Change excel_ui functions to output OrderedDicts #334
Merged
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.
Change
excel_ui.process_beads_table()
andexcel_ui.process_samples_table()
to outputOrderedDict
data structures keyed on IDs from corresponding Beads and Samples tables. Also update related code and documentation accordingly.Closes #232.
All unit tests pass in
Python 3.8 + Anaconda 2020.02
andPython 2.7 + Anaconda 4.4.0
.>python -m FlowCal.excel_ui -v -p -H -i ./examples/experiment.xlsx
works without error inPython 3.8 + Anaconda 2020.02
andPython 2.7 + Anaconda 4.4.0
. Outputs (plots and Excel file) generated before and after the change appear the same inPython 3.8 + Anaconda 2020.02
.>python ./examples/analyze_excel_ui.py
works without error inPython 3.8 + Anaconda 2020.02
.