[CRASH] Hot-Reload Assertion failed: !IsRooted()

Have consistent crashing on hot-reload in a project (but it works fine on initial editor load and rebuild). Project has a Blueprint Pawn based on an underlying C# class, and some AActorComponents (also written in C#) added to it.
Seems related to the OLD version of the object, being marked for garbage collection while it is still IsRooted
Crash log being hit on reload is:
Assertion failed: !IsRooted() [File:C:\Program Files\Epic Games\UE_5.5\Engine\Source\Runtime\CoreUObject\Public\UObject\UObjectBaseUtility.h] [Line: 179]

UnrealEditor_UnrealSharpEditor!`FCSReinstancer::StartReinstancing'::`2'::<lambda_2>::operator()<TMap<UScriptStruct *,UScriptStruct *,FDefaultSetAllocator,TDefaultMapHashableKeyFuncs<UScriptStruct *,UScriptStruct *,0> > >() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\Reinstancing\CSReinstancer.cpp:163]
UnrealEditor_UnrealSharpEditor!FCSReinstancer::StartReinstancing() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\Reinstancing\CSReinstancer.cpp:171]
UnrealEditor_UnrealSharpEditor!FUnrealSharpEditorModule::StartHotReload() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\UnrealSharpEditor.cpp:216]
UnrealEditor_UnrealSharpEditor!FUnrealSharpEditorModule::OnCompileManagedCode() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\UnrealSharpEditor.cpp:227]
UnrealEditor_UnrealSharpEditor!TBaseStaticDelegateInstance<void __cdecl(void),FDefaultDelegateUserPolicy>::ExecuteIfSafe() [C:\Program Files\Epic Games\UE_5.5\Engine\Source\Runtime\Core\Public\Delegates\DelegateInstancesImpl.h:779]
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_Slate
UnrealEditor_ApplicationCore
UnrealEditor_ApplicationCore
UnrealEditor_ApplicationCore
UnrealEditor_ApplicationCore
user32
user32
UnrealEditor_ApplicationCore
UnrealEditor
UnrealEditor
UnrealEditor
UnrealEditor
UnrealEditor
UnrealEditor
kernel32
ntdll

Added this PR to add some additional logging and skip garbage collecting in this case: https://github.com/UnrealSharp/UnrealSharp/pull/308
Will see if I can get a better idea of why/what is getting it into a bad state

With defensive code to stop the crash, can capture the following issue, where it thinks the Blueprint is 'Malformed'
LogUnrealSharpProcHelper: dotnet with args ("C:/Dev/ArcadeKit/Plugins/UnrealSharp/Binaries/Managed/UnrealSharpBuildTool.dll" --Action BuildWeave --EngineDirectory "C:/Program Files/Epic Games/UE_5.5/Engine/" --ProjectDirectory "C:/Dev/ArcadeKit/" --ProjectName ArcadeKit --PluginDirectory "C:/Dev/ArcadeKit/Plugins/UnrealSharp" --DotNetPath "C:\Program Files\dotnet\dotnet.exe") took 1.207725 seconds to execute.
LogUnrealSharp: Display: Unloading plugin ManagedArcadeKit...
LogUnrealSharp: Display: ManagedArcadeKit unloaded successfully!
LogUnrealSharp: Display: Successfully loaded plugin: ManagedArcadeKit
LogUnrealSharp: Display: Successfully loaded Assembly with path ../../../../../../Dev/ArcadeKit/Binaries/Managed/ManagedArcadeKit.dll.
Re-instancing ArcadePawn_C after reload.
LogOutputDevice: Warning: Script Stack (0 frames) :
LogStats: FPlatformStackWalk::StackWalkAndDump -  0.050 s
LogOutputDevice: Error: === Handled ensure: ===
LogOutputDevice: Error: Ensure condition failed: !OwnerClass || !OwnerClass->GetClass()->IsChildOf<UBlueprintGeneratedClass>() || OwnerClass->ClassGeneratedBy  [File:D:\build\++UE5\Sync\Engine\Source\Editor\UnrealEd\Private\Kismet2\BlueprintEditorUtils.cpp] [Line: 728]
LogOutputDevice: Error: Malformed Blueprint class (ArcadeCameraComponent_C_OLD_37496) - bad node dependency, unable to determine if the K2Node_VariableGet node (/Game/Core/BP_ArcadeCharacter.BP_ArcadeCharacter:EventGraph.K2Node_VariableGet_0) should be refreshed or not. Currently compiling: BP_ArcadeCharacter
LogOutputDevice: Error: Stack:
LogOutputDevice: Error: [Callstack] 0x00007ffb29e11b17 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2925766e UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb29335892 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb27838bd6 UnrealEditor-Kismet.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb277b38c8 UnrealEditor-Kismet.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb292a3202 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb29310ab1 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb293401b5 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2933e1d9 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2933f295 UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2933d89e UnrealEditor-UnrealEd.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00000223d9a3f6d8 UnrealEditor-UnrealSharpEditor.dll!FCSReinstancer::StartReinstancing() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\Reinstancing\CSReinstancer.cpp:147]
LogOutputDevice: Error: [Callstack] 0x00000223d9a3a569 UnrealEditor-UnrealSharpEditor.dll!FUnrealSharpEditorModule::StartHotReload() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\UnrealSharpEditor.cpp:216]
LogOutputDevice: Error: [Callstack] 0x00000223d9a33309 UnrealEditor-UnrealSharpEditor.dll!FUnrealSharpEditorModule::OnCompileManagedCode() [C:\Dev\ArcadeKit\Plugins\UnrealSharp\Source\UnrealSharpEditor\UnrealSharpEditor.cpp:227]
LogOutputDevice: Error: [Callstack] 0x00000223d9a2f267 UnrealEditor-UnrealSharpEditor.dll!TBaseStaticDelegateInstance<void __cdecl(void),FDefaultDelegateUserPolicy>::ExecuteIfSafe() [C:\Program Files\Epic Games\UE_5.5\Engine\Source\Runtime\Core\Public\Delegates\DelegateInstancesImpl.h:779]
LogOutputDevice: Error: [Callstack] 0x00007ffb2e1f73a5 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e1f7521 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e322c29 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e3280d0 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e2f488a UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e39145c UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e3af332 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e32aa19 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e13a7eb UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e1254a4 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e17f23f UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e16d5c6 UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffb2e161f9c UnrealEditor-Slate.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbb081aaf0 UnrealEditor-ApplicationCore.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbb0805786 UnrealEditor-ApplicationCore.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbb081d85e UnrealEditor-ApplicationCore.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbb08005b7 UnrealEditor-ApplicationCore.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbd6dd83f1 USER32.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbd6dd7eb1 USER32.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbb083bb26 UnrealEditor-ApplicationCore.dll!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70b632d UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70d57ac UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70d589a UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70d9114 UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70ebd04 UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ff6c70ef0ba UnrealEditor.exe!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbd623259d KERNEL32.DLL!UnknownFunction []
LogOutputDevice: Error: [Callstack] 0x00007ffbd7aaaf38 ntdll.dll!UnknownFunction []
LogStats:                SubmitErrorReport -  0.000 s
LogStats:                    SendNewReport -  0.755 s
LogStats:             FDebug::EnsureFailed -  0.806 s
Re-instancing ArcadeCameraComponent_C after reload.
Re-instancing ArcadeNetworkMovementComponent_C after reload.
LogSlate: External Image Picker: DecompressImage failed
LogUObjectHash: Compacting FUObjectHashTables data took   0.38ms
LogUnrealSharpReinstancer: Error: Cannot mark NetData_OLD_37504 for GarbageCollection because IsRooted
LogUObjectHash: Compacting FUObjectHashTables data took   0.37ms
Display: Reload/Re-instancing Complete: 3 classes changed, 1 scriptstruct changed

Editor seems to continue working after this...
Hitting 'Compile C#' in the editor again and the error doesn't show again, but if I make a change, or close the project and re-open it I can hit it again.  (This is possibly just the Old != New logic kicking in (where on first load Old is null so it updates))

Managed to whittle the cause down to just a [UStruct].  Minimum steps to reproduce:
Pre-Setup

Via the Unreal Launcher, create a new Blank (C++) Project using the latest version of UE (5.5.1)
Add /Plugins/Unreal (git pull latestest version from github)
Open the .uproject and 'Yes' to build the plug-in modules at the prompt
Select Yes to "No C# projects were found. Would you like to create a new C# project?" and accept defaults

Now, in the ManagedProject just add the following code:
using UnrealSharp.Attributes;

namespace ManagedBlankProject
{
    [UStruct]
    public struct FMyStruct
    {
        [UProperty]
        public float X;
    }
}

Hit 'Compile C#' and it all works fine
IMPORTANT: Close the Editor, and re-open via the .uproject
Hit 'Compile C#' and it crashes...
(at least for me, hopefully someone else can verify it's not my machine driving me crazy)

https://github.com/UnrealSharp/UnrealSharp/pull/311
Solved


Update a subset of data through Scenario.read_excel()

For some users who aim to import the data by reading that from Excel, there is only one option and that is to import the entire Excel workbook through ixmp.Scenario.read_excel(). This may be unnessary and time-consuming, if a user aims to edit a subset of data in Excel, e.g., only in one parameter, and update that part of data in their database. This type of workflow does not need to import the whole data again each time using read-excel().
It will be useful to add a feature to read_excel() to select a subset of data, either by sheet names, by filtering using the same syntax as ixmp, or combination of both.
For example: ixmp.Scenario.read_excel('filename.xlsx', sheet_name='bar', filters={'node':  'foo'})

Thanks for noting this @behnam-zakeri!
I removed assignments for myself and @OFR-IIASA because it has not been decided who will work on this.


(WIP) allow costumize faq tabs

Checklist

[ ] I have read the CONTRIBUTING guide
[ ] Good naming (as clear and simple as possible)
[ ] Correct behavior if external API endpoints are down, return 404, 504 (or no answer), 401 errors (ddos simulation)
[ ] I tested desktop/mobile resolution
[ ] I tested light/dark theme
[ ] I tested different languages
[ ] I checked the functionality once again (AFFECT MONEY)
[ ] I checked the work on the Testnet
[ ] I checked the work on the Mainnet
[ ] I checked the work in the plugin
[ ] I checked the PR once again

Tests
Please start auto tests as follows:

add a label swap test to start swap tests
add a label withdraw test to start withdraw tests

You can skip these tests if you completely sure that your changes aren't related to this functional
Original issue
https://github.com/swaponline/MultiCurrencyWallet/issues/5001

Video / screenshot proof

Мержить после этого https://github.com/swaponline/multi-currency-wallet-pro/pull/95


fix: representation function only create service user if id exists in message

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2178
PR adds a check to consumer function to only createOrConnect represented Or representative if the representedId and representativeId exists in the message.
Useful information to review or test
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improve documentation

Recommend Anaconda distribution for getting started.

We need to discuss this, and present a coherent plan to Vince/Sandia.
From: Pete Bachant<EMAIL_ADDRESS>Sent: Sunday, January 25, 2015 10:42 PM
To: UNH-CORE/RVAT-Re-dep
Subject: [RVAT-Re-dep] Improve documentation (#8)
Recommend Anaconda distribution for getting started.
—
Reply to this email directly or view it on GitHubhttps://github.com/UNH-CORE/RVAT-Re-dep/issues/8.

I think this is just about done. I've "dumbed down" the repository such that users can simply execute run.py and get the figures they want. Anything more will require additional programming.


If maxCount = 1, it'll crash.

Hello, thanks for your great job .but I found a bug, If maxCount = 1, it'll crash.

@lxszl thanks you for pointing out the crash. But setting maxCount to 1 still make no sense. So I just clarified this moment in documentation and added NSAssert for more information, why crash was caused.

I fetch something from sqlite to show in pages,the maxCount was set automatically,it maybe 1...
Thank you.


Enable feature flag for security metrics dashboard phase one

Description
Enables the new dashboard as the default.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
TODO(replace-me)

Current dependencies on/for this PR:

master

PR #2249   👈

This comment was auto-generated by Graphite.


Dvds4e4

new episode

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/shows/designer-vs-developer/podcast/DVDS4E04-prototyping-motion.md#L50 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/shows/designer-vs-developer/podcast/DVDS4E04-prototyping-motion.md#L38 - Common typo found: 'Material Design' Should it be 'material design'?

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/shows/designer-vs-developer/podcast/DVDS4E04-prototyping-motion.md#L50 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/shows/designer-vs-developer/podcast/DVDS4E04-prototyping-motion.md#L50 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful

:tada: This has been pushed live to https://developers.google.com/web/


Import ordering xnumpy

Minor fixes to the import ordering in xnumpy.

Unconcerned with coverage drop as this was an unused import. Merging...


can not support ESC/POS "cut paper"

can not support ESC/POS "cut paper" ,but node-serialprt can support

I'm not sure what your question is but I believe that node-serialport should be able to support any command that is sent to the printer.

browser-serialport can support any command that is sent to the printer except  "cut paper"，node-serialport can support all.


Improve resource loading

Pull request
Description
This PR fixes issues found with the current URN resource loading implementation.
Changes made

Fix multiple requests for the same content.
Make player item construction API consistent (though the linter complaints this is in general bad practice).

No additional unit tests have been written (I don't really see any meaningful way to meaningfully test a resource loader delegate yet).
Checklist

[x] Your branch has been rebased onto the main branch.
[x] APIs have been properly documented (if relevant).
[x] The documentation has been updated (if relevant).
[x] New unit tests have been written (if relevant).
[x] The demo has been updated (if relevant).

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
1.0.0 (120)

tvOS
-


Leptos Islands vs Reactive Components

First off, amazing work here! I'm really impressed with what you've put together, and it's given me a lot of insight to a project I'm working on. The ECS paradigm is new to me and I'm so intrigued with the possibilities outside of game development. Essentially I'm trying to build a fully running headless Bevy App with an integrated async runtime (bevy-tokio-tasks) which provides a similar starting point to build the idea of a Data Layer, and and axum server running inside of that async runtime. My goal was to build a Web UI in place of the regular bevy window, and use Leptos as the UI framework. I like Bevy's ECS paradigm as a way to manage sever side state (I'm hoping to build highly interactive multi-user web applications that primarily depend on an authoritative server for as much state management/syncing b/w clients as possible).
I've been testing some things, but I'm running into issues getting things to compile. I'm working on moving to a workspace setup similar to how you're doing it in your example app. I noticed in the readme you have two items that seem relevant:

Only Leptos Islands are supported - I'm assuming this is because you're focused on building static sites that don't hydrate on the server. Is there a technical reason you couldn't set this up to run a hydrated leptos app that runs on the client? I'm interested in potentially making this a feature for my apps, but not currently focused on it.
The "frontend" crate is separated from the "app" crate - It looks like you may have run into similar issues as I am currently, is this your workaround? Would you be willing to provide any more insight on why you had to implement it this way? I'm working on trying to mirror this currently to see if it resolves my issue, but you definitely are further along than me and have a lot more knowledge on the topic!

Thank you again for making this cool project, I definitely want to use it for some static sites in the future!

Hey, thanks for reaching out

Yes, Cinnog is focused on generating static pages. For that use case islands make a lot of sense in my opinion. The classic example would be a blog. Do you mean have the ECS running on the server or on the client? In Bevy games both has been done, so technically it should be possible. Integrating the synchronous systems with an asynchronous web server like axum could get interesting though.
I started with https://github.com/leptos-rs/start-axum-workspace and then enabled SSG and islands features. The workspace just made it significantly easier to reason about dependencies and target platforms for each crate. One big learning was that https://github.com/NiklasEi/start-axum-workspace-islands-ssg/commit/35d4ffb3bfa58926b2e7d680bdc1b355f3250855 is required for the hydration to work.

Thank you for your help with that, I was able to get something working, not quite there yet with the functionality of bevy as the datalayer, but I have a functioning way of getting a Leptos app inside of an async tokio runtime running within the context of the Bevy App. You can see the LeptosAppPlugin in crates/bevy_leptos.
async-bevy-web
Here is a working example, originally cloned from your Cinnog example and modified to be a bit simpler.
example
If there's any feedback you can provide, that would be amazing! Otherwise feel free to close this issue, again I appreciate your help and have learned a lot from your implementation! I'll be following your work closely!


[nfs-client] Error receiving resource lock

When starting the latest image of the nfs-client-provisioner, I get
error retrieving resource lock kube-system/softmax.app-nfs: endpoints "softmax.app-nfs" is forbidden: User "system:serviceaccount:storage:nfs-client-provisioner" cannot get endpoints in the namespace "kube-system"

over and over in my logs and it doesn't work. Pretty sure it is because of:
https://github.com/kubernetes-incubator/external-storage/blob/8052cf737caaf2a701f0caad491e08b8ac21573c/lib/controller/controller.go#L651
I don't use the kube-system namespace, do you need to assume kube-system?

As far as I can tell this was actually due to a changing in the RBAC role. However now my volumes arent getting deleted. Will go over the config files again before posting an issue on it

facing same error here, v2.1.1-k8s1.10 doesn't have this problem, definitely something related to the changes in v3.0

@chz8494 please try editing your clusterrole so it has these permissions https://github.com/kubernetes-incubator/external-storage/blob/master/nfs-client/deploy/auth/clusterrole.yaml#L19

@wongma7 thanks for quick reply, adding new permission works!

Was trying to find the file / line @wongma7 referred to in his comment above, but it looks like the file has moved and the above link goes to a 404.
I'm hitting this same issue (with the deploy manifests from master), and it looks like the correct link that should work for the future (pointing to a tag instead of master) is: https://github.com/kubernetes-incubator/external-storage/blob/v5.0.1/nfs-client/deploy/auth/clusterrole.yaml#L18-L20
Specifically, add  the following to the ClusterRole:
  - apiGroups: [""]
    resources: ["endpoints"]
    verbs: ["get", "list", "watch", "create", "update", "patch"]

I'm testing this fix now. It seems like the leader-locking-nfs-client-provisioner should provide this permission, but it looks like it might not be. With the default nfs-client layout in master, I am getting the following error every 6 seconds on the provisioner pod:
E1229 21:02:56.043228       1 leaderelection.go:234] error retrieving resource lock default/fuseim.pri-nfs: endpoints "fuseim.pri-nfs" is forbidden: User "system:serviceaccount:default:nfs-client-provisioner" cannot get resource "endpoints" in API group "" in the namespace "default"

Well, in my case I figured out the problem—the RBAC manifest I was editing locally (and I thought I was applying to the cluster) was not actually being applied. I had left the old NFS RBAC manifest file from the nfs and so the nfs-client provisioner was not being updated.
I reset the cluster, built it again from scratch, and made sure the nfs provisioner's RBAC manifest was not being applied (just the one from nfs-client), and now it's all magically working, yay!

I have the same problem with the current master and the lines geerlingguy pointed to, are missing now. I've added them and it works again.
I'm confused if these lines are still necessary, my local test suggests so.


Not working on HA 0.88

After upgrading home assistant to version 0.88, the andoridtv component no longer works. Even changing the folder structure in /config/custom_components/androidtv/media_player.py does not solve the problem.

Have you got the latest androidtv.py?

I can confirm it works on HA 0.88 using the latest version of androidtv.py found here.

Same problem as Supersimo (I guess)
But here is some info from the log:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 416, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/media_player/androidtv.py", line 196, in setup_platform
    adb_server_port=config[CONF_ADB_SERVER_PORT])
  File "/config/deps/lib/python3.7/site-packages/androidtv/__init__.py", line 249, in __init__
    self.properties = self.device_info()
  File "/config/deps/lib/python3.7/site-packages/androidtv/__init__.py", line 447, in device_info
    wifimac = re.findall(WIFIMAC_REGEX_PATTERN, wifi_out)[0]
IndexError: list index out of range

I tried to update to the latest version but it does not work
`
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
SLOW_SETUP_MAX_WAIT, loop=hass.loop)
File "/usr/local/lib/python3.7/asyncio/tasks.py", line 416, in wait_for
return fut.result()
File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/androidtv/media_player.py", line 200, in setup_platform
if not atv.available:
File "/config/deps/lib/python3.7/site-packages/androidtv/init.py", line 521, in available
adb_devices = self._adb_client.devices()
File "/config/deps/lib/python3.7/site-packages/adb_messenger/command/host/init.py", line 19, in devices
result = self._execute_cmd(cmd)
File "/config/deps/lib/python3.7/site-packages/adb_messenger/command/host/init.py", line 10, in _execute_cmd
conn.send(cmd)
File "/config/deps/lib/python3.7/site-packages/adb_messenger/connection.py", line 77, in send
return self._check_status()
File "/config/deps/lib/python3.7/site-packages/adb_messenger/connection.py", line 80, in _check_status
recv = self._recv(4).decode('utf-8')
File "/config/deps/lib/python3.7/site-packages/adb_messenger/connection.py", line 56, in _recv
return self.socket.recv(length)
ConnectionResetError: [Errno 104] Connection reset by peer`

The component in this repository has been updated.
Please note, the location of where the custom component should be placed, has changed.
See the main add-on README/manual for instructions.
@Supersimo88 ConnectionResetError: [Errno 104] Connection reset by peer is an unrelated error to this issue. Drop by on our Discord, since that seems to be a network related issue.


feat: add drag-and-drop for xlsx-format

Dragging an xlsx-file onto a livebook should load the code-snipplet.
In order for the DataTable to load the code assumes that a sheet called "Sheet 1" exists and that the first row contains column-labels.
There are still some corner-cases which have to do with loading data from Excel in the general sense.
The library https://github.com/xavier/xlsx_reader from https://github.com/xavier was used in the addition.
Hopefully some users will find the addition useful.

Please find a xlsx-file attached - which can be used to test the functionality.
livebook.xlsx
Kind regards,
Fabian

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:

Oh my - i believe "mix format" needs a run?
:) Thanks for the support - please let me know if you want me to do this.
Kind regards,
Fabian

No worries, fixed on main :)


[9.0] Add support for options on subscription swap

This resolves #580 - you can now pass in parameters when swapping a subscription, such as a coupon.
I believe this would also resolve #550, but I haven't tested it.

Please remove your comment.

Done.

This is a breaking change on the swap method for people who are overriding this and is best to be sent to the master branch.


feat: add api document(swagger)

Summary
This PR added api document, using swaggo.
Key Points

[ ] This is a breaking change
[x] New or existing documentation is updated

Description
Describe what this PR does, and aims to solve in a few sentences.
Does this close any open issues?
#292

Closing for now. Feel free to re-open


Add memcached support

Due to the slowness of some cloud storage solutions, we will be adding memcached support to create a responsive caching cluster for asset gets.
This ticket will track and associate the work involved. Also opened a ticket on the aperture process.

I believe the goal here is being targeted by my WHIP-LRU effort and Mike's precanned tooling ideas.

Closing for now.  May still be a useful idea in the future.


Allow to define text blocks manually

Certain courses would like to use text exercises without natural language solutions, but rather with formal proofs or there kinds of text. One example is the course IN0003 which plans to use Artemis for the repeat exam on July 4.
In such cases, it must be possible for tutors to assess submissions without predefined text blocks and to manually select the text blocks for the feedback.

Closed in #1705.


Point to a new omnibus branch to test a PGSQL change

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Notarization problem seems solved now: https://github.com/DataDog/datadog-agent-macos-build/actions/runs/5764001766/job/15627015719


Run Static Analysis Tools

Run BEAM and clang-analyzer on secvar-related code

Static analysis tools have been integrated into a semi-automatic build system. Fixes for issues flagged by the tool have already begun to be merged in.


Tooltip never disappears after opening

Bug, feature request, or proposal:
Bug with my dependencies may be.. or @angular/animations
What is the expected behavior?
Tooltip should disappear after the hideDelay
What is the current behavior?
Tooltip NOT disappear after the hideDelay
What are the steps to reproduce?
Same of material2 official doc
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Works with this version.
Plunker template: http://plnkr.co/edit/pr4AFA06aURbMxYI77BK?p=preview
Not works with this dependencies :
What is the use-case or motivation for changing an existing behavior?
N/A
Which versions of Angular, Material, OS, browsers are affected?
`
...
"@angular/common": "4.0.0-rc.1",
"@angular/compiler": "4.0.0-rc.1",
"@angular/core": "4.0.0-rc.1",
"@angular/flex-layout": "2.0.0-rc.1",
"@angular/forms": "4.0.0-rc.1",
"@angular/http": "4.0.0-rc.1",
"@angular/material": "2.0.0-beta.2",
...
"@angular/cli": "1.0.0-rc.0",
"@angular/compiler-cli": "4.0.0-rc.1",
"typescript": "2.2.1"
...
`
Is there anything else we should know?
I post my problem here : https://gitter.im/angular/material2.
I imported @angular/animations.
But nothing change.
Is it necessary that I import it in my code or angular does it alone ?
Thanks.

Please make sure that you have included the animation package for angular core. You can find more information about implementing the animation package here https://github.com/angular/angular/blob/master/CHANGELOG.md#400-rc1-2017-02-24

Related: https://github.com/angular/material2/issues/3321

@frankspin89 thanks, that works well! it's not obvious, you also need to add BrowserAnimationsModule too… (see http://stackoverflow.com/questions/42470364/using-angular-material-2-tabs-component-with-angular-4-0-0-rc-1/42471212#42471212).

I can't import BrowserAnimationsModule like import { BrowserAnimationsModule } from '@angular/platform-browser/typings/animations';
Error:
VM686:2 GET<EMAIL_ADDRESS>404 (Not Found)
(index):246 (SystemJS) XHR error (404 Not Found) loading<EMAIL_ADDRESS>	Error: XHR error (404 Not Found) loading<EMAIL_ADDRESS>	    at XMLHttpRequest.wrapFn [as _onreadystatechange] (http://localhost:5555/node_modules/zone.js/dist/zone.js?1488980193959:1190:29) [<root>]
	    at Zone.runTask (http://localhost:5555/node_modules/zone.js/dist/zone.js?1488980193959:166:47) [<root> => <root>]
	    at XMLHttpRequest.ZoneTask.invoke (http://localhost:5555/node_modules/zone.js/dist/zone.js?1488980193959:416:38) [<root>]
	Error loading<EMAIL_ADDRESS>as "@angular/platform-browser/typings/animations" from http://localhost:5555/app/app.module.js Not expecting this error? Report it at https://github.com/mgechev/angular2-seed/issues

If BrowserAnimationsModule should be used, then it should be available via index.ts / public_api.ts ... right?

I can confirm tooltip gets stuck in IE/Edge and Safari, but works fine in Chrome and Firefox. I believe this is due to web-animations polyfill and I've reported it there. This issue actually crashes my app so I've had to blacklist IE/Edge and Safari from even loading the web-animations polyfill.
Reproduce by hovering in and out of button rapidly.

As also added to https://github.com/web-animations/web-animations-js/issues/115 I think we got the same error:

Another strange fact is that

with Angular v2 and Angular Material v2.0.0-beta.1 we had the same bug including the JavaScript error
currently with Angular v2 and Angular Material v2.0.0-beta.2 the tooltips stuck and don't disappear but the app does not crash
with Angular v4.0.1 and Angular Material v2.0.0-beta.1 the app crashes again so we can't upgrade Angular

@karlhaas: I am using angular 4 and material 2 beta 2 and it's working great (as long as BrowserAnimationsModule is registered)

Without BrowserAnimationsModule animations won't work at all, right?
@victornoel in Safai with web-animations-js?

@karlhaas ha, maybe not in this context, sorry :) I just tried with chromium

I have the same issue when using IE11 or the latest Edge browser. I include web.animation.js to support animations.
When hovering over an element that has a tooltip multiple times in quick succession, the whole app crashes and following error occurs (as already mentioned above):

I just provided a PR to fix this in web-animations-js: https://github.com/web-animations/web-animations-js/pull/118

Seems to be working fine in the latest version.
Be sure to check out the changelog for the latest release in regards to changes w/ animations.

@jelbourn still the same error with material v. 2.0.0-beta.3 in Safari. Fix is available for web-animations-js: https://github.com/web-animations/web-animations-js/pull/118

@jelbourn I can also confirm that the error still happens with the new version in IE11 and Edge. As @karlhaas already mentioned it's a problem with web-animations-js and should get fixed there.


Make about page accessible from anywhere, easily

Some first-time users feel lost on our start page. Let's show them a friendly About link/button.
Suggestion: at the bottom of the page

duplicates #1658

i'd avoid putting it at the bottom, as we don't have a footer so far and potentially infinite scrolling lists (unless you fix it there, thus occluding screen-space for one link). it should be somewhere on the right side of the topnav.
#bikeshedding

There is a mockup in issue #2282, maybe we can adapt the current about page that way (+ including the necessary gdpr related information), and remove the old about page, since it is not at all up-to-date anymore (related to #2282)


Update Composer dependencies (2023-04-01-00-10)

Visual regression test passed!
View the visual regression test report


Prevent hashes from execution in DB, resulting in errors.

try User.find_by_guid(something: :somewhere)

Hi there @ValMilkevich    ,
In an effort to cleanup this project and prioritize a bit, we're marking pull requests that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this pull request. Obviously feel free to re-open it at any time if it's the right time or this was done in error! If you do, please rebase it with the latest master and explain why it's still needed.
We really appreciate your contribution, we're just trying to make this project manageable again to move it forward.
Hope all is well.

@ValMilkevich Can you explain a little bit about what is the purpose of this PR? I guess you mean User.find_by_uid (instead of User.find_by_guid). Otherwise in 1 week, we're closing this PR


Add SAML authentication support

Identify user with SAML2 IDP

We may be able to use the Meteor Accounts SAML package.

@preriasusi We are using OAuth and Meteor accounts package for authentication. The Meteor Accounts SAML package is not maintained. Hence, we suggest that this would not be an Apika feature (at least not APIKA MVP).

nope


Use obsidian tags

Problem
Could the tags column type pull from existing obsidian tags?  Or does it? Couldn't figure out a way to get those to show up or autocomplete when I was editing entries.
Solution
A field that looks like this when you edit it, and uses the obsidian tags:

Additional details
No response

It currently does not. I will add this issue to the roadmap


Add a panel button to push to remote

Hey!
I am helping a friend with their Kirby website and I’d like to enable them (non-tech person) to get the content backed up on git. I think this plugin is perfect for that (I was doing commits once in a while from SSH), but their hosting plan doesn’t offer cron job integration. The content is a lot of images, so doing commits after each change to /content folder sounds slow. But if the cron job would simply trigger an endpoint, would it be a lot of work for me to add a button in the panel that triggers the push? So the friend could simply run it once a week / once in a month or after they are done with a bunch of changes.
Thanks!

Hey gryzzly,
maybe a cheap solution would be to use the 'push'-route, that is implemented in the plugin already.
You could create a bookmark for your friend that links to <Your-URL>/git-content/push. That should push the content every time it is called.
I am not entirely sure but I think even for a simple button with a custom link you would have to create your own plugin and add a custom section in Vue that implements this button (you may want to take a look at this cookbook recipie: https://getkirby.com/docs/cookbook/extensions/first-panel-section)
In an old version of the plugin there was a button like the one you need, but I think it got removed when the panel was switched to Vue for v3 of kirby.
Maybe it will get implemented again - should not be too hard.

Very helpful, thanks for the kind advice!
I found this plugin called "Janitor" (https://github.com/bnomei/kirby3-janitor) that allows adding a button via fields in the blueprints, and I think adding a button to do an API call to /git-content/push should work!
I’ll update the issue once it’s working

@moddyn6
Hey, so I‘m trying to call the git endpoint like this:
'bnomei.janitor.jobs' => [
'contentPush' => function (Kirby\Cms\Page $page = null, string $data = null) {
// $page => page object where the button as pressed
$kirby->call('/git-content/push?secret=foobar');
return [
'status' => 200,
'label' => $page->title() . ' ' . $data,
];
},
],
'thathoff' => [
'git-content' => [
'commit' => true,
'cronHooksEnabled' => true,
'cronHooksSecret' => 'foobar',
'displayErrors' => true
],
]
where contentPush is a function that Janitor calls from a button in the panel. But it always show a dialog saying: No route found for path: "git-content/push" and request method: "GET".
However, I can see that the route is indeed installed at http://localhost:8000/git-content/push – if I visit it I receive the appropriate error message in JSON like this:
{"status":"forbidden","message":"Invalid secret passed"}

Do you see something obviously wrong with what I am doing?
Thanks!


Unable to install the package

Hi, i was try to install this package using npm install paymob-reactnative@https://github.com/PaymobAccept/paymob-reactnative-sdk.git but it is not installing.

Resolved using yarn


"No Project" reported by TSServer.js. Autocompleting doesn't works.

From @TheAifam5 on February 16, 2017 19:4

VSCode Version: 1.9.1
OS Version: ArchLinux

I tried to restart, reload etc.. Nothings help. I can't say how to reproduce it.
Workspace:
test.zip
Logs from DevTools:
Error processing request. No Project.
Error: No Project.
    at Object.ThrowNoProject (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:6146:23)
    at IOSession.Session.getFileAndProjectWorker (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:68162:42)
    at IOSession.Session.getFileAndProjectWithoutRefreshingInferredProjects (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:68156:29)
    at IOSession.Session.getCodeFixes (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:68558:31)
    at Session.handlers.ts.createMap._a.(anonymous function) (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:67627:61)
    at IOSession.Session.executeCommand (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:68664:28)
    at IOSession.Session.onMessage (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:68684:35)
    at Interface.<anonymous> (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:69836:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:188:7)

Copied from original issue: Microsoft/vscode#20762

From @TheAifam5 on February 16, 2017 22:23
tslog.zip

Thanks for the report. I'm unable to repo this using the small example project you provided
To help us investigate this, can you please try collecting some logs from the TypeScript server when this occurs. Here's information on to do this using the TSS_LOG environment variable: https://github.com/Microsoft/TypeScript/wiki/Standalone-Server-(tsserver)#logging
From a command prompt, run something like:
$ export TSS_LOG="-level verbose -file /PATH_TO_HOME/.tslog"
$ code

Reproduce the problem in VSCode and then share the .tslog file.
Let me know if you have any questions about this.
Thanks

From @TheAifam5 on February 16, 2017 22:54
I can say that when i was starting this example, was everything ok (was fine, completion etc) but at the end when i tried to edit a webpack.config.ts i got no completion.
Found the problem. I tested everything. Clean project, everything rewrited (but without tsconfig.json) and was ok. Also i checked with hex editor. I cleaned test project and copied "config.ts", "package.json", "tsconfig.json" and "webpack.config.ts"... and after restart was like before - no autocompletions... I decided to remove "tsconfig.json" and after "Reload Window" everything was fine. Then i decided to copy again "tsconfig.json" to check if really is that a problem .. and yes! "tsconfig.json" makes that problem. Without "tsconfig.json" everything works fine.
I also checked every option which settings make that happened.
Exactly this make that bug:
"include": [
    "./src/**/*"
  ]

Also when is empty:
"include": []

But when is set to current directory, works everything fine:
"include": [ "." ]

Yea. Thats all what i found.
PS. I didnt have any "src" folder.. The source is above in the first post.
PS2. That's obviously a bug :) "include" should be checked if path exists and if "include" is not empty.

Here's the first crash message from the logs:
TypeError: Cannot read property 'enable' of undefined
    at LSHost.resolveModuleName (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:64812:73)
    at LSHost.resolveNamesWithLocalCache (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:64852:68)
    at LSHost.resolveModuleNames (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:64911:29)
    at Object.compilerHost.resolveModuleNames (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:63902:104)
    at resolveModuleNamesWorker (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:51196:93)
    at resolveModuleNamesReusingOldState (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:51303:24)
    at processImportedModules (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:52028:35)
    at findSourceFile (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:51948:17)
    at processSourceFile (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:51859:27)
    at processTypeReferenceDirective (/opt/visual-studio-code/resources/app/extensions/typescript/node_modules/typescript/lib/tsserver.js:51983:21)


Ability to pull multiple images in run step

Hi,
I'm attempting to run tests of our whole suite (our clients and servers are in separate repos).
To do this, I've created a parent repo with the other repos as submodules. I've created a pipeline and docker-compose files in the parent repo so that Buildkite can build and run the child project images.
I would like to pre-build the images but find that when it hits the run step, it only pulls the image that I'm running, and then proceeds to rebuild the other images specified in docker-compose.yml.
I had a quick look through the plugin code and found this comment:

We only look for a prebuilt image for the service being run. This means that
any other services that are dependencies that need to be built will be built
on-demand in this step, even if they were prebuilt in an earlier step.

Is this something that's likely to change?
Thanks
Ritchie

In my case, I just wanted it to pull all containers in the docker-compose before running (so that old already-pulled versions aren't used) - they're all just images, no prebuild interactions. So, I've added a pull_all flag to my fork (in case someone else is in the same position)


Missing Wheel package format !?

Hello,
I see you have wheel support in your setup.cfg, but you didn't upload any .whel package to Pypi.
Could you consider doing it ?
python setup.py sdist bdist_wheel upload -r pypi

Bump! It would be great to push a wheel for the last release (especially if a release is not planned soon).

Wheel is there at https://pypi.python.org/pypi/Jinja2 so this issue can be closed.


Create CLI's ESLint plugin package

WHY are these changes introduced?
We have a lot of specific and custom ESLint rules, and since more teams are collaborating with the CLI but in other repositories, makes sense to have a shared set of rules that all cli-kit consumers should use.
Fixes https://github.com/Shopify/internal-cli-foundations/issues/545

WHAT is this pull request doing?

Extract all our rules (custom and not) into a new npm package called @shopify/eslint-plugin-cli
Make our main .eslintrc import that extend those rules (with some custom specific overrides)

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

@gonzaloriestra @pepicrft The eslint plugin is ready to review!
@pepicrft about the naming of the package, check this comment


Handle SSL frontend and backends

https://raymii.org/s/snippets/haproxy_ssl_backends.html
https://www.haproxy.com/doc/aloha/7.0/deployment_guides/tls_layouts.html
https://www.haproxy.com/doc/aloha/7.0/haproxy/tls.html
https://serversforhackers.com/c/using-ssl-certificates-with-haproxy
@sadlil , we need to think through all the cases. noSSL field is not enough.

#408


Update diff.js maybe faster

@aimwhy This looks like how diff.js used to work, and I switched to sparse Arrays for performance. Are you getting faster diffs with this approach?

Some quick testing seems to show there is little difference, so maybe my manual tracking of min is unnecessary here. Need more testing though.

I think it reduces the number of cycles.
No experiment

Yes, it would definitely reduce the number of iterations from within removeOrphanedChildren, though my thought was that would be more than offset by not having to mutate the Object and Array shapes.  Might try to get some others to chime in.

also reduce the number of iterations for next child

5 minutes ago

@aimwhy Right, yes, in the case where the min lower bound does not get incremented because the matching child is out of order.

this is my first use github ,  and my english is poor
With the aid of translation software
Hope useful

Very useful! I enjoy discussing performance any day :)

IMHO, splice will be much slower then sparse array especially you might need to call it a lot
if you want to reduce the amount of iterations down the chain you could collapse all the nulls using .filter,  though removeOrphanedChildrendoesif(child)` which is likely to be faster than making a new array anyway

I thought so too, though filter() seems to end up being slower than just looping (🐼).  I think the current sparse implementation ends up being the fastest, but it's not something that comes out in microbenchmarks very well because it's more about memory complexity than time complexity.
The strange thing is - I ran into a case a while ago where I had assumed looping over an Array and then setting .length=0 would be the fastest way to perform a queue-flush type operation, since it resulted in only a single mutation.  However, the tests showed that while(pop()) is actually the fastest by quite a bit:
https://esbench.com/bench/56bd078245df6895002e03c0
Sometimes I find my own conception of what will be fast ends up being predicated on assumptions I've made about the JavaScript engine that are untrue, at least in V8. It's humbling 😊

Going to close this one out - I've done enough testing locally to know that the sparse Array performs better (also see benchmark above), as noted by @kruczy.  I appreciate all your hard work on this, it's been a good exercise in performance analysis/tuning over the past few months. Thanks!!

Cool, thanks alot :)


'log-ios' broken, maybe invoke 'log' instead of 'syslog'

Environment
React Native Environment Info:
System:
OS: macOS 10.14
CPU: x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
Memory: 3.49 GB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.10.0 - /usr/local/bin/node
Yarn: 1.9.4 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 12.0, macOS 10.14, tvOS 12.0, watchOS 5.0
IDEs:
Android Studio: 3.1 AI-173.4907809
Xcode: 10.0/10A255 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.0-alpha.8af6728 => 16.6.0-alpha.8af6728
react-native: 0.57.3 => 0.57.3
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-git-upgrade: 0.2.7
Description
Currently, react-native log-ios seems to be broken, it does not display console.log/warm/error outputs, which is its main purpose...
After digging into the code, log-ios logic, which is located in local-cli/logIOS/logIOS.js, is invoking
syslog -w -F std -d /Users/john/Library/Logs/CoreSimulator/3FEB5B23-BA83-4AC0-A60D-CC07412FE6CC/asl

outputting lines like this
 com.apple.CoreSimulator.SimDevice.3FEB5B23-BA83-4AC0-A60D-CC07412FE6CC[17151] (com.apple.securityuploadd) <Notice>: Service only ran for 5 seconds. Pushing respawn out by 5 seconds.

What works
log stream --predicate '(processImagePath contains "<XCode Project Name") and senderImageUUID == processImageUUID' --style json

(case sensitive)
outputting the same lines (in eventMessage prop) as the XCode debug pane
{
  "traceID" :<PHONE_NUMBER>0663300,
  "eventMessage" : "Running application \"Foobar\" with appParams: {\"rootTag\":31,\"initialProps\":{}}. __DEV__ === true, development-level warning are ON, performance optimizations are OFF",
  "eventType" : "logEvent",
  "source" : null,
  "formatString" : "%s",
  "activityIdentifier" : 0,
  "subsystem" : "",
  "category" : "",
  "threadID" : 2354892,
  "senderImageUUID" : "5563B3FF-BDAD-33A2-BA67-FB3B6A6BF913",
  "backtrace" : {
    "frames" : [
      {
        "imageOffset" : 979817,
        "imageUUID" : "5563B3FF-BDAD-33A2-BA67-FB3B6A6BF913"
      }
    ]
  },
  "processImagePath" : "\/Users\/john\/Library\/Developer\/CoreSimulator\/Devices\/3FEB5B23-BA83-4AC0-A60D-CC07412FE6CC\/data\/Containers\/Bundle\/Application\/9BC93A5D-BCD6-4C90-9D4C-1E229B8327AB\/Foorbar.app\/Foobar",
  "senderImagePath" : "\/Users\/john\/Library\/Developer\/CoreSimulator\/Devices\/3FEB5B23-BA83-4AC0-A60D-CC07412FE6CC\/data\/Containers\/Bundle\/Application\/9BC93A5D-BCD6-4C90-9D4C-1E229B8327AB\/Foobar.app\/Foobar",
  "timestamp" : "2018-10-18 17:19:50.962092+0100",
  "machTimestamp" :<PHONE_NUMBER>43698,
  "messageType" : "Default",
  "processImageUUID" : "5563B3FF-BDAD-33A2-BA67-FB3B6A6BF913",
  "processID" : 17313,
  "senderProgramCounter" : 979817,
  "parentActivityIdentifier" : 0,
  "timezoneName" : ""
}

Here is a POC to demonstrate that this approach works: react-native-log-ios
npm install --global react-native-log-ios
react-native-log-ios <XCode Project Name>

17:19:50, Running application "Foobar" with appParams: {"rootTag":31,"initialProps":{}}. __DEV__ === true, development-level warning are ON, performance optimizations are OFF

Hello there 👋 this issue has been reported for an old version of React Native. Ideally we'd like everyone to be using 0.59 (see the awesome changes it brought) but we know updating can be a pain. We are going to close this issue because it's from a version before 0.57, which is really old.
But please, if it's actually still an issue with 0.59 please comment below and we can reopen it. Even better, please send us a pull request with a fix 😊

In 0.59, we have brand new CLI and it should work great for most cases. And please file an issue https://github.com/react-native-community/react-native-cli if the issue persists.


fix(aws): s3datetime_to_timestamp parse timestamp with Z(minio.io)

Hi!
Fix parse time like: 2020-03-27T19:15:42.462Z..
This is minio timestamp format.
Error trace:
hue_1       |   File "/usr/share/hue/desktop/libs/aws/src/aws/s3/s3fs.py", line 321, in listdir_stats
hue_1       |     res.append(self._stats_key(item, self.fs))
hue_1       |   File "/usr/share/hue/desktop/libs/aws/src/aws/s3/s3fs.py", line 207, in _stats_key
hue_1       |     return S3Stat.from_key(key, is_dir=is_directory_name, fs=fs)
hue_1       |   File "/usr/share/hue/desktop/libs/aws/src/aws/s3/s3stat.py", line 90, in from_key
hue_1       |     mtime = s3datetime_to_timestamp(s3_date) if s3_date else None
hue_1       |   File "/usr/share/hue/desktop/libs/aws/src/aws/s3/__init__.py", line 140, in s3datetime_to_timestamp
hue_1       |     assert datetime[-5:] == '.000Z', 'Time [%s] is not in GMT.' % datetime
hue_1       | AssertionError: Time [2020-03-27T19:15:42.462Z] is not in GMT.

Thanks!
Just there is currently a patch with a test for this one https://review.cloudera.org/r/15012/
If urgent feel free to do similar?
FYI @quadoss

Thanks a lot!


Problem with pubDate in some feeds (GatesNotes)

(filing issue as requested on IRC)
Adding GatesNotes RSS feed shows all articles with date 01 Jan 2008. With keep-articles-days 30 it does not show any articles even though there are some articles in last 30 days.
Feed URL: https://www.gatesnotes.com/rss
Newsboat version (copy from newsboat -v):

newsboat 2.12.0
System: Linux 4.17.14-102.fc27.x86_64 (x86_64)
Compiler: g++ 7.3.1 20180712 (Red Hat 7.3.1-6)
ncurses: ncurses 6.0.20170722 (compiled with 6.0)
libcurl: libcurl/7.55.1 OpenSSL/1.1.0h zlib/1.2.11 libidn2/2.0.5 libpsl/0.18.0 (+libidn2/2.0.4) libssh2/1.8.0 nghttp2/1.31.1 (compiled with 7.55.1)
SQLite: 3.20.1 (compiled with 3.20.1)
libxml2: compiled with 2.9.8

Other info you think is relevant:
The RSS feed has the following as pubDate: <pubDate>8/21/2018 9:00:00 AM</pubDate>. curl_getdate fails to parse it and returns -1.
Maybe this should be reported to libcurl.

Thank you for the report! The feed is malformed: the spec says that pubDate should use RFC 822 format, which looks like "Mon, 27 Aug 2018 22:10:36 +0300"—clearly different from what the feed uses.
I'd report that to the site, but I can't find the link to the RSS anywhere. Is the publisher even aware they have an RSS? I'm worried that they'll just remove the feed instead of fixing it. I'm subscribed too, but I've been for ages; perhaps the link existed back then. @Stevko-sk, how did you find the link to RSS, and did you do it recently?

Our existing parser tries to gradually extract more and more specific info: first the year, then the month, then the day, and so on. It stops as soon as it hits an error, and it doesn't check if it consumed the whole output. This is very much in line with the robustness principle, but also creates problems like the one above.
I think it'd be useful to curb that robustness a bit, and use a three-step algorithm:

try the format mandated by the specification;
if the above failed to match the whole input string, try an array of common formats. These should be compiled from a wide range of feeds;
if the above failed to match the whole input string, use current date and time.

(I believe we implement step 3 already, but I don't see it in the aforementioned function. Perhaps higher up the stack?)


framework: prevent double-delete of a renderdata not owned by the camera

Before adding the undertaker thread problems like this one have to be fixed.
The post effect renderData is its own hybridObject that is taken care of by Java as any other hybridObject.
GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Planning to merge on Tuesday


code-insiders: pastingAWS::*, Alexa::*, and Custom::*. into a markdown file converted to a link

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:

1.81.0-insider
daf9647eec0867d7bc32d935478fec4bd9de1f56
x64

OS Version:

No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.2 LTS
Release:	22.04
Codename:	jammy

Steps to Reproduce:

open vscode
paste AWS::*, Alexa::*, and Custom::
it returns this [Title](AWS:%253A%252A%252C%2520Alexa%253A%253A%252A%252C%2520and%2520Custom%253A%253A)

duplicate of https://github.com/microsoft/vscode/issues/187201


Import declarations may only appear at top level of a module

I get this error:
import declarations may only appear at top level of a module
I followed suggestions. Not sure what's wrong. I use the latest version of Redux and React.js.

can you show me where the error appear?

It only appears when I add the import rule on a layout view, which is the only one I have for the app. I'm a few hours away from demo hour but I will get an app example to confirm the problem later.

okay, it seems it's just a question of some code being before an import statement, very curious, I use this thing in all my projects and never had this error,  I can try to debug it with you if you are interested

I used redux-notifications and it worked. That too, has some problems, which I can live with for now. Still I like to check to create app to confirm the problem with this later.

I'm experiencing this error. I haven't actually used the import anywhere; the error is thrown because the module is in es6 and my webpack fails to transpile it, for some reason. Can you share your webpack  and babelrc? Here are my files:
//webpack

var HtmlWebpackPlugin = require('html-webpack-plugin')
var ExtractTextPlugin = require('extract-text-webpack-plugin')
var HTMLWebpackPluginConfig = new HtmlWebpackPlugin({
  template: __dirname + '/app/index.html',
  filename: 'index.html',
  inject: 'body'
});
var path = require('path')
var webpack = require('webpack');

module.exports = {
  devtool: 'source-map',
  entry: [
    'webpack-hot-middleware/client?reload=true',
    path.join(__dirname, './app/index.js')
  ],
  module: {
    loaders: [{
      test: [/\.js$/,/\.jsx$/],
      loader: "babel-loader",
      include: [
         path.resolve(__dirname, "node_modules/flash-notification-react-redux"),
         path.resolve(__dirname, "./app")
      ],
    },{
      test: /\.(css|scss)$/,
      loader: ExtractTextPlugin.extract('css!sass')
    },{
      test: /\.(png|jpg)$/,
      loader: 'url-loader?limit=8192'
    }]
  },
  output: {
    filename: 'index.bundle.js',
    path: __dirname + '/dist',
    publicPath: '/'
  },
  plugins: [
    HTMLWebpackPluginConfig,
    new ExtractTextPlugin('public/style.css', {
      allChunks: true
    }),
    new webpack.HotModuleReplacementPlugin(),
  ]
}

//babelrc
{
  presets: [ 'es2015', 'stage-0', 'react'],
  plugins: ['transform-decorators-legacy']
}

i can open a new issue if you'd like, thanks.


Updates to pip repository ....

Hi Jonathan:
I was wondering how often updates here are submitted to the PIP repository. I think there have been updates here (I could be wrong), but notice during 'conda-updates' (the Andconda Python update tool), that they are not reflected in the PIP repository. Should I just update manually from here instead?
Thank you! =:)

ptpython-0.34 is already on pip

Hi @stonebig. I see that now that you mention it. Anaconda has up to 0.32 (currently); but you're correct, explicitly running pip install --upgrade ptpython got me to 0.34. I'll use that method then. Thank you for the reply.

you may close your issue, if you consider the problem solved.

Closed. Thank you.


feat: rosbot demo

Purpose

Clear purpose of this PR

Proposed Changes
What does this PR add, remove or fix?
Issues

Links to relevant issues

Testing

How was it tested, what were the results?

closing, demo has been updated in https://github.com/RobotecAI/rai/pull/291


fix arm xdata for oopjit

xdata registration was not correct for arm. we heap allocate xdata on arm, so we should put it in native code data buffer when we are oop

:shipit:


SamlSecurityToken protected constructor should create a new SamlAssertion.

https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/dev/src/Microsoft.IdentityModel.Tokens.Saml/Saml/SamlSecurityToken.cs#L41
With this constructor, SamlSecurityToken.Assertion can be null. This is a difference between the Saml and Saml2 SecurityTokens. This constructor is not very helpful as the Assertion property can not be set.
Have the constructor set a new SamlAssertion.

@brentschmaltz Having the empty constructor for SamlSecurityToken set the Assertion property would require us to use the following constructor, as SamlAssertion does not have an empty constructor.
public SamlAssertion(             string assertionId,             string issuer,             DateTime issueInstant,             SamlConditions samlConditions,             SamlAdvice samlAdvice,             IEnumerable<SamlStatement> samlStatements             )
As DateTime cannot be null, what should be passed in? Or does it make sense to add an empty protected constructor for SamlAssertion as well?

I wonder how anyone could have used this constructor? I really would like to remove it, but back compat blocks us. These parameters can not be null or empty { assertionId, issuer, samlStatements } and code may depend on that. So defaults would be needed also for assertionId, issuer.
To answer your question: UTC.now for issueInstance.

@brentschmaltz I agree; it's unfortunate that we can't remove it.
I'm not sure what the appropriate default value for issuer would be... it is a settable property so a user will be able to change it if necessary, but I'm not sure if there's any default value that would necessarily make sense here.

Perhaps ClaimsIdentity.DefaultIssuer would be a choice.


⚡ Quick fixes Thursday 7 Jul 2022

A series of quick fixes to clear the backlog:

[x] Homepage content changes: added Elastic CI bootstrap link, test collector link and minor copy changes fe51a8040bbc0e3c2085d7464f4802ef4dc3e699
[x] DOC-431 pre-expand menu on TA: removes one click for customers to see what's under each menu heading
[x] DOC-437 no doc heading '#' on hover in note: fixed the bug @karensawrey found when you hover over this message the ::before ⚠ icon gets replaced with a # anchor
[x] DOC-439 shortcut contribute directly to edit: @JuanitoFatas's proposed change to shortcut to the edit view to make a contribution
[x] DOC-439 remove unused method: removed a dupe method that's not being used

I've manually checked the above in Chrome. Let me know if you're good with the above or need further fixes.

Homepage content changes

Thanks for the meticulous reviews 🙌
Option 1: "Customizing instances…" (noun noun noun…)
So the Customize/ing instances with a bootstrap script has another small niggle which is script gets orphaned.

Option 2: The current state "Customize instances…"
That's one reason I went with a shorter sentence, so I could fit everything into one line

Option 3: Maybe we could try to make the sentence longer?
If we want to do 'noun noun noun' then I could fudge it with "Customizing your instances…" so we don't get an orphan:

It won't fit into one line but I think it's ok. The sentence reads a bit nicer here.
Thoughts?

Yeah, I see how the orphan is a bit of a problem. A couple more options:
Option 4: Aggressively shortening (emphasis on instance)
Trim the "bootstrap script" as an implementation detail

Option 5: Aggressively shortening (emphasis on script)
Emphasize scripting instead of customization

Thanks for the reviews 🙌. Let me know if the changes are good or if more changes are needed.
As we're close to Friday, I'll hold off merging till Monday.


Inserting values at pointers

Currently trying out the new JSON pointer implementation and I think this would be very useful in many cases.
One thing I have needed is inserting or assigning arbitrary JSON values at locations specified by JSON pointers. Is it currently possible? How can I achieve it in the current state?

Please describe what you consider "JSON pointers". Do you mean some XPath like query syntax? Specifically, what syntax? Is there a standard for it?

Not a duplicate. That issue is about finding an element by its path, and this one is about inserting an element at a specified path.

Just to be sure, this is about creating intermediary objects, right? Do you also have any suggestions on how to deal with non-existant elements of arrays?

Yes, non-existent objects are to be created at the insertion. For example,
value jv1 {{"a": {{"b", 1}}}};
value jv2 {{"c": {{"d", 2}}}};

jv1.insert_or_assign("/x", jv2);
jv1.insert_or_assign("/a/b", jv2);

assert(jv1.at_pointer"/x/c/d" == 2)     // new value inserted at previously non-existent path /x
assert(jv1.at_pointer("/a/b/c/d") == 2) // new value assigned at the existing path /a/b

For non-existent array elements, I think std::vector assignment behavior might be preferable. Assign the value if the index is in range or else return an out-of-range/not-found error. Inserting to the back of an array can be specified using - character according to RFC 6901. A helper function to get the size of a json::array at a given pointer path may be needed as well.

Further investigations with the current API:
For objects:
value jv = {{"abc", {{"xyz", {12, 13, 15}}}}};

jv.at_pointer("/abc/xyz") = {"123", "1231"}; // works as desired
jv.at_pointer("/abc/klm") = {"123", "1231"}; // does not work but desired

For arrays:
value jv = {{"abc", {{"xyz", {12, 13, 15}}}}};

jv.at_pointer("/abc/xyz/0") = {"123", "1231"}; // works as desired
jv.at_pointer("/abc/xyz/3") = {"123", "1231"}; // does not work as desired
jv.at_pointer("/abc/xyz/-") = {"123", "1231"}; // does not work but desired

We're having differences of opinion on what jv.insert_at_pointer("/a/2", 5) needs to create, an array or an object. @grisumbras thinks it should create an object and set its "2" key to 5. I find this consistent but kind of not really doing what one would want in the majority of cases. It seems to me that it should create an array and set its third element to 5 (leaving the first two null).

There's no use case for (1) if you don't have (2)+(3).

(1) is useful if one has (3) and wants to maintain order independence as in
value jv = array();
jv.assign_at_pointer("/1/y", 5);
jv.assign_at_pointer("/0/x", 3);

giving the same result as
jv.assign_at_pointer("/0/x", 3);
jv.assign_at_pointer("/1/y", 5);

because there's an implicit null created by the first line.
On second thought, though, I no longer favor (3) because it makes denial of service attacks trivial ("/123456789"). So I agree that numeric indices should be constrained to [0, size()] (with - being the same as size().)

RapidJSON implements the behavior (3). The program below is problematic as @pdimov points.
#include <iostream>

#include "rapidjson/document.h"
#include "rapidjson/pointer.h"

using namespace rapidjson;

int main()
{
  const char* json = R"({"my_obj": { "my_arr": [10,11,12]}})";
  Document d;
  d.Parse(json);

  Document v;
  v.Parse("20");
  SetValueByPointer(d, "/my_obj/my_arr/999999999", v);

  return 0;
}

Interesting. So RapidJSON says yes to 1, 2, and 3. And even converts other kinds of values to arrays and objects, not just nulls. Maybe we need 2 kinds of functions: one conservative and one "helpful".

@doganulus @pdimov can you check out #713 to see if you agree with the choices made in that implementation.
The choices are:

Nulls are replaced when creating intermediate elements.
When creating intermediate elements segments are always treated as object keys, (in other words, only objects are created as intermediates).
Arrays are extended by 1 element at most.

One question: Can we set values at intermediate keys as follows?
jv = value();
result = &jv.set_at_pointer("/a/b/c", "m"); //
result = &jv.set_at_pointer("/a/b", "n");

Or the option replace_any_scalar just implies that we can only replace the leaf values?

value jv = 1;
jv.set_at_pointer("/a/b/c", "m"); // exception
jv.set_at_pointer("/a/b/c", "m", {.replace_any_scalar}); // success

value jv = 1;
jv.set_at_pointer("/a/b/c", "m"); // exception
jv.set_at_pointer("/a/b/c", "m", {.replace_any_scalar}); // success


HHH-9840 Allow custom key implementations for 2nd level cache regions

This is a follow up to https://github.com/hibernate/hibernate-orm/pull/974
Last commit replaced all CacheKey with Object. Because of statistics, added unwrap operation to all AccessStrategies - this is to be reviewed yet, since I had to modify SessionFactoryImplementor interface.

Wrt type safety: can I use Java 8 type annotations to do the compile type check?

@sebersole @Sanne @alexsnaps @galderz Any feedback?

I can't look at it today, will try to make some time tonight or tomorrow at
latest
On 11 Jun 2015 08:20, "Radim Vansa"<EMAIL_ADDRESS>wrote:

@sebersole https://github.com/sebersole @Sanne
https://github.com/Sanne @alexsnaps https://github.com/alexsnaps
@galderz https://github.com/galderz Any feedback?
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/976#issuecomment-111023553
.

Re: Wrt type safety: can I use Java 8 type annotations to do the compile type check?
I don't know, @sebersole would be able to say.
One alternative option for additions to EntityRegionAccessStrategy...etc could be:
public <T> T generateCacheKey(Object id, EntityPersister persister, SessionFactoryImplementor factory, String tenantIdentifier);
public <T> T getCacheKeyId(Object cacheKey);

By doing that, if the caller knows what the generated cache key or generated cache key is exactly, it doesn't need to do any casting. If it doesn't know, it can simply assign to object. As far as 2LC implementations is concerned, it would not change anything, we still treat keys as opaque, but maybe for the rest of Hibernate code base it helps?
Otherwise, the rest looks fine to me.

Hi Galder,
the caller (Hibernate ORM internals) doesn't use those objects so the type is not relevant for the caller.
Also, that signature would imply that the 2lc implementation needs to know what kind of key it's expected to create, while the goal is to allow implementors to have more flexibility (i.e. return whatever they see most fit from a performance perspective).
So we'll stick simply with 'Object' as it seems it's a non-problem :)

@Sanne Fine by me :)

I have rebased the PR and added commit with checkstyle fixes (can be squashed too).

merged! thanks a lot @rvansa


Ensure line feed in Makefile

Is it a good practice/causing issues?

Yup. eg when you cat Makefile, the terminal prompt then stays on the next line if you don't have smarter shell like zsh. There's basically line break missing. Printing multiple files without line feed at the end makes their last and first lines to join.

my IDE adds this automatically on save.


Issue on client query code generator

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
under some circustance the code generated by StrawberryShake generate classes that do not compile due to a
Compiler Error CS0542 - 'user-defined type' : member names cannot be the same as their enclosing type
this happens when in a query definition is present a type with a property named as "[TypeName]Data"
the code generator Generate a class for each type named "[TypeName]Data" so any property in that type that has that name produce the compiler erroe CS0542
Steps to reproduce

write a simple query selecting a type that has a property named the same as the typeitself + "Data"
Example Schema Type:

type Test {
TestData: String
}
Relevant log output
Compiler Error CS0542 - 'user-defined type' : member names cannot be the same as their enclosing type

Additional Context?
I'm not familiar on what library is doing the code generator, but i found this bugs including in my .net core 7 project the preview package StrawberryShake.Server/13.0.0-preview.91 with his dependency
Product
Strawberry Shake
Version
13.0.0-preview.91

Suggestioni on fix

Add a constrain during properties generation

Use Graphql alias defined in query as property name


rmul for sparse matrixes

I implemented rmul method for sparse matrixes.
Merge #239 first.
related to #36

LGTM! It will be merged after #239.


[meson] omit load extension when using MacOS

description
this PR makes checks if there is a bundled sqlite3 in OSX that does not support load extension and disables the support automatically

note: line feeds were changed from CRLF to LF

changes

meson_options.txt

changed SQLITE_OMMIT_LOAD_EXTENSION to SQLITE_OMIT_LOAD_EXTENSION (typo fix)

meson. Build

change the version to 3.2.1
changed SQLITECPP_USE_STACK_PROTECTOR to SQLITECPP_USE_STACK_PROTECTION (typo fix)
adds the SQLITE_OMIT_LOAD_EXTENSION option and enables it if running on MacOS with a bundled sqlite that does not supports it

additional notes
on wrapdb this can be added as a patch as is only a change in the meson files and not the project itself

note: related to mesonbuild/wrapdb#816

currently doing a code review for this update on wrapdb repo

Hello @ninjaoflight
Thank you again for all the work you are doing in this area!
Could you write a quick start on how to use meson? I think it's worth adding to a section in the readme, after the chapter on CMake.
Then I should add a new Github Actions workflow to test meson build automatically


[BUG] Deployment on Red Hat OpenShift

Description
I have tried to install the docker image in OpenShift, but it gives an error when building the image.
Reproduction steps
1. Go to workloads
2. Add container image
3. When installing it gives an error

Screenshots
![DESCRIPTION](LINK.png)

Logs
Traceback (most recent call last):
File "/app/app.py", line 3, in <module>
from theflow.settings import settings as flowsettings
File "/usr/local/lib/python3.10/site-packages/theflow/__init__.py", line 1, in <module>
from .base import Function, Node, Param, SessionFunction, unset
File "/usr/local/lib/python3.10/site-packages/theflow/base.py", line 30, in <module>
from .config import Config, ConfigProperty, DefaultConfig
File "/usr/local/lib/python3.10/site-packages/theflow/config.py", line 16, in <module>
class DefaultConfig:
File "/usr/local/lib/python3.10/site-packages/theflow/config.py", line 36, in DefaultConfig
default_backend = settings.BASE_BACKEND
File "/usr/local/lib/python3.10/site-packages/theflow/settings/__init__.py", line 64, in __getattr__
self.load_settings()
File "/usr/local/lib/python3.10/site-packages/theflow/settings/__init__.py", line 51, in load_settings
spec.loader.exec_module(module)
File "/app/flowsettings.py", line 34, in <module>
KH_APP_DATA_DIR.mkdir(parents=True, exist_ok=True)
File "/usr/local/lib/python3.10/pathlib.py", line 1175, in mkdir
self._accessor.mkdir(self, mode)
PermissionError: [Errno 13] Permission denied: '/app/ktem_app_data'

Browsers
No response
OS
No response
Additional information
No response

I found the cause of the error. OpenShift uses a non root user to run the docker/pod, so we need to modify the dockerfile to give permissions to the user group that is used to access the folders.

I found the cause of the error. OpenShift uses a non root user to run the docker/pod, so we need to modify the dockerfile to give permissions to the user group that is used to access the folders.

if you make any progress on a rootless container deployment it would be interesting to see the Dockerfile. I'm working on the same problem here.

I will post it on Monday.
El sáb, 2 nov 2024, 15:03, scheckley @.***> escribió:

I found the cause of the error. OpenShift uses a non root user to run the
docker/pod, so we need to modify the dockerfile to give permissions to the
user group that is used to access the folders.
if you make any progress on a rootless container deployment it would be
interesting to see the Dockerfile. I'm working on the same problem here.
—
Reply to this email directly, view it on GitHub
https://github.com/Cinnamon/kotaemon/issues/418#issuecomment-2453000142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG6XKXTT5AHQ5FXIBFCUS3Z6TLT3AVCNFSM6AAAAABQKIRWNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGAYDAMJUGI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

thanks :)
I ended up with this, which is a bit nasty.
FROM python:3.10-slim as base_image

# Use Python virtual environment for dependencies to avoid system-wide installs
ENV VENV_PATH=/app/venv
RUN python3 -m venv $VENV_PATH

# Set up PATH for the virtual environment
ENV PATH="$VENV_PATH/bin:$PATH"

# Common dependencies with non-root considerations
RUN apt-get update -qqy && \
    apt-get install -y --no-install-recommends \
    ssh \
    git \
    gcc \
    g++ \
    poppler-utils \
    libpoppler-dev \
    unzip \
    curl \
    cargo \
    tesseract-ocr \
    tesseract-ocr-jpn \
    libsm6 \
    libxext6 \
    libreoffice \
    ffmpeg \
    libmagic-dev

# Set environment variables
ENV PYTHONDONTWRITEBYTECODE=1
ENV PYTHONUNBUFFERED=1
ENV PYTHONIOENCODING=UTF-8
ENV TARGETARCH=${TARGETARCH}

# Create working directory with correct permissions
WORKDIR /app

RUN chmod -R 755 /app

# Set up NLTK data directory for cache
ENV NLTK_DATA=/app/nltk_data
RUN mkdir -p /app/nltk_data && chmod -R 775 /app/nltk_data

FROM base_image as dev

# Download pdfjs
COPY scripts/download_pdfjs.sh /app/scripts/download_pdfjs.sh
RUN chmod +x /app/scripts/download_pdfjs.sh
ENV PDFJS_PREBUILT_DIR="/app/libs/ktem/ktem/assets/prebuilt/pdfjs-dist"
RUN bash scripts/download_pdfjs.sh $PDFJS_PREBUILT_DIR

# Copy contents
COPY . /app
COPY .env.example /app/.env

RUN pip install --upgrade pip

# Install pip packages
RUN pip install --no-cache-dir wheel && \
    pip install --no-cache-dir -e "libs/kotaemon" && \
    pip install --no-cache-dir graphrag nano-graphrag future python-decouple theflow==0.8.6 && \
    pip install --no-cache-dir -e "libs/ktem" && \
    pip install --no-cache-dir "pdfservices-sdk@git+https://github.com/niallcm/pdfservices-python-sdk.git@bump-and-unfreeze-requirements"

# Install torch and additional packages
RUN pip install --no-cache-dir torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu && \
    pip install --no-cache-dir -e "libs/kotaemon[adv]" && \
    pip install --no-cache-dir unstructured[all-docs]

# Download NLTK packages explicitly
RUN pip install --no-cache-dir nltk && \
    python -c "import nltk; nltk.download('punkt', download_dir=nltk.data.path[0]); nltk.download('averaged_perceptron_tagger', download_dir=nltk.data.path[0])"

# Verify theflow installation
RUN pip freeze | grep theflow && \
    python -c "import theflow; print(theflow.__file__)"

RUN chmod -R 775 /app

RUN pip uninstall -y hnswlib
RUN pip uninstall -y chroma-hnswlib
RUN pip install --no-cache-dir chroma-hnswlib

# Expose the apps default port
EXPOSE 7860

# Let OpenShift automatically assign a random user
USER 1001

CMD ["python", "app.py", "--host", "<IP_ADDRESS>", "--port", "7860"]

I had a problem with hnswlib which I think stems from having multiple versions installed during the build.
app.py I edited to point to <IP_ADDRESS>:
import os

from theflow.settings import settings as flowsettings

KH_APP_DATA_DIR = getattr(flowsettings, "KH_APP_DATA_DIR", ".")
GRADIO_TEMP_DIR = os.getenv("GRADIO_TEMP_DIR", None)
# override GRADIO_TEMP_DIR if it's not set
if GRADIO_TEMP_DIR is None:
    GRADIO_TEMP_DIR = os.path.join(KH_APP_DATA_DIR, "gradio_tmp")
    os.environ["GRADIO_TEMP_DIR"] = GRADIO_TEMP_DIR

from ktem.main import App  # noqa

app = App()
demo = app.make()
demo.queue().launch(
    favicon_path=app._favicon,
    inbrowser=True,
    allowed_paths=[
        "libs/ktem/ktem/assets",
        GRADIO_TEMP_DIR,
    ],
    server_name="<IP_ADDRESS>",
)

This has deployed on an on-premise OpenShift cluster without any admin privileges.

this is mine
Lite version
FROM python:3.10-slim AS lite
---------------------------------------------------------------------------
Common dependencies
RUN apt-get update -qqy &&
apt-get install -y --no-install-recommends
ssh
git
gcc
g++
poppler-utils
libpoppler-dev
unzip
curl
cargo
Set environment variables
ENV PYTHONDONTWRITEBYTECODE=1
ENV PYTHONUNBUFFERED=1
ENV PYTHONIOENCODING=UTF-8
Create working directory
WORKDIR /app
Adjust permissions for OpenShift's random user ID
RUN mkdir -p /app/libs &&
mkdir -p /app/scripts &&
chmod -R g+rwX /app &&
chown -R 1001:0 /app
Download pdfjs
COPY scripts/download_pdfjs.sh /app/scripts/download_pdfjs.sh
RUN chmod +x /app/scripts/download_pdfjs.sh
ENV PDFJS_PREBUILT_DIR="/app/libs/ktem/ktem/assets/prebuilt/pdfjs-dist"
RUN bash scripts/download_pdfjs.sh $PDFJS_PREBUILT_DIR
Copy contents
COPY . /app
Adjust permissions after copying files
RUN chmod -R g+rwX /app && chown -R 1001:0 /app
Install pip packages
RUN --mount=type=ssh
--mount=type=cache,target=/root/.cache/pip
pip install -e "libs/kotaemon"
&& pip install -e "libs/ktem"
&& pip install graphrag future
&& pip install "pdfservices-sdk@git+https://github.com/niallcm/pdfservices-python-sdk.git@bump-and-unfreeze-requirements"
Clean up
RUN apt-get autoremove
&& apt-get clean
&& rm -rf /var/lib/apt/lists/*
&& rm -rf ~/.cache
Set permissions for Python packages installed in /usr/local/lib
RUN chmod -R g+rwX /usr/local/lib/python3.10/site-packages/
CMD ["python", "app.py"]
Full version
FROM lite AS full
Additional dependencies for full version
RUN apt-get update -qqy &&
apt-get install -y --no-install-recommends
tesseract-ocr
tesseract-ocr-jpn
libsm6
libxext6
libreoffice
ffmpeg
libmagic-dev
Install torch and torchvision for unstructured
RUN --mount=type=ssh
--mount=type=cache,target=/root/.cache/pip
pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu
Copy contents
COPY . /app
Create required directories and set environment variables
RUN mkdir -p /app/nltk_data && chmod -R g+rwX /app/nltk_data
ENV NLTK_DATA=/app/nltk_data
RUN mkdir -p /app/matplotlib && chmod -R g+rwX /app/matplotlib
ENV MPLCONFIGDIR=/app/matplotlib
RUN mkdir -p /app/fontconfig && chmod -R g+rwX /app/fontconfig
ENV XDG_CACHE_HOME=/app/fontconfig
Install additional pip packages
RUN --mount=type=ssh
--mount=type=cache,target=/root/.cache/pip
pip install -e "libs/kotaemon[adv]"
&& pip install unstructured[all-docs]
Clean up
RUN apt-get autoremove
&& apt-get clean
&& rm -rf /var/lib/apt/lists/*
&& rm -rf ~/.cache
Download nltk packages as required for unstructured
RUN python -c "from unstructured.nlp.tokenize import _download_nltk_packages_if_not_present; _download_nltk_packages_if_not_present()"
run the gradio app
CMD ["python", "app.py"]


Hitting a non-existent route results in infinite redirect. Same app behaves correctly locally and on vercel.

I have a nextjs app on v14, it uses page routing.
when I enter a non-existent route (/foobar) - the app 308 redirects to /404/ few dozen times, and then browser cancels the request.
Same app works fine when ran locally with npm run dev or on vercel.
Here is the link to a version deployed with sst: https://d2ia2gbaq4gtdc.cloudfront.net/
Working version on vercel: tripyacht.com
This is the only thing thats stopping me from fully embracing open next, and migrating away from Vercel.

Hi can you specify the open-next version and exact next version?
Can you also set up a min repo?
According to the latest versions
pages: https://dqijpnhzoveol.cloudfront.net/foo
app:  https://d1rjbhcyvwj9a2.cloudfront.net/foo
They both redirect to 404 and does not have an infinite redirect.

Thanks!
Next.js v14.2.1
OpenNext v2.3.7
Sorry, not sure what is "min repo", could you please elaborate?

A minimum reproduction project on github so we can see what's causing the issue.

@ilyastam can you try upgrading next to 14.2.2, according to the test, it passes: https://discord.com/channels/983865673656705025/1027265626085019769/1230685589926576309
But there might be some edge case in your setup that's causing the infinite loop that we haven't covered. We'll need a reproduction.

I'm going to close this old issue.
If that's still an issue, reopen and provide a reproduction


Add simple notebook comparing the results of ExactGP and viGP

I've created a simple example for people to compare the results of an ExactGP and a viGP. Nothing too fancy here.

Codecov Report

Merging #50 (53b126e) into main (a8e46a0) will increase coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   95.98%   96.01%   +0.02%
==========================================
  Files          44       44
  Lines        3541     3541
==========================================
+ Hits         3399     3400       +1
+ Misses        142      141       -1

Flag
Coverage Δ

unittests
96.01% <ø> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks! I feel it could benefit a bit from a discussion on when one should use a fully Bayesian (HMC/NUTS) approach vs variational inference approximation. In my experience,
HMC/NUTS:
Sensitivity to Priors: This can be perceived as a strength or a weakness, depending on the context. However, many researchers appreciate it because it offers a more intuitive grasp of the model.
Reliable Uncertainty Estimates: Offers robust evaluations of uncertainties as it directly samples from the posterior. The variational methods are known to lead to underestimation of uncertainties.
Integration with Classical Bayesian Models: This is particularly evident when you consider the combination of Gaussian Processes with traditional Bayesian models, as demonstrated in structured GP and hypothesis learning.
Comprehensive Convergence Diagnostics: Indicators such as n_eff, r_hat, and acc_prob for each inferred parameter.
Speed Limitations: One of the primary drawbacks is its computational speed.
SVI:
Efficiency: It's significantly faster and is memory-efficient (performs equally well with 32-bit precision)
Acceptable Trade-offs: For many real-world tasks, the slight decrease in the accuracy of predictive uncertainty estimates is either negligible or acceptable.
Convergence Indicator Limitations: The loss may not be a very good indicator of convergence - can easily overshoot or undershoot.

Would you like me to add this to the notebook or perhaps it belongs in a README/docs somewhere?

I would add it to the notebook for now. I plan to add some general advice to the README later on or maybe make a separate tutorial.

Ok will do. I'll add your text to the notebook a little later and will push the changes. I can also use this PR to run the smoke tests on all of the notebooks. Stay tuned.

Sounds good - looking forward to it


Open/Save Dialogs Leave TK Window open

If you save/open a dashboard the TK window stays open and no way to do anything with it.

This seems to have resolved itself after a reboot, going to mark as closed.


AP-5524: status 404 for page not found

What
Change response for page not found to 404
Link to story
These were previously redirecting/302 and then rendering
a page successfully/200.
In order to capture 404s using prometheus custom rules
we need respond with that status. This could help to
identify attacks or other issues.
Discussion
This mechanism returns 404 for the page_not_found page response. The actual cause of the redirect and therefore the 404 is lost however. A scripted attack, using curl for example, to probe known paths will not follow the redirect response necessarily and therefore not 404 at all.
This, for example, does not generate any 404s in OpenSearch logs, as each just gets a 302
for i in {2..10}; do curl -I https://ap-5524-status-404-for-page-no-applyforlegalaid-uat.cloud-platform.service.justice.gov.uk/nada; done

but using the browser to go to https://ap-5524-status-404-for-page-no-applyforlegalaid-uat.cloud-platform.service.justice.gov.uk/nada will follow the redirect and result in a 404.
So, it might be worth spiking changing the entire way we we handle 404s, possibly to use a more rails based approach?
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
The standards in the Git Workflow document on Confluence should be followed
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

superceded


Suppress non-critical errors on opening file (e.g E325)

Is E325 about the swap file?

I don't wish to turn the swap file off - I want to get the warning that the file is opened in another session or recovered from a crash (or instance shutdown) and to get the opportunity to open in read-only mode instead.
The error still occurs but it is presented by neovim rather than bubbled up to neo-tree to cause an error. We could check the status of the pcall and prevent the file opened event - but in the case of E325 the file is still opened. I'm not sure why that is an error but it seems like it requires every caller of this api to have a long list of valid and not-valid error codes to handle it gracefully.

The error still occurs but it is presented by neovim rather than bubbled up to neo-tree to cause an error.

OK, in that case you'll just need to figure out how to positively identify that you are in this specific situation and not some other valid error relating to opening a file.

I'm not sure how many more of these errors there are. Also selecting 'abort' on the prompt also throws e325 even though the file is not opened.

Current implementation is broken as it presents an error and doesn't process the file open even
Handling the specific e325 error is broken as it may process the file opened event even if it doesn't

Perhaps the file opened event should be driven by an autocmd to know when a file was really openend.
fyi nvm.tree had a similar bug and fix : https://github.com/kyazdani42/nvim-tree.lua/issues/880
I do think this is probably an neovim bug. There shouldn't be an error if the user presses 'Edit anyway' or 'Open readonly'. Or at the very least there should be different error codes depending on what they select.

I've added handling for E325 and a 'print' of the error, but I'm not sure either is the correct thing to do.
Looking at the list of errors for neovim there are a number of 'harmless' errors - I doubt there is a 'did_this_error_block_the_file_open()' api call that can be used.
I'm also not sure triggering an 'error' or even a 'print' is useful either. e.g in the case of a selecting 'abort' on the swap file prompt. It's not an error from neo-tree - just something the user chose to do.
I think the correct solution is to pass the result, err to the file_open event as parameters.

I created a new PR as I wasn't sure how to rework this one against a different branch.
I am still convinced that neo-tree should pass the result of the call through to the event handler rather than it trying to handle every perceivable 'harmless' error that neovim may generate. Or it should call a different handler such as 'file_opened_with_error' or something like that.

I am still convinced that neo-tree should pass the result of the call through to the event handler rather than it trying to handle every perceivable 'harmless' error that neovim may generate. Or it should call a different handler such as 'file_opened_with_error' or something like that.

It seems unusual to have a "harmless" error. Are there other errors that could happen where the file would actually be opened correctly?

I agree it is unusual - but as I mentioned above vim doesn't really have a way of indicating 'command succeed but with caveats' other than having an error. (although I would think a warning would be more appropriate). The only example I can see that might occur on open is E303, but this is far from an exhaustive search. There's a number that seem 'harmless' on close however.
I guess if we are now logging the error instead of "throwing" an error it will at least not interrupt the user. They'll just miss out on that file open event.