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

Progress bar for find_MAP() #8

Open
PrzeChoj opened this issue May 10, 2022 · 5 comments
Open

Progress bar for find_MAP() #8

PrzeChoj opened this issue May 10, 2022 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@PrzeChoj
Copy link
Owner

The user will run the M-H algorithm for a long time on his machine. The process has to be more informative and eye-pleasing than right now.

@PrzeChoj PrzeChoj self-assigned this May 10, 2022
@PrzeChoj PrzeChoj added the enhancement New feature or request label May 10, 2022
@PrzeChoj
Copy link
Owner Author

PrzeChoj commented May 10, 2022

The progress bar was added to Metropolis-Hastings.

However, I would love it to give out some information:

  • Estimate the time left
  • If the algorithm finds the new best permutation, print a different character

@PrzeChoj
Copy link
Owner Author

PrzeChoj commented Jun 2, 2022

The progress bar for the best_growth algorithm with max_iter=Inf is not supported, and I currently do not know how to design it. Any idea would be appreciated.

@PrzeChoj PrzeChoj added the good first issue Good for newcomers label Jun 2, 2022
@PrzeChoj PrzeChoj removed their assignment Aug 5, 2022
@PrzeChoj PrzeChoj removed the good first issue Good for newcomers label Aug 30, 2022
@PrzeChoj
Copy link
Owner Author

PrzeChoj commented Sep 8, 2022

When one calls find_MAP(return_probabilities = TRUE), calculating the probabilities is timely. I decided to add the second progress bar that starts to go after the first one ends to separate those processes. I'm unsure if I communicated it clearly in the documentation and if this could be done more intuitively.

@PrzeChoj PrzeChoj changed the title Progress bar for Metropolis-Hastings Progress bar for optimizers Jul 13, 2023
@PrzeChoj PrzeChoj changed the title Progress bar for optimizers Progress bar for find_MAP() Jul 13, 2023
@PrzeChoj
Copy link
Owner Author

There is a progress package in R, but:

  1. I think it does not support printing different characters when a new best is found
  2. works poorly for multithreading

@PrzeChoj PrzeChoj added the good first issue Good for newcomers label Jul 15, 2023
@PrzeChoj
Copy link
Owner Author

Packard CLI looks promissing…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant