Update utils.py to fix issue with find_markers()

Current find_markers() command triggers multi-dimensional indexing by numpy which was depreciated in numpy 1.23.
Current fix flattens expr_frac to avoid this

fixes #49


Bash and ZSH closes unexpectedly

A brief description
Whenever a command is issued that does not produce anything the terminal simply closes itself. This is especially painful when trying to tab-complete a typo, since that will result in a crash.

Expected results
Nothing should happen and the terminal should still be open.

Actual results (with terminal output if applicable)
The terminal closes itself.

Your Windows build number
10.0.14393 Build 14393

Steps / All commands required to reproduce the error from a brand new installation

Start bash or zsh in any way e.g. start cmd.exe and type bash.exe.
Press the down arrow or try to tab-complete something meaningless like asdjkfhjjasdf.
Bash closes itself.

Are you using anything like oh-my-zsh? Do you have a custom .zshrc?

I am. However the problem appears both when using regular zsh, oh-my-zsh and bash.

did you edit your .dot files with a windows editor?


ISPN-6457 Access protected interal caches over HotRod only with authz…

… or over loopback
https://issues.jboss.org/browse/ISPN-6457

Needs rebasing

-1 if I'm understanding this correctly:
The Javascript client needs to have the ability to remotely store scripts and currently I'm using that by putting those scripts directly in the script cache via a cache.put. It seems like your change will disable this?
If so, how is the Javascript client supposed to load scripts remotely?

The change doesn't affect behaviour when connecting over a loopback interface (e.g. localhost). So if your tests go over <IP_ADDRESS> or ::1 that will be fine. In all other cases, authorization must be enabled. You will then need either an authenticated client or go through the console/CLI.

The change doesn't affect behaviour when connecting over a loopback interface (e.g. localhost). So if your tests go over <IP_ADDRESS> or ::1 that will be fine. In all other cases, authorization must be enabled. You will then need either an authenticated client or go through the console/CLI.

Authentication has not yet been implemented in JS client so console/CLI would be the only alternative.
I had added a method to the JS client precisely to make it easier to add a script remotely (see here). Should this remain? I guess so. For now it'll only work when calling from localhost as you said, and eventually it should work from anywhere once authentication is in place.
Btw, the documentation says:

14.5.1. Installing scripts
Scripts are stored in a special script cache, named '___script_cache'. Adding a script is therefore as simple as put+ting it into the cache itself. If the name of the script contains a filename extension, e.g. +myscript.js, then that extension determines the engine that will be used to execute it. Alternatively the script engine can be selected using script metadata (see below).

Not only does not mention how to install scripts via console/CLI, but this whole section needs to be updated and I see no such changes in this PR.

Documentation to come later, integrating...

Actually, testing first

Testing looks good, integrating...

The change doesn't affect behaviour when connecting over a loopback interface (e.g. localhost). So if your tests go over <IP_ADDRESS> or ::1 that will be fine.

Doesn't this still leave a security hole? (Asking for a friend!)

@alanfx I've consulted with Chess on this and he's ok with it.

👍 I'll take his word for it, then!


Static files don't get proper cache control headers when running on nginx.

Responses generated by Utopia::Static when running on nginx don't get a Cache-Control header at all. Strangely responses generated by Utopia::Content don't have this problem. And it also works fine when running on puma with RACK_ENV=:development.

Hmm, that is odd.

public/ is normally served by web server, not application.


Add GitHub action and Makefile target for Windows installer

The new GitHub action "installer-for-windows" builds an NSIS installer with Tesseract for Windows.
It is not run automatically, but must be triggered manually.
Autoconf builds also have a new target winsetup, so the installer can be built with make winsetup.
This requires a build environment with all needed components (NSIS, Windows libraries, ...).

Example: https://github.com/stweil/tesseract/actions/runs/11620260190.

Here is the first result: https://github.com/tesseract-ocr/tesseract/actions/runs/11641358702.


Update LAB_04_MFAConditionalAccessandAADIdentityProtection.md

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review


remove redundant sentence

@MikeBazMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MikeBazMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This fix has been merged and should be live shortly. Thanks!


Bump openai version

A bug was introduced in the openai client which affects all CF installs. We had pinned openai under 1.47 to avoid a previous bug, but that's resolved so we can update the pin here. (Not sure why that pin didn't protect from this bug tbh, maybe the version was getting bumped elsewhere silently?)
The bug manifests as: Client.__init__() got an unexpected keyword argument 'proxies'
https://github.com/openai/openai-python/issues/1903

Is this supposed to be fixed? I'm still seeing the 'proxies' issue when trying to update a container at AWS. I've updated to OpenAI 1.55.3, and my code does not use Httpx at all. I'm using the Async client.
client = AsyncOpenAI(
File "/home/app/venv/lib/python3.9/site-packages/openai/_client.py", line 337, in __init__
super().__init__(
File "/home/app/venv/lib/python3.9/site-packages/openai/_base_client.py", line 1428, in __init__
self._client = http_client or AsyncHttpxClientWrapper(
File "/home/app/venv/lib/python3.9/site-packages/openai/_base_client.py", line 1325, in __init__
super().__init__(**kwargs)
TypeError: __init__() got an unexpected keyword argument 'proxies'


Declare Reuse code/section

Refer to the course website for how we can declare Reused code/sections in documentation (if applicable)


How do I change the frequency to 433Mhz for use with Omnipod?

I'm using Omnipod in the US.  On the Loop websites there's a screenshot of the settings page and it shows 433MHz - how do I do make that change?
Out of interest, my (home-built) RileyLink is working just fine with the Omnipod with it set at 916MHz so maybe I don't need to bother...?


3ds Max slow loading

Does anyone know why alembic files load considerably slower when there are other assets in the 3ds Max scene before loading as versus with an empty scene?
(3ds Max 2016)

They stream in live from the disk as you advance frames.
Thus if they are not quickly accessible they will be slow to access.
They are slower than point caches as well, even in the best case scenario.
We were not able to figure out how point caches in 3DS Max were as fast as
they were using standard SDK functions.
Best regards,
Ben Houston [Work:<PHONE_NUMBER>, Cell:<PHONE_NUMBER>, Skype: ben.exocortex]
Book a meeting: https://calendly.com/benhouston
http://exocortex.com | https://clara.io | http://threekit.com
On Tue, Sep 26, 2017 at 9:32 PM, Chad Travis<EMAIL_ADDRESS>wrote:

Does anyone know why alembic files load considerably slower when there are
other assets in the 3ds Max scene before loading as versus with an empty
scene?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Exocortex/ExocortexCrate/issues/68, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAj6_an3NZgikEvlJ6ZORTGJN6D97pAWks5smaWlgaJpZM4PlGJU
.


This is a clear commit message that explains what changes have been m…

…ade.
What you have changed
Why you changed it
Links to any relevant supporting material e.g. agile story
How to review
Who can review

Looks spiffing.


MSI GS43VR

Hey,
So i've been on corpnewts discord chat and compeltely followed this guide 3 times
https://www.reddit.com/r/hackintosh/comments/4sil5p/audio_mechanic_old_patchfix_removal_applealc/
but i still can't seem to get my audio working, i have a MSI gs43VR Laptop, it has ALC898 according to the detection software.
My HDEF after patched from HDAS is located at 1F rather than 1B though is that an issue?
I've tried every layout and can get you my config.plist if required.
If i use VoodooHDA it works fine except internal microphone but i would rather use native where possible.
Any advice you can give would be extremely helpful :)
Thanks.

Further more the layout is

<01 00 00 00> - this is the preferred layout - and supports a wide range of codecs

I forgot to mention the sound devices show up on almost all layouts, they just don't have any sound come out of them :(

i've just seen you are likely to just close this ticket? who can i speak to request further support with this? the guy with the gs63vr is likely to have the same layout i would imagine, can i provider any further help to get this layout added? i've tried to understand it might self but i'm a windows guy and  kext editing confuses me.

All questions about installation and usage ask in AppleALC threads
In English Insanelymac - http://www.insanelymac.com/forum/topic/311293-applealc-—-dynamic-applehda-patching/page-1
In Russian Applelife - https://applelife.ru/threads/applealc-dinamicheskij-patching-applehda.1171672/
Provide DarwinDumper and IOReg reports in one of this threads, describe your problem and community will help you.

i've added a reply to the thread with attached things http://www.insanelymac.com/forum/topic/311293-applealc-—-dynamic-applehda-patching/page-72#entry2411345


Why are keywords not highlighting if they have no whitespace after them (and another question)?

In my Lisp-Lexer-based text box, keywords don't get highlighted unless they're followed by some form of whitespace, such as a space or newline.
Additionally, I'm finding that negative numbers aren't being highlighted as numbers.
Can I fix either of these issues with some C# code?
Here's the entirety of my control's code -
https://github.com/bryanedds/Nu/blob/master/Nu/Nu/Nu.Gaia.Design/SymbolicTextBox.cs
I don't think I've done anything special that would warrant either of these two issues.
Thanks!

I saw that you created a workaround.. which I think is necessary in this case. The lexers that come built-in to Scintilla[NET] are mostly "all or nothing" functionality. If the behavior of the lexer isn't what you expected there is little you can do to influence the behavior.

Hmm, okay. Not much we can do then!

Not unless you want to submit code fixes for the native Scintilla component. :)


Missing default value in ArgGroup.

I understand that by default  defaultValue in ArgGroup are not applied.
The behavior is explained at : https://github.com/remkop/picocli/issues/876#issuecomment-558866624
(Note: The default behavior seems surprising to me at first sight but maybe it's justified)
I faced a situation where it seems that this behavior is not respected (but maybe I missed something)
Sorry for the not so short example, I was not able to make it smaller.
The goal is to have this kind of options : X AND ((1A AND 1B) OU (2A OU 2B))
I set default value for X, 2A and 2B.
I share the code :
@Command(name = "myCommand",
         mixinStandardHelpOptions = true,
         description = "A command with default value in section ?")
public class Example implements Runnable {

    @ArgGroup(exclusive = false,
              heading = "%n@|italic " //
                      + "Options to be used with group 1 OR group 2 options." //
                      + "|@%n")
    public OptXAndGroupOneOrGroupTwo optXAndGroupOneOrGroupTwo = new OptXAndGroupOneOrGroupTwo();

    public static class OptXAndGroupOneOrGroupTwo {
        @Option(names = { "-x", "--option-x" }, required = true, defaultValue = "Default X", description = "option X")
        private String x;

        @ArgGroup(exclusive = true)
        private OneOrTwo oneORtwo = new OneOrTwo();
    }

    public static class OneOrTwo {

        @ArgGroup(exclusive = false,
                  heading = "%n@|bold Group 1|@ %n%n"//
                          + "@|italic " //
                          + "Description of the group 1 ." //
                          + "|@%n")
        public GroupOne one = new GroupOne();

        @ArgGroup(exclusive = false,
                  heading = "%n@|bold Group 2|@ %n%n"//
                          + "@|italic " //
                          + "Description of the group 2 ." //
                          + "|@%n")
        public GroupTwo two = new GroupTwo();
    }

    public static class GroupOne {
        @Option(names = { "-1a", "--option-1a" },required=true,description = "option A of group 1")
        private String _1a;

        @Option(names = { "-1b", "--option-1b" },required=true,description = "option B of group 1")
        private String _1b;
    }

    public static class GroupTwo {

        @Option(names = { "-2a", "--option-2a" },required=true, defaultValue = "Default 2A", description = "option A of group 2")
        private String _2a;

        @Option(names = { "-2b", "--option-2b" },required=true, defaultValue = "Default 2B", description = "option B of group 2")
        private String _2b;
    }

    public void run() {
        System.out.println();
        System.out.println(" X = " + optXAndGroupOneOrGroupTwo.x);
        System.out.println("1A = " + optXAndGroupOneOrGroupTwo.oneORtwo.one._1a);
        System.out.println("1B = " + optXAndGroupOneOrGroupTwo.oneORtwo.one._1b);
        System.out.println("2A = " + optXAndGroupOneOrGroupTwo.oneORtwo.two._2a);
        System.out.println("2B = " + optXAndGroupOneOrGroupTwo.oneORtwo.two._2b);
    }

    public static void main(String... args) {
        Example example = new Example();
        int exitCode = new CommandLine(example).execute(args);

        System.exit(exitCode);
    }
}

When I run my command, it outputs value of each options.
E.g if I run it without any options :
 X = Default X
1A = null
1B = null
2A = Default 2A
2B = Default 2B

Now if I run it just with option -x : myCommand -x ANOTHER_VALUE, I get :
 X = ANOTHER_VALUE
1A = null
1B = null
2A = null
2B = null

Default value of 2A and 2B are lost I don't know if this is expected but this is a surprising behavior.

Not really linked to this issue but as 1A, 1B, 2A and 2B are required, I expected that when X is used (1A AND 1B) or (2A or 2B) will be required.

I would like to see the output.

[picocli DEBUG] Creating CommandSpec for test.picocli.examples.defaultvalue.Example@77c1870e with factory picocli.CommandLine$DefaultFactory
[picocli DEBUG] Creating CommandSpec for picocli.CommandLine$AutoHelpMixin@7371b5d5 with factory picocli.CommandLine$DefaultFactory
[picocli INFO] Picocli version: 4.6.1, JVM: 1.8.0_212 (Oracle Corporation OpenJDK 64-Bit Server VM 25.212-b03), OS: Linux 5.10.0-0.bpo.3-amd64 amd64
[picocli INFO] Parsing 2 command line args [-x, ANOTHER_VALUE]
[picocli DEBUG] Parser configuration: optionsCaseInsensitive=false, subcommandsCaseInsensitive=false, abbreviatedOptionsAllowed=false, abbreviatedSubcommandsAllowed=false, aritySatisfiedByAttachedOptionParam=false, atFileCommentChar=#, caseInsensitiveEnumValuesAllowed=false, collectErrors=false, endOfOptionsDelimiter=--, expandAtFiles=true, limitSplit=false, overwrittenOptionsAllowed=false, posixClusteredShortOptionsAllowed=true, separator=null, splitQuotedStrings=false, stopAtPositional=false, stopAtUnmatched=false, toggleBooleanFlags=false, trimQuotes=false, unmatchedArgumentsAllowed=false, unmatchedOptionsAllowedAsOptionParameters=true, unmatchedOptionsArePositionalParams=false, useSimplifiedAtFiles=false
[picocli DEBUG] (ANSI is enabled by default: systemproperty[picocli.ansi]=null, isatty=true, TERM=xterm-256color, OSTYPE=null, isWindows=false, JansiConsoleInstalled=false, ANSICON=null, ConEmuANSI=null, NO_COLOR=null, CLICOLOR=null, CLICOLOR_FORCE=null)
[picocli DEBUG] Initializing command 'myCommand' (user object: test.picocli.examples.defaultvalue.Example@77c1870e): 7 options, 0 positional parameters, 0 required, 1 groups, 0 subcommands.
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.helpRequested of type boolean to false.
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.versionRequested of type boolean to false.
[picocli DEBUG] [0] Processing argument '-x'. Remainder=[ANOTHER_VALUE]
[picocli DEBUG] '-x' cannot be separated into <option>=<option-parameter>
[picocli DEBUG] Found option named '-x': field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x, arity=1
[picocli INFO] Adding match to GroupMatchContainer [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]={} (group=1 [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]).
[picocli DEBUG] Creating new user object of type class test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo for group [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]
[picocli DEBUG] Created test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo@4c9458f6, invoking setter FieldBinding(test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo test.picocli.examples.defaultvalue.Example.optXAndGroupOneOrGroupTwo) with scope test.picocli.examples.defaultvalue.Example@77c1870e
[picocli DEBUG] Initializing --option-x=<x> in group [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]: setting scope to user object test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo@4c9458f6 and initializing initial and default values
[picocli DEBUG] Set initial value for field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x of type class java.lang.String to null.
[picocli DEBUG] Applying defaultValue (Default X) to field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x on OptXAndGroupOneOrGroupTwo@4c9458f6
[picocli DEBUG] 'Default X' doesn't resemble an option: 0 matching prefix chars out of 14 option names
[picocli INFO] Setting field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x to 'Default X' (was 'null') for field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x on OptXAndGroupOneOrGroupTwo@4c9458f6
[picocli DEBUG] Setting scope for subgroup [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]] with setter=FieldBinding(test.picocli.examples.defaultvalue.Example$OneOrTwo test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.oneORtwo) in group [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]] to user object test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo@4c9458f6
[picocli DEBUG] Initialization complete for group [[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]
[picocli DEBUG] 'ANOTHER_VALUE' doesn't resemble an option: 0 matching prefix chars out of 14 option names
[picocli INFO] Setting field String test.picocli.examples.defaultvalue.Example$OptXAndGroupOneOrGroupTwo.x to 'ANOTHER_VALUE' (was 'Default X') for option -x on OptXAndGroupOneOrGroupTwo@4c9458f6
[picocli DEBUG] Applying default values for command 'myCommand'
[picocli DEBUG] defaultValue not defined for field boolean picocli.CommandLine$AutoHelpMixin.helpRequested
[picocli DEBUG] defaultValue not defined for field boolean picocli.CommandLine$AutoHelpMixin.versionRequested
[picocli DEBUG] Applying default values for group '[[-x=<x>] [[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]]'
[picocli DEBUG] Applying default values for group '[[-1a=<_1a> -1b=<_1b>] | [[-2a=<_2a>] [-2b=<_2b>]]]'
[picocli DEBUG] Applying default values for group '[-1a=<_1a> -1b=<_1b>]'
[picocli DEBUG] defaultValue not defined for field String test.picocli.examples.defaultvalue.Example$GroupOne._1a
[picocli DEBUG] defaultValue not defined for field String test.picocli.examples.defaultvalue.Example$GroupOne._1b
[picocli DEBUG] Applying default values for group '[[-2a=<_2a>] [-2b=<_2b>]]'
[picocli DEBUG] Applying defaultValue (Default 2A) to field String test.picocli.examples.defaultvalue.Example$GroupTwo._2a on GroupTwo@77553f0b
[picocli DEBUG] 'Default 2A' doesn't resemble an option: 0 matching prefix chars out of 14 option names
[picocli INFO] Setting field String test.picocli.examples.defaultvalue.Example$GroupTwo._2a to 'Default 2A' (was 'null') for field String test.picocli.examples.defaultvalue.Example$GroupTwo._2a on GroupTwo@77553f0b
[picocli DEBUG] Applying defaultValue (Default 2B) to field String test.picocli.examples.defaultvalue.Example$GroupTwo._2b on GroupTwo@77553f0b
[picocli DEBUG] 'Default 2B' doesn't resemble an option: 0 matching prefix chars out of 14 option names
[picocli INFO] Setting field String test.picocli.examples.defaultvalue.Example$GroupTwo._2b to 'Default 2B' (was 'null') for field String test.picocli.examples.defaultvalue.Example$GroupTwo._2b on GroupTwo@77553f0b

 X = ANOTHER_VALUE
1A = null
1B = null
2A = null
2B = null

@remkop I have been working on this issue.  If I resolve it I will open a PR.

@MadFoal that would be great, thank you!

Update 1- looks like the default values do get properly stored in argSpec inside the applyGroupDefaults function correctly.  There is something about the way it gets merged back into this that is causing an issue.

Updated 2-
"Picocli will not initialize the @ArgGroup-annotated field (and so no default values are applied) if none of the group options is specified on the command line." (https://picocli.info/#_default_values_in_argument_groups)
If you do not specify at least one value within the argument group, it will not instantiate the default values, which is the observed behavior for this case.  Technically the program is working as advertised.
@remkop I will keep digging to investigate how to change this behavior unless you want me to stop.

@MadFoal I still think there is a problem, based on the debug output:
Applying defaultValue (Default 2A) to field String test.picocli.examples.defaultvalue.Example$GroupTwo._2a on GroupTwo@77553f0b
...

After parsing is done, picocli is trying to apply default values to all unmatched options in the command, where possible. Since the -2a and -2b options are in a group  (GroupTwo) whose user object already has an instance (see below), it should be possible to apply default values.
@ArgGroup(exclusive = false, ...)
public GroupTwo two = new GroupTwo();

However, in the debugger I can see that the default values are applied to a different instance of the GroupTwo class.
What I believe is happening is the following: initially, the OptionSpec for the -2a and -2b options are created with a reflective reference to the annotated field, and a reference to the original GroupTwo instance. When the -x option is matched, a new instance of OptXAndGroupOneOrGroupTwo, OneOrTwo and GroupTwo is created, but the OptionSpec for the -2a and -2b options within those subgroups still have references to the old GroupTwo instance. The GroupTwo reference is updated when the -2a and -2b options are matched on the command line (that is why I did not notice this issue before). But for default values, the reference is not updated, so the default values are applied to the original (now outdated) GroupTwo instance.
I need to dig a bit deeper into what would be the best way to fix this, but I do think the current behaviour is incorrect.

This is proving trickier than I thought...
When the -x option is matched, a new OptXAndGroupOneOrGroupTwo instance is created. Subgroups are updated to point to the new OptXAndGroupOneOrGroupTwo instance, but sub-subgroups are not updated.
When an options in a subgroup is matched, a new user object (e.g. a GroupTwo instance) for that subgroup is created, and this new object replaces the old reference; hence things work fine for arguments matched on the command line.
For default values there is no match, so this above logic is not invoked.
I have not yet found a way to fix this.

One idea was (when the new OptXAndGroupOneOrGroupTwo instance is created) to descend into the subgroups and update their object pointers but this is not trivial; while the group/subgroup hierarchy has not changed, the object hierarchy would need to be rediscovered via reflection. This would not work for programmatic models that do not contain annotated fields/methods.
Another idea was to invoke the same logic for applying default values as for when an option was matched; this does not work because it appears as if the option was actually matched, giving validation errors (for exclusive options etc.)

So while I can see that from @sbernard31's point of view the current defaulting behaviour is unsatisfactory, I am not sure this can be fixed.

@remkop  Okay thank you for that explanation.  I thought it was along those lines, however I am significantly less familiar with the code.  I noticed early that we were adding the default values but I was struggling with why they are not making it into the correct place.  This makes more sense.
I had been looking at how to change "applyDefaultValues" to look for the subgroup created by the -x option and add default values if the current value is null.

@MadFoal I am not sure there is a true fix.
One thing to note is that picocli can only show the default value of options in arg groups when the default value is specified in the annotation (it cannot derive the default value from the field's initial value like it can when showing help for options that are not in an arg group).
The documentation for default values in arg groups states the situation reasonably well.
Applications should probably:

put any default values in the option annotation so that picocli can display them in the usage help message
groups with subgroups should probably not instantiate the group class in the declaration (so, use @ArgGroup MyGroup g; instead of @ArgGroup MyGroup g = new MyGroup(); ). This allows applications to determine whether an option in the group was matched on the command line: the @ArgGroup-annotated field has a non-null value when a group option was matched.
Assign default values manually: if the group was not matched (the annotated field is null), then assign default values in the application logic. @MadFoal's suggestion to put default values in the declaration of the annotated option fields may be the most concise way to do this; the application then only needs to do something like if (g == null) { g = new MyGroup(); } to assign default values to all options in the group. For groups with subgroups, applications may need to do the same for the nested subgroups. For example, g = new MyGroup(); g.sub = new Subgroup();. And yes, that means some duplication, because default values would need to be specified both in the annotations as well as in the option field initial value.

@remkop
Yes Sir, that is correct.

@MadFoal Will you be able to provide a pull request to improve the documentation?

@remkop  I can do that.

Documentation is coming along.  Should have something tangible to submit in 24-48 hours.

Great, thank you for working on this!
Some nitpicking (I am very very picky when it comes to the documentation! 😅 ):

please spell the library as picocli (all lower case)
example code should be as minimalistic/short as possible, only leave what is absolutely necessary:

remove all public and private modifiers (except for the main method)
ultrashort meaningful names (ArgGroupDefaultValueTest -> MyApp,  _a -> a, OptionXAndGroup -> Outer, TwoArgGroup -> Inner)
don't qualify class names: ArgGroupDefaultValue.TwoArgGroup twoArgGroup = new ArgGroupDefaultValue.TwoArgGroup(); -> Inner inner;
do we need required = true in this example?
can we reduce the @Option annotation to one line? We can remove the long option name and the description, and make the default value shorter, like "AAA".

My recommendation to app authors using groups would be to not instantiate groups in their declaration:
// don't do this
class Outer {
    @ArgGroup Inner inner = new Inner();
}

but instead to just declare the field, so that it remains null when none of the options in the group were matched on the command line:
// do this instead
class Outer {
    @ArgGroup Inner inner; // this field will only be non-null if an option in this group was matched
}

The options in the groups can be initialized in their declaration, but should also have a default value in their annotation:
class Inner {
    @Option(names = "-a", defaultValue = "AAA") String a = "AAA";
    @Option(names = "-b", defaultValue = "BBB") String b = "BBB";
}

It may be a good idea to add a run method to the example that shows what the application logic needs to do.
// MyApp
public void run() {
    if (outer == null) { // no options in the group were specified on the command line; apply defaults
        outer = new Outer();
    }
    if (outer.inner == null) { // same for nested sub-groups
        outer.inner = new Inner(); // apply the default values of the inner group
    }
    // remaining business logic...
}

I'll make these changes and re-submit.

PR#1463 with the changes you recommended.  Let me know if there are more changes that I need to make.


Fix plotting non-unique categorical colors

IMPORTANT: Please search among the Pull requests before creating one.
Description

Fix plotting non-unique categorical colors in :func:squidpy.pl.spatial_scatter.
How has this been tested?

1 new test
Closes

N/A

Codecov Report

Merging #561 (439c971) into main (456c49a) will decrease coverage by 0.03%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main     #561      +/-   ##
==========================================
- Coverage   77.99%   77.96%   -0.04%
==========================================
  Files          37       37
  Lines        5171     5173       +2
  Branches      979      975       -4
==========================================
  Hits         4033     4033
- Misses        858      859       +1
- Partials      280      281       +1

Impacted Files
Coverage Δ

squidpy/pl/_spatial_utils.py
78.47% <50.00%> (-0.35%)
:arrow_down:

squidpy/pl/_color_utils.py
63.88% <66.66%> (ø)

interactive=["PyQt5>=5.15.0", "napari>=0.4.14,<0.4.16"],

yeah this also bugged me in the napari plugin. LGTM!


add automatic version string based on git-describe (tags)

Closes #194

I added a few more here https://github.com/zcash/lightwalletd/pull/232

In case you're wondering (as I was) ... This change causes make to run git -- what happens if that command fails for some reason? For example, git is not in the $PATH or fails?. I moved .git/ to a different name, and then:
$ make
GO111MODULE=on go build -ldflags="-X github.com/zcash/lightwalletd/common.Version=`git describe --tags`"
fatal: not a git repository (or any of the parent directories): .git
$ echo $?
0
$

So the make does succeed (exit code 0), although it prints that error. If git isn't in the $PATH, no error prints, but make succeeds. Either way, the version string is empty:
$ ./lightwalletd version
lightwalletd version
$

I think that's okay. This is what you see when it succeeds (example):
$ ./lightwalletd version
lightwalletd version v0.3.0-19-gcc80502
$

I should have explained above, the string is the latest tag, currently v0.3.0, the number of commits since that tag, 19, and the letter g followed by the short version of the latest commit hash, cc80502.

@benzcash, I'll close this PR and we can just go with your PR, #232. For the future, just a suggestion, it might be simpler for you to create your own branch (based on mine) to suggest further improvements, but then not create your own PR; you can just post the github URL of your branch here, and I can pull your new commit(s) over to my branch, to then be merged to master. That way we wouldn't have two PRs.


chore(actions): use newer method to set an output parameter

Motivation
The set-output command is deprecated and will be disabled soon.
For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Fixes #5736
Specifications

https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter

Solution
Apply the method used in the specifications.
Review
Anyone can review this
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

@teor2345 I just fixed the replacements I made by mistake.
About the remaining errors, I created a ticket for that:

https://github.com/ZcashFoundation/zebra/issues/6048


Dnd vue playground

As requested in #36
This version still has bugs when the droppable element leaves the parent container

Codecov Report

Merging #118 into master will decrease coverage by 0.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
- Coverage   60.43%   59.43%   -1.00%
==========================================
  Files          17       17
  Lines         417      424       +7
  Branches       68       71       +3
==========================================
  Hits          252      252
- Misses        135      142       +7
  Partials       30       30

Impacted Files
Coverage Δ

packages/dnd/src/Draggable/Draggable.js
53.96% <0.00%> (-0.88%)
:arrow_down:

packages/dnd/src/Droppable/Droppable.js
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

would you mind adding the new changes to the main readme like we've done before with draggable:
https://github.com/jalal246/dflex/tree/master/packages/draggable#examples

Will do!
Thanks for reviewing

Thank you!

@Tepexic Thank you for your contribution. You've expanded the usage of DFlex to Vue users which is awesome. And I hope you consider more contributions by opening new issues or applying PRs.

Thank you! I'll check the issues to see if I can tackle another one!


renv & suggested projectFolder name

E:/Covid19VaccineAesiIncidenceCharacterization
When using renv to get this package and setting this as the RENV_PATHS_CACHE = "E:/Covid19VaccineAesiIncidenceCharacterization" the name is too long for a windows machine and files to download the package at the renv::init() step.
When I changed it to something shorter it worked.
Thank you to @azzashoaibi to letting me know this.

@azzashoaibi: would you be able to provide the exact error message you received? It would be good to know why this limit exists, and what the maximum allowed length is.

Error installing package 'Covid19VaccineAesiIncidenceCharacterization':

installing to library 'D:/Projects/Covid19VaccineAesiIncidenceCharacterization/renv/library/R-4.0/x86_64-w64-mingw32/.renv/1'
installing source package 'Covid19VaccineAesiIncidenceCharacterization' ...
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
** help
*** installing help indices
converting help for package 'Covid19VaccineAesiIncidenceCharacterization'
finding HTML links ... done
Covid19VaccineAesiIncidenceCharacterization-package
html
Error: cannot open the connection
removing 'D:/Projects/Covid19VaccineAesiIncidenceCharacterization/renv/library/R-4.0/x86_64-w64-mingw32/.renv/1/Covid19VaccineAesiIncidenceCharacterization'
Error: install of package 'Covid19VaccineAesiIncidenceCharacterization' failed [error code 1]


GetOptionsFromProjectFile returns duplicates of project references on mono

This is essentially benign as they seem to be ignored, but should still be fixed.

ugh, thanks for detecting all these!!!

No longer the case following #478


deps/openssl: remove, require it as a prerequisite.

Also refactor (i.e. remove) RetDec's Crypto library.

lets run TC tests


BaseDir option not respected

Neither setting it in the config nor passing in as cli flag works.

Thank you, we'll merge this shortly along with a bunch of other things we've done in the dev branch.

Fixed in d2eccc98adfeb23868f21d677963e7a2d3e095ae

https://github.com/vattle/sqlboiler/releases/tag/v2.1.0


Unify code style using Prettier and updates ESLint rules

This PR includes:

unified package.json file, so it's almost exactly the same as raven-node one (missing author field)
prettier setup with husky, so it runs for every file that's about to be commited
updated grunt-eslint, which allows us to use eslint-disable-next-line clause (prettier moves eslint-disable-line clauses to the line below, which breaks it's functionality)
modifies .eslintrc so it allows for extra parens around logical conditions (which is done by prettier)

Notes:

no manual code changes has been made in this PR
setup and config has been done in a separate commit, and automated changes has been made in separate ones
all tests are passing

Related raven-node PR: https://github.com/getsentry/raven-node/pull/364

But other than that, nice job 👍

Agree. We should also consider removing ESLint rules that are touching anything related to code style, as they might clash with Prettier. For example by using https://github.com/prettier/eslint-config-prettier

@HazAT all updated. Same changes are reflected in https://github.com/getsentry/raven-node/pull/364

😱


task status is confusing and not working properly

We have some race conditions aside not very clear definition in the status field that make insecure to use it to check if a task is done (despite its resources are not yet being disposed).
This is specially a bad issue for CML

related to #357

A task is unequivocally “done” when status.succeeded is equal to the parallelism value. At least, when parallelism equals 1.
Can you add more information with regard to race conditions?

Still not sure whether this issue is going to be solved with code, documentation or both, but it's definitely worth the effort.

Off–topic: see https://airflow.apache.org/docs/apache-airflow/stable/concepts/tasks.html#task-instances for additional statuses.

I like apache airflow statuses, is what we defined originally.
Look at this three examples of task being refreshed after task completion
    "status":{"running":0} -> should be "status":{"running":1} ?
    "status":{"running":0,"succeeded":2} -> succeeded 2?

    "status":{"running":1}
    "status":{"running":1,"succeeded":1}
    "status":{"running":0,"succeeded":2} -> why again 2?

    "status":{"running":1}
    "status":{"running":1,"succeeded":1}

I have even seen "status":{"running":1,"succeeded":6}
Its succeeded related to the time the task is refreshed?

Look at this three examples of task being refreshed after task completion [...] I have even seen "status":{"running":1,"succeeded":6}

That's what I wanted to know, thank you very much! 🙏🏼 If this happened with parallelism set to 1, it's a bug. Can you please provide a minimal example to reproduce this condition?

Its succeeded related to the time the task is refreshed?

No, it shouldn't. 🙃

Running is attached to the disposal of the instance and its misleading. To me running should be reflecting the state of the task as a bash script and not tied to the hardware.

In that case, what should happen amidst two spot instances; waiting / suspended?
Currently, running means that there is a cloud instance running, and, implicitly, that an script has been launched on it. If the script fails or succeeds, the status will transition accordingly to failed or succeeded.

Also succeeded looks like if the task itself had succeeded

Yes, it means exactly that. If the current behavior is different, it's a bug.

As you say and what I wanted to say is from trial 1 to 2 both tasks were finished with valid results but the output is difficult to assimilate.

Can you please elaborate on the meaning of “trial 1 to 2”?

That's what I wanted to know, thank you very much! 🙏🏼 If this happened with parallelism set to 1, it's a bug. Can you please provide a minimal example to reproduce this condition?

Sure! no parallelism being used. This is the basic example that allowed me to reach that condition.
resource "iterative_task" "tpi-examples-basic4" {
    provider = iterative

    cloud     = "az"
    region    = "us-east"
    machine   = "Standard_D2S_v3"

    workdir {
        input = "./"
    }

    script = <<-END
    #!/bin/bash
    echo 'hello tpi' > data.txt
    printenv >> data.txt
    END
}

Yes, it means exactly that. If the current behavior is different, it's a bug.

But in the context of the script or just the internal execution of the task? If my script fails the task succeeded will still be true right?

In that case, what should happen amidst two spot instances; waiting / suspended?

We should somehow give feedback if the task is waiting for a new provisioning.

the status will transition accordingly to failed or succeeded.

This is weird, specially if we use console its more friendly to do the check against one value
if [ $(terraform console <<< 'iterative_task.train.status') == 'completed' ]; then

I might say that the majority of the cases what I want to know is that the task is completed. Maybe then if failed the script to be able to create proper checks.
Thats why I liked the status codes.
We might define 2X for 2 being terminated and 20 or 21 being terminated and success or terminated and failed.
Status code best benefit is the capability of expansion, which is definitely good to avoid being short and changing the definition in the near future.
A status code can always come with the description

If my script fails the task succeeded will still be true right?

Wrong. If the script fails, failed will be incremented, but succeeded won't.

I might say that the majority of the cases what I want to know is that the task is completed. Maybe then if failed the script to be able to create proper checks.
Thats why I liked the status codes.
We might define 2X for 2 being terminated and 20 or 21 being terminated and success or terminated and failed.
Status code best benefit is the capability of expansion, which is definitely good to avoid being short and changing the definition in the near future.
A status code can always come with the description

Status codes don't seem precisely user friendly. If detecting whether a task has completed (regardlessly of the succeeded or failed outcome) is so important, maybe we can add a third field with the sum of those two. 🤔

We should somehow give feedback if the task is waiting for a new provisioning.

With methods other than events it would be a bit hacky, if with “provisioning” you mean “spot instance”. Still, sounds interesting.

With methods other than events it would be a bit hacky, if with “provisioning” you mean “spot instance”. Still, sounds interesting.

This is a must. We need to know if the task is sleeping or running and we can already know it at least when terminating the spot instance

I do not find events useful right now, does not say a lot
In any case. Look at this scenario:
Task is running installing tensorflow like a boss
Mar 01 12:26:59 tpi000000 tpi-task[2070]: Processing triggers for man-db (2.9.1-1) ...
Mar 01 12:27:19 tpi000000 tpi-task[2070]: Processing triggers for libc-bin (2.31-0ubuntu9.2) ...
Mar 01 12:27:24 tpi000000 sudo[2069]: pam_unix(sudo:session): session closed for user root
Mar 01 12:27:25 tpi000000 tpi-task[7576]: Collecting tensorflow
Mar 01 12:27:26 tpi000000 tpi-task[7576]:   Downloading tensorflow-2.8.0-cp38-cp38-manylinux2010_x86_64.whl (497.6 MB)

however
"status": {
    "running": 0
  },

Not only is not reflecting the ongoing script, also says that we are not running (no spot here)

tpi000000 tpi-task[2070]

what is the goal of this?
Should not be the task name or something helpful to identify the task?

This is a must.

Yes, but “must” be duly tracked in a separate issue.

_We need to know if the task is sleeping or running

If I'm not mistaken, parallelism minus running is the sleeping value you want.

we can already know it at least when terminating the spot instance

That would be useful if we kept track of terminations though a central orchestrator or a leader, able to assign machines to “threads” for parallelism values greater than one.
Random self–quote from an internal conversation...
Termination notices
Spot instances can poll a special internal URL to know if they're going to be terminated soon:

Amazon Web Services — http://<IP_ADDRESS>/latest/meta-data/spot/termination-time
Google Cloud — http://metadata.google.internal/computeMetadata/v1/instance/preempted
Microsoft Azure — http://<IP_ADDRESS>/metadata/scheduledevents?api-version=2020-07-01

Termination notices are available up to ~2 minutes in advance, depending on the cloud... and the weather.
Shutdown signal
Spot instances transition to the G2 state before terminating, so init has a few seconds to send a termination signal to all the processes and wait for them to finish. We use this approach to detect termination on both virtual machines (aws, az, gcp) and containers (k8s) because it's much more idiomatic than polling termination notices.
The task code has been designed to copy machine files (e.g. checkpoints) as soon as they are created or updated. Thus, it doesn't need to synchronize any data after receiving a termination notice. In the worst case scenario, the last checkpoint will be lost if there isn't enough time to transfer it.
Spot instance chaining
In the beginning, our code listened to those termination signals and, if the task was still running (i.e. not finished), it requested a new spot instance to resume it. This approach was rather flaky and had a fair amount of downsides.
Finally, we chose to use orchestrators to manage our instances automatically. Clouds don't charge for virtual machine orchestrators, and they do exactly what we need: launch new spot instances to resume the task up to completion.

Amazon Web Services — Auto Scaling Groups
Microsoft Azure — Virtual Machine Scale Sets
Google Cloud — Managed Instance Groups
Kubernetes — Jobs

I do not find events useful right now, does not say a lot

Orchestrator events come directly from the cloud API, and there isn't much we can do about that, apart from logging our own “events” from inside the cloud instances.

Task is running installing tensorflow like a boss

Mar 01 12:26:59 tpi000000 tpi-task[2070]: Processing triggers for man-db (2.9.1-1) ...
Mar 01 12:27:19 tpi000000 tpi-task[2070]: Processing triggers for libc-bin (2.31-0ubuntu9.2) ...
Mar 01 12:27:24 tpi000000 sudo[2069]: pam_unix(sudo:session): session closed for user root
Mar 01 12:27:25 tpi000000 tpi-task[7576]: Collecting tensorflow
Mar 01 12:27:26 tpi000000 tpi-task[7576]:   Downloading tensorflow-2.8.0-cp38-cp38-manylinux2010_x86_64.whl (497.6 MB)

however

"status": {
    "running": 0
  },

Not only is not reflecting the ongoing script, also says that we are not running (no spot here)

Again, that “running” value comes from the cloud API and should be accurate. Are you sure that there was a machine actually running? If so, how an I reproduce that condition?

tpi000000 tpi-task[2070]
what does this mean?
Should not be the task name or something helpful to identify the task?
tpi000000
maybe parallelism 0?

This prefix is added automatically by systemd to all the logs, and tpi000000 is the host name. Azure sets the host name automatically to a prefixed hexatrigesimal value.
Mar 01 12:26:59 tpi000000 tpi-task[2070]: Processing triggers for man-db (2.9.1-1) ...
^^^^^^ ^^^^^^^^ ^^^^^^^^^ ^^^^^^^^ ^^^^   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
date   time     host      unit     pid    message

Blatantly off-topic note: see also #378 and journalctl --all --output=json for evil ideas.

Are we looking for something like starting, running, succeeded, failed?

"status": {
    "running": 1,
    "succeeded": 1
  },

is going to show running forever

So there are two different issues:

what keys we have (e.g. do we need to add starting and failed?)
the values are wrong (e.g. says 1 when it should be 0)

So there are two different issues

Yes, an issue and a feature request (?)

the values are wrong (e.g. says 1 when it should be 0)

Fixing with https://github.com/iterative/terraform-provider-iterative/pull/420

what keys we have (e.g. do we need to add starting and failed?)

We already have running, succeeded and failed. Missing would be starting and paused (waiting for spot instance), but it's more of an arithmetic issue than a technical issue.

Is queued more appropriate than paused? I think it's fine to have a few keys that are mathematically related.

Is queued more appropriate than paused?

queued would probably replace both paused and starting 👍🏼

About relation to parallelism, I think it's fine to have a few keys that are mathematically related.

👍🏼 Sounds good! Then, queued = parallelism - running

queued would probably replace both paused and starting 👍🏼

sounds good

@DavidGOrtega, care to open followup issues for running=0 from #420 and, perhaps, the “street math” queued status?

https://github.com/iterative/terraform-provider-iterative/pull/521 fixed running = 0 when the script exited with non–zero status; not while running; that's something I can't reproduce.

#471 adds all the “street math” queued status logic.


Add option to disable unchatter for individual keys

I have added possibility to disable unchatter for individual keys. Its useful for control keys (like you dont want to block CTRL or SHIFT) and for keyboard with volume sliders. Maybe there are more uses.
In the UI, the change is represented by checkbox column.

I don't use this anymore but I will take a look to the PR during the weekend Thank you!

Thank you. Feel free to ask any questions or propose improvements. Even if you do not use this, I still find it highly useful program. :)


Add support for node 18

Hi there, thanks for reporting.
I'm building them currently. Usually, I do that on the weekend.

added, please let me know how it works for you


Add a --all-contexts flag to list all the installations

- What I did
Using --all-contexts will list all your contexts, then list all the installations per context.
Adding a new column "CONTEXT", only showed if this flag is enabled.
- How to verify it
Run the unit tests, or test it manually:
$ docker app ls --all-contexts
INSTALLATION        APPLICATION                 LAST ACTION RESULT  CREATED MODIFIED REFERENCE               CONTEXT
app1                hello-world (0.1.0)         install     success 3 weeks 3 weeks  myuser/example:v1.0.0
   kube
cnab-with-status    cnab-with-status (0.1.0)    install     success 3 weeks 3 weeks                          default
cnab-without-status cnab-without-status (0.1.0) install     success 3 weeks 3 weeks                          default
app1                hello-world (0.1.0)         install     success 3 weeks 3 weeks  myuser/example:v1.0.0
  default

- Description for the changelog

Add --all-contexts flag to the docker app ls command

- A picture of a cute animal (not mandatory but encouraged)

Rebased, PTAL @chris-crone

PTAL @chris-crone

This works for the case where we keep the application state locally but won't scale if the state is stored remotely. I'm going to close this for now and we can relook at it later.


[Bug] Stacked bars do not work on time x axis when data is not fully synchronized

Version
5.5.1
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=bar-stack&lang=ts&code=PYBwLglsB2AEC8sDeAoWszGAG0iAXMmuhgE4QDmFApqYQOQCGAHhAM73EC-ANMcwEFWbQqhIYAniGoNIAW2qd0vYhKHtRxdGCkzY9AG6NsAV0Xc-6NrQjURsANpai47boYAjRqXqXXbMEYAYwBrBgAzLF9ndAATRkDCBwdoagB3WAARBOoACnoAJgAGAoAWAFoigEZKqoAVavwioqaigC16AEoeWAKAXT7nFXExVx1pT29o11gA4LD9SOBp13jExxiSFPSsnPzistrahpbm1o7u2Cq-vxntjOywPMKSiurjxrPmi57r29d7rsnvtXkdqg0Cq1zl1foMZnCSFxiIMuABuIA
Steps to Reproduce

View chart

Current Behavior
Rendering of series breaks (see first bar in the linked example) and bars are not stacked but instead overlapping.
Expected Behavior
Series should stack where timestamps overlap. When timestamps do not overlap the stack should only consider data that exists at the given timestamp.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
I have seen this comment in a similar issue. That comment advises to preprocess the data so that null values are written where timestamps do not overlap. However, if there is a lot of data it would be nice to avoid this preprocessing. In addition, it feels reasonable to assume that stacking will only occur where timestamps are the same even without preprocessing the data.
The documentation mentions that time and category axis are not supported, but I am assuming this applies to the y axis since the apache echarts stacked bar examples use category axis and it works fine. This would imply to me that time x axis is supported for stacking.
Am I missing something here? Is the preprocessing the intended way to use bar stacking? I feel that the functionality I described where preprocessing isn't needed sounds reasonable.
I opened a new issue because this seems slightly different than the linked one. To me, the linked one is saying stacking bars on a time axis does not work at all. Here, it seems that stacking bars on a time axis can sometimes work but sometimes breaks. And according to the documentation, it does not say that it is unsupported.

I agree. Sure I can do the pre-processing, but this is just confusing.
No one reasonably expects this data to output this graph.
Some may say it's not a bug, but it will look like a bug to any fresh eyes. And be reported as such, and lower trust in the lib. Plus it makes the lib less convenient, for very basic use-cases.

This is the natural output for the given data:


Add compatibility with RPGstats

https://www.curseforge.com/minecraft/mc-mods/rpgstats
RPGstats does not recognize blocks as being broken when broken with vanilla hammers.

Pulling from https://github.com/P03W/RPGStats/issues/70 , the only reason I can think of this happening is that the fapi block break event isnt being called (properly?), as thats the event I use for granting XP


[Feature]: Z1 | Z1 Extreme support

Suggestion Description
Hello,
Is there any plans to support ROCm drivers on recent APUs Z1 or Z1 Extreme ?
Have a good day
Operating System
Ubuntu/Debian
GPU
Z1 Extreme
ROCm Component
No response

@ZZ0R0 Please open this ticket under 'Discussions'. Closing ticket. Thanks!


Adding Highlight Support in PPL

Signed-off-by: forestmvey<EMAIL_ADDRESS>Description
Adding support for highlight in PPL with optional arguments and wildcard support in SQL and PPL.
Issues Resolved
Issue: 636
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[x] New functionality has javadoc added
[x] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I PPL with this query:
source=beer | where multi_match([Title, Body, Tags], 'taste') | fields highlight('Body'), highlight('T*');

A piece of the output:
-[ RECORD 3 ]-------------------------
highlight('Body') | null
highlight('T*')   | {'Tags': ['brewing taste']}
-[ RECORD 4 ]-------------------------
highlight('Body') | [away from direct sources of heat or light; too much exposure to light can cause beer to develop a foul taste]
highlight('T*')   | {}

There is NULL when regular highlight matches nothing, but empty STRUCT when wildcard highlight matches nothing.
I suppose this should be consistent - both should return NULL or both return empty STRUCT/ARRAY.
What to you think, @MaxKsyunz?

Great job, Forest!
I don't have any concerns/objections except this one.
Please, add query samples when you will post PR on upstream.


メニューバーの改善

説明 / Description

「デバッグ」のサブメニューが増えたので、各種フォルダを開くメニューを分離する。
「検討」と「詰み探索」の並び順を入れ替える。

チェックリスト / Checklist

MUST

[x] npm test passed
[x] npm run lint was applied without warnings
[x] changes of /docs/webapp not included (except release branch)
[x] console.log not included (except script file)

RECOMMENDED (it depends on what you change)

[ ] unit test added/updated
[x] i18n

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (940add8) 69.64% compared to head (68a6047) 69.64%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #670   +/-   ##
=======================================
  Coverage   69.64%   69.64%
=======================================
  Files          98       98
  Lines       18692    18695    +3
  Branches     2719     2719
=======================================
+ Hits        13018    13021    +3
  Misses       2968     2968
  Partials     2706     2706

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Load more entry should provide progress feedback

Testing https://github.com/Microsoft/vscode/issues/57301
like opening a parent node does.

There is progress feedback in the view.


[EditorDialog] Cannot update anything in the editor

Description
User is not able to edit anything in the editor when passed "item" resource has no resourceVersion field inside metadata
Impact
User is not able to edit anything in the editor when passed "item" resource has no resourceVersion field inside metadata
Set-up:

Pass this object as an "item" prop to EditorDialog component
{                              apiVersion: 'v1',                              kind: 'TestKind',                              spec: {                                  testProp: 'testValue',                              },                              metadata: {                                  name: 'testName',                                  namespace: 'testNamespace',                              },                          }
Try to update anything in the editor. If you can't - that's it.
Try to add resourceVersion field into metadata
{  apiVersion: 'v1',  kind: 'TestKind',  spec: {      testProp: 'testValue',  },  metadata: {      name: 'testName',      namespace: 'testNamespace',      resourceVersion: '123',  }, } . Now you can

My use case:
Passed "item" object doesn't have resourceVersion, because it is used to create resource from some template, so user can extend it or change anything

@callmevladik , I had a take at this but cannot reproduce the issue. I found other issues that hopefully will improve the experience, and I do know that with the dev tools, the editor gets slow (#1254) but I couldn't reproduce your problem here. Maybe it's also fixed for you? Would appreciate if you can take some time for checking it. And thanks for reporting!

I am sorry, I had to provide code examples at first. Please, try using these code blocks for bug reproduce
1.This is an example of a resource that has resourceVersion field and can be edited with no problems.
<EditorDialog
    open={editorOpen}
    item={{
        apiVersion: 'test',
        kind: 'test',
        spec: {
            testProp: 'test',
        },
        metadata: {
            namespace: 'test',
            resourceVersion: 'test',
        },
    }}
    onClose={() => setEditorOpen(false)}
    onSave={e => console.log(e)}
/>

2.This is an example of a resource that has no resourceVersion field and cannot be edited with no problems.
<EditorDialog
    open={editorOpen}
    item={{
        apiVersion: 'test',
        kind: 'test',
        spec: {
            testProp: 'test',
        },
        metadata: {
            namespace: 'test',
        },
    }}
    onClose={() => setEditorOpen(false)}
    onSave={e => console.log(e)}
/>

3.One more example of non-working editor where there is no metadata field.
<EditorDialog
    open={editorOpen}
    item={{
        apiVersion: 'test',
        kind: 'test',
        spec: {
            testProp: 'test',
        }
    }}
    onClose={() => setEditorOpen(false)}
    onSave={e => console.log(e)}
/>

Thanks for the details @callmevladik , we will look into it.

I believe I have fixed the issue and also made the code more efficient at https://github.com/headlamp-k8s/headlamp/pull/1301
Would be great if you could try it @callmevladik .

@joaquimrocha Yeah, I've tried it and faced first issues:

When changing code It throws "Unexpected token 'a', "apiVersion"... is not valid JSON" errors.
Example(look at "emptyProject" field:

As a result, I cannot save&apply(trigger onSave callback).
The piece of code in the editor in upper screenshots was validated through online yaml linter

I've dived again in the issue and found out that the problem is that in my case editorData("item" property) initial value was null. That's the root of my whole problem. If initial value is empty object {} and then when the value is updated editor works fine and I don't get those errors.
One more time to make it clear :slightly_smiling_face:
When some data that I pass as a "item" property into EditorDialog component is stored in useState and its initial value is null, EditorDialog's mount logic fails and then when the state value is updated the editor doesn't work.
However, third issue I described here is still with me.
If item doesn't have metadata.namespace field, It thinks that I have already made some changes when I haven't


TypeError defining a schema with an input_processor and array input

I'm trying to restrict the output keys in an array of hashes to those defined in the schema using the sanitizer input processor.
I added a spec to reproduce the error on this branch:
https://github.com/dry-rb/dry-validation/compare/master...rhec:bug-input-processor-with-array-input
The output of the spec is:
1) Dry::Validation::Schema for an array with input_processor = :sanitizer sanitizes output keys
     Failure/Error: type_compiler.(hash_node(schema_ast(ast)))

     TypeError:
       no implicit conversion of Dry::Types::Array::Member into Hash
     # ./lib/dry/validation/input_processor_compiler.rb:17:in `call'
     # ./lib/dry/validation/schema/deprecated.rb:14:in `input_processor'
     # ./lib/dry/validation/schema/class_interface.rb:166:in `default_options'
     # ./lib/dry/validation/schema/class_interface.rb:36:in `new'
     # ./lib/dry/validation.rb:23:in `Schema'
     # ./spec/integration/schema/array_schema_spec.rb:62:in `block (3 levels) in <top (required)>'
     # ./spec/integration/schema/array_schema_spec.rb:76:in `block (3 levels) in <top (required)>'

It doesn't appear to be specific to the sanitizer input_processor. json and form also fail, so I think it's array input and input_processors in general that don't work. I'd be happy to dig deeper, but fact that input_processor is in a file called deprecated and not mentioned in the docs makes me think I should ask first if there is another way to do this.

@rhec did you get any further solving this? I'm finding a similar problem.

@pmackay no I'm simply looping through my array data and applying a schema individually to each item instead as a workaround. I never got confirmation, but my suspicion was that fixing this bug might be a waste as the input_processor feature seems likely to change radically or go away completely, given it's apparently deprecated.

@rhec you're absolutely right. We're in the process of rewriting the library, input processors will be gone in the next version. At the moment, the new version doesn't have support for top-level arrays, we believe it can be added on top of the hash-first logic a little bit later (I hoper before the release since it's something I need). Good news is now unkown keys are filtered out by default :) Also, track https://github.com/dry-rb/dry-validation/pull/440 for progress.


FindDomNode warning with react strict mode

This is not really a bug but when using react in strict mode a warning appears linked to OutsideClickHandler :
Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of r which is inside StrictMode. Instead, add a ref directly to the element you want to reference.
To Reproduce
Run react in strict mode and use calendarContainer prop
Expected behavior
I think this can be solved with this https://github.com/Pomax/react-onclickoutside/pull/319
Thank you

Have the same issue when using custom Input prop.

Hello! Is this fixed? I'm facing a similar issue in a Next,js project.

I get the same issue when using showMonthDropdown and showYearDropdown

Any updates on this?

same, using showYearDropdown with react-hook-form


VSCode hangs while starting in WSL

This bug is while using insiders with the Remote WSL extension

VSCode Version:

+ λ code-insiders --version
1.37.0-insider
360e4e4dfa989135f28edc0d6f7e42df33cddf87
x64

OS Version: Windows 10 WSL

Steps to Reproduce:

Run code-insiders .
IDE loads, status says Opening Remote but the IDE just hangs in the loading screen.

Does this issue occur when all extensions are disabled?: Yes/No
This bug requires the remote extensions.
Log:
[2019-07-18 20:38:21.594] Starting VS Code Server inside WSL (Ubuntu)
[2019-07-18 20:38:21.595] Extension version: 0.38.10, Windows build: 18362. Multi distro support: enabled. WSL path support: enabled
[2019-07-18 20:38:21.999] WSL resolver response: undefined:57286

Is there another WSL window open? Can you see if there's more in that Windows WSL terminal?

There was another WSL window open when it happened, yes.

Closing. If you see it again, please report to https://github.com/microsoft/vscode-remote-release/issues


Feegrant: Is it possible to implement the content of a sponsored-transactions (user A's fee is paid by user B)?

Is it possible to implement the content of a sponsored-transactions on cosmwasm (user A's fee is paid by user B)

Fee grant support was discussed in #1200 but was there any further discussion on this topic? Can't find any implementation in the code.
This could be useful to have if one has to test a contract on mainnet and has insufficient/low funds to begin with.

just saw implementation commented out 😛

Here is a team that is issuing grants as a contract (i.e. contract is grantee):
https://github.com/astroport-fi/astroport-core/blob/52af83eab04c620ac40019f7cc9cee433d0c601e/contracts/periphery/fee_granter/src/contract.rs#L120-L181
It's great this works but is not very convenient yet. E.g. the GRANTS need to be tracked separatly instead of being queried from chain.

@webmaster128

Can the --allowed-messages in this feegrant be set to the specified `contract address?
I want dapp user to be able to use money from my account for gas when they can use my contract.

@99Kies can you open an issue for that in wasmd (and link it here)? Alex should be able to help over there.

I want dapp user to be able to use money from my account for gas fee when they can use my contract.

Other devs achieve something similar by allowing the users to spend the contract's funds. In this case the contract needs to create the grant itself. Not necessarily easier but seems to be possible.

https://github.com/astroport-fi/astroport-core/blob/52af83eab04c620ac40019f7cc9cee433d0c601e/contracts/periphery/fee_granter/src/contract.rs#L120-L181
@webmaster128 I see this function. It's actually executing wasmd tx feegrant grantContractAddress granteeContractAddress.
By this way can user A call the methods of grantee Contract without consuming the handling fee?

I tried this and specifying grantee as the contract address doesn't work. Since it's still the user who is initiating the transaction, the fee is still paid by the user.

@webmaster128 I tried this and specifying grantee as the contract address doesn't work. Since it's still the user who is initiating the transaction, the fee is still paid by the user.


Font-family issues

<button>, <input>, '' and '' render a different font-family (Helvetica on Mac).
This is due to Normalize v5 using font-family: sans-serif (line 257) compare to font: inherit in the past.
We can fix this by updating _reboot.scss and add font: inherit (line 316).
Can anyone see a problem with this?

Dupe of #21588. Related to issue necolas/normalize.css#642 which will be fixed by PR necolas/normalize.css#649.

Closed as dupe.


Password feature doesn't work

I click the button to turn the password feature on, but nothing changes and it doesn't ask me to set a password. I'd like to be able to lock the extension with a password so that it can't be easily disabled.
Thanks

transferred #12


Add Prettier Format to Frontend

Description

Add support for Prettier in the frontend via npm run format which matches the workflow in the backend projects

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

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

[ ] Test A
[ ] Test B

Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have already been accepted and merged

Further comments

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend - Vehicles available
Backend - DOPS available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

You might also want to disable eslint rules that conflict with prettier.
https://prettier.io/docs/en/integrating-with-linters.html https://github.com/prettier/eslint-config-prettier
We have done the same for backend components

Should be addressed now - I think I've matched up the configs between Backend and us.


Pagination

This PR adds pagination support to houdini 🎉
At a high level, pagination is driven by two new functions (paginatedQuery and paginatedFragment) and a new @paginate directive. Both functions return an object containing loadNextPage, loadPreviousPage, and pageInfo along with the usual data and loading stores . The fragment support for pagination relies on the same Node query and interface that relay does if it's not a fragment on the schema's query type. paginatedQuery works without any extra constraints on the api. An example:
<script>
  const {  data, loading, loadNextPage, pageInfo } = paginatedQuery(graphql`
      query UserList {
          friends(first: 10) @paginate {
              edges {
                   node {
                       id
                   }
              }
          }
      }
  `)
</script>

{#if $pageInfo.hasNextPage}
    <button onClick={() => loadNextPage()}> load more </button>
{/if}

For more information, you can check out the README.
Along the way, I also improved the artifact generation logic to avoid relying so heavily on recast's AST builder. Instead it builds up a javascript object (using the typedefinitions from the runtime) and then converts it to the appropriate AST before writing the file. In order to make this simplier, when and when_not were also updated to an api structure similar to the @with and @arguments.  I still have to add some bits to the readme but it's published under houdini@pagination so people can start testing it before I merge..
closes #55, closes #123

Thanks @pixelmund for testing this for me! @thormuller do you think you could install houdini@pagination and make sure everything still works for you?

Just received confirmation from thor that his app is still working as expected - going to merge this now 🎉


master -> main: update automation, setting

We are getting with the times and migrating from master to main as primary branch 🎉
This work should be ideally contained in a single PR for easy switchover.
Tasks in scope:

[x] Update all GH workflows and actions to reflect this change
[x] Identify all matching branch occurrences and update those as well across the repo, excluding app code.
[x] Update primary branch and branch protections in GH Settings (not part of PR but settings should be reviewed so we can execute on that at the time of PR merge.)

Not in scope:

[ ] Any application code

DoD: Single PR ready for review with all changes included.  Github settings identified, ready to edit and to save.


error on click "save code snippet"

no matter what I do, when I click "save code snippet", a notification appears in the bottom left:
{"message":"Internal Server Error"}

I'm taken to the page where my preview should be but all I have is a "+ add another" button.
I'm also given the options to "save" which leads to another error:
`Expected a value of type `Array<string & Pattern<^[\.a-z0-9\-]+$> & Length<0,30>> & Length<0,7>` for `codeImgIds` but received `[]`.

Same for me, I think he is working on it: https://twitter.com/benawad/status/1336696657950478338


flutter_gallery__transition_perf_e2e_ios32 failing repeatedly mid-test

https://ci.chromium.org/p/flutter/builders/prod/Mac_ios flutter_gallery__transition_perf_e2e_ios32
Started with https://github.com/flutter/flutter/commit/c4ba26e31ba000d23de402af5be0aa129a9fda8e, but it is a revert and looks unrelated.
/cc @gspencergoog who is the current gardener.
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: VMServiceFlutterDriver: Connecting to Flutter application at http://<IP_ADDRESS>:49535/nfyiMuDo9Wc=/
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: VMServiceFlutterDriver: Isolate found with number:<PHONE_NUMBER>638331
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: VMServiceFlutterDriver: Isolate is paused at start.
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: VMServiceFlutterDriver: Attempting to resume isolate
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stdout: flutter: 00:00 +0: flutter gallery transitions on e2e find.bySemanticsLabel
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stdout: flutter: 00:00 +0 ~1: flutter gallery transitions on e2e all demos
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: VMServiceFlutterDriver: Connected to Flutter application.
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stdout: flutter: > Shrine@Studies
...
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stdout: flutter: > Grid@Material
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: Unhandled exception:
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: DriverError: Failed to fulfill RequestData due to remote error
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: Original error: ext.flutter.driver: (112) Service has disappeared
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: Original stack trace:
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr:
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr:
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: #0      VMServiceFlutterDriver.sendCommand (package:flutter_driver/src/driver/vmservice_driver.dart:313:7)
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: #1      FlutterDriver.requestData (package:flutter_driver/src/driver/driver.dart:508:39)
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: #2      integrationDriver (package:integration_test/integration_test_driver.dart:71:29)
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: <asynchronous suspension>
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stdout: Screenshot written to /opt/s/w/ir/k/recipe_cleanup/flutter_logs_dir/drive_01.png
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] stderr: Failed to stop app
[flutter_gallery__transition_perf_e2e_ios32] [STDOUT] "/opt/s/w/ir/k/recipe_cleanup/tmpnt_xi_/flutter sdk/bin/flutter" exit code: 1
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] Task failed: Executable "/opt/s/w/ir/k/recipe_cleanup/tmpnt_xi_/flutter sdk/bin/flutter" failed with exit code 1.
[flutter_gallery__transition_perf_e2e_ios32] [STDERR]
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] Stack trace:
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:flutter_devicelab/framework/utils.dart 98:3        fail
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:flutter_devicelab/framework/utils.dart 363:5       _execute
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] ===== asynchronous gap ===========================
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] dart:async                                                 _asyncErrorWrapperHelper
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:flutter_devicelab/framework/framework.dart         _TaskRunner._performTask.<fn>
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:stack_trace                                        Chain.capture
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:flutter_devicelab/framework/framework.dart 238:11  _TaskRunner._performTask
[flutter_gallery__transition_perf_e2e_ios32] [STDERR] package:flutter_devicelab/framework/framework.dart 137:41  _TaskRunner.run

It doesn't strictly seem to be a problem with the device since mac_ios_native_ui_tests_ios32 test passed on the same flutter-devicelab-mac-11 host on the same commits...

It passed locally for me on my iPhone 4s.

Marking it flaky in https://github.com/flutter/flutter/pull/79317

Now the phone device has been configured to restart when task fails.
Is there anything I can help triage the issue from infra side?
Asking @godofredoc to check if the phone looks in a good state from devicelab.

Now the phone device has been configured to restart when task fails.

Is that true for the 32-bit iOS phones?  I thought those needed special handling.

Asking @godofredoc to check if the phone looks in a good state from devicelab.

It looks like it's crashing, it gets halfway through the test and then after the failure it's sitting happily on the home screen.
https://storage.googleapis.com/flutter_logs/flutter/f12d90c4358be1d0ec159c9f72ea3ccc02fd9f59/flutter_gallery__transition_perf_e2e_ios32/1cdc6c62-d46d-4079-be71-e978a6dfac06/drive_01.png
Unfortunately we need idevicesyslog output for the old iOS 9 devices to see what crashed.

Now the phone device has been configured to restart when task fails.

Is that true for the 32-bit iOS phones? I thought those needed special handling.

Configs are the same for all ios tests. When tasks fail, it will call idevicediagnostics restart, which I assume is the same for any ios device?

@keyonghan I was thinking of
https://github.com/flutter/flutter/blob/b0afe3464e7d280f736917528e1c87046d6959f3/dev/devicelab/lib/framework/framework.dart#L20-L23
Is there a different script that reboots?

Ah, yes. that one is for rebooting after a certain number of test runs.
We have implemented an auto reboot from bot_config side, which has a typo! The --device-os should be ios.
That bot_config detects any test failure, and will call device_doctor tool to do device reboot.
I will submit a CL to update the --device-os input.
At the same time, @godofredoc has helped manually reboot the device, let's see if that helps.

Latest run succeeds. So rebooting the device helps.

which has a typo! The --device-os should be ios

👍
These old 32 bit fellas are just flaky sometimes... 😞   Will revert https://github.com/flutter/flutter/pull/79317.

Fix has been submitted to staging.
Will wait for the prod push before closing.

@keyonghan we should not reboot the ios32 phone. After rebooting, it gets to a "no sym card found" state, the previous logic was skipping that phone explicitly.

ah, right..
I will add a similar logic to exclude the 32 phone as well.

Removing out of ticket queue.

This is blocking to move the auto-recovery logic to luci.flutter.prod.

Logic has been added to exclude 32bit phone when rebooting. Closing.


Using Reactstrap

From the best practices that I've read, it's best to use ReactStrap. I tried integrating that into this framework which I used from your Udemy tutorial and I'm getting into issues of webpack not being able to load ReactStrap.
A lot of the examples I've seen are using Webpack2, and instead of "loaders" they use "rules". Maybe it would be good to update this project to latest Webpack, and maybe also address the use of ReactStrap?

upgrading webpack also works fine, or you can just use the CDNs in index.html


fix: certbot config

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

v1.51.1
GitHub release

Your semantic-release bot :package::rocket:


Use nanos eslint configuration from neeto-commons-frontend

In this repo, we are using the ESLint configuration designed for web apps from neeto-commons-frontend. We need to replace this configuration with the one designed for nanos.
Ref: https://github.com/bigbinary/neeto-hotkeys/blob/main/.eslintrc.js#L1

@deepakjosp _a


I don't know what's wrong

Hi, my AlexaPi still hasn't talked yet and when I look at its status, this comes up:
● AlexaPi.service - Alexa client for all your devices
Loaded: loaded (/usr/lib/systemd/system/AlexaPi.service; enabled)
Active: active (running) since Sat 2017-10-14 17:04:06 EDT; 3 weeks 2 days ago
Docs: https://github.com/alexa-pi/AlexaPi/wiki
Main PID: 522 (python)
CGroup: /system.slice/AlexaPi.service
└─522 /usr/bin/python /opt/AlexaPi/src/main.py --daemon
Oct 14 17:04:06 raspberrypi systemd[1]: Starting Alexa client for all your d....
Oct 14 17:04:06 raspberrypi systemd[1]: Started Alexa client for all your de....
Oct 14 17:04:14 raspberrypi python[522]: INFO: Checking Internet Connection ...
Oct 14 17:04:14 raspberrypi python[522]: INFO: Connection OK
Oct 14 17:04:14 raspberrypi python[522]: INFO: AVS token: Requesting a new one
Oct 14 17:04:14 raspberrypi python[522]: INFO: AVS token: Obtained successfully
Oct 14 17:04:16 raspberrypi python[522]: [73e004d8] alsa audio output error:...r
Hint: Some lines were ellipsized, use -l to show in full.
Can you please help me? Thanks.

