Add parsing of nested Dictionaries/Arrays

Add new array2fields function that formats Arrays for Firestore
correctly.
Change dict2fields function to use array2fields on Arrays.
Both array2fields and dict2fields recursively call one another to
parse the entire object.
Return new object from dict2fields instead of modifying object passed
in, so that calling it twice on the same object doesn't break it.

Hi @timjrobinson ,
Is there a specific reason you would like to implement the array2fields method?
Are you having issues creating a dictionary of fields? Or is it limiting your flow in any way?

Hi @timjrobinson ,
Is there a specific reason you would like to implement the array2fields method?
Are you having issues creating a dictionary of fields? Or is it limiting your flow in any way?

@fenix-hub Just updated with test item that wasn't saving to Firestore. The new code saves it out correctly.

@fenix-hub Just updated with test item that wasn't saving to Firestore. The new code saves it out correctly.

@timjrobinson Maybe old changes broke compatibilty with Arrays. Thank you very much for your PR. It will be merged together with a little update with the Authentication module, later today.
Always feel free to contribute, but please always mention what is the issue you are encountering that your PRs will resolve, so we will be able to test them too and merge them faster. Thank you! :)

@timjrobinson Maybe old changes broke compatibilty with Arrays. Thank you very much for your PR. It will be merged together with a little update with the Authentication module, later today.
Always feel free to contribute, but please always mention what is the issue you are encountering that your PRs will resolve, so we will be able to test them too and merge them faster. Thank you! :)

Sure will do, thanks!

Sure will do, thanks!


Port remaining fixes from old branch

Looks like, with the exception of some transient image load timeouts, the CI tests passed, so I'm merging this.


Feature/extend cmd to mimic kafka

What
Extended the cmd functionality so that we can pass in a new published file path and update the sitemap.
How to review
check changes work as expected.

cd cmd/sitemapcli
go run main.go fake_scroll.go -robots-file-path=test_robots.txt -scroll-size=10 -scroll-timeout=2000 -api-url=http://localhost -sitemap-index=1 -zebedee-url=http://localhost:8082 -sitemap-file-path=test_sitemap -enable-fake-scroll=true

Test passes.
Who can review
anyone

As a general point - the linter is not working on cmd/sitemapcli/main.go - I would advise turning it on.


Upgrade pulumi-package-publisher version

Add version input with value fetched from the new provider-version-aciton.
Related to:

https://github.com/pulumi/pulumi/issues/16086

Deploying directly to pulumi-random: https://github.com/pulumi/pulumi-random/pull/908


Add txt file

@anar-shaim Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


[WIP][RFC] Introduce JavaScript null value

I believe that having Sass.NULL (and Sass.TRUE,
Sass.FALSE) is incorrect. We should be using
real JavaScript primitives instead, if only
to avoid object identity issues, or to allow
natural constructs like:
done(a === b)

I suggest assigning JavaScript 'null' to
sass.NULL and sass.types.Null.NULL names.
Current stage:

[x] handling primitive null values
[x] handling null items in list
[ ] handling null keys and values in maps
[ ] "a instanceof sass.types.Null" does not work

An attempt to instantiate sass.NULL object via:
 new sass.types.Null()

gives a different error message now:
 TypeError: sass.types.Null is not a function

I don't think that "null instanceof sass.types.Null" could be made true, I'd rather modify our test instead (not to depend on the implementation internals).

I'm not convinced this is a good idea discussed briefly in https://github.com/sass/node-sass/issues/1088#issuecomment-166467651
I would hold off on this until we reach a consensus on https://github.com/sass/node-sass/issues/1320


java.lang.NoSuchMethodError: org.apache.commons.lang3.StringUtils.countMatches(Ljava/lang/CharSequence;C)I

Hi. I am trying to get this working with spigot and this is giving this strange error:
java.lang.NoSuchMethodError: org.apache.commons.lang3.StringUtils.countMatches(Ljava/lang/CharSequence;C)I
        at net.dv8tion.jda.core.requests.Route.<init>(Route.java:162) ~[?:?]
        at net.dv8tion.jda.core.requests.Route.<init>(Route.java:33) ~[?:?]
        at net.dv8tion.jda.core.requests.Route$Self.<clinit>(Route.java:37) ~[?:?]
        at net.dv8tion.jda.core.entities.impl.JDAImpl.verifyToken(JDAImpl.java:147) ~[?:?]
        at net.dv8tion.jda.core.entities.impl.JDAImpl.login(JDAImpl.java:105) ~[?:?]
        at net.dv8tion.jda.core.JDABuilder.buildAsync(JDABuilder.java:355) ~[?:?]
        at net.dv8tion.jda.core.JDABuilder.buildBlocking(JDABuilder.java:378) ~[?:?]
        at org.supertassu.tyhjyydensyoma.api.discord.Discord.<init>(Discord.java:24) ~[?:?]
        at org.supertassu.tyhjyydensyoma.api.TyhjyydensyomaAPI.onEnable(TyhjyydensyomaAPI.java:25) ~[?:?]
        at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:292) ~[spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:337) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:405) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at org.bukkit.craftbukkit.v1_10_R1.CraftServer.enablePlugin(CraftServer.java:373) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at org.bukkit.craftbukkit.v1_10_R1.CraftServer.enablePlugins(CraftServer.java:323) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at net.minecraft.server.v1_10_R1.MinecraftServer.t(MinecraftServer.java:412) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at net.minecraft.server.v1_10_R1.MinecraftServer.l(MinecraftServer.java:377) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at net.minecraft.server.v1_10_R1.MinecraftServer.a(MinecraftServer.java:332) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at net.minecraft.server.v1_10_R1.DedicatedServer.init(DedicatedServer.java:271) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at net.minecraft.server.v1_10_R1.MinecraftServer.run(MinecraftServer.java:535) [spigot-1.10.2.jar:git-Spigot-72c2605-33071ec]
        at java.lang.Thread.run(Unknown Source) [?:1.8.0_111]

My POM.xml: http://pastebin.com/S65JB5aC
My code: http://pastebin.com/2PZpdGAt

You have a conflicting version of apache lang3 installed. I imagine that spigot comes with a version of it as well

Oh. Spigot comes with 2.5.

Wow.. that's out of date. JDA uses 3.5

As this isn't a JDA issue, I am closing it.

This actually is an issue, because char doesn't equal char sequence. You use something like countMatches(route, '{'). I downloaded the source and changed the ' to " and everything works perfectly because a String is a Charsequence, but a single char is not. The method countMatches requires two charsequences.

Version 3.4 introduced a method that allows to provide a character as second parameter.
Version 3.0 introduced a method that allows two CharSequence parameters.
Both are available in the version that JDA uses 3.5.

@DV8FromTheWorld Could someone possibly add some sort of solution to this, this issue was brought to me by someone who was using my dependency library, and them having to shade lang3 by using relocation sort of defeats the purpose, can we get some backwards compatibility?

There is a solution. When shading move the commons package that JDA uses.

Shading is the solution.
What you're asking from me is the same as if you were to ask spigot to update from 2.5 to 3.5. They would tell you to shade.
It is not JDA's responsibility to be interopt with another program, especially due to them using an out of date version of an extremely old lib. Both situations just call for shading.

Yes. Mojang does. Does it matter? No. Just relocate JDAs commons Lang when shading your plugin. It isn't hard. And there is absolutely no reason for the dependency to be outdated by JDA. It would make more problems than solutions. Very small part of the user are is on spigot, and changing the version of that fundamental library is a waste of time. As said before just shade and relocate the one uses by JDA. I think the discussion right now is pointless. That's the solution.. Or make Mojang update, but good luck with that.

I was only saying that for clarification and to show how improbable it would be for us to update it. Also I'm not asking for you guys to downgrade it..

In lang3 3.5, there's 2 methods for countMatches, one of them is new, and the other one is older. By using the older one you would retain backwards compatibility while also retaining intended functionality. It's a very simple solution to an issue blown way out of proportion and  all I'm asking is to have it considered.
For clarification:
countMatches(CharSequence, Char) is the one currently in use by JDA
countMatches(CharSequence, CharSequence) is the one I'm asking to be used instead, as it is in all of the versions, and not just the newer one

JDA will not downgrade to an older version. Shading your versions is something that you have to do.
Period.
We know about this change, I even mentioned it above in this very issue.
We do not intend to change the method. If you want to use JDA you will have to properly shade your dependencies, this issue is only caused by colliding dependencies and just because JDA uses the older method doesn't mean it resolves the main issue.
Your issue is not caused by JDA it is caused by your setup.

In addition
StringUtils.countMatches is not the only method from 3.5 that JDA is using.

The 2 methods being used that would conflict are

countMatches
truncate

the former being an easy fix, and the latter can simply be replaced with subString...
Also, is the entire lang3 library being shaded in JDA for only 6-7 methods and 2 classes?
@MinnDevelopment @DV8FromTheWorld

@Sxtanna if you don't want to shade a whole lib than just shade the affected classes.

So here is a bigger thing that you're missing: you're asking us to use the 2.5 method because the 3.5 method doesn't exist in the already included 2.5 version with MC, but the fact that it complains that the 3.5 method doesn't exist is because NONE of the 3.5 is loaded.
We use 3.5 in other places in JDA, thus it not being loaded will break JDA.
I've detailed why this happens here: http://stackoverflow.com/a/41629406/3813641
Yes, this is for MinecraftForge, but the exact same concept exists for Spigot.
The first use of 3.5 is encountered with countMatches, so that is the perceived point of failure, but it is a larger problem.
I do find it curious: why do you not want to shade? We've brought it up multiple times and each time you have skipped past it and pushed for us to change our usage. What do you not want to shade?

I know that shading and relocating the needed classes is a fix, however its overkill.
https://gist.github.com/Sxtanna/a9b67bd2b5b663de73a00fdf648013a3
Here I've shown everywhere lang3 is used in JDA, the only 2 things that are in 3.5 that aren't in 2.5 are

countMatches(CharSequence, Char)
and
truncate(String, int)

Which are both in StringUtils, its the only class that is different (in this case)
Countmatches is used once, and truncate is used twice, replacing countMatches with the one that will allow backwards compatibility would change nothing functionality wise, and replacing the two truncate methods (which in their usages do basically the same thing and should be pulled into a method of its own anyway), would be a simple refactor.
I also find it curious how backwards compatibility isn't something that's a tleast considered, and just thrown out for the reason of "just shade it yourself"? @DV8FromTheWorld

Additionally, we use lang3 for

Pair
MutablePair
ExceptionUtils
StringUtils

countMatches
truncate
equals
equalsIgnoreCase

But that really should matter. Why we use a lib or dont has nothing to do with this conversation.

Everything there is listed in the gist I posted. And you're right, I never question why you were using lang3, I asked if JDA was shading the entire thing, or just the needed classes?

JDA isn't shading anything

I was on mobile and your post wasn't on my screen when I typed my list.
It has nothing to do with 'not supporting backwards compatibility', the factor the matter is what if we decide to start using something that DOESNT exist in 2.5? What will you do then?
I get wanting backwards compatibility, but it isn't always best to bend yourself for it. This is such a case. Shading is not overkill. Why do you believe it is overkill? What reason do you have for not shading? Your plugin will have to have JDA in it anyways.

The point is you should setup a system that will work no matter what happens. Shading will safeguard against future changes in JDA using other features, not only with lang3, but also any other dependencies that we might end up using.

What do you mean by "bend yourself for it"?
Also the reason in this case is because distributing large jars with all dependencies shaded is usually not great xD. So I made a util that will download and load them at runtime. I honestly didn't even know JDA was a thing before someone told me it was incompatible.
With the Util, the plugin will have nothing in it, but its own compile output. JDA will be downloaded, and all dependencies will be downloaded aswell.

why don't you just add support for relocating dependencies to your dependency-loader then?

It doesn't modify files. It's supposed to be simple and as small as possible.

Do you plan to contact every incompatible lib dev team to downgrade dependencies for your dependency loader?

I never proposed downgrading anything actually... Only to consider backwards compatibility when creating APIs.

which is essentially the same...

How is that the same?

where is the difference between using an old version and using a new version but only the old features of it?

In this specific case the only new feature you're using is #truncate which is literally just a wrapper for subString. So the least that can be done is to use the method that is going to be in all versions...

that doesn't answer my question though...

Your question was insinuating something that isn't pertinent in this specific case, so I chose to explain what we're dealing with rather than what you're trying to imply.

Sorry to bring this up again but I would like to know how I can move or relocate the dependency.
Having the same issue but don't know how to fix it.

@TechXcrafter Use class relocation.
https://maven.apache.org/plugins/maven-shade-plugin/examples/class-relocation.html

Currently my pom.xml looks like this:
`

4.0.0
<groupId>me.TechXcrafter</groupId>
<artifactId>DiscordBridge</artifactId>
<version>1.0</version>

<repositories>
    <repository>
        <id>spigot-repo</id>
        <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
    </repository>
    <repository>
        <id>jcenter</id>
        <name>jcenter-bintray</name>
        <url>http://jcenter.bintray.com</url>
    </repository>
</repositories>
<dependencies>
    <!--Spigot API-->
    <dependency>
        <groupId>org.spigotmc</groupId>
        <artifactId>spigot-api</artifactId>
        <version>1.11.2-R0.1-SNAPSHOT</version>
        <scope>provided</scope>
    </dependency>
    <!--Bukkit API-->
    <dependency>
        <groupId>org.bukkit</groupId>
        <artifactId>bukkit</artifactId>
        <version>1.11.2-R0.1-SNAPSHOT</version>
        <scope>provided</scope>
    </dependency>
    <dependency>
        <groupId>me.TechXcrafter</groupId>
        <artifactId>TechPluginLibrary</artifactId>
        <version>1.0</version>
    </dependency>
    <dependency>
        <groupId>net.lingala.zip4j</groupId>
        <artifactId>zip4j</artifactId>
        <version>1.3.2</version>
    </dependency>

    <dependency>
        <groupId>org.apache.commons</groupId>
        <artifactId>commons-compress</artifactId>
        <version>1.5</version>
    </dependency>

    <dependency>
        <groupId>org.apache.commons</groupId>
        <artifactId>commons-lang3</artifactId>
        <version>3.0.1</version>
    </dependency>

    <dependency>
        <groupId>net.dv8tion</groupId>
        <artifactId>JDA</artifactId>
        <version>3.0.0_191</version>
    </dependency>

</dependencies>

<build>
    <resources>
        <resource>
            <directory>src/main/resources</directory>
            <filtering>true</filtering>
        </resource>
    </resources>
    <plugins>
        <plugin>
            <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-compiler-plugin</artifactId>
            <configuration>
                <source>1.8</source>
                <target>1.8</target>
            </configuration>
        </plugin>
    </plugins>
</build>

`
So and I need to add this part right:
<executions>           <execution>             <phase>package</phase>             <goals>               <goal>shade</goal>             </goals>             <configuration>               <relocations>                 <relocation>                   <pattern>org.codehaus.plexus.util</pattern>                   <shadedPattern>org.shaded.plexus.util</shadedPattern>                   <excludes>                     <exclude>org.codehaus.plexus.util.xml.Xpp3Dom</exclude>                     <exclude>org.codehaus.plexus.util.xml.pull.*</exclude>                   </excludes>                 </relocation>               </relocations>             </configuration>           </execution>         </executions>
But what do I need to put in these fields?
Btw thank you for this fast answer

Now my pom.xml looks like this. Still not working.
`

4.0.0
<groupId>me.TechXcrafter</groupId>
<artifactId>DiscordBridge</artifactId>
<version>1.0</version>

<repositories>
    <repository>
        <id>spigot-repo</id>
        <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
    </repository>
    <repository>
        <id>jcenter</id>
        <name>jcenter-bintray</name>
        <url>http://jcenter.bintray.com</url>
    </repository>
</repositories>
<dependencies>
    <!--Spigot API-->
    <dependency>
        <groupId>org.spigotmc</groupId>
        <artifactId>spigot-api</artifactId>
        <version>1.11.2-R0.1-SNAPSHOT</version>
        <scope>provided</scope>
    </dependency>
    <!--Bukkit API-->
    <dependency>
        <groupId>org.bukkit</groupId>
        <artifactId>bukkit</artifactId>
        <version>1.11.2-R0.1-SNAPSHOT</version>
        <scope>provided</scope>
    </dependency>
    <dependency>
        <groupId>me.TechXcrafter</groupId>
        <artifactId>TechPluginLibrary</artifactId>
        <version>1.0</version>
    </dependency>
    <dependency>
        <groupId>net.lingala.zip4j</groupId>
        <artifactId>zip4j</artifactId>
        <version>1.3.2</version>
    </dependency>

    <dependency>
        <groupId>org.apache.commons</groupId>
        <artifactId>commons-compress</artifactId>
        <version>1.5</version>
    </dependency>

    <dependency>
        <groupId>org.apache.commons</groupId>
        <artifactId>commons-lang3</artifactId>
        <version>3.0.1</version>
    </dependency>

    <dependency>
        <groupId>net.dv8tion</groupId>
        <artifactId>JDA</artifactId>
        <version>3.0.0_191</version>
    </dependency>

</dependencies>

<build>
    <resources>
        <resource>
            <directory>src/main/resources</directory>
            <filtering>true</filtering>
        </resource>
    </resources>
    <plugins>
        <plugin>
            <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-compiler-plugin</artifactId>
            <configuration>
                <source>1.8</source>
                <target>1.8</target>
            </configuration>
        </plugin>

        <plugin>
            <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-shade-plugin</artifactId>
            <version>3.0.0</version>
            <executions>
                <execution>
                    <phase>package</phase>
                    <goals>
                        <goal>shade</goal>
                    </goals>
                    <configuration>
                        <relocations>
                            <relocation>
                                <pattern>org.apache.commons.lang</pattern>
                                <shadedPattern>org.shaded.commons.lang</shadedPattern>
                            </relocation>
                        </relocations>
                    </configuration>
                </execution>
            </executions>
        </plugin>
    </plugins>
</build>

`
Thanks for helping

JDA no longer uses lang3 as of 2d19f09 which should resolve these dependency issues.


Cairo module

So this one was a pain, but it works and provides both cairo and cairo::pycairo
mattheath/puppet-cairo

@rafaelfranca @mattheath can this be merged in or closed?

Seems good to merge

Hi @mattheath, I can't get your tests to run successfully. Can you update your spec Puppetfile with appropriate version updates? At the very least you'll need:
mod 'boxen', '3.6.1', :github_tarball => 'boxen/puppet-boxen'
mod 'stdlib', '4.1.0', :github_tarball => "puppetlabs/puppetlabs-stdlib"

Sorry it's taken so long to respond to this.

Ok, not sure when I'll get around to it, but will update when I can. Needless to say the versions in that file are from when I submitted the PR so likely rather out of date :wink:

Closing due to this issue being stale. If you can still reproduce or would like to continue this discussion, please let me know and I'll reopen this.


how to enable log fields with envoy jaeger-native-tracing

Hi
there are some naming conventions for span tags and log fields
https://github.com/opentracing/specification/blob/master/semantic_conventions.md
but in the example on tracing there is no glue to add log fields or span tags
https://github.com/envoyproxy/envoy/tree/master/examples/jaeger-native-tracing
https://github.com/envoyproxy/envoy/blob/master/examples/front-proxy/service.py
how to add more log fields or span tags
thank you

Do you mean add log/tags from within the Envoy proxy (when handling your requests) or from within the service that is being proxied?

@objectiser i mean add log/tag from within the service
please help
thank you

Which language are you using for your service? You need to instrument your app with the appropriate OpenTracing API and Jaeger tracer.

i use python https://github.com/jaegertracing/jaeger-client-python,
just using sample
    with tracer.start_span('TestSpan') as span:
        span.log_kv({'event': 'test message', 'life': 42})

        with tracer.start_span('ChildSpan', child_of=span) as child_span:
            span.log_kv({'event': 'down below'})

because envoy auto generate_request_id: true
but i don't know how integrated with envoy jaeger-native-tracing
do i need to create new span from within service.py ??
thank you

You don't necessarily need to create a span within the service - only if you want to. The important thing is to propagate the context, which can be achieved using the extract and then inject functions.

thank you, got it
it should be update the example service.py :)

@alochym01 Unfortunately the service service.py is shared across multiple examples which use different tracers. If you can think of a way to implement that supports the different examples, feel free to submit a PR :)


Support uploading binary files in deploy API

From Slack:

Using PUT /application/v2/tenant/default/session/[session-id]/content/[path] endpoint, the file content got changed.  It looks like the binary content is read into a string and persisted in a file, the md5 hash is also based on the string content, which is not correct.

Also:

When I use the prepareandactivate endpoint to upload the zip file. The binary files included in the zip file is persisted to file correctly. However, the md5 hash is still calculated based on the string content, which is not correct.

