Limit to 15 devices independent of #define ESPALEXA_MAXDEVICES

Hi, I have the problem, that, independent of the define ESPALEXA_MAXDEVICES, Alexa will only find 15.
See the debug serial output: [Nodemcu V1.0 - ESP8266], Lib version 2.4.7
Constructing device 1
Adding device 1
Constructing device 2
Adding device 2
Constructing device 3
Adding device 3
Constructing device 4
Adding device 4
Constructing device 5
Adding device 5
Constructing device 6
Adding device 6
Constructing device 7
Adding device 7
Constructing device 8
Adding device 8
Constructing device 9
Adding device 9
Constructing device 10
Adding device 10
Constructing device 11
Adding device 11
Constructing device 12
Adding device 12
Constructing device 13
Adding device 13
Constructing device 14
Adding device 14
Constructing device 15
Adding device 15
Constructing device 16
Adding device 16
Constructing device 17
Adding device 17
Constructing device 18
Adding device 18
Constructing device 19
Adding device 19
Constructing device 20
Adding device 20
Espalexa Begin...
MAXDEVICES 25
Done
....
Responding search req...
Responding to description.xml ...
Send setup.xmlNot-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights
Body:
AlexaApiCall
ok
l0
lAll
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991840
Body:
AlexaApiCall
ok
l180991840
0
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991841
Body:
AlexaApiCall
ok
l180991841
1
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991842
Body:
AlexaApiCall
ok
l180991842
2
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991843
Body:
AlexaApiCall
ok
l180991843
3
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991844
Body:
AlexaApiCall
ok
l180991844
4
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991845
Body:
AlexaApiCall
ok
l180991845
5
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991846
Body:
AlexaApiCall
ok
l180991846
6
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991847
Body:
AlexaApiCall
ok
l180991847
7
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991848
Body:
AlexaApiCall
ok
l180991848
8
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991849
Body:
AlexaApiCall
ok
l180991849
9
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991850
Body:
AlexaApiCall
ok
l180991850
10
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991851
Body:
AlexaApiCall
ok
l180991851
11
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991852
Body:
AlexaApiCall
ok
l180991852
12
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991853
Body:
AlexaApiCall
ok
l180991853
13
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991854
Body:
AlexaApiCall
ok
l180991854
14
Not-Found HTTP call:
URI: /api/2WLEDHardQrI3WHYTHoMcXHgEspsM8ZZRpSKtBQr/lights/180991855
Body:
AlexaApiCall
ok
l180991855
15

Update: With Lib v 2.4.2 Alexa finds all devices, however, always reports "Device does not respond". However it turns lights on and off. With Lib v. 2.4.6 and 2.4.7 it only finds 15 devices, also switches these without an error.


[NP-1958] Fix wizardlet create args

Danny and I were able to produce an error by adding sections to the data model of a wizardlet with a createView override. We found out this was being passed as the context in wizardlet.cls_.create() instead of this.__subContext__.

You can pass either an explicit context or an object as the second argument to create. If you pass an object, it takes its subContext, so the two versions should be identical.
I would rather get to the root cause of this bug than making a unknown change that works for some unknown reason, that if someone sees, they might be tempted to revert.

Are you sure that this fixes the problem. What 'of' is being used?

This fix is no longer needed


Fix response body type for parse: none

Fixes # 62
Fix response body type for parse: none to be Buffer
Add parse: string to explicitly ask for body as a string
Add types for string parsing
Add tests for Buffer and string bodies
Regenerate documentation

Thank you very much.


Forms

Hello, In this feature branch I,
Highlights,

[ ] Created a method current_user in the ApplicationController to make current user data available to all controllers

