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

clib.Session: Remove deprecated virtualfile_from_data method, use virtualfile_in instead (Deprecated since v0.13.0) #3739

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

seisman
Copy link
Member

@seisman seisman commented Jan 1, 2025

Session.virtualfile_from_data was deprecated in #3225.

@seisman seisman added deprecation Deprecating a feature needs review This PR has higher priority and needs review. labels Jan 1, 2025
@seisman seisman added this to the 0.15.0 milestone Jan 1, 2025
@@ -128,35 +128,3 @@ def test_virtualfile_in_matrix_string_dtype():
assert output == "347.5 348.5 -30.5 -30\n"
# Should check that lib.virtualfile_from_vectors is called once,
# not lib.virtualfile_from_matrix, but it's technically complicated.


# TODO(PyGMT>=0.16.0): Remove this test in PyGMT v0.16.0 in which the old usage of
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a typo in #3722. This test should be removed in PyGMT v0.15.0, not PyGMT v0.16.0.

@michaelgrund michaelgrund added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Jan 2, 2025
@weiji14 weiji14 changed the title clib.Session: Remove the deprecated virtualfile_from_data method (Deprecated since v0.13.0) clib.Session: Remove deprecated virtualfile_from_data method, use virtualfile_in instead (Deprecated since v0.13.0) Jan 2, 2025
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Jan 3, 2025
@seisman seisman merged commit 4d60eac into main Jan 3, 2025
23 checks passed
@seisman seisman deleted the deprecate/virtualfile_from_data branch January 3, 2025 00:31
@seisman seisman mentioned this pull request Jan 9, 2025
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecation Deprecating a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants