Add performance optimization docs

Splunk has a decent doc on how to evaluate the performance overhead of the OTel Java Agent:
https://docs.splunk.com/Observability/gdi/get-data-in/application/java/performance.html
I think it would be great if OTel had similar content on our doc site. Jason suggested Splunk could possibly contribute their content.
cc @theletterf @breedx-splk

+1 for these ones as well :-)

Yep, contributions welcome for this. I'd say that a new doc called "Performance" under the "Automatic" folder would work.

@theletterf Thanks for contributing #3459.  Gentle reminder/nudge to consider contributing this doc as well.

Happy to! @breedx-splk WDYT?

Happy to! @breedx-splk WDYT?

I'm all about it. Thanks again!!

I think we can continue with this one. Let me reach out to Legal.

Any update @theletterf ?

Thanks for the reminder! Waiting for legal response this week.

Almost there. Final phase of legal review...

Got legal approval! Will create a PR soon.

Nice job!


Unable to install extensions on remote host

Type: Bug
Steps to Reproduce:

1 ssh to some remote server (my server is on Tencent cloud)
2 go to extensions and try to install any extension. For example, I searched for cmake and clicked "Install", and then the button changed to "Installing", and then the "Installing" stayed forever. I also tried to install React Native Tools, the button was "Install in SSH: x.x.x.x", but it's not clickable.
I also tried to reinstall the OS on my cloud server and still could not install extensions.

Log (Shared):
[2022-08-09 16:07:19.648] [sharedprocess] [info] Starting worker process with pid 17832 (type: fileWatcher, window: 1).
[2022-08-09 16:07:20.117] [sharedprocess] [info] WSL is not installed, so could not detect WSL profiles
[2022-08-09 16:07:27.699] [sharedprocess] [info] Starting worker process with pid 3336 (type: fileWatcher, window: 3).

Log (Window):
[2022-08-09 16:07:27.559] [renderer3] [info] [attempt 1] Invoking resolveAuthority(ssh-remote)
[2022-08-09 16:07:29.227] [renderer3] [error] [Extension Host] (node:9944) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use `Code --trace-deprecation ...` to show where the warning was created)
[2022-08-09 16:07:29.235] [renderer3] [info] [attempt 1] resolveAuthority(ssh-remote) returned '<IP_ADDRESS>:61309' after 1675 ms
[2022-08-09 16:07:30.021] [renderer3] [error] [xabikos.JavaScriptSnippets]: Unknown language in `contributes.JavaScriptSnippets.language`. Provided value: vue

Log (Remote Server):
[2022-08-09 15:44:38.639] [remoteagent] [info] Extension host agent started.
[2022-08-09 15:44:38.853] [remoteagent] [info] [<IP_ADDRESS>][149d3ab5][ManagementConnection] New connection established.
[2022-08-09 15:44:38.925] [remoteagent] [info] [<IP_ADDRESS>][35e7c2a0][ExtensionHostConnection] New connection established.
[2022-08-09 15:44:38.932] [remoteagent] [info] [<IP_ADDRESS>][35e7c2a0][ExtensionHostConnection] <3747> Launched Extension Host Process.
[2022-08-09 15:44:59.764] [remoteagent] [info] Getting Manifest... golang.go
[2022-08-09 15:45:00.384] [remoteagent] [info] Installing extension: golang.go
[2022-08-09 15:45:00.478] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:45:00.676] [remoteagent] [info] Installing extension: msjsdiag.vscode-react-native
[2022-08-09 15:48:35.142] [remoteagent] [info] [<IP_ADDRESS>][35e7c2a0][ExtensionHostConnection] <3747> Extension Host Process exited with code: 0, signal: null.
[2022-08-09 15:48:35.142] [remoteagent] [info] Last EH closed, waiting before shutting down
[2022-08-09 15:49:29.683] [remoteagent] [info] Got delay-shutdown request while in shutdown timeout, delaying
[2022-08-09 15:49:29.683] [remoteagent] [info] Cancelling previous shutdown timeout
[2022-08-09 15:49:29.900] [remoteagent] [info] [<IP_ADDRESS>][f2087f84][ManagementConnection] New connection established.
[2022-08-09 15:49:30.084] [remoteagent] [info] [<IP_ADDRESS>][ae6c7bfe][ExtensionHostConnection] New connection established.
[2022-08-09 15:49:30.105] [remoteagent] [info] [<IP_ADDRESS>][ae6c7bfe][ExtensionHostConnection] <5112> Launched Extension Host Process.
[2022-08-09 15:49:30.781] [remoteagent] [info] [<IP_ADDRESS>][149d3ab5][ManagementConnection] The client has disconnected, will wait for reconnection 3h before disposing...
[2022-08-09 15:49:30.783] [remoteagent] [error] Error: Unexpected SIGPIPE
    at process.<anonymous> (/home/lighthouse/.vscode-server/bin/da76f93349a72022ca4670c1b84860304616aaa2/out/vs/server/node/server.main.js:145:7922)
    at process.emit (node:events:390:28)
[2022-08-09 15:50:33.624] [remoteagent] [info] Downloaded extension: golang.go /home/lighthouse/.vscode-server/data/CachedExtensionVSIXs/golang.go-0.35.1
[2022-08-09 15:50:33.734] [remoteagent] [info] Extracted extension to /home/lighthouse/.vscode-server/extensions/.afb79b0a-bf79-4fdf-89e5-553550b13caf: golang.go
[2022-08-09 15:50:33.745] [remoteagent] [info] Renamed to /home/lighthouse/.vscode-server/extensions/golang.go-0.35.1
[2022-08-09 15:50:33.748] [remoteagent] [info] Extracting completed. golang.go
[2022-08-09 15:50:33.750] [remoteagent] [info] Extension installed successfully: golang.go
[2022-08-09 15:54:29.683] [remoteagent] [info] New EH opened, aborting shutdown
[2022-08-09 15:55:06.968] [remoteagent] [info] Getting Manifest... golang.go
[2022-08-09 15:55:07.180] [remoteagent] [info] Installing extension: golang.go
[2022-08-09 15:55:07.409] [remoteagent] [info] Extension installed successfully: golang.go
[2022-08-09 15:55:15.004] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:55:15.031] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 15:56:10.750] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:56:10.927] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 15:58:11.897] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:58:12.057] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 15:59:00.709] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:59:00.841] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 15:59:38.247] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:59:38.425] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 15:59:39.438] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 15:59:39.493] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.548] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:00:41.609] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.612] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.615] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.617] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.618] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:00:41.620] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:03:16.172] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:03:16.286] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:03:16.307] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:03:16.336] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:03:16.432] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:03:16.459] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:06:57.430] [remoteagent] [info] [<IP_ADDRESS>][ae6c7bfe][ExtensionHostConnection] <5112> Extension Host Process exited with code: 0, signal: null.
[2022-08-09 16:06:57.431] [remoteagent] [info] Last EH closed, waiting before shutting down
[2022-08-09 16:07:29.094] [remoteagent] [info] Got delay-shutdown request while in shutdown timeout, delaying
[2022-08-09 16:07:29.094] [remoteagent] [info] Cancelling previous shutdown timeout
[2022-08-09 16:07:29.307] [remoteagent] [info] [<IP_ADDRESS>][149d3ab5][ManagementConnection] Another client has connected, will shorten the wait for reconnection 5m before disposing...
[2022-08-09 16:07:29.308] [remoteagent] [info] [<IP_ADDRESS>][75dd1e72][ManagementConnection] New connection established.
[2022-08-09 16:07:29.395] [remoteagent] [info] [<IP_ADDRESS>][95c44b0e][ExtensionHostConnection] New connection established.
[2022-08-09 16:07:29.403] [remoteagent] [info] [<IP_ADDRESS>][95c44b0e][ExtensionHostConnection] <9077> Launched Extension Host Process.
[2022-08-09 16:07:29.806] [remoteagent] [info] [<IP_ADDRESS>][f2087f84][ManagementConnection] The client has disconnected, will wait for reconnection 3h before disposing...
[2022-08-09 16:07:29.807] [remoteagent] [error] Error: Unexpected SIGPIPE
    at process.<anonymous> (/home/lighthouse/.vscode-server/bin/da76f93349a72022ca4670c1b84860304616aaa2/out/vs/server/node/server.main.js:145:7922)
    at process.emit (node:events:390:28)
[2022-08-09 16:08:30.813] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:08:30.840] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:08:45.550] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:08:45.634] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:08:46.076] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:08:46.095] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:08:46.221] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:08:46.243] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:08:49.891] [remoteagent] [info] Getting Manifest... msjsdiag.vscode-react-native
[2022-08-09 16:08:49.937] [remoteagent] [info] Extensions is already requested to install msjsdiag.vscode-react-native
[2022-08-09 16:12:18.428] [remoteagent] [info] Getting Manifest... ms-vscode.cmake-tools
[2022-08-09 16:12:29.094] [remoteagent] [info] New EH opened, aborting shutdown
[2022-08-09 16:12:29.507] [remoteagent] [info] [<IP_ADDRESS>][149d3ab5][ManagementConnection] The reconnection short grace time of 5m has expired, so the connection will be disposed.
[2022-08-09 16:13:16.061] [remoteagent] [info] Installing extension: ms-vscode.cmake-tools
[2022-08-09 16:13:24.397] [remoteagent] [info] Getting Manifest... twxs.cmake
[2022-08-09 16:13:26.964] [remoteagent] [info] Installing extension: twxs.cmake

VS Code version: Code 1.70.0 (da76f93349a72022ca4670c1b84860304616aaa2, 2022-08-04T04:38:16.462Z)
OS version: Windows_NT x64 10.0.19043
Modes:
Remote OS version: Linux x64 4.18.0-348.7.1.el8_5.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-9350KF CPU @ 4.00GHz (4 x 4008)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (8.45GB free)

Process Argv
--crash-reporter-id 7656db13-88d1-4dc0-ab85-548188d9ff0d

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: <IP_ADDRESS>

OS
Linux x64 4.18.0-348.7.1.el8_5.x86_64

CPUs
Intel(R) Xeon(R) Gold 6133 CPU @ 2.50GHz (2 x 2494)

Memory (System)
1.68GB (0.49GB free)

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

JavaScriptSnippets
xab
1.8.0

go
gol
0.35.1

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30542924
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
vscaac:30438847
pylanb8912cf:30529770
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253

I set "remote.SSH.localServerDownload" to "always" and "remote.downloadExtensionsLocally" to true, and successfully installed extensions on remote server.
But it is still a problem. I didn't find error logs like "dial https://marketplace.visualstudio.com/xxx time out", it's difficult to troubleshoot.

I suspect there are some network issues on your remote machine and you can figure this by enabling log level to trace using command F1 > Set Log Level and retry the action.
Setting remote.downloadExtensionsLocally to true allows VS Code to download the extension locally and install it on the remote.


imgui-winit-support dependency problems

I am building an application that uses imgui and winit so I used the imgui-winit-support crate, but the compiler keeps throwing this error:
platform.attach_window(imgui.io_mut(), surface.window(), HiDpiMode::Rounded);
                                       ^^^^^^^^^^^^^^^^ expected struct `winit::window::Window`, found a different struct `winit::window::Window`

So I checked the dependencies and everything is up to date:
[dependencies]
winit = "0.26.1"
imgui-winit-support = "0.8.2"
imgui = "0.8.2"

So I checked the Cargo.lock file and there are two different versions of winit:
[[package]]
name = "winit"
version = "0.25.0"
....

[[package]]
name = "winit"
version = "0.26.1"

I don't know what this is caused by or how to fix it so any help is greatly appreciated.

Imgui 0.8 uses winit 0.25 by default, thus the two versions being resolved.
Winit 0.26 is available by using features=["winit-26"] (on phone so can't
check, but you might also have to disable default features)
The next release in current 'main' branch will use 0.26 by default
On Tue, 8 Mar 2022, 08:53 slackers, @.***> wrote:

I am building an application that uses imgui and winit so I used the
imgui-winit-support crate, but the compiler keeps throwing this error:
platform.attach_window(imgui.io_mut(), surface.window(), HiDpiMode::Rounded);
^^^^^^^^^^^^^^^^ expected struct winit::window::Window, found a different struct winit::window::Window
So I checked the dependencies and everything is up to date:
[dependencies]
winit = "0.26.1"
imgui-winit-support = "0.8.2"
imgui = "0.8.2"
So I checked the Cargo.lock file and there are two different versions of
winit:
[[package]]
name = "winit"
version = "0.25.0"
....
[[package]]
name = "winit"
version = "0.26.1"
I don't know what this is caused by or how to fix it so any help is
greatly appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/imgui-rs/imgui-rs/issues/620, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAD7JCDEOHSTPMIPFO54LU6Z6W3ANCNFSM5QETVGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

using
imgui-winit-support = {version = "0.8.2", features = ["winit-26"]}

works
thank you


Required fields with asterisk

Task: https://app.clickup.com/t/337yt2a

One pecularity is that we use showRequired for InputLabel and Fieldset, while for the other components we use required and hideRequired.

3. I think the `FieldsetFormField` should also have the `required`/`hideRequired` flags to match the FormField behavior. I know it's a bit weird because the `FieldSetField` works with `required`/`showRequired` but most of the time we'll use the FormField and it would be even weirder to have to set `showRequired` just for that FormField.

Regards point 3, what about always using the props required/hideRequired? It allows us to have a coherent API and put the logic of when to actually show the asterisk inside InputLabel and FieldSet only.

Regards point 3, what about always using the props required/hideRequired? It allows us to have a coherent API and put the logic of when to actually show the asterisk inside InputLabel and FieldSet only.

I see your point, but IMO hideRequired does not work well with InputLabel, Fieldset and FieldsetField because the logic is different. They do not have a separate required flag.

I discovered an unpleasant thing: the native required notice appears where the hidden input field is. Do you think it's possible to move it near the actual input element? I would not spend much time on this, just in case you find a solution quickly. You can also open a new ticket/PR for this. Thank you.

After a discussion with @alexlanz, the conclusion is to use showRequired for the following components:

InputLabel
Fieldset
FieldsetField
FieldsetFormField

After a discussion with @alexlanz, the conclusion is to use showRequired for the following components:

InputLabel
Fieldset
FieldsetField
FieldsetFormField

This seems a bit strange for FieldsetFormField. The only reason I specify the fields prop is to show the error state. Now I have to set showRequired in addition or it won't work.

Also, all the other FormFields have the hideRequired flag. This does not seem reasonable and consistent to me.

This seems a bit strange for FieldsetFormField. The only reason I specify the fields prop is to show the error state. Now I have to set showRequired in addition or it won't work. Also, all the other FormFields have the hideRequired flag. This does not seem reasonable and consistent to me.

Never mind, I got it wrong 🙈

Your input has not been ignored. The conclusion is basically that using showRequired for FieldsetFormField is not strange, because, like InputLabel, FieldsetFormField does not have a HTML required property.

No, sorry, I was just a bit confused. It's fine for me :)


[Beta4]: Combine bolt with chaining in F. Manipulator crash.

Tested with only Baubles and TC6b4.
Insert a blank focus into the Manipulator. Drag Bolt into the centre. Try to drag Chaining onto the manipulator and the client crashes.
crash-2018-02-08_22.05.34-client.txt

Same thing, but with all the possible combinations, if i drag a medium in the center, and try to add others gems, like effects, it crashes.
This happen even if i first put all the  effects and modifications, and then the medium.
crash-2018-02-08_15.27.50-client.txt

I opened another issue that was similar to this, exept my game didn't crash. (I didn't see this issue, I guess. Oops.)


Update coursier, coursier-launcher_2.13 to 2.1.19

About this PR
📦 Updates

io.get-coursier:coursier
io.get-coursier:coursier-launcher_2.13

from 2.1.14 to 2.1.19
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.get-coursier" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.get-coursier" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1451.


How to run it in Windows?

Is it possible to run Vegeta in Windows? And what about a Docker?

There are binary releases that includes window, but is not the latest.

Latest release includes Windows builds.


Correct answer prints but is not being recognized

Golf Code under JavaScript course
The test requests:
Strokes          Return
1                    "Hole-in-one!"
<= par - 2      "Eagle"
par - 1         "Birdie"
par                 "Par"
par + 1         "Bogey"
par + 2          "Double Bogey"
>= par + 3  "Go Home!"

My code:
javascript
function golfScore(par, strokes) {
  // Only change code below this line

  if (strokes == 1)
    {
      return "Hole-in-one!";
    }

  else if (strokes <= (par -2))
    {
      return "Eagle";
    }

  else if (strokes == (par -1))
    {
      return "Birdie";
    }
  else if (strokes == par)
    {
      return "Par";
    }

  else if (strokes == (par + 1))
    {
      return "Bogey";
    }

  else if (strokes == (par + 2))
  {
    return "Double Bogey";
  }
  else if (strokes >= (par + 3))
  {
    return "Go Home";
  }

  else
  {
  return "Change Me";

  }
  // Only change code above this line
}

// Change these values to test
golfScore(5, 4);

It says the following is not done:
golfScore(4, 7) should return "Go Home!"
golfScore(5, 9) should return "Go Home!"

@chocolatechimpcookie thanks for the issue but if you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Hint: look at the two tests you have pointed out and double check whether the strings it returns is exactly the same as what it requires. You have a missing character in the returned string. Happy coding!

you forget the "!" in in "Go Home!"
and  to remove the  following section:
else
{
return "Change Me"; ]

Exactly the same with me. Forgot the ! after "Go Home!"

function golfScore(par, strokes) {
// Only change code below this line
if (strokes == 1){return "Hole-in-one!";}
else if(strokes <= (par-2)){return "Eagle";}
else if(strokes == (par-1)){return "Birdie";}
else if(strokes == (par)){return "Par";}
else if(strokes == (par+1)){return "Bogey";}
else if(strokes == (par+2)){return "Double Bogey";}
else if(strokes >= (par+3)){return "Go Home!";}
else {return "Change Me";}
// Only change code above this line
}
// Change these values to test
golfScore(5, 4);

Thank's for help!   You must change (=)   (==).


Update gcc version for newer omnios versions.

I included a test for 151014 since it is the latest LTS release, but
looking at the release notes the version of gcc actually changed in
151008 so that is what I had the recipe test.

I think the build failure is do to fauxhai not being able to download the latest system definitions while being run inside travis.

I'm having a similar error running some local builds.  Is there a solution/workaround for the fauxhai failure?

Fauxhai has been updated so it should be shipping with the latest and greatest defs now.  Can you rebase this and we can see if we can get the tests passing.

Rebased and it looks like the tests are passing now.

Thanks for getting that fixed up


Move theming from Beta to Main

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Summary by Sourcery
Move theming from Beta to Main by introducing theme support for the auro-accordion component, allowing customization through design tokens. Refactor the component to use a new dependency versioning system for icons and update styling for consistency. Update documentation to reflect these changes and remove outdated information.
New Features:

Introduce theme support for the auro-accordion component, allowing it to be restyled using design tokens.

Enhancements:

Refactor the auro-accordion component to use a new dependency versioning system for icons.
Update the styling of the auro-accordion component to improve consistency and maintainability by using design tokens for border and color properties.

Documentation:

Add documentation on theme support for the auro-accordion component, including a code sample for restyling using design tokens.

Chores:

Remove outdated CSS custom property fallbacks documentation from README and other documentation files.

:tada: This PR is included in version 4.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


env variable to toggle MPT service

Hi @puagarwa, sorry to ask this question in this window. I have integrated, Microsoft playwright testing service integration into my project using NUnit/Playwright/C#. I'm able to run the code in the cloud, its working fine for me. But the issue is with NumberOfTestWorkers. I'm not able to run more than 1 worker. I have given configuration in .runsettings as well. Can you please help me on this.

Hi @uday2029, for NUNit, you can refer to the below documentations for configuring number of parallel workers. There's some limitations as well (outlined in the below links) on configuring parallel workers for NUnit.
https://github.com/microsoft/playwright-testing-service/tree/main/samples/.NET/NUnit#run-tests
https://playwright.dev/dotnet/docs/test-runners#running-nunit-tests-in-parallel
If the issue still persists, feel free to open an issue with additional details so that we can help you out.


20 Javascripts one liners that will make every coders life easier!

Hello, sir/mam thank you so much for providing this opportunity for open-source contribution. I have made these 20 Javascript one-liners that will really help every coder in their coding journey, hope you will like It, and if do consider it for Hectoberberfest.
Thank you!

Thank you so much sir🤗
On Tue, Oct 19, 2021, 10:54 AM Aman kumar sharma @.***>
wrote:

Thanks for your contribution
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shruti49/Hacktoberfest2021_beginners/pull/120#issuecomment-946377050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVK5LULYGLFZYFH6OII7BCDUHT6HTANCNFSM5GGP27KQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[Automation] Generate Fluent Lite from networkcloud#package-2023-07-01

[Automation] Generate Fluent Lite from networkcloud#package-2023-07-01

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-networkcloud


[CMake] update to 3.21.0

Library name: CMake from Kitware
New version number: 3.21.0
Other information that may be useful (release notes, etc...)
ref.:
scripts/vcpkgTools.xml
https://cmake.org/download/#:~:text=Latest Release (3.21.0)
https://cmake.org/cmake/help/v3.21/release/3.21.html#cmake-3-21-release-notes

https://github.com/microsoft/vcpkg-tool/pull/107
a new vcpkg-tool is also mandatory for cmake 3.21
fixes already landed there, I hope it can be released soon to unblock pipelines that are already using cmake 3.21

Can I second the request for a new vcpkg-tool release. Not having the commit https://github.com/microsoft/vcpkg-tool/commit/8b1c04b3b8e50db93d0245e761f3d8ebeb400309 means anyone using vcpkg migrating to CMake 3.21 will face doom and gloom. See https://gitlab.kitware.com/cmake/cmake/-/issues/22371 and https://gitlab.kitware.com/cmake/cmake/-/issues/22422 for at least two reports of the problem

@davidhunter22 We will release the next version in a few days.

It will affect a lot of github users. The latest Windows 2019 image provided by github actions works fine, because it uses cmake 3.20.5. (see https://github.com/actions/virtual-environments/blob/releases/win19/20210711/images/win/Windows2019-Readme.md).
But once they regenerate the image, vcpkg will crash. Usually they update the image weekly.

It will affect a lot of github users. The latest Windows 2019 image provided by github actions works fine, because it uses cmake 3.20.5. (see https://github.com/actions/virtual-environments/blob/releases/win19/20210711/images/win/Windows2019-Readme.md).
But once they regenerate the image, vcpkg will crash. Usually they update the image weekly.

This kind of eventually happened and our CI job fails running vcpkg command because Windows image now has CMake 3.21
@JackBoosY Any hints on the ETA for next vcpkg release ?

@9prady9 The update for cmake 3.21.0 is ready for merge now.


[GLUTEN-7012][SHIMS] Make spark shim provider configurable

What changes were proposed in this pull request?
Make spark shim provider configurable
closes #7012
How was this patch tested?
TODO

What's the use case of this feature? I think shim layers may require refactors in near future so we may hardly maintain a stable interface.

What's the use case of this feature? I think shim layers may require refactors in near future so we may hardly maintain a stable interface.

Our internal spark version is based on 3.5.0 and actually backports most of the features of the 3.5 branch. I want to make gluten spark 3.5 shim work by configuring.

I think shim layers may require refactors in near future so we may hardly maintain a stable interface.

I'm not trying to implement custom SparkShims, it's just that the current SparkShims loading is too strict.

I think shim layers may require refactors in near future so we may hardly maintain a stable interface.

I'm not trying to implement custom SparkShims, it's just that the current SparkShims loading is too strict.

So in your use case, the code of 3.5.0 shim will still be replaced in Gluten? Perhaps in your Gluten fork repo?

So in your use case, the code of 3.5.0 shim will still be replaced in Gluten?

Yeah, I want to make gluten spark35 shims work internal spark 3.5.0 version.

Perhaps in your Gluten fork repo?

Thank you, I will do that if this PR doesn't get accepted.

Thank you, I will do that if this PR doesn't get accepted.

My thought was, if you already have a Gluten fork, the we can put off merging this kind of PR, since the same job can be done in your forked repo.
Otherwise if you directly use upstream Gluten with 3.5.0 Spark, then perhaps we could proceed.

My thought was, if you already have a Gluten fork, the we can put off merging this kind of PR, since the same job can be done in your forked repo.

Thanks, this is feasible for us, we have a gluten fork repo.

SparkShimLoader: Using Shim provider: List(org.apache.gluten.sql.shims.spark35.SparkShimProvider@3a2a959e)

@zhztheplayer Sorry, I missed this log, actually org.apache.gluten.sql.shims.spark35.SparkShimProvider is effective for spark 3.5.0, so the current PR is unnecessary and I will close it.


Add 🏋♂🏋♀ heavy duty CI 🔁🔁

This adds some CI I worked on developing for a while. It validates with HACS and Hassfest, formats with pushes, checks formatting of PRs, and does some very strict lint.

Hey, do you see any problems in renaming afvalwijzer_scraper's domain to afvalwijzer_scraper?

@KTibow Thanks for your great help! This is very much appreciated!
I don't see any problems renaming it.
Thanks again!

BTW mind squash merging when you merge?


Update-Dbatools failed; Now can't run any commands

Slack Community Channel: https://sqlcommunity.slack.com/messages/C1M2WEASG/ (get an invite: https://dbatools.io/slack/)
Documentation within the module: Get-Help <command>
Database Administrator on Stack Exchange network, dbatools tag: https://dba.stackexchange.com/questions/tagged/dbatools
StackOverflow dbatools tag: https://stackoverflow.com/questions/tagged/dbatools

Dbatools was working, but I tried to update it by running Update-Dbatools which failed and told me to Import Dbatools. I couldn't see how to do that so ended up trying to uninstall and then I did a manual install.
Now if I run any dbatools command, I get for example this message: The 'Get-DbaUptime' command was found in the module 'dbatools', but the module could not be loaded. For more information, run 'Import-Module dbatools'.
If I run Import-Module dbatools I get the following:
Import-Module : The following error occurred while loading the extended type data file: , C:\Program Files (x86)\Microsoft SQL Server\140\Tools\PowerShell\Modules\dbatools\xml\dbatools.Types.ps1xml: The file was skipped because of the following validation exception: File C:\Program Files (x86)\Microsoft SQL Server\140\Tools\PowerShell\Modules\dbatools\xml\dbatools.Types.ps1xml cannot be loaded. The file C:\Program Files (x86)\Microsoft SQL Server\140\Tools\PowerShell\Modules\dbatools\xml\dbatools.Types.ps1xml is not digitally signed. You cannot run this script on the current system.
Get-Module dbatools -ListAvailable returns a single row which tells me my version is 1.0.107
It was suggested on Slack that I raise a new issue here. I hope someone can help me to fix dabtools.  Kind regards...

Anyone have any ideas how I can fix this?

Hi @FreedToFly is this still an issue?

@FreedToFly the update command we package with the module should only be used if you installed the module using the install script. If you are using the proper method for online installation from the PowerShell Gallery (Install-Module) this command should not be used.
If the version of the module on your machine is not importing properly I would recommend deleting it completely and then doing a new installation to pull the latest version.
If this is still causing problems please open a new issue and provide details on how you have installed the module.


[ISSUE] Navbar-logo

Hello,
The Navbar-logo of this website have responsiveness issue . When we open this website in phone view, the Navbar-logo have disappear . So, I will like to fix this ISSUE.
Thanks!

Assigned. You can work on it


Allow HTTPS

This pull request introduces four CLI flags allowing users to serve Storybook over HTTPS:
    --https                       Serve Storybook over HTTPS. Note: You must provide your own certificate information.
    --ssl-ca <ca>                 Provide an SSL certificate authority. (Required with --https)
    --ssl-cert <cert>             Provide an SSL certificate. (Required with --https)
    --ssl-key <key>               Provide an SSL key. (Required with --https)

@arbesfeld Thanks. The use case is pretty straightforward. We have an HTTPS-everywhere policy @nfl. Some of our components change state depending on user login state, but this requires access to cookies. Since our cookies are secure, they can only be read from a domain with a valid SSL cert. Hence the need.
I think I'll eventually stub that out so that I don't need actual cookies to display my component's state in Storybook, but this was a pain point during my prototype phase where I just wanted something quick and dirty to build my component. I'm sure similar use-cases exist for other SSL-sensitive APIs like Service Workers, LocalStorage, SessionStorage, etc.

@shilman Oh interesting. It works fine as a string. I'll update the code to handle either case, and make ca optional.

@shilman Done.
Also, it's out of scope for this PR, but I was thinking about adding support for a .storybookrc config file. As you can see, more CLI arguments can make for an unwieldy startup command. A config file would clean that up. It would also allow Storybook to stop relying on SBCONFIG_* environment variables to configure the app.

@doctyper Interesting idea about the config file. I agree that the growing list of command-line args is unwieldy, but I think the SBCONFIG env variables are useful for deployments, and it brings up a point that all the other args except for these new SSL args can be set using SBCONFIG vars. I'm fine with this inconsistency, but I just want to point it out as part of the review.
I agree that an alternative solution is outside the scope of this PR.

@shilman I can add the new args to the list of env vars if you'd like. Not a problem.

@doctyper I'm fine either way regarding the env vars, just wanted to point out the inconsistency if it wasn't intentional. Thanks for the quick fixes and the great feature!!

@arbesfeld In addition to  @doctyper's use case mentioned above, I think once we open up the server-side plugins as discussed in the video chat yesterday, there will be a lot of cases where one will want to run the server under HTTPS rather than serving from static files. (Whether this is best done by extending storybook-server or just pointing your client-side plugins at some completely different server is debatable.)

I was thinking, maybe at some point in the future we could create middleware/routers for stuff like express. That way people would be in full control of the web-app and storybook would just be a route on that app.


Using built-in user model

Hi,
i try to use the connector with the built-in user model. This works fine (also for other, non built-in models) as long as i do not attach the ACL-model to the couchbase datasource. If i do, i get this error in the explorer console (even when doing requests to other, non built-in models):
{
"error": {
"name": "Error",
"status": 500,
"message": "inq is an unknown key",
"statusCode": 500,
"stack": "Error: inq is an unknown key\n    at defaultHandler (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\strong-remoting\lib\rest-adapter.js:337:15)\n    at restErrorHandler (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\strong-remoting\lib\rest-adapter.js:327:14)\n    at Layer.handle_error (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\layer.js:58:5)\n    at trim_prefix (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:269:13)\n    at C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:238:9\n    at Function.proto.process_params (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:313:12)\n    at C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:229:12\n    at Function.match_layer (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:296:3)\n    at next (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\index.js:190:10)\n    at Layer.handle_error (C:\Users\Andre\WebstormProjects\couchbase-neu\node_modules\loopback\node_modules\express\lib\router\layer.js:54:12)"
}
}
This is my setup for datasources.json and model-config.json:
{
"db": {
"name": "db",
"connector": "memory"
},
"couchbase": {
"connector": "couchbase",
"database": "user_bucket"
}
}
{
...
"User": {
"dataSource": "couchbase",
"public": true
},
"AccessToken": {
"dataSource": "couchbase",
"public": true
},
"ACL": {
"dataSource": "couchbase",
"public": true
},
"RoleMapping": {
"dataSource": "couchbase",
"public": true
},
"Role": {
"dataSource": "couchbase",
"public": true
},
"Note": {
"dataSource": "couchbase",
"public": true
}
}
If i set ACL-datasource back to use the in-memory db everything works fine!? Do you have any clue whats going on?

We are in the process of writing a fix for this as some of the functionality for WHERE clauses was not completed.

Hi,  we have published a new version of the connector, which should fix this issue.  Please test and let us know.

Hi, thanks, thats great. Will test it tomorrow and give you feedback...


vimPlugins: warn if alias is used inside overrides

This changes the vimPlugins structure to be like stage.nix and top-level/aliases.nix.
vimPlugins was breaking evaluation if config.allowAliases was set to false, and we wanted to make sure it is harder to do that in the future.

cc @Mic92

@Mic92 Okay, I simplified it.

@Mic92 Do you approve of this as is? We should probably merge it quickly before someone wants to do something with vim plugins.


[BUG] Not compatible with Typescript >= 5.5.2

Are you using the latest version of this library?

[X] I verified that the issue exists in the latest next-safe-action release

Is there an existing issue for this?

[X] I have searched the existing issues and found nothing that matches

Describe the bug
parsedInput returning any, schema defined by Zod
Will provide a minimal repo soon.
Reproduction steps

Install Typescript 5.5.2
Create a safe action with Zod schema, check the type of parsedInput

Expected behavior
parsedInput should have the correct type information
Minimal reproduction example
Code Sandbox link
Operating System
MacOS 14
Library version
7.2.0
Next.js version
14.2.4
Additional context
No response

Will look into it soon, thanks.

I can confirm schema validations don't work with typescript 5.5.2

After some tests, I identified the problem in TypeSchema, which breaks with TypeScript 5.5. I've just opened an issue on the TypeSchema repo for this bug:

decs/typeschema#61

I know this is a pretty new issue, but a quick easy workaround for me was to just downgrade typescript. I just locked at typescript 5.4.5 (I use the LSP that comes with Mason) for now, it seems to work perfectly.

@RolandRFuchs92 yes, the workaround is to downgrade to TypeScript v5.4. The issue appears to be related to a compiler change in TS 5.5.2, which broke ts-patch and so TypeSchema as well.

I've just released next-safe-action v7.1.4-experimental.1, that should work fine with TypeScript 5.5. Note that this version only works with Zod, it's essentially the same codebase without TypeSchema support, as it's currently preventing the ability to upgrade to TS 5.5.
So, if you're using Zod and TS 5.5, you can install experimental next-safe-action with this command:
npm i next-safe-action@experimental # or pnpm, yarn

P.S.: Even though the tag is named "experimental", as explained above the codebase is essentially the same as the stable v7. It's named like that because I don't think this will be an actual library change, as it's nice to have multiple validation libraries support. However, it's also nice to use the latest TS version, so until TypeSchema is compatible with it, I think this is a good solution.

I've just released next-safe-action v7.1.4-experimental.1, that should work fine with TypeScript 5.5. Note that this version only works with Zod, it's essentially the same codebase without TypeSchema support, as it's currently preventing the ability to upgrade to TS 5.5.
So, if you're using Zod and TS 5.5, you can install experimental next-safe-action with this command:
npm i next-safe-action@experimental # or pnpm, yarn

P.S.: Even though the tag is named "experimental", as explained above the codebase is essentially the same as the stable v7. It's named like that because I don't think this will be an actual library change, as it's nice to have multiple validation libraries support. However, it's also nice to use the latest TS version, so until TypeSchema is compatible with it, I think this is a good solution.

Hey! Just to let you know is working for me using TS 5.5 and zod. Thanks for your amazing work!

I'm trying the experimental version, but I can't destruct the response because is possibly undefined.

@mattiaz9 that's not an experimental version bug, it's the expected behavior since v7.0.0. The explanation for that can be found here.

@mattiaz9 that's not an experimental version bug, it's the expected behavior since v7.0.0. The explanation for that can be found here.

I see. I was previously using a pre-release of v7 and it wasn't yet the case, so I thought it was an error.
It didn't help the fact that in the docs that behavior is not shown.
For instance in this example res is destructed without checking if undefined:
https://next-safe-action.dev/docs/getting-started#3-import-and-execute-the-action

@mattiaz9

It didn't help the fact that in the docs that behavior is not shown.

Fixed. It was left over from v6 docs.

This should probably be added to docs as well if not already there.

@aniketbiprojit https://next-safe-action.dev/docs/troubleshooting#schema-and-parsedinput-are-typed-any-broken-types-and-build-issues
But I'll add a note on the getting started page too, soon.

This issue should be solved in v7.2.0. next-safe-action now ships with a built-in system for supporting multiple validation libraries, at this time Zod (the default one), Valibot and Yup. You can now use them even with the Edge Runtime and obviously with TS >= 5.5. More information about that in this discussion and on this page of the website.
Please let me know if you encounter any bugs when updating to v7.2.0, thanks!


InvalidArAgumentError (see above for traceback): indices[478] = 5451 is not in [0, 5451)

I'm getting this error on Epoch 1 of run_20newsgroups.py:
InvalidArgumentError (see above for traceback): indices[478] = 5451 is not in [0, 5451)
[[node word_embed_lookup (defined at /Users/davidlaxer/Lda2vec-Tensorflow/lda2vec/Lda2vec.py:152)  = GatherV2[Taxis=DT_INT32, Tindices=DT_INT32, Tparams=DT_FLOAT, _class=["loc:@Optimizer/train/update_word_embedding/AssignSub"], _device="/job:localhost/replica:0/task:0/device:CPU:0"](word_embedding/read, _arg_x_pivot_idxs_0_1, word_embed_lookup/axis)]]
It seems that the 'word_embed_lookup ' tensor contains an embedding reference beyond the length of the embedding_matrix.  Any ideas where this 'off by one' issue could be?
# Word embedding lookup
        word_context = tf.nn.embedding_lookup(self.w_embed.embedding, x, name='word_embed_lookup')

similar to:
https://github.com/nateraw/Lda2vec-Tensorflow/issues/5

---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1333     try:
-> 1334       return fn(*args)
   1335     except errors.OpError as e:

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _run_fn(feed_dict, fetch_list, target_list, options, run_metadata)
   1318       return self._call_tf_sessionrun(
-> 1319           options, feed_dict, fetch_list, target_list, run_metadata)
   1320

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _call_tf_sessionrun(self, options, feed_dict, fetch_list, target_list, run_metadata)
   1406         self._session, options, feed_dict, fetch_list, target_list,
-> 1407         run_metadata)
   1408

InvalidArgumentError: indices[250] = 5451 is not in [0, 5451)
	 [[{{node word_embed_lookup}} = GatherV2[Taxis=DT_INT32, Tindices=DT_INT32, Tparams=DT_FLOAT, _class=["loc:@Optimizer/train/update_word_embedding/AssignSub"], _device="/job:localhost/replica:0/task:0/device:CPU:0"](word_embedding/read, _arg_x_pivot_idxs_0_1, word_embed_lookup/axis)]]

During handling of the above exception, another exception occurred:

InvalidArgumentError                      Traceback (most recent call last)
<ipython-input-3-7f4db3f7fa23> in <module>()
      6         num_epochs,
      7         idx_to_word=idx_to_word,
----> 8         switch_loss_epoch=switch_loss_epoch)
      9
     10 # Visualize topics with pyldavis

