Skip to content

Conversation

@chenemil
Copy link
Collaborator

updated with Jordan's hard tasks

workarenax.erpnext.create_payment_entry_without_cost_center.154,workarenax,erpnext,create-payment-entry-without-cost-center,test
workarenax.erpnext.create_payment_entry_without_cost_center.155,workarenax,erpnext,create-payment-entry-without-cost-center,test
workarenax.erpnext.create_payment_entry_with_cost_center_cheque_date_reference.156,workarenax,erpnext,create-payment-entry-with-cost-center-cheque-date-reference,test
workarenax.erpnext.create_payment_entry_with_cost_center_cheque_date_reference.157,workarenax,erpnext,create-payment-entry-with-cost-center-cheque-date-reference,test
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would have thought you just appended new tasks to the end, but I see ids have changed: create-payment-entry-with-cost-center-cheque-date-reference used to be 156 and 157 but now they are 142 and 143. How come?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ah i was grouping similar tasks together (so all the create_user tasks follow one another, etc.) but i can modify to just append if we need to maintain the same ids from previous runs

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that would be better, so every time we add new tasks, we are just adding. Otherwise, it may seem as we are replacing tasks

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

makes sense, i'll update the other PR with the jsonl so it also appends

Copy link
Collaborator

@marquezo marquezo left a comment

Choose a reason for hiding this comment

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

Thanks!

@chenemil chenemil merged commit 3639cb6 into scratch/workarenax_benchmarks Jan 26, 2026
11 of 13 checks passed
@chenemil chenemil deleted the scratch/update-workarenax-csv-with-hard-tasks branch January 26, 2026 16:56
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