Skip to content
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

Deprecate petab.sbml.globalize_parameters #381

Merged
merged 2 commits into from
Mar 13, 2020
Merged

Deprecate petab.sbml.globalize_parameters #381

merged 2 commits into from
Mar 13, 2020

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Mar 13, 2020

Function has nothing to do with PEtab's objectives, and is not usable
at the moment. Better functionality is available through
libsbml.SBMLLocalParameterConverter. Therefore, this should be removed.

@dweindl dweindl requested a review from yannikschaelte March 13, 2020 12:52
@codecov
Copy link

codecov bot commented Mar 13, 2020

Codecov Report

Merging #381 into develop will decrease coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #381      +/-   ##
===========================================
- Coverage    77.78%   77.74%   -0.04%     
===========================================
  Files           22       22              
  Lines         2066     2067       +1     
  Branches       475      475              
===========================================
  Hits          1607     1607              
- Misses         339      340       +1     
  Partials       120      120
Impacted Files Coverage Δ
petab/sbml.py 51.66% <0%> (-0.44%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7579689...a976568. Read the comment docs.

Copy link
Member

@yannikschaelte yannikschaelte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add note (e.g. in pr description) why this is deprecated now -- makes it easier to follow later.

@dweindl
Copy link
Member Author

dweindl commented Mar 13, 2020

please add note (e.g. in pr description) why this is deprecated now -- makes it easier to follow later.

"It should have never gotten in here"? :)

@yannikschaelte
Copy link
Member

please add note (e.g. in pr description) why this is deprecated now -- makes it easier to follow later.

"It should have never gotten in here"? :)

exactly :) now it's documented ;)

Function has nothing to do with PEtab's objectives, and is not usable
at the moment. Better functionality is available through
libsbml.SBMLLocalParameterConverter. Therefore, this should be removed.
@dweindl dweindl merged commit 3eebb99 into develop Mar 13, 2020
@dweindl dweindl deleted the dw_misc branch March 13, 2020 16:06
dweindl added a commit that referenced this pull request Mar 18, 2020
Release 0.1.5

Library:

* New create empty observable function (issue 386) (#387)
* Deprecate petab.sbml.globalize_parameters (#381)
* Fix computing log10 likelihood (#380)
* Documentation update and typehints for visualization  (#372)
* Ordered result of `petab.get_output_parameters`
* Fix missing argument to parameters.create_parameter_df

Documentation:
* Add overview of supported PEtab feature in toolboxes
* Add contribution guide
* Fix optional values in documentation (#378)
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.

2 participants