Comments (10)
@annakrystalli I was thinking about this whilst adding fixtures. It'd be nice to know at a glance which ones recycle info from say the client, and which ones don't.
from emodnetwfs.
I have been thinking about this again.
- keeping get for the method that just extract metadata
- having "download" in the name of https://emodnet.github.io/EMODnetWFS/reference/emodnet_get_layers.html
from emodnetwfs.
@annakrystalli @salvafern what's your take on this?
from emodnetwfs.
Related #53
from emodnetwfs.
Hi, I'm not aware of the best practices but seems like a valid point. Other packages use "download" instead of "get" indeed.
from emodnetwfs.
I was inspired by the Art of Readable Code where they wrote "Many Programmers are used to the convention that methods starting with get are 'lightweight accessors' that simply return an internal member."
But maybe we should follow ows4R lead to not confuse users 🤔
from emodnetwfs.
Well if you think it will improve the code readability I am all in!
from emodnetwfs.
Notes
- Even if creating the client beforehand,
emodnet_get_wfs_info()
needs an internet connection because ofguess_layer_format()
. So maybe its name should contain "fetch"? (or download, I thought of fetch because it's a shorter word)
from emodnetwfs.
emodnet_get_layer_info()
also performs a query.
from emodnetwfs.
emodnet_get_layers()
also performs a query.
from emodnetwfs.
Related Issues (20)
- new test failure
- pkgdown failure due to vignette HOT 4
- Get list of all OGC endpoints from catalog instead of hard-coded text file HOT 8
- Link to WFS endpoints in emodnetwfs.Rmd is broken HOT 2
- `emodnet_get_wfs_info(wfs_bio)` introduced twice HOT 1
- Question: Dump all data in one go? HOT 5
- add section about related packages to README HOT 1
- increase test coverage HOT 2
- investigate R CMD check error that pkgcheck is mentioning HOT 3
- All layers are of class data.frame? HOT 3
- explore wfs biology_new_data_products HOT 7
- why would one use rerddap HOT 2
- Could guess_layer_format() use ows4r's `$getGeometryType()`?
- Problem with a layer of the physics server HOT 6
- update example in request-params.Rmd HOT 5
- change lifecycle badge status once build green again
- could we rename the package to all lowercase? HOT 20
- Bug with a layer of biology_occurrence_data HOT 8
- Bug with a layer of chemistry_marine_litter HOT 7
- How to get the package listed from the FAQ
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 emodnetwfs.