Can't load HRNet Segmentation weights from PTH file

With get_segmentation_learner(architecture_name='hrnet', backbone_name='hrnet_w18').
Using the following callback to save the models during training:
SaveModelCallback(monitor='dice_multi', fname='best_model', with_opt=True)
The results of the predictions after loading "best_model.pth" with learner.load are zero-filled masks.
The prediction using the learner right after training are correct.

Hi, sorry, for super late reply. Where you using Windows?

No, we were using Google Collab Notebooks (Ubuntu
18.04.3 LTS 64-bit).

w32 & w48 not work, either.
PyTorch 1.12, Python 3.9 on Paperspace
learn = get_segmentation_learner(dls=dls, number_classes=2, segmentation_type="Semantic Segmentation",
                                 architecture_name="hrnet", backbone_name="hrnet_w32",
                                 splitter=segmentron_splitter,
                                 loss_func=CustomLoss(),
                                 metrics=[Dice, foreground_acc, JaccardCoeff],
                                 wd=1e-3).to_fp16()

RuntimeError                              Traceback (most recent call last)
File /usr/local/lib/python3.9/dist-packages/semtorch/models/archs/backbones/build.py:51, in load_backbone_pretrained(model, backbone)
     49     weights_path = download(model_urls[backbone], path=weights_path)
---> 51     msg = model.init_weights(pretrained=weights_path)
     52 else:

File /usr/local/lib/python3.9/dist-packages/semtorch/models/archs/backbones/hrnet.py:475, in HighResolutionNet.init_weights(self, pretrained)
    474     model_dict.update(pretrained_dict)
--> 475     self.load_state_dict(model_dict)
    476 return "HRNet backbone wieghts loaded"

File /usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py:1604, in Module.load_state_dict(self, state_dict, strict)
   1603 if len(error_msgs) > 0:
-> 1604     raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
   1605                        self.__class__.__name__, "\n\t".join(error_msgs)))
   1606 return _IncompatibleKeys(missing_keys, unexpected_keys)

complete error message.
hrnet-w32-error.txt

After deleting the previous cache file, the notebook can load the hrnet_w32 weights. It seems the PTH cache will always be ~/.cache/torch/checkpoints.


PredictableMemOpt: fix a crash, caused by a wrong instruction cast

Explanation:  Fixes a compiler crash in PredictableMemOpt caused by a wrong instruction cast. The fix is to check for the right instruction type and bail if the cast is failing.

Issue: rdar://125166206

Risk: Very low. The fix is a small change which checks for the right type to prevent the crash.

Reviewer: @meg-gupta

Main branch PR: https://github.com/apple/swift/pull/72490

@swift-ci test


Fix broken send_on_path_test

The function declaration of sort() is fn sort(&mut self) -> () and so these tests were simply asserting () == (). After correcting them they failed and so a change to paths_iter() is needed.

So, it turns out the paths_iter() condition was right, but the test was broken, which was hidden by the sort() bug you fixed. I pushed a commit to fix it, baiscally the idea is that the packets that are created in the test need to actually be sent to the server, so it can reply to the PATH_CHALLENGEs, which in turn makes the client update the paths' state correctly.

Anyway, good catch, and thanks for the fix!


Update add-your-organization-brand-to-encrypted-messages.md

updated page to indicate it's forced ome portal experience. @KCCross  please review

Already addressed.


updating to 0.3.9 on HA 2022.11

The problem
Hey,
I'm running HA Core 2022.11 and trying to update the integration but says this:

I guess I can manually update it, but not sure if that would create another issue
Related to this https://github.com/iMicknl/ha-nest-protect/issues/106
thanks!
What version of this integration (ha-nest-protect) has the issue?
0.3.9
What version of Home Assistant Core has the issue?
2022.11
Device / Model
Nest Protect Battery
Diagnostics information
No response
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

@iMicknl and @markvader thanks for fixing!


Make EncoderConfig non_exhaustive

This allows changing it in the future without performing an API breaking
change.

It is probably a good idea, but the current implementation in rust seems to trip in exactly the pattern that makes possible to write code w/out caring about additional fields...

Oh indeed, I've thought the ..Default::default pattern still works with non_exhaustive, but seems it's disallowed. Apparently so that you are allowed to add private fields in the future...
Should I close this?

At least for now, it seems that it has more cons than pros sadly =/

Ok then! Btw, I'm interested in this struct because of #151.


initial commit of Test-DbaServerName and Repair-DbaServerName

I tested replication (and only to the degree that I know how it works) but not mirror breakage.

I don't believe this supports named instances. Test may not either. It needs to. It can't change the \named  name, but can change the server name. This should be done before it's finished (and has to be done before my presentation in early oct. personal goal.)

Oh snap, Test-DbaServerName does support named instances! Cool, onto testing Repair-DbaServerName with an instance.

Repair support named instances, too. I'm dumm for thinking I was dumm. Testing now.

Making $rs null before checking for SSRS services fixed the issues I was having with servers showing they had SSRS when they didn't. Yay!


PHP versions greater than 8.0.* are unsupported

Bug report
When running the fixer with a PHP version greater than 8.0.* an exception is thrown with the following message:
PHP needs to be a minimum version of PHP 5.6.0 and maximum version of PHP 8.0.*.
Current PHP version: 8.1.3.
To ignore this requirement please set `PHP_CS_FIXER_IGNORE_ENV`.
If you use PHP version higher than supported, you may experience code modified in a wrong way.
Please report such cases at https://github.com/FriendsOfPHP/PHP-CS-Fixer .

I would expect any version greater than or equal to 7.4.* to be supported personally and since 8.0.* is supported, I think that the supported versions should actually be 7.4.* to 8.*.* since minor and patch versions shouldn't break the formatting rules currently in place.
Code snippet that reproduces the problem
Execute the fixer on any PHP version greater than 8.0.*.

Please, update your PHP CS Fixer version, in the latest release all 8 series version that are below 8.2.0 are supported: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v3.8.0/php-cs-fixer#L43

since minor and patch versions shouldn't break the formatting rules currently in place

I'm sorry, are you new to PHP or not aware how new features are introduced? All new minor releases in 7 and 8 series introduced  new operators, tokens or syntax changes - see dedicated tests for PHP versions here: https://github.com/FriendsOfPHP/PHP-CS-Fixer/tree/v3.8.0/tests/Fixtures/Integration/misc

it's very easy to fall into trap that PHP as a language follow SEMVER, especially for the interface of sth that is used only by very few projects like PHP tokenizer - no worry @jamesrweb , i fall into that trap myself one day ;)
Like Kuba said, just upgrade the Fixer and it will solve 8.1.x compatibility for you. But for each upcoming 8.x.0, we will need to put effort to make it compatible each time

@keradus @kubawerlos I have been using PHP for abut a decade now but there is a lot of inconsistency, yes. I was assuming since the PHP 7 move that SEMVER was to be respected but clearly not. Either way, thanks for the feedback and input 😄!

@jamesrweb if PHP follow SEMVER we'd have bigger version than in Java 😄


[Docs Request] add web demo

This will be useful in the future for bug hunters to be able to create simple reproducible examples of unexpected behavior in rustle.

That is something we should add, once the compiler is more complete. I have very little time these days to work on this project but this is a good idea, thank you!


[CSDiagnostics] Imporove requirement diagnostics originating in conte…

…xtual type
Detect that failed requirement comes from contextual type and use
that information to determine affected declaration.
Resolves: rdar://problem/47980354

@swift-ci please smoke test


Markdown Preview text color not controlled by styles.css

VS Code: 1.8.1
Why is it that the color for non-blocked text in the Markdown Preview can't be controlled by the styles.css?
This doesn't work.
body {
	color: #333;
}

The only way I can get the color to change is through changing my VS Code color theme from dark to light. Either VS Code's color theme or UI theme is preventing the Markdown Previewer from controlling that font color via styles.css.

You need to use a more specific css rule. The default text colors come from:
.vscode-light {
	color: rgb(30, 30, 30);
}

.vscode-dark {
	color: #DDD;
}

.vscode-high-contrast {
	color: white;
}

This allows customizing the style based on the current theme.
To use a custom color, you can either override one of these rules directly, or use a more specific rule that takes precedence over these rules. To inspect the dom and styling of the markdown preview, add the following keybinding
{"key": "cmd+alt+i", "command": "_webview.openDevTools"}

This will allow you to see where the style is coming from and how your rules are being applied.


