async="true" loses current thread identity

I'm setting async="true" for perf reasons and I know that causes targets to be called on a secondary thread. I was wondering how I can keep the user identity (even the identity.name) of the thread when the log message was created. I see that the layout renders have access to this but my custom targets do not have identity information cached anywhere. I think if this isn't possible, we should at least pass the existing identity along to the target.

Looks like the layout cache has this info but it's internal. It would be great if we could just expose this on the logger context!

do you mean ${windows-identity}?

In our nlog target I'd like to set the current user if it hasn't already been set. I try to use different contexts but I wanted a fallback to System.Threading.CurrentThread.Identity https://github.com/exceptionless/Exceptionless.Net/blob/master/Source/Platforms/NLog/ExceptionlessClientExtensions.cs#L11

If you write a custom layout renderer, it should work?

My target looks like this:
 <targets async="true">
    <target name="file"
      xsi:type="File"
      layout="${longdate} ${threadid:padding=4} ${level:uppercase=true:padding=5} ${logger:shortName=true} ${message} ${exception:format=tostring}"
      fileName="${logDirectory}/log.txt"
      archiveFileName="${logDirectory}/log.{#}.txt"
      archiveEvery="Day"
      archiveNumbering="Rolling"
      maxArchiveFiles="7"
      concurrentWrites="true"
      createDirs="true"
      autoFlush="true" />

    <target name="exceptionless"  xsi:type="Exceptionless">
      <field name="host" layout="${machinename}" />
      <field name="identity" layout="${identity}" />
      <field name="process" layout="${processname}" />
      <field name="project" layout="${event-context:item=project}" />
    </target>
  </targets>

I first noticed that we weren't setting the user for some exceptions but our layout had rendered a custom object that had it.. Upon investigation this was from the current thread and the authenticated web api user.. So I was hoping to pass this user through our exceptionless code as a fallback but it's not on any nlog context just internal cached layouts

I don't want to do that, I just want access to the current threads identity when the log was created.

Maybe i'm missing it, but ${identity} won't work well with async?

No it won't, but there is no reason it can't be passed along with all of the other contextual information in the context property of LogEventInfo. This information as well as all renderer info is there but thrown away and hidden.

Notice in this screenshot that I'm looking at my target that's being called in an async manner.. The current thread isn't set and I know why and that's a good thing.. But the private layoutcache contains that info and the context doesn't contain any data either.. There is no reason this and other useful info couldn't be passed along to the target.

${identity} won't work well with async?

No it won't,

I guess this should be fixed ?

Yes. Well it happens on a secondary thread and I'm not sure of the architecture here but this secondary thread might handle multiple different messages (which may mean each of those could be from a different Identity). I think it would be super useful to populate this info on each LogEventInfo context so it can be accessed no matter how the log target is being called,

I think it would be super useful to populate this info on each LogEventInfo context so it can be accessed no matter how the log target is being called,

Agree, but it's more important to have the bug fixed.

Works on my machine?
https://github.com/NLog/NLog/pull/1428

@304NotModified that test doesn't really test this issue. The layout render is right and always has been right. The issue is when the target is called in an async wrapper, the Thread.CurrentThread.Identity may not be set (which I don't think should be because it could be buffered messages?), and there is no way to get a hold of what the current thread identity was when the layout was rendered/log call was made

Can you give me an example/tesrlt repo? Because I don't get what isn't working.

I think we have two cases here, and that confusing

A bug? (according to your title), which I cannot reproduce
The feature request to read the layout renderer cache

I think it's wise to discuss them separate GitHub issues. I like to fix 1, if it is really a bug, but AFAIK all works.

If you want I'd be more than happy to launch a teamviewer session and show you this issue first hand. I'm on slack all day.
I think 1. is a bug and 2 kinda ties into this, it's not just the layout render cache but the context object should be populated with as much info as possible so the target can do anything the layout render can.

@niemyjski
Sorry about the delayed response. Also I was too busy for a (small) meeting.
Good news (or maybe not) is that the unit test I wrote sometimes fails: https://github.com/NLog/NLog/issues/1441. So there is something bad going on.
Don't understand why it fails, as the prerender always will be done in the calling thread.
About 2), reading the layout can be done by re-rendering the layout? E.g.
Layout l = "${identity};
string identity = l.render(logEvent); //logEvent with layoutCache

This is still an issue I guess?

Yes, when running in async you are not guaranteed to be on the same thread. My issues is not with layouts (which seems to have the correct info). It's that I want to get access to this cached info later in the process after layouts have ran. We need to copy this data some place and have layouts use it regardless of what thread you are on.

Is it possible to create a litre sample app or unit test? Clearly I don't fully understand the issue, but I think it could be fixed.

http://stackoverflow.com/questions/21838651/i-thought-await-continued-on-the-same-thread-as-the-caller-but-it-seems-not-to
http://blog.stephencleary.com/2013/11/there-is-no-thread.html

Those both are worth reading. You could create a test that does something like this
// log
// await Task.Delay(50);
// log
// await Task.Delay(50);

You'd probably want to use https://github.com/StephenCleary/AsyncEx/wiki/AsyncCountdownEvent to ensure that you hooks to verify logs have been created / event handlers called.  We do this in our tests :) https://github.com/exceptionless/Foundatio/blob/eb16268f037975f00482f41257d20f7fbe2c4093/test/Foundatio.Tests/Utility/ScheduledTimerTests.cs#L36-L59
Based on the task scheduler context, you are not guaranteed  to be running on the same thread. Which is exactly what happens when you have async=true.

Sure, but the async renderer isn't using await.

Forget the render.... I want all of this contextual information in my custom target which is called async...

Please provider a simple example / unit test.

I think we need a test application, otherwise we cannot fix this. Please provide one, otherwise I have to close this one. Thanks!

https://github.com/NLog/NLog/pull/1660

I didn't even have to startup a new thread, just a single await task delay..

It shows off how fragile this is, layouts will be rendered exactly how they should be but targets lose the contextual information. If you fix this by putting all of the contextual information that layouts have (the cached data), then this will work properly too if you do any buffered/bulk target writes.

Thanks for the unit test

No problem :) Sorry for the delay. I finally got some free time.

async="true" does not cause current user-identity-info to be lost, this is handled by NLog Layout Precalculate.
But you are right that the user-identity-rights are lost, when logging within impersonation-scope, and the actual logging happens on a background-thread after impersonation-scope-exit.
NLog doesn't have any magic to recreate the impersonation-scope of the original user. The only option currently available is https://github.com/NLog/NLog/wiki/LimitingWrapper-target
This is not a bug, but just working as designed.


"Pending" status is missing for offchain tx

Env: https://pr1397--safereact.review.gnosisdev.com/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/transactions
Have a safe with 2 owners needed to sign a tx
Create and sign any type of tx (send funds will do)
You will see that the mockup tx with the pending label status will NOT show up. After a while the tx with the "Awaiting confirmations" label will show up
If you sign that tx without executing (another off chain signature) the tx will not show the pending status once more. After a while the "Awaiting execution" status label will show up
Executing a transaction WILL show the Pending status label just fine. The issue is only with the off chain signatures
https://www.loom.com/share/059b53f6276843198ce0c39e55675e33

off-chain transactions weren't eligible for "Pending" status.
That behavior is intentional, as it's assumed that the off-chain transaction is almost immediately reflected in the Safe.

Ok, off chain tx have no "pending" status. I'll add that to the test cases


Connecting to IRCCloud bouncer with other clients

https://github.com/Libera-Chat/libera-chat.github.io/blob/c847b4b88129551460367156b3ca8efb53470bfe/content/_guides/clients.md?plain=1#L156-L163
I appreciate the mention of IRCCloud on the website, and I'm not asking for any further promotion, but I'd just like to note that the comment about the bouncer only working with our own clients is not accurate. Paid users are able to connect with other clients.
https://blog.irccloud.com/bouncer/

@jwheare

Paid users are able to connect with other clients.
I've PR'd a change to the IRCCloud section to note this. If you have a moment, I would appreciate if you could review the PR for accuracy.

I'd be interested in knowing if there are any clients not developed or maintained by IRCCloud Ltd that use IRCCloud's API.

I've left a review, thanks. I'm not aware of any full 3rd party clients that use the API.


Can you please provide more details in the script?

I have copied the script into a bot.rb and run it, how do I test it from my slack ?

@cherhan A little later today I'll put up a full working example.

@cherhan I have created a fully-working implementation of botsy here: https://github.com/rlister/slackbotsy_example. You will find full instructions and some example scripts.


07/28 Update

Hi @markcandelora, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


[BUG] Does not use the VERSION_STATIC property

Describe the bug
Does nothing with the version number assigned through VERSION_STATIC
Failed Action Log URL (Required)
A URL to the failed action log.
To Reproduce
Steps to reproduce the behavior:

Assign value to VERSION_STATIC

Expected Behavior
NuGet package should be created with the assigned version
Environment:

I ran into this issue as well, I have the Action create the version, and would like to set it manually for the NuGet package.

