Restructure CHILLED code to be more trackable and customizable

Reorganize CHILLED code to be more package-friendly
Main changes are:

Function (input/output) driven code for more package-friendly development and not just scripts
Usage of Config object for more consistent default inputs
Inputs housed in Github for traceability and flexibility
Can be run from command line (takes arguments as inputs)
Runnable on UniCC cluster

Changes to inputs and parameters

There now exists a Config object (in utils.config) that has the default parameters that should be consistently used in the model, but now we can easily change each parameter via the Config object instead of looking for it in places across different scripts.
For each "version" of the model (such as "ALPS2023"), make sure a folder with the version name exists in data/chilled/version with the following files

Changes to usage
Instead of modifying scripts that has parameters set in the script, now the parameters can be set via the Config object and modified via arguments. Currently, using the run scripts, these are the arguments that be given from the command line:

Version name (default: "ALPS2023")
GCM scenario (default: "GFDL-ESM4")
RCP (default: "baseline")

Preprocessing of rasters can be done first without considering GCM and RCP. This can be accomplished by running the script (in /chilled/):
python -m run_preprocess.py -version "version_name"

The part of the model that collects and processes the climate data to derive energy intensities can be run using:
python -m run_main.py -version "version_name" -gcm "gcm_name" -rcp "rcp_name"

Note that the code is now also easily runnable on the UniCC server, which is nice for setting off multiple GCM-RCP runs at once.
How to review
For @amastrucci or @byersiiasa: please let me know if the code does not make sense now or if the usage is not intuitive/efficient.
Perhaps @glatterf42 can also review if the code structure and docs are okay.
Thank you!

Quickly jumped in here to understand what's going on with the docs: overall, you were moving in the right direction.

There were some dependencies missing in pyproject.toml (which I noticed since I started in a new venv)
Sphinx could not understand some of the imports. E.g. importing from util.config in message_ix_buildings/chilled/preprocess/message_raster.py makes sphinx look at all top-level modules (like pandas, numpy, or message_ix_buildings) and it doesn't find util there. So we can either use absolute paths (like I did now) or relative paths. This would then be from .util.config import Config in this case. Not entirely sure which one is preferable.

With just these two changes, the docs now built successfully for me, including the code reference part. To replicate (in your venv on a command line of your choice):

Go to doc/
Run a command like sphinx-build -T -E -b html -d _build/doctrees-readthedocs -D language=en . _build/html
Host your own server via python -m http.server (and stop with ctrl+c once you're done)
Access the link it shows with your favourite browser (something like http://<IP_ADDRESS>:8000/)
Navigate to _build/html/ and have a look around :)

Thanks @glatterf42 for your review and edits! A follow-up:

Sphinx could not understand some of the imports. E.g. importing from util.config in message_ix_buildings/chilled/preprocess/message_raster.py makes sphinx look at all top-level modules (like pandas, numpy, or message_ix_buildings) and it doesn't find util there. So we can either use absolute paths (like I did now) or relative paths. This would then be from .util.config import Config in this case. Not entirely sure which one is preferable.

Using the absolute paths is giving me an issue now when trying to run the python scripts in command line. For example, with using absolute paths, I am now receiving the error:
  File "/home/mengm/repo/message-ix-buildings/message_ix_buildings/chilled/preprocess/message_raster.py", line 12, in <module>
    from message_ix_buildings.chilled.util.config import Config
ModuleNotFoundError: No module named 'message_ix_buildings'

Within message_raster.py, if I change the import line to use the relative path from .util.config import Config, I receive the following error when running the script in command line:
  File "/Users/meas/iiasagit/message-ix-buildings/message_ix_buildings/chilled/preprocess/message_raster.py", line 12, in <module>
    from .util.config import Config
ModuleNotFoundError: No module named 'preprocess.util'

I figure I needed to move the relative path to the directory before the preprocess submodule, but if I tried to change the import line to from ..util.config import Config instead, I receive the following error:
  File "/Users/meas/iiasagit/message-ix-buildings/message_ix_buildings/chilled/preprocess/message_raster.py", line 12, in <module>
    from ..util.config import Config
ImportError: attempted relative import beyond top-level package

Do you have any suggestions on how to proceed? The only import line seems to work in order to run the script in command line is from util.config import Config, but it seems that's causing issues for the documentation build.
Thanks!

Huh, that sounds odd. My first thought was that this might be dependent on where you are when you execute the scripts, but this would rather be an issue with relative imports than with the absolute paths we have now.
My next guess would be that you might not have an editable install of message-ix-buildings, but if you produce various error messages after only changing one line, you likely do have an editable install.
The (maybe unfortunate) thing is: I can't reproduce your error messages. For me, this is what I see:
fridolin@fridolin-Latitude-5520 ~> (buildings) cd message-ix-buildings/
fridolin@fridolin-Latitude-5520 ~/message-ix-buildings (chilled/restruct)> (buildings) python
Python 3.12.4 (main, Jun  8 2024, 18:29:57) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from message_ix_buildings.chilled.util.config import Config
>>>
fridolin@fridolin-Latitude-5520 ~/message-ix-buildings (chilled/restruct)> (buildings) cd message_ix_buildings
fridolin@fridolin-Latitude-5520 ~/m/message_ix_buildings (chilled/restruct)> (buildings) python
Python 3.12.4 (main, Jun  8 2024, 18:29:57) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from message_ix_buildings.chilled.util.config import Config
>>>
fridolin@fridolin-Latitude-5520 ~/m/message_ix_buildings (chilled/restruct)> (buildings) cd chilled
fridolin@fridolin-Latitude-5520 ~/m/m/chilled (chilled/restruct)> (buildings) python
Python 3.12.4 (main, Jun  8 2024, 18:29:57) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from message_ix_buildings.chilled.util.config import Config
>>>
fridolin@fridolin-Latitude-5520 ~/m/m/chilled (chilled/restruct)> (buildings) python preprocess/message_raster.py
fridolin@fridolin-Latitude-5520 ~/m/m/chilled (chilled/restruct)> (buildings) git status
On branch chilled/restruct
Your branch is up to date with 'origin/chilled/restruct'.

nothing to commit, working tree clean
fridolin@fridolin-Latitude-5520 ~/m/m/chilled (chilled/restruct)> (buildings) cd ../..
fridolin@fridolin-Latitude-5520 ~/message-ix-buildings (chilled/restruct)> (buildings) python message_ix_buildings/chilled/preprocess/message_raster.py
fridolin@fridolin-Latitude-5520 ~/message-ix-buildings (chilled/restruct)> (buildings)

In other words: wherever I am on my system, I can import from message_ix_buildings as from any other package and I can execute message_raster.py without error.
So just to clarify: you pulled the latest changes from here (especially including new __init__ files) and this directly adapted your install of message-ix-buildings because you installed it via pip install -e ...? What are you using to manage your venv? Is the correct venv active? If you run pip list, do you see message-ix-buildings in that list? With a version like 0.0.2.dev7+gd68f29b.d20240725? And finally, which OS are you using?

Hi all, thanks for this, looks very good.
Main suggestion would be to add a bit more intro to the README to explain more about CHILLED, cite relevant papers, and explain a bit more about the input data types and outputs that are generated.

Hi all, thanks for this, looks very good. Main suggestion would be to add a bit more intro to the README to explain more about CHILLED, cite relevant papers, and explain a bit more about the input data types and outputs that are generated.

Thanks Ed! If it's alright, my preference at the moment would be to get this PR merged and then add the documentation in another PR (so that this one does not get too large). 🙂 I have created an issue for the documentation: #21

Code could still be cleaned up (e.g. by adding mypy and ruff like we have them in message_ix and addressing linting/formatting errors or by cleaning up comments and type: ignore statements), but this could also happen in a later PR. If saved for later, please open an issue to not forget about this.

As a note, I just created an issue for this: #22

Its only for Fridolin to approve ;)

@glatterf42 If it's okay could I go ahead and merge? :)


Resources on creating a proper device tree

Thanks for your help on #7.
Now that I sucessfully flashed the svf (with one warning: Max frequency for clock '$glbnet$main_clkout0': 11.27 MHz (FAIL at 50.00 MHz)) on my board I was following the next steps.
Therefore I was wondering whether you have some more resources on how you created the .dts for your boards.

Therefore I was wondering whether you have some more resources on how you created the dft for your boards.

