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

Update server.js #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

RCLDevelopers
Copy link

So i figured i add these comments to provide a comprehensive overview of each core function's purpose, parameters, flow, and usage context. Each section describes the key responsibilities and interactions of the function within the larger system.

So i figured i add these comments to provide a comprehensive overview of each core function's purpose, parameters, flow, and usage context. Each section describes the key responsibilities and interactions of the function within the larger system.
@RCLDevelopers
Copy link
Author

This helps explaining the core functions of this server script

@luisliz
Copy link

luisliz commented Nov 6, 2024

Just imo but might be better to put this on an md file right in root folder. I guess you mostly care about this if you're reading the codebase but an md might be more intuitive since I might not look to the bottom of a file.

@raidendotai
Copy link
Owner

agreed

  • architecture modules (~next week) should have extensive docs alongside every function, and every inference tracing back to which architecture module called it
  • aim is also to have meta-prompts to spin up new architectures faster ; ie. describe general purpose and have the legos put together ; docs for every existing process (starting next releases) will help towards the direction

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