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

aiidateam / aiida_core / 4885
70%
develop: 74%

Build:
Build:
LAST BUILD BRANCH: fix_3032_doc_working_importexport
DEFAULT BRANCH: develop
Ran 10 Jan 2019 07:22AM UTC
Jobs 4
Files 412
Run time 28min
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
4885

push

travis-ci

web-flow
Simplify loading of node class from type string (#2376)

The type string of a `CalcJobNode` should now be just that of the
node class and not the sub class. Since the users are currently still
sub classing the node class and not a process class, we have to make
a special exception when generating the type string for the node.

Conversely, this type string, stored in the `type` column of the node,
should be used to load the `CalcJobNode` class when loading from
the database. The only classes that can legally exist in a database, and
therefore be loaded, are defined in `aiida-core`. Therefore, using the
entry point system to map the type string onto an actual ORM class is no
longer necessary. We rename the `aiida.plugins.loader.load_plugin`
function to the more correct `load_node_class`, which given a type string,
will return the corresponding ORM node sub class.

Note that the whole machinery around generating type and query strings
and loading the nodes based on them is still somewhat convoluted and
contains hacks for two reasons:

 1) Data is not yet moved within the `aiida.orm.node` sub module and as
    a result gets the `data.Data.` type string, which will not match the
    `node.Node.` type when sub classing in queries.

 2) CalcJobProcesses are defined by sub classing JobCalculation
    Until the user directly define a Process sub class that uses the
    `CalcJobNode` as its node class, exceptions will have to be made.

If these two issues are addressed, a lot of the code around type strings
can be simplified and cleaned up.

78 of 78 new or added lines in 12 files covered. (100.0%)

25220 of 36553 relevant lines covered (69.0%)

2.46 hits per line

Jobs
ID Job ID Ran Files Coverage
5 4885.5 (TEST_AIIDA_BACKEND=django TEST_TYPE="tests") 10 Jan 2019 07:22AM UTC 0
61.96
Travis Job 4885.5
6 4885.6 (TEST_AIIDA_BACKEND=django TEST_TYPE="tests") 10 Jan 2019 07:28AM UTC 0
61.9
Travis Job 4885.6
7 4885.7 (TEST_AIIDA_BACKEND=sqlalchemy TEST_TYPE="tests") 10 Jan 2019 07:50AM UTC 0
61.01
Travis Job 4885.7
8 4885.8 (TEST_AIIDA_BACKEND=sqlalchemy TEST_TYPE="tests") 10 Jan 2019 07:31AM UTC 0
60.96
Travis Job 4885.8
Source Files on build 4885
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #4885
  • 1e339c33 on github
  • Prev Build on provenance_redesign (#4883)
  • Next Build on provenance_redesign (#4890)
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

© 2025 Coveralls, Inc