Do you mean *.dts (device tree source files)? If so, it started out as a combination of the .dts generated by the RocketChip builder (see e.g. the DTS generated for the 64-bit-wide linux variant) and the MMIO (a.k.a. CSR) addresses generated by LiteX for the SoC peripherals: if you build with --csr-csv ./csr.csv or better yet, with both that and --csr-json ./csr.json, you can then run litex_json2dts.py --initrd-start 8 ./csr.json and use some of its output.
In your case, I'd start with nexys4ddr.dts (since it also uses the 64-bit-wide Rocket CPU variant):
- update here with the actual size of the RAM on your board (look for memory_region,main_ram,0x80000000,<SIZE>,cached, convert <SIZE> to hex, and you're good to go;
- replace this section with the matching portion of output from the litex_json2dts.py, and then INCREMENT each IRQ number in the pasted content by 1 (i.e., the IRQ for the UART changes from 0 to 1, and that of LiteETH from 2 to 3)!
In the near future, it is one of my goals for this project to "teach" litex_json2dts.py to generate the *.dts file automatically, and get rid of the individual board-specific examples in ./conf. But, in the mean time, doing something along the lines of what's listed above should take care of it.

Oh yes sorry for the type that's exactly what I meant.
Thanks for the help!

Which is the Max frequency for clock '$glbnet$main_clkout0': that you are getting for your (ECP5) boards?
I fail to upload the the boot.bin to the ULX3S and have read that this might be due to too low clock frequency. https://github.com/enjoy-digital/litex/issues/773
For the vexrisc cpu the frequency the Max frequency is above 50Mhz so it's all fine...

On Mon, Mar 29, 2021 at 10:36:05AM -0700, developandplay wrote:

Which is the Max frequency for clock '$glbnet$main_clkout0': that you are
getting for your (ECP5) boards?

Generally I get somewhere from 19MHz to 25MHz from nextpnr. It's
supposedly ultra-conservative, since the resulting design mostly
works OK at 50MHz regardless.

In case anyone else runs into this issue.
Make sure to run lxterm /dev/ttyUSB0 --kernel boot.bin --speed 115200 --kernel-adr 0x80000000 as the default address is 0x40000000 which of course won't work with rocket.
With this the upload over serial will work. However it takes about 1,5h.

Seems like my dts still wasn't set up properly however since BBL is not starting yet.

Turns out my riscv-pk was not up to date.
Now I finally get bbl to start on the arty.
However it gets stuck before the login screen.
bbl loader
              vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
                  vvvvvvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrr       vvvvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrrrrr      vvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrrrrrrr    vvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrrrrrrr    vvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrrrrrrr    vvvvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrrrrr      vvvvvvvvvvvvvvvvvvvvvv
rrrrrrrrrrrrr       vvvvvvvvvvvvvvvvvvvvvv
rr                vvvvvvvvvvvvvvvvvvvvvv
rr            vvvvvvvvvvvvvvvvvvvvvvvv      rr
rrrr      vvvvvvvvvvvvvvvvvvvvvvvvvv      rrrr
rrrrrr      vvvvvvvvvvvvvvvvvvvvvv      rrrrrr
rrrrrrrr      vvvvvvvvvvvvvvvvvv      rrrrrrrr
rrrrrrrrrr      vvvvvvvvvvvvvv      rrrrrrrrrr
rrrrrrrrrrrr      vvvvvvvvvv      rrrrrrrrrrrr
rrrrrrrrrrrrrr      vvvvvv      rrrrrrrrrrrrrr
rrrrrrrrrrrrrrrr      vv      rrrrrrrrrrrrrrrr
rrrrrrrrrrrrrrrrrr          rrrrrrrrrrrrrrrrrr
rrrrrrrrrrrrrrrrrrrr      rrrrrrrrrrrrrrrrrrrr
rrrrrrrrrrrrrrrrrrrrrr  rrrrrrrrrrrrrrrrrrrrrr

       INSTRUCTION SETS WANT TO BE FREE
[    0.000000] Linux version 5.12.0-rc4-157053-g2669e46c615f (martin@martin-ThinkPad-T14-Gen-1) (riscv64-unknown-linux-gnu-gcc (GCC) 9.2.0, GNU ld (GNU Binutils) 2.34) #1 Mon Mar 29 15:25:44 CEST 2021
[    0.000000] OF: fdt: Ignoring memory range 0x80000000 - 0x80200000
[    0.000000] Machine model: freechips,rocketchip-unknown
[    0.000000] earlycon: sbi0 at I/O port 0x0 (options '')
[    0.000000] printk: bootconsole [sbi0] enabled
[    0.000000] efi: UEFI not found.
[    0.000000] Zone ranges:
[    0.000000]   DMA32    [mem 0x0000000080200000-0x000000008fffffff]
[    0.000000]   Normal   empty
[    0.000000] Movable zone start for each node
[    0.000000] Early memory node ranges
[    0.000000]   node   0: [mem 0x0000000080200000-0x000000008fffffff]
[    0.000000] Initmem setup node 0 [mem 0x0000000080200000-0x000000008fffffff]
[    0.000000] On node 0 totalpages: 65024
[    0.000000]   DMA32 zone: 1016 pages used for memmap
[    0.000000]   DMA32 zone: 0 pages reserved
[    0.000000]   DMA32 zone: 65024 pages, LIFO batch:15
[    0.000000] SBI specification v0.1 detected
[    0.000000] software IO TLB: mapped [mem 0x000000008fbfa000-0x000000008fbfa800] (0MB)
[    0.000000] riscv: ISA extensions acdfim
[    0.000000] riscv: ELF capabilities acdfim
[    0.000000] pcpu-alloc: s0 r0 d32768 u32768 alloc=1*32768
[    0.000000] pcpu-alloc: [0] 0
[    0.000000] Built 1 zonelists, mobility grouping on.  Total pages: 64008
[    0.000000] Kernel command line: earlycon=sbi console=liteuart swiotlb=noforce
[    0.000000] Dentry cache hash table entries: 32768 (order: 6, 262144 bytes, linear)
[    0.000000] Inode-cache hash table entries: 16384 (order: 5, 131072 bytes, linear)
[    0.000000] Sorting __ex_table...
[    0.000000] mem auto-init: stack:off, heap alloc:off, heap free:off
[    0.000000] Memory: 242252K/260096K available (3729K kernel code, 4061K rwdata, 2048K rodata, 2808K init, 270K bss, 17844K reserved, 0K cma-reserved)
[    0.000000] SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1
[    0.000000] NR_IRQS: 64, nr_irqs: 64, preallocated irqs: 0
[    0.000000] riscv-intc: 64 local interrupts mapped
[    0.000000] plic: interrupt-controller@c000000: mapped 4 interrupts with 1 handlers for 2 contexts.
[    0.000000] random: get_random_bytes called from start_kernel+0x3a0/0x558 with crng_init=0
[    0.000000] riscv_timer_init_dt: Registering clocksource cpuid [0] hartid [0]
[    0.000000] clocksource: riscv_clocksource: mask: 0xffffffffffffffff max_cycles: 0x1d854df40, max_idle_ns:<PHONE_NUMBER>960 ns
[    0.000023] sched_clock: 64 bits at 1000kHz, resolution 1000ns, wraps every<PHONE_NUMBER>500ns
[    0.006260] Console: colour dummy device 128x32
[    0.008372] Calibrating delay loop (skipped), value calculated using timer frequency.. 2.00 BogoMIPS (lpj=10000)
[    0.013330] pid_max: default: 32768 minimum: 301
[    0.017647] LSM: Security Framework initializing
[    0.020518] Mount-cache hash table entries: 512 (order: 0, 4096 bytes, linear)
[    0.023993] Mountpoint-cache hash table entries: 512 (order: 0, 4096 bytes, linear)
[    0.047797] ASID allocator disabled
[    0.050736] EFI services will not be available.
[    0.056118] devtmpfs: initialized
[    0.080913] clocksource: jiffies: mask: 0xffffffff max_cycles: 0xffffffff, max_idle_ns:<PHONE_NUMBER>2750000 ns
[    0.085668] futex hash table entries: 256 (order: 0, 6144 bytes, linear)
[    0.093595] NET: Registered protocol family 16
[    0.212096] clocksource: Switched to clocksource riscv_clocksource
[    0.343163] NET: Registered protocol family 2
[    0.352595] tcp_listen_portaddr_hash hash table entries: 256 (order: 0, 4096 bytes, linear)
[    0.356830] TCP established hash table entries: 2048 (order: 2, 16384 bytes, linear)
[    0.360970] TCP bind hash table entries: 2048 (order: 2, 16384 bytes, linear)
[    0.364923] TCP: Hash tables configured (established 2048 bind 2048)
[    0.369107] UDP hash table entries: 256 (order: 1, 8192 bytes, linear)
[    0.372598] UDP-Lite hash table entries: 256 (order: 1, 8192 bytes, linear)
[    0.377886] NET: Registered protocol family 1
[    0.937063] workingset: timestamp_bits=46 max_order=16 bucket_order=0
[    1.181944] Block layer SCSI generic (bsg) driver version 0.4 loaded (major 254)
[    1.188538] LiteX SoC Controller driver initialized: subreg:4, align:4
[    2.088147] loop: module loaded
[    2.264756] libphy: Fixed MDIO Bus: probed
[    2.276462] liteeth 12001000.mac eth0: irq 2, mapped at ffffffd004003000
[    2.300900] NET: Registered protocol family 10
[    2.318173] Segment Routing with IPv6
[    2.320626] sit: IPv6, IPv4 and MPLS over IPv4 tunneling driver
[    2.333819] NET: Registered protocol family 17
[    2.338181] Warning: unable to open an initial console.
[    2.382536] Freeing unused kernel memory: 2808K
[    2.384950] Run /init as init process
[    2.386312]   with arguments:
[    2.387787]     /init
[    2.388917]   with environment:
[    2.390481]     HOME=/
[    2.391653]     TERM=linux

Success on the arty #13 !!!

I will try to see if I can get linux to boot on the ulx3s next.

Closing this as #14 and #12 are not dependent on it.

Hi @developandplay
I'm also getting hang at this step when trying on DE2-115 board
[   31.277582] sit: IPv6, IPv4 and MPLS over IPv4 tunneling driver
[   31.476334] NET: Registered PF_PACKET protocol family
[   32.024800] Warning: unable to open an initial console.
[   32.817778] Freeing unused kernel image (initmem) memory: 2700K
[   32.862982] Run /init as init process
[   32.867512]   with arguments:
[   32.876518]     /init
[   32.882666]   with environment:
[   32.886522]     HOME=/
[   32.892520]     TERM=linux

Can you share, what did you had done to fix this issue?

Rebuild kernel with default option + Enable SBI v1 make it work for me as https://github.com/litex-hub/linux-on-litex-rocket/issues/21


CASSANDRA-19340 - Support writing UDTs

This commit brings support for UDTs to the Bulk Writer. It converts Spark nested StructType(s) into UDT values.

Rebased/squashed/fixed CHANGES.txt conflict (no other changes/conflicts)


Create de_DE.lang

Not yet done. Will work on it and commit to this branch. Opened here requesting feedback from other native speakers.
I decided to not translate the words Flux and Warp as they are very hard to translate while keeping their meaning and not sounding stupid (I could not find an appropriate German word). Problem is that words like "Taintwood" cannot be translated well, it also wouldn't fit to the word tainted (which is being translated). I would really like to find a good word for "taint". Any suggestions? Best words I could find for "taint" would be "Verderben", "Fäulnis", "Schmutz" but neither really fit the true meaning of the word.
There is no good translation for the word "eldritch" either so I am switching between various words like "bizarr" and "unnatürlich" depending on context.
This mod has always been extremely fun to translate because it allows me to use a lot of old German words you don't see very often and also forces me to get creative with naming. I'd really like to find a good translation for the word "taint" to improve the quality of this translation.
Due to a lack of time, I likely will not be translating the Thaumonomicon entries. I am sorry about that.

Its cool that you guys are getting going with this so long, but just a word of warning - this is definitely subject to change. There is also a lot of old stuff in the lang file that will be removed at some point.


Fix issues when using protobuf-c from another project

I initially submitted the wrap after only testing that it would produce a valid .so, but I wasn't using it from another project yet.
This fixes a pretty important issue, which is the missing dependency declaration...

drive by comment here - but it looks like the upstream repo has moved on a lot from this original change.  not sure if djdeath's original commits exist in their repo either.

here's a commit that adds the files you're asking for.  (apply with git-am.  not sure how else i can edit/help/extend this PR).
0001-Update-readme-and-LICENSE.patch.txt


Xamarin.Android does not obfuscate on Deploy to Emulator/Device

I have configured according to https://docs.obfuscar.com/tutorials/xamarin.html#project-file for my xamarin android project.
Project

SampleLibrary.dll (netstandard)
Sample.Droid.dll

In Debug Mode I can Build and Deploy to emulator/device everything works and obfuscated.
In Release Mode I can Build and both .dll is obfuscated BUT when deploy to emulator/device the code is in plain text.
I used ILSpy on the dlls under "obj\Release\110\android\assets" SampleLibrary.dll is NOT obfuscated while Droid.dll is obfuscated.
Same problem for Archive and Distribute.
Did i miss some configuration or is it a bug?

Managed to solve it, in the exxample "" was used, had to change it to example "" and in csproj copy dll from that folder instead. Now Release build and deploy to emulator/device is obfuscated.


why not transfer ownership?

@juicycleff you can transfer repository ownership, which I think would be less confusing than copy-pasting for end-users, and maintain the github repo-relationship graph

Hey, @micimize Sorry had to relocate to Norway so was unavailable for a bit. How are you? Regarding your question Well, I will think about it though. Wil get back to you asap


Do not install gmock

Fixes the issue described in https://github.com/googlecartographer/cartographer/pull/1041.
The docker file is not to be merged, it is only here to help reproducing the issue and fix.

Verified also for cartographer_ros' Indigo docker image.

This fixes the issue for me :+1:
Tested on Ubuntu Bionic and Debian Stretch

To reproduce, use the docker file from the previous commit https://github.com/gaschler/cartographer/blob/7de7c8717f3a704d8a197095188bb89a21ec5057/Docker.reproduce

@wally-the-cartographer merge

Merge requested by authorized user gaschler. Merge queue now has a length of 2.


compatibility issue with django-oauth-toolkit==2.1.0

I am keep getting the "invalid client error" when I had django-oauth-toolkit==2.1.0 installed.
I had to change it  to django-oauth-toolkit==1.3.2

what version of django and drf did you use?

I think I didn't change django version - Django==3.2.14

I had some time to investigate this. Indeed, it does not work with django oauth toolkit equal or above 2.0.0. Reason is the new check_password(client_secret, request.client.client_secret) condition added in oauth2_providers/oauth2_validators, line 180. It is a bug on the password hasher.

@ehsan-g how did u change the version of django-oauth-toolkit?

@shrivardha
`django-environ==0.9.0
django-oauth-toolkit==2.1.0 // compability issue with social auth
django-oauth-toolkit==1.3.2
djangorestframework==3.12.4
drf-social-oauth2==1.2.1
social-auth-app-django==5.0.0
social-auth-core==4.3.0`


feat(jwt): enable token in cookie

Close https://github.com/honojs/hono/issues/437

Thank you for your quick response!


Roll Plugins from 4d15b7a16bda to a83d8efed219 (3 revisions)

https://github.com/flutter/plugins/compare/4d15b7a16bda...a83d8efed219
2021-05-20<EMAIL_ADDRESS>[image_picker] Fix camera alert localization analyzer warning (flutter/plugins#3939)
2021-05-20<EMAIL_ADDRESS>[camera] Migrate maven repo from jcenter to mavenCentral (flutter/plugins#3908)
2021-05-19<EMAIL_ADDRESS>[sensors] Update Cirrus image to Xcode 12.5, fix sensors analyzer warning  (flutter/plugins#3930)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Created Malayalam Readme file

Readme file translated to Malayalam (India)

 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.:x: san45 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


use MathX.DistanceSqr instead of MathX.Distance

allegedly this improves performance. if you look at the code for Distance it just calculates DistanceSqr and passes it into Sqrt so you avoid that sqrt. of course you need to square the other side of the equation but I think multiplication is more efficient than sqrt
https://github.com/Nytra/ResoniteDistantComponentOptimizer/blob/main/DistantComponentOptimizer/BasicPatcher.cs#L34

well sqrt is just raising it to the 0.5 power, rather than the 2 power for sqr
would be interesting to test if one is actually faster than the other

mathematically it is, but a computer does not simply "raise to a power". squaring a number is multiplying it by itself, sqrt is a little more complex

Fixed https://github.com/Nytra/ResoniteDistantComponentOptimizer/releases/tag/1.0.1


org settings: Add edit_topics_by_admins_only.

Add a realm setting that allows users to edit topics of any message.
I wanted to check in and see if the wording on the frontend pieces is correct -- the way I edited settings_org to display the success messages seems wordy.
I am going to continue with a few other pieces. In particular, I'll look through the tests to see what else I can add, and I'll try to add a function that updates the UI, similar to toggle_allow_message_editing_pencil.
Fixes #5947

Thanks @sarahcstringer for writing this!
I agree with you that the new text in settings_org feels wordy. The message editing section in the settings UI now feels complicated to me, too:

and I suspect those are related.
Here's an attempt at making the settings UI clearer:

Users can edit their messages

Users can edit topics on any message
(time limit)

I think flipping the sense of the option to match "Users can edit their messages" makes it clearer.
Also, thinking about this feature, I think we want a time limit to apply to editing others' topics, just like for one's own messages -- otherwise, in many orgs there will be people for whom the temptation for mischief is just too great. In my sketch above, that'd be the same time limit as for message content, but we could also make it different (maybe 60min by default?):

Users can edit their messages

(time limit)

Users can edit topics on any message

(time limit)

That feels a little more complex, but I think the right values are probably different for the two time limits.
@rishig, what do you think of the above?

hm, I hadn't thought about the temptation for mischief. I guess a time limit seems reasonable. I think we'd want admins to be able to edit forever (which is similar to what they can do now), and we'd still want to support a user simultaneously editing every message with a given topic, though they'd have to send a new message to that topic to reset the clock.
It would be nice to easily support editing the topics from the left sidebar itself, which this makes a bit harder. But I think that's probably fine.
Maybe 24 hours by default for the topic editing?

Yeah, admins should be able to edit forever, like now.
I think older messages from beyond the time limit should be out of reach
for a user to edit the topics. One form of obnoxiousness here would be to
reply to some other people's messages on a topic, and go edit the topic to
something that reflects your own idiosyncratic view of the matter, or makes
their messages look foolish. I think it's key that when someone edits your
message's topic, it should at least be recent enough that when you look at
the thread you'll remember what it originally was and know that they
changed it. So thinking through that, it does argue for a long enough time
limit to cover a solid discussion, longer than an hour.
24 hours could work for that, though I wouldn't want to go much beyond it;
I think I'd default to 12 hours. Past 12 or 24 hours, I think the
educational value in a Zulip-savvy user editing someone else's topics to
make the stream demonstrate good use of topics also rapidly diminishes, as
there are new conversations to set and read the topics on instead.

For 12 vs 24, in general I think it's better to set these defaults with newer realms in mind, since a realm getting even 10s of messages a day probably has an admin that can set the value appropriately for the realm.
For editing topics older messages, I guess the question is what should the last of the following three options do?

Hmm. Gosh, the text of those options is confusing -- the "to this topic"
could mean either "messages [sent] to this topic", or "change ... to this
topic", and I'm reading it both ways or alternately, like a Necker cube. :-)
I think a fine solution would be to cut out the third of these options.
Then the two choices would be "this message" and "this and following", and
the latter option is enough to avoid any tedious repetitive clicking --
just go to the first message you want to change. That's also already the
behavior we silently do when an admin edits a topic.

One important detail is that the "all messages to topic" one edits all messages with that topic in that stream's history, I think using the backend, and thus is more robust than hitting the home key repeatedly to find the first one.  That feature is certainly useless if we expect folks to restrict topic edits temporally.  We should look at the commit history for when it was added; I vaguely recall it was by user request like 4 years ago :).

Setting defaults for newer realms is a good point -- though I think the
quantitative threshold is different. There are a lot of org options, and it
takes a pretty energetic admin to actually go through them all and make
real choices. I'd guess maybe at 300+ messages/day, or like 10+ really
active users, that starts being a good assumption.

I actually would not at all assume that existing organizations will find out about new settings unless they are advertised somehow.  I've told the org admins of large organizations like RC about new settings 6 months after they were implemented, and seen them be excited about the new setting.

Yep, also a good point.

Thanks for the discussion! To clarify for moving forward:

Update the wording for this setting
Add a time limit for topic edits by users

Default is 12 hrs?
Is there a max time?

I need to add the dropdown box when editing older message topics, to select which ones to change; I'm not sure that displays currently in this PR.

On the from of the dropdown box for what to change, @rishig @gnprice I think we should try to make this PR be just about the new setting, and not about changes to the "Edit a topic" UI.  While this adding this setting may make us want to update that topic editing UI, I don't see a reason why the changes are actually related, and it's way easier to do 2 things separately than together...

yup, no need to change the dropdown for this PR, other than to perhaps remove that third option.
That being said, if there is some intermediate thing that can be merged (e.g. something that allows topics to be edited forever, if at all), it would probably be worth getting that mergeable now, so that we have it for gsoc. It shouldn't affect any existing realms, since the setting will be off by default.

Okay, I updated a few pieces of this PR but am working on a new one that adds the time limit.

Can you explain what you mean by "adds the time limit"?

Sorry, yes -- I was thinking the next step is adding the organization setting for a "topic editability" time limit for users, so that admins could choose to make topics editable up to a certain time after messages are created.

Thanks so much for this feedback @gnprice! I'll get started on updating this :)

I updated the migration file to reflect the original issue, which suggested that the default value for this setting be True, but that we should migrate all past realms in as False.

I made the following changes:

Changed the setting name
Added a max 24-hr limit for all users editing topics
Decoupled the message editing settings and made them part of the property_types framework.

Before I update all the frontend tests, I wanted to check in on the success message portion of this -- previously, the messages for "users can edit the topics of their message" and "users can edit the content of messages up to __ min __ old" went together. Should the messages remain this way, or should they all be separated out into separate success messages?

I think separated success messages is better (and the simpler the better, since I think we're going to get rid of them at some point.)

Okay, sounds good. Thanks!

Hello! Apologies, I've lost track of what changes I should make on this PR still. I was thinking I should go over the tests some more, but am no longer sure what other major fixes should happen.
I have other code for decoupling the success messages that I was going to submit separately once this feature was finished.

@sarahcstringer sorry for the slow reply!  I think just doing a rebase and testing it carefully is probably all I need from you before trying to merge this.

I just updated this PR and split the work up into a few different commits instead of one giant one :)

@sarahcstringer do you have time to rebase this?

@timabbott Looking at it now; apologies, this PR totally fell off my radar.

@sarahcstringer oops, did you update this and then not comment?  I'd had no idea, since GitHub doesn't send notifications if all you do is a push.
There's some merge conflicts with more of the feedback work we're doing on the settings pages.

@timabbott I just rebased and have been playing around with this feature. One piece that I need to fix is when a user has narrowed to a specific topic, and the someone changes all of those messages to have a different subject name. Currently, two browser errors pop up rather than being able to handle that change.

Update: I think the commit I just pushed fixes that browser error issue.

The UI commit merge conflicts with the recent rework completed on simplifying the organization settings codebase.  But I'm working on integrating the rest, since they don't require that commit.

@sarahcstringer I merged most of this PR!  A few things remain:

I switched the default for new organizations to False for now so this remains disabled until we're ready.  We'll want to add a migration to flip that default back to True when we do the final merge.
We should extend the message_edit node tests to cover the new editability code paths.  tools/test-js-with-node --coverage is helpful for seeing what it's doing (noted above).
Two checks in JavaScript for whether to show an "edit topic" option need to interact properly with the 24-hour time limit; I think the right approach involves using get_editability for those, but needs confirmation.  (I squashed the time commit into the other logic one, since the tests failed in between the two).
I didn't merge the message_viewport change, since I think it's the wrong fix; I'll work on a correct fix for it independently.  So you can not worry about that problem.
You'll need to rebase the "Settings UI" commit.

Thanks for all your hard work on this!  I think the rest of this should be pretty easy to integrate.

Closing in favor of #9176, which is merging shortly.  Thanks for all your work on this @sarahcstringer!


Make one-key-formatting also works in any title field

Now that parse-backticks feature works in issue titles, commit titles and more places, why not make one-key-formatting also works in any title field? I think many RGH users who enable parse-backticks feature have get used to adding backticks when writing a title. It would be a great convenience.
IMO, not all the symbols supported in comment field also need to be supported in title textarea. Maybe supporting [ ` ' " is enough.
BTW, it seems that one-key-formatting doesn't work consistently. But I don't know how to reproduce it. Just encouter this bug again and again🤣.
URL: issue page, pull request page and more

all the symbols supported in comment field also need to be supported in title textarea. Maybe supporting [ ` ' " is enough.

👍


[Question] Material UI (next) + Nextjs + Styled components

I have looked into the examples for nextjs+material ui (next) and nextjs + styled components but I am not sure if that require additional work.
Any suggestion?
Many thanks!

solution can be found here https://github.com/zeit/next.js/issues/3983


Add zoom feature

Works with mouse wheel + ctrl key

Looking great! Very nice feature! This makes me think we should start a "Powered by rdkit-js" section in the main rdkit-js repository.
Looking forward to try this project.

Closing this. Going with https://github.com/iktos/molecule-representation/pull/28 solution.


[UNDERTOW-804] fix incorrect session-avg-alive-time value.

https://issues.jboss.org/browse/UNDERTOW-804
upstream PR: https://github.com/undertow-io/undertow/pull/420

I think this whole approach may need to change, as there are some thread safety issues. In particular expiredSessionCount and averageSessionLifetime are not updated atomically.

I have changed this so it now has a sync block. I think there may have been issues with rounding errors as well as expired sessions get large.


Cannot install guvnor

Hi,
I've tried to install guvnor using
npm install -g guvnor --unsafe-perm
but after several seconds the process was stopped with a generic "killed" message on console.
[1]+  Killed                  npm install -g guvnor --unsafe-perm
Killed
Any idea?

npm sometimes takes a lot of memory when installing - how much memory does your system have available?

On 7 Jan 2016, at 18:50, Daniele Margutti<EMAIL_ADDRESS>wrote:
Hi,
I've tried to install guvnor using
npm install -g guvnor --unsafe-perm
but after several seconds the process was stopped with a generic "killed" message on console.
Any idea?
—
Reply to this email directly or view it on GitHub.


Update README.md

Allowing the first argument of m() to be an object (module) is quite brilliant. But it takes a review of the source to understand that a tag is created (per mithril.js) and the object continues to be available. I thought you might want to include a line showing the use of that object and maybe a bit of explanation.
className:line.attrs.className
I forked your example on Plunker which illustrates the point even further. The .item class is never used, thus the CSS applies to nothing. I also fixed errors in the CSS, so I believe the result is what you intended.
http://plnkr.co/edit/4m45IErwDBE6Zb08XbNZ?p=preview
I also added the (ctrl) argument to a view in this README.
REALY GREAT WORK.

Hi @pelonpelon. Actually the intended result is to auto merge the component attributes (onclick) into the outer element attrs (className). Your fix highlighted a Mithril.Elements bug that seems to have been introduced at the same time as this. And that compounded by my lousy CSS in the plunkr has not made my day :(
Thanks for taking the time to review. fixes Mithril.Elements#3


Use ctapipe for dl1 merging

I dont know if the lstchain tool supports more than the ctapipe merge tool, but if everything we need is in ctapipe, it might be worth to switch tools?

Not what we need for lstchain/hiperta, because the later stages use lstchain scripts and assume the lstchain format.
For ctapipe it has been added to #111


Improve documentation for cached pages usage

Hi,
I think an important information is missing in the documentation: when a page is cached, the request is not performed, but any check is silently ignored. So not only an user would think the checks are okay, but also any extraction / saveAs does not work with cached pages.
I spent 3 hours to understand this behavior, so maybe it could help someone else in the future.

It's all open source, feel free to contribute: https://github.com/gatling/gatling/tree/master/src/sphinx/http


Format week number

Codecov Report

Merging #934 into master will increase coverage by 0.04%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #934      +/-   ##
==========================================
+ Coverage   85.62%   85.66%   +0.04%
==========================================
  Files          12       12
  Lines         612      614       +2
  Branches      116      117       +1
==========================================
+ Hits          524      526       +2
  Misses          4        4
  Partials       84       84

Impacted Files
Coverage Δ

src/month.jsx
82.6% <ø> (ø)
:arrow_up:

src/calendar.jsx
86.48% <ø> (-0.24%)
:arrow_down:

src/datepicker.jsx
91.77% <100%> (+0.05%)
:arrow_up:

src/week.jsx
74.07% <60%> (+3.24%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8e85fa...5c11eb1. Read the comment docs.


Package Parse 1.7.0 is not compatible with netcoreapp1.0

+1

Closing as no details provided. Can't really address this fully by just the title. If this is still issue please provide an adequate description of what the issue is and we can open this back up.


Not able to update git domain credentials with out a re-add

Description of problem:
If the credentials need to be changed for the git domain, the only way to update them is to remove the domain, then re-add.
Version-Release number of selected component (if applicable):
5.10.14
How reproducible:
always
Steps to Reproduce:

Add a git domain through Automation > Automate > Import/Export
Attempt to change credentials

Actual results:
No option in the UI, has to be removed and re-added
Expected results:
option to update credentials
Note - The user is able to update the credentials without having to first remove the domain.
So, while there is no UI action to specifically update the git domains credentials, specifying the new credentials on the Automation -> Automate -> import/export page updates the credentials properly.
Created as a result of: https://bugzilla.redhat.com/show_bug.cgi?id=1799086

@h-kataria @skateman @himdel Can you please validate if this is a UI issue and if so, size it? Thx

I assume we don't know how they attempted to do step 2: "Attempt to change credentials"?
Because clearly the "No option in the UI" is negated by the note saying that you can do that in the place where you had to do it originally.

I think the reporter assumed that each datastore updates itself and has associated credentials, or something like that.
The UI right now looks more like once-off import, where you can set username/password.
If the UI is right, we can close this.
If there is indeed supposed to be some kind of auto update from git, then we need to change the UI accordingly.
Does anyone know which it is?

Hi @himdel
The UI is correct. :-)
We support a set of credentials for each git domain imported.
They could import multiple git domains, each with their own credentials.

@tinaafitz if the UI is correct, I am confused on what this issue is about.  My understanding of this issue is that the UI needs to be enhanced to allow user the option to update credentials.

Interestingly, the "Import Datastore via git" bit of the UI is using old forms, yet we also have an unused ImportDatastoreViaGit react implementation from https://github.com/ManageIQ/manageiq-ui-classic/pull/5180, reverted in https://github.com/ManageIQ/manageiq-ui-classic/pull/5792 to fix https://bugzilla.redhat.com/show_bug.cgi?id=1714493.
So how it works now is that MiqAeToolsController#retreieve_git_datastore (form submit) calls (ui) GitRepositoryService#setup which creates or updates a GitRepository model entity, does update_authentication on it, and calls (core) GitBasedDomainImportService#queue_import.
But I'm not sure if anything does repeated imports or refreshes on the GitRepository, is there something like that?
Is that something that can be done via the API too?
If the UI is the only way to update from git, the existing UI is fine, since it already allows you to set everything before you re-import.
If refresh of GitRepository is periodically scheduled, then the BZ is correct and we need a new UI screen to show GitRepository entities, and allow editing their associated credentials, as well as APIs for these.

Hi @chessbyte,
I think it's a user experience enhancement request.
I closed the BZ, and opened the issue to track the request.


Bug#148

Fixing a bug, removing mpld3 library.

I added a legend. I'm afraid that too many groups might make it look a little messy, but it is how it is.
I also changed the code in a way that it can be easily reused for plotting variants/groups of samples.
Have a nice weekend!

@Setnea I see you are trying to solve why travis builds fails unexpectedly. Can you try commenting out the total argument?
https://github.com/dizak/mothulity/blob/c16907efc6f46aabae449a27287763ca786eda6d/mothulity/utilities.py#L443

ascii argument in the tqdm function can accept bool or str. Can you please try with passing ascii=' 123456789#'? Yes, whitespace is the first char in this str


Add cmake support with cmake-format

Set the default formatter for cmake files to cmake-format. shiftwidth and textwidth are taken into account.

Thanks


When the column name includes sign, C3 cannot tell the difference between signs

C3 Version: v0.4.11
Browsers: Chrome(Version 50.0.2661.94 (64-bit))   Safari(Version 9.1)
*Bug: * One column name is with '+' and the other is '-', when I only select 'data+' from legend, it will highlight both.

Please, try to work around this behavior by adding other strings to the column names. For example, adding an empty string may force C3 to cast the label as such, whereas in your case it might be evaluating it.

The fix is released in v0.5.2.


dT in MSR Pronghorn input assumes uniform fuel temperature

A better measure of the delta temperature would be weighted by the mass flow rate
An even better measure would use the energy to compute the effective dT in terms of energy

Fixed in current set of inputs


The resource ibm_kms_key generates a warning use ibm_kms_key_policies

Using https://github.com/terraform-ibm-modules/terraform-ibm-cluster/tree/master/examples/secure-roks-cluster
The resource ibm_kms_key generates a warning.
│ Warning: Argument is deprecated
│
│   with module.kms.ibm_kms_key.key,
│   on .terraform/modules/kms/modules/key-protect/main.tf line 23, in resource "ibm_kms_key" "key":
│   23: resource "ibm_kms_key" "key" {
│
│ Support for creating Policies with the key will soon be removed, Utilise the new resource for creating policies for the keys => ibm_kms_key_policies

@powellquiring
Analysing the issue


Cannot create XWPFTable in XWPFDocument document

Hello, I make Android application with possibility to create docx file with text and table in it. For creating a file with table I use the next code:
 val fileOutPutStream =
                FileOutputStream(File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS).path + "/mytable.docx"))

            val document = XWPFDocument()
            val tableOne = document.createTable(2, 2)
            tableOne.getRow(0).getCell(0).text = "1"
            tableOne.getRow(0).getCell(1).text = "2"
            tableOne.getRow(1).getCell(0).text = "3"
            tableOne.getRow(1).getCell(1).text = "4"

            document.write(fileOutPutStream)
            fileOutPutStream.close()
            document.close()

And file is created well. But when I open it, I see only texts without tables. Here is screenshot of document:

What I make wrong? How to fix this? Thanks.

Does it work if you do the same with a normal Java application on a PC? Because this sounds like a problem of Apache POI itself, so you might be better off reporting the problem there if this is still an issue.

No response for a long time so closing this one for now, please reopen with more information if this is still an issue for you.


Refactor reading from connection

If you have problem with receiving data from server. Can it happen you will get only a part of chunk?
what about something like this?
$msg = '';
while (true) {
  $header = $this->connection->read(2);
  $length = unpack('n', $header)[1] ?? 0; //because 0x00 0x00 end mark is chunk of size 0

  if ($length > 0) {
    $chunk = '';
    $size = mb_strlen($chunk, '8bit');
    while ($size < $length) {
      $chunk .= $this->connection->read($length - $size);
    }
    $msg .= $chunk;
  } else {
    break;
  }
}

not tested.. written in this comment.
Originally posted by @stefanak-michal in https://github.com/neo4j-php/Bolt/issues/85#issuecomment-1057084967
Also you should read header until you readed length 2.

I added you to make you notified about updates


Reclassify VCS WorkingTree tests

Please have a look at the individual commit messages for the details.

@fviernau would you mind merging this despite the ongoing unrelated AppVeyor failure?


Use a custom http transport implementation

This allows us to log raw responses before decoding to json and
customize the transport for example by setting a timeout (not in this
PR).
For the implementation I used our gpv1 custom transport and rust-web3's
default transport as references and read the jsonrpc specification.
I discovered that that neither previous implementation takes into
account that the server is allowed to reorder responses for batch
requests. This is fixed here and something we might want to upstream
into rust-web3.
Related https://github.com/gnosis/gp-v2-services/issues/664
Test Plan
existing tests, manual tests with staging node (TODO)

Very good catch! According to the JSON RPC spec:

The Response objects being returned from a batch call MAY be returned in any order within the Array. The Client SHOULD match contexts between the set of Request objects and the resulting set of Response objects based on the id member within each Object.

Given how similar this transport is to what we wrote in GPv1, would it make sense to factor it in a reusable crate to that the safe gateweay, ido services and whoever else uses Rust and Web3 in the context of Gnosis can reuse it.

I would like to and am working on upstreaming this into rust-web3. Then everyone can benefit. If that doesn't work out then yes we could easily have this as a standalone crate.


Fixed build-status-icon to match Dev.Azure

Changed unittest-project to use net461. The standard netframework installed with VS2019.
Changed main-project to use net452. That matches the coming Akka.Net ver. 1.4 nuget-package.

@Danthar Updated to ver. 1.3.5

Downgrade to net 452 since that matches the Akka target. Fair enough.
Note that common.props settings are only for local build stuff. The build takes its parameters from releases.md.

Not sure I understand the "build takes its parameters from releases.md" but have now reverted the changes in common.props


Missing Authentication Guide

[ ] Add AUTHENTICATION.md file
[ ] Add AUTHENTICATION.md to .yardopts
[ ] Add AUTHENTICATION.md to *.gemspec

Does it make sense for generated gapic gems (google-cloud-security_manager-v1beta1) to have this guide? Or should this guide be present in the veneer gems (google-cloud-security_manager) only?


Enrichment rooms should be managed in Rectory Apps, not in Keystone

Right now the default enrichment rooms and descriptions are defined in Keystone, which has the two hour sync issue. Since the rest of Enrichment is managed in Rectory Apps, this component can be moved there as well.

Enrichment is probably going to be a rewrite with the new SIS, this bug will not be particularly valid


Automatically install the NuGet credential provider if NuGet feeds are configured

What are you trying to accomplish?
Allow users to authenticate with private Azure DevOps NuGet feeds without any additional configuration or complicated workarounds.
Reduce the number of issues users are having with private NuGet feed auth. e.g. see comments in:

https://github.com/tinglesoftware/dependabot-azure-devops/issues/921#issuecomment-2162273558
https://github.com/tinglesoftware/dependabot-azure-devops/issues/1201#issuecomment-2242074907
https://github.com/tinglesoftware/dependabot-azure-devops/issues/1206#issuecomment-2245030650
https://github.com/tinglesoftware/dependabot-azure-devops/issues/1232#issuecomment-2245129307

Changes
If DEPENDABOT_EXTRA_CREDENTIALS contains credentials for a NuGet feed, the workaround detailed in https://github.com/tinglesoftware/dependabot-azure-devops/issues/921#issuecomment-2162273558 will be applied-- more specifically, VSS_NUGET_EXTERNAL_FEED_ENDPOINTS will be set using the configured credentials, then the Azure Artifacts Credential Provider is installed to the container.
This automatically runs for both update_script.rb and update_script_vnext.rb.

There is an existing task in Azure Pipelines that installs & configures NuGet authentication:
https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/nuget-authenticate-v1
Would it be possible to glean information from that automatically instead of requiring another environment variable to be set?

There is an existing task in Azure Pipelines that installs & configures NuGet authentication: https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/nuget-authenticate-v1
Would it be possible to glean information from that automatically instead of requiring another environment variable to be set?

Unfortunately that task doesn't have any impact on the update process since Dependabot is run inside a Docker container which is isolated from the Azure pipeline.
With this change, the user does not need to set any additional environment variables; the script will use the auth provided in your dependabot.yml registries and pass it through to the Azure Artifacts Credential Provider so that NuGet and MSBuild have access to them.


fix warnings&errors of doc building

Description
Please describe here what changes are made and why. If this pull request fixes a GitHub issue, reference it.
Checklist

[ ] I have covered the code with tests
[ ] I have added comments to my code to help others understand it
[ ] I have updated the documentation to reflect the changes
[ ] I have performed a self-review of the changes

I would rather copy return type definition to each function definition as well

I would rather copy return type definition to each function definition as well

Yes, that's better.


"(out of sync)" labels re-aligned

The "(out of sync)" labels were all over the place...I've never noticed this because my wallet syncs for half a second or so...too fast to notice that
Before:

After:

Fixed the out-of-sync Out of Sync labels? hah, thanks!


fix(deps): :arrow_up: update all non-major dependencies

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@sentry/angular-ivy (source)
^7.75.1 -> ^7.76.0

ace-builds
1.31.0 -> 1.31.1

core-js
3.33.1 -> 3.33.2

Release Notes

getsentry/sentry-javascript (@sentry/angular-ivy)
v7.76.0
Compare Source
Important Changes

feat(core): Add cron monitor wrapper helper (#9395)

This release adds Sentry.withMonitor(), a wrapping function that wraps a callback with a cron monitor that will automatically report completions and failures:
import * as Sentry from '@sentry/node';

// withMonitor() will send checkin when callback is started/finished
// works with async and sync callbacks.
const result = Sentry.withMonitor(
  'dailyEmail',
  () => {
    // withCheckIn return value is same return value here
    return sendEmail();
  },
  // Optional upsert options
  {
    schedule: {
      type: 'crontab',
      value: '0 * * * *',
    },
    // 🇨🇦🫡
    timezone: 'Canada/Eastern',
  },
);

Other Changes

chore(angular-ivy): Allow Angular 17 in peer dependencies (#9386)
feat(nextjs): Instrument SSR page components (#9346)
feat(nextjs): Trace errors in page component SSR (#9388)
fix(nextjs): Instrument route handlers with jsx and tsx file extensions (#9362)
fix(nextjs): Trace with performance disabled (#9389)
fix(replay): Ensure replay_id is not added to DSC if session expired (#9359)
fix(replay): Remove unused parts of pako from build (#9369)
fix(serverless): Don't mark all errors as unhandled (#9368)
fix(tracing-internal): Fix case when middleware contain array of routes with special chars as @ (#9375)
meta(nextjs): Bump peer deps for Next.js 14 (#9390)

Work in this release contributed by @LubomirIgonda1. Thank you for your contribution!
v7.75.1
Compare Source

feat(browser): Allow collecting of pageload profiles (#9317)
fix(browser): Correct timestamp on pageload profiles (#9350)
fix(nextjs): Use webpack plugin release value to inject release (#9348)

ajaxorg/ace-builds (ace-builds)
v1.31.1
Compare Source
v1.31.0
Compare Source
Features

add nasal language (#5342) (4fff99e)

Bug Fixes

doc tooltip not filtering properly (e5034c7)

zloirock/core-js (core-js)
v3.33.2
Compare Source

Simplified structuredClone polyfill, avoided second tree pass in cases of transferring
Added support of SuppressedError to structuredClone polyfill
Removed unspecified unnecessary ArrayBuffer and DataView dependencies of structuredClone lack of which could cause errors in some entries in IE10-
Fixed handling of fractional number part in Number.fromString
Compat data improvements:

URL.canParse marked as supported from Chromium 120
Updated Opera Android 78 compat data mapping
Added Electron 29 compat data mapping

v3.33.1
Compare Source

Added one more workaround of possible error with Symbol polyfill on global object, #1289
Directly specified type: commonjs in package.json of all packages to avoid potential breakage in future Node versions, see this issue
Prevented potential issue with lack of some dependencies after automatic optimization polyfills of some methods in the pure version
Some minor internal fixes and optimizations
Compat data improvements:

String.prototype.{ isWellFormed, toWellFormed } marked as supported from FF119
Added React Native 0.73 Hermes compat data, mainly fixes of some issues
Added NodeJS 21.0 compat data mapping

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm WARN cli npm v10.2.2 does not support Node.js v20.2.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
npm ERR! code ERR_SOCKET_CONNECTION_TIMEOUT
npm ERR! errno ERR_SOCKET_CONNECTION_TIMEOUT
npm ERR! request to https://registry.npmjs.org/esbuild-register failed, reason: Socket connection timeout

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-11-02T17_36_03_947Z-debug-0.log


Persist successful retry

Fixes #21

Coverage increased (+0.004%) to 99.717% when pulling e7049b57daebb963635401dd4e53578371f75017 on persist-successful-retry into 5bb157c15e508813ac5e575cd256c848c750f7f0 on master.


Bug: signing up with another user's email breaks flow

Issue Summary
Clicking on a user invitation email takes you to the signup form. Here you can change your email address. If you change your email address to be the same as an existing user then you will correctly receive an error that the email address is in use:

After this, clicking on the Create Account button does not allow you to change email address and does not transition user to admin. From the console you can see that the validation token is now invalid and the user has been created. You can now log in using the original email address and password provided.

Possible solutions include:

API change - Not invalidating token or saving user if user.email is not unique
On ERROR: SQLITE_CONSTRAINT: UNIQUE constraint failed: users.email error - continue with  authentication and transition to user's profile page. Possibly show the error as alert or notification at this point.
Lock out email address for signup page

Technical details:

Ghost Version: master (latest commit: 0f9f0dd69f8c85569b087f70170de930bfb30511)
Client OS: Win 7 Ultimate x64
Node Version: 0.12.5
Browser: Chrome Version 44.0.2403.130 m
Database: SQLite

I am going to raise a PR using the 3rd solution idea as a quick fix for now but won't close this issue as it probably warrants further discussion as to the best approach.

I think not allowing users to change their email on signup was a totally sane fix for this. We've not had a single complaint about the restriction. Closing this as fixed.


Seperate training of reconstruction-task and property-prediction

Dear All,
In the paper it is mentioned that 250 000 drug-like molecules were used to train the autoencoder-system
And for training the  Gaussian-process 2000 Molecules were used.
However the provided command line tool only provides one input.
Therefore the question: Is it possible to train the property-prediction and reconstruction-task separately, or how was this separated training achieved in the paper?
How should the code be executed, assuming we have the following two data sets?:
• a limited number of Molecules with SMILES and properties and
• a large number of Molecules with SMILES and without properties, (maybe overlapping with the smaller dataset)
Kind regards!

Hello, do you figure out this question? Many thanks.

Hi All,
It sounds like you might like to do semi-supervised learning.
That is not currently supported by this repo, but you might be able to
adapt it for your purposes. I would suggest looking at this paper for some
ideas: https://arxiv.org/abs/1805.00108
Best,
Jennifer
On Fri, Jun 28, 2019 at 9:35 AM wkl000<EMAIL_ADDRESS>wrote:

Hello, do you figure out this question? Many thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_aspuru-2Dguzik-2Dgroup_chemical-5Fvae_issues_5-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DADT3XUHH4DLN5OT3IFWU2XLP4YHSLA5CNFSM4E3BGN72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY2C5HA-23issuecomment-2D506736284&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=UPzYrSHLXjnX3tYn90C8Ljjzb-yfrb1UtMOxOFh-tKk&m=qQfuQlGjhPcCvD22MpEG4HdqWm4ydziJI9XdkfoIEt4&s=gA8sbh69Qb1m6rNGN5AFpTIYFzWRID3tHKVAFtVc2e0&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ADT3XUGPYIARAV4WW3AAAPDP4YHSLANCNFSM4E3BGN7Q&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=UPzYrSHLXjnX3tYn90C8Ljjzb-yfrb1UtMOxOFh-tKk&m=qQfuQlGjhPcCvD22MpEG4HdqWm4ydziJI9XdkfoIEt4&s=miVhcpk8csZyND4dXeg_YPXWVUwaQRTEYI-cpRWUlfI&e=
.

Hi all,
I also encounter the same problem, so I modified the code by myself. Please take it for your reference.
After finishing training the VAE itself by a large number of Molecules with SMILES and without properties, and then I change the original data into the data which has a limited number of Molecules with SMILES and properties in the exp.json and add a key called "do_prop_pred_only" and set its value is true.
the whole parameter setting
https://github.com/AustinApple/modified_chemvae/blob/master/exp_property_training.json
In order to train the property predictor separately, I made some modifications in the train_vae.py
https://github.com/AustinApple/modified_chemvae/blob/master/train_prop.py
If there is any question, please let me know.

One problem I foresee is that if you don't train jointly the VAE with the property predictor, then the latent-space
will not be organized according to the property. So, you are doing something quite different from what is reported in their paper.


[ckks] Missing function EncodeNTTNew in Encoder interface

The function EncodeNTTNew is implemented, but is not anymore in the Encoder interface.

Hi ffsinger,
You are right, this method was mistakenly removed, and I'll add it back in the PR#96. Thank you very much for your feed back.

Hi ffsinger,
You are right, this method was mistakenly removed, and I'll add it back in the PR#96. Thank you very much for your feed back.


[CARBONDATA-1423] added integration test cases for presto

@chenliang613 can you please review this pr?

ok to test

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3430/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/332/

retest this please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/528/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/529/

@anubhav100  please fix conflict.

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/668/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/669/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/670/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/671/

Retest this please

@chenliang613 merge conflict is fixed

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/673/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/674/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/679/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/690/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/696/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/706/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/708/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/827/

retest this please

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/715/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/15/


Slurm-scripts

This PR adds examples on how to use slurm for parallel reconstructions for birefringence and phase.
Both pipilines convert the data to zarr, create an empty zarr store for the output and parallelize the reconstructions per position to simultaneously write to the output zarr.
These scripts run using our zenodo dataset, but bigger datasets really take an advantage using HPC.

I think I need to add documentation on how to run the code.
The scripts are bash scripts and the phase or birefringence reconstructions can be executed by bash batch_bire_recon.sh or bash batch_phase_recon.sh respectively. Thank you both!

Thanks for these utilities @edyoshikun.  Yes, a quick how-to on staging the data will be useful. These can be comments in specific scripts or a markdown guide linked in the docs.
Have you already implemented a utility that allows the reconstruction of data parallelized by positions, preceded by or followed by, the creation of metadata at all positions within the zarr store? Is iohub's metadata API now modular enough to do so?

Have you already implemented a utility that allows the reconstruction of data parallelized by positions, preceded by or followed by, the creation of metadata at all positions within the zarr store? Is iohub's metadata API now modular enough to do so?

I think this file does this (although using iohub internals which are now wrapped in public API, see following comment).

@edyoshikun this PR no longer needs a review, correct? Can you change to draft or close?


UI Shell: Missing "selected" state for header items

What package(s) are you using?

[x] carbon-components
[ ] carbon-components-react

Detailed description

Describe in detail the issue you're having.

We want to use the selected state of the UI Shell header item which is already presented on the documentation website: https://www.carbondesignsystem.com/components/UI-shell-header/style#menu-item

I couldn't find how that selected state is getting triggered and there's seems to be no reference to it on neither website nor on the header style file.

Is this issue related to a specific component?

UI Shell

What did you expect to happen? What happened instead? What would you like to
see changed?

There should be documentation on how to enable that selected state.

What browser are you working in?

Chrome

What version of the Carbon Design System are you using?

10.7.3

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

IBM ADDI

@asudoh Should I move this issue to the carbon-design-system/carbon repo?

@stanislavgeorgiev I see the website needs to be changed to use the live demo instead of a mockup, and thus moved this issue from http://github.com/carbon-design-system/carbon to here, but don't hesitate to speak up if you see otherwise - Thanks!

ok, I got what happened now. This issue is asking to have a selected class for the header items not to update the website to use live demo instead of a mock up thus I'd rather keep it in the Carbon repo where it was originally filed if that's ok with you.


Create swagger documentation

As a [role]
I need [function]
So that [benefit]
Details & Assumptions:

...

Acceptance Criteria:
Given [some initial state]
When [some event]
Then [some measurable outcome]

Not a current part of the requirements.


iptables v1.4.21: can't initialize iptables table `nat'

I am struggling to get the docker image to run using a CentOS 8 host.  I get the below error.
iptables v1.4.21: can't initialize iptables table `nat': Table does not exist (do you need to insmod?)
Perhaps iptables or your kernel needs to be upgraded.

My docker-compose.yaml is a follows:
version: '3.3'

services:

  vpn:
    image: 'wheelybird/openvpn-ldap-otp:v1.4'
    restart: always
    hostname: 'vpn.d1.innoapps.co.uk'
    environment:
      - OVPN_SERVER_CN=vpn.d1.innoapps.co.uk
      - OVPN_ENABLE_COMPRESSION=false
      - ACTIVE_DIRECTORY_COMPAT_MODE=true
      - USE_CLIENT_CERTIFICATE=false
      - LDAP_URI=ldap://<IP_ADDRESS>
      - LDAP_BASE_DN=dc=innoapps,dc=co,dc=uk
      - LDAP_BIND_USER_DN=CN=VPNService,OU=ServiceAccounts,DC=innoapps,DC=co,DC=uk
      - LDAP_BIND_USER_PASS=SECRET
      - OVPN_DNS_SERVERS=<IP_ADDRESS>,<IP_ADDRESS>
      - OVPN_DNS_SEARCH_DOMAIN=innoapps.co.uk
      - OVPN_PROTOCOL=tcp
    cap_add:
      - NET_ADMIN
    ports:
      - 1194:1194
    dns:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
    dns_search:
      - innoapps.co.uk

Do you have the iptables modules loaded on the host server?  It'd be strange if you didn't as Docker sets up some rules to get its networking stuff to work.  Still, give it a go:
sudo modprobe ip_tables; sudo echo 'ip_tables' >> /etc/modules
Otherwise (and always with CentOS) it could be SELinux issues.  Try disabling SELinux to see if that helps:  sudo setenforce 0.


📃: Add a Star Rating Component

:red_circle: Title : Star Rating Component
:red_circle: Aim : User can dynamically rate and give stars out of 5. After selecting , when the submit button is clicked then a thank you message is displayed in which the corresponding rating given by user is shown.
:red_circle: Tech Stack Used : HTML,CSS,JavaScript
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Meet Arora
What is your participant role? GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@meetarora10 go ahead


Use gitignore.in

Add .gitignore.in
Generate .gitignore from .gitignore.in

Codecov Report
All modified lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


YAML Inputs for Anomaly Detection

modified schema yaml
updated test cases with contamination

@ahosler This issue was recently resolved in feature/anomaly_detect branch. I rebased this branch with it and all the test cases are passing now.


Error ECONNABORTED sendFile

Hello :),
I have a question regarding res.sendFile function. What happens when there is an ECONNABORTED error in callback function ? Is there a way to handle it in my code ? It seems that express hangs forever after that error is thrown in sendFile. I am using express v. 4.17.1 and node v. 10.13.0
sendfile(res, file, opts, function (err) {
    if (done) return done(err);
    if (err && err.code === 'EISDIR') return next();

    // next() all but write errors
    if (err && err.code !== 'ECONNABORTED' && err.syscall !== 'write') {
      next(err);
    }
}

Hello Sewemark. I'm here to help as best I can with your issue. Do you have a small repo or gist than encapsulates your issue? I will setup a small sample repo if not so we can firstly reproduce your issue. Then we can take a look at it.

Closing due to no response.


Treat application/xhtml+xml as HTML

This makes rack-livereload work with XHTML as well.
This fixes nanoc/nanoc#1473.

@onesupercoder Hello! What needs to be done to get this merged and released?

@ddfreyne i'm sorry but i never saw this 😢

Thank you for fix.


Maven installation problem: Could not resolve dependencies

Been facing dependencies installation issues, for installing jota locally.

As you can see in the above image, this is the index which maven resolves to leveraging the pom.xml.
However the pom.xml file in the index redirects to other directory. Which is called jota.
<dependency>
<groupId>com.github.iotaledger.iota.lib.java</groupId>
<artifactId>jota</artifactId>
<version>0.9.7</version>
</dependency>

However if you traverse directly to this directory, jitpack says it doesn't exist. The below image suffices maven's order of fetching the jar.

There are two possiblities. Either the jota directory on that index is deleted or restricted.
Someone with access to jitpack could resolve the issue.

Another thing is that, the jar is sitting right there for the latest version, however the pom.xml is still redirecting to jota directory.

I ended copying the contents of pom.xml from the project repo and adding the jota dependency. I was able to make it run.
My pom.xml dependencies:
    <dependencies>
        <dependency>
            <groupId>com.github.iotaledger.iota~lib~java</groupId>
            <artifactId>jota</artifactId>
            <version>0.9.10</version>
        </dependency>
        <dependency>
            <groupId>org.bouncycastle</groupId>
            <artifactId>bcprov-jdk15on</artifactId>
            <version>1.58</version>
        </dependency>
        <dependency>
            <groupId>com.squareup.retrofit2</groupId>
            <artifactId>retrofit</artifactId>
            <version>2.3.0</version>
        </dependency>
        <dependency>
            <groupId>com.squareup.retrofit2</groupId>
            <artifactId>converter-gson</artifactId>
            <version>2.3.0</version>
        </dependency>
        <dependency>
            <groupId>org.apache.commons</groupId>
            <artifactId>commons-lang3</artifactId>
            <version>3.6</version>
        </dependency>
        <dependency>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-api</artifactId>
            <version>1.7.25</version>
        </dependency>
        <dependency>
            <groupId>commons-io</groupId>
            <artifactId>commons-io</artifactId>
            <version>2.5</version>
        </dependency>

        <dependency>
            <groupId>junit</groupId>
            <artifactId>junit</artifactId>
            <version>4.12</version>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>net.jadler</groupId>
            <artifactId>jadler-all</artifactId>
            <version>1.3.0</version>
            <scope>test</scope>
        </dependency>
    </dependencies>

Hope it helps.

Thanks @ijaureguialzo!

Hey @ahab94  @ijaureguialzo
I have updated the maven instructions in the README.
Our default jar is exported with just our classes, and no dependencies.
If you wish to get the dependencies jar instead, add the following to your <dependency>
<classifier>jar-with-dependencies</classifier>
Example:
<dependency>
    <groupId>com.github.iotaledger.iota~lib~java</groupId>
    <artifactId>jota</artifactId>
    <classifier>jar-with-dependencies</classifier>
    <version>17e0de8ed5</version>
</dependency>

You can replace the version with any other hash (first 10 characters), or a release tag.
Classifier does not work for 0.9.9 and 0.9.10 because those entire builds are broken.. A new one is on its way!
Please let me know if you have any questions!


iox-#1393 Construct DESTINATION_* variables from CMAKE_INSTALL_FULL_*DIR

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-#123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit messages are signed (git commit -s)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] Assign PR to reviewer

Notes for Reviewer
Implements the solution described in #1393:

This can be fixed by constructing DESTINATION_CONFIGDIR from CMAKE_INSTALL_FULL_LIBDIR and then getting rid of the concatenation in Config.cmake.in.

Checklist for the PR Reviewer

[x] Commits are properly organized and messages are according to the guideline
[x] Code according to our coding style and naming conventions
[x] Unit tests have been written for new behavior

[x] Each unit test case has a unique UUID

[x] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[x] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #1393

@elfenpiff I'm fine with not changing the header.


How to fetch coordinate of dynamic button before its getting added to FlexboxLayout

I have created two flex layout inside a relative layout. One is at top and other one is at bottom.
I am adding dynamic button in the bottom flexlayout and set onclick listener for each button.  In this onclick, Starting a animation from bottom flexlayout button coordinates till the top flexlayout coordinates where the button is going to be added.  I am using a temporay variable called lastposition for tracing the position of the view to be added in top flexlayout. How will i get to know row is fill with content in top flexlayout since my last position logic screws up in case of row getting filled and animation goes off screen.
Please see github repo https://github.com/akshayISTAR/TfyLanguage
Activity name : FlexActivity
layout name : activity_flex

Hi, let me make sure if I understand your issue correctly.
Do you want to know in which row a View is going to be added to the FlexboxLayout before it's actually added to FlexboxLayout?
If so, short answer is no. You can't know it beforehand.
Please reopen the issue if you think there is still an issue.
Thanks.

yeah i want the position with respect to the screen before its actually get added


Plugin version is 1.0.2, but database version is 0.1

I am using gitbucket 4.21.2 and I got the following error message.
Error during plugin initialization: gitbucket-4.10-asciidoctor-plugin-assembly-1.0.2.jar
java.lang.IllegalStateException: Plugin version is 1.0.2, but database version is 0.1
Any solutions?

Did upgraded from a very early version of the plugin? Have you tried uninstalling the old version, before installing the new one?
Besides, asciidoctor-plugin stores nothing in the database and has no need for any database migration after an update. If the above hint does not help, you could try if you can delete any old version information of this plugin in your installation.

Thanks. I deleted the old version info from the DB and it works.


German translation missing

There is no german translation available.
A german locale file should be added as src/locale/de.js.

Released as v0.2.7


Improve PQL regex query performance

Describe This Problem
The following regex query is very slow.
sum (rate(http_requests_counter{url=~"/api/query|/plugin/prometheus/api/v1/query.*"}[3m]))

But the performance is very good after splitting the query.
sum (rate(http_requests_counter{url="/plugin/prometheus/api/v1/query"}[3m]))
sum (rate(http_requests_counter{url="/api/query"}[3m]))

Proposal
Improve PQL regex query performance.
Additional Context
No response

I think we can add a datafusion OptimizerRule to convert regex filter to equal filter, eg: a=~"1|2" could be converted to a in (1, 2)

https://crates.io/crates/regex-syntax
This crate allow us to program based on regex syntax.

https://github.com/apache/arrow-datafusion/pull/6487 partially solved the problem.

https://github.com/apache/arrow-datafusion/issues/7185 Now we can support string which include _.


Not able to connect to aks cluster using az ask get-credentials

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks get-credentials Extension Name: aks-preview. Version: 0.5.19.
Errors:
The command failed with an unexpected error. Here is the traceback:
'name'
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.25.0/libexec/lib/python3.8/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/Users/khushboosharma/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 1945, in aks_get_credentials
    _print_or_merge_credentials(
  File "/Users/khushboosharma/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 3513, in _print_or_merge_credentials
    merge_kubernetes_configurations(
  File "/Users/khushboosharma/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 3589, in merge_kubernetes_configurations
    _handle_merge(existing, addition, 'clusters', replace)
  File "/Users/khushboosharma/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 3532, in _handle_merge
    if i['name'] == j['name']:
KeyError: 'name'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks get-credentials --resource-group {} --name {}

Expected Behavior
Environment Summary
macOS-12.4-x86_64-i386-64bit
Python 3.8.10
Installer: HOMEBREW

azure-cli 2.25.0 *

Extensions:
aks-preview 0.5.19

Additional Context

I had to delete the ~/.kube/config file to fix this issue.
I'm not sure why the config got messed up though.

route to CXP team


rename/move instead of copy possible for None's?

[Enter feedback here]
Test resources of analysers can confuse tools (e.g., a test.vsproj is discovered as a project within the solution instead of just being a resource).
A solution is to use changed extensions, like vsproj_.
We would like to change these extensions back to have the test case as similar to the real situation as possible.
The documentation is not clear enough whether that is possible or not.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 97e663f7-3bac-61af-28bb-f0e471dcdca6
Version Independent ID: 97985f56-e0ac-a03d-f651-b86cc2b85804
Content: Common MSBuild Project Items - MSBuild
Content Source: docs/msbuild/common-msbuild-project-items.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@pjljvandelaar -- thank you for your feedback.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

@ghogen -- please look into this issue.

@pjljvandelaar
Have you tried implementing your idea and run into problems?  In general, documentation can't cover all the possible scenarios, but if you can narrow your inquiry down to some details that we could either confirm or disconfirm, I could possibly update this document.

@pjljvandelaar
I think anything is possible, since MSBuild is very flexible.
I would imagine you could use the Copy or Move task in which the SourceFiles are given as an item group (could be @None or a custom group you define via ItemDefinitionGroup just for this purpose) and DestinationFiles, where the item group for Destination Files would be constructed by an item transform done by modifying the %Extension metadata of the None collection. If I have time, I might work up an example to test this idea.
Copy task - https://learn.microsoft.com/en-us/visualstudio/msbuild/item-definitions?view=vs-2022
Move task - https://learn.microsoft.com/en-us/visualstudio/msbuild/move-task?view=vs-2022
ItemDefinitionGroup - https://learn.microsoft.com/en-us/visualstudio/msbuild/item-definitions?view=vs-2022
MSBuild items - https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-items?view=vs-2022
Item metadata - https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-well-known-item-metadata?view=vs-2022
Transform item list using metadata - https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-items?view=vs-2022#BKMK_Transforming

This code takes all the None items and copies them to the output folder, with an _ appended to the extension. Perhaps you can adapt it to your situation.
`
	<ItemGroup>
		<SourceFile Include="@(None)" />
		<DestinationFile Include="@(SourceFile->'$(OutDir)\%(Filename)%(Extension)_')" />
	</ItemGroup>

	<Copy SourceFiles="@(SourceFile)"
		  DestinationFiles="@(DestinationFile)"
		  SkipUnchangedFiles="true" />
</Target>

`

I think it would be preferable to create a custom ItemDefinitionGroup for this purpose rather than reusing the 'None' item type which would enable you to set this as a Build Action in Visual Studio only for the project files you need this behavior on.  The way to get a custom item type to appear in VS is by using AvailableItemName - https://learn.microsoft.com/en-us/visualstudio/msbuild/visual-studio-integration-msbuild?view=vs-2022#additional-build-actions

@ghogen
Indeed, anything is possible, since MSBuild is more flexible than I was aware.
Your list with relevant pointers and the examples have helped tremendously.
It is always a challenge to get the right information and pointers in a document without overwhelming the reader with too much details. For me, some of the provided information could/should have been integrated in the documentation.
Thanks for your time!


Restore behavior and resource packs after update

After an update, the behavior and resource packs have disappeared. Is there a way to restore it automatically?

They get backed up to a sub directory that starts with "backup-pre-". You'll need to manually restore the content.

Thank you for your reply. True, but I'm looking for an automated solution for custom extensions.
I'm thinking about a mapped volume which, for example, is checked when the server starts and existing extensions are copied into the respective directory on the server.
I would also be open to alternative suggestions. I am happy to make my changes available to everyone.

Similar to https://github.com/itzg/docker-minecraft-bedrock-server/issues/194
you can move your behavior_packs, resource_packs and *_packs.json into your words/[your world name]/
I have not had issues with any update moving my mods, this is how my volume looks for reference:
$ ls worlds/*/
behavior_packs  level.dat      levelname.txt   valid_known_packs.json
db              level.dat_old  resource_packs  world_behavior_packs.json

That would be a great solution. I'll try it. Many Thanks

From discussion in Discord it sounds like it’ll work fine to keep the behavior pack directories in place and let the json regenerate.

...and re-closing since I now remember the complexity is that the top-level behavior packs includes all the Mojang provided ones too.

…however I’m good with re-opening if someone wants to PR a solution to selective restore pack directories after an upgrade.


C417 auto-fixer breaks code with late-binding

Version: ruff 0.0.276 (no regression though)
  nums = range(4)

- callbacks = map(lambda x: lambda: x, nums)
+ callbacks = (lambda: x for x in nums)

Now list(callbacks)[0]() will change it's result:
- 0
+ 3

Additionally, a new error appears (pointing out the issue):

B023 Function definition does not bind loop variable x

My expectation is that the either the rule doesn't trigger in this case, or at least the auto-fixer doesn't break code.

Is this the "real" code that triggered the violation? It'd be helpful to have some more examples to inform the fix.

A real (but harder to grok) code example is this:
infos = map(
    lambda aspect: ColumnInfo(
        aspect.value,
        ma.fields.Enum(models.Resolution, required=True, by_value=True),
        AttributeImpl(
            lambda d, ctx: d.aspect_resolutions(aspect).effective(
                ctx.aspects.device_resolution_quota(aspect).fallback
            ),
            display_name="",
        ),
    ),
    models.Aspect,
)

which got transformed to
infos = (
    ColumnInfo(
        aspect.value,
        ma.fields.Enum(models.Resolution, required=True, by_value=True),
        AttributeImpl(
            lambda d, ctx: d.aspect_resolutions(aspect).effective(
                ctx.aspects.device_resolution_quota(aspect).fallback
            ),
            display_name="",
        ),
    )
    for aspect in models.Aspect
)

for which then the following problem was caught:

B023 Function definition does not bind loop variable aspect

The problem effectively happens if the iterator variable is used inside a lambda, so a some logic like the following would would be needed:
for lambda_node in find_lambda_nodes(map_node):
  if loop_variable_name in find_all_names(lambda_node):
    # loop variable is late-bound to a lambda, using comprehension would introduce bugs, bail
    return

Thanks, yeah, this makes sense.


Modern Visual Media Circle: Basic Form Sketch Workshop

Basic Bean Bag
What makes your eyes follow the drawing that you see as if it's moving? Dynamics. In this session, we will focus on drawing bean bags that harmoniously display their movement through the contours.
By Modern Visual Media Circle (MVMC)
Session intent:
First session:

Participants will learn the basic drawing of contours with the help of drawing bean bags
Participants will have a brief introduction to perspective and foreshortening.

Second session:

Participants will be able to explore their imagination and draw using the skills that they have just acquired

Materials:

Bring your prefered drawing tools (Pen/Pencils/Drawing pad/Tablet)
Imagination
Note: MVMC usually hogs the canteen to have dinner together after our sessions. Do join us for the dinner to have a light chit-chat session.

Session Details:
Max Participants: 15
Venue: A Think Tank
Timing: First session: 31st May (7.00pm - 9.00pm)
Second session: 7th May (7.00pm - 9.00pm)
Workshop facilitators:

Christabel Dorothy : Avid drawer, especially for chibi. Art Instagram: @wyxion
Siti Nurbaya: Casual drawer. Art Instagram: @maxabre_

Logistics:

Projectors which is expected to come with the Think Tank
Extra Papers and Pencils just to be prepared

Point of Contact:
Siti Nurbaya @ 9847 9353 or<EMAIL_ADDRESS>

https://discover.sutd.dev/events.html#form-sketch
Please check if the info is correct

Hi I have edited the timing :) and can delete the dinner part as we have changed the timing. Thank you so much!

@simplisiti thanks for the update!
@Fishbiscuit can we confirm the venue for this?


Make the @yamada-ui/portal component headless

script: make-component-headless
package: @yamada-ui/portal
Description
Currently, the @yamada-ui/portal component has minimal styles set within the component's source code.
We want to make this style transferable to the theme and make the component headless, enabling more flexible UI customization.
Problem Statement/Justification
By making the @yamada-ui/portal component headless, we can achieve more flexible styling tailored to the project.
Proposed Solution or API
The styles within the component are mostly defined by css: CSSUIObject and set to a prop named __css. We will transfer these styles to the packages/theme/src/components.
The code below is just an example and not the actual code.
packages/components/input/src/input-group.tsx
const css: CSSUIObject = {
- width: "100%",
- display: "flex",
- position: "relative",
  ...styles.container,
}

packages/theme/src/components/input.ts
baseStyle: {
  container: {
+   width: "100%",
+   display: "flex",
+   position: "relative",
  },
  field: {
    width: "100%",
    minWidth: 0,
    outline: 0,
    position: "relative",
    appearance: "none",
    transitionProperty: "common",
    transitionDuration: "normal",
    _disabled: {
      opacity: 0.4,
      cursor: "not-allowed",
    },
  },
  element: {
    color: ["blackAlpha.600", "whiteAlpha.700"],
  },
},

[!TIP]
In cases where the styles within the component are dynamically changed, there may be instances where it is not possible to transfer the styles to the theme.
In such cases, it needs to be discussed in this issue. Please feel free to send comments to this issue.

Additional Information
The @yamada-ui/portal component exists in packages/components/portal/src. You need to check all the components in this folder and if they contain styles internally, you need to modify them.
This change could have a significant impact on projects using Yamada UI, so we will work on it within the scope of v2.0.
Please create the branch for working on this issue from the remote branch v2.0, and direct your pull requests to the v2.0 branch as well.
Please select major for the changeset and create a pull request.

@hirotomoyamada
Can I work this?

@Kei-Ta
Sorry. No styles existed for this package. Therefore, this issue will be closed.


[Enhancement] Add location message for show create external table

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/7286
Problem Summary(Required) ：

Add location message for show create external table
| test_iceberg | CREATE TABLE ice_catalog01.test.test_iceberg (
name VARCHAR(65533),
id BIGINT
)
LOCATION 'hdfs://hadoop/test/hive/warehouse/test.db/test_iceberg'
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:disappointed: fail : 3 / 5 (60.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/Table.java
0
1
00.00%
[254]

:large_blue_circle:
com/starrocks/catalog/IcebergTable.java
0
1
00.00%
[176]

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 5 / 11 (45.45%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/Table.java
0
1
00.00%
[220]

:large_blue_circle:
com/starrocks/catalog/IcebergTable.java
0
1
00.00%
[176]

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
5
9
55.56%
[732, 733, 734, 735]

[FE PR Coverage Check]
:disappointed: fail : 7 / 13 (53.85%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/Table.java
0
1
00.00%
[220]

:large_blue_circle:
com/starrocks/catalog/IcebergTable.java
0
1
00.00%
[176]

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
7
11
63.64%
[732, 733, 734, 735]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

LGTM

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.5

@Mergifyio pls create new backport, because 15192 has conflicts, when 15193 merged, new backport will not hava confilcts