Double Thumbnail loading (#1936) Fix

See Issue #1936 for more Infos
Brief description of the changes
Changes related to #1936
What browsers and operating systems have you tested these changes on?
Chrome on MacOS
Have you written unit tests? If not, explain why.
I have not written unit tests, because I dont have the complete overview over the code and do not have the time to working into it and setting up a proper environment to develop tests for this piece of code.
The fix I presented in #1936 is just a suggestion and is working in my situation. There should be someone other, who looks at it and test the change with a look at the full code.

Please explain your change in detail (the logic behind it, etc).

I searched a way to avoid duplicate adding Thumbnails to the thumbGenerationQueue.
I saw that the updateThumbnail is called two times:
One time in _addToList  and one time in _maybeUpdateThumbnail (part of qq.uiPrivateApi).
_addToList is called when _onSubmit runs and _maybeUpdateThumbnail is called when _markFileAsSuccessful runs.
At this point I dont have a deeper look how fineuploader is working. So I just compared the parameters given when updateThumbnail is called. I saw that the first time showWaitingImg is true and the second time undefined.
I also observed, that the second call of the update Function is not necessary to show the thumbnail image. So I just excluded the second push to the queue with:
if(showWaitingImg !== true) { return; }

May be not the best way to handle this problem. For me it was the best way to get this problem solved, without deep dive into the code and change major procedures.

Thanks for that. I’ll hopefully taje a closer look soon. The build will need to be fixed here as well, as one of the lint checks is failing

After looking at this a bit closer, I'm (first) not really sure I understand the problem. And second I don't think the changes here are appropriate at all. I will say the call to templating.updateThumbnail in the _maybeUpdateThumbnail method of uploader.api.js is incorrect in that it fails to pass a boolean for the third parameter (which is showWaitingImg). But it's not clear that this is related to the issue you're having.


Fix for KAFKA-7974: Avoid zombie AdminClient when node host isn't resolvable

When attempting to get topic list via KafkaAdminClient against a server that isn't resolvable, the worker thread can get killed as follows, leading to a zombie KafkaAdminClient:
ERROR [kafka-admin-client-thread | adminclient-1] 2019-02-18 01:00:45,597 KafkaThread.java:51 - Uncaught exception in thread 'kafka-admin-client-thread | adminclient-1':
java.lang.IllegalStateException: No entry found for connection 0
    at org.apache.kafka.clients.ClusterConnectionStates.nodeState(ClusterConnectionStates.java:330)
    at org.apache.kafka.clients.ClusterConnectionStates.disconnected(ClusterConnectionStates.java:134)
    at org.apache.kafka.clients.NetworkClient.initiateConnect(NetworkClient.java:921)
    at org.apache.kafka.clients.NetworkClient.ready(NetworkClient.java:287)
    at org.apache.kafka.clients.admin.KafkaAdminClient$AdminClientRunnable.sendEligibleCalls(KafkaAdminClient.java:898)
    at org.apache.kafka.clients.admin.KafkaAdminClient$AdminClientRunnable.run(KafkaAdminClient.java:1113)
    at java.lang.Thread.run(Thread.java:748)

It looks like cause is a bug in state handling between NetworkClient and ClusterConnectionStates:

NetworkClient.ready() invokes this.initiateConnect() as seen in the above stacktrace
NetworkClient.initiateConnect() invokes ClusterConnectionStates.connecting(), which internally invokes ClientUtils.resolve() to resolve the host when creating an entry for the connection.
If this host lookup fails, a UnknownHostException can be thrown back to NetworkClient.initiateConnect() and the connection entry is not created in ClusterConnectionStates. This exception doesn't currently get logged so this is a guess on my part.
NetworkClient.initiateConnect() catches the exception and attempts to call ClusterConnectionStates.disconnected(), which throws an IllegalStateException because no entry had yet been created due to the lookup failure.
This IllegalStateException ends up killing the worker thread and KafkaAdminClient gets stuck, never returning from listTopics().

This PR includes a unit test which reproduces the original issue (matching stacktrace) and verifies the fix.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Cc @cmccabe

Good find, @nickbp .  The fix seems a bit incomplete in the sense that there are more exceptions that we could get, besides UnknownHostException.  It seems like perhaps this line:
        } else {
            nodeState.put(id, new NodeConnectionState(ConnectionState.CONNECTING, now,
                this.reconnectBackoffInitMs, host, clientDnsLookup));
        }

Could catch the possible exceptions and then create a Disconnected connection, and rethrow, so that we didn't get more issues later on if something else throws here.
Or perhaps there is another way to make this more robust-- that was just the first that came to mind.

Good point. I was able to change the strategy of the fix to instead clean up when the host resolution actually occurs within ClusterConnectionStates.
Previously the resolve was happening automatically with the initial construction of the CONNECTING entry, leading to a missing entry. This fix switches to performing no resolution when the entry is first created, instead only performing it on-demand when currentAddress() is called. This allows NetworkClient to cleanly mark the entry as disconnected when resolution fails.

LGTM

retest this please

Looks like the build is green now, btw. Anything else needed before a merge?

@cmccabe looks like this is ready to be merged since you approved it and the build is green.

Thanks, @nickbp !

@cmccabe I cherry-picked this to the 2.1 and 2.2 branches as it seems like an important fix. Also cc @rajinisivaram who is familiar with this code.

@ijuma is this likely to go in to the 2.2 branch soon?

@manderson23 it's done now. I was running the tests locally before doing the push and had ran out of time.

I stumbled onto this issue a couple of days ago when trying to run an application in K8s. Any estimates for when a build will be ready with this fix?

I likewise originally encountered this in a K8s environment. The core issue leading to this bug arising was the result of DNS requests round-robining between CoreDNS instances which were individually eventually consistent but as a group not consistent at all, with entries popping in and out of existence as pods were brought up and some but not all CoreDNS instances knew about them at any given moment.
To avoid the inconsistency issues, I was able to use the following workarounds in my environment. With these workarounds a patch to the Kafka clients was not required to avoid the issue. YMMV:

kubectl edit service kube-dns --namespace=kube-system: Edit the sessionAffinity value to be ClientIP rather than None. This avoids rotating DNS lookups between CoreDNS instances, which can result in host records appearing to pop in and out of existence. Confirm the change with kubectl get service kube-dns --namespace=kube-system -o yaml
kubectl edit configmap coredns --namespace=kube-system: Edit the Corefile entry to have cache 30. By default this is 300. This ensures that DNS caches are updated every 30s rather than 5 minutes, which should result in pods deploying more quickly after having set sessionAffinity=ClientIP in the previous step. Confirm the change with kubectl get configmap coredns --namespace=kube-system -o yaml

@nickbp interesting...
I'll give that a go too. My workaround was to implement a simple, blocking, utility function that will attempt to resolve the host(s) for a finite duration n. If the host(s) cannot be resolved within the configured time, the application is terminated. It works for me at least.

Yeah the issue I was seeing was that after a utility check found the entry to be resolvable, it could still then become "unresolvable" due to dns lookup round-robining to a dns instance that didn't know about the host yet

Is this available in any release? I can't seem to find it...


Can't be compiled for > PHP 7 on Windows 7 x64

Hi
I am unable to compile it for:

PHP 7.4.3
PHP 7.2.28

I am getting this error message:
The system cannot find the file specified (php_protobuf.dll.res).
NMAKE : fatal error U1077: 'rc' : return code '0x1'
Stop.

I have encountered the same issue but i have fixes by remove the angle brackets < > in the CREDITS file at mentioned in https://phabricator.wikimedia.org/T208660
and this is the built dll file for php7.3 Thread Safe (TS) x64

@bcashier
I have encountered the same issue but i have fix it by remove the angle brackets < > in the CREDITS file at happens in https://phabricator.wikimedia.org/T208660
and this is the built dll file for php7.3 Thread Safe (TS) x64
For who want to know how to install the extension:

instruction: https://stackoverflow.com/a/58551223/8503381

Copy both php7.dll and php_protobuf.dll to System32.
Run a command prompt as Admin and navigate to C:\Windows\System32 and execute the command regsvr32 php_protobuf.dll then you will get a message that the DLL was loaded but entry point was not found.
Copy php_protobuf.dll to ext folder under the XAMPP php folder.
Also add extension=php_protobuf.dll to php.ini file.
Try to run php --ini if it works you will get no warning message.

hello Do you have  php_protobuf.dll

I have the same problem now，PHP 5.6 on Windows 10 x64  Do you know how to install it . think you

I have the same problem now，PHP 5.6 on Windows 10 x64 Do you know how to install it . think you

Yes i have, but it's for PHP 7.3 as in the link,
To install you can read the instruction that i have mentioned above.


ztunnel: drop default requests

CPU is more debatable, but the memory requests is egregiously large
based on current performance tests.

In response to a cherrypick label: new pull request created: #51145


documented warmup function does not work

Investigative information
The warmup function as described in the documentation generates an error.
Repro steps
Provide the steps required to reproduce the problem:
Use the code provided by the documentation:
function.json:
{
    "bindings": [
        {
            "type": "warmupTrigger",
            "direction": "in",
            "name": "warmupContext"
        }
    ]
}

init.py
import logging
import azure.functions as func

def main(warmupContext: func.Context) -> None:
    logging.info('Function App instance is warm 🌞🌞🌞')

