Auto-Healing the token for Undead Fortitude does not seem to work

Using current foundry and dnd5e system version; when rolling a save that passes the DC for Undead Fortitude, the token is not healed for 1 hp.

I just ran a test manually changing the health and rolling a sucess as well as using midi-qol. In both cases the token was healed to 1 HP.
I tested this with the both the Quick Save and the Advanced save.
Which version of Foundry and D&D are you on?

I think this might be an incompatibility with Ready Set Roll? In console it says that the result could not be parsed.

Yes, that seems to be the culprit. Looking at the comments in the code DnD5e Helpers (where this module was split from) didn't support Better Rolls, which is the previous module that Ready Set Roll is based on.

Doesn't look like there would be a way to handle this unfortuantely.


DATA: Contacts entity

to allow for the refugee or any other user entity to have specific contacts I propose the following data structure

email:string
phone?: string (using ISO e.10)

all fields are optional for contacting the user except email which I think should still be required for all users at this point in time.
so on Refugee entity the link would be 1-1

@Spectralgo  can we go ahead with this?

You can. But don't try to implement nullable with this Elvis notation or required with a notation attribute.
Make it simple and dumb, no logic should be in the model.

What about creating a configuration file?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Florian Renard @.>
Sent: Saturday, May 21, 2022 6:01:27 AM
To: hassanhabib/Refugee.Core @.>
Cc: Bhavesh Ramburn @.>; Author @.>
Subject: Re: [hassanhabib/Refugee.Core] DATA: Contacts entity (Issue #78)
You can. But don't try to implement nullable with this Elvis notation or required with a notation attribute.
Make it simple and dumb, no logic should be in the model.
—
Reply to this email directly, view it on GitHubhttps://github.com/hassanhabib/Refugee.Core/issues/78#issuecomment-1133533183, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACUTT3NMHCSBWJZZ7SM5UXTVLBU2PANCNFSM5WI4Z2WQ.
You are receiving this because you authored the thread.Message ID: @.***>


AUTO: test - 66

[Auto generated]
Number: [#66]

Closing issue for PR: AUTO: test.
PR was closed.


OSSM-4360 Avoid flaky in certmanager test cases

Jira: https://issues.redhat.com/browse/OSSM-4360
To avoid cert-manager test cases to be flaky I change this:

added certmanageroperator package with some definitions:

InstallIfNotExist to validate if the operator exists or not in the cluster, if not exist will install the operator
Uninstall to delete the operator

Delete the entire setup done in the main_test.go file and know we validate before the test execution if the operator is installed or not, with this we avoid the panic failure that we have before
Added a "hack" to avoid increasing the timeout for all the WaitPodReady, by default, WaitPodReady makes retry over the verification, so I wrapped in another retry (I know this is "ugly") but with this, we can make another retry to wait the cert-manager to be running (Takes almost 2 minutes to be ready)

I test locally with success:
--- PASS: TestIstioCsr (313.94s)
PASS
ok  	github.com/maistra/maistra-test-tool/pkg/tests/tasks/security/certmanager	317.325s

--- PASS: TestPluginCaCert (110.67s)
PASS
ok  	github.com/maistra/maistra-test-tool/pkg/tests/tasks/security/certmanager	114.074s

Tested the new changes:
--- PASS: TestIstioCsr (216.73s)
PASS
ok  	github.com/maistra/maistra-test-tool/pkg/tests/tasks/security/certmanager	220.899s

DONE 1 tests in 221.329s


Remove invoke & dev-null

dev-null is being replaced in favour of shhh helper in ember-reactive-helpers.
invoke can be accomplished in combination of shhh & compute i.e. {{shhh (compute ... )}}

/cc @zigahertz

thanks @taras! very nice! and I like the naming :-)


Fixing Solc 0.7 support

In order to support 0.7 and newer compilers, we must match ">0.6" instead of "^0.6".

Thanks @villesundell! I didn't think of doing this simple change. There may be other things that break, although it sounds unlikely. Have you tried this yourself?


Autocomplete for Request headers

Autocomplete for request header keys.
Like the user should not have to type the entire work "Content-Type"

We need to support the commonly used request header types as well as allow the user to enter custom headers.
Lets see if we can make this work without having to use any autocomplete react library.
I have implemented dropdown like this in an older project (that is not completed yet) see here
We will implement the Header Value autocomplete once this has been done for Header Names.

I have started reading the codebase and will work on this feature.

@helloanoop Can I get some more details on how to style the component? What kind of style should pop up or what should happen interactively, as we start typing?

@shash68i
How far are you in developing this?
I am thinking that it might not be a bad idea to use a lightweight library like https://github.com/reactjs/react-autocomplete
The list of headers can be taken from here https://github.com/for-GET/know-your-http-well/blob/master/headers.md
Here is the demo of the interaction https://jsbin.com/fireqebamo/edit?js,output
I would recommend not to worry too much about styling. You can keep it very simple and minimalist as seen in the above jsbin demo.

Sounds good!
Sorry for having you redo stuff. Thanks for understanding!

This has been completed. And is available in release 0.19.0


Upgrade @polkadot/api for parachain types

The API types for Rococo parachains are now bundled in the latest @polkadot/api. See hrmp.ts for example.
Have tested a parachain launch successfully after the upgrade. The parachain collator produced blocks successfully.
Also updated the README to reflect the de-facto installation method for the tool, as the existing steps will result in users getting an ancient incompatible version installed from NPM.

This is now quite out of date and shouldn't be merged, so closing for now.
To get polkadot-launch working with the very latest rococo-v1, need to run yarn upgrade @polkadot/api and also add types for MultiAddress to the config (polkadot now uses MultiAddress).

This is now quite out of date and shouldn't be merged, so closing for now.
To get polkadot-launch working with the very latest rococo-v1, need to run yarn upgrade @polkadot/api and also add types for MultiAddress to the config (polkadot now uses MultiAddress).


Need help

Hello,
I want to create an app and response route after event bus response.
Is it possible with apistar? or any alternative method for this situation?
Thank you very much for help!
Best Regards
This code try to response from hello_world def.
Here is the example code;
from apistar import ASyncApp, Route
import uvicorn
import time
from event_bus import EventBus

bus = EventBus()

@bus.on('hello')
def subscribed_event():
    print('World!')
    return {'hello': 'response from event bus'}

def some_func():
    bus.emit('hello')

async def hello_world() -> dict:
    some_func() #i want to response after this event response...
    #return {'hello': 'i do not want to response here...'}

routes = [
    Route('/', method='GET', handler=hello_world)
]

app = ASyncApp(routes=routes)

if __name__ == '__main__':
    app.serve('<IP_ADDRESS>', 999, debug=True)
    #uvicorn.run(app, host='<IP_ADDRESS>', port=999, debug=True, access_log=False)

Hiya,
We use the issue tracker for bugs and feature request on the package. For discussion and usage questions please see https://discuss.apistar.org (Although also not the big 0.6 change removes the server completely, see #624)


Upgrade to Jetty 9.2.0 - same version as in Camel 2.15.x

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/550/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/551/
Test PASSed.


cannot build PYTHON_VERSION=3.7.0  --> gpg: no valid OpenPGP data found.

I know y'all pushed a revolutionsystems/python:3.7.0-wee-optimized-lto image to dockerhub but I can't reproduce the build with the code in this repo.
Details:
Winebook:optimized-python-docker kevin$ env
PYTHON_VERSION=3.7.0
docker build .
[...]
Step 11/44 : RUN    curl -L -o /python.tar.xz "https://www.python.org/ftp/python/${PYTHON_VERSION%%[a-z]}/Python-$PYTHON_VERSION.tar.xz"     && curl -L -o /python.tar.xz.asc "https://www.python.org/ftp/python/${PYTHON_VERSION%%[a-z]}/Python-$PYTHON_VERSION.tar.xz.asc"         && gpg --keyserver ha.pool.sks-keyservers.net --refresh-keys 2>&1 | egrep -v 'requesting key|not changed'         && gpg --batch --verify /python.tar.xz.asc /python.tar.xz         && mkdir -p /usr/src/python         && tar -xJC /usr/src/python --strip-components=1 -f /python.tar.xz
---> Running in 59c774495749
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   162  100   162    0     0   1024      0 --:--:-- --:--:-- --:--:--  1031
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   162  100   162    0     0   1034      0 --:--:-- --:--:-- --:--:--  1038
gpg: keyring `/root/.gnupg/secring.gpg' created
gpg: key CA57AD7C: invalid direct key signature removed
gpg: key CA57AD7C: invalid direct key signature removed
gpg: key CA57AD7C: "PGP Global Directory Verification Key" 2 signatures cleaned
gpg: Total number processed: 1
gpg:     signatures cleaned: 2
gpg: refreshing 21 keys from hkp://ha.pool.sks-keyservers.net
gpg: key AA65421D: "Ned Deily (Python release signing key<EMAIL_ADDRESS>12 new signatures
gpg: key 18ADD4FF: "Benjamin Peterson<EMAIL_ADDRESS>3 new signatures
gpg: key 6F5E1540: "Ned Deily<EMAIL_ADDRESS>2 new signatures
gpg: key CAC5C305: "Alex Vandiver<EMAIL_ADDRESS>4 new signatures
gpg: key 8EFE0655: "Zev Benjamin<EMAIL_ADDRESS>4 new signatures
gpg: key A4135B38: "Benjamin Peterson<EMAIL_ADDRESS>1 new signature
gpg: key 830F6F7E: "Ernest W. Durbin III<EMAIL_ADDRESS>1 new user ID
gpg: key 830F6F7E: "Ernest W. Durbin III<EMAIL_ADDRESS>5 new signatures
gpg: key 498C1579: "Benjamin W. Smith (The Groundwork<EMAIL_ADDRESS>4 new signatures
gpg: Total number processed: 21
gpg:              unchanged: 13
gpg:           new user IDs: 1
gpg:         new signatures: 35
gpg: 3 marginal(s) needed, 1 complete(s) needed, PGP trust model
gpg: depth: 0  valid:   1  signed:   3  trust: 0-, 0q, 0n, 0m, 0f, 1u
gpg: depth: 1  valid:   3  signed:   6  trust: 3-, 0q, 0n, 0m, 0f, 0u
gpg: next trustdb check due at 2018-09-21
gpg: no valid OpenPGP data found.
gpg: the signature could not be verified.
Please remember that the signature file (.sig or .asc)
should be the first file given on the command line.
The command '/bin/sh -c curl -L -o /python.tar.xz "https://www.python.org/ftp/python/${PYTHON_VERSION%%[a-z]}/Python-$PYTHON_VERSION.tar.xz"     && curl -L -o /python.tar.xz.asc "https://www.python.org/ftp/python/${PYTHON_VERSION%%[a-z]}/Python-$PYTHON_VERSION.tar.xz.asc"         && gpg --keyserver ha.pool.sks-keyservers.net --refresh-keys 2>&1 | egrep -v 'requesting key|not changed'         && gpg --batch --verify /python.tar.xz.asc /python.tar.xz         && mkdir -p /usr/src/python         && tar -xJC /usr/src/python --strip-components=1 -f /python.tar.xz' returned a non-zero code: 2

Ah probably a bug that didn't get pushed to the repo.  Thanks for reporting it to us!
@gladiatr72 can you take a look at this in the near future?

Ugh.  Near future---far future :)  On it.  Thank you for the repot, @kbrown!

@kbrown Hey there. Finally got a chance to look at this--what you were seeing was a failure looking up public gpg keys on the sks-keyserver.net key server.  It's f*g amazing in how it's a free service that is only slightly wobbly, but, alas, it is a little wobbly.


this fixes a bug with fly version 3.9.2

the stream(pipe) is not being consumed, and blocks indefinitely
this solution writes the config file to a temp file
tests are still broken. this commit is to communicate what is broken,
and to unblock our team.
Signed-off-by: Mark Gangl<EMAIL_ADDRESS>

Thanks for submitting this PR! We are currently working on a similar solution so we will compare what we've done so far to this PR and figure out how to resolve this issue.

Thank you for this PR, however this issue has been fixed in the 0.19.0 of the release. Please feel free to take that release from the github releases and let us know if you run into any issues.

@anoop2811 & @ram-pivot


MTV-1722: AIO buffer performance

JIRA

MTV-1722

PREVIEW

Increasing asynchronous I/O (AIO) sizes and buffer counts for NBD transport mode

I think 9.11.1 section needs to be recompiled to have all in one better wording

need to edit section 9.11.2
since the forklift controller name prefix is dynamic  , and on the current example its hard coded
so please change  it to this :
`Validating Controller Pod support for AIO values
Ensure that the forklift-controller pod in the openshift-mtv namespace supports the AIO buffer values.
Since the pod name prefix is dynamic , check the pod name first  by running the folowing command:
oc get pods -n openshift-mtv | grep forklift-controller | awk '{print $1}'
output : forklift-controller-667f57c8f8-qllnx
This is the pod name prefix from the example : forklift-controller-667f57c8f8-qllnx
Check the environment variables of the pod by running:
`oc get pod forklift-controller-667f57c8f8-qllnx -n openshift-mtv -o yaml``

@dgur && @TzahiAshkenazi - thanks for the reviews lads
i think i have captured the changes you requested. Please could you do second-eyes
Thanks

It is important to Mention that those Settings only work with Cold Migration.

@dgur & @TzahiAshkenazi - thanks for the reviews
i have added the changes, where possible, and made compromises where not.
i have updated the preview
please can i ask you to do another review?
thanks

@mnecas && @fabiand - Please can I ask for your comments on this PR? I think we are currently saying Not to Publish.

@dgur & @TzahiAshkenazi - thanks for the comments. Changes made and the preview updated

@anarnold97 hi, Can you please update about the changes @TzahiAshkenazi asked to change?

@MrQuality - please be aware that this is not the only item in my work queue. However, i have updated this PR and the preview
@dgur - FYI

Reviewed Looks good for me

@TzahiAshkenazi & @dgur - updated and preview updated

@anarnold97
i have added missing command for section :
Editing ForkliftController Configuration
the missing line was under line 84 which need to include the oc command to edit the forklift-controller
@dgur F.Y.Y
the last section  number 9.11.4
is different from what we agreed on the main aio buffer document
do we want to change this section or to stay with the current one ?

I am ok with what Andy wrote, it is clear that there are required versions,
and we tested only with some. Others they need to test
On Sun, 15 Dec 2024, 10:48 Tzahi Ashkenazi, @.***>
wrote:

@anarnold97 https://github.com/anarnold97
i have added missing command for section :
Editing ForkliftController Configuration
the missing line was under line 84 which need to include the oc command to
edit the forklift-controller
@dgur https://github.com/dgur F.Y.Y
the last section number 9.11.4
is different from what we agreed on the main aio buffer document
do we want to change this section or to stay with the current one ?
—
Reply to this email directly, view it on GitHub
https://github.com/kubev2v/forklift-documentation/pull/598#issuecomment-2543562111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHGJQIUPUZRYOGGQSWP6BS32FU66TAVCNFSM6AAAAABTBHQAN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGU3DEMJRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@dgur - clearly this is not going to be approved. After numerous reviews i am closing it and not continuing.

Looks good for me, with the last fix from today, Thank you Andy!
Can you also generate  the updated preview, and publush?


file throws warning when downloading file over https in master-less setup

Describe the Bug
When downloading external resources via file in a master-less environment, puppet throws warnings as it is missing agent ssl files.
root@node:~  # /opt/puppetlabs/bin/puppet apply init.pp
Notice: Compiled catalog for node in environment production in 0.01 seconds
Warning: Private key for 'node' does not exist
Warning: Client certificate for 'node' does not exist
Notice: /Stage[main]/Main/File[/tmp/file]/ensure: defined content as '{mtime}2023-09-25 06:36:57 UTC'
Notice: Applied catalog in 0.10 seconds

Expected Behavior
No warning being thrown when downloading external resources over https.
root@node:~  # /opt/puppetlabs/bin/puppet apply init.pp
Notice: Compiled catalog for node in environment production in 0.01 seconds
Notice: /Stage[main]/Main/File[/tmp/file]/ensure: defined content as '{mtime}2023-09-25 06:36:57 UTC'
Notice: Applied catalog in 0.10 seconds

Steps to Reproduce

Install clean Debian 12
wget https://apt.puppetlabs.com/puppet8-release-bookworm.deb
dpkg -i puppet8-release-bookworm.deb
apt-get update
apt-get install puppet-agent
Create file init.pp with following content:

file { '/tmp/file':
  source => 'https://artifacts.elastic.co/GPG-KEY-elasticsearch',
}

/opt/puppetlabs/bin/puppet apply init.pp

Environment

Version: 8.6.0-1bookworm
Platform Debian 12.5

Additional Context
We found this issue when we tried to download apt key (via apt::keyring) over https from external source. There is no issue when downloading over insecure http.

This is a duplicate of https://github.com/puppetlabs/puppet/issues/9248, closing.


:honeybee: catalog: typing

@lucasrodes I spent some time trying to make it work, but with no success (even the simplest overriding of __getitem__ didn't work). I'd close this and use .loc[] rather than square brackets. It's possible that pandas might fix that in their core code in the future.

hey @Marigold, I think that makes sense for the [] issue. We could still add overloads for the other methods though.

I think we could still add overloads for the other methods though.

For sure. The question is whether to add them all at once or do it on the fly (which is what I'm doing). Some methods with a lot of arguments could end up having tons of @overload decorators.

Alright, yeah, will be a long run.
Should we then try to push these changes or discard them for the time being?

Sure, why not.

@Marigold still need to fix the tests not passing CI/CD. I'm looking at it.


Consider making as_option public

It would be nice to use in cases with pattern matching such as if let Some(val) = val.as_option() {...}. I think is flows better than if let Some(val) = Option::from(val) {...}.
However, the latter has the advantage that it will still work if I all of a sudden decide to drop in Option again instead of Optioned.
What do you think? I'd be happy to submit a PR if you think it's OK for the API.

Yeah, you're right on both thanks. I'll close this and just use into.

Thank you for bringing this up anyway, this may be a blind spot in the docs.

After refactoring quite a bit of code I have that processes arrays/Vecs of Optioned, I think this is worth re-opening. There were a lot of places where the compiler could not infer the types for some_val.into() and I had to type a lot of Into::<Option<f64>>::into(some_val).
Making as_option into into_option and making it public would read better, be shorter, and be just as clear I think.

Does .map(Option::from) work for you?

Nope. That just wraps the interior f64 in an Option and returns an Option<Option<f64>>.
Along that line of reasoning I did find this, which is weird to look at, but it works: .map(|inner| inner). That seems surprising at first glance and the coder's intentions aren't as clear as an .into_option() method would be.
What I'm really trying to do is convert between types Optioned and Option, which is where From and Into are idiomatic. But due to the limitations of type inference and generics the Into trait works inconsistently. The workarounds (so far) are either verbose or make the intention unclear.

Ah, I see. For those cases .into_option would indeed make sense. I'll gladly accept a PR


User can set specific schedule requests

See class UserShfitRequest in model.py

This is no longer relevant given the changes to the model


mpartner-default-mobile partner details is missing in kernel default properties

Describe the bug
auth.server.admin.allowed.audience should contian mpartner-default-mobile but it is missing
To Reproduce
1.Open [kernel.def](https://github.com/mosip/mosip-config/blob/qa-121/kernel-default.properties)
2. search for auth.server.admin.allowed.audience
Observed behavior
auth.server.admin.allowed.audience missing mpartner-default-mobile
Expected behavior
auth.server.admin.allowed.audience token should contain mpartner-default-mobile partner details
Screenshots

Smartphone (please complete the following information):

Device: android
MOSIP Version: 121
MOSIP server: qa-121.mosip.net

Additional context
As an impact credentials are stuck at downloading state in the app

verified from api, credential moved to printing state, and also VC is verified in https://verifiablecredential.io/playground

VC_qa-121.txt
ser-images.githubusercontent.com/102220709/192960254-efaf43ae-6d71-4dc4-85ec-d3d6b21a6cd8.png)

verified this issue in QA-121 env, issue is not observed any more, hense closing the bug


Prevent looping

i dont want my first frame to be linked with the last one.
If i put a constraint with head and tail prompt, in the region area, it tends just to not animate and put everything at once.
The only way that i found is to use less than 16 frames, which is pretty low.
So i need a way for the animation to not loop and to have more than 16 frames
and without controlnet

I asked the same question when it was first released.
Apparntly possible, but it would make things more complicated.
https://github.com/s9roll7/animatediff-cli-prompt-travel/issues/97


Queryset level ops

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           39        40    +1
  Lines         2533      2604   +71
=========================================
+ Hits          2533      2604   +71

Impacted Files
Coverage Δ

ormar/__init__.py
100.00% <100.00%> (ø)

ormar/models/modelproxy.py
100.00% <100.00%> (ø)

ormar/queryset/queryset.py
100.00% <100.00%> (ø)

tests/test_many_to_many.py
100.00% <100.00%> (ø)

tests/test_queryset_level_methods.py
100.00% <100.00%> (ø)


[FormRecognizer] Fix package version in build:autorest script

The generated code already have the correct version. This change might have been missed in previous commit.

/azp run js - formrecognizer - tests

Going to close this one as the other PR already contains this change.


compilation error:

I am getting following error when I use:
r2pm -i retdec-r2plugin
I have latest version of radare
radare2 5.0.1-git 25619 @ linux-x86-64 git.5.0.0
commit: ae9ffe4a833915efac4622d24d2dc3dfd4bed427 build: 2021-01-18__11:26:36

[ 40%] Built target main
[ 74%] Built target llvmir2hll
[ 86%] Built target fileinfo
[ 99%] Built target bin2llvmir
[ 99%] Built target retdec
[ 99%] Built target retdec-decompiler
[ 99%] Building CXX object src/r2plugin/CMakeFiles/core_retdec.dir/r2data.cpp.o
[100%] Building CXX object src/r2plugin/CMakeFiles/core_retdec.dir/registration.cpp.o
/Ruturaj/radare/confighome/git/retdec-r2plugin/src/r2plugin/r2data.cpp: In member function ‘std::string retdec::r2plugin::R2Database::fetchFilePath() const’:
/Ruturaj/radare/confighome/git/retdec-r2plugin/src/r2plugin/r2data.cpp:53:17: error: ‘RCore’ {aka
struct r_core_t’} has no member named ‘file’
   53 |  return _r2core.file->binb.bin->file;
      |                 ^~~~
gmake[2]: *** [src/r2plugin/CMakeFiles/core_retdec.dir/build.make:76: src/r2plugin/CMakeFiles/core_retdec.dir/r2data.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:2823: src/r2plugin/CMakeFiles/core_retdec.dir/all] Error 2
gmake: *** [Makefile:130: all] Error 2

Hi, did you manage to solve this issue?

I am experiencing the same error.

replace return _r2core.file->binb.bin->file; in line 53 at r2data.cpp to
	int fd = r_io_fd_get_current (_r2core.io);
	if (fd != -1) {
		RIODesc *d = r_io_desc_get (_r2core.io, fd);
		if (d) {
			return d->name;
		}
	}
	return "";

thats fixed in master, it was fixed 2 years ago in a pr that wasnt merged because of problems in the retdec side. please try again


incorrect constructN REPL

fixed incorrect wrapping parens causing the last parameter to not get called in R.map

Hi @jdkschang , thanks for the PR! Could you update it to keep the same indentation as before?

@kedashoe you got it!

Is it too far indented now? Or are tabs being inserted? Not trying to be difficult, I just can't really tell what is actually changed vs all the changes in the diff because of the spacing. Can you play with the spacing a little more and run git diff locally until just the lines you want changed are showing up in the diff?

Sorry for any inconsistent pull requests. I am new to this, but hopefully the last couple of commits will reflect the fix more explicitly than my first couple tries.
Cheers

Sorry for any inconsistent pull requests. I am new to this, but hopefully the last couple of commits will reflect the fix more explicitly than my first couple tries.
Cheers

Not a problem @jdkschang , thanks for sticking with it :) It's looking good, could you please squash the commits?

Hmm. As of c8d563fed7c6168d2028e0ed48fc267770670208 the only change I see is the addition of an unnecessary semicolon.

Hmm. As of c8d563f the only change I see is the addition of an unnecessary semicolon.

huh ya seems like github is getting confused? Sometimes the "Files Changed" tab shows the full diff and sometimes just that one semicolon. When I click "View" on the file the changes are there. And pulling it down locally looks fine..

@davidchambers The changes are actually in reference to the tag version v0.24.1 where the example has the R.map incorrectly wrapped as shown here.
I'm guessing the problem stems from my inexperience in initiating a pull request without following the proper channels (?). I've referred to the CONTRIBUTING.md but have little to no knowledge of the protocols regarding pull requests for a specific tag version: v0.24.1.
@kedashoe commits have been squashed

As far as I can tell it's just github. @davidchambers ?

$ git --no-pager diff master..jdkschang/constructN
diff --git a/src/constructN.js b/src/constructN.js
index 606a240..df5c820 100644
--- a/src/constructN.js
+++ b/src/constructN.js
@@ -22,7 +22,6 @@ var nAry = require('./nAry');
  *      function Salad() {
  *        this.ingredients = arguments;
  *      }
- *
  *      Salad.prototype.recipe = function() {
  *        var instructions = R.map(ingredient => 'Add a dollop of ' + ingredient, this.ingredients);
  *        return R.join('\n', instructions);

Looks like this was already done here: https://github.com/ramda/ramda/pull/2216
So maybe you had branched from before that was merged, @jdkschang ?

Must be. Sorry for all the trouble I may have caused @kedashoe @davidchambers 😅. I was under the incorrect assumption that the ramda docs were representative of the current master branch.


Removed Redudancy

Searching youtube with a song's link will give the same song as first option. We can use this to avoid writing a separate function for link support.

Why I can't merge your pull request now: -

You should test you code because it's not working when I am requesting a link.
Searching by links doesn't gives the item at first place.
I tried by searching this link -->  https://www.youtube.com/watch?v=Cn8tZ6SLGSc
https://www.youtube.com/watch?v=2tJyAXiLXFk
You should go and search for these links and also watch it by directly pasting it to the browser so that you can see the difference. :smiley:

Fixed that. As an added functionality one can also download from a list having a combination of links and song names.

Hey bro. It's not working when I am giving the url of the youtube link.
Please fix it so that I can commit your changes.

On Thu, Jul 21, 2016 at 1:59 PM, Tushar Tripathi<EMAIL_ADDRESS>wrote:

Fixed that. As an added functionality one can also download from a list
having a combination of links and song names.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aman-roy/mp3_downloader/pull/4#issuecomment-234189977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMs3Daq-9w0DDvy_4UDrq3oQKUSJ4vEEks5qXy4EgaJpZM4JRQD7
.

--
[image: --]
Aman Roy
[image: https://]My website
http://amanroy.me


Different distance results for the same code on different machines

Hello,
I run the following code on different machines (one just used CPU, one used GPU), but I got different results. For CPU, I got:
image1 | image2| cosine distance
001_002_0.5 | 001 | 0.471843
001_002_0.5 | 002 | 0.434614
001_003_0.5 | 001 | 0.565312
001_003_0.5 | 003 | 0.763006
001_004_0.5 | 001 | 0.666956
While for GPU one:
image1 | image2| cosine distance
001_002_0.5 | 001 | 0.01673
001_002_0.5 | 002 | 0.01159
001_003_0.5 | 001 | 0.03172
001_003_0.5 | 003 | 0.03836
001_004_0.5 | 001 | 0.02838
In my experience, the results from CPU one are reasonable, while GPU one is not. Do you have any idea what's wrong with it?
The deepface version I used is 0.0.51. The CPU on is i7-6800K, the GPU one is Xeon W-2175 with RTX 2080Ti. Plus, I try to use GPU on i7-6800K machine (1080Ti) with the code:
import os
os.environ["CUDA_VISIBLE_DEVICES"]="0"

But it didn't seem to work since GPU memory is not occupied.
The full code is as follows, where pairs_alpha05 is the path of my images, and path1 and path 2 is image 2 and image 1 shown above. Here is image 1 and 2.

model_name = "ArcFace"
model = DeepFace.build_model(model_name)

pairs_distance_lists = []

for j in range(len(pairs_name_alpha05)):

    print('{now}/{total}'.format(now = j, total = len(pairs_name_alpha05)))

    path1 = pairs_alpha05[j][0] # the path of frist image like 001_03_q30.jpg
    path2 = pairs_alpha05[j][1] # the path of second image like 001_03_R_morphed_0.5_002_03_R.png

    result = DeepFace.verify(path1, path2, model_name=model_name, model=model,\
                              distance_metric = "cosine", detector_backend = 'dlib')

    # extract distance from result
    a1 = pairs_name_alpha05[j][0]
    a2 = pairs_name_alpha05[j][1]
    a3 = result['distance']
    pairs_distance = [a1, a2, a3]
#     print(pairs_distance)
    pairs_distance_lists.append(pairs_distance)

    if (j+1)%1000 == 0:
        json_name = 'ArcFace_cosine_AMSL_distance_lists_sych_alpha05_' + str(j+1) + '.json'
        with open(json_name, 'w') as json_file3:
            json.dump(pairs_distance_lists, json_file3)
        print(json_name + ' saved!')

No idea :/ It might be a tensorflow related issue.

I think it could be the problem of tensorflow-gpu. After I disabled the gpu used in the code, the results in both computers are the same. For the environment requirement of deepface, it should not use  tensorflow-gpu, right?

it should not matter. it can be run on cpu or gpu.

I am closing this issue if it is oke, because it seems that it is independent from deepface.


(fix)[load] add thread pool for sinker

Proposed changes
Issue Number: close #18651
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.92 seconds
stream load tsv:          420 seconds loaded<PHONE_NUMBER>9 Bytes, about 169 MB/s
stream load json:         23 seconds loaded<PHONE_NUMBER> Bytes, about 97 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          29 seconds loaded 861443392 Bytes, about 28 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230414115134_clickbench_pr_129185.html

run buildall

run p0


[STORM-1208] Guard against NPE, and avoid using NaN values

Removes malformed preconditions that don't do anything, and if they did work would throw Errors that would crash nimbus.
When reading and writing average-type metrics, use 0 instead of NaN
Avoid NPE accessing some maps by doing (get m k) instead of (m k).

LGTM. +1

+1 NB

Nice fix. +1

it looks good

@hustfxj , @kishorvpatil , @zhuoliu , @redsanket,
Added checking for Infinity plus some refactoring as per @hustfxj's review.

still +1.

+1 LGTM


Subject 1-5 Data Download

Please close this issue once all your data has been downloaded.

Should we still download all the data even though we are currently planning to just look at one subject?

Let's close this for now, since we don't need to download the rest of the data for now


Translate bool attributes to BOOL when it is promoted to input

Currently bool is treated as int and will be promoted to INT64 when it is used as an input. For a Pythonic experience and to reduce errors it would be good to promote it to BOOL.
@gramalingam

Right. I think you may even have an earlier issue about this. I thought a bit about it. It is doable directly when promoting an attribute-reference of bool type to a value.
Another point to consider is another feature (which is currently missing in auto-cast): inserting casts on constants when the target type is known: e.g., in the call SomeOp(1, ...) where the first argument is known to be of "float type". We could insert a cast to float for the constant 1 here. Such a feature would also solve the above issue in almost all normal use-cases. In addition, it would also handle the case where an int attribute is used as an argument where a bool tensor is expected.
But if we do both, we may end up producing some redundant casts in cases like this. I guess that's one reason I didn't complete that right away. But maybe we should do this feature (bool attribute promotion) first, anyway ... any thoughts?

I think having the BOOL promotion first is going to be very useful, because it is going to eliminate a class of errors otherwise detected only when running the model.
For the other autocast feature I wonder how many ops there are where the type requirement is definitive, ie a single type like float? If the op takes multiple different types, do we need to look at other inputs for the analysis? Should we cast int to float16/float32/float64 for example? Should we do it for python constants, or onnx attributes / onnx constants as well? (I still think it should be a good idea, just typing out what I was wondering when exploring this!)
At the same time, I wonder if it is possible to do additional analysis to remove consecutive casts?

If the op takes multiple different types, do we need to look at other inputs for the analysis?

Yes. This is done already, and introduces a CastLike, so something like X+1 will be translated correctly for all types of X.

I wonder what happens for (const1 + const2) + dynamic_x. Will there be a castlike for each add? When the dtypes are different for each value, would that mean + is not commutative? Is that an issue?

Good question. I think nothing happens in this case, for these reasons. For now, the workaround is that the user does this themselves (eg., they can compute const1 + const2 outside the script (using python semantics, and doing whatever they wish) and use only the result inside the script.) So, it is a problem to be solved (in the converter).


Version conflict

Error:Execution failed for task ':app:processHousiDebugManifest'.

Manifest merger failed : Attribute meta-data#android.support.VERSION@value value=(25.4.0) from [com.android.support:design:25.3.1] AndroidManifest.xml:27:9-31
is also present at [com.android.support:preference-v14:26.1.0] AndroidManifest.xml:25:13-35 value=(26.1.0).
Suggestion: add 'tools:replace="android:value"' to  element at AndroidManifest.xml:23:9-25:38 to override.

The latest version of the library uses the latest version of the support libraries (26.1.0 at the time of writing this). You need to either upgrade your app to use 26.1.0 support libraries or use an older version of this library. You can see the available older versions here.


Support placeholder when content is empty

Since placeholder isn't supported currently, is there any workaround?

any updated?

+1

I have the same question

When are we getting this feature?

I will also be happy to have it (if there's anything I can do to help LMK)

Any updates folks?

This could be easily workaround by faking a placeholder, not saying that it's optimal but definitely a legit solution
e.g. my implementation in an Angular  app:
<div *ngIf="!editorState.text" class="placeholder" (click)="focusOnEmptyText()">
  <div class="placeholder-text"> {{placeholder}} </div>
</div>

  @keyframes fadeIn {
    from { opacity: 0; }
    to   { opacity: 1; }
  }

  .placeholder {
    position: absolute;
    font-family: Lato;
    font-style: italic;
    color: var(--input-border-color);
    top: 8px;
    left: 34px;
    animation: fadeIn 500ms;
    animation-timing-function: ease-out;
  }

Here is my solution: add another element to simulate placeholder
https://codesandbox.io/s/monaco-editor-placeholder-vue-1lurc?file=/src/components/MonacoPlaceholder.vue

It seems that vscode already supports this feature, any ideas?

I found the source code for this in VS Code: https://github.com/microsoft/vscode/blob/167b197e767beb791e3d191c24efb97865a98b42/src/vs/workbench/browser/parts/editor/untitledHint.ts
It looks like it is overlaying an element as other have suggested here.

Any update on this ?

你好！我是翁碧琴，您的邮件我已收到，谢谢~

Duplicate of #568.


Tammy wants a module to create groups and RBAC rules in the cluster to manage user access

What's the difference between this and #141?


please add bookmark and refresh shortcut

example: long press some button of tool bar can do this tow actions(open bookmark s list, refresh page).

This is an old problem, over time I made the extension floatly which brings some common actions in the form of a shortcut, including reloading the page and bookmarking.


Deprecated WebExtension proxy APIs

This is an epic to track three issues related to these deprecated APIs. The first involves marking the API as deprecated, the second adds the deprecation notice to Firefox 68 release notes, and the final one to add the removal of the API to the Firefox 71.

Handing this over as it goes with the issue I gave you for adding a notice on this to the release notes.


Looks like the carrot timezone is off. Times are appearing as 10/11/2017 05:44 instead of 17:44 as per the log

Cannot repro this -looks like an issue on the box


Some systems failed to build for Quicklisp dist

Building with SBCL <IP_ADDRESS>-55d27b14b / ASDF 3.3.5 for quicklisp dist creation.
Trying to build commit id d378e9faa1e69cccd4a2bd03adcd22cc138ac918
data-frame fails to build with the following error:
Unhandled SB-INT:SIMPLE-FILE-ERROR in thread #<SB-THREAD:THREAD tid=3619261 "main thread" RUNNING {1001710003}>: Failed to find the TRUENAME of /home/quicklisp/quicklisp-controller/dist/build-cache/data-frame/a705d11eecd0ad126891bc05ffce80053df27a3f/data-frame-20230902-git/src/random-sample.lisp: No such file or directory

data-frame/tests fails to build with the following error:
Unhandled SB-INT:SIMPLE-FILE-ERROR in thread #<SB-THREAD:THREAD tid=3619257 "main thread" RUNNING {1001710003}>: Failed to find the TRUENAME of /home/quicklisp/quicklisp-controller/dist/build-cache/data-frame/a705d11eecd0ad126891bc05ffce80053df27a3f/data-frame-20230902-git/src/random-sample.lisp: No such file or directory

Full log here

Fixed


Yatin: Watch kevin[\W_]+harvick[\W_]+cbd[\W_]+oil

Yatin requests the watch of the watch_keyword kevin[\W_]+harvick[\W_]+cbd[\W_]+oil. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

kevin[\W_]+harvick[\W_]+cbd[\W_]+oil has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by cigien in Charcoal HQ. Reason: 'There's a standard template for this that catches more of the text'


spelling mistake

pkg_config::Config::statik should be pkg_config::Config::static
...unless this is deliberate, for some reason?

Yeah unfortunately static is a keyword in Rust, so I had to pick the next best thing :)

Ah, I didn’t know being a keyword prevented it from being a function name. Fair enough. :)

On 11 May 2016, at 17:09, Alex Crichton<EMAIL_ADDRESS>wrote:
Yeah unfortunately static is a keyword in Rust, so I had to pick the next best thing :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/alexcrichton/pkg-config-rs/issues/25#issuecomment-218508135


Add support for Cockroach Serverless Cluster identifier and fix dropUnique from migrations

Previously we could add the cluster identifier as part of the database name. Laravel would work out of the box, but as of today (2022-09-22), it looks like they (Cockroach Labs) changed something on their side and made the artisan migrate command fails, as it was querying the migrations table in the database with the cluster identifier in its name.
So, I changed the Database Connector to add the missing options parameter to the DSN when the cluster parameter is present in the config.
This PR also fixes the dropUnique from the migrations, as CockroachDB doesn't support an alter table drop for a unique index ( See https://github.com/cockroachdb/cockroach/issues/42840?version=v22.1 )

@tryvin can you please split up this PR. I appreciate it feels easier to just add in additional issues but as a maintainer it makes my life easier to have seperate PRs and evaluate each as their own feature/fix

@peterfox I recreated the PRs. I'm closing this one.


Supplier "about you" data extract

For this story: https://www.pivotaltracker.com/story/show/121428561
See the story for details of the data extracted by this new script.  Output has been OK'd by Andrew on Preview and Staging, and Prod data passed to Cath, who likes the look of it.
The script takes over 2 minutes to run against production data, which means that if it was built in to the admin app (as in the story) then the request would time out.  Agreed with Cath and @allait that we won't embed in admin app right now, but instead create a Jenkins job to wrap this script so new versions of the data can be easily generated when required by CCS.

👍


Upload frigate-hass-card artifact on pull request

This should help to make it easier to test new versions without having to cloning the repository and building it by hand.
With this, one can take the zip artifact from GitHub Actions and extract the .js to the Home Assistant.

Thanks @felipecrs , great idea!


command to swap visible files between panes per issue #13061

per issue #13061 , I added a function to swap visible files between panes, keybinding to Ctrl + Alt + F , and a preference with a disabled default state.
The function itself is currently located within MainViewManager.js with the command and preference manager registration located in ViewCommandhandlers.js as it seemed the logical place.  There are three tests  covering the basic functionality, the case where the file to move already exists in the destination pane, and where only one pane contains open/visible files.  In the latter case I just call moveView.
If its felt this should not exist in the core, but in an extension I don't think that will be much of an issue. I'm interested in any Comments and/or Remarks.

Looks pretty good to me, I should be able to test tomorrow.

I like the code, I like the feature, works quite well, but two things:

Why do I have to enable the feature in preferences?
This breaks https://github.com/adobe/brackets/pull/12853 in a way that if I use ctrl+alt+f to switch panes, alt+w stops to work until I mouse-click to the second pane, can you have a look why is that?

@zaggino having it as a default disabled preference was a mention in the original issue, and wouldn't take much to implement/remove so I went in that direction.  As such there isn't any necessary reason for it to be so.  I'll look into why alt+w stops working, and while I'm at it also change the keybindings to something+w because I agree it makes more sense.


Implementation of the ranges (with delimiters)

Reincarnation of the https://github.com/pegjs/pegjs/pull/209.
Previous attempt, closed by technical reasons: #208
This is an implementation of the ranges proposal with batteries, i.e.:

ability to use numeric constants to specify minimum, maximum or exact repetition count:more2 = "a"|2..|;
upTo3 = "a"|..3|;

ability to use preceding label as a range boundary:list = count:n5 @n5|count|;
n5 = n:[0-9]|5| { return parseInt(n); };

ability to use function as a boundary:list = "a"|{ return options.listSize; }|;

The syntax chosen saves the < and > characters for the template definitions where their are more natural.
[ and ] already used for character class definitions and
( and ) already used for grouping.

What if there was a way to define parser-functions, and it would be just repeat(x, 2..) and repeat(x, ..3)?
I think by creating a new syntax for every feature it will resemble Perl and its regular expressions quite soon (arguably, it already resembles it way too much).
The thing not mentioned in PR description, but probably the most awaited by me, is delimited repetition (a|.., b|).

Is there a CI version of this PR to play with it?
What AST does it generate?
I remember there was a discussion in PEG.js issues to add it as a % b operator. Does this PR subsume that feature request?
I didn't find tests attempting to break parsing of || brackets. Can we be sure that some .. | .. | | .. , .. | .. , .. | | will get parsed properly?

You can checkout this branch, build minified version of a parsernpm run build
# or only
npm run rollup
npm run terser
npm run deploy

and go to docs/online.html:
You can use a recently added peggy --ast option to look at it
I personally against any syntax which does not clearly indicate where the repeat expression ends. Using a % b falls into this category. Last time I tried to summarize possible options here
Yes, but you should take into account, that you cannot put two suffix operators one after another, you must wrap the first in parentheses:// This is all forbidden
start1 = .. | .. | | .. , .. | .. , .. | |;
start2 = .**;
start3 = .++;
start4 = .??;

// This will work
start5 = .(. | .. |) | .. , .. | .. , .. | |;
start6 = (.*)*;
start7 = (.+)+;
start8 = (.?)?;

What if there was a way to define parser-functions

I think we will soon need to determine what our long-term syntax extensibility approach is, before we run out of adequate syntax to act as the extensibility point.  I don't think repeat(n) will work, since that looks like a rule named repeat followed by a group.  <repeat min="a" max="b">...</repeat> would work, but that's pretty ugly, and even I don't like XML that much.

@Mingun do you have time to work on this?  If not, I'll try to rebase it at least.

I'll look at weekend

Ok, the bug seems to be fixed, I'll check it once again on this week and add a changelog entry for plugins' authors.

The diff between the 7-month old version and the current:

added forgotten tests for function-based boundaries
fixes some minor errors in tests for variable-based boundaries
added a notice for plugins' authors
fix some minor misprints in tests

The last thing remains is to add tests for source-mapping functions from function-based boundaries

This is fantastic work.  Thank you very much.
I'm going to merge  as-is, then when I do a full review of the documentation page, there may be a few nits I clean up while fixing other things.


Whitespace changed causes recreation of the ruleset rules

Confirmation

[X] This is a bug with an existing resource and is not a feature request or enhancement. Feature requests should be submitted with Cloudflare Support or your account team.
[X] I have searched the issue tracker and my issue isn't already found.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.6.6
on darwin_arm64
+ provider registry.terraform.io/cloudflare/cloudflare v4.48.0

Affected resource(s)

cloudflare_ruleset

Terraform configuration files
# From cf-terraforming generate --resource-type cloudflare_ruleset
resource "cloudflare_ruleset" "ratelimit_ruleset" {
  kind    = "zone"
  name    = "default"
  phase   = "http_ratelimit"
  zone_id = local.zone_id
  rules {
    action = "block"
    action_parameters {
      response {
        content      = "{\"errors\":[{\"code\":\"too_many_requests\",\"message\":\"Too many requests. Please try again in a bit.\"}]}\"
        content_type = "application/json"
        status_code  = 429
      }
    }
    description = "Containing /prepare_verification (3req/10s)"
    enabled     = true
    expression  = "(http.request.uri.path contains \"/something\" and http.request.method in {\"POST\" \"PUT\" \"PATCH\"})"
    ratelimit {
      characteristics     = ["ip.src", "cf.colo.id"]
      mitigation_timeout  = 600
      period              = 10
      requests_per_period = 3
    }
  }
}

# After importing the resource and making the config more readable
locals {
  error_message = jsonencode({
    errors = [{
      message = "Too many requests. Please try again in a bit.",
      code    = "too_many_requests"
    }]
  })
}

resource "cloudflare_ruleset" "ratelimit_ruleset" {
  kind    = "zone"
  name    = "default"
  phase   = "http_ratelimit"
  zone_id = local.zone_id
  rules {
    action = "block"
    action_parameters {
      response {
        content      = local.error_message
        content_type = "application/json"
        status_code  = 429
      }
    }
    description = "Containing /prepare_verification (3req/10s)"
    enabled     = true
    expression  = "(http.request.uri.path contains \"/something\" and http.request.method in {\"POST\" \"PUT\" \"PATCH\"})"
    ratelimit {
      characteristics     = ["ip.src", "cf.colo.id"]
      mitigation_timeout  = 600
      period              = 10
      requests_per_period = 3
    }
  }
}

Link to debug output
https://gist.github.com/Dragotic/90193d616755c15356f079fbe3431fcf
Panic output
No response
Expected output
No changes. Your infrastructure matches the configuration.
Actual output
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # cloudflare_ruleset.ratelimit_ruleset will be updated in-place
  ~ resource "cloudflare_ruleset" "ratelimit_ruleset" {
        id      = "................00e"
        name    = "default"
        # (3 unchanged attributes hidden)

      ~ rules {
          ~ id          = "........4fb" -> (known after apply)
          ~ ref         = ".......4fb" -> (known after apply)
            # (4 unchanged attributes hidden)

          ~ action_parameters {
              ~ response {
                  ~ content      = jsonencode( # whitespace changes
                        {
                            errors = [
                                {
                                    code    = "too_many_requests"
                                    message = "Too many requests. Please try again in a bit."
                                },
                            ]
                        }
                    )
                    # (2 unchanged attributes hidden)
                }
            }

            # (1 unchanged block hidden)
        }
}
Plan: 0 to add, 1 to change, 0 to destroy.

After typing yes on the terraform apply the rules are recreated with new IDs
Steps to reproduce

import cloudflare_ruleset by using `cf-terraforming generate --email $CLOUDFLARE_EMAIL --token $CLOUDFLARE_API_TOKEN --resource-type cloudflare_ruleset --zone $CLOUDFLARE_ZONE_ID
Change the response message to a terraform map with jsonencode() function before passing it to the response of the cloudflare_ruleset resource
terraform apply
Changes are shown
apply the changes
rules are recreated

Additional factoids
It's not only that whitespace is recognised as change by the cloudflare API, but it also recreates the rules when applied. Effectively losing activity for last 24 hours
References
No response

jsonencode and the string there are not equivalent so the diff here makes sense. you can confirm the difference by expecting the state or running it through terraform console. if you don't want the diff, you should adjust the value or formatting accordingly.
if you define the ref field, the analytics UI keeps track of the changes. see the release notes for the latest updates and guidance on that.

@jacobbednarz Thanks for the quick reply. I didn't catch the ref fix on the v4.48 version, thanks for pointing it out.
Do the whitespace changes cause any other side-effect on Cloudflare API apart from the look?

the rulesets service doesn't care (and is not the issue here). it's just terraform seeing there is a drift between the two. once you address that, you'll see this drift go away.


java -jar issue: constructing graphs without -f parameter leads to an error

small issue: the help says TTL is the default but without -f TTL I get this exception:
Exception in thread "main" org.apache.jena.riot.RiotException: No graph writer for 'Lang:RDF/XML'

Hi @seebi,
Thanks for reporting this issue, it has been fixed in version 0.0.3.
Luigi


Add owner_id to the external-dns documentation example

Add owner_id to the external-dns documentation example
Discovered a missing example field when doing release testing. As this is required makes the example non functional.
How to use
Read the documentation
Testing done
used in a release test

Let's see if I can get the CI to be happy :)

I use the cluster name for this field.
owner_id = var.cluster_name

I agree with @surajssd
Rather than hard coding an example value lokomotive we should use a variable.
The value of cluster_name seems to be a good choice, however what if two clusters are created with the same cluster name ?


Enable the new model serialization for signair

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Updating eset_sumexp.Rmd to modern syntax.

This PR addresses syntax changes affecting the SummarizedExperiment class in eset_sumexp.Rmd. rowRanges replaces rowData and metadata replaces exptData. Comments are also added to help students follow along.
Also modified the .gitignore to cover .Rproj.user files.
Thanks!

On Fri, Oct 25, 2019 at 6:33 PM gillsignals<EMAIL_ADDRESS>wrote:

@gillsignals commented on this pull request.
In bioc/eset_sumexp.Rmd
https://github.com/genomicsclass/labs/pull/84#discussion_r339260718:

ExpressionSet
-dim(e)
+# exprs gives expression matrix
+dim(e)    # number of features and samples in ExpressionSet
exprs(e)[1:3,1:3]

Are you referring to line 27, which was unchanged in my previous commit? I
am happy to change that if you want, but that is the format in the textbook
and how it is shown in the course. Having said that, there are many places
in the Rmds where this would need to be changed if we choose to do that.

Would you like that done?

I would say this is a low priority.  If it seems useful to query students
on appreciation of the differences
then let's use the concept in a problem.  Otherwise ignore the comment.  I
will merge the PR.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/genomicsclass/labs/pull/84?email_source=notifications&email_token=ABDI5QVSKCF7SK5SICMLD2DQQNX3LA5CNFSM4JFEXQQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCJJ4NIY#discussion_r339260718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDI5QXX7AWFBOKYFQYCKZLQQNX3LANCNFSM4JFEXQQA
.

--
The information in this e-mail is intended only for the person to whom it
is
addressed. If you believe this e-mail was sent to you in error and the
e-mail
contains patient information, please contact the Partners Compliance
HelpLine at
http://www.partners.org/complianceline
http://www.partners.org/complianceline . If the e-mail was sent to you in
error
but does not contain patient information, please contact the sender
and properly
dispose of the e-mail.


Severe performance degradation when performing actions in Metadata Menu

Description
Over the past couple of days, I've noticed that whenever dealing with MetadataMenu, that it seems to cause a significant amount of hanging and slowness in Obsidian.
This seems most notable when opening FileClass fields view, adding FileClass fields, and modifying FileClass Fields.
Here is a video showing significant slowness in the UI becoming responsive after modifying a field name.
https://youtu.be/LbsrssNoiiQ
Whenever an action is performed, the issue with performance is triggered and  'violations' begin appearing in the console.
System Details
OS: Arch Linux x86_64
Host: Beastly-Beast_2.3
Kernel: 6.1.41-1-Arch
Uptime: 9 hours, 59 mins
Shell: zsh 5.9
WM: awesome
CPU: 13th Gen Intel i9-13900KS (32) @ 5.600GHz
GPU: Intel Raptor Lake-S GT1 [UHD Graphics 770]
GPU: NVIDIA GeForce RTX 4090
Memory: 6857MiB / 128628MiB
GPU Driver: Micro-Star International Co., Ltd. [MSI] Raptor Lake-S GT1 [UHD Graphics 770] [1462:7d25]
CPU Usage: 2%
Obsidian Details
Obsidian version 1.5.2
Issue Details
Violations begin to appear in the console whenever a field is edited or field editor is opened.
Other actions in Obsidian do not produce these violations.
[Violation] 'setTimeout' handler took <N>ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 4173ms
plugin:dataview:14861 [Violation] 'setTimeout' handler took 116ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 444ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 4022ms
plugin:dataview:14861 [Violation] 'setTimeout' handler took 105ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 411ms
plugin:image-captions:37 [Violation] 'message' handler took 4322ms
plugin:image-captions:37 [Violation] 'message' handler took 4237ms
[Violation] Forced reflow while executing JavaScript took 48ms

However, no consistent errors or exceptions occur.
This shows up more than any other:
Uncaught (in promise) TypeError: Cannot read properties of null (reading 'frontmatter')     at OnFileModifyAutomatorManager.onFileModify (plugin:metaedit:5331:22)
Here is a full trace from the moment a new field was added to the point where the UI became responsive again:
plugin:metaedit:5331 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'frontmatter')
    at OnFileModifyAutomatorManager.onFileModify (plugin:metaedit:5331:22)
7
[Violation] 'setTimeout' handler took <N>ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 5824ms
plugin:dataview:14861 [Violation] 'setTimeout' handler took 396ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 573ms
app.js:1 [Violation] 'setTimeout' handler took 70ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 58ms
app.js:1 [Violation] 'setTimeout' handler took 77ms
plugin:image-captions:37 [Violation] 'setTimeout' handler took 62ms
plugin:image-captions:37 [Violation] 'message' handler took 11413ms
app.js:1 Uncaught (in promise) Error: File system operation timed out.
    at e.kill (app.js:1:542802)
    at l (app.js:1:254714)
    at c (app.js:1:254830)
2
app.js:1 Uncaught (in promise) DOMException: Transaction timed out due to inactivity.
app.js:1 Sync Error!
app.js:1 TypeError: Cannot read properties of null (reading 'send')
    at e.send (app.js:1:2653122)
    at e.<anonymous> (app.js:1:2652313)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at app.js:1:236279
    at new Promise (<anonymous>)
    at v (app.js:1:236024)
    at e.request (app.js:1:2652182)
    at e.<anonymous> (app.js:1:2649153)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at a (app.js:1:236081)
app.js:1 Uncaught (in promise) Error: Disconnected
    at e.disconnect (app.js:1:2647328)
    at t.<anonymous> (app.js:1:2626880)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at app.js:1:236279
    at new Promise (<anonymous>)
    at v (app.js:1:236024)
    at t.asyncGetServer (app.js:1:2626711)
    at t.getServer (app.js:1:2626588)
    at t.<anonymous> (app.js:1:2628303)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at app.js:1:236279
    at new Promise (<anonymous>)
    at v (app.js:1:236024)
    at t._sync (app.js:1:2628044)
    at t.<anonymous> (app.js:1:2626044)
    at app.js:1:237258
    at Object.next (app.js:1:237363)
    at a (app.js:1:236081)

