Java.type() fails with a PolyglotException when attempting to launch a Karate server

I'm running into the following error when I attempt launch a server using a feature file with Java interop:
org.graalvm.polyglot.PolyglotException: TypeError: Access to host class JWTUtil is not allowed or does not exist.

I've never run into this before, but this is the first Gradle project I've worked on (all others I support are Maven), so perhaps that's the culprit?
Apologies if this is a silly oversight on my part.  I've tried a number of things to work around this -- moving things to the src/main folder; moving things to the /resources sub folder; using the ZIP release JAR instead of the regular karate.jar -- all to no avail.
Sample project is attached -- wouldn't be surprised at all if launching the server works for you, and this is an IntelliJ setting issue, so again, apologies in advance if that's the case!
Complete error details:
ERROR com.intuit.karate - mock-server background failed - test-server.feature:4
com.intuit.karate.KarateException: mock-server background failed - test-server.feature:4
	at com.intuit.karate.core.MockHandler.<init>(MockHandler.java:111)
	at com.intuit.karate.core.MockServer$Builder.build(MockServer.java:129)
	at com.intuit.karate.Main.call(Main.java:396)
	at com.intuit.karate.Main.call(Main.java:58)
	at picocli.CommandLine.executeUserObject(CommandLine.java:1953)
	at picocli.CommandLine.access$1300(CommandLine.java:145)
	at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2352)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2346)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2311)
	at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2179)
	at picocli.CommandLine.execute(CommandLine.java:2078)
	at com.intuit.karate.Main.main(Main.java:298)
Caused by: com.intuit.karate.KarateException: js failed:
>>>>
01: Java.type('JWTUtil')
<<<<
org.graalvm.polyglot.PolyglotException: TypeError: Access to host class JWTUtil is not allowed or does not exist.
- <js>.:program(Unnamed:1)

test-java-type.zip

@staffier this is clearly a classpath issue. I think you have missed the sourceSets, refer: https://github.com/karatelabs/karate/wiki/Gradle
for reference, here is the maven pom.xml that works:
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>

    <groupId>test</groupId>
    <artifactId>test</artifactId>
    <version>1.0-SNAPSHOT</version>
    <packaging>jar</packaging>

    <properties>
        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
        <java.version>8</java.version>
        <maven.compiler.version>3.6.0</maven.compiler.version>
        <karate.version>2.0.0</karate.version>
    </properties>

    <dependencies>
        <dependency>
            <groupId>com.intuit.karate</groupId>
            <artifactId>karate-core</artifactId>
            <version>${karate.version}</version>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>io.jsonwebtoken</groupId>
            <artifactId>jjwt</artifactId>
            <version>0.9.1</version>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>javax.xml.bind</groupId>
            <artifactId>jaxb-api</artifactId>
            <version>2.3.1</version>
            <scope>test</scope>
        </dependency>
    </dependencies>

    <build>
        <testResources>
            <testResource>
                <directory>src/test/java</directory>
                <excludes>
                    <exclude>**/*.java</exclude>
                </excludes>
            </testResource>
        </testResources>
        <plugins>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-compiler-plugin</artifactId>
                <version>${maven.compiler.version}</version>
                <configuration>
                    <encoding>UTF-8</encoding>
                    <source>${java.version}</source>
                    <target>${java.version}</target>
                </configuration>
            </plugin>
        </plugins>
    </build>

</project>

and the command:
mvn test-compile exec:java -Dexec.mainClass=com.intuit.karate.Main -Dexec.classpathScope=test -Dexec.args="-m src/test/java/test-server.feature"

I have also just made an improvement to the CLI for mocks so that this will also work (not the use of classpath:)
mvn test-compile exec:java -Dexec.mainClass=com.intuit.karate.Main -Dexec.classpathScope=test -Dexec.args="-m classpath:test-server.feature"

Thanks, Peter.  I tried adding a sourceSets section to my build.gradle file, and that didn't work.  I'm noticing now, thought, that all my Karate servers involving Java interop are failing with this issue, regardless of whether I used Maven or Gradle to build them.  I tried using your pom,xml file above with the command you provided and received the same error.
However, for some reason, using karate.start() to start this same server works fine.  It's only when I try starting the server using the the JAR file that it fails.

Sure -- if you try to launch schema_server.feature here, you'll get an error complaining about the DataGenerator class: https://github.com/staffier/Karate-Demo-Project/tree/main/src/test/java/mock_servers.
I attached a zipped project, as well, which is basically the same thing I provided earlier, only with Maven...
test-java-type-maven.zip

I had some more time to fiddle around with this -- I'm getting the error with Java.type() for dependency-related classes, too.  For instance, if I add this to my build file:
dependencies {
    testImplementation 'com.auth0:java-jwt:3.19.1'
}

...and then add this to the Background of my feature file I'm using as a server:
    * def JWT = Java.type('com.auth0.jwt.JWT')

...I end up with this error:
org.graalvm.polyglot.PolyglotException: TypeError: Access to host class com.auth0.jwt.JWT is not allowed or does not exist.
If I instead add * def JWT = Java.type('com.auth0.jwt.JWT') to a Scenario, the server will start, but will return a 500 error (with the aforementioned error details) any time that Scenario is used to respond.

I'm reopening this with a help wanted tag, really hard for me to figure out what's up with Gradle :|

Just to reiterate, I am getting the same sorts of errors using Maven.  It appears to be related to Java.type()...

@staffier but I didn't with the pom.xml and command that I provided. so I need some help here

@staffier if possible can you build from the latest develop (the version will be 1.3.0-SNAPSHOT) and see if this is fixed. just a guess because we overhauled the JS engine a bit

Been a while since I tried this out and can't remember what I was running from the command line to try to launch my server.  Fwiw, I tried adding the 1.3.0-SNAPSHOT jar file to my project and just using java -jar karate-1.3.0-SNAPSHOT.jar -m test-server.feature -p 123 and ran into the same issue I described above...but I'm not sure what to use as a Gradle equivalent to the Maven command you posted above.
For reference, here is the build.gradle I'm using:
plugins {
    id 'java'
}

repositories {
    mavenCentral()
}

dependencies {
    testImplementation 'com.intuit.karate:karate-junit5:1.2.0.RC1'
    testImplementation 'io.jsonwebtoken:jjwt:0.9.1'
    testImplementation 'javax.xml.bind:jaxb-api:2.3.1'
}

sourceSets {
    test {
        resources {
            srcDir file('src/test/java')
            exclude '**/*.java'
        }
    }
}

test {
    useJUnitPlatform()
    testClassesDirs = sourceSets.test.output.classesDirs
    classpath = sourceSets.test.runtimeClasspath
}

@staffier no worries, let me take a look tomorrow. wonder if the Java version also is a factor

@staffier took your maven project and made a few tweaks. attached: test-java-type-maven.zip
here's the thing - when you include some Java classes, the JAR approach will not work unless you do some extra things. explained here: https://stackoverflow.com/a/56458094/143475
so the Access to host class JWTUtil is not allowed or does not exist message is because it is not on the classpath
anyway you can try my sample. first run the Mock, then the Test, everything looks good. unfortunately I'm not good at Gradle, so see if you can get someone to check if the classpath is ok. notice what I did was use a test package, because maybe dumping things in the "root" causes problems.
in short - when you start a mock, the Java code you depend on should be on the classpath

@staffier one more theory is that when you do karate.start() the classpath may not be "normal" compared to starting things from a Java class

can reopen if needed

No need!  I was just testing this out now and it works just fine.  The stackoverflow link was very helpful, as well.  Thanks again!


Update connectors-create-api-servicebus.md

Update peek-lock action with new affinity feature.

maybe move the "by default" to earlier? Not sure what is the best wording.

Asked author to make update in the internal doc repo. #please-close


Interested in creating a more comprehensive Tor control module?

You are likely aware of the stem Python controller for Tor.  I wonder if you'd be interested in working on a stem like nodejs module?
I've begun work on the control port class here: https://github.com/mattcollier/tor-control

Yeah I've heard of stem. Your idea is interesting and I guess worth working on. So you're planning on creating a new nodejs module for that purpose, correct?
Unfortunately I'm not sure I can spend time working on it, as I'm very busy over the next months. I've created node-ths out of necessity, just like I guess you plan on building tor-control out of necessity too. I guess I can however have a look on it from time to time and give feedback if you'd like.


Error

09.04 19:45:15 [Server] Server thread/CRITICAL TypeError: "Argument 2 passed to pocketmine\entity\Entity::createEntity() must be an instance of pocketmine\level\Level, instance of pocketmine\level\format\Chunk given, called in /plugins/morph-master/src/morph/main.php on line 141" (EXCEPTION) in "/src/pocketmine/entity/Entity" at line 545
09.04 19:45:15 [Server] Server thread/CRITICAL Unhandled exception executing command 'morph add Chicken' in morph: Argument 2 passed to pocketmine\entity\Entity::createEntity() must be an instance of pocketmine\level\Level, instance of pocketmine\level\format\Chunk given, called in /plugins/morph-master/src/morph/main.php on line 141
Ryanshaw2 > /morph add Chicken
Could you fix this please?

@RyanClarke2 okey, I update plugin

ok thank you


support GpuConcat on ArrayType

Current PR is to support GpuConcat on ArrayType, which is required in issue #2013.
In addition, this PR also introduces some refinement on the implementation of string concatenation.

build

build

build

build

build

build

build

Still would like to see us prevent against the empty concat when folding is disabled, but it is such a rare corner case it is not important enough to hold this up over it.

I added missing support of empty concat. And I also found that cuDF concat can be simply bypassed in single column concat.

build

build


Startup blocking behavior in docker compose

I have a docker compose file that starts rabbitmq (currently using tutum image, need to switch to this one), but the dependent services are trying to connect before rabbitmq is listening on the port:
https://github.com/tleyden/open-ocr/issues/74#issuecomment-250772680
How can I block until rabbitmq is listening for connections?  Will switching to the official rabbitmq image help?

I've had my apps wait for active connection and when one isn't present after 5 seconds, close the attempted connection and retry.  It's neither elegant, nor 100% reliable, but it does seem to work most of the time.  Maybe this answer will serve as a bump for someone with a better solution to share?  TIA.


Switch to unix path separators before normalizing path for Windows compatibility

Turns out that the path module stubbed into a web environment does not support Windows, nor contains the posix / win32 variants of path (it's fixed posix, so path.sep returns / in a win32 environment).
Effectively, what happened here is that the fileName was C:\foo\bar and upon normalizing, remained C:\foo\bar (it should've switched to /, or more specifically, path.sep).
This aligns the behavior with the other path normalization process as seen here: https://github.com/facebookincubator/create-react-app/blob/b17fa4123e2d098d943f4b33ae9c5c2ac311fab6/packages/react-error-overlay/src/utils/unmapper.js#L67
Fixes #3078.

Hmm, @gaearon -- isn't the "Check your code at" part a little redundant?

Redundant in what way?

Redundant in what way?

Just that we're displaying the code with contextual highlighting, but still telling the user to check the code at that location.
Other errors (e.g. document.body()) don't add that extra instruction; I'm not sure of a place where it'd be particularly useful.

We should probably make it consistent, but this can be fixed in React itself rather than here.

Hmm, wouldn't that degrade the experience for those not using react-error-overlay?

Degrade how? I assume they'd still see the stack trace. Error overlay just strips it out.

Hmm, I suppose it wouldn't then.

@timer Do you want to cut a release for this?

Sure, if you'd like!
I can cut one in about 10 hours if you don't get to it before then; I'm pretty tied up tonight.

Hmm, what's odd is that lerna wants me to update many packages:
Joes-MacBook-Pro:create-react-app joe$ npx lerna updated
lerna info version 2.1.2
lerna info versioning independent
lerna info Checking for updated packages...
lerna info Comparing with<EMAIL_ADDRESS>lerna info result
- eslint-config-react-app
- react-dev-utils
- react-error-overlay
- react-scripts

I'd assume it'd only prompt me to do react-error-overlay and eslint-config-react-app (for docs); so I'm not sure why react-dev-utils and react-scripts (except that they're dependencies), but specified as caret ranges already ...
Should we do a manual release of a patch version on the overlay or a whole react-scripts release cycle?


Better support for Rust syntax highlight

Rust syntax highlighting is not as good as nord-vim (fixed in https://github.com/arcticicestudio/nord-vim/issues/138). It would be great to add similar support to vscode as well.

Hi @ashrithr :wave:, thanks for your contribution :+1:
I've added it to the backlog, but feel free to submit a PR if you like.

I tried my hand at doing this today, but it's a lot more confusing than I expected. This is easily my favourite theme and not having Rust support is kinda a let down. Hope someone can get around to this soon 🙏

Found this theme earlier and I really like it so far, but had to add some overwrites to my own config to make it more pleasing with rust.
    "editor.semanticTokenColorCustomizations": {
        "[Nord]": {
            "enabled": true,
            "rules": {
                "interface:rust": "#8FBCBB",
                "enum:rust": "#8FBCBB",
                "enumMember:rust": "#8FBCBB",
                "struct:rust": "#8FBCBB",
                "keyword:rust": {
                    "foreground": "#81A1C1",
                    "italic": true
                },
                "namespace:rust": "#8FBCBB",
                "typeParameter:rust": "#8FBCBB"
            }
        }
    }

Don't really have the time or energy right now to create a pr but maybe someone (@iipom) finds this useful.

Hey there, I've just created a PR for this issue. Inspired by vim version and some Typescript rules from vscode. If you have a minute, could you pls review it ?
https://github.com/arcticicestudio/nord-visual-studio-code/pull/240


Fix SQL error in downgrade scripts

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/70/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/110/


ERR_CONTENT_LENGTH_MISMATCH

Issue Type: Bug
Failed to load resource: ERR_CONTENT_LENGTH_MISMATCH
it is terrible that i cannot search on the website
i have such an error and i dont have an option to just seek ERR_CONTENT_LENGTH_MISMATCH  on the awesome offsite!
what should i do with the error?
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:51:32.889Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.89GB (8.55GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe D:\ONVEGO\GoOld\App\index.html

Screen Reader
no

VM
0%

Extensions: none

Could you describe what you were doing at the time this error occurred?

hi
nothing remarkable.. just refresh a page, and some important awesome files (like fontawesome-all.min.css or fontawesome-all.min.js) are not getting downloaded. thus the page looks ugly, you can expect

@vammex Are you reporting an issue for the website or for the desktop app?

website

What browser are you using?

chrome Version 67.0.3396.87

@vammex I'm on the same version. Can you show us a screenshot of what's happening? (With devtools open). Also what are the Chrome extensions you installed that might cause this (Adblockers, Tampermonkey, etc)?

it happened a week ago. i dont have a screen...
i didnt install addons

So it doesn't happen anymore? If it does, feel free to reopen.


Question about discrete data

Does anyone know how to implement attacks and defenses on discrete data where the cost function is not continuous and we can not make derivatives directly?

Hi @baderrasheed Thank you very much for using ART!
I think you are asking an interesting question. To facilitate the discussion, could you please share more information about your dataset and the definition of your noncontinuous cost function? Maybe with a short example?

Hi. I'm currently a PhD student and I'm working on adversarial attacks on
deep neural networks for security applications where the data is discrete
almost all the time. My current interest is applying adversarial attacks on
malware detectors. In such applications, the loss function is not
differentiable directly with respect to the input, and ART does not work
directly as far as I know.
On Mon, Jun 29, 2020 at 3:00 PM Beat Buesser<EMAIL_ADDRESS>wrote:

Hi @baderrasheed https://github.com/baderrasheed Thank you very much
for using ART!
I think you are asking an interesting question. To facilitate the
discussion, could you please share more information about your dataset and
the definition of your noncontinuous cost function? Maybe with a short
example?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IBM/adversarial-robustness-toolbox/issues/486#issuecomment-651070433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHBPX4A5ZS3MU233KQCVUBTRZB655ANCNFSM4OKCRLTQ
.

@baderrasheed I think if you are using a (common) neural network for classification you should be able to obtain gradients with respect to its input. As far as I understand based on your explanation the question is more how to apply these gradients. For example you could binarize your categorical features and apply ProjectedGradientDescent with appropriate arguments norm=np.inf, eps=1.0 and eps_step=1.0  and estimator clip_values=(0, 1) which in combination would leave all features either 0 or 1 while minimising (or maximising) your loss function.


Multiple-Source Telemetry for PIREPs

Is your feature request related to a problem? Please describe.
Right now, the API and phpVMS assumes that there's only one source of information to log for flight from a telemetry and logging perspective. I want to change the behavior of this to allow for multiple sources of telemetry, and logs, information to be stored with a pirep.
This change can:

Allow for multiple sources to provide telemetry, and not conflict.
Display multiple sets of logs or multiple telemetry lines for a flight (e.g. see if the VATSIM telemetry matches the ACARS telemetry)

Describe the solution you'd like

Create a new source column in the acars table as a string that's nullable.
Modify the API to allow for setting this value.

Describe alternatives you've considered
No reasonable alternative has been considered for this.
Additional context
I have a online network client module, similar to how IPS worked back in the day, in the wings, and this change is essential so the module can log data and not interfere with vmsACARS or smartCARS logging.

Instead of strings we can use a source column (int) with some enums defined maybe.

Where would the enum be defined? Create another table in the DB?
The reason for a string was so that the full name could be used. Though, I can see why that's also a bad idea because it would be very inefficient on space.

Here's what I have so far regarding schema for this. @FatihKoz is this what you were referring to with a integer?
        Schema::table('acars', function (Blueprint $table) {
            $table->foreignId('source_id')->nullable();
        });

        Schema::create('acars_sources', function (Blueprint $table) {
            $table->id();
            $table->string('slug');
            $table->string('name')->nullable();
        });

Still having the same question, why not having an ENUM defined for sources like others we have.
https://github.com/nabeelio/phpvms/tree/dev/app/Models/Enums
Then you can have a simple field added to the acars table, $table->unsignedTinyInteger('source'); (we can easily switch to small or normal unsigned integers too if needed)
<?php

namespace App\Models\Enums;

use App\Contracts\Enum;

class AcarsSource extends Enum
{
    public const VMSACARS = 0;
    public const SMARTCARS = 1;
    public const IVAO = 3;
    public const VATSIM = 4;

    protected static array $labels = [
        self::VMSCARS => 'vmsACARS',
        self::SMARTCARS  => 'SmartCARS 3',
        self::IVAO  => 'IVAO Tracker',
        self::VATSIM => 'VATSIM API',
    ];
}

And if someone needs to separate something really 3rd party, enum can be updated to have something assigned to that or even that field can be used directly with a value, imagine yourself developing something and using 108 for your specific constant identifier.
With your solution, somebody (the dev of the addon) should write to that acars_sources table, get his/her/its assigned ID and need to use it. Or he/she/it needs to send the slug to get a match for the ID on each request etc. etc.
I do agree on having a field for source but the rest, slugs, additional table looks a bit not necessary at the moment. This is why I asked about enums in the first place @BossOfGames

Still having the same question, why not having an ENUM defined for sources like others we have.
https://github.com/nabeelio/phpvms/tree/dev/app/Models/Enums
Then you can have a simple field added to the acars table, $table->unsignedTinyInteger('source'); (we can easily switch to small or normal unsigned integers too if needed)
<?php

namespace App\Models\Enums;

use App\Contracts\Enum;

class AcarsSource extends Enum
{
    public const VMSACARS = 0;
    public const SMARTCARS = 1;
    public const IVAO = 3;
    public const VATSIM = 4;

    protected static array $labels = [
        self::VMSCARS => 'vmsACARS',
        self::SMARTCARS  => 'SmartCARS 3',
        self::IVAO  => 'IVAO Tracker',
        self::VATSIM => 'VATSIM API',
    ];
}

And if someone needs to separate something really 3rd party, enum can be updated to have something assigned to that or even that field can be used directly with a value, imagine yourself developing something and using 108 for your specific constant identifier.
With your solution, somebody (the dev of the addon) should write to that acars_sources table, get his/her/its assigned ID and need to use it. Or he/she/it needs to send the slug to get a match for the ID on each request etc. etc.
I do agree on having a field for source but the rest, slugs, additional table looks a bit not necessary at the moment. This is why I asked about enums in the first place @BossOfGames

That makes a lot more sense, and 3rd party developers can easily modify that file to add additional clients as needed.
I'll change my draft around to reflect that. Thanks!

I understand the idea of using an enum, but I’m not sure if it’s the most easily modular approach. It would mean that people would need to submit a PR here every time they want to add a source linked to one of their modules... I understand that the additional table might seem unnecessary though. Do you think there might be a middle-ground way to manage this?

Interesting idea. I wouldn't use an enum, in this case. A string is fine. Are you envisioning this to be the same type of data going into the acars columns?

I understand the idea of using an enum, but I’m not sure if it’s the most easily modular approach. It would mean that people would need to submit a PR here every time they want to add a source linked to one of their modules... I understand that the additional table might seem unnecessary though. Do you think there might be a middle-ground way to manage this?

My original thought was to have a sources table with a slug and name field. That way sources can be added independently and dynamically.
The drawbacks are definitely management. For example, a client source would need to have a slug or would need to know its integer ID when submitting the request. If it provides a slug, then it'll be tapping the DB every request to figure out the integer ID.
I do have this somewhat sorted out, as in the PR, I have created a new API route that allows a client to get the list of supported sources from phpVMS (we could do the same for the FlightType enum to return a "localized list" of types, instead of hardcoding them in clients. Issue for another time)

Interesting idea. I wouldn't use an enum, in this case. A string is fine. Are you envisioning this to be the same type of data going into the acars columns?

The reason for having an integer, and why I agree with @FatihKoz's implementation is that strings, in this instance, can dramatically increase the size of the DB in the area with easily the most records in the whole system.
Yes, the idea is that you can have two fully independent clients writing to the same flight tables at the same time, whether its telemetry or logs.

I don't think size is too much of a concern. Let's say it's limited to 3 characters, with only non-phpvms sources being required to set it? Or do you think there might be just as much data?

I don't think size is too much of a concern. Let's say it's limited to 3 characters, with only non-phpvms sources being required to set it? Or do you think there might be just as much data?

Ideally, all sources would set it. For example, on the pireps page, you could have vmsACARS telemetry separated from VATSIM telemetry. If it is limited to 3 characters, how would we go about setting displaying that info with a fancy name easily?
With the above considerations, you begin to go down the path of either enums or database tables with set IDs.
If we implemented only a string to start, it makes it easier for us to define a table with a fancy name, description, etc. if we decide to later. Though, I would rather do a integer for space savings now, if we know we're going to go down that path.

Considering the consequences and effects of both methods, I would like to ask some different questions...
What will be that "external" telemetry ?
Why do we need it in acars table ?
What will we do with it ?
I mean, if it will be coordinates, speeds, headings, altitudes etc. why do we need it to stay along the acars telemetry side by side ? What will be total gain of having it, if there are some necessities (say for VATSIM or similar), then why in acars table ?
It is an external source after all, not from a source/client directly connected to the sim/aircraft.
Can't a developer, while creating an addon, create a separate model / db table for that telemetry if it is that much important or required ?
Then the contents can be displayed along with a pirep, either with extending the current pirep model by adding relationships or by just loading the content whenever required.
I think adding an enum or using a string should be considered/discussed later, after deciding to store non-acars data in acars model :)

As an example to an external source, I am providing support for StableApproach plugin (currently only for XP, hopefully will have it in MSFS too in near future).
It is 100% external and optional, I created a model (and a db table as expected), recording the data without any alterations to base models or controllers, displaying it along with the related pirep easily. I never thought storing the data in acars or pirep models for this. It uses a custom enpoint to communicate with v7 whenever needed and stores its own data, that simple was the logic.
Why can't we have something similar for VATSIM Telemetry (or any other external source) but trying to add stuff to current base models ?

As an example to an external source, I am providing support for StableApproach plugin (currently only for XP, hopefully will have it in MSFS too in near future).
It is 100% external and optional, I created a model (and a db table as expected), recording the data without any alterations to base models or controllers, displaying it along with the related pirep easily. I never thought storing the data in acars or pirep models for this. It uses a custom enpoint to communicate with v7 whenever needed and stores its own data, that simple was the logic.
Why can't we have something similar for VATSIM Telemetry (or any other external source) but trying to add stuff to current base models ?

The use case I am trying to apply for here is that the VATSIM telemetry will, optionally, take the place of an ACARS client entirely, where I can use a software-less client setup for communities. In effect, creating a free client by utilizing somebody else's infrastructure.
Since VATSIM telemetry provides the same data as ACARS, minus the events and ability to start and stop flights, and there's options and ways to use this telemetry (e.g. building maps), I think it's natural to put this data into the acars table. Also remember, the acars table stores the logs too, which can be utilized (e.g. creating a log entry when a flight enters controlled airspace, checking what COM 1 frequency is tuned to verify they're on the right frequency for the DEP/ARR airport, etc.).
The overall issue I have with the above implementation is that I want a case where my tracking will work in cases where an ACARS client is used or not. I don't think it's productive to write code that writes to two separate places based on if a client is used or not.

Users no, developers or a developer will have to put effort for fulfilling your wish :) So here it is you basically, trying to reduce the amount of your time and efforts by skipping at least two steps (model creation and api endpoint creation) in the beginning. Which is fair an understandable, no bad feelings there.
I still think then my first approach, having an enum and sending it with the api should be more than enough ;)
And if in near future somebody else wants to do a similar addon/client/integration for IVAO they can use the predefined enum or request a new item to be added.
Ps: There will more stuff missing with VATSIM telemetry, not just events but in a clientless install all can be acceptable, and this is not the topic of this conversation :)

@FatihKoz fair enough. I like the enum implementation the most, especially after you explained it.
@nabeelio since you have the final say here on what gets pulled, given the information above, what is the best course of action? Will you accept a solution that includes a enum?

An enum is messy and you could get conflicts pretty easily - let's say you decide to use 4 and then an update comes out adding 4.
A string with a 5 character limit makes the most sense. If you look at it this way - if you create the column with the latin1 encoding, which will ensure it's 1 byte per char, 100,000 rows at 5 bytes used at max, it's only 500KB of storage extra, with maybe a little more for overhead
I don't think that's worth the hassle and potential conflicts down the road

An enum is messy and you could get conflicts pretty easily - let's say you decide to use 4 and then an update comes out adding 4.
A string with a 5 character limit makes the most sense. If you look at it this way - if you create the column with the latin1 encoding, which will ensure it's 1 byte per char, 100,000 rows at 5 bytes used at max, it's only 500KB of storage extra, with maybe a little more for overhead
I don't think that's worth the hassle and potential conflicts down the road

Ok, that's fair. We can back that with an enum if we decide later. Plus, since MySQL ints are 4 bytes, I won't get hung up on an extra byte.
I'll modify my PR to remove the enum and change the column type to a string with a limit of 5.

Ok, that's fair. We can back that with an enum if we decide later. Plus, since MySQL ints are 4 bytes, I won't get hung up on an extra byte.

Yeah, in the grand scheme of that table, if someone really wanted to save space, there's other areas to reduce (lat/lon resolution, etc).


Add explanation for implicit component attribute passing

From https://github.com/vuejs/vue/issues/5846.

@posva Good call. 😄  I was looking at a cat when making up that example. Updated!


Vulnerabilities in npm audit results

Running npm audit gives these results:
found 58 vulnerabilities (28 low, 26 moderate, 4 high) in 6895 scanned packages

I realize these are all devDependencies and probably aren't much of a concern, but I thought it worth mentioning given how easy it would be to fix most of them.
(many of these are from grunt-contrib-nodeunit)

I can see from package.json that @greenkeeperio-bot is/was active on this repository for some time, but was perhaps silenced early last year.


 A new Tcl interpreter will
            be created. BASENAME will be used for the identification of the profile file (see
            readprofile).
            It is constructed from sys.argv[0] without extensions if None is given. CLASSNAME
            is the name of the widget class."""
        self.master = None
        self.children = {}
        self._tkloaded = 0
        # to avoid recursions in the getattr code in case of failure, we
        # ensure that self.tk is always _something_.
        self.tk = None
        if baseName is None:
            import os
            baseName = os.path.basename(sys.argv[0])
            baseName, ext = os.path.splitext(baseName)
            if ext not in ('.py', '.pyc'):
                baseName = baseName + ext
        interactive = 0
>       self.tk = _tkinter.create(screenName, baseName, className, interactive, wantobjects, useTk, sync, use)
E       _tkinter.TclError: Can't find a usable init.tcl in the following directories:
E           {c:\python37\tcl\tcl8.6}
E
E       c:/python37/tcl/tcl8.6/init.tcl: couldn't read file "c:/python37/tcl/tcl8.6/init.tcl": No error
E       couldn't read file "c:/python37/tcl/tcl8.6/init.tcl": No error
E           while executing
E       "source c:/python37/tcl/tcl8.6/init.tcl"
E           ("uplevel" body line 1)
E           invoked from within
E       "uplevel #0 [list source $tclfile]"
E
E
E       This probably means that Tcl wasn't installed properly.
c:\python37\lib\tkinter\__init__.py:2023: TclError

For tests it's a good idea to configure matplotlib to use a "headless" backend like "Agg". It can avoid a lot of issues like this (also avoid spurious windows popping up on dev machines when tests are run locally).

I just ran this test on my ubuntu subsystem for windows with gui-backend disabled and this test in addition to two others failed. The failing tests were:
In linalg/decompositions_test.py:
test_kak_plot_empty
test_scatter_plot_normalized_kak_interaction_coefficients
In cross_entropy_benchmarking_test.py:
test_cross_entropy_benchmarking
All of them failed with "_tkinter.TclError: couldn't connect to display ":0"". I hadn't previously attempted testing with the gui disabled so this hadn't come up before. I ran these two files in pytest on a native windows environment and the cross entropy benchmarking test opens a spurious window but the decomposition tests do not appear to. I hadn't noticed these before as the window closes itself before the testing is done and I had only ever looked at the results.


Abstract out api.PodSpec in kubernetes.go

api.PodSpec is being used 4 times in kubernetes.go for RC, Deployment, DS and Pod generation.
Just putting it out there that this can be abstracted out.

:+1:

@containscafeine taking this.


Multiple Scans, one screen?

I am running a facebook group for my town and I currently have the mobile link shared on the page so people in the group can scan in town, is there any way I can scan 5 times but show it all on one single screen? The link provided to my group would show the results of all 5 scans on 1 single map?

im trying to figure this out........but it seems when i do mutiple scans it stacks the scans already?


Fix RPC example (again)

Following pr #437 I believed the issue was fixed, but it came up again in pr #440.
test src/actor_ref/rpc.rs - actor_ref::rpc (line 86) has been running for over 60 second

I think this is fixed by the inbox update in #443, specifically pr https://github.com/Thomasdezeeuw/inbox/pull/10.
I'm removing it from the First Release milestone, but keeping it open in case it comes back.

Haven't seen this anymore, so closing as fixed by #443 and https://github.com/Thomasdezeeuw/inbox/pull/10.


Codespaces: Error: there is no registered task type 'dotenv'. Did you miss installing an extension that provides a corresponding task provider?

Please provide us with the following information:

This issue is for a: (mark with an x)
- [X ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Start the Codespackes, check the Tasks log.

Any log messages given by the failure

Error: there is no registered task type 'dotenv'. Did you miss installing an extension that provides a corresponding task provider?

Expected/desired behavior

No error message.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Could you file an issue in the azd repo?
https://github.com/Azure/azure-dev
I don't know of what we could do on our side here, since that dotenv task comes from the azd VS Code extension.

Could you file an issue in the azd repo? https://github.com/Azure/azure-dev I don't know of what we could do on our side here, since that dotenv task comes from the azd VS Code extension.

Hello,
Now it is open: https://github.com/Azure/azure-dev/issues/4460


novel agr types?

Hello,
If the output of AgrVATE indicates that there are missense mutations in AgrD, does that indicate that my isolate has a novel Agr type? My understanding is that there should be no mutations in the AgrD
Many thanks for your wonderful tool,

Related to the question above, why does the sequence for gp4 not match that of this paper https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4847544/. Could that be the reason why above is detecting missense mutations in AgrD? Thanks in advance for your clarification.

Hello,
If the output of AgrVATE indicates that there are missense mutations in AgrD, does that indicate that my isolate has a novel Agr type? My understanding is that there should be no mutations in the AgrD
Many thanks for your wonderful tool,

Hi smb20200615,
There may be missense mutations in AgrD. However, we don't believe that necessarily means it is a novel agr type, especially if the mutation does not fall in the 8 amino-acid autoinducer peptide (AIP) region. Even still, a mutation in the AIP alone cannot be considered a new agr type as there needs to be appropriate compensatory mutations in agrB and agrC to be able to recognize the novel AIP. We did however find that AgrVATE reports S. argenteus agr group-1 isolates as a novel agr group, as in this case - it matches the S. aureus agr group-1 kmers with a low match score. We did not find any un-typeable agr operons in our analysis in most cases. The exceptions are noted here
Does this answer your question? Please let me know if you need additional clarifications.
Thank you,
Vishnu

Related to the question above, why does the sequence for gp4 not match that of this paper https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4847544/. Could that be the reason why above is detecting missense mutations in AgrD? Thanks in advance for your clarification.

Hi sskashaf,
If you are referring to Fig3A in this paper, yes our AgrD group IV reference is different from that sequence by a single amino acid. In our paper, we did report observing multiple AgrD alleles. But we didn't consider these alleles novel agr groups as the autoinducer peptide sequence was still one of the four known types. We believe these alternate alleles are a result of the clonal-complex/sequence-type specific allelic differences in S. aureus .
Hope this helps. Please let me know if you have other questions.
Thank you,
Vishnu

That makes sense. I was wondering, did you look at whether there were a higher proportion of deactivating mutations in Agr in certain clonal complexes? I am not sure how to properly quantify that - would that be numb of nonsyn mutations relative to synon as predicted by agrVATE?

We didn't specifically look at dN/dS ratios but that would be a good thing to check. We did however attempt to train classifiers to predict the presence of agr frame shifts based on factors such as agr group, clonal complex, host body site, and infection/colonization status (Table S2) with no promising results suggesting there may not be such associations (at least no such associations detectable by this method). Our analysis may also limited by the fact that few CCs are overrepresented in NCBI.


Override USWDS Sass packages at source

This builds upon #336 to further restructure the folder structure to align to how files are managed in the upstream project.
Why?

Improved performance. Improves build time for the default package by nearly 50% (8.8s to 4.7s)

See related comment at https://github.com/uswds/uswds/pull/5241#issuecomment-1540686386

Better maintainability. Avoids needing to keep pacakges/_uswds.scss in sync as the upstream USWDS adds new components over time.
Coverage for different import semantics. Specifically, supports both import of the named package, as well as the "Using Package Source" performance optimization.
Clearer separation. Overrides are strictly separate from the base styles.

Unfortunately the diff didn't turn out as clean as I'd hoped, despite the fact that 99% of changes are file renames.
Essentially, this converts:
Before:
src/
 - scss/
  - packages/
   - _usa-accordion.scss

// _usa-accordion.scss

@forward './usa-accordion/index';

.usa-accordion {
  // ...customizations...
}

After:
src/
 - scss/
  - packages/
   - _usa-accordion/
    - src/
     -  _styles.scss
     - _overrides.scss
   - _usa-accordion.scss

// _usa-accordion.scss

@forward 'usa-accordion/index';
@forward 'usa-accordion/src/overrides';

// usa-accordion/src/_styles.scss

@forward 'usa-accordion/src/styles/index';
@forward './overrides';

// usa-accordion/src/_overrides.scss

.usa-accordion {
  // ...customizations...
}

The same general principle of intercepting USWDS files applies, but now applies for both the top-level import (@forward 'usa-accordion') as well as the package source (@forward 'usa-accordion/src/styles').
The only meaningful code change here was to eliminate $border-width variable in some of the SCSS files. I was running into an error message that seemed to be caused by multiple forwarded modules each defining the variable. Unclear why this wasn't an issue previously 😬
Error: Two forwarded modules both define a variable named $border-width.
  ╷
1 │ @forward 'usa-checkbox/index';
  │ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ original @forward
2 │ @forward 'usa-checkbox/src/overrides';
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ new @forward
  ╵
  _usa-checkbox.scss 2:1               @forward
  uswds-form-controls/_index.scss 4:1  @forward
  _uswds-form-controls.scss 2:1        @forward
  uswds/_index.scss 51:1               @forward
  _uswds.scss 2:1                      @forward
  src/scss/styles.scss 2:1             root stylesheet

I'm open to trying to break this down into a few separate pull requests to try to coerce git into showing a simpler diff, if it'd be helpful for review.


bump kcp chart to 0.8.2

bump kcp chart to include changes from https://github.com/kcp-dev/helm-charts/compare/4b520a0c26ce5693ee78bac4ef9b7e3ca9edcdfa...ce9db7f6a6ad1c2acb3ebd17a8a544d39e1905d6

/approve


Rare intermittent push failure

Every so often, I get this when I try to push.  In this case it was immediately after doing

dokku apps:create tmxdev

$ git push dev master
Counting objects: 1267, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (596/596), done.
Writing objects: 100% (1267/1267), 4.13 MiB | 1.09 MiB/s, done.
Total 1267 (delta 726), reused 1045 (delta 589)
remote: -----> Cleaning up...
remote: -----> Building tmxdev from buildstep...
remote: time="2015-04-20T07:08:55Z" level=fatal msg="Error response from daemon: no such id: b9cd32db2d879b5a0fd479f3bf5ef51016f55e279e48c3e0bac281c77caf1aac"
To dokku@dokku.splitatom.com:tmxdev
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@dokku.splitatom.com:tmxdev'

We'd need some more info to troubleshoot this. Specifically the dokku trace output from a failed deploy.

@joshco bump

Closing as there hasn't been any response in a few months.


M3-590 Close Button on Domain Delete Modal Crashing Page

Purpose
Previously when opening up the domain delete confirmation dialog, clicking 'cancel' would crash the page instead of closing the modal. Changed the function responsible for that logic to a fat arrow function, and now all is well.
#JavaScript

Coverage remained the same at 27.223% when pulling 2cd5f9f815e697dafbdbb2ba1f9012031cf3bc30 on martinmckenna:M3-590-domain-dismiss-modal into 19c889bde0b3b8e2c165ba89e971b9290127c020 on linode:develop.


Update old links and references

Update old links to working equivalents. Replace Mesosphere with D2iQ as appropriate.

Codecov Report

Merging #1891 into master will decrease coverage by 0.56%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1891      +/-   ##
==========================================
- Coverage   98.94%   98.38%   -0.57%
==========================================
  Files          40       40
  Lines        2857     2857
  Branches      136      136
==========================================
- Hits         2827     2811      -16
- Misses         29       45      +16
  Partials        1        1

Impacted Files
Coverage Δ

src/dcos_e2e/docker_versions.py
100.00% <ø> (ø)

src/dcos_e2e/node.py
96.13% <ø> (ø)

tests/test_dcos_e2e/test_cluster.py
100.00% <100.00%> (ø)

tests/test_admin/test_brew.py
59.52% <0.00%> (-38.10%)
:arrow_down:

Continue to review full report at Codecov.

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

cc @adamtheturtle


Is it possible to ARROW_DOWN after select a option

Is it possible to ARROW_DOWN after select a option?
Just like you enter a text and pressed "enter" in a cell.

Not sure what you mean? Can you explain in more detail?

@mydea
When you typing text in one cell and press enter, selected cell will go to next row.

And one more thing, when you selected a cell, and press "enter".
Option popup flash in and out, and first option selected.
I think the the popup option should stay and let me select with arrow key UP / DOWN and press ENTER to select.


Column datatype mismatch between Spark DataFrame & SQL Table Error

I have created the Spark Dataframe using the connector. I created another dataframe based on this and aligned with the schema as in the SQL table (due to the issue mentioned here)
Even though the schema of the dataframe and the target SQL table is similar, it throws an error when I try to save the data into a sql data table.
java.sql.SQLException: Spark Dataframe and SQL Server table have differing column data types at column index 6

There is strict schema check that validates dataframe with SQL table if that already exists. Error when there is a mismatch. Can u share the column types for dataframe and sql table?

I confirm the bug. It is impossible to load data into a table that have a NOT NULL column. This is the schema of my dataframe:
root
 |-- L_ORDERKEY: integer (nullable = true)
 |-- L_PARTKEY: integer (nullable = true)
 |-- L_SUPPKEY: integer (nullable = true)
 |-- L_LINENUMBER: integer (nullable = true)
 |-- L_QUANTITY: decimal(15,2) (nullable = true)
 |-- L_EXTENDEDPRICE: decimal(15,2) (nullable = true)
 |-- L_DISCOUNT: decimal(15,2) (nullable = true)
 |-- L_TAX: decimal(15,2) (nullable = true)
 |-- L_RETURNFLAG: string (nullable = true)
 |-- L_LINESTATUS: string (nullable = true)
 |-- L_SHIPDATE: date (nullable = true)
 |-- L_COMMITDATE: date (nullable = true)
 |-- L_RECEIPTDATE: date (nullable = true)
 |-- L_SHIPINSTRUCT: string (nullable = true)
 |-- L_SHIPMODE: string (nullable = true)
 |-- L_COMMENT: string (nullable = true)

and this is the table schema:
create table [dbo].[LINEITEM_LOADTEST]
(
	[L_ORDERKEY] [int] not null,
	[L_PARTKEY] [int] not null,
	[L_SUPPKEY] [int] not null,
	[L_LINENUMBER] [int] not null,
	[L_QUANTITY] [decimal](15, 2) not null,
	[L_EXTENDEDPRICE] [decimal](15, 2) not null,
	[L_DISCOUNT] [decimal](15, 2) not null,
	[L_TAX] [decimal](15, 2) not null,
	[L_RETURNFLAG] [char](1) not null,
	[L_LINESTATUS] [char](1) not null,
	[L_SHIPDATE] [date] not null,
	[L_COMMITDATE] [date] not null,
	[L_RECEIPTDATE] [date] not null,
	[L_SHIPINSTRUCT] [char](25) not null,
	[L_SHIPMODE] [char](10) not null,
	[L_COMMENT] [varchar](44) not null
)

If you try to load data into the table you'll get error
java.sql.SQLException: Spark Dataframe and SQL Server table have differing column nullable configurations at column index 0

Call stack is:
Py4JJavaError: An error occurred while calling o4542.save.
: java.sql.SQLException: Spark Dataframe and SQL Server table have differing column nullable configurations at column index 0
	at com.microsoft.sqlserver.jdbc.spark.BulkCopyUtils$.com$microsoft$sqlserver$jdbc$spark$BulkCopyUtils$$assertCondition(BulkCopyUtils.scala:526)
	at com.microsoft.sqlserver.jdbc.spark.BulkCopyUtils$$anonfun$matchSchemas$1.apply$mcVI$sp(BulkCopyUtils.scala:279)
	at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:160)
	at com.microsoft.sqlserver.jdbc.spark.BulkCopyUtils$.matchSchemas(BulkCopyUtils.scala:249)
	at com.microsoft.sqlserver.jdbc.spark.BulkCopyUtils$.getColMetaData(BulkCopyUtils.scala:207)
	at com.microsoft.sqlserver.jdbc.spark.Connector.write(Connector.scala:53)
	at com.microsoft.sqlserver.jdbc.spark.DefaultSource.createRelation(DefaultSource.scala:51)
	at org.apache.spark.sql.execution.datasources.SaveIntoDataSourceCommand.run(SaveIntoDataSourceCommand.scala:45)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:70)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:68)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.doExecute(commands.scala:86)
	at org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:150)
	at org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:138)
	at org.apache.spark.sql.execution.SparkPlan$$anonfun$executeQuery$5.apply(SparkPlan.scala:191)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:151)
	at org.apache.spark.sql.execution.SparkPlan.executeQuery(SparkPlan.scala:187)
	at org.apache.spark.sql.execution.SparkPlan.execute(SparkPlan.scala:138)
	at org.apache.spark.sql.execution.QueryExecution.toRdd$lzycompute(QueryExecution.scala:117)
	at org.apache.spark.sql.execution.QueryExecution.toRdd(QueryExecution.scala:115)
	at org.apache.spark.sql.DataFrameWriter$$anonfun$runCommand$1.apply(DataFrameWriter.scala:711)
	at org.apache.spark.sql.DataFrameWriter$$anonfun$runCommand$1.apply(DataFrameWriter.scala:711)
	at org.apache.spark.sql.execution.SQLExecution$$anonfun$withCustomExecutionEnv$1$$anonfun$apply$1.apply(SQLExecution.scala:112)
	at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:232)
	at org.apache.spark.sql.execution.SQLExecution$$anonfun$withCustomExecutionEnv$1.apply(SQLExecution.scala:98)
	at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:835)
	at org.apache.spark.sql.execution.SQLExecution$.withCustomExecutionEnv(SQLExecution.scala:74)
	at org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:184)
	at org.apache.spark.sql.DataFrameWriter.runCommand(DataFrameWriter.scala:711)
	at org.apache.spark.sql.DataFrameWriter.saveToV1Source(DataFrameWriter.scala:307)
	at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:293)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
	at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:380)
	at py4j.Gateway.invoke(Gateway.java:295)
	at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
	at py4j.commands.CallCommand.execute(CallCommand.java:79)
	at py4j.GatewayConnection.run(GatewayConnection.java:251)
	at java.lang.Thread.run(Thread.java:748)

you can force a column to be NOT NULLable in a dataframe by using COALESCE(nullableColumn, 'NULL') AS notNullableColumn

@gbrueckl sure I found a workaround, but with the previous connector I didn't have to do any workaround :) So I guess this is something it could be improved :)

@yorek sure, totally agree - I also stumbled over this bug when migrating from azure-sqldb-spark to this connector
I was only referring to your statement "It is also it is impossible to change the DataFrame colum nullability to False"

@gbrueckl oh I see now :) Thanks! :)

coming back to the original issue - it works just fine for me if the schema is identical in terms of column names, data type, nullability and column order

Solved in #52


Basic ES6 example application

We should have a basic (vanilla) ES6 example application. Regarding that I have question:
How do we handle the actual browser implementation? Do we transpile it while building and serve a browser-ready .js file? Or do we use an ES6-ready loader library like SystemJS for instance?
Since we did the transpiling thing on our website and considering the upcoming success of HTTP2 I would go for the SystemJS (or anything similar) solution, since this provides the possibility to actually make use of the advantage of having small components which are loaded on demand rather than serving one big .js file and only having the components as source files.
@gridonic/core Any feedback from you guys?

As of now, we don't have SystemJS anywhere except for one project. Until now, we've been using Grunt + browserify + babel. I would first do a version that works with this approach, and they do a SystemJS example. Thoughts?

Well the application itself won't have any differences regarding the way we implement it, right? It will export the application class an that's it.
So I will setup the example application and provide two (theoretical) ways of implementing. One pre-built and one on-demand.

Since I'm working on the yeoman generator, all our (upcoming) example applications will be documented at https://github.com/gridonic/js from now on. It's best to continue discussing any open topics over there.


Plug + airline + modeline = error?

environment

vim: NVIM 0.1.3
vim-airline: 1bb8b6278e901cbc282535cabb1f6e17f8059e2a
OS: Arch Linux
terminal: bash

steps to reproduce
vimrc:
call plug#begin('~/.vim/bundle')

Plug 'vim-airline/vim-airline'

call plug#end()

Opening the following file with vim -u vimrc-test -V5/tmp/vim.log results in lots of error (too long, posted at https://paste.xinu.at/RFUu2/#n520):
# vim: ft=sh:

Am 2016-04-13 09:55, schrieb Hexchain Tong:

ENVIRONMENT

vim: NVIM 0.1.3
vim-airline: 1bb8b62 [1]
OS: Arch Linux
terminal: bash

STEPS TO REPRODUCE
vimrc:
call plug#begin('~/.vim/bundle')
Plug 'vim-airline/vim-airline'
call plug#end()
Opening the following file with vim -u vimrc-test -V5/tmp/vim.log
results in lots of error (too long, posted at
https://paste.xinu.at/RFUu2/#n520):
vim: ft=sh:

That is an nvim issue.


[tracking] Use Verible's verilog_format for code formatting

Verible's verilog_format is a automatic code formatter. We want to be able to run it on our code base, ultimately in "enforcing" mode, requiring all submissions to be formatted with this tool. This is the same approach we take with clang-format and C/C++ files. This issue is to track the progress towards that.
Ibex TODO:

[x] Add Verible format support to edalize: https://github.com/olofk/edalize/pull/120
[x] Add a fusesoc target to our core files: #686
[x] Add Verible format in non-enforcing mode to CI: #686
[ ] Document how to use it
[ ] Clean up existing code once the auto-formatted style matches our expectations.
[ ] Switch to enforcing mode.

Bugs crashing the formatter:

[ ] "Some token partitions failed to complete within the search limit" https://github.com/google/verible/issues/346
[ ] "Formatted output is lexically different from the input" (Mismatched token enums.): https://github.com/google/verible/issues/347

Wrong style produced by the formatter according to the lowRISC style guide:

[ ] Use two space indents for port and parameter lists: https://github.com/google/verible/issues/348
[ ] Indentation of initializer lists: https://github.com/google/verible/issues/352
[ ] Comment in assign statement breaks indentation: https://github.com/google/verible/issues/353

Optional Verible formatter bugs:

[ ] https://github.com/google/verible/issues/228. Workaround for one specific case proposed in https://github.com/lowRISC/ibex/pull/693 to remove the blocking status of this, given that a full solution on the Verible side will take longer.
[ ] Continue comments in port lists: https://github.com/google/verible/issues/349
[ ] Line up comments in port lists: https://github.com/google/verible/issues/350
[ ] Closing parenthesis on its own line: https://github.com/google/verible/issues/351

The first set of code changes to get our code in line with what verible format produces is at https://github.com/lowRISC/ibex/pull/1014. The goal is to have that PR cover all the "easy" cases, getting us step-by-step closer to a full equivalence.

Thanks for your work in pursuing this, filing upstream issues, and tracking progress.
IMHO #351 should be higher priority - it seems to be a case of producing incorrectly formatted code, which is a much bigger barrier to enforcing verible formatting use than producing output that doesn't match our personal tastes. But once incorrect formatting bugs are addressed I agree with the thrust of your argument in #1014 - don't sweat the small stuff, and in return formatting becomes a solved problem.
We'll want to consult with downstream consumers (OpenTitan) prior to landing a large cosmetic change like #1014. I don't see why it should be problematic at this stage, but it couldn't hurt to explain the plan.


Make this plugin compatible with Daily Notes Editor

Enhancement Checklist

[X] I have searched existing issues and this is not a duplicate
[X] I am reporting a simple enhancement or improvement, not new features

Enhancement Type
Capability with other plugins
Enhancement Description
Daily Notes Editor seems to use hidden tabs to load recent daily notes together into one long page. Each daily note currently loaded shows up as a tab group in vertical tabs.

Clicking on any of these daily note tabs switches to DNE's long Daily notes page.
Additionally if the Deduplicate tabs option is toggled, each time I try to click on DNE's Daily notes page (either in the top tab bar or the vertical tabs sidebar), Obsidian immediately switches back to the previous tab.
(Not sure if this is helpful, but Mononote, another deduplication plugin, doesn't have this clashing behaviour with DNE.)
Motivation
Users get to use both plugins :)
Alternatives
I typically use Vertical Tabs in background mode, so the sidebar thing is personally not that important, but for now I'll leave Deduplicate tabs off, and use Mononote for that function.
Additional Information
I'm using Win10.

Hi @kazerniel ! Thank you for posting the request.

Daily Notes Editor seems to use hidden tabs to load recent daily notes together into one long page. Each daily note currently loaded shows up as a tab group in vertical tabs.

Well, I'm afraid that this is a bad design of Daily Notes Editor; and Vertical Tabs accidentally reveals the problem. I checked the code and find that some functionalities of Daily Notes Editor are not working properly, which are critical for Vertical Tabs to interact with it.
Vertical Tabs includes an unreleased feature called Continuous View that does the same thing of displaying multiple notes in a single page, but works for any notes and any tab groups.
Would you consider replacing Daily Notes Editor with Vertical Tabs' Continuous View if I make it publicly available?
If you still want Vertical Tabs to work with Daily Notes Editor, I may find a workaround, but it won't be efficient and may hurt the performance.

but Mononote, another deduplication plugin, doesn't have this clashing behaviour with DNE

Because we are using different algorithms for deduplicating, Mononote's solution may cause issues for some users. I'm improving mine as well.

Vertical Tabs includes an unreleased feature called Continuous View that does the same thing of displaying multiple notes in a single page, but works for any notes and any tab groups.
Would you consider replacing Daily Notes Editor with Vertical Tabs' Continuous View if I make it publicly available?

Thank you, that sounds great! Would the notes still be editable even in Continuous View? If so, then I'm happy to switch away from Daily Notes Editor (unfortunately it's quite laggy anyway 😅)

Would the notes still be editable even in Continuous View?

Of course!
My implementation takes a slightly different approach to better align with Obsidian’s design principles. In Continuous View, a group of tabs transforms into a single continuous page, rather than nesting multiple tabs inside another tab. This design integrates seamlessly with Obsidian’s UI. By combining Continuous View with Zen Mode, you can achieve a similar experience to Daily Note Editor—without the lag, thanks to its streamlined and consistent design.


4 players info chopped off

At top right of screen the panel that shows all the players scores does not show active players score.
Recommend using html/css flex container to work with different screens.  Example Nicole uses laptop monitor only

We don't really show the player scores, but I did just fix a problem where you couldn't see part of the resources on a small screen (not yet uploaded to the online version).  Please re-open if that was not the issue.


fix failing file upload, merge three file upload Cypress tests into one

Check the failing Cypress tests in main, the file upload test fails (I think the upload in the mock server doesn't work, no file is uploaded in the /upload folder, which is very odd).

You can check https://github.com/OpenCovenant/quill/issues/383 if you won't have much success in this, it is a much jollier issue :D

Solved, bug is fixed and I've merged three tests into one. There shouldn't be any issue on this anymore.


Consul Connect Should be allowed on any Server

Currently, the template for config.json has the following logic
{% if (item.config_version != 'client') and consul_connect_enabled %}

This requires either a bootstrap or server node in order to utilize the Consul Connect technology. With the rise in technologies like Hashicorp's Nomad, Connect is used to provided a secured mesh network between CLIENT nodes. Allowing this only on "Server" no longer mistakes and isn't even compliant with Nomad architecture.

Resolved by #401


Absolute path to access index.js

@josecorella is hitting the following error from using v1.7.1 within an MPL workflow
2024-07-15T16:37:48.1742150Z ##[group]Run node dist/index.js
2024-07-15T16:37:48.1742380Z node dist/index.js
2024-07-15T16:37:48.1774930Z shell: /bin/bash --noprofile --norc -e -o pipefail {0}
2024-07-15T16:37:48.1775170Z env:
2024-07-15T16:37:48.1775300Z   DOTNET_CLI_TELEMETRY_OPTOUT: 1
2024-07-15T16:37:48.1775490Z   DOTNET_NOLOGO: 1
2024-07-15T16:37:48.1775640Z   DOTNET_ROOT: /var/root/.dotnet
2024-07-15T16:37:48.1775830Z   INPUT_DAFNY-VERSION: 4.2.0
2024-07-15T16:37:48.1775990Z ##[endgroup]
2024-07-15T16:37:48.2328720Z node:internal/modules/cjs/loader:1031
2024-07-15T16:37:48.2329140Z   throw err;
2024-07-15T16:37:48.2329310Z   ^
2024-07-15T16:37:48.2329410Z
2024-07-15T16:37:48.2330660Z Error: Cannot find module '/private/tmp/codebuild/output/src962/src/a951722d_5a9e_4668_ab99_68c59d804d36/actions-runner/_work/aws-cryptographic-material-providers-library/aws-cryptographic-material-providers-library/dist/index.js'
2024-07-15T16:37:48.2331720Z     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1028:15)
2024-07-15T16:37:48.2332170Z     at Function.Module._load (node:internal/modules/cjs/loader:873:27)
2024-07-15T16:37:48.2333260Z     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
2024-07-15T16:37:48.2333710Z     at node:internal/main/run_main_module:22:47 {
2024-07-15T16:37:48.2333990Z   code: 'MODULE_NOT_FOUND',
2024-07-15T16:37:48.2334180Z   requireStack: []
2024-07-15T16:37:48.2334350Z }
2024-07-15T16:37:48.2340470Z ##[error]Process completed with exit code 1.

So the fix should be to change the invocation to node $GITHUB_ACTION_PATH/dist/index.js following this

Previously, this action was only a Javascript action, but I turned it into a composite action to install the right version of .NET for Apple Silicon runners, etc. It looks like my fix here is correct because someone ran into the same issue and applied the same fix so you can go ahead and approve.


Update imageFactory eol fied as null

Description 📝
End of life field is set to null in imageFactory As our e2e specs are using this factory to build mock data.
This fixes failed 2e2 test create-linode-from-image.spec.ts
How to test 🧪
Navigate to imageFactory validate end of life filed
How do I run relevant unit or e2e tests?
yarn cypress:debug
Then run spec file create-linode-from-image.spec.ts under Images

@jdamore-linode Created PR to address the failed spec create-linode-from-image.spec.ts.
Thought: Wondering it would be worth if we update the spec that leverages this imageFactory and build expected data from it. My thought is we can make this spec more robust and avoid breakages in future because of changing any default values in the imageFactory. But, I'm open your thoughts on this.

@jdamore-linode Created PR to address the failed spec create-linode-from-image.spec.ts.
Thought: Wondering it would be worth if we update the spec that leverages this imageFactory and build expected data from it. My thought is we can make this spec more robust and avoid breakages in future because of changing any default values in the imageFactory. But, open your thoughts on this.

@cpathipa Not a bad idea at all! I would probably suggest modifying the createMockImage() helper function in packages/manager/cypress/support/images.ts to explicitly set eol to null, and then it may be worth reconsidering whether we should change the factory default at all

@jdamore-linode @dwiley-akamai Sounds good to me, I meant the same. As, we have flexibility to build the mock data with custom values from factory w.r.t to test scenario. So, reverted the initial commit made to factory imageFactory and updated createMockImage


compatibility between sync and async provider

A proposal to pin the Airflow sync version of providers in the astronomer async provider
TODO: @pankajastro add details

is this still needed @pankajastro ?

is this still needed @pankajastro ?

No, since now our operator/sensor works just as a proxy


Run go bits inside a container

/cc @deads2k @stevekuznetsov
Depends on https://github.com/openshift/origin/pull/17681

That test requires direct access to the daemon... we've been asking for it to be refactored for what feels like more than a year now.
/cc @bparees @miminar @dmage

That test requires direct access to the daemon... we've been asking for it to be refactored for what feels like more than a year now.

what test does?

(and why the heck do we have a target named test-end-to-end-unit? I can't think of anything much more contradictory).

Hmm I may have tagged the wrong set of people -- I was certain registry tests fell into the bucket (but maybe they just moved out of Origin? I definitely remember them being in here). @knobunc it's currently router tests that expect direct Docker daemon interaction https://github.com/openshift/origin/blob/master/test/end-to-end/router_test.go

Yes presumably they moved when the registry was removed. (of course the
same problem may exist for them in their new location)
Ben Parees | OpenShift
On Dec 8, 2017 20:22, "Steve Kuznetsov"<EMAIL_ADDRESS>wrote:

Hmm I may have tagged the wrong set of people -- I was certain registry
tests fell into the bucket (but maybe they just moved out of Origin? I
definitely remember them being in here). @knobunc
https://github.com/knobunc it's currently router tests that expect
direct Docker daemon interaction https://github.com/openshift/
origin/blob/master/test/end-to-end/router_test.go
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/openshift/aos-cd-jobs/pull/930#issuecomment-350412395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEvl3nBwO-tPxLYbla7gwrRbjb32-urRks5s-eDMgaJpZM4Q7GVT
.

I believe it remains. @mfojtik can we get prioritization for that?


Replaces spring-test with custom JNDI mock

Since Spring 6.x the JNDI mock is no longer available.
We replace it with a simplified custom version based on Tomcat's JNDI implementation.
This change is specific to 3.x. The 2.x branch can continue using spring-test.

Closing this in favor of #2072.


Cursor is hidden behing context window

I use zt to scroll the current line to the top. A normal use case is to check a code block inside a long function.
However the top line will be overlapped by context line and this is annoying. Suggest remap zt to move the scroll the current line to the 2nd top line.

I'll see what we can do, I wouldn't be comfortable remapping fundamentals such as zt but we could provide a context-aware zt alternative for you to map to.

I have experienced this bug as well (the cursor being behind the float window). I would say this is more general then just a conflict with zt. You can see this behaviour with any/many? movements that move the cursor to a line that is visible in the screen area of the current window but hidden behind the floating window.
In order to test this you need any buffer with a deep enough context, for example lua/treesitter-context.lua from this repo. I am using bdc598b519cfbfdf61baf9a27953da2fc1642b23 so I am positioning line 77 at the top of my window (:77Gzt in neovim). Now you can move the cursor around (as long as the window does not scroll) and see it hiding behind the floating window and reappearing for example type 82G and then 78G. Or :79<CR> and then :77<CR>. You could even move up and done a litte with j and k.
The point is that you would have to remap everything. And I guess that will lead to conflicts with all kind of configs or plugins. I think the correct approach would be to find a more general solution.
Idea
The area hidden by the float feel as if it does not belong to my current window so the window should be scrolled when entering this area:

detect the size of the context window and check if the widow-line of the cursor is behind it
scroll the current window until the current line is below the floating window

Question/Problem
Will it be possible to run in an endless loop?  Some hypothetical situation where

the cursor is hidden
the window gets scrolled
but that changes the size of the context to display
so we should actually "undo" the scroll
but when the scroll is undone, the context gets bigger again and hides the cursor again
But maybe such a situation is impossible, I did not think it through.

I did something. Let me know if it's still a problem for you.

Had to revert the changes.

I think the title of this issue can be changed to be more general.

Perhaps as a workaround we could update the buffer-local &scrolloff setting to prevent the cursor from overlapping with the context window.
This approach has the drawback that scrolloff affects both the top and the bottom of the window.
If this approach is acceptable to those in charge here, I'll try to open a PR with such a patch.


sync downstream with upstream main

syncing downstream with upstream main

/lgtm
/approve

/lgtm
/approve

/lgtm
/approve


add support for more languages and upgrade shiki

Hey there thanks for making this plugin. I needed syntax highlighting for elixir and noticed that shiki was out of date. I updated that dependency and moved from the now deprecated oniguruma to vscode-oniguruma.

Closing this for now since it's gotten out of date. I believe the Shiki part has now been tackled but please submit another PR with the language changes if those are still needed


Fix derivedFrom attribute types and enumeratedValue minOccurs

I'm working on a SVDSuite, a Python package to parse, process, manipulate, validate, and generate CMSIS SVD files. During development, I noticed a discrepancy between documentation and schema.
According to the documentation, derivedFrom can be referenced from another scope for elements cluster, register, field, and enumeratedValues. However, in the schema, the type for the derivedFrom attribute is dimableIdentifierType which does not allow the . character. Furthermore, the enumeratedValue element in enumeratedValues must be specified (minOccurs set to 1), which doesn't make sense for deriving. This issue was already reported here.
This PR addresses the described issues. It contains the following changes:

add referenceIdentifierType (allows <dimableIdentifierType>.<dimableIdentifierType> with any depth, but not .<dimableIdentifierType>, .<dimableIdentifierType>., or <dimableIdentifierType>.)
change minOccurs for element enumeratedValue from 1 to 0 to allow deriving
change enumeratedValues derivedFrom attribute type to referenceIdentifierType to allow referencing
change field derivedFrom attribute type to referenceIdentifierType to allow referencing from another scope
change register derivedFrom attribute type to referenceIdentifierType to allow referencing from another scope
change cluster derivedFrom attribute type to referenceIdentifierType to allow referencing from another scope
modify svd_schema.txt so that the documentation corresponds to the schema

Please let me know what you think about the PR and if it needs any modifications.

Hi @thorstendb-ARM, @jkrech, @JonatanAntoni,
just wanted to check in on the status of the pull request I submitted around 4 months ago. I understand you might be busy, but I'd appreciate any updates or feedback you could share.
It seems that this issue and a new issue we received at cmsis-svd highlight that this is a real concern for the community. Addressing it would likely benefit a lot of users.


.bs domains

Problem
There's no whois server registered to the IANA Route Database Zone for all .bs domains.
Conclusion
That makes all .bs domains fails.

Please report to http://www.nic.bs/cgi-bin/search.pl for whois lookup


XHTML pages with  fail to load in Opera

Steps zur reproduce

Deliver the page specified as example code below with Content-Type application/xhtml+xml and place an image "red.png" next to it:
See that the image red.png is loaded in Chromium and in Firefox, but not in Opera.

Possible cause
In file booleanAttrs line 392 there is a special treatment for the src property for MSIE. Enabling this special treatment for Opera as well solves the problem.
Not that the spec for the setAttribute method simply says:

If an attribute with that name is already present in the element, its value is changed to be that of the value parameter

It does not mention anything about namespaces.
Looking at the DOM tree in Opera dragonfly reveals that Opera has actually set the src-Property correctly... It just overrode the ng:src property with the expected value.
Example code
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml" xmlns:ng="http://angularjs.org">
	<head>
		<script src="https://code.angularjs.org/1.3.0-beta.5/angular.js"></script>
		<script><![CDATA[
			angular.module('myApp', [])
				.controller('MyController', ['$scope', function ($scope) {
				}]);

			angular.element(document).ready(function() {
				angular.bootstrap(document, ['myApp']);
			});
			]]></script>
  </head>

  <body>
		red.png: <img ng:src="{{'red.png'}}" />
	</body>
</html>

Old Opera is not supported.


Simplify the API.

Remove static helper methods (RazorService) and give direct access to the instance instead (now Engine.Razor).

How to now? please?


Context to a conversation in PLATO-2

Is it possible to give a context to the conversation, so that the setting description/persona can be given to the pretrained models beforehand?
If not, is there a possibility of incorporating something like this with the pretrained models?

This is an excellent question, indeed. Currently, our pre-trained PLATO-2 model is not trained on Knowledge Grounded Dialogues. However, it is possible to put some persona in the context to make the model to have a certain persona/knowledge ground, etc. We've tested some of the cases and it can work well.
Just as being stated by GPT-3, language models can be few-shot learners. It might be interesting to incorporate some of those corpora (knowledge grounded dialogue, persona grounded conversation) during the pretraining stage. Unfortunately, most knowledge grounded corpora are rather small, which is more suitable in the fine-tuning stage. We are looking forward to more progress in this problem

Great, thanks for your response!
Could you please let me know where should I make the changes in the scripts/code to enable some persona and knowledge backgrounds during inference?

You can change interaction.py:
https://github.com/PaddlePaddle/Knover/blob/15d5279a4370b225b0c388a129b774c9469fcde4/interaction.py#L69
personas = ["your persona: i have three cats."]
example = Example(src=" [SEP] ".join(personas + context), data_id=0)

You can specify the personas as you want.


Drop or adapt tutorials that rely on Katacoda

Prompted by https://github.com/kubernetes/website/issues/33936
Follows https://github.com/kubernetes/website/pull/40292
Remove pages where we don't have a replacement online, interactive tutorial mechanism, changing them to an apology message
(aside: we don't have an easy way to serve 410 Gone responses)
Update pages where we have an way to let people run the tutorial locally.
/hold
We should not make this change until the end of March 2023.

I'm keen to get this approved and LGTMed (and held), so we are ready for the end of the month.

LGTM

/lgtm

/approve

/label tide/merge-method-squash
/lgtm

I reckon it's about time to ship this though.
/hold cancel

In the end, we adapted most of these tutorials for people to run them locally.


Traffic graph widget filter checkboxes

This refactors the Traffic Graphs Widget so that it has checkboxes for selecting the graphs to display, and so that the settings are stored in the "standard" format that is used by other widgets.

This sits on top of #3654 , which could be merged first.

This will [should] easily port back to 2.3.*


def_node_matcher for explicit and implicit nil

I have created a custom cop that raises violations on
_(data).must_equal nil
def_node_matcher :must_equal_nil?, <<~PATTERN     (send _ :must_equal (nil))  PATTERN
I have tried different approaches, how do I detect the node if the code has an expression/variable that is nil
value = nil  _(data).must_equal value
Any tips on how I can make this work? Or a cop I can look at for inspiration?

This is one way I could do it but the problem here is that it fails on every type of value and not just nil.
def_node_matcher :must_equal_nil?, <<~PATTERN
            {
              (send _ :must_equal (lvar _))
              (send _ :must_equal (nil))
          }
        PATTERN

Any way to just make it work for nil values?

I've tried several solutions, but none have worked so far. It's possible that we're dealing with a static analyzer and cannot obtain the value beforehand. I'm hoping someone from the Rubocop family can provide additional guidance.

AFAIK this is not possible with RuboCop, as it's not evaluating any code. At most, you could detect if value is a local variable, and if that local variable has been assigned the literal nil, but I would advise against that, as it doesn't seem very useful. If you must know if the method is being called with nil values, the only way to really know is to check at runtime. Another approach would be to use a type checker like Sorbet.

Yes, I have used the sorbet in some other context, which bulk up the project and I can't convince everyone to opt in for that in this particular case!
Thanks for the input

RuboCop is fundamentally a static analysis tool for single files, and therefore, it cannot perform analysis across multiple files. As already mentioned, it is impossible to determine if any case, other than nil literals and local variables, is nil. So, it might be possible to ascertain the value of local variables by traversing for nodes where the value is assigned.


Contribute 'New Java Project...' command to File/NewFile and File Explorer menus

Record the UI entry that triggers the "new java project" command.

Contribute "New Java Project..." command to the top "File -> New File..." menus.

Contribute "New Java Project..." command to the context menu of File Explorer.

yes, I will merge it first. Will use a separate PR to fix the ui test.


Use different sync engine depending on reference.

I ran some tests comparing how alass and ffsubsync perform using various options. What I noticed was that alass more consistently output correctly synced subs when syncing against a reference subtitle, while ´ffsubsync´ performed better than ´alass´ when syncing against audio.
Given that the script already supports both engines, I think it could be convenient if it was possible to allow for configuring the script to use ´alass´ when syncing against a subtitle and ´ffsubsync´ when syncing against audio. Alternatively, the script could simply ask for which engine to use when run, but I can imagine that would clutter up the UI and be annoying for people who want to configure once and be done with it.

I've implemented a menu that allows the user to choose between engines right before synchronizing. See this branch.

I've made the v0.32 branch the main branch of the project so that people will be redirected there instead of master, which is only for deprecated versions of mpv. I hope this makes it easier for issues and suggestions to go towards the branch in active development.

Great, thank you! Would be nice if the v0.33 branch was the default though because mpv v0.33 was released quite a few months ago.

Looks good! Thanks for the shoutout on your announcement page too! I was having some issues with the update initially because I simply replaced autosubsync.lua in the root of the script folder with the new version, but it clicked once I read the example git command properly. Still, it may be worth explicitly telling people to place both .lua files in a subfolder.
I could also see the new functionality having good synergy with being able to set a preferred engine per method as it would allow for very flexible configuration, such as:
audio_sync_tool=ask
internal_sub_sync_tool=alass
external_sub_sync_tool=ffsubsync    <= If implemented in the future.

There are basically two ways to install any mpv add-on. The first one is to place a *lua file to ~/.config/mpv/scripts. This will only work if the add-on is one file. The second one is to place a folder to ~/.config/mpv/scripts. The folder should have a file named main.lua for this method to work.

I see now that this stated in the mpv documentation. It was my first time seeing an mpv add-on consisting of multiple files, didn't realize it was standard practice. My mistake.


update issue templates and add new documentation issue template

Fixes Issue

#2353

Description
This PR includes updates for issue templates :

BUG-REPORT
FEATURE-REQUEST
DOCUMENTATION
HELP

Checklist

[x] Does npm test pass with this change (including linting)?
[x] Is the new or changed code fully tested?
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)? --> This PR doesn't do any modification to documentation

@agung2001 thanks for contributing! I'll review it soon


Feature request: ability to turn off repeated view tracking

Feature request:
Currently, Nylas tracks subsequent opens, not just the first open.
The problem I find is that many people will open your email multiple times that day, and often again the next day and the next day and so on. These are likely all the people who don't use the "inbox zero" approach and are constantly sifting through their email every day and previewing your email for a fraction of a second, which triggers the pixel and registers as another open.
The result is that my dropdown box in Nylas tells me when someone has opened this email every time. I might see over 100 notifications each day in Nylas, but 90% of them are the same old people re-opening old emails again. Which is a big waste of notifications. This might be of interest to some Nylas users, but I'm personally only interested in whether the recipient reads the email once.
So my feature request is a checkbox in Preferences to turn off repeated view tracking. That would be great!
Thanks for your time.
Nathan

We received another vote for this issue in the support channel today.

What I wanna know is why it counts opens from myself too. . . i.e. if I open the mail I sent in Nylas, it counts as an open in view tracking.

@ajcollett There is a new issue regarding it #3249.


First config add for webpack release and dev server

🔴 NOT READY FOR MERGING YET 🔴
ISSUE 👉 https://github.com/Jam3/generator-jam3/issues/331
This pull request contains all the configuration to run our projects under webpack, so far it's making half of our scripts obsolete and bundles are smaller by default thanks to scope hoisting, tree shaking and better modules overhead.
TODO:

Test with BigWheel project
Add support for LESS styles
Discuss ASSETS_PATH variable use

@jeremenichelli 👍  respect
I will def take a look on Monday.
but a little thought: I don't think we need to support bigwheel and less, at least for now. I personally don't think we are using them for most of the projects.

The other thing is the team might want to use it as an option for now? Instead of replacing budo? I don't know...

@neo I think that adding LESS support is super easy so that wouldn't be a problem. DOn't know the way we want to go on BIg Wheel but I wouldn't kill it under this PR.
Mainting budo and webpack on the same template is possible I think though a bit more complicated ot me since I don't have much experience doing heavy modifications to the generator but if that's the way the team wanna go I will change the PR for sure, just let me know :)

@jeremenichelli I'm not saying to kill them, I am just saying we don't have to support them at least for now.
And for sure, I know it's not gonna be easy. It will be a long way to go. But this is def a great first step! 👍

Just one thought: should we just use https://github.com/facebookincubator/create-react-app?

@neo it could be useful for some projects, the problem is that is hard to scalate the config I heard. So if you want to create specific vendor files or add some complex loader config you might have problems, and that happens really often in Jam3 projects.

@jeremenichelli You are right, that was just a thought anyway. We always use custom builds, but what we could do is refer to how they do it I guess and get something useful out of it.


Regression from #2078: Event can be null when Sneak navigation is used

I broke navigation with sneak mode with #2078, because the onClick handled is called without arguments

Ah, thanks for catching this @DeltaEvo ! 👍 Fix looks great!


GO:0018399 referred output obsoleted in ChEBI

GO:0018399 refers to:
intersection_of: GO:0018075 ! peptidyl-phenylalanine bromination
intersection_of: has_output CHEBI:21212 ! L-4'-bromophenylalanine

The compound is no longer to be found in ChEBI.

Hold on, this may be only a problem with their web version...

Sorry, the chebi.obo from Nov-01 actually has the entry, so not your problem.


init: LogException:23 FS: Could not start file system server

Please use the following bug reporting template to help produce issues which are actionable and reproducible, including all command-line steps necessary to induce the failure condition. Please fill out all the fields! Issues with missing or incomplete issue templates will be closed.
If you have a feature request, please post to the UserVoice.
If this is a console issue (a problem with layout, rendering, colors, etc.), please post to the console issue tracker.
Important: Do not open GitHub issues for Windows crashes (BSODs) or security issues.  Please direct all Windows crashes and security issues to<EMAIL_ADDRESS> Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".
Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.18362.86]
Hardware is a Lenovo Yoga C630 (Windows 10 on Arm, 8-core Snapdragon arm64 cpu).
Running WSL and Ubuntu 18.04.2 LTS.

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

% dmesg
[    0.021742]  Microsoft 4.4.0-18362.53-Microsoft 4.4.35
[    0.453819] <3>init: (1) ERROR: LogException:23: FS: Could not start file system server. Invalid argument @.\plan9.cpp:88 (CreateServerSocket)

What's wrong / what should be happening instead:

file system server should load without error.

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

No trace available.

For WSL launch issues, please collect detailed logs.

Happy to collect more logs if conditions require it.
See our contributing instructions for assistance.

I see the same on same model machine, updated today to 1903 (18362.145) with a freshly installed & updated Ubuntu WSL instance.
Without the file server, it's impossible to launch a GUI editor for files on the Linux filesystem; running something like code . or notepad.exe Cargo.toml simply never launches anything (strace gist). (Workaround: place files on Windows filesystem.)

For me, it cleared up after upgrading to the Fast ring build 18917; file server seems to work ok there.

i have the exact same hardware i.e.  Lenovo Yoga C630 (Windows 10 on Arm, 8-core Snapdragon arm64 cpu). and the latest Windows 10 insider build, Windows 10 Insider Preview 18932.1000 (rs_prerelease), fixe this issue for me

Same problem in HUAWEI MateBook E 2019(Windows 10 on ARM,  8-core Snapdragon 850, arm64), with build version 18363.476.

I have the same Problem with current version
10.0.18363 Build 18363 on a
LENOVO_MT_81F1_BU_idea_FM_Miix 630
(also an ARM machine).
Logs files are attached as desired.
lxcore.zip


Update to wapi.js resolves #776

A small update to the deleteMessage function

Actually, ignore this for now. It does not delete the message on the device. It just deletes it from the chat within that session. Still needs some work.

Ok now it should be working.


Window resize issue - When resizing it shifts to the right

On a big screen (Full HD) resolution, when resizing the black window with the console goes to the right.
You have to resize the window to a smaller size so that it goes again to the left.
It is pretty annoying with you maximize the window! You cannot see anything.

I ran into the same issue, exactly.  I use a high res HP ZDisplay at work, running at 2560x1660.  The bug has to do with the vertical scaling of SuperPuTTy.  If I have SuperPutty open with a large vertical footprint, say 150 - 200 lines for argument's sake, the active window behaves exactly like this when one tries to resize the width, for example.  However, if I decrease the vertical footprint to say ~ 100 lines, I can increase or decrease the width of the active PuTTy window without incident.
One can grab and drag the bottom right-hand corner of the SuperPuTTy window to resize the length and width of the active window.  This allows one to easily see when the issue occurs, and when it does not.
Hope this makes sense.


Fix out-of-bounds issue in getParameterNameAtPosition

Fixes #30171.

Started with wrong base branch. Closing this PR.


16.04 unity tweak tool

Hi.
Since I see you are preparing for 16.04, I found another small bug, no big deal really, it's just a bit annoying for my semi OCD ;)

It seems to work well with arc (https://github.com/horst3180/arc-theme) and numix (https://github.com/shimmerproject/Numix) in 16.04, so it should be possible to fix.
Have a nice day!

Fixed in #51


Tilt fails to apply namespace first

A user tried to load yaml containing both namespaces and objects declared in those namespaces.
Theoretically, tilt applies the namespaces first, so things work out.
For some reason, in this case, the objects declared in those namespaces are applied first, and yield an error because the namespaces have not yet been created. If you re-run tilt, the namespaces still exist, and things work fine.
Repro:
Save this into gist.yaml
make a Tiltfile that just has k8s_yaml('gist.yaml')
tilt up
expected:
the resources come up successfully
observed:
rook-ceph-operator fails to apply with:
       ERROR: kubectl apply:
       stderr: Error from server (NotFound): error when creating "STDIN":
       namespaces "rook-ceph-system" not found

(note, if reproing multiple times, you'll need to tilt down between attempts, to ensure the namespace gets deleted)
The same problem probably applies to CRDs.

When applying global yaml, tilt does sort by namespace. I think the issue here is that rook-ceph-operator is in a normal resource, not global yaml, and global yaml application happens in parallel with resource builds, so there's a race.
(also, tilt makes no attempt to apply CRDs first, and it should)

@landism
a. do you have a sense of how much work this'd be to fix?
b. does it seem that just removing the concept of global yaml (which we've been meaning to do for a while anyway) would address this issue?

do you have a sense of how much work this'd be to fix?

Nope! I'd lean towards applying global yaml synchronously and always making it higher priority than other resources, but I haven't really looked into what's involved in that (global yaml is currently handled via a separate controller, so there miiight be some complication to synchronize them?)

does it seem that just removing the concept of global yaml (which we've been meaning to do for a while anyway) would address this issue?

I've heard mention of this but I'm not entirely sure what it means in practice. We've got k8s objects (e.g., namespaces) that aren't really associated with tilt resources, and we need to apply them to the cluster at some point. Whether that's as part of a tilt resource or not, we still need code to take those non-tilt-resource objects (which one might refer to as "global yaml" or something else) and apply them to k8s at some point, and that seems unrelated to the tilt-resourciness of it.