Having less than 2 resumes causes an error: "IndexError: list index out of range"

[mohamedaleya]

Issue Title

Having less than 2 resumes causes an error: "IndexError: list index out of range"

Type

• [ ] Big
• [ ] Feature Request
• [ ] Info
• [x] Bug
• [ ] Documentation
• [ ] Other (please specify):

Description

If you have 2 resumes or less, you will receive an error after launching the app with the command streamlit run streamlit_app.py (or streamlit run streamlit_second.py). The slider defaults to the max index of the list. Resulting in an attempt to render a non-existent item. Also if you have only 1 resume, you shouldn't have a slider. And if you have no resume, some hint should be displayed to remind the user to add a resume.

This issue should help improve the UX for the time being until the new web app version is available to users.

Expected Behavior

• If the user has only one resume, remove the slider and display the text "There is 1 resume present.", not "There are X resumes." Then make that resume become the default selection.
• If the user has no resumes, display the text: "No resumes are present. Please add a resume, then refresh the app."
• If the user has 2 resumes, there should be two options on the slider, not 3 like now.
• If the user has more than 2 resumes present, each one will be represented by a slider breakpoint.

There should be no errors, no matter how many resumes the user has. The four points above make sure of that.

Current Behavior

If I have 1 resume, the slider will have 0, 1 and 2 indexes. 1 and 2 do not work. Therefore I need to set the slider to the index 0 to see relevant results.

Steps to Reproduce

1. Add no more than 2 resumes.
2. Use the cmd streamlit run streamlit_app.py.
3. The app will open to an error (in the resume section) as the slider's default selection is set to the maximum index (for this case, 2).

Screenshots / Code Snippets (if applicable)

Environment

• Operating System: Windows 11 Pro, x64
• Browser (if applicable): Brave, Version 1.57.49 Chromium: 116.0.5845.96 (Official Build) (64-bit)
• Version/Commit ID (if applicable):

Possible Solution (if you have any in mind)

Add conditions to display the appropriate content for no resumes, 1 resume, and 1 or more resumes available.

Additional Information

I've worked on a solution and will follow up on this issue with a PR.

[srbhr]

Hi @mohamedaleya, thanks for this PR. But Streamlit_second is for deploying to streamlit for demo purpose. You aren't required to run it.

[mohamedaleya]

Hi @mohamedaleya, thanks for this PR. But Streamlit_second is for deploying to streamlit for demo purpose. You aren't required to run it.

Sorry my bad. I have now updated the PR with changes applied to streamlit_app as well. Let me know if there's anything else I should rectify!

[srbhr]

This was fixed.