Deprecate DataFrameMixin

[adamamer20]

The DataFrameMixin was originally introduced in the space module to share functionalities between GridPandas and GridPolars. Since the pandas implementation has now been deprecated, this mixin is no longer necessary.

Deprecating the DataFrameMixin will simplify the codebase and make the Polars implementation cleaner. Additionally, it will make future refactoring and the transition to Polars LazyFrames easier.

Tasks:

• Identify and remove all usages of DataFrameMixin within the space module.
• Refactor the Polars-related implementations accordingly.
• Ensure existing functionality remains intact by updating unit tests.

[reyan-singh]

@adamamer20 @EwoutH I would like to work on this, can you please assign

[adamamer20]

@reyan-singh you can start working on it in its own PR.

[suryanshgargbpgc]

@adamamer20, I was working on this, at first I thought of removing the DataFrameMixin class itself, but AgentSetDF which seems to inherit it so I think methods should be changed there too however it is not a part of space module so should we change methods in agent module too. Or keep the class and just change the methods in the space module only.
But if we keep DataFrameMixin class so that it can be used in agent module then are not we just still keeping it so it wont simplify the code so should we extend the changes outside the space module.

I am a bit confused about this, please guide me on how should I approach this issue

[suryanshgargbpgc]

I am a bit confused about this, please guide me on how should I approach this issue

hi @adamamer20 looking for your help here : )

[adamamer20]

hi @adamamer20 looking for your help here : )

The original purpose of DataFrameMixin was to serve as a common interface for dataframe operations across two backends (pandas and polars). This allowed us to avoid duplicating logic in classes like GridDF. Now that we've fully adopted Polars, DataFrameMixin adds unnecessary complexity and isn't justified anymore.

Thus, we should:

• Deprecate DataFrameMixin.
• Keep GridDF as an abstract interface.
• Refactor GridPolars to directly use Polars operations without relying on DataFrameMixin.
• Additionally, since DataFrameMixin is also used in AgentSetDF.remove and AgentSetDF.pos, we should remove and refactor it from these methods as well.

I hope this makes the direction clear!

[suryanshgargbpgc]

• Additionally, since DataFrameMixin is also used in AgentSetDF.remove and AgentSetDF.pos, we should remove and refactor it from these methods as well.

Oh okay, If I am correct thats a part of Agent module, so we need to refactor beyond the space module too, thanks for helping I will work on this.