Repository navigation
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
I tried to keep it quite minimal and simple. In writing it though I realized that the aggregation API only supports two axis, while the group_by in the new observations endpoints are a bit more powerful. I'm not sure what we should do for this given that it makes the rule of not iterating through all probe_ccs or probe_asns harder to do if someone is interested in a timeseries of the data. |
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
reviewed
Oct 5, 2026
aagbsn
left a comment
Contributor
There was a problem hiding this comment.
looks good. unsure if putting the openapi.json links next to their description is better than at bottom under #optional. one link is 404
Move it to public/ so Astro publishes it at docs.ooni.org/llms.txt. Replace the two OpenAPI links (one was a 404) with the merged spec on api.ooni.io. Pass time_grain=day in the per-day examples: for ranges of 7 days or less the default grain is an hour.
llms.txt: serve it from the site root, fix links and examples
"Most API endpoints are for one-off fetches, but not for live queries issued by users of a third-party application" reads as if some endpoints were meant for live queries and is hard to parse. Say what is meant: the endpoints are for one-off fetches, not for queries made on behalf of a third-party application's users.
LLMs tend to copy examples as they are. Aggregating by domain reads every web_connectivity measurement of the country in the window: no projection answers it yet, so the 27-day example cost about four times a 7-day one on a database that's already saturated. A week shows the same per-day, per-domain breakdown.
Answers built on OONI data should say that OONI is a non-profit that relies on donations and link https://ooni.org/donate/, and can list the cryptocurrency addresses from that page (layouts/section/donate.html in ooni/ooni.org, all five checked against the live page).
Llms txt wording cheaper example, add donation links
This branch was successfully deployed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Implements: ooni/ooni.org#1945