-
Notifications
You must be signed in to change notification settings - Fork 651
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
REFACTOR: Add mypy checks for modin.core.dataframe.algebra #5073
Comments
Hello, @mvashishtha I'm so interested in contributing for this issue, and I just want to know the problem statement or what should I do to solve this issue ?? Thanks in advance |
Hi @mohanadtalat91, thank you very much for your interest. I want @sofezticated-gentleman to take this issue because I pointed it to him first in #4633. However, it would be great if you could take issue #5079 instead. Both issues are for adding mypy types. mypy is a type checker. Integrating it into the Modin codebase will make it easier to understand the Modin code and to write bug-free code. You can read more about mypy in its documentation. For #5079, the goal is to add types to all the files in modin/core/dataframe/base, so that mypy is satisfied with the types in all files in that directory. You can make a PR to fix the issue by following the steps in the first post of this issue, except for modin/core/dataframe/base instead of modin/core/dataframe/algebra. Please let me know if you have more questions. |
@mvashishtha after following the contributing guidelines, I have cloned the repository but I am stuck at adding a remote branch |
@sofezticated-gentleman you have to go into the cloned directory, e.g. if you cloned the repo into directory cd modin Then you should be able to add the remote. |
ooh thanks, I think so that should be added to contributing document |
@mvashishtha , I'm getting error : |
@mvashishtha also getting the same error for |
|
@mvashishtha also is there any discord channel or some forum of this org?? |
Let's continue this conversation in the Modin slack either on a new post in #modin-development or in DMs-- my name on slack is "Mahesh Vashishtha". For the ray error, could you please try to list the available versions of ray by following the instructions here? I'm surprised that a version of ray >= 1.4.0 is not available on your system. |
hey @mvashishtha i think so ray isnt available for windows sys but we can install it with help of anaconda and Docker |
@sofezticated-gentleman my impression is that ray is available on windows. Could you share some details of your environment, including operating system version, python version, conda version, pip version? Do you need any more help getting your development environment set up? |
@sofezticated-gentleman I'd encourage you to go ahead and join the Modin slack so that more folks can help you get onboarded with this. Thanks for taking on this issue! |
procedure:
modin/core/dataframe/algebra
to mypy.inimypy --config-file mypy.ini modin/core/dataframe/algebra
until it passesFor example, this line:
modin/modin/core/dataframe/algebra/binary.py
Line 26 in 3ee4fa0
should become something like
def register(cls, func: Callable, join_type: str ="outer", labels: str ="replace"):
The text was updated successfully, but these errors were encountered: