Skip to content
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

[Task]: Use DataFrame.map instead of DataFrame.applymap #33441

Closed
1 of 17 tasks
liferoad opened this issue Dec 23, 2024 · 5 comments
Closed
1 of 17 tasks

[Task]: Use DataFrame.map instead of DataFrame.applymap #33441

liferoad opened this issue Dec 23, 2024 · 5 comments

Comments

@liferoad
Copy link
Contributor

What needs to happen?

Fix this warning:

/jupyter/.kernels/apache-beam-2.61.0/lib/python3.10/site-packages/apache_beam/runners/interactive/display/pcoll_visualization.py:423: FutureWarning: DataFrame.applymap has been deprecated. Use DataFrame.map instead.
  rows = data.applymap(lambda x: str(x)).to_dict('split')['data']

Issue Priority

Priority: 2 (default / most normal work should be filed as P2)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam YAML
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Infrastructure
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner
@karangarg218
Copy link

Hi Can you assign this task to me. I have checked that this is in python sdk this is the file with warnings.catch_warnings():
# TODO(yathu) switch to use DataFrame.map when dropped pandas<2.1 support
warnings.filterwarnings(
"ignore", message="DataFrame.applymap has been deprecated")
data = data.applymap(lambda x: str(x) if isinstance(x, dict) else x)

@liferoad
Copy link
Contributor Author

Thanks!

@mohamedawnallah
Copy link
Contributor

Hi @karangarg218, are you currently working on this issue? I’d like to take it on otherwise. 🙏

cc: @liferoad

@mohamedawnallah
Copy link
Contributor

.take-issue

@mohamedawnallah
Copy link
Contributor

mohamedawnallah commented Jan 15, 2025

Hi @liferoad, It seems I used addresses instead of fixes in the current merged PR, so this issue won't be closed automatically. 🙏

@github-actions github-actions bot added this to the 2.63.0 Release milestone Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants