Kudu can't deploy console app referencing netstandard packages

I'm unable to deploy a Console .NET Framework csproj (as a web job) if it contains a reference to a NuGet package that references netstandard2.0.
A repro repository is here:
https://github.com/davidmatson/kuduissue2521repro
Here's the error message I get:
remote: Program.cs(8,40): error CS0012: The type 'Object' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]
remote: Program.cs(8,27): error CS0012: The type 'Dictionary<,>' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]
remote: Failed exitCode=1, command="D:\Program Files (x86)\MSBuild\14.0\Bin\MSBuild.exe" "D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj" /nologo /verbosity:m /t:Build /p:Configuration=Release;OutputPath="D:\local\Temp\8d4e435d99c9bd6\app_data\jobs\continuous\deployedJob";UseSharedCompilation=false /p:SolutionDir="D:\home\site\repository\.\\"
remote: An error has occurred during web site deployment.
remote:
remote: Error - Changes committed to remote repository but deployment to website failed.

Is Azure Web Apps currently running a preview version of .NET Core 2.0 tooling? I'm able to build locally when using VS 2017 Update 3 with the .NET Core 2.0 SDK (RTM versions of both).

As of this morning, it's running the official ,NET 2.0 tooling if you use dotnet.exe to build. However, when using msbuild, it's possible that it's missing the latest bits.
Are you able to repro by simply building from Kudu Console? That makes a simpler repro that doesn't involve any git deployment.

Thanks; the msbuild portion sounds likely to be the issue here.
I'm not sure this is the repro you're looking for, but I opened up a Kudu Console for this site (https://.scm.azurewebsites.net/DebugConsole), changed to the repository directory, and ran msbuild. I get the same error.
"D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj" (default target) (2) ->
(CoreCompile target) ->
  Program.cs(8,40): error CS0012: The type 'Object' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]
  Program.cs(8,27): error CS0012: The type 'Dictionary<,>' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]

    0 Warning(s)
    2 Error(s)

(Let me know if you're looking for something different.)

@watashiSHUN maybe the problem is that it runs msbuild 14 by default? Would it work with 15?

I tried downgrading to a package that references netstandard 1.3, but that also fails, though with a different error; I'm not sure if the root cause is the same or not. (I can fork this part to a different issue if you like.)
https://github.com/davidmatson/kuduissue2521repro/tree/tryNetStandard13
remote: CSC : error CS1703: Multiple assemblies with equivalent identity have been imported: 'D:\home\.nuget\System.Xml.ReaderWriter\4.3.0\ref\net46\System.Xml.ReaderWriter.dll' and 'D:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.7\Facades\System.Xml.ReaderWriter.dll'. Remove one of the duplicate references. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]
remote: CSC : error CS1703: Multiple assemblies with equivalent identity have been imported: 'D:\home\.nuget\System.Text.RegularExpressions\4.3.0\ref\net463\System.Text.RegularExpressions.dll' and 'D:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.7\Facades\System.Text.RegularExpressions.dll'. Remove one of the duplicate references. [D:\home\site\repository\ReferenceNetStandard20Library\ReferenceNetStandard20Library.csproj]

RE: MSBuild 15:
I tried running that from Kudu Console but couldn't find a v15 subdirectory where v14 is.
D:\Program Files (x86)\MSBuild>where msbuild
D:\Program Files (x86)\MSBuild\14.0\Bin\MSBuild.exe

D:\Program Files (x86)\MSBuild>dir
...
06/06/2017  04:25 AM    <DIR>          .
06/06/2017  04:25 AM    <DIR>          ..
06/06/2017  04:12 AM    <DIR>          12.0
06/06/2017  04:18 AM    <DIR>          14.0
06/06/2017  03:44 AM    <DIR>          Microsoft
06/06/2017  04:25 AM    <DIR>          Microsoft.Cpp
               0 File(s)              0 bytes
               6 Dir(s)   3,167,887,360 bytes free

It's in a different place, though it's not the final v15 yet: D:\Program Files (x86)\MSBuild-15.3-preview\MSBuild\15.0\Bin.
@watashiSHUN is there an app setting someone can use to switch to this build without hard coding the path?

Any thoughts on when the final v15 will be available? I tried running the preview version manually when I ran into a related problem, but it didn't work out.

We will replace preview with 15.3 with the next release

Great news - thanks! Any rough estimate for the next release?-
(#2528 is another issue hitting the same root cause.)

Any update on when the next release will be available? Thanks!

@watashiSHUN, is that change already in?

Actually, it's clear that it is, so closing.

I'm still getting the error message above when I deploy the repro repository. Is MSBuild 15 there but not the default?

@davidmatson msbuild15 is not the default builder when building console app
Were you able to build use MSBuild-15.3.409.57025 ?
if so, you can just add an app setting to your website
MSBUILD_PATH=D:\Program Files (x86)\MSBuild-15.3.409.57025\MSBuild\15.0\Bin\MSBuild.exe
this is (almost) equivalent of having msbuild15 by default
the catch is that during restore, nuget will go back to use msbuild14
but it might not matter for you, I tried on your same repo
if restore is an issue, you will need to add a custom deployment script:

pass the msbuild15.3 directory to nuget (nuget4 has dependency on it):
nuget.exe restore "{your .sln file path}" -MSBuildPath "%MSBUILD_15_DIR%"
use msbuild15.3 to build and publish:
"%MSBUILD_15_DIR%\MSBuild.exe {your build arguments}"

let me know if you need additional help

Thanks @watashiSHUN. I think the version of MSBuild for NuGet restore will matter for most of my real-world cases.
Any thoughts on when MSBuild 15 will be used by default?
Can this issue be kept open in the meantime (since it still repros)?
Thanks,
David

are you just building console app?

Yes, I have a number of repos that build console apps (which run as continuous web jobs).

And yes, the build does succeed if I run MSBuild.exe manually from the Kudu console specifying the full path to the new version.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Telegram Notifications and no-color option

This is great. Highly suggest implementing the no color option, allows for better Slack/external viewing.


Frame delimiter does not decode in some cases

See image attached:

Ticket #44832

See ticket 72810 for more information.
Two capture files provided below:
Captured HDLC Data (decoding error is here)
HDLC-captured-data-153k.sal.zip
Simulation HDLC Data
HDLC-bit-mode-sim-153k.sal.zip
In short, here are the timings found in the capture file for what seems to be the fill flag delimiter (image below).

Here are the timings found in a simulation capture generated by our HDLC analyzer (set to 153k bitrate).

For some reason, our HDLC analyzer is expecting the following pattern.
0 1111111 0 1111111 0......
i.e. [0][7x 1s][0][7x1 s][0]...
However, the specification seems to define the following, with the ability to share the 0 in between frame deliminters (so instead of 2x 0s, it should allow 1x 0):

The capture file seems to follow the specification above.


When will RxDataSources support Swift 5?

RxSwift already released its version to support Swift 5. When will RxDataSources support Swift 5? We are encountering compatibility issues already related to Swift 5 migration. Thanks!

We first need to release RxSwift 5.0.

There is already a latest release for RxSwift to support XCode 10.2. Isn't it the official release? Please confirm. https://github.com/ReactiveX/RxSwift/releases/tag/4.5.0

Follow up on this. RxSwift has already released version 5.0.0.  Thanks!
https://github.com/ReactiveX/RxSwift/releases

The release is out.


[ADnD 2E Revised] Bugfixes and spelling errors

Changes / Comments
All changes described on the forum: https://app.roll20.net/forum/permalink/10745067/
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Add iOS chat demo

Description of changes:
Add an iOS demo to demonstrate how to quickly add chat feature into iOS app using Amazon Chime SDK for messaging.
Testing

How did you test these changes? manually
Can these changes be tested using one of the demo application? If yes, which demo application can be used to test it? n/a

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can we merge both (android and ios) PRs close to the blog publishing time so that builders wont get confused on the blog link here?
But rest looks good.


[Bug]: Should Give an Error when click on Join on Free

What happened?

When click on Join for free then no event is happening. It should give the error or validation.
Kindly assign this issue to me
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

can add a toast there is a check if nothing is there in textfield

if this work is not finished can you assign it to me?
@punyakrit

@punyakrit I am lit busy @vaibhavi-singh-673 kindly assign issue to him


Apple TV - Netflix works, Hulu gets GeoBlock Message

So i've got my VM and docker up and running - Success!
Websites resolve, I've added whatismyip.com to my db.override and my IP comes up.
I can access Netflix on Desktop and Apple TV - Success!
However I can only access Hulu on desktop, it fails on Apple TV with a Geoblocked message.
I have a VM with 1TB of IN+OUT traffic so i'm not concerned about data usage as I can just spin a second server up however I'm wondering if there's a new DNS zone require for Hulu on apple tv?

From memory, when I was testing Hulu on ATV, they do some kind of additional checks in that specific app. I could work out if there was another domain or if they are using an Apple API to obtain a more accurate geo location (i.e. using Wi-Fi).
Have you tried loading it on your iPhone or iPad (assuming you have one) and AirPlaying to ATV instead?
Not sure if Hulu Android/iOS app has Google Chromecast support, but if it does, might be worth trying that too, given the devices are so cheap..
-- ab1

That's a shame. They are really trying harder then netflix. I also added level3.net to my DNS zone too because it appears they are using them for CDN.
I will try it on the Hulu app too and see if I can airplay to it or look at Chromecast...


rPackages.doubles: init at version 1.5.3

This PR adds the doubles package, however, unit tests fail with a very confusing error message:

Open to see error message
Successfully built doubles-1.5.3-py3-none-any.whl
Running phase: pythonRuntimeDepsCheckHook
Executing pythonRuntimeDepsCheck
Finished executing pypaInstallPhase
stripping (with command strip and flags -S -p) in  /nix/store/4vsak84k02c97y2jha23cqsllxc37amy-python3.12-doubles-1.5.3/lib
no Makefile or custom installCheckPhase, doing nothing
File "/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/pytest/main.py", line 9, in
File "/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/config/__in  File "/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/config/__in             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
, line 120, in _hookexec
, line 139, in multicall
, line 103, in multicall
self.hook.pytest_load_initial_conftests(
yield
res = hook_impl.function(*args)
it.py", line 581, in _set_initial_conftests
y'>, 'pastebin': <module '_pytest.pastebin' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/pastebin.py'>, 'assertion': <module '_pytest.assertion' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/assertion/init.py'>, 'junitxml': <module '_pytest.junitxml' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/junitxml.py'>, 'doctest': <module '_pytest.doctest' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/doctest.py'>, 'cacheprovider': <module '_pytest.cacheprovider' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/cacheprovider.py'>, 'freeze_support': <module '_pytest.freeze_support' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/freeze_support.py'>, 'setuponly': <module '_pytest.setuponly' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/setuponly.py'>, 'setupplan': <module '_pytest.setupplan' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/setupplan.py'>, 'stepwise': <module '_pytest.stepwise' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/stepwise.py'>, 'warnings': <module '_pytest.warnings' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/warnings.py'>, 'logging': <module '_pytest.logging' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/logging.py'>, 'reports': <module '_pytest.reports' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/reports.py'>, 'python_path': <module '_pytest.python_path' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/python_path.py'>, 'unraisableexception': <module '_pytest.unraisableexception' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/unraisableexception.py'>, 'threadexception': <module '_pytest.threadexception' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/threadexception.py'>, 'faulthandler': <module '_pytest.faulthandler' from '/nix/store/7xi8n243nis25ql9q5hmpjj5zp2p0ygb-python3.12-pytest-8.3.3/lib/python3.12/site-packages/_pytest/faulthandler.py'>, 'doubles': <module 'doubles.pytest_plugin' from '/build/source/doubles/pytest_plugin.py'>, 'capturemanager': <CaptureManager _method='fd' _global_capturing=<MultiCapture out=<FDCapture 1 oldfd=5 _state='started' tmpfile=<_io.TextIOWrapper name="<_io.FileIO name=6 mode='rb+' closefd=True>" mode='r+' encoding='utf-8'>> err=<FDCapture 2 oldfd=7 _state='started' tmpfile=<_io.TextIOWrapper name="<io.FileIO name=8 mode='rb+' closefd=True>" mode='r+' encoding='utf-8'>> in=<FDCapture 0 oldfd=3 _state='started' tmpfile=<_io.TextIOWrapper name='/dev/null' mode='r' encoding='utf-8'>> _state='started' _in_suspended=False> _capture_fixture=None>, '/build/source/test/conftest.py': <module 'conftest' from '/build/source/test/conftest.py'>}
error: builder for '/nix/store/cqd5inaw37bqs78j8948rwj2kqj295ic-python3.12-doubles-1.5.3.drv' failed with exit code 1

removing pytestCheckHook results in a successful build. Any idea?
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

sorry to tag you @natsukium but I don’t know who else to ask for help 🙏


Describe validation tests with appropriate maths

For example:

if during all the rest steps, the battery is actually resting

For this validation, in documentation, it might be helpful if you mention what exactly are you checking in terms of the code, i.e.

i = 0

as meaning of resting may not be apparent to someone who is new to the field. There are further other possible ways to test for resting, (voltage falling or constant, temp falling or constant, etc.) which method of validation you use here must be well documented as someone might want to add or edit it for their usecase. Let me know if I can explain this better.

This is already specified in the docstring underneath the function.
It says that it checks for resting by making sure the current is at 0.


mapping cfn-lint

https://github.com/aws-cloudformation/cfn-python-lint/

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.

Coverage decreased (-0.5%) to 71.23% when pulling a11aa924b324083a98b0553f22ad4873f3369c9e on PatMyron:patch-1 into 15208540da03fdacf48fcb0a8b88b26da76b64f3 on bndr:master.


9 - execute scan params on connection made

This is a PR for https://github.com/frawau/aioblescan/issues/9#issuecomment-361948555
Br,
Sven

Thanks @Snevzor

No, thank you! :)


ci: add build e2e test for Android

Description of changes

See https://github.com/aws-amplify/amplify-codegen/pull/513 for similar test for TypeScript apps.
Issue #, if available

PR to introduce the error: https://github.com/aws-amplify/amplify-codegen/pull/489
PR to fix the error: https://github.com/aws-amplify/amplify-codegen/pull/500
Description of how you validated changes
TK
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #526 (39fcd12) into main (750ba15) will decrease coverage by 0.38%.
The diff coverage is 88.53%.

@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
- Coverage   85.89%   85.51%   -0.39%
==========================================
  Files         143      148       +5
  Lines        7070     7263     +193
  Branches     1855     1909      +54
==========================================
+ Hits         6073     6211     +138
- Misses        906      956      +50
- Partials       91       96       +5

Impacted Files
Coverage Δ

packages/appsync-modelgen-plugin/src/index.ts
0.00% <0.00%> (ø)

...elgen-plugin/src/interfaces/introspection/index.ts
0.00% <0.00%> (ø)

...lugin/src/interfaces/introspection/model-schema.ts
0.00% <0.00%> (ø)

packages/appsync-modelgen-plugin/src/plugin.ts
0.00% <0.00%> (ø)

packages/appsync-modelgen-plugin/src/preset.ts
0.00% <0.00%> (ø)

...ges/appsync-modelgen-plugin/src/utils/constants.ts
100.00% <ø> (ø)

...ages/graphql-types-generator/src/compiler/index.ts
92.90% <ø> (ø)

...l-types-generator/src/typescript/codeGeneration.ts
95.19% <ø> (ø)

...graphql-types-generator/src/typescript/language.ts
91.95% <ø> (ø)

...-plugin/src/visitors/appsync-javascript-visitor.ts
97.82% <88.88%> (-2.18%)
:arrow_down:

... and 17 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add config inclusion feature

Hi @heiglandreas
Could you have a look at this, and let me know what you think?
This adds the possibility to include CaptainHook configurations in CaptainHook configuration files.
Basically it would look like this
{
  "config": {
    "includes": [
      "vendor/my-company/hook-package/valid-include.json"
    ]
  },
  ...
  "pre-commit": {
    "enabled": true,
    "actions": [
      {
        "action": "phpunit --configuration=build/phpunit-hook.xml",
        "options": []
      }
    ]
  },
  ...
}

If the included configuration would look like this
{
  "pre-commit": {
    "enabled": true,
    "actions": [
      {
        "action": "phpcs --standard=psr2 src",
        "options": []
      }
    ]
  }
}

and you run CaptainHook with the first configuration both pre-commit actions would be executed.
This way you could create  a "company base packages" that requires CaptainHook and maybe some plugins. Also the package could provide some base configurations that can be included in all project captainhook.json files.
This way if you want to add some validation to all projects the only thing you have to do is to update the package and run composer update on all projects.
I have not yet tested if a package can actually require a plugin, but I would hope that it's possible.

Looks awesome! I'll dig into that tomorrow morning...

The idea is great! But I do see some issues with the current implementation (that can be worked around IMO)

First, as far as I see it, the current captainhook.json is not modified. Which means that the inclusion of the file is done on each request of the File, which is on each and every invocation of a git hook. That can be rather time-consuming. Wouldn't it be easier/faster to create a captainhook.lock file (or whatever it is called) that contains the merged config which is then used instead of the captainhook.json? That .lock-file is created on every install or update as those are the only possibilities that the content can change. Alternatively when the lockfile is not available it will be created on the fly, which would mean that theinstallorupdateonly needs to remove that.lock`-file. Benefit would be, that it would be possible to load JSON from websites as well (with all the possible attack-vectors that suddenly opens up)
Secondly I'm not sure how you would use that. Would you always require --dev one company-package that then includes a project specific captainhook.json? Or would you set up The Captain individually with each project and then include a companywide captainhook.json-file? The first would require dependencies to install composer plugins while the later would list all the dependencies the project uses right in the right place. When using the first approach that would mean that the company-dependency would need to require The Captain and possible plugins as dependencies and not dev-dependencies as they would not be loaded otherwise. In the later case one could require a company-wide captainhook.json right from a webserver (given we allow such a thing – see first point).

I think those are my main concerns.

Thanks for taking the time to look at this
So let´s address your concerns.
CaptainHook lock file
I terms of the captainhook.lock file idea. I'm not sure I want to go there yet. Handling lock files is quite a bit of work. Creating hashes to ensure the validity of those files writing them to disk...
Composer is prove that it could be done reliably, but I want to avoid the complexity for now.
I benchmarked the config loading process by loading  a configuration with 2 included configuration files containing roughly 1k actions for different hooks each.
Loaded 3 configs in 0.0069s

For now I don't think loading the configuration every time is an issue in terms of performance we have to worry about. I would argue that in realistic scenarios one would include 2 or 3 configuration files with maybe 3 to 5 actions. Loading such a realistic configuration with 3 includes and an overall amount of 23 actions looked like this.
Loaded 4 configs in 0.0024s

General purpose
Every project still keeps its own captainhook.json to specify project specific settings. But as @aboks pointed out on twitter, if you have a lot of projects and want the same configuration for all of them, currently you have to manage it on copy&paste basis or write custom actions and plugins.
The idea is that you could create a basic composer package like this.
my-git-hooks/
  |-- composer.json
  |-- msg-hooks.json
  |-- crazy-workflow-hooks.json

Within its composer.json you would require (NOT --dev) all libs and plugins you need for the configured hooks in your configuration files (msg-hooks.json, crazy-workflow-hooks.json) as well as captainhook/captainhook or captainhook/plugin-composer.
To use your basic configuration you would have to require your basic package as --dev dependency.
composer require --dev my-company/my-git-hooks

Then create or edit the project captainhook.json and put something in like the following.
{
  "config": {
    "includes": [
      "vendor/my-company/my-git-hooks/msg-hook.json",
      "vendor/my-company/my-git-hooks/crazy-workflow-hooks.json"
    ]
  },
  ...
}

To update the base configuration you create a new version for your my-git-hooks package and run composer update in your projects.
HTTP handling
Opening this up to URLs would be problematic on many fronts.

The performance issue you mentioned before would get a lot more serious if we would do http instead of local files.
It would not be possible to require dependencies via http so you would have to figure that out in your local composer.json and manage all the dependency dependencies there. Not ideal to say at least.

So for now I would keep it as local file inclusion only feature.
To do
In this prove of concept implementation it currently works recursively, so you could include configurations in your included configurations. I'm not sure this is a good idea. No, I know this is not a good idea, I just had no time to make it 1 level of inclusion only. But that should be done before we release it.

Thanks for your work on this. I think @sebastianfeldmann already described my envisoned usecase better than I could have myself, and the current implementation seems perfectly suitable for that.
Personally I don't mind taking a minor performance hit when using included configs. As long as the case without included configs maintains almost its original performance, people can choose for themselves if they find the flexibility of included configs worth the slight performance hit.
As for recursive inclusion of configs, I personally probably won't use that, so feel free to limit it to 1 level. On the other hand, there probably isn't much harm in recursive inclusion. If people want to shoot themselves in the foot, they will always find a way to do so. If someone really wants to use 4 levels of configuration (and accepts the time necessary to load these), why prevent that?

Thanks for correcting me. The problem with the missing code when including remote URLs was something I didn't recognize in the first place. And after the clarification I'm absolutely with the current implementation. The performance issue seems to be negligible so that's OK. And should it become a problem it can then be addressed.
Only thing I ask myself is whether we can use composer to automatically include the files. Like creating a special package type that requires a special entry in their composer.json that can then be read by captain hook. But that is also an improvement that should not hold back the base implementation.

Only thing I ask myself is whether we can use composer to automatically include the files. Like creating a special package type that requires a special entry in their composer.json that can then be read by captain hook. But that is also an improvement that should not hold back the base
implementation.

That would be nice
I added a config setting maxIncludeLevel that defaults to 1. So per default I try to keep everybody from shooting themselves in the foot, but if they insist they can do it.
You can set it like this
{
  "config": {
    "maxIncludeLevel": 2,
    "includes": [
      "some-config.json"
    ]
    ...
}

From now on CaptainHook fails if an included configuration is not available and additionally I added the @internal annotation to all config related classes.


Bugfix/fix bad storage report

Fix some errors with the storage report.
I realized that the off_t type (in the struct pv_storage) is normally defined as a signed integer on POSIX and in another hand, the types inside of the struct statfs involved in the storage size calculation are __fsword_t that type is for internal use and the recommendation is to use an unsigned int and  fsblkcnt_t who is defined as "unsigned integer type" so I change those off_t to unsigned to avoid the overflow.
Also finally I change the pv_json_ser_int by pv_json_ser_number because the underlying library uses double

Looks good to me :)

I add a couple of commits because I found some problems with the PR:

FIrst, two functions return and receive an off_t that was changed to unsigned int
the calls to pv_log were using PRI64u that was replaced by %d


feat(DataExport): added export to csv functionality

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remeha.service tool <IP_ADDRESS> 9dee46a4 4f8c 4f9d b1d5 5501aa7d1e25

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate of #141271;


Anything about Verification?

Hi, every contributor on this work
Thanks for your work on face recognition, I have some issues on verification and need your help.
From your code on verification.py, it seems that "actual_issamne" is a prerequisite. Meanwhile, the definition "get_val_pair" in util.py  appears "issame". How do you generate the "{}/{}_list.npy" file? Could you provide me with an example img_list for paired images?
I'm looking forward to your reply, thanks.

I'm trying to run it with my own dataset and also need some insights on this "{}/{}_list.npy" file! Any hints?


Quiche roll<PHONE_NUMBER>2953

Commit Message: Update QUICHE from 8eb90efcb to d23adb74d
Additional Description: This is a standard dependency roll, no notable changes.
Risk Level: Low
Testing: Existing tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

/assign @RyanTheOptimist

Looks like there's a clang-tidy issue:
Message: invalid case style for function 'Http2HeaderBlockToEnvoyTrailers'
  ReplacementText: http2HeaderBlockToEnvoyTrailers

Thanks for the reviews. I fixed the clang-tidy issue.


d3d9 dll?

Recent updates started building d3d9.dll again instead of static build. Is it needed vs upstream dxvk dll or it's just a build artifact that's not needed to be used?

The former, as in it is prerequisite for upstreaming.  Haven't tested it myself but it shouldn't cause any issues with future updates.

Yeah from now on you're gonna need to copy d3d9.dll again.

I see, thanks. Are you preparing to upstream it soon?

It is a work in progress. Dunno if the merge will happen soon

Any news on merging or what is blocking it?

There is a goal of having it merged for dxvk 2.4

Merge is done!


EPIC: Add the ability to hard code parameters

Due to celestia's heavy reliance on social consensus, we are removing the ability for token holders to vote on a small set of parameters. This includes the inflation rates and the maximum block size.

removing this as a strict requirment for mainnet, since we will remove the inflation related params with #1570 and we will have some mechanism to hardcap the blocksize


Update unlikelihood.py

passing device through to the opts

I think we need to send through the device or later it complains it is missing the device definition

Thank you !


fastlane download_dsyms Unknown method 'train_number'

fastlane_version "2.86.0"
action: download_dsyms(version: "1.0.0")
Issue Description
******version_set=nil>, comparing to supplied version: 3.0.3
ERROR [2018-03-20 15:02:02.68]: Unknown method 'train_number'
WARN [2018-03-20 15:02:02.68]: Lane Context:
INFO [2018-03-20 15:02:02.68]: {:DEFAULT_PLATFORM=>:ios, :PLATFORM_NAME=>:ios, :LANE_NAME=>"ios refresh_dsyms"}
ERROR [2018-03-20 15:02:02.68]: To call another action from an action use other_action.train_number instead
'train_number' was used before it was declared.

This was fixed in 2.86.1 😊


Array Data Type in Step 1 in Migrate from CSV Doc Update

Re: Migrate from CSV
Issue Description
It may be worth noting in step 1 how to format Arrays in the CSV since it's a little different from other data types.
Suggested Resolution
Adding a line:
Arrays should be formatted like ARRAY['item 1', 'item 2']

Handing off to @lnhsingh and @mwang1026 for prioritization.


SM2 encryption in our providers

SM2 encryption support in our providers is currently lacking, but does exist and work in legacy form.

There's a vote going on whether we should simply drop SM2 encryption.


Running multiple gym_torcs experiments in parallel on the same machine

Hi,
I want to launch multiple gym_torcs environments on my machine so that I can run multiple simulations in parallel. Currently when I try to launch an extra simulation, the previous one crashes. The problem is likely the fact that the client and server always talk through the 3001 port. Is there a way to assign different ports to different instances of gym_torcs?
Thank you,
Anirban

Hi,
You can use https://github.com/ugo-nama-kun/gym_torcs/pull/3, though it is not merged yet.

@muupan I did and it worked! I appreciate your job! Hope your PR gets merged soon :)


NETStandard TLS Parity

Imports BouncyCastle when target is NETStandard for PEM Reading
Removes obsolete NatsTlsOpts methods
API Parity for NatsTlsOpts between NETStandard  and NET6/8

I'm wondering Bouncy dependency is going to be an issue for some people. Be nice if we can bring in a simple implementation for reading PEM. it doesn't need to be super efficient at all.

We could potentially require a different dependency for Bouncy Castle instead of bundling into Nats.Client.Core?  Then throw NotSuppportedException with a message that you have to install another dependency to use the pem-related fields on NETStandard

After further discussion on Slack, @mtmk and I thought that it would be best to

Remove Bouncy Castle dependency
Add support for importing PEM-encoded CA Certs on NETStandard
Remove support for importing PEM-encoded Client Cert/Key on NETStandard.  Too tricky to build an X509Certificate2 without any of the PEM methods to load/associate PEM-encoded Private Keys
Add support for using PKCS#12 .pem files for client certificates on all targets


Sentry 10 Installations fails with getsentry-ldap-auth

Like always added "sentry-ldap-auth" to requierements.txt and started install.sh from Sentry getting following error (file attached because of many lines).
commandline.txt
Without "sentry-ldap-auth" install.sh is succeeding.

Following i needed to add to Dockerfile for Sentry 10
# Install dependencies for LDAP authentication
RUN apt-get update && apt-get install -y build-essential libsasl2-dev python-dev libldap2-dev libssl-dev

Now it works :D

I'm just planning to upgrade a 9.x installation, @RogerSik can you confirm, that the plugin still works with Sentry 10?

@janLo still works 👌


:index => :some_string_attribute

Is it possible to include support for string attribute exact match index?
Useful for things like language codes etc

Yes it is possible, but I am not yet sure how to best implement it.
In your use case(s), is the string attribute a unique index or is it used to retrieve multiple records?
So is it something like Language.find_by_code('en'), or more like Person.where(:last_name => 'Jones').all?

Language.find_by_code('en') is exactly what it is


Recording rule support

Things the language server could do with recording rules, specified in the same YAML file:

Offer them as completions
Support go to definition
Show their definition and the information that it is a recording rule on hover

Current state of this effort:

Completion for recording rules is supported.
Hover works but does not deal well with duplicates.
Labels specified in the recording rule are not respected yet.
Go to definition is not supported yet.
There probably should also be warnings for duplicate recording rules.


Release Candidate Comments (Inclusive Leadership)

This issue was created to collect comments about the upcoming metrics release.
This thread is for comments about Inclusive Leadership
GitHub location: https://github.com/chaoss/wg-diversity-inclusion/blob/master/focus-areas/leadership/inclusive-leadership.md
See all release candidates of metrics are at:
https://chaoss.community/metrics-202008/

Overall, I think this metric is of utmost importance. However, I have one concern, on the "inclusive leadership cycle," six structured principles serve as a check-list. Are those related to the six listings below? Moreover, I don't know if there are sequential steps. For example, you must start from step 1, then step 2, etc., (clockwise) I think it's important to know the flow of things.
Signed-off-by: Armstrong<EMAIL_ADDRESS>

Thank you @foundjem for the feedback :)
We removed the confusing figure and removed the numbering to avoid implying sequential order.

Awesome addition to our community! I offer a minor edit in #298. Great work gang :smile:

Under Objectives, it should perhaps read "Signal to newcomers that everyone is welcome and everyone can be successful"
The way it's written now, it sounds like only certain people will be welcome.

One item that seems to be missing is where these guidelines should be documented. Ax example list maybe helpful (e.g. Code of Conduct, Participation Guidelines, Bylaws, or other governance documents).

Thanks @rpaik -- I issued a PR for the objectives statement. Do you have thoughts on where your guidelines documentation comment could live in the metric?

@GeorgLink is this metric ready?

Ready for release.


Does PList need to keep creating new StringBuilders ?

An observation / question whilst liiking at using CopyString intsead of GetString in JSONPListParser.
The reason I ask is that I made a small benchmark.net project that just loads the C# grammer and the dark theme, and with the current code (the System.Text.Json version), i get this:
BenchmarkDotNet=v0.13.5, OS=Windows 10 (10.0.19045.2728/22H2/2022Update), VM=Hyper-V
11th Gen Intel Core i7-1185G7 3.00GHz, 1 CPU, 4 logical and 2 physical cores
.NET SDK=7.0.202
  [Host]               : .NET 7.0.4 (7.0.423.11508), X64 RyuJIT AVX2
  .NET 6.0             : .NET 6.0.15 (6.0.1523.11507), X64 RyuJIT AVX2
  .NET 7.0             : .NET 7.0.4 (7.0.423.11508), X64 RyuJIT AVX2
  .NET Framework 4.7.2 : .NET Framework 4.8 (4.8.4614.0), X64 RyuJIT VectorSize=256

|            Method |                  Job |              Runtime |        Mean |     Error |    StdDev |     Gen0 |     Gen1 |  Allocated |
|------------------ |--------------------- |--------------------- |------------:|----------:|----------:|---------:|---------:|-----------:|
| LoadDarkPlusTheme |             .NET 6.0 |             .NET 6.0 |    32.65 us |  0.563 us |  0.603 us |  11.4746 |   0.9155 |   70.58 KB |
|     LoadCSGrammar |             .NET 6.0 |             .NET 6.0 | 1,077.55 us |  5.228 us |  4.891 us | 279.2969 | 121.0938 | 1719.05 KB |
| LoadDarkPlusTheme |             .NET 7.0 |             .NET 7.0 |    27.89 us |  0.439 us |  0.389 us |  11.5051 |   0.9155 |   70.58 KB |
|     LoadCSGrammar |             .NET 7.0 |             .NET 7.0 | 1,008.97 us | 12.943 us | 10.105 us | 279.2969 | 121.0938 | 1719.05 KB |
| LoadDarkPlusTheme | .NET Framework 4.7.2 | .NET Framework 4.7.2 |    53.95 us |  0.197 us |  0.164 us |  11.7798 |   0.9155 |   72.43 KB |
|     LoadCSGrammar | .NET Framework 4.7.2 | .NET Framework 4.7.2 | 1,652.90 us | 16.280 us | 15.228 us | 287.1094 | 134.7656 | 1768.32 KB |

but with this change I get:
|            Method |                  Job |              Runtime |        Mean |     Error |    StdDev |      Median |     Gen0 |    Gen1 |  Allocated |
|------------------ |--------------------- |--------------------- |------------:|----------:|----------:|------------:|---------:|--------:|-----------:|
| LoadDarkPlusTheme |             .NET 6.0 |             .NET 6.0 |    27.84 us |  0.550 us |  0.514 us |    27.55 us |   6.5613 |  0.4883 |    40.3 KB |
|     LoadCSGrammar |             .NET 6.0 |             .NET 6.0 |   954.05 us | 18.628 us | 27.881 us |   951.18 us | 167.9688 | 72.2656 | 1031.75 KB |
| LoadDarkPlusTheme |             .NET 7.0 |             .NET 7.0 |    23.26 us |  0.459 us |  0.948 us |    22.86 us |   6.5613 |  0.4883 |    40.3 KB |
|     LoadCSGrammar |             .NET 7.0 |             .NET 7.0 |   867.26 us | 17.322 us | 38.743 us |   847.65 us | 167.9688 | 94.7266 | 1031.75 KB |
| LoadDarkPlusTheme | .NET Framework 4.7.2 | .NET Framework 4.7.2 |    48.68 us |  0.407 us |  0.340 us |    48.77 us |   6.7749 |  0.4883 |   41.79 KB |
|     LoadCSGrammar | .NET Framework 4.7.2 | .NET Framework 4.7.2 | 1,608.01 us | 31.973 us | 66.739 us | 1,585.66 us | 173.8281 | 66.4063 | 1079.56 KB |

The timings are a bit variable running the test on my laptop, but the reported reduction in allocations was much larger that I expected (to the point I wondered if something else is going on, but the unit tests all pass so).
Anyway, I'll leave it here for comments.

Yes allocation are a little bit better and the change look safe. Let's merge it.


Clipboard Manger is TAURI_PLUGIN_CLIPBOARDMANAGER instead of TAURI_PLUGIN_CLIPBOARD_MANAGER

Hello! There is a typo in the injection where it injects __TAURI_PLUGIN_CLIPBOARDMANAGER__ in window when it should be __TAURI_PLUGIN_CLIPBOARD_MANAGER__

Also https://v2.tauri.app/plugin/clipboard/ it is documented as __TAURI_PLUGIN_CLIPBOARD_MANAGER__


Fix eks very-large-highmem template

There is test error here. It may not related to the current code, but we can have check why it failed randomly?


Run Set-AzVmDiskEncryptionExtension cmdlet again with new sequence version after adding a new data disk

Hello,
Please could you explain why "If auto encryption occurs on a newly added data disk, we recommend running the Set-AzVmDiskEncryptionExtension cmdlet again with new sequence version." ?
Thanks

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

ID: d7c63605-95f5-0953-3cf6-1e9c50532c1d
Version Independent ID: 8731daa8-2c15-78f7-adac-151401c13b6c
Content: Enable Azure Disk Encryption for Windows IaaS VMs
Content Source: articles/security/azure-security-disk-encryption-windows.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@giovannifl
Thanks for your feedback! We will investigate and update as appropriate.

#please-close


Allows checking if host MPI buffers are required at runtime

This PR adds the option to use host MPI buffers even if GINKGO_FORCE_GPU_AWARE_MPI is true. The option can be set in the communicator constructor. Additionally, a helper function is added that checks if the combination of executor and communicator needs to use host buffers.

format-rebase!

format!


Bug fix/propagate float to long

According to this thread I've changed float types to long for

e.NewLength;

private void OnVlcMediaLengthChanged(object sender, VlcMediaPlayerLengthChangedEventArgs e)
        {
           e.NewLength;
        }

and I've tested the project.

Nice, thanks.
Can you squash your commits into one an force-push in your branch please? Otherwise, I'll squash the commits when merging

Sir, that is my first pull request in the github so that I'm not familiar with and I am an amateur programmer.
I think I don't have the permission of squashing.

Only those with write access to this repository can merge pull requests.

how can I do that ? should I create a new pull request with another way ? or squash this existing pull request ?

Does it mean that my pull request is Unusable with out squashing !?
pardon me...

You pull request is nice and usable ( I still need to test though). I'm just nitpicking : I like having a clean git history and having a single commit to fix that, rather than 4 commits

I was also wondering to make a PR for LengthChanged too. but it seems that I should read more about github.
by the way thanks for your kindness to me.

You're welcome, I welcome everyone wanting to make contributions to this project.
For LengthChanged? Isn't this what this PR is about?
Your PR is fine, don't worry. I was talking about advanced git usage I didn't know until recently.

sorry my mistake.
I mean PositionChanged which is float too.
I've editted last comment.

Dear @jeremyVignelles special thanks for your assistance.

This PR has been merged manually, thanks for your contribution! A new package will be available in a few minutes : 3.0.0-develop303

now can I delete the branch ?

Yes, you can


[status] Add autodiscovery section in DCA status output

What does this PR do?
Adds a new section with information about Autodiscovery in the output of the datadog-cluster-agent status command.
The new section is the same as the one already shown in the node agent status.
It's useful to identify issues in the cluster check configurations fetched from Kubernetes services and endpoints. Currently we need to check the logs to get the same information.
This is what the new section looks like when there's an invalid JSON in the autodiscovery annotations of a Kubernetes service:
=============
Autodiscovery
=============
  Enabled Features
  ================
    kubernetes
    orchestratorexplorer

  Configuration Errors
  ====================
    kube_service://default/nginx
    ----------------------------
        cannot parse check configuration: invalid character '"' after object key:value pair

Describe how to test/QA your changes
Deploy on Kubernetes. Create a Kubernetes service that contains a check in the annotations. Use an invalid JSON and check that the output of datadog-cluster-agent status in the DCA pod contains the section shown above. Make sure that the errors goes away when the service is deleted or fixed.
Do the same with an endpoint check.
Docs for service and endpoint checks:

https://docs.datadoghq.com/containers/cluster_agent/clusterchecks/?tab=helm
https://docs.datadoghq.com/containers/cluster_agent/endpointschecks/?tab=helm

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Also, I think we could add in the PR description an example of the output with 2 service/endpoints check where one is erroneous.


AhrefsSiteAudit/6.1 (#1)

Mozilla/5.0 (compatible; AhrefsSiteAudit/6.1; +http://ahrefs.com/robot/)
First commit - not sure if there's any rules.  I hope note

Thanks a lot!


Bump doctrine/common dependency

Commiting the lock file will force dependencies to be locked for that specific version.
If you're locking the dependencies to a specific revision it can cause issues if we depend directly on doctrine/mongodb.

Composer does not consider the content of a dependency's lock file, only the lock file in the root project itself is ever used.


pubnubsub-handler to 1.0.5 (Wink)

Description:
Bump pubnubsub-handler to 1.0.5 to fix users seeing the following error
ERROR (EndpointThread-Subscribe-0) [pubnub] Exception in subscribe loop: HTTP Client Error (400): {"error":"Bad Request","message":"Client Error","statusCode":400}

Part of this update delays the pubnub subscription for 60 seconds after the Home Assistant started event is fired. This means some users may see things get out of sync if they change any Wink devices during that 60 seconds.
Related issue (if applicable): fixes https://community.home-assistant.io/t/wink-pubnub-not-updating/114580/209
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist

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

[X] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[X] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.

Hello @w1ll1am23,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

80275851b7e896738b101bd53c6828250bf95738 - This commit has something that looks like an email address (billy.scanlon@kroger.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:


.Net: Adding Kusto as an external memory

Motivation and Context

Description
Adding Kusto as an external memory.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

You guys have an impressive array of auto-checks. Kudos.

@shawncal , is there anything else required to in order to merge this?


SchemaService.addOrUpdateExtractor deserealization problem of io.hyperfoil.tools.horreum.entity.json.Schema

When running horreumClient.schemaService.addOrUpdateExtractor(extractorUpdate);  I am receiving:
InvocationTargetException: javax.ws.rs.ProcessingException: com.fasterxml.jackson.databind.exc.MismatchedInputException:
Cannot construct instance of `io.hyperfoil.tools.horreum.entity.json.Schema` (although at least one Creator exists):
no String-argument constructor/factory method to deserialize from String value ('uri:mcervena_test2')

This is my extractorUpdate object

Out of date, closing.


Support for Argo Workflows?

Hi There,
Based on how Hikaru is using OpenAPI/Swagger specs to generate types, would you consider supporting Argo Workflows? (Swagger 2.0 Spec).
If not, is there any way to generate my own types and use them with Hikaru?
Thanks!

Let me have a look. Hikaru's build system doesn't handle all of JSON Schema,  just the idiosyncratic use made by Kubernetes. This means the builder may not be able to generate Python properly or that it will work with the rest of Hikaru without modification. General JSON Schema support is on my radar in order to provide more seamless CRD support, but as I study the problem it's a big job. Your question is leading me to wonder if a more incremental approach. So let me have a tinker and I'll post what I find.

For the benefit of those who might be following this thread: I think I need a page that addresses "Can I use Hikaru to process the swagger for project X?" questions that arise from time to time. The answer is "maybe, but..."
The Kubernetes swagger has some specifc properties that allow Hikaru to turn that swagger into a swath of Python classes with methods:

K8s comprehensively creates JSON Schema definitions for every object that it uses; there are no naked JSON objects anywhere in the swagger-- if a piece of data is structured at all, uses of that data involve a reference to a definition.
K8s uses those definitions not only in the definitions of more complex objects, but more importantly as the primary argument to an operation. This allows me to associate the operation with an object, and is how we establish class and instance methods for these operations on the generated Hikaru classes.
K8s provides a generally consistent set of naming conventions that allow you to tie an operation with a function or method in the underlying K8s Python client. Eventually this will be less of an issue when Hikaru grows to simply use the underlying generic request mechanisms in the K8s client, but for now it uses this information to short-cut to methods that wrap these base facilities.

These properties are what allows Hikaru's builder do what it does with the K8s swagger easily. The JSON Schema spec used within swagger allows a lot more uses such as in-lined descriptions of complex objects for operations that would thwart things like automatic method creation on an object, and definition-less specifiction of objects that are used as parameters to operations or components to other objects.
It certainly is possible to create a parsing system that produced a structure that the user could then fill in missing object details or otherwise supply info that isn't in the swagger in order to allow the creation of a similar set of classes to those of Hikaru. This sort of use has been on my mind as it has direct applicability to richer support for CRDs. It is also a somewhat complex problem with a lot of work involved.
So what am I telling everyone here? I'm trying to set expectations. I welcome requests regarding the possibility of Hikaru supporting another swagger file and will take some time to have a look at what sort of issues lurk within, but I can tell you that, so far, I haven't come across any other requested swagger that Hikaru's builder can easily adapt to. Additionally, K8s has some idiosyncratic conventions that I leverage, and other swagger may not have those (or may have entirely different ones), and those take time to understand and adapt to.
So one thing people who request support may want to consider is how they can get some 'skin in the game', either by providing research or coding capacity to support this different swagger, or consider if their need is significant enough that they would be willing to enter a commercial arrangement with me to pursue this work for them.
As for the above Argo support request, I can share the following update for those who are interested: I've gotten some way through processing the Argo swagger and have had to only make minimal changes to the builder. But at the moment I'm stopped by an unexpected notation in how results may be reported, as the Argo API has some results that  aren't HTTP status codes, and I need someone to look into these to determine the semantic so I can understand how they should be handled. Additionally, I see more than a few operations that don't use defined types but just a set of parameters, so something new will need to be added in the case of an operation that can't be turned into a method automatically. I've invited @MattHodge to continue the details of this outside of this thread and will update here depending on how those discussions go.

This hasn't received any community support, so I'm considering closing this item unless someone can help out.

Closing due to apparent lack of interest by the original author.


use typed value for container type

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

Project coverage is 53.42%. Comparing base (b5b50ef) to head (b313060).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage   53.42%   53.42%
=======================================
  Files          19       19
  Lines        1065     1065
=======================================
  Hits          569      569
  Misses        436      436
  Partials       60       60

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


gulp.useref v3 error, update generator

I had an error with gulp.useref generator. Ubuntu 15.10, npm 3.4.1, node 5.0.0.
According to https://github.com/jonkemp/gulp-useref/issues/153, we shold change generator to avoid this error in new version.
I solved it with code below.
gulp.task('build-app', ['clean', 'inject-all'], function () {
  var assets = $.useref({searchPath: '{.tmp,app}'});
  var jsFilter = $.filter('**/*.js', {restore: true});
  var cssFilter = $.filter('**/*.css', {restore: true});

  var stream = gulp.src('app/index.html') // main html file
    .pipe(assets); // all assets (without index.html)

  if (options.minify) {
    stream
      .pipe(jsFilter)
      .pipe($.ngAnnotate({
        add: true,
        sourcemap: true
      }))
      .pipe($.uglify())
      .pipe(jsFilter.restore)
      .pipe(cssFilter)
      .pipe($.csso())
      .pipe(cssFilter.restore);
  }

  stream
    .pipe(gulp.dest(paths.dist));

  return stream;
});

We're gonna update the generator's dependencies soon and thus also gulp-useref. This might save us some time. Thanks!

fixed with bd715cb7e312166e3537b50562d722ba67b2a586 on #290.
Thank you very much for the heads-up! :)


Update to Fody Version 2

Fody Version 2 is out and you will need to do an update for your addin to be usable with it
Specifically:

Update to the newest Fody.Cecil nuget
Change the min dependency range for Fody to be at least 2.0

Please ping me if u need help

Updated!!!!


Design Panel: Changes do not save when clicking back into main story editor area

Bug Description
A forum user reported that when a user adds or changes text in the Accessibility field for an image and then clicks back into the main editor area, the Accessibility text is not saved.
Expected Behaviour
Accessibility text should be saved by default after the user clicks back into the main editor area.
Steps to Reproduce

Click on an image in the editor
In the right sidebar, enter text in the Accessibility field
Click anywhere in the main editor area
Click on the original image and see that Accessibility text was not added/updated

Screenshots
Video: https://jmp.sh/FwLr6y5
Additional Context

Plugin Version: v1.110
WordPress Version: v5.8.1
Operating System: macOS Big Sur
Browser: Chrome v93

I think this is true for all changes in the design panel. Can you confirm that?
cc @barklund

@swissspidy The user originally reported it for Accessibility and Attachment, but I can't replicate with Attachment – things are saving for me there.

Tested a few fields and this seems to be like this for most of the fields, e.g. when changing a value of Width of an element and then selecting the Page, the value does not get assigned.
It happens not when clicking anywhere in the main editor but when switching an element before blurring, the value doesn't get saved then.
It does not happen for all fields though, e.g. Page Attachment fields and Link fields save the value anyway. But yes, this is the default behaviour for most of the fields.

Thanks for confirming @miina!
Is this fixable?

Most likely fixable but not fully sure what would be the effort of this and if this would mean sacrificing any other features.
The issue here is that even though the fields have similar logic in the editor, then in the design system the elements behave differently. E.g. the numeric input is not actually passing onChange to the input directly but is instead saving the value locally on change, no matter what the editor has for the onChange logic. The numeric input was implemented in https://github.com/google/web-stories-wp/pull/6419
It also includes things like restoring the original value with Esc etc.
That's the same for TextArea (having internal onChange handler) which is used by the accessibility fields and was reported in this GH issue.
The link description and other fields that update as expected are using Input field from the design system which does not have inner custom handling of onChange.
For making it work for all the inputs, it would require reworking the inputs so that perhaps the design system would be purely presentational and not have its own logic for onChange. It would need a bit of further investigation to understand the complexity/possibility of maintaining the current features as well while removing the onChange internal handling.
Perhaps @samwhale remembers the requirements for needing internal onChange handler. I can see the following:

Saving the value only onBlur (to enforce not calling onChange with each character change)
Allowing to restore original value by using Esc

Were there any others that depended on this?
Related: https://github.com/google/web-stories-wp/issues/6690

👋 hello! Yes, these components are all controlled components except for the NumericInput.

Perhaps @samwhale remembers the requirements for needing internal onChange handlers. I can see the following:

Saving the value only onBlur (to enforce not calling onChange with each character change)
Allowing to restore original value by using Esc

The NumericInput is a little different - we didn't want to blow up the existing functionality in the app so this input tracks its own internal state to allow for reverting changes.
The NumericInput calls the onChange handler that's passed in when the input is blurred:
https://github.com/google/web-stories-wp/blob/37835ce9a2d110e71c7ddef1e40415efa688f04c/packages/design-system/src/components/input/useNumericInput.js#L49-L80
@miina is totally right! Reverting the state when pressing Esc and not wanting to push every update on every key stroke led us to using internal change handlers. The ⬆  and ⬇ keys also change the value, but that's it.

For making it work for all the inputs, it would require reworking the inputs so that perhaps the design system would be purely presentational and not have its own logic for onChange. It would need a bit of further investigation to understand the complexity/possibility of maintaining the current features as well while removing the onChange internal handling.

I think you're totally right about the reason this doesn't work. It's because we only save values on blur rather than on change BUT switching the selected element doesn't bubble up an event that would trigger a blur.
Ideally we'd save values with the onChange handler rather than blurring. We could even debounce network requests or saves so that we don't save on every single key stroke. Then we shouldn't run into this problem.

cc @barklund For prioritization


Multiplayer / Dedicated server error : event registration error

Can not join server, player disconnects with following error ( full log ):
`  34.155 Checksum for script LogisticTrainNetwork/control.lua:<PHONE_NUMBER>
34.162 Checksum for script LtnManager/control.lua:<PHONE_NUMBER>
34.166 Checksum for script LTN_Combinator/control.lua:<PHONE_NUMBER>
34.167 Checksum for script LTN_Content_Reader/control.lua:<PHONE_NUMBER>
38.760 Error ClientMultiplayerManager.cpp:1106: mod-LtnManager was not registered for the following events when the map was saved but has registered them as a result of loading: Custom event (ID 223)
38.760 Error ClientMultiplayerManager.cpp:1108: mod-LtnManager was registered for the following events when the map was saved but has not re-registered them as a result of loading: Custom event (ID 221)
38.760 Error ClientMultiplayerManager.cpp:98: MultiplayerManager failed: "" + multiplayer.script-event-mismatch + "
" + "
mod-LtnManager"
38.761 Info ClientMultiplayerManager.cpp:581: UpdateTick(28123446) changing state from(ConnectedLoadingMap) to(Failed)`

Could you send me a save file containing all of your enabled mods so I can test this?

Here is the save I am trying to add LTN Manager too.  I started a new blank save on the same server with the same mods and did not receive an error on join.
https://www.dropbox.com/s/znjynpwrjhzr2vw/Pepe_COVID_LIVE_HERE.zip?dl=0

Hmm, when I try to replicate it by running two instances of the game on my PC, I can't get it to happen.
Guess it's time to learn how to host a server!

:)  It's pretty easy but if you have any questions let me know.  You'll have to set the server settings .json  (../config/server-settings.json) and point the exe at the save game ( .\bin\x64\factorio --start-server "./saves/savegame.zip" --server-settings ./data/server-settings.json ) at a minimum to host an existing save game.
The server manager works well: https://github.com/mroote/factorio-server-manager  but the readme could explicitly state where to put your steam ID/pwd for the mod's to download and update.

Well... somehow it's fixed now.  I noticed from the first error log that RecipeBook was generating the same type of error, but only when I enabled LTNManager.  I disabled LTNManager and restarted a few times and then RecipeBook started generating the event manager error.  On a whim I disabled RecipeBook and enabled LTNManager (a combination I had already tried multiple times) and LTNManager now works.
I'll update this thread if I find any additional information but I had to guess it's a chance thing of events de-syncing during player join first time the mod is used?
Praise Kek a working LTN manager again!!!  Thanks raiguard!

O.o
I actually had this exact same issue with Recipe Book and Quick Item Search (one of my less popular mods), and it was being caused by the Death Markers mod. I never found the actual reason why it was erroring, just that enabling Death Markers made my event manager freak out. Maybe if I can set up my own dedicated server to use for testing I'll be able to pin it down.

Interesting.  While troubleshooting I deleted some unused (disabled) mods and one of them was Death Markers, but as it was disabled I don't know if it's related.  I tried re-enabling Recipe Book and both RB and LTNManager event handler error'ed again.  Removing RB allows LTNM to work as previously.

Does it need to be a Linux installation? I have a VM on my brother's server but it's a Windows VM.

I got it to reproduce by enabling Death Markers... I'm going to investigate and see what I can find. Though this kind of bug is extremely difficult to pin down. sigh

WTF... now I can't reproduce it at all. I have Death Markers, Recipe Book, and Quick Item Search enabled, and no dice. I do have a sneaking suspicion as to what could be causing it, though...
It involves a bit of rewriting on my translation module, but it should be doable.

I just pushed an update to RaiLuaLib. Could you please download it and see if the issue has been resolved?

I also released an LTN Manager update as well.

I get a new error when I include RecipeBook [attempt to index local 'player_data' (a nil value)].  Without RecipeBook seems to be working great.
Client and server logs: link

Whoops! I'll fix that pronto.

I pushed the update to Recipe Book. Hopefully nothing broke!

That makes sense and would explain the behavior I observed.  Fancy setup!  Thanks 1000x's for all your hard work!!!
I'm still getting (I think) the same error I got earlier with RecipeBook updated and enabled:
2020/04/11 22:28:12 Factorio Server:   83.827 Info ServerMultiplayerManager.cpp:923: updateTick(31857759) received stateChanged peerID(1) oldState(TryingToCatchUp) newState(WaitingForCommandToStartSendingTickClosures)
2020/04/11 22:28:12 Factorio Server:   83.827 Info GameActionHandler.cpp:4334: UpdateTick (31857759) processed PlayerJoinGame peerID(1) playerIndex(0) mode(connect)
2020/04/11 22:28:12 Factorio Server:   83.828 Error MainLoop.cpp:1202: Exception at tick 31857759: The mod Recipe Book (1.2.3) caused a non-recoverable error.
2020/04/11 22:28:12 [83.828 Error MainLoop.cpp:1202: Exception at tick 31857759: The mod Recipe Book (1.2.3) caused a non-recoverable error.]
2020/04/11 22:28:12 Factorio Server: Please report this error to the mod author.
2020/04/11 22:28:12 Factorio Server:
2020/04/11 22:28:12 Factorio Server: Error while running event RecipeBook::on_player_joined_game (ID 43)
2020/04/11 22:28:12 Factorio Server: RaiLuaLib/lualib/translation.lua:168: attempt to index local 'player_data' (a nil value)
2020/04/11 22:28:12 Factorio Server: stack traceback:
2020/04/11 22:28:12 Factorio Server:    RaiLuaLib/lualib/translation.lua:168: in function 'start'
2020/04/11 22:28:12 Factorio Server:    RecipeBook/control.lua:103: in function 'run_translations'
2020/04/11 22:28:12 Factorio Server:    RecipeBook/control.lua:165: in function 'handler'
2020/04/11 22:28:12 Factorio Server:    RaiLuaLib/lualib/event.lua:127: in function <RaiLuaLib/lualib/event.lua:28>
2020/04/11 22:28:12 Factorio Server:   83.828 Error ServerMultiplayerManager.cpp:90: MultiplayerManager failed: "The mod Recipe Book (1.2.3) caused a non-recoverable error.
2020/04/11 22:28:12 [83.828 Error ServerMultiplayerManager.cpp:90: MultiplayerManager failed: "The mod Recipe Book (1.2.3) caused a non-recoverable error.]
2020/04/11 22:28:12 Factorio Server: Please report this error to the mod author.
2020/04/11 22:28:12 Factorio Server:
2020/04/11 22:28:12 Factorio Server: Error while running event RecipeBook::on_player_joined_game (ID 43)
2020/04/11 22:28:12 Factorio Server: RaiLuaLib/lualib/translation.lua:168: attempt to index local 'player_data' (a nil value)
2020/04/11 22:28:12 Factorio Server: stack traceback:
2020/04/11 22:28:12 Factorio Server:    RaiLuaLib/lualib/translation.lua:168: in function 'start'
2020/04/11 22:28:12 Factorio Server:    RecipeBook/control.lua:103: in function 'run_translations'
2020/04/11 22:28:12 Factorio Server:    RecipeBook/control.lua:165: in function 'handler'
2020/04/11 22:28:12 Factorio Server:    RaiLuaLib/lualib/event.lua:127: in function <RaiLuaLib/lualib/event.lua:28>"
2020/04/11 22:28:12 Factorio Server:   83.828 Info ServerMultiplayerManager.cpp:774: updateTick(31857759) changing state from(InGame) to(Failed)
2020/04/11 22:28:12 Factorio Server:   83.829 Quitting: multiplayer error.

What the... I thought I fixed that! How strange.

Oh... frick. I did fix it because it was actually a problem with RaiLuaLib, not Recipe Book. Then I accidentally undid the change somewhere and forgot about it.
Pushing the update soon.

Sorry it took so long, I pushed the LtnManager update as well now. Hopefully everything should be good!

All is well here with Recipe Book included.  Sorry for the bag of worms and thanks again!

No problem! I'm glad we were able to get it sorted. This issue had been plaguing me for months.
And just in time too, since we just started a global Factorio library project and my translation engine will be a part of it. :D


twister: set verdict on tests after evaluating results

Do not set results before we finished parsing output.
Fixes #32514
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

I was thinking if in the part in a class below (Test(Harness)) the analog shift shouldn't be made, but I realized that there is probably some mess in the code. I've made a PR with refactoring: #33415

with this change, it could make all testcases judged as passed in generated xml files, however, the result in handler.log of "coredump.logging_backend" is still incorrect.

with this change, it could make all testcases judged as passed in generated xml files, however, the result in handler.log of "coredump.logging_backend" is still incorrect.

what exactly is not correct?

what exactly is not correct?

when I ran "coredump.logging_backend" and "coredump.backends.logging" together on frdm_k64f, the result in handler.log of "coredump.logging_backend" is as below:
  test_verify_stored_dump in 0.1 seconds
  Test suite coredump_backends succeeded
  PROJECT EXECUTION SUCCESSFUL

this is not the expected result of "coredump.logging_backend" according to testcase.yaml file, actually, it's the result of last testcase "coredump.backends.logging"

I think we should change the harness of testcase "coredump.backends.logging" to use test harness, then it could get all results including "PROJECT EXECUTION SUCCESSFUL", rather than stop at "E: #CD:END#" because it captured this expected string, then it won't influence the next case "coredump.logging_backend".

I'm not sure how it is related, but I see "coredump.logging_backend" failing on nrf5340dk_nrf5340_cpuappns. For me the output from "coredump.logging_backend" is the output of whatever was previously flashed, can you check @chen-png if this is the same for you (e.g. If you run hello_world before coredump.logging_backend, what would be the coredump.logging_backend output).
@ioannisg I haven't created new issue for the failure we're seeing as it might be related to this one ^^

@PerMac I tested to run hello_world before coredump.logging_backend on frdm_k64f board, the result were correct. And I found the result is incorrect only when I ran this two testcases "coredump.logging_backend" and "coredump.backends.logging" together.


The code editor sometimes reverts to a previous code file revision

Unfortunately I don't have exact steps to reproduce. Here's what I think I did to see this:
I created a new project
I added a new file in the new project. Exported a const from this new file.
I then opened app.js, I started writing code that imports this const, and insert it into the App component.
I may have pressed cmd + s
app.js snapped back to the original version, the one that came from opening the project, with the original wrong formatting before prettier runs on it.

Eni and I were able to reproduce this previously by following these steps (important: every time the open file was changed, it was triggered by selecting an element in the Navigator):

Create a new project
Click "Card" in the navigator to open app.js
cmd+s to save (and reformat) app.js
Add some property to the "Card" element in app.js and immediately cmd+s to save
Click "App" in the navigator to switch to storyboard.js
Make a change to storyboard.js (we were using the registerComponent function to add controls, but I highly doubt that is important), then immediately cmd+s to save
Switch back to app.js and repeat steps 4-7

After trying this enough times, we'd see one of the files revert (typically app.js).
However, this morning I have been unable to reproduce the bug, leading me to believe that it might have been fixed by us updating the version of VS Code.


InvalidProgramException

The line I copied directly from readme, throws an InvalidProgramException "Common Language Runtime detected an invalid program."
using (WordDocument document = WordDocument.Create(path)) // this line
{
    // some code
}

readme says "The main thing is - it has to work with .NET Framework 4.7.2, .NET Standard 2.0 and so on." so I assumed .NET 8.0 falls under "so on" ;) but I also tried to change my project's target framework to .NET 7.0 which is explicitly mentioned among dependencies, and it still throws the same exception. "Platform target" is "Any CPU" if that matters.
OfficeIMO.Word v 0.12.0 installed via VisualStudio nugget manager

And what happens if you get the whole project and run examples?

And what happens if you get the whole project and run examples?

There were some "file not found in C:/TEMP..." exceptions but when it comes to this particular issue, it seems to work perfectly fine.

Then you're doing something wrong when setting up project. Maybe copy one of examples and try to use them outside?

In my case, this one package was causing the issue. Uninstalling it solved the issue.
<PackageReference Include="DocumentFormat.OpenXml" Version="3.0.1" />

ye, it's newer version than we support:

https://github.com/EvotecIT/OfficeIMO/issues/192


Convert round doubles to ints in Firebase Database iOS plugin

Fixes flutter/flutter#10825

LGTM after travis is happy.
I wish we'd have a test for this.

Thanks. I hear you re: testing. If we can agree on a unit testing framework that runs on Travis I'd be very excited to write tests for the native side of plugins; I think integration testing would be useful as well.


Handle multiple output source directories - FAIL

** SHOULD NOT BE MERGED - ABANDON UPON MERGE OF: PULL#12 **
github.com/spotbugs/spotbugs-gradle-plugin/pull/12
Commit only contains the tests which verify that the fix contained within
pull request #12 fixes these issues.
These test will fail upon build. Check PR#12 to confirm that the fix
results in a succesful build.

Understood, thanks for your reproduction!


improve documentation of TunBuilder

Hi, I've just created issue #6 and was working on implementing it. Since I'm also not that versed in these detailed network technology I had to read up on some of the flags and functionalities. Because of that I have added some documentation to TunBuilder to include a little more details about what each parameter does.
Feel free to correct me on any of the added details :)

good job, thanks


Metadata Ingestion Workflow with SSL enabled in OpenMetadata server

Is some content missing, wrong or not clear?
After enabling SSL 2 steps are neccessary to run a successful metadata ingestion command via CLI:

Update the workflow section of YAML config file to the new https endpoint.
https://docs.open-metadata.org/openmetadata/connectors/database/hive/cli#metadata-ingestion

workflowConfig:
openMetadataServerConfig:
hostPort: https://localhost:8585/api

Specify an environment variable with the path to the certificate used for SSL (pam format):

export REQUESTS_CA_BUNDLE="/path/to/cacert.pem"
metadata ingest -c
Describe the solution you'd like
Enable a "certificate path" parameter in metadata command and/or document required steps in SSL configuration section:
https://docs.open-metadata.org/openmetadata/connectors/database/hive/cli

let's use this ticket for documenting the solution from the PR

This ticket was closed by: https://github.com/open-metadata/OpenMetadata/pull/10053 and https://github.com/open-metadata/OpenMetadata/pull/7502.


[SPARK-42996][CONNECT][PS][ML] Create & assign proper JIRA tickets for all failing tests.

What changes were proposed in this pull request?
This PR proposes to add comments for all pandas-on-Spark with Spark Connect failing tests, with related JIRA tickets.
Created all tickets for pandas API on Spark with Spark Connect are here: SPARK-42497
Why are the changes needed?
To assign appropriate tasks to efficiently fix the failing tests.
Does this PR introduce any user-facing change?
No, it's dev-only.
How was this patch tested?

