GDScript can't be singleton

Godot version:
3.1
OS/device including version:
macOS, runing inside of Godot
Issue description:
Singleton class don't be instance/create when run app.
But It create a Node with script, binding the node file with singleton. It works.
Here is the tip: https://docs.godotengine.org/en/3.1/getting_started/step_by_step/singletons_autoload.html?highlight=singleton
Steps to reproduce:

create a gdscript file, extends Node
add the class to "AutoLoad"
run game
stright typing the class name or get_node all don't get the object.

Minimal reproduction project:

What are your settings on the Autoload tab?

@MuffinManKen  here the snap

Are you maybe using it in a tool script? Does it also happen in 3.1.1?

@KoBeWi What's mean "tool script" ? I use whitch version from official website statble ver3.1

Well, if you don't know what is a tool script, then I doubt that's the problem xd
Another possibility is that your script has some error, so it can't compile, so it can't be recognized as singleton.
A reproduction project would be helpful, because it works for me (I even tested with the same name/path).

@KoBeWi
Originally I will create a simple project for this bug. But I found that all is works well either new project or my game.
Maybe it cause by app some where crashed. Sometimes I run my code with strange error. But I can't found wrong code. Then I restart the app and it works well.

Now I know that the Singleton gdscript forbid add "class_name" keyword

Ok, first thing: it doesn't make much sense to make a singleton with class_name. Well, it's practical if you want to make more instances of the singleton (then it's not singleton) or use it as argument somewhere and need a type name (you don't do that with singletons).
Second, if for whatever reason you want it, it should be possible, but make sure the singleton name is different than class name.
On Godot's side, this should probably result in some warning or error. (btw, I still can't reproduce your exact problem)

@KoBeWi Thanks


Indented code blocks weird behavior

I have noticed indented code blocks weird behavior. I think this started after the new obsidian update (v1.5.11, 19.03.2024).
Obsidian - v 1.5.11
CodeblockCustomizer - v 1.2.6

Editing (source mode)

Editing

Reading

Hmm. Interesting. The 4th picture is correct in editing mode. Can you send me the readme file?

Do you mean .txt file with this markdown text? I am a bit newbie)
test.txt

Hi @Aratopn, sorry for the delay, but I finally had time to look at this problem. I looked at the document you sent. Only the second code block is not working correctly.
So, the problem is, that if you simply create a code block and then push it right with TAB, then Obsidian actually "converts" it to inline code:

I probably could make this work, so that background color, and other colors are applied correctly, but the problem is, that syntax highlighting is also not applied. Even if I did all that, this code block would still not work as a normal code block does.
I see this as an Obsidian issue, but hopefully they will fix it soon.
However, if you want to indent a code block, you can do that, by adding it to a list. But I see from your document, that you already figured that out.


Using gitmirror with a github organization level hook

We've been using this project for some time now, it works perfect for our needs. Thanks :)
The way we use it is by configuring a github hook on our repos of interest so it sends a notification that is processed by a server running gitmirror. We found recently that we could set the github hook at our github organization level but we faced some issues with the gitmirror implementation.
If I understand this correctly, the path in the URL is used to create the local directory where to clone the repository. For instance,
curl http://localhost:8124/gitmirror.git

Will create a folder gitmirror.git or use it if it was previously created.
I think that is reflected on this line https://github.com/dustin/gitmirror/blob/master/gitmirror.go#L229
That would not work with an organization level hook, as it will use the same path for all the repos.
Is there a way (or would it be possible) to use the name of the repo obtained in the hook event payload to name the folder?

I think it could work with a flag to change it from using the path to resolve to using the repo name.  That wouldn't be a good idea in general, but it does make sense in your case.
All of the resolution is in the getPath function, though it might make more sense to modify handlePost to parse the payload and resolve the path from there instead if the flag is set.


Inexact numeric jacobian causes test failures

Using TensorFlow 2.4 with GPUs
Describe the current behavior
//tensorflow/python/keras/integration_test:gradients_test fails when run on a node with GPUs. I traced the problem to inexact computation in _compute_numeric_jacobian
Output is:
FAIL: testLSTMBatchJacobian (__main__.GradientsTest)
GradientsTest.testLSTMBatchJacobian
...
AssertionError:
Not equal to tolerance rtol=0.01, atol=1e-06
Mismatched value: a is different from b.
not close where = (array([0]), array([0]), array([2]))
not close lhs = [0.00074506]
not close rhs = [0.00076706]
not close dif = [2.20043e-05]
not close tol = [8.670623e-06]
dtype = float32, shape = (1, 1, 6)
Mismatch: 16.7%
Max absolute difference: 2.20043e-05
Max relative difference: 0.02868646
 x: array([[[-0.013396,  0.007078,  0.000745, -0.02031 ,  0.010461,
         -0.004366]]], dtype=float32)
 y: array([[[-0.0134  ,  0.007075,  0.000767, -0.020303,  0.010446,
         -0.004386]]], dtype=float32)

The numeric result is array([-0.013396,  0.007078,  0.000745, -0.02031 ,  0.010461, -0.004366], dtype=float32)
while the eager_result, function_result and backprop_result all are y: array([-0.0134  ,  0.007075,  0.000767, -0.020303,  0.010446, -0.004386], dtype=float32)
Standalone code to reproduce the issue
Reduced extracted test code:
import numpy as np
import tensorflow as tf

class GradientsTest(tf.test.TestCase):
  def testLSTMBatchJacobian(self):
    class HasLSTM(tf.keras.Model):

      def __init__(self):
        super(HasLSTM, self).__init__()
        self.lstm = tf.keras.layers.LSTM(units=5)
        self.dense = tf.keras.layers.Dense(1, activation=tf.nn.sigmoid)

      def call(self, x):
        return self.dense(self.lstm(x))

    m = HasLSTM()

    def jacobian(x):
      with tf.GradientTape() as tape:
        tape.watch(x)
        y = m(x)  # pylint: disable=not-callable
      return tape.batch_jacobian(y, x)

    inp = tf.nn.l2_normalize(tf.ones([1, 2, 3]), axis=[1, 2])
    eager_result = jacobian(inp)
    #function_result = tf.function(jacobian)(inp)
    #self.assertAllClose(eager_result, function_result)
    backprop_result, numeric_result = tf.test.compute_gradient(
        m, [inp], delta=1e-3)

    self.assertAllClose(numeric_result, backprop_result, rtol=1e-2)
    self.assertAllClose(tf.reshape(numeric_result, [-1]),
                        tf.reshape(eager_result, [-1]), rtol=1e-2)

if __name__ == "__main__":
  tf.test.main()

Other info / logs
Increasing the delta to 1e-2 makes the test pass.
Side note: Uncommenting the tf.function line yields a couple wrong-looking errors/warnings:
WARNING:tensorflow:Using a while_loop for converting CudnnRNNBackprop
pfor.py:1052] Using a while_loop for converting CudnnRNNBackprop
I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:116] None of the MLIR optimization passes are enabled (registered 2)
E tensorflow/core/grappler/optimizers/meta_optimizer.cc:592] function_optimizer failed: Invalid argument: Input 0 of node loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from has_lstm/lstm/PartitionedCall:6 incompatible with expected variant.
E tensorflow/core/grappler/optimizers/meta_optimizer.cc:592] function_optimizer failed: Invalid argument: Input 0 of node loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from has_lstm/lstm/PartitionedCall:6 incompatible with expected variant.
 W tensorflow/core/common_runtime/process_function_library_runtime.cc:805] Ignoring multi-device function optimization failure: Invalid argument: Input 0 of node loop_body/PartitionedCall/pfor/PartitionedCall/gradients/zeros_like/pfor/ZerosLike was passed float from has_lstm/lstm/PartitionedCall:6 incompatible with expected variant.

Was able to reproduce the issue with TF v2.3 and TF v2.4. Please find the gist of it here. Thanks!

I was able to reproduce the issue on tf-nightly 2.10.0-dev20220721. Kindly find the gist of it here. Thank you!

The issue persists and can be triggered using the same reproducer


State error b2c

Hello,
I request you people to create a clear documentation and the samples code itself have issues.
I have faced a lot of issues while doing refresh tokens and iam facing state routes Error now..

Hi @rnkpareek, we've been actively working on documentation and samples, and we definitely agree with your feedback that documentation could be improved.
If you can fill out the issue template and provide your msal versions, config, and usage, we can help with debugging or fixing any problems you may be seeing.

Closing this as I'm assuming the original question has been answered, if you have further questions please open a new issue. Thanks!


flutter drive not working

Steps to Reproduce

run flutter drive --target=xxx -v

**Target Platform: any(iOS simulator or Android real device)
**Target OS version/browser: MacOS 10.15.1
**Devices:**

logs
[  +73 ms] No devices found.
[   +9 ms] "flutter drive" took 574ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      DriveCommand.runCommand
(package:flutter_tools/src/commands/drive.dart:103:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand
(package:flutter_tools/src/runner/flutter_command.dart:620:18)
#3      _asyncThenWrapperHelper.<anonymous closure>
(dart:async-patch/async_patch.dart:74:64)
#4      _rootRunUnary (dart:async/zone.dart:1134:38)
#5      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#6      _FutureListener.handleValue (dart:async/future_impl.dart:140:18)
#7      Future._propagateToListeners.handleValueCallback
(dart:async/future_impl.dart:682:45)
#8      Future._propagateToListeners (dart:async/future_impl.dart:711:32)
#9      Future._completeWithValue (dart:async/future_impl.dart:526:5)
#10     Future._asyncComplete.<anonymous closure>
(dart:async/future_impl.dart:556:7)
#11     _rootRun (dart:async/zone.dart:1126:13)
#12     _CustomZone.run (dart:async/zone.dart:1023:19)
#13     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#14     _CustomZone.bindCallbackGuarded.<anonymous closure>
(dart:async/zone.dart:965:23)
#15     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#16     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#17     _runPendingImmediateCallback
(dart:isolate-patch/isolate_patch.dart:118:13)
#18     _RawReceivePortImpl._handleMessage
(dart:isolate-patch/isolate_patch.dart:175:5)

flutter doctor -v
[✓] Flutter (Channel master, v1.13.1-pre.47, on Mac OS X 10.15.1 19B88, locale zh-Hans-CN)
    • Flutter version 1.13.1-pre.47 at /Users/flutter
    • Framework revision ad20d368ff (3 hours ago), 2019-12-05 19:33:01 -0800
    • Engine revision 0506c65f59
    • Dart version 2.7.0 (build 2.7.0-dev.2.1 a9c77229c2)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0-rc3)
    • Android SDK at /Users/Library/Android/sdk/
    • Android NDK at /Users/Library/Android/sdk/ndk-bundle
    • Platform android-29, build-tools 29.0.0-rc3
    • ANDROID_HOME = /Users/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Connected device (2 available)
    • VOG AL00 • MQSGL19614000092 • android-arm64 • Android 10 (API 29)
    • macOS    • macOS            • darwin-x64    • Mac OS X 10.15.1 19B88

