Update jck tap file content to support enable jck_custom rerun link

If diagnostic=all, keep all outputs
if diagnostic=failure, filter the testcase and summarize result, same format as openjdk. This will

make it easy to know which test cases failed
enable rerun jck_custom link with failed test cases.

This seems like a nice feature to add, moving this into the 2024 1Q plan.

The interesting thing is for failed or error testcases sometimes the test outputs are stored to tapfile sometimes test outputs are not stored to tapfile. Suppose all those outputs should be kept in tapfile.

The reason that part of stderr message is missing is the parsing logic is if tests failed the output between testcase starts and string testcase_FAILED will be kept. For some reason the output messages are later than the testcase_FAILED message, which is quite common for jck tests( doesn't matter it's passed or failed test cases)
08:14:02  Setup time: 0 seconds
08:14:02  Cleanup time: 0 seconds
08:14:02  Tests found in the suite: 59,288
08:14:02  Test results: passed: 3,657; error: 1
08:14:02
08:14:02  Results written to /home/jenkins/workspace/Test_openjdk17_hs_extended.jck_arm_linux/aqa-tests/TKG/output_17065237431305/jck-compiler-lang-MOD_0/work
08:14:02  Error: Some tests did not pass
08:14:02  Report written to /home/jenkins/workspace/Test_openjdk17_hs_extended.jck_arm_linux/aqa-tests/TKG/output_17065237431305/jck-compiler-lang-MOD_0/report
08:14:03  -----------------------------------
08:14:03  jck-compiler-lang-MOD_0_FAILED
08:14:03  -----------------------------------
08:14:03  jckversion determined to be jck17
08:14:04  lang/MOD/mod004/mod00401m0/mod00401m0.html   Passed. compilation failed as expected
08:14:04  lang/MOD/mod004/mod00401m001/mod00401m001.html   Passed. compilation failed as expected
08:14:04  lang/MOD/mod004/mod00401m01/mod00401m01.html   Passed. compilation failed as expected
08:14:04  lang/MOD/mod004/mod00401m011/mod00401m011.html   Passed. compilation failed as expected
08:14:04  lang/MOD/mod004/mod00401m02/mod00401m02.html   Passed. compilation failed as expected
08:14:04  lang/MOD/mod004/mod00401m021/mod00401m021.html   Passed. compilation failed as expected
08:14

There are two solutions to correctly keep the failed testcases detail info.
One is to update the logic to consider both testcase failed string and the string Testcase Finish (for exampe jck-compiler-lang-MOD_0_FAILED and jck-compiler-lang-MOD_0 Finish). String jck-compiler-lang-MOD_0 Finish will always be at the end of testcase output.
The other option is for jck test if there are failed testcases summary.txt will always have all the testcase result summary.

Could also check if an option or setting in the underlying javatest framework would result in different behaviour.  (User Guide: https://docs.oracle.com/javacomponents/javatest-4-6/cli-user-guide/CLI.pdf).

I think it might be related with test itself as testcases running in a same build might have that output late issue some of them don't.


Avoid running sensu enterprise service in opensource installation

Not sure why this PR https://github.com/sensu/sensu-puppet/pull/409 was rolled-back in this commit https://github.com/sensu/sensu-puppet/commit/932324c44d06ecf2d2e039f70669ded4954ef5c2 but  as the first PR reported I'm getting:
Error: /Stage[main]/Sensu::Enterprise::Service/Service[sensu-enterprise]: Could not evaluate: Could not find init script or upstart conf file for 'sensu-enterprise'
Notice: /Stage[main]/Sensu::Enterprise::Dashboard/Anchor[sensu::enterprise::dashboard::begin]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu::Enterprise::Dashboard/Anchor[sensu::enterprise::dashboard::begin]: Skipping because of failed dependencies
Notice: /Stage[main]/Sensu::Enterprise::Dashboard::Config/File[/etc/sensu/dashboard.json]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu::Enterprise::Dashboard::Config/File[/etc/sensu/dashboard.json]: Skipping because of failed dependencies
Notice: /Stage[main]/Sensu::Enterprise::Dashboard::Config/Sensu_enterprise_dashboard_config[ubuntu1.vagrant.local]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu::Enterprise::Dashboard::Config/Sensu_enterprise_dashboard_config[ubuntu1.vagrant.local]: Skipping because of failed dependencies
Notice: /Stage[main]/Sensu::Enterprise::Dashboard::Service/Service[sensu-enterprise-dashboard]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu::Enterprise::Dashboard::Service/Service[sensu-enterprise-dashboard]: Skipping because of failed dependencies
Notice: /Stage[main]/Sensu::Enterprise::Dashboard/Anchor[sensu::enterprise::dashboard::end]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu::Enterprise::Dashboard/Anchor[sensu::enterprise::dashboard::end]: Skipping because of failed dependencies
Notice: /Stage[main]/Sensu/Anchor[sensu::end]: Dependency Service[sensu-enterprise] has failures: true
Warning: /Stage[main]/Sensu/Anchor[sensu::end]: Skipping because of failed dependencies

I'm working on fixing the tests failed :grinning:

There were test failing before my PR:
Finished in 33.67 seconds (files took 1.19 seconds to load)
362 examples, 7 failures

Failed examples:

rspec ./spec/classes/sensu_enterprise_spec.rb:128 # sensu on Debian when enterprise => true should contain Apt::Source[sensu-enterprise] with release => "sensu-enterprise"
rspec ./spec/classes/sensu_package_spec.rb:93 # sensu package repos ubuntu with puppet-apt installed default should contain Apt::Source[sensu] with ensure => "present", location => "http://repositories.sensuapp.org/apt", release => "trusty", repos => "main", include => {"src"=>false}, key => {"id"=>"EE15CFF6AB6E4E290FDAB681A20F259AEB9C94BB", "source"=>"http://repositories.sensuapp.org/apt/pubkey.gpg"} and before => "Package[sensu]"
rspec ./spec/classes/sensu_package_spec.rb:116 # sensu package repos ubuntu with puppet-apt installed unstable repo should contain Apt::Source[sensu] with repos => "unstable"
rspec ./spec/classes/sensu_package_spec.rb:121 # sensu package repos ubuntu with puppet-apt installed override repo url should contain Apt::Source[sensu] with location => "http://repo.mydomain.com/apt"
rspec ./spec/classes/sensu_package_spec.rb:123 # sensu package repos ubuntu with puppet-apt installed override repo url should not contain Apt::Key[sensu] with key => {"id"=>"EE15CFF6AB6E4E290FDAB681A20F259AEB9C94BB", "source"=>"http://repo.mydomain.com/apt/pubkey.gpg"}
rspec ./spec/classes/sensu_package_spec.rb:131 # sensu package repos ubuntu with puppet-apt installed override key ID and key source should not contain Apt::Key[sensu] with key => {"id"=>"FFFFFFFF", "source"=>"http://repo.mydomain.com/apt/pubkey.gpg"}
rspec ./spec/classes/sensu_package_spec.rb:158 # sensu package repos debian repo release should contain Apt::Source[sensu] with ensure => "present", location => "http://repositories.sensuapp.org/apt", release => "jessie", repos => "main", include => {"src"=>false}, key => {"id"=>"EE15CFF6AB6E4E290FDAB681A20F259AEB9C94BB", "source"=>"http://repositories.sensuapp.org/apt/pubkey.gpg"} and before => "Package[sensu]"

409 was rolled back due to it breaking the ability to use manage_service = false for enterprise users.
Your changes are inline with that I was thinking to fix https://github.com/sensu/sensu-puppet/issues/662
Let me know if you need assistance on this.

@dzeleski It'd be lovely to have the build of this repository fixed travis-ci url
There are some test failing related to sensu package repos ubuntu
Thank you for your help :+1:

@devcfgc I've reached out to Sensu and included @cwjohnston on this. It appears the apt module from puppet was updated a few weeks ago adding a new required fact which is causing the spec test to fail. We are still looking into it but I assume one of us will send a PR to your branch to resolve it.

@devcfgc do me a favor. https://github.com/devcfgc/sensu-puppet/blob/9913cf998fea3e18d9412cb83f0efb9677281d0e/spec/classes/sensu_enterprise_spec.rb#L121
Make that line look like this:
let(:facts) { { :osfamily => 'Debian', :lsbdistid => 'ubuntu', :lsbdistrelease => '14.04', :lsbdistcodename => 'trusty', :os => {'name' => 'Debian'}, }

Push that up and see if this entry goes away: https://travis-ci.org/sensu/sensu-puppet/jobs/242385983#L2169
If that works we can correct the others.

@dzeleski there wasn't any luck :'( , after adding this line:
let(:facts) { { :osfamily => 'Debian', :lsbdistid => 'ubuntu', :lsbdistrelease => '14.04', :lsbdistcodename => 'trusty', :os => {'name' => 'debian'}, } }
I'm still getting:
rspec ./spec/classes/sensu_enterprise_spec.rb:128 # sensu on Debian when enterprise => true should contain Apt::Source[sensu-enterprise] with release => "sensu-enterprise"
Let me know if you need me to do some other test :+1:

@devcfgc I have fixed the apt related failures in https://github.com/sensu/sensu-puppet/pull/663. Will you please rebase your branch on current master and see where that gets us?

nice!! good job, now all the test are green :+1:
@cwjohnston @dzeleski Thank you guys for your help.

Yep no problem. Apologies on re introducing that issue.
@cwjohnston let me know if we want to test this stuff on the enterprise side before merging I can pull it into our lab and test.


operator fails to deploy pods with init container

Error:
A security.properties file with the username and password needs to be supplied.
The file was not found in the properties directory /u01/oracle/properties
Weblogic kubernetes operator version: 2.2.1
POD deployment Yaml:
apiVersion: "weblogic.oracle/v4"
kind: Domain
metadata:
name: domain1
namespace: domain1-ns
labels:
weblogic.resourceVersion: domain-v2
weblogic.domainUID: domain1
spec:
domainHome: /u01/oracle/user_projects/domains/domain1
domainHomeInImage: true
image: "test/:weblogic-image:latest"
imagePullPolicy: "IfNotPresent"
imagePullSecrets:

name: acrcred
webLogicCredentialsSecret:
name: weblogic-admin2
includeServerOutInPodLog: true
serverStartPolicy: "IF_NEEDED"
serverPod:
annotations:
prometheus.io/scrape: "true"
containers:

name: custom
image: "test/:weblogic-image:latest"
volumeMounts:

name: workdir
mountPath: "/u01/oracle/user_projects/domains/domain1/servers/$(SERVER_NAME)/security/"
initContainers:

name: wget
image: "test/security:latest"
env:

name: NEXUS_USERNAME
valueFrom:
secretKeyRef:
name: nexus
key: username
name: NEXUS_PASSWORD
valueFrom:
secretKeyRef:
name: nexus
key: password
name: URL
value: "https://test/repository/weblogic-binaries/security/"
name: FILE
value: "security.tar.gz"
volumeMounts:
name: workdir
mountPath: "/work-dir"
volumes:

name: workdir
emptyDir: {}
shutdown:
shutdownType: Graceful
timeoutSeconds: 45
imagePullSecrets:
name: acrcred
restartPolicy: Never
env:
name: JAVA_OPTIONS
value: "-Dweblogic.StdoutDebugEnabled=true -Dweblogic.debug.DebugWebAppSecurity=true -Dweblogic.log.LogFileSeverity=Debug -Doracle.jdbc.fanEnabled=false"
name: USER_MEM_ARGS
value: "-XX:+UseContainerSupport -Djava.security.egd=file:/dev/./urandom "
adminServer:
serverStartState: "RUNNING"
adminService:
channels:

channelName: default
nodePort: 30701
channelName: T3Channel
clusters:

clusterName: "test-cluster"
serverStartState: "RUNNING"
replicas: 2

Hi @v-2vikc  can you please describe more details of when the error you showed happens?

A security.properties file with the username and password needs to be supplied.
The file was not found in the properties directory /u01/oracle/properties

Can you also upload the create-domain-inputs.yaml used to run the domain home in image sample?
The username and password are usually supplied when creating the domain and I believe the sample scripts update or create the properties file that is used at domain creation time.
Also, you list Operator 2.2.1, have you tried this sample with latest master or develop branch? The domain in image sample defaults to cloning https://github.com/oracle/docker-images/tree/master/OracleWebLogic/samples/12213-domain-home-in-image and this example was updated in past couple months so it's possible something has changed.

Hi @v-2vikc so I just tried adding initContainers to a domain YAML file I had from an existing sample domain to the adminServer section and this worked fine:
  :
  :
  # adminServer is used to configure the desired behavior for starting the administration server.
  adminServer:
    # serverStartState legal values are "RUNNING" or "ADMIN"
    # "RUNNING" means the listed server will be started up to "RUNNING" mode
    # "ADMIN" means the listed server will be start up to "ADMIN" mode
    serverStartState: "RUNNING"
    adminService:
      channels:
    # The Admin Server's NodePort
       - channelName: default
         nodePort: 30701
    # Uncomment to export the T3Channel as a service
    #    - channelName: T3Channel
    serverPod:
      initContainers:
      - name: test
        image: "container-registry.oracle.com/middleware/weblogic:<IP_ADDRESS>"
        imagePullPolicy: IfNotPresent
        command: ["sh","-c","echo The URL is $URL"]
        env:
        - name: URL
          value: "http://www.example.com"
  :
  :

after kubectl apply the updated domain.yaml file and the admin server started I could see the init container results:
$ kubectl logs domain1-admin-server -c test -n domain1-ns
The URL is http://www.example.com

So more details about when you see the error would help... Thanks!


Refactor metrics

This patch refactors the metrics support within fabio to
achieve several goals:

support labels for DogstatD, Prometheus and others
support raw events for statsd and others without aggregation
support multiple backends simultaneously to support migration
make integration of new metrics backends and/or libraries simple
being fully backwards compatible to not break existing setups

One of the main challenges is that fabio generates names for the metrics
targets through templates and that these templates are configurable. A
metrics backend which supports labels needs to generate a different name
for the same metric than one that does not support labels.
Combining this with the need to support multiple different metrics
backends at runtime means that none of the existing metrics libraries
like go-kit/metrics and hashicorp/go-metrics is a good fit. However,
they can be used as drivers which reduces the need for testing and makes
integration of new backends simpler since fabio does not need to rely on
a single metrics library.
This patch is work in progress with the new metrics code in the metrics4
package and the old metrics code in the 'metrics_old' package where it
is kept for reference until it has been migrated or removed. The basic
architecture should be good enough to add more providers and
functionality so that the community can help.
Right now there are two providers "flat" and "label" to demonstrate the
concepts. They provide counter and timers in statsd/dogstatd format and
can be configured with -metrics.target=flat[,label]. Other providers
should be added in the same way.
The metrics_old code should be converted to the new provider scheme.
The go-kit/metrics library currently supports most of the necessary
drivers and is the preferred way of integrating new drivers.
The list below is a probably incomplete list of things that need to be
done:

The configuration is not used except for metrics.target
The drivers from the metrics_old package need to be migrated
Additional drivers from go-kit need to be added
The old rcrowley/go-metrics code relies on 'registries' for
namespacing. After the routing table has changed the registry needs
to be pruned of services which no longer exist so that go-metrics
stops reporting them.
The RX/TX counters in the TCP and TCP+SNI proxy are probably
sub-optimal or broken.
Some counters may not have been initialized properly, e.g. WSConn
WSConn should probably be a gauge which also needs to be set to 0 on
startup.
The approach of injecting a noop metrics provider if none has been
set has already bitten me once since some metrics are reported while
others are not. I need to think about this a bit more.

Fixes #126
Fixes #165
Fixes #211
Fixes #253
Fixes #326
Fixes #327
Fixes #371
Closes #329
Closes #331
Closes #334
Closes #335

As for approach please see https://github.com/fabiolb/fabio/issues/211#issuecomment-375880738

Lets first try to add one go-kit/metrics driver to verify the approach and then we can spread the work to support more.

To test this I'm currently doing this:
# terminal 1
consul agent -dev

# terminal 2
cd $GOPATH/src/github.com/fabiolb/fabio/demo/server
go build
./server -prefix /foo

# terminal 3
cd $GOPATH/src/github.com/fabiolb/fabio
go build && ./fabio -metrics.target=flat,label

# terminal 4
curl -i localhost:9999/foo

I'm wondering whether https://opencensus.io/go.html might be a better foundation. Does someone have some experience with this?

This is getting old. Literally :)

+1

Yeah, I know. I was switching jobs and moving my family to a different country. Lots of changes, new language and all. Also, fabio needs a bigger community of contributors since I have become a bottleneck. I am starting to catch my breath again and get some energy on working on fabio again. However, this project is now way more than what I've ever needed to build. As much as I like programming I'll never be able to keep up with the demand by myself.
The metrics code is still bugging me and I'd like to solve this properly but I could really use some help here. If someone wants to work with me on this we could finally tick this one off.

Maintaining a project like this can be such an under-appreciated, and often times thankless investment, so thanks for letting us know what you've got going on and what help you need. Good luck on the move and all the transitions that has with it.
As far as getting metrics incorporated.. I won't be much help with go, but I would be happy to make myself available for testing metrics in an updated fabio. It'd be most easy for me to test with prometheus, but I can help with others too.

@magiconair what help do you need to get this branch where you're comfortable with merging it?

I'm planning to pick this back up.  I just squashed Frank's commits and rebased off master, it builds and tests pass.  I'm going to see if I can at least get a prometheus and statsd integration going with go-kit

so I have pushed a pretty major update here.  Currently all previously supported backends have been implemented, though there will definitely be changes required for users of statsd and likely graphite as well.  the statsd implementation was rather broken - counters never reset which is a violation of how counters are supposed to work in statsd. Graphite changed because the new implementation is based off the go-kit graphite adapter for their metrics framework.  The main changes are around histograms, since they use a different backend package.  This is unfortunate, however this gets us to the positives: namely, tag support for backends that can support them, and a way forward for backends that don't support tags.
Prometheus has been added, in order to use it you need to configure a listener with proto=prometheus .  this lets you configure it like any other listeners, with TLS support or not, with ACLs etc.
I would love some testers, and assuming people can suffer through the graphite / statsd pains changes, I'd like to get this slated for a 1.6 release sometime in the not too distant future.
I would really appreciate some help in testing the stats backends.  This is still very much a work in progress

Thank you for all your work on this @nathanejohnson!

I would love some testers, and assuming people can suffer through the graphite / statsd pains changes, I'd like to get this slated for a 1.6 release sometime in the not too distant future.
I would really appreciate some help in testing the stats backends. This is still very much a work in progress

Should we build the branch from source to test, or is there a build artifact somewhere we should pull or use?
I haven't been running the statsd backend, but I would be happy to test that out now (with datadog).

@ketzacoatl so I just pushed a new change to this branch that should support tagging with dogstatsd.  I don't have an artifact just now that you can get to easily, but if you just checkout the metrics4 branch and issue a make build that should get you a working binary.  If you'd like something prebuilt that's fine too, let me know GOOS / GOARCH and I can put something up somewhere for you to grab.  It would be nice to do a beta release, but I'm not sure yet the best way to do it without cluttering up the releases page on github.

I don't have an artifact just now that you can get to easily, but if you just checkout the metrics4 branch and issue a make build that should get you a working binary. If you'd like something prebuilt that's fine too, let me know GOOS / GOARCH and I can put something up somewhere for you to grab.

I don't have a golang env setup (though I have set it up in the past), so if you could build one for linux/amd64, that would be grand. I can figure it out if that's not easy to do.

It would be nice to do a beta release, but I'm not sure yet the best way to do it without cluttering up the releases page on github.

Maybe a RC can be released?

fabio_v1.5.15-7-ge2c0310.tar.gz

@ketzacoatl and anyone else interested, this is a tarball with mac and linux builds, as well as a signed manifest.

https://drive.google.com/drive/folders/1lMsA1Q3PFi_xXOvV-ZUNR9GPI4EVFXv8
This is going to be a "private" beta for now, though anyone with the link can download the files.  I did darwin and linux in amd64, I can add others if there is interest.

Sorry, I've been busy. This on my todo list though.

Sorry, I've been busy. This on my todo list though.


Does not work with dangerouslySetInnerHTML

Hello! I ran into this bug while using the library. Linkify does not work when wrapping a component that's created with dangerouslySetInnerHTML. The use-case for that is to generate templates (e.g. blog posts) that was saved in a database.
// This identifies the URI and wraps it with an anchor text
<Linkify>

    Go to https://google.com

</Linkify>

// This did not catch the link
<Linkify>
  <p dangerouslySetInnerHTML={{ __html: 'Go to https://google.com'}} />
</Linkify>

It's a pretty easy fix, the parse function just has to check __dangerouslySetInnerHTML as well as children, but this repo seems fairly dead.

Any updates on this issue?

Any update on this issue?

The is the work around and it worked just fine for me.
dangerouslySetInnerHTML={{ __html: ReactDOMServer.renderToString(<Linkify>Go to https://google.com</Linkify>) }}


Add a cache for "config"

Because getAvailableCommands() is used as the basis for many other
actions, and the payload is so gigantic, plus the fact that it does
not change very frequently, it makes sense to keep that response
cached.
The only way this would become out of sync is if you make changes to
the harmony setup (activities and devices).
To improve on this, it might make sense to see if we can detect when the hub needs a sync, and invalidate the cache then.  Or we can add a cache expiration time.
For most clients, the cache isn't needed. This is only useful for a client that keeps the connection open for a long time -- like a REST web service that dispatches commands to Harmony.

thank you for your contribution and sorry for my very late reply.
would you please have a look at the failed build and my review? i am happy to integrate your changes afterwards.

awesome @jhansche! going to release it right now :) thanks for your contribution!

awesome @jhansche! going to release it right now :) thanks for your contribution!


Compiling wat back to wasm?

I checked wabt and binaryen projects, they do not support features like recursive types, that are present in a module I'm playing with.
So, I wonder whether there is a public tool that can handle wat2wasm with latest proposals implemented. Thanks!

This project supports that with the wast or wat crates, which on the CLI is wasm-tools parse (some examples in the README)
Does that work for your use case?

That's what I need, thanks!

Great! In that case I'm going to close this. Feel free to comment though if something is amiss!


Add Python 3.8 support

Hello, are there plans to support Python 3.8 with the next release?
Mizux: yes target v7.5 release this month
Task:

[ ] Update the docker stuff to build the python package on Linux
[x] Update the release script on MacOS and

[ ]  Update my mac machine to add support to python 3.8

[x] Update the release script on Windows

[ ] Update my Windows VM machine to add support for python 3.8

Duplicate of https://github.com/google/or-tools/issues/1702

should be next release.

Reopen it to track the advancement...

https://github.com/google/or-tools/commit/4021c4ec517ba5c4bcfcb0923dcd873da3b8fa30


Imports or reference missing

In main:
-AXIS_DCT (should be utils.AXIS_DCT)
(https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L7270)
(https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L764)
(https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L789)
-torch.nn (https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L211)
-pred_to_nib (should be utils.pred_to_nib)
(https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L760)
(https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/main.py#L785)
In testing:
Conflict in test_HeMIS when from ivadomed.utils import SliceFilter with the import in loader (https://github.com/neuropoly/ivado-medical-imaging/blob/master/ivadomed/loader.py#L3)

-Need to import torch in the following: test_inference, test_HeMIS, test_training, undo_transform, test_sampler
-In test test_inference incorrect call of: pred_to_nib, IvadoMetricManager, threshold_predictions

[x] main
[ ] testing/


Choice spoilers for June cleaver adventures; the rest of the june cleaver items & effects; parse June Cleaver desc

Ready 2 roll!

Ahh, I had tried examining them manually (as I have found to work for
items), but wasn't aware of that CLI command. Thanks!
On Tue, Jun 7, 2022, 8:43 PM Veracity0 @.***> wrote:

@.**** commented on this pull request.
In src/data/modifiers.txt
https://github.com/kolmafia/kolmafia/pull/801#discussion_r891817358:

@@ -5048,6 +5050,7 @@ Effect	Arse-a'fire	Muscle Percent: +10, Initiative: +25
Effect	Artificially Sweet	Moxie Percent: +100
Effect	Artisanal Retentive	Avatar: "Foodie Giant"
Effect	Artisanal Satiaton	Experience: +3
+Effect	Ashamed	Experience: +10 ,Muscle:  -20%, Mysticality: -20%, Moxie: -20%

Question: How did you generate this line?
Are you aware that you can generate these lines automatically in the
correct formal?
test neweffect c9a095cc280c13e9ae6b44be77b61e79
puts the following in the gCLI - and in your session log, where you can
copy it easily with actual tabs.

2711	Ashamed	blush.gif	c9a095cc280c13e9ae6b44be77b61e79	neutral	none
Effect	Ashamed	Experience: +10, Muscle Percent: -20, Mysticality Percent: -20, Moxie Percent: -20
Similarly,
test newitem 836425101
does the same for items.

10923	fire-roasted lake trout	836425101	fishy2.gif	food	t,d	10
fire-roasted lake trout	1	1	good	0	0	0	0	Unspaded
Item	fire-roasted lake trout	Effect: "Fire-Roasted Feelings", Effect Duration: 50

2715	Fire-Roasted Feelings	fishybone.gif	ecbc67d56ee9229d98ff596524e48cba	neutral	none	eat 1 fire-roasted lake trout
Effect	Fire-Roasted Feelings	Cold Resistance: +3, Hot Damage: +15
Basically, if you know the item or effect descit, you can automatically
generate everything you need in the formal you need it in. No manual entry
and typos.
I am "unresolving" this, only so you can see it easily. I am not requiring
an additional change. :)
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/801#discussion_r891817358, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASZNQVLOTCU4ZLAJCBARU23VN7UBPANCNFSM5YDKVK3Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


RSL Error 1 of 2

When i tried to test Wrapper Classic, it says RSL Error 1 of 2, it only works on my old chromium localhost

i'm sorry to hear that. i think that it has something to do with the framework. i will have to fix that.


ci(github): Do not configure a custom linter version anymore

This is to check whether the issue at 1 is resolved.

Merging despite the unrelated Python test failures.


Implement syntax to remove a parent of a subtree with a hole

It would be convenient to have an ability to specify a rule that would remove an nested parent of some branch of an AST.
Such a parent may have various types of children. Therefore, they should be specified as holes.
An example is given below.
The source:
  graph TD;
      BinaryExpression -- 0 --> StringLiteral;
      BinaryExpression -- 1 --> EnclosedExpr;
      EnclosedExpr -- 0 --> Addition;
      Addition -- 0 --> Variable_a;
      Addition -- 1 --> Variable_b;

The target:
  graph TD;
      BinaryExpression -- 0 --> StringLiteral;
      BinaryExpression -- 1 --> Addition;
      Addition -- 0 --> Variable_a;
      Addition -- 1 --> Variable_b;

Syntax may be like:
EnclosedExpr(#1) -> #1

https://github.com/cqfn/astranaut/pull/55

@rultor release, tag is 0.1.5

@kniazkov OK, I will release it now. Please check the progress here

@rultor release, tag is 0.1.5

@kniazkov Done! FYI, the full log is here (took me 8min)

@rultor release, tag is 0.2.2

@rultor release, tag is 0.2.2

@kniazkov OK, I will release it now. Please check the progress here

@rultor release, tag is 0.2.2

@kniazkov Done! FYI, the full log is here (took me 8min)


CI: DockerHub and canary tests

Add a CI component that pushes to docker hub every release and add canary tests, after that, installation can also just download this DockerHub image instead of rebuilding everytime.

Hi, I would like to address this issue. Below are the steps I plan to take. Please approve them or explain any concerns you have:

I will create a GitHub Actions workflow that builds and pushes each image generated by the "make build" command in the docker-hadoop submodule directory.

I will change the commands from "docker build" to "docker pull" in the mentioned Makefile.

For the canary tests, I will pull the image of that release, run a container, and execute commands to ensure that the container works properly. (Please let me know if you have specific commands you would like to execute.)

After your approval, I will start.

Thanks @mohamadaminkarami, this would be great! You can start by including a test that runs this sample script https://github.com/binpash/dish/blob/main/scripts/sample.sh and we will see where we take it from there

Sure.

Hi,
I wanted to inform you that I have implemented the changes and tested them. Before merging, I need some configurations and updates from your side.

Please create two secrets named DOCKER_USERNAME and DOCKER_TOKEN in the dish repository, setting them to your desired username and token (or password) respectively. (link for help)

Create a repository for each image on DockerHub with the following names:

pash-base
hadoop-pash-base
hadoop-namenode
hadoop-datanode
hadoop-resourcemanager
hadoop-nodemanager
hadoop-historyserver
hadoop-submit (note: this image is not used in the docker-compose file, but it is built in the "build" target of the Makefile)

The repository uses the docker-hadoop submodule, for which I don’t have read/write access. Therefore, I had to create a fork of this repository to make changes. Here is the PR for the changes on this submodule. After merging it, I can update the submodule in my fork and create a PR for the dish repository.

PR link: https://github.com/binpash/docker-hadoop/pull/4

Hi Professor @angelhof, no problem at all!
Understood! For now, I’ve implemented only the jobs that build and push the images to the repository. Once this is merged, we can merge the PR for docker-hadoop. After that, I’ll create another MR to use the updated docker-hadoop and implement the job for the canary tests.

Thank you @mohamadaminkarami! The dockerhub_username can just be binpash right?

It should be the username name that has been set as DOCKERHUB_USERNAME. I think it must be set to adminpash because the images are pushed into these repositories right now:
https://hub.docker.com/u/adminpash

This was done as a mistake, the user is adminpash, but the repositories should go under the binpash organization. See my latest commit on dish:main to understand what I mean. Let me know if taht doesn't make sense.

This is where images should be: https://hub.docker.com/repository/docker/binpash/dish-pash-base/general

Oh, I get it now. I will continue tomorrow—my tomorrow, I mean. :)

Thanks! By the way, is there anyway we can modify the workflow to have separate steps for each docker image, and mark their dependencies? The base step fails for some curl timeout error and now all jobs have to be rerun (whereas if we had them as independent steps, each one could run on its own).

Hi @angelhof. Here is the version of the workflow with separated jobs, as you suggested.

Additionally, here is the version of docker-hadoop that pulls images from Docker Hub rather than building them.

Hi Professor @angelhof,
Here is the last PR.

Thanks a lot for all this Mohamad!


.github/pr-check-license.yaml: add license check scan CI

This PR is to fix #3 and add badge of license check.

Closes #3


Editable ComboBox background should be the same as a text field

Fixed to paint with same colors as text fields


[NEW ICON] Ionic

Skill Name
Ionic
Why?
Ionic Framework is a very popular library used by web developers for creating mobile apps.
Reference Image

You can try https://github.com/AnoyiX/dev-icons

+1

+1

+1


Is there a way to skip the "Unable to access all assets on the device"

onPermissionDenied method not working. I would like to remove the "Unable to access all assets on the device" screen and allow them to continue to the "Continue with limited access" screen automatically if they chose to only give limited access. How can i do that?

onPermissionDenied method not working. I would like to remove the "Unable to access all assets on the device" screen and allow them to continue to the "Continue with limited access" screen automatically if they chose to only give limited access. How can i do that?

I have same problem with you bro..
Please help help 😭

onPermissionDenied method not working. I would like to remove the "Unable to access all assets on the device" screen and allow them to continue to the "Continue with limited access" screen automatically if they chose to only give limited access. How can i do that?

 I have same problem with you bro..
Please help help 😭

My solution;
*Download the library manually from github and add it to the project main directory.

Give path extension from your pubspec.yaml file:

insta_assets_picker:
path: 'insta-assets-picker'
*Open the lib/src/assets_picker.dart file from the library files you added to your project and remove the following method

`ps = await _permissionCheck(); (x2)
`

Related to https://github.com/fluttercandies/flutter_wechat_assets_picker/issues/285. I will expose the limitedPermissionOverlayPredicate parameter so that you can use it.
InstaAssetPicker.pickAssets(
  context,
  ...
  limitedPermissionOverlayPredicate: (PermissionState state) => false, // here return whether to show the limited access screen or not based on the permission state
);

New versions 2.2.1 & 2.3.0-dev.2 were deployed with the change. Please check it out :)

Thanks

fluttercandies/flutter_wechat_assets_picker#285

This is the feature I was looking for! Thanks a lot :)


Adds CASSANDRA_MAX_CONNECTIONS to control pooled connections per host

Fixes #956

Updated to control local host connections per discussions


Add attraction_water_slide

Not as easy as I thought. On the left it is the actual rendering in the PR with two layers. On the right separated layers by the water_slide=* tag, but instead of two, there are 22 layers (missing 2 on the picture)... and finally the rendering is not the same as OpenStreetMap.

In OpenStreetMap the toboggans are separated and one above another one, and even one over himself after a turn. Maybe some features we don't have? I don't see how to do that. Do we choose simplicity and to be ok with the 2 layers version (or the 22 one) or do you want to deepen?

Okay thanks!


Can't connect using RemoteDesktop client but can using WebClient

For some reason when some of our users reset their passwords the RemoteDesktop client keeps bouncing back with a generic username or password is incorrect. However, when the user tries to login via the WebClient they can get in fine. It's almost as if the desktop client is caching credentials somewhere..
I've tried resetting user data within the app and then subscribing to our tenancy again but encounter the same issue. I've also tried to reinstall the app but get the same issue.
Any ideas?

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

ID: 80970cb8-947b-60cb-d28d-f0197cebb803
Version Independent ID: 031f04e5-03fb-09c7-c7ab-25676ec6bc88
Content: Troubleshoot Remote Desktop client Windows Virtual Desktop - Azure
Content Source: articles/virtual-desktop/troubleshoot-client.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@filipdimkin
Thanks for your Question! We are currently investigating and will get back on this.

@filipdimkin
Can you check this link for deleting the saved credentials (Windows Credential manager for stored credentials).

@VikasPullagura-MSFT
Tried removing all entries within Credential Manager and issue persists.
If it helps we can see in the WVD logs a SSL_ERR_LOGON_FAILURE error when the user tries to login using the DesktopClient and a Success message when they try to login using the WebClient.

@filipdimkin Thanks for trying the suggestion and responding back on this.
As this issues needs more investigation and live troubleshooting for resolution I would recommend you to contact azure support. If you have a support plan, requesting you to file a support ticket, else please do let us know, we will try and help you get a one-time free technical support.
In this case, could you send an email to AzCommunity[at]Microsoft[dot]com referencing this thread as well as your subscription ID. Please mention "ATTN: Vikas" in the subject line.
Thank you for your cooperation on this matter and look forward to your reply.

@filipdimkin Hope the issue will be addressed via Azure support.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


ng g --list shows the wrong collection

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [x] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
I don't know but the code makes it seem like it worked once upon a time
Description
 A clear and concise description of the problem...
Running ng g --list in a workspace which has a different default schematic than @schematics/angular prints out:
[--help stuff]

Available Schematics:
  Collection "@schematics/angular":
    library
    workspace-schematic

The schematics listed are indeed the schematics available in the default collection but the default collection is actually @nrwl/workspace.
It looks like the source code used to or intends to print out available schematics for all collections. I think this would be the preferred behavior.
🔬 Minimal Reproduction
npx create-nx-workspace@next ng-bug
cd ng-bug
ng g --list

🌍 Your Environment
CLI 8.0.0-rc.4
Anything else relevant?
This affects all Nx users + more?

@FrozenPandaz, just a note, --list doesn't exists. You are fall backing to --help


Improve handling for displaying readme controls

The readme controls are currently either made visible or hidden on the MouseMove event of the control. If the cursor is located over the readme box, they are shown, and if it is not, they will be hidden; however, if another window eclipses the form and the cursor is moved directly from the readme box to this window, the controls will remain visible. Having this functionality instead governed by the MouseEnter and MouseLeave events provides better behavior in this regard and intercepts fewer events. As an aside, there is currently a bug in which the form control box buttons will highlight if the mouse is moved near the top right corner of the readme box panel while the form is maximized. The message filter seems to be the culprit of this.
Additionally, FMsDGV has been modified to be made more visually appealing. The dark gray background of the grid is a rather jarring contrast to the appearance of the rest of the form; thus, a lighter background has been set to maintain the visual pattern, though I suppose this point is rather subjective. It has also been made impossible to have the individual columns appear "selected", as this should not be possible if the only way cells can be selected is by the entire row.

To the points:

Using the mouse events actually doesn't work like it seems to at first. It's the first thing I tried, and the reason I switched to the mouse hook is because it's not a single-control mouseover detect that's needed, but an area detect. The readme area can be in a state where the RichTextBox needs to be hidden but there still needs to be the readme controls showing (when the readme is HTML, the RichTextBox is hidden, a "View HTML readme" button is shown, and we still need the readme controls because there might be multiple readmes and we need to have the selection box up there). I also remember there being other cases where things didn't work right with the mouse events that weren't obvious at first, and unfortunately I don't remember what they were, but I definitely remember the mosue events came with way too many asterisks and how relieved I was to be able to use the nice mouse hook and skip the whole control hierarchy persnicketiness and just detect a region flat in the window.

I've been noticing the control box buttons highlighting bug myself recently but keep forgetting about it. It's a weird one, and I'll look into it.

I acknowledge that the readme controls stay visible when you mouse off the form, but it seems inconsequential, personally. In order for it to occur, you have to have another window partially overlapping and blocking the readme anyway, so having the top-right controls block a portion of it is the least of your problems then. Still, I'll see if I can maybe use a MouseLeave event in addition the hook to handle that. Of note is that I could use a global mouse hook to detect when the mouse moves out of the window entirely, but the global hook causes a problem where resizing the window becomes glacially laggy (which is random and bizarre). In fact, I'm starting to wonder if the mouse hook library I'm using is buggy. I know how to write my own p/invoke hook code and have just been lazy in using the library, so maybe I'll try that.

You're right that the datagridview background color is a heavy contrast. Personally I kinda like the sharp delineation between "something" and "nothing" (and being used to NDL which looks the same) but I guess it's neither here nor there... I wish more people would give me feedback so I would know whether it's a good idea to make a change like this. It might drive some people crazy. I dunno.

You're absolutely right that the column headers shouldn't highlight individually when the control is clearly set to a mode where there's no concept of individual columns. In fact it's a relatively new thing; you'll notice NDL doesn't have it, because it targets an earlier version of .NET Framework that hadn't yet introduced this. It's supposed to be an accessibility feature for better visibility of the current column, but as you can see it still does it even when we're in whole-row selection mode. I tried for quite a while to get rid of it, but it's controlled by an internal method that I can't even change with reflection. I can get rid of it by disabling a whole swath of accessibility features ("level 2") but who knows what genuinely useful stuff that might break. I see you fixed it by disabling visual styles, but I'm not keen on that solution. I'll tell you what though, .NET Core 3 is coming out very soon, and with it will come MIT-licensed WinForms source code. I'm planning to switch AL to .NET Core 3.1 (which is the "long-term service" release) when it comes out in November, and I'll be able to just grab the DGV code, modify it myself, and then we can keep visual styles but get rid of that godawful highlighting.

Quick testing shows the control box flickering issue goes away when I return from the message filter before the WindowFromPoint() call:
var hWnd = InteropMisc.WindowFromPoint(pos);
That's truly strange, as that call shouldn't be affecting anything, but only returning a window handle. Well, anyway, the only reason I'm using the message filter in addition the hook is that the hook library doesn't return the complete set of values from the p/invoke call (notably it's missing the "which keys are pressed" value so I can't tell if ctrl is pressed which I need).

The issues addressed here are indeed inconsequential to a certain extent, though I believe addressing them would serve to supplement the user experience. Somehow, in my testing, I neglected FMs with HTML readmes. This is easy to account for, fortunately, and brief testing seems to show that using MouseLeave events to supplement the mouse hooks is a viable solution. I have not thoroughly explored the employed hooks library, but if they are the source of this odd control box highlighting issue, perhaps a rewrite would be preferable.
I agree that disabling visual styles in the FMs list is not a particularly elegant approach. Unfortunately, this is the only for such property changes to take effect, as they seem to be completely ignored in favor of the default style. Though if you do plan to migrate to .NET Core 3.1 soon, it would be most advantageous to wait and address this in a more direct manner.
If there is nothing more of use in this request, I will close it.

I fixed the control box highlighting issue. It was a dumb logic error on my part: I was trying to get an x and y position from the prefiltered message before I'd checked whether the message's value was in fact supposed to be a position, thus getting garbage data if it wasn't.
Moving onto the move-off-the-window problem next.
Thanks for reporting these things though, even if I may whine about them sometimes, it's really good that you're finding these visual glitches and corner cases because I might never find them by myself, so thanks. And even if I reject a PR we often get useful ideas from the discussion nonetheless.

Your background color point reminds me again though of the idea of letting users customize the color scheme or theme to the extent practicable, some people were asking for a dark mode for instance. WinForms makes that a little trickier, but I should keep it in mind.

Fixed the readme mouse leave bug.

Excellent. All seems to work well, though including the return statement when intercepting the mouse leave call in the ComboBoxCustom class causes the box to remain highlighted after the mouse has left its boundaries.
The FMs list background color comes down to preference, I suppose. Providing an option may be welcome, provided other controls are recolored accordingly, as you say. Despite the fact that not all elements can be changed, perhaps a "dark mode" can be achieved to a certain extent.

You're right, my bad. Fixed.
As time has gone on I've gotten more and more used to doing my own work to make WinForms act the way I want. I'm pretty sure that scroll bars and tab controls can't be fully recolored by default, but I'm certain that with some custom work I could make it all happen. A custom tab control would be a good idea for other reasons too, and in fact I already essentially have one in the Settings form. I'll add this to my notes for .NET Core 3.

With that, it seems everything brought up here has been addressed. The rest should be facilitated by the general release of .NET Core 3.
Thank you for looking into these.

No prob, thanks for keeping me on my toes.


Question about the offsets of rep-points

Sorry for creating a similar issue with the closed one #21
In the following line, it seems  pts_out_init - dcn_base_offset    are the offsets of rep-points.

https://github.com/microsoft/RepPoints/blob/ee07b3fd1b00e2e1bbcc9116c8d56c2b67ce6536/src/reppoints_head/reppoints_head.py#L278

However, based on the this line, it seems pts_out_init are just the offsets of rep-points.

https://github.com/microsoft/RepPoints/blob/ee07b3fd1b00e2e1bbcc9116c8d56c2b67ce6536/src/reppoints_head/reppoints_head.py#L424

Really appreciate it if anyone could help me with this.

I think that self.offset_to_pts change pts to absolute coordinate in the image. For dcn_base_offset, see https://github.com/microsoft/RepPoints/issues/29


virtual cluster on pai

I wonder how to set virtual cluster before running tasks on pai? I do not want to use the default one, is there any option to use non-default virtual cluster?

Thanks @RayXu14 for the question, the current release does not support set VC for OpenPAI training service, we will consider adding it in later releases.

@RayXu14 , we just released NNI v0.4, which support virtual cluster configuration for OpenPAI.
You just need to add virtualCluster into nni trial config, like:
trial:
  command: python3 mnist.py
  codeDir: ~/nni/examples/trials/mnist
  gpuNum: 1
  cpuNum: 1
  memoryMB: 8196
  image: msranni/nni:v0.4
  virtualCluster: nni

Closing this item as the feature had released in 0.4, @RayXu14 - please feel free to reopen the issue if you find there are any problems or questions.


Add glob support in baseUrl for mono repos

Is it possible to use glob in baseUrl? It's very convenient for mono repos.

It sounds like you want to

inherit paths from a root tsconfig.json using the extends functionality.
have the same textual baseUrl (e.g. "./"), but have it resolved relative to the current tsconfig.json.

Can you confirm that's the scenario you have in mind?

@DanielRosenwasser
My project structure is like this:
|-- packages
    |-- a
        |-- tsconfig.json
    |-- b
        |-- tsconfig.json
    |-- c
        |-- tsconfig.json

And each tsconfig.json now have the same baseUrl: ./src.
Because those tsconfig.json are exactly the same, now I want to only maintain one tsconfig.json in the root of the project like this:
|-- packages
    |-- a
    |-- b
    |-- c
|-- tsconfig.json

So I hope the baseUrl could accept a glob like this: packages/*/src.

Do not think globs is the right thing to do here. you can just set the baseUrl to ./, then add Path mapping for the packages:
{
    "baseUrl": "./packages"
    "paths": {
        "a/*": ["a/src/*"],
        "b/*": ["b/src/*"],
        "c/*": ["c/src/*"]
    }
}


Fix snapshot not closing file

Leaving files unclosed in the autoload causes a warning that triggers and exception in the latest pytest version

/test

/test


Is there any way to install it on Majaro/Arch

Dear @divideconcept thanks for your great job.
I wanted to install this nice package on Manjaro. First, I tried to convert deb file to zst with debtap. Although it has installed successfully, it doesn't run and throws the following error:
torchstudio: /usr/lib/libQt6Svg.so.6: no version information available (required by torchstudio)
torchstudio: symbol lookup error: torchstudio: undefined symbol: _ZTIN11DivideFrame7QWidgetE, versionQt_6

I've also installed the Qt6.
Secondly, I tried to install it from the source. Unfortunately, it just will be run with conda.
I was wondering if is there any way to install it on the top of virtual environment?

Hi @javadr you won't find it as a conda package, it's a standalone app.
That being said, I think your best shot is your first attemp. After installing the zst, make sure you can see the following files:
/usr/local/bin/torchstudio
/usr/local/lib/torchstudio/lib/libQt6* (including libQt6Svg.so.6)
What is supposed to happen is: when launching /usr/local/bin/torchstudio, it's supposed to look in /usr/local/lib/torchstudio/lib/ for the required libraries. Are the files (executable and libraries) properly installed in those folders ?

Hi and thanks for your response.
Actually, the torchstudio has been installed at the following address in my system:
$whereis torchstudio
torchstudio: /usr/bin/torchstudio /usr/local/lib/torchstudio

and the /usr/local/lib/torchstudio/lib directory includes the files below:
ls /usr/local/lib/torchstudio/lib/libQt6*
/usr/local/lib/torchstudio/lib/libQt6Concurrent.so
/usr/local/lib/torchstudio/lib/libQt6Concurrent.so.6
/usr/local/lib/torchstudio/lib/libQt6Concurrent.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Core.so
/usr/local/lib/torchstudio/lib/libQt6Core.so.6
/usr/local/lib/torchstudio/lib/libQt6Core.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6DBus.so
/usr/local/lib/torchstudio/lib/libQt6DBus.so.6
/usr/local/lib/torchstudio/lib/libQt6DBus.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Designer.so
/usr/local/lib/torchstudio/lib/libQt6Designer.so.6
/usr/local/lib/torchstudio/lib/libQt6Designer.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6DesignerComponents.so
/usr/local/lib/torchstudio/lib/libQt6DesignerComponents.so.6
/usr/local/lib/torchstudio/lib/libQt6DesignerComponents.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6DeviceDiscoverySupport.a
/usr/local/lib/torchstudio/lib/libQt6EglFSDeviceIntegration.so
/usr/local/lib/torchstudio/lib/libQt6EglFSDeviceIntegration.so.6
/usr/local/lib/torchstudio/lib/libQt6EglFSDeviceIntegration.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6FbSupport.a
/usr/local/lib/torchstudio/lib/libQt6Gui.so
/usr/local/lib/torchstudio/lib/libQt6Gui.so.6
/usr/local/lib/torchstudio/lib/libQt6Gui.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Help.so
/usr/local/lib/torchstudio/lib/libQt6Help.so.6
/usr/local/lib/torchstudio/lib/libQt6Help.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6InputSupport.a
/usr/local/lib/torchstudio/lib/libQt6Network.so
/usr/local/lib/torchstudio/lib/libQt6Network.so.6
/usr/local/lib/torchstudio/lib/libQt6Network.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6OpenGL.so
/usr/local/lib/torchstudio/lib/libQt6OpenGL.so.6
/usr/local/lib/torchstudio/lib/libQt6OpenGL.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6OpenGLWidgets.so
/usr/local/lib/torchstudio/lib/libQt6OpenGLWidgets.so.6
/usr/local/lib/torchstudio/lib/libQt6OpenGLWidgets.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6PrintSupport.so
/usr/local/lib/torchstudio/lib/libQt6PrintSupport.so.6
/usr/local/lib/torchstudio/lib/libQt6PrintSupport.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Sql.so
/usr/local/lib/torchstudio/lib/libQt6Sql.so.6
/usr/local/lib/torchstudio/lib/libQt6Sql.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Svg.so
/usr/local/lib/torchstudio/lib/libQt6Svg.so.6
/usr/local/lib/torchstudio/lib/libQt6Svg.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6SvgWidgets.so
/usr/local/lib/torchstudio/lib/libQt6SvgWidgets.so.6
/usr/local/lib/torchstudio/lib/libQt6SvgWidgets.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Test.so
/usr/local/lib/torchstudio/lib/libQt6Test.so.6
/usr/local/lib/torchstudio/lib/libQt6Test.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6UiTools.so
/usr/local/lib/torchstudio/lib/libQt6UiTools.so.6
/usr/local/lib/torchstudio/lib/libQt6UiTools.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Widgets.so
/usr/local/lib/torchstudio/lib/libQt6Widgets.so.6
/usr/local/lib/torchstudio/lib/libQt6Widgets.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6XcbQpa.so
/usr/local/lib/torchstudio/lib/libQt6XcbQpa.so.6
/usr/local/lib/torchstudio/lib/libQt6XcbQpa.so.6.2.2
/usr/local/lib/torchstudio/lib/libQt6Xml.so
/usr/local/lib/torchstudio/lib/libQt6Xml.so.6
/usr/local/lib/torchstudio/lib/libQt6Xml.so.6.2.2

I moved the torchstudio file into the /usr/local/bin/ and now everything works like a charm.


[PDI-17830] In repository, File>Open has different functionality depe…

…nding on the perspective
@pentaho/tatooine @pentaho-lmartins
@mbatchelor @bmorrise
According to Bill Faulkner, Open File should have the same behaviour for both DI and Scheduler perspective. When not connected to a Repository it should look into the filesystem. When connected to a Repository it should look into the repository.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  ui/src/main/java/org/pentaho/di/ui/spoon/Spoon.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:


fix(jdbc): reintroduce improved support for Statement#getGeneratedKeys

Hello @gotson,
Over in https://github.com/xerial/sqlite-jdbc/issues/1018#issuecomment-1813516287 you mentioned that you'd be willing to look at a PR that restores getGeneratedKeys functionality if it could resolve the safety issues originally reported in #329.
This PR seeks to do that by adding back support for Statement#getGeneratedKeys with a new implementation that tightly scopes the generated keys to the Statement object. This makes it safe to use in an interleaved manner, across threads, and with multiple Statements.
In the interest of transparency, this PR does not address tracking keys for multiple inserts on the same statement (#468). However, that functionality was never previously available. That it could potentially be added in future enhancements to this model if it became a requirement.
Restoring this functionality would be a huge benefit to the users of this JDBC driver. Many applications and integrations were adversely affected by the removal of the basic getGeneratedKeys functionality that they came to depend on it over the past 16 years. This PR should restore the core functionality with critical improvements to scope and safety.
Please let me know if you have any questions or suggestions for improvements. Thank you for your consideration.

Hi @gotson - just checking in - do you have any thoughts about this PR?

Hi @gotson - just checking in - do you have any thoughts about this PR?

I didn't had time to look at it yet, but will do in January.

seems like code formatting failed, could you run mvn spotless:apply ?

all set, code formatting should be good now!

Thanks a lot for the quality PR!


Size +1 for files served via https

When http_server.c copies the javascript / jquery / json / html files, it calculates the length incorrectly which ends up copying one byte past the end of the text. This breaks the javascript.
0001-Subtracted-one-from-all-file-size-calculations.patch.txt

Hello,
I am not seeing the problem. If i request <IP_ADDRESS>/jquery.js for instance I can see the JS has the correct end. Same for the other files

I've tried this patch, it truncates all the pages by a character.
For instance  becomes </html


Print the Helm Secret

Output of helm version: version.BuildInfo{Version:"v3.8.1", GitCommit:"5cb9af4b1b271d11d7a97a71df3ac337dd94ad37", GitTreeState:"clean", GoVersion:"go1.17.8"}
Output of kubectl version: Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5", GitCommit:"c285e781331a3785a7f436042c65c5641ce8a9e9", GitTreeState:"clean", BuildDate:"2022-03-16T15:51:05Z", GoVersion:"go1.17.8", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{Major:"1", Minor:"21+", GitVersion:"v1.21.5-eks-bc4871b", GitCommit:"5236faf39f1b7a7dabea8df12726f25608131aa9", GitTreeState:"clean", BuildDate:"2021-10-29T23:32:16Z", GoVersion:"go1.16.8", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):  AWS EKS
I really want to print the Helm secret to the terminal. Honestly I don't know what to call this secret. IMHO it is the most mysterious aspect of Helm.
I am talking about the secrets like this which are created after installing a helm chart:
apiVersion: v1
kind: Secret
metadata:
  name: sh.helm.release.v1.apollo-server.v10
  namespace: default
  labels:
    modifiedAt: '1646255650'
    name: apollo-server
    owner: helm
    status: superseded
    version: '10'
data:
  release: >- ...

I would love to have a simple flag on the template/dry-runs like so: helm template foo ./chart --print-secret
There seems to be absolutely no way to get this, other than actually installing into a live cluster. Is there a reason why? Maybe I missed something and I actually can?

Under what circumstances do such requirements arise? @kferrone

One use case is to have Kustomize actually install a helm chart with this: https://kubectl.docs.kubernetes.io/references/kustomize/builtins/#helmchartinflationgenerator
Another use case is to add labels and annotations to the secret before I install.
Currently I have a Webhook to do this after install, probably way more code than if Helm just let me put a label on the secret. This is only possible with a Helm operator because it gives me something to look up and get the needed labels.

There's no feature to add annotations or labels to the storage resource. If you're interested in contributing this as a feature, take a look at https://github.com/helm/helm/blob/main/pkg/storage/driver/secrets.go and https://github.com/helm/helm/blob/main/pkg/storage/driver/cfgmaps.go (be sure to add unit tests). Since the storage driver is selected with environment variables, I'd propose that such a configuration could be exposed through the environment as well.


Crash

https://paste.dimdev.org/opoxuxeboy.mccrash

This doesn't look vw related, it looks like you pressed f3 + c.


Setting feature

I couldn't get pass this error A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0xa0 in tid 5413 (GLThread 66), pid 3757 (nderfootandroid). I don't really know why, but when I redesign the drawer, the error stopped. I think it had to do with database.
I added a setting feature to the, with just 1 item, I can add more items or remove that feature

I run pixel 3 API 31 and I don't have that error. my navigation is quite smooth. Try to invalidate caches and rebuild the app.
The error I do come in contact with is A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0xa0 in tid 22176 (GLThread 78), pid 22011 (nderfootandroid)

Yup, I cleaned the build and invalidated caches and I get exactly the same error on my Pixel 3, and when running a Pixel 3 / API 31 emulator.

What lead to this error?. I do not seem to get this error when I run the app and navigate rock and download fragment.
Try making download the home fragment, or I can help you build up the whole project from scratch.

I don't know what causes the error, but it happens consistently, after cleaning and invalidating caches, on device and in an emulator, whether the default fragments is rocks or water.

I don't know what causes the error, but it happens consistently, after cleaning and invalidating caches, on device and in an emulator, whether the default fragments is rocks or water.

I have a few fix. But since I do not get the error, that means that the error is at your end.

I think the error is in your fragment navigation, the fragment API you are using do not know how to navigate to that fragment.
What I think you should do is move the project to another folder and upgrade your android studio, and gradle. then create another project. now you can just copy paste your previous code to your new project. By consistently testing each stages of this new project you might find where the error is.
You mentioned something about following Google recommended practice . Then you should use jetpack compose. Since this is the future of mobile app development (because of KMM).
I hope theses options help, because I do not get the error when I run the app on my emulator
my emulator is {
Android studio dolphin
Pixel 4 API 31
gradle version 7.4
}

Fair enough. Thanks for the advice!


Argument 'sim' has incorrect type (expected mujoco_py.cymj.MjSim, got Physics)

Hello when i run https://github.com/yardenas/la-mbda, i tried to install this safety-gym environment, but i occurred this issue:
Traceback (most recent call last):
File "experiments/train.py", line 7, in
train_utils.train(config, LAMBDA)
File "/media/changquan/data0/sty/la-mbda/experiments/train_utils.py", line 139, in train
on_episode_end=lambda episode_summary, steps_count: on_episode_end(
File "/media/changquan/data0/sty/la-mbda/la_mbda/utils.py", line 101, in interact
pbar, len(episodes) < config.render_episodes and not training)
File "/media/changquan/data0/sty/la-mbda/la_mbda/utils.py", line 63, in do_episode
observation = environment.reset() if reset_function is None else reset_function()
File "/home/changquan/anaconda3/envs/lambda/lib/python3.7/site-packages/gym/core.py", line 259, in reset
observation = self.env.reset(**kwargs)
File "/home/changquan/anaconda3/envs/lambda/lib/python3.7/site-packages/gym/core.py", line 284, in reset
return self.env.reset(**kwargs)
File "/home/changquan/anaconda3/envs/lambda/lib/python3.7/site-packages/gym/core.py", line 232, in reset
return self.env.reset(**kwargs)
File "/home/changquan/anaconda3/envs/lambda/lib/python3.7/site-packages/gym/wrappers/time_limit.py", line 25, in reset
return self.env.reset(**kwargs)
File "/media/changquan/data0/sty/la-mbda/safety-gym/safety_gym/envs/engine.py", line 887, in reset
self.build()
File "/media/changquan/data0/sty/la-mbda/safety-gym/safety_gym/envs/engine.py", line 863, in build
self.world.reset()
File "/media/changquan/data0/sty/la-mbda/safety-gym/safety_gym/envs/world.py", line 324, in reset
self.build()
File "/media/changquan/data0/sty/la-mbda/safety-gym/safety_gym/envs/world.py", line 299, in build
render_context = MjRenderContextOffscreen(self.sim, device_id=-1, quiet=True)
TypeError: Argument 'sim' has incorrect type (expected mujoco_py.cymj.MjSim, got Physics)

Hi @SHITIANYU-hue, thanks for your interest in la-mbda :)
Seems like an off-screen rendering issue.
For some reason, it seems like your la-mbda installation didn't use the correct branch (dm_hack) when installing safety-gym (see more details here. To give some context: what this branch does is replacing the default mujoco-py backend with dm-control backend to use mujoco. The reason for that is exactly because I was facing issues with off-screen rendering when I used mujoco-py.
I'd suggest two ways to move forward with this issue:

Clone my fork of safety-gym, change branch to dm_hack, and run pip install -e .
Debug why your la-mbda installation and figure out why it doesn't install the correct branch.

Hope this helps!

Hello, thanks for you reply!
I tried to switch to your branch, and install mujoco-py again, it will have this error:       /tmp/pip-install-z2umchxi/mujoco-py_a423904d938b4189baba78cf038f2e6e/mujoco_py/gl/osmesashim.c:1:10: fatal error: GL/osmesa.h: No such file or directory
1 | #include <GL/osmesa.h>
|          ^~~~~~~~~~~~~
compilation terminated.
error: command '/usr/bin/gcc' failed with exit code 1
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for mujoco-py
Failed to build mujoco-py
ERROR: Could not build wheels for mujoco-py, which is required to install pyproject.toml-based projects

Thanks for the update. Two questions back:

Are you using a GPU for rendering? If that's the case, make sure to export MUJOCO_GL=egl before running experiments. This will make rendering use OpenGL rendering (which uses GPUs) instead of OSMesa (which runs on CPUs, and from my experience is quite slow)
Not sure why it's trying to compile mujoco-py but the failure happens there. I'd try compiling it before running LAMBDA. For example, run

import gym

env = gym.make("InvertedPendulum-v1")
# Should be building mujoco-py as it loads the mujoco bindings for the first time

Let me know if that helped


Add doc to enum

Adding enum doc to your client.
Does not require a release, since doc team pulls from Github directly

(message created by the CI based on PR content)

Codecov Report

Merging #2287 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #2287   +/-   ##
=======================================
  Coverage   52.76%   52.76%
=======================================
  Files        5627     5627
  Lines      133882   133882
=======================================
  Hits        70637    70637
  Misses      63245    63245

Impacted Files
Coverage Δ

azure-batch/azure/batch/batch_service_client.py
95.12% <100%> (ø)
:arrow_up:

...h/azure/batch/models/batch_service_client_enums.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


2.0: removeCookies() has no effect

This method does not result in an operation being added to the pipeline.
What's needed:

[x] a unit test for the expected behavior
[ ] a fix

Good catch, give me a sec

https://github.com/aidantwoods/SecureHeaders/commit/03abafff6730cf72805b88bcfbb8cdabeccddb64
This should fix, made a bit of a mess of copy pasting the test above originally 😬
Can you confirm fix?

If the test failed without the new code, and now passes, then all should be fine. :)
From a quick glance, the code looks sensible, though.

Tests were broken up until that final commit (but did test with/without the new operation)
Asking as I literally walked out the door the second after I did git push, maybe I'll give it another test later 😉


Use guzzle6 instead of guzzle 5

Solves https://github.com/jcalderonzumba/gastonjs/issues/14

Check https://travis-ci.org/jcalderonzumba/gastonjs/builds/83550068, test do not pass with this PR.

Also i've just check the error messages, guzzle6 does not support php 5.4. Could you elaborate a bit more on why we should use guzzle6 instead of guzzle5?.

Also i've just check the error messages, guzzle6 does not support php 5.4. Could you elaborate a bit more on why we should use guzzle6 instead of guzzle5?.

We as in Drupal would like to introduce frontend testing using phantomjs. We already use a the default mink driver, but would like to not have to use selenium.
In general I think php 5.4 is not a hard requirement anymore. Its hard to find even a hoster without support for 5.5, and well, all your users have your own servers / a proper hosting anyway.

Sorry I didn't really answer your question. We as in Drupal want to bundle our release in the next time
with a stable version of every library we use. Given that guzzle 6 will have support for much longer than guzzle 5. Its also the current stable version.

Ok, will give a thought about supporting 5.4 or not, because in the end this becomes an issue of dropping support of old PHP versions.
Thanks a lot for helping gastonJS to grow.
@stof What do you think about this?

given the small size of changes necessary here, it may be even better to support both Guzzle 5 and Guzzle 6 (detecting which version is installed).
this way, the driver will be usable for people using any of these versions (in many cases, updating from 5 to 6 is much more complex and so projects may still be stuck to guzzle 5)

+1 to supporting both version for now.

If this works this would be really cool!
What about doing that? Checking for the existence of that particular class?

Mh, anyone might have a clue why this fails?

Now is green, sometimes there are random phantomJS crashes. Is something that happens from time to time.

Cool, thank you!


fix: 🐛 fix nominatim address search

Due to a change in allowed URL formats, the nominatim OpenStreetMap search broke. This PR fixes it.
Background information

Issue and discussion on Nominatim: https://github.com/osm-search/Nominatim/issues/3134
Example of previous and now incorrect request and its result: https://nominatim.openstreetmap.org/search/?q=Augsburg&format=json

Merged since the only tests that failed are for Python 2.7 which are no longer supported by GitHub Actions


Revert worker test changes

@michael-berlin @enisoc
As mentioned in person, reverting this seems to significantly improve test flakiness.

LGTM
Sorry about that :-( Thanks for dealing with it.
I think it's still weird, for example for the second RebuildKeyspaceGraph at the end because MigrateServedTypes should take care of it.
Let's do the revert now and keep things this way. I may further dive into it when the resharding steps are automated.


Plugin complain when hand-written code is importing generated class

Hi,
I have a Java class, ContactPredicates, that import a a generated class, QContact.
When I compile the project (mvn clean compile), the apt-maven-plugin emits an information message (but does its job) when encountering this import statement.
I believe this is misleading. Shouldn't the plugin first generate the code before looking at the hand-written code? (BTW, why does the latter happen?)
Here is my plugin configuration:
<plugin>
	<groupId>com.mysema.maven</groupId>
	<artifactId>apt-maven-plugin</artifactId>
	<version>1.1.3</version>
	<executions>
		<execution>
			<goals>
				<goal>process</goal>
			</goals>
			<phase>generate-sources</phase>
			<configuration>
				<outputDirectory>target/generated-sources/java</outputDirectory>
				<processor>com.mysema.query.apt.jpa.JPAAnnotationProcessor</processor>
			</configuration>
		</execution>
	</executions>
</plugin>

And here is the output of the command:
11:45:13 /work/RTBF/svn/reptel/reptel-webapp/trunk $ mvnocolor clean compile
[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building Repertoire telephonique 1.1-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ reptel ---
[INFO] Deleting /work/RTBF/svn/reptel/reptel-webapp/trunk/target
[INFO]
[INFO] --- apt-maven-plugin:1.1.3:process (default) @ reptel ---
/work/RTBF/svn/reptel/reptel-webapp/trunk/src/main/java/be/rtbf/info/reptel/repository/jpa/ContactPredicates.java:3: cannot find symbol
symbol  : class QContact
location: package be.rtbf.info.reptel.model
import be.rtbf.info.reptel.model.QContact;
                                ^
[INFO]
[INFO] --- jaxb2-maven-plugin:1.6:schemagen (default) @ reptel ---
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ reptel ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 60 resources
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ reptel ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 133 source files to /work/RTBF/svn/reptel/reptel-webapp/trunk/target/classes
[WARNING] Note: Some input files use unchecked or unsafe operations.
[WARNING] Note: Recompile with -Xlint:unchecked for details.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 5.485 s
[INFO] Finished at: 2015-05-28T11:45:22+02:00
[INFO] Final Memory: 25M/81M
[INFO] ------------------------------------------------------------------------
``

Kindly let me know if I have done something wrong.

Kind regards,

Dimitri

Hi Dimitri,
I haven't been able to reproduce this error. Would you mind sharing your entities or a basic project?

Hi John.
Sure, I'm finalizing the squeezing ... :-)
In the meantime, please be informed that I could already pinpoint the problem to be only occurring with Java 6. It is fine with 7 and 8.

Please find 3 files in here: https://gist.github.com/DimitriHautot/26ee0f6234be112762f1
I'm running with Maven 3.2.3.

First of all, thank you for pointing out that this only happens with Java 6 and for providing a basic project.
The error message you're seeing occurs because the apt-maven-plugin scans all source files by default, searching for entities. As you pointed out, the plugin is still generating the Q classes correctly, but if you want to get rid of the error you can move ContactPredicates to a different package and add the following:
<configuration>
    ...
    <includes>
        <include>be.rtbf.info.reptel.model</include>
    </includes>
    ...
</configuration>

Unfortunately, you can only provide a package to the <include> directive.

Thanks a lot John, for the fast & accurate answer!
Indeed, I added the <include /> element to match the package, and it worked fine.
Cheers,
Dimitri


Fix field masks

This PR fixes the way field masks are created and checked. Previously, we assumed that field masks contain full field names. The standard FieldMaskUtil creates masks with short names.
This PR also addresses Error Prone and Javadoc warnings.

@dmitrykuzmin, PTAL.


Issue template format and URI

No functional change, issue template change only:

labels type is array not string.
Using actual labels from repo bug and enhancement.
label as heading not ending in '.', description as content ending in '.'.
URI pointing to hass-vantage not ludeeus.
YAML formatting (RedHat), indentation, quoted text.

Note:

Existing enhancement requests are unlabeled, add enhancement label.
Enhancement URI query searches for all enhancements, even closed, review for correctness.
Bug URI query searches for only open bugs, review for correctness.

Thanks @ptr727 !


ApplePay button via PaymentRequestButtonElement fails with "Page already has an active payment session."

Bug report
Describe the bug
Apple pay button fails with "InvalidAccessError: Page already has an active payment session." after opening and closing it a couple of times.
You do not have such a problem on your apple pay marketing page https://stripe.com/apple-pay
Not sure where to file this I am seeing this in my usage of <PaymentRequestButtonElement>, and it also shows up in this demo app.
Is there a know work around other than reloading the page? This is a pretty bad bug for SPA applications.
To Reproduce

Connect the safari debugger to the iPhone
Tap Apple Pay button
Cancel Apple Pay sheet
Repeat Tapping and Canceling until the button no longer summons an ApplePay sheet. For me it happens after about 4 times.

Expected behavior
The Apple Pay button should always summon the pay sheet
Screenshots

I have a video too but I need to blur out my address. If somebody can tell me of a free or cheap software I can use to blur out the address from the apple pay sheet then i'll post the video too.
If applicable, add screenshots to help explain your problem.
System information

OS: iOS 13.4.1
Browser: safari mobile

Additional context

not reproducible on iOS 14.0

I cannot reproduce, my device is on iOS 14.
It's also an error thrown and passed through from ApplePay itself. Can you reproduce on Apple's own demo site?

@hofman-stripe it is not reproducible on apple's demo site. It looks to be a combination of stripe implementation and older iOS versions, unfortunately i can't update to 13.5 through 13.7 as its 14 only now so I can't test those version :-( . I can't quite tell where this bug stops.
I still get a lot of 13.6 and up traffic onto the site I ant to use this on.

Hi @firemuzzy .
Did you find any solution on this ?
I am also facing same issue in IOS and IPAD.

Any news on this? I'm receiving this error from users with ios 14.6 as well as older ios 13 versions.

For anybody else running into this issue, the error is reproducible (both in this demo and Apple's demo per @hofman-stripe 's callout) by double/rage clicking the ApplePay button. I believe this might be fixable on the Stripe side of things with a no-op for calls to paymentRequest.show() until the returned promise is not pending (also equally fixable in userland if a bit inconvenient)

we see a lot of these @nopelluhh how do we fix in userland ?

Can confirm that @nopelluhh is correct on the diagnosis. We currently see a lot of this in our logs. One possible solution we are investigating is disabling the button submit and enabling on cancel event:
  const [disabled, setDisabled] = useState(false);

  const handleSubmit = (event: React.MouseEvent<HTMLButtonElement>) => {
    event.preventDefault();
    setDisabled(true);
    paymentRequest?.show();
  };

   paymentRequest.on('cancel', () => {
      setDisabled(false);
   });

Haven't tested this robustly yet

I can see same error on Sentry.


Calculation result decimals stay 0

Hello,
I am trying to calculate 2 floats in JQuery. One has 0 decimals, the other has 3 decimals. The numbers before the comma are calculated right. But the decimals itself are not (behind the ','). How can I fix this?
Example:
var inputAmount = '3'; // Value from input
var inputPrice = '2,567'; // Value from input

if(inputAmount == ""){
   inputAmount = 0;
}
if(inputPrice == ""){
   inputAmount = 0.000;
}

var calculatedPrice = (parseFloat(inputAmount) * parseFloat(inputPrice));

setTimeout(function() {
     $('input[name="inputResult"]').autoNumeric('set', calculatedPrice.toFixed(3));  // Set result to input
}, 500);

3 times 2,567 = 7,701
The result however becomes: 6000
Where is the mistake? I can't find it. I tried many different options. Does someone know where it goes wrong?

Matthijs,
If the ',' comma is a thousand separator then I believe your issue is
this line:
var  inputPrice=  '2,567';// Value from input
parseFloat(2,567) // becomes 2
If you are getting a a value from a autoNumeric controlled input use
autoNumeric 'get' method $(selector).autoNumeric('get'); //2567
Hope this helps - if so please close the issue.
Bob
On 8/10/2015 5:32 AM, Matthijs wrote:

Hello,
I am trying to calculate 2 floats in JQuery. One has 0 decimals, the
other has 3 decimals. The numbers before the comma are calculated
right. But the decimals itself are not (behind the ','). How can I fix
this?
Example:
var  inputAmount=  '3';// Value from input
var  inputPrice=  '2,567';// Value from input
if(inputAmount==  ""){
inputAmount=  0;
}
if(inputPrice==  ""){
inputAmount=  0.000;
}
var  calculatedPrice=  (parseFloat(inputAmount)*  parseFloat(inputPrice));
setTimeout(function() {
$('input[name="inputResult"]').autoNumeric('set', calculatedPrice.toFixed(3));// Set result to input
},500);
/3 times 2,567 = 7,701/
The result however becomes: /6000/
Where is the mistake? I can't find it. I tried many different options.
Does someone know where it goes wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/160.

That did the job :)
Thanks @BobKnothe !

Glad it helped
On 8/11/2015 4:19 AM, Matthijs wrote:

That did the job :)
Thanks @BobKnothe https://github.com/BobKnothe !
—
Reply to this email directly or view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/160#issuecomment-129786304.


Get-Help Potentially Incorrect Help Topic for New-ModuleManifest

Steps to reproduce
It appears the help topic for a parameter on New-ModuleManifest is incorrect. I can't be sure, but it doesn't look like the text goes with the parameter
Get-Help New-ModuleManifest -Parameter CompatiblePSEditions

Expected behavior
N/A

Actual behavior
Get-Help returns text for the parameter CompatiblePSEditions that does not appear to match the parameter name.
-CompatiblePSEditions <String[]>
    You can use this parameter to restrict the aliases that are exported by the module. It can remove aliases from the list of
    exported aliases, but it cannot add aliases to the list.

    If you omit this parameter, New-ModuleManifest creates an AliasesToExport key with a value of * (all), meaning that all aliases
    that are exported by the module are exported by the manifest.

    Required?                    false
    Position?                    named
    Default value                None
    Accept pipeline input?       False
    Accept wildcard characters?  false

Environment data

PS>( ( $PSVersionTable.GetEnumerator() | ForEach-Object { ( '{0,26} : {1,-20}' -f $PSItem.Key,($PSItem.Value -join ', ') ) } ) | Out-String -Width 100 ) | Set-Clipboard
                 PSVersion : 5.1.15063.608
                 PSEdition : Desktop
      PSCompatibleVersions : 1.0, 2.0, 3.0, 4.0, 5.0, 5.1.15063.608
              BuildVersion : 10.0.15063.608
                CLRVersion : 4.0.30319.42000
         WSManStackVersion : 3.0
 PSRemotingProtocolVersion : 2.3
      SerializationVersion : <IP_ADDRESS>

Looks like this was fixed already https://github.com/PowerShell/PowerShell-Docs/commit/6944160b09b2e45ea074e2443d3b326e6fc8c31e#diff-52086eee0a9f9d3b416db384cf54ef38, although it doesn't appear to have been published yet

Thanks. Was there some way I could have found that without bothering you?

@RickFlist good question!  In this case, all doc content issues should be in https://github.com/powershell/powershell-docs repo (not this one).  I looked to see if this was a known issue and it looks like our doc writer already fixed it (in the repo).  In this case, there wasn't an existing issue in that repo, so I just looked at the actual doc (it's under the reference folder) and it was already changed.

Thanks. I will try that next time.


Meo pr only

Okay, I though this had fixed it. However, the if condition is evaluating to FALSE even when the source repo is azurecitadel.
I have a test harness on my fork where this works as expected - I can't see the issue but I've run out of time for today :(.
It's a convoluted check as you need to get the source repo from a json file which means a bash shell to evaluate.  That's why I've used outputs and chained them...


Fix size printing format specifier

Fixes build error mentioned in https://github.com/newsboat/newsboat/pull/2650#discussion_r1485613029
@juntuu Thanks for reporting it!

coverage: 59.701%. remained the same
when pulling a37ccb060d9651b332f4c1ff298b508298e511a4 on dennisschagt:fix-size-print
into f5ada99a4413b085d8aeb946d72d258f0b933baf on newsboat:master.


Callback for succesful logging

For instance i have a worker, who just lost connection to redis. I want the process to exit after logging. But if I
log.crit('worker x died:' + err)
process.exit(1)

I never get the log entry, and if i wrap it in a setTimeout or some similar construct, there is the chance that it gets called multiple times as many requests to redis will be failing sequentially now, but i don't care about them as they are noise. So a callback would eliminate this problem and i could exit immediately after i know my entry is safe.
Perhaps this feature is already here somehow and i am over looking it.

By the nature of how logs are sent over tcp, it's unfortunately not possible to bind a callback to a specific log entry having been committed. The TCP connection is just a one-way continuous stream of text -- entries are delimited by newlines, and any given chunk being sent may include portions of one or more entries -- and we never hear anything back.
However I believe there's a way to handle the case you've described, something like this:
let somethingWentWrong = false;

// later, in the logic for exiting on error...
if (somethingWentWrong) return;
somethingWentWrong = true;
log.crit('worker x died:' + err);
log.on('finish', () => process.exit(1));
log.end();

Since the logger is in the worker, calling end() on the stream should be okay -- and then you're just listening for the stream to say it's done sending everything. However I'm not 100% sure this will work because there are really two streams involved. If this solution doesn't work, I'll have to dig in a bit -- the logic for closing the stream was never really as fleshed out as it ought to be.

Hmm that seems to be an awesome take, however :( Error: write after end is the outcome, it basically loops through there never actually sending to le and ending the process.

If you got a Write After End error, you still have handlers in there not obeying 'somethingWentWrong' after it's been set to true (you know, on reflection, I think writable streams have an ended property you can check instead of having a flag like this -- definitely a nicer approach). It may be that the finish event is firing before the network stream is actually terminated, but it also could be that the error caused by writing after end() is responsible. I'll try some tests over here soon though since what you've described should be easy to replicate.

    let errorThrown

    process.on('uncaughtException',finalErr)

    function finalErr(err){

        if (!errorThrown) {
            errorThrown = true
            console.log('final',err)
            log.crit("Worker dying: " + err)
            log.on('finish', function(){process.exit(1)})
            log.end()
        }

    }

    require('./worker')

final [Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED <IP_ADDRESS>:6379]
[Error: write after end]

This has been recently fixed in version 1.6.3


Server-side generation breaks down when the client closes the connection or stops the chat.

The current approach for chat streaming using actix_web sends stream responses to chat requests. However, actix_web lacks features to detect client disconnections, as identified in issue #1313. This causes the server to continue generating tokens even after the client has stopped the chat or disconnected from the server. We will attempt to migrate to axum for streaming chat responses.

Streaming web service migrated from actix_web to axum in #63

Streaming web service migrated from actix_web to axum in #63

After migration to axum, the chat streaming can be broken down if the client close the stream (in Python demo). While, the server unable to detect stream disconnections in the chat UI demo (written with typescript).


Add env variable for target directory

A ncTargetDirectory variable for advanced users determining the installation directory would be useful.

Done with e7b5cb45df0907a77c31cd212b98149c8c405722


Make clippy happy, bring back AHash feature, cleanup/clarify docs, cargo fmt

See subject. Basically everything that was lost from the recent merging mashup and then some.

LGTM. Thanks!


Enhanced RDoc for CSV::Table

Methods:

to_a
to_csv
inspect

Also:

Tuned up some of the call-seq.
Removed references to *-convertible, per new guide: https://github.com/ruby/ruby/blob/master/doc/method_documentation.rdoc#argument-description-if-necessary-

(I know, I've violated the one-topic principle.)

@kou, here's another PR.  This completes all the methods in CSV::Table except #dig.  I have to wait on a change from JeremyEvans for that.

Thanks!


unable to execute "beet popularity"

I recently tried to install beets-popularity with the command:
pip3 install beets-popularity
which gave me some "bdist_wheel" errrors, but ran through and and at the end it said:

Successfully installed beets-popularity [...]

I then uninstalled the package and some dependencies with pip-autoremoveautoremove beets-popularity which uninstalled the following:

munkres 1.0.12
jellyfish 0.5.6
PyYAML 3.12
musicbrainzngs 0.6

I then uninstalled all the other dependencies except beets itself
Then I installed the python wheel package with pip3 install wheel which was obviously missing (see bdist_wheel error above)
After that the installation of beets-popularity went through well but I could not execute
beet popularity because it says

error: unknown command 'popularity'

Installing it via the python3 setup.py install command goes through without error too...
What am I doing wrong?
Note: I am a noob about python and pip...

Just to be sure, did you add it to your config file as described in the documentation? Maybe I should add that to the installation instructions.

Yeah, I feel so dumb now...
I forgot to add 'popularity' to the plugins section within the beets config.
But actually there is no word about adding 'popularity' to the plugins section... Maybe it would be useful to add that information to the installation instructions.
Thanks for the hint though! and for the awesome software! 👍

You're welcome! I added a hint to the installation instructions (https://github.com/abba23/beets-popularity/commit/4cf06512af0e20200b667aaff3128fe6da87a53d), so that nobody will forget about enabling it in the future.


fix: LSDV-5066: Filter row validation to set filter just if value is added

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[x] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[x] Frontend

Describe the reason for change
If user add a new filter row and select the logic to OR, outliner filter is showing all regions again
What does this fix?
It's validating the filter row to set de filter just if the value field is filled.
What libraries were added/updated?
NA
Does this change affect performance?
no
Does this change affect security?
no
What alternative approaches were there?
NA
What feature flags were used to cover this change?
NA
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[x] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[x] integration
[ ] unit

/git merge master

/git merge master


Releases

@JuliaRegistrator register

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/46148
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.1 -m "<description of version>" 21e43ce3a83bb9e77e8cd54a2477d59ef2be4a3b
git push origin v0.3.1

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/50248
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.2 -m "<description of version>" 427b91e4c4ba3e551a908ed4b0b1e9a78811dba4
git push origin v0.3.2

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/56583
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.3 -m "<description of version>" 7f22623d072c4faf217d37f61568e3b7c60e5a3e
git push origin v0.3.3

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/57353
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.4 -m "<description of version>" da4bbfc825e929670f50c3a5d838994629a656ac
git push origin v0.3.4

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/64700
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.5 -m "<description of version>" b64237ed884fcfef35710d1dd908193b4146043e
git push origin v0.3.5


[🐛 BUG]: unknown flag for long config options

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
unknown flag: --config .rr.temporal.prod.yaml only happens when try to start docker container with command arguments. When inside the container all long options works. And short option names is works both inside and outside the container
Version (rr --version)
2023.3.6+
How to reproduce the issue?
version: '3'

services:
    temporal-worker:
        restart: "no"
        user: "1000:1000"
        image: ghcr.io/roadrunner-server/roadrunner:2023.3.10
        command: [
          "--config .rr.temporal.prod.yaml",
          "--override temporal.address=temporal:7233",
          "--override temporal.activities.num_workers=2",
          "--override temporal.activities.supervisor.max_worker_memory=128",
          "--override endure.grace_period=120",
          "serve"
        ]

Relevant log output
`unknown flag: --config .rr.temporal.prod.yaml`

Hey @astronom 👋
I guess, you should use commands separately, e.g.:
version: '3'

services:
    temporal-worker:
        restart: "no"
        user: "1000:1000"
        image: ghcr.io/roadrunner-server/roadrunner:2023.3.10
        command: [
          "--config",
          ".rr.temporal.prod.yaml",
          "--override",
          "temporal.address=temporal:7233",
          "--override",
          "temporal.activities.num_workers=2",
          "--override",
          "temporal.activities.supervisor.max_worker_memory=128",
          "--override",
          "endure.grace_period=120",
          "serve"
        ]

Because --config option treated by the process like a single flag.

Thanks, you are right! The issue can be closed