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

Montecarlo Packet Progress bar does not go to 100% #2216

Closed
2 tasks done
Rodot- opened this issue Feb 21, 2023 · 6 comments
Closed
2 tasks done

Montecarlo Packet Progress bar does not go to 100% #2216

Rodot- opened this issue Feb 21, 2023 · 6 comments

Comments

@Rodot-
Copy link
Contributor

Rodot- commented Feb 21, 2023

Describe the bug
When running the montecarlo main loop, the progress bar updates with an approximation for the number of packets completed when run in multithreaded mode. At the end of the loop, the progress bar is no longer updated so the display never shows it reaching 100%.

To solve this problem, at the end of the loop another update call to the progress bar should be made to fill out the remaining progress.

To Reproduce
Run a tardis model with >=2 threads

System

  • OS:

    • GNU/Linux
    • macOS
  • Environment (conda list):

Additional context

@AyushiDaksh
Copy link
Contributor

@Rodot- Are you still working on this? If not, can I pick it up?

@Rodot-
Copy link
Contributor Author

Rodot- commented Feb 28, 2023

I'm not working on this myself so it is definitely up for grabs

@shreyas3156
Copy link
Member

@Rodot- There seems to be a delay of a few seconds (even after the particular cell in the notebook has finished running), after which the packet progress bar does update to 100%. I could reproduce this for 1 <= no. of threads <= 4.

@shreyas3156
Copy link
Member

Hi @Rodot- I have made a PR that aims to solve this issue. It would be great if you could review it.

@atharva-2001
Copy link
Member

@Rodot- Does PR #2237 fix this issue?

@MarkMageeAstro
Copy link
Contributor

Seems to be fixed now

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

No branches or pull requests

6 participants