Skip to content

Update knowledge_level_agent_type_specification.md#459

Merged
edeutsch merged 3 commits into
masterfrom
mbrush-patch-1
Jan 16, 2024
Merged

Update knowledge_level_agent_type_specification.md#459
edeutsch merged 3 commits into
masterfrom
mbrush-patch-1

Conversation

@mbrush

@mbrush mbrush commented Sep 28, 2023

Copy link
Copy Markdown
Collaborator

Added note about short term plans for knowledge level annotations.

@andrewsu

Copy link
Copy Markdown

Personally, I don't love the link out to a google doc. I would rather capture the relevant information in the MD file itself...

@mbrush

mbrush commented Sep 28, 2023

Copy link
Copy Markdown
Collaborator Author

Yes, I get that, and am happy to translate the information into the markdown file. But it’s just stopgap measure for a couple months, and I just wanted people to know that they come to the specifications for guidance, and be able to see read what is going on in the short term while the permanent solution is implemented.
This specification document is a work in progress anyway, and definitely won’t have the information referenced externally when it is finalized in the coming months

@edeutsch

Copy link
Copy Markdown
Collaborator

Modeling is still ongoing in the EPC calls. We discussed at TRAPI call 2023-10-12 and decided to leave this PR open until a fully fleshed-out markdown can be added to the PR.

@mbrush

mbrush commented Jan 15, 2024

Copy link
Copy Markdown
Collaborator Author

@edeutsch I made significant changes to the spec here to reflect its current state of implementation, which will complement the updated developer documentation I am creating. This includes addition of more information to the spec md itself, rather than relying on linkouts to gdocs (there are still linkouts, but only for more supplementation info). Can you approve/merge the PR?

@edeutsch edeutsch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

great work, thanks!

@edeutsch edeutsch merged commit f046863 into master Jan 16, 2024
@edeutsch edeutsch deleted the mbrush-patch-1 branch May 30, 2025 22:35
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