PSNotApplyErrorActionToStderr does not work as expected with ForEach-Object -Parallel

Steps to reproduce
Enable PSNotApplyErrorActionToStderr - no longer needed
$ErrorActionPreference = 'Stop'
@(1,2,3) | ForEach-Object -Parallel { cmd.exe /c "echo $_ >&2" }

Expected behavior
Process all records, with no error records produced:
1
2
3

Actual behavior
Stops after first record, produces an error record for native command stderr:
1 # in red

Environment data
Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.20206
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

It works well without $ErrorActionPreference = 'Stop'.
/cc @PaulHigin

It works well without $ErrorActionPreference = 'Stop'.
/cc @PaulHigin

It works well without $ErrorActionPreference = 'Stop'.

Actually it doesn't. Stop only causes execution to continue, but it still creates error records (note the red output). If you remove -Parallel, no error records are created, as expected.

It works well without $ErrorActionPreference = 'Stop'.

Actually it doesn't. Stop only causes execution to continue, but it still creates error records (note the red output). If you remove -Parallel, no error records are created, as expected.

This is a known issue.  Current session state is not propagated to parallel running sessions.  This is currently by design, but there is a work item to address this to some degree.
https://github.com/PowerShell/PowerShell/issues/12240

This is a known issue.  Current session state is not propagated to parallel running sessions.  This is currently by design, but there is a work item to address this to some degree.
https://github.com/PowerShell/PowerShell/issues/12240

@PaulHigin How are experimental features flags related to a session? They are defined in powershell.config.json. Shouldn't every session load it independently?

@PaulHigin How are experimental features flags related to a session? They are defined in powershell.config.json. Shouldn't every session load it independently?

Sorry, I misread the issue.  I thought this was about propagating the $ErrorActionPreference built-in variable, and it looks like that is being done.  I am not sure about experimental features, but it is likely a similar problem in that it needs to be set in the parallel running sessions.
@aelij  Please add a comment to issue https://github.com/PowerShell/PowerShell/issues/12240, so that experimental feature enabling is also considered.  Then we can close this issue.

Sorry, I misread the issue.  I thought this was about propagating the $ErrorActionPreference built-in variable, and it looks like that is being done.  I am not sure about experimental features, but it is likely a similar problem in that it needs to be set in the parallel running sessions.
@aelij  Please add a comment to issue https://github.com/PowerShell/PowerShell/issues/12240, so that experimental feature enabling is also considered.  Then we can close this issue.

I think this should be a different issue.
This is about reading the correct config inside Parallel, not shared variables/session. It's likely the fix will be different too.

I think this should be a different issue.
This is about reading the correct config inside Parallel, not shared variables/session. It's likely the fix will be different too.

It is the same in how the sessions are set up before running in parallel.  There is no point in having multiple issues.  I'll add a note to that relevant issue and close this one.

It is the same in how the sessions are set up before running in parallel.  There is no point in having multiple issues.  I'll add a note to that relevant issue and close this one.

@PaulHigin I've investigated it a bit, and it doesn't seem that the issue is about experimental features at all:
1 | ForEach-Object -Parallel { Get-ExperimentalFeature PSNotApplyErrorActionToStderr }

yields:
Name                                Enabled Source                              Description
----                                ------- ------                              -----------
PSNotApplyErrorActionToStderr          True PSEngine                            Don't have $ErrorActionPreference affect stderr output

I think the problem is in NativeCommandProcessor and how it handles output from background processes. So it should work regardless of the suggested -UseCurrentState.
This code gets executed only in parallel:
https://github.com/PowerShell/PowerShell/blob/4a09585fe72c3679e23a1aa047b23ac827609c55/src/System.Management.Automation/engine/NativeCommandProcessor.cs#L657-L673
So please reopen as it's definitely unrelated to the other issue.

@PaulHigin I've investigated it a bit, and it doesn't seem that the issue is about experimental features at all:
1 | ForEach-Object -Parallel { Get-ExperimentalFeature PSNotApplyErrorActionToStderr }

yields:
Name                                Enabled Source                              Description
----                                ------- ------                              -----------
PSNotApplyErrorActionToStderr          True PSEngine                            Don't have $ErrorActionPreference affect stderr output

I think the problem is in NativeCommandProcessor and how it handles output from background processes. So it should work regardless of the suggested -UseCurrentState.
This code gets executed only in parallel:
https://github.com/PowerShell/PowerShell/blob/4a09585fe72c3679e23a1aa047b23ac827609c55/src/System.Management.Automation/engine/NativeCommandProcessor.cs#L657-L673
So please reopen as it's definitely unrelated to the other issue.

Bump - still relevant

@aelij If the setting in the json file is changed, additional propagation from an active session should use the current session value. Using the new setting after a parallel pipeline has already started could lead to inconsistent behavior across threads.  If the new setting is required, start a new session.   As of 7.4, it's still manual labor to propagate preferences (e.g., WhatIf!) that don't propagate.  It's easier to propagate the current session preferences via using than to check if the default has changed.  So, for once, it's easier to do the right thing.

@RandyInMarin

If the setting in the json file is changed, additional propagation from an active session should use the current session value.

Are you referring to the experimental flags JSON? That's not relevant anymore.
This issue is about how -Parallel handles stderr output differently. Just run the sample in the post.


Load video assets as needed

Closing as this is replaced by a (hopefully) improved approach in #28.


Move the streamer image resizing to css, remove pillow dependency

Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable
[ ] Documentation in docs/ or install-docs/ was updated, if applicable

No reason to use this large dependency anymore

This is what it looks like for me:


To run functional.py example locally, I need to escape """ in Python docstrings

Somehow, https://github.com/stanfordnlp/dspy/blob/main/examples/functional/functional.ipynb works for me on github, but when I try to run the
predictor = TypedPredictor(CodeSignature)
prediction = predictor(
    prompt=PythonCode(code=ds['test'][0]['prompt']),
    test=PythonCode(code=ds['test'][0]['test']),
    entry_point=ds['test'][0]['entry_point']
)

print(prediction)

section in a regular Python script, I keep getting JSON decoding errors.
To fix them, I had to change in dspy/functional/functional.py the return value of _unwrap_json():
return output.replace('"""', '\"\"\"')
Otherwise the """ in the generated code confuse JSON parser (you can replicate by just using json.loads() on the string below):
'{"code": "from typing import List\\n\\n\\ndef has_close_elements(numbers: List[float], threshold: float) -> bool:\\n    \\"\\"\\" Check if in given list of numbers, are any two numbers closer to each other than\\n    given threshold.\\n    >>> has_close_elements([1.0, 2.0, 3.0], 0.5)\\n    False\\n    >>> has_close_elements([1.0, 2.8, 3.0, 4.0, 5.0, 2.0], 0.3)\\n    True\\n    \\"\\"\\"\\n    for i in range(len(numbers)):\\n        for j in range(i + 1, len(numbers)):\\n            if abs(numbers[i] - numbers[j]) < threshold:\\n                return True\\n    return False"}'

Thank you @simonff ! Could be of interest to @thomasahle but it sounds like you sort of have a PR in mind @simonff ?

I need to get approval from work for submitting PRs, and I'm not sure I have all the right context here anyway, so I'd rather wait a bit for someone more knowledgeable to comment.

This is super interesting. I'll have a look. What language model are you using? Gpt-3.5-turbo?

Yes, gpt-3.5-turbo and also gemini-1.0-pro
Sorry for the typo above, the replacement string should have double backslash escapes before each quote.

I think the issue was actually that the signature was mistakenly defined to be:
class CodeSignature(Signature):
    prompt: str = InputField()
    test: PythonCode = InputField()
    entry_point: str = InputField()
    solution: PythonCode = OutputField()

Instead of
class CodeSignature(Signature):
    prompt: PythonCode = InputField()
    test: PythonCode = InputField()
    entry_point: str = InputField()
    solution: PythonCode = OutputField()

I fixed that now, and also added some robustness via parsing through ujson instead of json.
Do you mind checking in my fork whether it works for you as well?

Thank you, Thomas! Can confirm the problem is gone if I use your fork.


Circular reference : Router inside RestangularConfigFactory

When we inject the Router inside the module root.
RestangularModule.forRoot([Router],RestangularConfigFactory),
We have this error :
Unhandled Promise rejection: Maximum call stack size exceeded ; Zone: <root> ; Task: Promise.then ; Value: RangeError: Maximum call stack size exceeded
I used the router to do redirection on the request interceptors.
I didn't had this error with angular 4.
Here are my dependencies (angular5, ngx-restangular 2):
"dependencies": {
    "@angular/animations": "^5.0.0",
    "@angular/common": "^5.0.0",
    "@angular/compiler": "^5.0.0",
    "@angular/core": "^5.0.0",
    "@angular/forms": "^5.0.0",
    "@angular/http": "^5.0.0",
    "@angular/platform-browser": "^5.0.0",
    "@angular/platform-browser-dynamic": "^5.0.0",
    "@angular/router": "^5.0.0",
    "@swimlane/ngx-datatable": "^11.1.4",
    "angular-web-storage": "^1.0.1",
    "chart.js": "2.6.0",
    "console-emoji": "0.0.2",
    "core-js": "2.5.0",
    "lodash": "4.17.4",
    "moment": "2.18.1",
    "ng2-charts": "1.6.0",
    "ngx-bootstrap": "^1.9.3",
    "ngx-restangular": "^2.0.1",
    "ngx-toastr": "^6.5.0",
    "rxjs": "^5.5.2",
    "ts-helpers": "1.1.2",
    "zone.js": "0.8.17"
  },

@davidbonachera I've setup test project with your dependencies and there was no such error. Can you give me more details to solve this issue?


prebake animations/tweens for svg morphs

since svg morphs are quite computationally expensive it would be nice to be able to calculate the morphs and export them (e.g. to .json) similar to Lottie Files, so we could directly load and run the prebaked morphs without needing to recalculate them on every client

You can already and easily export them. just do
const tween= KUTE.fromTo(...args)

const from = JSON.stringify(tween.startValues.path);
const to = JSON.stringify(tween.endValues.path)
`

Reuse them in your future tweens ;)

so i would call them something like
const parsedTween = KUTE.fromTo('#elem',
    JSON.parse(from),
    JSON.parse(to)
)

and the calculation would be much faster?
(currently its unfeasable for me to use this package since everytime i refresh the page it blocks to page load for like 1sek per svg morph (and i have a couple))

It only takes a few milliseconds to parse, the data is already there.

It only takes a few milliseconds to parse, the data resulted from CPU gymnastics is already there.

thats what i wanted to hear :) nice!


page fault in nonpaged area

im trying using this injector https://github.com/Rhydon1337/windows-kernel-dll-injector with kdmapper  i got bsod
page fault in nonpaged area i try disable Security checks -GS + set DriverEntry without luck what could it be ?

Check the address in your BSOD file but if you didn't change anything https://github.com/Rhydon1337/windows-kernel-dll-injector/blob/467ec97eaaed1828cf2918a211539d42af047d98/DLLInjector/DLLInjector/main.cpp#L15 this entry point receive null pointers in manual map, which means that will automatically BSOD when the code try to access them
for example here
https://github.com/Rhydon1337/windows-kernel-dll-injector/blob/467ec97eaaed1828cf2918a211539d42af047d98/DLLInjector/DLLInjector/main.cpp#L25
or here
https://github.com/Rhydon1337/windows-kernel-dll-injector/blob/467ec97eaaed1828cf2918a211539d42af047d98/DLLInjector/DLLInjector/main.cpp#L34

Thank u SO MUCH IoCreateDriver Fixed it !


Conflict with Pagination inside Navs Tabs

Windows 7 64x
Chrome 60.0.3112.113 64x
Demo : https://codepen.io/seltix/pen/eEXawW
HTML source :
https://getbootstrap.com/docs/4.0/components/navs/
https://getbootstrap.com/docs/4.0/components/pagination/

Test case :

select TAB 2
select TAB 1
at this point booth tabs will be visible and the selected page on PAGINATION 1 will lose the active state.

thanks

Fixed by : https://github.com/twbs/bootstrap/pull/23087
will be available in our next release 👍


Allow easily sharing HTTPS servers

Product and Version: VSCode
OS Version: macOS
Live Share Extension Version: 0.3.11
Target Platform or Language: Node.js/docker
Steps to Reproduce / Scenario:
Our development server is a nodejs server running inside docker and uses https forwarded to the local machine on port 80/443.  Trying to use the vscode "Shared servers" feature to share localhost port 80 and 443 to other users results in other users not being able to load the page. Chrome, for example throws the following error:
This site can’t provide a secure connection
localhost sent an invalid response.
ERR_SSL_PROTOCOL_ERROR

Is this simply a limitation of trying to proxy https or is this a bug with the way vscode share is forwarding its shared servers?

Closing due to inactivity.

Re-opening this since it's actually a feature request not a bug.

We’re not able to prioritize this issue over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it’s for you.


INFINITY-1642 Treat COMPLETE+STARTING as IN_PROGRESS, instead of STARTING

Examples of current behavior:
{
  "phases" : [ {
    "id" : "6751e9b4-8002-4f02-8ee2-9516be5efdc4",
    "name" : "Deployment",
    "steps" : [ {
      "id" : "c49cec9c-1559-4ba6-930f-84f151ac0880",
      "status" : "COMPLETE",
      "name" : "kafka-0:[broker]",
      "message" : "com.mesosphere.sdk.scheduler.plan.DeploymentStep: 'kafka-0:[broker] [c49cec9c-1559-4ba6-930f-84f151ac0880]' has status: 'COMPLETE'."
    }, {
      "id" : "7a983d7e-196c-40a2-aa36-5dcbe8e6423b",
      "status" : "COMPLETE",
      "name" : "kafka-1:[broker]",
      "message" : "com.mesosphere.sdk.scheduler.plan.DeploymentStep: 'kafka-1:[broker] [7a983d7e-196c-40a2-aa36-5dcbe8e6423b]' has status: 'COMPLETE'."
    }, {
      "id" : "17e5b2d9-7425-4453-8f0d-ac34c9c661f6",
      "status" : "STARTING",
      "name" : "kafka-2:[broker]",
      "message" : "com.mesosphere.sdk.scheduler.plan.DeploymentStep: 'kafka-2:[broker] [17e5b2d9-7425-4453-8f0d-ac34c9c661f6]' has status: 'STARTING'."
    } ],
    "status" : "STARTING"
  } ],
  "errors" : [ ],
  "status" : "STARTING"
}

status of the cassandra "cleanup" plan: PENDING
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: IN_PROGRESS
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: STARTING
status of the cassandra "cleanup" plan: COMPLETE
status of the cassandra "cleanup" plan: COMPLETE
status of the cassandra "cleanup" plan: COMPLETE
status of the cassandra "cleanup" plan: COMPLETE

retest this please (turns out cassandra replace node flakes still arent fixed)


Handle side-by-side tokens with overlapping regexes

This fixes #46. As @mattdahl suggested, that bug is because the full regexes for "see" and "id." overlap (they both capture the space between them, but then ignore it by only grabbing what's in the match group). Unlike the regexes, the tokens generated from each regex have correct start and end offsets and don't overlap, so the core fix was just a one-liner to compare offsets on extractor.get_token(m) instead of the offsets of m.span() itself.
What made that tricky was that hyperscan works on bytes instead of strings, and dealing with the byte offsets was complicating everything. I realized that I could instead update the byte offsets right after they came out of hyperscan, and make it easier to work with everything downstream. Thus began refactoring.
So this has a bunch of changed lines, but it's hopefully just untangling some things to make any bugs like this easier to fix in the future.

LGTM aside from the one comment.

Yeah cause it's not defined yet (until py 3.10):
https://stackoverflow.com/a/53450349
(Dunno if the from future thing would be a good idea in general.)
On Thu, Mar 25, 2021 at 6:09 PM Mike Lissner @.***>
wrote:

@.**** commented on this pull request.
In eyecite/models.py
https://github.com/freelawproject/eyecite/pull/48#discussion_r601871041:

@@ -260,7 +260,7 @@ class Token(UserString):
end: int

 @classmethod

def from_match(cls, m, extra, offset=0):

def from_match(cls, m, extra, offset=0) -> "Token":

Should this be in quotes?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freelawproject/eyecite/pull/48#pullrequestreview-621618807,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC33UGNSEEMP7UPOO2S4FTTFOYABANCNFSM4Z2FYVYQ
.

Ah, I didn't grok what it was doing at first. I'll trust your judgment here.


✨ Overwrite helm chart version. No need for templating.

this commit adds a feature in csctl where one doesn't require templating and Chart.yaml and csctl will override the version field in Chart.yaml

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

@jschoone you can check this out!

Hi @kranurag7,
sorry for the late response.
For me it seems it still doesn't work for the cluster-addon
What I tried:
$ git clone -b syself/csctl-chart-templating https://github.com/SovereignCloudStack/csctl
$ cd csctl
$ make build
$ ./csctl version
csctl version: v0.0.2-12-g0a80b2c
commit: 0a80b2ccfdf934f9e3c67fe1240a592e980b7e92

$ ./csctl create -m hash tests/cluster-stacks/docker/ferrol -o .release
$ basename $(ls .release/*/*cluster-addon*.tgz)
docker-ferrol-1-27-cluster-addon-v0.tgz

When I change the value back to << .ClusterAddonVersion >> I get what I expect:
$ yq -i '.version = "<< .ClusterAddonVersion >>"' tests/cluster-stacks/docker/ferrol/cluster-addon/Chart.yaml
$ ./csctl create -m hash tests/cluster-stacks/docker/ferrol -o .release
$ basename $(ls .release/*/*cluster-addon*.tgz)
docker-ferrol-1-27-cluster-addon-v0-sha.0a80b2c.tgz

❯ ./bin/csctl create tests/cluster-stacks/docker/ferrol -m hash -o .test-releases
Creating output in .test-releases/docker-ferrol-1-27-v0-sha-a2b3cbb
.tmp/cluster-addon/Chart.yaml updating version from v0 to v0-sha.a2b3cbb
.tmp/cluster-class/Chart.yaml updating version from v0 to v0-sha.a2b3cbb
No provider specifig configuration in csctl.yaml. No need to call a plugin for provider "docker"
Created .test-releases/docker-ferrol-1-27-v0-sha-a2b3cbb

☸  kind-cso 📁 csctl on  syself/csctl-chart-templating [!?] via  v1.21.6
❯ ls .test-releases/docker-ferrol-1-27-v0-sha-a2b3cbb
cluster-addon-values.yaml  docker-ferrol-1-27-cluster-addon-v0-sha.a2b3cbb.tgz  docker-ferrol-1-27-cluster-class-v0-sha.a2b3cbb.tgz  hashes.json  metadata.yaml

this looks good I guess

Squashed


feat[CI]: Add golangci-lint

Fixes: #13

@iamrajiv isn't .golangci.yml required in the root directory (as here https://github.com/meshery/meshery-consul/blob/master/.golangci.yml)?  I'd expect running make check should catch the e.g. cases where file were not goimports-ed and go fmt'ed, as now in errorutil code. I always run make check and trust it. It used to create a lot of errors here in meshkit (hence the issue I created) and suddenly those were not reported anymore. Didn't have time to look into that then, unfortunately....

If I add the aforementioned .golangci.yml I get
❯ make check
golangci-lint run
utils/kubernetes/service_test.go:32: File is not `gofmt`-ed with `-s` (gofmt)
                                corev1.ServicePort{
cmd/errorutil/internal/error/summary.go:5: File is not `goimports`-ed with -local github.com/golangci/golangci-lint (goimports)
        "github.com/layer5io/meshkit/cmd/errorutil/internal/component"
        log "github.com/sirupsen/logrus"
cmd/errorutil/internal/coder/file.go:5: File is not `goimports`-ed with -local github.com/golangci/golangci-lint (goimports)
        "github.com/layer5io/meshkit/cmd/errorutil/internal/component"
        errutilerr "github.com/layer5io/meshkit/cmd/errorutil/internal/error"
        "github.com/sirupsen/logrus"
cmd/errorutil/internal/coder/node.go:5: File is not `goimports`-ed with -local github.com/golangci/golangci-lint (goimports)
        "github.com/layer5io/meshkit/cmd/errorutil/internal/component"
        "github.com/sirupsen/logrus"
logger/types.go:4:2: const `JsonLogFormat` should be `JSONLogFormat` (golint)
        JsonLogFormat = iota
        ^
utils/network.go:25:6: func TcpCheck should be TCPCheck (golint)
func TcpCheck(hp *HostPort, mock *MockOptions) bool {
     ^
broker/nats/nats.go:118:1: receiver name in should be consistent with previous receiver name n for Nats (golint)
func (in *Nats) DeepCopyInto(out broker.Handler) {
^
broker/nats/nats.go:123:1: receiver name in should be consistent with previous receiver name n for Nats (golint)
func (in *Nats) DeepCopy() *Nats {
^
broker/nats/nats.go:133:1: receiver name in should be consistent with previous receiver name n for Nats (golint)
func (in *Nats) DeepCopyObject() broker.Handler {
^
utils/utils.go:81:15: G107: Potential HTTP request made with variable url (gosec)
        resp, err := http.Get(url)
                     ^
utils/utils.go:145:19: G107: Potential HTTP request made with variable url (gosec)
        response, err := http.Get(url)
                         ^
cmd/errorutil/internal/coder/node.go:118:58: `analyses` is a misspelling of `analyzes` (misspell)
// handleValueSpec inspects node n if it is a ValueSpec, analyses and updates it (depending on update and updateAll). Returns true if any value was changed.
                                                         ^
models/oam/core/v1alpha1/doc.go:9:13: `Definiton` is a misspelling of `Definition` (misspell)
// 5. Scope Definiton
            ^
utils/kubernetes/expose/traverser.go:45:47: `occured` is a misspelling of `occurred` (misspell)
// while the other is the error that may have occured in the traverser
                                              ^
errors/errors.go:6: unnecessary leading newline (whitespace)
func NewDefault(code string, ldescription ...string) *Error {

errors/errors.go:18: unnecessary leading newline (whitespace)
func New(code string, severity Severity, sdescription []string, ldescription []string, probablecause []string, remedy []string) *Error {

errors/errors.go:32: unnecessary leading newline (whitespace)
func GetCode(err error) string {

utils/kubernetes/expose/expose.go:10:2: ST1019: package "k8s.io/api/core/v1" is being imported more than once (stylecheck)
        corev1 "k8s.io/api/core/v1"
        ^
utils/kubernetes/apply-manifest.go:8:2: ST1019: package "k8s.io/apimachinery/pkg/api/errors" is being imported more than once (stylecheck)
        "k8s.io/apimachinery/pkg/api/errors"
        ^
utils/kubernetes/service_test.go:9:2: ST1019: package "k8s.io/api/core/v1" is being imported more than once (stylecheck)
        corev1 "k8s.io/api/core/v1"
        ^

That's more like it :+1:

@mgfeller I have reverted the commit. Please check once.

@mgfeller Yeah I have updated the ci can u check once

@iamrajiv cool it works :smile: there are some warnings even... do you fix them? I can fix them if you like in this PR, they are from some of "my" errorutil code anyway.

@mgfeller there was an issue with a timeout so I added a configuration file fo golangci-lint where I added timeout: 5m. I will edit this configuration in separate PR where I will add other settings.


feat: update copyright text 2023

What does this PR do?
Changes the Copyright text in the documentation.

How to test it

Navigate to the /docs directory
Run make serve
Check the documentation at http://<IP_ADDRESS>:8000/traefik-mesh/

Additional Notes

The year of the copyright is already set for 2023

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


ssl error

Coinbase Pro Asyncronous Websocket Client version:
master 2c3c06301df5f762699cacc06ab0909c20c318b1

Python version:
Python 3.6.6 (v3.6.6:4cf1f54eb7, Jun 26 2018, 19:50:54)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)]

Operating System:
macos sierra 10.13.6

Description
When running the following script
# heartbeat.py
import asyncio
from copra.websocket import Channel, Client

KEY = '<mykey>'
SECRET = '<mysecret>''
PASSPHRASE = '<mypassphrase>'

if __name__ == '__main__':
    loop = asyncio.get_event_loop()
    ws = Client(loop, Channel('heartbeat', 'BTC-USD'))
    try:
        loop.run_forever()
    except KeyboardInterrupt:
        loop.run_until_complete(ws.close())
        loop.close()

I get the following exception related to ssl verification
commmand
python3 heartbeat.py

traceback
Task exception was never retrieved
future: <Task finished coro=<BaseEventLoop.create_connection() done, defined at /Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/base_events.py:681> exception=SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)')>
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/base_events.py", line 804, in create_connection
    sock, protocol_factory, ssl, server_hostname)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/base_events.py", line 830, in _create_connection_transport
    yield from waiter
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/sslproto.py", line 505, in data_received
    ssldata, appdata = self._sslpipe.feed_ssldata(data)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/sslproto.py", line 201, in feed_ssldata
    self._sslobj.do_handshake()
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py", line 689, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)

Based on your configuration, your problem sounds a lot like  https://github.com/Rapptz/discord.py/issues/423#issuecomment-272093801. Can you verify that you have the certificate installed?


Missing dependency in setup.py

This program requires
libusb1
enum34

It is stated in the readme but I should add something in the setup.py.


machine-id conflict

I'm using bionic-containers-rock64-0.7.11-1075-arm64.img.xz from 0.7.11: gitlab-ci-linux-build-75 in my Kubernetes cluster and I have problems with Weave overlay network. The nodes are not connecting to each other because all the boards have the same machine-id (/etc/machine-id).
I'm not a Linux expert but would it be possible to delete /etc/machine-id while bionic-containers-rock64 image creation (as the last step before the final shutdown) so that it can be recreated when I start my SBC the first time?

It will be part of next release, the 0.8.0rc5 or later.


CS0246 ItemContainerEventArgs, but Avalonia.Controls.Primitives are imported

Describe the bug
ItemContainerEventArgs not found, even if Avalonia.Controls.Primitives is imported

To Reproduce

Avalonia 11
Avalonia.Controls.Primitives are imported
use ItemContainerEventArgs in code

Expected behavior
I can use ItemContainerEventArgs
Avalonia version
11*
OS
Windows
Additional context
No response

ItemContainerEventArgs was long removed over two years ago by https://github.com/AvaloniaUI/Avalonia/pull/9677
I noticed you're using a third-party library that hasn't been updated for over two years and is on Avalonia 0.10.13. Such unmaintained libraries are not compatible with Avalonia 11.

Pull request description doesnt mention anything about ItemContainerEventArgs, thats bad, i dont know what to use then. Also i`am trying to port ExtendedToolkit to Avalonia11, i have my own fork

Pull request description doesnt mention anything about ItemContainerEventArgs, thats bad, i dont know what to use instead of ItemContainerEventArgs. Also i`am trying to port ExtendedToolkit to Avalonia11, i have my own fork

It was a large PR for a rewrite. The author can't specify every detail in text and readers need to actually examine what has been changed. I'm not sure if there's equivalent functionality exposed, but you could start a Discussion.


RPCError: got code -32000 with msg "insufficient funds for transfer"

i have "total_balance_formatted": "0.000528902815064832",
here is code
double amount = 0.000128902815064832;
BigInt bigIntValue = BigInt.from(amount * pow(10, 18));
EtherAmount ethAmount =
EtherAmount.fromBigInt(EtherUnit.wei, bigIntValue);
var apiUrl =
"https://eth-sepolia.g.alchemy.com/v2/XXXmy_APi_KeyXXX";
      var httpClient = http.Client();
  var ethClient = Web3Client(apiUrl, httpClient);

  EthPrivateKey credentials = EthPrivateKey.fromHex('0x$XXXMy_private_KeyXXX');

  EtherAmount etherAmount = await ethClient.getBalance(credentials.address);
  EtherAmount gasPrice = await ethClient.getGasPrice();

  log('SSS amount is -= $etherAmount');
  print('Balance in Wei: ${etherAmount.getInWei}');
  print('Balance in Ether: ${etherAmount.getValueInUnit(EtherUnit.ether)}');
  {

this is out put why is 0.0 ??
flutter: Balance in Wei: 0
flutter: Balance in Ether: 0.0
}
await ethClient.sendTransaction(
credentials,
Transaction(
to: EthereumAddress.fromHex(XXX_receiver_XXX),
gasPrice: gasPrice,
value: txValue,
),
chainId: 11155111,
);
  {

error is => RPCError: got code -32000 with msg "insufficient funds for transfer".
}
but I have amount in wallet but it's showing 0.0 and get this error

Check here if the address of your private key has a balance: https://sepolia.etherscan.io/
With chain id 11155111, that should be sepolia.
On Sepolia, you can just get ETH from a faucet.


Plugin error

Hi, I get the following problem after installing your plugin on Hyper 2.0.4

I also tried the provided installing method in README.md
What should I do?

Hi @balazsorban44 ,
Thanks for your feedback.
You can try to re-compile the abstract-socket module as explained in the error message.
To do so, run the following
$ cd ~/.hyper_plugins && npm install abstract-socket

Let me know if it works.

Unfortunately, it did not help either.

Can you please run node -v and npm version and paste the results here

I have node v8.6.0 and npm 5.4.2 installed.

Hi @balazsorban44 ,
I just released a new version of Hyper-spotify (v3.0.0) and I tested it on Ubuntu 17.04 with node 8.7.0 and nom 5.5.1.
Can you try to remove hyper-spotify plugin and add it again.

Unfortunately, I still get the same error.

Hi @balazsorban44 ,
Can you try to run the following commands and post here the resulting files
$ cd ~/.hyper_plugins
$ npm ls > ~/npm_deps_tree.txt
$ ls > ~/npm_deps_list.txt

Closed due to inactivity


turn image comparison tests back on and get them working

In #104 we turned the image comparison tests off because of some configuration issues, and didn't want that to interfere with 0.1.  They should be turned back on (and fixed!) in 0.2
cc @cdeil @bsipocz (also #81, although I think that's different from the original issue in #104 ?)

Marking Issue as stale and closing. Looks like it was fixed in #115. Please re-open if still affected by this Issue.


Creating an index with DynamicTemplates causes error

In the following code snippet to create an index with DynamicProperties passed in i get the following error :

  var response = await _elasticClient.Indices.CreateAsync(new CreateIndexRequest(fullIndexName)
            {
                Settings = indexSettings,
                Mappings = new TypeMapping
                {
                    Source = new SourceField()
                    {
                        Enabled = elasticIndexSettings.StoreSourceData,
                        Excludes = new string[] {IndexingConstants.DisplayTextAnalyzedKey}
                    },
                    Properties = new Properties
                    {
                        // We force some mappings for common fields.
                        {IndexingConstants.ContentItemIdKey, new KeywordProperty()},
                        {IndexingConstants.ContentItemVersionIdKey, new KeywordProperty()},
                        {IndexingConstants.OwnerKey, new KeywordProperty()},
                        {IndexingConstants.FullTextKey, new KeywordProperty()},
                        // We map ContentType as a keyword because else the automatic mapping will break the queries.
                        {IndexingConstants.ContentTypeKey, new KeywordProperty()},
                        {
                            IndexingConstants.ContainedPartKey,
                            new ObjectProperty()
                            {
                                Properties = new Properties()
                                {
                                    {nameof(ContainedPartModel.Ids), new KeywordProperty()},
                                    {nameof(ContainedPartModel.Order), new KeywordProperty()},
                                }
                            }
                        },
                        {
                            IndexingConstants.DisplayTextKey,
                            new ObjectProperty()
                            {
                                Properties = new Properties()
                                {
                                    {nameof(DisplayTextModel.Analyzed), new TextProperty()},
                                    {nameof(DisplayTextModel.Normalized), new KeywordProperty()},
                                    {nameof(DisplayTextModel.Keyword), new KeywordProperty()},
                                }
                            }
                        }
                    },
                    Meta = new FluentDictionary<string, object>() {{_lastTaskId, 0}}

                    , DynamicTemplates = new List<IDictionary<string, DynamicTemplate>>( new[]
                    {
                        new Dictionary<string, DynamicTemplate>
                        {
                            {
                                "*.Inherited",
                                new DynamicTemplate()
                                {
                                    MatchMappingType = "string",
                                    PathMatch = "\"*\" + IndexingConstants.InheritedKey",
                                    Mapping = new KeywordProperty()
                                }
                            },
                            {
                                "*.Ids",
                                new DynamicTemplate()
                                {
                                    MatchMappingType = "string",
                                    PathMatch = "\"*\" + IndexingConstants.IdsKey",
                                    Mapping = new KeywordProperty()
                                }
                            }
                        }
                    })
                }
            });

Invalid Elasticsearch response built from a unsuccessful (400) low level call on PUT: /default_dsdsfsd
 Exception: Request failed to execute. Call: Status code 400 from: PUT /default_dsdsfsd. ServerError: Type: mapper_parsing_exception Reason: "Failed to parse mapping: A dynamic template must be defined with a name" CausedBy: "Type: mapper_parsing_exception Reason: "A dynamic template must be defined with a name""

# Audit trail of this API call:
 - [1] BadResponse: Node: https://1a105802743d4f41b53ef6269761d4d1.us-central1.gcp.cloud.es.io/ Took: 00:00:00.1462256
# OriginalException: Elastic.Transport.TransportException: Request failed to execute. Call: Status code 400 from: PUT /default_dsdsfsd. ServerError: Type: mapper_parsing_exception Reason: "Failed to parse mapping: A dynamic template must be defined with a name" CausedBy: "Type: mapper_parsing_exception Reason: "A dynamic template must be defined with a name""
# Request:
<Request stream not captured or already read to completion by serializer. Set DisableDirectStreaming() on TransportConfiguration to force it to be set on the response.>
# Response:
{"error":{"root_cause":[{"type":"mapper_parsing_exception","reason":"A dynamic template must be defined with a name"}],"type":"mapper_parsing_exception","reason":"Failed to parse mapping: A dynamic template must be defined with a name","caused_by":{"type":"mapper_parsing_exception","reason":"A dynamic template must be defined with a name"}},"status":400}

@giannik Thanks for reporting! Could you please include the request JSON payload in this issue as well (DisableDirectStreaming must be used to capture the request payload).

@flobernd
here is the error with payload info

Unsuccessful (400) low level call on PUT: /default_testindexcc
 Exception: Request failed to execute. Call: Status code 400 from: PUT /default_testindexcc. ServerError: Type: mapper_parsing_exception Reason: "Failed to parse mapping: A dynamic template must be defined with a name" CausedBy: "Type: mapper_parsing_exception Reason: "A dynamic template must be defined with a name""

# Audit trail of this API call:
 - [1] BadResponse: Node: https://1a105802743d4f41b53ef6269761d4d1.us-central1.gcp.cloud.es.io/ Took: 00:00:00.3313462
# OriginalException: Elastic.Transport.TransportException: Request failed to execute. Call: Status code 400 from: PUT /default_testindexcc. ServerError: Type: mapper_parsing_exception Reason: "Failed to parse mapping: A dynamic template must be defined with a name" CausedBy: "Type: mapper_parsing_exception Reason: "A dynamic template must be defined with a name""
# Request:
{"mappings":{"_meta":{"last_task_id":0},"_source":{"enabled":true,"excludes":["ContentItem_DisplayText_Analyzed"]},"dynamic_templates":[{"*.Inherited":{"mapping":{"type":"keyword"},"match_mapping_type":"string","path_match":"\u0022*\u0022 \u002B IndexingConstants.InheritedKey"},"*.Ids":{"mapping":{"type":"keyword"},"match_mapping_type":"string","path_match":"\u0022*\u0022 \u002B IndexingConstants.IdsKey"}}],"properties":{"ContentItemId":{"type":"keyword"},"ContentItemVersionId":{"type":"keyword"},"ContentItem_Owner":{"type":"keyword"},"ContentItem_FullText":{"type":"keyword"},"ContentItem_ContentType":{"type":"keyword"},"ContentItem_ContainedPart":{"properties":{"Ids":{"type":"keyword"},"Order":{"type":"keyword"}},"type":"object"},"ContentItem_DisplayText":{"properties":{"Analyzed":{"type":"text"},"Normalized":{"type":"keyword"},"Keyword":{"type":"keyword"}},"type":"object"},"ContentItem_Vector":{"dims":1536,"index":true,"similarity":"cosine","type":"dense_vector"}}},"settings":{"analysis":{"analyzer":{"standard":{"type":"standard"}}}}}
# Response:
{"error":{"root_cause":[{"type":"mapper_parsing_exception","reason":"A dynamic template must be defined with a name"}],"type":"mapper_parsing_exception","reason":"Failed to parse mapping: A dynamic template must be defined with a name","caused_by":{"type":"mapper_parsing_exception","reason":"A dynamic template must be defined with a name"}},"status":400}

@giannik Thanks for the reply. I can see the problem now: Instead of putting both templates in a single dictionary, you have to specify multiple dictionaries with a single item. This is how the Elasticsearch server expects the templates to be sent (I know.. it's not the most intuitive design decision, but it's probably like this for historical reasons).

@flobernd thankyou.
That worked!


CVV doesn't work anymore

can you fix the code? becouse the Supreme site has been updated and the CVV code of the "Cop It Bot" doesn't work anymore.
Thanks.

should be fixed with
$("#vval").val(r.cvv);

idk if supreme will change the id of the cvv in the last second so this could just be a tentative fix...
I'll check out the website after the drop happens and look for any changes.

Thank you bro💪🏼

@AceroM you can always use a different selector like $('input[placeholder="CVV"]') instead.  that will work no matter what the id is.

the fix doesn't work for me! :(

someone says how to fix the code??

@robbymurphy
good selector thanks

@AceroM I fix the code with your code but i had to write "CVV" in lowercase


rpm not found

Adding RPM to worker chroot: filesystem-1.1-7.cm1.x86_64.rpm.
Adding RPM to worker chroot: kernel-headers-<IP_ADDRESS>-1.cm1.noarch.rpm.
Adding RPM to worker chroot: glibc-2.28-18.cm1.x86_64.rpm.
Elevated install failed for package glibc-2.28-18.cm1.x86_64.rpm, aborting. Inspect /mnt/c/Users/sylvian/CBL-Mariner/build/logs/worker_chroot.log for more info. Did you hydrate the toolchain?
make: *** [/mnt/c/Users/sylvian/CBL-Mariner/toolkit/scripts/tools.mk:148: /mnt/c/Users/sylvian/CBL-Mariner/build/worker/worker_chroot.tar.gz] Error 1
From the log file:
Adding RPM to worker chroot: filesystem-1.1-7.cm1.x86_64.rpm.
Found full path for package filesystem-1.1-7.cm1.x86_64.rpm in /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache: (/mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/x86_64/filesystem-1.1-7.cm1.x86_64.rpm)
rpm: RPM should not be used directly install RPM packages, use Alien instead!
rpm: However assuming you know what you are doing...
warning: /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/x86_64/filesystem-1.1-7.cm1.x86_64.rpm: Header V4 RSA/SHA256 Signature, key ID 3135ce90: NOKEY
Verifying packages...
Preparing packages...
filesystem-1.1-7.cm1.x86_64
Adding RPM to worker chroot: kernel-headers-<IP_ADDRESS>-1.cm1.noarch.rpm.
Found full path for package kernel-headers-<IP_ADDRESS>-1.cm1.noarch.rpm in /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache: (/mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/noarch/kernel-headers-<IP_ADDRESS>-1.cm1.noarch.rpm)
rpm: RPM should not be used directly install RPM packages, use Alien instead!
rpm: However assuming you know what you are doing...
warning: /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/noarch/kernel-headers-<IP_ADDRESS>-1.cm1.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 3135ce90: NOKEY
Verifying packages...
Preparing packages...
kernel-headers-<IP_ADDRESS>-1.cm1.noarch
Adding RPM to worker chroot: glibc-2.28-18.cm1.x86_64.rpm.
Found full path for package glibc-2.28-18.cm1.x86_64.rpm in /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache: (/mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/x86_64/glibc-2.28-18.cm1.x86_64.rpm)
rpm: RPM should not be used directly install RPM packages, use Alien instead!
rpm: However assuming you know what you are doing...
warning: /mnt/c/Users/sylvian/CBL-Mariner/build/rpm_cache/cache/x86_64/glibc-2.28-18.cm1.x86_64.rpm: Header V4 RSA/SHA256 Signature, key ID 3135ce90: NOKEY
Verifying packages...
Preparing packages...
glibc-2.28-18.cm1.x86_64
error: unpacking of archive failed on file /usr/libexec/getconf/POSIX_V6_LP64_OFF64;60d305a4: cpio: link failed - No such file or directory
error: glibc-2.28-18.cm1.x86_64: install failed
Elevated install failed for package glibc-2.28-18.cm1.x86_64.rpm, aborting. Inspect /mnt/c/Users/sylvian/CBL-Mariner/build/logs/worker_chroot.log for more info. Did you hydrate the toolchain?

I had the same issue, this error seems to be caused by trying to run the build in a folder over a network share. Probably something in with the chroot (symlinking perhaps?) doesn't work as expected leading to this error.
I cloned the repo directly inside the VM and build worked.

can't reproduce this very old issue, closing. please reopen if still an issue.


How to import into Angular project

Hi Im getting the following error when trying import into my angular project after importing.
import { Timer }  from 'tiny-timer';
/node_modules/tiny-timer/dist/tiny-timer"' has no exported member 'Timer'
Any ideas?
Thanks

The module is using default export, so you have to import without the curly braces:
import Timer from 'tiny-timer';

Cheers for the quick reply. that has fixed the import issue, however when  trying instatiate a basic Timer and compiling getting the following error:
[ng] Can't import the named export 'EventEmitter' from non EcmaScript module (only default export is available)
Thoughts?

@mathiasvr FYI I can use tiny-timer in react native by use eventemitter2 module instead of node 'event'.

Closing this as I think it is mostly solved now.
@gasolin Nice, I think removing the node event dependency is a good idea since it is the root of most issues with this module.


Fix nesting of multiple tabs

Fixes #27 but needs Leptos v0.5.4.

Thanks for taking your time looking into this! Using the <Provider> also increased SSR compatibility!


Update text on 2016 US tribal fishery

Will wait until more details about catch.

Tribal catch has been identified but wasn't showing up in our database. It's likely that our total catch therefore is missing these landings, but they represent less than 1% of total catch so it is not worth trying to revise the models at this stage, just ensure that we get everything next year (as noted in issue #252). The shore-based processor which received the landings in question is looking into the cause of the issue so I will hold off on revising text until I know more details.

Sorry for spacing out and not getting to this one. Will work on this now. Never got to bottom of why the tribal catch wasn't included, but have a few leads that I need to chase down. For now will just change text to replace zero with actual value and say it was not included in model but is small (<1% of total catch) and shouldn't make a big difference but will be included in the future.

Was just thinking about this one and had started some possible text to insert in the introduction section (paragraph starting line 81):
"Initial database extractions reported no landings of hake by tribal fisheries in \Sexpr{last.data.yr}.  However, the U.S. advisory panel report on the \Sexpr{last.data.yr} fishery (Appendix~\ref{chap:us-fishery-report}) indicated a tribal catch of 2,470 mt.  The Joint Technical Committee was not made aware of this catch until late in the assessment preparation process, thereby precluding an update to the overall catch this year.  However, this amount of catch is negligible relative to the total catch and thus would have negligible influence on model results and subsequent management forecasts. "
use any or none Ian

@aaronmberger, this text looks good to me so go ahead and commit rather than have me copy and paste and risk a mistake. If we want to get extra-obsessive we would add a note to Table a, Table 1, and Figure 4. However, I think that would be more distracting than helpful to say " U.S. tribal catches are included in the appropriate sector, except in 2016 when they were not included as noted in section 1.4.1."

Ok, I will just add this in the text as mentioned.  I agree that we don't need to discuss anywhere else as it just confuse people.


what is the value in the ratio methods?

Either lightenByRatio does not work or I don't understand what ratio to use.
Great library btw!

I get it, it's a number from 0 to 1, there was a bug on my side of the code, sorry. Still the docs could mention this. Thanks for a great library!

I agree. In hindsight I would have used a different word in the method names. I'll look at updating the docs in a few weeks (very busy currently); I'll happily accept a reasonable pull request for it though.

On 9/09/2016, at 9:12 PM, Daniel Sedlacek<EMAIL_ADDRESS>wrote:
I get it, it's a number from 0 to 1, there was a bug on my side of the code, sorry. Still the docs could mention this. Thanks for a great library!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


chore(fs): release<EMAIL_ADDRESS>

To be merged July 20.
Closes #5008

PTAL, @kt3k

LGTM once #5532 is merged. Though, for after stabilization, we can still improve documentation. Specifically, how different options for different APIs can be used.


Fix two VSIX issues stemming from Reunion -> WindowsAppSDK rename

After the rename, it was decided to revert the VSIX id to the original
name so as not to break upgradability and continuity. Unfortunately,
the repositoryId elements in the individual templates weren't updated.
Additionally, the original rename introduced a typo in the XmlPoke build
target responsible for replacing the placeholder package versions with
the actual package versions.

/azp run

        Query="/ns:VSTemplate/ns:TemplateContent/ns:CustomParameters/ns:CustomParameter[@Name='$WindowsAppSdkNupkgVersion$']/@Value"

This casing also needs to be fixed.

Refers to: dev/VSIX/Directory.Build.targets:218 in bade632. [](commit_id = bade63221ab759f110fcbde2f556daafba345557, deletion_comment = False)

        Query="/ns:VSTemplate/ns:TemplateContent/ns:CustomParameters/ns:CustomParameter[@Name='$WindowsAppSdkFoundationNupkgVersion$']/@Value"

This casing also needs to be fixed.

Refers to: dev/VSIX/Directory.Build.targets:221 in bade632. [](commit_id = bade63221ab759f110fcbde2f556daafba345557, deletion_comment = False)

        Query="/ns:VSTemplate/ns:TemplateContent/ns:CustomParameters/ns:CustomParameter[@Name='$WindowsAppSdkDWriteNupkgVersion$']/@Value"

This casing also needs to be fixed.

Refers to: dev/VSIX/Directory.Build.targets:224 in bade632. [](commit_id = bade63221ab759f110fcbde2f556daafba345557, deletion_comment = False)

        Query="/ns:VSTemplate/ns:TemplateContent/ns:CustomParameters/ns:CustomParameter[@Name='$WindowsAppSdkWinUINupkgVersion$']/@Value"

This casing also needs to be fixed.

Refers to: dev/VSIX/Directory.Build.targets:227 in bade632. [](commit_id = bade63221ab759f110fcbde2f556daafba345557, deletion_comment = False)

Fixed the cases mentioned by alwu.

@DefaultRyan Please cherry-pick fe787c733230e758b3c17dc6c7ea2a5972c5dac9 (https://github.com/microsoft/WindowsAppSDK/commit/fe787c733230e758b3c17dc6c7ea2a5972c5dac9) for the change the removes the Single-project MSIX Packaging project templates.

we want this change in main and experimental. But your change may only for experimental.

In reply to: 894549399

we want this change in main and experimental. But your change may only for experimental.
In reply to: 894549399

If it's broken, it's broken. In this case, I don't see much, if any benefit to isolating this change only to the experimental release branch.

/azp run

/azp run

/azp run


Release flutter preview

Description of changes

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] Breaking changes to existing customers are released behind a feature flag or major version update
[ ] Changes are tested using sample applications for all relevant platforms (iOS/android/flutter/Javascript) that use the feature added/modified

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

Codecov Report

Merging #360 (df256be) into master (6c91cc6) will increase coverage by 0.22%.
The diff coverage is 97.84%.

@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
+ Coverage   84.92%   85.15%   +0.22%
==========================================
  Files         142      143       +1
  Lines        6566     6681     +115
  Branches     1690     1720      +30
==========================================
+ Hits         5576     5689     +113
- Misses        899      901       +2
  Partials       91       91

Impacted Files
Coverage Δ

...nc-modelgen-plugin/src/visitors/appsync-visitor.ts
95.17% <ø> (+0.27%)
:arrow_up:

...delgen-plugin/src/visitors/appsync-dart-visitor.ts
97.75% <97.34%> (-0.23%)
:arrow_down:

packages/amplify-codegen/src/commands/models.js
83.62% <100.00%> (+0.58%)
:arrow_up:

.../validateAmplifyFlutterCapableZeroThreeFeatures.js
100.00% <100.00%> (ø)

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 779c71a...df256be. Read the comment docs.


Upgrade terraform provider signalfx to v6.22.0

Fixes #182
Fixes #181

Looking good. Could you please update to latest ci-mgmt using https://github.com/pulumi/ci-mgmt/actions/workflows/update-workflows-single-bridged-provider.yml to fix lint checks - then good to go!


NETOBSERV-678 Health dashboard link

Fix alert id calculation after monitoring operator update and add health dashboard link:

and here (bottom right):

Please note that this PR should be tested with the health dashboard PR

/lgtm

/ok-to-test

/approve

/ok-to-test


Deploy 070 - Add Flux Strategy

Contract
Address

FluxStrategyProxy
0x76bf500b6305dc4ea851384d3d5502f1c7a0ed44

FluxStrategy
0x57d49c28cf9a0f65b1279a97ed01c3e49a5a173f

Address

Deployer
0x58890a9cb27586e83cb51d2d26bbe18a1a647245

Proposer
0x6a6D776120f7e4a8dba5F6bF49b85cb340Cfe241

Governance proposal tx
Proposer is different from Deployer because the deployer doesn't have enough voting power to create proposal and the proposer address is a hardware wallet
Both contracts are verified on Etherscan
Have updated the deployment sheet with the steps

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[x] Code reviewed by 2 reviewers.
[ ] Copy & paste code review security checklist below this checklist.
[x] Unit tests pass
[x] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

I've verified

[x] The deployed contracts match the source code in the repo
[x] The constants and immutables set on deployment are correct. eg governor
[x] The steps in the proposal are correct


[APM] Fix potential integer overflow on conversion

What does this PR do?
This is a fix for https://datadoghq.atlassian.net/browse/VULN-7917
Use strconv.ParseUint instead of Itoa for integer to string conversion, ensuring the integer is not overflown in the int casting.
Motivation
https://datadoghq.atlassian.net/browse/VULN-7917

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Set tablet locations using conditional mutations

After #3251 is merged into the elasticity branch changes can be made to set the tablet location using conditional mutations.  These changes could also be made in the elasticity branch.  Switching accumulo metadata operations to use conditional mutations will have to be done piecemeal and setting the locations is a good place to start.
The following work is what would likely need to be done.

In the mananger set the future location with a condition that no location is set for the tablet AND no operation is active on the tablet.
In the tablet server set the current location with a condition that the future location is the tserver AND no operation is active on the tablet.
In the manager for the case of a dead tserver delelete the current location with a condition that the dead tserver is the current location AND no operation is active on the tablet.

Completed in #3317


chore: upgrade rusty_v8 to 0.23.0

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

LGTM. Do we have auto publish CI set up in this repo?

Yes


option to suppress tqdm in extract_bs

At line 119 of bispect.py, there is a tqdm loop for 'Extracting in the cube'.
If you want to loop over many files, you should be able to disable this progress bar.
I suggest using the disable keyword argument of tqdm : tqdm(..., disable=True).
Same applies to the 'CP covariance' tqdm in _compute_cp_cov

Hi @benjaminpope, thanks for your comment. In the last release v1.4, the CLI version of AMICAL can deal with many files (with a global progress bar over the files) and one (disappearing) progress bar for each individual file. The proper documentation will come soon but you can check how work the CLI with amical -h (command line).
Anyway, do you think it would be good to disable the progress bar? I do not know why but maybe you were facing some issues?

I don't think we want to disable it in general! But if you want to write a custom loop over many files, using Amical, you probably want options both to suppress verbose behaviour and tqdm, so that you can optionally control logging at the outside of the loop.

You can now add verbose = False in extract_bs() to turn the amical extraction in quiet mode (added in #128).

Yes, verbose was already implemented, I just add the disable of the tqdm (see #128).

Hero!
———————————————
Dr Benjamin Pope (he/him)
Lecturer in Astrophysics
University of Queensland
benjaminpope.github.io

From: Anthony Soulain @.>
Sent: Tuesday, February 15, 2022 8:56:35 PM
To: SydneyAstrophotonicInstrumentationLab/AMICAL @.>
Cc: Benjamin Pope @.>; Mention @.>
Subject: Re: [SydneyAstrophotonicInstrumentationLab/AMICAL] option to suppress tqdm in extract_bs (Issue #121)
Yes, verbose was already implemented, I just add the disable of the tqdm (see #128https://github.com/SydneyAstrophotonicInstrumentationLab/AMICAL/pull/128).
—
Reply to this email directly, view it on GitHubhttps://github.com/SydneyAstrophotonicInstrumentationLab/AMICAL/issues/121#issuecomment-1040132200, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABN6YFJ3R5SULX6HJKOESADU3IWOHANCNFSM5NLPNUMQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


gha-merge-up not always merging up

Problem 1 - not merging up when there are actual changes to files
With some modules (possibly all but I have only seen it on a couple), the merge-up isn't performed when there are changes only to workflow files. There may be other cases as well where the merge-up doesn't happen as expected.
https://github.com/symbiote/silverstripe-multivaluefield/compare/6...6.1

The mergeup job says

No changes found when merging-up 6.1 into 6. Skipping.

Problem 2 - not merging up when there are commits but no file changes to merge
Rhino shows branches as needing to be merged up based on commits, not based on changes to files. e.g. in these screenshots the 1.13 branch needs to be merged up to the 1 branch, but we can see no changes to actual files.

For best results, the merge-up action should always perform a merge when there are commits in one branch that aren't in another branch, even when there is no actual change to files. This will remove edge cases, line up with what Rhino reports, and would automatically resolve problem 1 above as well.
Acceptance criteria

gha-merge-up always makes a merge commit when one branch has commits that don't exist in another branch, regardless of whether there are changes to files

All the existing protections e.g. not merging for composer.json changes etc still apply

I can't replicate Problem 1
What I tried:

forked symbiote/silverstripe-multivaluefield to my emteknetnz account
Deleted the 6.1 branch
Created the 6.1 branch from the 6 branch
Changed a workflow file on the 6.1 branch
Ran the Merge-up workflowon the default 5 branch

Automatic merge went well; stopped before committing as requested
[6 b174234] Merge branch '6.1' into 6
To https://github.com/emteknetnz/silverstripe-multivaluefield
   a29bc37..b174234  6 -> 6
Succesfully merged-up 6.1 into 6

The 6 branch now has the change on it https://github.com/emteknetnz/silverstripe-multivaluefield/blob/6/.github/workflows/ci.yml#L13

Have created PR, linked in description. As mentioned in description, hopefully solving problem 2 fixes problem 1

PR merged. Reassigning to Steve in case you want to keep it open to let it run a few times - if not, feel free to close.

Will just close


Arcgis map is not working till openlayers 3.17.0

My map displayed with WMS Argics Tiles is not working anymore.
This is solved by https://github.com/openlayers/openlayers/issues/5430 in version 3.17.0
(STYLES parameter should be sent with = : &STYLES=&)
The error can be seen here : http://openlayers.org/en/v3.16.0/examples/wms-image.html?q=wms
And solved here : http://openlayers.org/en/v3.17.0/examples/wms-image.html?q=wms
So it would be great to upgrade openlayers to version 3.17.0 at least

#383

Ok, thx for reporting this. I'll take a look at #383 as soon as I can. so to avoid duplicates, I'm closing this one. thx again


Only subscribe users when $sync_member_fields fields have changed

I'm getting a lot of errors from MailChimp saying Recipient<EMAIL_ADDRESS>has too many recent signup requests, since subscribe is firing on every onBeforeWrite() of a Member, and sometimes writes happen frequently.
Perhaps we could add something along the lines of this to onBeforeWrite()? To prevent API calls to MailChimp when nothing has changed.
$syncFields = array_keys(static::get_sync_member_fields());
$matches    = [0];

foreach ($syncFields as $field) {
    $matches[] = isset($changed[$field]);
}

if (!max($matches)) {
    return;
}

Happy to review a pull request if you can put it together. Would be also good to have a test to check this 👍

Appreciate the immediate response! I'll look into it.

You are very welcome @janneklouman


Missed compiled script for 'csrc'

Very wonderful work on the decompostion of NeRF. I want to try you demo. But, I have encounterd a problem when running the training script.
ModuleNotFoundError: No module named 'cosine_triline_feature_cuda'
I found the lib is defined in the csrc, but the 'csrc' missed a script to compile it as a library.
Could you help me or share the script? @TE-KazukiYoshiyama
Thanks
Jie

Thanks for trying our work!
I guess by the 'csrc' missed a script to compile it as a library that you did not use DockerFile which contains the build code using the naive Makefile. Please try with the docker, that is the fastest way to use our work.

Thanks for your suggestion! I will try the docker.

Thanks for trying our work!
I guess by the 'csrc' missed a script to compile it as a library that you did not use DockerFile which contains the build code using the naive Makefile. Please try with the docker, that is the fastest way to use our work.

Hi, I used docker, but "ModuleNotFoundError: No module named 'cosine_triline_feature_cuda'" is still displayed. Could you please provide some help?


Add experimentation service

Adopt the vscode-tas-client package for running experiments

Looks good for the most part! Only other thing I think we need is to add the GDPR classification for the "query-expfeature" telemetry event (like we do in the remote-containers extension: https://github.com/microsoft/vscode-remote-containers/blob/9c5f61a5ff6c390e148c237ab6b0e110f204f015/src/utils/telemetry.ts#L9).

Thanks! Added that classification


What CPU's does Ghidra want support for?

From what point does a CPU architecture become eligible to be included in Ghidra?
For example, what about the CRIS architecture? The vendor that produced these chips doesn't anymore, and the Linux kernel has stopped supporting it in 2018, but it can be compiled for in GCC.
What are the requirements for a CPU architecture to be included?

Any CPU is fair game, especially if there might be some device in existence that someone might need to inspect in a reverse engineering capacity.  You never know.
Someone just submitted an MCS-48 implementation, of which there is most likely no device left other than possibly museums.  I don't see any reason it shouldn't be included as long as it is complete and well done, which it appears to be.  We just haven't had time to fully review it and accept it.  It is a very small processor so it shouldn't need much. In another case someone implemented a Cold War era processor for learning and historical purposes.
That said, how much time is spent developing and then reviewing a processor for acceptance would be based on the general applicability of the processor.  A lightly used processor might not need to be as fully spec'ed out or reviewed as closely as an ARM or X86 processor.  For a processor that might not get much use, having more user pcode ops for things that are very seldom used would be acceptable.
The real issue is how to properly evaluate a processor module during and after development to include a source of trusted sample binaries.


Updated layout and added auto hide for output menu options

Changed the layout of the output menu options to vertical and put them on top of the output box. The menu options will only appear when the user's mouse hovers on the output box.

Thanks, Timmy!
Two things:

Can you use Prettier (with default config) to format your code? So that

the code is consistent with our code base, and
make sure the diff contains exactly the changes you made for easier review.

Please attach a screenshot here to easily see the new UI

No problem, I'll work on fixing that.

You can just push to this branch , it will automatically update this PR.

The new UI.
Please let me know if I need to make additional changes.

Great PR. Thanks, Timmy!

Thanks @chimmyten. But the three options should appear in the output box instead of the code part.

The new UI. Please let me know if I need to make additional changes.

We had a discussion in #321 about the menu position.


Use Tox for testing multiple versions of Python/Django

We hadn't been testing multiple versions of Django; luckily, that sort of multi-environment testing is where tox shines. This PR is easiest to review changeset by changeset. There aren't actually too many changes; the mass deletes are around sphinx output.

Coverage remained the same at 92.979% when pulling ddbdffb97989e33969f4993b6cfa7371fb30eafd on cmc333333:tox into 4044c4168e852b6f3733c9afa3cf89b6d2190233 on eregs:master.

Coverage remained the same at 92.979% when pulling c2ff08d445bb643923a5b68fdd16f2fc07e935ad on cmc333333:tox into 4044c4168e852b6f3733c9afa3cf89b6d2190233 on eregs:master.


It returns object with id and status "initiation" instead of actual response from LLM

code that was working a while ago now returns {
id:string,
status:"initiation"
}
instead of code: number;     status: boolean;     model: string;     gpt: string;     original: string | null | {     code: number;     error: string;     message: string; }
i dont see any updates in docs. what changed?

Hi! For now, I don't have a specific date for the next update of this package.
Please note that it can be somewhat heavy when installing it in your project.
However, you can easily code it yourself by following this guide: https://nexra.aryahcr.cc/documentation/en

it seems the package is returning different response than the one it used to return before. did anything change?

I only changed the structure of my API, everything still works the same
just check the documentation, as the code is the same as in this package
I only modified the organization of the structure

thank you for your reply. and i appreciate you taking time to create this package,
however am not sure if i understand you correctly when you say everything works the same.
the same code await gpt.v1({       prompt: prompt,       model: "GPT-4",       markdown: true,     });
that was working suddenly stopped working without any changes from my end. as in my first comment, it is no longer returning the same json object as before. maybe gpt-4 is no longer supported without providing an api key or something?

once again thanks for this helpful package

This package is no longer working even the examples in readme do not work


Tutorial

Hello - I am extremely new to Python and crawler execution, so I am unsure of how to run this code. Would it be possible for you to record a brief video tutorial on:

How to install the required software (if any) required to run your code
How to properly place all of the code in a directory that can be accessed for running the code
How to install the wheel file
Where/how to run the the final file

I'm hoping this request isn't too difficult or time consuming!
Thank you!

Would you be able to further explain the number of threads, radius, and Boolean usage as well? I don't understand how to properly use those function inputs.
Thanks again!

I've tried pip installing this package but both pip and conda say they can't find it?

Yeah I had to re-install python and check the box for "add pip to PATH".
I do wonder if there is an easy way to download the data to a CSV, and I would like to also add phone number information to the data. Those would both be very useful to me.

There are currently no plans to make a video tutorial, however I updated the readme and hope everything is clearer now.
@alex-waters The repo is currently not available online, so cloning and installing locally is the way to go.
@deeznuts03 the number of threads allows multitasking and leads to a faster execution. the radius is for the radarsearch, check out https://developers.google.com/places/web-service/search#RadarSearchRequests and the all_places bool can be used if you want to retrieve all places in the location, not just the ones with available populartimes data.


Modifying the containerd config file at runtime is unreliable

What happened:
When providing a basic userdata configuration to the EKS node on startup, the node fails to start and join the cluster. If I remove the userdata, it works.
In versions prior to amazon-eks-node-1.29-v20240202, this userdata worked flawlessly.
What you expected to happen: I would expect breaking changes to be documented.
How to reproduce it (as minimally and precisely as possible):
Here is my userdata:
Content-Type: multipart/mixed; boundary="//"
MIME-Version: 1.0

--//
Content-Transfer-Encoding: 7bit
Content-Type: text/x-shellscript
Mime-Version: 1.0

#!/bin/bash
cat /etc/containerd/config.toml | sed -E 's/discard_unpacked_layers = true/discard_unpacked_layers = false/g' > /etc/containerd/config-foo.toml

set -ex
cat <<-EOF > /etc/profile.d/bootstrap.sh
export CONTAINERD_CONFIG_FILE=/etc/containerd/config-foo.toml
EOF
sed -i '/^set -o errexit/a\\nsource /etc/profile.d/bootstrap.sh' /etc/eks/bootstrap.sh

--//--

This is applied to an EKS node on startup.  The node no longer connects to the EKS cluster.
EKS 1.29, although similar reports in this issue on EKS 1.28 - the AMI is the common demoninator here.
Anything else we need to know?:
Environment:

AWS Region: us-east-2
Instance Type(s): many variable types
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion): "eks.1"
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version): "1.29"
AMI Version: amazon-eks-node-1.29-v20240202
Kernel (e.g. uname -a): Linux ip-10-205-184-145.us-east-2.compute.internal 5.10.205-195.807.amzn2.aarch64 #1 SMP Tue Jan 16 18:29:00 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux
Release information (run cat /etc/eks/release on a node):

BASE_AMI_ID="ami-086edf6f7e8d01cff"
BUILD_TIME="Fri Feb  2 16:46:37 UTC 2024"
BUILD_KERNEL="5.10.205-195.807.amzn2.aarch64"
ARCH="aarch64"

(x64 also affected)

Another user has posted some logs here: https://github.com/XenitAB/spegel/issues/217#issuecomment-1926776810

From those logs:
> Created symlink from /etc/systemd/system/multi-user.target.wants/kubelet.service to /etc/systemd/system/kubelet.service.
> Failed to start kubelet.service: Unit not found.

This is the output from these two lines: https://github.com/awslabs/amazon-eks-ami/blob/107df3fb9264e47cfd1e269ac5cc69ab4c4a8e3c/files/bootstrap.sh#L632-L633
Because systemctl enable kubelet succeeded, we know that /etc/systemd/system/kubelet.service exists.
So this error:
> Failed to start kubelet.service: Unit not found.

Is probably referring to one of the dependent services, my guess would be sandbox-image.service. That is created within this block: https://github.com/awslabs/amazon-eks-ami/blob/107df3fb9264e47cfd1e269ac5cc69ab4c4a8e3c/files/bootstrap.sh#L575-L582
When you create your custom containerd config file, that sed isn't changing anything:
> cat /etc/containerd/config.toml | sed -E 's/discard_unpacked_layers = true/discard_unpacked_layers = false/g' > /etc/containerd/config-foo.toml

# diff is empty
> diff /etc/containerd/config.toml /etc/containerd/config-foo.toml

And your custom containerd config file overwrites /etc/eks/containerd-config.toml: https://github.com/awslabs/amazon-eks-ami/blob/107df3fb9264e47cfd1e269ac5cc69ab4c4a8e3c/files/bootstrap.sh#L560-L562
So this comparison will never succeed:
https://github.com/awslabs/amazon-eks-ami/blob/107df3fb9264e47cfd1e269ac5cc69ab4c4a8e3c/files/bootstrap.sh#L575
Which agrees with that log file.

The creation of the sandbox-image.service unit could probably be moved outside that block; but I think you would encounter the same issue on previous AMI releases.
Is there a specific AMI release this worked on?

Is there a specific AMI release this worked on?

Anything prior to this one.

Hm okay, I think I see the problem. When we build the official AMI's with the cache_container_images=true we previously cached the sandbox image. That caused issues with kubelet image garbage collection on 1.29, and it was removed. As a result, we don't modify the default /etc/containerd/config.toml at build time: https://github.com/awslabs/amazon-eks-ami/commit/41dfa2217582a624a3cd582e5f0a93c25f951cad
So the line you're looking for (discard_unpacked_images = true) isn't there.

I'll see what we can do to unblock your use case, but in general I don't think we can guarantee that the sed will always produce the desired result. The --containerd-config-file option is intended for the user to define their entire containerd config, it's not well-suited for injecting a single value into the final config.

Thanks for this. I can probably inject the entire config file in the userdata if that's the recommended approach.

I know there's the risk of configuration sprawl but a flag/argument that could be passed to the bootstrap script to keep the container layers (and thus reverting the behaviour to how it was before the AMI defaulted to discarding them end of last year) would be good too.

@cartermckinnon @danielloader @tico24 we could look at using imports option in config.toml and let folks drop their own snippets in another directory?
https://github.com/containerd/containerd/blob/main/docs/man/containerd-config.toml.5.md#:~:text=as a duration-,imports,-%3A Imports is a

I think that using imports would be a great idea. Talos does a pretty good job using imports to customize Containerd configuration.
https://www.talos.dev/v1.6/talos-guides/configuration/containerd/
Importing from a directory would allow overrides to easily be written with minimal impact on existing solutions.

flag/argument that could be passed to the bootstrap script to keep the container layers

@danielloader What’s the context here?

flag/argument that could be passed to the bootstrap script to keep the container layers

@danielloader What’s the context here?

Using spegel to do intra node container image sharing. It depends on the layers being retained after pull.

Interesting, thanks!
I think https://github.com/awslabs/amazon-eks-ami/pull/1630 would cover that case, and my understanding of the merge behavior is that whatever is defined in the imports will override what EKS writes to /etc/containerd/config.toml.

@cartermckinnon yes you are right with your assumption that imports override existing configuration. I will add this to the documentation in Spegel as soon as we get a new AMI release with #1630, that way hopefully we wont get other users raising similar issues.

Hey guys any-update on the #1630?

This can now be closed and #1630 is merged and part of a release.

I updated the config.toml using below command as eks bootstrap script is using this file as the base file.
sed -i 's/discard_unpacked_layers = true/discard_unpacked_layers = false/g'  /etc/eks/containerd/containerd-config.toml
Once you update this, you can verify this using below command from your worker node.
containerd config dump

@zaidmnsr please check the thread above, we explicitly recommend against that approach, because it's very fragile

@cartermckinnon I agree with you that my approach is fragile.
The userdata provided at the top of the thread as problem statement will fail as default file in eks ami /etc/containerd/config.toml is all commented . So eventually using sed on that and explicitly setting conatinerd file (export CONTAINERD_CONFIG_FILE=/etc/containerd/config-foo.toml )will fail in joining nodes as all contents are hashed.
In the eks bootstrap script, it overwrites the file from location /etc/eks/containerd/containerd-config.toml.
So, I decided to go with that approach on directly updating  /etc/eks/containerd/containerd-config.toml .

@zaidmnsr here is an example of how you can override values without using sed. It is a better option because you are not relying on the Containerd config in the AMI to stay the same.
https://github.com/spegel-org/spegel/blob/main/docs/COMPATIBILITY.md#eks

Our docs touch on this as well: https://awslabs.github.io/amazon-eks-ami/usage/al2/#customizing-containerd-config

As containerd patches entire dictionaries, rather than merging keys, we are also opting to go with editing the EKS template config as part of our pre-bootstrap user-data:
sed -i 's/discard_unpacked_layers = true/discard_unpacked_layers = false/g'  /etc/eks/containerd/containerd-config.toml

While fragile, this is the best solution we've got until merging of imported dicts is supported: https://github.com/containerd/containerd/issues/5837. We cannot rely on customizing-containerd-config as there are tools (like nvidia-device-plugin) which modify the containerd configuration as well.


add calculator to find center of circle by its equation

Proposal
add calculator to find center of circle by its equation
(A clear and concise description of what the proposal is.)

i like to work this issue


chore(ts-webpack): setup e2e testing

All e2e tests work now, just as they do in the jspm-based es2016 and TypeScript skeletons. Couple of things to note:
In order to access the aurelia app element in the tests, in the other skeletons you can just use document.querySelector("[aurelia-app]". However, with the Webpack versions this attribute is not present on the element. So I have added this to the root element of the app:
const rootElem = document.body;
aurelia.start().then(() => aurelia.setRoot('app', rootElem));
rootElem.setAttribute('aurelia-app', '');

In loadAndWaitForAureliaPage in the aurelia protractor plugin, we use the aurelia-composed event to ensure that the page is loaded before executing each test. However, it seems with Webpack that the aurelia app is finished loading too quickly for this, so much so that the aurelia-composed event has actually been fired before the event listener is even added for this simple skeleton application. This would cause all the tests to fail every time. To account for this, we add a second condition that checks if the app is already loaded before also adding the listener:
if (document.querySelector("[aurelia-app]")) { cb("Aurelia composed") }

I'm not entirely sure if this is the best way to be sure the app has already loaded when we can't rely on the aurelia-composed event. For this skeleton I haven't had any further issues with it.
I can try and sort the e2e tests for the es2016-webpack skeleton when I get a chance too.

@axwalker If you can rebase on master that would be great. We had a lot of PRs related to the skeleton so we need to sync your stuff up. Also, if you think @atsu85 's recommendation is good, go ahead and make any changes related to that. Ping me when it's ready to do and I'll get it in.

Done the rebase. I left custom_typings where it was as typings has been added to .gitignore since I initially made this pull request. This way it still mirrors the other skeleton's directory structure. I think some of the typings stuff could be tidied up in both skeletons, but this might be best left to another PR.

I'll get it merged shortly. If you are interested in helping us improve the TS skeletons in any way, we would love that. These skeletons will be the basis for what our CLI will product in the future, so we'd love to make sure the output for our first CLI release is top notch :)


Phase 4 translated to brazilian portuguese

One more phase translated into portuguese. Task relationed this issue

lgtm 💯


fix: faq page filter

Closes #203
Description
Fix bug on FAQ filter that showing a wrong data when we search any keywords
Changes

[x] Add new prop checkDocSize: boolean on SearchForm
[x] Set to false on faq page.

Screenshots
Before

After

Help @ekamuktia to check


Please remove old pypi project

Please remove this old pypi project:
https://pypi.python.org/pypi/django-ckeditor-updated
thank you.

78 downloads in the last day
573 downloads in the last week
2041 downloads in the last month
it may breake things

Yes, you are right. But leaving a more explicit message, that this project gets no updates could be added.

hm.. is there a way to mark pip package as deprected or something like that? Aside of readme that existing users won't see?


Feature/display likes

These are the features implemented in this branch;

When the user clicks on the Like button of an item, the interaction is recorded in the Involvement API and the screen is updated.
When the user clicks on the "Comments" button, the Comments popup appears.

Hi @ Ayokunnumi1
Your task  is complete!  Well done here . it's time to merge it :shipit:

Highlights
📌 No Linters error ✅
📌 Descriptive pull request✅
📌 Good commit messages ✅
📌 Functionality working fine  ✅
Cheers and Happy coding!👏👏👏


Feature/fe patients list deactivation reactivation ui

address https://jira.movember.com/browse/TN-1252
add UI changes to allow staff admin to deactivate/reactivate patient accounts
feature is enabled by config value

deactivated accounts will be hidden by default
checkbox is added to allow viewing of deactivated accounts
UI button added to allow reactivation of a deactivated patient account

@ivan-c ivan, using ajax call is a good idea.  I will try if I get a chance.  The problem I was running into was trying to make sure the state of each data row (e.g. deactivated vs reactivated) stayed when the view of the bootstrap table changed - as when pagination or filtering is introduced.  Had to research the documentation a while (e.g. finding out which event to listen for and which method to use to update field data, etc).


Refactor Monsters Backend

[ ] Think more carefully about needs and refactor accordingly with learnings.

[ ] Write appropriate tests


Build Error: MainForm.resx - Invalid ResX-File.

While trying to build the gui, I got the following error:
Ungültige ResX-Datei. Der angeforderte Wert "Ctrl" konnte nicht gefunden werden.
Zeile 573, Position 5.
Translated: Invalid ResX-File. The required value "Ctrl" could not be found.
As it turns out, I had to translate

Ctrl

to

Strg

(for Germany), which fixed the error. This fix may be different if the locale of your System or Visual Studio is different.

should probably be reported on neo-project/neo-gui if this is still an issue


Table hover is not working when the code is used in the jsx

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
When I imported the code in my react project , in table property I tried using table-hover , but table hover is not working.
Reduced test cases
http://localhost:3000/
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
v 5.1.0

Closing due to missing example.


Fix incorrect behavior of EagerCursor::count() method with foundOnly false flag

This PR fixes issue:
Given document collection has 5 elements
When i create query with limit 2
And priming is enabled
And query is executed
Then eager cursor is returned
When count method is called with foundOnly false flag
Then count 5 is expected to be returned
But count 2 is returned instead
This happens because \Doctrine\MongoDB\EagerCursor that is used as base cursor of \Doctrine\ODM\MongoDB\EagerCursor does not support $foundOnly flag in count() method

This is a limitation of eager cursors: they fetch all data at once to immediately close the database connection and no longer run any queries for the lifetime of the cursor object. Unfortunately, this means that $foundOnly functionality is not supported. We had a discussion on how to deal with this in doctrine/mongodb#209 and decided it's best to leave functionality as is:

The newly created CursorInterface does not include the option as it is not common to all cursors
Regular cursors by default count all elements, including those not returned due to skip/limit options. This is consistent with the behavior of MongoCursor::count()
Regular cursors can be switched to $foundOnly = true, again consistent with MongoCursor::count()
Eager cursors are designed to query the database only once, which is why they can only handle $foundOnly = true.

@alcaeus  But, why eager cursor can't save both true and false foundOnly count values on initialization and return them later in count method ?

Please read the discussion in doctrine/mongodb#209, starting with this comment as to why storing an internal count is not an option.


using custom dataset

📚 Documentation Improvement

import torch

class MyCustomDataset(torch.utils.data.Dataset):
    def __init__(self, train: bool, image_size: int):
        ...

    def __getitem__(self, item):
        ...
        return inputs, targets # Or inputs, targets, additional_batch_items

Hello, the above class is from the documentation to create a Custom Dataset without recipe. Without any example, it is very hard to understand what should I write, for a custom pose dataset.
Anyway, I converted yolonas_pose_n recipe to code with the given script, which is a big help to be honest, and then simply modified data_loaders. Can you please write the above example class with a better example?
Thank you
I'll attach the modified recipe to code just in case
train_coco2017_yolo_nas_n.zip

Thanks for reporting!
For pose estimation you can refer to this Notebook on how to make a new dataset: https://github.com/Deci-AI/super-gradients/pull/1831


Error: Module version mismatch. Expected 48, got 46.

I'm seeing the following error
when I am running
const usb = require('usb');
Error: Module version mismatch. Expected 48, got 46.
node -v
v4.4.3
npm -v
2.15.1
Is there something to be updated?
I ran
npm install --build-from-source
as you suggested here https://github.com/nonolith/node-usb/issues/127, but I'm getting the same result.
Thanks

npm remove and installing after upgrading node and npm worked.

@wonjun27 Same here,
I try every way,but can't solve it .
using in electron
node v6.2.2
electron 1.3.3
log:
Uncaught Error: Module version mismatch. Expected 49, got 48
errer code in  ELECTRON_ASAR.js
return old.apply(this, arguments)

@qqnp1100 npm remove and installing after upgrading node and npm worked.


Support for Lyra

Remove the need for FlowWorldSettings so this plugin can be used in a Lyra based project

Honestly, I have no idea what happened here, as we have so many commits and changes here. It's like merging a version of a plugin from a year ago ;)
Also, I don't understand a change in a core element of a system to, allegedly, support a sample project. Which would wreak havoc on existing projects. It's not explained in the description.


UI Error shows when creating sample Pizzashack API in Publisher UI

Description:
The following UI Error shows when creating sample Pizzashack API in Publisher UI by a user with only creator permissions

This seems to be caused by the fact that only publisher users are able to deploy revisions. This causes the deployment of the Pizzashack API to fail.
It would be better if we could check the user permission before attempting to create/deploy a revision of the sample API.
Steps to reproduce:

Login to the publisher UI with a user who only has creator permissions
Click on the deploy sample API link in the home page.

Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Seems this is fixed in the master branch.


Re-enable SankakuComplexRipperTest

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[ ] a bug fix (Fix #...)
[ ] a new Ripper
[x] a refactoring
[ ] a style change/fix

Description
Re-enable SankakuComplexRipperTest. SankakuComplexRipper seems to have been fixed in #99 (@cyian-1756).
Testing
Required verification:

[x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x] I've verified that this change works as intended.

[x] Downloads all relevant content.
[x] Downloads content from multiple pages (as necessary or appropriate).
[x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[x] I've added a unit test to cover my change.

Coverage increased (+0.4%) to 24.063% when pulling 15c74f5d48d03fef8548333d1cde026a113948d2 on metaprime:test-sankaku into 657ee130ca1bbd1cdcd1b397b4bd9bfdeb854c70 on RipMeApp:master.


Works perfectly

Made my month!  Thanks, Microsoft!!!

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

ID: e40394ca-9a55-d9b5-d427-7c2a2c00cfc8
Version Independent ID: 5a5c17e2-a683-88f6-07a9-2002e814d54c
Content: Live Reload - Xamarin
Content Source: docs/xamarin-forms/xaml/live-reload.md
Product: xamarin
GitHub Login: @pierceboggan
Microsoft Alias: piboggan

Does the App class must be inherited directly from "Xamarin.Forms.Application" class?

Hi @nhwilly
Thanks, for the feedback. Glad you like it.


Adding resonant filter to AudioKit

See issue #453
So far I've added a resonantFilter.swift file (I may do an AKResonantFilter AKNode subclass to complement it but for now I just want to use it with computed parameters) and added it (along with reson.c, thanks @aure for updating Soundpipe 😀) to the OS X/iOS/tvOS projects.
Haven't tested this yet but thought it better to share early so I can get on the right track :)
Two questions:

To get this working, do I need to make sure reson exists within Sporth?
Once ready, are there any tests I should write?

Also, I remember, the Csound reson opcode had an iscl flag which we needed to tweak a bit in order to get it working right in AudioKit 2 (we set it to 1), I see that included in the sp_reson struct, will we need some kind of special handling for that here?
(Still trying to understand how all this ties together...)

I'm also having trouble installing AudioKit locally so I can test things out. Usually I'll have my project in a directory next to my AudioKit fork, and install by adding a line like pod 'AudioKit', path: '../AudioKit-fork' to my podfile and it gets installed as a Development Pod, but nothing shows up in my Xcode project. I've run build_frameworks.sh but still nothing. Am I overthinking this?

This won't work without the sporth version of reson installed into AudioKit.  It looks like Paul added it back to Sporth, so it means just copying some stuff over and changing some lines of code.  If you add me as a collaborator on your fork, I can push these changes to it so this all can live on one pull request.

Cool, just sent you an invite.
It's likely that this particular component won't work 'out of the box' - is there a way I can install AudioKit as a development pod somehow? I don't really want to have to spend ten minutes recompiling the whole framework every time I do a build...

You can't do framework development with CocoaPods.  I develop using playgrounds and I don't compile everything when I make any change.  Xcode does a pretty good job of knowing what needs to be recompiled.  I'll get this reson project jumpstarted for you.

Cheers buddy.
PS I was able to get it working by:

Dragging the AudioKit for iOS project into my own project
Building the AudioKit-iOS framework
Dragging AudioKit.framework into the Link Binary With Libraries build phase in my own project

Now Xcode is smart enough that if I change something in AudioKit, it will only recompile what it has to. I can drop breakpoints in AudioKit code, alt+clicking on keywords takes me to implementations, it's all working good.
Easy!

Looks good but... does this contain the reson Sporth integration?

This should be good to go.  If you make any other tweaks, just make a new pull request.


Log timestamps are missing the year

Probably a winston problem.
{"level":"verbose","message":"tolerance now at 1 synclet tweets for twitter","timestamp":"16 Jan 14:34:12"}

Timezone is unclear and assumed to be system as well.


As an admin, I want to have more options to search the engagements

As an admin, I want to have more options to search the engagements.
##Scenario 1##
Advanced Search button is enabled on the Engagement listing page. The button has an arrow showing if the advanced search is open or closed. Admin clicks the button to open advanced search.
https://xd.adobe.com/view/65ea110d-ea8c-496c-919b-10e890cdea9e-30b9/screen/c50b5e94-c060-44d5-a636-4be6c3004e64/
##Scenario 2##
Advanced Search is opened. We see the following search fields:
Label:
Status
Checkboxes:
Draft
Scheduled
Published
Open
Closed
Label:
Date Created - From
Input: date field
Label:
Date Created - To
Input: date field
Checkbox: Has comments that need review
Label:
Date Published - From
Input: date field
Label:
Date Published - To
Input: date field
Label:
Engagement Phase
Input: dropdown with engagement phases *Not developed yet
Buttons: "Reset All Filters" "Search"
https://xd.adobe.com/view/65ea110d-ea8c-496c-919b-10e890cdea9e-30b9/screen/49c929ce-c321-4ce2-9a12-7e75bc82b5d4/

Hey team! Please add your planning poker estimate with Zenhub @djnunez-aot @saravanpa-aot @tiagograf-AOT @VineetBala-AOT

Please add your planning poker estimate with Zenhub @saravanpa-aot


Feature/modernize

Changes

Removed Credential Handling both Project and the Credentials themselves
Added in Init command
Bumped Rake version and rspec
Cleaned up some misc rubocop complaints except long blocks
Added in small test for init
Readme changes

Not sure how you feel about the first one.. since its a pretty big change on how people would potentially use the tasks. But let me know. I can break this down more to add the init + version bumps with out removing creds if needed.

Thanks for this. I think my preference here is to merge everything except for credentials handling with a minor version bump, then go back and get rid of the credentials handling with a major version bump.
The other option would just be to merge all of this with a major version bump, but I think it would be preferable to allow the use of the other new features without requiring removing credential handling stuff.

Alright sounds good. Ill make two requests one with every change but removing creds and then a subsequent one that handles the removal with a major bump.


Offload some configurations into GBM.

Move tree heuristic configurations into GBTree.
Split up declarations and definitions of GBTree for testing.
Implement UseGPU in gbm to detect whether we are using GPU algorithm.

See  #4542 for more comments and plan.  This is part one of it, one or two more PRs are coming.

@RAMitchell Yes, added a test for that.

Codecov Report

Merging #4553 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4553   +/-   ##
=======================================
  Coverage   79.42%   79.42%
=======================================
  Files          10       10
  Lines        1735     1735
=======================================
  Hits         1378     1378
  Misses        357      357

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 a2042b6...07e235c. Read the comment docs.