Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Technical debt: Reference path not used in Crunchy #4263

Open
islean opened this issue Mar 4, 2025 · 0 comments
Open

Technical debt: Reference path not used in Crunchy #4263

islean opened this issue Mar 4, 2025 · 0 comments
Labels
technical-debt Complicating development in the repo

Comments

@islean
Copy link
Contributor

islean commented Mar 4, 2025

🧩 Technical Debt Report

Description

The init of the CrunchyAPI specifies a reference_path, which is also specified in the cg.yaml. As mentioned by SD, this parameter is not used anywhere and should be removed, however.


Location

  • File(s):
    • cg/apps/crunchy/crunchy.py
    • [cg.yaml](https://github.com/Clinical-Genomics/servers/blob/299dd00a0422bc101278968dbcf8c4f5c8a81238/config/hasta.scilifelab.se/cg.yaml#L288)
  • Module/Component:
    • e.g., CrunchyAPI

Impact

  • Performance: Is it causing delays or inefficiency?
  • Maintainability: Is it hard to update or debug?
  • Scalability: Does it hinder growth or adding new features?
  • Other: Any other impacts?

Only confusion and error prone.


Proposed Resolution

  • Refactoring: Just delete the lines

Estimated Effort

  • Effort Level: XS
  • Time Estimate: <1 hour

@islean islean added the technical-debt Complicating development in the repo label Mar 4, 2025
@islean islean mentioned this issue Mar 4, 2025
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical-debt Complicating development in the repo
Projects
None yet
Development

No branches or pull requests

1 participant