Segmentation Fault - maybe because of setting globals inside async

Hello,
I am trying to use Python with Julia's async tasks.
I've set
const pyglobals = get!(PythonCall.pydict, PythonCall.MODULE_GLOBALS, Main)

and just bring julia variables to python via pyglobals["variable"] = value. Maybe this is false?
Somehow when calling PythonCall inside Julia's async and redefining pyglobals inbetween leads to the following segmentation fault.
[24149] signal (11.1): Segmentation fault
in expression starting at none:1
_PyFrame_IsIncomplete at /usr/local/src/conda/python-3.12.0/Include/internal/pycore_frame.h:176 [inlined]
_PyFrame_GetFirstComplete at /usr/local/src/conda/python-3.12.0/Include/internal/pycore_frame.h:182 [inlined]
_PyThreadState_GetFrame at /usr/local/src/conda/python-3.12.0/Include/internal/pycore_frame.h:191 [inlined]
PyEval_GetGlobals at /usr/local/src/conda/python-3.12.0/Python/ceval.c:2355 [inlined]
PyImport_Import at /usr/local/src/conda/python-3.12.0/Python/import.c:2998
PyImport_Import at /home/ssahm/.julia/packages/PythonCall/wXfah/src/cpython/pointers.jl:299 [inlined]
macro expansion at /home/ssahm/.julia/packages/PythonCall/wXfah/src/Py.jl:131 [inlined]
pyimport at /home/ssahm/.julia/packages/PythonCall/wXfah/src/concrete/import.jl:11
deserialize_py at /home/ssahm/.julia/packages/PythonCall/wXfah/src/compat/serialization.jl:19
deserialize at /home/ssahm/.julia/packages/PythonCall/wXfah/src/compat/serialization.jl:28
unknown function (ip: 0x7fa2f13eff62)
_jl_invoke at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2758 [inlined]
ijl_apply_generic at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2940
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:880
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1499
unknown function (ip: 0x7fa2f1324406)
_jl_invoke at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2758 [inlined]
ijl_apply_generic at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2940
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:962
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
deserialize_expr at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:1277
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:896
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816
#5 at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:975
ntupleany at ./ntuple.jl:43
deserialize_tuple at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:975
handle_deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:859
deserialize at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Serialization/src/Serialization.jl:816 [inlined]
deserialize_msg at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Distributed/src/messages.jl:87
unknown function (ip: 0x7fa2dcbb5eb2)
_jl_invoke at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2758 [inlined]
ijl_apply_generic at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2940
jl_apply at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/julia.h:1880 [inlined]
jl_f__call_latest at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/builtins.c:774
#invokelatest#2 at ./essentials.jl:819 [inlined]
invokelatest at ./essentials.jl:816 [inlined]
message_handler_loop at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Distributed/src/process_messages.jl:176
process_tcp_streams at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/usr/share/julia/stdlib/v1.9/Distributed/src/process_messages.jl:133
#103 at ./task.jl:514
unknown function (ip: 0x7fa2f131f25f)
_jl_invoke at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2758 [inlined]
ijl_apply_generic at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/gf.c:2940
jl_apply at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/julia.h:1880 [inlined]
start_task at /cache/build/default-amdci5-5/julialang/julia-release-1-dot-9/src/task.c:1092
Allocations: 33967797 (Pool: 33946601; Big: 21196); GC: 49

Any help is highly appreciated

I think I found the root cause: It is a three-level nesting Julia - Python - Julia which breaks
Concretely, if

I am calling a python function from Julia, and
within the python function I am using julias sleep` for better task interactions, and
I am running another python code while the original function is in the julia sleep part

Then the above segmentation fault goes.
So I guess, one needs to make sure that python functions are always completely executed before a new python function is called.
Closing this, because this workaround sounds reasonable and works


CORE-3151 Replace membership group reader with the real implementation in the link manager

Integrate the MembershipGroupReader with the LinkManager. By default the stub component is still used, the real Membership Group Reader can be turned on using the option --without-stubs.
The real MembershipGroupReader had a slightly different API to the one used by our stub component (StubMembershipGroupReader): it needs to know the looking up member's holding identity, as different members can have a different view of the Membership group reader. To save time, because the stub code will soon be removed, I did not update the stub to behave in the same way. The stub ignores the looking up member's holding identity and always returns the same MemberInfo for a given lookupIdentity.
This API change meant I also needed to change the AuthenticationProtocol API. As there was no way to extract both members session public key hashes. I added a Lambda parameter as a part of validatePeerHandshakeMessage which does the pessary lookup. Data can be returned from the lambda to the layer which calls validatePeerHandshakeMessage via a template parameter. This was a simplest way I could find to prevent duplicate lookups in the LinkManager.

just a couple of comments - also, have you tried to run the link manager with stubs to confirm everything is wired up and starts successfully? I've noticed some components outside p2p do not start their child component themselves, so the it has to be done on our side. Not sure if there's anything like that here, but this would be an easy way to confirm.

I checked it and it started correctly. Deployed the corda cluster by helm charts ran sudo kubefwd svc -n corda  and java -jar ./applications/p2p-link-manager/build/bin/corda-p2p-link-manager-<IP_ADDRESS>-SNAPSHOT.jar  --without-stubs -mbootstrap.servers=prereqs-kafka:9092.
From logs:
2022-07-11 18:03:52.174 [lifecycle-coordinator-13] INFO  ForwardingMembershipGroupReader-1 - Status change: child net.corda.membership.read.MembershipGroupReaderProvider went up.


Issue#15

Issue#15
This  branch is get it from Issue#17 branch not from main! so that branch has in actual Issue#17 and current one.

[ ] You've read the Contributor Guide and Code of Conduct.
[ ] You've included unit or integration tests for your change, where applicable.
[ ] You've included inline docs for your change, where applicable.
[ ] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)
Description
{Detail}
Fixes #{bug number} (in this specific format)

Please update the PR message where says:
{Detail}
With a little bit of details of the changes
And where says:
Fixes #{bug number} (in this specific format)
With
Fixes #15


Can't get a resource value when a cached value is returned.

Issue
Can't get a resource value using getResourceValue() right after the resource value changed and cached value is returned.
Steps to reproduce
Tested Environment
* K64F
* macOS High Sierra Version 10.13.6
* node v8.9.4
*<EMAIL_ADDRESS>
Here are steps to reproduce this issue.
API Key
Get an API key from Pelion Portal.
Set up a board running QuickStart firmware
Prepare a board which runs the firmware provided in QuickStart.
Get a device ID (endpoint name) from device's serial output or Device List in  Pelion Portal.
Test program
I created a test program to reproduce this issue. Please download and set it up.
https://github.com/coisme/Pelion-NodeJS-Test-1
This test program calls getResourceValue() every 5 seconds and get the resource value.
Example Program Output
The button on the board was pushed once between 12:07:48 and 12:07:52. Data btnCnt is missing for a while after this.
$ node index.js
12:07:32, Call, Num=1
12:07:34,  Ret, Num=1, btnCnt=15
12:07:37, Call, Num=2
12:07:38,  Ret, Num=2, btnCnt=15
12:07:42, Call, Num=3
12:07:43,  Ret, Num=3, btnCnt=15
12:07:47, Call, Num=4
12:07:48,  Ret, Num=4, btnCnt=15
12:07:52, Call, Num=5
12:07:53,  Ret, Num=5, btnCnt={}
12:07:57, Call, Num=6
12:07:58,  Ret, Num=6, btnCnt={}
12:08:02, Call, Num=7
12:08:03,  Ret, Num=7, btnCnt={}
12:08:07, Call, Num=8
12:08:08,  Ret, Num=8, btnCnt={}
12:08:12, Call, Num=9
12:08:13,  Ret, Num=9, btnCnt={}
12:08:17, Call, Num=10
12:08:18,  Ret, Num=10, btnCnt={}
12:08:22, Call, Num=11
12:08:23,  Ret, Num=11, btnCnt={}
12:08:27, Call, Num=12
12:08:28,  Ret, Num=12, btnCnt={}
12:08:32, Call, Num=13
12:08:33,  Ret, Num=13, btnCnt={}
12:08:37, Call, Num=14
12:08:38,  Ret, Num=14, btnCnt={}
12:08:42, Call, Num=15
12:08:43,  Ret, Num=15, btnCnt={}
12:08:47, Call, Num=16
12:08:48,  Ret, Num=16, btnCnt={}
12:08:52, Call, Num=17
12:08:53,  Ret, Num=17, btnCnt=16
12:08:57, Call, Num=18
12:08:58,  Ret, Num=18, btnCnt=16
12:09:02, Call, Num=19

Additional Information
Python SDK's behavior
I created the same program with the latest Python SDK. This program works well even if the button is pushed.
https://github.com/coisme/Pelion-Python-Test-1
Example Output
The button on the board was pushed three times between 12:33:03 and 12:33:08 but the return value is handled correctly.
$ python pull-notification.py
12:32:53 btn=16
12:32:58 btn=16
12:33:03 btn=16
12:33:08 btn=19
12:33:13 btn=19
12:33:18 btn=19

Calls Service API directly
In addition, I tried using Service API directly. Here's my script:
https://github.com/coisme/Perion-API-Test-1
According to my experiment with this script, when in stable state, GET /v2/endpoints/{device-id}/{resourcePath} returns status code 202 and AsyncId. We can get the resource value from notification channel. However, when the button on the board is pushed, the API returns status code 200 and cached value for a while. The cached value is returned by the response's body and no data is coming from notification channel. If you call GET /v2/notification/pull in this context, it will get timeout.
Is this handled in Javascript SDK properly?
Javascript SDK v1.2.2 works well
If I use Javascript SDK v1.2.2, which is used by quick start, this issue doesn't happen.
https://github.com/coisme/Pelion-NodeJS-Test-1/tree/use-v1.2.2
If I update the module from v1.2.2 to v1.2.3, this issue occurs. It indicates something was injected between v1.2.2 and v1.2.3...

Related to JIRA IOTSTOOL-1661

Thank you for the analysis. Your observation on the async response was a problem we had with earlier versions of the SDKs as the behaviour wasn't originally documented and only discovered by accident during our own testing. From v<IP_ADDRESS>2 of the Python SDK and v 1.2.7 of the JavaScript SDK the SDKs use a newer endpoint /v2/device-requests/{device-id} to obtain resource values.

@madchutney Thank you for your comment. In the current master of JS SDK, ConnectAPI.getResourceValue() calls ResourcesAPI.getResourceValue(). In this function, it seems that /v2/endpoints/{device-id}/{resourcePath} is used.
https://github.com/ARMmbed/mbed-cloud-sdk-javascript/blob/f05738b5e6aa9151041d00ab4c66b6937c293e1d/src/_api/mds.ts#L753-L754
I guess that to use /v2/device-requests/{device-id}, DeviceRequestsAPI.createAsyncRequest() should be called. However, it seems not called, when I checked my program step by step with debugger.

 We didn't make the change because the old endpoint was still working and we didn't want to cause a regression.

K64F as well.
Could you try capturing the http response that produces the btnCnt={}.
If I can mock the response that causes the {}, then we may find an issue. If we can't reproduce this then I'm struggling to see how else we can help. The SDK is only base64 decoding the response it gets from the notification channel.

Hi @alexl0gan ,
I used a debugger and put the breakpoint at line 22 in index.js.
https://github.com/coisme/Pelion-NodeJS-Test-1/blob/f3840209f2e852a1da2388307616dd3b28073082/index.js#L22
When the button count is got successfully, ConnectAPI._pollRequest contains Response object and it has text field (highlighted blue in the picture below), which is JSON format result from the server.

On the other hand, right after the button on the board is pushed, i.e. shows{},  ConnectAPI._pollRequest doesn't contain Response object.

So getResourceValue() returns with no data.
If you need more info, let me know!

Could you put a breakpoint in node_modules/mbed-cloud-sdk/lib/connect/connectApi.js line 259 so you can see what the response from the long poll call is.

The duration of {} is always about 1 minute. This is very close to the time length which server returns status code 200.
Here's the script I used, directly calls Service API:
https://github.com/coisme/Pelion-ConnectAPI-Test-1
Output:
$ ./pull-notification.sh
09:42:58 btnCnt=7
09:43:07 btnCnt=7
09:43:14 btnCnt=7
09:43:22 btnCnt=7
09:43:29 btnCnt=8  (cached)
09:43:35 btnCnt=8  (cached)
09:43:42 btnCnt=8  (cached)
09:43:48 btnCnt=8  (cached)
09:43:55 btnCnt=8  (cached)
09:44:01 btnCnt=8  (cached)
09:44:09 btnCnt=8  (cached)
09:44:15 btnCnt=8  (cached)
09:44:21 btnCnt=8  (cached)
09:44:28 btnCnt=8
09:44:35^C

See the internal bug tracker IOTSTOOL-1661. It seems that we shouldn’t use GET /v2/endpoints/\{device-id}/\{resourcePath}...


[#117705909] Upgrade cf-test-helpers for password redaction

What
Our custom acceptance and performance tests currently expose the (now
temporary) admin password in the output on every run.
More recent versions of cf-test-helpers, as used by cf-acceptance-tests and
cf-smoke-tests, now redact the password. By upgrading our test suites we can
benefit from the same thing.
The output now looks like:
[2016-05-06 14:05:36.20 (UTC)]> cf auth admin [REDACTED]
API endpoint: https://api.dcarley.dev.cloudpipeline.digital
Authenticating...
OK
Use 'cf target' to view or set your target org and space

I've taken current HEAD of cf-test-helpers as used by cf-acceptance-tests
(not related to any particular cf-release version):

cloudfoundry/cf-acceptance-tests@3582729

The complete diff from the previous version is as follows:

<EMAIL_ADDRESS>
The commits relevant to password output are:

cloudfoundry-incubator/cf-test-helpers@7df92b1
cloudfoundry-incubator/cf-test-helpers@f4821ac

I think there's still an outstanding issue with our tests producing more
output than you'd normally expect. cf-acceptance-tests don't log cf auth
under normal circumstances (though they might during failure - which is why
we still need the temporary user).
How to review

Deploy the pipeline from this branch.
Run the pipeline.
Check that all the tests pass.
Check that the password does not appear in the test output.

Who can review
Not @dcarley

Can you rebase on master? It includes fix for:
Adding bosh-CA to root certificates
tar: can't open 'bosh-CA/bosh-CA.tar.gz': No such file or directory

Can you rebase on master?

Sure, done

Looks good, passwords are now redacted
cf auth custom-acceptance-test-user-TRTPidv4dc [REDACTED]

I'll wait for performance tests to finish.

Performance tests also pass with redacted passwords:
cf auth performance-tests-user-0GB5pkQT46 [REDACTED]

Merging....


Hero transitionOnUserGestures: element is missing after double back gesture

Here's example https://gist.github.com/skabber/8dbf7cdee1431b13d8b87d8ff9075fa3

Flutter doctor
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.15.1 19B88, locale en-RU)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.1)
[✓] VS Code (version 1.35.1)
[✓] Connected device (2 available)
upd:
harder to encounter in iOS, but still possible

Hi  @radroot1
I tried to reproduce your issue but it works fine on iOS
I used sample code https://gist.github.com/skabber/8dbf7cdee1431b13d8b87d8ff9075fa3
if you're still experiencing this problem with version of Flutter, please create a new issue and provide the information required in
the template.
Thank you


Added util functions to get nodes of different depth from a hierarchical nodes list

Description
Added util functions to get nodes of different depth from a hierarchical nodes list.
When one creates a hierarchical chunking especially with a DB backed vector store, one cannot create an index with nodes of a given chunk easily. This utils enables users to get nodes with different depth which correspond to different chunk sizes and add the chunk size as metadata to the nodes and filter nodes later based on this metadata.
This can be achieved in a different way as well; which is by adding chunk_size of the node_parser to the nodes as metadata when creating the nodes; however, it seems more reasonable to provide utils that enables users to achieve this themselves than making nodes bulkier for every user that might not need this metadata.
Type of Change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] Added new unit/integration tests

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@qiisziilbash if you can fix up the CI errors, we can get this merged
Seems you may need to update the __init__.py or update the import path in the tests
ImportError: cannot import name 'get_deeper_nodes' from 'llama_index.node_parser' (/home/runner/work/llama_index/llama_index/llama_index/node_parser/__init__.py)

Good catch; done!

@logan-markewich feel free to merge this; soes not seem like I have permission to do so

@qiisziilbash apologies, with v0.10.0 the project structure has changed slightly. Was focused on getting that landed.
I can take a stab at porting this PR in a bit, if you don't get to it first :)

@logan-markewich this should be good to go in

(ugh, merging main locally brought in extra changes, all good though, they will get merged anyways lol)


Valar Dohaeris bug

Turn 3 plot reveal.
Opponent = Greyjoy, Greensight. Plot revealed = "Heads on Spike"
Me = Baratheon Rose. Plot revealed = "Valar Dohaeris"
Sequence:

Opp won initiative, choose to go first and trigger heads.
Once heads hit my Greathall.
Valar did not trigger.
Greensight triggered.

Video of the game: https://youtu.be/lNceoRdIN7Q, forward to 9:28 for the incident.

Thanks for reporting. The video you linked is unavailable for me.

Had to verify YouTube. The video should be available now.
https://youtu.be/lNceoRdIN7Q?t=9m15s

the odd thing is that it was literally just passed over, it didn't appear as if Heads or greensight had anything to do with it.  Greensight triggered because of draw stage - its condition requires phase to be 'draw', so it's either something cascading from Heads on Spikes or dohaeris skipping itself with proceedToNextStep() - but you'd still expect it to print messages (Dohaeris has a ton of text that fires even when characters aren't selected).
This some spooky x-files level stuff.

I sense the Faceless men in action...

This is indeed spooky. 👽
Thanks for including the video! So far I haven't been able to reproduce it. A few theories for anyone else looking into it.

There may be some sort of obscure condition in ForcedTriggeredAbilityWindow that skips the remaining "When Revealed" ability after the first one is chosen. This might happen if the meetsRequirements method for Dohaeris went from returning true at the start of the prompt to false after Spikes was resolved. Though I don't immediately see what could cause that with Dohaeris.
There may be a race condition in ForcedTriggeredAbilityWindow. e.g. if the button were double clicked. However, I think at best it would have done Spikes twice and then done Dohaeris as normal given the code (unless there's a problem like the previous bullet point).
Some sort of crash in AbilityResolver could have cancelled the resolution of Dohaeris. However, it should have generated a "server error" message in chat (it didn't from the video), and sent us an error report (no matching report). So it wouldn't just silently fail.
The use of Withering Cold before the new set of plots is somewhat suspicious since it's a game flow altering effect but I don't really see how it could cause this. The fact that characters were kneeling when Dohaeris was supposed to trigger doesn't seem to matter either.
If Valar Dohaeris somehow saw no players players when it activated, it would complete with no messages at all. However, if the game is returning no players then a LOT of other things would be broken as well. Therefore this scenario is highly unlikely.

Has anyone seen this happen again (or similar) in the past two months? We significantly changed how ability prompts, including plot reveals, work in #1911 and a few other updates, so it's possible whatever this was has been (accidentally) fixed since it was filed.

Nope, I think it's fair at this point to close this as unreproducible. (And it can easily be reopened if it pops up again.) Thanks for noticing!


feat(controller): Container set template

@jessesuen this v1.0 is ready for proto review.

@whynowy  can I ask you to be the reviewer on this and give me your feedback today or tomorrow please?
@jessesuen can I get approval on the proto changes in the same timeframe please?

I'd like to revisit the syntax of nesting containers in both graph and sequence. Since a sequence is really just a specialization of the graph, I don't think we need a separate graph/sequence stanza that repeats the container list. The concept of sequentializing the graph is really just a convenience around not having to write depends on all of the containers.
I am proposing the following:
By default, a DAG style of container execution would be used:
      containerSet:
        volumeMounts:
          - mountPath: /workspace
            name: workspace
        containers:
          - name: A
            image: argoproj/argosay:v2
          - name: B
            image: argoproj/argosay:v2
          - name: main
            image: argoproj/argosay:v2
            depends: A && B

If user wanted sequenced run of their container, they would specify sequence option, without specifying depends everywhere:
      containerSet:
        sequence: {}
        volumeMounts:
          - mountPath: /workspace
            name: workspace
        containers:
          - name: A
            image: argoproj/argosay:v2
          - name: B
            image: argoproj/argosay:v2

I recall that one of the original reasons for having both graph and sequence was because there might be graph specific options such as target, and sequence specific options. I'm arguing that that is probably not the case since a sequence is a specialization of a dag. For example, if we were to introduce target, target would actually still be applicable to a sequence as well.
This would also put to rest the graph vs. dag debate as that is no longer part of the syntax.

I specifically don't want to do this. By having the containers perform double-duty as both a sequence and as graph I have to write additional logic and tests to validate that containers do not have dependencies when they are sequenced. I think this designs-in bugs.
I would be open to discussing removing sequenced completely, as all use cases can be done using a DAG.
Regarding depends vs dependencies.
I prefer and use 'dependencies' myself as the syntax is declarative, (a) more cloud-native (b) easier to reason about and (c) automatable. depends, while more powerful falls down on these fronts. I'd like to gather more data from the community on this.

I think this designs-in bugs.

I actually feel the opposite is true and that keeping both graph/sequence as separately treated entities would cause more design-in bugs, rather than treating sequence as syntatic sugar around depends. We would be repeating the approach that we made with steps and dag templates. Having different names/reference paths would cause some of the following problems:

Let's say in the future we needed to provide reference variables to the containers list. We will need to provide two sets of separate variables such as {{graph.XXX.outputs.YYY}} vs. {{sequence.XXX.outputs.YYY}} like we had to do with steps and dags.

Sometimes you need to teach other tooling, like kustomize, everywhere there are container references in the spec. An example of this is the argo-rollouts transform configuration in which we describe to kustomize, everywhere that you can have references to configmaps, secrets, volume mounts, labels:
https://github.com/argoproj/argo-rollouts/blob/master/docs/features/kustomize/rollout-transform.yaml#L2-L86

Similarly, Argo CD can be taught about that that portions of the spec can contain Containers, PodTemplates, etc...
https://github.com/argoproj/argo-cd/blob/5af0c5ad3a52ba8bfa16cce90855efce761395ed/docs/user-guide/diffing.md#known-kubernetes-types-in-crds-resource-limits-volume-mounts-etc

However, I would like to keep a flexible syntax that allows us to add features in the future, and for me that is very strong reason to keep graph and sequence.

Normally I would agree with you if the underlying entity had obvious differences (e.g. s3 vs. hdfs artifact types). However, in this case, I don't anticipate there will ever be any. Taking our DAG and Steps template as evidence, AFAIK there aren't any step or dag-specific features that wouldn't apply to both (even taking dag targets, the dag targets feature would apply to steps). My belief is that the only thing that we will ever have under graph or sequence is the container list, which will be unfortunate syntax if that's where we would ultimately land. Are there any that you can anticipate that you can think of? Maybe that would sway my mind.

Aside: your argument to adding depends goes directly contrary to removing graph and sequence.

Yes, I do realize depends is ignored/overridden if sequence is specified, but I didn't consider that as an issue, as it's common for fields to become ignored when not applicable.

I actually feel the opposite is true and that keeping both graph/sequence as separately treated entities would cause more design-in bugs, rather than treating sequence as syntatic sugar around a graph. We would be repeating the approach that we made with steps and dag templates. Having different names/reference paths would cause some of the following problems:

Both of these are a consequence of having sequence supported. It's syntactic sugar in both cases.
To remove this problem, we should remove sequence as an option.

To be clear

I don't believe it is necessary to provide examples of when extensibility could be needed - only to know that could be the case - any specific example could be argued away - the general problem cannot.
I don't think we should have context-dependant types in our spec - a sequenced flag introduces that problem.

To remove this problem, IMHO we should remove sequence as an option completely.

OK, this would be preferable to me over nesting containers under graph/sequence. Especially since we would never have more than a handful of containers in the list, only saving one line of YAML per container.

@alexec I am agreeing to your proposal to the remove the sequence feature, in order to prevent nesting of containers list under a graph/sequence stanza, which I think would cause more problems.

I'm aware we're going around trying to get an agreement on this. So these are my thoughts...
My 1st preference is to keep what we have currently. This has a more complex spec, but it is extensible. It maybe overly extensible, but that is better than not extensible enough.
My 2nd preference would be to have containers and eschew sequenced containers completely. Users would have to use dependencies/depends forever. The syntax and implementation will be simpler: both more user friendly and less buggy.
It'll me take about 2h to make the changes today.


Expand vars of Include while preserving type information

fix #139

@k1LoW
Can you give me your opinion on the implementation image?
May need refactoring, including the expand function

GREAT WORK!!! I think it is clear that it is at least better than what we have now.

I tested it by integrating it into the actual scenario and found another error.
Please wait a moment while we investigate.

Maybe this will work.
https://github.com/k1LoW/runn/pull/140/commits/e8cf3972399df62d8dadb12acf211290c7e27973

I will release a new version at least tomorrow!


esp_https_server: missing initializer for member 'httpd_config::uri_match_fn'

Hi guys,
I'm using current master branch of esp-idf under linux with cmake.
When compiling my program using esp_https_server I get the following warning
warning: missing initializer for member 'httpd_config::uri_match_fn' [-Wmissing-field-initializers] [build]     conf = HTTPD_SSL_CONFIG_DEFAULT();
I think within the header esp_https_server.h inside the macro HTTPD_SSL_CONFIG_DEFAULT() the initializer for httpd_config::uri_match_fn is missing.
Could you please add this?
Thanks
Chrissy

@chrissyheb It is already present or maybe you are referring to some different IDF version?
https://github.com/espressif/esp-idf/blob/140b6e3893317228debf5815b790e967c8562c62/components/esp_https_server/include/esp_https_server.h#L98

Seems it has been added two days ago:
https://github.com/espressif/esp-idf/commit/1af5d0d87cc2cf2f259ebd949da706f0d454d6bb
Thanks


Refactor S3, replace high-level resource/session API with low-level client API

While functionally they are the same, the session/resource stuff is not safe to use across multiple threads and subprocesses, and the client stuff is. The former is a bit easier to use directly than the latter, but this does not impact the user, because smart_open is doing all the work.
I also refactored the way we pass keyword parameters to boto3. Previously, we had a separate parameter for each boto3 function. This was a pain, because this made parameter lists longer for each new function, e.g.

resource_kwargs
multipart_upload_kwargs
singlepart_upload_kwargs
object_kwargs

This PR moves all of the above parameters into a single nested dict and introduces a wrapper that transparently injects the parameters into the required function call.
This does break backwards compatibility, so will need a major version bump when releasing.

@piskvorky Can you please have a look at this? I'd like to merge and release so I can use this in one of my projects.

@piskvorky Ping

This is a breaking change. People doing stuff with kwargs will have to switch from using resource paramaters to using client ones. This will mean a major version bump.

OK. And is this documented somewhere? A migration guide?

Good point. I added a section to the migration guide. Please have a look.

Updated https://github.com/RaRe-Technologies/smart_open/blob/client/MIGRATING_FROM_OLDER_VERSIONS.rst, please have a look.


Only using new plugin...

Create a new project...
App Engine isn't listed as an option anymore.   This seems like a deficit.

@lesv thanks for reporting. I assume you are referring to the new project generation wizard. Did you select Java (on the left panel)? Also are you using Ultimate Edition (this feature is not available in Community Edition)?

Yep - once I turned off the Intellij plugin, I was left w/o the option.  (ie. it's not in ours).

@lesv makes sense. I will close this issue then sense its currently working as expected. Like I mentioned, this will soon be available in our plugin once things get merged in.


Zendesk reference link is broken on this page - PCF 2.3

This is for PCF 2.3 topic Configure LDAP as an Identity Provider for PAS. There is a reference link provided on how to configure LDAP as mentioned below, which is broken. It has to be updated.
https://discuss.zendesk.com/hc/en-us/articles/204140418-Configuring-LDAP-Integration-with-Pivotal-Cloud-Foundry-

Thanks @parth-pandit ! Looks like one of our writers pushed a fix for this: https://github.com/pivotal-cf/docs-ops-guide/commit/d3610b69996e9d490d3c1786146a179a9392c5fc
Thanks @animatedmax !!


Tracking of Learning Style as Tags after lessons is logged

After recording of learning style: visual, auditory, reading/writing preference and kinesthetic (ENUMS?), will automatically put a tag of the student
Make subjects ENUMS + ability (with 1 being the weakest) (e.g -subject English 1)
Record weak subjects/topic

I've tried to implement it but encountered bugs.
It's in my personal repo but I'll not PR it in the interest of time,
:(


Busy loop on sentinel failure + fix

I'm running a Redis master/slave+slave with 3 sentinels
I'm connecting to it using NewFailoverClient() passing a list of all 3 sentinels.
If the first sentinel in the list fails then my app busy loops reporting:
redis: 2018/09/29 20:29:31 pubsub.go:149: redis: discarding bad PubSub connection: EOF
redis: 2018/09/29 20:29:31 sentinel.go:323: sentinel: ReceiveMessage failed: EOF
redis: 2018/09/29 20:29:31 sentinel.go:323: sentinel: ReceiveMessage failed: dial tcp <IP_ADDRESS>:26379: connect: connection refused
redis: 2018/09/29 20:29:31 sentinel.go:323: sentinel: ReceiveMessage failed: dial tcp <IP_ADDRESS>:26379: connect: connection refused
...
The following fix to sentinel.go fixes the problem:
diff --git a/sentinel.go b/sentinel.go
index c5f7149..9d10135 100644
--- a/sentinel.go
+++ b/sentinel.go
@@ -3,6 +3,7 @@ package redis
 import (
 	"crypto/tls"
 	"errors"
+	"io"
 	"net"
 	"strings"
 	"sync"
@@ -316,7 +317,7 @@ func (d *sentinelFailover) listen(sentinel *SentinelClient) {
 	for {
 		msg, err := pubsub.ReceiveMessage()
 		if err != nil {
-			if err == pool.ErrClosed {
+			if err == pool.ErrClosed || err == io.EOF {
 				d.resetSentinel()
 				return
 			}

I am going to improve that code to use Channel, but my understanding is that this code is responsible for detecting dead sentinel and reconnecting.

Closing as stale. Please re-open with more details if you still experience any problems.


Save image into docker storage

We need to be able to load pulled images into docker storage so after a
kubectl tag pull xyz
We can visualize the image with
docker images

Solved by https://github.com/ricardomaraschini/tagger/pull/56


Support 'columns' in aggregation view param of druid-geo

For restricting needless columns to be included in result.

Done


Support for HL-340

Hello Sir,
Must admit a great stable library to use in todays world. Keep up the good work!!!
I would like to know if HL-340 is supported on DroidTerm Android APK with pro version or using this library to achieve the USB-Serial functioanality.
Regards,
CPal

Hello @gurtajs4
I own one like that and it works well.

OK sir, I saw there was some issue in enumerating the device. Soon will share the log if get around.
Thanks
CPal


Responsive-UI workarounds

Minor changes which would involve margin fixes for instance within larger screens.

Thanks a lot for your contribution @jamesgeorge007! I'll reflect that in the contributors section
of the readme :)
I see two sets of changes:

ghuser-on-github: unfortunately I don't think this is the way to go because:

This button and the right panel aren't aligned anymore:

We don't want a workaround on one button, but full responsiveness instead. So I think we want
this part to become responsive:.page-content {
  min-width: 1012px;
}

and then we want the profile layout to fully reorganize itself. Ideally we should follow
GitHub's style and layout, so on mobile ghuser.io should be similar to the mobile version of
GitHub.

logo-with-punchline: this looks nice, thanks!

If the ghuser-on-github button doesn't fit on mobile, then OK. Because otherwise there is no point having a hamburger button since we have nothing else to put in the menu.
Also the mobile version of GitHub has a hamburger button, and that's the style/layout we want to follow, so :+1:


Installing the latest version

The version that's getting installed seemed to be quite old i.e v20160327.0 which is over a month ago.
If there a way to install a more up to date version or is that a bad idea?

@dottodot we have two release channels: stable and nightly.
We release the nightly channel every day (provided the master branch has passed CI), and we currently release to the stable channel manually when we feel a nightly version has been tested "in the real world".
By default, the install script will use the stable channel (which is currently pointing at v20160327.0) but you can instead use the nightly channel by setting FLYNN_UPDATE_CHANNEL=nightly when running the install script:
sudo FLYNN_UPDATE_CHANNEL=nightly bash < <(curl -fsSL https://dl.flynn.io/install-flynn)

See the Stability doc for more information.

Can I switch to nightly and later back to stable?

@philiplb in theory yes, but it is not a well tested approach (switching from nightly back to stable and then running flynn update could try and revert to an older version, something we do not currently make effort to support).
Feel free to try it though, the current channel is stored in /etc/flynn/channel.txt, so to switch to nightly run echo nightly > /etc/flynn/channel.txt as root.

@lmars Gotcha, thank you.

So when I see on your blog that there has been bug fixes I'm assuming that's only being applied to the nightly build.

@dottodot once a bug fix is merged into master, it will make it into a nightly release (provided master has passed CI) and we also blog that the change has been merged, but we tend to only release those fixes to the stable branch once we have verified there are no regressions "in the real world" for that release, which is currently a decision we make manually and at varying frequency.


MacOS createPlatformInAppWebViewWidget is not implemented on the current platform.

UnimplementedError: createPlatformInAppWebViewWidget is not implemented on the current platform.
See also: https://flutter.dev/docs/testing/errors
I used  flutter_inappwebview: ^6.0.0-beta.32 version, Flutter version: 3.13.1, MacOS 14.1. Release and Debug mode both not worked

That’s correct, it is not supported yet

Which macOS version support now?

Currently, you can use InAppBrowser or HeadlessInAppWebView on macOS.
InAppWebView is not supported yet because Flutter itself doesn’t support platform views on desktop.

Currently, you can use InAppBrowser or HeadlessInAppWebView on macOS. InAppWebView is not supported yet because Flutter itself doesn’t support platform views on desktop.

That is actually critical information that should be highlighted and emphasized in the macOS sections of this plugin readme.

@ekuleshov each flutter_inappwebview API has already its own Platform support list.
Check the official API Reference for it.

@ekuleshov each flutter_inappwebview API has already its own Platform support list. Check the official API Reference for it.

@pichillilorenzo you missed the readme part. It makes an illusion that all platforms are fully supported, yet you have to dig through API Reference pages to discover severe limitations of each individual platform.

Probably, for each main feature, such as InAppWebView, InAppBrowser, CookieManager, etc., I can add a simple platform support table.
But for each specific API, it’s not feasible
, so you should use the API Reference and/or doc comments.
There are a lot of classes, methods, and properties, so it’s not feasible to list them all.
For that, you would use the API Reference.
As you can see, this plugin is not simple and doesn’t contain just few methods.
If you want, feel free to make pull requests on the repo of the official website.

Also, WebViews are not a simple concept.
There is no magic here, just a lot of work on my free time.

For desktop support, follow https://github.com/pichillilorenzo/flutter_inappwebview/issues/460

I agree with the creator of this issue and the comments from @ekuleshov. I've decided to migrate to your package from webview_cef when I see the macOS support. However, it's not yet supported as intended as far as I understand. You may highlight this critical information @pichillilorenzo. Thank you for all your hard work and the package. Have a nice year 🙏

As I said, I will add info for the main features to have a quick overview for platform support also on the README and on the official website, but for methods and properties you should use the API Reference.
You can also help, adding documentation on the repo or updating the official website making pull requests 👍.
webview_cef seems to use its own “platform view” implementation.
Maybe it could be a possible way to implement the InAppWebView widget before Flutter itself releases a new version that natively supports it.

Hello contributors, in the Flutter InAppWebView version 6.0.0 release, the inAppWebView functionality is still not operational on MacOS (Sonama 14.2.1 (23C71)). While developing a desktop app, I've resorted to using the webview_windows plugin for Windows. However, I'm facing difficulties finding a similar plugin for MacOS.

Hello contributors, in the Flutter InAppWebView version 6.0.0 release, the inAppWebView functionality is still not operational on MacOS (Sonama 14.2.1 (23C71)). While developing a desktop app, I've resorted to using the webview_windows plugin for Windows. However, I'm facing difficulties finding a similar plugin for MacOS.

Hello, you may choose to use webview_cef for now. You should not add it directly as a dependency but you should add it as a path dependency like:
  webview_cef:     path: ./custom-packages/custom-webview_cef
Additionally, since its Pubdev page is not up to date, you need to refer to the GitHub page and GitHub readme.


[Question] Additional Options for QR Code Generation.

Hi Redth,
Would like to check currently does the ZXing.Net.Mobile included additional options when generating the QR code ? Do we have options like encoding, barcode size, error correction like this https://zxing.appspot.com/generator?
Also is there any limitation of length on the input text that used to generate the qr code ?
Thanks,
Kent Liaw

ZXing.Net.Mobile really doesn't do anything above what the core library ZXing.Net offers for generating barcodes, so best to check out the upstream project for that info: http://zxingnet.codeplex.com/
It does seem like you can pass in a IDictionary<EncodeHintType, object> hints parameter which depending on the barcode format you use, can suggest things like error correction.
Best thing you can do is pull down the code and have a look :)


Add ppc64le

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Pending https://github.com/conda-forge/libjxl-split-feedstock/pull/29

@conda-forge-admin, please restart ci


fix: fake transition blocking events

Closes #11
I tested it locally and everything seems to work fine but please double check, i might be missing some use case. I changed registerTransitionEnd to return a function and invoked it after the reflow so that element.getAnimations could have the latest and correct value for every element. After that instead of registering the event i just wait for every css animation finished.
This ensure that if an elements doesn't change because already has the same style the getAnimations will not return an animation for that avoiding the the Promise.all at the end to be blocked.

Haven't checked the implementation, but it seems to make the theme and language switch not reactive

Haven't checked the implementation, but it seems to make the theme and language switch not reactive

For me it works for everything except Vue 🤷🏻♂
I will check what's going on

Oh wait...i just switched to main and it's not working there either so it must be something else...does it work with other languages @antfu ?

@antfu there was a missing dependency on the watch in the playground that was making the change fail for vite. I think i fixed it on the latest commit (not that versed in vue so i might have messed something)

@antfu any update on this?

@antfu sorry for pinging you again but i'll soon need to use this library and i would love for this to be fixed before. Can you take a look?


IntelliJ can now debug the Agent

I wanted a decent debugging setup, so I modified the agent to be able to be debugged by IntelliJ. This lets me set breakpoints on both sides of the protocol.
This was a fairly straightforward change, but I have only used it for a few hours, so there may be issues. However it is very useful to be able to debug the agent, so I am committing it now.
There are two modes of operation supported: Cody can spawn the agent or IntelliJ can spawn it. The latter allows Cody to talk to any remote agent.
I added detailed instructions on how to set this up in CONTRIBUTING.md.
Test plan
Locally/manually tested. This is a dev-only code path, and tricky to unit test.

See also the agent-side PR.


*.deb package is of bad quality

When loading flickrdownloadr_<IP_ADDRESS>-0_amd64.deb on ubuntu 14.10
The package is of bad quality
The installation of a package which violates the quality standards isn't allowed. This could cause serious problems on your computer. Please contact the person or organisation who provided this package file and include the details beneath.
Lintian check results for~/Downloads/flickrdownloadr_<IP_ADDRESS>-0_amd64.deb:
Skipping~/Downloads/flickrdownloadr_<IP_ADDRESS>-0_amd64.deb: syntax error at line 15: Cannot parse line "Depends : mono-complete, gtk-sharp3"
You are still able to 'ignore and install' after this message

Well, you can't do this using deb or rpm because they act at super-user space, and this is something user related.
What you can do instead is adding a bash script installed as /usr/bin/flickr-downloader such as
#!/bin/bash

if [ ! -f ~/.config/flickr-downloadr/token.repos ]; then
  mozroots --import --ask-remove
fi

/opt/flickrdownloadr/bin/flickr-downloadr.exe

That just performs this thing once per user.

Thanks @3v1n0 - I will include this into the launcher script

@3v1n0 - The launcher script has been tweaked to run the mozroots --import --sync on the first run.

I'm working on that

[ ] Folder structure not match with a standard Debian program. See [some information].(https://sites.google.com/site/vysakhp/articles/create-debian-packages-for-mono-applications).
[ ] The current package have too many dependencies (195 packages).
[ ] Gtk-sharp3 dependency is incorrect, Gtk-sharp2 is the one used by the installer
[ ] I think that must be updated to 3.12.0 to use cert-sync


Vue-Validator keeps old value when use with v-model

Here is my code snippet.
<input type="text" v-validate:startdate="['required2']" v-model="current.start_at">
<month-year-picker :model.sync="current.start_at"></month-year-picker>

For test, I cloned built-in required validator, and found issue on it.
I don't change the input text content directly by typing text, and instead of doing it, tried to change v-model by synchronization on month-year-picker.
But while the v-model current.start_at value keeps new one, the required validator works with old value, that is, while current.start_at gets "01/01/2014", the entered value to required validator is "01/01/201", so keeps old 201 instead of 2014.
Is it unresolved issue or there is any solution of it?
I hope to get professional help from the vue-validator creators.

Thank you for your reporting!!
Can you provide the minimum reproduction code with jsfiddle or other please?

Close (in-activity)


Customizable actions

Thank you for the great work. I'd like to request a feature where you can define buttons with scripts, similar as seen here:
https://macbury.github.io/SmartHouse/HomeAssistant/Vacuum/
This would allow for go to target buttons of clean zone buttons etc.

I agree that this would be nice. I'd prefer there to first be an option for each default button to be able to hide them.
I may get to this at some point, but I need to take stuff a bit slower for a bit. So if someone else beats me to it, that'd be great :)
TODO list for this:

A new show option for each button in https://github.com/TheLastProject/lovelace-valetudo-map-card/blob/2a01bfcb8174ec113e56ccd3fb68e0b5a416e329/valetudo-map-card.js#L310-L319
A check for show_whatever needs to be added to each button in https://github.com/TheLastProject/lovelace-valetudo-map-card/blob/2a01bfcb8174ec113e56ccd3fb68e0b5a416e329/valetudo-map-card.js#L501-L594
A loop for custom_buttons or however it'll be called around https://github.com/TheLastProject/lovelace-valetudo-map-card/blob/2a01bfcb8174ec113e56ccd3fb68e0b5a416e329/valetudo-map-card.js#L582


Add Figma to BabylonJS to Community Extensions

Was asked by @PirateJC to add my Figma Plugin to the Community Extensions.
First PR so please feel free to correct me if I have made any mistakes. I have tested the site on local and everything is working as intended.

Are there any other issues? I resolved all the open change requests.

@RaananW, it looks like all of the feedback has been addressed. I will let you proceed with final approvals.


running task every 1 minute

After a while is not working anymore... and nothing is changing the state

Just want to confirm this. Had created a cron with:
var task = cron.schedule('* 6,10,14,18,22 * * *', function() {
  // My Function
});

It runs every minute!

Hello @esuarezz what node version are you using?


WIP: Apollo Federation V2 merge

Some notes on the unsupported / untested features of Apollo Federation V2:
Selective directive imports is not easily configurable currently, and could be used in tests.
Renaming imports is not supported. Namespacing doesn't work either, pretty sure?

lets use in the apollo project the new ArgumentException.ThrowIf... API to validate arguments
I kicked out the ApolloTag directive as we will use the one that we have in the HotChocolate.Core
The whole type initialization at the beginning is wrong ... we should only register a type if its used ... we can move this code.
In the type interceptor we will at the end of the schema building inspect the schema and look which types are registered and fill out the imports dynamically.
We will stick with ExtendServiceType ... there is no need to change this.
The schema registration code apollo did in their implementation is wrong. We will split that up and move part of this into the type interceptor, we have some internal hooks to make this really nice.

I probably have more as we move along.

Superseded by #6837


SSH key still required without migrations

Run<EMAIL_ADDRESS>  with:
    GIGALIXIR_USERNAME: ***
    GIGALIXIR_PASSWORD: ***
    GIGALIXIR_APP: ***
    MIGRATIONS: false
    GIGALIXIR_CLEAN: false
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.8.1/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.8.1/x64/lib
Error: Input required and not supplied: SSH_PRIVATE_KEY

Oh, this is probably due to "false" being truthy

@LostKobrakai can you try out #39 and tell me if it works for you?

Seems to have worked: https://github.com/LostKobrakai/hex-bobs-list/runs/2637150498


Italics and bold do not work in body text

Changing the body text to italic (with \textit{}) or bold (with textbf{}) does not result in an italic or bold text in the pdf.
I was able to fix it by changing the \setmainfont in https://github.com/latextemplates/scientific-thesis-template/blob/master/config.tex#L88 from
  \setmainfont{TeXGyreTermes-Regular}

to
  \setmainfont{TeXGyreTermes-Regular}[
       BoldFont       = TeXGyreTermes-Bold ,
       ItalicFont     = TeXGyreTermes-Italic ,
       BoldItalicFont = TeXGyreTermes-BoldItalic ]

Thank you for reporting and proposing a fix.
Could you please file a pull request?

Well, it was not my intention to already close the issue (I am going to reopen it again, if possible):
I am only available to verify the fix in the current MacTex / TexLive 2019. I do not have MikTex, which seems to be the officially supported platform for this template.
I created the pull request: https://github.com/latextemplates/scientific-thesis-template/pull/111


[QA] Sort out test-acceptance-core-api in cs3org/reva

PR #5430 changed the "make" target test-acceptance-core-api to test-acceptance-from-core-api in ocis.
cs3org/reva edge and master branches also have test-acceptance-core-api in their Makefile but it does not seem to be used by the drone CI of cs3org/reva - the drone CI fetches all the test code from the ocis repo, and uses the ocis Makefile for running the tests.

see if there is anything else that uses the test-acceptance-* Makefile targets in cs3org/reva (maybe they are useful for developers who want to run tests, or?)

if they are not used, then remove them from Makefile to avoid future confusion.
if they are used, then get the names consistent with what is used in ocis.

Currently, there are two make targets in reva edge branch Makefile, test-acceptance-api and test-acceptance-core-api the latter one was added in PR https://github.com/cs3org/reva/pull/3591, but the CI in Reva use make targets from ocis and make targets of reva itself have not used in CI.
IMO those make targets test-acceptance-api and test-acceptance-core-api in the cs3org/reva Makefile can be removed. @phil-davis what do you say?
I think, refactoring of README.md in cs3org/reva, where core is cloned to run the acceptance test is needed.

what do you say?

yes, remove both Makefile targets, and refactor the README instructions. Do that in a PR to reva edge.
Then I will make the same changes to https://github.com/cs3org/reva/pull/3592 (reva master) and that will be merged "some day".

@grgprarup please also make a PR to reva master to remove this stuff. (similar to https://github.com/cs3org/reva/pull/3622 )
Then I can review it and get it merged.


Add a symbol generation file similar to Laravel's Laravel IDE Helper for better IDE support

We generate a flat list of extending classes, this seems like the best way to do it, though laravel's adds methods for some reason. I wrote out everything to put the methods in too, so we can switch easily if it's required in the future.
Not sure how to tie this into grunt due to unfamiliarity, whoever adds this to grunt, please link me to the commit!
Best wishes,
Korvin

I realized that the reason why they have the methods is because IDE's will warn that you're calling non-static methods statically. Methods are now included.

@KorvinSzanto  I don't understand why... IMHO it should be legal to have
namespace {
    class Area extends \Concrete\Core\Area\Area {
    }
}

and call id(new \Area())->showControls() or \Area::get().
Even further, there's some final method, so having
namespace {
    class Area extends \Concrete\Core\Area\Area {
        final public static function get($c, $arHandle) {
        }
    }
}

is wrong...
It's not a big issue, but in my IDE, when I want to see the definition of \Area::get, I'd much prefer to see it's real implementation and not the wrapper in __IDE_SYMBOLS__.php...

If you know how make that happen,  please do. We inherit documentation in
these, and specifically call the parent so that it's easy to click through
to the real implementation.
On Wed, Apr 22, 2015, 12:50 AM Michele Locati<EMAIL_ADDRESS>wrote:

I realized that the reason why they have the methods is because IDE's will
warn that you're calling non-static methods statically. Methods are now
included.
@KorvinSzanto https://github.com/KorvinSzanto I don't understand why...
IMHO it should be legal to have
namespace {    class Area extends \Concrete\Core\Area\Area {    }}
and call id(new \Area())->showControls() or \Area::get().
Even further, there's some final method, so having
namespace {    class Area extends \Concrete\Core\Area\Area {        final public static function get($c, $arHandle) {        }    }}
is wrong...
It's not a big issue, but in my IDE, when I want to see the definition of
\Area::get, I'd much prefer to see it's real implementation and not the
wrapper in IDE_SYMBOLS.php...
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5-5.7.0/pull/114#issuecomment-95064944
.

In order to have just the class aliases, like this
namespace {
    class Area extends \Concrete\Core\Area\Area {
    }
}

we could comment out these lines of the ClassSymbol class (or add a parameter to control if we want to add methods or not).
If it's ok I can create a PR for this...

We have things like facades and decorators that require the methods be
included and marked as static. It may be possible to omit the methods on
anything that isn't a facade or a decorator, but it would require more than
just turning it off for everything.
On Wed, Apr 22, 2015 at 9:12 AM Michele Locati<EMAIL_ADDRESS>wrote:

In order to have just the class aliases, like this
namespace {    class Area extends \Concrete\Core\Area\Area {    }}
we could comment out these lines
https://github.com/concrete5/concrete5-5.7.0/blob/<IP_ADDRESS>/web/concrete/src/Support/Symbol/ClassSymbol/ClassSymbol.php#L105-L118
of the ClassSymbol class (or add a parameter to control if we want to add
methods or not).
If it's ok I can create a PR for this...
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5-5.7.0/pull/114#issuecomment-95251758
.


upgrade to es 7

https://www.elastic.co/blog/elastic-stack-7-0-0-released
Focus:
zen2 Excellent !!!
Faster top k queries
Intervals queries

We're already in the process of porting zen2 into CrateDB and have already upgraded to Lucene 8 (for the faster top k queries). Both will be part of the 4.0 release so I'm going to close this already


P330 highlight missing

On the module function (second line of the page), it now accepts two parameters name and requires which are new.

Fixed, thanks


[ThinkPad x270] Fingerprint reader does not work

Hello everyone,
TL;DR The built-in fingerprint reader of the ThinkPad x270 seems to works with the driver python-validity package, but it's not available in nixpkgs yet.
I tried to list my devices with:
nicolas@nixos ~ % lsusb
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 013: ID 138a:0097 Validity Sensors, Inc.
Bus 001 Device 003: ID 04f2:b5ab Chicony Electronics Co., Ltd Integrated Camera
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

I understand that the fingerprint reader is Bus 001 Device 013: ID 138a:0097 Validity Sensors, Inc.
And when running fprint-enroll it fails with:
Impossible to enroll: GDBus.Error:net.reactivated.Fprint.Error.NoSuchDevice: No devices available

I tried different options but get the same result:

I read this thread: https://discourse.nixos.org/t/how-to-use-fingerprint-unlocking-how-to-set-up-fprintd-english/21901/2
I enabled fprint in my configuration.nix file and first tried vfs0090 driver then goodix driver but without success in both cases.
services.fprintd.enable = true;
services.fprintd.tod.enable = true;
services.fprintd.tod.driver = pkgs.libfprint-2-tod1-vfs0090;

I read this other thread (for ThinkPad T480): https://discourse.nixos.org/t/thinkpad-t480-fingerprint-reader-support/17448/3 which confirms that there were no python-validity package for NixOs.

I then write a nix script for python-validity to add to my configuration:
{ lib, python3Packages }:
with python3Packages;
buildPythonApplication {
  pname = "python-validity";
  version = "0.12";

  propagatedBuildInputs = [ cryptography pyusb pyyaml ];

  driverPath = /usr/share/python-validity/6_07f_lenovo_mis_qm.xpfwext;

  src = ./.;
}

but getting the same error as before.

I also looked on Validity official support, where someone already asked about a Nix driver in 2019 for my device: https://gitter.im/Validity90/Lobby?at=5caf58edf851ee043d9d85d3

Thanks if you have any idea to help me :)

I also opened an issue in NixOS/nixpkgs/ https://github.com/NixOS/nixpkgs/issues/207116

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/thinkpad-x270-fingerprint-reader-support/24177/1

I also need python-validity for my T480. I managed to create some nix packages based on the AUR packages and it works for me now.
In total, three packages are needed to get it to work: open-fprintd, fprintd-clients, and python-validity.
However, I am completely new to NixOS and Nix. Hence, I wrote those packages by copying and adapting existing Nix package definitions and applying some guesswork.
Hence, someone more experienced with this than me probably needs to review these packages. Also, I currently load them manually with callPackage from local files. I still need to create a nixpkgs fork or a flake git repository to load them properly.
In any case, to get it to work, put the files in the following repository in a folder packages relative to your configuration.nix: https://github.com/ahbnr/nixos-python-validity
You can then load them in your configuration.nix like this:
...
let
  # load custom packages for driving the fingerprint sensor.
  # This probably conflicts with the default fprintd service, so do not enable services.fprintd
  open-fprintd = (pkgs.callPackage ./packages/open-fprintd/default.nix {});
  fprintd-clients = (pkgs.callPackage ./packages/fprintd-clients/default.nix {});
  python-validity = (pkgs.callPackage ./packages/python-validity/default.nix {});
in
  ...
  environment.systemPackages = with pkgs; [
    ...
    open-fprintd
    fprintd-clients
    python-validity
    ...
  ];

  ...
  # Enable services from custom packages
  systemd.packages = [ open-fprintd python-validity ];
  systemd.services.open-fprintd.enable = true;
  systemd.services.python3-validity.enable = true;

  # enable fingerprint scanning for sudo
  security.pam.services.sudo.text = ''
    # Account management.
    account required pam_unix.so

    # Authentication management.
    auth sufficient pam_unix.so   likeauth try_first_pass nullok
    auth sufficient ${fprintd-clients}/lib/security/pam_fprintd.so
    auth required pam_deny.so

    # Password management.
    password sufficient pam_unix.so nullok sha512

    # Session management.
    session required pam_env.so conffile=/etc/pam/environment readenv=0
    session required pam_unix.so
  '';

  ...
}

Then run
nixos-rebuild test
systemctl start open-fprintd
systemctl start python-validity

Now, fprintd-enroll, fprintd-verify etc. should work. Furthermore, if you do not enter a password in your sudo prompt (just press enter), sudo will authenticate you with your fingerprint instead.
It is a bit worrying, that open-fprintd and fprintd-clients have not received a commit/update in two years. Hence, is there even a chance that they might become official packages?

@ahbnr it would be best to just add those python packages to nixpkgs. You already did most of the work already. Than the nixos hardware profile in this repository can be extended as well.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/thinkpad-x270-fingerprint-reader-support/24177/3


I ask for proofs of TENS OF THOUSENDS Kills in Ukraine

This is not Politicaly Content or Page here.
Please give me the proofs of tens of thousands of civilians kills in the Ukrainian, if you can't show me the proofs, plese remove this politicaly incorect banner from your readme page.

https://www.ohchr.org/en/news/2022/04/ukraine-civilian-casualty-update-11-april-2022
If this is not a reputable source enough for you, I recommend you to take it to them.

@xelaz Look at the media of any other country other than Russia (and a couple of it's friends) and ask yourself which is more probable

Almost every country in the world is lying to their citizens because they somehow hate Russia AND the somehow have decided to report the same things (and no, the media is not just taking the word of Ukraine, but are witnessing the stuff first hand)
The things that are reported are true and the attacking country is lying to their citizens.

 You must be smart enough to understand which is true if you really think about it.


[BUG] Need to repair presentation with line chart after download 2-type chart presentation

Category

[ ] Enhancement
[x] Bug
[ ] Question
[ ] Documentation gap/issue

Version
Version of the library: [<EMAIL_ADDRESS>]
Version of PowerPoint: [ latest ]
Expected / Desired Behavior / Question
Download 2 working presentation, PPTX files should be able to open on Microsoft PowerPoint
Observed Behavior

Download first presentation with 2-type charts (bar and line)
Second presentation with line chart needs to be repair

Steps to Reproduce

Open codepen https://codepen.io/extnow/pen/pobLePw
Save 2-type chart presentation (1)
Save line chart presentation (2)
Open line chart presentation

Thank you for your feedback!

This is failing because you are adding the option:
barDir: 'col',
This is not a valid configuration for a line chart. Remove this and your second chart will generate fine.

This is failing because you are adding the option:
barDir: 'col',
This is not a valid configuration for a line chart. Remove this and your second chart will generate fine.

Thanks @ReimaFrgos


bug: New large generated code blocks almost always have garbage at the end

Cody Version
5.5.12-nightly
IDE Information
IntelliJ IDEA 2022.1 (Community Edition)
Build #IC-221.5080.210, built on April 11, 2022
Runtime version: <IP_ADDRESS>+1-b2043.25 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.5
GC: G1 Young Generation, G1 Old Generation
Memory: 752M
Cores: 10
Non-Bundled Plugins:
com.sourcegraph.jetbrains (6.0-localbuild)
Kotlin: 221-1.6.20-release-285-IJ5080.210
Describe the bug

Open a Java project in IntelliJ -- I used Redisson
Open one of the Java files -- I used ClusterConnectionManager.java
Put the cursor the middle of the file somewhere, between methods (but inside the outer class)
Open Edit Code dialog, type "add merge sort function", and choose Edit Code button
Wait for the streaming generation to finish.
At the end there is a high chance that there will be extra characters (several lines worth), as shown in the screencap above.
If not, then Undo and try again, and now the chance of generating garbage will be near-100%.

Expected behavior
It's clear that we are streaming something that doesn't belong; it's not part of the code that the LLM generated.
The expected behavior would be that the generated code has no parse errors, at the very least.
Additional context
This is almost certainly a Streaming-related bug in FixupController and/or its interaction with the JB client.

Note that in the screenshot above, lines 350-356 are all garbage.


Fix OAuth1 redirect URL when server is behind a proxy.

This is the same fix for OAuth1AuthenticationService that was applied to OAuth2AuthenticationService in commit 385e7ca (SOCIAL-447). Specifically, if the Host header is present in the request, the X-Forwarded-Proto and X-Forwarded-Port headers are used to generate the callback URL. This is because a reverse proxy may receive a request via HTTPS on port 443, but an internal application server will see the request as coming via HTTP over port 8080 (for example), and that information will subsequently be used to generate an incorrect redirect URL.
The callback URL automatically generated by OAuth1AuthenticationService was:
http://example.com/auth/twitter
...Despite our site being HTTPS. Due to Twitter recently beginning to enforce strict callback URL matching, Twitter logins were broken because our Twitter application configuration defined our callback URL as:
https://example.com/auth/twitter
I am using this fork in my production application and Twitter logins are now working. My specific setup is Spring Boot 2.0.5 running on Jetty, hosted behind nginx. The site is HTTPS-only, and HTTPS connections are terminated at nginx and forwarded to our internal Jetty host over HTTP, which is running on port 8080.
In case it's helpful for anybody, my nginx configuration for the reverse proxy section is as follows:
proxy_set_header Host $host;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Port $server_port;
proxy_set_header X-Forwarded-Proto $scheme;
proxy_set_header X-Real-IP $remote_addr;

Tagging @jaffadog because he was the author of the original OAuth2AuthenticationService fix, in case he has any interest in following this or providing any feedback.
Tagging @codeconsole because he has an open pull request (#259) that alters the corresponding logic in OAuth2AuthenticationService. You may want to include the same change in OAuth1AuthenticationService if/when this is merged.

@brendanjbaker Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@brendanjbaker Thank you for signing the Contributor License Agreement!

Thanks for tagging me, this really seems like it should be moved into AbstractSocialAuthenticationService.java to prevent code duplication (especially considering the original solution doesn't work in all environments).  #259 now includes your fix.

@codeconsole I agree fully. Since #259 has the same effect (and fix) as this, and since the other change included in #259 seems quite agreeable, I'm going to close this pull request.
@habuma -- Can you please review and/or approve/merge #259? Than you!


ENOENT: no such file or directory

Hi, I'm creating a docker-compose.yml file but when I use the compose configuration from the doc, I have this error message:

mongo-gui_1        | [Error: ENOENT: no such file or directory, stat '/app../public/index.html'] {
mongo-gui_1        |   errno: -2,
mongo-gui_1        |   code: 'ENOENT',
mongo-gui_1        |   syscall: 'stat',
mongo-gui_1        |   path: '/app../public/index.html',
mongo-gui_1        |   expose: false,
mongo-gui_1        |   statusCode: 404,
mongo-gui_1        |   status: 404
mongo-gui_1        | }

What's going on? What can I do to fix it?

It seems like there is a bug in server.js in this line app.get('/', (req, res) => res.sendFile(__dirname + '../public/index.html'));.  This has been fixed in the latest commit but I believe it hasn't build the latest docker image

I'm going to rebuild the docker image myself with the latest code to see if it fixed

After rebuilding the docker image with the latest commit, it works :)

not fixed, where new images on dockerhub?


Fix no data on VM Metrics

Problem:

No data on VM Metrics.
Solution:

Fix the changed metrics name.
Related Issue:
https://github.com/harvester/harvester/issues/5442
https://github.com/harvester/harvester/issues/5439
Test plan:

Per issue description.

@mergify backport v1.2

@mergify backport v1.3


Getting warning on python 3.10.4

This is due to some changes in python 3.10 https://docs.python.org/3/library/importlib.html#importlib.abc.MetaPathFinder

This is triggered by the bottle.ext module, which is actually a virtual module with its own loader that allows to import bottle_whatever as bottle.ext.whatever. Probably best to deprecate this high-magic-low-benefit functionality.


client_side_validations will break down if there are two forms in one view

The second one works well but the first one broke down

I'm guessing you're using the same object type for both forms. If that is the case you'll have to use the :as option on one of the forms to override the ID that it is given.

your guess is right.
according to what you said, now it works.
thank you @bcardarella

Glad I could help! :)

I was hit by the same basic problem but in my case I'm using Devise and had combined the sign in and sign up forms. In this case devise expects (AFAIK) both forms to be declared as:
form_for(resource, :as => resource_name....

This means the suggestion won't work.
Looking at the client_side_validations code the workaround is simple enough -You just need to set the ids of the fields as this causes overrides the specification from the :as option. Hopefully this helps anyone else who encounters this edge case...

@skeary can you provide some code to demonstrate your workaround? This should probably be in the wiki.

Inspired by what @skeary said, I tried to set different id to the forms, it also works
Here is some code snippet:
= form_for @user :validate => true, :html => { :method => :put, :id => 'one-form' } do |f|
= f.text_field :name
…
= form_for @user :validate => true, :html => { :method => :put, :id => 'another-form' } do |f|
= f.text_field :email
…


"Return value contains a constant value" spammed many times

Aim
global COUNT = 400;

fn main() -> pub BoundedVec<[Field; 1], COUNT> {
    let mut result = BoundedVec::new();
    let mut x = 0;

    for _ in 0..COUNT {
        if x < 1 {
            result.push([0]);
        }
    }

    result
}

Expected Behavior
A "return value contains constant value warning"
Bug
Many repeated warnings are issued, presumably one for each element of the returned array.
To Reproduce

Project Impact
None
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
No response
Installation Method
None
Nargo Version
No response
NoirJS Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Agreed we should deduplicate these for each Noir program. This issue also applies to other instances such as verify_proof, etc.

https://github.com/noir-lang/noir/issues/4365

Closed in #2936

Closed in https://github.com/noir-lang/noir/issues/2936
That issue seems unrelated. Typo?

Yep, I've fixed the link above.


Added distroless by GoogleContainerTools

https://github.com/GoogleContainerTools/distroless
Language focused docker images, minus the operating system.
"Distroless" images contain only your application and its runtime dependencies. They do not contain package managers, shells or any other programs you would expect to find in a standard Linux distribution.

So this line fails: https://github.com/veggiemonk/awesome-docker/blob/master/.travis.yml#L24
$ TOKEN=$GITHUB_TOKEN node buildMetadata.js && ./push.sh
error: pathspec 'master' did not match any file(s) known to git.
The command "TOKEN=$GITHUB_TOKEN node buildMetadata.js && ./push.sh" exited with 1.

been a while since i debugged travis builds...


style: fix some typos in comments, messages and documentation

All typos were found by codespell.
Pull Request Checklist
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[ ] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Fixed some typos which were found by codespell.

Changed

Fixed some typos.

Fixed

typos

What was wrong or missing?


Fix releaser workflows

@blink1073 this is a bit cheeky — I can't figure out why the releaser workflow isn't working. The traceback indicates that GITHUB_TOKEN isn't set. Do you have any ideas?
Run jupyter-server/jupyter_releaser/.github/actions/prep-release@v2
Run # Install Jupyter Releaser from git unless we are testing Releaser itself
+ command -v jupyter-releaser
+ pip install -q git+https://github.com/jupyter-server/jupyter_releaser.git@v2
Run export GITHUB_ACCESS_TOKEN= ***
+ export GITHUB_ACCESS_TOKEN= ***
+ GITHUB_ACCESS_TOKEN=
+ export GITHUB_ACTOR=agoose77
+ GITHUB_ACTOR=agoose77
+ export RH_REPOSITORY=
+ RH_REPOSITORY=
+ '[' '!' -z ']'
+ export RH_VERSION_SPEC=next
+ RH_VERSION_SPEC=next
+ export RH_POST_VERSION_SPEC=
+ RH_POST_VERSION_SPEC=
+ export RH_DRY_RUN=false
+ RH_DRY_RUN=false
+ export RH_SINCE=
+ RH_SINCE=
+ export RH_SINCE_LAST_STABLE=true
+ RH_SINCE_LAST_STABLE=true
+ python -m jupyter_releaser.actions.prep_release
Getting GitHub connection for agoose77/hatch-nodejs-version
/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/site-packages/ghapi/core.py:102: UserWarning: Neither GITHUB_TOKEN nor GITHUB_JWT_TOKEN found: running as unauthenticated
  else: warn('Neither GITHUB_TOKEN nor GITHUB_JWT_TOKEN found: running as unauthenticated')
Getting permission level for agoose77
HTTP Error 401: Unauthorized
====Error Body====
{
  "message": "Requires authentication",
  "documentation_url": "https://docs.github.com/rest/collaborators/collaborators#get-repository-permissions-for-a-user"
}

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/site-packages/jupyter_releaser/actions/prep_release.py", line 9, in <module>
    setup(False)
  File "/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/site-packages/jupyter_releaser/actions/common.py", line 19, in setup
    return prepare_environment(fetch_draft_release=fetch_draft_release)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/site-packages/jupyter_releaser/util.py", line 619, in prepare_environment
    raise RuntimeError(msg) from None
RuntimeError: Could not get user permission level, assuming user was not admin!

Did you add an ADMIN_GITHUB_TOKEN secret on this repo?

Yes, with the same scopes as that used in jupyterlab-myst (though it's not the same token).

I'm stumped, I don't understand why it looks like it sets the variable, but then the variable is empty.
Here's what it looks like in server:
+ pip install -q git+https://github.com/jupyter-server/jupyter_releaser.git@v2
Run export GITHUB_ACCESS_TOKEN=***
+ export GITHUB_ACCESS_TOKEN=***
+ GITHUB_ACCESS_TOKEN=***

I was stuck too. I recreated the token and re-added the secret, which seems to work. I'm not really sure what happened. Perhaps the token was empty in the secrets ledger, but I really doubt that.
Thanks for taking a look!


Create Design for Peer Discussion page

A peer discussion page is needed where

Where peers can talk with each other in form of blogs, resolve each others doubts etc
They can give course feedback
The healthy competition like solving problems etc could be made here
People can share notes, papers, their thoughts and forums in the form of threads

@Abhishek-kumar09 I want to work on this page. Please assign to me

@Abhishek-kumar09 I want to work on this page. Please assign to me

Hey, @poojadas07  if you need any sort of help from my side designing the Page in Figma or adobe xd just ping me out.
always ready help :p

@Abhishek-kumar09  i wanted ask is this not the same think i had asked in issue  #132 and you assigned it to me i believe the idea is the please tell me if what i am thinking is wrong

@san0808
I think that was a chat section you were talking about, and that could be made like a chat interface that will be more personalised.
This page more resembles to blogging page, forums and course discussion page etc.

@Abhishek-kumar09 I want to work on this page. Please assign to me

Hey, @poojadas07 if you need any sort of help from my side in designing the Page in Figma or adobe xd just ping me out.
always ready help :p

Okay


Ensure help can be printed at any time

We want to able to access the help at any point, so defer the nested VM check until after the flag parsing.
Today, ciao-down can print the nested VM error and exit before it has the change to print the help.

Coverage increased (+0.004%) to 65.625% when pulling f7619bba3956c9a40e07d8994cf1f491647598d1 on dlespiau:20170713-help-first into bff6ec8638851ea16e29661820962da020971e4a on 01org:master.


Make Index lens a tuple struct

You have made inner data public.
Druid is still "work in progress" and verbose way of doing things wanted.

I have doubts this pull request makes any sense.

I don't see any reason not to make the index public. (Same in lens::Constant)
But people(like me) still want to build stuff with it; reducing verbosity doesn't hurt.

If it were already a tuple struct I think that would be fine, but I don't see any advantage to changing it now. I get that Index(5) might feel better than Index::new(5) but I also think it's less idiomatic, and I don't expect this to actually matter much in practice.
in any case thanks for the PR, I do appreciate the interest in small improvements, I'm just not quite sold in this case. 😄
I'm going to close this; if you think I've missed something and there's a strong rationale behind this feel free to reopen.


CR: Clear Queue

Hi Supergiovane
Thanks for your great Node and it works like a charm.
I have just on little thing that would be very nice. I use your Node for sending my own Files to Sonos in a Alert System. Now it would be a greate Feature when the node has a switch for clearing the queue.
I hope you can spend time for this Change.
I look forward,
Cheers,
Chris

Hi Chris,
Thanks.
You need a msg input to clear the current TTS queue being played from the node and to force the node to stop the current speak, is that correct?

Hi Chris,
Thanks.
You need a msg input to clear the current TTS queue being played from the node and to force the node to stop the current speak, is that correct?

yes, that would be exactly what I need.
Thanks in advance

yes, that would be exactly what I need.
Thanks in advance

Hi Chris,
V. 1.0.13 is out with your request. Please see the README on how to use it.
Please star my node if you like it!
Thanks.

Hi Chris,
V. 1.0.13 is out with your request. Please see the README on how to use it.
Please star my node if you like it!
Thanks.

Hi Supergiovane,
Your extension of the node works perfectly.
I have only one question, is there a possibility to stop the node while it is playing something?
Thanks for your great support,
Chris

Hi Supergiovane,
Your extension of the node works perfectly.
I have only one question, is there a possibility to stop the node while it is playing something?
Thanks for your great support,
Chris

Hi Chris
Yes, not now, but i'll add this in the next version.
In the meantime, you can send this to the node:
// Stop msg.payload=""; msg.priority = true; msg.nohailing=true; return msg;

Hi Chris
Yes, not now, but i'll add this in the next version.
In the meantime, you can send this to the node:
// Stop msg.payload=""; msg.priority = true; msg.nohailing=true; return msg;

Hi Chris
I've pushed out the V. 1.0.14 with msg.stop = true command.
You should see it as soon as node-red palette manager has updated the package.

Hi Chris
I've pushed out the V. 1.0.14 with msg.stop = true command.
You should see it as soon as node-red palette manager has updated the package.


update configuration file for vcftools_filter clarity

The current nextflow.config lists filter criteria for filterGatkGenotypes.py under the param vcftools_filter. It would be good to have a separate gatkgenotypes_filter so that advanced users know which syntax to use for filtering.
// Mycosnp options
save_reference             = true
save_alignment             = true
sample_ploidy              = 1
coverage                   = 70 // Desired sample coverage used in seqtk_sample
rate                       = ''
gvcfs_filter               = 'QD < 2.0 || FS > 60.0 || MQ < 40.0 || DP < 10'
vcftools_filter            = '--min_GQ "50" --keep_GQ_0_refs --min_percent_alt_in_AD "0.8" --min_total_DP "10" --keep_all_ref'
max_amb_samples            = 10000000
max_perc_amb_samples       = 10
publish_dir_mode           = 'copy'
rapidnj                    = true
fasttree                   = true
iqtree                     = false
raxmlng                    = false
save_debug                 = false
mask                       = true
tmpdir                     = "$projectDir/tmp"
skip_samples               = ""
skip_samples_file          = null
skip_combined_analysis     = false
skip_phylogeny             = false

This would result to two params options a vcftools_filter and a gatkgenotypes_filter and provide a better insight into how the filter criteria is being applied within the workflow.

After digging a bit deeper into the workflow I see vcftools is only being used through the filtergatkgenotypes script. In this case it would make the most since to just change vcftools_filter to gatkgenotypes_filter.


Next-Generation UI/UX Enhancement for Guardian Platform with MRV Focus

Problem description
The Guardian platform has become a critical component for organizations managing policies, assets, and trust chains across various industries, with an emphasis on Monitoring, Reporting, and Verification (MRV) processes. As adoption grows, user expectations around ease of use, visual appeal, and overall user experience continue to increase. To maintain our competitive edge and to cater to a broader range of users, from policy authors to MRV specialists, system administrators, and asset managers, we need to deliver a next-generation user interface (UI) and user experience (UX) that not only meets but exceeds these expectations while highlighting the MRV capabilities of the platform.
Requirements

Responsive and modern UI design: The new UI should be visually appealing, easy to navigate, and optimized for use on various devices, including desktops, tablets, and smartphones. It should incorporate the latest UI design trends and best practices, ensuring a professional and polished look.
Intuitive navigation and user flows: The overall user experience should be designed with usability in mind, minimizing the learning curve for new users while maintaining efficiency and productivity for experienced users. This includes clear navigation, consistent layouts, and context-sensitive help.
Enhanced policy and asset visualization: Implement advanced visualization techniques for policies, trust chains, and assets to enable users to better understand their data and the relationships between various components. This may include interactive flow diagrams, hierarchical views, and geographic representations.
User feedback and continuous improvement: Implement a user feedback system to gather insights and suggestions from users to drive ongoing improvements to the UI/UX.

Definition of done

Functionality is implemented as per requirements
Documentation updated accordingly

Acceptance criteria

Commercial quality responsive UI

@dubgeis Sorry for my delayed response. I do indeed have some ideas, but they are probably best discussed during a Zoom session. Let's set up a call.


SAK-45628 site info > don't remove stealthed lti tools when manage tools list is edited

(cherry picked from commit 574b2a1e08f46b5082495b2523e4b931297f7e53)

Thanks, Sam!


Push notification null pointer exception (when app is closed)

Hi, first thank you for your project. Great work.
Issue:
When app is closed (when you swipe it to the right) and send message to it - instead push notification it gives me following exception:

java.lang.RuntimeException: Unable to start receiver co.chatsdk.firebase.push.DefaultBroadcastReceiver: java.lang.NullPointerException: Attempt to read from field 'boolean co.chatsdk.core.session.Configuration.inboundPushHandlingEnabled' on a null object reference
    at android.app.ActivityThread.handleReceiver(ActivityThread.java:2712)
    at android.app.ActivityThread.access$1700(ActivityThread.java:162)
    at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1425)
    at android.os.Handler.dispatchMessage(Handler.java:102)
    at android.os.Looper.loop(Looper.java:135)
    at android.app.ActivityThread.main(ActivityThread.java:5421)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:914)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:707)
 Caused by: java.lang.NullPointerException: Attempt to read from field 'boolean co.chatsdk.core.session.Configuration.inboundPushHandlingEnabled' on a null object reference
    at co.chatsdk.firebase.push.DefaultBroadcastReceiver.onReceive(DefaultBroadcastReceiver.java:24)
    at android.app.ActivityThread.handleReceiver(ActivityThread.java:2700)

It is this code that throws an exception (DefaultBroadcastReceiver.java):
        if(!ChatSDK.config().inboundPushHandlingEnabled) {             return;         }
It looks like when app is closed ChatSDK.config() returns null.
Thanks in advance for any help.

Got the same problem.
Had to add initializing on the OnCreate (DefaultBroadcastReceiver.java)
Configuration.Builder builder = new Configuration.Builder(context);         builder.firebaseRootPath("testdb");         try {             ChatSDK.initialize(builder.build(), new BaseInterfaceAdapter(context), new FirebaseNetworkAdapter());         }         catch (ChatSDKException e) {         }
before this line :
if(!ChatSDK.config().inboundPushHandlingEnabled) { return; }

I can confirm that the project doesn't compile when minify is enabled.

Currently quick reply isn't supported by the SDK. I've added this as a feature request.

ChatSDK implementation: 'sdk.chat:firebase-app:5.0.0rc21'
The above error still occures to me:
Fatal Exception: java.lang.RuntimeException
Unable to start receiver sdk.chat.core.push.DefaultBroadcastReceiver: java.lang.IllegalArgumentException: the bind value at index 1 is null

Caused by java.lang.IllegalArgumentException
the bind value at index 1 is null
android.database.sqlite.SQLiteProgram.bindString (SQLiteProgram.java:183)
android.database.sqlite.SQLiteDatabase.rawQuery (SQLiteDatabase.java:1906)
**org.greenrobot.greendao.database.StandardDatabase.rawQuery (StandardDatabase.java:32)**

Any way to fix this?

ChatSDK implementation: 'sdk.chat:firebase-app:5.0.0rc21'
The above error still occures to me:
Fatal Exception: java.lang.RuntimeException
Unable to start receiver sdk.chat.core.push.DefaultBroadcastReceiver: java.lang.IllegalArgumentException: the bind value at index 1 is null

Caused by java.lang.IllegalArgumentException
the bind value at index 1 is null
android.database.sqlite.SQLiteProgram.bindString (SQLiteProgram.java:183)
android.database.sqlite.SQLiteDatabase.rawQuery (SQLiteDatabase.java:1906)
**org.greenrobot.greendao.database.StandardDatabase.rawQuery (StandardDatabase.java:32)**

Any way to fix this?

Can you update to the latest version 5.1.1 and tell me if it still happens? We don't support old versions.

Can you update to the latest version 5.1.1 and tell me if it still happens? We don't support old versions.

Yeah sure, I didn't see any newer version on your github page
It will take me some time to test it out
But one more thing, after updating to sdk.chat:firebase-app:5.1.1
I am no longer able to look into your library files
Getting this message: decompiled .class file bytecode version 52.0 (java 8)
And when I try to download sources i get the following error:
* What went wrong:
Execution failed for task ':app:DownloadSources'.
> Could not resolve all files for configuration ':app:downloadSources_e8d9daa6-8f02-445c-88b5-637a14d4af98'.
   > Could not find sdk.chat:core:5.1.1@aar.
     Required by:
         project :app

Doo you have any suggestions on how to solve this? Everything I tried failed.

Yeah sure, I didn't see any newer version on your github page
It will take me some time to test it out
But one more thing, after updating to sdk.chat:firebase-app:5.1.1
I am no longer able to look into your library files
Getting this message: decompiled .class file bytecode version 52.0 (java 8)
And when I try to download sources i get the following error:
* What went wrong:
Execution failed for task ':app:DownloadSources'.
> Could not resolve all files for configuration ':app:downloadSources_e8d9daa6-8f02-445c-88b5-637a14d4af98'.
   > Could not find sdk.chat:core:5.1.1@aar.
     Required by:
         project :app

Doo you have any suggestions on how to solve this? Everything I tried failed.

Hi, @bensmiley
Just to get back to you, after extensive testing I can say that upgrading to the newer version of the library did fix the problem, thanks!

Hi, @bensmiley
Just to get back to you, after extensive testing I can say that upgrading to the newer version of the library did fix the problem, thanks!