forked from simbli/seacharts
-
Notifications
You must be signed in to change notification settings - Fork 3
Get review from Copilot #33
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
meeqn
wants to merge
113
commits into
pr_review_test
Choose a base branch
from
main
base: pr_review_test
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
Changes from 110 commits
Commits
Show all changes
113 commits
Select commit
Hold shift + click to select a range
5fbd7c4
Initial changes, started work on format separation
meeqn d316396
Further changes regarding early parsing
meeqn e8b5c22
3.03. changes - coordinates
nanatalia1 3582a85
Changes for autosize
meeqn 334d318
Merge branch 'Natalia_Branch' into miqn_branch
meeqn d7f379c
Merge pull request #1 from meeqn/miqn_branch
meeqn 8948ff8
3.03. changes - coordinates
nanatalia1 46d1995
3.03. changes - coordinates
nanatalia1 fecb285
3.03. changes - parser S-57
nanatalia1 a46a4dc
13.03. changes - zone
nanatalia1 cac154d
Delete .idea directory
nanatalia1 65e7ad6
config new
nanatalia1 bca170f
Merge remote-tracking branch 'origin/Natalia_Branch' into Natalia_Branch
nanatalia1 6a31161
lat long to UTM
nanatalia1 9ca1875
changed zone (one per map)
nanatalia1 6181aaf
pre pull
meeqn 1802f74
pre pull
meeqn d5fbf2f
Merge remote-tracking branch 'origin/Natalia_Branch' into miqn
meeqn 18d2acc
working early S-57 maps conversion + display for requested depths, si…
meeqn 0086640
Merge pull request #2 from meeqn/miqn_branch
meeqn f4a3896
updated configuration for different utm zones, display should now han…
meeqn bba05eb
Merge pull request #3 from meeqn/miqn
meeqn 52e73e1
fixed coordinate offsets
meeqn 00ad4db
Merge pull request #4 from meeqn/miqn
meeqn eb5eea3
weather download implementation
kdrozd-dev df81fe1
minor bug fix
kdrozd-dev 532c971
Update weatherManager.py
kdrozd-dev 9489804
weather data into layers
kdrozd-dev 4679eee
weather data into layers
kdrozd-dev 67fac0f
Merge branch 'konrad_branch' of https://github.com/SanityRemnants/sea…
kdrozd-dev 6ecf05a
added map clipping
meeqn e812a42
Merge branch 'miqn' of https://github.com/meeqn/seacharts_s57_WIP int…
kdrozd-dev 002673f
Merge pull request #1 from SanityRemnants/konrad_branch
kdrozd-dev 2868261
more global coordinate converter
meeqn f6effa6
weather operation UI, bugfix
meeqn 1e497a3
added weather display heatmap
kdrozd-dev c1fa69f
display update
kdrozd-dev d2c238a
update visualization
kdrozd-dev adfa935
proper demo
kdrozd-dev d842a2e
Update shape.py
kdrozd-dev 4ec82ab
Added weather data access from code
kdrozd-dev 9844a4f
Update weather.py
kdrozd-dev 482426d
Update display.py
kdrozd-dev 0d5a4d2
Merge pull request #5 from SanityRemnants/konrad_branch
meeqn 2022572
Update weather.py
kdrozd-dev 7f615e6
Merge pull request #6 from SanityRemnants/konrad_branch
meeqn 03fec2f
working on display fix
meeqn 5dcdd9b
v0.5 for displaying shoreline, layers display in S-57 fixed for light…
meeqn b284ce0
minor fixes
meeqn eeb067a
Merge pull request #7 from meeqn/s57
meeqn a1d81c0
minor display improvements
kdrozd-dev 7290b20
z_axis fixes
meeqn 2abc9e1
Merge pull request #8 from meeqn/s57
meeqn 19d2997
enable weather display correction for data cut by projection
kdrozd-dev 3c3ee7f
Update display.py
kdrozd-dev c03eb13
Update display.py
kdrozd-dev d3054b4
extra layers for S-57 maps, new config schema (with time separated fr…
meeqn 1113cd6
added improvements for dynamic z_order, and some error proofing for F…
meeqn bb5168f
Merge pull request #9 from meeqn/z_order_and_display_fixes
meeqn 23c7929
abstraction for DataClass and code improvements
meeqn 2039edd
initial commit adding functionality for having shapefiles for multipl…
meeqn 4818b8e
Merge pull request #10 from meeqn/z_order_and_display_fixes
meeqn 9d5ebdb
Merge pull request #11 from meeqn/multiple_maps_saved
meeqn b83cd0e
minor display improvements
kdrozd-dev ad6bb03
enable weather display correction for data cut by projection
kdrozd-dev 778ccf4
Update display.py
kdrozd-dev a3dc595
Update display.py
kdrozd-dev 3d24fb5
Download slightly larger area to better fit the display
kdrozd-dev c899e6b
implement basic arrowmap
kdrozd-dev 9292852
added extension for time in config to specify hours and minutes, minu…
meeqn f0e6903
Merge pull request #12 from meeqn/time_extended_to_hours
meeqn 3401541
Improve arrow map drawing
kdrozd-dev bcbdda0
Merge remote-tracking branch 'upstream/main' into konrad_branch
kdrozd-dev eeb6c58
slider handling
kdrozd-dev f3860cb
autoconfig for windows and minor fixes
meeqn 5a9e9c7
display WIP
kdrozd-dev a79cc6c
Merge branch 'konrad_branch' of https://github.com/SanityRemnants/sea…
kdrozd-dev 3b609ca
Merge remote-tracking branch 'upstream/main' into konrad_branch
kdrozd-dev 8c28bb4
Weather + UI integration
kdrozd-dev 49e2936
Merge pull request #13 from SanityRemnants/konrad_branch
meeqn 311c432
some stylistic changes
meeqn db447c7
added unit display to colorbar
kdrozd-dev a0d49a3
Merge pull request #14 from SanityRemnants/konrad_branch
meeqn 9eb9dce
Fix weather access
kdrozd-dev 56b200a
Merge pull request #15 from SanityRemnants/konrad_branch
meeqn 6c1d907
path fix and label for slider deleted
meeqn db45821
Merge branch 'main' of https://github.com/meeqn/seacharts_s57
meeqn ce727dd
shallow waters are not underlapping whole map now
meeqn 686ebaf
Merge pull request #16 from meeqn/reduce_depth_overlapping
meeqn a15aca8
loading shapefiles simplified
meeqn 7c6408f
Merge pull request #17 from meeqn/reduce_depth_overlapping
meeqn 28e4a73
get records
nanatalia1 15da5ac
get records
nanatalia1 bd9547e
Merge pull request #18 from meeqn/get_arguments
nanatalia1 01ca537
add warning for projection limit exceed
meeqn 63105cf
.
meeqn 8681063
Merge pull request #19 from meeqn/utility_updates
meeqn 3983e82
import fix, comments clarified in config, config file for setting up …
meeqn 117349f
added some description for users running SC for the first time (#22)
meeqn 293dbb9
Readme update (#20)
nanatalia1 e6b38a0
Display tweaks (#23)
meeqn a842b48
Documentation update (#24)
meeqn 8df73e7
Documentation update (#26)
meeqn cf237a3
fixes
meeqn 330a949
pushing user_doc to remote (#27)
meeqn abc2b25
Added get_value for weather data (#25)
kdrozd-dev 5753843
Minor fixes & tests update (#29)
meeqn 87f1426
getting parameters from layer (#30)
meeqn 06c65b4
removed the min depth constraint from schema (#31)
meeqn abe7e43
2025 premerge tweaks (#32)
meeqn 68d094f
Fixed some issues based on Copilot-generated review. (#34)
meeqn 0ef9751
Commit premerge suggestions (#35)
meeqn 080ff20
Add Table of Contents in README.md for improved navigation (#36)
meeqn 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -138,4 +138,7 @@ data/db/ | |
| data/shapefiles/ | ||
| data/vessels.csv | ||
| output/ | ||
| .vscode/ | ||
| .vscode/ | ||
|
|
||
| #pycharm | ||
| .idea | ||
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,43 @@ | ||
| # first, add conda-forge to channel using: | ||
| # conda config --add channels conda-forge | ||
|
|
||
| # when in directory with conda_requirements.txt file, use: | ||
| # conda create --name <env_name> --file conda_requirements.txt | ||
|
|
||
| # next, activate the environment (eg. select interpreter via Visual Studio Code UI or manually via CLI) | ||
|
|
||
| attrs=23.2.0 | ||
| cartopy=0.22.0 | ||
| cerberus=1.3.5 | ||
| certifi=2024.8.30 | ||
| click=8.1.7 | ||
| click-plugins=1.1.1 | ||
| cligj=0.7.2 | ||
| colorama=0.4.6 | ||
| contourpy=1.2.0 | ||
| cycler=0.12.1 | ||
| fiona=1.9.6 | ||
| fonttools=4.50.0 | ||
| gdal=3.8.4 | ||
| ipython=8.26.0 | ||
| ipywidgets=8.1.5 | ||
| jedi=0.19.1 | ||
| js2py=0.74 | ||
| jupyterlab_widgets=3.0.13 | ||
| kiwisolver=1.4.5 | ||
| matplotlib=3.8.3 | ||
| matplotlib-scalebar=0.8.1 | ||
| numpy=1.26.4 | ||
| packaging=24.0 | ||
| parso=0.8.4 | ||
| pillow=10.2.0 | ||
| pyparsing=3.1.2 | ||
| pyproj=3.6.1 | ||
| pyshp=2.3.1 | ||
| python-dateutil=2.9.0 | ||
| pyyaml=6.0.1 | ||
| requests=2.31.0 | ||
| shapely=2.0.3 | ||
| six=1.16.0 | ||
| tzdata=2024a | ||
| widgetsnbextension=4.0.13 |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Malformed markdown image syntax - missing closing parenthesis on line 208. Should be
or the description should be on the same line within the brackets.