Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a script that exports user travel diaries as geojson #848

Merged
merged 1 commit into from
Mar 24, 2022

Conversation

shankari
Copy link
Contributor

This reuses the same method as the the code to generate the diary for a
particular day for display in the UI. The related

For an example of working with the data, please see
https://github.com/e-mission/e-mission-eval-private-data/blob/master/OpenPATH_geojson_example.ipynb

@PatGendre in case this helps

This reuses the same method as the the code to generate the diary for a
particular day for display in the UI. The related

For an example of working with the data, please see
https://github.com/e-mission/e-mission-eval-private-data/blob/master/OpenPATH_geojson_example.ipynb

@PatGendre in case this helps
@shankari shankari merged commit 4c210c6 into e-mission:master Mar 24, 2022
@PatGendre
Copy link
Contributor

@shankari thanks !
just a question about this script : it uses geojson_feature_converter, and it is not completely clear which objects are exported.
I understand inferred trips and sections are exported, so the predicted mode / purpose computed later on the in pipeline is not exported, right ? Only the sensed_mode seems to be exported.
Also, I don't see if (and where) the manual/mode_confirm and manual/purpose_confirm are exported.

I ask this because the same code is used by the getTrips API which we use for transferring data to personal cozy clouds.

@shankari
Copy link
Contributor Author

shankari commented Mar 24, 2022

@PatGendre yes, this intentionally reuses the getTrips API. As part of a complete rewrite of the label display functionality in the UI (e-mission/e-mission-docs#674), which included scalability improvements, I made server side changes to work off the confirmed trips instead (78a3ebb). This means that the trip labels are indeed incorporated (at the trip level). The sensed modes continue to be exported at the sensed level.

@shankari
Copy link
Contributor Author

Note that I didn't quite get to the point where the user input is a generic JSON structure, but the UI directive is much simplified

@PatGendre
Copy link
Contributor

This means that the trip labels are indeed incorporated (at the trip level). The sensed modes continue to be exported at the sensed level.

Ok, fine, thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants