8000 Drop console application by filmor · Pull Request #2456 · pythonnet/pythonnet · GitHub
[go: up one dir, main page]
Skip to content

Drop console application #2456

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 1, 2024
Merged

Drop console application #2456

merged 1 commit into from
Oct 1, 2024

Conversation

filmor
Copy link
Member
@filmor filmor commented Sep 19, 2024

I don't really see the point of this one. We could keep it as an example, but
apparently it is even buggy (#2348, probably due to mixing the embedding modes).

@lostmsu
Copy link
Member
lostmsu commented Sep 19, 2024

I know we aren't supporting it, but:

  1. What do you mean by "mixing the embedding modes"?
  2. If a simple sample like this one has problems, are we not on fire?

@filmor
Copy link
Member Author
filmor commented Sep 19, 2024

It uses Py_Main which no one else uses. And if you would like to run .NET code from within Python, using the clr_loader path is the far more stable option.

@filmor filmor merged commit de7a1d2 into master Oct 1, 2024
29 checks passed
@filmor filmor deleted the drop-console branch October 1, 2024 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lostmsu lostmsu lostmsu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@filmor @lostmsu
0