firebase_auth Error on IOS

After adding the firebase_auth dependency to my pubspec file I'm getting the following IOS build error:
[   +1 ms] Failed to build iOS app
[        ] Error output from Xcode build:
           ↳
[        ] ** BUILD FAILED **

[        ] Xcode's output:
           ↳
[        ] === BUILD TARGET firebase_auth OF PROJECT Pods WITH CONFIGURATION Debug ===
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:5:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/Flutter.h:37:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterAppDelegate.h:11:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:80:42: warning: 'UIUserNotificationSettings' is only available on iOS 8.0 or newer [-Wunguarded-availability]
               didRegisterUserNotificationSettings:(UIUserNotificationSettings*)notificationSettings;
                                                    ^
           In module 'UIKit' imported from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Target Support Files/firebase_auth/firebase_auth-prefix.pch:2:
           /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.3.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIUserNotificationSettings.h:43:12: note: 'UIUserNotificationSettings' has been explicitly marked partial here
           @interface UIUserNotificationSettings : NSObject
                      ^
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:5:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/Flutter.h:37:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterAppDelegate.h:11:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:79:1: note: annotate 'application:didRegisterUserNotificationSettings:' with an availability attribute to silence this warning
           - (void)application:(UIApplication*)application
           ^
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:95:44: warning: 'UIBackgroundFetchResult' is only available on iOS 7.0 or newer [-Wunguarded-availability]
                     fetchCompletionHandler:(void (^)(UIBackgroundFetchResult result))completionHandler;
                                                      ^
           In module 'UIKit' imported from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Target Support Files/firebase_auth/firebase_auth-prefix.pch:2:
           /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.3.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIApplication.h:76:29: note: 'UIBackgroundFetchResult' has been explicitly marked partial here
           typedef NS_ENUM(NSUInteger, UIBackgroundFetchResult) {
                                       ^
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:5:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/Flutter.h:37:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterAppDelegate.h:11:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:93:1: note: annotate 'application:didReceiveRemoteNotification:fetchCompletionHandler:' with an availability attribute to silence this warning
           - (BOOL)application:(UIApplication*)application
           ^
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:140:58: warning: this block declaration is not a prototype [-Wstrict-prototypes]
                                 completionHandler:(nonnull void (^)())completionHandler;
                                                                    ^
                                                                     void
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:148:49: warning: 'UIBackgroundFetchResult' is only available on iOS 7.0 or newer [-Wunguarded-availability]
               performFetchWithCompletionHandler:(void (^)(UIBackgroundFetchResult result))completionHandler;
                                                           ^
           In module 'UIKit' imported from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Target Support Files/firebase_auth/firebase_auth-prefix.pch:2:
           /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.3.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIApplication.h:76:29: note: 'UIBackgroundFetchResult' has been explicitly marked partial here
           typedef NS_ENUM(NSUInteger, UIBackgroundFetchResult) {
                                       ^
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:5:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/Flutter.h:37:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterAppDelegate.h:11:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:147:1: note: annotate 'application:performFetchWithCompletionHandler:' with an availability attribute to silence this warning
           - (BOOL)application:(UIApplication*)application
           ^
           /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:70:41: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                   signInAnonymouslyWithCompletion:^(FIRAuthDataResult *dataResult, NSError *error) {
                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:7:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Private/Firebase/Firebase.h:13:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FirebaseAuth.h:4:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FIRAuth.h:493:73: note: passing argument to parameter 'completion' here
           - (void)signInAnonymouslyWithCompletion:(nullable FIRAuthResultCallback)completion;
                                                                                ^
           /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:103:41: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                                        completion:^(FIRAuthDataResult *dataResult, NSError *error) {
                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:7:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Private/Firebase/Firebase.h:13:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FirebaseAuth.h:4:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FIRAuth.h:590:61: note: passing argument to parameter 'completion' here
                            completion:(nullable FIRAuthResultCallback)completion;
                                                                       ^
           /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:131:37: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                                    completion:^(FIRAuthDataResult *dataResult, NSError *error) {
                                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:7:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Private/Firebase/Firebase.h:13:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FirebaseAuth.h:4:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FIRAuth.h:372:57: note: passing argument to parameter 'completion' here
                        completion:(nullable FIRAuthResultCallback)completion;
                                                                   ^
           /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:188:43: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                                          completion:^(FIRAuthDataResult *dataResult, NSError *error) {
                                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.m:5:
           In file included from /Users/aroba/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.5.10/ios/Classes/FirebaseAuthPlugin.h:7:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Private/Firebase/Firebase.h:13:
           In file included from /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FirebaseAuth.h:4:
           /Users/aroba/projects/sweetnest/sweetnest_app/ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FIRAuth.h:537:63: note: passing argument to parameter 'completion' here
                              completion:(nullable FIRAuthResultCallback)completion;
                                                                         ^
           4 warnings and 4 errors generated.

The Android build works.

I think you're experiencing this error https://github.com/flutter/flutter/issues/17721.  I ran in to this issue last night and was able to resolve it by following the fix mentioned in the thread.

Indeed same issue.
I followed the suggested solution here.


Test interlock

Build status:

Linux
Windows

CI failed due to this bug: https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/305

Build status:

Linux
Windows

Build status:

Linux
Windows


fix max_blocklength_observed

The code that update max_blocklength_observed was accidentally got deleted. And this PR fixed that.

close #4362


Trying to create {Module}RouteProvider

Hello ;)
I'm trying to migrate my application from nwidart module to yours. I have a few questions if you may:

Have you developed any large applications with this package?
How can I use my custom RouteProvider to add the API prefix to the routes in the account-routes-api.php file?

I've tried changing the composer.json file for the account module as shown in the screenshot, and I've also created a custom RouteProvider in the Wdc\Account\Providers\AccountRouteProvider file. However, the package still doesn't add the API prefix to the routes.

account module composer.json

{
  "name": "wdc/account",
  "description": "",
  "type": "library",
  "version": "1.0",
  "license": "proprietary",
  "require": {},
  "autoload": {
    "psr-4": {
      "Wdc\\Account\\": "app/",
      "Wdc\\Account\\Tests\\": "tests/",
      "Wdc\\Account\\Database\\Factories\\": "database/factories/",
      "Wdc\\Account\\Database\\Seeders\\": "database/seeders/"
    }
  },
  "minimum-stability": "stable",
  "extra": {
    "laravel": {
      "providers": [
        "Wdc\\Account\\Providers\\AccountServiceProvider",
        "Wdc\\Account\\Providers\\AccountRouteProvider"
      ]
    }
  }
}

Then in the Wdc\Account\Providers;\AccountRouteProvider:

<?php

namespace Wdc\Account\Providers;

use Illuminate\Foundation\Support\Providers\RouteServiceProvider as ServiceProvider;
use Illuminate\Support\Facades\Route;

class AccountRouteProvider extends ServiceProvider
{
    public function register()
    {
    }

    public function boot()
    {
        //dd(module_path('account/routes/account-routes-api.php'));
        $this->routes(function () {
            Route::prefix('api')
                ->middleware('api')
                ->namespace('Wdc\Account\Http\Controllers')
                ->group(module_path('account/routes/account-routes-api.php'));
        });
        //parent::boot();
    }
}

helpers.php
if (! function_exists('module_path')) {
    function module_path($path = ''): string
    {
        $path = $path
            ? DIRECTORY_SEPARATOR.$path
            : $path;

        $path = 'app-modules'.$path;

        return base_path($path ? DIRECTORY_SEPARATOR.$path : $path);
    }
}

The package recognizes account-routes-api, but it doesn't add the API prefix to them. I've verified that AccountRouteProvider is being read by using dd('something').
Thanks for your help´;)

I've changed the code to
<?php

namespace Wdc\Account\Providers;

use Illuminate\Foundation\Support\Providers\RouteServiceProvider as ServiceProvider;
use Illuminate\Support\Facades\Route;

class AccountRouteProvider extends ServiceProvider
{
    public function register()
    {
    }

    public function boot()
    {
        Route::prefix('api')
            ->middleware('api')
            ->namespace('Wdc\Account\Http\Controllers')
            ->group(module_path('account/routes/account-routes-api.php'));
    }
}

and it works now.
I don't know why because I've used RouteServiceProvider as a  base 😄

Hey, sorry—somehow missed this!
This package is used in a project with about 30 modules, and ~300,000 lines of application code. I don't know if that's big or small to you…
As for the routes issue. I think what's happening is that modular auto-loads anything in a module's routes/ directory for you. Just get rid of the whole service provider and put the Route::prefix… portion inside of account-routes-api.php. I think that should solve it for you!


Error running

When trying to run the program, I get this error:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Driver(DriverError(CUDA_ERROR_UNSUPPORTED_PTX_VERSION, "the provided PTX was compiled with an unsupported toolchain."))', /home/opfromthestart/.cargo/git/checkouts/dfdx-318e6e5ad83eea79/19da9fe/src/tensor_ops/select_and_gather/mod.rs:155:30

You need to update to cuda driver 530 for this version, 525 will not work


Find examples for elements of the standard

What should a README contain?
What should a CONTRIBUTING file contain?
What should a GOVERNANCE file contain?
What should a ChangeLog file contain?
Find two examples for each file showing "minimum to meet the bar" and "best of class" real-world examples.

Would we want "minimum to meet the bar" to be a file that only meets MUST requirements (and has good reasons to not meet the SHOULDs) or would it be a file that meets both MUST and SHOULD requirements and doesn't have the need for explanations/exceptions?

To me, a codebase having good circumstantial reasons for SHOULD exceptions is a different axis than understanding completeness between minimum-to-meet and best-of-class.
Thus, I think ideally it would be all MUST and all SHOULD, to avoid confusing those two dimensions.
However, if there happen to be SHOULD requirements which are explained as exceptions, I think that could be fine.
But, let's see what we find.

I agree. And then logically, a "best of class" then meets all OPTIONAL as well, right?

Given that most OPTIONAL are things that we would recommend, then I'd say likely, yes.


VSCode is not per monitor DPI aware

VSCode Version: 1.0.0
OS Version: Windows 10

Steps to Reproduce:

Connect second monitor and adjust DPI such as the second monitor has different DPI than the main one
Drag Vscode window to the second monitor - UI becomes all blurry.

Possibly related to #1896 and also this upstream Chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=426656

Also happens on VSCode Version: 1.1.0-insider; OS: Windows 10 14332

@ivanz @Chococat could you try our latest insider release to see if this issue still reproduces?
You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

@bpasero Not as far as I can tell. You can easily test this without multiple monitors on Windows by simply changing the size of text etc. As far as I can tell the work is still in progress on the Chromium side here and is currently available in their dev and canary builds only.

Ok but good to know they plan for it because that means we will eventually also benefit.

@ivanz we recently updated Chrome again to 53 in our insider builds, maybe this improved?
You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

Got it, thanks 👍

This is bothering, I'm running Google Chrome 54, and it works perfectly but not the Electron which is running with Chromium 53. I opened issue to Electron https://github.com/electron/electron/issues/8062

Electron 1.5 fixes this by updating to Chrome 54.

Todays VS Code insider build comes with Electron 1.6.x, would be interesting to hear if this update solves this issue for anyone: http://code.visualstudio.com/Download#insiders

I am afraid not. I changed the DPI of one of my monitors and with build from sha 61753c029226f43c12124a75020b5d8b86c60700 and date 2017-04-10T07:15:44.057Z when I drag across monitors it remains blurry. :(

Electron still hasn't resolved this on their end, right? https://github.com/electron/electron/issues/5429

True that, would still be happy for more people trying out this new electron release.

@bpasero it does for me :) I have a Surface Book and the Explorer panel used to look super tiny, now it's fine.

The default scaling factor ("zoom") is now more sensible out of the box (guess DPI aware) indeed.

It's now system DPI aware, but not yet per-monitor DPI aware.

Recently I started a test project with Electron, and I found it's now Per-Monitor Aware.
I'm using Electron 1.6.10, Renderer 56.0.2924.87, while VS Code 1.12.2 is using Electron 1.6.6 and same Renderer 56.0.2924.87.

Maybe another upgrade required? It's really annoying.

I temporarily have a setup with mixed-DPI screens and it is really annoying that VS Code isn't per-monitor DPI aware.

This is supposedly fixed in Electron 1.6.7: https://github.com/electron/electron/releases/tag/v1.6.7
Enabled per-monitor DPI awareness in the app manifest. https://github.com/electron/electron/pull/8786

(any chance of updating to the latest 1.6.x in the near term?)

Updating to Electron 1.7.x is on the plan for July.

I just found a great workaround for this that might be helpful to others.  You will need a recent build of Windows 10.  Find the shortcut you use to launch Visual Studio Code, right click on it and choose Properties.  Go to the Compatibility tab, check the "Override high DPI scaling behavior" checkbox, and choose Scaling performed by "Application".  Close and restart Visual Studio Code.

Now, when you drag VS Code between monitors with different DPI, it will initially fail to resize the text.  So going from a high DPI screen to a low DPI screen, the text will apear huge.  But all you have to do is resize the window on the new monitor and the text will instantly size properly for the monitor and be perfectly sharp.

@kring I have heard that choose System (Enhanced) will also work.
But anyway, after a month of postpone, Electron 1.7.4 is now in the master branch, so Insider builds from today should be per-monitor DPI aware now, at least in theory.

@kring (and others), yeah let us know if the insiders build (e.g. from today) fixes this.

Yep today's Insiders looks great, no settings tweak or manual resize necessary.  Thanks!

Marking as verified based on @kring comment.

After testing many text editors, this and Notepad are the only ones I could find that have per monitor DPI awareness, which is a big help to me, with a 4K monitor next to a 1080p one. Since Notepad still doesn't support Unix style line endings, and I often create text files on Linux, MacOS, and Android, that was not an option. I'm now using Visual Studio Code as my text editor for all my personal notes and lists. I am not a programmer, but I'm very glad VS Code now works this way. Many thanks to the Electron team for fixing this and to you folks making it work with Code.

Windows 8.1 x64, VS Code 1.17.2 x64, external 20" HP monitor @1600x900, here is a screenshot with VS Code on the left and Notepad++ on the right.

This is basically unusable, too blurry. Will this ever be fixed? I really want to use VS Code.

@alper6 VS Code has not done that for a while on any of my computers. As I drag it between monitors with different DPIs, it rescales so that it never becomes blurry. What version are you using?

@jnm2 the version is 1.17.2. I periodically test every latest version to see if anything has improved.

Random thing to check: Have you by any chance maybe set a "Custom scaling" in Windows for the monitor or any monitor? I think that turns off lots of multi-DPI goodness in Windows and reverts back to legacy behavior (not 100% sure)

@ivanz I mentioned details about my system in first post, top of screenshot (is it not visible?)
It's not Windows 10, but Windows 8.1
I'm not using multi-monitor, it's set as "show desktop only on 2nd monitor", here's a shot:

There's also no custom scaling, those settings are all at default, here's a shot:

Your Notepad++ is super blurry in your first screenshot too.

I would like to reopen this issue.
VSCode seems to support 'per monitor aware scaling', at my surface.
Link to some explanation about scaling modes.
However, Tooltips and Dialog Boxes are still only system aware.


[feat] Move cursor to "staging" section when opening neogit

Hi,
Is your feature request related to a problem? Please describe.
In the past, when I opened neogit my cursor was immediately put on the unstaged section.
However, recent changes added following section at the top of the neogit window:

and now the cursor is no longer at the unstaged section when I enter neogit window, but it is on the top of the new section.

I think that the previous UX was better.
Describe the solution you'd like
Opening neogit window should put the cursor on the top of unstaged section (or next section if there are no unstaged files).

Describe alternatives you've considered
For now I am manually moving my cursor there as I don't know any motion that would come in handy in this particular case.
Another way would be to use the motion described here https://github.com/NeogitOrg/neogit/issues/651 though I don't always want to expand these files.

Yea, that was a small regression, now that the headers are interactable. Thanks for the heads up :)


Repl not loading svelte compiler

Description
The repl is not loading the svelte compiler

(This message never goes away)
I suspected this might be an issue with my laptop so I tried opening the same page on my phone but to no avail
Checking the devtools reveals two typeErrors

I don't know a lot about this stuff 😅
Reproduction
Visit https://svelte.dev/repl

Fixed itself? Sorry for wasting anyones time

https://github.com/sveltejs/svelte/issues/8010 This was a bug in Svelte 3.53.0 with the compiler not being able to run in the browser - it's been fixed in 3.53.1.


Various typos in the Eve language handbook

I like Eve very much and worked through the Quickstart and handbook. I noticed a number of typos, most of them only spelling, a few in code examples itself. While "it is low hanging fruit", correcting them will give new users an even better experience, so I list them here:
(1)  http://docs.witheve.com/handbook/is/
“The output shows that y is false while z is true.”
This is incorrect, the output is false and 10.
The code should be rewritten as:		[#div text: "z is {{z}}"]
(2) Typo:
http://docs.witheve.com/handbook/functions/
“Also like records, arguments are stateted explicitly.”

Should be:
“Also like records, arguments are stated explicitly.”
angles in randians
should be:
angles in radians
(3) tictacto.eve:
product of their values. This means that if i = {0, 1, 2} and j = {0, 1, 2}, then i x j = {(1, 1), (1, 2), ... (3, 2), (3, 3)}. These are exactly the indices we need for our grid!
Should be: This means that if i = {1, 2, 3} and j = {1, 2, 3},
(4) http://docs.witheve.com/handbook/records/
Records are the predominate datatype in Eve
Should be:
Records are the predominant
(5) http://docs.witheve.com/guides/style/
Still uses match instead of search in:
// Good
match
[...]
bind
[...]
(6) Quickstart example:
search
schools = [#school address]
students = [#student school: school.name]
bind @browser
[#div text: "{{students.name}} attends {{schools.name}} at {{address}}"]
students = [#student school: school.name]
should be:
students = [#student school: schools.name]
(7) Quickstart:
 celia = [name: "Celia"]

Uses the wrong double quotes character " from Word probably; which can give wrong program execution. This occurs at several places, it
(8) Quickstart:  This block selects all students, and uses and if-then expression to set the student's
Should be an
(9) Quickstart:: calculated-age = if student.age then student.age
else if student.grade then student.grade + 5
Why so complicated and not just:
calculated-age = if student.grade then student.grade + 5

(10) http://docs.witheve.com/handbook/records/
commit
[name: "Jeremey" spouse: [name: "Wendy"]]
"Jeremey" should be "Jeremy"
(11)  http://docs.witheve.com/handbook/not/
friends not invited to the party
friends = [#friend]
not(friends = [#invited])
should be: // friends not invited to the party
(12) http://docs.witheve.com/handbook/joins/:
joins allow you to relate two records
(13)
http://docs.witheve.com/handbook/math/floor/
search
y = floor[value: 34.2]
bind @browser
[#div text: value]
Should be:
bind @browser
[#div text: y]
(14)
http://docs.witheve.com/handbook/bind/
Display a formatted time:
match
[#time hours minutes seconds]
(am/pm, adjusted-hours) = if hours >= 12 then ("PM", hours - 12)
else if hours = 0 then ("AM", 12)
else ("AM", hours)
match should be search
(15) http://docs.witheve.com/handbook/functions/
[#fn #function argument1, ... argumentN, value1, ..., valueN]
Should this not be: 	[#fn #function argument1: value1, ... argumentN: valueN, return: value]
(16) http://docs.witheve.com/handbook/functions/
The example in Set Semantics is correct, but does not illustrate any function, merely string embedding. It belongs probably in the docs for sets.
(17) http://docs.witheve.com/handbook/if-then/
In the syntax format: (value, ..., valueN)  should be (value1, ..., valueN)
[@"my party" date]
Should be [#"my party" date]
burgers = if guest = [@Arthur] then 3
should be #Arthur   (2 x)
Explanations from Description should be written between the examples.
(18) http://docs.witheve.com/handbook/string-interpolation/
eacother  should be each other
(19)
http://docs.witheve.com/handbook/add/
then := will create   should be  then += will create
(20) http://docs.witheve.com/handbook/action-phase : page does not yet exist
(21) the link time on http://docs.witheve.com/handbook/datetime/ does not work
(22)
http://docs.witheve.com/handbook/math/
Arithemtic
Should be
Arithmetic
(23)
In Eve 0.2.2;  Idem fix, tan
(24) http://docs.witheve.com/handbook/math/mod/
The example code does not have an output because of   [#angle value]
Replace by value = 30 for example.
(25) http://docs.witheve.com/handbook/math/range/
The   {   }  notation is used for sets instead of ( )
y = {1, 3, 5, 6, 9} should be y = (1, 3, 5, 7, 9)
(26) http://docs.witheve.com/handbook/statistics/count/
he pervious example
(27) http://docs.witheve.com/handbook/statistics/random/
First example for random does not display anything, should be replaced by something like:
search
[#time minutes seconds]
x = random[seed: seconds]
commit @browser
[#div text: "{{minutes}} {{seconds}} - random: {{x}}"]
2nd example:  should be:	bind @browser
Same for 3rd example
(28) http://docs.witheve.com/handbook/strings/join/
takes tokens tokens together    redundant tokens
or should be:  token tokens
(29)
// Join the tokens into a sentence again, with hyphens
search
[#token token index]
text = join[token, index, with: " "]
bind
[#div text] // Expected "the-quick-brown-fox"
 There is no definition for the function join
(30) http://docs.witheve.com/handbook/glossary/
Glosasry   should be Glossary
Referential Transparentcy should be Referential Transparency
Thanks!

Just realized that this is the wrong place. Moved it as an issue here:
https://github.com/witheve/docs/issues/36
Please close this issue


Revisit use of BoolEnum

We use BoolEnum here
https://github.com/Qiskit/openqasm3_parser/blob/3f74976b5f35a2fafc0de3985e2697013e1a41c8/crates/semantics/src/types.rs#L22-L28
We do this so we see things like Duration(IsConst) rather than Duration(bool).
@jakelishman suggested doing
enum Type {
  Bit { is_const: bool },
  ...
}

At the moment, the reason for choosing BoolEnum. But it's worth revisiting. If there is no advantage in using the external crate, better to get rid of the dependency.

The other concern is that when consuming these by enum variant destructuring, you don't see the type name, only the variable you pattern-match it with, which can be anything. Then using the variable is inconvenient, because you have to call .into() to get a bool, so you end up with code like
match expr.type() {
  Type::Bool(can_be_anything) => {
    if can_be_anything.into() {
      ...
    }
    ...
  }
}

(actually at the moment IsConst doesn't derive Copy, so can't automatically deref and needs to be (*can_be_anything).into(), but that's easy to change) whereas struct variants would require construction and destructuring to be
let t = Type::Bool { is_const: true };
match t {
    Type::Bool { is_const } => (),
}

where the name is required to match (or you have to do is_const: can_be_anything).


Should transition to BaseMappable

Hi,
Recent updates to ObjectMapper lifted the restriction on implementing init(map:) but the current Moya-ObjectMapper expects the objects conform to Mappable instead of BaseMappable. I'm opening this PR in a unmergeable state so that I remember to finish it tomorrow. But I'd be glad if you just take a look. Thanks!

Conflicts sorted, please take a look!

Looks good, merged!


Feature/add mac notification

Hi Mas @ariya , I saw this in the timeline and took a quick look and definitely interested in using this in my day-to-day, but turns out the notification doesn't support MacOS and Windows so I did a quick workaround using node-notifier dependency, I hope this helps!
I have tested this in MacOS and it works.
Please let me know if there's any suggestion :)

Hi @kamalhm, thank you for the PR!
 To see the problem yourself, rebase this feature branch to the latest main and then run npm run build. You will get the warning, e.g.
> Warning Cannot find module 'node-notifier' from …

But even worse, once the binary is produced, it won't run at all:
  Error: Cannot find module 'node-notifier'

I'm not sure if it's possible to fix it by tweaking how vercel/pkg is configured. But even if it's possible, seems that it's a bit of a hassle.
My suggestion: perhaps implement a simple macOS notification with osascript, see e.g. https://stackoverflow.com/questions/5588064/how-do-i-make-a-mac-terminal-pop-up-alert-applescript, similar to the Linux solution via notify-send.

I rebased my branch with the latest main and tried building it. While there were some warnings, I found the binary produced to be working well. Maybe it has something to do with me running the build on macOS?

Meanwhile I'll look into your suggestions and hopefully I can implement that instead.

Maybe it has something to do with me running the build on macOS?

Try running it on a different system (same OS) or at the very least, on a temporary empty directory.

Oh turns out it is broken when it's run on a directory without the node_modules. But I have fixed the PR using osascript, I tested it and it works even on an empty directory, please review again.

This is very good, thank you @kamalhm!


Could not install @aaa/google-docs-i18n-strings

@aaa/google-docs-i18n-strings seems not to be published on npm
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@aaa%2fgoogle-docs-i18n-strings - Not found
npm ERR! 404
npm ERR! 404  '@aaa/google-docs-i18n-strings@latest' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)

google-docs-i18n-strings is currently not open source and for internal use only. However, you can still use the ios-starter, just skip the googleSheetId parameter when asked by cookiecutter.
I updated the readme to clarify the installation steps. Thanks for pointing this out!


C# client with Azure SignalR Services: Unable to connect to the server with any of the available transports.

I had Signalr hub within asp.net core 2.0 mvc application and it was working fine. hosted it on azure and it was working. when tried to use Azure Signalr Services the client started to throw this exception
"Unable to connect to the server with any of the available transports."
The weird thing is that I can see connections on azure portal.
First question is the bug. all what changed and made it break are these:

services.AddSignalRCore().AddAzureSignalR(signalrConnectionString)
app.UseAzureSignalR(router => router.MapHub("/myhub")) instead of UseSignalR

Notes:

signalr resource connection string is set in configurations
app.UseFileServer() has no effect, and I wonder why it is there without any explanation
debuggin in iis express or hosting on azure both same result

This is the client, and I don't know where all other options have gone or how to configure logs!:
_hubClient = new HubConnectionBuilder()
                            .WithUrl(url, HttpTransportType.WebSockets)
                            .Build();

            await _hubClient.StartAsync();

no https no authentication no authorization. mvc controllers work, signalr client does not!
Second question is, if the client is not connected, what are these connections I see on the portal? is it my web app creating them or what? any documentation/explanation?

You need to update to the RC1 of ASP.NET Core and .NET Core.

@davidfowl , same issue. Here are the updated references:
on server:
<TargetFramework>netcoreapp2.1</TargetFramework>
...
<PackageReference Include="Microsoft.AspNetCore.All" Version="2.1.0-rc1-final" />
<PackageReference Include="Microsoft.AspNetCore.SignalR" Version="1.0.0-rc1-final" />
<PackageReference Include="Microsoft.Azure.SignalR" Version="1.0.0-preview1-10009" />

and client:
<PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>
...
<PackageReference Include="Microsoft.AspNetCore.SignalR.Client" Version="1.0.0-rc1-final" />


[Request] Apply Ears Effects to Offline/Fallback Skins

When I play offline, I use a resource pack to change the fallback skins to my skin, but Ears doesn't apply its affects there! Would you be able to add this functionality? If so, could we also apply it to Zombies, since they use basically the same format as player skins? I'd love to make a "Zombie Furries" resource pack haha!

@Flexico you can use a paper server with fake skins to circumvent this https://www.spigotmc.org/resources/skinsrestorer.2124/


feat(line): make lineDash configurable

Closes: #486

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

备注：merge 掉给祯老板用


libyaml: 0.1.7 -> 0.2.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/libyaml/versions.
These checks were done:

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.2.1 with grep in /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1

directory tree listing: https://gist.github.com/a183f8e1406cfb340cfc7e4809746583

du listing: https://gist.github.com/6731d94c1363109427d6793cc7cd010a

meta.description for libyaml is: '"A YAML 1.1 parser and emitter written in C"'.
cc @wkennington for testing.
This build is cached with Cachix. To use the cache follow these instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 15c30f43425b36509cad15df080aca2786f8616b && nix-shell --pure -I nixpkgs=. -p libyaml

Success on x86_64-linux (full log)
Attempted: libyaml
Partial log (click to expand)
make[2]: Leaving directory '/build/yaml-0.2.1/tests'
make[1]: Leaving directory '/build/yaml-0.2.1/tests'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1
shrinking /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1/lib/libyaml-0.so.2.0.5
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1/lib
patching script interpreter paths in /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1
checking for references to /build in /nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1...
/nix/store/bn221ypclyhac7pzicwf0kj7b6g03d4v-libyaml-0.2.1

Success on x86_64-darwin (full log)
Attempted: libyaml
Partial log (click to expand)
make[2]: Entering directory '/private/tmp/nix-build-libyaml-0.2.1.drv-0/yaml-0.2.1/tests'
make[2]: Nothing to be done for 'install-exec-am'.
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-libyaml-0.2.1.drv-0/yaml-0.2.1/tests'
make[1]: Leaving directory '/private/tmp/nix-build-libyaml-0.2.1.drv-0/yaml-0.2.1/tests'
post-installation fixup
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/99y0yk7imln93flfc7hsndg780j8qwys-libyaml-0.2.1/lib
patching script interpreter paths in /nix/store/99y0yk7imln93flfc7hsndg780j8qwys-libyaml-0.2.1
/nix/store/99y0yk7imln93flfc7hsndg780j8qwys-libyaml-0.2.1

Success on aarch64-linux (full log)
Attempted: libyaml
Partial log (click to expand)
make[2]: Leaving directory '/build/yaml-0.2.1/tests'
make[1]: Leaving directory '/build/yaml-0.2.1/tests'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1
shrinking /nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1/lib/libyaml-0.so.2.0.5
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1/lib
patching script interpreter paths in /nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1
checking for references to /build in /nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1...
/nix/store/k5i760gzjj2rf1803b5lmmwx3dgwma60-libyaml-0.2.1

Failure on x86_64-darwin (full log)
Attempted: libyaml
Partial log (click to expand)
cannot build derivation '/nix/store/j1d17m0qa3v9h7vj0kgcnw0w68azgfcp-libev-4.24.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/kv1dlql5svavxk6dfsa7rrqdag8b7g0w-perl-5.24.4.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/risl7ij45s34s0vs2v157iz0l84fxjvw-pkg-config-0.29.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/0n704zs2wp07ih48h3h415iyyxn88maq-openssl-1.0.2o.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/hd3zkqa9zry54ghv31nv658z91wx469y-libssh2-1.8.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/bfcryivplxsdgi7z0ngrslca0s76562g-nghttp2-1.32.0.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/fnzqbv0gcj1jxylq0i4zs4crfymrpih5-curl-7.60.0.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/qcvdki6s9c1zi4v26dx1fhsbah8ydxq2-yaml-0.2.1.tar.gz.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/259csawivfm32y6p0lk690slvbdgi9j0-libyaml-0.2.1.drv': 3 dependencies couldn't be built
error: build of '/nix/store/259csawivfm32y6p0lk690slvbdgi9j0-libyaml-0.2.1.drv' failed

Success on aarch64-linux (full log)
Attempted: libyaml
Partial log (click to expand)
make[2]: Leaving directory '/build/yaml-0.2.1/tests'
make[1]: Leaving directory '/build/yaml-0.2.1/tests'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1
shrinking /nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1/lib/libyaml-0.so.2.0.5
strip is /nix/store/h0nc84ybvb8g724fwysvcghh1675wjli-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1/lib
patching script interpreter paths in /nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1
checking for references to /build in /nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1...
/nix/store/0nlbph7ralc62rbc5lblmwyp6g731633-libyaml-0.2.1

Changelog: https://github.com/yaml/libyaml/blob/master/announcement.msg

This broke several tests of pythonPackages.pyyaml with some strange errors. Not sure how to proceed here.

I had already reverted it in 01fc00c (shown by GitHub directly above your comment).

I see, sorry, thanks.


Need support for ‘exclusive’ enclaves (Vatican library Issue #3 of 6)

The need for ‘exclusive enclaves’ arises when there is an enclosing region, an enclosed region, and where the annotation for the enclosing region DOES NOT apply to the enclosed region or enclave. This is illustrated in the figure below, where the enclosing region is highlighted in red and the enclosed region is highlighted in blue.
Without support for exclusive enclaves and without the support for irregular regions this had to be implemented as five separate annotations  for the enclosing red region, and one annotation for the enclosed blue region.

related to #709


Using x11docker with Docker Rootless Mode

Starting from Docker Engine v20.10 docker allows running the daemon as a non-root user.
I successfully installed it on a fresh Arch installation following this guide, and indeed, after setting the $DOCKER_HOST variable to unix://$XDG_RUNTIME_DIR/docker.sock, containers can be created and run as a regular user with the same command and syntax as previously with the root daemon.
However, seemingly x11docker is not working with it.
I tried to run this: x11docker x11docker/fvwm xterm
And it refused to start, complaining for not getting PID 1.
dockerrc: 40. check for PID 1: 0
time to say goodby (main)

Switching back to the root daemon, everything works fine (with x11docker prompting for password).
Is it possible to use x11docker with docker daemon not running as root? If not, is it planned to be made available in the future?
(I am almost completely new to the topic, so please forgive me if it's just me trying it the wrong way)
Thanks in advance.

Thank you for the ticket, I wasn't aware that docker supports rootless mode now.
I did some first tests and found that rootless docker fails with --userns=host. This is set by x11docker mainly to allow shared folders.
As a quick fix you can use option --user=RETAIN to suppress x11docker's container user setup. Options --home and --share won't work.
I'll look closer at this to support rootless docker well.
Currently I cannot get back from rootless to rootful mode and docker is entirely broken on my system. Even a purge and reinstall didn't fixed the issues. sigh.

Thanks for the quick reply (and sorry for the broken system :) ).
The fix is perfect for me now; using x11docker --user=RETAIN -- --v volume_name:/data -- IMAGE as a workaround for persistent storage without --home and --share works without any problems.
I hope support for rootless will be eventually added, I think there are numerous use cases for that. As for the system: now I am running a rootful and a rootless daemon at the same time, using sudo systemctl start docker and systemctl --user start docker; they are working fine, and do not have any conflict or whatsoever. Setting the DOCKER_HOST variable to the appropriate socket is enough for 'switching' between them. I hope it helps.

x11docker supports rootless docker now.
The setup is barely tested yet.
--home and --share are not supported and will be disabled along with a message.
You don't need --user=RETAIN anymore.

As for the system: now I am running a rootful and a rootless daemon at the same time, using sudo systemctl start docker and systemctl --user start docker; they are working fine, and do not have any conflict or whatsoever. Setting the DOCKER_HOST variable to the appropriate socket is enough for 'switching' between them. I hope it helps.

Now this works here as well. :-) Not sure what went wrong the first time.

Currently I'd rather recommend podman than docker to run rootless. podman runs rootless, supports --home and --share, does not need a dameon or an environment variable. (x11docker option --backend=podman #255)


Could not debug typescript when outDir is a symbolic link directory

VSCode Version: v1.29.1
OS Version: Windows 10

Steps to Reproduce:
1, check out https://github.com/chopdown/test_vscode_mklink.git
2, exec link.bat
3, set break point on Test.ts
4, run & debug

Does this issue occur when all extensions are disabled?: Yes
I want to build the typescript to another repository ( artist directory). They have no code permission, The program will read the configuration at that directory. So I mklink the directory to the code/dist directory.  (I don't want use  absolute path). It's OK,  except can't debug on this scenarios.

This works with
"runtimeArgs": [
    "--preserve-symlinks",
    "--preserve-symlinks-main",
],

added to the launch config. This is documented here: https://code.visualstudio.com/docs/nodejs/nodejs-debugging#_can-i-debug-if-im-using-symlinks


Cannot update items on Cloud

EdgeDB Version: 5.4 Cloud
EdgeDB CLI Version:EdgeDB CLI 5.1.0+7c5764f
OS Version: Ubuntu WSL Windows 10

Steps to Reproduce:
myorg/mydb:main> update AnyItem filter .external_type = <ItemType>'pubchemCompound' set {ext_type := ExternalType.pubchemCompound};
edgedb error: ClientConnectionEosError: end of stream while reading message
Reconnecting...
myorg/mydb:main> update AnyItem filter .external_type = <ItemType>'pubchemCompound' set {ext_type := ExternalType.pubchemCompound};
Reconnecting...
edgedb error: ClientConnectionEosError: end of stream while reading message
Reconnecting...

The update worked fine locally. There are 100'000's of AnyItems.

Schema:

Can you provide your schema? Or a pruned version of it?
If do I bit of guesswork, using ExternalType.pubchemCompound might mean that you are setting ext_type to all of .pubchemCompound of every ExternalType you have in your database.


Linux Packaging helper (arch, rhel)

Hey everyone,
I'm excited to share a tool I've created – the DEB Converter. It's designed to help with distributing Rust Tauri applications across various Linux distributions. This tool automates the conversion of .deb packages into other formats like .pkg.tar.zst for Arch Linux and .rpm for RHEL-based systems.
The DEB Converter integrates seamlessly with GitHub Actions, making it ideal for automating your build and release workflows for Rust Tauri projects.
Check it out here:
As an example of its application, take a look at this repository:
Feel free to explore, use it in your projects, and share any feedback or questions you might have!
Cheers!

@0-don the link is 404. Did you stop maintaining it?


chore(dependabot): remove bddckr from reviewers

Chris hasn't been actively part of the project for a while.

:tada: This PR is included in version 1.4.5 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.4.5
GitHub release

Your semantic-release bot :package::rocket:


Crash when building from source and launching new project on Ubuntu

Tested versions

Reproducible in 4.2.stable, 4.3.dev

System information
Godot v4.2.2.rc (d5064a7d4) - Ubuntu 22.04.3 LTS 22.04 - Wayland - GLES3 (Compatibility) - SVGA3D; build: RELEASE;  LLVM; () - Intel(R) Core(TM) i9-10850K CPU @ 3.60GHz (10 Threads)
Issue description
I've tried to build godot from source using: scons platform=linuxbsd use_llvm=yes linker=lld. After creating a new project with renderers Forward+ or Mobile, I always got the same error:
Godot Engine v4.2.2.rc.custom_build.d5064a7d4 - https://godotengine.org
Vulkan API 1.3.238 - Forward Mobile - Using Vulkan Device #0: Unknown - llvmpipe (LLVM 15.0.7, 128 bits)

WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
     at: _editor_init (modules/gltf/register_types.cpp:63)
ERROR: Caller thread can't call this function in this node (/root). Use call_deferred() or call_thread_group() instead.
   at: propagate_notification (scene/main/node.cpp:2236)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.2.2.rc.custom_build (d5064a7d443d4b335169c0d02e822ee720abcc5d)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x42520) [0x7f59bc242520] (??:0)
[2] llvm::CmpInst::Create(llvm::Instruction::OtherOps, llvm::CmpInst::Predicate, llvm::Value*, llvm::Value*, llvm::Twine const&, llvm::BasicBlock*) (??:0)
-- END OF BACKTRACE --
================================================================

On the other hand compatibility renderer works fine. I used many different scons arguments such as:
scons platform=linuxbsd
scons platform=linuxbsd debug_symbols=yes
scons platform=linuxbsd use_llvm=yes linker=lld dev_build=yes
Error message remained unchanged despite change in the arguments. I also deleted folders ~/.config/godot and ~/.local/share/godot before new builds to achieve clean build.
Steps to reproduce

Build godot using scons platform=linuxbsd
Run godot with ./godot/bin/godot.linuxbsd.editor.x86_64
Create new project with renderer Forward+ or Mobile

Minimal reproduction project (MRP)
N/A

What happens if you try adding --rendering-driver opengl3 to your launch command?  For example mine is ./Godot_v4.2.1-stable_linux.x86_64 --rendering-driver opengl3
Mine would crash with a similar message using Forward, but I didn't try Mobile.  After I tried that launch parameter it worked.

System information
Godot v4.2.2.rc (d5064a7) - Ubuntu 22.04.3 LTS 22.04 - Wayland - GLES3 (Compatibility) - SVGA3D; build: RELEASE; LLVM; () - Intel(R) Core(TM) i9-10850K CPU @ 3.60GHz (10 Threads)

SVGA3D is the software rasterizer used by VMWare. Please mention the fact you're trying to run Godot in a virtual machine, it's very relevant :)
And in that case it's a duplicate of https://github.com/godotengine/godot/issues/82435
The stackteace is a crash in the SVGA3D driver itself. Could be worth reporting to VMWare.

Just for the record I personally was having the similar crashes using the Linux binary.  I of course can't say what the OP was using.

@TheReaperKing Were you getting the same stacktrace pointing at llvm::CmpInst::Create? If not, and you're not using llvmpipe on an older Linux distro, it's unlikely to be the same crash.
The medium post you linked to seems to be a different issue too, from not having Vulkan drivers at all.

My apologies.  Maybe I didn't look close enough but it seemed like a very similar error.  I did notice the signal is different.  And I will freely admit I am just getting started with Godot.  This was my error:
WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
     at: _editor_init (modules/gltf/register_types.cpp:63)
ERROR: Caller thread can't call this function in this node (/root). Use call_deferred() or call_thread_group() instead.
   at: propagate_notification (scene/main/node.cpp:2236)

================================================================
handle_crash: Program crashed with signal 4
Engine version: Godot Engine v4.2.1.stable.official (b09f793f564a6c95dc76acc654b390e68441bd01)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x3c050) [0x7f2b850a5050] (??:0)
[2] llvm::CmpInst::Create(llvm::Instruction::OtherOps, llvm::CmpInst::Predicate, llvm::Value*, llvm::Value*, llvm::Twine const&, llvm::BasicBlock*) (??:0)
-- END OF BACKTRACE --
================================================================

Thank you so much for your clarification, I have a lot to learn!

That does seem similar. What OS are you on and what Vulkan drivers are you using? The first line of the Godot log should make it clear.


[0.11] FIRST(),LAST() no longer return the relevant timestamp and values seem off

Using the sample data from the docs:

first() and last() no longer return the relevant timestamp in 0.11
first() and last() don't return the same point in 0.10 and 0.11

0.10
> SELECT FIRST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
--------------
time			        first
2015-08-18T00:00:00Z	2.064
> SELECT LAST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
--------------
time			         last
2015-09-18T21:42:00Z	 4.938

0.11
> SELECT FIRST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
--------------
time			        first
1970-01-01T00:00:00Z	5.597
> SELECT LAST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
--------------
time			        last
1970-01-01T00:00:00Z	5.745

I believe 0.10 has the right answer:
#First values of water_level

> SELECT water_level FROM h2o_feet WHERE location = 'santa_monica' LIMIT 3
name: h2o_feet
--------------
time			        water_level
2015-08-18T00:00:00Z	2.064
2015-08-18T00:06:00Z	2.116
2015-08-18T00:12:00Z	2.028

# Last values of water_level

> SELECT water_level FROM h2o_feet WHERE location = 'santa_monica' ORDER BY time DESC LIMIT 3
name: h2o_feet
--------------
time			        water_level
2015-09-18T21:42:00Z	4.938
2015-09-18T21:36:00Z	5.066
2015-09-18T21:30:00Z	5.01

I've tried to reproduce the 'getting the wrong point' issue in another data set and I can't!

We decided with the new query engine that all aggregates and selectors, regardless of whether a group by is used or not, would either return the interval time or the zero time. In the future, we're going to come up with a syntax that allows selecting the unit time as part of a separate column.
@pauldix can you confirm this is a desired change in behavior?

Although I'm going to look into why the value is incorrect. That seems to be wrong.

@jsternberg I think that's right although I'm having trouble remembering why we'd want that. You'd almost always want the selector to return the time of the actual value and not the group by window.
The one thing I can think of is if you have multiple selectors and an aggregate mixed together. Then the output could have multiple points for the same group by windows, but with different values/point times for the selectors.
Did we create an issue for this anywhere?

I don't think we made an issue for it anywhere. I think the rationale for having the selectors return the interval time was because you felt it would be what people would expect when using GROUP BY and also to resolve the conflict when multiple selectors and aggregates are mixed together. The alternate syntax was the proposed method of giving the selected time as part of the query, but time would always be reserved for the interval.

Yeah, I think people will want the actual time for the selected point. Maybe include automatically? <selected_field>.time?

This has come back in 0.12:
Connected to http://localhost:8086 version 0.12.1
InfluxDB shell 0.12.1
> use telegraf
Using database telegraf
> select last(usage_user) from cpu
name: cpu
---------
time	last
0	1.2096774193903288

>

@jwilder @jsternberg @rkuchan

@beckettsean I think that was the desired behavior with a future feature request in #5926. @pauldix are we reversing that decision?

@jsternberg @pauldix what's the desirability of returning the selected point without a meaningful timestamp?

There was something around consistency of the API. However, we need a way to select the timestamp of the returned point. Should prioritize that to get in ASAP

There were two reasons that, combined, produced a compelling reason to keep it as it was.
The first was API consistency. The time field was to act as the interval. To be fair, I'm not completely sure we should bother returning that field for aggregate queries that don't have a group by since it's mostly useless.
The second reason was because of ambiguity with multiple selectors. Multiple selectors would make it impossible to figure out which one the time field should refer to and would have to just be ignored and zero'd out. It also introduced a complexity in the actual code since the part of the code that creates the iterator for that column doesn't know whether only one selector was used or multiple selectors were used. It wouldn't be too difficult to modify that section of the code though, so that's not the biggest deal.
Do we want me to prioritize the time selector from the other issue or just make it so the time field here gets filled with the selector if and only if there is only one selector? The first is harder, but likely more correct. The second is easier, but might be more appropriate for a later point release or 2.0.

I vote for the second for now. Let's do the easy thing that makes SELECT LAST(value) FROM foo do the right thing. If that means SELECT LAST(value), FIRST(value) FROM foo GROUP BY time(1m) doesn't do what's expected, that's okay. An argument can be made that GROUP BY intervals should always be the returned timestamps, even for selectors (better graphs).
Right now there is NO WAY to recover the timestamp for the point without selecting it by tags or using LIMIT or some other non-precise method. As long as a bare FIRST(), LAST(), MAX(), MIN() call returns the relevant timestamp, all problems can be solved with enough queries.

@gunnaraasen ^^ your thoughts?


Add fetch hook into tracer.

This is a first draft on how to hook static string data. This would allow you to build the string source of a file yourself, then pass it into the builder to resolve dependencies. Example:
builder.build('foo.js', 'bar.js', {
  fetch: function (data) {
    if (data.name.indexOf('foo.js') !== -1) { return 'var bar = 2;'; }
  }
});

Fixes #330

Okay I refactored it a bit to use the fetch override. One of the issues is that since we are completely overriding it, you can't pass in a string and have systemjs build the deps without the pass function. This kind of defeats the purpose so I added in a pass() that lets you fall back to the default implementation.

Thanks, this looks great. Agreed about the pass through concept - perhaps we could use the same pattern as we do for plugins then - https://github.com/systemjs/systemjs/blob/master/docs/creating-plugins.md#sample-css-plugin where the original fetch becomes the argument?

Ah okay, I did not see this! I was modeling it after the beforeEach(function (done) { done(); }) pattern from jasmine. I will update the PR and also add to the readme. :+1:

Great, thanks, just let me know when its ready as new commits don't send notifications.

Sorry for the delay on this. Updated my code and the README as well!

Thanks for following up here!

Oops, must have had some leftover changes from my previous method of doing this. I will update the branch with the fixes requested.


Make GenericMessageSubjectProvider the default?

As suggested by @slang25 here, we should consider making the message subject provider that supports generics the default implementation for JustSaying 7.0.0.

Discussion between myself, @brainmurphy and @iancooper yesterday afternoon was that we should make a yes/no decision on this imminently for inclusion in v7 or not.


Fix regressions after #419

Fix for https://github.com/androidx/media/issues/1126 and https://github.com/androidx/media/issues/1150

Hi @dsparano,
Considering we had the discussion already, could you please grant me write access so I can merge the PR?"

Hi @dsparano,
There are lots of tests failing in extractor.ts package with this change.
You can fix this by temporarily setting DumpFileAsserts.DUMP_FILE_ACTION to WRITE_TO_LOCAL.
Can you please run the tests to see if the changes in the dump files are correct?

Hi @dsparano,

could you please grant me write access so I can merge the PR?

Thank you for sending an invite to collaborate on the v-novaltd/androidx-media repository. When I accepted the invitation, it said I have View access and I'm unable to push changes to the repository now as well.
Is there something you can do from your end to fix this?

@rohitjoins sorry, you have now write access.
Re tests I'll look into them asap.

@dsparano, I have added the updated dump files to the PR.

Thanks @rohitjoins

There are test failures with this change in TransformerWithInAppMuxerEndToEndTest.

@dsparano We're temporarily rolling back the PR (https://github.com/androidx/media/pull/419) due to a bug fix release scheduled for next week. We'll reintroduce the changes once we've addressed the outstanding issues with this PR.

@rohitjoins I've cherry picked the #419 merge, re-applied the previous commits for fixing 1126 and 1150, and added a couple more to fix the H265 HLS cases which were not entirely correct and the TransformerWithInAppMuxerEndToEndTest. I think we should be good to go if you could please take a look.

Thank you for including the changes from #419 and fixing the tests. I'll take a look. Thanks :)

@rohitjoins sure this is it: https://venus.v-nova.com/daniele/hls/byterange/h265/bbb_720p25_h265_keyint15_60sec.m3u8
All the HLS streams I created for this ticket are under here: https://venus.v-nova.com/daniele/hls/


Infinite list scrolling

Can you also use FlexLayout to create an infinite list of pictures, for example each time after showing 20 pictures, load the next 20 pictures, and show all a scrollable list using FlexLayout?
Would be nice if possible to see an example

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

ID: b954b795-b38e-61ff-3ba1-26a0623b97e6
Version Independent ID: 65795b50-24d3-1997-038e-ffdb609eb20b
Content: The Xamarin.Forms FlexLayout - Xamarin
Content Source: docs/xamarin-forms/user-interface/layouts/flex-layout.md
Product: xamarin
GitHub Login: @charlespetzold
Microsoft Alias: chape

Hi @RobDaPraia
FlexLayout doesn't support that behaviour out of the box. However, it is possible. You have to implement incremental loading. See https://jmillerdev.net/incremental-listview-control-and-plugin-for-xamarin-forms/ for an example of doing this with a ListView.

HI David,
thanks for the fast response


integration tests for the ExternalUser class

Description
Unit tests are updated for the ExternalUser class and UserApps class
Integration tests are written for the ExternalUser class

[ ] create integraions tests

[x]  loginWithTonomy() then getUser()  (succeeds)
[x]  loginWithTonomy() then verifyLogin()  (succeeds)
[ ]  loginWithTonomy() then getUser()  (fails: user doesnt exsist on the blockchain)
[ ]  loginWithTonomy() then getUser()  (fails: no keys found)
[ ]  loginWithTonomy() then getUser()  (fails: User didn't login , keys are found user is found, but keys aren't on blockchain)
[ ]  loginWithTonomy() then verifyLogin()  (fails keys aren't valid)
hint: access JSdom-URL to check urls
Reference: https://github.com/Tonomy-Foundation/Tonomy-ID-SDK/blob/development/test/app.test.ts

[x] update unit tests for the login

because we have not handled the edge cases where the user doesn't login functionally, i have not implemented the test cases for them either. But if I did I would use scoped jest tests to do this


Performance of HeightField colliders and raycasts

In Bullet physics, there used to be a horrible performance problem of raycasts when heightfields were involved. This was caused by the fact the raycast was turned into an AABB (so it could cover a huge area), and due to how the abstraction worked, every single triangle of the heightmap was eventually checked against the rectangular region. Which is wasteful because raycasts are just segments :p
I made a PR to Bullet implementing a specialization of raycasts for heightfields, using a raytracing algorithm. The idea is that the ray is projected in 2D, and the algorithm iterates forward through each pixel it intersects with, and checks triangle intersections on them. This is fast enough to not even require any acceleration structure, which means it is compatible with editing the heightmap in realtime, without recomputing any cached structure. But of course if the terrain is not editable, precomputing a grid of AABBs can allow to run that same algorithm in a sort of broad-phase first, making it even faster. In the PR you can also find some profiling info.
How is it implemented in ReactPhysics3D?
It seems it does the same thing Bullet used to do, checking every single 2D triangle lying inside of the AABB: https://github.com/DanielChappuis/reactphysics3d/blob/23abaa905dff0dfb0672ac2e7c2e472135e44bab/src/collision/shapes/HeightFieldShape.cpp#L242
So it might benefit a lot from that technique.

Or is it what you did in https://github.com/DanielChappuis/reactphysics3d/commit/d0fa4c2755da4042d7794d55ad70992cf949fd6b ?

Or is it what you did in d0fa4c2 ?

Yes exactly. In the current released version (v0.8.0), the raycast tests are performed with all the triangles of the height field that are inside the ray AABB. Of course, this is slow. As you noticed, I have improved this in the current "develop" branch to only test triangles that are part of cells of the height field grid that the ray crosses.
Note that this is currently implemented in the "develop" branch that will be part of the next released version (v0.9.0).

I am closing this issue. Do not hesitate to reopen it if necessary.


Use vagrant boxes with Ceph already installed

Nice to develop on the plane :)

This won't probably happen... Closing
I prefer to use the official distro images.


Add Snapshots table metadata

@Fokko Can we merge this? I am almost done with "Files" table, so I can rebase my code before creating a PR.

Just have one question: I was thinking if later we need those metadata table classes, StaticTableScan, and StaticDataTask like what Java did. These may become useful when other engines (Daft, Ray) wants to represent the metadata tables in their dataframe. But since the metadata tables are normally not very large so using pyarrow as a bridge may be enough?

I'm open for that, but I would like to defer that to a later PR. I don't like the hard dependency on PyArrow, and would love to get rid of that, but I'm not sure what the best format is then. An Arrow table can be used in most engines without any copying.
I'll move this forward so @Gowthami03B can continue her work.


function.js

a reference to function.js file is made but the file isn't in the repo.

Nice catch! It's in my repo where I call the module but I forgot to put a sample function in the module itself. I'll correct this tonight or tomorrow. Thanks!

here is one i am using, it makes it so sub dirs can be called without index.html
function handler(event) {
    var request = event.request;
    var uri = request.uri;
    if (uri.endsWith('/')) {
        request.uri += 'index.html';
    }
    else if (!uri.includes('.')) {
        request.uri += '/index.html';
    }
    return request;
}

Yep that's the same one I was using. I just released 0.1.1  with that fix. Thanks for the catch!
https://github.com/loganmarchione/terraform-aws-static-site/tags


typo

I think mmdet should be mmcv
https://github.com/open-mmlab/mmdetection/blob/885b86bae849529735129c0945d729b91ff3b79a/mmdet/core/evaluation/mean_ap.py#L298

you are right, thank you!


SSH Adaptor could support config file in ~/.ssh

ssh allows for configuration using the ~/.ssh/config file. It would be nice if we read things like host aliases and user names from this config file.

Fixed in #231.


Add wizard and website creation code from vscode-azurewebservice

Needs tslint fixes before it can be checked in - might be easier to disable tslint temporarily, then fix after PR.

Merging, but some comments still pending fixes, plus tslint.


Improvement/466 remove not needed calls to tostring

Sorry, here should be only one commit. I will fix my PR today.

This pull request fixis #466.

Just a note - "fixis" is not one of recognized keywords.


Update to Electron 6.0.0

Update to Electron 6.0.0.
Update packages:

electron
electron-builder
electron-rebuild

Update the nodeIntegration default which was changed in Electron 5.0.0.
Add missing plist package required from updating the packages

Anyone able to help me figure out why the CI is failing. I don't seem to be getting all of these warnings and errors when I build the app locally

You need to update node-pty. the old version does not work with node 10+ afaik

Hey @razfriman are you willing to pick this up again?

@Tyriar any idea why this fails? https://travis-ci.org/zeit/hyper/builds/583470356?utm_source=github_status&utm_medium=notification

I seem to be having issues doing this upgrade actually. Things don't seem to be working well for me with node 12 and electron 6. Perhaps someone else can take this over

@Stanzilla You might need to also tweak the version of node you're building with. @deepak1556 what version of node.js should builds be using with Electron 6?

C:\projects\hyper\app\node_modules\node-pty>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>
it seems to be using an old version of node-gyp, not sure why

@Stanzilla You might need to also tweak the version of node you're building with. @deepak1556 what version of node.js should builds be using with Electron 6?

https://electronjs.org/blog/electron-6-0 says 12.4.0, CI seems to use gyp info using<EMAIL_ADDRESS>| win32 | x64 so should be fine?

@Stanzilla<EMAIL_ADDRESS>will break native module compilation on windows due to https://github.com/electron/electron/issues/20185, it has been fixed yesterday and new version will be released today, please update to that when its available.

@deepak1556 oh interesting, thanks! I hope this really is it then, because I tried with 6.0.0 before and it was the same issue

For reference, I can run and build successfully on Mac OSX and fully build (yarn run dist) Hyper on at this point in time. I will try to see if I can replicate the appveyor failures on a Windows machine. Not sure what else to try though

the is my pr #3781, one is to update the node-pty, another is to add a config for experimentalUseConpty. now this pr update the node-pty, @Tyriar I hope you can look at another problem. when open hyper with hyper cli, the node-pty exit with code 0. I test with experimentalUseConpty false. then the problem disappears. I found that the causes of this problem are that the node-pty create in the electron main process and hyper open by the cli. I hope it's helpful.

Okay thanks a lot for your help @Tyriar
Now that we got this working, @razfriman I tested it a bit and panes are a bit wonky when you split them up, can you repro that on macOS?

Yes I am noticing the same thing in MacOS. I will try to find the cause+fix but this is my first change to Hyper, so I don't know the codebase too well

I'm not sure what that would be, other than an issue with Electron 6 and canvas? VS Code just updated and a new build will be out on Monday so I'll probably find out quick if it is an Electron 6 issue.

@Tyriar anything came up in VSCode?

Haven't seen anything like this, related to your layout code I guess?

@raz-canva any news? did you find something? I didn't :(

merged the latest stuff from canary in and tried to get it working more, currently stuck because Windows thinks that 'build' is not recognized as an internal or external command, investigating...

next problem:

 internal/modules/cjs/loader.js:630
    throw err;
    ^

Error: Cannot find module 'C:\Users\Benjamin\VCS\Git\hyper\node_modules\electron\dist\resources\electron.asar\browser\init.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
    at Function.Module._load (internal/modules/cjs/loader.js:531:27)
    at Function.Module.runMain (internal/modules/cjs/loader.js:850:10)
    at internal/main/run_main_module.js:17:11 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
error Command failed with exit code 1.

I have not been able to figure out the reason for the distorted size of text when splitting the panes.
The “build” not found error should be resolved in the package json file by replacing
“Build” with “electron-builder” (that part should be in the PR diff)

@raz-canva yea I did that, I am on the next issue already

yarn run v1.19.1
$ electron target

 internal/modules/cjs/loader.js:630
    throw err;
    ^

Error: Cannot find module 'C:\Users\Benjamin\VCS\Git\hyper\node_modules\electron\dist\resources\electron.asar\browser\init.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:627:15)
    at Function.Module._load (internal/modules/cjs/loader.js:531:27)
    at Function.Module.runMain (internal/modules/cjs/loader.js:850:10)
    at internal/main/run_main_module.js:17:11 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
error Command failed with exit code 1.

@Stanzilla can you try to run this

@LabhanshAgrawal what's the difference? I only saw the changes for forwardRef. And nope, does not open with that change :(

@LabhanshAgrawal try this https://github.com/ivanwonder/hyper/tree/raz/electron. I remove the initialization of rows and cols when creating the node-pty session. now the step is
create node-pty session without rows and cols -> create xterm instance -> resize the node-pty

@Stanzilla https://github.com/ivanwonder/hyper/commit/730d34481e5a2e83edbbec654d4d4e425d4daae1

@ivanwonder cherry picked those over! thanks!

Looks and works perfectly so far!

@Stanzilla I have removed the useless fit(). just test on your compute make sure work as expected.

Yeah I saw, that commit i cherry-picked as well

oh, missed it. wait for the feedback of @LabhanshAgrawal about the initial load resize issue.

yup!

The initial load resize is working. I can see it being rendered a bit small and then resizing but it's very quick and isn't really a problem according to me.
The test rendering thing is still there though.

@LabhanshAgrawal can you make a screenshot/video of that remaining problem, please?

I think the text rendering is the problem of fonts or browser. https://stackblitz.com/edit/typescript-dausj9?file=index.ts you can test with different fonts.

@Stanzilla

Oh yeah, I'd probably blame that on fonts

@ivanwonder in this I haven't changed anything, the text looks moved up to me. Is this supposed to be like this?

Ok I tried with my terminal fonts. Maybe the issue is really with the fonts. If you're not facing any problem then you should ignore this as it might be an issue with my patched fonts.

Look the alphabet j, it's in the middle.

if you mind this, try render type dom. it shows ok.

it's ok, I'll look for new fonts

if #3848 has been solved. maybe close.

I will once we merge this one.

@ivanwonder I tried out DejaVu Sans Mono for Powerline still have that gap at bottom.
I don't have any issue with 3.0.2 so it shouldn't be a font issue .

@LabhanshAgrawal but that was before the electron update, caused by the xterm update, right? so not blocking this PR.

Checked with canary, no font issue there also

That does not answer my question though :D

I noticed it in xterm update, but it was working on using canvas renderer. But it's not working with this PR even with canvas renderer.

ok

But I think the issue is small, so it should not block the PR, we can track it separately.

yeah, agree

there's nothing else blocking this, right?

nope I just some last testing and everything looks good, so, merging!

because there are different version of chrome between them , run that page on hyper 3.0.2

发件人: Benjamin Staneck<EMAIL_ADDRESS>发送时间: Friday, October 11, 2019 3:20:47 AM
收件人: zeit/hyper<EMAIL_ADDRESS>抄送: ivan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>主题: Re: [zeit/hyper] Update Electron to v6 (#3785)
Merged #3785https://github.com/zeit/hyper/pull/3785 into canary.
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/zeit/hyper/pull/3785?email_source=notifications&email_token=ADYVH3DGCNUN7VZFZRKN3OTQN56A7A5CNFSM4IJZQPH2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUEVCCGI#event-2703892761, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADYVH3HDG7BZ536CNNESUYTQN56A7ANCNFSM4IJZQPHQ.


Consider separating custom property (vars) definitions for import

Some components define their custom properties at the top of the file, e.g. https://github.com/suitcss/components-grid/blob/master/lib/grid.css#L3-L6
This can be annoying when you want to redefine those variables.
This doesn't work:
:root {
  --Some-other-var: 'foo';
  --Grid-gutter-size: 30px;
  --Arrange-gutter-size: 30px;
}

@import 'suitcss-components-grid';

Instead you have to import first, define you vars, and rely on source order to override, which is probably not how most people would go about things naturally.
What do people think about separating the custom property definitions to their own file, so if you chose, you could import without the definitions?

I've had no problem with just including my project specific components and settings after any SUIT components (inside theme.css in this example) - https://github.com/simonsmith/simonsmith.io/blob/master/source/stylesheets/src/index.css
Doesn't feel counter-intuitive (at least to me) and easier than having to remember to include a load of custom property files separately.

Ok, it's probably fine then.
Just caused some brief confusion here a while back so thought I would bring it up.

I guess the part that felt "weird" to me was that the custom properties are pre-defined, but the custom medias aren't. But in processing it doesn't seem to matter, I think as long as the custom media is defined anywhere it will work.

Perhaps we do need a recommended setup guide, so that users don't need to discover the best way to work. It was mentioned before in #86 I think

Makes sense!

Or even just a repo of examples, much like webpack does.

That's my preferred approach. I don't think I would have made it though our webpack config without that repo!


Fail to build using micro_ros_static_library_builder docker

I'm not entirely sure this is a micro_ros_arduino issue, but I can't seem rebuild the library to add a custom message (and don't think it has to do with the custom message).  I'm following the directions from README.MD, but near the end I get rsync errors trying to change directory:

Setting up libpopt0:amd64 (1.16-14) ...
Setting up rsync (3.1.3-8ubuntu0.4) ...
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of start.
Processing triggers for libc-bin (2.31-0ubuntu9.9) ...
rsync: change_dir "/project/src/action_msgs/action_msgs" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1207) [sender=3.1.3]
rsync: change_dir "/project/src/actionlib_msgs/actionlib_msgs" failed: No such file or directory (2)

This is the command I used:
docker run -it --rm -v $(pwd):/project --env MICROROS_LIBRARY_FOLDER=extras microros/micro_ros_static_library_builder:galactic -p esp32
I did notice this earlier on in the output regarding the project directory:

=== ./control_msgs (git) ===
Cloning into '.'...
find: cannot delete ‘/project/src/’: Directory not empty

full pastebin:
https://pastebin.com/JuJ00Bsk
Any help would be.. helpful :)

Two pints here:

Are you using arduino galactic branch? Make sure the version used on the micro_ros_static_library_builder matches the repository branch.
Those errors can happen on a successful build. I understand that it can be confusing, did you check if the library was actually rebuild?

I'll check to see what branch is being rebuilt.  It's possible its not galactic (though I did try the same process using the humble docker version.. hard to believe its not one of those two).
And no, I didn't check as I just assumed it didn't work.  But I will!
Thanks.

@Acuadros95 Thank you for the help.  Turns out it did work after all.  I just assumed all that red text was bad.


On the Testing folder

So, I just opened https://github.com/econ-ark/HARK/pull/245 to clean up a file in the Testing folder as per recent discussions.
I wanted to touch on the rest of the files here.

https://github.com/econ-ark/HARK/blob/c394d0f779e78ba89081510cde6c0253b4dcfdc1/Testing/Comparison_UnitTests.py
This one is fit for inclusion in the HARK\tests folder I think. Solves some models and compares them in a meaningful way

https://github.com/econ-ark/HARK/blob/c394d0f779e78ba89081510cde6c0253b4dcfdc1/Testing/ModelTesting.py
Work in progress (?) that seems to test the behavior depending on what parameters are input to the models. Seems fit for HARK\tests but it's not clear to me that it's finished.

https://github.com/econ-ark/HARK/blob/c394d0f779e78ba89081510cde6c0253b4dcfdc1/Testing/MultithreadDemo.py
Not a test, but a demo of multhreading in python? Maybe move to DemARK to show how to use multithreading to quickly solve many variants of a given model on modern CPUs that often have many cores/threads.

https://github.com/econ-ark/HARK/blob/c394d0f779e78ba89081510cde6c0253b4dcfdc1/Testing/TractableBufferStockModel_UnitTests.py
Tests the consumption function against a hardcoded solution vector. May be directly transferable to HARK\tests, but we should probably be quite sure where that vector comes from. Is it just the solution that was there when the file was generated, put in place to avoid changes in behavior of the solution? Or is it meant as some sort of validation against an externally generated (external to HARK that is) solution?

@pkofod, can you @ or assign to whoever you'd like to reply to this/review this?

Okay, yes.

@pkofod
@mnwhite the idea here is to just try to solve the model for many values of parameters, right? We can use CI to test for errors thrown, or we can test for values of the solutions, but it's not 100% clear to me what the idea is with these tests
This does not seem to be a test at all, it's more of a DemARK, right ? @mnwhite states in the initial commit:

Three TBS testing files by DCL have been moved into /Testing, along with
MultithreadDemo (which is sort of a test, and doesn't really fit
elsewhere).  These files probably don't work now due to path changes,
will fix later today.

@pkofod

I'm not sure I wrote any of those files other than MultithreadDemo, which (yes) should be in DemARK.  The other files look like efforts by David Low and/or Nate Palmer to put testing in HARK.  They might be better suited to answer Qs about 1 and 2.
The message you reference is only the initial commit on MultithreadDemo.py because that file used to live elsewhere.

The message you reference is only the initial commit on MultithreadDemo.py because that file used to live elsewhere.

I figured. I'm adding 1 and 4 to the folder CI can find, and then I'll take a closer look at 2. 3 should either go to the cemetery or moved to DemARK.

For a potential sprinter, you may want to find/contact @llorracc (unless he has already written something here by then). Most of the files are handled here, but we still need to figure out what to do with the second file. @npalmer-professional may have something to say about the intent of those files, and how they might be turned into tests of predictable behavior so they are suitable for CI.

This is connected to  issue 18 in the OverARK: https://github.com/econ-ark/OverARK/issues/18 which proposes that we use existing DemARK and REMARK .py files as fodder for Travis.

This is connected two issues in the OverARK repo, #18 which discusses our general plan for unit testing and #29 which talks about using existing DemARK and REMARK .py files as fodder for Travis.
@llorracc @pkofod @mnwhite, can you make sure I've got the plan for this correct?
As I understand it, 1 and 4 are taken care of, 2 is being turned into a unit test, and 3 is being turned into a demark/remark?  Can sprinters help with both 2 and 3?

Testing exists no more!


BIDSLayout.get() returns a massive wall of text in interactive interpreters

BIDSLayout.get() currently returns a simple dict containing all the information a user may want to query. While I believe this is generally the correct behaviour (cf #29), the unfortunate side effect in an interactive console (e.g. ipython) is getting a massive, unusable wall of text when I call BIDSLayout.get() on even a few subjects. In pybids, the data is wrapped in classes with easy to digest repr (showing the file path).
As a more general solution, I would suggest transitioning to a slotted dataclass approach. The performance is roughly equivalent to dicts, and memory usage is better. You'll also get way better control over output, along with built in typing and intellisense, which is super helpful for development. Python has the builtin dataclass module, and there's also the wonderful attrs library. Although Models and all of its child classes are currently implemented as dicts, the current structure looks super clean and I don't think a migration would be too laborious (although I'm hardly intimate with the codebase). At any rate, it would increase maintainability, saving time long term.

@pvandyken thanks for the suggestion
It would be easily fixable as the __repr__() of the base class Model would just have to be adapted to shorten the dict.__repr__() string.
There is also #46 to use dataclasses instead of properties, but I have not yet experimented much with that idea.

Ah, apologies for the overlapping issue. At any rate, the title of the issue, to me, is the more important thing here, and any means of limiting the text output to something useful would be great.

@pvandyken the base Model.repr method is now overridden to summarize the object using only set simple (no lists/dicts) properties.


Enable allocation of new buffers with a warning to allow running decode with trace and prefill without trace

This is required by vLLM (https://github.com/tenstorrent/vllm/issues/14) since we need to interleave executions of a decode trace with prefill (not traced).

Merged to Metal main @ https://github.com/tenstorrent/tt-metal/commit/f0b2483529a55d1101eb142ae1c70eec5260ecf7


Handle redirect in discovery

In the latest version of Cattenbak, we've added support for redirect profiles from CAT. This means that the discovery may contain a simple redirect field for a profile, which means that the app should offer the user the provided link and open a browser when pressed. The flow ends there.
After adding these to the discovery, I noticed that the app hangs on these profiles (busy animation never terminates). Of course it makes sense that the app doesn't support this feature that did not exist when it was made, but it should crash more explicitly when anything is wrong.

Completion criteria:

App must NEVER crash or hang due to unexpected or missing values in discovery, but should show an error message that the institution is not supported
Redirect fields in the discovery lead to a page stating that the app cannot be used for this institution, but instructions on how to configure eduroam are available on the link.

#53

Fixed in commit #53

Handled in commit c5fefc303e5bf2fab7ef9c750d57576e0551aeee

I updated branch to open url outside the app.


sops.template docs with systemd DynamicUser

Thanks for adding sops.template -- it really simplified getting a key into my inadyn service.
Along the way I learned that you can pass credentials into a systemd unit that runs w/ DynamicUser = true. The key bit in the example below is LoadCredential = "inadyn.conf:${config.sops.templates."inadyn.conf".path}";, which exposes the template to the unit at ${CREDENTIALS_DIRECTORY}/inadyn.conf.
Wonder if it'd be worth updating the docs with these bits since I think it's pretty common to set the DynamicUser for better security? If not, figure at least having an example in this issue could be helpful to others in the future.
  sops = {
    defaultSopsFile = ./secrets.yaml;
    age.sshKeyPaths = [ "/etc/ssh/ssh_host_ed25519_key" ];
    secrets."inadyn/some_tld" = { };
    templates."inadyn.conf" = {
      content = ''
        custom namecheap {
          username    = some.tld
          password    = ${config.sops.placeholder."inadyn/some_tld}
          ddns-server = dynamicdns.park-your-domain.com
          ddns-path   = "/update?domain=%u&password=%p&host=%h"
          hostname    = "some.tld"
        }
      '';
    };
  };

  systemd.timers.inadyn = {
    description = "Sync inadyn every hour";
    wantedBy = [ "default.target" ];
    timerConfig = {
      OnBootSec = "60m";
      OnUnitActiveSec = "60m";
    };
  };

  systemd.services.inadyn = {
    description = "Syncs home IP to dynamic DNS.";
    requires = [ "network-online.target" ];
    serviceConfig = {
      DynamicUser = true;
      Type = "oneshot";
      ProtectSystem = "strict";
      ProtectHome = true;
      PrivateUsers = true;
      PrivateTmp = true;
      LoadCredential = "inadyn.conf:${config.sops.templates."inadyn.conf".path}";
      CacheDirectory= "inadyn";
      ExecStart = ''
        ${pkgs.inadyn}/bin/inadyn \
          --foreground \
          --syslog \
          --once \
          --cache-dir ''${CACHE_DIRECTORY} \
          --config ''${CREDENTIALS_DIRECTORY}/inadyn.conf
      '';
    };
  };

Does systemd replace ${CREDENTIALS_DIRECTORY}/inadyn.conf in ExecStart? Didn't knew that.

Yep -- Here are a few key snippets from the systemd docs:

The data is accessible from the unit's processes via the file system, at a read-only location that (if possible and permitted) is backed by non-swappable memory. The data is only accessible to the user associated with the unit, via the User=/DynamicUser= settings (as well as the superuser).

The LoadCredential= setting takes a textual ID to use as name for a credential plus a file system path, separated by a colon.

In order to reference the path a credential may be read from within a ExecStart= command line use "${CREDENTIALS_DIRECTORY}/mycred", e.g. "ExecStart=cat ${CREDENTIALS_DIRECTORY}/mycred".

Apparently this came out with v247 which released in 2020, so it's relatively recent.
Note: Systemd ran into an issue loading the credential when the name of my sops.template lead with a / (e.g. /etc/inadyn.conf instead of just inadyn.conf), I think because the resulting file path included a double-slash //.


A space before the first line of R code trunk

There is always a space before the first line of an R trunk:
For example, if I use the following, then there will be a space displayed before x, which is kinda annoying to my eyes.
x <- 1+1

Btw, thank you for making this beautiful theme!

Maybe you havn't enable the Highlight options ?
https://github.com/xianmin/hugo-theme-jane/blob/335d0c2f69149f63a0a7db0252a0628ac5a0c4a1/exampleSite/full-config.toml#L15-L20
Related issue: https://github.com/xianmin/hugo-theme-jane/issues/46

Highlight was on, and it seems that Blogdown which I used to create the hugo webiste does not support codes highlighting well. With Blogdown, we can create web pages using R markdown so that R codes can be run on the fly. However, with R markdown (instead of ordinary markdown), codes highlighting does not work well.

I think I have just solved the problem by following the first 2 steps from the following link to use highlight.pack.js for codes highlighting:
https://amber.rbind.io/blog/2017/11/15/syntax-highlighting/

I am glad to hear that you solved this problem! :smile:


Add clang format to clean up code

I add clang-format to auto-format the code, and I run it on the main arduino .ino file.
To run the formatter, simply do this from the command line:
./run_clang-format.sh

This should be done before all PRs are merged forever and onward into the future, as a standard practice by the person making the PR before we will merge the PR.
After merging this PR, I could use some help from you getting clang-format running on your Windows machine so you can document the process in the main readme. I've written it up already but am not on a Windows machine to test, so I did some guessing and online searching a bit.
Fixes #21

By the way, using github to review code can be challenging. I highly recommend using meld instead as a git difftool to view changes from one branch to another. It's an incredibly file comparison tool for Windows or Linux. See how to set it up here: https://stackoverflow.com/questions/14821358/git-mergetool-with-meld-on-windows/48979939#48979939

Yep windows version works with that installation instruction, you can update the readme, I approved the PR


error: model_initialize() got an unexpected keyword argument 'cell_fitness_pred' when running "Using Trained Model" tutorial

Hi,
Thank you for your wonderful work! However, when I follow the tutorial "Using Trained Model", I encounter the error
model_initialize() got an unexpected keyword argument 'cell_fitness_pred'

Perhaps this is because of the differences in version of gears. I found that the parameter cell_fitness_pred disappeared in the current version. Can you give some suggestions for this situation?

Tried downgrading the version to 0.0.1, that didn't help. Also, digressing a bit, what's the difference betweeen predict and GI_predict

Thanks for your comment! The Colab works if you downgrade to 0.0.4. I've updated the Colab, let me know if it still doesn't work for you

The Colab has now been updated and works out of the box with the up to date version of GEARS. No downgrading needed


Fresh install fails

Version
@nuxtjs/algolia: 1.1.1
nuxt: 3.0.0-rc.6
Reproduction Link
none
but with version 1.1.0 is seems to work
Steps to reproduce

install nuxt
install this plugin as described here: https://algolia.nuxtjs.org/getting-started/quick-start

What is Expected?
nuxt starts correctly
What is actually happening?
nuxt does not start anymore and throws following error:
 ERROR  Cannot start nuxt:  nuxtCtx.tryUse is not a function                                              15:01:10

  at useNuxt (node_modules/@nuxtjs/algolia/node_modules/@nuxt/kit/dist/index.mjs:47:28)
  at addPlugin (node_modules/@nuxtjs/algolia/node_modules/@nuxt/kit/dist/index.mjs:434:16)
  at setup (node_modules/@nuxtjs/algolia/dist/module.mjs:183:5)
  at Object.normalizedModule (node_modules/@nuxtjs/algolia/node_modules/@nuxt/kit/dist/index.mjs:624:29)
  at async installModule (node_modules/@nuxt/kit/dist/index.mjs:432:3)
  at async initNuxt (node_modules/nuxt/dist/index.mjs:1423:7)
  at async load (node_modules/nuxi/dist/chunks/dev.mjs:6750:9)
  at async Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6795:5)
  at async _main (node_modules/nuxi/dist/cli.mjs:46:20)

Hey, @podlebar ,
Could you please create a reproduction repository or a project in stackblitz? This issue is really strange and I have not encountered it yet

everything works fine with nuxt: 3.0.0-rc.8


Add an option to supply the dead domains list as an argument

In order to make the run more reliable, we could use the following approach:

First run: export the list of dead domains (will need #8 for that).
Second run after several months: again export the list of dead domains.
Compare them and compile a list of domains that are in both the old and the new list.
Final run: supply the list of dead domains and remove only them.

@ameshkov Is this tool for finding dead domains no longer being developed? this tool is very useful