langchain icon indicating copy to clipboard operation
langchain copied to clipboard

docs: `ecosystem` change

Open leo-gan opened this issue 2 years ago • 6 comments

move Additional_resources items into Ecosystem group Rationale: ToC groups composed from <3 items are too small. These small groups spent precious ToC space and distract user attention. Here I merge two small groups.

Maintainer responsibilities: @dev2049

leo-gan avatar Jun 21 '23 17:06 leo-gan

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jul 5, 2023 8:20pm

vercel[bot] avatar Jun 21 '23 17:06 vercel[bot]

we had it like this before but got feedback from other folks that ecosystem was a confusing grouping. maybe we wait to gather more feedback?

dev2049 avatar Jun 21 '23 17:06 dev2049

we had it like this before but got feedback from other folks that ecosystem was a confusing grouping. maybe we wait to gather more feedback?

I agree that "Ecosystem" is a too broad term and hence confusing. Still, PyTorch uses this term.

This PR is a trade-off between moving "Additional resources" items into "Ecosystem" or opposite, moving "Ecosystem" items into "Additional resources". The latter doesn't work because "YouTube" and "Gallery" can be part of the "Ecosystem" but the "Integrations" cannot be part of the "Additional resources". So, decision was to move "YouTube" and "Gallery" into the "Ecosystem".

How about renaming "Ecosystem" to "Resources" (as in PyTorch again)?

BTW @hwchase17 insisted on "Ecosystem" vs "Integrations" the last time.

leo-gan avatar Jun 21 '23 19:06 leo-gan

@hwchase17 Can you, please, give feedback about ^? Thanks!

leo-gan avatar Jun 27 '23 02:06 leo-gan

@baskaryan Please review it. Thanks!

leo-gan avatar Jun 30 '23 18:06 leo-gan

@baskaryan Please review it. Thanks!

leo-gan avatar Jul 07 '23 00:07 leo-gan

Closing it because nobody reviews it.

leo-gan avatar Jul 26 '23 17:07 leo-gan