Show detailed message why password is weak to user.

The goal of the PR is to show why the password is not valid to the user rather than showing Password is too weak. message. Here are some screenshots

There are two factors used for determining whether a password is valid.

Password length
Password quality

The password length should be greater than settings.PASSWORD_MIN_LENGTH and quality should be greater than settings.PASSWORD_MIN_ZXCVBN_QUALITY.
If these values are not specified by the organization admin the default value of 6 would be used for length and 0 for quality. This means that if these values are not specified the only factor which is used for checking whether the password is valid or not is the length of the password as the quality of the password is always greater than 0. So the user's password is accepted if it's greater than or equal to 6 characters and the only error message he will get is this (if the password length is less than 6).

Choosing a value for Password quality is kind of a tedious task for admins who don't have much time to research on what number it should be. This is how we caluclate the password quality
 var quality = Math.min(1,Math.log(1 + result.crack_times_seconds.
                                          offline_slow_hashing_1e4_per_second) / 22);

Instead of using this I propose to use the result.score(Integer from 0-4 provided by zxcvbn's for determining the password strength, Also pointed out by @rht).
0 # too guessable: risky password. (guesses < 10^3)
1 # very guessable: protection from throttled online attacks. (guesses < 10^6)
2 # somewhat guessable: protection from unthrottled online attacks. (guesses < 10^8)
3 # safely unguessable: moderate protection from offline slow-hash scenario. (guesses < 10^10)
4 # very unguessable: strong protection from offline slow-hash scenario. (guesses >= 10^10)
The admins can choose one of them and we can keep maybe 2 or 3 as the default. I have not implemented this part in the PR. What do you folks think?
Addresses #5052
@rishig @showell @gnprice  @adnrs96
For seeing error messages other than length you can set PASSWORD_MIN_ZXCVBN_QUALITY = 0.1(or some similar value) in settings.

Codecov Report

Merging #5500 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5500      +/-   ##
==========================================
- Coverage   86.65%   86.65%   -0.01%
==========================================
  Files         431      431
  Lines       53508    53541      +33
  Branches     2086     2092       +6
==========================================
+ Hits        46369    46397      +28
- Misses       7139     7144       +5

Impacted Files
Coverage Δ

static/js/common.js
100% <100%> (ø)
:arrow_up:

frontend_tests/node_tests/password.js
100% <100%> (ø)
:arrow_up:

static/js/popovers.js
10.52% <0%> (-0.02%)
:arrow_down:

frontend_tests/node_tests/hotkey.js
99.01% <0%> (ø)
:arrow_up:

zerver/lib/bugdown/__init__.py
89.44% <0%> (+0.23%)
:arrow_up:

static/js/hotkey.js
59.89% <0%> (+0.31%)
:arrow_up:

zerver/lib/str_utils.py
87.87% <0%> (+6.06%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a45c224...6e00239. Read the comment docs.

cool, thanks @hackerkid!
Currently, the min zxcvbn strength (and the min password length) is a server setting. Making it a realm setting is a totally reasonable future project, though probably not the highest priority at the moment.
Two minor comments:

atleast -> at least
We should make this work with i18n (both the min-length string, and the zxcvbn strings)

@rishig What do you think of using  result.score instead of our custom written quality function?

Currently, the min zxcvbn strength (and the min password length) is a server setting.

Oops. I should have written server admin instead of organization admin in the comment.

yeah, I think using result.score is better than using our custom quality function. But it is a somewhat big change, probably not worth making at the moment? (In particular, it'll be a complicated migration from people's current setting to the new setting). I think we should just make that change when we move from this being a server setting to it being a realm setting.

@rishig Updated :)

hm, is there some setting I have to change in the dev environment? I'm getting the following:

@rishig I think it is because you are missing Include translations bundle in register.html. commit. The error was caused because i18n was missing in register.html. You should not get the error after you rebase. I added that commit a a bit later.

Great, thanks! Got it working :).
Some thoughts: the user experience here still isn't feeling great, for some reason. Maybe:

Only start showing the error messages once the user defocuses for the first time
We need a message for when it is red and there are no other messages. Like "Password is too weak".
It is a bit confusing when you click register and nothing happens, since the error message is already there. Would it be hard to blink the error message when you click register?

What do you think?

@rishig Sounds good. Working on it :)

@rishig Done! Take a look when you are free.

Thanks @hackerkid! Left a few comments. Sorry for the extended back and forth; this ended up more involved than expected.

we should remove the || min_length here.

The change password form inpassword_warning don't have data-min-length and data-min-quality attributes. So we can't remove this from password_qualitynow without adding them first. The account_settings.handlebar which has the password change form is rendered client side. So we have to pass the value of min length and quality in page params to set the data-min-length and data-min-quality attributes and then we can remove them.
But I can remove the default value of min_length from password_warning function as we are not showing why the password id weak in password_warning. You made the comment also in this function but I thought it would be nice to remove them togather.

Also, in javascript, does the var have to be defined outside the if block?

I fixed that too. I didn't mean to commit those changes. Sorry.

@rishig Should I make any more changes here?

@r0fls Can you take a look at this. I have updated the PR.

@hackerkid I think you meant @rishig 😄  however, thanks for roping me in! zxcvbn looks awesome and I like the configurable levels of security.

@r0fls Oops. I meant @rishig. But thanks for the review :)

This looks great, merged, thanks @hackerkid!
A good follow-up project would be to make zulip's password change user experience nice.  I think we want to move it into a modal to simplify the main "your settings" page (similar to how we do email change), and then have the same nice responsive zxcvbn experience when actually changing it.


Recent change in semantics of version conflict resolution

Hi,
When updating to a newer version of smithy-language-server that uses maven-resolver(aether) instead of coursier for dependency resolution (introduced in https://github.com/smithy-lang/smithy-language-server/pull/113) we noticed a difference in behavior that was not expected.
I am skipping the details of that difference for brevity because it was quite complex. The end result was that the language server was seeing some old version of a dependency in the project's classpath - it was smithy-model version 1.27 instead of version 1.36 (artifacts/versions aren't important, I just use them to make it more concrete)
After closer inspection of the classpath we noticed that smithy-model is being included twice transitively by two different artifacts in two different versions - 1.27.0 and 1.36.0.
Previously, coursier in such cases was always picking the latest version. However, the behavior of maven-resolver is different. If you take a look into MavenRepositorySystemUtils the class that smithy-cli uses to instantiate DefaultRepositorySystemSession you will see that it defines ConflictResolver as follows:
DependencyGraphTransformer transformer = new ConflictResolver(
                new NearestVersionSelector(), new JavaScopeSelector(),
                new SimpleOptionalitySelector(), new JavaScopeDeriver());

where "nearest" means:

Nearest is defined as the declaration that has the least transitive steps away from the project being built.

In our cases the newer version was indeed one step deeper than the older one, but no one was paying attention to how many nesting steps of transitive dependencies there are.
This seems indeed to be the way of how maven works by default as at the time of writing there is no other implementation of VersionSelector besides NearestVersionSelector.
The solution in our case was simply - just add the smithy-model with the correct version as an explicit dependency in smithy-build.json.
Was this change intentional? Would you be interested in providing an option to restore old behavior, so that users have a smoother migration path?

The change to use maven-resolver in place of coursier was made to keep the Language Server and the core Smithy packages in sync in how they do dependency resolution.
Do you have a repro for this issue? We wouldn't want to re-introduce the dependency on coursier at this point, but it sounds like the Language Server pulling in two versions of a dependency is not correct.

but it sounds like the Language Server pulling in two versions of a dependency is not correct.

Let me be clear about this. In the end on the classpath there is only one version of each library (smithy-model in our case). The surprising part is how this particular version is chosen in the presence of version conflict due to transitive dependencies.

Do you have a repro for this issue?

I don't have it atm. I can prepare one, but I am very low on time recently so it might take a while.
In general it should be somewhere like this:

Have four libraries A, B, C, D.
Have A depend on B in version 0.1, D depend on B in version 0.2, and C depend on D.
Add A and C into the language server dependencies via buildConfig file.

{
  "version": "1.0",
  "maven": {
    "dependencies": [
      "A",
      "C",
    ]
  },
  "imports": [
    "./smithy"
  ],
  "languageServer": "com.disneystreaming.smithy:smithy-language-server:0.0.28"
}

The dependency graph will look as follows:
-A
 \-B(0.1)
-C
 \-D
   \-B(0.2)

maven-resolver will chose B(0.1) because it is nearer according to the definition, while coursier would chose B(0.2) because it is newer.


Suggest jest-enzyme for simplifying test matchers

Add jest-enzyme to documentation for simpler tests when using enzyme.

This doesn’t explain how to enable them. Should user import some package in src/setupTests.js?

@gaearon Yeah sorry. I'll update that.

Do you need me to squash, or do you do it on commit?

Looks good, thanks! Sorry it took so long to review.


findMany - SyntaxError: Unexpected end of JSON input

Bug description
Since this morning all my call to model.findMany are giving me this error when running on Heroku:

But it's working fine when i'm running my project locally. It's the same code on heroku but the findMany calls are not working...
This is the of code for the above error:
    const pendingBets = await prisma.bet.findMany({
        where: {
            status: 0
        },
    });

Environment & setup

OS: MacOs
Database: PostgreSQL
Node.js version: 16.15.0

Prisma Version
2.30.3

Hi, how are you?
I've some tips for you:
1 - Run script to generate build local, and run folder dist/build.
2 - Use npx prisma generate after run build.
3 - Verify if your migrations already run.

me too

@ThalKod
I 'm in the same situation.
But I changed Node version to 17.x as below, then it just worked!
// package.json
  "engines": {
+    "node": "17.x"
  },

I'm having the same issue, is it coming from a dependency? It works correctly locally

I updated from prisma 2 to prisma 4 and look like it’s working correctly again for me.

This also started occurring for me this about 7 hours ago.

Just started happening for us as well on<EMAIL_ADDRESS>

@chuckstock @ThalKod @bzuker @morifuji-koudai are you also on Heroku? Or is this issue occurring on other hosts.

Like @morifuji-koudai I was able bump my node to 17.x for Heroku and this resolved the issue. I wasn't able to to do a Prisma 2 -> 4 upgrade as this would require some code changes we're not ready to make yet.

@trek We're on fly.io.

@chuckstock so it doesn't seem to be related to a specific host and started happening for all of us suddenly. Is there possibly a dependency for prisma 2.x that wasn't locked to specific version and had a breaking change?

@chuckstock and just to clarify, you're not using Fly.io's "turboku" offering? https://fly.io/blog/new-turboku/

@trek That's correct. (I work with @chuckstock )

Experiencing this same issue. We are using a Nexus plugin that prevents us from upgrading to v4. Anyone on the Prisma team have ideas related to the broken dependency?

This broke for us (using Prisma v2.30.3) when upgrading from Node v16.16 to v16.17 (released Aug 16, 2022). Pinning Node to v16.16 fixed the issue. It appears Node updated its JSON escape sequences to align with the JSON spec (\v, specifically) and v2 of Prisma seems to reference that escape sequence in its code.
Changelog: https://github.com/nodejs/node/blob/main/doc/changelogs/CHANGELOG_V16.md#16.17.0
PR: https://github.com/nodejs/node/pull/43206
Commit: https://github.com/nodejs/node/commit/486dcd7c69b3601252373cca7a132b67a0125fb8

Anything for us at Prisma to do here? If this is isolated to Prisma 2.x and a specific Node version, and upgrades work to avoid that I feel like we can't really do more than to make sure everyone that could be affected can find this issue and fine these workarounds you thankfully documented here?

@janpio no, thanks, since this is isolated to specific version combinations of Prisma and Node, I wouldn’t expect anything to be done from Prisma other than keeping the issue discoverable for others who might run into it. It might help to make the issue title slightly more generic, as our issue wasn’t with “findMany” specifically. “Unexpected end of JSON input” is the bit that helped me locate this GitHub issue. We’ve already been planning to upgrade Prisma to a more recent version - this just provides some extra motivation 🙂

And closing the issue to signal that there is nothing for us to do.
Everyone finding this via Google, please read above how to fix this (upgrade Prisma or change Node version). Thanks.

For those who stuck on PrismaJS v2 it would be possible to mitigate the issue by enabling napi preview feature in the schema file:
previewFeatures = ["napi"]


[Issue 13220][blog] Publishing a community blog for Pulsar's 10k+ GitHub Stars milestone

Master Issue: #<13220>
Motivation
Publishing a community blog to celebrate Pulsar's 10k+ GitHub Stars milestone.
Modifications
*Created a new file in https://github.com/apache/pulsar/tree/master/site2/website/blog *
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

This post mixes StreamNative news with Apache Pulsar news.

@dave2wave I think Alice already updated the PR before you made the comment.

I see where she removed the links in the second commit which is a good first step. But it will still be confusing about things like FunctionMesh and KoP. Those are great, but they are StreamNative product.
For vendor products to be in Apache Pulsar Blog Posts, blog authors need to get input from others and become equitable to the whole ecosystem. That's vendor neutrality.

Hi Dave, thanks for pointing that out. I have removed KoP and Function Mesh from the blog.


Block list

User story
As a repo admin, I want to block specific usernames from being able to assign an issue to themselves, so that my bot accounts can tell human contributors to use the take-action command without assigning the issue to a bot.
Requirement
Allow users to provide a block list to prevent specific GitHub usernames from self-assigning issues.

hey @jamesgeddes, thanks for the PR. After thinking through the approach, I think adding accounts like name[bot] to an auto block list would be better. But limiting interactions is already a feature on GitHub. Would that already cover a block list use case?

I did not know GutHub actions came with a block list, so yes, that would certainly be better than doing it in bash.
What did you think about using guard clauses?

Looks like GitHub interaction limits are time limited?

Also, it would probably be bad to block all *bit* names due to the number of humans who have this in their name.
https://www.babynamespedia.com/names/boy/bot-end

I suppose they are time-limiting. That feature was created to limit spam during times like Hacktoberfest.
re: bot, all common bots on GitHub have the naming convention, [bot] in the name. As long as some don't have square brackets, we would be fine.
If you are interested, we can include a second PR to auto-block [bot]'s.


Use ActivatorUtilities.GetServiceOrCreateInstance for TaskActivity in Azure Functions

Changes ActiviatorUtilities.CreateInstance<T> to ActivatorUtilities.GetServiceOrCreateInstance<T> inside the source generated code for Azure Functions.
resolves #88

Also, consider updating CHANGELOG.md as part of this PR.

Testing would be great but having it in a different repo causes the tests to become broken soon as I introduce any breaking change. When/if we move the Azure functions extension package to this repo it will become much simpler.


Implement compiler checks

The original mal compiler implemented checks for various aspects of syntax validity, constraints, etc. These can be found here.
TODO

[ ] Read up on attack graphs, what they are, what they look like
[ ] Study the mal language. Here are some resources:

[ ] https://github.com/mal-lang/mal-documentation/wiki
[ ] https://github.com/mal-lang/exampleLang/

[ ] Understand the new mal compiler:

[ ] Study the mal syntax grammar, see this PR and the .g4 grammar file, until it gets merged.
[ ] Understand the visitor pattern of the compiler

[ ] Implement the validator checks

[ ] Understand each check in the original compiler's analyzer
[ ] Implement each check in the original compiler's analyzer

It might be useful for the person doing this to also have a look at the Language Graph(https://github.com/mal-lang/mal-toolbox/blob/main/maltoolbox/language/languagegraph.py).


Blurry rendering on non-retina

From @warpdesign  Here is what I found out:

removing max-width:100% and max-height:100%in the css applied to the image fixes the blurriness on the image
the image size is slightly smaller (a few pixels only) than the canvas size (see screenshot): that explains why it gets blurry when drawn onto the canvas

https://github.com/auchenberg/vscode-browser-preview/issues/89#issuecomment-482786179

Inspiration: https://cs.chromium.org/chromium/src/third_party/blink/renderer/devtools/front_end/screencast/ScreencastView.js?q=screencast&sq=package:chromium&dr=CSs&l=164

Any news regarding that issue ?

@melMass I don't have much time to work on this project these days. If you have time to contribute it would be much appreciated.

I'll try to find some time to look at this issue.

Looks like a solution have been found in the fork of BrowserPreview, https://github.com/antfu/vscode-browse-lite/commit/94b4064122aff4a9be515a0d0fe50c51c6ac0f28

+1

Looks like a solution have been found in the fork of BrowserPreview, antfu/vscode-browse-lite@94b4064

In VSCode-browse-lite too, it is blurry.

Duplicate of https://github.com/auchenberg/vscode-browser-preview/issues/75


C++ WinApi keywords not supported?

Standard WinApi keywords like "NULL", "BOOL" aren't highlighted in "cpp" mode.

Well, they aren't really part of the language but rather a library #defines (if I remember correctly). We could add such words in some cases but cpp.js is already big enough and I don't want to go on the slippery slope of adding more stuff that someone else would consider standard or widespread.
So in other words, this is by design :-)

Maybe they are not the part of the language but they are a BIG part of the C++ world.
It's just poor highlighting. SyntaxHighlighter with C++ parser from 2006 can highlight more things than highlight.js in 2014 :+1:
http://alexgorbatchev.com/SyntaxHighlighter/
And yes, I would have to switch back to it, who needs a syntax highlighter if it doesn't highlight anything? :-1:

Highlighting as many things as possible was never a goal for highlight.js. So please choose whichever highlighter works best for you!
P.S.


Preserve field order when updating kustomization.yaml file

This commit causes argocd-image-updater to preserve the order of fields in the kustomization.yaml file when updating the image(s). Doing so will minimize git diffs in the case that other tools also edit the kustomization.yaml file.
Fixes #502

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (7d93c7a) 66.27% compared to head (be96678) 66.06%.

Files
Patch %
Lines

pkg/argocd/git.go
25.00%
4 Missing and 5 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #666      +/-   ##
==========================================
- Coverage   66.27%   66.06%   -0.22%
==========================================
  Files          22       22
  Lines        2150     2160      +10
==========================================
+ Hits         1425     1427       +2
- Misses        591      595       +4
- Partials      134      138       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

The build_image job failed, but I'm pretty sure that's not due to my changes.

@jtdoepke can you also fix the DCO failure with sign-off?

@jtdoepke thanks!


Fixing the mimeType description to 'output'

Changed the config typo #584 and "re-opening" the last pr #587

Congrats on your first Runme contribution! This project can't live without the participation of the community. We would love to see more from you, so let us know if we can help to find interesting areas for you to contribute to. We appreciate you 🙏 ❤

Thanks! I'll be trying to contribute to the project!


!!/allspam should report all the user's posts to metasmoke

While discussing !!/allspam, @angussidney noted:

Using !!/report with multiple arguments is preferred however, as then we get a copy of the post on MS, we can give feedback ,etc

While this is true, it's also more cumbersome (and not doable if the user has more than x posts, as !!/report is rate limited). Instead, !!/allspam should be rewritten so that it reports all the user's posts to metasmoke.
Alternatively, we could make a new command !!/report-all <link to user> for this.

I agree that the requested behavior is what I naively expected from a command named like this.

It'd be great to be able to report all of a user's posts network wide. That could be quota-expensive (1 to get the user's site account, another to get the assoc accounts, and then up to 300ish (usually nowhere near that, more like 5) to get all posts.
Does anyone know of a more efficient way to get from a user to all their posts across the network?

We might be able to use http://api.stackexchange.com/docs/users-network-activity and use that to check recent posts. More than likely a true spammer only has a handful of things on this. That will at least narrow down which sites we have to query.
We still need to go from user account ID->network activity->listed activity (times how ever many sites listed here)

https://github.com/Charcoal-SE/SmokeDetector/tree/allspam

This is currently in the NG pull request: https://github.com/Charcoal-SE/SmokeDetector/pull/1135


[API Feature Request] Installing native node add-ons using node-gyp

Description
Right now, it looks like we cannot use any native node add-ons which are built using node-gyp. For instance, I tried various npm package to access sqlite database but failed: sqlite, sqlite3, better-sqlite3, etc
Steps To Reproduce

npm install and try to import in your extension any package relying on node-gyp (for instance: https://github.com/JoshuaWise/better-sqlite3)

The current behavior
You will get a nodejs error (such as Error: Could not find module root given file: "node:internal/modules/cjs/loader". Do you have a package.json file?)
The expected behavior
No error

👋 Hey, based on this issue: https://github.com/evanw/esbuild/issues/1051 is there still anything that can't be done here? My understanding is that esbuild simply grabs all the .node files and bundles them along? Though sqlite3 fails for weird reasons, better-sqlite3 looks like it could work with this native node modules plugin approach?

For now, there a workarounds such as marking certain modules as external via package.json and using other alternatives such as the mentioned wasm-based package above.

@fe9lix How to make it as external via package.json? Can you give an example?


Ignore path separators not cross platform

I had a bit of a struggle excluding my node_modules and similar from the Todo list. Turned out it was because the default settings used linux style path separators, and I am currently on win32.
My solution was to overwrite the default ignore paths with windows path separators.
  'todo-show':
    'ignoreThesePaths': [
      '\\node_modules'
      '\\vendor'
      '\\bower_components'
    ]

While this works at the moment does it make it hard for me to port my settings across computers and platforms. (E.g. using atom-sync-settings.)
Ideally the configs should use linux style path separators, and the package should determine if they need to be converted to win style.

Im running into the same Problem.
But by looking at the source i found that todo-show is internally using atom.project.scan (which is also used by the core-package "find-and-replace" i think). This API-Call ignores files and folders set by atoms core setting "Ignored Names". If you additionally check "Exclude VCS Ignored Paths" then all entries in your local-project-gitignore file are gonna ignored too.

This should be platform independent(works for me on my windows machine) so you dont have to add "todo-show.ignoreThesePaths" which is not.
This should be better documented in the README.

I accept PRs :)
On Fri, Feb 6, 2015 at 1:52 AM t3asdev<EMAIL_ADDRESS>wrote:

Im running into the same Problem.
But by looking at the source i found that todo-show is internally using
atom.project.scan (which is also used by the core-package
"find-and-replace" i think). This API-Call ignores files and folders set by
atoms core setting "Ignored Names". If you additionally check "Exclude VCS
Ignored Paths" then all entries in your local-project-gitignore file are
gonna ignored too.
[image: atom-core-ignores]
https://cloud.githubusercontent.com/assets/930604/6077233/19af6b18-aded-11e4-9a32-f1588f260fca.png
This should be platform independent(works for me on my windows machine) so
you dont have to add "todo-show.ignoreThesePaths" which is not.
This should be better documented in the README.
—
Reply to this email directly or view it on GitHub
https://github.com/jamischarles/atom-todo-show/issues/30#issuecomment-73212662
.

PR send ;)

Should be fixed now


Add an EVP demo for key encoding using EC

Fixes openssl#14117
This pull request looks really simple, regarding there is already demo for RSA. I had some ideas how to make it more interesting, but it would make the demo confusing. The PR can make it clear.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@paulidale still OK?

This pull request is ready to merge

Merged, thanks.


Add reserved IP range for gcp; to protect from director VM etc

The https://github.com/cloudfoundry-incubator/bosh-google-cpi-release/tree/master/docs/bosh terraform plan places the bastion/nat/bosh in <IP_ADDRESS>, <IP_ADDRESS> and <IP_ADDRESS>. This PR adds a reserved range to cover these.

Note this is purely for the "etc" part since director reserves its own IP. So it might be better to require users to explicitly add IPs via ops files if they're using them (since this change is not for reservation the IaaS requires).
Wonder what others think.

In GCP CPI's own docs tutorial the BOSH is going into the same network as the bastion + nat vms (as per image above). https://github.com/cloudfoundry-incubator/bosh-google-cpi-release/tree/master/docs/bosh
Is there a use case from other GCP users where they put the BOSH into a subnet that didn't have a bastion/nat already in it at the .2, .3 addresses?

The NAT/Bastion VMs don't have to be in the same subnet. I'm working on some ERT deployment scripts and I ended up creating a mgmt-subnet for the director/ops man/.. and a unmanaged-subnet for terraform created VMs (NAT, bastion). On GCP any subnets in the same network can route between one another by default.

Ok I will create a operator patch file in the tutorial.


Main: remove "main" app from are project

The main app was created by Lior Shtaimberg for the first demo and it is not needed anymore because we are going to create new views for are project

The tests do not pass because there are no tests in the project

Why delete the app and not use it to create the UI you want?

authentication

The UI will be created in the House app and in the Tips app
In the Django framework, every app is like a software component, the House and expenses is not a good example because we decided to break it into two separate apps but the Tips App is a good example
The Tips app can be built as a reusable Django app. for example another project can just add the routing and register the app in the setting file and it will reuse it.
About the authentication, we will not implement an authentication system for the third demo, maybe in the future if that's where the course will take us
Our plan in general:
3 apps-House, Expenses, Tips
2 places for the view(UI) - Tips and House
We can meet and talk about the architecture of the project If it's not clear or we want to change our plan


Alternative to remove local file

As discussed in #229, for local rendering only, this adds a select box at the left of the bench result's name (or just below depending on the available space) and a trash icon below the table.

If selected, then the trash icon displays the path of several files that are inside the outputs folder:

the selected .csv datasets,
the selected .html associated generated pages,
the cached json file (so the next time we call benchopt plot --all . for example the deleted benchs are not presented in the results table).

Then, one can simply click on copy to clipboard and paste the command in the terminal from the outputs folder.
For the current time, there is also a hide row button that will hide the selected rows (until the page is refreshed as it is static) but it can be useful to avoid clutter if we open the results in multiple tabs.
PS: this also separates the JS for the table page from the html. The only thing to adapt in the future would be the paths of the files (available as data-csv and data-html attributes of the checkboxes) when the user does not use benchopt plot but generate-results or a local make render (open issue #230).

I am +0 on this
I'll let vote those we feel the need for this

Forgot to mention but I like the feature! I am +1 on this one!

Yes sure. I haven't really had the time to go back on BenchOpt these past few days but I'll take care of it ASAP.

Codecov Report

Merging #232 (0eb4f39) into main (a026f35) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #232   +/-   ##
=======================================
  Coverage   62.89%   62.89%
=======================================
  Files          41       41
  Lines        2083     2083
  Branches      367      367
=======================================
  Hits         1310     1310
  Misses        677      677
  Partials       96       96

Thanks for the fix in .circleci config!
Just a comment for next time, please try to fix such issue in a separate PR so it is easier to review and merge each task.

If you prefer, I can revert the last commit and open a new PR with the .circleci fix now.

No need to revert it, you can open a new pr, merge it into main and merge main into this one


Update 30731.ts

Please confirm correct information. I am currently developing a project on movement. I have provided the correct movement devnet rpc and chainid information.
Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address: 0x76914803b100Df11D1329e7F916F83B72bb4A508

PR-Codex overview
This PR updates the chainId and networkId in the MOVE chain configuration from 30731 to 336, and adds an RPC endpoint for the MEVM devnet.
Detailed summary

Updated chainId and networkId to 336 in the MOVE chain configuration
Added an RPC endpoint for the MEVM devnet

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

hello @Quanionxyz, please instead correct this information on ethereum-lists/chains which will automatically update this data, a direct update here will instead be overridden by the automatic update as we sync with the upstream information


Fix semeval

changes as discussed in #580

Hi!
Please make the example answers lowercase too. Thanks

LGTM!


Your missing some of your found places

Hi,
You have found a very interesting and useful way of finding events based on location. Thanks for showing it openly.
I found that you are missing some of your places.
In index.js on line 101 you have this code:
  //Create array of 50 places each
  data.forEach(function(idObj, index, arr) {
    if (tempArray.length < idLimit) {
      tempArray.push(idObj.id);
    } else {
      ids.push(tempArray);
      tempArray = [];
    }
  });

  return ids;

That misses every 50th place and what is left in tempArray when the loop is done. I rewrote it like this:
      //Create array of 50 places each
      data.forEach(function (idObj, index, arr) {
        tempArray.push(idObj.id);
          if (tempArray.length >= idLimit) {
            ids.push(tempArray);
            tempArray = [];
          }
      });
      if (tempArray.length > 0) {
        ids.push(tempArray);
      }
      return ids;

Keep up the good work  //Big

Thanks for the report of this bug. I just fixed it with https://github.com/tobilg/facebook-events-by-location/releases/tag/0.4.1 and pushed the new version to NPM.


Add corresponding llvm to ghc's PATH

I just got bitten by having LLVM 3.5 in PATH and attempting to build with -fllvm. That led me to a thought -- why don't we add a wrapper to ghc which would prefix needed llvm version? IIRC there are even proposals to embed "recommended" version of llvm within GHC build (3.4, 3.5 and 3.6, again incompatible, in the future). This wrapper would also simplify things later when 7.10 is released for those who want to use latest possible LLVM for each compiler.
Upside is -fllvm working out of the box and different llvm version in PATH not messing with compilation.
One possible downside is that we gain another dependency on all the Haskell libraries (it can be avoided on GHC itself, however, by having wrapper in an another derivation).
cc @peti

IMHO, it's quite feasible to add a flag to the ghcWithPackages derivation that allows users to include llvm into the generated environment if they want. We cannot depend on llvm unconditionally, though, because the derivation is too large to include it for everyone, even if they don't use it.

Good idea, let's have it this way. I'll come up with a patch later, if you don't mind.

Merged to master.


Cannot be simplified

From #164
MathS.FromString("1+2x*-1+2x*2+x^2+2x+2x*-4+2x*4+2x*2x*-1+2x*2x*2+2x*x^2+x^2+x^2*-4+x^2*4+x^2*2*x*-1+x^2*2x*2+x^2*x^2").Simplify()

Fixed, I'll probably release <IP_ADDRESS> soon

Oof I just realized: I added polynomial parser to Simplify, but there're cases that it won't work. First, if there should be a replacement. Second, when some terms are not polynomial, e. g. 2x + 3x + ... + 10x + e ^ x

Continuation: #185


Fixed console error on manual localStorage clear

What kind of change does this PR introduce? (check one with "x")
[ x ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
When localStorage is cleared manually from Application > LocalStorage > clear button, this will throw console error TypeError: Cannot read property 'indexOf' of null.
What is the new behavior?
No console errors will be thrown from now.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
None
Other information:

@HackedByChinese could you please merge this one? This would resolve issues in my application.

@HackedByChinese Any updates? We are waiting for this to be in. Please merge it in as it is not a harmful code.


[wip][dnm] Akaris libxdp

- What this PR does and why is it needed

- Special notes for reviewers

- How to verify it

- Description for the changelog

/retest-required

/retest


Camera mode detection issue

It seems the camera mode starts assuming you're in 1st person, even when you start out in 3rd person really
1st example
This example shows how this behavior of detecting the camera mode is buggy at first when the scene loads
Example2.zip
Switch between 3rd and 1st person mode.
When the scene knows you're in 3rd person, the scene should display a message saying that "this experience is best viewed in 1st person". But in this case it doesn't detect you started in 3rd person until you switch back and forth to 1st person.
2nd example:
When you run the attached game.ts you will see that the scene runs the "first person" branch when clicking, even though the player loads into third preson view by default.
Then after pressing 'V' twice the cameraMode is returned correctly. (see floating textShapes)
https://app.zenhub.com/files/389651442/93218fb7-fd32-44bb-9c58-fb3b6971db7b/download

renderer: https://github.com/decentraland/unity-renderer/pull/1608


Opened EPS for the first time in a few months and I cannot connect my Electrum wallet

It keeps saying connecting and disconnected.  It was working perfect the last time I opened it, and I have not changed anything in between.

Probably related to the expired certificate, have a look here to fix it.

If

Probably related to the expired certificate, have a look here to fix it.

If I just wait will a fix eventually come to github or is this project basically dead?

If I just wait will a fix eventually come to github or is this project basically dead?

The issue is nothing more than an expired certificate. An updated certificate fixes it.
There's already a pull request that fixes this issue here so yes, you can just wait and it will be resolved.
If you want to use it today, simply follow the instructions here


User should be able to get outcome of individual dice

Todo: User should be able to get dice
Todo: If luck exceeds max potential a luck dice temporarily added

Resolved in v1.3.0
Individual dice are now available with luck. There is a difference between value and total.
If applicable luck exceeds maximum potential of dice a bonus is available that can be used by third part.


socket errors unhelpful

Messages such as these should include the host:port, otherwise its not possible to know what end point is causing problems in a process with multiple connections:
E0531 04:33:48.682259810       8 tcp_client_posix.c:145] socket error: connection refused
E0531 04:33:50.682653580       8 tcp_client_posix.c:145] socket error: connection refused
E0531 04:33:54.728829060       8 tcp_client_posix.c:145] socket error: connection refused
E0531 04:34:02.729196699       8 tcp_client_posix.c:145] socket error: connection refused
E0531 04:34:18.828834811       8 tcp_client_posix.c:145] socket error: connection refused

