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.

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

[BLOG] Add python-moa-introduction #633

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

[BLOG] Add python-moa-introduction #633

wants to merge 7 commits into from
+234 −0

Conversation

MarsBarLee
Copy link
Contributor

@MarsBarLee MarsBarLee commented Jan 23, 2023 •
edited
Loading

Link to blog post, successful build.

Links to other Quansight blogs:

There is no need to check for Blog2Blog links due to updated website build.

Text styling

  • The blog is written with plain language (where relevant).
  • If there are headers, they use the proper header tags in order to do so (with only one level-one header).
  • All links describe where they link to (for example, check the Quansight labs website).
  • Any kind of styling that the author uses (for example, bold for emphasis) is consistent throughout the blog.

Non-text contents

  • All content is represented as text (for example, images need alt text and videos need captions or descriptive transcripts).
  • If there are emojis, there are not more than three in a row.
  • Don't use flashing gifs or videos.
  • If it were to be read as plain text, the blog still makes sense and no information is missing.

@MarsBarLee MarsBarLee added labs 🔭 Items related to the Labs website type: content 📝 labels Jan 23, 2023
@vercel
Copy link

vercel bot commented Jan 23, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
consulting ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 14, 2023 at 5:11PM (UTC)
labs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 14, 2023 at 5:11PM (UTC)

@MarsBarLee
Copy link
Contributor Author

This is one the ipynb-to-md conversion. There's about 10 more to post.

@MarsBarLee MarsBarLee marked this pull request as ready for review February 1, 2023 19:28
@gabalafou
Copy link
Contributor

I changed the feature image for variation:

gabalafou
gabalafou requested changes Feb 14, 2023
View reviewed changes
apps/labs/posts/python-moa-introduction.md
Comment on lines +138 to +161
@numba.jit
def f(A, B):


n = A.shape[0]

m = A.shape[1]

k = B.shape[0]

l = B.shape[1]

_a21 = numpy.zeros(())

_a19 = numpy.zeros(())

_a21 = 0

for _i10 in range(0, m, 1):

_a21 = (_a21 + (A[(0, _i10)] + B[(0, _i10)]))

_a19[()] = _a21
return _a19
Copy link
Contributor

Choose a reason for hiding this comment

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

Indented code does not work in MDX.

Also, this looks like output to me, so we have the same issue here that we have described before:

@MarsBarLee MarsBarLee mentioned this pull request Mar 1, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabalafou gabalafou gabalafou requested changes

@trallard trallard Awaiting requested review from trallard trallard is a code owner

@rgommers rgommers Awaiting requested review from rgommers rgommers is a code owner

@pavithraes pavithraes Awaiting requested review from pavithraes pavithraes is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
labs 🔭 Items related to the Labs website type: content 📝
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MarsBarLee @gabalafou