aangelo9

Results 50 comments of aangelo9

Run_linter and run_fast_tests failed. ## For Run_linter: Error states: 1. Unpacking a string is disallowed > helpers/hdataframe.py:111: error: Unpacking a string is disallowed [misc] [mypy] In hdataframe.py, Refers to line:...

Should I proceed with the test fixes?

All tests are green. Awaiting review.

@Shaunak01 I have requested access for the [gdoc](https://docs.google.com/document/d/1cs50rwqxhRzQLt41brcPoQs5xL0QYOEFDcKHJB0NUo8/edit?tab=t.0#heading=h.rv0yjq809l2h).

@sonniki Yes, I do have access now. Thank you.

Approach to experiment with `GitHub Copilot Code Review`. 1. Enable GitHub Copilot and its code review settings 2. Load Markdown Guidelines into Copilot Code Review - Navigate to Copilot Code...

Understood, I will avoid the try-install approach. The [code](https://github.com/causify-ai/helpers/blob/5da1ffe71ca2fa0116f0ef229d51213be9333b21/dev_scripts_helpers/documentation/generate_readme_index.py) currently has a pseudo mock which accepts a `placeholder` argument to skip openai API calls. However, since `hopenai.py` is always imported,...

Understood, `importorskip` will be used for now. Are Interns allowed to work on/contribute to building the infra?

After some investigation: - In the old version of `lint_notes.py`, the `--in_place` argument explicitly controlled whether the input file was overwritten with the linted output. - In the current version,...

`import openai` raised an error 'No Module Found', which means that the docker container do not have openai installed. I also do not have the authority to install inside docker....