Hi.
Do you have formula or lookup fields? if so: try to toggle off "Auto-calculation" in the global settings and check if the performance improves.
If your fileclass view has many row per page x many fields, it can slow down the calculation of the table. Try to hide some columns and refuce the numer of row per page to 5 or something and see if performance improves.
Last: 0.6.8-beta and 0.6.9 weren't clearing objects properly → 0.6.10 does that better. Reload obsidian to flush the ram if you haven't done that already

Can you test with 0.6.11.
I've made several changes to avoid tableview being refreshed when it's not necessary → divides by 3 the time to open a tableview
The table won't refresh when the tab isn't displayed
The table won't refresh when there's a change in the fileclass. There's a button to rebuild the table after a fileclass change now.
I've alse default hidden the "insert field" buttons. There's a button to show them now→ reduces by 30% the building time of the building time of the table
Hopefully it will improve the experience for you

And wow, your hardware configuration is incredible!

Hi, I have the same problem but after your recommandations it's ok.
(row limit 5)
I confirm the slowdown with 0.6.11 version.
mais mon ordi n'est pas survitaminé ;-)

0.6.11 Does show an increase in performance but the greatest increase came from limiting the number of records returned for the Table View.
Once the Table view was limited to 5 everything works perfectly again.
Thank you so much for your assistance on this!
Happy Holidays

Hello, in fact not real improvements after downsizing the number of files in table view (juste one for me). My hardware configuration is very low tech... (I5-4570) perhaps a first limitation in my vault...(3500 files 1GB data)


SMP: Do not send phase 2 commands in CTKD over Classic

I just realized that mock methods are not released after the test, though there is no following tests right now. To solve this problem, we need to wrap all test cases into test class.

I just realized that mock methods are not released after the test, though there is no following tests right now. To solve this problem, we need to wrap all test cases into test class.

Oh not really. That can be protected by a with block.

Ready to review


style(all): add use strict to all src files

Hello,
I added use strict directive to all files. It is not a big deal. Can't not push myself to not fix it.
Best regards,
Aleh

Merged and released with v.3.3.4


Make Solver Communicate with Mempool

Given a solved set of intents, make solver have submit API. Given
appropriate intents structure, it is to be used to forward the intent
to the Mempool in an appropriate wrap. Currently this only supports
transparent transactions as intents.
Reformats the solver appropriately to suit this functionality.

Closes #1620


Unexpected token: apply

Describe the bug
There is a sql which i can run with mysql success, but if with jsqlparser, got an exception
To Reproduce
Steps to reproduce the behavior:

Example SQL

 SELECT  remark,apply FROM oms_data

Exception Log