Return the first user from the database,( Don't worry about proper authentication at this time. You will add it in the upcoming projects.)

[ ] Created forms to handle

create new post for current user.
create comment for current user

@KayLemba thank you for the review


CVE-2023-5217 (libvpx heap buffer overflow) tracking

CVE-2023-5217 is a heap buffer overflow in libvpx's VP8 encoder, as many things such as electron and more are being tracked in this issue, so we can fix them in nixpkgs.
This vulnerability is yet to be rated, but we can assume (as its a heap buffer overflow), that it might be a big deal.
Current status
We currently have patched the libvpx package with #257941, vendored dependencies are tracked below.
How to help

Review/merge any nixpkgs PR that you see pending in the task list below.
If you have an idea of how to fix/address the vulnerability in any of the packages listed in the task list below, don't hesitate to post a comment here and send pull requests! Feel free to cc me on PRs so I can make sure they're tracked and they don't get lost.

Task list

[X] libvpx is updated in master #257941

[X] libvpx is updated in release-23.05 #257960

[ ] electron needs to be updated to 26.2.4

[ ] backport fix to electron_25 (update to 25.8.4)
[ ] backport fix to electron_24 (update to 24.8.5)
[ ] backport fix to electron_22 (update to 22.3.25)

[ ] Vendoring

[ ] High risk stuff (mail clients, IM clients, web browsers)

[ ] signal-desktop needs to be updated to 6.32

[ ] signal-desktop-beta needs to be updated to 6.32

[ ] rocketchat-desktop
[ ] armcord
[ ] indigenous-desktop (homepage URL leads to 404, appears to be a chat app)
[ ] mattermost-desktop
[ ] threema-desktop
[ ] thedesk
[ ] gitter
[ ] aether (upstream hasnt had a commit since 2021, TODO mark/remove?)
[ ] caprine-bin
[ ] wire-desktop
[ ] keybase-gui
[ ] mailspring
[ ] brave needs to be updated to 1.58.135 #258060
[ ] tor-browser
[ ] mullvad-browser

[ ] Other stuff

[ ] octant-desktop #258061 (now archived, TODO mark/remove?)
[ ] figma-linux
[ ] insomnia
[ ] freeswitch
[ ] losselesscut-bin
[ ] popcorntime
[ ] snapmaker-luban
[ ] keeweb
[ ] zotero
[ ] polar-bookshelf (homepage link broken)
[ ] join-desktop
[ ] passky-desktop
[ ] isabelle
[ ] onlyoffice
[ ] jellyseer
[ ] yesplaymusic
[ ] obs-studio
[ ] atom
[ ] pulsar
[ ] simplenote (no upstream commits since 2021, TODO mark/remove?)
[ ] joplin-desktop
[ ] cypress (upstream has had a fix but hasn't version bumped yet)
[ ] premid
[ ] tidal-hifi
[ ] radicle-upstream
[ ] Jetbrains applications

[ ] mps
[ ] pycharm-community
[ ] idea-community

[ ] Libraries

[ ] qtwebengine
[ ] libtoxcore
[ ] termcolor
[ ] libaom
[ ] nwjs
[ ] jetbrains-jdk-jcef
[ ] python310Packages.get-video-properties #257947

This task list may or may not be complete, if you think we are missing something, please feel free to cc me! Here is the scan ran by @delroth for vulnerable dependencices

@Kiwi: simplenote, binary provenance, package seems obsolete

@maxhille: aether, binary provenance, package seems obsolete

@06kellyjac: octant-desktop, binary provenance, upstream is archived

@noneucat: polar-bookshelf, binary provenance, package seems obsolete

@WolfangAukang: indigenous-desktop, binary provenance, package seems obsolete

@Mic92, @equirosa, @urandom2: signal-desktop-beta, binary provenance, update available, priority is high

@WolfangAukang: indigenous-desktop, binary provenance, package seems obsolete

Not obsolete, they changed the repo name. I can build it from source, but the problem is that it does not work with an Electron version superior to 19.

Regarding threema-desktop (I'm the maintainer), there was a similar issue (https://github.com/NixOS/nixpkgs/issues/254798) that asked to remove lib/threema/threema-web, but as this package is using the bundled electron, it wasn't necessary to keep tracking.
Still I have this PR open that removes that component https://github.com/NixOS/nixpkgs/pull/255899

Brave is being updated in this PR #258060

Regarding thedesk (I'm the maintainer), this is a similar case as threema-desktop: application is using bundled electron version but the package contains the binary from the deb file.

@travisbhartwell, @manveru, @prusnak: could we get electron fixes underway?

@WolfangAukang: indigenous-desktop, binary provenance, package seems obsolete

Not obsolete, they changed the repo name. I can build it from source, but the problem is that it does not work with an Electron version superior to 19.

Then it should be marked knownVulnerable, because they use an end-of-life and vulnerable electron version. Please make that happen if you are opposed ti removing it.

@WolfangAukang: indigenous-desktop, binary provenance, package seems obsolete

Not obsolete, they changed the repo name. I can build it from source, but the problem is that it does not work with an Electron version superior to 19.

Then it should be marked knownVulnerable, because they use an end-of-life and vulnerable electron version. Please make that happen if you are opposed to removing it.

You mean knownVulnerabilities? Not able to find any examples with knownVulnerable.

New logs starting up by @delroth, should be more usable for reading seashell link

https://github.com/NixOS/nixpkgs/pull/258116
https://github.com/NixOS/nixpkgs/pull/258137
https://github.com/NixOS/nixpkgs/pull/258135
https://github.com/NixOS/nixpkgs/pull/258138

https://github.com/NixOS/nixpkgs/pull/258155

In addition to the VP8 CVE above, there's now a VP9 CVE https://github.com/NixOS/nixpkgs/pull/258295  so maybe updating everything will need to be redone again?

In addition to the VP8 CVE above, there's now a VP9 CVE #258295 so maybe updating everything will need to be redone again?

We are talking about what to do with the new CVE on the matrix room, we are going to use the same tracking issue, probably

Maybe the solution for mailspring and archord should be the same as this #258217, it should be noted that there is some effort to update the electron version of mailspring, though.

We should take this opportunity to drop atom. https://atom.io redirects to https://github.blog/2022-06-08-sunsetting-atom/ which explicitly states No more security updates and the repo is archived since last November.
#258440

As for gitter then it has been removed by #255784 as part of the another CVE.

Looks like Brave didn't get auto-backport properly. I can do a PR in the next day or two unless someone feels like grabbing it sooner.

One mass-ping for everyone:
mattermost-desktop: @jokogr
caprine-bin: @ShamrockLee
wire-desktop: @arianvp, @Kiwi, @toonn
keybase-gui: @Avaq, @rvolosatovs, @puffnfresh, @np, @Br1ght0ne, @kf5grd
figma-linux: @ercao, @kashw2
insomnia: @markus1189, @babariviere, @kashw2
freeswitch: drop, no maintainers (cc @mweinelt)
popcorntime: @onny
snapmaker-luban: @simonkampe
keeweb: @sikmir
zotero: @i077
polar-bookshelf: @noneucat
join-desktop: @SuperSandro2000
passky-desktop: @akkesm
isabelle: @jwiegley, @jvanbruegge
onlyoffice: @nh2, @GTrunSec
jellyseer: @camillemndn
yesplaymusic: @LostAttractor
obs-studio: @jb55, @MP2E, @deviant, @materusPL
pulsar: @COLAMAroro
simplenote: @Kiwi
cypress: @thorstenweber83, @mmahut, @Craftzman7
premid: @natto1784
tidal-hifi: @qbit, @spikespaz
radicle-upstream: @d-xo
jetbrains.mps: @rasendubi
jetbrains.pycharm-community: @GenericNerdyUsername, @tymscar
jetbrains.idea-community: @edwtjo, @gytis-ivaskevicius, @steinybot, @AnatolyPopov, @tymscar
qtwebengine: @milahu, @NickCao

libtoxcore: @peterhoeg, @ehmry
termcolor: @prusnak
libaom: @primeos, @kiloreux, @dali99
nwjs: @offlinehacker
jetbrains.jdk: @edwtjo
thats all! please update/drop your packages maintainers!

A fair warning concerning Pulsar
I didn't get the time to do it yet, but I will probably soon drop my maintainer status of Pulsar (due to lack of time, and some personal reasons). I gave every piece of documentation I could upstream so that someone else there can take over.
Beyond that, I doubt there will be an update to a newer Electron in the near future (because backporting the Atom codebase to a newer electron is a pain). "Fortunately", I think there is a low risk for this CVE on Pulsar. It's not as widespread as Atom once was, printing arbitrary HTML isn't quite straightforward, and probably will require a plugin, which can already run arbitrary JavaScript.
Pinging @mauricioszabo who has been leading this Electron version bump. If comes a patch for this CVE and nobody can take the maintainership, I'll see if I can find some time to update for nixpkgs.

dropping join-desktop in #259067

@COLAMAroro I am glad to say that you're wrong about the update to a newer Electron :).
Currently, I am using - even professionally - Pulsar under Electron 25 (can't bump to Electron 26 because it crashes on my Linux machine). In fact, it's been working so well that we're about to start a "next-gen" or maybe "beta" channel for it, so people that want to check if their packages run in this new version of Pulsar can actually try it.

As far as I can tell (and I have asked for clarification in the security room - but not yet received an answer), libaom does not vendor libvpx.
It is however a google media codec, and has inherited big parts of libvpx. I've tried finding similar code as what's been patched, but haven't found anything

Is there any place to see why individual packages were flagged? All of the seashell links are broken, presumably expired. I'm a maintainer for obs-studio, and can't find them vendoring libvpx— rather it's used via libavcodec, from ffmpeg, which pulls in libvpx from Nixpkgs.

IIRC obs-studio vendors a libcef.so which vendors libvpx.

@delroth Thanks for the pointer! It's actually vendoring files from Nixpkgs' libcef [pname is "cef-binary"] package, which should probably be on the list instead.

@delroth Thanks for the pointer! It's actually vendoring files from Nixpkgs' libcef [pname is "cef-binary"] package, which should probably be on the list instead.

Yeah, I'm not sure why it's not on the list, the automated tooling should have detected it too. I'll let @xyzeva double check this.
OTOH if this is really just vendored without any modifications it should probably be symlinked instead. Not only would that help us with security tracking in cases like this, but it's a 300MB binary duplicated for (likely?) no good reason.

Yeah, I'm not sure why it's not on the list,

I've just checked and the libcef package is already recent enough to be based on a version of Chromium that isn't vulnerable. So it's not actually an issue— meaning obs-studio should be safe. This issue was created prior to libcef being updated, so it wasn't a false-positive, just no longer valid.

OTOH if this is really just vendored without any modifications it should probably be symlinked instead. Not only would that help us with security tracking in cases like this, but it's a 300MB binary duplicated for (likely?) no good reason.

Good idea! I'm not sure why exactly, appears to be a copy+paste job. I'll create an issue for it.

I just updated everything according to the replies & new open PRs, thanks everyone for collaborating and fixing up my mistakes 😅

Freeswitch has implemented a fix for the CVE here: signalwire/freeswitch/pull/2259, but haven't version bumped yet.

I am not entirely sure, but isabelle might be fixed with the new version: https://github.com/NixOS/nixpkgs/pull/243497

Pale Moon's UXP / XUL bundles libvpx and upstream says they were affected, https://github.com/NixOS/nixpkgs/pull/259204 bumps palemoon-bin to a fixed version.

I created an issue at the upstream project of caprine-bin (sindresorhus/caprine#2081).

Zotero 6 is not affected by this according to the dev mailing list.

I'm a bit doubtful.  The list above was created by some scanning of the binary, right?  AFAIK zotero is based on Firefox/XUL, and default XUL binary does bundle libvpx inside...

I think it is based on XUL, but I think the version of Firefox that comes from is very old. In the JS console in version 6.0.26, Services.appinfo.platformVersion evaluates to "60.9.0", but running the same in the browser console of the most recent release of Firefox evaluates to "118.0.1". Firefox 60.9.0 was released in September 2019. I don't know exactly how the scan was done, as the link to the scan results in the OP is broken.
I can contact their security email or post in the forums about this, but I'm not really sure what the best course of action here is in the meantime. There is Zotero 7 in beta, which patched this vulnerability in a recent build.

qtwebengine uses system libvpx thus should not be affected.

I have applied a patch for the CVE in freeswitch in this pr: #259881

Suggesting to remove simplenote in: #259889

I looked at mattermost-desktop, and it doesn't use the shipped Electron version, rather it uses the Electron_26 from nixpkgs. Thus, if I haven't missed anything, it should be resolved, right?
See the line here. 😃

Also should mullvad-vpn not be on the list? It was also found by the vendoring checker https://clbin.com/haJzA. It has an upstream release bundling a new Electron version.

@xyzeva aether is now removed from master by https://github.com/NixOS/nixpkgs/pull/258556

For tidal-hifi there is a push to get it building from source: https://github.com/NixOS/nixpkgs/pull/252307 - I am not versed enough to be able to help.. maybe someone else can?

Mark figma-linux as vulnerable to this CVE in #261404 until fix is available.
Maintainer will update on next release see: https://github.com/Figma-Linux/figma-linux/issues/341#issuecomment-1764332718

I suggest marking pulsar, a vulnerable in #262376 because it uses a very outdated version of electron and doesn't seem to plan to bundle a new version anytime soon.

Update pycharm here: #262418

Have opened a draft pr fixing nwjs #258048, but I still need to do some more testing to ensure nothing breaks.

The current stable version of Zotero, based on Firefox 60 as discussed earlier here, is not going to get patched from what I can tell, so #262808 marks it as vulnerable until Zotero 7 hits GA.

The current stable version of Zotero, based on Firefox 60 as discussed earlier here, is not going to get patched from what I can tell, so #262808 marks it as vulnerable until Zotero 7 hits GA.

Hi, as much as I would love to see Zotero 7 come to GA, it seems Zotero 6 has been patched: "Changes in 6.0.30 (November 2, 2023), [Security] Disabled libvpx support (CVE-2023-5217)" (https://www.zotero.org/support/changelog).

The current stable version of Zotero, based on Firefox 60 as discussed earlier here, is not going to get patched from what I can tell, so #262808 marks it as vulnerable until Zotero 7 hits GA.

Hi, as much as I would love to see Zotero 7 come to GA, it seems Zotero 6 has been patched: "Changes in 6.0.30 (November 2, 2023), [Security] Disabled libvpx support (CVE-2023-5217)" (https://www.zotero.org/support/changelog).

Hi, for those interested, I managed to package Zotero 7 built from source in https://github.com/camillemndn/nixos-config/tree/main/pkgs/applications/office/zotero, but I could not upstream it since it is based on Firefox 102 which is already deprecated...

Opened https://github.com/NixOS/nixpkgs/pull/266033 to update Zotero 6

Hi, for those interested, I managed to package Zotero 7 built from source in https://github.com/camillemndn/nixos-config/tree/main/pkgs/applications/office/zotero, but I could not upstream it since it is based on Firefox 102 which is already deprecated...

Good job building Zotero 7 from source. Zotero seems like a Heath Robinson machine and the only way to distribute is is through its prebuilt which is what the Zotero 6 nixos package is doing I understand. I don't know why they need to vendor Firefox but that's another story.


1.2.8 breaks multiplayer?

Latest version (1.2.8?) breaks Multiplayer.
An error has occurred! Game version <IP_ADDRESS>46
Mods used: [IlLine1.0.0] [LDBTool2.0.6] [DSP Mod Save1.1.4] [DSP Common API<IP_ADDRESS>] [NebulaMultiplayerMod<IP_ADDRESS>] [NebulaMultiplayerModApi<IP_ADDRESS>] [BulletTime1.2.8] [Common API Nebula Compatibility<IP_ADDRESS>]
[Nebula Error] Nebula Multiplayer Mod is incompatible
Unhandled exception occurred while patching the game.

Also made an issue on Nebula Multiplayer:
https://github.com/hubastard/nebula/issues/580

It's the game 0.9.27 that breaks Nebula Multiplayer 0.8.10.
In case someone still wants to play in the old version, I make BulletTime v1.2.9 compatible with game 0.9.26.


WIP (Not ready): STYLE: Replace once_flag of ThreadPoolGlobals with static local variable

A static local bool variable is more lightweight than a std::once_flag data member, while its (static) initialization just as thread-safe as std::call_once.

Wow, segfaults and timeouts!
https://open.cdash.org/test/1187815471 says:

Test: PythonGetNameOfClass (Failed)
Build: Linux-Build8919-PR4171-Replace-ThreadPoolGlobals-ThreadPoolOnceFlag-Python (Azure.fv-az71-143)
...
Loading ITKVTK... done
itkTestDriver: Process exception: Segmentation fault

https://open.cdash.org/test/1187815689 says:

Test: PythonTemplateTest (Failed)
Build: Linux-Build8919-PR4171-Replace-ThreadPoolGlobals-ThreadPoolOnceFlag-Python (Azure.fv-az71-143) on 2023-08-23 16:04:40
Repository revision: 2e8ad9f6533ea6ea6063737c19250595af40a0fe
Test Details: Completed (Timeout)

https://open.cdash.org/test/1187868304 says:

Test: PythonExtrasTest (Failed)
Build: Darwin-Build8992-PR4171-Replace-ThreadPoolGlobals-ThreadPoolOnceFlag-Python (Azure.Mac-1692806138896) on 2023-08-23 16:02:47
Repository revision: 2e8ad9f6533ea6ea6063737c19250595af40a0fe
Test Details: Completed (Failed)
...
Traceback (most recent call last):
File "/Users/runner/work/1/s/Wrapping/Generators/Python/Tests/extras.py", line 328, in
itk.transformwrite(transforms, sys.argv[7], compression=True)
File "/Users/runner/work/1/s-build/Wrapping/Generators/Python/itk/support/extras.py", line 1446, in transformwrite
writer.Update()
RuntimeError: thread::join failed: Invalid argument
libc++abi: terminating with uncaught exception of type std::__1::system_error: thread::join failed: Invalid argument
itkTestDriver: Process exception: Subprocess aborted

To be continued...

Does anyone have an explanation about what goes wrong here, on both ITK.Linux.Python and ITK.macOS.Python? Is there a fundamental reason why a static local variable initialization in an ITK core CXX file might not be thread-safe, when doing a Python wrapping? And if so, why would std::call_once then be any better?

std::call_once might have some special treatment, while an ordinary local static variable does not.

Static variables get constructed at initialization time, while std::call_once gets executed at run time.

Static variables get constructed at initialization time, while std::call_once gets executed at run time.

This introduces a race condition with other global static variable and their initializations. The itk::ObjectFactory depends on static initialization so it cannot be used during static initialization, so ITK objects derived from itk::Object can not be used during static initialization.

Do ITK.Linux.Python and ITK.macOS.Python possibly load the ITKCommon lib multiple times concurrently, getting multiple instances of ITKCommon at the same time?

"Race condition" is not precise. It is more of a dependence on the order of the static initialization across the modules that is problematic and unspecified.

Sorry, I still don't get it! Looking at the code of this PR, the static local variable only gets initialized when ThreadPool::GetInstance() is called for the very first time. Which ensures that the lambda code only gets executed when ThreadPool::GetInstance() is called for the very first time. From C++11, static local variable initialization is officially thread-safe. So this should do exactly the same as the original std::call_once call: (It should do the same, but it does not.)
https://github.com/InsightSoftwareConsortium/ITK/blob/2e8ad9f6533ea6ea6063737c19250595af40a0fe/Modules/Core/Common/src/itkThreadPool.cxx#L69-L89

Maybe compiler does some special optimizations for local static variables, which it doesn't do for std::call_once?

Maybe compiler does some special optimizations for local static variables, which it doesn't do for std::call_once?

AFAIK, it is possible that compiler with eg -O3 can optimize away unused variable and doesn't run the lambda at all.
test with -03
void f()
{
    [[maybe_unused]] static bool x = []{ return true; }();
}

int main()
{
    f();
    return 0;
}

f():
        ret
main:
        xor     eax, eax
        ret

P.S. I fact, it does not prove what exactly happens in the PR. Maybe something else.

the static local variable only gets initialized when ThreadPool::GetInstance() is called for the very first time. Which ensures that the lambda code only gets executed when ThreadPool::GetInstance() is called for the very first time.

Small demonstration that the above statement is correct
#include <iostream>
#include <chrono>
#include <thread>

void f()
{
	static unsigned long long t0 = []()
	{
		return
			std::chrono::duration_cast<std::chrono::milliseconds>
				(std::chrono::system_clock::now().time_since_epoch()).count();
	}();
	std::cout << t0 << std::endl;
}

int main(int, char **)
{
	std::cout <<
		std::chrono::duration_cast<std::chrono::milliseconds>
			(std::chrono::system_clock::now().time_since_epoch()).count()
		<< std::endl;

	std::this_thread::sleep_for(std::chrono::milliseconds(3333));

	f();

	std::this_thread::sleep_for(std::chrono::milliseconds(1111));

	f();

	std::this_thread::sleep_for(std::chrono::milliseconds(1111));

	f();

	return 0;
}

r@deb2:~$ g++ s.cpp
r@deb2:~$ ./a.out
1692981873633
1692981876967
1692981876967
1692981876967

Thanks for having a look, @issakomi !

maybe it would be better to leave the code as it is (once_flag), the reward is not high, IMHO?

This particular PR is just a style improvement. If it works, I think a static local variable is preferable to a public once_flag member in a global (file scope) data structure and a call_once call. But only if it works, of course  😺
Moreover, I think it's worth knowing in general whether we can safely assume that static local variable initialization is thread-safe. And if so, how to use it properly!

AFAIK, it is possible that compiler with eg -O3 can optimize away unused variable and doesn't run the lambda at all.

The compiler is only allowed to do optimizations that do not change the run-time behavior. (According to the as-if rule.)  So if the lambda doesn't do anything observable, it may indeed be optimized away. But if the lambda has an observable side effect, it cannot be optimized away, e.g. (https://godbolt.org/z/7PPKe3Er4):
[[maybe_unused]] static bool x = []{ std::cout << 42; return true; }();`

the value is assigned (maybe re-initialized is correct term?) at the first run of the function.

Initialization is the correct term 😃  If a static local variable is initialized by a user-defined constructor, you may see that this constructor is called only once, by putting a print statement inside the constructor.
But as both of you (@dzenanz @issakomi )  mention compiler optimization as a possible cause of the CI failures, I can still try to see what happens of optimization is suppressed by adding a volatile keyword. Please wait...

But if the lambda has an observable side effect, it cannot be optimized away, e.g. (https://godbolt.org/z/7PPKe3Er4)

I run the above test with and without -O3 here is a lot of difference, specially with lambda. The f() in your example is not reduced to just 'ret', but reduced to printing '42'. Strictly, there is no lambda (s. assembler).

If it works, I think a static local variable is preferable to a public once_flag

IMHO, I still think it is a little bit a hack, to replace call_once with  initialization of a unused local static variable, for the compiler it may be not clear what do we want.

optimization is suppressed by adding a volatile keyword.

Maybe my 1st example is oversimplified, but volatile doesn't change anything
test
AFAIK, volatile prevents some optimization steps, related to read / write.

IMHO, I still think it is a little bit a hack, to replace call_once with  initialization of a unused local static variable

If you really want to call call_once, I think that at least in theory, it might be preferable to keep the once_flag as a local variable, instead of having it as a public data member of globally shared data:
ThreadPool::Pointer
ThreadPool::GetInstance()
{
  itkInitGlobalsMacro(PimplGlobals);

  static std::once_flag localOnceFlag{};

  std::call_once(localOnceFlag, []() {
    m_PimplGlobals->m_ThreadPoolInstance = ObjectFactory<Self>::Create();
    // ... some more things here...
  });

  return m_PimplGlobals->m_ThreadPoolInstance;
}

Of course, it would then still depend on proper (thread-safe) support of static local variables...  🤔

instead of having it as a public data member of globally shared data

Maybe make it private? In fact, I am still not 100% sure that the problem is the optimization, seems to be possible, but maybe something else.

tried, it works (with friend class ThreadPool;).

Could you then make all data members of PimplGlobals private? Would that compile? If so, please feel free to make it a pull request  😃

This (everything is private, except the ctor) works:
struct ThreadPoolGlobals
{
  ThreadPoolGlobals() = default;

  friend class ThreadPool;

private:

  // To lock on the various internal variables.
  std::mutex m_Mutex;

  // To allow singleton creation of ThreadPool.
  std::once_flag m_ThreadPoolOnceFlag;

  // The singleton instance of ThreadPool.
  ThreadPool::Pointer m_ThreadPoolInstance;

#if defined(_WIN32) && defined(ITKCommon_EXPORTS)
  // ThreadPool's destructor is called during DllMain's DLL_PROCESS_DETACH.
  // Because ITKCommon-5.X.dll is usually being detached due to process termination,
  // lpvReserved is non-NULL meaning that "all threads in the process
  // except the current thread either have exited already or have been
  // explicitly terminated by a call to the ExitProcess function".
  // Therefore we must not wait for the condition_variable.
  std::atomic<bool> m_WaitForThreads{ false };
#else // In a static library, we have to wait.
  std::atomic<bool> m_WaitForThreads{ true };
#endif
};

The ctor has to be public, otherwise there is the error:
In file included from /home/r/itk/ITK/Modules/Core/Common/src/itkThreadPool.cxx:25:
/home/r/itk/ITK/Modules/Core/Common/include/itkSingleton.h: In instantiation of ‘T* itk::Singleton(const char*, std::function<void(void*)>, std::function<void()>) [with T = ThreadPoolGlobals]’:
/home/r/itk/ITK/Modules/Core/Common/src/itkThreadPool.cxx:66:1:   required from here
/home/r/itk/ITK/Modules/Core/Common/include/itkSingleton.h:121:16: error: ‘constexpr itk::ThreadPoolGlobals::ThreadPoolGlobals()’ is private within this context
  121 |     instance = new T;
      |                ^~~~~
/home/r/itk/ITK/Modules/Core/Common/src/itkThreadPool.cxx:42:3: note: declared private here
   42 |   ThreadPoolGlobals() = default;

I didn't try Python wrapping and test, probable they should work, the ThreadPoolGlobals struct in in itkThreadPool.cxx file, nobody else see it.


log_plot: Support path to custom template

          > Yes, do you feel the link doesn't explain/imply it?

It wasn't clear to me either way -- I genuinely didn't know. I don't think we need to get into it in this PR. It was more of a product question that didn't occur to me until docs review.

Actually, I think custom templates might be broken as the user would need to pass the path relative to dvclive/dvc.yaml in order to work

So in DVCLive, should we do this?

Adjust the path to be relative to dvclive/dvc.yaml
Check that the template exists -- what should we do if not?

Originally posted by @dberenbaum in https://github.com/iterative/dvc.org/pull/4497#discussion_r1181602414

@dberenbaum I took a look into this and it just works:
https://github.com/iterative/dvclive/issues/569#issuecomment-1549406286
However, I think it is a problem as it means that DVC is not resolving the custom templates with respect to the dvc.yaml (at least for top level plots, didn't test stage level)

So it works but shouldn't work basically? So do I have this right?

dvclive saves it relative the working dir instead of relative to dvclive/dvc.yaml.
dvc resolves it from the repo root or working dir instead of from dvclive/dvc.yaml.


updated ppp to less than 10 pages

Pull Request Test Coverage Report for Build 1076

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.827%

Totals

Change from base Build 1066:
0.0%

Covered Lines:
3458

Relevant Lines:
4226

💛  - Coveralls


Esc Widget Not Working

Describe the bug
The Esc widget has not been working for a while. I've tried quitting and reloading Pock but the issue persists.
How to reproduce
N/A
Expected behavior
For instance, if I accidentally hit the screen capture shortcut, I would then hit the Esc on my touch bar and expect the screen capture to go away.
Screenshots
N/A
Crashlog
N/A
Versions:

macOS version: 14.0
Pock version:  0.9.0-22

Additional context
N/A

I had the same situation and fixed by checking the permissions: System Settings > Security & Privacy > Accessibility > Make sure Pock is checked.


MM-26959: display DMs and GMs in 'in:@' search suggestions

Summary

Display DMs and GMs in in:@ search suggestions.

Ticket Link

https://mattermost.atlassian.net/browse/MM-26959
Parent ticket: https://mattermost.atlassian.net/browse/MM-15477

Screenshots

racerx matches because that user's nickname is alice.

/update-branch

/update-branch


Toc numbering is weird when using defaults: numbered: true

When I use
defaults:
  numbered: true

globally in _toc.yml, some of the files are not numbered. And this changes with the number of times I run the build.
The screenshot is when I ran the build the first time:

The warnings I got during the build:

duplicate of #45


refactor: run backwards-compatibility tests in parallel

Closes #4206
Runs backwards-compatibility tests in parallel using parallel. This reduces the time to run backwards-compatibility in CI from ~21 minutes down to ~15 minutes and I'd expect the time savings to be material for the full backwards-compatibility matrix.
This requires refactoring how backwards-compatibility tests call test-ci-all.sh, where we now pass as arguments the versions to run each bash test function. The existing behavior of test-ci-all.sh is preserved by running as current.
For further improvements, we can consider merging the backwards-compatibility CI workflow with the build on linux test step. I've tested merging locally, which adds ~5 minutes to the build on linux tests step, so I'd prefer to keep backwards-compatibility separate for now.

https://github.com/fedimint/fedimint/pull/4418 some follow-ups.


Speck: added basic benchmarking.

I added this when I noticed that decrypt performance was significantly slower than encrypt performance.

I am currently getting these results on my laptop (CPU: Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz, 16383212 kB DDR4 @ 2133 MHz):
test decrypt      ... bench:          32 ns/iter (+/- 1)
test encrypt      ... bench:          25 ns/iter (+/- 1)
test generate_key ... bench:          36 ns/iter (+/- 1)

Thanks!


crc32 update adaptation from the author, mostly about

fixing inline assembly.

thanks!

What's your canonical source for this file? I see he seems to have two online, with different versioning and one updated more recently than this.
I feel like this is from the brotli source code but  I can't remember offhand.

It is ; but the changes are from the author himself.

Released in 1.6.21


feat: disable signup

Checklist

[X] No breaking changes
[X] Tests pass
[X] New features have new tests
[ ] Documentation is updated

can you update docs/environment_variables?

I've added AUTH_DISABLE_NEW_USERS to docs/environment-variables.md. Should I add a small description of how to use it to docs/configuration.md?


[Bug] Copy-DbaSsisCatalog - Could not load file or assembly Microsoft.SqlServer.IntegrationServices.Common.ObjectModel, Version=15

PowerShell version:
Name                           Value

PSVersion                      5.1.14409.1018
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1018
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
dbatools Module version:
Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.54\dbatools.psd1
Version : 1.0.54
Microsoft SQL Server 2016 (SP2) (KB4052908) - 13.0.5026.0 (X64)   Mar 18 2018 09:11:49   Copyright (c) Microsoft Corporation  Developer Edition (64-bit) on Windows 8.1 Pro 6.3  (Build 9600: )
Report
Using Copy-DbaSsisCatalog -Folder seems to look for version 15 of SSIS instead of grabbing any available version.
Host used

[X] powershell.exe
[ ] ISE
[X] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[X] Still buggy in powershell.exe (exactly the same)

Errors Received
writeErrorStream      : True
PSMessageDetails      :
Exception             : System.Exception: Could not load file or assembly
'Microsoft.SqlServer.IntegrationServices.Common.ObjectModel, Version=<IP_ADDRESS>,
Culture=neutral, PublicKeyToken=89845dcd8080cc91' or one of its dependencies. The system
cannot find the file specified. ---> System.Management.Automation.MethodInvocationException:
Exception calling "Create" with "0" argument(s): "Could not load file or assembly
'Microsoft.SqlServer.IntegrationServices.Common.ObjectModel, Version=<IP_ADDRESS>,
Culture=neutral, PublicKeyToken=89845dcd8080cc91' or one of its dependencies.
Steps to Reproduce
Try and use Copy-DbaSsisCatalog with the folder option. Don't have version 15 of Microsoft.SqlServer.IntegrationServices.Common.ObjectModel available (have an older version like 13.0 or 14.0, in my case 2016)
Copy-DbaSsisCatalog -Source $source -Destination $dest -Folder $folder
Expected Behavior
expected $folder` to appear on $dest.
Actual Behavior
Error: Could not load file or assembly

Thank you for the report. This command is problematic and I don't know how to fix it. it's likely that i'll remove the command soon unless someone who knows about DLLs can fix the issue.

Yeah, I am not sure either. I tried LoadWithPartialName (deprecated) to grab the available dll and that got me a bit further but then it still fails trying to grab the latest one. :(

what we coullllllllllld do is

Remove DLL and references
Require person to have SSIS installed locally
Use code in Get-DbaManagementObject to help determine the latest version, and load that
If that does not load, fail gracefully

I've been battling with this on and off for a week or so. What seemed to be the case was that the assembly MS.SQL.IS.Common.ObjectModel.dll is not included in the smo folder (mine only had two SSIS assemblies)
So I went into the SSMS install directory and copied all three of the assemblies (so they were the same version) into the smo directory and suddenly the command started working.

Also having SSIS installed locally did not help, I think the assemblies you need are shipped with ssms

Encountered the same issue trying to synchronize SSIS catalog between two SQL Server 2019 instances. Adding / replacing Microsoft.SqlServer.IntegrationServices.Common.ObjectModel.dll, Microsoft.SqlServer.Management.IntegrationServices.dll and Microsoft.SqlServer.Management.IntegrationServicesEnum.dll in folder dbatools\bin\smo, copied from an SSMS 18.4 installation resolved the issue.

Christians solution worked for me today too.


Wrong layer shape when loading checkpoint for inference

Hi,
Thank you so much for the repo!
I used it to train a single class model without problems, however, when I load a checkpoint for inference (after configuring the checkpoint model path and classes in 'inference_quad.py') I get this error:
Traceback (most recent call last):
File "inference_quad.py", line 32, in
prediction_model.load_weights(model_path, by_name=True)
File "/home/javier/.virtualenvs/efficientdet2/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training.py", line 182, in load_weights
return super(Model, self).load_weights(filepath, by_name)
File "/home/javier/.virtualenvs/efficientdet2/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/network.py", line 1371, in load_weights
saving.load_weights_from_hdf5_group_by_name(f, self.layers)
File "/home/javier/.virtualenvs/efficientdet2/lib/python3.6/site-packages/tensorflow_core/python/keras/saving/hdf5_format.py", line 754, in load_weights_from_hdf5_group_by_name
str(weight_values[i].shape) + '.')
ValueError: Layer #399 (named "class_net/class-predict"), weight <tf.Variable 'class_net/class_net/class-predict/pointwise_kernel:0' shape=(1, 1, 64, 12) dtype=float32> has shape (1, 1, 64, 12), but the saved weight has shape (1, 1, 64, 9).
Except for making the anchors explicit, all other arguments of 'efficientdet()' are the same as for training. Any idea why the architecture of the saved weights would be different from the one efficientdet() creates?
What puzzles me is that I can load the same checkpoint for continuing to train (in train.py)...
Thank you!

My bad!  found that out the issue was me not configuring the anchors the same way during training and inference. Sorry.

How did you fix it? I'm having the same error


Install failed on .NET 4.5

When I try to install it on .NET 4.5, I get the following error:

Install failed. Rolling back...
Package 'Alturos.Yolo.2.2.4' does not exist in project 'VisionTest'
Package 'Alturos.Yolo.2.2.4' does not exist in folder 'D:\softwares\VisionTest\packages'
Executing nuget actions took 12.15 sec
Install-Package : Could not install package 'Alturos.Yolo 2.2.4'. You are trying to install this package into a
project that targets '.NETFramework,Version=v4.5', but the package does not contain any assembly references or
content files that are compatible with that framework. For more information, contact the package author.
At line:1 char:1
+ Install-Package Alturos.Yolo -Version 2.2.4
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
+ FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackage
Command

The current package require 4.6.1 of .net framework, you can compile it self if you are use .net 4.5


chore(main): release 1.3.17

:robot: I have created a release beep boop
1.3.17 (2024-07-07)
Bug Fixes

update template (9c35993)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.3.17
:sunflower:


Make it easy to add back the mention of your current repo

Test plan

What about something like this?


IPv6 support

Please add ipv6 support if possible.

edit the nginx site file /etc/nginx/sites-available/default
below listen 80 add the line:
listen [::]:80;
and below listen 443 add the line:
listen [::]:443;
then restart nginx : sudo service nginx restart

I mean ipv6 support for rtorrent and libtorrent please.

As I understand it this will be in the next release of rtorrent, you can try and use the latest code available from the project page https://github.com/rakshasa/rtorrent but I have no idea what issues there may be wit it if any.


Cannot get the DriveItem when I renamed file (PATCH)

Category

[x] Question
[ ] Documentation issue
[ ] Bug

Hi.
I cannot get the DriveItem when I renamed file (use PATCH).
However, DriveItem can be obtained by executing delta after executing PATCH.
Is this behavior correct?
It's NG

PATCH
GET DriveItem -> NG. response is empty.

It's OK

PATCH
delta
GET DriveItem -> OK

PATCH -> GET DriveItem
PATCH (xxxxxx is comment out)
response: 200:
{
  "@odata.context":"https://graph.microsoft.com/v1.0/$metadata#drives('xxxxxx')/items/$entity",
  "@microsoft.graph.downloadUrl":"https://2vau8w.dm.files.1drv.com/xxxxxxxx",
  "createdDateTime":"2019-10-23T01:22:10.347Z",
  "cTag":"aYzo5QzdGMUY4OTRFMzFGNjU5ITE0MjIwMy4yNTc",
  "eTag":"aOUM3RjFGODk0RTMxRjY1OSExNDIyMDMuMg",
  "id":"xxxxxx",
  "name":"bbb",    <- renamed name

  snip

}

I can't get DriveItem. response is empty.
item: {}

PATCH -> delta -> GET DriveItem
PATCH
response: 200:
{
  "@odata.context":"https://graph.microsoft.com/v1.0/$metadata#drives('xxxxxx')/items/$entity",
  "@microsoft.graph.downloadUrl":"https://2vau8w.dm.files.1drv.com/xxxxxxxxxxx",
  "createdDateTime":"2019-10-23T01:27:08.787Z",
  "cTag":"aYzo5QzdGMUY4OTRFMzFGNjU5ITE0MjIwNy4yNTc",
  "eTag":"aOUM3RjFGODk0RTMxRjY1OSExNDIyMDcuMg",
  "id":"xxxxxx",
  "name":"bbb",    <- renamed name

  snip

}

delta
{
  "xxxxxxxx":
  {
    "value":
    [
      {
<EMAIL_ADDRESS>"#microsoft.graph.driveItem",
        "id": "xxxxxxxxxx",
        "name": "bbb",     <- renamed name
        "cTag": "aYzo5QzdGMUY4OTRFMzFGNjU5ITE0MzE1Ny4yNTc",
        "eTag": "aOUM3RjFGODk0RTMxRjY1OSExNDMxNTcuMg",
        "lastModifiedDateTime": "2019-10-23T06:43:37.19Z",

    snip

}

DriveItem
item:
{
<EMAIL_ADDRESS>'#microsoft.graph.driveItem',
  'id': 'xxxxxxxxxx',
  'name': 'bbb',    <- renamed name
  'cTag': 'aYzo5QzdGMUY4OTRFMzFGNjU5ITE0MjIwNy4yNTc',
  'eTag': 'aOUM3RjFGODk0RTMxRjY1OSExNDIyMDcuMg',
  'lastModifiedDateTime': '2019-10-23T01:28:09.34Z',

  snip

}

Which API did you call to make this request?

GET DriveItem -> NG. response is empty.
And which arguments did you provide?  ID or path?

sorry. reply later.

Which API did you call to make this request?

GET DriveItem -> NG. response is empty.
And which arguments did you provide?  ID or path?

I use GET /drives/{drive-id}/root/delta .
(ref.  Sync the contents of a drive - Microsoft Graph v1.0 | Microsoft Docs )

Might be confused, but for the case that we wanted to know more about, delta isn't being called - is it?
Could you please provide more information about the driveItem call that is returning an empty result?
https://docs.microsoft.com/en-us/graph/api/driveitem-get?view=graph-rest-1.0&tabs=http#http-request
Did you use itemId, item-path, or something else to get the renamed driveItem's metadata?


Question: config server applicationcontext started/refreshed/stoppend on each request to the config service

Hi,
We have some batches running every 5 minutes using the config server for retrieving the configuration. In the log files of the config server we see on each request from the batch that the applicationcontext is refreshed for each call.
sequence:

start application

refresh context

close context

09:24:33.449 [http-nio-6499-exec-1] INFO  - Starting application on s40jasstst01 with PID 5060 (started by susr_java_config_ser in D:\sxData\configservice) [o.s.b.StartupInfoLogger:logStarting:48]
09:24:33.449 [http-nio-6499-exec-1] INFO  - The following profiles are active: tst [o.s.b.SpringApplication:logStartupProfileInfo:670]
09:24:33.465 [http-nio-6499-exec-1] INFO  - Refreshing org.springframework.context.annotation.AnnotationConfigApplicationContext@48b5fc66: startup date [Fri Apr 22 09:24:33 CEST 2016]; root of context hierarchy [o.s.c.s.AbstractApplicationContext:prepareRefresh:578]
09:24:33.465 [http-nio-6499-exec-1] INFO  - Started application in 0.11 seconds (JVM running for 3327.168) [o.s.b.StartupInfoLogger:logStarted:57]
09:24:33.465 [http-nio-6499-exec-1] INFO  - Adding property source: file:/D:/sxData/java_io_tmpdir/configservice/config-repo-8950990226508586530/crmbatch.yml [o.s.c.c.s.e.NativeEnvironmentRepository:clean:224]
09:24:33.465 [http-nio-6499-exec-1] INFO  - Adding property source: file:/D:/sxData/java_io_tmpdir/configservice/config-repo-8950990226508586530/application.yml [o.s.c.c.s.e.NativeEnvironmentRepository:clean:224]
09:24:33.480 [http-nio-6499-exec-1] INFO  - Closing org.springframework.context.annotation.AnnotationConfigApplicationContext@48b5fc66: startup date [Fri Apr 22 09:24:33 CEST 2016]; root of context hierarchy [o.s.c.s.AbstractApplicationContext:doClose:960]
09:30:34.513 [http-nio-6499-exec-9] INFO  - Starting application on s40jasstst01 with PID 5060 (started by susr_java_config_ser in D:\sxData\configservice) [o.s.b.StartupInfoLogger:logStarting:48]
09:30:34.513 [http-nio-6499-exec-9] INFO  - The following profiles are active: tst [o.s.b.SpringApplication:logStartupProfileInfo:670]
09:30:34.513 [http-nio-6499-exec-9] INFO  - Refreshing org.springframework.context.annotation.AnnotationConfigApplicationContext@3cace960: startup date [Fri Apr 22 09:30:34 CEST 2016]; root of context hierarchy [o.s.c.s.AbstractApplicationContext:prepareRefresh:578]
09:30:34.529 [http-nio-6499-exec-9] INFO  - Started application in 0.125 seconds (JVM running for 3688.23) [o.s.b.StartupInfoLogger:logStarted:57]
09:30:34.529 [http-nio-6499-exec-9] INFO  - Adding property source: file:/D:/sxData/java_io_tmpdir/configservice/config-repo-8950990226508586530/crmbatch.yml [o.s.c.c.s.e.NativeEnvironmentRepository:clean:224]
09:30:34.529 [http-nio-6499-exec-9] INFO  - Adding property source: file:/D:/sxData/java_io_tmpdir/configservice/config-repo-8950990226508586530/application.yml [o.s.c.c.s.e.NativeEnvironmentRepository:clean:224]
09:30:34.529 [http-nio-6499-exec-9] INFO  - Closing org.springframework.context.annotation.AnnotationConfigApplicationContext@3cace960: startup date [Fri Apr 22 09:30:34 CEST 2016]; root of context hierarchy [o.s.c.s.AbstractApplicationContext:doClose:960]

Questions:

Why is the application context is started/stopped on each call? Why not just fetching the git repo for changes?
Is this behaviour not problematic in case of stoppen/starting a lot of applications at once?

Thanks in advance!
Regards
Peter

That's not the main context for the whole config server, it's just an implementation detail of how the properties are loaded for the EnvironmentController .


[OSD-17593] add hypershift binary to base image

build and add the hypershift binary to the base image, so that must gather can consume it directly

/lgtm


🤖 - DaData input

Description
https://dadata.ru/api/
TODO

That sounds good
Could you describe in more detail what would you like to do?

Interesting

@waterplea is it actual?

Let's revisit it after we refactored all textfields


Make it possible to pass custom Prism plugins

Can haz a way to pass custom Prism plugins to this 11ty plugin? Maybe this can be solved generically by allowing something like this in 11ty's plugin system:
-eleventyConfig.addPlugin(syntaxHighlight);
+eleventyConfig.addPlugin(syntaxHighlight, initializer);

For this particular plugin, initializer would be a function that gets called with the Prism object as an argument. Users could then do something like:
const initializer = (Prism) => {
  Prism.languages.myCustomLanguage = /* */;
};
eleventyConfig.addPlugin(syntaxHighlight, initializer);

What do you think?

I think that’s a good idea, yeah. :+1:
Maybe the this inside the callback should be the eleventyConfig object too?

I’m thinking plugins may want to expose multiple initializer arguments…
Any recommendations stylistically on ({ Prism, otherThing }) vs (Prism, otherThing)?
I think I like the latter better as it doesn’t seem to be as tightly coupled to the upstream plugin’s variable naming. The drawback there is that it has strict argument order ¯_(ツ)_/¯

I strongly prefer ({ Prism, otherThing }). Variable naming is part of the plugin’s API design.

https://github.com/11ty/eleventy-plugin-syntaxhighlight#options


Edits to Addlible, currentlib, and IFS texts.

More edits to fix typos, clarify and tighten up text.
Note the some files had blank # headings and a ## headings. Some have just # headings.  I went  with # & ##.  Not sure what you had in mind.

@SJLennon Another great PR. Thank you!


HW 14.10.16

Gradle  `
App Engine Module

Why did you used indexing api?

Automatically generated when create AppEngine fixed Indexing
Also fixed textUtils TextUtils

Start to work on your own project


Add option to toggle passing file name for spec at point

Right now when we run the spec at point, we also pass the file name. This can be good in that it is more restrictive in what gets run. However, it can also prevent global init code in other files from getting run.

As a workaround, you can set
(setq mocha-command "yarn test")

or
(setq mocha-command "npm test -- ")

and then add test script to the scripts in package.json and test all the files there. Then when you do mocha-test-at-point it gets all the filenames anyway.


Created context menu entry

added ability to format code without using the keybind

i am not sure about this
@akalongman


Add test for awaitData iterator error

Moved from #3956.

Description

It seems like awaitData does not block until the maxTimeMS expires, it returns an iterator error instead.
Readiness checklist

[ ] I added/updated unit tests (and they pass).
[ ] I added/updated integration/compatibility tests (and they pass).
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

This test passes now


[Feature Request] Interop with react-native-gesture-handler

In react-native-snap-carousel the ScrollView and FlatList implementations are hard coded to the ones provides by "react-native".
Can we add a prop to allow users to provide their own wrapped versions of ScrollView and FlatList, similar to how FlatList allows you to provide a wrapped implementation of ScrollView using the renderScrollComponent prop.
This would allow users to use the wrapped FlatList and ScrollView provided by react-native-gesture-handler.
Background
react-native-gesture-handler allows you to specify a waitFor prop on a ScrollView that tells the ScrollView to wait for other gesture handlers to become active or not. For example there could be a Slider component within a horizontal scrollview.

Waiting for this !

If someone wants to jump in and submit a clean PR for that, I'll be more than happy to merge it!

Here we go !


Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator

Is your bot running locally or remotely?
Are you supplying MSA appId and password, or running without credentials?
What are you supplying for the bot endpoint?
What port is your bot listening on?

Hi Eric,
My Bot is running locally(I am using BotBuilder-Samples from https://github.com/Microsoft/BotBuilder-Samples).
I am running it without credentials.
My end point is http://localhost:3978/api/messages
Port is : 3978
Below error I got every time
<EMAIL_ADDRESS>On more thing I would like to highlight here is, I am able to hit http://localhost:3978/api/messages from Advance Rest client but getting error “ERROR: ChatConnector: receive - no security token sent.“ so the concern is Node.js bot sample is listening on applied port properly but Bot emulator is not able to connect on that.
Regards,
Ravi Khatri
From: Eric Anderson<EMAIL_ADDRESS>Sent: Tuesday, March 21, 2017 1:29 AM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Is your bot running locally or remotely?
Are you supplying MSA appId and password, or running without credentials?
What are you supplying for the bot endpoint?
What port is your bot listening on?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-287879462, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVivjhzHPvORhQT11Jm6FY7LjJv5pks5rntqBgaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

Your screenshot of the error didn't come through via email. Can you repost it?

Please find attached error screenshot.
Regards,
Ravi Khatri
From: Eric Anderson<EMAIL_ADDRESS>Sent: Tuesday, March 21, 2017 10:49 PM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Your screenshot of the error didn't come through via email. Can you repost it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-288152359, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVjtiwpNly4jdNpxFKLJVWC1NgxXdks5roAZ0gaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

Still didn't make it. I think you'll have to copy/paste it directly into github. I guess github has problems receiving images over email.

The message ERROR: ChatConnector: receive - no security token sent is expected when the bot is running without credentials and accepts a connection from something that doesn't appear to be the emulator, so it isn't unusual that you saw it when using Advance Rest client.
Can you describe the error you are getting in the emulator?

Everything looks correct, I'm stumped. Maybe an invisible character in the appId or password field? You might try deleting your endpoint and re-adding it.

Yes Eric,
I tried all these but no luck.
From: Eric Anderson<EMAIL_ADDRESS>Sent: Wednesday, March 22, 2017 8:55 AM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Everything looks correct, I'm stumped. Maybe an invisible character in the appId or password field? You might try deleting your endpoint and re-adding it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-288287790, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVtiy88Q_X0pX5tv8J0BxJ-kjUbwpks5roJR4gaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

I think the next step would be to run fiddler as a local proxy to capture more detail about the error and facts leading up to it.

Okay, Sorry but I am not aware about fiddler.
Could you please help me on that.
From: Eric Anderson<EMAIL_ADDRESS>Sent: Wednesday, March 22, 2017 10:34 PM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
I think the next step would be to run fiddlerhttps://www.telerik.com/download/fiddler as a local proxy to capture more detail about the error and facts leading up to it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-288468287, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVnM61z5khL9qZJL4APcl782XjiMnks5roVRqgaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

I have the exact same issue, unable to use the bot framework emulator to connect to my local running bot.
When fiddler is running, I can see some details :
WARN: ChatConnector: receive - emulator running without security enabled. ChatConnector: message received. Error: Request to 'http://localhost:52672/v3/botstate/emulator/users/default-user' failed: [404] Not Found     at Request._callback (c:\usr\botpoc\messages\node_modules\botbuilder\lib\bots\ChatConnector.js:452:46)     at Request.init.self.callback (c:\usr\botpoc\messages\node_modules\request\request.js:188:22)     at emitTwo (events.js:106:13)     at Request.emit (events.js:191:7)     at Request.<anonymous> (c:\usr\botpoc\messages\node_modules\request\request.js:1171:10)     at emitOne (events.js:96:13)     at Request.emit (events.js:188:7)     at IncomingMessage.<anonymous> (c:\usr\botpoc\messages\node_modules\request\request.js:1091:12)     at IncomingMessage.g (events.js:291:16)     at emitNone (events.js:91:20)

@ravik12KPIT fiddler (downloadable from the link I provided earlier) is network inspection software. It lets you view all the network traffic sent and received from your computer. It can offer more detailed information about which network request is failing.

@sapson The emulator will return 404 from the state API if there's no active bot connection. Was the emulator connected to your bot?

@eanders-MS No the emulator is unable to connect to the bot, same symptoms as user @ravik12KPIT has reported. I can test the bot from the console, or after deployment to azure. But the emulator is unable to connect the bot running on localhost. Only when fiddler is active I get some more info (as shared).

After running fiddler the Bot Emulator is able to connect to my node.js code but when I say Hi then I am getting below error.
error ocurred [Error: Request to 'http://localhost:53524/v3/conversations/kl4kd1jennea659mc/activities/ic80ehll62hg1fhac' fail
ed: [403] Access Denied]
and below are the screenshots
<EMAIL_ADDRESS><EMAIL_ADDRESS>From: Ronny De Lafaille<EMAIL_ADDRESS>Sent: Tuesday, March 28, 2017 10:40 PM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
@eanders-MShttps://github.com/eanders-MS No the emulator is unable to connect to the bot, same symptoms as user @ravik12KPIThttps://github.com/ravik12KPIT has reported. I can test the bot from the console, or after deployment to azure. But the emulator is unable to connect the bot running on localhost. Only when fiddler is active I get some more info (as shared).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-289838803, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVi8AtacGsfpRvmVwG4BIjs8A_dZjks5rqT7VgaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

@ravik12KPIT With your bot and emulator running locally, could you send me:

the first few lines of your emulator log, I'd like to see the host & port your emulator is listening on.
the JSON content of a message sent from the emulator to your bot (copy/paste) from the emulator's detail view. I'd like to see what the serviceUrl is set to.
the JSON of a reply sent from the bot to the emulator (available from fiddler).

Thank you!

From: Eric Anderson<EMAIL_ADDRESS>Sent: Wednesday, April 5, 2017 9:21 PM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
@ravik12KPIThttps://github.com/ravik12KPIT With your bot and emulator running locally, could you send me:

the first few lines of your emulator log, I'd like to see the host & port your emulator is listening on.

     This is emulator Log

[12:42:00] Emulator listening on http://[::]:59757
[12:42:01] ngrok not configured (only needed when connecting to remotely hosted bots)
[12:42:01] Howto: Network tunneling with ngrok
[12:42:01] Configure ngrok
[12:42:01] Checking for new version...
[12:42:06] Application is up to date.
[12:42:25] -> POST 202 [conversationUpdate]
[12:42:25] -> POST 202 [conversationUpdate]
-  And below are node.js code log
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
D:\MS BotSetup\HRBotSample-master>node app.js
restify listening to http://[::]:3978
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
session.beginDialog(/)
/ - waterfall() step 1 of 2
/ - session.beginDialog(BotBuilder:Prompts)
.Prompts.choice - session.send()
.Prompts.choice - session.sendBatch() sending 1 messages
And error ocurred [Error: Request to 'http://localhost:59757/v3/conversations/3dcna28f49kamn077c/activities/3dd9b8ffn08i55fnf' fai
led: [403] Access Denied]

the JSON content of a message sent from the emulator to your bot (copy/paste) from the emulator's detail view. I'd like to see what the serviceUrl is set to.
{
"type": "message",
"text": "Hi",
"from": {
"id": "default-user",
"name": "User"
},
"locale": "en-US",
"timestamp": "2017-04-06T07:15:01.565Z",
"channelData": {
"clientActivityId": "1491462744798.8264607637491557.0"
},
"id": "3dd9b8ffn08i55fnf",
"channelId": "emulator",
"localTimestamp": "2017-04-06T12:45:01+05:30",
"recipient": {
"id": "default-bot",
"name": "Bot"
},
"conversation": {
"id": "3dcna28f49kamn077c"
},
"serviceUrl": "http://localhost:59757"
}

the JSON of a reply sent from the bot to the emulator (available from fiddler).

·         -= Fiddler Event Log =-
·         See http://fiddler2.com/r/?FiddlerLog for details.
·
·         12:30:10:7356 Fiddler Running...
·         12:30:10:8176 Windows 8+ AppContainer isolation feature detected.
·         12:30:13:7759 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:48:7789 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:52:5307 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:53:1571 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50658.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50660.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50653.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50657.
·         12:39:58:7856 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:45:03:2914 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-291905752, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVig7lRPzBxmxqDzVcl0F7ukvVNwHks5rs7hNgaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

@eanders-MShttps://github.com/eanders-MS Please go through below required data.
From: Ravi Khatri
Sent: Thursday, April 6, 2017 12:54 PM
To: 'Microsoft/BotFramework-Emulator'<EMAIL_ADDRESS>Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: RE: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
From: Eric Anderson<EMAIL_ADDRESS>Sent: Wednesday, April 5, 2017 9:21 PM
To: Microsoft/BotFramework-Emulator <BotFramework-Emulator<EMAIL_ADDRESS>Cc: Ravi Khatri <Ravi.Khatri<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
@ravik12KPIThttps://github.com/ravik12KPIT With your bot and emulator running locally, could you send me:

the first few lines of your emulator log, I'd like to see the host & port your emulator is listening on.

     This is emulator Log

[12:42:00] Emulator listening on http://[::]:59757
[12:42:01] ngrok not configured (only needed when connecting to remotely hosted bots)
[12:42:01] Howto: Network tunneling with ngrok
[12:42:01] Configure ngrok
[12:42:01] Checking for new version...
[12:42:06] Application is up to date.
[12:42:25] -> POST 202 [conversationUpdate]
[12:42:25] -> POST 202 [conversationUpdate]
-  And below are node.js code log
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
D:\MS BotSetup\HRBotSample-master>node app.js
restify listening to http://[::]:3978
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
session.beginDialog(/)
/ - waterfall() step 1 of 2
/ - session.beginDialog(BotBuilder:Prompts)
.Prompts.choice - session.send()
.Prompts.choice - session.sendBatch() sending 1 messages
And error ocurred [Error: Request to 'http://localhost:59757/v3/conversations/3dcna28f49kamn077c/activities/3dd9b8ffn08i55fnf' fai
led: [403] Access Denied]

the JSON content of a message sent from the emulator to your bot (copy/paste) from the emulator's detail view. I'd like to see what the serviceUrl is set to.
{
"type": "message",
"text": "Hi",
"from": {
"id": "default-user",
"name": "User"
},
"locale": "en-US",
"timestamp": "2017-04-06T07:15:01.565Z",
"channelData": {
"clientActivityId": "1491462744798.8264607637491557.0"
},
"id": "3dd9b8ffn08i55fnf",
"channelId": "emulator",
"localTimestamp": "2017-04-06T12:45:01+05:30",
"recipient": {
"id": "default-bot",
"name": "Bot"
},
"conversation": {
"id": "3dcna28f49kamn077c"
},
"serviceUrl": "http://localhost:59757"
}

the JSON of a reply sent from the bot to the emulator (available from fiddler).

·         -= Fiddler Event Log =-
·         See http://fiddler2.com/r/?FiddlerLog for details.
·
·         12:30:10:7356 Fiddler Running...
·         12:30:10:8176 Windows 8+ AppContainer isolation feature detected.
·         12:30:13:7759 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:48:7789 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:52:5307 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:35:53:1571 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50658.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50660.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50653.
·         12:37:00:8593 [Fiddler] No HTTP request was received from (firefox:11724) new client socket, port 50657.
·         12:39:58:7856 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
·         12:45:03:2914 HTTPSLint> Warning: ClientHello record was 508 bytes long. Some servers have problems with ClientHello's greater than 255 bytes. https://github.com/ssllabs/research/wiki/Long-Handshake-Intolerance
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-291905752, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVig7lRPzBxmxqDzVcl0F7ukvVNwHks5rs7hNgaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

@ravik12KPIT Are you running firewall or security software that is preventing the post back to the emulator? You may need to create a rule to allow emulator traffic.

Hi @ravik12KPIT, While we still don't have root cause for this issue, a workaround reported by another user is to change your endpoint protocol from http to https. YMMV. If it works, please let us know here.
https://localhost:3978/api/messages

Okay I will check it and let you know.
Sent from my Windows Phone

From: Eric<EMAIL_ADDRESS>Sent: 30-05-2017 23:20
To<EMAIL_ADDRESS>Cc: Ravi<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Hi @ravik12KPIThttps://github.com/ravik12kpit, While we still don't have root cause for this issue, a workaround reported by another user is to change your endpoint protocol from http to https. YMMV. If it works, please let us know here.
https://localhost:3978/api/messages
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-304956316, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVkv7rP0hJe30L3R7boBHhfUrvqf7ks5r_FbagaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

Still not working just changed http://localhost:3978/api/messages to https://localhost:3978/api/messages.
From: Ravi Khatri
Sent: Wednesday, May 31, 2017 7:52 AM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: RE: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Okay I will check it and let you know.
Sent from my Windows Phone

From: Eric<EMAIL_ADDRESS>Sent: 30-05-2017 23:20
To<EMAIL_ADDRESS>Cc: Ravi<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Not ableto connect with any BotBuilder-Samples(Node.js Samples) using Bot botframework-emulator (#132)
Hi @ravik12KPIThttps://github.com/ravik12kpit, While we still don't have root cause for this issue, a workaround reported by another user is to change your endpoint protocol from http to https. YMMV. If it works, please let us know here.
https://localhost:3978/api/messages
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BotFramework-Emulator/issues/132#issuecomment-304956316, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZQhVkv7rP0hJe30L3R7boBHhfUrvqf7ks5r_FbagaJpZM4MifJl.
This message contains information that may be privileged or confidential and is the property of the KPIT Technologies Ltd. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message. KPIT Technologies Ltd. does not accept any liability for virus infected mails.

I'm having the same issue and it appears that the Bot Emulator is configuring ngrok for some random local port - every time I restart the emulator it says "Emulator listening  http://[::]:some-random-port". Once I enter in an actual endpoint, the emulator works for that endpoint but the ngrok association isn't updated. If you go to the ngrok traffic inspector page on localhost:4040 you can see the port it is listening on and it isn't the port in my emulator configuration. Seems to me, ngrok shouldn't be initialized until someone configures the endpoint URL.

i have same issue :(

I have the same issue when I was following this tutorial:
https://docs.microsoft.com/en-us/azure/bot-service/bot-service-debug-emulator?view=azure-bot-service-3.0

It looks like various connection issues in the last two years have rolled into this one that may have been unrelated to each other. We're going to close this for now. Please open a new issue if you're on the latest version of the Bot Framework Emulator and experiencing connection issues while following a sample from BotBuilder-Samples.


Show/Hide info bar  should auto unhide when Toggling filter

Is your feature request related to a problem? Please describe.
Info bar that is hidden with the "i  Show/Hide info bar "  key binding, should automatically  be unhide when Toggling the filter with the f key binding.  Otherwise the Filter input is also hidden.

Good catch. Should be fairly simple to fix, would you like to try and fix it?

I won't have the time in the coming month or two.
Not a big deal for me.
Could be an easy win for anyone else that wants it though.


Concerns with Connector Classes: Improper Use of HttpClient and the Risk of Port/Socket Exhaustion

I'm analyzing the connector classes and I'm worried about the use of HttpClient. It seems that they are not following the guidelines outlined in https://learn.microsoft.com/en-us/dotnet/fundamentals/networking/http/httpclient-guidelines. This could potentially cause port/socket exhaustion problems and lead to the application shutting down.

hi @alexandrereyes thanks for looking into this, yes we're aware of the issue and we plan on start using IHttpClientFactory shortly. The work fits into a bigger story that covers refactoring the early prototype, introducing ServiceCollection and the usual .NET DI, which will then allow to support this easily.

The HttpClient has been updated across the kernel; please let us know if this issue still persists.


MidiInput is missing a method for removeMessageReceivedListener()

Because of race conditions, it seems it is possible to assign a listener twice. It would probably be nice to have a way to remove a listener before assigning a new one, e.g. when change midi inputs.
public interface MidiInput : dev.atsushieno.ktmidi.MidiPort {
public abstract fun setMessageReceivedListener(listener: dev.atsushieno.ktmidi.OnMidiReceivedEventListener): kotlin.Unit
}

Adding the same event handler twice itself is not strictly harmful, but it indeed seems error-prone. And having no way to reset is indeed problematic, as the listener argument is not nullable.
We should really switch to add/remove listener style. I kind of avoided that it is a breaking API change to the entire MidiAccess, but probably earlier to execute is better...

This is strange... I checked the ktmidi code and the listener is never a list, it's a single object being assigned. But still when switching between MIDI inputs I need to recreate the listener, otherwise it doesn't work, but I get the input events 2, 3, 4 times... Even tried making it null before creating a new one, but still get multiple input events
runBlocking {
            midiInput = JvmMidiAccess().openInput(portDetails.id)
            logger.info("Adding midi input handler for {}", midiInput!!.details.name)
            if (midiInputHandler != null) {
                midiInputHandler == null
            }
            midiInputHandler = MidiInputHandler()
            midiInput!!.setMessageReceivedListener(midiInputHandler!!)
            logger.info("Added midi input handler for {}", midiInput!!.details.name)
        }

Because you didn't close() the old MidiInput ?

Thanks, indeed closing en open midiinput before opening a new one, solves the double events!


[Fix #10524] Mark Style/RedundantInitialize as unsafe

Fixes #10524.
This PR marks Style/RedundantInitialize as unsafe because if subclass overrides initialize method with a different arity than superclass.
class Base
  def initialize
  end
end

class Derived < Base
  def initialize(arg)
  end
end

Derived.new(arg) # Requires one argument.

Therefore, the following is a false positive.
% bundle exec rubocop example.rb --only Style/RedundantInitialize
Inspecting 1 file
C

Offenses:

example.rb:8:3: C: Style/RedundantInitialize: Remove unnecessary empty initialize method.
  def initialize(a) ...
  ^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected

The same can be said for Module mix-in, which cannot be determined by static analysis.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Add ryutaroikeda.ovh

The link to the webring is the third link on my home page.

Very nice website, welcome to the webring :)


[SPARK-43427][PROTOBUF] spark protobuf: allow upcasting unsigned integer types

What changes were proposed in this pull request?
JIRA: https://issues.apache.org/jira/browse/SPARK-43427
Protobuf supports unsigned integer types, including uint32 and uint64. When deserializing protobuf values with fields of these types, from_protobuf currently transforms them to the spark types of:
uint32 => IntegerType
uint64 => LongType

IntegerType and LongType are signed integer types, so this can lead to confusing results. Namely, if a uint32 value in a stored proto is above 2^31 or a uint64 value is above 2^63, their representation in binary will contain a 1 in the highest bit, which when interpreted as a signed integer will be negative (I.e. overflow). No information is lost, as IntegerType and LongType contain 32 and 64 bits respectively, however their representation can be confusing.
In this PR, we add an option (upcast.unsigned.ints) to allow upcasting unsigned integer types into a larger integer type that can represent them natively, i.e.
uint32 => LongType
uint64 => Decimal(20, 0)

I added an option so that it doesn't break any existing clients.
Example of current behavior
Consider a protobuf message like:
syntax = "proto3";

message Test {
  uint64 val = 1;
}

If we compile the above and then generate a message with a value for val above 2^63:
import test_pb2

s = test_pb2.Test()
s.val =<PHONE_NUMBER>854775809 # 2**63 + 1
serialized = s.SerializeToString()
print(serialized)

This generates the binary representation:
b'\x08\x81\x80\x80\x80\x80\x80\x80\x80\x80\x01'
Then, deserializing this using from_protobuf, we can see that it is represented as a negative number. I did this in a notebook so its easier to see, but could reproduce in a scala test as well:

Precedent
I believe that unsigned integer types in parquet are deserialized in a similar manner, i.e. put into a larger type so that the unsigned representation natively fits. https://issues.apache.org/jira/browse/SPARK-34817 and https://github.com/apache/spark/pull/31921. So an option to get similar behavior would be useful.
Why are the changes needed?
Improve unsigned integer deserialization behavior.
Does this PR introduce any user-facing change?
Yes, adds a new option.
How was this patch tested?
Unit Testing
Was this patch authored or co-authored using generative AI tooling?
No

@rangadi do you mind taking a look? i've modified this to make it an option so that the default behavior doesn't change

bump on this @rangadi in case you get a chance 🙏

Merged to master.


Don't overwrite the Accept header if it is already set

APIGateway has at least one function, get_export, which relies on the use of the Accept header. We were over-writing that with application/json, so this checks to see if the header is already set before we put that in.
cc @kyleknap @jamesls

Can you confirm this doesn't break on response parsing?  If we're sending application/yaml for the accept header, wouldn't we get get yaml back?  I would expect this to break at the JSON response parser unless this was modeled as a streaming payload.

This function is indeed modeled as a streaming payload.

:shipit:  Ok cool.


Header Row setting does not adjust across data tabs

Current Behaviour (for problems)
When a Header row is set to checked, It applies to all data tabs however the setting value is not reflected in each data tab.
Expected Behaviour
Header row setting is applied to separately each data tab
Your Environment

Data Curator version: 0.5.2
Operating System and version: Windows

For future version we should discuss how this is best implemented.
I'm thinking:

show Header Row in Table Properties.
two-way interaction between menu item and header row table property

Yes @Stephen-Gates agreed - this is what I was also thinking to implement here.


Fix broken build when facter specs run first

Fix a race condition where the Rspec would error if the facter spec
tests ran before our other spec tests.
These spec tests for facter previously didn't require spec_helper
because they don't use rspec-puppet.
However, Rspec will error if we define example groups before configuring
the Rspec mock framework. If the tests are parallelised in such a
way as to trigger this behaviour, Rspec aborts with the following error:
Randomized with seed 31992
: RSpec's mock_framework configuration option must be configured before any example groups are defined, but you have already defined a group. (RSpec::Core::Configuration::MustBeConfiguredBeforeExampleGroupsError)
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:582:in `mock_with'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/puppetlabs_spec_helper-1.0.1/lib/puppetlabs_spec_helper/puppet_spec_helper.rb:122:in `block in <top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core.rb:97:in `configure'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/puppetlabs_spec_helper-1.0.1/lib/puppetlabs_spec_helper/puppet_spec_helper.rb:120:in `<top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/puppetlabs_spec_helper-1.0.1/lib/puppetlabs_spec_helper/module_spec_helper.rb:2:in `require'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/puppetlabs_spec_helper-1.0.1/lib/puppetlabs_spec_helper/module_spec_helper.rb:2:in `<top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/spec_helper.rb:4:in `require'
        from /home/travis/build/alphagov/govuk-puppet/spec_helper.rb:4:in `<top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/modules/loadbalancer/spec/defines/loadbalancer__balance_spec.rb:1:in `require_relative'
        from /home/travis/build/alphagov/govuk-puppet/modules/loadbalancer/spec/defines/loadbalancer__balance_spec.rb:1:in `<top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1327:in `load'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1327:in `block in load_spec_files'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1325:in `each'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/configuration.rb:1325:in `load_spec_files'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:102:in `setup'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:88:in `run'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:73:in `run'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/lib/rspec/core/runner.rb:41:in `invoke'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/gems/rspec-core-3.3.2/exe/rspec:4:in `<top (required)>'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/bin/rspec:23:in `load'
        from /home/travis/build/alphagov/govuk-puppet/vendor/bundle/ruby/1.9.1/bin/rspec:23:in `<main>'

I was able to reproduce this by running a facter spec test first:
bundle exec rspec modules/govuk/spec/facter/fqdn_metrics_spec.rb modules/loadbalancer/spec/defines/loadbalancer__balance_spec.rb

https://travis-ci.org/alphagov/govuk-puppet/builds/111979282

Also, remove an unused require.

:+1:

merge when ready


Update to work with keras 3.0

There will be a new multi-backend Keras 3.0 released soon. In theory, this should be a drop-in replacement, but this should be tested (an early version is available for testing).
Fairly high priority.

Closing as tracked in #229


RE1 Models and Door Rando Tooltips

Hi there, on a couple of seeds I noticed that RE1 NPCs can get stuck in cutscenes. For me it was Chris during the Leon, Ben, and Ada cutscene in the jail room, and Rebecca during the part when you play as Ada and come back with items. Not sure if they're getting stuck on geometry or their cutscene AI is bugged. RE2 NPCs seem to work just fine no matter what model is picked.
Another thing, the Door Rando tooltips on the program itself are a little confusing and don't explain what both options do (Number of Segments and Segment Size). It might be easier for others to understand if you explain specifically what each option does according to the rooms. I know a door rando obviously randomizes where doors take you, but in my few previous attempts I'm not sure what the extra options changed other than how long the rando took.
Anyway, this is a great mod and I look forward to future updates! Thank you for your hard work!

Wow, I completely skimmed over the README when first using the rando.
That's my mistake. As for tooltip suggestions;
Number of Segments - "Adjusts how many groups of rooms appear before
reaching the end."
Segment Size - "Adjusts how many rooms are in a group."
That's based on my understanding of the feature after reading the README.
Again, my mistake on that part. Another thing I did notice however, is that
the program seems to forget where the RE1 and RE2 directories are after
exiting it, so I have to select them again when starting up the program.
Not too big of a deal but would be nice if it was fixed.
On Mon, Dec 19, 2022 at 9:38 PM Ted John @.***> wrote:

The options are explained in more depth on the README, and there is a link
on the UI to go to the README. Do you have a suggestion on what the
tooltips should say?
I will disable the jail Ada NPC swap in the next version. Whether or not
it happens is based on seemingly random state and not the model. Ada and
Sherry are the only safe NPCs.
—
Reply to this email directly, view it on GitHub
https://github.com/IntelOrca/biorand/issues/90#issuecomment-1358796002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWXTGPXTSDAUOCN7D4LKPTLWOES4PANCNFSM6AAAAAATEANEOA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Another thing I did notice however, is that
the program seems to forget where the RE1 and RE2 directories are after
exiting it, so I have to select them again when starting up the program.
Not too big of a deal but would be nice if it was fixed.

In order to fix this, I need to know how to reproduce the issue. Could you send me your %APPDATA/biorand/settings.json file?

Here you go.
On Tue, Dec 20, 2022 at 7:41 AM Ted John @.***> wrote:

Another thing I did notice however, is that
the program seems to forget where the RE1 and RE2 directories are after
exiting it, so I have to select them again when starting up the program.
Not too big of a deal but would be nice if it was fixed.
In order to fix this, I need to know how to reproduce the issue. Could you
send me your %APPDATA%/biorand/settings.json file?
—
Reply to this email directly, view it on GitHub
https://github.com/IntelOrca/biorand/issues/90#issuecomment-1359369568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWXTGPX5ZQGDT5W7ORWH7FLWOGZO5ANCNFSM6AAAAAATEANEOA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@TharynP I don't think you attached anything

Oh sorry I replied via email, but github won't actually let me attach the JSON file here. Not a supported file type. You want me to just copy the contents to a TXT file or something?

@TharynP if you zip it first, it should be fine.

settings.zip
The bug actually seems to be gone now, which is strange. I haven't done anything differently, but it's saving the directory paths now. Also, I do want to suggest a feature for the rando which I think would be cool. Do you want me to submit that as a separate issue, or...?

Yeah whenever I ask people to send me the file, the issue goes away, making it hard to investigate.
Make a seperate issue for the feature request.