-
Notifications
You must be signed in to change notification settings - Fork 24
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
thredds_crawler revial #35
Comments
I decided to take a crack at the last one. See https://nbviewer.org/gist/ocefpaf/374ce5c8131343e789e5261d1fe7d83c I'm not an expert on either packages (siphon or thredds_crawler), but it seems that one can do everything described in the thredds_crawler's README (the only docs we have), with siphon. Siphon is also faster and looks in a better shape. |
@ocefpaf if u can assign a few tasks ,would like to take a crack at them- any specific prs that may not be a ton of work?? |
@muhd360 if you are interested in GSoC'25 I recommend you to wait until the orgs are selected. At this point we don't know if we'll be selected and we don't have a list of projects ideas. If you are interested in |
@ocefpaf no i was just asking generally if there are any prs u could point me 2 would be glad 2 help |
Try to familiarize yourself with the codebase and see what interests you. |
@ocefpaf I wanted to use |
I do not, but we can ask upstream. If it is something we can workaround and/or upstream to siphon, what other features do you believe |
I opened an issue in siphon to ask about this but haven't been back to this effort since then to follow up: Unidata/siphon#263 Doing some regex matching to include or exclude links I think is in thredds_crawler but not siphon and is handy. |
I'm an advocating we re-archive thredds_crawler, but siphon's performance made me think we should, at least, be using it under the hood as the crawler engine. |
To do list (kind of in order of priority):
The text was updated successfully, but these errors were encountered: