validation hints on interact fields do not allow to enter a 0 in nat type fields

Just recently introduced in https://github.com/baking-bad/bcd/commit/32e78c3638cbfef27f39ffa52e5db88263d2662b# src/components/schema/schemaForm/SchemaForm.vue line 177, that validation does not allow any more to enter a 0 in a nat type field.
But in some deployed smart contracts that would be needed, eg here (a youves contract):

where the dst_token_id is 0 (the token's ID is 0, the first one). But it can't be entered anymore.
I see plenty of other contracts using nat type fields for IDs, which can also be 0 in some cases (eg tokenIDs in FA2 tokens).
Best regards!

Thank you for the issue. I think you can use the interaction with this hint. It's just a hint. But definitely it's a bug and we'll try to fix it as soon as possible.

we fixed it

Hey @aopoltorzhicky, that was uper fast! Thank you. You are right for the hint. But in popup modals it was not possible to save the value, eg here:

Now it's fine. Thanks a lot!!


Removed access internal

Hi,
This change allows developers to reference to the "Word Templates Related Table" table object within their own code.
Without the internal access we are able to transfer word templates with all the related data we need between multiple environments and up to 300 companies per environment.
Please also note the Yammer conversation between Lorenz Roth and @JesperSchulz : https://www.yammer.com/dynamicsnavdev/#/Threads/show?threadId=1520243730882560

@PThiemann-AINO I read this thread on yammer, but I don't get what the agreement was.
I thought that @JesperSchulz agreed to add the copy functionality to the system application, but maybe that's a misunderstanding on my side.
I'm not quite sure if Jesper simple wanted you to remove the Access property from this table.

@pri-kise The copy function used to be part of our app before moving it from C/AL to AL. First a lot was missing but since then you brought back a lot of functionality we were depending on. One last thing missing on our side is the copy function and we can re-integrate it as soon the access property has been removed.
We might provide you with the functionality in future but currently this is still a decision to be made by us.

@pri-kise The copy function used to be part of our app before moving it from C/AL to AL. First a lot was missing but since then you brought back a lot of functionality we were depending on. One last thing missing on our side is the copy function and we can re-integrate it as soon the access property has been removed.
We might provide you with the functionality in future but currently this is still a decision to be made by us.

I'm not the one to decide if this access property is removed, since I'm not working for microsoft.
I only can tell, that in the past the Access properties weren't always removed, when someone created a PR here.
Therefore I tried to understand this yammer thread and guide you into the right direction.

Sorry that it took me so long to get involved in the discussion of this pull request. Better late than never I guess, so here we go:
Unfortunately there seems to have been a small misunderstanding in the Viva Engage conversation. While I would applaud a pull request here on GitHub to expand on the API you need to fulfill your business needs, what is suggested in this pull request is not what I meant. The "Word Template Related Table" is an implementation detail and should as such not get exposed. Opening up the table for extensions to tamper with, could lead to unwanted side-effects. It would also rob us of the opportunity to change and improve the implementation in the way we see fit going forward, which is one of the main goals with our modules. Rather than requesting access to the internal implementation details, you should look at the module's facade and expand it with public procedure(s) you require.
As this repository is getting retired for System Application pull requests and the provided pull request in its current shape cannot be accepted, I am taking the liberty of closing this PR. If you decide to expand the Word Template's API, please create an issue on the new BCApps  repository, reference this PR, and let's discuss the issue and possible solutions, before a new PR is created.
More details on the process for making contributions can be found here: https://github.com/microsoft/BCApps/blob/main/CONTRIBUTING.md
My apologies again for not being able to advise on this matter earlier. I hope we can find a solution which satisfies all parties on our new BCApps GitHub repository 😊


vit sizes

#18 Fixed issues, Main.ipynb should work.
Also added ability to change decoder size.
Fixed naming convention in ViT, since there was a bug in the projections and cls/mask tokens that I suspect was a result of naming: you were using mlp_dim rather than embed_dim, which did not give any erros since you had mlp_dim = embed_dim.

Fantastic! Looks great
Yes that mlp_dim / embed_dim problem was confusing me, thank you for fixing it!
Can confirm all works when I pulled it on my end


TabInputButton accessibilityId introduced

Current coverage is 73.31% (diff: 100%)

Merging #206 into master will increase coverage by 9.70%

@@             master       #206   diff @@
==========================================
  Files            63         18     -45
  Lines          3512        937   -2575
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2234        687   -1547
+ Misses         1278        250   -1028
  Partials          0          0

Powered by Codecov. Last update 22a3833...0a55bc0


Running npm start in server

while running npm start commad inside hospital repo directory haveing error says no module found "yarn install".can anyone help me?

Could you provide more info? I'm trying to install and run for the first time too and am not running into this one.
What you're describing sounds like Error: Cannot find module 'yarn install'. But I'm skeptical of that being the problem. Could you copy and paste the error verbatim here?

@akash65 This sounds like you're to start development on the frontend. You need to use https://github.com/HospitalRun/hospitalrun-frontend


Trino connector, support optional database field

Affected module
Ingestion Framework, Trino Connector
Describe the bug
Trino connector has an optional field 'database' described as follows: 'If you want to limit metadata ingestion to a single database, include the source.config.database field in your configuration file. If this field is not included, the connector will ingest metadata from all databases that the specified user is authorized to read'
When running ingestion ( openmetadata-ingestion, version 0.8.17dev1) with 'database' field empty or set to None, an error occurs that this field should not be None.
The reason seems to be that database field is not in fact optional:   https://github.com/open-metadata/OpenMetadata/blob/29e42051ecd4763d2f6b456ecb79377c54b6fd37/ingestion/src/metadata/ingestion/source/trino.py#L47
Having set that field to Optional[str] however, does not make the ingestion to work properly. It does not ingest any database/table but the job succeeds.
To Reproduce
Run a trino ingestion with database field set to None
Expected behavior
If no database is set, ingestion framework should iterate over all databases in the set catalog and run ingestion for each
Version:

Python version:3.9
OpenMetadata version: 0.8.4
OpenMetadata Ingestion package version:  openmetadata-ingestion[trino]==0.8.17dev1

Thanks for filing this @dvecerdea . We will fix it right away


Feature Request: Attach secrets to Circuit -> Providers

It would be nice to attach login credentials for web portals / access for providers

The advantage of storing secret data in NetBox is being able to retrieve and decrypt many secrets at once to, for example, generate a FreeRADIUS configuration file. There are plenty of better tools (KeePass, LastPass, etc.) available for general credentials storage.


React native 0.72.0

After upgrading react native to latest version android build stat to fail,
package.json
    "@amplitude/analytics-react-native": "^1.3.0",
    "react-native": "^0.72.0",

BUILD FAILED in 10s
error Failed to install the app. Command failed with exit code 1: ./gradlew app:installAlphaDebug -PreactNativeDevServerPort=8081 FAILURE: Build failed with an exception. * What went wrong:
Execution failed for task ':amplitude_analytics-react-native:compileDebugKotlin'.
> 'compileDebugJavaWithJavac' task (current target is 1.8) and 'compileDebugKotlin' task (current target is 11) jvm target compatibility should be set to the same Java version. Consider using JVM toolchain: https://kotl.in/gradle/jvm/toolchain * Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights. * Get more help at https://help.gradle.org BUILD FAILED in 10s.
info Run CLI with --verbose flag for more details.
error Command failed with exit code 1.

Hi @abgaryanharutyun, you are using @amplitude/analytics-react-native in the repo. This repo is for @amplitude/react-native.
I cannot reproduce your error. Could you run --stacktrace, --info, --scan and --verbose options to get more info? Thanks.

@Mercy811 , I will provide additional logs to reproduce the issue easily. However, it is easy to reproduce because it doesn't work with the new React Native version 0.72. Have you tested it with that version? Does it work properly on Android?

Hi @abgaryanharutyun, I got a similar but slightly different one by:

npx react-native init AwesomeProject_72 --version 0.72.0
yarn android

with environments:

"@amplitude/analytics-react-native": "^1.3.1",
"@react-native-async-storage/async-storage": "^1.19.0",
"react": "18.2.0",
"react-native": "0.72.0"
gradle plugin version 7.4.0 (you can find these two in Android Studio, File > Project Structure > Project)
gradle version 7.5

BUILD FAILED in 4s
error Failed to install the app. Command failed with exit code 1: ./gradlew tasks FAILURE: Build failed with an exception. * What went wrong:
Execution failed for task ':tasks'.
> Could not create task ':amplitude_analytics-react-native:compileDebugAndroidTestKotlin'. > Cannot use @TaskAction annotation on method AbstractKotlinCompile.execute() because interface org.gradle.api.tasks.incremental.IncrementalTaskInputs is not a valid parameter to an action method. * Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights. * Get more help at https://help.gradle.org BUILD FAILED in 4s
> Task :gradle-plugin:pluginDescriptors
> Task :gradle-plugin:processResources
> Task :gradle-plugin:compileKotlin
> Task :gradle-plugin:compileJava NO-SOURCE
> Task :gradle-plugin:classes
> Task :gradle-plugin:jar
> Task :gradle-plugin:inspectClassesForKotlinIC
> Task :tasks FAILED Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0. You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins. See https://docs.gradle.org/8.0.1/userguide/command_line_interface.html#sec:command_line_warnings
6 actionable tasks: 6 executed.
error Command failed with exit code 1.

Before proceeding, would like to confirm with you we are having the same error and the same environment.

Hey @Mercy811 ,
I wanted to let you know that I'm experiencing the same issue. It seems to be related to the latest React Native version 0.72. From what I've observed, the problem lies in incompatibility with the new RN version. Surprisingly, iOS is functioning properly, but Android is not.
I would appreciate it if we could address this problem as soon as possible. Can we work together to find a solution or fix?
Looking forward to your response.

Hi @abgaryanharutyun, thanks for your confirmation. It's on the top of my list and will keep you updated. Also really appreciate it if you could let me know if you have any findings.

Hey @Mercy811 I think this issue not related with amplitude package, I have fixed this issue on my end without amplitude change, so what I did just changed
        compileSdkVersion = 34
        targetSdkVersion = 34

    from

compileSdkVersion = 33
targetSdkVersion = 33

Solution
I didn't fixed it by changing compileSdkVersion and targetSdkVersion. But fixed the problem by the adding kotlinVersion to the latest in build.gradle
ext {
        ...
        kotlinVersion = "1.9.0"
        ...
}

Trace down
What I described here amplitude_analytics-react-native:compileDebugAndroidTestKotlin is not the same error as amplitude_analytics-react-native:compileDebugKotlin in the original post.
Step to reproduce

Update to the latest Android Studio (from Android Studio Electric Eel to Android Studio Flamingo
Change gradle plugin version to 7.4.0 and gradle version to 7.5
Run yarn android and get amplitude_analytics-react-native:compileDebugKotlin
There is an important line in the error message

<EMAIL_ADDRESS>(26, 13): Class 'kotlin.Unit' was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.7.1, expected version is 1.5.1.

It indicates that this error is caused by the incompatibility of Kotlin. The default Kotlin version compiled to is 1.5.1 however the needed version (probably required by React Native 0.72.0 is at least 1.7.1). So specify the latest Kotlin version in your React Native project should fix the problem.


[Mixer Deposit] mixer deposit should return a leaf to be compatible with TXButton

For now, the button that will trigger deposit action expects params are [MixerGroupId, Leaf].
however, the mixer .deposite will return Promise
A workaround to get the leaf:
    return new Promise<any[]>((resolve, reject) => {
      mixer
        .deposit(note, (leaf) => {
          depositLogger.trace('generated note ', note.serialize());
          depositLogger.info(`Getting params GroupID `, 0);
          depositLogger.info(`Full params are groupId, noteCommitment]`, [0, [leaf]]);
          resolve([0, [leaf]]);
          return Promise.resolve(0);
        })
        .catch((e) => {
          depositLogger.error(e);
          reject(e);
        });
    });

we fixed this too!


Using the same texture on TileMap and Sprite3D causes 'Cannot blit_rect in compressed or custom image formats.' and 'Can't get_pixel() on compressed image, sorry.'

Godot version
v4.1.2.stable.mono.official [399c9dc39]
System information
Godot v4.1.2.stable.mono - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>58) - Intel(R) Core(TM) i9-10900K CPU @ 3.70GHz (20 Threads)
Issue description
Using the same texture on TileMap and Sprite3D causes tons of
Cannot blit_rect in compressed or custom image formats.
Can't get_pixel() on compressed image, sorry.

and does not render correctly.
Steps to reproduce

create project (default settings)
import texture
create 3D scene
add SubViewport to root
add TileMap to SubViewport

new TileSet
add texture (x.png) as atlas
draw something on TileMap

add Sprite3D to root

set Texture to ViewportTexture (from previous SubViewport)

add Sprite3D to root

set Texture to x.png

Then, get tons of errors.
Minimal reproduction project
MRP.zip

This is expected behavior.
In the Import settings of the texture there is Detect 3D section. Change "Compress To" to "Disabled".
The textures get compressed by default when used in 3D context.

This is expected behavior. In the Import settings of the texture there is Detect 3D section. Change "Compress To" to "Disabled". The textures get compressed by default when used in 3D context.

Thank you!
So TileSet atlas does not support compressed image?
After scrolling up thousands of error messages, I noticed a gray line info telling me that the way of importing image had been changed, which I hadn't noticed before.

So TileSet atlas does not support compressed image?

It doesn't. Maybe we can detect that and display a single readable error instead of spamming unrelated messages.


[bug] Avatars sometimes do not load and randomly shuffle when changing tabs on "Who to follow"

First off, fantastic project, big fan!
On the "Who to follow" section of my profile the list of people shuffles/refreshes when changing focus (might be intentional) and sometimes the avatars do not load correctly (502 bad gateway)
On Firefox 96.0, MacOs 10.15, 2020 M1 Macbook Air
video
(w/ React devtools on if that helps, was too lazy to turn it off. cw: rapidly changing images)
https://user-images.githubusercontent.com/21983833/152965415-24d0f893-b972-4b8e-88b7-160bd49f382a.mov
Error
GEThttps://eu.ui-avatars.com/api/?rounded=true&background=574cfa&color=ffffff&name=dirkriehle
[HTTP/2 502 Bad Gateway 196ms]

GET
	https://eu.ui-avatars.com/api/?rounded=true&background=574cfa&color=ffffff&name=dirkriehle
Status
502
Bad Gateway
VersionHTTP/2
Transferred5,89 KB (5,38 KB size)
Referrer Policystrict-origin-when-cross-origin

    HTTP/2 502 Bad Gateway

    date: Tue, 08 Feb 2022 10:02:26 GMT

    content-type: text/html; charset=UTF-8

    set-cookie: cf_ob_info=502:6da405bc6a051764:EWR; path=/; expires=Tue, 08-Feb-22 10:02:56 GMT

    set-cookie: cf_use_ob=443; path=/; expires=Tue, 08-Feb-22 10:02:56 GMT

    x-frame-options: SAMEORIGIN

    referrer-policy: same-origin

    cache-control: private, max-age=0, no-store, no-cache, must-revalidate, post-check=0, pre-check=0

    expires: Thu, 01 Jan 1970 00:00:01 GMT

    cf-ray: 6da405bc6a051764-EWR

    server: cloudflare

    X-Firefox-Spdy: h2

    Accept
    	image/avif,image/webp,*/*
    Accept-Encoding
    	gzip, deflate, br
    Accept-Language
    	en-US,en;q=0.5
    Connection
    	keep-alive
    DNT
    	1
    Host
    	eu.ui-avatars.com
    Referer
    	https://www.researchequals.com/
    Sec-Fetch-Dest
    	image
    Sec-Fetch-Mode
    	no-cors
    Sec-Fetch-Site
    	cross-site
    TE
    	trailers
    User-Agent
    	Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:96.0) Gecko/20100101 Firefox/96.0

Welcome @ThomasFKJorna and thanks for reporting this. There is nothing lazy in you reporting - I appreciate any time you spend on doing this 😊
The issue of the avatar loads is popping up a bit more frequently than I had expected - it seems like https://eu.ui-avatars.com/ can't handle it as well. Do you feel it is too annoying or more a minor annoyance? (or something else)
With respect to the shuffling of the who to follow, it is intended at this point, but not set in stone. We do an auto refresh on window focus to ensure the feed is up to date, and then it also shuffles the suggested follows. With that information, would you prefer it to be different?

@all-contributors add @ThomasFKJorna for bug

It appears ui-avatars has improved their services to such a degree this is no longer happening. I couldn't reproduce this with a gazillion reloads. 😊


astyle lint is breaking mlkem/poly.c on branch only

Application of "astyle" to mlkem/poly.c is failing BUT on the poly_compress_contracts branch only.
On the "main" branch, it works fine, and doesn't change the file at all.
On the "poly_compress_contracts" branch, it destroys the formatting of the file from the poly_decompress() function onwards.]
e.g. in the NIX environment on either MacOS or Linux:
git checkout main
cd mlkem
astyle --options=../.astylerc --formatted poly.c
git st .
# reports no change...
git checkout poly_compress_contracts
astyle --options=../.astylerc --formatted poly.c
Formatted  /home/ubuntu/pqcp/refb/mlkem/poly.c
git st .

shows poly.c has been modified. git diff . shows the file has been reformatted incorrectly.
I am unable to work out what's going wrong here.

This was ultimately due to bad indentation in #91.


Failure to pose at the same XYZ position multiple times

Description
Attempting to sit in the same block multiple times- as in sitting, shifting to leave, then sitting again -will lead to the incorrect error of "Someone is already sitting here!" when nobody is sitting there. Presumably, this is caused by the Shift event not actually removing the player from that pose in memory.
Solution
The simplest solution to fixing the bug is removing the player's pose on that XYZ block from how you store it in memory. Though, what I believe to be the correct solution to this issue is to have a dictionary/hashmap of the player's GUID and the XYZ position they're in. This way, the player can only be in one pose at a time preventing "ghost poses" of what we're experiencing now.
Notes
Possibly caused by #8? Depending on the complexity of your mod, I might submit an untested PR.

https://user-images.githubusercontent.com/46751150/184520642-0cbece65-ebfd-4dfb-a425-016ce50390f3.mp4
Here's a repro. It seems like this is only caused through right-clicking though, not through the /sit command.

Should be resolved as of 4b86f4355ec390cc3f01a3eaac48180bd7b57891
I'll release version 1.5.3 in a sec, let me know how it goes :)

Works as intended! Thank you for the quick fix.


Failing Test in the Project

Hi, thanks for making this repo, I'm trying to use Jest and WDIO myself, its really helpful to have examples. However when I install and run the tests in the repo I get some error:

I've done some digging around but I haven't found a solution, I was hoping you might have an idea?
Thanks again!

Sorry, I'm away from laptop until next week. I wonder if maybe you don't have the right version of Webdriverio? Try a fresh npm install?

Thats alright, thanks for responding. The version I have installed is the same as what's in the package.json as<EMAIL_ADDRESS>I think it may have to do with the selenium-standalone dependency and compatibility issues with java 10. I was able to get it running when I updated the version to 6.15.6.


clojure-lsp: 2021.02.07-22.51.26 -> 2021.02.10-21.13.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/clojure-lsp/clojure-lsp/releases.
meta.description for clojure-lsp is: "Language Server Protocol (LSP) for Clojure"
meta.homepage for clojure-lsp is: "https://github.com/snoe/clojure-lsp"
meta.changelog for clojure-lsp is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2021.02.10-21.13.12 with grep in /nix/store/r58bvacy6ypkmfgf6d4if714xfac7b74-clojure-lsp-2021.02.10-21.13.12

directory tree listing: https://gist.github.com/a29362c084ac32be733625d424475943

du listing: https://gist.github.com/82be502c1b2785c0615f0562d04d2bae

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
clojure-lsp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r58bvacy6ypkmfgf6d4if714xfac7b74-clojure-lsp-2021.02.10-21.13.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A clojure-lsp https://github.com/r-ryantm/nixpkgs/archive/e524473fd9b766059092cbfd456a04e41a2779e7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r58bvacy6ypkmfgf6d4if714xfac7b74-clojure-lsp-2021.02.10-21.13.12
ls -la /nix/store/r58bvacy6ypkmfgf6d4if714xfac7b74-clojure-lsp-2021.02.10-21.13.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

clojure-lsp

Maintainer pings
cc @ericdallo for testing.

Result of nixpkgs-review pr 112778 at e524473f run on aarch64-linux 1

1 package built:

clojure-lsp

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/misc/clojure-lsp/default.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


[SPARK-29105][core] Keep driver log file size up to date in HDFS.

HDFS doesn't update the file size reported by the NM if you just keep
writing to the file; this makes the SHS believe the file is inactive,
and so it may delete it after the configured max age for log files.
This change uses hsync to keep the log file as up to date as possible
when using HDFS.
Tested with a SHS configured to aggressively clean up logs; verified
a spark-shell session kept updating the log, which was not deleted by
the SHS.

Test build #110796 has started for PR 25819 at commit 3819abd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15930/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110796/
Test FAILed.

Thank you for making this fix, @vanzin !

test this please

Test build #110800 has started for PR 25819 at commit 3819abd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15933/
Test PASSed.

Test build #110800 has finished for PR 25819 at commit 3819abd.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110800/
Test FAILed.

lgtm

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15956/
Test PASSed.

Test build #110824 has started for PR 25819 at commit 3819abd.

Test build #110824 has finished for PR 25819 at commit 3819abd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110824/
Test FAILed.

Test build #110834 has started for PR 25819 at commit dcd837c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15967/
Test PASSed.

Test build #110840 has started for PR 25819 at commit acd6820.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15974/
Test PASSed.

Test build #110834 has finished for PR 25819 at commit dcd837c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110834/
Test PASSed.

Test build #110840 has finished for PR 25819 at commit acd6820.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110840/
Test FAILed.

The last test failure was SPARK-29140

retest this, please

Test build #110881 has started for PR 25819 at commit acd6820.

Test build #110881 has finished for PR 25819 at commit acd6820.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110881/
Test PASSed.

Merging to master.


1.16.4 Crashing when I'm opening binary files

Problem on Windows 10.
We had the same problem on version 1.16.2

Found the problem. It was a very subtle thing and unfortunately I forgot to test for binary files! My bad, I need to create another release now.

Ok I did a new release just for this problem!


Add ConditionalModule

This can be used to conditionally bind a module based on a config
property or properties

Updated

Looks good, update CHANGES


Connection with Spyder

Hi,
Just wondering if there is a way to plug the Jupyter Scala in Ipython 3.0 inside Spyder.
Especially, using Connect to an existing Kernel  (Connection file with JSON or remote kernel)

I don't really know Spyder. It seems quite taylored for Python though, I'm not sure kernels for other languages would be usable from it. (Are they? What about e.g. Julia?)

Hello,
Spyder is an IDE integrating Ipython 3.0 inside (excellent quality, like matlab style).
The python code is evaluating through Ipython.
There is a communication process from
Ipython to others kernel.
Wondering if we launch Scala Jupyter in standalone mode, we can communicate with the kernel of Ipython.
My question:

How IPython is launched and communicated with the JVM ?

is there a background server ?

Thanks
Kevin
On 05 Apr 2016, at 16:57, Alexandre Archambault<EMAIL_ADDRESS>wrote:
I don't really know Spyder. It seems quite taylored for Python though, I'm not sure kernels for other languages would be usable from it. (Are they? What about e.g. Julia?)
―
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Closing, as the upcoming version is around the corner (currently on the develop branch).


pymbar 3.1 release not available through PyPI

I've just noticed that the pymbar v3.1 (LTS) release is not available via PyPI, see https://pypi.org/project/pymbar/#history
Is there a particular reason for this, or is it just an oversight?
For me personally it's not a big problem, installed pymbar from the source tarball obtained for the 3.1 tag in GitHub (https://github.com/choderalab/pymbar/releases/tag/3.1) seems to work just fine...

Thanks for calling attention to this. It was an oversight which i've resolved. Release is the version tagged at 3.1.0


Change dashboard layout

What Happens:

As a user, when landing on the site, I'm directed to a verbose page that requires a second click to get anywhere, and the places it links are already in the nav.

What Should Happen:

Put useful information on this page, e.g. submissions and grants associated with me, stats about submissions

@htpvu We have added some information, but it seems that this might be best addressed through separate tickets to add specific functionality. Proposing we close this.


Add support for multi-language files

Is your feature request related to a problem? Please describe.
Yes.
I'm trying to edit a PHP file which contains HTML embedded which contains some javascript embedded. Twinny's recommendations appear to assume I want PHP and even that the code is PHP when I'm editing the javascript portion.
Describe the solution you'd like
There are 2 separate and different difficulty level possibilities for this.

Identify that it's a multi-language file type and allow the user to clarify (in the chat box, could be the first word, for example) But it can also be a dropbox that is easily accessible.
Identify the language where the cursor is at which is used when prompting the bot
2.1. Doing this half arsed might do so-so reasonable results.... Maybe
2.2. Doing this for real (but doing that well is a PITA and I don't recommend)

Describe alternatives you've considered
I'm unable to find viable alternatives.

Hey thanks for the suggestion this is a good idea.
To be honest I am not sure how these small models handle FIM for multiple languages in the same prompt, probably not well is my assumption.
It might be easier in the chat as we can have a drop down as you mention.  In terms of detecting the current language from the editor, I am not sure what the best approach would be currently it might require more advanced techniques to establish it.

To be honest I am not sure how these small models handle FIM for multiple languages in the same prompt, probably not well is my assumption.

They aren't good with multiple languages in the same place but usually it's not required to have a lot of content outside the small area to get suggestions so usually it can adapt to the situation because (mostly) all it will see is that singular embedded language's code (E.g. javascript in an HTML document) or it's so small the help isn't useful.

It might be easier in the chat as we can have a drop down as you mention.

Yep. That's what I recommend too. Maybe with some kinks to make it fast to swap out, for example, fast tabbing or a shortcut only available in that window to open the dropdown.
For now, I'd just have a hardcoded list of languages for each defined language and accept PRs to make them more complete. For example, for PHP:

PHP
HTML
Javascript
CSS

The order is designed to make the most likely language to encounter in each language mode. In PHP, PHP should be the most common. Followed by HTML (because JS and CSS, most likely will be in external files). Then I went with js just because non-AI has gotten really good with CSS, so js is most likely to be asked about.
For other file types, for example, HTML, would be the 2,3,4 of the previous list.
A very different example: jinja2. Would probably get the same set as PHP except the 1st one is jinja2 or just straight up move the jinja2 to 2nd and select HTML 1st instead (Maybe leave the jinja2 help to the non-AI... Not sure. Dunno how good the AI is with jinja2).
There's a lot that would be correct so I'd start with some languages that are "multi-language" and hardcode possibilities for the drop down. After ppl start using this more, maybe someone will have good ideas how to proceed to viably please people.
For automatic detection... I'd probably start with half-arsed regex or similar to figure out some identifiers but I wouldn't try yet. It's full of issues and annoying (to the user) edge-cases that are not worthwhile yet.

For implementation, I'd just have a very simple translation table (can be a js object) from a name and an array of options for the drop box. If not defined in the translation table, just use the language provided.
For now, I'd be happy with:

PHP
HTML
jinja-py

There might be more and ppl with other embeds but this should work fine for now.
What do you think?

I'm unable to find viable alternatives.
You could make your own buttons for each language being hardcoded. Explain JS, Explain HTML and enable/disable the ones you do not want at that specific moment so it's not clutering the buttons too much. And you use these dedicated buttons to request explanation from whitin that given language instead of having one generic button like now:
{{{systemMessage}}}
Explain the following code;
{{{code}}}
Do not waffle on. The language is:
{{language}}

stale


Failed to pass SSL auth while curl.CURLOPT_CERTINFO is set

> php -v
PHP 5.4.17 (cli) (built: Aug 25 2013 02:03:38)
> curl -V
curl 7.30.0 (x86_64-apple-darwin13.0) libcurl/7.30.0 SecureTransport zlib/1.2.5

No matter CURLOPT_CERTINFO set to true or false, following request can't pass the SSL authentication.
<?php
require_once 'vendor/autoload.php';

use Guzzle\Http\Client;

$client = new Client('http://ata.***.com', array(

   'curl.options' => array(
       CURLOPT_CERTINFO => false[or true],
       CURLOPT_SSL_VERIFYPEER => false,
       CURLOPT_COOKIEFILE => './g.cookie',
       CURLOPT_COOKIEJAR => './g.cookie'
   )
));
$request = $client->get('/');
$response = $request->send();
echo $response->getBody();

But if remove CURLOPT_CERTINFO like below, it works
<?php
require_once 'vendor/autoload.php';

use Guzzle\Http\Client;

$client = new Client('http://ata.***.com', array(

   'curl.options' => array(
       CURLOPT_SSL_VERIFYPEER => false,
       CURLOPT_COOKIEFILE => './g.cookie',
       CURLOPT_COOKIEJAR => './g.cookie'
   )
));
$request = $client->get('/');
$response = $request->send();
echo $response->getBody();

Since PhpBrowser in codeception has a default config of
protected $curl_defaults = array(
    CURLOPT_SSL_VERIFYPEER => false,
    CURLOPT_CERTINFO => false,
);

We can't just remove that configuration

Just as a follow up to this problem:
On OSX 10.9 (Mavericks) Apple abandoned OpenSSL affecting cURL behavior with self-signed certs. You have some options cleverly pointed out in the link given.
PS: OSX 10.10 is also affected.

PHP curl_setopt and curl_setopt_array always return false when trying to set CURLOPT_CERTINFO.  This will cause curl_setopt_array to ignore any later parameters.
PHP 5.4.16, curl 7.29.0.


Add support for Jamba models

What model would you like?
Please incorporate the new Jamba model, which you can find at https://huggingface.co/ai21labs/Jamba-v0.1. It's likely necessary to first wait until llama cpp is compatible with this model architecture.

Closing as duplicate.


An option to create a non-reactive state?

What problem is this solving
At the moment I am trying to store an object instance (a Mapbox map instance created using "new" syntax) inside a store's state so that I can share this map instance in various components. However, it doesn't appear possible to use methods associated with this map instance because they seem to be hidden by the proxy object. I am assuming this is intended and a side-effect of how the new reactivity system works (though this used to work in Vue 2 / Vuex)
Proposed solution
Assuming there is no workaround to access the nested method (e.g. myStore.mapInstance.getZoom()) then it would be nice to have a way to turn-off reactivity for a specific item in the state, or else to have a separate non-reactive state where it is possible to store and share complex objects which otherwise wouldn't work.
Describe alternatives you've considered
Vuex 4.

Isn't markRaw from Vue what you're looking for?

I wasn't aware of markRaw and it is exactly what I needed. Everything is working perfectly now.
Thanks so much 👍


Problem infer return type of function with nullable argument

TypeScript Version:  3.2.0

Code
interface U { }
function test(Us?: U[] | null | undefined) {
    if (!Us)
        return;
    else return []
}

Expected behavior:
Return type should be void | any[]
Actual behavior:
any[]
Playground Link:
Playground

void means "I don't care the return type", which seems not the case here. undefined | any[] might make more sense.
And since strictNullChecks is not enabled, any[] is the same as undefined | any[].


Plugins are ignored (?)

Hi. Here's my config:
httplug:
    clients:
        test:
            http_methods_client: true
            plugins:
                - add_host:
                    host: 'test'

When i sendRequest (via Http\Client\Curl\Client by default) it (the request) is never modified by the plugin (i've also tried header_defaults plugin, to no avail). They're never called.
The error i'm getting is from Curl Client - url malformed. Which makes sense as there is no host.
If i manually create a PluginClient with AddHostPlugin and pass it CurlClient, then it works fine.
So it seems that the configuration does not work. Am i missing something or did i misconfigure it ? i did read the documentation and it seems like it should just work in this case.
version: "php-http/httplug-pack": "^1.1". Hope this is correct place for this question.

Hmm, nope it's probably not the right place to ask this question :D


MJ4 mathjax-tex monospace double-quote much wider than MJ2's

Issue Summary
MJ4 typesets a double-quote from MML monospace as much wider than MJ2. Ex:
<math display="block" overflow="scroll"><mrow><mo mathvariant="monospace">"</mo></mrow></math>

Steps to Reproduce:

Go to https://codepen.io/alexedgcomb/pen/YzoNdpO
Observed: The double-quote is quite wide

Expected: More like MJ2's:

Technical details:

MathJax Version: 4.0.0-beta.7
Client OS: Mac OS
Browser: Firefox 128.0.3

I am using the following MathJax configuration:
window.MathJax = { output: { font: 'mathjax-tex' } };

and loading MathJax via
<script<EMAIL_ADDRESS>
Supporting information:
Also reproduces in Chrome 127.0.6533.73

Here's the situation for this.  In MathML, the quotation mark is considered to be a "pseudo-script" as defined in the MathML specification section 7.2.2.  In order to handle both the recommended usage such as <msup><mi>f</mi><mo>"</mo> and the discouraged usage as <mi>f</mi><mo>"</mo>, MathJax needs to have the needed character in two different sizes: a smaller and raised version for the latter, and a larger, un-raised on for the former.  The mathjax-tex font does not have both for " (or for most of the pseudo-scripts), but it does have both sizes for things like the prime symbols (U+2032, U+2033, etc.), so quote-like pseudo-scripts are converted to those, when possible.  So the double-quote is converted to U+2033 when it appears in an <mo> (the quote as an operator is usually meant to indicate double-prime).  The monospace variant doesn't have U+2033, so MathJax tries to use two copies of U+2032 to fake it.  That is what is happening here.  Since both are fixed width, the are somewhat widely spaced.
This is not what happened in v2 because MathJax v2 didn't handle pseudo-scripts, whereas v4 does.
Of course, this substitution only occurs for <mo> elements, so if you want the actual " character from the monospace font, then you should use <mtext mathvariant="monospace">"</mtex> instead.
It is true that the use of using two U+2032 to replace U+2033 is not very good for a monospace font.  But even in v2, <mo mathvariant="monospace">″</mo> would produce the wide spacing.
The font tools do have a means of building new characters by combining others, and it would be possible to make a close-spaced version of U+2033 and the other multiple-prime characters for the monospace font.  The mathjax-tex font is supposed to be the same as the v2/v3 font, so I'm not sure if that should be done or not.
In any case, this is "expected behavior" in v4.

I forgot to add that you can disable the translation of " to U+2033 in <mo> elements using the following configuration:
MathJax = {
  startup: {
    ready() {
      const {MmlMo} = MathJax._.core.MmlTree.MmlNodes.mo;
      MmlMo.remapPrimes[0x0022] = 0x0022;  // don't remap double quote
      MathJax.startup.defaultReady();
    }
  }
};

@dpvc , thank you for your thoughtful explanation: Incredibly helpful insights into causes, MML standards, and options for going forward!


[AutoRelease] t2-elasticsan-2022-10-21-29939(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3196
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=5643

issue link:https://github.com/Azure/sdk-release-request/issues/3196

/azp run prepare-pipelines


Fix read the docs builds by updating rust compiler version

The read the docs builds are using conda to install rust and other
system dependencies required to install retworkx and build the docs.
However, the version of rust was pinned at 1.40 which is now too old
after #224 merged which bumped the minimum supported rust version from
1.39.0 to 1.45.0 because of a newer version of pyo3. This commit updates
the read the docs conda env to use the latest stable rust release 1.49.0
so that we can build retworkx for docs builds again.

Pull Request Test Coverage Report for Build 502279530

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.845%

Totals

Change from base Build 502277089:
0.0%

Covered Lines:
3091

Relevant Lines:
3259

💛  - Coveralls

Pull Request Test Coverage Report for Build 502279530

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.845%

Totals

Change from base Build 502277089:
0.0%

Covered Lines:
3091

Relevant Lines:
3259

💛  - Coveralls

This fixes the builds: https://readthedocs.org/projects/retworkx/builds/12822350/

This fixes the builds: https://readthedocs.org/projects/retworkx/builds/12822350/


Chatting 0-9 in loadout screen triggers recall loadout

The new loadout save/recall needs to be modified to not react to 0-9 key presses when the chat dialog is open.
Imported from trac ticket #118,  created by Xynth on 07-06-2010 at 18:00, last modified: 08-11-2010 at 20:45

r741
Trac comment by Imago on 07-17-2010 at 11:25


Feedback on API + integration

Continuing the discussion from reddit, I took another stab at integrating netman into oil.nvim. I'm keeping the work in a branch for now. I was able to get further along than I did before, and I have a working proof-of-concept that can do read-only browsing. A couple of notes:

The format for urls with an absolute path is scp://host///path/to/file.txt. For netrw, the equivalent would be scp://host//path/to/file.txt. That extra / threw me off for a while, because if it's missing I just get an obscure looking error. I personally prefer the netrw format (I don't see what the extra / is for), but if you stick with this format I'd recommend adding some earlier url validation.
I love the regular structure for the entries returned by providers. I know it's probably too early for this, but when 1.1 is finalized I'd like to see some documentation about which providers can provide which metadata fields, and for any enum fields I'd like to see a list of their possible values. What sparked that second one is I noticed that entries have TYPE = "regular file", whereas I'd expect that to be TYPE = "file" to match libuv. That made me curious what other values are possible in that slot.
I'd really love to see an async API. Currently it seems like the only API is synchronous and blocking, which makes the UI feel very clunky and unresponsive. It's possible that neovim 0.9 will ship with an async implementation (there's been a lot of chatter on https://github.com/neovim/neovim/issues/19624 recently), but there's always good old-fashioned callbacks.

This is probably as far as I'll go with the integration work until v1.1 is more stabilized. Nice work overall! Looking forward to seeing where this goes.

@stevearc Thank you for the feedback!

The format for urls with an absolute path is scp://host///path/to/file.txt. For netrw, the equivalent would be scp://host//path/to/file.txt. That extra / threw me off for a while, because if it's missing I just get an obscure looking error.

 For this event (a URI that is not able to find a match), you should an error that says netman cannot find a matching provider for the uri. I can word this in a better way if you would like, verbiage is quite important and if this error is not clear enough for a respective user of the framework, it needs to be updated to be more clear. That said, this error should crop up basically immediately when you attempt to do a api.read on an "invalid" URI. If you aren't seeing that, there may be a bug that I will have to further investigate.

I love the regular structure for the entries returned by providers. I know it's probably too early for this, but when 1.1 is finalized I'd like to see some documentation about which providers can provide which metadata fields, and for any enum fields I'd like to see a list of their possible values. What sparked that second one is I noticed that entries have TYPE = "regular file", whereas I'd expect that to be TYPE = "file" to match libuv. That made me curious what other values are possible in that slot.

So given that a provider must adhere to what Netman says it can return, the metadata that is returned is all standardized. It can be found here and you can see an example of how it can be read, in a standard way (here)[https://github.com/miversen33/netman.nvim/blob/ad4594b1c86d9cc113f7db9bb37400fdc6ba15ff/lua/netman/ui/neo-tree/init.lua#L157-L175]. I opted to go with the terminology LINK and DESTINATION since Netman isn't bound strictly to files or filesystems. Thus, a LINK can be treated as a directory and a DESTINATION can be treated as a file (for your purposes). I am iterating over this a bit to clean up how this will be returned by api.read as the explore content (which is poorly named) is quite clunky right now. That will be standardized by the merge of v1.1 into main.

I'd really love to see an async API. Currently it seems like the only API is synchronous and blocking, which makes the UI feel very clunky and unresponsive. It's possible that neovim 0.9 will ship with an async implementation (there's been a lot of chatter on Lua: async/await/await_all abstraction, structured concurrency neovim/neovim#19624 recently), but there's always good old-fashioned callbacks.

I am working through this right now actually! Though I made an active decision to not have read be asynchronous (as I am not quite sure how that should be handled by a consumer just yet). I can look into doing that for with v1.1 as I am working through trying to get searching (executing remote find basically) to be async as well.
I had no idea that async was going to be a main feature of Neovim, I am currently working through abusing libuv's spawn function for all of that right now.
Anyway! Thank you for the feedback, and once I get v1.1 merged into Main, that is when I would recommend taking a peek at this again (as I should have the kinks of consuming content from Netman worked out). I made a bit of a poor choice jumping to v1.0 when I did lol, but the decision is already made. If you would like, I can tag this issue when v1.1 is merged and I would be more than happy to help you integrate with Netman at that point (the less code I have to maintain, the better :smile: )

If you edit a file with two slashes as an absolute path (e.g. scp://root@host//root/test.txt), it produces this error:
Error detected while processing BufReadCmd Autocommands for "scp://*":
Error executing lua callback: ...nv/netman/data/nvim/lazy/netman.nvim/lua/netman/init.lua:51: attempt to index field 'data' (a nil value)
stack traceback:
        ...nv/netman/data/nvim/lazy/netman.nvim/lua/netman/init.lua:51: in function 'read'
        ...env/netman/data/nvim/lazy/netman.nvim/lua/netman/api.lua:247: in function <...env/netman/data/nvim/lazy/netman.nvim/lua/netman/api.lua:243>

I opted to go with the terminology LINK and DESTINATION since Netman isn't bound strictly to files or filesystems

That makes sense for the top-level API that you're trying to get all of your providers to conform to. It looks like there's some additional metadata for each of the entries, though (blksize, group, inode). Since you're already including that filesystem-specific information, I think it makes sense to also include a more detailed file type so we can distinguish a file from a symbolic link from a fifo, etc.
Feel free to ping this issue when v1.1 is released, but I'm following the releases so I should get notified anyway :)

Oh thats a true neovim error. I will look into that, you should get a netman error instead of the code crashing. Thanks for your help so far! I appreciate you :)

With v1.15 in testing now, I think I have hit a good point where this issue can be closed out. I wouldn't call this "complete" per say (as has been called out a few times, my documentation is still lacking), but I do think overall netman has moved in a far enough direction that the various feedback points here have been addressed.
@stevearc I may pull down oil and see if I can get netman integrated with it over christmas break. It has been on my list for a while and it seems like a decent start point for a new "thing" for netman. I don't promise anything usable but it is an interesting idea to poke at and spread netman :)


chore: add context to build functions

continuing the efforts started in #898
this is prep to add support for OpenTelemetry tracing.
the reason being, otel can (for the most part) work out of the box if you pass around contexts properly.

moving into draft until #898 is merged.
id rather take care of the conflicts than force them on sussman 😅


Table 335 - default value of the View Entry.

The description is slightly mangled:
"Default value: The first element in the Views array of the annotation specified in the RichMediaContent dictionary."
The Annotation does not have a Views array, the RichMediaContent does, but it does not specify an Annotation.
So, I believe this sentence was intended to say:
"Default value: The first element in the Views array of the RichMediaContent dictionary specified in the annotation."

Since this is a simple "point fix" do you want to duplicate it over into pdf-issues so this can be resolved?


[VS Code] Opening Terminal: Service not available: terminal

Error:
Opening Terminal: Service not available: terminal
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.905

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.38.1

platformName
MacOS

platformVersion
18.7.0

@susielau Thanks for reaching out. Can you give us more details about your scenario? Also, your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue

adding @IlyaBiryukov to follow up

Closing this due to inactivity. Please re-open this issue if you're still encountering this problem. Thanks!


Investigate new hangs on CI with RustConnection + simple_window_manager

First seen here: https://github.com/psychon/x11rb/pull/228#issuecomment-590106861
However, it actually occurred before on a merge to master (and I see this only now): https://travis-ci.org/psychon/x11rb/jobs/654088787?utm_medium=notification&utm_source=github_status
So, basically this started randomly failing right after RustConnection + simple_window_manager was added to CI. There were six successful builds before the first failure, so perhaps one of them is involved, but the commit messages sound unrelated.
For now my working theory is that this is not a new bug, but "was always broken".
Debugging idea: Push a branch that just runs simple_window_manager in a loop and let that fail on CI. Then, add lots of debugging println!s to figure out what is going on.

Also happened on AppVeyor: https://ci.appveyor.com/project/psychon/x11rb/builds/31053646
(This is the latest build for master)

Managed to reproduce it locally (but adding debuging-println!s makes the problem/race go away). GDB says:
Thread 2 (Thread 0x7ff333c14700 (LWP 34799)):
#0  __libc_recv (flags=<optimized out>, len=8192, buf=0x55d3ce082e40, fd=3) at ../sysdeps/unix/sysv/linux/recv.c:28
#1  __libc_recv (fd=3, buf=0x55d3ce082e40, len=8192, flags=0) at ../sysdeps/unix/sysv/linux/recv.c:23
#2  0x000055d3cd6e67fa in <std::sys::unix::ext::net::UnixStream as std::io::Read>::read ()
#3  0x000055d3cd6c24bb in <x11rb::rust_connection::stream::Stream as std::io::Read>::read (self=0x55d3ce085280, buf=...) at src/rust_connection/stream.rs:129
#4  0x000055d3cd6bde2b in <std::io::buffered::BufReader<R> as std::io::BufRead>::fill_buf (self=0x55d3ce085260) at /usr/src/rustc-1.40.0/src/libstd/io/buffered.rs:281
#5  0x000055d3cd6bd3be in <std::io::buffered::BufReader<R> as std::io::Read>::read (self=0x55d3ce085260, buf=...) at /usr/src/rustc-1.40.0/src/libstd/io/buffered.rs:245
#6  0x000055d3cd6bc181 in std::io::Read::read_exact (self=0x55d3ce085260, buf=...) at /usr/src/rustc-1.40.0/src/libstd/io/mod.rs:768
#7  0x000055d3cd682ff9 in x11rb::rust_connection::read_packet (read=0x55d3ce085260) at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:357
#8  0x000055d3cd6838df in x11rb::rust_connection::RustConnection<R,W>::read_packet_and_enqueue (self=0x55d3ce0851a0, inner=...) at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:158
#9  0x000055d3cd681dc2 in <x11rb::rust_connection::RustConnection<R,W> as x11rb::connection::RequestConnection>::wait_for_reply_or_error (self=0x55d3ce0851a0, sequence=10) at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:241
#10 0x000055d3cd644e8f in x11rb::cookie::Cookie<C,R>::raw_reply (self=...) at /home/psychon/projects/x11rb/src/cookie.rs:167
#11 0x000055d3cd643917 in x11rb::cookie::Cookie<C,R>::reply (self=...) at /home/psychon/projects/x11rb/src/cookie.rs:178
#12 0x000055d3cd667612 in x11rb::wrapper::LazyAtom<C>::atom (self=0x7ff333c13800) at /home/psychon/projects/x11rb/src/wrapper.rs:189
#13 0x000055d3cd685db4 in simple_window_manager::util::start_timeout_thread::{{closure}} () at examples/integration_test_util/util.rs:38
#14 0x000055d3cd6703d8 in std::sys_common::backtrace::__rust_begin_short_backtrace (f=...) at /usr/src/rustc-1.40.0/src/libstd/sys_common/backtrace.rs:129
#15 0x000055d3cd65e195 in std::thread::Builder::spawn_unchecked::{{closure}}::{{closure}} () at /usr/src/rustc-1.40.0/src/libstd/thread/mod.rs:469
#16 0x000055d3cd6771c8 in <std::panic::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once (self=..., _args=()) at /usr/src/rustc-1.40.0/src/libstd/panic.rs:317
#17 0x000055d3cd65c697 in std::panicking::try::do_call (data=0x7ff333c13bc0 "\220Q\b\316\323U") at /usr/src/rustc-1.40.0/src/libstd/panicking.rs:287
#18 0x000055d3cd6eb48a in __rust_maybe_catch_panic ()
#19 0x000055d3cd65c504 in std::panicking::try (f=...) at /usr/src/rustc-1.40.0/src/libstd/panicking.rs:265
#20 0x000055d3cd6771f8 in std::panic::catch_unwind (f=...) at /usr/src/rustc-1.40.0/src/libstd/panic.rs:396
#21 0x000055d3cd65dfa2 in std::thread::Builder::spawn_unchecked::{{closure}} () at /usr/src/rustc-1.40.0/src/libstd/thread/mod.rs:468
#22 0x000055d3cd6747b3 in core::ops::function::FnOnce::call_once{{vtable-shim}} () at /usr/src/rustc-1.40.0/src/libcore/ops/function.rs:227
#23 0x000055d3cd6ea3ff in <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once ()
#24 0x000055d3cd6de100 in std::sys_common::thread::start_thread ()
#25 0x000055d3cd6e9b16 in std::sys::unix::thread::Thread::new::thread_start ()
#26 0x00007ff333df9fb7 in start_thread (arg=<optimized out>) at pthread_create.c:486
#27 0x00007ff333d111af in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

Thread 1 (Thread 0x7ff333c16380 (LWP 34779)):
#0  futex_wait_cancelable (private=0, expected=0, futex_word=0x55d3ce084e7c) at ../sysdeps/unix/sysv/linux/futex-internal.h:88
#1  __pthread_cond_wait_common (abstime=0x0, mutex=0x55d3ce080d00, cond=0x55d3ce084e50) at pthread_cond_wait.c:502
#2  __pthread_cond_wait (cond=0x55d3ce084e50, mutex=0x55d3ce080d00) at pthread_cond_wait.c:655
#3  0x000055d3cd66a5d2 in std::sys::unix::condvar::Condvar::wait (self=0x55d3ce084e50, mutex=0x55d3ce080d00) at /usr/src/rustc-1.40.0/src/libstd/sys/unix/condvar.rs:69
#4  0x000055d3cd66a587 in std::sys_common::condvar::Condvar::wait (self=0x55d3ce084e50, mutex=0x55d3ce080d00) at /usr/src/rustc-1.40.0/src/libstd/sys_common/condvar.rs:41
#5  0x000055d3cd641d2d in std::sync::condvar::Condvar::wait (self=0x55d3ce085288, guard=...) at /usr/src/rustc-1.40.0/src/libstd/sync/condvar.rs:204
#6  0x000055d3cd683801 in x11rb::rust_connection::RustConnection<R,W>::read_packet_and_enqueue (self=0x55d3ce0851a0, inner=...) at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:152
#7  0x000055d3cd67f5bc in <x11rb::rust_connection::RustConnection<R,W> as x11rb::connection::Connection>::wait_for_event (self=0x55d3ce0851a0) at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:323
#8  0x000055d3cd6852d7 in simple_window_manager::main () at examples/simple_window_manager.rs:369

So... the "main thread" is waiting on the condition variable while the thread which is supposed to send a close event is blocked trying to read an event from the wire.
There are two pending replies:
(gdb) frame 7
#7  0x000055d3cd67f5bc in <x11rb::rust_connection::RustConnection<R,W> as x11rb::connection::Connection>::wait_for_event (self=0x55d3ce0851a0)
    at /home/psychon/projects/x11rb/src/rust_connection/mod.rs:323
323	            inner = self
[after some experimentation]
(gdb) print self.inner.data.value.pending_replies.tail
$10 = 3
(gdb) print self.inner.data.value.pending_replies.head
$11 = 5

Getting to these two replies is a bit harder, but:
(gdb) print *(self.inner.data.value.pending_replies.buf.ptr.pointer + 3)
$25 = (9, x11rb::utils::Buffer::Vec(alloc::vec::Vec<u8> {buf: alloc::raw_vec::RawVec<u8, alloc::alloc::Global> {ptr: core::ptr::unique::Unique<u8> {pointer: 0x55d3ce0856e0 "\001\000", _marker: core::marker::PhantomData<u8>}, cap: 32, a: alloc::alloc::Global}, len: 32}))
(gdb) print *(self.inner.data.value.pending_replies.buf.ptr.pointer + 4)
$26 = (10, x11rb::utils::Buffer::Vec(alloc::vec::Vec<u8> {buf: alloc::raw_vec::RawVec<u8, alloc::alloc::Global> {ptr: core::ptr::unique::Unique<u8> {pointer: 0x55d3ce085710 "\001\000", _marker: core::marker::PhantomData<u8>}, cap: 32, a: alloc::alloc::Global}, len: 32}))

At this point I needed raw pointers to get to the actual reply contents:
(gdb) x/32xb 0x55d3ce0856e0
0x55d3ce0856e0:	0x01	0x00	0x09	0x00	0x00	0x00	0x00	0x00
0x55d3ce0856e8:	0xef	0x00	0x00	0x00	0x00	0x00	0x00	0x00
0x55d3ce0856f0:	0x00	0x00	0x00	0x00	0x00	0x00	0x00	0x00
0x55d3ce0856f8:	0x00	0x00	0x00	0x00	0x00	0x00	0x00	0x00
(gdb) x/32xb 0x55d3ce085710
0x55d3ce085710:	0x01	0x00	0x0a	0x00	0x00	0x00	0x00	0x00
0x55d3ce085718:	0xf0	0x00	0x00	0x00	0x00	0x00	0x00	0x00
0x55d3ce085720:	0x00	0x00	0x00	0x00	0x00	0x00	0x00	0x00
0x55d3ce085728:	0x00	0x00	0x00	0x00	0x00	0x00	0x00	0x00

Uhm... yeah, no idea.
Anyway, I think somehow this ended up not seeing the reply even though it was already received.

Fun fact: Cannot reproduce after reverting e1c6fe3e78a026b9947449c20abacfff0747af55 :-/

Uhhh...
            Ok(mut lock) => {
                drop(inner);

                let packet = read_packet(&mut *lock)?;
                drop(lock);

                inner = self.inner.lock().unwrap();
                inner.enqueue_packet(packet);
                self.reader_condition.notify_all();
                Ok(inner)
            }

The thread that tries to read drops its lock called lock before acquiring self.inner.lock(). Thus, at the time this thread tries to lock self.inner, another thread could also end up calling read_packet_and_enqueue() and then proceed waiting on the connection. Even if the pending packet is for that other thread.
The fix is to reorder things: Do not drop(lock) before self.inner.lock().
Sadly, I do not have the time to test this theory yet. At least the symptoms fit this explanation.
Also, no idea how to write a regression test for this race...

This an explanation of what could be happening, step by step:

Thread 1 is in the event loop blocked on read_packet waiting for some data.
Thread 2 sends an InternAtom request (there are actually two, but lets simplify), but buffer do not get flushed.
Thread 2 waits X11RB_EXAMPLE_TIMEOUT
Thread 2 flushes the buffers before fetching for the reply
Thread 1 reads the reply and releases the lock on read (but does not lock inner yet, nor enqueue the packet)
Thread 2 enters read_packet_and_enqueue (because inner was unlocked) and waits on read_packet. This wait lasts forever because the reply has already been read (and the X11 server does not have anything else to send)
Thread 1 locks inner, enqueues the packet and notifies the condvar (but thread 2 is not waiting on the condvar)
Thread 1 waits on the condvar (because read is locked by thread 2), and stays there forever

Placing the drop(lock) after self.inner.lock() as suggested, will prevent step 6 from happening, because thread 2 would either:

if inner is locked: wait for the lock on inner to be released, so poll_for_reply will get the packet enqueued by thread 1
if inner is unlocked: wait on the condvar (because read is still locked), so it would get the notification from thread 1

When I wrote the code, my thought process was "the read lock is dropped
immediately after read_packet so I should do the same in the new
code". Which is of course faulty because the read lock wasn't actually
used to lock.

Fixed by #232.


crashes at launch

crash-2022-12-24_<IP_ADDRESS>28-quilt_loader.txt

@Sujeitom Either use Phosphor or Starlight.

https://modrinth.com/mod/starlight

Hi, please ask support questions in the forum or in discord.
Thanks!


Include other commit prefixes to changelog.md from pre-defined CHANGELOG_SECTIONS

I use release-please action and it works like a charm!
I employ simple setup for python:
      - uses: googleapis/release-please-action@v4
        with:
          token: ${{ secrets.RELEASE_PLEASE_TOKEN }}
          release-type: python

I have seen that most commit prefixes are not written to changelog file, probably based on hidden flag defined in CHANGELOG_SECTIONS
Is there an option in the github action itself to define which prefixes to include? If not, what is the most elegant solution to achieve this?
Thank you in advance!

Great question, I also have similar needs


BottomNavigationBarItem changes color when there are more than three items on the list

Steps to Reproduce
This is currently happening in Chrome and MacOS
Chrome:
https://user-images.githubusercontent.com/65375294/186971315-0af0afe4-8c19-4e86-be22-5e72b0ca9768.mov
MacOS:
https://user-images.githubusercontent.com/65375294/186971758-a670a275-3280-4e82-ab5f-93bac070a8ec.mov

Execute flutter run on the code sample (see "Code sample" section below)
Uncomment the last BottomNavigationBarItem and you will see that the color breaks.

I was expecting to see the same result as with three elements:

What I got was this:

Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

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

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Wrap Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Films'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int index = 0;
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: const Placeholder(),
      bottomNavigationBar: BottomNavigationBar(
              items: const [
                BottomNavigationBarItem(
                    icon: Icon(
                      Icons.movie,
                    ),
                    label: 'Movies'),
                BottomNavigationBarItem(
                    icon: Icon(Icons.event), label: 'Up and Coming'),
                BottomNavigationBarItem(
                    icon: Icon(Icons.theater_comedy), label: 'Genre'),
                //BottomNavigationBarItem(icon: Icon(Icons.place), label: 'Theaters',),
              ],
            )
    );
  }
}

Logs

Analyzing main.dart...
No issues found! (ran in 0.9s)

[!] Flutter (Channel master, 3.1.0-0.0.pre.2495, on macOS 12.5.1 21G83 darwin-arm, locale en)
    • Flutter version 3.1.0-0.0.pre.2495 on channel master at /Users/pedrozas/Progra/flutter
    ! Upstream repository git@github.com:snat-s/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to git@github.com:snat-s/flutter.git to dismiss this error.
    • Framework revision fefb2b00ba (22 hours ago), 2022-08-25 13:56:24 -0700
    • Engine revision b9ba679200
    • Dart version 2.19.0 (build 2.19.0-139.0.dev)
    • DevTools version 2.16.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/pedrozas/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio with Blaze.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

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

[✓] Android Studio
    • Android Studio at /Applications/Android Studio with Blaze.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.13+0-b1751.21-8125866)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 104.0.5112.101

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I think this is the default behavior for the code sample. Looking a bit more through the docs and realized that this is because of the following.
If [backgroundColor] is null, The
navigation bar's background color defaults to the [Material] background
 color, [ThemeData.canvasColor] (essentially opaque white).

More info in here.


Animation not working all the time with Jekyll

Hello,
I'm trying to achieve a simple on hover animation using the coffeescript below, my website is running on the latest version of Jekyll.
$(document).ready ->
  $('.site-title img').addClass 'animated jello'
  $('.page-link').hover ->
    $(this).toggleClass 'animated jello'

The first portion works - as soon as the page loads, the animated jello classes get added to .site-title img and it animates. However the second portion doesn't work. I can see that the classes are being added to .page-link on hover - however, no animation occurs.
What do you think might be causing this?

display: inline-block it. Animations will not work on inline elements. This is part of the CSS specs.

You nailed it first go - that was the issue. Thank you! :smile: :+1:


Adaptive: Detect and load graph cycles during type loading

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/2230 https://github.com/microsoft/botbuilder-dotnet/issues/3626 https://github.com/microsoft/botbuilder-dotnet/issues/3742
Simple 2-pass graph cycle detection algorithm integrated into the json.net infra. The first pass loads a cache mapping (jtoken, type) -> object until it finds a cycle. The second pass just stitches the missing pieces into the main object, using references stored in the cache.
Given that we have multiple types of json converters that need to benefit from cycle detection code, we externalize cycle detection into the CycleDetectionObserver. Converters can take observer registrations.
Note that the innate complexity of this problem is substantial. With that in mind, a significant percentage of the time spent here was trying to make this as simple to maintain and understand as possible. Additional requirements are:

Cycle detection needs to be reusable to different converters (from here we get that it needs to be externalized
Cycle detection should not use global state
2 resource explorers should be able to do cycle detection in parallel
a single resource explorer should be able to load types and do cycle detection in multiple threads at the same time
Given that this is a multi-pass algorithm, and the tasks that need to be done on each algorithm are different, some entity must coordinate all the pieces to know the pass. We achieve this by having a single cycledetector and registering it in all observable converters from the resource explorer

4 failing tests, submitting a fix in the morning :)

Something weird going on with tests, which run locally. Investigating.

            var evt = selection.First();

this should be called "trigger" or "condition"

Refers to: libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialog.cs:843 in 68e634b. [](commit_id = 68e634b592c44ddbf5c8f75d45e782cfea7f81b6, deletion_comment = False)


Bumped wascap to v0.10.1

Feature or Problem
This PR bumps wascap to 0.10.1 to bring in a bug fix for modules signed with v0.9.0. This should be released alongside wasmCloud v0.62.0 to ensure compatibility of signed modules
Related Issues
https://github.com/wasmCloud/wascap/issues/45
Release Information
v0.17.0, or whenever we release the version of wash up with v0.62.0
Consumer Impact
Folks using this version of wash will have to use wasmCloud v0.62.0 or greater, so this might be released as a minor bump.
Testing

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin
[ ] x86_64-windows

Tested on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin
[ ] x86_64-windows

Unit Test(s)
Nothing modified
Acceptance or Integration
Nothing modified
Manual Verification
I am testing and ensuring this works with the v0.62.0 release candidate

(Technically you don't need to specify 0.10.1 in the Cargo.toml, a cargo update is enough to pull in the latest patch version)


Show more information

Is your feature request related to a problem? Please describe.
Currently only alert and severity are shown.
Describe the solution you'd like
Meteoalarms (at least from eumetsat) have a short description and duration issued together with a code and level. It would be nice to have these shown along with the basic information.
Additional context
None

Not a desing goal of the card


Parse error at (k8s-monitoring/templates/_helpers.tpl:11): unclosed action

Started by following the docs at https://<your setup name>.grafana.net/a/grafana-k8s-app/configuration/cluster-config, then also tried cloning the chart locally and installing. Both returned the same error: Error: parse error at (k8s-monitoring/templates/_helpers.tpl:11): unclosed action
values.yaml (censored):
cluster:
  name: "scavenger-labs"

externalServices:
  prometheus:
    host: "https://prometheus-prod-36-prod-us-west-0.grafana.net"
    basicAuth:
      username: "***"
      password: "********"

  loki:
    host: "https://logs-prod-021.grafana.net"
    basicAuth:
      username: "***"
      password: "********"

opencost:
  opencost:
    exporter:
      defaultClusterId: "scavenger-labs"
    prometheus:
      external:
        url: "https://prometheus-prod-36-prod-us-west-0.grafana.net/api/prom"

Nothing special going on here, and no extra values provided in the helm upgrade/install command, just followed the normal install instructions from a Grafana cloud install.
kubectl version:
 kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"clean", BuildDate:"2021-04-08T16:31:21Z", GoVersion:"go1.16.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.4+k3s1", GitCommit:"8d0255af07e95b841952563253d27b0d10bd72f0", GitTreeState:"clean", BuildDate:"2023-04-20T00:33:18Z", GoVersion:"go1.19.8", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.21) and server (1.26) exceeds the supported minor version skew of +/-1

helm version:
helm version
version.BuildInfo{Version:"v3.5.4", GitCommit:"1b5edb69df3d3a08df77c9902dc17af864ff05d1", GitTreeState:"clean", GoVersion:"go1.15.11"}

As of now, this chart requires Helm CLI version 3.7 or greater. Try upgrading your version of Helm and trying again.

That did it! Thanks so much, just got a clean install.


Config

Review files for py3

Conflicting PR. Removed from build OMERO-docs#198. See the console output for more details.
Possible conflicts:

PR #2019 jburel 'exclude zeroc forum'

omero/conf.py

PR #2024 sbesson 'Define extlink for versioned Ansible documentation'

contributing/conf.py

--conflicts

All of the changes are pretty straightforward Python2->Python3 compatibility changes. Both Travis and Jenkins builds are passing. Merging ahead of the documentation work.


Rough Accelerate implementation, and updated finetuning colab

dont mind the commit messages regarding the colab

Can you make sure you do not replace the original files, because it won't work for multi-GPU? Just add a new file called train_finetune_accelerate.py would be fine.

Can you make sure you do not replace the original files, because it won't work for multi-GPU? Just add a new file called train_finetune_accelerate.py would be fine.

Sure, will do

@yl4579 done

I think the current state is too messy that I can't fix it. Can you just submit a PR with only train_finetune_accelerate.py? I can do that too but I want you to be credited.

I think the current state is too messy that I can't fix it. Can you just submit a PR with only train_finetune_accelerate.py? I can do that too but I want you to be credited.

will do

@korakoe Thanks I have credited you in README.

Thank you for doing this. Running on a 4 GPU ( p3.8xlarge AWS ) , both this script and the non single GPU, throws this error.
RuntimeError: Caught RuntimeError in replica 0 on device 0.
Original Traceback (most recent call last):
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/parallel/parallel_apply.py", line 61, in _worker
    output = module(*input, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/StyleTTS2/Colab/StyleTTS2/Modules/hifigan.py", line 458, in forward
    F0 = self.F0_conv(F0_curve.unsqueeze(1))
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1128, in _call_impl
    result = forward_call(*input, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 302, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 298, in _conv_forward
    return F.conv1d(input, weight, bias, self.stride,
RuntimeError: Given groups=1, weight of size [1, 1, 3], expected input[1, 199, 1] to have 1 channels, but got 199 channels instead ```

@vishalsantoshi See #126

@yl4579  thanks, saw the linked issue. Please note that the batch_size was 2 and the the batch_percentage was not touched ( remains 0.5 ). The same setup works with a single GPU instance.  This is straight up execution of the tuning notebook.  Will test it again and confirm.
config['batch_size'] = 2 # not enough RAM
config['max_len'] = 100 # not enough RAM

@martinambrus Yes it works too, but since it is too slow to train it from scratch using a single GPU, I don't think it's worth implementing.

@martinambrus Yes it works too, but since it is too slow to train it from scratch using a single GPU, I don't think it's worth implementing.

it might resolve the DDP issue if done properly, but I believe someone ended up fixing the DDP. here's the message they sent me as I don't have access to their fork
Basically, I place operators write to memory that has to be shared amongst all processes
So if they are all written at once
By different things and synced
Causes problems

not sure how useful this ends up being


Add double-bracket strings

Would close https://github.com/prql/prql/issues/721. I need to add to the docs.
I had previously implemented an rs"foo" string. But Actually I think this is fairly standard, and more closely aligns with the current problem. Definitely open to feedback though.

Currently the SQL formatter is changing {?foo} to ( ? foo }. I need to think about the best way of dealing with this — is there some generalized way of surrounding the expression so that the SQL formatter doesn't mangle it, but it's still valid?
Otherwise we'll have to replace it with something and then swap it back after the formatting, which would be a bit inelegant.

Possibly the way to do this is to surround double-bracket strings with dialect-specific "quotes" — e.g. for MySQL & BigQuery, we use backticks; for others we use double-quotes.
I can't see an escape hatch like we do for jinja expressions (https://github.com/prql/prql/blob/0.2.0/prql-compiler/src/sql/translator.rs#L46).

Currently the SQL formatter is changing {?foo} to ( ? foo }

fyi, just as a related data point, our PRQL pipeline already does this kind of replacing post-compilation as we use postgres's @? inside of an S-string, which gets mangled to @ ? (i.e. invalid sql)

Thanks for that case @mklopets
I increasingly think we should parse to some representation of "raw ident", e.g. backticks, and then based on the dialect convert to either backticks or double-quotes.

Not sure what's going on with the commit history but it'll be squashed...
Added some docs too; lmk if anything is unclear!


Command is not available error

Hi,
I am getting following error when I try to execute the command.
Error detected while processing function stackanswers#StackAnswers:
line    6:
E319: Sorry, the command is not available in this version: pyfile /home/devil/.vim/bundle/stackanswers.vim/autoload/stackanswers.py
line    7:
E319: Sorry, the command is not available in this version:         python stackAnswersVim("a:2")
Thanks

Do you have python installed?

Yes, I do. Suggested python modules are also installed.

Apparently this issue may happen if your version of vim doesn't support python. This plugin requires vim to be built with python 2. Does vim --version include "+python"?

Okay, that's a problem since this plugin expects python 2. I'll try and make it work with python 3 when I have some time.

This should be resolved now. Please let me know if you're still having issues.

Hey, so I'm still receiving this error:
Error detected while processing function stackanswers#StackAnswers:
line   14:
E319: Sorry, the command is not available in this version:         python stackAnswersVim("a:2")


Error with the just added terraform snippets

Hello, I'm getting this while editing terraform files:
2023-05-20T08:30:38 Messages  INFO Error executing lua callback: ...packer/start/LuaSnip/lua/luasnip/loaders/from_vscode.lua:54: bad argument #1 to 'concat' (table expected, got nil)
stack traceback:
	[C]: in function 'concat'
	...packer/start/LuaSnip/lua/luasnip/loaders/from_vscode.lua:54: in function 'get_file_snippets'
	...packer/start/LuaSnip/lua/luasnip/loaders/from_vscode.lua:100: in function 'load_snippet_files'
	...packer/start/LuaSnip/lua/luasnip/loaders/from_vscode.lua:271: in function '_load_lazy_loaded_ft'
	...packer/start/LuaSnip/lua/luasnip/loaders/from_vscode.lua:284: in function '_load_lazy_loaded'
	...e/nvim/site/pack/packer/start/LuaSnip/plugin/luasnip.lua:103: in function <...e/nvim/site/pack/packer/start/LuaSnip/plugin/luasnip.lua:102>
2023-05-20T08:30:38 Error  ERROR Erro detectado ao processar BufWinEnter Autocommands for "*":

Rolling back to b248e8db97ab23b189e65cd0864db42c472bfc54 stops the error messages.

This error is caused by the copyright object as it is missing the body field used by LuaSnip. Removing that line (and ensuring the resulting JSON file is syntactically correct) stops the error from occurring.

For some reason I could not reproduce the problem, @aleprovencio let me know if latest commit fixes the problem. Thanks @snoord for the fix.

Thanks for the input guys, but unfortunately this is still not a fix here.
When you first open a terraform file, the error will not show, but you can notice something is wrong because of the missing syntax highlighting.
After that if you open a file which is not a terraform one (like a README.md or maybe a playbook.yml) and then go back to that terraform file, now the error will pop up.


Unexpected artifacts from lg.JoinVertical

Describe the bug
JoinVertical is adding extra newLines (or otherwise something weird is happening) to the topmost passed text.
Setup
Please complete the following information along with version numbers, if applicable.

OS: Ubuntu
Shell ... the ubuntu shell?
Terminal Emulator [e.g. kitty, iterm]
Terminal Multiplexer: under tmux & with no multiplexer
Locale: unsure

To Reproduce
Steps to reproduce the behavior:
git clone http://github.com/nilock/tuido
cd tuido
go run .
[press ENTER] - opens context preview on the first listed todo item]

This will display the broken text from below.
To see the same text unbroken, replace return lg.JoinVertical(lg.Left, bodyWithPointer, foot) with return bodyWithPointer on line 44 of ./tui/peek.go, and rerun the above instructions.
Source Code
http://github.com/nilock/tuido
Expected behavior
See screenshots.
Screenshots
Broken text:

The same text not joined with the footer:

Additional context
The broken text is read at runtime from files on disk. The breaking newline behaviour only occurs when rendering .go files - the .md (eg, readme.md) file gets rendered correctly. AFAICT, both are Uni-8. I can't guess why the discrepancy, but this feels like a good clue.
Some other things are happening to this text before the JoinVertical

colorized with monokai
joined with the pointer + border on left via JoinHorizontal
styled with padding

... Manually removing each of these showed no effect - the described issue still presents.

I am running into something similar https://github.com/charmbracelet/lipgloss/issues/317


case-insensitive import collision

please make sure is： “Sirupsen/logrus” or “sirupsen/logrus”

Should be solved by https://github.com/MindFlavor/ftpserver2/pull/11.


Add leaky_relu operator property

Adding Leaky_Relu into operator property, which was missing, although quantized leaky relu was implemented. This resolve the issue: https://github.com/tensorflow/tensorflow/issues/33397

Thanks.


Fix/172 css grid restyling

Description
This PR removes the usage of !important for Grid and FlexGrid component. The front end was built to accommodate this !important override in the beginning so a lot of components need to be refactored in another issue.
To keep each PR small(fairly small), this one only refactored the Collection Step of the registration flow as an example.
To see the reason for this change please see #172 for detail.
Summary of changes

remove important for grid css in the component override file
refactor css styling for CollectionStep and SeedlotRegistrationForm
add button text and icon customization feature to SubmitModal component

Note:
Fixes #172
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Local testing
Checklist

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

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

As @mgaseta pointed out, this PR contains some changes for the parent tree step component which do not fit the scope of this issue. The reason being this task was branched out of the development of parent tree, in the future this practice will be avoided. For "efficiency" sake, these changes will be pushed to main this time.


Updated the changelog for the new release

Merge only after merging the profit calculations PR https://github.com/tellor-io/TellorMiner/pull/228
Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 333656283

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.3%

Totals

Change from base Build 333638875:
0.0%

Covered Lines:
1627

Relevant Lines:
4248

💛  - Coveralls


Overhaul junos.NewSession(), and provide support for alternative constructors

Issue #40

Hey @daemus! Sorry it's taken me so long to get this in, but I've merged it! Thank you for the submission!


Debian binary package - how to apply custom configuration

Gogs version (or commit ref): 0.9.71-1471414190.a00c932 (Debian package)
Git version: a00c932
Operating system: Debian 8
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:N/A

Description
I am trying to apply custom configuration and I've followed multiple guides but I am confused as to what I need to do after creating a /opt/gogs/custom/conf/app.ini file[1].
My custom app.ini contains:
[service]
DISABLE_REGISTRATION = true
REQUIRE_SIGNIN_VIEW = true

Then I restart the service systemctl restart gogs. But the configuration is not sticking.
I am also referencing the official "Config and run" doc[2] but it does not describe how to do this in production (with binaries).

https://www.progville.com/tools/gogs-setting-self-hosted-github-clone/
https://gogs.io/docs/installation/configuration_and_run
...

Hi, this package is not built by us (me), so I'm not sure how exactly. Maybe helpful: https://github.com/gogits/gogs/issues/2174

Was very helpful, thank you.