Looks like you didn't setup the audio correctly, here are the steps https://github.com/alexa-pi/AlexaPi/wiki/Audio-setup-&-debugging
can you post the complete log by executing main.py -l

Sorry, but for main.py -l it says the command is not found when I put it in terminal. Also, I don't know how to set it up using the wiki. Sorry I'm new to this.

No Problem!
Execute this  /opt/AlexaPi/src/main.py -l & Copy Paste the Log here

All 600 something lines?

And how do I copy all of it?

2017-11-07 13:37:04 INFO: Checking Internet Connection ...
2017-11-07 13:37:04 INFO: Connection OK
2017-11-07 13:37:04 INFO: AVS token: Requesting a new one
2017-11-07 13:37:05 INFO: AVS token: Obtained successfully
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 763, in run
self.__target(*self.__args, **self.__kwargs)
File "/opt/AlexaPi/src/alexapi/triggers/pocketsphinxtrigger.py", line 55, in thread
inp = alsaaudio.PCM(alsaaudio.PCM_CAPTURE, alsaaudio.PCM_NORMAL, self._config['sound']['input_device'])
ALSAAudioError: Device or resource busy [plughw:CARD=Device,DEV=0]
[73b01728] alsa audio output error: cannot estimate delay: Input/output error
Here, I found it

