Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

Run with new strax #198

Merged
merged 11 commits into from
Aug 27, 2021
Merged

Run with new strax #198

merged 11 commits into from
Aug 27, 2021

Conversation

JoranAngevaare
Copy link
Contributor

What is the problem / what does the code in this PR do

@JoranAngevaare
Copy link
Contributor Author

🤔 this doesn't look very nice, the testing time increases dramatically with this new strax(en):

Previous:
afbeelding

Current:
afbeelding

@WenzDaniel any ideas?

@WenzDaniel
Copy link

What is the difference for py3.9? For most of the tests the processing time doubled while in this case it increased by a factor of 6ish. Maybe that gives us a hint.

@zhut19
Copy link
Contributor

zhut19 commented Aug 27, 2021

I think most of the time is spent on calling cmt, as the cmt options are the default now.

@zhut19 zhut19 merged commit fc85b95 into master Aug 27, 2021
@zhut19 zhut19 deleted the run_with_new_strax branch November 9, 2021 14:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants