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

Add verbose argument for PowerSpectrumStatistics.load() #16

Open
echaussidon opened this issue Nov 30, 2022 · 2 comments
Open

Add verbose argument for PowerSpectrumStatistics.load() #16

echaussidon opened this issue Nov 30, 2022 · 2 comments

Comments

@echaussidon
Copy link

In order to avoid multiple line when loading one powerspectrum in several process, I propose to add verbose argument here:

def load(cls, filename):

@adematti
Copy link
Member

adematti commented Dec 1, 2022

I'm a bit reluctant to add an option that would be redundant with the logger level.
Wouldn't the best option (also to avoid unneccessary IOs) to load power spectra from a single process, then do comm.bcast(power, root=0)?

@echaussidon
Copy link
Author

Agree to track the I/O accross all the processes.

However it is quite heavy when reading 1000 power spectra to compute the covariance Matrix for instance.

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

No branches or pull requests

2 participants