Error with deltalake 0.15.0 linux-64 python3.9 package

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
I am trying to install the recently released deltalake 0.15.0 via conda-forge for linux-64 on python 3.9. I am able to install it, but it seems as if the package is empty. For example, from deltalake import DeltaTable is broken, and if I look at the package directory, I only see _internal.abi3.so instead any of the python files.
Installed packages
deltalake

Environment info
n/a

@dhirschfeld or @charlesbluca building this on my development machine with conda build provided a workable package for Python 3.9.  At first glance I'm unsure what the issue is with the Python 3.9 release pipeline.

Thanks @echai58 for reporting and @digi691 for trying things out locally - might I ask what conda build command you tried locally? Might also be worth giving this a spin with the local build script to see if that gives any different results.
My naive guess here is that the format of the source distribution has changed such that we are no longer automatically including the python modules we require - checking the sdist as of 0.15.0, it does look like 2 copies of the modules are being published in both deltalake/ and python/deltalake/, which wasn't being done before this.
As I'm not a maintainer of the main project, might be worth pinging @ion-elgreco (who cut the last two PyPI releases) to see if this change was intentional?

Python/deltalake is the python code for the library, python/src contains the rust lib with the bindings.
Deltalake I presume you are looking at crates/deltalake that's the rust crate.
So I am not sure where it's going wrong but we didn't make a change in the python binding in terms of the structure

Deltalake I presume you are looking at crates/deltalake that's the rust crate.

Nope, there is a top level deltalake directory in the latest source distribution that contains another copy of the python modules:

Are these meant to be consumed in some separate way from the python/deltalake code? Doesn't seem like these duplicate files were present in 0.14.0 (the last working conda-forge release):

This one I need to investigate

@charlesbluca I naively executed the below in the base of the project with a Python 3.9 environment activated.
conda mambabuild recipe/
The build produced the following files from the following artifact deltalake-0.15.1-py39h9bf148f_0.tar.bz2 which works in an python 3.9 environment.
.
├── deltalake-0.15.1-py39h9bf148f_0.tar.bz2
├── info
│   ├── about.json
│   ├── files
│   ├── git
│   ├── hash_input.json
│   ├── index.json
│   ├── licenses
│   │   └── licenses
│   │       └── deltalake_license.txt
│   ├── paths.json
│   ├── recipe
│   │   ├── conda_build_config.yaml
│   │   ├── meta.yaml
│   │   └── meta.yaml.template
│   └── test
│       ├── run_test.py
│       ├── run_test.sh
│       └── test_time_dependencies.json
└── lib
    └── python3.9
        └── site-packages
            ├── deltalake
            │   ├── data_catalog.py
            │   ├── exceptions.py
            │   ├── fs.py
            │   ├── __init__.py
            │   ├── _internal.abi3.so
            │   ├── _internal.pyi
            │   ├── __pycache__
            │   │   ├── data_catalog.cpython-312.pyc
            │   │   ├── data_catalog.cpython-39.pyc
            │   │   ├── exceptions.cpython-312.pyc
            │   │   ├── exceptions.cpython-39.pyc
            │   │   ├── fs.cpython-312.pyc
            │   │   ├── fs.cpython-39.pyc
            │   │   ├── __init__.cpython-312.pyc
            │   │   ├── __init__.cpython-39.pyc
            │   │   ├── schema.cpython-312.pyc
            │   │   ├── schema.cpython-39.pyc
            │   │   ├── table.cpython-312.pyc
            │   │   ├── table.cpython-39.pyc
            │   │   ├── _util.cpython-312.pyc
            │   │   ├── _util.cpython-39.pyc
            │   │   ├── writer.cpython-312.pyc
            │   │   └── writer.cpython-39.pyc
            │   │   └── writer.cpython-39.pyc
            │   ├── py.typed
            │   ├── schema.py
            │   ├── table.py
            │   ├── _util.py
            │   └── writer.py
            └── deltalake-0.15.1.dist-info
                ├── direct_url.json
                ├── INSTALLER
                ├── license_files
                │   └── deltalake_license.txt
                ├── METADATA
                ├── RECORD
                ├── REQUESTED
                └── WHEEL

I then ran the following script python build-locally.py linux_64_python3.9.____cpython which produced deltalake-0.15.1-py39hccdef0e_0.conda with the following contents matching what we see from the downloaded package from the conda-forge channel:
╰─ tree
.
├── info
│   ├── about.json
│   ├── files
│   ├── git
│   ├── hash_input.json
│   ├── index.json
│   ├── licenses
│   │   └── licenses
│   │       └── deltalake_license.txt
│   ├── paths.json
│   ├── recipe
│   │   ├── conda_build_config.yaml
│   │   ├── meta.yaml
│   │   ├── meta.yaml.template
│   │   └── recipe-scripts-license.txt
│   └── test
│       ├── run_test.py
│       ├── run_test.sh
│       └── test_time_dependencies.json
└── lib
    └── python3.9
        └── site-packages
            ├── deltalake
            │   └── _internal.abi3.so
            └── deltalake-0.15.1.dist-info
                ├── direct_url.json
                ├── INSTALLER
                ├── license_files
                │   └── deltalake_license.txt
                ├── METADATA
                ├── RECORD
                ├── REQUESTED
                └── WHEEL

How does that build environment compare to the one used on CI?

Also @echai58 could you please fill out the full issue template with a minimal reproducer?
This would really go a long way towards helping us resolve the issue

Was able to reproduce with the 3.12 linux-64 packages:
conda create -n test-deltalake-0150 -c conda-forge deltalake=0.15.0
conda activate test-deltalake-0150
python -c "from deltalake import DeltaTable"

I suspect this issue is present in all the linux-64 packages for >=0.15.0, and probably the other platforms as well (though I haven't had the chance to check, not sure how to inspect .conda packages outside of installing).

The environment I used for the working package is my default miniconda installation (base) with conda-build installed on my x86-64 Linux development machine.  Nothing special... The incomplete package from briefly reading the installation script is using a conda-forge docker image and their CI scripts.  I guess all that I've showed is that the recipe file may be correct, and something is missing for the conda-forge CI to pick up and package the artifacts appropriately in that CI container.

Was able to reproduce with the 3.12 linux-64 packages:
conda create -n test-deltalake-0150 -c conda-forge deltalake=0.15.0
conda activate test-deltalake-0150
python -c "from deltalake import DeltaTable"

I suspect this issue is present in all the linux-64 packages for >=0.15.0, and probably the other platforms as well (though I haven't had the chance to check, not sure how to inspect .conda packages outside of installing). Here's the environment info:
conda list:
conda info:

I believe cph will extract .conda files.  https://anaconda.org/anaconda/conda-package-handling

How does that build environment compare to the one used on CI?

The environment I used for the working package is my default miniconda installation (base) with conda-build installed on my x86-64 Linux development machine. Nothing special... The incomplete package from briefly reading the installation script is using a conda-forge docker image and their CI scripts. I guess all that I've showed is that the recipe file may be correct, and something is missing for the conda-forge CI to pick up and package the artifacts appropriately in that CI container.

Could you please create a diff between the two environments (yours that works and CI)? We would like to identify any differences

Hi folks,
the 0.15.1 package at conda-forge is still broken (for me, i.e. Linux, deltalake-0.15.1-py312hd892df0_0.conda).
'polars' suffered from apparently the same problem.
Please look at
https://github.com/conda-forge/polars-feedstock/issues/205
and
https://github.com/conda-forge/conda-smithy/issues/1827

Thanks for the context @elveshoern32! Looks like this issue should be resolved with #20 if anyone else would like to verify locally.
Beyond that, might be worth keeping our eyes on https://github.com/PyO3/maturin/issues/1911 to see if any changes are made to maturin to reflect this.

@charlesbluca 0.15.1 is working for me now :tada: i think 0.15.0 will still be broken, (don't see that updated today here: https://anaconda.org/conda-forge/deltalake/files), if that's a concern.

Thanks @echai58 :tada: the broken 0.15.0 packages aren't a problem for my use cases, unless there's a specific need to backport the fix I'm satisfied with marking those packages as broken

Thanks @echai58 🎉 the broken 0.15.0 packages aren't a problem for my use cases, unless there's a specific need to backport the fix I'm satisfied with marking those packages as broken

@charlesbluca same for me, thanks for helping with this!

Thanks @echai58 🎉 the broken 0.15.0 packages aren't a problem for my use cases, unless there's a specific need to backport the fix I'm satisfied with marking those packages as broken

There is actually an issue with 0.15.1 atm, during some refactors, and pushing a release to soon the GCP support got dropped. So could be useful to still get 0.15.0 in the conda feed.

Resolved by #20


bug/FPHS-729

Allow withdraw survey user entered reason to be de-selected if flag is set to YES.
Defaulted behavior to YES so that user can deselect it, which fixes a bug that once it was selected you could never delete the text in the pop-up or uncheck it.
The two FPHS story references are:
https://sagebionetworks.jira.com/browse/FPHS-739
https://sagebionetworks.jira.com/browse/FPHS-729

@dephillipsmichael I think it's reasonable to allow de-selection. The flag is really just confusing.

Sounds good will make the change

@syoung-smallwisdom fixed by removing the flag

looks good


when html snapshotting reddit it gets blocked. is there a way to just automatically have a singlefile thing that gets saved automatically get bookmarked instead?

As the title says is there a way to have something I singlefile save automatically get taken up my linkding instead? I remember seeing a url validation thing so I'm guessing I have to enable that so I can make it save localfiles as a url but does anyone have any advice to make this as simple as possible instead of having to singlefile the page wait for it to download and then manually drag and dropping into linkding? Thank you!

There is no integration with the singlefile extension that I'm aware of. There's an existing issue to make snapshots work with sites that use bot detection so I'm closing in favor of that: https://github.com/sissbruecker/linkding/issues/712
For Reddit specifically configuring a user agent for singlefile might be enough to make it work: https://github.com/sissbruecker/linkding/issues/690.


Sort the output of ProjectHealthConsoleReportBuilder on the printed suggestion

The Advice#compareTo is not the natural sorting that I was aiming for, this fixes it by doing it the old way of sorting the (to) print advice.

Thanks for keeping your branch up to date @mvegter. Are you interested in writing a unit test to validate the change?


Kube object protection and recovery status manager - sprint 5

[ ] Consider defining new kubeObjectsProtected and kubeObjectsRecovered status conditions to:
allow concurrent protection/recovery of PV and non-PV Kube objects (optimization post-MVP)
allow it to have a different state than clusterDataProtected and clusterDataReady conditions

consider scenarios to determine if necessary

[ ] add additional conditions for existing clusterDataReady and clusterDataProtected status conditions

Decided to re-use existing clusterDataReady and clusterDataProtected conditions and dividied them into sub-conditions


fix: stop leaking processors with multiple invoke

This commit fixes a bug where a processor would be invoked succesively
more and more times because the list of processors associated with a
given filter was accidentally mutated during the invocation.

Codecov Report

Merging #72 (d033873) into main (4064b46) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #72   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          425       426    +1
=========================================
+ Hits           425       426    +1

Impacted Files
Coverage Δ

src/event_processor/event_processor.py
100.00% <100.00%> (ø)

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 4064b46...d033873. Read the comment docs.


Bad accept invitation route

The automatically generated route for accept_user_invitation_url is not working for me.  My setup is pretty vanilla and all of my other routes and devise setup are working just fine.  However, the accept_user_invitation_url has an extraneous "." in it that is clearly breaking the route.
Example accept_user_invitation_url:
http://default.lvh.me:5000/users/invitation/accept.?invitation_token=DvTzf2eKyLtCDLojVanM
From routes.rb:
devise_for :users, :controllers => { :sessions => "admin/sessions", :registrations => "admin/registrations", :confirmations => "admin/confirmations", :invitations => "invitations" }
From rake routes:
accept_user_invitation GET    /users/invitation/accept(.:format)                               invitations#edit
Anyone else seeing this same behavior?

I have the same problem: http://localhost:3000/api/auth/invitation/accept.2?invitation_token=dr8xXsXdUa2FFjxUe6e8
It is adding the id (2) in the url.


Submit custom_fg_color instead of color

The conditional formats section of screenboards should submit a field called custom_fg_color instead of color. Docs - https://docs.datadoghq.com/graphing/graphing_json/widget_json/#conditional-format-schema
Should still be backwards compatible since I left the struct alone and only changed what its un-marshalled as.

LGTM, merging. Thanks for the PR!


cephfs: remove extra check for restore size

Looks like the cephfs snapshot size is buggy and it's getting removed in ceph fs. we cannot get the size of the snapshot during the CreateVolume call, so we cannot do any size check at CreateVolume to check if the restore size is smaller or not.
As we are removing this check it also fixes #3147 but we dont have any validation at the CSI level for smaller restore we need to depend on kubernetes external-provisioner for it.
fixes: #3147
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

Adding DNM and converting it to draft as the discussion is going on for snapshot-size

On Mon, Jun 13, 2022 at 11:51:55PM -0700, Madhu Rajanna wrote:

Looks like the cephfs snapshot size is buggy and it's getting
removed in ceph fs. we cannot get the size of the snapshot during the
CreateVolume call, so we cannot do any size check at CreateVolume to
check if the restore size is smaller or not.

If this is buggy, add a reference to the Ceph issue tracker please.

/retest ci/centos/mini-e2e-helm/k8s-1.23

@mergifyio rebase

@mergifyio rebase

Resolved merge conflicts and rebased

@mergifyio rebase

@humblec @nixpanic PTAL

@mergifyio rebase


Not working, I did "bash install.sh", but still no submenus available at all.

I'm using Zorin OS and Nemo file manager. It doesnt work in Nemo file manager whatsoever, the "scripts" option doesnt even come up in the submenu. But it does work in Nautilus, but wont work in Nautilus with avif, heic or jxl image files.
Please help.
Thanks.
Also, very important, see how Lich-Corals accomplishes this in Nemo, his only does certain types of conversion:
https://github.com/Lich-Corals/linux-file-converter-addon/issues/21
https://github.com/Lich-Corals/linux-file-converter-addon/tree/dev

Wait... I've found another bug for when you have multiple file managers installed. I'm working on this now... 😅

Hello, @Username23453 ! I think the bug is solved now.
I did the tests: Using Zorin OS 7.1 with Nautilus and Nemo installed. Also, I did conversions from  avif and heic to png.
Could you please retest?
See the screenshot (Nemo):

See the screenshot (Nautilus):

@Username23453 thanks for your feedback.
Could you please attach the sample files, the images heic and  jxl that you are using, for me to make some tests here?
After seeing the log, it seems to me that this is a bug (or a limitation) in the convert command (from ImageMagick).


newbedev.com

https://newbedev.com/customize-menu-bar-in-notepad
https://superuser.com/questions/1176224/customize-menu-bar-in-notepad
title, text and tags the same, images imbedded from i.stack.imgur.com

https://github.com/vladgba/Back2source/blob/26090da0d7bda5251ff29aac803c5a4f4a91c03d/back2source.user.js#L417-L421
Something like this?

That's great! Yes, something like that.


PATH variable not working at all in Terminal

Windows Terminal version
1.21.2911.0
Windows build number
10.0.22631.4460
Other Software
The Windows Terminal doesn't seem to be reading the PATH variable. You see, when I add the System32 to the PATH variable it should be working. I added the file path to both User and  System variables. And ive restarted terminal. Any help is appreciated, thank you
Steps to reproduce
When I type the command "ipconfig" it says it is not recognized as an internal or external command, operable program or batch file.
Expected Behavior
For the command "ipconfig" to work of course
Actual Behavior
"ipconfig" is not recognized as an internal or external command,
operable program or batch file.

You're also in system32 as your CWD in that screenshot, so CMD shouldn't even need the PATH to find it.
Here's a possibly silly question - is ipconfig somehow missing from system32/?
dir c:\windows\system32 | findstr ipconfig

should output something like 02/13/2024  01:29 PM            53,248 ipconfig.exe

'findstr' is not recognized as an internal or external command,
operable program or batch file.

Idk man, feel like i would either need to reinstall windows or get it fixed

Ok I kinda fixed it by pasting the full path of fndstr

Uh okay what
what does this dialog look like?

What does echo %PATH% emit?
If you hit win+R, then use the run dialog to launch conhost.exe, then run echo %PATH%, what does that output/?

And here is what CMD echo %PATH% says: C:\Windows\System32;C:\Program Files\dotnet;C:\HaxeToolkit;C:\Program Files\dotnet\sdk;C:\Windows\System32;C:\WINDOWS\System32;C:\Users\tahir\AppData\Local\Programs\Microsoft VS Code\bin;C:\Users\tahir\AppData\Local\Programs\oh-my-posh\bin;C:\HaxeToolkit\haxe;C:\HaxeToolkit\neko;C:\HaxeToolkit\haxe\haxelib.exe;C:\Users\tahir.dotnet\tools;C:\Program Files\mingw64\x86_64-w64-mingw32;C:\HaxeToolkit;C:\Program Files\Git\cmd;C:\Program Files\Git\bin;C:\mingw-64\x86_64-14.2.0-release-posix-seh-ucrt-rt_v12-rev0\mingw64\bin;C:\Program Files\dotnet;C:\Program Files\dotnet\sdk;C:\Windows\System32;C:\Program Files\dotnet;C:\HaxeToolkit;C:\Program Files\dotnet\sdk;C:\Windows\System32;C:\WINDOWS\System32;C:\Users\tahir\AppData\Local\Programs\Microsoft VS Code\bin;C:\Users\tahir\AppData\Local\Programs\oh-my-posh\bin;C:\HaxeToolkit\haxe;C:\HaxeToolkit\neko;C:\HaxeToolkit\haxe\haxelib.exe;C:\Users\tahir.dotnet\tools;C:\Program Files\mingw64\x86_64-w64-mingw32;C:\HaxeToolkit;C:\Program Files\Git\cmd;C:\Program Files\Git\bin;C:\mingw-64\x86_64-14.2.0-release-posix-seh-ucrt-rt_v12-rev0\mingw64\bin;C:\Program Files\dotnet;C:\Program Files\dotnet\sdk;C:\mingw-64\x86_64-14.2.0-release-posix-seh-ucrt-rt_v12-rev0\mingw64\bin;

Same, but yeah it's the same in Terminal.

Looks a lot like case sensitivity issues. https://learn.microsoft.com/en-us/windows/wsl/case-sensitivity
test1:
fsutil.exe file queryCaseSensitiveInfo c:\
fsutil.exe file queryCaseSensitiveInfo c:\windows
fsutil.exe file queryCaseSensitiveInfo c:\windows\system32

Case sensitive attribute on directory C:\ is disabled.
Case sensitive attribute on directory C:\Windows is disabled.
Case sensitive attribute on directory C:\Windows\System32 is disabled.

Is this good or not??

Is this good or not??

This is okay.
Maybe some registry key is involved.
https://devblogs.microsoft.com/commandline/per-directory-case-sensitivity-and-wsl/
HKLM\SYSTEM\CurrentControlSet\Services\lxss\DrvFsAllowForceCaseSensitivity

https://superuser.com/questions/266110/how-do-you-make-windows-7-fully-case-sensitive-with-respect-to-the-filesystem
HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\kernel\obcaseinsensitive

I think you need to dig in that direction.

Also check the following
cd \
cd C:\Windows\System32
ipconfig

This will work if it is a case sensitivity issue.

Also I've changed casesensitiveinfo to 0 in regedit yet still the same output: ipconfig is not recognized as an internal blah blah blah.

What does the following command output?
cd \
cd C:\Windows\System32
ipconfig

Exactly the same

cmd.exe does not distinguish between the case of drive letters and puts the one ordered, and in this situation only capital letters are allowed. It seems that you should change the drive letters to capital letters everywhere in the terminal configs.

So this looks like an issue on the cmd.exe side. They don't check for case when they need to.

How should I change the CMD's config files?

All occurrences of "commandline" option in settings.json must contain uppercase drive letters in their values.

What would it be in this case?

Replace %Systemroot% with C:\Windows.

Sorry, with C:\\Windows.

Woops figured it out. Do I have to restart my Laptop for it to work? Cause it doesn't right now.

No. It should work instantly. (Just create a new tab with cmd profile)

Could you show the output of the set command?

I'm trying to find a way to make my system case-sensitive to reproduce your experience, but so far no success. You have a lowercase drive letter C: coming from somewhere, it’s not clear where it comes from.

Please check the starting directory option

Ugh it still doesn't recognize ipconfig

Nope. Maybe I should dig around in regedit?

I don't know what else to look at or do. I'm also interested in how to get a system configuration like yours with case-sensitivity in the C drive partition.

Should I look somewhere for the system config then? Maybe if I send it you'll look at it, 'cause idk what 2 do anymore.

Maybe it's some side effect of some WSL v1 or v2 setting. Maybe something related to DrvFs or Plan9(P9) protocol setting. As soon as I find something to cling to, I'll write here. Maybe someone else will suggest something.

Thanks, I'll keep this issue open just in case.

Are you using WSL? If so, what does C:\Windows\System32\wsl.exe mount output? (or C:\Users\tahir\AppData\Local\Microsoft\WindowsApps\wsl.exe mount)

It says I need to install thorugh this command, but uhh, yeah.

Try C:\Windows\System32\wsl.exe --install.

If you haven’t installed WSL, don’t. If you haven’t installed it, it isn’t the cause of your issues. It is also not going to help you to diagnose them.
If you launch the original windows console host directly, by running conhost.exe with the Run dialog, does the problem persist?

@Mcknight231 Could you run the following?
subst c: C:\

@o-sdn-o same on my end

@DHowett Yes, it still persists.

I don't know what to grab onto, but what does the chcp command output?
C:\Windows\System32\chcp.com

Active code page: 850

Note: The subst.exe command internally aligns the case of all passed parameters, and this is probably why it didn't work.

So what's the solution?

Perhaps we need to find (or make) an analogue of the subst command that will accept parameters without changing their case.

Okay, I'll wait for some kind of solution then, thanks!

I'll try to make a powershell script that will do this.

Try to run the following powershell script.

subst_cs.ps1:$win32 = Add-Type -Name WinAPI -PassThru -MemberDefinition @'
[DllImport("kernel32.dll")] public static extern bool DefineDosDevice(int dwFlags, string lpDeviceName, string lpTargetPath);
'@
$rc = $win32::DefineDosDevice(0, "c:", "C:\")
C:\Windows\System32\cmd.exe /k "dir c:"
$rc

Most likely it won't work. Win32 API function DefineDosDevice converts the disk name to uppercase...

What does the command C:\Windows\System32\mountvol.exe output?

I have a theoretical suspicion that in your system in the global names section there may be a symlink with a name in the form of a lowercase c: along with an uppercase C:. This can be inspected using WinObj. There you can sort by name by clicking on the header of the corresponding column.

I found another clue. It may be related to locale. cmd.exe uses int ::towupper(wint_t c) function to change the case of drive letter, and the result depends on locale.
https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/toupper-toupper-towupper-toupper-l-towupper-l?view=msvc-170

The case conversion of towupper is locale-specific. Only the characters relevant to the current locale are changed in case.

It is possible that under certain conditions the string c:\Windows\System32\... is not converted to the required C:\Windows\System32\...

To be honest, CMD has about 500 million users. We would have heard about this.
@Mcknight231 would you be willing to install some debugging tools to send us some information? I can try to write them up over the weekend, but the latest I can do it is Monday.
In short, it would be to install Microsoft’s modern Windows Debugger (WinDbg) and recording a Time Travel Trace of CMD.
Install instructions: https://learn.microsoft.com/en-us/windows-hardware/drivers/debuggercmds/
If you follow Section 2 of this guide (https://learn.microsoft.com/en-us/windows-hardware/drivers/debuggercmds/time-travel-debugging-walkthrough#section-2-record-a-trace-of-the-displaygreeting-sample), but use…
Executable: C:\windows\system32\cmd.exe
Arguments: (nothing)
Starting Directory: C:\

And then reproduce the bug in the new CMD window after you follow the steps, it will make a huge file called cmd01.run that you can share with us.
Step 7 says to make the app crash, but you can just reproduce your bug and then exit!

This will allow a Microsoft engineer, or o-sdn-o, to step through the execution of CMD while it is looking for commands in your PATH.

Last question, to satisfy my curiosity please, what does the command output?
C:\Windows\System32\systeminfo.exe | C:\Windows\System32\findstr.exe Locale

c:\Windows\System32>C:\Windows\System32\systeminfo.exe | C:\Windows\System32\findstr.exe Locale
System Locale:             en-gb;English (United Kingdom)
Input Locale:              en-gb;English (United Kingdom)

I find another interesting clue. Could you run the following:
set PATHEXT=
find

In my setup I see the following:c:\Windows\System32>set PATHEXT="qqq"

c:\Windows\System32>find
'find' is not recognized as an internal or external command,
operable program or batch file.

c:\Windows\System32>set PATHEXT=

c:\Windows\System32>find
FIND: Parameter format not correct

c:\Windows\System32>

c:\Windows\System32>set PATHEXT="qqq"
c:\Windows\System32>find
'find' is not recognized as an internal or external command,
operable program or batch file.
c:\Windows\System32>set PATHTEXT =
c:\Windows\System32>find
'find' is not recognized as an internal or external command,
operable program or batch file.

No! Please run exactlyset PATHEXT= (without space before equal)

@DHowett i did it aand here it is:https://www.mediafire.com/file/4kxtvhvjec2cn3h/cmd01.run/file

@o-sdn-o

It seems a cure has been found. Having a PATHEXT variable with an incorrect value was apparently the cause of the problems. Just remove it if it is set somewhere.

Where the common, idk, locations for the PATHTEXT?

Or how can I change the PATHTEXT to be correct>

PATHEXT (not PATHTEXT) search and delete (configure) here

So should I modify it our just completely delete it?

OH MY GOD IT WORKED? But how can a variable mess up the commands for CMD? Tysm man!

Default value on Win10
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC

Huh, okay thanks! Im gonna close this issue now.

Wow I've never heard of PATHEXT getting messed up like that before.  Hopefully it should be helpful to future readers too


In TransitionMotion allow number proptype for "key"

The update makes it so that a key can be set as a number on styles and defaultStyles like so:
<TransitionMotion     defaultStyles={[{key: 1, style: {x: 0}}]}     styles={() => [{key: 1, style: {x: spring(10)}}]}> </TransitionMotion>

There's actually a reason why keys are string only; see https://github.com/chenglou/react-motion/commit/c44037dea6aebd87a188eea98499a031e2d40093. I'm hesitating to allow number keys here so I'll keep the PR around. Might reconsider in the future.

We can also add some specific warnings if we only allow string for keys.
I've encountered the same issue here and spent some time to figure out this.
I think this is a good PR.

Alternatively why not allow both string and integer props and internally cast the key to a string?


Move scholarsphere.yml to application.yml, ref #554

@awead I just went ahead and ticket the issues that I had with this PR.  That way we will not loose the fact that things are a bit off, but we can also get the code changes into the system.


Extend ITF format with variable types

At present, only the variable names and values are recorded. This can be a problem in reconstruction via the typed builder in the presence of empty sets/sequences, because empty collections need type tag annotations in the builder.
Concretely, we should either

Change the "vars" field to an object, the keys of which are the variables, and the values of which are string representations of TT1 types, or
Add a new field "varTypes" (or similar), with that same info, preserving "vars" as a list, as not to break compatibility.

I think (1) is a nicer option.
I think this also opens up an opportunity to make the trace much more concise and readable!
At the time ITF was designed I argued against the current data representation, because it's treatment of scalars and compound values is so different: https://github.com/informalsystems/apalache/blob/main/docs/src/adr/015adr-trace.md#discussion
At that time, Igore argued that

The more regular approach is less concise. In the future, we might want to add a flag that lets the user choose between the regular output and the output proposed in this ADR, which is more ad hoc.

However, if we put the variable types in this header data, then we can get rid of all the #map and #set tags, cause all that data would be readily available in the variable's types. WDYT?

RE compound types, that's more of an orthogonal discussion, but the # values did strike me as odd at the time, I wouldn't mind changing them in the future.

This will be much easier to communicate and gather feedback on with an example.
Here's a fragment of the current ITF format, as per ADR015:
{
  "#meta": {
    "source": "MC_MissionariesAndCannibalsTyped.tla"
  },
  "vars": [ "bank_of_boat", "who_is_on_bank" ],
  "states": [
    {
      "#meta": { "index": 0 },
      "bank_of_boat": "E",
      "who_is_on_bank": {
        "#map": [
          [ "E", { "#set": [ "c1_OF_PERSON", "c2_OF_PERSON",
                             "m1_OF_PERSON", "m2_OF_PERSON" ] } ],
          [ "W", { "#set": [] } ]
        ]
      }
    },
    {
      "#meta": { "index": 1 },
      "bank_of_boat": "W",
      "who_is_on_bank": {
        "#map": [
          [ "E", { "#set": [ "c1_OF_PERSON", "m1_OF_PERSON" ] } ],
          [ "W", { "#set": [ "c2_OF_PERSON", "m2_OF_PERSON" ] } ]
        ]
      }
    }
   ]
}

Note the way that maps and sets are marked with the # prefixed tags. That empty set in the W bank of the first state is the kind of problem @Kukovec is pointing out.
The alternative representation using variable-typing info:
{
  "#meta": {
    "source": "MC_MissionariesAndCannibalsTyped.tla"
  },
  "vars": { "bank_of_boat": "Str", "who_is_on_bank": "{ Str : Set(PERSON) }" },
  "states": [
    {
      "#meta": { "index": 0 },
      "bank_of_boat": "E",
      "who_is_on_bank": {
        "E": ["c1_OF_PERSON", "c2_OF_PERSON", "m1_OF_PERSON", "m2_OF_PERSON"],
        "W": []
      }
    },
    {
      "#meta": { "index": 1 },
      "bank_of_boat": "W",
      "who_is_on_bank": {
        "E": ["c1_OF_PERSON", "m1_OF_PERSON"],
        "W": ["c2_OF_PERSON", "m2_OF_PERSON"]
      }
    }
  ]
}

IMO, this is a huge win for readability, for humans and for computers! Having to inspect each data type to see whether it has a special tag or not, then inferring it's type based on this always sounded like a big PITA to me.

that's more of an orthogonal discussion

I think it is parallel, and maybe even coincident, rather than orthogonal :D

ITF isn't really mature enough where we'd need to care about compatibility IMO.

I think this is true as well, also if there are large sets of traces being used somewhere, a migration script should be fairly trivial.
(3) A third option of course is just to infer the type of empty collections from non-empty ones in the same trace. If that cannot be done, we could either throw an error or fall back to some other inference mechanism.
If we pursue (1) or (2) it really makes the odd # tags redundant, so I think we should seriously consider coupling these changes. Still, we can introduce them as separate changes, giving us time to get Igor's feedback and perhaps feedback from MBT.

Yeah, I agree (3) seems unappealing. Even more so to me given that (1) and (2) solve other problems whereas (3) introduces new problems :D

The types given in the variable should JSON serialized, not using our human notation, otherwise all code consuming ITF would need to be able to parse our type specification syntax.

Should the types given in the variable should JSON serialized, not using our human notation? Otherwise all code consuming ITF would need to be able to parse our type specification syntax.

Are you asking if we want a JSON tree-like structure for compound TT1 types, e.g. Str -> Int (= FunT1(StrT1,IntT1) written as
"type": {
  "kind": "FunT1"
  "args": {
    "from": "StrT1"
    "to": "IntT1"
  }
}

or something similar? Or is the question just whether the type written in the ITF file is Str -> Int or FunT1(StrT1,IntT1)?

Yeah, I'm thinking we should use the same encoding for types as is generated by our json serialization of IR.
Whether we intend to only use these in Apalache or whether we depend to direct users to refer to these to interpret the values in variables, it seems like it's most sensible if we use a serialized representation of the types rather than requiring another parsing pass.

I've spun up a new issue for the fields. This one will be closed with just the "vars" change, but we can discuss the rest under the new issue.

After a discussion we agree to go forward with the following:

Types in ITF traces are optional in general, so the variables field should remain unchanged
Types in ITF traces for Apalache use are mandatory and must be present in #meta.


ARTEMIS-5202 require Java 17+ (i.e drop support for Java 11)

This commit drops support for Java 11. All code changes were related to output from ErrorProne that prevented the build from completing.

I tested the examples with my branch and they all work except for the CDI example. It fails with:
$ mvn verify
[INFO] Scanning for projects...
[INFO]
[INFO] --------------< org.apache.activemq.examples.broker:cdi >---------------
[INFO] Building ActiveMQ Artemis CDI Example 2.39.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-enforcer-plugin:3.5.0:enforce (enforce-maven-version) @ cdi ---
[INFO] Rule 0: org.apache.maven.enforcer.rules.version.RequireMavenVersion passed
[INFO]
[INFO] --- maven-enforcer-plugin:3.5.0:enforce (enforce-java-version) @ cdi ---
[INFO] Rule 0: org.apache.maven.enforcer.rules.version.RequireJavaVersion passed
[INFO]
[INFO] --- maven-remote-resources-plugin:3.2.0:process (process-resource-bundles) @ cdi ---
[INFO] Preparing remote bundle org.apache.apache.resources:apache-jar-resource-bundle:1.5
[INFO] Copying 3 resources from 1 bundle.
[INFO]
[INFO] --- maven-resources-plugin:3.3.1:resources (default-resources) @ cdi ---
[INFO] Copying 2 resources from src/main/resources to target/classes
[INFO] Copying 3 resources from target/maven-shared-archive-resources to target/classes
[INFO]
[INFO] --- maven-compiler-plugin:3.13.0:compile (default-compile) @ cdi ---
[INFO] Nothing to compile - all classes are up to date.
[INFO]
[INFO] --- maven-resources-plugin:3.3.1:testResources (default-testResources) @ cdi ---
[INFO] skip non existing resourceDirectory /home/jbertram/jboss/src/activemq-artemis-examples/examples/features/standard/cdi/src/test/resources
[INFO] Copying 3 resources from target/maven-shared-archive-resources to target/test-classes
[INFO]
[INFO] --- maven-compiler-plugin:3.13.0:testCompile (default-testCompile) @ cdi ---
[INFO] No sources to compile
[INFO]
[INFO] --- maven-surefire-plugin:3.3.0:test (default-test) @ cdi ---
[INFO]
[INFO] --- maven-jar-plugin:3.4.2:jar (default-jar) @ cdi ---
[INFO]
[INFO] --- artemis-maven-plugin:2.39.0-SNAPSHOT:create (create0) @ cdi ---
[INFO]
[INFO] --- artemis-maven-plugin:2.39.0-SNAPSHOT:cli (start0) @ cdi ---
[INFO] awaiting server to start
[INFO] awaiting server to start
server0-out:     _        _               _
server0-out:    / \  ____| |_  ___ __  __(_) _____
server0-out:   / _ \|  _ \ __|/ _ \  \/  | |/  __/
server0-out:  / ___ \ | \/ |_/  __/ |\/| | |\___ \
server0-out: /_/   \_\|   \__\____|_|  |_|_|/___ /
server0-out: Apache ActiveMQ Artemis 2.39.0-SNAPSHOT
server0-out:
server0-out:
[INFO] awaiting server to start
server0-out:2024-12-09 10:36:08,285 INFO  [org.apache.activemq.artemis.integration.bootstrap] AMQ101000: Starting ActiveMQ Artemis Server version 2.39.0-SNAPSHOT
server0-out:2024-12-09 10:36:08,313 INFO  [org.apache.activemq.artemis.core.server] AMQ221000: Primary message broker is starting with configuration Broker Configuration (clustered=false,journalDirectory=./data/journal,bindingsDirectory=./data/bindings,largeMessagesDirectory=./data/large-messages,pagingDirectory=./data/paging)
server0-out:2024-12-09 10:36:08,346 INFO  [org.apache.activemq.artemis.core.server] AMQ221012: Using AIO Journal
server0-out:2024-12-09 10:36:08,399 INFO  [org.apache.activemq.artemis.core.server] AMQ221057: Global Max Size is being adjusted to 1/2 of the JVM max size (-Xmx). being defined as<PHONE_NUMBER>
server0-out:2024-12-09 10:36:08,424 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-server]. Adding protocol support for: CORE
server0-out:2024-12-09 10:36:08,425 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-amqp-protocol]. Adding protocol support for: AMQP
server0-out:2024-12-09 10:36:08,425 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-hornetq-protocol]. Adding protocol support for: HORNETQ
server0-out:2024-12-09 10:36:08,426 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-mqtt-protocol]. Adding protocol support for: MQTT
server0-out:2024-12-09 10:36:08,426 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-openwire-protocol]. Adding protocol support for: OPENWIRE
server0-out:2024-12-09 10:36:08,427 INFO  [org.apache.activemq.artemis.core.server] AMQ221043: Protocol module found: [artemis-stomp-protocol]. Adding protocol support for: STOMP
server0-out:2024-12-09 10:36:08,484 INFO  [org.apache.activemq.artemis.core.server] AMQ221034: Waiting indefinitely to obtain primary lock
server0-out:2024-12-09 10:36:08,484 INFO  [org.apache.activemq.artemis.core.server] AMQ221035: Primary Server Obtained primary lock
server0-out:2024-12-09 10:36:08,694 INFO  [org.apache.activemq.artemis.core.server] AMQ221080: Deploying address DLQ supporting [ANYCAST]
server0-out:2024-12-09 10:36:08,695 INFO  [org.apache.activemq.artemis.core.server] AMQ221003: Deploying ANYCAST queue DLQ on address DLQ
server0-out:2024-12-09 10:36:08,701 INFO  [org.apache.activemq.artemis.core.server] AMQ221080: Deploying address ExpiryQueue supporting [ANYCAST]
server0-out:2024-12-09 10:36:08,702 INFO  [org.apache.activemq.artemis.core.server] AMQ221003: Deploying ANYCAST queue ExpiryQueue on address ExpiryQueue
[INFO] awaiting server to start
server0-out:2024-12-09 10:36:09,023 INFO  [org.apache.activemq.artemis.core.server] AMQ221020: Started EPOLL Acceptor at <IP_ADDRESS>:61616 for protocols [CORE,MQTT,AMQP,STOMP,HORNETQ,OPENWIRE]
server0-out:2024-12-09 10:36:09,025 INFO  [org.apache.activemq.artemis.core.server] AMQ221020: Started EPOLL Acceptor at <IP_ADDRESS>:5445 for protocols [HORNETQ,STOMP]
server0-out:2024-12-09 10:36:09,027 INFO  [org.apache.activemq.artemis.core.server] AMQ221020: Started EPOLL Acceptor at <IP_ADDRESS>:5672 for protocols [AMQP]
server0-out:2024-12-09 10:36:09,053 INFO  [org.apache.activemq.artemis.core.server] AMQ221020: Started EPOLL Acceptor at <IP_ADDRESS>:1883 for protocols [MQTT]
server0-out:2024-12-09 10:36:09,055 INFO  [org.apache.activemq.artemis.core.server] AMQ221020: Started EPOLL Acceptor at <IP_ADDRESS>:61613 for protocols [STOMP]
server0-out:2024-12-09 10:36:09,058 INFO  [org.apache.activemq.artemis.core.server] AMQ221007: Server is now active
server0-out:2024-12-09 10:36:09,058 INFO  [org.apache.activemq.artemis.core.server] AMQ221001: Apache ActiveMQ Artemis Message Broker version 2.39.0-SNAPSHOT [<IP_ADDRESS>, nodeID=62f1df75-b642-11ef-a46e-3ce1a1d12939]
[INFO] Server started
[INFO]
[INFO] --- artemis-maven-plugin:2.39.0-SNAPSHOT:runClient (runClient) @ cdi ---
Dec 09, 2024 10:36:10 AM org.jboss.weld.bootstrap.WeldStartup <clinit>
INFO: WELD-000900: 2.4.8 (Final)
Dec 09, 2024 10:36:10 AM org.apache.deltaspike.core.impl.config.EnvironmentPropertyConfigSourceProvider <init>
INFO: Custom config found by DeltaSpike. Name: 'META-INF/apache-deltaspike.properties', URL: 'jar:file:/home/jbertram/jboss/src/activemq-artemis-examples/examples/features/standard/cdi/target/cdi-2.39.0-SNAPSHOT.jar!/META-INF/apache-deltaspike.properties'
Dec 09, 2024 10:36:10 AM org.apache.deltaspike.core.util.ProjectStageProducer initProjectStage
INFO: Computed the following DeltaSpike ProjectStage: Production
Dec 09, 2024 10:36:10 AM org.jboss.weld.environment.deployment.discovery.ReflectionDiscoveryStrategy processAnnotatedDiscovery
INFO: WELD-ENV-000014: Falling back to Java Reflection for bean-discovery-mode="annotated" discovery. Add org.jboss:jandex to the classpath to speed-up startup.
Dec 09, 2024 10:36:10 AM org.jboss.weld.bootstrap.WeldStartup startContainer
INFO: WELD-000101: Transactional services not available. Injection of @Inject UserTransaction not available. Transactional observers will be invoked synchronously.
Dec 09, 2024 10:36:10 AM org.jboss.weld.event.ExtensionObserverMethodImpl checkRequiredTypeAnnotations
INFO: WELD-000411: Observer method [BackedAnnotatedMethod] protected org.apache.deltaspike.core.impl.message.MessageBundleExtension.detectInterfaces(@Observes ProcessAnnotatedType) receives events for all annotated types. Consider restricting events using @WithAnnotations or a generic type with bounds.
Dec 09, 2024 10:36:10 AM org.jboss.weld.event.ExtensionObserverMethodImpl checkRequiredTypeAnnotations
INFO: WELD-000411: Observer method [BackedAnnotatedMethod] protected org.apache.deltaspike.core.impl.exclude.extension.ExcludeExtension.vetoBeans(@Observes ProcessAnnotatedType, BeanManager) receives events for all annotated types. Consider restricting events using @WithAnnotations or a generic type with bounds.
Dec 09, 2024 10:36:10 AM org.jboss.weld.event.ExtensionObserverMethodImpl checkRequiredTypeAnnotations
INFO: WELD-000411: Observer method [BackedAnnotatedMethod] protected org.apache.deltaspike.core.impl.interceptor.GlobalInterceptorExtension.promoteInterceptors(@Observes ProcessAnnotatedType, BeanManager) receives events for all annotated types. Consider restricting events using @WithAnnotations or a generic type with bounds.
Dec 09, 2024 10:36:10 AM org.jboss.weld.logging.ValidatorLogger$7 log
WARN: WELD-001478: Interceptor class org.apache.deltaspike.core.impl.throttling.ThrottledInterceptor is enabled for the application and for the bean archive /home/jbertram/.m2/repository/org/apache/deltaspike/core/deltaspike-core-impl/1.7.1/deltaspike-core-impl-1.7.1.jar. It will only be invoked in the @Priority part of the chain.
Dec 09, 2024 10:36:10 AM org.jboss.weld.logging.ValidatorLogger$7 log
WARN: WELD-001478: Interceptor class org.apache.deltaspike.core.impl.lock.LockedInterceptor is enabled for the application and for the bean archive /home/jbertram/.m2/repository/org/apache/deltaspike/core/deltaspike-core-impl/1.7.1/deltaspike-core-impl-1.7.1.jar. It will only be invoked in the @Priority part of the chain.
Dec 09, 2024 10:36:10 AM org.jboss.weld.logging.ValidatorLogger$7 log
WARN: WELD-001478: Interceptor class org.apache.deltaspike.core.impl.future.FutureableInterceptor is enabled for the application and for the bean archive /home/jbertram/.m2/repository/org/apache/deltaspike/core/deltaspike-core-impl/1.7.1/deltaspike-core-impl-1.7.1.jar. It will only be invoked in the @Priority part of the chain.
[INFO] AMQ571000: Discovered configuration class org.jboss.weld.bootstrap.events.ProcessManagedBeanImpl$1@6a261998
[INFO] AMQ571001: Discovered client configuration class org.jboss.weld.bootstrap.events.ProcessBeanImpl$1@76980c75
[ERROR]
java.lang.reflect.InvocationTargetException
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:569)
    at org.apache.activemq.artemis.maven.ArtemisClientPlugin.doExecute (ArtemisClientPlugin.java:86)
    at org.apache.activemq.artemis.maven.ArtemisAbstractPlugin.execute (ArtemisAbstractPlugin.java:80)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
    at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute2 (MojoExecutor.java:370)
    at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute (MojoExecutor.java:351)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:215)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:171)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:163)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:299)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:193)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:106)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:963)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:296)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:199)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:569)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)
Caused by: org.jboss.weld.exceptions.WeldException: WELD-001524: Unable to load proxy class for bean Implicit Bean [javax.enterprise.event.Event] with qualifiers [@Default] with class interface javax.enterprise.event.Event using classloader ClassRealm[plugin>org.apache.activemq:artemis-maven-plugin:2.39.0-SNAPSHOT, parent: jdk.internal.loader.ClassLoaders$AppClassLoader@30946e09]
    at org.jboss.weld.bean.proxy.ProxyFactory.getProxyClass (ProxyFactory.java:397)
    at org.jboss.weld.bean.builtin.AbstractFacadeBean.initializeAfterBeanDiscovery (AbstractFacadeBean.java:61)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:132)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:123)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:62)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:55)
    at java.util.concurrent.FutureTask.run (FutureTask.java:264)
    at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1136)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:635)
    at java.lang.Thread.run (Thread.java:840)
Caused by: java.lang.ExceptionInInitializerError
    at org.jboss.weld.bean.proxy.ProxyFactory.newClassFile (ProxyFactory.java:518)
    at org.jboss.weld.bean.proxy.ProxyFactory.createProxyClass (ProxyFactory.java:473)
    at org.jboss.weld.bean.proxy.ProxyFactory.getProxyClass (ProxyFactory.java:389)
    at org.jboss.weld.bean.builtin.AbstractFacadeBean.initializeAfterBeanDiscovery (AbstractFacadeBean.java:61)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:132)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:123)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:62)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:55)
    at java.util.concurrent.FutureTask.run (FutureTask.java:264)
    at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1136)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:635)
    at java.lang.Thread.run (Thread.java:840)
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @56499781
    at java.lang.reflect.AccessibleObject.checkCanSetAccessible (AccessibleObject.java:354)
    at java.lang.reflect.AccessibleObject.checkCanSetAccessible (AccessibleObject.java:297)
    at java.lang.reflect.Method.checkCanSetAccessible (Method.java:200)
    at java.lang.reflect.Method.setAccessible (Method.java:194)
    at org.jboss.classfilewriter.ClassFile$1.run (ClassFile.java:238)
    at java.security.AccessController.doPrivileged (AccessController.java:569)
    at org.jboss.classfilewriter.ClassFile.<clinit> (ClassFile.java:233)
    at org.jboss.weld.bean.proxy.ProxyFactory.newClassFile (ProxyFactory.java:518)
    at org.jboss.weld.bean.proxy.ProxyFactory.createProxyClass (ProxyFactory.java:473)
    at org.jboss.weld.bean.proxy.ProxyFactory.getProxyClass (ProxyFactory.java:389)
    at org.jboss.weld.bean.builtin.AbstractFacadeBean.initializeAfterBeanDiscovery (AbstractFacadeBean.java:61)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:132)
    at org.jboss.weld.bootstrap.ConcurrentBeanDeployer$AfterBeanDiscoveryInitializerFactory.doWork (ConcurrentBeanDeployer.java:123)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:62)
    at org.jboss.weld.executor.IterativeWorkerTaskFactory$1.call (IterativeWorkerTaskFactory.java:55)
    at java.util.concurrent.FutureTask.run (FutureTask.java:264)
    at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1136)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:635)
    at java.lang.Thread.run (Thread.java:840)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  5.907 s
[INFO] Finished at: 2024-12-09T10:36:10-06:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.activemq:artemis-maven-plugin:2.39.0-SNAPSHOT:runClient (runClient) on project cdi: null: MojoFailureException -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

It looks like something related to JPMS given this error:
java.lang.reflect.InaccessibleObjectException: Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @56499781

Any ideas?

...and ninja'd, lol.
The cdi-cleint bits use ancient deps in both the example and the main tree tests, which pre-date modern versions and just dont work on them. I previously had them skip out when enabling the PR build to work on 16+: https://issues.apache.org/jira/browse/ARTEMIS-3306. It seems like they need ripped and replaced.

It seems like they need ripped and replaced.

I'm investigating that possibility now. Newer versions appear to need Jakarta-specific dependencies which might be a problem.

One option is to just add jvm args to allow it if they can't be replaced.
--add-opens=java.base/java.lang=ALL-UNNAMED

It seems like they need ripped and replaced.

I'm investigating that possibility now. Newer versions appear to need Jakarta-specific dependencies which might be a problem.

Latest versions will yep, could awkwardly need non-latest. Though I seem to recall trying that during the JUnit 5 migration and having 'non-latest uses JUnit 4' issues, which is why I quickly just left it as it was and used junit-vintage.

One option is to just add jvm args to allow it if they can't be replaced.
--add-opens=java.base/java.lang=ALL-UNNAMED

FWIW I tried this workaround out locally on 17 and it worked ok for the cdi-client module tests. Diff:
https://github.com/gemmellr/activemq-artemis/commit/abcbf0c34c724a219c6a452078bb497ca0877d72
Didnt try it on 21+ (though my forks actions will be running it now). The example would also still need a different change to the similar effect, since it doesnt use surefire.

I just tested the same on the example and verified it works. The problem is that I needed to set
export MAVEN_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED"

I'm not sure how to set this in the pom.xml.

I just tested the same on the example and verified it works. The problem is that I needed to set
export MAVEN_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED"

I'm not sure how to set this in the pom.xml.

You need to configure surefire plugin, for example
<plugin>
          <groupId>org.apache.maven.plugins</groupId>
          <artifactId>maven-surefire-plugin</artifactId>
          <configuration>
            <argLine>--add-opens java.base/java.lang=ALL-UNNAMED</argLine>
          </configuration>
        </plugin>

I just tested the same on the example and verified it works. The problem is that I needed to set
export MAVEN_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED"

I'm not sure how to set this in the pom.xml.

You need to configure surefire plugin, for example
<plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-surefire-plugin</artifactId>
  <configuration>
    <argLine>--add-opens java.base/java.lang=ALL-UNNAMED</argLine>
  </configuration>
</plugin>

The examples dont use surefire.

I just tested the same on the example and verified it works. The problem is that I needed to set
export MAVEN_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED"

I'm not sure how to set this in the pom.xml.

Yeah, not sure if it can be currently, think the example application bits run in the same VM as the maven plugin, i.e its too late at that point.
Could maybe abuse a .mvn/jvm.config file, though I think those are meant to be in top-level.

I'm trying out a update for the example that gets it to work on 17 without tweaks. Giving it a try out in CI now against a rebase of these changes. If that works I'll update the example.


Resource manager docs

Motivation and Context
Documentation push for the ResourceManager and related files/classes. Some related refactoring and cleanup included.
Feedback, proofing, etc... welcome. 👍
How Has This Been Tested
Local build.
Types of changes

[x] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

CI fails including cmake generated header esp/gfx/configure.h for doc linking in ResourceManager.h.  Works on local build. Happy for suggestions if anyone knows a simple fix. Otherwise it's not that important and can be removed.

Looks like esp/gfx/configure.h didn't get added to the diff :)

Looks like esp/gfx/configure.h didn't get added to the diff :)

I think this is actually built from esp/gfx/configure.h.cmake into build/esp/gfx/configure.h. I don't have that file locally in the src directory either. However, my local build works and the CI build failed. 🤔

This is why it's failing: https://github.com/facebookresearch/habitat-sim/blob/master/src/esp/gfx/CMakeLists.txt#L66
The configure file needs to be run always.

This is why it's failing: https://github.com/facebookresearch/habitat-sim/blob/master/src/esp/gfx/CMakeLists.txt#L66
The configure file needs to be run always.

That looks like the culprit, thanks for finding this. I'll update it. 👍

Any other thoughts on this before merge?


Try to parse Nginx logs in JSON format

Hi,
I am trying to analyze the logs of Nginx send on stdout in a container, the logs are in json format.
Below my Nginx log format:
log_format vhost  '$host $remote_addr - $remote_user [$time_local] ''"$request" $status $body_bytes_sent ''"$http_referer" "$http_user_agent"';
And here is a line of logs from the Nginx container:
{"log":"\u001b[0;33;1mnginx.1    | \u001b[0mdrive.mysite.com <IP_ADDRESS> - - [26/Dec/2018:12:29:09 +0000] \"GET /storage/1.5/1/storage/clients?full=1\u0026limit=1000 HTTP/2.0\" 200 844 \"-\" \"Firefox/64.0 (Linux x86_64) FxSync/<IP_ADDRESS>81212012434.desktop\"\n","stream":"stdout","time":"2018-12-26T12:29:09.74181707Z"}
So I want to interpret with Goaccess this whole log file, where each line is a json.
So I tried this:
./nginx2goaccess.sh '$host $remote_addr - $remote_user [$time_local] ''"$request" $status $body_bytes_sent ''"$http_referer" "$http_user_agent"';
- Generated goaccess config:
time-format %T
date-format %d/%b/%Y
log_format %v %h - %^ [%d:%t %^] "%r" %s %b "%R" "%u"

