Why fastJoin is preferred over using populate?

This sentence is repeated on both hooks related documentation sections:

https://hooks-common.feathersjs.com/hooks.html#fastjoin
https://hooks-common.feathersjs.com/hooks.html#populate

fastJoin is preferred over using populate

But I didn't find any link or explanation about why.
So I am asking: why? :slightly_smiling_face:

I am asking because I consider populate more "easier to use" than fastJoin. We can have the related data with just some configuration lines:
I used fastJoin to calculate a price of a record:
fastJoin(
  {
    joins: {
      totalPrice: () => async (ride) => {
        ride.totalPrice = getTotalPrice(ride.price);
      },
    },
  },
),

This works, but you will certainly say I should use alterItem for that.
Plus, you show alterItem example on the documentation to load something that looks like a relation:
alterItems(rec => {
  rec.userRecord = (async () => await service.get(...) )()
})

Well, I am pretty confused. :upside_down_face:
I hope you may clarify this. :+1:


Document pre-req for manageTLS: runtime must load certificates

manageTLS mounts the certificates onto the runtime pods, but the runtime is responsible for loading the mounted certificates. Liberty containers load the certificates automatically into the default keystore / truststore at container startup. tWAS runtime doesn't have such equivalence. Document that the inorder for manageTLS to work end-to-end, runtime must load the certs.
Document this in the limitation / troubleshooting section

The current workaround to install tWAS applications using RCO v1.2.0 is to use spec.route.termination: edge with insecureEdgeTerminationPolicy: Redirect or insecureEdgeTerminationPolicy: Allow or spec.route.termination: edge.passthrough
For example:
spec:
  route:
    termination: edge
    insecureEdgeTerminationPolicy: Redirect
  service:
    type: ClusterIP
    port: 9080


Deleting the namespace holding a CR usually orphans cluster-scoped resources "owned" by this CR

The issue
Let's say we have an helm-based operator for a Foo kind, whose backing helm chart renders some namespace-scoped resources, as well as some cluster-scoped resources, say a ValidatingWebhookConfiguration called ${foo-namespace-and-name}-foo-validation.
Now take the following sequence of events:
Setup

a user creates a foohome namespace,
in that namespace the user creates a myfoo instance of Foo
the operator adds a uninstall-helm-release finalizer to myfoo
the operator runs helm to create resources in foohome as well as the foohome-myfoo-foo-validation ValidatingWebhookConfiguration object; this also creates the helm "release" secret

Now we have two deletion scenarios
Deletion of CR alone

user does kubectl delete foo myfoo -n foohome
operator runs helm to delete the resources it has previously deleted. This works fine since the "release" secret is still there and has a list of what needs to be deleted
operator removes the finalizer from myfoo
kubernetes finishes the deletion and myfoo vanishes

So everything works as intended.
Deletion of the namespace

user does kubectl delete ns foohome
kubernetes performs a cascading deletion of all objects in the foohome namespace
usually kubernetes gets to delete the helm "release" secret first
not long after the operator tries to process the myfoo deletion. But by then the release secret is gone. So all that the operator seems to do is log INFO	controllers.Helm	Release not found, removing finalizer and remove the finalizer.
kubernetes completes the deletion of the namespace-scoped resources and the foohome namespace as well

In this scenario, the foohome-myfoo-foo-validation stays around indefinitely.
Moreover, if it was not for the disambiguating prefix on the cluster-scoped resource name, then when the user now retries steps (1) and (2) with a different name, the operator will fail to reconcile with an error such as:
rendered manifests contain a resource that already exists. Unable to continue with install: ValidatingWebhookConfiguration \"foo-validation\" in namespace \"\" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key \"meta.helm.sh/release-namespace\" must equal \"newns\": current value is \"foohome\""

I used "usually" in the title because this is a race, but a race that I found to be usually lost by the helm operator.
Ideas for fixing this
Put a finalizer on the helm release secret as well, and only remove it when about to remove the finalizer on the CR.
Alternative: when CR is being deleted, then instead of depending on the release secret, do a dry-run processing of the helm chart, and delete everything it spits out.  Note that this not reliable, because the CR spec might have changed between the moment when the controller has last acted on it and now (leading to a change in the list of resources produced).

Put a finalizer on the helm release secret as well, and only remove it when about to remove the finalizer on the CR.

With this option, what would happen if a user just went in and manually tried to delete the helm release secret? It seems like the following steps would occur:

Helm release secret gets a deletion timestamp, but is not deleted because it has a finalizer
CR gets reconciled since we're watching the release secret (what happens here?)

The deletion of the helm release secret would hang until the finalizer was deleted, and we would be unable to process any further helm upgrades (you can't update an object that has a deletion timestamp)
It seems like the only option would be to have the reconciler remove the finalizer, let the deletion proceed, and then re-install the release to get the secret re-created.
But that seems like it would making handling the "delete the namespace" case problematic since you can't re-create anything in a terminating namespace.
I'm not a huge fan of the alternative solution for the reason you mention (there's no guarantee the secret matches the CR)

In general (and unfortunately) there are lots of issues with deleting namespaces without cleaning up inside them first. Off the top of my head:

If the namespace also happens to contain the operator itself, the operator deployment might be deleted before the CRs in that namespace. That would cause the namespace deletion to hang because the CRs were not able to be finalized.
If a CRD has a webhook served by a deployment in the namespace and the namespace contains CRs, the webhook might get deleted before the CRs, and then since the webhook is unavailable, the namespace deletion hangs because the API server is unable to reach the webhook to process the remaining CRs.

My point being, perhaps a simpler immediate solution is to document that namespaces containing CRs managed by helm-based operators should not be deleted.
Nonetheless, I'm definitely open to ideas to come up with a way to solve this to help avoid these races altogether.

Currently I think the only safe option would be to:

(optionally) put up an error in the CR status warning about the unexpected secret deletion, and
refuse to reconcile anything other than a deletion of the CR.
This would solve the "namespace deletion" case. As for the "user decided to delete just the release secret" case, this should at least guide them to what they did wrong - by following the breadcrumbs of secret -> ownerRef -> CR -> status.

In addition to conflicting with the namespace deletion, I don't see how letting the secret deletion proceed and re-installing the release could be made reliable. AFAICT the release secret is the only state storage mechanism we have. Once we let it disappear, we lose track of what resources should be deleted - the set of resources derived from current CR spec might be different than the one that just got deleted, since spec might have been changed in the meantime.
Eventually, I think this is a perfect use case for the liens a.k.a. in-use-protection feature that is proposed.

Eventually, I think this is a perfect use case for the liens a.k.a. in-use-protection feature that is proposed.

Oh TIL about that KEP. Big 👍 from me. That feature would be a big win for operators.
One of the root causes of this problem is that namespace-scoped objects (the CR controlling a helm release) can't be owners of cluster-scoped objects or objects in other namespaces. If the CR was cluster-scoped:

It couldn't be deleted as part of a namespace deletion
It could own any other object in the cluster, making the helm uninstall step much less important, because kubernetes garbage collection would take care of anything that was missed.

So I'm wondering if we should investigate support for cluster-scoped CRs in the helm operator. It would be much easier to say: "Any helm chart that deploys cluster-scoped objects should be managed by a cluster-scoped CR".
It seems like this would be fairly straightforward to implement, and the only big hurdle to overcome would be where to store the release secret (and that could likely be easily solved by a CLI flag, environment variable, or maybe even an annotation in the custom object).
In fact, I was experimenting with a completely unrelated project recently and decided to try using this library to make a cluster-scoped object own and reconcile a helm release, and it generally seemed to work. This is all I had to do. That's clearly begging for a better interface, but its clear to me that its pretty do-able.

While adding support for cluster-scoped CRs might be a useful feature, I don't think we should recommend to operator developers to make their CR cluster-scoped just because their helm chart happens to contains a couple of cluster-scoped resources. There are valid reasons to keep things in namespaces, access control and resource quotas being primary ones.  So I don't think it counts as a solution to this issue.
Incidentally, when designing the API of the project we use helm-operator-plugins for, one of the hardest part was deciding whether the CRs should be cluster- or namespace-scoped. One of our CRs represents a typical web app, which is perfectly suited to be namespaced-scoped.
The hard part was what to do with a handful of cluster-scoped resources that the associated helm chart happened to create:

PersistentVolume and StorageClass which were convenience features in the helm chart
SecurityContextConstraints - to allow using hostDir storage
PodSecurityPolicy - ditto
ClusterRole - needed only because of the presence of the PodSecurityPolicy

In the end we decided to go namespace-scoped and rely on finalizer to help us clean up resources that k8s GC would not handle.
At that time we did not even know that the helm operator did not support cluster-scoped CRs, but it would have been another hint in that direction.
After some time, we changed to using one of the stock SCC, and now that PSPs are on their way to removal, most of the reasons to go cluster-scoped are gone. If we had decided to go cluster-scoped we would be in an awkward place, since the scope of a CR can never be changed, even across API versions.
In general, exactly because of the asymmetry you mentioned, it seems to me like going cluster-scoped is a slippery slope. Once you go down that route, you find that more and more things need to be cluster scoped.

Since https://github.com/kubernetes/enhancements/pull/2840 is still in KEP phase, I'd like to restart the discussion assuming we might not have it.
Coming back to your question:

With this option, what would happen if a user just went in and manually tried to delete the helm release secret?

I have a followup question: what happens currently, let's say in the worst possible corner case of operator controller being down for a moment and the CR getting changed or deleted in the meantime? I'm assuming we do not lose track of the operand resources, thanks to the owner references. So what is the impact actually? If none, then why do we need this secret anyway? I must be confused :thinking:


Need to update platform-server package to match @angular

Missing package for "@angular/platform-server": "4.0.0", and "@angular/platform-server": "4.0.2"
Short description of what this resolves:
Resolves npm install errors and dependencies that are unmet
Changes proposed in this pull request:
Add  "@angular/platform-server": "4.0.0" to ChangeLog docs
Add "@angular/platform-server": "4.0.2" to ChangeLog docs
Ionic Version: 3.x

