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

Separate post-run analysis #24

Open
anufrievroman opened this issue Feb 22, 2024 · 2 comments
Open

Separate post-run analysis #24

anufrievroman opened this issue Feb 22, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@anufrievroman
Copy link
Owner

In the future, I plan to separate the running of the simulation from the analysis so that the analysis could be performed separately with many different parameters. This will free the config files from unnecessary options, and will allow creating pretty plots without rerunning the entire simulations. Example of things that could be redone after the simulation:

  • Re-plotting maps without phonons in hole, or with white patches of holes on top.
  • Re-plotting distributions with desired ranges and bins.
  • Re-plotting with different color maps and other cosmetics
  • Changing in which time segments thermal conductivity is calculated

Most of this can be achieved just by user re-plotting things from raw data. I am not sure what would be the best way to do it? Create a separate command, or just provide a bunch of scripts for analysis, or keep all the settings in the config and allow rerunning freepaths without rerunning the simulation, like with --idle flag.

@anufrievroman anufrievroman added the enhancement New feature or request label Feb 22, 2024
@phigas
Copy link
Contributor

phigas commented Feb 25, 2024

One challenge with this are the timeframes. The information is put into the timeframes at runtime and so it would be difficult to chantge the timeframes after the run. I propose to use a high number of timeframes during the run so that they can be summed together into different lenghts at the plotting stage. The TC calculation would then also fall into the plotting stage.

@anufrievroman
Copy link
Owner Author

Yes, that's precisely my current vision. Let's say we always have NUMBER_OF_TIMEFRAMES, which is supposed to be about 10 or so. And all profiles for each timeframe are saved. Then, we can do all:

  • Show the convergence
  • Output thermal conductivity calculated over last N timeframes, plus some uncertainty
  • Recalculate the thermal conductivity afterward for different N if needed.

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

No branches or pull requests

2 participants