-
Notifications
You must be signed in to change notification settings - Fork 40
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
Enh/m av2 #455
Conversation
@tstavenga I see that the last change you added broke the tests, might be worth checking out why that is. I also see that this PR essentially contains two big changes.
I would propose the following
Let me know if I can be of any help. |
pycqed/analysis_v2/base_analysis.py
Outdated
@@ -288,6 +296,22 @@ def extract_data(self): | |||
self.raw_data_dict = new_dict | |||
self.raw_data_dict['timestamp'] = self.timestamps[0] | |||
self.raw_data_dict['timestamps'] = self.timestamps | |||
|
|||
def extract_data_json(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this block seems to be misplaced?
pycqed/analysis_v2/base_analysis.py
Outdated
@@ -461,38 +488,31 @@ def save_fit_results(self): | |||
""" | |||
Saves the fit results | |||
""" | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a bugfix and needs to be merged over
use numpy array for transpose
This is not working. Created a new branch #458 to realize this request. |
Please use the following template for a pull request.
Fixes #issuenumber.
Push my changes to the new analysis
It is not finished but i think it is good to have a common starting point
Changes proposed in this pull request:
@mention AdriaanRol
In order for the pull request to be merged, the following conditions must be met:
Whenever possible the pull request should
Tests are not mandatory as this is generally hard to make for instruments that interact with hardware.