The generated create mutation are missing foreigners ids

My schema:
type Foo {
  id: ID!
  title: String!
  bars: [Bar]!
}

type Bar {
  id: ID!
  description: String!
}

and this is the generated mutation for createBar
mutation createBar($description: String!) {
  createBar(input: {description: $description}) {
    ...BarFields
  }
}

fragment BarFields on Bar {
  id
  description
}

and the error when I perform the mutation:
[GraphQL error]: Message: Field BarInput.fooId of required type ID! was not provided., Location: [object Object], Path: undefined
[Network error]: ServerError: Response not successful: Received status code 400

the correct mutation should be this:
mutation createBar($fooId: ID!, $description: String!) {
  createBar(input: {fooId: $fooId, description: $description}) {
    ...BarFields
  }
}

Graphback: 0.10.0-rc2

@wtrocki no it was broken also in the previous version

Fixed in https://github.com/aerogear/graphback/pull/609

@craicoverflow is this fixed int the 0.10.0 or in the 0.10.1?

@craicoverflow is this fixed int the 0.10.0 or in the 0.10.1?

0.10.1 which should be published soon (its failing at the minute):
https://circleci.com/gh/aerogear/graphback/1950?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

@craicoverflow testing this out I find that it's working for one to one but not for one to many. I'm reopening it to track it.
If you are already fixing this but in another issue just close it again

@craicoverflow testing this out I find that it's working for one to one but not for one to many. I'm reopening it to track it.
If you are already fixing this but in another issue just close it again

Ah yeah this is because you need to explicitly add the field to the type on the M:1 side in order for it to be generated.
So
type Foo {
  id: ID!
  title: String!
  bars: [Bar]!
}

type Bar {
  id: ID!
  description: String!
}

generates the following schema:
type Foo {
  id: ID!
  title: String!
  bars: [Bar!]!
}

type Bar {
  id: ID!
  description: String!
}

So if you want the field to added to create mutation you need to do this:
type Foo {
  id: ID!
  title: String!
  bars: [Bar!]!
}

type Bar {
  id: ID!
  description: String!
  note: Note
}

which creates the mutation as we want it:
mutation createBar($description: String!, $noteId: Int!) {
    createBar(input: {description: $description, noteId: $noteId}) {
      ...BarFields
    }
  }

@craicoverflow testing this out I find that it's working for one to one but not for one to many. I'm reopening it to track it.
If you are already fixing this but in another issue just close it again

Ah yeah this is because you need to explicitly add the field to the type on the M:1 side in order for it to be generated.
So
type Foo {
  id: ID!
  title: String!
  bars: [Bar]!
}

type Bar {
  id: ID!
  description: String!
}

generates the following schema:
type Foo {
  id: ID!
  title: String!
  bars: [Bar!]!
}

type Bar {
  id: ID!
  description: String!
}

So if you want the field to added to create mutation you need to do this:
type Foo {
  id: ID!
  title: String!
  bars: [Bar!]!
}

type Bar {
  id: ID!
  description: String!
  note: Note
}

which creates the mutation as we want it:
mutation createBar($description: String!, $noteId: Int!) {
    createBar(input: {description: $description, noteId: $noteId}) {
      ...BarFields
    }
  }

We have created another issue to cover this problem:
https://github.com/aerogear/graphback/issues/614


Dev Mode: Store model state on update

If we allowed a dev mode that stored the model state on update, than a hot reload would not reset the page to the beginning.

So I think this is just two things:

Cmd - HotReload.boot(enabled = true|false, key: String, decode: String => Model) used during init
Sub - `HotReload.snapshot(enabled = true|false, interval = 1.second, encode: Model => String) as a subscription.

And basically they both just use local storage.
You'd be limited in terms of the amount of data but is that really a problem?


bug: Menu can not be closed by clicking backdrop

Bug Report
Ionic version:
[x] 4.x
Current behavior:
When clicking the backdrop area of an overlay menu, the menu is not closed in MS Edge.  Other browsers like Chrome, Firefox or Safari are working properly.
Expected behavior:
The menu should be closed.
Steps to reproduce:
Open an app with a menu in MS Edge, open the menu, click the backdrop area.
Related code:
The issue can be reproduced even in the example of the official documentation:
https://ionicframework.com/docs/api/menu

Thanks for the issue. I am going to close this as we recently released Ionic Framework v5 which dropped support for IE11 and older versions of Edge. Moving forward, Ionic Framework v4 will only receive critical security patches.


Logging drivers to be able to deliver log messages as is

Docker logging drivers is a fantastic feature. However, I would like some logging drivers (such as the Splunk driver) to deliver log messages as is, i.e. without wrapping them in a JSON object. This indeed means we will lose the extra metadata like tags and labels. The issue with wrapping the raw log message is that we need to reconfigure the recipient to unwrap the message. This can be inefficient with high volume situations and adds complexity for DevOps. A "--log-opt" option to disable logging metadata would be useful in my opinion.

This looks like a duplicate of https://github.com/docker/docker/issues/17020, which was rejected, so unfortunately, I'll have to close this one, but feel free to comment after I closed

I am afraid you haven't really understood the scenario I described. I have seen that other issue which was closed, but what I proposed is not the same.
As I mentioned, I would like a logging driver such as the Splunk driver to NOT wrap the raw message within a JSON object. I am not interested in using a plain text driver such as syslog because it won't be forwarding messages to Splunk.
For example, a log message forwarded to Splunk by the Splunk logging driver looks similar to this:
{"line":"2016.07.21 15:23:57 samples.xmpp.ClientSession - Session created for user [abcd]","source":"stdout","tag":"443f6ddafd36"}
Instead of the above, I would like the Splunk driver to forward the raw log message:
2016.07.21 15:23:57 samples.xmpp.ClientSession - Session created for user [abcd]
I hope this explanation helps for this requested feature to re-evaluated.
Thank you.

@LakshithadSilva I think you may be in luck; there's an open pull request for the Splunk driver; https://github.com/docker/docker/pull/24652

Awesome, exactly what I wanted! I hope this pull request will go into the next Docker release :)
Thanks again for your quick response @thaJeztah. Much appreciated.


[Issue #2101] Use WEBAPP_VERSION in documentation link

Fixed #2101.
Will link to latest if version contains -dev.
I also made the URL https.

Codecov Report

Merging #2102 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2102      +/-   ##
==========================================
+ Coverage   76.16%   76.17%   +<.01%
==========================================
  Files          80       80
  Lines        7813     7814       +1
  Branches     1544     1544
==========================================
+ Hits         5951     5952       +1
  Misses       1609     1609
  Partials      253      253

Impacted Files
Coverage Δ

webapp/graphite/settings.py
76.68% <100%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for fixing that, @wridgers !
Merging.


mismatch between installed package and package called in init to set the version

The cookiecutter sets the package name using cookiecutter.repo_name here: https://github.com/MDAnalysis/cookiecutter-mdakit/blob/main/{{cookiecutter.repo_name}}/pyproject.toml#L9
but then sets the version using cookiecutter.package_name here: https://github.com/MDAnalysis/cookiecutter-mdakit/blob/main/{{cookiecutter.repo_name}}/pyproject.toml#L9
In some cases cookiecutter.repo_name != cookiecutter.package_name (for example when @orbeckst created mdgeomkit, repo_name=mdgeomkit, and package_name=mdgeom). When this occurs the init version setter fails because it can't find the installed package.

I believe this was fixed in #116.


aws-ebs-csi-driver component can't connect to EC2 instance metadata

It currently crashes with the following error:
+ kubectl logs -f ebs-csi-controller-585d6d6f56-hgn5d -c ebs-plugin
I0826 15:01:28.649833       1 driver.go:62] Driver: ebs.csi.aws.com Version: v0.5.0
panic: EC2 instance metadata is not available

goroutine 1 [running]:
github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver.newControllerService(0xc000191380, 0xc00001e960, 0x0, 0x16)
        /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver/controller.go:76 +0x103
github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver.NewDriver(0xc000115f70, 0x3, 0x3, 0xc000062900, 0xd4d780, 0xc000191260)
        /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/pkg/driver/driver.go:82 +0x3d9
main.main()
        /go/src/github.com/kubernetes-sigs/aws-ebs-csi-driver/cmd/main.go:31 +0x117

My guess is that EC2 instance metadata blocking has been implemented after this component has been added + there is a bug in the controller, that it reports that it's ready, even though it's not, which tricks our CI that the component is healthy.

Reported upstream: https://github.com/kubernetes-sigs/aws-ebs-csi-driver/issues/548.


Eventually consistent global state

I'm thinking it can be useful if there was a way to create some kind of shared object between all peers and a mechanism to synchronize it.
For example, a bloom filter with an expiration date(while the previous one is valid (not expired) system tries to  build the next one.) that stores IDs of all online users. This way we can know in advance if some peer is unavailable.
Removed nonsense

Or a website state snapshot. Can be really great with something like Om    server can sign  webapp's state object and the p2p network will distribute it. Or with JWT...  Sky is the limit :smile:


⚠ Bofrost.fr has degraded performance

In aeb387c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 996 ms

Resolved: Bofrost.fr performance has improved in cb304a6 after 7 minutes.


Update docker-opentelemetry.md

removing duplicate command from example.
Purpose of this pull request
This pull request...
Select the type of change

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

thanks, @bradtho ! I'll add you to our CLA approved list and then I'll merge your PR.


App hangs on use of Actions.refresh()

Version
Tell us which versions you are using: 4.0.0-beta.24
Expected behaviour
App should not hang after rendering view using Actions.refresh(); on NavigationBar after performing some action such as API call.
Actual behaviour
App hangs after rendering view using Actions.refresh(); on NavigationBar after performing some action such as API call / in-app logic.
Error: Maximum call stack reached
XHR finished loading: POST "http://localhost:8082/callback_result".
XHR finished loading: POST "http://localhost:8082/call_method".

@simonso3 I encountered this when calling Actions.refresh() both on componentDidMount and componentWillReceiveProps. Former is to set a default navigation bar component, latter is to update it. If Actions.refresh() fired too quickly, RNRF will loop in Navigation/SET_PARAMS and cause app to freeze.

Use setTimeout for this.


Delayed vesting account changes

Codecov Report

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

@@           Coverage Diff           @@
##             main     #264   +/-   ##
=======================================
  Coverage        ?   18.48%
=======================================
  Files           ?      134
  Lines           ?    20959
  Branches        ?        0
=======================================
  Hits            ?     3875
  Misses          ?    16449
  Partials        ?      635

Continue to review full report at Codecov.

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


leveldb iterator is invalid

Running an experiment with reutersidf10k dataset via python dec.py reutersidf10k fails with an error
I0721 18:03:54.192920 24690 net.cpp:67] Creating Layer data
I0721 18:03:54.192975 24690 net.cpp:358] data -> data
I0721 18:03:54.193002 24690 net.cpp:96] Setting up data
I0721 18:03:54.193032 24690 data_layer.cpp:45] Opening leveldb reutersidf10k_total
python: db_iter.cc:68: virtual leveldb::Slice leveldb::<unnamed>::DBIter::value() const: Assertion `valid_' failed.

