enhancement : focus to preconfigured server

Hello,
Could you add the possibility to focus to servers preconfigured in servers.xml ?
Preconfiguration of "currentServerUrl".
Others config parameters could be fine
Or just use a config.json for all users
Because by default it focus to add new server
thank you

Seems to be okay with #1850

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Newly created worlds don't load.

Same problem with v0.9.3 v0.9.4 and v0.10.3.
Whenever I make a new world it says world created. Then when I try to load the world it says that the world doesn't exist. But it does exist in /list and I can rename the world. When I log off and open the world again then it stays infinitely on the loading screen.
I have Minecraft version 1.8.9 installed with forge <IP_ADDRESS>.1890.
In the following link I will upload a screenshot of all the mods that I have installed on it.
http://imgur.com/a/HE2bi
Let me know if you need any more information.
I hope you can help fix my problem.
(btw for on sponge forum, tell the guy asking about enabling pixelmon spawns to add the dimension to the list of pixelmon spawns in pixelmon config spawning>spawn dimensions)

I tried it with only spongeforge and ProjectWorlds and I got the same result. (Same spongeforge and forge version)

I believe this is connected to this https://github.com/trentech/ProjectWorlds/issues/28
I'm also convinced it's isolated to SpongeForge. I'm checking it out.

Yeah it seemed connected but I wasn't sure if I should comment there or make a new issue since things where a bit different.
And thanks for looking at it :D


automatic groups all_* are being removed when reloading groups

Home Assistant release (hass --version):
HASS.io 0.63.3
Python release (python3 --version):
Component/platform:
group
Description of problem:
After booting/restarting hass.io, there are some automatically generated groups with an all_-prefix (group.all_covers, group.all_automation, group.all_devices, group.all_lights etc). These groups have an attribute auto: true
When you call "reload groups" in the settings panel, those all_-groups are being removed. This can lead to problems with scripts/automations depending on those scripts.
If you restart hass.io, the all_-groups are loaded again (until you reload groups again)
Expected:
Reloading groups should not lead into the group.all_ groups being removed (aka groups with attribute: auto: true)
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

There's already an open issue about this. https://github.com/home-assistant/home-assistant/issues/12012

Duplicate of #12012.


No remote archive indexed for encoding key

Error reported by EverywhereSean on Discord:
Having an issue with exporting the Queen Azshara Naga model
Here's my log
[03:54:49] Previewing model creature/azsharanaga/azsharanaga.m2
[03:54:49] Loading remote CASC file 1820347 (creature/azsharanaga/azsharanaga.m2)
[03:54:49] Failed to open CASC file: No remote archive indexed for encoding key: aa8a41149c0618ebe80d9ec9e2f39e49

I can reproduce this issue on remote builds. Seems like CASC doesn't map the file correctly? Not sure what we can do this about. Needs more testing.

Files not in present in index but present in encoding will be unarchived.
http://blzddist1-a.akamaihd.net/tpr/wow/data/aa/8a/aa8a41149c0618ebe80d9ec9e2f39e49


add truncate parameter for shortening inputs to match model token length limits

options are -

"NONE" (default), do not truncate and raise an exception if the input is too long
"END", truncate tokens from the right side of the input
"START", truncate tokens from the left side of the input

future: consider END/RIGHT/BOTTOM and START/LEFT/TOP aliases for better language support
note: NONE is the default because it is the service's default, most users will pick END
fixes #20

@sduttanv will this address the issue?

Hi @mattf, this solves the issue. Thank you so much!!


Remove madetech users from pttp/mojo repo's

Removed RichRace and NeilKidd as he has been added back via GitHub Org level.

Reopened to kick pipeline


Henry Ecker: Watch 1-866-332-0276

Henry Ecker requests the watch of the watch_number 1-866-332-0276. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Makyen in Charcoal HQ


🐛🎨Fixed errors in the community page and updated the design template

.Users couldn't access their chats even after using their custom Project ID. The error is fixed.
.Updated the community design template.

https://github.com/N-45div/Resilient-Response/assets/113259853/bab514a2-0421-44fb-a747-70cdeff15b53

Very nicely made @Akangkha....Well done !


[Feature] Introduce the Debug\Deprecation utility component

Description
This PR introduces the CodeIgniter\Debug\Deprecation namespace containing a set of classes and traits to be used in deprecating certain elements in the framework. This can be used in the future for debugging deprecated elements that needs eventual removal.
1. CodeIgniter\Debug\Deprecation\Deprecation
This final static class is the "trigger" for the deprecation proper. It hosts a variety of methods to call in a deprecation message for a particular element (e.g., a class, method, property, or method parameter).
This has two modes on how to handle a deprecation. If mode is Deprecation::LOG_MESSAGE, which is the default mode, the deprecation call will just log the message to our registered loggers with a default log level of error (to conform with the default logging threshold and increase the chance the message is logged and read by the user). If mode is set to Deprecation::THROW_EXCEPTION, a DeprecationException will be thrown containing the deprecation message thus halting the script execution. Mode can be set by Deprecation::setMode() and retrieved by Deprecation::mode().
2. CodeIgniter\Debug\Deprecation\DeprecationException
This exception class is a child of ErrorException class. The original approach was to use the regular mechanic of trigger_error($message, E_USER_DEPRECATED) but I found out that our error handler catches this error and throws an ErrorException instead. So, to make a short cut, this class is thrown instead directly. This class adds a minor tweak to the error message and ensures $severity level is E_USER_DEPRECATED.
3. CodeIgniter\Debug\Deprecation\DeprecatedClassMethodTrait
This support trait is intended to be imported by classes with deprecated methods and doesn't want those to be used outside the class. This trait can also be used by parent classes to prevent child classes to use their deprecated methods. The trait intercepts calls to methods via the magic __call and __callStatic methods. (More info on the class definition)
4. CodeIgniter\Debug\Deprecation\DeprecatedClassPropertyTrait
This is similar to the DeprecatedClassMethodTrait except that this trait intercept calls to properties instead using the magic __get and __set methods. This trait is available only for properties in object context because PHP do not currently have __getStatic and __setStatic methods. (More info on the class definition)
Localized deprecation messages are stored in system/Language/en/Deprecation.php.
When using the traits, please be aware of the notion of overloading in PHP. If the inaccessible property/method is declared in the current scope, the traits' specially tweaked magic methods won't be invoked. That is, accessing a deprecated private property within the class will not emit a deprecation message since this is accessible.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[x] Conforms to style guide

Pending:

[x] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Integration into the framework internals

{
   ...
  "scripts": {
      ...
      "post-package-update": "CodeIgniter\\ComposerScripts::postPackageUpdate"
  },
  ...
}

I like this. I'll include this in the integrations. However, I think it is quite intrusive to always show the deprecation message whether after package update or app running. Users would want this lengthy message to show if they are actually using those deprecated elements. This script however will be useful for the internal updating of the framework to locate all deprecated elements at once.

@paulbalandan

In such cases I am thinking of aliasing the method during import.

it's magic methods :(

Apparently, it works. https://3v4l.org/IBA3o

@paulbalandan yes, that's work, but why you think what's better than something like Deprecated/BCClass?

@paulbalandan

Apparently, it works. https://3v4l.org/IBA3o

I try say: it's not work as magic method, it's can't work as "class using trait use trait's methods, but that's methods call class methods if they defined".

@paulbalandan

The original approach was to use the regular mechanic of trigger_error($message, E_USER_DEPRECATED) but I found out that our error handler catches this error and throws an ErrorException instead.

fixed in #4680

Since this is being reviewed anyway, I am requesting review now to know others' opinions on this before I continue work on this.

@paulbalandan ok, I vote to use simple helper like as https://github.com/koseven/koseven/blob/devel/system/classes/KO7/Core.php#L1050

It looks like @paulbalandan has put a lot of work into this PR, but if the main goal here is to "mark" deprecated methods then I would be in favor of doing it in a very simple way, as suggested by @WinterSilence (helper function + detection of the depreciation tag).
This is just my preference. I like to keep things simple if it's possible - less code is usually better. Both from the perspective of maintaining the code and from the perspective of those who will use it later.
I'm still curious about others' opinions.

I'd like to get this resolved. I agree @paulbalandan has done lots of work, and it is quality code. It is definitely more complex than the simple function. I'm still open to hearing more opinions, but here is where I am at right now:

If this utility is something we can offer to developers as a framework feature to enhance their own project & library workflow, I think it is worthwhile pursuing it as a development component (which the framework uses as well, of course)
If this ends up mostly being an internal tool to keep us from forgetting to put deprecate('Use something else') into our classes then the complexity may be too daunting, especially in the interest of encouraging more contributors

Thanks, @MGatner !  🎉 I originally intended this to be mostly internal while it is on its early stage but while working on it I conclude we can offer this to "advanced" users, especially CI4 module creators.
From a bit of research long ago in support of my side, I came across doctrine/deprecations which is somehow similar to this PR's architecture. So I think this could be useful internally and externally.

Late to the party, but here's my $0.02. I also prefer things kept simpler when possible.
In this case, I see a lot of value from having deprecated methods show up in the logs. Having the framework add a call to trigger that seems simple enough and very helpful.
I also like the two different modes. Nice touch.
I'm not convinced at all about the traits to intercept calls. To me, those feel like more complexity than necessary, and potential gotchas waiting to happen.


[google_maps_flutter] setMapStyle takes time to load, user is presented with blank map view

I'm using a custom map style for the Google Maps Flutter package.
Implemented with this code
  void _onMapCreated(GoogleMapController controller) {
    setState(() {
      _mapController = controller;
      _mapController.setMapStyle(_mapStyle).whenComplete(() {
        showMap = true;
      });
    });}

Current "problem" (more like an imperfection): the style is set after the map is loaded, so the user first gets a white screen, then the standard Google Maps style and after all that the custom style. This happens very fast, but there is a noticeable "flash". Especially when you're using a dark mode map style.
What I tried:

putting with Stack a black container on top of the google map widget (that' visible when showMap is set to false), so I can set the bool showMap to true if everything is loaded. Doesn't work because whenComplete get's fired when the setMapStyle is almost done (so there's still a flash). (tested in debug).

Solutions I could think of with the current possibilities:
loading the google maps widget in the background on another screen, so when the user taps on the screen that has the google map widget, the map is already fully loaded.
I tried an approach to this with the cupertinotabbar and a statefull widget (with the google maps widget inside, that has keepAliveClientMixin enabled) and stack but the map kept reloading. Are there other approaches to this?
A nice feature would be:
Possibility to set a map style before the map is created
Or if the above feature is really difficult / almost impossible to implement:
Possibility to set a background color when the map isn't loaded yet (currently this color is white and has a big contrast with custom map style's that are darker).

Reproduces with current google_maps_flutter 0.5.28+1.

complete code sample
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';
import 'package:flutter/services.dart' show rootBundle;

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Google Maps Demo',
      home: MapSample(),
    );
  }
}

class MapSample extends StatefulWidget {
  @override
  State<MapSample> createState() => MapSampleState();
}

class MapSampleState extends State<MapSample> {
  GoogleMapController _controller;
  String _mapStyle;

  static final CameraPosition _kGooglePlex = CameraPosition(
    target: LatLng(37.42796133580664, -122.085749655962),
    zoom: 14.4746,
  );

  @override
  void initState() {
    super.initState();

    rootBundle.loadString('assets/map_style.txt').then((string) {
      _mapStyle = string;
    });
  }

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: GoogleMap(
        initialCameraPosition: _kGooglePlex,
        onMapCreated: (GoogleMapController controller) {
          setState(() {
            _controller = controller;
            _controller.setMapStyle(_mapStyle);
          });
        },
      ),
    );
  }
}

map_style.txt

flutter doctor -v
[✓] Flutter (Channel master, 1.20.0-8.0.pre.57, on Mac OS X 10.15.5 19F101, locale en-GB)
    • Flutter version 1.20.0-8.0.pre.57 at /Users/nevercode/dev/flutter
    • Framework revision 8bd2e6585b (2 days ago), 2020-07-11 12:15:19 -0700
    • Engine revision 9b3e3410f0
    • Dart version 2.9.0 (build 2.9.0-20.0.dev 06cb010247)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/nevercode/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (5 available)
    • Redmi Note 7 (mobile)              • 6345c469      • android-arm64  • Android 9 (API 28)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • macOS (desktop)                    • macos         • darwin-x64     • Mac OS X 10.15.5 19F101
    • Web Server (web)                   • web-server    • web-javascript • Flutter Tools
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 83.0.4103.116

• No issues found!

@VladyslavBondarenko Just updated to that version and tested it on iOS 13. Looks the same. Do you have any recommendations to make it less noticeable?

Issue reproduced on the latest google_maps_flutter: ^2.0.3
Preview

Android
iOS

code sample
import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';
import 'package:flutter/services.dart' show rootBundle;

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Google Maps Demo',
      home: MapSample(),
    );
  }
}

class MapSample extends StatefulWidget {
  @override
  State<MapSample> createState() => MapSampleState();
}

class MapSampleState extends State<MapSample> {
  late GoogleMapController _controller;
  late String _mapStyle;

  static final CameraPosition _kGooglePlex = CameraPosition(
    target: LatLng(37.42796133580664, -122.085749655962),
    zoom: 14.4746,
  );

  @override
  void initState() {
    super.initState();

    rootBundle.loadString('assets/night_mode.json').then((string) {
      _mapStyle = string;
    });
  }

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: GoogleMap(
        initialCameraPosition: _kGooglePlex,
        onMapCreated: (GoogleMapController controller) {
          setState(() {
            _controller = controller;
            _controller.setMapStyle(_mapStyle);
          });
        },
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.6, on macOS 11.3 20E232 darwin-x64, locale en-GB)
    • Flutter version 2.0.6 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 1d9032c7e1 (5 days ago), 2021-04-29 17:37:58 -0700
    • Engine revision 05e680e202
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554                        • android-x86    •
      Android 10 (API 29) (emulator)
    • iPhone 12 (mobile)                 • 4D5BE4C8-BF1D-46CD-B595-FD81A7140B52 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)                    • macos                                • darwin-x64     •
      macOS 11.3 20E232 darwin-x64
    • Chrome (web)                       • chrome                               • web-javascript •
      Google Chrome 90.0.4430.93

• No issues found!

[✓] Flutter (Channel master, 2.3.0-1.0.pre.226, on macOS 11.3 20E232 darwin-x64, locale en-GB)
    • Flutter version 2.3.0-1.0.pre.226 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 51ca131e39 (10 hours ago), 2021-05-03 19:54:04 -0700
    • Engine revision 378e4dbc41
    • Dart version 2.14.0 (build 2.14.0-48.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1.1)
    • IntelliJ at /Volumes/Extreme/IntelliJ IDEA CE.app
    • Flutter plugin version 55.1.5
    • Dart plugin version 211.7233

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

[✓] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554                        • android-x86    •
      Android 10 (API 29) (emulator)
    • iPhone 12 (mobile)                 • 4D5BE4C8-BF1D-46CD-B595-FD81A7140B52 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)                    • macos                                • darwin-x64     •
      macOS 11.3 20E232 darwin-x64
    • Chrome (web)                       • chrome                               • web-javascript •
      Google Chrome 90.0.4430.93

• No issues found!

Is anyone maintaining the issue seams to me that there is no progress on it since one year and the issue is still existing.

Indeed I still have the same issue.
Anyone has found a workaround?

More than year passed from issue was initiated. Is there any news on this subject?

I use the following code as a work around to give the map time to load and provide a smooth transition using animatedOpacity to display it when loaded. Hope this can help you.
`
// Declare variables
GoogleMapController? _googleMapController;
final Future _mapFuture = Future.delayed(const Duration(milliseconds: 1000), () => true);
// Build map using FutureBuilder
FutureBuilder
(
future: _mapFuture,
builder: (context, AsyncSnapshot snapshot)
{
if(!snapshot.hasData)
{
return const Center
(
child: CircularProgressIndicator()
);
}
return AnimatedOpacity
(
  curve: Curves.fastOutSlowIn,
  opacity: isMapVisible ? 1.0 : 0,
  duration: const Duration(milliseconds: 600),
  child: Consumer<NewListingAddMarkers>
  (
    builder:(context, markers, child) => GoogleMap
    (
      markers: markers.markersList,
      myLocationButtonEnabled: false,
      zoomControlsEnabled: false,
      mapToolbarEnabled: false,
      compassEnabled: false,
      buildingsEnabled: false,
      initialCameraPosition: _initialCameraPosition,
      onMapCreated: (controller) async
      {
        _googleMapController = controller;
        _googleMapController!.setMapStyle(MapStyle.mapStyle);
        Future.delayed
        (
          const Duration(milliseconds: 550),
          () => setState
          (()
            {
              isMapVisible = true;
            }
          )
        );
      },
    ),
  ),
);

}
),
`

Still very relevant as of today.
Flutter 3.10.4
google_maps_flutter: ^2.3.0
google_maps_flutter_web: ^0.5.0+1

Still. Even I use AnimatedOpacity, the first frame of google map still is normal style. If you use dark style, you can obviously see the flicker between white and dark!

How to solve the problem of blank map in dark mode?


fix(types): use typescript 5.0

Description

TypeScript 5.0 has been released with standard decorator support and various config improvements, notably including --moduleResolution bundler which is recommended by Evan You and Daniel Rosenwasser for Vite users.
Additional context

Note that due to TypeScript not following semver, 4.9 and 5.0 are both releases with breaking changes, so this upgrade should be roughly as compatible as previous major and minor upgrades.

What is the purpose of this pull request?

[x] Bug fix
[x] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

I'm not sure this is the best name, assuming it'll be copied into the changelogs. Is there a way to tweak the changelog for each package separately? The bulk of the interesting changes are in create-vite.

@fi3ework Thanks for the warning, fixed it. The other packages don't seem to be dependent.


Port to GameBoy

It would be basically my dream to use a strongly typed language for gbdev.

That would be completely feasible --- the GB processor is similar to the 8080, and there's already a core for that. It would need a custom emulator to run compiler tests on but that wouldn't be hard.

It's an 8080/Z80 hybrid monster, but yeah :slightly_smiling_face: I agree it should be feasible given the goals!


Add missing cluster creation step

We have already created the cluster in the steps in 1a (or if following the steps from 1b, we are going to get credentials for an already existing cluster).


Fixes #3314

Cleans up and simplifies (a bit) the code for perceiving chirality
based on a conformer and wedged bonds.

@greglandrum, I'd like to understand the changes a bit better, but I have a tentative approval here.

@greglandrum, I'd like to understand the changes a bit better, but I have a tentative approval here.

Great. Let me know if you have any questions.


Charts with d3.js

Hi!
Propose add a new object to the library.
Regards, Maxim.

I like the idea.
Btw, in the "prototype", you have declared this.last.g twice. The second one (line 41) should probably be g1.

Sorry, right - should be g1.

I'm new in Githab. If changes after PR but before its adoption, it is necessary to cancel and create a new PR?

I am not 100% sure about charts, as I feel it deserves its own repository. I have worked with d3 extensively and do have some samples too. Let's give it a try and see where it will lead. Should it stay as a single w2ui.js file or be split into its own file, not sure yet.

Thank, this is my first pull request :)

I am not 100% sure about charts, as I feel it deserves its own repository.

+1 for their own repository (at least what NPM and Bower is concerned), i.e. W2UI should include only the required changes (if any) to allow charts easily, but no dependencies or charts IMHO.
(many already use the one or the other charting library, so this extension would be useful more for new projects that haven't decided upon a charting lib yet)
The Demos could include the charts too, since it makes no sense to host many different demos.


Minor code moves in controller_binding.go

The event handlers were not grouped together in this file as they are in other controller_*.go; I had to hunt for delete while adding some logging.

LGTM

LGTM


Contributing?

I really like the mission of Weld and would love to contribute! How would you suggest I get started? Is there a Slack (or similar) workspace to chat?

Awesome! We just started a chat forum here: https://spectrum.chat/weld, where we can discuss some ideas. There's a ton of stuff to do depending on what your interests are, ranging from new compiler optimizations in Rust, to making the Python API nicer, to adding/improving some of the tools that can be used to understand and debug Weld programs.
I'll also begin adding some starter projects as GitHub issues and labeling them as such as well.


License confusion

Thanks for sharing this. There is a confusion about the license. If I'm not missing something here, Emu is based on llama 1 and commercial use is prohibited but Emu2 is llama 2 and hence commercial is ok till 700m user. Also Apache2 is mentioned as the license, so what is exactly the license for Emu 2 ??
Thanks in advance

Both the training of Emu1 and Emu2 use the llama 1 weights as the pretrained checkpoint, so the use of Emu weights should follow the llama1 license. But the code itself of Emu follows the Apache2.0 license, it means you can use our code to train your own model for any scenarios which are allowed in Apache2.0.


flowbit-tooltip is behind card issue fixed

Description
Fixed the flowbite tooltip which appears on the hover of the sidebar. The tooltip goes behind a card which is fixed now.
Before:

After:
\
Fixes #359
Type of change

[ ] Bug fix (non-breaking change which fixes an

Breaking changes
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

Thanks.


Use tracemap instead of sourcemap

https://github.com/jridgewell/trace-mapping

Not really, we need more power than that


Babel transpilation issue on block aliases

Looking into #2281, it looks like babel is changing these lines:
if ( this.owner.template.z ) {
    fragment.aliases = { [this.owner.template.z[0].n]: model };
}

into this:
if (this.owner.template.z) {
    fragment.aliases = defineProperty({}, this.owner.template.z[0].n, model);
}

(which fails because it should be { value: model }).
However, if I use the babel online cli for:
var foo = { bar: 'bar' }
var o = { [foo.bar]: 12 };

It generates a babel defined _defineProperty:
function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }

var foo = { bar: 'bar' };
var o = _defineProperty({}, foo.bar, 12);

Maybe conflict with babel somehow, HOWEVER probably better to skip babel on this one and just do:
const aliases = {};
aliases[this.owner.template.z[0].n] = model;
fragment.aliases = aliases };

(Note there are multiple occurrences in that file)

@martypdx I couldn't figure out why, but the iterative aliasing tests were running fine in Chrome and refused to run in phantom. Swapping out the computed properties makes all of the tests work in phantom too. It also fixes the #2264 failure. I shall henceforth be more wary of newer ES features.


fix: fix np.int64 issue

We add l2 normalization in HNSW before index and search, but this will break when the input type is np.int64. This pr fix this issue.

why should we support np.int64? I cannot see in which case the dtype of embedding should be in int64 format.


Please update React.js dependency.

However<EMAIL_ADDRESS>wants<EMAIL_ADDRESS><=0.13.0-beta.x" :
"peerDependencies": {
"react": ">=0.12.0 <=0.13.0-beta.x"
}
Could you please update it?

@sAbakumoff  Please try latest 0.5.12 release: https://github.com/yahoo/flux-router-component/releases/tag/v0.5.12

Thank you! I think fluxible also needs to be updated: https://github.com/yahoo/fluxible/blob/master/package.json
shall I enter the new issue in fluxible repo?

@sAbakumoff Done: https://github.com/yahoo/fluxible/releases/tag/v0.2.4


xmessage: Add test

