-
Notifications
You must be signed in to change notification settings - Fork 370
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
Fix broken URLs #1125
Fix broken URLs #1125
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -36,15 +36,6 @@ class Chesapeake(RasterDataset, abc.ABC): | |
Center (CIC) in partnership with the University of Vermont and WorldView Solutions, | ||
Inc. It consists of one-meter resolution land cover information for the Chesapeake | ||
Bay watershed (~100,000 square miles of land). | ||
For more information, see: | ||
* `User Guide | ||
<https://chesapeakeconservancy.org/wp-content/uploads/2017/01/LandCover101Guide.pdf>`_ | ||
* `Class Descriptions | ||
<https://chesapeakeconservancy.org/wp-content/uploads/2020/03/LC_Class_Descriptions.pdf>`_ | ||
* `Accuracy Assessment | ||
<https://chesapeakeconservancy.org/wp-content/uploads/2017/01/Chesapeake_Conservancy_Accuracy_Assessment_Methodology.pdf>`_ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Some of these links are broken. Most of these docs are linked on the main dataset page, so I don't think we need to explicitly reference them. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fine with me |
||
""" | ||
|
||
is_image = False | ||
|
@@ -415,7 +406,7 @@ class ChesapeakeCVPR(GeoDataset): | |
additional layer of data to this dataset containing a prior over the Chesapeake Bay | ||
land cover classes generated from the NLCD land cover labels. For more information | ||
about this layer see `the dataset documentation | ||
<https://zenodo.org/record/5652512#.YcuAIZLMIQ8>`_. | ||
<https://zenodo.org/record/5866525>`_. | ||
If you use this dataset in your research, please cite the following paper: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,7 +35,7 @@ class EnviroAtlas(GeoDataset): | |
This dataset was organized to accompany the 2022 paper, `"Resolving label | ||
uncertainty with implicit generative models" | ||
<https://openreview.net/forum?id=AEa_UepnMDX>`_. More details can be found at | ||
https://github.com/estherrolf/qr_for_landcover. | ||
https://github.com/estherrolf/implicit-posterior. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Many of these are repo renames. The old URL still works and redirects to the correct page, but figured we should update them anyway. |
||
If you use this dataset in your research, please cite the following paper: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -159,10 +159,7 @@ class VHR10(NonGeoDataset): | |
"md5": "d30a7ff99d92123ebb0b3a14d9102081", | ||
} | ||
target_meta = { | ||
"url": ( | ||
"https://raw.githubusercontent.com/chaozhong2010/VHR-10_dataset_coco/" | ||
"master/NWPU%20VHR-10_dataset_coco/annotations.json" | ||
), | ||
"url": "https://raw.githubusercontent.com/chaozhong2010/VHR-10_dataset_coco/ce0ba0f5f6a0737031f1cbe05e785ddd5ef05bd7/NWPU%20VHR-10_dataset_coco/annotations.json", # noqa: E501 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should never download files from the master branch, they may change or move to a different location. Instead, we should use permalinks when available. |
||
"filename": "annotations.json", | ||
"md5": "7c76ec50c17a61bb0514050d20f22c08", | ||
} | ||
|
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.
Dataset was renamed and link broke