make e2e-test-kind fails

I am following the instructions from https://github.com/cloudnative-pg/cloudnative-pg/blob/main/contribute/e2e_testing_environment/README.md and trying to run E2E tests locally for my changes. I am pointing it to my docker hub repo.  It looks like the script does not like some of the arguments.  What should we do different?
amcohen@us.ibm.com@Aaron-IBM-MacBook-Pro temp % git clone git@github.com:cloudnative-pg/cloudnative-pg.git
Cloning into 'cloudnative-pg'...
remote: Enumerating objects: 31571, done.
remote: Counting objects: 100% (658/658), done.
remote: Compressing objects: 100% (323/323), done.
remote: Total 31571 (delta 424), reused 498 (delta 322), pack-reused 30913
Receiving objects: 100% (31571/31571), 16.91 MiB | 4.92 MiB/s, done.
Resolving deltas: 100% (23989/23989), done.
amcohen@us.ibm.com@Aaron-IBM-MacBook-Pro temp % cd cloudnative-pg
amcohen@us.ibm.com@Aaron-IBM-MacBook-Pro cloudnative-pg % export BUILD_IMAGE=true
amcohen@us.ibm.com@Aaron-IBM-MacBook-Pro cloudnative-pg % export CONTROLLER_IMG=amcohen/test_pg:devel
amcohen@us.ibm.com@Aaron-IBM-MacBook-Pro cloudnative-pg % make e2e-test-kind
hack/e2e/run-e2e-kind.sh
Creating kind cluster pg-operator-e2e-v1-26-0 with version v1.26.0
Creating cluster "pg-operator-e2e-v1-26-0" ...
 ✓ Ensuring node image (kindest/node:v1.26.0) 🖼
 ✓ Preparing nodes 📦 📦 📦 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
 ✓ Joining worker nodes 🚜
Set kubectl context to "kind-pg-operator-e2e-v1-26-0"
You can now use your cluster with:

kubectl cluster-info --context kind-pg-operator-e2e-v1-26-0

Not sure what to do next? 😅  Check out https://kind.sigs.k8s.io/docs/user/quick-start/
fs.inotify.max_user_watches = 524288
fs.inotify.max_user_instances = 512
fs.inotify.max_user_watches = 524288
fs.inotify.max_user_instances = 512
fs.inotify.max_user_watches = 524288
fs.inotify.max_user_instances = 512
fs.inotify.max_user_watches = 524288
fs.inotify.max_user_instances = 512
v1.14.3-debian-forward-1.0: Pulling from fluent/fluentd-kubernetes-daemonset
Digest: sha256:c783e3bff97bcd44ab34f318be7fc083813b96eedc1e63fd769bfa831bf328a1
Status: Image is up to date for fluent/fluentd-kubernetes-daemonset:v1.14.3-debian-forward-1.0
docker.io/fluent/fluentd-kubernetes-daemonset:v1.14.3-debian-forward-1.0
Image: "registry.dev:5000/fluentd-kubernetes-daemonset:local" with ID "sha256:5abf0a5cd34310603a1abeb547f2bd18950914b774359b4caf8668618d177afa" not yet present on node "pg-operator-e2e-v1-26-0-control-plane", loading...
Image: "registry.dev:5000/fluentd-kubernetes-daemonset:local" with ID "sha256:5abf0a5cd34310603a1abeb547f2bd18950914b774359b4caf8668618d177afa" not yet present on node "pg-operator-e2e-v1-26-0-worker2", loading...
Image: "registry.dev:5000/fluentd-kubernetes-daemonset:local" with ID "sha256:5abf0a5cd34310603a1abeb547f2bd18950914b774359b4caf8668618d177afa" not yet present on node "pg-operator-e2e-v1-26-0-worker", loading...
Image: "registry.dev:5000/fluentd-kubernetes-daemonset:local" with ID "sha256:5abf0a5cd34310603a1abeb547f2bd18950914b774359b4caf8668618d177afa" not yet present on node "pg-operator-e2e-v1-26-0-worker3", loading...
serviceaccount/fluentd created
clusterrole.rbac.authorization.k8s.io/fluentd created
clusterrolebinding.rbac.authorization.k8s.io/fluentd created
configmap/fluentd-conf created
daemonset.apps/fluentd created
FluentD is Ready
Done creating kind cluster pg-operator-e2e-v1-26-0 with version v1.26.0
ERROR: unknown command -r

Usage<EMAIL_ADDRESS>[-e {kind|k3d}] [-k <version>] [-r] <command>

Commands:
    prepare <dest_dir>    Downloads the prerequisite into <dest_dir>
    create                Create the test cluster
    load                  Build and load the operator image in the cluster
    load-helper-images    Load the catalog of HELPER_IMGS into the local registry
    deploy                Deploy the operator manifests in the cluster
    print-image           Print the CONTROLLER_IMG name to be used inside
                          the cluster
    export-logs           Export the logs from the cluster inside the directory
<EMAIL_ADDRESS>    destroy               Destroy the cluster
    pyroscope             Deploy Pyroscope inside operator namespace

Options:
    -e|--engine
        <CLUSTER_ENGINE>  Use the provided ENGINE to run the cluster.
                          Available options are 'kind' and 'k3d'. Default 'kind'.
                          Env: CLUSTER_ENGINE

    -k|--k8s-version
        <K8S_VERSION>     Use the specified kubernetes full version number
                          (e.g., v1.26.0). Env: K8S_VERSION

    -n|--nodes
        <NODES>           Create a cluster with the required number of nodes.
                          Used only during "create" command. Default: 3
                          Env: NODES

    -r|--registry         Enable local registry. Env: ENABLE_REGISTRY

    -p|--pyroscope        Enable Pyroscope in the operator namespace

To use long options you need to have GNU enhanced getopt available, otherwise
you can only use the short version of the options.
make: *** [e2e-test-kind] Error 1

Also tried k3d and got this error so I wonder if the problem is with a configuration in my terminal.
make e2e-test-k3d
hack/e2e/run-e2e-k3d.sh
Creating k3d cluster pg-operator-e2e-v1-26-0 with version v1.26.0
parse error: Invalid numeric literal at line 1, column 7
make: *** [e2e-test-k3d] Error 1

ERROR: unknown command -r

Hi. From what you pasted, it looks like you have a jq executable on your system, but it is not the one our script expects (it doesn't accept a -r option).
I do not see jq in the list of the software needed to run the e2e tests, so this is indeed a bug in our documentation.
Please verify the jq version by executing jq --version.
On my system, I have 1.6 which works perfectly.

I have probably misinterpreted the error message I pasted in the previous comment.
It comes from https://github.com/cloudnative-pg/cloudnative-pg/blob/main/hack/setup-cluster.sh#L632
I guess that your version of getopt doesn't support having the flags at the end, so the call at https://github.com/cloudnative-pg/cloudnative-pg/blob/main/hack/e2e/run-e2e-kind.sh#L56 doesn't work.
Can you please try to run hack/setup-cluster.sh -r create and see if it gives you an error?

PR https://github.com/cloudnative-pg/cloudnative-pg/pull/2053 seems to be working.


Reactive-RPC hardening

[x] Make sure on protocol error WebSocket is disconnected or can gracefully handle garbage data.

:tada: This issue has been resolved in version 7.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Websocket

ServerBase will create a websocket channel to push telemetry events to the client.
Removed ipc channel's event hub implementation.
General Guidelines

[x] If introducing new functionality or modified behavior, are they backed by unit tests?
[x] Have all unit tests passed locally? (by running npm run test command)
[x] Have you updated the README.md with new screenshots if significant changes have been made?
[x] Have you update the package version if the current version in package.json is not higher than the version released?

#562 #573 #561


Experiment from entangled90

Could you please rebase this one, so as to resolve the merge conflict and then remove the "needs rebase" label again? Also please make sure that running ./test.sh entangled90 passes. Thex

Done, sorry about that!

Hey @entangled90, tests pass now, but the output is different from that one of the baseline implementation.

There was an error with rounding. Thanks for catching it!

I've found an issue in this entry, if we add the following line in the print results:
    System.out.println("Processed: " + processors.entrySet().stream().mapToLong(e -> e.getValue().count).sum());

We can see that it is only processing 312843874 records instead of a billion, it seems to miss about 2/3rds of the entries for some reason. Can't really pinpoint where the issue is though (yet).

Ah, it has to do with the size of the queue and the MAX_MAPPED_MEMORY. If I make both 1*1024*1024 in size, we get<PHONE_NUMBER> entries.

Thanks for spotting this! Issue is I was using  ArrayBlockingQueue.offer (which does not block if queue is full). On my pc there is no difference in time so probably depends on the machine/core available

00:18.380, nice. Thx for particpating!


[FEATURE 💡] Carousel for Customers

Description
A carousal feature would be a nice addition to this project

Add a carousel for the customer reviews
Add more dummy reviews
The carousel must rotate automatically initially
Must have a pagination item at the bottom to pause and navigate next or previous reviews
Any animation would be nice

Screenshots

Additional information
Make sure to read the CONTRIBUTING and SETUP docs before proceeding
Happy contributing. 💐
Star my other Repositories here

Hey @warmachine028 can you assign this to me please?

Sure @VadimDez assigning you

@VadimDez what's the status?

@warmachine028 I can work on this if it's alright with you.

@warmachine028 Can I work on these?

@akash0708 , @debajoti both of you can work on this issue. Since the previous assignee has left this issue as stale. I would accept any of the PR's which are made first.


[Cosmos] Updates to kwargs and docstrings

Partial fix for #7729 and #7788
Also address #8278

Can one of the admins verify this patch?

/azp run python - cosmos - tests

/azp run python - cosmos - tests

/azp run python - cosmos - tests


Access to low-level transcoder APIs

For projects implementing support for Basis Universal data without relying on a .basis wrapper file, e.g. via KTX2, it will be necessary to expose additional features of the low-level transcoder. Currently the WASM transcoder only accepts a .basis file as input, for example.
In https://github.com/BinomialLLC/basis_universal/pull/51 some comparable changes are proposed to enable libktx to access the low-level decoder. The PR is specific to libktx, and doesn't expose anything new on the WASM transcoder yet, but is similar in its goals.
I would like to eventually reach a state where multiple libraries can use the low-level transcoder: libktx for complete KTX2 support, a lightweight JS implementation for the web, and perhaps a lightweight native implementation like tiny_ktx.
@richgel999 would you be open to exposing additional APIs? Or am I asking for the wrong thing, and I should be considering a different high-level API rather than direct access to the low-level transcoder? Open to feedback on how to approach this, thanks.

Two notes about PR #51. First there is nothing specific to KTX2 about it. It simply creates a variant of the existing low-level API that does not require a .basis file header pointer. The header is used only to get 2 bits of information about the data. The additional signature passes those bits as explicit parameters in place of the .basis header pointer.
Second I need to update the PR for all the new transcode options. Part of adding those options involved adding parameters to the low-level transcode API. I need to add those same parameters to the new signature.

@richgel999 would you be open to exposing additional APIs? Or am I asking for the wrong thing, and I should be considering a different high-level API rather than direct access to the low-level transcoder? Open to feedback on how to approach this, thanks.
Sorry for the delay. I explicitly wanted to get UASTC support in first, before replying. That's going in today (in hours).
I'm totally open to API changes. Whatever it takes to make KTX2 happy!

@richgel999 at least pre-UASTC, KTX2 is happy with the changes proposed in PR #51. However it probably is too low-level for easy or general use. Within libktx I created a ktxBasisImageTranscoder which is at roughly the same level as transcode_image_level but is independent of container format. This is what I have exposed in msc_basis_transcoder.js. You may wish to have something similar in your library.

Low-level transcoding API's are in (for both native and JavaScript).


Course-of-Action objects should have an appropriate external reference.

I found course-of-action objects in STIX 2.0 that I retrieve by TAXII protocol contain non-compliant external reference values.
For example, I found an external reference in a course-of-action object.
external_references": [
    {
        "source_name": "mitre-attack",
        "url": "https://attack.mitre.org/mitigations/T1156",
        "external_id": "T1156"
    }
]

I think the external_id should start with "M". Moreover, the URL is invalid (404 Not Found).
I recommend improving the two points below.

external_id should be a mitigation ID that corresponds with a course-of-action
URL should indicate a mitigation page.

This STIX file is in the below URL.
https://github.com/mitre-attack/attack-website/blob/master/data/stix/enterprise-attack.json

Hi @stmtstk,
Thank you for reaching out! The STIX data on this repo is a mirror of that on our MITRE/CTI repository, which is the more relevant place to open such issues for the future (and also contains additional documentation about the data model and methods for working with the STIX).
With regards to the course-of-action, that object (and other CoAs with such odd IDs) are leftovers from before the ATT&CK version 5 release which changed the way mitigations are represented in ATT&CK. Prior to ATT&CK v5 such mitigations had a 1:1 relationship with techniques and as such shared their ATT&CK IDs and lived on the same page as them, which is why the data looks like that for those old course-of-action objects.
Those course-of-actions are deprecated (marked with the x_mitre_deprecated field) to mark them as invalid for current versions of ATT&CK. We keep them in the data so as not to break any code which references them. Please see the USAGE document on MITRE/CTI for more info on working with deprecated (and revoked) objects.

Hi @isaisabel,
Thank you for your kind reply.
I missed a x_mitre_deprecated property.
I also understand before the ATT&CK version 5 dataset.
Your explanation is very helpful for me.
Thank you again!


public folder does not work as advertised

When having just the ./public folder the default behaviour is serving an empty page. So I changed the text accordingly.
I rather had the API pages still intact and it would be great that get access to it even after creating a ./public/index.

Hmmm ... my conclusion was wrong. Curl reports 404 Not found but the body contains {} maybe the right respons?
curl -vvvv http://localhost:3000/
*   Trying ::1...
* connect to ::1 port 3000 failed: Connection refused
*   Trying <IP_ADDRESS>...
* Connected to localhost (<IP_ADDRESS>) port 3000 (#0)
> GET / HTTP/1.1
> Host: localhost:3000
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 404 Not Found
< X-Powered-By: Express
< Vary: Origin, Accept-Encoding
< Access-Control-Allow-Credentials: true
< Cache-Control: no-cache
< Pragma: no-cache
< Expires: -1
< X-Content-Type-Options: nosniff
< Content-Type: application/json; charset=utf-8
< Content-Length: 2
< ETag: W/"2-mZFLkyvTelC5g8XnyQrpOw"
< Date: Sun, 07 Feb 2016 11:58:23 GMT
< Connection: keep-alive
<
* Connection #0 to host localhost left intact
{}%

This now seems to me as when creating a public folder you  must add an index.html with it.


no viewport meta tag needed

next/head adds this meta tag automatically
See https://github.com/vercel/next.js/pull/44454
Thanks to @leerob for pointing this out. 🙇🏼

Rebased to get the new CI workflow running on this PR: https://github.com/replicate/getting-started-nextjs/pull/8 🍏


Fix bug: Error - not able to save profile (User 303 on Fullstory)

TODO: watch fullstory to see what was going on

I found one potential cause of profiles not saving: invalid data. I added a guard against one such error on the client and logging to the server. Hopefully if this comes up again the improved logging will make it easy to see what failed. Generally client-side validation is lacking, although in fact most fields on the faculty profile page are optional.


Add others to Hasura project

Add others (Chanaka, Dulaj and Mahel) to the new Hasura project (#2) as ADMINS. Give us all the permissions possible.
My email is<EMAIL_ADDRESS>@dulaj93 What is your email? (You have to sign up to hasura.io to create your account. #8 )

My email<EMAIL_ADDRESS>


Fix missing cast to hash in repo discovery script

Fix this exception in discovery-repos.rb script
cam/steps/discover-repos.rb:125:in `<main>': undefined method `values' for an instance of Array (NoMethodError)
lines = [found.first[1].keys.join(',')] + found.values.map { |m| m.values.join(',') }
Did you mean?  values_at

https://github.com/yegor256/cam/issues/211

@howcanunot thanks for the fix! Would be great to supplement the fix with a unit test. Otherwise, how do we know anything was fixed?

@howcanunot thanks for the fix! Would be great to supplement the fix with a unit test. Otherwise, how do we know anything was fixed?

What is the best way to add such a test? Would it be enough to add my case to tests/steps/test-discover-repos.sh?

some changes in commit:

in dry_run script mode we should get not only single repo but page-size for every iteration, just like in normal mode
extra test to verify that we were able to grep a sufficiently large number of repos

@yegor256 Good afternoon! Can you please check this pr?

@rultor merge

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 17min)


Setup dcp2.data.humancellatlas.org instance for data browser

Gitlab: https://gitlab.azul.data.humancellatlas.org/clever-canary
Azul: https://service.azul.data.humancellatlas.org/
I'll start an email thread about AWS access.
Please review https://gitlab.azul.data.humancellatlas.org/ucsc/azul/-/blob/prod/terraform/gitlab/gitlab.tf.json.template.py#L523 and file Github ticket against Azul about what the DB-specific IAM permissions should look like on the prod instance of Gitlab.

Please keep me advised about any changes you make to the data.humancellatlas.org hosted zone on the old AWS prod account.

Scratch that. Sanger just switched the NS record so to make changes to that domain please file PR against https://github.com/DataBiosphere/dcp2/blob/develop/prod/route53.tf.json.

Got it thx.
D

@hannes-ucsc I have created pr https://github.com/DataBiosphere/azul/pull/2578 for the updates to the resource.


Jamaica missing black

Just as the title says – there's a bit of black missing on Jamaica's CSS

Which browser, version and OS are you using?

Firefox 47.0 on Mac OSX 10.10.5
On Wed, Aug 3, 2016 at 10:37 AM, Chris Monkman<EMAIL_ADDRESS>wrote:

Which browser, version and OS are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pixelastic/css-flags/issues/7#issuecomment-237165429,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALMGvZ_H5lX7SkMkQDuM1wt2xLzvURwOks5qcEVPgaJpZM4JbYFg
.

--
Kumbi Madzinga
www.kumbimadzinga.com http://kumbimadzinga.com

Ahh you're right. I didn't even spot that.

Could you post a screenshot? I haven't tested it on Firefox at all.


Uncaught exception when pushing

PHP error is as follows:
Fatal error: Uncaught exception 'Pmp\Sdk\Exception\LinkException' with message 'Unable to retrieve urn:collectiondoc:form:issuetoken from the home document' in /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk/AuthClient.php:95 Stack trace: #0 /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk/AuthClient.php(75): Pmp\Sdk\AuthClient->_authLink('urn:collectiond...', Array) #1 /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk/AuthClient.php(46): Pmp\Sdk\AuthClient->_request('urn:collectiond...', Array) #2 /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk/AuthClient.php(34): Pmp\Sdk\AuthClient->getToken() #3 /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk.php(62): Pmp\Sdk\AuthClient->__construct(NULL, NULL, NULL, Object(Pmp\Sdk\CollectionDocJson)) #4 /www/indpub/wp-content/plugins/pub in /www/indpub/wp-content/plugins/public-media-platform/vendor/publicmediaplatform/pmpsdk/src/Pmp/Sdk/AuthClient.php on line 95

This is with pmp-wordpress release 0.2.8, PHP 5.4.16, and Wordpress 3.9.3.

Does search-for/pulling content work on that server?

Yep, search/pull works fine.

Any new ideas on this? We are hoping to get push working as soon as we can.

@eobanb If you downloaded the plugin via Github or are tracking the plugin using Git and haven't updated Composer requirements lately, I would try that first. From the directory you installed PMP plugin into, try running the following command:
composer install

If you downloaded the plugin via WordPress.org, you should already have all requirements satisfied (these files live in the vendor directory).
Based on the error you've described, it seems like the plugin is getting hung up on retrieving some basic information from the API when it first requests a handle.
Can you access the "Home" document directly (for live and sandbox versions of the API) to verify they are working properly? If yes, you'll notice that the same urn:collectiondoc:form:issuetoken from the exception you shared is included in the Home document.
Being unable to access the Home document for some reason (maybe some temporary network connectivity issue?) could conceivably cause this issue.
If it turns out to be a permissions issue with your API credentials, you'll need to talk to someone at PMP or NPR. @cavis can you recommend anyone?

@rnagle we're using a copy downloaded from the wordpress.org plugin directory.
Other than accessing the home document from my browser (it appears to download just fine, and I see urn:collectiondoc:form:issuetoken referenced near the end) is there something else I should be doing?
This has happened weeks/months apart and very consistently so I don't think any temporary network issue is the problem.

@eobanb Thanks for clarifying those points. It sounds like you have everything in order. A few more thoughts for you...
Have you tried to replicate the issue on multiple servers/hosts? For instance, if it works fine on a local development (or staging, test, etc.) site, that would point to a problem with the host that the live site is deployed to.
If you are able to replicate the problem on another host, then I would definitely want to investigate the permissions for your API credentials to see if the problem originates there.

Looks like i'm late to the party!
I'm pretty sure this is tied to a specific server setup, rather than an API/credentials problem.  That Pmp\Sdk\Exception\LinkException will really only occur if you somehow authenticate with the API before fetching the home-document.  (I thought the SDK prevented this scenario, so I'm very curious where this originates in the pmp-wordpress codebase).
Maybe somewhere in the codebase, it tries to create-and-save a PMP doc without fetching the home doc?  But why it would only impact your specific setup, i have no clue.
@rnagle have you ever tried running the plugin against a 3.9.x Wordpress?  I'm pretty sure I have not - and the unit tests have always been against a 4.x.  Could there be any differences there?

@cavis Thanks for the information.
I've been using 4.x+ for development and testing.
I don't think that WordPress' settings API (used to store/retrieve the PMP API credentials) has changed so much that it would cause invalid parameters to be passed to the \Pmp\Sdk constructor.
There are two places in the code base where the SDK is instantiated if you're interested in checking that out: here and here.
As far as I know, the plugin always uses the SDK to access the API.
The only other place it communicates with the PMP is during the notification subscription process.
We could try adding a 3.9.x configuration to the travis.yml file to see if the unit tests reveal anything. What do you think?

OK, I have a 4.0.x install up and running (same PHP and plugin versions), same exact error.

New update: I've disabled every single plugin on our testing site and determined there's a conflict with this plugin:
https://wordpress.org/plugins/wordpress-mu-sitewide-tags/
Given that the above plugin hasn't been updated in a couple years, I'm going to go ahead and assume the best thing to do here would be figuring out how to discontinue our site's use of it.

That...is very good to know! Very glad you got it sorted out.


Fix command being cut off short

Previously the command could be cut off short with no easy way to see the rest of it.
Fixed by placing the command in a code block which allows horizontal scrolling.
Before on my machine:

Thanks for the fix! This loses the emboldening of -cheerp-make-module=es6 - I'll add that with expressive-code syntax before merging


Make gltf loader a lot more generic

Relates to #85

Adds a validation example for lighting and shadows
Adds an example that renders the Sponza scene with custom tweaked lighting
Revamps the gltf loading utility, I've broken it up into sub utilities that are now all available generally:

gfx.load_scene(path) which can load any file trimesh can handle (there's a lot) and return a list of Meshes (with potentially submeshes, etc)
gfx.trimesh_material(material) which will convert a Trimesh Material to a pygfx Material
gfx.trimesh_geometry(mesh) which will convert a Trimesh Geometry to a pygfx Geometry (we already had this)
gfx.pillow_image(image) which will convert a Pillow image to a pygfx Texture

When comparing test screenshot output the diffs are now computed lazily, which makes executing this tests a bit faster

Fixes:

I've moved the uv coordinate conversion from load gltf to gfx.trimesh_geometry so that it is applied generally
I noticed that the AxesHelper with its default configuration is unusable. I've updated it a little bit to more sensibly size the arrow heads. Really the problem here is that the line thickness is in screen space and every other parameter is in world space. We can address that in the future.

Wow!

Note: I tried enabling cast and receive shadow but the WGSL compilation process just kept blowing up.

Maybe #322 will help?

Note: I tried enabling cast and receive shadow but the WGSL compilation process just kept blowing up.

I fixed it in #351, 😅
BTW, when we render a large scene with multiple objects (especially when the rendering logic is complex), the performance may be poor. There are many objects outside the field of view, which do not need to be rendered. Before rendering, We should exclude them according to the camera's view frustum.
This part of the logic should be completed in the CPU, I think it should be in the render() method of _render.py.

Before I merge main I will add a test/example to prevent regressions.

I added an example that combines MeshPhongMaterial and MeshStandardMaterial with PointLight, AmbientLight, SpotLight and DirectionalLight to check that all combinations are working properly. I confirmed that it failed before merging main.

I had a ton of fun positioning and configuring the lighting. I'm really pleased with this result, doesn't it look amazing?

It's amazing!


Is this ready now?

This project is temporally stopped after I read the sentence of the paper that authors will release the code with TensorFlow.


Handling Ordered Version Files

Taking advantage of versions files that will contain the correct sorted set of versions.
I need to update all existing blob storage locations with the appropriate versions files before we check this in.

Can one of the admins verify this patch?

Existing docs are transitioned over to new methodology as well.


Unable to backup RWO PVCs

Hi
When trying to backup RWO PVCs K8up doesn't check on which node the pod with said PVC mounted is running.
Since RWO volumes can only be mounted multiple times on the same node, K8up should check on which node the pod is running and create the backupjob on the same node.
As of now, the backupjob will never start since it's not running on the same node.
Is it possible to implement that check?
As a workaround we'll try to add an application aware backup command to the pod; basically "tar"-ing all of the files to stdout. But it would be much nicer if we could just use the PVC annotation.
Many thanks,
gi8lino

HI @gi8lino
Thank you for opening an issue.
It's correct that K8up is currently not capable of backing up RWO PVCs as mentioned here: https://k8up.io/docs/0.1.9/index.html#_current_limitations
K8up will only schedule one wrestic worker per namespace. So if the namespace contains multiple RWO PVCs it's sometimes not even possible to schedule it, because they could be mounted on different hosts.
We're aware that this is a problem and using the backup commands is currently our recommended workaround for that.
Best
Simon

for the sake of completeness here is the backup command:
k8up.syn.tools/backupcommand: tar czvf - /app

@gi8lino Is it okay for you that we close here?


JPA 2.1 Entity Graphs

Please, add a capability to QueryDSL that allows to construct JPA 2.1 Entity Graphs in a type-safe manner.
Thanx,
Antón

+1
I also need this for performance.
thanks

+1
Would be great indeed!


Add instructions for primer design from RL Regions in docs

In the documentation, add instructions for how to use RLRegions in UCSC Genome Browser to enable primer design

Fixed with https://github.com/Bishop-Laboratory/RLBase/commit/ffbfeffc930fce38819ae2013b24a3dff489d5dd


upload files is failing

I have done a test with curl and redmine 3.3.0: File is loaded (checked on $REDMINE_DIR/files)
response is 201:
{
"upload": {
"token": "32.69abd4abf577d7cfd6d370f146611fea"
}
}
but when it is loaded through redmine-java-api it returns a 201 but no file is uploaded on $REDMINE_DIR/files and from response: token is:
"33."
so seems last part of token is missing too
I can make any kind of test if needed. I went through transport.upload() but get lost in redmine-java-api internals

relates with #278
https://redmine.lighttpd.net/issues/1104 redmine do not accept Transfer-encoding: chunked, but httpcomponents decide to send attachment that way.


Search icon is not visible when using the dark header

In the theme settings > site wide section.
If you change the Header style to "Dark" the search magnifiying glass icon is no longer visible in the header.

Hey @gregforcey, thanks for the report! Try re-installing the theme, and the issue should be fixed.


[BUG] Identity may return an expired token in an isolated function instance

Library name and version
Azure.Identity 1.7.0
Describe the bug
I have the following code:
private static readonly DefaultAzureCredential DefaultAzureCredential = new();
public static string GetToken()         {             return DefaultAzureCredential.GetToken(                 new TokenRequestContext(scopes: new string[] { "https://database.windows.net/" })).Token;         }     }
This is being called by a .Net6.0 isolated process, through an http trigger in a function app.
I routinely get Login failed for user ''. Token is expired.
Expected behavior
When I call GetToken() for a new token, I will get a refreshed token (with a new expiration date).
Actual behavior
I get the same token if it hasn't expired, which means it can expire between the call to get the token and the call using the token.
Reproduction Steps
Run the following code:
private static readonly DefaultAzureCredential DefaultAzureCredential = new();
public static string GetToken()         {             return DefaultAzureCredential.GetToken(                 new TokenRequestContext(scopes: new string[] { "https://database.windows.net/" })).Token;         }     }
Wait for the token to nearly expire and try to use it.
Environment
Visual studio 2022 17.3.6
Azure Function -- .Net6 Isolated

//cc: @schaabs

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Expected behavior
When I call GetToken() for a new token, I will get a refreshed token (with a new expiration date).

There is no guarantee that a new token is received each time one is requested from the authority. The caller actually has no control over when a fresh token is received.
How close is the current token to expiration before you are calling GetToken and still receiving the same one?

If I use the older Microsoft.Azure.Services.AppAuthentication library, this call does not cause any issues:
' public static readonly AzureServiceTokenProvider tokenProvider = new();'
tokenProvider.GetAccessTokenAsync("https://database.windows.net/").ConfigureAwait(false);
I only moved to the new library, since the older one recommends it.

How close is the current token to expiration before you are calling GetToken and still receiving the same one?
Does it matter which credential you end up using as part of the DefaultAzureCredential chain?

This is to feed an api so there are many calls/sec. I can't get a new token until the one requested expires, so it really doesn't matter how close it is.
 I just want to find the best mechanism to obtain and keep tokens fresh.

To answer your question about time, I had it set to 30 min before expiration to try for a new token.

GetToken should return a new token when one returned most recently is nearing expiration. For example, in the Azure SDK clients, we attempt to refresh when the cached token is within 5 minutes of expiration.  Can you try a similar threshold?
Note: Azure.Identity has no control over when AAD will return a cached token or a new token with a new expiration.

I will try it at 5 min. BTW, is there a reason the older library has no issue? The reason I know the older library doesn't have this issue, is because that is currently running in production.

BTW, is there a reason the older library has no issue? The reason I know the older library doesn't have this issue, is because that is currently running in production.

I don't know why it would work differently - both result in a call to AAD and only it controls when a new token is returned.

In the original call there was no concept of scope. Here I am asking specifically for database access (which is all I need).
This would be the new proposed solution you are recommending?

Again, I am somewhat confused with why the old Microsoft.Azure.Services.AppAuthentication always provides a proper token and does not give a token expired error. This seems to be the simpler library. Is there any reason I should use it?
This is the only call I need vs the method above:
await DBConnection.tokenProvider.GetAccessTokenAsync("https://database.windows.net/").ConfigureAwait(false);

I'm not recommending this specific solution, but it appears to refresh within the 5 minute window which should avoid expiration issues.
As a side note, I just wanted to mention the Directory Default authentication feature for SQLClient, in case that helps you in this scenario. It allows you to provide a connection string to the client which uses DefaultAzureCredential under the covers.

That is the most elegant solution, thank you! I'll use the default authentication feature.


Tiny typo ruining lives.

I'm sorry

Saddest pull request ever. Don't be sorry! It was probably me.

Definitely me :(


Emphasis

In the example for the element Emphasis no word is in italics.

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

ID: d28c9871-f3a6-2dc1-1494-93f981f50238
Version Independent ID: 8e95f7bc-399a-7648-36bd-8dd7dc90f347
Content: Formatting common text elements - Microsoft Style Guide
Content Source: styleguide/text-formatting/formatting-common-text-elements.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Oh, yes, that's an error. The word "never" is supposed to be italic. We'll get that in the queue to be fixed. Thanks for letting us know!


startAngle seems

How do I make the animation start at the top of the circle??

Set your animationStartValue to 1 and your
startAngle: -Math.PI/2
This should set it to the top. I haven't used this for 3 years so I'm going by memory.

You can calculate the angle with this. Just enter the degree in this formula.
Math.PI * ((degree / 360) * 2 - 0.5)
0 ° will start from the top.


Masonry Grid

A masonry grid system would be very useful, especially for articles and media galleries, both horizontal and vertical, but mainly horizontal is what I need.
Here is an implantation in another CSS framework I like a lot.
https://getuikit.com/assets/uikit/tests/grid-masonry.html

Looks like grid masonry is not cross-browser supported yet
https://caniuse.com/?search=masonry
https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Grid_Layout/Masonry_Layout

We can simulate a masonry with column-count https://codepen.io/leo-bnu/pen/YzRYwBm
But the order of things is not in "Z pattern". And this is not responsive, needs to add some media queries to change the collumn count according to user screen.

It appears as though Material Design doesn't mention masonry directly (it is shown in the Cards guidance section) since it is in the web standards (just not implemented anywhere). So as such, if beercss's primary goal is implementing material, then it would imply that one should get masonry specific from another as that's not a specific design pattern for Material.

Closed, because this is inside another issue.


[iOS] Align "Back to profile" button title

Open Profile -> Edit -> tap Delete account -> observe "Back to profile" button: button name is moved down.
This issue applies to iPhone and iPad.

Need to align the button title
As a possible solution - we can simply remove this button, since at the top left there is a "Back" button that performs the same action

Hello @sdaitzman
Are you ok if we get rid this button or it's needed for some design purposes?
Thank you

Hi @rnr, thank you for tagging me to check about this. I think it would be preferable to keep the Back to Profile button if possible.
From a design standpoint, screens that allow a destructive action should include a quick and clear default action to opt out of that destructive action. It's not a frequent situation, but e.g. if a user has entered their password but realized that they do not want to delete their account, it should be clear that the default action is to go back without deleting their account.
On this screen, I also think the Delete Account button should eventually use the red error/danger color with an outlined (secondary) button style, and the Back to Profile button should use the primary button style. This would emphasize which action is destructive. Those changes may fall under some future design enhancements within account settings/profile.


Something always fails because of codemeta

Describe the bug
In my case somesy always fails in codemeta, bevause packagename = pkg.name
happens in all cases for me, through hooks, cli, etc.
What I found so far is:
https://stackoverflow.com/questions/67212594/attributeerror-pathdistribution-object-has-no-attribute-name
that it might have to do with the version of  importlib-metadata<3.4.0.
To Reproduce
Steps to reproduce the behavior:

Have a codemeta.json file
install somesy (pypi)
Run it
See error

Expected behavior
A clear and concise description of what you expected to happen.
Traces
Stack trace of the error
ERROR    Error: 'PathDistribution' object has no attribute 'name'                                                                                util.py:18
DEBUG    Traceback (most recent call last):                                                                                util.py:19
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/cli/util.py", line 15, in wrap_exceptions
             return wrapped(*args, **kwargs)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/cli/sync.py", line 146, in sync
             run_sync(somesy_input)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/cli/sync.py", line 169, in run_sync
             sync_command(somesy_input)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/commands/sync.py", line 39, in sync
             _sync_codemeta(conf)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/commands/sync.py", line 98, in _sync_codemeta
             if update_codemeta(conf):
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/codemeta/__init__.py", line 77, in
         update_codemeta
             cm_harvest = gen_codemeta(cm_sources)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/somesy/codemeta/exec.py", line 25, in gen_codemeta
             g, res, args, _ = build(
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/codemeta/codemeta.py", line 656, in build
             codemeta.parsers.python.parse_python(newgraph, res, source, crosswalk, args)
           File "/home/j.broeder/.cache/pypoetry/virtualenvs/data-harvesting-m9-r40y6-py3.9/lib/python3.9/site-packages/codemeta/parsers/python.py", line 226, in parse_python
             packagename = pkg.name
         AttributeError: 'PathDistribution' object has no attribute 'name'                                                                                ```

** Env **

CodeMetaPy==2.5.1
importlib-metadata==3.4.0
python 3.9
somesy==0.1.0

Do you get the same problem when you use codemetapy directly?
somesy is only wrapping codemetapy and this error looks like it comes from inside the codemetapy parsing pipeline.

The problem you have is unrelated to somesy, you get the same error when running codemetapy directly on your pyproject.toml.
No input files specified, but found python project (pyproject.toml) in current dir, using that...
Note: You did not specify a --baseuri so we will not provide identifiers (IRIs) for your SoftwareSourceCode resources (and others)
Initial URI automatically generated, may be overriden later: file:///pyproject-toml
Processing source #1 of 1
Obtaining python package metadata for: pyproject.toml
Loading metadata from pyproject.toml via pyproject-parser
Failed to process pyproject.toml via pyproject-parser: The 'project.license' table should contain one of 'text' or 'file'.
Fallback: Loading metadata from pyproject.toml via PEP517

[notice] A new release of pip is available: 23.2.1 -> 23.3.1
[notice] To update, run: pip install --upgrade pip
Traceback (most recent call last):
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/bin/codemetapy", line 8, in <module>
    sys.exit(main())
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/codemeta.py", line 335, in main
    g, res, args, contextgraph = build(**args.__dict__)
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/codemeta.py", line 656, in build
    codemeta.parsers.python.parse_python(newgraph, res, source, crosswalk, args)
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/parsers/python.py", line 226, in parse_python
    packagename = pkg.name
AttributeError: 'PathDistribution' object has no attribute 'name'

There you can see that your project.license is wrong for the setuptools format, it must be license = {text = "MIT"}.
Fixing that gives a different error message:
No input files specified, but found python project (pyproject.toml) in current dir, using that...
Note: You did not specify a --baseuri so we will not provide identifiers (IRIs) for your SoftwareSourceCode resources (and others)
Initial URI automatically generated, may be overriden later: file:///pyproject-toml
Processing source #1 of 1
Obtaining python package metadata for: pyproject.toml
Loading metadata from pyproject.toml via pyproject-parser
Traceback (most recent call last):
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/bin/codemetapy", line 8, in <module>
    sys.exit(main())
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/codemeta.py", line 335, in main
    g, res, args, contextgraph = build(**args.__dict__)
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/codemeta.py", line 656, in build
    codemeta.parsers.python.parse_python(newgraph, res, source, crosswalk, args)
  File "/local/home/a.pirogov/.cache/pypoetry/virtualenvs/testcmpy-m3kZsOkJ-py3.9/lib/python3.9/site-packages/codemeta/parsers/python.py", line 339, in parse_python
    if value.startswith("http"):
AttributeError: 'Readme' object has no attribute 'startswith'

I also want to note that it is strange that you have both "setuptools" and "poetry" style metadata. This definitely would cause issues with somesy and I can imagine also confuse other tools.
But in any case, this is unrelated to somesy, so please open an issue for codemetapy should you confirm that there is in fact a bug and your pyproject.toml file is valid.

Thanks.
I know. I would only want the project part, not the poetry part, but since poetry fails to use the project part, I need to duplicate it, that poetry is aware of it. Other tools usually read and should only read the usual project part in my opinion. One reason for me not to use poetry.
The license thing we have corrected badly when enabling somesy, since before it was dynamic.


Update Search.php

Questions
Answers

Description?
Filter products by customer group. Allow to show some products only to a specific group.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?
Create one custom customer group, attach a customer to this group, block a product only for this customer group, then only this customer will see this product.

This change is

Hi,
Since we had no news from you for more than 30 days, I will follow our guidelines and I'll close this pull request as it seems stale. Feel free to reopen or open another one if you think it's still relevant.
Thanks!


Remove grandparent bottom-panel

This removes the grandparent bottom-panel on close of a script. Without doing so, the empty bottom-panels hang around and since each has a height of 1px, they start to stack up and take up more and more space.

Yay! A PR! Thanks @efatsi!

Thanks again for this. This has been shipped.
I've also given you commit access so you can help maintain if available.

No problem. Actually came across the fix while working on my own package which owes it's good looks to atom-script. I'll port over any other fixes I come across in the future. Thank you for your work on this!


In essential metabolic tasks: NEFA blood pool in[x] should be changed for NEFA blood pool in[s]

Essential task 46 (Uptake and beta oxidation of all NEFAs) fails in version 1.9. NEFA blood pool in[x] should be changed for NEFA blood pool in[s]; this solves the issue.

@CadavidJoseL Thanks for finding this, it should be changed to the [s] compartment as you suggest.
However, this task should not fail, even with the [x] compartment being used. Did you remember to add the boundary metabolites before running checkTasks?:
model_boundary = addBoundaryMets(model);

If the boundary metabolites are not present, then the task check results will not be valid.

@CadavidJoseL Thanks, will look into this

You're right, I did forget to add those boundary mets; that also fixes the issue. Thanks!

If it's ok with you, @CadavidJoseL, I'd like to keep this issue open until we correct the compartment of the NEFA blood pool in in the task file. Otherwise we may forget to make the fix.

This issue was resolved in #331.


Attribute producing adapter based on Sets/Maps stored in CRDs

@geeknoid @mandarjog @rshriram
The Mixer expression language is quite powerful but implementing large set membership or large keyspace maps would be extremely unwieldly. I think this would be a relatively cheap yet powerful feature to deliver in 0.2.
I propose we add:

A standard CRD mechanism for defining large sets and maps (with types?)
Standard namespace ACL controls for writing these maps/sets
A simple built in attribute producing adapter that uses these sets or maps to produce values keyed by other attributes in the context
A simple 'Predicate' Check adapter which uses a the Mixer expression language to allow or deny check requests

This functionality would provide a generic mechanism for:

whitelisting and blacklisting
looking up parameters to downstream adapters. E.g distinct quota limits
Defining policy maps which affect which handlers are selected based on context

E.g logical flow
CRD: name:Admins, type:Set -> ("steve", "dana", "rachel")
Attribute Producing adapter: isAdmin = request.user in 'Admins'
Check : allow request if isAdmin
Maybe someone in the community could pick it up to help validate the Mixer adapter architecture stuff (that someone could even be me :)

@timothyhinrichs did you guys provide any in-language mechanisms for large sets/maps in OPA or was the intent to handle this out-of-band using extensions points like what I described above. Would be good to be lined up on this.
@tristonianjones largely same question based on Firebase experience ?

We have part of what we need for this today: the generic list checker takes a map from its configuration and checks that the result of an expression is in its set (or not in it, for blacklists).
If we push Mixer config into CRDs (AFIAK @jmuk is working on this now) then technically we can do this as soon as Mixer CRDs land, with no other changes. If we want do something like read a configmap in the adapter as the input, then some code changes are required.

I've wanted to create a generic map-producing Attributes adapter for a while as I think it's generally useful. Such an adapter doesn't exist today and could be used to help solve this case too.

I think list checker might be a little too constraining as it couples the
value lookup to the behavior in a limiting way. i.e. can't use a predicate
expression that combines the value of the lookup with other attributes in
the context.
On Thu, Aug 10, 2017 at 11:31 AM, Zack<EMAIL_ADDRESS>wrote:

I've wanted to create a generic map-producing Attributes adapter for a
while as I think it's generally useful. Such an adapter doesn't exist today
and could be used to help solve this case too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/mixer/issues/1049#issuecomment-321635885, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIoKPC6FcgVZGcwsyMoJN4GojDj2Xsakks5sW0yAgaJpZM4OzzUa
.

OPA does more than storing sets with datalog-style reasoning. Can Mixer do a join operation across two tables?

I think Mixer already materializes all data sources. This is really a database problem, and can be improved as we get to larger scale and need multi-level caching.

Mixer 0.2 is pretty much locked down, we still have a number of things to
deliver. So let's talk about this in the context of 0.3. This will give us
more time to design this feature properly.
On Thu, Aug 10, 2017 at 12:47 PM, Kuat<EMAIL_ADDRESS>wrote:

I think Mixer already materializes all data sources. This is really a
database problem, and can be improved as we get to larger scale and need
multi-level caching.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/mixer/issues/1049#issuecomment-321654613, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHXOrMgHELtWep9GKh4VzKdVJM1uxks5sW15qgaJpZM4OzzUa
.

Happy to treat as '3rd party' extension while we work this out

Apologies for the late reply. Maps and lists with a limited number of entries are typically handled within code, but for larger datasets (say > 500 entries), Firebase prefers an extension that permits policy writers to query the database for the portion of the dataset required to answer the question.
The reasoning for the extension based approach is two-fold:

Large datasets written inline in a policy make the policy much more difficult to manage
All data provided into the policy must be marshalled into a format the policy expects and marshalling data can easily dominate evaluation time.

@louiscryan I'm happy to work on this.
I'll start with a bit of prototyping, but follow up with a design doc to circ before I go whole hog.

@louisryan yes, you can load arbitrary attributes into OPA and then refer to them in policies/queries.
Currently the data is stored using native Go types (e.g., maps, slices, etc). This has been benchmarked on a container scheduling use case where the data set was in the 100s of MBs. We have plans to optimize the storage format in the future, e.g., with string de-duplication, better number encoding, etc.
OPA was designed from the beginning with this kind of “attribute production” use case in mind.
In OPA, you write rules that define attributes (JSON documents) that can be queried. When you query OPA, the engine evaluates the rules and produces attribute values.
For example, given the some data set…
apps = [{

  name: web
  ports: [443, 8080, 80]

}, {
  name: db
  ports: [443]

}, …]

insecure_ports = {80, 8080}

You could define an attribute production rule as follows…

# generate “is_insecure_port” attribute w/ value “true” if ...
is_insecure_port = true {

  request.dest_app = data.apps[i].name                # lookup destination app in data set
 based on incoming request
  data.apps[i].ports[_] = data.insecure_ports[_]      # check if app exposes insecure port
}

FWIW, we have demo'd logical flow you described above several times as part of Kubernetes Admission Control use cases (e.g., for allow/deny decisions, app placement, etc.)
Check out the Kubernetes docs to see how this is done for app placement in Federation: https://kubernetes.io/docs/tasks/federation/set-up-placement-policies-federation/#configuring-placement-policies-via-configmaps
/cc @TristonianJones @timothyhinrichs

thanks @spikecurtis for picking up the baton

We're having a discussion in the forums on the general idea of adapters reading CRDs. I am fairly strongly convinced that this would be a substantial architectural mistake.
https://groups.google.com/d/msg/istio-security/4nR-qQhtPKY/cVU9C41aBAAJ

Check returns a thumbs up when no actions match.

The config I have so far looks like this:

Could you give a brief explanation of what this is expressing? It's a bit confusing..

@geeknoid I've updated with some commentary.  Hopefully it is helpful.

Thanks, the comments are very helpful :-)
It seems likely to me that we'll want fine-grained ACLs on who can change which map/set and hence these should be kept as separate resources.
It's also worth considering whether this could be all made easier to consume if we expand the Mixer's expression evaluation code to provide set/map related functions such as contains. Basically, if we believe sets and maps such as shown here are generally useful, let's make them first class within our model. Mixer already understands STRING_MAP attributes, so extending our semantic and syntactic support to include sets and manipulation of sets and maps seems natural.

Maps should be separate resources, whose ACLS are managed like any other istio resources.
This is part of the original description.
I believe that providing a uniform way to store resources and connect them to Mixer EL is important and useful. This feature request fits perfectly in that. Resources are owned by the attribute producing adapter, but they are managed with all other Istio storage.

I was thinking about set/map functionality in CEXL as I was working through the prototype.
CEXL supports map evaluation with literals, but does it support it with attributes today?  E.g. if I have attributes:
user="spike"
location = {"spike": "Seattle", "alex": "San Francisco"}

can I write an expression
location[user] == "Seattle"


[SUPPORT] tryComposeIndexFilterExpr in dataskip util could support InSet expression of spark?

spark has a threshold spark.sql.optimizer.inSetConversionThreshold by default it is 10,
when filter is IN expression for example in ('1', '2', '3' ... '100'), when the values of in set > 10
IN will transform to InSet, but the filter expression transform doesnt check InSet, it only has In

@alexeykudinkin can you take a look pls?

@xushiyan you can assign this one to myself

@Zhangshunyu this is a great catch!
I've filed HUDI-4851 to address. @xushiyan we can close this one.


Polygon and freehand ROIs (partly) read in incorrectly

Hey,
I have been using your function to read cell outlines marked in ImageJ into my python pipeline without problems for a year now.
Now I moved to considerably larger images (10.000 x 10.000 pixels, before 1.000 x 1.000 pixels). I marked a couple of ROIs, saved as zip, and used the read_roi_zip() function to read. I got a weird pattern of errors:
-Some polygon ROIs are read in correctly.
-For some polygon ROIs, all x coordinates have a large negative offset (ca -60.000), while the y is correct
-With the freehand ROIs, I observed all four cases: correct ROIs, large negative offsets in x while y is ok, the reverse, and both x and y with large negative offsets.
I also found out that the following function from another github user works correctly on the same ROI file: https://github.com/tdsmith/ijroi
I attach my ROI file, a screenshot of the correct ROIs in Image J and the incorrectly read-in ROIs plotted in python.

First five coordinates of the read in ROIs (ID refers to legend in above figure).
Note that for all plots, I inverted the y coordinate to have the ROIs oriented as they are on the ImageJ screenshot below.
0 x [2104, 2302, 2573, 2729, 2833]
  y [6647, 6605, 6428, 6251, 6167]
1 x [4709, 4063, 3875, 3844, 3646]
  y [5073, 4751, 4615, 4417, 4313]
2 x [5605, 6032, 6345, 6522, 6741]
  y [5521, 5834, 6032, 6188, 6428]
3 x [5188, 5199, 5167, 5240, 5334]
  y [4928, 4844, 4761, 4709, 4709]
4 x [4844, 4678, 4553, 4553, 4542]
  y [9126, 8897, 8720, 8616, 8397]
5 x [-57316, -57128, -57087, -57149, -57285]
  y [4230, 4323, 4177, 4042, 4021]
6 x [-59066, -59108, -59108, -59108, -59108]
  y [3334, 3365, 3386, 3396, 3406]
7 x [1583, 1541, 1531, 1521, 1510]
  y [3907, 3907, 3907, 3907, 3907]
8 x [2062, 2104, 2115, 2115, 2135]
  y [-57649, -57670, -57681, -57691, -57733]
9 x [-57431, -57472, -57493, -57504, -57514]
  y [-57076, -57087, -57128, -57139, -57149]

Original ROIs in imageJ:

My code for plotting/reading in:
import original_read_roi
import matplotlib.pyplot as plt
import numpy as np
a = original_read_roi.read_roi_zip('hajdu_20180205_0_RBPMS_referenceROIs_day2_nozoom.zip')

for i,k in enumerate(list(a.keys())):
    plt.plot(np.array(a[k]['x']),-(np.array(a[k]['y'])),label=str(i))
    print(i,'x',a[k]['x'][:5],'\n  y',a[k]['y'][:5])

plt.title('all rois')
plt.legend(loc=6)
plt.show()

plt.figure(figsize=(10,10))

for i,k in enumerate(list(a.keys())):

    if a[k]['type']=='polygon':
        plt.plot(a[k]['x'],-(np.array(a[k]['y'])))
plt.title('polygon rois, uncorrected')
plt.show()
#
plt.figure(figsize=(10,10))

for i,k in enumerate(list(a.keys())):

    if a[k]['type']=='polygon':
        if a[k]['name']=='00005-04172-08334':
            plt.plot(np.array(a[k]['x'])+65307,-(np.array(a[k]['y'])),label='corrected outlier (added 65307 to y coordinate)')
        else:
            plt.plot(a[k]['x'],-(np.array(a[k]['y'])))
plt.title('polygon rois, corrected')
plt.legend()
plt.show()

for i,k in enumerate(list(a.keys())):

    if a[k]['type']=='freehand':

        plt.plot(np.array(a[k]['x']),-(np.array(a[k]['y'])),label=str(i))
        plt.title('only freehand rois')
plt.legend()
plt.show()

The ROI file used to obtain the above plots:
hajdu_20180205_0_RBPMS_referenceROIs_day2_nozoom.zip
I hope this helps to improve your code!
Cheers,
Jan.

I am having a similar issue with both versions 1.4.2 and 1.5.0.  Here is an image of bounding boxes plotted on the original image (the y-axis is inverted in comparison to ImageJ).  As you can see, when the roi value goes above ~4500 read_roi (both zipped and unzipped) returns wildly large numbers.... suspiciously close to 65000...

To me; it appears that some sort of bit-wise function in read_roi is getting its bits confused and converting from 12bit to 16bit depths (4096 vs 65536).  I never learned binary (zzzzzzzz) so I dont have a solution for this other than adding 65536 to that super negative number (like -59043).
Hope this helps yall find a fix.
-JT

Hello both and thank you for the report. I don't have time to look into that in the short term. If you have an idea what is going on please feel free to submit a PR. Thank you!

Here is how to get rid of the unexpected large negative offsets:
This is caused by a part of the code for which I did not understand the intention (in _read_roi.py around line 140):
    if top > 6000:         top -= 2**16     if left > 6000:         left -= 2**16
Removing those lines solved the issue for me; I did not notice any other issues that would be introduced by removing those lines.
Hope this helps!

Thank you @OSchoppe. I'll have a look at any PRs someone could submit with this fix.

I have similar issues as attached.
@OSchoppe  I did not find the code part you mentioned in 1.5.2. Have it been fixed? I still have this issue though.

Fixed!


Broken ruby / gem setup

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.0.1
VirtualBox 'vboxmanage --version'
ansible <IP_ADDRESS>
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/valic/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.14 (default, Sep 23 2017, 22:06:14) [GCC 7.2.0]

Your OS

macOS (Sierra)
Linux (Ubuntu 17.10)

Full console output
TASK [geerlingguy.ruby : Install configured gems.] *****************************
failed: [foreo] (item=compass) => {"changed": false, "cmd": "/usr/bin/gem install --user-install --no-document compass", "item": "compass", "msg": "WARNING:  You don't have /home/vagrant/.gem/ruby/2.3.0/bin in your PATH,\n\t  gem executables will not run.\nERROR:  Error installing compass:\n\tERROR: Failed to build gem native extension.\n\n    current directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\n/usr/bin/ruby2.3 -r ./siteconf20180214-10687-1xc1xl7.rb extconf.rb\nchecking for ffi.h... no\nchecking for ffi.h in /usr/local/include,/usr/include/ffi... no\nchecking for shlwapi.h... no\nchecking for rb_thread_blocking_region()... no\nchecking for rb_thread_call_with_gvl()... yes\nchecking for rb_thread_call_without_gvl()... yes\ncreating extconf.h\ncreating Makefile\n\nTo see why this extension failed to compile, please check the mkmf.log which can be found here:\n\n  /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/mkmf.log\n\ncurrent directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\nmake \"DESTDIR=\" clean\n\ncurrent directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\nmake \"DESTDIR=\"\nRunning autoreconf for libffi\nautoreconf: Entering directory `.'\nautoreconf: configure.ac: not using Gettext\nautoreconf: running: aclocal --force -I m4\nautoreconf: configure.ac: tracing\nautoreconf: running: libtoolize --copy --force\nautoreconf: running: /usr/bin/autoconf --force\nconfigure.ac:187: error: possibly undefined macro: LT_SYS_SYMBOL_USCORE\n      If this token and others are legitimate, please use m4_pattern_allow.\n      See the Autoconf documentation.\nautoreconf: /usr/bin/autoconf failed with exit status: 1\nlibffi.mk:6: recipe for target '\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a' failed\nmake: *** [\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a] Error 1\n\nmake failed, exit code 2\n\nGem files will remain installed in /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21 for inspection.\nResults logged to /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/gem_make.out", "rc": 1, "stderr": "WARNING:  You don't have /home/vagrant/.gem/ruby/2.3.0/bin in your PATH,\n\t  gem executables will not run.\nERROR:  Error installing compass:\n\tERROR: Failed to build gem native extension.\n\n    current directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\n/usr/bin/ruby2.3 -r ./siteconf20180214-10687-1xc1xl7.rb extconf.rb\nchecking for ffi.h... no\nchecking for ffi.h in /usr/local/include,/usr/include/ffi... no\nchecking for shlwapi.h... no\nchecking for rb_thread_blocking_region()... no\nchecking for rb_thread_call_with_gvl()... yes\nchecking for rb_thread_call_without_gvl()... yes\ncreating extconf.h\ncreating Makefile\n\nTo see why this extension failed to compile, please check the mkmf.log which can be found here:\n\n  /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/mkmf.log\n\ncurrent directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\nmake \"DESTDIR=\" clean\n\ncurrent directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c\nmake \"DESTDIR=\"\nRunning autoreconf for libffi\nautoreconf: Entering directory `.'\nautoreconf: configure.ac: not using Gettext\nautoreconf: running: aclocal --force -I m4\nautoreconf: configure.ac: tracing\nautoreconf: running: libtoolize --copy --force\nautoreconf: running: /usr/bin/autoconf --force\nconfigure.ac:187: error: possibly undefined macro: LT_SYS_SYMBOL_USCORE\n      If this token and others are legitimate, please use m4_pattern_allow.\n      See the Autoconf documentation.\nautoreconf: /usr/bin/autoconf failed with exit status: 1\nlibffi.mk:6: recipe for target '\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a' failed\nmake: *** [\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a] Error 1\n\nmake failed, exit code 2\n\nGem files will remain installed in /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21 for inspection.\nResults logged to /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/gem_make.out\n", "stderr_lines": ["WARNING:  You don't have /home/vagrant/.gem/ruby/2.3.0/bin in your PATH,", "\t  gem executables will not run.", "ERROR:  Error installing compass:", "\tERROR: Failed to build gem native extension.", "", "    current directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c", "/usr/bin/ruby2.3 -r ./siteconf20180214-10687-1xc1xl7.rb extconf.rb", "checking for ffi.h... no", "checking for ffi.h in /usr/local/include,/usr/include/ffi... no", "checking for shlwapi.h... no", "checking for rb_thread_blocking_region()... no", "checking for rb_thread_call_with_gvl()... yes", "checking for rb_thread_call_without_gvl()... yes", "creating extconf.h", "creating Makefile", "", "To see why this extension failed to compile, please check the mkmf.log which can be found here:", "", "  /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/mkmf.log", "", "current directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c", "make \"DESTDIR=\" clean", "", "current directory: /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c", "make \"DESTDIR=\"", "Running autoreconf for libffi", "autoreconf: Entering directory `.'", "autoreconf: configure.ac: not using Gettext", "autoreconf: running: aclocal --force -I m4", "autoreconf: configure.ac: tracing", "autoreconf: running: libtoolize --copy --force", "autoreconf: running: /usr/bin/autoconf --force", "configure.ac:187: error: possibly undefined macro: LT_SYS_SYMBOL_USCORE", "      If this token and others are legitimate, please use m4_pattern_allow.", "      See the Autoconf documentation.", "autoreconf: /usr/bin/autoconf failed with exit status: 1", "libffi.mk:6: recipe for target '\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a' failed", "make: *** [\"/home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21/ext/ffi_c/libffi-x86_64-linux-gnu\"/.libs/libffi_convenience.a] Error 1", "", "make failed, exit code 2", "", "Gem files will remain installed in /home/vagrant/.gem/ruby/2.3.0/gems/ffi-1.9.21 for inspection.", "Results logged to /home/vagrant/.gem/ruby/2.3.0/extensions/x86_64-linux/2.3.0/ffi-1.9.21/gem_make.out"], "stdout": "Successfully installed sass-3.4.25\nSuccessfully installed multi_json-1.13.1\nSuccessfully installed compass-core-1.0.3\nSuccessfully installed compass-import-once-1.0.5\nSuccessfully installed chunky_png-1.3.10\nSuccessfully installed rb-fsevent-0.10.2\nBuilding native extensions.  This could take a while...\n", "stdout_lines": ["Successfully installed sass-3.4.25", "Successfully installed multi_json-1.13.1", "Successfully installed compass-core-1.0.3", "Successfully installed compass-import-once-1.0.5", "Successfully installed chunky_png-1.3.10", "Successfully installed rb-fsevent-0.10.2", "Building native extensions.  This could take a while..."]}
ok: [foreo] => (item=sass)
changed: [foreo] => (item=less)
changed: [foreo] => (item=zen-grids)

RUNNING HANDLER [geerlingguy.firewall : restart firewall] **********************

Summary
When vagrant box setup is running with Ruby setup and additional gem install it is broken (see pasted output above)
Have solved this with building Ruby from source
ruby_install_from_source: true
But maybe someone should take a look. This setup was working well for some time until yesterday.

@valicm - Thanks for reporting this; please see the upstream issue in the Ruby role here: https://github.com/geerlingguy/ansible-role-ruby/issues/42
This was fixed and is currently in the master branch of Drupal VM (latest dev revision). It isn't yet in a tagged Drupal VM release though. But as you say, installing Ruby from source temporarily resolves the issue if you need to stay on a tagged version of Drupal VM (and would continue working in the future as well).


Web page only provides x86 msix packages and not the ARM ones

[Enter feedback here]
The bulk installer section that indicates to download the bootstrap and the msix only has the x86 version to downlaod. The ARM version is not avaliable to download and it is needed to make an offline package

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

ID: 4b73fb04-90e1-de2e-e81d-3495c004b2b5
Version Independent ID: 4b73fb04-90e1-de2e-e81d-3495c004b2b5
Content: Bulk deploy the new Microsoft Teams desktop client - Microsoft Teams
Content Source: Teams/new-teams-bulk-install-client.md
Service: msteams
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Duplicate of https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10451

Contacting the product team.

@get-itips @LuisAmadeo   The articles have now been updated to include all 3 options listed -- x86, x64, & ARM 64.


Impossible to make non-C++ DeviceApi

It's possible to register new APIs from other languages by registering a global device_api.my_device, but it breaks down because the actual implementation depends on being a C++ DeviceApi with methods like AllocDataSpace. One solution is to create a "wrapper" DeviceAPI which calls out to globals like device_api.ext_dev.alloc_data_space.

what are you trying to do that couldn't be supported by a Rust DeviceApi

(@Ravenwater) The issue is that one cannot make a device API from non-C++ languages because creating one requires generating a pointer to a DeviceAPI which is used within C++ (and isn't C ABI-compatible POD). This PR just makes it possible to make a device API in any language (incl. Python).
Of course, what the PR does is independent of why it exists. The reason is because I wanted to write a VTA simulator in Rust (and eventually Scala) for easier integration testing. @tqchen is right, though: the runtime should be kept minimal. I'll batch these changes until either 1) the other languages become self-sufficient or 2) I accumulate enough of these changes to make a cohesive contrib library to be hidden behind compiler flags.
I'll close this issue for now.

@nhynes turns out that we have a lot in common! I am looking into how to connect our Golang simulation environment that models are tensor processor and masquerade it as a VTA widget.
Since, our sim environment can run at scale, modeling hundreds of thousands of processors, we went with a GRPC/HTTPS interface that marshals the command and data streams of the processor. So I was planning to create a C++ stub that can create and manage that connection. This simplifies the integration point and decouples the build and run environments, keeping TVM/VTA pure C++, but being able to talk to a remove cluster of simulation capability.
I would think that is the way you want to go as well, particularly if you want to push the modeling into a Scala universe.

Iiinteresting. I'm actually also trying to extract the commands and data from VTA programs. I tried extracting the cmds and uops directly from the IR, but it's easier to just run the compiled program.

C++ stub that can create and manage that connection

Right, I see. If it's C++, you could just register your own device_api.ext_dev and replace the VTA simulator with your own high-performance, distributed implementation. It's an extra layer of indirection, but it keeps TVM simple and focused, so that makes complete sense.

architecture pattern a github tracking issue

Quite possibly. There's definitely some common functionality here. Also, once the VTA Chisel port is ready, we'll need a way to streamline building TVM modules, extracting their data, pushing the memories into the (Chisel) simulator stack, and running the test harness.
Right now I'm trying to get the data extraction rolling. If I understand correctly, you're working on getting VTA programs into your simulation cluster. It seems like the common bit of functionality is flat-packing the VTA module and design in a common format that can be shipped to an arbitrary simulator. In that case, the current sw sim, viviado xsim, the eventual treadle sim, and your setup can all consume the same underlying format.
If the above is true, what beyond the initial memories are you trying to extract? It might be worth putting together a proper RFC to see what else people need.

We need to elevate this to a real discussion topic

Yes, this should be done now before too much real discussion gets lost in this thread.

there are two more dimensions you will need

I think that the first dimension is something that can reasonably be expected of a DeviceAPI since synchronization comes up in any context that's not (host cpu)=(target cp). I'm not so sure about the second, though.
Dimension 1: Concurrency

multiple kernels running, possibly within a time-shared resource manager and multiple users

Such functionality would be amazing to see and is certainly a logical goal. My only concern would be that the scheduler implementation is highly specific to each deployment (e.g., multi-tenancy on an F1 instance vs time/user sharing a device fleet in an on-prem datacenter) and would be more suited to a collection of externally maintained libraries. If anything, the core project would only have stubs that allow such extensions.
Dimension 2: Device Control
I agree that a DeviceAPI should be able to fully control

include a full software emulation of that object

Not quite sure I agree here. Part of the issue is that the TVM IR can be wildly different before and after scheduling for a given platform. Emulation wouldn't offer a reliable estimate of performance unless the program is fully lowered. In that case, you'd be stuck writing an emulator for each and every platform (and, indeed, is what we see with the bespoke VTA C++ simulator). I don't think that I fully understand the use case, but if the goal is simply to test the computation, then there's always LLVM/LLI as the common denominator.

deal decisively with failures and observability

Would something in-band and generic like tf.add_check_numerics_ops not suffice? If so, then this would probably fall under the purview of concurrency and synchronization.

not pleasant to rely on a real hw driver

Do you mean specifically for simulation or in general?


upd(ci): upgrade GitHub Actions to newer versions

Updated various GitHub Actions used in the CI workflows (build.yml, gh-pages.yml, test.yml) to their latest major versions. This includes actions for checking out code, setting up Go, uploading artifacts, configuring Docker, and deploying to GitHub Pages. The update is part of routine maintenance to ensure compatibility with the latest features and improvements provided by these actions.
With loves, your's dependabot :)

Thanks!

https://github.com/AlexxIT/go2rtc/releases/tag/v1.9.0


pfx File

Hi There,
I am currently wishing to use FileTypeDetector but when I try to run the downloaded code it cannot find FileTypeDetector.pfx
Could I ask if the file is available, if not, what is the format of the .pfx file so that I may create it myself.
Kind Regards
Phil sargeant

Solved - I figured that you signed the assembly.

I figured out that you signed the assembly. Now works fine so I have closed the issue.
Regards
Phil


NGAP/MCP Pilot: Pilot Nucleus Deployment for MESSENGER MDIS Data

💡 Description

https://github.com/NASA-PDS/nucleus/issues/52
https://github.com/NASA-PDS/nucleus/issues/53
https://github.com/NASA-PDS/nucleus/issues/54

📆 May status: Delay of 1 sprint expected due to delays in onboarding.

📆 June status: In progress. On schedule.

📆 July status: In progress. Task is taking much longer than expected due to large data volumes being worked. Plan to process subset by end of build. POC task that does not impact build delivery.

📆 August status: Deferring to B14.1. Issues with AWS changing permissions on us in NGAP. Completed a workaround to load the MESSENGER data without Nucleus for the time being.


Unable to remove logger

I understand how to remove a transport within a logger and have implemented it, but I have not been able to find any information on how to remove a logger.  I have a site that can have multiple games played that only last 10 minutes or so.  Each game instance adds a logger to log to its own file by game id.  It does not seem to make sense to keep the logger in memory and only remove the transport for it.
Is it acceptable to simply delete the object or is there a better way?
delete Winston.loggers.loggers["192"]

Use winston.loggers.close('id')


clang-format after moving to c++

previous PR from Jeremy didn't update source style
disable indentation of namespaces
upgrade clang-format  to 5.0

Why are we changing clang minimal version ?

no specific reason but 4.0 is quite old now and when we install clang-format (say with brew), clang-format 5 is installed. Hence I thought we should start using latest version.

p.s. note that most of changes in this PR are not specific to 5.0


[BUG] ...

Describe the bug
When I try to open the toolkit, the following message pops up:
"An error occurred in Toolkit for YNAB (a browser extension you installed to enhance YNAB). Click here to report the issue to the YNAB Toolkit project."
with a refresh button.
To Reproduce
Steps to reproduce the behavior:

Open YNAB
Click on "Toolkit Reports"
Error pops up

Expected behavior
I expected the toolkit to open, but only the error box appears
Desktop (please complete the following information):

OS: Chrome and Windows 7
Browser: Chrome
Toolkit Version Unknown (just updated)

Setting Export
Unable to open

Same thing is happening to me, starting today.
OS:  Windows 10
Browsers:  Chrome Version 84.0.4147.89 and Edge Version 84.0.522.59
Toolkit Version:  2.26.0

Duplicate of #2087

This has been fixed in the current release. It'll likely take Google a couple of business days to get the new release approved so you'll have to be patient. Closing as a duplicate.


[seamlisten] remove double slash comments in css

The headers of css files contain copyright information with double slash comments which are invalid css. Modify them to /* */ comments.

This issue was already resolved.

@Celebio
I can work on this issue


Is opencv-python-rolling still built every week?

Is opencv-python-rolling still built every week? I found the latest version was built on Feb. 11.

Looks like something broken in CI. I'll take a look.

opencv-python-rolling is recovery, so close this issue


Adding prism on dangerouslySetInnerHTML

Is there a way I can highlight code in the React's dangerouslySetInnerHTML?
I tried it but it seams the Prism doesn't highlight the code that way...

Did you find any fix for this ?


These are serialised without problems, but the deserialiser assumes presence of everything.
Owl only sparsely populates CrewMembers in most cases

:shipit:


Template x carousel block

Fixes following issues|Adds following features:

Creates a basic carousel that will allow a centered card on mobile that can be navigated from left to right and maintain the card in focus centered while lowering the opacity of the cards on either side to 0.5 and keeping the card in focus at 1.
This will be used for the International Template-X carousel requirements.
I've discussed this approach with Jingle and he has approved this approach.
The rationale for creating a new carousel for these features is that the current carousel has a very high surface area and touches many parts of Express and is also considered buggy - so any changes would force a site wide regression.
Further, since this requires CSS changes to enable the product requested Template-X changes we can silo those CSS changes in this carousel for that use case without adding more bloat to the existing Template-X CSS.

Resolves: MWPW-155657
Before - Does not exist
After
Mobile

Desktop

Pages to check for regression and performance:

https://template-x-carousel-block--express--adobecom.hlx.page/drafts/jsandlan/blog-page

Codecov Report
Attention: Patch coverage is 12.80000% with 109 lines in your changes missing coverage. Please review.

Project coverage is 43.49%. Comparing base (0c5ee58) to head (dd34d55).
Report is 30 commits behind head on stage.

Files with missing lines
Patch %
Lines

express/blocks/shared/basic-carousel.js
8.92%
102 Missing :warning:

express/blocks/shared/carousel-wrapper.js
50.00%
5 Missing :warning:

express/blocks/template-x/template-x.js
33.33%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #1295      +/-   ##
==========================================
+ Coverage   42.72%   43.49%   +0.76%
==========================================
  Files          97       98       +1
  Lines       22896    22765     -131
==========================================
+ Hits         9782     9901     +119
+ Misses      13114    12864     -250

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


ARROW-14283: [CI][C++] Disable Gandiva on macOS GHA builds

Workaround failure to find LLVM during the CMake config step.

@kou Feel free to push it here :-)


Added methods to easily obtain keys

Could there be cases where current projects have keys directories with underscores that FileNameFilter will try looking for dashes on instead?


InvalidArgumentError: Negative dimension size caused by subtracting 2 from 1 for 'max_pooling2d_10/MaxPool' (op: 'MaxPool') with input shapes: [?,1,112,128]

When I run the lesson1 notebook, I get the above mentioned error. Quick google search brought me here. How do I resolve this?

It looks like your Keras is set to use TensorFlow instead of Theano (part 1 notebooks use Theano).
One thing to check is the keras.json file in your .keras directory. I suggest you try modifying your keras.json using the "keras.json.for_Theano" template from my repo. This should hopefully solve the issue.

It did not work.

Find this line:
from keras import backend as K
and put after it:
K.set_image_dim_ordering('th')

@ragvri
In case you have not yet solved the issue (also using the alternative suggestion thanks to @vteodorescu) it would help to see a full error log + the actual Keras version + the keras.json file.


Handle SAML SLO calls

It only works for front-channel calls, but the back-channel support can be added by customization.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Setting autoStartServer to false gives spurious warnings

steps
In ~/.sbt/1.0/global.sbt, I have added:
Global / autoStartServer := false

This works, but gives some warnings:
[warn] there's a key that's not used by any other settings/tasks:
[warn]
[warn] * Global / autoStartServer
[warn]   +- /home/hrj/.sbt/1.0/global.sbt:4
[warn]
[warn] note: a setting might still be used by a command; to exclude a key from this `lintUnused` check
[warn] either append it to `Global / excludeLintKeys` or call .withRank(KeyRanks.Invisible) on the key

problem
Although it works, the warning makes me wonder if I am setting it correctly.
expectation
If my usage of the key is correct, it would be better if the sbt doesn't print the warning.

Same here, thank you for opening this issue.
I added Global / lintUnusedKeysOnLoad := false as a workaround until properly fixed.


Migration documentation

The documentation on migrations is not clear that most migrations will run by just incrementing the schemaVersion number

At the very minimum all we need to do is to update the version with an empty block to indicate that the schema has been upgraded (automatically) by Realm.

A list of what is migrated automatically would be very helpful.  There is some mention in the same code block, but this is easy to overlook.

Hi @dmorrow. Thank you for reaching out about this. I'll have someone respond with some more information or refer you to an existing resource soon. Cheers!


Cann't open the online encrypted post

Expected Behavior
When I tested the site locally, it seems fine, the post shows up after I type in the password, yet there is no response when doing online test.
Actual Behavior

Steps to Reproduce the Problem

[x] hexo clean && hexo g && hexo d, didn't work
[x] remove hexo-blog-encrypt and resinstall, didn't work

Specifications
(The version of the project, operating system, hardware etc.)
hexo: 3.8.0
hexo-cli: 1.1.0
os: Darwin 18.5.0 darwin x64
node: 11.5.0
v8: 7.0.276.38-node.13
uv: 1.24.0
zlib: 1.2.11
ares: 1.15.0
modules: 67
nghttp2: 1.34.0
napi: 3
llhttp: 1.0.1
http_parser: 2.8.0
openssl: 1.1.0j
icu: 63.1
unicode: 11.0
cldr: 34.0
tz: 2018e

It looks like jQuery link is invalid.
Can you change the jquery link to https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js, you can find this link in your template file or your settings.
You can refer to this： https://github.com/MikeCoder/hexo-blog-encrypt/pull/73

Thanks for your response, I use the theme next and I've looked up its source directory tree, I have no idea of where this template file is.

Can you give me a link to your page?
Also I mean template file is /node_modules/hexo-blog-encrypt/template.html, sorry for the misleading.

No, problem https://complicatedphenomenon.gitlab.io/2018/01/27/life/

/node_modules/hexo-blog-encrypt/template.html

</script>

</script> type="text/javascript">

</script>

  <input type="password" class="hbe-form-control" id="pass" placeholder="{{message}}" />
    {{message}}

{{decryptionError}}

{{noContentError}}

{{content}}

The link is already the latest one, no need to modify

I get you problem. In your post, the JQuery link is at the last, but the plugin call jQuery in the middle.

I don't know whether you customized template in your post, like https://github.com/MikeCoder/hexo-blog-encrypt#change-template. I don't find
 in your post.
I recommend you to add a jquery link before blog-encrypt.js.
The default one is:

  <input type="password" class="hbe-form-control" id="pass" placeholder="{{message}}" />
    {{message}}

{{decryptionError}}

{{noContentError}}

{{content}}

It's wired, what I provide is indeed different from yours. But I didn't modify a thing of this plugin. Moreover, I also host this website on Gitlab, this problem appears too. on Gitlab, CI & and CD is done automatically with .gitlab-ci.yml.
image: node:11.5.0

cache:
  paths:
    - node_modules/

before_script:
  - npm install hexo-cli -g
  - test -e package.json && npm install
  - hexo generate

pages:
  script:
    - hexo generate
  artifacts:
    paths:
      - public
  only:
    - master

Try adding  in https://github.com/theme-next/hexo-theme-next/blob/master/layout/_partials/head/head.swig. As
<meta charset="UTF-8"/>
<meta http-equiv="X-UA-Compatible" content="IE=edge"/>
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=2"/>
<meta name="theme-color" content="{{ theme.android_chrome_color }}"/>

...

It may help.

The solution works, although I didn't make myself clear, you find the problem finally. Thanks 🍻 :

Actually, no... I don't know why your blog remove the jquery link generated by hexo-blog-encrypt. But accroding to you, it works fine offline. So it's possible caused by CI or something between you and the website.

文章封面总是多出来一串字符


NDK release v 1.0.4

Memory leak fix:

Turn on ANRMonitor, update tests
Re-arrange code to fix memory leak, add logs
Added Codecov and GHA

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@3c4be8f). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             main     #48   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?       9
  Lines           ?     419
  Branches        ?      66
======================================
  Hits            ?       0
  Misses          ?     419
  Partials        ?       0

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


docs: add create-react-app example

I once tried to use unocss in an existing create-react-app project, refer to  https://github.com/unocss/unocss/discussions/1781 , for the create-react-app project, it is not easy to import unocss to use, it took me a while to solve it. the main thing is you have to figure out the problem and modify and override the default configuration, In order to prevent others from encountering the same problem, I would like to add a usage example

I fixed the eslint script error and added some comment

I am not even sure if CRA should still be recommended today - their team seems not actively maintain it, and migrating CRA project to Vite sounds straightforward.
Anyway, I am ok to have it, as long as we can keep the example minimal as possible, which means:

No assets (images, robot.txt, etc), we could use icons preset for the logo
No documentation of CRA/React themselves - as it's not related to the example
No CSS if possible

Thanks

I once tried to use unocss in an existing create-react-app project, refer to #1781 , for the create-react-app project, it is not easy to import unocss to use, it took me a while to solve it. the main thing is you have to figure out the problem and modify and override the default configuration, In order to prevent others from encountering the same problem, I would like to add a usage example

I'm stuck right here just now ,thanks for your demo :)

Did you met HMR problem ?  i try use this configuration, but the HMR is broke.
When i change class to update styles, i have to refresh my browser. The same problem also occur in webpack project without CRA,maybe it's a @unocss/webpack issue ?