Contributing to homebrew-xorg:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass
brew audit --strict <formula> (where <formula> is the name of the formula you're submitting)?
[x] Have you built your formula locally prior to submission with brew install <formula>?

Not entirely sure this is going to work on CI, but guess we'll find out.

Hmm, apparently brew style failed:

== /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/linuxbrew/homebrew-xorg/Livecheckables/intel-media-driver.rb ==
C:  4: 78: Redundant escape inside regexp literal
== /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/linuxbrew/homebrew-xorg/Livecheckables/libva-intel-driver.rb ==
C:  4: 70: Redundant escape inside regexp literal
== /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/linuxbrew/homebrew-xorg/Livecheckables/libva-internal.rb ==
C:  4: 57: Redundant escape inside regexp literal
== /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/linuxbrew/homebrew-xorg/Livecheckables/libva.rb ==
C:  4: 57: Redundant escape inside regexp literal
== /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/linuxbrew/homebrew-xorg/Livecheckables/libvdpau-va-gl.rb ==
C:  4: 68: Redundant escape inside regexp literal

Since I was the last person to touch all of those files, I'll take a look at them later this week (but obviously anyone is welcome to fix them themselves if they want to)

It was a trivial fix. Submitted separately as #654

Testing linuxbrew/xorg/xmessage
==> xmessage -timeout 1 Hello Homebrew
Error: Can't open display:
Error: linuxbrew/xorg/xmessage: failed

Yeah, I was wondering if it was gonna do that... This might require some additional thought as to the best way to handle tests that require an X server.

Looks like fake X11 is finally working.
I'm going to squash my commits into one.

@sjackman , have a look at this magic: we're testing an X11 app here 😎

@rwhogg , I'm going to stop tinkering with xvfb-run and merge this "as is"
I wanted to use xvfb-run -a brew test-bot... because this is a one-line change compared to what I had to do here... but it doesn't work for some reason.

have a look at this magic: we're testing an X11 app here 😎

Woohoo! That's awesome!


Support specialized equality methods

Feature summary
Often times, classes define specialized equality methods that have different semantics than Object#equals. It would be nice to be able to reference these directly via a BiPredicate<T, T> and have it work with isEqualTo and other relevant methods.
Example
assertThat(actual)
    .usingEqualityMethod((x, y) -> x.equalsStrict(y))
    .isEqualTo(expected);

// or

assertThat(actual)
    .usingEqualityMethod(MyObject::equalsStrict)
    .isEqualTo(expected);

I can currently hack around this by (ab)using usingComparator, but it's not ideal.
assertThat(actual)
    .usingComparator((x, y) -> x.equalsStrict(y) ? 0 : -1)
    .isEqualTo(expected)

I agree allowing to express equals with BiPredicate<T, T> would be nice.
Would usingEquals be as clear as usingEqualityMethod ? No strong opinion on my side, just thinking of other namings

usingEquals is a less verbose and makes sense to me.

PR welcomed 😉


CertificateRequest is not getting approved

I have tried installing it through the deployment manifests through kubectl apply and also through the helm chart, but I cannot seem to get the CertificateRequest approved.
I tried deleting and reapplying the certificate to get it to approve but it is not working. I got all the RBAC manifests applied as well.
Any ideas I could try to get it working?

This repository doesn't ship anything that would approve CertificateRequests, as that requires knowledge of your environment along with your organization's access and change control policies.
However, cert-manager's default configuration configures an approver that automatically approves all requests. Have you changed this configuration, and have you consulted the cert-manager logs?

@terinjokes I thought the ClusterRole and the ClusterRoleBinding shipped in this repo made sure that the CertificateRequests would get approved. I did not check the cert-manager logs. I saw that I used the bitnami cert-manager helm chart, so I am now trying the jetstack one. Maybe that will already fix it. Otherwise I'll check the cert-manager logs

The ClusterRole and ClusterRoleBinding allow cert-manager to approve the requests. If you deployed cert-manager with a different service account name or into another namespace, you may be running into #83.

@terinjokes it does now work after moving to the jetstack/cert-manager instead of the bitnami/cert-manager... Completely my fault 🤦🏻  Thanks for your quick response!


How to refresh Azure Analysis services replica server/management server using REST API?

I am trying to refresh my azure analysis services replica server's model with :rw at the end of the server name and it says server not found error when I submit POST request for the URL below from Azure data factory, but the same is working fine when actual server is provided in the URL.
REST API URL not working for refresh:
https://westus.asazure.windows.net/servers/******:**rw**/models/***Cube/refreshes
REST API URL working for refresh:
https://westus.asazure.windows.net/servers/******/models/***Cube/refreshes
Please let me know if REST API support is available to refresh the replica server(management server) or just the processing server only?
[Enter feedback here]

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

ID: 6f706b7d-0a82-4535-f7ec-3851bd204bf2
Version Independent ID: fefdb693-fb98-25a0-684c-d828bb709114
Content: Learn about asynchronous refresh for Azure Analysis Services models
Content Source: articles/analysis-services/analysis-services-async-refresh.md
Service: analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@swathichmsft
Thanks for your feedback! We will investigate and update as appropriate.

@Minewiskan
Could you please review add comments on this, update as appropriate.

Hi @swathichmsft
It appears that you are attempting to update your Azure Analysis Services replica server's model by adding:rw to the end of the server name, but when you make a POST request for the URL, you receive a server not found error. When the actual server is included in the URL, the same thing functions properly.
According to Document we need to follow this URL format.
you can try using the following URL format to refresh the replica server's model:
https://.asazure.windows.net/servers//replicas//models//refreshes
Replace  with the region where your server is located,  with the name of your server,  with the name of your replica server, and  with the name of your model.

Hi @swathichmsft
It appears that you are attempting to update your Azure Analysis Services replica server's model by adding:rw to the end of the server name, but when you make a POST request for the URL, you receive a server not found error. When the actual server is included in the URL, the same thing functions properly. According to Document we need to follow this URL format. you can try using the following URL format to refresh the replica server's model:
https://.asazure.windows.net/servers//replicas//models//refreshes
Replace with the region where your server is located, with the name of your server, with the name of your replica server, and with the name of your model.
Hi @Naveenommi-MSFT ,

I have tried with the below URL adding replicas after servers in the URL, but it still throws the error - Server not found from ADF pipeline.Please let me know if the URL need to be passed differently?
URL used:
(https://westus.asazure.windows.net/servers/replicas/<>:rw/models/<>/refreshes)
Error:
Invoking Web Activity failed with HttpStatusCode - '404 : NotFound', message - 'The requested resource does not exist on the server. Please verify the request server and retry'

Hi @swathichmsft
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1


Fix Tsan issue with lemming_test

Add more logging so that the server state is easy to see in logs
Add unittest for stop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 16 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 67.508%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
401

Relevant Lines:
594

💛  - Coveralls


User unable to add LUIS or QnA resource from composer

Describe the bug
customer is using bot framework composer 2.0.0, when he tries to add LUIS for the project, he gets the login page, after successfully login.
he could see the azure directory and subscription, however the LUIS resource never displays like following:
while i dont have this issue in my lab, not sure if this is related with the sponsorship sub?
Please see ICM issue:
https://portal.microsofticm.com/imp/v3/incidents/details/247570921/home
Version
2.0.0
Browser

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

OS

[ ] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
See ICM

@Dewain27 is there a way for us to get access to a "Sponsorship sub" to see if we can recreate this issue? It may be a permission issue as @luhan2017 mentioned in the ICM.

@benbrown, there are the cases that one developer doesn't have the permission to all the resources under the subscription (they may have the permission only for his own resource group.)  @zhixzhan had a PR to fix this issue, #8160 . The fix is that we just ignore and don't show the resources which user have no permissions to list keys. This fix applies both for luis and qna.


SDXL: neuron_config of text_encoder_2 is CLIPTextNeuronConfig instead of CLIPTextWithProjectionNeuronConfig

I tried to search in the codebase and am not sure how they got chosen, but it appears that pipeline.text_encoder_2.neuron_config is optimum.exporters.neuron.model_configs.CLIPTextNeuronConfig while I suspect it should be CLIPTextWithProjectionNeuronConfig

wondering is this at least verified that it exists?

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

not stale? @JingyaHuang

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

Will check! Sorry for the late reply.

I kinda got where this question comes from, there is a bug on sdxl when we reconstruct neuron config of text encoder 2 from a json file 😅. The test I did above was during the export, but not the neuron config we reconstructed during the inference:
https://github.com/huggingface/optimum-neuron/blob/3359516a49e2c0c68e3db28d35ece53f38193439/optimum/neuron/modeling_diffusion.py#L524

hi! yes!
I was going to reply similar just now 😂 I was looking at during loading/inference, and here's what I got:

I also have a feeling that the json file of text encoder 2 might be mistaken too:
https://huggingface.co/aws-neuron/stable-diffusion-xl-base-1-0-1024x1024/blob/main/text_encoder_2/config.json#L30-L34
the text encoder 2 output should include pooler_output based on my shallow understanding?

From transformers' clip modeling code, text encoder (CLIPTextModel) shall be the one which has pooler_output as output while text encoder 2 (CLIPTextModelWithProjection) outputs text_embeds which is the text emb obtained by applying the projection layer to the pooled output.

you're right 😅 I had it the other way around
what I was trying to say is that the two jsons should look different in terms of the neuron. output_names?
or in my case, the text_encoder.neuron_config.outputs

should be fixed by #581


Examples/switch space

Adding an example to get switches with /mnt/flash /var/log or /var/core that are over 90% utilised

If you don't have the tooling for the linting installed you can run make dev-setup to install them

thanks Cian for reviewing, I have made the changes to resolve the flake8 errors and corrected the date.


Download block FED

Download(s) block.
 It's unusual to have hard-coded colors like that.

Yeah, good question. There are some hover styles, but I've not gotten to them yet. The only one I've seen in figma so far is the main nav (open vs closed state, actually):

So I think I'm going to have to go and update these icons so that the icon "shadow" colour is changeable. Usually we'd use currentColor as the icons would be one colour only. But in this case the main bit of the icon is always(?) black and it's the shadow bit that changes, but not to match the text.

yeah, you could use currentColor for the shadow (setting color on the icon rather than inheriting), or you could update a variable?


log-level: cloud.prefab and prefix tweaks

This updates the docs for the latest ruby version.
TODO: Make sure the java logger is using cloud.prefab.client as a prefix (or at least cloud.prefab)

I'm going to merge this optimistically. @jdwyah, LMK if you have any addendums or edits.


Logic for using quotes around variables

Hello,
I have a marathon deployment script which sets some ENV vars, as shown below:
"env": {
   "NEW_RELIC_APP_NAME" : "MyApp",
   "NEW_RELIC_HOST_DISPLAY_NAME" : "${HOST}"
   "NEW_RELIC_LICENSE_KEY" : "xxx",
   "NEW_RELIC_LOG" : "STDOUT",
}

You can see that each of the variables uses quotes around the value (to satisfy the json requirement).
However, when the container starts, I notice that the NEW_RELIC_APP_NAME and NEW_RELIC_HOST_DISPLAY_NAME variables have single ' marks around them, which causes the $HOST variable to not expand.  The other two variables don't have this:
NEW_RELIC_APP_NAME='MyApp'
NEW_RELIC_HOST_DISPLAY_NAME='${HOST}'
NEW_RELIC_LICENSE_KEY=xxx
NEW_RELIC_LOG=STDOUT

Can someone tell me how this happens?

Hi @cookandy
environment variables are simple key-value string pairs at the moment.
There is already an issue to keep track of this feature request here https://github.com/mesosphere/marathon/issues/1828 and here https://github.com/mesosphere/marathon/issues/2679.
Thanks for your feedback!


[Please ignore] Run matmul test in loop.

So in CI I get 2 / 1000 fails:

The failure pattern is different to what I get locally. In CI it is:

Whereas locally (where I got 536/1000 failures) it is more localized:

(although the failure pattern is different across runs).

Just in the interest of ruling things out, can you do with llvm-cpu target instead of amd-aie and local-sync instead of xrt device and verify there are no failures? Interesting bug, thanks for catching it! I wonder if there is still some intermediate file dependency we havent caught yet. The reason I suspect that is, I think your setup is in a slow NFS file system while the CI is not which could explain the rareness on CI.

Just in the interest of ruling things out, can you do with llvm-cpu target instead of amd-aie and local-sync instead of xrt device and verify there are no failures?

--target-backend llvm=cpu --device xrt
Command failed 0 times out of 1000 runs.
--target-backend llvm=cpu --device local-sync
Command failed 0 times out of 1000 runs.

I wonder if there is still some intermediate file dependency we havent caught yet. The reason I suspect that is, I think your setup is in a slow NFS file system while the CI is not which could explain the rareness on CI.

When I run with with --n_runs 1 I still get a high failure rate (locally). I guess that means if it a file system thing, it's maybe not related to a file name issue. I remove results_dir_tmp between runs where I'm running
  ./run_matmul_test.sh   results_dir_tmp ../../../iree-build-with-peano/   $MLIR_AIE_INSTALL_DIR    $PEANO_INSTALL_DIR  /opt/xilinx/xrt  $VITIS_INSTALL_PATH

Thanks @nirvedhmeshram, let me know if you have any more test ideas to run (at this point I'm a bit in the dark as to what to try!).

Additional data points. I run the following sequence of experiments locally. I modified the TestShape returned in tests/matmul/generate_e2e_matmul_tests.py and the -n_runs flag in  build_tools/ci/run_matmul_test.sh.
The wall clock time is included -- notice that for some shape it is 10x slower per iteration of the run -- I can't explain this.
real 0m9.861s                                                                                # 10 / 10 failures                                                                                # TestShape(m=64, k=8, n=64, accumulate=False)
                                                                                # real 0m1.594s                                                                                # 7 / 10 failures                                                                                #TestShape(m=64, k=16, n=64, accumulate=False)
                                                                                # real 0m1.896s                                                                                # 6 / 10 failures                                                                                #TestShape(m=64, k=32, n=64, accumulate=False)
                                                                                # Fails to compile.                                                                                # TestShape(m=64, k=48, n=64, accumulate=False),
                                                                                # real 0m9.971s                                                                                # 0 / 10 failures                                                                                #TestShape(m=64, k=64, n=64, accumulate=False)
                                                                                # real 0m10.104s                                                                                # 0 / 10 failures                                                                                # TestShape(m=64, k=96, n=64, accumulate=False)
                                                                                #real 0m9.871s                                                                                # 0 / 10 failures                                                                                # TestShape(m=32, k=16, n=64, accumulate=False)
                                                                                # real 0m29.174s                                                                                # 4 / 1000 failures                                                                                # TestShape(m=32, k=16, n=32, accumulate=False)
                                                                                # Then I ran m=8 k=16 n=32 and it froze (20 seconds, not a single run). Ctrl+C to stop.
                                                                                # Then running TestShape(m=32, k=16, n=32, accumulate=False) again resulted in 10/10 failures. So the state of the machine had changed.


Prepare detailed document for brownfield service onboard

We are now helping compute service onboard TypeSpec and corresponding SDK, during which there is many manual adjustments. This is OK for one service, but not sustainable for the following 200+ services. We need a detailed document/TSG guide following services do these adjustments themselves.

est: 5


Error when running Primer: PydanticImportError

While I trying to run the example from the hello world chapter in the Ought Primer I consistently get this error:
Traceback (most recent call last):
  File "/Users/ptmalmgren/src/ice-demo/hello.py", line 3, in <module>
    from ice.recipe import recipe
  File "/Users/ptmalmgren/Library/Caches/pypoetry/virtualenvs/ice-demo-25sS_li7-py3.10/lib/python3.10/site-packages/ice/recipe.py", line 24, in <module>
    from pydantic import BaseSettings
  File "/Users/ptmalmgren/Library/Caches/pypoetry/virtualenvs/ice-demo-25sS_li7-py3.10/lib/python3.10/site-packages/pydantic/__init__.py", line 210, in __getattr__
    return _getattr_migration(attr_name)
  File "/Users/ptmalmgren/Library/Caches/pypoetry/virtualenvs/ice-demo-25sS_li7-py3.10/lib/python3.10/site-packages/pydantic/_migration.py", line 289, in wrapper
    raise PydanticImportError(
pydantic.errors.PydanticImportError: `BaseSettings` has been moved to the `pydantic-settings` package. See https://docs.pydantic.dev/2.1.1/migration/#basesettings-has-moved-to-pydantic-settings for more details.

For further information visit https://errors.pydantic.dev/2.1.1/u/import-error

Because setup.cfg has no pinned version for Pydantic, I believe it is defaulting to the latest Pydantic 2.1.1 release which contains backwards-incompatible changes.

Fixed in #314.


TextField can still be modified even if it's disabled

The user can still enter/modify text in a disabled TextField! @smoti123

Fixed!


Update zio, zio-test, zio-test-magnolia, ... to 2.0.19

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-magnolia
dev.zio:zio-test-sbt

from 2.0.13 to 2.0.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 = "dev.zio" } ]

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

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

Superseded by #33.


PowerRename default setting option to exclude Subfolder and Files for Stability.

Description of the new feature / enhancement
Having a setting in the PowerRename application menu or better yet an option in the PowerRename Menu in the PowerToys settings menu that toggles the inclusion/exclusion of sub folders and files.
This way when someone selects a small list of folders in a location, PowerRename can skip searching through those folders all together.
This would also improve speed and overall stability of the tool, reducing the number of files it has to scan through and display.
Scenario when this would be used?
Scenarios when you have maybe 15 or more folders you want to quickly re-label, but those folders have more than a lot of files in them.

Supporting information
Not much outside of my personal constant experience of crashing due to the large volume of files it has to scan through unnecessarily.
But here are a few screenshots that show that the tool is indeed crashing.

Related to #16919

/dup #16162


Update assignment.md

hi, you can add this to a discussion on the discussion boards; this is not the right place for it, but thank you for working through a solution!


docs: Added missing languages, sorted alphabetically

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[ ] FIX - a PR of this type patches a bug.
[ ] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[x] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[ ] CHORE - a PR introduces other changes than the specified above.

@DamirSQA do you mind updating the SupportedLanguage type with the newly added language codes in this PR?
https://github.com/near/wallet-selector/blob/bfd37545aa54b4e56c5c79458b1d8676463de9bc/packages/core/src/lib/translate/translate.ts#L45-L53

While checking I found two strings untranslated for hr language.

While checking I found two strings untranslated for hr language.

This can be addressed in a separate/new task. This PR is not about adding new or missing keys.


fix: expose the errors in the errorList

Codecov Report

Merging #226 (2b0fbb7) into develop (dd7ab15) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #226      +/-   ##
===========================================
- Coverage    35.28%   35.10%   -0.18%
===========================================
  Files           49       49
  Lines         4356     4361       +5
===========================================
- Hits          1537     1531       -6
- Misses        2667     2676       +9
- Partials       152      154       +2

Flag
Coverage Δ

unittests
35.10% <0.00%> (-0.18%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/utils/errors.go
53.84% <0.00%> (-7.92%)
:arrow_down:

pkg/ghissue/buglink.go
90.32% <0.00%> (-9.68%)
:arrow_down:

pkg/utils/paths.go
5.00% <0.00%> (-5.00%)
:arrow_down:

Continue to review full report at Codecov.

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


fix(registrar module): fix for php-sdk and OTE connectivity

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


Lacks CSS styling for contents directive

Apart from sphinx's custom toctree directive, restructured text also provides the local contents directive for printing a local table of contents. Currently, the styling for this directive is missing. Especially, indentations of lower-level sections are missing.
Currently, this code
.. contents:: Overview of required steps
   :depth: 2
   :local:

renders this:

"Use a Cookiecutter template" and "Manually set up a project" are child headings of "Boostrap the project" and should at least be indented.

What's the difference between using contents and toctree?

contents lists the local headings inside a single file
Here's my quickfix again:
.contents.local .topic-title {
  text-transform: uppercase;
  font-size: 0.875rem;
  line-height: 1.25rem;
  --text-opacity: 1;
  color: #4a5568;
  color: rgba(74, 85, 104, var(--text-opacity));
  letter-spacing: 0.1em;
}

.contents.local li {
  padding-left: 1rem !important;
}

.contents.local ul.simple > li {
  padding-left: 0 !important;
}

https://stackoverflow.com/questions/24129481/how-to-include-a-local-table-of-contents-into-sphinx-doc

Thank you so much! Here's a preview: https://deploy-preview-191--sphinxawesome-theme.netlify.app/demo/contents/

Thanks for the quick fix!
Is there a reason why the vertical distance between entries in "Contents" is different from the ones in "On this page".

Yes, there is a reason but not a good one. The contents directive produces

... and the toctree directive has
 without the
 in between. I hate these inconsistencies between docutils and Sphinx. Probably I'll add the extra margin/padding to the toctree directive as removing the padding in the contents directive makes it a bit cramped when you have subelements.


feat: upgrade airbyte to 0.50.25

Summary

Test Plan

Checklist

[ ]  No images hosted from dockerhub
[ ]  Are dashboards present to understand the health of the application.  There must be at least 1 of these

[ ]  all databases should have dashboards
[ ]  ideally also have at least cpu/mem utilization dashboards for webserver tier of the app
[ ]  you can use plural from-grafana to convert a grafana dashboard found via google to our CRD

[ ]  Are scaling runbooks present

[ ]  all databases must have scaling runbooks
[ ]  you can use the charts in pluralsh/module-library to accelerate this

[ ]  do you need to add config overlays?

[ ]  inputing secrets
[ ]  configuring autoscaling

[ ]  If there’s a web-facing component to the app, we need to support OIDC authentication and setting up private networks if no authentication option is viable
[ ]  All major clouds must be supported

[ ]  Azure
[ ]  AWS
[ ]  GCP

Ah ya, I see. I tested with their most recent helm chart and everything I seemed to be working properly. Sounds good, I'll close for now.


[Bug]: vllm failing on tpu v5litepod-8

Your current environment
collect_env.py doesn't report any gpu information on tpu nodes.
Model Input Dumps
.
🐛 Describe the bug
When I build and run the docker container according to the vllm tpu guide I get the following error:
ERROR 10-21 02:38:59 worker_base.py:464] requests.exceptions.ConnectionError: HTTPConnectionPool(host='metadata.google.internal', port=80): Max retries exceeded with url: /computeMetadata/v1/instance/attributes/tpu-env (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f01780d21a0>: Failed to establish a new connection: [Errno 111] Connection refused'))
Process SpawnProcess-1:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 199, in _new_conn
    sock = connection.create_connection(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/connection.py", line 85, in create_connection
    raise err
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/connection.py", line 73, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 789, in urlopen
    response = self._make_request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 495, in _make_request
    conn.request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 441, in request
    self.endheaders()
  File "/usr/local/lib/python3.10/http/client.py", line 1278, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/local/lib/python3.10/http/client.py", line 1038, in _send_output
    self.send(msg)
  File "/usr/local/lib/python3.10/http/client.py", line 976, in send
    self.connect()
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 279, in connect
    self.sock = self._new_conn()
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 214, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7f01780d21a0>: Failed to establish a new connection: [Errno 111] Connection refused

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 667, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 843, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/retry.py", line 519, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='metadata.google.internal', port=80): Max retries exceeded with url: /computeMetadata/v1/instance/attributes/tpu-env (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f01780d21a0>: Failed to establish a new connection: [Errno 111] Connection refused'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/usr/local/lib/python3.10/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/workspace/vllm/vllm/engine/multiprocessing/engine.py", line 390, in run_mp_engine
    engine = MQLLMEngine.from_engine_args(engine_args=engine_args,
  File "/workspace/vllm/vllm/engine/multiprocessing/engine.py", line 139, in from_engine_args
    return cls(
  File "/workspace/vllm/vllm/engine/multiprocessing/engine.py", line 78, in __init__
    self.engine = LLMEngine(*args, **kwargs)
  File "/workspace/vllm/vllm/engine/llm_engine.py", line 335, in __init__
    self.model_executor = executor_class(
  File "/workspace/vllm/vllm/executor/ray_tpu_executor.py", line 39, in __init__
    super().__init__(*args, **kwargs)
  File "/workspace/vllm/vllm/executor/executor_base.py", line 47, in __init__
    self._init_executor()
  File "/workspace/vllm/vllm/executor/ray_tpu_executor.py", line 51, in _init_executor
    self._init_workers_ray(placement_group)
  File "/workspace/vllm/vllm/executor/ray_tpu_executor.py", line 198, in _init_workers_ray
    self._run_workers("load_model",
  File "/workspace/vllm/vllm/executor/ray_tpu_executor.py", line 264, in _run_workers
    driver_worker_output = self.driver_worker.execute_method(
  File "/workspace/vllm/vllm/worker/worker_base.py", line 465, in execute_method
    raise e
  File "/workspace/vllm/vllm/worker/worker_base.py", line 456, in execute_method
    return executor(*args, **kwargs)
  File "/workspace/vllm/vllm/worker/tpu_worker.py", line 111, in load_model
    self.model_runner.load_model()
  File "/workspace/vllm/vllm/worker/tpu_model_runner.py", line 151, in load_model
    model = get_model(
  File "/workspace/vllm/vllm/model_executor/model_loader/__init__.py", line 19, in get_model
    return loader.load_model(model_config=model_config,
  File "/workspace/vllm/vllm/model_executor/model_loader/loader.py", line 398, in load_model
    model = _initialize_model(model_config, self.load_config,
  File "/workspace/vllm/vllm/model_executor/model_loader/loader.py", line 175, in _initialize_model
    return build_model(
  File "/workspace/vllm/vllm/model_executor/model_loader/loader.py", line 160, in build_model
    return model_class(config=hf_config,
  File "/workspace/vllm/vllm/model_executor/models/llama.py", line 515, in __init__
    self.model = LlamaModel(config,
  File "/workspace/vllm/vllm/compilation/decorators.py", line 71, in __init__
    old_init(self, *args, **kwargs)
  File "/workspace/vllm/vllm/model_executor/models/llama.py", line 305, in __init__
    self.start_layer, self.end_layer, self.layers = make_layers(
  File "/workspace/vllm/vllm/model_executor/models/utils.py", line 419, in make_layers
    [PPMissingLayer() for _ in range(start_layer)] + [
  File "/workspace/vllm/vllm/model_executor/models/utils.py", line 420, in <listcomp>
    maybe_offload_to_cpu(layer_fn(prefix=f"{prefix}.{idx}"))
  File "/workspace/vllm/vllm/model_executor/models/llama.py", line 307, in <lambda>
    lambda prefix: LlamaDecoderLayer(config=config,
  File "/workspace/vllm/vllm/model_executor/models/llama.py", line 217, in __init__
    self.self_attn = LlamaAttention(
  File "/workspace/vllm/vllm/model_executor/models/llama.py", line 170, in __init__
    self.attn = Attention(
  File "/workspace/vllm/vllm/attention/layer.py", line 85, in __init__
    self.impl = impl_cls(num_heads, head_size, scale, num_kv_heads,
  File "/workspace/vllm/vllm/attention/backends/pallas.py", line 122, in __init__
    if torch_xla.tpu.version() < 4:
  File "/usr/local/lib/python3.10/site-packages/torch_xla/_internal/tpu.py", line 187, in version
    env = get_tpu_env()
  File "/usr/local/lib/python3.10/site-packages/torch_xla/_internal/tpu.py", line 181, in get_tpu_env
    metadata = _get_metadata('tpu-env')
  File "/usr/local/lib/python3.10/site-packages/torch_xla/_internal/tpu.py", line 89, in _get_metadata
    resp = requests.get(path, headers={'Metadata-Flavor': 'Google'})
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 700, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='metadata.google.internal', port=80): Max retries exceeded with url: /computeMetadata/v1/instance/attributes/tpu-env (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f01780d21a0>: Failed to establish a new connection: [Errno 111] Connection refused'))

I've tried manually curling the metadata endpoint and ensuring that all the results are set to env vars, but I still get this exception.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

I am also facing this issue

I think it has to do with ray workers don't have access to metadata in gcp
Temporary workaround is this ,
change it from line 112 in pallas.py
`        #if torch_xla.tpu.version() < 4:
#    raise NotImplementedError("TPU version must be 4 or higher.")
    self.megacore_mode = None
    #tpu_env = torch_xla.tpu.get_tpu_env()
    #tpu_type = (tpu_env.get("ACCELERATOR_TYPE", None)
    #            or tpu_env.get("TYPE", None)
    #            or tpu_env.get("TPU_ACCELERATOR_TYPE", None))
    #assert tpu_type is not None
    #tpu_type = tpu_type.lower()

    #if "lite" not in tpu_type:
    #    if self.num_kv_heads % 2 == 0:
     #       self.megacore_mode = "kv_head"
    #    else:
            # NOTE(woosuk): If the batch size is not a multiple of 2, the
            # megacore mode will be None.
    self.megacore_mode = "batch"

`

Something in the last few PRs (or on Google's end idk) changed and now llama3-8b and a mistral/nemo merge 12b both run fine on v5litepod-8s.


Support remote URLs for logo

For our project, we have a dedicated CDN for static resources that is shared across multiple properties. E.g. for our logo we use:
https://static.bokeh.org/branding/logos/bokeh-logo.svg
It does not seem possible to configure html_logo with a full URL. (The value ends up appended to the project base URL as if it was a relative path, even if it starts with "http"). Currently, in order to use our static CDN log, we have to manually override navnar-logo.html as below:

We would like to reduce the amount of customization (especially template customization) over time. Adding support for logo to use a remote URL would let us drop this template customization entirely.

I had the same problem in my the documentation of one of my projects: https://github.com/12rambau/sepal_ui
When I asked the question of using external resources on SO I was said that it could lead to security issues.
Then I added 1 extra line in my conf.py file to download it from the url at each build:
# -- Load the images from the master sepal-doc ---------------------------------
urlretrieve(
    "https://raw.githubusercontent.com/openforis/sepal-doc/master/docs/source/_images/sepal.png",
    "_image/dwn/sepal.png",
)
urlretrieve(
    "https://raw.githubusercontent.com/openforis/sepal-doc/master/docs/source/_images/favicon.ico",
    "_image/dwn/favicon.ico",
)

That avoids the custom template. (note that the content of the dwnfolder is ignored by git.
That's a patch before we find a solution in the theme.

@12rambau preference, of course, but that's not really an improvement over a custom template to me. In particular, if the central resource changes, then none of the previously published docs will pick up the change since they have vendored the image.


[HOLD for payment 2024-02-07] Focus mode modal can show incorrectly when signing up for a new account after logging out of an existing account.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
Action Performed:

Signed up for expensify.com using a gmail account
Get directed to new.expensify.com (good :white_check_mark: )

Expected Result:
We should only show this modal for users who have more than 30 chats.
Actual Result:
Saw [the focus mode] modal first (bad :x:)
This was a brand new account and I had only one chat.
Workaround:
👍
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014e891ac070cbe788
Upwork Job ID:<PHONE_NUMBER>725543424
Last Price Increase: 2024-01-19

I'm guessing I'm here for the C+ payment of the review?

Yes. I've got a PR in the works already.

@c3024 reviewed the associated PR.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x]  [@c3024] The PR that introduced the bug has been identified. Link to the PR: #30664
[x]  [@c3024] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: The present fix is made by the same PR author so not commented.
[x]  [@c3024] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: Not started because this could not have been caught earlier.
[x]  [@c3024] Determine if we should create a regression test for this bug. Yes
[x]  [@c3024] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal
Tests here are suitable regression tests.

Tests
Verify focus mode switch still works and there are no regressions

Ensure signed out of both OldDot and NewDot
Sign into a NewDot account that has 30+ chats (must be a new account)
Verify the focus mode alert still works correctly

Verify focus mode modal is not incorrectly showed for brand new account after signing out of large account

Ensure signed out of both OldDot and NewDot
Sign into an account that has many hundreds of chats in NewDot
Sign out
Create new account on a public domain in OldDot
Verify you are redirected to NewDot
Verify the Focus mode modal does not show incorrectly

👍 or 👎

Regression steps look good, proposed they be added in https://github.com/Expensify/Expensify/issues/366901.
Payment due on Wednesday.

Payment summary:
C+: @c3024 to be paid $500 via manual request.

We're all done here, closing this one out.

I'm not eligible for manual requests yet. I can only be paid on Upwork. @bfitzexpensify

Gotcha - I've just made the Upwork job public, can you apply to it and let me know when you've done so?

Thanks. Applied. Here is my Upwork profile as well.
https://www.upwork.com/freelancers/~0105555e2f227dbf47

Thanks, all paid out - now we're done


[RELAY][OP] Fix doc of strided_slice

According to the topi computation, I believe the end index should be exclusive.
cc @tqchen @siju-samuel

Thanks @vinx13 this is merged


GDB Error 1 in Visual Studio Code on Windows

Clicking the bottom debug icon in Visual Studio Code produces an error:
[proc] Executing command: "C:/Program Files/Raspberry Pi/Pico SDK v1.5.1/gcc-arm-none-eabi/bin/arm-none-eabi-gdb.exe" --version
[proc] The command: "C:/Program Files/Raspberry Pi/Pico SDK v1.5.1/gcc-arm-none-eabi/bin/arm-none-eabi-gdb.exe" --version exited with code: 1
[proc] Executing command: gdb --version
[proc] The command: gdb --version exited with code: 1

Are the forward slashes correct?

The forward slashes are correct, but you need to use the configured profile for debugging, so the debug button on the bottom bar does not work. Our VS Code configuration hides that button, did you follow the steps in the tutorial to set that up and start debugging?

The VS Code configuration (i.e. .vscode folder) was supplied with the SDK and installed by pico-setup-windows-x64-standalone.exe.
A zip file named pico-examples.zip contained the .vscode folder.
I should not that there was a step in the SDK installation process that failed.  I attempted to install the examples to the C:\Program Files folder instead of the C:\Users\  Documents directory.  The installer failed to extract the examples from github due to permissions error.

I reinstalled.  Looks like the SDK installer does the same thing I did manually which was unzip pico-examples.zip to the documents directory.
Are you saying the debug icon should not be here?

I've come to the conclusion that CMake is a mess.  Sure it works however debugging issues like this is a nightmare.  There is so much cache and crap left behind that pin-pointing the root cause of an issue is pretty much impossible.  Even resetting and clearing doesn't get rid of everything.

Yes, you're not supposed to use the debug button on the bottom status bar, the .vscode configuration that we provide is supposed to hide that for this reason. I'll see if I can replicate this behavior.
The recommended way to start debugging is documented in the tutorial (which is included and installed as readme.txt if you use this installer.)


Dragover events won't fire in Chrome

I got a problem with react-dnd.  I can't run this tutorial-app in my Chrome. The dragover events doen't seem to be triggered. But it works well with Firefox and Sarfri. I got confused. Here's the version of my Chrome: Version 56.0.2924.87 (64-bit).
What is the reason for that and how to fix it?

looks like its broken. I don't see it working in Firefox nor Safari.
@cc2victoria - are you able to move the horse to different boxes? (in Firefox and Safari)

Mine's same version chrome work well

@theTechie  Yes, I can move horse to different boxes in FireFox.

Yes. Looks like it's fixed now. It should also work in Chrome now!
Please close this if it's no more a issue.

@theTechie Sorry, it still doesn't work in my Chrome or my colleague's. I can't even see the horse picture or the color in my chessboard when I drag the white horse.

Something very strange. We are using the exact same version of Chrome and its working for me.
Could you check for any errors in the browser console ?
It will be helpful if you can share a GIF when its breaking for you.

@theTechie There's no error in the browser console. But with you proposal, I suddenly got it. When I drag the horse, something else showed up in my page.
I have installed a extension named Dragdis in my Chrome. And it seems to have some conflicts with react-dnd. I think it may be the horse picture which fired both. So I removed the Dragdis extension. Now, the demo works well in my chrome as well as my own project.
You are very heplpful, thank you very much!


update to react 17

plz update to react 17 to still be relevant
also the older react-boilerplate wy not give deprecation notice and link to this project?

It is not always best to update to the latest version. You can easily do so in 5 seconds in your project. Sometimes newer versions come with problems and it's better for the sake of the majority to stay a bit behind and safe. I will consider it after some time.
I wanted to do so in the other repo but after huge discussions, I decided not to proceed.

@Can-Sahin "Sometimes newer versions come with problems and it's better for the sake of the majority to stay a bit behind and safe. "
This not a boilerplate for new projects?
So wouldnt you want it to be most up to date not already use older stuff when you start a project knowing you have technical depts to begin with in the future?  It makes no sense to me what so ever to use older stuff when starting a project.. unless there are big bugs.. making you stop updating to a newer package...
you dont want to start a project updating all dependencies ... also how you know old dependencies are still safe to use?
the dont have any snyk.io vulnerability issues.. (something you might want to run nightly to give insights)
my 2 cents on the matter

Agree and disagree :) See this : https://github.com/react-boilerplate/react-boilerplate-cra-template/issues/86
Here the new version had a bug and I updated it before waiting that much. It takes a while for the community to find bugs -> report -> get fixed. That's what I saw in the past years and concluded that its safer to stay some months behind.
You don't need to update dependencies though. Dependency management is a tedious task and definitely project-specific. I'm trying to find a safe middle ground only. Recently, I switched to npm7(5 months old) and it took us 1 day to workaround a tiny bug of it in cli.

"Dependency management is a tedious task" that i can agree up on for sure.


Add initial .appveyor.yml

This is the follow-up companion to https://github.com/docker-library/docker/pull/75. :+1:

:muscle: :metal:


Don't overwrite a higher score

Check existing score and only overwrite if new score is higher.

Did not fix. Will examine later in a proper dev environment.


Chatgpt Agent and Agent Do Task Function

TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[x] Feature
[x] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
You can test the agent without openai key.
from flytekit import task, workflow
from flytekitplugins.chatgpt import ChatGPTDispatcher
# from flytekit.dispatcher.chatgpt_dispatcher import ChatGPTDispatcher

chatgpt_job = ChatGPTDispatcher(
    name="chatgpt",
    config={
        "openai_organization": "org-NayNG68kGnVXMJ8Ak4PMgQv7",
        "chatgpt_conf": {
            "model": "gpt-3.5-turbo",
            # 'messages': [{'role': 'user', 'content': 'Say this is a test!'}],
            "temperature": 0.7,
        },
    },
)

@task()
def t1(s: str) -> str:
    s = "Repsonse: " + s
    return s

@workflow
def wf() -> str:
    message = chatgpt_job(message="hi")
    return t1(s=message)

if __name__ == "__main__":
    print(wf())

Tracking Issue
https://github.com/flyteorg/flyte/issues/3936

What do you think of moving this into a plugin? I'm a bit worried that a chat gpt integration into flytekit itself is a scope creep. Other task types for specific providers like aws or gcp bigquery are not integrated into the core library to keep it lean but are their own package.

Thanks for the good question.
I think the reason is because of the file sensor feature here.
https://github.com/flyteorg/flytekit/pull/1790
I have discussed this with @pingsutw before, and we think that the dispatcher can be used for lots of cases such as LangChain, OpenAPI and other do-task cases.

We can add BaseDispatcher into flytekit, but ChatGPT task should definitely added to the plugin.

We can add BaseDispatcher into flytekit, but ChatGPT task should definitely added to the plugin.

This proposal sounds good to me 👍

btw, @fg91 If you come up with a better name than dispatcher, let us know.  We are not good at naming things 😂

btw, @fg91 If you come up with a better name than dispatcher, let us know. We are not good at naming things 😂

At the example of this plugin: Do I understand it correctly that instead of launching a task pod in a workflow to send the request to the OpenAi API, a long living agent pod makes the respective request asynchronously?

btw, @fg91 If you come up with a better name than dispatcher, let us know. We are not good at naming things 😂

At the example of this plugin: Do I understand it correctly that instead of launching a task pod in a workflow to send the request to the OpenAi API, a long living agent pod makes the respective request asynchronously?

Yes, you are right!

btw, @fg91 If you come up with a better name than dispatcher, let us know. We are not good at naming things 😂

At the example of this plugin: Do I understand it correctly that instead of launching a task pod in a workflow to send the request to the OpenAi API, a long living agent pod makes the respective request asynchronously?

Yes, you are right!

How about one of the following more descriptive (but also longer) names: ExternalApiRequestDispatcher, RequestDispatcher, ExternalApiTask, AsyncRequestTask, ApiRequestTask?

btw, @fg91 If you come up with a better name than dispatcher, let us know. We are not good at naming things 😂

At the example of this plugin: Do I understand it correctly that instead of launching a task pod in a workflow to send the request to the OpenAi API, a long living agent pod makes the respective request asynchronously?

Yes, you are right!

How about one of the following more descriptive (but also longer) names: ExternalApiRequestDispatcher, RequestDispatcher, ExternalApiTask, AsyncRequestTask, ApiRequestTask, AgentApiRequestTask?
I personally tend towards a name that contains "task" as that is a familiar concept in Flyte whereas "dispatcher" is new.

I think ExternalApiTask or AgentExternalApiTask is great!
@pingsutw how do you think?

no changes here yet?

no changes here yet?

I will update it later.
I originally thought that only a certain part needed to be changed.
I will update the syncAgentBase class, xd

I am considering the following changes.

rename api_task to sync_agent_task.
rename task_executor to sync_agent_executor.
rename external_api_task to sync_agent_task

This PR will be derprecated.
We can refer to this PR for generic sync agent executor in the future.


refactor(tests): remove duplicate relay environment mocks #FF

The type of this PR is: Enhancement

Description
This removes duplicate relay environment mocks in some files cause we already have this in setupJest.tsx

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

remove duplicate relay environment mocks from tests - gkartalis

This PR contains the following changes:

Dev changes (remove duplicate relay environment mocks from tests - gkartalis)

  Generated by :no_entry_sign: dangerJS against 4d898414bff96400e34bad69ccb1da661e7957c5


Update license to 2019

There have been no copyrightable changes in 2019; the license year is not 2019 yet.


Add logic to prevent fetching from invalid test matrices

Fixes #
Checklist

[ ] Documented
[ ] Unit tested
[ ] release_notes.md updated

Codecov Report

Merging #693 into master will decrease coverage by 0.03%.
The diff coverage is 64.28%.

@@             Coverage Diff              @@
##             master     #693      +/-   ##
============================================
- Coverage     77.24%   77.21%   -0.04%
  Complexity      639      639
============================================
  Files           112      112
  Lines          2580     2594      +14
  Branches        366      367       +1
============================================
+ Hits           1993     2003      +10
- Misses          350      354       +4
  Partials        237      237


Exception when using Horizontal.fullWidth() with series with just 1 entry

How to reproduce
Set up a chart with a full width horizontal layout, and provide a series with just one entry, for example:
@Preview(widthDp = 400)
@Composable
fun Chart(
) {
    val entries = listOf(
        Entry(x = 0, y = 1),
    )
    CartesianChartHost(
        chart = rememberCartesianChart(
            rememberLineCartesianLayer(),
            startAxis = rememberStartAxis(),
            bottomAxis = rememberBottomAxis(),
        ),
        model = CartesianChartModel(
            models = listOf(
                LineCartesianLayerModel.build {
                    series(
                        x = entries.map { it.x },
                        y = entries.map { it.y }
                    )
                },
            )
        ),
        horizontalLayout = HorizontalLayout.fullWidth(
            unscalableStartPadding = 10.dp,
            unscalableEndPadding = 10.dp,
        ),
    )
}

Observed behavior
This results in an exception:
    java.lang.IllegalArgumentException: Cannot coerce value to an empty range: NaN..0.0.
    at kotlin.ranges.RangesKt___RangesKt.coerceIn(_Ranges.kt:1470)
    at com.patrykandpatrick.vico.compose.chart.scroll.VicoScrollState.setValue(VicoScrollState.kt:75)
    at com.patrykandpatrick.vico.compose.chart.scroll.VicoScrollState.setMaxValue$compose_release(VicoScrollState.kt:85)
    at com.patrykandpatrick.vico.compose.chart.scroll.VicoScrollState.update$compose_release(VicoScrollState.kt:149)
    at com.patrykandpatrick.vico.compose.chart.CartesianChartHostKt$CartesianChartHostImpl$6.invoke(CartesianChartHost.kt:255)
    at com.patrykandpatrick.vico.compose.chart.CartesianChartHostKt$CartesianChartHostImpl$6.invoke(CartesianChartHost.kt:224)
    at androidx.compose.ui.draw.DrawBackgroundModifier.draw(DrawModifier.kt:116)

Adding one more entry, or removing the layout results in no crash.
Expected behavior
No exception.
Vico version(s)
Latest unstable version
Android version(s)
14
Additional information
This is with alpha 9. The same code worked with alpha 8 and earlier.

Hello! Thanks for the report. We’ll look into this.

This crash occurs when there’s no scalable content. You can work around the issue by adding the following Zoom implementation to your project and setting initialZoom to Zoom.max(Zoom.static(), ContentZoom) for scrollable charts, and to ContentZoom for charts with scroll disabled. We’ll be releasing a hotfix soon.
val ContentZoom =
    Zoom { context, horizontalDimensions, bounds ->
        val scalableContentWidth = horizontalDimensions.getScalableContentWidth(context)
        if (scalableContentWidth == 0f) {
            1f
        } else {
            (bounds.width() - horizontalDimensions.unscalablePadding) / scalableContentWidth
        }
    }

Vico 2.0.0 Alpha 10 fixes this. Cheers!

Thanks, alpha 10 works perfectly!

I’m glad to hear that! You’re welcome, and thanks again for letting us know about the bug.


fix(router): Handle comma delimited X-Forwarded-For values

Fixes #167 (at least partially; we cannot necessarily fix idiosyncrasies of GCP)
To test this, you'd be best off using GKE.  The TCP load balancer that k8s provisions for you when you install workflow would need to be replaced with an HTTP load balancer.
At that point, the router will receive inbound requests with multi-valued X-Forwarded-For headers.
Router should have it's router.deis.io/nginx.proxyRealIpCidr configures as follows:
router.deis.io/nginx.proxyRealIpCidr: <IP_ADDRESS>/8,<load balancer public IP>/32

Requests to any all applications should yield access log messages form the router that contain your public IP.
See #167 for more information.

@krancour code LGTM. https://github.com/deis/router/pull/168#discussion_r60940854 is not blocking, just wanted to point it out. Added LGTM1 and removed in progress

@krancour do you have an HTTP load balancer setup handy for testing this? How did you set yourself up on GKE?

my comment triggered jenkins. Sorry about that!

my comment triggered jenkins. Sorry about that!

No worries.  It happens.  fwiw, I actually opened an issue against the Jenkins Github Pull Request Builder plugin with relevant logs.  I've heard nothing back so far.
Re: Testing this in GKE, it's quite a pain to set up.  I'll set it up and point you in its direction.

@bacongobbler... an easier way for you to test this might be as follows:

Deploy locally on vagrant. (This is so you're operating somewhere where there's no load balancer between you and the router.  You will play the role of the LB. 😄)

Deploy deis-router from this branch: $ make deploy

Add the following annotation to the deis-router RC.  This will relax some constraints to allow IP spoofing:

router.deis.io/nginx.proxyRealIpCidrs: <IP_ADDRESS>/0

4. curl an app, whilst explicitly setting the `X-Forwarded-For-Header`:

    ```
$ curl -H "X-Forwarded-For: <IP_ADDRESS>,<IP_ADDRESS>" <any valid app URL; event the controller>

Check router's access logs.  If working, requests should appear to have originated from <IP_ADDRESS>.

$ kubectl logs  --namespace=deis

LGTM:
><> curl -H "X-Forwarded-For: <IP_ADDRESS>,<IP_ADDRESS>" deis.<IP_ADDRESS>.xip.io

Not Found

The requested URL / was not found on this server.

><> kd logs deis-router-6acz9
2016/05/02 18:12:59 INFO: Starting nginx...
2016/05/02 18:12:59 INFO: nginx started.
2016/05/02 18:12:59 INFO: Router configuration has changed in k8s.
2016/05/02 18:12:59 INFO: Reloading nginx...
2016/05/02 18:12:59 INFO: nginx reloaded.
[02/May/2016:18:13:39 +0000] - <IP_ADDRESS> - - - 404 - "GET / HTTP/1.1" - 415 - "-" - "curl/7.38.0" - "~^deis\x5C.(?<domain>.+)$" - <IP_ADDRESS>:80 - deis.<IP_ADDRESS>.xip.io - 0.006 - 0.006

@bacongobbler thanks for taking the time.  fwiw, not need to delete the router pod to apply configs in the future.  Router is always watching its own annotations and applying new config when it spots diffs.

I figured that the pod annotations are separate from the rc annotations so I just wanted to make sure. Thanks for the tip!


http://useraccounts-ratchet.meteor.com doesn't seem to exist

Am getting an error at http://useraccounts-ratchet.meteor.com "This site is under construction. Come back later please ;-)" - I have been several times so I don't think it's meteor.com lazy load...

It was me putting the message "This site is under construction. Come back later please ;-)".
The this is we still miss a boilerplate for ratchet... :-(
Would you be willing to create it?
In case, just start of from one of the others and make a PR to add it to the repo.
That would be much appreciated!
Thanks,
Luca

deployed!


uploader: if clicking the X on one tag will remove all tags

This is due to the nature of the complex query input, this should be fixed with that


Make MyBoolVariable unsigned ?

I am actually not sure if the example for MyBoolVariable should use unsigned types (intead of signed) types. You might try it.

I tried it and it looks like it makes no difference to Matlab.
As long as you set the MAT_F_LOGICAL flag Matlab (2015b) evaluate it as a bool, means:
value = 0 -> false
value != 0 -> true
This behavior works with INT as parameter, too:
int intbool = 42;
matvar_t *variable = Mat_VarCreate(fieldname, MAT_C_INT16, MAT_T_INT16, 2, dim, &intbool, MAT_F_LOGICAL);

Evaluated as bool with value true in Matlab (Octave ignores the flag -> INT16 -> 42)
When reading the value with matio you have to check the flag and then evaluate with the expression given above (value != 0)
Maybe it would be nice to add automatic convertion to matio.