Comments (3)
Lewis, everything works perfectly well and all the documentations are clear! However, I have 2 sugestions:
-
Although it doesn't change the output, line 60 and 65, you are using twice the name "df" for two different things, which may be a bit confusing. I would suggest changing the first one (name of the sequential vector giving the latitudinal scale) to something like "scl" or "lat_scale". I'm going to pull a request anyway.
-
I totally understand your ambitions with the "fit" argument. The fact is that, if one user specifies a bin size that is not a factor of 180, the output will by default return an exact binning of that size, naturally ommiting extremal portions of the [-90, 90] latitudinal spectrum. There is no possibility to tune that (e.g. saying "I want the southern pole region to be included but not the northern pole one"). I'm wondering who would want to exclude some pre-defined regions like that, with the risk to loose samples that would fall there. I'm probably wrong but wouldn't it make more sense to just implement the "fit=TRUE" case as a conditionnal situation triggered by "size = x such as 180%%x != 0"?
Otherwise, if one user REALLY wants a binnig that is not a factor of 180, maybe could we implement the possibility for him/her to choose the zones of the map to exclude? Do you see what I mean?
But again, the function works very well!
from palaeoverse.
Thanks for your thoughtful comments Lucas! I will address them point by point.
- I do not think this is really an issue. In fact, it prevents too many additional variables being generated and taking up more memory than neccesary (though not really an issue for the size of objects here!).
- I see your point here, and I think it is pretty valid. However, I think how the function is implemented gives greater control to the user without any surprises. I don't think we should force a specific latitudinal bin size on the user as we do not really know their intentions. However, I agree that 99% of time, users will want to cover the entire latitudinal range. I will leave the function as is for now, but this something to get feedback on from users.
Thanks again!
from palaeoverse.
Hi Lewis,
-
Okay, I see the point. I think my mind is a bit OCD. Maybe could we leave a comment in the code saying that these 2 dfs are different? But it's just a suggestion, with your justification it seems fair enough to leave the code as it is.
-
Perfect! Let's see if the question is raised by the users then :)
All in all, you did something really neat!
from palaeoverse.
Related Issues (20)
- Rotation (MULLER2019) HOT 2
- Unnecessary columns generated in palaeorotate HOT 1
- Multi-model call bug in palaeorotate
- Atdabanian/Botomian in interval_key incorrect
- Summarise abundance in occurrence datasets
- Quantify ghost ranges HOT 1
- Make all links more accessible
- sp retirement HOT 3
- Multimodel binding issue in palaeorotate
- Allow custom time bins for tax_expand_time HOT 1
- Update CONTRIBUTING.md
- Range plot for stratigraphic sections
- Greater plot customisability in tax_range_time HOT 1
- dateline issues with bin_space plot HOT 12
- Midpoint age falling right on boundary HOT 1
- Age arguments for bin_time
- Convert occurrence ID certainty into binary vector
- Superfluous columns in palaeorotate
- GPlates API now returns original coordinates instead of NA when outside age scope HOT 5
- Equal area latitudinal bins
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 palaeoverse.