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

set up continuous integration #58

Closed
gwenchee opened this issue Oct 2, 2018 · 7 comments
Closed

set up continuous integration #58

gwenchee opened this issue Oct 2, 2018 · 7 comments

Comments

@gwenchee
Copy link
Collaborator

gwenchee commented Oct 2, 2018

The continuous integration should check if:

  1. python install.py cyclus:FlanFlanagan:time_series
  2. python install.py cycamore:FlanFlanagan:d3ploy_update
  3. python setup.py install builds d3ploy
  4. cyclus test_cycamore.xml
  5. check if the sqlite output has all the timeseries tables
@jbae11
Copy link
Collaborator

jbae11 commented Oct 2, 2018

this error occurrs when building through travis. Any ideas?

@FlanFlanagan
Copy link
Contributor

Yep. This is because the cyclus version is wrong. I wonder if I just need to update the time_series branch to fix this. Let me give it a shot.

@jbae11
Copy link
Collaborator

jbae11 commented Oct 11, 2018

I think at this point there's two ways to solving this:

  1. Fix the build issue in cyclus:FlanFlanagan:time_series and continue on
  2. Wait until the FlanFlanagan branches get merged into cyclus/cyclus:master and use that as a ci build point
    @FlanFlanagan @gwenchee thoughts?

@FlanFlanagan
Copy link
Contributor

Wait for the merge. Those PRs are ready and just need merging.

@jbae11
Copy link
Collaborator

jbae11 commented Oct 12, 2018

even after the PR merge, I am still getting that error here, but I can build it in my machine. Can you explain why this error is happening?

@gwenchee
Copy link
Collaborator Author

gwenchee commented Oct 15, 2018

This error could be because a pre 2.7 version of python is being used? Someone encountered the same error here: https://groups.google.com/forum/#!searchin/cyclus-dev/process$20hdf5_back_gen%7Csort:date/cyclus-dev/j3o7G7uyfOs/STXAjVxvBAAJ ... But i see you are using python 3.6.5

@jbae11 jbae11 mentioned this issue Oct 16, 2018
@jbae11
Copy link
Collaborator

jbae11 commented Oct 25, 2018

#58 was merged.

@jbae11 jbae11 closed this as completed Oct 25, 2018
jbae11 pushed a commit that referenced this issue Dec 6, 2018
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

No branches or pull requests

3 participants