Exposed method to control the readonly query parameter

Summary
This PR is meant to improve the control over the readonly option sent by the client to the CH database. Right now, it is defined by some methods like Query::do_execute, but most of the public methods like Query::fetch, Query::fetch_one, or Query::fetch_all don’t have control over this option. This makes it impossible to query the database with queries that contain a lot of columns and require temporary table creation on the database side, which results in the following error:

Code: 164. DB::Exception: db_user: Cannot execute query in readonly mode. (READONLY) (version <IP_ADDRESS>12 (official build))

By adding the changes in this PR, we will gain control over this option, making it possible to request data that requires the creation of temporary tables by CH.
Checklist

[x] A human-readable description of the changes was provided so that we can include it in CHANGELOG later

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

CC @loyd @serprex have a look please ^

Hey @slvrtrn. Sure, I can check/extend tests and docs. But I’m just thinking if overwriting with 0 is the proper way to go, I mean including the readonly option all the time, since afaik it can be configured on the DB side. Basically, the user can be configured with the required access level. It will just limit the client and require handling that on both sides, which is a bit redundant and needs additional maintenance. But, as long as I can perform SELECT queries without encountering confusing exceptions, I'm okay with that :)

would it make more sense just to make it so that .with_option("readonly," "0") overrides the default hardcoded read-only param?

Based on this code with hardcoded flag and this PR, I assume it will not overwrite it by default if it is already present in the options.

Perhaps we can just remove readonly setting, which is enforced by default; IIRC, no other language client does that. Maybe this approach will be even more flexible (readonly can still be attached using with_option) https://github.com/ClickHouse/clickhouse-rs/pull/184

Sure, this will work as well. Thanks.


Aesthetic issue when announcing to player when they've crossed over the border

Hey TehNut,  The mod works great as server side only; the client was bungied back even when teleporting without any glitches I noticed.  The only issue is that the UUID gets announced directly after the message. See here

Update to 1.0.0-2. I fixed that at b77094fdea7e58796dcefde910f4dc9537b786a2


Windows support

QrSync should be tested and adapted to run on windows.
The main blockers could be:

Interface selection and IP address discovery
QR rendering

This commit https://github.com/crisidev/qrsync/commit/81be2b178adfa5170ab90b46744f5c93fba53b99 allows cross compilation to succeed. Testing is still missing.


启动服务后报无法更新组件错误

devScripts.js:5836 Warning: Cannot update a component (i) while rendering a different component (Layout). To locate the bad setState() call inside Layout, follow the stack trace as described in https://fb.me/setstate-in-render
in Layout (created by Context.Consumer)
in withRouter(Layout) (created by LoadableComponent)
in LoadableComponent (created by Context.Consumer)
in Route (created by RouterComponent)
in Switch (created by RouterComponent)
in Router (created by RouterComponent)
in RouterComponent
in Provider (created by _DvaContainer)
in _DvaContainer
in WEBPACK_DEFAULT_EXPORT
in WEBPACK_DEFAULT_EXPORT

warning只是警告而已，不是错误。
听作者说，有大版本要升级，期待下吧

我也遇到这个问题，期待尽快吧

+1


MacOS Retina graphics not working

Version of Archi, Operating System
4.3 beta 1
Expected Behaviour
Retina Graphics
Actual Behaviour
Blurred text, fuzzy fonts - was the same in version 4.2
Steps to Reproduce the Behaviour
Unknown - just running this on MacOS.

This is the same for version 4.2 you say? I've not had any other reports from Mac users about this.
Can you provide a screenshot?

Something to try I found from here:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=382972#c4

Right-click the Archi.app
"Show Package Contents"
Dig into Contents folder
Edit "Info.plist" and add this:

<key>NSHighResolutionCapable</key>
<true/>

May need to log out and back in again.

Attached is a an Info.plist file that you could try. It's in a zip file.
Info.zip

Hi yes - I have the same on 4.2.

I tried the Info.plist fixes, no luck. I also tried: https://apple.stackexchange.com/questions/75891/how-can-i-completely-remove-apples-java-6-and-make-oracles-java-7-default-for
(in my case 1.8)
Somewhere there's a cached setting or something.