Did you met HMR problem ? i try use this configuration, but the HMR is broke. When i change class to update styles, i have to refresh my browser. The same problem also occur in webpack project without CRA,maybe it's a @unocss/webpack issue ?

seem to be the webpack did not detect the virtual file virtual/__uno.css has changed, i guess

Closing it as I consider CRA to be soft sunsetting (new React docs no longer mention it).


Update the editor CheckButton icons

Unfortunately, I no longer have the original files from Inkscape, so I put the optimized SVGs in both folders. There's probably no visible difference between the original and optimized SVG anyway, as the icons are very simple.
See https://github.com/godotengine/godot/pull/24116.

Bump :slightly_smiling_face:

Superseded by #57.


class_or_bound_method decorator: use descriptor

Refs #10

Codecov Report

Merging #11 into master will decrease coverage by 0.82%.
The diff coverage is 86.95%.

@@            Coverage Diff            @@
##           master     #11      +/-   ##
=========================================
- Coverage   87.62%   86.8%   -0.83%
=========================================
  Files          16      16
  Lines         404     394      -10
  Branches       25      24       -1
=========================================
- Hits          354     342      -12
- Misses         39      40       +1
- Partials       11      12       +1

Impacted Files
Coverage Δ

lock_tokens/utils.py
89.28% <84.61%> (-4.37%)
:arrow_down:

