fix ingestion

[emrgnt-cmplxty]

Important

Fixes current_usage initialization in vector_storage_pipe.py to ensure correct usage calculation per message.

• Behavior:
  • Fixes current_usage initialization in _run_logic() in vector_storage_pipe.py to occur inside the loop, ensuring correct usage calculation per message.
  • Assumes only one owner per message for current_usage calculation.
• Logging:
  • Logs error if user exceeds maximum allowed chunks in _run_logic() in vector_storage_pipe.py.

^{This description was created by for 6092ad8. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 6092ad8 in 9 seconds

More details

• Looked at 29 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. py/core/pipes/ingestion/vector_storage_pipe.py:68

• Draft comment:
  The current_usage check and assignment can be moved outside the loop since it only needs to be set once per message batch. This will make the code cleaner and more efficient.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The current_usage variable is being set inside the loop but only once, which is correct given the assumption of one owner per message. However, the logic for checking and setting current_usage can be optimized.

Workflow ID: wflow_yz5BYtCcyXlFKRsy

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.