Skip to content

Tuning Correction Improvements #277

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

Open
4 of 8 tasks
AdityaSavara opened this issue Nov 10, 2021 · 0 comments
Open
4 of 8 tasks

Tuning Correction Improvements #277

AdityaSavara opened this issue Nov 10, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@AdityaSavara
Copy link
Owner

AdityaSavara commented Nov 10, 2021

  • createMixedReferencePatterns should be set to false if no standard or existing (external) file is provided.
  • exporting improvements as noted here: https://github.com/AdityaSavara/MSRESOLVESG/issues/267
  • if UserChoices['measuredReferenceYorN']['on'] ='no' then the UserChoices['measuredReferenceYorN']['referenceFileStandardTuning'] = ['GasMixtureNISTRef.csv'] needs to be set to blank during user validity. Same thing for the existing pattern.
  • The variables referenceFileStandardTuning and referenceFileExistingTuning in userinput need to be changed to say "andForm" at the end of the variable name. Same for referenceFileDesiredTuning
  • With the gas mixture tuning corrector, There is lots of printing of "moleculechooser" that is probably not intended. An optional argument or temporary turning off of verbose should be implemented for the relevant line(s).
  • After making a mixed reference pattern from tuning corrector, need to remove molecules again if chosen molecules is on and they are not in chosen molecules. Undesired behavior was observed in 211105BenDataAnalysisGasMixtureOneExperiment\7
  • In workshop directory 8 https://github.com/AdityaSavara/MSRESOLVE_Workshop, there is a problem where the resolveConcentrations are not getting plotted. (maybe not calculated at all? or maybe calculated wrong? May need to print out matching_abscissa and other things to trace this issue.)
  • IN workshop directory 8, there is a warning printed out about uncertainties not being completely programmed for tuning corrector.
@AdityaSavara AdityaSavara added the enhancement New feature or request label Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant