Comments (15)
A few of them appear to be from the CHIANTI dbase zipfile not being transferred so no big deal there.
The last two are failures result from the changed abundanceName keyword from #66 . I'll update those testsradLoss
.
Actually its that the continuum
class is used in radLoss
with the abundanceName
keyword. Yay for tests!
from chiantipy.
Travis CI creates a build for each PR. I would recommend waiting to merge each PR until all tests have passed. That way any bugs can be fixed before they are merged into the main codebase.
from chiantipy.
You are right about that. I have been a little too quick to just do the PR
from chiantipy.
This can be closed. Fixed by #68
from chiantipy.
one final comment. The CHIANTI database is also available for download file-by-file. Perhaps this would be better than downloading the whole tar file. SSWIDL get these files from:
from chiantipy.
the previous comnet was suposed to include
http://chiantidatabase.org/sswidl/dbase
from chiantipy.
Maybe so. On RTD and Travis I had just downloaded the whole database because I wasn't sure which files were being read at import.
I think the reason the download phase in the build might be a bottleneck is because four workers are all trying to download the tar file at the same time. Is it possible that this kind of traffic is overwhelming the CHIANTI server? I'm assuming its not used to sudden bursts of traffic like that.
This can be solved by just limiting the number of concurrent jobs we run on Travis.
from chiantipy.
Some times the server keeps up with the traffic and sometimes not. This weekend was OK.
I think that tools/util/data is what reads the necessary data. These include
the Ionization potential file, the ionization equlibrium files,the abundance files, and the masterlist
if things get too difficult, we could try limiting the build to these. However, I have learned how to restart the various steps in the build, one-by-one and this gets by that problem.
Also, I send you an invitation to join the team. I was not too clear on what that meant but what is was wondering, is whether you @wtbarnes wanted to be able to merge PR requests on your own?
from chiantipy.
Yes I'd be happy to help merge in some PRs. I think if you grant me permission to merge PRs, this should also let me restart RTD and Travis builds as well when failures come up. Hopefully this takes some of the burden off of you!
I'm happy to merge PRs (either from myself or from others), but I think its important to still have your approval on these (in the form of a 👍 or +1 in the comments to the PR ) before they get merged into the main codebase. That way there is agreement on changes to the codebase before it is merged. The SunPy folks have outlined a procedure for reviewing PRs. It might not be a bad idea to use a system similar to this one.
from chiantipy.
It seems you are now a member. It seems like I need to make you an owner to do PRs. That is fine with me. The point of getting my approval sounds good. We need to find some way to make sure that we are not working on the same pieces of SW. Right now it is just in the form of comments here an there. The SunPy procedure is a bit much for really a 2 active member team.
from chiantipy.
I believe you can create groups in order to customize permissions ? I'm not overly familiar with doing this on GitHub, but I think by creating a group (e.g. "Maintainers" or something), you can grant PR access without having full ownership over the repository?
Yes I agree the SunPy procedure is a bit over the top for this repo at the moment. Perhaps the minimum requirements for merging should be:
- A +1 or 👍 from you (@kdere)
- All tests passing on Travis CI
from chiantipy.
I think the best way to ensure we aren't working on the same pieces of code is to open a PR every time you begin working on a feature. As you make commits to your fork while you work on the feature, these will show up in the PR.
When someone is ready for the PR to be merged, they can tag you with a review request (e.g. @kdere please review) and if you're happy with it and the tests have passed, then merge. That way, all work is done out in the open and it is obvious to everyone what features/pieces of code are being worked on.
from chiantipy.
from chiantipy.
I will look into making a group, maybe maintainers.
from chiantipy.
This issue can also be closed. Looks like the original problem was resolved a while back.
from chiantipy.
Related Issues (20)
- KeyError on windows 10 HOT 5
- Error on running provided jupyter notebooks in the repo HOT 3
- line identifications in spectrum plot HOT 13
- Ability to give custom abundance files HOT 8
- Error while using spectrum with elementList only when all the ions are not present in the masterlist HOT 7
- Unexpected behavior when setting minAbund lower than lowest abund HOT 3
- Error when giving "Zn" in elementlist HOT 2
- How much memory does the mspectrum process consumes? HOT 1
- Chianti is taking more threads than specified HOT 2
- Clarify emission measure units in the function documentations. HOT 2
- Question: Support for photons/keV completed? HOT 10
- Getting spectrum from Ch.spectrum without smoothing/filtering HOT 4
- IDL and ChiantiPy gives different results. HOT 4
- Possible typo in function's documentation HOT 1
- NaNs generated when calculating spectra under certain conditions HOT 9
- Upload 0.15.1 to PyPI HOT 3
- Non-ionization equilibrium? HOT 2
- Unable to load ioneq HOT 10
- Two-photon calculation? HOT 1
- Problem with mspectrum? HOT 9
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from chiantipy.