Skip to content

fix: installation bug due to missing domain files#525

Merged
gcroci2 merged 4 commits intomainfrom
hotfix_gcroci2
Nov 17, 2023
Merged

fix: installation bug due to missing domain files#525
gcroci2 merged 4 commits intomainfrom
hotfix_gcroci2

Conversation

@gcroci2
Copy link
Copy Markdown
Collaborator

@gcroci2 gcroci2 commented Nov 14, 2023

Now the relevant files are included in the installation. Fixes #524

@gcroci2 gcroci2 requested a review from DaniBodor November 14, 2023 16:05
@gcroci2 gcroci2 self-assigned this Nov 14, 2023
Copy link
Copy Markdown
Collaborator

@DaniBodor DaniBodor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for some reason the last few lines of some of the files was duplicated. Can you remove those before merging, please.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't every folder supposed to have an init.py file? Not sure why, but I thought that was the case.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the official documentation, it seems that it's not needed.

@gcroci2
Copy link
Copy Markdown
Collaborator Author

gcroci2 commented Nov 16, 2023

for some reason the last few lines of some of the files was duplicated. Can you remove those before merging, please.

Thanks for noticing, no clue about what happened. It's fixed now :)

@gcroci2 gcroci2 merged commit ca9dd46 into main Nov 17, 2023
@gcroci2 gcroci2 deleted the hotfix_gcroci2 branch November 17, 2023 09:05
@gcroci2 gcroci2 added the JOSS label Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Tutorial failing when running deeprank2 as Dockerfile

2 participants