java.lang.NullPointerException at com.microsoft.azure.maven.utils.MavenArtifactUtils.isExecutableJar (MavenArtifactUtils.java:50)

Plugin name and version
<groupId>com.microsoft.azure</groupId>
<artifactId>azure-spring-cloud-maven-plugin</artifactId>
<version>1.9.0</version>

Plugin configuration in your pom.xml
				<plugin>
					<groupId>com.microsoft.azure</groupId>
					<artifactId>azure-spring-cloud-maven-plugin</artifactId>
					<version>1.9.0</version>
					<!-- https://github.com/microsoft/azure-maven-plugins/wiki/Azure-Spring-Cloud -->
					<configuration>
						<runtimeVersion>Java_11</runtimeVersion>
						<allowTelemetry>false</allowTelemetry>
						<appName>${azure.appName}</appName>
						<clusterName>${azure.env}-XXX-asc</clusterName>

						<!-- https://github.com/microsoft/azure-maven-plugins/wiki/Authentication#service-principles-in-plugin-configuration -->
						<auth>
							<serverId>azure-auth</serverId>
						</auth>

						<public>true</public>
						<isPublic>true</isPublic>

						<subscriptionId>${azure.subscriptionId}</subscriptionId>

						<!-- ??? -->
						<advancedOptions>true</advancedOptions>

						<!-- https://github.com/microsoft/azure-maven-plugins/wiki/Azure-Spring-Cloud%3A-Configuration-Details -->
						<deployment>
							<environment>
								<SPRING_PROFILES_ACTIVE>${azure.spring.profiles.active}</SPRING_PROFILES_ACTIVE>
							</environment>
						</deployment>

						<!-- About the lack of skip option -->
						<!-- https://github.com/microsoft/azure-maven-plugins/issues/1936 -->
					</configuration>

					<executions>
						<execution>
							<id>deployWebApp</id>
							<phase>deploy</phase>
							<goals>
								<goal>deploy</goal>
							</goals>
						</execution>
					</executions>
				</plugin>

Expected behavior
A self-expressive error if something is wrong in my configuration
Actual behavior
Caused by: java.lang.NullPointerException
    at com.microsoft.azure.maven.utils.MavenArtifactUtils.isExecutableJar (MavenArtifactUtils.java:50)
    at java.util.stream.ReferencePipeline$2$1.accept (ReferencePipeline.java:176)
    at java.util.ArrayList$ArrayListSpliterator.forEachRemaining (ArrayList.java:1655)
    at java.util.stream.AbstractPipeline.copyInto (AbstractPipeline.java:484)
    at java.util.stream.AbstractPipeline.wrapAndCopyInto (AbstractPipeline.java:474)
    at java.util.stream.ReduceOps$ReduceOp.evaluateSequential (ReduceOps.java:913)
    at java.util.stream.AbstractPipeline.evaluate (AbstractPipeline.java:234)
    at java.util.stream.ReferencePipeline.collect (ReferencePipeline.java:578)
    at com.microsoft.azure.maven.utils.MavenArtifactUtils.getExecutableJarFiles (MavenArtifactUtils.java:84)
    at com.microsoft.azure.maven.springcloud.config.ConfigurationParser.toDeploymentConfig (ConfigurationParser.java:39)
    at com.microsoft.azure.maven.springcloud.config.ConfigurationParser.parse (ConfigurationParser.java:22)
    at com.microsoft.azure.maven.springcloud.AbstractMojoBase.getConfiguration (AbstractMojoBase.java:124)
    at com.microsoft.azure.maven.springcloud.AbstractMojoBase.initTelemetryProxy (AbstractMojoBase.java:100)
    at com.microsoft.azure.maven.AbstractAzureMojo.execute (AbstractAzureMojo.java:505)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)

Steps to reproduce the problem
My command is mvn azure-spring-cloud:deploy -Pazure,dev,deploySpringCloud -DskipTests -PskipStyle -X, but I suppose this denotes something wrong in the considered input fatJar.
Given https://github.com/microsoft/azure-maven-plugins/blob/develop/azure-maven-plugin-lib/src/main/java/com/microsoft/azure/maven/utils/MavenArtifactUtils.java#L46-L54
public static boolean isExecutableJar(File file) {
    try (final FileInputStream fileInputStream = new FileInputStream(file);
         final JarInputStream jarInputStream = new JarInputStream(fileInputStream)) {
        final Manifest manifest = jarInputStream.getManifest();
        return manifest.getMainAttributes().getValue("Main-Class") != null;
    } catch (IOException e) {
        return false;
    }
}

I suppose manifest.getMainAttributes() is null in my case, and the exception should report this explicitly.

The path to the faulty jar would also be helpful.

@blacelle Thanks for your report and sorry for the late reply, we will looking into this issue soon, as a workaround, you may use following configuration to set the artifact to deploy
            <configuration>
                ...
                <deployment>
                    ....
                    <resources>
                        <resource>
                            <filtering/>
                            <mergeId/>
                            <targetPath/>
                            <directory>${project.basedir}/target</directory>
                            <includes>
                                <include>{your_fat_jar}.jar</include>
                            </includes>
                        </resource>
                    </resources>
                </deployment>
            </configuration>

Hello,
I feel my issue is not a lack of jar. It seems related to https://stackoverflow.com/questions/37783069/why-cant-i-extract-a-spring-boot-executable-jar
-> Hence, JarInputStream (supposedly) fails opening this file.

@blacelle Thanks for your message, could you please help check is there any fat jar in your target directory? If not, you may use spring-boot-maven-plugin to generate one, refers and specify the fat jar in spring cloud maven plugin configuration?
We will fix the NPE issue to get main class asap, will update here once we have any updates

could you please help check is there any fat jar in your target directory? If not, you may use spring-boot-maven-plugin to generate one, you may refer document here.

@Flanker32 There is a fat jar, else your code would throw differently. it is generating using spring-boot-maven-plugin, but with true</ executable> (as suggested by the pointer to https://stackoverflow.com/questions/37783069/why-cant-i-extract-a-spring-boot-executable-jar).

https://docs.spring.io/spring-boot/docs/current/maven-plugin/reference/htmlsingle/#goals-repackage-parameters-details-executable

(I suppose this is a normal migration step from AppService (with a custom Dockerfile, referring to an entrypoint.sh script starting sshd then the fatjar with executable=true)).

@blacelle

The NPE is unquestionably a bug, we'll fix it in the next release.
It may not startup normally even if it's successfully deployed to Azure Spring Cloud because I guess Azure Spring Cloud does not support this format of jar. (We'll confirm with Azure Spring Cloud team about this, @yuwzho can you help?)

BTW, is it necessary for you to run it directly? is java -jar not acceptable, can you try Spring Boot with Docker?

BTW, is it necessary for you to run it directly? why is java -jar not acceptable?

It is acceptable as a general rule of thumb, but not for our current way of deploying to AppService -> during the migration, we need to have both deployment OK (AppSerice and SpringCLoud) -> it is totally OK to require executable false (for SpringCloud) ; I suppose azure-spring-cloud documentation should be improved, and maybe the maven plugin should reject them explicitely.
-> For my own case, this can be closed. However, the doc + maven plugin code can be improved for other people in my case.

@blacelle

I suppose azure-spring-cloud documentation should be improved, and maybe the maven plugin should reject them explicitly.

You are right, we'll fix the NPE and improve the error message, thanks

hi, @blacelle , we have fixed this issue, and the fixed be included in the coming released.
you can try on branch endgame-202204.deprecation, the updated version of the plugin is 1.10.0


feat(consensus): consensus default config enables single node execution

This change is

#1373  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on  Graphite

Merge activity

Oct 15, 7:50 AM EDT: A user started a stack merge that includes this pull request via Graphite.


Add fake ServerFactory

This aligns with the ClientFactory type generated for ARM SDKs.
Fixes https://github.com/Azure/autorest.go/issues/979

@tadelesh FYI


Add CMake option to toggle stack protection