Yeah I'm a bit unclear on how this is supposed to be integrated, or if this should just be more of a boolean flag?
GitVersioning just as an example seems to already "come up with" the version itself, and it produces a package file with that version accordingly (assuming doing GeneratePackageOnBuild.  I'm tinkering around a bit with this build action right now and running into that problem.
Seems like some examples that show off the use of this feature may be worthwhile, like how it all fits together.  Though based on some of the issues and prs in this repo, perhaps it's been abandoned?


Implement mux and demux

Adding some frequently used combinational blocks. These are just 2:1 and 1:2 implementations, it would be useful if we can give option to the user to choose the variant on selecting the module. But we can use 2:1 muxes to easily create 3:1, 4:1 etc.

Awesome, ty


dxvk and time reporting errors...

Compatibility Report
ESO - works great except for whn i used an addon called libdebuglogger
Steam AppID of the game:i dont know that 306130?
System Information
sudo inxi -Fxzmc0
System: Host: manjaroed Kernel: 5.6.13-1-MANJARO x86_64 bits: 64 compiler: gcc v: 10.1.0 Desktop: MATE 1.24.0
Distro: Manjaro Linux
Machine: Type: Desktop System: Gigabyte product: AB350M-DS3H v: N/A serial: N/A
Mobo: Gigabyte model: AB350M-DS3H-CF v: x.x serial: N/A UEFI [Legacy]: American Megatrends v: F50a date: 11/27/2019
Battery: Device-1: hidpp_battery_0 model: Logitech M570 charge: 90% status: Discharging
Device-2: hidpp_battery_1 model: Logitech Wireless Keyboard K360 charge: 55% (should be ignored)
status: Discharging
Memory: RAM: total: 15.65 GiB used: 2.92 GiB (18.7%)
Array-1: capacity: 128 GiB slots: 4 EC: None max module size: 32 GiB note: est.
Device-1: DIMM 0 size: No Module Installed
Device-2: DIMM 1 size: 8 GiB speed: 3000 MT/s type: DDR4
Device-3: DIMM 0 size: No Module Installed
Device-4: DIMM 1 size: 8 GiB speed: 3000 MT/s type: DDR4
CPU: Topology: Quad Core model: AMD Ryzen 5 2400G with Radeon Vega Graphics bits: 64 type: MT MCP arch: Zen
L2 cache: 2048 KiB
flags: avx avx2 lm nx pae sse sse2 sse3 sse4_1 sse4_2 sse4a ssse3 svm bogomips: 57513
Speed: 1421 MHz min/max: 1600/3600 MHz Core speeds (MHz): 1: 1420 2: 1420 3: 1460 4: 1523 5: 1592 6: 1405 7: 1486
8: 1426
Graphics: Device-1: Advanced Micro Devices [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] vendor: ASRock
driver: amdgpu v: kernel bus ID: 01:00.0
Display: server: X.Org 1.20.8 driver: amdgpu,ati unloaded: modesetting resolution: 1920x1080~60Hz
OpenGL: renderer: Radeon RX 570 Series (POLARIS10 DRM 3.36.0 5.6.13-1-MANJARO LLVM 10.0.0) v: 4.6 Mesa 20.0.6
direct render: Yes
Audio: Device-1: AMD Ellesmere HDMI Audio [Radeon RX 470/480 / 570/580/590] vendor: ASRock driver: snd_hda_intel v: kernel
bus ID: 01:00.1
Device-2: Advanced Micro Devices [AMD] Family 17h HD Audio vendor: Gigabyte driver: snd_hda_intel v: kernel
bus ID: 0a:00.6
Sound Server: ALSA v: k5.6.13-1-MANJARO
Network: Device-1: Realtek RTL8111/8168/8411 PCI Express Gigabit Ethernet vendor: Gigabyte driver: r8169 v: kernel
port: e000 bus ID: 05:00.0
IF: enp5s0 state: up speed: 1000 Mbps duplex: full mac:
Device-2: Intel Wi-Fi 6 AX200 driver: iwlwifi v: kernel port: e000 bus ID: 08:00.0
IF: wlp8s0 state: down mac:
Drives: Local Storage: total: 1.37 TiB used: 362.41 GiB (25.9%)
ID-1: /dev/nvme0n1 model: Sabrent Rocket Q size: 931.51 GiB
ID-2: /dev/sda vendor: SanDisk model: Ultra II 500GB size: 465.76 GiB
ID-3: /dev/sdb type: USB vendor: Generic model: Flash Disk size: 1.92 GiB
Partition: ID-1: / size: 48.97 GiB used: 26.20 GiB (53.5%) fs: ext4 dev: /dev/nvme0n1p3
ID-2: /boot size: 968.3 MiB used: 186.0 MiB (19.2%) fs: ext4 dev: /dev/nvme0n1p2
ID-3: /home size: 865.62 GiB used: 208.49 GiB (24.1%) fs: ext4 dev: /dev/nvme0n1p4
Sensors: System Temperatures: cpu: 43.0 C mobo: N/A gpu: amdgpu temp: 35 C
Fan Speeds (RPM): N/A gpu: amdgpu fan: 1495
Info: Processes: 331 Uptime: 8m Init: systemd Compilers: gcc: 10.1.0 clang: 10.0.0 Shell: bash v: 5.0.16 inxi: 3.0.37
I confirm:
that I haven't found an existing compatibility report for this game.
The game works.
that I have checked whether there are updates for my system available.
I am up to date with the latest version of the game and with proton and dxvk
Have no idea whgat that is
Symptoms
2 things. When playing with the addon libdebuglogger which is used for many other addons, the time in game shifts from my local system time to UTC
Also when trying to use mangohud Vulkan reports time in utc but opengl reports local system time
https://lh3.googleusercontent.com/UOJaNfyNyTg7ZNlrD6fCxK4kfHnZI5vabkITO2w2UN05zTGVL2NaAgnsppXXpusKfEo-Pwc8RWmt9POfP3CMxSH9IfTP9dUWmxsI0yeXfN4Mom-X3JW7q_Tenzh9fjdljsCdvr5mrg=w2400
https://lh3.googleusercontent.com/a36uOuiIpgi0iS1WHqysA3f9ikTfRh4mvbxnD05Z-yswLXa-08pEVc0uTHnMUuycZrP5OAVuy-8xCRbKlQZb7aT6Ng_S-ZKd9F37yTlhAOr2FgQBj_vrtAklhP4gLVEUvbhUgJpDvA=w2400
Something seems wonky with dxvk i think? but i dont know anyone else having this issue so maybe its something with my system hardware? i have the latest bios and the bios is on the correct local time.
I filed a request for for with libdebuglogger at esoui because something in that causes the in time game to shift from local to utc but it isnt just there. while running mangohud the time in vulkan driver is reported as utc? but in opengl it is local time?
https://www.twitch.tv/videos/620189454
someone even uderstands what is happening here because i do not... i tried to switch my rtc to local time and rebooted but no improvement...

sigh
As mentioned in your Proton bug report, this has nothing to do with DXVK but with MangoHud.

mangohud is not the only program showing the error, libdebiglogger does as
well
On Thu, May 14, 2020 at 3:10 PM Philip Rebohle<EMAIL_ADDRESS>wrote:

Closed #1628 https://github.com/doitsujin/dxvk/issues/1628.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/doitsujin/dxvk/issues/1628#event-3338039193, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEJCUBXSPIFPFBT3ZFIHXHDRRQ62LANCNFSM4NA6MBBA
.


Ensure access limits are not sent for live items.

Although the publish command destroys any access limit items, we have
seen occasions when the access_limited hash has been populated in the
live content store. This ensures that it will never be sent for
published items, and notifies airbrake if it encounters this situation.

Looks good.


Validator.registerAdapter() doesn't work properly with Retrolambda

We're currently using an anonymous ViewDataAdapter to get the String from the EditText of a TextInputLayout, like this:
validator.registerAdapter(TextInputLayout.class, new ViewDataAdapter<TextInputLayout, String>() {
    @Override
    public String getData(TextInputLayout view) throws ConversionException {
        return view.getEditText().getText().toString();
    }
});

When adding Retrolambda to the project, lint informed us that this could be replaced with a lambda expression, like this:
validator.registerAdapter(TextInputLayout.class, view -> view.getEditText().getText().toString());

However, doing this resulted in the following exception when validation was attempted:
java.lang.UnsupportedOperationException: To use 'com.mobsandgeeks.saripaar.annotation.Length' on 'android.support.design.widget.TextInputLayout', register a 'com.mobsandgeeks.saripaar.adapter.ViewDataAdapter' that returns a 'java.lang.String' from the 'android.support.design.widget.TextInputLayout'.

It appears that Reflector.findGetDataMethod() is interpreting the return type as Object rather than String.

Reproduced and verified this bug. The return type of the anonymous ViewDataAdapter is considered as Object.

Type verification has to be deferred to support lambda expressions.

Was this bug fixed? Just encountered it with version 2.0.0


Feature request: use local files to generate a wallet address

Just throwing a suggestion here. Keeping track of seed words - especially when you deal with multiple wallets - is a pain and not user friendly at all. Also keeping the seed phrase in physical form is as safe as writing your credit card password in a piece of paper and hoping no one will ever find it.
So here is the suggestion
Would it be possible to generate wallet addresses using the entropy generated from any combination of random files a user sends to daedalus (offline?). For example, I'd rather have my wallet generated from the picture I took from my first newborn coupled with the image from a frame of the video of my wedding at 15 minutes and 2 seconds. Add my favorite snippet from a book or part of a song that I like. A seed phrase could still be generated from that and securely stored as usual (and that might be a very long seed), but then that's not a single point of failure anymore.
The benefits of this approach are: you can still use a seed phrase, but you are not forced to. No one knows what images/videos/files you used. You can save everything online in multiple places with a lot of other stuff alongside it (google drive for example), you have a great chance of being able to recover the funds if you forget or lose the seed phrase. It's pretty much impossible for someone to find out what images/video frames/files were used, and in which order. I'm not an expert but this should make the wallet quantum resistant as well as it doesn't depend on random combinations of words that can be guessed.

You can always just run your files through a hashing algorithm to generate the entropy yourself. Generating a recovery phrase from entropy can be done through https://iancoleman.io/bip39/ so it's not impossible to do right now -- just requires a bit more knowledge of how these things work. Daedalus could introduce a feature to make this flow a bit more accessible but you have to be really careful about it because for example somebody might change their picture to a different file format or change the metadata in the image and it would generate a different recovery phrase.

Thanks @SebastienGllmt , there doesn't seem to be an "ADA" or "Cardano" coin option in the page you linked (https://iancoleman.io/bip39/). I'm wondering what would be compatible with a daedalus wallet seed and also a paper wallet seed.
If we could give daedalus multiple random files to generate the entropy and make it ensure it's random enough, that would help a lot.

The coin is only for the BIP32 Root Key section which you don't need. You just need the BIP39 Mnemonic of the right length (12 for Byron-era reward, 15 for Byron-era Yoroi, 15 for any Shelley wallet)


Inventory example

Hi, can you share an example of your inventory please to better understand the HA section.
Good job👍

Hi,
@eshikhov Do you think you could add an example section to the README how you set up your own HA?
btw: Thanks for adding that feature!

enable the parameter
microk8s_enable_HA: true
add a group with hosts to the inventory file
[microk8s_HA]
node1
node2
node3
if you don't like the name of the group microk8s_HA, you can redefine it in the variable microk8s_group_HA for example microk8s_group_HA: "k8s_HA"
[k8s_HA]
node1
node2
node3

Would be great to support pod code, service cidr custom. Using an external load balancer. Etc...

Thanks for adding your example

@metabsd Do you think you could add something like that?

This inventory.yaml works for me:
all:
    multinode_cluster:
      children:
        microk8s_HA:
          hosts:
            node3:
            node4:
            node5:
        microk8s_WORKERS:
          hosts:
            node1:
            node2:

    k8s:
      children:
        single_node_clusters:
        multinode_cluster:


[snapshot] luci-mod-system: implement system.description, system.notes

Implement two new text "options" for UCI system config, intended to
help humans describe the device.
"system.description" is a short, single-line description suitable for
selector UIs in remote administration applications, or remote UCI (over
ubus RPC), etc.  It would also be suitable as a default for LLDP/SNMP
"system description".
"system.notes" is a multi-line, free-form text field that can be used in
any way the user wishes, e.g. to hold installation notes, or unit serial
number and inventory number, location, etc.
Signed-off-by: Henrique de Moraes Holschuh<EMAIL_ADDRESS>

Related to #4839

Thank you!


added detectScale to FlTouchData flag and scale gesture detection option

added detectScale to FlTouchData flag and scale gesture detection option in the touch callback to all charts.
enabling scale detection and pan gesture detection is not supported, so enabling scale detection will disable pan detection and vice versa
related issues:
This PR addresses the problem described in issue #1720 and #71

This needs attention and merge possibly. @imaNNeo could you look when you have time?

As @adamsocrat provided a simpler solution, we will go with #1728
Thanks for contributing!


Latest release bug: pressing on buttons not working all the times

I noticed that in the latest release I have to press the buttons really long to get an effect.
If I press them normally, no button callback is triggered, the alert just disappears.
I believe it was introduced by 3.5.0.
3.3.0 I am unable to install (does not resolve in Xcode).
On 3.1.0 I am unable to reproduce.
My code to show an alert:
.customAlert(confirmationSubtitle, isPresented: $showSelectionConfirmaionAlert) {
   // view code with labels and image
} actions: {
   Button("test 1") {
      // only called when long-pressing button
   }

   Button("test 2") {
      // only called when long-pressing button
   }
}

Update: I was able to get 3.3.0 installed, on this version I am unable to reproduce the bug either.
So I am pretty sure that the issue has been introduced by the latest version 3.5.0.
Hope that helps!

Thank you for opening this issue. Yeah I moved the gesture to the AlertButtonStyle, but that doesn't seem to work properly.
I reverted that change with 3.5.1

Thanks for the quick fix!

I'm on 3.7.0 and still facing this issue :/

Cannot reproduce the issue with iOS 17.5 (both real device and simulator) using the Demo project on the main branch.
@kasem-sm Maybe a specific iOS version is affected? Could you share some details about your setup on how to reproduce this issue? iOS version, Xcode version, Swift version, Code sample, etc.
Thank you in advance


不确定性

score_top, _ = score.topk(k=3, dim=1)

RuntimeError: selected index k out of range 这个怎么解决

There is a problem with the version we uploaded, and the code has been updated. Please refer to the following methods to modify or re-download "data.py" in the root directory.

我们上传的版本有问题，代码已更新。请参考以下方法修改或重新下载根目录下的“data.py”。
0%|          | 0/754 [00:00<?, ?it/s]Epoch [0/400]
Traceback (most recent call last):
File "/home/djj/msk/my code/U-MLP-main/train.py", line 414, in
main()
File "/home/djj/msk/my code/U-MLP-main/train.py", line 351, in main
train_log = train(config, train_loader, model, criterion, optimizer)
File "/home/djj/msk/my code/U-MLP-main/train.py", line 135, in train
iou,dice = iou_score(output, target)
ValueError: too many values to unpack (expected 2)
0%|          | 0/754 [00:01<?, ?it/s]    是不是训练文件也有问题

Please remove the HD95 metric from the metrics.py file, and only return the IoU and Dice coefficients.


ninja use "-std=gnu++1y" in build.ninja while it's only "set(CMAKE_CXX_STANDARD 17)" in CMakeList.txt

I was trying to build TransformerEngine with pip install .
it gives fatal error: filesystem: No such file or directory
while I'm using GCC 11.2.1.
After checking, in the CMakeLists.txt of TransformerEngine it uses set(CMAKE_CXX_STANDARD 17) and in build/cmake/build.ninja the build command for some files using -std=gnu++1y then it cause the error of cannot find filesystem because it's only fully support in c++17 and needs extra flag in c++14.
I want to know why ninja choose -std=gnu++1y when it's set(CMAKE_CXX_STANDARD 17) in CMakeLists.txt and how can I force it to c++17.
I'm using ninja 1.10.2 installed from conda.

It was actually CMake who did that, best to ask here: https://stackoverflow.com/questions/ask?tags=cmake


Need to create IS-KM-5.8.0

Since we do not have  a APIM release parallel t IS 5.8.0 we need to create a custom IS-KM-5.8.0 pack with is-5.8.0 dependencies.
Parallel to this we also need to create the docket image for is-km-5.8.0

This can be used with IS-5.9.0


allow for exceptions to throw more relevant errors

Description
Quick hotfix to allow std::exceptions that are caught to display correct errors
Fixes #
General Checklist:
Please ensure the following points are checked:

[ ] My code follows the coding style
[ ] I have reviewed file licenses
[ ] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ ] I have commented any added functions (in line with Doxygen)
[ ] I have commented any code that could be hard to understand
[ ] My changes do not add any new compiler warnings
[ ] My changes do not add any new validation layer errors or warnings
[ ] I have used existing framework/helper functions where possible
[ ] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[ ] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[ ] This PR describes the scope and expected impact of the changes I am making

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

OK, I'll merge on Tom's recommendation.  LMK if we need to revert.


Transaction list: remove infinite loading

The Chore
Inifinite loading works in some places, but it is not a good system for looking through history for transactions. We should remove it and switch to a manually paged list of transactions
Tasks

[ ] This should change all TX lists - the one on the party page and the one on the transactions page
[ ] Remove infinite loading wrapper
[ ] Remove react-window wrapper around table
[ ] Add more standard pagination controls. As a starting point, let's

[ ] This can be iterated on, but must have:
[ ] Previous page button, which should be disabled on page 1
[ ] Current page number
[ ] Next page button

[ ] Page size should be 25

[ ] A follow up ticket can make this configurable with a dropdown

Notes

This doesn't need to change the layout. That's a separate ticket.
It can change the layout if it's better done all at once.

Sort of semi blocked by https://github.com/vegaprotocol/vega/issues/8545 but I'll implement a hack


PR for evolved enum annotator and manual review components

May combine with #23

use fao_class_enum  from tests/resources/mixs/terms.yaml for tests.


DateTimePicker is not appearing the second time.

Ask your Question

When I press on the input for the first time the Time Picker popup appears and let me select a time, after selecting time the time picker popup hides but when I press the input again the popup doesn't appear. why is this happening??

Please check the issue template before submitting an issue, thanks!

Any solution? Date picker not appearing second time. (Only in android), ios working fine.

I am facing same issue with android device. Can you please help?

Hi,
I am also facing the same issue. Is there any workaround or anything we could get as help!

I was facing the same issue but realised I did not hide the modal manually in the onConfirm handler.
Make sure you set your isVisible variable to false in the onConfirm handler.
Example :
const [isDatePickerVisible, setDatePickerVisibility] = useState(false);

...
const handleConfirm = (date) => {
setDatePickerVisibility(false);
};


Running on Windows... everything installs find with mongo running

I can run node run node server/index.js OK. I open http://localhost:3001/ and it renders Cannot GET /.
Your repo is the closest to what I am looking for to start my foray into reactjs.
I prefer to run on windows for the client app that will hit my Laravel API already setup for CORS.
I see a base.html in the app DIR. What am I missing here?

@LarryEitel Can you run npm run build and then run npm start. Then you can access http://localhost:3000 instead of 3001.
Let me know if that helps.

I hacked package.json to accommodate setting environment variables in Windows:
    "start": "set NODE_ENV=production && node server/index.js",
    "dev": "set NODE_ENV=development && npm run buildDev && node server/index.js",

It works now. Can't think of an elegant way to accommodate linux AND windows for setting environment variables in one line.
I am very happy to get you project running locally. Thank you for sharing!! :)
Do you intend to maintain this project along with your reap version? I am working with MySql so I will be exploring that project for ideas. :)

Moving on... I now run npm run watch / watchDev. Everything seems to initialize however nothing renders when I hit http://localhost:3000/.
This may be informative:
npm run watch

><EMAIL_ADDRESS>watch F:\_vms\HomesteadCode\xapp
> webpack --watch --progress --config ./webpack/webpack.config.js

 69% 220/222 builHash: 8256dd13e9508875fafa
Version: webpack 1.11.0
Time: 4957ms
                               Asset     Size  Chunks             Chunk Names
3f1998aca65909c0bd077ca705e7d25c.png  6.88 kB          [emitted]
ddaf6d6ec871786ec6ee85b2869447b0.png  5.68 kB          [emitted]
751390a95ba609212014ff3c73ebf976.png  5.38 kB          [emitted]
                       app.server.js  88.3 kB       0  [emitted]  app
                    header.server.js  8.65 kB       1  [emitted]  header
                     styles/main.css  4.86 kB       0  [emitted]  app
    + 52 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Hash: e5a16b858b9f9794b4b1
Version: webpack 1.11.0
Time: 6327ms
              Asset     Size  Chunks             Chunk Names
             app.js  1.27 MB       0  [emitted]  app
    styles/main.css   4.9 kB       0  [emitted]  app
         app.js.map  1.49 MB       0  [emitted]  app
styles/main.css.map  12.2 kB       0  [emitted]  app
    + 251 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules
Child extract-text-webpack-plugin:
        + 2 hidden modules

Chrome Network  tab showed failed attempt to load localhost.

@LarryEitel

I am very happy to get you project running locally. Thank you for sharing!! :)

That made my day! Thanks! :)

Do you intend to maintain this project along with your reap version? I am working with MySql so I will be exploring that project for ideas. :)

I will be maintaining this project actively in my spare time. The reap version was more of a proof-of-concept for working with Postgresql. :)

Everything seems to initialize however nothing renders when I hit http://localhost:3000/

If you run npm run watch, you will still need to have another tab open running npm start for the server to start. npm run watch recompiles on any file changes, but does not restart the server.
If you want to try making changes with live reloads, you can just run npm run dev which starts the server and reloads your components as you save.

@choonkending SO CLOSE!!! In one tab I run npm run watchDev and another I run npm run dev. I hit http://localhost:3000/ and BADA BANG it works!!! However, when I make a change to code, although both tabs refresh/rerun their stuff, BADA BOOM!!!. Even if/when I manually refresh or it refreshes with LiveReload, the change is not reflected in the browse UNLESS I rerun npm run dev.
As you can imagine, that would be a very painful workflo...
If you don't work on Windows, I can appreciate where this falls on your list of priorities.
Question: Is there any reason why this won't work on Windows? If it should work, I will run with any ideas/suggestions to work this out. Perhaps someone else tuning in may have an observation.
Thanks again. :)

Ok, got things to a workable state using gulp and LiveReload. Used this tutorial: Tutorial – How to use Livereload with Nodemon in Gulp to Automatically Reload your Browser and Restart your App
I ran: npm install --save-dev gulp gulp-livereload gulp-notify gulp-nodemon express (express can be excluded)
Here's my adapted gulpfile.js file:
// Dependencies
var gulp = require('gulp');
var nodemon = require('gulp-nodemon');
var notify = require('gulp-notify');
var livereload = require('gulp-livereload');

// Task
gulp.task('default', function() {
    // listen for changes
    livereload.listen();
    // configure nodemon
    nodemon({
        // the script to run the app
        script: 'server/index.js',
        ext: 'js'
    }).on('restart', function(){
        // when the app has restarted, run livereload.
        gulp.src('server/index.js')
            .pipe(livereload())
            .pipe(notify('Reloading page, please wait...'));
    })
})

So with LiveReload watching my app directory and enabled, in one tab I run npm run watch and in another tab I run gulp. Browse to localhost:3000. Make a change to code like in app/base.html, save and watch browser reload with change reflected.
This is in production mode. So next will be to try in dev mode to get sourceMaps.

@LarryEitel Sorry it took months (literally) to get back on this. Let me know if you still need help with the current version of the repo. Closing for now. :)


update to 2023 :)

use golang-image builder step
update to recent alpine linux version
deactivate senseless unit test

[X ] mage test passes
[ ~] unit tests are included and tested

LGTM thank you @colpari


docs: update the azure video

Add Azure video to docs
This is a

[x] Doc improvement

Thank you, @royra


Updated ru_ru.json

changed translation for corn kernel bag

Thank you, I will manually change it myself


Preserving scroll position when Link is clicked

Now the blueprint uses scroll-behavior and we always scroll to top on page change.  Normally this is great, but I have a case at the moment where I don't want that.
I've noticed that scroll-behavior has a new feature that makes it possible to prevent the scroll position from changing via a callback.
I've worked out how to edit the creation of the history object in client.js to call this:
const history = useScroll(() => browserHistory)({
  shouldUpdateScroll: () => {
    return shouldScroll;
  }
});

But I can't work out how to pass this down stream so that shouldScroll can be set in the relevant containers/components when clicking on a Link tag. For example could I attatch a reference to a setter in the props for example?
Does anyone know how I would do that?

@skywickenden
This maybe a solution:

in client.js

import {getValue} from 'redux/scrollState';
const pool = {};
const history = useScroll(() => browserHistory)({
  shouldUpdateScroll: () => {
   //BUT, how to get current path from url???
    return getValue(path);
  }
});
const dest = document.getElementById('content');
const store = pool.store = createStore(history, client, window.__data);

2.create a scrollState.js file to config scroll state
const info = {
  patients: false
};

export function getValue(key) {
  return info[key];
}

export function setValue(key, value) {
  info[key] = value;
}

Thanks @tearsofphoenix
I adapted it slight as so:
In client.js
import { getScrollToTop, setScrollToTop } from './helpers/scrollHelper';

const history = useScroll(() => browserHistory)({
  shouldUpdateScroll: () => {
    const scrollToTop = getScrollToTop();
    setScrollToTop(true);
    return scrollToTop;
  }
});

In /src//helpers/scrollHelper.js
let scrollToTop = true;

export function getScrollToTop() {
  return scrollToTop;
}

export function setScrollToTop(value) {
  scrollToTop = value;
}

In any component:
...
import { setScrollToTop } from 'helpers/scrollHelper';
import { Link } from 'react-router';
...
  render() {
    return(
      ...
      <Link onClick={setScrollToTop(false)} >A link</Link>
      ...
    );
  }
...

This feels dirty. But it works, and since the flag is only set temporarily it doesn't effect Redux state or the history adversely.


QPPSE-317: Refactor decocer/encoder static variables

Information

Refactored decoder and encoder constant static variables to a common file

Checklist

[ ] All JUnit tests pass (mvn clean verify).
[ ] New unit tests written to cover new functionality.
[ ] Added and updated JavaDocs for non-test classes and methods.
[ ] No local design debt. Do you feel that something is "ugly" after your changes?
[ ] Updated documentation (README.md, etc.) depending if the changes require it.

pretty much puts the static variables into a model package, looks good to me


Event processing host errors

I've tried to use @azure/event-processor-host": "1.0.2
I've created new event hub with 32 partitions and set access policies to manage.
Then I tried to initialize 4 instances of EventProcessorHost that I created from connection string and all of them were pointing to same event hub (I think this should still work or do they need to be on separate computers or processes to test?).
I then started all 4 of them and started receiving some packets that were sent to this event hub.
Even though some packets were received there were tons of errors reported until my service crashed with transfer after detach error.
Here are the list of different errors I received - most of them were repeated a lot of times:
MessagingError: The messaging entity 'streams:eventhub:test-hub~2047|consumers' could not be found. TrackingId:93b327c5-2dab-43d7-beb8-635e6c68f53d_B17, SystemTracker:streams:eventhub:test-hub~2047|consumers, Timestamp:9/20/2018 1:39:05 PM TrackingId:012a846c3e644424a5bef547be9eeb51_G30, SystemTracker:gateway5, Timestamp:9/20/2018 1:39:05 PM

[connection-2] error on read: ProtocolError: attach received on invalid channel 2  (buffer:0x00,...)

[connection-3] error on read: ProtocolError: detach received on invalid channel 4  (buffer:0x00,...)

MessagingError: New receiver with higher epoch of '5' is created hence current receiver with epoch '4' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:ff92ddfe000003c30004c7925ba3a318_G9_B20, SystemTracker:streams:eventhub:test-hub~24575|$default, Timestamp:9/20/2018 1:39:41 PM

MessagingError: The request with message_id "706920c4-d036-415c-900a-97514c05e7bc" to "$management" endpoint timed out. Please try again later.

MessagingError: The request with message_id "340e306b-fadf-4c82-abdf-8f70b0e8273c" to "$cbs" endpoint timed out. Please try again later.

MessagingError: Unable to create the amqp receiver 7ed373cf-2922-4f7d-b271-c87daef60fdd due to operation timeout.

MessagingError: Unable to create the amqp session due to operation timeout.

MessagingError: Unauthorized access. 'Listen' claim(s) are required to perform this operation. Resource: 'sb://streams.servicebus.windows.net/test-hub/consumergroups/consumers/partitions/17'. TrackingId:8b064d004b2f46e6bc041e7c319c6ed8_G4, SystemTracker:gateway5, Timestamp:9/20/2018 1:45:08 PM

MessagingError: A link to connection '275148' $management node has already been opened.

\event-processing-service\node_modules\rhea\lib\session.js:360
console.pub.js:42
        throw Error('transfer after detach');

Since node.js is single threaded, I would recommend creating different processes. In that way the event loop does not get stressed between receiving messages for all the eph instances and managing leases for them as well.
As explained in the overview doc, eph uses the epoch receiver feature of event hubs. This allows EPH to ensure that at any given time, only 1 receiver will be actively listening from a partition within the provided consumer group.
Hence, this error
MessagingError: New receiver with higher epoch of '5' is created hence current receiver with epoch '4' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:ff92ddfe000003c30004c7925ba3a318_G9_B20, SystemTracker:streams:eventhub:test-hub~24575|$default, Timestamp:9/20/2018 1:39:41 PM

makes sense. Because another EPH instance was trying to load balance, it created an epoch receiver with a higher epoch value, which caused the receiver from the previous EPH to get disconnected. I will send a PR to not bubble up such errors as well. This should avoid confusion.
MessagingError: The messaging entity 'streams:eventhub:test-hub~2047|consumers' could not be found. TrackingId:93b327c5-2dab-43d7-beb8-635e6c68f53d_B17, SystemTracker:streams:eventhub:test-hub~2047|consumers, Timestamp:9/20/2018 1:39:05 PM TrackingId:012a846c3e644424a5bef547be9eeb51_G30, SystemTracker:gateway5, Timestamp:9/20/2018 1:39:05 PM

AZ:  This looks like it was not able to find the event hub.
[connection-2] error on read: ProtocolError: attach received on invalid channel 2  (buffer:0x00,...)

[connection-3] error on read: ProtocolError: detach received on invalid channel 4  (buffer:0x00,...)

AZ: Would look into these two. It looks like a lower level protocol error.
MessagingError: The request with message_id "706920c4-d036-415c-900a-97514c05e7bc" to "$management" endpoint timed out. Please try again later.

MessagingError: The request with message_id "340e306b-fadf-4c82-abdf-8f70b0e8273c" to "$cbs" endpoint timed out. Please try again later.

MessagingError: Unable to create the amqp receiver 7ed373cf-2922-4f7d-b271-c87daef60fdd due to operation timeout.

MessagingError: Unable to create the amqp session due to operation timeout.

AZ: These errors are due to everything being run in the same process. The event loop is busy doing too many things. Hence the operations timeout.
MessagingError: Unauthorized access. 'Listen' claim(s) are required to perform this operation. Resource: 'sb://streams.servicebus.windows.net/test-hub/consumergroups/consumers/partitions/17'. TrackingId:8b064d004b2f46e6bc041e7c319c6ed8_G4, SystemTracker:gateway5, Timestamp:9/20/2018 1:45:08 PM

AZ: This one looks like the cbs token expired and the client was not able to renew token. Token renewal happens at an interval of 45 minutes (token expires in 60 minutes). Which again indicates that the event loop was busy doing too many things.
MessagingError: A link to connection '275148' $management node has already been opened.

\event-processing-service\node_modules\rhea\lib\session.js:360
console.pub.js:42
        throw Error('transfer after detach');

AZ: This error should not have happened. Will have to investigate further.

@MatejSkrbis - New version of @azure/event-processor-host: 1.0.3 has been published which fixes this issue.

Closing this issue as fixed.


 Please try again.
	status code: 400, request id: 67b61c29-cd84-4b01-bbd8-3e645d2c084e '/aws/lambda/notify_slack'

  on .terraform/modules/notify_slack/terraform-aws-modules-terraform-aws-notify-slack-415fe44/main.tf line 26, in resource "aws_cloudwatch_log_group" "lambda":
  26: resource "aws_cloudwatch_log_group" "lambda" {

I thought may be resource already exits. so ran terraform destroy to give a clean restart and verified manually. Again, failed when it is tried to create log Group which is correct as it already exists. Any help will be appreciated. :)

UPDATES
I rolled back the tag to 2.3.0  and able to setup the resources.
May be we can close this but curios on why it failed on master? :)


big fix: last prompt is duplicated in jsonMap['messages'] in file groq_chat.dart

Logging  jsonMap['messages'] after line 76 in groq_api.dart, we can see that the last prompt sent is repeated.
This is because in line 75  messages.add(prompt.toJson()); we add the prompt to our of messages, but from line 415   _chatItems.add(RequestChatEvent(request)); in groq_chat.dart we already added the prompt to our list of chatItems.

Thank you for the fix! 👍

I can add the new feature in open issue if you'd like


.NET 6.0 Isolated functions' in Visual Studio 2022: WorkerConfig for runtime: dotnet-isolated not

When I am running the .NET 6.0 Isolated functions' locally in Visual Studio 2022 i am getting the error:
A host error has occurred during startup operation 'dcaec2a4-21c8-4c41-a4e2-cb06e5f32e64'.
[2022-04-25T10:59:39.918Z] Microsoft.Azure.WebJobs.Script: WorkerConfig for runtime: dotnet-isolated not found.
Value cannot be null. (Parameter 'provider')

I am running Visual Studio 17.1.5 on Window 10

Project can be found in this GitHub Repo: FunctionAppTest

Thanks for providing the GH repo for repro.  I tried it in 2 different Windows 11 machines running VS 17.5.1 and it worked fine without any issues.
@sulmanm Could you check whether azurite is running when you are getting this error. You could also try using a valid azure storage account connection string as the value of AzureWebJobsStorage setting in local.settings.json and see that works. That information will help us to investigate this further.
@satvu Could you also try the repro on your machine?

I just tried the repro on a Windows 10 machine with the same OS and version as yours @sulmanm and it worked for me - here's some extra info on my setup:
Azure Functions Core Tools
Core Tools Version: 4.0.4483 Commit hash: N/A  (64-bit)
Function Runtime Version: <IP_ADDRESS>73
VS Version 17.1.5

@sulmanm Is it possible for you to try your repro in a second machine/environment?

You can open a command prompt/terminal and run azurite command. If it is not already running, it will start.

After manually starting azurite, try running the azure functions project and see if that works.
Ideally, Visual studio should start azurite automatically for you when you do F5 on an azure functions project.  If that is not happening, perhaps something somewhere is corrupted. Can you try repairing your VS installation and see that helps?

azurite is already running.

I think there's 2 things that we need to look at to narrow down the issue:

Is the build output on the "bad" machine correct?
If so, is there something in the environment that's causing the config to not be found on startup

To see if the output looks correct, can you try taking the build output (the whole /bin folder) from one of your "working" machines and copy it to your "broken" machine. Then try to use Functions Core Tools to run it? Navigate to the folder and run func host start.
If that also gives the same error, can you try running dotnet --info on the two machines to compare and see if there's something that looks different? You can even paste that here.
If that doesn't work, we may have to have you file a support ticket so we can get some more detailed information from your setup to try to narrow things down.

Having this issue as well, also tried via docker
Preface: My company is moving towards the .NET 6 LTS and making sure we are using the out-of-process model as it seems that will be the way of things here soon.  So this is our first Function app migration to .NET 6 and out-of-process since .NET Core 3.1 with in-process model (using func v3).
The setup here was using VS 2022 Enterprise, selected the azure function template to create the project. This included docker support out of the box.  Made some edits to that dockerfile to use the net6.0 instead of core 3.1.  Our web apps across the board use Serilog, so those dependencies were added as well, and program code modified to be as similar to our web app approach as possible.  Though it seems app insights may be provided through a default provider and unchangeable, so that is currently in our serilog config pending investigation.
Win 10 Pro (Version	10.0.19044 Build 19044)
VS 2022 Enterprise 64 bit (17.2.1)
Core Tools Version:       4.0.4544 Commit hash: N/A  (64-bit)
Function Runtime Version: <IP_ADDRESS>86
Azurite appears to be running
Error message:
A host error has occurred during startup operation '024d099f-24d1-4b09-9132-6b7276ecd4af'.
Microsoft.Azure.WebJobs.Script: WorkerConfig for runtime: dotnet-isolated not found.
Failed to stop host instance 'c9a932c3-01c2-4790-882b-37b174235a8e'.
Microsoft.Azure.WebJobs.Host: The host has not yet started.
Value cannot be null.


Webcontext can start with /

The doc currently reads:

The web-context is a string that uniquely identifies the web context of a request to be dispatched to this service. It MUST NOT start with a / character. It can have / characters inside as segment separators in case of multi-segment contexts. And it is allowed to be "" for root context. If multiple services match the request, the longest context match takes precedence.

We should actually allow it to start with a /.  Otherwise, it might get annoying while troubleshooting in development.

Yes, this just bit me for about 30 minutes.  I figured it out by changing the value until it worked.  However, I did want to find out if it was documented.  The inability to search the docs made that painful.  I did find the relevant doc by doing a github search.
If you really don't want, I'd consider throwing an exception or logging an error/warning.

I agree, we should either throw an exception or should support /.  We need to re-enable search, please see https://github.com/paypal/squbs/issues/534.


evalCodeBlocks should execute blocks in parallel and display loading indicators

This should make the script much faster and provide more useful feedback.

Extracted these issues from the original description: #36, #37, #38, #39, #40, #41, #42
I repurposed this issue to be about the last remaining item.


[OpenVR] left-right buttons are swaped

Hey,
Playing with the project in the thought of either forking it to something new, or just learning Godot.
There seems to be a left-right controller issue.

left saber is blue
right trigger triggers left saber click
Heptics are swapped

Using Godot on linux with HTC Vive.
Any idea where this is set up  in the code so I can see how to fix this?

Default Controller Button Mappings (For Oculus Quest Touch Controller) are here: https://github.com/NeoSpark314/BeepSaber/blob/52c7ed298de7f9d16a1917c18f88117ab9e5a278/OQ_Toolkit/vr_autoload.gd#L183
BeepSaber uses an older version of the OQ_Toolkit; in the current version I have a function to remap buttons: https://github.com/NeoSpark314/godot_oculus_quest_toolkit/blob/51fe3b7ccf0c4f5d41e0b375b5b5aa18e3812799/OQ_Toolkit/vr_autoload.gd#L248 but it does not yet have specific code for the Vive; if this is really a consistent issue with godot that the contorller button mappings do not match between vive and touch this could be a place to fix it.


Return more usefull failure message for Token.NEWLINE_PARSER

Token.NEWLINE_PARSER returns a newline or carriage return as failure message. This is not very useful because it moves the cursor to the beginning of the line. In my project I'm using a logger class which writes timestamps and class names at the beginning of the line. Because of the carriage return this gets overwritten. Example:
Validator version: 2.1.6-SNAPSHOT
' expected40 ERROR massbank.Validator - '
16:49:00.343 ERROR massbank.Validator -   99.00676994602972 57.0 57.0
16:49:00.347 ERROR massbank.Validator -                            ^
16:49:00.347 ERROR massbank.Validator - Error in 'records/recdata/BU000001.txt'.

More readable and understandable would be:
Validator version: 2.1.6-SNAPSHOT
17:46:28.226 ERROR massbank.Validator - 'NEWLINE' expected
17:46:28.230 ERROR massbank.Validator -   99.00676994602972 57.0 57.0
17:46:28.236 ERROR massbank.Validator -                            ^
17:46:28.236 ERROR massbank.Validator - Error in 'records/recdata/BU000001.txt'.

Coverage remained the same at 99.02% when pulling ee14b4be74a9ae2f15dfd5308abcd69d80196ab3 on meier-rene:master into c16ef478a4927ac6da48551619d4110758877475 on petitparser:master.

Thank you for your contribution, feel free to add yourself to AUTHORS.

I now added tests; and made it more general so that all character parsers have readable error messages: 1c2a6e11b0c5df79075a84dfb8a255d3fea1a845


Add configuration refresh/ dynamic configuration for python provider

@PoonamS25 currently the only way of do this is by calling the load operation multiple times. But it is the next feature we plan on adding.

@PoonamS25 we have released a beta version of this feature https://github.com/Azure/AppConfiguration-Announcements/issues/32.

Released https://github.com/Azure/AppConfiguration-Announcements/issues/35 version 1.1.0. Let us know if you have any question.


fix doc build issues

Docs now build without errors with RUSTDOCFLAGS=-Dwarnings cargo doc
If we want, we can add a check for this to CI

LGTM.  Other than the merge conflicts with main,  any reason we are holding off on the merge?

Nope, this one can merge


Open Street Map and validation buttons in the same webpage: misleading?

This is for @peterkwells in particular, quickly discussed in the office with @Floppy, @pezholio and @pikesley .
See the screenshot below, showing the current styling of the page we use to get user validation for an address.

I see two problems with it:
a) It may look like we're asking people to validate the address' existence vs the map. It could be read as "is the address where the pointer on the map is?" or "you are a human so you can easily tell if the pointer does or does not fall on no. 65: can you confirm that?"
b) Even if (a) wasn't true, it makes the map a "part" of the validation process, to the point of saying that the validation information is derived work from the map, from a licensing perspective.
What do you think?
Two possible way of solving that: more wording (booo) or hiding the map in the previous page, when one clicks on "Help us improve the UK's address data! Tell us what you know about this address.".

Good catch, the latter solution (drop the map at this point in the journey) makes sense to me.
Obviously the next step is to flip to a map which is compatible with CC-BY-4.0, possibly even serving that up to those who need it. As verbally discussed this could be "John's map". A roadmap feature is being raised to reassure people that "John's map" doesn't mean John quickly flipping through his collection of paper maps, snapping an image, uploading it and then pressing "GoGoGo" to load the screen/API response.

@floppy see above. For the time being, hiding the map after one clicks on ""Help us improve the UK's address data! Tell us what you know about this address." will be the solution.
The longer term solution is on the roadmap as https://github.com/OpenAddressesUK/roadmap/issues/83 .

This is irrelevant now, as we're using OS instead.


Update Bootstrap.ps1

Without git installed, the DataServicesLogonScript.ps1 will throw an unexpected error:
Failed to initialize: Bad git executable.
The git executable must be specified in one of the following ways:

be included in your $PATH
be set via $GIT_PYTHON_GIT_EXECUTABLE
explicitly set via git.refresh()

Hi @YRollHid thanks for your contribution.
Before we merge the PR in (although it's non intrusive), I'd like to better understand where you ran into this error - i.e. where git is being referenced in the script.
Are you able to share a screenshot/reference which line in DataServicesLogonScript.ps1 you ran into this error message?

@YRollHid heads up - we've reproduced this issue elsewhere as well and are isolating the RC - thanks for bringing this up

@YRollHid thank you. We are currently working with the Azure Arc and CLI product teams on a globally fix rollout as this issue impacting all Jumpstart scenarios that leveraging the k8s-extension cli functionality. I will update this thread once we will have it but will not be merging this PR for now as it offers a temp solution and solved the issue at it's core (as mentioned).
You can also see another reference to the issue in the Azure CLI repo here

@YRollHid you should be able to successfully re-run the scenario (without the choco install git) successfully. Please let us know if you run into any issues/needed support.


helm installation with sentinel cluster and broker-transport-option

Hello. I am gonna deploy celery-exporter by using helm. I have 3 nodes redis sentinel cluster. So how can apply CE_BROKER_URL for sentinel cluster, for one redis I just give the one IP but here no idea and also how can I apply "broker-transport-option master_name=my_master" for example. By using docker commands I can do it but how can apply these values with helm installation. For example should I use CE_BROKER_TRANSPORT_OPTION as an env in values yaml?

@adinhodovic helm expert

Hi there 👋 I am trying to do the same thing but can't make it work. I used this values:
    env:
    - name: CE_BROKER_URL
      value: sentinel://redis-node-0.redis-headless.redis.svc.cluster.local:26379/0;sentinel://redis-node-1.redis-headless.redis.svc.cluster.local:26379/0;sentinel://redis-node-2.redis-headless.redis.svc.cluster.local:26379/0
    - name: CE_BROKER_TRANSPORT_OPTION
      value: 'master_name=default-master'
    - name: CE_LOG_LEVEL
      value: DEBUG

The config looks good, the exporter start as expected, cf the logs:
2023-05-31 14:15:34.116 | DEBUG    | src.exporter:run:255 - Setting celery broker_transport_option master_name=default-master
2023-05-31 14:15:34.138 | INFO     | src.http_server:start_http_server:66 - Started celery-exporter at host='<IP_ADDRESS>' on port='9808'

But I get no metrics exported on /metrics compared when I use the following (I deployed a small haproxy that target the redis leader)
    env:
    - name: CE_BROKER_URL
      value: redis://haproxy.redis.svc.cluster.local:6379/0

At this point I don't really know how to dig the issue, if someone have any idea I'm all ears

Using the redis-sentinel bitnami chart, for me the following worked:
env:
- name: CE_BROKER_URL
  value: "sentinel://redis-sentinel.web.svc.cluster.local:26379"
- name: CE_BROKER_TRANSPORT_OPTION
  value: "master_name=redis-master"

I am able to see the metrics through the Grafana dashboards

Work well with version 0.10.10 but not 0.10.L7

I have a question regarding Helm installation to connect to a Sentinel cluster as well, how would the equivalent helm values env look like for the example provided connection with docker? Is it possible to declare multiple --broker-transport-option through the env values for Helm chart installation? I am having difficulty passing in the password for the sentinel_kwargs.
docker run -p 9808:9808 danihodovic/celery-exporter --broker-url=sentinel://sentinel.service.consul/1 \
  --broker-transport-option master_name=my_master \
  --broker-transport-option sentinel_kwargs="{\"password\": \"sentinelpass\"}"

Helm values:
env:
- name: CE_BROKER_URL
  value: "sentinel://sentinel.service.consul/1"
- name: CE_BROKER_TRANSPORT_OPTION
  value: "master_name=my_master"
# how can --broker-transport-option sentinel_kwargs="{\"password\": \"sentinelpass\"}" be defined?

@zqyong you can try cloning the project, starting the docker containers running the Python program to see how the --broker-transport-options translate to environment variables. I'm not sure myself.

Yeah I poked around and found out how it can be done, to anyone trying to pass multiple broker_transport_option values through a single environment variable this can be done by:
env:
- name: CE_BROKER_URL
  value: "sentinel://sentinel.service.consul/1"
- name: CE_BROKER_TRANSPORT_OPTION
  value: "master_name=my_master"
- name: CE_BROKER_TRANSPORT_OPTION
  value: 'master_name=my_master sentinel_kwargs={"password":"sentinelpass"}'

the values are parsed by Click, and by default Click uses whitespace as a separator. Thank you @danihodovic for your quick reply and the great work!


Why can't I find the fa-compress class?

Why can't I find the fa-compress class? I can't find the fa-compress class, so some of my ICONS don't show up.

OK,i got it. Need npm install font-awesome

Yes, use fontawesome without writing more CSS. But you also write a css to display icon yourself instead of using fontawesome.


Change ingestor scaling to match loggregator recommendation of

1 to 1 with traffic controllers
also fixes typo in redis worker count recommendation
[#166452522]

Thank you!


Refactor type specification

Changes
Refactored the type specification methods to its own method.
Issues
None
Additional information
It supports optional or non optional use like in declaration (optional) and parameter (non optional)

warning: unneeded `return` statement
   --> tutara-interpreter/src/parser/parser.rs:369:3
    |
369 |         return None;
    |         ^^^^^^^^^^^^ help: remove `return`: `None`
    |
    = note: `#[warn(clippy::needless_return)]` on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return

Can you make mister clippy happy?


Fixed #28805 -- Added regular expression database functions.

This is a work in progress attempt to implement RegexpReplace(), etc. for ticket-28805.
Backend Support:
PostgreSQL:

regexp_count(expr, pattern, position?, flags?) -- v15+ only
regexp_instr(expr, pattern, position?, occurrence?, return_opt?, flags?, subexpr?) -- v15+ only
regexp_replace(expr, pattern, replacement, position?, occurrence?, flags?)

position and occurrence for v15+ only

regexp_substr(expr, pattern, position?, occurrence?, flags?, subexpr?) -- v15+ only
substr(expr from pattern) -- required for <v15

MySQL:

REGEXP_REPLACE(expr, pattern, replacement, position?, occurrence?, flags?)
REGEXP_SUBSTR(expr, pattern, position?, occurrence?, flags?)
REGEXP_INSTR(expr, pattern, position?, occurrence?, return_opt?, flags?)

MariaDB:

REGEXP_REPLACE(expr, pattern, replacement)
REGEXP_SUBSTR(expr, pattern)
REGEXP_INSTR(expr, pattern)

Oracle:

REGEXP_REPLACE(expr, pattern, replacement, position?, occurrence?, flags?)
REGEXP_SUBSTR(expr, pattern, position?, occurrence?, flags?, subexpr?)
REGEXP_INSTR(expr, pattern, position?, occurrence?, return_opt?, flags?, subexpr?)
REGEXP_COUNT(expr, pattern, position?, flags?)

SQLite:

re.sub(pattern, replacement, expr, count?, flags?)

Implementation:

RegexpReplace(expression, pattern, replacement, flags)
RegexpSubstr(expression, pattern, flags)
RegexpStrIndex(expression, pattern, flags)

Notes:

position is only supported on MySQL, Oracle, and PostgreSQL 15+ and is used to indicate where matches should begin. Support for this will not be implemented, save to pass in the default value of 1 where subsequent function arguments are required.
By default PostgreSQL replaces the first occurrence only unless 'g' is passed in flags to replace all occurrences. MySQL & Oracle replace all occurrences by default (occurrence=0)  or the specified occurrence. SQLite, using Python's re.sub() will replace all occurrences by default (count=0) or up to count occurrences in the string. MariaDB only supports replacing all occurrences.
We will pass 1 to the underlying count/occurrence parameter by default and accept 'g' in flags to pass to PostgreSQL or 0 to the underlying count/occurrence. MariaDB will ignore the 'g' flag as it will always replace everything.
return_opt is only supported on MySQL and Oracle for REGEXP_INSTR and is used to control which position value is returtned. Support for this will not be implemented, save to pass in the default value of 0 where subsequent function arguments are required.
The case-sensitive ('c') and case-insensitive ('i') flags are supported by most backends. It seems that later flags specified take precedence over earlier ones. SQLite, using Python, is always case-sensitive by default and only supports 'i', but if 'c' is present after 'i', we can cancel the case-insensitivity.
MariaDB doesn't support being passed only supports inline flags. We can support 'c' and 'i', by prefixing pattern with (?-i) and (?i) respectively. It also seems that MariaDB is case-insensitive by default (unless we have some weird collation configuration on the Django CI).
MySQL also seems to be case-insensitive by default.
The multi-line flag ('m') seems to work similarly across all backends. PostgreSQL supports the value 'm', but the canonical value for the flag is 'n'.
The dotall flag ('s') seems to work similarly across all backends, but is the default on PostgreSQL. We may want to pass the 'p' flag by default for PostgreSQL to get it to behave like other backends -- it is also documented. Oracle and MySQL use the value 'n' for this flag instead.
The extended flag ('x') is not supported by MySQL. Oracle does not support comments in the pattern when using 'x', but all other backends do.

Issues:

~MySQL tests are currently skipped as Django CI doesn't have 8.0.4+.~ (No longer a problem - this was started so long ago!)

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

The main issue for me is that's it's not adjustable for 3rd-party database backends. Ideally, we could add a several hooks and call them in as_{vendor} methods instead of checking multiple vendors in as_sql().

So I've rejigged this in a way that should address this concern. Feel free to review those changes.
I'm now just waiting for the oracle tests to complete to see if I still get the failures I was experiencing. I suspect that I'll have to follow the advice recently added in 9369f0cebba1f65909a14dec6aa3515ec1eb2557.

I'm now just waiting for the oracle tests to complete to see if I still get the failures I was experiencing. I suspect that I'll have to follow the advice recently added in 9369f0c.

So the failures with Oracle remain:
db_functions.text.test_regexpreplace.RegexpReplaceFlagTests.test_dotall_flag
db_functions.text.test_regexpreplace.RegexpReplaceFlagTests.test_extended_flag
db_functions.text.test_regexpreplace.RegexpReplaceFlagTests.test_global_flag
db_functions.text.test_regexpreplace.RegexpReplaceFlagTests.test_multiline_flag
db_functions.text.test_regexpsubstr.RegexpSubstrFlagTests.test_dotall_flag
db_functions.text.test_regexpsubstr.RegexpSubstrFlagTests.test_extended_flag
db_functions.text.test_regexpsubstr.RegexpSubstrFlagTests.test_multiline_flag

I've noticed that the failing tests all have one thing in common: They pass Article.text as their first argument - a TextField - and not Article.title as the other tests use - a CharField.
@felixxm I'm wondering whether this is a more general problem with Oracle? I see that the majority of tests for text database functions don't test with TextField inputs. You mentioned being unable to reproduce the issue with LPad or Left, but were you using a CharField? Do you get the same issue if you pass in a TextField, e.g. Article.text, as the first argument? (I don't currently have an Oracle instance to test...) :detective:

buildbot, test on oracle.

buildbot, test on oracle.

@felixxm This has been nearly ready for a long time so I thought I'd try to finish it.
I've tested it out with the change that you suggested in https://github.com/django/django/pull/12438#discussion_r472006730 and all the tests pass.
It doesn't appear to affect anything else. Is there any reason why that wouldn't be acceptable?

Thanks for all you efforts +1 and really sorry for the late reply.

It happens!

I'm still a bit skeptical about juggling so many database-specific flags, it's can be really hard to maintain. Personally, I'm -0 about this feature.

After two years of keeping this going with no discouragement, I hope we don't jump to a no too quickly...
We'll have to see if we can break things up further to deal with the flags better.

I left some comments, but we need to make few more things to make it reviewable again:

Have rebased, blackened, re-targeted to 4.1, and addressed the majority of comments. A couple of the comments will require a bit more thought, however.

As an aside, PostgreSQL 15 looks like it is going to learn REGEXP_INSTR, REGEXP_REPLACE and REGEXP_SUBSTR - among others - which will eventually allow some of the hacks for PostgreSQL to be phased out:

https://www.postgresql.org/docs/devel/functions-string.html
https://commitfest.postgresql.org/34/3042/

buildbot, test on oracle.

buildbot, test on oracle.


HTTP

Pinpoints and Map Features are not working with DJI Pilot 2 App Android: V<IP_ADDRESS>

#10 Maybe your problem is the same.

Hi @DJIsean, Any plan when the new Pilot2 (that supports HTTP) will release?

Pilot 2 was suppose to be fixed. BUT ITS NOT!!!!!  <IP_ADDRESS>

Is the screenshot attached the same issue?

HTTP is already supported in Pilot2 version V6.1 or higher. https://www.dji.com/matrice-30/downloads


[BEAM-9110] Bump jboss-marshalling to 2.0.10.Final

Bump jboss-marshalling to 2.0.10.Final

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java_Examples_Dataflow PreCommit

R: @iemejia
It's used in grpc and has passed the test so it seems safe to update

I am not sure how can we properly test this upgrade given that this is in the vendoring dependency, and fully testing it would probably require a release of the vendored dependency.

@iemejia Thank you for your comment! I see.
I found suztomo-san doing this kind of PR right now. It may be better to wait for this to end.
https://github.com/apache/beam/pull/14028
If you know of any other people you should consult with, please let me know.  🙏

@masahitojp - Would you be able to look at the last question from @suztomo ?

Now jboss-marshalling has been removed from the vendored gRPC dependencies. I think you don't need this PR any more.


Consider fixing V8 warnings

#336 disabled warnings in two files pertaining to V8 in order to enable building with warnings treated as errors. Ideally, we would want to fix these warnings rather than just turn them off, but there are complications in both cases.

In js_native_api_v8.cc, that file may also exist in other repositories and for that reason we may wish to be cautious about introducing changes to it.
In AppRuntimeV8.cpp, the errors I turned off were coming from files inside the V8 NuGet package, over which we have no direct control whatsoever, so I'm not sure there's a better way for us to handle that situation than to simply turn the errors off.

We should take a new version of N-API from the node.js repo before even considering fixing anything here.

Most likely V8 has already fixed all of these. We just need a newer version.

See https://github.com/pmed/v8-nuget/issues/34.


Datapoint UI

Changed the UI of the datapoints to make them easier to view

Conflicting binaries, will recommit


UWP - Icons not displaying for contribution areas

Appears to be an issue after build drop. No icons are being displayed for the MVP contribution areas.

Appears this also affects the MVP status page also.

Possible missing ms-appx prefix?

Possibly. It may also be due to the fact that I only appear to have generated scale factor 400 images. Everything works locally when building through VS. Can take a look at this.

I believe this to be now fixed in the current master branch having tested the package output for the store. Will hold off on closing this off and publishing out for now.

This has been pushed out in the latest release and appears to be working fine.


AirPlay Devices over tunnel?

Is it possible to use AirPlay devices lets say from network two with the Roon Core on network one?

I honestly don't know.  Maybe you could try it and report back!

Closing since no further contact with user


Receiving frequent errors after upgrading to 0.1.1

Getting this error several times a night, which is only resolved with a hardware reset of the Mini DSP 2x4HD: error; EzBeq.getdevices failed, HTTP status 500. Log entry indicates the following: ProcessTimedOut: ('Process did not terminate within 10 seconds', ['/usr/bin/minidsp', '--tcp', '<IP_ADDRESS>']).

Im assuming (because of the ezbeq reference) that you're running this on a raspberry pi and have installed the debian package, is that right?
When the problem occurs, are you able to run minidsp from the console?
If not, does journalctl -u minidsp show any errors?

Yes, I am running on a Raspberry Pi 4 and the Rasbian Pi (Debian) OS. When I got the issue, I lost connectivity and had to reboot the mDSP to get connectivity back. I reverted to version 0.0.8 earlier today so can’t run any commands to test. Sorry about that… Thanks for all your great work! I can’t tell you how much it means to be able to wirelessly access m mDSP from my laptop! It’s awesome! I never was able to get a wi-dg to work because its IP conflicted with my cable modem and I was stuck using usb until you came out with mini dsp-rs.
Sent from Yahoo Mail for iPad
On Wednesday, June 16, 2021, 5:51 PM, Mathieu Rene @.***> wrote:
Im assuming (because of the ezbeq reference) that you're running this on a raspberry pi and have installed the debian package, is that right?
When the problem occurs, are you able to run minidsp from the console?
If not, does journalctl -u minidsp show any errors?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Closing since this had no activity for a long time, and no other reports.


Unused locale argument in Tokenizer.DefaultTokenizer

The optional string arg locale is not used at all in Tokenizer.DefaultTokenizer.
public class Tokenizer
{
    public static List<Token> DefaultTokenizerImpl(string text, string locale = null)
    {
        var tokens = new List<Token>();
        Token token = null;

        // Parse text
        var length = text != null ? text.Length : 0;
        var i = 0;

        while (i < length)
        {
            // Get both the UNICODE value of the current character and the complete character itself
            // which can potentially be multiple segments.

            int codePoint = char.IsSurrogatePair(text, i)
                    ?
                char.ConvertToUtf32(text, i)
                    :
                Convert.ToInt32(text[i]);

            var chr = char.ConvertFromUtf32(codePoint);

            // Process current character
            if (IsBreakingChar(codePoint))
            {
                // Character is in Unicode Plane 0 and is in an excluded block
                AppendToken(tokens, token, i - 1);
                token = null;
            }
            else if (codePoint > 0xFFFF)
            {
                // Character is in a Supplementary Unicode Plane. This is where emoji live so
                // we're going to just break each character in this range out as its own token.
                AppendToken(tokens, token, i - 1);
                token = null;
                tokens.Add(new Token
                {
                    Start = i,
                    End = i + (chr.Length - 1),
                    Text = chr,
                    Normalized = chr
                });
            }
            else if (token == null)
            {
                // Start a new token
                token = new Token
                {
                    Start = i,
                    Text = chr
                };
            }
            else
            {
                // Add on to current token
                token.Text += chr;
            }
            i += chr.Length;
        }
        AppendToken(tokens, token, length - 1);
        return tokens;
    }
}

This code looks very bug prone, as the logic around tokenizing is quite complex. For fun, I looked up what Lucine does and the answer is "People can define their own tokenizers".

The type of the tokenizer is in the same file, this seems pretty clear.
Possibly we could move the delegate definition to the Tokenizer? Please open an issue if that makes sense.


Configure spark nodes

We need to update/create `conf/spark-env.sh' with following values:
a) for master node:
export SPARK_MASTER_IP="xxx"
export SPARK_LOCAL_IP="xxx"
b) for worker node:
export SPARK_LOCAL_IP="xxx"
where xxx is a external IP adress.

@hekonsek I need fix for this one because work around I done for master and worker is not working for 'smolok spark submit' and I need to set SPARK_LOCAL_IP when I want to run client deploy mode.
My proposal for handle this one would be to update start-worker.sh, start-master.sh and submit.sh with following code:
if [ -z "${SPARK_LOCAL_IP}" ]; then   export SPARK_LOCAL_IP=${SPARK_LOCAL_IP} fi
I think this should work, what do you think?
Assign this task to me if you think this approach is ok.

Actually I think that setting up those options on the Docker level would be fine.  I have uploaded the most recent Docker image to DockerHub, so just pull the latest 0.0.1-SNAPSHOT.
[1] https://github.com/smolok/smolok/commit/54b6d07a4d99621a3ae5c6f41215cc74b442fc8c

Yes it works. Thank you.


[match] fix match not making new profiles for force_for_new_devices

Fixes #15118
Motivation and Context
To force match to recreate new profiles (again) when a new device is added
Description
Fixing a regression inserted with #15023. This PR introduced a new variable called force which replaced which params[:force] everywhere since there was some logic needed about when to use force. There was an instance of params[:force] that wasn't replaced which caused force_for_new_devices to never trigger generation of a new profile 😱
But we should be good now 🙃

@mollyIV For sure 👌 IMO, params should not be used as a "global" variable storage since it gets hard to trace things so this would be 💯 But also match, might end up getting reworked a bit with the use of the new App Store Connect APIs so that might also be a good time to clean up some of the params usage 🙃 Thoughts?


add volume as integer to CEUR-WS data

select max(eventId) from event_CEURWS

gives
Vol-999
we need the most recent volume number for certain use cases

followup issue:
Proceedings of the 1998 European Workshop,
June 6 - June 8, 1998.Submitted by: Frank van Harmelen|id=Vol-16
is multiline


Update request dependency to v2.81.0

This prevents the google-auth-library, by way of a dependency on an older version of request, from emitting a node-uuid is deprecated message during installation.

Coverage remained the same at 94.233% when pulling 5b172454e0b9050de46122613fe930e347287548 on kohlmannj:master into d38f6c5c276851b8f36f3da240d070bf7d598ec6 on google:master.

Coverage remained the same at 94.233% when pulling 228384e3392add49242b1ff959106d8633a7f22d on kohlmannj:master into d38f6c5c276851b8f36f3da240d070bf7d598ec6 on google:master.

Coverage remained the same at 94.233% when pulling 228384e3392add49242b1ff959106d8633a7f22d on kohlmannj:master into d38f6c5c276851b8f36f3da240d070bf7d598ec6 on google:master.

Note: I've reviewed the Travis build failure for #244.3, and I'm not convinced I know enough about the project to determine why it failed.

Sorry for the late response. This has already been addressed in another PR. Closing.


Enum renders kind of weird

Provide a minimally reproducible example
#| export
from enum import Enum

#| export
class Role(Enum):
    SYSTEM = "system"
    USER = "user"
    ASSISTANT = "assistant"
    FUNCTION = "function"

creates the following doc:

The first question here is what should an Enum like this look like in the docs? I think even just showing the mappings would be a good starting point.  So
Role

System = "system"
USER = "user"
ASSISTANT = "assistant"
FUNCTION = "function"

Maybe a comment down here?


Add common retry helper for SeverManager

VSTS https://github.com/aspnet/IISIntegration/issues/1049


Update jobs.md

JavaScript Engineer(Full time, 40h/week)
Responsibilities
Build high-quality web applications and electron-based desktop applications;
Keep optimizing code and applications in codebase, performance and user interface;
Build dev-tools/libs for the whole ecosystem of nervos network CKB project and godwoken project;
Minimum Qualifications
Comp. Sci or Comp. Eng degree or demonstrable skill and equivalent experience;
Experienced in Front-end development, familiar with HTML, CSS and TypeScript, have a clear understanding of Web standards(Accessibility, Security);
Have worked on generic front-end scope such as Node.js app, electron app;
Proficient in utilities of front-end, such as scaffold, css pretreats, and template engines;
Prefer React and TypeScript;
Have knowledge of Blockchain and have participated in related products design;
Fluently in English reading and writing.
Preferred Qualifications
Live in open source community;
Have GitHub or Blog

Product/Project Manager(Full time, 40h/week)
Responsibilities
Build and manage robust, scalable and delightful products that are aligned with the company's mission;
Clearly define product goals, value propositions, milestones and success metrics. Document requirements and acceptance criteria to ship the right product;
Use good product thinking and judgment to manage product development and lifecycle, coordinating teams and resources to follow through from conception, development, testing, release management, and promotion;
Own and track product success metrics and manage product feedback loops to drive improvements.
Minimum Qualifications
3+ years of experience working as a product manager in the software development or internet industry;
Strong understanding of product and design thinking methodologies;
Experience managing all aspects of a successful product throughout its lifecycle;
Proven experience in developing product and marketing strategies and effectively communicating recommendations to executive management;
Strong problem solving skills and a high level of accountability;
Passion, knowledge and activity in the blockchain ecosystem;
Hold cryptocurrency assets in a decentralized blockchain and be an active user of DeFi;
Honest, clear-headed, self-driven, a team player and always eager to learn;
Fluent in English reading and writing.
Preferred Qualifications
None
Ruby Engineer(Full time, 40h/week)
Responsibilities
Understanding business requirements and being responsible for product design and development;
Being responsible for the performance optimization of existing products;
Maintaining blockchain related tools;
Collaborate with Ops to deploy and maintain projects.
Minimum Qualifications
Comp. Sci or Comp. Eng degree or demonstrable skill and equivalent experience;
3+ years of practical experience working with Ruby, Ruby on Rails;
Experience with daily database design and development as well as optimization (MySQL, PostgreSQL);
Familiarity with Git/GitHub and RESTful standards;
Familiarity with underlying middleware, distributed technologies (including redis caching, messaging systems, etc.);
Fluently in English reading and writing.
Preferred Qualifications
A proven track record of open source projects or open source community contributions;
Experience with database tuning;
Experience with system performance optimization;
Experience with blockchain product development;
Experience with full-stack development;

Junior Tester(Full time, 40h/week)
Responsibilities
Particiate in product and technical design to bring better products to nervos community;
Test the full application stack using a range of testing approcches from automation to exploratory;
Respond to, and support, issues from community;
Constantly learn and expand knownledge about blockchain and nervos CKB.
Minimum Qualifications
Drive to support continuous improvement in testing practices;
Experienced with system testing, integration testing;
Experienced building or executing test automation across layers of the software stack;
Have knownledge on Blockchain and have particiated in related products;
Fluently in English.
Preferred Qualifications
Live in open source community;
Have GitHub or Blog

Hi @fangqiao1 , could you submit the JD through github issue? Since we have actions to process the new added issues automatically, and the new add issue will be published in our WeChat Official Account and our website.


What is CodePlay's plan to update TF SYCL integration to the latest TensorFlow version

I am working on using TensorFlow SYCL device to support my GPU. What is CodePlay's plan to update TF SYCL integration to the latest TensorFlow version?

We are right now focusing on two priorities: Upstreaming Eigen and improving performance with the use of SYCL-DNN and SYCL-BLAS. These two jobs will then make it much easier to update TensorFlow in the future as well as much easier to optimize for different devices. If you want to contribute to the project, then doing the merge from upstream TensorFlow will be very much appreciated and help all of us a lot. We'd be happy to accept a merge request for this work.

@rodburns Does it means if I want to try Tensorflow SYCL, I have to get tensorflow source from here and
the upstream Tensorflow (https://github.com/tensorflow/tensorflow) doesn't support SYCL so far?
Thank you.

@wangliwei-intel Yes this is broadly correct. If you want to use the SYCL TensorFlow implementation you need to follow the instructions here that specify what branches, changelists and versions to use. https://developer.codeplay.com/products/computecpp/ce/guides/tensorflow-guide/tensorflow-amd-setup
We do plan to upstream the changes but first we are upstreaming the Eigen work we have done to that project.

@rodburns
Actually, I'm trying that pull request of Eigen, but I can't find the source code of benchmarks, and I've
raised the question there, do you know what the answer is?
https://bitbucket.org/eigen/eigen/pull-requests/442/enabling-optimised-eigen-sycl-backend-for/activity
Thank you!

Any update?
It would be cool if you could hit at least 1.12, so that it would be usable with python 3.7.

As TensorFlow 2.0 will be released soon, we are assessing if we can migrate directly to that version and hopefully we can provide an update on that in due course

Issue was answered so closing.


feat: add key management provider resource

Description
What this PR does / why we need it:

Adds a new Key Management Provider resource which will supersede the Certificate Store
This new resource will encompass all key + certificate management from various configured KMS providers.
Adds an error if Key Mangement Provider is applied while an existing Certificate Store exists on the cluster. Error is in the Ratify logs.

It is NOT recommended to use both CertificateStore and KeyManagementProvider resources together.
If using helm to upgrade Ratify, users need to delete any existing CertificateStore resources.
For self-managed CertificateStore resources, users should migrate to the equivalent KeyManagementProvider.
If migration is not possible and both resources must exist together, please make sure to use DIFFERENT names for each resource type.
Ratify is configured to prefer KMP resources when a matching CertificateStore with same name is found.
All functionality in CertificateStore is present in KMP. Users will be pointed to docs to migrate (TODO). The names of some fields are different.

Adds e2e tests and UTs

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #1294
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Helm Chart Change (any edit/addition/update that is necessary for changes merged to the main branch)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

Hi@akashsinghal , would you be able to add some test results/ screenshots / logs so we could review what the CRs look like for back compatibility purpose? For example, for existing customers using AKV code path, on ratify upgrade, will they see both CertStore and KMP CR?
From the code, one CR (hopefully they reconcile in a deterministic fashion ), should be in a disabled with a clear msg ( might need doc to clarify this is expected for existing customer).

Hi@akashsinghal , would you be able to add some test results/ screenshots / logs so we could review what the CRs look like for back compatibility purpose? For example, for existing customers using AKV code path, on ratify upgrade, will they see both CertStore and KMP CR?
From the code, one CR (hopefully they reconcile in a deterministic fashion ), should be in a disabled with a clear msg ( might need doc to clarify this is expected for existing customer).

@susanshi, @binbin-li, and I had a discussion offline about this. The goal is to transition users to use the new KeyManagementProvider (KMP) resource. New users ideally should be installing KMP instead of CertificateStore. For upgrade scenarios, users would already have an existing CertificateStore resource on a cluster and we cannot force the user to update while causing potential Ratify downtime. Introducing should not be a breaking change initially until v2.0.0. As such, here is the proposed approach:

All existing CertificateStore support will remain. Implementation wise, CertificateStore will take precedence over KMP if both exist with SAME unique metadata.name.
The helm chart will be updated to install the KMP resource instead of CertificateStore. If the KMP controller detects the existence of a CertificateStore resource already, an error will be logged showing the conflict however BOTH resources will be operational. Deference will be given to the CertificateStore resource.
In Ratify v2.0, the CertificateStore will be entirely deprecated and it's implementation/support removed. Once done, the KMP will not perform any check for existence of CertificateStore.

Hi@akashsinghal , would you be able to add some test results/ screenshots / logs so we could review what the CRs look like for back compatibility purpose? For example, for existing customers using AKV code path, on ratify upgrade, will they see both CertStore and KMP CR?
From the code, one CR (hopefully they reconcile in a deterministic fashion ), should be in a disabled with a clear msg ( might need doc to clarify this is expected for existing customer).

@susanshi, @binbin-li, and I had a discussion offline about this. The goal is to transition users to use the new KeyManagementProvider (KMP) resource without breaking them. New users ideally should be installing KMP instead of CertificateStore. For upgrade scenarios, users would already have an existing CertificateStore resource on a cluster and we cannot force the user to update while causing potential Ratify downtime. Introducing KMP should not be a breaking change initially until v2.0.0. As such, here is the proposed approach:

All existing CertificateStore support will remain. Implementation wise, CertificateStore will take precedence over KMP if both exist with SAME unique metadata.name.
The helm chart will be updated to install the KMP resource instead of CertificateStore. If the KMP controller detects the existence of a CertificateStore resource already, an error will be logged showing the conflict however BOTH resources will be operational. Deference will be given to the CertificateStore resource.
In Ratify v2.0, the CertificateStore will be entirely deprecated and it's implementation/support removed. Once done, the KMP will not perform any check for existence of CertificateStore.

Documentation and TODOs will be added with above behavior
Here's a sample of logs when a user updates to newer version of Ratify from current version:
time=2024-03-20T18:33:02.798193985Z level=error msg=Error: key management provider and certificate store cannot be configured together, Code: KEY_MANAGEMENT_CONFLICT, Plugin Name: gatekeeper-system/kmprovider-akv, Component Type: keyManagementProvider, Detail: certificate store already exists

Community Meeting 3/20/24 discussion:

We need to prioritize the existing Certificate Store. KMP should be an added feature that existing users can decide to switch to
It makes sense to have v1.2 helm chart switch to KMP so that new users start using KMP from start
We need to add documentation, improve error message in the log, and add BREAKING CHANGE guidance in the release notes.
The plan outlined above originally, will be largely adhered to
The main point of contention is for users who do not use helm chart for Ratify install/upgrade. If they have an existing CertStore and then attempt to apply a new KMP with the SAME name as the CertStore, then the new KMP is essentially disregarded since notation will give preference to CertStore. This scenario we need to highlight and tell users to preferably delete the old CertStore or use a different named KMP. This name conflict will not affect helm chart upgrades since the new KMP resource will have a different name then the old charts with the CertStore. The notation verifier will accordingly be updated to reference the new KMP name instead so things should just work out of the box.

@susanshi appreciate another round of review. From your last review: I updated an error message to be more descriptive. I switched the behavior so that now a KMP resource is favored over a CertificateStore resource when both have the same metadata.name. This behavior change is in response to our discussions with @fseldow .


Update README.md

Thanks for the PR. Would you please append instead of the prepend your entry? :)

Thanks for the PR. Would you please append instead of the prepend your entry? :)

✅