Setting the Java won't help as Archi has its own bundled Java in the App.
When you replace the Info.plist you might try renaming things, or rebooting to clear the cache (ckec the thread at https://bugs.eclipse.org/bugs/show_bug.cgi?id=382972). Right-click on Archi app and "Get Info" should show "Open in Low Resolution" as not checked.
Eclipse itself should be similar. You could download it and see if it works in hi-res mode:
https://www.eclipse.org/downloads/download.php?file=/eclipse/downloads/drops4/R-4.7.3a-201803300640/eclipse-SDK-4.7.3a-macosx-cocoa-x86_64.dmg

One thing you could also try in Archi is "Help - > Reset To Defaults..." and restart Archi. This will clear Archi layout cache and visual settings.

@Phillipus - ah - it is checked (Open in Low Resolutions): but greyed out and I can't untick it even with privileges.
But making a copy of the App (so that MacOS can notice the change) worked! Thanks so much, it looks so much better.

Ah, that's great news. Thanks for reporting this.
Some observations:

I don't have a retina Mac so I rely on users who do to report this type of issue, so strange that no-one else has noticed this. Thanks for reporting this. :-)
The updated Info.plist will be included in Archi 4.3 beta 2
I'm now wondering how retina Mac users who update to Archi 4.3 can force the change. I found this https://apple.stackexchange.com/questions/162141/how-do-i-properly-force-osx-to-reload-the-info-plist-file-of-an-application - moving the app to a different location seems to do it.

The updated Info.plist is now present in Archi 4.3 beta 2.
Forcing the cache to update seems to involve moving the Archi.app from one location to another.

@stephanbuys Are you seeing the issue reported in #396 ?


State and test support for big-endian machines

[ ] Clarify which backends of mlkem-native work on big-endian systems.
[ ] Add a CI test for each.

It looks like

The C code works on big-endian targets
The AArch64 assembly does not work on big endian Arm systems, but it also isn't enabled on those (because we the decisive architecture flag is __ARCH64_EL__)


We should include platforms information for each controls

Can we have platforms support for every controls? so that users know what this control will be available on, for example, tooltips and menubar are only available on windows

Hi @Strypper
Fundamentally everything is available on every platform, unless the doc specifically calls out that it's not available on a platform. When it's not available on a platform it's generally because the platform doesn't support it.
As for your suggestion, I like it. It's actually something that's being worked on for the API docs, so that you'll see which platforms the type is available on. I suspect it'll be a while before it's rolled out though.
Thanks again for your suggestion.


Ability to click on a variation and edit that in a comment.

This way you won't have to read the letters and try to recreate it, you can just click it and press edit or something.

That's a good idea; I've been struck for the need for something like that, too, especially when responding to a long sequence someone else has suggested. I'm still not sure exactly how it would work, but I'll definitely think about it.


Banish unsafePerformIO in Cardano.CLI.Tx.Generation

There are a few uses of unsafePerformIO in the Cardano.CLI.Tx.Generation module. The purpose of this PR is to remove them.

@mrBliss I've removed unsafePerformIO, please see changes in Cardano.CLI.Tx.Generation.

If it doesn't have to be a mutable var at all and can just pass ordinary values as params, that's even better and worth doing.

Currently I kept only one single commit in this PR (which actually removed unsafePerformIO), because https://github.com/input-output-hk/cardano-node/issues/359 is exactly about that.
Converting availableFunds from TVar to a pure value is the next step, and I'll open a new issue for it.

bors r+

bors r+


Crashes  since the last fix "Ghidra loses '#' symbol in display section "

Since the last fix
https://github.com/esaulenka/Ghidra_C166/commit/c64dd2a5e5b6132ba6f38d0b1293e7cfe684fb2a
Ghidra crashes with a null pointer exception.
I disassemble the same binary file with the same settings as last week.
Before I have never seen that crash.
Now it happens always.
How to reproduce:
I create a new project.
Select C166
Set base address = C00000
Select an area with the mouse which I want to disassemble
right click the selected area
click "Disassemble" in the menu
The first bytes are disassembled fine until Ghidra finds invalid bytes in a section which does not contain valid code.
Before your last fix Ghidra simply skipped that invalid bytes and continued later on where it found valid code.
Now it crashes here:
It seems to have a problem with the byte CC
      c081bc cc              ??         CCh
      c081bd 76              ??         76h    v
      c081be c0              ??         C0h
      c081bf 00              ??         00h
      c081c0 3e              ??         3Eh    >
      c081c1 77              ??         77h    w
      c081c2 c0              ??         C0h
      c081c3 00              ??         00h
      c081c4 42              ??         42h    B
      c081c5 77              ??         77h    w
      c081c6 c0              ??         C0h
      c081c7 00              ??         00h
      c081c8 aa              ??         AAh
      c081c9 77              ??         77h    w
      c081ca c0              ??         C0h
      c081cb 00              ??         00h
      c081cc 74              ??         74h    t
      c081cd 77              ??         77h    w
      c081ce c0              ??         C0h
      c081cf 00              ??         00h
      c081d0 70              ??         70h    p
      c081d1 77              ??         77h    w
      c081d2 c0              ??         C0h
      c081d3 00              ??         00h
      c081d4 bc              ??         BCh
      c081d5 77              ??         77h    w
      c081d6 c0              ??         C0h
      c081d7 00              ??         00h
      c081d8 c0              ??         C0h
      c081d9 77              ??         77h    w
      c081da c0              ??         C0h
      c081db 00              ??         00h
      c081dc c6              ??         C6h
      c081dd 77              ??         77h    w
      c081de c0              ??         C0h
      c081df 00              ??         00h
      c081e0 8e              ??         8Eh
      c081e1 77              ??         77h    w
      c081e2 c0              ??         C0h
      c081e3 00              ??         00h
      c081e4 8a              ??         8Ah

If you cannot reproduce the problem I can send you the binary file.

yes it would be better, if you share your binary.
I am tested it only with one binary, and it works ok.

yes it would be better, if you share your binary.
I am tested it only with one binary, and it works ok.

I uploaded the file here:
https://netcult.ch/elmue/FlashMemory.bin
When you import the file set the base address C00000
The flash memory has code sections and data sections.
The code section which is interesting for me and which causes the crash is from C08000 until the end (C1FFFF)
Mark that section and click "Disassemble" and you will see.

I uploaded the file here:
https://netcult.ch/elmue/FlashMemory.bin
When you import the file set the base address C00000
The flash memory has code sections and data sections.
The code section which is interesting for me and which causes the crash is from C08000 until the end (C1FFFF)
Mark that section and click "Disassemble" and you will see.

No  answer anymore ?
Could you reproduce the problem?

No  answer anymore ?
Could you reproduce the problem?

Well, at least i found a workaround:
Instead of selecting "Disassemble" in the menu, use "Disassemble Static"
I have no idea what is the difference but it works.


Throw an error when an invalid transfer.id is provided for accounting/transfers.ts#createTransfers()

Maybe we change CreateTransferOptions to be a union type for creating new, committed, or voided transfers:
https://github.com/interledger/rafiki/blob/dc909a6e73aec9064fd144b0a79fa74aceb07c88/packages/backend/src/accounting/transfers.ts#L19-L31

actually, what is an invalid transfer.id?
This currently throws
https://github.com/interledger/rafiki/blob/dd07d2ff6f157c8b3fd671e9d74f7ea86e91ee5b/packages/backend/src/accounting/transfers.ts#L57
https://github.com/interledger/rafiki/blob/dd07d2ff6f157c8b3fd671e9d74f7ea86e91ee5b/packages/backend/src/accounting/utils.ts#L5-L13

@wilsonianb @sabineschaller I had a look at the logic.
It has been some time since I looked at the code. :-) With regards to the toTigerbeetleId function, I believe we are covered and this issue may be closed.
I see the following PR ensures that the id is valid:
https://github.com/interledger/rafiki/commit/2f8e82a1c91b0d2c4ff554b0851947351d6afa03
But I remember another issue where in some cases the account id was a number and not bigint.
If I remember correctly, it was during an internal lookup where the type wasn't always a bigint.
Sorry, I will have a look at where I have noted it, I remember discussing it with you @wilsonianb , do you perhaps recall the conversation?

But I remember another issue where in some cases the account id was a number and not bigint.

@koekiebox That rings a bell. I'll take another look. Thanks

I still haven't found an error case not covered by toTigerbeetleId
Closing for now


Readme

@CodiumAI-Agent /review

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
1, because the PR is straightforward and involves only updates to the Readme.md file with no complex code changes.

 🧪 Relevant tests
No

 🔍 Possible issues

Typographical Errors: There are several typographical errors and grammatical mistakes that could affect the professionalism of the document.

Ambiguity in Language: Some sentences are ambiguous and may confuse the reader.
 🔒 Security concerns
No

 Code feedback:
relevant fileReadme.md
suggestion

Correct the spelling of "consume" to "consuming" to improve grammatical accuracy. [important]

relevant lineI am excited to present an innovative idea that has a chance to completely transform your consume drinks and cocktails.
relevant fileReadme.md
suggestion

Replace "Weather" with "Whether" to correct the word usage for the intended meaning. [important]

relevant lineFinding the perfect cocktail can often be difficult. Weather it's for a party or a casual evening at home.
relevant fileReadme.md
suggestion

Change "Acient" to "Ancient" to correct the spelling error. [important]

relevant lineIt's a destination where you find all types of drinks also Mocktails (for non-alcoholic person). We have a collection of all types of Cocktail from Acient to Modern.
relevant fileReadme.md
suggestion

Modify "greate" to "great" to fix the spelling mistake. [important]

relevant lineWe also think the cooperation and community have greate power. So, experience bartini person also share their own recipe/tricks for new bartini guys.

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


CASSANDRA-19612 Make sure IntersectFilteringQueryTest#shouldNotWarnOrFailOnIndexQuery waits for indexes to be available before making queries

patch by Caleb Rackliffe; reviewed by Stefan Miklosovic for CASSANDRA-19612

Committed as https://github.com/apache/cassandra/commit/1d04457a2414c5b631e84f2dc198842eefbd90c9


chore: setting up dependency factories

Merging this into the core branch. So it’s easier for us to review and request reviews.


VM doesn't have json multidimentional array support

Description:
type of multidimensional json array is a single dimensional json array in VM side
Steps to reproduce:
import ballerina.io;

public function main (string[] args) {
    json[][][][][][][] jsonMulti = [];
    io:println(typeof jsonMulti);
}

above code prints "json[]"
Affected Versions:
current master and previous releases(last release 0.965.0)
Note - fix test cases in PR - https://github.com/ballerina-lang/ballerina/pull/5261 after fixing this

Typeof is not available anymore.


tensorflow-serving apt-get not working

System information

OS : Linux
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.0.0
Python version: 3.7
Installed using virtualenv? pip

Describe the problem
Upon following instructions in https://www.tensorflow.org/tfx/serving/setup for apt-get method to install tensorflow-model-serving-universal
get the following error
Err:3 http://storage.googleapis.com/tensorflow-serving-apt stable InRelease   503  Service Unavailable [IP: <IP_ADDRESS> 80]
Provide the exact sequence of commands / steps that you executed before running into the problem
echo "deb [arch=amd64] http://storage.googleapis.com/tensorflow-serving-apt stable tensorflow-model-server tensorflow-model-server-universal" | sudo tee /etc/apt/sources.list.d/tensorflow-serving.list &&
curl https://storage.googleapis.com/tensorflow-serving-apt/tensorflow-serving.release.pub.gpg | sudo apt-key add -
apt-get update && apt-get install tensorflow-model-server

