Added project management guidelines #2061
Conversation
Signed-off-by: Manjita Pandey <[email protected]>
for more information, see https://pre-commit.ci
manjitapandey
changed the title
| - Click on _I would like to include my own imagery layer for reference_ | ||
| in the first step to add TMS URL. You can get the URL by uploading it in openaerialmap. | ||
|
|
||
| **ODK Central Credentials** |
There was a problem hiding this comment.
Best to use nested headings here so they appear on the sidebar (####), otherwose I'm sure the markdown linter will complain 😄
There was a problem hiding this comment.
Thanks @spwoodcock , This helps me to fix the error.
| - Ensure you have the map features ready for the area you plan | ||
| to survey before starting project creation. | ||
| - The files should be in GeoJSON format, use the WGS coordinate | ||
| system with EPSG 4326, and must not include a Z-coordinate. |
There was a problem hiding this comment.
Do we not strip a z coord if present? Of we don't, we probably should! This info can be omitted as it should be handled automatically for the user
There was a problem hiding this comment.
If we can do it from backend, I can create a ticket regarding so and remove it from here. But for now, this is the behavior.
cc: @Sujanadh
There was a problem hiding this comment.
We do have a function to strip z coordinates, but the current implementation doesn't work as expected. I will update the function to make sure it removes the z coordinate from geometry. Most of the cases when data extract do not work during splitting are due to missing tags. I think we don't need to check tags if it is custom extracts so will update that as well so there won't be any issue with custom extracts if they are valid. 👍
There was a problem hiding this comment.
No worries @Sujanadh I made a new repo to handle this: https://github.com/hotosm/geojson-aoi-parser
Thought it might be handy to have all this logic in a module we can use in FMTM / Drone-TM / raw-data-api / other tools 😄
| Healthcare only. We plan to add more types of features moving ahead. | ||
| - Project managers can also upload supporting map features. Note that these | ||
| secondary features can’t be surveyed but selected for respective primary features. | ||
| To understand the use cases, please follow the link here. |
There was a problem hiding this comment.
Should there be a link somewhere?
There was a problem hiding this comment.
Yes Sorry, I was thinking I could dig in to some use cases of supporting map feature, document it somewhere and link it here but for now I can remove this from here.
manjitapandey
force-pushed
the
manjitapandey-patch-4
branch
from
5b5628d to
3b15359
Compare
manjitapandey
force-pushed
the
manjitapandey-patch-4
branch
from
b51c3c2 to
c1779e4
Compare
for more information, see https://pre-commit.ci
What type of PR is this? (check all applicable)
Related Issue
Example: Fixes #921
Describe this PR
This PR contains the mapping guidelines for field mappers and training and project creation guidelines for project managers that may help them in using FMTM smoothly. It also includes the contraints FMTM has and encourage them to follow the certain workflow FMTM has to offer.
Screenshots
Please provide screenshots of the change.
Alternative Approaches Considered
Did you attempt any other approaches that are not documented in code?
Review Guide
Notes for the reviewer. How to test this change?
Checklist before requesting a review
[optional] What gif best describes this PR or how it makes you feel?