Closing as setup for ionic v3 does not use @angular/platform-server.
Perhaps I'll do another pull request to specify that this is confusing when updating...
Perhaps we could update the docs to say that it is safe to remove @angular/platform-server???


Touch outside the Alert to dismiss?

Is it possible to touch outside the box and the alert to dismiss at this poit?

// Handle ContainerView tap gesture
func handleContainerViewTapGesture(sender: AnyObject) {

//         cancel action
let action = actions[cancelButtonTag - 1] as! DOAlertAction
if (action.handler != nil) {
action.handler(action)
}
self.dismissViewControllerAnimated(true, completion: nil)
}


Improve configuration parameters for Kafka #1359

Signed-off-by: chandresh-pancholi<EMAIL_ADDRESS>
Which problem is this PR solving?

Improve configuration parameters for Kafka #1359

Short description of the changes

I have added three Kafka Producer config.
Old PR: https://github.com/jaegertracing/jaeger/pull/1524


MultiWorkerMirroredStrategy:Training is very slow with 50 workers

I am trying to use 50 workers to train my model and made it.However,training is very slow.I'd like to know if there are some tips to
improve the performance in terms of reducing training time.I would appreciate it if you could give me some advice.

@Mullich123
Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Thanks for you reply,I've got some tips on Tensorflow tutorial and it's sorry to trouble you for my carelessness.I will change next time.

@Mullich123
Its absolutely no issues, please let us know the issue faced and share details for us to help.

Sorry I am a newbie in Tensorflow2.Could you please delete this issue.

@Mullich123
Please confirm if your issue is resolved.

Good night
I'm so sorry.It's my default.I raised the issue in a wrong way.By debugging my code
and referencing tutorials,I found I knew too little.In term of the issue,I use 50 ports
of localhost to do experiment and make it ambiguous.So please delete it and I would
raise another questions carefully.
Yours Sincerely,
Mr. Mullich


Consider increasing session expiry

Hello, could you increase the default session expiry, or allow it to be configurably extended?
E.g. with a "keep me signed in" check box?
https://github.com/efdevcon/devcon-website/blob/f29bd2bc16ff122b68b40df0dbab8bfce491f45e/src/pages/api/account/token.ts#L30

Hi @gnattishness, thanks for your feedback. Extending session expiration is high on our priority list.
Will keep you updated!

Solved with #168
Thank you @randylien 🙏


feat add etcd

@lachie83 helm is a cool stuff, thanks for that.

@ingvagabund @bprashanth - Merged svc and petset manifest as requested.

@ingvagabund - I was just in Brno. I'll have to come say hello next time I'm in town

@ingvagabund - I was just in Brno. I'll have to come say hello next time I'm in town

what a pity, next time let's have some beer then

Up to the grammar nit, I am fine with the PR. This is the first example, I expect other improvements over time. Let's move the effort to the second PR (no matter who is going to send it).

@lachie83 would you mind pulling in the last changes from master (reorg) then putting this chart in the incubator directory?

@viglesiasce sure no worries

@viglesiasce moved chart to incubator

ok to test

All worked as expected. Thanks @lachie83!!!
Tested on a 1.3.6 cluster with alpha flags enabled (kube-up.sh on gce). Used the etcdctl client from an adjacent fedora pod in the local namespace.
[root@fedora-230265774-v0z87 /]# export ENDPOINT='http://exciting-gophe-etcd:4001,http://exciting-gophe-etcd:2379'
[root@fedora-230265774-v0z87 /]# etcdctl --endpoint $ENDPOINT mkdir vic
[root@fedora-230265774-v0z87 /]# etcdctl --endpoint $ENDPOINT ls
/vic
[root@fedora-230265774-v0z87 /]# etcdctl --endpoint $ENDPOINT cluster-health
member c319a202ef817fa is healthy: got healthy result from http://exciting-gophe-etcd-2.exciting-gophe-etcd:2379
member 154dceee85b49a84 is healthy: got healthy result from http://exciting-gophe-etcd-0.exciting-gophe-etcd:2379
member 3b40565e3013c8bb is healthy: got healthy result from http://exciting-gophe-etcd-1.exciting-gophe-etcd:2379
cluster is healthy
[root@fedora-230265774-v0z87 /]


Spike into how to update users package.json file for them

What
Investigate how to apply updates to the package.json file for a user. This might be complicated, I think the only way to do this is with a three-way merge that includes the contents of the file as released in the users current version of the kit, the contents of the file as released in the version to be upgrade to, and the contents of the file in the users project.
This ticket is to spike ways of achieving this three way merge with the update script, or something alternative that has the same effect. The output should be an approach, and another ticket with more details on how to fix this.
Why
We want to simplify the update process for users, so we can make releases more often. One of the currently problems with upgrading is that users lose any packages they have installed personally, and have to reinstall themselves.
Who needs to work on this
Developer
Who needs to review this
Developer
Done when

We know how to make the update script do a three-way merge of the package.json file

The problem of losing packages may not require a 3 way merge, just looking at what the user had in package and bringing it over.
In addition to losing packages, another scenario is users deliberately using for example an older version of Step by step, and the kit has a newer version. This is complicated by the fact some users may want or need to update a package that the kit has updated,
This is a separate problem and may be too hard to resolve at the same time

The problem of losing packages may not require a 3 way merge, just looking at what the user had in package and bringing it over.

It depends on whether it's a problem if a package we have removed from the release gets put back in by the update script. I think that would be; we don't really want users to be installing things they don't need to, and in some cases it could be a security issue.

great point

I had a go at doing this with the standard command line tool diff3.
This worked well for small version jumps, but as the difference between package.json grew as the version jump got larger, its output became less useful as it found many more conflicts between lines. Unfortunately while you can change how aggressively it will try to resolve conflicts, you can't really give it any guidance on how to resolve conflicts; it seems to be mainly designed for human intervention when it comes to conflicts. I guess we could think about presenting the user with conflicts, but I'd like to think spend a little more time on this spike thinking about how to create a script that does the update without user intervention.

I have a JavaScript script that does the merge of the package.json dependencies for us:
https://github.com/alphagov/govuk-prototype-kit/commit/b9aaa7a52bd4c47451eabace7364f76a4b8fe617
It's fairly complicated, but it seems to work well.
Would appreciate testing from others though, before we decide if this is the way we want to go.

Some scenarios I hadn't considered but should be:

User has upgraded multiple times
User has downgraded step-by-step extension

looks great thanks - another small note from our testing session: standardise on originalPackage vs oldPackage
and a question that might be clarified by a code comment: why do we need to sort twice?
  const mergedPackageNames = Object.keys(merged)
    .sort()
    .sort((a, b) => {


cabal: Could not resolve dependencies

Hi! Thanks for providing this tutorial! Total beginner here.
When I tried:
docker build . -t minimal/checker

it failed with:
Step 17/18 : RUN cabal new-install
 ---> Running in e6f50d3e2379
cabal: Could not resolve dependencies:
[__0] trying: Checker-<IP_ADDRESS> (user goal)
[__1] next goal: base (dependency of Checker)
[__1] rejecting: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: Checker =>
base>=4.13.0 && <4.14)
[__1] skipping: base-<IP_ADDRESS> (has the same characteristics that caused the
previous version to fail: excluded by constraint '>=4.13.0 && <4.14' from
'Checker')
[__1] rejecting: base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>,
base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>, base-<IP_ADDRESS>
(constraint from non-upgradeable package requires installed instance)
[__1] fail (backjumping, conflict set: Checker, base)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: base, Checker

The command '/bin/sh -c cabal new-install' returned a non-zero code: 1

Hm, that's weird.
They seem to have bumped the Haskell Image, so as I see there is a "too new" base version.
@diegoo It should be fine if you remove the version number from base, but I'll have a look too and add a version number to the haskell-image

@diegoo  Ok I had the same issue, I now specified it to
FROM Haskell:8.8.3-buster

...

And it works for me. Can you confirm?
Its updated in the repository, thanks for reaching out.

@Twonki Yes, adding that version works for me. Thanks!
(Doesn't seem to be committed though.)


Not able to paste values inside select box

I need to paste the options in the dropdown which need to be selected. I tried using ";" for differentiating between options but it is not working.
Can I get some help here?

I having trouble when pasting in the multiple selection input, when I copy and paste, ng2-select sometimes puts a '[' instead of the copied value in chrome and 'à' in firefox.
is there a way to solve this issue?


Bryce minor fixes

Ok, I guess this really isn't minor at this point . . .
I am happy to explain anything that I did. If it needs to be explained, then I probably should have commented on my code better (or written it more clearly). Most of the changes are not really needed but they either make the code look nicer to me or show off a more "Pythonic" way to do something that really makes no difference. There are some significant improvements to colour handling, LaTeX handling, and poll display. Many of the minor fixes do technically improve speed but not to any level that is significant to anyone. I am sure that I also did enough things that decrease the speed that it balances out.
Now because it took me so long to do this, I had to merge in new commits. But I did this by rebasing which changed history and therefore makes it look like this request also includes all of those changes. It also makes it so that it can't be merged automatically. I've managed to fix it but I do not know if that will have any adverse effects. I will merge like a normal person in the future.

I've fixed a number of issues brought up in review.


Update to Xcode 8.0 / Swift 2.3

A few minor changes, including:

The private dependencies have been updated for Xcode 8.0 / Swift 2.3,
Dobby-Mac is now Dobby-macOS, and
CircleCI uses Xcode 8.0.

Perfect timing! Thank you 🎉
I think we can merge this one 👍


Improve the command splitting

This matches the split used in run_command_cd_instead_of_workdir.rego

is there a case where the existing implementation fails?

I can't think of any off the top of my head other than if someone has a typo like ls /usr/bin&& ls /root where the space is missing before or after the && which would cause a failure to build the image anyway since it's not proper shell syntax.
That said, the old way does not match the spaces and could result in the spaces finding their way into commands_list, which probably isn't intended.


BASH Ultimate Edition - Fixing bug, please apply this ASAP

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Page layout is broken from last change, this fixes it.
First time using GitHub. Something went wrong uploading files last time.

Character Sheet Info Roll20 Internal Use only.


Xinywa/update

Summary
Describe the goal of this PR. Mention any related Issue numbers.
Requirements (place an x in each [ ])

[ ] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[ ] I've read and agree to the Code of Conduct.

The following point can be removed after setting up CI (such as Travis) with coverage reports (such as Codecov)

[ ] I've written tests to cover the new code and functionality included in this PR.

The following point can be removed after setting up a CLA reporting tool such as cla-assistant.io

[ ] I've read, agree to, and signed the Contributor License Agreement (CLA).

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


FetchContent support

I saw the README said glog can be incorporated into custom projects using the CMake command add_subdirectory. I tried it like this
FetchContent_Declare(
  glog
  GIT_REPOSITORY https://github.com/google/glog.git
  GIT_TAG        v0.6.0
)
FetchContent_GetProperties(glog)
if(NOT glog_POPULATED)
  FetchContent_Populate(glog)
  add_subdirectory(${glog_SOURCE_DIR})
endif()

When building as a submodule of my projects, I found those template codes in xxx.h.in files have not been replaced to generate xxx.h file. How to do things right? Thanks for your help.

I found the solutions.
FetchContent_Declare(
  glog
  GIT_REPOSITORY https://github.com/google/glog.git
  GIT_TAG        v0.6.0
)
FetchContent_GetProperties(glog)
if(NOT glog_POPULATED)
  FetchContent_Populate(glog)
  add_subdirectory(${glog_SOURCE_DIR} `${glog_BINARY_DIR}`)
endif()

And I will give a PR later.

add_subdirectory mentioned in the docs does not use FetchContent. You need to specify the build directory only for the latter explicitly.

The issue is not actionable. You can however extend the README with a link to CMake reference documentation that explains how to use FetchContent correctly if you want.

@sergiud I opened a PR which gives an example to show the usage of FetchContent and add_subdirectory. https://github.com/google/glog/pull/1067


Does the regional-limit table also denote all regions the feature is in preview at ?

It is said that the feature is GA-ed in a few regions, and in all other regions which it's enabled, it's in preview.
However, it wasn't entirely clear what are the exact regions which the feature is in preview at.
Namely, is this available in government clouds ?

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

ID: 81e00caf-dd95-441b-d2d5-0b3c979cb11b
Version Independent ID: 83ed8e59-7ae8-332e-9ce0-7d80e8a973f8
Content: Deploy container group to Azure virtual network - Azure Container Instances
Content Source: articles/container-instances/container-instances-vnet.md
Service: container-instances
GitHub Login: @dlepow
Microsoft Alias: danlep

@dany74q Thanks for the Feedback. I have assigned the issue to content author for checking this and updating the doc as applicable.
@dlepow Can you please look into this doc enhancement request and add your comments on the doc update as appropriate.


the auto completion does not work for more than half of the solidity keywords

Example: public, modifier, returns.
Poor autocompletion, Will it be better someday?

True, these are built as snippets, highlighting but not as auto-complete items.


docs: add link to wiki page

Pretty trivial change, just wanted to see what @ariporad thinks of it. It gives a shoutout to the two most popular projects in the wiki.

@ariporad I rebased this. I changed the wording a little bit. I also removed the docs for the make command, and moved these to the wiki (and a link is provided). The rationale behind this is that I want less junk in the way of the command reference, which is probably the most important part of the README.

LGTM!


Invalid getter for ProxyUri and ProxyScheme?

Hi,
I've just found in class Message that getters for ProxyUri and ProxyScheme get option OptionType.Accept but set valid option.
Is that correct?
I was very surprised when I set option Accept and the Utlis.OptionsToString returned something in ProxyScheme.

Oops... These are no doubt careless miswrittens in class Message. Would you like make a pull request and help us fix this? Or you can wait a couple of days before we have some new commit.


fix(select): clear selection enhancement

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix remaining YARD exception, enable --fail-on-warning

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This is a follow-up to https://github.com/Homebrew/brew/pull/14354#issuecomment-1379304944 that resolves the remaining YARD error and enables --fail-on-warning to the docs action. It does so by using YARD's parse directive to register ENV as a class. While ENV is not actually a class, this is a common workaround in ruby documentation.
While it would make more sense to place the @!parse directive near the extend invocation at the end of the file, YARD seems to be picky about where directives can go: a class/module declaration, a def, the top of the file, or in isolation. (The last one meaning that the directive is followed by 3 or more newlines, which rubocop didn't like.)
cc @EricFromCanada

Including a screenshot, in case it's helpful:

Thanks again @dduugg, much appreciated!


Adding Docker support

Hey :)
Thanks for the amazing tool, I bumped some versions and added docker support :)
to run it at the moment use: docker run -d --rm -p 80:80 lumberjackotters/visualsubnetcalc:latest
There is also a github action workflow so you can build the container and push it to dockerhub (with your username in it :))
Do you want me to update documentation ?
Cheerz
PurpleBabar

@PurpleBabar This looks good at first glance. Let me look over this in a bit more detail over the next day or two and I'll merge it and start publishing to my own docker repo.


Unable to locate _dl_open symbol

Hi David,
I'm testing this code and getting this error.
Can you help me?

Which distro are you running?

I'm using ubuntu 20

This is my ld-2.31.so

I'm also having this issue on Ubuntu 20.
Searching through commits on binutils
git log --all --grep='dl_open' doesn't turn up any results since 2016
git grep -I 'dl_open' turns up none as well, which seems weird since it contains ld's source code.
From some googling around and playing with the binary itself I started to dig into GLIBC_PRIVATE I found this thread I believe something changed that now labels it private so you can use it within the library but you as an external caller can't locate the address offset. That's my current working theory but need to get some more time to play around with it.

If the symbol is no longer exported, then I have a vague idea for a workaround - which is, to locate the function via heuristics. I'll need to see if any of the exported functions reference it, and if so, disassemble them to extract its offset.

I got the same issue on Ubuntu 21.
@DavidBuchanan314,
Is there quick workaround to fix issue?
Thanks
Tim

Note to self: investigate __libc_dlopen_mode

Upon further investigation, these changes were made because libdl is now directly part of libc - so I suppose we can just use dlopen directly, on newer libcs?
https://lwn.net/Articles/864920/

@DavidBuchanan314, we just moved a tool that started out as a fork of dlinject into public beta today (https://github.com/BishopFox/asminject). One of the main reasons I built it was this particular issue. In case it's helpful, I think the root cause is that different Linux distributions expose the library-loading function differently. Some of them have a function in ld called _dl_open, like dlinject is looking for. Some of them have a function in libc called __libc_dlopen_mode. Some of them have a function in libdl called dlopen. For the library-injection payloads in asminject.py, I created variations that would handle _dl_open and dlopen. Every distribution I ran across that had __libc_dlopen_mode also had one of the other two, so I haven't made a payload for that variation yet.
I've listed the variations I found in my testing here: https://github.com/BishopFox/asminject/blob/main/docs/examples-shared_library_injection.md

You only need to change 86 lines of code to ‘’if re.match(r"./libc-..so", ld_path):‘’ ，this error is not reported.

You only need to change 86 lines of code to ‘’if re.match(r"./libc-..so", ld_path):‘’ and change 94 lines of code to "dl_open_offset = lookup_elf_symbol(ld_path, "__libc_dlopen_mode")" and chane 33 lines to "symtab = elf.get_section_by_name(".dynsym")", then it'll be succeed.

Have you tried that on several Linux distributions? As I mentioned in my reply, there seems to be at least three different models that different Linux distributions use. So I would expect your approach to work on Debian and its derivatives (although maybe not newer versions of Ubuntu), but not on OpenSUSE or Arch, for example.


Notification UI design missing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

What performance issue do we need to solve?

Notification Icon: A small icon to represent notifications, such as a bell icon or an envelope icon.
Notification Counter: Display the number of unread notifications next to the notification icon. This helps users quickly see how many new notifications they have.
Notification List: When users click on the notification icon, a dropdown or modal appears displaying a list of their notifications. Each notification includes:
Notification Content: A brief description of the notification.
Timestamp: The time when the notification was received.
Action Buttons: Depending on the type of notification, include action buttons like "Mark as Read," "Dismiss," or "View."
Clear All Button: Include a "Clear All" button at the bottom of the notification list to allow users to dismiss all notifications at once.
Notification Settings: Provide a link or button to access notification settings where users can customize their notification preferences, such as opting in or out of specific notification types or channels.
Close Button: Include a close button to allow users to close the notification dropdown or modal without taking any action.
Design Consistency: Ensure that the notification UI design aligns with the overall look and feel of the application for a cohesive user experience.

What is the impact of this on end-users?
List specific user experiences that will be improved by solving this problem e.g. app boot time, time to for some interaction to complete, etc.
List any benchmarks that show the severity of the issue
Please also provide exact steps taken to collect metrics above if any so we can independently verify the results.
Proposed solution (if any)
Please list out the steps you think we should take to solve this issue.
List any benchmarks after implementing the changes to show impacts of the proposed solution (if any)
Note: These should be the same as the benchmarks collected before any changes.
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on Upwork

Instructions here to join the channel

@MelvinBot I was sent 2 emails but thee is no update also I sent few suggestion solutions discuss in different threads comment's but seem not response.

@MelvinBot Any reported bugs, new features suggestion, testing or bug fixings should consider and get paid.

I sent my suggestion about notification

UI missing, will agree?


Created Test scenario - Example Pull Request

I have created a number of tests which have "issues".  This is intended
to be given to a technical interview candidate to find and resolve the
issues.

Keeping the branch separate


Update SRBWaterfallEffects.netkan

SpaceDock still says:

https://spacedock.info/mod/2992/SRB Waterfall Effects
The forum thread says it "will NOT work without these", before saying it's not a hard requirement, which sounds like a contradiction:

https://forum.kerbalspaceprogram.com/index.php?/topic/207196-112x-srb-waterfall-effects/


On mobile, after clicking aside link, menu is not closed

aside {overflow: hidden;}
aside > div {overflow-y: scroll; overflow-x: hidden;}


Tate games

Hi
I've noticed that I can place Arcade roms is several folders, like: arcade, mame2003 and mame.
I use a Mame 2003+ non merged romset that belongs to the mame2003 folder.
These have a mix of horizontal and Tate games in there.
What would you do if you needed to create a separate folder only for Tate games?
Being in a separate folder is important because Tate games need a RetroArch screen tweak to change the proportions (and no rotation on the rgb30).
My current method implies a separate override for each Tate game, which is tedious.
Thank you in advance for any advice.

Hi,
You can edit /etc/emulationstation/es_systems.cfg, create a duplicate copy of the arcade system entry, have it point to a different folder name something like tate or vertical.  Some themes may have an entry for vertical only that you may be able to use with that system as well.  nes-box has a verticalarcade system.

It worked! Thanks! 👍🏼


@json directive doesn't render with the correct $echoFormat

Laravel Version: 8.83.27
PHP Version: 7.4.30
Database Driver & Version: mysql 8.0.30

Description:
The BladeCompiler class has the method withDoubleEncoding. This method configures the $echoFormat variable in such a way that subsequent calls to htmlspecialchars from the e echo helper method used by Blade, are called with the $double_encode argument set to true.
This works fine when using the regular echo syntax of Blade {{ json_encode($data) }}, however it doesn't apply at all when using the json directive @json($data).
Steps To Reproduce:

Let's say we have the given example data $data:
$data = ['foo' => 'bar'];

And the following view that get's passed the $data variable:
<some-vue-component v-model="@json($data)"></some-vue-component />
{{-- vs --}}
<some-vue-component v-model="{{ json_encode($data) }}"></some-vue-component />

I would expect both to produce the same output. However, the compiled output looks like the following:
<some-vue-component v-model="{"foo":"bar"}"></some-vue-component />

<some-vue-component v-model="{"foo":"bar"}"></some-vue-component />

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so, please open up a new issue and we'll help you out.
Thanks!

I can confirm this issue is still present in the latest version of Laravel.
The @json Blade directive is implemented in the following file: https://github.com/laravel/framework/blob/9.x/src/Illuminate/View/Compilers/Concerns/CompilesJson.php#L28 which hasn't seen any changes in the past 5 years.
If you want, I can propose a fix for this.


Add 'Create Folder' button in Install Project dialog

Fixes #25454

I first went in the direction of creating a new UI element for installing the project but I later found out that there is already an element like that that we can use, so this PR eventually got reduced to a line change.
Open for suggestions!

Screenshots for reference:

Please squash commits together, the second one completely obsoletes the code added in the first one.

@akien-mga Squashed the redundant commits

You also rebased an unrelated commit in the process it seems. Try git rebase upstream master to fix that. (Provided you added an upstream remote pointing to godotengine/godot).

Rebased the branch, now only my own changes are up

Thanks!


Remove boto and table_logger requirements

Remove several python sdk requirements as they introduce side effects and are not used by us.

Looks good to merge.


Proposal for required services

Summary:

[ ] /v1/notify-about-new-device, returns the mounting name of a device
[ ] other proposals can be added here

It would be good if the Mounting Orchestrator would offer a service, to notify other applications if a new device has been added to the controller (of course it should also provide the mount name of the new device in its notification).
Possible usage e.g. in the MWDI. If a new device is added the MWDI needs to retrieve that devices control construct. With the notification, the control construct would be derived immediately after the device has been added, whereas otherwise there would be some delay.

[ ] Controller CRUD operations

[ ] MountPoint (Device) CRUD operations

[ ] List of supported devices CRUD operations

Notifications

[ ] /v1/notify-about-new-device, returns the mounting name of a device

[ ] /v1/notify-acceptance-ready-link-endpoint

Services regarding performance, auditing, logs...

[ ] other proposals can be added here


added in pro pages that were deleted [ambassador-docs #193]

@LukeShu I shifted some things around because I'm very nitpicky about table layout. Content wise, I updated some capitalization and added that one sentence at the top.

I assume the reference/core/annotations and reference/pro/* docs are already re-added in another PR?

I assume the reference/core/annotations and reference/pro/* docs are already re-added in another PR?

@nbkrause annotations is already in there, and /pro pages got moved to be just in the /reference, but they exist as well


Text Generator

Tired of boring lorem ipsum?
 Yess??
Proposal
Here is a thing for you, a text generator in JS to help you generate random text paragraphs
Demo

Assign this to me & I will get it done!

@TidbitsJS I like it!! Go for it :)


Doesn't recognize custom wavtools

Acknowledgement

[X] I have read Getting-Started and FAQ

🐛 Describe the bug
When using Track Settings to set the wavtool, it still only lists "convergence" and "simple" despite me having three custom wavtools (moresampler.exe + moreconfig.txt, wavtool.exe, wavetool4vcv.exe).
If I click "Location" it opens the Wavtools folder where they are, so it should be the correct folder, it just seems the program isn't recognizing it.
The resampler selection works fine.
Explains how to reproduce the bug

Place wavtools in Wavtools folder
Create a track
Click the gear (Track Settings) on the track to open the resampler/wavtools menu

OS & Version
Windows 11, OpenUtau ver. 0.0.942.0
Logs
2022-09-25 05:05:01.678 -07:00 [INF] Logging initialized.
2022-09-25 05:05:01.704 -07:00 [INF] Microsoft Windows NT 10.0.22621.0
2022-09-25 05:05:01.705 -07:00 [INF] Microsoft Windows 10.0.22621 X64 X64
2022-09-25 05:05:01.705 -07:00 [INF] OpenUtau v0.0.942.0 win10-x64
2022-09-25 05:05:01.889 -07:00 [INF] Initializing application.
2022-09-25 05:05:02.063 -07:00 [INF] Initializing culture.
2022-09-25 05:05:02.064 -07:00 [INF] Initialized culture.
2022-09-25 05:05:02.064 -07:00 [INF] Initializing theme.
2022-09-25 05:05:02.074 -07:00 [INF] Initialized theme.
2022-09-25 05:05:02.074 -07:00 [INF] Initializing OpenUtau.
2022-09-25 05:05:02.093 -07:00 [INF] Search all singers: 00:00:00.0085202
2022-09-25 05:05:02.103 -07:00 [INF] Search all plugins: 00:00:00.0094459
2022-09-25 05:05:02.106 -07:00 [INF] Loaded plugin LyricParser2.1 C:\LegacyApps\OpenUtau\Plugins\Lyric Parser 2.1\LyricParser.exe
2022-09-25 05:05:02.106 -07:00 [INF] Search all legacy plugins: 00:00:00.0027006
2022-09-25 05:05:02.106 -07:00 [INF] Initialized OpenUtau.
2022-09-25 05:05:02.106 -07:00 [INF] Initializing audio.
2022-09-25 05:05:02.107 -07:00 [INF] ReleaseSourceTemp C:\LegacyApps\OpenUtau\Cache
2022-09-25 05:05:02.139 -07:00 [INF] Initialized audio.
2022-09-25 05:05:02.139 -07:00 [INF] Initialized application.
2022-09-25 05:05:02.139 -07:00 [INF] Framework initialization completed.
2022-09-25 05:05:02.168 -07:00 [INF] Creating main window.
2022-09-25 05:05:02.318 -07:00 [INF] Initialized main window component.
2022-09-25 05:05:02.380 -07:00 [INF] SchedulePreRender
2022-09-25 05:05:02.381 -07:00 [INF] Publish notification Load project
2022-09-25 05:05:02.387 -07:00 [INF] Main window checking Update.
2022-09-25 05:05:02.387 -07:00 [INF] Created main window.
2022-09-25 05:05:02.795 -07:00 [INF] Checking update at: https://github.com/stakira/OpenUtau/releases/latest/download/appcast.win-x64.xml
2022-09-25 05:05:03.824 -07:00 [INF] undoGroup started
2022-09-25 05:05:03.825 -07:00 [INF] ExecuteCmd Add track
2022-09-25 05:05:03.869 -07:00 [INF] undoGroup ended
2022-09-25 05:05:03.869 -07:00 [INF] SchedulePreRender
2022-09-25 05:05:03.869 -07:00 [INF] Publish notification Pre-render notification.
2022-09-25 05:05:05.435 -07:00 [INF] undoGroup started
2022-09-25 05:05:05.436 -07:00 [INF] ExecuteCmd Change singer
2022-09-25 05:05:05.440 -07:00 [INF] undoGroup ended
2022-09-25 05:05:05.440 -07:00 [INF] SchedulePreRender
2022-09-25 05:05:05.441 -07:00 [INF] Publish notification Pre-render notification.
2022-09-25 05:06:34.662 -07:00 [INF] Exiting.
2022-09-25 05:06:34.666 -07:00 [INF] Exited.

Did you change the resampler?
Worldline seems to only support the default wavtools


Kafka topics filter and blacklist usage issues

Hello,
I'm having issues with setting up the following configuration for Kafka topics to back up.
I want to have the ability to run Secor as follows:
secor.kafka.topic_filter=.* (the entire existing topics)
secor.kafka.topic_blacklist=topic1,topic2,topic3,topic4... (list of topics that will be excluded).
This configuration is not working though, what am I missing?
Are there any exmaples for the blacklist configuration usage?
Thanks,
Lior

Are your using profile kafka-2.0.0 during maven build?  I think for
kafka-2.0.0, you might need to use the kafka new consumer property:
kafka.new.consumer.topic.list=
Take a look at the latest src/main/config/secor.common.properties
On Tue, May 19, 2020 at 7:55 AM Lior Yakobov<EMAIL_ADDRESS>wrote:

Hello,
I'm having issues with setting up the following configuration for Kafka
topics to back up.
I want to have the ability to run Secor as follows:
secor.kafka.topic_filter=.* (the entire existing topics)
secor.kafka.topic_blacklist=topic1,topic2,topic3,topic4... (list of topics
that will be excluded).
This configuration is not working though, what am I missing?
Are there any exmaples for the blacklist configuration usage?
Thanks,
Lior
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/1333, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYJP76TJ26NAY56ACP4HZTRSKMXJANCNFSM4NFCQB3A
.

Hey @HenryCaiHaiying,
I'm using the kafka-2.0.0 profile.
I used this configuration and it seems to work well, it reads only the topics I have specified.
On the other hand, if I can use blacklist instead of the topics list it will be more convenient.
Does the blacklist as well works with kafka-2.0.0, similar to the topics list, with commas seperation?
Thanks,
Lior

Looks like you can do some filtering: if kafka.new.consumer.topic.list is
empty, it will use secor.kafka.topic_filter
https://github.com/pinterest/secor/blob/8b0ac6e5a05a45543297a321c3e3c894d8a05046/src/main/java/com/pinterest/secor/reader/SecorKafkaMessageIterator.java#L144
public void subscribe(RebalanceHandler handler, SecorConfig config) {
ConsumerRebalanceListener reBalanceListener = new
SecorConsumerRebalanceListener(mKafkaConsumer, mZookeeperConnector,
getSkipZookeeperOffsetSeek(config),
config.getNewConsumerAutoOffsetReset(), handler);
;
String[] subscribeList = config.getKafkaTopicList();
if (subscribeList.length == 0 || Strings.isNullOrEmpty(subscribeList[0])) {
    mKafkaConsumer.subscribe(Pattern.compile(config.getKafkaTopicFilter()),

reBalanceListener);
} else {
mKafkaConsumer.subscribe(Arrays.asList(subscribeList),
reBalanceListener);
}
}
On Wed, May 20, 2020 at 4:55 AM Lior Yakobov<EMAIL_ADDRESS>wrote:

Hey @HenryCaiHaiying https://github.com/HenryCaiHaiying,
I'm using the kafka-2.0.0 profile.
I used this configuration and it seems to work well, it reads only the
topics I have specified.
On the other hand, if I can use blacklist instead of the topics list it
will be more convenient.
Does the blacklist as well works with kafka-2.0.0, similar to the topics
list, with commas seperation?
Thanks,
Lior
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/1333#issuecomment-631427452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYJP76EXH5XP7ATIQ4IT6DRSPALZANCNFSM4NFCQB3A
.


Error 429

Saving and restoring cache seems to work, but then this happens:
ERROR: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/rust/manifests/sha256:de6e081a4e9c10e70731071cec6f629892da1bbaa24b478a96bdc59060674117: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
------

and i have no clue why. Any luck?

It is mentioned in in the error message.

Server message: toomanyrequests: You have reached your pull rate limit.
You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

The Docker webpage mentions:

The rate limits of 100 container image requests per six hours for anonymous usage, and 200 container image requests per six hours for free Docker accounts are now in effect. Image requests exceeding these limits will be denied until the six hour window elapses.

Should not be a problem with the library, but rather the CI hitting the limits specified by Docker.


cum_count returns incorrect values when used as trans_primitives

Bug/Feature Request Title
cum_count returns incorrect values when used as trans_primitives
Bug/Feature Request Description
import featuretools as ft
es = ft.demo.load_mock_customer(return_entityset=True)
print(ft.__version__)

0.9.1
print(ft.show_info())
None
from pprint import pprint

import sys
print(sys.version)

3.6.8 |Anaconda custom (64-bit)|
fm, features = ft.dfs(
  entityset = es,
  target_entity = "customers",
  agg_primitives = ["sum"],
  trans_primitives = ["month", "cum_count"],
  max_depth=2,
  #instance_ids = [4],
)
pprint(features)

[<Feature: zip_code>,
<Feature: SUM(transactions.amount)>,
<Feature: MONTH(join_date)>,
<Feature: MONTH(date_of_birth)>,
<Feature: SUM(sessions.CUM_COUNT(customer_id))>]
fm.reset_index(inplace=True)
print(fm.query("customer_id == 4").T)
customer_id                                 4
zip_code                                60091
SUM(transactions.amount)              8727.68
MONTH(join_date)                            4
MONTH(date_of_birth)                        8
SUM(sessions.CUM_COUNT(customer_id))       95
The above SUM(sessions.CUM_COUNT(customer_id)) = 95 seems incorrect!
Checking: Verify that SUM(sessions.CUM_COUNT(customer_id)) = 95 is incorrect:
(1) Get the list of session_id by customer_id == 4
session_id_customer_4 = es["sessions"].df.query("customer_id == 4")["session_id"].to_list()
print(session_id_customer_4)

[3, 5, 8, 11, 12, 13, 21, 22]
(2) If CUM_COUNT is calculated correctly, it should have returned sum([1, 2, 3, 4, 5, 6, 7, 8 ]) = 8*(8+1)/2 =  36. However, it seems to return the session_id_customer_4 itself, i.e., as an identity function, sum([3, 5, 8, 11, 12, 13, 21, 22]) = 95, which seems incorrect.
(3) To verify the above, let calculate the session_id list for each customer_id
session_ids_customer = dict()
for _c in es["customers"].df["customer_id"].to_list():
  session_ids_customer[_c] = es["sessions"].df.query("customer_id == @_c")["session_id"].to_list()

for _c in sorted(session_ids_customer.keys()):
  print("customer_id = ", _c, ", session_ids = ", session_ids_customer[_c], ", sum of list = ", sum(session_ids_customer[_c]))

customer_id =  1 , session_ids =  [4, 6, 9, 14, 18, 26, 27, 29] , sum of list =  133
customer_id =  2 , session_ids =  [1, 10, 15, 16, 17, 31, 33] , sum of list =  123
customer_id =  3 , session_ids =  [7, 19, 23, 25, 34, 35] , sum of list =  143
customer_id =  4 , session_ids =  [3, 5, 8, 11, 12, 13, 21, 22] , sum of list =  95
customer_id =  5 , session_ids =  [2, 20, 24, 28, 30, 32] , sum of list =  136
(4) Check the value of SUM(sessions.CUM_COUNT(customer_id)) for each customer_id
print(fm[["customer_id", "SUM(sessions.CUM_COUNT(customer_id))"]])
customer_id  SUM(sessions.CUM_COUNT(customer_id))
0            1                                   133
1            2                                   123
2            3                                   143
3            4                                    95
4            5                                   136
(5) Expected result from SUM(sessions.CUM_COUNT(customer_id))
def cum_count_x(_list):
  from featuretools.primitives import CumCount as cc
  _cumcount = cc()
  _result = _cumcount(_list).sum()
  return _result

for _c in sorted(session_ids_customer.keys()):
  print("customer_id = ", _c, ", session_ids = ", session_ids_customer[_c], ", cum_count of list = ", cum_count_x(session_ids_customer[_c]))

customer_id =  1 , session_ids =  [4, 6, 9, 14, 18, 26, 27, 29] , cum_count of list =  36
customer_id =  2 , session_ids =  [1, 10, 15, 16, 17, 31, 33] , cum_count of list =  28
customer_id =  3 , session_ids =  [7, 19, 23, 25, 34, 35] , cum_count of list =  21
customer_id =  4 , session_ids =  [3, 5, 8, 11, 12, 13, 21, 22] , cum_count of list =  36
customer_id =  5 , session_ids =  [2, 20, 24, 28, 30, 32] , cum_count of list =  21

Sorry for the late reply @bigheadming , the issue is that if you want CumCount to filter by an id column (like customer id) it should be added to the groupby_trans_primitives list instead of trans_primitives, like this:
fm, features = ft.dfs(
  entityset = es,
  target_entity = "customers",
  agg_primitives = ["sum"],
  trans_primitives = ["month"],
  groupby_trans_primitives = ["cum_count"],
  max_depth=2,
  #instance_ids = [4],
)

This will create a different feature, SUM(sessions.CUM_COUNT(customer_id) by customer_id), that has the values you were expecting:
>>> useful_cols = ["customer_id", "SUM(sessions.CUM_COUNT(customer_id) by customer_id)"]
>>> print(fm[useful_cols])
   customer_id  SUM(sessions.CUM_COUNT(customer_id) by customer_id)
0            5                                               21.0
1            4                                               36.0
2            1                                               36.0
3            3                                               21.0
4            2                                               28.0

Closing this issue but feel free to re-open if you have additional questions or would like more clarification!


Want anonymous TCP port binding

Port 0 binds, but its awkward to learn the port the server is listening on.
What would be convenient would be for the Listener() interface to offer and Address() routine that returned the actual service address (URL format) suitable for connecting.  This could then be returned to surveyors to create a dynamic distributed service network using service discovery.

:rocket:


[ISSUE] Elasticsearch fails to start when provided with image values

Describe the issue:
Elastic search does not start when provided with image values. Seems to have something to do with volume mounting.
Actual behavior:
The error is the following:
ERROR: Missing logging config file at /usr/share/elasticsearch/config/log4j2.properties
sometimes it will say jvm.options not found as well.
Expected behavior:
How to reproduce:
elasticsearch:
    image:
      registry: "artifactory.blabla.ca"
      repository: "bla"
      tag: latest
      pullSecrets:
        - artifactory
      pullPolicy: Always

I wrote a custom image to include s3 plugin:
FROM artifactory.bla.ca/elasticsearch:8.8.2
RUN bin/elasticsearch-plugin install --batch repository-s3

Logs:
ERROR: Missing logging config file at /usr/share/elasticsearch/config/log4j2.properties
Environment:
Please note: Without the following info, it's hard to resolve the issue and probably it will be closed.

Platform: openshift
Helm CLI version: 3.13.1
Chart version: 8.3.2
Values file: see above

@jpan8866 just for my understanding, do you want to create a custom ES image and use it in the Camunda chart?
This is out of the scope of the Camunda side.
Feel free to open the issue again if it's not about creating a custom ES image.

Mainly, I am trying to add s3 repository plugin and credentials to ES keystore, but the bitnami image used by the camunda chart neither has bin/elasticsearch-keystore nor bin/elasticsearch-plugins. So I tried making a custom image on the official ES image. Perhaps i'm going about it wrong. Any pointers appreciated


⚠ ClamAV has degraded performance

In 15814a7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 29d5708.


Remove mnemonic requirement for eth

Addresses https://github.com/Sailias/cryptocoin_payable/issues/3
I found one way to generate Ethereum addresses without relying on the mnemonic / private key. I had to switch to using BIP32 derivation and use .node_for_path() without the full derivation path. I'm honestly not sure why the method for computing these is different from the Bitcoin case but the generated addresses seem to match https://iancoleman.io/bip39/

@mhluska this looks much better

@Sailias I wonder if we should be incrementing the version at this point. On one hand this change will affect all generated Ethereum keys/addresses going forward so should probably get a major version bump. On the other hand, we haven't published a gem yet and the code is still in flux.

if you test and merge this pull request, I would change the version, and create a tag.
We can decide to publish it after we deem it completely stable.


I can not request same url after upgrading from 3.35.3 to 3.38.1

why I need requesting same url?  because my login form would make a fake url '/user/login'.
say I am at homepage '/', then click user login btn, url is '/user/login', then I click homepage logo, the url should become '/', but it does not change, remaining 'user/login'
I have not found any config about this.
so what can i do to allow request the same url?

Cannot reproduce the problem from your info. Can you show me a demo? And can you identify the version that caused the problem, like 3.35.3 to 3.35.4?

Cannot reproduce the problem from your info. Can you show me a demo? And can you identify the version that caused the problem, like 3.35.3 to 3.35.4?

Thank you.
my website performed well with 3.35.3
recently I upgrade pjax-api to 3.38.1 and today I found that i can not request the same url

now I downgrade to 3.35.3,   everything go very.
I guess the new version of pjax-api maybe take a check and render the new document only when the new url is different

Too little info for debugging.


V-sync and interaction button problem

I have a strange problem too that with vsync activated sometimes it skips the interaction button presses (b by default, but i tried swapping it too) and does not record them properly. Which results in you sometimes having to tap a few times to interact with objects on the map or other characters, seems random. It only happens when you control the character and it doesn't seem to happen in the menus. I have solved it by disabling vsync. I dont use any other unity 50fps camera fixes or mods. Im using the official Xbox Elite 2 V2 controller and my monitor is a Dell 2k 165hz.
Edit: I still have the problem with vsync off from time to time, I don't know what it can be. I dont think its a problem with the hack itself, it seems fps cap or hz related, the red room menu cursor was speed up too. Only game where all this happens although I experienced something similar unlocking Nier Automata fps with Special K because some actions are tied to the framerate. Only fix that seems to work right now is fps cap at 60fps, could you make it possible to enter the fps cap values manually like the monitor hz? That would help a lot because its hard to select what you need now with the mouse slider only. Thanks.  Any other ideas around it? I cant go over 60fps (tried 120 too) right now without getting this issue.

Unless you go exclusive fullscreen - Unity uses monitor refresh rate (if Vsync is enabled) and ignores target framerate. If you disable V-sync, then target framerate is used and you can configure it to whatever you want - technically if you use borderless it will likely result in somewhat odd frame pacing, but shouldn't introduce tearing, as this game is not using DXGI Flip Model.
As for the input being ignored - again, not something I have experienced, with my game being capped at 144. Unity itself processes inputs per frame, unless for whatever reason, devs write some inputs to be handled in FixedUpdates. But with SteamInput added on top, maybe some mistakes were made.

Thanks for the input. I can't seem to get around this problem either, maybe because my monitor is the odd dog at 165hz? In the end the only solution for me is to play at caped 60fps.

Yes, sometimes you have to move against the object for it to detect them or even wait for a character to look back at you even at 60fps vanilla game. But this exact problem is easily recognizable from that because sometimes you have to hit the action button up to 3 or 5 times to enter a vending machine (for example) in front of it and other occasions it works just fine.


[ISSUE #924]🔥Support producer send rpc message-2🚀

Which Issue(s) This PR Fixes(Closes)

Fixes #issue_id
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


Boolean Offline Status

This issue is intended to be the sole issue associated with the Boolean Offline Status design doc.
Please reference the document for the complete problem statement and detailed implementation plan.
Problem
Parts of our app are reliant upon a 3rd-party service called Pusher to communicate with our servers in real-time. There are a couple scenarios where the app may have an internet connection but still be unable to receive data from Pusher, for example:

You are connected to the Pusher service but not subscribed to your user channel
The Pusher service is down

In these scenarios, we currently have an inconsistent UX, where those parts of our app that are reliant upon Pusher (eg. "user is typing", comments being edited, etc.) won't work as expected, while others will. This will be confusing for users.
Solution
Implement Boolean Offline Status according to the following logic:
If you are authenticated with Expensify (have an authToken), you are considered online if and only if:

You have an internet connection (determined by NetInfo), AND
You are connected to Pusher, AND
You are subscribed to your private user Pusher channel

If you are NOT authenticated with Expensify (do not have an authToken), you are considered online if and only if:

You have an internet connection (determined by NetInfo), AND
You are connected to Pusher

If you are not online, you’re offline. Period.
Outstanding Issues:

https://github.com/Expensify/App/issues/9211

Commenting so this hits my participating filter.

Pro-tip – you can also just press Subscribe/Unsubscribe on the side 😄

Putting this on HOLD per this slack thread

@roryabraham Yeah unfortunately that doesn't work for the "participating" filter in my GH inbox. Thanks though!

Got a draft PR up, just need to spend a bit more time testing it and working on the PR writeup before it will be ready for review: https://github.com/Expensify/App/pull/9230/files

Going to prioritize this over the next few days. Updated the PR on Friday and was fighting Storybook, of all things.

Prioritized Comp tool improvements last week due to the deadline being today. Picking this back up this week

No update. This really only covers a sort of rare edge case where pusher or our web servers are down, so despite being under the Offline-First umbrella it's really not as high priority as some other items.

Speak of the devil: https://expensify.slack.com/archives/C03SC9DVD/p1659625927469579

Not prioritizing over other N7-blocking initiatives

Still not prioritizing over other N7 issues

Still not prioritizing, though a tangentially-related issue has cropped up here

@roryabraham I'm going to swap this to NewFeature. Let me know if you disagree.

We could, but I like keeping it open as a monthly for now. I also think that implementing this will be a bit simpler once this proposal is implemented

Ok, cool. Thinking more on this, you're right. We still need to do this. I was also probably too wrapped up in the specific implementation of boolean offline status, and where it fits. Assigning myself as well to keep it top of mind.

Added the WAQ hold for clarity.

Still on hold for WAQ.

No longer on hold for WAQ, but still not a priority in my book.

No update

Out of curiosity, I wonder if we should consider this one for the "buddy" process?

Chatted with @JmillsExpensify, we're just going to close this for now


failed to call webhook: unexpected error when reading response body

Hello,
I am trying to deploy cert-manager in my lab and experiencing the issue while creating cert issuer. I have tried with cert-manager version 1.11, 1.12 and 1.13 but getting the same error in my lab environment which is running on k8s version 1.23.
Initially I tried to install cert-manager using helm but it was failing like the issue mentioned on the link https://github.com/cert-manager/cert-manager/issues/4646. I tried all the possible solution from this link but unfortunately nothing worked for me.
Now I have installed the cert-manager using kubectl apply mentioned on the official doc page https://cert-manager.io/docs/installation/kubectl/ and I can see the pod and services running but while I am trying to create an issuer getting the below error.
Error from server (InternalError): error when creating "test-resources.yaml": Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook: unexpected error when reading response body. Please retry. Original error: context deadline exceeded
Error from server (InternalError): error when creating "test-resources.yaml": Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook: unexpected error when reading response body. Please retry. Original error: context deadline exceeded
kubectl get all -n cert-manager
NAME                                          READY   STATUS    RESTARTS   AGE
pod/cert-manager-784dc8659d-qt2rv             1/1     Running   0          48m
pod/cert-manager-cainjector-8b96b8fb5-65ps4   1/1     Running   0          48m
pod/cert-manager-webhook-86994fcd8f-mlg8j     1/1     Running   0          48m
NAME                           TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
service/cert-manager           ClusterIP   x.x192.15            9402/TCP   48m
service/cert-manager-webhook   ClusterIP   x.x.124.158           443/TCP    48m
NAME                                      READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/cert-manager              1/1     1            1           48m
deployment.apps/cert-manager-cainjector   1/1     1            1           48m
deployment.apps/cert-manager-webhook      1/1     1            1           48m
NAME                                                DESIRED   CURRENT   READY   AGE
replicaset.apps/cert-manager-784dc8659d             1         1         1       48m
replicaset.apps/cert-manager-cainjector-8b96b8fb5   1         1         1       48m
replicaset.apps/cert-manager-webhook-86994fcd8f     1         1         1       48m
#cat test-resources.yaml
apiVersion: v1
kind: Namespace
metadata:
name: cert-manager-test
apiVersion: cert-manager.io/v1
kind: Issuer
metadata:
name: test-selfsigned
namespace: cert-manager-test
spec:
selfSigned: {}
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
name: selfsigned-cert
namespace: cert-manager-test
spec:
dnsNames:
- training.com
secretName: selfsigned-cert-tls
issuerRef:
name: test-selfsigned
kubectl get nodes
NAME        STATUS   ROLES    AGE    VERSION
x.x.x.66   Ready    master   3d6h   v1.23.8
x.x.x.68   Ready    worker   3d1h   v1.23.8

I got this error because network policies were blocking the requests. I'll try to investigate exactly what requests got blocked and see if I can create a policy that allows them. If I figure it out, would you be interested in a PR that adds the network policy to the cert-manager manifests/chart? More and more organizations nowadays run very restrictive network policies and expect app applications to explicitly open up the communication required...

@wallrj FYI, this relates to your docs PR

@inteon thanks for pinging me.
I've published some broad network policy recommendations on the cert-manager.io website.

https://cert-manager.io/docs/installation/best-practice/#network-requirements-and-network-policy

It includes a summary of the cert-manager network interactions that you need to account for.
This is only a start. I stopped short of giving full NetworkPolicy examples because I reached the limit of my knowledge with Calico and I don't have experience with other CNI plugins. We can add some example manifests in later PRs.
Please read and feedback.

I noticed that openstack tenant network was created on vxlan and packet encapsulation was happening on both at k8s and openstack level due to which packet reading was not happening properly and it was causing the issue for me. I changed the openstack network from vxlan to vlan and then tried and it worked for me.
Though I am not sure if any other workaround for the same but this resolved my problem.

I stopped short of giving full NetworkPolicy examples because I reached the limit of my knowledge with Calico and I don't have experience with other CNI plugins. We can add some example manifests in later PRs.

I'm happy to try and help with this. Having had a swift read of the docs you linked, I think it looks reasonable. Adding a concrete example is always good, but I think it would be even better to just implement the required NetworkPolicies as part of the chart.
If the user is not running a CNI plugin that supports NetworkPolicies, then the policies we create will have no effect. If the user do have a CNI plugin that supports NetworkPolicies, then they will be helped by the policies we create. We need to be careful so the rules we create are correct, but apart from that I see no downside to create them.
Rather than improving the documentation further, would you be open to a PR that adds  the required polices? I'll even make sure to verify them on Calico and Cilium.

Rather than improving the documentation further, would you be open to a PR that adds the required polices? I'll even make sure to verify them on Calico and Cilium.

@lindhe I'm open to it, but I have come to the conclusion that it's too difficult to create Kubernetes NetworkPolicy resources that allow cert-manager to components to connect to the Kubernetes API server, when the Kubernetes API server is outside the cluster (rather than in a Pod inside the cluster), such as on Google GKE.
Read the comments in https://github.com/cert-manager/cert-manager/issues/2334 first and announce your PR there so that all the people in that issue can try your proposed NetworkPolicy.

https://github.com/cert-manager/cert-manager/issues/2334

/close
Because @pram0596 and @lindhe both seem to have solved their webhook problems.
There is some documentation explaining the networking requirements.
And @lindhe may open a PR to help users install best practice NetworkPolicy resources with the Helm chart.


Link to A Super-simple Guide to Mechanical Ventilation... is incorrect

Hi,
The link goes to a different (tho interesting) document...
Looking forward to this guide!
Christine
A Super-simple Guide to Mechanical Ventilation For Open Source Engineers Designing Ventilators for the COVID-19 Pandemic -- This was written by me (Robert L. Read). It's my best attempt at an introduction.

Fixed: https://docs.google.com/document/d/1p-oYUv_6FadipV67g2O5yfnQvpdvZDm4WadYVubhoH4/edit?usp=sharing


Declaring transforms should run eagerly if possible, even for deferred inputs

 Issue by nex3
Originally opened as dart-lang/sdk#18270

Suppose there exists a lazy transformer L followed by a declaring transformer D. L declares that it emits assets X and Y; D is primary for only Y, and declares that it emits Z. If X is requested, L will generate both X and Y. Currently D will not generate Z, but since it's only declaring and not lazy it should do so once its input is available.

 Comment by nex3

Fixed by dart-lang/sdk@d30621796b90796240c95d7d225bfc65c8413372.

Added Fixed label.

 Comment by nex3

Added Started label.


feature(logbook): add map modal for current vessel position

Coverage remained the same at 42.382% when pulling e8ddd44f6e4173e0f02e4ccde515e9691273ef91 on feature/logbook into c6a2f667ef71ac4b8cdaea186e30fcad083d6b50 on master.


Update Cmake Files

Examples/CmakeLists.txt needs to be updated to just pull in every file and do the cmake magic.
Update for release/debug builds.
Add in compiler flags for debug mode (printf everywhere!), even test, release..
Look into setting up for installing..
mkdir build
cd build
cmake ..
make
sudo make install

Thinking about just keeping it simple: user will be able to download a zip of the librequest.a and the header file then user can import the header file and compile with the .a file
Also, give build instructions from source with debug info etc..


add onelogin/php-saml security issues

Based on the warnings in the README file of the library, I added these 2 security related matters. I am not sure if the CVE mentioned in the README should be added. I can't find the CVE in the official CVE database.
https://github.com/onelogin/php-saml/blob/master/README.md

I can't find the CVE in the official CVE database.

if a CVE has been assigned, add it. Information in CVE database may be released later when more info about the vulnerability is provided.

Alright. I added it.

thank you @peternijssen


Dogecoin fatal error while downloading wallet

I am having issues with the Dogecoin wallet download, at about 80%, and now suddenly the application does not start up anymore, and gives a fatal error:
"A fatal internal error occure, see debug log for details"
This is the error in the debug log:
2021-03-02 14:58:01 ERROR: CheckAuxPowProofOfWork : block does not have our chain ID (got 0, expected 98, full nVersion 0) 2021-03-02 14:58:01 ERROR: ReadBlockFromDisk: Errors in block header at CBlockDiskPos(nFile=240, nPos=10406780) 2021-03-02 14:58:01 *** Failed to read block 2021-03-02 14:58:02 Loaded 65219 addresses from peers.dat 482ms
Please help, since there are dogecoins sent to this wallet, and since they did not yet arrive, I can not retrieve them, thank you.
I am using Windows 10 and Dogecoin version 1.14.2

What version did you upgrade this from?

This is my first version, I started it newly

in the %AppData%\Dogecoin\blocks folder, are there any files with a higher number than blk00240.dat?

yes blk00241 and 242,dat

okay. This sounds like a data corruption on disk (because the client will never accept a blockheads with version 0), but to be absolutely sure, I'd recommend the following steps:

shut down the wallet software completely
in %AppData%\Dogecoin make a copy of peers.dat to elsewhere
delete peers.dat (DO NOT remove wallet.dat)
in %AppData%\Dogecoin\blocks, make a copy of blk00240 / 241 / 242 .dat files to elsewhere
remove the 240 / 241 / 242 files
restart dogecoin-qt.exe from the command line as: dogecoin-qt.exe -reindex

it will take a bit of time to run through the reindex (couple of hours) but that should reset you.

Okay, thank you, I will try this, hopefully it can resolve the issue

Hello, I have done all you advised me, and now I want to  restart dogecoin-qt.exe from the command line as: dogecoin-qt.exe -reindex, but when i do it, my system tells me dogecoin-qt.exe -reindex is not recognized as an internal or external command, operable program or batch file. Please let me know where I should use this command line, I now used it in C:\users in windows
Thanks !

Sorry! It's most likely C:\Program Files (x86)\Dogecoin\ or C:\Program Files\Dogecoin\

hmm, ok thanks, but both also don't work.....

You can find the exact location of your executable by searching dogecoin in the start menu, and then choose "open file location", which should open an explorer window to the location where it is installed.

ok, found it, but it does not have any progress......already 2 hours running

update, its re-indexing the blocks on disk now, thank you so much for your support!!

Was this now solved?

Yesterday it was finally updated and I can use the wallet now, thank you!
On Thu, 11 Mar 2021 at 12:27, Patrick Lodder @.***>
wrote:

Was this now solved?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dogecoin/dogecoin/issues/1779#issuecomment-796668477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATB5QJTIP2KU2SJ6SOJ6R2DTDCSLZANCNFSM4YPZTH4A
.


Take Control Hearbeat Failures

Take Control heartbeats fail once implementing the rules as outlined but the actual Take Control functions still work.  Anyone else seeing this or know the fix?

I solved this - had to add two more conditions for heartbeat to pass.
(      (         http.request.uri.path eq "/dms2/services2/ServerMMS2" and          http.user_agent eq "Agent-Probe" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/bosh/bosh/" and          http.user_agent eq "" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerEI2" and          http.user_agent eq "Mozilla/5.0 (compatible)" and          http.request.method eq "POST"      ) or      (          http.request.uri.path contains "/images/agent/" and          http.user_agent eq ""      ) or      (          http.request.uri.path contains "agentAssetImageMap.txt"      ) or      (          http.request.uri.path contains "/download/"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerII2" and          http.user_agent eq "Mozilla/5.0 (compatible)" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/FileTransfer/"      ) or      (          http.request.uri.path eq "/commandprompt/"      ) or      (          http.request.uri.path contains "/LogRetrieval"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerMMS2" and          http.user_agent eq "gSOAP/2.8" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerEI2" and          http.user_agent contains "MSP%20Anywhere%20Daemon (unknown version)" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerII2" and          http.user_agent contains "MSP%20Anywhere%20Daemon (unknown version)" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerII2" and          http.user_agent eq "CodeGear SOAP 1.3" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/services2/ServerII2" and          http.user_agent eq "Mozilla/4.0 (compatible; MSIE 6.0; MS Web Services Client Protocol 4.0.30319.42000)" and          http.request.method eq "POST"      ) or      (          http.request.uri.path eq "/dms2/hello"      ) or      (          (              http.request.uri.path eq "/dms2/services2/ServerII2" and              http.user_agent contains "Take%20Control%20Viewer%20for%20N-central/1 CFNetwork/" and              http.request.method eq "POST"          ) or          (              http.request.uri.path eq "/dms2/services2/ServerII2" and              http.user_agent contains "Mozilla/5.0 (Windows NT 10.0; Win64; compatible) MSPAnywhere/<IP_ADDRESS>5" and              http.request.method eq "POST"          ) or          (              http.request.uri.path eq "/dms2/services2/ServerII2" and              http.user_agent contains "Mozilla/4.0 (compatible; MSIE 6.0; MS Web Services Client Protocol 4.0.30319.42000)" and              http.request.method eq "POST"          )      ) or      (          http.request.uri.path eq "/dms2/services2/ServerMMS2" and          http.user_agent contains "Mozilla/4.0 (compatible; MSIE 6.0; MS Web Services Client Protocol" and          http.request.method eq "POST"      )  )


Cleanup logging

Looking through the logs in splunk, the cacheable gem logs a line like this for every request:
[Cacheable] Browser gzip: false, Suggested key: , Raw cacheable.key: 8506570,false,text/html,www.shopwestindieswear.com,/,,,false,,,:7700401694961521622, cacheable.key: cacheable:618e467be6a84698402813bd6ebb718f, If-None-Match:

Now, "Browser gzip" is useless information -- it's usually true, and when it's not, a separate line is logged that looks like this:
[Cacheable] uncompressing for client without gzip

"Suggested key" and "If-None-Match:" are almost always nil, so I made them conditional.
r: @csfrancis

Ping @csfrancis :-)

@burke This is a very old PR I found in my list. Rebased, revived and it's ready to ship (except for the "class is too long" style warning which I'm not going to fix...).


CUDA/PyTorch version mismatch

Hi.
I receive this error when I launch the setup.py script:
RuntimeError:
The detected CUDA version (11.7) mismatches the version that was used to compile
PyTorch (10.2). Please make sure to use the same CUDA versions.
I did several tests uninstalling and installing several CUDA and PyTorch versions without success.
What is the correct CUDA/PyTorch versions pair to use?
Note that I followed the instructions to create the correct dependencies by the environment.yaml file and it ends with no errors.

@buemi The official anaconda installation will install Pytorch 1.10 which is compatible with CUDA 11.3.


Environment name error

Following the README file and running the following code with the latest version would yield error: Environment llf-Gridworld-v0 doesn't exist.
import llfbench as gym
env = gym.make('llf-Gridworld-v0')

I looked into the __init__ file, line 5 is causing the issue env = gym.make(env_name), where the "gym" is the gymnasium rather than llf-bench.

@BeibinLi  The code is okay, just the registered env_name is llf-gridworld-v0. Can you give it a try?

Thanks! The new env name fixed the issue.


Bump to 2023 year in testdata

Close #63.

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

CI is failing fort other reasons.  Sigh...
We'll need to clean this up.

Can this be fixed permanently with Replace of current year to "{current_year}" on both sides before comparison? The periodic fixes are problematic for distros which run tests when building the package.

Can this be fixed permanently with Replace of current year to "{current_year}" on both sides before comparison? The periodic fixes are problematic for distros which run tests when building the package.

I agree we need a better approach.  However allowing the tests to pass would prevent user from remembering these files need to be changed. But I haven't looked at the code in a while so I might be missing something.

Such a failure sounds suitable for your CI (upstream), but I think basically anyone else will be unhappy with that breakage.  There are situations when even older versions are desirable to use (e.g. bisecting regressions across history) and then similar newly-appearing failures can be annoying.

Damn it, I was cleaning up some public forked repos.


relativeTo issue with empty folder

I have an issue with generating an relative url where the current browsers url contains an empty folder:
history.pushState(null,'test','/a//c/d') // browser path: example.com/a//c/d
var uri = new URI('http://example.com/a').relativeTo(new URI()) // uri = "../../a"
history.pushState(null,'test2',uri.toString()) // browser path: example.com/a/a

The problem is that normalizePath convert '/a//c/d' to '/a/c/d'. Maybe you can find a solution for me?

All clients I know translate /{2,} to /, meaning /// gets normalized to /. I've always considered a//b the result of a bad join operation (e.g. "a/" + "/" + "b").
I have not found anything in RFC 3986 that mandates this behavior. In fact I have not found any mention of // in the path segment aside from section 3.3 saying:

If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//").

Here's a good summary on the behavior of // in paths in Linux. It is also the way Python normalizes paths. Exactly the behavior URI.js show.

Maybe we should talk about the semantics of your a//b/c thing. What's the reason you're using // in there?

Well you are right, currently i have the // because of a bug in an other place of our routing and we are trying to remove it, which should also solve the problem. Also you are right that // should not be used and looks wired.
So far I tested the code i posted in Chrome and was surprised to get example.com/a/a as a result instead of the expected example.com/a.
So far i noticed the also google used the // as you can see here: http://static.googleusercontent.com/media/www.google.com/en//webmasters/docs/search-engine-optimization-starter-guide.pdf
If you remove in the link the // it wont work.

Does that mean we can close this issue now?

Feel free to close this ticket, as I found the solution for me to use absolute URIs,which works for our project.
But I would say that relativeTo does not work correct at least in Chrome as you can see in my example.

using the url resolution of URL in Chrome I can see the following happening, suggesting that URI.js may indeed be a bit overzealous in normalizing paths:
var url = new URL('../../d', 'https://google.com/a//b/c');
ulr.pathname === '/a/d'


Revert "chore: Add custom browserslist (#2290)"

GDS can't handle it so reverting it for now.

Though I think it's likely, I'd like to test that this will properly unblock us on deploying main before actually merging it. My proposal would be:

Deploy this branch to dev.
Temporarily update just one of the GDS E-inks to point to the dev version of its URL.
Monitor to see whether it continues making data requests. (Ideally, the only consequence is that the screen briefly shows "dev" in its header in small text, but otherwise displays the same information.)
Revert the update to the E-ink device, pointing it back to production.
Assuming this test was successful, merge this branch and deploy to production.

Let me know if that makes sense.

Picking EIB-101 to test this out. Copied its prod config into dev. Will let the branch sit for about 10 minutes and then update the URL.

Starting to see successful api requests: https://mbta.splunkcloud.com/en-US/app/search/search?sid=1732308914.7805057. I think this was the issue like we thought. Will leave this up for ~10 more minutes, switch the URL, and merge the change after approval. Then we can deploy main again.

Still looking good: https://mbta.splunkcloud.com/en-US/app/search/search?sid=1732309570.7807810. I'm now confident in this being the issue.


Create Type id cells

Requirement: Support Creating Cells Based on Type ID
Background:
talks：https://talk.nervos.org/t/en-cn-misc-single-use-seals/8279/2
In the Type ID method on CKB, every seal on a seal chain must contain the first seal’s ID as genesis information. Type ID ensures that the type script gives cells unique IDs and restricts them from being changed throughout the seal chain’s lifecycle. This ID allows verifiers to retrieve the latest seal easily from a universal indexer on CKB.
Requirement:
We need the SDK to support the following features:

Generate Type ID: The SDK should be able to generate a unique Type ID.
Create Cell with Type ID: The SDK should be able to create a cell with the generated Type ID, ensuring that the Type ID remains immutable throughout the cell’s lifecycle.

@SpectreMercury We've created an example demonstrating how to create a TypeID cell with Lumos at #731. We hope this example is helpful


Add mask match to us/can

Take two of the pull request.  Issue https://github.com/findthemasks/findthemasks/issues/561

@nburt Done!  Please look at https://github.com/findthemasks/findthemasks/pull/586


recrawlableResolver - minimal frequency stopped working

hello Pascal, could you please re-open #741 ? As REJECTED_PREMATURE stopped working properly after fixing the sitemap issue (#738 ) .
Thanks!

#741 Re-opened.


[Android] Support theme-color on Android Lollipop+(5.0+)

Support css meta theme-color on Android Lollipop+(5.0+). For Android
version older than 5.0, this doesn't work.
BUG=XWALK-4305
(cherry picked from commit 3d89fb817123d85fdbb7a34eda7fd060b94cdb1a)

@lincsoon  @fujunwei  @halton  @rakuco

LGTM

The crosswalk-12 branch is dead and we do not intend to make any additional Crosswalk 12 releases, so I'm closing this PR.


Is 'TANGO2 deficiency' actually 'metabolic encephalomyopathic crises, recurrent, with rhabdomyolysis, cardiac arrhythmias, and neurodegeneration' (MONDO:0014812)

Mondo term (ID and Label)
metabolic encephalomyopathic crises, recurrent, with rhabdomyolysis, cardiac arrhythmias, and neurodegeneration (MONDO:0014812)
Synonym to be added

Transport And Golgi Organization protein 2 (TANGO2) deficiency
TANGO2 deficiency
(https://www.ncbi.nlm.nih.gov/pubmed/31339582)

Is this also the same as this?
3. tango2-related-metabolic-encephalopathy-and-arrhythmias
(https://rarediseases.info.nih.gov/diseases/13423/tango2-related-metabolic-encephalopathy-and-arrhythmias)
Also see for patient-centered phenotypes: https://tango2research.org/signs-symptoms/
Synonym subtype: ie, broad/exact/narrow/related
exact

@GARDinfocenter, @ahamosh, @cmungall: do any of you have an opinion if MONDO:0014812 'metabolic encephalomyopathic crises, recurrent, with rhabdomyolysis, cardiac arrhythmias, and neurodegeneration' is the same as the GARD term: tango2-related-metabolic-encephalopathy-and-arrhythmias
(https://rarediseases.info.nih.gov/diseases/13423/tango2-related-metabolic-encephalopathy-and-arrhythmias)
These seem like they could be equivalent to me. In which case, we should add an xref to GARD.

These are the same.  I can't tell you how much I wish that people would stop putting gene symbols in disease names....

Thanks @ahamosh!

@ahamosh me too!

Thank you for your prompt action on this.

@nicolevasilevsky GARD has listed the two terms as synonyms since 3/5/2018. The change was prompted by research conducted by GARD due to an inquiry from a patient organization.


Integration with NextJS

Hello, this project seems to be very good, do you know how I could integrate this library in a project with Next?
Thanks

I tried with this library in an project with Next.
And I get this error:
TypeError: dispatch is not a function

I reproduced only the example from de main page.
Any suggestions?
thanks!

Heya! Thanks for the issue. I tried to reproduce the error with nextjs create app template but failed.

Could you give me more details? If you press useMetamask.js?cdc:138 url in second line of the error you got, this may route you to code block causing the issue.

perfect!, you can share this example?

sure, here you go. but please let me know the cause of the code line by doing instruction from my previous message. so I can see what I can improve.
nextjsapp.zip

Perfect, now I'll look at the code you sent, and I'll let you know.

I have the issue.

Oh, I guess I know the issue now. Did you wrap your root component with <MetamaskStateProvider> ?
As shown here in first step.
Your root component must look like this;
import React                     from 'react';
import ReactDOM                  from 'react-dom';
import { MetamaskStateProvider } from "use-metamask";
import App                       from './App';

ReactDOM.render(
    <MetamaskStateProvider>
      <App />
    </MetamaskStateProvider>
  document.getElementById('root')
);

You can also see _app.js file that I shared as nextjs example with you.

Hi, yes I have wrapped it over pages/_app.js

Hi, yes I have wrapped it over pages/_app.js

That's interesting. Is there any chance I can see your implementation if it's open source?
The only way I was able to reproduce your issue, when I don't use MetamaskStateProvider wrapper, thus;
  const dispatch = useContext(MetaDispatchContext);

in my hook implementation returns undefined. The component you call useMetamask hook must be child or descendant of this wrapped root component. Other than that I could not find any other case this can happen.

Hey @damianlluch! Is it still a problem on your side? Is there anything I can help there?

Closing this issue due to inactivity, feel free to open if anything related this topic arise.