memgraph / mage

MAGE - Memgraph Advanced Graph Extensions :crystal_ball:
Apache License 2.0
246 stars 24 forks source link

[main < T324] Reliable llm_util output #324

Closed antepusic closed 1 year ago

antepusic commented 1 year ago

Description

This PR addresses the issue of checks failing because the order of properties in llm_util.schema('prompt_ready') is not deterministic. The fix is to sort the properties before outputting them, and e2e tests are updated to reflect that.

Pull request type

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

######################################

imilinovic commented 1 year ago

You are missing newlines at the end of some tests, but that can be added after tests pass.

Also it should probably be added to the schema documentation that the properties are sorted now.