Project/AOI data limitations

Right now there are not data limits on an AOI project. Not only can you create an AOI the size of the whole earth, but if https://github.com/azavea/raster-foundry/issues/2040 is addressed, there's no limit to how far back in time you can go.
I think there are several tactics for addressing this:

Limit the size of the AOI
Limit the number of scenes that can be added to an AOI and calculate those before starting an ingest
Limit the amount of data that can be processed by a user in a day

I don't think tactic 1 is a good tactic (although it is a common one). I think 2 suffers from the problem of disparity in "scene" sizes across data types (i.e. it's not a good proxy for the problem we're trying to address, which is really compute resources). I think 3 will take a boatload of effort to monitor and visualize for the user/create reasonable warnings for, but it's probably the best fit for solving the problem.

This is also an issue with regular projects; we don't place a limit on the number of scenes you can put in a project so someone could make a project containing every scene in the DB.

that's called "Pulling a Santucci"


Replace AnyTypes with apiextensions.JSON

Adds a pipeline stage to convert AnyTypes to k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1.JSON, which can store arbitrary JSON values in the API. This enables generating all of the packages that were previously being filtered out because they generated interface{} fields.
Removes the filterOutDefinitionsUsingAnyType stage, although leaves the ensureDefinitionsDoNotUseAnyTypes check at the end just in case later stages introduce AnyTypes again.
A lot of packages include fields that come out as map[string]map[string]v1.JSON (especially for UserAssignedIdentities fields), but controller-gen can't generate deep-copy code for these types. To handle these the stage collapses them down to map[string]v1.JSON as well. https://github.com/kubernetes-sigs/controller-tools/issues/287
Adds special-case handling in the ARM conversion generation to ensure that JSON fields are deep-copied when converting to/from ARM types.
There are still a couple of packages that need to be excluded because the ARM conversion code generated for them has an issue or hits controller-gen limitations.
Fixes #133.

This came out nicely!

Coverage decreased (-23.7%) to 3.954% when pulling ad869cedaf67463dead150bdffa8e0a7f2b09839 on babbageclunk:json-fields into 155e5a41f8da034a030f0184e65a9f33a7fdee38 on Azure:master.


Could not run phased build action using Gradle distribution

Could not run phased build action using Gradle distribution 'https://services.gradle.org/distributions/gradle-6.1.1-all.zip'.
Could not open cp_init remapped class cache for 6a5krabdzij62jglcll542e66 (C:\Users\ashut\.gradle\caches\6.1.1\scripts-remapped\eclipsePlugin_3p8l27g9m3tj7wg6futj69bc8\6a5krabdzij62jglcll542e66\cp_init3607aee355f62839c5e6f549478ccc87).
Could not open cp_init generic class cache for initialization script 'C:\Users\ashut\AppData\Roaming\Code\User\workspaceStorage\30fff57d479a798977bdef98d8f73fa4\redhat.java\jdt_ws\.metadata\.plugins\org.eclipse.buildship.core\init.d\eclipsePlugin.gradle' (C:\Users\ashut\.gradle\caches\6.1.1\scripts\6a5krabdzij62jglcll542e66\cp_init\cp_init3607aee355f62839c5e6f549478ccc87).
Could not initialize class org.codehaus.groovy.vmplugin.v7.Java7

Can someone tell me why this error pops? Unable to solve this is this a bug?
I want to write Native Java Code for my flutter app
`flutter doctor -v Output
[√] Flutter (Channel stable, 1.20.3, on Microsoft Windows [Version 10.0.19041.508], locale en-IN)
• Flutter version 1.20.3 at S:\Softwares\flutter
• Framework revision 216dee60c0 (4 weeks ago), 2020-09-01 12:24:47 -0700
• Engine revision d1bc06f032
• Dart version 2.9.2
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at S:\Softwares\Android
• Platform android-30, build-tools 30.0.2
• Java binary at: S:\Softwares\Android Studio 4.0\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Android Studio (version 4.0)
• Android Studio at S:\Softwares\Android Studio 4.0
• Flutter plugin version 49.0.2
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] Connected device (1 available)
• SM G965F (mobile) • 37caa7a607027ece • android-arm64 • Android 10 (API 29)
• No issues found!`

Hi @Ashutosh3011
This issue doesn't seem to describe a bug related Flutter or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


fixed mysqlWorkbench Uri

Hi,
mysql.com has changed the download uri.
I've changed it in the manifest, this is the pullrequest.
best regards

I need to recreate the commit signed from another workstation.


loss progress

Hi Matteo,
I was wondering how was the loss progress during your training? For example, after 3 epochs loss didnt change at all for me. It is around 8.1 and not dropping.
I was wondering how much loss was for you when you finished training, and how much we can expect to be after 49 epochs?
All the best.

Hello @Sh0lim
unfortunately I don't have the logs of the experiments anymore so I hope to remember right. I remember that at the beginning I had to have patience, after a few epochs the loss started decreasing. But you could try to lower the learning rate by a factor of 10 earlier.
At epoch 49 the loss should be around 2.7...something.

Yes, patience. :)
Anyway, that was useful. Thank you.

 Can you tell me about your result. I got a similar problem.

Hi @donghn,
i had problem with learning rate. I modify code a bit and there was bug. I was reducing learning rate by 10 every step, so learning rate was almost 0.
When I correct it, loss was doping down normally. Loss after 3 epoch was around 4.5, after 7 epochs was around 3. I started with learning rate 0.001. After 10 epochs i divide learning rate by 10. training is still running, I let you know what was the loss at the end. But I have old gpu, so I will not go to the end. Ill stop it after 20 epochs.
All the best.

@Sh0lim Yes. Thank so much. !!

try with this. gpu_memory_fraction goes up to 1.
gpu_options = tf.GPUOptions(per_process_gpu_memory_fraction=gpu_memory_fraction)
sess = tf.Session(config=tf.ConfigProto(gpu_options=gpu_options, allow_soft_placement=True, log_device_placement=False))
with sess.as_default():
...
Best,
M.

@Sh0lim may i ask about the details of your workable settings? learning rate test_step and batch_size etc.
This will really helpful, i also encounter some loss stuck issues.

Hi guys
if you happen to find better hyper parameters just submit a PR. That would be appreciated! 👍


SCM Sync: View is strange when checked out on a branch without upstream

Testing #194016

Could not repro. Closing for now. Please reopen if you have a repro. Thank you!


JSON CRDT benchmarks

[ ] Text editing benchmarks.

Dupe of https://github.com/streamich/json-joy/issues/129


Rss views

bad queryset pagination.
don't need this explicit return. Seems like it was a bandage that is no longer needed.
@mparent61 @MichaelButkovic

:+1:


afun60: add "Before session begins" helpers

Add some tips to setup the demo machine for prime time.

Nice work @cmaneu


Integrate Scheduled Meetings day view into the Admin App

See https://github.com/Azure/azure-sdk-tools/issues/3944#issuecomment-1218677468_ for more context.
@maririos created a new view that will tell the admins what meetings have been created every day, it would be useful if they could add it to the admin app too.

This is actually documented on https://github.com/Azure/azure-sdk-tools/issues/3857
Closing this issue since it is a duplicate


Total Grid Import: Does not match with actual grid import

You Inverter
Details about your Inverter: Sunsynk 8KW Single Phase - Firmware COMM:E426 MCU:6022-1724
Describe the issue/bug and what you expect
The total grid import sensor does not seem accurate. As per #134 I think this again comes down to the mode the inverter is set in.
Things I am noticing:

When in limit to load or export modes it does not count grid charging towards grid import
When in limit to load only the grid import for essentials are counted, non-essentials are not counted towards grid import.

I can see according to the sensor this data is retrieved directly from modbus registers 78 and 80 on the inverter and can confirm that the sunsynk app also does not align correctly with actual grid import so I guess this is one of those issues around should it be fixed in the addon or not. I have also read on social media about people complaining about the numbers on the synsynk app does not match that of actual readings on accounts and it varies quite a bit.
I have not had time to go through a more detailed look at the modbus document to see if there are registers one can combine to get to the right numbers so would like to hear anyone else's findings on this.

To test this theory, I have gone ahead and set up a Riennman Sum Integral sensor on the CT grid sensor value, and then setup a Utility meter sensor on the Riennman Sum Integral sensor so it resets every day.
Will see how it goes over the next couple of days and report back with my findings.

I've never thought of that, but now that you mention it, I am probably seeing something similar.
During the evenings I switch to "limit to load" so that I do not drain my batteries with the geyser. But I have also been seeing a mismatch between my reported energy use and my prepaid meter. This would explain it.
It would not be easy to fix. You could:

try an integral on the grid_ct_sensor as you suggested.
sum the total_grid_import with the total_non_ess (external meter), but you will have to account for the inverter program, since you should not sum these during zero export/export to grid

How is your sensors working out? Do you mind posting the config here, or even adding it to the examples in the docs?

According to my calculations the custom sensors are working as they should and reporting correctly. I am just waiting for my next monthly bill to confirm 100% things align. I'd be happy to share the sensor config, however you prefer.
I have emails some people in Sunsynk about this and also opened a ticket, but yet to receive any response.

Same issue here with exporting and the "Total Grid Import" value. When mode was "Zero Export" then sum value was correct and included non-essentials and reflected correctly on sunsynk.net. After enabling export, non-essential KWH were not included in the grid import nor consumption figures. The instantaneous read sensor "Non-Essential power" (KW) does pick up the geyser, etc but cannot be used to feed the energy dashboard which wants a cumulative measure. If this sensor was available as a sum, then you could correct the grid import value.

It’s unfortunate that these sensors change when you change the export mode.
The addon does not keep any state. And this is not something I plan on adding soon. To calculate the energy, you will have to keep state
@kloon solution of doing this in HASS is probably your best option today

I finally got a reply from Sunsynk on this issue after 3 weeks, no surprise it was a generic one saying to update to the latest firmware, which I did, yet the issue is still present.

Adding a wontfix label as this is the Inverter's bahaviour, but will keep the issue open for reference


pkg: grub: add measurefs command patch for grub 2.06

I Adapted the patch that @mikem-zed added for the 2.02 version of grub and added measurefs cmd for arm64.

              Edge Virtualization Engine

................   ..............   ................
 ................   ............   ................
              ....    .........   ....
    .[    8.434559] random: crng init done
............... [    8.439682] brcmfmac: brcmf_fw_alloc_request: using brcm/brcmfmac43455-sdio for chip BCM4345/6
  .......   ................
     ................   .....   ................
                    ...   .   ....
        ................    [    8.460764] brcmfmac: brcmf_c_preinit_dcmds: Firmware: BCM4345/6 wl0: Apr 15 2021 03:03:20 version 7.45.234 (4ca95bb CY) FWID 01-996384e2
 ................
          ...............   ................

              Edge Virtualization Engine
linuxkit-dca632b1cd3d login: root (automatic login)

EVE is Edge Virtualization Engine

Take a look around and don't forget to use eve(1).
linuxkit-dca632b1cd3d:~# [   10.916141] cgroup: cgroup: disabling cgroup2 socket matching due to net_prio or net_cls activation
[   11.962629] usbcore: registered new interface driver smsc75xx
[   11.980938] usbcore: registered new interface driver cp210x
[   11.986711] usbserial: USB Serial support registered for cp210x
[   12.001519] nicvf, ver 1.0
[   13.769362] EXT4-fs (mmcblk1p9): mounted filesystem with ordered data mode. Opts: (null)
[   22.815406] usbcore: registered new interface driver cdc_wdm
[   22.853303] usbcore: registered new interface driver cdc_acm
[   22.859057] cdc_acm: USB Abstract Control Model driver for USB modems and ISDN adapters

And no debug messages from grub after measurefs command:
U-Boot 2022.10-00001-g5a6603ad90-dirty (Nov 27 2022 - 22:58:35 -0100)

DRAM:  7.9 GiB
RPI 4 Model B (0xd03114)
Core:  177 devices, 18 uclasses, devicetree: board
MMC:   mmc@7e300000: 0, mmc@7e340000: 1
Loading Environment from FAT... Card did not respond to voltage select! : -110
** Bad device specification mmc 0 **
In:    serial
Out:   serial
Err:   serial
Net:   eth0: ethernet@7d580000
PCIe BRCM: link up, 5.0 Gbps x1 (SSC)
starting USB...
Bus xhci_pci: Register 5000420 NbrPorts 5
Starting the controller
USB XHCI 1.00
scanning bus xhci_pci for devices... 2 USB Device(s) found
       scanning usb for storage devices... 0 Storage Device(s) found
Hit any key to stop autoboot:  0
Card did not respond to voltage select! : -110
switch to partitions #0, OK
mmc1 is current device
Scanning mmc 1:1...
Card did not respond to voltage select! : -110
** Unable to read file ubootefi.var **
Failed to load EFI variables
Unable to find TPMv2 device
BootOrder not defined
EFI boot manager: Cannot load any image
Found EFI removable media binary efi/boot/bootaa64.efi
1040384 bytes read in 64 ms (15.5 MiB/s)
Booting /efi\boot\bootaa64.efi
Welcome to GRUB!

script/script.c:65: free 0x3c8edec0
script/script.c:65: free 0x3c8edf00
script/script.c:65: free 0x3c8edf40
script/script.c:65: free 0x3c8eda00
script/script.c:65: free 0x3c8eda60
script/script.c:65: free 0x3c8edb00
script/script.c:65: free 0x3c8edb60
script/script.c:65: free 0x3c8edbc0
script/script.c:65: free 0x3c8edca0
script/script.c:65: free 0x3c8edd80
script/script.c:65: free 0x3c8edde0
script/script.c:65: free 0x3c8ede20
script/lexer.c:336: token 288 text [set]
script/script.c:50: malloc 0x3c8edce0
script/script.c:50: malloc 0x3c8edca0
script/script.c:163: arglist
script/script.c:50: malloc 0x3c8edc40
script/lexer.c:336: token 289 text [root=]
script/script.c:50: malloc 0x3c8ed9a0
script/script.c:50: malloc 0x3c8ed960
script/lexer.c:336: token 289 text [dev]
script/script.c:50: malloc 0x3c8ed900
script/script.c:50: malloc 0x3c8ed8c0
script/lexer.c:336: token 289 text []
script/script.c:50: malloc 0x3c8edb60
script/script.c:50: malloc 0x3c8ed880
script/script.c:163: arglist
script/script.c:50: malloc 0x3c8ed820
script/lexer.c:336: token 259 text [
]
script/script.c:50: malloc 0x3c8ed7c0
script/script.c:50: malloc 0x3c8ed780
script/script.c:198: cmdline
script/script.c:50: malloc 0x3c8ed720
script/lexer.c:336: token 0 text []
script/script.c:50: malloc 0x3c8ede00
script/script.c:50: malloc 0x3c8eddc0
script/script.c:294: append command
script/script.c:50: malloc 0x3c8edd80
kern/verifiers.c:212: string: set root=hd0,gpt2, type: 2
script/script.c:65: free 0x3c8edd80
script/script.c:65: free 0x3c8eddc0
script/script.c:65: free 0x3c8ede00
script/script.c:65: free 0x3c8ed720
script/script.c:65: free 0x3c8ed780
script/script.c:65: free 0x3c8ed7c0
script/script.c:65: free 0x3c8ed820
script/script.c:65: free 0x3c8ed880
script/script.c:65: free 0x3c8edb60
script/script.c:65: free 0x3c8ed8c0
script/script.c:65: free 0x3c8ed900
script/script.c:65: free 0x3c8ed960
script/script.c:65: free 0x3c8ed9a0
script/script.c:65: free 0x3c8edc40
script/script.c:65: free 0x3c8edca0
script/script.c:65: free 0x3c8edce0
script/lexer.c:336: token 288 text [configfile]
script/script.c:50: malloc 0x3c8edc20
script/script.c:50: malloc 0x3c8edbe0
script/script.c:163: arglist
script/script.c:50: malloc 0x3c8edb80
script/lexer.c:336: token 289 text [(]
script/script.c:50: malloc 0x3c8ed940
script/script.c:50: malloc 0x3c8ed900
script/lexer.c:336: token 289 text [dev]
script/script.c:50: malloc 0x3c8ed8a0
script/script.c:50: malloc 0x3c8ed860
script/lexer.c:336: token 289 text [)/EFI/BOOT/grub.cfg]
script/script.c:50: malloc 0x3c8ed800
script/script.c:50: malloc 0x3c8ed7a0
script/script.c:163: arglist
script/script.c:50: malloc 0x3c8ed740
script/lexer.c:336: token 259 text [
]
script/script.c:50: malloc 0x3c8ed6e0
script/script.c:50: malloc 0x3c8ed6a0
script/script.c:198: cmdline
script/script.c:50: malloc 0x3c8ed640
script/lexer.c:336: token 0 text []
script/script.c:50: malloc 0x3c8edd80
script/script.c:50: malloc 0x3c8edd40
script/script.c:294: append command
script/script.c:50: malloc 0x3c8edd00
commands/wildcard.c:535: no expansion needed
commands/wildcard.c:594: paths[0] = `(hd0,gpt2)/EFI/BOOT/grub.cfg'
kern/verifiers.c:212: string: configfile (hd0,gpt2)/EFI/BOOT/grub.cfg, type: 2

measurefs: Measuring hd0,gpt2 into PCR-13
                             GNU GRUB  version 2.06

 ┌────────────────────────────────────────────────────────────────────────────┐
 │*Boot 0.0.0-pr2946-3fc6e355-kvm-arm64                                       │
 │ Set Boot Options                                                           │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 │                                                                            │
 └────────────────────────────────────────────────────────────────────────────┘

      Use the ▲ and ▼ keys to select which entry is highlighted.
      Press enter to boot the selected OS, `e' to edit the commands
      before booting or `c' for a command-line. ESC to return previous
      menu.
   The highlighted entry will be executed automatically in 0s.

I need to study the problem.

Yes, I just tested everything, the problem with kernel hangs was due to an image write error, after rewriting EVE to sdcard everything worked.
I get this values from eve exec vtpm tpm2_pcrread command
sha1:
  0 : 0xF40C4B5B8EC48D5812A47B9117BE084CB446799D
  1 : 0x629391551BD72410650B39D3ABECF488883D66A8
  2 : 0xB2A83B0EBF2F8374299A5B2BDFC31EA955AD7236
  3 : 0xB2A83B0EBF2F8374299A5B2BDFC31EA955AD7236
  4 : 0x171E8800AA35DCA62E25987E72EEBF844BCAE8A8
  5 : 0x731C0516F71254D7F6A0C6EB8B8C1FB8D940658E
  6 : 0xB2A83B0EBF2F8374299A5B2BDFC31EA955AD7236
  7 : 0x1F5A197D36B62E30E5BC22C88B6E4A4556BB9539
  8 : 0x06A206B240175C6F39DCE90DC5312AE3315E296F
  9 : 0xF2747308E8D208C6687C75440582C06180F013C2
  10: 0x0000000000000000000000000000000000000000
  11: 0x0000000000000000000000000000000000000000
  12: 0x0000000000000000000000000000000000000000
  13: 0x0B90DB4347B5F6115693AB0160454555D0229930
  14: 0x0000000000000000000000000000000000000000
  15: 0x0000000000000000000000000000000000000000
  16: 0x0000000000000000000000000000000000000000
  17: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  18: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  19: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  20: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  21: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  22: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  23: 0x0000000000000000000000000000000000000000
sha256:
  0 : 0x046F1225A3A2FEDD476930D68C5FDC7C9F355F206F0555A0FBC966231447613D
  1 : 0xD88B8D993E6AEE365B9BA3384E1FE7AD6D04F33AF2AF7F681B2453C65875D102
  2 : 0x3D458CFE55CC03EA1F443F1562BEEC8DF51C75E14A9FCF9A7234A13F198E7969
  3 : 0x3D458CFE55CC03EA1F443F1562BEEC8DF51C75E14A9FCF9A7234A13F198E7969
  4 : 0x68A816FD57A9F35EE31E31DA45C27853F7DA9B55CA34A127E849B027A197D9D0
  5 : 0x86A20D6013CC1DC8F8FDE77389F88964AE693AF97FC7B26074F7709C2E9F6D38
  6 : 0x3D458CFE55CC03EA1F443F1562BEEC8DF51C75E14A9FCF9A7234A13F198E7969
  7 : 0x9674AECD3F40B4A936AE19C8848AB95A8799D8897FFC40480599652E55D49332
  8 : 0x6301D31A77000315E575B4C8B37DEA49A26031BAA9065E7C90404E91B4263913
  9 : 0x8C0DA3C568A1CCF7443E737C0E30B1433F6360588F12D257C428DC07104CA55F
  10: 0x0000000000000000000000000000000000000000000000000000000000000000
  11: 0x0000000000000000000000000000000000000000000000000000000000000000
  12: 0x0000000000000000000000000000000000000000000000000000000000000000
  13: 0x3EB6713D587021F8F57EE432C93CFACF1D16A6692D9F511DDED4F47721C99F95
  14: 0xECF7D7F8449D0F783A28715D2A2EDA576069FEEFE8FAC17B9D0D47527DA0DE60
  15: 0x0000000000000000000000000000000000000000000000000000000000000000
  16: 0x0000000000000000000000000000000000000000000000000000000000000000
  17: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  18: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  19: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  20: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  21: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  22: 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF
  23: 0x0000000000000000000000000000000000000000000000000000000000000000

LGTM

@mikem-zed any chances you upstream the patch?

or if @Insei have connections in the grub community can help with that?

we need to move to the latest LTS GRUB which is going to be 2.12 soon. Then we can think about up streaming. This is not a very easy task, vanilla grub is not suitable for us because it is missing some important functionality like efilinux. this is why we have an initiative to use Ubuntu's 2.06 grub but the whole process is not decided yet


Set a common group tag

Fixes #121
This addresses an issue in micrometer where CB with different tags end up not showing up in the metrics.
See https://github.com/micrometer-metrics/micrometer/issues/877

@jonatan-ivanov would like your thoughts if you have any knowledge here.
If you think this looks OK, I would probably make this configurable as well as add a property to disable it.


BZ1288132: Same 'Completed indexing' message 3 times in the server log

See https://bugzilla.redhat.com/show_bug.cgi?id=1288132
Batch Indexing was being flagged as "started" in the asynchronous handler, which itself may not have been scheduled by the JVM to run until after additional requests for the FileSystem had been received. Each request checks if the FileSystem has been indexed and, if not, schedules a batch index. This change moves flagging the batch index as "started" from the asynchronous handler to where the request is made. This prevents multiple batch indexing on successive requests for an (unindexed) FileSystem.

+1 (thx for fix my sh*t)

Thanks.
master: 7d6764e69434756f22c54f965cbbb8ccfff943ee


[5.3] Illuminate\Database\Query\Builder::count() return 0 when offset is enabled.

Laravel Version: 5.3
PHP Version: 7.0
Database Driver & Version:
PostgreSQL 9.2

Description:
Illuminate\Database\Query\Builder::count() return as expected when offset = 0. But if we setup the offset, the count return 0 (not expected).
Steps To Reproduce:
use the query builder and set the offset, get the value of Builder::count().

Please try with Laravel 5.4 and open a new issue if this remains.

@ordentid also, this is the correct behaviour.
I'm not sure what oyu expect, but the standard does exactly expect not to return any rows if rowcount < offset
see: http://sqlfiddle.com/#!15/9eecb7db59d16c80417c72d1e1f4fbf1/14934
SELECT COUNT returns only ONE row. offsetting by 15 results in 0 rows as expected


Clocks are not trustworthy

Timestamps are used in the last write wins commit strategy, but timestamps are unreliable.  Clocks on computers exhibit strange behavior, sometimes jumping forward or even backward.  A malicious party can set the timestamp to an arbitrary value.  Therefore, a timestamp should only be accepted if it comes from a trusted party or if it was signed by one.

@alanhkarp there are no wall clock timestamps used in the spec to determine logical ordering of any kind, there is a clock property that represents a simple incrementing integer/global vector. Additionally, data is only allowed to be written in based on permissions the user allows, so random attackers have no ability to just overwrite this clock value. If Alice permissions an app to write and they overwrite an object with a greater vector value, then that's simply a poorly coded app that she should not use.

Here is the definition of the clock param: The object MUST contain a clock property, and its value MUST be an integer representing an incrementing logical counter. - it's not a timestamp, and the processing instructions do not rely on any concept of relativistic time: https://identity.foundation/identity-hub/spec/#processing-instructions

My bad.  The document specifies that the datePublished "MUST be an Unix epoch timestamp".  However, I was wrong, since those time stamps are not used in the commit decision.  Can an incorrect datePublished cause problems?
Insider attacks are common.  A user with a legitimate permission to update an object may choose a large clock value to guarantee the user's update will be considered the last writer, blocking updates from other users.  Also, is there a need to provide for clock values wrapping?

Discussed at DIF:
@alanhkarp : There is no hard anchoring on clock timestamps. Given this perspective in DIF, are you OK with us closing this out? Adding motion to close tag.

Was my question "Can an incorrect datePublished cause problems?" answered in the documentation?
Was my question "Also, is there a need to provide for clock values wrapping?" answered in the documentation?
Even though the timestamps used for commit order decisions aren't clock values, there is a question of whether they can be manipulated.  If they can, you either need to mitigate the problem or explain any impacts.  The other choice is to rely on consensus of arrival order.

Discussed on DIF:
Lost where the language is on this. We should repoint this issue to the updated commit strategy language.


kornia integration

Hi guys ! this is a really nice paper !!
We would like to have BPnP integrated inside kornia, however there's a drawback with your code which is that uses opencv and we don't allow third-party libraries other than PyTorch.
Do you guys have any alternative so that we can collaborate ?

Hi Edgar,
Thanks for the invitation. That's a cool idea. I would love to have BPnP inside kornia. We will need to work out an alternative PnP solver then.

Hi Edgar,
Thanks for the invitation. That's a cool idea. I would love to have BPnP inside kornia. We will need to work out an alternative PnP solver then.

Hi Edgar,
Thanks for the invitation. That's a cool idea. I would love to have BPnP inside kornia. We will need to work out an alternative PnP solver then.

Hi Edgar,
Thanks for the invitation. That's a cool idea. I would love to have BPnP inside kornia. We will need to work out an alternative PnP solver then.

you are welcome :) - for the PnP there's the option to include EPnP.
At some point we discussed with one of the original authors to provide the author implementation of EPnP based on the original codes since PyTorch3d has already implemented but with his own copyright and that fact didn't like at all to the original authors.
It can be hard to adapt the code since one of the authors looked at the code at apparently they have taken exactly same name functions and everything. /cc @lferraz have been on working on this problem for a while

you are welcome :) - for the PnP there's the option to include EPnP.
At some point we discussed with one of the original authors to provide the author implementation of EPnP based on the original codes since PyTorch3d has already implemented but with his own copyright and that fact didn't like at all to the original authors.
It can be hard to adapt the code since one of the authors looked at the code at apparently they have taken exactly same name functions and everything. /cc @lferraz have been on working on this problem for a while

Has there been any progress on integrating this PnP code into kornia? The dependence on OpenCV and for loops in the backward pass slow down the training significantly.

hi @ankurhanda, from kornia side we haven’t started direct actions but during this last months we introduced a vanilla pnp and ransac solvers that after looking at BPnP could be used as starting point. The ransac is far from opencv or any related c++ performance, but still worth trying to combine to solve 6dof pose. /cc @ducha-aiki @thatbrguy @lferraz

@edgarriba It would be good to discuss on slack what to add next wrt PnP for Kornia. I would likely have more time to contribute in January or so.


Incorrect candle data & SQL record duplication on insert

Despite the errors, trading advice is being generated.
Error messages
error: Strategy error Cannot read property 'low' of undefined {"stack":"TypeError: Cannot read property 'low' of undefined\n    at Strategy.update (/root/stockml/TraderCore/src/strategies/bb_pure/index.js:41:20)\n    at Emulator.update (/root/stockml/TraderCore/src/emulator/emulator.js:72:29)\n    at process._tickCallback (internal/process/next_tick.js:68:7)"}
error: SQL errorDuplicate entry '142-BCH/USDT-1566206580000' for key 'strategy_guid' {"code":"ER_DUP_ENTRY","errno":1062,"sqlState":"23000","sqlMessage":"Duplicate entry '142-BCH/USDT-1566206580000' for key 'strategy_guid'","stack":"Error: Duplicate entry '142-BCH/USDT-1566206580000' for key 'strategy_guid'\n    at PromisePool.query (/root/stockml/TraderCore/node_modules/mysql2/promise.js:322:22)\n    at LiveEmulator.save_advice (/root/stockml/TraderCore/src/emulator/live_emulator.js:168:18)\n    at LiveEmulator.single_update (/root/stockml/TraderCore/src/emulator/live_emulator.js:128:16)\n    at process._tickCallback (internal/process/next_tick.js:68:7)"} error: Strategy error Cannot read property 'low' of undefined {"stack":"TypeError: Cannot read property 'low' of undefined\n    at Strategy.update (/root/stockml/TraderCore/src/strategies/bb_pure/index.js:41:20)\n    at Emulator.update (/root/stockml/TraderCore/src/emulator/emulator.js:72:29)\n    at LiveEmulator.single_update (/root/stockml/TraderCore/src/emulator/live_emulator.js:114:63)\n    at process._tickCallback (internal/process/next_tick.js:68:7)"}
Strat settings

Should be fine from now.


Generalize implementation of extern functions

Extern functions are meant as mechanisms for users to allow noir code to call into external C code - for example to provide a database call that supplies extra parameters to a function. The difficulty implementing the "call into C code from the noir compiler" side stems from needing to know the type of the function called at compile time so that it can be casted into a function pointer. For example, if we were to use libloading:
let lib = libloading::Library::new("/path/to/liblibrary.so")?;
let func: libloading::Symbol<unsafe extern fn() -> u32> = lib.get(b"my_func")?;
func()

Getting this unsafe extern fn() -> u32 type known at compile-time for all extern functions noir may call into implies either us limiting the set of possible functions to call into, or standardizing the signatures on the C side into some combination of type-erased void*s. With this in mind, I see 4 main ways of implementing the "call into C code" functionality of extern from the rust compiler:

Have a limited number of extern functions in noir that are all builtin - the user can't define their own. This way we can just match over the name and cast to the corresponding type in each branch.
Allow users to define their own extern functions, but limit their maximum number of parameters. This way, we can match over the Noir function type to gradually build up the actual function type. This match will be exponentially uglier the more parameters that are allowed however. It would require 1 + M^N match branches if we want to allow functions with a maximum of N parameters to use M available noir types. Even if we only allow 4 primitive noir types to be used for functions up to 4 parameters, implementing this would still require 1 + 4^4 = 257 match branches, making it impractical.
Allow users to define their own extern functions, but standardize things on the C side, so that noir always expects to call a C function with the signature noir_value my_extern_fn(noir_args args); where noir_args and noir_value are both presumably typedefs for void** or void*. This would rely on the user to do any argument checking they want/need. It may also require we provide a small C header for managing noir_args. Example:

noir_value my_extern_fn(noir_args args) {
    assert(noir_args_len(args) == 2);
    int first = noir_int(noir_args_get(args, 0));
    noir_field second = noir_field(noir_args_get(args, 1));
    ...
    return noir_make_int(5);
}

Hybrid of 2 and 3 in which the C functions take in each argument as a void* but still have separate arguments compared to 3 where they are all packed into a single void**. Compared to option 3, this would require less casting and checking on the users part, and compared to 2 our implementation would still be limited to a finite number of parameters, but matching on the size would grow linearly (N+1 cases for N parameters) rather than exponentially.

typedef void* noir_value;

noir_value my_extern_fn(noir_value param1, noir_value param2) {
    int first = *(int*)param1;
    noir_field second = *(noir_field*)param2;
    ...
    // open question: do we require this return value to be owned so we may free it?
    int* ret = malloc(sizeof(int));
    *ret = 5;
    return ret;
}

This last option also allows a slightly simpler version since the different pointer types are binary compatible:
int* my_extern_fn(int* param1, noir_field* param2) {
    int first = *param1;
    noir_field second = *param2;
    ...
    return ret;
}

We went with a different strategy after-all for this where users can define foreign functions in brillig in order to do things like database calls. This ultimately allows us to abstract away the fact that C code is being used etc


:lipstick: change shevron style on testimonial qna

Improve Testimonial Q&A Component: Enhance SVG Arrow Animation
The current implementation of the SVG arrow in the Testimonial Q&A component uses a 180-degree rotation to indicate the expansion of a question. While functional, this approach results in a misalignment of the SVG when a question is opened, detracting from the overall user experience. Replace the 180-degree rotation animation with a transition to a minus symbol ("-").

Type of Change
Please delete options that are not relevant:

[x] 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)
[ ] Documentation update
[ ] Other (please describe):

Checklist
Please ensure your PR fulfills the following requirements:

[x] I have read and followed the Contributing Guidelines.
[x] My code follows the code style of this project.
[x] My code respect as much as possible the W3C standards with corresponding ARIA roles and html semantics.

[x] I have performed a self-review of my code.
[x] I have commented my code (if necessary), particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation (if applicable).
[ ] My changes generate no new warnings.

Screenshots (if applicable)
If your PR includes UI changes, please include screenshots or GIFs showing the new behavior.
Additional Context
Please provide any additional context or information that might be useful for the reviewers, such as links to relevant discussions, references, or issues.

Thanks for the contribution! I’ll review it as soon as possible.
We might create a variant for this behavior since it uses client-side JavaScript. The original component was designed to work without any JavaScript, which had its own advantages. However, your version definitely makes the UI cleaner so we will definitively create a variant for this. 😉

Thanks again for the PR! I’ve reviewed it, and I think the idea is great, as I mentioned in the issue. However, one of the key advantages of this Q&A component is that it doesn’t require any JavaScript. The solution you propose could be an interesting variant, but it would introduce JavaScript where it’s not currently needed.
The position of the arrow can actually be adjusted with simple CSS by modifying the top property. Additionally, the arrow animation in your solution feels a bit random. The animation proposed in the issue should serve as the inspiration, as it offers a much smoother SVG transformation.

Thanks again for the PR! I’ve reviewed it, and I think the idea is great, as I mentioned in the issue. However, one of the key advantages of this Q&A component is that it doesn’t require any JavaScript. The solution you propose could be an interesting variant, but it would introduce JavaScript where it’s not currently needed.
The position of the arrow can actually be adjusted with simple CSS by modifying the top property. Additionally, the arrow animation in your solution feels a bit random. The animation proposed in the issue should serve as the inspiration, as it offers a much smoother SVG transformation.

Thanks for feedback, I have updated the as per requirement plz have a look!

The changes look great, but I still believe there's no need for JavaScript in this case (which includes the useRef from React). Additionally, the 180-degree rotation isn't necessary anymore, thanks to the smooth animation that transforms the minus into an arrow.
Since Cuicui aims to offer the easiest way to copy-paste, it would be better to keep the toggleIcon in the same file as the variant. This avoids having an unknown import at the top of the file.
I've made some adjustments, but I can't create a PR directly on your branch. However, here are the modifications for variant1 as an example.
"use client";

import { cn } from "#/src/utils/cn";

export default function GradientQnAVariant1() {
  return (

        <AdvancedGradiant className="opacity-50" />

        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />
        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />
        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />

  );
}

const Testimonial = ({
  title,
  content,
}: {
  title: string;
  content: string;
}) => {
  return (

            {title}

          <ToogleIcon />

{content}

  );
};

const AdvancedGradiant = ({
  className,
}: {
  className?: string;
}) => {
  return (
    <>

    </>
  );
};

const ToogleIcon = () => {
  return (
    <button
      type="button"
      className="relative size-5 text-neutral-600 dark:text-neutral-300"
    >

    </button>
  );
};

The changes look great, but I still believe there's no need for JavaScript in this case (which includes the useRef from React). Additionally, the 180-degree rotation isn't necessary anymore, thanks to the smooth animation that transforms the minus into an arrow.
Since Cuicui aims to offer the easiest way to copy-paste, it would be better to keep the toggleIcon in the same file as the variant. This avoids having an unknown import at the top of the file.
I've made some adjustments, but I can't create a PR directly on your branch. However, here are the modifications for variant1 as an example.
"use client";

import { cn } from "#/src/utils/cn";

export default function GradientQnAVariant1() {
  return (

        <AdvancedGradiant className="opacity-50" />

        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />
        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />
        <Testimonial
          title="What is the best thing about this product?"
          content="The best thing about this product is that it is very easy to use and it is very user-friendly. It is also very affordable and it is very easy to set up."
        />

  );
}

const Testimonial = ({
  title,
  content,
}: {
  title: string;
  content: string;
}) => {
  return (

            {title}

          <ToogleIcon />

{content}

  );
};

const AdvancedGradiant = ({
  className,
}: {
  className?: string;
}) => {
  return (
    <>

    </>
  );
};

const ToogleIcon = () => {
  return (
    <button
      type="button"
      className="relative size-5 text-neutral-600 dark:text-neutral-300"
    >

    </button>
  );
};

Thanks for the suggestions, and changes incorporated.


Text rendering

Took this out for a spin, and am impressed by the results!
I did find a regression, however, in the text input window. The cursor is always rendered at the start of the text, which does not occur on master. Perhaps the text width is incorrectly passed on?

I've noticed a few assert(false) calls. What's your intuition behind them? I presume you have a reason to believe the code won't be reached — perhaps leave a comment stating why.

Requires MSVC fix. Pinging @duncanspumpkin @IntelOrca

#undef small is needed after all includes of <windows.h>

https://stackoverflow.com/questions/27793470/why-does-smallgive-an-error-about-char
Sounds like we'll either have to #undef small every time windows.h is included, or use a different name. The latter is probably cleaner, considering the file is included in 9 places.


Fix QoL bug where exception occurs if you click console ID format button with no NAND loaded

I wrote this last night, I would have merged it then if i'd known the release was coming out so soon, oops! :)

I wrote this last night, I would have merged it then if i'd known the release was coming out so soon, oops! :)

Will check out tomorrow, thanks buddy :D
Josh


Joomla 4 and Bootstrap 4 compatibility

Sorry this was meant for some other repo. Kindly delete the issue


chore: 🤖 upgrade Storybook to 5.2

:tada: This PR is included in version 12.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Upgrade LibAVIF & use new flags for building only AVIF encoder/decoder

See the discussion in https://github.com/AOMediaCodec/libavif/issues/254 where this was implemented.
This allows us to avoid using ERROR_ON_UNDEFINED_SYMBOLS and build a truly separate encoder/decoder libs.

Note on the further changes above: also changed the build scripts to optimise the directory structure.
Before we were producing:
libavif-enc
  ...AVIF sources...
  build
    ...AVIF built with required params...
  ext
    aom
      ...AOM sources...
      libavif.build
        ...AOM built with required params...
libavif-enc-mt
  ...AVIF sources...
  build
    ...AVIF built with different params...
  ext
    aom
      ...AOM sources...
      libavif.build
        ...AOM built with different params...
libavif-dec
  ...AVIF sources...
  build
    ...AVIF built with different params...
  ext
    aom
      ...AOM sources...
      libavif.build
        ...AOM built with different params...

Now I changed the way AVIF looks for AOM and directories could be simplified to:
libavif
  ...AVIF sources...
libaom
  ...AOM sources...
build
  enc
    libavif
      ...AVIF built with required params...
    libaom
        ...AOM built with required params...
  enc-mt
    libavif
      ...AVIF built with different params...
    libaom
        ...AOM built with different params...
  dec
    libavif
      ...AVIF built with different params...
    libaom
        ...AOM built with different params...

That is, we no longer have to unpack multiple copies of same sources, but instead unpack them once and only produce different builds.


Development mode breaks at scale

Seems fsevents just wants to crap the bed:
(FSEvents.framework) FSEventStreamStart: register_with_server: ERROR: f2d_register_rpc() => (null) (-21)

Switching to a polling method doesn't fix either -- we need to explore a new watcher.

@Timer, have you tried to run brew install watchman? I recall having the similar issue which was resolved by installing watchman. Not sure if I had the same scale project (only about 400-500 js files).

I do have watchman installed but that only fixes Jest afaik, not webpack. It'd be awesome to switch webpack's watcher to a new one.

Jest uses Sane to abstract over it: https://www.npmjs.com/package/sane

Can you try the latest release and see if this has improved? If not please open a new issue.


nestopia-ue: 1.52.0 -> 1.52.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/0ldsk00l/nestopia/releases.
meta.description for nestopia-ue is: Cross-platform Nestopia emulator core with a GUI
meta.homepage for nestopia-ue is: http://0ldsk00l.ca/nestopia/
meta.changelog for nestopia-ue is: https://raw.githubusercontent.com/0ldsk00l/nestopia/1.52.1/ChangeLog
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.52.1 with grep in /nix/store/a3b9kh5ix57pzvhfvalh60jgy22alhkn-nestopia-1.52.1
found 1.52.1 in filename of file in /nix/store/a3b9kh5ix57pzvhfvalh60jgy22alhkn-nestopia-1.52.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nestopia-ue

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/c16b28422d58f0e5536d196cf3fb2b2c5d47592e#nestopia-ue

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a3b9kh5ix57pzvhfvalh60jgy22alhkn-nestopia-1.52.1
ls -la /nix/store/a3b9kh5ix57pzvhfvalh60jgy22alhkn-nestopia-1.52.1/bin

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

1 package built:

nestopia-ue

Maintainer pings
cc @AndersonTorres for testing.

@NixOS/nixpkgs-merge-bot merge


Airdrop contract(s) using SignatureBouncer

🎉 Description
It'd be cool if we had a simple Airdrop contract that used SignatureBouncer.sol to issue tokens (with emission variants like the crowdsale's Mintable/Allowance types) to those with valid signatures from a server.
Bonus points for a nodejs script that runs across a list of addresses and generates signatures for each of them.
More bonus points for a simple rails/express based api endpoint for generating these signatures which follows a flow like:

user signs some challenge data using metamask/hardware-wallet/whatever
user submits that signature to the server
server recovers the address from the signature
server checks to see if address is in off-chain whitelist
if so, server signs a SignatureBouncer hash allowing the address to call the Airdrop contract
user can then submit that signature to the airdrop contract and receive their tokens

[ ] 🐛 This is a bug report.
[x] 📈 This is a feature request.

Yeah. I expect a lot of contracts will want to have single-use signatures. Do you think it would be more gas efficient to keep track of the nonce per-sender (with perhaps a range check like nonce must be less than 1 for single-use) and then sign signatures with a nonce on the backend or would it be better to just do a mapping of bytes32 -> boolean to keep track of whether or not the signature has been used before. Probably the nonce version is better?

Not sure, the nonce version is probably more optimized but harder to code.
The nonce in my example was mainly to allow multiple issuances of the same amount to the same address


Wrong cursor position with line-height: normal

Description

Cursor renders in wrong position with this settings in user stylesheet:
atom-text-editor {
  font-weight: bold;
  line-height: normal;
}

Here is how it looks

Steps to Reproduce

Open user stylesheet (edit -> stylesheet)
Insert snippet that was desribed above
Save

Expected behavior:

Cursor renders exactly at selected line.
Actual behavior:

Cursor renders below the selected line.
Reproduces how often:

Always.
Versions
Atom 1.39.1
Ubuntu 19.04

Additional Information

Font on screenshot: this.

Verified using Atom Nightly v1.41.0-nightly7 on macOS 10.14.6

I've found that Atom use transform: translate for setting cursor position. Also we can add some position calculations for given line height, but will it be performance-cheap? I think, cursor position can be corrected by playing with position property in user stylesheet.
In my case, the main aim of setting line-height was getting more code on my low-resolution laptop screen, and I've done it by scaling editor view with ctrl + mouse wheel .

Closing this issue, adding position calculations is not performance-cheap. Since this is introduced with a user style mod. The fix should also be through a modification on the same user style.


[W-12550313] add accelerators icons

ref: W-12550313
added multiple icons for the upcoming accelerators launch, following the instructions from here. Sorry for the massive changes in icondefs.js, gulp decided that when it was building the UI bundle to format the file.
To see the actual changes, cmd+f the component names listed in https://github.com/mulesoft/dev-docs-internal/blob/integration/internal/antora-navigator.yml#L3, plus accelerators_home.
Validated via local build.

this also fixes some defects in the footer section of the docs site


Bump Dependencies #11858549891

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/11858549891

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


feat: use a proper block builder factory in batcher creation

This change is

#1384  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite


Migrate repos in https://github.com/cnf-testsuite to https://github.com/orgs/cnti-testcatalog

Migrate repos in https://github.com/cnf-testsuite to https://github.com/orgs/cnti-testcatalog

[ ] https://github.com/cnf-testsuite/tar
[ ] https://github.com/cnf-testsuite/find
[ ] ...

TODO: list all repos to migrate from https://github.com/cnf-testsuite in tasks above


Language system optimizations

Optimize the language system

[x] Generate languages on startup and use fallback language
[x] Serve only requested language, or default
[x] Fallback language is German
Use Zanata (optional) - maybe later

* [ ]  Fallback language is German

Maybe it would be better to use English as fallback language.
Or the best solution would be if you could change it in the config with the Language_Code.

I chose German because it will always be a complete translation because I use it.
A configurable option could be dangerous if the language was not complete.

Implemented in 948750b6008af433d7210dea667a0c85caedbb93


Fix IMG_SHA256 to match v0.5.0 release

Fix IMG_SHA256 to match v0.5.0 release
From https://github.com/genuinetools/img/releases/tag/v0.5.0

oops thanks I should really automate that :)


Add Aborted and Failed statuses, and new endpoint to support ingesting broad metrics

Description
This is a draft PR while I seek feedback for https://github.com/dockstore/dockstore-support/pull/478. Will add tests once it's out of draft mode.
This PR adds the ABORTED and FAILED execution statuses so that we can ingest Terra metrics with these status. Interesting side note, the Terra metrics also have Aborting and Running statuses (about 2000). I did not include those because those aren't completed states.
This PR also adds an endpoint that lists published workflows by path. This is used by https://github.com/dockstore/dockstore-support/pull/478 to help determine the TRS ID from the source_url of the execution. More details of how it's used in the other PR.
Review Instructions
Describe if this ticket needs review and if so, how one may go about it in qa and/or staging environments.
For example, a ticket based on Security Hub, Snyk, or Dependabot may not need review since those services
will generate new warnings if the issue has not been resolved properly. On the other hand, an infrastructure
ticket that results in visible changes to the end-user will definitely require review.
Many tickets will likely be between these two extremes, so some judgement may be required.
Issue
SEAB-5963
Security and Privacy
If there are any concerns that require extra attention from the security team, highlight them here.
e.g. Does this change...

Any user data we collect, or data location?
Access control, authentication or authorization?
Encryption features?

Please make sure that you've checked the following before submitting your pull request. Thanks!

[ ] Check that you pass the basic style checks and unit tests by running mvn clean install
[ ] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[ ] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[ ] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[ ] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[ ] Do not serve user-uploaded binary images through the Dockstore API
[ ] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[ ] Do not create cookies, although this may change in the future
[ ] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

(approved in theory I guess)


Update CSR log

Please provide a description for what this PR is for.
Add SDS connection information into CSR logs. For each CSR, we need to know what SDS request from which SDS connection triggers the CSR. If a CSR fails or certificate parsing fails, we need to know which SDS client is affected. Currently resource name is logged in CSR. For workload SDS, there could be multiple SDS requests in parallel, with same resource name and different connection ID. We need to associate every CSR activity to each SDS client.
Also add some method interface changes that unify log pattern and simplify parameter list.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
https://github.com/istio/istio/issues/13439

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ
To complete the pull request process, please assign wattli
You can assign the PR to them by writing /assign @wattli in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@JimmyCYJ: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-lint-master
99bbebfef976c6e0f2cd454103c85820ce46a845
link
/test istio-lint-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JimmyCYJ, quanjielin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [quanjielin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-framework-k8s-presubmit-tests-master

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

integ-galley-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-galley-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-mixer-k8s-presubmit-tests-master

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

integ-galley-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-galley-k8s-presubmit-tests-master

istio-racetest-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

integ-galley-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-galley-k8s-presubmit-tests-master

istio-racetest-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-racetest-master
/test integ-galley-k8s-presubmit-tests-master
/test integ-framework-k8s-presubmit-tests-master

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
d3d757cdf9f320f766a3b06641691e59fc7ad0f2
link
/test integ-new-install-k8s-presubmit-tests-master

integ-framework-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-framework-k8s-presubmit-tests-master

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

integ-galley-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-galley-k8s-presubmit-tests-master

istio-racetest-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-mixer-k8s-presubmit-tests-master

istio-pilot-e2e-envoyv2-v1alpha3-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3-master

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-istioctl-k8s-presubmit-tests-master

istio-racetest-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-racetest-master

integ-mixer-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-mixer-k8s-presubmit-tests-master

istio-pilot-e2e-envoyv2-v1alpha3-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

integ-new-install-k8s-presubmit-tests-master
7959b24d44960c51878ed60cea571bd85c736ef2
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Better export of interface in cmake.

From:
https://github.com/ExCALIBUR-NEPTUNE/NESO/pull/186#discussion_r1243953040
users should be able to do
target_link_libraries(${LIBRARY_NAME} PUBLIC neso::particles)

This should also remove variables like NESO_PARTICLES_LINK_FLAGS.

Closing as @seimtpow has implemented this and it is merged.


[5.2] Session doesn't store the 'user_id' in DB when using multiple user tables

The Illuminate\Session\DatabaseSessionHandler@getDefaultPayload method doesn't set the 'user_id' when using multiple user tables. It works only with the default users table.

What do you mean by a multiple user tables?

I mean a multi-auth setup where you have for example 2 tables: users & customers.
So in this case the App\User is the default Eloquent model shipped with Laravel and the App\Customer is my custom Eloquent model. I'm not using custom guards or service providers - just tweaking the auth.php config file like shown here.

I think these steps are working fine.
@themsaid
It's time to close this issue now.


feat: add report analytics settings [HEAD-975]

Description
This PR introduces the analytics flag. yesAnalyticsReport enables users to opt-in to analytics reporting. This setting allows the extension to collect usage statistics to help improve the product.
Note: this is a continuation from #392. You can dismiss #392 changes by looking at the commits of this branch. Please merge that first :)
Checklist

[x] Tests added and all succeed
[x] Linted
[x] CHANGELOG.md updated
[x] README.md updated, if user-facing

Screenshots / GIFs

Analytics Enable
Analytics Disabled

Change not relevant now 👋


fix CI error(xml file doesn't exist)

xml file

Omit raw xml file and add build.jl to download xml automatically to save git size.


Add a benchmark test for reading/ parsing data

Adds a benchmark test requiring the Mentat to create a class and function to read, parse, and store data from a json file. The data is a set of people with sever attributes. The hardest part for Mentat turned out to be the parsing (originally I thought it would be the class architecture). The hard part about the parsing for Mentat is people in a family are represented differently than single people. Most of the mistakes Mentat makes are just 5 or 6 different things.
I created 4 different prompts with varying levels of prompt engineering. The first three I tested 20 times each and they had these pass rates:
easy - 95%
easy_medium - 75%
medium - 40%
hard - probably 0% (I didn't record the tests)
I've set the test to use the easy_medium prompt for now because I think we could improve its performance just by improving the general Mentat prompt. But one thing I was considering was moving this test to another file and breaking it out into 1-3 tests so we could see how all of the prompts do across updates to the general Mentat prompt.

Nice benchmark! Certainly more involved than our older benchmarks. We needed this
I'll take another look after you address Paul's comments

Left a couple more comments about the import function; after resolving those and simplifying the test cases like I mentioned earlier this should be good to go.

This is fairly outdated and we have new benchmarks now, so I'm marking this closed.


Report bug link doesn't work on mobile

Tries to go to a relative path

How did you report this then? 🤔

on desktop :P


erroneous message about obstruction co-ordinates

I am setting an obstruction over openram so it can be routed to without causing DRC errors.
set ::env(GLB_RT_OBS)  "li1  0    0   2920    3520,
       met1 344.0 475.5 823.78 873.0,
       met2 344.0 475.5 823.78 873.0,
       met3 344.0 475.5 823.78 873.0,
       met4 344.0 475.5 823.78 873.0"

These are lower left and upper right co-ordinates.
Then when I run the flow I get this warning message:
[WARNING]: Specifying a routing obstruction is now done using the coordinates
[WARNING]: of its bounding box instead of the now deprecated (x, y, size_x, size_y).
[INFO]: Obstructions added over li1  0    0   2920    3520,
       met1 344.0 475.5 823.78 873.0,
       met2 344.0 475.5 823.78 873.0,
       met3 344.0 475.5 823.78 873.0,
       met4 344.0 475.5 823.78 873.0.

The message is erroneous because I'm already using co-ords of lower left and top right corners as per the docs.

It should be gone by now, yeah, I'll open a PR

Fixed in https://github.com/The-OpenROAD-Project/OpenLane/pull/1244


Add write batch

I have added write batch and updated unit tests.
Sorry for diff from previous PR.

@trK54Ylmz I have merged #6, please rebase to the latest master branch. Thank you!

I have removed repository today, but when I see the comment forked again and updated changes but PR does not sync the updates. is it okay, if I close this one and create new PR @xyb?
Also I started to get the same exact error on my Mac<PHONE_NUMBER>,
Caused by:
  process didn't exit successfully: `/Users/runner/work/rocksdb3/rocksdb3/target/release/build/librocksdb-sys-2b31[40](https://github.com/xyb/rocksdb3/runs/5054861329?check_suite_focus=true#step:10:40)e0065ce4f7/build-script-build` (exit status: 101)
  --- stderr
  error: header 'rocksdb/include/rocksdb/c.h' does not exist.
  thread 'main' panicked at 'unable to generate rocksdb bindings: ()', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/librocksdb-sys-6.28.2/build.rs:44:10
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Traceback (most recent call last):
  File "/Users/runner/work/rocksdb3/rocksdb3/tests/build.py", line 16, in <module>
    subprocess.run(["cargo", "build", "--release"], check=True, cwd=str(ROOT))
  File "/Users/runner/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['cargo', 'build', '--release']' returned non-zero exit status 101.
Error: Process completed with exit code 1.

@trK54Ylmz sure, please do what you find handy.


fix broken tests

please build

please build

please build

please build

please build

please build

please build

please build

please build

please build


Type callable for getParameterType() in ParameterDeriverTrait

Hi,
For the ParameterDeriverTrait::getParameterType method there is no declared type for the argument $callable.
https://github.com/php-fig/event-dispatcher-util/blob/29725c77467dddf637d08579e63b28a14209dbff/src/ParameterDeriverTrait.php#L19
There is a comment below explaining why but I'm not sure it's correct.
https://github.com/php-fig/event-dispatcher-util/blob/29725c77467dddf637d08579e63b28a14209dbff/src/ParameterDeriverTrait.php#L21
I have updated the trait locally in order to add the type callable and I tried the code below and it works well :
$listenerProvider = new RandomListenerProviderUsingTheTrait();

class CallableHolder
{
    public function onStdEvent(\stdClass $event)
    {
    }
}

$callabelHolder = new CallableHolder();

var_dump($listenerProvider->getParameterType([$callabelHolder, 'onStdEvent']));

Am I missing something?
Thank you!

See the comment on isClassCallable.  The code nominally supports ['ClassName', 'methodName'] for an instance of ClassName, whereas AFAIK that only qualifies as a valid callable-array in PHP if methodName is static.
Although, when I add the type hint locally and run the tests, they pass, despite having a test for a class/method that isn't yet instantiated.  Hm.  I wonder if that changed in some PHP version at some point?  I'm on 7.4 right now, but the code was written on 7.2 originally...  Anyone have a 7.2 system lying around to test on? :smile:

Ah, here we go.  It's if the class in question is not defined at the time of the call, but would be defined later.  cf: https://3v4l.org/BYiL2
I know I ran into that at some point during development of that routine, but I don't recall precisely what circumstances it was.  It's possible we could add the type hint, but I'm wary of discovering whatever that edge case was again, the hard way.

Improved the comment in #17 to explain why we can't type that param.

Nice job ! :)


Make django Admin interface support slightly better UX wise

Display some fields on table view ["task_name", "queue_name", "attempts", "status"]
Allow to filter on status
Display Events inline

Sorry no screenshot.
Closes ø

Successful PR Checklist:

[ ] Tests

[x] (not applicable?)

[ ] Documentation

[x] (not applicable?)

PR label(s):

[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A breaking 💥
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A feature ⭐
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A bugfix 🕵
[x] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A miscellaneous 👾
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A dependencies 🤖
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A documentation 📚

Hi there ! Thank you for your contribution ! As mentioned in the 2.0 release note:

The Admin panels are very very basic for now

😅
Thank you for tackling this, but if you have some spare time, I think we could go a bit further!

I think it would make sense if the first list_display element stays the pk. That or the call_string (potentially in a <code> tag)
OR I think it makes sense if the kwargs stay displayed in the listing view. It might make sense to pretty-print them and put them in a <pre> tag.
It might make sense to only display the name of the task function and not the module names, optionally keep the initials (p.d.d.t.index_book)
Would it make sense to include a brief summary of the last event too ? ("deferred 5 minutes ago" with a title mentioning the exact date time)

I tend to find proper use of bold, italics, underline etc, and the occasional emoji very nice. Anything that helps.
Colors can be nice though with some caveats: there are colorblind people, so color should duplicate some info, not be the only way to get some info. Can be text color or background color. Django Admin now supports both light and dark mode, so we need to check contrast with both.
Regarding the details view, I like the addition of events as inline. I think the only thing worth bikeshedding is the order of fields. I'd say: id, task name, args, status, queue, lock, queuing lock, priority, scheduled at, attempts
Thanks a lot :) If you'd rather not spend longer on this, please let us know, we can merge thing and follow up for other changes.

Thanks for the feedback. I'll to see how far I can get to incorporate all the suggestions.
Stay tuned

Would it make sense to include a brief summary of the last event too ? ("deferred 5 minutes ago" with a title mentioning the exact date time)

It would mean introducing another dependency similar to humanize to pretty print the timedelta.
I don't think it's worth it.
I will push my interpretation of your feedback. They improve even more the readability. I hope it matches your expectations :)

I could be wrong but timesince doesn't require humanize, does it ? (There's a python callable version in django.utils.timesince, I believe) (note: maybe I missed a deprecation notice or something?)

I could be wrong but timesince doesn't require humanize, does it ? (There's a python callable version in django.utils.timesince, I believe) (note: maybe I missed a deprecation notice or something?)

you didn't missed anything. I did. Thanks for the pointer. I will look into it.

I've added a few commit for additional ideas. Here's the result:
https://github.com/user-attachments/assets/06034308-8554-42da-af76-5602d1d5d158
I'm ready to merge. If you're ready too (in case you want to read/comment my commits), then let's merge.

Very nice Thanks for the additions. You can merge it.


Changing OSC ip Address cause application to hang and crash

When using vrcfacetracking, trying to change the ip to my other computer in the OSC address causes the application to freeze/crash
example:
If the other computer's ip would be <IP_ADDRESS> it would crash but when using the ip of the local machine <IP_ADDRESS> it would be fine.
This issue is consistent on my windows 11 machine and my windows 10 machine
VRCFT V: <IP_ADDRESS>
Logs didn't say anything about the OSC

Dumping some info from when I last looked into this on Discord: currently, VRCFT both listens and sends to the IP address configured in the UI. IIRC the crash occurs because it can't bind to an address that isn't on the local machine. Fixing this for the non-oscquery implementation would require different UI config option for IP addresses for listening/sending.
The second part of the issue is that OSCQuery hardcodes the address it advertises to "<IP_ADDRESS>", so if VRC is running on a different computer, it won't be able to send/receive to VRCFT. This seems intentional due to this code/comment, but this breaks setups where VRCFT is running on a separate PC -- especially so for someone trying to use standalone VRC.

Dumping some info from when I last looked into this on Discord: currently, VRCFT both listens and sends to the IP address configured in the UI. IIRC the crash occurs because it can't bind to an address that isn't on the local machine. Fixing this for the non-oscquery implementation would require different UI config option for IP addresses for listening/sending.
The second part of the issue is that OSCQuery hardcodes the address it advertises to "<IP_ADDRESS>", so if VRC is running on a different computer, it won't be able to send/receive to VRCFT. This seems intentional due to this code/comment, but this breaks setups where VRCFT is running on a separate PC -- especially so for someone trying to use standalone VRC.

i don't think that this is an issue since you are able to change the ip in launch settings for vrchat like stated here, and the ports / i do think that it is an binding issue on the part of the application, i have just build a version localy and confirmed two issues with the ip field
if you clear the ip field than the application will crash and next time you launch it the same thing will happen, i looked into the appdata/roaming/VRCFaceTracking folder for the settings json and there the ip would be blank, if you replace that with a valid ip the app would work correctly again, so something is going wrong when validating the saved address and when setting it to blank it doesn't correct its self to the default <IP_ADDRESS>,
i have also been able to confirm that the app hangs when setting it to a remote ip instead of straight up crashing

