Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test displaying the resume text #48(new) #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aaryamantriescode
Copy link

Also removed line strip

@aaryamantriescode
Copy link
Author

@nicobrenner hey, I think everything should be fine now !! I Thought it would be easier to just start a fresh repo😂

@nicobrenner
Copy link
Owner

Thank you @aaryamantriescode please check out comments on the PR, also always make sure the tests run before submitting the code

Looking forward to merging this PR soon

@aaryamantriescode
Copy link
Author

Yeah, ok ill do the tests. I don't have docker installed, I will have to set it up.

@nicobrenner
Copy link
Owner

nicobrenner commented Mar 19, 2024 via email

@aaryamantriescode
Copy link
Author

aaryamantriescode commented Mar 19, 2024

Yeah, I should probably try that because I am not getting how to do it in terminal I put the api key and followed the instructions but something is wrong
image

@nicobrenner
Copy link
Owner

Yeah, I should probably try that because I am not getting how to do it in terminal I put the api key and followed the instructions but something is wrong image

Oh, interesting, hadn't seen that

Looking around, it seems like it's an issue with the environment not setting a TERM variable: https://stackoverflow.com/questions/66834732/python-curses-doesnt-recognize-terminal

Given that the issue is happening inside of docker, make sure that when you run docker, that the docker-compose.yml has these two lines:

    tty: true  # Allocate a pseudo-TTY
    stdin_open: true  # Keep STDIN open

So that when the app runs, it has access to a terminal. If that doesn't work, maybe try setting the terminal-related environment variables inside docker-entrypoint.sh (if your docker-compose file includes it), or in the docker-compose.yml file (inside the environment section)

@aaryamantriescode
Copy link
Author

aaryamantriescode commented Mar 19, 2024

Yeah, I should probably try that because I am not getting how to do it in terminal I put the api key and followed the instructions but something is wrong image

Oh, interesting, hadn't seen that

Looking around, it seems like it's an issue with the environment not setting a TERM variable: https://stackoverflow.com/questions/66834732/python-curses-doesnt-recognize-terminal

Given that the issue is happening inside of docker, make sure that when you run docker, that the docker-compose.yml has these two lines:

    tty: true  # Allocate a pseudo-TTY
    stdin_open: true  # Keep STDIN open

So that when the app runs, it has access to a terminal. If that doesn't work, maybe try setting the terminal-related environment variables inside docker-entrypoint.sh (if your docker-compose file includes it), or in the docker-compose.yml file (inside the environment section)

Yeah, I tried all that I don't know what to do now, even python isn't working. Can you check if my code works on your pc?
image

@nicobrenner
Copy link
Owner

About the error you are getting, this is what I get in my terminal:

echo $TERM
xterm-256color

This is what chatgpt says (try doing export TERM=xterm before running docker):
image

When running the code in the PR, I don't get the TERM error, but I do get an error having to do with the test code

@aaryamantriescode
Copy link
Author

aaryamantriescode commented Mar 19, 2024

I'll try it in the evening now I have to prep for entrance exams😅I will try to get it working by tomorrow! Thank you for being helpful!

@nicobrenner
Copy link
Owner

Good luck, hope it works!

@aaryamantriescode
Copy link
Author

aaryamantriescode commented Mar 20, 2024

image
image
I got the venv file to work now but python says I don't have the curses module, but I have it installed 🤦‍♂️its been 2 hours just trying to run this 🥲 i even tried the "export TERM=xterm" before running docker

@nicobrenner
Copy link
Owner

nicobrenner commented Mar 20, 2024

Sorry, haven't tried running it in windows. You are being a pioneer. Thank you for sharing your experience here, it's super valuable to learn how to make it work in more platforms

The suggestions I see online are to do either pip install windows-curses which looks like you already did, or bpython-curses (pmbarrett314/curses-menu#18 (comment)), I haven't tried it though

@aaryamantriescode
Copy link
Author

Sorry, haven't tried running it in windows. You are being a pioneer. Thank you for sharing your experience here, it's super valuable to learn how to make it work in more platforms

The suggestions I see online are to do either pip install windows-curses which looks like you already did, or bpython-curses (pmbarrett314/curses-menu#18 (comment)), I haven't tried it though

Yeah, 😅 I'm planning to shift to Linux after my exams cause windows is wayy too slow :( . I'll try to look more into it by tomorrow, hopefully should find a way to run it !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants