Skip to content

feat: add nasa problem downloader#688

Open
j8xixo12 wants to merge 3 commits intosolvcon:masterfrom
j8xixo12:download_nasa_data
Open

feat: add nasa problem downloader#688
j8xixo12 wants to merge 3 commits intosolvcon:masterfrom
j8xixo12:download_nasa_data

Conversation

@j8xixo12
Copy link
Collaborator

@j8xixo12 j8xixo12 commented Mar 7, 2026

This PR is derivatived from #684 and adding a downloader for downloading the nasa problem dataset and untar the data.

Copy link
Member

@yungyuc yungyuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Points to address.

  • Keep only one author in copyright header.
  • Use full text for BSD license in the copyright header.
  • Add docstrings to explain how to use this tool from command line.

@@ -0,0 +1,13 @@
# Copyright (c) 2026, Chun-Shih Chang <austin20463@gmail.com>,
# Chun-Hsu Lai <as2266317@gmail.com>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep only one author. @j8xixo12 @ThreeMonth03 , you decide who is listed in the copyright header.

@@ -0,0 +1,13 @@
# Copyright (c) 2026, Chun-Shih Chang <austin20463@gmail.com>,
# Chun-Hsu Lai <as2266317@gmail.com>
# BSD 3-Clause License, see COPYING
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use full text for license in the file copyright header.

Initialize download/load configuration.
"""

self.url = "https://techport.nasa.gov/api/file/presignedUrl/380503"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make URL an argument.

# BSD 3-Clause License, see COPYING

"""
track: Track nasa flight dataset with Kalman Filter.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add docstrings to explain how to use this tool from command line.

@yungyuc yungyuc requested a review from ThreeMonth03 March 7, 2026 12:52
@yungyuc yungyuc added the array Multi-dimensional array implementation label Mar 7, 2026
@j8xixo12 j8xixo12 force-pushed the download_nasa_data branch from 60c4216 to 91b523b Compare March 7, 2026 15:43
Copy link
Collaborator Author

@j8xixo12 j8xixo12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yungyuc This PR is ready for the next review.

# OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
# LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
# NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
# EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yungyuc I have put the full text of BSD-3 clause license.

@@ -0,0 +1,38 @@
# Copyright (c) 2026, Chun-Shih Chang <austin20463@gmail.com>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThreeMonth03 I put your name as the author!

track: Track NASA flight dataset with Kalman Filter.

How to download the dataset:
PYTHONPATH=. python3 -m modmesh.track.dataset
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added the dataset download cli example here.

Comment on lines +113 to +114
"https://techport.nasa.gov/api/file/presignedUrl/380503",
"DDL-F1_Dataset-20201013.zip",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To accommodate different file names for different url. I have modified the NasaDataset to take url and file name as arguments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

array Multi-dimensional array implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants