-
Notifications
You must be signed in to change notification settings - Fork 54
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
Update size of (continuous) benchmarks / Add benchmarks for new features #1661
base: main
Are you sure you want to change the base?
Conversation
@JuanPedroGHM do you think we should...
|
The first one would be quite easy, because you can choose a number of GB per process, resulting in a number of elements per process which then results in a total number of elements for the data set that then can result in a concrete shape (different for square, tall-skinny, very tall-skinny etc. data, but all with the same global and process-local amount of data) |
…om:helmholtz-analytics/heat into features/1615-Stable_meaningful_benchmarks
looks like that it will become difficult to have a uniform size of the benchmarks... |
It's a good idea to align all our benchmarks by using standardized sizes, easier to design and change everything in the future if we get again more powerful hardware. For the time being, I'm going to raise the time of the slurm job so that you can experiment better. |
Raised the time limit to 2 hours, but once you have more data fine tune it better. |
Due Diligence
Description
Issue/s resolved: #
Changes proposed:
Type of change
Memory requirements
Performance
Does this change modify the behaviour of other functions? If so, which?
yes / no