Question: possible to use sht_rails as the top level view from a controller?

When I call render from a controller method using a handlebars template it doesn't seem to be rendering. Is it possible to use sht_rails to render from a controller (with or without layout) instead of only rendering partials? Thanks.

It should work.


false positive: transfer.adttemp.com.br

The domain "transfer.adttemp.com.br" is a public instance of the transfer.sh software, used for temporary file hosting (similar to catbox.moe). It's currently on the blacklist. There's nothing inherently bad about it.
It's currently in this file of yours: https://raw.githubusercontent.com/hectorm/hmirror/master/data/urlhaus/list.txt

This repository is just a mirror, you should report this upstream.


Issues with analytics when seeking with keyboard

If using analytic plugin, when seeking with keyboard, video is not paused. So there is no way to tell that user skipped some part of video (seeking with mouse handles this fine: video is paused at old time and resumed at new). Solution would be to pause video when user starts seeking(and not pause when user seeks again to avoid "pause" event triggered more than once), wait for "pause" event to be processed(to provide curtentTime at seeking start , and allow user to resume video manually. If this change is approved, I can create pull request.

@igorko sure Igor, any fix or improvement is always more than welcome. Mediaelement depends on community contributions. Thanks to be an active member :-)

Looks like this timeout in 1100 miliseconds is bad when seeking by clicking arrow key few times. In some cases (when pressing arrow key with some interval near second) play is triggered (from keyAction), and you still can seek by one keypress (inside keydown event listener). As a result play event is triggered with outdated time(you seek once more, time is changed, but pause/play is not triggered once more). Setting timeout to 0 causes more side effects - in some cases play is not triggered at all. I am not familiar with all the logic for key events. Could someone look into it?

I have even reproduced testcase when you can add more thet one keypress after media was resumed and this will not be tracked (you hear playing media for a moment each time you press a key, than when you leave it for a while - about second - all starts to work correctly)


How to compile this project?

I tried to compile this project using Arduino IDE, but some errors occured.
I think my project setting is not right.
I would appreciate it if you could tell me your project settings.
Environment

OS: macOS High Sierra Version 10.13.6
Arduino IDE: v1.8.5

Board Setting: WEMOS D1 & mini

Board: WEMOS D1 mini

What I tried

follow this tutorial for board setting:

install CH340G driver
install hardware package using git

setup project

clone this repository
install two libraries

push compile button on Arduino IDE

Error
In disply.ino, the following error occured.
display:1054: error: macro "putchar" passed 3 arguments, but takes just 1
     putchar(charArray[i], (i % 21) * 6, (i / 21) * 8);
                                                     ^
display:1096: error: macro "putchar" passed 3 arguments, but takes just 1
     putchar(c, regx * 6, regy * 8);
                                  ^
display:1123: error: macro "putchar" passed 3 arguments, but takes just 1
     putchar(s[i], i * 6, y);
                           ^
display:1128: error: macro "putchar" passed 3 arguments, but takes just 1
 void putchar(char c, uint8_t x, uint8_t y) {
                                          ^
exit status 1
macro "putchar" passed 3 arguments, but takes just 1

This error was fixed by changing putchar to putchar_, but it's not a solution to the root of the problem, so other errors happened one after another.
I think my project setting is not right.
I would appreciate it if you could tell me your project settings or how to solve this problem.
Thanks.

Sorry for bothering you.
I moved this issue into esp8266_game_engine, so I close this.


Overwrite a read-only permission file succeed

Alluxio Version:
2.7.1
Describe the bug
Overwrite a read-only permission file succeed.
To Reproduce
$ echo hello > test_file
$ ll test_file
-rw-rw-r-- 1 test test 6 Jul 13 11:55 test_file
$ chmod 400 test_file
$ ll test_file
-r-------- 1 test test 6 Jul 13 11:55 test_file

$ echo hello > test_file
$ ll test_file
-rw-r--r-- 1 test test 6 Jul 13 11:56 test_file

We can found that the original file was deleted and a new file created with default permission.
Expected behavior
Failed with "Permission denied" as the local filesystem.
Urgency
ACL does not take effect
Are you planning to fix it
Yes

could you try with mount option "default_permission"?

@LuQQiu  Any comment here?

could you try with mount option "default_permission"?

default_permissions works, thanks a lot @yyongycy


Is it possible to access the models in the runBefore method?

We have a plugin that implements a custom QueryBuilder and applies tenant-level filters with onBuild, works great. We would now like to inspect all of the models before they are inserted (eventually updates/patches too) to ensure the operation is not attempting to associate a record to a tenant they are not part of.
We have access to runBefore method in the QueryBuilder but I don't see any method to access the models that are about to be inserted/updated.
We are trying to avoid $beforeInsert or beforeInsert hooks because it is far too easy to forget to also add the super call if these hooks are also implemented on the model. If this is forgotten or accidentally omitted these methods in the plugin with its authorization logic will not run. Tests should catch this if it is omitted, but we are trying to minimize this risk as much as possible.
Perhaps there is a different or better way to approach this? Or if not, is there any way to access the models in the runBefore method on a QueryBuilder?

With regards to the $beforeInsert and beforeInsert option, would there be a way to test a model to ensure await super.$beforeInsert(context); was added to the function. We will test for behavior but I also want to test that single line of code is added to every model's $beforeInsert method.
I suppose I could stringify the function and check for it but hoping there is a more reliable way, especially with TypeScript as it may change the output of the compiled code.

Ended up going with $beforeInsert or beforeInsert and leaning on tests to ensure super is added.


Support loading webhook config from versioned file

What this PR does / why we need it:
This PR adds support for loading configuration following the k8s.io/component-base recommendations (https://docs.google.com/document/d/1FdaEJUEh091qf5B98HM6_8MS764iXrxxigNIdwHYW9c/edit#heading=h.ctbveyrh07ch).
It adds a new type, WebhookConfiguration, which today contains a field for each of the current options we expose as flags.
This will allow us to e.g. rename fields, as well as pave the way for dynamically loading config (i.e. one day these types could be used as CRDs, with our various components watching for their own config objects).
Special notes for your reviewer:
Most of the complexity in this is within cmd/webhook and is there to handle 'layering' of configuration. In accordance with component-base recommendations, all options specified as flags on the CLI by the user will always take precedence over the equivalent field in the WebhookConfiguration object.
Release note:
Add support for loading webhook flags/options from a WebhookConfiguration file on disk

/cc @wallrj

/milestone v1.7

'm +1 on making this easier, and naturally Helm does make sense, but given this is an optional feature for now I think it'd be easier to provide a ConfigMap example that is well documented, and have users create the ConfigMap if they do want to make use of this config API type (we can have the chart deploy with an optional: true configmap with a particular name to make this easy for users to use)

ConfigMap makes sense, but I think we need to allow to specify a volume to be mounted for webhook deployment via Helm chart.

ConfigMap makes sense, but I think we need to allow to specify a volume to be mounted for webhook deployment via Helm chart.

My plan was to not expose this as a config option in the Helm chart, and instead update the Helm chart to always configure the webhook with a volume/volumeMount for a ConfigMap with a 'known name' such as cert-manager-webhook-config, but set it to optional: true - this way, all a user has to do to configure configuration is create this ConfigMap and it'll then be picked up by the webhook with no further changes. If the ConfigMap does not exist, the webhook will start up just fine as the volume is optional :)

My plan was to not expose this as a config option in the Helm chart, and instead update the Helm chart to always configure the webhook with a volume/volumeMount for a ConfigMap with a 'known name' such as cert-manager-webhook-config, but set it to optional: true - this way, all a user has to do to configure configuration is create this ConfigMap and it'll then be picked up by the webhook with no further changes. If the ConfigMap does not exist, the webhook will start up just fine as the volume is optional :)

Thanks for explaining that, makes sense 👍🏼
I think this PR should be good to go in once the Helm chart option and the flag deprecation is there

@wallrj The API group should be webhook.config.cert-manager.io instead of config.webhook.cert-manager.io - Irbe noticed this during review and I've not addressed it yet 😅

Yep. That was it. It's working now:
kubectl -n cert-manager logs cert-manager-webhook-657bdc5ffb-9694b
W1202 17:29:27.137527       1 client_config.go:615] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
W1202 17:29:27.138778       1 client_config.go:615] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
I1202 17:29:27.138882       1 webhook.go:76] cert-manager/webhook "msg"="using dynamic certificate generating using CA stored in Secret resource"  "secret_name"="cert-manager-webhook-ca" "secret_namespace"="cert-manager"
I1202 17:29:27.139181       1 server.go:141] cert-manager/webhook "msg"="listening for insecure healthz connections"  "address"=":6080"
I1202 17:29:27.139975       1 server.go:174] cert-manager/webhook "msg"="running go profiler on"  "address"="localhost:6060"
I1202 17:29:27.140040       1 server.go:205] cert-manager/webhook "msg"="listening for secure connections"  "address"=":8443"
I1202 17:29:27.141535       1 reflector.go:219] Starting reflector *v1.Secret (1m0s) from external/io_k8s_client_go/tools/cache/reflector.go:167
I1202 17:29:27.141561       1 reflector.go:255] Listing and watching *v1.Secret from external/io_k8s_client_go/tools/cache/reflector.go:167
I1202 17:29:28.214329       1 dynamic_source.go:273] cert-manager/webhook "msg"="Updated serving TLS certificate"

by default when deployed with Helm a configmap will be created, but will be empty and --config flag will not be passed

Yeah I wonder if we're better to not create it at all... not sure. Needs some thought, but I think we can follow up later.

with the static manifests, it is up to the users to create a configmap with the WebhookConfiguration, mount the volume and set the --config flag

👍 correct!

at the moment there is no mechanism to ensure that webhook will reload the config when the contents of the configmap has changed. This may be solved in the future by having the webhook watch a WebhookConfiguration CR

Yeah we can look at a follow up for this. Either annotating pods with a hash of the config provided (which I think is a common Helm pattern) or improving the webhook to actually handle reloads when it changes 😄

Retesting after https://github.com/jetstack/testing/pull/612
/test pull-cert-manager-e2e-v1-22

I'm not convinced that Helm users should care what port the webhook process listens on,
only what port the webhook Pod and perhaps the webhook service use,
neither of which can be changed from the WebhookConfiguration file.