Could you help me create this log for Goaccess
Thanks.

Are \u001b[0;33, \u001b[0m typos or actual escape sequences?

Also, please post several lines from your access log. Thanks!

it works, thank you !

Great! Closing this.


Implement Ably Kinesis Rule Resource

Implement Ably Rule Resource with Kinesis Target
As a team, we decided to implement the Ably rule with a Kinesis target first. This PR includes the following items

tfplugindocs generated documentation
Ably Kinesis rule resource with inbuilt source and target.
Updates to provider and models to represent Ably Rule for Kinesis target
Playground tf templates updated
Resource example updated for kinesis integration rule
Acceptance testing configured

NOTE: There's an issue with the staging control API endpoint that we use for testing. Because of this, tests are currently failing. I've used the prod endpoint to verify that acceptance tests work correctly. Output:
~/Documents/src/terraform-provider-ably (feature/IAC-44-Ably-Rule-Resource ✗) make testacc TF_ACC=1 go test $(go list ./... | grep -v 'vendor') -v  -timeout 120m ?       terraform-provider-ably [no test files] === RUN   TestAccAblyApp --- PASS: TestAccAblyApp (10.72s) === RUN   TestAccAblyKey --- PASS: TestAccAblyKey (11.47s) === RUN   TestAccAblyNamespace --- PASS: TestAccAblyNamespace (11.87s) === RUN   TestAccAblyQueue --- PASS: TestAccAblyQueue (12.19s) === RUN   TestAccAblyRuleKinesis --- PASS: TestAccAblyRuleKinesis (9.73s) PASS


Revert "Fix: fix android studio 4.1 plugin path for macOS"

Reverts flutter/flutter#69364 due to https://github.com/flutter/flutter/issues/69863

LGTM

Reverted without waiting for checks to cycle (because it is a clean revert).


Add Japanese README file.

Add README-ja.md file
Add link to README files.

I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

This is brilliant! どうもありがとうございます


[RFC] Add support for systemd initcpio setups

Significant rewrite/expansion of the scencrypt hook to allow it to work with systemd-based initcpios.
Instead of replacing the systemd-cryptsetup flow, this method piggybacks on it by using a generator to add dependencies and overrides for systemd-cryptsetup disk targets, as follows:

systemd-initramfs-gpg-init.service initializes the GPG homedir inside the initramfs by importing all of the public keys which were packed into the initramfs.
<EMAIL_ADDRESS>gets the passphrase or PIN from systemd-ask-password and passes it to gpg, which stores the keyfile at a known location (based on the key's filename).
<EMAIL_ADDRESS>copies the key from the decrypted location to the /run/cryptsetup-keys.d directory where it is automatically picked up by systemd-cryptsetup and used to decrypt the disk.

Because it uses systemd-ask-password, this means you can even use plymouth!
Demo video running in a virtual machine with a real Yubikey passed through on USB: https://fuhry.com/b/initramfs-scencrypt-systemd-demo.webm
This does break backward compatibility with version 1.x, because there is no way to make systemd-cryptsetup work when the key-file column (column 3) of crypttab points to a file. It has to be set to none and the pgp keyfile must be passed in an option instead. A migration script (aptly named scencrypt-migrate) is included which mostly automates the process of modifying crypttab.

Hey, did you maybe forget to include the scencrypt-migrate file?


Code Review

The ICCS RSE Team collectively reviewed this GitHub repository together using the JOSS and SSI review criteria, which we used as the basis of this document. The criteria focus on the quality of the software project as sustainable, open source software. The process included installing and running the software, as well as looking at the associated documentation, metadata and aspects of governance.
The results of the review are captured in a public document.
ICCS Code Review Output
PyRealm - Sustainability Evaluation
The recommended actions are included here as a task list which can be checked off once each is raised as a distinct issue. It is recommended that each of these be tagged with a 'Code Review' label.

[x] #55
[x] #56
[x] #57
[x] #58
[x] #59
[x] #60
[x] #61
[x] #62
[x] #63
[x] #64
[x] #65
[x] #66

This issue can be closed when the separate code review tasks are created as their own issues.

Actions all converted into issues tagged https://github.com/davidorme/pyrealm/labels/code review
Grouped the issues under a project https://github.com/users/davidorme/projects/1/views/1


Windows install error

This is the error I get when trying to install through Cargo on Windows 10. Is there any way to troubleshoot this error ? Thanks

Details
error: failed to run custom build command for `ncurses v5.101.0`

Caused by:
  process didn't exit successfully: `C:\Users\decla\AppData\Local\Temp\cargo-install0KOJbC\release\build\ncurses-198b7c22f92c184a\build-script-build` (exit code: 101)
warning: build failed, waiting for other jobs to finish...
error: failed to compile `git-branchless v0.3.1`, intermediate artifacts can be found at `C:\Users\decla\AppData\Local\Temp\cargo-install0KOJbC`

Caused by:
  build failed

@pwnorbitals Thanks for raising this. It should be solved by @waych's #13. Since the fix hasn't been released to Cargo, you can instead install it directly from source with cargo install --git https://github.com/arxanas/git-branchless.

Thank you very much with your quick response !

@pwnorbitals FYI I just sent out another PR which fixes various test and execution issues on Windows.  See PR #20.


Scope enqueues/dequeues to subject set

This was making the subject and classification routes unbearably slow.
For grouped workflows we can assume that subjects are only queued in
their respective group queue and make sure we're only dequeuing from
that install of all the queues

Once I have this passing I'm going to merge it in. Sorry all, this should resolve the slowness from today.

Also this will only fix the controllers not the worker. We need to figure out something else for them still.


migration state: add source pod and failure reason

What this PR does
Before this PR:
Migration state has no source pod and no failure reason
After this PR:
Migration state has a source pod and a failure reason

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

More information in the migration state of VMI / migration objects

/cc @vladikr

/hold
I broke unit tests

/hold cancel

/lgtm

/approve
Looks good to me.

Thanks @jean-edouard !

@jean-edouard should we back port this?

/cherry-pick release-1.2


Add pre-commit hook ForbiddenBranches

This hook fails on an attempt to commit to a branch matching any of the glob patterns specified in the branch_patterns hook setting.
Closes #327

Thanks!
Merged in 556f78981a693116488b25d62465a9b4ccb2c6c4.


Joplin 1.8.5 is old

Joplin already is at Version 2.0.9. Please update!
https://github.com/laurent22/joplin

Ah no, that is a pre-release. My bad.


[Add-ins] (Yo Office) Add note about spaces not being permitted in project name or folder path.

This PR adds a note to call out a bug that's currently present in the Yeoman generator for Office Add-ins: OfficeDev/generator-office#476. It also removes the screenshots of Yo Office prompts (since screenshots show a currently-unpermitted value for project name). When the bug in Yo Office is fixed, we'll remove this note and restore the screenshot in each article that's updated by this PR.

@AlexJerabek this PR is to update the Outlook quick start and tutorial.

Docs Build status updates of commit 29a8091:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

docs/add-ins/quick-start.md
:white_check_mark:Succeeded

docs/includes/note-yeoman-generator-bug-201908.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ability to change different font colors in text input

this means that text elements can have other spans. BRs, bolds, and italics should also be displayed in the left-hand layers pane so that engineers can explicitly change them.

works via contenteditable


Add forced value-setting as Edge workaround

This is a hackish compensation for a bug in Edge. I don’t think it’s necessarily something you’d even want in your codebase because it’s so unsightly, but I’m submitting it to get your feedback. (Also it’s branched off v2.2.2 because I haven’t yet converted to contextual components.)
Without this, a select with the required flag is marked as invalid until you change the value. You can change the value yourself and then change it back to the default, but even that workaround fails if the select only has one value.
I don’t really feel like this is an acceptable solution, but travis-web users on Edge are unable to add cron jobs on repositories with only one branch because of it. Even on repositories with multiple branches, all three fields have to be changed before a cron can be added:

Do you have any better ideas on how to address this? I’d love to find a cleaner way!

shakes fist at edge
I think we should hide this behind a UA check since this is a pretty gross browser bug.

Closing this PR in favor of #170


[homepage] add last post link somewhere in the top

Now it's not possible to see new content on the homepage
See first line

Done on homepage


add unicore board

Should address the request in https://github.com/v923z/micropython-builder/issues/20

Hi, automatic firmware generation for ESP32 is not working. Could you check it and make corrections?

Yes, I know. All ESP32-based boards fail in the CI. I believe espressif might have switched versions of the IDF, but I haven't yet had time to find that out. However, I'm always happy, if people contribute;-)


Easy language link (DE) only (EXPOSUREAPP-10220)

Remove old keys from DE strings to clean them up
Add news keys to non_translatable.xml as easy language is only available in German

Testing

Check First Onboarding Screen and About the App screen in DE -> easy language should be visible and clickable
other language -> no link

Ticket
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-10220

oh .. why this?
Why hide this link from all users except those that have chosen German as primary language in cwa?
Actually I liked the Android solution very much, because:

the audience for easy language are not only German natives, but also users with limited German
especially bi-lingual users might appreciate the easy language
last not least, the linked content is not only easy language but also sign language

So, please consider to let the users decide themselves if they want to open that link

oh .. why this? Why hide this link from all users except those that have chosen German as primary language in cwa?
Actually I liked the Android solution very much, because:

the audience for easy language are not only German natives, but also users with limited German
especially bi-lingual users might appreciate the easy language
last not least, the linked content is not only easy language but also sign language

So, please consider to let the users decide themselves if they want to open that link

@thomasaugsten FYI

oh .. why this? Why hide this link from all users except those that have chosen German as primary language in cwa?
Actually I liked the Android solution very much, because:

the audience for easy language are not only German natives, but also users with limited German
especially bi-lingual users might appreciate the easy language
last not least, the linked content is not only easy language but also sign language

So, please consider to let the users decide themselves if they want to open that link

I think this is a valid point.

oh .. why this? Why hide this link from all users except those that have chosen German as primary language in cwa?
Actually I liked the Android solution very much, because:
* the audience for easy language are not only German natives, but also users with limited German

* especially bi-lingual users might appreciate the easy language

* last not least, the linked content is not only easy language but also **sign language**

So, please consider to let the users decide themselves if they want to open that link

Hi @ndegendogo , Thanks for your feedback. The easy language link is appearing in two places in the App

Onboarding Flow -> was only visible in DE
App Information screen -> is visible in all languages ,but led to the German explanation and it is considered as a bug in Android side.

I see your point. if it has to visible in the App information regardless of user's locale. it has to be visible everywhere for consistency. I will transfer your feedback internally and come back to you

We got a feedback that it should be fine to have it visible in other locales too


[INTERNAL] .editorconfig: Add more file extensions

Coverage remained the same at 94.706% when pulling 9eedcb33bcef0c82b8b52275360b3b4961347806 on update-editorconfig into 07a5be2b6d9aa23cf78ddd17951c832d6dec7bef on main.


QA work for null states

[x] Got show images on iPad to update width properly
[x] Hides blurb if there is no blurb
[x] Hides Current & Upcoming Shows if there are none
[x] Uses 'work' instead of 'works' for singular works in related artist component
[x] Hides articles if there are no articles for artist
[x] Hide show data if missing
[x] Hide ‘Other Works’ if all works are for sale. (see https://www.artsy.net/artist/nicole-cohen/works)

Fixes #68.
Fixes #67.
Fixes #66.
Fixes #64.
Will fix #65.

Added another TODO for when all works are for sale.

I'm going to make a separate PR for the images on iPad because that's going to be its own investigation. Also, @briansw , what should we display if an artist has no shows (see https://www.artsy.net/artist/kaikai-kiki)?

@sarahscott is it simple enough to hide the shows tab in that case?

@briansw  I'll look into it. @alloy might know better.

@sarahscott @briansw I don’t think that should be a problem, you have those counts before any view is rendered, so you should be able to use that to determine what tabs to render.

Something like:
const tabs = ['ABOUT', 'WORKS'];
if (this.artist.previous_shows.length && this.artist.upcoming_shows.length && this.current_shows) {
  tabs.push('SHOWS');
}
…
<TabView titles={tabs}>
  …
</TabView>

You’ll need to pull the show fragments that define the previous, upcoming, and current shows lists from their respective components into the Artist component, otherwise you can’t access them.

Merging including those tap delays, but please let me know what the deal is with them.

🎉
Great work, @sarahscott :clap:


Bug on airflow of Openmetadata

Checks

[X] I have checked for existing issues.
[X] This report is about the Openmetadata Helm Chart.
[X] This report is about the Openmetadata Dependencies Helm Chart.

Description
Bug Description:
When install openmetadata-dependencies. after it done. i have this bug. after that. when using OM ingres metadata from other source. this worker of airflow always error. I think this is some misstake in here. How can i fix it.

Steps to reproduce
No response
Chart Version
new
App Version
new
Kubernetes Version
v1.26.5
Helm Version
v3.13.2
Relevant Logs
No response
Custom Helm Values
No response
Have you joined Openmetadata Slack community?
Yes

Hello, I have exactly the same problem. Have you by any chance found a solution?
Regards,

Unfortunately, we have exactly the same problem. With the latest version of the charts as well as with version 1.3.3. Has anyone been able to solve the problem?

@Doritanh, @FleischerT . I resolved the error. The issue was with the MySQL chart in the Helm chart, which had an error I couldn't fix. You can find the log on the MySQL pod; it contains a warning. When I tried deploying a new MySQL database for Airflow and OP, the error was resolved.


solve linting problems

closes https://github.com/SABS-R3-Epidemiology/epistrains/issues/61

looks good to me!


new namespaces on resource creation

New namespaces to the repository are only correctly added when going through resource.update().
For example, if a new resource is created, and a triple is added before create that contains a namespace new to the repository, this namespace is not added correctly.
This happens if:

the namespace prefix and uri exist in repo.context, and thus the nascent resource's rdf.prefix
or, even if resource.add_namespace is run prior to create

But it does not happen if the resource has been created and update is run, even if adding namespace.  This would seem to suggest it's related to new namespaces in .create, which does not correctly pass along this new namespace.
The only successful way to introduce a new namespace appears to be through a PATCH request, as is done with resource.update()?

More fuel for the fire: it's not tenable to add triples to a resource before creation, so this might all be moot.
Tried instantiating a resource, adding a triple, then creating without specifying a URI (thereby issuing a POST request), which failed with the following error:
ERROR 22:45:43.016 (PersistingRdfStreamConsumer) http://localhost:8080/rest/ is not in the topic of this RDF, which is http://localhost:8080/rest/a8/9f/a1/19/a89fa119-0b79-4ad5-aaab-6c827ee53296.```

Which is now unsurprising, as this resource was attempting to write triples for a URI it did not yet know.  So, perhaps it makes sense to restrict `add_triple` unless `resource.exists` is True?  This is set during creation and retrieval.  Downside to this would be creating resources with `auto_refresh` off, as it would not yet know it exists....

Perhaps work to catch this error and inform, but not prevent?

Might be a little trickier, as DirectContainer and IndirectContainer both create triples on creation, outside of controlled channels like resource.update.
For example, for an empty repository without namespace prefixes pcdm or ore, creating Direct or Indirect containers that automatically created triples with those prefixes -- if those prefixes have not been created through update -- results in bad prefixes.
Not quite "bad" in the same way, these are not ns001, but instead use the entire URI like http://www.openarchives.org/ore/terms/.

One two-prong approach might work:

when a repository instance is started, it checks for prefixes it has as part of repo.context, and issues an update to the root node to propagate those namespaces throughout the repository
prevent add_triple from working before resource creation, or inform as mentioned above

This would add those namespaces to the repository before any Direct or Indirect containers could do so automatically, and would also prevent resources that had not been created (which we've determined is not a good idea) from adding triples.

https://groups.google.com/forum/#!topic/fedora-community/Q38vsSiyyu8

See forum post above - fixed in Fedora 4.8.  Continuing to use 4.7 for testing, as this behavior has not yet affected tests, but will update docker image for travis if need be, and certainly when 4.8 is released.


reset all (from x button) fixed

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
No new function
[x] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Before:
When clicking reset all (X) button, then it remove all filter and only show toggle filter (but not visible what it filter)
Screencast from 06-09-2023 09:21:54 AM.webm
After:
Screencast from 06-09-2023 09:22:18 AM.webm
This fixed #6722

@Valpuia The reason there are 2 separate methods removeTableFilters and resetTableFiltersForm is that they do two separate things. The removeTableFilters removes all the filters independent of the default values, while the resetTableFilter resets everything to the defaults.
The reason the difference is important is because in your example, if you use the default() method on the Filter and then test your PR, when you click the Remove Table Filters button, it doesn't actually remove the filter since the default is for it to on.
The fix to the problem you discovered, try adding this to the removeTableFilters method:
$field->state(match (true) {
      is_array($state) => [],
      $state === true => false,
      $state === false => false, // <- add this line
      default => null,
  });

This fixes the actual issue which is that the match function isn't picking up on the false state and instead sets it to null which causes the issues you found.

@Valpuia The reason there are two separate methods removeTableFilters and resetTableFiltersForm is that they do two separate things. removeTableFilters removes all the filters independent of the default values, while resetTableFilter resets everything to the defaults.
The reason the difference is important is because in your example, if you use the default() method on the toggle Filter and then test your PR, when you click the Remove Table Filters button, it doesn't actually remove the filter since its default state is “on”.
To fix the problem you discovered, try adding this to the removeTableFilters method:
$field->state(match (true) {
      is_array($state) => [],
      $state === true => false,
      $state === false => false, // <- add this line
      default => null,
  });

This fixes the actual issue which is that the match function isn't picking up on the false state and instead sets it to null which causes the issues you found.
And I'm glad you found this bug since it also affects some of my apps. No client has brought it to my attention yet, but I'm glad I can fix it before anyone notices! pray

Yes, now I understand the logic a little bit, and changes my PR according to your suggestions..

I'm 99% sure this would break ternary filters, can you check those please?

I think you're totally right. We'll have to figure out a different solution to the problem. Maybe checking that it isn't an instance of TernaryFilter? Or is an instance of Filter? Not sure...on my phone.

I check again and this doesn't break anything if i recall..
But one thing I notice is the ternary itself is not working (it cannot filter) but this filter clear is not the problem..
Ternary Filter cannot filter boolean.. but clearing (with my PR) doesn't break anything..
Let me know if I'm missing something..

I can confirm as well that this doesn't appear to break Ternary Filters. I tested with a normal boolean ternary filter, a nullable ternary filter, as well as using the Deleted records filter (which is just a ternay filter that has true/false/blank queries already set up).
I agree Dan that it seems like this SHOULD break ternary filters. Since ternary filters can have a false state it seems that by adding the false match there, you'd think this would actually set the ternary filter, but it doesn't seem to. Weird.
Is there another combination of ternary filters we could test against?
And @Valpuia regarding ternary filters not working, I'm not seeing that problem. Ternary filters are working for me. Maybe try casting your column to boolean?

Tables\Filters\TernaryFilter::make('is_admin') this filter is not working in User (model/resource), and yes I casts as boolean also.. but this same filter is working in another model/resource..
Screencast from 06-12-2023 10:37:54 AM.webm

Hi, I got email about fix-code-style, Attempt #2: All jobs have failed what should I do?


Add cri-containerd hostname tests

Adds a hostname test for a workload container in each of:
runhcs-wcow-process, runhcs-wcow-hypervisor, runhcs-lcow runtimes.

Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

CRI: https://github.com/jterry75/cri/pull/44


Fixed bug in combining data range and soft_range

The dimension_range utility which is responsible for combining different types of ranges did not correctly handle soft ranges if one of the limits is not defined.

[x] Fixes https://github.com/pyviz/pyviz/issues/117
[x] Fixes https://github.com/ioam/holoviews/issues/3021
[x] Added unit test

@jlstevens Ready to review and merge.

Looks good. Merging.


Thermistor

Not sure if it makes sense to have a thermistor per cell or let the host controller handle reading a single thermistor for the whole pack.

Added per-cell thermistor: https://github.com/scttnlsn/cell-monitor/commit/843964791c5e5899a78a108f2de5d110ddb3e3c5


Add create action from Dttp::Provider to Provider

Context
On the list of provider results from the DTTP lookup add an 'Create' button that creates a provider record for the selected provider.
Changes proposed in this pull request

Change Dttp::Provider#dttp_id column to be a uuid instead of string.
Add a relationship between Dttp::Provider and Provider based on dttp_id to easily check against existing records
Add a create button that imports the Dttp::Provider record over to the Provider list.

Guidance to review
Navigate to system-admin/dttp_providers, and from the list, choose a provider to import and click the "Create" button against that record.
On successful creation, the user is redirected to the providers page.

 [M] Add provider from DTTP


Implement CZPowGate00 and CZ00

This is the first part of https://github.com/quantumlib/Cirq/issues/2587

@Strilanc I agree.


dashboard variable for fields specified for each SLO dashboard

currently only added to Global overview dashboards for top 4 common labels/fields

closed with https://github.com/SumoLogic-Incubator/slogen/pull/15


Please invite me to the Recode-Hive GitHub Community Organization

Name
amar jha
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


[BUG]How we can add ElasticSeach SSL certificate

Describe the bug
To Reproduce
Steps to reproduce the behavior:

I tried to install botkube helm chart and integrate it with Elasticseach which is installed in one of VM'

POD get to install successfully but in logs, I can see below error
ERRO[2021-09-21T10:04:06Z] Failed to get index. Error: Head "https://xxxyz:9201/esskube-2021-09-21": x509: certificate signed by unknown authority

in the helm option also I don't see any parameter where we can pass the certificate.

Expected behavior
while running the helm chart we should override ES certificate

After adding skipTLSVerify: true into a secret now I am not getting the above error but now indexes are not getting created and also nothing in error.
And also in doc nothing mentions do we need to create an index before or not.


⚠ Turnos 24x7 has degraded performance

In f714c49, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 3237 ms

Resolved: Turnos 24x7 performance has improved in 4eccf54.


Hack->Config: queue depth tabulation

Coverage increased (+0.2%) to 67.363% when pulling 6f22e9d672df5a5121fd700cf1c58fbd3fcfb63b on queue_depth_tabulation into 85ea27e1bf84fcdf82b0174c9b55b3b709728435 on master.

Coverage increased (+0.06%) to 67.177% when pulling a3850656b649cf56d3c49cfccb4d8ef566a3bf9c on queue_depth_tabulation into 85ea27e1bf84fcdf82b0174c9b55b3b709728435 on master.


Release 1.2.0-30 (mvp3)

Recommendations and Forecasts
A personalized immunization schedule can be accessed from the homepage, in addition, immunization health records that are part of a series will now display a forecast of future doses when viewing the record details.
Communication Banner
Service outages and other communications from Health Gateway will be displayed on the homepage and can be dismissed once read.

Thanks for reviewing this Amit! Here are my considerations:

We shall remove e.printstacktrace() if its sneaked anywhere in the code.

The idea is to avoid handling them silently and better understand the root cause of the errors. A use case is when we are fetching the records and one of them fails, without that we can't know which one (and why) it failed and triggered the error notification. Is there a specific problem we wanted to avoid?

Would like know if we tested DB migration on top of current PROD app.

Yes, a specific ticket was created to test the migrations: HAPP-1013. They were done on stage build and a smoke test has been done on prod. I also tested the migrations in the release build (1 to 2, 1 to 3, 2 to 3)

Merge conflicts needs to be resolved and please test the app post that.

Done!


Make OS X builds

Like in #33 we need to set up automatic OS X builds from Travis to make sure that it builds. We also need to find someone to test the build.

Note: Travis has OS X support but it is in closed beta atm.

Built OK on OS X Yosemite 10.10.1 with cmake 3.0.2.

Thank you Tommy!

This is all I see when I run the client on OS X:
fogleman:client$ ./konstructs play.konstructs.org

Sorry, the documentation is not completely up-to-date, I'm experimenting with a primitive GUI. "Press T" and follow the instructions. Command line arguments are ignored atm.

"T" just shows Server>. I tried "help" but that didn't work.

Hehe, yeah, I think this needs some more usability tests :)

Press T
enter "play.konstructs.org"
enter username
enter a password

A new user will be created, if needed.
Think it is time to update the documentation :)

I see.
Looks like the max password length isn't very long. My first attempt crashed. :)

Hehe, noted!

What was the end result here, did #154 improve things enough to close this issue?

@petterarvidsson yes, it once again builds cleanly on OS X.
We need to publish the OS X build like we do with Windows and Linux. When that's done we can finally close this one!

@nsg Do you think is something that could be done for release 7? Shall I add it?

@petterarvidsson release 8 or 9 probably in the speed that we release code at the moment :)

@tommymalmqvist are you willing to try to install a dmg-file if I succeeds to make one? Is dms still a valid option on macOS these days?

.dmg is perfectly fine! Homebrew or (brew cask in this case) is even more awesome.

Thank you for the feedback, I will give it a try!


Drag-and-drop of GCT file does not work (in Chrome).

Describe the bug
Dragging and dropping a .GCT file on the landing page doesn't work in Chrome.
Instead of loading the file it opens a new browser tab with the contents of the file displayed.
I assume this is a browser issue with Chrome because it works as expected in Safari.
Opening the same file using the "Enrich my RNA-seq Data" button works fine.
To Reproduce
Try to drag and drop a .GCT expression file onto the landing page in Chrome. (Contact me if you need a .GCT file, or use the one Veronique sent on Dec 7, I don't want to post the link here publicly).
Expected behavior
Should take me to the Class Chooser UI.

@mikekucera, the GCT extension was added to the whitelist so this can be closed, right?

yes


Implement azure boards

This should be quick, using https://github.com/fsspec/adlfs. Looking back, I have everything set up for CI in .env.dev.

Released in v0.0.6: https://github.com/rstudio/pins-python/releases/tag/v0.6.0
# set these environment variables:
# AZURE_STORAGE_ACCOUNT_NAME
# AZURE_STORAGE_ACCOUNT_KEY

from pins import board_azure
board <- board_azure("some_container")


Create personas

create at lest n+1 for n-sized team (at least 4 for us)

[ ] Persona 1 (Andrew)
[ ] Persona 2 (Andrew)
[ ] Persona 3 (Beatriz)
[ ] Persona 4 (Aron)

Created two personas: Jessica/Clarence and Beth. Here is the link to the Jamboard.
https://jamboard.google.com/d/1haQbWqZEbhgPHvbV2ly60fo2-vuMrqLiXxJ_AHvrjAY/viewer?f=2

New working link:
https://jamboard.google.com/d/1haQbWqZEbhgPHvbV2ly60fo2-vuMrqLiXxJ_AHvrjAY/edit?usp=sharing

Added two more personas:
https://jamboard.google.com/d/1haQbWqZEbhgPHvbV2ly60fo2-vuMrqLiXxJ_AHvrjAY/edit?usp=sharing

Feedback from professor:
follow format given to really show different characteristics of the persona

Made changes to Marc Persona to reflect template and add context/pain point/goals/ etc

added liz & Jay

Need to also fix Jam Board since personas arent the only issue

Changed Clarence + Jessicas Personas

Added Eli and George

n

Adjusting personas based on Professor feedback

Tell more about night time care, who is taking care of baby and how often?

Updated Marc:

Updated Eli and George

Updated Jessica and Clarence:

Updated Liz and Jay:

Updated Mei:


Fix user verify

Coverage increased (+0.03%) to 82.838% when pulling 6b5a69a8be174e3323a9f49309fcb62a68484190 on josenavas:fix-user-verify into 5345a850e814df95d9629f79578d62a72c9561fc on biocore:master.

General question: Why move this one piece from verify to create? It was originally in verify because a user can't log in until they are verified, so we saved extra message rows for users that sat unverified. Also, any messages that expire before a user verifies shouldn't matter since they expired before the user can log in and see them.

@squirrelo the original code has a pathological case: if a user registers, then messages get added, and the user tries to verify their email, the system will fail cause the messages have already been added:
ValueError: Error running SQL query:
Query: INSERT INTO qiita.message_user (email, message_id)
                             SELECT %s, message_id FROM qiita.message
                             WHERE expiration > %s
Arguments<EMAIL_ADDRESS>datetime.datetime(2016, 3, 31, 4, 49, 43, 242531)]
Error: duplicate key value violates unique constraint "idx_message_user"
DETAIL:  Key (email<EMAIL_ADDRESS>16) already exists.

Closing in favor of: https://github.com/biocore/qiita/pull/1731


Replace table shortcode with render hook

New on Sep 3 2024 in 0.134: https://gohugo.io/render-hooks/tables/

This is probably solved in c9095e948be4e3ff52123053e63658653383a4d0, simply by using markdown attributes to manually add class instead. Then content can choose what type of styling to use, if adding more in the future.


keyTap only works in debug mode

I'm using node v5.11.1, socket.io 1.4.6 and robotjs 0.4.2.  My code is very simple ...
var robot = require("robotjs");

function pausePlayer(){
	robot.keyTap('z','alt');
}

However, the pausePlayer() function is called by a handler on socket.io.
	socket.on('pause player', function(data){ 		pausePlayer(); 	});
If I run the program in a debugger, with a breakpoint on the keyTap() line, it works fine and has the expected results.  But if I run it real-time, it doesn't seem to do anything.
Using setKeyboardDelay() with a larger number seemed in incur a delay, but the key-press still didn't have any effect.
Not sure how to debug this one further. Your help would be appreciated.

With more investigation, this may have something to do with Google Chrome consuming all Global Hotkeys while it has the focus.  Jumping to the debugger took focus away from Chrome.  I had tried many combinations of keys with modifiers, so I had assumed I would have hit an unused combination sometime, but it seems Chrome consumes them all.

Interesting, I'm glad you figured it out! Let me know if you need anything.


Change in compile_commands.json doesn't update syntax highlighting of opened files

After updating my CMakeLists files -for example to add an external dependency with find_package()/target_link_libraries()- and running CMake configure again to update compile_commands.json (through CMake tools extension), syntax highlighting is not updated accordingly in opened files.
I have to close/open those files in the editor to see proper changes in syntax highlighting (no need to restart language server, so I guess compile_commands watcher properly works).
Here is my settings.json (I also use conan but I think that the issue is orthogonal):
{
    "C_Cpp.intelliSenseEngine": "Disabled",
    "cmake.buildDirectory": "${workspaceFolder}/build/${buildKit}_${buildType}",
    "cmake.configureArgs": [
        "-DCMAKE_TOOLCHAIN_FILE=${workspaceFolder}/build/${buildKit}_${buildType}/.conan/conan_toolchain.cmake"
    ],
    "cmake.configureOnEdit": false,
    "cmake.configureOnOpen": false,
    "cmake.copyCompileCommands": "${workspaceFolder}/build/.clangd/compile_commands.json",
    "cmake.preferredGenerators": [
        "Ninja"
    ],
    "cmake.statusbar.visibility": "compact",
    "clangd.arguments": [
        "--compile-commands-dir=${workspaceFolder}/build/.clangd",
        "--query-driver=/usr/bin/clang++"
    ]
}

System information
Clangd version (from the log, or clangd --version): 13.0.0
clangd extension version: 0.1.13
Operating system: Macos Big Sur

I would like to voice support for reparsing opened files upon compile_commands.json change!  A number of times as I have been setting up my build environment, I keep thinking I've done something wrong when syntax highlighting doesn't change; only to eventually realize that I just had to resave/alter the opened files 🙃

I feel this hard working with unreal, I spam ctrl+s a lot to get clangd to reprocess a file.