VPN Gateways

What is the difference between "virtualnetworkgateways" and "vpngateways"?
Also, the network-specific guidance immediately states "For VPN Gateways, you must move IP addresses, virtual network gateways, and all associated connection resources."
https://docs.microsoft.com/en-us/azure/azure-resource-manager/move-limitations/networking-move-limitations
That info makes this page's assertion that "vpngateways" can NOT be moved to different RGs or subs a bit unclear. It seems that it IS possible to move them as long as you move the vnet also. Instead of a simple yes/no, I think we could at least have an asterisk or some other indication that resources can't be moved by themselves but can be moved with the dependent resources.

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

ID: 41f4e1d2-5b58-3d7c-6a97-d0e7bcacd402
Version Independent ID: b627e946-e267-2f94-1e5b-9877a343d611
Content: Move operation support by Azure resource type
Content Source: articles/azure-resource-manager/move-support-resources.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@dcbrown16 ,
Virtual network Gateway is the actual gateway which you create in the Gateway Subnet.
VPN gateway is the gateway which is created as the part of VWAN. So it cannot be moved and the documentation is correct mentioned.
I think this should clarify your question.

@msrini-MSFT is it possible to show me the difference in a screenshot? Where would I find a "VPN gateway" resource? Searching for this only brings up the Virtual Network Gateways.

@dcbrown16 , I figured out using the powershell commands. Use a get-virtualnetworkgateway command and get-vpngateway command. You can understand the difference.

Ok, thank you :)


Prettier error on Windows

Description

On Windows, the prettier command in the lint-staged pre-commit hook failed because Windows cannot handle the parsed linter command properly. We add the option --shell to the lint-staged command in the pre-commit hook to skip parsing the linter command (see https://github.com/okonet/lint-staged#command-line-flags).

I also removed the prettier scripts to prevent that they get out of sync. E.g. in the pre-commit command, the client tests case got included, whereas they are still not included in the prettier scripts. Prettier can instead by executed with the command yarn lint-staged which executed the pre-commit hook.

I added the prettier scripts again as the bamboo jobs need them. I also adjusted the file pattern of the scripts to include the client test files.


Intents not working when fired from cold start

This issue seems to be old, but is still present, please fix?

Use this fork, it works: https://github.com/jmcrthrs/cordova-plugin-intent


Access response detail when the response is not successful

Have you read the FAQ and checked for duplicate open issues?
Yes, found something related here: https://github.com/shaka-project/shaka-player/issues/3533
What version of Shaka Player are you using?
4.0.1
Please ask your question
Any way to see the response detail (headers, status code) when the status code is >= 400?
I tried to use:

ResponseFilter: can see only successful responses
downloadfailed event: no details about the response only the request. httpResponseCode is 0

Is downloadfailed only showing the request because the response is available with some other method?
Thanks!

I will make a PR to expose more information inside the downloadfailed event.

Actually, this is looking like it might not be possible.
I tested an experimental change on a manifest someone sent for testing that produces a 403 issue.
The fetch call throws the error: TypeError: Failed to fetch before even creating a response object, which means we have no way of getting what the error code was. The XHR API also did not seem to have any way to get the info of a failed request, though I admit I did not spend as much time looking with that.
According to this Chromium bug report, this might be intentional?
https://bugs.chromium.org/p/chromium/issues/detail?id=118096#c5
I don't understand the reasoning myself, but I tested this on Firefox also and it has the same behavior so I guess that must be part of the spec.


Fix compliance logos in dark mode

Incorrect antialiasing in the images

Footer images defined here: packages/docs/docusaurus.config.js
Images here: packages/docs/static/img/compliance/

Hi @codyebberson, I am a new contributor here.
I would love to start my contribution to this project with this issue. Could you please assign it to me?


Add ability to specify a list of additional security groups to attach to

PR o'clock
Description
This addresses #47 - I have been using this in a local fork and noticed there was an open issue for it. Basically, this allows the user to specify a list of additional existing security groups which will merge with worker_security_group_id if specified. This might make worker_security_group_id unnecessary, although it's assumed that the additional security groups are unaffiliated with the EKS cluster and more for VPC transit to other areas.
Please let me know what you think.
Checklist

[x] terraform fmt and terraform validate both work from the root and examples/eks_test_fixture directories (look in CI for an example)
[ ] Tests for the changes have been added and passing (for bug fixes/features)
[ ] Test results are pasted in this PR (in lieu of CI)
[x] Docs have been updated using terraform-docs per README.md instructions
[x] I've added my change to CHANGELOG.md
[x] Any breaking changes are highlighted above

Hey @hobbsh! Thanks for the PR and sorry for the late reply. Can you resolve your conflict? Then I think we can merge.

I'll defer to @brandoconnor. I think it would be good to have a per worker group SG list and also a list that applies to all work groups (which you have here in this PR). We don't need to tick both things in the same PR though 🙂

I believe this would be a valuable feature as implemented to start with.  I currently need this feature for my company.  Anything I can do to help?  My apologies to the team for creating a duplicate issue #110 and nearly duplicate PR #111 related to this same feature.
Adding the per worker group proved tricky in my PR due to the way that TF currently handles interpolations.

@mr-joshua How is that different from this PR? I see no functional differences. You basically pulled in the code here into that PR with added tests.

@hobbsh the PR I submitted covers both use cases of Issue #47.  #112 in workers.tf line 27 attaches cluster-wide security groups (which is also covered in this PR as you point out).  Extending this pr and also on line 27 is the logic to attach security groups to specific worker group configs via
"${compact(split(",",lookup(var.worker_groups[count.index],"additional_security_group_ids","")))

@mr-joshua Ok, I see it now - thanks. I just didn't see a default for that in variables.tf which is probably worth adding.

I merged the other PR since @mr-joshua was pretty quick at rebasing and also hassling me 🙃
Sorry about how this one eventuated @hobbsh, I don't have much free time but should have noticed there was 2 similar PRs and sorted it immediately.

@hobbsh and @max-rocket-internet my apologies

No problem at all @mr-joshua @max-rocket-internet - I'm a bit bummed but glad it got into master. I also didn't stay on top of this as well as I could have.


[speechx] Fix json2_kalditype compile bug

PR types

Bug fixes
PR changes

Describe

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.9 out of 11 committers have signed the CLA.:white_check_mark: zh794390558:white_check_mark: lym0302:white_check_mark: Honei:white_check_mark: QingshuChen:white_check_mark: jerryuhoo:white_check_mark: yt605155624:white_check_mark: KPatr1ck:white_check_mark: SmileGoat:white_check_mark: Jackwaterveg:x: root:x: pollyyanroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Bofrost.lu has degraded performance

In 91ba1b6, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1265 ms

Resolved: Bofrost.lu performance has improved in 4f74400 after 10 minutes.


MeteorJS package

Hey all,
I'm thinking of writing a package for MeteorJS. I understand that there are already a few, however those are either based on fabien's repo or of the other fork of alertifyJS.
Does anyone know if this has been done already, and if not would there be any interest in this?

you can use it from npm since 1.3: npm install alertify.js --save on project root and you just need to add it on your code with " var alertify = require('alertify.js')


Improve font and colour contrast in content

https://tw.pycon.org/2020/en-us/about/pycontw/
年紀大了眼睛不好看得很累，字種很細，顏色和背景的差別也不大 :|

我來跟設計師討論討論!

Should take this into consideration for the next year!


[fix][bookie] Correctly handle list configuration values

Motivation
When the metadataServiceUri or the zkServers configuration for BookieRackAffinityMapping is provided as a comma delimited list, it is automatically parsed into an ArrayList by the configuration class because the Bookkeeper configuration class relies on the defaults in the org.apache.commons.configuration.AbstractConfiguration class. Here is a sample error:
2022-07-29T19:25:43,437+0000 [main] ERROR org.apache.bookkeeper.client.RackawareEnsemblePlacementPolicyImpl - Failed to initialize DNS Resolver org.apache.pulsar.zookeeper.ZkBookieRackAffinityMapping, used default subnet resolver
java.lang.ClassCastException: class java.util.ArrayList cannot be cast to class java.lang.String (java.util.ArrayList and java.lang.String are in module java.base of loader 'bootstrap')

Also, see #6349 and https://github.com/apache/pulsar/issues/6343 for context.
Modifications

Move the castToString method out to a shared class.
Use the castToString method to safely get the configuration value.

Verifying this change
This PR includes a test.
Documentation

[x] doc-not-needed

@lhotari @eolivelli - PTAL, thanks!


Engine singletons portability issues

This issue lists the current problems regarding Engine singletons and the ClassDB API portability. The purpose of this issue is to try to find a solution for these problems.
Before starting, I will describe the rules for registering Engine singletons while respecting API portability. Any Engine singleton that breaks any of these rules will at best cause an API hash mismatch, which is already bad enough for the mono module.

The class must be exposed to the scripting API (ClassDB::register_class). (Otherwise only GDScript would be able to use it, and that is only because right now GDScript doesn't check whether a class is exposed or not).
 This applies to Engine singletons as they must be exposed. The class can be replaced with a dummy one on unsupported platforms.
The class must not be exposed only on builds with tools disabled.
The class may be exposed only on builds with tools enabled if it's exposed as part of the Editor API (ClassDB::current_api at the time it is registered).

However, there's currently a problem that prevents the last rule (4) from working properly. The class VisualScriptEngineSingleton has a property to select between the list of Engine singletons. This doesn't differentiate between singletons exposed as part of the Core and Editor APIs, so the hint string will variate between builds with tools enabled and disabled. This is important because hint strings are considered when calculating the API hash.
I'm not sure how to properly solve this problem. We could simply check if the class is part of the Editor API before adding it to the hint string, but that would mean Editor-only singletons like NavigationMeshGenerator will not be selectable and that may be a problem.
Having made all that clear, I'm listing the current singletons that are causing API portability issues:

JavaClassWrapper is added as a Engine singleton only on Android; this breaks rule 2. The class JavaClassWrapper is not even exposed to the scripting API via register_class, breaking rule 1.

NavigationMeshGenerator is added as an Engine singleton only on builds with tools enabled. This is actually fine as NavigationMeshGenerator is exposed to the Editor scripting API, adhering to rule 4. However, as described above, rule 4 is not working right now because of the VisualScriptEngineSingleton problem.

The JavaClassWrapper issue can be fixed easily. Just make sure to expose the class to the scripting API (register_class) and add it as an Engine singleton, both on all platforms. On non-Android platforms, a dummy class would be used. The reason I haven't done this is because it would need to be moved from its current location at platforms/android/ to somewhere else, and I'm uncertain where the new location should be.
I don't know what to do about the VisualScriptEngineSingleton issue.

cc @reduz Maybe you have a suggestion.

Here's an example of how the VisualScriptEngineSingleton's property hint string differs between builds. The first is from a desktop tools build, the second is from an android non-tools build (export template).
-"hint_string": "Performance,ProjectSettings,IP,Geometry,ResourceLoader,ResourceSaver,OS,Engine,ClassDB,Marshalls,TranslationServer,Input,InputMap,JSON,JavaScript,GodotSharp,NavigationMeshGenerator",
+"hint_string": "Performance,JavaClassWrapper,ProjectSettings,IP,Geometry,ResourceLoader,ResourceSaver,OS,Engine,ClassDB,Marshalls,TranslationServer,Input,InputMap,JSON,JavaScript,GodotSharp",

Notice how the desktop tools build includes NavigationMeshGenerator while the android non-tools build doesn't, and how the android build includes JavaClassWrapper while the desktop build doesn't.

I think it might be possible to write VisualScriptEngineSingleton's property hint in _validate_property. Maybe the one that is registered in _bind_methods could then include only non-editor singletons.

@bojidar-bg Interesting, doing it in _validate_property may actually work.


Cannot create orders due to Google API error

Scenario:

create new driving order
save order to database
order hangs: displaying empty map tile

Browser console display:

Change of policy @google

Policy Change -> New Business Cases (RansomWare?):

Added key to all googleapi requests, as required in first comment (above), no more problems encountered. The choice of STANDRAD vs. PREMIUM is open, a request for information has been sent to Google Sales for clarification.


Cython speedups

Summary
Provide cython generated speedups. For now this feature will be opt-in with the use of HIKARI_EXTENSIONS environment variable. This will later be replaced with an opt-out by making the use of HIKARI_NO_EXTENSIONS
Example:
HIKARI_EXTENSIONS=1 pip install hikari

TODO

[ ] Cythonized test runs
[ ] Cythonize before package release
[ ] Profile and optimize further

Decide whether to use .pxd (type annotations) or .pyx (full rewrite of files in Cython)

Checklist

[ ] I have run nox and all the pipelines have passed.
[ ] I have made unittests according to the code I have added/modified/deleted.

Related issues
Closes #535

I am going to be closing this for now, as there are more important things to focus on at the moment. Cythonizing and installing the whole cythonized package takes much longer than I expected it to. Might open another one in the future with some sort of other compiled speedup.


Vickel: Watch Golden[\W_]+Globes[\W_]+Awards[\W_]+Live[\W_]+Free[\W_]*+2024

Vickel requests the watch of the watch_keyword Golden[\W_]*+Globes[\W_]*+Awards[\W_]*+Live[\W_]*+Free[\W_]*+2024. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Golden[\W_]*+Globes[\W_]*+Awards[\W_]*+Live[\W_]*+Free[\W_]*+2024 has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jesse in Charcoal HQ


python311Packages.hahomematic: 2023.5.1 -> 2023.5.2

Changelog: https://github.com/danielperna84/hahomematic/releases/tag/2023.5.2
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.hahomematic
python311Packages.hahomematic


mx.nd.contrib.boolean_mask doesn't support GradientAddition

Description
Hi, there.
I found mx.nd.contrib.boolean_mask gives a wrong gradient when gradient addition on it.
mx.nd.contrib.boolean_mask doesn't support Gradient Addition.
Environment info (Required)
----------Python Info----------
Version      : 3.7.3
Compiler     : GCC 8.2.1 20181127
Build        : ('default', 'Mar 26 2019 21:43:19')
Arch         : ('64bit', 'ELF')
------------Pip Info-----------
Version      : 19.0.3
Directory    : /usr/lib/python3.7/site-packages/pip
----------MXNet Info-----------
Version      : 1.5.0
Directory    : /home/wkcn/proj/incubator-mxnet/python/mxnet                                                                   Hashtag not found. Not installed from pre-built package.                                                                      ----------System Info----------
Platform     : Linux-5.1.6-arch1-1-ARCH-x86_64-with-arch
system       : Linux
node         : MiraiT
release      : 5.1.6-arch1-1-ARCH
version      : #1 SMP PREEMPT Fri May 31 15:17:53 UTC 2019

Package used (Python/R/Scala/Julia):
Python
Build info (Required if built from source)
Compiler (gcc/clang/mingw/visual studio): gcc
MXNet commit hash:
b0be6c50ae686a59c340c56b9c11493a5c84ff72
Build config:
make -j 5 USE_OPENCV=1 USE_BLAS=openblas USE_MKLDNN=0
Minimum reproducible example
import mxnet as mx

a = mx.nd.array([1,2,3,4,5])
a.attach_grad()

bi = mx.nd.array([1,1,0,0,0])

ci = mx.nd.array([1,0,0,1,0])

with mx.autograd.record():
    b = mx.nd.contrib.boolean_mask(a, bi)
    c = mx.nd.contrib.boolean_mask(a, ci)
    su = b.sum() + c.sum()

su.backward()
print(a.grad)
'''
[2.0000000e+00 1.0000000e+00 0.0000000e+00 1.0000000e+00 5.0171812e+04]
<NDArray 5 @cpu(0)>
'''

Steps to reproduce

Run the reproduce example
The last gradient is wrong and random.

What have you tried to solve it?

I read the code, and I found mx.nd.contrib.boolean_mask doesn't support GradAdd.

The issue has been fixed. Close it.


Replace all occurances of ST_AsBinary when checking types

There are issues with checking types using ST_GeometryType that this doesn't fix, but this does fix the issue of UNIONed tables and mixed types.
In addition to this applying to 0.5.x, it should be backported to 0.4.x, where the regression was introduced.
It fixes the specific error in #320 but that one should be left open or a new issue created about the problems of relying on string replacement like this.

@pnorman could you base this pull request off of v0.6.0. v0.6.0 is the head of our development; not master. For us master is the current stable.

@pnorman good catch! I'm going to spin this into another issue and base it off v0.6.0. I still need to respond to your issue around SQL parsing. One detail you brought up about not using strings.ToLower() has been fixed in the v0.6.0 branch which we're hoping to cut today. Thanks for submitting this fix

@pnorman I just reviewed the code in v0.6.0 and this has been fixed. You can see the fixes here.

@pnorman could you base this pull request off of v0.6.0. v0.6.0 is the head of our development; not master. For us master is the current stable.

The intent for this is for 0.5.1, not 0.6.0

@pnorman we're so close to cutting v0.6.0 and this has been fixed in v0.6.0, so let's hold off on the patch release.

Even if 0.6.0 were out, as a bug fix shouldn't it make it into 0.5.x?


config-tool fails on some secure passwords

Quay: v3.4.0
config-tool v0.1.0
Red Hat Enterprise Linux release 8.3 (Ootpa)
Quay container won't start after upgrade to Quay v3.4.0 caused by some trouble with config-tool
$ config-tool validate -c /quay-registry/conf/stack/ --mode online
panic: runtime error: slice bounds out of range [1:0]
goroutine 24 [running]:
github.com/quay/config-tool/pkg/lib/fieldgroups/database.ValidateDatabaseConnection(0x7ffd55ca9661, 0x6, 0xc0004a9980, 0xc0001882d0, 0x0, 0x0, 0x0, 0x0)
        /remote-source/app/source/config-tool/pkg/lib/fieldgroups/database/database_validator.go:90 +0xca5
github.com/quay/config-tool/pkg/lib/fieldgroups/database.(*DatabaseFieldGroup).Validate(0xc000450fc0, 0x7ffd55ca9661, 0x6, 0xc0004a9980, 0xc0002a7848, 0x0, 0x0)
        /remote-source/app/source/config-tool/pkg/lib/fieldgroups/database/database_validator.go:55 +0x22a
github.com/quay/config-tool/commands.glob..func3.1(0xc0004a97a0, 0xc0004e2cd0, 0xc0004a9980, 0xc000451700, 0xc00040e222, 0xc00040eea0)
        /remote-source/app/source/config-tool/commands/validate.go:108 +0xd4
created by github.com/quay/config-tool/commands.glob..func3
        /remote-source/app/source/config-tool/commands/validate.go:93 +0x4eb

It seems DB_URI contains some problematic content in secure database password. The problematic sign is "?"
After removing this, config-tool works well. Of course I have to change the password in database backend ;-)
Thanks!

Okay, the second complecated sign in the password string ("^") I struggled here: https://github.com/quay/config-tool/blob/f713d004cd2872c8a73fcc176c1f2055fd24a931/pkg/lib/fieldgroups/database/database_validator.go#L46

Thanks for this report! Would you mind opening a ticker in our jira? If not, we can open one on your behalf.
https://issues.redhat.com/projects/PROJQUAY/issues

@thomasmckay  sure: https://issues.redhat.com/browse/PROJQUAY-1585
thx!

Feedback from RedHat support. Password must be urlencode in configuration settings:
origin:  1234567?a^b
urlencode: 1234567%3Fa%5Eb


Garbage collection running wild

Description
Since old tags for our images were piling up, I enabled garbage collection to run with keep_latest: 10 and older_than: 90. From the documentation, I assume this means remove all tags but the last 10 and don't consider any tag younger than 90 days.
However, what happened was a ton of old, admin and template images being removed entirely. The images weren't updated recently, but frequently pulled, as we need them in our daily deployment process.
Steps to reproduce

Enable garbage collection:

PORTUS_DELETE_GARBAGE_COLLECTOR_ENABLED=true
PORTUS_DELETE_GARBAGE_COLLECTOR_OLDER_THAN=90
PORTUS_DELETE_GARBAGE_COLLECTOR_KEEP_LATEST=10

Restart portus
docker-compose up -d
Watch your images diappear

Expected behavior: Remove tags older than 90 days and keep at least 10 around
Actual behavior: Entire images were removed

Providing logs of the moment when the issue has happened would also be
useful. If you are in production, you might want to set the PORTUS_LOG_LEVEL
to debug to get a more verbose log.
Deployment information
Deployment method: docker-compose.yml
version: "3.7"
services:
portus:
image: opensuse/portus:2.4
restart: unless-stopped
environment:
- PORTUS_MACHINE_FQDN_VALUE=xxx
  # DB. The password for the database should definitely not be here. You are
  # probably better off with Docker Swarm secrets.
  - PORTUS_DB_HOST=db
  - PORTUS_DB_DATABASE=xxx
  - PORTUS_DB_PASSWORD=xxx
  - PORTUS_DB_POOL=5

  # Secrets. It can possibly be handled better with Swarm's secrets.
  - PORTUS_SECRET_KEY_BASE=xxx
  - PORTUS_KEY_PATH=/certificates/2019_key.pem
  - PORTUS_PASSWORD=xxx

  # SSL
  - PORTUS_PUMA_TLS_KEY=/certificates/2019_key.pem
  - PORTUS_PUMA_TLS_CERT=/certificates/2019_cert.pem

  - RAILS_SERVE_STATIC_FILES=false
ports:
  - <IP_ADDRESS>:8063:3000
links:
  - db
volumes:
  - /srv/xxx/portus/certs:/certificates:ro
  - /srv/xxx/portus/assets:/srv/Portus/public
extra_hosts:
  - "xxx:<IP_ADDRESS>"
logging:
  driver: "gelf"
  options:
    gelf-address: "tcp://xxx:5555"
    tag: "portus-frontend"
    mode: non-blocking
    max-buffer-size: 1g

background:
image: opensuse/portus:2.4
restart: unless-stopped
environment:
- CCONFIG_PREFIX=PORTUS
- PORTUS_MACHINE_FQDN_VALUE=xxx
  # DB. The password for the database should definitely not be here. You are
  # probably better off with Docker Swarm secrets.
  - PORTUS_DB_HOST=db
  - PORTUS_DB_DATABASE=xxx
  - PORTUS_DB_PASSWORD=xxx
  - PORTUS_DB_POOL=5

  # Secrets. It can possibly be handled better with Swarm's secrets.
  - PORTUS_SECRET_KEY_BASE=xxx
  - PORTUS_KEY_PATH=/certificates/2019_key.pem
  - PORTUS_PASSWORD=xxx

  # Enable garbage collection
  - PORTUS_DELETE_GARBAGE_COLLECTOR_ENABLED=true
  - PORTUS_DELETE_GARBAGE_COLLECTOR_OLDER_THAN=90
  - PORTUS_DELETE_GARBAGE_COLLECTOR_KEEP_LATEST=10

  - PORTUS_BACKGROUND=true
links:
  - db
volumes:
  - /srv/xxx/portus/certs:/certificates:ro
extra_hosts:
  - "xxx:<IP_ADDRESS>"
logging:
  driver: "gelf"
  options:
    gelf-address: "tcp://xxx:5555"
    tag: "portus-backend"
    mode: non-blocking
    max-buffer-size: 1g

db:
image: library/mariadb:10.0.23
restart: unless-stopped
command: mysqld --character-set-server=utf8 --collation-server=utf8_unicode_ci --init-connect='SET NAMES UTF8;' --innodb-flush-log-at-trx-commit=0
environment:
- MYSQL_DATABASE=xxx
- MYSQL_ROOT_PASSWORD=xxx
volumes:
- /srv/xxx/portus/database:/var/lib/mysql
registry:
image: library/registry:2.6
restart: unless-stopped
command: ["/bin/sh", "/etc/docker/registry/init"]
environment:
# Authentication
REGISTRY_AUTH_TOKEN_REALM: https://xxx/v2/token
REGISTRY_AUTH_TOKEN_SERVICE: xxx
REGISTRY_AUTH_TOKEN_ISSUER: xxx
REGISTRY_AUTH_TOKEN_ROOTCERTBUNDLE: /secrets/2019_bundle.pem
  # SSL
  REGISTRY_HTTP_TLS_CERTIFICATE: /secrets/2019_cert.pem
  REGISTRY_HTTP_TLS_KEY: /secrets/2019_key.pem

  # Portus endpoint
  REGISTRY_NOTIFICATIONS_ENDPOINTS: >
    - name: portus
      url: https://xxx/v2/webhooks/events
      timeout: 2000ms
      threshold: 5
      backoff: 1s
volumes:
  - /srv/xxx/portus/registry/storage:/var/lib/registry
  - /srv/xxx/portus/certs:/secrets:ro
  - /srv/xxx/portus/registry/config/config.yml:/etc/docker/registry/config.yml:ro
  - /srv/xxx/portus/registry/init/init:/etc/docker/registry/init:ro
ports:
  - <IP_ADDRESS>:xxx:5000
  - <IP_ADDRESS>:xxx:5001 # required to access debug service
links:
  - portus:portus
logging:
  driver: "gelf"
  options:
    gelf-address: "tcp://xxx:5555"
    tag: "portus-registry"
    mode: non-blocking
    max-buffer-size: 1g

Configuration:
Please note that I had to turn off garbage collection for now to stop portus from causing more damage.


Browsing save data for GBA VC titles.

This simply doesn't work, we get "Failed to open file listing archive" as error message.
GBA VC titles use a different save format, using the same header as the AGBSAVE NAND savegame with the actual save (in GBA native format) attached to it.
http://3dbrew.org/wiki/3DS_Virtual_Console#NAND_Savegame
So, I don't expect you to make this browsable, but maybe a nicer error message would be possible?

Is there a way to detect a GBA title from userland? If I just hardcoded GBA VC title IDs for an error message, it wouldn't work for custom GBA VC injections.

Depends on what you have access to. You can safely detect it from the NCCH ExHeader (via the core version, requires AGB_FIRM):
https://3dbrew.org/wiki/NCCH/Extended_Header#ARM11_Local_System_Capabilities
The savegame itself has '.SAV' as magic number.

I don't have access to the exheader or the raw save data.

Closing as I don't know of any way to get this done in userland.


Grafana dashboards

Description
giant blobs of json converted to a file in manifest folder. @hogepodge

Deploy preview for calico ready!
Built with commit 51599038c13441478cc006d7b5bc2635c09e523a
https://deploy-preview-3593--calico.netlify.app


[DO NOT MERGE] Testing a switch to Cmake

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Attempting to switch to CMake build instead of autotools.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@gonuke, did you want to switch to cmake before we merge #55 ?

@gonuke, did you want to switch to cmake before we merge #55 ?

Thanks for checking in.  I lost track of what was failing here, so I've relaunched the tests to see if I  can take another shot at this

#55 got merged so you should probably rebase and test on that version instead.

Yeah - just did that

Quick question @gonuke. Is this to get conda working on Windows?

That's definitely a primary motivation

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

I see this PR has a few conflicts that I can't resolve as write access to the branch is needed. I was interested in seeing if the ever changing eco system has made the cmake build more possible since this PR was started. So I made an updated version of this PR over on #92 .


fix Injectors not working due to error on use

This fixes the Injectors/implants to fail installing on 1.4 with the error message
JobDriver threw exception in toil Use's initAction for pawn Foobar driver=JobDriver_UseItem (toilIndex=2) driver.job=(UseItem (Job_1426443) A=Thing_ArchotechBioadapter977863)
System.InvalidCastException: Specified cast is not valid.

The issue is simply that <compClass>CompUseEffect_DestroySelf</compClass> is no longer defined and instead has been replaced with <li Class="CompProperties_UseEffectDestroySelf" />.

Thank you! Should be added in 50e2b75


Add sensible defaults to the OSV evaluator to allow running without any configuration

Summary
Configuring the OSV evaluator is a bit of a PITA, but at the same time,
the configuration is normally not really needed for public projects as
they would all use the same OSV instance and the same package databases.
Let's just hardcode sensible defaults so that profiles can be leaner and
UIs don't have to ask users to input all this data.
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Create a very minimal profile:
# sample profile for validating artifact signatures
version: v1
type: profile
name: acme-github-profile-pr-vuln-check
context:
  provider: github
  pull_request:
    - type: pr_vulnerability_check
        def: {}

Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 48.089% (-0.08%) from 48.168%
when pulling 8c51e1351a598ab7a60278c5bf9a81bf7cfcb943 on jhrozek:osv_defaults
into 6e21fc08480819079519b88ded0f6530d073935b on stacklok:main.


net: icmpv4: Zephyr replies to ICMPv4 echo request with broadcast destination IPv4 address

Zephyr replies to ICMPv4 echo request with destination IP address set to <IP_ADDRESS> broadcast address.
These packets should be dropped instead.
icmpv4-bcast-dest.pcap.gz

Can you detail a bit which icmpv4 packet type? (echo req?)
Note there is an option: CONFIG_NET_ICMPV4_ACCEPT_BROADCAST which is set by default (I think it should not actually). Unset it and re-run the test.

The packet is an echo request. If config CONFIG_NET_ICMPV4_ACCEPT_BROADCAST is unset the test case passes. RFC 1122 in <IP_ADDRESS> says "ICMP Echo Request destined to an IP broadcast or IP multicast address MAY be silently discarded." Config should probably be disabled by default.

Thanks for testing this, let's disable it by default


chore: add support for html-webpack-plugin v4

I am using html-webpack-plugin v4
and After reading this issue:
https://github.com/arthurbergmz/webpack-pwa-manifest/issues/56
I tried to solve my injection problem, also following html-webpack-plugin README.md examples helped me through
Hope it might help

Can we please merge this? we need this in our environment

@silentsakky feel free to use my repo temp branch https://github.com/JeandeCampredon/webpack-pwa-manifest/tree/temp directly in you module

please, merge this.

Please merge, otherwise this plugin is unusable with Create React App that uses html-webpack-plugin 4.0.0beta.

@arthurbergmz could we get this merged please?

Would be awesome to get this merged. Thanks advance!

@arthurbergmz can you publish to nom? :) thanks a lot!

@arthurbergmz thank you for merging this! However latest npm publish has an issue with missing dist-directory.


Add embark-target face

Describe your request
I propose to add support for embark-target.
The embark-target face has the following definition:
(defface embark-target '((t :inherit highlight))
  "Face used to highlight the target at point during `embark-act'.")

@tecosaur suggested:

We could always blend it a bit with the background colour

Edit: per some of the tests below, the issue may be more doom-one than a general issue?
Briefly explain its use-case
Now, in Doom One (note the "cite" at top):

As the highlighting remains active until the user exits embark, this is too strong in my view.
Implementing a solution would restore balance :-)
Options
Here's 0.7:
(custom-set-faces!
  `(embark-target :background ,(doom-blend (doom-color 'bg) (doom-color 'highlight) 0.7)))

... and 0.8, which I think looks the best, at least on this theme:

0.9 is too subtle.

Perhaps ideally might tweak the foreground too?
I tried this with most of the themes, and it is mostly an improvement. But there are a few light themes where it's a step back; for example, it doesn't look right on this:

doom-nord-light

Another approach would just be not to inherit from highlight, or to modify highlight to be a bit more subtle on doom-one?
cc @iyefrat

Personally I think that it's good that it's strongly highlighted, but that's just me. I'm not sure how to reconcile my personal theming preferences with what would be a good default for the package.

Personally I think that it's good that it's strongly highlighted, but that's just me. I'm not sure how to reconcile my personal theming preferences with what would be a good default for the package.

I can post a few screenshot examples if someone gives me a starting point code for set-face-attribute and doom-blend.
My experiments haven't worked.

I update the main post with a couple of quick examples.

Since this needs a poke anyway: I think it's nice if the embark highlight doesn't look like a normal selection or a super-fat cursor. I feel like a good mix factor would be something in the 0.3-0.7 range.


discoverServices() retrurning null service?

Hi. I'm facing with discoverServices() error.
BLE logs below. Just tell if You need something more.. Or what is wrong in this code?
final Subscription subscription = bleDevice.establishConnection(false)
                .takeUntil(disconnectTriggerSubject)
                .flatMap(new Func1<RxBleConnection, Observable<RxBleDeviceServices>>() {
                    @Override
                    public Observable<RxBleDeviceServices> call(RxBleConnection rxBleConnection) {
                        Log.d(TAG, "BLE connection to " + bleMac + " done!");*

                    // Add this connection to a map for later use
                    connectionMap.put(bleMac, rxBleConnection);

                    return rxBleConnection.discoverServices();
                }
            })
            .map(new Func1<RxBleDeviceServices, ConnectedDevice>() {
                @Override
                public ConnectedDevice call(RxBleDeviceServices rxBleDeviceServices) {
                    Log.d(TAG, "Got service BLE from " + bleMac);

                    BluetoothGattService ngService = null;
                    for (BluetoothGattService service : rxBleDeviceServices.getBluetoothGattServices()) {
                        Log.e(TAG, "ngServices UUID: " + service.getUuid());
                        Log.e(TAG, "ngServices Instance Id: " + service.getInstanceId());
                        if (NG_SERVICE_UUID.equals(service.getUuid())) {
                            ngService = service;
                            break;
                        }
                    }

                    // If there are no proper service, just throw an error
                    if (ngService == null) {
                        throw new IllegalArgumentException("Connected device not a proper ng device");
                    }

BLE LOGS:
E/BleManager: isConnected: RxBleConnectionState{DISCONNECTED}
D/BluetoothGatt: connect() - device: E5:07:96:5E:94:CD, auto: false
D/BluetoothGatt: registerApp()
D/BluetoothGatt: registerApp() - UUID=04b2518b-01e4-4160-9296-461bfea33036
D/BluetoothGatt: onClientRegistered() - status=0 clientIf=14
D/BluetoothGatt: onClientConnectionState() - status=0 clientIf=14 device=E5:07:96:5E:94:CD
D/BleManager: BLE connection to E5:07:96:5E:94:CD done!
D/BluetoothGatt: discoverServices() - device: E5:07:96:5E:94:CD
D/BluetoothGatt: onSearchComplete() = Device=E5:07:96:5E:94:CD Status=0
D/BleManager: Got service BLE from E5:07:96:5E:94:CD
E/BleManager: Connected device not a proper ng device

Hello @Esperanz0
First of all could you give info what device do you use, OS version, library version does your peripheral is changing services or you use BluetoothGatt.refresh() and how reproducible is this scenario? So far it seems that you are facing a native Android BLE bug.

Hello @dariuszseweryn.
BUG: Sony Xperia Z5 - Android 7.0 ( Updated to 7.1 and still the same problem)
OK: Sony Xperia Z5 Premium - Android 6.0.1
Library:  'com.polidea.rxandroidble:rxandroidble:1.2.3'
I'm not using BluetoothGatt.refresh().
Reproduction:
Connect Phone with Ble device. Restart both ( app and ble device). Try connect again.
After 15-20s its working again OK.
Looks like phone issue? Not a platform?  Any idea how to handle this?

0x85 = 133 which is the dreaded GATT_ERROR. There is no difference in the discovery mechanism in between OS versions. Does your peripheral use Service Changed characteristic?

To be honest. I'm not responsible for peripheral device and don't know.

But you can check on the device that does not exhibit the but if a Generic Attribute Service is discovered.

@Esperanz0 Any luck? Have you tried to postpone service discovery for at least 2 seconds after getting the connection?

@dariuszseweryn Hey. Sorry have no time for this issue now. For now I'm just trying connect again. But will check it soon.

Closing due to inactivity. Too small amount of info. Feel free to reopen if something new will appear.

@dariuszseweryn Speaking of Generic Attribute Service, strangely, when I try to discover services, this one is not found on Nexus 5. But it is found on an Android 8. How could it be possible to find some services but not the Generic Attribute Service ?

Never seen this behaviour


Styles

React.js SASS
Im wondering where is the code responsible for compiling scss file, and minifying the css.
Second question about styles is Im structruing my components along with their scss files, like so
Components/
|--Header/
|------ Header.js
|------ Header.scss

Im now migrating my code to use it with the megaboilerplate, any suggestions on how to get this done?

@upq It's done via node-sass-middleware. If you want to import individual styles into React components, take a look at how this project is doing it. https://github.com/chentsulin/electron-react-boilerplate/tree/master/app/components

@upq Could you please close this issue if that answers your question?


Implement Matrix exercise

I wanted to use this one as an excuse to teach structs, but didn't want to tie them to using a particular internal implementation, except for when rows/columns are output, which is why the example has matrix and transposed_matrix, where the one the users see only has matrix, and the test just checks that it's not nil, and that to_string returns the same string representation as was passed in.

@devonestes Fixed your request for the test, and renamed the module to Matrix, but it'll rely on https://github.com/exercism/xelixir/pull/332 before it passes the CI build.


Static files

Route to net/http static files handler

Too easy to implement, added "static" example.
That should be enough.


Date pipe does not match the UTC in current challenge page

"2021-12-31" will be stored as "2021-12-31T00:00:00Z" in database ending with "Z", which represents UTC. Just using  "2021-12-31T00:00:00Z" | date: "yyyy-MM-dd" will minus one day and show 2021-12-30.

We can create custom Angular pipes if needed. Here are some I created for another project two years ago.

@rrchai Is this issue closed?


trainer is not working

Nice job for making the trainer!
I am getting an error:
Environment ready!
objc[40735]: Class FIFinderSyncExtensionHost is implemented in both /System/Library/PrivateFrameworks/FinderKit.framework/Versions/A/FinderKit (0x7fff94ee23d8) and /System/Library/PrivateFrameworks/FileProvider.framework/OverrideBundles/FinderSyncCollaborationFileProviderOverride.bundle/Contents/MacOS/FinderSyncCollaborationFileProviderOverride (0x122b4bf50). One of the two will be used. Which one is undefined.
Positive Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Positive
Negative Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Negative
Positive Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Positive
Negative Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Negative
(base) Nishchays-MacBook-Pro:app nishchay$ chmod -R a+x ./app
chmod: ./app: No such file or directory
(base) Nishchays-MacBook-Pro:app nishchay$ python3 main.py
Starting...
git version 2.28.0
sh: opencv_createsamples: command not found
Environment ready!
objc[40763]: Class FIFinderSyncExtensionHost is implemented in both /System/Library/PrivateFrameworks/FinderKit.framework/Versions/A/FinderKit (0x7fff94ee23d8) and /System/Library/PrivateFrameworks/FileProvider.framework/OverrideBundles/FinderSyncCollaborationFileProviderOverride.bundle/Contents/MacOS/FinderSyncCollaborationFileProviderOverride (0x128c25f50). One of the two will be used. Which one is undefined.
Positive Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Positive
Negative Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Negative
Output Dir selected:  /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training
sh: -c: line 0: syntax error near unexpected token (' sh: -c: line 0: find /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Negative -name '.png' -o -name '.jpg' > /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Negative/index.txt'
Successfully generated negative images index
sh: -c: line 0: syntax error near unexpected token (' sh: -c: line 0: find /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Positive -name '.png' -o -name '.jpg' > /Users/nishchay/Dropbox/My Mac (Nishchay’s MacBook Pro)/Desktop/Training/Positive/index.txt'
Successfully generated positive images index
ERROR: Unable to open positive images index file.

Thanks for your comment @Determine06
I am looking into this issue. This is the same issue as this one.
The issue is because you don't have essential build tools installed or not included in the path. Check in the terminal if you can run "opencv_traincascade" and "opencv_createsamples" in your terminal.
This is a very common yet not popular issue with the installation of OpenCV. I am looking into the issue and would probably add the instructions soon.

Might be because of openv-cv v4 - The main app might have to be forced to installing only v3 in case it installs v4 in line 35 of main.py? The open-cv documentation mentions that opencv_createsamples and opencv_traincascade is deprecated in open-cv v4: https://docs.opencv.org/3.4/dc/d88/tutorial_traincascade.html
if 'opencv' in missing:
            if python_version[0] =='3':
                os.system('sudo apt-get install -y python3-opencv')
            else:
                os.system('sudo apt-get install -y python-opencv')```


proof of assignment in Verify

The Verify function as generated by Recordflux contains this code:
                     Ctx.Cursors (Successor (Ctx, Fld)) := (State => S_Invalid, Predecessor => Fld);

The predicate seems to be difficult to prove for SPARK currently. Previously, the code was duplicated for each field using a case statement, which helped proof but increased code size. This was recently replaced by a for loop:
               for F in Field loop
                  if Fld = F then
                     Ctx.Cursors (Successor (Ctx, Fld)) := (State => S_Invalid, Predecessor => Fld);
                  end if;
               end loop;

For SPARK, this is somewhat similar to the case statement, as the loop is unrolled and we get each case separately. However the loop unrolling has a threshold of 20. For any message with more than 20 fields, the loop is not unrolled and proof breaks down (it would require a loop invariant). This happens in SPDM code (message Alg_Struct).

The naive solution would be using the loop only in cases where there are at most 20 fields, and the previous approach otherwise. Is there a better solution?

It seems we can now remove the loop, please check out my pull request.


Broken Link for the Build your own connector

When you click on the link Build your own connectors link in the section Include custom external data in search experience, we are getting a broken link.

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

ID: a854a342-c67a-d53f-6b95-3d7366ac3f0e
Version Independent ID: 81581c12-a7a8-706f-ec0d-e2471baef2ca
Content: Microsoft Search API in Microsoft Graph overview - Microsoft Graph
Content Source: concepts/search-concept-overview.md
Product: search
Technology: microsoft-graph
GitHub Login: @snlraju-msft
Microsoft Alias: MSGraphDocsVteam

Closing as this has been fixed. Thanks.


Cost too much memory

I'm not sure whether I used the point-transformer correctly: I just implemented one block for training, and the data shape of (x, pos) in each gpu are both [16, 2048, 3], later I was informed that my gpu is running out of the memory(11.77 GB total capacity)

Oh yup, this type of vector attention is quite expensive. In the paper, they did KNN on each point, and only attended to the local neighbors

You can use this great library to do the clustering https://github.com/rusty1s/pytorch_cluster

@JLU-Neal Hi, I decided to add in the feature for attending only to the k nearest neighbors, which can be set with the num_neighbors keyword argument. Let me know if it works for you! https://github.com/lucidrains/point-transformer-pytorch/commit/2ec1322cbecced477826652c567ed8bc2d31c952

@JLU-Neal Hi, I decided to add in the feature for attending only to the k nearest neighbors, which can be set with the num_neighbors keyword argument. Let me know if it works for you! 2ec1322

Wow, surprised by your efficient work! Wish you a happy Chinese new year!

kNN is only specified for the "transition down" layer. They don't seem to mention it for the general point transformer layer.
So is this just an added bonus or am I missing something from the original paper?

kNN is only specified for the "transition down" layer. They don't seem to mention it for the general point transformer layer.
So is this just an added bonus or am I missing something from the original paper?

In the section 3.2 Point Transformer Layer, author mentioned that "the subset X(i) ∈ X is a set of points in a local neighborhood (specifically k nearest neighbors) of xi."

kNN is only specified for the "transition down" layer. They don't seem to mention it for the general point transformer layer.
So is this just an added bonus or am I missing something from the original paper?

In the section 3.2 Point Transformer Layer, author mentioned that "the subset X(i) ∈ X is a set of points in a local neighborhood (specifically k nearest neighbors) of xi."

On that note, do you think they talk about k-nearest neighbours in the point space? They always refer to the coordinates as p and not X throughout the paper. I've always read that as the KNN in the feature space which although might be less stable, may increase the receptive field quite a bit.

@lucidrains I have the same issue, and cannot get a single layer to run with 12GB GPU Memory. Maybe my understanding of the layer is incorrect but I have made the following observation:
I have a pointcloud in (160000, 3) Format which I put into the layer as both feature and position with nearest neighbors k=8. However I noticed (using smaller input data) that in the forward function the relative position takes up almost pos**2 in memory and does not change with k. Is the implementation here correct?
rel_pos = pos[:, :, None, :] - pos[:, None, :, :]

@lucidrains I have the same issue, and cannot get a single layer to run with 12GB GPU Memory. Maybe my understanding of the layer is incorrect but I have made the following observation:
I have a pointcloud in (160000, 3) Format which I put into the layer as both feature and position with nearest neighbors k=8. However I noticed (using smaller input data) that in the forward function the relative position takes up almost pos**2 in memory and does not change with k. Is the implementation here correct?
rel_pos = pos[:, :, None, :] - pos[:, None, :, :]

I believe this code is designed to support both global and local attention mechanisms by switching knn on and off. The following of code checks whether knn is on and starts to select specific q, k, v, and position embedding based on indices from knn.
https://github.com/lucidrains/point-transformer-pytorch/blob/f9d4e56a26ceee70deb60da230fef40c656396e6/point_transformer_pytorch/point_transformer_pytorch.py#L78


liquidprompt: unstable-2018-05-21 -> 1.12.0

Motivation for this change
Update to version 1.12.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/242


Add ability to construct a new WeatherAPI instance with credentials

Because reading a WeatherAPI.properties on the classpath is too inflexible.

As a quick heads-up - Snowplow wintern @AALEKH is working on a PR to add this functionality. We need this to deliver the weather enrichment in Snowplow...


Add a throttle on the autocomplete requests

Is this issue related to the errors in my console saying "Failed to load resource: the server responded with a status of 429 (Too Many Requests)
https://search.mapzen.com/v1/autocomplete?text=537 2nd&api_key=search-Ff4Gs8o Failed to load resource: the server responded with a status of 429 (Too Many Requests)" ?

^ @dsjoerg yes!
the only change I want to make on your throttle PR is to drop the timeout from 1000ms to 100, the server should be able to handle 100ms just fine :)

