Skip to content

ec.source: one format, one parser and a record of tuning - #91

Open
edkerk wants to merge 1 commit into
developfrom
feat/ec-source-format
Open

edkerk wants to merge 1 commit into
developfrom
feat/ec-source-format

Conversation

@edkerk

@edkerk edkerk commented Nov 8, 2026

Copy link
Copy Markdown
Collaborator

parse_kcat_source and format_kcat_source read and write the provenance text of model.ec.source, <origin>[ (<detail>)][ | tuned=<method>[+<method>...] (prior=<value>)], for example brenda (wc=0, origin=2) | tuned=cmaes (prior=6.55), and KcatSource holds the parts of one. format_kcat_source(origin, detail=None, tuned=(), prior=None) replaces format_kcat_source(source, wildcard_level, origin) and writes the same string for the same non-empty source, wildcard level and origin. Every writer builds its text with the formatter, and the readers use the parsed origin, so a source with a detail or a tuning status is recognised.

  • set_kcat_for_reactions writes the origin manual (it wrote setKcatForReactions) and drops any tuning status.
  • sensitivity_tuning and cmaes_kcat_tuning keep the origin and detail of every kcat they change and add the method (sensitivity, cmaes) to its tuning status, with the kcat before the first tuning as prior. sensitivity_tuning no longer writes the old kcat and source into ec.notes.
  • assign_standard_kcat and remove_standard_kcat keep a standard row that has a tuning status.
  • The text of every other writer is unchanged.

Part of raven-gecko-parity#151.

parse_kcat_source and format_kcat_source read and write <origin>[ (<detail>)][ | tuned=<methods> (prior=<value>)]. The writers build their strings with the formatter and the readers use the parsed origin. set_kcat_for_reactions writes manual. sensitivity_tuning and cmaes_kcat_tuning keep the origin and detail and add the tuning method, with the kcat before the first tuning as prior, and sensitivity_tuning no longer writes the old source into ec.notes. A standard row with a tuning status is kept by assign_standard_kcat and remove_standard_kcat.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant