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

Language InterOp Blog based on cppyy and Cling paper from 2023 #167

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

QuillPusher
Copy link
Contributor

  • High Level Blog Post to introduce to Language Interoperability, based on the following Paper:

  • "Efficient and Accurate Automatic Python Bindings with cppyy & Cling" (https://arxiv.org/pdf/2304.02712.pdf)

  • Date of the Blog is kept as the Date that the paper was published on. Later we can share this post on LinkedIn etc, using a #tbt (ThrowBack Thursday) or Feature Highlight type post

  • tested on localhost:

Blog_sample3_cppyy

- High Level Blog Post to introduce to Language Interoperability, based on the following Paper:

- "Efficient and Accurate Automatic Python Bindings with cppyy & Cling"

- https://arxiv.org/pdf/2304.02712.pdf

- Date of Blog is kept as the Date that the paper was published on. Later we can share this post on LinkedIn etc, using a  #tbt (ThrowBack Thursday) or Feature Highlight type post
@QuillPusher
Copy link
Contributor Author

Hello @maximusron
Vassil suggested that you'd be an excellent candidate to review/contribute to this blog post 🚀
Please have a look 😎

@davidlange6
Copy link
Contributor

some meta comments

  • lets keep this positive. Instead of shortcomings, we should talk about the new functionality provided by our work. Its description is missing. Probably section 2 of the paper is a good source of information.
  • The benchmarks need some more description. In particular, in the blog article the speedups up pitiful. However, they are actually really awesome because its in comparison to a numpy-enabled operation. The text in the article just above the same table is useful
  • A conclusion is probably helpful.

- more positive view of why C++ is used
- more positive language around Numba enhancements
- Added merits of using wach language (C++ and Python), while highlighting their strengths.
- Added description to Benchmarks section
- Added a Conclusion/Summary
@QuillPusher
Copy link
Contributor Author

Thanks @davidlange6 for the insights. Please see the latest commit to review the requested changes.

@QuillPusher
Copy link
Contributor Author

@maximusron I've added you as a collaborator to my fork of the website (branch: interop_blog_post) in case you would like to directly make edits to the post. Any feedback would help

@vgvassilev
Copy link
Contributor

@maximusron ping.

@aaronj0
Copy link
Collaborator

aaronj0 commented May 21, 2024

@QuillPusher I have made some edits, can you send me the invite again?

@QuillPusher
Copy link
Contributor Author

@QuillPusher I have made some edits, can you send me the invite again?

@maximusron Invite sent 👍
let me know if it works

- Incorporated Vassil's Review changes

- Minor grammar and formatting changes as well
@QuillPusher
Copy link
Contributor Author

Thanks @maximusron for the changes.

@vgvassilev requested changes have been made. Please see if this looks good overall
CC: @davidlange6

@QuillPusher
Copy link
Contributor Author

Hello @davidlange6
Did you get a chance to have a look at this? It is in much better shape than before 😅

@davidlange6
Copy link
Contributor

i did, but unfortunately the text is now very circular and difficult to understand what was accomplished or enabled. I do not yet have a clear idea of how to progress with it. But I have three suggestions

  • add a simple but real example of code that is now supported that wasn't before
  • decide if the article is about adding numba support to cppyy or adding cppyy support to numba
  • avoid meaningless statements like "advanced C++ features" in favor of specific ones like "you can now call a c++ function in python loops that numba can interpret"

@QuillPusher
Copy link
Contributor Author

Thanks @davidlange6 , I'll look at this with fresh eyes to review the flow

@QuillPusher
Copy link
Contributor Author

@maximusron , can you spot a suitable code example:

add a simple but real example of code that is now supported that wasn't before

P.s., @maximusron does the example presented in the paper qualify, or have you come across a better option?

code

@aaronj0
Copy link
Collaborator

aaronj0 commented Aug 2, 2024

@QuillPusher Yes this example should work, it is the more simple example

- Added the Code example suggested by Aaron
- Focused on Achievements of the research
- Removed verbose sections about C++/Python
- Revamped the document structure with clear headings, for better readability
@QuillPusher
Copy link
Contributor Author

Thanks @aaronj0 😎

Hello @davidlange6
Kindly review the following changes:

  • Added the Code example suggested by Aaron
  • Focused more on Achievements of the research
  • Removed verbose sections about C++/Python
  • Revamped the document structure with clear headings for better readability

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants