Socket.recv_multipart is missing from typestubs

Socket.recv_multipart is missing from typestubs. I believe it should go to to zmq/backend/__init__.pyi.

recv_multipart is defined in zmq/sugar/socket.py (the low-level backend does not have multipart), and imported in zmq/sugar/__init__.pyi. Where do you see it being missing? All my tests show type checks working with recv_multipart, but I don't know how everybody works with types.
You might check with 23.0.0b2, which has some fixes for typing.

Hmm, I see. PyCharm tells me it is unresolved. Maybe it got confused since both classes are called Socket and the one from backend is renamed on import to SocketBase. Now going through the code more carefully I also think is should work. However, I do not have much experience with typing.
So it might be something to report against PyCharm?

I do see Any for 22.3, but I get fully typed info for 23b2. With this script:
import zmq

ctx = zmq.Context()
s = ctx.socket(zmq.PULL)
reveal_type(ctx.socket)
reveal_type(s.recv_multipart)
msg = s.recv_multipart()
reveal_type(msg)

mypy 0.942 gives the correct type of msg to be list[bytes]. So maybe it's a PyCharm issue? Or maybe you could give a script example where it gets confused? I'm pretty new to types (I don't use them much myself) and the generator/inheritance stuff in pyzmq is unkind to the strictness typing tools expect.

So maybe it's a PyCharm issue?

That seems to be the case. It must be using some kind of lazy indexing system that got confused - it noticed the constants being available right after installing 23b2 but it didn't correctly decipher the return type of ctx.socket. After some time and/or PyCharm restart, now it does not complain anymore. So for me it :+1: for types in 23b2 and sorry for bothering you guys - I just thought I'll try to make pyzmq better and didn't know the context well enough.

For posterity - the PyCharm snap on ubuntu updated itself between the first and second test so it might have been fixed on the PyCharm side (the current version being 2022.1.1 (Community Edition)).


Fixing issue when subclassing models

This change creates related names that contain the class name, so that related names won't clash when subclassing taggit models.

Closing due to inactivity.


Update start and end dates for budgets

When updating a budget we need specific changesets and triggers for updating the

[x] start and end date of a budget
[x] amt_in and amt_out of a budget

These can have been covered in the update_budget action of the the money_logs context here it is a question of passing in the correct attributes when updating


The description of how to delete an Azure Migrate project is incomplete.

Hello dear developer friends who are expanding these very helpful Docs.
The description of how to delete an Azure Migrate project is incomplete in my opinion.
Under the item "How do I delete Azure Migrate projects and associated Log Analytics workspaces?" it is not described how to delete a actual Azure Migrate project. The deletion of the Log Analytics workspaces is kinda described there, but not the deletion of the Migrate project.
It doesn't seem to work via the Web Portal, because I couldn't find an option for it anywhere.
I was able to remove the resource using the Powershell CMDlet Remove-AzResource.
Thank you so much for your efforts!

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

ID: 4e91f6f9-23d7-da62-b7f9-4f77e1924653
Version Independent ID: 813a0912-51f8-13c0-2096-efe6c9fe1514
Content: Troubleshoot Azure Migrate issues
Content Source: articles/migrate/troubleshooting-general.md
Service: azure-migrate
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

Now added. Thanks!
#please-close


Update on_deliver* and add new on_session_expired hook

Add new on_deliver* hook variant to include QoS and Retained flag modifiers and new on_session_expired hook.
Part of https://github.com/vernemq/vernemq/issues/1346

Looks good to me. Thanks :)
One thing I have to understand is how the "retain as publish" thing might interfere in the v5 version of the on_deliver hook you propose. (https://github.com/vernemq/vernemq/blob/d4c54592305d3ff609e6865490e39db3e26538a7/apps/vmq_server/test/vmq_retain_SUITE.erl#L389)

Good point @ioolkos  I'm gonna write a test and check that later. Another thing I'm concerned is that people with already developed plugins and upgrade vernemq_dev will have breaking changes. I'm not quite sure what is a good answer here but i've mentioned before that we need a changelog and versioning for this package to avoid these things.

Thanks, agree. Should we add a tag/release for the vernemq_dev master as is, and then add a tag after your PR is merged?

Thinking about how to express this on the plugin development documentation...

Created a 1.0.0. release, so I think I can merge this PR.

Sounds good to me @ioolkos . A bit busy today but feel free to merge.


Try not to change line endings when editing files

Is your feature request related to a problem?
When writing file edits try not to change the line endings.
The tool might be run on a different operating system to the native line endings of the files, and whatever is used should be retained where possible, rather than relying on the behaviour of Environment.NewLine.
Describe the solution you'd like
Add a helper to try and detect the line endings in a file (\r\n vs. \n) and emit the right line endings as required (e.g. when updating a Dockerfile which uses File.WriteAllLinesAsync()).
Describe alternatives you've considered
No nothing, let users handle their own conflicts.
Additional context
None.

This is mostly resolved by #48, but some of the JSON file edits don't currently support preserving the line-endings (or comments) as the JsonNode APIs don't support that.


Switched to using dns and http clients from clustertest

What this PR does
Fixes https://github.com/giantswarm/giantswarm/issues/31135
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

🤔  Looks like I broke the http client. Maybe it using the resolver isn't right. I'll do some more testing.

/run cluster-test-suites

/run cluster-test-suites TARGET_SUITES=./providers/capa

/run cluster-test-suites TARGET_SUITES=./providers/capa

Looks like CAPA China is playing up today but other test suites are passing ok so I'm going to add the skip label.


ARROW-12907: [Java] Fix memory leak on deserialization errors

CC @liyafan82 or @lidavidm would you mind reviewing?

In practice this can happen if the thread running this code is interrupted.

CC @liyafan82 or @lidavidm would you mind reviewing?

@emkornfield Thank you for working on this. I will take a look two or three days later, when I come back from vocation.

Good catch, the fix looks good.

Merging. Thanks for the PR. @emkornfield


Additional styling

I've added some extra styles to my config. Maybe you'd be interested in using them here?
Here's all of the visual styling that I have in place:
# https://draculatheme.com/kitty
foreground #F8F8F2
background #282A36
selection_foreground #282A36
selection_background #F8F8F2
color0     #000000
color8     #4D4D4D
color1     #FF5555
color9     #FF6E67
color2     #50FA7B
color10    #5AF78E
color3     #F1FA8C
color11    #F4F99D
color4     #BD93F9
color12    #CAA9FA
color5     #FF79C6
color13    #FF92D0
color6     #8BE9FD
color14    #9AEDFE
color7     #BFBFBF
color15    #E6E6E6

# URL styles
url_color #BD93F9
url_style single

# Cursor styles
cursor #E6E6E6
cursor_shape beam

# Tab bar colors and styles
tab_fade 1
active_tab_foreground #282A36
active_tab_background #F8F8F2
active_tab_font_style bold
inactive_tab_foreground #F8F8F2
inactive_tab_background #282A36
inactive_tab_font_style normal

Thanks for sharing this @cweagans

@cweagans sorry I missed this issue! Please send a PR and I'll merge it in.

@keegancsmith done


How know encode type when I parse content a mail?

I have created a mail app using Postal Framework and my mail server send the message to app with different type data, so how I know a data in body is encode/decode by type?
In my example I don't know data should encode with NSUTF8StringEncoding or NSISO2022JPStringEncoding :(
msg.body?.allParts.forEach({ (singlePart) in
    let meesage = String.init(data: (singlePart.data?.rawData)!, encoding: NSUnicodeStringEncoding)
    if singlePart.mimeType == MimeType.textPlain {
        mailInfo.messageDescription = meesage?.stringByTrimmingCharactersInSet(NSCharacterSet.whitespaceAndNewlineCharacterSet()) ?? ""
    } else if singlePart.mimeType == MimeType.textHtml {
        mailInfo.message = meesage ?? ""
    }
})

Thanks for support!

Sadly Postal give you only a few encoding type as referred here: https://github.com/snipsco/Postal/blob/fad90b6a8f8dd2cbb9002f9823c43ef99bbc21ae/Postal/ContentEncoding%2BParsing.swift#L28 .
It's clearly an issue and I suggest you to try to find the encoding type manually. If using NSString isn't an issue for you I suggest you to use the method NSString.stringEncoding(for:encodingOptions:convertedString:usedLossyConversion:).
Documentation here: https://developer.apple.com/reference/foundation/nsstring/1413576-stringencoding


add a modified version of process.py from pysilcam

this PR adds most of the functions within pysilcam.process and pysilcam.postprocess (now call statistics.py)
functions making use of pysilcam config settings should now be taking these arguments directly instead of from the settings. References to settings should now be removed.

Just to check - should this file be called process.py? I think that was the case, but double check, that this is process and not supposed to be called something else.
I also think my review here is somewhat unqualified, as I don't fully understand how this has changed silcam's process.py, but assume you just took out the relevant parts, but left those parts unchanged.

Yes, looking at my notes, I think we wanted to call this process.


Enter the client and flash a white screen

Enter the client and flash a white screen

Hi @zhangxiansheng123,
I am currently focused on fixing this issue, which I believe might be caused by an error with the database file, as discussed in PR #1711. However, I am not certain about the exact cause in your case. Please follow these steps:

If your data does not involve privacy concerns, you can share the MQTTX.db file with me at<EMAIL_ADDRESS>for further investigation.
Delete the MQTTX.db file:

Linux: ~/.config/MQTTX/
macOS: ~/Library/Application Support/MQTTX/
Windows: %USERPROFILE%\AppData\Roaming\MQTTX/

Restart the app.
If you can still access the settings page, please click on data export to check for the error cause.
Let me know your current version, and if necessary, upgrade to the latest version.

Thank you for your feedback.
Best regards

The issue has been fixed in https://github.com/emqx/MQTTX/releases/tag/v1.10.1. Please update and use it. Thank you.


update NEWS

I went through all the pr's since the last version and tried to update NEWS.md with the more important changes. Since we are now about 50 pull requests ahead of the last version I thought it would be helpful to highlight a few bigger changes in a separate section. I also moved the current changes back under ## master. Feel free to make changes to whatever needs them.
Here's a list of all pull requests merged since 0.15.3. I marked the ones that are mentioned, added a pr number (without the hashtag to avoid pinging all of them) and added a note on what is affected (probably not entirely accurate).

[x] 1393 breaking release

[x] 1192 cleanup scenes, boundingboxes, compile time latency [Makie]
[x] 1222 logical expressions for ispressed (more programmable hotkeys) [Makie]
[x] 1223 simplify plotting barplot by group [Makie]
[x] 1307 Node -> Observable [All]
[x] 1382 Add depth_shift (basically a fine grained overdraw) [GLMakie, WGLMakie]
[ ] 1392 fix order dependence of LScene and Axis (805) [Makie]
[x] 1394 restrict Camera3D to last clicked/added scene (fixes scenes fighting over control) [Makie]
[x] 1395 add depth to 3D contours and isosurfaces (i.e. allow other objects to intersect with them) [GLMakie]

[ ] 1413 add code of conduct [Pkg]
[x] 1404 cleanup volumeslices (more control over attributes) [Makie]
[x] 1411 cleanup menu render order & priority [Makie]
[x] 1401 add rotation arg to labels in barplot [Makie]
[ ] 1408 improve latex docs [Docs]
[x] 1260 allow lines of offsetarrays [Makie]
[ ] 1421 fix camera replacement  [Makie]
[x] 1438 add resize_to_layout! [Makie]
[x] 1434 cleanup volume lighting [GLMakie, WGLMakie]
[ ] 1425 clarify plotting in REPL [Docs]
[ ] 1440 document dodge and dodge_gap [Docs]
[ ] 1446 fix set_theme! [Docs]
[ ] 1447 fix trimspine for n ticks < 2 and add example [Makie]
[ ] 1441 add aspect tutorial [Docs]
[ ] 1445 fix applyconvert [Makie]
[ ] 1452 disable rectangle zoom if we don't have inverse [Makie]
[ ] 1453 fix single point line bug [CairoMakie]
[ ] 1457 add lineheight and justification [Makie, Docs]
[ ] 1456 remove used attributes [Makie]
[x] 1463 cycled color and automatic markercolor for scatterlines (fixes 1461 1306 1450) [Makie]
[x] 1310 Add ECDF plot [Makie]
[x] 1248 add Cycled object for direct cycler access [Makie]
[ ] 1473, 1475 Update colors.md [Docs]
[ ] 1477 Improve layout tutorial [Docs]
[ ] 1487 fix log ticklabels with very small or large exponents [Makie]
[x] 1418 Order independent transparency [GLMakie]
[ ] 1439 update NEWS.md and prepare breaking release [All?]
[x] 1494 Fix blurry text [GLMakie, WGLMakie]
[ ] 1488 error on colormap lookup with NaN, which should be handled elsewhere [Makie]
[x] 1504 Disconnect other cameras when switching to PixelCamera [Makie]
[ ] 1448 Fix recipe docs [Docs]
[ ] 1423 Fix UndefVarError: not_implemented_for not defined (1209) [Makie, MakieCore]
[ ] 1481 start adding tests for primitives [Makie/Tests]
[ ] 1474 Create PULL_REQUEST_TEMPLATE.md [Pkg]
[x] 1506 Make OIT less sensitive to avoid infinities (fix 1505) [Makie, GLMakie]
[ ] 1510 fix typo in port forwarding [Docs]
[ ] 1512 Remove redundant imports [All]
[ ] 1513 Fix occasional long ticks [Makie]
[ ] 1516 Default for DataInspector (use current_figure as default) [Makie]
[ ] 1499 Text layouting tests [Makie/Tests]
[ ] 1503 Fix remaining issues [All?]

[ ] change texture atlas font path, so users don't need to delete it manually
[ ] fix cairomakie 3d depth rendering issue
[ ] update new test images, make sure new basic tests pass on all backends
[ ] better unique test names, that are not depend on position in the file

[ ] 1518 Update volume() docstring [Docs]
[ ] 1435 bump compat for MathTeXEngine [REVERTED in 1489]
[ ] 1531 Typo in recipe docs [Docs]
[ ] 1526 Add documentation for attributes of primitive plots [Docs]
[ ] 1522 colormap documentation [Docs]
[ ] 1527 cleanup docs and last bug fixes [Docs, RPRMakie, Makie]

I removed the deprecation and added this PR & Release note to the NEWS to have a reference to all changes.
I think this is now mergeable :)