I think i can also add changing the listen port to 0 to allow OCS Query only the application hangs on startup/launch if there's no OCS Query service detected. Example if VRChat wasn't opened before the application was launched.


encode: reduce allocations

A buffer that can hold the whole packet is allocated early on. Instead of reconstructing most of the packet in order to compute the signature it is computed on sub buffers of the packet.
Still TODO is:

[x] tls-crypt encode
[x] avoid having to allocate a dummy hmac just to pass the hmac size
[x] the hmac dance could probably be simplified

I did similar changes for tls-crypt, but they are untested (for now)

this looks great! some minor remarks, and one train of thought is: should we use create_unsafe since we fill the entire buffer anyways?

It would be great to make similar changes for decode regarding hmac. That is, compute the hmac on the input buffer directly instead of reconstructing it from the parsed data. However, the decoding is done in parts and we "lose" the op_key part early on. We can reduce allocations in to_be_signed a bit, but ideally we would remove it.

We can for sure do that for decoding :) I also agree that decoding and reassembling / reconstructing it just for the hmac isn't a great idea.

this is a nice improvement. Feel free to merge. any further improvements can be done in separate PRs :)


query, query frontend: Add TLS support to redis_client

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Adds support for TLS connections to redis
Modified original PR #5312 to use NewConfigClient as per pharau
Adds tls_client_cert, tls_client_key and tls_ca_cert options for Client Certificate and custom Root CA.
Verification
Expanded unit tests to validate new configuration

While looking to update the docs (& changelog), I noticed that http_config block puts the TLS config in a subobject tls_config
Would it be better to structure the redis TLS config using the same idiom?

While looking to update the docs (& changelog), I noticed that http_config block puts the TLS config in a subobject tls_config
Would it be better to structure the redis TLS config using the same idiom?

I thought about it more and decided to go with the tls_config struct layout.


Crashed upon startup

Minecraft version
1.20.1
Describe the bug
Crashes
Steps to reproduce

Just run it

...
Logs
https://gist.github.com/mas6y6/4950f7df4aa8d38a98022c79707a272b
Additional context
No response

Please read the issue template more carefully. For modpack related issues, use the support channel on our discord server.

Before reporting, please ensure that:

You've installed a minimal set of mods required to reproduce the issue.
Issues with modpacks and excessive amount of mods will not be accepted, as they take a long time to diagnose.
Knowing which mods are causing problems allows us to focus on fixing the issue as soon as possible.
If you're unsure which mods might be at fault, try using binary search - removing half of installed mods
repeatedly until the faulty mod is found.

To upload logs, use an external paste site, such as Github Gist (recommended), Ubuntu Pastebin or Pastebin


Deprecate sui move utf8

We don't need our utf8.move since upstream string implements utf8.
As part of https://github.com/MystenLabs/sui/issues/4172

@ebmifa
This PR removes a Move module, which we want to mention in the release notes for 0.8.0
Whats the process for this?

@ebmifa This PR removes a Move module, which we want to mention in the release notes for 0.8.0 Whats the process for this?

We are now tracking this in the next release notes


promtool should support checking a single expression

Currently promtool support checking multiple expressions as defined in a rule file. If you want to lint a single expression, you need to arbitrarily/needlessly stick the expression in a rule file.
I propose extending promtool with the sub-command promtool check expressions that takes a number of expressions from a file or stdin, with one expression per line to be checked. Notably, this proposal does not support multi-line rules. After all, we're not in the context of YAML.
This functionality is especially useful when preparing promql expressions for other systems like Grafana. In my case, I'm managing Grafana dashboards in Git so I don't have the Grafana GUI to quickly tell me that my promql is invalid. I'd like to lint my Grafana JSON, extract the promql, and run it through a checker.

Maybe, a flag like -i for instantaneous checking of expressions might be better after promtool check expressions for implementing this.

If we need this feature to be implemented, I would like to pick this up.

Looking forward to use this feature 🙂

Following the lead of check metrics, I think it makes sense to pass in any file contents over stdin:
cat expressions.txt | promtool check expressions

For single expressions (besides just echo "expr" | ...) we could support:
promtool check expression expr

But I'm not sure if it's necessary

Hello from the bug scrub, this has been implemented via the promtool promql format command line tool.


Support set the ice-ufrag and ice-pwd for connectivity check

Checking the HTTPS API or UDP connectivity for WHIP tests can be difficult. For example, if the UDP port isn't available but the API is fine, OBS only says it can't connect to the server. It's hard to see the HTTPS API response or check if the UDP port is available.
This feature lets you set the ice username and password in SRS. You can then send a STUN request using nc and see the response, making it easier to check UDP port connectivity.

Use curl to test the WHIP API, including ice-frag and ice-pwd queries.
Use nc to send a STUN binding request to test UDP connectivity.
If both the API and UDP are working, you should get a STUN response.

我们需要合并到5和6

还有个相关的改进，使用HTTP GET访问WHIP API时，错误消息不够明确，参考 https://github.com/ossrs/srs/issues/3838


Huffman_Problem

I have solved the above problem in Hacker rank and the problem is in c++

Please accept the pull request for hacktoberfest2022

I've added the label, this will be counted as one contribution. Later I'll review and merge if needed. Thanks :)


[Bug]: Combobox doesn't render the selected item when using objects as values

Environment
Developement/Production OS: macOS Sonoma (14.4.1)
Node version: v18.20.3
Package manager<EMAIL_ADDRESS>Radix Vue version: 1.9.3
Vue version: 3.4.35
Client OS: macOS Sonoma (14.4.1)
Browser: Chrome 127.0.6533.89

Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-bdpgs4
Steps to reproduce
When binding objects (second combobox):

Click on the "chevron" icon
Select an option

Describe the bug
Input placeholder is not replaced with the selected value.
Expected behavior
Input placeholder is replaced with the selected value.
Compare this to using string values as a binding (first combobox)
Context & Screenshots (if applicable)
No response

I think it's really a documentation issue. Afaik, in case of object binding display-value prop must be be passed to ComboboxRoot. It's used to get string value of selected item.
https://github.com/radix-vue/radix-vue/blob/9c665d687387830c4047940e2aa847771e8902c4/packages/radix-vue/src/Combobox/ComboboxRoot.vue#L194-L196
Would be nice if Combobox could produce a warning in case of object value but missing display-value.
Fixed repro: https://stackblitz.com/edit/vitejs-vite-ihsric?file=src%2Fcomponents%2FComboboxObjects.vue

@romansp thanks! I think you're right. Setting display-value fixed it for me.
I'll send a PR to clarify the docs.


Update README

This updates the current main project README.md with a little more description and information relating to sass-lint and its current status/functionality. It also provides more of a quick start guide to some of the most frequent questions we receive regarding the CLI and also provides clear links to all parts of our documentation in an effort to reduce the number of configuration style issues we get.
Related to many issues but most recently #563
<DCO 1.1 Signed-off-by: Dan Purdy<EMAIL_ADDRESS>

Changes Unknown when pulling d37d4bab351a0114469eca7d362ed2edda13193b on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Changes Unknown when pulling 5f08adc545cc464a01679d0c8ff7de0ec5f80faf on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Changes Unknown when pulling 699f0ee521b9dcfe16b7d0e828ff45bb2deea450 on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Changes Unknown when pulling f15e0469730f2aa1f4ae8042f2796c2d2c948925 on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Changes Unknown when pulling 5da5f050b2ff4e4da7221d555927cbefade84b74 on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Changes Unknown when pulling 7dfe9974549c47459f69b0f1cb769e82e1598cbc on DanPurdy:feature/update-readme into ** on sasstools:develop**.

Should hopefully help with the issues.


NFS plugin should use correct mounter

Depends on #8044
The nfs volume plugin is using the incorrect mounter -- it should use the one passed through the NewBuilder and NewCleaner methods.
@thockin @eparis

LGTM


DS: JSON + Namespace + Included Elements

If you are setting up a JSON based dynamic Data Source, the namespace Duplicator will still be shown. Shouldn't it be hidden?
Does the Included Elements field make sense in this context? I cannot use xPath for JSON, can I?

Yeah, namespaces don't make a lot of sense in this case. What should we do about this? Pickable?

Does the Included Elements field make sense in this context? I cannot use xPath for JSON, can I?

You can, the JSON is converted into XML.
{
	"name": 'Brendan',
	"age": 23
}

Becomes:
<data>
	<name>Brendan</name>
	<age>23</age>
</data>

What should we do about this? Pickable?

Sounds like a good plan.

Unsure of how to implement Pickable in this situation as we don't want to switch between two elements, we just want to hide the existence of one depending on the option selected. Is pickable still the best option?

We've got two options:

either we use Pickable
or we write a few lines of JavaScript for this purpose.

Pickable should work just fine in this context: if we don't provide a pickable element for JSON the namespaces will be hidden and nothing will be shown. Isn't that exactly what we'd like to achieve?

Add this to admin.js and it should work (around line 34):
$('select[name="fields[dynamic_xml][format]"]').symphonyPickable({
	pickables: '#xml'
});

Ah, and the div around the namespace Duplicator should be id="xml".

Cheers, I'll implement that in a sec.


redshift carry over

We need to add redshifts to the object multi-band data frames.

According to #234, we have decided that in order to do this correctly it would require a refactoring of the code.  This is beyond the scope of what is necessary for the user facing tutorial notebook.


trying to fix git file conflicts, and change example app id/name

Manually modify in /example macos, linux, ios, android to use
package_id: com.getx.GetXDemo
name: GetX Demo
removed some uneeded project files for intellij

hahaha you really put a commit with "i hate git" message hahahhaa

I too hahaha
LGTM


Bugfix: Blood and Doom v1.0.2

Multiple minor tweaks

Added missing XP field
Added a Notes field below Equipment
Added Armor Proficiency
Add more space to some fields in the Path tab

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


[W5.7][W14-4] Yao Tengxiong

Added new command 'find_email', with the JUnit and IO test cases for it.

@yao-tx great work. Everything looks good!

@fzdy1914 The find_email command is meant to match the exact keyword given only. I think the description for the command in the user guide might have caused some confusion.
I will update the user guide for a better and clearer description. Thank you for the pointing that out.


Revert signatures of _register_s3_events and _register_s3_control_events

Reverts the signatures of "private" functions _register_s3_events and _register_s3_control_events to what they were before botocore 1.28.0.

Codecov Report
Base: 93.45% // Head: 93.39% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (853369f) compared to base (c6b5eef).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2808      +/-   ##
===========================================
- Coverage    93.45%   93.39%   -0.06%
===========================================
  Files           63       63
  Lines        13298    13298
===========================================
- Hits         12427    12420       -7
- Misses         871      878       +7

Impacted Files
Coverage Δ

botocore/client.py
97.83% <100.00%> (ø)

botocore/compat.py
91.33% <0.00%> (-1.34%)
:arrow_down:

botocore/httpsession.py
91.43% <0.00%> (-0.39%)
:arrow_down:

botocore/utils.py
78.26% <0.00%> (-0.20%)
:arrow_down:

botocore/credentials.py
98.54% <0.00%> (-0.11%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.