@ajayaraman I am going to close this issue here. Please post this issue in tensorflow/serving repository as this issue is more apt to that repository.


tensorflow 2.0

File "F:/workspace/python/tf-fit/keras_clr/clr.py", line 603, in
inp = Input(shape=(10,))
File "E:\Anaconda3_5_0_0\envs\DeepLearning\lib\site-packages\keras\engine\input_layer.py", line 178, in Input
input_tensor=tensor)
File "E:\Anaconda3_5_0_0\envs\DeepLearning\lib\site-packages\keras\legacy\interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "E:\Anaconda3_5_0_0\envs\DeepLearning\lib\site-packages\keras\engine\input_layer.py", line 39, in init
name = prefix + '_' + str(K.get_uid(prefix))
File "E:\Anaconda3_5_0_0\envs\DeepLearning\lib\site-packages\keras\backend\tensorflow_backend.py", line 74, in get_uid
graph = tf.get_default_graph()
AttributeError: module 'tensorflow' has no attribute 'get_default_graph'

Keras does not support Tensorflow 2.0 as of version 2.2.4. You'll need to wait until TF 2.0 is officially released, and then Keras master is updated.


solve android bug, it can use in android now

i will send you a email to tell you,thank you

Thanks @joeyzone  a big change, will you please wait for me with more test & code style lint

Hi @joeyzone, what bug does this fix exactly? The most recent version of this component on Github seems to run fine on Android.

I'm closing this as it no longer seems to be relevant.


After bosh update cloud-config, deploying failed at subnet overlaps in 'create_vm' CPI method

I upload cloud config using <IP_ADDRESS>/24 subnet in https://github.com/cloudfoundry/bosh-deployment/blob/master/warden/cloud-config.yml and deployed a deployment called zookeeper as in https://raw.githubusercontent.com/cppforlife/zookeeper-release/master/manifests/zookeeper.yml; then I updated cloud config using https://github.com/cloudfoundry/cf-deployment/tree/master/bosh-lite, and deploy cf on bosh- lite following https://github.com/cloudfoundry/cf-deployment. It says  Error: CPI error 'Bosh::Clouds::CloudError' with message 'Creating VM with agent ID '088d75b3-eebc-4800-a1ad-639de487ece3': Creating container: the requested subnet (<IP_ADDRESS>/22) overlaps an existing subnet (<IP_ADDRESS>/24)' in 'create_vm' CPI method. I changed zookeeper.yml to use the setting in the new cloud config, it still did not solve any problem. Ideally, when we upgrade cloud config, we should be able to redeploy successfully.

The cloud-config from cf-deployment uses <IP_ADDRESS>/22, but the cloud-config from bosh-deployment has already configured <IP_ADDRESS>/24 (and has existing VMs there). Currently BOSH does not support shrinking the network size, particularly if there are existing VMs running.
To fix, you'll either want to recreate your bosh-lite and avoid testing the zookeeper config, or you'll want to...

delete the zookeeper deployment
update cloud-config from cf-deployment
deploy cf

Note: you can redeploy zookeeper alongside cf, just don't try to resize the network.


Resetting frontBio after read now.

Resetting the transport layer event after reading all data.
When the event is not manually reset, the client main loop transforms to a busy loop consuming lots of CPU
This bug was uncovered by #2750 which fixed events getting lost because of automatically reset events and the WaitForMultipleObjects followed by specific WaitForSingleObject calls on the bio event handle.

I guess this issue already exists in Linux version because the Linux style of implementation is always 'manual reset', because in winpr/libwinpr/synch/event.c:
if (!event->bManualReset)
{
    WLog_ERR(TAG, "auto-reset events not yet implemented");
}

By the way, I realize that PR #2750 miss the code for Linux:
ptr->hEvent = CreateFileDescriptorEvent(NULL, FALSE, FALSE, (int) ptr->socket);
Although it is equivalent to "CreateFileDescriptorEvent(NULL, TRUE, FALSE, (int) ptr->socket);" in current winpr implementation, it is wrong semantically.

@realjiangms The linux path is fixed by #2765

I have not looked at the transport thing from a long time, but does your patch work also work with async transport ?

@hardening Should work (have not tested extensivly with async), as it just reads that stuff in its own thread.

+1
Tested and worked well.

Does not work for me, see #2790

@akallabeth @llyzs did you test on linux? This problem is really quite obvious.

I think we should not have to reset event in libfreerdp/core/transport.c
What need to be fixed is still in libfreerdp/core/tcp.c:
WSAEventSelect(ptr->socket, ptr->hEvent, FD_READ | FD_WRITE | FD_CLOSE);
==========>
((It attach to FD_WRITE event. This should not be expected. Because WSAEventSelect attaches to FD_WRITE, it is always in 'set' state even when we drain the read buffer.))
I think it should only attach to FD_READ, same as Linux part:
ptr->hEvent = CreateFileDescriptorEvent(NULL, FALSE, FALSE, (int) ptr->socket, WINPR_FD_READ);
When I looked at this PR firstly, I did realized a question that why we only have problem in windows version. But I didn't think much of it.

@akallabeth I guess the same 'busy loop issue' also exists in Linux version with PR #2765

I tried to remove the FD_WRITE in WSAEventSelect, but it doesn't work. The busy loop issue still exists.
The event created by WSACreateEvent need WSAEnumNetworkEvents or WSAResetEvent to reset the event manually. It seems currently the fix in 2770 is the best solution


Publish docker container for certify proxy

We should really publish a docker container for the certify proxy.

Would be cool to use Github registry for this. Be careful not to push any images we wouldn't be happy to keep as they don't delete images from the registry.

Somehow I forgot I did this for docker hub already: https://hub.docker.com/r/jfbrandhorst/certify


first commit

Button in the navbar of contact us
Closes #558
Since, the user was not able to get the contact button option in the navigation bar, which was quite necessary feature, so I have contributed toward that.
Screenshots

[ x] My code adheres to the established style guidelines of the project.
[ x] I have included comments in areas that may be difficult to understand.
[ x] My changes have not introduced any new warnings.
[ x] I have conducted a self-review of my code.

I have added a button in the navbar of contact us

We don't need a separate button for contact as there is no space for buttons in the mobile view.


zh-cn with 4 missing filled

zh-cn with 4 missing filled

...and released - thanks for your contribution!


Incorrect prototype of eel_set_casts()

The prototype of eel_set_casts() in EEL_register.h is wrong, it does not match the implementation in e_cast.c:
EEL_xno eel_set_casts(EEL_vm *vm, int from, int to, EEL_cast_cb cb);
vs
EEL_xno eel_set_casts(EEL_vm *vm, EEL_classes from, EEL_classes to, EEL_cast_cb cb);
When I found this I thought it might be useful to do some more cleanup. I renamed a few variables so that they have identical variable names in the prototype and the implementation.
There are still some inconsistencies about how the variables are named between the functinos, eg some objects are named "o" while others are named "object". But that is to much work to fix and wouldn't really contribute to anything useful but to please my OCD. ^_^

I think the reason for that is the value/class mess, with different enums... Since I've just unified them, there should be less need for that, and the more reason to use the (now single) enum type instead.


appcenter codepush using Xamarin / Android

I'm trying to publish a .NET MAUI app using the appcenter CLI.
using appcenter codepush release with the options --update-contents-path, -a, and --target-binary-version I get the error Deployment "Staging" does not exist.
When I try to create a deployment (appcenter codepush add Staging), the command fails with Not supported platform / OS combination: Xamarin / Android.
It looks like the deployment name is required with appcenter codepush release, but not supported with this platform / OS combination.
AppCenter CLI version: 2.10.10

Hi @christiannagel , thank you for reaching out!
AppCenter CodePush supports ReactNative and Cordova only. See docs.
Unfortunately, there is no way to use codepush with Xamarin.
You can use appcenter distribute release for your app.
I'm closing this item, but feel free to ask any additional questions.


Hint user to install Language Support for Java if its not installed

Most of the features in this extensions requires Language Support for Java to be installed.
We should hint to install it in the welcome view to make sure full features can be provided.

To verify this (only used for test):

If Language Support for Java is not installed. hint user to install
If it's in LightWeight mode, hint user to import project
If workspace contains no Java projects, hint user to create one or open a project folder
Otherwise, show the project view as normal


Sunburst layout toggle

Similar to #8, this would be an option to draw the flamegraphs in the sunburst layout (radial coordinates).
First use of it I think came from here by @paulirish: https://bugs.chromium.org/p/chromium/issues/detail?id=452624
I've also written about the layout here: http://www.brendangregg.com/blog/2017-02-06/flamegraphs-vs-treemaps-vs-sunburst.html
We can have it as a mode. This is why I wanted flamegraphs to go d3 in the first place, so we can make it more interactive and manipulatable.

Found this recent implementation that might help getting started: https://bl.ocks.org/kerryrodden/7090426
Happy to give it a go later this week unless someone beats me to it.

TBH, I'm much less fond of the sunburst than I used to be.
The general challenge with flamegraphs is focusing their user's attention on the wide&flat parts and ignoring spiky flames. I feel like the sunburst draws attention to the spikes even more than the typical flamegraph layout.
Of course it could always be interesting to toggle to sunburst, but I wouldn't necessarily encourage it these days. :)
IMO if sunburst were attempted, I think it definitely needs the adjacent tree (like we see in Baobab).


Managing updates for Windows Remote Desktop

[Enter feedback here]
is anyone else seeing the issue where the registry setting to manage updates doesnt exist? there is no HKLM\Software\Microsoft\MSRDC\Policies
do I have to manually create it?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b56197c8-e490-f6fe-a494-a8aa584a8faa
Version Independent ID: 46d1a544-517b-a4eb-68a2-cd7194aca44b
Content: Windows Desktop client for admins
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windowsdesktop-admin.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Whitespace at beginning of build output

If there is whitespace at the beginning of a build line the file won't open and instead attempt to create a new file. The error can be recreated when building from munit.
# fails to open on double click
   source/Main.hx:141: characters 10-14 : Unknown identifier : foo

# works
source/Main.hx:166: characters 16-21 : Unknown identifier : bar

Thanks.


add Energy_consumption.txt

In an effort to make ODS more compatible with electric fleets, I am proposing the following addition of Energy_Consumption.txt
vehicle_type_id
Reference ID to the proposed Vehicles.txt.

Required

route_id
Reference to route_id from routes.txt (but practically from trips.txt).

When blank it is assumed a global value.

direction_id
Reference to direction_id from trips.txt.

When blank it is assumed a value for any direction.

from
Referencing either a stop_id or ops_location_id as a start point.

Unclear if we need to separate this into two fields for stop_id and ops_location_id.

to
Referencing either a stop_id or ops_location_id as an end point.

Unclear if we need to separate this into two fields for stop_id and ops_location_id.

start_time
This is the start of when these values are valid.

When left blank, it is considered valid all day.

end_time
This is the start of when these values are valid. When end_time is blank, start_time must also be blank.

When left blank, it is considered valid all day. When start_time is blank, end_time must also be blank.

kwh_per_distance
Consumption per preferred distance unit (although I would really prefer that we force distance units to be described somewhere :) )
consumption_scenario
enum value representing the state of the vehicle. This is mainly to distinguish between a stationary vehicle and one that is moving. This could also allow us to avoid having a stop_id and ops_location_id field for the "to" and "from" fields.

0 is a trip
1 is a deadhead
2 is a stationary vehicle, but is powered on.

service_id
Represents the service_id that these consumption values are valid for.

Some questions:/comments

What is the thinking behind having service_id in here? Are we assuming that because we only require start and endpoints that consumption might differ based on what the vehicle is doing in between those points?
Should we not be using trip_id instead of route_id? We would expect there to be many trips for every one route, but we don't seem to have a way of distinguishing which trip we need to get the route from.
Referring to trip_id would also obviate the need for us to include direction_id or service_id
if route_id is blank, then I think direction_id must also be blank
re: from and to, would it make more sense to get the stop sequence from the associated trip (via stop_times.txt) instead of pulling stop_id from stops.txt?

I like this with one modification to support use cases beyond just BEBs: changing kwh_per_distance to power_per_distance and adding an optional foreign key to the Vehicle_type_id  (per #30) to distinguish charging for different vehicle types where applicable.


Protocol spec incorrectly refers to Swap NFTs

Describe the bug
The protocol spec describes "Swap NFTs": https://protocol.penumbra.zone/main/dex/swap.html
This hasn't been how the system has worked for about two years. We need the description of the system to match how it actually works (we have swap commitments).
We should edit that page to correctly refer to swap commitments.

@redshiftzero there's still code comments referencing swap NFTs that should be removed


add support for && and ; characters in python agent

Add support for && and ; characters in shell commands in the Python Empire agent.  This is an enhancement.  Prior behavior did not support shell commands with those characters in them.
New behavior:
(Empire: MROXAHQX) > shell ls; uname
(Empire: MROXAHQX) >
Desktop
Documents
Downloads
Music
Pictures
Public
src
Templates
Videos
Linux
..Command execution completed.
(Empire: MROXAHQX) > shell ls && uname
(Empire: MROXAHQX) >
Desktop
Documents
Downloads
Music
Pictures
Public
src
Templates
Videos
Linux
..Command execution completed.

@caleb-sfdcsec Thanks again. Completed testing without issue.


Add Neutron l3 Agent Sensu Check

I have modified this check to run using python-neutronclient.
I have set a maximum number of routers within the script to 100.
This is in accordance with: https://github.com/blueboxgroup/ursula/pull/2301

I've included both commits to show the requests method, and then the python-neutronclient method. I can squash when the latest one is accepted.

Let me know if these comments are not clear...we should be able to hammer them out quickly. Ping directly if anything is not clear.

is this check only going to be active/making neutron calls on controller with ucarp?

@nirajdp76 I have the task in Ursula set to only run on the first controller.
when: inventory_hostname in groups['controller'][0]

This check is fine but the Ursula logic will not work.  What happens when the controller fails over?

@craigtracey Even if we set the task to run on the current UCARP primary, only that controller will contain the json config for the check in /etc/sensu/conf.d/checks/. So after a post-install fail over, the check will still run from the new UCARP secondary controller whenever it reboots.


[Documentation] - Explicit say what Provide API the user should Create Instance

Item 3)
Click Create Instance to create a new deployment. The default kfdef provided will deploy the latest release of the [ODH Core components](https://opendatahub.io/docs/tiered-components). If you accepted the default name, this will trigger the creation of an Open Data Hub kfdef object named opendatahub and start the rollout of the [ODH Core components](https://opendatahub.io/docs/tiered-components).

There are 5 "Create Instance"
The one that the user must select is Kf Def

cc @VaishnaviHire


OrderDetails.Shipping.Address does not contain the Phone Number

Hello,
Task GetOrderAsync(int orderId, CancellationToken cancellationToken = default(CancellationToken));
the returned OrderDetails.Shipping.Address  does not contain the Phone Number as detailed in the Bricklink docs here:
https://www.bricklink.com/v3/api.page?page=resource-representations-order
[shipping.address.phone_number] | String | The buyer's phone number
Thank you
Iain

Hey lain,
Thanks for reporting. I'll add it in the next days.
Best

Thank you kindly, I will look out for it... Iain
On Sat, Apr 8, 2023 at 9:35 AM Jens Eisenbach @.***>
wrote:

Hey lain,
Thanks for reporting. I'll add it in the next days.
Best
—
Reply to this email directly, view it on GitHub
https://github.com/gebirgslok/BricklinkSharp/issues/16#issuecomment-1500826030,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYU44ILXRLVX6MDTKGS7WDXAEPL7ANCNFSM6AAAAAAWUPKOV4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I just added the phone no to the response object.
PS: don't 4get to leave a star for this repo :)


--parser with value postcss is deprecated

This line is being prepended to my *.css files when formatting with vim-prettier:
--parser with value postcss is deprecated. Prettier now treats it as: --parser=css.
I am using prettier version 1.7.1.

Hi @nathanspecht thanks for reporting that, will work on a fix for it tonight and do a new release

this is also happening
https://github.com/prettier/prettier/issues/2936

Had the same problem.

Trying to figure out a solution for this problem, in the meantime if you guys can use an older prettier version 1.7.0 that would be great.
I have just pushed to master a pin to force using prettier 1.7.0 dependency version while I try to figure out how to fix things in here and on prettier itself, so please update vim-prettier and
in the meantime you can do that by:
let g:prettier#exec_cmd_path='~/.vim/plugged/vim-prettier/node_modules/.bin/prettier'

Just did a 0.1.3 release that should fix this issue. Please feel free to reopen if you still have the same problems


converting datetime to json gives wrong time

CrateDB version: 2.2.6
JVM version:
OS version / environment description:
Problem description:

Steps to reproduce:

Use case: saving time in a table.

select time from data;
then use the returned time value
select time from data where time<=? order by time desc limit 1;
The returned time is different to the time received from the first query.

Feature description:
Propose to change th line client/http.py:86 to
return int(o.timestamp()*1e3) #delta.microseconds / 1000.0 + (delta.seconds + delta.days * 24 * 3600) * 1000.0)

Tested with python 3.6

Hi @mojovski
unfortunately, i'm not able to reproduce this issue. Can you provide me with some additional information (e.q. the code causing this behaviour and the timestamps you are using)?
Thanks, Christian.

@mojovski is it possible that you forget to do a refresh table xx after insert? This needs to be done because of CrateDBs eventually consistent approach (see https://crate.io/docs/crate/reference/en/latest/general/dql/refresh.html).

@mojovski does this work for you now?


Add Addons namespace to Custom Ops

Fixes issue #379
https://github.com/tensorflow/addons/issues/379
This requires the nightly or the upcoming TF2 RC2, it does not work with the TF2 RC1

Thanks very much Tomer this is a big help. Mind quickly formatting with:
bash tools/run_docker.sh -c 'make code-format'

Done!

Done, but it looks like the way I rebased onto master messed up the CLA bot.

Done, but it looks like the way I rebased onto master messed up the CLA bot.

No worries we can override CLA when time to merge.

Nice job, Tomer!

@ewilderj Hi, Edd, could you help override cla status? Thanks 😄

@WindQAQ @fsx950223 Hi, could you take another look? I'd like to merge it if you both agree :-)

Thanks, all!


💡[Feature]: Adjusting the font mismatch in the home page.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The header font in the home page if very off-putting in Random Disco Light Simulator.
The first two words are comparatively larger than the other two.
Use Case
This increases the project usability and enchances the user experience.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anijit18 are you working on this task as its been a month

@aditya-bhaumik i will provide the PR by today itself.

@anijit18 tommorrow is the last day to contribute try to add the pr by today


Mavros update

WIP
@thomasgubler : this updates tests and demo nodes for MAVROS 0.11
More fixes are coming (another NS change) as soon as I get 0.11.1 on Ubuntu, will wait for that before merging.
Fixes #1979

Needs PX4/containers/pull/12

Looks good on my local installation, will merge as soon as the container on Docker hub is built.

SITL CI: test this please

Thanks for doing the update :+1:


pcp: init at 0.4.0

Motivation for this change
Adds pcp which is a terminal file sharing program much like magic-wormhole and croc.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

All feedback so far has been resolved.

@legendofmiracles I hastily fixed your suggestion without realising you had made a suggestion via Github. I have added your Co-author string, since I did not realise and failed to click the "commit suggestion" button. Thank you.

Tests fail on darwin ofborg but they passed for me locally.
Probably a sandboxing issue, __darwinAllowLocalNetworking may fix it (may still fail on hydra anyway) but I'm fine with just setting doCheck = !stdenv.isDarwin.

commit should be pcp: init at 0.4.0

Same thing for the PR title.

Test fails on darwin, shall we disable it and/or report it to upstream?

Result of nixpkgs-review pr 152869 run on x86_64-linux 1

1 package failed to build:

pcp

Result of nixpkgs-review pr 152869 run on x86_64-linux 1

1 package failed to build:

pcp

Tests fail on my system
=== RUN   TestTimeCriticalProtocol_Bootstrap_connectsBootstrapPeersInParallel
    protocol_test.go:127:
                Error Trace:    protocol_test.go:127
                Error:          Max difference between 50ms and 76.904548ms allowed is 2.5e+07, but difference was -2.6904548e+07
                Test:           TestTimeCriticalProtocol_Bootstrap_connectsBootstrapPeersInParallel
--- FAIL: TestTimeCriticalProtocol_Bootstrap_connectsBootstrapPeersInParallel (0.17s)
=== RUN   TestProtocol_Bootstrap_errorOnNoConfiguredBootstrapPeers
--- PASS: TestProtocol_Bootstrap_errorOnNoConfiguredBootstrapPeers (0.00s)
=== RUN   TestProtocol_Bootstrap_cantConnectToOneLessThanThreshold
--- PASS: TestProtocol_Bootstrap_cantConnectToOneLessThanThreshold (0.00s)
=== RUN   TestProtocol_Bootstrap_cantConnectToMultipleLessThanThreshold
--- PASS: TestProtocol_Bootstrap_cantConnectToMultipleLessThanThreshold (0.00s)
=== RUN   TestProtocol_Bootstrap_cantConnectButGreaterThanThreshold
--- PASS: TestProtocol_Bootstrap_cantConnectButGreaterThanThreshold (0.00s)
=== RUN   TestProtocol_DiscoveryIdentifier_returnsCorrect
--- PASS: TestProtocol_DiscoveryIdentifier_returnsCorrect (0.00s)
FAIL
FAIL    github.com/dennis-tra/pcp/pkg/dht       0.634s
FAIL

@onny I underclocked my laptop to 500mhz and ran nix build, and indeed the tests fail. If you're running the tests on too slow a machine, they will fail. The clue is in the name of the test TestTimeCriticalProtocol. It is "time critical", which means it will fail if the test does not complete fast enough. Not sure what to do to fix this, other than disable the checks entirely.

@onny I underclocked my laptop to 500mhz and ran nix build, and indeed the tests fail. If you're running the tests on too slow a machine, they will fail. The clue is in the name of the test TestTimeCriticalProtocol. It is "time critical", which means it will fail if the test does not complete fast enough. Not sure what to do to fix this, other than disable the checks entirely.

Oh wow thats interesting. We should disable this specific test if possible

Result of nixpkgs-review pr 152869 run on x86_64-linux 1

1 package built:

pcp

Thank you!


[ci] R macOS CI jobs failing on 'brew install basictex'

Description
For at least the last 24 hours, macOS CI jobs have been failing with the following error:
==> Caveats
You must restart your terminal window for the installation of MacTex CLI tools to take effect.
Alternatively, Bash and Zsh users can run the command:

  eval "$(/usr/libexec/path_helper)"

==> Downloading http://mirror.ctan.org/systems/mac/mactex/mactex-basictex-20210325.pkg
==> Downloading from https://ctan.math.illinois.edu/systems/mac/mactex/mactex-basictex-20210325.pkg
curl: (22) The requested URL returned error: 404
Error: Download failed on Cask 'basictex' with message: Download failed: http://mirror.ctan.org/systems/mac/mactex/mactex-basictex-20210325.pkg
sudo: tlmgr: command not found
sudo: tlmgr: command not found
installer: Error - the package path specified was invalid: '/Users/runner/work/LightGBM/LightGBM/R.pkg'.

Which comes from here:
https://github.com/microsoft/LightGBM/blob/d811f3a074d30924a88aebe93eea66df70ecb7dc/.ci/test_r_package.sh#L74-L77
This results in the following related error during R CMD check:
* checking PDF version of manual ...  WARNING
LaTeX errors when creating PDF version.
This typically indicates Rd problems.
* checking PDF version of manual without hyperrefs or index ... ERROR
Re-running with no redirection of stdout/stderr.
Hmm ... looks like a package
Error in texi2dvi(file = file, pdf = TRUE, clean = clean, quiet = quiet,  :
  pdflatex is not available
Error in texi2dvi(file = file, pdf = TRUE, clean = clean, quiet = quiet,  :
  pdflatex is not available
Error in running tools::texi2pdf()
You may want to clean up by 'rm -Rf /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T//RtmpJ2Qw8R/Rd2pdf2a345d471e9'
* checking for non-standard things in the check directory ... NOTE
Found the following files/directories:
  ‘lightgbm-manual.tex’

Reproducible example
Example build from #4811: https://github.com/microsoft/LightGBM/runs/5826112507?check_suite_focus=true.
But this is failing on master and multiple PRs.
This is affecting all macOS R CI jobs.

I found a relevant thread: https://github.com/Homebrew/homebrew-cask/pull/121499#issuecomment-1088161324
This seems similar to #3454 which we've seen repeatedly...inconsistencies between TeX installers and the mirrors they hit can lead to errors like this, and it sometimes takes a significant amount of time, on the order of hours or days, for changes to be replicated to all mirrors.
I'll try re-building in an hour or two. I'll also addd a PR to make the R CI jobs fail earlier on such download errors.

The tests on https://github.com/microsoft/LightGBM/pull/4977 and other recently-rebuilt PRs are now passing, so I think this issue is resolved.


Incorrect .d.ts generated for destructuring arrow function with default

TypeScript Version:
1.8.2 / nightly (1.9.0-dev.20160224)
Code
main.tsx:
import React = __React;
namespace A {
    export const Greeter = ({name = 'world'}) =>
Hello, {name}!
;
}

module1.tsx:
const B = ;

react.d.ts
// latest version from the DT repo

Action
I run the following commands:
> tsc -d --jsx react react.d.ts main.tsx
> tsc --jsx react react.d.ts main.d.ts module1.tsx

Expected behavior
Both commands successfully complete.
Actual behavior
On TS 1.9.x:
> tsc --jsx react react.d.ts main.d.ts module1.tsx
main.d.ts(3,22): error TS2371: A parameter initializer is only allowed in a function or constructor implementation.

On TS 1.8.2:
> tsc --jsx react react.d.ts main.d.ts module1.tsx
main.d.ts(3,22): error TS1131: Property or signature expected.
main.d.ts(3,37): error TS1128: Declaration or statement expected.
main.d.ts(4,14): error TS1109: Expression expected.
main.d.ts(5,6): error TS1128: Declaration or statement expected.
main.d.ts(5,8): error TS1128: Declaration or statement expected.
main.d.ts(6,1): error TS1128: Declaration or statement expected.

Notes
It seems that the generated typings file, main.d.ts, is invalid:
import React = __React;
declare namespace A {
    const Greeter: ({name = 'world'}: {
        name?: string;
    }) => JSX.Element;
}

Minimal repro (run with --d)
const k = ({x = 'y'}) => { }

Good call @RyanCavanaugh, all the React stuff above is not relevant to this bug. I happened to stumble across this when trying to use stateless functional components introduced in 1.8, so I'd guess that others might come across it soon as well :)

sorry @JKillian for the delay. fix should be in typescript@next later tonight.

I'm seeing this in the 1.8.10 official release. Was it ever patched or do I need to switch to the nightlies to get the fix?

@joewood, this was fixed after TS 1.8 was forked and release (note the milestone is "TypeScript 2.0"). there is no intention to back port the fix to 1.8.

OK, thanks. Any ETA on a dev release of 1.9 or 2.0? (i.e. should I change my code to remove default values on de-structured parameters)

Adding an explicit type annotation should circumvent the issue. e.g. const k = ({x = 'y'} : { x: string }) => { }

that didn't work for me...
const k = ({x = 'y'}:{x:string}) => { }

still produces:
declare const k: ({x = 'y'}: {
    x: string;
}) => void;

And with typescript@next it's:
declare const k: ({x}: {
    x: string;
}) => void;

apologies for this, should have tried it before recommending it as a workaround. you are right. it does not work..
that leaves us with either not using the default, or changing the meaning  to make the initialization on the whole parameter (i.e. const k = ({x} = {x: 'y'} ) => { });

yes, right - or doing a beta release soon 😄


NullPointerException thrown during AbstractRedisAsyncCommands.flushCommands

Bug Report
Possibly there is a bug in lettuce, where NPE is thrown during flushCommands operation. See some simplified code, where this is happening. I have included all custom options we set on the client. This code runs against clustered enterprise redis.
I understand that it might be also caused by our environment or some specific options we use, but we're blocked in investigation with hardly any ideas on how to proceed with it. Code runs in grid and issue appears randomly without ways to debug it. Therefore if you don't have any suggestions on what to change, could you please at least share ideas on how to improve the situation by let's say extra logging or similar?
Current Behavior
The application throws with NullPointerException, when executing AbstractRedisAsyncCommands.flushCommands

Stack trace
	at x.x.x.x.x.X.executeAll(PipelineExecutor.java:97)
	... // some more calls done in application
Caused by: java.lang.NullPointerException
	at io.lettuce.core.internal.AsyncConnectionProvider$Sync.doWithConnection(AsyncConnectionProvider.java:266)
	at io.lettuce.core.internal.AsyncConnectionProvider.lambda$forEach$3(AsyncConnectionProvider.java:190)
	at java.util.concurrent.ConcurrentHashMap$ValuesView.forEach(ConcurrentHashMap.java:4707)
	at io.lettuce.core.internal.AsyncConnectionProvider.forEach(AsyncConnectionProvider.java:190)
	at io.lettuce.core.cluster.PooledClusterConnectionProvider.flushCommands(PooledClusterConnectionProvider.java:555)
	at io.lettuce.core.cluster.ClusterDistributionChannelWriter.flushCommands(ClusterDistributionChannelWriter.java:377)
	at io.lettuce.core.RedisChannelHandler.flushCommands(RedisChannelHandler.java:326)
	at io.lettuce.core.AbstractRedisAsyncCommands.flushCommands(AbstractRedisAsyncCommands.java:539)
	at x.x.x.x.x.X.executeAll(PipelineExecutor.java:55)
	... 19 more

Input Code

Input Code
public class PipelineExecutorClass<T> {

    public Map<String, T> execute() {

        ClusterClientOptions.Builder optionsBuilder = ClusterClientOptions.builder();
        optionsBuilder.autoReconnect(true).cancelCommandsOnReconnectFailure(true);
        optionsBuilder.disconnectedBehavior(ClientOptions.DisconnectedBehavior.REJECT_COMMANDS)
                    .topologyRefreshOptions(ClusterTopologyRefreshOptions.builder().enablePeriodicRefresh(Duration.ofSeconds(5))
                            .enableAllAdaptiveRefreshTriggers().build());
        // set SSL options
        ClusterClientOptions options = optionsBuilder.build()

        RedisURI redisUri = RedisURI.create("redis_url");
        // set password
        io.lettuce.core.cluster.RedisClusterClient client = io.lettuce.core.cluster.RedisClusterClient.create(uri);
        client.setOptions(options);

        StatefulConnection<String, String> connection = client.connect.get();
        AbstractRedisAsyncCommands<String, T> advancedClusterAsyncCommands = ((StatefulRedisClusterConnection<String, String>) connection).async()
        advancedClusterAsyncCommands.setAutoFlushCommands(false);

        int attempts = 0;
        Set<String> keySet = Sets.newHashSet(/* some keys */);

        Map<String, RedisFuture<T>> futures = Maps.newHashMap();
        for (String key : keySet) {
            try {
                futures.put(key, advancedClusterAsyncCommands.get(key));
            } catch (Exception e) {
                // Exception occurred while setting key
            }
        }
		Map<String, T> result = Maps.newHashMap();
        try {
            do {
                advancedClusterAsyncCommands.flushCommands();
                boolean ready = LettuceFutures.awaitAll(30000, TimeUnit.MILLISECONDS,
                        futures.values().toArray(new RedisFuture[futures.size()]));

                for (Map.Entry<String, RedisFuture<T>> item : futures.entrySet()) {
                    final RedisFuture<T> future = item.getValue();

                    if (future.isDone()) {
                        // add to result and handle errors
                    }
                }

                if (!ready) {
                    // "Timeout. Will retry to handle following keys again: ...
                }
                attempts++;
            } while (!futures.isEmpty() && attempts < 3);
        } catch (Exception e) {
            // Exception occurred while handling keys
        } finally {
            advancedClusterAsyncCommands.setAutoFlushCommands(true);
        }
        return result;
    }
}

Expected behavior/code
Application should not throw NPE.
Environment

Lettuce version(s): [e.g. 5.2.1.RELEASE]
Redis enterprise version: [e.g. 5.0.5], clustered

Possible Solution

Additional context

Thanks for the report. Looking at the code, there should be no way that the action callback object is null. We're going to add a guard to the execution callback. Other than poking at the code, there's nothing really what you could do.

Thanks for answering Mark. Feel free to close this if you don't have any other suggestions.
The only other option I'm left with are retries I guess. I have to decide on which level to retry though. My initial thought is that whole process must be restarted as the connection might be somehow malformed at this time and next attempt to get data will fail too.
BR,
Konrad

If this issue comes back, then I'd advise you to introduce a few log/debug lines in a custom build of Lettuce to trace where the issue comes from. Closing the issue for now as we cannot do here without further knowledge.

Thanks, and speaking of some logs - do you have any suggestions where can I add them to get some information that could help tracking the bug?


cf unbind-service error

currently have version 0.5.0 of the broker deployed and seeing this on an instance that was renamed (and renamed back to the original). This instance may have been created prior to 0.5.0 where there were issues with renaming instances.
The error is:
Error: An unbind operation for the service binding between app mp-adapter and service instance mongodb-dso failed: Service broker error: cannot find plan for instance “instance id”: instance metadata has the wrong type map[string]interface {}

Waiting on more info from customer

Hi @jasonmimick ,
May I know what info are you looking for please?

Hi @jasonmimick ,
May I have an update please?
Did I provide all info you need?
Thanks a lots.

Hi,
This happened because the service instance was created with an older (pre-v0.5) version of the Broker. It's not possible to use versions v5.0 and newer with these old instances as there was no backwards compatibility during these early stages of Broker development.
If you only have this one instance with such issue and are not hitting it with later versions of the Broker, please remove the broken instance by hand (delete from Atlas + cf purge-service-instance).
That being said, in the more recent releases we're able to provide better backwards compatibility since the whole design is getting a lot more stable.
Closing since this should not happen in latest versions.

Hi @jasonmimick ,
We updated to v0.7.0 but still see this issue, please kindly advice.

there's actually 4 of the broker-state realm apps in the SEA preview org.
We are waiting for @vasilevp to implement a fix to the broker and also provide a script to fix the duplicate realm apps.
We can manually fix the instancesif needed.


Disable ireturn linter in workflow

Issues associated with this PR
#2256
Solutions
Disable ireturn in workflow file

Could you please also check this message? there might be other lints that need to be disabled.
https://github.com/mosn/mosn/actions/runs/4477912086/jobs/7870029509#step:3:94
The golint CI show happy may be due to some other reason, I mean it actually is unhappy.


Create Activity Feed Component

The Activity Feed is a small component that lives on the right side of the main area and shows recent activity on the site.

This depends on saving favorites functionality.
After user accounts are added, we can show the user who created a favorited plant

The feed now exists as well as user accounts.
We probably need to manage userName state in the Plants layout component rather than in the Nav. That way, it can be accessed in other components such as the Activity Feed here.


Determine return type for non-paginated listing operations

Paginated listing items return PagedFlux<T>. We need to decide if single-request listing operations should return a PagedFlux<T> that will only ever fetch one page, or if they should return Mono<List<T>> (or Mono<SomeTypeWithAListInIt> depending on the specific API).

PagedFlux<T> is more user-friendly, in my opinion. I would improve PagedFlux to have a more appropriate constructor so you are not forced to pass in null for the follow-up page arg. I would also improve PagedFlux to clearly document that it may be used in cases where there is only a single page of results. If you agree, we can work to make this so. Thanks.

Issues exist for the resolution of these discussions.


units table: sorting with subset of units

When displaying the first 30 rows, it's best to first sort the units and then restrict to the first 30 rows. Right now it's done in the other order which is not ideal.
@jsoules This might be a bit tricky, not sure.

See figurl issue #2 and PR #4.


Ensure Pac4JUserProfiles are ClusterSerializable

Not the prettiest fix for #145 but it works.

Thanks for your contribution


publish<EMAIL_ADDRESS>

Requested by: @AbhiPrasad
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[x] npm
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

#retract


Only allow requests through CloudFront

This locks down the API server's load balancer so that it only forwards requests to the actual service if it has a header with a secret value. CloudFront is already configured to set that header (see #1186), so this effectively allows only requests from CloudFront.
Fixes #1181.

:shipit: 🤞


G42

Setting aside a dedicated block for reviewing these tomorrow (3 Nov)

So I took just the g42 and ran it through the validator https://validator.w3.org/ and took a look locally in my browser; there's a few things that can be fixed: errors but also I notice the code examples go into a wide horizontal box so I've been newlining those so they'll all show up on a page, indented etc so developers can recognise what's going on (even tho a lot of this code will get wiped).
Did you want to go through those? I will be sending 11 soon.

So what I need to do is to run all the guidelines that I have copied through the validator and to correct the errors and fix the code examples, right? And what do you mean by "sending 11 soon"?


Installed Todo app & added paths to todo.urls

added a Todo app in settings.py file
created path of index , add , delete  in 'Todo apps'  urls.py file
Added a path to todo.urls for '' and path to todo.urls for 'todo/'url pattern in urls.py

Now app running without any issues.

would be nice to add this repo to hacktoberfest  topic  or a hacktoberfest-accepted   topic .

Hey, please have a look at CONTRIBUTING.md and limit your PR to a specific issue as PRs addressing multiple issues won't be accepted. You could break this into 2 PRs addressing on 2 separate issues.

ok , I will close this and create 2 new PRs

Cool, thanks


CTS tests failing - xtdb

The CTS tests for xtdb have been been failing for over a week.
For example https://github.com/odpi/egeria-test-cts/actions/runs/5239075143 reports:

The xtdb test times out - details showing:

It's not obvious that there is a specific xtdb or base change that has caused this behaviour

Actions to consider

Update test script to ensure the timeout for running the test is enough under 6 hours, so that we can at least continue the script if the test times out. For example using 5:45 may be prudent, in addition to adding some steps to extract k8s console logs
Use the built in 'ssh' capability so that the test is initiated, and then interactive k8s activities can be used to identify what is going on
try running CTS for xtdb locally, or in another more accessible environment (openshift, rancher etc)

I've added a PR to adjust timeouts & try to capture console log.
Meanwhile I ran a session with ssh access, and can see that there is a failure to start the tests:
Mon Jun 12 11:35:48 GMT 2023 cts Information CONFORMANCE-SUITE-0008 The Open Metadata Repository Conformance Workbench repository-workbench is waiting for server tut to join the cohort
Mon Jun 12 11:35:58 GMT 2023 cts Information CONFORMANCE-SUITE-0008 The Open Metadata Repository Conformance Workbench repository-workbench is waiting for server tut to join the cohort
Mon Jun 12 11:36:08 GMT 2023 cts Information CONFORMANCE-SUITE-0008 The Open Metadata Repository Conformance Workbench repository-workbench is waiting for server tut to join the cohort
Mon Jun 12 11:37:04 GMT 2023 cts Information CONFORMANCE-SUITE-0008 The Open Metadata Repository Conformance Workbench repository-workbench is waiting for server tut to join the cohort
Mon Jun 12 11:37:29 GMT 2023 cts Information CONFORMANCE-SUITE-0008 The Open Metadata Repository Conformance Workbench repository-workbench is waiting for server tut to join the cohort

This likely means the tut never started -- and the most common cause of that is library loading issues
In fact in the logs, there are zero log entries from the tut server itself -- and even the cts server is reporting restarts
Possible dependency clash? will check locally.

The  logs show that an exception occurs initializing the server:
"OCF-CONNECTION-400-005 Unknown Connector Provider class org.odpi.egeria.connectors.juxt.xtdb.repositoryconnector.XtdbOMRSRepositoryConnectorProvider passed in connection <Unknown>","The system is unable to create the requested connector instance because the Connector Provider's class is not known to the JVM.  This may be because the Co

(truncated)
Note that this is logged when configuring the cts -- so not in the platform output log....
The cause is that the download of the connector (also not clearly reported) fails, due to
wget http://repository.sonatype.org/service/local/artifact/maven/redirect\?r\=central-proxy\&g\=org.odpi.egeria\&a\=egeria-connector-xtdb\&v\=LATEST\&c\=jar-with-dependencies
--2023-06-12 16:08:01--  http://repository.sonatype.org/service/local/artifact/maven/redirect?r=central-proxy&g=org.odpi.egeria&a=egeria-connector-xtdb&v=LATEST&c=jar-with-dependencies
Resolving repository.sonatype.org (repository.sonatype.org)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to repository.sonatype.org (repository.sonatype.org)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://repository.sonatype.org/service/local/artifact/maven/redirect?r=central-proxy&g=org.odpi.egeria&a=egeria-connector-xtdb&v=LATEST&c=jar-with-dependencies [following]
--2023-06-12 16:08:01--  https://repository.sonatype.org/service/local/artifact/maven/redirect?r=central-proxy&g=org.odpi.egeria&a=egeria-connector-xtdb&v=LATEST&c=jar-with-dependencies
Connecting to repository.sonatype.org (repository.sonatype.org)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 401 Unauthorized

Username/Password Authentication Failed.

This previously failed and was reported at https://community.sonatype.com/t/issues-or-something-changed-cannot-download-artifact-without-user-pass-prompt/10593
It was then fixed, but in the last week+ has again failed.
A more resiliant fix would be to

enable charts to download via maven API instead
when building xtdb etc, post in github as an artifact, and use within egeria test cts

Both require dev work.
Some improvements may be useful in the way failed downloads are detected/reported.
cc: @mandy-chessell @lpalashevski

One of the sonatype community leaders has posted to the thread advising that the team has been informed, so hopefully someone will look at the download issues & then our CTS will be running again.


Logic: Updated Remarks to use Edit/Add commands instead.

This PR resolves #65.

It removed the remark command and merged it with add and edit instead.

Updated the test cases to follow this format.

Updated Developer Guide Title

Updated Team Collated Codes. This resolves #97 .

Coverage decreased (-0.7%) to 88.201% when pulling de96be961b59f3b65364e0ead69cbe43bacd668b on hanselblack:Remark into 8a74c71102794727887286fa1834e8f9b5516262 on CS2103AUG2017-W11-B4:master.

Coverage increased (+0.4%) to 89.372% when pulling 0326c197039155ed4c88109e18c5a4c369274c8d on hanselblack:Remark into 8a74c71102794727887286fa1834e8f9b5516262 on CS2103AUG2017-W11-B4:master.


Config not initialized

when run 'generate.sh' meet problem
AttributeError: 'Config' object has no attribute 'overwrite'
I guess it is because inside config.py instance() method:
@classmethod
def instance(cls):
if cls._instance is None:
print('Creating new instance')
cls._instance = cls.new(cls)
# Put any initialization here.
return cls._instance
there is no initilization automatically call?

Sure, here is the stacktrace
Generating site from /pathhide/PhotoGallery/photos
Importing /pathhide/PhotoGallery/photos/Tianjin
Detecting Faces...
Importing /pathhide/PhotoGallery/photos/Tianjin/1 as tianjin > 1 (tianjin-1)
 --> Processing /pathhide/PhotoGallery/photos/Tianjin/1/787A6964.JPG...
 --> Processing /pathhide/PhotoGallery/photos/Tianjin/1/787A7059.JPG...
Creating new instance
Creating new instance
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/pool.py", line 48, in mapstar
    return list(map(*args))
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 266, in _process_photo
    return (photo_file, Photo.process_photo(external_path, photo_file, filename, unique_slug, album_folder, _process_photo.people_q))
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 193, in process_photo
    if Config.instance().overwrite or not os.path.exists(new_original_photo):
AttributeError: 'Config' object has no attribute 'overwrite'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/pathhide/PhotoGallery/fussel-main/fussel/./fussel.py", line 83, in <module>
    main()
  File "/pathhide/PhotoGallery/fussel-main/fussel/./fussel.py", line 44, in main
    generator.generate()
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 442, in generate
    Albums.instance().process_path(Config.instance().input_photos_dir,
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 314, in process_path
    self.process_album_path(
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 388, in process_album_path
    self.process_album_path(
  File "/pathhide/PhotoGallery/fussel-main/fussel/generator/generate.py", line 359, in process_album_path
    results = P.map(_process_photo, jobs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/pool.py", line 367, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/pool.py", line 774, in get
    raise self._value
AttributeError: 'Config' object has no attribute 'overwrite'

I get the same.

Also getting the same, has anyone found a solution?

同样得到同样的问题，有人找到解决方案了吗？

_process_photo  函数是在线程里面执行，无法获取外面的Config.instance()。传进去就可以了

Can you try the new 2.1.0 version and let me know if its fixed? I believe it should be.


Impossible to send STEEM with api

I got this :
{ AssertionError [ERR_ASSERTION]: Expected version 128, instead got 176
code :
steem.broadcast.transfer(acc_password, acc_username, "oxward", "0.100 STEEM", "testing tx", function(err, result) {
	  console.log(err, result);
	});

Any solution ?
Thx.
UPDATE ----
I use now the Posting key, i can upvote but i can't send STEEM :
{ RPCError: missing required active authority:Missing Active Authority peneinc

Does not work.
RPCError: missing required active authority:Missing Active Authority

Fixed : u need to use your ACTIVE key


Fixed problem with inserting element to arraylist on large index

Inserting element on position larger then size was causing exception - IMO it's counter intuitive because if we can insert element on position larger then size of list (list size + 1) then probably we should be able to insert it on any position.

That's strange - logs said that some configuration errors caused this fail and now I don't have access to this logs...

Ok, I've been doing a little digging in this stuff and found out, that some jdks are not supported anymore by travis afaik.


bugfix(reported_issues #304)

1.fix bug #304 , make onViewRecycled()，onViewAttachedToWindow() and onViewDetachedFromWindow() function to support situations where the position is zero

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

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

@fengivy fen 麻烦签一下 CLA 哈

@longerian 已经签了，出来了2个一样的，第二个打开显示已经签了


Raise usage error when updating profile does not receive any argument

Error when profile update does not receive any arguments
Will take care of some feedback from QA

Note that the current CHANGELOG entry also basically overs this change

I get a test that fails locally but no in CI/CD