Going to merge this... Let me know if there was anything missing!


[vm/ffi] KeepAlive marker interface

We currently have a Finalizable interface that serves two purposes:

It keeps objects alive until the end of the syntactic scope.
It can have NativeFinalizers attached.

Due to (2), we disallow sending objects implementing Finalizable to other isolates. As that that will most likely be an error.
However, when attaching finalizers via the dart_api.h, we also need the behavior of (1), and we actually want to send objects to other isolates (that's why we resorted to using finalizers from the dart_api.h in the first place.)
We should introduce another marker interface that only has behavior 1.
Maybe the interfaces should even be related.
Ready for bike shedding on the name:
a. KeepAlive
b. IsolateGroupFinalizable
The marker interface should probably live in dart:ffi as well. Because it's mostly used when sending objects via FFI with Handles. Also it would enable relating the two interfaces, helping with documentation. But I can be convinced of other options.
Use case:

https://github.com/dart-lang/http/issues/1260#issuecomment-2223660760

cc @HosseinYousefi @mkustermann

@mkustermann: name option NativeFinalizale.

(I misremembered how this works in today's meeting)
The current setup is:

We can use Finalizer API with any dart object (no restriction) - under the assumption that invoked finalizer callbacks are delayed in event loop, delayed enough to ensure no finalization-before-use issues occur (**)
We can use NativeFinalizer API only with Finalizables

I think the main reasons we currently disallow Finalizers to be shared across isolates is that:

If the NativeFinalizer object is GCed then the finalizers are silently / don't trigger anymore
=> That is IMHO a mistake - if C code needs to free up something, we should invoke it, whether the NativeFinalizer object is still alive or not
=> It also leads to non-determinism as an unreachable NativeFinalizer may or may not be collected before the collection of Finalizables. So whether or not the callback is inovked is dependent on how GC happens to be implemented.
On isolate shutdown we eagerly run all native finalizers (only if the corresponding NativeFinalizers haven't been collected yet)
=> Any other isolate using the objects after isolate shutdown would have use-after-free error
Objects are copied when sending across isolates: Objects with native resources cannot be copied
=> We do have @pragma('vm:deeply-immutable') now for that.

So marking a class as extends Finalizable and marking it as @pragma('vm:deeply-immutable') allows sending/sharing across isolates but

is dangerous because isolate death will eagerly run NativeFinalizers and therefore possible create use-after-free scenarios.
is safe to do if one uses the C APIs - e.g. Dart_NewFinalizableHandle as those are a) tied to an isolate group and are b) guaranteed to run (neither of which are NativeFinalizers are not - see above)

Eventually (with shared multi threading) we want a finalizer API that works across isolates (just like our existing C API works across isolates). So one way would be to have

isolate-local resources & finalizer APIs - those prevent sending/sharing (shouldn't even work with deeply immutable objects)
non-isolate-local resources & finalizer APIs - those allow sending/sharing (should work with deeply immutable objects)

But this may require two different marker interfaces or otherwise leaves room for user bugs where objects that are isolate-local are shared.
We could go instead the other direction:

We establish a guarantee that native finalizers are always run
They run even if the NativeFinalizer object was GCed
Don't run them on isolate shutdown (but isolate group shutdown)

=> This would effectively align the NativeFinalizer behavior with that of the C API.
=> This would make it safe to allow sharing Finalizable & deeply immutable objects across isolates.
So the main question is: Do we really care about early reclamation of isolate-local resources on unexpected isolate shutdown? This is only important if isolates are short lived and native resources aren't eagerly freed up by user code (or e.g. isolates get often killed) and we cannot wait for the GC to eventually run them a bit later.
/cc @mraleph wdyt?
(**) This assumption only makes sense for synchronous code. In asynchronous code one can easily have
foo() async {
  final obj = Foo(field);
  finalizer.attach(foo, ...., () => ...);
  final field = obj.field;
  await <...>
  // ^^^^
  //   - takes long time
  //   - may cause GC
  //   - may collect `obj` (as it's not used anymore & it's not `Finalizable)
  //   - GC may enqueue finalizer,
  //   - may run finalizer
  useField(field);  // <-- use field after finalizer has run
}

=> This would effectively align the NativeFinalizer behavior with that of the C API.

That's not entirely true, canceling finalizers from another isolate doesn't work, from a previous discussion of ours:

However, detach would not work shared. Trying to detach from a non-shared static nativeFinalizer would lead to a no-op on another isolate. Because it would be a different native finalizer (that happens to have the same callback address.)

https://github.com/dart-lang/sdk/issues/55511#issuecomment-2066124750

Do we really care about early reclamation of isolate-local resources on unexpected isolate shutdown?

I don't know of any such use cases. So I like the idea of trying to make NativeFinalizer shared always instead of introducing a shared native finalizer later, from another previous discussion of ours:

https://github.com/dart-lang/sdk/issues/55062

I believe we could make the existing NativeFinalizer sharable (and thus make detach work from any isolate) by making it's backing store for entries (a map), concurrency safe. (We would need to do the same work for introducing a SharedNativeFinalizer later anyway.)

That's not entirely true, canceling finalizers from another isolate doesn't work, from a previous discussion of ours:

The code may not care about detachment at all, it may only detach on the main isolate anyway, helper isolates could ask main isolate to detach or it may (in future) use static shared NativeFinalizer = ... (in which case one gets the same semantics as the C api)

Trying to detach from a non-shared static nativeFinalizer would lead to a no-op

Wouldn't it throw a nice exception: "You cannot detach object foo from this finalizer as it was not attached" ?

I believe we could make the existing NativeFinalizer sharable (and thus make detach work from any isolate) by making it's backing store for entries (a map), concurrency safe. (We would need to do the same work for introducing a SharedNativeFinalizer later anyway.)

Making NativeFinalizer shared (i.e. make them work in upcoming shared multithreading shared isolates) isn't necessary for this.
What is necessary is committing to the semantic changes of NativeFinalizer: Make them always run (even if NativeFinalizer object is GCed), make them not run on isolate shutdown.

Right, if you cannot send the NativeFinalizer to another isolate, you cannot try to detach there. 👍
➕  For changing NativeFinalizers to be run on isolate group shutdown, and not GCable themselves if they still have attachments.

Wouldn't it throw a nice exception: "You cannot detach object foo from this finalizer as it was not attached" ?

It does not. https://github.com/dart-lang/sdk/issues/56632


Include missing RP label

Codecov Report

Merging #1570 (96fe78b) into main (fac92d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1570   +/-   ##
=======================================
  Coverage   95.00%   95.00%
=======================================
  Files           2        2
  Lines          20       20
=======================================
  Hits           19       19
  Misses          1        1

Flag
Coverage Δ

dotnet
95.00% <ø> (ø)

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


Incorrect stub generation

Phalcon 2.0.9
Phalcon\Session\AdapterInterface
    /**
     * Sets session options
     */
    public function setOptions(array! options);

used zephir stubs
Expected
    /**
     * Sets session options
     */
    public function setOptions(array $options);

Actual
    /**
     * Sets session options
     *
     * @param array $options
     */
    public function setOptions($options);

Issue still actual

This issue was moved to phalcon/ide-stubs#4


Actual type of dwResult

In this structure dwResult is DWORD, that is , plain win32 error code. But IBidiRequest::GetResult() specifies it as HRESULT, a COM error code. Which one is semantically correct ?

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

ID: 2d752c7c-6018-3d0b-055e-87a06a0f505d
Version Independent ID: 05e640d9-06bd-6f00-e681-0b19b985b970
Content
Content Source
Service: unspecified

Self-assigned @barrygolden for investigation.

Contacted the print driver feature team for clarification on construct elements in the _BIDI_RESPONSE_DATA structure and the IBidiRequest::GetResult method.

Opened internal work item 16710618 to address this issue. Will republish topic when the changes to clarify error code type have been completed,


Use transparent theme for compose lambda

Fix https://github.com/takahirom/roborazzi/issues/67

The failure of unchanged check test / test (pull_request) is OK. It is expected to fail.

It might be useful to demonstrate how to get back to a Theme background, should this be an option?

I think the new Compose function hasn't been widely used yet. People can create their own custom captureRoboImage(). I'm going to mention this PR comment in the release notes.
fun captureRoboImageNoTransparent(
  file: File,
  roborazziOptions: RoborazziOptions = RoborazziOptions(),
  content: @Composable () -> Unit,
) {
  if (!roborazziEnabled()) return
  // Use non-transparent activity
  val activityScenario = ActivityScenario.launch(ComponentActivity::class.java)
  activityScenario.onActivity { activity ->
    activity.setContent {
      content()
    }
    val composeView = activity.window.decorView
      .findViewById<ViewGroup>(android.R.id.content)
      .getChildAt(0) as ComposeView
    val viewRootForTest = composeView.getChildAt(0) as ViewRootForTest
    viewRootForTest.view.captureRoboImage(file, roborazziOptions)
  }
}

However, it seems difficult to view images without a background color. We are not currently using background colors in other parts, and users can use Rgb565 as a workaround. Therefore, I believe it is a good idea to use transparency in this PR.

Transparent works for me, once this is allowed you can choose any other strategy.  But critically if transparent isn't supported, it's hard to put it back later.
I actually found the background was the big difference between Screen and Component level screenshots. Maybe something you could support different strategies, Fill, 50PctAlpha, Transparent, DeviceTheme?
For example
https://github.com/google/horologist/blob/main/roboscreenshots/src/main/java/com/google/android/horologist/screenshots/ScreenshotTestRule.kt#L248
    @Composable
    private fun ComponentDefaults(
        fakeImageLoader: FakeImageLoader,
        content: @Composable () -> Unit
    ) {
        fakeImageLoader.override {
            Box(
                modifier = Modifier
                    .wrapContentSize()
                    .background(Color.Black.copy(alpha = 0.5f)),
                contentAlignment = Alignment.Center
            ) {
                DefaultTheme {
                    content()
                }
            }
        }
    }


Dynamically tuning [joint_trajectory_controller] PID gain

In ros1, the change of P parameters of effort_controller/JTC in runtime can be use to change the force of the joint. But in ros2, I have tried to change the P parameters using the /set_parameters service provided by the JTC in runtime, but the forece still remain same.
I just wonder, when the JTC in ros2 can have the ability to accept the change of the PID gains in runtime like ros1?

Yes, in iron and rolling.
Iron might not be released yet, you have to build it from source.
For humble you can use the the branch from this PR and build it from source.

Sorry, for iron it was already released with 3.18 too.

thanks a lot!!!!


Use separate gRPC client executor for workers

Workers handles long running blocking streams which will exhaust the common ForkJoinPool (by default cpu core - 1 parallelism) pretty quickly if user holds on to streams concurrently. This change makes the worker client use default gRPC executor which is not bounded by the number of CPU cores.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2103/
Test PASSed.


No way to transfer ownership of a library

I am the original author haxe-continuation, I want to transfer the ownership of https://lib.haxe.org/p/continuation to @AxGord .
If it is not possible in the cli program,  @Simn could you please modify the database of https://lib.haxe.org/ for the transfer?

Afaik, You can append the new maintainer user name in haxelib.json and release that as new version. After that the new maintainer can release under his account.

Thank you!

@AxGord Could you please register a user in haxelib, so I can append you as an owner?

@Atry Yes, my user in haxelib: https://lib.haxe.org/u/AxGord/


fix(turn_signal, lane_change, goal_planner): add optional to tackle lane change turn signal and pull over turn signal

Description
General, update the turn signal decider to align the rule with traffic rules.
Lane Change
For lane change action, the lane change will decide whether there is a left/right lane for the current lanelet or the next lanelet.
There are cases when the lane change starts at a point inside a lanelet without a left/right lane, but the target lane is beside the "next" lanelet.
So whether shiftSideLane exists should be decided by the Lane Change module, instead of the turn signal decider. (If there is no shiftedSideLane, Lane Change will not act instead). So the LC module should skip that judgment.
Goal Planner
When the goal planner is pulling over by the side of a wide lane, it should turn on the turn indicator no matter whether there is a shift side lane or the path will cross the lane boundary. So the goal planner module should skip that judgment.
Related links
Parent Issue:

Link

Private Links:

Internal Issue

Internal Issue

How was this PR tested?
PSim and Internal Evaluator.
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

thansk!! LGTM


Add some more features

There are some feature, you may add to make it more usable,

ctrl + backspace should delete the previous typed characters until a space is encountered.
Statistics of your previous typing tests

Hey @kunal1234523! Thank you for the suggestions.

ctrl + backspace should delete the previous typed characters until a space is encountered.

There was an issue for this already (#18) and ctrl-w has been added already and is available in v0.4.0. ctrl-backspace could not be implemented.

Statistics of your previous typing tests

This is great idea. It has been suggested before (#11).

I could not find time to work on this in the past couple of months, so if you know some Rust, you can try contributing too!

I'm closing this issue for now since there are other issues specific to the features you requested. Feel free to open more issues if you have other suggestions!


Convert to a CLI/npm module

This is already super helpful, but I think it opens up more possibilities and probably better overall ergonomics if this was available to run like this (for example):
index.js -level 3 -url_limit 10 -key #### -whiteListUrls whiteListUrls.txt
A few of those could take either config files or just params (like, whiteListUrls could potentially be just a list of urls passed directly, and I could see a config param to pass in a config file versus using level, url_limit, etc.
The end goal though is that when this is set, we could wrap this up as an npm module so people could eventually have something like:
npx wpt-crawler -level 3 .....
Which would make it easier to drop into other workflows.
What do you think, @abdulsuhail?

yeah, totally makes sense. This would make lives much easier, will work on it.

The Issue has already been resolved and can be closed


Add gtm.js runtime caching

Library Affected:
workbox-google-analytics.
Browser & Platform:
all browsers.
Issue or Feature Request Description:
Support runtime caching of gtm.js.
I would like to add runtime caching for gtm.js because it's responsible of fetching analytics.js so without caching it the offline google analytics will not work if the user starts the app with no connection.

This was fixed by https://github.com/GoogleChrome/workbox/pull/1869


update to use vNet integration

I think that this doc could be updated to use vNet Integration instead of public endpoints for SQL Managed Instances. I am not a Data and AI guy, and I don't know all of the in's and out's of these services to make the changes.

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

ID: e4785005-a130-a392-7b4a-c8c6d36bad37
Version Independent ID: 6beb58e0-167c-c08d-46ff-13f72ddbaf31
Content: Use Azure Functions to perform a database clean up task
Content Source: articles/azure-functions/functions-scenario-database-table-cleanup.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @cchapin-ms,
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@cchapin-ms you are probably right if the goal is to establish a fully-locked-down topology with Functions and SQL Database. However, this article is a more basic demonstration of using SqlClient in a function, which a much more modest scope.
+ @cachai2 on whether this suggestion is a scenario we should consider documenting in the future. In the meantime, I'm going to close this issue. #please-close


Adds filtering of trace/log messages

Allows the user to filter trace/log messages by level.

The "component model" (i.e. "view model") concept has been split into separate components, the model itself and optional state which can reside with the React component (UI) and persist across models.  (This enables the filter selection to be "sticky" as the user switches between requests, making comparisons of logs easier.)
The styling of the tab has also been updated to match the latest redlines.

Related to #78.

Would be interesting to see how responsive the UI feels with a large number of log messages.  E.g.,500?

Would be interesting to see how responsive the UI feels with a large number of log messages. E.g.,500?

I experimented with a request that generated 1500 messages; the UI feels pretty responsive at that level.  I think if we were to adopt Redux, we could probably do better as it makes it pretty easy to ignore re-renders for components not affected by the state change.

Great about the responsiveness.  This LGTM.  What needs to happen to merge this in?

Just waiting for @avanderhoorn to also sign-off.

After that one minor style change/revert, :shipit:


Dropdown menu title

[x] This is a question about using the theme.
[x] This is a feature request.
[ ] I believe this to be a bug with the theme.

[ ] I have updated all gems with bundle update.
[ ] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version: 4.6
Jekyll version:
GitHub Pages hosted: yes
Operating system: Mac OS

Expected behavior

Is it possible to have a dropdown item in the nav bar, like bootstrap?
Steps to reproduce the behavior

The menu currently follows the "priority plus" pattern and isn't something I'm interested in supporting.
If you want a navigation bar similar to Bootstrap's you'll need to use a Bootstrap theme or customize the theme. Here's some Jekyll resources to get you started if you want to build your own.

https://thinkshout.com/blog/2014/12/creating-dynamic-menus-in-jekyll/
http://planetjekyll.github.io/snippets/nav-with-data.html
https://github.com/forestryio/jekyll-menus
https://gist.github.com/kasperisager/9416313

Michael, thank you for your prompt reply.
However, I like your theme so I'm going to try to customize it.
Have a nice day


python310Packages.bless: init at 0.2.4

Description of changes
Library for creating a BLE Generic Attribute Profile (GATT) server
https://github.com/kevincar/bless

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 187335 run on x86_64-linux 1

2 packages built:

python310Packages.bless
python39Packages.bless


PDF document icon change

Windows 11
Version 117.0.5938.157 (Official Build) (64-bit) AVX2
Hello, I have html and pdf files and I recently changed to Thorium all good, except then in edge PDF documents had their own icon that resembled a document, on Thorium looks like another .html and cannot distinguish between them.
Is there a way for me to change the icon to a more pleasing document-like shaped icon instead of the default Thorium?, I already uninstalled Edge in its totality so I don't want to deal with the install of it anymore, I would like to open pdf's on thorium I just would like the icon to be a document, yet I haven't found a solution to do it on windows.
Thank you so much for your help.

This problem seems to be related to Windows' own file association mechanism and icon mechanism
@GuevaraStrings

Alright


How to increase Broker configuration value message.max.bytes

Hey,
I'm using your container but have the need to increase the broker configuration value  message.max.bytes but unfortunately I could not find anything about this property.
Can I parametrize message.max.bytes while starting up the container? Unfortunately, I could not find anything about that.  Thanks for your effort :)
Best regards,
Kon

Sure. Try something like:
-e KAFKA_MESSAGE_MAX_BYTES=2048000

Thanks :)


Windows installatoin issues & Nvidia Cuda NvToolsExt

I found 3 issues
1:
git submodule update --init --recursive --jobs 0
does not work, but this does:
git submodule update --init --recursive --jobs 1
By the way, I was unable to clone with:
git clone git@github.com:lfranke/TRIPS.git
Instead, I used this:
git clone https://github.com/lfranke/TRIPS.git

2:
cmake -Bbuild -DCMAKE_CUDA_COMPILER="$ENV:CUDA_PATH\bin\nvcc.exe" -DCMAKE_PREFIX_PATH=".\External\libtorch" -DCONDA_P_PATH="$ENV:CONDA_PREFIX" -DCUDA_P_PATH="$ENV:CUDA_PATH" -DCMAKE_BUILD_TYPE=RelWithDebInfo .
I get this error:
CMake Error at External/saiga/cmake/Dependencies_cuda.cmake:21 (enable_language):
  The CMAKE_CUDA_COMPILER:
    $ENV:CUDA_PATH\bin\nvcc.exe
  is not a full path and was not found in the PATH.

Then I replaced this line with:
cmake -Bbuild -DCMAKE_CUDA_COMPILER="%CUDA_PATH%\bin\nvcc.exe" -DCMAKE_PREFIX_PATH=".\External\libtorch" -DCONDA_P_PATH="%CONDA_PREFIX%" -DCUDA_P_PATH="%CUDA_PATH%" -DCMAKE_BUILD_TYPE=RelWithDebInfo .

3:
It works much better, but now, I have this error:
CMake Error in src/lib/CMakeLists.txt:
  Imported target "torch" includes non-existent path
    "C:/Program Files/NVIDIA Corporation/NvToolsExt/include"
  in its INTERFACE_INCLUDE_DIRECTORIES.

My Cuda 11.8 installation has no C:/Program Files/NVIDIA Corporation/NvToolsExt
I found some files here:
C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.8\include\nvtx3
nvToolsExt.h
etc.
I'm not sure what is wrong here
I installed Cuda 11.8 selecting this:
Development/*
Runtime/Libraries/*
Visual Studio Integration

It seems that this zip is for Cuda 11.6 - not 11.8
right?
https://download.pytorch.org/libtorch/cu116/libtorch-win-shared-with-deps-1.13.1%2Bcu116.zip

Ah perfect! I will add this to the README!


pygame requires extra dependency

Because pygame is a dependency an extra dependency is required, because the installation fails with:
Running pygame-1.9.2rc1/setup.py -q bdist_egg --dist-dir /tmp/easy_install-Lxwqwp/pygame-1.9.2rc1/egg-dist-tmp-Td5P4G

WARNING, No "Setup" File Exists, Running "config.py"
Using UNIX configuration...

Hunting dependencies...
sh: 1: sdl-config: not found
sh: 1: sdl-config: not found
sh: 1: sdl-config: not found
WARNING: "sdl-config" failed!
error: Setup script exited with Unable to run "sdl-config". Please make sure a development version of SDL is installed.

I'd really prefer not to define it in setup.py because I don't need it and it's causing problems like this. If it's optional than people can install these extra dependencies, inc. pygame.

The actual package that also needs to be installed: sudo apt-get install libsdl1.2-dev

But then tox eventually fails with:
WARNING, No "Setup" File Exists, Running "config.py"
Using UNIX configuration...

Hunting dependencies...
SDL     : found 1.2.15
FONT    : not found
IMAGE   : not found
MIXER   : not found
PNG     : found
JPEG    : found
SCRAP   : found
PORTMIDI: not found
PORTTIME: not found
FREETYPE: found 2.6.1

Warning, some of the pygame dependencies were not found. Pygame can still
compile and install, but games that depend on those missing dependencies
will not run. Would you like to continue the configuration? [Y/n]:y

If you get compiler errors during install, doublecheck
the compiler flags in the "Setup" file.

Continuing With "setup.py"
error: [Errno 2] No such file or directory: 'version.py.in'

Also, shouldn't this import be higher up; in the pygame class?

What distro and arch are you on that requires SDL for pygame?
I have installed it on ubuntu (x64), macOS, debian (RPi), armbian (OrangePi) and haven't seen any of these specific issues (although, I can't claim that libsdl wasnt already installed on any of them).
Which version of python?

Also, shouldn't this import be higher up; in the pygame class?

Possibly, but I originally wanted to call the emulator class pygame (maybe in hindsight not a good idea) and this clashed; additionally I didn't want to introduce an import unnecessarily if it wasn't going to be used.

That's on Ubuntu 16.04 LTS with compiled Python 3.5.2.

Possibly, but I originally wanted to call the emulator class pygame (maybe in hindsight not a good idea) and this clashed; additionally I didn't want to introduce an import unnecessarily if it wasn't going to be used.

Because all the emulators inherit that pygame import now, even though they're not using it.

ah, sorry, all the emulators do actually use pygame: the scaling is done by pygame's built-in scale2x and smoothscale transforms

Hmm, are these not available in Pillow? I'll have to take a closer look. With my suggested Qt emulator (#60 ) I was thinking it would all be done by Pillow and Qt in that case.

Am seeing the same problems locally now as you.. Previously it was working for me with pygame-1.9.2b6, but seems like this new release candidate is causing me grief... it was released 2 days ago
Looking at https://travis-ci.org/rm-hull/ssd1306#L265 but thats OK

Do you have mock installed? Does the code look for python 3 mock first? I thought that might be the problem but haven't tested..

Pygame 1.9.2rc1 introduces some extra dependencies that aren't automatically met any more on some platforms.
--> Removed install dependency of pygame - using the emulators will now raise a runtime exception with message Emulator requies pygame to be installed
Fixed in commit 135afd473d07787db22caf480f74c5c24ee9af60


Change GL bindings (proposal)

Proposal
Eventually we will need to change our GL bindings when we make the full switch to SDL and remove the osuTK package.  There are a number of ways we can do this.
Requirement of #2448
1. Wrap osuTK GL bindings
The idea being that there would be no direct references to osuTK GL types and functions.  Once we remove osuTK, the wrapper would load procs solely with SDL_GL_GetProcAddress.
Pros

No references to osuTK.Graphics.ES30 or SDL outside of the wrapper

Cons

Requires copying enums and structs
Wrapping could introduce overhead depending on how it's done
A lot of work

2. Split osuTK GL bindings to a standalone package
Rather than copying code or making wrappers, it might be worth retaining part of the osuTK library solely for its GL bindings.  This is something we could remove in future when we look at going API-agnostic, but for now it would save a lot of effort and reinventing the wheel.
Pros

Requires minimal changes to osu!framework
SDL PassthroughGraphicsBackend would continue to work fine
Can be retained once we are ready to remove the other osuTK packages
We know it works

Cons

osuTK GL bindings use osuTK's Vector*, Matrix*, and Color4 types
Could be a lot of other cross dependencies we would need to resolve when splitting the package

3. Move to another GL binding library
We could also look at finding another cross-platform OpenGL binding library.  There are plenty available, but we'd need to ensure they're netstandard compatible and work fine on Xamarin.
Pros

Keeps GL bindings self-contained (though splitting osuTK would serve the same purpose)

Cons

Potentially requires a lot of changes to osu!framework
Might not work the same on all platforms
Could introduce regressions

Summary
My personal preference would be to split the osuTK GL bindings, as it requires the fewest changes in osu!, osu!framework, and consumers' projects.  It's also proven to work on all platforms we currently target.
I welcome other suggestions and opinions.

Personal preference is splitting out osuTK's GL bindings. One thing that may be considered is removing all the typed overloads for methods since the future is for consumers to not use any native methods at all, and instead submit commands via the o!f graphics abstraction layer which then invoked appropriate native commands whether vulkan/opengl/d3d/etc.

One thing that may be considered is removing all the typed overloads for methods since the future is for consumers to not use any native methods at all

I agree, since this would also avoid the dependency issue of needing osuTK's colour and math types.  It might be worth moving some of the typed overloads into osu!framework as [Obsolete] extension methods, just to ease migration in the short term.  There are only a handful we use, and these can be removed along with the Colour4 and System.Numerics changes.

Doesn't need to be internal, we can re-expose them under the correct namespace from within o!f. Not sure about obsoleting just yet - I envisioned the implementation of the abstraction layer to have a "raw" command even if only temporarily, something like:
SubmitViewport(...);
SubmitOrtho(...);
SubmitUnsafeRaw(api =>
{
    /* do custom stuff with whatever graphics API you wish, keeping in mind this is unsafe */
    GL.Scissor(...);
});
SubmitVertices(...);
...

Looking more into it, the GL bindings in osuTK are incredibly tightly coupled with the rest of the library.  Splitting might be more effort than it's worth.
As for an abstraction layer, that's still a ways down the track.  I was hoping to get in something small and self-contained to replace the osuTK bindings so that we can eventually remove it entirely.

Looks like there's a fourth option that combines 2 and 3.  Someone has already done the work for us. :)
https://github.com/giawa/opengl4csharp

This code started as a limited subset of the Open Toolkit bindings to OpenGL, but has evolved to include the latest functionality by parsing the OpenGL man pages directly.

I've converted the framework to these bindings and it seems to work fine with the SDL backend.  Will check to see what needs to be done in regards to the osuTK backend.

I have an initial attempt at splitting osuTK into GL and core packages.  Low priority, but the point is that it can be done without too much effort.
https://github.com/swoolcock/opentk/tree/package-split

For those who aren't in the discord: more than happy to help out where I can if the team are willing to use Silk.NET as I'm the main author over there and can fill in missing functionality where needed.
If the Silk.NET route were to be pursued, what would end up happening is we'd use the Silk.NET.OpenGLES package but with a custom GetProcAddress (potentially SDL as described in option 1) and custom context.
Let me know if you have any questions - it's probably best you direct them to me in the Discord, I'm @Perksey over there too, ping me but just don't spam me with pings k :P

Oh and btw, Silk.NET like OpenTK does provide a pretty solid and pretty extensible windowing and input API (currently only supports desktop via GLFW though, mobile support won't come until November) so as a potential future/stretch goal we could move over to that instead of GameWindow later down the line - not right now obviously as that'd be a lot of work.

One last thing (I promise I'll stop with the comments now :D ) I notice that you've currently got a IWindow abstraction in the works, and Silk.NET already has a pretty solid abstraction so even if you only wanted an abstraction and not necessarily our first-party implementation, you can still consume the APIs defined in Silk.NET.Windowing.Common (we have an IView interface and an IWindow interface, the latter of which building on the former)

What's the reasoning behind the November timeline?  Is it purely lack of manpower, or are you specifically waiting for .NET 5?

We’re lining up the release of Sik.NET 2.0 with the .NET 5 release yeah
because we’ll be using some of the latest language and framework features
On Fri, 15 May 2020 at 01:13, Shane Woolcock<EMAIL_ADDRESS>wrote:

What's the reasoning behind the November timeline? Is it purely lack of
manpower, or are you specifically waiting for .NET 5?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ppy/osu-framework/issues/3538#issuecomment-628951470,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVEYI4W7VX6IQVYMJF7S33RRSCL3ANCNFSM4M6QAOFA
.

Closing this one as we have mostly abstracted away bindings with the new renderer framework, and will continue to track this there.


Safari version

Any plan to release this extension for Safari?

Correct me if I'm wrong, but it seems Safari is deprecating support for Extensions.

It seems like the new form of Safari extensions have to be written in Swift, so it'd require a complete re-write, and parallel updates between the JS and Switch code each commit :/ I don't see this happening, @Muffo
Thoughts, @boukestam?

Yes, that's a really pain! I agree it would be a maintenance nightmare unless someone comes up with a framework to easily port extensions from Chrome/Firefox.


Slack channel requires a stackstorm email address

I tried to sign into the slack community chat today and wasn't allowed in because I don't have an @stackstorm email address.

You can sign up here - https://stackstorm.typeform.com/to/K76GRP and an invite should automatically be sent to your email address.
You can then use this invite to join http://stackstorm-community.slack.com Slack instance

@robzienert You should have an invite now. The problem was not with non stackstorm email address. We have a sensor that links typeform to invites. It appears typeform APIs were returning 5XXs and our sensor crashed. It couldn't respawn because this https://github.com/StackStorm/st2/pull/1798 is not deployed to that box yet. We'll work on upgrading the box to latest st2. Thanks for the bug report and apologies for the delay in the invite. I just got my invite for a non-stackstorm email. LMK if you don't have the invite and I can debug further. Logs show you were sent out an invite.

Heh, I assumed something like this might be going on, but I wasn't sure that the form was used, so I rather asked :)
In any case, it might be worth adding some text to that form which says you should contact us and include your email address if you don't receive an invite in an hour or something along those lines.

That server is now running the latest version of st2.
root@pinkiepie:/home/lakshmi/st2incubator# st2 sensor list
+-------------------------------------+----------+----------------------------+-----------------------+
| ref                                 | pack     | name                       | trigger_types         |
+-------------------------------------+----------+----------------------------+-----------------------+
| linux.FileWatchSensor               | linux    | FileWatchSensor            | linux.file_watch.line |
| typeform.TypeformRegistrationSensor | typeform | TypeformRegistrationSensor | typeform.registration |
+-------------------------------------+----------+----------------------------+-----------------------+
root@pinkiepie:/home/lakshmi/st2incubator# less /var/log/st2/st2sensorcontainer.log
root@pinkiepie:/home/lakshmi/st2incubator# ps auxww | grep typeform
root      4143  2.8  1.0<PHONE_NUMBER>4 pts/4    S    23:40   0:01 /opt/stackstorm/virtualenvs/typeform/bin/python /usr/lib/python2.7/dist-packages/st2reactor/container/sensor_wrapper.py --pack=typeform --file-path=/opt/stackstorm/packs/typeform/sensors/registration_sensor.py --class-name=TypeformRegistrationSensor --trigger-type-refs=typeform.registration --parent-args=["--config-file", "/etc/st2/st2.conf"] --poll-interval=30
root      4235  0.0  0.0  10460   940 pts/4    S+   23:41   0:00 grep --color=auto typeform
root@pinkiepie:/home/lakshmi/st2incubator# st2 --version
st2 ('0.13dev.179',)
root@pinkiepie:/home/lakshmi/st2incubator#

With this, I also picked up the bug fixes for the typeform sensor https://github.com/StackStorm/st2incubator/pull/238.
Closing issue. Will open another issue for adding text to contact us on invite SLA breach.


xliff parse errors: "Message {id} misses a translation" error for all messages in the application's native language

I have an application that I am natively building in English, and translating into other languages. I'm trying to add the ngx-translate/i18n-polyfill to augment the native Angular i18n capabilities, but I get prolific errors whenever I load the app in its native en-US locale: "Message {id} misses a translation" is repeated for every message in my messages.en-us.xlf file.
Its clear that ngx-translate/i18n-polyfill is just not aware that en-US is the native locale, and so is searching for translations that are not required, but I cannot see any way to configure the polyfill to treat en-US as the native locale.
Providing MISSING_TRANSLATION_STRATEGY with MissingTranslationStrategy.Ignore in the default locale does not fix the issue; the XliffParser still throws the same errors.
providers: [
  {
    provide: MISSING_TRANSLATION_STRATEGY,
    useFactory: (locale: any) => {
      return locale === defaultLocale ? MissingTranslationStrategy.Ignore : MissingTranslationStrategy.Error;
    },
    deps: [LOCALE_ID],
  },
  { provide: TRANSLATIONS_FORMAT, useValue: 'xlf' },
  {
    provide: TRANSLATIONS,
    useFactory: (locale: any) => {
      return require(`raw-loader!../locale/messages.${locale}.xlf`);
    },
    deps: [LOCALE_ID],
  },
  I18n,
],

Is there some config that I am missing, or a work-around that doesn't involve providing redundant translations for the native English?

I found a work-around, by providing an empty string for the translations when the locale is the default:
{
  provide: TRANSLATIONS,
  useFactory: (locale: any) => {
    return locale === defaultLocale ? '' : require(`raw-loader!../locale/messages.${locale}.xlf`);
  },
  deps: [LOCALE_ID],
}


ditching python 3.9 due to known pytorch segfault

also separated install-build and tests phase

@ardizzone @tbung it's a bit risky, but I don't see any other choice: Please merge this at plain sight.
So I invested this morning into fixing the github actions workflow. According to my observations, the current configuration of the CI pipeline performs a peculiar thing. When I send a PR to the upstream repo VLL-HD/FrEIA from my fork, the topic branch from which I am sending the code gets checked out on the test node and used for executing the tests.
But just before running the tests, master is merged in! :shrug: This will evidently lead to failing tests, because master may have evolved while the topic branch has worked on.
For the time being, the only thing I can imagine, is that I misconfigured the workflow in some not so obvious way in the past and thus master is stuck with this behavior.
If any github actions experts read this, feel free to chime in. At this point, I don't see from the docs what to do differently.

@ardizzone @tbung it's a bit risky, but I don't see any other choice: Please merge this at plain sight.
You mean we should merge it now? (It's still a draft PR, but I can change that)

Go for it.


Srun3mvp

Changes / Comments
[Shadowrun3e] new character sheet
submitting a new character sheet for shadowrun third edition.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


iOS: Account Login Field should switch first responder on return

Currently, when the user enters his username / email into the corresponding field he has to manually select the password field.
This should happen when the user presses return.
Also the keyboard should be dismissed when the user presses the action button.
Optional: dismiss the keyboard when a view other than the text fields or the action button is touched.
Note: I already fixed this on my side. I can prepare a PR.

I added the functionality that the login is happening when the users presses return while the password field is first responder. This is obvious. Silly me 🤷🏻♂


Update README regarding PyPI and related package

Some useful notes.

Indeed, pythonhosted.com documentation hosting upload has ended; see pypa/warehouse#582 for future plans.


No cache hits after calling cache.recreate_keys

The problem
No cache hits after upgrading requests-cache from 0.9.8 to 1.0.1 and calling the session.cache.recreate_keys() method.
Expected behavior
recreate_keys() would make previously cached responses reusable in the current version.
Steps to reproduce the behavior

pip install requests-cache==0.9.8

Make some requests:
import requests_cache

session = requests_cache.CachedSession()

for url in (
    'https://www.volby.cz/opendata/prez2023/xml/perk.xml',
    'https://www.volby.cz/pls/prez2023/vysledky_kraj?kolo=1&nuts=CZ010',
    'https://www.volby.cz/pls/prez2023/vysledky_kraj?kolo=2&nuts=CZ010',
):
    response = session.get(url)
    print(response.from_cache, response.cache_key)

Output:
False 429de4a616434ff0
False e29ed7cff14f5ea8
False f3bba3a9c21f72cb

pip install requests-cache==1.0.1

Recreate the keys:
import requests_cache
session = requests_cache.CachedSession()
session.cache.recreate_keys()

Check database contents:
$ sqlite3 -header -column http_cache.sqlite 'select * from responses'
key               value  expires
----------------  -----  -------
c6d760d2c8662f37  ��
ddf1a5b74bb3b36a  ����
9cdd12e45c6517e3  ���\

As expected - all the keys have changed.

Make the same requests as in step 2.
Output:
False 480cd5448f89ed57
False 5d39231999fbd7fa
False 6b272b1ecd346ee1

Unexpected - three cache misses instead of three cache hits. Three new rows have been added to the database:
$ sqlite3 -header -column http_cache.sqlite 'select * from responses'
key               value  expires
----------------  -----  -------
c6d760d2c8662f37  ��
ddf1a5b74bb3b36a  ����
9cdd12e45c6517e3  ���\
480cd5448f89ed57  ��O
5d39231999fbd7fa  ���
6b272b1ecd346ee1  ��]

Environment

requests-cache version: 0.9.8 and 1.0.1
Python version: 3.11.3
Platform: Ubuntu 22.04

A request's body in version 0.9.8 is stored as b'None', whereas in version 1.0.1 it is b''. This causes the difference in cache keys. It was already discussed here.

This issue can be easily worked around by manually fixing the empty request bodies before recreate_keys() is called. For example:
import requests_cache

def normalize_empty_bodies(cache):
    for key, response in list(cache.responses.items()):
        if response.request.body == b'None':
            response.request.body = b''
            cache.responses[key] = response

session = requests_cache.CachedSession()
normalize_empty_bodies(session.cache)
session.cache.recreate_keys()

Thank you for the thorough bug report! I think that fix is simple enough that I'll go ahead and include that in recreate_keys().

Your example now works as expected, after changing step 3 to install from the current git repo:
pip install git+https://github.com/requests-cache/requests-cache

This will be included in the next release.


jquery.html5loader Not Working

Hi Gianluca,
I've written a small test HTML file to try out your loader app: http://troncon.ca/ESO/test.html
Included in the same directory on the server are the following files:
jquery-2.1.4.js
jquery.html5Loader.js
preload.js
test.mp4
The preload.js file has the following content:
{
'files': [
{
'type':'VIDEO',
'sources': {
'h264':{
'source':'test.mp4',
'size':5,979,174
}
}
}
]
}
I've added an alert() to the onComplete function but it is not showing, which indicates to me that I'm doing something wrong. Can you point me in the right direction.
Thanks,
Ted

I've changed the extension of the json file to json but alert is still not showing. Any ideas?

Change 'size':5,979,174 in your json into 'size':5979174

I made that change but it still didn't work. It's such a simple test page I can't fathom what I'm doing wrong : (


System.TypeInitializationException when building projects after 15.5 update

Summary
After updating to latest version of MSBuild <IP_ADDRESS>428 via the Visual Studio Installer not a single project can be built with MSBuild. Exception being thrown:
MSBUILD : error MSB1025: An internal failure occurred while running MSBuild.
System.TypeInitializationException: The type initializer for 'Microsoft.Build.Shared.NativeMethodsShared' threw an exception. ---> System.PlatformNotSupportedException: Operation is not supported on this platform.
Steps to reproduce

Update to latest version of MSBuild
Try to build any existing project with msbuild.exe <projectName>.csproj for example

Expected  behavior
Being able to build a project using MSBuild.exe
Actual behavior
A System.TypeInitializationException is being thrown
msbuild <projectName>.csproj

New project in VS

Solution Explorer in VS

Trying to Reload a project in VS

Environment data
MSBuild version : Build Engine version <IP_ADDRESS>428 for .NET Framework
Output:
MSBUILD : error MSB1025: An internal failure occurred while running MSBuild.
System.TypeInitializationException: The type initializer for 'Microsoft.Build.Shared.NativeMethodsShared' threw an exception. ---> System.PlatformNotSupportedException: Operation is not supported on this platform.
at System.Runtime.InteropServices.OSPlatform.get_Linux()
at Microsoft.Build.Shared.NativeMethodsShared.get_IsLinux()
at Microsoft.Build.Shared.NativeMethodsShared..cctor()
--- End of inner exception stack trace ---
at Microsoft.Build.Shared.NativeMethodsShared.get_IsMono()
at Microsoft.Build.BackEnd.Logging.LoggingService..ctor(LoggerMode loggerMode, Int32 nodeId)
at Microsoft.Build.Evaluation.ProjectCollection.CreateLoggingService(Int32 maxCPUCount, Boolean onlyLogCriticalEvents)
at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary2 globalProperties, IEnumerable1 loggers, IEnumerable1 remoteLoggers, ToolsetDefinitionLocations toolsetDefinitionLocations, Int32 maxNodeCount, Boolean onlyLogCriticalEvents)    at Microsoft.Build.CommandLine.MSBuildApp.BuildProject(String projectFile, String[] targets, String toolsVersion, Dictionary2 globalProperties, ILogger[] loggers, LoggerVerbosity verbosity, DistributedLoggerRecord[] distributedLoggerRecords, Boolean needToValidateProject, String schemaFile, Int32 cpuCount, Boolean enableNodeReuse, TextWriter preprocessWriter, Boolean debugger, Boolean detailedSummary, ISet1 warningsAsErrors, ISet1 warningsAsMessages, Boolean enableRestore)
at Microsoft.Build.CommandLine.MSBuildApp.Execute(String commandLine)
Unhandled Exception: System.TypeInitializationException: The type initializer for 'Microsoft.Build.Shared.NativeMethodsShared' threw an exception. ---> System.PlatformNotSupportedException: Operation is not supported on this platform.
at System.Runtime.InteropServices.OSPlatform.get_Linux()
at Microsoft.Build.Shared.NativeMethodsShared.get_IsLinux()
at Microsoft.Build.Shared.NativeMethodsShared..cctor()
--- End of inner exception stack trace ---
at Microsoft.Build.Shared.NativeMethodsShared.get_IsMono()
at Microsoft.Build.BackEnd.Logging.LoggingService..ctor(LoggerMode loggerMode, Int32 nodeId)
at Microsoft.Build.Evaluation.ProjectCollection.CreateLoggingService(Int32 maxCPUCount, Boolean onlyLogCriticalEvents)
at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary2 globalProperties, IEnumerable1 loggers, IEnumerable1 remoteLoggers, ToolsetDefinitionLocations toolsetDefinitionLocations, Int32 maxNodeCount, Boolean onlyLogCriticalEvents)    at Microsoft.Build.CommandLine.MSBuildApp.BuildProject(String projectFile, String[] targets, String toolsVersion, Dictionary2 globalProperties, ILogger[] loggers, LoggerVerbosity verbosity, DistributedLoggerRecord[] distributedLoggerRecords, Boolean needToValidateProject, String schemaFile, Int32 cpuCount, Boolean enableNodeReuse, TextWriter preprocessWriter, Boolean debugger, Boolean detailedSummary, ISet1 warningsAsErrors, ISet1 warningsAsMessages, Boolean enableRestore)
at Microsoft.Build.CommandLine.MSBuildApp.Execute(String commandLine)
at Microsoft.Build.CommandLine.MSBuildApp.Main()
OS info:
Windows specifications
Edition: Windows 10 Enterprise
Version: 1709
OS Buld: 16299.98
Some additional notes. Visual studio can't load or create any projects either after the update. Also older versions of MSBuild work just fine. I haven't fiddled with my OS in any way other than installing the standard MS updates whenever prompted.
So far I have tried:
Repairing VS
Reinstalling VS
Reinstalling VS Installer
Manually uninstalling/installing MSBuild Tools (which includes msbuild.exe)
Running older versions of MSBuild (this has worked)

Team triage: closing due to age. If you're still seeing this problem, please let us know and we can reactivate it.


nativecall: Add language to address #2684

The syntax for NativeCall callable signature constraints under Rakudo
deviates from the usual syntax. There is an issue at
rakudo/rakudo#2878 addressing this, but until this is addressed, some
minimal explanation for the strange syntax is needed.

Please don’t merge this yet; it’s for discussion at #2684.


qemu hook stopped/release not getting bridge name anymore

Hi @doccaz,
I've done some adjustments to your script:

fixing a bug: XML does not contain the bridge name in stopped/release actions anymore. The bridge is removed from XML before calling qemu "stopped/release" hook. I am using virsh version 5.0.0 on debian 10. As a side effect #1 is fixed as well, no "cat" is needed anymore, user specify the bridge on his own. How to test if the bug exists in your setup, stop VM and call
virsh dumpxml <VM> | xpath -q -e /domain/devices/interface[1]/source
in my case stopped VMs do not have the bridge.
fixing of double usage of "reconnect" action (bug #2)
host IP can be specified as well, which allows to configure "all ports forwarding" from a specific external IP to a guest IP (I've used your script to achieve a solution described in this article)
assign/remove host IP to/from network interface when VM starts/stops. Why? Example: if guests and host have ssh running on the port 22 (possible in my script adaption) and a guest will be stopped, SSH trafic of the guest will be processed by the host leading to ECDSA key mismatch (and difficult debugging of network problems)
code refactoring - the script in "my" taste in PS

Feel free to use the version in PS/part of it or just keep it here for the others.
Thank you, a nice idea!
Brani
PS:
#!/bin/bash
#
# Hook script for QEMU
#
# adds port forwards via IPtables to your VMs
#
# Erico Mendonca<EMAIL_ADDRESS># dec/2018
#
# https://github.com/doccaz/kvm-scripts
#
# Adapted by Branislav Siarsky
# apr/2020
#

log() {
        logger -t addForward "$1"
}

addForward() {
   VM=$1
   HOST_NIC=$2
   HOST_IP=$3
   HOST_PORT=$4
   GUEST_NIC=$5
   GUEST_IP=$6
   GUEST_PORT=$7

   IPTABLES="/usr/sbin/iptables"
   IP="/usr/sbin/ip"
   HOST_IP_MASK="24"
   IPTABLES_ACTION=""
   IPTABLES_TEXT=""
   IP_ACTION=""
   IP_TEXT=""

   if [ "${VM}" == "${VM_NAME}" ]; then
	   if [ "${ACTION}" == "stopped" ]; then
		   IPTABLES_ACTION="-D"
		   IPTABLES_TEXT="Removing forwarding rules for VM ${VM}"
		   IP_ACTION="del"
		   IP_TEXT="Removing IP ${HOST_IP} from ${HOST_NIC}"
	   fi

	   if [ "${ACTION}" == "start" ] || [ "${ACTION}" == "reconnect" ]; then
		   IPTABLES_ACTION="-I"
		   IPTABLES_TEXT="Adding forwarding rules for VM ${VM}: host port ${HOST_PORT} will be redirected to ${GUEST_IP}:${GUEST_PORT} on interface ${GUEST_NIC}"
		   IP_ACTION="add"
		   IP_TEXT="Adding IP ${HOST_IP} to ${HOST_NIC}"
	   fi

	   if [ "${IPTABLES_ACTION}" == "" ]; then
		   log "Action ${ACTION} on domain $VM not configured, ignoring"
		   exit 0
	   else
		   log "Action ${ACTION} on domain $VM called"
	   fi

	   log "${IPTABLES_TEXT}"
	   $IPTABLES ${IPTABLES_ACTION} FORWARD -o ${GUEST_NIC} -d $GUEST_IP -j ACCEPT
	   $IPTABLES -t nat ${IPTABLES_ACTION} PREROUTING -d $HOST_IP -p tcp --dport $HOST_PORT -j DNAT --to $GUEST_IP:$GUEST_PORT

	   log "${IP_TEXT}"
	   $IP address ${IP_ACTION} ${HOST_IP}/${HOST_IP_MASK} dev ${HOST_NIC}
   fi
}

## main program
VM_NAME=${1}
ACTION=${2}

### declare your port forwards here
### format: addForward <VM> <source nic> <source address> <source port> <destination nic> <destination address> <destination port>
#Port 80 forward:   addForward debian10-vm2 enp3s0f0 <IP_ADDRESS> 80 virbr0 <IP_ADDRESS> 80
#All ports forward: addForward debian10-vm2 enp3s0f0 <IP_ADDRESS> 1:65535 virbr0 <IP_ADDRESS> 1-65535
addForward debian10-vm1 enp3s0f0 <IP_ADDRESS> 1:65535 virbr0 <IP_ADDRESS> 1-65535
addForward debian10-vm2 enp3s0f0 <IP_ADDRESS> 1:65535 virbr0 <IP_ADDRESS> 1-65535
addForward debian10-vm3 enp3s0f0 <IP_ADDRESS> 1:65535 virbr0 <IP_ADDRESS> 1-65535

Thank you for your contribution! Merged your changes.


Getting error when try to run webpack in watch mode.

I am getting error when try to run webpack in watch mode.
'UV_THREADPOOL_SIZE' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: UV_THREADPOOL_SIZE=8 node ./node_modules/webpack/bin/webpack.js -d --watch
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\mrokicki\AppData\Roaming\npm-cache_logs\2018-10-22T07_13_27_415Z-debug.log

Hey, what Node version do you have? Make sure that you're running Node > 10.x. (I'm using 10.11.0 with latest master and everything works well).

Getting Same thing with node 10.13
npm.log

To make it clear - you're running npm start and get this error? What version of Saleor are you using? I'm looking at the current package.json and the in start command UV_THREADPOOL_SIZE variable isn't used (I remember it was a long time ago).

Please reopen the issue if it still occurs.


R2 PUT operation not working

I've got the setup with a small Svelte project, and any attempt to use a PUT operation with R2 fails.
const r2 = event.platform?.env.TORCH_R2 ?? connectR2('R2_STORE');
if (!r2) throw new Error('Torch R2 binding not found');

await r2.put('newvalue', 'test'); <-- stack points here

Wrangler-proxy Console Error:
[wrangler:inf] POST /instruction 200 OK (2ms)
[wrangler:inf] POST /data 500 Internal Server Error (3ms)

Svelte Console Error
Error: {"name":"TypeError","message":"Provided readable stream must have a known length (request/response body or readable half of FixedLengthStream)","stack":"TypeError: Provided readable stream must have a known length (request/response body or readable half of FixedLengthStream)"}
    at W (file:///C:/Users/XXX/development/svelte-test/node_modules/wrangler-proxy/dist/index.js:1:197)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async o.post (file:///C:/Users/XXX/development/svelte-test/node_modules/wrangler-proxy/dist/index.js:1:714)
    at async Module.init (C:\Users\XXX\development\svelte-test\src\lib\middleware\init.ts:29:3)
    at async Object.handle (C:\Users\XXX\development\svelte-test\src\hooks.server.ts:9:14)
    at async Module.respond (C:\Users\XXX\development\svelte-test\node_modules\@sveltejs\kit\src\runtime\server\respond.js:282:20)
    at async file:///C:/Users/XXX/development/svelte-test/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:502:22
j {
  host: 'http://<IP_ADDRESS>:8787',
  name: 'R2_STORE',
  metadata: {},
  data: null <-- null?
}

It looks like data is null, but I am passing a value.

I've got the setup with a small Svelte project, and any attempt to use a PUT operation with R2 fails.
const r2 = event.platform?.env.TORCH_R2 ?? connectR2('R2_STORE');
if (!r2) throw new Error('Torch R2 binding not found');

await r2.put('newvalue', 'test'); <-- stack points here

Wrangler-proxy Console Error:
[wrangler:inf] POST /instruction 200 OK (2ms)
[wrangler:inf] POST /data 500 Internal Server Error (3ms)

Svelte Console Error
Error: {"name":"TypeError","message":"Provided readable stream must have a known length (request/response body or readable half of FixedLengthStream)","stack":"TypeError: Provided readable stream must have a known length (request/response body or readable half of FixedLengthStream)"}
    at W (file:///C:/Users/XXX/development/svelte-test/node_modules/wrangler-proxy/dist/index.js:1:197)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async o.post (file:///C:/Users/XXX/development/svelte-test/node_modules/wrangler-proxy/dist/index.js:1:714)
    at async Module.init (C:\Users\XXX\development\svelte-test\src\lib\middleware\init.ts:29:3)
    at async Object.handle (C:\Users\XXX\development\svelte-test\src\hooks.server.ts:9:14)
    at async Module.respond (C:\Users\XXX\development\svelte-test\node_modules\@sveltejs\kit\src\runtime\server\respond.js:282:20)
    at async file:///C:/Users/XXX/development/svelte-test/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:502:22
j {
  host: 'http://<IP_ADDRESS>:8787',
  name: 'R2_STORE',
  metadata: {},
  data: null <-- null?
}

It looks like data is null, but I am passing a value.

I made a small reproduce here: https://github.com/chientrm/wrangler-proxy-r2-bug
It seem to put correctly.
Please try update wrangler-proxy version to the latest and restart the proxy process.

I appreciate your example.
The difference appears to be R2 doesn't work locally. If I add the --remote flag to when running wrangler-proxy, it does indeed work correctly. I presumed this library supports local workflows as well, or am I incorrect in that assumption? D1 seems to work perfectly locally.

I appreciate your example.
The difference appears to be R2 doesn't work locally. If I add the --remote flag to when running wrangler-proxy, it does indeed work correctly. If I remove it, then the above error occurs again.
I presumed this library supports local workflows as well, or am I incorrect in that assumption? D1 seems to work perfectly locally.

I have no idea lol.
wrangler-proxy was originally designed to work with remote mode. When I first started, local mode was not available.

Ah, ok then! That would totally explain my issue then lol


Unified and synced up all branches

lets fix this build mess once and for all

Well you tagged the wrong person 3 days ago, that's why I didn't have any notifications about this. My account is manhnt9, manhnt is someone else.

set(PNG_BUILD_ZLIB ON CACHE BOOL "Custom zlib Location, else find_package is used")
Maybe your CMake cache is already set and the set command is ignored.


Can't run PowerShell code until i hit Enter in Terminal

Issue Type: Bug

open VSC
VSC opens with last edited PS script
select part of its code and hit F8 to run selection
nothing happens in Terminal
disabling PS extension doesn't help
click Terminal
totally blank
hit Enter
now we see path with command prompt
run selected code again (F8)
works

VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:14.102Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.82GB (10.16GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

This is a bug in Windows 10 1809, unfortunately we have no way to work around the issue so you will need to wait until you get the next version of Windows 10 (1903).
/duplicate https://github.com/Microsoft/vscode/issues/57803


fix: widget borders does not reflect widget height

Description
Reported issue
If a widget - not a container like widget - has a minimum height of 4 rows, and the auto height is enabled for the widget, the widget borders signify a much smaller height. Example - checkbox widget.
Results of RCA
This was happening because, we set height: auto for all widgets, when it should have been done only for container like widgets. The height:auto in the resizable library, allowed the library to show the widget borders completely hugging the contents of the widget, when it should have been restricted to a minimum of 4 rows (40px). This is because, in the platform, the minimum height any widget can have is 4 rows (40px) in Fixed mode (not auto layout mode)
Solution:
We exclude widgets which do not have isCanvas property set to true from setting the autoHeight flag. This allows the library to set the provided height instead of hugging contents.
PR fixes following issue(s)
Fixes #23690
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing
How Has This Been Tested?

[x] Manual
[x] Cypress

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/ok-to-test

ads review not needed here 👍

/build-deploy-preview skip-tests=true

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test


Launch/Task Configuration Options in User Settings being Added to Debug: Add Configuration

Version: 1.81.1 (user setup)
Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794
Date: 2023-08-09T22:22:42.175Z
Electron: 22.3.18
ElectronBuildId: 22689846
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

When either selecting Debug: Add Configuration, or one of its many shortcuts such as "create a build file" on the Run and Debug panel, a list of potential configurations drop down, typically including a launch.json and tasks.json. Many extensions add onto this list with varying options. Additionally, users may add a launch config in their user settings, which will thus act as if added to all projects.
The problem, however, is the former is not modifiable by users easily, requiring creating an entire extension themselves for minute changes, and the latter only works on that users' systems. Where the .vscode builds files would want to be shared with others, neither of these solutions work.
A simple change would be to add any launch configuration from the user settings to the Add Configuration dropdown menu as well (instead of being usable, but not attachable to the project without manual copy/paste).

What I would suggest here is using a normal snippet, registered for the jsonc language. This will show up in launch.json and let you take advantage of the normal snippet infrastructure to fill in your preferred launch config. The disadvantage is that it will show up in other json files as well, but I don't want to try to reinvent snippets just for launch.json.


failure to cache productlist in the adobe_tools branch

as of commit 5822bc94, the __init__ of AdobeAPIObject was failing to gather and cache the productlist member of the cached data, and so the product list is always fetched from the server.
See lines 170-172 of adobe_api.py for details.  I would suggest calling gather_product_list at that point in the initialization.

Fixed in a171b85b8adf4c08bd05a4227fb8e13078a7d300.


"Self referencing loop detected" json serialization error while doing bool query.

Branch 2.0
Simplest repro:
https://gist.github.com/michaelbudnik/b1c552c34dc39ab2e067
It works when you delete the boolean container.
Error comes from CustomJsonConverter class.
The exact message is:
Self referencing loop detected with type 'Nest.QueryContainerDescriptor`1[...]'.
Path 'query.bool.must'.

It seems like the boolean queries are not working at all for me.

Seems to be working when I debug in and set the ReferenceLoopHandling setting to "Serialize" just before serialization. The json hitting the server is correct.
The error seems wrong then. There aren't any loops? I'm really struggling to understand what's going on.

Trace from Newtonsoft.Json:
2015-11-18T10:56:08.990 Info Started serializing Nest.SearchDescriptor`1[Console
Application2.Program+Model]. Path ''.
2015-11-18T10:56:09.053 Info Started serializing Nest.QueryContainerDescriptor`1
[ConsoleApplication2.Program+Model] with converter Nest.ReserializeJsonConverter
`2[Nest.QueryContainer,Nest.IQueryContainer]. Path 'query'.
2015-11-18T10:56:09.068 Info Started serializing Nest.BoolQueryDescriptor`1[Cons
oleApplication2.Program+Model]. Path 'query.bool'.
2015-11-18T10:56:09.068 Info Started serializing System.Collections.Generic.List
`1[Nest.QueryContainer]. Path 'query.bool.must'.
2015-11-18T10:56:09.068 Info Started serializing Nest.QueryContainerDescriptor`1
[ConsoleApplication2.Program+Model] with converter Nest.CompositeJsonConverter`2
[Nest.ReadAsTypeJsonConverter`1[Nest.QueryContainer],Nest.CustomJsonConverter].
Path 'query.bool.must'.
2015-11-18T10:56:09.068 Error Error serializing Nest.QueryContainerDescriptor`1[
ConsoleApplication2.Program+Model]. Self referencing loop detected with type 'Ne
st.QueryContainerDescriptor`1[ConsoleApplication2.Program+Model]'. Path 'query.b
ool.must'.
2015-11-18T10:56:09.068 Error Error serializing System.Collections.Generic.List`
1[Nest.QueryContainer]. Self referencing loop detected with type 'Nest.QueryCont
ainerDescriptor`1[ConsoleApplication2.Program+Model]'. Path 'query.bool.must'.
2015-11-18T10:56:09.068 Error Error serializing Nest.SearchDescriptor`1[ConsoleA
pplication2.Program+Model]. Self referencing loop detected with type 'Nest.Query
ContainerDescriptor`1[ConsoleApplication2.Program+Model]'. Path 'query.bool.must
'.

Hmm, this may very well be an issue with our custom serialization as we haven't completely finished refactoring and testing the query DSL bits in the 2.0 branch (which is a blocker for the alpha release).  We'll definitely dig into this further in the coming days- @Mpdreamz and I are both travelling this week so may be a little slow to reply.
Thanks for reporting!

Currently fix for that is that you set
connectionSetting.SetJsonSerializerSettingsModifier(m => m.ReferenceLoopHandling = ReferenceLoopHandling.Serialize);

#1636 :smile:

Just tested the serialization with the latest in 2.0 branch with the following, based on your example
[U]
public void Test()
{
    var client = new ElasticClient();

    var createIndexResponse = client.CreateIndex("model", c => c
        .Mappings(m => m
            .Map<Model>(mm => mm
                .AutoMap()
            )
        )
    );

    var result = client.Search<Model>(s => s
        .Index("model")
        .Query(q => q
            .Bool(b => b
                .Should(m => m
                    .Match(qs => qs
                        .Field(f => f.Name)
                        .Query("asdsadsa"))))));
}

public class Model
{
    public int Id { get; set; }
    public string Name { get; set; }
}

This works and produces the following query DSL
{
   "query": {
      "bool": {
         "should": [
            {
               "match": {
                  "name": {
                     "query": "asdsadsa"
                  }
               }
            }
         ]
      }
   }
}

I'm going to close this issue as fixed. Thanks again for reporting :smile:


"Finish the Last Level" button returns to the skillmap without a focus on the last level

After finishing the 4th level of the balloon skillmap, the reward window pops up. If you click on "Finish the Last Level" it just returns to the skillmap - the same result as just closing the pop-up. Instead, I would expect the last level to be selected.
Pop-up window:

After clicking "Finish the Last Level":

Expected:

Great idea!  It's an engineering thing, though.

Can we remove the "Finish the last level" from skillmap? It is not serving any purpose.

In the education biz, it's what we call a "prompt."  They're more inclined to finish the last level if someone suggests they finish the last level.

There are too many prompts in that dialog. I am not sure it is serving that purpose. Your call though.

Sounds good


Appwright reporter

Appwright uses a saveVideo fixture to attach artifacts to the report. This increases test execution time and should be carried out in a reporter.

I get an error TypeError: providerClass.downloadVideo is not a function after tests complete

Thanks for flagging this @dreybuzz. Just released 0.1.37 with the fix


Update Manifest: IObit.RandomPasswordGenerator version 1.3

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @KarbitsCode,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 886.)

So... Is all good?

So... Is all good?

Yes, this one slipped through the "cracks", sorry.


feat: drain module account once the bridge is shutdown

This PR adds a check in Endblocker so all funds (not only Umee tokens) are sent to the "drain address", in case there are none, it just skips the send.
TODO: add test
Didn't mean to push the removal of the burn msg, I had removed it locally so tests would run 😓

thanks


how to concat multi layers?

Hi,
It's great work you made.
I came across one problem, python code looks like below
flatten = mx.symbol.Flatten(data=relu3)
fc1 = mx.symbol.FullyConnected(data=flatten, num_hidden=512)
fc21 = mx.symbol.FullyConnected(data=fc1, num_hidden=11)
fc22 = mx.symbol.FullyConnected(data=fc1, num_hidden=11)
fc23 = mx.symbol.FullyConnected(data=fc1, num_hidden=11)
fc24 = mx.symbol.FullyConnected(data=fc1, num_hidden=11)
fc2 = mx.symbol.Concat(*[fc21, fc22, fc23, fc24], dim=0)
label = mx.symbol.transpose(data=label)
label = mx.symbol.Reshape(data=label, target_shape=(0, ))
mx.symbol.SoftmaxOutput(data=fc2, label=label, name="softmax")

Is it possible to implement this using ConvNetSharp?
thanks!

Hi,
It should be possible using the 'Flow' part of ConvNetSharp (by creating a computation graph).
I will try to implement your example to ConvNetSharp soon and will post it here.

It would look something like:
var input = cns.PlaceHolder("input");
var flatten = cns.Flatten(input);
var fc1 = cns.Dense(flatten, 512);
var fc21 = cns.Dense(fc1, 11);
var fc22 = cns.Dense(fc1, 11);
var fc23 = cns.Dense(fc1, 11);
var fc24 = cns.Dense(fc1, 11);
var fc2 = cns.Concat(cns.Concat(fc21, fc22), cns.Concat(fc23, fc24));
var model = cns.Softmax(fc2);

var label = cns.PlaceHolder("label");
var cost = cns.CrossEntropyLoss(model, label);

Cool , thank you so much.
I'll have a try and give some feedback again.


Multi certificate support

Fixed recursive .gitignore
Updated Access Control doc
Move CertificateOptions to it's own type
Added SSH permissions generator function
Added support for multiple certificates in CLI filesystem
A certificate is now generated per jumphost in Auth Lambda response

Worth nothing that ci/expected-output-jumpbox-sshconf.txt has to have two trailing new lines..


🐞 [Bug]: adding worker on k8s is not functional

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
while adding worker to k8s after adding it the worker is not shown in the master
Steps To Reproduce

deploy k8s with one worker
add a worker to the same cluster
go to the master ssh on it and use kubectl get nodes
the other worker is not shown

which network/s did you face the problem on?
Main
version
2.5.2
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
na

tried on different deployment and faced this

The issue isn't Reproducible anymore.


Make the recovery phrase confirmation and Account recovery more user friendly

Instead of asking the user to type in his whole recovery phrase, we could ask to type in the 3rd, 7th and 12th word for instance.
This should go together with an account recovery screen with suggestions once the user has typed 3 letters (in case they stored it with a typo).

Proposed by @fckt

Alexey
Encrypted by a verified device
hey! I've got a really simple solution on how to solve this problem: "we want to force users to backup their seed, but we don't want to be annoying and force users to do what they don't want to do"
Encrypted by a verified device
so the idea: on the "seed check screen" we can add a bottom, entitled as "Check later" and then add a flag to that account which means "there is no prove that this account is recoverable" and show it somehow in the interface
Alexey
15:07Encrypted by a verified device
we could go further even. If, for example, user stores some good amount of ETH on account  for which we have no proof that it could be recovered we can warn user somehow

If I'm not mistaken Exodus does this: the Backup step is optional, but if you don't backup they show a warning sign.

That definitely a good idea, Status and mist are doing it in a similar manner too.


IntelliSense is lost on empty line immediately following directives

Issue:
@page
|

don't type the pipe but try and invoke intelliSense at the pipe.
You'd expect for HTML IntelliSense but in actuality you don't get IntelliSense. This seems to be due to our SpanKind.None addition; basically we mark the end of @page as SpanKind.None (expected) but that has an adverse effect of turning off IntelliSense on the line following it (if there's no content). We should be adding a marker symbol after our SpanKind.None to indicate to the editor that we expect HTML at that location.
This was originally reported in TFS: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/584082
Putting this in the VS15.7 milestone to correspond with either preview3 or 4.

9dc7c65205ef1e0a85c39c0fb067acbdcd1bf0b4


[fix] fix get job metrics get NPE

Purpose of this pull request
part fix of https://github.com/apache/seatunnel/issues/8081

Does this PR introduce any user-facing change?
no

How was this patch tested?
exist tests

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

part of fix, to avoid get NullPointException. We still need find why the running job can't be find in the IMAP


Reducing the complexity of the examples with singleton and pipelines in new lines

It’s not technically better, but understanding has improved.

Hi, yep, I think this will improve readability and remove di noise.


[cli] Answers file ignored when installing app from a catalogue as it's impossible to avoid interactive question dialogue

Rancher versions:
rancher/cli: 2.0.5
rancher/server or rancher/rancher: 2.1.0
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
macOS, Darwin 17.7.0
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
not applicable
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
not applicable
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
not applicable
Steps to Reproduce:
Install rancher catalogue chart and provide answers file using:
rancher app install --answers myanswers.json appchart appname
Results:
Instead of respecting the contents of answers file, rancher-cli starts interactive dialogue for question responses to fill out some missing pieces if not provided in answers file.
Looking at the code, when installing catalogue chart, it always runs interactive question dialog no matter what, see https://github.com/rancher/cli/blob/master/cmd/app.go#L565.
This ruins all the scriptability as there is no way to avoid it. I suggest turning off the interactive question mode when answer file is provided and fall back to defaults where applicable. Or at least provide a toggle for that mode.

Following the same steps described in the ticket. The bug is reproduced on CLI 2.0.5 , and the bug fix is validated on CLI v2.2.0-rc5
Result: there is no prompt when adding --no-prompt option and even the answer file does not contain everything.


Use mocks for remote/integration testing

As developer of HSC I would like to use a mock service to test for misc. external resources as the real resources are sometimes not stable which leads to frequent flaky errors and unstable builds.

Hey @ascheman
I have some experience with testing. I would love to contribute!
Can you help me on where to start from

Hi @RehanChalana,
thanks a lot, I appreciate your offer.
My idea would be to mock away external services which are used for testing (and are somewhat unstable in their availability).
If you take htmlSanityCheck-core/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenHttpLinksCheckerSpec.groovy as a starting point, you will find several good and bad URLs referring to external sites. If you run the test case while not connected to the Internet, it will run into (false) errors.
Hence, I would like to start a (local) mock service and run the tests against the mocked endpoints. Suitable technologies could be Wiremock, and perhaps Testcontainers? But I am happy if you see another approach.

Thank you so much for the detailed response!
To start, I am working on htmlSanityCheckcore/src/test/groovy/org/aim42/htmlsanitycheck/check/BrokenHttpLinksCheckerSpec.groovy I’m trying to use WireMock as you mentioned, but I’ve encountered an issue since its latest version requires Java 11 or higher. I’m still figuring it out but its fun!

Are you on (the Hacktoberfest) Discord? You can reach out to me (gerdaschemann) so we could have a chat about the problems you encounter (or if you need any other support).
I am also happy if you will contribute to the development documentation of the project if you manage to solve your issues. A second PR wrt. to docs could increase your Hacktoberfest score.

Yes i am on discord! as well as participating in hacktoberfest!
I sent you a friend request on discord as well. my username is rc__22.
Thanks !


Select all button

Sometimes you want to be aggressive with your tagging strategy. This would be a nice option to enable if you are interested in starting with everything clicked and want to take the irrelevant tags away.

Ah... if you mean a checkbox or link for each image to select/deselect all keyword checkboxes, I'm not sure that this is possible to do. I'll have to look into it further, but I don't think that the Lr SDK makes it very easy to add these sort of "select all/deselect all" toggle checkboxes that are so common in a Web-based GUI. I actually had it in mind at some point and, at the time, I didn't see a simple solution for this. I'll keep this issue open, though, since I agree that that could be a nice-to-have.