Warnings need to be fixed
Closed this issue · 5 comments
The #148 PR is passing with the exception of travis killing it due to the log being too long.
If you download the raw log here, you can see that message at the bottom.
https://travis-ci.com/StanfordAHA/lassen/builds/129343573
The log is too long due to there being warnings in the code. These warnings are coming from peak.auto_assembler
This should help #150
Depends on a peak branch https://github.com/cdonovick/peak/tree/move-assembler-utils
Waiting for the travis build
It looks like metamapper is still triggering a warning
metamapper/metamapper/peakmapper.py:6: DeprecationWarning: auto_assembler.py is deprecated please move to assembler
I'm debugging the buildkite build, it looks to be an environment issue.
I have a branch on Metamapper called "warnings" that fixes these issues. Trying to fix the travis on that repo before I merge though.
Okay tests seem to be proceeding along in buildkite https://buildkite.com/stanford-aha/lassen/builds/333#b73aa5bb-7a8b-4ede-bf1a-828105b9b4ca, with the metamapper warning branch this build should be back to green soon
Closing