Switch widget color doesn't follow defined ColorScheme on Android

Looks like Switch widget still needs deprecated accentColor to be defined in ThemeData in order to not use the default color on Android. In my project I defined a whole ColorScheme to not use deprecated color values in ThemeData and saw that Switch ignores secondary defined in the app theme. I can only change color by adding accentColor property.
Attached a sample project with fully defined ColorScheme and Switch not following it.
This is how Switch looks with just ColorScheme defined:

This is how Switch should look like and how it looks when I add accentColor to ThemeData:

Steps to Reproduce

Create new project
Define full ColorScheme (with all defined colors)
Add Switch widget and observe that it has the default Material blue color on Android.

Expected results: Switch follows secondary color as it is suggested in deprecation message for accentColor
Actual results: Switch has default Material blue color on Android

Code sample
bug.zip

flutter doctor output
// Attaching only Flutter related info, since other is irrelevant for the issue
[✓] Flutter (Channel stable, 2.5.3, on macOS 11.6.1 20G224 darwin-x64, locale en-UA)
    • Flutter version 2.5.3 at /Users/vb/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

Thanks for the report. I see the same behavior.

 stable, master flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.5.3 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (2 days ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/dhs/Library/Android/sdk
    • Java binary at: /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/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
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 93.0.4577.82

• No issues found!

[✓] Flutter (Channel master, 2.6.0-12.0.pre.737, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-GB)
    • Flutter version 2.6.0-12.0.pre.737 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee87f6113e (2 hours ago), 2021-11-17 05:43:05 +0300
    • Engine revision<PHONE_NUMBER>
    • Dart version 2.16.0 (build 2.16.0-14.0.dev)
    • DevTools version 2.8.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/dhs/Library/Android/sdk
    • Java binary at: /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/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
      1.8.0_242-release-1644-b3-6915495)

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

[[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 93.0.4577.82

• No issues found!


Remote-write should check that label names are valid

Proposal
Prometheus remote-write receiver should check that labels conform to the documented restrictions.
Not checking allows callers to create series which cannot be queried via PromQL.
To illustrate, change the test data for TestRemoteWriteHandler and the test still passes:
@@ -30,7 +30,7 @@ var writeRequestFixture = &prompb.WriteRequest{
        Timeseries: []prompb.TimeSeries{
                {
                        Labels: []prompb.Label{
-                               {Name: "__name__", Value: "test_metric1"},
+                               {Name: "__name__", Value: "test.metric1"},
                                {Name: "b", Value: "c"},

You can also pass the test with blank labels, no __name__, and pretty much everything else I tried.
Related: #11505

Yes, this is something known. The only remote write client we support is Prometheus. I guess we have to be more careful now that 3rd parties also use the protocol.

Hello from the bug scrub.
Apologies for the delay in looking at this.
@alexgreenbank to look at a proposal to enforce the existing spec alongside other related issues.


Update action version in README

via 3e51a0050e3249e7932b81fe358fbf70cf75bdb7 -- thanks for the patch (I made sure to commit it as you so you get credit!)


Minor: improve join / join_on docs

~Draft as it builds on https://github.com/apache/arrow-datafusion/pull/7805~
Which issue does this PR close?
Related to https://github.com/apache/arrow-datafusion/pull/7805 and https://github.com/apache/arrow-datafusion/issues/7766
Rationale for this change
The capabilities of datafusion with joins could be better documented, so let's fix that.
What changes are included in this PR?
Update documentation comments
Are these changes tested?

Are there any user-facing changes?

Thanks for the review @jackwener


Add ascii art

This adds my ascii art to the packages maintained by me
Ascii art:
 __      __________   ______
/  \    /  \_____  \ /  __  \
\   \/\/   //  ____/ >      <
 \        //       \/   --   \
  \__/\  / \_______ \______  /
       \/          \/      \/

Should I merge

Bruh, why did the checks fail xD

Something wrong with the github action. It works correctly on the first one but not the second or others


fix(redis): do not update metadata with negative ttl

Hi!
This attempts to fix the regression from #215 where *RedisStorages instantiated without a TTL will fail during update_metadata().
The failure stems from PTTL returning -1 if the key exists but does not have a TTL.
Improvements to this commit would involve E2E testing of hishel to prevent introducing similar regressions in the future.
I might suggest VCR-esque tests for each of the storages, but I haven't gone that far in this bugfix attempt.

Hi!
Oh, that's great! Thank you for the fix :pray:

Improvements to this commit would involve E2E testing of hishel to prevent introducing similar regressions in the future.
I might suggest VCR-esque tests for each of the storages, but I haven't gone that far in this bugfix attempt.

I guess, we do have some integration tests here.
Maybe we could just add some tests like test_{storage_name}_metadata_updating?

Seems like we also should run scripts/lint, to generate the synchronous code for this fix.

Seems like we also should run scripts/lint, to generate the synchronous code for this fix.

Good point, missed that -- done. ✅

Great, thanks!


TypeScript helper separation

Does TypeScript have a runtime-style module? Or are helpers concatenated per-build?
Currently the build is inlining all helpers, which is obviously not ideal.

@guybedford Currently we've made a few modifications in TypeScript (should be available in 1.6)

we emit helper inside the module so it does not mess up module kind detection logic in SystemJS
there is a noEmitHelpers compiler options that suppresses generation of these helpers so user can provide its own global implementation of __extends, __decorate etc.

Is it enough for your scenarios or you would expect runtime library to be a module so in case if user writes some code that uses runtime helpers we would emit some code like
System.register(["typescript-runtime"], function(exports_) {
    var typescriptRuntime; // set in setter
    var __extends = (this && this.__extends) || typescriptRuntime.__extends;
    ....

})

there is a noEmitHelpers compiler options that suppresses generation of these helpers so user can provide its own global implementation of __extends, __decorate etc.

@vladima are you guys planning on shipping a file with all the global helpers?

@vladima that sounds great. A runtime library module would definitely be nice to reference rather than relying on globals.
Further than that, Babel has a transform option to output modular helpers:
System.register(['helpers/core', 'helpers/class'], function(exports_) {
  var __extends, __decorate;
  return {
    setters: [
    function(core) { __extends = core.__extends; },
    function(class) { __decorate = class.__decorate; }
    ],
  };
});

etc. That said this now has the reverse problem that too much granular modularity actually becomes itself a performance bottleneck - something inbetween the Babel approach and a single runtime library might also be useful to split up just the needed helper modules as well (I guess only useful if the TypeScript runtime is very large to split it up?)

we are considering this as an option but we need to figure out some questions about the format of helper: should it be just a set of globals or it should be a UMD module. If helper library is a module then it affect the codegen - we need to inject a runtime library in the list of modules

@vladima the helper library can be treated just like any other library - distributed however the consumer desires - UMD / CommonJS / ES6. In terms of how we run TypeScript in this project, we would do a single file transpile and then set a custom path to the runtime library (my/custom/helper/base sort of like a base path to the helpers being at my/custom/helper/base/core etc), then most likely using a global, CommonJS or UMD build.
In terms of generating the helper library as part of a full tree build, then yes it would need to be injected for those cases using a standard format.

Any news about how we could get a prelude containing the helpers required by our code to append it once to built files?
Passing --noEmitHelpers is part of the solution, but currently we concatenate fixed helpers that we copied from TS source.
This is fragile. For instance, TS 1.8 has changed its code for __awaiter, so our code broke, without build-time error of course: we caught it in testing.
We use gulp to build, it would be nice if TS could provide a file with just the helpers required by our code in its results. Babel does that for instance.
Alternative: let us use babel for all transpiling by providing a new ESnext target, so that no TS helpers at all are required.

Work on TypeScript use cases is happening through the plugin https://github.com/frankwallis/plugin-typescript, as this internal TypeScript transpilation will be deprecated in the next breaking release.

@guybedford sorry, you can dismiss my comment. I got a notification from this issue (I subscribed to it) and added a comment assuming I was in the TS repo... Didn't pay attention to the fact this was systemjs! Too many subscriptions it seems, I get confused!


Notification support for mobile devices

Great project!
However the description suggests support for all devices.
However it does not work on mobile devices. You do get the request the allow notifications, but a notification is never sent.
Is this support anticipated soon?

Theoretically, it should work cross-platform, but mobile devices were never tested. I actually started this project to complement a desktop-oriented website I was building, so I never thought of testing it on phones. However, now that more people are using it, I'll definitely look into what the issue might be.

Currently investigating this issue on mobile Chrome (Android N). It seems to be throwing the following error:
Uncaught TypeError: Failed to construct 'Notification': Illegal constructor. Use ServiceWorkerRegistration.showNotification() instead.

As it turns out, mobile Chrome does not support the new Notification() constructor (as it is being depreciated) and instead requires a service worker. Some justification as to why can be found here. This should be a fairly straight-forward fix though. Working on a fix :)

A quick update on this issue: apparently the new ServiceWorker approach requires the host site to use SSL in order to function properly. SSL is enabled on the GitHub page, but I'm trying to get a local SSL server running to test before I commit anything.

Exciting news! Mobile chrome support was just added with 4edcc9d6ec16476378c33a829efb7c82bbdb5631. You can even make the device vibrate when you get a notification :) Enjoy! Let me know if you run into any bugs. Closing this for now.

Great! You're fast! :)
Is it correct that the demo wasn't updated yet? At least it doesn't work there.
Did you also test is on other mobile browsers (Safari, etc).
Thanks!

Apparently mobile Safari doesnt even have the ability to send push notifications, so it was not tested. And the demo was updated, although the site needs to be using a valid SSL certificate in order for it to work properly (something mobile Chrome requires, not me). So try https://nickerson.github.io/push.js/demo.html.

Great! It works on Android. However on IOS on an Iphone it still doesn't work. Would be great if that is supported as well! You rock!

Haha thanks.  :)

Just checking now, it seems that the Notifications API in general is not supported on iOS, in neither Chrome or Safari. According to this Verge article from last year, Chrome still cannot send push notifications on iOS. Looks like Apple will once again need to play catch-up with Google.

Damn iOS always has to complicate everything.

On my mobile notification not shown.


bugfix for multiple shadow threads rendered at the same time, small probability crash.

Summary
bugfix for multiple shadow threads rendered at the same time, small probability crash.
Changelog
Android
Fixed
ViewManagersPropertyCache.class
Background:
on the Tab Page, every tab has a React Page. If the React page is not using the same ReactInstanceManager, multiple shadow threads will be generated.
A crash can occur when multiple pages are rendered at the same time.
Reasons of issue: PropSetter class VIEW_MGR_ARGS, VIEW_MGR_GROUP_ARGS, SHADOW_ARGS, SHADOW_GROUP_ARGS is static.
one shadow put data in static array, it was changed by another shadow thread before method invoke.
The solution:
use ThreadLocal instead of static array.
Test Plan
Test on my App 20 times. it is not crash again.

I am not change anything about iOS. why  "ci/circleci: test_ios_unit_jsc — Your tests failed on CircleCI"

@chenmo187 Can you give more insight into the problem this is solving and a reproduction? Additionally, can you format your PR?

@chenmo187 Can you give more insight into the problem this is solving and a reproduction? Additionally, can you format your PR?

OK, I will make a new PR

Cool, you can request a review from me.


Heading of 2D Detections

Is there a way to get the heading of the objects using the 2D bounding boxes (vehicles)?

Are you using the perception dataset? Our bounding boxes should have 3D info including heading here: https://github.com/waymo-research/waymo-open-dataset/blob/c9bf4d829bdf10f48a0dec2ef0ef2e5e05a8a0db/src/waymo_open_dataset/label.proto#L38


Fail to update addon

Hey @creativetags , could you please share more information about the device which you are using for home-assistant?
For now it looks like https://github.com/fuslwusl/homeassistant-addon-victoriametrics don't support your device architecture. I would be great if you run uname -a from terminal share the output of this command.

It’s a raspberry pi 4

It’s a raspberry pi 4

What OS you run on raspberry? There is some different behavior with uname -m output for example on debian or arch linux. Please share the output:

from uname -m
from uname -a
from lscpu

Thanks!

aarch64
Linux a0d7b954-ssh 6.1.73-haos-raspi #1 SMP PREEMPT Wed Mar 13 12:10:55 UTC 2024 aarch64 Linux
lscpu is not installed. It's the standard Home Assistant install from https://www.home-assistant.io/installation/raspberrypi The only non-standard thing is I've moved data to a USB memory stick via Settings > System > Storage > Three Dots Menu >  Move datadisk

Still an issue with version 1.9.2
Error updating Victoria Metrics: The command '/bin/ash -o pipefail -c /bin/bash -c 'set -ex &&     ARCH=$(uname -m) &&     if [ "$ARCH" == "x86_64" ]; then         wget https://github.com/VictoriaMetrics/VictoriaMetrics/releases/download/$VERSION/victoria-metrics-linux-amd64-$VERSION.tar.gz -qO- | tar xvz -C /;     elif [[ "$ARCH" == "arm"* ]]; then         wget https://github.com/VictoriaMetrics/VictoriaMetrics/releases/download/$VERSION/victoria-metrics-linux-arm-$VERSION.tar.gz -qO- | tar xvz -C /;     elif [ "$ARCH" == "aarch64" ]; then         wget https://github.com/VictoriaMetrics/VictoriaMetrics/releases/download/$VERSION/victoria-metrics-linux-arm64-$VERSION.tar.gz -qO- | tar xvz -C /;     else         echo "Unsupported architecture $ARCH";         return -1;     fi'' returned a non-zero code: 1

Hey @creativetags @robvolgers , please share the output of uname -m and echo $SHELLfrom your devices.

aarch64
/bin/sh

aarch64
/bin/sh

Update: Home Assistant crashed  shortly after. After a new clean install + recent backup I could update Victoria Metrics with no problem.

I am closing this ticket as there has been no feedback for two months, so I believe the problem has been resolved. If the problem occurs again, please reopen the ticket.

Sorry for not replying. I have now tested and it's working well thanks.


When using --graph-output and -p, the output file contains all installed packages deps.

When using --graph-output and -p, the output file's graph contains all installed packages deps.
I would expect the graph contains only the selected packages and dependencies.
When I use the -p option alone I do see the expected selected content. When additionally use the --graph-output flag, the option -p is simply ignored, but I would like to see the graph of the selected packages only.

Hi, this has been a known issue and there was an attempt to fix it earlier but had to be reverted as it broke other things (See #87).
This tool was originally written with just printing the output to stdout/stderr in mind and graph support was added later. So the fix will require fair amount of refactoring. I am planning to pick it up after the current task that I am working on, which is sorting out the tests which have become messy and flaky over time.

A pre-release version 2.0.0b1 has been pushed to pypi that fixes this. A stable 2.0.0 version will be released soon. Closing the issue.


Modify Dockstore to use latest AMI

The AMI used for Dockstore should be updated to the latest one due to security issues. Instead of hard coding the AMI ID, grab the latest.

One way to fix the AMI issue to refer to the latest of a given type:
data "aws_ssm_parameter" "ami" {
  name = "/path/to/ami"
}

resource "aws_instance" "nginx" {
  ami           = data.aws_ssm_parameter.ami.value # pull value from parameter store
  instance_type = "t2.micro"

  provisioner "remote-exec" {
    inline = [
      "sudo yum install nginx -y",
      "sudo service nginx start"
    ]
  }
}


Potential bugs for the llama inference example

Hi team,
Sharing some observation for potential bugs as below:

https://github.com/aws-neuron/neuronx-distributed/blob/4f954715f39b2cc9e628ded79274957401bea086/examples/inference/llama2/neuron_modeling_llama.py#L283-L312 this seems to be duplicated. The first class definition should be changed to ResBlock
https://github.com/aws-neuron/neuronx-distributed/blob/main/examples/inference/modules/model_base.py#L453-L486 these lines seems to be not relevant, and should be deleted and replaced with return [res] + updated_kv_cache

Thank you!

@yinsong1986 thanks for reporting! Our inference team will take a look.


BUG:the accept section of the page is not alligned properly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
the accept section of the page is not alligned in a manner

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

which section , image is not visible ?  please provide a detailed explanation of the issue u want to resolve .


SAK-43621 soft-delete student submission by modifying STATUS flag

Hard deleting student submissions is problematic as student responses may contain good info or info to be retrieved by a support team. Why not use the columns we already have like STATUS?

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ samigo/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/evaluation/GrantSubmissionListener.java  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ samigo/samigo-app/src/java/org/sakaiproject/tool/assessment/ui/listener/evaluation/GrantSubmissionListener.java  -4

See the complete overview on Codacy


Changing language in admin breaks the page explorer menu

Steps to Reproduce
On a fresh Wagtail install, log in to the admin and go to Account settings -> Language preferences and set language to French. Opening the page explorer then gives a server error, with this error in the console:
Internal Server Error: /admin/api/main/pages/
Traceback (most recent call last):
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/django/core/handlers/base.py", line 179, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/django/views/decorators/cache.py", line 44, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/admin/urls/__init__.py", line 127, in wrapper
    return view_func(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/admin/auth.py", line 170, in decorated_view
    response = view_func(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/api/v2/router.py", line 63, in wrapped
    return func(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/viewsets.py", line 125, in view
    return self.dispatch(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/api/v2/views.py", line 116, in handle_exception
    return super().handle_exception(exc)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/admin/api/views.py", line 98, in listing_view
    response = super().listing_view(request)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/api/v2/views.py", line 74, in listing_view
    return self.get_paginated_response(serializer.data)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/serializers.py", line 745, in data
    ret = super().data
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/serializers.py", line 246, in data
    self._data = self.to_representation(self.instance)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/serializers.py", line 663, in to_representation
    return [
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/rest_framework/serializers.py", line 664, in <listcomp>
    self.child.to_representation(item) for item in iterable
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/api/v2/serializers.py", line 287, in to_representation
    meta[field.field_name] = field.to_representation(attribute)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/api/v2/serializers.py", line 62, in to_representation
    return page.full_url
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/core/models.py", line 1609, in get_full_url
    url_parts = self.get_url_parts(request=request)
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/core/models.py", line 1587, in get_url_parts
    if get_supported_content_language_variant(translation.get_language()) == language_code:
  File "/Users/matthew/.virtualenvs/langwithoutwagtail/lib/python3.9/site-packages/wagtail/core/utils.py", line 271, in get_supported_content_language_variant
    raise LookupError(lang_code)
LookupError: fr

Related: #6510

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.9
Django version: 3.1.3
Wagtail version: 2.11
Browser version: Chrome 86

Fixed in #6521


update angular version

@kukac7
localForage 1.5 is a breaking change, so I'm loathe to just merge this in without making a major bump. Maybe we could make it backwards compatible for our users by setting Safari to WebSQL and adding a note? Any thoughts?


Are you thinking of building a Soner style structure?

example: https://github.com/xiaoluoboding/vue-sonner

Hi @0fatihyildiz, Notivue will never ship a Sonner-like repositioning experience.
The reason behind that is very simple: I developed this package according to my needs and my beliefs.
I'm not sure the macOS-way to group and clear notifications can be the new default for web apps' generic toasts.
However, the package you linked seems a good solution and I suggest to use it instead of Notivue.


any plans to use localForage?

In order to support more database types, are there any plans to use localforage etc?

nope, no plans at all, unfortunately.


Fix recursive copy pathing

This didn't run for me out of the box and it was because the ./blank/. path was blowing up. This code gets around the issue by changing paths. It works both on OSX and Linux. Along the way I added a couple of utility functions including one to handle all the error checking and exiting. If you hate that I'll rebase with just the pathing change.
Awesome work @didip

I couldn't merge this automatically.. Will look in the morning.

Hm... I just went through a refactor last night, can you rebase and move these functions under package helpers?

@relistan Just out of curiosity, could you post the error you were getting with the ./blank/. path?

@gmas it only worked if you were in the directory when you ran it. So If I were somewhere else . did not contain a directory named blank. I've long since lost the error in my scrollback.
Now you can just go go get github.com/go-bootstrap/go-bootstrap followed by go-bootstrap --dir bitbucket.org/relistan/<my project> from anywhere and it works.
@didip this is rebased

@relistan thanks, makes sense


GCMPlayground doesn't seem to be talking to the server app

So I can load up my GCMPlayground page at ipAddress:3000, and I can create a request just fine. However, when I attempt to submit the request, it doesn't seem to talk to the server. I see the pop-up saying "Sending message", but message doesn't actually get sent.
There's nothing wrong with the request itself -- if I cut-and-paste the data into a curl call, everything works just fine.
 No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://xxx.xxx.xx.xxx:3000' is therefore not allowed access. I'm going to guess that's probably related?

I'lll follow this up by saying it looks like this was user error. I was accidentally using the port of my Docker host (2376) when I should have been using the port of my server (4260, as noted in server.go)
I'll close this for now.


replicate.stream() gives blank spaces at the beginning of output stream

Hello, I got trouble when I want to stream the chat model output from replicate. I'm using replicate version 0.25.1. My code is look like this:
for event in replicate.stream(
    "meta/llama-2-7b-chat",
    input={
        "debug": False,
        "max_new_tokens": 800,
        "min_new_tokens": -1,
        "temperature": 0.75,
        "top_p": 0.95,
        "top_k": 50,
        "repetition_penalty": 1,
        "prompt": prompt,
        "system_prompt": SYSTEM_PROMPT,
        "min_new_tokens": -1,
        "prompt_template": "[INST] <<SYS>>\n{system_prompt}\n<</SYS>>\n\n{prompt} [/INST]",
    },
):
    print(str(event), end="", flush=True)

When I run it for the first time, it give me good result and desired output. But, for the 2nd run it gives me output with one blank spaces in the first line. For the 3rd run it gives me two blank spaces in the beginning of the output. Here's what my output looks like when I run the code 5th times (I got 4 blank spaces in the beginning of the output):

I don't know if it's caused by my prompt or bugs. Can someone help me with this?

I think it's not caused by my prompt. I tried to change the prompt and system prompt like this:
"prompt": "Hello how are you?",
"system_prompt": "Answer the question",

And I still faced the same issue:

Any help or suggestion will be appreciated :)

Hi @yosuaw this should get fixed in coming versions. I the mean time - you can strip away leading newlines from the output as a quick fix.

Thank you so much for your response @Pwntus. Please let me know if the bug is fixed.

Hi @yosuaw. I believe this was the same issue as #287, which is now fixed in 0.25.2. Please update to the latest version of this library and let us know if that doesn't resolve your issue. Thanks!


training or eval question

Hi. Thank you for sharing the code! I think I have trouble training or evaluating. Runing the code provided, it stayed here for several hours, even I change the small contrastive batch size.
clean contrastive=6.242605 	 adv contrastive=6.484784
contrastive bs 64

What GPUs do you use?  The code is evaluated on 8 2080Ti GPUs. The training part uses less GPU as we only train two layers to readout for contrastive loss. The testing needs to backward through the backbone model and consumes more memory.

Thanks! I only have two 1080Ti GPUs to use. I’m not sure it is enough. Could you please offer me any suggestions about this problem?


handle default (identity) matrix

This function is missing the case where an element uses the default (identity) matrix.

Yes, I left it out to simplify the SVG. If nothing is returned, then SvgRenderer._render_element will not add a <g transform="matrix(1, 0, 0, 1, 0, 0)"> element, which I feel is redundant: it adds extra indentation when you format the SVG and increases file size a tiny amount.
The advantage isn't that big, though. Do you run into cases where returning None instead of an identity matrix is confusing? If so, I could have get_matrix return an identity matrix and apply the optimization in SvgRenderer. Or, are there cases where you want the nesting of <g> elements to reflect the exact nesting of XFL elements? If so, then I can just return an identity matrix.

It's not confusing. The problem is that gradient.py doesn't expect get_matrix to return None. It should be fine if it's fixed either in get_matrix or in gradient.py.

Oh, I didn't realize that! Thanks for pointing it out.

Hmm, I don't see anything, so I'm curious what it is. Go for it.


fixed incorrect laue prediction variances

This fixes a bug in computing the SigIpred for Laue data

Codecov Report

Merging #33 (cbff1c9) into main (1597169) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #33   +/-   ##
=======================================
  Coverage   85.73%   85.73%
=======================================
  Files          38       38
  Lines        1290     1290
=======================================
  Hits         1106     1106
  Misses        184      184

Flag
Coverage Δ

unittests
85.73% <100.00%> (ø)

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

Impacted Files
Coverage Δ

careless/models/merging/variational.py
97.18% <100.00%> (ø)

Continue to review full report at Codecov.

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


Property definitions adjustment + inclusion of GNU Units definitions.units

Implements the suggestion discussed in PR #376 after it was merged that the property definitions need a mandatory property definition format version number (brought up by @vaitkus). This is a good idea for several reasons, but not the least to properly accommodate possibly backwards breaking changes in future revisions of JSON schema, which still does not have a "real" release.

Updates the reference to definitions.units to point to the latest version of that file.

Adds the file itself to our repository to not force people to dig through archives to find the right version, and, to also demonstrate what we believe is a valid way of redistributing the file in a repository with otherwise licensed files.

Adds a note in our main README.md about this.

By the way, would it make sense to use even more detailed license identification for definition.units ("GPL 3.0 or later"), or is it OK to let the readers to figure it out by looking at the header of definition.units file?

I try my best to avoid "gotchas" in terms of embedded version numbers etc. that one will surely forget to update when something changes. The file COPYING is included with the verbatim license, and linked.
We still have to be careful to always copy COPYING as well as the new definition.units from GNU Units every time we update it, just in case something changes there.

By the way, would it make sense to use even more detailed license identification for definition.units ("GPL 3.0 or later"), or is it OK to let the readers to figure it out by looking at the header of definition.units file?

I try my best to avoid "gotchas" in terms of embedded version numbers etc. that one will surely forget to update when something changes.

Completely agree.

The file COPYING is included with the verbatim license, and linked.

Strictly speaking, license text in COPYING does not distinguish between "GPL 3.0 only" and "GPL 3.0 or later", thus definition.units should be consulted for the exact license. But this is OK.

Thanks for the rapid reviews. Clearly, even for quite small text edits like this I need to remember to invoke a spell checker; sorry for the trouble with that. Now I think everything above has been addressed.

@ml-evs You are still labeled as "requesting changes". Has everything been addressed?

Sorry @vaitkus and @merkys for another minor change requiring a re-review. (Maybe @ml-evs can approve first; so we know if we are ready.)


Make components headless for Motion component

Description
Migrate styles from components to themes.
Since component styles in themes inherit styles from other components using mergeStyle etc., duplicate styles should be removed.
Problem Statement/Justification
Move static styles from motion component to theme.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
Yes
Additional Information
No response

Closed due to no migration required.


Update DAL issue tracking

Issues are tracked and maintained by pkg/dal to provide some context onto why a configuration is not valid and will cause other dependant operations to fail.
We introduced issues instead of outright preventing updates since DAL models (and schema alterations) are detached from Compose resources (other resource may follow the same pattern in the upcoming future).
The current issue tracking needs to be revised in order to allow some extra metadata to be passed through such as the type of the issue (to know if we need to display schema alteration modals).

This one can wait a bit longer to be resolved in some patch release


Roll Engine from 774ae0313d26 to 6d5dd9639f04 (3 revisions)

https://github.com/flutter/engine/compare/774ae0313d26...6d5dd9639f04
2020-07-15<EMAIL_ADDRESS>[null-safety] update web_sdk build rules to use package_config (flutter/engine#19741)
2020-07-15<EMAIL_ADDRESS>Roll Skia from 79d88fe4d182 to 13711ebe682c (2 revisions) (flutter/engine#19770)
2020-07-15<EMAIL_ADDRESS>Roll Dart SDK from 89fe12b12aab to 06183779d7aa (2 revisions) (flutter/engine#19769)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


[BUG] Customer Lookup set value not resolved

EasyRepro Version
Microsoft Dynamics 365 Online Version 9.1
Serverversion: 9.1.0000.10329
Clientversion: 1.3.2472-191112-195614
Easy Repro
Library used is the version currently on Develop Branch
UCI or Classic Web

UCI (XrmApp)

Online or On Premise

[ ] Online

Browser

[ ] Chrome

Describe the bug
We are testing our sales/quote process automaticaly and implemented several testcases.
On the 18.11.209 we noticed several of our tests fail now.
The last successful execution was on 13.11.2019.
We noticed the cause for the failing tests is that the "customerid" field on the Quote is no longer set correctly. Other lookups still work fine.

-> Feature affected is "Entities (Create, Update, Delete, Duplicate Detection, Assign)"
Special formatting / display
No Special formatting. It is a lookup of type customer (so it can contain accounts or contact)
Code to reproduce
```csharp``
xrmApp.OnlineLogin.Login(_xrmUri, _username, _password);
//we jump directly to quotes by using the accoring url
//eg. /main.aspx?app=d365default&pagetype=entitylist&etn=quote
_xrmApp.CommandBar.ClickCommand("Neu");
_xrmApp.Entity.SetValue(new LookupItem() { Name = OrderFieldHelper.PotentiellerKunde, Value = customerValue, Index = 0 });
//_xrmApp.Entity.SetValue(new LookupItem() { Name = OrderFieldHelper.PotentiellerKunde, Value = customerValue }, 0); does the same
 -->

**Expected behavior**
Problem: Entered text is matched but entry is not selected. Search suggestions are displayed and do not disappear even if other commands are sent (like SetValue on a other LookUp).

Expected: Entry with index 0 is selected

I've had the same problem before, you can try the following code
_xrmApp.Entity.SetValue(new LookupItem() { Name = “OrderFieldHelper.PotentiellerKunde”, Value = “3H44 AG” });

I've had the same problem before, you can try the following code
_xrmApp.Entity.SetValue(new LookupItem() { Name = “OrderFieldHelper.PotentiellerKunde”, Value = “3H44 AG” });

Hey Jinfengv
I am afraid the suggested solutions does not fix the issue.
No matter if I set the index or not it does not work.

try give Schema Name instead of the value mentioned


Merged local feature branch to remote develop branch

new avatar added and new quote records

great, no merge conflict with new quote record


DO NOT USE THIS PACKAGE

cc @rennokki

FWIW, I've been keeping a discrete main branch in a stopgap fork here:
https://github.com/cuppett/php-k8s/commits/main/
And including this separately in composer.json:
{
    "require": {
        "renoki-co/php-k8s": "dev-main"
    },
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/cuppett/php-k8s"
        }
    ]
}

It has a couple other nits included as well to consolidate some of the activity and keep up with other activity/bumps.
Not looking to go alone, but would be willing to help a bit with upkeep in renoki-co/php-k8s if wanted. This package has worked well for me.


Loft curves

l1 = Line(0, 5)
l2 = Pos(Y=-10) * Line(-5, 10)
l = loft([l1, l2])

Result
ValueError: More than one wire, or a wire and a vertex is required

Expected: Trapezoid

loft if defined to work with 2D shapes:
def loft(
    sections: Union[Face, Sketch, Iterable[Union[Vertex, Face, Sketch]]] = None,
    ruled: bool = False,
    clean: bool = True,
    mode: Mode = Mode.ADD,
) -> Part:

Would you like to extend it to work with 1D objects?


Jill92.ogg greatly extends cutscenes due to its thirty-second length

For example, Marvin's scene lasts roughly 1:30, if REmake 3 Jill replaces him, the scene can take 2:30 because Jill repeatedly reuses Jill92.ogg which is thirty seconds long. This voice clip could be split up into three or four seperate ones easily to mitigate this.

Could make Jill92 a narrator kind


port_compiler: generate clang compilation db

In order for newer clang tools to work, they require the presence of a
compilation database in the form of compile_commands.json.  Therefore,
we adapt port_compiler to write such a file.

More info: http://eli.thegreenplace.net/2014/05/21/compilation-databases-for-clang-based-tools


Add tests that things throw when they should

\c @alazier @tgoyne

Other than naming ambiguity :+1:

@tgoyne could you take a look at this?

Still uses the word assert for something that is not an assertion. assertPositive(x) should be a sanity check in a place where it is not possible for x to be non-positive, not a precondition check in a place where x is not allowed to be non-positive.

Can one of the admins verify this patch?

@tgoyne changed the naming on assertPositive

I would prefer the names ensurePositiveIndex(index: Int) or throwForNegativeIndex(index: Int) and not take a second argument, since it is the same for all cases.

@segiddins please just rename to throwForNegativeIndex(index: Int) and let's be done with this. You get my :+1:.

Changed.


Tag Input

⚠ Warning
~This PR is branched off from "Project Input" (https://github.com/toggl/toggldesktop/pull/2907), so it's critical to merge it before reviewing this PR.~
~I will in charge of doing it~
📒 Description
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] Implemented new design for Tag View
[x] Implement new design and logic for Tag Token (Including hover, X button and gradient)
[x] Quick hack 💥 to support TagTextField inside the "TagAutoCompleteView" (See next section for the detail)
[x] Logic for fetching Tags from Library
[x] Logic for Tag DataSource (Including the Tag Selection (Select/De-select))
[x] Logic for creating new tag (but I have a on-going bug, which is the Tag in Library doesn't reflect correctly) (see the Hack 3 section below)
[x] Unable to Tab to focus on Create new tag  (see the Hack 2 section below)
[x] Logic for replacing long tags to "..." tag

👫 Relationships
Close #2851
💥 The Hack
1. Move TagTextField to AutoCompleteWindow
Technically, there is a hidden Tag TextField in EditorController. Then it is “moved” deliberately to AutoCompleteWindows after presenting and move back after the windows is dismissed. By doing this hack, the original logic for presenting the AutoComplete is retained, and the state of TagSearchTextField is retained too. At the first experiment, I try to “clone” itself, but by this approach, more boilerplate codes.
2. Tab to focus on Tag Create Button doesn't work
It turns out that when pressing the Tab button, the TagAutoCompleteView is collapsed unexpectedly, so it's impossible to focus on Tag Create Button.
The hack is simply ignore the TextFieldDidEndEditing in TagTextField, so the Windows is still remained (https://github.com/toggl/toggldesktop/pull/2913/commits/2b05a2768bf8cbe9139128a9348d197fbe1190fe)
3. New tags doesn't appear after creating
After creating new Tag, for some reasons, the Library doesn't return any new tags (https://github.com/toggl/toggldesktop/blob/ca782683aabc180ab3332473f2e33406445461ca/src/ui/osx/TogglDesktop/test2/AppDelegate.m#L1631) (even the Syncer is done). Until I have to close the TimeEntry, then trigger the on_tag again.
The hack is just simply keeping all newTags, and appending it if need.
🔎 Review hints

Able to create new tag
Able to select / de-select tag and the UI is corresponding properly
Able to filter tag
Same Design

Updated ### Hack section. It's ready to review @IndrekV


Rename ancillary dir

Make the ancillary dir name consistent w/ the base output dir (include the entire version string).

Perhaps we want to remove reference to "eCDR" in "official" directories and filenames?

Yes, good point. I'll drop the eCDR from these dirs!


[v12] Manual Input Dice — roll for location

Not really a bug or feature request, more like something to consider.
With Foundry v12, some people will probably want to use the manual dice input - mostly people who use Foundry for in-person games, but maybe others too.
With that in mind, there is a question of separate roll for location. With Manual Input it asks for two roll values for every attack.
I think this could be a setting (replacing "Manual Chat Cards") with following dropdown options:

Manually input Roll for Hit Location (as is now)
Reverse Weapon Roll - default
Roll for Hit Location randomly in the background (using evaluate({allowInteractive: false})

The option above would matter only if users use manual input for their d100.

I don't think there's a real demand for manually rolling hit location, unless people just like wasting time. The intended default would be 3, with the option of 2, although I still prefer to have 3 as the only choice because 2 is just worse.

I don't think there's a real demand for manually rolling hit location, unless people just like wasting time. The intended default would be 3, with the option of 2, although I still prefer to have 3 as the only choice because 2 is just worse.

That may well be true, I thought some people may want that. I think it could be something to ask community, but you're probably right


how to store the conversion chat in dialogflow.

how can we store the every conversion chat in dialogflow and based on the past conversion it going to train the model.

I don't believe that DialogFlow currently stores every conversation.


dev: Add ddsignfiles wrapper

This file is a wrapper around the DDSignFiles.dll tool that is present in pipelines that have signing setup and enabled. It has been copied from source because it's easier than consuming an npm package. We can use this js helper to sign files with a js script in each extension instead of requiring a .csproj and building it with MSBuild. I think I'd rather stay in the nodejs world if we can.

Docker and several others do not use the dev package. Can we put this in the pipelines instead?

Docker and several others do not use the dev package. Can we put this in the pipelines instead?

Rats. The pipelines folder isn't currently published as a package. Do you know if I include this script in the pipelines folder if it will be accessible within a pipeline run that depends on the templates? I'll try it out.

I'm starting to think it might just be easier to use MSBuild and write a csproj file into each repo 😢. Let me try that out too.


feat(tabs): add pagination

Adds pagination to tab group so we can add more navigation tabs that will be visible in small screens
TODO

[ ] On arrow tap, scroll left / right
[ ] Move active logic to tabs group
[ ] Remove arrows if tabs fit screen

#626  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


[Bug]: Can not change Document Format of E-Document Service without "Auto-save as you work" feature

Describe the issue
It is not possible to set up E-Document Services other than "Data Exchange" when the "Auto-save as you work" feature is not enabled yet.
The value is reset every time you select a different value, because there's a CurrPage.Update(false) in the validate trigger:
https://github.com/microsoft/ALAppExtensions/blob/07aa6c5540cf21e8bc90dfbace5d9b137055aa0f/Apps/W1/EDocument/app/src/Service/EdocumentService.Page.al#L36-L44
Expected behavior
Possible to change the Document Format.
Steps to reproduce

Open any E-Document Service card without the "Auto-save as you work" feature (Feature Management) enabled.
Try to change the "Document Format" to another value

Additional context
No response
I will provide a fix for a bug

[x] I will provide a fix for a bug

Approved.


Added Basic API Endpoints Support

This PR intends to support basic API's, Once User hosted the Finance Web App, User can access the finance data using API. This will be  useful for mobile app integration.
I'm planning to develop a mobile app, which will have self-hosted URL or IP. Once User added the hosting detail. The mobile App will sync the data from the server.
[+] Main advantage of having mobile app is to Read SMS from directly

Thanks, @ahaneef29 for your efforts.
Actually adding API just for the mobile is redundant because we already have GraphQL API set up and we can the latter for mobile clients as well we just need to update the middleware here.
However, even we have a mobile app, I did some research on reading the SMS directly but it seems not possible on Apple devices due to privacy constraints. I'm still doing some research on how to automate reading the SMS instead of manually adding them to the parser, for now, we can live with the current implementation. Also, planning to integrate https://plaid.com/ or something else but seems now not supported by all banks around the globe.


fix: rlp decode deposit tx

closes #2615
rlp decoding order following the specs

looks correct, do we have some samples for tests?

Added a test that does the following:

decoded tx -> rlp bytes
rlp bytes -> decoded tx

I also had to move the signature up in the match statements as it's not part of deposit transactions


Improve username behavior

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Did it


BUG: to_datetime can return NaTType

import pandas as pd
out = pd.to_datetime("dajsldaljd", errors="coerce")
reveal_type(out)

wrong type returns
note: Revealed type is "pandas._libs.tslibs.timestamps.Timestamp"

Also, mypy on
import pandas as pd
out = pd.to_datetime("dajsldaljd", errors="coerce")
out is pd.NaT

returns
error: Non-overlapping identity check (left operand type: "Timestamp", right operand type: "NaTType")

We made a conscious decision to not have pd.to_datetime() return Union[pd.NaT, Timestamp] because then you'd have to use cast and write this in lots of code:
out = cast(pd.Timestamp, pd.to_datetime("2022-07-05"))

to force out to be pd.Timestamp, which is what you want most of the time.
There's not a way in static typing to test if the timestamp is valid.

@Dr-Irv why not @overload with errors:Literal["coerce"] with a return type [Timestamp, NaTType] and errors:Literal["ignore","raise"]  with return type Timestamp?  This type of overloading would let most users ignore the NaT and the cast while letting those who use errors="coerce" get correct typing.
Reading python/mypy#5090 both convinced me that str and Sequence[str] are the same, and that the role of a typechecker is to be as honest as possible with the underlying language, including undesirable warts.

@Dr-Irv why not @overload with errors:Literal["coerce"] with a return type [Timestamp, NaTType] and errors:Literal["ignore","raise"] with return type Timestamp? This type of overloading would let most users ignore the NaT and the cast while letting those who use errors="coerce" get correct typing.

Reasonable suggestion.  Open to a PR.  Have reopened the issue.

Reading python/mypy#5090 both convinced me that str and Sequence[str] are the same, and that the role of a typechecker is to be as honest as possible with the underlying language, including undesirable warts.

Not sure that is relevant here - I think  you're referring to #82, so comment there on your perspective on this issue.


Update main bower file paths

Last missing piece after renaming .css files :)

Isn't what I've done in the last release ?

You've changed the names everywhere except for the bower.json file.

Roooooo ! My Bad, I merge and release ...

Done in 2.0.16


Introducing relationships for references and provenance

@rartino, @muhrin, @ml-evs, others (sorry, I forgot the GitHub handles, feel free to add them in) and I have put together a proposal to describe the provenance and publication references (closes #27). The proposal relies on JSON API Relationships and introduces a new entry type reference with fields based on BibJSON specification.
To implement filtering by references, the reference list of any entry MUST be duplicated in references property, as we currently have no way to filter by relationships (#96).

Some beautiful stuff here, thanks @merkys! sunglasses

Thanks @CasperWA for approval! :)

Can be merged after comments have been addressed

Current status is expressed in a review comment by @merkys, linked here, rewritten here:

I still have to sort this thing out. BibJSON seems to inherit properties from BibTeX, but the only list of BibTeX properties I was able to find on the spot is this: http://bibtexml.sourceforge.net/btxdoc.pdf (year 1988, so no DOIs and stuff). Will come back to this later.

Hmm, BibJSON does not seem to have a proper specification:

BibJSON is not a complete standard, the aim is to demonstrate the simplicity with which we can represent this data in a JSON object, using namespaces to extend keys as necessary.

(from Jones et al., 2011)
Although BibJSON uses BibTeX fields, BibTeX v0.99b spec does not contain fields url and doi, which we urgently need. I see two options how we can continue:

define the BibJSON as we understand it, including url, doi and other fields (essentially a fork of the BibJSON spec);
choose another standard (suggestions?)

Pinging provenance group: @fawzi, @rartino, @ml-evs.

Ditched BibJSON as it does not seem to have a formal specification. I've placed a note that we support the BibTeX specification + doi and url.

Before merging, could you replace (ONLY in the new examples that you added)
 ```json

with
 ```jsonc

?
(see also #118 and PR #121)


Don't check Chrony PID

If the docker is stopped and the PID is still present chrony will restart with a warning message :
2020-07-07T10:26:48Z chronyd version 3.5 starting (+CMDMON +NTP +REFCLOCK +RTC +PRIVDROP -SCFILTER +SIGND +ASYNCDNS -SECHASH +IPV6 -DEBUG)
2020-07-07T10:26:48Z Fatal error : Another chronyd may already be running (pid=9), check /var/run/chrony/chronyd.pid

Remove PID file at startup

thank-you for your contribution @clementperon! good catch - clearly i don't cycle my containers often enough 😱


(Feature Request) Add support for Force client settings

Hello, it would be really useful to get back the option to use "Force client settings" module with the UI. Or maybe a checkbox for GM to set its settings as world settings if it's easier to do.

There's not a lot I can do on my end, it seems to be just how submenus work, and converting back to non-submenus has problems as well.
However, Force Client Settings has a tool for exactly this reason:

Using this, you can still enable the lock for settings that are hidden in submenus, you just don't get the hint text.


Migrate the deprecated moment.js to date-fns

Migrate away from the deprecated library moment. A good replacement is date-fns which is significantly smaller and built for modern tooling (https://date-fns.org/)

This was done in cf127524df4e96f4d86cb237c182127d0cfeefe9


client: check consistency immediately before waiting

So we don't unconditionally burn 50ms every call to Get()
and List(). Also return the cache mutex RLocked so that we
don't have to lock/unlock/lock checking consistency and then
doing the actual operation.
@trozet @jcaamano @dave-tucker

/retest

I don't know how to trigger a retest on the CI here, the normal command doesn't seem to work.

/lgtm


Update instructions and add configs for remaining CPU apps

Update parallelcluster instructions and add configs for mixbench, mt-gemm, nekrs, osu, quicksilver, and stream.

Updated the PR per your recommendations!


command to list user's rooms

When I used gitter-export-room I had to find the rooms id's using curl:
curl -i -H "Accept: application/json" -H "Authorization: Bearer {{token}}" "https://api.gitter.im/v1/rooms"
So, I think it would be very handy to have a command to show your current rooms and its ids.

LGTM, thanks


Change month in weekview

Is it bug or feature?
If I change days in same month in weekview - just selecting another day
If I change days in different months in weekview - it animate scroll to new month, selecting first day of month - it is not right to animate scroll and autoset day(if disable autoselect not select any day) to first day of month or week(in prev month)
like so:

Will it be solved in 2.0? I make some workaround in code but it so horrible because I don't understand route, as I see there are 3 weeks that changes, but also change month - can somebody help with it? it is major for me

@elsesiy can you help with it?

@EvgenyKondrashkin This is still an issue?

Will check it tomorrow
понедельник, 11 апреля 2016 г. пользователь Jonas-Taha El Sesiy написал:

@EvgenyKondrashkin https://github.com/EvgenyKondrashkin This is still
an issue?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Mozharovsky/CVCalendar/issues/232#issuecomment-208388830

Still here

@elsesiy @Mozharovsky Truly it is major bug, I try to solve it but got horrible solution - I messed up with previous, present and following weeks in CVCalendarWeekContentViewController with initialLoad and insertWeekView. As I see I solve it but create more issues. Can you check this and 243 issues?

@EvgenyKondrashkin So you want to scroll to previous/next month without autoselection of the first right?

No, I want to select day in weekview that contains 2 month without scroll,
like selecting days in any other weekviews. It is missbehavior scroll in
weekview and in code there us very strange data in prev, present and
following week
суббота, 7 мая 2016 г. пользователь Jonas-Taha El Sesiy написал:

@EvgenyKondrashkin https://github.com/EvgenyKondrashkin So you want to
scroll to previous/next month without autoselection of the first right?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Mozharovsky/CVCalendar/issues/232#issuecomment-217531149

@EvgenyKondrashkin No that's not true, the transition between two month is happening so there's gotta be an animated scroll. It's the same with the Apple Calendar, select a specific date and you'll be taken into the month with the respective date selected or the detail view will be presented automatically. I understand that it might be wrong to select the first of the month automatically, but the transition should be fine.

@elsesiy you are not right about transition between two month in weekview - I think that there should be no any transition in weekviews even in changing month. In weekview should be only one transition - user scroll or ask for current day. Like in apple calendar(ad another gif):

there no special autoscroll transition for selecting day in next or previous month - it is userfriendly
but true in monthview there should be transition and animation, but not in weekview.
And yes there is misbehavior in selecting days - should select days that user choose, not first or current day, but it is more global problem - it happens in weekview, monthview


nix-prefetch-git: remove extraneous output from JSON

Motivation for this change
It was breaking the emacs2nix use of this tool.
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Without this change, the JSON output from nix-prefetch-git is
corrupted by the pushd/popd output:
mdorman@aching:~$ nix-prefetch-git --fetch-submodules --url http://repo.or.cz/r/anything-config.git --rev 2d7e0450e13ab04b20f4dff08f32936e78677e58 2>/dev/null
/tmp/git-checkout-tmp-MKaDNZ03/anything-config-2d7e045 /home/mdorman
/home/mdorman
{
  "url": "http://repo.or.cz/r/anything-config.git",
  "rev": "2d7e0450e13ab04b20f4dff08f32936e78677e58",
  "date": "2015-10-19T11:03:43+09:00",
  "sha256": "077gdqd6vhnfryf21mbgs2bdp8k7bdgaiajdw72rq4rlcg4xpjv9",
  "fetchSubmodules": true
}

By redirecting the output of pushd/popd to stderr, we can preserve it in
STDERR (just after 'Switched to a new branch'):
mdorman@aching:~$ ~/src/nixpkgs/pkgs/build-support/fetchgit/nix-prefetch-git --fetch-submodules --url http://repo.or.cz/r/anything-config.git --rev 2d7e0450e13ab04b20f4dff08f32936e78677e58
Initialized empty Git repository in /tmp/git-checkout-tmp-6EK8ZHTC/anything-config-2d7e045/.git/
remote: Counting objects: 30, done.
remote: Compressing objects: 100% (28/28), done.
remote: Total 30 (delta 0), reused 16 (delta 0)
Unpacking objects: 100% (30/30), done.
From http://repo.or.cz/r/anything-config
 * branch            HEAD       -> FETCH_HEAD
Switched to a new branch 'fetchgit'
/tmp/git-checkout-tmp-6EK8ZHTC/anything-config-2d7e045 ~
~
removing `.git'...

git revision is 2d7e0450e13ab04b20f4dff08f32936e78677e58
path is /nix/store/5b3acbmxpwhp7i54zw3kh0zm0s0i50zy-anything-config-2d7e045
git human-readable version is -- none --
Commit date is 2015-10-19 11:03:43 +0900
hash is 077gdqd6vhnfryf21mbgs2bdp8k7bdgaiajdw72rq4rlcg4xpjv9
{
  "url": "http://repo.or.cz/r/anything-config.git",
  "rev": "2d7e0450e13ab04b20f4dff08f32936e78677e58",
  "date": "2015-10-19T11:03:43+09:00",
  "sha256": "077gdqd6vhnfryf21mbgs2bdp8k7bdgaiajdw72rq4rlcg4xpjv9",
  "fetchSubmodules": true
}

But it does not appear in the JSON output on STDOUT:
mdorman@aching:~$ ~/src/nixpkgs/pkgs/build-support/fetchgit/nix-prefetch-git --fetch-submodules --url http://repo.or.cz/r/anything-config.git --rev 2d7e0450e13ab04b20f4dff08f32936e78677e58 2>/dev/null
{
  "url": "http://repo.or.cz/r/anything-config.git",
  "rev": "2d7e0450e13ab04b20f4dff08f32936e78677e58",
  "date": "2015-10-19T11:03:43+09:00",
  "sha256": "077gdqd6vhnfryf21mbgs2bdp8k7bdgaiajdw72rq4rlcg4xpjv9",
  "fetchSubmodules": true
}

I noticed this problem when all non-github git pulls for emacs2nix were
failing, so this is not a theoretical problem.

On the other hand, I doubt it was intentional to print the "pushd" directory in the first place. Maybe just redirect to /dev/null to restore previous behaviour?

Basically, this fixes a regression caused by commit f56ab9e5e4 ("nix-prefetch-git: Include the date in the machine-readable [...]").

Well pushd/popd were only introduced a couple of weeks ago (and, I suspect, only merged in the last few days), so I don't know that there's any "previous behavior" to revert to.
As for whether >/dev/null would be the better thing?  I dunno---there are more instances of 1>&2 than there are of >/dev/null, so I just followed the majority usage, and I tend to err at least a little bit on the side of more diagnostics when things go wrong.

I feel quite confident that's a regression. I fixed it like this: 68205d0adeefa7ed5a3627f5d5c47e7a33b93f86.

@bjornfor I noticed the issue, found the problem, produced a fix---and, to be clear, it was a perfectly reasonable fix, more consistent with the rest of the script than yours---opened a PR, and because you prefer another fix, you push a different commit and relegate me to "Reported by"?
I cannot begin to express how disrespectful and demotivating your actions were.

Ouch, I'm sorry! Sometimes I feel I discuss things too much in PRs and just want to fix things and move on. I just wanted to make a reference to the commit that introduced the change and to completely revert the change. I honestly thought this was the best way.
Again, I'm sorry!

Oh, sorry guys!  I just added the pushd thing yesterday.  I forgot that it prints stuff to stdout; that's so lame.  I just added it so we could use it as an alternative to cd, so I think its output should be directed to /dev/null.

@mdorman Thanks for reporting the bug and all the work you put into identifying and fixing it!
However, @bjornfor's solution is objectively better because there is no reason to be printing those directory names to the standard error pipe.  Part of the Unix philosophy is the Rule of Silence: "Rule of Silence: When a program has nothing surprising to say, it should say nothing."  It was pretty clear what the right solution was and I think it's OK that he just went ahead and fixed it.  Piping to /dev/null is standard practice, and was already done in other places in this script.
What exactly is demotivating for you?  Are you sad that you didn't get your name included as a committer/author in this repository?
Actually, I think you should look at this more positively.  You reported a bug, and within an hour, one of the (presumably unpaid) maintainers of this project took a look at it and fixed it.  That's a sign that nixpkgs is a well-run project, and they care about the issues that you report.


'illegal offset type' whenever I create a resource.

Hello,
First, I would like to thank you for the great package.
Recently, I have the issue 'illegal offset type' whenever I want to create a resource. I think this happened in Laravel Nova v3.8.4 because before that I was able to create a resource fine.
here is my code,
MapMarker::make('Map')
            ->defaultZoom(13)
            ->latitude('latitude')
            ->longitude('longitude'),

is there any temporary solution for this issue?
Thank you.

Hi @khaledBawz, if you can provide the entire stack trace that will help us debug the issue. Thanks! :)

i have same issue 😢

Same issue, the problem happens whenever you submit a form containing the MapMarker component, to be specific
file nova/src/PerformsValidation.php
line 237 return [$item->attribute => $item->name]; throws an ErrorException because $item->attribute returns an array containing the component object's attributes
Laravel v7.25
Nova v3.8.4

@semihkeskindev Thanks for the additional details. I will work on a fix.

Looks like this is a recent change in Nova that broke this. Working on it now.

Opened issue with Nova: https://github.com/laravel/nova-issues/issues/2834


Mention style doesn't work correctly

When mentioned text has an icon at the end of it, the style of normal text will follow to mention style. So any ideas? The screen shoot is attached below

any updates?

Thanks for your feedbacks. I'm working on the next major update and will try to include a fix for this bug.

Thanks 👍🙏
On Fri, 29 Apr 2022, 16:57 David, @.***> wrote:

Thanks for your feedbacks. I'm working on the next major update and will
try to include a fix for this bug.
—
Reply to this email directly, view it on GitHub
https://github.com/dabakovich/react-native-controlled-mentions/issues/66#issuecomment-1113281179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGCPBNQUIHEJWM4PQ46N5JLVHPMEBANCNFSM46W4Q7HQ
.
You are receiving this because you commented.Message ID:
@.***
com>

@tiranavayan alpha v3 is released now. Can you test is the bug still reproducing on new version?
Please, use this branch to get usage info and example.
You can install alpha version using those commands:
yarn add react-native-controlled-mentions@alpha
// or
npm install --save react-native-controlled-mentions@alpha

@tiranavayan can you install updated alpha version and test how the input work with emojis now?
yarn add react-native-controlled-mentions@alpha
// or
npm install --save react-native-controlled-mentions@alpha

I'm getting the same issue. It's look like it is an iOS bug. As workaround, you can use option isInsertSpaceAfterMention. With space after mention it should work as expected.

I use this prop, but after deleting space I'm getting same issue ((

Yeah, that sad. Unfortunately, I can't do with this bug anymore yet.

So sad

any update? am facing same issue

any update @dabakovich , you can test it by typing fast

I think this is an issue related to react-native: https://github.com/facebook/react-native/issues/29890

Facing the same problem, has anyone been able to fix it?


test_supervise and test_supervise_flask_server fail for KeyError when run stand-alone

this is a byproduct of changes made in #522

multiple tests failing:
test_thermostat_api
test_thermostat_common
test_supervise
test_supervise_flask_server


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=630122780
https://github.com/Azure/azure-cli-extensions/commit/f30a59736b3743775793477d97dd7432f1d5f52a

Docs Build status updates of commit ba0fce6:
:clock10: Full build: the average full build time is 7 min(s) 9 sec(s), this is based on the last 327 build(s) for this repository.

Docs Build status updates of commit ba0fce6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update helm installation command

Update the helm installation command and remove the --set option from it.
Add a new helm installation command setting the trivy.ignoreUnfixed configuration as an example of using --set

Fix #1379

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

@chen-keinan

lgtm 🚀  thank you for the contribution


cant get this to build locally but maybe it'll work if I'm using it i…

…n our project

I have forked the repo here: https://github.com/safaiyeh/react-native-cookie-store
We can create issues and PRs there to continue the development of the project.


[SPARK-40192][SQL][ML] Remove redundant groupby

What changes were proposed in this pull request?
Remove redundant groupby invoking in code.
Why are the changes needed?
For Code optimization.
Dataset.agg() has invoked the function groupBy(). We don't need to call groupBy again before executing agg().
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing UT

The pr title should be [SPARK-40192][SQL][ML] Remove redundant groupby

cc @zhengruifeng @LuciferYang @cloud-fan

LGTM for the ML side
But for the SQL side, all changes are in test suites, I am not sure whether those groupby were added intendedly

LGTM for the ML side
But for the SQL side, all changes are in test suites, I am not sure whether those groupby were added intendedly

+1, Agree. Need to be check the groupby is not intentionally added

There is no functional difference, as .agg() is the same as .groupBy().agg(). I'm OK with it but it doesn't optimize anything

Merged to master


[ᚬrc/v0.11.0] add more tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (rc/v0.11.0@70621bd). Click here to learn what that means.
The diff coverage is 91.66%.

@@              Coverage Diff              @@
##             rc/v0.11.0     #150   +/-   ##
=============================================
  Coverage              ?   94.32%
=============================================
  Files                 ?        9
  Lines                 ?      194
  Branches              ?       38
=============================================
  Hits                  ?      183
  Misses                ?       10
  Partials              ?        1

Impacted Files
Coverage Δ

packages/ckb-sdk-rpc/src/resultFormatter.ts
100% <ø> (ø)

packages/ckb-sdk-utils/src/index.ts
100% <100%> (ø)

packages/ckb-sdk-address/src/index.ts
88.88% <88.88%> (ø)

packages/ckb-sdk-utils/src/ecpair.ts
90.9% <90%> (ø)

Continue to review full report at Codecov.

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


New version: Microsoft.WindowsSDK.10.0.22621 version 10.0.22621.755

Pull request has been created with Komac v1.11.0 :rocket:

Resolve #120433

Microsoft Reviewers: Open in CodeFlow

I don't know why it fails

Automatic Validation ended with:

2023-09-30 19:54:27.836 [CLI ] Starting: 'C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Microsoft.WindowsSDK.10.0.226<IP_ADDRESS>21.755\winsdksetup.exe' with arguments '/q /log "C:\Users\Validator\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Microsoft.WindowsSDK.10.0.226<IP_ADDRESS>21.755-2023-09-30-19-54-27.835.log"'
2023-09-30 19:57:09.979 [CLI ] ShellExecute installer failed:<PHONE_NUMBER>

This package installs normally interactively. Is an installer switch missing, possibly to accept something?

@stephengillie
I don't know anything about this package. Is there any engineer of Microsoft could help out?

Feel free to reach out to the team that manages this package.

@stephengillie Do you know who manage this package?

@just-ero

Stale PR; Please update or close
[Policy] Needs-Author-Feedback

I think setup might need to download some additional cab file to install.

The root URL is https://download.microsoft.com/download/7/9/6/7962e9ce-cd69-4574-978c-1202654bd729/windowssdk. Currently, you just saw 1 out of 332 files. Because that setup will continue downloading the file while running the setup. Metadata File
Metadata.txt
Metadata.xlsx
Source file for download that is required to install WindowsSDKSetup.
Download.txt
Download.xlsx

That setup always responds to commands /? which is help interface.

UnattendedFile.txt

Automatic Validation ended with:

Installation failed with exit code -1978335226
2023-09-30 19:57:09.979 [CLI ] ShellExecute installer failed:<PHONE_NUMBER>

Manual Validation ends with the package uninstalling. I'm not sure why.

I am going to suspect installer file logs.

I think the error code is coming out from Scope: user which there is no admin.  So why I requested some change to see if it is work.

It look like there is a little bits weird in that pipelines. MSI UpgradeCode failed

Is winget problem?
2023-10-28 01:11:58.607 [REPO] Examining ARP entries for User | X64
2023-10-28 01:11:58.828 [CLI ] Installer args: /q /log "C:\Users\Brandon Wan\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Microsoft.WindowsSDK.10.0.226<IP_ADDRESS>21.755-2023-10-28-01-11-58.828.log"
2023-10-28 01:11:58.829 [CLI ] Starting: 'C:\Users\BRANDO~1\AppData\Local\Temp\WinGet\Microsoft.WindowsSDK.10.0.226<IP_ADDRESS>21.755\winsdksetup.exe' with arguments '/q /log "C:\Users\Brandon Wan\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir\WinGet-Microsoft.WindowsSDK.10.0.226<IP_ADDRESS>21.755-2023-10-28-01-11-58.828.log"'
2023-10-28 01:24:26.665 [CLI ] ShellExecute installer failed:<PHONE_NUMBER>
2023-10-28 01:24:26.704 [CLI ] Terminating context: 0x8a150006 at D:\a_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\InstallFlow.cpp:1fd

Any suggestions?

It look like there is still having some problem to add this manifest. I will report it to support.microsoft.com  and answers.microsoft.com.

This one was able to add into pipelines. https://download.microsoft.com/download/b/8/5/b85bd06f-491c-4c1c-923e-75ce2fe2378e/windowssdk/winsdksetup.exe There is no error issue detected

It look like there is still having some problem to add this manifest. I will report it to support.microsoft.com  and answers.microsoft.com.

https://answers.microsoft.com/en-us/insider/forum/insider_apps-insider_other/it-look-like-winget-silent-install-failed-with-the/410a01a3-734a-4b13-a085-3943969bf641

Do not close ^^

I haven't found a way to silently install this package successfully.

I saw the .NET Framework 4.8.1 was required in the logs. So I tried with Microsoft.DotNet.Framework.DeveloperPack_4 as a dependency. But the install still failed.

Oh, I see. That dependcies is in catalog.

Do not close ^^ again

/AzurePipelines run


removing outdated Docker image link and easier to copy README commands

https://quay.io/ssro/ec2instances.info

Definitely easier to just use the local Dockerfile. Nice change - thanks!


📝 Correction | "Unfortunately the Android version of Firefox doesn't support managing tabs as containers." points to resolved bug

Description
On https://www.privacytools.io/browsers/#browser you write "Unfortunately the Android version of Firefox doesn't support managing tabs as containers." and link to this bug https://bugzilla.mozilla.org/show_bug.cgi?id=1398097 which has been marked as resolved.

We should update that to point to https://github.com/stoically/temporary-containers/issues/465 instead as it's still not currently possible.


Prepare for wheel building

compare https://github.com/mundialis/actinia-gdi/pull/25.
This PR also adds a Dockerfile which builds this addon, creates a github release and publishes the wheel to github as a release asset.

This PR also changes the Flask Version from Flask==0.12.3 to Flask>=0.12.3. If requirements are installed after actinia_core, this is no problem, as it uses the existing Flask.

Also see https://github.com/mundialis/actinia_satellite_plugin/pull/3

Besides the shebang typo it looks fine to me!

As reviewed, I'll merge now


[Fix #9345] Make Style/AsciiComments allow copyright notice

Fixes #9345.
This PR makes Style/AsciiComments allow copyright notice "©" by default.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

jruby-9.2-ascii_spec CI matrix is failing. I will investigate it.

jruby-9.2-ascii_spec CI matrix is failing. I will investigate it.

CI is successful by workaround and I've opened jruby/jruby#6528 to JRuby team.

CI is successful by workaround and I've opened jruby/jruby#6528 to JRuby team.

Thanks!

Thanks!


Archive partitions to S3

Copies a given partition in CSV format to a local tempfile, then uploads that tempfile to an S3 bucket and protects it with server-side encryption.
https://www.pivotaltracker.com/story/show/143054107
TODOs:

[ ] Partition removal/truncation (we might be able to have Partman do this for us)
[ ] Partition selection (may be moot given the above)
[ ] Gzip files before uploading

@alanpeabody This LGTM 👍
We should have grandstand staging data, so we could throw it up on there.

Oh, I forgot the most important part in my comment.
Once archived we run the partman task drop_partition_time (https://github.com/keithf4/pg_partman/blob/master/doc/pg_partman.md#destruction-functions). This basically takes an "age" to remove from the partition manager and drop.
/cc @Justin-Holmes


DSP-702 Add template for PRs

resolves DSP-702

Why is the test failing? I changed nothing in the code. I added only a github PR template by following this documentation:
https://docs.github.com/en/github/building-a-strong-community/creating-a-pull-request-template-for-your-repository

@kilchenmann I don't know, but I also been getting mysterious timeouts on another PR (#1717), which I can't reproduce locally. I'll try to figure this out.

OK, that seems to have fixed the timeout here, let's merge this one.

Thank you @benjamingeer


instructions unclear, image files in folder all changed to white circles

title. I selected an image folder I wanted to be used, but it didn't work. I just now went into the folder to discover that nearly every image has been replaced with a white circle, with no clear way to restore them. any help would be appreciated.

 Will be closing this now.


accelerometer_timestamp_relative is always '0'

I am not sure if its a bug or not.
Looking at plots of sensors_combined message, I can see that accelerometer_timestamp_relative is always 0.
I am not sure about the logic that related to this var, and if it just log issue.
I can say that magnetometer_timestamp_relative and baro_timestamp_relative are not constant 0.

The accel timestamp is relative to the gyro timestamp. And because they come from the same driver with the same publication rate, it's always 0.

@bkueng  Thanks.


@aws-sdk/core is missing when using typescript-codegen in 0.20

Hi,
I want to ask for clarification whether providing aws.api#service trait is required alongside aws.protocol#restJson1.
The official example for generating typescript clients mentions only @restJson1 trait.
https://smithy.io/2.0/guides/using-code-generation/update-model.html#id4.
Including @service annotation triggers aws-sdk specific codegen to be triggered (including code regarding regions etc), which isn't desireable in case of generating plain http clients (not part of aws sdk).
It seems to me that it wasn't required to add @service annotation in 0.19.
Originally posted by @Petroniuss in https://github.com/smithy-lang/smithy-typescript/issues/1229#issuecomment-2049982875

The aws.protocol#restJson1 protocol trait, or any AWS protocol trait, is designed to usable in isolation from the other AWS traits (e.g. aws.api#service).

It seems to me that it wasn't required to add @service annotation in 0.19.

Are you experiencing codegen errors without the aws.api#service trait applied, or is there a case where codegen fails when using only aws.protocol#restJson1?  If so, please provide a full stack trace so we can investigate.

Awesome! Ff that's the case, I'll close this issue.
Please reopen the issue if there is a problem at codegen time when using aws.protocol#restJson1 in isolation.


installing via npm versions > 3.x.x

Tried this on a couple of machines and getting same error.
λ Tjs-MacBook-Air steam-node → npm install steam --save
<EMAIL_ADDRESS>/Users/tj/Documents/code/projects/steam-node
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
└──<EMAIL_ADDRESS>extraneous

npm WARN EPACKAGEJSON<EMAIL_ADDRESS>No description
npm WARN EPACKAGEJSON<EMAIL_ADDRESS>No repository field.
npm WARN EBUNDLEOVERRIDE Replacing bundled steam > steam-resources with new installed version
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "steam" "--save"
npm ERR! node v5.0.0
npm ERR! npm  v3.4.0
npm ERR! path /Users/tj/Documents/code/projects/steam-node/node_modules/steam/node_modules/steam-resources/node_modules/protobufjs/node_modules/yargs/node_modules/window-size/cli.js
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod '/Users/tj/Documents/code/projects/steam-node/node_modules/steam/node_modules/steam-resources/node_modules/protobufjs/node_modules/yargs/node_modules/window-size/cli.js'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/tj/Documents/code/projects/steam-node/npm-debug.log

Node Version: 5.0
NPM Version: 3.4.0
Tried all basic troubleshooting you can think of. All other modules are installing fine for me currently.

This is an issue with npm. Updating to >3.5.0 should solve it.

Yeah, that did the trick. Thank you for the fast response. Didn't think 3.4.0 was that old. Should we update the README on minimum npm version support?

Looks like one of deps doesn't exist: https://www.npmjs.com/package/steam-resources

Well, i had to run npm install -g npm just because fedora repos are outdated af

steam-resources is a bundled dependency.

@seishun You can safely close this issue. This is no longer a problem with latest npm which is being shipped. Pre-3.5.0 installations are outdated anyway and the chance that someone runs into this is very low as well.

Agreed.


Implement multi-select in left-panel file tree

It would be good to be able to move/copy/delete/etc. multiple files in the tree view at once using combinations such as:
Mouse

shift+left click:  Select all items from the currently focused item to the clicked item
ctrl+left click: Add the clicked item to the selection

Keyboard

shift+↑: Add the item above the focused item to the selection (or reduce the selection)
shift+↓: Add the item below the focused item to the selection (or reduce the selection)
ctrl+↑: Move the focus up without selecting the item
ctrl+↓: Move the focus down without selecting the item
ctrl+space: Add/remove the focused item from the selection

+1 - was just trying to delete multiple files.

Hey I just wanted to do a +1 on this one, this would be a really useful addition!

+1

+1
I don't like having to right click and and doing "Reveal in Explorer" (If that is the correct label and doesn't exist on Linux) and then doing what I want there.

:+1:

:+1: +1

this is important especially when needing to restructure an app by moving multiple files into other folder locations or deleting many files at once. we tend to do this often when in the early stages of web app development.

+1

+1

+1

:+1:

Hi all, prefer :+1: reactions on the original issue comment as that helps us with prioritization.

:+1:

I updated the issue with more detail and additional desired keybindings.

👍

We need this feature

+1

👍

(thumbsup)

+1

👍

👍

+1

Hi all, prefer :+1: reactions on the original issue comment as that helps us with prioritization. If you're on a mobile device you can use reactions by first hitting the "Desktop version" button at the bottom of the page.

+1

I would love to see this : )
It seems very counter intuitive since most all file explorers have ctrl+click for multi selecting as well as shift clicking for selecting a range of files between file A and file B

👍  It's important, and I need. Mutil-select will make vscode be more convenient for developers.

+1 Still needed. Since 1 year now...

+1

+1
I was surprised that is doesn't work.

I just noticed ctrl+left click currently opens the file in the next editor group, we would need to introduce a setting and/or override that.

Don't forget "open selected files" :D

Mostly just waiting for someone to pick it up, it's high on my personal want list so I might pick it up once I'm freed up.

+1

i need this feature

+1

+1

+1

+1

+1

Please prefer :+1: reactions on the original issue comment as that helps us with prioritization and doesn't send notifications to everyone listening to the issue for updates.

ctrl+left click should toggle the selection rather than just select

👍

👍

+1

@Tyriar did this feature made it onto product roadmap ?

+1

+1

1

This is especially annoying since you have watch the confirmation dialog animation (on a Mac at least) every freaking time you delete files one by one.
It would be great if there was no animation, even better, just don't show the confirmation dialog and remove to trash, which has a convenient "Put back" option in case one mistakenly deleted something.

+1 when is this coming? need this too.

+1 please :)

+1

+1

+1

+1

+1

+1

+1

+1 x10000000

+1 this is important feature for me, please.

+1 Need this!

Not support now?? terribel

Is it that hard to implement? Just black mask the file that's selected (like while you have not files opened) and then start working with multiple files selection, is that a really big problem?

hell yeah.. why still not add this feature yet?  so hard?

+1

+1

+1000

+1

Please prefer :+1: reactions on the original issue comment as that helps us with prioritization and doesn't send notifications to everyone listening to the issue for updates.
As an update: it's not that this is particularly hard, it just takes some time to do right and the team is all busy doing other stuff. This is near the top of the list of issues sorted by 👍 reactions now though which probably means it will be picked up in the coming months.

+10086

+1

👍

+1

+1

+1

+1

This issue is 1.5 year old, really, Microsoft, y u no listen?
(╯°□°)╯︵ ┻━┻

+1

Not yet implemented? This should be a priority.

This is an obvious one. Still not delivered? Wooow!

WOW. +1. Really handy feature. Should be implemented. 👍  :)

+1

+1 👍

+1

+1

It's really pretty irritating ;) but massive 👍 for vscode in general, just works

+1

+1

+1

+1

+1

+1. This feature is required before VSCode can be really considered as a serious IDE.

+1 - Seriously this is not a thing!!!! - I love VS CODE until now....

Pretty Please?

+1

+1

+1

I feel this is a no-brainier feature. +1000

+1

+1
I strongly believe that this feature should be there in any IDE.

+1
This is a must have!, this is standard in almost all IDEs.

+1
Please add this feature in to vscode

+1

1

+1

can we get an estimate from the team when this well be added or if this is planned?

It's not planned right now. I'd love to have it, we just need people to free up.
The best way to influence us working on this is by hitting :+1: on the top comment.

+1

is it possible now ?

+1

totally agree on you. I found what you said is very necessary for me in daily work. You spoke it out, great!

Oh, my... I can't believe this...

+1

+1

+1, I really need this feature

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

there is an estimation for this feature?

+1

So... still have no solution???

 I guess I meant it is a small but important feature, don't know about the difficulty in implementing it.

Thanks for clarification. Well, it's certainly not technically trivial, especially if you need support for keyboard controls.
I've just made the mistake to subscribe to this issue to be aware of any progress but I mostly get notifications about people spamming "+1" or complaining that such an "easy feature" is not done yet...

Yeaaah. I don't know what to say to that except https://help.github.com/articles/unsubscribing-from-conversations/

+1

+10086

+1

+1

+1s

Selecting two items should show the "compare" option in the context menu.

+1

+1

+1

+1

Why do everyone vote with a +1? Please please please stop it! GitHub has a feature called reactions since March 2016. Please use that instead of flooding with unnecessary +1's. With every +1 everyone of the 128 subscribers gets a useless notification 🙄

You are right @screendriver, +1

As far as I know feature requests are sort by thumbs up reactions:
https://github.com/Microsoft/vscode/issues?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc
So the +1 posts will not help anyone. The only thing you can really do is a thumbs up reaction for the first post. Just saying...

Plus you (or the maintainer) can sort GitHub issues accordingly the reactions and can see which issue is most important. He can't do that with +1s.

As I said 😉

So the +1 posts will not help anyone. The only thing you can really do is a thumbs up reaction for the first post. Just saying...

Already did. +1 is for rebellion! XD

So... Is it so really unfixable bag??????

I have idea - if you do this like "feature" and will sell it for 1$ )) I'm sure everyone will buy it!!

what a shame

+1

It must be just an Easter egg or something. We must discover how to activate this hidden feature.

+1

+1

+1

+1

Still waiting for this important feature :(

Is this feature still not supported?

+1. I don't understand why it is still not implemented..

just got pissed off by this missing feature, again. Only thing that makes me frustrated about the VS Code :)

Multiple files selection feature doesn't work on my Mac!!! help please. Is this a bug?

@binshu2013 this feature can't work on Mac because this issue is a feature request and the feature is not implemented yet ;)

Martin,
Thank you for your reply. Any idea when this feature will be implemented?
In my opinion, multi files selection is a must-have feature.
-Bin
On Tue, Nov 21, 2017 at 2:33 AM, Martin-Wegner<EMAIL_ADDRESS>wrote:

@binshu2013 https://github.com/binshu2013 this feature can't work on
Mac because this issue is a feature request and the feature is not
implemented yet ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/1023#issuecomment-345953101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFPFeWetHicw8fQelBgVvWxOxdpYGp2_ks5s4orZgaJpZM4GvF60
.

+19260817 +1s

+1

+1

+1

+1

-1
Maybe we should try minuses instead.

🙈
Or a monkey

+1

The intent is to provide developers with a sense of pride and accomplishment for managing files one by one.

+1
I'm shocked this has not been added yet

It's been two years! Congrats to the team🥇  Your persistence to core values (whatever they are) is admirable :)

Any update here? Most of VS Code is designed with the goal of increasing user efficiency and development speed. The lack of a multi-select feature slows down the workflow and seems contradictory to the overall ambition of the editor.
I am crossing my fingers for an imminent update which includes that feature.
Otherwise, the editor is great.
PS: The docs around launch configurations and tasks are slightly messy and they could use some clean-up work. I spent two days trying to understand how to debug transpiled ES6 code. It is possible that I am a bit slow, but it should not be that hard to understand.
Thanks again for a great editor.

@serioth the only update is that it's planned in the 6 month roadmap https://github.com/Microsoft/vscode/wiki/Roadmap

such a long fuss about a simple thing!!!! come on!!!!!! implement it now!!!!!! geez

s this feature still not supported?

why？？？

wtf this is incredible...

Really? This is the first thing you implement in an editor, to make it an editor.

+1

It's Dec 23 2017 now. 750 days passed.

I just started using vscode yesterday and today found myself looking for a way to select multiple files to copy them to a different sub-folder.

As far as I can say from the previous example, this comment will be downvoted.
However, the same story. The first day, first 10 minutes after WebStorm. Started to look for this feature. Found this thread...

As far as I can say from the previous example, this comment will be downvoted.

Everyone who contributed to this issue is being informed that another comment was added. Noone needs +1 posts.
Did you upvote the issue?

Yes.

Added +1 to the main issue description.

Thanks!

zzzzz

+1