This is probably also true for Windows sockets.
On May 30, 2015 21:37, "Matt Wytock"<EMAIL_ADDRESS>wrote:

Messages such as these should include the host:port, otherwise its not
possible to know what end point is causing problems in a process with
multiple connections:
E0531 04:33:48.682259810 8 tcp_client_posix.c:145] socket error:
connection refused
E0531 04:33:50.682653580 8 tcp_client_posix.c:145] socket error:
connection refused
E0531 04:33:54.728829060 8 tcp_client_posix.c:145] socket error:
connection refused
E0531 04:34:02.729196699 8 tcp_client_posix.c:145] socket error:
connection refused
E0531 04:34:18.828834811 8 tcp_client_posix.c:145] socket error:
connection refused
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/1840.

One of the things I'd like for us to do in coming weeks is to establish an internal api for capturing and reporting more context for all of our error messages. Just host:port will be insufficient when building a client side load balanced channel, for instance.

This duplicates #49


Simplify not enough args handling

It's currently just handled in each argument handler itself. Putting it in a more standard place would be nice.

Fixed by 9acdeedec6c1e54b230c44b446c01319fe11fd20


Debloat fontawesome?

   10.6 MiB [##########] /fontawesome-free-5.1.0-web
  303.0 KiB [          ] /leaflet
  267.0 KiB [          ] /bootstrap4
  140.0 KiB [          ] /x-editable
   81.0 KiB [          ] /select2
   54.0 KiB [          ] /bootstrap-daterangepicker
   49.0 KiB [          ]  jquery.min.js
   41.0 KiB [          ]  jquery-3.5.1.slim.min.js
   29.0 KiB [          ]  moment.min.js
   25.0 KiB [          ] /multi-level-dropdowns-bootstrap
   13.0 KiB [          ]  popper.min.js

fontawesome takes 10MB and flask-admin is only using a tiny portion of that, can we remove the unused files to reduce the package size?

This definitely feels like something we should do, yes. Would be happy to accept a PR that addresses this.


Support tags mode in autocomplete

What problem does this feature solve?
ability to fille the autocomplete field with multi-values.
What does the proposed API look like?
same like nzMode="tags" with select but instead of a dropdown list it will be a normal field with autocomplete , also if we will have the ability to customize these tags like adding icons to it it will be great

Ant Design does not provide this feat, you can submit a issues to ant-design/ant-design, it will be sync if ant-design provide this feat.


[Feat][C++] Remove the  custom namespace setting feature

Is your feature request related to a problem? Please describe.
Currently GraphAr C++ provides support user to set custom namespace to avoid name conflict. But since the namespace graphar is not a widely-used names, I suggest we can remove the feature.
Allowing users to set a custom namespace for a C++ library during the CMake configuration step is not considered a best practice in the C++ community.Here are several reasons why this approach is generally discouraged:

Consistency: A consistent namespace across all deployments of the library aids in maintainability and user understanding. Changing namespaces can lead to confusion, especially when dealing with community support or third-party tutorials and examples.
Documentation: Documentation becomes more challenging to write and maintain if the namespace can vary, as examples in the documentation might not match the user's configured namespace.
Code Clarity: When working in a codebase that uses the library, having a standard namespace makes it clear which library the code is interfacing with.
Tooling and Integration: Tools that perform static analysis, code indexing, and refactoring may not be aware of the dynamic namespace configuration, leading to incorrect behavior.
-Build Systems and Package Managers: Package managers and build systems expect library names and namespaces to be consistent to manage dependencies correctly.
Namespace Collisions: The primary motivation for allowing configurable namespaces might be to avoid namespace collisions. However, such collisions are rare, and when they occur, they can often be resolved using namespace aliases or other techniques that don't require changing the library's namespace.

We should make the design right in the GraphAr's beginning
Describe the solution you'd like
Remove the custom namespace setting feature.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

/cc @lixueclaire @Thespica


Re-enable dask testing.

stuck again :-(

Only one stuck instead of two, that's progress :)

Codecov Report

Merging #584 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #584   +/-   ##
=======================================
  Coverage   86.66%   86.66%
=======================================
  Files          76       76
  Lines        7193     7193
  Branches     1439     1439
=======================================
  Hits         6234     6234
  Misses        765      765
  Partials      194      194

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60de327...6e89ba1. Read the comment docs.

Superseded by PR #585


MoveFile failed; code 183. Cannot create a file when that file already exists

VSCode Version: Latest as at this post (can't check since it will not run)
OS Version: Windows 10

Steps to Reproduce:

Open VS Code and wait for a update
Ask it to update
Error pops up. The file is askpass.sh
Try deleting the file causing the problem (didn't work on my machine)
Click ignore (VS Code Insider no longer runs

This is from the insider channel. When updating the main version, I got the same message, asked it to ignore. Currently, that instance crashes every few minutes. Not sure if the ignore was the cause.
Running from the command line gives the error below
{ errorCode: 'load',   moduleId: 'https-proxy-agent',   neededBy: [ 'vs/base/node/proxy' ],   detail: { [Error: Cannot find module 'https-proxy-agent'] code: 'MODULE_NOT_FOUND' } }

Duplicate of #3560
Please reinstall Code


Chained handlers

I have the following routes:
First one work as expected, both handlers are invoked:
router.get("/private").handler(new CheckRoleHandler(UserRole.Admin));
router.get("/private").handler(infoHandler);

I have also tried this:
router.get("/private2").handler(new CheckRoleHandler(UserRole.Admin)).handler(infoHandler);

but only the second handler gets invoked and the CheckRoleHandler is skipped.
Don't know if this is by design, but one would expected that all handlers in a given chain are executed ?

You can only add a handler to a route (this is by design but might need to be changed in the future) and there is no proper documentation either. We should for now atleast throw an exception stating that there is already an handler or log a warning at runtime

Sure ... it took me at least 10min to find the problem ... was going with the second approach.
Would it be this hard to allow chaining? I think it could be useful in some cases ...

For now it will be hard because we need to rework the internals of the route object. The reason is not just to allow multiple handlers but to correctly respond to content negotiation requests https://github.com/vert-x3/vertx-web/issues/456 and also to properly answer to https://github.com/vert-x3/vertx-web/issues/594.

I'll add the exception as proposed above for the next point release and from there we'll start working on the overall issue as described above.

The warning has been already raised to an exception: https://github.com/vert-x3/vertx-web/issues/434

I'll close this since the fix is already on master and the remaining tasks will happen in their own issues.


Latest GQL is not compatible - changed to: "graphql": "^15.5.0"

@julianwagle: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


libsodium generate a conflict with my project

Hello,
I am trying to use web3swift but libsodium generate a conflict and i got a lot of errors.
For example:
While building module 'Foundation' imported from ...
System/Library/Frameworks/Foundation.framework/Headers/Foundation.h:8:9: In file included from /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS11.4.sdk/System/Library/Frameworks/Foundation.framework/Headers/Foundation.h:8:
What could be happening?
Regards

My podfile is:
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, ‘9.0’
post_install do |installer|
installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
config.build_settings['ENABLE_BITCODE'] = 'NO'
end
end
end
target “project” do
use_frameworks!
pod 'AFNetworking', '~> 2.5.4'
pod 'AFNetworkActivityLogger'
pod 'Google/Analytics', '~> 1.0.0'
pod 'Firebase'
pod 'Firebase/Storage'
pod 'Firebase/Messaging'
pod 'Firebase/RemoteConfig’
pod 'TesseractOCRiOS'
pod 'web3swift', '~> 0.9.0'

end

Hello @corocraft
Never had a problem like this myself. Will create a dummy project with your podfile and check.
Sincerely, Alex

Hello @shamatar,
Thank you for your answer. Finally, I could solve the problem. This was because libsodium is in C and it generates a lot of errors with header's librarys. I solved this renamed each file extension from .c to .m.
I don't know because this happened.
Regards,

libsodium removed from web3swift


Make importing the library fast

import resource_retriever is now slow due to the roslib.load_manifest and instantiation of rospkg.RosPack on import time.
In this PR,

Removed the roslib.load_manifest as it is not necessary on the latest ROS distribution
Delay instantiation of rospkg.RosPack object to right before the object is used.

@ahcorde @clalancette Kindly ping to the maintainers.


error when using 'guidedfilter2d_color'

Thanks for the author to share this code. But I meet some error here, when I run this demo:

        target = torch.randn((2, 1, 10, 10))
        net = GuidedFilter2d(radius=4, eps=1e-4)
        res = net(target, im)
        print(res.shape)

I got the error information as below:
Traceback (most recent call last):
  File "/projects/guided-filter-pytorch-master/guided-filter-pytorch-master/guided_filter.py", line 204, in <module>
    demo()
  File "/projects/guided-filter-pytorch-master/guided-filter-pytorch-master/guided_filter.py", line 187, in demo
    res = net(target, im)
  File "D:\anaconda<IP_ADDRESS>\envs\torch1\lib\site-packages\torch\nn\modules\module.py", line 493, in __call__
    result = self.forward(*input, **kwargs)
  File "/projects/guided-filter-pytorch-master/guided-filter-pytorch-master/guided_filter.py", line 17, in forward
    return guidedfilter2d_color(guide, x, self.r, self.eps)
  File "/projects/guided-filter-pytorch-master/guided-filter-pytorch-master/guided_filter.py", line 117, in guidedfilter2d_color
    mean_a = torch.stack([boxfilter2d(a[:, i], radius) for i in range(3)], 1) / N
RuntimeError: The size of tensor a (3) must match the size of tensor b (2) at non-singleton dimension 1

But there is no problem when I change the image to gray and run this code.  So I'm now running the gray version of this code. All in all, thank you very much for the sharing the code

The shape of N is incorrect.
I fixed it.


fix(TDI-43734): update spring-boot-starter version and bump daikon

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-43734
What is the new behavior?
new version of daikon should include new Spring boot version.
Do not merge till PR will be merged to maintenance/0.31:
https://github.com/Talend/daikon/pull/588
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: success :star:
Sonar results


Add --exclude-target to exclude targets when using --all

As I mentioned in #61, sometimes I'd like to exclude some older targets from the list node-abi gives you when you do --all.

Are you using --all because you're not using Node-API? If so, I'm reluctant to approve this PR because we want to drop support of non-napi: https://github.com/prebuild/prebuildify/issues/57#issuecomment-979894415


Small tweaks to Rust code after a review pass

I've done a review pass through all of the Rust code. This branch contains a handful of small fixes/tweaks I made along the way since I couldn't leave review feedback on already merged code :-)
Feel free to treat these changes as if they were suggestions. I'm happy to drop any that aren't to your taste. Probably best reviewed commit-by-commit.
One extra note: I was going to add Nightly coverage to CI but sqlx and the current nightly are interacting badly due to a rustc bug. Happy to revisit that once the upstream issues are fixed.

This is perfect! There is only one change that I'm not sure about (see this comment), but I'm going to merge right away and we can sort it out later.


feat(pipettes): add specific can messages for 96 channel tip motors

Overview
Rather than having a specific node id for the tip motors on the 96 channel, I decided to create specific CAN messages for certain motor functionalities. Unlike the gripper, the tip motors are pretty intrinsic to the rest of the pipetting functionality and these CAN messages are only used in very specific instances.
See the python side of this change here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ff206fb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #531   +/-   ##
=======================================
  Coverage        ?   83.83%
=======================================
  Files           ?       88
  Lines           ?     2704
  Branches        ?        0
=======================================
  Hits            ?     2267
  Misses          ?      437
  Partials        ?        0


Supporting the now optional {format} argument

The swagger server now supports optional format in the URL and that broke the UI

in swagger-service.js starting at line 111 the code doesn't take the optional formatString into account


Observed and Simulated Sample at Start

Maybe there is a good reason, but why are the simulated (at start) and observed samples not 100% identical when using the same initialization file to simulate a sample and start an estimation?
est.grmpy.txt
notes.txt
test.grmpy.txt

closed see #70.


🐛 BUG: allowArbitraryAttributes doesn't support alpine . modifiers

Describe the Bug
The setting supports most alpine attributes, however the . character breaks the types... And the . is used for quite a lot!
Example: x-on:click.outside="isOpen = false"
Some docs:
https://alpinejs.dev/directives/on#keyboard-events
https://alpinejs.dev/directives/on#modifiers
Steps to Reproduce

npm init astro using template
astro add alpinejs
Add x-on:click.outside="console.log('!')" to an element
Error! You'll see an error message.

Even worse...

Add x-on:click.outside="console.log('!')" to an element within a loop.
Error! Syntax highlighting is completely broken.

Example:
<div>
  {
    ['!'].map((str) => (
        <div
          class="subMenu"
          x-data="{isOpen: false}"
          x-on:click.outside="isOpen = false"
        >
          {str}
        </div>
      ))
  }
</div>

That helps a lot. I just want to point out that within loops, it no longer breaks everything, but there's still a highlighting issue. Thanks!

Ah yeah, that one is a bit challenging since we don't provide the highlighting inside JSX expressions ourselves..

Hello, this is not related to allowArbitraryAttributes. It's caused by the current version of the extension not being able to parse attributes with . in them as it is invalid in JSX.
You can try out the Pre-release version of the extension, it has a fix for this.

Where can I find the pre-release to try?

The pre-release has been released many months ago, this should now work just fine in any recent versions!


A few lines translated into Russian

A few lines translated into Russian

@snuyanzin Could you sign our CLA so we can accept your PR.

@Bryukh , ok, no problem. I've just done it

Thanks!


add dependencies

github.com/google/go-github/github

golang.org/x/oauth2 - recommended for dealing with auth
github.com/google/go-querystring - dependency that for some reason glide did not install it

Does our contributing doc need to change then? I'm confused by it as it says updating should be in its own PR but the last bullet contradicts it

Good point. I have created https://github.com/Originate/git-town/pull/951 to address this.

I'll merge this with #937


Synthesize a track setting on sync state.

lg!
Do we need to also think about flipping other disabled tracks back to non-disabled? thinking about if it's possible for the client to set something enabled when the server didn't receive that message.

That would come in the Subscribed part right? All the subscribed tracks are in that list and by default they are enabled. Do we need an explicit enable?

Will merge this and address separate enable in a subsequent PR if needed.


Limit size of archives put to HSI

According to the NERSC HPSS documentation (https://docs.nersc.gov/filesystems/archive/#avoid-very-large-files), files over 2TB are inefficient when put to HPSS. They recommend breaking files up into 500GB chunks if they get over that limit.
The hsi handler mcscript.task.archive_handler_hsi() should:

[x] Inspect the size of an archive file to put.
[x] Put the file directly using hsi if smaller than threshold, or
[x] use split to break up the file into smaller segments and put those with hsi.

In addition (so that consumers of archives don't need to be aware of this splitting behavior):

[ ] mcscript should also provide a wrapper function to fetch and reassemble archives,

c849af4887cd94fe9141cbdd11eac50ba3a8799a implements the logic in archive_handler_hsi()


Explore implementing Switch as X

Requirements

[x] Air circulator
[x] Mini air circulator
[x] Thruple lamp
[x] Thumb lamp
[x] Marquee
[x] Row lamp

Resources

2022.4: Groups! Groups! Groups!
Switch as X

Resolved in #589


Does this tcmalloc has a release version

Hi guys,
Does this tcmalloc has a release version or plan?
Thanks!

It would be extremely helpful to have semi-regular releases so that this library can be pulled at stable points. Currently the only way I've seen to do that is to pull gperftools, but that project is quite slow to release as it is nearing a year out-of-date and I've seen significant improvements to this project since
I also see a PR for CMake which would also be very, very useful as bazel is hard/impossible to get running with custom build systems that can't write to system directories or need special steps. I build >200 opensource components and none use bazel. cmake is the new standard - replacing autotools

Having proper versions would make benchmarking easier.


Added OSM's BANO (French Open address database)

I tried to write the json spec for the dataset, but I'm not quite sure about the conform part.

dataset needs to be un-gzipped
there is no header in the csv file, headers are given in their readme (https://bano.openstreetmap.fr/data/lisezmoi-bano.txt)

Hope this can help and the DB is elligible

Yep, for reference https://github.com/openaddresses/openaddresses/pull/236.


Build long prefix package using conda-build 2.x

Create a conda package with a long prefix (255 characters) using conda-build 2.x

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Could you please take a look and (if acceptable) merge, @aseyboldt ? This is required as part of upgrading to conda-build 2.


Write design document of keycloak's CIBA support

Based on the prototype in https://github.com/jsoss-sig/keycloak-fapi/issues/47 , Write design document of keycloak's CIBA support, submit it to keycloak-community repository, and be merged.

merged https://github.com/keycloak/keycloak-community/pull/105
merged design document can be found in https://github.com/keycloak/keycloak-community/blob/master/design/client-initiated-backchannel-authentication-flow.md

@tnorimat Does Keycloak not support PUSH mode yet please ?

@vanhoanHoang Unfortunately the current keycloak does not support CIBA push mode, only support poll and ping mode.
Are there any actual use cases where push mode is beneficial?


Odd inconsistency in auto generation of pairs

I've noticed something which seems like an odd(as in: I don't think this is supposed to happen) inconsistency in the auto generation of pairs.
If I am in a place where I haven't disabled nvim-autopairs then autopairing is still disabled if there is text to the right of the cursor and this makes sense. If I have text to the left of the cursor then autopairing works(this makes sense) except for the pair ''. So foo( automatically becomes foo() and foo" becomes foo"" but foo' doesn't automatically become foo''. If instead I have whitespace to the left of the cursor then ' becomes '' automatically. Is this supposed to happen?
I'm using the same init.vim that I used in another recent issue:
call plug#begin('~/.config/nvim/plugged')
Plug 'nvim-treesitter/nvim-treesitter'
Plug 'nvim-treesitter/playground'
Plug 'windwp/nvim-autopairs'
call plug#end()
lua << EOF
ts_config = require('nvim-treesitter.configs')
ts_config.setup{
	ensure_installed = "maintained",
	autopairs = {enable = true}
}
ap_config = require('nvim-autopairs')
ap_config.setup({
	check_ts = true,
	ts_config = {
		cpp = {'string_literal', 'comment'},
		c = {'string_literal', 'comment'},
		python = {'string', 'comment'}
	}
})
EOF

it is normal because  English language  use  single quote inside text sample  doesn't  or didn't

You're right, it now makes a lot more sense than I originally thought!


Dangling actions-object anchor links in Check Runs / Create a check run

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/rest/reference/checks#create-a-check-run--parameters
What part(s) of the article would you like to see updated?
The actions object link should work:

See the actions object description.

Additional information
No response

@jsoref Thanks for opening this issue ✨. You or anyone else is welcome to open a PR with a fix 💖

Hi, isn't the link going to https://docs.github.com/en/rest/reference/checks#actions-object - this link itself ? Could you please clarify what's the change required ?

Hmm, when I click on the link, I end up with something like this:

This is in Chrome (Version 99.0.4844.51 (Official Build) (x86_64)) on macOS
Since the link is for actions object, I'd expect a heading actions object that scrolls into view and is clearly related to a link for actions object.

Whatever this thing is, it doesn't scroll into view
If I scroll up, this is what I see:

Note that actions items isn't a clear match to actions object.
You're right it isn't technically dangling, it's just practically dangling.
It's ok for a link in a document to link to another portion of the same document. It isn't ok for it to link to a random portion of the document with no clear indication as to why that's the right portion, which is what I'm experiencing today (and when I filed this bug 8 days ago).

@jsoref @akhilalekha I apologize for triaging this to good first issue and help wanted. Since this is rest/reference document, this has to be fixed internally.
@akhilalekha Thank you so very much for taking out your time to look into the issue. I apologize for the confusion 💛.
I will get this issue moved to an internal repo for tracking.
Thanks so much for taking your time in contributing to the GitHub docs 💖

Thanks for clarifying @jsoref  @ramyaparimi


qri.get().readme only returning Readme("")

Running qri.get('xristosk/bot_sentinel_trolls').readme is returning Readme("")
Same happens with other datasets I have locally.

Fixed by e92bd6f544ebd0f7adcf421ec9fb3882455c9345. You can now do: qri.get('me/my_ds').readme.script to get the markdown text, and qri.get('me/my_ds').readme.render() to see the rendered html. The readme itself will render depending upon the environment its running in: markdown for console, html for jupyter.
This is working in the development version, but now in the pip release, as other issues are blocking a new release.


Login button takes you to Github?

I'm not sure what logging in is supposed to do in any case, errant button?

Hi @oluoluoxenfree,
The current behaviour (should be) an oauth flow to github to allow adding projects, it also allows the user to see which of their repositories have been added already.
I don't think we need to have a user login at all currently- all it does is give permission to project#new and a very sparse user page that lists their projects already added to contribulator.
@andrew might have his own opinions on this so feel free to chat about it at http://gitter.im/andrew/contribulator or submit a pull request with some ideas.
Thanks :+1:

The main reason I added it was because we need a certain number of valid oauth tokens to be able to download repo information from the github api, which we have now, so I don't see why the log in requirement can't removed now.


how to take rowData and pass it to dispatch function

I want to store the selected item using Redux.
I am using the onRowClick property, to move to the next component that should show me the information of the item selected, but I don't know how to use the dispatch function and how to capture rowData and pass it as payload. Thanks in advance and thank you for this excellent compoennt.!
..
render() {
const columns = [
{ name: "name", label: "Name" },
{ name: "username", label: "Username" },
{ name: "email", label: "Email" },
{ name: "phone", label: "Phone" }
];
const options = {
filterType: "dropdown",
responsive: "scroll",
onRowClick: (rowData) => this.props.history.push('/user/create')
};
return (
  <MUIDataTable
    title={"Test"}
    data={this.state.users}
    columns={columns}
    options={options}
  />
);}}

const mapDispatchToProps = (dispatch) => {
return {
onStoreSelectedUser: (rowData) => dispatch({type:actionTypes.STORE_SELECTED_USER,
payload:rowData})
};
};
export default connect(null, mapDispatchToProps)(withRouter(UserList));
Your Environment

Tech
Version

Material-UI
^3.9.2

MUI-datatables
^2.0.0-beta-54

React
^16.8.1

browser
Chrome Version 72.0.3626.119

etc

The short answer is that you need to fire onStoreSelectedUser from onRowClick and pass it whatever you want.
Redux can be quite complicated, and this seems like a redux issue, so I'm not sure that this is the best forum for your question. It's probably going to be more helpful for you to seek help from redux-specific forums or stack overflow questions. As such, I have to mark this as invalid since it really doesn't pertain to this library.

Thanks for answering Gabriel !!
let's forget Redux
How can I pass the data of the selected row to the other component?
onRowClick: (rowData) => this.props.history.push ('/ user / create', rowData)
Is this the right way?

Another question, I need to move on to the next component and also fire my redux function if I do this ..
onRowClick: () => this.props.history.push ('/ user / create'),
onRowClick: (rowData) => this.props.onStoreSelectedUser (rowData)
He gives me the following warnings:
Line 51: Duplicate key 'onRowClick' no-dupe-keys

Thank you very much Grabriel for answering!
It works for me correctly, I just had to do this:
const options = {
onRowClick: (rowData) => {
this.props.onStoreSelectedUser (rowData);
this.props.history.push ('/ user / create');
},
};
I didn't know how I could combine the two functions within onRowClick. I must have elaborated the question badly,  for what you thought it had to do with redux.


Eslint 9

This PR gets rid of old cruft and adds dependencies to eslint 9 and latest prettier.
Closes https://github.com/victronenergy/venus-influx-loader/issues/177

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add fact to facts.json

Added a new fact regarding first time open source contributor in json file.
Short description of what this resolves:
Contributes fact to #414
Changes proposed in this pull request:
Added a fact for first time open source contributor!

Fixes: #

I am excited to get my first PR merged in master.
Can't wait to contribute more to open source!!


Exception: TFManager already started on worker1, ppid=25378, state='running'

hello, thank you for your work to open tensorflow on spark.
I can run the example where num-executors=2, but when num-executors= 4, I get some errors:

Exception: TFManager already started on worker2, ppid=7751, state='running'
Exception: TFManager already started on worker1, ppid=25378, state='running'
Exception: Duplicate cluster node id detected (host=worker1, ppid=25378).  Please ensure that the number of executors >= number of TensorFlow nodes, and TFCluster.shutdown() is successfully invoked when done.

spark-submit \

--master yarn
--deploy-mode client
--queue default
--num-executors 4
--executor-cores 1
--executor-memory 600M
--py-files tfspark.zip,examples/mnist/spark/mnist_dist.py
--conf spark.executorEnv.LD_LIBRARY_PATH="$JAVA_HOME/jre/lib/amd64/server"
--conf spark.dynamicAllocation.enabled=false
--conf spark.yarn.maxAppAttempts=1
examples/mnist/spark/mnist_spark.py
--images mnist/csv/train/images
--labels mnist/csv/train/labels
--mode train
--model mnist_model

17/03/03 10:48:17 INFO YarnSchedulerBackend$YarnDriverEndpoint: Launching task 2 on executor id: 1 hostname: worker1.
17/03/03 10:48:17 INFO TaskSetManager: Finished task 0.0 in stage 0.0 (TID 0) in 11836 ms on worker1 (1/4)
17/03/03 10:48:17 INFO TaskSetManager: Starting task 3.0 in stage 0.0 (TID 3, worker2, partition 3, PROCESS_LOCAL, 5391 bytes)
17/03/03 10:48:17 INFO YarnSchedulerBackend$YarnDriverEndpoint: Launching task 3 on executor id: 2 hostname: worker2.
17/03/03 10:48:17 INFO TaskSetManager: Finished task 1.0 in stage 0.0 (TID 1) in 11927 ms on worker2 (2/4)
17/03/03 10:48:18 WARN TaskSetManager: Lost task 3.0 in stage 0.0 (TID 3, worker2): org.apache.spark.api.python.PythonException: Traceback (most recent call last):
File "/data/hadoop/tmp/nm-local-dir/usercache/spark/appcache/application_1488455522150_0012/container_1488455522150_0012_01_000003/pyspark.zip/pyspark/worker.py", line 172, in main
process()
File "/data/hadoop/tmp/nm-local-dir/usercache/spark/appcache/application_1488455522150_0012/container_1488455522150_0012_01_000003/pyspark.zip/pyspark/worker.py", line 167, in process
serializer.dump_stream(func(split_index, iterator), outfile)
File "/usr/local/spark/python/lib/pyspark.zip/pyspark/rdd.py", line 317, in func
File "/home/spark/TensorFlowOnSpark/tfspark.zip/com/yahoo/ml/tf/TFSparkNode.py", line 91, in _reserve
Exception: TFManager already started on worker2, ppid=7751, state='running'
    at org.apache.spark.api.python.PythonRunner$$anon$1.read(PythonRDD.scala:193)
    at org.apache.spark.api.python.PythonRunner$$anon$1.<init>(PythonRDD.scala:234)
    at org.apache.spark.api.python.PythonRunner.compute(PythonRDD.scala:152)
    at org.apache.spark.api.python.PythonRDD.compute(PythonRDD.scala:63)
    at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:319)
    at org.apache.spark.rdd.RDD.iterator(RDD.scala:283)
    at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:70)
    at org.apache.spark.scheduler.Task.run(Task.scala:86)
    at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:274)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)

17/03/03 10:48:18 INFO TaskSetManager: Starting task 3.1 in stage 0.0 (TID 4, worker2, partition 3, PROCESS_LOCAL, 5391 bytes)
17/03/03 10:48:18 INFO YarnSchedulerBackend$YarnDriverEndpoint: Launching task 4 on executor id: 2 hostname: worker2.
17/03/03 10:48:18 WARN TaskSetManager: Lost task 2.0 in stage 0.0 (TID 2, worker1): org.apache.spark.api.python.PythonException: Traceback (most recent call last):
File "/data/hadoop/tmp/nm-local-dir/usercache/spark/appcache/application_1488455522150_0012/container_1488455522150_0012_01_000002/pyspark.zip/pyspark/worker.py", line 172, in main
process()
File "/data/hadoop/tmp/nm-local-dir/usercache/spark/appcache/application_1488455522150_0012/container_1488455522150_0012_01_000002/pyspark.zip/pyspark/worker.py", line 167, in process
serializer.dump_stream(func(split_index, iterator), outfile)
File "/usr/local/spark/python/lib/pyspark.zip/pyspark/rdd.py", line 317, in func
File "/home/spark/TensorFlowOnSpark/tfspark.zip/com/yahoo/ml/tf/TFSparkNode.py", line 91, in _reserve
Exception: TFManager already started on worker1, ppid=25378, state='running'
    at org.apache.spark.api.python.PythonRunner$$anon$1.read(PythonRDD.scala:193)
    at org.apache.spark.api.python.PythonRunner$$anon$1.<init>(PythonRDD.scala:234)
    at org.apache.spark.api.python.PythonRunner.compute(PythonRDD.scala:152)
    at org.apache.spark.api.python.PythonRDD.compute(PythonRDD.scala:63)
    at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:319)
    at org.apache.spark.rdd.RDD.iterator(RDD.scala:283)
    at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:70)
    at org.apache.spark.scheduler.Task.run(Task.scala:86)
    at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:274)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)

17/03/03 10:48:18 INFO TaskSetManager: Starting task 2.1 in stage 0.0 (TID 5, worker1, partition 2, PROCESS_LOCAL, 5391 bytes)
17/03/03 10:48:18 INFO YarnSchedulerBackend$YarnDriverEndpoint: Launching task 5 on executor id: 1 hostname: worker1.
17/03/03 10:48:28 INFO TaskSetManager: Finished task 3.1 in stage 0.0 (TID 4) in 10163 ms on worker2 (3/4)
17/03/03 10:48:28 INFO TaskSetManager: Finished task 2.1 in stage 0.0 (TID 5) in 10111 ms on worker1 (4/4)
17/03/03 10:48:28 INFO YarnScheduler: Removed TaskSet 0.0, whose tasks have all completed, from pool
17/03/03 10:48:28 INFO DAGScheduler: ResultStage 0 (collect at /home/spark/TensorFlowOnSpark/tfspark.zip/com/yahoo/ml/tf/TFCluster.py:189) finished in 22.397 s
17/03/03 10:48:28 INFO DAGScheduler: Job 0 finished: collect at /home/spark/TensorFlowOnSpark/tfspark.zip/com/yahoo/ml/tf/TFCluster.py:189, took 22.526369 s
{'addr': ('worker1', 33132), 'task_index': 0, 'port': 50483, 'authkey': UUID('16300861-aa35-4fe3-b775-a90cfa9e5883'), 'worker_num': 0, 'host': 'worker1', 'ppid': 25378, 'job_name': 'ps', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-8w7uN9/listener-O49cY4', 'task_index': 0, 'port': 50548, 'authkey': UUID('c3d5da9a-b35f-4d0b-9d0f-e8986bacb7da'), 'worker_num': 1, 'host': 'worker2', 'ppid': 7751, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-NBT2JR/listener-DzP5fR', 'task_index': 1, 'port': 49292, 'authkey': UUID('a988596d-16a8-44f8-b173-e9b8e0f3e4c5'), 'worker_num': 2, 'host': 'worker1', 'ppid': 25378, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-V2AMeX/listener-rtKT5z', 'task_index': 2, 'port': 58283, 'authkey': UUID('fb48c221-c612-4f43-b5e1-c6a8be1a5eb6'), 'worker_num': 3, 'host': 'worker2', 'ppid': 7751, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}
Traceback (most recent call last):
File "/home/spark/TensorFlowOnSpark/examples/mnist/spark/mnist_spark.py", line 70, in
cluster = TFCluster.reserve(sc, args.cluster_size, num_ps, args.tensorboard, TFCluster.InputMode.SPARK)
File "/home/spark/TensorFlowOnSpark/tfspark.zip/com/yahoo/ml/tf/TFCluster.py", line 207, in reserve
Exception: Duplicate cluster node id detected (host=worker1, ppid=25378).  Please ensure that the number of executors >= number of TensorFlow nodes, and TFCluster.shutdown() is successfully invoked when done.
17/03/03 10:48:28 INFO SparkContext: Invoking stop() from shutdown hook
17/03/03 10:48:28 INFO ServerConnector: Stopped ServerConnector@49c82a94{HTTP/1.1}{<IP_ADDRESS>:4040}

@dataecho the main problem appears here in the cluster_info:
{'addr': ('worker1', 33132), 'task_index': 0, 'port': 50483, 'authkey': UUID('16300861-aa35-4fe3-b775-a90cfa9e5883'), 'worker_num': 0, 'host': 'worker1', 'ppid': 25378, 'job_name': 'ps', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-8w7uN9/listener-O49cY4', 'task_index': 0, 'port': 50548, 'authkey': UUID('c3d5da9a-b35f-4d0b-9d0f-e8986bacb7da'), 'worker_num': 1, 'host': 'worker2', 'ppid': 7751, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-NBT2JR/listener-DzP5fR', 'task_index': 1, 'port': 49292, 'authkey': UUID('a988596d-16a8-44f8-b173-e9b8e0f3e4c5'), 'worker_num': 2, 'host': 'worker1', 'ppid': 25378, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}
{'addr': '/tmp/pymp-V2AMeX/listener-rtKT5z', 'task_index': 2, 'port': 58283, 'authkey': UUID('fb48c221-c612-4f43-b5e1-c6a8be1a5eb6'), 'worker_num': 3, 'host': 'worker2', 'ppid': 7751, 'job_name': 'worker', 'tb_pid': 0, 'tb_port': 0}

Again, we expect each executor to host only one TF node.  And from this, it appears that you have two physical hosts (worker1, worker2), but only two executor processes (ppid=7751, 25378) that ran the reserve tasks.  Are you sure that you're actually getting the 4 executors that you requested?  Perhaps you need to reduce --executor-memory to "fit" more executors onto the same host (e.g. 600M for 2 executors => 300M for 4 executors)?
Note: I've seen cases where YARN can actually return fewer than requested hosts, and Spark will attempt to proceed...

@leewyang  is TFOnSpark can't run on more executors? if a node has lanuch more executors,that will throw error like this issue

@ali8zake the issue is not not related to running multiple executors on the same node, but rather the "reservation" task not being distributed evenly across all these executors.
Currently, there's a simple time.sleep() statement to force Spark to distribute these to all executors, but I think it may not be long enough for all the various setups folks are using.  You can try increasing the sleep amount (as a possible short term fix... I'm looking into a more deterministic alternative in the meantime).

@leewyang thank you for your advice. That's the problem of yarn, yarn return fewer than requested hosts, after changing yarn's config, I solved the problem.

@dataecho  which yarn config，can you expain it ,thanks.


Update Zinc used by Scala plugin to 1.7.1

Zinc 1.7.1 includes a number of bug fixes including under-compilation when Java annotation changes.
Release notes:

https://github.com/sbt/zinc/releases/tag/v1.7.0
https://github.com/sbt/zinc/releases/tag/v1.7.1

Diff: https://github.com/sbt/zinc/compare/v1.6.1...v1.7.1

We're now up to 1.8.0

We have a PR to update to the latest - Zinc 1.9.3

#25887

Any reason to be cautious?


How do you add Custom Domains for Azure Functions on Consumption plan (Y1)?

From my Azure function I can get to it's underlying/parent App Service plan, but no Custom Domains link is provided on the left Nav bar in the Azure Portal.
This section of this document specifically mentions Azure Function Apps in a consumption plan but the subsequent steps are not valid, it seems
"Prepare the app
To map a custom DNS name to a web app, the web app's App Service plan must be a paid tier (Shared, Basic, Standard, Premium or Consumption for Azure Functions). In this step, you make sure that the App Service app is in the supported pricing tier"
After much external searching (e.g. stackoverflow), I found the answer is here: https://docs.microsoft.com/en-us/azure/app-service/scripts/cli-configure-custom-domain
Clarification on the difference between App Service / Web Apps versus Function Apps (outside of a paid App Service plan) would be helpful along with a link to the cli-configure-custom-domain article.
I.e. you have to use Azure CLI for Function Apps in a consumption plan.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 224b73dd-c59d-a833-0d4d-a778af6086a9
Version Independent ID: c6b167c9-9589-45d0-ee6a-7a47a4aec41e
Content: Tutorial: Map existing custom DNS name - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-custom-domain.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hi @DTronD, thank you for the detailed question. We are currently investigating and will update you when we have accurate information to provide.

@DtronD I just looked and I see "Custom domains" listed in a consumption function app, in the Platform features tab. Please check that you're actually looking at the settings for the function app, not for its App Service plan. Custom domains are not plan settings.
@ggailey777 @craigshoemaker FYI
#please-close

Thanks, @cephalin.  Looking now on the Platform Features for the Function does show Custom Domains.  But, I still think the doc should be modified as it specifically directs users to the left navigation....which does NOT contain the Custom Domains menu item so described for Web Apps.  A line or two that mentions the different location for Function Apps versus Web Apps would be very helpful to more than just me, I think.  Please.


hardware

did you just get the starter pack ?
https://www.amazon.de/nanoleaf-NL22-0002TW-9PK-Light-Panels-Lichtpanels/dp/B01M0W7NIP/ref=lp_7465101031_1_1
Or maybe you needed some controller to be able to interact with it ?

I have a "Starter" and "Expansion" for a total of 12 panes. As long as you have the controller/power cord that comes with the starter kit, you should be fine.

thanks you
On Sat, 12 May 2018 at 14:29 Nick Gerakines<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/ngerakines/auroraops/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ngerakines/auroraops/issues/1#event-1623175471, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATuCwg2lCIWvDI4VYua2c5pZXqzYS1qZks5txtWPgaJpZM4T8V9G
.


Detected index js file change. Contact me to check if it's safe to continue: https://t.me/scripts_hub

Detected index js file change. Contact me to check if it's safe to continue: https://t.me/scripts_hub

Fixed, check.


Fix MySQL remove_index

Fixed bug about MySQL adapter for mysql-use-alter option.

Rename remove_index! to remove_index because remove_index! hasn't been called.
Use index_name_for_remove to get index_name, just like ActiveRecord.

Related line:
https://github.com/rails/rails/blob/master/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb#L795

Thank you! 👍


lib/parser.js formatPRIVMSG: Cannot read property 'split' of undefined

C:\Users\TheBlackParrot\Documents\Share\twitch-plays-bj2\node_modules\twitch-bot\lib\parser.js:33
    parsed.username = username.split('!')[0]
                               ^

TypeError: Cannot read property 'split' of undefined
    at Object.formatPRIVMSG (C:\Users\TheBlackParrot\Documents\Share\twitch-plays-bj2\node_modules\twitch-bot\lib\parser.js:33:32)
    at TLSSocket.irc.on.data (C:\Users\TheBlackParrot\Documents\Share\twitch-plays-bj2\node_modules\twitch-bot\lib\bot.js:77:32)
    at emitOne (events.js:116:13)
    at TLSSocket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:246:13)
    at TLSSocket.Readable.push (_stream_readable.js:208:10)
    at TLSWrap.onread (net.js:607:20)

Any chance to get more information about the version and the input that failed to be parsed?

Any updates on this @TheBlackParrot ?

Haven't seen it happen again recently, and I had no logs at the time. I've got it running under PM2 now so if it happens again I should be able to catch logs of what was going on.
Is there a debug output I can enable if it happens again?

I had this happen today:
...\node_modules\twitch-bot\lib\parser.js:33
    parsed.username = username.split('!')[0]
                               ^

TypeError: Cannot read property 'split' of undefined
    at Object.formatPRIVMSG (C:\...\node_modules\twitch-bot\lib\parser.js:33:32)
    at TLSSocket.irc.on.data (C:\...\node_modules\twitch-bot\lib\bot.js:77:32)
    at emitOne (events.js:116:13)
    at TLSSocket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:246:13)
    at TLSSocket.Readable.push (_stream_readable.js:208:10)
    at TLSWrap.onread (net.js:597:20)

I got this error when someone "raided" (fucntionality of twitch) my stream, hopfuly this info is helpful.

What version of the bot are you using? You can find out this information by looking at the version no. in your package.json file of your project.

@kritzware Version 1.2.5

I get the same error when anyone hosts me
version:  "twitch-bot": "^1.2.5"

Same error here "version": "1.2.5"

    | /app/node_modules/twitch-bot/lib/parser.js:33 server_1    |     parsed.username = username.split('!')[0] server_1    |                                ^ server_1    | server_1    | TypeError: Cannot read property 'split' of undefined server_1    |     at Object.formatPRIVMSG (/app/node_modules/twitch-bot/lib/parser.js:33:32) server_1    |     at TLSSocket.<anonymous> (/app/node_modules/twitch-bot/lib/bot.js:76:32) server_1    |     at TLSSocket.emit (events.js:210:5) server_1    |     at addChunk (_stream_readable.js:308:12) server_1    |     at readableAddChunk (_stream_readable.js:285:13) server_1    |     at TLSSocket.Readable.push (_stream_readable.js:223:10) server_1    |     at TLSWrap.onStreamRead (internal/stream_base_commons.js:182:23)
Has the same issiue and whole node crashed on it......
Just check after split if there is an index 0 not get it straight.
For simple fix try parsed.username = username.split('!')[0] || ''
I will try to fix it when i got more time...
Best Dan


Add footer

description
Add footer

can u please elaborate , where to add footer

Also do specify the contents of the footer.

contents we can dd to Footer are @avinal @ankitapuri  :
1)Find us
2)Call us
3)Mail us
4)Website logo
5)What this website is about
6)Follow us
-Instagram link with logo
-LinkedIn link with logo
-Twitter link with logo
-etc
7)Useful links
-Home
-Services
-About Us
-Contact Us etc
8)Subscribe
-Email address
9)Copyrights
10) All the above with responsive UI

Hi , can i work on this issue?

I want to work on this issue.

I myself want to work on this, hence made the issue :)

any update on this?  @ankitapuri

We don't need so detailed footer . It will occupy so much space . Moreover , we already have one PR for the same for the login page .


K8s 1.27

This PR attempts to address the different incompatibilites from the dependabot PRs regarding K8s 1.27.

controller-runtime don't follow semver, and there were API breakage introduced between v0.14.6 and v0.15.0-beta.0 (used in this PR)

controller-runtime don't follow semver, and there were API breakage introduced between v0.14.6 and v0.15.0-beta.0 (used in this PR)

aha, that's annoying..


[tutorial] Improve "next steps" at end of tutorial

Description (required)
This PR fixes the duplicate "next steps" at the end of the tutorial, and makes them prettier with the <LinkCard> components!

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
tutorial/6-islands/3.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

!coauthor


HIVE-27300: Upgrade Parquet to 1.13.0

What changes were proposed in this pull request?
Upgrading Parquet to 1.13.0

Why are the changes needed?
Parquet 1.12.x has issues with buffers nog being closed properly.

Does this PR introduce any user-facing change?

How was this patch tested?
Compiled locally

If the test executes then it will be ok. cc: @akshat0395 @deniskuzZ


Set the XMLInputFactory property for RDF files

Fixes #90
See Jena issue 2331 for more information:
https://issues.apache.org/jira/browse/JENA-2331
Signed-off-by: Gary O'Neall<EMAIL_ADDRESS>

@zvr I think this fixes the first error message you see when running the tools with any RDF file.
If you wouldn't mind a quick review - let me know if you see any issues with this approach.

Looks fine, but I am not really qualified to review this code.


Introducing Ring Door Bell Camera (including StickUp cameras) and WiFi sensors

Description:
Extended Ring DoorBell to support camera (including stickup cameras) playback and wifi sensors

Bump python-ringdoorbell to version 0.1.6

Support to camera playback via ffmpeg (ring doorbell and ring stickup camera)

Extended ringdoorbell sensors to report WiFi attributes

Extended unittests
This PR also change the configuration making all monitored_conditions enabled if not specified. Not sure if this is considered a breaking change.

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3682
Example entry for configuration.yaml (if applicable):
#configuration.yaml
ring:
  username: foo
  password: bar

binary_sensor:
   - platform: ring

camera:
  - platform: ring

sensor:
   - platform: ring

ffmpeg:

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Thanks @pvizeli
I'm going to work on the changes requested tonight.

Please address the lint and we can merge it

@pvizeli  thanks for your review. It seems the lint test failed due to timeout on Travis.
It passed to me on my personal branch https://travis-ci.org/tchellomello/home-assistant/builds/290744227

@tchellomello thank you for this PR! I have been waiting for something like this. I was wondering, do you think it might be a good idea to add some kind of event to notify HA that there is a new video to download?

@etsinko I've created another sensor with Ring video URL and use that to trigger automation. May be we can have an event fired in case of new video. I love that idea

@arsaboo which sensor do you use, template_sensor? And use an automation that triggers on state change?

@etsinko Yes, I use a template sensor for the Ring video URL and trigger an automation based on that.

Please don't use merged PRs for discussion. Take it in the forum or chat. If there's a bug, open an issue. Thanks!


More supported languages

languages.yml in the GitHub linguist project has a good list of languages but I'll need to look up and register how strings and comments are written in each language.
Maybe having a similar languages.yml file and then deriving the scanner configuration from that would be the best way to maintain it. Right now the code is just statically written which may be a bit cumbersome.
Top 250 languages on GitHub
This is based on the BigQuery public data table. Many important file formats are missing from this (e.g. YAML, TOML, etc). I think this is because while they are in many many repos the data only includes the languages that make up a large % of the repo contents and config files are using just a small part.
Probably I'll call this good after supporting the top 100 or so.

[x] JavaScript
[x] Python
[x] Java
[x] PHP
[x] Ruby
[x] C++
[x] HTML
[x] C
[x] CSS
[x] C#
[x] Shell
[x] Go
[x] Objective-C
[x] Swift
[x] TypeScript
[x] Haskell
[x] CoffeeScript (#702)
[x] Scala
[ ] Clojure
[x] Lua
[x] Perl
[x] R
[x] Rust
[ ] Jupyter Notebook
[ ] TeX
[ ] Arduino
[ ] Matlab
[ ] PowerShell
[x] Erlang
[ ] Groovy
[x] Makefile
[ ] Emacs Lisp
[ ] VimL
[ ] Puppet
[ ] Kotlin
[ ] Elixir
[ ] Julia
[ ] Vim script
[ ] Dart
[ ] Visual Basic
[x] Assembly
[ ] OCaml
[ ] Common Lisp
[ ] F#
[ ] Processing
[ ] ActionScript
[ ] ApacheConf
[ ] Pascal
[ ] CMake
[ ] Vue
[x] Dockerfile
[ ] XSLT
[ ] D
[ ] Haxe
[ ] Scheme
[ ] Batchfile
[ ] Elm
[ ] Eagle
[ ] SCSS
[ ] QML
[ ] MATLAB
[ ] FORTRAN
[ ] Racket
[ ] Verilog
[ ] ASP
[ ] VHDL
[ ] Cuda
[ ] Vala
[ ] Crystal
[ ] Fortran
[ ] HCL
[ ] Nginx
[ ] Prolog
[ ] OpenSCAD
[ ] Groff
[ ] ColdFusion
[ ] PLpgSQL
[ ] Tcl
[ ] AutoHotkey
[ ] PureScript
[ ] PLSQL
[ ] Mathematica
[ ] KiCad
[ ] Smarty
[ ] Objective-C++
[ ] Smalltalk
[ ] Vim Script
[ ] Perl6
[ ] Apex
[ ] SourcePawn
[ ] SQF
[ ] SaltStack
[ ] BitBake
[ ] AppleScript
[ ] Logos
[ ] LiveScript
[ ] GLSL
[ ] Roff
[ ] PostScript
[ ] Nimrod
[ ] GDScript
[ ] AutoIt
[ ] DM
[ ] IDL
[ ] Coq
[ ] OpenEdge ABL
[ ] Ada
[ ] Nim
[x] XML
[ ] LabVIEW
[ ] Max
[ ] Perl 6
[ ] Handlebars
[ ] SQLPL
[ ] Web Ontology Language
[ ] Pure Data
[ ] Standard ML
[ ] Squirrel
[ ] SuperCollider
[ ] Protocol Buffer
[ ] Nix
[ ] Jinja
[ ] Stata
[ ] Awk
[ ] Xtend
[ ] TSQL
[ ] NSIS
[ ] M
[ ] M4
[ ] SystemVerilog
[ ] Hack
[ ] API Blueprint
[ ] LookML
[ ] FreeMarker
[ ] Game Maker Language
[ ] XQuery
[ ] Idris
[ ] Forth
[ ] DIGITAL Command Language
[ ] Liquid
[ ] Yacc
[ ] Gettext Catalog
[ ] Agda
[ ] GAP
[ ] GCC Machine Description
[ ] AGS Script
[ ] Less
[ ] MoonScript
[ ] BlitzBasic
[ ] PAWN
[ ] QMake
[ ] ABAP
[ ] CartoCSS
[ ] nesC
[ ] Scilab
[ ] Cucumber
[ ] Ceylon
[ ] ANTLR
[ ] Raku
[ ] PureBasic
[ ] NetLogo
[ ] xBase
[ ] Objective-J
[ ] Gnuplot
[ ] SAS
[ ] Gosu
[ ] ooc
[ ] Delphi
[ ] Visual Basic .NET
[ ] Gherkin
[ ] Brainfuck
[ ] Starlark
[ ] LilyPond
[ ] Monkey
[ ] UnrealScript
[ ] Lex
[ ] Rebol
[ ] Eiffel
[ ] AMPL
[ ] Inno Setup
[ ] Rich Text Format
[ ] Propeller Spin
[ ] Xojo
[x] SQL
[ ] Mercury
[ ] ShaderLab
[ ] 1C Enterprise
[ ] Diff
[ ] Brightscript
[ ] COBOL
[ ] RobotFramework
[ ] Modelica
[ ] Dylan
[ ] Isabelle
[ ] RAML
[ ] CLIPS
[ ] KiCad Layout
[ ] Bro
[ ] LSL
[ ] Io
[ ] Lasso
[ ] Hy
[ ] Smali
[ ] Bison
[ ] LLVM
[ ] Twig
[ ] VBA
[ ] Pony
[ ] Cython
[ ] Frege
[ ] DOT
[ ] Ren'Py
[ ] Pug
[ ] AspectJ
[ ] IGOR Pro
[ ] MAXScript
[ ] Thrift
[ ] Vim Snippet
[ ] J
[ ] Papyrus
[ ] G-code
[ ] Sass
[ ] Stylus
[ ] EJS
[ ] PigLatin
[ ] VCL
[ ] POV-Ray SDL
[ ] SMT
[ ] Augeas
[ ] Mako
[ ] Parrot
[ ] HLSL
[ ] Lean
[ ] Jsonnet
[ ] Pawn
[ ] Component Pascal
[ ] Uno
[ ] Nunjucks
[ ] Boo
[ ] Slash
[ ] Bluespec
[ ] Nemerle
[ ] Pan
[ ] Clean
[ ] Rascal
[ ] Blade
[ ] q
[ ] ATS
[ ] ChucK
[ ] Inform 7

Leasot has a supported languages list: https://github.com/pgilad/leasot/blob/master/media/supported-languages.md

Need to rework languages so they each have their own parser to support idiosyncrasies in how some languages are parsed.
For example VimL uses quote characters (") for both strings and line comments. A parser would need to determine the difference between the two.

Will close once there are a total of 50 supported languages.


XML handling

We should stop rely on plain string templates for domain XML generation. Instead we should fill initial values in structs, operate on them and only once at the end generate from this XML, cutting down need to parse it at first.
So this is placeholder issue for future refactor.

As a follow-up of libvirt GO bindings there has come up another project with libvirt Go XML parser from the same author: https://github.com/libvirt/libvirt-go-xml. I think, this will be the answer to our XML handling issue.

Other options which may be tried just in case

Automatically convert libvirt domain description (http://libvirt.org/schemas/) to go structs, using

https://github.com/gnewton/chidley  converts any XML to Go structs (and therefor to JSON)
https://github.com/metaleap/go-xsd

Use XPATH-like access  launchpad.net/xmlpath (https://godoc.org/gopkg.in/xmlpath.v1)
Use DOM-like style, requires CGO (like Nokogiri - ruby-based) https://github.com/moovweb/gokogiri - LibXML bindings for the Go programming language.