Show project homepages in the Hydra Landscape - #3381
Open
adarshsm wants to merge 1 commit into
Open
Conversation
Landscape entries only linked to a repository. A listing may now also carry an optional homepage, shown next to the repository link, and entries without one are unchanged. - refresh_landscape_metadata.py requests homepageUrl and records it verbatim apart from surrounding whitespace, so an unset homepage stays None. - build_landscape_review.py reports the homepage in the review queue, which is where a maintainer decides whether to promote it into a listing. - build_public_landscape.py accepts an optional listing.homepage, requires HTTPS, rejects a homepage that merely repeats listing.url, and omits the key entirely when absent. - The website validator and the Landscape card handle the optional field. - decisions.json records the homepage GitHub reports for 61 of the 104 included projects. Closes hydra-ecosystem#3375
Collaborator
|
Thanks.
|
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
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.

Closes #3375.
Landscape entries linked only to a repository. A listing may now also carry an optional
homepage, shown beside the repository link; entries without one render exactly as before.What changed
refresh_landscape_metadata.pyhomepageUrland records it verbatim apart from surrounding whitespace, so an unset homepage (GitHub returnsnullor"") becomesNone.build_landscape_review.pybuild_public_landscape.pylisting.homepage, requires HTTPS, rejects one that merely repeatslisting.url, and omits the key entirely when absent.validate-landscape.mjsLandscapeDirectorydata/decisions.jsonThe repository link is untouched and still present on every card.
Verification
PYTHONPATH=tools/landscape pytest tools/landscape→ 95 passed. Reverting the three tool changes makes 10 of the new tests fail, so they pin the new behaviour rather than passing vacuously.npm run validate:landscapepasses. I also confirmed the new validator rules actually bite by mutating the generated file: ahttp://homepage, a whitespace-only homepage, and a homepage equal tourleach fail with the intended message; an unknown field is still rejected.npm run buildsucceeds and the builtdocs/landscape/index.htmlcontains exactly 61Homepagelinks, matching the 61 populated entries.landscape.jsonbyte-for-byte, and thedecisions.jsondiff contains only added"homepage"lines.Two curation calls that are yours, not mine
1. Ten of the 61 homepages are publications rather than project sites. These are what the projects themselves set on GitHub, so I recorded them faithfully, but a reader clicking "Homepage" may not expect a paper:
HazyResearch/hyena-dna,PRIME-RL/TTRL,SWivid/F5-TTS,SysCV/sam-pt,aiming-lab/SkillRL,pdebench/PDEBench(arXiv) ·eloialonso/iris(OpenReview) ·microsoft/mattergen(Nature) ·mir-group/nequip(RSC) ·langfengQ/verl-agent(HF papers).DBD-research-group/BirdSetpoints at a HF dataset.Dropping them is a one-line filter in the ledger if you prefer. Note
mir-group/nequip's URL returns 403 to automated probes — publisher bot protection, the page is real.2. Three repositories declare an HTTP-only homepage and are therefore not included:
dssg/aequitasandmees/calvinhave no working HTTPS at all, so they are omitted rather than silently upgraded.hydra-ecosystem/hydradeclareshttp://hydra.cc/, which does serve HTTPS, so it is recorded ashttps://hydra.cc/.Smaller decisions worth a glance
schemaVersionstays at 2. The field is purely additive and the only consumer ships in this repo. Happy to bump it if you would rather every shape change be visible.nameandurl. Two listings from one monorepo can legitimately share a homepage, so I deliberately left them out of the uniqueness sets.Homepagebeside it, which is the smallest change satisfying every bullet in the issue. If you would rather see two explicitly labelledHomepage/Repositorylinks instead of the slug, that is a small edit toLandscapeCard.