@Linsanity77 You are still on an older version of AlexaPi. Please upgrade to 1.6 https://github.com/alexa-pi/AlexaPi/wiki/Q&A-(FAQ)#how-do-i-upgrade-alexapi

Thank you, I have now updated it. I checked the status again and this comes up:
● AlexaPi.service - Alexa client for all your devices
Loaded: loaded (/usr/lib/systemd/system/AlexaPi.service; enabled)
Active: active (running) since Sat 2017-11-11 13:39:44 EST; 3min 23s ago
Docs: https://github.com/alexa-pi/AlexaPi/wiki
Main PID: 564 (python)
CGroup: /system.slice/AlexaPi.service
└─564 /usr/bin/python /opt/AlexaPi/src/main.py --daemon
Nov 11 13:39:48 raspberrypi python[564]: ALSA lib pcm.c:2239:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
Nov 11 13:39:48 raspberrypi python[564]: ALSA lib pcm_dmix.c:1022:(snd_pcm_dmix_open) unable to open slave
Nov 11 13:39:48 raspberrypi python[564]: Cannot connect to server socket err = No such file or directory
Nov 11 13:39:48 raspberrypi python[564]: Cannot connect to server request channel
Nov 11 13:39:48 raspberrypi python[564]: jack server is not running or cannot be started
Nov 11 13:39:52 raspberrypi python[564]: INFO: Checking Internet Connection ...
Nov 11 13:39:53 raspberrypi python[564]: INFO: Connection OK
Nov 11 13:39:53 raspberrypi python[564]: INFO: AVS token: Requesting a new one
Nov 11 13:39:53 raspberrypi python[564]: INFO: AVS token: Obtained successfully
Nov 11 13:39:55 raspberrypi python[564]: [718004d8] alsa audio output error: cannot estimate delay: Input/output error
What do I do now?

Hi, I get the same error.
I tried debugging the audio with this guide: https://github.com/alexa-pi/AlexaPi/wiki/Audio-setup-&-debugging
However, on the "Checking if your system audio works" part when I'm testing recording, it requires your_input_device. How do I find your_input_device? Is it an ID? the device name?
OP, have you tried that audio setup & debugging guide?

Due to a large number of issues, this one is going to be closed because we think your problem is solved.
If you still have any problem, please read this first and feel free to re-open it!

Dec 03 04:40:22 raspberrypi python3[998]: CRITICAL: Your input_device 'plughw:1,0' is invalid. Use one of the following:
Dec 03 04:40:22 raspberrypi python3[998]: USB Microphone: Audio (plughw:1,0)
Dec 03 04:40:22 raspberrypi python3[998]: USB PnP Sound Device: Audio (plughw:2,0)
do I just put the input device as 'USB Microphone: Audio (plughw:1,0)'


[FEATURE] ✨ Make IPublish dotnet agnostic

Is your feature request related to a problem? Please describe.
IPublish component should not be dotnet specific
Describe the solution you'd like
Pipelines should have components that extends IPublish and that are specific to dotnet, docker and so on

Related to #46


bug: CSS issues while using VPhoneInput (Vue Vuetify)

Expected Behavior

Actual Behavior

Steps to Reproduce the Problem
I used the code provided on GitHub, but it seems like the CSS is not being applied correctly.

Specifications

OS: Window
Browser: Chrome
Browser version: 130
Package version: ^4.4.0
Vuetify version: ^3.7.4

Hello @bipingyawali! This looks similar to #29. Could you please provide a reproduction on stackblitz or GitHub?

Closing due to inactivity.


From raw to lux value

Hello everyone,
I'm trying to figure out how to convert the raw values to the real lux value. I suppose that it depends on the gain and integration time..
Could you help me?

The data sheet, available on the Adafruit product page and in the learn guide, says "Refer to Appendix A for the lux formula", but there is no Appendix A in the linked data sheet. Web searches for the data sheet yield copies at other locations, also without Appendix A. Searching specifically for Appendix A yields a copy of Appendix A (is it authoritative? seems so, but not sure):
https://cdck-file-uploads-europe1.s3.dualstack.eu-west-1.amazonaws.com/arduino/original/3X/8/d/8ddfc2d335f178e70e20974439a94ce31dd9df35.pdf
Looks to be a thorough description of the calculation to get a lux value.

Re-opening, this would ideally be added to the library.


refactor(LinearAlgebra/BilinearForm/Basic): Delete infered instances

Following #11278 the instances in LinearAlgebra/BilinearForm/Basic can be infered without further proof, and therefore the instance statements are no longer required.

bors d+
Thanks!

bors r+


[5.4] Add select function to Collection

Hi,
I added a select function to Collection, which made the collection behave like SQL select.
Thanks

Laravel 5.4 is not supported anymore.
What is the difference between select and only?
You did not provide enough information what this PR does and what is the use case for something like this.

Sorry, 5.4 is closed. Please send to 5.6.


Debugger does not show the 'Fields' of a record variable

Describe the bug
Very often while debugging, the variables group contains a local / global variable of type "Record", showing Field : Count = X. Normally, one would expect to see the '>' icon to drill down to the actual fields and see the content. however, very often the '>' symbol is not available.
Only workaround to use is to hover over the variable in code and to drill down to the field contents.

PS: In some scenarios it might refresh if you navigate to the next line, however this might be too late since the function / publisher / ... might have exited already...
Expected behavior
Get real time insights to the record variable field values in the variables group of the debugger.

5. Versions:

The updates to VSCode that was just released will hopefully improve that, and the hover ability. Will likely take the team some time to implement.

Can you try out our preview build? We are working on uptaking the new VSCode API.

If you're confident the above will be solved in the newest builds, you can close the issue. I'm currently still running on v15 and it will take some months before we will actually adopt v16 / newer builds. Will log another issue if the issue would still appear.


Picker showing seconds-selector when there is an escaped 's' in specified moment format

Fiddle.
Essentially just trying to display custom text after the date and time per moment's escaping format through the data-date-format attribute. In the fiddle above, the first input shows the seconds selector because there is a lower-case 's' in 'Eastern' -- the second omits the 's' with 'Eatern', and no seconds selector is shown.
Using version 4.7.14, but replicated this on the latest version as well.

Duplicate of #1645


Fix test script-2

Closing. Thanks for coming up with these improvements @yrong, but some of them aren't in the direction we want to go. Also this branch  is very out-of-date compared to our current start-services script.


Fetch Vlan IDs

This PR add a support for fetching VlanIDs for the DVPortGroups and standart porgroups.

Please link some issues showing why/where you need it.
It does not need to be issue directly on Forklift but could be from you project.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.

Project coverage is 15.62%. Comparing base (4d6a3f1) to head (dabe1d1).

Files with missing lines
Patch %
Lines

pkg/controller/provider/container/vsphere/model.go
0.00%
14 Missing :warning:

...controller/provider/container/vsphere/collector.go
0.00%
1 Missing :warning:

pkg/controller/provider/web/vsphere/network.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1225      +/-   ##
==========================================
- Coverage   15.63%   15.62%   -0.02%
==========================================
  Files         112      112
  Lines       23107    23123      +16
==========================================
  Hits         3612     3612
- Misses      19210    19226      +16
  Partials      285      285

Flag
Coverage Δ

unittests
15.62% <0.00%> (-0.02%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


routing: tracking visited nodes in findPath, fixing TestNewRoutePathTooLong test

These commits add a visitedNodes map that tracks if the Djikstra's algorithm in findPath has already visited this node before.  Previously, when evaluating the best node, we would look at all of the incoming edges and repeat the processEdge flow for already visited nodes.  The second commit enables the TestNewRoutePathTooLong test by fixing the excessive hops graph to correctly be parsed.
I think having the visitedNodes map is a good idea and generally good design, but I'm not sure if it really makes things that much more efficient.  It does prevent evaluating edge weights for edges we won't take though.

I think this will not have the desired effect, as we are not guaranteed that the first time we visit a node is the "shortest path" getting there. We must therefore evaluate all other incoming edges before we can consider the node properly "visited".

@halseth But we are popping the best entry from the heap so we should have the shortest path from source to this node, no?

Hm, but aren't we actually attempting to visit the node one the other end of the edge than the best node?

Ah, okay I see what it achieves now. So it is an optimization to avoid the extra calculation before reaching the line
if tempDist >= distance[fromVertex].dist {

?
I think you might be right then, since we assume we have processed all edges from that node at that point. The only exception is maybe the additional eddges?

Yeah good catch, the visitedNodes[pivot] = struct{}{} should be below the additional edges call, mb.  And yeah since we've processed all the edges from the node and it was the best node, there's no need to examine it in later calls.

I suppose instead of having the visitedNodes map we can just add the visited nodes to ignoredNodes and it would achieve the same thing.

Do you actually see any speedup by using this change?

I think I will need to create a test graph and test this out.  I think if the check is done before edgeInfo.FetchOtherNode, there should be some efficiency gain.  Like this:
var fromBytes [33]byte
switch {
case bytes.Equal(edgeInfo.NodeKey1Bytes[:], pivot[:]):
	fromBytes = edgeInfo.NodeKey2Bytes
case bytes.Equal(edgeInfo.NodeKey2Bytes[:], pivot[:]):
	fromBytes = edgeInfo.NodeKey1Bytes
default:
	return fmt.Errorf("node not participating in this channel")
}

if _, ok := visitedNodes[Vertex(fromBytes)]; ok {
	return nil
}

channelSource, err := edgeInfo.FetchOtherNode(
	tx, pivot[:],
)
if err != nil {
	return err
}

I was thinking about the findPath code today and I discovered that a particular node can be the "best node" in the heap multiple times.  Luckily, we will always evaluate the shortest distance to this node first and subsequent iterations can be ignored by the same visitedNodes map.  If we have a graph like this:
//		    ┌──────────┐250 ┌──────────┐
//		    │    D     │─── │    B     │
//                  └──────────┘    └──────────┘
//	     100  /                       |     \  50
//		 /                        |      \
//  ┌──────────┐/                         |       ┌──────────┐
//  │     S    │                          | 30    │     T    │
//  └──────────┘\                         |       └──────────┘
//		 \                        |      /
//	     100  \                       |     /  100
//		    ┌──────────┐250 ┌──────────┐
//		    │    C     │─── │    A     │
//	            └──────────┘    └──────────┘

then the current findPath algorithm will start with T, evaluate its neighbors (A, B) and add them both to the heap.  However, it will add A to the heap with a distance of 100 (for example's sake... not an accurate edge weight per the edgeWeight func) and B with a distance of 50.  Then the best node is B so we take it.  It evaluates its neighbors (A, T, D) and adds A to the heap with a distance of 80 (since it's less than 100) and D to the heap with a distance of 300.  The next best node is A with a distance of 80, so it's taken.  After that, the next best node is also A but with a distance of 100.  We can get efficiency gains if we check the best node before iterating over all channels with ForEachChannel (if we haven't iterated over the best node, we would also check the neighbors to see if we've already iterated over any of them as well - keeping the changes mentioned in the previous comments).

Ran a test with QueryRoutes in a loop. Performance improvement seems to be around 15%

LGTM. You can squash the fixup commits.

@joostjager squashed


Preventing loading Google Fonts - using local fonts

Dear all
due to legal relevance (DSGVO in Europe) I'd like to prevent loading Google Fonts from Google. As far as I understand this is possible and done this wonderful theme so far. (e.g. icomoon; fa-...)
But how to use the Montserrat font locally? Downloading an copying the files into the local /fonts/ folder and changing the custom.css doesn't seem to work.
An online check resulted in the usage of one Google font online. The full-text search found a link only in /layouts/partials/footer.html line No. 39 . Deleting this line or leaving all blank between " " works without errors but the look of the site suffers then.
Is it really correct to edit the footer.html file? It also seems that 'Montserrat' font is part of the local stored icomoon anyway? How to prevent any load of Google fonts?
Thank you.

Added googleFonts option to config. Disabling it will load fonts from local.
However, Thin, Light, SemiBold, ExtraBold, and Black fonts are not available. (We will add this option if requested.)

Thanks a lot. This is fine for me.