Expected behavior
"function all instance is warm" to be logged.
Actual behavior
[04/09/2020 19:53:26] Worker failed to function id <GUID>.
[04/09/2020 19:53:26] Result: Failure
[04/09/2020 19:53:26]  INFO: Received FunctionLoadRequest, request ID: <GUID>, function ID: <GUID>
[04/09/2020 19:53:26] Exception: FunctionLoadError: cannot load the warmup function: type of warmupContext binding in function.json "warmupTrigger" does not
match its Python annotation "Context"
[04/09/2020 19:53:26] Stack:   File "/usr/lib/azure-functions-core-tools-3/workers/python/3.7/LINUX/X64/azure_functions_worker/dispatcher.py", line 245, in _handle__function_load_request
[04/09/2020 19:53:26]     function_id, func, func_request.metadata)
[04/09/2020 19:53:26]   File "/usr/lib/azure-functions-core-tools-3/workers/python/3.7/LINUX/X64/azure_functions_worker/functions.py", line 208, in add_function
[04/09/2020 19:53:26]     f'type of {param.name} binding in function.json '

Known workarounds
N/A

@bmc-msft can you please confirm that you are running in the Premium plan.

Yes, PremiumV2.

Hi @bmc-msft,
The PremiumV2 is a SKU of AppService plan.
The premium plan Anirudh indicated is Elastic Premium EP1/2/3.
https://docs.microsoft.com/en-us/azure/azure-functions/functions-premium-plan
I know this is a bit confusing

Can the documentation be cleaned up to discuss the ambiguity of two types of instances, both with the required "premium" identifier, but where only one is supported?

Yes we can. Do you have a suggestion where we can make this fix.

Perhaps the document I linked to in the initial issue comment?


Fixed use-select & use-autocomplete logic

Closes #2179
Description
Add if statements to use-select.ts and use-autocomplete.ts.
Is this a breaking change (Yes/No):
No

@zonoryo17
Thank you!

@Shunii85 @zonoryo17 @nakayan5
I have corrected it.Please review again.

@hirotomoyamada @Shunii85
I just realized, isn't there a missing test case to assert the initial state of the Select component when the defaultValue prop is set?


More C++11 makefile refactoring

Submission Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Declare copyright holder and open-source license: see below

Summary:
Changes needed to square these makefiles with the ones upstream in the Stan repo - soon these will be included by the Stan repo's makefile. Also changed a couple of things to look like the versions of these files present in the Stan repo that I think follows the intentions more closely.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Well, here's what I was thinking: the last setup didn't distinguish between flags for the C compiler vs C++ compiler, so functionally the equivalent ("send flags to both") is CPPFLAGS. I wasn't sure in these cases if we really did want that behavior or if they were for just C++ files (-> CXXFLAGS). So to preserve the old behavior in the new setup I changed it like so.
That said, if you know that these flags were just intended for the C++ compiler (i.e. our projects and not sundials or whatever else we build) then I can go ahead and change them to be CXXFLAGS?

I'm ok with either as long as we're consistent. It's slightly easier for me
to get my head around things with CXX just because it's what the make doc
uses and it's designed for C++, but there's value in self-consistency. It's
not like anyone else can use our makefiles.
In the last setup, it wasn't that we didn't distinguish flags for C vs C++.
It's that we actually did something very non-standard and used all the C
variables for building C++.
In my head, the libraries that need to be built with a true C compiler
should just be handled separately from the C++ build portion. A lot of the
complication in our old build was from us trying to do too much with make.
On Tue, Aug 15, 2017 at 11:24 PM, seantalts<EMAIL_ADDRESS>wrote:

Well, here's what I was thinking: the last setup didn't distinguish
between flags for the C compiler vs C++ compiler, so functionally the
equivalent ("send flags to both") is CPPFLAGS. I wasn't sure in these
cases if we really did want that behavior or if they were for just C++
files (-> CXXFLAGS). So to preserve the old behavior in the new setup I
changed it like so.
That said, if you know that these flags were just intended for the C++
compiler (i.e. our projects and not sundials or whatever else we build)
then I can go ahead and change them to be CXXFLAGS?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/602#issuecomment-322656878, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_FyEtmBJxaTpvN5rQ26RLqjHr4vaQks5sYmDrgaJpZM4O4KRL
.

Okay, I'll switch these all to CXXFLAGS because it sounds like they were all intended just for the C++ compiler. I'm not really sure what separating out the library compilation from make would look like, but I think we can leave that to a future pull request.


How to use extraEnv, extraVolumes and extraVolumeMounts

to add extra environment variables
    env:

{{- if .Values.extraEnv }}
{{ toYaml .Values.extraEnv | indent 8 }}
{{- end }}
to add extra volumes and mount paths.
    volumeMounts:

{{- if .Values.extraVolumeMounts }}
{{ toYaml .Values.extraVolumeMounts | indent 8 }}
{{- end }}
   volumes:

{{- if .Values.extraVolumes }}
{{ toYaml .Values.extraVolumes | indent 8 }}
{{- end }}
++++in values.yaml file, how to insert environment variables, volumes, mount paths.
extraEnv:

name: URL
value: "https://sample:80/admin"
extraVolumes:
name: ext-pvc
persistentVolumeClaim:
claimName: ext-pvc
extraVolumeMounts:
name: ext-pvc
mountPath: /opt/config
readOnly: true

Hi Team,
The above one is not an issue, it is just documentation/ usage snippet of extraEnv, extraVolumes and extraVolumeMounts, might help others.
Thanks,
Ananda Reddy C.

Sorry, I neither understand how to use the extraVolumes and extraVolumeMounts. In my case, I would like to use it with elasticsearch helm. I would like to provide a /backup mount point with VolumeClaim of my default storage class.


[Bug]: Cancelling the Geolocator.getPositionStream will make the app crash

Please check the following before submitting a new issue.

[X] I have searched the existing issues.
[X] I have carefully read the documentation and verified I have added the required platform specific configuration.

Please select affected platform(s)

[ ] Android
[X] iOS
[ ] Linux
[ ] macOS
[ ] Web
[ ] Windows

Steps to reproduce

Setup an app with geolocator, start listening to the getPositionStream and store the returned StreamSubscription to streamSubscriptionLivePosition.
After a while cancel the Subscription await streamSubscriptionLivePosition?.cancel(). At this point the app just gets completely unresponsive and hangs when in debugger. When in release mode, the app crashes.

Expected results
It should be possible to stop listening to position updates and start listening to it at another time.
Actual results
This is the crash report of firebase crashlytics:
Fatal Exception: NSInvalidArgumentException
-[GeolocationHandler stopListening]: unrecognized selector sent to instance 0x30332a1c0

          Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x827cc __exceptionPreprocess
1  libobjc.A.dylib                0x172e4 objc_exception_throw
2  CoreFoundation                 0x1888c8 +[NSObject(NSObject) _copyDescription]
3  CoreFoundation                 0x20b08 ___forwarding___
4  CoreFoundation                 0x20430 _CF_forwarding_prep_0
5  geolocator_apple               0x6cb0 (Missing UUID c0281c3bde253271948bb8b1d00890a9)
6  Flutter                        0x5d55e4 (Missing UUID 4c4c445055553144a1a5c934648e82e4)
7  Flutter                        0x5f2e8 (Missing UUID 4c4c445055553144a1a5c934648e82e4)
8  libdispatch.dylib              0x2370 _dispatch_call_block_and_release
9  libdispatch.dylib              0x40d0 _dispatch_client_callout
10 libdispatch.dylib              0x129e0 _dispatch_main_queue_drain
11 libdispatch.dylib              0x125fc _dispatch_main_queue_callback_4CF
12 CoreFoundation                 0x56204 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__
13 CoreFoundation                 0x53440 __CFRunLoopRun
14 CoreFoundation                 0x52830 CFRunLoopRunSpecific
15 GraphicsServices               0x11c4 GSEventRunModal
16 UIKitCore                      0x3d2eb0 -[UIApplication _run]
17 UIKitCore                      0x4815b4 UIApplicationMain
18 UIKitCore                      0x7bbfa8 block_destroy_helper.23
19 Runner                         0x474c main (AppDelegate.swift)
20 ???                            0x1b6dfeec8 (Missing)

If needed I can provide the full crash report.
Code sample
Code sample
StreamSubscription streamSubscriptionLivePosition = Geolocator.getPositionStream(locationSettings: const LocationSettings(accuracy: LocationAccuracy.best)).listen((position) {
      debugPrint(position.toString());
  }

await streamSubscriptionLivePosition?.cancel();

Screenshots or video

Screenshots or video demonstration
[Upload media here]

Version
13.0.2
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.5, on macOS 15.1 24B83 darwin-arm64, locale en-US)
    • Flutter version 3.24.5 on channel stable at /Users/chris/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (27 hours ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/chris/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.16.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)

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

[✓] Connected device (5 available)
    • Chris’s iPhone (mobile)         • 00008130-000970383EE0001C            • ios            • iOS 18.1 22B83
    • iPhone 16 Pro Max (mobile)      • 3EC01824-C903-460F-AB65-B58F2D386A96 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-18-1 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.1 24B83 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.1 24B83 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 130.0.6723.117
    ! Error: Browsing on the local area network for Chris’s Apple Watch. Ensure the device is unlocked and discoverable via Bluetooth. (code -27)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @justChris,
We briefly had introduced a bug in the goelocator_apple package yesterday (version 2.3.8). This bug resulted in the error you listed.
This bug was fixed within a few hours in version 2.3.8+1. Please verify in your pubspec.lock file if the correct version is used by the app.
Apologies for the inconvenience.