The existing CI should pass

merged to master


how to use custom https://pay.mollie.nl/checkout/v2/css/global.css ?

I need to know how to use custom css for global.css.
I tried via {CSS}mollie_global_ito.css in module but to no avail.

This feature will be fixed in next 3.4.4 release but if you want to manually fix it fallow this steps.
1)Create custom_css.tpl in views/templates/front/ and paste this line.

2) go to mollie.php file and find function hookDisplayHeader() line: 2201.
There you will need to change everything to this:
        $this->addCSSFile($this->_path.'views/css/front.css');
        $this->context->controller->addJS($this->getPathUri() . 'views/js/apple_payment.js');
    $this->context->smarty->assign(array(
        'custom_css'   => Configuration::get(static::MOLLIE_CSS),
    ));
    return $this->display(__FILE__, 'views/templates/front/custom_css.tpl');

After this you should be good to go. Inserting links to css file will work on front-office.


certificate verify failed when REQUESTS_CA_BUNDLE environment variable is set

When the REQUESTS_CA_BUNDLE environment variable is set, pykube is unable to connect to a server:
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 346, in _make_request
    self._validate_conn(conn)
  File "/usr/lib/python3.6/site-packages/urllib3/connectionpool.py", line 850, in _validate_conn
    conn.connect()
  File "/usr/lib/python3.6/site-packages/urllib3/connection.py", line 326, in connect
    ssl_context=context)
  File "/usr/lib/python3.6/site-packages/urllib3/util/ssl_.py", line 329, in ssl_wrap_socket
    return context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/lib/python3.6/ssl.py", line 407, in wrap_socket
    _context=self, _session=session)
  File "/usr/lib/python3.6/ssl.py", line 814, in __init__
    self.do_handshake()
  File "/usr/lib/python3.6/ssl.py", line 1068, in do_handshake
    self._sslobj.do_handshake()
  File "/usr/lib/python3.6/ssl.py", line 689, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:777)

It appears that this is because the session object of the HTTPClient doesn't play nice when REQUESTS_CA_BUNDLE is set. This is apparently a known issue in requests, and it doesn't look like there's an ETA for a fix (they're pushing it off to v3.0.0):
https://github.com/requests/requests/issues/3829
I can confirm this is the case by passing the verify argument to the get method:
>>> api.session.get('https://<IP_ADDRESS>:8443')
...
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:777)
>>> api.session.get('https://<IP_ADDRESS>:8443', verify=api.session.verify)
<Response [200]>

This might be able to be worked around for now by adding the verify argument to kwargs in HTTPClient.get_kwargs. I'll investigate further and open a PR if that's the case

Never mind, I see this is already fixed in master. If anyone else comes across this issue, you can work around it (at least temporarily) by installing pykube>=0.16a1.


