flutter 1.22.4 -> 2.0.3, dart 2.10.5 -> 2.12.2

Motivation for this change
Flutter major upgrade to version 2.0.3.
I've upgraded all patches and made sure they apply and build. I did not check if new patches are required due to new version.
I also had to upgrade dart since we need dart in order to build flutter itself. Since the dart sdk is now included in flutter itself, I'm not quite sure if we need this at all, or if it would require changes to the flutter build declaration. To discuss? Maybe someone more knowledgable than me has a better idea :slightly_smiling_face:
I've been able to create a new flutter app with the new built nix flutter package and build it for android.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117116 at 3be90191 run on aarch64-linux 1

1 package failed to build (new failures):

dart

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/interpreters/dart/default.nix:49:3:
   |
49 |   installPhase = ''
   |   ^

Since the dart sdk is now included in flutter itself, I'm not quite sure if we need this at all

It is included for quite sometime, but we still prefer to build it separately since this way we can put them separately on environment.systemPackages (so if someone only needs Dart they don't need to install Flutter too, for example). This also reduces the total derivation size if you need both installed at the same time.

@thiagokokada I see, makes sense then. Should I split the flutter and dart update in two separate commits?

Hi, I also tried to update this derivation and found a couple of things that would be good to merge in here too:
diff --git a/mkFlutter.nix b/mkFlutter.nix
index 4a7aa0d..61d1327 100644
--- a/mkFlutter.nix
+++ b/mkFlutter.nix
@@ -30,6 +30,7 @@
 , libXi
 , libXext
 , libGL
+, chromium
 , nspr
 , nss
 , systemd
@@ -45,7 +46,6 @@ let

     postPatch = ''
       patchShebangs --build ./bin/
-      find ./bin/ -executable -type f -exec patchelf --set-interpreter $(cat $NIX_CC/nix-support/dynamic-linker) {} \;
     '';

     buildPhase = ''
@@ -104,6 +104,9 @@ let
         which
         xz

+        # needs chrome for web development
+        chromium
+
         # flutter test requires this lib
         libGLU

@@ -137,6 +140,7 @@ runCommand drvName
     #!${bash}/bin/bash
     export PUB_CACHE=''${PUB_CACHE:-"$HOME/.pub-cache"}
     export ANDROID_EMULATOR_USE_SYSTEM_LIBS=1
+    export CHROME_EXECUTABLE=${chromium}/bin/chromium
     ${fhsEnv}/bin/${drvName}-fhs-env ${flutter}/bin/flutter --no-version-check "$@"
   '';
   preferLocalBuild = true;
@@ -155,7 +159,6 @@ runCommand drvName
   };
 } ''
   mkdir -p $out/bin
-
   echo -n "$startScript" > $out/bin/${pname}
   chmod +x $out/bin/${pname}
 ''

More in details:
-      find ./bin/ -executable -type f -exec patchelf --set-interpreter $(cat $NIX_CC/nix-support/dynamic-linker) {} \;

All executables files in ./bin are actually shell scripts and patchelf will complain about it during compilation.
+    export CHROME_EXECUTABLE=${chromium}/bin/chromium

Flutter 2 will now have web development as a compilation target, hence flutter doctor will complain about it if no browser is present. chromium here looks like a good choice to avoid enable allowUnfree.
Hope it was useful! ;)

Anyway, there is an issue with the current approach of separating dart and flutter. That is, the dart compiler cannot find flutter by default (i.e. if you add both flutter and dart to a mkShell).
This is sometimes needed, for example when running
$ dart pub outdated --mode=null-safety
Package doesn't exist (the Flutter SDK is not available).

Running the same command as flutter pub outdated --mode=null-safety works.
Is there an easy way to "enter" the flutter's fhsEnv? That would allow to have a working dart and could also be handy for development, I believe.

Hi, I also tried to update this derivation and found a couple of things that would be good to merge in here too:
diff --git a/mkFlutter.nix b/mkFlutter.nix
index 4a7aa0d..61d1327 100644
--- a/mkFlutter.nix
+++ b/mkFlutter.nix
@@ -30,6 +30,7 @@
 , libXi
 , libXext
 , libGL
+, chromium
 , nspr
 , nss
 , systemd
@@ -45,7 +46,6 @@ let

     postPatch = ''
       patchShebangs --build ./bin/
-      find ./bin/ -executable -type f -exec patchelf --set-interpreter $(cat $NIX_CC/nix-support/dynamic-linker) {} \;
     '';

     buildPhase = ''
@@ -104,6 +104,9 @@ let
         which
         xz

+        # needs chrome for web development
+        chromium
+
         # flutter test requires this lib
         libGLU

@@ -137,6 +140,7 @@ runCommand drvName
     #!${bash}/bin/bash
     export PUB_CACHE=''${PUB_CACHE:-"$HOME/.pub-cache"}
     export ANDROID_EMULATOR_USE_SYSTEM_LIBS=1
+    export CHROME_EXECUTABLE=${chromium}/bin/chromium
     ${fhsEnv}/bin/${drvName}-fhs-env ${flutter}/bin/flutter --no-version-check "$@"
   '';
   preferLocalBuild = true;
@@ -155,7 +159,6 @@ runCommand drvName
   };
 } ''
   mkdir -p $out/bin
-
   echo -n "$startScript" > $out/bin/${pname}
   chmod +x $out/bin/${pname}
 ''

More in details:
-      find ./bin/ -executable -type f -exec patchelf --set-interpreter $(cat $NIX_CC/nix-support/dynamic-linker) {} \;

All executables files in ./bin are actually shell scripts and patchelf will complain about it during compilation.
+    export CHROME_EXECUTABLE=${chromium}/bin/chromium

Flutter 2 will now have web development as a compilation target, hence flutter doctor will complain about it if no browser is present. chromium here looks like a good choice to avoid enable allowUnfree.
I also noticed that in mkDerivation, we do not specify the version, which should make the upgrade system not work as expected. I will therefore suggest to also add an:
inherit patches version;

Hope it was useful! ;)

I think that adding chromium to the derivation is overkill. If you want to use Chrome you can export CHROME_EXECUTABLE environment variable yourself, and this is also good because maybe you don't have Chromium but have Google Chrome/Brave/Any other Chromium based browser installed, and can use what you want.
But yeah, maybe it would be better to document this somewhere.

Anyway, there is an issue with the current approach of separating dart and flutter. That is, the dart compiler cannot find flutter by default (i.e. if you add both flutter and dart to a mkShell).
This is sometimes needed, for example when running
$ dart pub outdated --mode=null-safety
Package doesn't exist (the Flutter SDK is not available).

Running the same command as flutter pub outdated --mode=null-safety works.
Is there an easy way to "enter" the flutter's fhsEnv? That would allow to have a working dart and could also be handy for development, I believe.

I think using flutter pub instead is a good workaround no :thinking: ?

I've added the inherit version and removed the find command, looks like it works fine (building an app worked without issues, but I obviously didn't test everything).

But yeah, maybe it would be better to document this somewhere.

Actually, if you run flutter doctor without chrome specified it will tell you to set CHROME_EXECUTABLE:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.3, on Linux, locale en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Android Studio (version 4.0)
[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

So I think it'll be fine if we don't really document this as flutter already made that env variable easily discoverable.

Ah, not including chromium is indeed better! I thought that FHS meant it will be some sort of isolated environment ala nix-shell --pure, but I definitely misunderstood it xD

This is important to me, what needs to be done so that this MR can be merged?

@roosemberth A review from at least @babariviere would be great 🙂
If you need to, you could also just clone my branch and build flutter yourself and use that for now.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 117116 run on x86_64-linux 1

3 packages built:

dart
flutter (flutterPackages.stable)
hover

Package wise it seems good, still needs to test.

Tested:
$ flutter creater new_project
Your application code is in new_project/lib/main.dart.

$ flutter build web
Compiling lib/main.dart for the Web...                             18.6s

$ flutter run
Unable to run "adb", check your Android SDK installation and ANDROID_SDK_ROOT environment variable:
/nix/store/divwb1bx12x84nps3iak4izd0w37569v-platform-tools-29.0.6/libexec/android-sdk/platform-tools/adb

$ flutter build apk
/nix/store/0w6dqz95lgdwgnp24cc95m184g7yhgka-openjdk-11.0.9+11/lib/openjdk/bin/java: symbol lookup error: /usr/lib/libc.so.6: undefined symbol: _dl_fatal_printf, version GLIBC_PRIVATE
Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                       71ms
Gradle task assembleRelease failed with exit code 127

Hover also seems broken, but maybe we could merge this and fix it later?

Which things are broken?

Which things besides hover are broken?

Just see my tests, flutter build apk is broken, flutter run also seems to not be working (this last one maybe is the lack of an connected Android device in my system, since adb itself works fine).

BTW, flutter build apk is maybe broken because I am running release-20.09, and I am not sure that this package is hermetic so maybe it is leaking somewhere.

I see, so I'm fine in merging and fix the rest later

About the hover issue: I think it was already broken before, it is just that I never needed to run hover init.

About the hover issue: I think it was already broken in master, it is just that I never needed to run hover init.

Yeah, the issue with hover is completely unreleated: https://github.com/NixOS/nixpkgs/pull/118187

@thiagokokada interesting flutter build apk did not work for you as I was able to build an apk just fine. What project were you trying to build?

I am able to run flutter run and flutter build apk on the provided branch (cf. attached flake).
PS: Github won't allow me to attach a file, so I've pasted it on the details section:

```
{
  inputs.nixpkgs.url = "github:kolaente/nixpkgs?ref=ba619289b62453a4957e540c109880d5d261ce36";
outputs = { self, nixpkgs }: let
systems = [ "x86_64-linux" "i686-linux" ];
forAllSystems = f: nixpkgs.lib.genAttrs systems (system: f (import nixpkgs {
inherit system; overlays = [ self.overlay ];
config.android_sdk.accept_license = true;
}));
in {
overlay = final: prev: with final; {
android = androidenv.composeAndroidPackages {
platformVersions = [ "28" ];
abiVersions = [ "x86_64" ];
buildToolsVersions = [ "29.0.2" ];
};
emulator = androidenv.emulateApp {
name = "emulator";
enableGPU = true;
platformVersion = "28";
abiVersion = "x86_64";
};
};
devShell = forAllSystems (pkgs: with pkgs; stdenv.mkDerivation {
  name = "android-env";
  ANDROID_HOME = "${android.androidsdk}/libexec/android-sdk";
  ANDROID_JAVA_HOME = "${jdk.home}";
  JAVA_HOME = "${jdk.home}";
  nativeBuildInputs = [ android.androidsdk emulator flutter ];
  allowSubstitutes = false;
});

# Export all packages from the overlay attribute.
packages = forAllSystems (pkgs: with pkgs.lib;
  getAttrs (attrNames (self.overlay {} {})) pkgs
);

apps = forAllSystems (pkgs: with pkgs; {
  emulator.type = "app";
  emulator.program = "${emulator}/bin/run-test-emulator";
});

};
}
</details>

Hooray! :tada:

@kolaente Like I said, I think this package is not hermetic enough. This GNU GLIBC error probably means that there is a mismatch between GLIBC versions (I am using release-20.09, I think most people are testing this in unstable).
This is something that should be eventually fixed, since non hermetic packages are bad since they're non reproducible.

@thiagokokada How about working that out in a new issue describing what is expected and how it differs from the result under what circunstances?


Interface for Enfo stack tags

Create a well documented interface for our stack tags.

See discussion on https://github.com/enfogroup/aws-cdk-compliance/pull/50 for more information


Is the slide resolution taken into account when extracting tiles ?

Hello,
Thank you for this nice package !
I used histoprep on two slides with different resolutions.
One was scanned at a mpp of 0.25, the other one with a mpp of 0.5. It seems that when extracting the tiles with the following code, it reads images of size 512x512 at the lowest openslide level, without taking in account the resolution. This implies that the cell scale etc.. differs between the images extracted on the two different slides (see images below, first is mpp 0.25, second is mpp 0.5).
from histoprep import SlideReader

# Read slide image.
reader = SlideReader("./slides/slide_with_ink.jpeg")
# Detect tissue.
threshold, tissue_mask = reader.get_tissue_mask(level=-1)
# Extract overlapping tile coordinates with less than 50% background.
tile_coordinates = reader.get_tile_coordinates(
    tissue_mask, width=512, overlap=0.5, max_background=0.5
)
# Save tile images with image metrics for preprocessing.
tile_metadata = reader.save_regions(
    "./train_tiles/", tile_coordinates, threshold=threshold, save_metrics=True
)

Is this an intended behaviour or am I missing something?
Best

This is intended behaviour. If this feature is something that you need feel free to add a pull request! This could be implemented for example as a helper method get_width_from_mmp, which could then be passed into get tile coordinates.


fix dependencies versions

These are package fixes for icondusk-e2e, but since we dont have yet jenkins tests I will merge it for testing it with jenkins

launch jenkins dawn

launch jenkins dawn4py


az webapp create fails because Azure CLI reports app service exists in a different resource group then specified (which is not the case)

This is autogenerated. Please review and update as needed.
Describe the bug
We can no longer run out deployment scripts against out PROD environment since Azure CLI has been updated to 2.10.1
Command Name
az webapp create
Errors:
The webapp '<app-service>' exists in resource group '<group>_devtest' and does not match the value entered '<group>'. Please re-run command with the correct parameters.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp create --resource-group {} --name {} --plan {} --only-show-errors

Expected Behavior
The command should pass without making any changes since the web app already exists
Environment Summary
Linux-4.19.104-microsoft-standard-x86_64-with-debian-buster-sid
Python 3.6.10
Installer: DEB

azure-cli 2.10.1

Additional Context
The app service we create here already exists in the resource group <group> and not in <group>_devtest.  <group>_devtest is a resource group we use for DEV purposes and there is and there is an app service named  '-dev' in there. As of this version of the Azure CLI some check seems to indicate that the app service is in a different resource group then it really is
az webapp show --resource-group {} --name {} with the same name an resource group returns the app service object as well

webapp

Thanks @qwordy for fixing this 👍

Thanks @calvinsID for fixing this 👍


WAMR implementation

[x] WAMR on a Linux device
[ ] WAMR on Pico

As the PR is already big enough, I've decided to merge it with root and do the Pico specific WAMR implementation in a new PR.


Support includes in preview and preview synchronisation

When a document has includes a bunch of files, changing location in the source document does not align in the preview. This is a pretty cool feature of Asciidoc and supporting it in this extension is a worthy (but non-trivial goal).
What should be the correct behaviour?
One option:

With a preview of the "main" document, the line numbers should align correctly when the user scrolls in the source.
If while the preview of the "main" document is open and an "included" file is opened, then the preview should scroll to the appropriate location in the included file.

I'm looking into doing this by writing a "data-line" and "data-file" attribute to the output. Information for both of these are available when sourcemap is set on asciidoctor. This is semantically better than the current solution of putting a role with the line number in (which is hard to do with the file name without doing weird stuff) but requires the use of templates and converters. I'm not sure whether a better approach is possible.

@danyill I agree that addRole is not the proper way to do it, the current use for data-line-number is ugly. Unfortunately I couldn't find an easy way to set attributes, like you mention, we probably need to change the html5 backend to handle those.

hi @joaompinto
Yes, I hope it's not an impossible goal. I have done a little preliminary research. At least for Ruby this can be done using Tilt templates and a Treeprocessor to expose the information based on adding an attribute to a block.
I have a prototype I'll hope to share in some time. It works as follows:

Use a TreeProcessor extension in Ruby. This extension adds to each block an attribute for filename and an attribute for line number:

class LineNumberTreeprocessor < Extensions::Treeprocessor

    def process document
        return unless document.blocks?
        if (document.attr? 'apply-data-line')

            process_blocks document
        end
    end

    def process_blocks document
        (document.find_by).each do |block|
            if !block.lineno.nil?
                block.set_attr 'data-file', block.file
                block.set_attr 'data-line', block.lineno.to_s
                # this is not semantically righteous
                block.add_role 'data-line-' + block.lineno.to_s
            end
        end
    end

end

Create a template which overrides the html output and includes this information. For instance in haml the file block_paragraph.html.haml (Asciidoctor uses the filenames to determine what to overwrite):

- if attr? :'data-file'
  %div{:id=>@id, :class=>['paragraph', role], :'data-file'=>(attr :'data-file'), :'data-line'=>(attr :'data-line')}
- else
  %div{:id=>@id, :class=>['paragraph', role], :'data-line'=>(attr :'data-line')}
- if title?
  .title=title
%p<=content

Call this extension using either the sourcemap attribute or another extension from https://github.com/asciidoctor/asciidoctor-extensions-lab (the docs for it is missing it but there is enable-sourcemap-preprocessor.rb). I use:
asciidoctor --trace --verbose -E haml -b html5 -T ./lib/line-number-treeprocessor/templates -r ./lib/enable-sourcemap-preprocessor.rb -r ./lib/line-number-treeprocessor.rb lib/line-number-treeprocessor/sample.adoc

Which produces an output with:
div class="paragraph data-line-139" data-file="/home/mulhollandd/Documents/asciidoctor-extensions-lab/lib/line-number-treeprocessor/sample.adoc" data-line="139">

Integer sollicitudin magna blandit leo condimentum fermentum.
Aliquam sit amet consectetur purus.
Aliquam sed dui blandit ligula fermentum tincidunt.
Donec vitae augue non urna lobortis dignissim non dictum velit.
Maecenas ac metus volutpat, vehicula mauris at, dignissim odio.
Suspendisse at enim facilisis, aliquam ligula vel, imperdiet lectus.
Proin augue leo, finibus ac leo in, volutpat vulputate enim.
Aliquam rutrum est vitae ultricies semper.
Nullam a lacinia urna.

Warning

Suspendisse at enim facilisis, aliquam ligula vel, imperdiet lectus.

Cheshire Cats

Chapter 6 (an excerpt)

There was certainly too much of it in the air. Even the Duchess sneezed occasionally; and as for the baby, it was sneezing and howling alternately without a moment’s pause. The only things in the kitchen that did not sneeze, were the cook, and a large cat which was sitting on the hearth and grinning from ear to ear.

‘Please would you tell me,’ said Alice, a little timidly, for she was not quite sure whether it was good manners for her to speak first, ‘why your cat grins like that?’

‘It’s a Cheshire cat,’ said the Duchess, ‘and that’s why. Pig!’

At the moment I've only overwritten the paragraph but you can see the data-line and data-file attributes here. The source file has an include so you can see this changing.
What do you think? Could we do something like this? It is very verbose to add the file to each paragraph but I don't see a better option. It is also quite verbose to have to overwrite all the templates to get similar performance to adding role but I don't think the other extension hooks allow anything else (as far as I can  tell).

@danyill I am still thinking on the functional implications of this, right now the preview window is totally refreshed when you switch to a different source file. As it is on your point 2, besides switching/scrolling to the include file would also replace the preview content to the included file only, would that be ok ?

@joaompinto yes, I'm not sure what the best way is to do this smoothly.
Either:

Only when the include file is open change focus and scroll to it.
Ignore include files when scrolling across the "main document" (at least renumber this correctly so that it represents the file which is open)

or

Open include files as the user scrolls (but probably not close them afterwards)

There's probably other options.
Both seem slightly tricky UI issues and I'm not sure what is possible within VS.

Maybe we need 2 view modes, preview mode, which will render the includes, and edit mode which will just place a button signalling there is an includes.
I am still researching on how to trigger VS editor actions from within the preview actions, (this is required (preview -> editor), the preview window runs in a sandbox process, so it's not trivial to implement bidirectional behaviour.

@joaompinto see my PR above for this code in a better form.
From the point of view of minimising preview -> editor interaction:

Perhaps the easiest "first option" would be only to scroll when data-file = the open file.
Is it reasonable to allow the preview to stay open if the "main file" remains open and any of the 'include files" (which strictly should include files included from other files as well!).
I need to look into how to get a list of include-ed files from the parent Asciidoc.


Migrated to .NET Core 3

This works 100%, was mostly a drop in replacement. Waiting on .NET Core 3 to leave preview so that I can officially support this.

Just FYI: .NET Core 3 is now officially out. It's actually finally, really out. After all these years :smile:

Awesome! I'll see if I can snag enough time this weekend to do some final tests and merge it in! Thanks for the poke @josteink !

no promises, of course, because I'm bad at keeping this project up to date, but the plan is to actually merge this in this weekend, and setup GitHub Actions at the same time (it looks like GitHub actions will be easier to use with dotnet core).

but the plan is to actually merge this in this weekend

The weekend ended up being about 8 months long, but I'm finally merging this. I'm very far behind on production releases of this. The new plan is to merge this in right now! And then start to address some of the new open issues before doing a new prod release and catching up.
Apologies for the dead repo people! I feel like I've neglected this project. I don't use a windows PC often at home anymore, so I haven't had a need to many any changes here.


Support OIDC in eventshub receiver

Fixes https://github.com/knative/eventing/issues/7334
Changes

:gift: Add OIDC support to eventshub receiver

As soon as an eventshub receiver was setup with an audience (via OIDCReceiverAudience(aud)), it will check verify the JWTs of incoming requests (and respond with a 401, if failed).
Hints for reviewers:

Added the OIDCSinkAudience(aud) option too, to define the audience of the sink, which is helpful/needed, if the audience of the sink can't be determined by the source itself (e.g. when the sink does not provide the .status.address.audience field)
We can add the OIDC service account name to the eventInfo and then later base some asserter on it, but I didn't see a real usecase for it at that time of the OIDC implementation
We verify the tokens with the TokenReviewAPI to not add another dependency to some OIDC library (e.g. coreos/go-oidc) and safe the setup.

Support OIDC in eventshub receiver

/lgtm

Failing due to
panic: test timed out after 10m0s

As the expiredToken test takes at least 10 minutes, I increased the timeout in 0145c0b840bae978dc9ec4a595809a377808ff45

@pierDipi can you recheck? I only increased the timeout in 0145c0b840bae978dc9ec4a595809a377808ff45

/cc @pierDipi
Could you recheck? I needed to increase some timeouts for the expiredtoken test


Allow overriding date on documents

Can we do the same date "hack" for documents as we have for other content types?
https://github.com/gbif/portal16/blob/cf1ecebecadc2108cb99c92efe877016963e1356/app/views/pages/resource/key/document/documentContent.nunjucks#L9
main.fields.meta.drupal.created
I've added the meta field to the content model on DEV...

Thanks. Works like charm. I've updated the prod content model as well. Please deploy to prod whenever convenient...


Issues Noticed with Auth

I have been working with auth via different methods.

JWT expired and the session lost 😩 I noticed it’s expired after the period written in Supabase which can be maximum 1 week. Wish if we can stay logged in until it’s logged out by the user Like instagram.

Provider sign in has to ways a. Sigin method which doesn’t lunch the url you have to lunch it by yourself b. SigninWithProvider which return bool

both methods doesn’t return the session and don’t trigger onAuthStateChange

Phone auth is cool but I would highly recommend if we can sign users with phone directly “ In my case Im using other sms provider api so I had to workaround and signup the user first”

Thanks and hope it will be fixed soon🔥

I'm curious why the session isn't automatically being refreshed. Probably a bug that can be looked into.
There's a PR for signInWithProvider to return the correct value.

@DanMossa
Just checked your PR it would be amazing if it's fixed.
It could be in this way or any other way as long as it allowed me signin with phone without sigin it up.

@DanMossa
#73 as a way to fix the issue i can see you return the GotrueSessionResponse instead of bool "That's cool but still the response will return 2 things url and provider name.
I believe the main issue is in the _handleProviderSignIn here where the session should be given

#33

I guess we can close this a duplicate of the above issues


Add eslint file

Decide the default rules, we can begin with the old jslint rules and take care about rules in ES5, ES2016 and react(with JSX)

I'd recommend starting with airbnb rules for both plain js and jsx
https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb

I like it too

Yeah, our style guide (https://github.com/Jam3/Javascript-Code-Conventions) is based off airbnb with some slight modifications, so its a good place to start

Some additional hoops to jump through for integrating with React and Babel:
https://github.com/yannickcr/eslint-plugin-react
https://github.com/babel/babel-eslint
Also need to make sure Sublime is parsing the file using Javascript (Babel) as the syntax and not JSX

Add in the documentation more data like which IDE plugin should be used

On Gout we used this which seemed to work really well for our team:
https://github.com/Jam3/prj-gout/blob/master/.eslintrc
It required some additional devDependencies. We also had a auto-formatter which was useful when pulling in code from another project.
We also needed this ignore:
https://github.com/Jam3/prj-gout/blob/master/.eslintignore
Maybe @jmckinnell and @namniak can chip in.

I will take it a look @mattdesl and merge the rules with the current one. Thanks


Crash immediately after character selection

The Crash happens immediately after character selection. Using Gw Launcher and version 3.8.1
crashes only for 1 alt account.

crash dump:
crash dump.txt

Can you attach the file from C:\Users\sang\AppData\Local\GWToolboxpp\crashes\3.8.1-20200621-174111-28048-11996.dmp please?

Hi, I can't seem to attach dump files in github. Any other way to send the file?

right click and zip it

3.8.1-20200621-174111-28048-11996.zip

Fixed for next release https://github.com/HasKha/GWToolboxpp/commit/f15682ce8b61bb0a09e872d9ca458ef7ec55f785


Class property inference from constructor does not work with private class fields

TypeScript Version:  4.0.0-dev.20200803

Search Terms: class property inference, private class fields
Expected behavior: private class field's type is inferred from its assignment in the constructor
Actual behavior: typechecker reports implicit any for private class field

Related Issues:
Code
class Example {
  #test;

  constructor(test: number) {
    this.#test = test;
  }

  get test() {
    return this.#test
  }
}

Output
"use strict";
var __classPrivateFieldSet = (this && this.__classPrivateFieldSet) || function (receiver, privateMap, value) {
    if (!privateMap.has(receiver)) {
        throw new TypeError("attempted to set private field on non-instance");
    }
    privateMap.set(receiver, value);
    return value;
};
var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, privateMap) {
    if (!privateMap.has(receiver)) {
        throw new TypeError("attempted to get private field on non-instance");
    }
    return privateMap.get(receiver);
};
var _test;
class Example {
    constructor(test) {
        _test.set(this, void 0);
        __classPrivateFieldSet(this, _test, test);
    }
    get test() {
        return __classPrivateFieldGet(this, _test);
    }
}
_test = new WeakMap();

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "alwaysStrict": true,
    "esModuleInterop": true,
    "declaration": true,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "moduleResolution": 2,
    "target": "ES2017",
    "jsx": "React",
    "module": "ESNext"
  }
}

Playground Link: Provided

I'm not sure if TS can support this, the syntax sugar for adding the type in the constructor generates a public field but the #syntax isn't allowed in the constructors params

and

If we added support for this we might end up taking syntax space from JavaScript

@orta That's not what I was referring to. I was referring to this recently released behavior which exists for non-private fields.
https://devblogs.microsoft.com/typescript/announcing-typescript-4-0-rc/#class-property-inference
The playground that is linked demonstrates the bug without need for modification.

Yep, I know what you're talking about 👍🏻
My point was that that syntax space is already taken for the implicit field which is created how it currently works. We'd have to add support for putting #fieldname in a constructor in order to disambiguate,

I'm afraid I don't really understand what you mean. The playground I linked doesn't use any TypeScript-specific syntactic sugar. Are you referring to parameter properties?
https://www.typescriptlang.org/docs/handbook/classes.html#parameter-properties
// no implicit field created
constructor(test: number) { }

// implicit field created
constructor(public test: number) { }

Except for the one type annotation, the playground I linked is valid vanilla JS so I don't really understand what that has to do with what is or isn't valid in the constructor's params.
Here's another demonstration that's maybe more clear.

class Example {
  #test;  // should not be implicit any error

  constructor() {
    this.#test = 42;
  }
}

This works with no errors in noImplicitAny
class Example {
  test;

  constructor(test: number) {
    this.test = test;
  }

  get publicTest() {
    return this.test
  }
}

But the original example code doesn't. That should be fixed.

@ahejlsberg think this would be trivial to fix before the 4.0 stable?


Update sensitivity-labels-versions.md

Split PDF support table entry to encryption and non-encryption parts.

Not needed at this time

Not needed at this time

Not needed at this time

Alright thanks Carol.


Consider changing MeanToMid() to CvtToFuzzyMeanToMid()

The MeanToMid operator converts raw data to fuzzy data, similar to how the other "Convert to Fuzzy Operators" work (e.g.,  CvtToFuzzyCurve, CvtToFuzzyCurveZScore, CvtToFuzzyCat, etc).
For consistency, I think we should change the name of this operator from MeanToMid() to CvtToFuzzyMeanToMid()
https://github.com/consbio/mpilot/blob/9d830eee5b61b23e66eff006af61b9432c778fc8/mpilot/libraries/eems/fuzzy.py#L216
Both EEMS3.0 for ArcGIS and EEMS Online  call the operator CvtToFuzzyMeanToMid(), so currently uploads will fail if a model contains a MeanToMid() operator.

Sounds good to me!
On Tue, Apr 6, 2021 at 2:05 PM mikegough @.***> wrote:

The MeanToMid operator converts raw data to fuzzy data, similar to how the
other "Convert to Fuzzy Operators" work (e.g., CvtToFuzzyCurve,
CvtToFuzzyCurveZScore, CvtToFuzzyCat, etc).
For consistency, I think we should change the name of this operator from
MeanToMid() to CvtToFuzzyMeanToMid()
https://github.com/consbio/mpilot/blob/9d830eee5b61b23e66eff006af61b9432c778fc8/mpilot/libraries/eems/fuzzy.py#L216
Both EEMS3.0 for ArcGIS and EEMS Online call the operator
CvtToFuzzyMeanToMid(), so currently uploads will fail if a model contains a
MeanToMid() operator.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/consbio/mpilot/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7CJ2YTL2GPH56KVUJNXDLTHNZSLANCNFSM42PPWYGA
.

Fixed in 1.0.0b18.


feature request: let user change cache folder

we spoke about this in the S2S AI call on thursday. would it be possible to let the user set the location of the downloaded files?
in case that location gets filled up? I thought it was /tmp but I cannot find the data there. I understand that the filesnames are hashed. its just that I dont want to spill a location on disk I cannot control.

as far as I can see you wrote the downloading and caching in climetlab from scratch: you could also see if you want to build on top of intake and fsspec. my usage example: https://renkulab.io/gitlab/aaron.spring/s2s-ai-competition-bootstrap/-/blob/master/notebooks/ECMWF_intake.ipynb
fsspec caching: https://filesystem-spec.readthedocs.io/en/latest/features.html#caching-files-locally

This already works, see https://climetlab.readthedocs.io/en/latest/reference/settings.html

great. thanks @b8raoult


Ping to default gateways (per network) to see if local connection is …

…alive.
This helps to distinguish between the case that pi does not have connection to the phone,
and the case that connection to the phone exists but phone has no external IP connection.
Also in the case that both WiFi and BT tethering is enabled one can see which of them is used (the one with the lower metric).
Here is an example to the output (when both WiFi and BT are enabled.):
At Mon 23 Dec 2019 04:50:08 PM IST my local Bluetooth IP is: <IP_ADDRESS>
At Mon 23 Dec 2019 04:50:10 PM IST ping to default gateway <IP_ADDRESS> (bnep0 metric = 0) passed
At Mon 23 Dec 2019 04:50:10 PM IST ping to default gateway <IP_ADDRESS> (bnep0 metric = 214) passed
At Mon 23 Dec 2019 04:50:11 PM IST ping to default gateway <IP_ADDRESS> (wlan0 metric = 302) passed
At Mon 23 Dec 2019 04:50:11 PM IST my public IP is: <IP_ADDRESS>
Signed-off-by: Tzachi Dar<EMAIL_ADDRESS>

Yws, only diagnostics.


fix: Add missing calls to addMessageToConsole in console client's tim…

…e() and timeEnd()

PR Checklist

[x] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
[x] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
[x] You have signed the CLA.
[x] All existing tests are passing: https://github.com/NativeScript/ios-runtime/blob/master/CONTRIBUTING.md#contribute-to-the-code-base
[ ] Tests for the changes are included.

What is the current behavior?
console.time() and console.timeEnd() are not working with Chrome DevTools debugger.
What is the new behavior?
Both methods print proper values in DevTools console.
Fixes #888 .

The PR build has failed. You seem to have forgotten to update WebKit with making ConsoleMessage objects copyable.


[Enhancement] Sort function for array type

$subject please. This will help enable effective usage of arrays and quick inbuilt ability to sort an array.

We do have a sort function to sort an array of strings. But that is not enough.

This is added with the array lang library. https://github.com/ballerina-platform/ballerina-lang/blob/master/langlib/lang.array/src/main/ballerina/src/lang.array/array.bal#L170


Improvements upon set filter modifier

First commit is unrelated to my changes, but was necessary since pypika's version 0.37.6 changed how functions were formatted. Fyi aliases of nested args are no longer serialised. Therefore that broke some tests in fireant.
Despite that, this PR adds a couple new features to the ResultSet filter modifier, such as the ability to break down the complement set. Some fixes, especially regarding the use of share dimensions/totals
with sets were also addressed. I've added some documentation on the aforementioned features as well, given API is stabler now.

Coverage increased (+0.1%) to 91.499% when pulling 7a00a9c2bc734118283b59bf7012a203f60d87ce on x8lucas8x:improvements-upon-set-filter-modifier into 2f7bc012e697452912ce37d8e5a9a23c33231f7a on kayak:master.


DO NOT MERGE: Add debug logging to help fix weird WSLg bug

@Omotoye reported in a private repository that when restoring fullscreen on WSLg, they get a crash. This adds a bunch of logging to help diagnose what is going wrong.
Please help test this:
git clone git@github.com:emilk/egui.git
cd egui
git checkout emilk/debug-fullscreen-failure

Then run this:
RUST_LOG=debug cargo r -p egui_demo_app --no-default-features --features wgpu,persistence

toggle fullscreen (F11), exit, and then run the same command again.
Does it work, or fail in the same way?
Please share the log from the two runs!

Thanks for testing! Interesting that you cannot replicate now.
I've added some more settings to make it more similar to Rerun. Can you please git pull and again run:
RUST_LOG=debug cargo r -p egui_demo_app --no-default-features --features wgpu,persistence

Still fine, no errors (at least the kind that we expect)... Logs below
terminal log

omotoye@Omotoye:~/egui$ git status
On branch emilk/debug-fullscreen-failure
Your branch is up to date with 'origin/emilk/debug-fullscreen-failure'.

nothing to commit, working tree clean
omotoye@Omotoye:~/egui$ git pull
remote: Enumerating objects: 75, done.
remote: Counting objects: 100% (75/75), done.
remote: Compressing objects: 100% (17/17), done.
remote: Total 75 (delta 46), reused 74 (delta 45), pack-reused 0
Unpacking objects: 100% (75/75), 10.79 KiB | 290.00 KiB/s, done.
From https://github.com/emilk/egui
   739d918e..5e02cfa2  emilk/debug-fullscreen-failure -> origin/emilk/debug-fullscreen-failure
 * [new branch]        emilk/simplified-usage         -> origin/emilk/simplified-usage
   4e8341d3..37fd141d  master                         -> origin/master
Updating 739d918e..5e02cfa2
Fast-forward
 crates/egui_demo_app/src/main.rs | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)
omotoye@Omotoye:~/egui$ RUST_LOG=debug cargo r -p egui_demo_app --no-default-features --features wgpu,persistence
warning: /home/omotoye/egui/crates/eframe/Cargo.toml: unused manifest key: target.cfg(not(target_arch = "wasm32")).dependencies.glutin.es
   Compiling egui_demo_app v0.20.0 (/home/omotoye/egui/crates/egui_demo_app)
    Finished dev [unoptimized + debuginfo] target(s) in 1m 14s
     Running `target/debug/egui_demo_app`
2022-12-14T19:48:36.294268Z DEBUG eframe: Using the wgpu renderer
2022-12-14T19:48:36.332956Z DEBUG eframe::native::run: Entering the winit event loop (run_return)…
2022-12-14T19:48:36.337483Z DEBUG eframe::native::file_storage: Loading app state from "/home/omotoye/.local/share/eguidemoapp/app.ron"…
2022-12-14T19:48:36.341398Z  INFO eframe::native::epi_integration: Monitor size: 2256x1504 pixels, 2256x1504 points
2022-12-14T19:48:36.341560Z  INFO egui_winit::window_settings: Clamping size to 2256x1504 points
2022-12-14T19:48:36.341582Z  INFO egui_winit::window_settings: Restoring window size to 2256x1504 points, with fullscreen=false
2022-12-14T19:48:36.561494Z DEBUG crossfont::ft: Loaded Face Face { ft_face: Font Face: Book, load_flags: TARGET_LIGHT, render_mode: "Normal", lcd_filter: 1 }
2022-12-14T19:48:37.396925Z  WARN wgpu_hal::vulkan::instance: Unable to find layer: VK_LAYER_KHRONOS_validation
2022-12-14T19:48:37.642479Z  WARN wgpu_hal::gles::egl: Re-initializing Gles context due to Wayland window
WARNING: lavapipe is not a conformant vulkan implementation, testing use only.
2022-12-14T19:48:37.861476Z DEBUG egui_winit::clipboard: Initializing arboard clipboard…
2022-12-14T19:48:37.864066Z DEBUG egui_winit::clipboard: Cannot initialize smithay clipboard without a display handle
2022-12-14T19:48:39.132821Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:48:43.892017Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:48:43.892299Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:48:43.962813Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:48:43.963077Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to EwResize
2022-12-14T19:48:44.260353Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to EwResize
2022-12-14T19:48:44.260586Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:48:53.709909Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:48:53.710197Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:48:53.782492Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:48:53.782729Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:12.311930Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:12.313389Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:49:12.532540Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Crosshair
2022-12-14T19:49:12.534536Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Text

2022-12-14T19:49:12.724663Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Text

2022-12-14T19:49:12.725196Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:13.710164Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:13.710670Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to NsResize
2022-12-14T19:49:14.031138Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to NsResize
2022-12-14T19:49:14.032082Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:24.143246Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:28.823197Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:28.823471Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to EwResize
2022-12-14T19:49:29.664036Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to EwResize
2022-12-14T19:49:31.105770Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to EwResize
2022-12-14T19:49:31.106393Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Hand

2022-12-14T19:49:31.339310Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Hand

2022-12-14T19:49:31.339535Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:31.748795Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:33.211310Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:33.730512Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:38.760015Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:38.801311Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:39.069814Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:40.362926Z ERROR sctk_adwaita::pointer: Failed to set cursor
2022-12-14T19:49:40.702548Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
2022-12-14T19:49:43.333219Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
^C
omotoye@Omotoye:~/egui$ RUST_LOG=debug cargo r -p egui_demo_app --no-default-features --features wgpu,persistence
warning: /home/omotoye/egui/crates/eframe/Cargo.toml: unused manifest key: target.cfg(not(target_arch = "wasm32")).dependencies.glutin.es
    Finished dev [unoptimized + debuginfo] target(s) in 0.52s
     Running `target/debug/egui_demo_app`
2022-12-14T19:50:02.766622Z DEBUG eframe: Using the wgpu renderer
2022-12-14T19:50:02.772840Z DEBUG eframe::native::run: Entering the winit event loop (run_return)…
2022-12-14T19:50:02.772966Z DEBUG eframe::native::file_storage: Loading app state from "/home/omotoye/.local/share/eguidemoapp/app.ron"…
2022-12-14T19:50:02.773783Z  INFO eframe::native::epi_integration: Monitor size: 2256x1504 pixels, 2256x1504 points
2022-12-14T19:50:02.773853Z  INFO egui_winit::window_settings: Clamping size to 2256x1504 points
2022-12-14T19:50:02.773897Z  INFO egui_winit::window_settings: Restoring window size to 2256x1504 points, with fullscreen=false
2022-12-14T19:50:02.782821Z DEBUG crossfont::ft: Loaded Face Face { ft_face: Font Face: Book, load_flags: TARGET_LIGHT, render_mode: "Normal", lcd_filter: 1 }
2022-12-14T19:50:02.826722Z  WARN wgpu_hal::vulkan::instance: Unable to find layer: VK_LAYER_KHRONOS_validation
2022-12-14T19:50:02.860677Z  WARN wgpu_hal::gles::egl: Re-initializing Gles context due to Wayland window
WARNING: lavapipe is not a conformant vulkan implementation, testing use only.
2022-12-14T19:50:02.903340Z DEBUG egui_winit::clipboard: Initializing arboard clipboard…
2022-12-14T19:50:02.904329Z DEBUG egui_winit::clipboard: Cannot initialize smithay clipboard without a display handle
2022-12-14T19:50:03.402545Z  WARN winit::platform_impl::platform::wayland::seat::pointer: Failed to set cursor to Default
^C
omotoye@Omotoye:~/egui$


Asset Model Issue

Within the View Asset Model: The corresponding asset options are not appearing correctly. Asset options are either empty or not corresponding with the asset that was selected.

Fixed following the recent 1.2.0 release


Added section on how to specify the package name for module federation

Hello,
I added a little section on how to specify the package name when using shared with module federation plugin. This should fix #5476.

@Christian24 mind rebasing?

@Christian24 mind rebasing?

@evenstensberg give me a couple hours. :)

@evenstensberg is there something else I need to do here?

I think we can merge. Deploy previews from forks need spproval on vercel, not sure who has the access.

I can too but need to figure out what is wrong with the deployment phase. For reference, I can approve the deploy but it doesn't update the pipeline or comments the deployment url...


provide definition for 'observational data item about drug-drug interaction'

This was meant to represent observations like the ones underlying case reports.

added the following definition:
"A data item that is about a drug-drug interaction and the creation of which was triggered by an observation outside a controlled context (e.g. a clinical trial)."


[Enhancement] Move records folder from home directory

Describe the bug
On my Archlinux install, using SpeedRunIGT in MultiMC, the mod creates a speedrunigt folder directly in my home directory.
The speedrunigt folder in the directory of the Minecraft instance remains unused.
Expected behavior
I would prefer, if the records were stored in the directory of the Minecraft instance, or alternatively, if the folder was renamed to .speedrunigt to make it hidden by default.
Config, cache and other application folders are not usually stored as unhidden folders in ~
Screenshots or logs

Versions info :

MC Version: 1.16.1 in MultiMC 0.7.0-3772
SpeedRunIGT Version: 13.6+1.16.1
Fabric Loader Version: 0.14.21

Fixed from #93

By creating a .hidden file in the home directory with the simple contents speedrunigt, it should hide the folder. This may not happen by default for some machines, so see the full answer I found here: https://superuser.com/a/816004


Add stubs directory to search path for transitive linking deps

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Resolves #40

@conda-forge-admin, please re-render

@bdice @jakirkham

Closing as this issue is fixed in newer cmake versions. see discussion https://github.com/conda-forge/cuda-cudart-feedstock/pull/30#discussion_r1608861874


NR database - no node for taxid ID

Hi all,
I am getting started with mmseqs and tried to download the nr database with
mmseqs databases NR mmseqs2_nr tmp
However at the end of the process there are many lines like:
No node for taxID 2742838, ignoring it.
No node for taxID 2742840, ignoring it.
No node for taxID 2742841, ignoring it.
No node for taxID 2742842, ignoring it.
No node for taxID 2742843, ignoring it.
No node for taxID 2742845, ignoring it.
No node for taxID 2461416, ignoring it.
Could you let me know how to proceed here? Thanks!

Hi there - was wondering if this is not the place for this question? could you let me know? thanks!

We were a bit busy with a paper submission.
This behavior is kind of okay and expected.
The procedure to map an NR entry to a taxonomic label is a bit involved since NCBI's NR, accession2taxid and taxdump don't match very well and it's difficult to bring them into a consistent state.
This message happens when we can parse a taxonomic identifier either by looking the accession up in one of the accession2taxid files, but the taxonomic identifier is not contained in the taxdump anymore. In this case we either try to take the taxonomic identifier of the next sequence that was clustered into this entry or leave this entry unannotated (thus it cannot contribute to LCA computation despite being findable through searching).
For nearly all sequences the mapping procedure works correctly (331375261 vs 331433500) the last 50K entries are however kind of tricky and we don't have a good solution yet to get a taxonomic identifier automatically for those.


Update dashboard template details view

Update dashboard template details view to use design system and match figma: https://www.figma.com/file/bMhG3KyrJF8vIAODgmbeqT/Design-System?node-id=467%3A13745

Acceptance Criteria

No regressions in a11y usability of detail view
View should match updates in figma

@bmattb I'm sorry i'm adding to scope in the final 3 days of the sprint, I didn't realize I'd missed the template detail view when I made tickets for the dashboard rewrite. I'm almost done with this now.

Testing directions copied from PR:
Go to 'explore templates' and click on 'see details' when you hover over any template.

See that pagination still works as expected
See that 'use template' button still works as expected
See that 'close' button still works as expected

Verified in QA


did:web Create() URL-safe encoded path parameters

The spec specifically describes how to handle a port in a given domain https://w3c-ccg.github.io/did-method-web/#example-creating-the-did-with-optional-path-and-port
...but what about other special characters in a path. For example, what about a space /something/with a/space/did.json?
Should all path params be percent encoded?
What about query params?

Right now our code handles the port scenario %3A but what about other special characters?

It's unclear to me from the did:web spec if other special characters ought to be percent encoded; the port is specifically referenced in the spec (link in the description) for obvious reasons being that the colon character is in direct conflict with the DID syntax, but unclear to me if other special characters should be percent encoded.
To further confuse the matter, did:web's paths are to be delimited by a colon, but on the other hand the DID Core spec supports paths on the DID URL.
Step one in this ticket is to determine what the ruleset here is, and that may include reaching out to the did:web author's.


Implement Info, cleanup, and post/delete notification V2 API tests

fix #444
Signed-off-by: Ginny Guan<EMAIL_ADDRESS>

The following testcase fails because of https://github.com/edgexfoundry/edgex-go/issues/3534
https://github.com/edgexfoundry/edgex-taf/blob/b57668a398475d4efdc0e62b8ccca8a7042bea02/TAF/testScenarios/functionalTest/V2-API/support-notifications/notification/DELETE.robot#L17-L27


In the local distributed environment, can not create table

What type of bug is this?
Unexpected error
What subsystems are affected?
Datanode
What happened?
In the local distributed environment, the table cannot be created. The error is as follows:
mysql> CREATE TABLE dist_table(
    ->     ts TIMESTAMP DEFAULT current_timestamp(),
    ->     n INT,
    ->     row_id INT,
    ->     TIME INDEX (ts)
    -> )
    -> PARTITION BY RANGE COLUMNS (n) (
    ->     PARTITION r0 VALUES LESS THAN (5),
    ->     PARTITION r1 VALUES LESS THAN (9),
    ->     PARTITION r2 VALUES LESS THAN (MAXVALUE),
    -> )
    -> engine=mito;
ERROR 1815 (HY000): Failed to execute query: CREATE TABLE dist_table(
    ts TIMESTAMP DEFAULT current_timestamp(),
    n INT,
    row_id INT,
    TIME INDEX (ts)
)
PARTITION BY RANGE COLUMNS (n) (
    PARTITION r0 VALUES LESS THAN (5),
    PARTITION r1 VALUES LESS THAN (9),
    PARTITION r2 VALUES LESS THAN (MAXVALUE),
)
engine=mito, source: Failed to execute statement, source: Failure during query execution, source: Failed to request Meta, source: Illegal state from server, code: 1, error: No active datanodes

What operating system did you use?
MacBook Pro M1
Relevant log output and stack trace
No response
How can we reproduce the bug?
I followed the steps of the documentation.
https://docs.greptime.com/getting-started/distributed

But the command to start meta is different.
cargo run -- metasrv start --selector LoadBased

@Fengys123 What's the root cause of this bug?

In the logic of heartbeat reporting, due to some problems in error handling, datanode reports the region-number of heartbeat is -1.
If the region number in the heartbeat sent by datanode to meta is -1, it means that something went wrong. In the current implementation of LoadbasedSelector, it relies on the region number in the heartbeat. If the region number is equal to -1, LoadbasedSelector will exclude the datanode.
This issue has been resolved in #1207. But we may need to add some integration tests later.

In #1167 region_stats was introduced to table trait with the default implementation of returning UnsupportedSnafu. During datanode starting up, [numbers](https://github.com/GreptimeTeam/greptimedb/blob/b2a09c888a9e93a0b7ea6a78e87bb8ca8edaafb3/src/catalog/src/remote/manager.rs#L410) table was registered as NumebrTable which doesn't explicitly implement region_stats.
When heartbeat task tries to iterate over tables to collect region stats, numbers table would fail, stopping the whole collecting process and return -1 as the final result. Then the datanode is excluded by the meta-srv as #Fengys123 explained above.


Add server cache image

This change adds a Dockerfile which can be used to build a front-side cache for Pathling server.

Made redundant by #1414.


[polyserve] 0.27.4 modules with .js extension are not resolved if importing by absolute path

Using --module-resolution "node":
import {SomeService} from '../services/some.service' - works (relative path, no extension)
import {SomeService} from '/src/services/some.service.js' - works (absolute path, with extension)
import {SomeService} from '/src/services/some.service' - 404 (absolute path, no extension)
Expected: .js extension should be auto-resolved when using absolute path module identifier.
polyserve: 0.27.4
node: 9.11.1
os: macos

Might be the same as #9

@web-padawan might be related, as you get 404 depending on path, but I'm not sure.
This issue is about absolute path identifiers.

0.27.6 - the issue is still reproduced.


[Builtins] Add the 'dropList' builtin

This experiment adds the dropList builtin so that folks can play with it. Not intended for merging.

/benchmark validation

/benchmark validation

/benchmark validation

Comparing benchmark results of 'validation' on 'ac8c225aeb' (base) and 'd36d0f87f4' (PR)
Results table
ac8c225 d36d0f8 	Change
TOTAL 	46.71 ms 	42.41 ms 	-9.2%

Oh, come on!


Added IPython debugger

I use this library nearly daily and I just wanted to add this debugger for having debugger command history when I debug.

Thank you for your pull request, it's much appreciated. Here are a few comments:

Use snake-case for function parameters (ipython=True)
IPython shouldn't be a requirement for this module, as implemented rpdb will crash when installed on a fresh installation since it will be missing IPython.
Since you have a lot of repetition in this for no good reason (IRpdb and Rpdb are basically identical), I'd suggest wrapping your import in a try/except and keep everything in a single class.

@tamentis is README the only blocker for this PR? I can help with that ,  it is very useful option

I tried both Python 3.7 and 2.7 and couldn't get this to work. Am I doing anything wrong?
Test script (test.py):
#!/usr/bin/env python

import rpdb; rpdb.set_trace(ipython=True)

Terminal 1:
$ cd rpdb
$ virtualenv venv
$ . ./venv/bin/activate
$ pip install -e .
$ pip install ipdb
$ python test.py
pdb is running on <IP_ADDRESS>:4444

Terminal 2:
$ nc localhost 4444
--Return--
> /private/tmp/test.py(4)<module>()->None
-> import rpdb; rpdb.set_trace(ipython=True)
(Pdb)

It works for me with python 3.6.10
`(pydss) ➜  ~ nc localhost 4444
--Return--
None

/Users/val/testipython.py(3)()
1 #!/usr/bin/env python
2
----> 3 import rpdb; rpdb.set_trace(ipython=True)

ipdb>`
However, auto-completion does not work ;)


Hangs on import MouseTools

Hello,
I found your project while looking for a way to get realtime Disney wait times to display on an LED matrix.  Thought this would be a fun way to bring some Disney magic into my home during COVID.
  After installing and running it hangs on import MouseTools.  I was patient the first few times as instructed and after an hour I noticed that MouseTools.db stays at 0KB so I'm not sure if it's even getting to Disney's site.  I know that's not much to go on but any tips would be greatly appreciated.
Thanks!
Steve

I let it run all night as well and had the same result... even running py -m pydoc MouseTools hangs

My semester just ended so I'll try to take a look over my break.

I should have something working within the next few days.

I should have something working within the next few days.

I put a beta version on pypi. I would recommend uninstall your current version of MouseTools and then installing the beta with "pip install MouseTools==2.1.0b1". This removes the whole database stuff as I wasn't able to find a way to connect again. I assume credentials changed or something. Some functions were also changed and others removed and I haven't been able to fully test. So just reply to this thread with any comments or issues.

I put a beta version on pypi. I would recommend uninstall your current version of MouseTools and then installing the beta with "pip install MouseTools==2.1.0b1". This removes the whole database stuff as I wasn't able to find a way to connect again. I assume credentials changed or something. Some functions were also changed and others removed and I haven't been able to fully test. So just reply to this thread with any comments or issues.

Quickly checked it out on my lunch break and it's perfect!  I wanted to be able to iterate through each WDW park's attractions and see their wait time and this does the trick:
for park_id in MouseTools.ids.WDW_PARK_IDS:     park = MouseTools.Park(park_id)     print(park.get_name())     for attr_id in MouseTools.Park(park_id).get_attraction_wait_times():         attr = MouseTools.Attraction(attr_id)         print(attr.get_name())         print(attr.get_wait_time())
I have to get back to my real job but cannot wait to get this info onto my LED matrix hopefully this weekend.  Thanks so much!!!

Quickly checked it out on my lunch break and it's perfect!  I wanted to be able to iterate through each WDW park's attractions and see their wait time and this does the trick:
for park_id in MouseTools.ids.WDW_PARK_IDS:     park = MouseTools.Park(park_id)     print(park.get_name())     for attr_id in MouseTools.Park(park_id).get_attraction_wait_times():         attr = MouseTools.Attraction(attr_id)         print(attr.get_name())         print(attr.get_wait_time())
I have to get back to my real job but cannot wait to get this info onto my LED matrix hopefully this weekend.  Thanks so much!!!

You could also do something like this:
for park_id in MouseTools.ids.WDW_PARK_IDS:
     park = MouseTools.Park(park_id)
     for id, values in park.get_wait_times_detailed().items():
         print(f"{values['name']}: {values['wait_time']}")

get_wait_times_detailed() returns a dictionary of dictionaries with the following key:value pairs:
{
id: {
name,
status
wait_time
last_updated
entityType
}
}
An example:
{
'80069748': {
'name': 'Country Bear Jamboree',
'status': 'Operating',
'wait_time': None,
'last_updated': datetime.datetime(2021, 1, 11, 16, 2, 7, 6000),
'entityType': 'Attraction'}
}

You could also do something like this:
for park_id in MouseTools.ids.WDW_PARK_IDS:
     park = MouseTools.Park(park_id)
     for id, values in park.get_wait_times_detailed().items():
         print(f"{values['name']}: {values['wait_time']}")

get_wait_times_detailed() returns a dictionary of dictionaries with the following key:value pairs:
{
id: {
name,
status
wait_time
last_updated
entityType
}
}
An example:
{
'80069748': {
'name': 'Country Bear Jamboree',
'status': 'Operating',
'wait_time': None,
'last_updated': datetime.datetime(2021, 1, 11, 16, 2, 7, 6000),
'entityType': 'Attraction'}
}

Even better!
I couldn't wait and got everything transferred to my Pi and LED matrix.  Cannot wait to get this formatted with text that fits, park and ride logos, etc.
Thanks again!

Even better!
I couldn't wait and got everything transferred to my Pi and LED matrix.  Cannot wait to get this formatted with text that fits, park and ride logos, etc.
Thanks again!


dump.py 微信，提示缺少工具gadget-ios.dylib

need gadget to attach; its default location is: /Users/xking/.cache/frida/gadget-ios.dylib
官网上没有找到该动态库，求解答，十分感谢！

Get your gadget from https://github.com/frida/frida/releases/

放到指定位置就OK了,感谢

I am also facing this issue, I have frida 15.2.2 on my PC and 15.2.2 on iOS, I have downloaded the gadget from the latest version which is currently 15.2.2 but doesn't work


Update Composer dependencies (2024-06-18-00-19)

Visual regression test passed!
View the visual regression test report


Servlet init parameters ignored

@WebServlet(name = "myservlet", initParams = [WebInitParam(name = "foo", value = "bar")])
class MyServlet : VaadinServlet()

object Main {
    @JvmStatic
    fun main(args: Array<String>) {
        VaadinBoot()
            .withServlet(MyServlet::class.java as Class<Servlet>)
            .run()
    }
}

The servlet init parameters are ignored; and also the servlet name is not taken into account. The entire WebServlet annotation seems to be completely ignored.

More info here: https://stackoverflow.com/questions/13952661/embedded-jetty-with-annotated-servlet-patterns


UWP: Clicking 'X' to dismiss the sign-in dialog returns the error "Portal item type incorrect"

I noticed on the dev branch that when the app is configured to view a web map that is not public, and when dismissing the sign-in dialog that appears by default, the map fails to load (which is expected), but cites an inaccurate error message.

In this case, an incorrect 'Map failed to load' error is given:

Signing in with valid credentials has no problem launching the expected web map.

This has been fixed:


check and potentially fix prod deployment

So far we have only relied on the latest stage for adoption tests. This is almost ready for prime time now.

so far, so good! Versioned prod docker on dockerHub.


[WFCORE-6214] Fixed Flaky Test HelpSupportTestCase.testStandalone

What is the purpose of this PR

This PR fixes the error resulting from the flaky test org.jboss.as.cli.impl.aesh.HelpSupportTestCase
The mentioned test may fail or pass without changes made to the source code when it is run in different JVMs due to HashSet's non-deterministic iteration order.

Why the test fails

HelpSupportTestCase calls SynopsisGenerator, which uses a HashSet as its data structure for the conflicts object. As per Java 11 documentation, the ordering of HashSet is not constant. So, the test may fail as the given order can be different from the expected order.

Reproduce the test failure

Run the test with NonDex maven plugin. The command to recreate the flaky test failure is
mvn -pl cli edu.illinois:nondex-maven-plugin:2.1.1:nondex -    Dtest=org.jboss.as.cli.impl.aesh.HelpSupportTestCase#testStandalone
Fixing the flaky test now may prevent flaky test failures in future Java versions.

Expected result

The tests should run successfully when run with NonDex.
Expected: <...server-groups] | [--[replace] | [--server-groups]] )>

Actual Result

We get the failure:
[ERROR] testStandalone(org.jboss.as.cli.impl.aesh.HelpSupportTestCase)  Time elapsed: 0.254 s  <<< FAILURE!
org.junit.ComparisonFailure: org.jboss.as.cli.impl.aesh.Commands$Standalone$Command10. EXPECTED [command1 [<argument>] ( [--all-server-groups] | [--replace] | [--server-groups] )]. FOUND [command1 [<argument>] ( [--all-server-groups] | [--server-groups] | [--replace] )] expected: <...server-groups] | [--[replace] | [--server-groups]] )> but was: <...server-groups] | [--[server-groups] | [--replace]] )>

Fix

Changed the object type of conflicts and conflicts2 from HashSet to LinkedHashSet in method SynopsisGenerator.addSynopsisOption(SynopsisOption) and imported LinkedHashSet.

Jira Issue:
https://issues.redhat.com/browse/WFCORE-6214
This PR is similar to some merged PRs in other Wildfly projects:
https://github.com/wildfly/wildfly/pull/13728
https://github.com/wildfly/wildfly/pull/11833
https://github.com/wildfly/jboss-ejb-client/pull/534

Hello, RishabhPrabhu5. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

Hi @RishabhPrabhu5 , thank you for contributing to WildFly!
Could you add the Jira number to the commit message? Like what you did with the title, having the Jira number in the commit message is a requirement for us. It helps us to track the history with the Jira issue.

@RishabhPrabhu5 thank-you for your fix. I added a comment.
Running your plugin I found another one in : https://github.com/wildfly/wildfly-core/blob/main/cli/src/test/java/org/jboss/as/cli/impl/BootScriptInvokerTestCase.java#L217
Should be a LinkedHashSet. Would you be ok to update the JIRA description and also fix this one? The JIRA would be the findings running the plugin against CLI unit tests.
Thank-you!

Thank you for the feedback, I have made the changes requested and have updated the JIRA description. Please let me know if there is anything more I can do.
Thank you!

/ok-to-test

Windows - Bootable JAR - JDK 11 is unrelated to this PR.

@RishabhPrabhu5 , merged!, thanks for contributing to WildFly, and especially for the great PR description, that helps in doing the review.

Thank you for your email and accepting my PR. I hope my fix was useful, and
I look forward to contribute more in the future. Thanks!
On Wed, Feb 15, 2023 at 5:06 AM Yeray Borges @.***>
wrote:

@RishabhPrabhu5 https://github.com/RishabhPrabhu5 , merged!, thanks for
contributing to WildFly, and especially for the great PR description, that
helps in doing the review.
—
Reply to this email directly, view it on GitHub
https://github.com/wildfly/wildfly-core/pull/5366#issuecomment-1431059020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYYIO436OBSDNIMQQNH5QDTWXSTCVANCNFSM6AAAAAAUKVD7EY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Next 10 support

What kind of change does this PR introduce?
Feature
What is the current behaviour?
Supports Next v.9.
What is the new behaviour?
Drop support for Next v.9
Support Next v.10
Does this PR introduce a breaking change?
Yes. Users should update their project to consume Next v.10
Other information:
Extend getServerSideProps context with resolvedUrl field.
Please check if the PR fulfills these requirements:

[X] Tests for the changes have been added
[X] Docs have been added / updated

Coverage remained the same at 100.0% when pulling 12ebc1378419b8a3feb927f694d186f8fbfa9305 on next10 into d9af2dabb7a659990c7bb4ad1886180acf206dd4 on master.


IGN_LAUNCH_CONFIG_PATH could accept path lists

Desired behavior
ign launch should try to prepend all colon-separated paths from IGN_LAUNCH_CONFIG_PATH when searching for a launch file to launch. Currently, this variable can contain only a single path.
Alternatives considered
All alternatives for combining multiple launch packages are bad.
Implementation suggestion
Shouldn't be difficult, but before submitting a PR, I wanted to know whether this feature would be desired/welcome.

Sounds like a good idea to me :+1:

Fixed by #93.


[Harvesting - Scheduling] Implement a configurable dynamic scheduler

I have already implemented a way to have dynamic scheduling with celery beat and django. The implementation is present in geonode/geonode#7317
It is using a similar pattern as the one described in this article:
https://medium.com/swlh/dynamic-task-scheduling-with-django-celery-beat-f2591d52e15
A brief overview:

We add django-celery-beat (an official celery project) as an additional dependency to GeoNode. It is able to store celery beat schedule in the django database and exposes models to update this schedule in a dynamic way. Each scheduled task is represented by a PeriodicTask model and it has properties such as the update frequency, whether it is enabled or not, which celery task shall be executed, etc.

Then I have a Harvester model, which is connected to PeriodicTask via a 1-1 relationship - this means that every harvester has an associated periodic task, and is therefore able to be scheduled by celery beat in a dynamic way. Each harvester keeps track of its own properties

This seemed to work really well when I did the initial development. However, when I started adding more harvesters I realized that the scheduling does not seem to be working correctly. I seem to be facing an issue similar to what is described here:
https://github.com/celery/django-celery-beat/issues/330
Basically, I am seeing that only two periodic tasks seem to be able to run at the same time. This seems like a strange bug. I plan to debug it in order to see if it can be made to work reliably. As such, it is currently unclear to me how much time/effort this will still require. I will know better when I have the time to do some debugging.

@meomancer
can you please debug this to see if it is still relevant?
There have been new releases of django-celery-beat, so maybe this has been solved in the meanwhile?
For debugging you will need to:

Make sure you run a celery beat scheduler by launching celery with celery --app geonode.celery_app beat -l DEBUG  as the scheduler uses celery beat
Add some three or four harvester instances to the django DB,
Set their scheduling_enabled attribute to True
Set their update_frequency attribute to a value suitable for testing - this is interpreted as minutes
Monitor the celery beat logs in order to check whether all harvesters are being scheduled. When I originally implemented this code I found that some of the harvesters were triggered but others were not

You can try to update the django-celery-beat version mentioned in the geonode requirements.txt and setup.cfg to the latest version and check whether that fixes the issue

Hi @ricardogsilva
It is still just running the first 2 task from harvesters (using latest celery and celerybeat)
But i'm still not be able to debug it and still can't find the problem

can we close this @ricardogsilva?

@giohappy , yes a dynamic scheduler has been implemented and merged upstream. I'll go ahead and close this


Add better repr to Color class

print(Color(*(.3,.5,.6)))

Returns: Color: (0.30000001192092896, 0.5, 0.6000000238418579, 1.0)
Color(*(.3,.5,.6))

Returns: <build123d.geometry.Color at 0x12f6e8eb3d0>

After 6eb00f1ef94fa4d5eb2b94c7ff6c39c0839b80f7:
>>> str(Color(*(.3,.5,.6)))
'Color: (0.30000001192092896, 0.5, 0.6000000238418579, 1.0) ~ LIGHTBLUE3'
>>> Color(*(.3,.5,.6))
Color(0.30000001192092896, 0.5, 0.6000000238418579, 1.0)

Wow that was fast, thanks!

The string now returns the closest named color which is a good sanity check that the color is correct.


Flash Drive not visible (different output from diskutil)

Etcher version: 1.0.0
Operating system and architecture: macOS 10.12.5
Image flashed: Zorin OS 12.1 Ultimate x64 (Ubuntu-based Linux distro)
Do you see any meaningful error information in the DevTools? No
Other (possibly) relevant details: I'm using a Dell USB-A to USB-C adapter to flash it from my 2016 Macbook Pro.

This relates to #649 and #828.
I have a SanDisk 32GB flash drive than I'm using to flash, and like the previous two, Etcher didn't see the drive. However, my output from diskutil was different from the other two; instead of seeing:
   Device Location:          External
   Removable Media:          No

I see:
   Device Location:          External
   Removable Media:          Fixed

It sounds to me from this link that the fate of Removable: Fixed devices is the same as Removable: No - is this correct, or can you work with Fixed devices and just not Removable: No?
I got it flashed by using unsafe mode (a godsend, never want to use another flashing program at this point) - just curious about the rest..
Full diskutil output if you want..
diskutil list
...
  /dev/disk2 (external, physical):
     #:                       TYPE NAME                    SIZE       IDENTIFIER
     0:     FDisk_partition_scheme                        *31.4 GB    disk2

diskutil info /dev/disk2
   Device Identifier:        disk2
   Device Node:              /dev/disk2
   Whole:                    Yes
   Part of Whole:            disk2
   Device / Media Name:      Extreme

   Volume Name:              Not applicable (no file system)
   Mounted:                  Not applicable (no file system)
   File System:              None

   Content (IOContent):      FDisk_partition_scheme
   OS Can Be Installed:      No
   Media Type:               Generic
   Protocol:                 USB
   SMART Status:             Not Supported

   Disk Size:                31.4 GB<PHONE_NUMBER>2 Bytes) (exactly 61282631 512-Byte-Units)
   Device Block Size:        512 Bytes

   Read-Only Media:          No
   Read-Only Volume:         Not applicable (no file system)

   Device Location:          External
   Removable Media:          Fixed

   Virtual:                  No
   OS 9 Drivers:             No
   Low Level Format:         Not supported

Hi @clounie,
Thanks for the detailed report. You are right about the fixed property. It basically means "removable = No".
We investigated in the past whether we should only rely on the device location being external, but that includes external hard drives as well.
I guess there's not Etcher can do (for safety reasons) if the adaptor reports to the MacBook Pro that the drive is fixed.
Maybe worth investigating if macOS could somehow be tweaked to detect this adaptor as something removable?


Reset Sensor value?

Is it possible to reset (or set) a sensor value?
I would like to null the gyro at program start and maybe several times in the program.

It depends on the sensor. For example, the LEGO EV3 Gyro sensor can be reset by momentarily changing to a different mode and then changing back to the desired mode. Most of the other gyro sensors out there don't have a reset function. You just have to maintain an offset variable in your program and use that to "zero" the sensor value.

Ah, ok thanks. I was talking about the LEGO Sensor.I thought i could avoid keeping track of the sensor value at certain positions.


⚠ Bofrost.at has degraded performance

In a0f6dc6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1381 ms

Resolved: Bofrost.at performance has improved in 1d96411.


Run autosens in ns-loop

Currently we run autosens in its own loop in cron.  This results in autosens occasionally running at the same time that ns-loop is refreshing carbhistory.json, which causes autosens to generate different results (since it can't properly exclude meal periods).
This PR updates oref-ns-loop.sh to run autosens after uploading to NS, which ensures that it happens after the carbhistory refresh.  It also checks that glucose data is fresh before and after running autosens, to ensure that a slow autosens run doesn't prevent us from getting new BG data for long.

I'm 👎  on integrating the autosens loop in the ns-loop. It's not separating concerns, but combining them. The same rationale as https://github.com/openaps/oref0/pull/759#issuecomment-340147866 applies.
The autosens should also work for offline usage, and therefore it's should not be tied/integrated into ns-loop.
The ns-loop cgm part is essential for those using CGM from Nightscout. The loop getting the CGM data from Nightscout should be as small as possible.  The risk of "a slow autosens run doesn't prevent us from getting new BG data for long." should be made impossible from a safety perspective. Getting fresh BG data is much more important than running autosens.

Locking is the classic solution to this problem.

Ok, if you prefer https://github.com/openaps/oref0/pull/767 I'll close this one.


Why Only lowercase ascii characters, underscores, and digits are allowed ?

Why isn't it allowed to use Upper case letters in the name of steps. I have disabled it and its running fine on my local machine. Am I missing anything?

We lint the user code to respect the PEP-8 guidelines.


Update License

Hi,
Can you please fill in the license field in package.json?
https://docs.npmjs.com/files/package.json#license

Will do thx.

Just a quick update, testing out a new release on the TypeScript branch. This will be fixed there.

Fixed: https://github.com/drudru/ansi_up/commit/c6bb6f04c98020224a42ba84fb3bae2a2c225974


GoogleMaps/GoogleMaps.h file not found

Is this a bug report?
Yes

Have you read the Installation Instructions?
Yes
Environment

Steps to Reproduce

Manually install react native maps using Xcode and then execute npm run ios
Expected Behavior

Build should be passed. I changed the react native maps version to 0.17.1 and it worked.
Actual Behavior

iOS build failed and the error is GoogleMaps.h file doesn't exist

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Duplicate #2159

Upgrading to latest react-native-maps(0.23.0) resolved my problem
npm i<EMAIL_ADDRESS>react-native link


Where can I get the SYNLA_Plus_1024.npy file?

I am trying to retrain the model, however, I am not able to get the SYNLA_Plus_1024.npy file. I tried downloading the original images from the Synla dataset, but I don't know exactly how to convert the images into the .npy file, so as to successfully create a dataset from that file.
Can anyone please guide me as to How do I create the input dataset .npy file?
Thanks in advance!

The. npy files are found in the downloads, they are zipped as a .tar archive. https://github.com/bloc97/SYNLA-Plus/releases/tag/v1.0
However note that the SYNLA dataset is used to benchmarking, testing and prototyping models, as it fits in memory and allows faster training and accurate testing. A real dataset of anime + art images is the used to train the final model.
Using SYNLA only might yield poor results on large and overparameterized models.
Real datasets have a lot of variance and the same network trained with the same dataset might give different results if trained a second time just because the real dataset might have been shuffled differently. Using SYNLA just prevents training variance caused by random dataset drift.

Oops, I forgot that the .tar archives contained png files and I didn't include the script to generate the .npy files, sorry for the confusion.
You can use this python script to generate the .npy datasets. You need python 3.x, PIL and numpy. You can modify dataset_folder and np_file if needed. Also be careful, you will need at least triple the size of the dataset as memory to account for decompression, concatenation and saving. There might be more memory-efficient methods out there but they might be more complex...
from PIL import Image
import numpy as np
import os

dataset_folder = "/hdd/sda/SYNLA_Plus_1024/"
np_file = "/hdd/sda/SYNLA_Plus_1024.npy"
np.save(np_file, np.array([np.array(Image.open(dataset_folder + d)) for d in os.listdir(dataset_folder)]))

Ohh I see! Thank you very much for your help :)


UCR Doesn't Seem To Be Able To Send Anything To vJoy

I have been tried for a couple of days now, installing and reinstalling, including troubleshooting based on a myriad of suggestions I found online.
I have used Joystick Gremlin(v13.3) and UCR (v0.9.0 Alpha1+8) to work as a feeder to vJoy(<IP_ADDRESS>), but still I can't make it to read. Albeit vJoy seems to be able to read information sent by the "vJoy Feeder (demo)". UCR works perfectly with ViGem. But I would like the advanced features(axis) of vJoy for other games (like Tachyon)
I know I am missing something or doing something wrong.
The controllers I am trying to make vJoy to identify are the Thrustmaster Warthog and Saitek Rudder Pedals. ( I have 4 MFDs and 2 Saitek Throttle Quadrants, but I am not using them for vJoy and neither are being recognized either)
I've attached screenshots of the setup. The problem is present even if it is 32 or 128 buttons. (using vjoy 1 or vjoy 2).

Output device here is vJoy stick 2

This seems to indicate that you only have vJoy Device #1 enabled, and no vJoy device #2

Although you say "using vjoy 1 or vjoy 2", so maybe that's not the problem.
One thing I would recommend trying is disabling the combining for the TM device - maybe they use vJoy to do the combining and it's interfering with UCR controlling the vJoy device.
Also ensure that the vJoy demo feeder is CLOSED when trying to use UCR to drive vJoy - only one program at a time can control a vJoy device.

Thanks for the reply. I tried your suggestions but still no go. With, or without Thrustmaster Combined powered on. vJoy demo feeder is off.
Here are the screenshots of the test. Including the list of devices that UCR displays.

Did you not say that you could not get Joystick Gremlin to send to vJoy
either?
Sounds like there is something fundamentally broken on your system?
Maybe try uninstalling and reinstalling vJoy?
Just because the demo feeder works, does not mean that other apps will
definitely work, because the demo feeder is basically in the same folder
structure as the vJoy DLLs and so will always be able to find them, whereas
eg apps like UCR may be looking in the registry to find the location of the
vJoy DLL and may not be able to find it or load it properly
On Fri, 13 Aug 2021 at 20:54, RealDrJester @.***> wrote:

Thanks for the reply. I tried your suggestions but still no go. With, or
without Thrustmaster Combined powered on. vJoy demo feeder is off.
Here are the screenshots of the test. Including the list of devices that
UCR displays.
[image: image]
https://user-images.githubusercontent.com/3292341/129411734-b360585c-4f2d-4cf8-9d58-53acbc9f9a3c.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Snoothy/UCR/issues/246#issuecomment-898687618, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANDZBFCUKZJPC6VLTB7Y73T4VZ67ANCNFSM5CCHC23A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Did you not say that you could not get Joystick Gremlin to send to vJoy either? Sounds like there is something fundamentally broken on your system? Maybe try uninstalling and reinstalling vJoy? Just because the demo feeder works, does not mean that other apps will definitely work, because the demo feeder is basically in the same folder structure as the vJoy DLLs and so will always be able to find them, whereas eg apps like UCR may be looking in the registry to find the location of the vJoy DLL and may not be able to find it or load it properly
…
On Fri, 13 Aug 2021 at 20:54, RealDrJester @.***> wrote: Thanks for the reply. I tried your suggestions but still no go. With, or without Thrustmaster Combined powered on. vJoy demo feeder is off. Here are the screenshots of the test. Including the list of devices that UCR displays. [image: image] https://user-images.githubusercontent.com/3292341/129411734-b360585c-4f2d-4cf8-9d58-53acbc9f9a3c.png — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#246 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AANDZBFCUKZJPC6VLTB7Y73T4VZ67ANCNFSM5CCHC23A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

I've tried uninstalling and reinstalling as well. It doesn't work either. I've also tried an early version too, and still didn't work. <IP_ADDRESS> and the <IP_ADDRESS>.
It is definitely quite puzzling! What address does UCR search for? So I can check on my registry for it.

I had the same issue.
I noticed that the vJoy dlls coming with the UCR installer are a bit older: vJoyInterface.dll is v<IP_ADDRESS> and vJoyInterfaceWrap.dll is v<IP_ADDRESS> so I replaced them with the latest ones provided by vJoy (taken from Program Files\vJoy\x86\) and now it works.


[BUG] Rail visualiser doesn't work if the lines are tabbed in

Describe the bug
The world map previewer doesn't strip leading tabs/spaces
To Reproduce
Steps to reproduce the behavior:

Tab in a line in the railway file
open the world map previewer
See error

Expected behavior
The program should strip the tabs before reading the rail lines, and yet it does not
many of these rails are going to provinces with ids that are actually used in the file to represent length - province 9 is the very popular one off of south america
Screenshots

Desktop (please complete the following information):

OS: idk it wasn't me who took the screenshot
VSCode version: idk
Extension version: idk

Fixed in https://github.com/herbix/hoi4modutilities/commit/31efcd6d928a08e032c30dca32e76be1e087664b.


Add Eli Shalnev pledge

Individual pledge by Eli Shalnev aka 0mnius

Added in https://github.com/opentffoundation/manifesto/pull/266


registry override change  queue-proxy image  not update  config-deployment  configmap after finished install

Describe the bug
Add  queue-proxy registry override after knative serving finished, but  operator not update queueSidecarImage in config-deployment  configmap.
Expected behavior
A clear and concise description of what you expected to happen.
operator change  queueSidecarImage in config-deployment  configmap.
To Reproduce
Steps to reproduce the behavior.
knative-serving.yaml
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeServing
metadata:
  name: knative-serving
  namespace: knative
spec:
  registry:
    override:
       activator: xxx.com/k8s/knative-activator:v1.1.0
       autoscaler: xxx.com/k8s/knative-autoscaler:v1.1.0
       autoscaler-hpa: xxx.com/k8s/knative-autoscaler-hpa:v1.1.0
       controller: xxx.com/k8s/knative-controller:v1.1.0
       domain-mapping: xxxx.com/k8s/knative-domain-mapping:v1.1.0
       domainmapping-webhook: xxxx.com/k8s/knative-domain-mapping-webhook:v1.1.0
       net-istio-controller/controller: xxx.com/k8s/knative-net-istio-controller:v1.1.0
       net-istio-webhook/webhook: xxxx.com/k8s/knative-net-istio-webhook:v1.1.0
       migrate: xxx.com/k8s/knative-apiextensions-storageversion-migrate:v1.1.0
       webhook: xxx.com/k8s/knative-serving-webhook:v1.1.0

apply knative-serving.yaml, wait operator finished install knative serving components.
add queue-proxy registry override

    queue-proxy: xxx.com/k8s/knative-queue:v1.1.0

kubectl get configmaps -n knative config-deployment -o yaml

Knative release version
The Knative version, where you find this issue.
v1.1.0
Additional context
Add any other context about the problem here such as proposed priority

To change queueSidecarImage or queue-sidecar-image in current version, need configure spec.config.deployment  in KnativeServing operator
config:
    deployment:
      queue-sidecar-image: xxxx.com/k8s/knative-queue:v1.1.0
      queueSidecarImage: xxx.com/k8s/knative-queue:v1.1.0

@wu0407 I think this is the expected behavior by the way.
spec.registry.override can override the images for all the containers in the deployment resource.
If you want to change any configurations in ConfigMap, you have to configure as you described here: https://github.com/knative/operator/issues/918#issuecomment-1010675784.


Added PHP 7.1

While PHP 7.1 is relatively new.  It is the latest stable version, and many popular frameworks and libraries are dependent on this version.  Example: Doctrine DBAL's latest version is 7.1+
Adding PHP 7.1 into the Azure Run Stack would greatly enhance the dev experience for modern PHP developers who wants to quickly get started with Azure Web App on Linux.

This will also partially address Issue #1

Merge it, pls

closed in favour of 7.2, 7.3

Why is 7.1 excluded from the supported versions?

we already have 7.2 and 7.3, is there a reason you need 7.1 in particular?

Simply for completion.
Azure App Services is a platform.  we won't know why a customer needs a particular version of a language..
It might be for a legacy library version (an older version of Doctrine or Laravel, where PHP 7.2 is incompatible with certain versions). Or they might have even more legacy software that they want to incrementally upgrade to 5.6 to 7.0 to 7.1

We'll add it if there is customer demand, Otherwise we don't want to support something that will soon be end of life.


🚀[Feature] Support GitHub Installation tokens

GitHub Apps
We use GitHub Apps to automate actions on the GitHub platform. By creating a JWT, we authenticate to GitHub and request an installation token that functions the same as a PAT for that specific organization that installed the GitHub App.
When we use it for authentication we run into the following errors:
connect-gitHubAccount -AccessToken -Silent -Owner WeAreInSpark
! Enter your personal access token: ****************************************
Write-Warning: A parameter cannot be found that matches parameter name 'ForegroundColor'.
WARNING: Unexpected access token format: ghs_*
Invoke-GitHubAPI:
Invoke-GitHubAPI: ----------------------------------
Invoke-GitHubAPI:
Invoke-GitHubAPI: Request:
Invoke-GitHubAPI:
Invoke-GitHubAPI: StatusCodeVariable      : APICallStatusCode
Invoke-GitHubAPI: Method                  : 1
Invoke-GitHubAPI: ContentType             : application/vnd.github+json; charset=utf-8
Invoke-GitHubAPI: Authentication          : Bearer
Invoke-GitHubAPI: FollowRelLink           : True
Invoke-GitHubAPI: HttpVersion             : @{Major=2; Minor=0; Build=-1; Revision=-1; MajorRevision=-1; MinorRevision=-1}
Invoke-GitHubAPI: ResponseHeadersVariable : APICallResponseHeaders
Invoke-GitHubAPI: Token                   : @{Length=40}
Invoke-GitHubAPI: Uri                     : https://api.github.com/user
Invoke-GitHubAPI: Headers                 : @{Accept=application/vnd.github+json; charset=utf-8; X-GitHub-Api-Version=2022-11-28}
Invoke-GitHubAPI:
Invoke-GitHubAPI:
Invoke-GitHubAPI: ----------------------------------
Invoke-GitHubAPI:
Invoke-GitHubAPI: Message:
Invoke-GitHubAPI: Response status code does not indicate success: 403 (Forbidden).
Invoke-GitHubAPI:
Invoke-GitHubAPI: ----------------------------------
Invoke-GitHubAPI:
Invoke-GitHubAPI: Response:
Invoke-GitHubAPI:
Invoke-GitHubAPI: Version             : @{Major=2; Minor=0; Build=-1; Revision=-1; MajorRevision=-1; MinorRevision=-1}
Invoke-GitHubAPI: Content             : @{Headers=System.Object[]}
Invoke-GitHubAPI: StatusCode          : 403
Invoke-GitHubAPI: ReasonPhrase        : Forbidden
Invoke-GitHubAPI: Headers             : {@{Key=Server; Value=System.Object[]}, @{Key=Date; Value=System.Object[]}, @{Key=x-github-media-type;
Value=System.Object[]}, @{Key=x-github-api-version-selected; Value=System.Object[]}…}
Invoke-GitHubAPI: TrailingHeaders     : {}
Invoke-GitHubAPI: RequestMessage      : @{Version=; VersionPolicy=0; Content=; Method=; RequestUri=https://api.github.com/user; Headers=System.Object[];
Properties=System.Object[]; Options=System.Object[]}
Invoke-GitHubAPI: IsSuccessStatusCode : False
Invoke-GitHubAPI:
Invoke-GitHubAPI:
Invoke-GitHubAPI: ----------------------------------
Invoke-GitHubAPI:
Invoke-GitHubAPI:
Exception: Response status code does not indicate success: 403 (Forbidden).

It's a shame because it works like a breeze.

@Manbearpiet, Again thank you for the feedback.
I have not gotten into testing this scenario just yet. Mostly been using it as a CLI replacement as its neat with piping commands into each other. Your use case does make sense how ever. Its on the list for testing. Think there is some spaghetti code on this too that needs tidying up as it was implemented somewhat in a rush to get the auth and secret store working, similarly as it would for GitHubCLI.
Cannot promise a timeline for this though. But let me get on top of GitHub Apps first then verify so I can verify the scenario. I have plans to make a repo and org configurator that takes a setting file and process it with this module, likely using Azure Function Apps or so.

@Manbearpiet, Again thank you for the feedback.
I have not gotten into testing this scenario just yet. Mostly been using it as a CLI replacement as its neat with piping commands into each other. Your use case does make sense how ever. Its on the list for testing. Think there is some spaghetti code on this too that needs tidying up as it was implemented somewhat in a rush to get the auth and secret store working, similarly as it would for GitHubCLI.
Cannot promise a timeline for this though. But let me get on top of GitHub Apps first then verify so I can verify the scenario. I have plans to make a repo and org configurator that takes a setting file and process it with this module, likely using Azure Function Apps or so.

Hey Marius, I can talk you through it? I also have a blog post coming up about the GitHub Apps part.
Let me know if I can help 😄

Also taking a look at this while I'm checking on improvements. The issue here seems to be that the -AccessToken parameters sets the command on a path to use an interactive session for PAT tokens. Would it be typical to run this interactively. Thinking that we could default to use the environment variables for this, like $env:GH_TOKEN or $env:GITHUB_TOKEN. When doing so, it will not run into the last issue at hand which is the /user API, which i guess is the main issue.
What would be the preferred way to trigger this logon flow? Parameters or environment var? In either case, do we need another name for it or can we use the existing env var?

Fixed #91

@Manbearpiet : Also taking a look at this while I'm checking on improvements. The issue here seems to be that the -AccessToken parameters sets the command on a path to use an interactive session for PAT tokens. Would it be typical to run this interactively when using an installation token/JWT? Thinking that you could use the environment variables for this, like $env:GH_TOKEN or $env:GITHUB_TOKEN. When doing so, it will not run into the last issue at hand which is the /user API, which i guess is the main issue.
What would be the preferred way to trigger this logon flow? Parameters or environment var? In either case, do we need another name for it or can we use the existing env var?

You got it right in your PR. So will close