Hi @mvanbeusekom
Thank you for your quick reply!
I am indeed using geolocator_apple: 2.3.8 instead of 2.3.8+1 and there is a transitive update available for geolocator_apple when I run dart pub outdated.
But your last update did not update this dependency. It seems like something is holding it back from updating. Even if I run dart pub upgrade this dependency will not be updated.
Do you need to do anything on your side?

I need to correct myself. I have just run dart pub upgrade again and not it has been updated.
Thank you very much for your fast and professional support!


python310Packages.geopandas: 0.11.0 -> 0.11.1

Description of changes
https://github.com/geopandas/geopandas/releases/tag/v0.11.1

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages failed to build:

python310Packages.rlax
python39Packages.rlax

12 packages built:

python310Packages.asf-search
python310Packages.bsuite
python310Packages.geopandas
python310Packages.osmnx
python310Packages.plotnine
python310Packages.wktutils
python39Packages.asf-search
python39Packages.bsuite
python39Packages.geopandas
python39Packages.osmnx
python39Packages.plotnine
python39Packages.wktutils

rlax is broken already.


make 'TenderlySolcConfig' depend on 'SolcConfig', then modify the 'libraries' type

Current dependencies on/for this PR:

master

PR #84   👈

PR #85

This comment was auto-generated by Graphite.


Decoupling: Hide 'copy story url' from dashboard menu when url is undefined

Context
Hide “Copy story URL” and "Open in new Tab" links when URL is undefined. Also because they look redundant in storybook
Summary

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Partially addresses https://github.com/google/web-stories-wp/issues/2918