Using the MinGW-W64 compiler on Windows results in a segfault when a database is initially opened. This seems to be the result of the -fstack-protector flag, which has patchy support in MinGW.
Note some other major projects that have encountered this, and don't add this flag when compiling with MinGW on Windows:
https://github.com/bitcoin/bitcoin/issues/8732
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86832
I can reproduce this with the example program and embedded sqlite3 library -- Building with MSVC works fine, and building with MinGW-W64 with the flag removed works fine, but segfaults at sqlite3_open_v2() with the flag added.
This PR adds a CMake option, SQLITECPP_USE_STACK_PROTECTION, defaulting to enabled, which allows disabling of this flag. Some other pull requests on this project to improve hardening (see #262) also mention this flag. In KOLANICH's PR, he excludes this flag unless the platform is Linux. I think that a good short-term solution short of refactoring all the hardening flags is to just keep this behind an option and document it with a comment in the CMakeFile.

Coverage remained the same at 100.0% when pulling 1aa82864ceb6c567189b8cbd78d7ec7e3e6f2cd0 on chrisdalke:master into f757b640b1a900544241346fe4711de7cb5c8ffd on SRombauts:master.

Thanks a lot, I'll also have a look at the aforementioned PR!
Cheers!
Sébastien


RxJs as a side-effect free library

RxJs is marked as being free from side effects in its package.json:
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/package.json#L6
For dead code elimination purposes, being side effect free means that any call to a function in this library whose value is not saved, returned, or thrown can be dropped.
In other words, since nothing was done with the result of the function call, and the library has no side effects, calling or not calling that function is the same.
While doing some testing on this topic I found a couple of places where RxJs seems to have side effects, causing code to be dropped:

these lines are dropped since _enable_super_gross_mode_that_will_cause_bad_things is false at the module toplevel, and calling config.useDeprecatedSynchronousErrorHandling =  true should have no side effects
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/config.ts#L23-L28

the following function calls are dropped because nothing is done with their return value
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/Immediate.ts#L16
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeTo.ts#L18-L19
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToArray.ts#L9
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToArray.ts#L12
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToIterable.ts#L9
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToIterable.ts#L12
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToIterable.ts#L20-L24
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/util/subscribeToPromise.ts#L5-L14
https://github.com/ReactiveX/rxjs/blob/d7bfc9ddff01bd65c91ed3e4e6cc6930d4ceeae7/src/internal/Observer.ts#L12

Additional information:
I came across this while testing Build Optimizer with ES2015 modules (https://github.com/angular/devkit/pull/981) in Angular.io. The version of RxJs used there is 6.0.0-uncanny-rc.7.
When using Build Optimizer with ES2015 modules on Angular.io, the page would load, not show any console errors, but remain in a permanent content loading state with a red loading bar at the top. Top nav bar would show but no content was loaded beneath it.
The specific dead code elimination that causes Angular.io to break is the dropped lines in src/internal/util/subscribeTo.ts.
I do not know why this is not a problem with ES5 code.
cc @IgorMinar @benlesh @jasonaden @hansl @gkalpak

Theory: Isn't the problem in that setters/getters and lambdas are considered by build optimizer as top level functions? This is wrong. Once the code is transpiled into es5 they all become regular functions and then they are processed correctly.

@IgorMinar's theory was correct.
This stemmed from me misunderstanding the semantics of "sideEffects": false. I thought it meant library usage was free from side effects but it only means module loading.
Discussed this further with @IgorMinar and he explained it:

Pure for build optimizer and webpack means that loading the module has no non-local side-effects.
So if the code in a module is used loading the module will execute whatever is at the top level. If we stop using that module, we will stop executing the top level code. If this code had some side-effects outside of the module (monkey patching dom, registering with a global registry, etc) then we could be breaking stuff.
If however the side effects were local to that module (e. g. Monkey patching a class defined in the same module) then loading or not loading the module has no observable side effects outside of that module.
And in those cases we can remove the code without anyone noticing.


Gdax to Coinbase Pro

Are you guys going change API URL from Gdax to Coinbase Pro?

I forked and made a one line change.  Everything works 'as is' so far.
coinbase-exchange-ruby/lib/coinbase/exchange/api_client.rb
        @api_uri = URI.parse(options[:api_url] || "https://api.pro.coinbase.com")

Could you please send a Pull Request. Thanks.


Solves Issue #222 (documenting move to Redux)

I was scared when reading the documentation that I'd have to bloat my dependencies to flux and Redux 👍 but don't have to 🍒 !

Thanks for doing this!


WIP - Enable MongoDB replica set for the results cache and initialize it when starting the package.

Description

Validation performed

Search query performance compassion
With & Without the replica set enabled, performed a search query in the WebUI with a screen recording of 30fps, and extract below times:

T0 (relative 0 second): the query was started.
T1: the first batch of results were available and displayed in the browser.
T2: the total number of results are available (either from the results-metadata or summed from the aggregated time buckets) and displayed in the browser.
T3: the job is marked as done and displayed in the browser.

| T0 | T1 | T2 | T3
-- | -- | -- | -- | --
Trial 1 (with replica-set) | 0 | 0.7 | 1.266667 | 2.1
Trial 2 (with replica-set) | 0 | 0.633333 | 1.433333 | 2.166667
Trial 3 (with replica-set) | 0 | 0.433333 | 1.4 | 2.1
Trial 4 (w/o replica-set) | 0 | 0.466667 | 1.433333 | 2.1
Trial 5 (w/o replica-set) | 0 | 0.5 | 1.433333 | 2.066667
Trial 6 (w/o replica-set) | 0 | 0.6 | 1.433333 | 2.2

The speed performance does not seem to significantly differ with and without replica-set enabled.

An issue is found when in clp-config.yml I configure a non-27017 port on host localhost for the replica set:
pymongo.errors.OperationFailure: No host described in new configuration with {version: 36921, term: -1} for replica set rs0 maps to this node, full error: {'ok': 0.0, 'errmsg': 'No host described in new configuration with {version: 36921, term: -1} for replica set rs0 maps to this node', 'code': 74, 'codeName': 'NodeNotFound'}

However, such issue is only observed on localhost hosts. If the host is not localhost / <IP_ADDRESS> (e.g., my WSL's virtual eth0 interface gets assigned <IP_ADDRESS>), with any port the replica set initialization would succeed and there is no issue connecting to the clp-configured mongo address.
By looking into the community Mongo server source code, it was found they have two ways to check whether a node is up before the server configures itself as a replica node: isSelfFastPath and isSelfSlowPath.

isSelfFastPath is purely logic based as it tries to check identity of the server's address with a given config's host and port. It first checks if the port matches then it checks for host.
isSelfSlowPath handles the case that the fast path cannot find a match of the server's address in any of the configs. It tries to make a connection with the given config host and port and issue a command to check for identity.

Why localhost: any non-27017 port fails
Since our mongo server instance runs within a Docker container with only the docker's localhost's port 27017 mapped to the Docker host, if we request replica set to be configured on localhost:27017, the fast path check should succeed; if we request for localhost: any non-27017 port, the fast path wouldn't find a match because the server is started with 27017, and the slow path wouldn't find a match because in the Docker container's own network, the Docker host's port (configured by clp-config and used in the replica set config) is not reachable.
Why any non-localhost host: any port works
The fast path would fail but the slow path would succeed as Docker knows non-localhost is outside of its network and Docker routes the connection to the Docker host.
Proposed solution

Create a docker network and put all services (docker containers) that use Mongo into the same network.
docker run the result cache with --network "host" so that the mongo Docker container sees / exposes everything on the Docker host.

Considering existing CLP utilities, Option 2. seems the quickest way. @haiqi96 may comment on the impacts on CLP cloud.

You may also see from the {version: 36921, term: -1} in the error message that they may have some uninitialized memory issue, lol


CSI Serial Access

This patch provides serial access to volume resources via the CSI module to both the libStorage bridge and any native CSI plug-ins.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1000      +/-   ##
==========================================
- Coverage   34.15%   34.11%   -0.04%
==========================================
  Files          36       36
  Lines        2896     2896
==========================================
- Hits          989      988       -1
- Misses       1805     1806       +1
  Partials      102      102

Impacted Files
Coverage Δ

libstorage/api/types/types_localdevices.go
86.76% <0%> (-1.48%)
:arrow_down:

Continue to review full report at Codecov.

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


Show all PDF authors in case there are no affiliations available

we need a little hack to display all authors in case there have been no affiliations specified. the authblk library needs it, apparently, otherwise it uses the standard latex methodology, which would be like \author{author1 \and author2 \and ...}
fixes #265

closing this in favor of #268


Change Server Salt References to Service Salt

Description Of Changes

Changed the name of the Variable being set in Set-SslSecurity.ps1 from ServerSalt to Service Salt so the Register-C4bEndpoint.ps1 script gets generated with the correct parameter name.
Changed $ServerSalt variable in ClientSetup.ps1 script to be $ServiceSalt so it matches the name of the config setting it applies to more accurately.
Also made one note change in ClientSetup.ps1 to also reflect correct config setting being set.

Motivation and Context
Made sense to change all the Server Salt references to Service salt since the actual config setting this deals with is named centralManagementServiceCommunicationSaltAdditivePassword and not server.
Testing
Change Types Made

[X] Bug fix (non-breaking change)
[ ] Feature / Enhancement (non-breaking change)
[ ] Breaking change (fix or feature that could cause existing functionality to change)
[X] PowerShell code changes.

Related Issue
Fixes #114
Change Checklist

[ ] Requires a change to the documentation
[ ] Documentation has been updated
[ ] Tests to cover my changes, have been added
[ ] All new and existing tests passed.
[ ] PowerShell v2 compatibility checked.

LGTM!


User-defined types - How to define property with any name

Is your feature request related to a problem? Please describe.
Let's say I have type that is object similar to tags. I want to be able to define that tags can accept any property name but the value for that property must be string. May be something like this
type tagsType = {
  *: string
}

Is there some way to define this?
Describe the solution you'd like
see above.

This feature was implemented in #9511 and will be included in the next release.


Maintenance

Changes:

Modify comments - remove unnecessary, change grammar
Modify code style in app.py
Add basic unit test support for Flask app.py

LGTM!


The dashboard menu to select 'stages' is broken

On https://flutter-dashboard.appspot.com/#/build

Click the gear icon and uncheck the Stages, and click apply.
You see all results disappear as expected.
Click the gear icon and check the Stages, and click apply.
While your selecting the checkbox you see the results appear, but when you click the apply button, they don't appear.

Note that the query parameters remain as false for the settings in the URL.

\cc @CaseyHillers @XilaiZhang
possibly related to https://github.com/flutter/cocoon/pull/1744

\cc @codebird

If you manually remove the url query parameter, it works as expected, so that part of the plumbing seems to be good.

I'll check it out


Tries to open nonexistent .d file in LLVM ASM build

Hi,
Another weird one (sorry!). After #78 was fixed I was able to build .cpp files from the LLVM repo on Windows, but I run into problems with ASM files, here is the error:
read lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir\blake3_avx2_x86-64_windows_msvc.asm.obj.d: The system cannot find the file specified. (os error 2)

Here is the command invocation:
C:\PROGRA~1\MIB055~1\2022\PROFES~1\VC\Tools\MSVC\1436~1.325\bin\Hostx64\x64\ml64.exe -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_HAS_EXCEPTIONS=0 -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -IC:\code\llvm\llvm-project\out\debug\lib\Support\BLAKE3 -IC:\code\llvm\llvm-project\llvm\lib\Support\BLAKE3 -IC:\code\llvm\llvm-project\out\debug\include -IC:\code\llvm\llvm-project\llvm\include  -c -Fo lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir\blake3_avx2_x86-64_windows_msvc.asm.obj C:\code\llvm\llvm-project\llvm\lib\Support\BLAKE3\blake3_avx2_x86-64_windows_msvc.asm
failed: Building ASM_MASM object lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir\blake3_avx2_x86-64_windows_msvc.asm.obj

I am guessing that n2 tries to find the dependency file for the .asm file, but ml64 doesn't output (maybe no assemblers do?). Hopefully an easy fix.

Any thoughts on this one @evmar ?

Sorry for not responding!  Can you find the relevant snippet of the build.ninja file that runs that command?  n2 is only supposed to look for that .d file if the build.ninja mentions a depfile attribute, maybe something is going wrong in how n2 is parsing the file?

(Alternatively, you could attach your entire build.ninja file and I could take a look)

Ping @tru :)

I'll wait for someone else to repro, no prob.

Hi! So sorry I didn't reply, it's been busy days for me. But I pulled latest from the n2 repo and reproduced it with llvm-project

n2.exe installed as ninja.exe
ran ninja LLVMSupportBlake3

C:\PROGRA~1\MIB055~1\2022\PROFES~1\VC\Tools\MSVC\1437~1.328\bin\Hostx64\x64\ml64.exe -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_GLIBCXX_ASSERTIONS -D_HAS_EXCEPTIONS=0 -D_LIBCPP_ENABLE_HARDENED_MODE -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -IC:/code/llvm/llvm-project/out/debug/lib/Support/BLAKE3 -IC:/code/llvm/llvm-project/llvm/lib/Support/BLAKE3 -IC:/code/llvm/llvm-project/out/debug/include -IC:/code/llvm/llvm-project/llvm/include  -c -Fo lib/Support/BLAKE3/CMakeFiles/LLVMSupportBlake3.dir/blake3_sse41_x86-64_windows_msvc.asm.obj C:/code/llvm/llvm-project/llvm/lib/Support/BLAKE3/blake3_sse41_x86-64_windows_msvc.asm
failed: Building ASM_MASM object lib/Support/BLAKE3/CMakeFiles/LLVMSupportBlake3.dir/blake3_sse2_x86-64_windows_msvc.asm.obj

Attached is the build.ninja file.
build.ninja.zip

@evmar did you see my reply on this one? I just retried it today and it still have the same problem. I can provide new files if you want.

I didn’t, thanks for pinging oops!

It looks like the relevant snippet is
build lib/Support/BLAKE3/CMakeFiles/LLVMSupportBlake3.dir/blake3_sse2_x86-64_windows_msvc.asm.obj: ASM_MASM_COMPILER__LLVMSupportBlake3_unscanned_Debug C$:/code/llvm/llvm-project/llvm/lib/Support/BLAKE3/blake3_sse2_x86-64_windows_msvc.asm || cmake_object_order_depends_target_LLVMSupportBlake3
  DEFINES = -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_GLIBCXX_ASSERTIONS -D_HAS_EXCEPTIONS=0 -D_LIBCPP_ENABLE_HARDENED_MODE -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
  DEP_FILE = lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir\blake3_sse2_x86-64_windows_msvc.asm.obj.d
  INCLUDES = -IC:/code/llvm/llvm-project/out/debug/lib/Support/BLAKE3 -IC:/code/llvm/llvm-project/llvm/lib/Support/BLAKE3 -IC:/code/llvm/llvm-project/out/debug/include -IC:/code/llvm/llvm-project/llvm/include
  OBJECT_DIR = lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir
  OBJECT_FILE_DIR = lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir
  TARGET_COMPILE_PDB = lib\Support\BLAKE3\CMakeFiles\LLVMSupportBlake3.dir\
  TARGET_PDB = ""

but I don't see the definition of ASM_MASM_COMPILER__LLVMSupportBlake3_unscanned_Debug here, I think it might be in one of the adjacent .ninja files CMake generates?

@evmar i don't have access to a computer right now, but it just does depfile = $DEP_FILE for rule definition. I then compared how ninja handled this and stepped through it in the debugger and noticed that it just ignored the not found error. So I think this is the "right" fix.

Some notes from random digging:

it seems like the ASM_MASM related logic is part of CMake, not LLVM specific
some discussion of depfiles with assembly here https://gitlab.kitware.com/cmake/cmake/-/issues/20426
as best as I can gather, cmake basically unconditionally sets DEP_FILE, while masm is the MS assembler that doesn't support writing depfiles

So I conclude treating missing files as empty is more or less right.


feat(custom-pu-grid): custom pu grid geometry

unpacking shapefile and adding geometries - https://github.com/Vizzuality/marxan-cloud/pull/525 (this)
exposing endpoint to submit & api events - https://github.com/Vizzuality/marxan-cloud/pull/527
exposing endpoint to get the request status - https://github.com/Vizzuality/marxan-cloud/pull/528
updating project planning_unit_grid_shape - https://github.com/Vizzuality/marxan-cloud/pull/529

In terms of validation, it is part of the characteristic of a grid that their members are adjacent geometries. in order to validate it, they should not intersect with each other, but every single geometry should be intouch (connected) with at least one. If we want to do that validation with javascript, i think turf will allow you to intersect and check if the result is a line in order to ensure that they only share borders.

Can we validate this assumption with postgis?

In terms of validation, it is part of the characteristic of a grid that their members are adjacent geometries. in order to validate it, they should not intersect with each other, but every single geometry should be intouch (connected) with at least one. If we want to do that validation with javascript, i think turf will allow you to intersect and check if the result is a line in order to ensure that they only share borders.

Can we validate this assumption with postgis?
would https://postgis.net/docs/ST_IsSimple.html be fair enough (not sure about mentioned tangency) - checked, seems like not, as touching edges make it "invalid"
https://postgis.net/docs/postgis_usage.html#eval_spatial_rel

not something I have much experience with, but I would be surprised if this wasn't the case, and it would be much more preferable than doing this at the API layer. We can deal with this in a separate validation story, if effectively needed.


nuitka compiled exe in Windows is indentified as a virus/trojan

Any Onefile compiled python code in Windows is identified as a virus/trojan by avast and many others.
Some standalone builds are also reported as viruses/Trojans but not all seems depending on specific code compiled
I know that there are already reported similar issues but they're old, and some does not give code examples

Nuitka version, full Python version, flavor, OS, etc. as output by this exact command.

python -m nuitka --version

1.9.7
Commercial: None
Python: 3.8.18 (default, Aug 28 2023, 15:16:02) [MSC v.1900 64 bit (AMD64)]
Flavor: CPython Official
Executable: C:\Program Files\Python38\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: ~\gcc (gcc 12.1.0).

Actually it's Windows 11 Pro

How did you install Nuitka and Python
Using installer from here
https://github.com/adang1345/PythonWindows/blob/master/3.8.18/python-3.8.18-amd64-full.exe
Here are it virustotal report
https://www.virustotal.com/gui/file/c61ab0ecc6a529492015d29c3c9e15e5d4340e78c7e232fe59b0d5d870eb02e2

The specific PyPI names and versions
It should be taken from this output if there specific packages involved, e.g. numpy, you are
expected to shorten this to the relevant ones.

python -m pip freeze

gmsh==4.12.0
Nuitka==1.9.7
numpy==1.24.4
numpy-stl==3.1.1
ordered-set==4.1.0
parea==0.1.1
psutil==5.9.7
PyQt5==5.15.10
PyQt5-Qt5==5.15.2
PyQt5-sip==12.13.0
python-utils==3.8.1
screeninfo==0.8.1
shapely==2.0.2
trimesh==4.0.8
typing_extensions==4.9.0
zstandard==0.22.0

Many times when you get an error from Nuitka, your setup may be special
Any python code compiled as OneFile in Windows is reported  as virus/trojan. If the same file is build as standalone no problem is reported when running with anti-virus installed.

Also supply a Short, Self Contained, Correct, Example
Call this code mypython_version.py and build it as a very simple example:

import sys

print("Python version:")
print(sys.version)

Provide in your issue the Nuitka options used
python -m nuitka --onefile --remove-output mypython_version.py

Here is its virustotal report
https://www.virustotal.com/gui/file/bbd9f9e104cf6bc334aa57dcf002b5ca619fa3c97f8ff540a9734d0e4a1e71d5
Compiling same as a standalone, include folder in a zip file upload to virustotal shows a similar problem too:
https://www.virustotal.com/gui/file/bdf60fcdca73e9f4deb58ec6ec720560ea772bf5da185a0e893a592d44e74779
Uploading a single exe from inside standalone build:
https://www.virustotal.com/gui/file/f2dc4a670a46ba80a6ad9a5f61a07c543dd8677dfa0bc2da660bb37fd3be6597
This is a another python example that's not reported a virus if built standalone for major anti-virus program like avast despite having more complex code and dependencies, report is much cleaner but not 100% on virustotal. But it's reported so if built as onefile:
Call it p-area.py and compile as python -m nuitka --standalone --remove-output --output-filename=parea p-area.py or python -m nuitka --onefile --remove-output --output-filename=parea p-area.py
# -*- coding: utf-8 -*-
import re
import sys
from parea import main
if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

Here is its standalone report
https://www.virustotal.com/gui/file/5e032f573c07c625304dcc1b076e49c79a09a7497bf6e19aa6e93e4585559eee
Here is its onefile report
https://www.virustotal.com/gui/file/3c7b754e63968d032332b87f158cf378021470484941ae5d22759eb2fff72991

pip uninstall zstandard
Then manually pack with upx.

Nuitka is the preferred choice of malware authors due to its protection of the source code, and the inability to investigate behavior of the program without deep analysis. This leads to Nuitka compiled binaries being reported as malware based on being similar to it, even for small programs, esp. for programs that use Crypto libraries and that deal with Crypto payments in particular.
There are ways around this, but further action will be needed, they are described here: https://nuitka.net/doc/user-manual.html#windows-virus-scanners
Naive compilation however is exactly what malware authors do, so it's not going to be sufficient to get your binaries not flagged.

pip uninstall zstandard

compile manually with nuitka.
Then manually pack with upx.

@7gxycn08
hi  how to compile ? with onefile o standalone? if standaalone how pack to one file all dll and exe ?


Avantis Soft Keys

The Avantis module has in its list to control soft keys, but there are not any selectable controls for soft keys.  Is this being developed at all.  The avantis does not have macros and soft key control would really help with creating usable macros in Companion.

Hi, i am busy rewriting it to work with the latest version of the companion software. It is a work in progress currently as i can only test with the actual device like once a week every other week. i will see what i can do.

Hi, is there any update on this issue?


Include Oxyplot as a submodule instead of NuGet package

https://github.com/oxyplot/oxyplot-xamarin/issues/100#issuecomment-572769837

I recommend including the OxyPlot Xamarin as a git submodule/subtree and building these projects yourself from the latest source code.

Sorry, but I don't think we should take the responsibility to build it ourselves.
Fabulous.XF extensions are meant only as wrappers around external packages maintained by others.
We already have enough to do with Fabulous alone.

external packages maintained by others

But the package is currently unmaintained which is over 3 years old now. The submodule approach is the maintained alternative.

As per https://github.com/oxyplot/oxyplot-xamarin/issues/84, I only really see it as a way to offload the maintenance to whoever uses the package because they don't have the time/people willing to maintain it properly and make sure it works on all platforms. Which is fine.
But if we were to build and release an OxyPlot.XF package, we would take all the responsibilities:

Set up all the build and release processes
Make sure it works on all platforms
Fix bugs and update it every now and then
Also people would come here to file issues and features for OxyPlot.XF

And those aren't the goals of Fabulous.
We only aim to provide a compatible wrapper to make OxyPlot usable in Fabulous.
If the wrapped package ends up unmaintained, people are free to participate to the open-source repository of OxyPlot and release a new version there. Fabulous won't do it for them.


HSV doesn't seem to work

Always 0, 100, 100, 1 no matter what:

Even in simplest case where no parameters passed at all:
<script lang="ts">
  import ColorPicker from 'svelte-awesome-color-picker'
</script>

<ColorPicker />

rgb, hex – they work tho

Can you please provide a Repl and the os/browser you are using?
It is working fine here: https://svelte.dev/repl/aab96e19ae3e4b96a592322497b232a7?version=3.59.1

By some reason, it works for me in Repl, but doesn't work when I run a pretty simple Svelte application dev server. Here's gif:

OS: Windows 10, version 22H2, build 19045.4046
Browser: MS Edge Version 122.0.2365.52 (Official build) (64-bit)

BTW:

HSV works perfectly fine in 3.0.0-beta.8 somehow, and it worked fine until 3.0.0-beta.11
3.0.0-beta.12 is non-existent so idk, but in 3.0.0-beta.13` it's broken since then

I'm using<EMAIL_ADDRESS>so maybe that's related.

Just checked with<EMAIL_ADDRESS>– it's compatible with the latest library 3.0.5
So, it's incompatibility with Svelte 5, indeed.

I'm closing this issue in favor of the issue #69


Feature: sort interface properties, object keys and array values if code has annotation

Describe the rule
There is existing package that implements similar feature: https://github.com/ronparkdev/eslint-plugin-annotation
I personally think it would be great if eslint-plugin-perfectionist provided thie rule since it's also related to sorting.
Code example
|  |  |
Additional comments
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Currently, you can use ESLint comments to define plugin rule settings.
/*
  eslint perfectionist/sort-objects: [
    'error',
    {
      type: 'alphabetical',
      order: 'asc',
      partitionByNewLine: true
    }
  ]
*/

Using annotations looks beautiful and convenient. But I'm not sure it will be in demand.
I would like to hear more opinions from other users.

I understand the use case for sort-array-includes: it would allow users to sort arrays who do not have .include if the user wishes.
However, I'm not sure to understand the use case with the other rules such as sort-objects: all objects get sorted if the rule is active, why would an annotation be needed?

@sardor01 Perhaps the ESLint comment solves your problem?


Incorrect guidance - 'choose Reload project'

Under section title Set properties, edit, build, and debug it says:
After you have set up a Clang configuration, right-click again on the project node and choose Reload project.
The Reload Project options are not available at this point, because the project was never unloaded previously.  The changes mentioned do not cause the project to be unloaded either, if there is such a thing.
Since the need for a reload is implied in the text, I unloaded and reloaded.

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

ID: 47cabdd4-5ef1-1c19-4a82-5448a5901e8a
Version Independent ID: 56fd7d96-62ec-07df-40df-aaf155d504d3
Content: Clang/LLVM support in Visual Studio projects
Content Source: docs/build/clang-support-msbuild.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Moved this issue to an internal work item tracking system because we are moving away from using GitHub issues to track C++ doc work.


Change custom_error_response variable type

https://github.com/terraform-aws-modules/terraform-aws-cloudfront/blob/ddf49a7b736cf25057f716e13f7c11a193624ac9/variables.tf#L118-L122
I think that would be better having the right type for the variable custom_error_response, since I didn't find any example using this variable.

This example should help
custom_error_response = {
    error404 = {
      error_code            = 404
      error_caching_min_ttl = 300
      response_code         = 200
      response_page_path    = "/index.html"
    }
  }

Yes, also there can be multiple custom_error_response values in a map as @cageyv showed.
Closing the issue.


Multiple refactorings

Bad practice to do such a huge PR, but it just kinda turned out this way.
The aim of this PR is to refactor stuff better. I started with pretty conservative aims but ended up doing a lot of work. I made sure there are very few changes that affect any interfaces or functionality.

All parsers have been implemented as regular classes, none of that inline class thing that was happening.
Imports have been optimized
Got rid of lodash as a dependency and just bundled the code that was needed.
Fixed the circular dependency issue #37
Get rid of some namespaces
Implement some combinators using advanced TypeScript features instead of long lists of overloads (Or, Then)
Clean up the doc comments at the top of each file... they were there for a reason but they are ugly

I see there are more CI issues. I'll figure them out

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-35 of 709 (95.06%)  changed or added relevant lines in 45 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.5%) to 92.065%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/lib/internal/combinators/many1.ts
28
29
96.55%

src/lib/internal/combinators/reason.ts
13
14
92.86%

src/lib/internal/combinators/recover.ts
14
15
93.33%

src/lib/internal/combinators/replace-state.ts
15
16
93.75%

src/lib/internal/combinators/stringify.ts
19
20
95.0%

src/lib/internal/combinators/then-pick.ts
15
16
93.75%

src/lib/internal/functions/index.ts
7
8
87.5%

src/lib/internal/parsers/char-code-where.ts
17
20
85.0%

src/lib/utils.ts
21
46
45.65%

Totals

Change from base Build<PHONE_NUMBER>:
2.5%

Covered Lines:
1138

Relevant Lines:
1217

💛  - Coveralls

Oh yes, really happy to see this!

All parsers have been implemented as regular classes, none of that inline class thing that was happening.

👍🏻  Love it. Maybe less surprising to see and work with, although personally didn't have an issue with this.

Imports have been optimized

👍🏻
suggestion(optional): how would you feel about automating this with some kind of tool? I have used these at work in various projects:

https://www.npmjs.com/package/prettier-plugin-organize-imports is a prettier plugin, zero config, can be annoying if you have auto save on editor defocus on (although you can always undo)

(I think if you like to group the imports it's still possible by using empty lines between groups)

hmm I think there was another one but I forgot which one it was 😄

Got rid of lodash as a dependency and just bundled the code that was needed.

👍🏻

Implement some combinators using advanced TypeScript features instead of long lists of overloads (Or, Then)

Absolutely love this 👍🏻

About the optimize imports plugin: Yes! I had no idea something like that existed. Thank you. I'll also integrate it at work.


Use inclusive words - denylist / allowlist

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: #1214
Pull Request Description
Use inclusive words(Change blacklist/whitelist to denylist/allowlist).

I guess we should change DenyLists, because Deny is offensive to all people.
Deny is an album of Ex Pistols band.
It contains a song with a title "Nicotine Addict".
We should change it to ForbidList.
Also "Allow" is not a good term.
Allow has a meaning of granting freedom to someone.
"to allow a servant his liberty".
We should change it to AcceptList, I guess. Or PermitList.

@lolgear
Thanks! Your explanation made sense to me(but I didn't know the album of Ex Pistols band 😅).
And I prefer AcceptList to PermitList.
@CocoaLumberjack/collaborators
Any other opinions? If there are no opinions after waiting for a while, I'll use forbiddenList/acceptableList.
Thanks in advance.

@sushichop
Sorry!
I thought it was obvious and will be treated as a joke :)
( But "Allow" is still something that "master-slave" is ).

@ffried
Others are illogical.
Because Whitelisting is...
"Whitelisting (also referred to as allow-listing[1]) is the practice of explicitly allowing some identified entities access to a particular privilege, service, mobility, access or recognition."
Changing a name doesn't changing a meaning.

I have updated PR and resolved conflict.
I used typedef and category for deprecated classes and methods.
As a result, when we remove the deprecated ones in the future, I think we should only need to delete deprecated {h,m} files and remove #import <CocoaLumberjack/DDContextFilterLogFormatter+Deprecated.h> from CocoaLuimberjack.h .
Thanks.

Thanks is mine🙂


fix(headers): remove extra $ symbol in require

Sorry, due to my mistake, this PR merge was lost. I will make this modification again on the new commit. I will still use the changes of your PR, please see here for details.


Attention is all you need  dataset wmt 2014 english-german dataset

Requirements

[ ] Read about input embeddings technique (byte-pair encoding) used by Google's team on "Attention Is All You Need" paper.
[ ] Design the input embeddings pipeline for wmt 2014 english-german dataset

Attention is all you need  dataset wmt 2014 english-german dataset
Dataset: https://nlp.stanford.edu/projects/nmt/
from the paper:
Sentences were encoded using byte-pair encoding.
see Massive Exploration of Neural Machine Translation Architectures

Originally posted by @AlexisTercero55 in #11

References

Massive Exploration of Neural Machine Translation Architectures
Attention Is All You Need

Dataset structure
The wmt 2014 english-german dataset contains two text files that contains 4.5M sentence pairs of English-German:

train.en
train.de

Acoording "Attention is all you need" paper the python implementation of transformer architecture is placed in this repo/script

https://github.com/tensorflow/tensor2tensor/blob/master/tensor2tensor/models/transformer.py


[Question] Should "continue training" load the vecnormalize.pkl as well as the model.zip?

❓ Question
Hi,
I saw in zoo3 "continue training" will load the pretrained model.zip at the beginning.
I wonder should the corresponding vecnormalize.pkl file be loaded as well?
Thanks!
Checklist

[X] I have checked that there is no similar issue in the repo
[X] I have read the SB3 documentation
[X] I have read the RL Zoo documentation
[X] If code there is, it is minimal and working
[X] If code there is, it is formatted using the markdown code blocks for both code and stack traces.

Hello,
it should and it is:
https://github.com/DLR-RM/rl-baselines3-zoo/blob/2d5db11f6aee6d231d2154849715b0f7a9435038/rl_zoo3/exp_manager.py#L553-L563
In the worst case, statistics tends to quickly converge if started from scratch.


fix flaky python test

Issue Type

[x] bug report
[ ] feature request

Current Behavior
There is one python test that is quite flaky: test_should_not_allow_to_withdraw_inputs_and_outputs_when_ifes_processing_interchanges
https://app.circleci.com/pipelines/github/omisego/plasma-contracts/1977/workflows/2cf05327-796a-4bcc-9ee9-3723107f0741/jobs/10277
In some previous PR, we have experienced that some unrelated change causing the test to fail.
And see the slack discussion: here
Expected Behavior
Be stable
Suggested Fix
My current guess is that the priority does not strictly follow our first come first server order as the timestamp in blockchain is not really that reliable when two tx comes consecutively.
As a result, the order of priority queue might be a bit unexpected.

https://app.circleci.com/pipelines/github/omgnetwork/plasma-contracts/2323/workflows/ee053da0-4923-4dc9-a431-68a01bdd2205/jobs/12198
Started re-appearing.

sigh, anyone has idea what can we try? @kevsul @pgebal @thec00n

rechecked why previous attempt failed. During piggyback, it will enqueue with exit-able timestamp base on the following formula:
return uint32(Math.max(_blockTimestamp + (_calculator.minExitPeriod * 2), _now + _calculator.minExitPeriod));
So the problem is that it is using the _blockTimestamp + (_calculator.minExitPeriod * 2) part instead of _now + _calculator.minExitPeriod as the minExitPeriod (4min in python test setup) is longer than what I time travelled the ganache. Basically, the two competing txs are and should be of same priority 🤷♂.
In fact, the test setup does not makes much sense as they are two competing txs with one suppose to be canonical (as included in a block) and the other non-canonical. However, no challenge was involved in the test. I will scan through all tests in test_process_exits.py and come up with a proposal to update/remove/change this test or not.


Bug Azure Login (user locked by last company)

Hello,
I worked for one campany, who gives me the access to suscription Azure as User.
Then, I left for another campany, we work on Azure as Well. They give me access to their suscription azure, but I CAN'T Login
The problem is 'your account has been locked. Contact your support person to unlock it, then try again.'
I think my last company has locked my account, but who is the support person pls? When I login azure, It is directly redirected to my last company, how to swith to the new company, I have already the access for the new company.
If my last company lock my personal account, I can not login for the new company?
Thanks

I am pretty confident that they two company are on different tenant and so so one company deactivating an use account should not be an issue . I think you should work with your admin and if they want they can raise a support with Microsoft and that should help .

@himanshusinha-msft thanks for your response.
Two company have two different tenants. But who is my admin? It is my personal account.
When I login azure portal with this account, it is redirected to the auth page of my last company, and I can not event go to the portal, because my account is locked.
So why my personal account is locked, I can't event go to the portal page
Thanks


Add deleting and fix something else

Hi, i added deleting and adding languages from the GUI, changed something in the index etc. you can check it on the Video or ScreenShot. Be happy!
Some changes for #24
I commented annoying alert, do you think this is #11 ?

@themsaid Ok i tested it really advanced and is problem with scan keys for new language. If you will try scan keys for new language in the scanForKeys() action you have forEach function which is going to addNewKey, i don't know why, but if this keys was added in other language, you can't add keys for translate new language, because you will get alert with "This key already exists" and it will end it because  this is return.

I fixed empty field for creating new language and adding empty file witout name, mean .json.
I added new action addNewKeysForScan() for scanForKeys() - this actions conflicted when you tried scan keys for new language(displayed information about "this key already exists" many times as many was keys). We do not need check key exists for scaning keys for new language so i needed add new action for adding keys for new language. After testing everything working perfectly.

You can test it, it should working normally, i fixed everything what could be annoying (i hope so...).

+1

1

Sorry I know you've put much effort into this but I don't really want to allow removing languages from GUI, also on adding new language I don't want to force refresh the screen since users might be having unsaved keys.
Thanks again for your time :)


fix: speed up time to wallet signing

Time to sign popup:
before: 8328.5390625 ms
after: 6050.569091796875 ms
And some additional light touch code organization


test: add generator and namespace tests

What type of PR is this?
/kind feature

What this PR does / why we need it:
add generator and namespace tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.6%) to 71.242%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/parser/parser.go
3
74.36%

pkg/engine/operation/parser/spec_parser.go
4
56.1%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
5279

Relevant Lines:
7410

💛  - Coveralls


Maintain indentation when pasting in code

Hello,
It would be great if FlashDevelop could keep alignment of formatting when you paste a block of code. Problem is when you paste lets say 4 lines the top line will be in the correct position. But the following lines are an indentation back. It is such a small issue but would be great if it didn't do this:

Edit -> Smart Paste (Ctrl+Alt+V)


Fix cleanup of replace mode

Sorry I forgot to add following logic at cleanup method when I sended PR #15
if (task.getMode().equals(Mode.REPLACE)) {}

good catch:-) thank you.

Thanks you. I merged :+1:


Datasets should be in alphabetical order

When the user first lands on this page, the datasets should automatically appear in alphabetical order:

Same for when a user select a letter. Here the order should be: Afghanistan Records, Australia Department of finance, Australia New South Wales, Austria Digiwhist.

fixed

@t-mrazek @hrubyjan I'm testing again and this is still an issue. Is there something I'm doing wrong?

Checked again after Jakub pushed the fixes live. Looks good this time :)

Closing as Deployed.


find is return the same data,but what I want is got diffrient data

req.models[tablename].find({
fid: data.fid,
myid: data.myid
}, function(err, result) {
if (err) {
res.send(500, err);
} else {
console.log(eval(result.chat))
}
})
any problems with the code? I change the key to find diffrient data, the first time I got the data right。 but after it, it always return the first data I have got .

use diffrient keys but got the same data, why?


Only set CPUSchedulingPolicy once

As of af05e97 both CPUSchedulingPolicy and IOSchedulingClass were
disabled in the freeswitch systemd service for compatibility with LXD.
But however, the first option was then re-enabled in the config.yml,
triggering a restart of freeswitch every time.
But outside of virtualised environments, there is on need to remove the
CPUSchedulingPolicy option^1. And the IOSchedulingClass is set to
realtime by default^2, but was also disabled for the LXD
compatibility.
To stop freeswitch from being restarted when there is no actual
configuration change and dial back the removal of IOSchedulingClass the
tasks where consolidated into one task so it's idempotent.
The default value for bbb_freeswitch_ioschedule_realtime is set to
false by default for backwards compatibility with setups that require
this configuration.
Close #59

I am not sure, if the default of bbb_freeswitch_ioschedule_realtime you proposed should be the default. Since https://github.com/n0emis/ansible-role-bigbluebutton/pull/68/files#diff-f8020d83f5c78e15b9807b118f54493c15b8b217402b0c02fe644bc8a20202d4R208 deletes the line, this is a "breaking change", since freeswitch get's restarted.
Therefor I would propose to change the default of bbb_freeswitch_ioschedule_realtime to realtime, because this is also the default for freeswitch and is a sensible default. Although this would mean that any LXC-based deployments would break, but I think it would be sufficient to warn about this in the readme.


regression for Ocaml keyword match when upgrading to bucklescript 7.0.1

This looks to be the same problem as in https://github.com/facebook/reason/issues/2494 that @anmonteiro fixed for most keywords in https://github.com/facebook/reason/pull/2380.
  type match = {id: string};
  type matches = array(match);

results in
This type constructor's parameter, `match__`, can't be found. Is it a typo?

Not broken in master.


Update remix-dev to use isbot@4

Closes #8407

Oops - just saw #8385 - closing this in favor of that


Pointing out that the nabo matrix must stay in scope.

This cost me an entire day of looking for bugs. Grrr.

Can one of the admins verify this patch?

Yes, this is a very problematic thing as it is typically not expected like it is!
Btw. this was also the core behind https://github.com/ethz-asl/libnabo/issues/34 as it can also store the reference to a temporary, which will silently happen if implicit type casting is necessary (not exact type match). I should have explained that back then... Sorry.
I think the best would be to take a pointer instead of a reference or even a shared pointer. At least temporaries are then prohibited.
And I also believe it will make people more think about the expected live time of matrix, without looking into the documentation.
Talking about life time : I think the better term for the documentation would be that it needs to stay alive or in memory at least as long as the nabo object.
Staying in scope is only the appropriate way to achieve that for local variables.

ok to test

Alive it is. Good to got then? Thanks for clarifying about #34!

Yes, lgtm for the moment. For the future I added #39 .

:+1: :+1: :+1: thanks for caring!

Could you merge? I don't have the rights...


Convert curl syntax to Python, Node.js, PHP

I used link https://curl.trillworks.com/#python to convert curl command to python requests syntax. But it show me error "Error parsing curl command".
Syntax of curl command
curl https://android.googleapis.com/gcm/send -H 'Content-Type: Application/json' -H  'Authorization: key=sdfwer24s:345sdfwer25dfg9dVtpddf345dfg' -d '{"registration_ids":["sdfg34fsg3s:345fw4tsfgsdftg3w5esrert-345"], "data":{"data":"sdf23sdf dsf"}}'
There are problems  in AUTH_KEY and registration Ids, because they contains symbols :.

I am having trouble reproducing this issue.

Any more information you can give me?


Incorrect zero primal returned for ComplexF64 inputs

Now that #758 is fixed, if we use the same example another issues appears, where if the return activity is Const and the eltype is Float64, the primal value returned is 0.0:
julia> n = 10;

julia> x, y = randn(ComplexF64, n), randn(ComplexF64, n);

julia> ∂x, ∂y = map(zero, (x, y));

julia> _dot(x, y)
-1.628395690775859 + 1.9870438237721155im

julia> autodiff(ReverseWithPrimal, _dot, Const, Duplicated(x, ∂x), Duplicated(y, ∂y))
((nothing, nothing), 0.0 + 0.0im)

Strangely enough, this is fine if we use ComplexF32:
julia> x, y = randn(ComplexF32, n), randn(ComplexF32, n);

julia> ∂x, ∂y = map(zero, (x, y));

julia> _dot(x, y)
-0.91027236f0 + 0.34108874f0im

julia> autodiff(ReverseWithPrimal, _dot, Const, Duplicated(x, ∂x), Duplicated(y, ∂y))
((nothing, nothing), -0.91027236f0 + 0.34108874f0im)

Can you repost the entire reproducer again here for ease?

Sure, here it is:
using Enzyme, LinearAlgebra

_scalar_dot(x, y) = conj(x) * y

function _dot(X::StridedArray{T}, Y::StridedArray{T}) where {T<:Union{Real,Complex}}
    return mapreduce(_scalar_dot, +, X, Y)
end

function EnzymeRules.augmented_primal(
    config::EnzymeRules.ConfigWidth{1},
    func::Const{typeof(_dot)},
    RT::Type{<:Union{Const,Active}},
    X::Duplicated{<:StridedArray{T}},
    Y::Duplicated{<:StridedArray{T}},
) where {T<:Union{Real,Complex}}
    r = func.val(X.val, Y.val)
    primal = EnzymeRules.needs_primal(config) ? r : nothing
    shadow = EnzymeRules.needs_shadow(config) ? zero(r) : nothing
    tape = (copy(X.val), copy(Y.val))
    return EnzymeRules.AugmentedReturn(primal, shadow, tape)
end

function EnzymeRules.reverse(
    config::EnzymeRules.ConfigWidth{1},
    ::Const{typeof(_dot)},
    dret::Union{Active,Type{<:Const}},
    tape,
    X::Duplicated{<:StridedArray{T}},
    Y::Duplicated{<:StridedArray{T}},
) where {T<:Union{Real,Complex}}
    if !(dret isa Type{<:Const})
        Xtape, Ytape = tape
        X.dval .+= dret.val .* Ytape
        Y.dval .+= dret.val .* Xtape
    end
    return (nothing, nothing)
end

n = 10
x, y = randn(ComplexF64, n), randn(ComplexF64, n);
∂x, ∂y = map(zero, (x, y));
_dot(x, y)
autodiff(ReverseWithPrimal, _dot, Const, Duplicated(x, ∂x), Duplicated(y, ∂y))

x, y = randn(ComplexF32, n), randn(ComplexF32, n);
∂x, ∂y = map(zero, (x, y));
_dot(x, y)
autodiff(ReverseWithPrimal, _dot, Const, Duplicated(x, ∂x), Duplicated(y, ∂y))

Fixed by https://github.com/EnzymeAD/Enzyme.jl/pull/764

@wsmoses, I just checked, and on #main I still get the same issue (incorrectly returning a primal of 0.0) using the above reproducer.

Are you sure you're on the right commit? I reproduced locally and resolved via the PR, confirming success.

Add a PR from main with it as a test, and lets find out?

Are you sure you're on the right commit?

Yes, I confirmed.

Add a PR from main with it as a test, and lets find out?

Done! #769

Okay this PR should now fix: https://github.com/EnzymeAD/Enzyme.jl/pull/770


Simple.iOS.Sample Loading Placeholder is broken

I downloaded the code cause I was seeing this is my own project. But the same issue is here. Run the sample and view the output.
System.IO.FileNotFoundException: placeholder
at FFImageLoading.DataResolvers.BundleDataResolver+d__1.MoveNext () [0x006a9] in E:\Eds Projects\FFImageLoading-master\source\FFImageLoading.Touch\DataResolvers\BundleDataResolver.cs:140
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:187
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (Syst
em.Threading.Tasks.Task task) [0x0002e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:156
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:128
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1+ConfiguredTaskAwaiter[TResult].GetResult () [0x00000] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:535    at FFImageLoading.Work.ImageLoaderTask2+d__108[TImageContainer,TImageView].MoveNext () [0x001b1] in E:\Eds Projects\FFImageLoading-master\source\FFImageLoading.Common\Work\ImageLoaderTask.cs:380

I have essentially the exact same setup in my code. Had to roll back from 2.2.8.

updated to 2.2.9 from 2.1.8 and got the same. Placeholder on iOS from bundled resource is not working:
2017-03-11 17:36:27.106 FoodsageiOS[306:95014] Setting placeholder failed
System.IO.FileNotFoundException: Perfil_Placeholder
at FFImageLoading.DataResolvers.BundleDataResolver+d__1.MoveNext () [0x00517] in <8242b6346c404272a5f37bad37b4ee65>:0
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:187
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:156
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:128
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1+ConfiguredTaskAwaiter[TResult].GetResult () [0x00000] in <54e129cad3ee4c61a6b46c3680ddc69a>:0    at FFImageLoading.Work.ImageLoaderTask2+d__108[TImageContainer,TImageView].MoveNext () [0x001bb] in <2d17e89ae1144fbc85484e7be1030866>:0
2

I'm having the same issue. It happens to ErrorPlaceholder as well.
I think the error is related to Asset Catalogs. In the past this AssetCatalogDataResolver existed but it was deleted and now CompiledResources are handled by BundleDataResolver.
This is the stacktrace I'm getting:
Setting placeholder failed
System.IO.FileNotFoundException: ic_calendar
  at FFImageLoading.DataResolvers.BundleDataResolver+<Resolve>d__1.MoveNext () [0x00517] in <8242b6346c404272a5f37bad37b4ee65>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:187
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:156
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:128
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1+ConfiguredTaskAwaiter[TResult].GetResult () [0x00000] in /Library/Frameworks/Xamarin.iOS.framework/Versions/<IP_ADDRESS>/src/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:535
  at FFImageLoading.Work.ImageLoaderTask`2+<ShowPlaceholder>d__108[TImageContainer,TImageView].MoveNext () [0x001bb] in <2d17e89ae1144fbc85484e7be1030866>:0

Was this DataResolver intentionally deleted? Are AssetCatalogs really supported by BundleDataResolver?

Resolver is now here: https://github.com/luberda-molinet/FFImageLoading/blob/master/source/FFImageLoading.Touch/DataResolvers/BundleDataResolver.cs#L18
Also sample project was some broken code (legacy code), i fixed it but it still doesn't show placeholder. Do you have any idea why?

It should be fixed (newest prerelease), please reopen if you still experience it.


OptionsDialog and buttons CSS styling updates

This PR fixes #

CSS styling for options button, dialog header, checkbox alignment and option label text alignment etc

ScreenShots:

@ssreerama  which issue are you trying to fix? please add the screenshots that show the differences with vs without your changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 9 (77.78%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 42.121%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
0
1
0.0%

extensions/sql-database-projects/src/dialogs/publishOptionsDialog.ts
3
4
75.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
72.73%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
28171

Relevant Lines:
62400

💛  - Coveralls

@ssreerama which issue are you trying to fix? please add the screenshots that show the differences with vs without your changes

These styles are for the table row - checkbox and option label to align vertically middle. I'll try if i can apply these styling at extension level code. Thanks.

@ssreerama from the screenshots, the checkbox is still aligned to the top.

@ssreerama please have before and after screenshots so we can see the differences

@alanrenmsft That checkbox alignment is not fixed with this PR as this should be fixed on code side, which i'm planning to work on that but not able to put some time on that yet. will try if i can make some time to send a PR on core side.


Refactoring hermitian conjugation etc functions out of operator_utils

PR #632 has cleaned up most of operator_utils, but the hermitian conjugation routines remain there. I'd like to refactor them into SymbolicOperator/PolynomialTensor - perhaps as an op.hc() or op.herm() routine (kinda tempting to go the numpy route and make op.conj().T but the extra processing requirements to implement this function twice seems like a bad idea). Also, this will break a lot of code (both internally and externally), what do people think?
(I guess we can perhaps implement hermitian_conjugated as a function for a while and then depreciate it later.)
Starting this issue to discuss the best way to implement what is a pretty important routine --- it's probably best we get this right the first time.

Why not have hermitian conjugation as a util function and an abstract method on SymbolicOperator, where each subclass implements their own version of .hc()?


Progressive encoding JPEGs crashes CJXL

Describe the bug
Encoding .jpegs with the progressive option crashes cjxl.
To Reproduce
I'm attaching the .jpeg I used for the sake of documenting it, but this error occurs with many (all?) jpegs, on cjxl versions v0.8.1 c27d4992 and v0.9.0 6fb35528 (latest as of 5th March, 23). Binaries are static linux binaries from https://artifacts.lucaversari.it/libjxl/libjxl.
turtle.zip
$: ./cjxl turtle.jpg  turtle.jxl -v --progressive

JPEG XL encoder v0.9.0 6fb35528 [AVX2,SSE4,SSSE3,SSE2]
Note: Implicit-default for JPEG is lossless-transcoding. To silence this message, set --lossless_jpeg=(1|0).
Read JPEG image with 225719 bytes.
Encoding [Container | JPEG, lossless transcode, effort: 7 | JPEG reconstruction data],
./lib/jxl/enc_frame.cc:908: JXL_CHECK: enc_state_->passes.size() == 1
fish: Job 1, './cjxl turtle.jpg  turtle.jxl -v…' terminated by signal SIGILL (Illegal instruction)

Expected behavior
If progressively and losslessly encoding a .jpeg is not possible, cjxl should inform the user it's not possible and exit, instead of crashing.
Environment

OS: Linux pop-os 6.1.11-76060111-generic #202302091138~1675975749~22.04~f771a7f SMP PREEMPT_DYNAMIC Thu F x86_64 GNU/Linux
CPU type: x86_64
cjxl version string: JPEG XL encoder v0.9.0 6fb35528 [AVX2,SSE4,SSSE3,SSE2]

Related

https://github.com/libjxl/libjxl/issues/1871
https://github.com/libjxl/libjxl/pull/1814  (Suggests progressive .jpeg encoding is possible)
https://github.com/libjxl/libjxl/pull/1820

Perhaps I should explicitly note that lossy encoding jpegs works alright, even with progressive.
./cjxl turtle.jpg turtle.jxl -v --progressive -d 0.5 --lossless_jpeg=0

JPEG XL encoder v0.9.0 6fb35528 [AVX2,SSE4,SSSE3,SSE2]
Read 1920x1200 image, 225719 bytes, 248.5 MP/s
Encoding [VarDCT, d0.500, effort: 7],
Compressed to 269509 bytes (0.936 bpp).
1920 x 1200, 4.15 MP/s [4.15, 4.15], 1 reps, 12 threads.


arch: riscv: make arch_current_thread_set() safer

Use the do { } while (0) construct so the macro behaves like an actual
C statement in all cases. Also evaluate the macro argument only once.

@npitre I wonder if this caused this failure (or rather, timeout) in the renode-based benchmark here in the latest Twister integration build? https://github.com/zephyrproject-rtos/zephyr/actions/runs/12149971105/job/33881763419
INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) benchmark.sched_queues.scalable on m2gl025_miv/miv failed (Timeout)
INFO    -
INFO    - To rerun the tests, call twister using the following commandline:
INFO    - west twister -p <PLATFORM> -s <TEST ID> --no-detailed-test-id, for example:
INFO    -
INFO    - west twister -p m2gl025_miv/miv -s benchmark.sched_queues.scalable --no-detailed-test-id
INFO    - or with west:
INFO    - west build -p -b m2gl025_miv/miv tests/benchmarks/sched_queues -T benchmark.sched_queues.scalable
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - Run completed

I wonder if this caused this failure (or rather, timeout) in the
renode-based benchmark here in the latest Twister integration build?

Weird. This patch should be a no-op.
...
But it isn't. Compiled binary differs from before to after.
...
Hmmm. Very weird. I can't explain why the difference would cause some
failure yet, but if you have a renode setup please try this which brings
the build to what it was:
diff --git a/include/zephyr/arch/riscv/arch_inlines.h b/include/zephyr/arch/riscv/arch_inlines.h
index 2752b43d977..c97413a5f8f 100644
--- a/include/zephyr/arch/riscv/arch_inlines.h
+++ b/include/zephyr/arch/riscv/arch_inlines.h
@@ -33,7 +33,7 @@ register struct k_thread *__arch_current_thread __asm__("gp");
 #define arch_current_thread() __arch_current_thread
 #define arch_current_thread_set(thread)                                                            \
 	do {                                                                                \
-		_current_cpu->current = __arch_current_thread = (thread);                          \
+		__arch_current_thread = _current_cpu->current = (thread);                          \
 	} while (0)
 #endif /* CONFIG_RISCV_CURRENT_VIA_GP */

if you have a renode setup please try this which brings
the build to what it was:

Test passes with this change
$ west twister -p m2gl025_miv/miv -s benchmark.sched_queues.scalable --no-detailed-test-id
INFO    - Using Ninja..
INFO    - Zephyr version: v4.0.0-1449-gafba775304db
INFO    - Using 'zephyr' toolchain.
ERROR   - Platform nucleo_h745zi_q not found in the list of platforms
ERROR   - Platform stm32h745i_disco not found in the list of platforms
ERROR   - Platform nucleo_h755zi_q not found in the list of platforms
ERROR   - Platform stm32h747i_disco not found in the list of platforms
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/ycsin/zephyrproject/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  built (not run):    0, filtered:    0, failed:    0, error:    0
INFO    - 1 test scenarios (1 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
INFO    - 1 of 1 executed test configurations passed (100.00%), 0 built (not run), 0 failed, 0 errored, with no warnings in 70.43 seconds.
INFO    - 1 of 1 executed test cases passed (100.00%) on 1 out of total 879 platforms (0.11%).
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /home/ycsin/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/ycsin/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/ycsin/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed


issues with liquibase diff when testing with azure sql server 12 - Adventure Works  example database

Environment
Liquibase Version: 4.3
Liquibase Integration & Version: CLI
Liquibase Extension(s) & Version: CLI
Database Vendor & Version: MICROSOFT SQL SERVER 12.00.214
Operating System Type & Version: WINDOW 10 home 64 BIT
Description
A clear and concise description of the issue being addressed.
I have the following error when i tried to compare a sql server snapshot with a onlien database , the error are coming from this code
https://github.com/liquibase/liquibase/blob/4.3.x/liquibase-core/src/main/java/liquibase/util/ObjectUtil.java
https://github.com/liquibase/liquibase/blob/4.3.x/liquibase-core/src/main/java/liquibase/database/OfflineConnection.java
https://github.com/liquibase/liquibase/blob/4.3.x/liquibase-core/src/main/java/liquibase/structure/core/Data.java
i am updating the document.json in txt file
documento.txt.txt
i am giving all the error log but the important thing is in here
Caused by: liquibase.exception.LiquibaseParseException: liquibase.parser.core.ParsedNodeException: liquibase.exception.UnexpectedLiquibaseException: Cannot call public liquibase.structure.core.Index liquibase.structure.core.Index.setTable(liquibase.structure.core.Table) with value of type java.lang.String

[2021-06-30 08:31:51] GRAVE [liquibase.integration] Unexpected error running Liquibase: Cannot parse snapshot offline:mssql?snapshot=documento.json&catalog=samepldb&version=12.00.2148&productName=Microsoft SQL Server&caseSensitive=true
liquibase.exception.DatabaseException: liquibase.exception.UnexpectedLiquibaseException: Cannot parse snapshot offline:mssql?snapshot=documento.json&catalog=samepldb&version=12.00.2148&productName=Microsoft SQL Server&caseSensitive=true
        at liquibase.integration.commandline.CommandLineUtils.createDatabaseObject(CommandLineUtils.java:131)
        at liquibase.integration.commandline.Main.createReferenceDatabaseFromCommandParams(Main.java:2050)
        at liquibase.integration.commandline.Main.doMigration(Main.java:1484)
        at liquibase.integration.commandline.Main$1.lambda$run$0(Main.java:364)
        at liquibase.Scope.lambda$child$0(Scope.java:160)
        at liquibase.Scope.child(Scope.java:169)
        at liquibase.Scope.child(Scope.java:159)
        at liquibase.Scope.child(Scope.java:138)
        at liquibase.integration.commandline.Main$1.run(Main.java:363)
        at liquibase.integration.commandline.Main$1.run(Main.java:193)
        at liquibase.Scope.child(Scope.java:169)
        at liquibase.Scope.child(Scope.java:145)
        at liquibase.integration.commandline.Main.run(Main.java:193)
        at liquibase.integration.commandline.Main.main(Main.java:156)
Caused by: liquibase.exception.UnexpectedLiquibaseException: Cannot parse snapshot offline:mssql?snapshot=documento.json&catalog=samepldb&version=12.00.2148&productName=Microsoft SQL Server&caseSensitive=true
        at liquibase.database.OfflineConnection.<init>(OfflineConnection.java:111)
        at liquibase.database.DatabaseFactory.openConnection(DatabaseFactory.java:186)
        at liquibase.database.DatabaseFactory.openConnection(DatabaseFactory.java:175)
        at liquibase.database.DatabaseFactory.openDatabase(DatabaseFactory.java:140)
        at liquibase.integration.commandline.CommandLineUtils.createDatabaseObject(CommandLineUtils.java:96)
        ... 13 more
Caused by: liquibase.exception.LiquibaseParseException: liquibase.parser.core.ParsedNodeException: liquibase.exception.UnexpectedLiquibaseException: Cannot call public liquibase.structure.core.Index liquibase.structure.core.Index.setTable(liquibase.structure.core.Table) with value of type java.lang.String
        at liquibase.parser.core.yaml.YamlSnapshotParser.parse(YamlSnapshotParser.java:62)
        at liquibase.database.OfflineConnection.<init>(OfflineConnection.java:101)
        ... 17 more
Caused by: liquibase.parser.core.ParsedNodeException: liquibase.exception.UnexpectedLiquibaseException: Cannot call public liquibase.structure.core.Index liquibase.structure.core.Index.setTable(liquibase.structure.core.Table) with value of type java.lang.String
        at liquibase.snapshot.DatabaseSnapshot.load(DatabaseSnapshot.java:632)
        at liquibase.parser.core.yaml.YamlSnapshotParser.parse(YamlSnapshotParser.java:55)
        ... 18 more
Caused by: liquibase.exception.UnexpectedLiquibaseException: Cannot call public liquibase.structure.core.Index liquibase.structure.core.Index.setTable(liquibase.structure.core.Table) with value of type java.lang.String
        at liquibase.util.ObjectUtil.setProperty(ObjectUtil.java:143)
        at liquibase.util.ObjectUtil.setProperty(ObjectUtil.java:172)
        at liquibase.snapshot.DatabaseSnapshot.load(DatabaseSnapshot.java:597)
        ... 19 more

this is my current liquibase properties file, the sql server is hosted in azure and the database is the adventures work sample database that comes by default
#driver: com.mysql.jdbc.Driver
#url: jdbc:mysql://localhost:3306/example2
driver: com.microsoft.sqlserver.jdbc.SQLServerDriver
url: jdbc:sqlserver://sqlservergabriel89.database.windows.net:1433;databaseName=samepldb
#url: offline:mssql?snapshot=schema_before1.json&catalog=samepldb&version=12.00.2148&productName=SalesLT&caseSensitive=true
classpath: mssql-jdbc-9.2.1.jre8.jar
#classpath: mysql-connector-java-8.0.25
username: user2
password: **********
#username: root
#password: *********
logLevel: debug
changeLogFile: mydatabase_changelog.mssql.sql
#diffChangeLogFile=mydatabase_diffchangelog.mssql.sql
#outputChangeLogFile=mydatabase_outputchangelog.mssql.sql
#changeLogFile: mydatabase_changelog.xml
defaultSchemaName: SalesLT
defaultCatalogName: samepldb
#referenceUrl: offline:mssql?snapshot=schemabefore.json
referenceUrl: offline:mssql?snapshot=documento.json&catalog=samepldb&version=12.00.2148&productName=Microsoft SQL Server&caseSensitive=true
#referenceUrl: offline:mysql?snapshot=schema_before.json
#referenceDefaultSchemaName: SalesLT
#referenceDefaultCatalogName: samepldb
outputDefaultCatalog: true
outputDefaultSchema: true
includeCatalog: true
includeSchema: true

Steps To Reproduce
List the steps to reproduce the behavior.

deploy a azure sql server and use a adventuresworks example database provided by azure
make a snapshot of database

./liquibase.bat --defaultSchemaName=SalesLT --defaultCatalogName=samepldb --outputDefaultCatalog=true --outputDefaultSchema=true --includeCatalog=true --includeSchema=true --driver=com.microsoft.sqlserver.jdbc.SQLServerDriver --classpath=mssql-jdbc-9.2.1.jre8.jar "--url=jdbc:sqlserver://sqlservergabriel89.database.windows.net:1433;databaseName=samepldb" "--username=user2" "--password=*****" --outputFile=documento.json --logLevel=debug snapshot --snapshotFormat=json

do a

./liquibase.bat  diff

Actual Behavior
A clear and concise description of what happens in the software with the version used.
testing with sample database adventurers work in sql server fails
Expected/Desired Behavior
A clear and concise description of what happens in the software after a fix is created and merged.
testing with sample database adventurers work in sql should work
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.

the adventure table have user defined data types, does liquibase can know about this data types? i dont think it can because , i dont see the definition in the snapshot i take but i dont know if this is related with the error mentioned above

i tried with liquibase pro trial license but still cant retrieve defined data types, please in would like some suport about this case ,thanks!

Hi @gabrielcuadrosinkafarma Thanks for submitting this issue. I'm not sure that user defined types are supported in snapshots in Liquibase at this time.

This issue have gone stale. If you think we should add defined types support, you can always submit an enhancement request and/or a PR with a solution for this. Im closing this issue for now. But feel free to re-open it


fix: error on unvalidated config

Summary:
Briefly describe the changes made in this PR.
Issue Reference(s):
Fixes #1517
/claim #1517

Build & Testing:

[ ] I ran cargo test successfully.
[ ] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[ ] I have added relevant unit & integration tests.
[ ] I have updated the documentation accordingly.
[ ] I have performed a self-review of my code.
[ ] PR follows the naming convention of <type>(<optional scope>): <title>

Summary by CodeRabbit
Summary by CodeRabbit

Refactor

Enhanced configuration validation to prevent unknown fields from being accepted, improving overall system reliability and configuration accuracy.

@35C4n0r There are some lint errors.

@35C4n0r There are some lint errors.

@tusharmath Fixed.


Update docker example in migration guide

Fixes #4054

I didn't know if the desired example was to run on port 80 or 9090...
Also, I've changed the prometheus version there because it was an old release... but I can change it back if requested

LGTM, any comment @simonpasquier @SuperQ


Cannot enter bootloader mode

I had my Plaid working for a month. Now I want to flash a new layout.
I understand that to enter bootloader mode, I have to:

Plug into USB cable
Push and hold RESET SW
Push and hold BOOT SW
Release RESET SW
Release BOOT SW

However, when I pushed the BOOT button, it just output "edc", i.e the keys on the 4th column, so now I cannot enter bootloader mode to flash my new layout.
Again, aside from the BOOT button, all the other keys of my Plaid still works fine.
How should I go about solving this? Thanks.

check if your RESET button works, it will disconnect and reconnect the board.
Will return "edc" if switch button is not pushed down while pressing the BOOT button

Cool, just ordered a new atmega chip with a pre-loaded bootloader on it. Will check on this method when I finally get it. Thanks!

check if your RESET button works, it will disconnect and reconnect the board.
Will return "edc" if switch button is not pushed down while pressing the BOOT button as BOOT switch is using the same pin as COL3 (See schematics).


First Time Bounty Hunter Tutorial

The bot should actually provide a custom tutorial message instead of just responding with the help menu without context. For example, it should say:
Welcome to the ubiquity-dollar repository. Please comment "/assign" on any issue if you would like to work on it. UbiquiBot will immediately assign you to the issue. Please comment "/wallet" at any time with your wallet address following (e.g. /wallet ubq.eth) to register your payment address for task completion.
As part of the bounty, I would expect that whoever implements this should think about ways to improve the message.

          ### Available commands

- /assign: Assign the origin sender to the issue automatically.
        ex: /assign

- /wallet <WALLET_ADDRESS | ENS_NAME>: Register the hunter's wallet address.
	ex1: /wallet 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
	ex2: /wallet vitalik.eth

@chrlschwb
Originally posted by @ubiquibot[bot] in https://github.com/ubiquity/ubiquity-dollar/issues/536#issuecomment-1491282411

Consolidating into https://github.com/ubiquity/ubiquibot/issues/96


Not working correctly with inAppBrowser

great plugin.  in testing i did run into an issue.  i used the apache "inAppBrowser" and just used google.com as the url.  I then simply hit the home button.  in the console i get the message "Snapshotting a view that has not been rendered results in an empty snapshot. Ensure your view has been rendered at least once before snapshotting or snapshot after screen updates."  testing the rest of the app in all the other functions it worked great.  I am using cordova 3.6.3 on a iPad Air 2 running ios 8.1.  Not that it should matter, but i am also using JqueryMobile 1.4.3.  Any idea on how to fix this i did not see any settings that i had to change.
thanks for any input...

You can modify "InAppBrowser" plugin.
this.getWindow().setFlags(WindowManager.LayoutParams.FLAG_SECURE,
WindowManager.LayoutParams.FLAG_SECURE);
add above line into "InAppBrowserDialog" constructor.

There is a PR for inAppBrowser right now that might be what we need.
https://github.com/apache/cordova-plugin-inappbrowser/pull/164
If it works, I am hoping to have a look and get consensus to merge it in.

Ugh. The PR was never merged in it looks like.
Not sure what the next step is here...

apache/cordova-plugin-inappbrowser#164 is work for inAppBrowser,
but can't work on themeableBrowser

error: cannot find symbol
...
if (!features.disableAnimation) {
dialog.getWindow().getAttributes().windowAnimations
= android.R.style.Animation_Dialog;
}
            int flags = cordova.getActivity().getWindow().getAttributes().flags;
            if ((flags & WindowManager.LayoutParams.FLAG_SECURE) != 0) {
                // dialog.getWindow().addFlags(WindowManager.LayoutParams.FLAG_SECURE);
                dialog.getWindow().addFlags(LayoutParams.FLAG_SECURE);
            }

this plugin doesn't work on iOS10.0-10.3


Fixes #2322 - count tasks without explicit state as staging

Generally strange that we consider tasks with empty states as staged – but in order to keep this bugfix small and not introduce other issues that might depend on empty task state, this LGTM!


Add check to link put and add validate for invocation

Add check to duplicated link put, and some tiny fix

Thanks @Taction and @thomastaylor312.  Will review and get merged this week 🙂

TY! On the list for first thing in the morning


Resource definition - Different metrics formula for different plan.

Hi,
Currently the pricing config in account-stub has plans in it(which will allows different plans to have different prices). For example object-storage has one plan called 'basic'.
On the other side, the object-storage resource definition in provisioning-stub does not use any plan in it. This would not create any problem if all plans are assumed to use the same meter, accumulate, rate, summarize, and charge functions.
In some situation, different plan might have different functions. Example for object-storage:
plan basic: no incremental model( #152  ). plan super: use incremental model. Or where different plan have certain discounts / disregarding a metric in the rate function (plan super will have x gb storage free while basic is not.)
We might not face this problem now, but it is very possible in the future.
Any thoughts? Thanks

We'll need to add a plan_id to the resource and price configs and also pass that plan_id through the APIs that retrieve them.

This is now in with c619c2f5212bd2debecd4986956fb316582f2423
The resource config structure has changed from
{
	"resource_id":"",
	"effective":0,
	"metrics":[],
	"measures":[]
}

to
{
	"resource_id":"",
	"effective":0,
	"plans": [{
		"plan_id":"",
		"metrics":[],
		"measures":[]
	}]
}


debug mode in test.sh gives error (-5:Bad argument) in function 'line'

I am trying to visualize the intermediate results of testing. As @mrnabati pointed out in this issue I tried to enable debug by putting --debug 4 \ in the test.sh file. However, I get the following error:
Traceback (most recent call last):
  File "test.py", line 224, in <module>
    prefetch_test(opt)
  File "test.py", line 131, in prefetch_test
    ret = detector.run(pre_processed_images)
  File "/home/fabrizioschiano/repositories/CenterFusion/src/lib/detector.py", line 159, in run
    self.show_results(self.debugger, image, results)
  File "/home/fabrizioschiano/repositories/CenterFusion/src/lib/detector.py", line 435, in show_results
    debugger.add_bird_view(
  File "/home/fabrizioschiano/repositories/CenterFusion/src/lib/utils/debugger.py", line 424, in add_bird_view
    cv2.line(bird_view, (rect[e[0]][0], rect[e[0]][1]),
cv2.error: OpenCV(4.5.3) :-1: error: (-5:Bad argument) in function 'line'
> Overload resolution failed:
>  - Can't parse 'pt1'. Sequence item with index 0 has a wrong type
>  - Can't parse 'pt1'. Sequence item with index 0 has a wrong type

my version of opencv is the following
python -c "import cv2; print(cv2.__version__)"
4.5.3

Did anyone experience this issue?
Thanks!

It now works as you can also see from the following screenshot. As you can see, many pictures are generated for each sample. Therefore, next step would be to understand exactly what are all the pictures generated in the debug folder.


Different Kylin with Mondrian

Hi,
I am newbie in OLAP world!
What is different between Kylin and Mondrian ?.
Why do you need to mix between them ?
Thank you

Hi pribadihcr,
it is not complicated:

Mondrian: Enables multidimensional queries against data, using the powerful MDX query language, translates MDX queries to SQL.
Kylin: Provides multi-dimensional analysis (OLAP) on Hadoop with SQL interface (no MDX queries).

Result when you combine Kylin and Mondrian: MDX on Hadoop

I have a question, can I use this method on mondrian 3.12 branch ?


Fix sampling bug, improve code

There is only one thing, are we suppose to prefix size_t with std:: in header files ? If so, you have some changes to do in cdf.h.

We will. Until now, we don't prefix size_t anywhere but it's not very logical. At some point we'll do a big search & replace and make sure it is prefixed everywhere. Until then, and out of consistency, I prefer not to add the prefixes.
Thanks for the review!


Exception fired when generating from HTML

Based on the example in the readme, when I use generateFromHtml I get the following.
Fatal error: Uncaught exception 'RuntimeException' with message 'The file '/tmp/bill-123.pdf' was not created (command: /Users/jmorton/Sites/tests/wkpdf/wkhtmltopdf --lowquality "/var/folders/EH/EH1NBNSiGU0hy8p-WZ78Bo2kqHI/-Tmp-/knp_snappy4e7223ebe7a27.html" "/tmp/bill-123.pdf").' in /Users/jmorton/Sites/tests/wkpdf/Knp/Snappy/AbstractGenerator.php:260 Stack trace: #0 /Users/jmorton/Sites/tests/wkpdf/Knp/Snappy/AbstractGenerator.php(116): AbstractGenerator->checkOutput('/tmp/bill-123.p...', '/Users/jmorton/...') #1 /Users/jmorton/Sites/tests/wkpdf/Knp/Snappy/AbstractGenerator.php(126): AbstractGenerator->generate('/var/folders/EH...', '/tmp/bill-123.p...', Array, false) #2 /Users/jmorton/Sites/tests/wkpdf/index.php(14): AbstractGenerator->generateFromHtml('Bill<p...', '/tmp/bill-123.p...') #3 {main} thrown in /Users/jmorton/Sites/tests/wkpdf/Knp/Snappy/AbstractGenerator.php on line 260
If I try to have it send the pdf for download then the pdf is corrupted and wont open.

This is indeed something related to wkhtmltopdf and not Snappy