Skip to content

APPENG-3315 Allow the user to insert custom Amazon dataset#12

Merged
fax4ever merged 4 commits into
RHEcosystemAppEng:mainfrom
fax4ever:APPENG-3315
Jul 16, 2025
Merged

APPENG-3315 Allow the user to insert custom Amazon dataset#12
fax4ever merged 4 commits into
RHEcosystemAppEng:mainfrom
fax4ever:APPENG-3315

Conversation

@fax4ever
Copy link
Copy Markdown
Contributor

@fax4ever fax4ever requested a review from itay1551 July 15, 2025 09:43
Comment thread service/dataset_provider.py Outdated
import pandas as pd
from feast import FeatureStore

from models import data_util
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.

This will do problems on the backend which copy into the public/, need to add to the copy the right path

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right! Good catch.
We copy the models and service directory on on /* in the workflow container and on /public/* in the backend container...

Ideally I think that eventually we should try to have the same scaffolding.

@fax4ever
Copy link
Copy Markdown
Contributor Author

Thanks @itay1551 for the review. I added a commit with the workaround for the import.

@fax4ever
Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
Collaborator

@itay1551 itay1551 left a comment

Choose a reason for hiding this comment

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

Great work!!!
Thank you @fax4ever

@fax4ever fax4ever merged commit e1eac24 into RHEcosystemAppEng:main Jul 16, 2025
1 check passed
@fax4ever
Copy link
Copy Markdown
Contributor Author

Merged thanks!

@fax4ever fax4ever deleted the APPENG-3315 branch July 16, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants