Comments (9)
@volodymyrss should be fixed in the last upload to pypi. However, it might be best to clone or install directly from master if there are any other bugs at the moment for faster turnaround. Thanks!
from ronswanson.
Uff. I thought I had fixed that. Will correct.
from ronswanson.
Thanks it works now. I wonder how it works in relation to openjournals/joss-reviews#4969 , it has to be somehow well-defined version which is being reviewed. As it's iterative, it would be the master branch. But some commands in the doc would not work until the version is uploaded. Maybe this is not imporatant. It's more of a question to @dfm.
from ronswanson.
@volodymyrss the best approach is to review the development version of the software and then we always tag a new release at the end of the review to archive the reviewed version.
from ronswanson.
@volodymyrss the best approach is to review the development version of the software and then we always tag a new release at the end of the review to archive the reviewed version.
Ok, I see, the comments/issues can be on master.
But I suppose the approval should be on a fixed revision, right? Else there can be a change on the master branch which is not approved.
How do I track what I am approving?
In principle, this is the case not only for final approval but for every qualification.
E.g. if I check "[ ] Installation: Does installation proceed as outlined in the documentation?", how do I indicate that it this particular version which proceeds, but not the other one?
edit: it's like in the PR: a new commit can be made to break the approval.
from ronswanson.
@volodymyrss Good question! We typically are not strict about that for JOSS reviews - you're definitely not expected to go through the full checklist a second time when there is a final version minted. I'd say that this is partly because of logistics (the scope is too broad, and it's hard to synchronize the two reviews, for example), but also partly because the point of a JOSS review is more to provide constructive feedback than to bring a specific judgement. From my perspective, it's totally sufficient to make sure that at the time you're going through it, the development branch satisfies the criteria in the checklist. That being said, if there are elements that are brittle or give you other reasons to be concerned, it's good to note that, and when I'm reviewing I do sometimes quickly go through the checklist one last time before signing off, but that's certainly not required! Please let me know if you have further questions - I'm happy to continue this conversation!
from ronswanson.
@volodymyrss Good question! We typically are not strict about that for JOSS reviews - you're definitely not expected to go through the full checklist a second time when there is a final version minted. I'd say that this is partly because of logistics (the scope is too broad, and it's hard to synchronize the two reviews, for example), but also partly because the point of a JOSS review is more to provide constructive feedback than to bring a specific judgement. From my perspective, it's totally sufficient to make sure that at the time you're going through it, the development branch satisfies the criteria in the checklist.
I see, thanks.
That being said, if there are elements that are brittle or give you other reasons to be concerned, it's good to note that, and when I'm reviewing I do sometimes quickly go through the checklist one last time before signing off, but that's certainly not required! Please let me know if you have further questions - I'm happy to continue this conversation!
Installation is quite likely to break in the future, especially if the requirements are not frozen. I suppose this is normal and not a fault of the software.
Evolving requirements may even change the behavior, or at least performance.
If we are concerned here with making the review reproducible and verifiable, I'd attach some form of environment snapshot, with all fixed versions, in a container.
But at minimum, I could point out the revision I am looking at when making a statement.
Maybe your bot could make use of this information in the future.
from ronswanson.
Great - totally agreed! And thanks for the suggestion - I'll bring this point up with the other JOSS editors to see how we'd like to handle these versioning issues that you've brought up. Thanks again!!
from ronswanson.
Thanks to you! This is interesting.
from ronswanson.
Related Issues (9)
- Pip Installation Error -- can't find xsFortran.h HOT 3
- Energy Grid in ronswanson HOT 1
- Do the authors describe how this software compares to other commonly-used packages? HOT 7
- Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support HOT 1
- !simulation_build sim_build.yml Traceback (most recent call last): File "/opt/conda/bin/simulation_build", line 5, in <module> from scripts.simulation_build import simulation_build ModuleNotFoundError: No module named 'scripts' HOT 4
- Broadening the scope of ronswanson: building table models for other software? HOT 4
- Docs: separate the intro and make a clearer case for ronswanson HOT 4
- Improving the example in the documentation - eventually adding more HOT 5
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 ronswanson.