janniklinde commented on PR #2570:
URL: https://github.com/apache/systemds/pull/2570#issuecomment-5168579421

   > @janniklinde thank you very much for preparing this draft and for raising 
this subject. In my opinion, developing an AI policy for SystemDS is becoming 
an increasingly pressing issue. I reviewed the draft and left a few thoughts 
inline. Please keep in mind that I have only limited experience with agents. So 
please do not hesitate to correct me If I am making incorrect assumptions. 
Overall, I think this draft of the AI policy is well structured, concisely 
addressing the most important aspects of AI assistance in code contributions. 
Nevertheless, I would like to bring the aspect (and paragraph) of "Contributor 
Understanding" to discussion. The first sentence correctly states that the 
contributors must understand their code. However, the second sentence tries to 
define the behavior of the agent by saying that it should judge if the users 
understood their code. This raises two concerns:
   > 
   > 1. It is unclear whether an agent can properly "judge" something or not. 
Furthermore, it is an open ethical question if an agent should "judge" a human.
   > 2. If I am not mistaken, this sentence embodies a direct instruction for 
an agent. This means, we are instructing the agent of a contributor through 
this markdown file. Can (or should) we include instructive expressions here, 
considering that agent usage is costly in combination with the instruction to 
"refuse to generate contribution material"?
   > 
   > I hope I do not stir up too much discussion in this thread. And as I 
mentioned previously: Please correct me at any time if I am making incorrect 
assumptions about agents and generative AI. Thank you and all the best, David
   
   Thanks for reviewing :) 
   This draft is meant to be up for discussion so I'm happy about your feedback.
   
   You are right about the judgement part and it really comes down to what 
types of contributions we expect. I'd say modern agents are able to judge 
understanding at a high level and the aim of this instruction is to mainly 
affect low effort external contributions and contributors. In my tests an agent 
would never block requests that are specific enough (though I only tested it 
for a couple of examples). I am open to leaving this part out, but given that 
we occasionally receive submissions from inexperienced contributors, it could 
be a good tradeoff. In terms of cost, I did not run any experiment but given 
that the agent is supposed to judge from existing context, it should not create 
much additional cost (agents mainly get expensive when interacting with the 
environment) and giving project-specific instructions in this file is fairly 
standard. There are several repositories (e.g., llama.cpp) that include similar 
instructions in their AGENTS.md.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to