Sonya Nikiforova issues

Results 34 issues of


                                            Sonya Nikiforova

Use scratch space instead of tmp file

In [`test_amp_doc_formatter.py`](https://github.com/causify-ai/helpers/blob/master/linters/test/test_amp_doc_formatter.py) we are using a temporary file created with `tempfile`. Instead, we should use our [`get_scratch_space()`](https://github.com/causify-ai/helpers/blob/78529b03738a8ad9fc40eeadf87d49629764639a/helpers/hunit_test.py#L1230) and create the file there.

good first issue

Onboarding Peeyush Dyavarashetty

- Source: [`intern.onboarding_checklist.reference.md`](https://github.com/causify-ai/helpers/blob/master/docs/onboarding/intern.onboarding_checklist.reference.md) ### Org - [x] **HiringMeister**: File an issue with this checklist - The title is "Onboarding {{Name}}" - Copy-and-paste the whole checklist starting from [here](#checklist) - The...

Onboarding Shreyash Bhatkar

Sudo password requested running invoke on Linux locally

As reported by @samarth9008 in https://github.com/causify-ai/helpers/pull/385#pullrequestreview-2695820910: After setting up our dev env locally on a Linux laptop, any invoke like `i docker_bash` at some point asks for sudo password: ```...

Onboarding Neomi Sule

Unified comments in unit tests

Our [doc about writing unit tests](https://github.com/causify-ai/helpers/blob/master/docs/coding/all.write_unit_tests.how_to_guide.md) should mention that we strongly encourage the use of consistent concise comments in unit tests that establish a unified unit test structure: ``` #...

good first issue

Replace data with df in hdataframe

In [helpers/hdataframe.py](https://github.com/causify-ai/helpers/blob/master/helpers/hdataframe.py) there is a TODO: ``` # TODO(gp): data -> df ``` Let's fulfill the TODO and replace `data` with `df` in this file.

good first issue

Clean up intern fast tests failures

Interns have reported failures while running `i fast_tests`: see https://github.com/causify-ai/helpers/issues/392#issuecomment-2745413503 and below, https://github.com/causify-ai/helpers/issues/391#issuecomment-2746813006 and below. We should go over the failing tests: - If some of them are expected to...

Rename is_inside_ci to is_ci_csfy

As proposed in #401, we want to rename `hserver.is_inside_ci()` to `hserver.is_ci_csfy()`

Remove is_dev4 and is_ig_prod

Remove `is_dev4()` and `is_ig_prod()` from `helpers/hserver.py`. Make sure to remove all references to them in the codebase and check that nothing is breaking as a result. FYI @gpsaggese