~/Lda2vec-Tensorflow/lda2vec/Lda2vec.py in train(self, pivot_words, target_words, doc_ids, data_size, num_epochs, switch_loss_epoch, save_every, report_every, print_topics_every, idx_to_word)
    244
    245                 # Run a step of the model
--> 246                 summary, _, l, lw2v, llda, step = self.sesh.run(fetches, feed_dict=feed_dict)
    247
    248             # Prints log every "report_every" epoch

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    927     try:
    928       result = self._run(None, fetches, feed_dict, options_ptr,
--> 929                          run_metadata_ptr)
    930       if run_metadata:
    931         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1150     if final_fetches or final_targets or (handle and feed_dict_tensor):
   1151       results = self._do_run(handle, final_targets, final_fetches,
-> 1152                              feed_dict_tensor, options, run_metadata)
   1153     else:
   1154       results = []

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _do_run(self, handle, target_list, fetch_list, feed_dict, options, run_metadata)
   1326     if handle is None:
   1327       return self._do_call(_run_fn, feeds, fetches, targets, options,
-> 1328                            run_metadata)
   1329     else:
   1330       return self._do_call(_prun_fn, handle, feeds, fetches)

~/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1346           pass
   1347       message = error_interpolation.interpolate(message, self._graph)
-> 1348       raise type(e)(node_def, op, message)
   1349
   1350   def _extend_graph(self):

InvalidArgumentError: indices[250] = 5451 is not in [0, 5451)
	 [[node word_embed_lookup (defined at /home/ubuntu/Lda2vec-Tensorflow/lda2vec/Lda2vec.py:152)  = GatherV2[Taxis=DT_INT32, Tindices=DT_INT32, Tparams=DT_FLOAT, _class=["loc:@Optimizer/train/update_word_embedding/AssignSub"], _device="/job:localhost/replica:0/task:0/device:CPU:0"](word_embedding/read, _arg_x_pivot_idxs_0_1, word_embed_lookup/axis)]]

Caused by op 'word_embed_lookup', defined at:
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel_launcher.py", line 16, in <module>
    app.launch_new_instance()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/traitlets/config/application.py", line 658, in launch_instance
    app.start()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/kernelapp.py", line 486, in start
    self.io_loop.start()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tornado/platform/asyncio.py", line 127, in start
    self.asyncio_loop.run_forever()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/asyncio/base_events.py", line 427, in run_forever
    self._run_once()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/asyncio/base_events.py", line 1440, in _run_once
    handle._run()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/asyncio/events.py", line 145, in _run
    self._callback(*self._args)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tornado/platform/asyncio.py", line 117, in _handle_events
    handler_func(fileobj, events)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tornado/stack_context.py", line 276, in null_wrapper
    return fn(*args, **kwargs)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/zmq/eventloop/zmqstream.py", line 450, in _handle_events
    self._handle_recv()
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/zmq/eventloop/zmqstream.py", line 480, in _handle_recv
    self._run_callback(callback, msg)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/zmq/eventloop/zmqstream.py", line 432, in _run_callback
    callback(*args, **kwargs)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tornado/stack_context.py", line 276, in null_wrapper
    return fn(*args, **kwargs)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/kernelbase.py", line 283, in dispatcher
    return self.dispatch_shell(stream, msg)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/kernelbase.py", line 233, in dispatch_shell
    handler(stream, idents, msg)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/kernelbase.py", line 399, in execute_request
    user_expressions, allow_stdin)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/ipkernel.py", line 208, in do_execute
    res = shell.run_cell(code, store_history=store_history, silent=silent)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/ipykernel/zmqshell.py", line 537, in run_cell
    return super(ZMQInteractiveShell, self).run_cell(*args, **kwargs)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2662, in run_cell
    raw_cell, store_history, silent, shell_futures)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2785, in _run_cell
    interactivity=interactivity, compiler=compiler, result=result)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2903, in run_ast_nodes
    if self.run_code(code, result):
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2963, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-1-2177bca49b35>", line 41, in <module>
    save_graph_def=save_graph)
  File "/home/ubuntu/Lda2vec-Tensorflow/lda2vec/Lda2vec.py", line 82, in __init__
    handles = self._build_graph()
  File "/home/ubuntu/Lda2vec-Tensorflow/lda2vec/Lda2vec.py", line 152, in _build_graph
    word_context = tf.nn.embedding_lookup(self.w_embed.embedding, x, name='word_embed_lookup')
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/ops/embedding_ops.py", line 313, in embedding_lookup
    transform_fn=None)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/ops/embedding_ops.py", line 133, in _embedding_lookup_and_transform
    result = _clip(array_ops.gather(params[0], ids, name=name),
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/ops/array_ops.py", line 2675, in gather
    return gen_array_ops.gather_v2(params, indices, axis, name=name)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/ops/gen_array_ops.py", line 3332, in gather_v2
    "GatherV2", params=params, indices=indices, axis=axis, name=name)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
    return func(*args, **kwargs)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3274, in create_op
    op_def=op_def)
  File "/home/ubuntu/anaconda/envs/ai/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1770, in __init__
    self._traceback = tf_stack.extract_stack()

