1st part of upgrading registerElement to customElement

Replace registerElement with customElement
This is a general improvement, and a prerequisite before any electron upgrade.
I was able to do it for the javascript files. But I am stuck with the coffeescript files as I have described in https://github.com/atom/atom/pull/22011#issuecomment-799753706
Those probably require some bigger overhaul that I am not able to deal with.
This should be ok for merging (maybe tests need some updates?) but if any of the devs could please help out with the other files that would be much appreciated!

@aminya thanks for the hint. Lint caught the useless constructor. No lint errors anymore now

Weirdly the tests say:

Trying it out on a current browser:

Found a stackoverflow: https://stackoverflow.com/questions/43836886/failed-to-construct-customelement-error-when-javascript-file-is-placed-in-head
Apparently attributes (e.g. this.classList) cannot be modified in a constructor.
In this case, some implementations don't enforce that, so that could explain why there's no constructor error.
But maybe it still created htmlunknownelement and that's why the tests are failing.
This affects spec\panel-container-element-spec.js, src\pane-container-element.js, src\pane-element, src\panel-container-element.js, src\text-container-element.js, and src\workspace-element.js
Maybe there should be a constructor2 method, and then a function createCustomElement that automatically calls both

Also there's another thing about createElement
class example1 extends HTMLElement {}
customElements.define('example-one', example1)
document.createElement('example-one')

// But here....
class example2 extends HTMLDivElement {}
customElements.define('example-two', example2, { extends: 'div' })
document.createElement('div', { is: 'example-two' }) // different

So ~line 1352 in spec/text-editor-element-spec.js should be changed
      const parent = document.createElement(
---     'text-editor-component-test-element'
+++     'div', { is: 'text-editor-component-test-element' }
      );

Thanks for all your feedback and research @icecream17
Maybe one of the official Atom developers can propose what they think is the preferred approach forward? @sadick254 @darangi

[ ]

Replace registerElement with customElement
This is a general improvement, and a prerequisite before any electron upgrade.
I was able to do it for the javascript files. But I am stuck with the coffeescript files as I have described in #22011 (comment)
Those probably require some bigger overhaul that I am not able to deal with.
This should be ok for merging (maybe tests need some updates?) but if any of the devs could please help out with the other files that would be much appreciated!

This PR has been surpassed by @sadick254's new patches


Update netty-handler to 4.1.100.Final

About this PR
📦 Updates io.netty:netty-handler from 4.1.81.Final to 4.1.100.Final
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.netty", artifactId = "netty-handler" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.netty", artifactId = "netty-handler" }
}]

labels: library-update, commit-count:1

Superseded by #1591.


user login status

upload php work for user log in status

the code has been uploaded

Yeah I use the code in login page. The username variable is Session['name']


A CMake build system for the C++ script?

I'm not exactly sure how to support that with C++ scripting, but it would be nice since it trivializes adding external libraries.

Discussed with @fwsGonzo
Can add existing libraries to godot-sandbox projects.
Open question how to support with a docker build container.
We can only currently add single-header-only libraries.

Implemented here: https://github.com/libriscv/godot-sandbox/pull/148


forward port TransportExecutor changes from cross-joins

I left out the newBranch/leaveBranch because it is unused and only included
the other changes.

ack


Implement mutant grouping in the @stryker-mutator/typescript-checker

Is your feature request related to a problem? Please describe.
The TypeScript checker is currently very slow since it is checking mutants one by one.
Describe the solution you'd like
Improve the typescript checker performance by checking mutants in groups.
Describe alternatives you've considered
We've investigated a lot of alternatives in @danny12321 and @ThierryRietveld 's graduation assignments. Grouping seems to be the best solution.
Additional context
All investigative work is already done, see #3366 for inspiration.

Closed with #3900 🚀


AutoFac runtime error when attempting to initialize TweetInvi in Xamarin Android

This one has me stumped.  When I build using Debug configuration, everything runs fine.  But when I build using Release configuration, the build goes fine but then the app crashes as soon as I try to initialize the TweetInvi library.  I wasn't able to find anything helpful on Google/SO/etc.
Here's the adb stack dump:

--------- beginning of crash
08-19 00:34:44.475  9382  9382 E AndroidRuntime: FATAL EXCEPTION: main
08-19 00:34:44.475  9382  9382 E AndroidRuntime: Process: com.companyname.BirdieMobileApp, PID: 9382
08-19 00:34:44.475  9382  9382 E AndroidRuntime: android.runtime.JavaProxyThrowable: System.TypeInitializationException: The type initializer for 'Tweetinvi.TweetinviConfig' threw an exception. ---> Autofac.Core.DependencyResolutionException: An error occurred during the activation of a particular registration. See the inner exception for details. Registration: Activator = TweetinviSettingsAccessor (ReflectionActivator), Services = [Tweetinvi.Core.ITweetinviSettingsAccessor], Lifetime = Autofac.Core.Lifetime.RootScopeLifetime, Sharing = Shared, Ownership = OwnedByLifetimeScope ---> No constructors on type 'Tweetinvi.Core.TweetinviSettingsAccessor' can be found with the constructor finder 'Autofac.Core.Activators.Reflection.DefaultConstructorFinder'. (See inner exception for details.) ---> Autofac.Core.DependencyResolutionException: No constructors on type 'Tweetinvi.Core.TweetinviSettingsAccessor' can be found with the constructor finder 'Autofac.Core.Activators.Reflection.DefaultConstructorFinder'.
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Activators.Reflection.ReflectionActivator.ActivateInstance (Autofac.IComponentContext context, System.Collections.Generic.IEnumerable1[T] parameters) [0x00051] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.InstanceLookup.Activate (System.Collections.Generic.IEnumerable1[T] parameters) [0x0001a] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:    --- End of inner exception stack trace ---
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.InstanceLookup.Activate (System.Collections.Generic.IEnumerable1[T] parameters) [0x0004d] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.InstanceLookup.<Execute>b__5_0 () [0x00007] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Lifetime.LifetimeScope.GetOrCreateAndShare (System.Guid id, System.Func1[TResult] creator) [0x0002f] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.InstanceLookup.Execute () [0x00051] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.ResolveOperation.GetOrCreateInstance (Autofac.Core.ISharingLifetimeScope currentOperationScope, Autofac.Core.IComponentRegistration registration, System.Collections.Generic.IEnumerable1[T] parameters) [0x0005f] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.ResolveOperation.ResolveComponent (Autofac.Core.IComponentRegistration registration, System.Collections.Generic.IEnumerable1[T] parameters) [0x00000] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Resolving.ResolveOperation.Execute (Autofac.Core.IComponentRegistration registration, System.Collections.Generic.IEnumerable1[T] parameters) [0x00013] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.Core.Lifetime.LifetimeScope.ResolveComponent (Autofac.Core.IComponentRegistration registration, System.Collections.Generic.IEnumerable1[T] parameters) [0x00041] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.ResolutionExtensions.TryResolveService (Autofac.IComponentContext context, Autofac.Core.Service service, System.Collections.Generic.IEnumerable1[T] parameters, System.Object& instance) [0x00023] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.ResolutionExtensions.ResolveService (Autofac.IComponentContext context, Autofac.Core.Service service, System.Collections.Generic.IEnumerable1[T] parameters) [0x00030] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.ResolutionExtensions.Resolve (Autofac.IComponentContext context, System.Type serviceType, System.Collections.Generic.IEnumerable1[T] parameters) [0x00007] in <773915280c53497dbfb0899f8d1f1426>:0  08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Autofac.ResolutionExtensions.Resolve[TService] (Autofac.IComponentContext context, System.Collections.Generic.IEnumerable1[T] parameters) [0x00000] in <773915280c53497dbfb0899f8d1f1426>:0
08-19 00:34:44.475  9382  9382 E AndroidRuntime:   at Tweetinvi.Injectinvi.AutofacThreadContainer.Resolve[T] (Tweetinvi.Core.Injectinvi.IConstructorNamedParameter[] parameters) [0x0002b] in <93de26df2bc84d029febbba83cd5d75f>:0
08-19 00:34:44.476  9382  9382 E AndroidRuntime:   at Tweetinvi.Injectinvi.AutofacContainer.Resolve[T] (Tweetinvi.Core.Injectinvi.IConstructorNamedParameter[] parameters) [0x00005] in <93de26df2bc84d029febbba83cd5d75f>:0
08-19 00:34:44.476  9382  9382 E AndroidRuntime:   at Tweetinvi.TweetinviContainer.Resolve[T] () [0x00020] in <93de26df2bc84d029febbba83cd5d75f>:0
08-19 00:34:44.476  9382  9382 E AndroidRuntime:   at Tweetinvi.TweetinviConfig..cctor () [0x00000] in <93de26df2bc84d029febbba83cd5d75f>:0
08-19 00:34:44.476  9382  9382 E AndroidRuntime:    --- End of inner exception stack trace ---

I don't know why Release is causing it to throw up like this.  I tried reinstalling TweetInvi in the NuGet console but that had no effect.
Any ideas?  Releasing the app in Debug configuration is obviously not an option.
Thanks!

Here's the exact line where it's failing:
TweetinviConfig.CurrentThreadSettings.TweetMode = TweetMode.Extended;
I've tried everything I can think of in terms of fiddling with the app settings but TweetInvi just won't work in Xamarin unless I run it in Debug configuration.  Does anybody know why this is happening?

Hi @sirkris , I have never encountered such a problem but will be looking into it.
Though please note that at the moment there is a more urgent issue (#850)  that I would like to solve as top 1 priority.

I have updated the version of Tweetinvi to 4.0.2 so that it becomes compatible with .NET Framework 4.7.2.
Please note that the it came with a cost which is the fact that for the moment Tweetinvi is no longer a StrongName package.
I will be glad to hear feedback on the subject before I close the issue.

5.0 has been tested with Xamarin Android.


Properly handle readOnly and writeOnly properties

Describe the solution you'd like
As described at original issue and pull request openapi-typescript-codegen does not handle readOnly and writeOnly properties of openapi schemas.
With these properties, it is easier to model properties not for modifiable models.
OpenAPI example:
type: object
properties:
  id:
    # Returned by GET, not used in POST/PUT/PATCH
    type: integer
    readOnly: true
  username:
    type: string
  password:
    # Used in POST/PUT/PATCH, not returned by GET
    type: string
    writeOnly: true

Expected example types (not necessary to be generated separately, as @tiholic suggested in his PR):
UserGetResponse: { id: number, username?: string }
UserPostRequest: { id: number, username?, password?: string }
UserPutRequest: { id: number, username?, password?: string }
UserPatchRequest: { id: number, username?, password?: string }

@tajnymag just want to clarify the expected example types. Aren't they meant to be like this?
UserGetResponse: { id?: number, username?: string }
UserPostRequest: { username?: string, password?: string }
UserPutRequest: { username?: string, password?: string }
UserPatchRequest: { username?: string, password?: string }

@tajnymag just want to clarify the expected example types. Aren't they meant to be like this?
UserGetResponse: { id?: number, username?: string }
UserPostRequest: { username?: string, password?: string }
UserPutRequest: { username?: string, password?: string }
UserPatchRequest: { username?: string, password?: string }

Yeah, they should.  I pushed some changes yesterday that extract types for data objects when using --useOptions, more cool stuff to follow.
P.S. I believe --useOptions should be the default setting, and plan to support mostly that flag in the future

@tajnymag a little update: this will work only with --useOptions, so please make sure you're using that flag. Out of interest, are you able to share your config?
I explored this a bit and the conclusion is that we will need to rework types quite a bit, it's not a quick fix. That pull request you linked doesn't implement it properly either and I'd rather wait until it works as expected than go through multiple iterations here

@mrlubos Unfortunately I don't have anything to share. As I really needed multipart forms supported I've used the official openapi generator. Similarly, as the readOnly and writeOnly properties weren't well supported by any generator I had found, I manually created schemas for requests and responses.

@tajnymag are you not currently using this package then?

@tajnymag are you not currently using this package then?

Unfortunately I am not. I would like to though. The official generators are lacking in so many other ways.

Why don't you use https://github.com/drwpow/openapi-typescript @tajnymag?

Why don't you use https://github.com/drwpow/openapi-typescript @tajnymag?

I did't want to hardcode any urls into the client.

Unfortunately I am not. I would like to though. The official generators are lacking in so many other ways.

Don't worry, we'll get there. Keep listing any issues you're seeing elsewhere and we'll figure it out!

Hi @mrlubos

That pull request you linked doesn't implement it properly either and I'd rather wait until it works as expected than go through multiple iterations here

I'm the author of the pull request @tajnymag mentioned. I am planning on migrating to this package from openapi-typescript-codegen to keep things simple moving forward.
When I originally worked on this PR, my primary focus was ensuring the following use case:
1.	A schema (e.g., User -> id, name) has readOnly properties (User.id) used in the OpenAPI spec.
2.	When this schema is used for POST/PUT service payloads, OmitReadOnly excludes the readOnly properties from the requestBody to efficiently allow re-use of a singe schema for both Read/Write APIs.
I acknowledge there is one particular case I didn’t account for - handling deeply nested schemas with readOnly properties. For example, this schema wouldn't work with that implementation: Category -> (id [readonly],  name, SubCategory -> id [readonly], name)


Issue with the webview_flutter when getting the current address in certain pages.

Currently working with webview_flutter and getting the current address with the OnPageStated() method and updating it in a text field. It gets the new address when navigating in certain webpages such as Google but when navigating in other webpages such as the mobile versions of Facebook or YouTube it doesn't update the new addresses, only once I call the reload method in the WebView the address in the textfield gets updated.  I know that probably it is because these webpages really doesn't charge another page when updating the urls (Probably there's some Javascript framework behind). Is this something normal to happen with this plugin? Does anyone know a work around for this? Or maybe this can be some kind of new feature that can be added in the plugin.

class WebViewWidget extends StatelessWidget {
  final HomeController homeController;

  const WebViewWidget({Key key, this.homeController}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return WebView(
      initialUrl: homeController.url,
      javascriptMode: JavascriptMode.unrestricted,
      onPageStarted: (String address) {
        homeController.onPageStarted(address);
      },
      onPageFinished: (_) {
        SystemChannels.textInput.invokeMethod('TextInput.hide');
      },
      onWebViewCreated: (WebViewController controller) {
        homeController.setWebViewController(controller);
      },
    );
  }
}

void onPageStarted(String address) {
_addressBarController.text = text;
}

Hi @gabxap
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

Hi @gabxap
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

Hi @gabxap
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample along with the actual vs expected results?
Thank you

Flutter doctor -v

[✓] Flutter (Channel stable, 1.22.6, on macOS 11.1 20C69 darwin-x64, locale en-PE)
    • Flutter version 1.22.6 at /Users/jumpbotron/programming/installations/flutter
    • Framework revision 9b2d32b605 (6 days ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0-rc3)
    • Android SDK at /Users/jumpbotron/Library/Android/sdk
    • Platform android-30, build-tools 29.0.0-rc3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

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

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

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

[✓] Connected device (1 available)
    • Mi A3 (mobile) • ba2b9fba2ef7 • android-arm64 • Android 10 (API 29)

• No issues found!

Code sample:
WebView
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:retro/controllers/browser_controller.dart';
import 'package:webview_flutter/webview_flutter.dart';

class WebViewWidget extends StatelessWidget {
  final BrowserController browserController;

  const WebViewWidget({Key key, this.browserController}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return WebView(
      initialUrl: browserController.url,
      javascriptMode: JavascriptMode.unrestricted,
      onPageStarted: (String address) {
        browserController.onPageStarted(address);
      },
      onPageFinished: (_) {
        SystemChannels.textInput.invokeMethod('TextInput.hide');
      },
      onWebViewCreated: (WebViewController controller) {
        browserController.setWebViewController(controller);
      },
    );
  }
}

Business Logic
class BrowserController extends GetxController {
  WebViewController _controller;
  TextEditingController _addressBarController;
  String _url = 'https://www.google.com/';

  WebViewController get controller => _controller;
  TextEditingController get addressbarcontroller => _addressBarController;
  String get url => _url;

  @override
  void onInit() {
    super.onInit();
    _addressBarController = new TextEditingController(text: "://home");
  }

  void setWebViewController(WebViewController someController) {
    _controller = someController;
  }

  void onPageStarted(String address) {
    _addressBarController.text = address;
    _url = address;
  }

TextField

class SearchBar extends StatelessWidget{
@override
      Widget build(BuildContext context) {
          return TextField(
                controller: browserController.addressbarcontroller,
          );
     }
}

Expected result:
A normal browser such as Brave updates the current url at any time. Including Youtube urls when clicking a video

Actual result:
My browser does not update the urls when navigating in certain webpages such as YouTube (clicking a video )but in other yes such as searching in Google yes. I am using the OnPageStarted() method of the WebView plugin which has a parameter called String adress.

Hi @gabxap
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample along with the actual vs expected results?
Thank you

Flutter doctor -v

[✓] Flutter (Channel stable, 1.22.6, on macOS 11.1 20C69 darwin-x64, locale en-PE)
    • Flutter version 1.22.6 at /Users/jumpbotron/programming/installations/flutter
    • Framework revision 9b2d32b605 (6 days ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0-rc3)
    • Android SDK at /Users/jumpbotron/Library/Android/sdk
    • Platform android-30, build-tools 29.0.0-rc3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

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

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

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

[✓] Connected device (1 available)
    • Mi A3 (mobile) • ba2b9fba2ef7 • android-arm64 • Android 10 (API 29)

• No issues found!

Code sample:
WebView
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:retro/controllers/browser_controller.dart';
import 'package:webview_flutter/webview_flutter.dart';

class WebViewWidget extends StatelessWidget {
  final BrowserController browserController;

  const WebViewWidget({Key key, this.browserController}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return WebView(
      initialUrl: browserController.url,
      javascriptMode: JavascriptMode.unrestricted,
      onPageStarted: (String address) {
        browserController.onPageStarted(address);
      },
      onPageFinished: (_) {
        SystemChannels.textInput.invokeMethod('TextInput.hide');
      },
      onWebViewCreated: (WebViewController controller) {
        browserController.setWebViewController(controller);
      },
    );
  }
}

Business Logic
class BrowserController extends GetxController {
  WebViewController _controller;
  TextEditingController _addressBarController;
  String _url = 'https://www.google.com/';

  WebViewController get controller => _controller;
  TextEditingController get addressbarcontroller => _addressBarController;
  String get url => _url;

  @override
  void onInit() {
    super.onInit();
    _addressBarController = new TextEditingController(text: "://home");
  }

  void setWebViewController(WebViewController someController) {
    _controller = someController;
  }

  void onPageStarted(String address) {
    _addressBarController.text = address;
    _url = address;
  }

TextField

class SearchBar extends StatelessWidget{
@override
      Widget build(BuildContext context) {
          return TextField(
                controller: browserController.addressbarcontroller,
          );
     }
}

Expected result:
A normal browser such as Brave updates the current url at any time. Including Youtube urls when clicking a video

Actual result:
My browser does not update the urls when navigating in certain webpages such as YouTube (clicking a video )but in other yes such as searching in Google yes. I am using the OnPageStarted() method of the WebView plugin which has a parameter called String adress.

Hi @gabxap
Can you please provide a code sample that does not use any third-party plugin/package? Also, please try to provide a complete code that I can just copy/paste and run it.
Thank you

Hi @gabxap
Can you please provide a code sample that does not use any third-party plugin/package? Also, please try to provide a complete code that I can just copy/paste and run it.
Thank you

Hi @gabxap
Can you please provide a code sample that does not use any third-party plugin/package? Also, please try to provide a complete code that I can just copy/paste and run it.
Thank you

Hey don't worry I already solved it constantly asking the route with another method.


profile speed vs. CatGT

preprocessing will run slower than CatGT (pure C++), see how much slower for CatGT supported functions ((tshift, CAR, filtering)

Kind of unrelated, but it would be cool to see if SI preprocessing can be jitted

On further consideration, there is no point thinking about this now. The aim of this project is to wrap SI, in which special attention and a lot of consideration already paid to performance.


Why there're many pages of wiki are mission(I can't find)?

Mean in title.

Sorry, that word in title "Mission" mean "Can't find"

What pages are missing from what wiki exactly? Minecraft wiki? The wiki of Traincarts itself isn't really maintained. https://minecraft.gamepedia.com/Custom_servers/Bukkit/TrainCarts

...
ok
(I mean github)

Yeah the wiki hosted here should probably be removed, or just point to the minecraft wiki. Only the custom mods minecraft wiki page is actively updated at this time. There were some plans to do something with the github-hosted wiki, but nothing much came of it.


Text Case Converter and Inspector - Incorrect behaviour for text in scripts that don't support casing

Description
When applying casing options to text written in a script that doesn't support casing, Text Case Converter and Inspector appear to modify the contents of the string incorrectly. It should either leave the source string as it is and if it also depends on spacing, only the spacing rules should be applied.
Steps To Reproduce
No response
Expected behavior
I've currently tried this for Tamil language text. Casing options that don't depend on spacing (Sentence case, Title case, UPPERCASE, lowercase, aLtErNaTiNg cAsE and InVerSe Case) seem to output the same string which is the correct behaviour, however other options seem to change the string incorrectly. This may be due to incorrect handling of Unicode strings.
Input string: தமிழ் மொழி
PascalCase: தமழமழ (Expected: தமிழ்மொழி)
snake_case: தம_ழ_ம_ழ (Expected: தமிழ்_மொழி)
CONSTANT_CASE: தம_ழ_ம_ழ (Expected: தமிழ்_மொழி, same as snake_case)
kebab-case: தம-ழ-ம-ழ (Expected: தமிழ்-மொழி)
COBOL-CASE: தம-ழ-ம-ழ (Expected: தமிழ்-மொழி, same as kebab-case)
Train-Case: தம-ழ-ம-ழ (Expected: தமிழ்-மொழி, same as kebab-case)
Another option would be to simply error out or keep it the same and show a message that the casing option couldn't be applied to the string. I'll try strings in more scripts and let you know.
DevToys Version
<IP_ADDRESS>
Relevant Assets/Logs
Not required.

#549


upgraded to django1.8

are we ready to go to 1.8? We've moved python-opencivicdata-django.

I ran opencivicdata.org locally and had no issues.

Sweet. LGTM!


Remove six module due to the full switch to the Py3

Hello @blokhin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file aiida_crystal/calculations/common.py:

Line 46:80: E501 line too long (99 > 79 characters)
Line 47:80: E501 line too long (101 > 79 characters)
Line 48:80: E501 line too long (85 > 79 characters)

In the file aiida_crystal/calculations/properties.py:

Line 33:80: E501 line too long (99 > 79 characters)
Line 34:80: E501 line too long (101 > 79 characters)
Line 35:80: E501 line too long (96 > 79 characters)

@ansobolev ping


Secret Doors

Is there any way to add secret doors? I tried with the tools from foundry, but that does not seem to work..

Right now there isn't, beyond manually modifying the walls and doors yourself (and then not modifying the dungeon any more, since it will regenerate walls and doors).
But, if this is something you think would be useful, I can add it to the TODO list. There's a similar open request for the ability to draw a dotted-line wall that would block movement but not sight.

if this is something you think would be useful, I can add it to the TODO list.

That would be awesome!

Being able to set the door default to secret would be fantastic. I use Foundry to play IRL, and I don't like having the door icons visible on the map. Especially since they show through the FoW I use. Secret doors solves this problem.

Cool. I'll plan to work on this feature next. Maybe a new tool button ("Add Secret Door"), and it's visible as a normal wall for players, and as a wall with an S through it for GMs? Sort of like:

Awesome. I realize that I might be a very special edge case, but for my use, I want it to look like a normal door as a map visual, but be registered as a secret door in the Foundry wall settings, as I just don't want the extra icon that Foundry shows for doors.
If the settings would let me choose the combination of Foundry door/Dungeon Draw door, that would be perfect.
I don't have the slightest clue about how much that complicates programming though :p

Ok, so maybe two config settings for a secret door: how it appears to GMs and how it appears to players... and you could choose between s-secret door, normal-looking door, or blank wall for each. And then for the auto-generated Foundry lighting wall, Is Door? will be set to "Secret" for all cases.

Sounds perfect for my needs at least :D
The biggest issue for me, with the current setup, is that if I change anything at all, every door is reset to a normal door.
And thank you for being so responsive here. The speed of development for this module has been quite impressive.

Implemented in upcoming 0.12.0.

![Screen Shot 2021-11-30 at 4 12 21 PM](https://user-images.githubusercontent.com/189172/144136473-18de19e5-bcb0-4585-bae0-10a0e4d04e08.png

)

Uuuuh it's working great! Thank you very much! :)


Add new category Scinece

from API  https://opentdb.com/api_config.php

Before adding a new category, we need to generalize some models more especially the Quiz model


BOOST_ASSERT as KIWAKU_ASSERT no longer works

Now that you have started using iostream constructs in invocations of KIWAKU_ASSERT this fails to compile when BOOST_ASSERT is used
https://user-images.githubusercontent.com/340735/222702614-350a6e2f-67aa-438f-97a5-dc62f82c2084.png
The first fix is to remove BOOST_ASSERT support and let/us other provide our own KIWAKU_ASSERT implementation. The core issue for us is the use of iostreams (yuck :D) and also a different 'abort' point (more then one place to place a break point and/or stacktrace printouts for CI environments).

For ergonomics purpsoe, we need rich assert message that carries the runtime values that may have failed, hence the changes.
We can easily replicate the BOOST_ASSERT_HANDLER protocol for KIWAKU_ASSERT no problem.

OK, plan can be:

make ASSERT takes the string of << infos into a tuple
pass the tuple to a assert_handler that do either stream stuff or w/e you want when overridden

This way we keep the rich info where we want and if you need a degraded support of assert, you can safely ignore them and not need any ostream thing.

took a stab at this - can you please take a look at https://github.com/jfalcou/kiwaku/actions/runs/4373496578/jobs/7651745697 ?
is this some unhappy kiwaku<->std::tuple interop issue?

Yes. THink you need ot use kumi::tuple.

Fix merged


Upmerge 01.03.2024

Mcuboot upmerge 01.03.2024

With the latest push the history of the upmerge branch was rewritten to align with the history cleanup on main.


Setup project

In this pull request, we have made the following changes:

[x] Setup Linters
[x] Setup devise
[x] Setup RSpec and Capybara

STATUS: APPROVED ✅

Hi🖐 @lorenzoworx,

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit: Congratulations! 🎉

Highlights
✅  setup well-implemented
✅ ... Green linters

Optional suggestions

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.

Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me(@joskalenda) in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


fix(watch): remove instance unmounted short circuit in getter of watchEffect

Ref: https://github.com/vuejs/core/pull/7330#issuecomment-1872461341
This is solution 1, relatively high risk.
Solution 2: #9949

/ecosystem-ci run

/ecosystem-ci run

I took a deeper look and this is already guarded by this line here https://github.com/vuejs/core/blob/main/packages/runtime-core/src/apiWatch.ts#L333 so it should be safe to remove.


[Feature Request] el-popover to show after content is entirely fetched

Existing Component
Yes
Component Name
el-popover
Description
If the content of the popover has to be fetched e.g. from one API popover show first an empty tooltip and load the content as soon as it has been fetched.
The position of the popover is not readjusted after the content has been loaded but remains based on the empty content from the beginning.
Repro link:
https://jsfiddle.net/dLwcsdj1/
And how it is expected to be after the content has loaded (from the position):
https://jsfiddle.net/5tjcxj69/

You can call updatePopper after data is fetched: https://jsfiddle.net/leopoldthecuber/dLwcsdj1/5/ . updatePopper is an inner method, but is stable enough to be used in production.

That's it! Thanks for the quick answer!

I have met something similar today. And I debugged, then found
.
If query changes, the updatePopper is called. Just assume nextTick is called via microTask, and if we send a xhr, and won't update at last. And the first update is useless maybe?
And I tried to call updatePopper manually and fixed.
I just found the issue and post my experience. But I don't think it's an elegant way. Can we have another better method to do this or enhance the api?
Anyway, thx for ur great work on Element-ui.


Simplify circuit code

Simplifies circuit code a bit more, removing some of the more "fancy" code in favor of simple loops
Updates client dependency
Removes unused core dependency

@ytham -- CI failing?

closing, please use https://github.com/axiom-crypto/axiom-quickstart/pull/14 instead


[BUG]  chat bubble 3 time the same

Who notified us about the bug?
@gal-agmon
Describe the bug
when I am looking at a municipality in the Philippines (lowest admin area) I see the same chat bubble 3 time
Production:

Demo:

To Reproduce
Steps to reproduce the behavior:

Go to 'philippine flodd'
Click on 'province' -' municipality'
See the same chat bubble 3 time

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@gulfaraz I'm pretty sure this is a symptom if the bug that we recently solved already? If so, @gal-agmon this should already be fixed on ibf-test, not on ibf-demo yet.

@jannisvisser  this is test

Ok, clear @gal-agmon, let's tag @BlaiseSelvan then to prioritize.

reproduced locally (with 2 speech bubbles, not 3)


Assignment count mismatch failure on go get

go get github.com/golang/lint/golint
github.com/golang/lint
../../go/src/github.com/golang/lint/lint.go:1432: assignment count mismatch: 2 = 3

Solution: go get -u golang.org/x/tools
Duplicates 99 97 93 92

This is also mentioned in the "guidelines for contributing" that was prominently displayed when you were filing this issue. Unfortunately I can't change that link text to make it clear that you should read it before filing issues too.

Would it be useful to add this to the install instructions?
On Tue Jan 27 2015 at 12:42:22 PM David Symonds<EMAIL_ADDRESS>wrote:

This is also mentioned in the "guidelines for contributing" that was
prominently displayed when you were filing this issue. Unfortunately I
can't change that link text to make it clear that you should read it before
filing issues too.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/lint/issues/103#issuecomment-71723850.

I don't know. It's not a problem if it's the first time you're getting the golang.org/x/tools repo (since you'll get the latest version), and I'm loathe to suggest always updating it since you could run into problems in the other direction (other packages depending on some older version). Right now I think people installing this from source just need to know what they're doing somewhat.


Update http4s-circe, http4s-dsl to 0.23.19

About this PR
📦 Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.23.11 to 0.23.19
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #468.


Add option to set the prisma-fmt binary path in vscode options

I am using prisma on NixOS which has no precompiled binaries available
so I think the easiest solution is to add an option to set the prisma-fmt binary path in vscode options, this would allow me to compile prisma-fmt from source and provide it's path to the extension

Would an ENV variable also do the job?

yes that would also work


fix: base href does not work #11401 #10194 #2134

Fix #11401
Fix #10194
Fix #11643
Fix #7767
Related #2134
Motivation
Support BASE_HREF on the document https://argoproj.github.io/argo-workflows/argo-server/

Modifications

Add handle base href for static file and test

Verification
make test

More
As I check this issue, I see the lib to embed static files into Go binary has been deprecated (https://github.com/bouk/staticfiles). We should replace it.

@agilgur5 what "partial rootpath support" mean?

With just base href, the proxy in front should be rewriting these subpaths
=> Can you give me an example?

The Argo Server still sees all requests from /
=> I tried to keep this compatible with other functions, ex: readiness check

@agilgur5 what "partial rootpath support" mean?

As in a partial implementation of #7767.

=> Can you give me an example?

#3080 has many examples of ingress config with rewrites. This one from the top is pretty concise and uses the common NGINX IGC

@agilgur5 I'm sorry, but I do not understand. Let's me know if you want any info here

@agilgur5 I'm sorry, but I do not understand.

Let me try two other explanations:

<base> is an HTML element. Per the MDN docs, the href attribute only affects HTML links. It does not change anything on the back-end.

With a Single Page App (SPA) on the UI, JS links also need to be changed. Hence the JS I linked before.

/argotest/main.js should never occur. The proxy in front of the server is supposed to rewrite that to /main.js. The server only sees /main.js and never sees /argotest/main.js because there is a proxy in front.

Without rootpath support (i.e. #3080), the back-end basically has no knowledge of the rewrite. Rootpath support is similar to implementing the rewrite within Argo.

I don't think this is actionable as a "fix" to base href. base href is working correctly.

Can you make a test? I don't think it work like the document and I also made some test before. That's why many people report this in some issues. Also you can read the fix code to know the reason

@agilgur5 with ingress rewrite, I think this is not a issue. I tried without ingress rewrite for this

Can you make a test?

The back-end is working correctly. Per https://github.com/argoproj/argo-workflows/issues/11401#issuecomment-1685877020, the Server logs are correct, correctly detecting the environment variable. Per https://github.com/argoproj/argo-workflows/issues/11643#issuecomment-1685871836, the Server correctly sets <base href="/argo/">.
That is the only impact to the back-end that base href has. All other impacts are front-end facing.

That's why many people report this in some issues

Per above, both users' own issues show that it is working correctly.
Per https://github.com/argoproj/argo-workflows/issues/10194#issuecomment-1685866941, I think many users are misinterpreting the base href feature for rootpath, but those are two separate features.
Base href is for the front-end, rootpath is for the back-end.

@agilgur5 with ingress rewrite, I think this is not a issue. I tried without ingress rewrite for this

Yes exactly! Base href is intended to be used in conjunction to an ingress rewrite

Yes exactly! Base href is intended to be used in conjunction to an ingress rewrite

@agilgur5 Maybe ingress rewrite added to the document after have this issue. This fix can run without ingress rewrite. The document updated after this issue https://github.com/argoproj/argo-workflows/issues/3080

This fix can run without ingress rewrite.

That's just a separate feature entirely. That is part of rootpath support #7767. And totally feel free to work on #7767!
Base href and rootpath are just two separate (but related) features. Note that the opening comment of #7767 uses both --basehref and --rootpath together.
"Base href" specifically is a front-end only feature designed to be used with ingress rewrites. It is making the front-end aware of a back-end configuration. The back-end configuration can be done either with an ingress rewrite or with --rootpath.

@agilgur5
I think it this project basehref and rootpath is the same. You will see that they are set to same value.
Also please note on https://github.com/argoproj/argo-workflows/issues/7767, purpose of the the issue is to fix running argo-workflows on a subpath requires rewrites on the reverse proxy . The suggest about rootpath is a way to archive the purpose. I think this PR also fix #7767

I think in this project basehref and rootpath is the same.

No they are separate, it's just that rootpath was never implemented.
I also think it is good for Argo projects to be consistent where possible (and where it makes sense), so if Argo CD has those as two separate flags, Argo Workflows can implement that consistently.
Moreover, "base href" refers to a Web standard, literally the HTML base element and its href attribute. I don't think we should redefine a Web standard. Things like that tend to cause more confusion rather than less. We should try to teach users about the existing standards and how they work -- which is why those were good docs changes. We might be able to further improve those docs.
--rootpath also would require more changes. Ingress rewrites change the external API route as well (e.g. my-domain.com/api -> my-domain.com/argo/api). So we'd need to do the same thing for --rootpath. https://github.com/argoproj/argo-workflows/issues/7767#issuecomment-1031207499 shows several of the changes in CD required (like the GRPC gateway).

@agilgur5 Could you give me a real life example for rootpath and basehref with different values for running Argo Workflow?

my-domain.com/argo/apiwith prefix /argo/ -> should work without rewrite on additional controller for #7767

I think my PR do not handle this

@agilgur5 Maybe we also need rootpath to handle all cases. let me try to figure it out

@agilgur5 Could you give me a real life example for rootpath and basehref with different values for running Argo Workflow?

The main one is an ingress rewrite. For instance, if I have my-cluster.com as my main cluster domain and all services as subpaths, then I could have my-cluster.com/argo-cd/, my-cluster.com/argo-workflows/, my-cluster.com/argo-events/, my-cluster.com/prometheus/, etc.
If all those services did not support rootpath, it would be easier to configure ingress rewrites for all of them in the same way -- only the subpath would be different between them. If they did all support rootpath, they may all also require different configuration (e.g. a configmap vs a flag vs an env var, etc), so rewrites may be easier/more consistent as well.
I can think of some other wacky use-cases, but that's the only one I can think of off the top of my head that is a "real life example" that I and others have definitely used before.

I think my PR do not handle this

@agilgur5 Maybe we also need rootpath to handle all cases. let me try to figure it out

Yea that's what I meant when I said that full rootpath support to handle all cases would require a larger PR.
Feel free to take that on if you want!


[RFC] Rename Bardic Inspiration identifier to align with community modules

The D&D Beyond importer module uses a more precise name for the bardic inspiration scale than that which is currently used by the SRD classes available with the dnd5e system. Automation modules are therefore likely to follow suit, and some have already done so (e.g. Chris's Premades).
Flip the SRD identifier to better match developing community expectations.
See e.g.:

https://github.com/MrPrimate/ddb-importer/blob/e7fcbfec18000b671519928e0e2c51dd614df296/src/parser/classes/index.js#L52
https://github.com/chrisk123999/chris-premades/blob/master/packs/cpr-class-features.db (CTRL+F<EMAIL_ADDRESS>

Disclaimer: haven't looked at what migration/how migrations are handled. Happy to do the work if this change is palatable.

This seems to touch the class item only, but not the features that use the scale value. Those would have to be adjusted as well, in which case there is a consideration for what would happen for existing actors that have the class, but not all the features, as well as those that use the SRD bard class in conjunction with their own bard subclass item.
When imported from the SRD compendium or from user's personal compendiums, each user would have to manually change the scale value to match one or the other. The alternative is to continuously migrate the name of the scale value (inspiration to bardic-inspiration) for all classes and feature items, in which case users will be fully unable to use their own (and shorter) name for the scale value.
There are probably nuances to a possible migration method I overlook here, but might it actually be more straightforward for importers and premade items modules like the ones mentioned to adjust and align with the SRD, given a user base that more often continuously updates their actors' items either manually or automatically?
I don't see what's best here, but those are my personal thoughts on the matter.

Disclaimer: haven't looked at what migration/how migrations are handled. Happy to do the work if this change is palatable.

I'm not even sure what a migration would look like here. Already-existing Actors would have Advancements configured with the old identifier and would continue to work fine. Newly-created Actors would get the new identifier when they dragged the Bard class from the SRD compendium. It would probably only be a problem if there are external modules or macros that are looking for a specific identifier, but those would run into trouble regardless since identifiers are user-editable and can be whatever the user wants, ultimately.
That said, unless there are severe technical challenges preventing them from doing so, we would generally encourage modules to align with the reference implementation where it exists. I don't see why this particular situation would be any different, so I'm tentatively marking this PR as won't change.

I'm not even sure what a migration would look like here. Already-existing Actors would have Advancements configured with the old identifier and would continue to work fine.

Actually, I just realised this would not be the case. An Actor that levels up will pull in an Item configured with the new identifier from the SRD compendium as part of their Advancement, while all their existing Items use the old identifier, creating an issue.
This would be a pretty complex migration to target all class and subclass Items across all Actors. So it makes even less sense to make this change and place a migration burden on all users of the dnd5e system for the sake of modules that they might not even use.

Reasonable


[SPARK-29118][ML] Avoid redundant computation in GMM.transform

What changes were proposed in this pull request?
avoid double computation by obtaining the prediction (double) from its probabilty prediction(vector)
Why are the changes needed?
it avoid predict twice
Does this PR introduce any user-facing change?
no
How was this patch tested?
existing tests

Test build #110769 has started for PR 25815 at commit f2dd5e8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15906/
Test PASSed.

Test build #110772 has started for PR 25815 at commit 1179d63.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15907/
Test PASSed.

Test build #110769 has finished for PR 25815 at commit f2dd5e8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110769/
Test PASSed.

Test build #110772 has finished for PR 25815 at commit 1179d63.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110772/
Test PASSed.

Merged to master

@srowen Thanks for reviewing!


GeoDjango form fields do not work on Django 1.11

Hi,
To illustrate, I've modified the tests to test a GeoDjango form field. The resulting tests run fine on Django 1.10, but they fail on 1.11 with TypeError: type object got multiple values for keyword argument 'geom_type'. Any idea on how to fix that could be useful, as I feel a bit lost.
======================================================================                                                                             [86/3511]
ERROR: test_bound_class (bootstrap3.tests.FormTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/tests.py", line 365, in test_bound_class
    res = render_template_with_form('{% bootstrap_form form %}', {'form': form})
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/tests.py", line 139, in render_template_with_form
    return render_template_with_bootstrap(text, context)
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/tests.py", line 128, in render_template_with_bootstrap
    return render_template("{% load bootstrap3 %}" + text, context)
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/tests.py", line 119, in render_template
    return template.render(context)
  File "/home/anthony/nosync/development/django/django/template/backends/django.py", line 66, in render
    return self.template.render(context)
  File "/home/anthony/nosync/development/django/django/template/base.py", line 207, in render
    return self._render(context)
  File "/home/anthony/nosync/development/django/django/test/utils.py", line 107, in instrumented_test_render
    return self.nodelist.render(context)
  File "/home/anthony/nosync/development/django/django/template/base.py", line 990, in render
    bit = node.render_annotated(context)
  File "/home/anthony/nosync/development/django/django/template/base.py", line 957, in render_annotated
    return self.render(context)
  File "/home/anthony/nosync/development/django/django/template/library.py", line 203, in render
    output = self.func(*resolved_args, **resolved_kwargs)
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/templatetags/bootstrap3.py", line 330, in bootstrap_form
    return render_form(*args, **kwargs)
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/forms.py", line 45, in render_form
    return renderer_cls(form, **kwargs).render()
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/renderers.py", line 87, in render
    return mark_safe(self._render())
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/renderers.py", line 227, in _render
    return self.render_errors() + self.render_fields()
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/renderers.py", line 193, in render_fields
    bound_css_class=self.bound_css_class,
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/forms.py", line 61, in render_field
    return renderer_cls(field, **kwargs).render()
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/renderers.py", line 87, in render
    return mark_safe(self._render())
  File "/home/anthony/nosync/development/django-bootstrap3/bootstrap3/renderers.py", line 545, in _render
    html = self.field.as_widget(attrs=self.widget.attrs)
  File "/home/anthony/nosync/development/django/django/forms/boundfield.py", line 120, in as_widget
    **kwargs
  File "/home/anthony/nosync/development/django/django/forms/widgets.py", line 220, in render
    context = self.get_context(name, value, attrs)
  File "/home/anthony/nosync/development/django/django/contrib/gis/forms/widgets.py", line 74, in get_context
    **attrs
TypeError: type object got multiple values for keyword argument 'geom_type'

It looks like you may have found a bug that is not detected by the current tests. Could you submit a Pull Request with a failing test?

Seems fixed in Django 1.11.1 :)

Indeed, on 1.11.1 I no longer have the problem.


Validate crashtrace and coredump with java app

Resolves #1333

I'm merging after a convo with @iapaddler.  We have more work to do to understand all ins and outs of java w.r.t. signal handling, but the code changes here are significant improvements which we want in release/1.3.  It's not the end; we'll continue to improve in this area, but want to get the benefit of what has been completed up to this point.


Decoupled the layers tests and the sequential tests.

Summary
Currently we have 2 types of models (Sequential, Model) and 25+ types of layers.
In the tests, we test each type of layer in each type of model.
But Sequential derives from Model, so in fact, if layers work in Model they're expected to work in Sequential. We need other types of tests for Sequential. Most of them already exist in keras/tests/keras/test_sequential_model.py, I just added a few which look like what we tested before.
The effect of this PR is that it halves the running time of the layer tests in all backends (see the logs).
For some reason, the theano tests are not faster with python 3.6 which is very strange (caching maybe?).

TF 2.7 : 12m17s -> 10m28s
TF 3.6 : 11m33s -> 9m25s
TH 2.7 : 10m20s -> 10m32s
TH 3.6 : 12m11s -> 10m28s
CNTK 2.7 : 6m13s -> 6m36s
CNTK 3.6 : 6m23s -> 6m11s

Related Issues
#8990
PR Overview
Quite some code was moved around in test_utils.py, but very few has changed. I wanted to be able to reuse the functions in test_sequential_model.py.

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thank you for the review. I did quite a lot of suppositions concerning what should be tested and what should not. I think I understand better what you want. Allow me to open a new PR which will be way simpler and closer (I think) to what you want.

I opened #11115 as a simpler version of this PR.


Design an admin interface for adding tasks and permissions

Starting this ticket to focus on the admin interface. As of now it is:

Once we move the details of the task #63 to the main view, we can dedicate this section for adding new projects and editing details of existing projects from #158, and specific users could see another tab for user permissions from #159.
cc @ajithranka

Done.


Totals and sub-totals

Hi Alan,
This is a great utility and  would be nice if we are able to get the totals and sub-totals based on our definition. Is there any workaround or trick by which we can get that?

There's a PR is for support a table footer, but there's some bug so I still wait the creator to fix this issue, actually, I think there's no any good workaround to solve this issue, but I can support a simple table footer in recently. Anyway, you can describe your requirements detail, I'll consider to add them in near future.

This would indeed be a great feature.
Could be a footer or a custom component that can be provided.
The idea would be to have access to the rows currently shown (with filtering/paging/searching/sorting taken into account) so that we could add relevant information for those rows e.g. a total

Could be a footer or a custom component that can be provided.

ok, I will

The idea would be to have access to the rows currently shown (with filtering/paging/searching/sorting taken into account) so that we could add relevant information for those rows e.g. a total.

yap, it should be provided

EDIT: It might already be possible (or easy to add ^^) in v3.0.0 with custom components. I still need to check this new version.

I prefer to support this feature on v3.0.0, but will step by step, I've no many time to work on it, so it really need some times.
Finally, PR is welcome :)

whats a PR?

I mean PR is welcome.

Hi I have done the logic for Grand-Total. Please have a look and let me know what you think.


Ability to export and import all repos currently installed

Summary / 概要
The ability to export and import all repos currently present in alcom's config. Preferably to some human readable format (i.e. JSON) but that's not a necessity. Said export of all repos could then be put into version control, and in case one of the parties collaborating on the project is missing some repos - they can pass ALCOM that file to import any missing repos they might have.
Purpose / 目的
I collaborate on some VRC projects, I also sometimes work from different computers. Git solves most of my issues, but sometimes there's new repos that I add that I immediately start using on one of the machines and that means the other machines will fail to resolve the related VPM packages because they lack the repos. It would be nice to keep those repos in sync by the means of importing an export from the machine that added that repo to the project.
Additional context / 補足
No response

There already is export / import repository list implemented in #1209 (issue #643).
Is this lacks feature?

Oh! My ALCOM hasn't notified me of the update, so I didn't have the issue. If it exports/imports the repos altogether with the headers, I'm going to close this issue. Sorry.

Okay it appears to not export headers. I'm going to open a new issue then. Thank you for clarifying.

I opened #1459 to make it more clear instead of reusing this FR.


Changes to DynamicJsonObject

This will help with automapping libraries and unit testing

Is there any argument against these changes? I'm using them in a custom build, but it would be super if I could just rely on the official released DLLs rather than maintaining my own fork/DLL.


WIP: use async_snapshot instead of raw API in GC

What is changed and how it works?

Issue Number: Close #13319
What's Changed:

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
Please add a release note.

Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with None.
If this PR will be picked to release branch, then a release note is probably required.

/test

/test

What is changed and how it works?

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

How much is the performance regression?

/test

/test

/merge


msfs-gltf export tool  not showing in export options

Current Behavior
msfs-gltf export tool  not showing in export options after i had to reinstall windows, i never had any problems before
Expected Behavior
for the tool to show up in my options
Steps To Reproduce

Environment
- OS:windows11
- Blender:3.3.19
- glTF-Blender-IO-MSFS:

Anything else?
No response

@WhazoPremier it has NEVER shown in the import / export prompt.
The exporter works through its own tab in the 3D window. Press N and look on the right to the panels


nb multi-select: cannot select between cells

Testing #235047
tried doing chtrl + shift + l but it doesn't get variables in other cells
https://github.com/user-attachments/assets/41b31eb6-dba9-4ac0-98a2-8a8cc0c13f5e

jk fixed with proper settings


add support for mapped facet labels

I flattened the configuration keys. I used "type" and "labelPath" as the configuration keys for now, but we could use a longer, more descriptive names, such as "valueType" or "dataApiValueType".
The changes are simple without appCustomText support for dynamic facet labels. Otherwise, a lot more changes are required.
Tested locally.
I will add the configuration to AP json file after the code change is deployed.

Coverage remained the same at 82.871% when pulling 8968e80d45957c994e1a19d4c2b52a71dcebe6c7 on CAB-3586 into e9e496ee1564032a04466e3c243c23f3340e24e1 on develop.


"0 chapters" Novel Updates Source

Source information
Novel Updates
Steps to reproduce

Open Novel Updates source
Tap on any novel available/listed on the source extension
The novel information page is open

Expected behavior
The information page showing the available chapters matching the website version
Actual behavior
The information page showing "0 chapters" instead of the available chapters on the website version
LNReader version
1.1.18
Android version
Android 11
Other details
I already checked the website if the novels have chapters and they do, so it should be available on the app.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.18.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Login is now a requirement for Novel Updates
Use webview to login, then exit out the source and reenter and novels should work just fine

Thank you !! It works !!

when i login and update it works but it doesn't fetch the covers, they go blank. is it possibile to fix that? it's probably due to a different view of the page when you're logged in (not sure tho).


How to add to the following list

_documentList.document = streamSnapshot.data.documents.toList();
Im fetching the documents from the firestore. How do i add a new element to the above list ?
I tried adding, however i'm getting the following error
Unhandled Exception: type '_InternalLinkedHashMap<String, dynamic>' is not a subtype of type 'DocumentSnapshot' of 'element'

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .


Checking the number of times a method has been called

Hi @bobalicious! Love the project!
Is there any way we can tell how many times a method has been called?
Some use cases for this feature:

we're not worried about what parameters have been passed to a method, just that it has been called x times.
asserting that some methods are called, and ignoring other calls
ignoring the order methods are called in, but asserting they have been called

Sorry - no idea how I missed this, and it's probably way too late for you.
All you can do at the moment is 'Spy' this after the call.
For example:
  Integer numberOfCalls = controller.countOf( 'methodToCheck' )

meaning you can:
Assert.areEqual( 4, controller.countOf( 'methodToCheck' ), '"methodToCheck" is called 4 times );


Minor corrections on the main page

Some small minor corrections and simplifications on the main page.

Coverage remained the same at 90.545% when pulling 05ef0550d7fab34d063fbcc5cd0f8de72924ea30 on magikstm:minor-corrections into af9d05217a4986aa63a5744018316ee6a56fafa7 on tarent:master.


Always reset IV for CBC mode on cipher context reinitialization

This is necessary to keep compatibility with 1.1.1 implementation
of the CBC mode ciphers.
Fixes #14704
Checklist

[x] tests are added or updated

CFB mode was mentioned in the bug report. Is that covered here?

Good catch! Actually both CFB and OFB modes need this as well.

One wonders if we should support this IV reset for all mode?
Or do we want it to go away quietly?

IIRC these are the only modes for which the "IV" changes -- the other modes don't need a reset because it would be a noop.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master. Thank you for the review!


Don't dockerize when run requirements install

Usually you do "sls requirements install" for local testing, but if you have "dockerizePip: true" on serverless.yml it will download the linux libraries. This is a problem if you want to make a "sls invoke local" on another environment. I think is best to use the docker container only when "sls deploy"

I suggest using a virtualenv and normal pip installation for local development like you would an ordinary(non-serverless) python project.


Add role columns

Add ROLE column to apps, pods and tasks list
Add ENFORCE_ROLE column to group list
JIRA issues:
DCOS_OSS-5380

@mesosphere-ci run linux integration tests

@ArmandGrillet Do you have any idea why the ITs fail? It doesn't really seem related to this PR, but the ITs are failing for a while now...

Yes it's https://jira.mesosphere.com/browse/DCOS-57734

The CI is fixed 🎉.

@mesosphere-ci run linux integration tests

tests/integrations/test_marathon_pod.py::test_pod_list FAILED
This test needs to be updated.

@mesosphere-ci run linux integration tests

@mesosphere-ci run linux integration tests

@ArmandGrillet Who is landing this PR? I don't have write access to the repo.


custatevec for aarch64/ppc64le requires glibc 2.27+

See https://github.com/conda-forge/cuquantum-feedstock/pull/47#issuecomment-1700751650. Once the conda-forge infrastructure is ready, we should restore these two lines
https://github.com/conda-forge/cuquantum-feedstock/blob/ce8944d672469f8aef72cc7f6952f07df4be9909/recipe/meta.yaml#L99-L102
Right now they are commented out because we need to consume them from within the CI (to build cuquantum-python).

Note: When the infra is ready, we should switch this line
https://github.com/conda-forge/cuquantum-feedstock/blob/d21a50c51af96049b43d82971bda690b979a7af8/conda-forge.yml#L20
to use native_and_emulated to turn on QEMU at the test stage.


Bridge discovery approaches

Hello,
I have been using your lib since several days, and I must say I'm very pleased with the result. Thank you so much for your work on it.
The only thing I have trouble with is bridge discovery on the network.
Philipps provides a nice guidance as how to discover a Hue bridge:

Use UPNP
Use N-UPNP (uses https://discovery.meethue.com)
Use mDNS
Use network scan

Looking at the source code of HttpBridgeLocator, I clearly see that only the approach 2 (NUPNP) is supported by this lib.
When looking in the issues of this repo, I came across some issues talking about UPNP discovery (#3 and #151) and referencing the SSDPBridgeLocator class.
However, it seems it was deprecated in 3.8.1 without a single explanation on why. So now I am quite confused.
My questions are:

Why did you deprecate SSDPBridgeLocator? Was it not working? Should we use something else?
Do you plan to support more way of discovering the bridge? Especially for non-internet environment?

Thank you for your time.

The SSDPBridgeLocator used to work, but only on the .Net 4.5 framework.
It was deprecated because this library moved to netstandard. I've tried to get it to work on .netstandard, but without success. With some changes the code compiled, but it never found any bridges anymore. So because I couldn't get it to work, it was removed.
If somebody creates a PR with a working UPNP solution, I'd be happy to include it in the library again., but I'm not planning to work on it myself.
There are also some other libraries that have more knowledge about this subject, for example:

https://github.com/Yortw/RSSDP
https://github.com/kakone/SSDP

Maybe it's easier to do the bridge discovery using those libraries, or ask their owners how to implement bridge discovery.
If you have working code using one of those libraries it would also be helpful to share it here.

Thank you for your reply. I'll try to look into it and eventually make a PR.
Would you accept PR regarding other bridge discoveries? Like mDNS or network scan?
How would you prefer these new locating approaches to be defined? In other class? Project?

Yes, a PR with bridge discovery using whatever method is welcome :)
The bridge discovery should implement the 'IBridgeLocator' interface:
https://github.com/Q42/Q42.HueApi/blob/master/src/Q42.HueApi/Interfaces/IBridgeLocator.cs
And the code should be in a class library with at least netstandard2.0 support.
If no additional NuGet package references are needed, it can be included in the main Q42.HueApi class library. But if the bridge discovery uses another external library, it's best to create a new project for example Q42.HueApi.BridgeDiscovery.

Alright. I'll keep you posted 😸

When the last PR lands, I would love to have a Nuget release please 😺
Then I'll share here an example of an implementation of a robust bridge discovery using all of these locators.

Yes, I'm also waiting for this to land on NuGet. :D
With the current .13 package, was something subsequently fixed that may explain this issue:
System.AggregateException: One or more errors occurred. (Only one usage of each socket address (protocol/network address/port) is normally permitted.)
---> System.Net.Sockets.SocketException (10048): Only one usage of each socket address (protocol/network address/port) is normally permitted.
at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
at System.Net.Sockets.Socket.DoBind(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.Sockets.Socket.Bind(EndPoint localEP)
at Q42.HueApi.SsdpBridgeLocator.LocateBridgesAsync(TimeSpan timeout)
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification)    at System.Threading.Tasks.Task1.get_Result()
at HueDream.Hue.HueBridge.FindBridges() in D:\DEV\HueDream\Hue\HueBridge.cs:line 158
I'm calling reuseaddress on all of my socket instances, and I see you're setting it in the SsdpBridgeLocator I'm calling at line 158 in my code, so I'm guessing that I'm just outdated yet.

@d8ahazard thanks for reporting this issue.
After investigating it, I found a way to fix and improve the locators.
@michielpost I'll submit a new PR after the MDNS one lands.

Version 3.14.0 has been submitted to NuGet.org and should be there shortly. Thanks for all the PRs!

Here is a gist with some approaches I came up with:
https://gist.github.com/Indigo744/fd55456077450bce1382cf1f957b6b59
It is inspired from the Philips documentation flowcharts:

N-UPNP, MDNS and SSDP are run in parallel for better performance.

@Indigo744 do you think it would be a good idea to include your HueBridgeDiscovery examples in the library?

If you want, I have no problem with that 😺

Added the HueBridgeDiscvoery to Q42.HueApi: https://github.com/Q42/Q42.HueApi/pull/208


Example is broken

I just run it and try to crop immediately:
- (instancetype)initWithCroppingStyle:(TOCropViewCroppingStyle)style image:(UIImage *)image
{
    NSParameterAssert(image);

It seems you just call the following in the incorrect order:
UIGraphicsGetImageFromCurrentImageContext
UIGraphicsEndImageContext
UIGraphicsBeginImageContextWithOptions

But "begin image context" should be before other calls. The order is broken because you call it with dispatch_async


Add push notifications channel

Description & motivation
The GA4 team has added the 'Mobile Push Notifications' channel group (see https://support.google.com/analytics/answer/9756891). This is an important channel for news outlets and other publishers that use push notifications.
I have renamed the channel from 'Mobile Push Notifications' to just 'Push Notifications' because 'push' is a common medium value for browser push notification services too.
This PR would close issue #151 with the channel name change.
Checklist

[x] I have verified that these changes work locally
[na] I have updated the README.md (if applicable)
[x] I have added tests & descriptions to my models (and macros if applicable)
[x] I have run dbt test and python -m pytest . to validate existing tests

The regex matches the rules in the docs (push$ ensures that push is only matched at the end of the string). As for the name, I had doubts and explained the reasons in issue #151. I have no problem with the required change, just wanted to point there because there's a comment in favour of the rename.
I should have set the PR as a draft before having more feedback on #151 to avoid duplicate discussions, sorry about that!

@dgitis yea I think he's all set on the regex using push$. I don't have a really strong opinion on the default channel grouping name but I think if we need a tie breaker, we should go with what Google recommends. @gblazq do you mind making that 1 change? Then I think we can merge.

Sorry, I missed the push$.
I also didn't see the issue where you had discussed Push Notification versus Mobile Push Notification. I wouldn't approve a deviation from GA4 without a good reason, but it sounds like you have a decent reason for the change.
I'll withdraw my change request.


Da loop dynamic #2

This PR builds upon the previous da looping PR. The other PR contains the operational components to run data assimilation in a block test format the same way qlats were handled. In this new PR we are replacing the blocks with a start and end date with a new block size. The code then can derive the block sizing of da files and build the same blocks that were manually entered in the yaml in an automatic fashion. This will streamline large testing of much larger time spans with looping. Mostly operates within .io file to run the function def build_da_date_range(data_assimilation_parameters): in order to achieve this goal.
Should be integrated after da_looping PR. Need to add preservation of block input approach AND dynamic block building approach for da input files.

[ ] add warm state capability
At some level, we'll need to do more in here to handle the warm states. For instance, the lastobs_df returned here is going to need to be specified in the input file, and we need to verify that the function is doing what we expect. And the q0 warm-state will need a corresponding lastobs value passed out of the compute process and made available to hand back to the next portion of the looped execution.

-[ ] ensure functional wrapping
All the algorithmic details in the main_v03 function need to be wrapped in a sub-function or be one-liners in order to be compatible with eventual use with the asynchronous I/O capability.


WIP Publish flow between metrics and alerts

This is a WIP PR to complete the publishing filtering defined between metrics and alerting.
It needs a merge of https://github.com/hawkular/hawkular-metrics/pull/591 and releases from metrics and alerts.
Basically it adds additional ISPN configuration and tests into hawkular-services.
It should be tested using this specific setup for srcdeps:
mvn clean install -Pitest
UPDATE: Pointing srcdeps to upstream of metrics/alerts, no need to use forked repos to test this PR.
Waiting to release of metrics/alets to bump srcdeps to release versions and final merge.

Also a self-reminder.
Once this one is merged the publishing changes should be propagated at hawkular-client-ruby and ManageIQ.

@lucasponce Upgrade to Alerting 1.1.3.Final has been released!

@jpkrohling This is ready for merging now.
metrics and alerting srcdeps versions have been bumped with released ones.
CI passed.
/cc @stefannegrea @jshaughn

LGTM


Tokens across notebooks and readme fixes

Addresses #44
Includes fix for the error in clear cache

Changes are deployed on JH beta and verified parallel execution of notebooks works.


Model matrix not getting applied to tileset correctly

This bug goes back to at least 1.42.
The modelMatrix property seems to have some weird interaction with the ContextCapture tileset posted here. The tileset uses the CESIUM_RTC extension but is in local coordinates.
Default view at center of the Earth:

With model matrix applied

Debugging Sandcastle demo
The good thing is when upgrading the tileset with 3d-tiles-tools (links to the 2.0-tools branch) the tileset is correct. Also needs to be running on a branch that combines https://github.com/AnalyticalGraphicsInc/cesium/pull/7035 and https://github.com/AnalyticalGraphicsInc/cesium/pull/6805. Hopefully it is easy to track down.

Debugging Sandcastle demo

The reason for the bad positions in the second image is CESIUM_RTC applies its translation after the model matrix. Saw the same problem in here: https://github.com/AnalyticalGraphicsInc/cesium/pull/6755#issuecomment-425143807.
The summary here is CESIUM_RTC doesn't work well with model matrices that have rotation. Upgrading the tileset with 3d-tiles-tools, which changes CESIUM_RTC to RTC_CENTER, is the best fix.

Same issue in this forum thread, retiling with Cesium ion fixes it.

I can not upgrade my model , the problem as above

The reason for the bad positions in the second image is CESIUM_RTC applies its translation after the model matrix. Saw the same problem in here: #6755 (comment).
The summary here is CESIUM_RTC doesn't work well with model matrices that have rotation. Upgrading the tileset with 3d-tiles-tools, which changes CESIUM_RTC to RTC_CENTER, is the best fix.

I can not resolve it , the 3d-tiles-tools is not working properly

@engineerhe just to narrow down the issue here, can you confirm if tiling your source data with Cesium ion produces a tileset that works as expected?

@OmarShehata,Thanks for your patient answer,I followed the steps below to solve the problem.
Firstly,i use obj23dtiles tool tiling my data and generate 3dtiles.
Secondly , i use 3d-tiles-tools to upgrade tileset ,but it  always reports an error which Binary gLTF version is not 1.
so,i want to know besides upgrading the tileset or retiling the tileset ,is there any other way to solve the problom?

@OmarShehata,i have solved it . Thanks！

@engineerhe can you describe what the solution was?

Sorry, I didn't see the news until now.The solution is very sample , you just switch obj23dtiles tool into branch 2.0.

Sorry, I didn't see the message until now.The solution is very sample , you just switch obj23dtiles tool into branch 2.0.
obj23dtiles the address  https://github.com/engineerhe/objTo3d-tiles??


SQLRowCount: number of affected rows for insert/update/etc

AffectedRows seems to work. Can this now be closed?

Nope. The value of AffecteedRows can't be retrieved by the ODBC application yet.


Virtual service no route match when simple host is used

Bug Description
Hi,
I defined a simple dns name(foo) and it's resolved only internaly in my network / dns server. It resolves to istio-ingressgateway IP.
When I call http://foo/bar/v1/health  it should redirect to the bar service inside my k8s, but I receive the no route match message in ingress gateway log.
Defined Gateway:
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: foo
  namespace: istio-system
spec:
  selector:
    istio: ingressgateway
  servers:
  - hosts:
    - '*'
    port:
      name: http
      number: 80
      protocol: HTTP

Defined VistualService:
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: bar
spec:
  gateways:
  - istio-system/foo
  hosts:
  - 'foo'
  http:
  - match:
      gateways:
      - istio-system/foo
      uri:
        prefix: /bar/
    rewrite:
      uri: /
    route:
    - destination:
        host: bar.default.svc.cluster.local
        port:
          number: 80
      weight: 100

If I change the host in the VirtualService to '*' it works
Version
❯ istioctl version
client version: 1.14.1
control plane version: 1.14.5
data plane version: 1.14.5 (210 proxies), 1.14.3 (184 proxies), 1.14.1 (48 proxies), 1.14.4 (73 proxies), 1.15.3 (5 proxies)

~
❯ kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-12T10:57:26Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.15+IKS", GitCommit:"e982eaaac65ba290ce43154f935e895ee21640c1", GitTreeState:"clean", BuildDate:"2022-09-21T19:55:23Z", GoVersion:"go1.16.15", Compiler:"gc", Platform:"linux/amd64"}

Additional Information
No response

Does it work when you add the -HHost:foo to your curl call (or adding the header to the client you use)?

Hi @ymesika , thanks for the comment
I identified that the VirtualService treats simple hosts as services within the same namespace where the VirtualService is located, that is, it understands that the host FQDN is 'foo.default.svc.cluster.local'. I will close this bug and open an improvement discussion.


Feat/google tag manager

:tada: This PR is included in version 1.188.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Handle exception when the bio_text is too long

I didn't realise that the original Twitter bio is appended to the text provided via bio_text, so I ended up having too much text. That produced the error:
2021-04-29 08:33:06,479 pleroma_bot ERROR: Exception occurred
Traceback (most recent call last):
  File "/home/federico/git/federico/mastodon/lib64/python3.9/site-packages/pleroma_bot/cli.py", line 435, in main
    user.update_pleroma()
  File "/home/federico/git/federico/mastodon/lib64/python3.9/site-packages/pleroma_bot/_pleroma.py", line 243, in update_pleroma
    response.raise_for_status()
  File "/home/federico/git/federico/mastodon/lib64/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 422 Client Error: Unprocessable Entity for url: https://mastodon.technology/api/v1/accounts/update_credentials

(Oddly, the update seemed to go through, or at least the profile images were updated at some point.)
Verified by shortening the text and seeing the error went away.
The error is not particularly useful. At a minimum it could be a good idea to remind the user to check the variables which might make the request fail, perhaps?

Hi!
You raise a good point, I changed the output of the exception for error code 422 when updating the profile so it's a little more readable an user-friendly when it comes to identifying what caused it.
82c776b802d8305d118b89c30e3338029e1b554f


Future direction: Non-copyable Span

Given that Span should be ended exactly once, and should not be shared between subprocesses or preserved after termination, it seems a potential client of the recently accepted SE-0390. With Swift 5.9+ we can mark Span as non-copyable, and only allow the operation to “borrow” the span. span.end() will consume the span once. We can also add span.end() in deinit to automatically terminate the span.
Note that this is unfortunately very breaking. Haven’t thought about if there’s a way to land this safely in the 1.x period.

Yeah it would be nice but I'm a bit worried about just doing it for 5.9+ for example...
Definitely worth keeping in mind so let's keep this issue open


Unable to get good bind path format / docker: Error response from daemon

I've been beating on this for 3 days and been through all sorts of forum issues and posts and cannot resolve. I'm trying to package numpy in a function, individually building requirements (I have multiple functions with multiple requirements that I'd like to keep separate).
Environment:
Windows 10 Home
Docker Toolbox for Windows:
 Version:       18.03.0-ce
 API version:   1.37
 Go version:    go1.9.4
 Git commit:    0520e24302
 Built: Fri Mar 23 08:31:36 2018
 OS/Arch:       windows/amd64
 Experimental:  false
 Orchestrator:  swarm

Server: Docker Engine - Community
 Engine:
  Version:      18.09.0
  API version:  1.39 (minimum version 1.12)
  Go version:   go1.10.4
  Git commit:   4d60db4
  Built:        Wed Nov  7 00:52:55 2018
  OS/Arch:      linux/amd64
  Experimental: false

serverless version 6.4.1
serverless-python-requirements version 6.4.1

serverless.yml
service: test
plugins:
  - serverless-python-requirements
custom:
  pythonRequirements:
    zip: true
    dockerFile: Dockerfile
    dockerizePip: non-linux
provider:
  name: aws
  runtime: python3.6
  stage: dev
  environment: ${file(./env.yml):${opt:stage, self:provider.stage}.env}
  region: ${file(./env.yml):${opt:stage, self:provider.stage}.aws.region}
  profile: ${file(./env.yml):${opt:stage, self:provider.stage}.aws.profile}
package:
  individually: true

Dockerfile
# AWS Lambda execution environment is based on Amazon Linux 1
FROM amazonlinux:1

# Install Python 3.6
RUN yum -y install python36 python36-pip

# Install your dependencies
RUN curl -s https://bootstrap.pypa.io/get-pip.py | python3
RUN yum -y install python3-devel mysql-devel gcc

# Set the same WORKDIR as default image
RUN mkdir /var/task
WORKDIR /var/task

I have my project files in C:\Serverless\test. I run npm init, followed by npm i --save serverless-python-requirements, accepting all defaults. I get the following on sls deploy -v. even though I've added C:\ to Shared Folders on the running default VM in VirtualBox, and selected auto-mount and permanent.

Serverless: Building custom docker image from Dockerfile...
Serverless: Docker Image: sls-py-reqs-custom

  Error --------------------------------------------------

  Unable to find good bind path format

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Stack Trace --------------------------------------------

Error: Unable to find good bind path format
    at getBindPath (C:\Serverless\test\node_modules\serverless-python-requirements\lib\docker.js:142:9)
    at installRequirements (C:\Serverless\test\node_modules\serverless-python-requirements\lib\pip.js:152:7)
    at installRequirementsIfNeeded (C:\Serverless\test\node_modules\serverless-python-requirements\lib\pip.js:451:3)

If I move my project to C:\Users, I get this instead:
Serverless: Docker Image: sls-py-reqs-custom
Serverless: Trying bindPath /c/Users/Serverless/test/.serverless/requirements (run,--rm,-v,/c/Users/Serverless/test/.serverless/req
uirements:/test,alpine,ls,/test/requirements.txt)
Serverless: /test/requirements.txt

  Error --------------------------------------------------

  docker: Error response from daemon: create "/c/Users/Serverless/test/.serverless/requirements": "\"/c/Users/Serverless/test/.serv
erless/requirements\"" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If you in
tended to pass a host directory, use absolute path.
See 'docker run --help'.

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Stack Trace --------------------------------------------

Error: docker: Error response from daemon: create "/c/Users/Serverless/test/.serverless/requirements": "\"/c/Users/Serverless/test/
.serverless/requirements\"" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If y
ou intended to pass a host directory, use absolute path.
See 'docker run --help'.

    at dockerCommand (C:\Users\Serverless\test\node_modules\serverless-python-requirements\lib\docker.js:20:11)
    at getDockerUid (C:\Users\Serverless\test\node_modules\serverless-python-requirements\lib\docker.js:162:14)

I'm a bit at a loss as to what to do next and advice would be greatly appreciated. TIA.

I'm not sure this will friends but I had the same issue myself, Windows 10 1803. Simply un-sharing my C drive and re-sharing it got me past the Unable to find good bind path format error.
I can't even take credit for the solution, I pulled it from this thread: #210
I have a new error that I'm sure is waiting just beyond this one for you. docker: Error response from daemon: invalid mode: /test.. I will investigate and file an additional issue if I cannot resolve it.

The unsharing/resharing has not worked for me.  I've been at this for almost 3 weeks with no resolution at this point.

I was unable to make the plugin work but I found a better solution anyhow - Lambda Layers. This is a bonus because it reduces the size of the lambda and allows code/file reuse. There is a pre-built lambda layer for numpy and scipy that you can use, but I built my own to show myself how it all works.  Here's how I made it work:
Create a layer package:

Open an EC2 instance or Ubuntu or Linux or whatever - This is needed so we can compile the runtime binaries correctly
Make sure pipenv is installed: sudo pip install pipenv
Create a pip virtual environment and install packages

pipenv --python 3.6
pipenv shell
pipenv install numpy==1.15.4   # or pipenv install -r requirements.txt

Make a dependencies package zip - Must use the directory structure python/lib/python3.6/site-packages for python to find during runtime

mkdir -p tmpdir/python/lib/python3.6/site-packages
pipenv lock -r > requirements.txt
pip install -r requirements.txt --no-deps -t tmpdir/python/lib/python3.6/site-packages
cd tmpdir zip -r ../py_dependencies.zip .
cd ..
deactivate
pipenv --rm
rm -r tmpdir requirements.txt Pipfile Pipfile.lock

Push layer zip to AWS - requires latest awscli

sudo pip install awscli --upgrade --user
sudo aws lambda publish-layer-version \
--layer-name py_dependencies \
--description "Python 3.6 dependencies [numpy=0.15.4]" \
--license-info "MIT" \
--compatible-runtimes python3.6 \
--zip-file fileb://py_dependencies.zip \
--profile python_dev_serverless

To use in any function that requires numpy, just use the arn that is shown in the console or during the upload above

f1:
    handler: index.handler_f_use_numpy
    include:
      - functions/f_use_numpy.py
    layers:
      - arn:aws:lambda:us-west-2:XXXXX:layer:py_dependencies:1

As an added bonus, you can push common files like constants to a layer as well.  Here's how I did it for testing use in windows and on the lambda:

import platform

# Set common path
COMMON_PATH = "../../layers/common/"
if platform.system() == "Linux": COMMON_PATH = "/opt/common/"

def handler_common(event, context):
    # Read from a constants.json file
    with open(COMMON_PATH + 'constants.json') as f:
        return text = json.load(f)

Yeah, if all you need is numpy & scipy, I definitely recommend using AWS's layer.
Creating your own on EC2 or with docker (but you'll need to figure out the bind path 😉) or a VM is definitely a good option, you can also use the serverless framework to publish your layer: https://serverless.com/blog/publish-aws-lambda-layers-serverless-framework. I'd recommend the last technique used, the one with cloudformation exports.

I only use the EC2 because I have a nice development AMI that is essentially a fully configured Ubuntu machine with remote desktop connection.  Ubuntu for Windows would work ok just fine as well.  The reason I push the layers separately from serverless is because I want to use the layers across multiple services (e.g., a py36_core_dependencies layer with numpy, scipy, scikit-learn, etc.).  It's easy to then add the add the layers via an env.yml or the serverless.yml.  While I wish I had made this plugin work, I think layers is a better overall solution because it reduces lambda package size and allows code reuse across services.

Ok so, to get all this stuff to fly on Windows 10 (today's challenge, as I had to switch from a work Mac)...

Installed Docker Toolbox (as I also use VMware, so couldn't use the Hyper-V based Desktop one)
Go into VirtualBox, make sure that your C drive is automount shared as /C in the configuration there, and restart the virtual machine - go in with docker-machine ssh to check that /C does indeed reflect your C drive - this deals with not being able to find bind paths.
Now for a bit of code hacking - in node_modules/serverless-python-requirements/lib/pip.js, inside dockerPathForWin, change the return value for windows to be

return `${path.replace(/\\/g, '/')}`;

(we're removing the extra double quotes - this deals with the complaints about incorrect characters in paths)

And finally, inside your serverless.yml file , make sure to specify

custom:
  pythonRequirements:
    dockerizePip: non-linux
    pythonBin: python

(this deals with the package defaulting to 'python.exe' which clearly won't work inside the docker container).
And thus, I have been able to sls deploy from Windows 10.

I solved the problem by changing the name of the shared folder from C_DRIVE to c. Try it.

I'm trying to debug this.. does anyone know how to run docker toolbox in windows in a VirtualBox VM? I don't have a windows machine :/

Hey 👋 I'm closing this ticket as it looks like it's heavily outdated, we can of course reopen it if needed 👍


powers to Arabic

I would like to change the power to be on the left instead of the rightbut I couldn't find that part in the source code.
Can you point it out please. thanks

It's not quite clear what you are asking.  Are you using the Arabic third-party extension?  If not, perhaps this already does what you need?

thank you, that's what I needed