Discover local dependencies outside of immediate node_modules

I really hope that I'm just missing something obvious, so please close this issue if I just misread the documentation or something.
Let's say I have a very simple module with a single index.js that I want to test. If I have this in my tachometer.json:
{
  "benchmarks": [
    {
      "url": "./benchmark.html",
      "expand": [
        {
          "name": "this-change"
        },
        {
          "name": "tip-of-tree",
          "packageVersions": {
            "label": "tip-of-tree",
            "dependencies": {
              "my-package-name": {
                "kind": "git",
                "repo": "https://github.com/org/my-package-name.git",
                "ref": "master"
              }
            }
          }
        }
      ]
    }
  ]
}

And then I use this in my benchmark.html:
import myPackage from 'my-package-name'

This actually works for the remote dependency, but it doesn't work for the local dependency. Instead, Tachometer doesn't transform the import statement, so the browser can't find the module.

Here is a small repro.
The only solution I've found is to manually add symlinks so that node_modules/my-package-name is linked locally:
ln -s .. node_modules/repro-tach-dependency

Then Tachometer works as expected - the local dependency is resolved locally, and the remote one is resolved by fetching it from npm.
I've also observed this issue in a monorepo, where the project structure is like this:
packages/my-package-name
packages/my-package-name/index.js
packages/benchmark
packages/benchmark/benchmark.html

In this case, if benchmark.html tries to do import 'my-package-name', it won't resolve locally.
This might have something to do with subtleties of dependency hoisting. In a monorepo project without dependency hoisting, packages/benchmark/node_modules would indeed contain the my-package-name package. But in the case I ran into, dependency hoisting moves all the dependencies to the top-level node_modules, so Tachometer can't find it, because it seems to check only the immediate node_modules directory.
The solution I've found is again to do the node_modules symlinking, e.g.:
ln -s ../../my-package-name ./packages/benchmark/node_modules/my-package-name