Implemented in https://github.com/vespa-engine/vespa/pull/32055, released in 8.382.22


Difference in count of available tests

Please have a look at the Flash video referenced below.
It shows a difference in the number of tests available when I create a new Session.
http://screencast.com/t/KC9WKrCfV

Are the missing tests theories?

Nope. In the whole suite there is only one Theory. :-(

OK. Now this is weird....
http://screencast.com/t/yAZotrDvC

I believe this is a ReSharper issue rather than the plugin - at least, there has been a bug with a very similar description fixed in the latest 10.1 EAPs. However, xunit isn't ready for EAP yet. It will likely be EAP5.

Closing as not reproducible. If there are still problems, please file an issue with ReSharper (as the xunit plugin is now a part of ReSharper)


Error code 2010 when buying a coin

Good day, every time I try to buy a coin I get the following error message:
APIError(code=-2010): Account has insufficient balance for requested action
I am just trying to open an order of 25 Dollars, when in the balance I have over 60.
Is it a problem of the price changing between the request of the price and the placement of the order?
If that is the case I would suggest to replace this with a clearer message.
Here is the complete config file:
  # Switch between testnet and mainnet
  # Setting this to False will use REAL funds, use at your own risk
  TEST_MODE: FALSE
  LOG_TRADES: True
  LOG_FILE: 'trades.txt'

  # Set this to true if you are accessing binance from within the United States of America
  # Need to change TLD
  AMERICAN_USER: False

# These options apply to the trading methods the script executes
trading_options:

  # select your base currency to use for trading (trade for example USDT pairs)
  PAIR_WITH: USDT

  # Total amount per trade (your base currency balance must be at least MAX_COINS * QUANTITY)
  # Binance uses a minimum of 10 USDT per trade, add a bit extra to enable selling if the price drops.
  # Recommended: no less than 12 USDT. Suggested: 15 or more.
  QUANTITY: 20

  # List of trading pairs to exclude
  # by default we're excluding the most popular fiat pairs
  FIATS:
    - EURUSDT
    - GBPUSDT
    - JPYUSDT
    - USDUSDT
    - DOWN
    - UP

  # Maximum number of trade 'slots' at any time (your USDT balance must be at least MAX_COINS * QUANTITY)
  MAX_COINS: 2

  # the amount of time in MINUTES to calculate the difference from the current price (minimum: 1).
  TIME_DIFFERENCE: 1

  # Number of times to check for TP/SL during each TIME_DIFFERENCE (minimum: 1).
  # Don't spam the Binance API, you will be banned (max 1200 requests per minute per IP).
  RECHECK_INTERVAL: 6

  # the difference in % between the first and second checks for the price.
  CHANGE_IN_PRICE: 1

  # define in % when to sell a coin that's not making a profit.
  STOP_LOSS: 5

  # define in % when to take profit on a profitable coin.
  TAKE_PROFIT: 1

  # Use custom tickers.txt list for filtering pairs.
  CUSTOM_LIST: True

  # Name of custom tickers list
  TICKERS_LIST: 'tickers.txt'

  # whether to use trailing stop loss or not; default is True
  USE_TRAILING_STOP_LOSS: True

  # when hit TAKE_PROFIT, move STOP_LOSS to TRAILING_STOP_LOSS percentage points below TAKE_PROFIT hence locking in profit
  # when hit TAKE_PROFIT, move TAKE_PROFIT up by TRAILING_TAKE_PROFIT percentage points
  TRAILING_STOP_LOSS: .4
  TRAILING_TAKE_PROFIT: .1

  # Trading fee in % per trade.
  # If using 0.75% (using BNB for fees) you must have BNB in your account to cover trading fees.
  # If using BNB for fees, it MUST be enabled in your Binance 'Dashboard' page (checkbox).
  TRADING_FEE: .075

  SIGNALLING_MODULES:
    - pausebotmod
    - signalsamplemod```

Do you have some BNB for the fees?


Quick Question For  A Beginner

Hello, Can I just start off by saying thank you for such an incredible project.  I've created something similar but far less complicated. My question is this, Is it possible to call an image picker on view did load. For the demo that you provided in VIewController at view did load I called for your show method providing the needed specifications.
I end up getting an error <DKImagePickerControllerDemo.DKImagePickerController: 0x12582ee00> on <DKImagePickerControllerDemo.ViewController: 0x124e458f0> whose view is not in the window hierarchy!. Not sure where I am going wrong
override func viewDidLoad() {
super.viewDidLoad()
showImagePickerWithAssetType(
.AllAssets,
allowMultipleType: false,
sourceType: .Photo,
allowsLandscape: true,
singleSelect: false
)
}

Trying to call in viewDidAppear.

Great, i was actually able to accomplish by adding this to viewDidLoad pickerController.didSelectAssets = { (assets: [DKAsset]) in
print("didSelectAssets")
print(assets)
}
    pickerController.assetType =  .AllAssets
    pickerController.allowsLandscape = true
    pickerController.allowMultipleTypes = true
    ///pickerController.sourceType = .Photo
    pickerController.singleSelect = false
    //			pickerController.showsCancelButton = true
    //			pickerController.showsEmptyAlbums = false
    //			pickerController.defaultAssetGroup = PHAssetCollectionSubtype.SmartAlbumFavorites

    // Clear all the selected assets if you used the picker controller as a single instance.
    //			pickerController.defaultSelectedAssets = nil
    pickerController.defaultSelectedAssets = self.assets

    pickerController.didSelectAssets = { [unowned self] (assets: [DKAsset]) in
        print("didSelectAssets")

        self.assets = assets
        //self.previewView?.reloadData()
    }

    if UI_USER_INTERFACE_IDIOM() == .Pad {
        pickerController.modalPresentationStyle = .FormSheet;
    }

    self.presentViewController(pickerController, animated: true) {}.

One more question, what parameters need to be met in order to call for a separate camera instead of DKCamera

Currently don't support that. If you can provide a PR, that would be great :)

Let me know if you have any questions


Height not auto adjusted with the content

I am react-slick to show some carousal, but if first carousal content is more that height automatically set for other slides too.
Second every slides wrapping up inside a div, which I dont want.
Below setting I am using,
const settings = {
infinite: true,
speed: 500,
slidesToShow: 1,
slidesToScroll: 1,
arrows: false,
adaptiveHeight: false,
rows: 0,
responsive: [
{
breakpoint: 1024,
settings: {
slidesToShow: 1,
slidesToScroll: 1,
infinite: true,
},
},
{
breakpoint: 600,
settings: {
slidesToShow: 1,
slidesToScroll: 1,
infinite: true,
},
},
{
breakpoint: 480,
settings: {
slidesToShow: 1,
slidesToScroll: 1,
infinite: true,
},
},
],
};
What is the expected behaviour?
1.Height should be calculated for every slides based on content rather than set default height based on the first content
2. I want to remove that wrapping div which causing me UI issues. I tried to apply style to that particular wrapping div which did not work out. I tried below code,
.slick-slider .slick-active .slick-current > div{
height: 100% !important;
}
====================================================================
What is observed behaviour?
1 Height set default based on the first slide content.
2. Need to remove the wrapping div.

Have you tried setting the .slick-slide height equally?


riscv: Fix remaining asm checks

There are additional asm checks which don't check for OPENSSL_CPUID_OBJ causing the build to still fail on riscv [1], so fix them in the same manner as ff279597692f9f19dca5b147944d3d96f2e109f8
[1] https://bugs.gentoo.org/923956
Fixes: https://github.com/openssl/openssl/issues/22871

This should be fixed. But I also think Gentoo shouldn't be offering a no-asm build.

I've signed and sent the CLA via e-mail.

ping for second review

This pull request is ready to merge

Merged to the master, 3.3 and 3.2 branches. Thank you for your contribution.


Activating Open Collective

Hi, I'm making updates for Open Collective. Either you or a supporter signed this repo up for Open Collective. This pull request adds backers and sponsors from your Open Collective https://opencollective.com/jest ❤
It adds two badges at the top to show the latest number of backers and sponsors. It also adds placeholders so that the avatar/logo of new backers/sponsors can automatically be shown without having to update your README.md. [more info]. See how it looks on this repo.
We have also added a postinstall script to let people know after npm|yarn install that you are welcoming donations (optional). [More info]
You can also add a "Donate" button to your website and automatically show your backers and sponsors there with our widgets. Have a look here: https://opencollective.com/widgets
P.S: As with any pull request, feel free to comment or suggest changes. The only thing "required" are the placeholders on the README because we believe it's important to acknowledge the people in your community that are contributing (financially or with code!).
Thank you for your great contribution to the open source community. You are awesome! 🙌
And welcome to the open collective community! 😊
Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other open source sustainers!

Some screenshots:
Updated README Header

New Credits Section to README & CONTRIBUTING

New postinstall message

Codecov Report

Merging #6458 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6458   +/-   ##
=======================================
  Coverage   63.85%   63.85%
=======================================
  Files         234      234
  Lines        8800     8800
  Branches        3        3
=======================================
  Hits         5619     5619
  Misses       3180     3180
  Partials        1        1

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 9a8af70...7513f3a. Read the comment docs.

I added the backers and contributors to the homepage here: https://github.com/facebook/jest/pull/6537

You guys rock.

Please, try to rethink presence of opencollective banner in the postinstall script...
See my arguments in this discussion: https://github.com/invertase/react-native-firebase/pull/578#issuecomment-375198631

@vonovak this is only for postinstall in development

@rickhanlonii Oh, you are right, because this is monorepo...
Sorry, in this case it totally ok!

IT IS FIXED! The image should be fine now. There was an error in the reference to the repo.

Wow thanks @monkeywithacupcake, works great now!


[3.9] Upgrade to Platform 5.8.13

Pickup Jackson Databind <IP_ADDRESS> (CVE-2020-36518)

will be included in #3012


ACN_CD_SymantecEndpointProtection_Parser02

Proposed Changes

The initial parser was written for a SyslogMessage that included the syslog header:
<51>Nov 12 09:46:30 SAMPLESERVER SymantecServer: AA1030546,<IP_ADDRESS>...

The SyslogMessage, if ingested into the workspace properly will start with the Server details first, dropping the syslog header:
AA1030546,<IP_ADDRESS>...
The following revisions will correct this.
(updated sample data included)
12/15 -

Added parsing for Agent Risk Logs
Corrected RemoteHostName field that was of datetime type

Replaced by https://github.com/Azure/Azure-Sentinel/pull/1501


feat: Store client certificate paths in collection settings as relative to collection and display them the UI.

Description
The UI for collection level certificates does not reveal which file is used for which domains, so the only way to make sure you are using correct keys is to remove and re-add them. Also, the paths to key and cert files are kept in bruno.json file as absolute paths, making it unsuitable for version control and sharing between users with various setups.
With this change, the paths picked with file picker are relative to current collection and displayed in the Client Certificates view of collection settings.

Note, the absolute paths are still supported, so collections using old format are not affected.
fixes #2420
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

@sanjai0py @helloanoop consider this one when merging of #2336 . Thanks.

@pietrygamat Can you fix the conflicts?
Otherwise this is GTG

@helloanoop done.
As a sidenote: from my testing, when running bruno on Windows the path.relative produces wrong results (i.e. posix implementation is fired even though win32 one should). I am not sure it's my build issue, or electron issue.
I work around this by using:
      if (isWindowsOS()) {
        relativePath = slash(path.win32.relative(root, e.files[0].path));
      } else {
        relativePath = path.posix.relative(root, e.files[0].path);
      }

I think there are a couple more places in the application, that this may cause problems.

@sanjai0py , is it still GTG?

Really looking forward to this! Could simplify a lot of our setup :)

Merged! Thanks @pietrygamat


Unrecognized mode

What version of Remix are you using?
16.1.2
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
NODE_ENV=preview remix build
Expected Behavior
Not to log Unrecognized mode: preview and exit.
Actual Behavior
It logs Unrecognized mode: preview and exits without any context.

Can confirm. Workaround for the moment seems like (for example) NODE_ENV= remix build

I have found a similar bug, when deploying to cloudflare pages if you set NODE_ENV = production in the environment variables section, and you use tailwind, the tailwind css file is not generated. It is solved by removing NODE_ENV from the env variables

Fixed by #7113


complete list of Cs & Ps (TERN Ontology and imported / reference ontologies)

Can TERN please provide a complete list of all the ABIS-defined Classes & Properties (TERN Ontology and Messages stuff) AND other terms expected for use (from imported / reference ontologies) in converting DwC? (issue generating these ourselves as per Issue 80.)

Please see https://linkeddata.tern.org.au/viewers/tern-ontology.
Also note that there is a PR https://github.com/ternaustralia/ontology_tern/pull/55 which will refactor the specialised classes of tern:Value and align the TERN Ontology more with SSN.
TERN Ontology only deals with the domain-specific details and does not include the BDR messaging system by Surround [1].

ABIS-defined Classes & Properties ... in converting DwC

Once there's some consensus with https://github.com/ternaustralia/ontology_tern/discussions/53, the details of converting Darwin Core data into the TERN Ontology will be documented in the cookbook.
[1] https://github.com/surroundaustralia/abis

You are quite correct and fairly state that you can't be expected to respond to matter relating to Surround Messaging - (poorly worded request).
A better request would have been:
"Can TERN please provide a complete list of all the ABIS-defined Classes & Properties available within  TERN Ontology from the Tern Ontology and AND other terms expected for use (from imported / reference ontologies). (issue generating these ourselves as per Issue 80.)"
(I possibly should have posted this in the Discussions section - oops)

As mentioned in https://github.com/ternaustralia/ontology_tern/issues/80, please try TopBraid Composer.
You can use TopBraid Composer to open any RDF file (including the TERN Ontology) and view the instance data or the class/property hierarchy list. It will also handle imports stated by owl:imports. You can use this to view the imported classes and properties.

There is a 'Free Edition' of TBC available here https://www.topquadrant.com/topbraid-composer-install/
Note that it is version 6, and does not include the diagramming.
A license for the full version ('Maestro Edition') is a significant investment, but I find it is worth it.


NVM restore from an Aeotec  Z-Stick Gen5 device to an Aeotec  Z-Stick Gen7 failed

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
PKG executable
Z-Wave JS UI version
<IP_ADDRESS>a6f57
ZwaveJS version
10.14.0
Describe the bug
When restoring an Aeotec  Z-Stick Gen5 NVM backup to an Aeotec  Z-Stick Gen7, treatement fails
To Reproduce
Backup Z-Stick Gen5 NVM.
Swap sticks.
Restore NVMxxxxxxxxxxxxxxxxx.bin
Conversion reaches 100%
Restore aborts with the following message
INVALID NVM JSON: Controller is part of another network but has no learned Home ID! (ZW0281)
Expected behavior
Restore & Conversion end succesfully.
Additional context
Soft Reset & Hard Reset executed on Aeotec Gen 7 where useless
z-ui_2023-07-22_logs.zip

Was your Gen5 a secondary controller?

Hello,
thanks for your answer
it is my primary (and unique) controller
Regards
Le lun. 24 juil. 2023 à 11:47, AlCalzone @.***> a
écrit :

Was your Gen5 a secondary controller?
—
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/6055#issuecomment-1647578743,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBN3JTE3GBDGU7XQZDDBC7TXRZABNANCNFSM6AAAAAA2VE2TPQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

To investigate this, I'll need your NVM backups from both sticks. Send them to the email in my profile please.

backups sent

Interesting - there is one byte set to 255 I've never seen have that value before.

Great ! and how quickly !
Thank you


Reduced mipmap level

Exporting with a square (1024x1024) texture will increase the vertical size of the output image slightly and reduce the mipmap levels of the texture.
How can I export at 1024x2048 size?

change line 79 to nopow = False

Thanks!


Introduce gRPC API versioning

Can we have a planned meeting discussing about which types needs to be versions vs which one shouldn't be?


add IONOS Deploy Now

Please add IONOS Deploy Now as one more static hosting provider

Thanks for contributing!


Default dataset to cleaned alpaca dataset from HF

Simple change to default the dataset to the Hugging Face dataset repo for the Cleaned Alpaca dataset, rather than loading the local json, which will likely be out of date.

Makes sense. How often do you plan to update the Hugging Face dataset, by the way?

Wasn't that merged already
https://github.com/tloen/alpaca-lora/issues/167

Wasn't that merged already #167

Only the README.md was updated.  The finetune.py was not.

Wasn't that merged already #167

Only the README.md was updated.  The finetune.py was not.

Makes sense. How often do you plan to update the Hugging Face dataset, by the way?

Will generally keep the HF dataset in sync with the github repo. Will likely not sync each individual minor commit; however, after collecting a series of commits/PR's that are reviewed, planning to make a release and push to HF.

My reasoning for not updating the default in the code was that I didn't want to have any external dependencies in the code itself (besides my own repo, of course). Users can get a generally better experience pointing their clients to the decapoda HF checkpoints and the cleaned huggingface dataset, which is why I add them to the README, but I do want the default settings to work every time. I'd be comfortable pointing the code directly to the upstream repo if I had a sense that it wouldn't ship any dangerous/breaking changes.

Ah whatever. Move fast and break things


Staging next

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

there should be 1 more staging-next cycle after this for branch off, which is currently projected to be  September 4th.

@FRidh did you want to do a python package set update? I'm going to go to bed soon, but I'll have most of my sunday to do stabilization.

@jonringer Yes I'd like to have one for the next cycle. I can make a start with it.

qtquickcontrols failure is blocking KDE https://hydra.nixos.org/build/125404486. cc @petabyteboy @ttuegel


Support for tvOS

Not sure what it takes to make it work but it would be great if Starscream can add support for tvOS

Would also benefit from this

#129 added support.


Query Extraction

To avoid any problems with typos in the future, I extracted the keys from any query values I could find to a static class
This will also solve any maintenance problems should the query keys ever change, as they are all in a single place now

Thanks for your PR @lfmundim


[Tech Design] Terra Select Responsive Dropdown

Tech Design
Description

Tech design for https://github.com/cerner/terra-core/issues/2383
TLDR

Consolidate use of Dropdown and DropdownMenu

<DropdownMenu /> is an internal implementation detail of <Dropdown /> and it's unnecessarily complicating the component by rendering the menu as a renderProp in the <Select /> component

Implement (name pending) <ResponsiveDropdown /> (or <Modal />, whatever the UX defined "responsive dropdown" is)
Remove render props pattern in Select.jsx

In Frame, if this.state.isOpen, then use either the <Dropdown /> or <Modal />

Flex between <Dropdown /> and <Modal /> based on if we're mobile or at 'small' or 'tiny' breakpoints

Add isMobile method to SharedUtil which uses the UserAgent to check for iPhone, iPad, and Android
Use ActiveBreakpointContext to determine the activeBreakpoint
const useModal = isMobile() || activeBreakpoint === 'tiny' || activeBreakpoint === 'small';

Details
Dropdown, DropdownMenu, and the Unnecessary renderProps Pattern
<DropdownMenu /> is just an internal implementation detail of <Dropdown />. We don't need to be rendering it via render props in <Select />. We can just render the <Frame /> and delegate the dropdown and responsive behavior to the Frame. This allows us to reduce the complexity of the Select component while also giving us the ability to add new behavior easily to the Frame.
Responsive Dropdown Implementation
NOTE: See additional screenshots below for visuals.
The responsive dropdown workflow will live inside a DialogModal. When rendered, it will have an action header and action footer, while the content will be displayed with an input styled just like the current dropdown variant (including tags/pills) and a list of all options.
Action Header
We will need to give the action header a label to display as the title. The action header will call the responsive dropdown component's onClose prop when the action header's close button is pressed/clicked and close the modal.
Content
The content will contain an input (for multiple/search/tag variants) and a list of options as well as any extra visual indicators for workflows that exist outside the happy-path (e.g. "No results found") or normal usage (e.g. "Enter ‘x’ characters to search").
Action Footer
The action footer will have two buttons. The start button is labeled "Clear All" and will call the responsive dropdown's onClear prop when pressed/clicked. The end button is labeled "Apply" and will simply approve/commit the state change authored by the user. It will call the responsive dropdown's onApply prop when pressed/clicked. It is assumed that changes will only be committed when the Apply button is used, while also closing the modal.
isMobile()
class SharedUtil {
  // ...

  static isMobile() {
    const { userAgent } = navigator;
    const isAndroid = userAgent.includes('Android');
    const isIOS = userAgent.includes('iPhone') || userAgent.includes('iPad');

    return isAndroid || isIOS;
  }
}

Flexing between  and  workflows
Right now in Frame, we render a <Dropdown /> (using the renderProps pattern to pass props to a <DropdownMenu /> in <Select />) based on the boolean this.state.isOpen. We can use this plus another boolean to flex between <Dropdown /> and <Modal /> workflows. Using the above isMobile() method and the ActiveBreakpointContext, we can determine if we want to render the <Modal /> or not:
const activeBreakpoint = useContext(ActiveBreakpointContext);
const useModal = isMobile() || activeBreakpoint === 'tiny' || activeBreakpoint === 'small';
const { isOpen } = this.state;

return (
  <div {...stuff}>
    {isOpen &&
      useModal ? <Modal {...modalProps} /> : <Dropdown {...dropdownProps} />
    }
  </div>
);

Additional Context / Screenshots

@ Mentions

@bjankord
@StephenEsser

Also of note: rudimentary tests with a similar layout as above show iOS voiceover working as expected since all the children of the dialog modal are siblings of the input, rather than visually appearing to be.

+1 on tech design.

A little more context about the render prop so the decision to remove it isn't done under misapprehension.
The Select menu was originally designed to allow a custom implementation of the menu within the dropdown. The render prop was added to allow an entry point for this functionality. A user could implement a Select dropdown using the Frame and provide their own menu which would receive all of the require props from the render prop.
<Frame dropdown={dropdownProps => (
  <CustomMenu {...dropdownProps} />)}
/>

This would allow a user to create a custom menu implementation to meet application specific requirements. This requirement is also why the Frame itself exists as a separate component from the Select.
With the introduction of the accessibility changes, general complexity of this component, and the drive towards a standard behavior a custom menu is becoming increasingly more difficult and discouraged to implement.
If we decide to remove the custom render functionality we should also investigate merging the Frame into the Select to cut down on complexity and potential extra renders.

@StephenEsser good point!
when writing up the tech design, I had a similar thought: if we remove the renderProp pattern, we're basically just rendering the frame directly now so why do we need it?
👍 👍 👍

Do you foresee any issues customizing the Dialog Modal to display with a different positioning and height? The height would likely need to flex based on the Select variant that will invoke the keyboard.
The same question would apply to the Action Header and Action Footer. Are we going to alter the existing components to match the callouts or will they display with the default style?

Background Colors
Close Icon
Apply and Clear All Buttons
Borders

Will Clear All within the modal be conditionally rendered based on the presence of allowClear?
What are the disadvantages / advantages to using the components over creating a simple flex container?
Is the Input "sticky"? Does it scroll with the set of available options or does it remain fixed at the top of the Modal? The callouts look like it might scroll with the options, but we should get some clarification from UX on this.
( This is if it remains fixed at the top of the Modal )
What happens when the selected options width grows to be the height of the Dialog Modal? Previously this problem was somewhat mitigated by the ability to scroll the page. With the introduction of the Dialog Modal the page is not scrollable. When enough options have been selected the Options will no longer be visible.
We discussed offline about making the label of the Select required to enforce a label to be present when presented in a dialog.

We'll want to avoid using the user agent sniffing. There is a significant variability to the adherence to APIs and standards. As well as is against terra's general guideline to avoid device specific implementations, and code in favor of viewports and/or parent element sizing exclusively.
Here's also an article about it on MDN, about it shouldn't be used in this fashion:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Browser_detection_using_the_user_agent
We'll likely want to avoid the terra-dialog-modal, and use the terra-abstract-modal with specific implementation and styling.

@StephenEsser

The same question would apply to the Action Header and Action Footer. Are we going to alter the existing components to match the callouts or will they display with the default style?

What are the disadvantages / advantages to using the components ( Action Header / Footer ) over creating a simple flex container?

We can definitely roll our own headers & footers here. Doing so will help decouple the implementations between these components and allow for more control over styling (this appears to be the standard practice in terra).

Will Clear All within the modal be conditionally rendered based on the presence of allowClear?

That seems reasonable.

Is the Input "sticky"? Does it scroll with the set of available options or does it remain fixed at the top of the Modal? The callouts look like it might scroll with the options, but we should get some clarification from UX on this.

Yeah, I'll reach out to UX on this. From this image:

it does look like the input + list items scroll together.

What happens when the selected options height grows to be the height of the Dialog Modal? Previously this problem was somewhat mitigated by the ability to scroll the page. With the introduction of the Dialog Modal the page is not scrollable. When enough options have been selected the Options will no longer be visible.

From Dave's comment, I'll be using the abstract modal now. I wasn't aware of the dialog modal (or abstract modal) not allowing scrolling. Looking now, I see that overflow: hidden; is set for the abstract modal, but the dialog modal's body is set to overflow-y: auto;

We discussed offline about making the label of the Select required to enforce a label to be present when presented in a dialog.

This seems reasonable (or making the fallback what it currently is). Since this will be a breaking change, it makes sense to add props and/or change required-ness.

@dkasper-was-taken

We'll want to avoid using the user agent sniffing.

We already do it in the select shared utils here. Are you suggesting removing that? One of the use cases that @bjankord brought up is that we'll want iPads to also use the responsive dropdown since they'll also succumb to the iOS voiceover a11y concerns driving this redesign. As far as I'm aware, the iPad breakpoints can be pretty big and may map to large (or even huge).
Here's what the simulators for an ipad pro show when visiting https://engineering.cerner.com/terra-ui/#/components/terra-breakpoints/breakpoints/about

We'll likely want to avoid the terra-dialog-modal, and use the terra-abstract-modal with specific implementation and styling.

👍

Just a few questions from briefly looking at this:

My initial thought was how do you close the modal? It appears there is an x in the input, but I would assumed that is to clear all the content. However, there is an x on each option and a clear all option in the footer. Is clicking apply the way to close?

Is there a max height on the input that displays the selected options? Should there be an input? How will this height and the dropdown height be managed when keyboard is rendered?

For determining the mobile sizing, should we match what is currently implemented in terra-time-input? https://github.com/cerner/terra-framework/blob/master/packages/terra-time-input/src/TimeUtil.js#L191
I agree with Dave that determining the agent is likely not a great approach in supporting multiple browser and browser versions. Could there be screen reader issues on non-IOS tablets that use firefox or

Originally I was thinking we'd need to use UA detection to ensure this modal shows up on all iOS devices (including the larger iPad Pro devices), even though I'm not a big fan of user agent sniffing and instead prefer feature detection.
I'd recommend we check for the following conditions to trigger showing the modal:

The large breakpoint is not active and it is not the default variant or the browser supports touch events and is not the default variant.

The first condition covers most "mobile" devices and we don't want to trigger the modal for the default variant, that can keep using the dropdown
The second condition covers the larger iPad devices (we need to include these since the issue we are working to fix with with is with VoiceOver on iOS devices). For the second condition I'd only check for 'ontouchstart' in window and not worry about max-touch-points.
Let's move forward with a PR for this @dev-cprice and we can iterate on it on the PR and address questions as we get this built out some more.

Update: We've talked about this offline and have landed on implementing a solution that renders the input within the portal instead of rendering the select as a modal on mobile devices. Closing this issue as we've decided on a direction to go with this that won't require the implementation described in the tech design.


HTTP UserAgent cleaner (extension, XUL/XPCOM)

for those of us running a FF fork that supports the older XUL/XPCOM extensions, i think HTTP UserAgent cleaner might be worth a serious look
this is not a simple UA cleaner, so don't be fooled by the name
it's no longer being developed for FF because of the limits imposed by the WebExtension API, though it should work with the Moon, Waterfox and also Basilisk, which i just now learned of - apparently this is a Pale Moon project that supports WebExtensions (all of my extensions installed fine it appears)
the advantage i see in HTTP UserAgent cleaner is that it makes it easy to toggle the static settings in the ghacks prefs config and since it works mostly at the header level, i believe one can avoid adding exceptions to the ghacks prefs (untested) - it can also potentially eliminate the need for several other extensions dealing with things like canvas, UA, cookies/storage, WebGL, push, fonts, web sockets, SSL, https, etc.
if i remember correctly, HTTP UAC works at the header level, like i said, and so it does not interfere with other extensions or prefs - so for example if you block cookies at an extension, browser prefs, or user.js level, but allow them in HTTP UAC, they will still get blocked and will not override the settings in the browser
regarding Basilisk, it appears from their release notes that they are keeping up with the latest security patches - i don't know if Waterfox does this? - there are also packages for it in the AUR repo
links for HTTP UserAgent cleaner...

AMO (depreciated)
website
direct download (xpi): http://huac.8vs.ru/2.3.4-b05.xpi

there is no real EN documentation that i am aware of - with the """help""" (you will note the triple quotes) of the dev, i wrote an EN doc covering the add-on as thoroughly as i could (was on my site), but given the language barrier and the fact that he was so difficult to deal with i eventually canned the extension and deleted everything related to it - that said, i'm trying it out again on Basilisk
apparently he plans on porting it to a WebEx if and when the required API's are made available

oh shit, i see this should've gone here...
https://github.com/ghacksuserjs/ghacks-user.js/issues/492
sorry - can u move?

I don't know if Waterfox does this?

According to the blog, Waterfox is patched with the ESR security fixes.

OK, lets get this over and done with .. with all due respect don't anyone get upset .. here is my position, and by extension (pun intended), this repo's stance
I do not care for FF forks (except TBB) for all the obvious reasons (do you really need me to list these?). My information may be slightly out-of-date, as I am not interested in wasting my time following them.
 Palemoon is so behind the curve it's re-DiCK-ulous, and there are a lot of dubious decisions IMO (I do not have the full picture, but take for example the removal of e10s and sandboxing - maybe it's because the code is so divergent, NFI and I don't care).  Basilisk can also go in the PM pile - why would you trust your most important probably piece of software in your system (you know, the one that connects to the internet all the time, and handles almost ALL your online shit) with a "one-man-band" (note the speech marks) unvetted browser. If you're willing to do this, then I have two rocks to sell you - they keep tigers away, very valuable.
Secondly, I do not care for legacy extensions. Except for a couple of things (eg NoRedirect, extensions able to see/control other extensions etc), nothing is lacking in FF except what I term workflow items - and my answer to that, is to work around them .. adapt or die. Some may come back, some definitely won't. What are you going to do, stick with legacy BS all your life?
I removed all the legacy extensions listed on the wiki. I and this repo do not care for FF forks and legacy BS - end of story.
/end of rant / lesson
Now, before anyone gets all upset, remember this -> :kiss: :heart:

I removed all the legacy extensions listed on the wiki. I and this repo do not care for FF forks and legacy BS - end of story.

I guess we need a list of extensions which should be pkrted to WebExt. Maybe exploiting WebExt experiments mechanism.


Fix billboard and label clamping

Turns out that billboard and label clamping were fundamentally broken because the QuadtreePrimitive was processing the tile queue in the wrong order.  It was always pulling new tiles from the back of the array rather than the front, which meant that data would get processed in the wrong order causing old tiles to take precedence over newer tiles.
Additionally, there was a bad if block in Label.js which caused the initial position of the individual label billboards to not be properly set when clamping was on, instead we should always set the positions before calling _updateClamping (if needed).
Fixes #4396 and #4062
I already talked to @bagnell about this and he agreed it was a straight up bug.  I would love to add unit tests for these, but I'm not sure what the best strategy would be.
There are still several other clamping related issues that I'm looking into; but I wanted to keep these PRs small for easier review.

Am i doing something wrong? #4026 doesn't seem to be fixed.  The billboards look all over the place to me.
var viewer = new Cesium.Viewer('cesiumContainer', {
    baseLayerPicker: false,
    terrainProvider: new Cesium.CesiumTerrainProvider({
        url : 'https://assets.agi.com/stk-terrain/world',
        requestWaterMask : true,
        requestVertexNormals : true
    })
});
var options = {
    camera : viewer.scene.camera,
    canvas : viewer.scene.canvas,
    clampToGround: true
};
viewer.dataSources.add(Cesium.KmlDataSource.load('../../SampleData/kml/facilities/facilities.kml', options));

Works for me, are you waiting long enough, it can be slow?

Since #4497 has been retargeted, this is ready unless there are other comments.

Will look at this shortly.

For #4062, the labels didn't clamp to terrain for me:

Did you turn terrain on after you loaded the KML?  If so they won't clamp because that's fixed with #4497

Actually, I think changing terrain is yet another unrelated clamping bug, I'm looking at that separately now.

Works for me, but when you switch terrain, depth testing comes on, and due to #4090 KML billboards are centered, so they occlude a bit.
Facilities isn't a great example, but here's how it looks when you turn on terrain with clamping in this branch:

Works for me, but when you switch terrain, depth testing comes on, and due to #4090 KML billboards are centered, so they occlude a bit.

Thanks for testing @denverpierce. I actually think this is a bug in the facilities.kml file itself, but I have to check.  Also, did you turn terrain on after the fact?  I think there is a separate bug where terrain needs to be turned on before the data is added but I'm not sure.

This fixes #4396 and the misplaced glyphs of a label, but it does not fix #4062 completely.


Delete fails when table name <> class name

This occurs deleting an Order in ai_customer_orders example - app described here

Fixed in version 9.04.


Sync rel/1.2.1 to master

@franklin-feingold Why not just:
git fetch upstream
git checkout rel/1.2.1
git merge upstream/master
git push

It seemed like it is merging in too, is there a difference? I was on the GUI so doing this seemed easier

The main difference is now this will be in the changelog.

I can delete the entry right now (at least for stable), for latest it will reappear. next release I'll do cli


🔥 remove unused cps

no issue

settings/general controller contained a number of xImageSource CPs that are no longer being used

Changes Unknown when pulling f06e273ee7982ac158b794150ff00ac2a15f9da1 on kevinansfield:remove-unused-cps into ** on TryGhost:master**.


jscript

🔍 Before submitting the issue
🐞 Description of the bug
Hello - I used the code (https://discuss.ansys.com/discussion/1084/how-to-generate-the-report-preview - which works inside wb mech. script console), but gives an error inside pymech. (as we said it runs fine inside wb mech):
PublishType = {"SingleFile": 1, "FiguresSubFolder": 2, "SingleFolder": 3}
cmd = """DS.Script.doReportPreviewWindow();"""
cmd1 = """DS.Script.doReportRegenerate();"""
cmd2 = """DS.Script.publishReport("{0}",{1});""".format(r"D:/Data/sample.htm", PublishType["FiguresSubFolder"])
ExtAPI.Application.ScriptByName("jscript").ExecuteCommand(cmd)
ExtAPI.Application.ScriptByName("jscript").ExecuteCommand(cmd1)
ExtAPI.Application.ScriptByName("jscript").ExecuteCommand(cmd2)

📝 Steps to reproduce
See above
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
2023 R2
🐍 Which Python version are you using?
3.10.10 (anyhow the pymech script example bolted flange with internal pressure works, only when the jscript is added it does not work and gives the error)
📦 Installed packages

Closing, we don't support jscript.


How do we work with this folder?

Hi! Maybe I totally missed some planning here, but is there a plan to the workflow of the course?
Do we add everyone as collaborators? Do they all make their own branch or rather their own fork? Will anyone be main responsible for maintaining the repo?

Also, how does this repo and the tutorial one from pangeo-data interact? Will that one just stay as is with the tutorial material, and then we add our own material to this repo? If so, I suggest a structure along the lines of last year.

yes - lets do as last year - can you prepare this in this repo @sarambl ?

I can try, but can't promise I'll have time. I'm a bit over-worked!
We might want to copy something like this: https://nordicesmhub.github.io/forces-2021/setup/organize.html?highlight=github
In general I think it's not a bad practice if everyone makes their own fork, but maybe you have an opinion on best practice here @annefou ? I don't quite remember what we did last year, but I think we might have had a branch per group or something?

But if everyone only work on their own fork, it is difficult to share and also look for inspiration among the students... we can always have a different branch for each group and commit from the forks to that particular branch and it that way share. If the students are not too familiar with git it might be messy... But you have more experience with this than me.
Do I need to remove my plankton folder? I can do so :-)

So, the idea would be to regularly pull from the main repo and also to submit pull requests.. Much like with a branch, but I see how people might forget to do that!

@adagj , no need to remove anything, I think whoever does a tidying can just move it to the right place.
@MichaelSchulzMETNO Honestly I would prefer if someone else does the cleaning and set up, because I am really at max capacity already for a while!

ok, I can do that over the weekend. And suggest a way to git-work together.

We usually advice not to use this repo and use the organisation eScience-course and the template we prepare. I suggest one repo per group as one group should use similar datasets. This will make their workflows much simpler. Each group mentor needs to admin his/her own repo and add other team members as collaborators of their group repo.
You can add examples to GitHub.com/pangeo-data/escience-2022 (pull request).

Honestly, I prefer one repo for the course. In addition to the pangeo repo. It worked well in the last years. And we want to foster exchange in the whole course group. There is also data use overlap.

Of course, if all group repos are public, exchange is possible, I am not dogmatic on this.

Several groups work on quite similar subjects and will use the same data and exchange methods and scripts (I hope), so I also prefer one repro. The scripts I have uploaded so far can be useful for all groups I think, and I'm afraid it will be confusing to copy and update among many repos if we have one for each group. But I don't have very strong opinions on this, just expect the students to be easily git confused :-)

We usually advice not to use this repo and use the organisation eScience-course and the template we prepare. I suggest one repo per group as one group should use similar datasets. This will make their workflows much simpler. Each group mentor needs to admin his/her own repo and add other team members as collaborators of their group repo.
You can add examples to GitHub.com/pangeo-data/escience-2022 (pull request).

@annefou Which repo is "this repo"? (sorry for being slow..)

This repo ==  https://github.com/NordicESMhub/escience-2022
I think you should do what you want.
My experience from the previous years is that students are unable to learn and use GitHub properly during the course. We usually need to clean their mess on a regular basis (which I won't do this year).  Having their own repo and/or per group is much easier and can give them a bit more confidence and this is anyway what they usually do during their day to day work.
Examples could be added to GitHub.com/pangeo-data/escience-2022 (I could add mentors, etc. as collaborators to the repo).

Of course, if all group repos are public, exchange is possible, I am not dogmatic on this.

Having private repos would certainly not help students to learn and understand Open Science practices...

Ok suggestion:
Lets try to do one public repository per mentor-group this year. I made a test and created a https://github.com/eScience-course/Tjaernoe2022, which is mainly for my test purposes. Should be easy to do for all assistants. I chose the MIT licence.
Examples will be visible to all if repos are public
Suggested group name by number: https://github.com/eScience-course/Tjaernoe2022-group1
See google drive escience-2022 participants  for group names and github names.
Private slides and presentations : we could store in the course google drive for all participants.


mkDerivation: add overrideAttrs function

Motivation for this change
This is similar to overrideDerivation, but overrides the arguments to
mkDerivation instead of the underlying derivation call.
Also update makeOverridable so that uses of overrideAttrs can be
followed by override and overrideDerivation, i.e. they can be
mix-and-matched.
Extracted out of #17886, as this seems to have existing support (e.g. see #10721), so there is no need to keep it with the cmakeFlags changes as a motivator.
Things done

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

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

TODO: this needs documentation, I would like some pointers on which docs need updating.
IMO almost all cases of overrideDerivation are better served by overrideAttrs, so I would like to update the docs to replace those references. Since that changes a fairly visible part of nixpkgs, it may be better to put this in after 16.09.

Looks useful!

@abbradar I'm happy to run some benchmarks, but would appreciate some more concrete instructions on how to do so.

I have completely forgot about this, sorry. I've found an old issue about this with associated benchmarks: https://github.com/NixOS/nixpkgs/issues/10721#issuecomment-154841265 Let's see if things are better now...

@abbradar This PR just adds a new function but doesn't use it anywhere yet, (unlike the old PR which changes overrideDerivation), so those benchmarks will just report the same thing both times. A blind search/replace doesn't seem like a good idea as the new overrideAttrs has slightly different semantics and will act differently in some cases, so I think using a few selected test cases would be best. Can you help me pick a few (important) packages that currently use overrideDerivation that I can bench after switching to overrideAttrs?

I think the main performance impact of the old patch was adding things to makeOverridable, therefore effectively growing closure of every function wrapped in it. I'm not sure however,

I have updated this with some documentation, please review and let me know how it can be improved.
@abbradar there is a trailing comma on your last comment, is part of it missing? I'm not sure what you're looking for in terms of benchmarking.

You are right, fixed; I was going to bed around this time and was sleepy -- sorry! Disclosure: personally I have urges to just merge this because IMO it's better to have a sane overrideDerivation even at a performance cost. However, others may not be so passionate about it, so we need numbers to present.
Also notice that there're very few overrideDerivation calls in nixpkgs and it's generally considered slow, so I don't think performance penalty of actual overrideAttrs calls need to be measured (I don't think it has much more impact that just a callPackage from what I see) -- rather we are interested only in added overhead for makeOverridable.

I wrote a quick benchmarking script. Here are some sample results; note that it varies between runs.
cowsay
                     before     after     delta     change
Elapsed time          0.109     0.094    -0.015  -13.7615%
Max RSS (Kbytes)    27663.6   27684.8     +21.2   +0.0766%

diffoscope
                     before     after     delta     change
Elapsed time           0.26      0.26      +0.0   +0.0000%
Max RSS (Kbytes)    51450.4   51534.0     +83.6   +0.1625%

firefox
                     before     after     delta     change
Elapsed time          0.288      0.27    -0.018   -6.2500%
Max RSS (Kbytes)    52466.8   52707.6    +240.8   +0.4590%

gnome3
                     before     after     delta     change
Elapsed time          0.596     0.595    -0.001   -0.1678%
Max RSS (Kbytes)    95241.6   95707.6    +466.0   +0.4893%

I don't think this patch will cause any significant performance problems.

Seems good to me; cc @edolstra

/cc @domenkozar

I think right after the 16.09 release is a good time to get this in since it's a replacement for (most uses of) overrideDerivation, allowing time before the next release to update references in the code and docs, etc.
I'm also looking for feedback on the documentation I added.

I'll review this shortly.

@domenkozar, just a kindly reminder :)

Clean PR with docs and simple implementation. Merging.

Nice idea! I personally see very little use for overrideDerivation, now that we have this, as the derive primitive is hidden too deep in typical expressions.

Thanks @domenkozar! Can I request that this be cherry-picked to 16.09/stable?

This is great; wanted for a long time! But I wish we could call this overrideDerivation, and the old one overrideRawDerivation, or something like that.

Also nice if this could use the self: super: _ ideom.

I thought of overrideMkDerivation, especially if we also changed override to overrideCallPackage, but the names would be very long.
Oh, I didn't recall lib.extends. We still don't have a way for the two-argument style on most places. Perhaps we should introduce some new parallel naming for using this more powerful style.

I don't think silently changing what overrideDerivation does is a good idea, and I never liked that name anyways.
Also, what's the benefit of the self: super:  idiom here? I've never felt the urge to use a self argument when doing overrides, usually just oldAttrs: rec { ... } works for me.

@aneeshusa check out the agda stuff, which does an adhoc mkDerivation arguments self: super:

@Ericson2314 I took a look and think you're talking about the code at the bottom of pkgs/build-support/agda/default.nix. I can't yet understand what's going on or what the purpose of the extension arg is (which seems to be the raison d'être for the self/super distinction). I can't find docs in-tree, in the manual, in commit messages or PR comments, and I see only extension = self: super: {} being passed for that arg. Can you please provide some more explanation and/or concrete examples?

Well, self is supposed to refer to the final version, i.e. even after some other overrides were (potentially) applied.
It's not terribly useful, but it can come in some complex cases... out of the top of my head, say the package/override (conditionally) applies a patch that should be gone by version x and after that you want to additionally apply override of the version to a newer one – in that case it's better to use optional self.version than just optional version.

@aneeshusa sorry for being slow. The simple answer is mkDerivation rec { .. } is fairly common, and now suspect. Each case probably ought to be probably be a mkDerivation (self: with self; { .. }), because one probably wants other arguments to be updated based on overrides.
rec { .. } in general is kind of a smell for this reason.


In use pin    no longer exists on node  Add Root Bone . Please refresh node or break links to remove pin.

Error in BP_SkeletonBoneFunctions function Add Root Bone

Windows 10
Unreal Engine 5.0.3 Source Build
This occurs when updating an existing installed plugin

Please wait until v0.4 is officially released then the issue should be gone, it seems to be an Unreal Engine issue. It will available during the next days. Basically when this gets merged to master, https://github.com/tuatec/TTToolbox/pull/12.

Sorry I thought it was released (but you just renamed the branch)! I will now take look at your full implementation!

Now v0.4 is already released. If you have further issues please do not hesitate to reopen this issue if you still have the same issue. 😉


include room id to the data being transmitted by start and stop typing events

include the room_id in the data being sent by the start and stop typing events so that bots can also receive these events and assign them to the respective room. This information can be logged to calculate writing and idling time of a user

isn't this implicitly contained with the user_id because users can only be in one room at a time?


Fixed VLA definition because not every compiler supports that

Fair enough, support for VLA on is not certain. Leaving aside Visual Studio, rare platforms could be hurt.
Thanks.

This has been merged into a maintained fork here https://github.com/fast-debug/PHP-CPP/pull/2

#424 duplicates #326 but takes a different approach to address the lack of VLA support in MSVC.
Can we pick one approach and use a wrapper, e.g.,?
#if __STDC_NO_VLA__
        zval* arguments = static_cast<zval*>(alloca(argc * sizeof(zval)));
#else
        zval arguments[argc];
#endif


Consider how a FIT R universe page can supplement the FIT website.

Some information about how other communities are using R universe

We use it for our time-series packages: https://atsa-es.r-universe.dev/packages. So easy to set up and make much much easier for people to install your GitHub packages esp if they include C++ code (e.g. TMB) because they don't have to build them. Works like installing from CRAN.

We do have this experimental FIT one.
I think the other advantages is you need the whole r-tools chain for github pkgs with c++ code if you install from r universe.

Recent value added: devtools::install_github() isn't working on NOAA laptops, and pak::pak() required the package build tool chain. Installation from R universe does not, so the user was able to install from R universe as a "quick fix" way to get the package nmfspalette, which is not on CRAN.
R universe can be a helpful tool for users and developers of the toolbox; We could provide instructions on how to install using R universe on the R package pages?

I think there are some clear uses of R universe that could be helpful. I think when developing the FY25 plan, we can consider specific action items based on what we know about R universe. For example:

Committing to maintain an R universe for the FIT and advertise it(we have one, but it is not well advertised)
Add install instructions using R universe to all of the FIT R packages


[feat] Add Detached Washboard

Related Issues

wasmcloud/wasmcloud#321

Feature or Problem
This work comes out of this wasmCloud RFC which aims to make the wasmCloud dashboard UI (AKA the washboard) detached from the host and able to connect to any lattice.
Technology decisions
We discussed various frontend frameworks like Svelte and Solid, but ultimately decided on React because of community size and support. Here are some of the packages chosen alongside React:

UI

Tailwind with cva for utility-first approach with smallest stylesheet footprint
Radix with shadcn/ui for a large, accessibility focused, theme-able component library
React Router for best routing experience
React Hook Form with zod for form handling and validation

State management

RxJs for event-based data handling (pairs nicely with NATS subscriptions)
NATS.ws for communicating with lattice

Features
The aim is to be as close to feature parity with the existing OTP host dashboard. Here is a breakdown/comparison of each dashboard
✅ - supported
⛔ - not supported
⏱ - planned
🔨 - in progress

OTP Host
Detatched

Hosts

View Status
✅
✅

Stop Host
⛔
⏱

Providers

View Status
✅
✅

Deploy from file
✅
⛔ 1

Deploy from registry
✅
⏱

Remove
✅
⏱

Actors

View Status
✅
✅

Deploy from file
✅
⛔ 1

Deploy from file (hot-reload)
✅
⛔ 1

Deploy from registry
✅
⏱

Remove
✅
⏱

Links

View Status
✅
✅

Define
✅
⏱

Remove
✅
⏱

Lattice

Connect to lattice by URL
⛔2
✅

Connect using credentials
⛔2
⏱

Notes

sending actor/provider bytes over NATS is not (yet?) supported
OTP-Host Washboard is built into host

Considerations
This PR assumes only one method of distribution—bundling assets and serving via the wash cli tool—but this doesn't have to be the final solution. In fact, since the app itself compiles to static web assets, multiple options could be implemented with little effort maximizing flexibility. Alternates include:

separately built binary that is only downloaded when wash ui is called for the first time
as above, but packaged as a wash cli plugin
bundled as a wasmCloud Actor component
bundled as a Tauri application
provided as static files only

Merging with failing windows build because it's failing everywhere


Not enabled in Outlook Web app

I'm using the Outlook Web app on Chrome, and the Markdown Here button is frequently disabled when in composing mode.
Is there a way to manually trigger the conversion, since the auto-detect of applicable text field is not working correctly? I'm not sure what the actual reason might be. Perhaps Outlook is making it difficult to detect..

Another interesting behavior I just discovered - if I focus (click) on the email address To field, and then focus back on the email body, then the button is enabled.


yarn serve运行项目报错

⚠ IMPORTANT ⚠ Please check the following list before proceeding. If you ignore this issue template, your issue will be directly closed.

[ ] Read the docs.
[x] Make sure the code is up to date. (Some bugs have been fixed in the latest code)
[ ] This is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Describe the bug
failed to load config from G:\demo\vue-keycloak-js-master\vben-admin-thin-next\vite.config.ts
error when starting dev server:
Error: Build failed with 3 errors:
package.json:1:0: error: Unexpected "b"
package.json:1:0: error: Unexpected "b"
tsconfig.json:1:0: error: Unexpected "b"
at failureErrorWithLog (G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:1224:15)
at buildResponseToResult (G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:936:32)
at G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:1035:20
at G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:568:9
at handleIncomingPacket (G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:657:9)
at Socket.readFromStdout (G:\demo\vue-keycloak-js-master\vben-admin-thin-next\node_modules\vite\node_modules\esbuild\lib\main.js:535:7)
at Socket.emit (events.js:315:20)
at addChunk (internal/streams/readable.js:309:12)
at readableAddChunk (internal/streams/readable.js:284:9)
at Socket.Readable.push (internal/streams/readable.js:223:10)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Reproduction
Please describe the steps of the problem in detail to ensure that we can restore the correct problem
System Info

Operating System:
Node version:
Package manager (npm/yarn/pnpm) and version:

npm run bootstrap
npm run serve


PHP7.4 compatibility issues

Hello!
It looks like certain parts of this code aren't compatible with PHP7.4 sadly. When doing a search for constellations and ore type we come across this error:
2020/05/27 01:17:25 [error] 369#369: *88927 FastCGI sent in stderr: "line 177PHP message: PHP Notice:  Trying to access array offset on value of type float in /var/www/bleh/module/VposMoon/src/View/Helper/VposMoonHelper.php on line 177

The line in question being the third line in this function:
   private function sortWorthForRenderMoonMateriallist($a, $b)
    {
        return $a["worth"] <= $b["worth"];
    }

Would it be possible to get it updated for PHP7.4 compatibility to ensure it's good for the next 18-30 months?
https://www.php.net/supported-versions.php

Do you experience any problems with the application?

Yeah, I gave a quick rundown above. If you'd like I can re-enable and grab more logs, it's basically just install php7.4 & necessary php-modules - switching the socket and testing.
I'll debug it a bit more this evening when I get some time.

put up a pr with a proposed fix, please check it over - I'm terrible at PHP


Need optimization Oap heartbeat

At Executor side, Oap will report FiberCacheHeartbeat,IndexHeartbeat and FiberCacheMetricsHeartbeat to driver, spark.sql.oap.heartbeatInterval default value is 2s.
We found the size of heartbeat about 80k+ before deserialize, 220k+ after deserialize and deserialize  one heartbeat package cost 5 ~ 10ms, the heartbeat rate is too high,  package size is too large and deserialize time too slow, it will affecting the processing performance of spark event bus and let important events not be processed immediately in high concurrency scenario.
Our testing environment use 80 instances and 49 core per instances, we config spark.sql.oap.heartbeatInterval to 5s problem have a little improved.

cc @jerrychenhf @xuanyuanking

Can #817 make this faster?


Support new upstream api

https://github.com/Mashape/kong/pull/1735

Seconded.  Would love the ability to add multiple upstreams/targets to an api via the config file.  Use case is using kong to route upstream to containers in a rancher cluster without using DNS and being able to control what containers belong to what api (ie swapping container sets in a blue/green deploy scenario)


When to reset value of $_WD_HTTPRESULT?

I'm wondering whether almost each _WD_**** function call should clear $_WD_HTTPRESULT at start to be sure it is trully LastHTTPResult returned by _WD_LastHTTPResult() ?
Originally posted by @mlipok in https://github.com/Danp2/au3WebDriver/pull/247#discussion_r820866604

Currently it is reset in __WD_Post() :
Func __WD_Post($sURL, $sData)
	Local Const $sFuncName = "__WD_Post"
	Local $iResult, $sResponseText, $iErr

	If $_WD_DEBUG = $_WD_DEBUG_Info Then
		__WD_ConsoleWrite($sFuncName & ': URL=' & $sURL & "; $sData=" & $sData)
	EndIf

	$_WD_HTTPRESULT = 0

Right... each of these functions performs an HTTP request, so it makes sense to me that the variable is reset in each of these functions.

I think that $_WD_HTTPRESULT should be like a @error it should be reset on each _WD_*** function call.
It is user/scripter/coder resposibility to check them and store in correct time.

There are functions that don't perform any WinHTTP requests (the new _WD_Json* functions for example). Would calling one of these also reset the value of  $_WD_HTTPRESULT?

This would be a script breaking change. Therefore, it shouldn't be made without great consideration.

My approach to it is different: it is making order from previously not fully/correctly specified functionality.

...not fully/correctly specified way of how $_WD_HTTPRESULT works/behaves. Last should be really last and you don't need to focus on which _WD_*** function resets / changes this value. .... nothing more to add here.

Just because it doesn't behave the way you think it should doesn't necessarily make the current implementation wrong. 😉

Yes, I know the current implementation works.... in most cases.  As long as you check the value at the right moment. The question is what is right moment?
If coder is experienced then he will try to do it ASAP.
You can do it right after executing the function or later ......... and here it starts the question, how much other  _WD_**** code can you use before losing the previous value?
Ok.
We must to slow down 1 day. Let's do 24H brake.

Finally, I will ask one question, how many scripts have you written based on the analysis of this value?
In my case basically none.
They are all based on @error analysis only $_WD_HTTPRESULT is used only internally in UDF for better way of debug by showing additionall info i logs.
The question is did you use them in any of your main script (outside Au3WebDriver UDF) ?
I think that other user can confirm that they not rely on this value.
I mean it will not be Script Breaking Change as internally UDF will be consistent. Using them out of UDF will give you 0 or value. And it will be still valid.

Btw. we can always resetting this valu not for all functions for example excluding _WD_Json****.
And I hope you agree that this will be a rresonable step forward to reset them in all functions that calls __WD_Post() __WD_Get() __WD_Delete()  but in internally in those 3 function.

Btw. we can always resetting this valu not for all functions for example excluding _WD_Json****. And I hope you agree that this will be a rresonable step forward to reset them in all functions that calls __WD_Post() __WD_Get() __WD_Delete() but in internally in those 3 function.

Here is changes which i was in my mind: wd_core.au3 $_WD_HTTPRESULT = 0 presetting
Same should be done for other wd udf files but only where __WD_Post() __WD_Get() __WD_Delete() is called.
if you check wd_core.au3 then you notice that I omnit function that not use  __WD_Post() __WD_Get() __WD_Delete() .

To answer your question, I've used _WD_LastHTTPResult in one place in my main script, and I'm using it immediately affer a call to _WD_Action, so there isn't any issues here. However, that doesn't mean that someone else isn't using it in such a way that your proposed changes will break their scripts.
P.S. What happened to waiting 24 hours?! 😵💫
P.P.S. You misspelled "resetting" 😝
P.P.P.S. There's no point in resetting the value in __WD_Post/Get/Delete if it is being done separately elsewhere

To answer your question, I've used _WD_LastHTTPResult in one place in my main script, and I'm using it immediately affer a call to _WD_Action, so there isn't any issues here. However, that doesn't mean that someone else isn't using it in such a way that your proposed changes will break their scripts.

No. Because only those _WD_*** functionions that relays on __WD_Post/Get/Delete will reset this value at start.

P.S. What happened to waiting 24 hours?! 😵💫

Coca cola + coffee

P.P.S.

P.P.P.S. There's no point in resetting the value in __WD_Post/Get/Delete if it is being done separately elsewhere

Sure.

take also look on: Func _WD_Window
If you use it like this:
_WD_Window($sSession, 'mistake')

it will lead to $_WD_ERROR_InvalidDataType but  none of __WD_Post/Get/Delete will be called so without $_WD_HTTPRESULT = 0 ; Reseting result of last WinHTTP request it will still have old values.
And it is getting to more and more diffucult to relays on $_WD_HTTPRESULT without resteting them at start.

This would have been easier to fix if it were a 1 line function.

take also look on: Func _WD_Window

Oh, I understand the problem. I just don't want the "solution" to cause more issues than it fixes.

Oh, I understand the problem. I just don't want the "solution" to cause more issues than it fixes.

I agree here.
Just give me a day or two to finish my corrections.
Then you will see the whole thing and probably you will point out some further corrections.
But the direction is right.

I think this ISSUE can be closed as all is processing (suplementation is in progress) in https://github.com/Danp2/au3WebDriver/pull/247

I think this ISSUE can be closed as all is processing (suplementation is in progress) in #247

What is your opinion ?


Add dutch translations

Enhances the translations with the dutch language (because I needed them in my project)

Thanks @wesselvanderlinden !


Adds clearing errors on ngForm.$setPristine() invocation and fixes possible memory leak.

ngForm.js:
It is more convenient for consumers to clear all form's validation errors on ngForm.$setPristine() instead of additional call to ngForm.$aaFormExtensions.$clearErrors().

ngModel.js:
It seems there was a little memory leak in element's 'blur' event listener (the 'showErrorReasons' array is increased on every event).

Nice, good idea. Let me mull this over a little bit and I'll get back to you. Thanks!

There is one concern related to wrapping $clearErrors in $setPristine:
$clearErrors - async function (at least as for now), $setPristine - sync. After wrapping, $setPristine will be async too. But it is a lack of convention and can cause potential issues (e.g. flashing error notifications on UI).
So, there are 2 solutions I can suggest:

extract sync part of $clearErrors() into a separate method and use it inside of $setPristine() and $clearErrors().
Advantages:
- extended theForm.$setPristine()  method will actually do what its name states;
- $aaFormExtensions.$clearErrors() method still can be used for some advanced scenarios with no breaking change.

reconsider $clearErrors() method as a synchronous.

What do you think? Or maybe I missed some important things?

Yeah unfortunately the $clearErrors() had to be async due to some issues I was having getting everything to clear in a single digest (thinking $evalAsync would work) (or even without a $timeout 0 for some reason). I'll mess with it again, maybe its something that has been improved since 1.2. Let me try it out to see how we should proceed. Thanks!


Codeunit 2000020 DomiciliationJnlManagement "CreateDomiciliations"

can we have an event in CU 2000020 DomiciliationManagement in function "CreateDomiciliations"
DomJnlLine.Copy(DomicJnlLine);
DomJnlLine.SetRange("Journal Template Name", DomJnlLine."Journal Template Name");
DomJnlLine.SetRange("Journal Batch Name", DomJnlLine."Journal Batch Name");
if DomJnlLine."Bank Account No." <> '' then
DomJnlLine.SetRange("Bank Account No.", DomJnlLine."Bank Account No.");
if DomJnlLine."Posting Date" <> 0D then
DomJnlLine.SetRange("Posting Date", DomJnlLine."Posting Date");
OnAfterSetFilters(var DomJnlLine); //new event
DomJnlTemplate.Get(DomJnlLine."Journal Template Name");
REPORT.RunModal(REPORT::"File Domiciliations", true, false, DomJnlLine);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

This is not ok in BC15.4: the procedure is not an event:
[Scope('OnPrem')]
procedure ModifyPmtDiscDueDate(DomicJnlLine: Record "Domiciliation Journal Line")
var
    CustLedgEntry: Record "Cust. Ledger Entry";
begin
    with DomicJnlLine do begin
        CustLedgEntry.SetCurrentKey("Document No.");
        CustLedgEntry.SetRange("Document Type", "Applies-to Doc. Type");
        CustLedgEntry.SetRange("Document No.", "Applies-to Doc. No.");
        CustLedgEntry.SetRange("Customer No.", "Customer No.");
        if CustLedgEntry.FindLast then begin
            CustLedgEntry.Validate("Remaining Pmt. Disc. Possible", -"Pmt. Disc. Possible");
            CustLedgEntry.Validate("Pmt. Discount Date", "Posting Date");
            CustLedgEntry.Modify(true)
        end;
    end
end;

local procedure OnCreateDomiciliationsOnAfterSetFilters(var DomiciliationJournalLine: Record "Domiciliation Journal Line")
begin
end;

this is just created as local function, so useless. this should be created as an event.


Being able to edit project view as a fully dashboard view

Being able to edit project view as a fully dashboard view

The project list views are editable just like the standard views.  However, integration with the Dashboard plugin or other views is not exactly a high priority.  I'll accept pull requests.

Moving to https://issues.jenkins-ci.org/browse/JENKINS-32250