Do we need credit card for external services for all type of subscriptions (EA/non-EA/CSP). Can someone clarify in the documentation as to what scenarios require credit card for external services?

Can you please update the following and clarify which scenarios are for EA and non-EA customers?
https://docs.microsoft.com/en-us/azure/billing/billing-understand-your-azure-marketplace-charges#how-external-services-are-billed

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

ID: 64f88290-1358-7873-a5ca-f4510869c7dd
Version Independent ID: 03942194-fb02-17d2-1fcd-51934df5f79e
Content: Understand your Azure external service charges
Content Source: articles/billing/billing-understand-your-azure-marketplace-charges.md
Service: billing
GitHub Login: @adpick
Microsoft Alias: banders

@Praveen-410 Thanks for the questions! We will assign this issue to the content owner for further review and update as appropriate.

#in-process

@bandersmsft Any update on this?

@Praveen-410 I'm following up on this with the enginnering team.

@SadiqhAhmed-MSFT FYI.


Fix bug in how failed batteries are tracked in PowerDistributor

There are tests for this case coming with the PowerManager, so not adding them here.

This fixes a bug introduced in a previous PR.  No changes have been made since the last release.

@llucax maybe this is another possible check for flake8 - when using set().union(), the results can't be ignored.  Although not sure how easy it is.


fix: border in search field

When hovering over the search field, the 2px border makes it expand and take up more space, causing the content around it to be shifted.
Using an outline, the search field doesn't use additional space.
This is more noticiable in sites like https://sphinx-book-theme.readthedocs.io/

ping @gabalafou, this seems like something in your wheelhouse --- you may have even already done this on the feature-focus branch?

Correct. It should be fixed in #1564.
For some reason the latest Read the Docs preview build failed, but you can verify the fix in the last successful Read the Docs build

While I appreciate work done in this PR to improve the theme, I would prefer if we do not merge this code because Bootstrap uses box-shadow for a number of effects, and so to play nicely with Bootstrap and avoid double halos around elements, where you get a :hover outline and a :focus-visible box-shadow, #1564 uses box-shadow instead of outline.

Oh! I didn't see your PR before. Your fix looks better, kudos! I will close this one.

@gabalafou do we have a list of items to get into the feature-focus branch before merging it? Or more generally, what's the process by which we'll decide it is ready to merge and then do so?

There are 3 remaining PRs waiting be merged into the feature-focus branch. Once those are merged, and any failing checks are addressed, then the whole PR will be ready for review. Review should hopefully be swift, as all of the code in the PR already went through one round of review. I'm thinking that review would be giving the whole set of code changes a high-level scan, as well as some spot checking against the preview build.

Amazing - that sounds like a good plan to me.

I'm thinking that review would be giving the whole set of code changes a high-level scan, as well as some spot checking against the preview build

I agree with this take - IMO it's already been vetted through your numerous PRs already, so there should be a quick pass for any really obvious regressions, but if nothing obvious pops out we just merge it and iterate from there.


grunt wont let running

I followed the instruction the steps
unfortunately, it wont let running on localhost:3000
it said
$ grunt --force

Local Npm module "grunt-contrib-less" not found. Is it installed?
Local Npm module "grunt-node-inspector" not found. Is it installed?

why? What did I missing?

@ironheartbj18 Sounds like you need to run npm install

I am also experiencing this very problem when executing npm install (build error with "grunt-node-inspector"). A problem compiling one of it's dependencies perhaps?
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1

Using:
{ http_parser: '1.0',
  node: '0.10.28',
  v8: '<IP_ADDRESS>',
  ares: '1.9.0-DEV',
  uv: '0.10.27',
  zlib: '1.2.3',
  modules: '11',
  openssl: '1.0.1g',
  npm: '1.4.28',
  meanjs: '0.4.0' }

Argh.
"grunt-node-inspector" lacks a pre-built binary for OS X so it has to compile one.
Problem was in the copying of the files – "copy" was choking on spaces in the directory names.

@francisdaigle To clarify, did you resolve your issue?
@ironheartbj18 Does that help answer your question?

Yes I got it work finally , because I did not install "npm install -g yo"  and "npm install -g generator-meanjs", "yo meanjs" and "grunt" from http://meanjs.org/generator.html. obviously...
wondering keeping it said
>> Local Npm module "grunt-node-inspector" not found. Is it installed?

I went to https://www.npmjs.com/package/grunt-node-inspector
trying
npm install grunt-node-inspector --save-dev

the version currently is  "version": "0.2.0",
it seems not working.

Uncaught ReferenceError: angular is not defined
When I run everything with grunt it seems that the server is running, but angular is not connecting. I cannot figure out what is going on. When I go to the locahost:3000 page the page is blank and the console reads: Uncaught ReferenceError: angular is not defined
I have tried doing a bower install and npm install bower several times, but nothing is working. Can someone help me out? What am I missing?
my github is https://github.com/ironheartbj18/jobfinder.

@ironheartbj18 I just cloned your repo, ran npm install then grunt and the page loaded just fine with no errors.

@codydaig your different than mine hmm? it just showing as blank page.  I dont know which one the file to be back as normal showed as http://meanjs.herokuapp.com/ since I am very new with mean stack to get started.
its just showing like this...

What node and npm version are you running?

@ironheartbj18 Also, did you use the generator or clone the master branch from the project?

@codydaig  I checked node -v its a 0.12.3 the currently is Current Version: v0.12.7. I m going to fork or clone the master branch from them.

I did not clone the master branch from the project. I solved it. When I looked at the '.bowerrc'
{
	"directory": "public/lib"
}

so I created a new folder and type $bower install and $npm install -g bower (upgrade to : "1.5.2") it seem working now. just strange... maybe I did not create the folder as for lib. it was not existed.


DEV2-2518 fix publish jobs

Issue
the output of ./gradlew -q currentVersion is like so (idk when it was changed):
➜  ./gradlew -q currentVersion
channelName: alpha
0.8.13

you can see here that the job thinks that ${{ steps.get_build_version.outputs.build_version }} == channelName: alpha:

Fix
add | tail -n1 to the end of ./gradlew -q currentVersion
this takes the last line of the output.
Its also idempotent, i.e. if there's only one line, it'll take this line.
You can see that if I run tail -n1 twice it still outputs the correct version:
➜  CURRENT_VERSION=$(./gradlew -q currentVersion | tail -n1)
➜  echo $CURRENT_VERSION | tail -n1
0.8.13

Why do we need the channel name in the output? Wasn't it just a debug message that was accidentally left there?

@amircodota is fixing in another pr


holding down backspace crashes 0.8 iOS

reported by @Krishma-Bawa

https://github.com/lockbook/lockbook/assets/821972/80274bd4-2996-4d31-9c0a-866f9d9779b1

I was able to reproduce on mobile but not on the iOS simulator. Is there a way to view logs on my device? I also have to check if I have the USB-C to lightning cable to deploy to my phone

confirmed broken in 0.8 and fixed on master


Uptake latest SSI SDK

Uptake https://github.com/TBD54566975/ssi-sdk/commit/6d87cd19531450238b82e48f731f4ff7dff9e2e4

Codecov Report

Merging #385 (a1bc20d) into main (8aac18d) will not change coverage.
The diff coverage is 76.92%.

@@           Coverage Diff           @@
##             main     #385   +/-   ##
=======================================
  Coverage   20.42%   20.42%
=======================================
  Files          48       48
  Lines        5376     5376
=======================================
  Hits         1098     1098
  Misses       4114     4114
  Partials      164      164

Impacted Files
Coverage Δ

pkg/server/router/manifest.go
9.96% <ø> (ø)

pkg/server/router/presentation.go
3.46% <0.00%> (ø)

pkg/service/credential/storage.go
1.68% <0.00%> (ø)

internal/keyaccess/jwt.go
44.62% <90.00%> (ø)

internal/keyaccess/dataintegrity.go
59.09% <100.00%> (ø)

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


Add support to python3

Used six package.
Fixed some pep8 issues.

Thanks @drorasaf. Sorry for the late response, but I'm merging this now and will be releasing a new version on PyPI the next few days.


Integrate schnelltesttest.de into the Corona-Warn-App

Feature description
https://schnelltesttest.de offers the option to scan the barcode printed on the packaging of rapid antigen tests for self usage. After scanning the barcode, the site shows the BfArM rating for this test (how well the test can detect an infection, etc.)
Problem and motivation
To help users make an informed decision which test to buy, the Corona-Warn-App should help by showing how precise the test is.
In the past there were discussions on whether rapid antigen tests (for self usage) are really working and that only some are able to detect an infection reliably. This unsettled many people and CWA could help, with this feature, to restore trust in the tests.
Additional information
https://schnelltesttest.de is provided by https://zerforschung.org (https://github.com/zerforschung).
The project is Open-Source, the code is available here: https://github.com/zerforschung/schnelltesttest.de

Internal Tracking ID: EXPOSUREAPP-11505

Thanks for your suggestion @Ein-Tim. We have created an internal ticekt for it and will raise this topic internally.
Internal Tracking ID: EXPOSUREAPP-11505

Corona-Warn-App Open Source Team

@bunchofsage Only in an EU digital test certificate the test used is shown.

@bunchofsage as @Ein-Tim already pointed out, the EU certificate contains the test name. I've attached two examples of two tests I took last year that show the test name. From there one you can simply use a website like schnelltesttest.de or the PEI sensitivity document to check the test's sensitivity by searching for its name.
picture 1
picture 2

schnelltesttest.de is now referred to in coronawarn.app/en/faq/#schnelltesttest.

The website https://schnelltesttest.de has a disclaimer on top saying that it is not updated anymore, due to changes in the way how data is published. Also, as the CWA project went into ramp-down mode, I don't expect this feature to be implemented. I'm therefore closing this issue.


v1.1.2 has tests dir export ignored but needed for patches

In v1.1.2, the tests dir is export-ignore in the .gitattributes file.  But, in the composer.json, the files in the tests folder are needed to run the patches.  Can you either remove the tests dir from .gitattributes or remove the patch files from composer.json?

@dystopianblue: Is the issue that was accidentally added as a dependency? See fix in: https://github.com/predishq/predis/commit/6dd55d7f9661b135964d6df49e1a126d7d1351aa
I assume you can't Composer install currently, or what's the issue?

Yes, when attempting to install predis in composer for Drupal, I get this error:
  - Installing phpunit/phpunit (7.5.20): Loading from cache
  - Applying patches for phpunit/phpunit
    ./tests/phpunit_php7.patch (Fix PHP 7 compatibility)
   Could not apply patch! Skipping. The error was: The "./tests/phpunit_php7.patch" file could not be downloaded: failed to open stream: No such file or directory

Does installing v1.1.x-dev work for you?

Secondly, if you set composer-exit-on-patch-failure to false what's your output when installing the package?

@tillkruss Both v1.1.1 and 1.1.x-dev work.  When I set composer-exit-on-patch-failure to false at ^1.1, I get this:
  - Installing phpunit/phpunit (7.5.20): Loading from cache
  - Applying patches for phpunit/phpunit
    ./tests/phpunit_php7.patch (Fix PHP 7 compatibility)
   Could not apply patch! Skipping. The error was: The "./tests/phpunit_php7.patch" file could not be downloaded: failed to open stream: No such file or directory
    ./tests/phpunit_php8.patch (Fix PHP 8 compatibility)
   Could not apply patch! Skipping. The error was: The "./tests/phpunit_php8.patch" file could not be downloaded: failed to open stream: No such file or directory

FYI I also have "cweagans/composer-patches" as a require in my composer

Okay, can you post your full composer.json to double check this?

I can confirm I am getting the same errors with each of the PHP patches.
The patch looks for this file /src/Util/Getopt.php
However I cannot see this file in https://github.com/predishq/predis/tree/v1.1.2/src
I can also confirm there isn't a patching issue with v1.1.1

@joz-d7 when installing v1.1.x-dev instead, does that resolve the issue for you?

@dystopianblue @joz-d7 composer install --prefer-source might help also.
@tillkruss I suggest releasing a patch version for v1.1 with the dependencies issue fix

@b1rdex: Yeah, just waiting for confirmation that this is resolved in v1.1.x-dev.

I am not getting errors on 1.1.x-dev where I previously had issues. I assume there has been some changes made?

Great, we'll tag 1.1.3 soon. The change was made here: https://github.com/predis/predis/commit/6dd55d7f9661b135964d6df49e1a126d7d1351aa

Still occurs on second composer update with 1.1.3. I think this is due to the code only being checked out for predis/predis the first time, but the second time it tries to patch phpunit but can't find the files which are in vendor/predis/predis/test.
simon@t580:% composer update -n
Gathering patches for root package.
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 0 installs, 2 updates, 0 removals
Gathering patches for root package.
Gathering patches for dependencies. This might take a minute.
  - Updating predis/predis (v1.1.1 => v1.1.3):  Checking out 2ce537d75e
Package container-interop/container-interop is abandoned, you should avoid using it. Use psr/container instead.
Package phpunit/php-token-stream is abandoned, you should avoid using it. No replacement was suggested.
Writing lock file
Generating autoload files
composer/package-versions-deprecated: Generating version class...
composer/package-versions-deprecated: ...done generating version class
53 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
Updating Shepherd scaffold files.
Creating necessary directories.
Creating settings.php file if not present.
Removing write permissions on settings files.
11:44:40 simon@t580:% composer update
    1/1:	http://repo.packagist.org/p/provider-latest$cb7bd9d33a179c2b36c5143866752ee3bc85120b0cec89b098eb9943941a1255.json
    Finished: success: 1, skipped: 0, failure: 0, total: 1
Gathering patches for root package.
Removing package phpunit/phpunit so that it can be re-installed and re-patched.
  - Removing phpunit/phpunit (7.5.20)
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 1 install, 1 update, 0 removals
Gathering patches for root package.
Gathering patches for dependencies. This might take a minute.
  - Installing phpunit/phpunit (7.5.20): Loading from cache
  - Applying patches for phpunit/phpunit
    ./tests/phpunit_php7.patch (Fix PHP 7 compatibility)
Deprecation Notice: strpos(): Non-string needles will be interpreted as strings in the future. Use an explicit chr() call to preserve the current behavior in phar:///home/simon/bin/composer/src/Composer/Util/RemoteFilesystem.php:234
Deprecation Notice: strpos(): Non-string needles will be interpreted as strings in the future. Use an explicit chr() call to preserve the current behavior in phar:///home/simon/bin/composer/src/Composer/Util/RemoteFilesystem.php:234
   Could not apply patch! Skipping. The error was: The "./tests/phpunit_php7.patch" file could not be downloaded: failed to open stream: No such file or directory

  [Exception]
  Cannot apply patch Fix PHP 7 compatibility (./tests/phpunit_php7.patch)!

update [--prefer-source] [--prefer-dist] [--dry-run] [--dev] [--no-dev] [--lock] [--no-custom-installers] [--no-autoloader] [--no-scripts] [--no-progress] [--no-suggest] [--with-dependencies] [--with-all-dependencies] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--ignore-platform-reqs] [--prefer-stable] [--prefer-lowest] [-i|--interactive] [--root-reqs] [--] [<packages>]...

composer install does the same thing.
I think its because the patches are in the composer.json for predis, but trying to patch phpunit, so probably need more path information?

@singularo could you please try removing vendors folder (rm -rf vendor/) and running composer update again? Please attach composer.json and composer.lock if the error still occurs.

I'd also try executing composer using the -vvv flag for maximum verbosity, maybe we can get more details out of it.

FWIW, had the issue, but adding a "conflict" : { "predis/predis": "< 1.1.4" } fixed it for me, thanks for the updates.
My project does not directly depend on predis, but it brought in by vimeo/vimeo-api, so adding a conflict felt more appropriate than a direct dependency.

I tried all of the suggestions above as I was installing https://gitlab.com/weitzman/drupal-test-traits on a project. The only thing that worked was pegging predis to 1.1.1. After remove vendor and composer.lock and running composer install, everything worked fine.
If I did composer require predis/predis:^1.1.3 and repeated the above steps, I was still seeing the patch error when phpunit was attempting to be installed.
Steps I was running:

composer remove predis/predis
composer require predis/predis:^1.1.3
composer require weitzman/drupal-test-traits (or whichever package is dependent on phpunit)
Path error occurred
composer remove predis/predis
composer require predis/predis:1.1.1
composer require weitzman/drupal-test-traits (or whichever package is dependent on phpunit)
No errors
Remove vendor and composer.lock
composer install to confirm no errors when it builds remotely


Remaining evac time in comms ui has excessive precision displayed.

Description
Lots of decimal places in countdown now, see screencap.
Reproduction

Call shuttle
Check comms console.

Screenshots

Additional context
NA

Duplicate
https://github.com/space-wizards/space-station-14/issues/30279

Duplicate of #30279


chore: bump go version to 1.22 (#4590)

Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description
Upgrading Go version to 1.22

There might be a solution here: https://github.com/github/codeql/issues/15647

The biggest change in go 1.22 is the variable scoping in for loops. While the code should work as expected, whenever the variable is deliberately not scoped inside the iteration, should be adjusted, as with go 1.22 that variable will have the scope in iteration. I do not know of such case in the bee codebase, but it should be good to validate by other reviewers, as well. Have in mind that for loop variable scoping in 1.22 have a lot of strange consequences that may not be obvious and are very indirect.
It appears that CodeQL analyzer is a blocker for this change https://github.com/ethersphere/bee/actions/runs/9160600677/job/25183926395?pr=4687#step:4:103 as it does not support go 1.22, which is unfortunate.

There is tooling support we should look into.
After running go build -gcflags=all=-d=loopvar=2 ./cmd/bee, this is what I got:
# runtime
/usr/local/go/src/runtime/proc.go:1907:7: loop variable freem now per-iteration, stack-allocated
/usr/local/go/src/runtime/proc.go:3269:7: loop variable enum now per-iteration, stack-allocated
/usr/local/go/src/runtime/mgcmark.go:810:6: loop variable d now per-iteration, stack-allocated
/usr/local/go/src/runtime/traceback.go:633:7: loop variable iu now per-iteration, stack-allocated
/usr/local/go/src/runtime/traceback.go:990:7: loop variable iu now per-iteration, stack-allocated
# vendor/golang.org/x/net/dns/dnsmessage
/usr/local/go/src/vendor/golang.org/x/net/dns/dnsmessage/message.go:1161:10: loop variable q now per-iteration, stack-allocated
/usr/local/go/src/vendor/golang.org/x/net/dns/dnsmessage/message.go:1168:10: loop variable a now per-iteration, stack-allocated
/usr/local/go/src/vendor/golang.org/x/net/dns/dnsmessage/message.go:1175:10: loop variable a now per-iteration, stack-allocated
/usr/local/go/src/vendor/golang.org/x/net/dns/dnsmessage/message.go:1182:10: loop variable a now per-iteration, stack-allocated
/usr/local/go/src/vendor/golang.org/x/net/dns/dnsmessage/message.go:2633:9: loop variable o now per-iteration, stack-allocated
# runtime/pprof
/usr/local/go/src/runtime/pprof/pprof.go:386:9: loop variable r now per-iteration, heap-allocated
/usr/local/go/src/runtime/pprof/proto.go:363:6: loop variable e now per-iteration, stack-allocated
/usr/local/go/src/runtime/pprof/proto.go:612:9: loop variable frame now per-iteration, stack-allocated
/usr/local/go/src/runtime/pprof/protomem.go:29:9: loop variable r now per-iteration, heap-allocated
# github.com/ethereum/go-ethereum/rlp
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/rlp/encbuffer.go:68:9: loop variable head now per-iteration, stack-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/rlp/encbuffer.go:84:9: loop variable head now per-iteration, stack-allocated
# net
/usr/local/go/src/net/cgo_unix.go:74:9: loop variable addr now per-iteration, stack-allocated
/usr/local/go/src/net/dnsclient_unix.go:567:9: loop variable ip now per-iteration, stack-allocated
/usr/local/go/src/net/interface_linux.go:27:9: loop variable m now per-iteration, stack-allocated
/usr/local/go/src/net/interface_linux.go:150:9: loop variable m now per-iteration, stack-allocated
/usr/local/go/src/net/sockopt_posix.go:28:9: loop variable ifi now per-iteration, heap-allocated
/usr/local/go/src/net/interface.go:147:9: loop variable ifi now per-iteration, heap-allocated (loop inlined into /usr/local/go/src/net/interface.go:139)
/usr/local/go/src/net/interface.go:147:9: loop variable ifi now per-iteration, heap-allocated (loop inlined into /usr/local/go/src/net/interface_linux.go:159)
# github.com/ethereum/go-ethereum/p2p/netutil
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:93:9: loop variable net now per-iteration, stack-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:130:9: loop variable net now per-iteration, stack-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:130:9: loop variable net now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:144)
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:130:9: loop variable net now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:146)
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:130:9: loop variable net now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:156)
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:130:9: loop variable net now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/p2p/netutil/net.go:158)
# crypto/x509
/usr/local/go/src/crypto/x509/x509.go:1208:11: loop variable name now per-iteration, stack-allocated
/usr/local/go/src/crypto/x509/x509.go:1220:11: loop variable ipNet now per-iteration, stack-allocated
/usr/local/go/src/crypto/x509/x509.go:1228:11: loop variable email now per-iteration, stack-allocated
/usr/local/go/src/crypto/x509/x509.go:1240:11: loop variable uriDomain now per-iteration, stack-allocated
/usr/local/go/src/crypto/x509/verify.go:856:9: loop variable ext now per-iteration, heap-allocated
/usr/local/go/src/crypto/x509/verify.go:871:10: loop variable ext now per-iteration, heap-allocated
# github.com/casbin/casbin/v2/rbac/default-role-manager
../../go/pkg/mod/github.com/casbin/casbin/v2@v2.35.0/rbac/default-role-manager/role_manager.go:103:10: loop variable domain now per-iteration, stack-allocated
../../go/pkg/mod/github.com/casbin/casbin/v2@v2.35.0/rbac/default-role-manager/role_manager.go:195:10: loop variable domain now per-iteration, stack-allocated
# github.com/gogo/protobuf/proto
../../go/pkg/mod/github.com/gogo/protobuf@v1.3.2/proto/extensions_gogo.go:163:9: loop variable e now per-iteration, stack-allocated
../../go/pkg/mod/github.com/gogo/protobuf@v1.3.2/proto/extensions_gogo.go:179:9: loop variable e now per-iteration, stack-allocated
# github.com/ethereum/go-ethereum/accounts/abi
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/accounts/abi/abi.go:229:9: loop variable errABI now per-iteration, heap-allocated
# github.com/casbin/casbin/v2/model
../../go/pkg/mod/github.com/casbin/casbin/v2@v2.35.0/model/model.go:306:13: loop variable assertion now per-iteration, stack-allocated
# crypto/tls
/usr/local/go/src/crypto/tls/common.go:1146:9: loop variable cert now per-iteration, heap-allocated
/usr/local/go/src/crypto/tls/ticket.go:121:10: loop variable extra now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/ticket.go:143:10: loop variable chain now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/ticket.go:150:12: loop variable cert now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/ticket.go:374:9: loop variable key now per-iteration, heap-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:195:12: loop variable proto now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:233:12: loop variable ks now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:268:12: loop variable psk now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:276:12: loop variable binder now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:355:11: loop variable binder now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:693:12: loop variable sct now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:1186:14: loop variable ca now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:1389:7: loop variable i now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:1389:10: loop variable cert now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_messages.go:1411:15: loop variable sct now per-iteration, stack-allocated
/usr/local/go/src/crypto/tls/handshake_client.go:1099:9: loop variable chain now per-iteration, heap-allocated
/usr/local/go/src/crypto/tls/handshake_server_tls13.go:186:10: loop variable ks now per-iteration, heap-allocated
# internal/profile
/usr/local/go/src/internal/profile/encode.go:251:10: loop variable l now per-iteration, stack-allocated
/usr/local/go/src/internal/profile/encode.go:285:9: loop variable i now per-iteration, stack-allocated
# go.opencensus.io/trace/tracestate
../../go/pkg/mod/go.opencensus.io@v0.24.0/trace/tracestate/tracestate.go:67:13: loop variable entry now per-iteration, heap-allocated (loop inlined into ../../go/pkg/mod/go.opencensus.io@v0.24.0/trace/tracestate/tracestate.go:78)
../../go/pkg/mod/go.opencensus.io@v0.24.0/trace/tracestate/tracestate.go:105:9: loop variable entry now per-iteration, heap-allocated (loop inlined into ../../go/pkg/mod/go.opencensus.io@v0.24.0/trace/tracestate/tracestate.go:128)
# google.golang.org/protobuf/internal/impl
../../go/pkg/mod/google.golang.org/protobuf@v1.32.0/internal/impl/message_reflect.go:238:10: loop variable x now per-iteration, stack-allocated
../../go/pkg/mod/google.golang.org/protobuf@v1.32.0/internal/impl/encode.go:156:9: loop variable x now per-iteration, stack-allocated
../../go/pkg/mod/google.golang.org/protobuf@v1.32.0/internal/impl/encode.go:177:10: loop variable x now per-iteration, stack-allocated
../../go/pkg/mod/google.golang.org/protobuf@v1.32.0/internal/impl/merge.go:60:12: loop variable sx now per-iteration, stack-allocated
../../go/pkg/mod/google.golang.org/protobuf@v1.32.0/internal/impl/checkinit.go:69:9: loop variable x now per-iteration, stack-allocated
# gopkg.in/yaml.v2
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:81:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1265:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1322:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1378:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1435:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1605:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1645:6: loop variable i now per-iteration, stack-allocated
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:81:6: loop variable i now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1227)
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:81:6: loop variable i now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1237)
../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:81:6: loop variable i now per-iteration, stack-allocated (loop inlined into ../../go/pkg/mod/gopkg.in/yaml.v2@v2.4.0/emitterc.go:1251)
# golang.org/x/text/language
../../go/pkg/mod/golang.org/x/text@v0.14.0/language/coverage.go:126:10: loop variable t now per-iteration, stack-allocated
../../go/pkg/mod/golang.org/x/text@v0.14.0/language/match.go:382:9: loop variable tag now per-iteration, stack-allocated
../../go/pkg/mod/golang.org/x/text@v0.14.0/language/match.go:391:9: loop variable tag now per-iteration, stack-allocated
../../go/pkg/mod/golang.org/x/text@v0.14.0/language/match.go:455:9: loop variable ww now per-iteration, stack-allocated
../../go/pkg/mod/golang.org/x/text@v0.14.0/language/match.go:495:10: loop variable t now per-iteration, stack-allocated
# encoding/xml
/usr/local/go/src/encoding/xml/typeinfo.go:81:13: loop variable finfo now per-iteration, stack-allocated
# go.opencensus.io/trace
../../go/pkg/mod/go.opencensus.io@v0.24.0/trace/spanstore.go:130:10: loop variable b now per-iteration, stack-allocated
# resenje.org/web
../../go/pkg/mod/resenje.org/web@v0.4.3/auth.go:127:10: loop variable network now per-iteration, stack-allocated
../../go/pkg/mod/resenje.org/web@v0.4.3/auth.go:136:10: loop variable network now per-iteration, stack-allocated
# github.com/uber/jaeger-client-go/utils
../../go/pkg/mod/github.com/uber/jaeger-client-go@v2.24.0+incompatible/utils/localip.go:63:9: loop variable iface now per-iteration, stack-allocated
# github.com/koron/go-ssdp/internal/multicast
../../go/pkg/mod/github.com/koron/go-ssdp@v0.0.4/internal/multicast/interface.go:30:9: loop variable ifi now per-iteration, stack-allocated
../../go/pkg/mod/github.com/koron/go-ssdp@v0.0.4/internal/multicast/multicast.go:70:9: loop variable ifi now per-iteration, stack-allocated
../../go/pkg/mod/github.com/koron/go-ssdp@v0.0.4/internal/multicast/multicast.go:116:9: loop variable ifi now per-iteration, heap-allocated
# github.com/google/gopacket/routing
../../go/pkg/mod/github.com/google/gopacket@v1.1.19/routing/routing.go:166:9: loop variable m now per-iteration, stack-allocated
../../go/pkg/mod/github.com/google/gopacket@v1.1.19/routing/routing.go:217:9: loop variable iface now per-iteration, stack-allocated
# github.com/huin/goupnp
../../go/pkg/mod/github.com/huin/goupnp@v1.3.0/network.go:50:9: loop variable iface now per-iteration, stack-allocated
# github.com/libp2p/go-netroute
../../go/pkg/mod/github.com/libp2p/go-netroute@v0.2.1/netroute_linux.go:52:9: loop variable m now per-iteration, stack-allocated
../../go/pkg/mod/github.com/libp2p/go-netroute@v0.2.1/netroute_linux.go:103:9: loop variable iface now per-iteration, stack-allocated
# github.com/miekg/dns
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/types.go:1513:9: loop variable p now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/msg.go:797:9: loop variable r now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/msg.go:919:10: loop variable r now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/msg.go:983:9: loop variable r now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/ztypes.go:308:9: loop variable x now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/ztypes.go:841:9: loop variable e now per-iteration, stack-allocated
../../go/pkg/mod/github.com/miekg/dns@v1.1.58/msg_truncate.go:66:9: loop variable r now per-iteration, stack-allocated
# github.com/uber/jaeger-client-go
../../go/pkg/mod/github.com/uber/jaeger-client-go@v2.24.0+incompatible/span.go:349:11: loop variable ld now per-iteration, stack-allocated
../../go/pkg/mod/github.com/uber/jaeger-client-go@v2.24.0+incompatible/jaeger_thrift_span.go:70:9: loop variable tag now per-iteration, stack-allocated
# github.com/libp2p/go-libp2p/p2p/net/upgrader
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/net/upgrader/upgrader.go:247:9: loop variable m now per-iteration, heap-allocated (loop inlined into ../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/net/upgrader/upgrader.go:240)
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/net/upgrader/upgrader.go:247:9: loop variable m now per-iteration, heap-allocated (loop inlined into ../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/net/upgrader/upgrader.go:259)
# github.com/prometheus/client_golang/prometheus
../../go/pkg/mod/github.com/prometheus/client_golang@v1.18.0/prometheus/metric.go:181:10: loop variable e now per-iteration, stack-allocated
../../go/pkg/mod/github.com/prometheus/client_golang@v1.18.0/prometheus/expvar_collector.go:44:12: loop variable desc now per-iteration, stack-allocated
../../go/pkg/mod/github.com/prometheus/client_golang@v1.18.0/prometheus/go_collector_latest.go:198:9: loop variable d now per-iteration, stack-allocated
# github.com/klauspost/compress/huff0
../../go/pkg/mod/github.com/klauspost/compress@v1.17.6/huff0/compress.go:538:9: loop variable v now per-iteration, stack-allocated
../../go/pkg/mod/github.com/klauspost/compress@v1.17.6/huff0/compress.go:553:9: loop variable v now per-iteration, stack-allocated
# github.com/libp2p/go-nat
../../go/pkg/mod/github.com/libp2p/go-nat@v0.2.0/natpmp.go:69:9: loop variable iface now per-iteration, stack-allocated
../../go/pkg/mod/github.com/libp2p/go-nat@v0.2.0/upnp.go:30:10: loop variable dev now per-iteration, stack-allocated
../../go/pkg/mod/github.com/libp2p/go-nat@v0.2.0/upnp.go:87:10: loop variable dev now per-iteration, stack-allocated
../../go/pkg/mod/github.com/libp2p/go-nat@v0.2.0/upnp.go:306:9: loop variable iface now per-iteration, stack-allocated
# github.com/prometheus/statsd_exporter/pkg/mapper
../../go/pkg/mod/github.com/prometheus/statsd_exporter@v0.22.7/pkg/mapper/mapper.go:334:9: loop variable mapping now per-iteration, heap-allocated
# github.com/libp2p/go-libp2p/p2p/host/eventbus
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/host/eventbus/basic.go:245:6: loop variable i now per-iteration, stack-allocated
# github.com/libp2p/go-libp2p/p2p/host/resource-manager
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/host/resource-manager/limit_defaults.go:428:9: loop variable l now per-iteration, stack-allocated
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/host/resource-manager/limit_defaults.go:500:9: loop variable l now per-iteration, stack-allocated
# github.com/libp2p/go-libp2p/p2p/net/swarm
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/net/swarm/swarm_dial.go:435:9: loop variable addr now per-iteration, stack-allocated
# github.com/klauspost/compress/zstd
../../go/pkg/mod/github.com/klauspost/compress@v1.17.6/zstd/fse_decoder.go:277:9: loop variable v now per-iteration, stack-allocated
# github.com/ethereum/go-ethereum/core/types
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/core/types/bloom9.go:110:11: loop variable b now per-iteration, heap-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/core/types/bloom9.go:124:10: loop variable b now per-iteration, heap-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/core/types/gen_log_rlp.go:13:9: loop variable _tmp2 now per-iteration, stack-allocated
# github.com/quic-go/quic-go
../../go/pkg/mod/github.com/quic-go/quic-go@v0.42.0/server.go:511:10: loop variable p now per-iteration, stack-allocated
# github.com/ethereum/go-ethereum/accounts
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/accounts/manager.go:249:9: loop variable wallet now per-iteration, stack-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/accounts/manager.go:263:9: loop variable wallet now per-iteration, stack-allocated
# github.com/ethereum/go-ethereum/signer/core/apitypes
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/signer/core/apitypes/types.go:629:9: loop variable field now per-iteration, stack-allocated
../../go/pkg/mod/github.com/ethereum/go-ethereum@v1.13.4/signer/core/apitypes/types.go:713:10: loop variable typeObj now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/bzz
pkg/bzz/utilities.go:9:9: loop variable v now per-iteration, stack-allocated
# github.com/libp2p/go-libp2p/p2p/transport/webtransport
../../go/pkg/mod/github.com/libp2p/go-libp2p@v0.33.2/p2p/transport/webtransport/cert_manager.go:183:9: loop variable certHash now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/transaction
pkg/transaction/transaction.go:621:9: loop variable abiError now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/storer/internal/cache
pkg/storer/internal/cache/cache.go:271:9: loop variable addr now per-iteration, stack-allocated
pkg/storer/internal/cache/cache.go:289:10: loop variable addr now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/file/joiner
pkg/file/joiner/joiner.go:383:6: loop variable i now per-iteration, stack-allocated
pkg/file/joiner/joiner.go:383:9: loop variable addr now per-iteration, heap-allocated
# github.com/ethersphere/bee/v2/pkg/storer/migration
pkg/storer/migration/reserveRepair.go:111:7: loop variable i now per-iteration, stack-allocated
pkg/storer/migration/step_05.go:27:8: loop variable item now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/storer
pkg/storer/reserve.go:183:9: loop variable batchID now per-iteration, stack-allocated
pkg/storer/compact.go:54:6: loop variable shard now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/pkg/puller
pkg/puller/puller.go:457:6: loop variable bin now per-iteration, stack-allocated
# github.com/ethersphere/bee/v2/cmd/bee/cmd
cmd/bee/cmd/db.go:521:11: loop variable root now per-iteration, stack-allocated
./pkg/storer/migration/reserveRepair.go:111:7: loop variable migration.i now per-iteration, stack-allocated