org.xmlresolver.Resolver error

Hi there!
When giving the ZIP distribution a trial run, I'm getting the following error:
<EMAIL_ADDRESS>) java -jar libs/docbook-xslTNG-1.5.3.jar samples/article.xml -o:article.html
Transformation failed: Failed to instantiate class org.xmlresolver.Resolver (does it have a public zero-argument constructor?)

I'm following along in the guide.
Any suggestions for what may be going wrong? Thanks in advance!

This is looking like a "me" issue, as this step works correctly for me on a different machine. It may be outside the scope of this issue, but I'm wondering if there's a way to progressively debug this. I'm trying to execute this command on a FreeBSD machine. Here's my JVM info:
<EMAIL_ADDRESS>) java -version
openjdk version "11.0.11" 2021-04-20
OpenJDK Runtime Environment (build 11.0.11+9-1)
OpenJDK 64-Bit Server VM (build 11.0.11+9-1, mixed mode)

Is there something I can try to run to work towards debugging this? @ndw If I'm thinking about this correctly, I might be able to stand up the xmlresolver separately and see if I can get that working. Does that sound approximately correct?
Thanks very much for any suggestions you might have.

Apologies for not replying sooner. I feel like I have about a thousand plates to keep spinning. There's definitely a zero argument constructor on org.xmlresolver.Resolver so my guess is that the "does it have" message is a red herring.
OpenJDK 11 could be relevant. I feel like I saw a backwards incompatibility there recently. Can you try with JDK 1.8?
Can you stand up and run the sample application, https://github.com/xmlresolver/sampleapp ?

@ndw hey, please don't apologize! I should have explicitly said: "when you have some time". And you're exactly right: using OpenJDK8 seems to have addressed the problem; i.e. article.html is generated without a hitch.
I really appreciate your time! Thanks for the help!

You can also try more recent version such as 1.15 or 1.16. I once noticed a bug related to URIs (not related to catalog resolving though, IIRC) in 1.11 that wasn’t present in previous Java versions and that has apparently been resolved in 1.13.

@gimsieke thanks for that suggestion. I'm giving the xmlresolver/sampleapp a spin with different (Open)JDKs and can report back.

I've pushed version 1.5.4 which updates the resolver dependency.


Client-Side Persistent Wrapper Caching

Add the capability to optionally cache wrappers client-side, so that the next time the application starts it can easily fetch & run the wrapper.

I think ability to cache wrapper on local-storage for browser and temp dir on node would improve experience a lot.


Latest update crashes on iOS 9.0.2

Hi,
The latest update causes crashes in the settings app for me, it would appear that the 9.2+ changes don't work with earlier versions of iOS 9.
Crash log
https://pastebin.com/UQANkRZU

This should be fixed in the latest version (3.5).


InputAutocomplete broken in .NET MAUI

Due to this issue the InputAutocomplete component will not work correctly. Clicking an item in the list does not select the item.
This is an issue in the following versions of the .NET MAUI libraries. The last known good version is 8.0.7.

8.0.10
8.0.14
8.0.20
8.0.21
8.0.40

As far as I know this is the only component that is currently affected.

Fixed in MAUI 8.0.70, released 1.1.0 for this project.


fffffff

Thank you for opening a Pull Request!

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

apologies for the inadvertent PR.   I was trying to mess with a private fork for testing purposes and sent it directly to you


Test lapi register

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 62.635%

Files with Coverage Reduction
New Missed Lines
%

pkg/leakybucket/manager_run.go
4
73.31%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
9471

Relevant Lines:
15121

💛  - Coveralls


bug: npm install -g ionic@beta npm ERR! cb() never call

From @arun0009 on April 15, 2016 19:59
Type: bug
Ionic Version: 2.x
Platform: all
When I try to run
npm install -g ionic@beta
I get an exception:
npm verb afterAdd /Users/CXI624/.npm/ionic/2.0.0-beta.25/package/package.json written
npm ERR! cb() never called!
Note: I am able to install npm install -g<EMAIL_ADDRESS>however @beta didn't work even after uninstalling ionic and clean up cache etc

Copied from original issue: driftyco/ionic#6183

Hello! Thanks for opening this issue with us! Since this issue is related to the ionic-cli repo i will be moving this issue over there. Thanks again!

Hey, not sure what the cause is, but it seems to affect a lot of packages when using node v5.9.0.  If you upgrade to v5.10.0 it appears to be fixed.
Going to close, but let us know if you're still having issues, thanks!


Gnome File Sharing doesn't work

Describe the bug
Gnomes Network sharing doesn't work
can't access the "Public" Folder from other Systems when activating Sharing in GNOME Settings
Steps To Reproduce
Steps to reproduce the behavior:

open gnome settings and enable file sharing
enter the link on another system
doesn't work

Expected behavior
i am supposed to be able to access the public folder from other devices in the same LAN
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

will in the comments once i find out who it is
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Did you ever figure this out? I'm in the same boat at the moment.

Nope, am Sorry. I used Rsync at first and then set up a NAS.

Fair enough, setting up a NAS will happen here some day too, for now it's just an external driver I move around :sweat_smile:


is statements

Something that can handle union types

Here are some ideas:
a: A | B = f()
b = is f() < A else { empty_a() }

a: A | B = f()
b = if f() as a is A { a } else { empty_a() }

a: A | B = f()
b = f() is A else empty_a()

a: A | B = f()
b = f() when A else empty_a()

b: A = f() else empty_a()

a is A

b = if f() is A else empty_a()

Some of these examples are #260. The actual is-comparator is separate and is implementable right now.


github actions: Prettier into rust.yml?

@utxo-detective can i insert the Prettier here like this? or does it go into a separate .yml?
see pr #3676 for the variant in separate prettier.yml

see also pr #3676
this approach seems to execute the "Prettier" on all files in the repo. Whereas it should only "pretty" the added/changed files in the pr.
different approch with "Pretty-Quick" in pr #3679


Link to ready-to-train preprocessed datasets not working

Is it possible to get the data on an other way or to renew the dropbox link?
Best Regards

Thanks for your interest! Which dropbox link isn't working for you? I just checked and both still seem to be active and downloadable.

thanks for the fast reply. I just checked and the dropbox link is working fine again.


Support for ExternalName Services

I've been using the Cloudflare ingress controller with services inside Kubernetes successfully for months (running 0.6.5). I'd also like to be able to use the same ingress controller to route traffic to services accessible to, but outside of, Kubernetes.
In this scenario, I want to be able to expose Vault's UI (accessible on an AWS ELB), to Kubernetes as a Service, which can then be used as a backend service for a Cloudflare ingress. This would make use of the ExternalName service: https://cloud.google.com/blog/products/gcp/kubernetes-best-practices-mapping-external-services
I believe something like this is what I'd need to do, but it seems Cloudflare can't support it yet:
apiVersion: v1
kind: Service
metadata:
  name: vault-ui
spec:
  type: ExternalName
  externalName: vault-ui.eu-west-1.elb.amazonaws.com
  ports:
  - name: https
    port: 443
    targetPort: 443
    protocol: TCP
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    kubernetes.io/ingress.class: argo-tunnel
  name: vault-ui
spec:
  rules:
  - host: vault-ui.mydomain.com
    http:
      paths:
      - path: /
        backend:
          serviceName: vault-ui
          servicePort: https

I think the reason it doesn't work yet is that a Service with ExternalName doesn't have any Endpoints; however, the DNS and port can be curl'd from within the Cloudflare ingress container. I get error: err: \"endpoints 'default/vault-ui' do not exist\""
Is this something that could be added?

Part of the ingress-controller's tunnel start/stop conditioning is a check against the existence of service endpoints, which won't exist for this type of service. Instead of using the ingress controller to do this, you should stand up a tunnel directly (a cloudflared pod that maps directly to the service)

I also got into this error bypassing from a externalName to ingress.
I created a cluster service and a manual endpoint pointing to the internal lan ip i was sharing over the ingress controller. this works in my use case now


App crash at Firebase Anaytics

<FIRAnalytics/INFO> Firebase Analytics enabled
fatal error: unexpectedly found nil while unwrapping an Optional value

Is this in the Analytics sample app?

@travoux How is classtree different from AnalyticsExample? What are the steps needed to reproduce?

@paulb777  It happened only one time. Could not reproduce it. Classtree did not use any of the analytics APIs from the sdk. If we have time, we will try to find out and let you know

@travoux It could crash at some points after Firebase Analytics was enabled but it does not mean that Firebase Analytics created the crash. It would be best to have some stack trace to pin point which crashed the app. Is your app using Swift? I looked up the crash but it seems to come mostly from Swift and Firebase Analytics is not Swift.


Release with new merged features

Hi!
Just found out there are unreleased merged features, including some new rules.
Is there any plan or help wanted to make a new release?
Thx!

Is there anything the community could do to free up some time for the maintainers? Whether it be in this project or others to get these new features and bug fixes out. @benmosher


Task: smoothly replace script.py with main.py

Describe the task
The old ETL pipeline is set up as script.py. The new one is main.py, and relies on a set of new files in the new_etl subdirectory. This task involves smoothly replacing the old pipeline with the new. This is going to be a really messy ticket until I have a chance to walk through everything that needs to happen; if you're interested, add a comment and I'll come back with more context/info.
Along the way, there's some more cleanup that can be done, including (but not limited to):

consolidating classes and imports
getting FORCE_RELOAD to load data from postgres again (not working rn)
moving vars to the env and/or config files where relevant
moving loaders.py back into featurelayer.py and converting database.py to a class instead of a function (possibly)

@rmartinsen if you're interested in tackling another issue, this would be a huge help. I'm traveling this week and don't have bandwidth to make this happen--any chance you're interested?