fix(dataset): push dataset with name

[ArslanSaleem]

Important

Enhance push() in base.py to include dataset name in parameters and ensure dataset is saved before pushing.

• Behavior:
  • In push() method, raise ValueError if self.path is None, ensuring dataset is saved before pushing.
  • Add name to params in push() method, using self.name or empty string if None.

^{This description was created by for d8e764d. It will automatically update as commits are pushed.}

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to d8e764d in 34 seconds

More details

• Looked at 22 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. pandasai/dataframe/base.py:239

• Draft comment:
  Consider suggesting the user to use the save method in the error message for clarity.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR adds a check to ensure that the dataset has been saved before it can be pushed. This is a good addition for error handling. However, the error message could be more informative by suggesting the user to use the save method.

Workflow ID: wflow_d4Mzgpfua01Rrpeb

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.