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

[FEATURE] Flexible CRS #135

Open
5 tasks
e-lo opened this issue Apr 4, 2022 · 3 comments
Open
5 tasks

[FEATURE] Flexible CRS #135

e-lo opened this issue Apr 4, 2022 · 3 comments
Assignees
Labels
feat New feature or request MetCouncil Things for MetCouncil

Comments

@e-lo
Copy link
Collaborator

e-lo commented Apr 4, 2022

User Story

As a Lasso user, I'd like to be able to create project cards using a network from a different projections
As a Lasso user, I'd like to be able to write networks to a projection which I specify

Issue Checklist

  • Sufficiently Defined
  • User Story Addressed
  • Tests Added
  • Documentation Updated
  • Tests Pass

Priority

High

Level of Effort

Low-medium

Resolution Ideas

Use code implemented for MTC.

Project

MetCouncil

Who should be involved?

Users: @RachelWikenMC
Reviewers: @i-am-sijia

Risk

Will this potentially break anything?
Low

Tests

What are relevant tests or what tests need to be created in order to determine that this issue is complete?
Will write one

@e-lo e-lo added feat New feature or request MetCouncil Things for MetCouncil labels Apr 4, 2022
@e-lo e-lo added this to the MetCouncil release milestone Apr 4, 2022
@e-lo e-lo self-assigned this Apr 4, 2022
@e-lo e-lo changed the title [FEATURE] Write [FEATURE] Flexible CRS Apr 4, 2022
@e-lo
Copy link
Collaborator Author

e-lo commented Apr 4, 2022

@RachelWikenMC @DavidOry I just want to make sure I understand the problem appropriately...the develop branch currently allows you to specify the output CRS for the network either when doing the write-out or in the parameters:

roadway_standard_to_met_council_network(self, output_epsg=None):

Is this not working?
Or is this not part of the problem you need fixing?

@e-lo e-lo mentioned this issue Apr 4, 2022
1 task
@DavidOry
Copy link
Member

DavidOry commented Apr 5, 2022

Thanks @e-lo. The issue is not writing the network. The issue is:

  1. We make an edit in Cube and record a log file.
  2. Because the Cube network is in the model CRS, the log file records new nodes in the model CRS.
  3. Network Wrangler is expecting lat/lng CRS. So when the Project Card is read in, it fails.

For MTC, @i-am-sijia added functionality to write out the Project Card in lat/lng CRS.

@RachelWikenMC
Copy link
Collaborator

Thanks @DavidOry , I was out yesterday. That correctly summarizes the issue. I need the create project card step to output in lat/long and not model CRS. Currently having to correct project cards by hand for new node coordinates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request MetCouncil Things for MetCouncil
Projects
None yet
Development

No branches or pull requests

3 participants