OK its there


/media/sd-mmcblk0 folder is not there after update to ccu3

While I had the CCU2 firmware running there was a log file which went to:
DEVLOGFILE=/media/sd-mmcblk0/devlog.txt
however, the folder media/sd-mmcblk0 (used to be the SD card) is gone :-(

This is correct. The original CCU3 does not have this folder anymore and because of this piVCCU3 does not have this folder.
I suggest to use an external usb storage or a folder below /usr/local


Could you support initialUrl? Because a black background page flickers for a moment.

Could you support initialUrl? Because a black background page flickers for a moment.

support initialUrl will bring some broken changes, this is not good for maintenance and expansion, so I didn't do that
maybe you can use some other way to load webview?
  String _urlPath;

  @override
  void initState() {
    initialUrlDynamic();
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: _urlPath == null ? buildLoadingWidget(): buildWebView(),
    );
  }

@asjqkkkk
Thanks for your suggestion. I solved this problem by using about:blank

Hi,
does anyone know when this feature will be available?
It seems to be a great solution for one of my biggest problems.
Thanks for all your work!

Does anyone know if this will be merged in further releases?

Any ETA for the merged release ?

cc @bparrishMines

Possible ETA?

The PR Triage has last been updated on 13 August. Is there a more up to date indication when this will be available on the master branch?

@asjqkkkk why did you end up closing this?

@asjqkkkk why did you end up closing this?

My branch was not updated for too many days, so I think there should be another fresh pr to fix this issue

I take it that's bad news?

Local html file is loading but local assts like image and css are still not loading in Flutter 2.2 stable.


Support typescript indentation

It doesnt seem like tslint indent is supported, which exposes things like interfaces to inconsistent spacing.

Hi @dwasyluk, thanks for submitting this issue. Could you please provide an example?
Any kind of documentation will also be very helpful (maybe a link to TSLint?)

#219

closing this in favor of #201


[Future request] Dial color via tags

Is it possible to make dial color customization via bookmark tags? So if there is a tag starting with # and is a valid color, use it. Bookmark tags are fully synced with bookmarks, so it would be fully integrated and wouldn't require any addition storage and options.

Hi @regs01! Thank you for the suggestion. Based on my research, it doesn't seem possible for an extension to read bookmark tags. Also, Chrome doesn't have tags, and I want to keep functionality the same between Firefox and Chrome. I do plan to add the ability to customize dial colors soon.

I'll leave it here
https://bugzilla.mozilla.org/show_bug.cgi?id=1225916


[DO NOT MERGE] feeds hackery

adds a new configuration section for vulnerabilities provider and associated data feeds
update feed sync process to download the grype-db from the open source url by default (configurable)
minor fixes to grype_wrapper for returning the checksums and not deleting active db directory
circle ci change to push the container images from this branch to dockerhub

No longer relevant


[rockspec] fix luarocks rockspec

so apicast can be installed from luarocks
broken since #486

Travis build fails because there's a broken link, but it's unrelated to this PR.

Fixed.


[LOC] Back-Translation "ヘルプ"→"Help"

https://docs.microsoft.com/ja-jp/nuget/tools/cli-ref-spec

The PR is not being built because the target branch live is filtered in .openpublishing.publish.config.json on live branch.

@hyoshioka0128 様、
ご提案いただきありがとうございます。 ご提案に同意のため、この PR をマージします。ご提案は、次回のアーティクルの更新でも使用されます。
敬具
Microsoft DOCS グローバル エクスペリエンス チーム

The PR is not being built because the target branch live is filtered in .openpublishing.publish.config.json on live branch.

Hello, @hyoshioka0128
Thank you for your feedback and suggestions.
This proposed file change has been accepted. The suggestion will be merged to this repository.  We will release this article on the next localization release round.
Many thanks for your support.
Kind regards,
Microsoft C&E Localization Team


feat: introduce ui tests based on vscode-extension-tester

Signed-off-by: Ondrej Dockal<EMAIL_ADDRESS>

@ssbarnea I had to update npm script "webpack" with npm run compile in order to compile the ts classes on out folder. also their placement in out/client/ui-test is no quite fortunate. It all could be probably done better.

@ssbarnea I had to update npm script "webpack" with npm run compile in order to compile the ts classes in out folder. also their placement in out/client/ui-test is no quite fortunate. It all could be probably done better.

While trying to setup chai, I tried to use the test/ root repository but I did not know all the tricks required for making ts compile automatically. If we could add a follow-up to enable us to use npx mocha directly it would be even better.

@webknjaz Shortly npm test is just a shortcut to npm run test and we had no real test running.
#204 added the first part of extest preparation of vscode and installation of the extension, this channels is the follow-up for completing the test enablement. The are no reasons to have different commands, especially as you cannot really test the extension without installing it with extest.


Document response phase.

New :response() plugin method.

Does this change mean that there is no longer a limitation for Go plugins (ie, previously no header_filter or body_filter phases)?

Go plugins still can't define header_filter or body_filter methods (because those OpenResty phases are non-yielding).  But now they can define a response method.

Go plugins still can't define header_filter or body_filter methods (because those OpenResty phases are non-yielding). But now they can define a response method.

What's the upside/downside of this approach for Go plugins? In the Enterprise doc, currently the lack of header_filter and body_filter is listed as a limitation; by having response instead, does it provide equal configurability, or is there a downside (eg, I can see "lack of flexibility" with these methods being a "limitation").

not sure if there's any missing information.  _filter phases work but can't yield, so Go plugins can't define them.  we added the new :response(), which can yield so are usable from Go.   as it relies on buffering, it's turned on automatically and as noted, can't handle HTTP/2 upstreams.
for Go plugins it's not much of a pros/cons thing, since it's the only way to handle responses


Print nice message when odigos install is called when odigos is already installed

Currently, when odigos is already installed, and one calls odigos install, the following output is ejected to console:
➜ odigos install
Installing Odigos version v0.1.75 in namespace odigos-system ...
Creating namespace odigos-system                  X
ERROR namespaces "odigos-system" already exists

I think it would be nicer to print some useful friendly message, like:
Odigos is already installed in namespace "odigos-system".
If you wish to re-install, run "odigos uninstall" first.

@HeetVekariya Since you are already working on the odigos uninstall fix, would you be interested in fixing this one as well?

To check if odigos is already installed, one can use GetOdigosNamespace function to check if there is an existing odigos installation in the cluster.

@blumamir Sorry for the silence, as i was in college.
I see that you have assigned me two issue, thank you for that.

closed this as completed


HTML injection issue with linter

It seems that HTML code is not properly escaped before displaying.
In the example screenshot I expect it says Expect '}' to match '{' from line 1 and instead saw '
Hello World
'. (E020) at line 2 col 14 instead of rendering the html code.

Fixed in master

exactly where and how was this fixed? I'm interested in seeing the commit.

@AsaAyers I was gonna do a PR but accidently pushed to master (caffine level = low), it's one commit where it replaces html with text in legacy.
Here's the commits https://github.com/AtomLinter/Linter/commit/d78dc2e86072f6649b410d7bbb5d56b3ee97c946 https://github.com/AtomLinter/Linter/commit/630fecc5bd285b13ba32781b17835a0f38867701


[risk=low][RW-6187] rm user.data_access_level and fix a few typos

Relates to https://github.com/all-of-us/workbench/pull/4786 and https://precisionmedicineinitiative.atlassian.net/browse/PD-6040

Actually you can't remove BQ columns!  Never mind.


Change Outline Explorer font size

Description of your problem
What steps will reproduce the problem?

 What do you see instead?
The Outline Explorer font size should increase.
Instead, I see that it does not.
Please provide any additional information below
Versions and main components

Spyder Version: 3.2.0
Python Version: 3.6.1
Qt Version: 5.6.2
PyQt Version: 5.6.0
Operating system: Mac OS X 10.12.6

Dependencies
IPython >=4.0    :  5.3.0 (OK)
cython >=0.21    :  0.25.2 (OK)
jedi >=0.9.0     :  0.9.0 (OK)
nbconvert >=4.0  :  5.2.1 (OK)
numpy >=1.7      :  1.13.1 (OK)
pandas >=0.13.1  :  None (NOK)
psutil >=0.3     :  5.2.2 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.5.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.6.4 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.6.2 (OK)
sympy >=0.7.3    :  None (NOK)

Thanks for reporting. We'll fix this problem in a future release.

@ccordoba12 I don't want to be a pain with this hehe, but if we do this for the Outline Explorer, then why not for the File explorer, the Find in Files, the Project Explorer, the File Switcher, the Breakpoint explorer, etc. And while we are at it, why not also changing the font of the dialog, the buttons, and the menus... You see where I'm going with this hehe? This is kind of related to the discussion we had in Issue #5185.
Since the font option in general is not used to change the font of the spyder GUI overall, I think that the font size for the Outline Explorer should remain the job of the OS. I still think we should add a little description of what this option do exactly, so that people do not expect that changing the font size will affect the entire GUI.

Hmmm, thanks for pointing this problem out. We need to think carefully how to address this problem....

I'm using spyder3 on a 4K monitor in a lab, actually.
And these fonts appear really small and tiring to read along the day.
I'd be glad to know what setting in MacOS can increase the font size of :

Outline Explorer, File explorer, Find in Files, Project Explorer, File Switcher, Breakpoint explorer, etc.

If you where this setting is, please let me know :-)

Maybe trying this could help in the meantime https://www.laptopmag.com/articles/increase-text-size-mac ?

@sebma, if you are using a 4K screen, please adjust your resolution in
Preferences > General > Screen resolution

That should fix this problem for you.

We should maybe allow a "zoom in/zoom out/zoom reset" option to all of our plugins and we should probably save these levels of zoom in the config.
Having options on the preferences for each pane is definitely a no go, but if someone want a specific panel to be bigger, Context menu and default shortcuts should probably offer that option.
Thoughts @spyder-ide/core-developers ?

Thoughts?

Sounds good.


eda genomics: wgcna network : link to gene list: new tab if different website (maybe even if same site?)

one can't use the right click to open in a new tab..

We are already opening eda in a new tab, so this would mean we are opening two new tabs, which doesn't seem ideal to me.
My proposal is that we do not open these things in new tabs, but use links, so that users can ctrl-click or right-click.


there was an issue deleting the selected datasets

Trying to bulk delete (example) datasets raises There was an issue deleting the selected datasets: Fatal error, but the datasets have actually been deleted.

Actual results
docker-compose log
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:41 +0000] "DELETE /api/v1/chart/?q=!(57,56,55,54,53,52,51,50,49,48,47,46,45,44,43,42,41,40,39,38,37,36,35,34,33) HTTP/1.1" 200 32 "http://<IP_ADDRESS>:8088/chart/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc&viewMode=table" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:41 +0000] "GET /api/v1/chart/?q=(order_column:changed_on_delta_humanized,order_direction:desc,page:0,page_size:25) HTTP/1.1" 200 1348 "http://<IP_ADDRESS>:8088/chart/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc&viewMode=table" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:41 +0000] "GET /api/v1/chart/favorite_status/?q=!(32,31) HTTP/1.1" 200 61 "http://<IP_ADDRESS>:8088/chart/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc&viewMode=table" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:44 +0000] "GET /health HTTP/1.1" 200 2 "-" "curl/7.88.1"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:45 +0000] "DELETE /api/v1/chart/?q=!(32,31) HTTP/1.1" 200 31 "http://<IP_ADDRESS>:8088/chart/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc&viewMode=table" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:45 +0000] "GET /api/v1/chart/?q=(order_column:changed_on_delta_humanized,order_direction:desc,page:0,page_size:25) HTTP/1.1" 200 668 "http://<IP_ADDRESS>:8088/chart/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc&viewMode=table" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:46 +0000] "GET /api/v1/dataset/_info?q=(keys:!(permissions)) HTTP/1.1" 200 98 "http://<IP_ADDRESS>:8088/tablemodelview/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:46 +0000] "GET /api/v1/dataset/?q=(order_column:changed_on_delta_humanized,order_direction:desc,page:0,page_size:25) HTTP/1.1" 200 7023 "http://<IP_ADDRESS>:8088/tablemodelview/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:47 +0000] "POST /superset/log/?explode=events&dashboard_id=10 HTTP/1.1" 200 20 "http://<IP_ADDRESS>:8088/tablemodelview/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | 2023-07-25 10:32:55,473:ERROR:flask_appbuilder.api:Parent instance <SqlaTable at 0x7f9704082d30> is not bound to a Session; lazy load operation of attribute 'database' cannot proceed (Background on this error at: https://sqlalche.me/e/14/bhk3)
superset_app          | Traceback (most recent call last):
superset_app          |   File "/usr/local/lib/python3.8/site-packages/flask_appbuilder/api/__init__.py", line 110, in wraps
superset_app          |     return f(self, *args, **kwargs)
superset_app          |   File "/app/superset/views/base_api.py", line 122, in wraps
superset_app          |     raise ex
superset_app          |   File "/app/superset/views/base_api.py", line 113, in wraps
superset_app          |     duration, response = time_function(f, self, *args, **kwargs)
superset_app          |   File "/app/superset/utils/core.py", line 1594, in time_function
superset_app          |     response = func(*args, **kwargs)
superset_app          |   File "/usr/local/lib/python3.8/site-packages/flask_appbuilder/api/__init__.py", line 182, in wraps
superset_app          |     return f(self, *args, **kwargs)
superset_app          |   File "/app/superset/utils/log.py", line 266, in wrapper
superset_app          |     value = f(*args, **kwargs)
superset_app          |   File "/app/superset/datasets/api.py", line 775, in bulk_delete
superset_app          |     BulkDeleteDatasetCommand(item_ids).run()
superset_app          |   File "/app/superset/datasets/commands/bulk_delete.py", line 49, in run
superset_app          |     security_manager.find_view_menu(model.get_perm()) if model else None
superset_app          |   File "/app/superset/connectors/sqla/models.py", line 719, in get_perm
superset_app          |     if self.database is None:
superset_app          |   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/attributes.py", line 481, in __get__
superset_app          |     return self.impl.get(state, dict_)
superset_app          |   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/attributes.py", line 941, in get
superset_app          |     value = self._fire_loader_callables(state, key, passive)
superset_app          |   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/attributes.py", line 977, in _fire_loader_callables
superset_app          |     return self.callable_(state, passive)
superset_app          |   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/strategies.py", line 862, in _load_for_state
superset_app          |     raise orm_exc.DetachedInstanceError(
superset_app          | sqlalchemy.orm.exc.DetachedInstanceError: Parent instance <SqlaTable at 0x7f9704082d30> is not bound to a Session; lazy load operation of attribute 'database' cannot proceed (Background on this error at: https://sqlalche.me/e/14/bhk3)
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:55 +0000] "DELETE /api/v1/dataset/?q=!(23,22,21,20,19,18,17,16,15,14,13,12,11,10,9,8,2,1,7,6,5,4,3) HTTP/1.1" 500 26 "http://<IP_ADDRESS>:8088/tablemodelview/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
superset_app          | <IP_ADDRESS> - - [25/Jul/2023:10:32:55 +0000] "GET /static/assets/6abe473e8ad84b72d7f9.chunk.js HTTP/1.1" 200 829 "http://<IP_ADDRESS>:8088/tablemodelview/list/?pageIndex=0&sortColumn=changed_on_delta_humanized&sortOrder=desc" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"

Screenshots

Environment

browser type and version: Chromium 114
superset version: 2.1.0 and 2.1.1rc2

I will try other versions too.
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
none

@eschutho I can't reproduce the error on master so the fix is already merged but I don't know which PR. I'm not sure how easy it is to cherry pick it into 2.1.1. If it's hard, the fix will be available in 3.0.

Maybe we can try to git bisect to see where the fix was and try to patch it into 2.1.2, although I agree that 3.0 will most likely be released first.

I think the fix was in https://github.com/apache/superset/pull/24488, which means being a migration, I won't be able to pull this into 2.1.2, but I'll close this ticket since it's fixed in the master branch and in 3.0.


VALVE should trim the structure column before testing whether it is empty

See this line of code which checks to see whether the structure column is empty. This will crash VALVE if the structure contains nothing but whitespace. We should trim the structure of whitespace first before doing this check.

After a bit of investigation, this is going to involve slightly more work than just a one-line change. (Which is not to say that the one-line change in the example above won't work, but that's only one instance, and this is a general problem.) To fix this properly we will need to systematically map the rows as we read them from the config files (i.e., right after calling read_tsv_into_vector() throw a function that trims the values in each column of the row. It's not a big change but it's non-trivial, and this issue isn't a priority right now so will have to get to it later.

After a bit of investigation, this potential problem shows up in multiple places and not just at the line indicated. It won't be a very big job either to make all the corrections one by one or systematically, but since this isn't a huge priority right now (just make sure not to have extra whitespace in your configuration files), it will go lower down on the list.

I've added some code to the function that is used to read configuration files, so that we emit a warning whenever a value in one of the configuration files has leading/trailing whitespace. This should be enough to enable the user to figure out the problem. That commit is on the new-schema branch, which will be merged once I've implemented the new schema for VALVE.

UPDATE: Instead of just a warning I've made it so that an error message is emitted instead, after which VALVE just exits. I don't think there is any need to support leading/trailing whitespace in configuration file cells, and it is very easy to fix for the user. The relevant commit is here: https://github.com/ontodev/valve.rs/commit/b4fc0e16c42aea7131e553d165de6099b388992c


我截取的图片不完整

我运行的 page2image https://www.baoquan.com
截取的图片不完整
我在Windows10上使用的

please using template to create an new issue and below is my screenshot by using page2image CLI commend. it seems quite complete 🤔
page2image https://www.baoquan.com/

you could just comment below and don't need to create an new issues 😀
Node.js & npm version
Node.js: node -v;
npm: npm -v
Operating system:
Description of the bug:
Example CLI command or javascript code to reproduce the error

Error infos:

Node.js:  v7.10.0
npm: 4.2.0
Operating system: Windows10
Description of the bug:

我截取下来的图片：

It seems like because this page uses the React framework and the timing of the screenshot is networkidle by default. Although the network is idle in this moment, React still have not finished render this page yet.
So there is the problem.
You cloud using below command wait until they rendered the copyright element
page2image https://www.baoquan.com --sleep=.copyright

Or you cloud call page2image via the Module to more accurately determine the timing to take a screenshot


Support for listing online alternative

In light of Covid-19 lockdowns, physical distancing etc...
If a meeting type is marked TC (i.e. has a Temporary Closure), could we allow meetings to publish an online alternative, with a clickable link to the online platform? e.g. Zoom, Google Meet, Skype etc.
I have no idea if I'm escalating this need in the right fashion, but I am noticing many meetings are currently discussing the need for online alternatives. I feel like it would be wise to streamline this so that physical meeting attendees can look up their usual meeting and find the online alternative on the same screen.

This is sort of a big question. I've been helping online intergroup rethink their meeting finder and move to a database over here: https://github.com/code4recovery/oiaa
The ultimate goal would be go get online meetings into the Meeting Guide app. For that, we'd need to create a fully new spec for online meetings (no address but yes timezone) and a new section of the app to list online meetings cares about time and not geolocation. (Pending GSO's approval.)
This is a weird situation though, where we want to enable meetings that are geographical to list an online alternative, so it might be as simple as adding a video_conference_url property to the feed to enable a button?
One concern Is that the URL is so open-ended, what if people used it for a Discourse website or Google group or Venmo account? I suppose we could annotate that it's only Zoom and Google Hangouts (and potentially others) for now?


Broker prefix is now configurable

Pull Request Test Coverage Report for Build 301

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 91.03%

Files with Coverage Reduction
New Missed Lines
%

cf/config.go
2
84.09%

Totals

Change from base Build 298:
0.0%

Covered Lines:
548

Relevant Lines:
602

💛  - Coveralls


Log threads at the time SQL deadlock exceptions to labkey.log

We already do this for exception reports. This will log locally as well.

Looks good to me.


Enabling glClipControl for OpenGL renderer

In renderer_gl.cpp, lines 2354-2359 read:
//				if (s_extension[Extension::ARB_clip_control].m_supported)
//				{
//					GL_CHECK(glClipControl(GL_LOWER_LEFT, GL_ZERO_TO_ONE) );
//					g_caps.originBottomLeft = true;
//				}
//				else

Can this code block be made enable-able with a #define? Eg
#define BGFX_OPENGL_HETEROGENEOUS_DEPTH 1
Cheers!

This code is not guaranteed to be available.
You can enable it in your fork.

Thanks for getting back to me. I've written a script to uncomment it in my fetch-from-github-and-compile script. Reverse Z is now working fine in my OpenGl renderer
Cheers!


Enables linting across the entire project

Enables lint checking in pre-commit
Skips examples/ directory
Fixes type annotation with Variant; Charles confirmed this change
Ensures all files (non-examples) have annotations from __future__

This is a pre-curser to considering to enable the static analyzer. Approximately 250 analyzer errors (out of nearly 3000) got cleaned up as a consequence of making the linter happy.

The linter will accept both types of docstrings and I haven't decided on one over the other yet. We should discuss as a team/group and pick one.
I would lean towards Sphinx so we can auto-generate docs and I find it is a little easier to work with and probably easier for a computer to parse.
One thing we definitely shouldn't do is duplicate the typing info in the docstrings as most tools should be able to parse both and mypy/VS Code only looks at the type annotations.


Blazor forms and validation topic feedback

This documentation has gaps in it larger than Florida sink holes.    All of this leads to lower engineer productivity.  What gaps?  Let me list them:

Where is the code example?  If you are going to do code snippets, it should be required that there be an accompanying code example.  That is the first and best way to fill in the gaps.
Can Model and EditContext coexist?  There is nothing here that says they can or cannot.  So I'm reduced to trial and error (Boom! Error: System.InvalidOperationException: EditForm requires a Model parameter, or an EditContext parameter, but not both.)  Why couldn't that have been noted in this doc?
The form may be valid, but that doesn't mean the business transaction/flow can complete.  (e.g. "User already exists.") How to show that form data is incorrect (vs acceptable)?  A bank withdraw is another example.  Entering $100 as the amount to withdraw may be an acceptable form entry, but incorrect for the business transaction since there is only $50 in the account.  How is this kind of error communicated back to the form?
"You just gonna leave me hangin', bro?"  in the example just above "InputText based on the input event"  there is the beginning of an answer to 3, but a powerful code example was replaced with ellipses.  ("Hah, no matter, I can just go to the code example....")

All of the beautiful architecture, elegant design and hard work of developing Blazor is WORTHLESS without comprehensive documentation.  Documentation makes adoption easier, increasing productivity, and isn't Developer Productivity one of the tenets of this whole paradigm shift?  Close the loop and make great documentation to go along with a great product.
P.S. The bloggers are great, but, they rarely revisit a topic unless there is a major change.  Just look at how many use @function{} in their Blazor examples!  Authoritative documentation has to come from the source.

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

ID: eef47457-c8cf-74a4-b884-c1fe5c2ef52d
Version Independent ID: 09a02d10-2834-467d-d8c2-3fd19ceb3863
Content: ASP.NET Core Blazor forms and validation
Content Source: aspnetcore/blazor/forms-validation.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @bdnts ... Thanks for your feedback, and I agree with your editorial remarks. [btw -- ❤ your Florida reference ... I live in Pensacola! 🏖] Let's take a look at your points ...

The snippets are "code examples;" so when you say "code example," do you mean a running sample app? If so, it's a dup of https://github.com/dotnet/AspNetCore.Docs/issues/14216. We plan to perform a big sample app overhaul to re-align the sample app with the text. The reason that the sample app diverged from the topics was that our workload for basic scenario coverage in the topics required all of our time. The plan is to get several of the missing scenarios, such as the forms and validation coverage, into the sample app for live demonstration.
Sure thing, we can get that surfaced.
(and 4) Yes, we can flesh that out further. On the first version of the topic, we wanted to show the basic pattern. I'm interested in using a custom validation attribute with the ServerValidate method, if possible. I don't have any such example right now to show you ... I'll see what we can do and what engineering suggests when I work this issue.

Artak will add his notes/ideas and get this prioritized. Note that we're all 🏃🏃🏃😅 for the upcoming 3.2 release at the moment, so there could be a delay of up to eight weeks. That's a worst case scenario ... I don't think it will take that long. I'm just letting you know that we're a very small team with a heavy workload and backlog. We'll get to this as soon as we can. Worst case scenario is that we get to this after 3.2 releases (May), but I'll try to get to it before then.

Hi Luke,
Thanks for the response.  I try to be constructive in my criticisms, but, boy you should have seen the first draft!  Lol.  It seems the big points on everything from MS are done well, but when you get down to making something real, there are these gaps that waste hours doing research.  I’m 4 months behind on a critical project—and I’m the boss!—because I’m dying by a thousand questions, that I feel should be in the documentation.
I was employee 38 for BEA Systems (Tuxedo, Weblogic) and their first EA back in the 90s.  I know the time pressures when it comes to delivering product.  I saw on one of the requests for example around LinkGenerator that broke in a new version.  It took 6 months for the team to deliver.  Whhooaa!  I feel sorry for you all, there must be serious resource constraints, because the willingness was 110% there.  You guys must really be under the gun to deliver new products.  I am very sympathetic.
And I’ll add I love Blazor.  I’ve been developing for nearly 40 years, on just about anything you can imagine, and some things you can’t.  Blazor and SignalR represent breakthrough technological approaches (the socket), which was pretty much abandoned for the web.  With the return of a continuous circuit, the possibilities are endless.  Once I get through my current deliveries, I’ll be posting full examples to Github to share.

Yes, running application.  I believe everything must be QAed.  One part of QAing documentation is testing with a running example to insure the snippets are bug free.  I will check out #14216.
Thank you.
And 4 Cool.  Taking multiple sweeps at a topic is called the Nautilus pattern in literature—working down from the macro in broad strokes down to the minute details, and is a powerful technique.  ServerValidate is also where my interest lies.  The big rock I’m trying to move at the moment is actually Identity with Blazor.  I have test project at https://github.com/bdnts/BlazorIdentity where I’m trying to work out the details of scaffolding MS Identity in an existing Blazor project.  Then I found BlazorBoilerPlate and BlazorWithIdentity.  I’m not convinced about their wholesale use of APIs, so I’m working a new branch and have converted Register.cshtml/.cs to Register.razor.  This is where I’m using the  and failure testing with duplicate names.  Working my way through EditContext, that’s where all this came up. If I come up with anything on ServerValidate, I’ll let you know.

Cheers
Brian Douglass
<EMAIL_ADDRESS><EMAIL_ADDRESS>702-254-5485
From: Luke Latham<EMAIL_ADDRESS>Sent: Wednesday, March 18, 2020 6:38 PM
To: dotnet/AspNetCore.Docs<EMAIL_ADDRESS>Cc: bdnts<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/AspNetCore.Docs] Gaps bigger than Florida sink holes (#17360)
Hello @bdnts https://github.com/bdnts  ... Thanks for your feedback, and I agree with your editorial remarks. [btw -- ❤ your Florida reference ... I live in Pensacola! 🏖] Let's take a look at your points ...

The snippets are "code examples;" so when you say "code example," do you mean a running sample app? If so, it's a dup of #14216 https://github.com/dotnet/AspNetCore.Docs/issues/14216 . We plan to perform a big sample app overhaul to re-align the sample app with the text. The reason that the sample app diverged from the topics was that our workload for basic scenario coverage in the topics required all of our time. The plan is to get several of the missing scenarios, such as the forms and validation coverage, into the sample app for live demonstration.
Sure thing, we can get that surfaced.
(and 4) Yes, we can flesh that out further. On the first version of the topic, we wanted to show the basic pattern. I'm interested in using a custom validation attribute with the ServerValidate method, if possible. I don't have any such example right now to show you ... I'll see what we can do and what engineering suggests when I work this issue.

Artak will add his notes/ideas and get this prioritized. Note that we're all 🏃🏃🏃😅 for the upcoming 3.2 release at the moment, so there could be a delay of up to eight weeks. That's a worst case scenario ... I don't think it will take that long. I'm just letting you know that we're a very small team with a heavy workload and backlog. We'll get to this as soon as we can. Worst case scenario is that we get to this after 3.2 releases (May), but I'll try to get to it before then.
—
You are receiving this because you were mentioned.
Reply to this email directly,  https://github.com/dotnet/AspNetCore.Docs/issues/17360#issuecomment-600941184 view it on GitHub, or  https://github.com/notifications/unsubscribe-auth/ANLUPMSE3BS24OQUI2OGRNDRIFZO7ANCNFSM4LO5JIQQ unsubscribe.  https://github.com/notifications/beacon/ANLUPMQHFFC7Y3OOALXTESLRIFZO7A5CNFSM4LO5JIQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEPI2FAA.gif

Related: #17746.


The fixes a bug where nil date gets converted to 0 which is 01/01/1970

This applies changes that was approved and merged into simperium repo. see
https://github.com/Simperium/simperium-ios/pull/548

LGTM


Add 'platform_code' in 'stops.txt'

platform_code comes from the GTFS specification (see stops.txt). Please review carefully as I'm not a data and/or model expert.

I'm wandering about adding some restriction on this field with a location_type=0 (stop_point) and 4 (boarding_position). Thoughts about that ?

I'm wandering about adding some restriction on this field with a location_type=0 (stop_point) and 4 (boarding_position). Thoughts about that ?

sound great, a platform code as no meaning for a stop_area, zone and even less for path way objects

@woshilapin if it's ok with you, can you add this in the description field ? thanks !

thanks :+1:


symfony 4 support

Hi,
Do you have any plans to implement symfony 4 support?
Best regards,
Michal

Hi Michal,
Short answer is yes, but in 3-6 months.
Long answer is that this GDPR bundle is part of a larger application which is still in development on Symfony 3.4.
I plan to migrate the entire application to Symfony 4 once I get everything to stable in 3.4. Thats probably 3-6 months away.
Feel free to fork and modify for your needs. I can merge any pull requests so long as they are backward compatible with 3.
Regards,
Mark

Hi,
We've made a start on adding Symfony 4 support to this bundle, by removing the dependency on symfony/symfony and replacing it with dependencies on individual components. The composer.json in the GdprBundle references the symfony-4 branch of EncryptBundle. That would need to be changed if it were merged in.
GdprBundle: https://github.com/dailyinfo/GdprBundle/tree/symfony-4
EncryptBundle: https://github.com/dailyinfo/EncryptBundle/tree/symfony-4
We also made a Flex recipe to automate some default configuration for EncryptBundle. It's using Flex's generate(secret, 32) to generate an encryption key. generate() uses bin2hex so not sure if that's suitable.
Flex recipe: https://github.com/dailyinfo/flex-recipes/tree/master/specshaper/encrypt-bundle/1.0

@danabrey
Thanks for your contribution, I have merged your alterations.
If they're working satisfactorily for you then I will close this issue.
Cheers,
Mark

@eventhorizonpl @mogilvie I've opened a pull request on the Flex recipes contrib repository to add the recipe for EncryptBundle: https://github.com/symfony/recipes-contrib/pull/349
It will be available to anyone who has enabled the contrib repository as soon as it's merged.
GdprBundle doesn't require a recipe, as the only direct configuration required is to enable the bundle in the kernel, which Flex does by default.


[Event Request] report 699 Calculate Plan - Req. Wksh.	OnAfterItemOnAfterGetRecord

Hello,
Is het possible to get the following event:
              RespectPlanningParm);

            if PlanningAssignment.Find('-') then
                repeat
                    if PlanningAssignment."Latest Date" <= ToDate then begin
                        PlanningAssignment.Inactive := true;
                        PlanningAssignment.Modify();
                    end;
                until PlanningAssignment.Next() = 0;
            **// DINNL <<
            OnAfterItemOnAfterGetRecord(ReqLine, Item, CurrTemplateName, CurrWorksheetName, FromDate);
            // DINNL >>**

            Commit();

**[IntegrationEvent(false, false)]
local procedure OnAfterItemOnAfterGetRecord(var ReqLine: Record "Requisition Line"; Item: Record Item; CurrTemplateName: Code[10]; CurrWorksheetName: Code[10]; FromDate: Date)
begin
end;**

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


add_features fails

It seems they have removed **DataFrame.to_dense ** from pandas. So I cannot add features to my data set.
"  Removed Series.to_dense, DataFrame.to_dense (GH26684)"

neurosynth is officialy deprecated, so the only possible solution will be to downgrade pandas
see: https://github.com/neurosynth/neurosynth/issues/96

I updated requirements.txt to recommend 0.x series pandas (i.e. not 1.x). Closing this as a dupe of #96


Upgrade to Rails 4.2 latest version.

It removes info from Readme, adds specific values, set ruby-saml dependency to master, add unnecesary files...

The original REAMDE content is very old out of box Rails readme, make no sense for Rails 4.2 now.
specific values can be removed if you think it's right.
ruby-saml master is necessary, if can release a new version I can use the new version.


Add ConnectionError retrying Mixin

High Level Description
Modifies the Marathon API client in test_util to ignore Connection and Timeout errors
Related Issues
https://jira.mesosphere.com/browse/DCOS-14969
Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: DEFLAKE WORK
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

@lingmann I will go ahead and add a logging statement. Good tip


operator configuration

How to change the operator's configuration correctly?
When the config map is out of the mine, you need to restart the operator, which is inconvenient.
I tried to use the ClickhouseOperatorConfiguration crd, an entry appears in the logs:
I0415 08: 43: 33.394833 1 announcer.go: 215] addChopConfig (): clickhouse-operator / configuration-v1: new, previously unknown config, need to apply
but the configuration change is not applied.
If you restart the pod, the changed configuration will be used.

Operator configuration changes are rare, therefore hot configuration deployment was not implemented. This is in TODO list, but not very high due to over priorities. For now pod restart is considered as an acceptable solution.


update to use with multiple html & scss in head

Hi Brad
Issue
I wanted to use the Boilerplate for my project and quickly found out that it does not really work if you have multiple html files.
So I changed the build run script.
Second Issue
After that I found out that if you include the SCSS in the JavaScript, for some reason it will only work in one html file and not in the other. So I included the SCSS file in the head according to the documentation of parcel, which will recognize the file and will complie it to CSS with the hash.
This has some advantages:

Able to use it with multiple html files
If a site has no Script, you don't need to extra add a JS script to have your CSS (like a 404 page...)
It's closer what people are used to, to put the CSS in the head

What's still not so nice
The dev script will only work for the index.html file unless it's changed to for example about.html
But I did not really find a solution for that
I hope it helps and thanks for the great videos.
Sam

the template.content in return template.content.clodeNode(true) in the App.js file is undefined


ENH: Circulant matrix inverse and multiplication

Is your feature request related to a problem? Please describe.
Scipy currently does not include functionality to multiply and calculate inverse of circulant matrices.
Describe the solution you'd like.
Since circulant matrices are diagonalized by the DFT matrix, multiplication and inversion operations are cheap and fast. Would there be interest in having these implemented in scipy?
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

Much of my response will borrow from discussion on #11346, if you want to see detailed reasoning instead of a summary.  The main argument against adding similar functions on that PR seems to be that it adds an additional function to the SciPy API, and for such simple functions they weren't sure of the utility.
Since implementations of multiplication and inversion of circulant matrices without error-checking are one-liners, I suspect similar concerns would be raised here.  I think the awareness of such methods is still useful, so might you consider adding those examples to the documentation of la.circulant?  The other options would be starting a collection of tutorials for performant linear algebra on structured matrices or adding the methods to a package collecting similar methods such as PyLops.