InvalidArgumentError (see above for traceback): indices[250] = 5451 is not in [0, 5451)
	 [[node word_embed_lookup (defined at /home/ubuntu/Lda2vec-Tensorflow/lda2vec/Lda2vec.py:152)  = GatherV2[Taxis=DT_INT32, Tindices=DT_INT32, Tparams=DT_FLOAT, _class=["loc:@Optimizer/train/update_word_embedding/AssignSub"], _device="/job:localhost/replica:0/task:0/device:CPU:0"](word_embedding/read, _arg_x_pivot_idxs_0_1, word_embed_lookup/axis)]]

The 'pivot_ids' and the 'target_ids' in 'skipgram.txt', appear to be 'one-based',
but the embedding_matrix is 'zero-based':
E.g. -
print(np.sort(pivot_ids))
[   1    1    1 ... 5451 5451 5451]
print(np.sort(target_ids))
[   1    1    1 ... 5451 5451 5451]

The Keras Tokenizer appears to output 1 based ids' and states:
"0 is a reserved index that won't be assigned to any word."
https://keras.io/preprocessing/text/#tokenizer

Strange...This totally could be because I was running those other experiments I talked to you about via email. I might have made changes that I shouldn't have pushed back up here.

In load_glove() the indexes are zero-based.
But the skipgram indexes are one-based.
Could this have something to do with the exception?
embeddings_index = dict(get_coefs(*o.split(" ")) for o in open(EMBEDDING_FILE))

        all_embs = np.stack(embeddings_index.values())
        emb_mean,emb_std = all_embs.mean(), all_embs.std()
        embed_size = all_embs.shape[1]

        # word_index = tokenizer.word_index
        nb_words = self.vocab_size
        embedding_matrix = np.random.normal(emb_mean, emb_std, (nb_words, embed_size))
        for word, i in self.word_to_idx.items():
            if i >= self.vocab_size: continue
            embedding_vector = embeddings_index.get(word)
            if embedding_vector is not None: embedding_matrix[i] = embedding_vector

``

I get this error even with: load_embeds = False

On Mar 31, 2019, at 12:45 PM, Nathan Raw<EMAIL_ADDRESS>wrote:
Strange...This totally could be because I was running those other experiments I talked to you about via email. I might have made changes that I shouldn't have pushed back up here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nateraw/Lda2vec-Tensorflow/issues/43#issuecomment-478374086, or mute the thread https://github.com/notifications/unsubscribe-auth/AC9i24UfR0yT6vLVcSTcDA4KQxj8yuBBks5vcRBPgaJpZM4cQhKV.

Found the source of this issue. Fix coming. It's actually a tensorflow version issue, I think. I don't get the issue on TF v1.5.0. Either way, I'll fix it.

That make sense. Any details on the issue?

On Apr 1, 2019, at 9:24 AM, Nathan Raw<EMAIL_ADDRESS>wrote:
Found the source of this issue. Fix coming. It's actually a tensorflow version issue, I think. I don't get the issue on TF v1.5.0. Either way, I'll fix it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nateraw/Lda2vec-Tensorflow/issues/43#issuecomment-478648568, or mute the thread https://github.com/notifications/unsubscribe-auth/AC9i20H_R5VHhjPECHzBoKk2aTIHA9oSks5vcjLLgaJpZM4cQhKV.

I believe the easy fix is to pass an unk_token="<SKIP>" when initializing the keras tokenizer within nlppipe.py. Will check tonight and push. I've also made better installation instructions using conda environment so you can make sure you're on the same versions of everything.

Issue isn't that keras tokenizer starts at 1. The issue is that we don't have a token representation of idx 0, so it skips it when doing the loop in load_glove(). If you try to just assume everything is from 1-vocab_size, then you get a different error in TF.

I'm no longer getting the error:
InvalidArAgumentError (see above for traceback): indices[478] = 5451 is not in [0, 5451

 is appearing in the list of 'closest' words per topic.
 I remember messing with that function a while back when something wasn't working. It was actually related to this issue, most likely. Again, the tensorflow version I'm using didn't give me the InvalidArgumentError, so I didn't know why there was a bug.
I'll take a deeper look at the topics function. I think theres probably only a couple small tweaks that need to be made. Will close this issue for now, as this specific problem has been solved.


Out of Date

I can find no such thing as a Notification Template in either NAV 2018 or Dynamics 365. If there is, can you please include on this page and other pages that reference them a specific navigation direction?

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

ID: 12bd6782-3076-cb3f-73ab-e66d3aa6217d
Version Independent ID: cdb8ea99-5a17-b4c0-4bfd-0523cad05adf
Content: Setting Up Workflow Notifications - Dynamics NAV App
Content Source: dynamics-nav-app/across-setting-up-workflow-notifications.md
Product: dynamics-nav-2018
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hi - Application help for Dynamics NAV is no longer updated/maintained by Microsoft, nor are GitHub issues like this one systematically tracked by Microsoft. Therefore, whenever you find something missing or wrong in NAV Help, please refer to the help for Dynamics 365 Business Central help, the online replacement of NAV, in which the application scope is 99% the same. For example, see this topic about setting up workflow notifications: https://docs.microsoft.com/en-us/dynamics365/business-central/across-how-to-specify-when-and-how-to-receive-notifications


Kernel Error

Hi,
I'm having an issue where when I launch jupyter notebook, it gives me a Kernel Error. I have tried using anaconda, and also uninstalling anaconda and running from the base python with the same results. I have no clue what's going on. I have read the similar responses here, but nothing there has seemed to work. Any thoughts on what I can do?
The error looks like this:
File "/Users/meganicka/anaconda/lib/python2.7/subprocess.py", line 1024, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
Presumably it's looking for python in the wrong place, but I don't know how to get it to look in the correct place.
Thanks in advance for your help.
Best,
Megan

Is using the default Anaconda failing as well? Can you post the complete stack trace? Any other particular things in your deploy you believe could help to know what is happening?

Closing as stale, please re-open if you have more information. Cheers.

I am getting a similar error.  If I go to the "Conda" tab in the notebook, I get two popups saying "An error occurred while listing Conda environments." and "An error occurred while retrieving package information."  Each also says "Internal server error".  In the console where I launched the notebook, this stack trace appears:
[E 13:50:02.409 NotebookApp] Unhandled error in API request
    Traceback (most recent call last):
      File "C:\FakeProgs\Anaconda3\envs\nb_conda_kernels\lib\site-packages\notebook\base\handlers.py", line 516, in wrapper
        result = yield gen.maybe_future(method(self, *args, **kwargs))
      File "C:\FakeProgs\Anaconda3\envs\nb_conda_kernels\lib\site-packages\nb_conda\handlers.py", line 229, in get
        data = searcher.list_available(self)
      File "C:\FakeProgs\Anaconda3\envs\nb_conda_kernels\lib\site-packages\nb_conda\handlers.py", line 213, in list_available
        bufsize=4096)
      File "C:\FakeProgs\Anaconda3\envs\nb_conda_kernels\lib\subprocess.py", line 676, in __init__
        restore_signals, start_new_session)
      File "C:\FakeProgs\Anaconda3\envs\nb_conda_kernels\lib\subprocess.py", line 955, in _execute_child
        startupinfo)
    FileNotFoundError: [WinError 2] The system cannot find the file specified

This seems to be an issue with nb_conda, not nb_conda_kernels. I would try to reproduce it, but we should put it in the proper issue tracker.
Can you post your notebook, nb_conda_kernels and nb_conda versions? A conda list from your environment should be enough to get this info.

