Remove KerasCV and KerasNLP; new structure for KerasHub and KerasTuner #1993
Conversation
mattdangerw
requested review from
fchollet,
MarkDaoust and
pcoet
as code owners
|
Not all KerasCV guides have a KerasHub replacement yet. These guides no longer show up on the site, but I've left them in the repo so they can be ported. |
|
would https://keras.io/keras_cv/ take to keras_cv page or the keras_hub page? |
|
@divyashreepathihalli KerasCV page will be gone! keras.io/keras_cv will redirect to keras.io/keras_hub You might want to try rendering locally to get a sense of the new structure. |
I will check it out! |
mattdangerw
force-pushed
the
new-structure
branch
from
c58a6d5 to
68fdde7
Compare
There was a problem hiding this comment.
Thanks for the PR! It looks great.
The redirects need to be more granular, especially for KerasCV -- anytime there's a relevant page we can point to, we should. Examples:
/api/keras_cv/layers/augmentation/*- >/api/layers/preprocessing_layers/image_augmentation//api/keras_cv/models/backbones/efficientnet_v2/-> corresponding API page in KerasHub/api/keras_cv/models/tasks/stable_diffusion/-> corresponding API page in KerasHub (note, this one has quite a few links that point to it!)- etc.
Please go over each one by hand and determine what the best page to redirect to is.
|
Thanks! Yeah, that is done for KerasTuner/KerasHub/KerasNLP since there's a one-to-one correspondence. But yeah KerasCV will need a more careful touch. Will do! |
mattdangerw
force-pushed
the
new-structure
branch
from
68fdde7 to
bb24e30
Compare
|
@fchollet improved the CV links a bit, though a good bit doesn't have a perfect replacement. E.g. for bounding boxes and preprocessing layers in CV I'm just linking to core Keras preprocessing layers. |
mattdangerw
force-pushed
the
new-structure
branch
from
bb24e30 to
a6fbf1d
Compare
guides/keras_hub/xx->keras_hub/guides/xx.api/keras_hub/xx->keras_hub/api/xx.