lock_tokens/models.py
91.11% <90%> (-0.29%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d938451...9d6a52e. Read the comment docs.


Adding meta information does not overwrite default values

   "html-webpack-plugin": "^2.24.1",
   "html-webpack-template": "^5.4.1",

webpack.config.js file:
    plugins: [
		new HtmlWebpackPlugin({
		title: 'Appname',
		inject: false,
		filename: PATHS.prodFile + '/index.html',
		template: require('html-webpack-template'),
		appMountId: 'appContainer',
		meta: {
		    'theme-color': '#FF5A64'
		},
		mobile: true,
		}),
		new FaviconsWebpackPlugin(PATHS.statics + '\\ashojash.png')
							]

I want to add a  theme-color meta to produced files.
I noticed that there is a theme-color by default with content #fff So I deducted that adding theme-color should overwrite it but it doesn't.
Now there is to theme-color meta tag in my file and seems like the browser follows the default one.
What am I doing wrong here?

I found that FaviconsWebpackPlugin was generating that. thanks.


Add a loading indicator

Add support for a loading indicator that can be shown or hidden. Optionally, show the loading indicator by default if there's no data in the grid.

Fixed in PR #11


Branch comparison

When a new branch is ready to be pushed to github,
these scripts can be run to evaluate the effect of
the changes brought to the code on model outputs.
The companion python script of this bash script
(simulation_comparison_setup.py) will produce
a series of graphs for this purpose.
The product of this comparison will be a pdf
named result.pdf
This pdf should be provided with the push of any new branch.
To run the comparison,
(1) open simulation_comparison_setup.sh
(2) edit the flags as indicated
(3) run the bash script.
result.pdf

Replaced by #634