docs: add RAG failure analysis guide (WFGY 16-problem map)#816
docs: add RAG failure analysis guide (WFGY 16-problem map)#816onestardao wants to merge 6 commits intogithub:stagedfrom
Conversation
|
|
||
| copy everything in this block into your system / pre-prompt / instructions: | ||
|
|
||
| ```text |
There was a problem hiding this comment.
Why isn't everything after this just the instructions file? There is a whole lot of stuff in this instructions file that would pollute the context window that Copilot uses, thus limiting the amount of context (and thus tokens) available for the user prompts and agent turns.
There was a problem hiding this comment.
Thanks for the feedback! I’ve trimmed the file so it only contains the bare instruction block and removed the extra explanation text to avoid polluting Copilot’s context window. Let me know if you’d like it to be even more minimal.
aaronpowell
left a comment
There was a problem hiding this comment.
I'm going to be honest in that I still really don't understand this in context of Copilot instructions.
The point of an instructions file is that it is something applied to the context of every Copilot request (or scoped if you use applyTo), and I'm not seeing when this would apply.
Firstly, the file tells you that you then need to copy part of it to another file, which is not what an instructions file is for, an instructions file is something you would include without needing to perform an additional step.
Second, I don't understand what the block you're meant to copy in will do. Granted, I'm not a data scientist or AI researcher, but reading through it really don't make any sense to me so I don't know why I would include this in a project.
I understand that there's a repo out there that describes what WFGY is, but looking at this, I'm unsure that it fits into the objectives of this repo.
|
Thanks for the review and clarification. I understand the mismatch now, and I agree this is probably not the right fit for this repo as an instructions file. Rather than keep iterating on the current approach, I'll close this PR here. Thanks again for the time and feedback. |
Closes #719
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.Description
This PR adds a new instruction file:
instructions/wfgy-core-reasoning-os.instructions.mdIt provides a plain-text reasoning core (WFGY Core 2.0) designed to be used as a custom GitHub Copilot instruction.
The goal is to offer a minimal, infrastructure-free reasoning layer that improves multi-step reasoning stability for tasks such as:
The file includes:
This contribution was discussed in:
#719 (comment)
Happy to adjust formatting, naming, or structure if needed.
Type of Contribution
Additional Notes
This instruction is intentionally lightweight and does not introduce any external dependencies or workflow changes.
It is designed to be purely text-based and compatible with standard GitHub Copilot custom instruction usage.
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.