-
Notifications
You must be signed in to change notification settings - Fork 1
MVP routing module #175
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
Open
ppinchuk
wants to merge
65
commits into
main
Choose a base branch
from
pp/routing_module
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
MVP routing module #175
Changes from 48 commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
5dd5ddd
Update
ppinchuk a30e668
Update
ppinchuk 7f603ad
Merge remote-tracking branch 'origin/main' into pp/routing_module
ppinchuk 515b2ac
Update lockfile
ppinchuk a3044b0
Add new exception type
ppinchuk b0e4dc3
Add another error type
ppinchuk 60c1fca
Log class names
ppinchuk c880945
Update tests to check for name
ppinchuk 1601907
Update logic to look for shape dataset
ppinchuk 74bd763
Add first few tests for routing
ppinchuk 6fec6e9
Routing WIP!!
ppinchuk a3f4c3d
More tests
ppinchuk 953561b
Add test
ppinchuk 9213d5b
Add test
ppinchuk c1bc645
Fix typo
ppinchuk f483595
Fix bug
ppinchuk 08b14b3
Add tracked layers test
ppinchuk 5bffe8a
Add geometry test
ppinchuk 7e5df95
A few more tests
ppinchuk fd86ec3
Add `__init__.py` to module
ppinchuk 97f49fb
Updates
ppinchuk 4e82865
Don't allow negative costs
ppinchuk 9b31040
Add tests for invalid start/end costs
ppinchuk dce81b3
Check for error message logs
ppinchuk fe8ee72
Add routing utility tests
ppinchuk a80c677
Add utilities module
ppinchuk 2438141
Update docstrings
ppinchuk 54ca855
Add first pass of cli file
ppinchuk 57f558e
Minor updates
ppinchuk d82b8ae
Connect first routing CLI
ppinchuk fe8f4aa
Add changes to points logic
ppinchuk fa64388
Minor docstring updates
ppinchuk 2faaa31
Update
ppinchuk b16bef0
Partially fix tests
ppinchuk e9fa521
Fix tests
ppinchuk ed91627
Add docstrings
ppinchuk 2fe36fb
Minor updates
ppinchuk 3117127
Fix docs
ppinchuk b12b5e6
No extra log
ppinchuk 73dd974
Fix recursion error
ppinchuk fbec893
Add tests
ppinchuk 7cd8bd4
MVP routing CLI tests
ppinchuk c78f4f6
Minor refactor
ppinchuk e614fa6
Minor formatting
ppinchuk 15cd004
update tests
ppinchuk 024e837
Update tests
ppinchuk 3c796a0
Merge remote-tracking branch 'origin/main' into pp/routing_module
ppinchuk 45e5d39
Update lockfile
ppinchuk 8ad80fe
Merge remote-tracking branch 'origin/main' into pp/routing_module
ppinchuk 06dad47
MInor formatting
ppinchuk b53749e
Update instructions
ppinchuk d4968ac
Break out dictionary extraction
ppinchuk eb0fe31
Minor update
ppinchuk bc2a0fc
Docstrings
ppinchuk bb49f8d
Add a few tests
ppinchuk dbaad90
Update
ppinchuk cd2d559
Minor fix
ppinchuk 21fe9b6
More messages
ppinchuk 891b822
Use `da.max`
ppinchuk 90dc0c3
Fix rust tests
ppinchuk 6d0edd8
Merge remote-tracking branch 'origin/main' into pp/routing_module
ppinchuk 91d72d7
Minor refactor
ppinchuk 0dd2217
Much lower memory limit
ppinchuk 1188678
`add_layer_to_data` now returns `Result` object
ppinchuk 22e2519
More error types
ppinchuk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,7 +5,7 @@ build-backend = "maturin" | |
| [project] | ||
| name = "NREL-reVRt" | ||
| version = "0.2.1" | ||
| description = "National Renewable Energy Laboratory's (NREL's) Transmission for reVX tool" | ||
| description = "National Laboratory of the Rockies' (NLR's) Transmission for reVX tool" | ||
| readme = "README.rst" | ||
| authors = [ | ||
| {name = "Guilherme Castelão", email = "[email protected]"}, | ||
|
|
@@ -16,7 +16,7 @@ maintainers = [ | |
| {name = "Paul Pinchuk", email = "[email protected]"}, | ||
| ] | ||
| license = {text = "BSD-3-Clause"} | ||
| keywords = ["NREL", "routing", "transmission", "reVRt", "reV", "reVX"] | ||
| keywords = ["NLR", "routing", "transmission", "reVRt", "reV", "reVX"] | ||
| requires-python = ">= 3.11" | ||
| classifiers=[ | ||
| "Development Status :: 2 - Pre-Alpha", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| """reVRt routing logic""" |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.