The same error occurs with the mnist and the reutersidf datasets. I haven't tried with the stl dataset.
I compiled the Caffe library provided in this repo, downloaded the dataset, and ran python make_reuters_data.py as given in the instruction.
The error happens when iter_->value() is called in: https://github.com/piiswrong/dec/blob/master/caffe/src/caffe/layers/data_layer.cpp#L122
It seems https://github.com/piiswrong/dec/blob/master/caffe/src/caffe/layers/data_layer.cpp#L53 creates an invalid iterator; putting a line CHECK(iter->Valid())  immediately after it fails.
My leveldb version is 1.0.7

@yalesong did you solve this problem? I am being bewildered with it.

@lipond No I haven't. Are you getting the same problem?

When  python dec.py mnist, do you have the error : AttributeError: type object 'Net' has no attribute 'set_phase_train'

@hukangrong It appears that you are importing the official, public version of the CAFFE library. This repo contains a modified version of CAFFE and you would have to compile & import that one instead.

@yalesong Thank you for your reply. Compiling it need much time. I would change the author code rather than build it again.


Update everything to Go 1.12

It appears as though some of our stuff is being built with Go 1.12 while some of our test jobs are using Go 1.11. We need to be using a consistent tool chain for everything.
Check out https://github.com/istio/istio/pull/14410 for evidence of this.   It looks like you have the work under control.  If you need any help, please reach out.  It seems like from the other issue tracker, prow may need attention as well.  Could you comment?
Cheers
-steve

hmm most of our prow testing/build is using this image:
gcr.io/istio-testing/istio-builder:v20181008-db31a9fd
Which has: go version go1.10.4 linux/amd64
I will need to figure out which tests are using go1.11 instead.. The last time I tried to upgrade the image to use go.1.12(with KinD package included), e2e-simpleTests test was failing. Let me create a new image with go.1.12(without KinD package) and update the prow tests and see the impact.
Do we want to update ALL branches' tests to use go.1.12? or only specific branch?

@icygalz I am assuming that we need to do master first and then cherry-pick back to 1.2 as this issue has a ``1.2 milestone.

@icygalz The failure I saw with prow tests was I had made a Makefile change to GO_VERSION_REQUIRED:=1.10 making it 1.12. It failed all prow tests IIRC (as prow was at 1.10.4 as you mention).

We need to have everything consistently building with and single compiler suite. In particular, all tests must be built with the same tool suite as the product. I think that means we want everything we do in release-1.2 and in master in every repo to be using Go 1.12.5

With a few PRs: https://github.com/istio/istio/pull/14610, https://github.com/istio/test-infra/pull/1310, and https://github.com/istio/test-infra/pull/1314, I believe we are now using Golang 1.12 everywhere in the master branch.
To get things using Golang 1.12.5 in the release-1.2 branch I think we need to cherry-pick https://github.com/istio/istio/pull/14610 forward so tests will pass and then we can merge https://github.com/istio/test-infra/pull/1298.
@linsun @fpesce I see this is labeled a release 1.2 item. Should we move forward with getting the PRs merged (in the correct order) in release 1.2?

Given how close we are to release 1.2, I'm in favor of not porting this to 1.2 unless we know of a user impact.  I think it is great we have got this fixed in master!!!
@fpesce pls chime in if you disagree. :)

@ericvn I don't think we have 1.12 everywhere yet, our releases are still in 1.10
$ dk run gcr.io/istio-release/pilot:master-20190610-09-15 version
version.BuildInfo{Version:"master-20190610-09-15", GitRevision:"b034753e104adff9faad7969d9246a2157fa6bd1-dirty", User:"root", Host:"6194d580-8b60-11e9-a7d1-0a580a2c6405", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.7-989-gb034753"}

It would be great if we could get master builds built on 1.12 so we can get some testing before 1.3. And distroless (not related to this)

The main advantage of golang 1.12 is improved performance.  The main disadvantage of moving to golang 1.12 for 1.2 is no serious testing has been done with Istio running with the golang 1.12 toolchain.
Doesn't seem to be worth the risk to me.
Cheers
-steve

@howardjohn Let me look into this one some more based on what you posted. I'm wondering about the GitTag. Need to investigate part of the build more.

GitTag one is tracked in https://github.com/istio/istio/issues/13602. Should we fixed as well though.

OK. Found the issue. The image we thought should have Golang 1.12 in it for the build doesn't so will try moving the build again (but first in staging branch). As this issue now seems to me master/1.3, I'll lower the priority a bit to get some of the 1.2 doc/testing done.

@ericvn are you saying that the build image we use for the 1.2 branch does not use Golang 1.12 so this isn't a blocker for the 1.2 release?
What version of Golang is used for the build images used for the 1.2 branch?   1.11?

@duderino Right now, the images used to build both the master branch and the release-1.2 branches use Golang 1.10.4. We moved the master branch to an image that should have contained Golang 1.12.5, but because of some of the issues with the CI pipeline, that is no longer the case. As John noted above, master is still using Golang 1.12.5. We have created a new build image and today we are going to try building the current master code in a staging branch. If tests and builds pass, we will attempt to merge that into master. I hope that makes sense. Then we could entertain moving the release-1.2 branch if so desired.

https://github.com/istio/istio/pull/14757 was created (using prow-staging brach) to build/test.

@ericvn thanks for the update
Both 1.2 release managers (@linsun and @fpesce) voiced concern about updating 1.2 to go 1.12 so close to the release.  We thought we were in a bind that we were testing with 1.12 but building with an earlier version.  So the fact that 1.2 still uses an older version of go is great news.
Please do not update go for the 1.2 branch.  This is a great thing for master and, in the future, 1.3.
I'm re-prioritizing accordingly

@duderino I believe we are testing with Golang 1.12.5 and building with Golang 1.10.4 in release-1.2 branch (have to go back and verify that).
As of earlier this morning with https://github.com/istio/test-infra/pull/1334, we should be totally on Golang 1.12.5 in master.

prow uses go1.10.4 for building and testing release-1.2.
I am not sure about circleci.

circleci uses Golang 1.12.5 in both master and release-1.2.

Based on the @duderino comment that this should be done in master and we shouldn't do this in release-1.2 (even though the milestone target is still 1.2), I will close this issue.
Note that the work done in the PRs was to update the builds to using Golang 1.12.5, but does not include migrating things to using go modules during the builds instead of vendor. There are other on-going discussions like https://github.com/istio/istio/issues/14795 which discuss this issue.
Please re-open if you think there is additional work needed in this issue.


Workaround: Exclude cudf_log.txt in RAT check

Printing headers for text files without a valid license header...
=====================================================
== File: /home/jenkins/agent/workspace/jenkins-rapids_premerge-github-10377/sql-plugin/cudf_log.txt

Currently the output of cudf logger is failing blossom-ci, try exclude it from the repo to unblock CI for now.

build

build

build


object has no attribute 'tenant'

I had the error object has no attribute 'tenant', how resolve?

Hello @sadson ! Thank you for the bug find. I've fixed the issue, you can pull the latest code from pypi.


allow passed children as node type

Please let me know how you think about this change.

@ngovietbao Cannot do that change. Clamping an element will cause error when rerender. #6


DoH blocked?

Description
The REQUEST-949-BLOCKING-EVALUATION.conf file (ID: 949110) seems to block DoH requests. If I remove the file, everything works. If I do the same requests on another host which does not proxy to a DoH server, I get a 404, no 403, so the problem seems to come from the response of the DoH server.
How to reproduce the misbehavior (-> curl call)
No Idea how a curl command could look like, but using doh-cli
doh-cli --url https://dns.domain.de/dns-query www.example.com A on a nginx host (with this ruleset), which proxies to an adguardhome server results in: 403 Client Error: Forbidden for url: https://dns.domain.de/dns-query?dns=r2wBAAABAAAAAAAAA3d3dwdleGFtcGxlA2NvbQAAAQAB
Logs
ModSecurity: Access denied with code 403 (phase 2).
In your debug.log, there is this part:
[168451336790.523740] [/dns-query] [4] (Rule: 920420) Executing operator "Rx" with param "^[^;\s]+" against REQUEST_HEADERS:Content-Type.
[168451336790.523740] [/dns-query] [9] Target value: "application/dns-message" (Variable: REQUEST_HEADERS:content-type)
[168451336790.523740] [/dns-query] [7] Added regex subexpression TX.0: application/dns-message
[168451336790.523740] [/dns-query] [9] Matched vars updated.
[168451336790.523740] [/dns-query] [4] Running [independent] (non-disruptive) action: setvar
[168451336790.523740] [/dns-query] [8] Saving variable: TX:content_type with value: |application/dns-message|
[168451336790.523740] [/dns-query] [4] Rule returned 1.

It seems that rule 920420 triggered, which increases the anomaly score by 5 - this is enough to trigger the rule 949110.
Perhaps you should set your own allowed_request_content_type variable in crs-setup.conf.

so the problem is that the content type is blocked? and no, nginx only logs the part I sent at the beggining. But another question, below the lines you mentiioned arfe http version listed and they don't contains http3, could this also cause a block, since I compiled nginx with http3 support?

so the problem is that the content type is blocked?

Yes, you need to allow application/dns-message content type, which you can do in crs-setup.conf, as @airween already mentioned.

But another question, below the lines you mentiioned arfe http version listed and they don't contains http3, could this also cause a block, since I compiled nginx with http3 support?

You did a HTTP2 request, see: request: "POST /dns-query HTTP/2.0"

You did a HTTP2 request, see: request: "POST /dns-query HTTP/2.0"

yes in this example, but would it mean that HTTP/3 requests would be fast blocked?

so the problem is that the content type is blocked?

Yes, you need to allow application/dns-message content type, which you can do in crs-setup.conf, as @airween already mentioned.

will this be something I need to add forever now to the crs-setup.conf, or will this be added to the coreruleset at some point?

You did a HTTP2 request, see: request: "POST /dns-query HTTP/2.0"

yes in this example, but would it mean that HTTP/3 requests would be fast blocked?

It depends on your configuration, see crs-setup.conf.

so the problem is that the content type is blocked?

Yes, you need to allow application/dns-message content type, which you can do in crs-setup.conf, as @airween already mentioned.

will this be something I need to add forever now to the crs-setup.conf, or will this be added to the coreruleset at some point?

You need to add it to your configuration, it won't be part of CRS.

so the problem is that the content type is blocked?

Yes, you need to allow application/dns-message content type, which you can do in crs-setup.conf, as @airween already mentioned.

will this be something I need to add forever now to the crs-setup.conf, or will this be added to the coreruleset at some point?

You need to add it to your configuration, it won't be part of CRS.

thanks, adding application/dns-message to the crs-setup.conf fixed it.

You did a HTTP2 request, see: request: "POST /dns-query HTTP/2.0"

yes in this example, but would it mean that HTTP/3 requests would be fast blocked?

It depends on your configuration, see crs-setup.conf.

You did a HTTP2 request, see: request: "POST /dns-query HTTP/2.0"

yes in this example, but would it mean that HTTP/3 requests would be fast blocked?

It depends on your configuration, see crs-setup.conf.

Will HTTP/3 be added to coreruleset at some point?

if I set tx.allowed_http_versions to HTTP/3, will it be added to the default or will it override the default?

if I set tx.allowed_http_versions to HTTP/3, will it be added to the default or will it override the default?

It will override the default so you probably want to keep all versions.

is it possible to add it without override?

No.

Will HTTP/3 be added to coreruleset at some point?

See https://github.com/coreruleset/coreruleset/pull/3218.


[10.x] Added FQDN(fully qualified domain name) validation

Just added FQDN ('fqdn') validation rule to the list of rules in src/Illuminate/Validation/Concerns/ValidatesAttributes.php
FQDN stands for Fully Qualified Domain Name. For example subdomain.domain.com or just domain.com
https://en.wikipedia.org/wiki/Fully_qualified_domain_name#:~:text=A fully qualified domain name,domain and the root zone.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


fails to import in the browser because process is not defined

Hi, when I try to import this in my browser (building with esbuild) I immediately get an error because in the browser, process is not defined: https://github.com/kumavis/browser-process-hrtime/blob/38920d3d06ddecf0ad1b51f0f6872db946e3c035/index.js#L1
Shouldn't there be an if statement that checks for process being defined, so that this module can be used in the browser? I'm not sure how others have gotten this to work or what I'm missing. Thanks!

try https://github.com/cabinjs/browser-hrtime

try https://github.com/cabinjs/browser-hrtime


datastore: proper handling of unexported anonymous struct fields

Given that bigquery will soon be encoding/decoding data into user structs (https://github.com/GoogleCloudPlatform/google-cloud-go/issues/238) with proper handling of unexported anonymous struct fields, eg:
type A struct {
	b
}

type b struct {
	C string
}

myA := A{b: b{C: "hello"}}

will (and should) be marshaled like:
{"C":"hello"}

then we should now fix the datastore package, which currently marshals the above as:
{}

even though this will change the behavior for users. We should be consistent with bigquery, and we should fix this broken behavior before it's too late.
@zombiezen @jba thoughts?

+0.5, especially if we can leverage the new fields package.

I think it would be great if you fixed this. Very soon. Should be easy with the fields package.


feat: tensor algebra of free module over integral domain is a domain

Provide instances:

Nontrivial (TensorAlgebra R M) when M is a module over a nontrivial semiring R
NoZeroDivisors (FreeAlgebra R X) when R is a commutative semiring with no zero-divisors and X any type
IsDomain (FreeAlgebra R X) when R is an integral domain and X any type
TwoUniqueProds (FreeMonoid X) where X is any type (this provides NoZeroDivisors (MonoidAlgebra R (FreeMonoid X)) when R is a semiring and X any type, via TwoUniqueProds.toUniqueProds and MonoidAlgebra.instNoZeroDivisorsOfUniqueProds)
NoZeroDivisors (TensorAlgebra R M) when M is a free module over a commutative semiring R with no zero-divisors
IsDomain (TensorAlgebra R M) when M is a free module over an integral domain R

Mentioned on zulip here

This is all very nice, thanks!
I wonder whether this result should be more "generic" in terms of (Two)UniqueProds for Lists, Multisets, Finsets, as TwoUniqueProds.instTwoUniqueProdsProdInstMul works for products.
In fact, using a Lex order on Lists and an arbitrary linear order on the underlying type X should give an argument very similar to the one for the MVPolynomials result.

I wonder whether this result should be more "generic" in terms of (Two)UniqueProds for Lists, Multisets, Finsets, as TwoUniqueProds.instTwoUniqueProdsProdInstMul works for products.

This should indeed work for TwoUniqueSums and Multiset, but none of the other types have an Add or Mul with the right semantics, so I don't think this can work there.

I tried that, but it needs a linear order.

I'll try this when I have some more time.  What I had in mind was to use a lex ordering on Lists and any linear order on the base type X, i.e. one that you obtain via a LinearExtension.
In the multiset case, I think that you are also extracting both the min and the max for a lex order on any linear extension.  Anyway, I may have some time to try this out tonight or tomorrow, but this is just a way of golfing further the PR: I am happy with the results, even if I think that the proofs can be streamlined.

This is what I had in mind, but it requires an extra import:
import Mathlib.Order.Extension.Linear

@[to_additive "Any `FreeAddMonoid` has the `TwoUniqueSums` property."]
instance FreeMonoid.instTwoUniqueProds {κ : Type*} : TwoUniqueProds (FreeMonoid κ) :=
  -- put an arbitrary linear order on `κ`
  let _ : LE κ := { le := (· = ·) }
  have : PartialOrder κ := {
    le_refl := fun _ => rfl
    le_trans := fun _ _ _ ↦ Eq.trans_le
    le_antisymm := fun _ _ a _ ↦ a }
  let _ : LinearOrder κ := inferInstanceAs (LinearOrder (LinearExtension κ))
  -- use the lexicographic order on `List κ` induced by the order on `κ`
  let LO : LinearOrder (FreeMonoid κ) := List.instLinearOrderList
  -- show that a `CovariantClass` holds for `List`
  have : CovariantClass (FreeMonoid κ) (FreeMonoid κ) HMul.hMul LO.lt :=
    ⟨fun a b c bc => List.Lex.append_left _ bc .. ⟩
  TwoUniqueProds.of_covariant_right

I'd argue that that approach is backwards; we shouldn't have to put a weird order on FreeMonoid, there should be a  lemma stated in terms of a suitable map f : FreeMonoid _ -> Y, where the order goes on Y.

I'd argue that that approach is backwards; we shouldn't have to put a weird order on FreeMonoid, there should be a lemma stated in terms of a suitable map f : FreeMonoid _ -> Y, where the order goes on Y.

I always find that this issue with random orders appearing is probably due to my initial input to UniqueProds.  This comes from the fact that a standard way of proving that UniqueProds holds for X, is to put an order on X and extract mins or maxs from Finsets of X.
However, the statement of UniqueProds makes no reference to a background order.  All of this is to say that, when you are putting an arbitrary order on something, just to prove UniqueProds, it is probably an indication that a previous lemma that I put in the library assumed an order, but it could have been generalised to something without an order!

Anyway, I think that this discussion highlights the need for an improvement of the basics of the UniqueProds file.  This improvement, however, is out of scope for this PR.
I am happy with the current PR!

There's a trivial refactor I can do:
@[to_additive]
theorem of_graded_mul {X G : Type*} [Mul X] [Add G] [IsRightCancelAdd G] [LinearOrder G]
    [CovariantClass G G (· + ·) (· < ·)] [CovariantClass G G (Function.swap (· + ·)) (· < ·)]
    {f : X → G} (hf : ∀ x y : X, f (x * y) = f x + f y)
    (hinj : ∀ a b c d : X, a * b = c * d → f a = f c → a = c ∧ b = d) :
    TwoUniqueProds X where
  uniqueMul_of_one_lt_card {A} {B} h := by
    sorry

and
/-- Any `FreeMonoid` has the `TwoUniqueProds` property. -/
@[to_additive "Any `FreeAddMonoid` has the `TwoUniqueSums` property."]
instance FreeMonoid.instTwoUniqueProds {κ : Type*} : TwoUniqueProds (FreeMonoid κ) :=
  TwoUniqueProds.of_graded_mul (List.length_append) (fun _ _ _ _ => by exact List.append_inj)

It doesn't seem terribly useful. Maybe there's a way to do it that fits in with the API around GradedMonoid.

There's a trivial refactor I can do:

That looks like a nice refactor to me.

There's a trivial refactor I can do:

That looks like a nice refactor to me.

79faa1b10c
I'm not sure "graded" is precisely the word I want here. It's suggestive, at least.

Reviewers, thank you very much for your work on this. It's been an education.
Do you have any further suggestions?

Do you think "grade" is ok? Can you see a better phrasing?
More docstrings?
All the rw's should be rewrite's, since the implicit with_reducible rfl is just a waste of time

note: the saving, in heartbeats, would be 67 / 3566 (1.9%)
note: I invite you to pass over this point without comment, inarguable though it may be; I'm a C++ programmer at heart, and we love our obsessive parsimony
note: every occurrence of "rewrite" in current Mathlib is by me

No more improvements have come to mind. @eric-wieser, any suggestions? Do you think this should be merged, and do you think it's ready? Anything else I should do (like ask another person to review)?

@adomani Do you remember why you called this lemma mulHom_image_of_injective in #6723? I can't see image in the statement and I think of_injective_mulHom might be a better name.

I've completed a refactor cuts off 18 lines and proves the more general
@[to_additive] theorem of_mulHom (f : H →ₙ* G)
    (hf : ∀ ⦃a b c d : H⦄, a * b = c * d → f a = f c ∧ f b = f d → a = c ∧ b = d)
    (uG : TwoUniqueProds G) : TwoUniqueProds H where

Merged. of_mulHom is great!

@adomani Do you remember why you called this lemma mulHom_image_of_injective in #6723? I can't see image in the statement (but the proof uses mulHom_image_iff) and I think of_injective_mulHom might be a better name.
theorem mulHom_image_of_injective (f : H →ₙ* G) (hf : Function.Injective f) (uG : UniqueProds G) :
    UniqueProds H

I think that your name is indeed better.  I suspect that the name that I gave referred to an older version of the lemma that might have contained an image...

maintainer merge


Set up Codecov

We need to use sudo: required for now, to work around travis-ci/travis-ci#9061. 🔧:shipit:

Codecov Report

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

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   92.14%
=========================================
  Files             ?        5
  Lines             ?     1337
  Branches          ?        0
=========================================
  Hits              ?     1232
  Misses            ?      105
  Partials          ?        0

Continue to review full report at Codecov.

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


setup: test for app

Please Review for this setup test with https://github.com/testing-library/dom-testing-library/issues/260.
if, approved. we can review 6-adding-subscription https://github.com/evilfactorylabs/nadya/pull/19 & basic test of shell.js https://github.com/evilfactorylabs/nadya/pull/20 .
thanks

Current deployment URL: https://nadya-app-kx8jjqka9.now.sh


any way to customise color?

thanks for creating react components for mdl. I am interested in using it. But I cannot see a way to customise color. As the cdn version of mdl can specify different color. Is it possible to make the css to work with different color like blur?

Unfortunately if you're using our custom css to fix an issue with the layout, it's not possible to customize the color. I'm trying to find a better solution to this

thanks!

Any updates about this case?

Not that I'm aware of.. But some people got something in this thread https://github.com/tleunen/react-mdl/issues/148


Add DLP samples

Node samples: https://github.com/googleapis/nodejs-dlp/tree/gapic-v2

LGTM. Let's wait for tests to pass.


[Bug]: Adding query.http.tls.enabled=true breaks jaeger-query

What happened?
I want to setup Jaeger with TLS enabled for a secure connection between Grafana and Jaeger as data source. When adding the option query.http.tls.enabled=true to my deployment yaml, the jaeger-query pod does not start. Collector and agent do start with the same option set to true.
Setting option query.http.tls.enabled=false works as expected. The option should be supported (https://www.jaegertracing.io/docs/1.36/cli/). Im guessing it's a bug in Jaeger. However, some help or examples if I did something wrong would be appreciated (I haven't found any documentation or examples on how to work with Jaeger + TLS).
Changing the strategy to "all-in-one" does allow the pods to start up, but there is still no TLS.
Steps to reproduce
Deploy Jaeger operator on K8s cluster. Deploy Jaeger (see Deployment configs).
Expected behavior
Jaeger-query pod starts and has enabled TLS on the server.
Relevant log output
{"level":"info","ts":1666868672.982716,"caller":"app/server.go:273","msg":"Starting HTTP server","port":16686,"addr":":16686"}
{"level":"error","ts":1666868672.9827507,"caller":"app/server.go:284","msg":"Could not start HTTP server","error":"open : no such file or directory","stacktrace":"github.com/jaegertracing/jaeger/cmd/query/app.(*Server).Start.func1\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/server.go:284"}

Screenshot
No response
Additional context
No response
Jaeger backend version
v1.36.0
SDK
No response
Pipeline
No response
Stogage backend
ElasticSearch 7
Operating system
No response
Deployment model
Kubernetes
Deployment configs
apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
  name: jaeger
  namespace: observability
spec:
  agent:
    strategy: Sidecar
    options:
      admin.http.tls.enabled: true
  query:
    options:
      query.http.tls.enabled: true
  collector:
    options:
      collector.http.tls.enabled: true
    maxReplicas: 5
    resources:
      limits:
        cpu: 100m
        memory: 128Mi
  storage:
    options:
      es:
        server-urls: https://elastic-es-http.elastic:9200
        tls:
          ca: /es/certificates/ca.crt
          enabled: true
          skip-host-verify: false
    secretName: jaeger-secret
    type: elasticsearch
  volumeMounts:
      - mountPath: "/es/certificates/"
        name: certificates
        readOnly: true
  volumes:
      - name: certificates
        secret:
          secretName: elastic-es-http-certs-public
          items:
            - key: ca.crt
              path: ca.crt
  strategy: production

Same issue here, very strange.

Is the deployment created? If yes, could you share the event log?

Deployment is created yes. kubectl events gives:
LAST SEEN   TYPE      REASON              OBJECT                                   MESSAGE
52s         Normal    Scheduled           pod/jaeger-collector-7b65ff566f-n5mn8    Successfully assigned observability/jaeger-collector-7b65ff566f-n5mn8 to ip-xxxxxxxxxxx.eu-central-1.compute.internal
50s         Normal    Pulling             pod/jaeger-collector-7b65ff566f-n5mn8    Pulling image "jaegertracing/jaeger-collector:1.35.2"
45s         Normal    Pulled              pod/jaeger-collector-7b65ff566f-n5mn8    Successfully pulled image "jaegertracing/jaeger-collector:1.35.2" in 4.268128423s
23s         Normal    Created             pod/jaeger-collector-7b65ff566f-n5mn8    Created container jaeger-collector
23s         Normal    Started             pod/jaeger-collector-7b65ff566f-n5mn8    Started container jaeger-collector
23s         Normal    Pulled              pod/jaeger-collector-7b65ff566f-n5mn8    Container image "jaegertracing/jaeger-collector:1.35.2" already present on machine
7s          Warning   BackOff             pod/jaeger-collector-7b65ff566f-n5mn8    Back-off restarting failed container
22s         Warning   Unhealthy           pod/jaeger-collector-7b65ff566f-n5mn8    Readiness probe failed: Get "http://[xxxxxxx]:14269/": dial tcp [xxxxxxx]:14269: connect: connection refused
52s         Normal    SuccessfulCreate    replicaset/jaeger-collector-7b65ff566f   Created pod: jaeger-collector-7b65ff566f-n5mn8
53s         Normal    ScalingReplicaSet   deployment/jaeger-collector              Scaled up replica set jaeger-collector-7b65ff566f to 1
52s         Normal    Scheduled           pod/jaeger-query-57f9948bcc-gzsc4        Successfully assigned observability/jaeger-query-57f9948bcc-gzsc4 to ip-xxxxxxxx.eu-central-1.compute.internal
50s         Normal    Pulling             pod/jaeger-query-57f9948bcc-gzsc4        Pulling image "jaegertracing/jaeger-query:1.35.2"
46s         Normal    Pulled              pod/jaeger-query-57f9948bcc-gzsc4        Successfully pulled image "jaegertracing/jaeger-query:1.35.2" in 4.45897922s
17s         Normal    Created             pod/jaeger-query-57f9948bcc-gzsc4        Created container jaeger-query
17s         Normal    Started             pod/jaeger-query-57f9948bcc-gzsc4        Started container jaeger-query
45s         Normal    Pulling             pod/jaeger-query-57f9948bcc-gzsc4        Pulling image "jaegertracing/jaeger-agent:1.35.2"
42s         Normal    Pulled              pod/jaeger-query-57f9948bcc-gzsc4        Successfully pulled image "jaegertracing/jaeger-agent:1.35.2" in 3.114319514s
42s         Normal    Created             pod/jaeger-query-57f9948bcc-gzsc4        Created container jaeger-agent
42s         Normal    Started             pod/jaeger-query-57f9948bcc-gzsc4        Started container jaeger-agent
17s         Normal    Pulled              pod/jaeger-query-57f9948bcc-gzsc4        Container image "jaegertracing/jaeger-query:1.35.2" already present on machine
7s          Warning   BackOff             pod/jaeger-query-57f9948bcc-gzsc4        Back-off restarting failed container
52s         Normal    SuccessfulCreate    replicaset/jaeger-query-57f9948bcc       Created pod: jaeger-query-57f9948bcc-gzsc4
53s         Normal    ScalingReplicaSet   deployment/jaeger-query                  Scaled up replica set jaeger-query-57f9948bcc to 1

Oh seems like the pod is crashing. Could you also provide the logs from your pod?

I've shared the relevant log output in the section Relevant log output in the original post. After that part the pod shuts down. Let me know if you need more logs than that and I'll dig them up

Ahh i see. It says "Could not start HTTP server","error":"open : no such file or directory". Looks like you need to define the path to the certs too, right?
https://github.com/jaegertracing/jaeger/blob/v1.36.0/pkg/config/tlscfg/flags.go#L60-L69

I did include a clientCA, and the error persisted. Are all the options listed in the url required to be set (e.g. tlsCert, tlsKey, tlsClientCA)?
I havent had time yet to test putting all options in, but will report here when I do.
What gets me confused is when I create the deployment with the all-in-one strategy, it does not show an error when I put       query.http.tls.enabled: true, and the pod and http server start without issue.

Adding tls.cert and tls.key worked, thanks. Though I am now running into https://github.com/jaegertracing/jaeger/issues/2976.
Even with a clean Jaeger installation. And even when rolling back to using just http.
As for the TLS part, it would be useful to have a more descriptive error message other than "no such file or directory"


Commands are not executed by my artisan scheduler

Hi,
Commands are not executed by my artisan scheduler since I've made updates in composer.json. Eveyrhing worked fine bedore. Is the version 7.0 could be used with Laravel 6.1 ?
If I manually execute the command it's working.

Do you know the solution for this kind of problem ?
Thanks for your help
Regards

As pointed in the readme file, 7.0 should not be used with laravel 6. Please use the correct version.
Besides that I don't see why your commands are not scheduled, it should work as expected. Maybe worth trying using version 6 of this package which corresponds to your laravel version.

Everything is working by upgrading to Laravel 7 and Nwidart package version 7. I think it's on my side nothing wrong in your package. Sorry for that ;)


Add support to de-register workflows and activities

I'm seeing some errors in the temporal workers regarding unknown activities (which were previously registered but no workflow uses anymore):
2021/05/14 15:54:24 util.go:27  Task processing failed with error%!(EXTRA string=Namespace, string=default, string=TaskQueue, string=jobs, string=WorkerID, string=1@temporalworker@, string=WorkerType, string=ActivityWorker, string=Error, *errors.errorString=unable to find activityType=Activity1. Supported types: [Activity2, Activity3])

This is happening on our staging environment since we test workflows in development there and we cannot just cleanup the database. If I list all the workflows with tctl, there is no workflow that is using Activity1.
There's really no way I can tell to avoid this problem. I don't want to register some previous unused activity just to avoid this error from happening if that activity is not going to be used (and I don't want it to be used).
I tried toying with tctl admin db scan and tctl admin db clean but I get a Cassandra connection issue. (I'm not using Cassandra or ES, I'm using Posgress). See: https://github.com/temporalio/temporal/issues/598

I think there might be some misunderstanding here.
You don't need to "de-register" activities explicitly, you do that by deploying a worker that doesn't register an activity during startup.
If your question is about what happens to the older workflows that are still using old activity that you've deleted then you have two options here:

Identify and terminate or reset all workflows that are using old activity.
Use versioning to branch your code and differentiate between old and new workflows. This way you would keep "old" activity around as long as workflows that are using it exist in the system and then you could go ahead and delete it fully once you only have new workflows that are no longer using it.
I'm going to resolve this issue as I don't see any work on the SDK that needs to be done for it, if you have more questions, please visit our community forum and post your questions there.


add android course track by harvard

What does this PR do?
This PR adds a free course to teach Android basis made by Harvard
For resources
Description
This is a track course about Android from Harvard.
Why is this valuable (or not)?
This is a course made by Harvard teaching Android.
How do we know it's really free?
At the 'CS50 Certificate' section you can read that you only need to pay for the certificate but the course is Free.
For book lists, is it a book? For course lists, is it a course? etc.
This is a course.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

I think you want https://www.edx.org/course/cs50s-introduction-to-computer-science
Please add author names where appropriate.

@eshellman  I updated the authors name, but I think that its important to keep this Track as a course. Specially because its a course and Its easier to people who are looking for Android material to find in a specific place.
What do you think?

If you invent a title for this, it will be hard in the future to avoid unintentional duplicates. Use the title given by the resource.

@eshellman  I updated the course with the youtube playlist and the correct title. What do you think?

Looks good. Thanks!


Use the simulator's frame rate for video rendering in the results ingestion pipeline

Following https://github.com/NextCenturyCorporation/MCS/pull/308, it would help a lot for the results analysis UI if the rendered videos used the true frame rate.

@bzinberg What specific difference are you expecting? I thought that they're already using the correct frame rate (20 FPS), but @deanwetherby and @brianpippin can correct me if I'm wrong.
For example, this gravity support scene, which lasts for 20 steps, has a 3 second video: http://<IP_ADDRESS>:3000/analysis?eval=Evaluation 3.5 Results&category_type=gravity support&test_num=18&scene=1

If they're already using the right frame rate then great!  I didn't know whether this was happening, and https://github.com/NextCenturyCorporation/MCS/pull/308#discussion_r615948943 mentions showing the framerate number in the UI, but want to also make sure the videos actually use the frame rate (presumably another place in the codebase to touch).  And of course, that any hard-coding of the 20FPS number in the analysis pipeline be adjusted to pull from the appropriate config.
It's all stuff that might already be tracked, but I just hadn't seen any issues about it.  Thanks @ThomasSchellenbergNextCentury for following up.

Thanks for keeping us honest Ben. Our ingest software is now updated with https://github.com/NextCenturyCorporation/mcs-ingest/pull/12


Route finder finds routes with invalid assignments.

It happens often:

Clear all data.
Load planes, jobs data.
Start route finder.
Click "add to my flight plan".
Invalid Assignment parameter

Sample failing route:
blob:https://fse-planner.piero-la-lune.fr/32e97ae3-0772-4fd6-a927-cd41224141f8

Hi, your example does not load for, could you share the PDF file instead of a link (I assume this is what you are sharing?).
Have you checked FSE ? Do you see the corresponding assignments in FSE (ie the button does not work but the assignments exist), or not (ie invalid assignments).
How long between loading the jobs and running the route finder? Could it be that another use has taken the assignments before you could have the chance to add them to your flight?

Hi, the pdf was from sever https://fse-planner.piero-la-lune.fr/. So, I don't have control overt it.

I seem to run into the same issue a lot of the time now. It seems to find jobs that don't seem to exist at some airports when it generates the route. In the PDF below, if you compare the first starting airport, the small sub 100KG loads do not exist and this was right after a fresh data pull and route generation.
debug.json
FSE Route.pdf

Are you certain the incriminating jobs (those that do not exist) are not in your "My Assignment" queue? When you add jobs to your assignments, FSE Planner now keep those in this queue, until they are marked as flown (or the queue is cleared). They will be used by the Route Finder, so if you have already flown those jobs but did not refresh the data in FSE Planner, you will encounter the issue as the Route Finder will try to make you fly those jobs again.


Fix rounding bug

This resolves #506

take fractions into account when dividing the remainder (largest fraction first)
add testcases
fixed some spec types

Just a note: The compare() use there is broken, see moneyphp#503. This will at least affect merging your PR and mine, but it's easily fixed.

Easy fix indeed 😄

Replaced by #526. In that PR conflict is solved and included example of #492


Thinking about adding ZSH completions repo. Thoughts?

https://github.com/zsh-users/zsh-completions

Already part of prezto since 15 days before you posted :-D maybe you only had to enable the module...


Audit deploy after processing incoming event

So that unauthorised deploys are caught. For now we only check if the deploy version is on master.
If deployed version is not on master we log a warning and send a notification to Slack.

ping @FundingCircle/black-bat please review

@dideler I think everything is addressed, except from starting delayed job conditionally. I think we can do it in the next iteration of this feature, would really like to have this in production to observe the behaviour.

LGTM :+1:

to confirm: the nodes are provisioned with the new Slack webhook env var.


Single record in Eager loaded relation

Im currently trying to eager load a relation on main table (Model).
By default it eager loads all records on relation.
My child relation has many record and I dont want all record to be loaded.
Infact i just want to load single last record (latest) record from this relation to be loaded.
is this possible with eager loading.?
See my post at SO

Got it resolved by:
in my model I added a custom relation:
public function latestDeviceLog()
{
    return $this->hasOne('Myhome\Myplugin\Models\Devicelog')->latest();
}

Now I can access with:
Device::with('latestDeviceLog')->get();


feat: change --save behavior from skip to Override

--save flag skips the Kptfile editing if it finds the fn in the mutators list. This PR changes the behavior to override the Kptfile with the new function arguments.
bug fix: modify the Kptfile read/write path.

related:

#2249
https://github.com/GoogleContainerTools/kpt/pull/2841#issuecomment-1081295669

+CC @morgante


Patch mapping for sets?

Do we depend entirely on external programs to map patches to specifc patches? If not, there needs to be a way to reorder the patches into sets in some intelligent way (with a config file I guess). This would also be a good place to map patches to "song names" or whatever.

I believe the controller should work like a synth: patch numbers are patch numbers. The addition of support for program change should allow external programs to handle this.


GraphQL query for Mirror token supply needs to be updated

Querying https://fcd.terra.dev/v1/circulatingsupply/mir returns:
{
  "type": "API_ERROR",
  "message": "400 - {\"errors\":[{\"message\":\"Cannot query field \\\"mirTotalSupply\\\" on type \\\"Statistic\\\". Did you mean \\\"mirSupply\\\"?\",\"locations\":[{\"line\":3,\"column\":13}],\"extensions\":{\"code\":\"GRAPHQL_VALIDATION_FAILED\"}},{\"message\":\"Cannot query field \\\"mirCirculatingSupply\\\" on type \\\"Statistic\\\".\",\"locations\":[{\"line\":4,\"column\":13}],\"extensions\":{\"code\":\"GRAPHQL_VALIDATION_FAILED\"}}]}"
}

Due to this error, CoinGecko currently fails to display MIR's circulating supply:

The cause of this is that FCD uses an outdated query here to fetch data from Mirror's GraphQL server:
# outdated
query statistic {
  statistic {
    mirTotalSupply
    mirCirculatingSupply
  }
}

This needs to be updated to
query statistic {
  statistic {
    mirSupply {
      circulating
      staked
      liquidity
    }
  }
}

Thanks @larry0x. It's fixed.


SampleGen: generate skeletons for C# standalone samples

Add a standalone_sample.snip for CSharp to start working on generating standalone samples, similar to this one.

@googleapis/samplegen @chrisdunelm PTAL

Codecov Report

Merging #2638 into master will increase coverage by 0.04%.
The diff coverage is 96.42%.

@@             Coverage Diff              @@
##             master    #2638      +/-   ##
============================================
+ Coverage     86.64%   86.69%   +0.04%
- Complexity     5554     5571      +17
============================================
  Files           466      467       +1
  Lines         22119    22175      +56
  Branches       2423     2427       +4
============================================
+ Hits          19165    19224      +59
+ Misses         2100     2097       -3
  Partials        854      854

Impacted Files
Coverage Δ
Complexity Δ

...i/codegen/viewmodel/StaticLangSampleClassView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...en/transformer/StaticLangApiMethodTransformer.java
95.82% <0%> (-0.19%)
63 <0> (ø)

...rmer/csharp/CSharpStandaloneSampleTransformer.java
100% <100%> (ø)
10 <10> (?)

...ansformer/csharp/CSharpGapicClientTransformer.java
97.09% <100%> (+0.19%)
44 <0> (-10)
:arrow_down:

...oogle/api/codegen/gapic/GapicGeneratorFactory.java
88.52% <50%> (-0.26%)
12 <0> (ø)

...transformer/csharp/CSharpApiMethodTransformer.java
95.41% <96.62%> (+2.39%)
17 <13> (+12)
:arrow_up:

...gle/api/codegen/transformer/SampleTransformer.java
98.62% <0%> (+0.68%)
26% <0%> (+1%)
:arrow_up:

...m/google/api/codegen/transformer/SurfaceNamer.java
73.43% <0%> (+0.89%)
214% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #2638 into master will increase coverage by 0.04%.
The diff coverage is 96.42%.

@@             Coverage Diff              @@
##             master    #2638      +/-   ##
============================================
+ Coverage     86.64%   86.69%   +0.04%
- Complexity     5554     5571      +17
============================================
  Files           466      467       +1
  Lines         22119    22175      +56
  Branches       2423     2427       +4
============================================
+ Hits          19165    19224      +59
+ Misses         2100     2097       -3
  Partials        854      854

Impacted Files
Coverage Δ
Complexity Δ

...i/codegen/viewmodel/StaticLangSampleClassView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...en/transformer/StaticLangApiMethodTransformer.java
95.82% <0%> (-0.19%)
63 <0> (ø)

...rmer/csharp/CSharpStandaloneSampleTransformer.java
100% <100%> (ø)
10 <10> (?)

...ansformer/csharp/CSharpGapicClientTransformer.java
97.09% <100%> (+0.19%)
44 <0> (-10)
:arrow_down:

...oogle/api/codegen/gapic/GapicGeneratorFactory.java
88.52% <50%> (-0.26%)
12 <0> (ø)

...transformer/csharp/CSharpApiMethodTransformer.java
95.41% <96.62%> (+2.39%)
17 <13> (+12)
:arrow_up:

...gle/api/codegen/transformer/SampleTransformer.java
98.62% <0%> (+0.68%)
26% <0%> (+1%)
:arrow_up:

...m/google/api/codegen/transformer/SurfaceNamer.java
73.43% <0%> (+0.89%)
214% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #2638 into master will increase coverage by 0.04%.
The diff coverage is 96.42%.

@@             Coverage Diff              @@
##             master    #2638      +/-   ##
============================================
+ Coverage     86.64%   86.69%   +0.04%
- Complexity     5554     5571      +17
============================================
  Files           466      467       +1
  Lines         22119    22175      +56
  Branches       2423     2427       +4
============================================
+ Hits          19165    19224      +59
+ Misses         2100     2097       -3
  Partials        854      854

Impacted Files
Coverage Δ
Complexity Δ

...i/codegen/viewmodel/StaticLangSampleClassView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...en/transformer/StaticLangApiMethodTransformer.java
95.82% <0%> (-0.19%)
63 <0> (ø)

...rmer/csharp/CSharpStandaloneSampleTransformer.java
100% <100%> (ø)
10 <10> (?)

...ansformer/csharp/CSharpGapicClientTransformer.java
97.09% <100%> (+0.19%)
44 <0> (-10)
:arrow_down:

...oogle/api/codegen/gapic/GapicGeneratorFactory.java
88.52% <50%> (-0.26%)
12 <0> (ø)

...transformer/csharp/CSharpApiMethodTransformer.java
95.41% <96.62%> (+2.39%)
17 <13> (+12)
:arrow_up:

...gle/api/codegen/transformer/SampleTransformer.java
98.62% <0%> (+0.68%)
26% <0%> (+1%)
:arrow_up:

...m/google/api/codegen/transformer/SurfaceNamer.java
73.43% <0%> (+0.89%)
214% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

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

Apologies for not reviewing yet, I've been busier than I expected. I'll do it tomorrow if you can wait another day.

@chrisdunelm No worries at all.

Thanks for reviewing!


Uploaded Mid Sem Papers ECE

Checklist

[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Added 4 midsem papers ece to the repository.

Thank you for the PR
I think its well enough to be merged.
Let others review as well.


Remove link to "browse source code"

This link is dead. I don't know what functionality it offered originally but it is no longer useful.

Current coverage is 93.35%

Merging #2115 into master will not affect coverage as of 2e301b3

@@            master   #2115   diff @@
======================================
  Files          566     566
  Stmts        35946   35946
  Branches      2315    2315
  Methods
======================================
  Hit          33557   33557
  Partial        761     761
  Missed        1628    1628

Review entire Coverage Diff as of 2e301b3

Powered by Codecov. Updated on successful CI builds.

This site was similar to https://source.roslyn.io, but for the code in this project.
@pdelvo is this (site being down) a temporary thing?

I tried to bring it back up but I have some troubles getting https to work. Maybe we can change the link to use http instead for now

I fixed the https version of the page.


Add File Explorer to app structure

Please go the the Preview tab and select the appropriate sub-template:

Applications
Others

Outdated


GPFS kafka package installation time improvement

Signed-off-by: Rajan Mishra<EMAIL_ADDRESS>

Hi Rjaan,
I looked at your code changes and it looks okay for me !


Implement note-based (pitch + duration) encoder/decoder.

The current melody model uses time-based encoder/decoders; each RNN step is one 16th note by default.  We should also try note-based encoder/decoders, where each RNN step is an event of the form "play note with pitch P for duration D".

Perhaps it could be resolved using sigmoid output with every pitch/note encoded as multi-class labels. That way we could keep the 16th note resolution which helps inform the RNN of rhythmic patterns


Updates to project tables

Dashboard

staff

applicant

Project tables

Filter - "missed report" etc

Looks good, happy for this to go to test

All acceptance criteria were visible and functional. Thanks! :boom:


update bam coverage calculation

@rfeng2023 there are some updates last week from Frank on RNA_calling.ipynb mainly with improved documentation but now results in merge conflict. I think it is easier for you to resolve it because your patch is a lot smaller than his. Would you please check out the latest version of RNA_calling.ipynb and manually apply your changes to that document, and send in a PR again? I'm closing this one for now. Thanks!


Add verbose (client predicted!) examine text to Gas Miners

About the PR

Adds a detailed examine text to Gas Miners so that their behavior can be better understood. The examine text shows if the miner is currently producing gas or not and shows all the information about the gas being mined.
Why / Balance

I think the positive feedback about weather or not the miner is actually producing gas is the most important part, but showing all the information that the miner actually uses to mine the gas turns the miner from a deeply mysterious object into a completely understandable one.
Technical details

I turned GasMinerComponent into a Shared component so that the examine text can be client-predicted. I don't think this exposes any overly-exploitable information to the client.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

GasMinerComponent is now Shared

Changelog

tweak: Gas Miners now have detailed examine text

This PR complements #30352

Not prediction examine text: 😔
Predicted examine text: 🥳 🥳 🥳 🥳
Looks great!

Will the examine text only show up when the player is in close range? If yes, is it possible to see it from outside the miner chamber through the window? Otherwise this would only be useful for ghosts.

it specifically has a 6 tile special range check instead of using standard examine range


protocol execution: fix party index

Introduced in 79610b4d96632c346e4384fcc8de4f47dc375755, this bug
resulted in miscalculated party indices starting from -1:
https://github.com/cryptobiu/MATRIX/blob/79610b4d96632c346e4384fcc8de4f47dc375755/Execution/fabfile.py#L111

It's not mistake. Take a look at this scenario:
Let's assume that we have 3 parties that act as clients and one server that coordinate between them, total of 4 different machines.
The server needs to be on air when the parties tries to connect., so he will be the first party, i.e. party_id= 0.  The clients id are running from 0 to 2.
because the server defined as party_id= 0 and the clients are also need to start from 0 the line is run('. ./%s %s %s' % (executable_name[idx], party_id - 1, values_str)) and not run('. ./%s %s %s' % (executable_name[idx], party_id , values_str))

Hi Lior,
the line I referenced above handled the case without a coordinating party.
However, you have fixed the off-by-one in ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f here:

https://github.com/cryptobiu/MATRIX/commit/ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f#diff-c437b24436406fb96d62b02c66bee442L115
https://github.com/cryptobiu/MATRIX/commit/ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f#diff-c437b24436406fb96d62b02c66bee442R123

Thanks!

With the changes from ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f, the parties.conf file is no longer uploaded to the servers if the (external) protocol is run without coordinating parties. See

https://github.com/cryptobiu/MATRIX/commit/ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f#diff-c437b24436406fb96d62b02c66bee442L113

and maybe

https://github.com/cryptobiu/MATRIX/commit/ef6ef8ad34ccce85cc581d86518d5ad0b7cd0a7f#diff-c437b24436406fb96d62b02c66bee442L109


Investigate Alt + Key sideeffect behavior for: Macos

ALT + T yields †. The current logic isn't able to process these special characters accordingly.

See https://angular.io/guide/event-binding#binding-to-keyboard-events

Closing since code :)


404 'Not Found' Error

In the closed issues for this page there is reference to a 404 'Not Found' error when trying to use the query generated by Time Series Insights in Power BI Desktop. A fix is shown and comments made about permission levels.
I am experiencing this error using the March 2020 version of Power BI Desktop. My IoTHub, Time Series Insights environment and Power BI Tenant are all located in South East Australia. I am using the same user account to create and operate the process and that account has Owner, Contributor and Reader roles (overkill probably!) for the Time Series Insights environment. I have tried the fix available but I understand that this is superseded by the December 2019 version of Power BI Desktop anyway and I am beyond the one.
Any help with this would be much appreciated.

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

ID: 83bec0a3-a1b6-2d48-6cc1-4087f5900e7a
Version Independent ID: 6eef57fd-f177-9dd1-45d7-71f3b227a9c9
Content: Connect your environment to Power BI - Azure Time Series Insights
Content Source: articles/time-series-insights/how-to-connect-power-bi.md
Service: time-series-insights
GitHub Login: @deepakpalled
Microsoft Alias: dpalled

@JackHorlock Thanks for the feedback! We are currently investigating and will update you shortly.
@lyrana Please check this issue and provide your feedback.

@lyrana I have repeated my test today using the April 2020 release of Power BI Desktop. The 404 error is exactly the same.

@JackHorlock - Thanks for testing this again, can you share the screenshot of the error you get? Also, do you have access to more than one AAD tenant?
@@Sandip-Raiyani and @shreyasharmamsft - Do we know what might cause this issue?

@deepakpalled - I've had the same issue on two tenants and from two TSI environments. I've tried in with everything based in Singapore and also South East Australia, I am in New Zealand.
I can't paste the screen shot of the error in this comment. I'll try getting it to you on GitHub

Here's the screen shot

Hey @JackHorlock ,

I checked our logs and  I can see that the queries made from this environment(Id starting with 206f0f03-) since past few days are Cold Store queries. The last Warm Store query was made on 6th May 2020. Can you please double check whether the Warm Store queries is still on? PBI connector currently works on Warm Store queries only.

If the Warm Store queries is on and if you used our Fix  anytime, make sure to delete  "C:\Users<username>\Documents\Power BI Desktop\Custom Connectors" folder and "Clear permissions for "Login into Azure Time Series Insights Preview"(if present). Go to Power BI Desktop - Go to File -> Options & Settings -> Data Source Settings -> Login into Azure Time Series Insights Preview."

If you are still facing the issue after double checking the above things, let's schedule a phone call sometime this week. It would be easier to debug the issue through screen share. Thanks.

Hi @Sandip-Raiyani ,
Thanks for getting back to me. I just checked the configuration of my TSI environment and warm store is only retained for the default 7 days. I have now changed this to 31 days and will spin up an IoT device simulation to get some new data into it. I can then test again.
I have used your fix previously but had already deleted the custom connectors folder. I have also now cleared permissions.
I'll post an update once I have retested with a screen shot of the error if needed. Thanks.

Hi @Sandip-Raiyani,
I have run some new data into my TSI solution now and managed to get a connection for both raw and aggregated data. It looks like clearing the permissions and making sure I was working with warm store did the trick.  Thanks for your help. I think you can close this issue now.

Thanks @JackHorlock . Happy to know that it worked out. Feel free to provide feedbacks and improvements required anytime.
@PRMerger9 @lyrana @deepakpalled  Can we close this issue? Who can close this issue? I can't see it from my side. Thanks.

#please-close


blender: 3.3.0 -> 3.3.1

Description of changes
Bumps blender to 3.3.1
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

blender
blender-hip
python310Packages.bpycv
python39Packages.bpycv

</details

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 198539 run on x86_64-linux 1

4 packages built:

blender
blender-hip
python310Packages.bpycv
python39Packages.bpycv

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

python310Packages.bpycv:

got build log for '/nix/store/fyinw5n7414wg8kql6amyixivvdr20lj-python3.10-bpycv-0.2.43' from 'daemon'
When evaluating attribute ‘python310Packages.bpycv’:
warning: unused-argument
Unused argument: fetchurl.
Near pkgs/development/python-modules/bpycv/default.nix:5:3:
  |
5 | , fetchurl
  |   ^

warning: unused-argument
Unused argument: writeText.
Near pkgs/development/python-modules/bpycv/default.nix:6:3:
  |
6 | , writeText
  |   ^

python39Packages.bpycv:

got build log for '/nix/store/7xygi865wjz5ndaxk8lz8874halrpbm1-python3.9-bpycv-0.2.43' from 'daemon'
When evaluating attribute ‘python39Packages.bpycv’:
warning: unused-argument
Unused argument: fetchurl.
Near pkgs/development/python-modules/bpycv/default.nix:5:3:
  |
5 | , fetchurl
  |   ^

warning: unused-argument
Unused argument: writeText.
Near pkgs/development/python-modules/bpycv/default.nix:6:3:
  |
6 | , writeText
  |   ^


cmd/go: remote build cache

What version of Go are you using (go version)?

$ go version
go version go1.15.3 darwin/amd64

What are you suggesting?
Go's build cache (GOCACHE) provides a hash-driven mechanism for storing intermediate compilation results. This hash takes into consideration a number of different variables (including Go versions, architecture, flags, paths, etc). As such, it should be possible to share the cache across different concurrent build jobs (both for developers and CI jobs).
There are a lot of different ways to share thousands of small files across machines, but for most people this will involve NFS (AWS EFS, GCP Filestore, etc). NFS is suboptimal for this use case (provisioning IO throughput across many machines is problematic, builds should not be punished for NFS outages, file interactions need to be atomic, metadata (stat) lookups are rarely cached, etc, etc).
Instead, this is a proposal to support URLs in GOCACHE. This would allow CI servers to have a simple, central HTTP server which would accept GETs for cache lookups and PUT/POST for cache writes. For distributed systems, local proxies off the central HTTP server would provide an additional speedup.
Why would it be valuable?
As anyone who has had to interact with NFS in their career would agree, this is a much simpler configuration than trying to mount a shared filesystem.
This would also lead to the possibility of using memcache/etc URLs in the future.
What are you looking for?
What is the process to officially propose this idea and have it accepted? If it was accepted, I would like some design constraints. I could take it from there and put together a prototype.

/cc @bcmills @jayconrod @matloob

Some prior art: Bazel supports remote caching over both HTTP and gRPC based protocols.
Some of the folks that worked on Bazel RBE prototyped this in the go command last year. We found that this had the potential to speed up long-running tests, since test results can be cached remotely. However, it's detrimental for builds. Most actions performed by the go command are very fast, and the overhead of sending lots of small files over the network tends to dominate any gains we might see.
Implementing remote caching and execution adds a lot of complexity to the go command. Since it would benefit a fairly small number of users, we chose not to go forward with it at that time.

I'd love to get the speed up for caching tests since were we spend alot of time. Is there a way to identify test cache results so we could slurp them up GOCACHE and then lay them back down before the next clean  build?  (maybe storing/retrieviing at git commit).
Having a seperate GO_TEST_CACHE might also make this pretty feasible.

@jayconrod - is there a public reference to the design and the attempt done by the RBE people?
I wonder if we can still utilize remote cache for CI builds done on ephemeral machines. For organizations working in monorepo and cloud-based CI this could be a real game-changer (given that the org does not want to adopt Bazel).
Is there a way to plugin to go cache so we can replace its implementation with some custom code? That would allow us to offer remote caching implementation while keeping the code of go slim and lean.

I'd also be interested in plugging into Go's caching. https://github.com/firebuild/firebuild implements a generic process cache working with a many compilers and reproducible commands, but not Go, because Go's build insists on using its internal caching.
https://github.com/firebuild/firebuild is a local cache that can be fine-tuned to not cache quick operations or filter by command.

See the proposal at #59719, which pulls most of the work out of cmd/go.

See #64876 for the followup


ListView Component's Text Alignment Properties

About PR

This PR adds two new designer properties into the ListView component for setting TextAlignment of Main Text and Detail Text.
Fixes Issue #2594

Note for Future Work

[ ] Designer Preview for TextAlignemrt Properties.

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test


Spikes in PID output

I am experiencing some odd spikes in the PID output. I am a bit of a novice on PID but, the image below doesn't seem like it should be right.  Would fluctuations in my temperature readings cause these large spikes?
I am in the process of trying to figure out the cause, but thought I would post here in case there are any obvious known causes.
I appreciate this is not much to go on, just want to flag the issue while I investigate.

The library output is smooth, whilst my logging and display are not.


Suggestion: Add an XML output

This could be convenient :o)

I cannot agree more, I'll see what I can do... :wink:

Here you go, reports can now be exported as XML files.
Invoke-PrivescCheck -Report MyReport -Format XML

This will produce the file MyReport.xml.
Here is an example with the check Invoke-BitlockerCheck. The standard output would be something like this:
+------+------------------------------------------------+------+
| TEST | HARDENING > BitLocker                          | INFO |
+------+------------------------------------------------+------+
| DESC | Check whether BitLocker is configured and enabled on  |
|      | the system drive. Note that this check will yield a   |
|      | false positive if another encryption software is in   |
|      | use.                                                  |
+------+-------------------------------------------------------+
[*] Found 1 result(s).

Name        : HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\BitLockerStatus
BootStatus  : 0
Description : BitLocker isn't enabled.

The same output in XML would be something like this:
<?xml version="1.0"?>
<Objects>
  </Object>
  <Object Type="System.Management.Automation.PSCustomObject">
    <Property Name="Id" Type="System.String">HARDEN_BITLOCKER</Property>
    <Property Name="Category" Type="System.String">Hardening</Property>
    <Property Name="DisplayName" Type="System.String">BitLocker</Property>
    <Property Name="Description" Type="System.String">Check whether BitLocker is configured and enabled on the system drive. Note that this check will yield a false positive if another encryption software is in use.</Property>
    <Property Name="Type" Type="System.String">Info</Property>
    <Property Name="Compliance" Type="System.String">N/A</Property>
    <Property Name="Severity" Type="System.String">Info</Property>
    <Property Name="ResultRawString" Type="System.String">

Name        : HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\BitLockerStatus
BootStatus  : 0
Description : BitLocker isn't enabled.

</Property>
</Objects>

This should be pretty easy to parse. The output XML document contains 1 or more Object elements, which all have the same attributes: an ID, a category (e.g.: Services), a name, a description, a type (Info or Vuln), a compliance result (OK, KO or N/A), a severity (Low, Medium, High) and finally the raw result of the check.

Can you confirm that's ok for you so I can close this issue?

Perfect :) Thanks !


New version: MLJLinearModels v0.2.0

Registering package: MLJLinearModels
Repository: https://github.com/alan-turing-institute/MLJLinearModels.jl
Created by: @tlienart
Version: v0.2.0
Commit: 202e058a2a6fe98871c74c85f3e0a8c1cd7d0a7e
Reviewed by: @tlienart
Reference: https://github.com/alan-turing-institute/MLJLinearModels.jl/commit/202e058a2a6fe98871c74c85f3e0a8c1cd7d0a7e#commitcomment-35708270

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


fix(modeling): expose all versions of hull functions

These changes expose all versions of the hull functions, including a new function to hullPoint3(), which will allow users to choose the level of functionality that best suites their needs.
All Submissions:

[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 update/change?
[X] Does your submission pass tests?

@platypii @hrgdavor this isn't 100% complete but wanted to get some feedback.

@platypii @hrgdavor please review. all comments welcome.


Adding resources samples

This PR includes:

Sample to create a resource group
Sample for deleting a resource group that shows manual polling usage

Can one of the admins verify this patch?


Fix: Correct List Ordering for Subnets and Address Blocks in IPAM Federation

Subnet and address block, When more than 3 realms are assigned, it causes ordering issue, this PR addresses this

In the AttrTypes for both subnet and address blocks , federation_realms is still defined as a types.ListType{ElemType: types.StringType} . Since we have changed it to unordered list , the new type should be internaltypes.UnorderedListOfStringType

can you please add/update acceptance tests for this, and also post the results of them passing?

Address block -

Subnet -


Can't Build Project for Release

I have an app using Tesseract which runs just fine in development, but when I want to archive it, I get a series of warnings:
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/liblept.a' of type archive.ar for architecture armv7
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/libtesseract_all.a' of type archive.ar for architecture armv7
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/liblept.a' of type archive.ar for architecture armv7s
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/libtesseract_all.a' of type archive.ar for architecture armv7s
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/liblept.a' of type archive.ar for architecture arm64
warning: no rule to process file '...../Pods/TesseractOCRiOS/TesseractOCR/lib/libtesseract_all.a' of type archive.ar for architecture arm64

And, of course, archiving fails due to undefined symbols for architecture.
Any tips on this?
I've read this can be caused by having the wrong files in "Compile Sources", but as far as I can tell, I only have .m files and core data models in there.

Hi Kevin,
I just created a brand new iOS 8.1 project in Xcode 6.1.1, added this line to a blank Podfile:
pod 'TesseractOCRiOS', '3.4.0'
and nothing else and did pod install. When I open my project Snap.xcworkspace, I see the following:

The framework files in the Pods project are red, because they refer to /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS7.1.sdk/System/Library/Frameworks/Foundation.framework
which does not exist since I only have iOS8.1 installed.
How can I make it work so that I can get to try the awesome sample?
Thank you!

Hi, @pwansch.
Could you try this #111 and tell us if it helps?

@pwansch, I created a new project using the same steps as you did, and although I also see those items in red like you do, building and running the project works perfectly fine (see screenshot below).
The only red items that refer to the iOS7.1-related path are Foundation.framework and UIKit.framework in Pods/Frameworks/iOS. This seems to be an issue with CocoaPods itself, but I can assure you that everything will still build correctly regardless. Here's the related issue on CocoaPods: CocoaPods/CocoaPods#2324
Screenshot of the 3.4.0 Cocoapod running fine using the same version of Xcode as you:

Thank you @kevincon. This perfectly answered my question.


Turn builtin signatures a(b, c) into Tokay-style syntax a : @b, c

Currently, this is a valid builtin definition:
tokay_function!("ast(emit, value=void, flatten=true, debug=false)", {
    ...
}

It is some kind of mix between Python/Rust and Tokay types, but I also don't know why I didn't do it right at first.
Anyway, this is the Tokay style of a signature definition:
tokay_function!("ast : @emit, value=void, flatten=true, debug=false", {
    ...
}

To make the relevance to Tokay more clean, I would recommend latter syntax to be used by all builtin functions. This requires the tokay-macros to be updated.

Additionally, when syntax is extended to allow for constant chaining, this could also be possible:
dict_get_item : dict_get : @dict, item, default=void

This would register the same function both as dict_get_item and as dict_get.


Subcommand infrastructure and NEST

This PR introduces a new command line structure for subcommands, which allows access to the old embed tool used for generating source files for plugins and modules, as well as the new fuse tool.
It also adds NEST, which was previously a separate utility. Once this PR is merged and released, the NEST repository will be put into archive mode.
This PR also removes the old jo_gif recorder functionality, as it doesn't work with the decoupled game loop at this time.

This PR is feature complete. Documentation needs to be updated to account for combined functionality in both the "Distribution" and "Plugin" guides.


How to solve raspberry pi

@zhongyuhong as you can see it into the logs, just install the FLAC
apt-get install flac
and after it if you find any other issues kindly do report


synchronize ecs-logging spec

What
ECS logging specs automatic sync
Why
Changeset

https://github.com/elastic/ecs-logging/commit/d3ffcf1 Adding service.environment to the spec (https://github.com/elastic/ecs-logging/pull/62)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-16T12:01:35.018+0000

Duration: 17 min 37 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
192

Skipped
0

Total
192

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Quantum v. Quantum Multiplayer mode ?

I think it would be cool to have a multiplayer mode where both players play as 3-qubit computers.

See new issue #55

No need to close this issue yet as #55 has been merged. Openning.


URL rebase not working as expected

I'm struggeling to get some url rebasing correctly with your plugin. Maybe I'm doing something obviously wrong, but I couldn't get the expected results even after trying for multiple hours.
I created a demo repository that should show what I'm trying to achieve:
https://github.com/mgreter/bug-gulp-concat-css
The directory structure is pretty simple:
/
  design
    src # source css files
      assets # css assets
    dst # generated stuff

I have one styles.css in the src folder and would like to "copy" it over to dst, while adjusting/rebasing the urls to point to the original locations. I was under the impression this is what rebaseUrls is supposed to do!?
src/styles.css:
BODY { background: url("assets/image.png"); }

Expected dst/styles.css:
BODY { background: url("../src/assets/image.png"); }

Actual results:
BODY { background: url("../bug-gulp-concat-css/assets/image.png"); }

I seem to be able to get this working if I do process.chdir("src").
But doing process.chdir is not an option for me, as it is:
a) not working if I only set it locally inside the gulp tasks (it needs to be global)
b) should be avoided as it affects all threads (unsure if v8 does a real posix chdir)
Also tried various other approaches (ie. giving cwd to gulp src/dest).
Maybe somebody could clarify if I'm just using it wrong or if this is a bug!?
Thanks

Did you try to set the base?
gulp.task('test', function () {
  var dst = gulp.src(['design/src/style.css'], {base: 'design'})
    .pipe(concatCss('dst/test-02.css'))
    .pipe(gulp.dest('design/dst'));
  return dst;
});

@mariocasciaro thx, just tried it but that doesn't give me the results I would expect.

- /src/css
- 1.css
.demo1 { background-image: url("../img/1.png"); }
- /sub/2.css
.demo2 { background-image: url("../../img/2.png"); }
        .pipe(concatCSS("main.css"))
        .pipe(gulp.dest(CSS_DIST_PATH))
        .pipe(rename({
            suffix: '.min'
        }))
        .pipe(cleanCSS())
        .pipe(gulp.dest(CSS_DIST_PATH))

- /dist/css
- main.css
- main.min.css
      /*main.css*/
      .demo1 { background-image: url("../img/1.png"); }
      .demo2 { background-image: url("../../img/2.png"); }

      /*main.min.css*/
      .demo1 { background-image: url("../img/1.png"); }.demo2 { background-image: url("../img/2.png"); }


Records are created despite requests timing out

I was doing some load testing for the app I'm building using Pocketbase as a framework and I came across this issue.
Under load, when creating records (users in this case) after ~60s some requests will timeout, but records are still processed and created in the db. I tried with Users but my guess is that this will happen with any request that need time to be completed. My expectation would be that if the create request times out the operations should not be completed or at least should be reverted.
Steps to reproduce:

Run Pocketbase in a small server. (if too fast will be more difficult to get it to 100%)
Run load test hitting /api/collections/users/records  with some random data
Test result will be something like: 219 complete and 184 interrupted iterations
Go to to the admin UI or open data.db - 403 records were created.

Example of my results:

$ sqlite3 /opt/pb/pb_data/data.db                                                                                130 ↵
SQLite version 3.37.2 2022-01-06 13:25:41
Enter ".help" for usage hints.
sqlite> select count(*) from users;
403

K6 Script I used
import http from "k6/http";
import { check } from "k6";
import { randomString } from "https://jslib.k6.io/k6-utils/1.4.0/index.js";

export const options = {
  scenarios: {
    open_model: {
      executor: "constant-arrival-rate",
      rate: 10,
      timeUnit: "1s",
      duration: "1m",
      preAllocatedVUs: 100,
      maxVUs: 2500,
    },
  },
};

export default function () {
  const url = "http://<IP_ADDRESS>:8090/api/collections/users/records";

  const pass = `${randomString(10)}`;
  const payload = JSON.stringify({
    username: `user_${randomString(10)}`,
    email<EMAIL_ADDRESS>    emailVisibility: true,
    password: pass,
    passwordConfirm: pass,
    name: "test",
  });

  const params = {
    headers: {
      "Content-Type": "application/json",
    },
  };

  const res = http.post(url, payload, params);

  check(res, {
    "response code was 200": (res) => res.status == 200,
  });
}

As I was quite surprised how log the requests were taking (barely achieving 10rps) I investigated a bit and basically all the time is spent on bcrypt

profile

As I was quite surprised how log the requests were taking (barely achieving 10rps) I investigated a bit and basically all the time is spent on bcrypt

Just to clarify, since I forgot this part: bcrypt is intended to be slow mostly to prevent brute-force attacks. We currently use a default round of 13, which is OK for most 2vCPUs+ but on lower hardware could be a little too much. I'll consider sometime in the future changing it to 12 (and/or maybe provide an option to adjust it during initialization).

thanks for your answers. I didn't relized


pip install ubelt incorrectly installs jaraco on linux

The jaraco module is only needed on windows for the symlink functionality. As such it should not be a requirement when installing ubelt on Linux. However, when you pip install ubelt (current pypi version is at 0.2.0), it will install this package. This is undesirable because it makes it appear that ubelt is more heavyweight than it really is.
I'm not sure why this is happening the requirement is specfied in requirements-win32.txt and setup.py should only see if sys.platform.startswith('win32'). Perhaps pypi is parsing the that file? Not exactly sure yet.

This should be fixed in #26

FIxed.


Difficulty with docs: Langchain?

While using Langchain is great (and many use it), it unfortunately isn't compatible with the feedback mechanism, because it's impossible to retrieve response headers. Because of that, 'helicone-id' header can't be retrieved. Is there any other way to still integrate the feedback functionality while using Langchain?
Path: /features/advanced-usage/feedback

Hi @ErwinAI Thanks for your message. I added docs on how to do this
https://github.com/Helicone/helicone/pull/956
You can see the docs here:
https://docs.helicone.ai/features/advanced-usage/feedback
But the idea is that you can create the request_id before hand

import uuid
from langchain.chat_models.openai import ChatOpenAI
import requests
import json
HELICONE_AUTH = "Bearer <API_KEY>"

def provide_feedback(heliconeId, rating):
    url = "https://api.hconeai.com/v1/feedback"
    headers = {
        "Helicone-Auth": HELICONE_AUTH,
        "Content-Type": "application/json",
    }
    data = {
        "helicone-id": heliconeId,
        "rating": rating  # True for positive, False for negative
    }

    response = requests.post(url, headers=headers, data=json.dumps(data))

request_id = str(uuid.uuid4())
ChatOpenAI.openai_api_base = "https://oai.hconeai.com/v1"
llm = ChatOpenAI(
    openai_api_base="https://oai.hconeai.com/v1",
    headers={
        "Helicone-Auth": HELICONE_AUTH,
        "Helicone-Request-Id": request_id
    }
)

llm_result = llm.predict("What is a Helicone")
provide_feedback(request_id, True)

@chitalian Ah gotcha, I did not know I can use the 'Helicone-Request-Id' to set an ID I build myself beforehand.
This works, thank you 🙇!

@ErwinAI no worries! Thanks for bringing it to our attention and helping us improve our docs


Checkbox help

I have tried many variations for getting a checkbox to show on load that it is true or false.
I am programatically setting the value.  I can see my value is true or false but when I add that to either checked or defaultChecked it does not show as checked when page render().  always shows unchecked.  ???
  or

It is uncontrolled so I was trying defaultChecked first.

Well we got it figure out.  Changed the DefaultChecked to value and that worked.


2 little bugfixes for git stree split

Hi, I just installed git-stree and start using it. How awesome !
Just found 2 little things I corrected in git stree split :

I first got an infinite loop.
For a reason I don't explain, the regexp match ~= always match, even where there is no .. in $path.
I replaced the loop with readlink

I got the git error "This command needs to be run in the root directory". So I encapsulate all with pushd/popd.

Just my 2c: The infinite loop issue is already known, see https://github.com/tdd/git-stree/issues/24. The readlink tool is not available on Git for Windows (at least on my installation).

Thanks for pointing that out, and referencing the bug (I admit I didn't think searching through the bugs)

Just pushed another commit, to avoid readlink :
for existing paths, a simple pushd in the directory will normalize the path.
I didn't analyze code to know if normalize_prefix can be called on non exiting paths, but if the answer is yes, then you can drop the two elsif/else.

Hey there,
We're rewriting the normalizing code and beefing it up with tests, so I'm not merging this in, but thanks for bringing the use case to our attention, we'll add it in the tests!

Hey there,
We're rewriting the normalizing code and beefing it up with tests, so I'm not merging this in, but thanks for bringing the use case to our attention, we'll add it in the tests!


Minor UX Changes

Changing the helper infer_compressor_from_model_config to ModelCompressor.from_pretrained to match more closely with QuantizationConfig.from_pretrained
Also adding a device default to decompress to fix some errors in the sparseml tests

@Satrat sweet. Let's get this in as well as https://github.com/neuralmagic/compressed-tensors/pull/41 and then push the new pypi package.

Or better yet, I merged those two PRs.


Failing to save changes after making changes on the visual editor

It doesn't happen every time but to reproduce it you can make some changes on the visual editor then hit Save Changes. Sometimes it will display the success alert but the actual content was not saved. That may happen due to malformed template that we're not validating or some race condition, still needs more investigation.

If someone can find specific reproduction steps I'll dig into this

If someone can find specific reproduction steps I'll dig into this

@cibernox I was not able to reproduce locally but it was reported to happen on our staging env. I don't think the Visual Editor component itself is causing it, but would be helpful if you can help find the root cause.