modin icon indicating copy to clipboard operation
modin copied to clipboard

FEAT-#5394: Reduce amount of remote calls for Map operator

Open Retribution98 opened this issue 1 year ago • 2 comments

What do these changes do?

  • [x] first commit message and PR title follow format outlined here

    NOTE: If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

  • [x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
  • [x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
  • [x] signed commit with git commit -s
  • [x] Resolves #5394
  • [x] tests added and passing
  • [x] module layout described at docs/development/architecture.rst is up-to-date

Retribution98 avatar Mar 28 '24 13:03 Retribution98

Is this PR ready for review?

YarShev avatar Apr 10 '24 11:04 YarShev

@Retribution98 you need to change branch to main

anmyachev avatar Apr 18 '24 12:04 anmyachev

@Retribution98 I see your graphs above, but I don’t really understand what the axes mean. Please label them.

anmyachev avatar May 02 '24 10:05 anmyachev

@Retribution98 I see your graphs above, but I don’t really understand what the axes mean. Please label them.

@anmyachev Thanks, updated it.

Retribution98 avatar May 02 '24 16:05 Retribution98

@YarShev any more comments?

anmyachev avatar May 03 '24 10:05 anmyachev