• No issues found!

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter drive --target=test_driver/app.dart -v
exception
NoSuchMethodError: NoSuchMethodError: The getter 'id' was called on null.
Receiver: null
Tried calling: id
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:51:5)
#1      IOSSimulator.isAppInstalled (package:flutter_tools/src/ios/simulators.dart:270:52)
#2      _startApp (package:flutter_tools/src/commands/drive.dart:243:30)
<asynchronous suspension>
#3      DriveCommand.runCommand (package:flutter_tools/src/commands/drive.dart:121:51)
<asynchronous suspension>
#4      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:490:18)
#5      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#6      _rootRunUnary (dart:async/zone.dart:1132:38)
#7      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#8      _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#9      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#10     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#11     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#12     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:552:7)
#13     _rootRun (dart:async/zone.dart:1124:13)
#14     _CustomZone.run (dart:async/zone.dart:1021:19)
#15     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#16     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#17     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#18     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#19     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#20     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

flutter doctor
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.15.1 19B88, locale zh-Hans-CN)
    • Flutter version 1.9.1+hotfix.6 at /Users/ippan/Documents/flutter
    • Framework revision 68587a0916 (3 months ago), 2019-09-13 19:46:58 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/ippan/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/ippan/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Android Studio
    • Android Studio at /Applications/Android Studio 4.0 Preview.app/Contents
    • Flutter plugin version 41.2.3
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] IntelliJ IDEA Ultimate Edition (version 2019.2.4)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[!] IntelliJ IDEA Community Edition (version 2019.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] VS Code (version 1.40.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.7.0

[✓] Connected device (1 available)
    • iPhone 11 Pro Max • 77601539-763A-4C42-BCC2-9BFB44B48727 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-2 (simulator)

! Doctor found issues in 2 categories.

Hi @iptton
i see you are describing two different issues;
in the first, using flutter channel master your logs show
[  +73 ms] No devices found.
[   +9 ms] "flutter drive" took 574ms.

in the second, using flutter channel stable
appear to be an error from the code.
Could you please describe the issue in details an provide
a minimal code sample or the steps to reproduce the problem?
Thank you

@ianloic sorry for that confuse , It may be two issues at the same codebase.
issue 1 : cannot found devices, but actually flutter devices can find my phone
issue 2 : same as https://github.com/flutter/flutter/issues/43258

@ianloic trying to reproduce the problem in other computer. found something:
Maybe I'm doing wrong with flutter drive:
code puts here: https://github.com/iptton/inview_notifier_list
when using stable branch: the same code with flutter drive --target=test_drive/main.dart: output:

AndroidManifest.xml could not be found
but in master branch, the output is :

[  +73 ms] No devices found.
[   +9 ms] "flutter drive" took 574ms.
#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      DriveCommand.runCommand

solved
I do two things wrong about flutter drive:

test_driver must under a dir that has android or iOS dir. so testing an extension should be in the example?
in test_driver/xxx_test.dart file must not contains any widget imports. In my case, I've import a dart file for a const value.


[directxmath] Fix to support MSBuild usage

The port was installing the headers to include\directxmath which worked fine for CMake scenarios. For MSBuild, however, this would require changing all the source to use #include <directxmath\DirectXMath.h>.
This patch just installs it to include so no client code changes are needed for MSBuild+VCPKG compared t CMake+VCPKG. This change was also made upstream to will be included in future updates as well.

Seems reasonable, thank you!


186049138 - Remove apk package installation

Description:

cf-uaac-image-resource already has g++, musl-dev, make so this line is unnecessary
See successful run at https://deployer.dev02.dev.cloudpipeline.digital/teams/main/pipelines/create-cloudfoundry/jobs/pipeline-lock/builds/73

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

superseded by https://github.com/alphagov/paas-cf/pull/3505


[BUG]  Multiple Recipients only works for TO not CC or BCC

Library name and version
Azure.Communication.Email
Describe the bug
when using the CC and BCC  after passing in the list of emailAddresses to the constructir
the EmailRecipients  CC and BBC property yielded no results...
EmailRecipients emailRecipients = new EmailRecipients(emailAddresses, emailAddressesCC, emailAddressesBCC);
Expected behavior
EmailRecipients  CC and BCC should be set via constructor
Actual behavior
the EmailRecipients  CC and BBC property yielded no results...
Reproduction Steps
https://github.com/Azure/azure-sdk-for-net/blob/Azure.Communication.Email_1.0.0-beta.1/sdk/communication/Azure.Communication.Email/README.md#send-an-email-message-to-multiple-recipients
Send an Email Message to Multiple Recipients
Environment
Inisde an Azure function .net 6

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Identity:0.08975528,Communication:0.0837231,Azure.Core:0.08369434'

ok ignore this i have worked throught it again :-)  looks ok  must have been the lack of coffee


fix(app): android integration tests

Fix android integration tests
Update ios podfile
Update usaged document date and commit

👀


fix(deps): update module github.com/jedib0t/go-pretty/v6 to v6.6.5

This PR contains the following updates:

Package
Type
Update
Change

github.com/jedib0t/go-pretty/v6
require
patch
v6.6.4 -> v6.6.5

Release Notes

jedib0t/go-pretty (github.com/jedib0t/go-pretty/v6)
v6.6.5
Compare Source
What's Changed

text: Wrap* should account for display width; fixes #344 by @jedib0t in https://github.com/jedib0t/go-pretty/pull/347

Full Changelog: https://github.com/jedib0t/go-pretty/compare/v6.6.4...v6.6.5

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Successfully tested usage of updated library


Doesn't handle multiple, similar, reference to the same file

See original issue here: https://github.com/rickharrison/broccoli-asset-rev/issues/116
Given the input:

And a fingerprint of https://mycdn.com/
The output is:

(Note the incorrect slash at the start of the second URL.)
This can be avoided by using the exact same URLs everywhere in your source code. It would be nice, however, if the rewrite engine handled this automatically (eg a sort to process the longer replacements first?)

This seems to be very similar to the issue I reported. See: https://github.com/rickharrison/broccoli-asset-rewrite/issues/63
I think it's a problem with the regex but I'm afraid to change the regex since I can not estimate if and how a change of this regex could break existing apps.


Call GtkImContextFilterKeypress after issuing the KeyDown event

I'm writing an app in avalonia that depends on being able to accurately read all of the KeyDown and KeyUp events.
I'm doing my development on Linux but I'm also testing on windows.
I've found that in order to get the same sequence of events for Control.KeyDown, KeyUp and TextInput on Linux as I get on windows I need to have the KeyDown event issued before the call to GtkImContextFilterKeypress, as that call will then issue the TextInput event via the OnCommit callback in WindowBaseImpl.
The other effect of this change is that both the KeyDown and the TextInput event now get issued on Linux, whereas before the KeyDown event would never be issues if GtkImContextFilterKeypress returned true.
With this change I now get the same events issued to my app when it runs on Linux as I do when I run it on Windows.
Checklist:

[ ] Added unit tests (if possible)?
No unit tests -- all testing done manually.
[ ] Added XML documentation to any related classes?
No documentation needed.
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation
No documentation needed.

https://developer.gnome.org/gtk3/stable/GtkIMContext.html#gtk-im-context-filter-keypress

Allow an input method to internally handle key press and release events. If this function returns TRUE, then no further processing should be done for this key event.

Basically this allows the input method to preprocess key events and signal application to ignore them

https://github.com/AvaloniaUI/Avalonia/issues/2076

GtkImContextSimple is still handling Compose key sequences that don't require special rendering. RAlt + O, C prints "©", for example.
So we currently do support  multi-key input in our GTK backend.

Ok, so I think the question is whether handling key sequences is more important than receiving the same events between operating systems for the moment.

We could implement basic infrastructure for checking if focused control expects text input, but I'm not quite sure if my initial approach in #2076 is correct. "Virtual" text store and some kind of per-control IM-context seems to be a better solution.

Yeah, I'm really not sure either... I just don't have enough experience in this area to be able to give useful input. @tdaffin do you have any experience in this area?

We could try somewhat mirroring GTK's API, but that will require to pass the native handle pointer through the whole event pipeline. In that case the control handling events could decide if it wants them to be processed via IM.
The problem with that approach (aside from the need to handle native objects) is that in Avalonia it's way too easy to intercept events for child controls, so some top-level control logic might break IM interaction, which will be unnoticed by application developer (who will often only do testing on european language localized Windows-machine) until app is deployed on some CJK-machine. That way even our standard TextBox could be broken, which is a bad experience for ends users.

I'm certainly not an expert, but it appears to me that there shouldn't be any reason not to treat the KeyDown/KeyUp events differently from the TextInput events.
In my use case I'm using the keys for an interactive game where I need to know every time any particular key on the keyboard is pressed or released, and I don't need to know if it is going to create a character sent to TextInput or not -- as far as I know the letters associated with keys being used [W, A, S, D, Ctrl, Shift, Alt, Spacebar] are not important at all.
Is it really necessary to prevent code from knowing when keys are pressed or released?  As far as I could see the change I made would still allow the input information into the IME even though it still allows the KeyDown/KeyUp through.
It seems to be that that 'no further processing' is really only relevant to the output of the TextInput event...

I'm using the keys for an interactive game

That's the problem. Some controls (most of them actually) don't care about text input and IME, some controls need key events to be handled and filtered by IME. The problem is that if key events should be handled by IME, we shouldn't allow parent controls to intercept them through the routed events system.
For now your change is breaking the following scenario:

application has a hotkey for Alt+C
user types Alt+(O,C) and expects to see © without triggering Alt+C hotkey

I'm going to merge this right now since it's how OSX backend is currently working.
We'll introduce a proper event filtering for text input later.

Ok, I see -- on windows you hold Alt and type '0169' on the numeric keypad to get ©
I note that this does work in Avalonia on windows -- I et all the relevant KeyDown and KeyUp events but no TextInput events until I let go of the Alt key, at which point I get a single TextInput with just the copyright symbol.
I'll see if I can find a way to get Linux to produce the copyright symbol in case I can find a simple way to support that as well as the KeyDown/KeyUp events.

Because that won't work for complex input methods that present their own UI and need more key presses which shouldn't be handled by application.
Please, at least install a proper input method of all three OS-es (Windows, Linux, OS X) and try entering Chinese or Japanese text before further discussion on input methods.


preact magic?

What is the magical piece I am missing to make preact work with this?  I just keep getting that react is not found yet preact is definitely aliased in package.json, module-alias is present, and i aliased in webpack config.


Add license to README and create issue templates (Closes #177)

This PR creates issue templates and adds the license to the README.md.

Closes #177

Thanks @SebastianWolf-SAP, I committed your suggested changes.


Documentations v1.4 rev1

Updated User and Developer Guide for final release.

Pull Request Test Coverage Report for Build 576

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.282%

Totals

Change from base Build 47:
0.0%

Covered Lines:
3109

Relevant Lines:
3562

💛  - Coveralls

Pull Request Test Coverage Report for Build 48

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.282%

Totals

Change from base Build 47:
0.0%

Covered Lines:
3109

Relevant Lines:
3562

💛  - Coveralls


style: Transition of Navbar from dark to light and vice-versa is now smooth

Fixes issue #24478
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Included in the issue itself
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

closing/reopening to kick-start CI. Hopefully this passes.


active contact is now sometimes gone

this is some regression:

[x] if contact a@b/1 is active (unexpanded), and a@b/2 gets online (with higher priority)
[x] if contact a@b/1 and a@b/2 are online, /1 is active (unexpanded) and gets offline
[x] if contact a@b/1 is active, goes offline, and a@b/2 comes online
[x] if self contact comes online another time (again, priorities etc)
[x] if ctrl-q pressed, which jumps to an unexpanded contact

might be more cases as well... expand before focus/unfocus (or again: focus bare jid into unexpanded contact -- but i believe this gets worse)

I like to preserve several invariants:

there's always a single active contact visible, which is in reverse color mode in the buddy list
the program itself does not change the active contact on its own (ok to do for broken xmpp clients with random resources)
the program itself does not modify the expand/collapsed state of contacts

the third invariant is already sometimes violated (if contact a@b/1 and a@b/2 are online, and 1 goes offline, it is collapsed). the other two are imho more important than the third (esp the second, since otherwise you might send messages to a resource with an encryption state which you did not intent to do).
thus: I'll expand at my own will within jackline. if someone can think of a better solution, please tell.

this is all still (as of deed63f401a6c332b42f1e04f9f10ddcc3143518) horribly broken:

if multiple sessions are online, the contact is always expanded
offline messages from a bare jid won't highlight a single full contact (but rather stupidly blink around)

all crap, needs more thinking and proper code

focus on bare contact, then F5, focus on offline full contact -> nothing highlighted; also, single full contact shown should always contain all the history (e.g. if other sessions are offline and thus not displayed)
some more thinking needs to be done:

when/what to notify (subscription: bare, message: full)
when to expand (if notification of something not shown is there; corner case: similar session)
which history to show

i again claim this is fixed now. let's see how long until i reopen this issue again...


[BUG]: keyInfo usage

Is your feature request related to a problem? Please describe...
I have just updated to 4 version and confused by validateSignatureValue function.
I have duplicated <KeyInfo> inside SamlRequest and SamlMetadata. Similar to Okta example http://saml.oktadev.com/.
That's mean that loadSignature functions will initialize this.keyInfo by request key, and validateSignatureValue will use it preferable over metadata certificate without any option to choose another behavior.
       // loadSignature
        const keyInfo = xpath.select1(".//*[local-name(.)='KeyInfo']", signatureNode);
        // TODO: should this just be a single return instead of an array that we always take the first entry of?
        if (xpath.isNodeLike(keyInfo)) {
            this.keyInfo = keyInfo;
        }

        // validateSignatureValue
        const key = this.getCertFromKeyInfo(this.keyInfo) || this.publicCert || this.privateKey;

Describe teh solution you'd like...
Another order of keys.
Describe the alternatives you've considered...
Configurable keyInfo

Thanks for flagging this. This was also picked up by another user and discussed in #399. The user pointed out that there is a viable workaround in this comment.
tldr: consider doing the following to declare getCertFromKeyInfo() a no-op:
new SignedXml(
  {
    // ...
    getCertFromKeyInfo: () => undefined
  }
);

4.x introduced a breaking change where we now adhere to section 3.2.2 of W3C's XML Signature Syntax and Processing (Second Edition)^1, ie, if a certificate is present in the KeyInfo element of an XML document, we will use that for validation.

getKeyInfoContent is now a noop by default. This will show in the next semver-major release.

Thanks for flagging this. This was also picked up by another user and discussed in #399. The user pointed out that there is a viable workaround in this comment.
tldr: consider doing the following to declare getCertFromKeyInfo() a no-op:
new SignedXml(
  {
    // ...
    getCertFromKeyInfo: () => undefined
  }
);

4.x introduced a breaking change where we now adhere to section 3.2.2 of W3C's XML Signature Syntax and Processing (Second Edition)1, ie, if a certificate is present in the KeyInfo element of an XML document, we will use that for validation.
Footnotes
1. https://www.w3.org/TR/2008/REC-xmldsig-core-20080610/#sec-CoreValidation [↩](#user-content-fnref-1-6bd00c995b16e7b73ddf894a6d7c52fd)

By that logic you should also resolve the SecurityTokenReference when in wssecurity mode.
https://docs.oasis-open.org/wss/v1.1/wss-v1.1-spec-pr-x509TokenProfile-01.htm

By that logic you should also resolve the SecurityTokenReference when in wssecurity mode.
https://docs.oasis-open.org/wss/v1.1/wss-v1.1-spec-pr-x509TokenProfile-01.htm

Feel free to submit a PR.


⚠ Site has degraded performance

In 2ae7edb, Site (https://fyle.uk) experienced degraded performance:

HTTP code: 200
Response time: 3241 ms

Resolved: Site performance has improved in 70ab45f after 8 minutes.


Update deps

Giant, gross PR to update all dependencies in the various libraries.
This also fixes some prettier and linting issues.
I've confirmed that the same tests are failing in the same way as on master so I'm assuming all is as well as it was before. 😬

FML – dependency hell. I'm closing this and I'll update the libraries separately.


feat(conan): add conan toolchain import third-party C/C++ dependencies

Closes #58
This is a rebase of #58. I no longer have push access to this repository and can't update #58 myself, so I created this separate PR to run CI on the changes. See #58 for a detailed description and motivation.

The import error is:
2023-10-15T10:18:36.176308Z ERROR cli_log: Error in changeset processing: Unknown error: This file path is illegal and cannot be imported: buck2/prelude/toolchains/conan/TARGETS.v2

I suspect this is a bug with our import/export configuration. Ours is a little tricky because the files are internally named TARGETS, and then renamed for export (we want to change the source of truth to BUCK eventually, once Buck1 dies). Perhaps if you change prelude/toolchains/conan/BUCK to prelude/toolchains/conan/BUCK.v2 it might work?

Nothing to be done about the PR failures - we've fixed them on master. I've successfully imported the diff and we are reviewing internally.

I've tested the merged state and it works. Thank you for pushing this through!


Contextual comments are lacking in the files

As a Developer that is looking at the Open Source Guardian Code, I want to able to see comments at the beginning of important files that give an introduction as to what the code below is all about.
Definition of Done: Commenting should follow TSDOC Format
Description
Files are uncommented in the https://github.com/hashgraph/guardian/blob/main/guardian-service/src/api/
Can we please add information on what these files are and provide some basic commenting?
Steps to reproduce
Example: https://github.com/hashgraph/guardian/blob/main/guardian-service/src/api/approve.service.ts
Additional context
No response
Hedera network
other
Version
any
Operating system
No response

This has been completed. Here is the Pull Request for DOD / Acceptance Criteria for this ticket:
https://github.com/hashgraph/guardian/pull/124
@sergmetelin @danielnorkin @anvabr


feat(headless-client): require explicit DNS method

Closes #5063
Will this affect customers?
### Tasks
- [ ] Confirm existing customers using the headless Client won't be affected
- [ ] Smoke-test the Linux GUI
- [ ] Smoke-test the Windows GUI

Agreed. I really liked the idea of not having any default at all to protect from Hyrum's Law but systemd-resolved is fine as a default and if we ever had to drop it for some odd reason, we could make it a required arg when that happens

Agreed. I really liked the idea of not having any default at all to protect from Hyrum's Law but systemd-resolved is fine as a default and if we ever had to drop it for some odd reason, we could make it a required arg when that happens

First time I heard of this law!

#6163 supersedes this


Add new keys GridPower and LoadPower

envertech-pv.0 | 2024-01-27 18:34:27.401 | warn | [station] object GridPower not configured - report to developer.
envertech-pv.0 | 2024-01-27 18:34:27.400 | warn | [station] object LoadPower not configured - report to developer.

implemented at 1.2.0


Issue with docs

The left hand side says pass-through authentication current limitations but on the right hand side the subject says Azure Active Directory Pass-through Authentication security deep dive

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

ID: eafd62b3-145a-7c93-e88d-a7fb29814a0d
Version Independent ID: 999181be-96fa-1be6-36f3-b604f43613ed
Content: Azure Active Directory Pass-through Authentication security deep dive
Content Source: articles/active-directory/hybrid/how-to-connect-pta-security-deep-dive.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

Attached what i mean here. This is quite strange

Hi @limjianan Thank you for your feedback! We have assigned this issue to the author to investigate further and take the right course of action.

This is not my experience.  The articles are correct.
#please-close


I want to build a Light Factory and construct wheeled units from it

Depends on #9 and #25
Acceptance criteria:

[x] ability to build and place it
[x] ability to build a trike and quad
[x] once unit is built, it will appear next to the building on a free cell
[x] when no free cells are found then no unit will be deployed (for now)

more info:
http://duneii.com/buildings/#312

This is finished by merging #128


update sendgrid endpoint to handle new payload structure

SendGrid apparently sends an array of event objects in a single payload, this change updates the controller method to loop through each event and decide what to do.
Also removed some tests that don't make sense with this new behavior.

Ugh, sendgrid. +1


[v3] Performance issue since migration from 2.0 -> 3.0

Description
So I followed the migration guide from unistyles 2.0 -> 3.0
I have this tab button which opens a bottom sheet, and since I moved over to the new version it renders extremely slowly, before it worked almost instantly now it is noticeably choppy.
Is there something I am doing incorrectly in my approach, my new code is below.
I also sometimes get Style is not bound error! but I'm not sure why, as I don't see where Im merging styles anywhere.
(tabs)/_layout.tsx
import { Tabs } from 'expo-router';
import FontAwesome from '@expo/vector-icons/FontAwesome';

const ThemedTabs = createUnistylesComponent(Tabs, theme => ({
  screenOptions: {
    tabBarActiveTintColor: theme.colors.surface['900'],
    tabBarInactiveTintColor: theme.colors.surface[100],
    tabBarStyle: {
      backgroundColor: theme.colors.surface['000'],
      borderTopWidth: 0,
      height: 90,
    },
  },
}));

const ThemedFontAwesome = createUnistylesComponent(FontAwesome, theme => ({
  color: theme.colors.surface['900'],
}));

<ThemedTabs>
... other tabs
<Tabs.Screen
        name="plus"
        options={{
          headerShown: false,
          tabBarShowLabel: false,
          tabBarIcon: ({ color }) => <FontAwesome size={28} name={'plus'} color={color} />,
          tabBarButton: () => {
            return (
              <GenericBottomSheet
                snapPoints={['45%']}
                triggerComponent={
                  <View
                    style={{
                      display: 'flex',
                      paddingHorizontal: 25,
                      height: '100%',
                      borderRadius: 10,
                      justifyContent: 'center',
                      alignItems: 'center',
                    }}
                  >
                    <View style={styles.plusButton}>
                      <ThemedFontAwesome size={24} name={'plus'} />
                    </View>
                  </View>
                }
              >
                <View style={styles.bottomSheetContent}>
                  <BottomSheetNavigationItem
                    path="/(auth)/log-meal"
                    emoji={'greenApple'}
                    color={'red'}
                    title="Log a meal"
                    description="Add a meal to your diary"
                  />
                  <BottomSheetNavigationItem
                    path="/(auth)/log-workout"
                    emoji={'weightLifting'}
                    color={'blue'}
                    title="Log a workout"
                    description="Add a workout to your diary"
                  />
                  <BottomSheetNavigationItem
                    path="/(auth)/log-progress"
                    color={'green'}
                    emoji={'ruler'}
                    title="Track progress"
                    description="Update weight or measurements"
                  />
                </View>
              </GenericBottomSheet>
            );
          }
/>
</ThemedTabs>

generic-bottom-sheet.tsx
import React, {
  useCallback,
  useRef,
  ReactNode,
  ReactElement,
  Children,
  isValidElement,
  cloneElement,
} from 'react';
import { View, TouchableOpacity } from 'react-native';
import {
  BottomSheetModal,
  BottomSheetBackdrop,
  BottomSheetBackdropProps,
  BottomSheetScrollView,
  BottomSheetView,
} from '@gorhom/bottom-sheet';
import { StyleSheet } from 'react-native-unistyles';

type Props = {
  children: ReactNode;
  triggerComponent: ReactNode;
  snapPoints?: (string | number)[];
  onChange?: (index: number) => void;
  scrollable?: boolean;
};

const DEFAULT_SNAP_POINTS = ['50%', '90%'];

export default function GenericBottomSheet({
  children,
  triggerComponent,
  snapPoints = DEFAULT_SNAP_POINTS,
  onChange,
  scrollable = false,
}: Props) {
  const bottomSheetModalRef = useRef<BottomSheetModal>(null);

  const renderBackdrop = useCallback(
    (props: BottomSheetBackdropProps) => (
      <BottomSheetBackdrop {...props} appearsOnIndex={0} disappearsOnIndex={-1} />
    ),
    []
  );

  const handlePress = () => {
    bottomSheetModalRef.current?.present();
  };

  const handleSheetChanges = useCallback(
    (index: number) => {
      if (onChange) {
        onChange(index);
      }
    },
    [onChange]
  );

  const dismiss = () => {
    bottomSheetModalRef.current?.dismiss();
  };

  const renderChildrenWithDismiss = (children: ReactNode): ReactNode => {
    return Children.map(children, child => {
      if (isValidElement(child)) {
        return cloneElement(child as ReactElement, {
          dismiss,
          children: renderChildrenWithDismiss((child as ReactElement).props.children),
        });
      }
      return child;
    });
  };

  return (
    <View>
      <TouchableOpacity onPress={handlePress} activeOpacity={0.8}>
        {triggerComponent}
      </TouchableOpacity>
      <BottomSheetModal
        ref={bottomSheetModalRef}
        snapPoints={snapPoints}
        onChange={handleSheetChanges}
        backdropComponent={renderBackdrop}
        backgroundStyle={styles.backgroundStyle}
        handleIndicatorStyle={styles.handleIndicator}
      >
        {scrollable ? (
          <BottomSheetScrollView style={styles.container}>
            {renderChildrenWithDismiss(children)}
          </BottomSheetScrollView>
        ) : (
          <BottomSheetView style={styles.container}>
            {renderChildrenWithDismiss(children)}
          </BottomSheetView>
        )}
      </BottomSheetModal>
    </View>
  );
}

const styles = StyleSheet.create(theme => ({
  container: {
    paddingHorizontal: 16,
    backgroundColor: theme.colors.surface['000'],
  },
  backgroundStyle: {
    backgroundColor: theme.colors.surface['000'],
  },
  handleIndicator: {
    backgroundColor: theme.colors.surface['300'],
  },
}));

bottom-sheet-navigation-item.tsx
import { Text, TouchableOpacity, View } from 'react-native';
import React from 'react';
import { StyleSheet } from 'react-native-unistyles';
import { Href, router } from 'expo-router';
import typography from '@/constants/typography';
import { EmojiKey } from '@/constants/emojis/types';
import { ColorName } from '@/constants/colors/types';
import { getColor } from '@/constants/colors/utils';
import { getEmojiByKey } from '@/constants/emojis/utils';

type Props = {
  path: Href;
  title: string;
  description: string;
  emoji: EmojiKey;
  color: ColorName;
  dismiss?: () => void;
};

export default function BottomSheetNavigationItem({
  path,
  title,
  color,
  emoji,
  description,
  dismiss,
}: Props) {
  const colorHex = getColor(color);
  const emojiCode = getEmojiByKey(emoji);

  return (
    <TouchableOpacity
      style={styles.container}
      onPress={() => {
        if (dismiss) {
          dismiss();
        }
        router.push(path);
      }}
      activeOpacity={0.8}
    >
      <View style={styles.iconContainer(colorHex)}>
        <Text style={styles.emoji}>{emojiCode}</Text>
      </View>
      <View style={styles.textContainer}>
        <Text style={styles.title}>{title}</Text>
        <Text style={styles.description}>{description}</Text>
      </View>
    </TouchableOpacity>
  );
}

const styles = StyleSheet.create(theme => ({
  container: {
    display: 'flex',
    flexDirection: 'row',
    alignItems: 'center',
    justifyContent: 'space-between',
    padding: theme.padding.md,
    borderBottomWidth: 1,
    gap: 20,
    borderBottomColor: theme.colors.surface['020'],
  },
  emoji: {
    fontSize: 28,
  },

  iconContainer: (color: string) => ({
    display: 'flex',
    alignItems: 'center',
    justifyContent: 'center',
    width: 50,
    height: 50,
    borderRadius: 10,
    backgroundColor: color,
  }),

  textContainer: {
    display: 'flex',
    flexDirection: 'column',
    flex: 1,
    alignItems: 'flex-start',
    justifyContent: 'center',
  },
  title: {
    ...typography.headings.h4,
    color: theme.colors.text['900'],
  },
  description: {
    ...typography.body.bodySmall,
    color: theme.colors.text['500'],
  },
}));

Steps to Reproduce
Untested on android
Snack or Repository Link (Optional)
No response
Unistyles Version
3.0.0
React Native Version
0.76.3
Platforms
iOS
Expo
Yes

I don't see anything specific. Can you give me some repro so I can play around?

Thanks and no worries.
Will definitely take a look before releasing beta.2!

Is it ok now? 😄
https://github.com/user-attachments/assets/3f577fe8-e316-4700-8d31-ecbd4f2fe98d

Thank you!

Hey @jpudysz hope you are well sorry to tag but this issue is currently closed.
the issue is improved for me in 3.0.0 beta 2 but its still very low fps compared to unistyles 2.0.0 is there a some debugging steps I can do to provide you more information.

Hey, how do you know it has a lower FPS? Does it look like the video I posted above?
How does it look like?

Hey @jpudysz, sorry to be likely surfacing errors unrelated to your project more than once now 😭.
After reading the following I believe it may just be a new architecture issue with react native bottom sheet.
Issue on gorham bottom sheet package
https://github.com/user-attachments/assets/382c398d-44b9-4029-9af1-dc2be60cd2bb
This is a video from my device, although I will note it looks noticeably choppier on the device than this video for whatever reason.
It just basically has some quite noticeable lag when opening the bottom sheet, according to the issue it is affecting multiple packages.
The changes you previously made has made an improvement however! But assume we can keep this closed.

Ah! Thanks for the reference. That’s what I though as I don’t apply any animations 😇

@samducker yeah experiencing the same behaviour also not as noticeable in recordings. Most likely issue with reanimated


Decorator to get position of the first/last non-empty element of a merkle tree.

Add a decorator to get the position of the first/last non empty leaf of a Merkle tree.
Context: https://github.com/0xPolygonMiden/crypto/issues/170

One possible design is to have the target position of the element on stack and the Merkle tree root:
Stack: [el_pos, tree_root ...]

And the decorator would push the leaf position to the advice stack.
The Advantage of the above is that a single decorator supports not only the first/last position, but also arbitrary values in-between. The disadvantage is that to get the position of the last element either the number of elements must be known, or a magic value needs to be used.

I'd probably go for the simpler approach for now as figuring out the correct position will be quite complicating, I think.
A couple things to clarify:

We probably want to return both the index and the depth as zero nodes could theoretically be located at different depths in different parts of the tree.
What should we return if there are no zero nodes in the tree?

correct position will be quite complicating, I think

It should be easy:
container.inner_nodes().filter(|n| n.is_leaf()).nth(index)

The complexity is basically O(n) where n is the number of leaves.

What should we return if there are no zero nodes in the tree?

Undefined? We can easily check if the tree is empty by looking at the root value.

It should be easy:
container.inner_nodes().filter(|n| n.is_leaf()).nth(index)

Not sure this will work as in the MerkleStore the order of nodes is not defined.

What should we return if there are no zero nodes in the tree?

Undefined? We can easily check if the tree is empty by looking at the root value.

Comparing against empty tree root in MASM would require comparing words. I think we we return a well-defined value for empty trees we can make it more efficient (i.e., need to compare only single element).
Assuming, this decorator puts (depth, index) on the top of advice stack, we could say that (0, 0) means that the tree is empty. Or maybe just returning depth=0 is good engouh.


URGENT: Server gives @ to everyone when joining an unregistered channel

Apparently the server doesn't recognize anymore who is the first user on the channel and gives @ to everyone.
I haven't tested with a registered channel yet.
Screenshot: http://imgur.com/a/KRaF4

Yeah I stuffed this up while writing the channel registration code. I've pushed a fix to master, just performing some final tests and I'll push out a new build, thanks.

This is fixed in the new release here, thanks very much for the yell on this!


🐛 FIX: Singlehtml builder error when adding to another project.

fixes https://github.com/executablebooks/sphinx-proof/issues/65 .
Apparently adding a singlehtml handle for a node, breaks other project's node which don't handle singlehtml.
Also, the addition of integrating handling of searching fignumbers for singlehtml builders.

Codecov Report

Merging #66 (dc3fab9) into master (80443cc) will decrease coverage by 0.28%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   96.80%   96.51%   -0.29%
==========================================
  Files           5        5
  Lines         313      316       +3
==========================================
+ Hits          303      305       +2
- Misses         10       11       +1

Flag
Coverage Δ

pytests
96.51% <75.00%> (-0.29%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sphinx_proof/__init__.py
91.66% <ø> (ø)

sphinx_proof/nodes.py
93.20% <75.00%> (-0.80%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix #1047

Fixes #1047
Changes Summary

Add www. redirection

This pull request is ready for review.

When this is merged we're going to need to add the SSL block to both server blocks, not just one.


Add documentation to explain the use of multiple GitHub access tokens

I like the idea behind this, but I worry that A) most of the code examples are very specific to bundle-size and B) this doesn't provide much beyond what https://developer.github.com/apps/differences-between-apps/ documents. I'm not sure if this deserves its own doc, as opposed to a small section in the top-level repo README linking to the GitHub docs on auth types

A) addressed
B) none of this information is in the link you provided. This isn't something that seems to be documented correctly in their official docs :(

PTAL at recent changes


SQLite drives seems to skip quoting table and column names

Version: 0.10.0
Module: quill-jdbc
Expected behavior
Queries should have identifiers properly escaped (e.g., "Transaction").
Actual behavior
Identifiers are not escaped, resulting in invalid queries like:
INSERT INTO Transaction (id,timestamp,amount) VALUES (?, ?, ?)

Steps to reproduce the behavior
case class Transaction(id: Long)
val ctx = new JdbcContext[SqliteDialect, Literal](ConfigFactory.parseMap(Map(
    "driverClassName" -> "org.sqlite.JDBC",
    "jdbcUrl" -> "jdbc:sqlite:test.db"
  ).asJava))
ctx.run(ctx.quote(
  query[Transaction].insert(lift(Transaction(0))
))

@getquill/maintainers

Related to #405

Closed in favor of #405


feat(dataset): Restore a dataset to original state during a revision

Need
As a user, during a revision, if I replace a dataset and later realize I made a mistake, I need a way to restore a dataset to its last published state without restarting the revision.
Approach
Add an endpoint to restore a dataset to the last published version of the dataset.
Definition of Done

during a revision, a modified dataset can be returned to its published state without starting a new revision.

Not needed at this time.


feat: File Field Plugin

Summary
A basic "file" field should be supplied.
Basic example
{
  name: "frontmatter.pdf",
  label: "PDF",
  component: "file"
}

Motivation
Currently there is an "image" upload but it is specifically for uploading images.

tinacms-fields has you covered: https://github.com/mmintel/tinacms-fields/tree/master/packages/tinacms-file-field


Process: support tiers for hardware for Zephyr v3.3 and later

This issue covers a process discussion for introducing support tiers for boards with upstream Zephyr project support.
The goal of this issue is to define a high level process for defining the support level for different Zephyr boards. This will aid users in knowing which platforms are well tested and debugged. For a related release criteria issue, see https://github.com/zephyrproject-rtos/zephyr/issues/46759.
Proposal for board tier names and descriptions:

Tier 0: emulation boards used in CI; failures can block PRs. Supported by the Zephyr project itself, commitment to fix bugs in releases. One is required for each new architecture.
Tier 1: "real hardware" with commitment from a specific team to run tests using twister device testing for the "Zephyr compatibility test suite" (details TBD) mentioned above on a regular basis using open drivers. Commitment to fix bugs in time for releases. Not supported by "Zephyr Project" itself. Other quality of developer experience criteria TBD. General availability for purchase (modulo silicon supply chain crisis).
Tier 2: board implementation is available in upstream, no commitment to testing, may not be generally available. TL;DR no guarantees, if it breaks you get to keep both pieces. Has a dedicated maintainer who commits to respond to issues / review patches, however.
Tier 3: Deprecated board. Board implementation is available, but no owner or unresponsive owner. No commitment to support is available. May be removed from upstream if no one works to bring it up to tier 2 or better. If the last board for a particular SoC is removed, the SoC and drivers will be removed as well.

This will also affect the release criteria for future versions of zephyr:

maintainers will guarantee testing for boards in higher-priority support tiers
bugs in boards in higher-priority support tiers are more likely to block the release

Once the high level process is defined, the goal is for the testing working group to manage the details with input from interested stakeholders.
Previous discussion:
https://docs.google.com/document/d/1LOhgronKx3TxUypYLy9nex0_3UC58FMMjbXV4wbiuiE/edit#

related discussion: https://github.com/zephyrproject-rtos/zephyr/discussions/35872

Process WG:

@nashif : want to avoid repeating testing work on two different boards with the same hardware
@mbolivar-nordic : designating this at the board level misses somethings, like sensor drivers
@nashif : also ties to missing documentation at the SoC level, not just board level. E.g. a change to an SoC right now rebuilds everything. If we can maintain meaningful information on what SoCs are present on a board, that helps for coverage, documentation, drivers
@mbolivar-nordic : propose we move the conversation over to how we write this metadata down in a machine-readable way for the documentation rework that's ongoing
@dleach02 : how to start tracking which platforms are priority for different HW features, though?
@nashif : probably worth defining deeper criteria for testing and associating them with release criteria

Support tiers:

@stephanosio : highest would be supported by company, second community support, etc
@dleach02 what does community support mean?
@nashif let's look at https://github.com/zephyrproject-rtos/zephyr/discussions/35872 again
@mbolivar-nordic I would push back on "silicon vendors" being a requirement for "reference" / "tier 1" boards -- the main thing that is needed is a committed maintainer or team of maintainers who commit to testing and fixing bugs on their boards for every release
@stephanosio need commitment to testing and reporting results as well
@nashif the least we should expect from tier 1 is to use device testing via twister on a regular basis and submit results. In the future we should work towards making boards available on the cloud, but let's not block this effort on that idea.
@dleach02 important to note the consumers for this data: a primary one is the release management team
@keith-zephyr what is the state of hardware testing now?
@dleach02 depends on the vendor. @hakehuang on NXP does our testing on a shared range; he says he tries to run once a week right before the testing wg meeting. Intel does a daily run, right?
@nashif we do, yes -- results reported on https://github.com/zephyrproject-rtos/test_results
@mbolivar-nordic this is an ask for people to commit to; I think we should list the tier 1 boards in a prominent place so their vendors can claim that their boards are well supported to customers etc.
@galak what about other drivers?
@nashif we do need this for e.g. sensor drivers, but let's cover platform support first
@mbolivar-nordic we agreed in the binary blobs discussion (https://docs.google.com/document/d/1heqcv7dzGvM5rA9xpTMW3kyKJLpZsl2Gjsmr5eqBje8/edit#) to define a suite of tests that targets must be able to pass. I think Tier 1 boards should be able to pass all of these tests on every release.
@nashif yes, at least this.
@galak should we require open source implementations for key features for tier 1 boards?
@nashif I think that makes sense
@galak what about public documentation for the SoC?
@nashif it's tricky. This is a plus, but not everybody does it the same way; not everybody has permissions to share these.
@galak perhaps public availability of documentation for the SoC could give you some points. I bring this up because when I review a vendor driver, if I don't have HW docs, I can't fully review.
@nashif this would also enable third parties to write drivers. This is a plus, but certain items on the checklist shouldn't outweigh others
@dleach02 we're also really talking about general availability for purchase here. Some board owners don't want to release the board directory, but may want to report testing results. This still is useful for testing results.
@galak additional things that might matter here are schematics, etc.
@stephanosio two relevant things for board "tier level" here: 1. support level, 2. developer experience (docs, schematics, etc.)
@galak there's a separate conversation about lifecycle to be had here.
@stephanosio we can do a period re-evaluation with a nomination process for moving between tiers.

Proposal for board tier names and descriptions:

Tier 0: emulation boards used in CI; failures can block PRs. Supported by the Zephyr project itself, commitment to fix bugs in releases. One is required for each new architecture.
Tier 1: "real hardware" with commitment from a specific team to run tests using twister device testing for the "Zephyr compatibility test suite" (details TBD) mentioned above on a regular basis using open drivers. Commitment to fix bugs in time for releases. Not supported by "Zephyr Project" itself. Other quality of developer experience criteria TBD. General availability for purchase (modulo silicon supply chain crisis).
Tier 2: board implementation is available in upstream, no commitment to testing, may not be generally available. TL;DR no guarantees, if it breaks you get to keep both pieces. Has a dedicated maintainer who commits to respond to issues / review patches, however.
Tier 3: Deprecated board. Board implementation is available, but no owner or unresponsive owner. No commitment to support is available. May be removed from upstream if no one works to bring it up to tier 2 or better.

Test selection:

@nashif : another issue is deciding which tests to run on a board; need to establish the scope of things to cover for testing. It's more about hardware testing of specific functionality: MPU, SMP, etc.

Process WG:

@mbolivar-nordic: Nordic is interested in having Tier 1 boards. We are looking for a process owner for test reporting infrastructure.
@nashif the testing WG needs to define requirements
@dleach02 : release managers are a stakeholder here as well

Process WG:

@nashif we discussed this at the testing WG this week. I asked everyone to explore and come up with proposals, tomorrow hopefully we will know more at the testing WG meeting tomorrow.
@mbolivar-nordic any leading candidates for test management systems, or still open?
@nashif we had been looking at an open source solution, but on its own it was not enough, as it was an open core solution with attempted lock in via commercial "brother". The commercial solution looked good, but we need more testing and a decision on cloud-based vs self-hosted. Infrastructure "team" (@stephanosio ) will need to manage whatever is decided.
@mbolivar-nordic I think the test suite mentioned in https://github.com/zephyrproject-rtos/zephyr/pull/47678/ is going to form the initial scope for testing for some tier here, and we'll need to move the list of tests in the binary blobs PR over to a dedicated "tier support" test suite definition

Process WG:

@stephanosio : Testing WG is still discussing
@dleach02 : I discussed with @hakehuang ; this is ongoing for tomorrow's meeting as well. General feedback on the NXP side is to start looking for boards to promote internally.

Process WG:

@stephanosio : "opensearch" was proposed by Nordic at the last testing WG. @nashif was not there so no feedback from him yet; discussion continues. Opensearch is like grafana but has a filtering function that allows you to view what tests ran on what platforms.
@nashif let's keep this in progress here and revisit next week; need to discuss some other process related items

Process WG: nothing on this this week

Process WG:

@nashif (who could not attend) requests that we make the board tier level recommendations to the TSC, with details to be sorted out by the testing WG assuming TSC approval
no objections, moving forward with that

We've had TSC approval for this in principle for a while, and have now moved the implementation work to the testing WG's domain


Possibility to pass rifmReplacer prop (or smth like that) down to Rifm's 'replace' prop along with rifmFormatter

Is your feature request related to a problem? Please describe.
Feature is related with masking input when custom rifmFormatter is passed to KeyboardDatePicker. Current behavior is that when one pass custom rifmFormatter fn, then mask is gone. This is because of the Rifm component is lacking proper mask function and then it's not be able to show mask properly (mask is gone). Here is sandbox forked from rifm example https://codesandbox.io/s/istarkovrifm-date-format-d85xs. Here if you comment out line 69
// replace={addMask}
then mask is gone.
Describe the solution you'd like
I think it would be nice to have a possibility to pass a prop to KeyboardDatePicker called for example 'rifmReplacer', that would do proper masking for me and would be passed down to Rifm component.

Make sure that we are using only 0.8 version of rifm

There is already refuse prop

Could you please describe how can refuse prop be useful in that?
AFAIK refuse prop has nothing to do with masking itself. Refuse prop is of type RegExp that say smth like 'user cannot write other characters than numbers' (which is default behavior) or 'make the behavior different based on passed regexp'.
What I meant with this feature request was ability to pass custom format function along with rifmFormatter, because when I use rifmFormatter, then mask (//____) is gone and is not shown at all at the start of typing the date.
Thanks.


Change project license [v2]

fix issue with v1 #69

@bdilly There are some files that does not have licenses in it:
soleta_module/samples/light.fbp
The json configuration files (Do they need license?)

Most of json files don't need it (soletta's description json files have fields for license)
These other files you just spotted require license headers. I'll add it. Thank you

@cabelitos these files you mentioned don't exist in our repository

merged, thanks


Analysis of super bullet flag

Thank you for your feedback; patch pushed.
@allejo the change and addition included. The matter about hiding inside (or behind) objects is included in the last paragraph already, though, so I skipped it.
@jwmelto your suggested change is simply a rewording of what is already written in the second paragraph of the summary. I have removed the conflicting paragraph entirely.


Add Url to nuspec

Please add a link to the github project (https://github.com/vain0x/DotNetKit.Wpf.AutoCompleteComboBox) in the description of your nuget

Fixed on v1.13.1. Thanks!


[FEATURE] Display TOC using a drawer component on mobile

Describe the feature you want 描述你的功能需求
Something like this https://mui.com/material-ui/react-drawer/
The open button will be on top of the return-to-top button.
Useful reference 有价值的参考
https://github.com/HEIGE-PCloud/DoIt/issues/757

https://github.com/wes-goulet/side-drawer
I also need to solve the toc id issue https://discourse.gohugo.io/t/tableofcontents-without-tableofcontents-id/23529/2

Should use https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dialog instead


Remove similar letters (big-i and small-L) from char preset

Some characters, like the ones you suggest, are really hard to distinguish from each other. It’s a valid change for the font this module is using now.
However, in the future, I want to add the option that allow users to use their own font. In that case, it would be inappropriate to take big-I and small-L out of the presets.
Instead of hard code the presets, do you think you can make your change an option to the captchaText function?

Good idea! I have updated my PR.

thx


Use of checkers on docker_cli_info_test.go

Signed-off-by: liaoqingwei<EMAIL_ADDRESS>

LGTM 🐯

LGTM


Refactor out global service

create HttpUtilsService
create ConstantsService
use Routes not selectedSiteId
remove unused fields
renamed global.service to misc-utils.service

All tests pass.

Looks all good. Well done.
We may need to add "export * from './http-utils.service';" to src/client/app/shared/global/index.ts.


fix missing lookahead

the command prefix should match as given - with undocumented space as separator wildcard it should not allow the prefix to re-appear rather than perform lookahead, removes errors like {pref}{pref}{pref}{pref}\s\s{pref} and such.
Fix produces following examples (prefix ~, green is new matching):
// matching
~register
~ register

// not matching
~~register
~~~~~~~~register
~ ~register

Previously all results would match.

This breaks for any commands that contain the prefix within their name. Remember that prefixes can just be plain words, and command names can have symbols.


Multi-line aliases, runtypes, loadtypes etc

In MOS 3 we have support for aliases, but currently they are restricted to performing a single command
RISC OS systems would support multiple commands within a single alias.  commands would be split using |M (ctrl-M, a carriage-return character).
We should support this too.  This applies to command aliases, runtypes, and loadtypes too.
(Hotkey definitions should ideally support this as well, however that likely requires a completely different approach to be taken for how hotkeys are handled in the line editor.)

Multi-line handling of hotkeys is likely strongly related to #18


(fix+feat): improve DiffField.jsx to work with complex pages

The HistoryDiff requires a redesign, as the current layout is broken on complex pages and isn't functioning as expected. My plan is to avoid treating the HTML as a simple string and layering classes on top, which could cause issues. Instead, I'll take a more structured approach by properly parsing the HTML.
Before:

After:

@dobri1408 having the .snap test files changed means this is a breaking change to the code. I wonder which of your logic makes it so that the content is without the span tag.
I would also like to see a before and after of the actual volto stock demo instead of the EEA website diffing in action.

I made a change by removing the DefaultView for the RenderBlocks (https://github.com/plone/volto/pull/6309/files#diff-1d656037a4db345e1154459194c480148b2b6a85acf74691f76a113f5df89199L81). The reason for this is that the DefaultView also renders elements like the footer and navigation bar, which is not ideal. We want to focus solely on the differences in the blocks. This is why one span tag is removed.
Here are some screenshots :
Before:

After

@ichim-david The coloring is controlled by the CSS style found here: https://github.com/plone/volto/blob/a2d997b3e7a9746970f97c5cd7bec30a08a46daf/packages/volto/theme/themes/pastanaga/extras/main.less#L250. This style applies only to elements where the background can be modified, such as text, so it doesn't affect images. However, I don't believe it's necessary to extend it to images since changes to them are visually apparent. The coloring is mainly intended for text. That said, if you think adding something like a border for images would be better, I can implement that. Let me know what you prefer.


Designing an Interactive Services Section for Salon Website Landing Page

Issue Description:
Develop an interactive and engaging services section for the salon's website landing page. This section will highlight key service offerings such as hair care, skin care, and custom care. Each service category should be represented in a way that is visually appealing and in harmony with the website's overall theme. The main requirements for this section include:

Interactive Service Categories:

Distinct sections or icons representing each service: Hair Care, Skin Care, Custom Care.
These should be designed with engaging shapes and be visually distinctive.

Clickable Elements with Animation:

Upon clicking a service category, there should be a subtle yet noticeable animation to enhance user interaction.
Clicking on any service category should tell about the particular service offered by salon.

Consistency with Website Theme:

The design of this section must align with the overall theme and aesthetic of the website.
The color scheme, typography, and graphical elements should be consistent with the website's branding.

Acceptance Criteria:

Visual Appeal and Engagement:

Each service category (Hair Care, Skin Care, Custom Care) must have a unique, visually appealing representation.
The design should encourage user interaction and be intuitive.

Functionality of Clickable Elements:

Clickable elements must function correctly, with each leading to its respective detailed service section.
The animation upon clicking should be smooth and not overly distracting.

Design Consistency:

The overall design of this section must seamlessly integrate with the rest of the website.
Colors, fonts, and graphical styles must be in line with the website's established design language.

Responsive Design:

The services section should display correctly on all devices (desktops, tablets, smartphones).
Elements should adjust in size and layout according to the screen size without losing functionality or aesthetic appeal.

Created the services section of website with visuallt appealing design and services like Hair Wash, Skin care etc


Repo sync for protected CLA branch

The pull request is created from main638005361542591179sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit a1c2695:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


[M8] Hive/OCM CRUD Operations, rebased

A rebase of #2332, opened for CI

thanks for the rebase Amber. I rebased my PR after reviewing this/your branch and will close this PR now as it looks like I also was missing a make client to update the swagger spec/sha hash.


build(nodejs): remove 14 support

What's included

build(nodejs): remove 14 support

Notes

remove nodejs 14 support, 16 is the lowest

How to test

Coverage and basic unit test check

update the NPM packages with $ yarn
$ yarn test
confirm tests come back clean

Example

...
Updates issue/story

ongoing

Codecov Report

Merging #59 (4966207) into main (3a00958) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main      #59   +/-   ##
=======================================
  Coverage   92.14%   92.14%
=======================================
  Files           5        5
  Lines         242      242
  Branches       98       98
=======================================
  Hits          223      223
  Misses         18       18
  Partials        1        1

Continue to review full report in Codecov by Sentry.

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


ServiceNow tool

@njhale I think this is in progress?

@njhale I think this is in progress?

@cjellick yes it is, missed updating this one.


Installation / Usage manual link is 404

Original issue: https://charm.cs.illinois.edu/redmine/issues/103

The manual section on building and running Charm++ got moved into an appendix of the Charm++ manual. The website still links to a separate spot, though, which now produces a '404 Not Found' error.

Original date: 2013-05-03 03:26:13

We've got a whole bunch of tutorial presentations coming up in the next few weeks, and people may look at our documentation in advance of attendance. This needs to get fixed, the sooner the better. It shouldn't take very long, either.

Original date: 2013-05-14 18:10:44

Could you correct the link from https://charm.cs.illinois.edu/help to point at the current URL, http://charm.cs.illinois.edu/manuals/html/charm++/A.html

Original date: 2013-05-14 18:14:18

The "Tutorials from 2008 Charm++ Workshop" link is badly munged, too. There are other 404s for the various FEM-related manuals, and for Charisma, but those are less critical to resolve.

Original date: 2013-06-04 05:49:54

Ping; this isn't something that we should leave broken, since it can put off prospective users rather quickly.

Original date: 2013-06-26 20:02:11

Just moving so this isn't visible as part of the release list.


[5.1] Support unavailable and obsoleted attributes on extensions

Members of an extension should inherit the availability of the extension. This was previously supported for introduced and deprecated, but not unavailable and obsoleted.
PR on master #24997

@swift-ci Please smoke test

@swift-ci Please test source compatibility


Justine/logout command

To Test:

Log in and type command "/logout"

Should log you out and redirect to the login page

Log in again and enter a room, send messages, and then type "/logout" in the chat

Should log you out and redirect to the login page

Tests as expected. '/logout' command works both in the lobby and in a chatroom. :shipit:

The logout command works in Chrome, IE, and Firefox, but doesn't work in Edge.


Add commatose by glass beach

Song Name
commatose
Artist Name
glass beach
Source
Youtube
Youtube Link
T4t1vYNY0mA
Direct File Link
No response
Song ID
1362953
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 110653463
Downloaded: No

reject
already in the mod


Consider using Flask Restless to build an API

Flask-Restless — Flask-Restless 1.0.0b1 documentation
Python REST API Framework — Eve 0.6.5.dev0 documentation

I played with Flask Restless recently, it looks better than Eve.

This issue has become obsolete. See #135.


Whether to consider using pb to define api ？

Is your feature request related to a problem? Please describe.
Now the api uses a set of syntax created by go-zero, whether it can support the use of pb files to generate api, the purpose is to use pb to easily replace api. Some api special functions can be realized through pb opt.
Describe the solution you'd like
Use pb instead of writing api files.
Describe alternatives you've considered
After pb is written, you can directly use pb reflection to parse the pb file and generate code similar to that generated by the previous api file.

You can use pb in gateway, but not for API files.

But pb gateway cannot support middleware jwt ......


[Improvement][定时任务]定时任务添加多时区支持

*For better global communication, please give priority to using English description, thx! *
Please review https://dolphinscheduler.apache.org/en-us/docs/development/issue.html when describe an issue.
Describe the question
A clear and concise description of what the improvement is.
What are the current deficiencies and the benefits of improvement

A clear and concise description of the current deficiencies and the benefits of this improvement.
在设置任务的定时调度时间时，发现一个痛点，定时默认只能按照服务器时间来设置，如果需要使用多个不同时区的时间来调度，只能自己在原服务器时间的基础上，按照不同时区进行增减小时数来设置定时时间，不方便。比如服务器时间为0时区时间，但是用户时区为8时区时间，希望按照8时区时间来定时

Which version of DolphinScheduler:
-[1.2.1-preview]
Describe alternatives you've considered
A clear and concise description of any alternative improvement solutions you've considered.

please update the title in English


unable to deploy speaker with operator due to podsecurity

im having a similar issue as to whats describe in a post in the main metallb repo https://github.com/metallb/metallb/issues/1457
the speaker pods wont run due to podsecurity
this can be fixed using namespaces in yaml and helm, but not sure how to fix this with operator?
where do i add the following lines?
 labels:
    pod-security.kubernetes.io/enforce: privileged
    pod-security.kubernetes.io/audit: privileged
    pod-security.kubernetes.io/warn: privileged

When you deploy the operator you apply manifests, one of which is the "metallb-system" Namespace resource.
You need to add those labels to this yaml. like we have here:  link

its ok i have moved away from metalb now, thanks for the help tho


Make ICustomDrawOperation to use ImmediateDrawingContext

What does the pull request do?

Changes ICustomDrawOperation to use ImmediateDrawingContext as a rendering context instead of Impl interface.
Removes IDrawingContextImpl.Custom method as it seems useless after refactoring. @kekekeks please correct me if I am wrong. This method also wasn't used before, i.e. ICustomDrawOperation.Render(IDrawingContextImpl) was called directly from non-impl classes.

ImmediateDrawingContext usage was copied from the custom composition visual, but I wonder if it's fine to allocate this wrapping context on each render call. Either way this implementation detail can be improved later.
Fixed issues
Fixes #11160

You can test this PR using the following package version. 11.0.999-cibuild0034039-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

Hey guys, just wondering why you need to introduce so many breaking changes. It really hurts.

@MaryanneLynch yeah for 11.0 we had to do this. The API grew in the past when it was a side project first. Now was the point in time where we tried to iron things out while making the framework both, faster and more stable. So we hope that 12.0 will have way less breaking changes. In the 11.x timeframe we try to avoid them completely unless there is a very very good reason.
Overall we think that updating to 11.0 is worth the effort. However, feel free to stick with 0.10 if you can't effort the needed work.
Hope you enjoy Avalonia anyway
BR Tim


AzureOpenAI content filtering gives unclear error

Issue
Using dspy-ai==2.4.5, when the AzureOpenAI API's content filtering is triggered, DSPy returns a very unhelpful error.
AttributeError: 'NoneType' object has no attribute 'strip'

From within/site-packages/dsp/primitives/predict.py:78.
Expected behaviour
The AzureOpenAI LM class should throw a clear error stating that the content has been filtered.

Are you open to pull requests here? Happy to make this change myself.

Thanks @erst-neaste, please do contribute PRs!
Could you also share snippets of related code and the full error stack trace? This seems to be an error in configuring your LM with your DSPy program.

This code will replicate the issue:
gpt3 = dspy.AzureOpenAI(
    deployment_id=gpt3_model_name,
    api_key=gpt3_openai_api_key,
    api_base=gpt3_azure_endpoint,
    model_type="chat",
    api_version=openai_api_version,
    temperature=0.1,
    max_tokens=4096,
)

dspy.settings.configure(lm=gpt3)
signature = dspy.Predict('input -> output')
signature(input='Translate "period is over" to danish')

Stack trace:
File "/usr/local/lib/python3.11/site-packages/dspy/predict/predict.py", line 49, in __call__
    return self.forward(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/dspy/predict/predict.py", line 91, in forward
    x, C = dsp.generate(template, **config)(x, stage=self.stage)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/dsp/primitives/predict.py", line 78, in do_generate
    completions: list[Example] = [template.extract(example, p) for p in completions]
                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/dsp/primitives/predict.py", line 78, in <listcomp>
    completions: list[Example] = [template.extract(example, p) for p in completions]
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/dsp/templates/template_v2.py", line 152, in extract
    raw_pred = raw_pred.strip()
               ^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'strip'

Thanks for opening this! We released DSPy 2.5 yesterday. I think the new dspy.LM and the underlying dspy.ChatAdapter will probably resolve this problem.
Here's the (very short) migration guide, it should typically take you 2-3 minutes to change the LM definition and you should be good to go: https://github.com/stanfordnlp/dspy/blob/main/examples/migration.ipynb
Please let us know if this resolves your issue. I will close for now but please feel free to re-open if the problem persists.


fix(test): fix hanging by channel for kong.response.exit

Fix #169.
Channel stateChange isn't buffered channel. The e.Finish() method writes the  channel, but the consumer is in the same go routine. So it causes a dead lock.
Changing the channel to buffered can solve the issue.

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

I think this is good to go, but @gszr could you take another look?

@gszr @nowNick I've added these changes as well as the requested test to my PR (#174) that addresses the hanging channel issue and adds support for shared ctx handlers in the test utility.
In the interest of getting this breaking change fixed could we look at moving forward with my PR? Would you rather I break the PR into two separate PRs (one for hanging channel issue, one for shared ctx handlers)?
I'm only persistent in bringing this up because in its current state, the testing utility does not work and prevents developers from moving forward with the tool.

Closing this in favor of #174.


Backend - RSVP - Allow events to hold attending users

Set up event model to hold attendee rsvps.

Events put method needs to be able to update users_attending

***************TESTING

event model appropriately allows event to have a manyToMany relationship w/ User model
-front end has RSVP button to correspond (an issue is already made to deal with allowing users to un-rsvp to an event)


fix: improve examples for Android Native Modules addListener/removeListeners

Improve Android Native Module examples for using addListener  and removeListeners with pattern that mimics the one in RCTEventEmitter on iOS. The current examples are not self explanatory, as removeListeners is called with not-obvious count parameter. I've sent 1 hr figuring out how all of these should work and want to save other readers from having to do the same research.

@mdjastrzebski Thank you so much for taking the time to add these details in all the versioned docs!


docs.helm.sh: can't scroll with keyboard in firefox

On docs.helm.sh using firefox, I can only scroll using my mouse's scroll wheel.  Arrow keys, page up/down are both broken.

re-opened at https://github.com/helm/helm-www/issues/139. Thanks!


StorageSync ResourceProvider : Bug Fix to change parameter name

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-js

@azuresdkci Retest this please

@jhendrixMSFT How can we fix the issues with go build here

This is the pertinent part of the error. "FATAL: System.InvalidOperationException: codegen for preview swagger Microsoft.StorageSync/preview/2018-10-01/storagesync.json must be under a preview subdirectory"
I went ahead and fixed up the readme.go.md file.

Thanks Joel, appreciate the help! :)

@AutorestCI rebuild azure-sdk-for-ruby

Automation for azure-sdk-for-ruby


Issues with importing AzureAD Applications

Hello,
I am getting a lot of errors when I try to import AzureAD applications. Details are below:
Terraformer Version:  v0.8.24
Terraform Version: 1.5.3 also tried with 0.13.5
Azure AD Plugin Version: 2.53.1 also tried with 2.44.1
Running all on MacOS Sonoma 14.5. Everything is installed via homebrew. I do have sufficient permissions on azure AD
When running terraformer import azuread --resources=application I get the errors below:
2024-07-30T15:56:43.842-0400 [ERROR] plugin.terraform-provider-azuread_v2.44.1_x5: Response contains error diagnostic: @module=sdk.proto diagnostic_severity=ERROR tf_proto_version=5.4 tf_resource_type=azuread_application @caller=github.com/hashicorp/terraform-plugin-go@v0.19.0/tfprotov5/internal/diag/diagnostics.go:58 diagnostic_detail="" tf_req_id=73c06693-0e4f-1548-5fca-0bcf795f445f
  diagnostic_summary=
  | parsing "796c5e6d-caf9-44e5-91ce-a10807ede7e1": parsing the Application ID: the number of segments didn't match
  |
  | Expected a Application ID that matched (containing 2 segments):
  |
  | > /applications/00000000-0000-0000-0000-000000000000
  |
  | However this value was provided (which was parsed into 0 segments):
  |
  | > 796c5e6d-caf9-44e5-91ce-a10807ede7e1
  |
  | The following Segments are expected:
  |
  | * Segment 0 - this should be the literal value "applications"
  | * Segment 1 - this should be the user specified value for this applicationId [for example "00000000-0000-0000-0000-000000000000"]
  |
  | The following Segments were parsed:
  |
  | * Segment 0 - not found
  | * Segment 1 - not found
   tf_provider_addr=provider tf_rpc=ImportResourceState timestamp=2024-07-30T15:56:43.842-0400

2024/07/30 15:56:43 ERROR: Unable to refresh resource tfer--KnowBe4-0020-Security-0020-Awareness-0020-Training
2024/07/30 15:56:43 Filtered number of resources for service application: 0
2024/07/30 15:56:43 azuread Connecting....
2024/07/30 15:56:43 azuread save application
2024/07/30 15:56:43 azuread save tfstate for application

Same issue here. Looks like Terraformer is not correctly forming the Application ID for the Terraform input commands when parsing through the list of application ObjectIDs it detected. At a glance it appears a simple fix of appending '/applications/' to the front of each ObjectID retrieved somewhere before 'return resources' on Line 36 of application.go would do the trick.

I'm still not having any luck with this. I have been trying to modify application.go to add the needed "/applications/" segment that the Terraform import commands require. I have been playing around with the appendResource function in my attempts to no avail. Alas, my development knowledge is limited and I've never used Go, so I've been trying to wrap my head around pointers vs values without any success. For algorithm I'm trying to take resource.ID, append the needed string segment to the front of it, and pass the new value as id to az.appendSimpleResource. I imagine it takes something like storing the value from the pointer in a temp variable, storing the address from the pointer, updating the temp value, and overwriting the value at the pointer address. If anybody is familiar with how to do that with Go, I'd love some assistance. :)

Created a PR that fixes this issue. I tested this and it worked for me. I was able to terraform existing applications. If you can't wait until it is merged. Download the source, update the application.go file, and overwrite the func at line 39 with the code below:
func (az *ApplicationServiceGenerator) appendResource(resource *msgraph.Application) {
	id := resource.ID
	// Prepend "/applications/" to the application ID
	fullID := "/applications/" + *id
	az.appendSimpleResource(fullID, *resource.DisplayName, "azuread_application")
}

Then build using go mod download and build using go build -v once built, move the binary from the terraformer repo to user usr/local/bin and then run the terraformer command to terraform the applications. Let me know if this works for you

Thank you! This fix worked like a charm. I had been doing that exact thing but missed the step about copying the new binary over to usr/local/bin. I knew it had to be something simple and turns out it was my very limited exposure to Unix/Linux OS instead ha. I appreciate the quick response. Sorry it took so long to test it out in my environment. I had been detained by a more pressing issue last week.

Thanks a lot @max-blue
It's the same thing for all other resources by the way (group, user..)


feat(m-extensions): Adds divu and remu

We are now adding DIVU and REMU instructions as part of the M-Extensions

This was merged into the main jolt repo


T0 JobAccountant crashes due to multiple jobs generating same logArchive files

Impact of the bug
JobAccountant crashes due to logArchive duplicated files.
Describe the bug
Two or more jobs create output files with the same name. JobAccountant tries to add them to the DB and fails due to a ORA-00001: unique constraint (WMBS_FILDETAILS_UNIQUE) violated
How to reproduce it
Deploy a Tier0 replay with a significant amount of jobs. As soon as the firsts batches of Express jobs finish, the JobAccountant will crash.
Expected behavior
Each job should create files with unique names
Additional context and error message
This is tightly connected to #10870. I decided to create a new issue because the mentioned issue focused on the issue with ROOT files, which will require a fix at the CMSSW level. See issue:
github.com/cms-sw/cmssw/issues/37240
However, we have also seen with T0, duplicated lfns that involve the logArchive executor
    "dup_lfns": [
      "/store/unmerged/data/logs/prod/2021/10/13/Express_Run343082_StreamCalibration_Tier0_REPLAY_2021_v2110131547_211013_1547/Express/0001/0/Express-ac5ffd57-dc01-493f-ba17-951d97d7a523-0-logArchive.tar.gz",
      "/store/unmerged/data/logs/prod/2021/10/13/Express_Run343082_StreamCalibration_Tier0_REPLAY_2021_v2110131547_211013_1547/Express/0001/0/Express-ac5ffd57-dc01-493f-ba17-951d97d7a523-0-logArchive.tar.gz"
    ]

This is coming from:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/Steps/Executors/LogArchive.py#L283-L287
In this case, we generate the uuid generated by `self.job['name']
https://github.com/dmwm/WMCore/blob/1bb206d606c4acc09c29a19c08a057a51de2c235/src/python/WMCore/WMBS/Job.py#L69-L71

Update: There was a bug in the script that finds these duplicated LFNs. After this fix:
https://github.com/amaltaro/ProductionTools/commit/af153ac26bea0d6437e24227719338247826007c
No records of duplicated logArchive files have been found so far, so I'm just closing this issue.
If we do spot a duplicated logArchive related LFN, we can reopen this issue.


Remove semvar PATCH version number

The patch number is by definition only for non-breaking changes i.e.,
security updates, we should not explicitly state the semvar patch
number in our manifests.

bors r+


set up gallery with DemoCards

This is a similar PR to https://github.com/JuliaPlots/PlotDocs.jl/pull/255 and I guess maybe you'll also be interested in it.
The demos are organized into folders, and the folder structure is mapped as the gallery sections/cards.
One caveat: The cover is generated on the fly, which means we have to run demo.jl once to generate the png file, and the second time to generate demo.ipynb, and finally Documenter will run it for the third time. Thus the build time might be 3x longer. I have plans to make ipynb optional but this package is not of very high priority, to be honest.
There's a broken docs/gallery/gallery/basic/makie_embedding.jl.bak file, which I don't know how to generate the cover. Thus I'll leave it to you :)
The docs make.jl seems broken at present so I can't build it successfully, here's the result of DemoCards.preview_demos("docs/gallery")

Other live DemoCards examples:

JuliaImages: https://juliaimages.org/dev/examples/
Augmentor.jl: https://evizero.github.io/Augmentor.jl/dev/operations/

Thanks for the PR! I'm picking this up in #260 (so that I can do it together with some docs reorg I had in mind).
The build times did get a bit worse with this, it'd be really cool to at least save on the jupyter notebook generation. I guess making it optional is just a matter of adding a flag somewhere in DemoCards, or is it more complex than that?
The "broken docs" are because one of the democards (with geographic data) has a footnote, and in strict mode documenter throws an error (see https://github.com/JuliaPlots/AlgebraOfGraphics.jl/pull/258/checks?check_run_id=3302846071#step:6:29).
Can this be fixed on the DemoCards side somehow, to support cards with footnotes without throwing?
In non-strict mode, everything seems to work but with a warning. Unfortunately here the docs are also a "test suite", so I'd prefer to keep the strict mode.


Display controls on hover

Use CSS only for displaying/hiding dropdown icons
Separate render logic for displaying dropdown menus

cheers for https://github.com/x-team/auto-ui/pull/182/commits/c577a9297bffb15288ed8f0771a28e8447c2d4e6 @bernardodiasc :clinking_glasses:

Updated this PR to use conditional props for displaying icons.