Comments (2)
This was fixed with the first two file changes in 3c90597 . It is not stripping private_location field out of the grid.json (it is obfuscating the data), but this is actually good because we need that information in the front end. What we need on the front end is some indicator of wether or not an observation is to be represented by a circle or flag marker. Since the tileserver has to check both geoprivacy and presence of private coordinates to make that determination, the front-end code needs the same information to make the exact same determination.
To be clear - we're either returning private_location: "" to represent no private location, or private_location: "[object Object]" to represent the presence of a private location. We never send any actual private location data other than presence/absence.
We could have some step in-between fetching the data from ES and rendering the .grid.json to create some kind of "is_obscured" boolean (alternatively we could add that to the ES index itself), but currently elasticmaps isn't set up to have any logic in between fetch results and rendering. So there are various things we can do, though I do worry a little about slowing down the tileserver with more logic. If we really wanted to change it, I'd vote for adding a field to ES which would make the logic clearer for both the tileserver styles and the UTFGrid interaction offsets.
from inaturalistapi.
I'm fine with the way we have it. Not ideal, but workable, and not revealing anything we shouldn't.
from inaturalistapi.
Related Issues (20)
- Trad project user species count not showing the correct number HOT 1
- Allow exact string search queries (for things like taxa and projects) HOT 2
- Allow restricting taxon search to only valid names HOT 1
- Add PUT observations/:id/viewed_updates to API v2
- v2 fetch endpoints with a single UUID should 404 if there's no record
- how does paging work in taxa and taxa/autocomplete endpoints? HOT 1
- Document order and order_by params of /v1/taxa endpoint
- Support DELETE observation in v2
- Support updating observation photos and sounds in API v2
- Add application field to observations/search HOT 1
- Support GET observations/deleted in API v2 HOT 1
- Allow authenticated log requests from anywhere
- admin_level documentation is outdated
- Add a rank_level filter param to the fetch taxa endpoint
- Allow users to search by ORCID HOT 2
- Taxa `rank_level` returns floating point HOT 3
- API endpoint to reset a password
- [v2] GET observations returns 500 with search_phase_execution_exception HOT 1
- Additional fields for User API HOT 1
- Fix some recurring errors in logs
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 inaturalistapi.