[Integration Tests] Improve and fix integration tests

Motivation
Integration test are basically just not working in apache CI. there are multiple reasons:

PULSAR_MEM is set to very high (~2G). If we start more than 6 containers, it will quickly go up to ~12GB.
We start containers for each tests, which is very inefficient.
There is a regression from #2214. #2214 tried to listen on a hostname that is configured in worker config,
which the hostname is internally to docker environment.

Changes

Set PULSAR_MEM to use no more than 128M.
Switch to use test suite, trying to start containers only once as possible and use them across test suites.
so we only start the cluster for a set of tests. hence reorganize those tests and get rid
of using dataProvider, which doesn't work well with current approach.
Remove binding to hostname in WorkerServer, so it binds to all interfaces.

@rdhabalia there is a regression from #2214 causing integration tests to fail. I fixed that in this PR, so please take a look.

Unfortunately I can’t break this down because all these are coupled, if you don’t pull these together, the integration tests never pass.

run java8 tests


add tenant switch support in the sign-in area

Is your feature request related to a problem? Please describe.
Today if I use the graph explorer to GET /beta/servicePrincipals, it will return only the service principals on the main tenant I'm part of. But I might have created other service principals on other tenants using the Azure Portal for example.
Whenever I'm writing an application, I can switch tenant by requesting different tokens to MIP. When using Azure I can switch tenant from the portal's UI. It'd be interesting to allow users to switch between the tenants they are part of so they can access different datasets.
Describe the solution you'd like
Next to the profile card, in the little gear, a "switch tenant" button that lists out the tenant you are part of. Click on any tenant and you are redirected to complete auth on that specific tenant instead.

Describe alternatives you've considered
No alternatives possible using the graph explorer, using Postman collections or hand-written code is the only option at this point.
AB#7492

@baywet might #568 have solved this?

@thewahome I believe it partially does, all what's missing now is to query the same API the azure portal uses to get the list of tenants I'm part of and add some kind of UI switch. Or at least make it discoverable so people know it exists.

@RabebOthmani do you think there's more that should be done on this issue or should it be closed?


Leak in call to __cxa_demangle if it fails

https://github.com/bombela/backward-cpp/blob/aa3f253efc7281148e9159eda52b851339fe949e/backward.hpp#L505-L515
If __cxa_demangle fails here then result is NULL and the buffer is leaked.

https://gcc.gnu.org/onlinedocs/libstdc++/libstdc++-html-USERS-4.3/a01696.html
Not really clear about what happens if it returns NULL while an existing buffer was passed. Especially since the function can call realloc at will. I will look at the implemention!
thanks for the finding

https://github.com/llvm-mirror/libcxxabi/blob/master/src/cxa_demangle.cpp#L344
Definitively a leak. Will put up a fix.

This was fixed by #157


Data limit of each member

Data limit of each member is NOT 1024MB if there are less than 10 members in the plan.

Fixing this bug in Pull Request #5
The issue will be closed when the PR is merged into master branch.


mtools: 4.0.33 -> 4.0.34

Motivation for this change
Fix bug for recursive use, see release announcement: https://lists.gnu.org/archive/html/info-gnu/2021-07/msg00010.html
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132211 at 7887182e run on aarch64-linux 1

4 packages marked as broken and skipped:

bootiso
unetbootin
wimlib
woeusb

7 packages built successfully:

fwanalyzer
fwup
gnome.gnome-boxes
ipxe
memtest86-efi
mtools
multibootusb

2 suggestions:

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/tools/filesystems/mtools/default.nix:23:5:
   |
23 |     license = licenses.gpl3;
   |     ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/tools/filesystems/mtools/default.nix:19:3:
   |
19 |   meta = with lib; {
   |   ^

Result of nixpkgs-review pr 132211 run on x86_64-linux 1

1 package marked as broken and skipped:

multibootusb

3 packages failed to build:

bootiso
wimlib
woeusb

8 packages built:

fwanalyzer
fwup
gnome.gnome-boxes
ipxe
memtest86-efi
mtools
syslinux
unetbootin

Result of nixpkgs-review pr 132211 run on x86_64-linux 1

1 package marked as broken and skipped:

multibootusb

3 packages failed to build:

bootiso
wimlib
woeusb

8 packages built:

fwanalyzer
fwup
gnome.gnome-boxes
ipxe
memtest86-efi
mtools
syslinux
unetbootin

packages failed to build:

I have no idea what this means. Were they not building before this change? Where’s the build logs?
I’m just going to merge for now, the package seems to build just fine.


Personalization

Dashed Lines / Outlines
It would be nice to add a way to set the outlines of the features as dashed lines with a custom array like
https://maplibre.org/maplibre-style-spec/layers/#paint-line-line-dasharray
Cursor style
An option to let the user choose which cursor style wants. This because I think is better to have the move style

when hovering a selected future in select mode. Also, to display this style ONLY when hovering the selected feature.

@alanOracle thanks for suggesting - I will see how realistic this would be when I get some time. One thing I've tried to ensure is approximate uniformity in styling across the different providers so would want to try and keep this property where possible. Let me see what I can do!

I will close this of favour of individual issues that can be tracked #48 and #49


IntelliJ and fastlane integration

Question Subject
Installed IntelliJ IDEA Ultimate 2019.1 with Ruby support
Question Description
Is it possible to configure intellij to recognize fastlane script as a ruby properly?
expected:

all actions should be properly mapped to the source code, Cmd+Click should navigate developer to implementation
private_lane and lane declaration should be recognized by IDE as methods
debugger support
(Optional) Support unit tests for fastlane script

the max I achieve so far is - https://medium.com/@olku/fastlane-setup-in-intellij-enterprise-ffb56051065a

That's an interesting question.
I have no idea, but my guess would be that the way fastlane works does not make it easy for any IDE to properly connect the action names to the code that is executed behind the scenes automatically - there is quite a lot of magic involved to go from the "method" call to the executed code.


Implement method GetAllInfo() and key --info-all to request informations for all instruments with the same types

Save response to .xlsx file

Duplicated by #46


Add no-store configuration option for UEFI

We don't need OSSL_STORE in the firmware. Add an option to disable it.
cf<EMAIL_ADDRESS>

Shouldn't the UEFI target have it disabled by default, then? I.e an added disable attribute?
        disable         => [ 'store' ],

I imagine more could be added in such an attribute. Is the openssl command interesting at all for firmwares? I would disable 'apps' as well (not available for the configuration command line, but if you look in apps/build.info, you will see that someone's listening)

Finally, I gotta ask, what else does UEFI not need? PEM? crypto/x509/by_*.c? The more I think about it, the more it looks like a huge slashing job...

Shouldn't the UEFI target have it disabled by default, then? I.e an added disable attribute?
I imagine more could be added in such an attribute…

How much do you really want to know? :)
Intel's EDK2 is the open source implementation of the UEFI core which most of the production UEFI firmware in the world is based on. Its build system is… baroque. I stopped short of trying to beat the OpenSSL Makefiles to work with it. Instead the EDK2 build system runs Configure and then extracts the file list from OpenSSL's config data to create the OpensslLib.inf file for its own build system. It only does so for crypto/ and ssl/ and doesn't do apps. You can see the gory details at https://github.com/tianocore/edk2/blob/master/CryptoPkg/Library/OpensslLib/process_files.pl
So yes, there's a bunch of stuff which could be automatically disabled in the UEFI target, but it's also slightly redundant to do so.

Finally, I gotta ask, what else does UEFI not need? PEM? crypto/x509/by_*.c? The more I think about it, the more it looks like a huge slashing job...

The "huge slashing job" was when I went through and made OPENSSL_NO_STDIO/OPENSSL_NO_FP work :)
For a lot of things, if they're in standalone C files and not directly referenced from elsewhere, they just drop out of the build completely. Store wasn't like that because parts of it are referenced from elsewhere — it all got pulled in through things like err_load_crypto_string_int() which referenced those C units.
As for crypto/x509/by_*.c I think those survive by virtue of the fact that the file-based BIOs still exist even in the OPENSSL_NO_STDIO build. There might be merit in hiding them, and some other bits, behind an OPENSSL_REALLY_NO_FILE_IO_AT_ALL_SO_DONT_EVEN_PRETEND but I'm not massively inclined to go there.
PEM does get used for certificates.

... it all got pulled in through things like err_load_crypto_string_int() which referenced those C units.

That's surprising, I would have thought that only crypto/store/store_err.c gets involved...  I haven't looked very closely at that one, though.
It sounds like we could do with some refactoring, dividing some files into smaller units.  That would be healthy anyway, gods know that we have some units that are way too large and contain way too many things at the same time.

That's surprising, I would have thought that only crypto/store/store_err.c gets involved... I haven't looked very closely at that one, though.

Yeah, I'm not sure it all got pulled in. I haven't looked at the precise trigger for them attempting to disable the store code, but it's a new directory full of code that they'd prefer to disable. It's also possible that it would have fallen out of the final link but actually failed to build in the first place, if there's (for example) a missing OPENSSL_NO_STDIO check around using BUFSIZ or something like that again.
Certainly the hacks on the UEFI build side (in the thread I replied to and added you to Cc) aren't the right way to do it. I think the STORE stuff fits squarely into the category of things that normally would have had no-xxx options to turn them off, and adding no-store is consistent and reasonable.
Now, it's entirely possible to go overboard and demand that we don't have a single machine instruction in the UEFI build of OpenSSL that isn't actively useful. That would be silly. I think we've settled on a relatively sane middle ground where we disable things that can be reasonably disabled, and live with the rest. Obviously, feedback on that strategy is welcome.
Honestly, I was glad that no-stdio already existed when I started trying to clean this stuff up, because then I was only "fixing" it and not trying to persuade you to take that as a new concept. :)

The way I would like store to go, it's as sensible to disable it as it is to disable, say, PEM.
It would make sense to disable selected loaders (such as the file loader).

Another way to put it is that it makes sense to disable back ends, it makes less sense to disable the framework...

Makes sense. I'll try an EDK2 build with it included and look at excluding specific back ends, and see what it looks like. Thanks.

Could you please rebase the PR to resolve conflicts?
Marking as inactive to be closed at the end of 3.4 dev barring further input.


Form Validation

Email should force .edu email accounts, not unomaha accounts.
Phone numbers should be validated to only be numbers.
Names should not have numbers.

Only phone numbers need to be fixed. Might require model change.


Docker Build for MADlib

JIRA: MADLIB-920

Add docker files that would help developers download a docker image
with Postgres-9.6 and MADlib depedencies installed. A developer's
local source code changes can be built on this image's container
to quickly build and run install-checks. Requires docker installed
on the developer's environment.
Add a bash script (jenkins_build.sh) that would be a starting
point towards getting a Jenkins build for MADlib master branch.
This is work under heavy development and we would want to add in
similar support for Greenplum as well in the future. There is a
placeholder Dockerfile for GPDB in this commit, must be modified
to get it working.

Looks good to me. I was able to run through the steps. Only comment would be to update documentation to reflect the madlib docker hub account.


fix(alert): add spacing only when onRequestClose is present

:tada: This PR is included in version 4.6.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rabbitmq example is not longer deployable with latest node.js buildpacks

-----> Installing binaries
engines.node (package.json): 8.x
engines.npm (package.json): unspecified (use default)
ERROR Unable to install node: no match found for 8.x in [10.18.1 10.19.0 12.16.0 12.16.1]
Failed to compile droplet: Failed to run all supply scripts: exit status 14
Need to fix node version in package.json

Thanks. Updated to the latest supported Node version.


[BUG] Power Automate Editor Import OpenAPI 3.0 - allOf not changed

Type of Connector
Custom Connector
Name of Connector
GitHub Gists
Describe the bug
I was able to successfully import the connector definition (after removing non-Gists-related definitions) from:
https://raw.githubusercontent.com/github/rest-api-description/main/descriptions/api.github.com/api.github.com.json
In the Swagger Editor, I can see that the definition has been downgraded to 2.0. However, the allOf keyword has still been included. So, I have two questions:

Does the Editor saving without an error mean that the custom connector can be used with the allOf parameters?
The paconn CLI validate command completes with warnings, but should these be fixed before submission as an Independent Publisher connector? (I may fix and submit anyways, as it is only two warnings)

Is this a security bug?
No, this is not a security bug
What is the severity of this bug?
Severity 3 - Minor issue
To Reproduce
Import OpenAPI definition that includes allOf keyword (I'm guessing this is similar for oneOf, anyOf, and not as well)
Expected behavior
Editor downgrade removes the need for allOf keyword
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build)
CLI version (paconn --version)
OS version
Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Additional context
None

Hi, is there a fix, or known workaround for this? I've had this same issue pop up when importing a 3.0 spec generated by swagger-php.


Example notebooks to not specify what version of pymc-marketing was used when last executed

The MMM example notebook, the Budget Allocation notebook, the Lift Test Calibration notebook, and likely others specify the versions of python, matplotlib, arviz, numpy, pandas,  pymc, and pytensor. Strangely, the version of pymc-marketing is not included. This makes it very difficult to know what features described in the notebook might require in terms of package versions.  The version of pymc-marketing seems much more important than the version of something like matplotlib.
Furthermore, the CLV quickstart notebook has no package versions listed at all.
The only notebooks I could find that include a pymc-marketing version are the Pareto/NBD Model notebook, the "Shifted Beta Geometric model with individual customer data" notebook, the "Custom Models with MMM components", and the "other samplers" notebook.

Good thinking. Weird because the watermark command seems to be the same and version of that package is the same. Likely best to just be explicit with pymc marketing.
How are these as steps?

[ ] ensure all notebooks have watermark
[ ] be explicit in watermark command at end of notebook


How can we do functional tests in CI/CD pipeline by mocking kubernetes resources for GO operators

How can we do functional tests in CI/CD pipeline by mocking kubernetes resources for GO operators. Apart from envTest, as no services/pod from inside the tests are created in envtest.
Question
What did you do?
Creating a GO operator for our project.
What did you expect to see?
functional end-to-end test for the operator and check the resources created
What did you see instead? Under which circumstances?
No able to check the resources created
Environment
Operator type:

/language go

Kubernetes cluster type:
Openshift
$ operator-sdk version
v1.0.0
$ go version (if language is Go)
1.15
$ kubectl version

Additional context

Apart from envTest, as no services/pod from inside the tests are created in envtest.

What do you mean by this? You can create any k8s object for which a scheme is registered with envtest using the *rest.Config returned by envtest.Environment.Start(). I recommend creating functional tests this way.
Alternatively you can create custom tests using scorecard custom tests, which requires your operator be running in a cluster beforehand. However custom tests should be used to test operator states and not necessarily end-to-end scenarios.
/triage support

can we manage a new image resource  pod created from inside the controller in the Reconcile method and communicate with it.? Also, if I delete the CR object, would the service and pods for that deployment be deleted automatically in the envTest environment as it does on openshift?

/assign


Support stylesheet variations

There are no labels, some labels, and more labels versions of the stylesheets. It would be nice to add support to easily toggle between them

💯

Closing in favor of https://github.com/mapzen/android/issues/372


8.2

I'm using Pentaho Kettle 8.2 on fedora 28 64 bits;
I get this error
"Unable to open dialog for this step
org.eclipse.swt.SWTError: No more handles [MOZILLA_FIVE_HOME="](java.lang.UnsatisfiedLinkError:Could not load SWT library. Reasons: no swt-mozilla-gtk-4335 in java.library.path..."
"...libxpcom.so: cannot open shared object file: No such file or directory..."
I've been looking for an answer but all of I have found do not solve this problem.
The last on was instal xulrunner and set MOZILLA_FIVE_HOME to xulrunner directory.

Hello, This is an automated comment from our build system. Since this pull request originated from a user  not in the Pentaho organization, we will not proceed  with automatic validation of the pull request.  However, rest assured that it will be reviewed manually as soon  as we get the chance. Thanks for your contribution!

Please open an issue at jira.pentaho.com or post a thread at the community forums: https://community.hitachivantara.com/community/products-and-solutions/pentaho
Did you see this link? http://diethardsteiner.blogspot.com/2014/02/installing-pentaho-kettle-depencencies.html
Seems to be the exact same problem you're having.


"Nginx Ingress" and "Metrics Server Monitoring" disabled for RKE clusters.

Rancher versions: Latest build from master
Steps to Reproduce:
Create a custom cluster .
In cluster option I see "Nginx Ingress" and "Metrics Server Monitoring" being disabled even though I see the default value being correctly set in cluster schema .
Also When u i try to enable "Nginx Ingress" , it disables "Metrics Server Monitoring" automatically in UI

Version - 2.1 master 9/12
Network provider default & Project Network Isolation has been resolved.  Wes asked me to move the Ingress & Metrics default issue out to another bug. #15559


Problem including ng2-vis in Angular2

Hi,
I just wanted to include your code into my library and I hang at the moment at an error I do not understand.
I included in my project vis.js 4.16.1 properly. Furthermore I included in my Angular2 application your demo code examples and do not encounter any more issues with wrong folder settings etc.
You have maybe a clue what the problem is?
Thanks!

I found the error - I tried to include vis.js via my angular-cli.json. That caused some trouble.
Furthermore, I still need the vis.css in my angular-cli.json, else the timeline example makes trouble...

I'm glad that the problem is solved. In my sample project there is no need to reference the vis.css because webpack knows how to load the css files. I'll bet you can get help from the angular-cli community on this problem :)


fix lint.sh - it ignored whitespace differences

Codecov Report

Merging #1283 (27de500) into develop (fdcdf2f) will decrease coverage by 6.43%.
The diff coverage is n/a.

:exclamation: Current head 27de500 differs from pull request most recent head 432ed35. Consider uploading reports for the commit 432ed35 to get more accurate results

@@             Coverage Diff             @@
##           develop    #1283      +/-   ##
===========================================
- Coverage    59.57%   53.14%   -6.44%
===========================================
  Files           84       18      -66
  Lines         4183      382    -3801
  Branches         0       22      +22
===========================================
- Hits          2492      203    -2289
+ Misses        1691      177    -1514
- Partials         0        2       +2

Impacted Files
Coverage Δ

...gle-lang/__tests__/Reducer/Reducer_TestHelpers.res

...ct/ReducerProject_tutorial_5_calling_functions.res

packages/squiggle-lang/__tests__/TestHelpers.res

...c/rescript/Distributions/DistributionOperation.res

...le-lang/src/rescript/Distributions/GenericDist.res

...butions/PointSetDist/AlgebraicShapeCombination.res

...rescript/Distributions/PointSetDist/Continuous.res

...c/rescript/Distributions/PointSetDist/Discrete.res

.../src/rescript/Distributions/PointSetDist/Mixed.res

...cript/Distributions/PointSetDist/PointSetTypes.res

... and 92 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Flows in the Player Ability API

Describe the problem related to your feature request.
The current api to manage players abilities have currently one big flows, is that you can't change the gamemode and the abilities in the same tick.
It can be useful to pass the player in survival but maintain the flying state.
What solution would you like?
Rethink the order of execution of systems.

assigned to myself

Did you mean "flaw" instead of "flow"?

Yes, english hard


Order Contents Decorator undefined method 'grab_line_item_by_variant'

When using spree 3.7.0 it fails to run with gem due to "grab_line_item_by_variant" no longer being a method in the order contents model.

+1

+1

Hey @Mbuckley0
Spree 3.7 support is now available in https://github.com/vinsol-spree-contrib/spree_gift_card/tree/3-7-stable branch
Let us know if you face any issues with this branch


Link to particular notebook

Migrated from https://github.com/binder-project/binder/issues/18
Q: Is there a way to link to a particular notebook hosted on mybinder?
A: from https://github.com/binder-project/binder/issues/18#issuecomment-139273270
Not yet but definitely possible! @andrewosh and I just sketched it out.
And the behavior we're imaging is something like this for linking to a notebook at the top level
mybinder.org/repo/user-name/repo-name/notebook-name.ipynb

this for linking to a sub-folder (presumably one with many notebooks)
mybinder.org/repo/user-name/repo-name/folder-name/

and this for a notebook in a sub-folder
mybinder.org/repo/user-name/repo-name/folder-name/notebook-name.ipynb

If that's not what you had in mind, definitely let us know what would be most intuitive!

@freeman-lab yes, this looks great to me

This is now in production, let us know if you run into trouble @mrocklin ! Thanks again for the suggestion.


Expand warning about big sysdump

fixes: https://github.com/cilium/cilium-cli/issues/1371

Forgot to re-push :facepalm:


Channel ID too high?

Description

Looking at the logs for this, I see channel numbers high like 65,

<ChannelView cid=65 from=0xE7e04Dd236395570Fd0053c4Fa2AEf9561349710 to=0xc976608eDe1503e351B896360D17AB468ca1a58f capacity=0>
However, the scenario file only has 5 nodes and 5 channels configured.
Expected Behavior
As the channel ids increase monotonically, the highest channel should have an id of 5, not 65. So either we reuse the same token network for multiple scenarios or something else goes rogue.

Expired


Auto released binaries and images are not tagged with proper release

The artifacts published by auto-release job (see<PHONE_NUMBER>773928448 build) are not tagged with proper tag - in this case v0.26.0. They use the v0.4.0-13-g7f0b0fd as version, which is the git describe output without having v0.26.0 tag on repo.
Release script and probably magetasks needs to adjusted to properly tag artifacts.
$ curl -LO https://github.com/knative-sandbox/kn-plugin-event/releases/download/v0.26.0/kn-event-linux-amd64
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   628  100   628    0     0   2114      0 --:--:-- --:--:-- --:--:--  2114
100 52.0M  100 52.0M    0     0  37.2M      0  0:00:01  0:00:01 --:--:-- 67.0M

$ chmod +x kn-event-linux-amd64

$ ./kn-event-linux-amd64 version
kn-event version: v0.4.0-13-g7f0b0fd
sender image: gcr.io/knative-releases/kn-event-sender@sha256:2784c4e206fa9148773372c8d651a0be6311954e820498ac0a322335508328be

$ skopeo inspect docker://gcr.io/knative-releases/kn-event-sender@sha256:2784c4e206fa9148773372c8d651a0be6311954e820498ac0a322335508328be | jq -r '.Labels.version'
v0.4.0-13-g7f0b0fd

/kind bug

As evidenced by build<PHONE_NUMBER>907949568 (dot-release), this problem still persists. Looks like #87 didn't fix it, fully.


Equivalent to setcontent

In Twig for Bolt, content can explicitly be set using "setcontent" inside the template. This is useful for common blocks which are used in asides, footers, etc, without having to change the template itself.
I haven't managed to work out a way of getting similar behaviour in Gutenberg. Is this currently possible?

I'm not familiar with Twig/Bolt but, if I understand setcontent correctly, then I think you can accomplish the same thing with the set Tera filter (perhaps combined with other Gutenberg features to get a particular target value).
For example, the first setcontent example in Twig's docs is
{% setcontent about = 'page/about' %}

{{ about.title }}

{{ about.introduction|raw }}
link

I believe you can do the same thing in Gutenberg with:
{% set about = get_page(path="about.md") %}

{{ about.title }}

{{ about.summary | safe }}
link

Would that do what you're looking for, or did I misunderstand what you're asking?

Thanks both - get_page does indeed do exactly what I was trying to do.


Create PVCs for each pod

We need to specify a data dir for the stacks node and bitcoin node pods, and we'll need a PVC to write to the fs. We'll need to create manifests for those PVCs and test to confirm that memory usage is staying low on the devnets when this is actually deployed on a cluster.

This one may not be necessary and requires testing.

Everything is deployed and working well; closing this as not planned


Rx problem in both Vhf 162Mhz and Uhf 462Mhz

Hi
I m testing and using different Firmwares on my two K5 wakie talkies from many days and comparing Rx with my Baofeng.
I have fonud Rx problem in both Vhf 162Mhz and Uhf 462Mhz in this FW. Signal reception is weak then Custom firmware. All other functions are working good and unbeatale.
Sorry if you mind my observation.

Some are saying it's deaf, some are saying the opposite, can't keep up
I need people to put there radios onto signal generators and report the actual sensitivity reading at various vhf/uhf frequencies (no out of band reports as they don't count) so as we know either way.

Sir
general isue is being faced. I am checking on two k5 radios keeping in same invoirment ans receiving the signals of a transmeter with is hundred miles for away from me.
Its fieldd test. I think its reliable. Ihave chageed different Fw many yimes again when i confermed this isue then I thought to give in your knowlege.

oo I don't want any rewards or hardware or anything, but thank you for the offer.
I'm doing it while I can, I don't know how long I can keep going though due to health reasons, time will tell.

oo I don't want any rewards or hardware or anything, but thank you for the offer.
I'm doing it while I can, I don't know how long I can keep going though due to health reasons, time will tell.

Sir
May God give you health and happiness. Your acheavments are blessings for us.
Sir I have Checked reception on MCFW v29.1C, its Rx on  Band and 70cm  is best ,as equal to iCom V85 walkie Talkie.
Sir I have not much knowledge about softwares , you can find whats happening that is decreasing signals.

I need people to put there radios onto signal generators and report the actual sensitivity reading at various vhf/uhf frequencies (no out of band reports as they don't count) so as we know either way.

By that I'm assuming hardware similar to a NanoVNA. I may get one now that the AGC fiasco has me stumped on this sensitivity issue. I have four K5s, three with their own quirks. Gotta be something between them.

Well, if you can figure out the BK4819 chip, well done !!  don't think anyone else has (yet), though some are using registered not listed anywhere, how they do that I do not know.

Well, if you can figure out the BK4819 chip, well done !! don't think anyone else has (yet), though some are using registers not listed anywhere that I'm aware of, how they do that I do not know.

Thought I'd dig deeper and it turns out fagci has found some, including interesting entries like REG_62 (not set).
https://github.com/fagci/uv-k5-firmware-fagci-mod/blob/f71cee19d0f6cfb581f9286a034821fa868066d9/app/spectrum.h#L174
I'm going to play with these and see what happens.

You are working great, we are addicted of this.


Updated screendiff golden screenshots for Linux platforms.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

The change doesn't affect any existing Beam code or tests.


Fix build and tests for the 'hyper_client' feature.

Thanks for catching this, looks like we need to improve the CI here


Minify disabling right click context menu by disabling it in JS or with return!1 in HTML

Minified HTML demo https://codepen.io/burntcustard/pen/BaZVoqX
Discussion: https://js13kgames.slack.com/archives/C1EHVV2DN/p1632204464031100
Should maybe put in as a build step, which replaces all occurrences of "return false" in the HTML with return!1

document.addEventListener('contextmenu', event => event.preventDefault());

document.addEventListener('contextmenu', event => event.preventDefault());

Yep that would work for putting it in the JS - but it seems we can also return false, or even return !1 instead of preventing-default.
I'll have a go later and see what actually saves the most bytes!


Poor performance using one(T) inside Polyester.@batch  loop

I am running Julia 1.9 with two threads. If I include one(T) inside a Polyester.@batch loop, where T is a type known to the function, I obtain slow performance with a lot of allocations.  Consider the following example
using Polyester
using BenchmarkTools

function test_no_poly!(y::Vector{T}, x::Vector{T}) where{T}
    for i in 1:length(x)
        y[i] = exp(x[i] + one(T))
    end
end

function test_poly_one!(y::Vector{T}, x::Vector{T}) where{T}
    Polyester.@batch for i in 1:length(x)
        y[i] = exp(x[i] + one(T))
    end
end

function test_poly_1!(y::Vector{T}, x::Vector{T}) where{T}
    Polyester.@batch for i in 1:length(x)
        y[i] = exp(x[i] + 1.0)
    end
end

x = rand(1_000_000)
y = similar(x)

@btime test_no_poly!($y, $x)
@btime test_poly_one!($y, $x)
@btime test_poly_1!($y, $x)

This gives me
  4.411 ms (0 allocations: 0 bytes)
  84.761 ms (2000001 allocations: 30.52 MiB)
  2.413 ms (0 allocations: 0 bytes)

It's because of Julia's not-specializing-on-type heuristics.
You could make a PR that has the macro call wrap all arguments to batch, and then unwrap them at the start of the loopbody
struct WrapType{T} end
wrap_type(@nospecialize(x)) = x
wrap_type(::Type{T}) where {T} = WrapType{T}()

unwrap_type(@nospecialize(x)) = x
unwrap_type(::WrapType{T}) where {T} = T


Modernize VoteServiceTest

Bring this test up to date with the latest practices: use assertTableEquals
to check table contents and don't run it as a mock user.

#2621  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


move revolver to detective inventory and more

About the PR

Moves the detectives revolver from their locker to their holster. Removed the pistol mag for a rubber speedloader and lethal speedloader.
This should bring them more in line with standard security as a side result, but for the most part it was just to remove the redundant pistol mag.
Moved print analyzer and paper from their pockets to their bag while I had the opportunity.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Whisper

tweak: The Detective's gun has been moved to their holster from their cabinet. It comes with one speedloader each of the standard rubber and lethal bullets of officers.
tweak: The Detective's scanner and pad were moved to their backpack from their pockets.
remove: Detective no longer spawns with a pistol magazine.

It seems to me that this change is problematic, there are cases when an ordinary officer becomes detectives due to their absence, and now it turns out that the officer will not be able to have a revolver

It seems to me that this change is problematic, there are cases when an ordinary officer becomes detectives due to their absence, and now it turns out that the officer will not be able to have a revolver

All officers spawn with a pistol now whereas the detective spawns with... nothing currently actually. Just their prints kit in their pockets unfortunately

Yes, but they still has a revolver in their closet, a revolver is still a special attribute of a detective, a cmo will not spawn with a hypospray in a backpack, and a bartender with a shotgun, too

Because these are special attributes of their profession and if there is no main person in this profession, another person will be able to take a place and take possession of the attributes

The gun is not any more a key item of the detective as a pistol is for secoffs, I don't believe giving the detective his gun on roundstart takes away from anything a secoff already has or can obtain.
Argueably, the print kit and gloves are they key items of that role, not the weapon.

this is neato and det can always pick up a secbelt in a locker


Fixed #458 - Fixed Cordova Plugin Social Sharing reference

Replaced cordova-plugin-socialsharing with cordova-plugin-x-socialsharing

Ok the fix is in the dev branch here: https://github.com/phonegap/phonegap-app-developer/commit/82a3888fa2165efb63e48c99005c11fce81b98f7.
Thanks!


Fixed wrong PSR answer

PSR-0 is only about autoloading.

Hi,
thank you Jerome, don't hesitate to add more questions if you use this repository ;)
Mickaël


WIP Opentracing

Overview
Implement RFC-011 (introduce opentracing support)
Testing recommendations
make check
Related Issues or Pull Requests

https://github.com/apache/couchdb-documentation/pull/440

Checklist

[ ] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

I hoped to trigger a CI but it didn't worked out.


Pre-PR for initial feedback: add iscisi device path resolver for softlayer

Do not merge this. This is to allow me to make comments to @EdwardStudy

I had finished adding test cases and fixing mistakes.
Some replies had also be added.
@maximilien Please help to review, thanks.

@EdwardStudy please update the PR when you have made necessary changes and comment here. I am not sure which of the feedback is addressed.

@maximilien We should close it and re-open a formal PR about this feature.

Closing to @edwardstudy can resubmit with fixes


add support for new gpt-3.5-turbo-16k model

sadly this repo is abandoned by the creator

Thanks for doing this.
I have added it to my fork.

Thanks for this!

遗憾的是这个仓库被创建者放弃了

为啥放弃了？有其他新的项目吗？


Adding URLs and blank templates for all apps

Adding URLs to all apps.
Adding a blank HTML template page to every feature.
Linking URLs to templates via views
fix #71

What is the difference between signing and login?

Login is for registered users, and sign-in is for new users that want to register.
For now< we add users only via the admin screen, so the sign-in page isn't at top priority.
Still, I thought it would be worthwhile to prepare the ground for the moment we want to add this feature in the Feature.

What is the difference between signing and login?

Login is for registered users, and sign-in is for new users that want to register. For now< we add users only via the admin screen, so the sign-in page isn't at top priority. Still, I thought it would be worthwhile to prepare the ground for the moment we want to add this feature in the Feature.

Sign in and login means the same thing, it's supposed to be sign up or register. @OmerCS8

Should we add views/urls to the direct_message model if its not merged yet?

We already have a direct_message app so i think we should add the views and URLs. For now it's a blank template, but once the direct_message model will be merged we can alter the file.
(BTW we can work on the HTML and CSS of the direct messaging app even if the model hasen't been merged yet)

As we agreed, I removed the sign-up URL 👍

@OmerCS8 Please squash all the commits

@OmerCS8 Please squash all the commits

Squashed all the commits and ready to merge 👍 . @natifridman @halleelyasaf @abiton1

@OmerCS8 Please squash all the commits

Squashed all the commits and ready to merge 👍 . @natifridman @halleelyasaf @abiton1

@OmerCS8 Great! we need one more approval


Nature Unable to Locate Bindings File in RunKit.

When attempting to run Natural from  NPM site in RunKit. One encounters

Error: Could not locate the bindings file. Tried:
→ /app/available_modules/1493023968000/natural/node_modules/webworker-> threads/build/WebWorkerThreads.node

This can be reproduced by clicking on the link above and clicking running, this will throw the error immediately.
I should note that I have not been able to reproduce the above locally.

should be fixed as of (#354)

Error: Could not locate the bindings file. Tried:
→ C:\coding\projects\blog\node_modules_webworker-threads<EMAIL_ADDRESS>→ C:\coding\projects\blog\node_modules_webworker-threads<EMAIL_ADDRESS>
I got this

Got the same error when running on electron.
Posted a proposition here https://github.com/NaturalNode/natural/issues/368

Adding Webworker-threads is manual now. So no problems should occur anymore in using natural with a framework like electron.
Hugo


This is a security bug bounty test... Please ignore  lkbzpw

This is a security bug bounty test... Please ignore  lkbzpw

@marsartan closing these PRs.
Pleas do not probe on public repositories.


add name field to recordTime field

We weren't passing the name of the recordTime field when updated the IDK schema from the avro schema in the kafka consumer. This was preventing time quantums from being used. This PR corrects that mistake.

Do we have a test for this that actually uses it, like, takes a thing from the avro schema in kafka, stores data, and verifies that the correct time quantums are getting populated?
Can we easily add one? I feel like that would be a really good thing to confirm/deny.

I'm willing to approve this, but if you can easily add a test that actually confirms that data submitted this way is showing up in the right time quantums, I would prefer that.

@seebs I've added some additional testing. Let me know what you think.


Support formations and partnerless events

No points for rankings, but useful when watching a competition live.

Closed by pull request #145


Remove build number from build phases

Moving to make BB the source of truth for deployment. Using timestamp as build number.

Confirmed this worked, just submitted 1.14.0 to the App Store.


Duplicated unsubscribe links at bottom

I would also make the link to jupyter.org https:// and in general make sure link can be https when they can be.

@katiewhite360 this is still the case , can you remote the unsubscribe and update subscription preferences from our template.


  Would be nice to validate that window is not undefined and maybe some other properties to validate it is an actual window and if not throw a better error.  I'll can submit a PR if you tell me what other properties to validate.

Would be nice to validate that window is not undefined

That's actually what this error is saying. I guess the message itself could be better. If you'd like to improve it feel free -- the code is just here:
https://github.com/krakenjs/post-robot/blob/master/src/interface/client.js#L16-L18


added mongoid support for state_machine-audit_trail

Hello, I just added some support for Mongoid.
I ported the specs to Mongoid and they are all passing. In some cases I got :start (symbol) instead of "start" (string) but this seems to be a known issue and so I have used .to_s to coerce the result to a string.
I am about to begin using this in my app, so will add more specs as necessary.
thanks for the nice gem.
cheers
svs

Looking good on first inspection. Thanks for your contribution?
@jstorimer do you have Mongo installed so we can actually run the tests? :)

Managed to set up mongo myself and run the tests.
I did some refactoring to make sure that the plugin is now fully independent of the backend: it will only load ActiveRecord or Mongoid if necessary.

Now released as version 0.1.0.

Awesome. my first merged pull request.
thanks guys.

Thanks for contributing!


Horizontal scrolling through homepage carousels triggers scroll-to-top

Steps to reproduce:

Open app
Navigate to home page
Scroll down a bit
Lightly scroll a carousel that isn't the featured carousel (not enough length to scroll right a whole card.

Expected: Carousel snaps to one card or another
Actual: Along with the expected result, the app also suddenly scrolls to the top of the home page.
AHS Mobile v.3.1.1
Android v.5.1.1
LG G Stylo

The carousel bug seems to be API specific. On an API level 28 phone, this does not happen but it occurs on an API level 24 phone.


Median Filtering

Hello, I want to work on Median filtering in Image Soothing. Please assign me.

Assigning you with the same @shivangi-pandey.


Does this work per repository?

Every time I switch from one repository to another it seems to only want to use the last profile I selected. I would much prefer having a profile-to-repository association so I never have to click anything again. Is this possible?

@tylerclark how's the issue going? if you have solved the problem, please share it with me

I haven't solved the problem. Will likely make my own extension since this one doesn't really do much.

hello, it was developed to work per repository. If I remember correctly, the main reason was that I did not want to overwrite if you open a repository that already has user details. I might revisit and see if there is a better way to address your issue but not anytime soon...


Question:  Are there any sites similar to 1-Sec Mail? Temporary Email Solution API Alternative.

Hello just wanted to know if there is any site similar to that. I don't mind if it's subscription based.
Also, sincere apologies as this isn't an issue but a general question.

Hello,
I didn't pay for something like that.

You can make a website like that with your domain and server. (Maybe you can find sources here (I didn't check them): https://github.com/topics/disposable-email)

Some websites:

https://tempmail.lol/custom.html
https://temp-mail.io/en/subscription
https://smailpro.com/pricing
https://temp-mail.org/en/premium/login
I don't know anything about these websites, I share my search results.

Thank you, I'll close this.


[DO NOT MERGE] Merge / release template-retail-react-app v5 features previously held from develop

Description

Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

git merge failed to determine which commit SHA comes first, overriding the changes from v5/template-retail-react-app. Closing


Rock Paper Scissor Game.

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
It will be a rock paper scissor game.
Tech Stack
Web Development

Have you checked, if this project is avalable in the provided web-dev folder?

Is Spock rock game equal to rock paper scissors ? @manisha-27

Yes, have you seen the project, there, including rock paper scissor, that project contains 2 more element. Also the Working will be same.

Ok ma'am my fault. 😅😔😶


Can't flash on Lilygo T3 V1.6.1

Thanks for the development!
But I need help...
I have a Lora Lilygo T3 V1.6.1 and no matter how much I follow your tutorial I can't flash this :/
I know I shouldn't put this here, but I don't know where else to turn, sorry
Ursprünglich gepostet von @nosfaratu in https://github.com/matthias-bs/BresserWeatherSensorReceiver/issues/143#issuecomment-2010911201

Mathias, thanks for opening the topic.
I'm a bit of a code lammer and I tried to follow all your steps, but without success.
Board - lilygo model T3 V1.6.1
So I'm using Arduino IE as you can see in this link to have a chance of doing everything right, but not :/
https://github.com/matthias-bs/BresserWeatherSensorReceiver/blob/main/BUILD.md
code for BresserWeatherSensorMQTT2.ino
https://gist.github.com/nosfaratu/a1c46ffddb8e8281a7018ab253c5f4f0
code for WeatherSensorcfg.h
https://gist.github.com/nosfaratu/6b9cf40bd2a2667aee947fadecb6ac58
I compile the code and save it in another folder.
Then I'll check ok, when I upload it gives an error, but the upload finishes.
Set it to reset but the board does not reset.

https://gist.github.com/nosfaratu/3bd3a127fa0f1e063de87ca436db1833
Thanks in advance!

Hi,
in your WeatherSensorCfg.h, you have uncommented two defines in the board section:
// This define is set by selecting "Board: TTGO LoRa32-OLED" / "Board Revision: TTGO LoRa32 V2.1 (1.6.1)"
// in the Arduino IDE:
#define ARDUINO_TTGO_LoRa32_V21new

[...]

// DFRobot Firebeetle32
// https://github.com/espressif/arduino-esp32/tree/master/variants/firebeetle32/pins_arduino.h
//
// This define (not very specific...) is set by selecting "FireBeetle-ESP32" in the Arduino IDE:
#define ARDUINO_ESP32_DEV

Please leave those defines commented out as it was before. If you select this board and board variant in the IDE (as you did as I see in the screenshot), the correct settings are used automatically.
For the first test, I recommend using the basic example. I any case, please use the examples provided when trying the first steps. You are trying BresserWeatherSensorMQTT2.ino - how should I know which changes you made?
After flashing the board, please switch to the serial console (same serial port as for flashing).
Regards,
Matthias

@matthias-bs
Success! Using just BresserWeatherSensorBasic.ino from the examples, without changing anything it flashed!

The log appears on the serial monitor!

But I can't pass anything to HA xD

Did you check the part related to secure/non-secure MQTT and the related port and WiFi settings?
You have to change
#define USE_SECUREWIFI        // use secure WIFI
// #define USE_WIFI              // use non-secure WIFI

to
//#define USE_SECUREWIFI        // use secure WIFI
#define USE_WIFI              // use non-secure WIFI

in order to try a non-secure connection on port 1883.
Check the MQTT broker's logfile to see why the connection fails.

A secure connection requires USE_SECUREWIFI, port 8883 and some specific credentials.

Regarding the display - have a look at find_transmitter.ino how to use it.

mqtt is already resolved! I had to set sleep to false, because it almost always appeared offline
I can now listen to the topic on homeassistant

Now it remains to create the sensors.
Regarding the screen, do I just put that file and flash it along with the others?

No, you have to modify the MQTT sketch to use the display. You can use the other sketch I mentioned as an example for using the Adafruit SSD1306 library.

Seems to be solved - closing.


Zed's Crash When Closing and Reopening Cameras

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
When I have a list of cameras, if I continue opening and closing them, it eventually fails and my terminal outputs something similar to "can't claim interface 0: -6"
My purpose for doing this is that I want to collect data in a low resolution, but extrinsically calibrate a single camera at a time with high resolution. I only have 3 cameras on at once, and this issue happens frequently.
Steps to Reproduce
Make a while True loop where you turn cameras on and then off.
Expected Result
Procedure should be safe to repeat
Actual Result
Eventually camera's cannot be turned back on again.
ZED Camera model
ZED2
Environment
I am using the most recent SDK, ubuntu 22, and have zed minis and zed 2's.

Anything else?
No response

I wanted to bring attention to this issue again whenever you guys have the chance. The memory leak issue I mentioned in another thread is more blocking than this though :)

Noted. We are working on this

Just to check- are you guys able to reproduce the issue?

Hey! Can you guys please address this.


Possible memory leak in SimpleStreamSender

Currently SimpleStreamSender has an overloaded constructor that takes only an IlpOverHttpLink. This then creates an ExecutorService to handle any ILP packet sends. However, once the client is done using SimpleStreamSender it will be garbage collected but the ExecutorService it created will never be shutdown or garbage collected.
Possible solutions:
A. Add a shutdown method to SimpleStreamSender that calls the shutdown method on the executor service. Clients will be responsible to call SimpleStreamSender#shutdown to avoid memory leak.
or
B. Remove the overloaded constructor that creates an ExecutorService and require the client to always provide one. Clients will be responsible to create and shutdown the ExecutorService they provide to SimpleStreamSender.

StreamSender has been deprecated in-favor of ILP Pay functionality. Closing this.

StreamSender has been deprecated in-favor of ILP Pay functionality. Closing this.


Refine Contract & Document Class Attributes

MOSCOW: Machine Readable User-defined Attributes
Main challenge is to figure out the attributes to attach to Contract and Document for the customer. Do a first pass over the notes and analysis and add as needed. Will (hopefully) be tested and refined after.
Difficulty of implementation should be low. No parsing of the Contract should be required.
Note this will be primarily accessed through a search system down the line.

Write comments in the code for User / Company / Assosiate links that do not exist yet.

Read on

ActiveRecord Basics https://guides.rubyonrails.org/active_record_basics.html
Rails https://guides.rubyonrails.org/getting_started.html
ActiveRecord Migrations (to add to our Contract and Document) https://guides.rubyonrails.org/active_record_migrations.html

The Contract TItle / Document Title may also need rework.


fix: stuck in invalid state after last validation constraint is removed while invalid

Fixes #400

Quite a lot of changes in this PR. All the tests seem to pass with the following additional observer to vaadin-text-field-mixin (could be extended by numberfield if -mixin shouldn't know about min, max, step):
__constraintsChanged(required, minLength, maxLength, pattern, min, max, step) {
  if (!required && !minLength && !maxLength && !pattern && !min && !max) {
    this.invalid = false;
  }
  if (this.invalid) {
    this.validate();
  }
}

Am I missing something essential? If so, there should be more tests to cover all the changes.

Quite a lot of changes in this PR. All the tests seem to pass with the following additional observer to vaadin-text-field-mixin (could be extended by numberfield if -mixin shouldn't know about min, max, step):
__constraintsChanged(required, minLength, maxLength, pattern, min, max, step) {
  if (!required && !minLength && !maxLength && !pattern && !min && !max) {
    this.invalid = false;
  }
  if (this.invalid) {
    this.validate();
  }
}

Am I missing something essential? If so, there should be more tests to cover all the changes.

The problem with this simpler approach is that it will run validation only when all those are undefined, but not when there is still some active constraint but one or more other constraints were unset.
It's a shame that the multi dependency observers in Polymer don't provide the old values so you don't know which of those dependencies changed without manually tracking the old values elsewhere. The single property observer provides old value but I also ran to some issue when thinking about using individual observers.
I'll add a test case where there are many constraints applied and one is unset.

After taking a closer look, the simple __constraintsChanged observer approach seemed to work good after all considering all the cases I could think of.
I rebased this branch to latest master (with the flaky test fixes) and updated to use this simplified approach. Also added one more test and fixed some random typos in the test files.


Default devtools behavior, enabled or disabled?

Please vote in the comments below. An explanation around your preference would be nice, but is not necessary.
This is to determine whether the base npm start command should run with devtools enabled or disabled. The starter kit currently has devtools enabled by default, where they can be optionally turned off by using npm run dev:no-debug.
Switching this would invert the commands to how they used to be in the early days of the starter kit, so npm start would run the webpack-dev-server but not include devtools, and they could be enabled with npm run dev:debug.
This goes back to the old question of: how often do you use devtools? Do you find yourself disabling/hiding them more often than not? Or do you use them frequently and want to keep them as the default behavior?
Thanks everyone.

Well either everyone but me hates DevTools or someone is dedicated enough to switch IP several times and mess up with the vote. Isn't it a bit strange that the vote counter stayed at 2 for few hours and now suddenly it's at 80?

@fuczak something must be messed up. We can just do the poll in this issue, I'll update the description.

Personally my preference would be hide by default via visibleOnLoad=false so we can toggle visibility when needed. If this is not an agreeable solution then I would vote for disabling by default.

If we go with setting visibleOnLoad to false then I'd probably just add a key for that option in the project configuration so it's easier to switch to the desired functionality. Might not be bad.

That sounds resonable. For the sake of expressing my thoughts I'd personally go with visibleOnLoad=true since I'm using DevTools all the time.

I find myself hiding it more often than not, but honestly, one of the reasons I wen't with this starter kit was to get a feel of how the devtools were implemented. So for new people it's probably the best way to get them on board. It will give that "WOW"-effect when you launch the starter-kit and see the devtools up and running.
So I think it depends on where YOU want to focus on with this starter-kit. Do you want people to get on board up and running with it, or do you want to supply the people that already work with it. I never ran npm run dev:no-debug, because I'd rather hide it and open it when I have to.

I agree @inooid that it's more about highlighting the starter kit's functionality than customizing it for users already familiar with the devtools. As often as I personally hide/disable devtools, I think it makes sense to keep them enabled and visible by default, and then people are free to customize it as they see fit; it's fairly trivial.


Add BaseTracker

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (4446441) to head (04a1f5a).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #280   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           96        97    +1
  Lines         3025      3058   +33
  Branches       358       370   +12
=========================================
+ Hits          3025      3058   +33

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


Revert "fix To ensure compatibility with the cloud upgrade"

Reverts apache/doris#37708

run performance


Is there a bodyParser.urlencoded({ extended: true } equivalent in Hono?

Hello,
I'm making my move from Express/node.js to Hono/Bun, and it feels refreshing!
Some stuff are really nice and useful: for example: set()/get()!
But sometimes, I'm struggling...
Consider the simple c.req.parseBody() below:
{
  id: "123456"
  "timeslots[0][start]": "",
  "timeslots[0][end]": "",
  "timeslots[1][start]": "",
  "timeslots[1][end]": "",
  "timeslots[2][start]": "",
  "timeslots[2][end]": "",
}

With express, it is automatically parsed using bodyParser.urlencoded({ extended: true }, and timeslots is returned as an array.
Maybe I'm missing something, but is there something equivalent in hono? Thanks.

Hi @charnould!
Thank you for using Hono.

Maybe I'm missing something, but is there something equivalent in hono?

Hono doesn't automatically parse them as an array like Express. Actually, c.req.parseBody() is just a shortcut to manipulate req.formData(), which returns a FormData object:
FormData - Web APIs | MDN
You can use the FormData object directly like this:
const formData = await c.req.formData()
const timeslots= formData.getAll('timeslots'))

I'm not sure if this will give you the result you expect, so please give it a try and let me know if it works or not.

@yusukebe
Thanks for your help. Unfortunately, it doesn't :-)
I've posted a question on Stack Overflow, but current answer looks super complicated for something that was a bodyParser.urlencoded({ extended: true }-away with express.
Don't you think it could be a hono feature?

@yusukebe
I saw your smiley reaction (👀) but don't know how to interpret it 😅...
Is my suggestion a bad idea? Or maybe SO answer looks super-simple to you?
Let me know (if you want, I can close this comment).

Sorry.
Current my thought is it's not within Hono's scope. If you want to do that, you can make the shortcut method by yourself.
But, I was thinking it's a good idea to make an "Extended Body Parser Middleware". So, it's no problem to keep this issue still opened.

I think you can get the same result by using:
import qs from 'qs';

// ...

const text = await c.req.text()
const data = qs.parse(text);

@yusukebe is it possible to add qs to hono? Probably like if the user pass the argument to parse it with qs then it will parse it automatically
const body = await c.req.json({ extended: true });

and inside the request.ts we can do it this way
async json<T = any>(parse:? { extended?: boolean; options?: Nonullable<unknown> }): Promise<T> {
    const body = await this.cachedBody('json');

   if (!parse || !parse.extended) return body;

   const qs = require('qs');
   return qs.parse(body, parse?.options);
 }

Hi @krsbx,
Thank you for the suggestion. But, the answer is no – we can't add qs nor port it directly.
Firstly, Hono should not depends on external libraries. Secondly, introducing functionality like qs would be more of a "Helper" due to it will increase the amount of code. Nevertheless, if you have a use case that requires its functionalities, you can directly use qs alongside Hono.

@yusukebe how about if I make a PR where a middleware reassign the json and text function from c.req so it will parse it automatically? I'll try to create my own version of qs

@yusukebe, @charnould just made a PR so we can have some sort of qs in hono out of the box in #1389


No longer require Ref for broadcasting distributions

Bug report from a student!
It appears that Distributions is now "broadcast friendly".  We should look for uses of Ref with broadcasting with it (e.g. pdf.(Ref(d), [1.0, 3.0]) no longer requires the Ref) and remove it.  This might be a good time to review all uses of Ref( in the notes, since others might have changed.

Done at bf5efe5.


docs: move to monorepo

From https://github.com/textlint/textlint/issues/424#issuecomment-354314908. Current textlint/docs:

Have some broken links, stale type interface (#424)
Require time consuming process to edit/test as it will always start with full build (we have yarn run textlint though.)

Proposal
In order to solve issues above and make space for future improvement, we would like to move docs to monorepo.
Tasks
Rough steps to implement:

Add docs/ as new workspace to textlint/lerna.json and textlint/package.json—think this directory as a project, instead of just a bucket of documents
Add new package.json to docs/ and introduce its own dependencies, test, linting, etc.
Add new test script(s) to textlint/package.json e.g. "test:docs": "lerna run test --scope docs"
Run CI build and see how it works

References

#270 Import * to monorepo
Yarn Workspaces
Lerna Flags#scope
Docusaurus
almin/almin for monorepo + Docusaurus setup

📝 Not current, But future 🌟
I have a motivation to introduce Docusaurus.
The motivation related with release notes. (Of course, we want to get documentation site)
GitHub Releases page is hard to see.
Because, textlint is monorepo and, tagged multiple git tag in a single release.
This issue is resolved by introduced Docusaurus's blog system, I think.
Release and publish a blog post instead of release note.
Prettier does it.
https://github.com/prettier/prettier/pull/3604


proposal: Go 2: require field name for struct composite literal when the struct reference is with a selector

Background
Adding a new field to a struct in a package is currently a breaking change. This means that adding a field to any public struct in a package requires a new major package version, creating a fair amount of churn for users when coupled with modules.
One major reason this is a breaking change is because of the second rule of Composite literals that allows to not specify each individual key name, and use a list instead, i.e. image.Point{10, 20}.
Adding methods to a struct is generally not considered a breaking change (even if it is) and it is frequently done in the Go standard library.
The net result is that it is very hard to write future proof structs, as adding a struct fields is a generally considered a breaking change due to the allowance of keyless composite literal rule for struct imported from an external package.
Precedent
go vet already warns on keyless struct composite literal.
Proposal
When a struct composite literal appears in the code, and the struct symbol reference uses a selector, that is, it is not in the current package, then field names are required.
This means changing the composite literal spec from:
For struct literals the following rules apply:

A key must be a field name declared in the struct type.
An element list that does not contain any keys must list an element for each struct field in the order in which the fields are declared.
If any element has a key, every element must have a key.
An element list that contains keys does not need to have an element for each struct field. Omitted fields get the zero value for that field.
A literal may omit the element list; such a literal evaluates to the zero value for its type.
It is an error to specify an element for a non-exported field of a struct belonging to a different package.

to
For struct literals the following rules apply:

A key must be a field name declared in the struct type.
An element list that does not contain any keys must list an element for each struct field in the order in which the fields are declared. This shorthand format only allowed if the struct literal type is not using a selector.
If any element has a key, every element must have a key.
An element list that contains keys does not need to have an element for each struct field. Omitted fields get the zero value for that field.
A literal may omit the element list; such a literal evaluates to the zero value for its type.
It is an error to specify an element for a non-exported field of a struct belonging to a different package.

This is effectively turning the go vet composites rule into a compile rule, removing the need for the go vet check.
Rationale
Enable package authors to add fields to struct the same way they feel safe to add methods to struct.
Automatic fix
While this is a breaking change in the language, as current code that use struct composite literal from imported packages with keyless list would stop compiling, it is a relatively easy fix that could be 100% automated.
Caveats
Breaking change due to embedding
Adding a field to a struct would still be a breaking change when the struct is embedded into another one, as described in this post. Still, the cases where this occurs are expected to be significantly lesser than struct composite literal, resulting in a net benefits.
Verbosity
The rationale to disallow non-keyed list only when a selector is used enables using non-keyed list in unit tests, which are often verbose.
The obvious downside is that other packages will be more verbose in the struct composite, but long term forward compatibility is worth the price.
Example
Synthetic
Source foo.go
package foo

type struct T {
  A string
}

In foo_test.go, it would be allowed to use the short form T{"hi"} because the type T is referred to without a selector.
On the other hand, in source bar.go:
package bar

import "foo"

var t = foo.T{A: "hi"}

Using foo.T{"hi"} would be disallowed because type T is referred to via selector foo..
Real world
I'd like to add a new field to the struct physic.Env as the BME680 sensor supports gas measurement, but I can't add a field to this struct without bumping periph's major version from 3 to 4. This is because I don't know which code will break.

I think it's worth mentioning the section on struct literals in the Go 1 compatibility doc: https://golang.org/doc/go1compat#expectations .

Thanks, that's a good point. The proposal is essentially replacing the recommendation with enforcement, for better future proofing for package authors, including the stdlib.

IMHO this proposition would make code harder to read in a few domains that use short vector-like structs (for coordinates, colors...), especially for graphics programming, indie game development, and probably other math-heavy domains.
These tend to have not only quite a few literals in the code, but also conversion to/from separate variables, leading to stuttering: p = image.Point{X: x, Y:y}
Maybe unkeyed literals could be allowed only for structs where all field names are one-letter long?
Note that, in these cases, using arrays instead of structs is not a good option either, because it obfuscates the code that make use of the fields: c = color.RGBA{nc.R*nc.A, nc.G*nc.A, nc.B*nc.A} is much easier to understand than c = color.RGBA{nc[0]*nc[3], nc[1]*nc[3], nc[2]*nc[3]}

Closing as dup of #2794.


Add incrementCounter method to make it compatible with presto

PrestoDB and TrinoDB depends on the old incrementCounter method which has two parameters. Therefore, although we have replaced this the method with another one that has three parameters, we still need to reserve the old method to make it compatible with PrestoDB/TrinoDB .

alluxio-bot, merge this please

alluxio-bot, merge this please

alluxio-bot, merge this please.


feat(cli): use BufWriter for diagnostic printing

As per suggestions in Discord, I've updated diagnostic printing to use BufWriter rather than the println!() macro.

For performance improvements, it would be great if we post the perf gains as well ;-)


Note that Firebase Admin Node.js SDK doesn't support WLIF

Is there an upstream issue tracking support we can link to? I'm not sure what "authenticate via the firebase sdk" means.

@sethvargo Added a link.
For context, the Firebase SDKs provide another interface to certain GCP services (Firestore, Storage, others) and Firebase projects are GCP/pantheon projects. Generally, authenticating with gcloud etc suffices for our server SDKs, but WLIF support on Node isn't there yet.
https://github.com/firebase/firebase-admin-node/issues/1377
https://firebase.google.com/docs/admin/setup#initialize-sdk

Got it - thanks! I agree we should document this.


Session is not working on IIS with CORS

From @devilsuraj on June 28, 2016 8:19
I am using sessions to manage application state in ASP.NET CORE and it's configured as below.
 services.AddSession(options =>
            {
                options.CookieName = ".my.Session";
                options.IdleTimeout = TimeSpan.FromSeconds(20);
            });

It's working on localhost but on remote IIS 8 it's not creating cookies so not able to get the values. I also have enabled CORS and don't know what exactly caused this problem. In log it's not showing error either. It's creating session getting stored id but no cookie at client side.
Copied from original issue: aspnet/Session#118

From @kichalla on June 28, 2016 16:36
Could you share a sample repro project?

From @devilsuraj on June 28, 2016 19:23
thanks for reply i am following this https://github.com/aspnet/MusicStore
this is the code where session is created
private static string GetCartId(HttpContext context)
        {
            var cartId = context.Session.GetString("Session");

            if (cartId == null)
            {
                //A GUID to hold the cartId.
                cartId = Guid.NewGuid().ToString();

                // Send cart Id as a cookie to the client.
                context.Session.SetString("Session", cartId);
            }

            return cartId;
        }

but tis not working on CORS i am getting cookie in response header but its not set in browser .
everytime it just send me new set cookie in response header , the cookie is never set ,it may be stupid to ask but i am not sure, is it cors related or i have add any cookiedomain or something so it can work with any domain... I am also using angular2 as client

From @Tratcher on June 28, 2016 22:53
What is your CORS policy? Some CORS policies do not allow cookies.

From @devilsuraj on June 28, 2016 23:30
well its not working on subdomains either this is the response header
Set-Cookie:.mysession.Session=e506554c-34b1-8990-de71-a9d12cd4a3ff; domain=.mydomain.com; path=/; httponly
my session config
services.AddSession(options =>
           {
               options.CookieName = ".mysession.Session";
               options.IdleTimeout = TimeSpan.FromSeconds(20);
               options.CookieDomain = ".mydomain.com";
                          });

my CORS policy
 services.AddCors(options =>
            {
                options.AddPolicy("AllowndSTFU",
                    builder => builder.AllowAnyOrigin()
                                     .AllowAnyMethod()
                                     .AllowCredentials()
                                    .AllowAnyHeader());
            });

its generating cookie from backend.mydomain.com and   i need to access it on app.mydomain.com

From @devilsuraj on June 28, 2016 23:14
@Tratcher as for now i have allowed everything in CORS , anyways after researching a lot i found cookies wont work on cross domain for security reasons. anyways i will get the app on subdomain , will test and close the issue

From @Tratcher on June 29, 2016 17:25
(not directly related, but it looks like you're still using RC2, you should move to 1.0).

i updated to 1.0 i think its just Id is not opaque now but its not working with domain name
Set-Cookie:.swapKart.Session=CfDJ8BRCECbMEPVJohJqV5kdf0FLqOw6ICpUMnv%2BC0YsNjAEHsfYe7XkQUuxpwsqKKZeiti1yKdNtDnxEy%2Fms3hdfBVB62fj4XJFHchAv1H3TKALF8RDGj0SKZZZj9VkSxD3rXoROxDYygv79uf%2B2gBp40q%2BJJCwVTaj%2BTU5XG55%2Bzgx; domain=mydomain.com; path=/; httponly
however any update on issue??

cc @Eilon

With "Passing Credentials in Cross-Origin Requests" option, I was able to have the session in CORS over IIS. I am using the option mentioned in this link
http://www.asp.net/web-api/overview/security/enabling-cross-origin-requests-in-web-api#credentials

@kichalla can you continue investigating this? Do we think there's a bug here?

I tried this scenario on IIS and using the article @prasannapattam had shared but for .NET Core. I was not able to repro the issue.
My verification steps:

I created couple of hosts file entries for app.mydomain.com and backend.mydomain.com both pointing to <IP_ADDRESS>

Added site bindings for the above entries in IIS's website

Published the ASP.NET Core site to IIS. The idea here is to simulate the scenario where the UI on app.mydomain.com is trying to access an api hosted on backend.mydomain.com (of course in this case i used the same website for the domain names but i hope this shouldn't matter)

In Startup.cs

services.AddSession(o =>
{
    o.CookieDomain = ".mydomain.com";
});

services.Configure<CorsOptions>(options =>
{
    options.AddPolicy("AllowAll",
        builder => builder.WithOrigins("http://app.mydomain.com:8080")
                        .AllowCredentials()
                            .AllowAnyMethod()
                            .AllowAnyHeader());
});

In controller

    [EnableCors("AllowAll")]
    public class HomeController : Controller
    {
        public IActionResult SetSession()
        {
            HttpContext.Session.SetString("CartId", "10");

            return View(viewName: "Info");
        }

        public IActionResult GetSession()
        {
            return View(viewName: "Info");
        }
    }

In Info.cshtml

        Cart id: @Context.Session.GetString("CartId")

        <button onclick="javascript: work()">Make cross domian request</button>

Closing this issue now. Feel free to re-open if this still does not work for you.


Multi drive pool resize and px restart

What this PR does / why we need it:
Added below scenarios

Multi drive pool resize
pool resize and px restart when resize is in progress
Stablised pool with IOs selection

Which issue(s) this PR fixes (optional)
Closes #PTX-14477,PTX-14478
Special notes for your reviewer:
MultiDriveResizeDisk.log
ResizeWithPXRestart.log
http://aetos.pwx.purestorage.com/resultSet/testSetID/39666/testCaseID/83809
http://aetos.pwx.purestorage.com/resultSet/testSetID/39694/testCaseID/83878
Please note tests are marked failed as pod restart counts are not matching

can you please run this job on your changes? https://jenkins.pwx.dev.purestorage.com/job/Torpedo/job/eks-pool-expand/

can you please run this job on your changes? https://jenkins.pwx.dev.purestorage.com/job/Torpedo/job/eks-pool-expand/

Regression run:
https://jenkins.pwx.dev.purestorage.com/job/Torpedo/job/eks-pool-expand/392/console


Fix CVE-2023-3978

Description
CVE-2023-3978 is probably not included in the compiled binaries but will be detected by tools like Nancy.
Remove deprecated github.com/golang/protobuf/ptypes/empty
Issue link
https://nvd.nist.gov/vuln/detail/CVE-2023-3978
https://jenkins.nordix.org/job/meridio-periodic-security-scan/331/artifact/_output/report.txt
Checklist

Purpose

[x] Bug fix
[ ] New functionality
[ ] Documentation
[ ] Refactoring
[ ] CI

Test

[ ] Unit test
[ ] E2E Test
[ ] Tested manually

Introduce a breaking change

[ ] Yes (description required)
[x] No

/reverify


Unable to locate package libphonenumber on Ubuntu

Any hints? it seems libphonenumber is no longer in ubuntu repos
apt-get install -y libphonenumber
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libphonenumber

Good catch. Should be apt-get install -y libphonenumber-dev. I'll update the docs.

Good catch. Should be apt-get install -y libphonenumber-dev. I'll update the docs.


Route incorrectly seen as orphan route

This route is seen as an orphan route in Knooppuntnet. On Openstreetmap it is visible that the route is part of the Velden en Meersen network.

@A67-A67 Thanks for reporting this problem.
The document with id "network:8300603" in couchdb does not contain route 8304581. This is why the route is seen as an orphan route.  This is strange because in de overpass database, network 8300603 does contain the route, and the last time the network was retrieved from overpass, the query result also contained the route.
This has to be further investigated.

relation(8300603);(>>;);out meta;
./05/08/20/37/54/relation-8300603.xml.gz
./05/08/20/37/57/relation-8300603.xml.gz
After recreating the database from scratch, all is ok.


Update the file with an existing Cargo version

Please have a look at the individual commit messages for the details.

The Cargo update was already done in https://github.com/oss-review-toolkit/ort/pull/7248... but it does not help if no one reviews it 😉

Closed in favor of https://github.com/oss-review-toolkit/ort/pull/7248.

Note that the second commit in this PR is probably still valid... you could have just rebased this once #7248 is merged.

Yes I know, I won't forget.
This commit will piggyback another PR 😄


Support AlphaFold 3 PDB dataset curation

Adds initial support for AlphaFold 3 PDB dataset curation via the new alphafold3_pytorch/pdb_dataset_curation.py script.
The NOTE: references spread throughout the dataset curation script denote areas where the implementation could use additional feedback or where it relies on specific assumptions that are explicitly stated.
That being said, any and all feedback on this script is welcome!

lgtm!


ManagedPolicyArns throws error referring to policy created in the same template.

*cfn-lint version: cfn-lint 0.58.3
Description of issue.
Create IAM:
InstanceIamPolicy:
    Type: 'AWS::IAM::ManagedPolicy'
    ...

Refer the policy in a new IAM:Role
Ec2InstanceRole:
    Type: "AWS::IAM::Role"
    ManagedPolicyArns:
        - !Ref: InstanceIamPolicy
        ...

Linting shows an error
[cfn-lint] E3002: Property Resources/Ec2InstanceRole/Properties/ManagedPolicyArns/0 has an illegal function Fn::Ref:

It should not be !Ref: but just Ref
Ec2InstanceRole:
    Type: "AWS::IAM::Role"
    ManagedPolicyArns:
        - *Ref*: InstanceIamPolicy


[WIP] Peer management

This PR adds preliminary functionality to facilitate management of a network of replicas, including adding, removing, and getting status reports.
I've implemented the rpc methods such that they work with the current leveldb database, and peers can be managed via functions defined in replica.go rather than interacting directly with the database. In this implementation, peers are not really deleted, just tombstoned. This required updating the protocol buffer definitions of Peers to include deletion date and other metadata fields, which is why the pb files have been impacted in this commit.
Marking as WIP since the code still requires tests to make sure everything works. I also haven't tested out the gdsutil commands yet, since I was hoping we could clarify some of the requirements together before I finalized anything.

Codecov Report

Merging #64 (6a13ff8) into v1.1 (c639b08) will decrease coverage by 0.79%.
The diff coverage is 6.55%.

:exclamation: Current head 6a13ff8 differs from pull request most recent head 0b93e00. Consider uploading reports for the commit 0b93e00 to get more accurate results

@@            Coverage Diff             @@
##             v1.1      #64      +/-   ##
==========================================
- Coverage   20.86%   20.07%   -0.80%
==========================================
  Files          26       26
  Lines        2856     2969     +113
==========================================
  Hits          596      596
- Misses       2124     2237     +113
  Partials      136      136

Impacted Files
Coverage Δ

pkg/gds/global/v1/global.pb.go
11.65% <ø> (ø)

pkg/gds/global/v1/global_grpc.pb.go
0.00% <0.00%> (ø)

pkg/gds/models/v1/models.pb.go
19.18% <ø> (ø)

pkg/gds/replica.go
0.00% <0.00%> (ø)

pkg/gds/store/leveldb/leveldb.go
31.01% <0.00%> (-5.01%)
:arrow_down:

pkg/gds/store/store.go
24.13% <ø> (ø)

pkg/gds/emails/bindata.go
47.02% <100.00%> (ø)

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 c639b08...0b93e00. Read the comment docs.


az vm create wont work with --image 'MicrosoftSQLServer:SQL2017-WS2016:Standard:14.1.200114'

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
I try to run:
az vm create    -n  'xxxxx'                     -g  'xxxxx'
--admin-password 'xxxxx'                     --admin-username 'xxxxx'
--vnet-name 'xxxxx'                     --subnet 'xxxxx'
--image 'MicrosoftSQLServer:SQL2017-WS2016:Standard:14.1.200114' `
-l  'NorthEurope'
and getting:
--- Logging error ---
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 636, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init.py", line 306, in call
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core_init_.py", line 493, in default_command_handler
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli\command_modules\vm\custom.py", line 830, in create_vm
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-mgmt-resource\azure\mgmt\resource\resources\v2019_07_01\operations_deployments_operations.py", line 2695, in create_or_update
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-mgmt-resource\azure\mgmt\resource\resources\v2019_07_01\operations_deployments_operations.py", line 2642, in _create_or_update_initial
msrestazure.azure_exceptions.CloudError:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands\arm.py", line 103, in handle_template_based_exception
AttributeError: 'str' object has no attribute 'message'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\knack\knack\cli.py", line 206, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 608, in execute
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 666, in run_jobs_serially
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init.py", line 657, in _run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands\arm.py", line 105, in handle_template_based_exception
knack.util.CLIError:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\logging_init_.py", line 993, in emit
msg = self.format(record)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\logging_init_.py", line 839, in format
return fmt.format(record)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\logging_init_.py", line 576, in format
record.message = record.getMessage()
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\logging_init_.py", line 336, in getMessage
msg = str(self.msg)
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\msrestazure\msrestazure\azure_exceptions.py", line 193, in str
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\msrestazure\msrestazure\azure_exceptions.py", line 120, in str
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 912: ordinal not in range(128)
Call stack:
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli_main.py", line 41, in
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli_main.py", line 30, in cli_main
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\knack\knack\cli.py", line 218, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core_init_.py", line 107, in exception_handler
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\util.py", line 47, in handle_exception
Message: CloudError("The template deployment 'vm_deploy_TBgw6rQGH9rQGc8PZCMcXEZ940UNVRVF' is not valid according to the validation procedure. The tracking id is 'b4704780-3edc-4c0a-9d63-13f562dd71eb'. See inner errors for details.",)
Arguments: ()
--- Logging error ---
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 636, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init.py", line 306, in call
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core_init_.py", line 493, in default_command_handler
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli\command_modules\vm\custom.py", line 830, in create_vm
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-mgmt-resource\azure\mgmt\resource\resources\v2019_07_01\operations_deployments_operations.py", line 2695, in create_or_update
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-mgmt-resource\azure\mgmt\resource\resources\v2019_07_01\operations_deployments_operations.py", line 2642, in _create_or_update_initial
msrestazure.azure_exceptions.CloudError:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands\arm.py", line 103, in handle_template_based_exception
AttributeError: 'str' object has no attribute 'message'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\knack\knack\cli.py", line 206, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 608, in execute
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init_.py", line 666, in run_jobs_serially
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands_init.py", line 657, in _run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\commands\arm.py", line 105, in handle_template_based_exception
knack.util.CLIError:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\logging_init_.py", line 993, in emit
msg = self.format(record)
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\knack\knack\log.py", line 78, in format
msg = str(self.msg)
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\msrestazure\msrestazure\azure_exceptions.py", line 193, in str
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\msrestazure\msrestazure\azure_exceptions.py", line 120, in str
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 912: ordinal not in range(128)
Call stack:
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli_main.py", line 41, in
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli\azure\cli_main.py", line 30, in cli_main
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\knack\knack\cli.py", line 218, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core_init_.py", line 107, in exception_handler
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-3pnsb50r\azure-cli-core\azure\cli\core\util.py", line 47, in handle_exception
Message: CloudError("The template deployment 'vm_deploy_TBgw6rQGH9rQGc8PZCMcXEZ940UNVRVF' is not valid according to the validation procedure. The tracking id is 'b4704780-3edc-4c0a-9d63-13f562dd71eb'. See inner errors for details.",)
Arguments: ()
To Reproduce
Steps to reproduce the behavior.
run above
Expected behavior
A clear and concise description of what you expected to happen.
vm is created based on image
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
windows server,
visual studio code powershell integrated terminal (latest updates),
Python 3.8.0
az version -
"azure-cli": "2.1.0",
"azure-cli-command-modules-nspkg": "2.0.3",
"azure-cli-core": "2.1.0",
"azure-cli-nspkg": "3.0.4",
"azure-cli-telemetry": "1.0.4",
"extensions": {
"aks-preview": "0.4.29",
"dev-spaces": "1.0.3"
Additional context
to get the image list available run:
az vm image list --sku Standard --offer SQL2017-WS2016  --publisher MicrosoftSQLServer --location NorthEurope --all -o table

Ok I found:
it works with python 3.5.2.
thanks ......

hi @dariuszbz is it ok to close the issue now?

Hi @yungezz I think you have to install the proper python version with az, or check the python version to give the proper error message to the user.
But it's ok for me to close. It's working now with the python 3.5.2.


Browser spoofing feature

I am attempting to use Bowser with Firefox Hello, but at this time Firefox Hello is seeing my iOS user agent string and giving me an error about iOS not supporting WebRTC. Normally this would be a correct assumption, but thanks to Bowser this is not the case. I plan to send feedback to Mozilla in regards to this, but I also suggest a user agent spoofing feature for Bowser. This will ensure users won't encounter similar issues and be stuck without a solution.

Yes, this is a recurring problem since various services makes assumptions on capabilities instead of doing feature probing. It is quite easy to change the UserAgent in code (see https://github.com/EricssonResearch/bowser/blob/master/bowser-ios/Bowser/BowserAppDelegate.m#L34) but a better solution would be to have a UI in the app for switching between different options.
If you are willing and able to help out we would really appreciate it.


Fix error in example

Hi!
Thanks for this PR, this is indeed a typo in the docs, well spotted!
Currently on the stable release branch 2.2 this has been corrected by https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2677 , however it seems that fix was not applied to the master branch.
(ping @keradus , would you want to merge 2.2 -> master, or take a new PR for master?)

This PR and #2677 fix typos in different places.
I checked and it seems #2677 is merged to master

You are absolutely right!
Can't believe I missed the typo again  :|
Nice fix, thanks again :)

Thank you @mleko.


Is it possible to enable SOURCEMAPS for debugging?

I spent quite some hours trying to figure out this, is it possible? I used several flags and none of them seems to override the default mechanism of CSS concatenation of Rails.

+1

+1

See https://github.com/rails/sass-rails/issues/314#issuecomment-75968784


helm pull --repo fails for oci helm charts

Running helm pull with --repo for WordPress Helm Chart fails,
helm pull wordpress --repo oci://registry-1.docker.io/bitnamicharts/wordpress --version 17.1.17

Is --repo not a supported flag for OCI?
Output of helm version:
version.BuildInfo{Version:"v3.12.0", GitCommit:"c9f554d75773799f72ceef38c51210f1842a1dea", GitTreeState:"clean", GoVersion:"go1.20.3"}
Output of kubectl version:
lient Version: v1.27.2
Kustomize Version: v5.0.1

helm pull --repo ... allows pulling a chart from a chart repository (which is different to an OCI repository). Without doing a helm repo add
For OCI, this would be:
helm pull oci://registry-1.docker.io/bitnamicharts/wordpress --version 17.1.17
Agree, this could be better documented

Great, thank you for clarifying.


Pulsar standalone 2.10 won't start with Java 8

Describe the bug
Tried to upgrade Pulsar 2.9.1 to newer version 2.10, but 2.10 won't start while 2.9.1 has no problems.
Error:
bin/pulsar standalone

Unrecognized option: -Xlog:gc*:logs/pulsar_gc_%p.log:time,uptime:filecount=10,filesize=20M
Error: Could not create the Java Virtual Machine.

Java Version:
java version "1.8.0_202"
Java(TM) SE Runtime Environment (build 1.8.0_202-b08)
Java HotSpot(TM) 64-Bit Server VM (build 25.202-b08, mixed mode)

Is a newer Java version now mandatory?
Docs still say Java 8 64bit (or higher) is required

do you have available JAVA_HOME variable?

Ahh you're right, JAVA_HOME and PATH wasn't correctly set!
Thanks for the quick help


Fix CI badge image

I guess the name of the CI build has been changed which led to a broken image.

Nice, thanks!


Attempting to reconnect modal should not be an overlay?

When working locally and watching logs from a server, if the server/process dies or is stopped, there still might be valuable debugging information in the logs that I would like to read/copy even after the server is stopped.
The reconnect overlay makes reading these hard, copying not possible. It could very well be some modal dialog element that doesn't block the page this way and improve the UX.

@kvenkatrajan I think we should do this in preview 5, it's a long standing annoying work item.
@SteveSandersonMS is there anything we should know WRT blazor to make this reconnet experience a bit better? Background retries making it possible to navigate would be great but that likely requires more work (and client state?).

@davidfowl We're tracking the general need to improve the reconnection experience at https://github.com/dotnet/aspnetcore/issues/32113.
We've prototyped a possible improved UX at https://github.com/SteveSandersonMS/StatefulReconnection. If you're interested in trying that out in the Aspire dashboard, we'd definitely appreciate any feedback on it. Of course, we're open to other suggestions too.
As for the default reconnection modal making things unselectable, this is something that Blazor developers can control, replacing it with any HTML/CSS: https://learn.microsoft.com/en-us/aspnet/core/blazor/fundamentals/signalr?view=aspnetcore-3.1#reflect-the-server-side-connection-state-in-the-ui

@SteveSandersonMS unfortunately it doesn't look like that UX meets our needs, as it still appears as a modal


GenerateChangePhoneNumberTokenAsync too long!

I am use identity add a phone number, and generate an token, but when I send sms, the supplier told me the token it's too long!

Could provide a method it can generate some  number code ?

Yeah this has been fixed in 2.0.3 and 2.1 nightly builds, dupe of https://github.com/aspnet/Identity/issues/1388