Improve speed regression testing

[ ] Add commit hashes to .csv
[ ] Add computer identifier to rerun all tests on different computers
[ ] Show improvements as percent % instead of no units
[X] Self hosted server to avoid network delays

Self hosted server to avoid network delays

Fixed by #108

Self hosted server to avoid network delays

Fixed by #108

@CohenArthur @Skallwar any specific reason for bringing python and http.server ?
httpmock for rust would provide portablity and runs either standalone or via API in sync and async context if you wish to decide adopt async later and include it easily within tests.
httpmock runs on hyper (like reqwest) and also features delay setting and I've been pondering contributing there for more network behaviour scenarios.
nginx also would provide sendfile and other optimisations if standalone is still required and it's portable and fast.
nginx also is event based reactor (+prefork) which means no messing with threads while you serve static content but simulating (without breaking portability) delay scenarios and other network blips is harder with limit_req, limit_req_zone, rate=20r/s
Also instead of cloning the linux repo there could be "a seed" setting that readily generates into repeatable data for various test cases as there can be different performance scenarios for tons of small files, large files, error handling etc.
If you are keen I could push a PR

@CohenArthur @Skallwar any specific reason for bringing python and http.server ?

Not really :D It was simple, and this is what the first testing benchmark was originally written in. It would scrape http://books.toscrape.com/ and time it, which was bad for a number of reasons. I am definitely not against functional testing/benchmarking being done in Rust, and I don't think @Skallwar is either.

Also instead of cloning the linux repo there could be "a seed" setting that readily generates into repeatable data for various test cases as there can be different performance scenarios for tons of small files, large files, error handling etc.

The test clones the linux repository in order to benefit from a big folder with known data. We checkout the v5.9 branch so that every test runs against the same content. Seeding it would be interesting, but maybe not for this speedtest specifically.

If you are keen I could push a PR

That would be absolutely fantastic and we'd love it! I believe that we are using some form of HTTP mocking in the tests already, so this would definitely fit. Thanks a lot!

@CohenArthur @Skallwar Are you okay if I change to lib-bin pattern from binary only now?
Essentially I would split all the library logic to library and binary that requires different things e.g. for shell related libraries.
This is a common pattern and will make organising code, testing and dependency management much easier between two sets of needs.


Implement the post page with dynamic content

This issue is related to issue #17. This issue is the actual implementation of the design.

Task List:

[ ] Implement a rich text serializer using prismic-reactjs with appropriate styles for each element.
[ ] Design the main image/content section.


ARROW-12388: [C++][Gandiva] Implement cast numbers from varbinary functions in gandiva

Can you look at the failing tests

@projjal can you please confirm the errors on the builds is not related.

@projjal can you please confirm the errors on the builds is not related.

Build failures are unrelated

@projjal Needs a rebase

@jvictorhuguenin Can you rebase. There is merge conflict.

@praveenbingo Could you use the latest dev/merge_arrow_pr.py? The latest dev/merge_arrow_pr.py doesn't list squashed commits.

@kou Will pull the latest in. Thanks - sorry if my merge caused any trouble.

Thanks!


feat(schedules): add template selector when exporting meeting schedules

:tada: This PR is included in version 2.130.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fixed - cameraFilter bitmask doesn't work for values < 0

This PR (delete as applicable)

Fixes a bug

Describe the changes below:
The cameraFilter bitmask is only checked if its value is more than 0. A full bitmask is equal to -1, so I've changed the check to specifically check for !== 0.
I discovered this when coming up with a way of easily only showing something on one camera - foo.cameraFilter = ~camera.id, which for a camera id of 1 returns -2.

Thanks. I've also made the change to the Game Object class, as it's more useful there!


Commit changes and create a pull request for the styling into develop

Merge the advancements.

Done into develop.


AVX2

When running the code in the Service Definition step in 00_Quickstart.ipynb, the command prompt reported that "Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2". Will you be able to support AVX2? Thanks.

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

ID: 25565324-f64d-a8db-bc1c-e50956e6e878
Version Independent ID: a6c9ffa3-c085-b4fc-7153-06adb8a50ce4
Content: How to deploy a model as a web service on an FPGA with Azure Machine Learning
Content Source: articles/machine-learning/service/how-to-deploy-fpga-web-service.md
Service: machine-learning
GitHub Login: @tedway
Microsoft Alias: tedway

@nuiwos Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@tedway Hi, there is a product enhancement idea, can you please check it? Thanks a lot!


minor typo

Hi, when setting up a multi-sig wallet on my phone I noticed a minor typo in one of the error messages.  Looks like this occurs in many files.

Thank you for reporting this to us. We should have this fixed for the next release


[WIP] Run CPO conformance test in multinode Kubernetes cluster

Fixes https://github.com/kubernetes/kubernetes/issues/91739

@ramineni @chrigl could you please have a look at this PR? I've set up a local environment using the proposed ansible playbook, but the kubetest command failed with random test cases.


Preview not loading

Cannot load preview, even for very simple notes. Loader icon just spins forever.
App version: 0.1.8, build 18
Device: Kindle Fire HDX 8.9
OS: Android 4.4 (Fire OS <IP_ADDRESS>)
(The device/OS is of course rather old. Don't know if it's supposed to work on legacy devices? Rest of the app works fine.)

I just tried it out on an Android 4.4 Emulator and could reproduce the issue. It is caused because flutter platform views (which are required for the preview browser) are only supported on Android Version 5.0 and up. Because of this I will disable the preview feature if the app is running on an Android 4.4 device. That's of course not a perfect solution, but it's still better than dropping support for legacy devices i hope.

changed in https://github.com/redsolver/noteless/commit/7023336aba4941dce76733c01bd85c8fee34606e


Install script is not signed

Why is the PowerShell script unsigned? How do we know we are running a legitimate script when it's downloaded from internet on each build?

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

ID: a2e484a9-5e0c-eb3e-1413-7b507c4fa07d
Version Independent ID: 5bc7caa5-f1c2-d7d5-fe02-80c65cca9672
Content: dotnet-install scripts - .NET Core CLI
Content Source: docs/core/tools/dotnet-install-script.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @mairaw
Microsoft Alias: mairaw

@jozefizso I'm going to move this issue to the cli repo where the source of the script is.

This issue was moved to dotnet/cli#13159

It should probably go to the sdk repo instead.


fix #133: XSS injection

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed tickets
#133

License
MIT

Doc
PR

See https://michelf.ca/blog/2010/markdown-and-xss/ for the explanation about why this is not solving the issue

For those who don't want to read the entire article:

So the conclusion is that, if you want real security, you need to filter Markdown’s output, not the input. There’s no other choice.

@javiereguiluz oh, but Markdown’s output is already html. it will be not so easy to filter it

Does a post XSS injection really problem for us in this case? Posts could add administrator only, so if he need to use html tag - that's ok I think. User haven't enough premissions and couldn't create a posts.
What about comments, IMO we should use strip_tags or htmlentities and do it only on create/update comment, then we read already escaped value from DB. Require a vendor bundle for it in this demo isn't a good idea I think.

@bocharsky-bw you're right, strip_tags could solve the problem

@bocharsky-bw @voronkovich no it won't. Please read the article I sent
@bocharsky-bw we should not advocate that Markdown is output a safe value. And allowing XSS in admin forms because you are trusting admins is a bad idea. you should not trust users.

Closing it for not being the right solution.


Generated projects doesn't compile with maven 3.x

Generated projects use exec-maven-plugin version 1.2.
        <groupId>org.codehaus.mojo</groupId>
        <artifactId>exec-maven-plugin</artifactId>
        <version>1.2</version>

Maven 3.X has changed mvn.bat to mvn.cmd and due to this http://jira.codehaus.org/browse/MEXEC-118 bug generated projects do not compile with it.
I suppose that exec-maven-plugin version should be upgraded to latest version in pom templates.

Codehaus has shut down.
Termination
All Codehaus services have been terminated
Redirection
If you would like your projects links redirected then please see our redirector repository - create a sane squashed pull request and it will be added to the redirection system - you may add some 302s initially, but ultimately all redirects will be amended to 301s over time.

Hi astafyev,
Thanks for pointing this out. We have upgraded to exec-maven-plugin:1.4.0 and that solves this issue.
Closed https://wso2.org/jira/browse/TOOLS-2850 and https://wso2.org/jira/browse/TOOLS-2819 respectively.


Data Type usage: users cannot 'deselect' selected non-concrete root elements

This only happens if there are no concrete children.  In this case the DT should be removed


Fix config load order bug with LA

it was solved by the order change
this runner_config.light_actions if runner_config.light_actions else []  is incase someone messes with the config and removes it all together(or adds a typo there) to make sure there wont be issues with the sinks being configured


Should Shell32 on Windows be required by default?

Currently Shell32 is loaded dynamically by default. Linking is possible via compile-time version switch.
Both dmd and gdc link executable against Shell32 (according to Dependency Walker), so it probably does not make sense to load the library at runtime. Need to check if ldc does this too.

Now Windows version prefers SHGetKnownFolderPath. Optlink can't find this symbol if linking to shell32. So we need runtime loading anyway.


[Bug] Maven commands fail randomly in CI/CD workflow

Plugin Name
@nxrocks/nx-spring-boot
Plugin Version
4.1.0
Nx Version
14.4.3
Expected Behaviour
./mvnw commands should successfully completes on first execution.
Actual Behaviour
This project provides the script ./mvnw to run maven commands when executing project targets (e.g. nx build <project>). In my main CI/CD workflow (GH Action), the execution of targets that rely on ./mvnw randomly fail.
Here are the high-level commands that may fail in my CI/CD work.
      - run: yarn nx affected --target=build --parallel --max-parallel=3
      - run: yarn nx run-many --all --target=test --parallel --max-parallel=2

Here is an example of error that randomly make my CI/CD workflow fail.
Error:  Error executing Maven.
java.lang.NullPointerException
	at java.base/java.util.concurrent.ConcurrentHashMap.putVal(ConcurrentHashMap.java:1011)
	at java.base/java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:1006)
	at java.base/java.util.Properties.put(Properties.java:1301)
	at java.base/java.util.Properties.setProperty(Properties.java:229)
	at org.apache.maven.cli.MavenCli.populateProperties(MavenCli.java:1656)
	at org.apache.maven.cli.MavenCli.properties(MavenCli.java:612)
	at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:282)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:196)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:282)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:225)
	at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:406)
	at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:347)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.apache.maven.wrapper.BootstrapMainStarter.start(BootstrapMainStarter.java:47)
	at org.apache.maven.wrapper.WrapperExecutor.execute(WrapperExecutor.java:156)
	at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:72)
> nx run shared-java-challenge-util:build
Executing command: ./mvnw package
Failed to execute command: ./mvnw package
Error: Command failed: ./mvnw package
    at checkExecSyncError (node:child_process:828:11)
    at execSync (node:child_process:899:15)
    at runBuilderCommand (/__w/challenge-registry/challenge-registry/node_modules/@nxrocks/common/src/lib/core/jvm/utils.js:19:38)
    at runBootPluginCommand (/__w/challenge-registry/challenge-registry/node_modules/@nxrocks/nx-spring-boot/src/utils/boot-utils.js:15:43)
    at /__w/challenge-registry/challenge-registry/node_modules/@nxrocks/nx-spring-boot/src/executors/build/executor.js:10:62
    at Generator.next (<anonymous>)
    at /__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:117:75
    at new Promise (<anonymous>)
    at Object.__awaiter (/__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:113:16)
    at buildExecutor (/__w/challenge-registry/challenge-registry/node_modules/@nxrocks/nx-spring-boot/src/executors/build/executor.js:8:20)
Error
    at /__w/challenge-registry/challenge-registry/node_modules/@nxrocks/nx-spring-boot/src/executors/build/executor.js:12:19
    at Generator.next (<anonymous>)
    at /__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:117:75
    at new Promise (<anonymous>)
    at Object.__awaiter (/__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:113:16)
    at buildExecutor (/__w/challenge-registry/challenge-registry/node_modules/@nxrocks/nx-spring-boot/src/executors/build/executor.js:8:20)
    at /__w/challenge-registry/challenge-registry/node_modules/@nrwl/tao/src/commands/run.js:147:23
    at Generator.next (<anonymous>)
    at /__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:117:75
    at new Promise (<anonymous>)
    at __awaiter (/__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:113:16)
    at runExecutorInternal (/__w/challenge-registry/challenge-registry/node_modules/@nrwl/tao/src/commands/run.js:127:34)
    at Object.<anonymous> (/__w/challenge-registry/challenge-registry/node_modules/@nrwl/tao/src/commands/run.js:219:54)
    at Generator.next (<anonymous>)
    at /__w/challenge-registry/challenge-registry/node_modules/tslib/tslib.js:117:75
    at new Promise (<anonymous>)

Restarting the failed job in the GH Actions may lead to a successful run as well as another failed run (seems random).
Based on the log, could it be that maven or the maven wrapper (./mvnw) executions can not be reliably run in parallel? I run 2-3 tasks in parallel in my CI/CD workflow. The next troubleshooting step would be for me to test without running targets concurrently.
@tinesoft Have you ever observed this behavior with ./mvnw?
Steps to reproduce the behaviour

Fork https://github.com/Sage-Bionetworks/challenge-registry.
The GH workflow .github/workflows/ci.yml may randomly fail.

Hi @tschaffter

@tinesoft Have you ever observed this behavior with ./mvnw?

No I haven't. sorry

Based on the log, could it be that maven or the maven wrapper (./mvnw) executions can not be reliably run in parallel? I run 2-3 tasks in parallel in my CI/CD workflow.

I would say so too, yes.  It has nothing to do with the plugin per se, but rather in the concurrent capabilities of the projects that were ran.

The next troubleshooting step would be for me to test without running targets concurrently.

Yes that would be a good test indeed. You could also try to pinpoint from the nx affected command output, which ones of your projects were actually built and try to restrict the scope to these specific projects.
Then, creating a simple test that build those x projects in parallel , by calling for example:
 it('test parallel build', async () => {
   	for (let i = 0; i < 20; i++) {
   		execSync(`nx run-many --projects project1,project2,projectx --parallel 3`);
   	}
 });

in a loop, could help reproduce the issue locally...
Looking at the source code of Maven itself, can also help:
https://github.com/apache/maven/blob/9b656c72d54e5bacbed989b64718c159fe39b537/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java

I spent some time exploring the issue and came up with the solution. The issue stems from the fact that concurrent executions of Maven is not safe. For example, two or more concurrent executions of Maven may attempt to download the same dependency at the same time and save it into ~/.m2/repository, which I've seen resulting in errors (Issue A).
A similar error occurs when multiple project targets that rely on@nxrocks/nx-spring-boot are executed in parallel, which is the default when running an nx run-many command. By default, @nxrocks/nx-spring-boot relies on the Maven wrapper (mvnw) for the sake of portability. Any execution of ./mvnw will trigger the download of the mvn binary, which is installed in ~/.m2/wrapper by default. In this context, errors may arise when parallel executions of an @nxrocks/nx-spring-boot targets attempts to download the same version of the Maven binary at the same time (Issue B). Below is the stack trace of an occurrence of this issue.
$ nx run-many --all target=build

...
Exception in thread "main" java.util.zip.ZipException: zip END header not found
	at java.base/java.util.zip.ZipFile$Source.findEND(ZipFile.java:1469)
	at java.base/java.util.zip.ZipFile$Source.initCEN(ZipFile.java:1477)
	at java.base/java.util.zip.ZipFile$Source.<init>(ZipFile.java:1315)
	at java.base/java.util.zip.ZipFile$Source.get(ZipFile.java:1277)
	at java.base/java.util.zip.ZipFile$CleanableResource.<init>(ZipFile.java:709)
	at java.base/java.util.zip.ZipFile.<init>(ZipFile.java:243)
	at java.base/java.util.zip.ZipFile.<init>(ZipFile.java:172)
	at java.base/java.util.zip.ZipFile.<init>(ZipFile.java:186)
Exception in thread "main" java.nio.file.NoSuchFileException: /root/.m2/wrapper/dists/apache-maven-3.8.6-bin/67568434/apache-maven-3.8.6-bin.zip.part
	at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:92)
	at org.apache.maven.wrapper.Installer.unzip(Installer.java:207)
	at org.apache.maven.wrapper.Installer.createDist(Installer.java:110)
	at org.apache.maven.wrapper.WrapperExecutor.execute(WrapperExecutor.java:151)
	at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:76)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
	at java.base/sun.nio.fs.UnixCopyFile.move(UnixCopyFile.java:429)
	at java.base/sun.nio.fs.UnixFileSystemProvider.move(UnixFileSystemProvider.java:266)
	at java.base/java.nio.file.Files.move(Files.java:1432)
	at org.apache.maven.wrapper.Installer.createDist(Installer.java:95)
	at org.apache.maven.wrapper.WrapperExecutor.execute(WrapperExecutor.java:151)
	at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:76)
Exception in thread "main" java.nio.file.NoSuchFileException: /root/.m2/wrapper/dists/apache-maven-3.8.6-bin/67568434/apache-maven-3.8.6-bin.zip.part
	at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:92)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
	at java.base/sun.nio.fs.UnixCopyFile.move(UnixCopyFile.java:429)
	at java.base/sun.nio.fs.UnixFileSystemProvider.move(UnixFileSystemProvider.java:266)
	at java.base/java.nio.file.Files.move(Files.java:1432)
	at org.apache.maven.wrapper.Installer.createDist(Installer.java:95)
	at org.apache.maven.wrapper.WrapperExecutor.execute(WrapperExecutor.java:151)
	at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:76)

The above issues become more and more likely to happen randomly as the Nx workspace grow in terms of number of Maven-based projects. Issue B can be solved by using a globally-installed Maven binary, which can be achieved by specifying the option --ignoreWrapper to the @nxrocks/nx-spring-boot executor.
Ultimately, I decided to find a solution to Issue A and Issue B that still enables projects to use different version of Maven for enhanced project isolation. First, I added the following target to my Java project:
    "prepare-java": {
      "executor": "@nrwl/workspace:run-commands",
      "options": {
        "commands": [
          "./mvnw dependency:go-offline -DexcludeGroupIds=org.sagebionetworks.challenge || true"
        ],
        "cwd": "apps/challenge-api-gateway"
      }
    },

This target downloads Maven (mvw) and all the dependencies of the project. Because I have Java projects that depend on a shared local library, the command ./mvnw dependency:go-offline would fail because it would not find my local library that has not even been built at this stage. I may have been able to solve this using Nx project dependncies, but this would complicate this "preparation" stage. Instead, I'm specifying the option -DexcludeGroupIds=org.sagebionetworks.challenge to prevent Maven from throwing an error when it attempt to download my shared local libraries. Unfortunately, the command mvn dependency:go-offline has several bugs and one of them is that the option -DexcludeGroupIds is not evaluated. The workaround I found was to add || true to silence any errors that mvn dependency:go-offline may generate. I really don't like doing that but since a failure of this prepare-java target should not affect subsequent targets (lint, build, etc.), I can live with this shortcoming.
In my CI workflow, I run the following command to ultimately 1) install Maven and 2) install all the project dependencies (minus my shared local libraries) sequentially for the affected projects.
      - run: yarn nx affected --target=prepare-java --parallel=1
      - run: yarn nx affected --target=lint
      - run: yarn nx affected --target=build
      - run: yarn nx affected --target=test

This is how I solved Issue B while maintaining Java project isolation and most importantly, Issue B.
The release notes of the latest version of Maven mention improvements for concurrent builds. I have not evaluated the impact of these improvements on the reported issues but it sounded like a good idea to improve the version of Maven used by the wrapper and update the wrapper itself at the same occasion.
For each Java project, I update the .mvn/wrapper/maven-wrapper.properties file with the reference to the latest version of Maven and Maven wrapper:
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.8.6/apache-maven-3.8.6-bin.zip
wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.1.1/maven-wrapper-3.1.1.jar

Once again, anyone using this plugin should face Issue B at some point as the number of Java projects and targets increase in the workspace. Idem for Issue A, unless Maven finds a way to safely handle the case where concurrent executions attempt to modify the same file at the same time. Because of the pseudo random nature of the issue, it may be worth adding a note about it to the README of this plugin.
Thanks!

Hi Thomas,
Thanks for your continuing interest in the plugin and for sharing the result of your investigation in such an exhaustive way!
The issues you create on my repo are always very well detailed, for that, I officially name you, my "n°1 issue reporter" !
I will look into a way to implement (and document) your workaround, with:

a new go-offline executor (that will download everything locally)

My only concern so far, is the compatibility when using Gradle (instead of Maven) as build system...
There is an  offline mode in Gradle too,  but it works differently from Maven's, as it requires  the --offline parameter to be passed along with each command,  whereas for Maven the offline mode can be activated once (via mvnw dependency:go-offline) and then benefit to subsequent commands.
I'm still researching the best way to achieve that, that would be transparent for final users of the plugin, wether they use Maven or Gradle.

We've also ran into several problems with this, which forced us to disable parallel builds, sadly. But there seems to be workarounds to make maven concurrent safe. Maybe you coud add it into your plugin?
@tinesoft
http://takari.io/book/30-team-maven.html#concurrent-safe-local-repository

This is still an issue for my monorepos where I have to run some Gradle tasks with without concurrency.
The Takari project seems archive. One of their GH repo indicates that there may be a solution for projects managed with Maven: https://maven.apache.org/resolver/local-repository.html#shared-access-to-local-repository


BREAKING(get rid of inner component)

Initial spike of removing the inner component and switching to ember-popper 0.7.0, which uses a new approach for interacting with the popper element.

[x] Get dummy app working

[x] Make sure options can still be toggled (aka fix bug in ember-popper)
[x] Fix weird style bug
[x] Fix renderInPlace (should add tests to prevent regressions) https://github.com/kybishop/ember-popper/pull/58

[x] Get tests working
[x] Make sure animations work as before
[x] Fix issue where position is altered between arg changes (like arrow true/false)

I'll squash these commits when it's finally time to merge.
No rush, but @rwwagner90 I'd love your thoughts on this before I push it through.

@kybishop can you give me a TLDR of why we made this switch? There is no inner component at all now? Just one component? I'm curious how it effects the issues I was having with outer and inner classes and showing and hiding.

Hey @rwwagner90, here are the bullet points:

The inner component was able to be removed thanks to ergonomic improvements to ember-popper.
class is now passed directly to the inner div
{{#ember-attacher}} is now {{#attach-popover}}
The popper <div> (the outer div) is now hidden/shown, and receives the aria-hidden attribute.

We can't remove the inner <div> entirely because popper.js's positioning CSS conflicts with the animation CSS. To get around the issue, we apply animations to the inner div instead.
In regards to your previous issues with class, they'd disappear entirely with this PR. This effectively gets us to parity/compatibility with ember-tooltips.

@kybishop sounds good to me! I briefly looked at the code, and didn't see anything glaringly wrong, so fine with me to ship it 👍


Issue-7 - Bucket this scoping issue.

Coverage remained the same at 100.0% when pulling 2f97dd765efbb90b78750aa69c9abca23cb236e1 on issue-7 into c621238eba2fa3ca690863da5e60b5cff5a0469c on master.


Add @Name annotation support

Not a breaking API change either :)

Open questions:

Should this just be for methods and make @Burst have an optional name method?

Related: if keeping field support for the @Name annotation, should it warn or fail if there's not also a @Burst annotation on it?

@Name, @BurstName, @Named, @Banana?

This looks exciting! Can't wait to see it in the next release.

I see how this would be useful, but not sure about the output format; when I see Drink.PEPSI I think Drink is an enum type.
Maybe we should just change . to = (for both named an unnamed params) so it doesn't look like Java syntax. Or alternatively, we could keep unnamed params as is (like Soda.PEPSI), and  do drink=Soda.PEPSI for named params. WDYT?

I think I prefer the latter, despite it making the name longer. I don't really care in the end though.

I'd prefer the former regardless and the latter optionally. The main benefit for me with this can make it easier to grok at a glance. The soda example perhaps doesn't make the value prop clear, but to use another example:

In this case, forcing = and lengthening the name would just make things worse. I'd rather make the explicit drink=Soda bit or its inverse (just drink=) optional.

Hm, maybe we should just drop the enum type names in all cases, i.e.

@Burst Soda soda -> COKE
@Name("drink") @Burst Soda soda -> drink=COKE

It would mitigate the length concern, and I feel that unqualified names are kind of more canonical since that's what Enum#toString gives.
We'll have some cases like your OnboardingState.UNKNOWN where UNKNOWN isn't meaningful without the type, but @Name would take care of that; onboardingState=UNKNOWN seems just as good.

I think I'd be OK with that

Ping for updates.

Do you have an opinion on what we ended up on above?

Nope! Either is fine by me too.
On Sat, Jun 4, 2016, 10:18 PM Zac Sweers<EMAIL_ADDRESS>wrote:

Do you have an opinion on what we ended up on above?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/burst/pull/54#issuecomment-223789114, or mute
the thread
https://github.com/notifications/unsubscribe/AAEEEYqfsz3tL8NX3LzGUzDWYaoqzXqKks5qIjHtgaJpZM4IFIMH
.

@JakeWharton @dlubarov updated. Now the output is like this:
@Burst Soda soda -> COKE
@Name("drink") @Burst Soda soda -> drink=COKE

Looks good apart from those minor things 👍

@dlubarov addressed some comments, but I'm not sure I fully grok what you're getting at regarding the "args.length == annotations.length" bits.

I'd like to revisit this - any further thoughts?

Rebased, but addressed the last two comments:

Revert back to former arraycopy approach: 211f73c
Use empty array rather than nullable params: a94fed2

Sorry I missed your update, thanks for seeing it through!


Align [Shell|Process]Task#identifier

I have still not understood why we have #identifier in shell and process task and it still feels like we are leaking an implementation detail but if we do so it should  be called id to align with Extension#id and SourceControl#id

We have an identifier to reference a task contributed by an extension in the tasks.json.

So I can create an API task that is referenced in tasks.json to create another (or replacement) task? I didn't know that was possible. How do I know those identifiers and if how do I know what those referenced tasks actually do?
I see how I can hook up a somewhat generic NPM task with a problem matcher but I wonder we can't just have some like this
{
	"version": "2.0.0",
	"tasks": [
		{
			"taskName": "gulp: build",
			"command": "npm run compile",
			"problemMatcher": "$tsc"
		}
	]
}

with really good IntelliSense for command which is done via an extension that simply registers a completion item provider for **/.vscode/tasks.json? With that we would avoid the task+tasks.json<>task loop

This is exactly what we want to get rid of. It is better that these commands come from an extension instead of having them statically in the tasks.json. This is what makes it hard for extension to evolve there stuff (for example if they detect the need to change the command or pass different arguments for different OS, ...)
An you don't need to know these ids. You can simply do:

All the extensions needs to do is to keep the identifier stable.

Moving to June since we moved the API back to internal.

Should we maybe have something like a TaskTemplate which is such a cookie cutter task and then a ordinary Task which cannot be customised? I think of a task that publishes a build and has a custom output reader. My feeling is that it would feel alien in tasks.json

In the early days of task 0.1.0 we had two sections for this. tasks and something like taskAnnotations. Was confusing at least from the feedback at that time. So in task 0.1.0 we decided to do the matching by name. However since task can now come from extensions there is a potential of a name conflict (two calling their tasks build :-)). So I decided to go with an id which makes it more stable. In 0.1.0 with gulp autodetection this looked like this:
{
	"version": "0.1.0",
	"command": "gulp",
	"isShellCommand": true,
	"tasks": [
		{
			"taskName": "build",
			"problemMatcher": ["$tsc"]
		}
	]
}

which basically annotates the build task coming from gulp with a $tsc problem matcher.

Addressed in the new API.


Android app crashes when pressing connect

Describe the problem
When pressing connect button in android app, aplication will crash
Usign netbird cloud
app version 0.0.27
screenshot with error summary

Same version, same thing happening to me. I don't know how to submit the bug report, but I am on Android 10. LG Stylo6
Was working fine for a long time up until I tried it today. My device may have updated apps in the playstore, Netbird being one.

Fwiw I rolled back to v 0.24 and it works.
Hopefully this will be fixed for the next version
😄

same problem
version 0.0.27 from Google Play


Add spotify

component added

Incorporate into floorplan

Show album image
Create Scripts for play/pause, next, previous, volume up/down
address tv appearing


Fix Depth-Prepass transparency mode

Add missing bits of the implementation and ensure depth-prepass objects are only rendered in the transparency pass.
Fixes #50344

Thanks!


Environment variables for tests: #21

It's very simple, but I think it will be useful
tests = $disorderCheckEnvAll TestRunFast

$ ./mafia test
+++ OK, passed 1000 tests.

$ DISORDER_RUN_FAST=5000 ./mafia test
+++ OK, passed 5000 tests.

ping @charleso

should it be FAST=fewer tests, more agile?

OK - I will rename to "TestRunMore" "TestRunNormal" and "TestRunFewer"

:+1: Nice
The only question I have is whether the 3 "sizes" here are enough and the right value?

I can't say whether three sizes is enough.
Another option would be to throw away the size datatype altogether, and just have the tests specify the number of iterations as usual. Then, we could have a DISORDER_RUN_MULTIPLIER environment variable that just multiplied the number of tests. I don't really know which is better

Like the idea have a :hamburger:


Demonstrate change to occluded layer dragging

Layers or groups of layers can only be dragged if they are
not occluded at the point of dragging. Otherwise the layer(s)
are deselected and the occluding layer is selected instead.

Closes #705

This demonstrates the desired behavior, but I'm pretty sure it's not the right way to do it in Rust. Any tips would be appreciated.

@dchiasson It is easy to understand and it works. If you really wanted to avoid the unwrap you could do something like intersection.last().map(|last| selected.contains(last)).unwrap_or(false) but that may be less readable.

Are you part of the Discord server for the project, by the way? I'd like to promote your role to Code Contributor once this is merged, if so. Welcome and thank you for this contribution!

Yeah, thanks! My server nickname is the same as my github id.


OptionSerializer fails to serialize parameterized/wildcard Option fields

adding the following to OptionPlainTest:
public static class Parameterized<T> {
    public Option<T> value;
    public Parameterized(Option<T> value) {
        this.value = value;
    }
}

@Test
public void writeWrappedParameterizedSome() throws IOException {
    Parameterized<Integer> object = new Parameterized<>(Option.some(1));
    assertEquals("{\"value\":1}", mapper().writeValueAsString(object));
}

gives:

com.fasterxml.jackson.databind.JsonMappingException: No serializer found for class java.lang.Integer and no properties discovered to create BeanSerializer (to avoid exception, disable SerializationFeature.FAIL_ON_EMPTY_BEANS) ) (through reference chain: javaslang.jackson.datatype.Parameterized["value"])
at com.fasterxml.jackson.databind.ser.impl.UnknownSerializer.failForEmpty(UnknownSerializer.java:69)
at com.fasterxml.jackson.databind.ser.impl.UnknownSerializer.serialize(UnknownSerializer.java:32)
at javaslang.jackson.datatype.serialize.ValueSerializer.serialize(ValueSerializer.java:53)
at javaslang.jackson.datatype.serialize.OptionSerializer.serialize(OptionSerializer.java:41)
...

same happens with Option<?> and in "array" serialization mode

Hi @mpulkkinen
Which version you are using?

first noticed using 2.0.4, wrote that test using latest master 3...

thank you, I'll tale a look

@mpulkkinen please check 3.0.0-SNAPSHOT
@danieldietrich I plan to release next bugfix release. The version will be <IP_ADDRESS>. What's your opinion about fourth digit?

@ruslansennov we have to re-release every Javaslang module then when the core is released, even if the module did not change.
It will save us from total chaos, if we have to break compatibility for some reason. But it is only a matter of organisation to get semantic versioning right.
I would keep major and minor in sync and use the patch version for each module on its own.

@ruslansennov seems to work, closing


Add ESLint and Prettier configs; Apply linting;

The PR introduce ESLint and Prettier support 🍾

Linting is set up on a pre commit hook so it is impossible to commit code without compiling and linting it
There are base configs for both ESLint and Prettier
Editor extension are available for prettier so a code will be formatted on every save

Only cli and lib projects are linted at that moment and warnings are ignored. We have to try this config first and make sure we are comfortable with it. Next steps are:

Linting tests as well
Linting other projects
Fixing warnings

I belive that code consistency is more important than personal preferences yet I want to be absolutely sure that you guys – @spalladino and @jcarpanelli  are comfortable with these rules. Let me know if anything feels wrong and we'll figure it out.
PS As @spalladino requested semicolons are in.

Absolutely we'll remove it!


style config options are not working

Configuring style not working.
    {
            "dgox16/oldworld.nvim",
            lazy = false,
            priority = 1000,
            config = function(opts)
                    require("oldworld").setup({
                            styles = {
                                    functions = { italic = false},
                            },
                    })
                    vim.cmd [[ colorscheme oldworld ]]
            end
    }

Thank you for the feedback. I've already fixed it. I had a flaw when setting default styles. From now on, all styles are disabled by default, and you can choose which ones to activate. In your case, it's not necessary to disable the style for functions now, but to achieve the previous design, you may want to enable italics in comments again.


Missing .editorconfig

Shouldn't we create an .editorconfig file?

We definitely should.

Go ahead @Nipher -san!

I'm looking at some files, and I'm seeing a lot of different settings...
like, some of them are:
indent_style = tab
indent_size = 4

Others are:
indent_style = space
indent_size = 2

and even:
indent_style = tab
indent_size = 8

So I'm confuse about what settings to use, but I like the second one, for all files...
and of course, others settings as well:
# Unix-style newlines with a newline ending every file
[*]
end_of_line = lf
insert_final_newline = true

# Set default charset
[*]
charset = utf-8

# Indentation
[*]
indent_style = space
indent_size = 2

What do you guys think?

I like, go ahead with your configs.
What do you think @davegomez ?

I've added this line too:
max_line_length = 80
and theses for .md file
[*.md]
max_line_length = 0
trim_trailing_whitespace = false


New class Mmap.

Change Fiber to use Mmap instead of LibC.mmap.
Move mmap related functions and constants from libc.cr to mmap.cr.
@asterite This class has several future uses.

Thread stack allocation so multithreading can finally work with fibers.
Everything else you would use mmap for (shared semaphores, zero copy io, databases, etc).

We need to review this with @waj, but since it's mostly for the scheduler and fibers there's no rush because it's internal API.

Yeay. Mmap definitely deserves not being just one of a number in LibC.

@asterite It's not just for those 2 use cases.  The user below wanted an efficient replacement for seek/read with random access files.

buggs   is there an mmap interface?
buggs   for ruby i might just adabt a mmap extension i have

Another discussion:

jhass   then it's cheaper to mmap/read into memory and read all \n positions
BlaXpirit       then seek from the end, leave the rest in the buffer, read into it again
jokke   jhass: hm mmap would be nice indeed

There is certainly utility and some desire for a public mmap interface.

@technorama yes I'd like to make a Crystal version of BoltDB (https://github.com/boltdb/bolt)
Not in any rush to do so, but would be nice if it's possible

I'm closing this. Not that it's wrong or bad, but I wouldn't use the MMap wrapper internally because it adds a bit of overhead (it's a class). Maybe Mmap can be defined a shard, I don't know if it needs to be in the standard library.


Fix styling bugs

There are several styling bugs which have not been there in an earlier version. The screenshot show a comparison between a mininote instance hosted on my private server that was installed several months ago and an instance that was installed today. The first screenshot shows the way it should look like, the second one how it currently looks like.
It might be due to version changes in the frontend's dependencies or something else. Please feel free to help me find out and fix this :slightly_smiling_face:
Right

Wrong

Right

Wrong

what browser did you use?

nevermind, i think i know what are the issues. btw, the first image doesnt quite clear, you want the alert to be on top of the form?

It appears in both Firefox and Chrome.
Concerning the first screenshot: the alert should overlap the search bar, but instead it shows up above it and shifts the entire page down while it's there (maybe a problem of relative vs. absolute positioning?).

yeah, that's the case. i figured that you set the position to fixed but it got overwritten to relative. checking it out right now


Compilation warning

==> xcribe
Compiling 19 files (.ex)
warning: Phoenix.json_library/0 is undefined (module Phoenix is not available or is yet to be defined)
  lib/config.ex:112: Xcribe.Config.json_library/0

Generated xcribe app

#55


Several bugs for node-exporter/pod-gpu-node-exporter-daemonset.yaml

I'd like to collect node/pod metrics for GPU, and when applying pod-gpu-node-exporter-daemonset.yaml, I find several bugs and suggestion

the output of nvidia-dcgm-exporter can NOT be integrated into node-exporter, the root cause is: the metrics of nvidia-dcgm-exporter is redirect to /run/prometheus/dcgm.prom, but in node-exporter it is set to collect from /run/dcgm , so of course you can get NOTHING

pod exporter does NOT work and throw an error says

failed to get devices Pod information: failure connecting to /var/lib/kubelet/pod-resources/kubelet.sock: context deadline exceeded

I check the /var/lib/kubelet/pod-resources under my node but it's an empty folder. I am using k8s 1.14 where kubelet.sock is located under /var/lib/kubelet/device-plugins.
Then following this issue, I modify the hostpath to the folder of   kubelet.sock, while the pod-exporter raise a new exception:

failed to get devices Pod information: failure getting pod resources rpc error: code = Unimplemented desc = unknown service v1alpha1.PodResourcesLister

I assume this is due to incompatible k8s protobuf, so may you provide the corresponding docker image?

From the guide, I assume the key of GPU metrics between node-exporter and pod-exporter is totally seem, such as dcgm_gpu_utilization, which is not a good pratice as I can not simply distinguish them. You should give them different key, like dcgm_gpu_utilization for node level, and pod_gpu_utilization for pod level

Moreover, not sure if pod exporter can expose the metrics of GPU limits and requests, which is provided when user submit a pod?

Do you have all the pre-reqs completed https://github.com/NVIDIA/gpu-monitoring-tools/tree/master/exporters/prometheus-dcgm/k8s/pod-gpu-metrics-exporter#prerequisites Seems like you have not set the KUBELET_EXTRA_ARGS=--feature-gates=KubeletPodResources=true. For pod resources there should be a separate socket under /var/lib/kubelet/pod-resources when you set enable the PodResources feature gate

I gave same problem with k8s v1.16 which set KubeletPodResources true in default.
In a gpu server,  I try to run curl localhost:9400/gpu/metrics , but it returns connection refused.
I assume that exporter fails to bind 9400 port in the server with silence.

In a gpu server, I try to run curl localhost:9400/gpu/metrics , but it returns
E1204 04:49:12.763582       1 http.go:48] error responding to <IP_ADDRESS>:9400/gpu/metrics: open /run/dcgm/dcgm-pod.prom: no such file or directory

@kuromt this endpoint is available only inside the container unless you do port-forwarding. You are combining docker run instructions with k8s. If you want to see the metrics on the host, you will need to do
$ kubectl port-forward "dcgm-pod-name" 9400

@damon008 this means you have error running the exporter. Please make sure you follow the pre-reqs and run exported on a supported GPU.


Adguard for Mac 2.16.2 crashes during HTTP/3 filtering

Separate problem from comment:
https://github.com/AdguardTeam/CoreLibs/issues/1944#issuecomment-2561228160

@cassperine AdGuard for Mac 2.16.3 is out with a fix


Sudden maximum CPU usage for istio pilot

Bug Description
I am running into an issue where I have istiod pods suddenly increasing to the maximum CPU limit. Regardless on how high I set the CPU limit on the pod, it increases to 100% of the CPU limit. It then also seems that updates to the istio service mesh are not propagated anymore when it's in this state, as if the pilot process is stuck in a loop or blocking.
Deleting the pod, which of course starts a new pod, resolves the issue for a while, but it will eventually get into this broken state again. It normally takes a couple of hours, sometimes a day or so. It's not clear what is triggering the event. It has happened on times where I was not doing any updates on the mesh.
As seen on this screenshot, the CPU usage suddenly spikes to the limit, and stays there:

I've ran a CPU profile task using go tool pprof -http=:8888 localhost:8080/debug/pprof/profile which resulted in the following graphviz output:

This result doesn't tell me a lot. I'm not sure how to proceed further investigating this behavior. I'm not seeing any error logs on the pilot container.
Are there any other logs or tools I should be looking at?
I hope someone can help me with this, and I'm happy to provide more information if necessary.
Version
Istio version: 1.14.3

Additional Information
No response

Did you enable PILOT_ENABLE_ANALYSIS to true?

Already fixed in https://github.com/istio/istio/pull/40595, just not merged.

We had PILOT_ENABLE_ANALYSIS set to true indeed. Thank you for fixing the issue!


WebView_Flutter_Web - When navigating to a page using a webview containing a google form link, a new tab is opened

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a similar example to what is provided with the WebView_Flutter_Web package but have the user land initially on a page and navigate to the page containing the google form webview using the get package.

Host or run the web app on a local network and connect to the app via google chrome browser on an android phone.

Navigate to the webview page and noticed that a new tab is opened instead of displaying the webview within the current web app page.

Expected results
Like other URL's provided to the PlatformWebView Widget, we expect the app to open a webview of the google form within the current app page; as opposed to opening a new browser tab.
Actual results
When the user navigates to the page containing the google form webview, they are navigated to a new tab, and grey area is displayed on the page of the web app where the webview should show.
Code sample
Code sample
// Copyright 2013 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

// ignore_for_file: public_member_api_docs

import 'dart:async';
import 'dart:typed_data';
import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:webview_flutter_platform_interface/webview_flutter_platform_interface.dart';
import 'package:webview_flutter_web/webview_flutter_web.dart';

void main() {
  WebViewPlatform.instance = WebWebViewPlatform();
  runApp(const GetMaterialApp(home: HomePage()));
}

class _WebViewExample extends StatefulWidget {
  const _WebViewExample();

  @override
  _WebViewExampleState createState() => _WebViewExampleState();
}

class _WebViewExampleState extends State<_WebViewExample> {
  final PlatformWebViewController _controller = PlatformWebViewController(
    const PlatformWebViewControllerCreationParams(),
  )..loadRequest(
      LoadRequestParams(
        uri: Uri.parse('https://forms.gle/jUJHbzs6sHPS9rFE6'),
      ),
    );

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Flutter WebView example'),
        actions: <Widget>[
          _SampleMenu(_controller),
        ],
      ),
      body: PlatformWebViewWidget(
        PlatformWebViewWidgetCreationParams(controller: _controller),
      ).build(context),
    );
  }
}

enum _MenuOptions {
  doPostRequest,
}

class _SampleMenu extends StatelessWidget {
  const _SampleMenu(this.controller);

  final PlatformWebViewController controller;

  @override
  Widget build(BuildContext context) {
    return PopupMenuButton<_MenuOptions>(
      onSelected: (_MenuOptions value) {
        switch (value) {
          case _MenuOptions.doPostRequest:
            _onDoPostRequest(controller);
            break;
        }
      },
      itemBuilder: (BuildContext context) => <PopupMenuItem<_MenuOptions>>[
        const PopupMenuItem<_MenuOptions>(
          value: _MenuOptions.doPostRequest,
          child: Text('Post Request'),
        ),
      ],
    );
  }

  Future<void> _onDoPostRequest(PlatformWebViewController controller) async {
    final LoadRequestParams params = LoadRequestParams(
      uri: Uri.parse('https://httpbin.org/post'),
      method: LoadRequestMethod.post,
      headers: const <String, String>{
        'foo': 'bar',
        'Content-Type': 'text/plain'
      },
      body: Uint8List.fromList('Test Body'.codeUnits),
    );
    await controller.loadRequest(params);
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Home'),
      ),
      body: Center(
          child: ElevatedButton(
        child: Text('Go'),
        onPressed: () {
          showDialog(
              context: context,
              builder: (context) => Dialog(
                    child: Text("Pressed"),
                  ));
          Get.to(_WebViewExample());
        },
      )),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.2 22D49 darwin-x64, locale en-US)
    • Flutter version 3.10.6 on channel stable at /Users/brycerichards/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (8 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[!] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/brycerichards/Library/Android/Sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/brycerichards/Library/Android/Sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.80.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] VS Code (version 1.70.2)
    • VS Code at /Users/brycerichards/Downloads/Visual Studio Code 2.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.2 22D49 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.179

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Hi @BJRichardsAZ
I think this works as intended with the mechanism to automatically open a new tab of google form. Although I haven't found specific technical reference yet, check out the demo with the following pure HTML example:
https://github.com/flutter/flutter/assets/104349824/f683ac24-1031-4455-bb70-ec95f2cf65e1

HTML sample code
<!DOCTYPE html>

My First Heading

My first paragraph.

This is a link

So I'm going to close this as it's not issue with webview plugin. If you disagree, please write in the comments with more information and I will reopen it.


Yandex.Disk Error: Eof on big files

What is your rclone version (eg output from rclone -V)

rclone v1.39
os/arch: linux/amd64
go version: go1.9.2

Which OS you are using and how many bits (eg Windows 7, 64 bit)

Ubuntu 16.04.2 LTS

Which cloud storage system are you using? (eg Google Drive)

Yandex.Disk

The command you were trying to run (eg rclone copy /tmp remote:tmp)

rclone copy --log-level=DEBUG --transfers 4 --size-only --buffer-size 500M wdmycloud: yandexdisk:wdmycloud
rclone copy wdmycloud: yandexdisk:wdmycloud
rclone copy --log-level=DEBUG --transfers 1 --buffer-size 500M wdmycloud: yandexdisk:wdmycloud

A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)

2018/01/14 16:59:46 DEBUG : <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
2018/01/14 16:59:46 DEBUG : HTTP RESPONSE (req 0xc420457800)
2018/01/14 16:59:46 DEBUG : Error: EOF
2018/01/14 16:59:46 DEBUG : <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
Initial discussion: https://forum.rclone.org/t/rclone-copy-stucks-at-99/4575/10

seems it works in case when I'm connecting direct to my NAS from local network.
Previously I did it from digital open vps, NAS is exposed via router. maybe router adds some networking issues.
I have copied many of 1gb files, seems I can close this issue.
thanks for your help. Maybe I'll get back to experiment with NAS - Router - VPS - Yandex again later for big files.

Glad you've got closer to finding the issue - thanks for testing.


Correct YAML indentation

Remove some idle indentations and fix env block indents 'cause of inability to use envs alongside envFromSecrets

Sorry for the late reply. Why would we remove the usages of nindent?


Add blocking progress support for NVLINK in UCX (not UCX-PY)

UCX-py relies on ucp_worker_arm, ucp_worker_get_efd and ucp_worker_signal to wait for send/recv event completion especially in the context of unblocking coroutines. It's the responsibility of different UCTs like IB, TCP, cuda-ipc, to provide this capability. IB and TCP have native capability to notify completion of events through file descriptor whereas CUDA doesn't. In addition, cuda_ipc was never conceived with blocking progress in mind and we will have to rely on stream callbacks or invoking host functions to support this. Current status is to overcome high overheads in stream callbacks when event query occurs on the same stream as the one when where callbacks are issued.

Additions on this branch address this issue: https://github.com/Akshay-Venkatesh/ucx/tree/topic/cuda-ipc-event-notif-fd
Able to improve performance from 350 MB/s -> 7 GB/s but still not peak.

UCX-py is still not seeing the benefits of adding blocking progress in CUDA-IPC. Looks like some other capability needs to be added but this may not be a huge delta but about figuring out why CUDA-IPC UCT isn’t being picked up with server-client mode.

