Skip to content

Update readme, refactor fcviz#158

Merged
PastorD merged 7 commits intomasterfrom
pastord-update-readme
Mar 31, 2026
Merged

Update readme, refactor fcviz#158
PastorD merged 7 commits intomasterfrom
pastord-update-readme

Conversation

@PastorD
Copy link
Copy Markdown
Contributor

@PastorD PastorD commented Mar 20, 2026

Remove outdated items in README.md

  • add latest Ubuntu
  • add instructions to install cogapp with apt
  • add instructions to use fcviz as a script with inline dependencies, avoiding global pip

Add doc action (not tested until we merge to master)

  • add action workflow to push to github pages
  • fix doxygen file issues

Refactor fcviz.py:

  • add inline dependencies to follow PEP 723
  • refactor graph generation to its own function
  • add type annotations and docstrings
  • fix linting and format

@dwai-wai @JosephBowkett @kwehage @preston-rogers

Copy link
Copy Markdown
Collaborator

@JosephBowkett JosephBowkett left a comment

Choose a reason for hiding this comment

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

great!!!

@PastorD PastorD merged commit bcec358 into master Mar 31, 2026
4 checks passed
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.

3 participants