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

Add Noir #6432

Merged
merged 32 commits into from
Aug 29, 2024
Merged

Add Noir #6432

merged 32 commits into from
Aug 29, 2024

Conversation

critesjosh
Copy link
Contributor

@critesjosh critesjosh commented May 25, 2023

Description

Adding the Noir programming language.

Checklist:

@critesjosh critesjosh marked this pull request as ready for review May 25, 2023 20:53
@critesjosh
Copy link
Contributor Author

critesjosh commented May 26, 2023

I'm not sure what is going on with Github search, but there are fewer files found today than yesterday. There should be more--I pushed a public repo and I know for a fact at least 3 other people did as well.

lib/linguist/languages.yml Outdated Show resolved Hide resolved
vendor/grammars/vscode-noir Outdated Show resolved Hide resolved
lib/linguist/heuristics.yml Outdated Show resolved Hide resolved
@critesjosh critesjosh requested a review from a team as a code owner May 30, 2023 16:07
@critesjosh
Copy link
Contributor Author

@lildude Do you need anything else from me on this?

I am wondering how to best track popularity. When I first drafted this search query, there were 376 files. Now its only showing 358. I know more people have been pushing public repos, so it doesn't seem to be picking up on them for some reason.

lildude
lildude previously requested changes Jun 1, 2023
Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

I think you might have put the samples/Noir/roff.nr sample in the wrong directory or accidentally added it as it looks more like Roff than Noir to my naïve eye.

The same applies for the samples/Noir/roff1.nr sample which is also massive and waaay bigger than we need. ProTip: if the diff suppresses the content, the file is too big.

@lildude
Copy link
Member

lildude commented Jun 1, 2023

I am wondering how to best track popularity. When I first drafted this search query, there were 376 files. Now its only showing 358. I know more people have been pushing public repos, so it doesn't seem to be picking up on them for some reason.

There's only search, which is expected to fluctuate as people add and remove repos and files. This is only really noticeable as there are so few files. When things reach close to our requirements, a deviation of 18 files is trivial and not really an issue.

@critesjosh
Copy link
Contributor Author

critesjosh commented Jun 1, 2023

I think you might have put the samples/Noir/roff.nr sample in the wrong directory or accidentally added it as it looks more like Roff than Noir to my naïve eye.

The same applies for the samples/Noir/roff1.nr sample which is also massive and waaay bigger than we need. ProTip: if the diff suppresses the content, the file is too big.

Do you need examples of Roff, since it has the same extension as Noir? Should I add them to samples/Roff/roff.nr? Or since there is already a samples/Roff/trekmanual.nr, can I delete them?

@critesjosh
Copy link
Contributor Author

I am wondering how to best track popularity. When I first drafted this search query, there were 376 files. Now its only showing 358. I know more people have been pushing public repos, so it doesn't seem to be picking up on them for some reason.

There's only search, which is expected to fluctuate as people add and remove repos and files. This is only really noticeable as there are so few files. When things reach close to our requirements, a deviation of 18 files is trivial and not really an issue.

Just curious, how do you assess when things reach close to your requirements, since the search doesn't return the number of repos?

@lildude
Copy link
Member

lildude commented Jun 1, 2023

Do you need examples of Roff, since it has the same extension as Noir? Should I add them to samples/Roff/roff.nr? Or since there is already a samples/Roff/trekmanual.nr, can I delete them?

Noir and Roff are very different so the classifier shouldn't have problems distinguishing between the two. That said, there's no harm adding one (the smaller) to the Roff samples directory. As we'll then have two for each language, the "Classifier cross-validation (pull_request)" test will then start to test the classifier for the .nr extension.

Just curious, how do you assess when things reach close to your requirements, since the search doesn't return the number of repos?

See #5756 (this is referenced in the docs too). I'm slowly reassessing this now the new search is open for all and we can get better results. When I've come up with a good reliable strategy, I'll update the docs and close that issue.

@critesjosh
Copy link
Contributor Author

Thanks for your help with this! Let me know if I need to make any more updates.

@TomAFrench
Copy link

The submodule could probably do with a bump btw @critesjosh. We've made some changes to the highlighting recently.

@signorecello
Copy link

@lildude how are we doing with this integration? There are more and more people exploring zero-knowledge proofs so it would be really cool to have support 😊

@lildude
Copy link
Member

lildude commented Mar 1, 2024

Still got failing tests. Run this cmd to fix. I'll re-evaluate popularity when I come to make the next release which will in the next two weeks, assuming the failures have been addressed by then.

@critesjosh
Copy link
Contributor Author

Thanks! Just ran the command and pushed changes

@lildude
Copy link
Member

lildude commented Mar 12, 2024

Things don't look popular enough once we exclude the AztecProtocol user that is responsible for nearly 650 of the found files.

@critesjosh
Copy link
Contributor Author

Huh, I only see a difference of ~200 files (1.9k vs 1.7k), but I guess we can wait a few more weeks. 🙂

image

@signorecello
Copy link

Hitting 1.8k again, when is a new release planned @lildude ?

@lildude
Copy link
Member

lildude commented Apr 9, 2024

Same as usual... approx 3-4 months after the last one.

@Savio-Sou
Copy link

1.8k now (with both AztecProtocol and noir-lang excluded):

image

@lildude lildude dismissed stale reviews from Alhadis and themself July 10, 2024 09:03

Requested changes applied. Pending popularity now.

@Savio-Sou
Copy link

2.1k 🚀 @lildude curious if that's sufficient popularity?
image

@aguzmant103
Copy link

@lildude I vouch this PR as a user :)
(don't know if it makes a difference but would appreciate this, lol)

@Savio-Sou
Copy link

Savio-Sou commented Aug 20, 2024

@lildude / @Alhadis wonder if we're all set to remove the Pending Popularity label + resolve pending reviews?

lib/linguist/heuristics.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Alhadis Alhadis left a comment

Choose a reason for hiding this comment

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

LGTM. 👍

@Savio-Sou
Copy link

Thanks! @lildude would be grateful for your re-review.

@lildude lildude added this pull request to the merge queue Aug 29, 2024
Merged via the queue into github-linguist:master with commit 34ae2f4 Aug 29, 2024
5 checks passed
@Savio-Sou
Copy link

Thank you all on behalf of the Noir community for the help along the way ❤️

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.

9 participants