fatal: can not remove last element from an empty collection

Runtime error causing fatal crash when using finder extension toolbar actions
Error was present in prior versions as well
Steps to reproduce

Enable finder extension in settings

Add actions to finder toolbar

Select 'Create link for'

Attempts to use finder actions again will demonstrate proof of fatal crash, both finder actions and context actions will disappear until manually ending process and reopening application or log in/out or reboot.

Console crash report
Process:               quick-symlink-extension [56034]
Path:                  /Applications/Utilities/quick-symlink.app/Contents/PlugIns/quick-symlink-extension.appex/Contents/MacOS/quick-symlink-extension
Identifier:            org.ololx.quick-symlink.quick-symlink-extension
Version:               1.0 (1)
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           quick-symlink-extension [56034]
User ID:               ***

Date/Time:             2021-09-11 08:49:13.739 -0500
OS Version:            macOS 11.5.2 (20G95)
Report Version:        12
Anonymous UUID:        0704A1A4-9780-5E31-3D9F-AF8CB5C7572D

Sleep/Wake UUID:       D8D99E98-AA45-4CD3-AFE2-79B50E9B21EC

Time Awake Since Boot: 840000 seconds
Time Since Wake:       45000 seconds

System Integrity Protection: enabled

Crashed Thread:        2  Dispatch queue: com.apple.NSXPCConnection.user.endpoint

Exception Type:        EXC_BAD_INSTRUCTION (SIGILL)
Exception Codes:       0x0000000000000001, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Illegal instruction: 4
Termination Reason:    Namespace SIGNAL, Code 0x4
Terminating Process:   exc handler [56034]

Application Specific Information:
Fatal error: Can't remove last element from an empty collection

Thread 0:: Dispatch queue: com.apple.main-thread
0   libsystem_kernel.dylib        	0x00007fff204a72ba mach_msg_trap + 10
1   libsystem_kernel.dylib        	0x00007fff204a762c mach_msg + 60
2   com.apple.CoreFoundation      	0x00007fff205d5b2f __CFRunLoopServiceMachPort + 316
3   com.apple.CoreFoundation      	0x00007fff205d420f __CFRunLoopRun + 1328
4   com.apple.CoreFoundation      	0x00007fff205d361c CFRunLoopRunSpecific + 563
5   com.apple.HIToolbox           	0x00007fff28819a83 RunCurrentEventLoopInMode + 292
6   com.apple.HIToolbox           	0x00007fff288197e5 ReceiveNextEventCommon + 587
7   com.apple.HIToolbox           	0x00007fff28819583 _BlockUntilNextEventMatchingListInModeWithFilter + 70
8   com.apple.AppKit              	0x00007fff22ddb172 _DPSNextEvent + 864
9   com.apple.AppKit              	0x00007fff22dd9945 -[NSApplication(NSEvent) _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 1364
10  com.apple.AppKit              	0x00007fff22dcbc69 -[NSApplication run] + 586
11  com.apple.AppKit              	0x00007fff22d9fe6c NSApplicationMain + 816
12  libxpc.dylib                  	0x00007fff2022b3eb _xpc_objc_main + 919
13  libxpc.dylib                  	0x00007fff2022acd3 xpc_main + 116
14  com.apple.Foundation          	0x00007fff21384986 -[NSXPCListener resume] + 262
15  com.apple.pluginkit.framework 	0x00007fff2b0e0c17 0x7fff2b0c5000 + 113687
16  com.apple.pluginkit.framework 	0x00007fff2b0e089f 0x7fff2b0c5000 + 112799
17  com.apple.pluginkit.framework 	0x00007fff2b0e0fdd 0x7fff2b0c5000 + 114653
18  com.apple.ExtensionKit        	0x00007fff318f803c EXExtensionMain + 70
19  com.apple.Foundation          	0x00007fff213f360f NSExtensionMain + 208
20  libdyld.dylib                 	0x00007fff204f7f3d start + 1

Thread 1:: com.apple.NSEventThread
0   libsystem_kernel.dylib        	0x00007fff204a72ba mach_msg_trap + 10
1   libsystem_kernel.dylib        	0x00007fff204a762c mach_msg + 60
2   com.apple.CoreFoundation      	0x00007fff205d5b2f __CFRunLoopServiceMachPort + 316
3   com.apple.CoreFoundation      	0x00007fff205d420f __CFRunLoopRun + 1328
4   com.apple.CoreFoundation      	0x00007fff205d361c CFRunLoopRunSpecific + 563
5   com.apple.AppKit              	0x00007fff22f6123a _NSEventThread + 124
6   libsystem_pthread.dylib       	0x00007fff204dc8fc _pthread_start + 224
7   libsystem_pthread.dylib       	0x00007fff204d8443 thread_start + 15

Thread 2 Crashed:: Dispatch queue: com.apple.NSXPCConnection.user.endpoint
0   libswiftCore.dylib            	0x0000000107f02170 specialized _fatalErrorMessage(_:_:file:line:flags:) + 96
1   libswiftCore.dylib            	0x0000000107d4b7ae RangeReplaceableCollection<A>.removeLast() + 590
2   org.ololx.quick-symlink.quick-symlink-extension	0x0000000107ca0c0b CopyPathAction.execute() + 699
3   org.ololx.quick-symlink.quick-symlink-extension	0x0000000107c983b7 FinderSync.copyPathToClipboard(_:) + 119
4   org.ololx.quick-symlink.quick-symlink-extension	0x0000000107c98408 @objc FinderSync.copyPathToClipboard(_:) + 56
5   com.apple.finder.FinderSync   	0x00007fff6ea3265e -[FIFinderSyncExtension executeCommandWithMenuItemDictionary:target:items:] + 298
6   com.apple.Foundation          	0x00007fff213c91be __NSXPCCONNECTION_IS_CALLING_OUT_TO_EXPORTED_OBJECT_S3__ + 10
7   com.apple.Foundation          	0x00007fff2136db6f -[NSXPCConnection _decodeAndInvokeMessageWithEvent:flags:] + 2246
8   com.apple.Foundation          	0x00007fff21325039 message_handler + 206
9   libxpc.dylib                  	0x00007fff20222c24 _xpc_connection_call_event_handler + 56
10  libxpc.dylib                  	0x00007fff20221a9b _xpc_connection_mach_event + 938
11  libdispatch.dylib             	0x00007fff203328a6 _dispatch_client_callout4 + 9
12  libdispatch.dylib             	0x00007fff20349aa0 _dispatch_mach_msg_invoke + 444
13  libdispatch.dylib             	0x00007fff20338493 _dispatch_lane_serial_drain + 263
14  libdispatch.dylib             	0x00007fff2034a5e2 _dispatch_mach_invoke + 484
15  libdispatch.dylib             	0x00007fff20338493 _dispatch_lane_serial_drain + 263
16  libdispatch.dylib             	0x00007fff203390e0 _dispatch_lane_invoke + 417
17  libdispatch.dylib             	0x00007fff20342c0d _dispatch_workloop_worker_thread + 811
18  libsystem_pthread.dylib       	0x00007fff204d945d _pthread_wqthread + 314
19  libsystem_pthread.dylib       	0x00007fff204d842f start_wqthread + 15

Thread 2 crashed with X86 Thread State (64-bit):
  rax: 0x0000000000001e0b  rbx: 0x0000000108098680  rcx: 0x0000000000000000  rdx: 0x00000000000aea80
  rdi: 0x000000010842a080  rsi: 0x0000000108876e00  rbp: 0x0000700004967ba0  rsp: 0x0000700004967b70
   r8: 0x0000000000000030   r9: 0x0000000000000003  r10: 0x00007f8e0de00000  r11: 0x00007f8e0de46f40
  r12: 0x0000000108097b79  r13: 0x000000000000000b  r14: 0x0000000000000002  r15: 0x0000000000000032
  rip: 0x0000000107f02170  rfl: 0x0000000000010202  cr2: 0x00000001028f7000

Logical CPU:     3
Error Code:      0x00000000
Trap Number:     6

Hi, @zgover,
I didn't reproduced this on my machine; But I think I figured out where this error occurs and will try to fix it today. After my main job.
Thanks for finding this bug and reporting it.

@ololx feel free to share your reason, I can always contribute and make a PR if I get enough downtime to debug and fix before you do.

@zgover,
Could you be so kind to check this fast fix and let me know that everything is fine.
I also wanted to warn you that before completing work on #3, I want to refactor the code - it is already starting to resemble a trash can =)
I will continue to keep you up to date with the development process.

@ololx confirmed, the recent release v0.8.1 has fixed the finder extension toolbar crash.


matrix.cpp:310: error: (-215) s >= 0 in function cv::setSize

matrix.cpp:310: error: (-215) s >= 0 in function cv::setSize
i am getting this error at:
-> face_recognizer.train(faces, np.array(labels))
why?
this is a labels np.array:
[0 0 0 1 2 2 2 3 3 3 3 4 4 4 5 5]
and this is a list of faces:
[array([ 36, 22, 244, 244]), array([ 20, 23, 249, 249]), array([ 32, 20, 245, 245]), array([ 39, 26, 239, 239]), array([ 19, 23, 248, 248]), array([ 28, 22, 235, 235]), array([ 28, 11, 243, 243]), array([ 39, 26, 238, 238]), array([ 27, 26, 243, 243]), array([ 21, 21, 257, 257]), array([ 35, 30, 245, 245]), array([ 33, 28, 246, 246]), array([ 24, 20, 257, 257]), array([ 32, 27, 250, 250]), array([ 35, 31, 232, 232]), array([ 35, 25, 236, 236])]
what is a problem?

it was my fault sorry

what was the issue, I am getting the same Error.
The error i am getting is as follows-
error: (-215:Assertion failed) s >= 0 in function 'cv::setSize'


HTC M8 for Windows Phone

Hi guys,
I have a HTC M8 for Windows and I just got the app. But just can't login. All I'm having access to is the home screen and every time I try to get in, the app crashes.
ps: I have the windows 10 already.

This have already been documented before.
It's not a device specific issue but rather an API problem.


None role type exists

http://policykit.org/main/policyengine/policykitdeleterole/add/ (see base user and none)

in add/delete user roles etc.

is this still a bug?


support to airflow 2.6.0

just loosing up the requirements to allow airflow 2.6.0
linked to: #65

Hi @CorsettiS thank you for your contribution! In fact, the tests for 2.6.0 have not been run: you have to update airflow-version matrix in .github/workflows/tests.yml too. Please do 🙏

@bryzgaloff thanks for noticing it ! just did it

Thank you @CorsettiS! This change is released as v0.11.0: https://github.com/bryzgaloff/airflow-clickhouse-plugin/releases/tag/v0.11.0
I am also adding to the contributors list in README 👏

Resolves #65


Update user labels

Given the Users List & User Details pages, the Disabled/Deleted fields should read "Disable After"/"Delete After", so that the fields better communicate what will happen after the duration has passed.
Additional Context
The current labels of "Disabled In"/"Deleted In" give the impression that these hooks might trigger immediately when the duration is up. This is not the case, and updating the labels better communicates this behavior.

/backport v2.7.patch1

/forwardport v2.9.0

Tests pass on v2.8.4-rc5-s4


Fix unit tests on x-plat

Assume case sensitivity.
Change \ to /.
@muratg @BrennanConroy

:shipit:

YAY! It's great to finally see StaticFiles green @davidfowl

Keep some Windows specific paths and tests. Just skip them xplat. We need to cover everything


Move fullscreenKeyboardWatcher() into hs.finalcutpro

Actually, I think this should be moved into a plugin.

I feel like the watcher itself should be part of hs.finalcutpro, but the functions called by the watcher should be in the plugin.
I think eventually we should have hs.finalcutpro.watcher that allow us to trigger things when Final Cut Pro is opened, closed, hidden, gone into full screen mode, the Command Editor is opened or close, the preferences files has been updated, etc.

What is it watching exactly? The fullscreen window? Or the keyboard?
On Sat, 7 Jan 2017 at 10:53 pm, Chris Hocking<EMAIL_ADDRESS>wrote:

I feel like the watcher itself should be part of hs.finalcutpro, but the
functions called by the watcher should be in the plugin.
I think eventually we should have hs.finalcutpro.watcher that allow us to
trigger things when Final Cut Pro is opened, closed, hidden, gone into full
screen mode, the Command Editor is opened or close, the preferences files
has been updated, etc.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/fcpxhacks/fcpxhacks/issues/161#issuecomment-271082153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADbEw_pw-bziDSk95MZdlTRUKX3nmBVTks5rP4rQgaJpZM4LZMEr
.

I've created watch/unwatch methods in FullScreenWindow in the #217 issue, and the function has been migrated into a plugin. Can close this now.


Errors: unexpected token: '!' + AreaChart is not defined

Hi,
I tried to play a little bit with this wonderful analyzer, I must admit that it was a bit a struggle but I got it working. The page loads but sadly it won't show any graphs and print two erros in the console:

SyntaxError: unexpected token: '!'
ReferenceError: AreaChart is not defined

Does anyone have any idea how to fix it ?
Thanks,
K

I'm guessing this is related to #1570. Did you build from development/upstream?

Indeed !

Same here on sources from master. Reverted back to v1.3 and it's all good.

This should be fixed per #1595.
Closing it, feel free to reopen it if needed.


Create ArtemisMQ

Ive created an ArtemisMQ which uses the ActiveMQ Artemis JMS Client. Also have made the code use async sending with completion listener (jms 2.0 feature). And some hopefully broker side tunings.
I am not a Scala expert as such please let me know if any scala issues.

Can one of the admins verify this patch?

I think something is wrong with this setup - I'm getting 3 messages/second :)

annoyingly without access to see what, nor any logs its hard to work out, no doubt its something obvious, or my scala code for the producer, as there are nuances between scala and java. maybe too many changes in one go.
With lack of access, i guess best option will be for me to send a PR with just a subset of the changes and we can go from there. (I've also just pinged someone else to cross check in case i made a silly mistake)

someone else's eagle eyes (thanks @franz1981) just spotted we changed the journal from asyncio to mapped and turned off fsync, but forgot to change the buffer timeout to 1 as we don't need this.
Can you change this from:
736000
To:
1

Also I think it could be my Scala code in jms2mq. As such the pr i sent makes artemismq extend jmsmq (sync send) just to eliminate that

@adamw

I think something is wrong with this setup - I'm getting 3 messages/second :)

Probably it is due to an unlucky mix of configurations, with a Raspberry 3 I'm getting order of magnitude more eheh
In order to help better on it and just to summarize:

2.1 Artemis broker
736000
MAPPED journal
datasync false
core JMS client

I'm expecting OS and HW are the same of the post, right?
Anyway thanks for the time spent on it :+1:

@franz1981 i spotted the issue, its my async client Scala code i added, using latches that is the cause.
I update a PR for @adamw which moves him back to sync style code, but keeps all the basic bits.
Move to use Artemis Client
Mapped
datasync false
I did enable his transaction so will do batch transaction send (as noted clebert already advised this, but i don't think clebert noticed @adamw was using the 5.x client not the artemis one).
As noted on that PR @adamw if you could merge and run.
I will have to look at my async code, but tbh with artemis client, batch transaction, mapped, data sync false it should have some improvement.


Is there a way to conditionally render select options.

I was trying to make two columns select one column options depends upon the other. Basically i need to filter the options in second select from the value selected in first select column. Is it possible?

Hey @bytesagar I don't think that's possible, you would need to fork the project.

Hello @masiulis. Is there any way that I can access column value inside fielts.fieldType.options? I tried playing with the code but was not able to.


Did behavior of SQL output pane change?

I'm using the 0.6.1 version of the VS Code extension. If I open a PRQL query from a file, then hit Cmd-Shift-P to open the PRQL Extension, the right half of the window fills with the generated SQL as expected.
But if I open another PRQL query file in the left half, the SQL in the right side does not change.
My impression is that this is different from previous versions, where changing the PRQL query displayed changes the SQL Output window. Or am I simply mistaken... Thanks.

Yes, this seems to happen — the preview is now linked to a specific file rather than being generic for whichever file was last open.
Without understanding all the tradeoffs, I marginally preferred the old approach (i.e. possibly there are other things this approach enables).
One other thing I see — with two files open, each with previews available, when switching between PRQL files, the focus is grabbed by the preview. Ideally the preview would never grab focus (there's not much we can do with it...)

I wanted to follow up on my initial report. After playing with it more, I finally realized that the 0.6.1 version is designed to make it easier to preserve the SQL results from the PRQL query.
I strongly prefer the prior (0.6.0?) behavior. In my workflow, I like to treat the PRQL query files as the sole "source of truth". I don't find a need to keep the SQL results. They're just interim data that I copy/paste into my database.
Here are the features that I appreciate/treasure from the extension, and how I'd like to operate again:

The SQL Preview Pane simply displays the current PRQL file.

When I open the SQL Preview Pane, it displays the results of the current PRQL file open on the left.
Clicking another tab on the left or opening a new .prql file should switch the SQL Preview Pane to show the result of that query.

I like the "Copy SQL to Clipboard" icon at the top
For myself, I don't need either the "Generate SQL File" icon at the top (I don't preserve SQL results), or the "Open SQL Preview" icon (since it would always be present)
I like the "Copied SQL to Clipboard" confirmation. It can go away automatically after ~5-10 seconds.

Thanks.

Sql Preview has been refactored and changed to support multiple PRQL document output views, serialization, and restore on vscode reload and many other features centered around individual PRQL documents linked to the corresponding SQL Preview views rather than using a single view as before. See #60 and #108 for more info and changed behaviour.

In the refactoring, I have lost the simplicity of earlier versions. Here's the result I would like: when I open any ".prql" file, I want to copy the SQL and paste it into my database. This means:

The "Copy SQL to Clipboard" icon would always active active.
The SQL Preview pane doesn't need to be open/visible. (If I want to view the SQL, I can always open the preview pane)
All the rest of the icons (Generate SQL File, Open SQL Preview, View PRQL Setting) can remain the same

I am hopeful that this is a straightforward change. Thanks again for a terrific tool.

@richb-hanover That is not how Copy SQL to Clipboard fucntionality was designed, and that functionality did not exist in prior version. You need an open SQL Preview to Copy generated SQL from it. You can also use Generate SQL file that opens the resulting SQL you can copy from a code editor and not commit that sql to your project repo, if you don't want to keep sql files in your project.
In any case, none of that Copy SQL or generate SQL file functionality was in the prior vscode extension v0.4.x you are referencing, and those changes were not made to remove the "simplicity" of the earlier versions. They are both new features and shortcuts added to work with PRQL in VSCode. Also, current SQL Preview works as most Previews work in other custom extensions, providing a preview for the open/active document.
You can review those changes in PR #49, #59, #118 with demo gifs if it's still not clear how that functionality works.
We moved on from PRQL code tools and it's up to @max-sixty  and other devs contributing here if they want to rollback those features and added functionality, or close this ticket with the clarifications of those features we've provided here.

Just to level-set on expectations:

Overall the extension added a lot of features as a result of @RandomFractals 's work. I think it's much much better, I'm grateful for the contribution.
As with any change, as we explore the space we might regress on some dimensions. That's why this sort of feedback from @richb-hanover is valuable
I think there's space to re-introduce the existing "single generic preview pane" output that we had before, while keeping the new features
My understanding is that @RandomFractals is working less on the extension now
Others are welcome to contribute! That's the nature of open-source... If @RandomFractals wants to, he's of course very welcome to, but zero obligation

@RandomFractals Thanks for this note. I didn't understand how the new model worked, so I created this ticket. I'm going to close it now, and as you suggest, create a new feature request.
I'll be interested to see your Data Notebook extension. Good luck!

@richb-hanover great! thanks for opening new ticket for your feature request. I shared my notes on implementing it in #165.
Happy we were able to find a good middle ground solution for your use case.
Btw, I would love to share more info about new Data Notebook ext. with PRQL support , but I did not find any public social media handles you use. Even though I know @max-sixty doesn't like us sharing out of context bits here, you and others can find more info about Data Notebook ext. on twitter, where we shared a few demo gifs, etc.: https://twitter.com/search?f=live&q=(%23DataNotebooks) (from%3ATarasNovak)&src=typed_query


[v1.1] add test for submodule setup

add test for submodule setup

if setup is successful, does that mean there won't be any issue in install? Or should we follow line43 to also test on install?

Yes, twine will check the package.

UT passed. Please help to merge. Thx~@xuechendi


iOS: Focus events not being called when navigating into tab stack then into another tab quickly

Current Behavior
Using a stack navigator with a tabs screen and some other full screen - When navigating back from the full screen onto the tabs screen and pressing a tab QUICKLY navigation listeners are not executed.
For example a screen which is contained as a tab with the following:
useEffect(() => {
    const didFocusListener = navigation.addListener("didFocus", handleFocusChanged(true));
    const didBlurListener = navigation.addListener("didBlur", handleFocusChanged(false));
    return () => {
      didFocusListener.remove();
      didBlurListener.remove();
    };
  }, [navigation]);

The didFocusListener is not called if the user pressed the tab fast enough after navigating back to the tabs screen and into the tab.
Expected Behavior
The listener is called
How to reproduce
I apologise I don't have time to make the application for reproduction I will try make time as soon as I have time
A stack navigator containing 1 bottom tab navigator (using createBottomTabNavigator) and one other screen as a react component.
Eg
export const StackNavigator = createStackNavigator(
  {
    [Routes.MainTabs.routeName]: MainTabNavigator,
    [Routes.CameraPermission.routeName]: SomeOtherScreen,
  },
  {
    initialRouteName: Routes.MainTabs.routeName,
  }
);

MainTabNavigator:
export const MainTabNavigator = createBottomTabNavigator(
  {
    [Routes.ScreenOne.routeName]: ScreenOne,
    [Routes.ScreenTwo.routeName]: ScreenTwo,
    [Routes.ScreenThree.routeName]: ScreenThree,
  },
  {
    initialRouteName: Routes.ScreenOne.routeName,
  }
);

In ScreenOne add the following effects:
useEffect(() => {
    const didFocusListener = navigation.addListener("didFocus", handleFocusChanged(true));
    const didBlurListener = navigation.addListener("didBlur", handleFocusChanged(false));
    return () => {
      didFocusListener.remove();
      didBlurListener.remove();
    };
  }, [navigation]);

  useEffect(() => {
    console.log("Running effect:" + Math.random());
  }, [isFocused, navigation]);

Navigate from SomeOtherScreen to ScreenTwo, as quickly as possible tap the tab for ScreenOne.
Observe no console log.
Your Environment

software
version

iOS
13.3

react-navigation
4.0.10

react-navigation-tabs
2.5.6

react-native
0.61.4

node
10.16.3

yarn
1.17.3

We have the exact same issue.


fix sample ui on mobile devices

@khajra please review and merge. this should fix the issue on mobile devices

@jtrieu Looks good!


Deploy on Reef Mainnet

Related PRs:

https://github.com/witnet/witnet-solidity-bridge/pull/299
https://github.com/witnet/witnet-price-feeds/pull/160
https://github.com/witnet/data-feeds-explorer/pull/278
https://github.com/witnet/data-feeds-explorer/pull/279

Published REEF/USDT on DFE:

https://feeds.witnet.io/reef/reef-mainnet_reef-usdt_6


[Event Request] codeunit 6501 "Item Tracking Data Collection" OnSelectMultipleTrackingNoOnBeforePageRunModal

Hello,
Is it possible to add the following Event:
   ItemTrackingSummaryForm.AutoSelectTrackingNo();

    Window.Close;

    // DINNL <<
    IsHandled := false;
    OnSelectMultipleTrackingNoOnBeforePageRunModal(DINIsHandled);
    if not IsHandled then
    // DINNL >>
    if not (ItemTrackingSummaryForm.RunModal = ACTION::LookupOK) then
        exit;
    ItemTrackingSummaryForm.GetSelected(TempEntrySummary);
    if TempEntrySummary.IsEmpty() then
        exit;

    // Swap sign on the selected entries if parent is a negative supply line
    if CurrentSignFactor > 0 then // Negative supply lines
        if TempEntrySummary.Find('-') then

[IntegrationEvent(false, false)]
local procedure OnSelectMultipleTrackingNoOnBeforePageRunModal(var IsHandled: Boolean)
begin
end;

Please explain how you are going to us this event without any parameters except IsHandled?

If we have a certain app installed we don't want that form to run. So we just check if our own app is active.

In this case why don't you use event OnBeforeSelectMultipleTrackingNo in beginning of this procedure? Do you need to run all code before run page, I assume this is not needed in your case?

We are going to check that. Please feel free to close this one in the meanwhile.
Thank you Alexander.


Move examples into a separate examples.js file

Previously the examples was executed in main.js, which had some
unintended consequences.

 Datepicker


🔧 Dependabot updates 2023-01-01

bump coverage
bump iosrt
address security concern in certifi
TODO need to bump mysql-connector-python as soon as they allow updating protobuf >= 3.20.2

closes #104
closes #105

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (24e495a) compared to base (b391523).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #106   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        18
  Lines          549       547    -2
  Branches        91        58   -33
=========================================
- Hits           549       547    -2

Flag
Coverage Δ

3.10
?

3.11
?

3.7
?

3.8
?

3.9
100.00% <ø> (ø)

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

Impacted Files
Coverage Δ

pydapper/commands.py
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


added Hugs

You may want to choose another lang id / lang name. Another alternatives for lang id: haskell-hugs, hugs98.

Went with https://tio.run/#haskell-hugs. I've also enabled command-line arguments.


Fire() method do not write message to fluent if hook.tag is set

setMessage called only if hook.tag == nil

@marrrvin  Thank you for reporting the issue.
I fixed it in #10, but if it still reproduces then please tell me! 🐛


Safari not working with Annotation-Tool

Hi, I just tried the annotation tool demo with safari and I have the problem that the video is not loading correctly, the video screen hangs in "start loading video" step. If I use Chrome I don't have any problems with the annotation-tool demo.
User agent details:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/11.1.2 Safari/605.1.15

This is a somewhat known issue in the sense that the tool is currently mostly tested in Chrome; at least that's what I develop against. It's good to have an issue to track this stuff, the thing is just I don't know when/if I will get to it. For Safari specifically, I don't even have a (simple) way of testing at all, unfortunately. :disappointed:
On the bright side: From internal discussions I gathered that some problems just disappear when you change your user agent string to something that resembles Chrome, so you might have some luck trying that. :slightly_smiling_face:

LABEL: blocker
Changing the user agent string is no option for our teachers/students.

Updated to Safari 12, The issue is still present.

The javascript console shows 404 that can be seen in the screenshot

FYI: The 404s in the console are unfortunate, but don't have anything to do with this problem.

This issue was solved


[ETL-694] Specify python-version in actions/setup-python

Fixes bug described in ticket.

There appears to be another bug with the Snowflake action we are using, as described here: https://github.com/Snowflake-Labs/snowflake-cli-action/issues/28
I think this will be resolved by Snowflake-Labs within the next few days so let's wait on them, rather than specifying a specific version of the action, which would need to be reverted later.

There appears to be another bug with the Snowflake action we are using, as described here: https://github.com/Snowflake-Labs/snowflake-cli-action/issues/28
I think this will be resolved by Snowflake-Labs within the next few days so let's wait on them, rather than specifying a specific version of the action, which would need to be reverted later.

Lets wait a week to see if it gets resolved by snowflake labs.  If not we don't want it to block our work - will the GX work be able to be resumed?

@thomasyu888

will the GX work be able to be resumed?

Technically this is not a blocker, since we aren't making any changes to Snowflake we don't need to worry about breaking anything Snowflake related. We can merge regardless.
But if we wait a few days to see how this plays out, we can see all the green check marks before merging without impeding any dev work, since the GX work is not related to Snowflake, either.

Snowflake-Labs used an unfortunate tag v1 for their initial tag, rather than v1.0. This caused @v1 to point to a specific tag, rather than use the latest minor version of 1.x. The fix at this point seems to be to specify a specific tag, @v1.5, unless Snowflake-Labs fixes the v1 tag, but it's not clear when they might do so.

Snowflake-Labs used an unfortunate tag v1 for their initial tag, rather than v1.0. This caused @v1 to point to a specific tag, rather than use the latest minor version of 1.x. The fix at this point seems to be to specify a specific tag, @v1.5, unless Snowflake-Labs fixes the v1 tag, but it's not clear when they might do so.

Make sense - unfortunate... Agree that lets move forward with V1.5 tag

@thomasyu888 Can I get an approving review? Sorry for the formality.


DynamicViewPresenter UIKit extensions break VIPER-based apps

Great package @vmanot! I was eager to try it, but as my app is currently a VIPER-based UIKit that I'm slowly moving towards SwiftUI, there are still a number of VIPER modules in the codebase. In the VIPER architecture, most UIViewControllers already have a presenter class member, so I've been getting errors like this all across my codebase:
Overriding non-open property outside of its defining module

and
Property 'presenter' with type 'HomeSplitViewControllerPresenter?' cannot override a property with type 'DynamicViewPresenter?'

I realized it's due to this SwiftUIX extension:
https://github.com/SwiftUIX/SwiftUIX/blob/e4acefdd5b6e65574e8b251fd0ce7a89a624d707/Sources/Intramodular/Presentation/DynamicViewPresentable.swift#L19-L27
Surprisingly, all I did was add SwiftUIX to my SPM deps and this error started to crop up even in files where I wasn't importing SwiftUIX directly. Figured I would let you know. Happy to open a PR renaming this member to viewPresenter or something non-conflicting.

@Nickersoft thank you for raising this issue! I'm planning on removing the public conformances completely and using some type erased wrapper for all my UIKit → SwiftUIX presenter needs. SwiftUIX ideally should not be publicly extending any UIKit symbols.

Closing this for now because unfortunately I'm completely out of bandwidth. An exhaustive change is planned, but I cannot commit to any timeline right now due to a lack of time.


Only check validation cookie with enabled amendment

It looks like bySeq_ is only used for that cookies-related check, so I'm not sure if we want more of it behind the amendment enabled check.

Jenkins Build Summary
Built from this commit
Built at 20180503 - 23:42:44
Test Results

Build Type
Log
Result
Status

docs
logfile
1 cases, 0 failed, t: 2s
PASS :white_check_mark:

clang.debug.unity
logfile
1022 cases, 5 failed, t: 251s
FAIL :red_circle:

coverage
logfile
1022 cases, 5 failed, t: 517s
FAIL :red_circle:

clang.debug.nounity
logfile
no test results, t: 33s
FAIL :red_circle:

msvc.debug
console
no test results, t: n/a
FAIL :red_circle:

clang.debug.unity, PARALLEL_TESTS=false
logfile
1022 cases, 5 failed, t: 482s
FAIL :red_circle:

gcc.debug.nounity
logfile
no test results, t: 34s
FAIL :red_circle:

gcc.debug.unity
logfile
1022 cases, 5 failed, t: 244s
FAIL :red_circle:

msvc.debug, PROJECT_NAME=rippled_classic
console
no test results, t: n/a
FAIL :red_circle:

clang.release.unity
logfile
1021 cases, 5 failed, t: 327s
FAIL :red_circle:

gcc.release.unity
logfile
1021 cases, 5 failed, t: 350s
FAIL :red_circle:

gcc.debug.unity -Dstatic=true
logfile
1022 cases, 5 failed, t: 242s
FAIL :red_circle:

msvc.release
console
no test results, t: n/a
FAIL :red_circle:

closing in favor of https://github.com/ripple/rippled/pull/2506


Automatically generate password to avoid using Convert-To-SecureString

A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

/azp run

/azp run

/azp run


fix(deallocate_middle): advance to next in try_insert_after

If the next hole node is still lower than the hole to be added, advance the cursor to the next in the list.
For this to trigger, this patch modifies the deallocate_middle() test.
It requires two holes in the list before the one to be deallocated.
Signed-off-by: Harald Hoyer<EMAIL_ADDRESS>

Hey @haraldh, I think I just hit the same issue in #64, I'll be adding more tests.
Sorry for the issue!

Superceded by #64, thank you for reporting @haraldh!


[material-ui][docs] Incorrect material ui version in stackblitz demos

Steps to reproduce
Steps:

Open this link to live example: (required) https://mui.com/material-ui/react-text-field/
click on edit in stackblitz
notice material ui version

Current behavior
material-ui is v5.x
Expected behavior
material-ui version should be v6.x
Context
https://github.com/user-attachments/assets/adbd8ca8-2378-42a8-b9ba-f57833db47b6
Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: docs version

This should be fixed in a few hours. I pushed the latest tag an hour ago.

It's working correctly now.


accept JobConfig and PackageConfig equally

[x] fix tests

since we want JobConfig to be as powerful as PackageConfig, we need to
make sure that packit work with it via API
Follow-up to: #858
Needed for packit-service/packit-service#418
Signed-off-by: Tomas Tomecek<EMAIL_ADDRESS>

Would using CommonPackageConfig instead work?
My concern with the current implementation is that a JobConfig is not a PackageConfig (it's missing job entries, has metadata, so on..), so mixing types like this can create confusion and open the door for those ugly check-the-type-before-doing-something "solutions".

Would using CommonPackageConfig instead work?
My concern with the current implementation is that a JobConfig is not a PackageConfig (it's missing job entries, has metadata, so on..), so mixing types like this can create confusion and open the door for those ugly check-the-type-before-doing-something "solutions".

okay, that makes sense, gonna give it a go
there is only a single place in packit codebase where jobs are being processed so it shouldn't be an issue right now, but could be in future if packit works with jobs more


train my custom dataset

Thank you for your contribution, I want to train my custom dataset with senet,What should I do?

Hi, check https://github.com/pytorch/examples and replace the network with SENet in this repository.


Add cure53 review of node_exporter

DCO broken by that web commit, but I am fine merging as is.

:+1:


Unable to use some property controls in ACEs

Category

[ ] Enhancement
[x] Bug
[ ] Question

Version
Please specify what version of the library you are using: [3.4.0]
Cause of the issue
Some of the PnP SPFx property controls are importing IPropertyPaneField , PropertyPaneFieldType and other data from @microsoft/sp-webpart-base instead of @microsoft/sp-property-pane.

Expected / Desired Behavior / Question
Property pane controls should be usable in ACEs like we use them in SPFx web parts.
Observed Behavior
When some property pane controls are added to ACEs an error is seen when we run gulp serve and hence when we edit the ACE the property pane doesn't load.
Steps to Reproduce

Create an SPFx ACE
In the property pane add a PnP SPFx property control (e.g. PropertyFieldCollectionData or PropertyFieldGuid)
Save and run gulp serve
See errors in the terminal window

@anoopt many thanks for raising this issue and providing detailed information.
This is now fixed and will be available as part of the next major release.
It will also be available in the beta release that should be published within the next few minutes.

Thank you @joelfmrodrigues


Backport fix for #6261

I'm experiencing the same warnings reported in #6261 on storybook v4. It seems like the fix for that (#6267) needs to be ported back.

Seems something is wrong:
ERR! => Failed to build the manager
ERR! ./.storybook/addons.js
ERR! Module build failed (from /tmp/storybook/node_modules/babel-loader/lib/index.js):
ERR! Error: [BABEL] /tmp/storybook/examples/cra-kitchen-sink/.storybook/addons.js: Invalid Option: corejs is not a valid top-level option.
ERR!         Maybe you meant to use 'targets'? (While processing<EMAIL_ADDRESS>ERR!     at validateTopLevelOptions (/tmp/storybook/node_modules/@babel/preset-env/lib/normalize-options.js:49:13)
ERR!     at normalizeOptions (/tmp/storybook/node_modules/@babel/preset-env/lib/normalize-options.js:160:3)
ERR!     at _default (/tmp/storybook/node_modules/@babel/preset-env/lib/index.js:168:37)
ERR!     at /tmp/storybook/node_modules/@babel/helper-plugin-utils/lib/index.js:19:12
ERR!     at loadDescriptor (/tmp/storybook/node_modules/@babel/core/lib/config/full.js:165:14)
ERR!     at cachedFunction (/tmp/storybook/node_modules/@babel/core/lib/config/caching.js:33:19)
ERR!     at loadPresetDescriptor (/tmp/storybook/node_modules/@babel/core/lib/config/full.js:235:63)
ERR!     at config.presets.reduce (/tmp/storybook/node_modules/@babel/core/lib/config/full.js:77:21)
ERR!     at Array.reduce (<anonymous>)
ERR!     at recurseDescriptors (/tmp/storybook/node_modules/@babel/core/lib/config/full.js:74:38)
ERR!     at loadFullConfig (/tmp/storybook/node_modules/@babel/core/lib/config/full.js:108:6)
ERR!     at process.nextTick (/tmp/storybook/node_modules/@babel/core/lib/transform.js:28:33)
ERR!     at _combinedTickCallback (internal/process/next_tick.js:132:7)
ERR!     at process._tickCallback (internal/process/next_tick.js:181:9)
ERR!  @ multi ./.storybook/addons.js /tmp/storybook/lib/core/dist/client/manager/index.js main[0]

Hm, it sounds like maybe it’s the wrong fix:
https://github.com/storybooks/storybook/issues/6261#issuecomment-485574481

Thanks anyway 😃


Add system property to avoid whitelisting apim rest apis scopes

Purpose
This PR will add a system property to avoid whitelisting product rest apis scopes in km connector in iskm setup.
If someone needs to restrict whitelisting product rest apis scopes, he needs to define the follwing 2 system propertise as true.
Sytem properties:
restrict.apim.restapi.scopes=true
restrict.unassigned.scopes=true
Startup:
./wso2server.sh -Drestrict.unassigned.scopes=true -Drestrict.unassigned.scopes=true
Fixes

https://github.com/wso2/api-manager/issues/2363

This PR is not needed now since the code has moved to the IS side, and this is covered from https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2328. Hence closing this.

https://github.com/wso2-extensions/apim-km-wso2is/pull/129


Fixed issue #2

https://github.com/bioidiap/bob.io.matlab/issues/2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/core
I managed to fix the issue adding -pthread  in the CFLAGS and -lpthread in LDFLAGS, but I don't understand why do I have to do this to make it work with conda.
Also, after building I had an issue when I tried to load the library and the issue was fixed exporting this variable
DYLD_FALLBACK_LIBRARY_PATH=$PREFIX/lib.
Why should I do that? Btw, I took this tip from here https://github.com/conda-forge/libmatio-feedstock/blob/master/recipe/build.sh#L13
Thanks

@conda-forge/core @tiagofrepereira2012 means this error:
ERROR: Failure: ImportError (dlopen(/Users/tiagodefreitaspereira/Documents/buildOutProjects/bioidiap/bob.io.matlab/bob/io/matlab/_library.so, 2): Library not loaded: @rpath/libmatio.4.dylib
  Referenced from: /Users/tiagodefreitaspereira/Documents/buildOutProjects/bioidiap/bob.io.matlab/bob/io/matlab/_library.so
  Reason: image not found)

LGTM. Thanks @tiagofrepereira2012.

Sorry nobody got back to you @tiagofrepereira2012. I have an answer to the question you ask - I presume it is a surprise to you to need the -pthread option for bob.io.matlab? Could it be that the python you normally compile with already has that in the python-confg flags?
$ python-config --cflags

-I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -fno-strict-aliasing -fno-common -dynamic -arch x86_64 -arch i386 -g -Os -pipe -fno-common -fno-strict-aliasing -fwrapv -DENABLE_DTRACE -DMACOSX -DNDEBUG -Wall -Wstrict-prototypes -Wshorten-64-to-32 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -DENABLE_DTRACE

I have no pthread in there, but maybe you do when not compiling with conda?


[logs] Refactor old Hue logs views into new public APIs

What changes were proposed in this pull request?

Current log methods in Hue are super old and returning server-side mako based view responses.
This refactoring helps in improving the core logic and present it as a public API.

This will also help the ongoing revamp of Hue server logs page to consume the new APIs. Once the new server logs page is ready, we can cleanup all related old code.

How was this patch tested?

Manually
Unit tests

Will be adding unit tests shortly

If possible, could you please add tests for "download_hue_logs" as well?

If possible, could you please add tests for "download_hue_logs" as well?

I tried but there are lots of moving pieces so integration tests are better for it.


Module compiled with Swift 5.0 cannot be imported by the Swift 5.1.3

Hi, I'm facing issues to run and check your code in the latest Swift version. Requesting you to set BUILD_LIBRARY_FOR_DISTRIBUTION = YES in your framework.
Source: https://stackoverflow.com/questions/58654714/module-compiled-with-swift-5-1-cannot-be-imported-by-the-swift-5-1-2-compiler
Thanks in advance.

@SimonFernandes7 Thanks for addressing the issue. Fix is available, please clone the latest.


Implement string joining in the compiler

We have string 'split' but no corresponding 'join' method.
This should be implemented as a 'convert_method_call' method. The actual string joining should be done in C++. Like the implementation of split which defers to runtime_functions.string_find which calls nativepython_runtime_string_split in _runtime.cpp.
In this case, you'll produce a function like
void nativepython_runtime_string_join(StringType::layout* outString, StringType::layout* separator, ListOfType::layout* toJoin)

that defers to functions implemented in StringType to do the actual work. You should be able to look at the split code for examples.
We mimic python string internals: each string uses 1,2, or 4 bytes per codepoint depending on the largest codepoint in the string. You'll need to compute the number of bytes per codepoint across all the strings, compute the appropriate size of the output string, and then bitcopy all the relevant substrings in (making sure to cast each string to the max number of bits used as you go).
The string_wrapper needs to make sure it passes a ListOf(str), but the function should work for any iterable. So it should first pass the argument into a function (using call_py_function) that checks whether its argument is ListOf(str) and if not, use call_py_function to call something that iterates the arguments, verifies they're strings, and pushes them into a ListOf(str).

The layout class is.
    class layout {
    public:
        std::atomic<int64_t> refcount;
        typed_python_hash_type hash_cache;
        int32_t count;
        int32_t reserved;
        uint8_t* data;
    };```

The passed as list is `ListOfType::layout* toJoin`.

How can I iterate through the list if I don't have the size of the element?

In this case, you know you are joining a ListOf(str), so the element Type*
is String::Make(). You must ensure that this is the case from the calling
context.  Also, for the most part, the best way to interact with the layout
object is to produce the relevant list type and use that.  For instance,
you can see StringType::split has code like
    static ListOfType* listofstring = ListOfType::Make(StringType::Make());
    listofstring->resize((instance_ptr)&outList, 0, 0);

which you can see is producing the type for ListOf(str) listofstring and
using it to interact with the layout.
On Fri, Jul 19, 2019 at 5:54 AM Szymon Lipiński<EMAIL_ADDRESS>wrote:

The layout class is.
class layout {
public:
    std::atomic<int64_t> refcount;
    typed_python_hash_type hash_cache;
    int32_t count;
    int32_t reserved;
    uint8_t* data;
};```

The passed as list is ListOfType::layout* toJoin.
How can I iterate through the list if I don't have the size of the element?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/APrioriInvestments/nativepython/issues/191?email_source=notifications&email_token=AB6OHBEGJRFRCQTZWQ7SH53QAGFLZA5CNFSM4IEHEPU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2LFT6Y#issuecomment-513169915,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6OHBBJ4WX5GKA3K2UHDZDQAGFLZANCNFSM4IEHEPUQ
.

Thanks. I need your help with two things:

Could you review this code, please? https://github.com/APrioriInvestments/nativepython/compare/szymon-dev

I get error and I'm not sure about the proper fix.

typed_python/StringType.cpp:826:35: error: invalid use of ‘StringType::layout::layout’
     int maxCodePoint = separator->layout->bytes_per_codepoint;

Even if I remove the wrong code, then the test fails with there is method join for string, and I'm not sure what I'm missing.


Strange interaction between Blast and Velocity on multiple divs

Hey, I've noticed a problem when you try to use the same function twice on 2 divs with the same class. I've made a demo on codepen here, illustrating the problem. The function only works on the last div.
http://codepen.io/sdras/pen/8d7f8116818ec9ec00f8f82be65838d5
I forked your demo on delimiters, just so that we know for sure I'm using versions of jquery, velocity, velocity ui, and blast that do work in isolation.

Fixed. See new release.


GH-2055: Containers Must Implement DisposableBean

Resolves https://github.com/spring-projects/spring-kafka/issues/2055
If context initialization fails, Lifecycle.stop() is not called.
Containers must be stopped from DisposableBean in this case.
cherry-pick to 2.7.x, 2.6.x, 2.5.x

... and cherry-picked to all supported versions.


request help: The APISIX dashboard should have its own independent docker image

Issue description
First of all, the dashboard should not be deployed on the forwarding node, but to expose services through a route (ingress)
Secondly, Skywalking and Kong, etc., their dashboard images are independent.

Environment

apisix version (cmd: apisix version):
OS:

our contributor @bzp2010 helps us to add docker support for apisix-dashboard.


Move the Config portion of Inspect into libpod

While we're at it, rewrite how we populate it. There were several potential segfaults in the optional spec.Process block, and a few fields not being populated correctly versus 'docker inspect'.

LGTM assuming happy tests.  One open question for consideration.

LGTM

/hold
/lgtm

/hold cancel
Looks green

/retest


How to keep incomplete array of data from spanning through the whole chart?

I am having an issue with plotting my data on the LineChart. I have days of the month (0 -> 30) on the x axis, and it works great when data contains 30 elements. But when it contains 10 items, for example, the line doesn't stop at day 10, but spans even more. I want the line to go up to day 10. Is there any way to do this because this is very common for charts.

Hey did you find a work around for this?

Same.

I found a solution.
You can achieve this by adding one more full-size dataset whose opacity is 0 and whose dots are disabled.
But, you can't use shadow because the shadow of the incomplete line won't match the line self.


chore: Stop logging uncaught user generated error into sentry

Description

Sentry can capture and log unhandled errors from web workers. The global error handlers added earlier to prevent sentry from logging user exception doesn't prevent the default behaviour, thereby propagating the error further into sentry error handling logic. This PR adds preventDefault to the unhandled errors.

PR fixes following issue(s)
Fixes # (issue number)

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

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

/ok-to-test

/build-deploy-preview env=release

/ok-to-test sha=d94b06e


Logging with callstacks

I'm merging the modified branch instead of this one.
Thanks for this contribution. Great stuff!


Gracefully handle negative position in getWordAtOffset

I received reports of crashes due to a negative position in getWordAtOffset, and this prevents them.  It looks like the existing logic was meant to handle negative positions anyway, but the terms were reversed?
I was not able to actually reproduce the crash.  I can't figure out a way for getPositionForOffset to return offset: 0, affinity: upstream.
In order to test this without a real repro, I made getWordAtOffset @visibleForTesting.  Is there any other way to test it?

Looking at the SkParagraph implementation of getPositiomForOffset  it seems unlikely/impossible that it would ever return affinity: upstream. For reference https://github.com/google/skia/blob/e8432a50c55ef88e9ab9b2d04e0087d4dc4920c3/modules/skparagraph/src/ParagraphImpl.cpp#L838 . Not sure if there's another way to test this case.

@Renzo-Olivares I thought I could get it by using a grapheme cluster based on that code but I still never saw 0, upstream.  It seems like it's probably possible though.


Delayed updates

One of the sensor configuration steps is to enable delayed updates (if not done that by the installation parameter of the sensor install). I feel that that step is missing from this page. Could that be added (or a reference to a different page) to this document?

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

ID: 4bac5b74-85a1-6a19-3cf8-fc5d1899eaaf
Version Independent ID: 4bac5b74-85a1-6a19-3cf8-fc5d1899eaaf
Content: Configure Microsoft Defender for Identity sensor settings
Content Source: ATPDocs/configure-sensor-settings.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for the suggestion. I've added a note to the document explaining how to configure delayed updates.


Handle full hostname when computing host tag on GCE

The current code assumes the full domain name will not be included,
which is not always the case. This patch adds support for computing the
host tag from a fully qualified domain name.

ok to test

LGTM.
This highlights our need for consistency in our expectations about names, but this PR definitely doesn't hurt.


virtctl: add commands to dynamically set SSH keys and passwords on a VM

What this PR does / why we need it:
Added 3 commands to virtctl:

virtctl credentials add-ssh-key - Adds an SSH public key to one of the secrets that are specified in AccessCredentials field of a VM or VMI.
virtctl credentials remove-ssh-key - Removes an SSH key from one of the secretes.
virtctl credentials set-password - Sets a user's password in one of the secretes.

Release note:
Added "virtctl credentials" commands to dynamically change SSH keys in a VM, and to set user's password.

/cc @0xFelix @ksimon1

/retest

@akrejcir does the secret need already to exist before adding the ssh key? If so, shouldn't we create the secret together with the first key we want to add?

Good point. Yes the secret needs to exist. We cannot change running VM, but we can create the secret for a stopped VM.
/hold

What happens if the propagation method for the ssh key injection is cloud-init? Is this command not supported?

 I'd like to see it split up into smaller an easier to review PRs.
Added some comments.

IMO, if we try to reduce the code duplication into functions, the size of the pr should reasonably decrease

Added new functionality.
Can you take another look?

/hold cancel

/retest

@0xFelix @alicefr, Can you take another look and approve, if it's ok?

/retest

I've added usage examples.

/retest

/retest

/approve
Left a couple of questions and nit comments.
I'm also missing the answer to https://github.com/kubevirt/kubevirt/pull/9818#issuecomment-1567987453 . If not addresses could be done in a follow PR

/cherry-pick release-1.0


accessiblity map: We should be able to  upload a previously saved map to visualize it

When generating an accessibility map, we have the ability to save it to files.
We should be able to load previously generated map to visualize them.
@tahini We can also link this to some comparing functionality.

I'd like another term than "compare", but yeah, we can link this to "that which I don't have a word for yet"

For accessiblity map, a good term would be difference.

It's more that that, you only want to visualize a map previously saved. It's "analysis"? (I hate that term, in that context... everything is an analysis :p)

I was responding to your question about compare.
This issue is only to visualize


Added 'as Administrator'

...to PowerShell module section

Closing this PR as we have added the commit directly on PR #https://github.com/MicrosoftDocs/office-docs-powershell/pull/4468


Support string types (datetimes) in Trace x or y axis

currently they are restricted to one type of SomeNumber.  I hacked together a version that adds a second generic type to Trace, not restricted to SomeNumber, and use that for the x axis, and I was able to make a nice looking datetimes plot.  Here is what it looked like:
  Trace*[T: SomeNumber, R] = ref object
    xs*: seq[R]
    ys*: seq[T]
    xs_err*: ErrorBar[R]
    ys_err*: ErrorBar[T]
    marker*: Marker[T]
    text*: seq[string]
    opacity*: float
    mode*: PlotMode
    `type`*: PlotType
    fill*: PlotFill
    name*: string
    yaxis*: string

It makes the code a bit uglier and I had to modify the Plot type and functions as well. it Also doesn't allow for strings in the y-axis.

I tried to support this but there is a bug that limits having multiple generic types in some cases, e.g.: https://github.com/nim-lang/Nim/issues/7794
You can get around this by not setting xs and setting text instead. E.g. this works:
import plotly

var t = Trace[int](text: @["a", "b", "c"], ys: @[5, 10, 2],  `type`: PlotType.Bar)
Plot[int](traces: @[t]).show()

This is cheating since you may want the text to be a longer phrase for the hover.
Maybe your setup works because you don't have a type constraint on R which I guess is fine. I don't like having to specify, e.g. Plot[int,string](), but I guess that might be the best way forward. Thoughts?

I would also like this to be supported, in my case I want a times on the y axis and dates on the x axis, which I'm guessing is not possible right now.

anyone want to make a PR for this?

As a workaround, you can omit setting the xs property and it will display whatever the text property is

As a workaround, you can omit setting the xs property and it will display whatever the text property is


[App Check] Migrate AppAttestKeyIDStorage from UserDefaults to Keychain

Updated FIRAppAttestKeyIDStorage to use the Keychain as its underlying storage instead of NSUserDefaults. Since we use kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly in GULKeychainUtils, this means that the key IDs will not be migrated to new devices when restoring from backup.
Context: "The keys that you generate remain valid through regular app updates, but don’t survive app reinstallation, device migration, or restoration of a device from a backup." -- Start over on reinstallation
#no-changelog

After discussion, we decided to put on hold while investigating a more targeted approach based on a specific error message.

@andrewheard, is this PR related to this issue? https://github.com/firebase/firebase-ios-sdk/issues/11926

@andrewheard, is this PR related to this issue? #11926

Yes it is related and could solve it. However, I think the ideal approach would be to leave in the UserDefaults and like @paulb777 mentioned is target a specific error message to reset the key like my work around does.

@andrewheard, is this PR related to this issue? #11926

Yes it is related and could solve it. However, I think the ideal approach would be to leave in the UserDefaults and like @paulb777 mentioned is target a specific error message to reset the key like my work around does.

@umairalitail As @jostster said, this is related and could solve the issue in some scenarios (e.g., device migration) but it's unclear, at least to me, that it would resolve the issue in all scenarios (e.g., app reinstallation on same device). I'm holding off on this PR in favour of handling the specific DCError codes, e.g., DCErrorInvalidInput (code 2) and DCErrorInvalidKey (code 3), that we've been seeing in error reports.

@andrewheard, is this PR related to this issue? #11926

Yes it is related and could solve it. However, I think the ideal approach would be to leave in the UserDefaults and like @paulb777 mentioned is target a specific error message to reset the key like my work around does.

@umairalitail As @jostster said, this is related and could solve the issue in some scenarios (e.g., device migration) but it's unclear, at least to me, that it would resolve the issue in all scenarios (e.g., app reinstallation on same device). I'm holding off on this PR in favour of handling the specific DCError codes, e.g., DCErrorInvalidInput (code 2) and DCErrorInvalidKey (code 3), that we've been seeing in error reports.

Would it make sense to also make the resetAttestation public so developers could reset in other rare use case scenarios?


SessionCallback + 'static issue

Hello there 👋
I had an implementation issue with the 'static constraint for my callback :
https://github.com/dequbed/rsasl/compare/development...Mathieu-Lala:rsasl:fix/callback-static
What do you think about these changes ?
Is taking a Box<dyn SessionCallback> a problem ?

I don't think changing the config builder to take a Box<dyn SessionCallback> is a good call. I'd like to later on weaken the trait bounds to allow callbacks living shorter than 'static and taking a Box would make that change semver-breaking. It also adds an allocation in the IMO rather common case of the callback containing only some database handles and thus being Sized & 'static like so:
struct DbCallback {
    userdb: DB<User>,
    passdb: DB<Passwords>,
}

That being said, you can also implement SessionCallback on a Box<T> if you defined T, so
struct MyCallback { … }

impl rsasl::callback::SessionCallback for Box<MyCallback> {
    …
}

is possible.
I can see the merit of also implementing SessionCallback for Box<dyn SessionCallback> however, which would need to happen in rsasl itself. I'll add that to the next release, but if you need something like that working right now you can define a wrapper type:
#[repr(transparent)]
pub struct CallbackWrap(Box<dyn SessionCallback + Send + Sync>);
impl SessionCallback for CallbackWrap {
    fn callback(&self, session_data: &SessionData, context: &Context, request: &mut Request) -> Result<(), SessionError> {
        self.0.callback(session_data, context, request)
    }

    fn validate(&self, session_data: &SessionData, context: &Context, validate: &mut Validate<'_>) -> Result<(), ValidationError> {
        self.0.validate(session_data, context, validate)
    }
}

This will be as cheap at runtime as the proper solution and will make upgrading to the next rsasl release painless :)
Does that help you or is your case something I haven't convered here?

Thank you for your detailed response!
I will try this on my implementation and let you know if it fix my issue :+1:

Thank you :)
I'll close this issue then for now, please do re-open it if it turns out that my approach doesn't fix your problem ^^


Problem Working with DateTime in MySQL

For sunrise and sunset fields we need to save the data in DateTime format but we get an error when running and we have to change the data type field in the database to the string.

Issue solved in 2bc6e


Use multi-arch image of nginx, curl, alpine, busybox and k8s-example-web-app for tests

How to categorize this PR?

/area delivery
/kind enhancement
What this PR does / why we need it:
This PR updates the test images (except the redis image) to use multi-arch image.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/backlog/issues/19
Special notes for your reviewer:
Depends on https://github.wdf.sap.corp/kubernetes/guestbook/pull/3
Release note:

NONE

/assign

/milestone v1.52

/hold
I don't see the redis image adapted - ref https://github.com/gardener/gardener/blob/c840b5b50d6cad209d2f34a28b771bd514b93a4d/test/framework/applications/guestbooktest.go#L137-L138

I am also fine if you revert 70e87d96e9ed86bc6b7c0505fb55ec8a5ac50097 and follow-up with the redis image in another PR.

/test pull-gardener-e2e-kind

/test pull-gardener-e2e-kind

/test pull-gardener-e2e-kind


Packaged Typescript Definitions

Basically all that would be required in v3 is copying the definitions from here into the project. Going forward, it would mean that new releases of React Router (especially those with breaking changes to the API, such as with v4) would immediately have access to TypeScript typings instead of relying on the community to scramble to update the definitions.
Refer to this document for more information on publishing NPM packages with built-in TypeScript types.

I'm against this because of the version management. TS and Router have two different release cycles, so we might end up getting ahead or behind whatever the major version of TS is. If 3.0 just came out and we published a patch version of Router that used TS 3.0-only features, that would actually bump the version of the Router up to a major. It becomes kind of a mess really easily.
Also, no one on the maintainer list is a TS expert (AFAIK). So, we would have to rely on the community to help maintain them. And if that's the case, it's best left on DefinitivelyTyped as an external addon. The tooling handles this with the @types npm namespace pretty well as-is.

Agree w @timdorr. Unfortunately nobody who maintains react-router uses TS, so we'd always have to rely on someone to come in and update them for us. This is something that is better left to people who really care about TS.

@timdorr just to clarify, I'm only talking about including the definitions in the RR project, not porting the whole project over to TS - it's a little hard to tell which you're talking about.
In any case, thanks for the response.

To add to the above comment, type definitions help with code completion, type checking and other IDE hints. This is a proposal for ReactTraining to maintain the definitions instead of the community (package linked above).

To add further to what @GeKorm wrote, the TS definitions also help with vanilla JavaScript type hinting/"intellisense" in editors like VSCode.

@tills13 Yep, that's what I'm talking about too. We're on the same page.
The core thing is this has to be community maintained regardless. We don't know how to best build typedefs ourselves. Whether we set up a react-router-types subpackage or it lives on DT, we're not going to be the ones writing that code. And given its proliferation and ability to progress relative to TS, not Router's release schedule, DT would seem to be the best way to maintain those typings.

Sorry for this cross post from https://github.com/ReactTraining/history/pull/377#issuecomment-288392859. I think having type declarations would help the documentation. IMHO currently it is a little bit ambiguous, if some prop can be undefined or not.

Why separate the tightly coupled?
Whilst DT exists, a lot of pain would be spared if each library shipped with its own typings attached. See 'Including declarations in your npm package' in https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html
The problem with having @types/react-router is that the two projects has to be kept in sync, and there is no authority to approach when it comes to typings issue.
Getting hang of typings isn't quite rocket science, and most of it, if not all, can be automatically generated using tsc. Or, you can just copy what's on DT, and ping the OP when issues pop up here.

Hello everyone, just bumping this thread.
@timdorr I could be wrong, but I believe you can include the definition files without actually placing a dependency on typescript. Just having the index.d.ts on the top level of your package is enough for consumers reap the benefits. With this in mind, I'm a little confused about why you think the release schedule could cause trouble. This thread isn't about converting code to TS or adding dependencies, it's simply adding one file to the project, no more. It's affect on the library is exactly the same as adding additional documentation.

If 3.0 just came out and we published a patch version of Router that used TS 3.0-only features, that would actually bump the version of the Router up to a major.

Is this really the case? In definition files, you're not providing any implementations. No part of the react-router package needs to be in typescript. We're simply adding definitions to reflect the state of the javascript api. Unless typescript completely deprecates it's type system for something new, I don't think we have any worries in this department.
I understand the argument that no one on the core team is a personal user of TS, and therefore the typings are best left to the community because it probably means someone with a higher understanding of TS will write the files for you, which is beneficial for everyone.
With that said, downloading the typings separately from another package is the common route these days, but is that correct? This is only the case because of all of the existing libraries when typescript was released, but we're seeing more and more projects written in TS, and even seeing definition files exported by projects written in JS, for the consumer's convenience.
Personally, I feel that even more projects will pick this up, and it's only making our lives as developers easier.
Would you and the other core members consider another contributor who is a bit more TS savvy? This person could help with regular issues, but also be responsible for keeping the definitions in sync.
This would add the benefit of

official typings exported from the library (who doesn't love branding?)
typings that are reliably in sync with the api (less of these types of issues)
JS and TS users can reap the benefits. Using an editor like vscode or something else with the ternjs system will pick up these types and allow for Intellisense on regular JS files.

I really do think that this could be beneficial for both Typescript users and regular Javascript users alike.
Any thoughts from the team?


Dev Task: Beastiary feature

Jdungeons bestiary is an the a dictionary that the player can look up in order to access information about various creatures such as lore bits, weaknesses, attack skills and so on.
It the amount of information is dependent on the number of times the player has killed a specifik creature.

In rural districts they can be a real trouble if they get into farmers grain stores. This makes most farmers turn to cats, poison and traps for protection.”
30 kills unlock (Weakness - Though a rat doesn't have any so it is replaced with this flavor text):
“During your many encounters with Rats you have identified a weak point called the head. It’s located above the neck in the general region where it’s eyes and mouth are. This information has made your mercenary company more efficient at killing rats.”
100(or many) kills unlock:
“Legend has it that a powerfull rats resides in the mud flats east of the Merctown city.”
Reveals a location where the players can fight powerfull rats and their boss The Mad Ratter

Keep in mind that this is just a very generic example.
That way it'll work code-wise is that there's a dictionary in the playerdata that looks something like this:
var bestiary = {
	"Creature name": number of kills,

When the bestiary is opened the server (for) loops through the dictionary then tells the client to display the entries that has a value greater than 1. The bestiary entry also displays the number of kills and maybe some indicator as to when the dictionary entry is complete.
Each entry is a dictionary that looks something like this:
var rat_bestiary_entry = {
	1 : "This furry mammal squeaks a lot when poked with swords and pointy objects.",
        10 : “In cities, rats mostly reside in sewers where they live off edible leftovers from human waste as well as insects, worms and other tiny creatures. [Line break] In rural districts they can be a real trouble if they get into farmers grain stores. This makes most farmers turn to cats, poison and traps for protection.”

And so on.
Other scripts can be implemented in ordet to make the dictionary tie in with:

quests (like the mad ratcatcher in the example
Achievements (Ratsbane)
Buffs (+1 damage to rats)
Dialogue choices ("I've killed a lot of rats so i believe i know the reason for the grain shortage")

And so on..

I like the idea of getting a damage boost after killing enough of an enemy.
If you already fought plenty, you probably want the fight to end quicker.
We probably want to make this work as a component, so the way i'd do it is to give a Bestiary component to every creature.
The component only allows unlocking entries while used by a player, and it is limited to only containing entries when used by an enemy.
Now you just have to connect a "signal killed_creature( creature_class: String)" signal to the bestiary and let it run itself.
Unless we want to use Resources, we can use the translation system to keep this as lightweight as possible for the server. (The server doesn't have to do the TranslationServer look up, only the client does.)
It's a simple matter of giving each entry a unique name like "rat_10_kills_bestiary" and the translation system does the rest.


class_parameters doesn't populate on a data resource

I have been doing some testing and, specifically on the solidserver_ip_subnet and possibly others that use class_parameters, the data for the class_parameters does appear to be in the raw query data when the data resource is queried, but it doesn't seem to populate the class_parameters field.  I found a code snippet in the data_source_ip_subnet.go file that seems to "filter" the class params.
I don't really know what that field is meant to be, as there doesn't seem to be a good representation of what that is in the sparse solidserver rest documentation I could find.  Perhaps there is a better source for documentation on the rest API than the stuff in the gitlab python repo, but I haven't found it.
For the subnet, and I don't know if this is implementation specific or not, but on our setup, the class_parameters is where the gateway for the subnet is stored.  I would like to programatically pull that information, but at the present I cannot.
I was toying with an approach to this, but figured I would open an issue and see what @alexissavin's thoughts are.
My idea is this:
Add a field called "class_params_filter" to the data resource and then change the code where it filters out the class parameters to look something like this:
      retrievedClassParameters, _ := url.ParseQuery(buf[0]["subnet_class_parameters"].(string))
      computedClassParameters := map[string]string{}
      currentClassParameters := d.Get("class_params_filter").(map[string]interface{})
      if(len(currentClassParameters) == 0 ){
        currentClassParameters = d.Get("class_parameters").(map[string]interface{})
      } else {
        currentClassParameters = d.Get("class_params_filter").(map[string]interface{})
      }

      for ck := range currentClassParameters {
        if rv, rvExist := retrievedClassParameters[ck]; rvExist {
          computedClassParameters[ck] = rv[0]
        } else {
          computedClassParameters[ck] = ""
        }
      }

Preliminary testing shows this to work, but I don't know if this is the desired approach.

Hello,
Many thanks for reporting the issue and please forgive the latency in the answer.
Problem comes from the fact that only terraform declared class parameters are synchronized for resource (that is an implementation choice).
This filtering makes no sense for data sources. I have fixed in the previously reported commit.
Will be integrated within the next-release.


Add help to add annotation and network header modal dialog

:pushpin: References

Issue: Closes https://app.clickup.com/t/1xwfh93

:memo: Implementation
Added help icon with tooltip on hover in "add annotation" and "network" header modal dialog.
:art: Screenshots

Thanks for your review @tokland!


Minimize the use of session/local storage while creating board

In this project we are using local and session storage to get render the boards/widget etc. Is there any way to transfer control to developer?

feeding in the board's data structure programmatically as input is definitely doable. At some point I actually wrote the code to do that. All you need to do is create an @input property on the GridComponent and associate it with the model class variable. The value of the Input property can be derived from app/services/configuration-sample-boards.model.ts You can pass it in via the BoardComponent's view.html.

@jayhamilton - Thanks for your inputs.
I don't want to change in existing code so is there any other way to do this. something like extending the existing components and then write all code over there. Please let me know if you have any other different way.


Shims ICE when a scalar argument isn't actually a scalar

For instance, consider:
extern "C" {
    fn pipe(pipefd: [i32; 2]) -> i32;
}

fn main() {
    let mut fds: [i32; 2] = [0; 2];
    assert_eq!(unsafe { pipe(fds) }, 0);
}

I think to fix this properly we need to completely re-do the way we handle shim arguments: every OpTy needs to be transmuted to the right type before we do anything else with it. And we can't just just call OpTy::transmute as that can't be used for arbitrary transmute, e.g. changing ABIs can fail spectacularly.
Once this is consistently done everywhere, we can also remove the ScalarSizeMismatch error from the interpreter. This was added because otherwise ICEs can be triggered by using an incorrect scalar type in the signature of a function that has a Miri shim, but as we have seen, ICEs can still be triggered -- and once we use our own hard-coded types everywhere, ScalarSizeMismatch can no longer be triggered at all.
We probably want a macro that helps with that.

I think we want a new function like get_arg or so that preprocesses the OpTy. Given an op: OpTy and ty: Ty, we do a check like

let layout = layout_of(ty);
Check layout.abi.eq_up_to_validity(&op.layout.abi). If that is false, the argument type is invalid -> raise UB.
Return op.transmute(layout).

Similar ICEs can occur when the return type of the function is wrong -- we end up calling write_scalar with a scalar of the wrong size. This either fails to initialize a part of the result (if the scalar is smaller than the return place), or ICEs since we are exceeding the bounds of our AllocRefMut (if the scalar is bigger than the return place).


Feature Request: Copy and paste aura ring settings for token

Please add an option for aura rings set for token to be copied and pasted to another token.

This is available in 2.5.0


Bug: Uncaught TypeError: Cannot read property 'isSame' of null

Hey @clauderic
First of all, thanks so much for a great component. It's amazing. Not sure if this is a bug or intended but I get the following exception when rendering react-infinite-calendar with  a minimum date and null selectedDate I get the following error:
TypeError: Cannot read property 'isSame' of null
I've created a fork at: https://jsfiddle.net/abury/o80kxzha/
It's referring to line 119 of index.js. It looks like we just need to check for the existence of selectedDate in the preceding if. Let me know if its intended or not and I'll create a fork with a fix.
Thanks!
Aron

Hey @abury, thanks for the detailed report ❤
I tried replicating the issue you speak of, but to no avail. https://jsfiddle.net/o80kxzha/1/
I think perhaps you sent the link to the wrong fiddle? 😅
Either way, this is most certainly not intended behaviour, so absolutely a PR would be most welcome!

@clauderic hmm, it doesn't seem to be happening for me in the fiddle either (I'm new to the JS world so still learning how these things work, does JSFiddle show errors in a console anywhere?)
It's definitely occurring consistently in my project, I might spend some more time trying to look into it today. 👍

Hey @abury, FYI, I just shipped version 2.0.0, which replaces moment.js in favor of date-fns. This should no longer be an issue, though let me know if you run into any other issues


Support for 8.2 Simulator?

What about support for the 8.2 Simulator?
error: Device 'iPhone 6 (8.1 Simulator)' not found. Available device types: ["Resizable iPad (8.2 Simulator)", "Resizable iPhone (8.2 Simulator)", "iPad 2 (8.2 Simulator)", "iPad Air (8.2 Simulator)", "iPad Retina (8.2 Simulator)", "iPhone 4s (8.2 Simulator)", "iPhone 5 (8.2 Simulator)", "iPhone 5s (8.2 Simulator)", "iPhone 6 (8.2 Simulator)", "iPhone 6 Plus (8.2 Simulator)"]. Use --trace to view backtrace

Sorry, I haven't tried snapshot with t Beta release of Xcode yet

Okay. Should not be a big issue?! I'll wait for the update.

Perfect, glad to hear :+1:

As 8.2 is out now, it would be helpful to add this to the documentation of the snapfile.

I already prepared snapshot to automatically use the latest iOS version. In the mean time, please just insert the ios_version to your Snapfile.


Implement show_batch for unet

This PR implements show_batch for icevision.models.fastai.unet.
Screenshot

Colab notebook
https://colab.research.google.com/gist/dnth/88a76edb18ece126c36741ff0882c787/getting_started_semantic_segmentation.ipynb
Closes https://github.com/airctic/icevision/issues/1068

Thanks @dnth. That's a good idea!

Thanks a lot @dnth .
Are we ready to merge?

Yup! I'm good if there are no changes needed.

Merged!
Thanks a ton Dickson!


Unable to update "Card not match response" section in Default QnAMaker Dialog

Describe the bug
When we use QnAMaker Dialog in our dialog tree, The option "Card not match response" refuse tu applied any update from text input. We can only update it from code area. Also, it is not posssible to add adaptive card in this section (even if authorised it).
Version
v2.1.2
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[ ] Windows
[x] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to "one trigger and select a dialog"
Add QnAMaker Dialog in the tree and click on it to access properties
Scroll down to  "Card not match response"
Try update the card. The change will not applied in the tree

Expected behavior
We want to make sure that the update in  "Card not match response"  is applied synchroniously to the tree dialog. Also, we want to add an adaptative card to  "Card not match response"  (instead of raw text).
Screenshots

Additional context
We also seen some others problems of synchronisation between graphic input and raw json code.

hi @Levis0045

When we use QnAMaker Dialog in our dialog tree, The option "Card not match response" refuse tu applied any update from text input.  We can only update it from code area.

Are you able to use that response template (${QnAMakerDialog_CardNoMatchResponse_TcA5vF()}) in a simple Send a response action in that same dialog? I'm guessing there's a disconnect. You can also click on the Template Name link at the bottom of the response edit field to take you to that template (or use Bot Responses on left nav and go to that dialog). Hopefully it will show something there that may be a clue on what the issue might be.

Also, it is not posssible to add adaptive card in this section (even if authorised it).

I'm not sure on that part. Let me investigate and let you know what I find.

Also, it is not posssible to add adaptive card in this section (even if authorised it).

It is possible. I just tested it successfully.
Steps to accomplish for Card no match response:

Click Show response editor
Click + icon to use attachments
Click Add new attachment
Select Create from template
Select Adaptive card
Replace the adaptive card with your own. I used:

- ```{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2",
  "type": "AdaptiveCard",
  "body": [
    {
      "type": "TextBlock",
      "text": "Sorry, I did not find an answer.",
      "weight": "bolder",
      "isSubtle": false
    }
  ]
}```

Hi @dmvtech , thanks for your answer. But it was exactly the way that we have tried in composer:

Click Show response editor we have this warning message, and then we click Confirm

Click + icon to use attachments
Click Add new attachment
Select Create from template
Select Adaptive card
Replace the adaptive card with your example, however, we get still the same error: the answer is just a $.

Same result with your first answer, we have tried to change the answer template, but it doesn't work (message is $)

@tingting0118
Interesting.
Let me share mine so you can contrast it against yours.
Here is my QnAAction from my dialog
{
  "$kind": "Microsoft.QnAMakerDialog",
  "$designer": {
	"id": "NmXKQO"
  },
  "knowledgeBaseId": "d1d265d0-3689-4ba0-b906-78330ff246df",
  "endpointKey": "92af40bc-6dd1-43d5-9e2c-f2170d6ddac8",
  "hostname": "https://AAAAAAAAA.azurewebsites.net/qnamaker",
  "threshold": 0.3,
  "activeLearningCardTitle": "Did you mean:",
  "cardNoMatchText": "None of the above.",
  "top": 3,
  "isTest": false,
  "noAnswer": "${QnAMakerDialog_NoAnswer_NmXKQO()}",
  "cardNoMatchResponse": "${QnAMakerDialog_CardNoMatchResponse_NmXKQO()}"
}

Sharing my noAnswer only for sake of completion
# QnAMakerDialog_NoAnswer_NmXKQO()
- Didn't find anything!

Here is the template that cardNoMatchResponse uses for the activity to send. This has the attachment which calls another template (QnAMakerDialog_CardNoMatchResponse_NmXKQO_attachment_IyAtZj)
# QnAMakerDialog_CardNoMatchResponse_NmXKQO()
[Activity
    Attachments = ${json(QnAMakerDialog_CardNoMatchResponse_NmXKQO_attachment_IyAtZj())}
]

And this is the template that actually has the adaptive card
# QnAMakerDialog_CardNoMatchResponse_NmXKQO_attachment_IyAtZj()
- ```{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2",
  "type": "AdaptiveCard",
  "body": [
    {
      "type": "TextBlock",
      "text": "Sorry, I did not find an answer.",
      "weight": "bolder",
      "isSubtle": false
    }
  ]
}```

Please go over all these aspects in your dialog as well as bot responses and double check the syntax (all $,{,},(,), etc.). Additionally, make sure that the formatting of the card template is correct. It should start with:
-```{

And end with:
}```

Don't combine or split those lines
If you do not find any bugs/typos, then please try using my card above. I am curious if perhaps something might be happening due to the non-english-alphabet/accent characters in the . What locale is this bot running in?

@dmvtech Hi, thanks for this explication, but we don't find any bugs in out bot. We have tried with your example without change a character, but the answer of bot is still just a $. Our bot is in French, and we are in Paris, we have tested with English, French with accent characters, French without accent characters, but it doesn't work.
Is it possible to send your project with this example in zip to us?

Is it possible to send your project with this example in zip to us?

Absolutely. You can get it from a repo here:
https://github.com/dmvtech/CardNoMatchResponse/tree/master/CardNoMatchResponse
I am using the chit-chat content for QnA KB and here is what I get using the linked bot:

@Levis0045 Any luck using my repo?

Closing due to lack of reply.


docker打包报错：ERROR: failed to solve: process "/bin/sh -c pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple &&     pip install bitsandbytes --upgrade &&     pip install vllm==0.3.3 &&     pip install --no-cache-dir -r /workspace/requirements.txt &&     pip uninstall transformer-engine -y" did not complete successfully: exit code: 1

提交前必须检查以下项目 | The following items must be checked before submission

[X] 请确保使用的是仓库最新代码（git pull），一些问题已被解决和修复。 | Make sure you are using the latest code from the repository (git pull), some issues have already been addressed and fixed.
[X] 我已阅读项目文档和FAQ章节并且已在Issue中对问题进行了搜索，没有找到相似问题和解决方案 | I have searched the existing issues / discussions

问题类型 | Type of problem
安装环境 | Installation environment
操作系统 | Operating system
Linux
详细描述问题 | Detailed description of the problem
# 请在此处粘贴运行代码（如没有可删除该代码块）
# Paste the runtime code here (delete the code block if you don't have it)
docker build -f docker/Dockerfile.vllm -t llm-api:vllm .

docker打包报错
ERROR: failed to solve: process "/bin/sh -c pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple &&     pip install bitsandbytes --upgrade &&     pip install vllm==0.3.3 &&     pip install --no-cache-dir -r /workspace/requirements.txt &&     pip uninstall transformer-engine -y" did not complete successfully: exit code: 1
Dependencies
# 请在此处粘贴依赖情况
# Please paste the dependencies here

运行日志或截图 | Runtime logs or screenshots

我也遇到了，解决了吗

我也遇到了，解决了吗

没有啊，没人回复


Generic arg count as different type in inheritance

It seems like when you are passing on generic arguments in inheritance that they count as different types when the compiler tries to resolve a method call.
class Foo(A)
  def initialize(@value : A)
  end
end

class Bar(B) < Foo(B)
end

biz = Bar.new(25)

This will give me an error telling me that A and B are different types and nothing matches.
Error in line 9: no overload matches 'Bar(B).new' with type Int32
Overloads are:
 - Bar(B).new(value : A)

Though when I was using this I was expecting that the type that A refers to would be the same as B. I guess they are but it somehow thinks they are different?

FWIW, it seems to work fine if you use class Bar(A) < Foo(A)

Yeah noticed that, should have written that as well. Using that for now.
I reported it as I imagined this wasn't intended behaviour.


IllegalStateException during OnSaveInstanceState

About this issue
We are observing an IllegalStateException when OnSaveInstanceState is called.
Library version 10.4.0
Implementation:
class ViewLicensesActivity : AppCompatActivity() {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            MyTheme {
                Scaffold(
                    topBar = {
                        MyTopAppBar(
                            titleId = R.string.open_source_licenses_title,
                        )
                    },
                ) {
                    LibrariesContainer(
                        modifier = Modifier.fillMaxSize()
                    )
                }
            }
        }
    }
}

Steps to reproduce:

Turn on don't keep activities
Navigate to license activity
Click on an item
Background the app
Foreground the app

Details
[10.4.0] Used library version
[7.2.2] Used gradle build tools version
[1.7.10] Kotlin version
[1.2.1] Compose version
[1.3.0] Compose compiler version
[Chipmunk Patch 2] Android Studio version
Checklist

[x] Searched for similar issues
[x] Checked out the sample application
[x] Read the README
[x] Checked out the CHANGELOG
[X] Read the MIGRATION GUIDE

@danielbanks would you please be so kind to add the stacktrace.

@mikepenz edited to include stacktrace :)

OK thanks for the fix @mikepenz


Stop trying to connect if the test host exits unexpectedly

Taking into consideration host launch failures while waiting to setup connection, so we bail out sooner.
Related issue
#1689

Guys, can you shed the light on when this fix is going to make its appearance in SDK?
Fixed 4 months ago, supposedly merged 14 days ago, and yet .NETCore SDK v2.2.3 released like 7 days ago still does not have it.


Fix cs_CZ.yml BUTTON_REPLY_ALL typo

Issue #1650

Thanks, but you should use https://www.transifex.com/rainloop/rainloop-webmail/ to improve localization.


Stacks should have "stopped" state

When a stack is fully stopped by the user the stack shouldn't show up a degraded but instead should have a stopped status.
In my use case I have some stacks i'm still in the process of designing or aren't in use currently so are effectively turned off. However in my UI i see several degraded services, only 1 of which is actually having problems. This is kind of misleading and doesn't make it easy to spot actually degraded services.

Adding a new 2.0 UI.

Then, is this feature tracked somewhere else? I subscribe to GitHub issues so that I know when features are merged.


remove npx from package.json, add types for typescript

Rather removed npx 😁

Rather removed npx 😁

Hello guys, if it's not difficult for you to do it, please merge and publish, as in ide there is an error during import

Hello guys, if it's not difficult for you to do it, please merge and publish, as in ide there is an error during import

@Sitronik  published, sorry for late.

@Sitronik  published, sorry for late.


Mekanism Autocrafting

Issue description:
Autocrafting system no longer recognizes various Mekanism machines as the correct crafting item.
What happens:
Say I want to craft an elite level machine (ex: elite energized smelter). The RS system doesn't recognize previous tiers as the correct component, and will not craft. This was not an issue in previous RS versions.
What you expected to happen:
I expected the machine to craft without issues.
Steps to reproduce:

Tell crafter to create basic/advanced/elite level Mekanism machine
System will say previous tier item does not exist.

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>39
Refined Storage: 1.6

Does this issue occur on a server? [yes/no]
No.
If a (crash)log is relevant for this issue, link it here:
N/A

That is because the mekanism recipes are nbt sensitive. RS can't account for that.


dfuse patch on eosio 1.6.x dm cannot compile

https://github.com/dfuse-io/eos/tree/release/1.6.x-dm

eosio 1.6 is very old.  current is 2.0

@JasonPan2008 We officially support only 2.0.x currently. The old patches, even on 1.8.x do not include latest features and most probably event if they compile, wouldn't work.
From a 1.8 chain, you can definitely sync using 2.0 without requiring to upgrade the BPs. On 1.6, I assume it would work also when syncing with a 2.0 nodeos.
The patch is required only to extract information from the chain, so syncing on the same network with a 2.0 is a correct use case.
Tell us how it goes, don't hesitate to get in touch with our team over Telegram or Email if you have specific business requirements.
I'm closing this.


Incorrect node selection state

The node (action)'s selected state has a blue border, it should be the brand color instead per design.
AB#15994752

After discussing with Ricki, let's follow the latest Fluent guidance:

Closing due to original report being a non-issue


Add support for the workspace protocol

Affected Packages

Problem
Both pnpm and Yarn v2 support workspace ranges.

About workspace ranges in pnpm
About workspace ranges in Yarn v2

When I ran changeset status on the pnpm monorepo, I got lots of the following errors:
Package "@pnpm/tarball-fetcher" must depend on the current version of "@pnpm/error": "1.2.0" vs "workspace:1.2.0"

Workspace ranges are really useful in monorepos, so I think it is crucial to support them. And it shouldn't be hard to support. You could probably just strip the workspace: part. Only workspace:* is a special case. Before publish, it is replaced with the current version present in the monorepo.
Proposed solution

Are there any differences between pnpm and berry in this regard?

I don't think so. This was added by @arcanis to Yarn, I liked the idea and implemented the same in pnpm.

I'll take a look into adding support for this in the following days.

I need this now, so I'll submit a PR

@zkochan sure, that's cool - couldn't find enough time to get to it last week and was planning to address it this weekend. Thanks for taking care of this though! I really appreciate it.


Stable marriage problem implementation in Elvish

This is an algorithm I came up with so it needs to be checked.

[lang: elvish]


Mock bindings (or higher level API)

Currently feature/bindings provides the barebones for compiling the app code when in a bolos environment.
This is okay and more ergonomic bindings will be added as features are needed.
Here's the catch:
The bindings are, of course, invalid when there's no bolos environment or emulator to run the app, rendering unit tests impossible currently.
So this is the tracking issue to provide mocking of such bindings (or the ergonomic wrappers on top) so that unit tests will be possible once again
Current ideas:

provide a separate module in the app directly with all the mocked implementation and expose it thru the sys module, instead of bolos-sys and zemu-sys
provide said module in a crate instead for better reusability (and to not track all needed dependencies in the app manifest)

what is the status of this issue, from what I have seen I think this can be closed?? @becominginsane ?

Indeed


Publish 12/08/2021, 3:30 PM

Docs Build status updates of commit 84b5d29:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2016-ps/provisioning/Export-Trace.md
:white_check_mark:Succeeded

docset/winserver2019-ps/provisioning/Export-Trace.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

docset/winserver2022-ps/provisioning/Export-Trace.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


feat: add security headers

Description
Add missing security headers to UI related responses.
Fixes #297

Code LGTM
Do you think we can add the no cache header at the same time? It was mentioned in this discussion and seems related to the changes already in this PR.

@edlerd  Yes, I already added it to another PR in admin UI so it's quick and easy