To solve both these issues (if this is indeed an issue and I didn't just misconfigure something), it seems to me that the Tachometer module resolver should:

search for packages at the root level via package.json (for the simple single-package case)
search recursively up the tree for node_modules (for the hoisted monorepo case)

Thanks for reading this far, and thanks for creating Tachometer! It's a great tool, and I can use it just fine with the symlinking workaround, but I thought I'd report the issue in case others ran into it or I missed something. :slightly_smiling_face:

I wrote a better repro of the issue.
After a bit of debugging, it seems that koa-node-resolve correctly resolves some-package  to ../node_modules/some-package, but then at some point this gets transformed into <rootDir>/node_modules/some-package/dist/index.js, which is wrong because it should be <rootDir>/../node_modules/some-package/dist/index.js. My hunch is that Koa is refusing to serve files outside of the directory that the server is being run from.
The workaround for now still seems to be to put your tachometer.json file at the root level – not in a subdirectory. Or, do the ln hack above.

After thinking about this more deeply, I think this can be solved with:

root (as described here: https://github.com/google/tachometer/issues/244#issuecomment-2267191898)
not referring to your own package as a bare 'my-package-name' but instead as e.g. './index.js'

I'm not sure why the resolution works differently for remote vs local tests, but in any case, Tachometer is probably right that it would be weird to resolve 'my-package-name' to the current directory when running inside the my-package-name directory itself. For example, this doesn't work in Node:
mkdir foo
cd foo
npm init --yes
node -e "require('foo')" # Error: Cannot find module 'foo'

So since the resolution doesn't work in plain Node, you wouldn't really expect it to work in Tachometer either. So referring to your own local package as e.g. './index.js' makes a lot more sense.
As for the monorepo issue, I think this is the same as #244 and can be resolved with root. Closing this issue. Sorry for the noise!

For the record, this issue was actually trickier to resolve than I thought. I'll put a summary here in case it helps anyone else.
Situation: you have a package, e.g. my-package, and you want to test the local branch versus a remote (e.g. main). There is no monorepo; you want to test a single package and have Tachometer swap it out.
Resolution: use a placeholder package in your package.json:
"devDependencies": {
  "@scope/placeholder": "file:."
}

Then import from that placeholder in your benchmark scripts:
import foo from '@scope/placeholder'

Then tell Tachometer to swap out the placeholder package when resolving the main branch dependency:
  "expand": [
    {
      "name": "this-change"
    },
    {
      "name": "tip-of-tree",
      "packageVersions": {
        "label": "tip-of-tree",
        "dependencies": {
          "@scope/placeholder": {
            "kind": "git",
            "repo": "https://github.com/scope/package.git",
            "ref": "main"
          }
        }
      }
    }
  ]

It's a little gnarly, but it works! Note the @scope to avoid accidental dependency confusion attacks.


Restructure code

This commit contains following changes:

Move assets to root folder.
Delete template folder.

solved


Abbreviated when clause unexplained

Nice article.
However, there is a "race condition" in the text:

In the preceding example, the when clause was omitted on the final case.

Up to that time in the article, the term "when clause" neither occurs or is getting explained.
The previous section merely introduces a "property pattern" and doesn't provide any relation between a "property pattern" and a "when clause", nor does it introduce the term "when clause".
So, the above mentioned sentence leaves behind a buffled reader.

I propose to introduce the term "when clause" in the introduction to "Add occupancy pricing" and create a relation between a "property pattern" and a "when clause".
Something like this may be appropriate (I adorned my edit in italics):

These rules can be implemented using the property pattern in the same switch expression. A property pattern is a special case of when clause, comparing a property value to a particular (constant?) value. The property pattern examines properties of the object once the type has been determined.

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

ID: 7bc5c533-a9ca-cc70-af20-21308e819082
Version Independent ID: ef6af5d2-b640-0feb-d9d3-7aae0ca92264
Content: Tutorial: Build algorithms with pattern matching
Content Source: docs/csharp/tutorials/pattern-matching.md
Product: dotnet-csharp
Technology: csharp-whats-new
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @SetTrend
I like your suggestion. I made a couple edits to match our style guide:

These rules can be implemented using the property pattern in the same switch expression. A property pattern is a when clause that  compares a property value to a constant value. The property pattern examines properties of the object once the type has been determined.

I've added this to our backlog, and I've added the "up for grabs" label if you or someone else wants to create a PR before I make the updates.

Thanks for noting this @SetTrend
I like your suggestion. I made a couple edits to match our style guide:

These rules can be implemented using the property pattern in the same switch expression. A property pattern is a when clause that  compares a property value to a constant value. The property pattern examines properties of the object once the type has been determined.

I've added this to our backlog, and I've added the "up for grabs" label if you or someone else wants to create a PR before I make the updates.


Allow import of PKCS12 (pfx) certificates. Issue #8645

[ ] Redmine Issue: https://redmine.pfsense.org/issues/8645
[ ] Ready for review

Allows user to select a PKCS #12 certificate store to import a certificate. Optionally allows for import of bundled intermediate certificates. Tested with a couple of different certificates.
updated PR https://github.com/pfsense/pfsense/pull/4063 with resolved conflicts
original PR can be closed

added the ability to import X.509 without private key, see https://github.com/pfsense/pfsense/pull/4102
tested on 2.5.0.a.20200227.1722

@vktg please rebase your fork and resolve conflicts

Rebased


Add explicit support for freestanding environments

Sunder the language does not strictly require a hosted environment or even an operating system to run (although the semantics of the syscall construct would be iffy in such a case). It would be nice if there was explicit support for compiling Sunder code in a freestanding environment. A specific use case would be the development of an Operating System kernel.
I think the only major change that would be required in order to support freestanding environments would be a way to customize which file is used for sys.asm (the equivalent of C's crt0 and friends). Currently this file is hard coded to be $SUNDER_HOME/lib/sys/sys.asm, so someone could work around this issue by replacing the sys.asm file in a local install, but it would be nice if there was a way to explicitly override this behavior.
Perhaps the compiler could check for a usually-undefinded environment variable like SUNDER_SYS_ASM and if it is defined then that file will be used instead of $SUNDER_HOME/lib/sys/sys.asm.

Added in 3a935efa3d6d54839f9d943ca92c027a9b52b531 and add62fe14fae4878c89ba954d336ce9e5ad34010.


gz_ros2_control on jazzy: Use correct branch

Blocked by https://github.com/ros-controls/gz_ros2_control/issues/395


Fix the carriage returns in RFM69HCW

Signed-off-by: Tzachi Dar<EMAIL_ADDRESS>Without this change, the lines on the md doc seems very hard to understand.

After creating the PR, I have noticed that things look well before my change on github (for example on: https://github.com/tzachi-dar/docs/blob/master/docs/docs/Gear Up/edison.md) but not on
https://openaps.readthedocs.io/en/latest/docs/Gear Up/edison.html.
However, most people look at openaps.readthedocs.io.
So, I guess that the simplest way around this, is to replace the text with a picture.
Does this sound reasonable?

I would do both. (Essentially, add a picture below for clarity, but leave the text so it's easy to update it + generate a new picture in the future)

Closing this, as https://github.com/openaps/docs/pull/1428 was opened.


[WIP] Fixed wrong operator use in condition

Hello,
this fix will allow Unit of Work to stop early when there is nothing to do. Currently Doctrine starts transaction and commits it each time flush is called as this condition is invalid.

Looks like the condition is okay, I just worked with entities in a bad way.


Remove the Persistence folder reference

The Persistence folder did not exist in project and Xcode show warning, so remove it.

👍


Vickel: Watch imediasolutions.in

Vickel requests the watch of the watch_keyword imediasolutions\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

imediasolutions\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Ollie in Charcoal HQ


chore: changes display message in api docs for PutRead

What does this PR do

changes the message displayed in the docs to be more clear
mark message as read

Screenshot

Fixes issue #388

Hi @robdefeo I did it manually at first. However, I've done it using the command and pushed the changes.

Awesome @rickynyairo this is great. Using make openapi is better as it will soon be part of the build process. Thanks for your first contribution! 🎆


Fixed work with sinon.useFakeTImers() and Rhino.js 1.7R3

This is fix for my blog post about Sinon.js and Fake timers: Headless JavaScript testing, Sinon.js, Fake Timers and Rhino.
As promised, I've made environment with Rhino and Sinon.js where test for Fake Timers is not work properly (download). To execute tests use ./runtests.sh.
The main reason of issue is that Rhino.js use global object called global which cause conflict with Sinon.js global object recognition.
I've not wrote regression tests because most of the tests are failed within Rhino.js.

Thanks. I'll look into making the tests run properly on Rhino too.


How does the size work?

When using with an input like this:
<input class="input is-small">

I was expecting the tags-input to keep the correct input size, but instead it became much bigger. Looking at the transformed html, I now have this:
<div class="tags-input">
  <input class="input is-small">
</div>

But looking at the sass file, it seems the size will only work if is-small is applied on .tags-input, which is not the case here. Is it a bug? or an option to give when using new BulmaTagsInput(selector, options);

Actually even when I add the is-small class on the tags-input, the input it still much bigger than it should

Temporary solution:
.tags-input {
  min-height: auto;

  .input {
    border: 0;
    height: initial;
    margin: initial;
  }
}


Title message not used correctly when validating on keyup [bug]

When fields are validated on key up, the title is not used as the error message... The default error message will show.
However when any other validation call is made, the title attribute message is used?

Ah, this seems to be an issue with the pattern plug-in...
`$.validator.addMethod("pattern", function(value, element, param) {
if(this.optional(element)) return true;
if(typeof param === "string") param = new RegExp("^(?:" + param + ")$");
return param.test(value);

}, 'Invalid format.');`
It will show the default message sometimes when the title attribute is needed... hmmm

hmm on further inspection of core.js it looks like it is just not using the title attribute when running a customer method for some reason... I can't figure out why or how to fix/patch it.

some other unrelated jquery plugin was momentously removing title attribute on key press... No bug.


usbc: explicitly declare sizes of state machines arrays

The asserts in the USB-C stack functions were causing compilation errors when asserts were enabled in Kconfig.
This was caused because the asserts checked if the functions parameters were within limits specified by the state machine arrays.
At the position of these asserts, these arrays were only declared without specifying the size of them, what prohibited the asserts from knowing the size of arrays and checking if specified value is within limit of it.

It is not clear from the commit message. How exactly does it relate to CONFIG_ASSERT?

Sorry for not being descriptive enough. In the USB-C stack code, there were asserts that checked if the function parameter is within the limits of the states array size, e.g. function void pe_set_state(const struct device *dev, const enum usbc_pe_state state) has assert
__ASSERT(state < ARRAY_SIZE(pe_states), "invalid pe_state %d", state);
Without asserts enabled, this line is not compiled, however enabling the asserts makes this functional and the assert fails because the pe_states has unknown size at the moment, because the array is declared earlier, but defined later at the end of file.

It is not clear from the commit message. How exactly does it relate to CONFIG_ASSERT?

Sorry for not being descriptive enough. In the USB-C stack code, there were asserts that checked if the function parameter is within the limits of the states array size, e.g. function void pe_set_state(const struct device *dev, const enum usbc_pe_state state) has assert __ASSERT(state < ARRAY_SIZE(pe_states), "invalid pe_state %d", state);
Without asserts enabled, this line is not compiled, however enabling the asserts makes this functional and the assert fails because the pe_states has unknown size at the moment, because the array is declared earlier, but defined later at the end of file.

I see, thank you. Maybe you can update the update commit message.

DNM until @sambhurst has had a chance to review.

Maybe you can update the update commit message.

Sure, done.

@tmon-nordic please take a look


BUTTON REMAPS - Adds edit feature for button remaps, and major clean up of code.

Advmame Button Remaps Changes:
Add Edit feature to custom button remaps.
Major clean up of code - Before it was messy and the function signatures were long and complicated. This revision majorly simplifies everything.
I have tested the latest revision a bit, give me a day or two to test it more thoroughly but from what I've tested it's looking good.

@shantigilbert
These changes are real sexy dude, much better code. LMFAO ^_^

Ok, I'll test these changes formally tomorrow. I havent tested that the button assignments actually take effect in game properly.

Ok, I'll test these changes formally tomorrow. I havent tested that the button assignments actually take effect in game properly.

Thanks!

@shantigilbert
I am confident that all the changes are working properly. I tested this patch thoroughly selecting, creating, deleting, and editing and all is working fine and could not find any bugs. Latest version ready to merge.

Ok, give some time to test it and I will merge it ASAP. Thanks

Due to the ammount of commits I cannot test this as a patch, can you squash them into one please.

There is one small issue, when you go into create remap you select say button X first, then the X button gets set to "none" now when you move to the next, and when you are moving around with the buttons if you change it to X then the X gets set to "none: making it really hard to configure. This again assuming if you use left and right to select the buttons. I suggest removing that check

Ah ok, I will make it so it only checks if you use the popup. I rarely change it using the left and right. Ok put commit on hold and I will make that change today.

I rarely change it using the left and right.

But a lot of people (including me) do use the left and right, and also, it could be done by mistake

Close this PR I'll make a clean new one!


Create Code_of_conduct-ben.md

Bengali translation of the Code of conduct
What does this PR do?
adds info in another language
Why is this valuable?
valuable for the ones who are comfortable in Bengali.

use bn language code, not ben

I have used the language code bn

I have used bn language code, please review the newly made changes
On Sun, 16 Oct 2022 at 03:27, Eric Hellman @.***> wrote:

use bn language code, not ben
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/8400#issuecomment-1279839363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWVBOVLKJQ7KNOVZCUITTELWDMSGJANCNFSM6AAAAAARF6EFM4
.
You are receiving this because you authored the thread.Message ID:
@.***>

please add link in docs/README.md


Delete-until-character operations incorrectly move cursor to new line

Example source code:
if (foo == 'bar')
  console.log('ok')

Place the cursor on the letter b
Perform any vi edit-until-character operations (e.g. dt' or df')

Expected
The cursor should remain on the letter b after the deletion is performed (or, at least, this is how vi/vim/neovim behave).
Actual
The cursor, for some reason, is moved to the beginning of the next line.
Note
Only the d command with cursor motion instructions (i.e. dt* and df*) seem to be affected.
di*, ci*, vt* etc. are not affected by this behaviour (the cursor doesn't jump to the next line after execution).

not reproducible. try with empty init.vim

The issue seems to have been resolved by itself somehow. I tried with an empty init.vim and everything worked, then I tried to restore my init.vim (the exact same one that replicated the issue) and everything still worked. That's a bit weird - I'll close this issue for now and I'll reopen it if the issue pops out again.

@asvetliakov the issue had previously gone only because I had commented large swaths of configuration in my init.vim.
Once I uncommented everything and restored all the plugins, the issue came back.
After a lot of bisecting to find the culprit, I realized that the issue isn't caused by any of my custom mappings but by a vim plugin - in my case yankring.
When the plugin is disabled, typing e.g. dt. works as expected. When the plugin is enabled, the command moves the cursor to the line below.
Since I haven't really used that plugin in ages (and the last commit on that repo is also >7 years old), removing it isn't a big deal for me, and I haven't really investigated what's happening under the hood that messes the state of the cursor only in vscode but not in nvim.


Assistant does not wok on cherishOS (using Pixel Launcher port from Pixel 5)

Assistant does not work even with the pixel 5 launcher port. April security patch
Video of issue: https://streamable.com/5eshmo

When i disable the pixel props module it goes back to normal and assistant 2.0 opens, but if i enable it breaks assistant again :(

Assistant does not work even with the pixel 5 launcher port. April security patch
Video of issue: https://streamable.com/5eshmo

okay that didn't open i think you can attach mp4 in issues but okay let me try with vpn

https://user-images.githubusercontent.com/68574209/118525699-7cbab200-b70d-11eb-9975-ce08003e7757.mp4

Your GA 2.0 overlay is working fine only glow is missing. In the last issue did you use the pixel launcher module with navbar stuff?

Since google app crashed when you triggered for assistant. I would suggest cleaning it's data.
In nutshell
For glow to work

Use these variants - Pixel Launcher R v3.1 or v3.1.1  from here - https://forum.xda-developers.com/t/magisk-pixel-launcher-for-android-11.4213619/

Also once this get's installed update the pixel launcher to latest release from here - https://www.apkmirror.com/apk/google-inc/pixel-launcher/pixel-launcher-11-release/

That's what i also had to do on my cherish OS 2.5 for realme 6

yes i downloaded the latest pixel launcher apk and i have the variant with all the navbar things, i tried clearing data, re-installing if i remove the module it starts working again with glow an everything, something that the module does is breaking it

https://streamable.com/k9gayl

update: i kinda fixed it, i used Weiju to change the google app device fingerprint to a random one and it worked, its ga 1.0 tho still better than no ga at all tho

update 2.0: one time i got the module and ga 2.0 working but after a few seconds it reverted back to ga 1.0, very weird

https://streamable.com/iafivn

https://streamable.com/eawpw8

and ok google is also broken

https://streamable.com/k9gayl This video i show that it sometimes opens but its bugged

First things first-  Your Cherish OS doesn't seem to me to be gapps build to me seeing the videos you linked.
And also
Google never asks for permission like this, its a system-priv app in gapps build

So if you flashed vanilla and used  some package of gapps after that. Then all your weirdness is because of it.

and ok google is also broken

remove the module and check google hotword detection if it works or not.

Which device is yours?

Im using an s20 fe 5g with braialindo’s offcial gsi, includes gapps but its a very minimal gapps with only play store and some other stuff by default, i had to install google app from play store

Out of scope...


Multiple RTSP stream

Hi
I am using this application and I would like to use more than one camera live stream from our office, so I changed the source for Multiple URI and added the 3 cameras rtsp addresses but in the end, I only got one frame and kafka messages from the frame only.
Any recommendation to display the 3 frames and get messages regarding the different livestreams?
Thanks

Hi JanetSe,
how did you add the 3 cameras? please share the config file.

change you file which is dstest_occupancy_analytics.txt with below code
################################################################################
Copyright (c) 2018-2020, NVIDIA CORPORATION. All rights reserved.

Permission is hereby granted, free of charge, to any person obtaining a
copy of this software and associated documentation files (the "Software"),
to deal in the Software without restriction, including without limitation
the rights to use, copy, modify, merge, publish, distribute, sublicense,
and/or sell copies of the Software, and to permit persons to whom the
Software is furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in
all copies or substantial portions of the Software.

  IN NO EVENT SHALL
THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
DEALINGS IN THE SOFTWARE.
################################################################################
[application]
enable-perf-measurement=1
perf-measurement-interval-sec=5
#gie-kitti-output-dir=streamscl
[tiled-display]
enable=1
rows=1
columns=2
width=1280
height=720
gpu-id=0
nvbuf-memory-type=0
[source0]
enable=1
type=4
uri=stream1
num-sources=1
gpu-id=0
nvbuf-memory-type=0
smart-record=2
smart-rec-start-time=1
smart-rec-file-prefix=smart_record
smart-rec-cache=10
[source1]
enable=1
type=4
uri=stream2
num-sources=1
gpu-id=0
nvbuf-memory-type=0
smart-record=2
smart-rec-start-time=1
smart-rec-file-prefix=smart_record
smart-rec-cache=10
[sink0]
enable=1
type=2
msg-conv-config=msgconv_sample_config.txt
msg-conv-msg2p-lib=/opt/nvidia/deepstream/deepstream-6.0/sources/apps/sample_apps/deepstream-occupancy-analytics/bin/jetson/libnvds_msgconv.so
msg-conv-payload-type=0
msg-broker-proto-lib=/opt/nvidia/deepstream/deepstream-6.0/lib/libnvds_kafka_proto.so
msg-broker-conn-str=localhost;9092;quickstart-events
source-id=0
[sink1]
enable=1
type=2
msg-conv-config=msgconv_sample_config.txt
msg-conv-msg2p-lib=/opt/nvidia/deepstream/deepstream-6.0/sources/apps/sample_apps/deepstream-occupancy-analytics/bin/jetson/libnvds_msgconv.so
msg-conv-payload-type=0
msg-broker-proto-lib=/opt/nvidia/deepstream/deepstream-6.0/lib/libnvds_kafka_proto.so
msg-broker-conn-str=localhost;9092;quickstart-events
source-id=1
[osd]
enable=1
gpu-id=0
border-width=1
text-size=10
text-color=1;1;1;1;
text-bg-color=0.3;0.3;0.3;1
font=Arial
show-clock=0
clock-x-offset=800
clock-y-offset=820
clock-text-size=12
clock-color=1;0;0;0
nvbuf-memory-type=0
[streammux]
gpu-id=0
live-source=1
batch-size=2
batched-push-timeout=40000
width=1920
height=1080
enable-padding=0
nvbuf-memory-type=0
[primary-gie]
enable=1
gpu-id=0
batch-size=2
bbox-border-color0=1;0;0;1
nvbuf-memory-type=0
interval=0
config-file=pgie_peoplenet_tao_config.txt
[tracker]
enable=1
tracker-width=640
tracker-height=384
gpu-id=0
ll-lib-file=/opt/nvidia/deepstream/deepstream-6.0/lib/libnvds_nvmultiobjecttracker.so
ll-config-file=../../../../samples/configs/deepstream-app/config_tracker_NvDCF_perf.yml
enable-batch-process=0
[nvds-analytics]
enable=1
config-file=nvdsanalytics_config.txt
[tests]
file-loop=0


Fixed Tcl_AsyncDelete crash after closing dialog (#4993)

Reverting the change introduced in the fix for #1466 and fixing it in a different way without the side effect of #4993.

I added an acceptance test, but I'm not sure if it should be merged as failing it crashes RF and leads to false negatives. I am willing to improve the test, but I would need some pointers in the right direction.

I tested this locally and the added new test succeeded also without the fix. I got it to fail by calling Evaluate before Pause Execution and the provided change fixed it. I'll update the test accordingly and merge this PR. Thanks for contribution!

I'm happy if I could help a little. Thank you for taking the time to review the pull request and even improving the test. I checked the submitted test only on one machine, and I apologize for the inconvenience this caused for you. If any related issues arise, feel free to mention me so I can take a look.


data:Ukaypromise

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@SaraJaoude I have made the required changes.


NuGet.org accepted licenses underspecified

Issue moved from dotnet/docs#20963

Please respond to @boringcactus.

From @boringcactus on Wednesday, October 7, 2020 4:43:22 PM
The documentation on the PackageLicenseExpression property says

NuGet.org accepts only OSI or FSF approved licenses when using license type expression.

However, I tried to use the OSI-approved zero-clause BSD license, which is listed at the very top of the SPDX version permalinked in that section of the docs, and it was rejected as unsupported. Evidently only a subset of OSI or FSF approved licenses are allowed by NuGet.org, in which case the documentation should say which ones are allowed.

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

ID: 06a19d7d-253d-f17f-6421-624c9867473d
Version Independent ID: cc37eb17-f715-080c-7bfc-9eae868d50ba
Content: .nuspec File Reference for NuGet
Content Source: docs/reference/nuspec.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

Issue moved from dotnet/docs#20963

Please respond to @tdykstra.

From @tdykstra on Wednesday, October 7, 2020 6:12:59 PM
cc @rrelyea @nkolev92

Issue moved from dotnet/docs#20963

Please respond to @tdykstra.

From @tdykstra on Wednesday, October 7, 2020 6:12:59 PM
cc @rrelyea @nkolev92

@boringcactus, could you please clarify what exactly fails in your case? I just tried publishing a package with 0BSD license expression and it was published fine: https://www.nuget.org/packages/Andrei.0bsd/0.2.10
I found the we have trouble displaying the license text here: https://licenses.nuget.org/0BSD, we'll look into it.

@boringcactus, could you please clarify what exactly fails in your case? I just tried publishing a package with 0BSD license expression and it was published fine: https://www.nuget.org/packages/Andrei.0bsd/0.2.10
I found the we have trouble displaying the license text here: https://licenses.nuget.org/0BSD, we'll look into it.

honestly, this was so long ago that I forget what the error was.

honestly, this was so long ago that I forget what the error was.

Didn't realize the original issue was from October. I will close the issue for now, if stumble across the repro, feel free to reopen the issue with details.

Didn't realize the original issue was from October. I will close the issue for now, if stumble across the repro, feel free to reopen the issue with details.


[React] WebGridClipboardOperationsColumnInit event args type

This handler is only used in clipboard-operations sample and it is not a xplat one. It is a custom sample made here: https://github.com/IgniteUI/igniteui-react-examples/pull/325. So I am not really sure if we sure merge this or abandon the PR. @dkamburov What do you think?

Yeah, we shouldn't have these at all if the sample is custom and used in that sample only.

Closing this because of https://github.com/IgniteUI/igniteui-xplat-examples/pull/364


Organizer can create a profile which has their contact info

Component for user creation
Ideally a user form

User should be able to select profile type: Organizer, Family, Donator
Form should be re-usable

Form field templates with place holder labels for easy instantiation

@jwu910 @psyduc we need to finish this so we can start a final PR from frontend-content to development


Inquiry about MCP2025 and Possible Alternatives

Good day,
Where did you purchase the MCP2025 that you use for the steering wheel buttons? Are there alternatives such as the MCP2515?
Best regards,
Niko de Vries

Hello,
I bought the MCP 2025 on tme.eu . You can also use the MCP 2003 or other LIN bus transceiver. The MCP 2515 is for the CAN bus.

Did you order them in 3.3V or 5V?


Streams not loading in 2020?

Has anyone successfully gotten this to work in 2020? The app builds and runs fine - it allows me to activate a device and select a game and a stream - but no stream will ever play. Instead I get an error message:
"Server Error: Got unexpected server response. There might not be any streams available for this item. Please try again. Error:"
Upon poking around, I think this may be related to the current code pulling from API v1, and Hehestreams currently using API v2 to deliver stream data? Would love any guidance in getting this up and running.

I checked hehestreams and it looks like its still using v1. I dont see any reference to v2. If you wanted to change the api url:
https://github.com/wftllc/hahastream/blob/master/Haha Stream/HahaService/HahaService.swift#L46

I checked hehestreams and it looks like its still using v1. I dont see any reference to v2. If you wanted to change the api url:
https://github.com/wftllc/hahastream/blob/master/Haha Stream/HahaService/HahaService.swift#L46


Autodiscover fails if POST /autodiscover/autodiscover.xml is used instead of GET /autodiscover/autodiscover.xml

Hi,
I have client application using cloud based Office 365. After upgrade to the ews-java-api 2.0 from previously used Microsoft's EWSJavaAPI 1.2 library, autodiscover stopped working correctly.
After analysing logs I found that in the autodiscover process the previous EWS library was making a GET request to autodiscover service :
2015-11-04 06:39:02.066-08 | FINE | Startup-1_31_6 | org.apache.commons.httpclient.Wire | wire | >> "GET /autodiscover/autodiscover.xml HTTP/1.1[\r][\n]" 2015-11-04 06:39:02.144-08 | FINE | Startup-1_31_6 | org.apache.commons.httpclient.Wire | wire | << "HTTP/1.1 200 OK[\r][\n]" 2015-11-04 06:39:02.144-08 | FINE | Startup-1_31_6 | org.apache.commons.httpclient.Wire | wire | << "X-SOAP-Enabled: True[\r][\n]" 2015-11-04 06:39:02.144-08 | FINE | Startup-1_31_6 | org.apache.commons.httpclient.Wire | wire | << "X-WSSecurity-Enabled: True[\r][\n]"
the ews-java-api 2.0 is using POST instead of GET :
2015-11-04 06:04:24.626-08 | FINE | Startup-1_30_6 | org.apache.http.wire | wire | http-outgoing-3 >> "POST /autodiscover/autodiscover.xml HTTP/1.1[\r][\n]" 2015-11-04 06:04:24.736-08 | FINE | Startup-1_30_6 | org.apache.http.wire | wire | http-outgoing-3 << "HTTP/1.1 200 OK[\r][\n]"
The response for POST to POST /autodiscover/autodiscover.xml is also HTTP 200 OK, however the response is missing the X-SOAP-Enabled: True and X-WSSecurity-Enabled: True headers.
Because of that, the ews-java-api assumes only Legacy endpoint is enabled
2015-11-04 06:04:24.782-08 | FINEST | Startup-1_30_6 | microsoft.exchange.webservices.data.misc.EwsTraceListener | trace | AutodiscoverConfiguration - <Trace Tag="AutodiscoverConfiguration" Tid="19" Time="2015-11-04 14:04:24Z">
 Host returned enabled endpoint flags: [Legacy]
 </Trace>
and reports error:
microsoft.exchange.webservices.data.autodiscover.exception.AutodiscoverLocalException: The Autodiscover service couldn't be located.
When changed the code to use GET instead of POST in AutodiscoverService.tryGetEnabledEndpointsForHost() , things started to work correctly
<Trace Tag="AutodiscoverConfiguration" Tid="19" Time="2015-11-06 10:25:29Z">
 Host returned enabled endpoint flags: [Legacy, Soap, WsSecurity]
 </Trace>
note also that AutodiscoverService.java:1522 has
request.setRequestMethod("GET");
however HttpClientWebRequest.java:102 is always using POST request
httpPost = new HttpPost(getUrl().toString());

Do you have a code sample of what you were using to test the AutoDiscover service?
I tried this myself against Office365 using v.2.0 and it worked, but was very slow. (Took around 30 seconds.)

I dont have isolated test code, only app. code, relevant fragments are:
AutodiscoverService autodiscoverService = new AutodiscoverService(exchangeVersion); //2010
autodiscoverService.setCredentials(exchangeCredentials);
WebProxy proxy = createWebProxy(proxyConfiguration);
            autodiscoverService.setWebProxy(proxy);

response = autodiscoverService.getUserSettings(userSmtpAddress, InterestedUserSettings);
Map<UserSettingName, Object> userSettingsMap = response.getSettings();

		if (autodiscoverService.isExternal()) {
			EWSUrl = (String) userSettingsMap
					.get(UserSettingName.ExternalEwsUrl);
		} else {
			EWSUrl = (String) userSettingsMap
					.get(UserSettingName.InternalEwsUrl);
		}
return EWSUrl;

--------------------

private static final UserSettingName[] InterestedUserSettings = new UserSettingName[] {
			UserSettingName.InternalEwsUrl, UserSettingName.ExternalEwsUrl,
			UserSettingName.UserDN, UserSettingName.UserDisplayName,
			UserSettingName.MailboxDN, UserSettingName.CasVersion,
			UserSettingName.InternalRpcClientServer,
			UserSettingName.InternalMailboxServerDN};

public static WebProxy createWebProxy(ProxyConfiguration proxyConfiguration) {
        if (proxyConfiguration.getCredentials() != null && !StringUtil.isEmptyOrNull(proxyConfiguration.getCredentials().getUsername())) {
            return new WebProxy(proxyConfiguration.getHostname(), proxyConfiguration.getPort(),
                    new WebProxyCredentials(proxyConfiguration.getCredentials().getUsername(),
                            proxyConfiguration.getCredentials().getPassword(), null));
        } else {
            return new WebProxy(proxyConfiguration.getHostname(), proxyConfiguration.getPort());
        }
	}

the "fix" I made was
public class HttpClientWebRequest extends HttpWebRequest {
...
private HttpRequestBase httpPost = null;
...
public void prepareConnection() {
    httpPost = "GET".equals(this.getRequestMethod()) ? new HttpGet(getUrl().toString()) : new HttpPost(getUrl().toString());

Interesting.
I'm using a different method to look up the URL.
Here is my code:
public static URI autoDiscoverURI(String emailAddress, String password)
{
	URI url = null;
	ExchangeService service = new ExchangeService();
	ExchangeCredentials credentials = new WebCredentials(emailAddress, password);
	service.setCredentials(credentials);
	try
	{
		service.autodiscoverUrl(emailAddress, new IAutodiscoverRedirectionUrl()
		{
			@Override
			public boolean autodiscoverRedirectionUrlValidationCallback(String url) throws AutodiscoverLocalException
			{
				return url.startsWith("https:");
			}
		});
		url = service.getUrl();
	}
	catch (Exception e)
	{
		System.out.println(e.toString());
	}
	finally
	{
		service.close();
	}
	return url;
}

We are also running into this issue with Office365 accounts.

I've managed to reproduce this as well and also confirmed that mbialkowskigmc's fix works correct. From the code it definitely appears that the intended request is GET but this is overridden by the default POST request which does not result in the X-SOAP-Enabled being set in the response. This causes AutodiscoverService to fallback to the legacy endpoint which also slows down the autodiscover process noticeably. I'm not familiar enough with the whole autodiscover process (seems like black magic to me and not well documented) but it definitely seems like a GET request was intended, and this is what the C# managed api seems to be doing a well.
I can submit a pull request.

I have same issue.
EWSEditor.exe shows me:

Ordered List of Autodiscover endpoints:
https://exchange2010.intern.mycompany.com/autodiscover/autodiscover.xml

I have same code which avromf had post.
Additionaly i´ve imported cer-certificate into keystore.
Now it looks like that:

Properties systemProps = System.getProperties();
systemProps.put("javax.net.ssl.trustStore",
			  "C:/Program Files/Java/jdk1.8.0/jre/lib/security/cacerts");
systemProps.put("javax.net.ssl.trustStorePassword","changeit");
System.setProperties(systemProps);
service = new ExchangeService(ExchangeVersion.Exchange2010_SP1);
ExchangeCredentials credentials = new WebCredentials("FirstnameSecondname", "myPW","EXCHANGE2010.intern.mycompany.com");
service.setCredentials(credentials);
service.setTraceEnabled(true);
service.setTraceFlags(EnumSet.allOf(TraceFlags.class));
service.setTraceListener(new ITraceListener() {
			public void trace(String traceType, String traceMessage) {
				System.out.println("Type:" + traceType + " Message:" + traceMessage);
			}
		});
<EMAIL_ADDRESS>new                       IAutodiscoverRedirectionUrl() {
				public boolean autodiscoverRedirectionUrlValidationCallback(String url)
						throws AutodiscoverLocalException {
							System.out.println("url: " + url.toLowerCase().startsWith("https://"));
					return url.toLowerCase().startsWith("https://");
				}
			});

But it does not work :(
Trace:

Type:AutodiscoverConfiguration Message:
Determining which endpoints are enabled for host etecture.de

Type:AutodiscoverConfiguration Message:
Host returned enabled endpoint flags: [Legacy]

Type:AutodiscoverConfiguration Message:
No Autodiscover endpoints are available  for host etecture.de

Type:AutodiscoverConfiguration Message:
Determining which endpoints are enabled for host autodiscover.etecture.de

Type:AutodiscoverConfiguration Message:
No Autodiscover endpoints are available for host autodiscover.etecture.de

Type:AutodiscoverConfiguration Message:
Trying to get Autodiscover redirection URL from http://autodiscover.etecture.de/autodiscover/autodiscover.xml.

Type:AutodiscoverConfiguration Message:
No Autodiscover redirection URL was returned.

Type:AutodiscoverConfiguration Message:
Trying to get Autodiscover host from DNS SRV record for etecture.de.

Type:AutodiscoverConfiguration Message:
DnsQuery returned error 'DNS name not found [response code 3]'.

Type:AutodiscoverConfiguration Message:
No appropriate SRV record was found.

Type:AutodiscoverConfiguration Message:
No matching Autodiscover DNS SRV records were found.

Type:AutodiscoverResponse Message:
Autodiscover service call failed with error 'The Autodiscover service couldn't be located.'. Will try legacy service

Type:AutodiscoverConfiguration Message:
Trying to call Autodiscover for<EMAIL_ADDRESS>on https://etecture.de/autodiscover/autodiscover.xml.

Type:AutodiscoverRequestHttpHeaders Message:
POST /autodiscover/autodiscover.xml HTTP/1.1
Keep-Alive : 300
Content-type : text/xml; charset=utf-8
Accept : text/xml
User-Agent : ExchangeServicesClient/<IP_ADDRESS>
Connection : Keep-Alive

Hello,
I am currently running into a problem with autodiscovery:
I/O exception (java.net.SocketException) caught when processing request to {s}->https://autodiscover.omitted.onmicrosoft.com:443: Network is unreachable (connect failed) using the same type of code as sVeloper.
I'm wondering if this fix will address that, and if so, when can it be expected?
Thank you.

Hello,
Could you show piece of your code? And what is Java version you use?

Hello,
Could you show piece of your code? And what is Java version you use?

    if (endpoint.getAutodiscover()) {
        try {
            this.autodiscoverUrl(userOfAccount.getUsername(), new                       IAutodiscoverRedirectionUrl() {
            public boolean autodiscoverRedirectionUrlValidationCallback(String url)
                    throws AutodiscoverLocalException {
                        System.out.println("url: " + url.toLowerCase().startsWith("https://"));
                return true;
            });
        } catch (Exception e) {
            logger.error(e.getMessage(), e);
            throw new IllegalArgumentException(e.getMessage(), e);
        }

"this" extends server
Java - 1.8.0_144

Hello,
Could you show piece of your code? And what is Java version you use?

Hi pkropachev.  I think I fixed my problem.  Thanks!

Hello @dan-oneil !
Sorry, I didn't have time to check it. Could you share your solution?

I realized the credentials were being set after the call was made.  I made the call after setting credentials in the class that extended ExchangeServer and all is well!  Thanks :-)

What's the update on this? I am still getting this issue.


Build status 404 right after build is finished (for a few seconds)

"I have noticed a brief period where Serval returns 404s just after the build has finished uploading, but then it resolves itself for me."

Who is this quote from? What is meant by "uploading"? Does this mean a few minutes after it says it's complete, it returns 404s?

I believe it was from @Nateowami  - can you check with him on the specifics?  I don't know if I got all the details (after uploading files?  After attaching a corpus?  After starting a build?  Which command was returning 404's?

After uploading the corpus and telling the build to start.

@pmachapman may be able to provide exact details.

SF uploads 520 files (2 chapters of the Bible) and then tries to build within a few milliseconds.  I don't know if the file write operations take a few seconds to complete after responding positively.

The only way that Serval would return a 404 on the start build endpoint is if the engine does not exist. I'm guessing that it isn't the start build endpoint that is returning a 404.

@johnml1135 @Nateowami @Enkidu93 I think this occurred from a combination of SF not handling state correctly while uploading to Serval, and the issue with the S3 keys. I am not having it occur while retesting at my end.
I think we can close this, and if I have it happen again (and have a consistent procedure for recreating it) I can reopen the ticket.

Sounds good!


have dartfix only depend on published packages

Dartfix currently depends on a few non-published packages (it looks like specifically just pkg/analysis_server/).
We should refactor it so that it doesn't depend on non-published packages, so that dartfix itself can be published.
It looks like the use of pkg/analysis_server/ is just for the JSON communications; if that's the case, we can move to using a package like packageanalysis_server_lib: (https://pub.dartlang.org/packages/analysis_server_lib).

I had originally expected / intended analysis_server_client (in sdk/pkg) to fulfill that roll. It has the advantage that it can easily be kept consistent with changes to server. In either case, we should consolidate and have a single package to support Dart-based clients that use server's APIs.

Both analysis_server_client and dartfix are now published.


Inconsistent snapping intervals when resizing cubes using the new gizmo

Godot version
4.2.dev5.mono.official
System information
Godot v4.2.dev5.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>99) - Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz (6 Threads)
Issue description
If you hold CTRL while resizing a cube(or any object, for that matter), the interval that it snaps to will change depending on what size the cube was when you started resizing.
Steps to reproduce

Create a CSGBox3D node at (0, 0, 0), with a size of (1, 1, 1).
Select the cube
While holding CTRL, click and drag the top face's "handle" and move it upwards until the box's size is (1, 2.5, 1).  Notice that while you were doing this, the size was snapping to multiples of 0.5 (1.5, 2, 2.5, etc.)
Release the left mouse button
While still holding CTRL, click and drag the same "handle" again, and try to move it upwards until the box's size is (1, 4, 1).  Notice that this is now impossible; it will end up snapping to the values 2.75, 3.25, 3.75, 4.25.
Release the left mouse button
While still holding CTRL, click and drag the same "handle" again, and try to move it upwards until the box's size is (1, 5, 1).  Now it will snap to even weirder values.

Expected result: the size stays snapped to a grid of 0.5, hitting the values 1.5, 2, 2.5, 3, 3.5, 4, 4.5, and 5.  This should be consistent even when you let go of the left mouse button and then click again.
Actual result: Every time you release the left mouse button and re-click, the size will become progressively more misaligned from the grid.
Minimal reproduction project
N/A

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/81916.


Time out of sync for players in multiplayer

Contains
Fix proposal for Issue #3670 .
Most of the code comes from @godfather2 's pull request #3890 , closed on 29 August 2020, which attempted to fix time synchronisation when connecting with another client, and when changing time in Debug mode.
Using setWorldTime command on either client should now update time on both clients.
How to test

Start a local multiplayer game
Start up another client
Wait 5 minutes or so before connecting.
-> The second client should now have its time (almost) synchronized with the first one (see next section).
Change time on either client, using arrow keys in Debug mode.
-> Both clients should now have their times updated.
Use setWorldTime 0.9 on either client.
-> Time should now be set to 0.9 on both clients.

Outstanding before merging
As opposed to the description of the issue, time is already (mostly) synchronized when connecting to a multiplayer game.
As explained in PR #3890 , the host's time is sent at the beginning of the connection process, which causes a delay of about 0.03 day (or more, depending on the capability of the computer used) when the new client eventually connects.

Just tested this out and it works like a charm 🎉 Code-review will follow in a bit.
@PaulBeslin As a follow-up I would propose that both setWorldTime and changing the world time with the arrow-keys should be only possible with cheat commandos as in a real game the client should not be able (except with cheat permissions) to change the world time. Only the host should be able to do so.

Thank you for taking time to check this!
The "anti-cheat" idea you proposed seems indeed necesary, and I'll try to work on it.
Should it be part of this PR, or should a new issue be made?
Thanks again.

The "anti-cheat" idea you proposed seems indeed necesary, and I'll try to work on it.
Should it be part of this PR, or should a new issue be made?
Thanks again.

No, I would say this is not necessary for this first iteration.
I'll merge this now and if you'd like you can work on the "anti-cheat" aspect as a follow-up :+1:


 Where to commit them?

 These files should be in the path specified in your Sphinx conf locale_dirs.

Closing as duplicate of #1746


Adds The TSqlDecimalValue Data Type

I have added tests for the new data type. Which
also include some validation tests to make sure
invalid objects can be created.

I'm tweaking some of the value objects because it's not entirely what I had in mind. Will surface on master.


Credentials required to be in two different places?

I'm trying to set up IaC via Pulumi.
One of the challenges is to set up appropriate permissions.
So I needed to get an Azure AD group object id:
var group = Output.Create(
    GetGroup.InvokeAsync(
        new GetGroupArgs
        {
            Name = "Administrators"
        }));

However, I got the following error:
PS C:\dev\___> pulumi preview
Previewing update (dev):
     Type                 Name          Plan     Info
     pulumi:pulumi:Stack  Frontend-dev           1 error

Diagnostics:
  pulumi:pulumi:Stack (Frontend-dev):
    error: Running program 'C:\dev\___\bin\Debug\netcoreapp3.1\Frontend.dll' failed with an unhandled exception:
    Grpc.Core.RpcException: Status(StatusCode=Unknown, Detail="invocation of azuread:index/getGroup:getGroup returned an error: Error building AzureAD Client: Authenticating using the Azure CLI is only supported as a User (not a Service Principal).

    To authenticate to Azure using a Service Principal, you can use the separate 'Authenticate using a Service Principal'
    auth method - instructions for which can be found here: (THERE IS NO LINK HERE)

    Alternatively you can authenticate using the Azure CLI by using a User Account.")
       at Pulumi.GrpcMonitor.InvokeAsync(InvokeRequest request)
       at Pulumi.Deployment.InvokeAsync[T](String token, InvokeArgs args, InvokeOptions options, Boolean convertResult)
       at Pulumi.Output`1.ApplyHelperAsync[U](Task`1 dataTask, Func`2 func)
       at Pulumi.Output`1.Pulumi.IOutput.GetDataAsync()
       at Pulumi.Serialization.Serializer.SerializeAsync(String ctx, Object prop)
       at Pulumi.Deployment.SerializeFilteredPropertiesAsync(String label, IDictionary`2 args, Predicate`1 acceptKey)
       at Pulumi.Deployment.SerializeAllPropertiesAsync(String label, IDictionary`2 args)
       at Pulumi.Deployment.RegisterResourceOutputsAsync(Resource resource, Output`1 outputs)
       at Pulumi.Deployment.Runner.WhileRunningAsync()

There's no link where it was supposed to be.
Somewhere in the documentation, we're asked to put the credentials in the Pulumi.<stack>.yml via pulumi config set commands which led me to believe that that should be enough.
However, it seems to me now that we must have the credentials also set up as environment variables.
In powershell, you may type the following:
$env:ARM_CLIENT_ID="0000000000000000000000000"
$env:ARM_CLIENT_SECRET="0000000000000000000000000"
$env:ARM_TENANT_ID="0000000000000000000000000"
$env:ARM_SUBSCRIPTION_ID="0000000000000000000000000"

After doing that, my Pulumi stack was able to retrieve the Azure AD group object id successfully.
Could this be sorted please? Is it expected that we duplicate the credentials or is there a better way?
Original question:
https://stackoverflow.com/questions/62259552/how-to-get-group-in-azure-ad-in-pulumi/62263761#62263761

You only need either the environment variables or variables setup in your config. Please see https://www.pulumi.com/docs/intro/cloud-providers/azure/setup/ which describes the configuration.

Hi @leezen,
Thank you for your response.
I have the credentials in my config Pulumi.dev.yml which were added as follows:
pulumi config set azure:clientId "00000000000000000000000"
pulumi config set azure:clientSecret "00000000000000000000000" --secret
pulumi config set azure:tenantId "00000000000000000000000"
pulumi config set azure:subscriptionId "00000000000000000000000"

The thing that's bugging me is that the service principal credentials are initially recognized by pulumi as I can see in the debug entries. Running:
$ pulumi preview --debug

Presents us with the following:
Line 55.
debug: 2020/06/09 10:31:16 Testing if Service Principal / Client Certificate is applicable for Authentication..
debug: 2020/06/09 10:31:16 Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
debug: 2020/06/09 10:31:16 Testing if Service Principal / Client Secret is applicable for Authentication..
debug: 2020/06/09 10:31:16 Using Service Principal / Client Secret for Authentication
debug: 2020/06/09 10:31:16 Getting OAuth config for endpoint https://login.microsoftonline.com/ with  tenant....

^^^ This shows that the service principal and the client secret configs were recognized.
Line 94.
debug: 2020/06/09 10:31:17 Testing if Service Principal / Client Certificate is applicable for Authentication..
debug: 2020/06/09 10:31:17 Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
debug: 2020/06/09 10:31:17 Testing if Service Principal / Client Secret is applicable for Authentication..
debug: 2020/06/09 10:31:17 Testing if Managed Service Identity is applicable for Authentication..
debug: 2020/06/09 10:31:17 Testing if Obtaining a token from the Azure CLI is applicable for Authentication..
debug: 2020/06/09 10:31:17 Using Obtaining a token from the Azure CLI for Authentication

^^^ And then Pulumi forgets about the service principal...
And, at the end, it's as if the service principal crendentials were never presented.
Grpc.Core.RpcException: Status(StatusCode=Unknown, Detail="invocation of azuread:index/getGroup:getGroup returned an error: Error building AzureAD Client: Authenticating using the Azure CLI is only supported as a User (not a Service Principal).

    To authenticate to Azure using a Service Principal, you can use the separate 'Authenticate using a Service Principal'
    auth method - instructions for which can be found here: **** There's no link presented here.

    Alternatively you can authenticate using the Azure CLI by using a User Account.")

However, adding the service principal credentials as environment variables worked.
This has forced me to place the credentials in two places. The Pulumi.dev.yml and the environment variables. Not ideal.
After several hours on this, it seems that possibly the feedback to help the devops may be improved in some way to help us get to the source of the problem faster.
Can you help with this please?
Thanks


Akka style 'tell' for fire-and-forget async messaging

Awesome library,  I look forward to the re-write in version 0.10.  In other actor frameworks it's a frequently used pattern to use an async fire-and-forget pattern.  Akka uses 'ask' and 'tell' for a send type method that returns a response and the corresponding async pattern.
This is clearly a very fast framework that doesn't actually use message passing locally but in a remote / foreign setup serialization overheads will slow things down and it's often the case that an actor does not need a response.  I'd love to see similar functionality added here ?

Hi @stevehayles, thanks so much for the feedback!
Fluxion used to implement fire-and-forget messages quite a while back (I think the commands were tell and request). The reason why those no longer exist is because I have been trying to move Fluxion away from dependency on an async executor. Currently in version 0.9, quite a bit of boilerplate is required just to enable spawning supervisor tasks in an exector agnostic manner. Version 0.10 will completely remove all of that boilerplate by using a new system that "simulates" message passing. The only downside of this is that fire-and-forget messages can no longer be implemented directly in Fluxion without re-introducing all of that boilerplate.
While I don't think I would be able to implement fire-and-forget messages directly in Fluxion (without quite a few compromises), I do have an alternative. You can use the async executor of your choice to spawn a new task for a given message:
tokio::spawn(async {
    actor_ref.send(message).await
});

This is almost exactly what Fluxion version 0.9 does under the hood, except in version 0.9 the task is spawned on the other side of a channel as seen here. Additionally, in Fluxion 0.10 this may not be required if the delegate spawns a new task or uses a channel. I would need to double check, but I believe that it may be possible for a delegate to skip waiting for a response if the result type is ().
So while it may not be possible to add the tell functionality without comprimising Fluxion's size and executor agnosticism, it may be possible to significantly reduce the impact of messages that do not require a response. When I am working on a basic networked foreign message system, I will make sure to keep all of these in mind.
Additionally, On my backlog, I plan on creating an event system that can integrate with Fluxion to provide these fire-and-forget messages separately, including targeting messages at individual or even sub-groups of actors. I am also planning on creating a crate that allows actors to be defined in an external scripting language, which will hide the entire tokio::spawn behind a nicer API.
Thanks again for the feedback, and have a great day!


resource is hidden by a resource

i have two user-system and each one have its own table in database. so i'm trying to override the template of each user-system so that the "action" in  change with every user-system
so i'm trying the b) method "Create A Child Bundle And Override Template"  but i got this error :
"/var/www/html/test/app/Resources/RollerworksMultiUserBundle/views/UserBundle/Registration/regis
ter_content.html.twig" resource is hidden by a resource from the "GAGVendeurBundle" derived bund
le. Create a "/var/www/html/test/app/Resources/GAGVendeurBundle/views/UserBundle/Registration/register_content.html.twig" file to override the bundle resource
and when i create the "/var/www/html/test/app/Resources/GAGVendeurBundle/views/UserBundle/Registration/register_content.html.twig" file, it overrdide all the user-systeme templates
help me plz

First of sorry for the late reply,
Its hard to determine if this is a bug in RollerworksMultiUserBundle bundle or a wrong usage.
Can you provide more details and if possible a Forked Symfony-standard edition to demonstrate the problem.
Are you setting the RollerworksMultiUserBundle as a parent of your own bundle?
Because that is what the error seems to be saying.


bad repo after last merge

Just cloned the repo after these changes
Context: monterey, running λ npm install && jspm install command, I get these questions
Would you like jspm to prefix the jspm package.json properties under jspm? [yes]:
Enter server baseURL (public folder path) [./]:
Enter jspm packages folder [.\jspm_packages]:
Enter config file path [.\config.js]:
Configuration file config.js doesn't exist, create it? [yes]:
Enter client baseURL (public folder URL) [/]:
Do you wish to use a transpiler? [yes]:
Which ES6 transpiler would you like to use, Babel, TypeScript or Traceur? [babel]:

I understand all questions,  it is safe to take all defaults (true?) but I did not have to do that until this new version of monterey that is approx 1 hour old. What's different?
Added a few minutes later:
monterey fails to start with:
Failed to load resource: net::ERR_FILE_NOT_FOUND
<EMAIL_ADDRESS>Failed to load resource: net::ERR_FILE_NOT_FOUND
file:///C:/work/monterey-framework/monterey/app/jspm_packages/system.src.js Failed to load resource: net::ERR_FILE_NOT_FOUND
config.js:1 Uncaught ReferenceError: System is not defined
index.html:22 Uncaught ReferenceError: System is not defined
C:\work\monterey-framework\monterey\node_modules\electron-connect\lib\client.js:27 [2016-07-05T21:39:01.854Z] [electron-connect] [client: 1] connected server

So, jspm install failed because something (jspm section of package.json ?) is missing. I will leave this for @JeroenVinke's update (hopefully tomorrow)

a side effect of https://github.com/monterey-framework/monterey/pull/46 is that config.js lives in the app directory. I have updated the readme:
1. npm install
2. cd app
3. npm install
4. jspm install
5. gulp watch (manually reloading electron with ctrl+R is required) or gulp watch-r to automatically reload electron

Thank you for the explanation


Change host engine opt type to list

1347

so how are we handling the transition from old string based host settings to the new array?

It's handled automatically in os-config.yml when that's updated. For user settings it'll have to be manually upgraded.

its for the user that i'm worried about - is there any way we can start having versioned yml files?

Versioning configuration was something I was planning on doing post-GA. I think it'd be a pretty large effort for now.


Enhancement: Unified refresh

Goal-State/What/Result
Refreshes for two or more data connectors with the same acceleration parameters should be unified (only one refresh happens).
Why/Purpose
To enable the in-memory primary, file backup acceleration fallback pattern.
By When
A target date for done-done completion of the entire enhancement.
Issue/Spec written and reviewed:
Done-Done:
Done-Done

[ ] First Principles Driven
[ ] The Algorithm
[ ] PM/Design Review
[ ] DX/UX Review
[ ] Threat Model / Security Review
[ ] Tests
[ ] Performance / Benchmarks
[ ] Documentation
[ ] Samples/Quickstarts/Tutorials
[ ] Release Notes

The Algorithm

[ ] Every requirement questioned?
[ ] Delete (Scope) any part you can.
[ ] Simplify.
[ ] Break down into smaller iterations/milestones.
[ ] Opportunities for automation.

Spec
E.g.
datasets:
  - from: s3://blah/blah
     name: primary
     refresh_interval: 5m
  - from: s3://blah/blah
     name: secondary
     refresh_interval: 5m

Should only cause one refresh every 5 mins, but both datasets to have the refreshed data.
How/Plan
A clear and concise plan of how this should be implemented.
Implementation (Sub Issues)

Sub Issue 1
Sub Issue 2

QA

Sub Issue 1
Sub Issue 2

Target: Done by Tue


Nuka carousel

Adding Nuka Carousel in to the bookmark component

Also please look at what can be removed from here: https://github.com/boundlessgeo/sdk/blob/master/scripts/install.js#L14

Coverage increased (+0.4%) to 69.563% when pulling ea685ec2fab63ec833281c6df27d835d659f8a28 on nuka_carousel into 4612f5de380283e2923618e20255581107c57edd on master.

Coverage increased (+0.5%) to 69.621% when pulling 86b6f25bd42d56223cc72b3dcdbbc6db3c03f447 on nuka_carousel into 4612f5de380283e2923618e20255581107c57edd on master.


GitHub Release listings contain invalid links to referenced commits

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Area
- [x] devkit
- [ ] schematics

Versions
angular-cli version 6.1.0-beta2
Repro steps

Visit Github page for @angular/cli.
Click on "releases" link.
Find the 6.1.0-beta2 release.
Click on an image in the Commit column in a table, which has a hash and commit type.

The log given by the failure
GitHub shows a 404 error. The URL linked to is https://github.com/angular/devkit/commit/_HASH_
Desired functionality
GitHub shows the commit page for the referenced commit. The links should be to https://github.com/angular/angular-cli/commit/_HASH_
Mention any other details that might be useful
These image/commit links appear to be new, previous usages of just text-based links relied on GitHub auto-linkifying them to this repository.

Fixed now, thanks!


CheckBox borderWidth

Hi,
How to change checkbox border width?

Any update?

you can not change border width directly as checkbox created using images if you really want you can images in node_modules/react-native-check-box/img/


Is it possible to increase screen resolution?

I'm only using 800x600 so far

Current the size is responsive by the first time you open VNC web page.
You may restart the container, then open the page that resolution will fit the browser window size.

I don't see any re-sizing or re-scaling. The window is not "fitting" the browser size.
Where can I hard code the vnc screen resolution?

Resize only trigger when first starting the container

Close because no response

Hi, I'm using VNC client, how to change resolution in this case?

@axot
Hi, that's impossible to support VNC client because RFB protocol do not define changing resolution by VNC client. What I can do in this project is to pass default resolution via Docker environment. Is it usable for you?

How to change the size based on browser size?

Resize only trigger when first browsing the web page


What is the proper syntax for VCPKG_ENV_PASSTHROUGH?

What is the proper syntax for VCPKG_ENV_PASSTHROUGH in vcpkg triplets on Windows x64?

I found information here "https://devblogs.microsoft.com/cppblog/vcpkg-2019-07-update/" but it does not give the syntax.
I also found a discussions here "https://github.com/Microsoft/vcpkg/issues/3712", "https://github.com/microsoft/vcpkg/issues/7127" and "https://github.com/microsoft/vcpkg/issues/7328"
I tried:

set(VCPKG_ENV_PASSTHROUGH INCLUDE;LIB;LIBPATH) in a custom triplet
set VCPKG_ENV_PASSTHROUGH=INCLUDE;LIB;LIBPATH in an environment variable

Neither method allowed the variables to pass into a clean .\vcpkg.exe env command.
I did find a discussion on VCPKG_KEEP_ENV_VARS here  "https://github.com/microsoft/vcpkg/blob/master/docs/users/config-environment.md", "https://vcpkg.readthedocs.io/en/latest/users/config-environment/#vcpkg_keep_env_vars" and "https://github.com/microsoft/vcpkg/issues/8950"

set(VCPKG_KEEP_ENV_VARS INCLUDE;LIB;LIBPATH) in a custom triplet
set VCPKG_KEEP_ENV_VARS=INCLUDE;LIB;LIBPATH in an environment variable

both worked and passed the variables into a clean .\vcpkg.exe env

Since it seems both serve the same purpose, which should one use and if VCPKG_ENV_PASSTHROUGH is the correct method, what is the correct syntax?

I close, I no longer use product.

See https://github.com/microsoft/vcpkg/blob/master/triplets/community/arm-mingw-dynamic.cmake


Can we use this for Tron ( Trx ) chain ? Suggest one.

Can we use this for Tron ( Trx ) chain ? Suggest one.

There is no such feature yet. If it's EVM compatible, you can add it as custom network

Hey, @akhilccrb  have your initial issue been resolved? Please let us know if it can be closed.
If you still experience any problems, feel free to add more details in this issue or open another one.


[Feature][ui] Start from force success

What is the purpose of the pull request

Add a button 'Start From Force Success' at instance list page, available for instance with state FORCED_SUCCESS
Add a new execution type START_FROM_FORCED_SUCCESS for instance, for zh-cn, it calls 从强制成功处继续
Add related locale words.

Related issues

[Feature][*] Force Task Success #3575

Brief change log

dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/instance/pages/list/_source/list.vue Add a button and handlers for force success operation;
dolphinscheduler-ui/src/js/module/i18n/locale/en_US.js and dolphinscheduler-ui/src/js/module/i18n/locale/zh_CN.js Add related locale words.

Codecov Report

Merging #4160 (cb647e9) into dev (61fcad7) will increase coverage by 1.31%.
The diff coverage is 37.05%.

@@             Coverage Diff              @@
##                dev    #4160      +/-   ##
============================================
+ Coverage     40.04%   41.35%   +1.31%
- Complexity     2926     3113     +187
============================================
  Files           459      467       +8
  Lines         21793    22168     +375
  Branches       2652     2720      +68
============================================
+ Hits           8727     9168     +441
+ Misses        12238    12120     -118
- Partials        828      880      +52

Impacted Files
Coverage Δ
Complexity Δ

...inscheduler/alert/utils/EnterpriseWeChatUtils.java
64.58% <0.00%> (+1.31%)
14.00 <0.00> (ø)

...cheduler/api/controller/AccessTokenController.java
8.33% <0.00%> (ø)
2.00 <0.00> (ø)

...hinscheduler/api/controller/ProjectController.java
5.55% <0.00%> (ø)
2.00 <0.00> (ø)

...nscheduler/api/controller/ResourcesController.java
1.98% <0.00%> (-0.11%)
2.00 <0.00> (ø)

...scheduler/api/controller/TaskRecordController.java
25.00% <ø> (ø)
2.00 <0.00> (ø)

...lphinscheduler/api/controller/UsersController.java
2.77% <0.00%> (ø)
2.00 <0.00> (ø)

...nscheduler/api/service/ProcessInstanceService.java
73.83% <0.00%> (ø)
36.00 <0.00> (ø)

...nscheduler/api/service/impl/LoggerServiceImpl.java
77.77% <ø> (ø)
11.00 <0.00> (ø)

...rg/apache/dolphinscheduler/common/enums/Event.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../dolphinscheduler/common/enums/SqoopQueryType.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

... and 179 more

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 61fcad7...cc2068b. Read the comment docs.

could you solve conflict files locally？


Swipe back gesture doesn't work in editor

Every Door 3.0 on iOS 16.0 doesn't allow to go back from the editor by swiping from the left edge of the screen. This is very confusing, because it's a default go back gesture expected to work in any context. At the same time, it does work correctly in settings, it's broken only in the editor.

This was an UX issue: according to https://github.com/flutter/flutter/issues/14203, "back" gesture cannot be conditional on iOS. And the editor page checks for unsaved changes. Hence, as recommended, I transformed the page into a full screen dialog, which doesn't imply it can be swiped.
There is one UI hole left (creating a new amenity), but at the moment I've no idea how to patch that. Still, looks more consistent now in 3.1. Thanks for reporting, I would never notice that myself :)


Create server-authoritative 3D physics example

I've created an example of clients controlling server-authoritative, 3D physics objects. The spaceships example also showcases clients controlling server-authoritative physics but it has a lot of extra features such as pre-spawning and wasm that make it more difficult to follow. This is intended to be a more streamlined example. Plus, it demonstrates 3D physics which is neat.
avian_3d_physics_demo.webm

Thanks, that's really awesome!

What settings did you use for the video? I tried input_delay_ticks = 4 and the corrections were very noticeable, it was much better with input_delay_ticks = 6
I notice that there are constant rollbacks happening. This can be fixed in a future PR, but do you have any idea why?

One window was cargo run --release -- client-and-server and the other was cargo run --release -- client -c 1. I did not change the settings from what's in the commit.
I wish I knew why there are constant rollbacks 😅 I tried incorporating https://github.com/Jondolf/avian/commit/63580d38a50322a1f80ba55925094db47749d68e but that didn't stop the rollbacks. I think the movement logic is responsible. When the character looks like it isn't moving it is still getting a non-zero, very small force applied to it in order to keep it in place. That may result in a very small linear velocity that isn't in sync with the server. I was hoping that because the physics engine is deterministic that the resulting calculated movement force would also be deterministic but maybe not.

If I could get a snapshot of the world when the client resets the world to a tick when performing a rollback and also get a snapshot of the world on the server at that tick then I could then compare the two snapshots and see what's different. Is that possible?

Even better would be a snapshot of the world on the starting rollback tick and the tick after that plus the same ticks on the server. Then I could verify how the first tick changed.

Usually I just print out the value of Position at every tick on the client and server and look for discrepancies.
In general avian is deterministic so it might be something else

That is much easier, thanks!


Add example for object filter on object.

There was already an example for using an object filter on an array. Clarify that object filters can be used on objects as well and that the result is an array, not an object.
Why:
Closes #11762.
What's being changed:
Content only; refer to rich diff.
Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@Kurt-von-Laven
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Great addition @Kurt-von-Laven - thanks much.  I made one minor suggestion but this is 👍

Thank you for the review, @ethomson!

I believe the build failure is unrelated to this pull request.

I rebased on the latest version of main, and this time all of the checks passed. I suspect the previous failure may have been caused by a flaky test.

@Kurt-von-Laven We are looking into the fix. I will get this merged once the tests are passing. Thanks so much for your patience 💖

Sounds good. The flaky test theory is supported by the fact that the tests sometimes fail and sometimes succeed when all I am doing is rebasing on the latest changes.

@ethomson, I rebased on the latest main, and now all tests are passing.

Please process


feat: 🎸added disable option in menuItem for disabling dropdown option

feat: 🎸 In this feature we can disable dropdown option
In this feature we can disable SQFormdropdown option specifically by
providing isDisable value
✅ Closes: 118
screenshot:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: