Removed requirements.txt reference from README file

[jafraustro]

Existing Sample Changes

Description

Removed requirements.txt reference since is an unnecessary step to run the sample in README file

Fixes Issue# Improve documentation

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Command Line
• oneapi-cli
• Visual Studio
• Eclipse IDE
• VSCode
• When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

[Ankur-singh]

A small suggestion, can't we just delete requirements.txt file in Modin_GettingStarted directory and add update the sample.json file accordingly? Please refer the sample.json file in Modin_Vs_Pandas directory.

[jafraustro]

A small suggestion, can't we just delete requirements.txt file in Modin_GettingStarted directory and add update the sample.json file accordingly? Please refer the sample.json file in Modin_Vs_Pandas directory.

Hi @Ankur-singh,

We made some corrections to our previous commit.

• Tests should be run in modin env to avoid install modin from pip
• Base environment has matplotlib, for this reason we remove the requirements.txt file but matplotlib is not present in the modin environment, so we add requirements.txt file again.

[Ankur-singh]

I mean to say, use pip install matplotlib in sample.json. This will make it explicit.

[jafraustro]

Done

[Ankur-singh]

@jafraustro requirements.txt file is no longer required, right?

[jafraustro]

Hi @Ankur-singh,
I make it explicit in the Readme file and removed the requirements.txt file.

Matplotlib is not part of the AI tool selector commands for 2025.1.0 release