The versions:
nb_conda                  2.0.0                    py35_0
nb_conda_kernels          2.0.0                    py35_0
notebook                  5.0.0                    py35_0

I think I have an idea what the problem is.  On Windows, the various Anaconda tools (like conda) in each environment are batch files that use the call shell command to call the main version in the root environment.  But since call is a shell command, it doesn't work unless you use shell=True in the various subprocess calls, which nb_conda is not doing.
I think the use of these subprocess calls also means it won't work if I try to start the notebook without activating its conda environment (since the path won't be set right), but I guess that's more of an overall issue with the conda architecture.

I think I have an idea what the problem is. On Windows, the various Anaconda tools (like conda) in each environment are batch files that use the call shell command to call the main version in the root environment. But since call is a shell command, it doesn't work unless you use shell=True in the various subprocess calls, which nb_conda is not doing.

I was suspecting the same... thanks for confirm it.

I think the use of these subprocess calls also means it won't work if I try to start the notebook without activating its conda environment (since the path won't be set right), but I guess that's more of an overall issue with the conda architecture.

I am not sure about this, can you elaborate further? I need to understand better the example you are proposing.

Maybe I am wrong.  My thinking is that something like subprocess.call(['conda', 'info']) will do whatever conda info means in the current environment.  I guess this will be okay as long as all that's being done is stuff like conda info (which in theory should be consistent across all environments).
What won't work is trying to use the nb_conda_env tab from a notebook that's started when no conda environment at all is active (since then conda won't be on the path at all).  But maybe that is reasonable, since after all the name of the package is nb_conda_kernels so it's not outrageous that it wouldn't run if your notebook isn't running in a conda environment.

What won't work is trying to use the nb_conda_env tab from a notebook that's started when no conda environment at all is active (since then conda won't be on the path at all). But maybe that is reasonable, since after all the name of the package is nb_conda_kernels so it's not outrageous that it wouldn't run if your notebook isn't running in a conda environment.

Yep, if you don't have conda installed things will fail... if it is installed, the executable lives in the root environment and it is accessible from every new env you create, then it will not fail (even if you don't have conda installed in the new environment, because, in fact, you can not install conda in new environments).


ImportError: No module named xml.parsers.expat

I'm using youtube-dl on my openwrt router when I run the command it says ImportError: No module named xml.parsers.expat
root@AngelBeats:/mnt/usb# youtube-dl -v
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/bin/youtube-dl/main.py", line 16, in
File "/usr/bin/youtube-dl/youtube_dl/init.py", line 16, in
File "/usr/bin/youtube-dl/youtube_dl/options.py", line 8, in
File "/usr/bin/youtube-dl/youtube_dl/downloader/init.py", line 3, in
File "/usr/bin/youtube-dl/youtube_dl/downloader/common.py", line 8, in
File "/usr/bin/youtube-dl/youtube_dl/compat.py", line 132, in
ImportError: No module named xml.parsers.expat

That python is missing xml support, you'll have to rebuild it or install one with the support enabled.
And don't run youtube-dl as root user, it's only needed in some setups for updating.


net: sockets: Fixes net_pkt leak in accept

There is a net_pkt leak in sockets.c which can happen when the tcp stack is under stress and reuses net_context.
The idea is that when zsock_accepted_cb() is called it installs callback zsock_received_cb which allows tcp stack to start putting net_pkts in the recv_q. However if the connection is closed by a peer when the connection is not yet accepted by the application, the reference count of the net_context is not increased, because zsock_accept_ctx() fails and returns -1 to the application.
Now that when tcp stack calls tcp_conn_unref(), the net_context has only reference count of 1, hence can be reused by another connection, without having the recv_q empty. And finally when the net_context is reused it just k_fifo_init() the recv_q leaking all the net_pkts in recv_q.
My solution simply increases the reference count earlier with instalment of the zsock_received_cb() callback. And consequently flushes recv_q and decrement reference count if zsock_accept_ctx() fails.

Compliance check script reports some issues
Error: See https://docs.zephyrproject.org/latest/contribute/#coding-style for more details.

-:11: WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#11: FILE: subsys/net/lib/sockets/sockets.c:336:
+		/* TCP context is effectively owned by both application
+		* and the stack: stack may detect that peer closed/aborted

- total: 0 errors, 1 warnings, 52 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES C99_COMMENT_TOLERANCE COMPLEX_MACRO CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME DT_SCHEMA_BINDING_PATCH DT_SPLIT_BINDING_PATCH ENOSYS FILE_PATH_CHANGES IS_ENABLED_CONFIG MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE NETWORKING_BLOCK_COMMENT_STYLE PREFER_KERNEL_TYPES PREFER_SECTION PRINTK_WITHOUT_KERN_LEVEL REPEATED_WORD SPDX_LICENSE_TAG SPLIT_STRING TRAILING_SEMICOLON UNDOCUMENTED_DT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.
Error: See https://docs.zephyrproject.org/latest/contribute/#commit-guidelines for more details

3: UC4 Line exceeds max length (150>75): "Fix net_pkt leak by increasing net_context the reference count earlier in the zsock_accepted_cb() with instalment of the zsock_received_cb() callback."
4: UC4 Line exceeds max length (103>75): "And consequently flushing recv_q and decrement net_context reference count if zsock_accept_ctx() fails."
Error: Process completed with exit code 1.

I am sorry about that coding style, the checkpath.pl does not work in my project under Windows.

Should this be backported to v2.7?


[Feature Request] Organize the converted files

Pull data from the *arr APIs and organize the converted files into TV Shows and Movies folders, and for TV shows, organize them into folders for each show. This makes re-importing into *arr much easier.
For example:
Convert Show S01E01 and put it into ConversionOutputDir/TV Shows/Show
Convert Movie (2021) and put it into ConversionOutputDir/Movies

Implemented in Beta v0.14. The series/movie file naming convention will follow the source file, but it will be placed into series/movie folders based on the "cleantitle" pulled by the API. I may add a setting to customize this a bit more later, but using cleantitle should allow the importer to grab it, as well as maintain compatibility across win/linux.


AAD Domain Services: add 09-01-2022 API version

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ X ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
October 2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
October 2022
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ X ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ X ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ X ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ X ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

@mentat9, Is the Swagger SDK Breaking Change check required?  Or is there a way to override it?

@mentat9, Is the Swagger SDK Breaking Change check required? Or is there a way to override it?

@ehunter1878 - I don't know: that's in the purview of the Azure SDK team who own this repo, including all the tooling, PR review, approval, and merge, etc. ARM reviewers only handle reviewing the REST layer for ARM-friendliness. The owner of your PR from the Azure SDK will always be the assignee (@leni-msft in this case).
Note: your PR is not on ARM reviewers' radar. Be sure to use the instructions in this comment to facilitate ARM review: https://github.com/Azure/azure-rest-api-specs/pull/20768#issuecomment-1258866732.

@tadelesh please help review the Go sdk breaking changes

@tadelesh please help review the Go sdk breaking changes

Approved for remove unreferenced types.

@ehunter1878 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree [company="{Microsoft}"]

@microsoft-github-policy-service agree [company="Microsoft"]

@ehunter1878 the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree

and
@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree company="Microsoft"

@microsoft-github-policy-service agree company="Microsoft"

@microsoft-github-policy-service agree

@mentat9, why is the license/cla agreement not getting resolved?  Am I not responding correctly?

@mentat9, why is the license/cla agreement not getting resolved? Am I not responding correctly?

@ehunter1878 - Sorry, I don't know about that: looks OK to me. @leni-msft, any idea?

@ehunter1878 Can you try again? I closed and reopened this PR, following the comments in https://github.com/microsoft/ContributorLicenseAgreement/issues/54

@microsoft-github-policy-service agree company="Microsoft"

@raych1 please help check the ModelValidation errors

@raych1 could you help with the ModelValidation errors?  I attempted a fix but that caused more problems.

/azp run


TYPO3-project in subfolder leads to wrong configuration path

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
Diagnostic shows no errors.
Expected Behavior
When setting up a TYPO3-project in a subfolder /src everything should run normal.
Actual Behavior
When setting up a project in a subfolder /src and defining docroot: src/public and composer_root: src in config.yaml there is an additional /src/src/public/typo3conf/AdditionalConfiguration.php generated.
Steps To Reproduce
Setting up a fresh&clean project with
ddev config --project-type=typo3 --docroot=src/public --create-docroot --composer-root=src
ddev composer create --no-install "typo3/cms-base-distribution:^11.5"
ddev composer install

Until now, everything is fine. But when doing a
ddev start

the additional /src/src/public/typo3conf/AdditionalConfiguration.php is generated.
Anything else?
Only TYPO3 v9-11 are affected. With v12 it seems to behave well.

Thanks for this! And thanks for the extra troubleshooting.
Do I understand correctly that this does not cause any operational problem when it appears in TYPO3 v9-11?

As long as there are no config changes which result in an adapted DDEV-controlled AdditionConfiguration.php there should be no problem. Until now I had no problemes in two projects.

I can confirm this problem using TYPO3 11 and a setup with docroot: src/projectname results in an AdditionalConfiguration.php being created at src/projectname/src/projectname.

@woemar that's the opposite of what @rtoenjes said...

i have the same problem with several projects with various php versions. the problem only occurs when composer_root is set.
@rfay what @woemar is the same problem, the composer_root gets somehow attached/used for the creation of the AdditionalConfiguration.php
ddev version v1.21.6

Can confirm the bug and it's only related to v11 or lower. Bugfix is on the way...


Year should be year of original publication

Great project! Thanks for this.
While looking through it I noticed that you've updated the copyright year to 2017 but actually, the aim of the year in a copyright notice is to indicate when the work was originally published, in case of any dispute. So I thought I'd raise a PR.
You're not in the UK but this, from the UK's copyright service is useful: http://www.copyrightservice.co.uk/copyright/p03_copyright_notices, item 4, and the rules will be the same in the US.

Actually, it should be a range (2015-2017), since each change is potentially significant enough to qualify for copyright.

👍

Per a SO answer:

if you made changes that year, do include the year in a comma-separated list in your copyright notice.
if did not make copyrightable changes that year, do not include that year in your copyright notice.

Seems fair.


Optimize initial requests to BE after #34

After the implementation of #34  the request to retrieve initial folder list or inbox are duplicate.

Logic has been changed:

Only when first login of a user (no IndededDB record exists) an initial request of folders and INBOX will be performed to the backend.
For all the other cases, when user logs in and application displays the folder list with a selected folder, the refresh poll function is launched.

This function will retrieve the updated list of folders and the updated cache of messages for the selected folder.


CoreFX

What does the "FX" stand for?

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

ID: aa071095-4eb6-20c1-ca26-f34da6fcbdf8
Version Independent ID: 2b2efd49-b0c8-9a13-525a-d04c802d2bef
Content: .NET Glossary
Content Source: docs/standard/glossary.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @gewarren
Microsoft Alias: gewarren

I believe it stands for "framework".
"CoreFX is the foundational class libraries for .NET Core. It includes types for collections, file systems, console, JSON, XML, async and many others."

Well, I thought so, too, but there is no "x" in Framework.
Since the stated goal of the glossary is to "clarify meanings of selected terms and acronyms that appear frequently in the .NET documentation", I think that this information (after it has been discovered) should be added here.

Sure, I can add that info to the glossary. I think that "x" is commonly used to shorten a word. For example, doctors write "Tx" as shorthand for treatment.
Here's another example of Fx being used to mean "framework": https://docs.microsoft.com/en-us/sharepoint/dev/spfx/sharepoint-framework-overview
@mairaw Can you officially state if the "Fx" in CoreFx means "framework"?

I think so. @karelz @danmosemsft can you confirm?

Yes, FX stands for Framework.

Yes, but we don't use the CoreFX term any more. We use "core .NET Libraries". It's OK to have it in a glossary as it comes up from time to time.

@danmosemsft Should we update the title and TOC name for this page then?
https://docs.microsoft.com/en-us/dotnet/core/compatibility/corefx

@gewarren good point, yes please (I hadn't seen this doc before). I suggest "Core .NET Libraries"


Can't add a nil AVCaptureInput

Crashes with following error
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[AVCaptureSession addInput:] Can't add a nil AVCaptureInput'

Solved by deploying to real device instead of emulator


cloudlist: 1.0.3 -> 1.0.4

Diff: https://github.com/projectdiscovery/cloudlist/compare/v1.0.3...v1.0.4
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release 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

[ ] Fits CONTRIBUTING.md.

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

1 package built:

cloudlist


3-ipscend img error

img at the top of https://ipfs.io/blog/3-ipscend/ is broken

Looks like an invalid url. Should be: https://ipfs.io/blog/3-ipscend/img/ipscend.png


🐛 Add validation of ei suported version

The source_version is validated before full executions.

Solves #16 . Thanks!


returning commands strings

Hello
I am looking for a way to handle MCcommands
I am designing a datapack generator in Python (documentation is not the best)
https://github.com/Loumardes/BlocCode
currently MCcommands are inserted as strings arguments in the scriptings functions
if your code allow to intuitively insert commands and return commands strings, i would like to use it

Actually, no, command generation is done via string concatenation, for example:
if len(order) == 1:
    return 'kill @s'
else:
    return 'kill ' + selector.upgrade(props['selector'])

Main goal of my project was to parse pre-1.13 commands and generate correct 1.13+ commands that would do the same thing. So, I have a parser but I haven't even tried to implement proper classes for command generation.

allright
have a look to :
https://github.com/BranislavBajuzik/1.12-to-1.13
also in my project I avoided Object Oriented Programming to try help users to check and inprove source code

I don't think that OOP is bad actually but it's your right to decide whether to use it or not.
Seems like the only thing that might be useful for your BlocCode is the commands.pre_1_13.nbtstr.types module as it has an implementation of NBT strings (not actual NBT though) used in commands.


error in lcvl_search()

Hey,
I am trying to standardise plant names and I am using the lcvc_search() function.
When I run the command I get this error:
'Error in if (!all(hybrids)) { : missing value where TRUE/FALSE needed'
my splist contains also non-species entries (e.g. bare ground or higher taxonomic information for unknown species).
When trying to run lcvc_group_search(splist, search_by = "Genus")  I get the same error.
Do you have any idea why I get this error?
Thank you in advance

Hi @paschatz, thanks for reporting the issue. Could you provide any reproducible example?


Make union/except/intersect chainable as SelectManager

Fixes #118.
This is the same as #239, but against master instead of 4-0-stable. It would be really nice to get this in so we don't need to keep freshening it! Or if there's a reason it isn't being merged, feedback would be great.

It is frustrating AREL doesn't support this, there are a lot of use cases for chaining union operators. It would be great if this was resolved.

@amalagaura: Yeah, I rely on this behavior and keep needing to update this branch to work with new versions. If the maintainers have no interest in merging this functionality, perhaps I should instead figure out a way to do it with a monkey-patch or a separate library.
@rafaelfranca / @tenderlove / other maintainers: I'm not sure if you've seen this, but a thumbs-up or thumbs-down on this functionality would be really nice, so that I have a sense for whether or not I'm wasting my time on it. If the functionality is desired, but you don't like the implementation, I'm willing to put together a different PR that speaks to that.

This seems like a no-brainer. I also have 3 Arel::SelectManagers that I need to merge and I am forced to do a .to_sql  and join(' UNION ').
Arel needs union to not return a Node::Binary and instead return a proper SelectManager, otherwise union really is broken since SQL definitely supports chainable unions.

I'd also like to see a better way to work with unions. Feedback on @jsanders  pull request please?

How has this been ignored a year? @tenderlove ??

I sincerely apologize for not having time for this.  Can you please rebase it and I will review.  I'm really sorry.

@tenderlove: No worries! I'll find some time to rebase and ping you again.

@jsanders Did you rebase? We'd love to have this. Thanks!

@tenderlove: Rebased this onto current master and things look pretty good. Feedback welcome!
Sorry for the delay, folks.

@tenderlove Waiting on your tender PR love. Thanks.

@tenderlove Any updates? I have another use case and don't want to use join(' OR ')

Hello. What about this?
Arel::Nodes::Union.new(select_manager_1, Arel::Nodes::Union.new(select_manager_2, select_manager_3))

This gives us "chained" unions. Some over-parenthesized but workable solution. We can generalize it a'lil:
def build_union(left, right)
  if right.length > 1
    Arel::Nodes::UnionAll.new(left, build_union(right[0], right[1..-1]))
  else
    Arel::Nodes::UnionAll.new(left, right[0])
  end
end

managers = [select_manager_1, select_manager_2, select_manager_3]
build_union(managers[0], managers[1..-1]).to_sql
# => ( (SELECT table1.* from table1)
#    UNION ALL
#    ( (SELECT table2.* from table2)
#    UNION ALL
#    (SELECT table3.* from table3) ) )

This works in Arel 4.0.2... No need to join lots of to_sql...

Per #523, Arel development is moving to rails/rails.
If this PR is still relevant, please consider reopening it over there.