It is possible to change the listen port of the process - that's exactly what securePort does. It's not possible to change the mapping of 443->securePort in the Service however.

Will you be introducing a feature flag for this configuration file stuff?

It was decided to not have a feature flag for this, given it requires explicitly enabling anyway to get this turned on. Also, when looking in the upstream project, there was not a feature flag for the config file work (only for the dynamic config work where the the configuration is read from an actual CRD in the apiserver)

Is there any code here which has been copied from Kubernetes? And if so, perhaps we should add links to the upstream code: for our own benefit so that we can easily check upstream to see what improvements they've made in this area, and to give some copyright credit to the original authors.

There's not any code that's been copied 1:1 - the majority of this is 'inspired by' the upstream work, and follows the component-base/configuration guidelines.
Thanks for all the reviews everyone!
/hold cancel


[AUTO-PR] Automatically generated new release 2021-04-23T15:13:36.446Z

What are you changing?

[ ] Releasing a new version of Notify
[ ] Changing kubernetes configuration

Provide some background on the changes
⚠ The production version of the Terraform infrastructure is behind the latest staging version. Consider upgrading to the latest version before merging this pull request.
NOTIFICATION-API

fix: adjust log level when email address is invalid (#1277) by Antoine Augusti
feat: add tests when generating callbacks (#1276) by Antoine Augusti

NOTIFICATION-ADMIN
NOTIFICATION-DOCUMENT-DOWNLOAD-API
NOTIFICATION-DOCUMENT-DOWNLOAD-FRONTEND
NOTIFICATION-DOCUMENTATION
If you are releasing a new version of notify, what components are you updating

[ ] API
[ ] Admin
[ ] Document API
[ ] Document UI

Checklist if releasing new version:

[ ] I made sure that both API and Admin changes are present in Notify
[ ] I have checked if the docker images I am referencing exist - ex: https://gallery.ecr.aws/v6b8u5o6/notify-admin

Checklist if making changes to Kubernetes:

[ ] I know how to get kubectl credentials in case it catches on fire

Works in staging!


Handle max voting power restrictions

    @tombeynon it seems like restake is working with teritori except in the case where the validator cannot accept more staking

my other 2 validators are restaking fine, just this one that is overcapacity will not restake and only throws that error if I manually compound
Originally posted by @techmik in https://github.com/eco-stake/restake/issues/638#issuecomment-1304380754

I have been manually claiming rewards for this one while letting my other 3 delegations continue auto-compounding and everything usually is fine. Today when I manually claimed from that same validator the transaction appeared to go through, but the reward disappeared. It did not go to my wallet and was not compounded (which it shouldnt have been, I chose "claim rewards"). It wasnt much, but I wonder where it did end up going? This is the 1st time that happened and I just claimed again and it appeared to work fine this time. Just that 1 transaction earlier that was weird.


Text editing becomes less responsive while debugger is active

While the debug message window is open, typing into the editor becomes somewhat unresponsive, even if there is no debug activity.
Scrolling also becomes very choppy.

Thanks @Wuerfel21.  Turns out, this is a duplicate of issue #137 I just entered this morning, although your title is much better than mine for the general public.
This issue will be addressed soon.

This is fixed in v2.5.3.


Replaced renamed pep8 with pycodestyle and added install dependency

The pep8 module was renamed to pycodestyle. flake8 already switched to pycodestyle 2.0.0. Because of that using flake8 and flake8-todo results in a dependency glitch. I also added a install dependency on pycodestyle. Until now this dependency was only indirectly given because flake8 had a dependency on pep8.

Thanks. Can you please pin the dependency as flake8 is doing ('pycodestyle >= 2.0, < 2.1').

Done!

any chance of getting this merged and having a new release of the package?

Yes, I'll try to find time for a release at the weekend.

Thanks!


show icon drag #8051

@matallo could you take a look? cc/ @saleiva

Fix https://github.com/CartoDB/cartodb/issues/8051

:es:


Warehouse subscription image semverConstraint showing only very last tag

Checklist

[x] I've searched the issue queue to verify this is not a duplicate bug report.
[x] I've included steps to reproduce the bug.
[x] I've pasted the output of kargo version.
[x] I've pasted logs, if applicable.

Description
In the CRD reference it's specified that kargo supports semver for the container image (looking into the code I see based on https://github.com/Masterminds/semver), but when I tried the below Warehouse I got only and always the very last tag, even if the constraint is way broader.
apiVersion: kargo.akuity.io/v1alpha1
kind: Warehouse
metadata:
  name: kargo-demo
  namespace: kargo-demo
spec:
  subscriptions:
  - image:
      repoURL: nginx
      semverConstraint: ^1
      # or
      # allowTags: "[0-9]*.[0-9]*.[0-9]*"

Actually kargo should present way more tags...
Is it an expected behaviour?
What about document more the Warehouse available fields and how it works?
Screenshots

Steps to Reproduce
I followed the kargo quick-start guide in the documentation.
Version
Client Version: v0.3.2
Server Version: v0.3.2

Logs
No special logs in any of the kargo pods

but when I tried the below Warehouse I got only and always the very last tag, even if the constraint is way broader.

@bygui86 I'm not sure if I am misinterpreting you... the point of a warehouse is to select one version from each artifact (image, config, chart) repo you subscribe to. The SemVer selection strategy picks the latest that matches the constraint.
There are docs on all fields here:
https://doc.crds.dev/github.com/akuity/kargo

@krancour you mentioned also:

There are docs on all fields here: https://doc.crds.dev/github.com/akuity/kargo

In which is written (Warehouse > subscriptions > images > semverConstraint):

SemverConstraint specifies constraints on what new image versions are permissible. The value in this field only has any effect when the TagSelectionStrategy is SemVer or left unspecified (which is implicitly the same as SemVer). This field is also optional. When left unspecified, (and the TagSelectionStrategy is SemVer or unspecified), there will be no constraints, which means the latest semantically tagged version of an image will always be used. Care should be taken with leaving this field unspecified, as it can lead to the unanticipated rollout of breaking changes. Refer to Image Updater documentation for more details.

SemverConstraint specifies constraints on what new image versions are permissible. and This field is also optional. When left unspecified, (and the TagSelectionStrategy is SemVer or unspecified), there will be no constraints, which means the latest semantically tagged version of an image will always be used. are in contrast to what you wrote above the point of a warehouse is to select one version from each artifact (image, config, chart) repo you subscribe to. The SemVer selection strategy picks the latest that matches the constraint.
So how kargo use the Masterminds/semver library?
I would like to highlight an important point: CRD reference is absolutely important, but it cannot replace a good documentation explained in a more extended way and providing examples!
Anywhere in https://kargo.akuity.io I can't find any example of how to use a Warehouse in details or any example of how to use semverConstraint or allowTags or ignoreTags or else...

Then 1 Warehouse 1 version... then how can I subscribe for multiple versions (for example multiple container image tags)?

I think there may be a fundamental misunderstanding about the sort of use cases Kargo is intended to address.
If what you want to do is explicitly deploy one specific version of an image, you don't really need to involve Kargo. Or at least you don't need to involve it in watching images.
Kargo is for watching for new versions of images (or manifests or charts) over time. A semantic version constraint of ^1.24.0, for instance, does not express an interest in all versions 1.x.y. It expresses a desire to create new freight when a new version appears in the top of that range.
A meaningful comparison could be made to a package.json file (or the dependency manifest for pretty much any package manager). I don't believe anyone interprets"react": "^18.2.0" as a dependency on all versions of react in the 18.x.y range, rather, it is well understood to express a dependency on one version at the top of that range.

CRD reference is absolutely important, but it cannot replace a good documentation explained in a more extended way and providing examples!

I 100% agree, but I ask that you not lose sight of the fact that Kargo is alpha-level software right now and we're not done making breaking changes. Having useful docs without overcommitting effort to things that are likely to change feels, at times, like a delicate balancing act. The situation will improve incrementally as Kargo matures. In the meantime, the CRD docs are, at least, always up to date.

@krancour thanks for the explanation! Now usage of Warehouse is a bit more clear!

Kargo is for watching for new versions of images (or manifests or charts) over time. A semantic version constraint of ^1.24.0, for instance, does not express an interest in all versions 1.x.y. It expresses a desire to create new freight when a new version appears in the top of that range.

Just to be sure, so after creating the Warehouse I can expect to grab all new container image versions once released, right?

I 100% agree, but I ask that you not lose sight of the fact that Kargo is alpha-level software right now and we're not done making breaking changes. Having useful docs without overcommitting effort to things that are likely to change feels, at times, like a delicate balancing act. The situation will improve incrementally as Kargo matures. In the meantime, the CRD docs are, at least, always up to date.

Got it thanks. I thought kargo was in a bit more advanced stage, something like closer to beta :)

Just to be sure, so after creating the Warehouse I can expect to grab all new container image versions once released, right?

After creating a new Warehouse, you can expect to immediately get one Freight containing the one image at the top of the specified range. After, and on an ongoing basis, you'll get a new Freight with the new image at the top of the specified range each time one is pushed.

I thought kargo was in a bit more advanced stage, something like closer to beta :)

We're inching toward that, so you can expect us to step up our docs game quite a bit very soon.

Hello
I'm facing a similar situation, weirdly, it gets a 7 days old image, while there are about ten new since.
spec:
  subscriptions:
  - image:
      imageSelectionStrategy: SemVer
      repoURL: europe-docker.pkg.dev/test-infra112/ns/app

I only see one old image v2.108.0-f467ee7, and nothing else, new images have been pushed since it is running...

You can see it is the 12th image in the list:

@stephaneetje you're selecting the image with the highest semantically versioned tag. That's v2.108.0-f467ee7, which semantically speaking, is newer than your newest image (v2.108.0-e82fd8e).

I'm trying to select anything but tags ending with -$ShortSHA, i used NewestBuild strategy with following configuration :
  subscriptions:
  - image:
      imageSelectionStrategy: NewestBuild
      ignoreTags:
        - ^v.*-[0-9a-z]{7}$

But it keeps selecting an image with a ShortSHA, as if my ignoreTags field isn't effective.

Strange it isn't supported as it is apparently for allowTags.

That's a relic of Image Updater. We used to use Image Updater as a library for selecting images. That's what they support -- regex for allow and list for ignore. I'm not sure why they did it that way.
We could think about changing that, but I wouldn't rush into that change. The way I have seen it until now is this:

Use a regex to say what you allow
Use ignores to make specific exceptions to the regex without needing to complicate the regex

For your case, I think something like this works:
spec:
  subscriptions:
  - image:
      repoURL: europe-docker.pkg.dev/test-infra112/ns/app
      imageSelectionStrategy: SemVer
      allowTags: '^v\d+\.\d+\.\d+$'

That will select the semantically latest version from tags that don't have anything after the patch version.

Thanks for your reply!
It doesn't suit my needs, because i just want to exclude 'vx.x.x-0123456', but the other tags with - are still needed, for exemple i have a tag v2.0.8-RC1.

I would guess the range of acceptable patterns after the dash isn't infinite except for those resembling a short sha.
If that is so, it should be easy enough to do, right?
allowTags: ^v\d+\.\d+\.\d+(-(ALPHA|BETA|RC)(\.\d)?)?$


Easy dev setup

Make new dev onboarding as easy as possible with docker based workflow. We can use a postgres container to quickly spin up an instance

https://github.com/nycmeshnet/nycmesh-api


artifact generation + dataspace impl

Adds ArtifactGenerationExtension to provide specification for generating an artifact given a packageable elements
Adds Dataspace implementation

I'm adding the "bug" label as this is also fixing the "bug" introduced by my earlier PR that disabled the maven warning.
Maybe the label needs to be " bug fix" >?


compose: notice when drafts are saved

Fixes #7539

Thanks @AasthaGupta!

I think we can make the text "Saved as draft" (no exclamation mark for this), and make it green (same color as the one we use for "Copied!" in single_message.handlebars).
The notice is appearing way out in the right sidebar for me, rather than in the compose bar.

I don't think we need a new div for this.  We should be able to use the existing div and follow the examples of other compose-related messages in the code.  You should be able to track these down by grepping, or ask on chat if you get stuck.

@showell
Using existing div the message can be shown when the user changes streams.

But in the case when the compose box is closed and the message is saved as draft then I think we will need a new div to show the status above the panel.

I think we only want the notice when the compose box closes, not when you explicitly save a draft.
And maybe the notice should show above the "Drafts" button on the left, to help new users notice that button?
We should talk on https://chat.zulip.org/#narrow/stream/design about the best UI here, since we can get a few more opinions.

Yeah, the styling here should be similar (can just be identical, including the animation) to the "Copied" here:

It can go just to the right of the drafts button, whenever the compose box closes and a draft is saved. Here:

@rishig @showell
I have updated the pull request.

@AasthaGupta Have you tested this with a narrow window?
@rishig Do we want to test-deploy this?  I'm thinking we can just merge it and fix forward, since it's gotten a lot of design input.

@showell Yes I did test it with narrow by stream and by clicking outside of the compose box.
It worked as it should have.

Thanks @AasthaGupta, this is looking great! We don't need the test deploy this, but a few comments:

"Saved as draft" should be vertically centered and 7 pixels to the left of where it currently is
If you write a message and change the topic, it currently shows the "Saved as draft", which I think is not correct? I.e. I think we only save a draft when you close the compose box, so it should never appear in the open compose box?
Saved as draft should not have an ! at the end.

@rishig I have made the changes you asked for.

@AasthaGupta thanks for the quick turnaround; looks great from my end!
Sending to Steve for final review, but @brockwhittaker fyi. (In particular, wasn't sure if the "Saved as draft" should be vertically centered or if should be aligned to some sort of text baseline?)

@AasthaGupta I merged this, but there's a subtle flaw in the current scheme.  If you use the "preview" feature in the compose box, a draft gets created right at that point.  And then if you subsequently hit escape, you don't seem to get a new draft.
We should try to follow up on that, so I'll re-open #7735

Hrm now I'm seeing it work right on previews, so it was possible I was just confused.


Question: Validate clientside multiple fields as 'distinct' rule

Versions

vee-validate: 2.0.9
vue: 2.5.16

This is more of a question than a bug IMO. I literally searched through the entire API, the community online, the issues on github but couldn't figure a way to deal with this problem.
So I have a form which contains dynamic sections. Each section contains a dropdown (let's keep it simple). For all of my dropdown, I want to make sure that the same value is not picked more than once.
I thought of making a custom rule that would call our Laravel backend, but this is not possible currently because of our implementation of validation on this side.
Currently, this is how I manage backend errors coming in:
manageBackendErrors(errors) {
      for (let fieldName in errors) {
        // gives us an error in the ErrorBag-item format
        const fieldErrors = errors[fieldName].map(msg => {
          return { field: fieldName, msg };
        });

        const field = this.$validator.fields.find({ name: fieldName });

        // then for that field, make a correlation on the id of the error and the field
        // and add the error to our Error Bag
        fieldErrors.forEach(error => {
          // we can access the id using field.id
          error.id = field.id;
          error.rule = 'distinct';
          this.verrors.add(error);
        });

        // finally update the flags, which also updates the opposite flags
        // (i.e: valid-invalid, dirty-pristine)
        field.setFlags({
          valid: !! fieldName.length,
          dirty: true
        });
      }
    }

This work well for single dropdown; the error is mapped correctly. But when I change the value of any dropdown, I would like the other errors of these fields to react as well.
Is there any way I could let Vee-Validate know that multiple fields are binded to each other so that when one change, the other's error re-evaluate as well?

Thanks for your answer. Do you know when such enhancement would be implemented?
Also, I'm seeing that Vee-Validate will soon have a 2.1 stable version, did you planned a release date yet?
Thank you!

I don't have a planned release date unfortunately as the beta releases introduced a major re-write, and the amount of time is severely limited due to my army mandatory service. Now I have a few features in mind that I would like to address before the 2.1 is ready:
https://github.com/baianat/vee-validate/projects/3

closed in https://github.com/logaretm/vee-validate/commit/2dfcd97fecc67082fcaef0526dfb32ebda15e9b9

Is this available in some way yet? I can't seem to find anything in the docs.. or will this just be released in v3.0 is that it?
Thanks!

@kevrousso it will be available in v3, this cannot be implemented in v2.


fixed 1st Gen and typos

Docs Build status updates of commit 9d3a4f2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrtk-porting-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Bug] 1.1.1q retrieving randomness from PRNGD

I'm having some issues retrieving randomness from PRNGD in test_ecparams. The code works fine elsewhere, but in this test, just not. This has been doing on for a few patches - I cannot determine where as the environment has changed. PRNGD is supplied by the system, but I'm wondering whether OpenSSL is supplying its own for this test. This happens regardless of the build type and platform, so x64 and ia64 are involved.
An example of this failure is:
#   Failed test at ../test/recipes/15-test_ecparam.t line 29.
checking elliptic curve parameters: failed
0:error:2406C06E:random number generator:RAND_DRBG_instantiate:error retrieving entropy:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:335:
0:error:2406C06E:random number generator:RAND_DRBG_instantiate:error retrieving entropy:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:335:
0:error:2406B072:random number generator:RAND_DRBG_generate:in error state:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:588:
0:error:2406C06E:random number generator:RAND_DRBG_instantiate:error retrieving entropy:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:335:
0:error:2406C06E:random number generator:RAND_DRBG_instantiate:error retrieving entropy:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:335:
0:error:2406B072:random number generator:RAND_DRBG_generate:in error state:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:588:
0:error:2406C06E:random number generator:RAND_DRBG_instantiate:error retrieving entropy:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:335:
0:error:2406B072:random number generator:RAND_DRBG_generate:in error state:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/rand/drbg_lib.c:588:
0:error:10089003:elliptic curve routines:ec_GFp_simple_points_make_affine:BN lib:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-1.1_Pipeline/crypto/ec/ecp_smpl.c:1269:
../../util/shlib_wrap.sh ../../apps/openssl ecparam -noout -check -in ../recipes/15-test_ecparam_data/valid/secp224r1-named.pem => 1
not ok 66

I would appreciate some help in tracking this down.

One option I am considering is to down port the hardware randomizer I wrote for 3.0 back to 1.1.1, but that will not work in the ia64 platform

I don't think that that test does anything special, it's just running the openssl command line utility.
Is it picking up an odd configuration file (or not picking one up)?
It a stretch since other tests don't do anything special with configuration.
Does running it manually work?
./util/shlib_wrap.sh apps/openssl ecparam -noout -check -in test/recipes/15-test_ecparam_data/valid/secp224r1-named.pem

I don't think that that test does anything special, it's just running the openssl command line utility.
Is it picking up an odd configuration file (or not picking one up)? It a stretch since other tests don't do anything special with configuration. Does running it manually work?
./util/shlib_wrap.sh apps/openssl ecparam -noout -check -in test/recipes/15-test_ecparam_data/valid/secp224r1-named.pem

I got:
checking elliptic curve parameters: ok. So that looks ok.

Argh!  Another bit of craziness in the test harness.

Argh!  Another bit of craziness in the test harness.

Apparently, I am the canary in the coal mine: Randall, when the test breaks

Strangely, this does not work in isolated tested but works correctly when built in our Jenkins CI/CD system.


speech to text display : simple formatting

Right now the text is treated as one long sentence.  Can we use pause duration, and mark a sentence boundary at >1 sec pause?

I think we were doing that for the old demo. we were capitalize the sentence and put a .

Yeah, that was better.  Could we put that in the new demo?  May not need to do a line break for every sentence, but at least a '.' and capitalize.

Yes


Reduce a number of external dependencies

Do we really need ejplugins and pymatgen?

Given the code is PY3-only, why six is needed?

About ejplugins and pymatgen: these are also remnants of the old code base, now deleted.
About six: the code is both PY2 and Py3 compatible, please see Travis CI configuration

pymatgen is still in travis.yml

ejplugins is still in aiida_crystal/io/out.py and in aiida_crystal/parsers/migrate.py

pymatgen in travis.yml has been commented out since 01.03; however, string deleted. All ejplugins files are from the old code base, now also deleted


Replace nssm restart by powershell commands

Very frequenty nssm commands (stop and start) fail because the service
gets into STOP_PENDING or START_PENDING state.
powershell commandlets such as restart-service have the good taste to
wait for the service to stop/start completely.

I don't know too much about nssm to be honest. I did a little searching on the page and it seems like there are configurations where you can tweak some changes.
http://nssm.cc/usage

/cc @Ginja since he wrote the Windows service provider.

I haven't seen this too much in our environment.

As of version 2.22, nssm can apply a mandatory delay between restarts of the application. This could be used, for example, to run a command at regular intervals such as an hourly batch script.

To specify a restart delay, create an integer (REG_DWORD) value HKLM\System\CurrentControlSet\Services\servicename\Parameters\AppDelay and set it to the number of milliseconds to wait between restarts.

The service will report its state as Paused while waiting for the next restart. Sending it a Continue control will temporarily cancel the delay and trigger a restart immediately.

Please see the section on restart throttling below for notes on how nssm works when both throttling and restart delays are configured.

thanks @Ginja for you reply. As far as I understand the restart delay option it only deals with a delay between the service stop and the moment nssm tries to start it again.
In my case, a simple restart command send to nssm is causing the state.
It seems to be due to consul taking some time to stop (writing some information to disk) that might be long (a few seconds for a 2k node cluster) and nssm does not wait properly for process termination thus declaring the service as "stop pending".
Apparently powershell commandlets handle this correctly.

@kamaradclimber oh, thanks for the clarification. That makes sense, I don't manage that large of a fleet in one Consul data centre, which probably would explain why I never ran into this issue. I just tested using those cmdlets locally & it looks like it'll work fine with NSSM.
I'd say go ahead & merge this, @johnbellone.

thanks!


Missing User RoleBindings in the Project Access Web UI

Describe the bug

We've one RoleBinding for a project admin which includes two users.
Via the CLI we can see the two users if you describe the rolebinding.
"[root@amu1123 ~]# oc describe rolebinding gitops-user-binding
Name:         gitops-user-binding
Labels:       argocd.argoproj.io/instance=ns-dev-c2-0eu1-
Annotations:  Role:
Kind:       ClusterRole
Name:       admin
Subjects:
Kind  Name      Namespace
User  GRESS
User  HERRNB_F"
But in the Web UI only the first user is displayed. (Developer -> Project -> Project Access)

If you search for RoleBindings two RoleBindings will be displayed:

Also If you delete the RoleBinding from the Project Access Web UI the complete RoleBinding will be deleted not only the first User will be removed
Version

4.5.0-0.okd-2020-10-03-012432 - UPI
How reproducible

Just create a RoleBinding which contains two users.
apiVersion: authorization.openshift.io/v1
kind: RoleBinding
metadata:
  name: gitops-user-binding
roleRef:
  name: admin
subjects:
- kind: User
  name: GRESS
- kind: User
  name: HERRNB_F
userNames:
- GRESS
- HERRNB_F

Log bundle

Please file a bug for https://github.com/openshift/console


[FEATURE REQUEST] Add boot from USB/SSD support

Please add support for booting from USB. I flashed the 1TB SSD with the Umbrel-OS image but it fails to start properly and I receive the ERR_CONNECTION_REFUSED error in the browser.
On screen, I see the following service failing:
[FAILED] Failed to starte External Storage Mounter. See 'systemctl status umbrel-external-storage.service' for details. [DEPEND] Dependency failed for Umbrel Startup Service. [DEPEND] Dependency failed for Connection Details.
At the bottom the cursor is just blinking away at:
umbrel login:

@reinto  You can try if https://github.com/getumbrel/umbrel/suites/1809392088/artifacts/34767027 works.

@reinto  You can try if https://github.com/getumbrel/umbrel/suites/1809392088/artifacts/34767027 works.

Thanks, I will! I'll update here how that turns out.
Can I ask what the main motivation is to use different devices (SD card and SSD disk) as opposed to running all from a single disk? That is what it says on the pull request.
My motivation not to use an SD card is durability and speed, although perhaps those are not that important if the SD card only holds the OS and, for example, Page files and other high I/O stuff are turned off.

Thanks, I will! I'll update here how that turns out.
Can I ask what the main motivation is to use different devices (SD card and SSD disk) as opposed to running all from a single disk? That is what it says on the pull request.
My motivation not to use an SD card is durability and speed, although perhaps those are not that important if the SD card only holds the OS and, for example, Page files and other high I/O stuff are turned off.

The OS on the SD allows easily reflashing the OS in case anything goes wrong without loosing all data, because all data except the docker container images are on the SSD.

The OS on the SD allows easily reflashing the OS in case anything goes wrong without loosing all data, because all data except the docker container images are on the SSD.

 If it indeed saves your data and let's you re-flash the image to SD, that is a major win. Will it save both the synced full node and any open LN channels?
I can test the bits, if that would help. However, I will probably end up running the OS on an SD card for easy recovery now.

 If it indeed saves your data and let's you re-flash the image to SD, that is a major win. Will it save both the synced full node and any open LN channels?
I can test the bits, if that would help. However, I will probably end up running the OS on an SD card for easy recovery now.

Yes, everything of the data is on the SSD, only the OS is on the SD

Yes, everything of the data is on the SSD, only the OS is on the SD

The OS on the SD allows easily reflashing the OS in case anything goes wrong without loosing all data, because all data except the docker container images are on the SSD.

IMO this is not a good tradeoff considering exceptionally poor performance reported frequently by community and SD card corruption for so much as a power outage. The SSD can be partitioned, and the Umbrel could live in one of the partitions.

Would operate faster.
Would be cheaper to fix.
Would not take 1 hour to load the admin interface (yes, official PS and bench-tested USB3 cable)


Flaskik ПЕРЕВЕДИТЕ НА РУССКИЙ, Ю ХЕВ ТУ РАЙТ ОН РАШАН

fixes #

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

Hi, if you are interested in translating Flask docs, feel free to submit your request here.


Add tag for grip

Add tags for grip
I think dots is a good tags for grip, and drag and move is also good for grip-vertical.
How do you think so? It is OK, I will send a PR :)

@PeterlitsZo yess I agree with you. Don't hesitate to create a PR!

See #598 please.


Shift route highlight for other tools

On Priority Roads tool, holding Shift highlights the route that will be affected - it's great feedback for user as it simultaneously indicates that a route will be changed and also shows what that route is.
Can we get the same feature added to:

Speed Limits tool
Parking Restrictions tool

Maybe fade overlay (well, make more transparent) for any roads along route that don't have parking; still show the road as part of route, but faded overlay gives cue nothing will happen on that segment

Vehicle Restrictions tool

This will ensure great consistency across all tools in relation to Shift key visual feedback.
cc: @kianzarrin

Also, it might merit making a start on something akin to #630

Tagging relevant PRs where this was added: #708, #709 and #721


Added Iraq.

Hello, I added Iraq to the regions.yaml
Thanks.

Thank you @Colonel-Dixie ; I'll take care of the last steps in the next few days. 👍

🚀 Deployed: https://welcome.osm.be/asia/iraq


Trealla does not print a newline / suffix continued construct

GProlog
% [ /home/jacob/projects/logtalk3/examples/errors/errors_loader.lgt loaded ]
(14 ms) yes
| ?-

SWI
% [ /home/jacob/projects/logtalk3/examples/errors/errors_loader.lgt loaded ]
true.
?-

Trealla
% [ /home/jacob/projects/logtalk3/examples/errors/errors_loader.lgt loaded ]
true

... The suffix on Trealla does not print in the same manner as SWI or GProlog (prompt). I wonder if this is a Trealla version problem.

Note: this leads to awkward continuous script-calling & eventual shutdown.

Trealla leaves a choice-point. Possibly a predicate indexing issue.


Image captioning: num-words=0 turned to None, load features with variable length

(description of the change)
Pull Request Checklist

[x] Changes are complete (if posting work-in-progress code, prefix your pull request title with '[WIP]'
until you can check this box.
[x] Unit tests pass (pytest)
[x] Were system tests modified? If so did you run these at least 5 times to account for the variation across runs?
[x] System tests pass (pytest test/system)
[x] Passed code style checking (./style-check.sh)
[x] You have considered writing a test
[ ] Updated major/minor version in sockeye/__init__.py. Major version bump if this is a backwards incompatible change.
[ ] Updated CHANGELOG.md

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

Done


[Bug] Image does not rendered in proper way (GPU rendering)

Describe the Bug
In case if we are using CPU rendering it works fine, but in case GPU this image inverted.
Please see attached anonimized file
DICOM sample Issue with Invertion.zip
Steps to Reproduce
run GPU rendering

run CPU rendering

The current behavior
Image inverted and not rendered in proper way
The expected behavior
Image should be rendered in the correct way and the result should not depend on the rendering type.
OS
Windows 10, Mac os
Node version
v18.14.1
Browser
Chrome 120.0.6099.131

old cornerstone was rendering it correctly in and OHIF v2

@sedghi  any updates?

No not yet

Hi @sedghi , Our customer is facing this issue, Would your team please help to check that or any clue we can address?

@sedghi
could this issue related to function toLowHighRange(
  windowWidth: number,
  windowCenter: number
): {
  lower: number;
  upper: number;
} {
  const delta = 0.001; // small value to prevent zero width
  const width = windowWidth > delta ? windowWidth : delta;
  const lower = windowCenter - 0.5 - (width - 1) / 2;
  let upper = windowCenter - 0.5 + (width - 1) / 2;

  // upper and lower should be non equal (upper should be bigger than lower)
  if (upper - lower < delta) {
    upper = lower + delta;
  }
  return { lower, upper };
}

export { toWindowLevel, toLowHighRange };

Upper and lower should be non equal (upper should be bigger than lower).
and for this image we have
Window Width: 1
Window Center: -2
so as a result lower === upper === -2.5

The display is still not perfect with the change suggested by @Max-Kharitonov, but it is a lot better. The issue is that the vtk colorTransferFunction doesn't seem to correctly deal with the case when the window width is set to 1 (threshold value) or we are encountering a rounding issue.

@s2ramana I fixed some issue with window level recently can you check again?

@sedghi which PR was this changed made in?

@sedghi still reproducible

@sedghi could you please take a look?
https://github.com/cornerstonejs/cornerstone3D/pull/1135


Mlhades3 - Updated pull request for Hades update

Log of changes across commits:
6/16
-working on test script

change Op attribute behavior on bt and !bt
-add function that finds the concept id from a concept set expression

Between 6/16 - 7/1
-create unit tests for cohort components
-set up Hades structure
7/6/21
-Update connection details language
-create unit tests for attributes and user functions
-small edits to attribute functions

include addAttributeToQuery Function to Capr (from Phenotype Librarian)
add check concept field to get info from concept s4

7/7/21
-add final unit test for User facing functions (save, load, read)
-update README, NEWS and DESCRIPTION
-ready for first release
7/13/21
-add vignette for complex cohort design
-refresh .gitignore and delete .Rhistory

@mdlavallee92 -- please remove .Rhistory and add that file name to .gitignore


TypeError: Cannot read property 'destroyed' of null

<ListView class="list-group" [items]="albums"  >
     <ng-template let-album="item">
         <StackLayout (loaded)="loadComplete()" class="list-group-item">

        <WrapLayout *ngIf="album.images.length>0" width="100%"  class="list-group" >
               <WebImage  stretch="aspectFit"  backgroundColor="gray"
                   [src]="image.thumbnailUrl" *ngFor="let image of album.images"
                   width="20%" [height]="imageElement.getActualSize().width"  #imageElement></WebImage>
         </WrapLayout >
      </StackLayout>
    </ng-template>
</ListView>

if we use above code then app is starting without any errors and output is as expected.
but using below code i get the error that "ERROR TypeError: Cannot read property 'images' of null "
<ScrollView class="list-group"   >
     <StackLayout>
         <StackLayout *ngFor="let album of albums" (loaded)="loadComplete()" class="list-group-item">

           <WrapLayout *ngIf="album.images.length>0" width="100%"  class="list-group" >
               <WebImage  stretch="aspectFit"  backgroundColor="gray"
                   [src]="image.thumbnailUrl" *ngFor="let image of album.images"
                   width="20%" [height]="imageElement.getActualSize().width"  #imageElement></WebImage>
            </WrapLayout >
         </StackLayout>
    </StackLayout>
</ScrollView>

we can clearly state that both code are equivalent and should give similar output.
but last gives error "ERROR TypeError: Cannot read property 'images' of null"
Detailed error is like this:
JS: ERROR CONTEXT [object Object]
JS: ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'destroy
ed' of null
JS: TypeError: Cannot read property 'destroyed' of null
JS:     at ViewContainerRef_.move (file:///data/data/org.nativescript.bunkerzon/
files/app/tns_modules/@angular/core/bundles/core.umd.js:11506:20)
JS:     at file:///data/data/org.nativescript.bunkerzon/files/app/tns_modules/@a
ngular/common/bundles/common.umd.js:2652:38
JS:     at DefaultIterableDiffer.forEachOperation (file:///data/data/org.natives
cript.bunkerzon/files/app/tns_modules/@angular/core/bundles/core.umd.js:7451:17)

JS:     at NgForOf._applyChanges (file:///data/data/org.nativescript.bunkerzon/f
iles/app/tns_modules/@angular/common/bundles/common.umd.js:2641:17)
JS:     at NgForOf.ngDoCheck (file:///data/data/org.nativescript.bunkerzon/files
/app/tns_modules/@angular/common/bundles/common.umd.js:2627:22)
JS:     at checkAndUpdateDirectiveInline (file:///data/data/org.nativescript.bun
kerzon/files/app/tns_modules/@angular/core/bundles/core.umd.js:12410:19)
JS:     at checkAndUpdateNodeInline (file:///data/data/org.nativescript.bunkerzo
n/files/app/tns_modules/@angular/core/bundles/core.umd.js:13931:20)
JS:     at checkAndUpdateNode (file:///data/data/org.nativescript.bunkerzon/file
s/app/tns_modules/@angular/core/bundles/core.umd.js:13874:16)
JS:     at debugCheckAndUpdateNode (file:///data/data/org.nativescript.bunkerzon
/files/app/tns_modules/@angular/core/bundles/core.umd.js:14767:76)
JS:     at debugCheckDirectivesFn (file:///data/data/org.nativescript.bunkerzon/
files/app/tns_modules/@angular/core/bundles/core.umd.js:14708:13)
JS:     at Object.eval [as updateDirectives] (ng:///HomeModule/HomeComponent.ngf
actory.js:89:9)
JS:     at Object.debugUpdateDirectives [as updateDirectives] (file:///data/data
<EMAIL_ADDRESS>.js:14693:21)
JS:     at checkAndUpdateView (file:///data/data/org.nativescript.bunkerzon/file
s/app/tns_modules/@angular/core/bundles/core.umd.js:13840:14)
JS:     at callViewAction (file:///data/data/org.nativescript.bunkerzon/files/ap
p/tns_modules/@angular/core/bundles/core.umd.js:14191:21)
JS:     at execEmbeddedViewsAction (file:///data/data/org.nativescript.bunkerzon
/files/app/tns_modules/@angular/core/bundles/core.umd.js:14149:17)
JS: synct contact success [object Object]
JS: ERROR TypeError: Cannot read property 'images' of null
JS: ERROR CONTEXT [object Object]

i had also tried binding label text to function in which local album variable gets printed like below

conDir(a){         console.log(a.id)         console.log(a.name)         return a.name;     }
Looking at logs i can verify that "album" is not null and has correct value as it should have.
so i have no idea what can be wrong.

Hi, @bhavincb,
Could you provide sample project, which demonstrates the problem with the ngFor?
The issue might be related to #1206.

hii @tsonevn ,
At the moment i'm unable to provide sample project because problem we are facing is inside very large project so separating it will take some time.
but as i said i had implemented 2 UI html which should give same output, but second one which is implemented using "ScrollView" and "*ngFor" gives the error i had mentioned above, while first implementation using "ListView" and "ng-template" works perfectly fine.
There is not a single changes made to TS files only HTML files changed , so i can say that "albums" object generated remains same in both cases, and i also checked same by debugging app in chrome dev tools.

hi @tsonevn ,
As you had mentioned that there is some issue with *ngIf while using conditions , so i had replaced *ngIf with "visibility", and also tried removing *ngIf but still it is not working and giving same error.

Hi, @bhavincb,
I notice that you are using nativescript-web-image-cache plugin in your application.
Regarding that, could you replace WebImage with the Image component provided by NativeScript core modules and check if the issue is related to the plugin's component. After replacing the WebImage, remove the platform with tns platform remove <platform name> and rebuild the app tns run <platoform name>.

hii @tsonevn ,
there were no issue with WebImage. i had found problem and solved it.
i'm also adding full information if anyone may need it in future.
<ListView class="list-group" [items]="albumObj.albums"  >
     <ng-template let-album="item">
         <StackLayout (loaded)="loadComplete()" class="list-group-item">

        <WrapLayout *ngIf="album.images.length>0" width="100%"  class="list-group" >
               <WebImage  stretch="aspectFit"  backgroundColor="gray"
                   [src]="image.thumbnailUrl" *ngFor="let image of album.images"
                   width="20%" [height]="imageElement.getActualSize().width"  #imageElement></WebImage>
         </WrapLayout >
      </StackLayout>
    </ng-template>
</ListView>

in above code albumObj is shared service which has "albums" named array.  this service initially fetch previous Albums from the local database. and also request fro new albums data and updates it.
i had getting error i had mentioned in previous comment.
loadComplete(){
    this.cd.detectChanges();
}

above function was real culprit. if i remove the this.cd.detectChanges() code runs without any error.
but doing changeDetection  on StackLayouts loaded event and running code i was getting above error and in html file "album" object becomes null.
but Note that while logging checking "albumObj.albums" on 100milisecond time interval i can see that it has perfect value and all albums has images property.
So in conclusion i can say that when using this.cd.detectChanges() in StackLayouts loaded event it destroys objects value in html view. but works perfectly in TS file.
also i really think there is something wrong "loaded" event or "change Detection Ref" please look into this if possible.

We're having the same issue.  We're using a directive to add some attributes to a TextField on Android, no change detection at all. Even just binding a (loaded)="doesNothing()" is enough to trigger this error.
If we use a single Label inside the for and set (loaded)=printThis(item) we don't get a crash, but we do get an error accessing "item" and the function prints "null". Immediately after it prints the correct object, as if the Label was loaded twice.
I can set up a playground if needed.

Hi @edusperoni,
Can you provide a sample project and more info about your environment(CLI, tns-core-modules, nativescript-angular, runtime versions)?

@tsonevn Here you go:
https://play.nativescript.org/?template=play-ng&id=hpl6OJ

HI @edusperoni,
If you are facing a problem with RadListView, please log a new issue in nativescript-ui-feedback repository.

I am also having same problem with ngx-pagination, while there is data available it works fine, if there is no data(null) then its replicating pagination page numbers.

I'd like to provide more info about this issue, in hopes it gets into the next release. As you may have seen, I've commented on https://github.com/NativeScript/nativescript-angular/issues/848#issuecomment-420673404 saying it's basically impossible to get the android view as it loads due to both these bugs.
After some testing, I've arrived to the conclusion it's the angular event binding specifically that causes this bug. I've updated my example to exemplify it:
https://play.nativescript.org/?template=play-ng&id=hpl6OJ&v=4
If you use (loaded) and (unloaded), you get:
ERROR TypeError: Cannot read property 'item' of null
and ultimately a crash:
TypeError: Cannot read property 'destroyed' of null
but if you use my directive which adds safeLoaded and safeUnloaded. You get no error at all, and that's done with this.el.nativeElement.on("loaded",...).
If you start listening to the event on the directive constructor (set ensureItemExists = false), before the element is loaded, you still get
LOADED! null
But no ERROR TypeError: Cannot read property 'item' of null.
If you do it in ngAfterViewInit (set ensureItemExists = true), you sometimes have to manually trigger a loaded event in case the element hasn't loaded (this happens sometimes in android, as seen in #848), but once that is done, you get the item AND no crash/errors.
Maybe this is enough to shed some light on the issue?

Hi all,
With the latest NativeScript version, we introduced a new event layoutChanged. Can you check whether you will have the same problem while using layoutChanged instead of loaded?

I can confirm layoutChanged doesn't trigger the error. So far, it only seems to happen on loaded and unloaded.
Added layoutchanged to example:
https://play.nativescript.org/?template=play-ng&id=hpl6OJ&v=5

Ok, I found why this error occurs. Wrapping the callback in NgZone.run breaks the code.
See: https://play.nativescript.org/?template=play-ng&id=hpl6OJ&v=6
            this.el.nativeElement.on("loaded", this.getZonedCb((evt: EventData) => {
                this.safeLoaded.emit(evt);
            }));

    getZonedCb(callback: any) {
        return (...args) => {
            this.zone.run(() => {
                callback.apply(undefined, args);
            });
        };
    }

this is the same logic being used in nativescript-angular renderer https://github.com/NativeScript/nativescript-angular/blob/309ed7b0de824a869b1b1c07b4a25928f7a754e1/nativescript-angular/renderer.ts#L296
It seems ngzone tries to check if the element is destroyed, but at the time of loading and unloading, it's null, so it throws the error.

After over 1 year of thinking about this on and off I FINALLY found the solution, and the problem is way deeper than I originally thought.
The problem is specifically with onItemLoading.

onItemLoading calls view.detectChanges() inside a Zone, but NOT THE ANGULAR ZONE
ngFor starts detecting changes
ngFor creates/deletes views
view call loaded/unloaded
renderer enters the angular zone and calls the callbacks
renderer leaves the zone
zone checks if it's stable (nothing else running in the angular zone). Triggers change detection again. Remeber, view.detectChanges() is running in a zone, but not in the angular zone
second change detection running simultaneously with the first
ngFor (inside ngzone) starts checking it's changes again, finishes without a problem
ngFor (outside ngzone) finishes running it's change detection, but it's references are dirty, so it throws errors.

This happens for 2 reasons:

Nativescript events are never delayed, which means loaded and others will always trigger when they happened, even if there are other tasks running. This is different from the web event loop, where the event callbacks are queued

1.1. This means you can intercept events quickly and act upon them, but they WILL impact applications in unforeseeable ways.
1.2. If you implement a callback queue, like I did in my testing, you'll miss events like unloaded because the element will have been destroyed by then, as it happens DURING the destroy. This resulted in my (unloaded) never firing as it waited for the whole destruction process to fire, and by then the subscription had been closed.
Some angular calls are not running in the angular zone. When they eventually enter and leave the ngzone, it'll trigger CD during CD, causing all sorts of issues.

This discovery raises some issues:

all core polyfills (setTimeout, etc) are using Zone, but I'm not sure they're running on the Angular zone. I believe most of them are, but my newest addition requestAnimationFrame has not been patched in zone-nativescript https://github.com/NativeScript/nativescript-angular/blob/master/nativescript-angular/zone-js/dist/zone-nativescript.js#L1676
there are other places in nativescript-angular where detectChanges or markForCheck are being called outside the ngzone. That I could find: dialogs, DetachedLoader, and templated-items-comp.

I'll write a PR wrapping all detectChanges in the ngzone for the time being, but the other points must be considered.

@edusperoni - That is awesome debugging!
Are you sure about that it is running in a zone.    The Zone code to my knowledge is ONLY for Angular; so if it is already running in a Zone, then it should be a Angular zone.   Zones were not part of NativeScript until Angular was added.   And I personally felt it polluted the core modules to add the extra code that did nothing for NSCore.   So it would seem very strange that it is in a zone, but the zone didn't do anything...     You might want to verify it is actually in a zone, and if it is angular related and why it isn't doing what it is supposed to be...   Maybe their is a bug actually in the zone code...

@NathanaelA While writing the PR and tests, the problem seems to be with nativescript-zone.
If you trigger fixture.detectChanges() or fixture.autoDetectChanges(true), the problem does not happen, but if you trigger fixture.autoDetectChanges(true) and add a setTimeout you'll get the following error:
TypeError: Cannot read property 'id' of null--Pendng async tasks are: [type: macroTask, source: setTimeout, args: {handleId:37,isPeriodic:false,delay:1000,args:[object Arguments]},type: macroTask, source: setTimeout, args: {handleId:39,isPeriodic:false,delay:0,args:[object Arguments]}]

Angular has an "NgZone" which contains an inner zone (actual angular zone) and outer zone (code running without angular "knowning").  The code from onItemLoading is called on the <root> zone (since it's being bound by on and not by (event)=...). The zones in NS core seem to be doing:
const zone = Zone.current;
zone.run(callback);

which means if a request came from the angular zone, Zone.current.name is angular, so it'll trigger CD when it's finished.
Now for some reason, this bug only happens when you have other tasks queued up (which is often the case in real apps), so it can't be reproduced in unit testing, which probably means the issue is not with running the code inside the angular zone, but something with zone itself. My solution of running it inside the ngzone is actually more of a workaround.
On all cases, onItemLoading was called in zone <root>.


Explicit Python version for port_server

In the same way that it's specified in the shebang line.
This matters in cases where the default Python version isn't a 2.x one (ie, Arch Linux).

LGTM


Shapeless is failing to extract (starting in July 2016) — withLogRecompileOnMacro related

last good runs were Jul 9, first bad runs Jul 16. both JDK 6 and 8 affected.
JDK 6:

last success (Jul 9): https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/275/
first failure (Jul 16): https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/276/

[shapeless] [info] Compiling 1 Scala source to /home/jenkins/workspace/scala-2.11.x-jdk8-integrate-community-build/target-0.9.5/extraction/d95196dffef13b2b973c2560c0fa3e88ff981e4d/projects/42bf871dd7737f169d47f1cd19ee6de251e7c9df/project/target/scala-2.10/sbt-0.13/classes...
[shapeless:error] /home/jenkins/workspace/scala-2.11.x-jdk8-integrate-community-build/target-0.9.5/extraction/d95196dffef13b2b973c2560c0fa3e88ff981e4d/projects/42bf871dd7737f169d47f1cd19ee6de251e7c9df/build.sbt:40: error: value withLogRecompileOnMacro is not a member of sbt.inc.IncOptions
[shapeless:error]   incOptions := incOptions.value.withLogRecompileOnMacro(false),
[shapeless:error]                                  ^
[shapeless] sbt.compiler.EvalException: Type error in expression
[shapeless] [error] sbt.compiler.EvalException: Type error in expression
...
[info] >>> The dbuild result is------------: EXTRACTION FAILED (failed: shapeless)

looks like the culprit is milessabin/shapeless@78aa3542810ed3b62a51463dc68192b83c2e90d3 and simply bumping our own sbt version to 0.13.12 as well should fix it

test run with sbt version bump: https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/281/consoleFull

extraction succeeded, but let's wait for the whole run, to make sure nothing regressed elsewhere

all good, only failure was because some project's JavaScript dependency moved to a different URL


com.mysql.jdbc.exceptions.jdbc4.MySQLNonTransientConnectionException: No operations allowed after statement closed.

Hi, great team!
The error stack is as follows

The HikariCP configuration information is as follows

The mysql configuration is as follows

Version information
HikariCP 4.0.3
mysql-connector-java 5.1.29
Java 8
mysql 5.7
Where's the problem? Thanks！

This is a problem with running at 2 a.m！

@brettwooldridge

I can't tell you where the problem in you code is. This error only occurs when code has already called close() on the Connection object. I don't have a good recommendation for how to trace that. It may have to do with Hibernate session management. I recommend asking on Stackoverflow with the tag #hibernate.


fix: Issue #713

Avoid removing userlicence when running reconcile with source only flag
Checklist

[ ] Adhere to Contribution Guidelines: Yes
[ ] Updates to documentation at DX@Scale Guide required? No
[ ] Tested changes? Yes

Thanks @genoud as always!


examples added and updated

Summary
slightly changed some examples and added new ones

[x] I have tested my changes.

[ ] I have run task pyright and fixed the relevant issues.

[ ] I have updated the documentation to reflect the changes.

Avoid Intents.all as much possible (I don't see why the old code was changed?).
PR overall seems unnecessary


Remove confusing UIActivityIndicatorView.animating extended property

https://github.com/RACCommunity/Rex/pull/134#discussion_r69709244

I don't find the previous one confusing, but I prefer this one, since it doesn't leak implementation details.

My thought:

Having both animating and isAnimating() seems confusing
The latter might be renamed as isAnimating in Swift 3, that makes the thing more complicated


Investigate how node-uuid is so fast

See: https://github.com/broofa/node-uuid/blob/master/benchmark/benchmark.js
Try it:
$ git clone https://github.com/broofa/node-uuid.git && cd node-uuid
$ npm install uuid uuid-js
$ node benchmark/benchmark.js

#### v4
nodeuuid.v4() - using node.js crypto RNG: 2840909 uuids/second
nodeuuid.v4() - using Math.random() RNG: 2747252 uuids/second
nodeuuid.v4('binary'): 2293577 uuids/second
nodeuuid.v4('binary', buffer): 2369668 uuids/second
uuid(): 171585 uuids/second
uuid('binary'): 162022 uuids/second
uuidjs.create(4): 433651 uuids/second
140byte.es_v4: 266951 uuids/second

#### v1
nodeuuid.v1(): 988142 uuids/second
nodeuuid.v1('binary'): 563063 uuids/second
nodeuuid.v1('binary', buffer): 851788 uuids/second
uuidjs.create(1): 320718 uuids/second

Reading the history of the module is a good lesson in improving performance over time.
https://github.com/broofa/node-uuid/commits/master/uuid.js
I highly recommend it to anyone curious about high performance (but terse) JS.

The most interesting thing I learned was how to use Bitwise Operators >>> and <<  to shift bits.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Bitwise_Operators


Why doesn’t JVMMetricsSender notify the status as DISCONNECT when sending a network error?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion

Question

What do you want to know?

Why doesn’t JVMMetricsSender notify the status as DISCONNECT when sending a network error?
Is it by design or is it missed? I noticed that if there is a network error, TraceSegmentServiceClient/MeterSender/ServiceManagementClient will notify DISCONNECT.
https://github.com/apache/skywalking/blob/871fdf2fa251e3e2c3082a2373e4ead6c4aec8fd/apm-sniffer/apm-agent-core/src/main/java/org/apache/skywalking/apm/agent/core/jvm/JVMMetricsSender.java#L70-L89

What do you mean? I can see JVMMetricsSender#statusChanged exists. What do I miss?

I mean, why not add reportError in line 86, like this:
 catch (Throwable t) {
    LOGGER.error(t, "send JVM metrics to Collector fail.");
    ServiceManager.INSTANCE.findService(GRPCChannelManager.class).reportError(t);
}

I mean, why not add reportError in line 86, like this:

 catch (Throwable t) {

    LOGGER.error(t, "send JVM metrics to Collector fail.");

    ServiceManager.INSTANCE.findService(GRPCChannelManager.class).reportError(t);

}

I think it's missed, will you open a pull request to add it?


yang.settings file should be associated as JSON by default

Currently manual association has to be added globally or per project/workspace .vscode/settings.json. This could be avoided by the plugin doing it instead of the developer.

Since the YANG LSP yang.settings supports // comments, it would be good to classify it as "JSON with Comments" to allow for documentation of the settings

Merged


Video Duration Limit of Video Recording Ignored on Android 11 with Image Picker Plugin

Steps to reproduce

Initialize the Image Picker plugin in a Flutter project.
Set the maxDuration parameter for video recording to 15 seconds.
Deploy the app on an Android 11 device.
Start recording a video using the Image Picker.
Observe the behavior when the recording reaches 15 seconds.

Expected results
The video recording should automatically stop at 15 seconds, as specified by the maxDuration parameter.
Actual results
The video recording continues indefinitely beyond the 15-second limit on Android 11 devices, not respecting the maxDuration parameter. This issue is not present on iOS devices and Android 13, where recording stops correctly at 15 seconds.
Code sample
final pickedFile = await ImagePicker().pickVideo(
source: ImageSource.camera, maxDuration:  Duration(seconds: 14));
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.5 23F79 darwin-arm64, locale en-GB)
• Flutter version 3.22.2 on channel stable at /Users/wakassharif/Developer/flutter310
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 761747bfc5 (9 weeks ago), 2024-06-05 22:15:13 +0200
• Engine revision edd8546116
• Dart version 3.4.3
• DevTools version 2.34.3
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /Users/wakassharif/Library/Android/sdk
• Platform android-34, build-tools 34.0.0
• ANDROID_HOME = /Users/wakassharif/Library/Android/sdk
• Java binary at: /Library/Java/JavaVirtualMachines/jdk-17.0.2.jdk/Contents/Home/bin/java
• Java version Java(TM) SE Runtime Environment (build 17.0.2+8-LTS-86)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15F31d
• CocoaPods version 1.13.0
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
[✓] Android Studio (version 2022.2)
• Android Studio at /Applications/Android Studio 2.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
[!] Android Studio (version unknown)
• Android Studio at /Users/wakassharif/Library/Android/sdk/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
✗ Unable to determine Android Studio version.
✗ android-studio-dir = /Users/wakassharif/Library/Android/sdk
✗ Android Studio not found at /Users/wakassharif/Library/Android/sdk/Contents
• Try updating or re-installing Android Studio.
• Consider removing your android-studio-dir setting by running:
flutter config --android-studio-dir=
[✓] VS Code (version 1.92.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.94.0

@wsharif92 Even though I don't have physical device with Android 11 version, but I tried by creating emulator of same configuration, ran image_picker plugin example which has set duration of 10 seconds. I selected ImageSource.camera with which the video recording lasted for set duration as expected. See below which shows the video I recorded which stopped at 10 seconds.
https://github.com/user-attachments/assets/ed7c3fcb-d328-43f8-9764-f130efe5d61f
Can you check if there's any device setting that could be leading to the behavior since you said it only occurs in Android 11 ?
There's a similar issue open but for ImageSource.gallery : https://github.com/flutter/flutter/issues/83630

Also, what is the exact device that you are testing with? The behavior of the video picker intent handler (including whether it honors this option) is entirely controlled by the application handling the intent. So for instance, a device with a non-standard OEM picker intent handler may not choose to support the limit.


add imagePullSecrets "dockerhub-dockerconfig" to SAs

As part of the authenticated pull from DockerHub, we'll be setting the  "dockerhub-dockerconfig" imagePullSecret as default for 'bitnami.libsonnet' library.
Any project using this library can assign this imagePullSecret to a ServiceAccount by using
nginx_ingress_service_account: bitnami.ServiceAccount("nginx-ingress")
This snippet will be added to the SA:
   "imagePullSecrets": {
      "name": "dockerhub-dockerconfig"
   },

The secret will be only available on those clusters who make use of ClusterSecrets, which replicates the source secret to desired namespaces in the cluster.
KOPS managed clusters doesn't need any imagePullSecret definition since use a Docker config in the nodes, so the pull is authenticated out of K8s

I think this it's not needed anymore if we include imagePullSecret-patcher or any other software.
Putting it in the freezer


fix: Get port from Scheme

closes #54

What I did:
Why I did it:

Codecov Report

Merging #55 into master will decrease coverage by 1.65%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   62.70%   61.05%   -1.66%
==========================================
  Files           3        3
  Lines         185      190       +5
==========================================
  Hits          116      116
- Misses         44       49       +5
  Partials       25       25

Impacted Files
Coverage Δ

cache/cache.go
51.42% <0.00%> (-1.91%)
: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 6c123be...b924f76. Read the comment docs.


About the traning code.

Thanks a lot for your great work!
When shall you make the training code available?

Thanks a lot for your great work!
When shall you make the training code available?

Thank you for your recognition of our work! We are currently conducting further exploration based on our current efforts, so we have decided to open source the code and training weights at a later time.


[DO NOT SUBMIT] Java regen: oneof -> inheritance (Loggin only)

Please only look at the second commit (the one that matches the title of the PR)
To demonstrate https://github.com/googleapis/protoc-java-resource-names-plugin/pull/25

Updated to move factory methods into the Any* classes, PTAL


Add parse() to exported module

Overview
This PR adds parse() to the exported module so that programmatic usage of renv can benefit from the same git project auto detection as offered in the CLI API.
TODO before wrapping up:

[x] add documentation to README

/cc @archlichking @chaseadamsio

👍


Axes Are Weapons Allowing Wrong Enchants

Hello!
Using AxesAreWeapons-1.5.1-fabric [1.17 and 1.18], on Fabric for 1.18.1, the mod allows armor enchants and crossbow/bow enchants to be applied to axes in survival.
I am unsure what these two config settings are:
**	"enableForEnchantingTable": true,
"enableDamageInEnchantingTable": true,
**
Would this cause it?

No, that would definitely not cause it, so let us see what's going wrong.
First of all, are you sure you are actually in survival mode? Because you can apply any enchantment to any item when in creative mode.
If that's not it, I'll need a lot more info:

Does the issue happen without Axes Are Weapons?
Is this about the Anvil or the Enchanting Table?
Which other mods do you have installed? (Especially mods touching Enchantments/Anvil/Enchanting Table are relevant.)
Which enchantments specifically are we talking? Are they modded?
Is the axe modded?

Also, to understand those settings better, it's best to look at them in Mod Menu:

enableForEnchantingTable is "Enable the above in the Enchanting Table too", so in this case, Looting would appear for axes in the Enchanting Table as well.
enableDamageInEnchantingTable is "Enable Damage Enchantments for Axes in the Enchanting Table", which adds Sharpness, Smite and Bane of Arthropohs for axes to the Enchanting Table.

It was some wonky interaction with a trident enchant datapack that got left in whenever I would recreate worlds to test my mods (apologies).


Incorrect plan when there are several targets per domain name

It is possible for a source to return several endpoints with the same DNSName. For example, services may have more than one load balancer ingress records. However, a Plan computation logic
(the code that compares records returned by provider/registry and source) is going to be wrong for such case.
For each Source record, Plan computation searches for the first match in the list of records, returned from a registry by comparing DNSName alone. On the first round, all records for the domain name will be added because there is no match yet. However, on the second round, both records returned from the Source are going to be compared with the first of one in provider's list. As a result one of them is going to land in the Changed collection with incorrect UpdateOld record even though no changes were made. If we add 3rd endpoint later it will land in the Changed list as well.
Similar problem is going to happen if Source would return several endpoint of different types for the domain name. In this case the situation is going to be even worse because not only a Plan computation doesn't take record type into account, but also forcibly overwrites source record type with that found in provider even though they are (almost) unrelated.

I see that there is a PR #243 that is supposed to fix support for multi targets. My idea was instead of changing the Endpoint structure to contain list of targets to leave it as is but instead make a plan computation smarter. Grouping several targets in a single endpoint doesn't solve the problem when there are targets with different type. From what I see in the code, endpoints are considered to be unique by (DNSName, Target) pair elsewhere. Also current code already can produce several records with the same DNSName. So for me, changing plan computation algorithm seems less radical than changing the main data structure and all the providers that produce it. However, I may not see the big picture.
Are there any decisions made already? Can anybody comment on this before I submit my PR? Just don't want to have two competing PRs.

@istalker2 thanks for the thoughtful and detailed issue.
This PR #243 is orthogonal to the issue u are raising. The problem can occur if you simply created two services trying to acquire same dns names (but apparently different load balancers). This is something we should handle one way or another. Let me create a small PR with the test which will make the plan fail and I will ping you there.

The discussion is in #260, proposed fix in #261


WARNING: Too many active WebGL contexts. Oldest context will be lost.

Excuse me
First of all, feel the author with such a good library. First time use have a few problems, problems to solve
In single-page applications using pixi. Js.  Now use the MovieClip and PIXI. Elves animated Sprite.
A canvas corresponds to a PIXI object.  Due to performance issues.  I take multiple canvas element. Shared a requestAnimation.
The following code：
maskSprite = new PIXI.Sprite(this.maskTexture[0]);
stage1.addChild(this.maskSprite);
addQueue(function() {
    renderer.render(stage1);
})

movie = new PIXI.extras.MovieClip(textures);
stage2.addChild(movie.maskSprite);
addQueue(function() {
   renderer.render(stage2);
})

addQueue = function(fn) {
    list.pust(fn)
    requestAnimation(list)
}

Running is normal, But page will prompt error a few times,
WARNING: Too many active WebGL contexts. The Oldest context will be lost.
I don't know if I destroy mode has a problem.I only in the renderer method invoked on the destruction.
The following code：
renderer.destroy(true)
The problem:

WARNING, because the wrong to destroy?

If I didn't describe clearly, I send the source code
sprite.js
https://github.com/JsAaron/es6-magazine/blob/master/src/lib/component/pixi/sprite.js
queue.js
https://github.com/JsAaron/es6-magazine/blob/master/src/lib/component/pixi/core/queue.js

Duplicate of #2233?

hello! this has been addressed in v4 (latest dev branch)
We are now destroying the context with the WEBGL_lose_context extension if it is available.


Client app resets the frame when frame index is 0

Fixes #50

Looks good. May be worth a test that uses subscribe_to_frames as well as subscribe_to_all_frames, I don't expect any difference for the client since they use the same frame merging method, but I could imagining it catching problems where the server has a bug in aggregating frames or something.

Looks good. May be worth a test that uses subscribe_to_frames as well as subscribe_to_all_frames, I don't expect any difference for the client since they use the same frame merging method, but I could imagining it catching problems where the server has a bug in aggregating frames or something.

Done :)


Revert "Merge pull request #246 from artichoke/lopopolo/remove-versio…

…n-sync"
This reverts commit ad03e44081640d743d6e5b89b1c6945929049520, reversing changes made to 8c5422d3ee78bf9aef15f8fcd1570772f1b3d4a6.
Cargo does the right thing with the lower version constriant.

Reverts #246.


Oracle InstantClient

After installing (and using) Oracle InstantCliente 11.2 cannot run bash.
First time runs OK but after using SQLPLUS cannot run bash again generating Error: 0x80070005

This is the output of my console:

Microsoft Windows [Version 10.0.14393]
(c) 2016 Microsoft Corporation. All rights reserved.

C:\Users\winuser>bash
-- Beta feature --
This will install Ubuntu on Windows, distributed by Canonical
and licensed under its terms available here:
https://aka.ms/uowterms

Type "y" to continue: y
Downloading from the Windows Store... 100%
Extracting filesystem, this will take a few minutes...
Please create a default UNIX user account. The username does not need to match your Windows username.
For more information visit: https://aka.ms/wslusers
Enter new UNIX username: user
Enter new UNIX password:
Retype new UNIX password:
passwd: password updated successfully
Installation successful!
The environment will start momentarily...
Documentation is available at:  https://aka.ms/wsldocs
user@WINHOST:/mnt/c/Users/winuser$ sudo mkdir -p /opt/oracle
[sudo] password for user:
user@WINHOST:/mnt/c/Users/winuser$ cd /opt/oracle/
user@WINHOST:/opt/oracle$ sudo su -
root@WINHOST:~# apt-get install zip libaio1
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages were automatically installed and are no longer required:
  libfreetype6 os-prober
Use 'apt-get autoremove' to remove them.
The following extra packages will be installed:
  unzip
The following NEW packages will be installed:
  libaio1 unzip zip
0 upgraded, 3 newly installed, 0 to remove and 43 not upgraded.
Need to get 425 kB of archives.
After this operation, 1052 kB of additional disk space will be used.
Do you want to continue? [Y/n] yes
Get:1 http://archive.ubuntu.com/ubuntu/ trusty/main libaio1 amd64 0.3.109-4 [6364 B]
Get:2 http://archive.ubuntu.com/ubuntu/ trusty-updates/main unzip amd64 6.0-9ubuntu1.5 [157 kB]
Get:3 http://archive.ubuntu.com/ubuntu/ trusty/main zip amd64 3.0-8 [262 kB]
Fetched 425 kB in 0s (770 kB/s)
Selecting previously unselected package libaio1:amd64.
(Reading database ... 25022 files and directories currently installed.)
Preparing to unpack .../libaio1_0.3.109-4_amd64.deb ...
Unpacking libaio1:amd64 (0.3.109-4) ...
Selecting previously unselected package unzip.
Preparing to unpack .../unzip_6.0-9ubuntu1.5_amd64.deb ...
Unpacking unzip (6.0-9ubuntu1.5) ...
Selecting previously unselected package zip.
Preparing to unpack .../archives/zip_3.0-8_amd64.deb ...
Unpacking zip (3.0-8) ...
Processing triggers for man-db (<IP_ADDRESS>-1ubuntu1) ...
Processing triggers for mime-support (3.54ubuntu1.1) ...
Setting up libaio1:amd64 (0.3.109-4) ...
Setting up unzip (6.0-9ubuntu1.5) ...
Setting up zip (3.0-8) ...
  All rights reserved.

Enter user-name: orauser/password@//oracletest:1521/ORCL

Conectado a:
Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
With the Partitioning, OLAP, Data Mining and Real Application Testing options

SQL> select banner from v$version;

BANNER
--------------------------------------------------------------------------------
Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
PL/SQL Release <IP_ADDRESS>.0 - Production
CORE    <IP_ADDRESS>.0      Production
TNS for Linux: Version <IP_ADDRESS>.0 - Production
NLSRTL Version <IP_ADDRESS>.0 - Production

SQL> exit
Desconectado de Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
With the Partitioning, OLAP, Data Mining and Real Application Testing options
user@WINHOST:~$ exit
exit

C:\Users\winuser>bash
Error: 0x80070005

C:\Users\winuser>

Windows 10 Creators Update 1505x+ is ok

Thanks for the feedback, marking as fixed for insider builds per @kxjhlele.

Windows 10 Creators Update 15063 not ok

Thanks @kxjhlele, are you saying that the behavior regressed?  If so could you please provide the error message and a strace?

run:
apt-get install execstack
execstack -c instantclient_12_1/libclntsh.so.12.1
exit bash:
C:\Users\winuser>bash
Error: 0x80070005

Thanks @kxjhlele, sorry for the delay.  Can you provide the full repro steps so we can try this out locally?  Also, could you please clarify why you are running execstack on libclntsh.so.12.1?

Windows 10 Creators Update 15063.502 not ok and I verified this is still happening. I tried installing 11gR2 binaries without executing the execstack step and got the same issue. Bash for Windows no longer loads at this point. Where and how do you troubleshoot this?

This one was lost in the noise. It isn't clear what was fixed in 1505x+ (comment), or if it was fixed, or if it was fixed and regressed. 15063 and 16299 both shipped with fixedinsiderbuilds status.
@jacobrillema if you (or anyone else) is still seeing an error in an Oracle InstantClient scenario, please open a fresh issue following CONTRIBUTING.md (CLI repro steps from clean Ubuntu install and strace) and we can reboot this. Closing this one as needs-repro.


Minor code cleanup and documentation updates

Checklist

[x] I have read the CONTRIBUTING doc.
[x] I have added/updated unit tests to cover my changes.
[x] Unit tests pass locally.
[x] I have added appropriate documentation for my changes.
[x] I have updated the README to reflect any changes that will affect the component's public API (if applicable).

Problem

Some of the ways the component's code had been structured (ie, where constants and util functions live) is a bit weird and inconsistent
The documentation isn't as easy to navigate as it should be when browsing on Github or npm
Some documentation has gotten stale and needs to be updated

Solution

Adds components.js file that is dedicated for component constants, moves getVideoState util function into the utils.js file since it seems like it belonged there in the first place.
Adds a table of contents to the README and updates the demo site to extract that and use it for the navbar's contents rather than manually constructing everything
Minor documentation cleanup/updates

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix anonymous class violation file name prepending

PHPMD version: 2.9.1
PHP Version: 7.4.10
Installation type: composer
Operating System / Distribution & Version: Alpine Linux in Docker

Next step for #829
Current Behavior
:9 Violation

Expected Behavior
FileName.php:9 Violation

Steps To Reproduce:
Run cleancode rule:
<?php
class TestCase
{
    public function getAnonymousClass()
    {
        return new class {
            public function get(): string
            {
                return '';
            }
        };
    }
}

Checks before submitting

[x] Be sure that there isn't already an issue about this. See: Issues list
[x] Be sure that there isn't already a pull request about this. See: Pull requests
[x] I have added every step to reproduce the bug.
[x] If possible I added relevant code examples.
[x] This issue is about 1 bug and nothing more.
[x] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

@kylekatarnls is this already fixed in master? I tried to reproduce it but did get the filename in the error. Else I miss something and got the wrong error.


How to disable SSL certificate validation?

I have a WebDAV server on a local network, to access it over TLS/SSL I have to set up a certificate with a dummy CA, which is quite annoying.
Can I disable the SSL certificate validation? (Yes I know that this way I'd be losing the trusted CA)
I've found a solution for an HttpClient here, but I'm not sure if/how to employ it with WebDavClient

Hi @lmattcris,
You can pass an instance of HttpClient when creating WebDavClient. That should do the trick.

@skazantsev That sounds fine, however, I have to admit I have no idea how to pass on the credentials that previously I entered with WebDavClientParams.Credentials ... I mean that otherwise I just filled in a WebDavClientParams object and pass it to the WebDavClient constructor, as in base examples

Oh well, I made it, no matter how dumb I am 😄
I just changed the code to this
            var httpClientHandler = new HttpClientHandler();
            httpClientHandler.ServerCertificateCustomValidationCallback = ( message, cert, chain, sslPolicyErrors ) =>
            {
                return true;
            };
            httpClientHandler.Credentials = new NetworkCredential( "user", "password" );
            HttpClient httpClient = new HttpClient( httpClientHandler ) { BaseAddress = new Uri( server ) };
            WebDavClient client = new WebDavClient( httpClient );

And that worked, even removing the dummy CA, so without a trusted CA. Great!
Still: this is a restricted use, on a local network, possibly not even connected to the internet. I understand that using it outside a controlled environment isn't a good idea!


[PR][CLOSED] Added jenkins scripts to archive script codes of CI server

 Issue by geunsik-lim
Wednesday Jun 27, 2018 at 02:39 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/63

This PR is to archive the scripts that have run in the aaci.mooo.com
server. The scripts are written by prefix "taos-platform-".

Jenkins: http://aaci.mooo.com:8080/

Changes proposed in this PR:

Added jenkins scripts to maintain CI server

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
geunsik-lim included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/63/commits

:octocat: cibot: Thank you for posting issue #87. The person in charge will reply soon.

 Comment by git-bot-sec
Wednesday Jun 27, 2018 at 02:39 GMT

If you want to monitor the current build status of your PR, Please refer to http://aaci.mooo.com//TAOS-CI/ci/standalone/ci-server/monitor.php

 Comment by git-bot-sec
Wednesday Jun 27, 2018 at 02:39 GMT

:dart: cibot: @STAR has updated the pull request.


Adding perf profile tab on Kruize UI & fixing RecommendationComp typo

Raising this PR to support Performance profile tab and minor typo changes in file names

@rbadagandi1 Please fix the conflicts

Conflicts resolved @dinogun. Please review.


[Excel] (Errors)

My PR from yesterday #2898 was based on incorrect error strings provided by eng. Eng detected mistake. This PR updates docs to match actual error strings.

Thanks for the review — yesterday and today — @ElizabethSamuel-MSFT!