Skip to content
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

refactor documentation #38

Merged
merged 8 commits into from
Jan 22, 2024
Merged

refactor documentation #38

merged 8 commits into from
Jan 22, 2024

Conversation

aaguinal
Copy link
Contributor

This PR updates the documentation found on https://algebraicjulia.github.io/AlgebraicRewriting.jl/dev/.

More specifically, this:

  • Provides a gentle overview of the package on the homepage
  • Maintains the examples such that they are compatible with Catlab 0.16.5 and AlgebraicRewriting 0.3.0
  • Compiles the library reference docs

@aaguinal aaguinal self-assigned this Jan 17, 2024
@aaguinal aaguinal linked an issue Jan 17, 2024 that may be closed by this pull request
Copy link

codecov bot commented Jan 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fc2269a) 81.55% compared to head (4ab460d) 76.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   81.55%   76.97%   -4.59%     
==========================================
  Files          28       28              
  Lines        1681     1681              
==========================================
- Hits         1371     1294      -77     
- Misses        310      387      +77     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@kris-brown kris-brown left a comment

Choose a reason for hiding this comment

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

Thanks so much Angeline, this is very helpful! I look forward to keeping up with the docs for new features in the future! I noticed the unicode art in the docstrings gets mangled, but happy to address that later.

```

#### Colimit-of-representables instantiation (`@acset_colim``)
Copy link
Collaborator

Choose a reason for hiding this comment

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

extra backtick

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

```

This documentation provides a basic guide to using the AlgebraicRewriting package in Julia.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this line belong at the beginning?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, you're right. It was a straggler. Moved it up.

If using the **static** approach, you must fully specify the ACSet functors and natural transformation. Here is a rule that defines the ACSet statically.

In this example, the rule swaps player between two teams.
Copy link
Collaborator

Choose a reason for hiding this comment

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

swaps players?

Also the rule doesn't swap player #1 and #4, not sure if that's intended.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that is. It was intended to simulate a trade of players, one from each team. I revised the sentence to hopefully make it clearer.

@kris-brown
Copy link
Collaborator

also I suspect that src/generated should be added to the .gitignore

@kris-brown
Copy link
Collaborator

Looks great to me, thanks!

@kris-brown kris-brown merged commit 448e085 into main Jan 22, 2024
8 of 9 checks passed
@kris-brown kris-brown deleted the 37-refactor-documentation branch January 22, 2024 23:03
repo="github.com/AlgebraicJulia/AlgebraicRewriting.jl.git",
branch="gh-pages",
devbranch="dev"
Copy link
Member

Choose a reason for hiding this comment

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

This should not be dev, this should be main

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.

Refactor documentation
3 participants