fix(examples): add missing BaseDataSource import to RFNet/ERFNet sedna_predict.py files#539
fix(examples): add missing BaseDataSource import to RFNet/ERFNet sedna_predict.py files#539Aamod-Dev wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates multiple sedna_predict.py files across different benchmark examples to import BaseDataSource alongside IndexDataParse from sedna.datasources. There are no review comments provided, so I have no additional feedback to offer.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
/assign @MooreZheng |
…a_predict.py files Signed-off-by: Aamod007 <aamodkumar2006@gmail.com>
a6ca60a to
11b6194
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Aamod007 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/assign @jaypume |
What type of PR is this?
/kind bug
What this PR does / why we need it:
This PR adds the missing
BaseDataSourceimport to multiplesedna_predict.pyfiles in the RFNet/ERFNet examples.Previously, these files instantiated
BaseDataSource(data_type="test")in thepre_data_process()method but only importedIndexDataParsefromsedna.datasources. If this execution path is hit without the import, it results in aNameError: name 'BaseDataSource' is not defined.This commit adds
BaseDataSourceto the existingsedna.datasourcesimport statement across the 5 affected files.Which issue(s) this PR fixes:
Fixes #522
Special notes for your reviewer:
N/A