Worked with UCX developer @bureddy to narrow down the cause of this issue to configuring cuda_ipc iface (https://github.com/openucx/ucx/pull/3479) and to the test itself
We're able to reach close to expected perf now

I believe we can close this.  The new bindings support blocking and non-blocking API calls.  We can reoopen if cuda_ipc transfers is not performant


FormatException: Unexpected character (at character 1)

Describe the bug
If the Edge Function completes its execution due to a timeout, we receive a JSON parsing error. Although, it would be more logical to receive a timeout error
To Reproduce

Write a function that will run for a very long time.
Call this function in a Flutter application.
Expect an error.

Expected behavior
It would be correct to receive a timeout error
Screenshots

Version (please complete the following information):
├── supabase_flutter 1.10.14
│   ├── supabase 1.11.1
│   │   ├── functions_client 1.3.2
│   │   ├── gotrue 1.11.2
│   │   ├── postgrest 1.5.0
│   │   ├── realtime_client 1.2.1
│   │   ├── storage_client 1.5.1

The best fix for this issue needs some breaking changes, which will come in v2. For now, I suggest you to wrap it in a try catch and handle the error yourself.

This is now fixed with v2 #673


Split "dumb" and "smart" components

Consider using something like Context for passing props around

 Split dumb and smart components

Currently Pages and Sections hold the "smart" part and the components in ~component hold the "dumb" part. This can be closed


Change encoding for json.loads

Remove encoding argument from json.loads for support to pyhton 3.9 #19

Hey @nbering, can this patch be merged? Python 3.9 is out since early October this year and it is only a matter of time before it starts rolling out into distros.

@nbering Are you able to get this merged? I am using @obilodeau solution to work on python 3.9

@dverbeek84, for the record, this is not my patch. It is @emrahcetiner's.
It can be quickly applied like this:
cd YOUR_TERRAFORM.PY_LOCATION
wget https://patch-diff.githubusercontent.com/raw/nbering/terraform-inventory/pull/20.diff
patch -p1 < 20.diff
rm 20.diff

@nbering: Arch Linux is now on Python 3.9. terraform.py will now issue a fatal TypeError over there. Can you please merge this patch? Thanks!

Any update on this? Homebrew Python is on 3.9 and this script fails

Any update on this? Homebrew Python is on 3.9 and this script fails

Fedora 34 is Py3.9 now too so it's broken there as well without this patch


updated first.php fixed issue #5

Signed-off-by: manu553<EMAIL_ADDRESS>

rejected my push comes first


[Site] Fix header size

Description
Reduce the header length to make it more compact and accordingly resize the smp logo to fit the header. This will take some attempts at playing around with different sizes for the header and the smp logo to achieve the perfect combination.
Steps to reproduce

Go to https://smp-spec.io/
Observe the header

I would love to take up this task

Go ahead 👍  @St80ene

@St80ene Thank you for taking this on! Let me know if you need any help 🎈

Closed by #122


Add docs for upgrade command in cli

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Add docs for upgrade command in cli
Issue reference
Closes: #1163

@msfussell I can't approve my own PR, but the changes looks good. @AaronCrawfis I think we'll need your approval here to get it merged.


Nested ManyToMany Relationships In Relay Nodes Are Broken

I have read the docs thoroughly before making this bug report.

[X] Yes

I have read through other open issues, and my issue is not a duplicate.

[X] Yes

What version of the library you are using?
0.2.6
Which python version are you using?
3.12
What operating system are you on?
Mac
Description
When querying a ManyToMany field as a nested relay node, I run into the following error:
2024-03-07T04:02:05+0000 | INFO | project.logging.resolve:40 | Traceback (most recent call last):
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/tests/project/logging.py", line 38, in resolve
    return next(root, info, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/miniconda3/envs/gql-py11/lib/python3.11/site-packages/graphene/relay/node.py", line 106, in node_resolver
    return cls.get_node_from_global_id(info, id, only_type=only_type)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/miniconda3/envs/gql-py11/lib/python3.11/site-packages/graphene/relay/node.py", line 131, in get_node_from_global_id
    return get_node(info, _id)
           ^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/types.py", line 78, in get_node
    return optimize_single(queryset, info, pk=pk, max_complexity=cls._meta.max_complexity)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/compiler.py", line 91, in optimize_single
    optimized_queryset = optimizer.optimize_queryset(queryset)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/optimizer.py", line 72, in optimize_queryset
    results = self.compile(filter_info=filter_info)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/optimizer.py", line 110, in compile
    self.compile_prefetch(name, optimizer, results, filter_info)
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/optimizer.py", line 140, in compile_prefetch
    queryset = self.paginate_prefetch_queryset(queryset, name, filter_info=filter_info)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rahulsomani/git/graphene-django-query-optimizer/query_optimizer/optimizer.py", line 174, in paginate_prefetch_queryset
    field_name: str = field.remote_field.attname
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'ManyToManyRel' object has no attribute 'attname'

This is the GQL setup on the Python side (slightly modified from the example):
class DeveloperNode(DjangoObjectType):
    class Meta:
        model = Developer
        interfaces = (relay.Node,)

class HousingCompanyNode(IsTypeOfProxyPatch, DjangoObjectType):
    real_estates = DjangoConnectionField(RealEstateNode)

    class Meta:
        model = HousingCompanyProxy
        interfaces = (relay.Node,)

Breaking queries:
# Fails
{
  pagedHousingCompanies {
    edges {
      node {
        name
        developers {
          edges {
            node {
              id
            }
          }
        }
      }
    }
  }
}

# Fails
{
  housingCompany(id:"SG91c2luZ0NvbXBhbnlOb2RlOjIw") {
    developers {
      edges {
        node {
          id
        }
      }
    }
  }
}

This only happens when querying developers as a DjangoConnectionField. If we set it up as a DjangoListField, there's no issues in the query

Seems like I missed a case for forward many-to-many relations when determining partitioning field name for prefetch queryset windowing. Quick fix, added in 0.2.7.


fix hackage link in readme

otherwise github thinks it's a relative link to a file within this repo

Thanks!


Reset time on local server is tooooooo long.

Reset time on local server (for development purpose) is tooooooo long.
Currently, it takes about 5-10 mins, which is super inconvenient, and which does not make much sense.
As a comparison, if you reset local cosmos db emulator, it takes only a few seconds.
Is it possible to reduce the reset time on local server?
Thanks in advance.

As a user/developer, I need reset local cluster very often. I desperately wish the reset time could be reduced. Thanks.

Also stop/start local cluster is very slow too. Is it possible to restart?

any update on this?

My team and I also share this concern.
Is there any way to update the application parameters without having to wait five or six minutes for the local cluster to reset? That would be a very helpful step toward more efficient development.


Option to validate only api and ignore the rest

Tried:
pyramid_swagger.enable_path_validation = false
pyramid.includes = pyramid_swagger
but enable_path_validation doesn't seem to work, the pyramid_swagger still validates the paths not in swagger.json
also tried:
pyramid_swagger.exclude_paths = ['^(?!/api/).*$']
pyramid.includes = pyramid_swagger
but exclude_paths seem to either exclude all the paths, or none at all.
Help will be appriciated.
Maor

Hi Maor,
Can you help me out with a bit more diagnostic information? It would be useful to know:

What version of pyramid_swagger you're using
The complete configuration you're using
A minimal swagger spec that reproduces the issue
A specific curl that isn't behaving like you expect

I had the same problem.
I think registry.settings.get(..) always returns string, and causes the regex compiler to generate the wrong pattern.
you should try with something like this:
pyramid_swagger.exclude_paths = ^(?!/api/)

Ah thanks @deubs. Sounds like it's just the INI format being picky about strings then?

I guess so, i think you shouldn't test instance types here .As always are strings.
Maybe to define multiple patterns would be convenient to use multi line values, or comma separated values (without quotes) and then split...


Define the Dataset

There are various datasets listed in the README.md file in the dataset-defination branch that are available in the Dataset directory, yet their defination is not available.
Explore such datasets and add a separate markdown file in order to define them. This will make the other contributors to easily understand and work with their desired datasets.
The name of the file should be same as that of the dataset (i.e. <dataset-name>.md) and provide a link to the newly created file in readme, as well.

@all-contributors please add @ravi-prakash1907 for Data


Add CI workflow with model repo, related to to issue #122

@allaway You can see the working chained workflows here:

https://github.com/gf-dcc/data_curator/actions/runs/2332796101 (update model URL in config)
https://github.com/gf-dcc/data_curator/actions/runs/2332798496 (shiny deploy triggered by above)

This is currently a draft because I need to confirm/resolve two things for NF:

For workflows to trigger other workflows, one needs to use a PAT instead of the default action token. See https://github.com/ad-m/github-push-action/issues/32. For GF, I set up a bot account, same idea as https://github.com/google-github-actions-bot.
The PAT needs to have repo scope and be available to this repo -- so we can use REPO_PAT already used for shiny-deploy or set up something new. Once that's confirmed, the workflow can refer to the correct secret.

We have a bot account, @nfosi-service. I'll share the credentials with you on LP! I cannot recall whether a PAT already exists in LP but probably best to just create a new one anyway...

Alright, I just set @.*** as the default email address
for 2FA emails, and shared the creds with you on LP!
robert allaway (he/him)
principal scientist
sage bionetworks
seattle, wa
On Tue, May 17, 2022 at 9:20 AM Robert Allaway @.***>
wrote:

We have a bot account, @nfosi-service https://github.com/nfosi-service.
I'll share the credentials with you on LP! I cannot recall whether a PAT
already exists in LP but probably best to just create a new one anyway...
—
Reply to this email directly, view it on GitHub
https://github.com/nf-osi/NF_data_curator/pull/133#issuecomment-1129068127,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWYPLFY663EDE5YBOWQ4SLVKPBLDANCNFSM5WBT4GCQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks, this is great! Looking forward to seeing this in action!


C# client Stop can take a while if it spams Sends then tries to Stop

while (someCondition == false)
{
    _ = hubConnection.SendAsync("Ping");
}

// Takes a long time to complete
hubConnection.StopAsync();
// or
hubConnection.DisposeAsync();

The SendAsync's are all waiting on a lock and when StopAsync/DisposeAsync tries to run it waits on the same lock. And I've observed it taking over a minute for the Stop/Dispose to finish

We should have an internal CancellationToken that we trigger 😎


Fix testGroupByAggregationWithLimitZero expected type

https://github.com/apache/pinot/actions/runs/12479172654/job/34827888272 from https://github.com/apache/pinot/pull/13564

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

Project coverage is 0.00%. Comparing base (59551e4) to head (40821d5).
Report is 1505 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (40821d5). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (40821d5)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14710       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


excessive regrid warning

We shouldn't have this warning since regridTool is set to esmf as default.
/export_2/zhang40/anaconda2/envs/acme_diags_env/lib/python2.7/site-packages/cdms2/avariable.py:1142: Warning:
avariable.regrid: We chose regridTool = esmf for you among the following choices:
   Tools ->    'regrid2' (old behavior)
               'esmf' (conserve, patch, linear) or
               'libcf' (linear)
  warnings.warn(message, Warning)

@chengzhuzhang @dnadeau4  Is the default "conservative" or "linear"?   I was just looking at Curt Covey's code this morning and assumed it was using "regrid2"... guess I was wrong.
g = cdms2.open('dailyMean034Jul1999-2005std_of_dailymeans.nc')
v = g['dailySD'] # Randomly chosen variable on CESM-LE grid
outgrid = v.getGrid()
print 'Target-grid shape:', outgrid.shape
tvarbnew = tvarb.regrid(outgrid)

The default is tool="libcf",  method="linear" when data has no bounds defined.
The default is tool="esmf",  method="linear" when data has bounds.
The default is esmf if method=conservative or patch.
The default for non-uniform grids, tool=esmf, method=linear

@dnadeau4 Thanks... good to know!

@dnadeau4 we used 'regridTool' and regrid_tool  = 'esmf'. it doesn't seem like an misspelling...
mv2_reg = mv2.regrid(mv_grid, regridTool=regrid_tool,
regridMethod=regrid_method)

@dnadeau4 I didn't put the keyword 'regridTool=' and 'regridMethod=' in one regrid line, this triggered the warning messages...This is fixed in commit https://github.com/ACME-Climate/acme_diags/commit/25e01721911ad0896802ee7d14bab2904a36f47e
Thank you for troubleshooting with us!

@dnadeau4 sorry to bother you one more time...if 'conserve' (this is in the warning message) or 'conservative' both okay for conservative regrid regard method?

We had the arguments mixed up, where the regrid tool and regrid method were swapped. Hence, that's why it defaulted to esmf with linear regridding.
Everything is fixed in 25e01721911ad0896802ee7d14bab2904a36f47e.


Resource tree code merge

Codecov Report

Merging #2322 into dev will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #2322      +/-   ##
============================================
- Coverage     31.07%   31.05%   -0.03%
+ Complexity     1708     1706       -2
============================================
  Files           337      337
  Lines         17679    17679
  Branches       2206     2206
============================================
- Hits           5494     5490       -4
- Misses        11623    11627       +4
  Partials        562      562

Impacted Files
Coverage Δ
Complexity Δ

...scheduler/alert/template/AlertTemplateFactory.java
42.85% <0.00%> (-28.58%)
2.00% <0.00%> (-1.00%)

...he/dolphinscheduler/api/service/LoggerService.java
91.30% <0.00%> (-8.70%)
7.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

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


support 3.6.0

Codecov Report

Merging #574 (a2f821e) into master (ba23131) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #574   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines          755       751    -4
  Branches       106       106
=========================================
- Hits           755       751    -4

Impacted Files
Coverage Δ

nbqa/config/config.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


"Sel: 0|0" duplication in status bar

System information:

Operating system (distribution) and version: KDE Neon based on Ubuntu 18.04
DBeaver version: 6.3.2
Additional extensions: none

Describe the problem you're observing:
After using for a while, the status bar ends up with multple "Sel: 0|0" entries which don't appear to change or represent anything.

Steps to reproduce, if exist:
No specific steps but seems to come from general usage, opening and switching between SQL Editors.

Seems to be Eclipse bug. Try to reset UI settings (main menu File)

That works as it restarts DBeaver and restarting seems to fix things temporarily.  I've also noticed the "Set Encoding..." menu item in the "Edit" menu also starts to duplicate until restarted.

is it still reproducible?

I haven't seen it recently but I'm now working from home on a different operating system. I won't be able to check my work machine for quite a while.


Remove Extra Mount Targets

We added extra mount targets to the FSS to improve performance. Since we have deprecated the FSS we can remove these unused mount targets.
This can be done from the OCI Console.

Did this a while back.


⚠ Jenkins Plugins Artifactory has degraded performance

In b2633f7, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1330 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 89b7144 after 57 minutes.


Provide 'speed' option for libimagequant

https://sharp.pixelplumbing.com/api-output#png
PNG output has many options for libimagequant but speed option is missing. https://pngquant.org/lib/#liq_set_speed
It would be great to get access to that param.

Hi, you'll need to add support for this to libvips first.

Hi, you'll need to add support for this to libvips first.

v0.30.0 now available with this feature.


Remove Jquery and Use fetch Api

This is failing because of lint issues. Please rebase + fix the lint.


tf lite model error

Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: FULLY_CONNECTED. Here is a list of operators for which you will need custom implementations: IdentityN.

Hey @prakHr
I'm attempting to triage issues in order to speed up the debugging process. Can you please give an example of the operators you tried to run to see if this is a bug or an intentional exclusion? If this is not a bug it might be best labeled as type:feature and comp:lite just to get the right eyes on this project!

@prakHr,
If you are using the Python converter, with the latest version of TF/TFLite (nightly maybe?) could you try adding the following line pre-conversion:
converter.experimental_new_converter = True
If that doesn't work, it looks like we don't support the op yet :-(. You could try using Select TF ops if you are using our C++/Java APIs. Thanks!

@prakHr, Is this still as issue ?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Expose the insight component as a JavaScript API in the NPM package

Task list:

[ ] Create an Elm application called Insight.elm inside the cli folder.

The application should be initialized similar to this: https://github.com/finos/morphir-elm/blob/6be48f261a69292395b57619926d3ec973f4664f/cli/src/Morphir/Web/DevelopMain.elm#L29-L35
It should take the IR through Flags as described here: https://guide.elm-lang.org/interop/flags.html

The output of morphir-elm make is a Distribution that can be decoded from JSON using this function: https://github.com/finos/morphir-elm/blob/6be48f261a69292395b57619926d3ec973f4664f/src/Morphir/IR/Distribution/Codec.elm#L65-L66

It should take the value expression through a Port as described here: https://guide.elm-lang.org/interop/ports.html

The value expression will be passed in as a Value that can be decoded from JSON using this function: https://github.com/finos/morphir-elm/blob/6be48f261a69292395b57619926d3ec973f4664f/src/Morphir/IR/Value/Codec.elm#L200-L201
The decode function requires two additional decoders for type and value annotations. We don't expect any annotations to be passed in so we can pass Decode.succeed () to always pass Unit without decoding.

Once both of those are available the view function should invoke the visualization code the same way as it's done here: https://github.com/finos/morphir-elm/blob/6be48f261a69292395b57619926d3ec973f4664f/cli/src/Morphir/Web/DevelopMain.elm#L239-L241

[ ] Add a build step to compile Insight.elm into insight.js.
[ ] Add insight.js to the files section in package.json
[ ] Publish the NPM package as described here: https://github.com/finos/morphir-elm/blob/master/docs/publishing.md

This step can only be done by a maintainer for now.

Building the Elm application will require understanding The Elm Architecture (TEA) so if you are not familiar please read this first: https://guide.elm-lang.org/architecture/
Let me add more details on the design. The component's lifecycle should be the following:

To initialize the component the consumer should pass in the JSON that was previously generated by morphir-elm make. This should be done by using Flags.
The Elm application should decode the JSON into a Morphir.IR.Distribution.Distribution using the decoder mentioned above.
If the decoding succeeds the application should store the Distribution in the model and show an empty  to signl successful initialization.
It the decoding fails it should show an error message.
When the consumer is ready to specify which function should be explained they should pass in the fully-qualified name of the function. This should be done using a Port.
Based on the fully-qualified name the application should look up the function definition in the Distribution.
If the function is found it should invoke Morphir.Visual.ViewValue.view with the body of the function. This will show a visualization of the function without data.
If the function is not found it should show an error message.
When the consumer is ready to pass in the arguments for the function they should pass in a list of arguments as a list of JSON objects that can be decoded into Morphir.IR.Value.Value as described above. This should be done through a port.
The application should decode each argument and pass it to the Morphir.Visual.ViewValue.viewWithData function. If the function doesn't exist please use a placeholder that simply ignores the data and calls the view function instead.
If the decoding fail the app should display an error message.

To sum up the application will interact with JavaScript through:

Flags to get the Distribution at startup.
A port to get the fully-qualified name of the function to explain (without data).
A port to get  the arguments of the function to explain with data.

The model should capture the following states of the application:

Initialized with a distribution - this should show an empty .
Function name set - this should show the explanation without data.
Function arguments set - this should show the explanation with data.
Failed - this should show an error message.

Building the Elm application will require understanding The Elm Architecture (TEA) so if you are not familiar please read this first: https://guide.elm-lang.org/architecture/
Let me add more details on the design. The component's lifecycle should be the following:

To initialize the component the consumer should pass in the JSON that was previously generated by morphir-elm make. This should be done by using Flags.
The Elm application should decode the JSON into a Morphir.IR.Distribution.Distribution using the decoder mentioned above.
If the decoding succeeds the application should store the Distribution in the model and show an empty  to signl successful initialization.
It the decoding fails it should show an error message.
When the consumer is ready to specify which function should be explained they should pass in the fully-qualified name of the function. This should be done using a Port.
Based on the fully-qualified name the application should look up the function definition in the Distribution.
If the function is found it should invoke Morphir.Visual.ViewValue.view with the body of the function. This will show a visualization of the function without data.
If the function is not found it should show an error message.
When the consumer is ready to pass in the arguments for the function they should pass in a list of arguments as a list of JSON objects that can be decoded into Morphir.IR.Value.Value as described above. This should be done through a port.
The application should decode each argument and pass it to the Morphir.Visual.ViewValue.viewWithData function. If the function doesn't exist please use a placeholder that simply ignores the data and calls the view function instead.
If the decoding fail the app should display an error message.

To sum up the application will interact with JavaScript through:

Flags to get the Distribution at startup.
A port to get the fully-qualified name of the function to explain (without data).
A port to get  the arguments of the function to explain with data.

The model should capture the following states of the application:

Initialized with a distribution - this should show an empty .
Function name set - this should show the explanation without data.
Function arguments set - this should show the explanation with data.
Failed - this should show an error message.

Is there any function defined in the Distribution file which can lookup function definition in distribution?

Is there any function defined in the Distribution file which can lookup function definition in distribution?

Is there any function defined in the Distribution file which can lookup function definition in distribution?

No. There is a way to look up a specification but not a definition yet. Please implement it using the lookupValueSpecification as a template.

Is there any function defined in the Distribution file which can lookup function definition in distribution?

No. There is a way to look up a specification but not a definition yet. Please implement it using the lookupValueSpecification as a template.

Merged a PR that added some of the missing features mentioned above: https://github.com/finos/morphir-elm/pull/278

Merged a PR that added some of the missing features mentioned above: https://github.com/finos/morphir-elm/pull/278


error messages: suggest field/method with that name exists

Description
When a user types uses a field instead of a method, or the other way around, the error message is not very clear:
error: content does not contain field "location"

It would be great if Typst could provide a hint with the error message. Here are some examples:
error: content does not contain field "location". Did you mean to use a method instead of a field?

error: content does not contain field "location". Did you mean "location()"?

error: content does not contain field "location". A method with that name exists.

error: content does not contain field "location" but has method "location()".

Use Case
I often forget which fields are methods or fields. When I encounter this message, I think I either made a typo or I am performing the field lookup on the wrong variable. Debugging would be faster when hints like this are provided.

Closed by https://github.com/typst/typst/pull/2977


Automate - fix inherits problem.

Modified test.
https://bugzilla.redhat.com/show_bug.cgi?id=1301123

@mkanoor
Please Review

@mkanoor
Removed inheritance test from miq_ae_engine_spec.
Please Review

:+1:

@gmcculloug
Please Review


Wrong SteamId

Linked to this issue : https://github.com/Facepunch/webrcon/issues/16
The UserId is send as an Integer and interpreted as well by the js part. But it exceed the maximum js integer, which is<PHONE_NUMBER>740991 (16 digits), the steamId is 17 digits long. That's why the last digit was rounded.
The other way is to send the UserId as a string instead of force the js part to re-interpret it.

Can confirm this fixes the issue.

Will fix this on the serverside


ImportError: No module named 'balkans'

Hi Eser,
I need to use this nlp library for my python project and I always get below error, unfortunately. Please help me.
Sincerely,
Enise

from engerek import ZemberekStemmer
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.4/dist-packages/engerek-1.0b0-py3.4.egg/engerek/init.py", line 1, in
from balkans import BalkansDeasciifier
ImportError: No module named 'balkans'

I have same problem, did this problem fix?

I have same problem now, have the problem been solved?

I have solved finally.
if you change the init.py file as:
from .balkans import BalkansDeasciifier
from .tokenizer import EngerekTokenizer
from .zemberek import ZemberekStemmer

Problem must be fixed.


Add os/cpu/processes info for all builders

Most recipes have been collecting os info for better understanding the bot status when test fails. We should enforce that on all recipes. Currently it is misses on

packages
firebaselab
cocoon
drone recipes

https://flutter-review.googlesource.com/c/recipes/+/46321 completes this.


Uncaught TypeError: Cannot read property '$apply' of undefined

Open a Markdown Document

Atom Version: 1.8.0
System: Mac OS X 10.11.6
Thrown From: markdown-document package, v0.2.0
Stack Trace
Uncaught TypeError: Cannot read property '$apply' of undefined
At /Users/max/.atom/packages/markdown-document/lib/markdown-document-view.coffee:238

TypeError: Cannot read property '$apply' of undefined
    at MarkdownDocumentView.mdOutline (/Users/max/.atom/packages/markdown-document/lib/markdown-document-view.coffee:238:13)
    at /Users/max/.atom/packages/markdown-document/lib/markdown-document-view.coffee:84:9
    at tryToString (fs.js:414:3)
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:401:12)

Commands
     -3:15.5.0 editor:newline (atom-text-editor.editor.is-focused)
     -3:00.2.0 core:move-right (atom-text-editor.editor.is-focused)
     -2:59.4.0 editor:newline (atom-text-editor.editor.is-focused)
     -2:55.8.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
     -2:55.2.0 core:backspace (atom-text-editor.editor.is-focused)
     -2:52.6.0 core:save (atom-text-editor.editor.is-focused.autocomplete-active)
     -2:38.9.0 core:move-right (atom-text-editor.editor.is-focused)
     -2:38.7.0 editor:newline (atom-text-editor.editor.is-focused)
  3x -2:36.7.0 core:save (atom-text-editor.editor.is-focused)
  7x -2:19.2.0 markdown-preview:preview-file (atom-text-editor.editor)
  2x -1:02.4.0 markdown-document:toggle (atom-text-editor.editor.is-focused)
  2x -0:43.3.0 markdown-preview:toggle (atom-text-editor.editor.is-focused)
  5x -0:16.1.0 core:select-right (atom-text-editor.editor.is-focused)
     -0:15 core:backspace (atom-text-editor.editor.is-focused)
     -0:14.7.0 core:save (atom-text-editor.editor.is-focused)
  3x -0:11.8.0 markdown-preview:toggle (atom-text-editor.editor)

Config
{
  "core": {
    "audioBeep": false,
    "disabledPackages": [
      "haskell-ghc-mod",
      "haskell-pointfree",
      "ide-haskell",
      "autocomplete-haskell",
      "ide-haskell-cabal",
      "ide-haskell-repl",
      "language-haskell",
      "build-grunt",
      "grunt-runner",
      "browser-plus",
      "build",
      "build-gulp",
      "build-npm-apm"
    ],
    "themes": [
      "atom-material-ui",
      "atom-material-syntax"
    ]
  }
}

Installed Packages
# User
angularjs, v0.3.5
atom-beautify, v0.29.10
atom-material-syntax, v0.4.6
atom-material-ui, v1.3.3
atom-mocha, v0.0.6
atom-typescript, v10.1.5
bottom-dock, v0.4.4
busy, v0.6.0
docker, v0.7.0
file-icons, v1.7.17
flex-tool-bar, v0.10.2
gulp-manager, v0.2.19
highlight-selected, v0.11.2
jenkins, v0.3.5
linter, v1.11.14
linter-docker, v0.1.2
linter-eslint, v7.2.4
linter-jsonlint, v1.3.0
linter-manager, v0.2.12
linter-markdown, v1.5.2
linter-tslint, v0.10.0
markdown-document, v0.2.0
markdown-scroll-sync, v2.1.2
minimap, v4.24.7
minimap-bookmarks, v0.3.1
minimap-find-and-replace, v4.5.1
minimap-git-diff, v4.3.1
minimap-highlight-selected, v4.4.0
minimap-selection, v4.4.0
node-debugger, v1.7.0
project-manager, v2.9.7
rest-client, v1.2.0
terminal-plus, v0.14.5
todo-manager, v0.2.9
tool-bar, v1.0.0
tool-bar-main, v0.0.9
travis-ci-status, v1.1.1

# Dev
No dev packages

Do you experience the error when you don't use markdown-preview?

Fwiw I'm seeing this as well, when markdown-preview is enabled or disabled.

Atom 1.10.2
Mac OS 10.12

The same problem here :( I like the scrivener-idee and would like to use it - somehow.


Datetime question

This utility is awesome, thanks for all the work. Question about datetime. I was looking in the Rochester, MN area, and received a result that looks like this;
{'name': u'QUIET BUILDING! CLOSE TO DOWNTOWN! BEST DEAL AROUND!', 'area': u'875ft2', 'url': u'https://rmn.craigslist.org/apa/d/rochester-quiet-building-close-to/6748094294.html', 'where': u'Rochester NW', 'price': u'$825', 'bedrooms': u'2', 'geotag': (44.036915, -92.464017), 'repost_of': u'6576850170', 'has_image': True, 'datetime': u'2018-12-12 16:15', 'has_map': True, 'id': u'6748094294'}

The datetime displayed is 2018-12-12. However, if I look at the listing based on its url, the date posted shows as: 2018-11-13. Is this something you've come across?

More info - it looks like datetime is showing the 'last updated' timestamp, rather than the initial post timestamp.

Yeah I noticed this as well, if I recall. My/most needs require storing scheduled results. So, do a check on your sql to see if the record is unique, then if it is, insert it, if not, don't. This way you can also track the posts update history. You're right though, you lose the ability to see exactly when it was posted, if you're checking every day it'll get you there though.
If you have any questions let me know

That's a valid point. In 66d1d6f I'm adding 'last_updated' as a key with the same value as 'datetime' (keeping 'datetime' for backwards compatibility). The information of when was the entry actual created is in the detail, so if include_details is set to True, the key 'created' will be included with the datetime of when was it actually created.
Hope this helps.


When disabling Scrollify page doesn't scroll at all

I am working on a project in Webflow, and for some reasons I need to disable scrollify on mobile. I tried setting touchScroll: false or calling $.scrollify.disable(); with a page width condition, but the page instead of scrolling normally, stopped scrolling at all. Am I getting something wrong?
I am using jQuery v3.3.1.

That’s odd, I never seen that behaviour. Both the things you’ve tried should work fine.

Had the same issue.  I removed "scrollbars: true" from my options and it works.

I had the same issue. I removed "scrollbars: false" and it now works.

I'm also experiencing the same issue. Have y'all just settled on having the scrollbar show in order to work around this @dynainteractive  @meta4creations ?

@mmoga-uf Sorry, I don't recall what I did or what project I used this on.

Hey guys, it's been a while since you started this topic but I think you got the answer I'm looking for !
Like @simonszalai I'm working on a Webflow project and tried to implement Scrollify.
It went well on desktop but I need to disable it on mobile.
I'm not a code master, I'm a designer, and I've been scratching my head for hours now..
This is the code I used :

I've tried to add lines with  $.scrollify.disable(); mentioning screen width and things like
function disableScrollify(toggle){
if(toggle){
$.scrollify.disable();
} else {
$.scrollify.enable();
}
}
Like I said I'm pretty shitty at coding so I can't figure it out
Hope you can help me on this !
thank you

@Marin-polish I have applied this logic but scrolling is not working at all in chrome and firefox

@simonszalai can you please share the solution if you found one.


GemRank daily rank test failing

:clock11: When did the problem start?
Consistently failing for two days, intermittent failures for a few days before

:camera: Live badge

https://img.shields.io/gem/rd/facter.svg

:wrench: Is the live badge working?
Yes seems to be

:link: CircleCI link
https://circleci.com/gh/badges/daily-tests/424

:beetle: Stack trace
GemRank daily rank (valid) [ GET /rd/rails.json ] - [ GET /rd/rails.json ]

ValidationError: message mismatch: "value" with value "invalid rank" fails to match the required pattern: /^[1-9][0-9]*(ᵗʰ|ˢᵗ|ⁿᵈ|ʳᵈ) daily$/

:bulb: Possible solution
Not sure yet but will take a look at it.

Our GemRank badge services uses BestGems.org and according to their docs

Results of trends API are not perfect. Often, results have chasm as indicated by the following example. You don't expect that perfect data are returned.

I see this mirrored in the recent API response where there is no record for some days, and has a rank of null in others:
 {
  "date": "2019-06-25",
  "daily_ranking": null
},

Hopefully this will get resolved in the upstream BestGems API. If not, then perhaps we could consider tweaking the logic here a bit to check an additional record or two (i.e. the value from the day before) to try to find a data point.

This appears to have stabilized so going to close this for now. Can revisit if the issue resurfaces


Update index.html / added vis 5 data

added vis 5 data

Some components are missed, can you clean the previous PRs then collect components for making a single PR?


Nondeterministic output on test crates

The output on test crates (make test) seems to be nondeterministically adding or removing a few lines, particularly on dependency-ex, see the bot updates from CI:
bot update 1
bot update 2
The only thing that changed between the two was cargo fmt so I don't think the actual output should change. It may not be possible to replicate exactly bc the bot is running on Ubuntu (elk is Arch Linux), but we should look into what is happening here.

Another case, it looks like it goes beyond just this particular line
bot update 3