• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

LoLab-VU / pysb / 552
79%

Build:
DEFAULT BRANCH: master
Ran 01 May 2017 09:08PM UTC
Jobs 2
Files 86
Run time 55s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
552

push

travis-ci

jmuhlich
Fix PysbFlatExporter with compartmental models (#254)

* Fix PysbFlatExporter with compartmental models
* Fix `is_equivalent_to` for compartmental models

## Original commit messages

* Fix PysbFlatExporter with compartmental models

The PysbFlatExporter fails with compartmental models due to `Compartment`'s
`repr()` implementation, which means parameters and parent compartments
get printed in full rather than just their name, which leads to a
`ComponentDuplicateNameError` with compartmental models when using the
PysbFlatExporter. This commit fixes the issue.

Resolves: pysb/pysb#210

* Fix `is_equivalent_to` for compartmental models

Currently, the `is_equivalent_to` method does not consider species
with a defined Monomer compartment, species compartment or both to
be equivalent, i.e. the following are all different:

 * A() ** C1 # monomer compartment
 * ComplexPattern([A()], compartment=C1) # species compartment
 * ComplexPattern([A() ** C1, compartment=C1) # species and monomer
                                              # compartments

These should all be equivalent, which is fixed in this patch.

I've also added a check in the `is_equivalent_to` method to check
that the ComplexPatterns are concrete, i.e. species, and not
abstract patterns, as the matching rules are subtly different.

Resolves: pysb/pysb#257

* Add error to prevent compartment override with **

In the following scenario with two compartments C1 and C2:

    In [95]: (A() ** C1) ** C2
    Out[95]: A() ** C2

This syntax should throw an error, rather than override the existing
compartment.

See also: pysb/pysb#258

* Deprecate is_equivalent_to with non-concrete pats

* Fix synthesis initials with compartmental models

One initial condition based on __source should be generated for each
compartment. A unit test has been added to check for this.

5196 of 7026 relevant lines covered (73.95%)

1.31 hits per line

Jobs
ID Job ID Ran Files Coverage
1 552.1 01 May 2017 09:09PM UTC 0
71.76
Travis Job 552.1
2 552.2 01 May 2017 09:08PM UTC 0
71.5
Travis Job 552.2
Source Files on build 552
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #552
  • 43ecfa60 on github
  • Prev Build on master (#551)
  • Next Build on master (#553)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc