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

Name correction #41

Merged
merged 1 commit into from
May 4, 2020
Merged

Name correction #41

merged 1 commit into from
May 4, 2020

Conversation

Wealing
Copy link
Contributor

@Wealing Wealing commented May 1, 2020

No description provided.

@codecov
Copy link

codecov bot commented May 1, 2020

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   70.92%   70.92%           
=======================================
  Files          46       46           
  Lines        3401     3401           
=======================================
  Hits         2412     2412           
  Misses        989      989           

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 58a0b86...a34a619. Read the comment docs.

@psalvy psalvy merged commit f1467b2 into EPFL-LCSB:master May 4, 2020
psalvy added a commit that referenced this pull request Mar 4, 2021
Bypassing Travis because failing on GLPK not sympy anymore. See PR #49 

* Dev: Incremental update (#29)

* MNT: Pypi details

* MNT: Adding veriable creation logs

* FIX: typo

* FIX: Issue #28, incorrect variable referencing

* TST: adding tests for some of the analysis functions

* FIX: more fixes from #28

* FIX: fixed transport reaction function

* TST: Better tests, with a smaller model for I/O

* TST/FIX: relative<- absolute paths for small_model resources

* FIX: typo in paths to small_model resources for tests

* VER: Bump v0.9.0-b2

* VER: bump to v0.9.1

* Update LICENSE.txt

* Fixing a couple of minor bugs and added new variable class (#35)

Thank you @remidhum 

* FIX: fixed the apply_directionality function

solution.raw dataframe index does not contain the fwd and bwd use variables.

* ENH: added new binary variable class

This class is usefull to deal with inactive reactions in a model.

* ENH: deal with models without an objective function

Objective function is set to Zero (symbol("0") does not work!) if there is no defined objective function

* OOPS: change proper function ...

* FIX: fixed the failing test

As suggested, I added a check to determine what object type is passed.

* MNT: solution object type testing improved

* LIC: Create CLAI

* Name correction (#41)

Thank you for this correction :) Since I copied this from Google scholar, maybe check that they got your name right!
Cheers,
Pierre

* Allow saving of imported mat files as json (#42)

* Allow saving of imported mat files as json

When importing .mat models, the membranePot type is numpy's int16. The change makes it a standard int so that it can be saved as json.

* convert to float

* VER: Bump to 0.9.2

* MNT: removing warnings from deprecated usage of logger.warn

* FIX: metabolite thermo data was not recovered upon serialization

* FIX: Equilibrator update + reqs for building Dockerfile

* FIX: newer pypi version

* FIX: actual newer pypi version

* Update README.rst

* Update utils.py

* Zero import

Co-authored-by: realLCSB <31034133+realLCSB@users.noreply.github.com>
Co-authored-by: Pierre Salvy <psalvy@users.noreply.github.com>
Co-authored-by: Pierre Salvy <pierre.salvy@epfl.ch>
Co-authored-by: RémiDhum <rdhumeaux@gmail.com>
Co-authored-by: Daniel F Hernandez <dan.fhg@gmail.com>
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