Existing updates from Armin #30
Merged
Conversation
This file contains 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
|
Tests of random program numbers all seem ok. I see log messages about batch queries, which is the solution to the timeout errors that Armin was working on solving. Webpages are created as long as uncal files are downloaded. |
|
@arminrest are you happy with how these changes look? If so I can merge them. |
bhilbert4
changed the title
This was referenced Mar 10, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds in a number of changes Armin had implemented locally.
Among them is a fix for the MAST timeout errors that were interfering with some searches. These were solved by querying in batches, at the recommendation of the MAST developers.
This also introduces sortable.js, for creating sortable tables when results are saved in html files.