The one thing that bugs me here is that it's very unintuitive, because of the way menuItems and menuItemActions are constructed in the dashboard.
I don't see why these are two separate things.
As a dev I'd have expected action to be part of the menuItem, e.g. something more like menuItems = [ { label: '...', value: '...', capability: '', action: '...' } ] , that would make the filtering easier: if there's no action, just remove it.
We could even turn capability into a more general purpose callback that is called in the menuItemsFiltered filter.
Maybe something like this:
export const STORY_CONTEXT_MENU_ITEMS = [
  {
    label: __('Open in editor', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.OPEN_IN_EDITOR,
    isEnabled: (story) => Boolean(story?.capabilities.hasEditAction),
  },
  {
    label: __('Open in new tab', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.OPEN_STORY_LINK,
    isEnabled: (story) => Boolean(story.previewLink),
  },
  {
    label: __('Copy Story URL', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.COPY_STORY_LINK,
    isEnabled: (story) => Boolean(story.previewLink),
  },
  {
    label: __('Rename', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.RENAME,
    separator: 'top',
    isEnabled: (story) => Boolean(story?.capabilities.hasEditAction),
  },
  {
    label: __('Duplicate', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.DUPLICATE,
    isEnabled: (story) => Boolean(story?.capabilities.hasEditAction),
  },
  {
    label: __('Delete Story', 'web-stories'),
    value: STORY_CONTEXT_MENU_ACTIONS.DELETE,
    separator: 'top',
    isEnabled: (story) => Boolean(story?.capabilities.hasDeleteAction),
  },
];

// And then in generateStoryMenu():

const menuItemsFiltered = menuItems.filter(
  (item) => typeof item.isEnabled === 'undefined' || item.isEnabled(story)
);

@samwhale refactored the context menu, let's pause on this until that can get merged: https://github.com/google/web-stories-wp/pull/9960/files

@samwhale refactored the context menu, let's pause on this until that can get merged: https://github.com/google/web-stories-wp/pull/9960/files

That PR got merged yesterday into main.

Am I correct in this being to resolve storybook showing these redundant options like so?

@BrittanyIRL Yes, thanks for testing.

Some failures all over the place now 👀

Seeing this error in the console when visiting the dashboard

But couldn't test the main idea: How do you set a story to have an undefined url?

npm run storybook and open the dashboard there

Tested in storybook and looks good. Thanks @swissspidy


Create a set of datasets with multiple object and scenes for training and testing

SimulatorDataManager class allows you to filter objects and save them into a pkl.


add mobilenetv2 as a new task

By @martinjaggi from FeAI
can use a standard public TF.js model for a first demo, and we can use our existing image preprocessing pipelines with it (or add a new one, whichever is easier)

Meant for privately.


[ubuntu] Refactor Software Report helpers

Description
This PR:

Removes Restore-UserOwner function and invokes command directly
Renames Take-OutputPart function to Get-StringPart with approved verb
Removes unused Get-AptSourceRepository function
Combines Get-PathWithLink and Get-LinkTarget to a single function for simplicity
Contains minor style changes

Related issue: https://github.com/actions/runner-images-internal/issues/5606
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu2004,ubuntu2204


Show timer status in status panel

The status panel only shows that the timer mode is active, it doesn't show the actual time or count direction. Something like that would be helpful, especially for debugging if people are having trouble.

done! The status panel now mirrors the current expected output and state. It's already helped with debugging!


internal: add logging library for more consistent logging output

Description
Add logging library to be used to centralize log output and levels in logging. No updates to any other files or main have happened to configure or use this. If this is done this PR will be more than one file as all instances of the console call will be replaced.
Motivation and Context
Logging output is inconsistent and there is no central way to change the logging level.
How has this been tested?
Heavily used in SQL fork.
Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

By creating a Pull Request you agree to our Code of Conduct. For further guidance on contributing please see contributing guide

In the name of the PR, it says 'more consistent logging output' - what does that mean? Consistent with what, please?

Consistent in formatting of the output and the way levels are handled, instead of a hard coded trace/debug/etc console call it can be configured up and down.
Also if anything needed to be changed at a later date it is one place now instead of every console.* having to be updated.

Awesome, I am excited about more configurability for logging! I am on mobile, so I have not looked at the code yet, but if I wanted to make a follow-up PR that changed the recent perf-related debug logging to use this logging system, is there an example in here of how I would do that? Thanks!

Thanks, Anna. I also wondered what the code for both setting up logging and using it would look like. That's the kind of thing that is worthwhile having in code docs.
Also Anna FYI, from memory, sytone previously added quite a lot of logging calls to his fork, and so I imagine that he will want to merge some or all of that back in to this project, for code that already exists here.

Summary
This is all good. I will then:

[ ] merge it
[ ] fix the plugin name (in 2 places)
[ ] add code to main.ts to hook it up
[ ] and then add what I learned to the docs.

Trying it out
I had a go at using the logging code, by copying in code from tasks-x.
Here is what I added:
diff --git a/src/main.ts b/src/main.ts
index 2df6080..ac39f9f 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -10,6 +10,7 @@ import { QueryRenderer } from './QueryRenderer';
 import { getSettings, updateSettings } from './config/Settings';
 import { SettingsTab } from './config/SettingsTab';
 import { EditorSuggestor } from './Suggestor/EditorSuggestorPopup';
+import { log, logging } from './lib/logging';

 export default class TasksPlugin extends Plugin {
     private cache: Cache | undefined;
@@ -17,7 +18,20 @@ export default class TasksPlugin extends Plugin {
     public queryRenderer: QueryRenderer | undefined;

     async onload() {
-        console.log('loading plugin "tasks"');
+        logging
+            .configure({
+                minLevels: {
+                    '': 'debug',
+                    taskssql: 'debug',
+                    'taskssql.perf': 'debug',
+                    'taskssql.QuerySql.QuerySql': 'debug',
+                },
+            })
+            .registerConsoleLogger();
+        log(
+            'info',
+            `loading plugin "${this.manifest.name}" v${this.manifest.version}`,
+        );

         await this.loadSettings();
         this.addSettingTab(new SettingsTab({ plugin: this }));
@@ -44,7 +58,10 @@ export default class TasksPlugin extends Plugin {
     }

     onunload() {
-        console.log('unloading plugin "tasks"');
+        log(
+            'info',
+            `unloading plugin "${this.manifest.name}" v${this.manifest.version}`,
+        );
         this.cache?.unload();
     }

Firstly, it's really nice to know how to access the plugin version number - thank you.
This was the output:
[20220803104521][info][taskssql] loading plugin "Tasks" v1.11.0

With all the work being done on performance improvements in Tasks, over the next few releases, I'm going to be having users look at the console a lot.
I can see how to fix the [taskssql] bit, and seeing the code, it's clear what is going on.
Controlling log levels
I added a line of debug output:
        log('debug', 'wibble - debug');

It was displayed in the console.
Then I changed the initialisation code like this:
        logging
            .configure({
                minLevels: {
-                     '': 'debug',
+                     '': 'info',
                    taskssql: 'debug',
                    'taskssql.perf': 'debug',
                    'taskssql.QuerySql.QuerySql': 'debug',
                },
            })
            .registerConsoleLogger();

And then the debug log message was not written out.

I was going to wire in but did not want to add multiple changes to this PR.

Hi @sytone, many thanks for this - now merged.
I won't have time to do anything with this for a few days, so if you were able to hook this in to the existing code now, then thank you.


rename rid to resource in method arguments

I've talked to a few people confused by "rid" in the method args-- they don't expect to be able to pass in an alias reference (e.g. {"alias": ""}).

released on 0.13.4


python312Packages.homematicip: 1.1.3 -> 1.1.4

Diff: https://github.com/hahn-th/homematicip-rest-api/compare/refs/tags/1.1.3...1.1.4
Changelog: https://github.com/hahn-th/homematicip-rest-api/releases/tag/1.1.4

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362729

x86_64-linux

:white_check_mark: 5 packages built:

home-assistant-component-tests.homematicip_cloud
python311Packages.homematicip
python311Packages.homematicip.dist
python312Packages.homematicip
python312Packages.homematicip.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 362729

x86_64-linux

:white_check_mark: 5 packages built:

home-assistant-component-tests.homematicip_cloud
python311Packages.homematicip
python311Packages.homematicip.dist
python312Packages.homematicip
python312Packages.homematicip.dist


GUI improvements

@bls337 I thought it would helpful to keep track of GUI-realted things I run into while testing:

[ ] On the data panel, both the "save images" checkbox and datastore selection (which includes RAM) are present. To me this seems redundant since RAM datastore would be not saving. Or is not saving images related to some kind of hardware test?
[x] It would be very helpful for testing (and also eventually for users) if the GUI remembered settings between different runs. In particular remembering the saving folder and name would save me a lot of clicking.
[x] The add and remove channels I assume are supposed to have icons or labels but they are not big enough to display on my screen:

[ ] How should the acq eng callback to the GUI to let it know an acq is finished? Right now the stop acqusition never returns to start acquisition

I fixed the add and remove channels buttons, they are readable now.

The acquisition settings object is saved when the plugin closes, and when the plugin opens the acquisition settings object is loaded and the gui is populated with it's values. Most of the gui controls are being initialized from the acquisition settings, but not all of them are. In the case of the data tab, the gui wasn't being initialized with the acquisition settings and the save path on the Datastore itself was not set correctly. I changed the createDatastore method to take the path as a string, it's now being created at the beginning of the acquisition.
Do we need to close the Datastore at the end of an acquisition? It would be nice to incorporate the freeze method to prevent the Datastore from being modified after an acquisition as well.

I removed the RAM datastore option from the radio button. I don't think there's any reason to have it, it was just left over from throwing things together. I haven't connected that part of the gui to the Datastore yet, it seems like users would want to be able to change between SINGLEPLANE and MULTIPAGE TIFF formats?

I will think about a good solution for the acquisition engine to communicate with the user interface. Maybe having the acquisition engine emit a signal that gui can register to listen to, so the observer pattern, or Google event bus would work.

The acquisition settings object is saved when the plugin closes, and when the plugin opens the acquisition settings object is loaded and the gui is populated with it's values.

Okay good to know. It might worthwhile to have the acquisitions settings save whenever they are changed in addition to when the plugin closes. This would, for example, mean settings aren't lost in case of a crash.

Do we need to close the Datastore at the end of an acquisition? It would be nice to incorporate the freeze method to prevent the Datastore from being modified after an acquisition as well.

Right now its not using the Datastore interface, but rather going into NDTiff directly through its API, but I plan to add datastores shortly and this functionality as well.

seems like users would want to be able to change between SINGLEPLANE and MULTIPAGE TIFF formats?

Were single plane images used in the diSPIM plugin? I know that writing code is much slower than the multipage tiffs, so it may not be worth it to include it

I will think about a good solution for the acquisition engine to communicate with the user interface. Maybe having the acquisition engine emit a signal that gui can register to listen to, so the observer pattern, or Google event bus would work.

Yes, I think that sounds good. The event bus should work, though I have found that code that uses it can be hard to debug. If its just a one or two functions, you might just consider adding API methods to the GUI directly


Optimize Neuron#activate method

Avoid creating an intermediate array.
Running a benchmark shows this optimization converts into approximatelly 10% in performance gain on final Network#run method.
$ crystal run --release bench.cr
original   1.86M (536.63ns) (± 4.24%)  1.13× slower
modified   2.11M (474.37ns) (± 4.32%)       fastest

Very nice! Thanks again @hugoabonizio :)


Upgrade JSON-Smart dependency

Issue
 Multiple dex files define Lorg/objectweb/asm/AnnotationVisitor; error in Android dex compilation phase. When using JSON-Path with Mockito.
Probable cause
The JSON-Smart dependency packages asm classes under default namespace.
See Issues:

https://code.google.com/p/json-smart/issues/detail?id=52 ( see last comment from project maintainer )
https://code.google.com/p/json-smart/issues/detail?id=51

Can the JSON-Smart dependency version be upgraded to latest release ?

We are running 2.1.1. This is the latest release http://search.maven.org/#search|ga|1|json-smart
/Kalle

Has anyone resolved this?

Should be resolved in json-smart 2.2 (see #108)

We're at 2.2 now.

resolved in #108


x64: Add support for some BMI2 instructions

This PR adds support for some isntructions in the BMI2 instruction set, namely sarx, shlx, shrx, and rorx. The shifting instructions can't encode immediates but use a 3-operand form and place no constraints on the amount being shifted by (as opposed to sar-style instructions which require that in %cl). The x-suffixed instructions are prioritized for this reason when BMI2 is enabled so long as an immediate shift amount isn't used. These new instrutions additionally support loading the value-to-shift from memory, unlike the old instructions (they store the result to memory as opposed to the new instructions which store the result in a register), but this functionality is not exposed at this time.
For rorx the benefits are similar, but the amount being shifted by is required to be an immediate. This means that rotation instructions, either left or right, will unconditionally use rorx so long as the amount-to-rotate is an immediate. This theoretically reduces register allocator pressure since the destination register can be different from the source.
I'll note that I don't personally have a compelling performance reason for adding these. I was poking around at instruction set extensions awhile back and never got around to making a PR for this until just now.

I've additionally pushed a commit for bzhi as well which is more along the lines of https://github.com/bytecodealliance/wasmtime/pull/6484 where it can cut down on a few emitted instructions in niche situations.


MacOS: focusing status/command window interrupts state

Not sure if there's a way around this as it's standard Reaper behavior(maybe??). But just in case.
Interruption happens in all modes. ESC will close the focused command window and clear the RK state. You can only get back to previous state by clicking "Clear" in the command window or somewhere that won't affect where you left off(transport and tool bar area). Very few exceptions exist. So far I've only found that in Visual Mode h and l and their variants continue to work with varying degrees of consistency.
EDIT: should say that ESC on focused RK window closes the window and suspends state, not clear it. If you continue what you were doing after focusing the rk window and escaping or clicking out of focus, state resumes as it was.

If you click on the main window and leave the RK window unselected, does it still interrupt?

@gwatcha hi! Assuming that you mean clicking the main after having focused rk window? In this case clicking focus back to the main window allows state to continue where it left off. I edited my last comment because escaping rk window from focus also returns you to the state you had before interruption. So I suppose the issue becomes a question of can state continue independent of whether the rk window is focused or unfocused?

So I suppose the issue becomes a question of can state continue independent of rk window focus state?

Unfortunately no, when the message window is focused, no key presses are sent to the scripts.
The intended way to use the window is to leave it on the side and have the main window focused. If you want to get rid of it though, there's an option that will be released soon called 'log_level' that can turn off the messages.

FocusMain is used with FX plugins

t


[BUG] Prediction with DatetimeIndex as ForecastingHorizon values

Describe the bug
There is an issue with DatetimeIndex in ForecastingHorizon when doing prediction. Seems this came up by recent commits?
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-5-0c038f527191> in <module>
      9 forecaster = NaiveForecaster(strategy="last", sp=12)
     10 forecaster.fit(y_train)
---> 11 y_pred = forecaster.predict(fh)
     12 y_pred

~\Desktop\sktime\sktime\sktime\forecasting\base\_sktime.py in predict(self, fh, X, return_pred_int, alpha)
    216         self.check_is_fitted()
    217         self._set_fh(fh)
--> 218         return self._predict(self.fh, X, return_pred_int=return_pred_int, alpha=alpha)
    219
    220     def compute_pred_int(self, y_pred, alpha=DEFAULT_ALPHA):

~\Desktop\sktime\sktime\sktime\forecasting\base\_sktime.py in _predict(self, fh, X, return_pred_int, alpha)
    616
    617         # all values are out-of-sample
--> 618         if fh.is_all_out_of_sample(self.cutoff):
    619             return self._predict_fixed_cutoff(
    620                 fh.to_out_of_sample(self.cutoff), **kwargs

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in is_all_out_of_sample(self, cutoff)
    346             cutoff.
    347         """
--> 348         return sum(self._is_out_of_sample(cutoff)) == len(self)
    349
    350     def to_indexer(self, cutoff=None, from_cutoff=True):

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in _is_out_of_sample(self, cutoff)
    328         """Get index location of out-of-sample values"""
    329         # return ~self._in_sample_idx(cutoff)
--> 330         return self.to_relative(cutoff).to_pandas() > 0
    331
    332     def is_all_out_of_sample(self, cutoff=None):

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in to_relative(self, cutoff)
    223                 values = _coerce_duration_to_int(values, unit=_get_unit(cutoff))
    224
--> 225             return self._new(values, is_relative=True)
    226
    227     @lru_cache(typed=True)

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in _new(self, values, is_relative)
    160         if is_relative is None:
    161             is_relative = self.is_relative
--> 162         return type(self)(values, is_relative)
    163
    164     @property

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in __init__(self, values, is_relative)
    137         if not isinstance(is_relative, bool):
    138             raise TypeError("`is_relative` must be a boolean")
--> 139         values = _check_values(values)
    140
    141         # check types, note that isinstance() does not work here because index

~\Desktop\sktime\sktime\sktime\forecasting\base\_fh.py in _check_values(values)
    105     # check values does not contain duplicates
    106     if len(values) != values.nunique():
--> 107         raise ValueError("`values` must not contain duplicates.")
    108
    109     # return sorted values

ValueError: `values` must not contain duplicates.

To Reproduce
from sktime.forecasting.all import *

y = load_airline()
y.index = y.index.to_timestamp()
y_train, y_test = temporal_train_test_split(y, test_size=44)

fh = ForecastingHorizon(y_test.index[0:5], is_relative=False)

forecaster = NaiveForecaster(strategy="last", sp=12)
forecaster.fit(y_train)
y_pred = forecaster.predict(fh)
y_pred

Versions
Current master

That seems related to #534, no? Actually, when I run this it, it works? But the predict call is extremely slow, very strange. Still have to look into this.

Sry, I was not sure if this is the same issue. Btw I tried this also on my Linux Azure VM and I got a different error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/aiwalter/notebooks/sktime/sktime/sktime/forecasting/base/_sktime.py", line 218, in predict
    return self._predict(self.fh, X, return_pred_int=return_pred_int, alpha=alpha)
  File "/home/aiwalter/notebooks/sktime/sktime/sktime/forecasting/base/_sktime.py", line 620, in _predict
    fh.to_out_of_sample(self.cutoff), **kwargs
  File "/home/aiwalter/notebooks/sktime/sktime/sktime/forecasting/base/_sktime.py", line 654, in _predict_fixed_cutoff
    fh, X, return_pred_int=return_pred_int, alpha=alpha
  File "/home/aiwalter/notebooks/sktime/sktime/sktime/forecasting/naive.py", line 166, in _predict_last_window
    last_window = np.tile(last_window, reps=reps)
  File "<__array_function__ internals>", line 6, in tile
  File "/anaconda/envs/py37_default/lib/python3.7/site-packages/numpy/lib/shape_base.py", line 1258, in tile
    c = c.reshape(-1, n).repeat(nrep, 0)
MemoryError: Unable to allocate 98.5 GiB for an array with shape<PHONE_NUMBER>, 12) and data type float64

Not sure what it tried to allocate here

Yes I think this is due to what I describe in #543 where it takes "MS" months start as "ms" for milliseconds. That explains why it becomes so slow in my case too. I'll close this one in favour of the other one.

okay thx for having a look into it!


v3 proposal

It extends #24 and add php 7.1 support
ping @weierophinney @Maks3w @bakura10

Originally posted by @snapshotpl at https://github.com/zendframework/zend-validator/pull/177

@snapshotpl
I see some problems, because you've mixed different problems in one commit:

this breaks the version control system
no way for a cherry-pick
reading the commit history is a mess
a review is very difficult

Each individual change should be one commit. Please create separate commits for each problem. Thank you in advance!

Originally posted by @froschdesign at https://github.com/zendframework/zend-validator/pull/177#issuecomment-316069729

@snapshotpl Totally missed this when I submitted #181 earlier.
@froschdesign For large refactors, we simply cannot do one change at a time. In such cases, I would recommend posting a WIP with some of the ideas codified, and then creating an RFC in the contributors section of the forums. I'll be posting one related to #181 tomorrow, as I've done some significant work trying to address forwards and backwards compatibility at this point.

Originally posted by @weierophinney at https://github.com/zendframework/zend-validator/pull/177#issuecomment-316213664

@weierophinney

For large refactors, we simply cannot do one change at a time.

You can, because a problem can change more than one line or one file!

In this PR, there's a single commit that does two different things:

Updates travis.yml and composer.json to PHP 7.1
Adds Result.php, Message.php and the changes ValidatorInterface.php.

It seems to me that these should be two separate PRs. I agree with Matthew that more explanation on the reasons behind a significant change would be useful.

Replace the word "change" with "problem".

In this PR, there's a single commit that does two different things

Right!

I agree with Matthew that more explanation on the reasons behind a significant change would be useful.

I agree also.

I wanted to point out the problem with the "all-in-one-commits". No more.
We should add a new paragraph to the contributors guide to respect the principles of a version control system.

Originally posted by @froschdesign at https://github.com/zendframework/zend-validator/pull/177#issuecomment-316284942


Update formatting-a-csv.md

Updated punctuation.
Updated punctuation:
Improving reading flow:
Link to original source:

Closes #

Sorry, added the comma!

Hello @misselliev,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @misselliev,Thanks again for the PR!                                         You have earned 6 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


Why my Python extension doesn't work?

Environment data

VS Code version: 1.56.0-insider
Extension version (available under the Extensions sidebar): 2021.3.680753044
OS and version: win10家庭中文版
Python version (& distribution if applicable, e.g. Anaconda): 3.8
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: 3.8
Relevant/affected Python-related VS Code extensions and their versions: 2021.3.680753044
Value of the python.languageServer setting: Jedi

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
Just run.
Actual behaviour
It didn't work.
Here are some images:

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

I have same issue, i can't user Interpreter Selecter at state bar, my right click can not get a funcatin defination.
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 15ms, has a falsy return value, Arg 1: Uri:f:\Code\IMP_GCN\IMP_GCN, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 16ms, has a falsy return value, Arg 1: Uri:f:\Code\IMP_GCN\IMP_GCN, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 17ms, has a falsy return value, Arg 1: Uri:f:\Code\IMP_GCN\IMP_GCN, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: Python Extension: interpreterManager.refresh [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 1ms, has a falsy return value, Arg 1: undefined, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 1ms, has a falsy return value, Arg 1: undefined, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]
Error 2021-04-09 20:08:58: autoSelectInterpreter, Class name = f, completed in 3ms, has a falsy return value, Arg 1: undefined, Arg 2: , Return Value: undefined [TypeError: Invalid Version: 2020.7
at new $ (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:77201)
at Function.Y [as compare] (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:80546)
at c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70759
at Array.sort ()
at Object.t.sortInterpreters (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:1:70724)
at y.getBestInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:730186)
at h.onAutoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:61:701269)
at h.autoSelectInterpreter (c:\Users\yk727.vscode\extensions\ms-python.python-2021.3.680753044\out\client\extension.js:9:280757)
at processTicksAndRejections (internal/process/task_queues.js:97:5)]

@yltx You don't have a python file open. Please open a python file, that will activate the extension.
@YangTakeshi What happens if you select the interpreter? If you can, please provide a GIF to explain what is going on. Also this looks like it is a different issue so I recommend filing a separate issue.

@yltx You don't have a python file open. Please open a python file, that will activate the extension.
@YangTakeshi What happens if you select the interpreter? If you can, please provide a GIF to explain what is going on. Also this looks like it is a different issue so I recommend filing a separate issue.

I can use command to select interpreter, but my state doesn't have  select interpreter button.

Also, my right click meau doesn't have go to defination

I solved the problem by deleting the C:\Users$USERNAME\AppData\Roaming\Code

@yltx You don't have a python file open. Please open a python file, that will activate the extension.
@YangTakeshi What happens if you select the interpreter? If you can, please provide a GIF to explain what is going on. Also this looks like it is a different issue so I recommend filing a separate issue.

I can use command to select interpreter.

But my state doesn't have select interpreter button.

Also, my right click meau doesn't have go to defination

事实上，我是看他半天没反应之后才切掉的窗口的。否则，右下角也不会提示“相应时间过长了”。

Hi @yltx 👋 Unfortunately I'm not very clear on the issue you're facing and I'm unable to reproduce it from what I understand. What doesn't work? Is it the status bar display?
Please provide the logs as mentioned in the issue template.
Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Note we investigate issues in order based on priority and severity, which includes the impact it has on your ability to use the extension to do productive work, and the number of people affected. If other users come forward and leave a comment demonstrating they are seeing/reproducing the problem then we will raise this issue's priority.
If you think your issue is more of a question or configuration problem rather than a bug, please ask on Stack Overflow with the visual-studio-code and python labels for help.
Thanks for your understanding and patience!

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.

When I right-click in the PY3 file and select "run selection in Python terminal (Shift + Enter)", the python plug-in starts and waits for a long time without response.

Closing as dup of https://github.com/microsoft/vscode-python/issues/16768


The ConverterParameter should be mentioned in this page.

The new MAUI community toolkit has a ConverterParameter which is the type of the enum doing the conversion. In other versions this parameter wasn't required but now it is since if you try to convert back from an int to an enum, the code will throw argument null excepetion here: https://github.com/CommunityToolkit/Maui/blob/6d3ecd66810077bb102436cf2d264219d3cd7c04/src/CommunityToolkit.Maui/Converters/EnumToIntConverter.shared.cs#L34

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

ID: e79e0ccd-3a58-690b-3d39-a4a93a16ecae
Version Independent ID: e79e0ccd-3a58-690b-3d39-a4a93a16ecae
Content: EnumToIntConverter - .NET MAUI Community Toolkit - .NET Community Toolkit
Content Source: docs/maui/converters/enum-to-int-converter.md
Product: dotnet-communitytoolkit
GitHub Login: @CliffAgius
Microsoft Alias: bramin

@emorell96 Thank you for raising the issue. I'll look over the details and see if it's something we need to fix in the docs or whether the code change really was necessary.
cc @brminnick

The code change was definitely nice IMO, since now the converter actually checks that the int is defined in the given enum. It was just a surprise when that part of the code started to throw exceptions.

This issue is related to this one over on the code repository: https://github.com/CommunityToolkit/Maui/issues/379


Unrecognised tokens are only passed through if a parameter is matched

Best explained with some code:
import cyclopts
from cyclopts import Parameter
from typing import Annotated
app = cyclopts.App()
@app.default
def main(is_set: bool = False, *tokens: Annotated[str, Parameter(show=False, allow_leading_hyphen=True)]):
    print(f"is_set = {is_set}")
    print(f"extras = {tokens}")

# This works
app(["--something-else", "--is-set"])

# This fails with "unknown option: --something-else"
# I'd expect it to call `main("--something-else")` (or something like that?)
app(["--something-else"])

Running cyclopts 2.9.4 on Python 3.10.11

Thanks so much for such a quick and helpful response, @BrianPugh!
I had previously thought that the *tokens field was somehow handled specially, but now I see it actually isn't - it's just a standard, required (but possibly empty) list of positional arguments.  Thanks again!
Thanks for an awesome library; this is the forth I've tried (argparse, fire, pyinvoke, click) and is the first I'm really happy with!


Fixed leak and removed no-shuffle tag in scheduler_test.dart

This PR fixed the problem that has prevented scheduler_test.dart being shuffled. Part of #85160.
One test schedules a task, but events are locked so the scheduled task does not execute and this messes up other tests.
This PR adds a teardown that executes the scheduled task after the test, when events are unlocked.
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

@googlebot I consent.


error building in cluster config: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined

deploy the virtual node   binary
./bin/virtual-node --nodename <IP_ADDRESS> --provider k8s --kube-api-qps 500 --kube-api-burst 1000 --client-qps 500 --client-burst 1000 --kubeconfig /root/certs/master/admin.config  --client-kubeconfig /root/certs/node/admin.config

FATA[0000] error building in cluster config: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined

when  set
export KUBERNETES_SERVICE_HOST=<IP_ADDRESS>
export KUBERNETES_SERVICE_PORT=6443
root@virtual-kubelet:~/tensile-kube# ./bin/virtual-node --nodename <IP_ADDRESS> --provider k8s --kube-api-qps 500 --kube-api-burst 1000 --client-qps 500 --client-burst 1000 --kubeconfig /root/certs/master/admin.config  --client-kubeconfig /root/certs/node/admin.config
Error: error building in cluster config: open /var/run/secrets/kubernetes.io/serviceaccount/token: no such file or directory
Usage:
  virtual-node [flags]
  virtual-node [command]

Available Commands:
  help        Help about any command
  providers   Show the list of supported providers
  version     Show the version of the program

binary deployment description is very simple，how to solve it?

--kubeconfig /root/certs/master/admin.config

is this path exits? if not, it will run as InCluster

--kubeconfig /root/certs/master/admin.config

Does this path exits? if not, it will run as InCluster

thank you for your reply.   remote debug found that the path was not exists.
// Check if the kubeConfig file exists.
	if _, err := os.Stat(configPath); !os.IsNotExist(err) {
		// Get the kubeconfig from the filepath.
		config, err = clientcmd.BuildConfigFromFlags("", configPath)
		if err != nil {
			return nil, errors.Wrap(err, "error building client config")
		}
	} else {
		// Set to in-cluster config.
		config, err = rest.InClusterConfig()
		if err != nil {
			return nil, errors.Wrap(err, "error building in cluster config")
		}
	}

I think maybe add a log here

--kubeconfig /root/certs/master/admin.config

Does this path exits? if not, it will run as InCluster

thank you for your reply. remote debug found that the path was not exists.
// Check if the kubeConfig file exists.
	if _, err := os.Stat(configPath); !os.IsNotExist(err) {
		// Get the kubeconfig from the filepath.
		config, err = clientcmd.BuildConfigFromFlags("", configPath)
		if err != nil {
			return nil, errors.Wrap(err, "error building client config")
		}
	} else {
		// Set to in-cluster config.
		config, err = rest.InClusterConfig()
		if err != nil {
			return nil, errors.Wrap(err, "error building in cluster config")
		}
	}

I think maybe add a log here

You can open a PR to do this if you  like, thanks!

--kubeconfig /root/certs/master/admin.config

Does this path exits? if not, it will run as InCluster

thank you for your reply. remote debug found that the path was not exists.
// Check if the kubeConfig file exists.
	if _, err := os.Stat(configPath); !os.IsNotExist(err) {
		// Get the kubeconfig from the filepath.
		config, err = clientcmd.BuildConfigFromFlags("", configPath)
		if err != nil {
			return nil, errors.Wrap(err, "error building client config")
		}
	} else {
		// Set to in-cluster config.
		config, err = rest.InClusterConfig()
		if err != nil {
			return nil, errors.Wrap(err, "error building in cluster config")
		}
	}

I think maybe add a log here

You can open a PR to do this if you like, thanks!

done


Highlight to display open apps is thinner when large icons are used

Describe the bug
When large icons are used the highlight shown here is very thin.
Possibly related: https://github.com/ChrisAnd1998/TaskbarX/issues/683
To Reproduce
Steps to reproduce the behavior:

Change the tray icons to large.

Expected behavior, this is what is suppose to show.

Screenshots Problem

Desktop (please complete the following information):

Microsoft Windows 10 Pro (Build 19042)

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


chore(launch): Return updated runSpec after pushToRunQueue query

https://wandb.atlassian.net/browse/WB-11577
Description
We now update the run spec on the backend based on the default resource configuration for a queue, so we want to reflect that to the user when printing
Testing
new test, should fail for old versions of the backend, but only the test.

Tests are super annoying here, we need to support older versions of the query that doesn't return runSpec. We can either catch that error like implemented here, or do introspection...

One test should fail, on the manual pushToRunQueueByName, until the new backend path is in

add mocked test


error: 'rdpSettings' has no member named 'width' when compiling

Trying to compile head from a recursive github clone(git clone --recursive https://github.com/neutrinolabs/xrdp) on Debian 8.
./configure --enable-kerberos --enable-neutrinordp
Finishes cleanly. Running make then fails on the above message, followed by several other similar errors:
make[2]: Entering directory '/home/msimmons/xrdp/neutrinordp'
  CC       xrdp-color.lo
  CC       xrdp-neutrinordp.lo
xrdp-neutrinordp.c: In function 'lxrdp_start':
xrdp-neutrinordp.c:74:13: error: 'rdpSettings' has no member named 'width'
     settings->width = w;
             ^
xrdp-neutrinordp.c:75:13: error: 'rdpSettings' has no member named 'height'
     settings->height = h;
             ^
xrdp-neutrinordp.c:76:13: error: 'rdpSettings' has no member named 'color_depth'
     settings->color_depth = bpp;
             ^
xrdp-neutrinordp.c:79:13: error: 'rdpSettings' has no member named 'encryption'
     settings->encryption = 1;
             ^
xrdp-neutrinordp.c:80:13: error: 'rdpSettings' has no member named 'tls_security'
     settings->tls_security = 1;
         ^
xrdp-neutrinordp.c:81:13: error: 'rdpSettings' has no member named 'nla_security'
     settings->nla_security = 0;
             ^
xrdp-neutrinordp.c:82:13: error: 'rdpSettings' has no member named 'rdp_security'
 settings->rdp_security = 1;
             ^

I'm sure it's not going to show up in this paste, but the carrot is pointing to the last arrow in each case.
Also, compiling without --enable-neutrinordp works fine.

I believe you got FreeRDP installed side by side with NeutrinoRDP which is
FreeRDP 1.0.1 fork and it's got an older API.
Did you install NeutrinoRDP at all?
Xrdp configure script is looking for a pkgconfig file NeutrinoRDP which is
also called freerdp.pc and might find this file but of newer freerdp
version.
On Sun, Jul 16, 2017 at 3:35 AM Matt Simmons<EMAIL_ADDRESS>wrote:

Trying to compile head from a recursive github clone(git clone --recursive
https://github.com/neutrinolabs/xrdp) on Debian 8.
./configure --enable-kerberos --enable-neutrinordp
Finishes cleanly. Running make then fails on the above message, followed
by several other similar errors:
make[2]: Entering directory '/home/msimmons/xrdp/neutrinordp'
CC       xrdp-color.lo
CC       xrdp-neutrinordp.lo
xrdp-neutrinordp.c: In function 'lxrdp_start':
xrdp-neutrinordp.c:74:13: error: 'rdpSettings' has no member named 'width'
settings->width = w;
^
xrdp-neutrinordp.c:75:13: error: 'rdpSettings' has no member named 'height'
settings->height = h;
^
xrdp-neutrinordp.c:76:13: error: 'rdpSettings' has no member named 'color_depth'
settings->color_depth = bpp;
^
xrdp-neutrinordp.c:79:13: error: 'rdpSettings' has no member named 'encryption'
settings->encryption = 1;
^
xrdp-neutrinordp.c:80:13: error: 'rdpSettings' has no member named 'tls_security'
settings->tls_security = 1;
^
xrdp-neutrinordp.c:81:13: error: 'rdpSettings' has no member named 'nla_security'
settings->nla_security = 0;
^
xrdp-neutrinordp.c:82:13: error: 'rdpSettings' has no member named 'rdp_security'
settings->rdp_security = 1;
^
I'm sure it's not going to show up in this paste, but the carrot is
pointing to the last arrow in each case.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/neutrinolabs/xrdp/issues/818, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADTH1MoKkHWNBTwaBfmmnsimhdCDKPZoks5sOVrhgaJpZM4OZKcX
.

--
Idan Freiberg
PGP FP: 8108 7EC9 806E 4980 75F2  72B3 8AD3 2D04 337B 1F18

--enable-neutrinordp requies NeutrinoRDP installed.


Minor link fix

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/graph-api/import-messages/import-external-messages-to-teams.md
92
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

No change after merging from main :-) The change was made to main via another PR so closing this PR.

Docs Build status updates of commit 3457d3c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add optional version param to jsDelivr badges

:clipboard: Description
It would be nice if we could call for example /hd/jquery/jquery/3.3.1.svg

How would we go about implementing this?
Asking because of an in-progress refactor, but I'm also happy to take this up.

An optional param can be added by using a ? e.g:
https://github.com/badges/shields/blob/951aa0f9ead34637ac5843bb1316d434d1aa0a58/services/jenkins/jenkins-plugin-installs.service.js#L97-L102
so you could modify the pattern in
https://github.com/badges/shields/blob/951aa0f9ead34637ac5843bb1316d434d1aa0a58/services/jsdelivr/jsdelivr-hits-github.service.js#L6-L11
to
static get route() {
  return {
    base: 'jsdelivr/gh',
    pattern: ':period(hd|hw|hm|hy)/:user/:repo/:version?',
  }
}

If I call
https://data.jsdelivr.com/v1/package/gh/jquery/jquery/stats/date/week
for example, I could write some code like
const data = await this.fetch(/*params*/)
const version = '3.3.1'
let downloads = 0
for (date in data.dates) {
  if (version in data.dates[date].versions) {
    downloads += data.dates[date].versions[version]
  }
}

to iterate through each date and sum the downloads for only a specific version.

Or just ask for that version 😛<EMAIL_ADDRESS>

..or that :smile:

So currently we can call<EMAIL_ADDRESS>to get only that version, but would it be worth either adding some documentation to show this or adding an optional version parameter to make it easier?

Yes - this is a good point - you could just add an example to the examples array to show calling with the package@version syntax. I hadn't clocked that would work as the code stands.
One other thing we usually do with this style of badge is modify the label if version is specified. e.g:
 - https://img.shields.io/crates/d/rustc-serialize.svg
 - https://img.shields.io/crates/d/rustc-serialize/0.3.24.svg
 - https://img.shields.io/jenkins/plugin/i/view-job-filters.svg
 - https://img.shields.io/jenkins/plugin/i/view-job-filters/1.26.svg

Maybe it’s better to support it with a slash as usual, and use a stricter regex on the version that rejects it as part of the package name. That’s a bit friendlier in the UI which is the main reason; it also requires less custom logic to detect the version and render the label.


Ignore node_modules for the root and react-router-website folders

I know @mjackson has some opinions about this. However, it's consistent with the other packages' .gitignores, so I'm going to merge it in. Feel free to revert if you don't want it in there for a specific reason.

Yes. I made this PR because I saw other packages ignores them. So, I made it consistent with others.


[REVIEW] Fix incorrect output from averages with filters in partial only mode

Fixes #155 .
This is WIP for a couple reasons and also as I want to get more comments on the approach shown in the PR.
As the associated issue explains in some detail, how we are ending up with nulls being sent down to the CPU's final aggregation. The fix tries to circumvent this by special casing how averages are handled by the case-when of the filter. One thing to note is, we need this special case since we don't have a clean way to pass the filter down to the GpuAverage DeclarativeAggregate itself. When a null comes into the filter we want it not pass down any further but instead default to (0.0, 0). This seems to be specific for averages while other aggregates like count require nulls. I had to use another case-when with isnotnull to make this happen. I have some concerns on the else condition added in this PR and would like to know what others think is a better way to do this. I tried some games with having initialValues be reused in the filter it was not straight forward what do to with their data types. At the end, I am also open to use this PR to morph the solution to put the hammer on filters+avg+partial_only_conf to fall back on the CPU, but that is not my first preference. Tagging @abellina for some discussion to get this PR going.

Addressed review comments @abellina and additionally took out a test that was redundant after this fix. Please take a look.

build

build

Waiting for approvals and CI, after which, I will rebase-squash and sign off.

Squashed and signed off, re-runing CI

build


Module '"vue3-promise-dialog"' has no exported member 'DialogWrapper'

Installed vue3-promise-dialog in existing webpack/typescript vue app, PromiseDialog is imported fine but there doesn't appear to be a DialogWrapper component exported (at least as far as typescript knows).
import { DialogWrapper } from 'vue3-promise-dialog';

Oh sorry. I only tested the project on Stackblitz and no error was reported.
I use Vite library mode to build this lib. Unfortunately, it doesn't generate Typescript definition files by itself. I fiddled a lot with this but couldn't find a proper solution. I'll take a look at it again.
Any help on this is appreciated.

No worries, also noticed I have strict typescript checking on so I'm undoubtedly more "errors" than I should. Will see if I can resolve it.

I'll take a look at this plugin and see if it solves the problem : https://github.com/qmhc/vite-plugin-dts

It should be fine now.


add support for album name scrobbling, custom_song_export, Now Playing, configurable scrobble threshold, improved logging

Currently, CHScrobbler assumes that the first line of currentsong.txt is always the song title, and the second line is always the artist name. However, Clone Hero has a setting (custom_song_export) that allows you to change the format of currentsong.txt. This PR adds functionality to CHScrobbler so it uses the value of custom_song_export to ensure it always parses currentsong.txt correctly.
In addition to the above, this PR also enables CHScrobbler to include the album name when scrobbling a track. This only works if you add %b to your custom_song_export pattern. It does not matter where you put %b in the export pattern. If you don't do this, CHScrobbler will still work just fine, it will scrobble without album names like it always has.
I also added support for Last.fm Now Playing. As soon as you begin playing a track, it calls the Last.fm API method track.updateNowPlaying. There is no API method to reset Now Playing, this is handled automatically by Last.fm.
Lastly, I made the scrobble threshold configurable. The default is still 25 seconds.
Some thoughts about the code:

I opted not to add a dependency to a library that can parse .ini files, instead it just reads the lines of settings.ini until it finds "custom_song_export =".
I made it so that it logs an error if %s (track) or %a (artist) is missing in custom_song_export, but it won't stop the program from running, even though it will run into trouble when it starts reading from currentsong.txt.
Instead of giving a warning, perhaps the CHScrobbler setup could automatically add %b to the custom_song_export setting? I'm assuming everyone would want the album name to be included in the scrobble data.
For performance, I wrote it so that it will only use regex to parse currentsong.txt when it has to. It won't use regex if custom_song_export is unchanged.

From a quick glance, everything looks good, but I will have a better look later today.
Thank you for your contribution!

Excellent job here! I requested a few changes to follow last.fm's API rules. Thanks for improving the logging, I always keep missing that mark.
I also didn't know that Clone Hero supported custom formats for the custom song display! I haven't played CH in a while, which is why this project hasn't been updated.

Thanks for taking the time to review the code. I have made the suggested changes!

I'm looking into ways to get the duration of the song, so CHScrobbler will scrobble the song when it's played halfway (or when 4 minutes have passed), but I'll leave that for another PR, IF I get it working.

Great! Will you be creating a new release soon?

Just released! Sorry it took so long, I wanted to do some testing and I noticed a couple bugs while doing so, such as:

NPE is thrown when custom song format doesn't have album and an attempt to scrobble is made because the album string was not set
Scrobbles are continuously attempted due to the way the if checks were handled regarding the album string.

I also wanted to see if it was possible to make this app work for Mac and Linux and I haven't had the chance to use either platform until today.
Please try out the release and let me know if there's any issues you encounter. Thanks!

Nice work! I did not consider at all what would happen when a custom song doesn't have an album in it's song.ini.