2021-04-07 09:55:02.471  WARN 41488 --- [nio-8080-exec-1] c.b.m.e.p.i.PaginationInnerInterceptor   : optimize this sql to a count sql has exception, sql:"SELECT
remark,apply
  FROM oms_data", exception:
net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "apply" "APPLY"
    at line 2, column 8.

Was expecting one of:

    "!"
    "("
    "*"
    "+"
    "-"
    ":"
    "?"
    "@"
    "@@"
    "ACTION"
    "ANY"
    "BYTE"
    "CASCADE"
    "CASE"
    "CAST"
    "CHANGE"
    "CHAR"
    "CHARACTER"
    "COLUMN"
    "COLUMNS"
    "COMMENT"
    "COMMIT"
    "CREATE"
    "CURRENT"
    "CYCLE"
    "DESCRIBE"
    "DISABLE"
    "DO"
    "DOUBLE"
    "DUPLICATE"
    "ENABLE"
    "END"
    "EXTRACT"
    "FALSE"
    "FIRST"
    "FN"
    "FOLLOWING"
    "GROUP"
    "GROUP_CONCAT"
    "IF"
    "IIF"
    "INDEX"
    "INSERT"
    "INTERVAL"
    "ISNULL"
    "KEY"
    "LAST"
    "LEFT"
    "LIMIT"
    "MATCH"
    "MATERIALIZED"
    "NEXTVAL"
    "NO"
    "NOLOCK"
    "NOT"
    "NULL"
    "NULLS"
    "OFFSET"
    "ON"
    "OPEN"
    "OPTIMIZE"
    "ORDER"
    "OVER"
    "PARTITION"
    "PATH"
    "PERCENT"
    "PRECISION"
    "PRIMARY"
    "PRIOR"
    "PROCEDURE"
    "RANGE"
    "READ"
    "REPLACE"
    "RIGHT"
    "ROW"
    "ROWS"
    "SCHEMA"
    "SEPARATOR"
    "SESSION"
    "SET"
    "SIBLINGS"
    "SIZE"
    "START"
    "TABLE"
    "TEMP"
    "TEMPORARY"
    "TOP"
    "TRUE"
    "TRUNCATE"
    "TYPE"
    "UNSIGNED"
    "VALIDATE"
    "VALUE"
    "VALUES"
    "VIEW"
    "XML"
    "ZONE"
    "{"
    "{d"
    "{t"
    "{ts"
    "~"
    <K_DATETIMELITERAL>
    <K_DATE_LITERAL>
    <K_TIME_KEY_EXPR>
    <S_CHAR_LITERAL>
    <S_DOUBLE>
    <S_HEX>
    <S_IDENTIFIER>
    <S_LONG>
    <S_QUOTED_IDENTIFIER>

Expected behavior
A clear and concise description of what you expected to happen.
System

Database you are using: MySQL 5.6.16
Java Version: Oracle 1.8.0_251
JSqlParser version: 3.2

There is also has the exception when i update JSqlParser vertion to 4.0

apply is a keyword.  I checked to adapt the grammar to allow apply as a column name. Unfortunately this is not trivial.


[iOS] Swiping and Tapping Quickly Causes Unintended Swipe of Next Card

Bug Report: Card Swiping Issue on iOS
Issue Description:
On iOS, when swiping a card and quickly tapping the next card before the previous one is fully swiped away, the next card is unintentionally swiped. This seems to occur when interacting with the next card before the swiped card has completely left the screen.
Steps to Reproduce:

Swipe a card.
Before the swiped card is fully gone, quickly tap the next card.
Observe that the next card is unintentionally swiped.

Expected Behavior:

The next card should not be swiped when tapped immediately after the previous card is swiped.

Actual Behavior:

The next card gets swiped when tapped, even though a swipe gesture wasn't initiated.

Affected Platforms:

iOS

Environment:

react-native-deck-swiper version: ^2.0.17
Expo version: ~51.0.28
React Native version: 0.74.5
React version: 18.2.0
Device tested on: iPhone 15 (Simulator), iOS 17.4

Code Example:
import { StyleSheet, View, Button, Text } from 'react-native';
import Swiper from 'react-native-deck-swiper';
import React from 'react';

export default function HomeScreen() {
  return (
    <View style={styles.container}>
      <Swiper
        cards={['DO', 'MORE', 'OF', 'WHAT', 'MAKES', 'YOU', 'HAPPY']}
        renderCard={(card) => {
          return (
            <View style={styles.card}>
              <Text style={styles.text}>{card}</Text>
            </View>
          );
        }}
        onSwiped={(cardIndex) => console.log(cardIndex)}
        onSwipedAll={() => console.log('onSwipedAll')}
        cardIndex={0}
        backgroundColor={'#4FD0E9'}
        infinite={true}
        stackSize={3}
      >
        <Button
          onPress={() => console.log('oulala')}
          title="Press me"
        />
      </Swiper>
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: '#F5FCFF',
  },
  card: {
    flex: 1,
    borderRadius: 4,
    borderWidth: 2,
    borderColor: '#E8E8E8',
    justifyContent: 'center',
    backgroundColor: 'white',
  },
  text: {
    textAlign: 'center',
    fontSize: 50,
    backgroundColor: 'transparent',
  },
});

package.json
{
  "name": "test-swiper",
  "main": "expo-router/entry",
  "version": "1.0.0",
  "scripts": {
    "start": "expo start",
    "reset-project": "node ./scripts/reset-project.js",
    "android": "expo start --android",
    "ios": "expo start --ios",
    "web": "expo start --web",
    "test": "jest --watchAll",
    "lint": "expo lint"
  },
  "jest": {
    "preset": "jest-expo"
  },
  "dependencies": {
    "@expo/vector-icons": "^14.0.2",
    "@react-navigation/native": "^6.0.2",
    "expo": "~51.0.28",
    "expo-constants": "~16.0.2",
    "expo-font": "~12.0.9",
    "expo-linking": "~6.3.1",
    "expo-router": "~3.5.23",
    "expo-splash-screen": "~0.27.5",
    "expo-status-bar": "~1.12.1",
    "expo-system-ui": "~3.0.7",
    "expo-web-browser": "~13.0.3",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "react-native": "0.74.5",
    "react-native-deck-swiper": "^2.0.17",
    "react-native-gesture-handler": "~2.16.1",
    "react-native-reanimated": "~3.10.1",
    "react-native-safe-area-context": "4.10.5",
    "react-native-screens": "3.31.1",
    "react-native-web": "~0.19.10"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@types/jest": "^29.5.12",
    "@types/react": "~18.2.45",
    "@types/react-test-renderer": "^18.0.7",
    "jest": "^29.2.1",
    "jest-expo": "~51.0.3",
    "react-test-renderer": "18.2.0",
    "typescript": "~5.3.3"
  },
  "private": true
}

same here

same here


[Bug] [Module Name] Cannot run in Centos6.6's env

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I use command
tar -zxvf apache-seatunnel-xx-2.3.2.tar.gz in Centos6.6,I run a sample example,it goes wrong
the error image : NoclassDeffoundError: com/sun/jersey/client/impl/copyOnWriteHashMap

SeaTunnel Version
2.3.2
SeaTunnel Config
the origion config

Running Command
./bin/seatunnel.sh --config ./config/v2.batch.config.template -e local

Error Exception
the error image : NoclassDeffoundError: com/sun/jersey/client/impl/copyOnWriteHashMap

Flink or Spark Version
None
Java or Scala Version
1.8.0_171-b11
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

please help

add

lib  can resolve

maven  xml is
org.apache.seatunnel

                    <artifactId>seatunnel-hadoop3-3.1.4-uber</artifactId>
                    <version>${project.version}</version>
                 </dependency>

add  lib can resolve  maven is org.apache.seatunnel

                    <artifactId>seatunnel-hadoop3-3.1.4-uber</artifactId>
                    <version>${project.version}</version>

请问，我直接把这个包放入解压后的seatunnel的lib下面可以吗

可以的

add  lib can resolve  maven is org.apache.seatunnel

                    <artifactId>seatunnel-hadoop3-3.1.4-uber</artifactId>
                    <version>${project.version}</version>

请问，我直接把这个包放入解压后的seatunnel的lib下面可以吗

可以的

seatunnel-hadoop3-3.1.4-uber

非常感谢，祝你天天艳遇不断，天天发大财

problem solved！

how get seatunnel-hadoop3-3.1.4-uber-2.3.2-SNAPSHOT-2.11.12.jar file?


Second review corrections

Pull Request Test Coverage Report for Build 81

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 76:
0.0%

Covered Lines:
374

Relevant Lines:
374

💛  - Coveralls


Fail bundle install if a formula is installed but not linked

We have a situation where people install software outside of homebrew, and running brew bundle install will install the formulae but then fail creating the symlinks. Trying again though will succeed, although the symlinks are not enabled, thus hiding the problem.
At least my expectation is that brew bundle install is deterministic.
No problems: we installed AND linked n formulae.
Problems: we installed n but failed to link m formulae.
Right now it's not. Either there are no problems, or if there are, trying again and again and again will lead to a success message, with problems being ignored.

if i understand this correctly, this is less of a bundle problem and more of a brew + overwriting problem. what does the bundle file look like? what arguments are you passing in? what is the PATH setup on these systems?

Can you provide a minimal Brewfile and, ideally, brew commands that reproduce this behaviour?


application-title-bar: 0.6.8 -> 0.6.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for application-title-bar is: KDE Plasma6 widget with window controls
meta.homepage for application-title-bar is: https://github.com/antroids/application-title-bar
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.9 with grep in /nix/store/p64hga3847s99wlx50v5p25h09hixlgi-application-title-bar-0.6.9
found 0.6.9 in filename of file in /nix/store/p64hga3847s99wlx50v5p25h09hixlgi-application-title-bar-0.6.9

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

application-title-bar

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p64hga3847s99wlx50v5p25h09hixlgi-application-title-bar-0.6.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A application-title-bar https://github.com/r-ryantm/nixpkgs/archive/d2be09bc893776bb76257711ed3959eb1754ad06.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d2be09bc893776bb76257711ed3959eb1754ad06#application-title-bar

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p64hga3847s99wlx50v5p25h09hixlgi-application-title-bar-0.6.9
ls -la /nix/store/p64hga3847s99wlx50v5p25h09hixlgi-application-title-bar-0.6.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

application-title-bar

Maintainer pings
cc @HeitorAugustoLN for testing.

@NixOS/nixpkgs-merge-bot merge


FileSmoosher requested changes

Addresses https://github.com/druid-io/druid/issues/3640 and https://github.com/metamx/java-util/pull/55 requested changes.

@cheddar @drcrallen Thanks for reviewing this PR, uploaded a new patch set to address code review comments.

👍

👍


Fix some small errors

Forgot to delete one _release
Allow most of tests in test_wgpu_vertex_instance to run even if multi-draw-indirect isn't available.  This will be useful in the future as we add more features. Also simplified the code a little bit.
Fixed one test where I believe naga's behavior is incorrect.  I updated the test to accept either naga's current behavior or the behavior of the JS implementation.

I don't have sufficient privileges (write access) to do the merge.


Basic example: After reconnect messages aren't being received

I adapted the basic example to just listen for incoming messages, if the websocket server goes down, recws correctly retries to connect but then the messages are no longer received.
Expected behaviour, the messages continue to be received OK. You can see the messages received before disconnect but not afterwards.

Adapted example
package main

import (
	"context"
	"github.com/recws-org/recws"
	"log"
	"time"
)

func main() {
	ctx, cancel := context.WithCancel(context.Background())
	ws := recws.RecConn{
		KeepAliveTimeout: 10 * time.Second,
	}
	ws.Dial("wss://echo.websocket.org", nil)

	go func() {
		time.Sleep(2 * time.Second)
		cancel()
	}()

	for {
		select {
		default:
			if !ws.IsConnected() {
				log.Printf("Websocket disconnected %s", ws.GetURL())
				continue
			}

			_, message, err := ws.ReadMessage()
			if err != nil {
				log.Printf("Error: ReadMessage %s", ws.GetURL())
				return
			}

			log.Printf("Success: %s", message)
		}
	}
}

I figured it out :)
	go func() {
		for {
			select {
			default:
				if !ws.IsConnected() {
					time.Sleep(1 * time.Second)
					log.Printf("Websocket disconnected %s", ws.GetURL())
					continue
				}

				_, message, err := ws.ReadMessage()
				if err != nil {
					time.Sleep(1 * time.Second)
					log.Printf("Error: ReadMessage %s", ws.GetURL())
					continue
				}

				log.Printf("Success: %s", message)
			}
		}
	}()


Rename PlayAgainButton.tscn to game/src/UserInterface/PlayAgainButton…

….tscn
changing folder

Resource checked


Implement pagination on queries with hardcoded limit

A few queries didn't have any limit implemented so I added an hardcoded limit of 100 to them in PR https://github.com/liteflow-labs/starter-kit/pull/362 to limit possible API related issues but without a pagination.
So we now need to implement a pagination system on those queries when needed.
I added the comment # TODO: implement pagination to the codebase to find those queries easily.

@ismailToyran could you check what other queries have hardcoded pagination. If that's only the collection's trait value I think we can close this issue for now.


Port Encoding (UTF-8/UTF-16/UTF-32) performance comparison test to BenchmarkDotNet

We have some Utf8/Utf16 encoding performance tests that need to be ported from xunit.performance to BenchmarkDotNet.
https://github.com/dotnet/corefxlab/blob/22c2d4edc6fc16ed50489f6709ee54ab8aede113/tests/System.Text.Primitives.Tests/Encoding/Performance/PerfTests.CoreCLR.cs#L9
https://github.com/dotnet/corefxlab/blob/679861d43063637d523bd36d54c7dd25994c475e/tests/System.Text.Primitives.Tests/Encoding/Performance/PerfTests.Span.cs#L14
Follow the examples from previous efforts to port the performance tests:
https://github.com/dotnet/corefxlab/pull/2240
https://github.com/dotnet/corefxlab/pull/2241
https://github.com/dotnet/corefxlab/pull/2242
https://github.com/dotnet/corefxlab/pull/2243
https://github.com/dotnet/corefxlab/pull/2249
https://github.com/dotnet/corefxlab/pull/2254
https://github.com/dotnet/corefxlab/pull/2260
cc @adamsitnik, @GrabYourPitchforks

will take a look


Consistent env vars for testinfra's Ansible calls

Leverage the same mechanism we use to set env vars for ansible-playbook for testinfra as well
Fixes #164

Does this replace #164 ?

@retr0h Yep.
Replaces #164

I'm happy to +1 if it solves @elementalvoid's problem.

👍

Thanks guys. I can confirm that this resolves the issue.


Building for production

Having an issue when trying to build the app for production: https://ibb.co/HrsK1F3.
Here is my nuxt config:
export default defineNuxtConfig({
   css<EMAIL_ADDRESS>
   // add modules
   modules: ["@nuxtjs/i18n",  "vuetify-nuxt-module"],

   i18n: {
      locales: [
         { code: "en", iso: "en-US", file: "en.ts" },
         { code: "es", iso: "es-ES", file: "es.ts" },
      ],

      lazy: true,
      langDir: "lang",
      defaultLocale: "en",
      detectBrowserLanguage: false,
      strategy: "no_prefix",
      baseUrl: "https://deploii.no/",

      vueI18n: "./config/i18n.config.ts",
   },

   vuetify: {
      moduleOptions: {
         styles: {
            configFile: "assets/scss/_variables.scss",
         },
      },
      vuetifyOptions: "./config/vuetify/vuetify.config.ts",
   },
})

My dependencies:
"devDependencies": {
      "@nuxt/devtools": "latest",
      "@nuxt/eslint-config": "^0.1.1",
      "@nuxt/image": "1.0.0-rc.1",
      "@nuxtjs/eslint-config-typescript": "^12.0.0",
      "@nuxtjs/i18n": "8.0.0-beta.13",
      "@pinia/nuxt": "^0.4.11",
      "@types/node": "^18",
      "eslint": "^8.42.0",
      "eslint-config-prettier": "^8.8.0",
      "nuxt": "^3.6.2",
      "pinia": "^2.1.4",
      "prettier": "^2.8.8",
      "sass": "^1.63.3",
      "vuetify-nuxt-module": "^0.5.6"
   },

Help would be much appreciated.

@AasmundN you need to add vuetify as a dependency.

@AasmundN  Try disabling inlineSSRStyles in your Nuxt config file, I'll try to figure out what's happening, the paths for scss files using absolute paths, should be prefixed with /@fs/, Vuetify needs a deep review (disabling inline SSR Styles, the error should go away):
experimental: {
    inlineSSRStyles: false,
}

@AasmundN you need to add vuetify as a dependency.

No? vuetify-nuxt-module adds it as one of its dependencies.
@userquin Disabling inlineSSRStyles seems to have fixed the problem. I'm guessing doing that shouldn't be necessary?

@userquin Disabling inlineSSRStyles seems to have fixed the problem. I'm guessing doing that shouldn't be necessary?

Experimental inlineSSRStyles is enabled by default, we need to change the custom Vuetify styles plugin here to resolve/load SSR styles properly, I'm trying to fix it.

It seems using Vuetify configFile styles and Nuxt inlineSSRStyles are imcompatible, there is no way to get individual chunks

Okay. Is that a problem? I'm not really familiar with what inlineSSRStyles does. Or how the styles are handled by Nuxt and Vite for that matter.

The problem is the nuxt nitro renderer, since we've some .sass references we will receive the request, but it is missing, I have it working just returning an empty string when receiving the id in Vite load hook with SSR flag: we'll receive requests like this /plugin-vuetify/lib/components/VDialog/VDialog.sass?inline&used.
Since we're using Vuetify styles via configFile we'll get the styles in the vuetify-theme-stylesheet, maybe I can fix the error, but I have no idea if it has side effects...

This is the page without inlining SSR styles:

Inlining SSR styles:

Vuetify should avoid using SASS...

It is not working...

@AasmundN You can try it adding this simple Vite plugin to your nuxt config file, removing inlineSSRStyles: false from experimental option:
vite: {
  plugins: [{
    name: 'vuetify-nuxt-inline-ssr-styles',
    enforce: 'pre',
    load(id, options) {
      if (options?.ssr && id.startsWith('/plugin-vuetify/lib/'))
        return ''
    }
  }]
}

Adding the Vite plugin also seems to work, and it does reduce the size of the css-file like you showed. However when I load the page from the production build there is a little "flick" of the styles, as if the styles are being applied a little after the html appears.

Yeah, the vite plugin doesn't seem to be working properly. Is using inlineSSRStyles important?

Still in the process of deciding what framework to choose for styling in my Nuxt project, and I would really like to use Vuetify as I have used it in other SPA, Vite projects. However, adding Vuetify to Nuxt has been quite painful and things don't seem to work properly at every other turn. I'm not so familiar with how Vite works, but I've been trying to understand wether or not this inline styles thing is a deal breaker or not, and thus if I should choose some other styling framework instead. Or would it just marginally change the load time. If it is important, will eventually work more into fixing it? Thank you for your work, a module like this one is really needed.


Bundle version number still 1.2.0 in 1.3.0 download

hey @deseven  - I just wanted to report that ICHS' bundle version wasn't updated to 1.3.0 (still shows 1.2.0)
and the short version string is empty

Oops :)
Thank you!

I have to ask, how did you notice that? Do you use some software that tracks app versions?
I don't really want to re-release it now as 1.3.1 with such a minor change, might be a good idea to wait for some feedback and fix possible bugs, since 1.3 added some important features.

hi @deseven
we are working on 'MacUpdater' ( https://www.corecode.io/macupdater/ ) which allows users to track and auto-update apps like 'iCanHazShortcut' and a few thousand more. obviously there will be issue whenever the embedded version number in an app is broken so we tend to send bug-reports where-ever possible.
it definitely makes sense to wait with 1.3.1 before fixing this issue. outside of the use of app-updaters this is probably only a minor irritation and confusion.
thanks!

Ok, understood. I'll keep that in mind from now on.


Maximum number of diagnostics exceeded.

I keep getting this warning on a compilation unit that otherwise compiles without warnings.
Don't know if this is the right place to ask but is there something that I don't know about this warning?
I'm using defaults with .ycm_extra_conf.py from ycmd.
Here's my debug info:
Ycmd debug information for buffer main.cpp in c++-mode:

((python
  (executable<EMAIL_ADDRESS>  (version . "2.7.15"))
 (completer
  (items
   ((value . "None")
    (key . "compilation database path"))
   ((value . "[u'-std=c++17', u'-x', u'c++', u'-stdlib=libc++', u'-Wall', u'-Wextra', u'-pedantic', u'-DDEBUG', u'-Wno-gnu-case-range', u'-isystem', u'/usr/local/opt/llvm/include', u'-isystem', u'/usr/local/opt/llvm/include/c++/v1', u'-I', u'/Users/jvillasante/Downloads/test_cpp', u'-resource-dir=/Users/jvillasante/Hacking/software/ycmd/ycmd/../clang_includes', u'-isystem', u'/Library/Developer/CommandLineTools/usr/include/c++/v1', u'-isystem', u'/usr/local/include', u'-isystem', u'/Library/Developer/CommandLineTools/usr/include', u'-isystem', u'/usr/include', u'-isystem', u'/System/Library/Frameworks', u'-isystem', u'/Library/Frameworks', u'-isystem', u'/Library/Developer/CommandLineTools/usr/lib/clang/9.1.0/include', u'-isystem', u'/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include', u'-fspell-checking']")
    (key . "flags"))
   ((value . "/Users/jvillasante/Downloads/test_cpp/src/main.cpp")
    (key . "translation unit")))
  (name . "C-family")
  (servers))
 (extra_conf
  (path . "/Users/jvillasante/Downloads/test_cpp/.ycm_extra_conf.py")
  (is_loaded . t))
 (clang
  (has_support . t)
  (version . "clang version 6.0.0 (tags/RELEASE_600/final)")))

Server is running at: <IP_ADDRESS>:49558

Ycmd Mode is enabled

--------------------

Ycmd version:   1.3snapshot (package: 20180520.353)
Emacs version:  26.1
System:         x86_64-apple-darwin17.6.0
Window system:  mac

Also, there are some flags that I'm not setting myself, does ycmd does this by itself?
All of this flags are not added by me:
u'-resource-dir=/Users/jvillasante/Hacking/software/ycmd/ycmd/../clang_includes', u'-isystem', u'/Library/Developer/CommandLineTools/usr/include/c++/v1', u'-isystem', u'/usr/local/include', u'-isystem', u'/Library/Developer/CommandLineTools/usr/include', u'-isystem', u'/usr/include', u'-isystem', u'/System/Library/Frameworks', u'-isystem', u'/Library/Frameworks', u'-isystem', u'/Library/Developer/CommandLineTools/usr/lib/clang/9.1.0/include', u'-isystem', u'/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include', u'-fspell-checking'

For completion, here's my .ycm_extra_conf.py
import os
import ycm_core

# These are the compilation flags that will be used in case there's no
# compilation database set (by default, one is not set).
# CHANGE THIS LIST OF FLAGS. YES, THIS IS THE DROID YOU HAVE BEEN LOOKING FOR.
flags = [
  '-std=c++17',
  '-x', 'c++',
  '-stdlib=libc++',
  '-Wall',
  '-Wextra',
  '-pedantic',
  '-DDEBUG',
  '-Wno-gnu-case-range',
  '-isystem', '/usr/local/opt/llvm/include',
  '-isystem', '/usr/local/opt/llvm/include/c++/v1',
  '-I', '.',
]

# Set this to the absolute path to the folder (NOT the file!) containing the
# compile_commands.json file to use that instead of 'flags'. See here for
# more details: http://clang.llvm.org/docs/JSONCompilationDatabase.html
#
# You can get CMake to generate this file for you by adding:
#   set( CMAKE_EXPORT_COMPILE_COMMANDS 1 )
# to your CMakeLists.txt file.
#
# Most projects will NOT need to set this to anything; you can just change the
# 'flags' list of compilation flags. Notice that YCM itself uses that approach.
compilation_database_folder = ''

if os.path.exists( compilation_database_folder ):
  database = ycm_core.CompilationDatabase( compilation_database_folder )
else:
  database = None

SOURCE_EXTENSIONS = [ '.cpp', '.cxx', '.cc', '.c', '.m', '.mm' ]

def DirectoryOfThisScript():
  return os.path.dirname( os.path.abspath( __file__ ) )

def IsHeaderFile( filename ):
  extension = os.path.splitext( filename )[ 1 ]
  return extension in [ '.h', '.hxx', '.hpp', '.hh' ]

def GetCompilationInfoForFile( filename ):
  # The compilation_commands.json file generated by CMake does not have entries
  # for header files. So we do our best by asking the db for flags for a
  # corresponding source file, if any. If one exists, the flags for that file
  # should be good enough.
  if IsHeaderFile( filename ):
    basename = os.path.splitext( filename )[ 0 ]
    for extension in SOURCE_EXTENSIONS:
      replacement_file = basename + extension
      if os.path.exists( replacement_file ):
        compilation_info = database.GetCompilationInfoForFile(
          replacement_file )
        if compilation_info.compiler_flags_:
          return compilation_info
    return None
  return database.GetCompilationInfoForFile( filename )

def FlagsForFile( filename, **kwargs ):
  if not database:
    return {
      'flags': flags,
      'include_paths_relative_to_dir': DirectoryOfThisScript()
    }

  compilation_info = GetCompilationInfoForFile( filename )
  if not compilation_info:
    return None

  # Bear in mind that compilation_info.compiler_flags_ does NOT return a
  # python list, but a "list-like" StringVec object.
  final_flags = list( compilation_info.compiler_flags_ )

  return {
    'flags': final_flags,
    'include_paths_relative_to_dir': compilation_info.compiler_working_dir_
  }

Thanks in advance!

This means that ycmd found more than 30 diagnostics when parsing the translation unit. This limit is configurable through the max_diagnostics_to_display option (a value of 0 means no limit). However, emacs-ycmd doesn't seem to provide a way to configure this option.

But why or how does ycmd gets to that number of diagnostincs. The file compiles with all -Wall -Werror et. al. without any warnings.

It seems that ycmd is trying to parse system files as can be seen in this extract of the ycmd logs
(((kind . "ERROR")
  (text . "use of undeclared identifier 'wcschr'")
  (ranges)
  (location
   (filepath . "/usr/local/opt/llvm/include/c++/v1/wchar.h")
   (column_num . 77)
   (line_num . 137))
  (location_extent
   (start
    (filepath . "/usr/local/opt/llvm/include/c++/v1/wchar.h")
    (column_num . 77)
    (line_num . 137))
   (end
    (filepath . "/usr/local/opt/llvm/include/c++/v1/wchar.h")
    (column_num . 83)
    (line_num . 137)))
  (fixit_available . :json-false))

Is there a way to overcome this?

Changing .ycm_extra_config.py to this
.ycm_extra_config file solves the issue. This one can be closed.

Thanks!

I have the same problems and this ycmd extra config does not solve it. I am wondering why ycmd diagnostics does not show first max_diagnostics_to_display entries, rather showing one message that the limit is exceeded. What could be helpful if I could see first errors and may be figure out what am I missing, what is wrong with the code.

I have just cloned the repo and loaded from source. where I changed max_diagnostics_to_display to 0. After that I was able to see some helpful information, it was a single warning. Weird...


Add Private DNS Resolver Outbound Endpoint azurerm_private_dns_resolver_outbound_endpoint

Description of your changes
Added the resource azurerm_private_dns_resolver_outbound_endpoint which is already available in the terraform registry.
Fixes #
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
The code was tested locally with a KIND cluster and by running
kubectl apply -f examples/network/v1beta1/privatednsresolveroutboundendpoint.yaml

Results

/test-examples="examples/network/v1beta1/privatednsresolveroutboundendpoint.yaml"

/test-examples="examples/network/v1beta1/privatednsresolveroutboundendpoint.yaml"


RHBPMS-4435 Fixes behaviour when interval scanning is stopped when kieScanner.scanNow() is called

When scanNow() was called after interval scanning was started with kieScanner.start(),
the interval scanning was stopped, because "status" of kieScanner was set to STOPPED.
Adds reproducer for mentioned issue and proposed fix.

Looks good, thanks @baldimir.
Two questions regarding the test: (1) is there any chance to remove the Thread.sleep()?, (2) should we stop/shutdown the scanner after the test? It seems the background thread will be running until the JVM shutdown, which is not ideal.

After discussion with @psiroky I will add new commit with changes.

@psiroky, requested changes added.

Thnks @baldimir! This is what I had originally in mind https://github.com/droolsjbpm/drools/pull/1065/commits/ba546362b139d2e2cfce6736d48b49b2577d2efd. Please let me know what you think.

@psiroky thanks, that will do, I added the commit also to https://github.com/droolsjbpm/drools/pull/1066


BufferWhileBy should emit empty array if primary source finished first

BufferWhileBy should emit empty array if primary source finished (without any value) before the secondary one.

Could be related to #113

Fixed by #129


Wrong Final Dimension size for wav2vec2 large tpu config

Hi,
I was trying to run wav2vec2 on TPU when I found a strange thing:

In the config file here The final dimension layer is of size 256 which is not the case with wav2vec2_large.
There can be two potential issues:

Either the name of the config is wrong i.e. wav2vec2_large_librivox_tpu-pod should be wav2vec2_base_librivox_tpu-pod

Or the the final dimension layer should be of size 768

Through my inspection I think it should be case 1.

@alexeib Can you please confirm on this?

yeah those dimensions are for the base model. please just copy the "model" part of the config from the librivox and i'll update the tpu configs when i get a chance

Thanks Alexi.
This is fixed here


Add register_custom_call_target to xla_client API docs

This function is (for better or worse) user facing for custom call users. I think it's worth having this in the API docs.
Preview: https://jax--22150.org.readthedocs.build/en/22150/_autosummary/jaxlib.xla_client.register_custom_call_target.html

Looks good - will you be referring to this func in the FFI docs?

will you be referring to this func in the FFI docs?

Yes - that's why I discovered that it wasn't included in the current API docs!

Sounds good, thanks!


Move dco check from pre-git-commit to github actions.

Implement DOC check in github action.
Remove make check because it already run in prow.: https://github.com/openshift/release/blob/7a5ed8d7d0a1e315ed88d7249542d0451d268e89/ci-operator/config/stolostron/managedcluster-import-controller/stolostron-managedcluster-import-controller-main.yaml#L68C1-L71C14

/assign @zhiweiyin318

/approve
/lgtm