-
Notifications
You must be signed in to change notification settings - Fork 127
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
Cleanup dataframes #1360
Merged
nkanazawa1989
merged 31 commits into
qiskit-community:main
from
nkanazawa1989:cleanup/more_composition
Feb 6, 2024
Merged
Cleanup dataframes #1360
Changes from 9 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
78a967a
Update internals of AnalysisResultTable
nkanazawa1989 7a83179
Update internals of ScatterTable
nkanazawa1989 3bc9525
Removed unused mixin
nkanazawa1989 68013a8
Fix index mismatch issue after JSON serialization
nkanazawa1989 5032e86
Add more tests
nkanazawa1989 7736f19
Bug fixes
nkanazawa1989 8bbaa15
Merge branch 'main' of github.com:Qiskit/qiskit-experiments into clea…
nkanazawa1989 fc9273e
Unpin pandas 2.2
nkanazawa1989 0cae116
Update old pattern
nkanazawa1989 2fb28dc
Fix cross-reference
nkanazawa1989 ac972fd
Update curve analysis tutorial
nkanazawa1989 01471bb
Add shortcut methods
nkanazawa1989 8dc6c4f
Bugfix autosave
nkanazawa1989 144127a
Raise user warning when numbers contain multiple series
nkanazawa1989 a81f97c
Merge branch 'main' into cleanup/more_composition
nkanazawa1989 7c0662c
Bugfix: Missing circuit metadata in composite analysis
nkanazawa1989 92cfc92
Replace class_id with data_uid
nkanazawa1989 346d23a
Add documentation for filtering triplet
nkanazawa1989 ee03161
Apply review comments
nkanazawa1989 ee5b34d
Wording suggestions
nkanazawa1989 38abdff
Remove DEFAULT_
nkanazawa1989 9e27f16
Reorganize the doc
nkanazawa1989 b870be3
Remove _data
nkanazawa1989 cc905c6
Remove key from add_data
nkanazawa1989 0dc4eb2
Remove type cast depending on the entry number
nkanazawa1989 f8c1efe
Minor docs formatting
nkanazawa1989 ee92f1d
Add more tests for result table
nkanazawa1989 03aac67
Performance optimization
nkanazawa1989 ac5bdd8
name, data_uid -> series_name, series_id
nkanazawa1989 58671eb
Add more tests for construction
nkanazawa1989 7ff2c6a
Update Ramsey analysis
nkanazawa1989 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -39,6 +39,7 @@ | |
.. autosummary:: | ||
:toctree: ../stubs/ | ||
|
||
ScatterTable | ||
SeriesDef | ||
CurveData | ||
CurveFitResult | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
It still surprises me that it is better to iterate over numpy arrays point by point and add them to them to lists to add to a new dataframe rather than just adding the numpy arrays to a new dataframe.
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.
Handling of the empty column is expensive because it requires careful handling of missing values. Without doing this shots column may be accidentally typecasted to float because numpy doesn't support nullable integer. This means we first need to create a 2D object-dtype ndarray and populate values, then convert it into dataframe. Since current
_lazy_add_rows
buffer assumes row-wise data list, arrays needs to be converted into this form internally.