feat: adding finemapping method to studylocusid hash

[Daniel-Considine]

✨ Context

Currently studyLocusId column is generated from studyId and variantId. This isn't a unique identifier for cases where the same top variant from the same study is fine-mapped with multiple methods (pics. susie, SuSiE-inf)

🛠 What does this PR implement

Includes the finemappingMethod column into the hash generation

🙈 Missing

🚦 Before submitting

• Do these changes cover one single feature (one change at a time)?
• Did you read the contributor guideline?
• Did you make sure to update the documentation with your changes?
• Did you make sure there is no commented out code in this PR?
• Did you follow conventional commits standards in PR title and commit messages?
• Did you make sure the branch is up-to-date with the dev branch?
• Did you write any new necessary tests?
• Did you make sure the changes pass local tests (make test)?
• Did you make sure the changes pass pre-commit rules (e.g poetry run pre-commit run --all-files)?

[project-defiant]

Please check the comments.

[project-defiant]

@Daniel-Considine can you just add the same logic to the of the StudyLocusId recalculation with finemappingMethod column to the pics step?

gentropy/src/gentropy/method/pics.py

Line 256 in bb8558c

)

[project-defiant]

@DSuveges @Daniel-Considine let's keep the updated cs StudyLocusId as @Daniel-Considine developed it for now as I need it for the downstream testing of the credible set without rerunning the dags. In the meantime I will prepare the version with md5 hash instead.