I do not know how to interepet the above yet but it's a start

I do not know how to interepet the above yet but it's a start

That is indeed a nice start, especially as there are not so many reported for loops under pkg/. It is nice that every module as dependency declares the supported go version and the for loop scoping will be used according to it, so we do not have to care about the dependencies when we are upgrading go version in our own module (bee).


Wait for dialog to be closed in E2E tests

In this PR we are waiting for dialogs to be closed in orbit_capture_repeatedly and orbit_symbol_locations, so Alt+F4 close Orbit and not the dialogs.
For capture_repeatedly we are waiting for the capture option button to be enabled again
(and reverting https://github.com/google/orbit/pull/4173) and for symbol_locations we are waiting
for the ui dialog to be None.
Test: Run capture_repeatedly locally, trigger E2E test in the signed build.
Bug: http://b/246302999

Offline discussion: Symbol_locations issue will be fixed in https://github.com/google/orbit/pull/4226

Tests passed in http://sponge.corp.google.com/target?id=175663d0-a03f-575e-b008-5cc49cb41ae4&target=//chrome/cloudcast/tools/cli/internal/integrationtest/testrunner/tests:cloudcollector_test&showTestCases=ALL


Tutorial Doc Step 9

In tutorial doc in step 9, doesn't exists information about what file need be to created.

This has been addressed in PR #1212. Can this be closed now?

Closed


fix: MySQL plugin: add check for null column value for JSON column type

Description

Add check for column value being null when column type is JSON. This check was mistakenly not added before. Adding it now as part of this PR.
Add JUnit TC.

PR fixes following issue(s)
Fixes #27645
Media

Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[x] JUnit

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

@sumitsum Can u please generate a DP link for this PR

/build-deploy-preview skip-tests=true


Get an error when I try to "undo" copied-pasted node.

Describe the Bug
I use Collaborative, undo-redo, and copy-paste functionality, so I had to modify a bit of code which is in the examples. There is an error from some files under the hood of the library. So If I add a new node and press the "undo" button all works excellent. But If I copy and paste a new node, and after trying to "undo" the pasted node I get an error. I Will share the video with the problem, but I do not know if can I share the code, coz this is a paid functionality.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

add node
copy node
paste node
press the "undo" button
get error

Expected behavior
remove a pasted node from the chart.
Screenshots or Videos
https://github.com/wbkd/react-flow/assets/64188262/cc834de0-5ca6-4175-8c15-8d1737ba2379
Platform

OS: [e.g. macOS]
Browser: [e.g. Chrome, Safari]

Additional context
No response

If this can help, I take the previous past state where I have only one node and update the current nodes. nodes are updated as we can see there were 2 nodes before and only one left. But I can't get where this error comes from

after some debugging, I saw that the node was removed from the layout, and "nextNodeInternals" became undefined, and the error came only after that.

We revised the pro examples. Could you check if the issue persists? There was an issue with the copy/paste example because we didn't reset the node when it got pasted. You need to set dragging: false / selected: false as seen in the current example. Does this help?

Hi, Apologies for missing your message earlier. I'll review it shortly and get back to you as soon as possible. Thank you.

Sorry, I think we can close this issue, you are right. I reset the selected state to false after pasting and do the same, before pressing the back button. Coz if you select the pasted node manually and press back, you will see the same issue.


increase number of colored divs in layout

This PR addresses issue #62
Colored div limit was 20 before increased it to 50.

Sorry for your trouble. In digging in more, I've decided I don't want to support this many divs, I believe it will cause confusion in how grid layouts can be made. Thanks!


Don't anonymize local IP addresses

IP anonymization now skips IP addresses obtained from the local machine.

This change is

Pull Request Test Coverage Report for Build 714

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 84.767%

Totals

Change from base Build 710:
0.1%

Covered Lines:
1074

Relevant Lines:
1267

💛  - Coveralls

PTAL


meta: Install dev tools in devcontainer

summary

[x] Install grpcurl
[x] Using docker-compose instead of Dockerfile for devcontainer
[x] Add volume mount parameters: cached as a performance improvement  for macos, windows

grpcurl がインストールされていることを確認

grpcurl -h
Usage:
	grpcurl [flags] [address] [list|describe] [symbol]

The 'address' is only optional when used with 'list' or 'describe' and a
protoset or proto flag is provided.

If 'list' is indicated, the symbol (if present) should be a fully-qualified
service name. If present, all methods of that service are listed. If not
present, all exposed services are listed, or all services defined in protosets.

If 'describe' is indicated, the descriptor for the given symbol is shown. The
symbol should be a fully-qualified service, enum, or message name. If no symbol
is given then the descriptors for all exposed or known services are shown.

If neither verb is present, the symbol must be a fully-qualified method name in
'service/method' or 'service.method' format. In this case, the request body will
be used to invoke the named method. If no body is given but one is required
(i.e. the method is unary or server-streaming), an empty instance of the
method's request type will be sent.

The address will typically be in the form "host:port" where host can be an IP
address or a hostname and port is a numeric port or service name. If an IPv6
address is given, it must be surrounded by brackets, like "[2001:db8::1]". For
Unix variants, if a -unix=true flag is present, then the address must be the
path to the domain socket.

Available flags:

LGTM


Bloom r us

chore(*): update parquet file format version
feat(bloomfilter): add sbbf
fix(types): ensure number coercion
WIP - add the ability to write bloom filters to parquet file

Hey, @enddynayn, I noticed you closed this but no merge is indicated here.  Did this work make it to the right upstream location via some other route?
Best regards,
-Karl

@kfogel yes we forked the repository and got merged here https://github.com/LibertyDSNP/parquetjs.

Thanks, @enddynayn.  Glad to hear it, and now the loop is closed here!


Nested Batches

Hello there,
I wanted to know if and how it is currently possible to use the BatchMapping with only one request for this situation:
Basically currently when mapping on type extension and field data it requested data from my server twice. Is it possible in this situation to avoid a second call and call everything in one go?
type Query{
  query: ReturnObject
}
type ReturnObject{
  id: ID
  header: [Header]
}
interface Header{
  id: ID
  description: String
}
interface Extension{
  id: ID
  data: String
}
type ObjectA implements Header & Extension{
  id: ID
  description: String
  data: String
}
type ObjectB implements Header {
  id: ID
  description: String
  objectC : [ObjectC ]
}
type ObjectC implements Extension{
  id: ID
  data: String
}

There were improvements for interface mappings in 1.3, but overall it's not clear what you mean with one vs two requests. Could you provide more information, what version you're trying this with, maybe a sample query and explain what the two requests are.

Thank you for your reply.
The situation is as follows: I am returning two types of objects, both of which implement the Header interface (Object A and Object B).
Object B contains a list of Object C.
Now Object A and Object C (the one in Object B) both share a field data from the shared Interface Extension which I address with @Batchmapping(typeName= Extension, field= data) in my code. When the header is addressed, the function handles both objects at the same time and the function is only active once. For the interface Extension, the function is called twice. Once for object A and once for object C. Is there a way to avoid this?

Rather than explaining that in plain text, could you show a sample schema, a request and a controller method signature?
I'm afraid I don't understand the current description. Code snippets would be clearer.

This is my Schema:
type Query{
  query: ReturnObject
}
type ReturnObject{
  id: ID
  header: [Header]
}
interface Header{
  id: ID
  description: String
}
interface Extension{
  id: ID
  data: String
}
type ObjectA implements Header & Extension{
  id: ID
  description: String
  data: String
}
type ObjectB implements Header {
  id: ID
  description: String
  objectC : [ObjectC ]
}
type ObjectC implements Extension{
  id: ID
  data: String
}

This is my BatchMapping for the Extension
@Batchmapping(typeName= Extension, field= data)
fun getExtensionData(extensionObjects: List<Extension>, context: GraphQLContext): Map<Extension, Data>{
    return requestToAPI() // not relevant
}

A query would be like:
query {
  query {
    id
    header {
      ... on ObjectA {
        data
      }
      ... on ObjectB {
        objectC {
          data
        }
      }
    }
  }
}

Hope this helps. It's a really confusing situation for me too, unfortunately.

I am having a similar issue and don't know if I should open a new ticket.
@Segelente,
You mean, that aliasing and passing arguments to the graphql query doesn't work?
I figured, that I can get the BatchLoaderEnvironment, GraphQLContext, Principle.
But I don't see the information that I found in the DataFetchingEnvironment in the past.
Does it mean, that BatchMapping is only for very simple use cases (almost none in my case)?
I would expect something like @Argument("filter") Map<FilterClass, List> being possible....

@Segelente could you please use https://start.spring.io to create a starter project with the relevant versions, and then add minimal code that helps to demonstrate the issue. As I said in my previous comment, the behavior depends on the version you're using and a number of other things, so it's best if you iterate over something until you can demonstrate the issue. If that's easy then it shouldn't take long. If it is not, then you can try different things and compare to your code until you get demonstrate it.

Hello everyone,
Thanks for your patience. I really appreciate your help. I have created a project: https://github.com/Segelente/demo-nested-batches
I hope this helps. Unfortunately, I am not responsible for the design choices and cannot just change the way the schema is set up. If you can't help, I fully understand. Have a nice day.


Spanner: Tests faling with duplicate column name.

Affecting all Spanner tests since this is happening in the fixture: Error: Google.Cloud.Spanner.Data.SpannerException : Duplicate column name Albums.MarketingBudget.
See sample CI output.

I think this was a flake. I haven't seen the same failure after this instance. For example in this which ran after the CI test above.
In a normal run of the tests the output looks like this:
Starting test execution, please wait...

A total of 1 test files matched the specified pattern.
Data inserted.
Added the MarketingBudget column.
Added the AlbumsByAlbumTitle index.
Added the AlbumsByAlbumTitle2 index.
Database my-test-database already exists.
Backup my-test-database-backup already exists.
Added the Revenue column.
...

In this failed case, the output looks like:
Data inserted.
OperationStopped: FAILED with exit code 1
[xUnit.net 00:00:29.77]     AddNumericColumnTest.TestAddNumericColumn [FAIL]
[xUnit.net 00:00:29.78]     CreateBackupTest.TestCreateBackup [FAIL]
[xUnit.net 00:00:29.78]     CreateDatabaseAsyncTest.TestCreateDatabaseAsync [FAIL]
[xUnit.net 00:00:29.78]     CreateDatabaseWithRetentionPeriodAsyncTest.TestCreateDatabaseWithRetentionPeriodAsync [FAIL]
[xUnit.net 00:00:29.79]     CustomTimeoutsAndRetriesTest.CustomTimeoutsAndRetries [FAIL]
[xUnit.net 00:00:29.79]     DeleteUsingDmlCoreAsyncTest.TestDeleteUsingDmlCoreAsync [FAIL]
[xUnit.net 00:00:29.81]     DeleteUsingPartitionedDmlCoreAsyncTest.TestDeleteUsingPartitionedDmlCoreAsync [FAIL]
[xUnit.net 00:00:29.82]     InsertUsingDmlCoreAsyncTest.TestInsertUsingDmlCoreAsync [FAIL]
[xUnit.net 00:00:29.82]     LogCommitStatsAsyncTest.TestLogCommitStatsAsync [FAIL]
  X AddNumericColumnTest.TestAddNumericColumn [1ms]
  Error Message:
   Google.Cloud.Spanner.Data.SpannerException : Duplicate column name Albums.MarketingBudget.
  Stack Trace:
     at Google.Cloud.Spanner.Data.SpannerCommand.ExecutableCommand.ExecuteDdlAsync(CancellationToken cancellationToken)
   at Google.Cloud.Spanner.Data.SpannerCommand.ExecutableCommand.ExecuteDdlAsync(CancellationToken cancellationToken)
   at AddColumnAsyncSample.AddColumnAsync(String projectId, String instanceId, String databaseId) in /tmpfs/src/github/dotnet-docs-samples/spanner/api/Spanner.Samples/AddColumnAsync.cs:line 30
   at SpannerFixture.InitializeDatabaseAsync() in /tmpfs/src/github/dotnet-docs-samples/spanner/api/Spanner.Samples.Tests/SpannerFixture.cs:line 117
   at SpannerFixture.InitializeAsync() in /tmpfs/src/github/dotnet-docs-samples/spanner/api/Spanner.Samples.Tests/SpannerFixture.cs:line 48
  X CreateBackupTest.TestCreateBackup [1ms]
....

It's not very easy to tell what happened from the logs but I think a DML call probably timed out, so some retry mechanism kicked in and tried adding the column again, after which the database started returning an error.
According to the code, there's only one time where AddColumnAsync could be called, assuming that we're not trying to run tests for both the old version of the the samples and the new ones.

I'm not sure that there is anything to fix here but @amanda-tarafa let me know your thoughts.

Yep, it's a flake for sure, I think I saw it once before but no more than that. I also took a brief look and couldn't make out how it would happaned. I thought it might have been the tests running at the same time on Linux and Windows, but then the failure would have been on database creation, not here. And we are not running the old tests either.
Maybe catch that exception, check that the error code indicates a duplicate, and ignore it in that case? I'd like to understand it better, but I agree it's probably related to some retry that's being erronously reported by the backend.

Definitely this seem like flaky and not able to reproduce it locally. I have loop it for 50 time to create database and add column but it's working fine.
Based on SpannerFixture I just copy the sample and created below program:
    class Program
    {
        static string ProjectId = "project-id";
        static string InstanceId = "instance-id";
        static void Main(string[] args)
        {
            for (int i = 0; i < 50; i++)
            {
                Console.WriteLine($"Start --> {i}");
                RunSamples().Wait();
                Console.WriteLine($"Finish --> {i}");
            }
        }

        static async Task RunSamples()
        {
            try
            {
                var databaseId = $"my-db-{DateTimeOffset.UtcNow.ToUnixTimeMilliseconds()}";
                CreateDatabaseAsyncSample createDatabaseAsyncSample = new CreateDatabaseAsyncSample();
                InsertDataAsyncSample insertDataAsyncSample = new InsertDataAsyncSample();
                InsertStructSampleDataAsyncSample insertStructSampleDataAsyncSample = new InsertStructSampleDataAsyncSample();
                AddColumnAsyncSample addColumnAsyncSample = new AddColumnAsyncSample();
                await createDatabaseAsyncSample.CreateDatabaseAsync(ProjectId, InstanceId, databaseId);
                await insertDataAsyncSample.InsertDataAsync(ProjectId, InstanceId, databaseId);
                await insertStructSampleDataAsyncSample.InsertStructSampleDataAsync(ProjectId, InstanceId, databaseId);
                await addColumnAsyncSample.AddColumnAsync(ProjectId, InstanceId, databaseId);
            }
            catch (Exception ex)
            {
                throw;
            }
        }
    }

I'll close this one for now as not reproduceable. I'll reopen if I see it again.


[UPDATE] keygen and sign QR code share png

To update and improve the standard of our QR codes for keysign and keygen, please match the design on figma.
Include the padding and description updates.

@Rockindash on main it still does not look like in figma, I see that my description may not have been accurate. Please update the pairing views accordingly.:

@Rockindash please check

Hey @realpaaao this issue required the update to the QR share images.
For the pair view itself, they were fixed in the following PR:
https://github.com/vultisig/vultisig-ios/pull/1420


Update cd.yml

Ticket
https://wicmtdp.atlassian.net/browse/{TICKET_ID}
Changes

What was added, updated, or removed in this PR.

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

Testing

Screenshots, GIF demos, code examples or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

Closing WIP in favor of alsia/manage-envs


Display "powered by Google" logo?

According to Google's place autocomplete documentation:

Note: When you display predictions from Place Autocomplete, you must include the 'powered by Google' logo. This logo is included in the results list by default, as shown in the above example.
- Source

We can add that ourselves via the component's customContainerRender prop, but it seems like something the component should handle natively / automatically. Do you agree? If so, our team would be happy to contribute a PR to add it to the module.

You are right. I would be grateful if you contribute with a PR to help me about this :)
Could you add a prop to disable it in case of someone would like to customize the render with customContainerRender?
Thank you very much

We are working on a fix to this and adding a prop to selectively show the "Powered By Google" logo with the default view. However, because the customContainerRender is a function provided by the consumer, the consumer would have to add the "Powered By Google" logo themselves since we have no access into the container function itself.
To alleviate this issue, we would provide a component that the consumer can use in their layout, since we cannot append it within their customContainerRender function. Thoughts on this approach?

It could be a good idea. An other way could be to use customContainerRender to customize only list result. The google logo will be at the bottom and still manageable by the prop you added

Fixed with v3.7.0


Rewrite

This is a complete rewrite of the library that is meant to keep the API intact. There are a couple of reasons why a rewrite turned out to be the best option:

Over time we realized that defensive behavior of the library when it comes to altering configs is problematic. The idea was good but it was enforced implicitly thus making it literally impossible to use dry-configurable in certain scenarios (ie having a subclass that needs more settings than its parent while the parent already finalized its config)
Tweaking existing behavior turned out to be too difficult (me and Nikita tried, separately, and concluded the same)
This is the very first dry-rb library! It served us extremely well for almost 5 years but as a result it gathered some dust and so cleaning it up before its 1.0.0 release makes sense

Summary
Behavior changes
There are a couple of behavior changes but they do not change the public API:

Accessing config does not finalize/freeze it and you can change it in multiple steps easily now and finalize the config once you know it's safe
config uses method_missing to read and write variables - I think it's much much simpler and unless we hear reports about performance issues, it's OK to keep it like that

Low-level additions

You can easily restore config to its initial state via Config#pristine (which is used internally by the test helper)
class App
  extend Dry::Configurable

  setting :db do
    setting :user, 'root'
    setting :pass
  end
end

App.configure do |config|
  config.db.user = 'jane'
end

App.config.db.user
# "jane"

App.config.pristine.db.user
# "root"

The DSL is now based on AST - this simplified the implementation a lot (as it's usually the case) and we now have a nice, pure-data representation of settings:
class App
  extend Dry::Configurable

  setting :db do
    setting :user
    setting :pass
  end
end

ast = App.__config_dsl__.ast
# [[:nested, [[:setting, [:db, Undefined, {}]], [[:setting, [:user, Undefined, {}]], [:setting, [:pass, Undefined, {}]]]]]]

compiler = Dry::Configurable::Compiler.new
# #<Dry::Configurable::Compiler:0x00007f94b88b4878>

settings = compiler.(ast)
# #<Dry::Configurable::Settings:0x00007f94b626d160 entries=1 default_proc=nil>

settings[:db]
# #<Dry::Configurable::Setting::Nested name=:db options={:value=>Undefined, :default=>Undefined, :settings=>#<Dry::Configurable::Settings:0x00007f94b626cf08 entries=2 default_proc=nil>}>

TODO

[x] add new YARD docs
[x] refactor to use an AST-based DSL, get rid of the anonymous settings class used for nested settings
[x] setting :foo
[x] setting :foo, 'value'
[x] setting :foo, 'value', reader: true
[x] setting(:foo, 'value') { |val| val.upcase }
[x] inheritance
[x] ensure reserved keywords still work ie setting :if, true
[x] see if we can stick to method_missing
[x] readd test_interface
[x] ensure relevant old specs pass and remove them once they do
[x] nested settings

[x] ensure that deep-nesting works

[x] default values

[x] when constructor is defined and default is nil
[x] set to nil implicitly (I think?)

[x] ensure reader: true with nested settings works as expected
[x] ensure dup + clone behavior stays the same
[x] finalize! class method (consider deprecating this, using freeze would be enough)
[x] finalize! instance method (same as above)

It makes no sense to review the diff - please just go through the code 😃


fix: campaign proposition responsive behaviour

Pull Request Test Coverage Report for Build 437748567

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.006%) to 73.924%

Totals

Change from base Build 437668967:
-0.006%

Covered Lines:
1647

Relevant Lines:
2031

💛  - Coveralls


how to concatenate depth image and shaded image？

Hello, I would like to know how to concatenate depth image and shaded image? Is it just adding the depth image as alpha channel into the shaded image and processing, or training separately and combining at the view-pooling layer?

I'm actually concatenating the input as 6 channels, but I guess it's similar to using the alpha channel, just note that I'm using the imagenet pre-trained model so the first layer needs to be averaged over channel if you do so.

thank you very much : )

Can you explain it in more detail? I tried adding the depth map into the alpha channel and changing the in_channel of the first Conv2d layer from 3 to 4, but the result was worse than the 3-channel RGB map.

I append the depth as another RGB image, so the input is 6-channel. The weights of the first layer are replicated along the channel (if using the pre-trained model).


Support for custom resources

This is great. Thank you for sharing! Any ideas about adding support for custom resources?
Something along the lines of:
kubectl get crd -o json | jq '[.items[].spec.names] | map([.kind,.singular,.plural, ["g","d","rm"], null])'

would get pretty close to the shape already being used:
https://github.com/ahmetb/kubectl-aliases/blob/0533366d8e3e3b3987cc1b7b07a7e8fcfb69f93c/generate_aliases.py#L42-L51
However, seems this would be tricky without an abbreviation field defined in the spec.names.. I'm happy regenerating locally after manually adding my own abbreviations, but maybe you have some ideas about how to support this more generally?

@montmanu ahh, CRDs would be tricky, especially because we can't make them work for everyone and we statically generate the aliases. I'd say this is a WONTFIX for now, mostly because we don't even have shorthands for lots of other native Resource Types (like NetworkPolicy, ClusterRole[Binding], Role, Job, CronJob, StatefulSet, PersistentVolume[Claim] etc).


USHIFT-1733: 1733 reenable after cleanup

Continuation of #2417
/assign @pacevedom

/cherry-pick release-4.14


Investigate:🔍 Multiple sln files

Multiple solution files in a project has been mentioned a couple of times and I need to make an assessment.

What practical benefit is there from having multiple solution files
What is the drawbacks of implementing support for multiple solution files
How does Visual studio/rider handle this situation
Where are these sln files placed in the file structure (siblings, child, parent etc)

Anyone feel free to chime in here

I have seen multiple solution files in the same repo to handle cases where you want to include/exclude different projects, e.g. one solution file for all projects and another excludes test projects. If you want to just load certain projects in VS, a better option is to use solution filters. Another scenario would be a monorepo with different applications/services, each with their own solution.

If you try to run dotnet build without specifying solution or project when there are multiple solutions/projects, you will get an error: MSBUILD : error MSB1011: Specify which project or solution file to use because this folder contains more than one project or solution file. For this extension, letting the user choose project/solution to build is a nicer experience IMHO. Maybe we can consider adding Build All functionality, .e.g in PowerShell: Get-ChildItem -Path <RootDir> -Filter *.sln -File -Recurse | % { dotnet build $_.FullName }.

You usually open a project or solution in VS. There is an option to open a folder which I have rarely used.

IMHO best not to make assumptions regarding solution file location.

I was merely stating what I have seen in professional settings regarding multiple solutions :smile:
Being allowed to use Neovim at work would be tough challenge 🙁
I don't work on C# code outside work much and would probably not need multiple solutions. IMHO it is a niche use case.
My thoughts are:
Keep status quo: only support single solution (put disclaimer regarding this in README.md).
OR
Add a solution picker. If you do so, would be nice to also include an option to build all solutions.

Why would being allowed to use Neoivm at work be a challenge? Due to security concerns?

Yes
On Fri, Sep 13, 2024, 9:53 PM Gustav Eikaas @.***>
wrote:

Why would being allowed to use Neoivm at work be a challenge? Due to
security concerns?
—
Reply to this email directly, view it on GitHub
https://github.com/GustavEikaas/easy-dotnet.nvim/issues/114#issuecomment-2350834777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVJQJZZ3YZOAS6AOHLJAIDZWO6LXAVCNFSM6AAAAABODTWVPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJQHAZTINZXG4
.
You are receiving this because you commented.Message ID:
@.***>

Yes, mostly security reasons.

Okay great!
Ill prioritize getting this done as one of my next features
Any input on how to do it?

Scan for solution files in current directory and traverse down ~5 depths
If multiple solution files are found spawn a telescope picker
Persist the solution file the user chose and default to using this for any operation going forward
Add a command Dotnet solution select? or something similar to spawn the prompt and pick a different solution

Draft implementation in #185, feel free to test and give feedback


PNG pictures are wasteful

Source video doesn't use lossless compression, PNG is pointless, and wastes bandwidth and storage space. JPG, WEBP or AVIF should be used instead.

Thx for opening issue, all file has completed convert to jpg, only the copy2clipboard function need png.


Undefined symbols for arm64 SDLUIKitDelegate

Hi, I'm using this extension with an android and iOS app. Android works just fine but iOS has an issue when compiling. It comes up with an error of:
Undefined symbols for architecture arm64:
"_OBJC_CLASS_$_SDLUIKitDelegate", referenced from: ltmp27 in libfacebook.a(a5504ceb_SDLUIKitDelegate+ExtnesionFacebook.o) l_OBJC_$_CATEGORY_SDLUIKitDelegate_$_ExtensionFacebook in libfacebook.a(a5504ceb_SDLUIKitDelegate_ExtensionFacebook.o)
Any idea what might be the cause of this? Something in my setup, or something with the code?

Wich version of openfl / lime do you use on mac ?
Do you use  legacy version when SDL wasn't the backend yet ?
Try to use the legacy branch maybe.

Thanks for the quick reply, everything is on the current latest version for openfl and lime. I've not had to use legacy on ios before, is it a case of using the same -Dlegacy flag as Android do you know?

By legacy i meant previous version of lime when SDL was not yet the system backend.
If you use last lime and openfl with master branch of the extension its normally ok.
I see in lime changelog that sdl have been updated, maybe it's what causing the issue.
I understant from the error you linked that the SDLUIKitDelegate class is not found, as if it changed name with the new version or got remove, I can't tell yet. It's strange.
Sadly I don't have access to a mac anymore, I don't have my own and won't be able to buy one before weeks so I won't be able to fix this rapidly.
One thing you can try is to set lime to 2.8.3 and try to run again. Of it compile then the error probably come from that sdl upgrade.

I'll give that a go, A clean build of the project changed the problem from arm64 to armv7 but I'm assuming that it's essentially the same issue. I'll roll back lime and give that a go

Unfortunately setting lime to 2.8.3 wasn't a solution in this case

If there is a way that I can somehow help to narrow down this issue by proxy I am happy to do what I can to work out what's going on? I'm happy to poke around and try some things, I understand you personally can't look to reproduce at the present time but my client is looking for solutions ASAP. Give me some direction and I can head off and investigate ;)

hello,
Hello,
sorry for not answering this week.
I got a mac and triied to reproduce your problem on a fresh new install but couldn't reproduce it, no linking error. The only error I got was a code signing error 'cause I don't have any iDevice yet, but normaly those errors occurs after linking I think.

I really can't see why you have this error.
I'll try to get a cheap iPhone and let you know if I found anything.

Hi, thanks for taking the time. I'm going to go and double check what I'm compiling against on the Mac and hopefully this'll just turn out to be some dumb thing I've overlooked (I'm not experienced with OpenFL Mac development as I'm sure you've gathered!) I'll leave another message once I have some more information as to what I'm using.

Hello, hope you found a solution.
Just in case, I finally got an iPhone and just tested... no linking error on my side.
I don't know how to reproduce your problem but in my opinion the problem comes from your environment.
Check you use last lime (or at least a version based on SDL) and you use the master branch of this repo.
then rebuild the extension using lime rebuild facebook ios.
then lime update ios and haxelib run facebook project.xml.
this should work unless you use -Dlegacy to build your project
If you use -Dlegacy, switch to the legacy branch of this repo and rebuild the extension.

Having the same problem...

openfl: [3.6.1]
hxcpp: [3.3.49]
flixel: [4.0.1]
lime: [2.9.1]

Just ran into this when building for iOS simulator, and we're also using -dLegacy.
Ld build/Debug-iphonesimulator/Game.app/Game normal i386
    cd /game/export/ios
    export IPHONEOS_DEPLOYMENT_TARGET=8.1
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin"

    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++
     -arch i386
     -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.3.sdk
     -L/game/export/ios/build/Debug-iphonesimulator
     -L/game/export/ios/Game/lib/i386-debug
     -L/game/export/ios/Game/lib/i386
     -F/game/export/ios/build/Debug-iphonesimulator
     -F/usr/local/lib/haxe/lib/facebook/git/dependencies/ios
     -filelist /game/export/ios/build/Game.build/Debug-iphonesimulator/Game.build/Objects-normal/i386/Game.LinkFileList
     -mios-simulator-version-min=8.1
     -Xlinker
     -no_deduplicate
     -Xlinker
     -objc_abi_version
     -Xlinker 2
     -lstd
     -lregexp
     -llime-legacy
     -lfacebook
     -lApplicationMain
     -stdlib=libc++
     -ObjC
     -framework UIKit
     -framework OpenAL
     -framework OpenGLES
     -framework QuartzCore
     -framework CoreAudio
     -framework CoreGraphics
     -framework AudioToolbox
     -framework AVFoundation
     -framework Foundation
     -framework CoreMotion
     -framework MediaPlayer
     -framework FBSDKCoreKit
     -framework FBSDKLoginKit
     -Xlinker
     -dependency_info
     -Xlinker /game/export/ios/build/Game.build/Debug-iphonesimulator/Game.build/Objects-normal/i386/Game_dependency_info.dat
     -o /game/export/ios/build/Debug-iphonesimulator/Game.app/Game

Undefined symbols for architecture i386:
  "_OBJC_CLASS_$_SDLUIKitDelegate", referenced from:
      anon in libfacebook.a(520600ac_SDLUIKitDelegate+ExtensionFacebook.o)
      l_OBJC_$_CATEGORY_SDLUIKitDelegate_$_ExtensionFacebook in libfacebook.a(520600ac_SDLUIKitDelegate+ExtensionFacebook.o)
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Building on Mac, and since it's flixel it has to be on legacy. Gonna try now the legacy branch

@TBaudon works with legacy branch. Need any help debugging this?

Hello, thanks for your help.
I don't think this is a bug actually.
As specified in the readme, if you use the ios legacy backend you have to use the legacy branch of this extension.
Using ios legacy backend means using the -dLegacy tag when building for ios, maybe it's not clear enough in the readme, I'll change that.
The problem is, legacy backend used a nme based class to handle every system related stuff, but then on the next lime version, SDL was used to replace this part. So I made two branch, one to use lime backend on legacy, and one to use sdl on master.
The legacy branch is not outdated, it's called legacy to correspond to the lime backend's name.
I don't really know how to do it in another way.
Maybe at compile time, the extension would need to check wich backend is in use and generate the corresponding categorie class...
It's easier to maintain two branch I think.

Ah @TBaudon that makes sense!
In haxe there's
#if openfl_legacy

#end

Maybe that would be better, as the legacy branch doesn't have the recent improvements added ( such as java version forcing and string = null setting )


https://crackmes.one/crackme/64ee0f7fd931496abf909393

#custom-algorithm,#string-manipulation

@maracujazuul sorry I am not adding these two labels because they are too broad. Most crackmes would somehow manipulate the string and a fair amount of them uses custom algorithms

I'm agree with you.


[Adjust Rule] SIM118 - possible issue with objects having a method "keys()"

Example
Imagine this scenario:
class X:
    def keys(self):
        return [1, 2, 3]

x = X()

if 1 in x.keys():
    print('yes')

If I run the linter, I get:

SIM118 Use 1 in x instead of 1 in x.keys()

Which will obviously lead to a crash. I know that I wouldn't name a method keys() but still I think the rule might cause some problems.

Rule(s): SIM118
Adjustment: In which way should it/they be adjusted?

We need some check that it's really a dict if we want to auto-remove the keys...
Explanation
We might crash somebodys code when somebody is auto-fixing it (via ruff) or just blindly following the linter.

Hi,
this actually crashed our code yesterday.
Please try to to this only for objects that act like dicts. Maybe something in here can help to identify these structures? https://docs.python.org/3/reference/datamodel.html?emulating-container-types=#emulating-container-types


[BUG] "Not production ready" impacts users

Currently with "WARNING: Snippets are not production ready." people are tempted to look at the code, are discouraged from looking at any of the code, for fear that the snippets aren't going to be useful.
I personally ignored this project because of that warning, and only on being reminded about it did I stop to think about why I was ignoring it.
I recommend that the project-wide warning be removed as a global warning, and be added instead to the relevant snippets that are likely to impact production.

Thanks for the suggestion. Many of the snippets are not really production-ready, meaning there are edge-cases and performance issues with them, if users want to do something that we may not have foreseen. This is in contrast with a library like Lodash, which has been built and bundled for production.
However, the criticism that the warning might discourage users is a valid one. I already migrated our tests from Tape to Jest yesterday to make the testing more robust and allow for more extensive snippet testing later down the line. Adding more tests for snippets should be one of the steps we take to ensure snippet quality and production-readiness. I'd really like for the community to pitch in and help out by adding more tests and setting them up so we can validate which snippets are ready for production.

Update: After a lot of testing and some evaluation of the code, I believe we can change the site-wide disclaimer into more of a note that some snippets are not yet optimized for production. Disclaimers for individual snippets are still there and we might add a few more over time to address more issues that we uncover, but I think this issue is not gonna bother anyone anymore.


Control the mdns behaviour of multicasting / advertising

Are there in the mdns library configuration settings available to control the amount and timeframes of the mdns multicasting / advertising?
I'm a bit confused why my homebride sends about 4 or more multicast resquests per minute to the entire network. (tcpdump checked)
Are these multicasts needed by any devices who are already paired with my homebrigde?
Since HAP-NodeJS moved from dbus and avahi away the configurations in the avahi-daemon config are useless.
e.g. these...
[publish]
disable-publishing=yes
disable-user-service-publishing=yes
[server]
ratelimit-interval-usec=0
ratelimit-burst=0

This is not very well documented and may not have the feature you are looking for but, please review issue #1957 for details on how to configure the mDNS package.  And the options are documented here
https://github.com/mafintosh/multicast-dns#mdns--multicastdnsoptions

In my test environment I just added a debug line around that if statement and can confirm that parameter passing works as designed, but for the feature itself I can't confirm if it works or not.
If that feature is a required feature for your self, you could try opening an issue with multicast-dns, and see if they will take it on as an enhancement.

OK, i think I've found the problem. It's not in the multicast-dns package. It's in the a forked implementation bonjour-hap that is used by hap-nodejs:
https://github.com/KhaosT/bonjour/blob/master/lib/Service.js
Filepath in my Installation:
/usr/lib/node_modules/hap-nodejs/node_modules/bonjour-hap/lib
"gitHead": "2db14fae2cb66886f920d78710d902f4fe535a20"
<EMAIL_ADDRESS>@KhaosT I'm not able to create an Issue in your fork of the bonjour project. So I'm give it a try here...

The announcements never stops, because, the variable "this.delay=1000" is overwritten every time the  "announce: function ()" is getting called by the "onAnnounceComplete: function ()".
This variable should be defined in the constructor "var Service = function (opts) {..."

It would be great if the 2 Paramaters in that file would be configurable.
var REANNOUNCE_MAX_MS = 60 * 60 * 1000
var REANNOUNCE_FACTOR = 3

So the currently never ending "PTR Homebridge-..." mdns announcements should be stopping after a while.


Jen Morse LiDAR Uncertainty .py and .ipynb files for review.

Pull Request Template
Review Checklist
Reproducibility

[ ] Are the data downloaded in the code
[ ] Are paths created to ensure they work on all operating systems (using  os.path.join)
[ ] Are comments used to clarify the contents of the code that can’t be clarified using expressive variable and function names alone? (not too many comments - just enough)
[ ] Does the notebook run from start to finish?

PEP 8 standards & Code Readability
Functions

[ ] Do functions follow PEP 8 format conventions?
[ ] Are function docstrings clear (all  inputs and outputs clearly  described and defined)
[ ] Are function names expressive (the name describes what the function does)?
[ ] Are functions easy to understand and read?
[ ] How many tasks does each function do? (ideally a function does one thing well).

Package imports

[ ] Are standard modules (those included with the base  python install) vs. third party (related but externally developed tools) import groups correct with appropriate spacing in between each group?
[ ] Are variable names throughout the code, expressive?
[ ] Suggest changes if not, highlight what was done well
[ ] Is the code overall easy to understand and read? Are there things that would make it more clear / cleaner?

DRY Code

[ ] Are segments of code repeated in the file or is the code DRY?
[ ] Are loops used to optimize DRY code?
[ ] Are functions used to optimize DRY code?
[ ] Are there any areas that could be potentially improved (you can suggest improvements OR you can just highlight parts of the code where you suspect it could  be cleaner / more efficient.

Novel Approaches to Problem solving

[ ] Highlight any novel approaches to completing the assignment.

@allisonw5 Please review this PR #56  (Canvas isn't working so let's try GitHub directly!)

nice work, Jen! Great use of classes and functions. Next step - make sure to include parameters and returns and attributes in your docstrings so others can know what your code does.


[Parse] Fix source range of SPI attributes

Fix the SPI attribute source range to cover only the attribute. This also fixes the fix-it that deleted the next token too.
rdar://problem/63554849

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

@swift-ci Please smoke test Linux


BrokenResourceError on metrics endpoint

They mention here that it's related to MemoryStream middleware, but I'm not sure it's relevant.
https://github.com/tiangolo/fastapi/issues/4041
Versions:
PFI: 5.8.2
FastAPI 0.78
Starlette 0.19.1
Any ideas how to fix it?

I've got exactly the same problem, here is a bigger stack trace screenshot:

I'm using 5.8.1, there seems to be quite a few changes in 5.8.2. I'll try that.

It happens randomly on all endpoints. Maybe your health endpoint is used the most, therefore it seems most problematic for you.

@mdczaplicki yeah, it happens on all endpoints, but the health endpoint restarts my service when it fails too often, which is a problem. It's also a pretty simple endpoint, so I thought it's worth including for illustrative purposes.

@mdczaplicki I upgraded FastAPI, prometheus-fastapi-instrumentator and uvicorn to the latest versions, and I haven't seen that this error in the last hour :crossed_fingers:

I did as well. Only FAPI was to be updated, though didn't go away. :/

@circlingthesun, what other 3rd party packages do you use in your project? Do you still not get the error?
I've realized that our pods are also restarted, because health endpoint fails.

@mdczaplicki I don't use any other middleware (which seems to be the culprit) beside the starlette/fastapi CORS middleware. If you think you can make progress by looking at my dependencies here you go:
alembic==1.8.1
aio_pika==7.2.0
Babel==2.10.1
bandit==1.7.4
beautifulsoup4==4.11.1
base58==2.1.1
billiard==<IP_ADDRESS>
celery==4.4.2
flake8==4.0.1
GeoAlchemy2==0.12.2
icalendar==4.1.0
intervaltree==3.1.0
iso3166==2.0.2
Jinja2==3.1.2
lxml==4.8.0
PyJWT==1.7.1
bcrypt==3.2.2
mock==4.0.3
mypy==0.971
pika==1.2.1
Pillow==9.2.0
prometheus-client==0.14.1
types_python_dateutil==2.8.18
pytz==2022.1
types_pytz==2022.1.1
requests==2.25.1
sentry-sdk==1.8.0
simplejson==3.17.2
types_simplejson==3.17.7
SQLAlchemy==1.4.39
SQLAlchemy-Searchable==1.4.1
WeasyPrint==56.1
uvicorn[standard]==0.18.2
fastapi==0.79.0
pydantic==1.9.1
python-multipart==0.0.5
websockets==10.3
databases==0.6.0
asyncpg==0.26.0
cachetools==4.2.1
types-cachetools==5.2.1
email-validator==1.2.1
httpx==0.23.0
watchdog==2.1.7
argh==0.26.2
python_jose[cryptography]==3.3.0
bleach==5.0.0
types-bleach==5.0.3
passlib[bcrypt]==1.7.4
aiobotocore==1.2.2
boto3==1.16.52
asyncache==0.1.1
dacite==1.6.0
locust==2.10.1
watchfiles==0.16.1
prometheus_fastapi_instrumentator==5.8.2
asgiref==3.5.2

@mdczaplicki I forgot I'm also using sentry's middleware here are the relavant lines:
app.add_middleware(SentryAsgiMiddleware)

# Make sure this is last? https://github.com/tiangolo/fastapi/issues/1663
app.add_middleware(
    CORSMiddleware,
    allow_origin_regex="https?://.*",
    allow_credentials=True,
    allow_methods=["*"],
    allow_headers=["*"],
)

Instrumentator().instrument(app).expose(app)

Hmm, and you're sure that you get rid of BrokenResourceError? We are trying to get rid of the CORSMiddleware from starlette, we have a suspicion, that it's the culprit.
I suppose you have starlette==0.19.1, right?

@mdczaplicki yes, starlette==0.19.1, I don't lock it, I just get whatever version FastAPI depends on. Also just to be complete, I get the CORS middleware via from fastapi.middleware.cors import CORSMiddleware.

Uuu, thanks for the hint. We use from starlette.middleware.cors import CORSMiddleware. If  we'll get rid of this error, by removing starlette's CORSMiddleware - we'll switch to fastapi's

@mdczaplicki its not completely, gone, but since upgrading my dependencies on the 26th, there has been exactly 3 occurrences:

🤣

@mdczaplicki ha ha, yeah, I vaguely recalled that being the case.

@circlingthesun, hey man. I'm coming back to you - to let you know that the problem is not with this library.
It's related to Sentry middleware. You should update to newest Sentry SDK (1.9.0). In 1.8.0 they've introduced new FastAPI integration method. However, don't rush into updating yet, because this will cause new errors to pop up - much more harsh.
Better wait until this PR is merged: https://github.com/getsentry/sentry-python/pull/1532
New integration configuration: https://docs.sentry.io/platforms/python/guides/fastapi/#configure

@mdczaplicki it's funny that without sentry we wouldn't be aware of the problem but also wouldn't have the problem. Thanks for the heads up :+1:


Add missing #include

This is necessary to make code compile with a recent libc++ (after https://reviews.llvm.org/D146097).

Thanks! Do you need a release quickly?


Don't retry a failed commit

Session would retry a transaction even if it failed on commit. This is not desirable because the driver can't know if the transaction got successfully committed on the server's side or not.

Actually there is a BoltIncompleteCommitError that's meant for internal use only. That's why Transaction translates it to a ServiceUnavailable Error.


Close tick around command

Thanks @mnordine, closing via https://dart-review.googlesource.com/c/sdk/+/184105


Graphviz image update

It's entirely cosmetic, but the graphviz network should be updated with odo -- perhaps there is some new functionality which could be merged into the network?

@quasiben I'm not sure what you mean here. Are there things that are missing in the graph?

Many of the nodes have into.foo

Oh well that simply won't foo :)

I can handle this if no one else is.
On Thu, Mar 12, 2015 at 7:11 PM, Phillip Cloud<EMAIL_ADDRESS>wrote:

Oh well that simply won't foo :)
—
Reply to this email directly or view it on GitHub
https://github.com/ContinuumIO/odo/issues/124#issuecomment-78749088.

Or rather, I'm on this.  Let me know if I'm overlapping.  Mostly this just
involves creating an environment with everything in it, running
from odo.dot import dot_graph
dot_graph()
And then saving files to the appropriate places in the repository.
On Thu, Mar 12, 2015 at 7:20 PM, Matthew Rocklin<EMAIL_ADDRESS>wrote:

I can handle this if no one else is.
On Thu, Mar 12, 2015 at 7:11 PM, Phillip Cloud<EMAIL_ADDRESS>wrote:

Oh well that simply won't foo :)
—
Reply to this email directly or view it on GitHub
https://github.com/ContinuumIO/odo/issues/124#issuecomment-78749088.

Fixed by https://github.com/ContinuumIO/odo/commit/2f4c7fce02b9974b3eaaae8a32f6a678606e9ede


About Exception reporting APIs

I noticed the last versions of Application Inisghts removed support for exceptions reporting in favor to HockeyApp. The problem is that HockeyApp for Xamarin still doesn't support (known) exception reporting, only crashes.
So right now users don't have a way of doing that, since everyone (Xamarin Insights and AppInsights) is dropping support and telling us to use a feature that isn't available yet on HockeyApp SDK for Xamarin.
What is the recommended way for doing this now? Would Application Insights consider putting back the exception APIs until HockeyApp releases a more complete version of the SDK for Xamarin?

These ApplicationInsights SDKs are not maintained any longer. We are working on adding support for handled Exceptions to HockeyApp, but we don't have an ETA at this moment.

Since there is no ETA, could Application Insights put this back until it is available on HockeyApp?

To my knowledge Application Insights has no plans on supporting mobile apps besides this: https://azure.microsoft.com/en-us/blog/access-hockeyapp-data-in-ai-with-hockeyapp-bridge-app/


feat: load test cli

I think we can close this in flavor of https://github.com/dashpay/rs-platform-explorer/pull/13


Update Kali Install

Pictures are outdated and need to be updated to something more current

Could we also include recommended installation for MacOS M1 chip machines that have trouble using the regular Virtual Box installation of Kali Linux?


LIFX & Harmony 0.41.0

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.41.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
LIFX & Harmony Hub
Description of problem:
Both platforms had been working on 0.40.0 but now have errors starting.
Traceback (if applicable):
17-04-01 22:05:36 ERROR (MainThread) [homeassistant.components.light] Error while setting up platform lifx
Traceback (most recent call last):
File "/srv/hass/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 148, in _async_setup_platform
entity_platform.async_schedule_add_entities, discovery_info
File "/usr/lib/python3.4/asyncio/coroutines.py", line 141, in coro
res = func(*args, **kw)
File "/srv/hass/lib/python3.4/site-packages/homeassistant/components/light/lifx.py", line 59, in async_setup_platform
partial(aiolifx.LifxDiscovery, hass.loop, lifx_manager),
AttributeError: 'module' object has no attribute 'LifxDiscovery'
17-04-01 22:05:43 WARNING (MainThread) [homeassistant.setup] Setup of remote is taking over 10 seconds.
17-04-01 22:05:43 WARNING (MainThread) [homeassistant.setup] Setup of wemo is taking over 10 seconds.
17-04-01 22:05:44 ERROR (MainThread) [homeassistant.setup] Not initializing remote.harmony because could not install dependency pyharmony==1.0.12
17-04-01 22:05:44 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform remote.harmony: Could not install all requirements.
17-04-01 22:05:45 WARNING (MainThread) [homeassistant.setup] Setup of tts is taking over 10 seconds.

I've got exactly the same problem. Any solution yet?

Tried to update aiolifx
xxxx xxxxx@beaglebone:/root$ pip3 install --upgrade aiolifx
but there seems to be a problem with installing this package. Result shown below
Collecting aiolifx
Downloading aiolifx-0.4.3.tar.gz
Collecting bitstring (from aiolifx)
Exception:
Traceback (most recent call last):
File "/srv/hass/lib/python3.4/site-packages/pip/basecommand.py", line 215, in main
status = self.run(options, args)
File "/srv/hass/lib/python3.4/site-packages/pip/commands/install.py", line 335, in run
wb.build(autobuilding=True)
File "/srv/hass/lib/python3.4/site-packages/pip/wheel.py", line 749, in build
self.requirement_set.prepare_files(self.finder)
File "/srv/hass/lib/python3.4/site-packages/pip/req/req_set.py", line 380, in prepare_files
ignore_dependencies=self.ignore_dependencies))
File "/srv/hass/lib/python3.4/site-packages/pip/req/req_set.py", line 620, in _prepare_file
session=self.session, hashes=hashes)
File "/srv/hass/lib/python3.4/site-packages/pip/download.py", line 809, in unpack_url
unpack_file_url(link, location, download_dir, hashes=hashes)
File "/srv/hass/lib/python3.4/site-packages/pip/download.py", line 715, in unpack_file_url
unpack_file(from_path, location, content_type, link)
File "/srv/hass/lib/python3.4/site-packages/pip/utils/init.py", line 599, in unpack_file
flatten=not filename.endswith('.whl')
File "/srv/hass/lib/python3.4/site-packages/pip/utils/init.py", line 484, in unzip_file
zip = zipfile.ZipFile(zipfp, allowZip64=True)
File "/usr/lib/python3.4/zipfile.py", line 937, in init
self._RealGetContents()
File "/usr/lib/python3.4/zipfile.py", line 978, in _RealGetContents
raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

Have finally solved the problem pip will not install aiolifx or pyharmony in a virtualenv. Had to use easy_install while in the environment. Restart and all good.

Hey Trevor, could you give me a bit more info on how this works for the pyharmony package. Im a noob... Thx a lot.