Skip to content

Add KerasRS API documentation and examples #2083

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

Merged
merged 30 commits into from
Apr 29, 2025

Conversation

abheesht17
Copy link
Collaborator

@abheesht17 abheesht17 commented Apr 8, 2025

Do not review now.

TODO:

  • Add other layers when they are added to the repo.
  • Fix images and links in examples.
  • Add generated files [they aren't here on this PR yet, because we can generate them once we release].

@abheesht17 abheesht17 marked this pull request as ready for review April 29, 2025 15:47
@abheesht17 abheesht17 requested a review from hertschuh April 29, 2025 15:48
@abheesht17 abheesht17 changed the title [WIP] Add KerasRS API documentation and examples Add KerasRS API documentation and examples Apr 29, 2025
@@ -392,6 +394,12 @@ def sync_tutobook_templates(self):
os.makedirs(dst_dir)
copy_inner_contents(dir_path / "md", dst_dir, ext=".md")

shutil.copytree(
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a TODO here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No, this isn't a TODO. We have to copy over the examples to KerasRS

@@ -42,5 +43,6 @@
KERAS2_API_MASTER,
TUNER_MASTER,
HUB_MASTER,
RS_MASTER,
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't that the thing you should remove for now so that KERAS_RS doesn't appear in the menu?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Nope, all of that is in the HTML files

@hertschuh hertschuh merged commit 7a014ee into keras-team:master Apr 29, 2025
3 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