Skip to content

Enh/hierarchical catalog#913

Merged
tturocy merged 8 commits into
masterfrom
enh/hierarchical_catalog
Jun 5, 2026
Merged

Enh/hierarchical catalog#913
tturocy merged 8 commits into
masterfrom
enh/hierarchical_catalog

Conversation

@rahulsavani
Copy link
Copy Markdown
Member

@rahulsavani rahulsavani commented Jun 3, 2026

organised via the following top level directories:

  • books
  • journals
  • conf (for conferences)

Closes #912

@rahulsavani
Copy link
Copy Markdown
Member Author

@edwardchalstrey1

Locally this did not work in the following sense:

I still had all the old entries. I think this may be because catalog/img is not cleaned out or something like that? In any case, please take a look.

@tturocy

Are you OK with the top-level directories?

@rahulsavani rahulsavani requested a review from tturocy June 3, 2026 06:23
@rahulsavani
Copy link
Copy Markdown
Member Author

@edwardchalstrey1 @tturocy

Separately I forgot we use these in the tests. Will fix that aspect later today.

@tturocy
Copy link
Copy Markdown
Member

tturocy commented Jun 3, 2026

@edwardchalstrey1

Locally this did not work in the following sense:

I still had all the old entries. I think this may be because catalog/img is not cleaned out or something like that? In any case, please take a look.

@tturocy

Are you OK with the top-level directories?

Generally. Was going to ask about your handling of Reiley et al 2008 - that's a journal article but it's in "other" as the top level. It's a journal we're not likely to have a lot of catalog entries from, so should that be perhaps journal/other rather than misc? misc seems like a good name for unpublished games that are useful but don't have a clear provenance.

Copy link
Copy Markdown
Member

@tturocy tturocy left a comment

Choose a reason for hiding this comment

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

See comment in conversation

@rahulsavani
Copy link
Copy Markdown
Member Author

rahulsavani commented Jun 3, 2026

@edwardchalstrey1
Locally this did not work in the following sense:
I still had all the old entries. I think this may be because catalog/img is not cleaned out or something like that? In any case, please take a look.
@tturocy
Are you OK with the top-level directories?

Generally. Was going to ask about your handling of Reiley et al 2008 - that's a journal article but it's in "other" as the top level. It's a journal we're not likely to have a lot of catalog entries from, so should that be perhaps journal/other rather than misc? misc seems like a good name for unpublished games that are useful but don't have a clear provenance.

@tturocy

Totally agree for Reiley et 2008. I am very happy with you suggestion

What about this one:

Shapley, L.S. (1974). A note on the Lemke-Howson algorithm. In: Balinski, M.L. (eds) Pivoting and Extension. Mathematical Programming Studies, vol 1. Springer, Berlin, Heidelberg

It's a (hard to find) chapter in a series that didn't continue. It's an edge case for sure.

journals/other
misc/

something else?

@tturocy
Copy link
Copy Markdown
Member

tturocy commented Jun 3, 2026

journals/other seems good. Although "journal" is slightly a misnomer, but we're unlikely to have lots of games that originate in series rather than more standard journals.

@edwardchalstrey1
Copy link
Copy Markdown
Member

Locally this did not work in the following sense:

I still had all the old entries. I think this may be because catalog/img is not cleaned out or something like that? In any case, please take a look.

@rahulsavani See the note on step 3

You can use the --regenerate-images flag when building the docs locally for a second time to force any changes to be picked up.

@edwardchalstrey1
Copy link
Copy Markdown
Member

@rahulsavani I didn't make the changes directly on this branch in case you are in the midst of making changes but #915 can be merged into this branch to update the tests and docs with the new hierarchy

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rahulsavani rahulsavani requested a review from tturocy June 3, 2026 19:45
@rahulsavani
Copy link
Copy Markdown
Member Author

@rahulsavani I didn't make the changes directly on this branch in case you are in the midst of making changes but #915 can be merged into this branch to update the tests and docs with the new hierarchy

Thanks, I sadly missed before I had done most of the work myself, but there we go. We are close now! Once this is merged it will allow me to get others contributing games effectively hopefully in the coming days, while you are still around.

@edwardchalstrey1
Copy link
Copy Markdown
Member

edwardchalstrey1 commented Jun 4, 2026

@rahulsavani I've updated the catalog page so there is now a collapsible structure that reflects the hierarchy you added - there's a new config called catalog_hierarchy.yaml which can be used to give a name to Books, Conferences, Journals and articles within journals - the names are needed so the hierarchy dropdowns are labelled in a way that makes sense. Can you check the names make sense? Page: https://gambitproject--913.org.readthedocs.build/en/913/catalog.html#catalog

@rahulsavani
Copy link
Copy Markdown
Member Author

@rahulsavani I've updated the catalog page so there is now a collapsible structure that reflects the hierarchy you added - there's a new config called catalog_hierarchy.yaml which can be used to give a name to Books, Conferences, Journals and articles within journals - the names are needed so the hierarchy dropdowns are labelled in a way that makes sense. Can you check the names make sense? Page: https://gambitproject--913.org.readthedocs.build/en/913/catalog.html#catalog

These are perfect, at least for now. Let's get this merged in, and I will do my best to get some more games in for 16.7.

@edwardchalstrey1
Copy link
Copy Markdown
Member

@rahulsavani I've updated the catalog page so there is now a collapsible structure that reflects the hierarchy you added - there's a new config called catalog_hierarchy.yaml which can be used to give a name to Books, Conferences, Journals and articles within journals - the names are needed so the hierarchy dropdowns are labelled in a way that makes sense. Can you check the names make sense? Page: https://gambitproject--913.org.readthedocs.build/en/913/catalog.html#catalog

These are perfect, at least for now. Let's get this merged in, and I will do my best to get some more games in for 16.7.

Needs an approval from @tturocy

@tturocy tturocy merged commit 4f76fc1 into master Jun 5, 2026
26 checks passed
@tturocy tturocy deleted the enh/hierarchical_catalog branch June 5, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Catalog]: Way to view catalog via its hierarchical structure

3 participants