Issue building android duplicate versions of jetified-kotlin-stdlib-1

Description
running npx react-native android
I get the following error:
Execution failed for task ':app:mergeDebugJavaResource'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
   > More than one file was found with OS independent path 'META-INF/kotlin-stdlib-jdk7.kotlin_module'.

This repros in CI as well
Version
0.63.5
Output of npx react-native info
System:
OS: macOS 13.0.1
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 136.32 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 14.18.0 - ~/.nvm/versions/node/v14.18.0/bin/node
Yarn: 1.22.18 - /usr/local/bin/yarn
npm: 8.19.2 - ~/.nvm/versions/node/v14.18.0/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 4.2 AI-202.76<IP_ADDRESS>51085
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: <IP_ADDRESS> - /usr/bin/javac
Python: 3.10.3 - /Users/Jake/.pyenv/shims/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.5 => 0.63.5
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
npx react-native android
Snack, code example, screenshot, or link to a repository
NA

I believe this is the same issue I've been battling with for a couple of hours now.
`

What went wrong:
Execution failed for task ':app:checkReleaseDuplicateClasses'.

A failure occurred while executing com.android.build.gradle.internal.tasks.CheckDuplicatesRunnable
Duplicate class kotlin.collections.jdk8.CollectionsJDK8Kt found in modules jetified-kotlin-stdlib-1.8.0 (org.jetbrains.kotlin:kotlin-stdlib:1.8.0) and jetified-kotlin-stdlib-jdk8-1.4.10 (org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.4.10)
`

I found a potential fix of upgrading kotlinVersion to 1.8.0 in build.gradle, but that didn't help. Apparently because React Native has version 1.4.10 on it.

I believe this is the same issue I've been battling with for a couple of hours now.
* What went wrong:
Execution failed for task ':app:checkReleaseDuplicateClasses'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.CheckDuplicatesRunnable
   > Duplicate class kotlin.collections.jdk8.CollectionsJDK8Kt found in modules jetified-kotlin-stdlib-1.8.0 (org.jetbrains.kotlin:kotlin-stdlib:1.8.0) and jetified-kotlin-stdlib-jdk8-1.4.10 (org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.4.10)

I found a potential fix of upgrading kotlinVersion to 1.8.0 in build.gradle, but that didn't help. Apparently because React Native has version 1.4.10 on it.

I thought I was the only one, I have also been getting this same issue since today.

Our nightly build started exhibiting this issue ~ 18 hours ago.

Our nightly build started exhibiting this issue ~ 18 hours ago.

This was when I started seeing this issue as well.

This fix worked for me in the meantime:
     constraints {
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
            because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
        }
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
            because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
        }
    }

(android/app/build.gradle)
I think if using a modern Kotlin Gradle Plugin you can also add the Gradle property kotlin.stdlib.jdk.variants.version.alignment=false
(https://kotlinlang.org/docs/whatsnew18.html#usage-of-the-latest-kotlin-stdlib-version-in-transitive-dependencies)

This also worked for me!

This fix worked for me in the meantime:
     constraints {
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
            because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
        }
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
            because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
        }
    }

(android/app/build.gradle)
I think if using a modern Kotlin Gradle Plugin you can also add the Gradle property kotlin.stdlib.jdk.variants.version.alignment=false (https://kotlinlang.org/docs/whatsnew18.html#usage-of-the-latest-kotlin-stdlib-version-in-transitive-dependencies)

Perfect, works! Thank you.

Fixed work for me.

[Checkout here](https://youtrack.jetbrains.com/issue/KT-55297/kotlin-stdlib-should-declare-constraints-on-kotlin-stdlib-jdk8-and-kotlin-stdlib-jdk7)

This fix worked for me in the meantime:
dependencies {
     constraints {
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
            because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
        }
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
            because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
        }
    }
}

(android/app/build.gradle)
I think if using a modern Kotlin Gradle Plugin you can also add the Gradle property kotlin.stdlib.jdk.variants.version.alignment=false (https://kotlinlang.org/docs/whatsnew18.html#usage-of-the-latest-kotlin-stdlib-version-in-transitive-dependencies)

Worked for me. Thanks

Closing as the workaround is reported as working fine.

In my company's case I did a comparison of the gradle dependency graph to identify what introduced the change in the build dependencies.
The problem was introduced by this library's use of a version wildcard:
https://github.com/proyecto26/react-native-inappbrowser/issues/398
And the solution we went with was to specify androidXAnnotation = "1.5.0" as an ext in the android/build.gradle file

This fix worked for me in the meantime:
dependencies {
     constraints {
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
            because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
        }
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
            because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
        }
    }
}

(android/app/build.gradle)
I think if using a modern Kotlin Gradle Plugin you can also add the Gradle property kotlin.stdlib.jdk.variants.version.alignment=false (https://kotlinlang.org/docs/whatsnew18.html#usage-of-the-latest-kotlin-stdlib-version-in-transitive-dependencies)

This can lead to other problems with kotlin files like

Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.8.0, expected version is 1.6.0.

Solution by @RyanThomas73 fixes the root cause: We also had a library where the androidx.core:core used a wildcard. Specifying a pinned version fixed the issue.

constraints {
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
}
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
}
}

This works for me also

implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.0"))

This worked for me.

In my company's case I did a comparison of the gradle dependency graph to identify what introduced the change in the build dependencies.
The problem, in our case, was introduced by this library's use of a version wildcard: proyecto26/react-native-inappbrowser#398
And the solution we went with was to specify androidXAnnotation = "1.5.0" as an ext in the android/build.gradle file

this should be the best workaround

In my company's case I did a comparison of the gradle dependency graph to identify what introduced the change in the build dependencies.
The problem, in our case, was introduced by this library's use of a version wildcard: proyecto26/react-native-inappbrowser#398
And the solution we went with was to specify androidXAnnotation = "1.5.0" as an ext in the android/build.gradle file

worked for me, the solution from @jbaez didn't work for some reason.

This fix worked for me in the meantime:
dependencies {
     constraints {
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.0") {
            because("kotlin-stdlib-jdk7 is now a part of kotlin-stdlib")
        }
        implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0") {
            because("kotlin-stdlib-jdk8 is now a part of kotlin-stdlib")
        }
    }
}

(android/app/build.gradle)
I think if using a modern Kotlin Gradle Plugin you can also add the Gradle property kotlin.stdlib.jdk.variants.version.alignment=false (https://kotlinlang.org/docs/whatsnew18.html#usage-of-the-latest-kotlin-stdlib-version-in-transitive-dependencies)

YOU ARE MY HERO

Below fixed work for me.
implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.0"))

Checkout here

Saved my day

implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.0"))

Me too. But I sinceraly don't understand why after creation Java-project with Android Studio's wizard I have any problems with Kotlin.

你好
我是聋人，第一次接触Andorid开发，不知道怎么入手。。求指导下，谢谢

Below fixed work for me.
implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.0"))

Checkout here

Wow, it helped me too. My issue of below has been solved to.
Reason of my issue

I was migrating from RN v0.71.7 to RN v0.72.0 & that issue raised. Your solution worked 🔥

Error screenshot

I had the same error message in an unrelated project.
I found these two lines in my build.gradle.kts file:
implementation(libs.androidx.material3)
implementation(libs.androidx.material3.android)

I removed the second line and then it worked.


command "Create MCProject files" not woking

I can't use "Create MCProject files" command
It return the error:

Should be fixed in v3.2.81


Update Selenium Grid for WinAppDriver information

I am looking at this guide here.
https://github.com/Microsoft/WinAppDriver/wiki/Working-with-the-Selenium-Grid
There is some good information, and there is a link for 'here' where there is supposed to be more information. This link is no longer working.
Anyone else know what resource this took you too or any other guides as far as getting started with WinAppDriver Selenium Grid?

@Tree55Topz What are you trying to do?  Reading to what goal?

@liljohnak sorry I ment to close this one - I have since gotten further, but am now stuck trying to get the Hub to actually pass tests out to the nodes. Here is the latest Issue, closing this one
https://github.com/microsoft/WinAppDriver/issues/1231


Show the correct tab for subfield errors

WHY
To correctly detect the crud fieldName for subfields with validation errors and display the relevant tab to maintain consistency with the behaviour of other fields that have validation errors.
BEFORE - What was wrong? What was happening before this PR?
When the first error is on a subfield, I expect that tab to be the active tab as it would be with other fields.
Because the fieldname is in dot notation array format, this does not match the crud fieldname e.g. someRelation.0.name does not match the fieldName someRelation, therefore the error is skipped when determining the tab to activate
AFTER - What is happening after this PR?
The crud fieldName is parsed from the current field under test and will match the parent fieldName, allowing it to set the active tab appropriately.
HOW
How did you achieve that, in technical terms?
I stripped the crud fieldName from the start of the subfield name
$fieldName = preg_split('/\.\d+\./',$fieldName)[0];
Is it a breaking change?
No
How can we test the before & after?
Create a validation error on a subfield that is not on the first tab.
Before: the errors are shown, but the subfield's tab is not active.
After: The tab with the subfield is the active tab.

Thank you @krjohnston 🙏
Much appreciated. Will tag a new version later today with the changes!
Cheers

Thanks @pxpm


fix to show the tinymce editor for the content field

this works for me to get at least the editor up for the content field

Hello,
effectively the editor is showing up, but is not enough...
But thank you for your contribution.


[Bug]: Cannot push to multiple remotes simultaneously

Describe the bug
The plugin can only push to the last remote it has pushed. When there are several remotes, it cannot simultaneously push to them all.
Relevant errors (if available)
No response
Steps to reproduce
Set up multiple remotes, and run the Push command of the plugin in Obsidian.
Expected Behavior
It should be able to push to all the remotes simultaneously.
Addition context
on macOS Mojave
Operating system
macOS
Plugin version
2.5.1

But what does happen if you try to push to multiple remotes?

@Vinzent03
Sorry for my naive question, but how to push with a single git command line?

According to the SE answer, even when I have created a new "remote" combining the two current remotes, so I can push to all the remotes in a single command. The push in Obsidian still only push to only one remote.

Did you keep the old remote and just added all? I guess the upstream branch, which is used by the plugin is still the old one. I see there is no command to change it. So you will have to edit the config.

@Vinzent03
Thank you for the discussion, problem solved according to this SE answer.


Reduce babel config

Will break common build as now all common modules contains import-export

Super now we are again have one source of babel plugins. Thank you.


Rearrange Dict

Codecov Report

Merging #11 into master will increase coverage by 0.36%.
The diff coverage is 69.38%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage    48.8%   49.17%   +0.36%
==========================================
  Files          13       14       +1
  Lines        1889     1873      -16
==========================================
- Hits          922      921       -1
+ Misses        919      906      -13
+ Partials       48       46       -2

Impacted Files
Coverage Δ

jen/lit.go
75% <ø> (+16.86%)
:white_check_mark:

jen/group.go
56.36% <100%> (+1.64%)
:white_check_mark:

genjen/render.go
98.72% <100%> (ø)
:white_check_mark:

jen/dict.go
65.9% <65.9%> (ø)

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 78e962e...897a360. Read the comment docs.

Breaking change:
Before:
Dict(map[Code]Code{Lit("a"): Lit("b")})

After:
Values(Dict{Lit("a"): Lit("b")})

Before:
DictFunc(func(m map[Code]Code){
    m[Lit("a")] = Lit("b")
})

After:
Values(DictFunc(func(d Dict){
    d[Lit("a")] = Lit("b")
}))

Before:
Dict(nil)

After:
Values()


adapt CI workflows and test examples (GDEV-404)

Title for squash and merge:
adapt CI workflows and test examples (GDEV-404)

Description for squash and merge:
- reduce test examples to minimum (caused too much work when adapting the repo to a service)
- restructure test fixtures
- added script and CI workflow to update/check the `example_config.yaml` and the `config_schema.json`
- added CI workflow to check integrity of DB migration scripts

Can the GH action workflow boilerplate code be moved to a composite action and be included via uses? I mean

Get/export important runtime variables
Set up Python 3.9
Install Dependencies

Maybe all the adaptation hints could be made testable by using a dedicated placeholder keyword like __CHANGEME__ or so which could be searched for by a commit hook

Thanks @lkuchenb, that are great ideas. Could you take that over?

Codecov Report

Merging #55 (4ff8dce) into dev (c61ac29) will increase coverage by 6.74%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##              dev      #55      +/-   ##
==========================================
+ Coverage   61.79%   68.53%   +6.74%
==========================================
  Files          13       14       +1
  Lines          89       89
==========================================
+ Hits           55       61       +6
+ Misses         34       28       -6

Impacted Files
Coverage Δ

my_microservice/__main__.py
0.00% <0.00%> (ø)

my_microservice/dao/__init__.py
0.00% <0.00%> (ø)

my_microservice/dao/db.py
0.00% <0.00%> (ø)

my_microservice/dao/db_models.py
0.00% <0.00%> (ø)

my_microservice/api/deps.py
100.00% <100.00%> (ø)

my_microservice/api/main.py
100.00% <100.00%> (ø)

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

my_microservice/core/__init__.py
100.00% <100.00%> (ø)

my_microservice/models.py
100.00% <100.00%> (ø)

my_microservice/core/greeting.py
91.66% <0.00%> (-8.34%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #55 (3076b16) into dev (c61ac29) will increase coverage by 6.74%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##              dev      #55      +/-   ##
==========================================
+ Coverage   61.79%   68.53%   +6.74%
==========================================
  Files          13       14       +1
  Lines          89       89
==========================================
+ Hits           55       61       +6
+ Misses         34       28       -6

Impacted Files
Coverage Δ

my_microservice/__main__.py
0.00% <0.00%> (ø)

my_microservice/dao/__init__.py
0.00% <0.00%> (ø)

my_microservice/dao/db.py
0.00% <0.00%> (ø)

my_microservice/dao/db_models.py
0.00% <0.00%> (ø)

my_microservice/api/deps.py
100.00% <100.00%> (ø)

my_microservice/api/main.py
100.00% <100.00%> (ø)

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

my_microservice/core/__init__.py
100.00% <100.00%> (ø)

my_microservice/models.py
100.00% <100.00%> (ø)

my_microservice/core/greeting.py
91.66% <0.00%> (-8.34%)
:arrow_down:

Continue to review full report at Codecov.

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

The license checker script is failing because of the year. This can be ignored until a fix is rolled out.


Update shell.nix for newer LLVM versions and aarch64-darwin

Does MACOSX_DEPLOYMENT_TARGET also need to be updated?

Does MACOSX_DEPLOYMENT_TARGET also need to be updated?

Sounds reasonable, although tbh I don't know what's a sensible X in 11.X

I've only been using nix for a day, decided to try not using homebrew on a new computer, but this PR worked for me first try
/t/crystal (feature/nix-shell)> nix-shell
these 2 derivations will be built:
  /nix/store/8c5yzvwwd0rz20qnzb8g3p7cxifr30v2-boehm-gc-8.2.0.drv
  /nix/store/a8ml099barsdjffjcxw87wfmqymf9hbk-crystal-binary.drv
these 128 paths will be fetched (168.57 MiB download, 1033.73 MiB unpacked):
...on and on and on....

[nix-shell:/tmp/crystal]$ crystal --version
Crystal 1.6.1 [fc61bd678] (2022-10-21)

LLVM: 10.0.0
Default target: aarch64-apple-darwin

@HertzDevil @beta-ziliani What needs to be done with regards to MACOSX_DEPLOYMENT_TARGET to move this forward?

On a second thought, if we still can compile it in 10.11, I don't see the reason to change it.


MNT: rerender and migrate Python versions

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #109.
Here's a checklist to do before merging.

[x] Bump the build number if needed.

great many thanks @molinav :beer:


More details about Bookings with Me required

Could you include some documentation on actually setting up Bookings with me, and how people should access the Bookings with me page to configure and share their own Bookings with Me?

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

ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Version Independent ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Content: Bookings with me
Content Source: microsoft-365/bookings/bookings-in-outlook.md
Service: bookings
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@kwekuako Please help us in resolving this issue. Thanks


Add WidgetSDK documentation workflow

Jira issue:
https://glia.atlassian.net/browse/MOB-3137
What was solved?
This PR adds the necessary script for Bitrise to execute so that documentation page can be automatically generated and uploaded during release of the Widget SDK.
MOB-3137
Release notes:

[ ] Feature
[X] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Tests fixed, added? Unit, acceptance, snapshots?
[ ] Logging necessary for future troubleshooting of customer issues added?

Screenshots:
Widgets doc page:
http://ios-sdk-docs.salemove.com.s3-website-us-east-1.amazonaws.com/widgets/index.html

I have no idea why Bitrise fails with error

Failed to process Step inputs:
simulator UDID lookup failed:
runtime (com.apple.CoreSimulator.SimRuntime.iOS-17-5) is incompatible with device type (com.apple.CoreSimulator.SimDeviceType.iPhone-8-Plus)

!squash


python3Packages.pyxeoma: init at 1.4.1

Motivation for this change
Python wrapper for Xeoma web server API.
https://github.com/jeradM/pyxeoma
This is a Home Assistant dependency.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 109292 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma

Result of nixpkgs-review pr 109292 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109292 run on x86_64-linux 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109292 run on x86_64-linux 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109292 run on x86_64-darwin 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109292 run on x86_64-darwin 1

3 packages built:

python37Packages.pyxeoma
python38Packages.pyxeoma
python39Packages.pyxeoma


Catch exceptions for request to private Pidroid forum messages

https://github.com/JustAnyones/Pidroid/blob/ccb24a4e59388e6a4f70fe2af7281bf8fb128f2b/bot/cogs/ext/tasks/forum_messages.py#L32
This should be catched as the requests can sometimes fail and automatically stop the task.

Fixed by https://github.com/JustAnyones/Pidroid/commit/5140ef757de0e1521d3077ca8e17b5295bbc9b50


Split Client/Broker Code / reduce dependencies

Description
I'm trying to integrate the MQTTnet server/broker into one of my asp.net core 3.1 projects and found out, that the MQTTnet nuget package has tons of dependencies that I don't need (i.e. websockets, name resolution).
Related to

Client
ManagedClient
Server

Describe the solution you'd like
Split out several dependencies into separate projects, so that one can leverage composition and reduce dependencies.

split client/server
move websocket stuff out
move file system stuff out (message retention)

Hi, extracting the server from the core library is already planned for 3.1.x (but this might be not released released next summer).
I am wondering where the "tons" of dependencies are coming from. I had a look at MQTTnet.TestApp.NetCore which has, depending on the build target, 4, 7 or 14 external dependencies. Mostly these are .NET references. And the most are required when using net461. I added netcoreapp3.1 which "only" requires 7 libraries. These can be already reduced to 4 when MQTTnet.Extensions.WebSocket4Net.dll is not used.
I agree that having a hard dependency to Newtonsoft.Json.dll can also be omitted in the future by moving the storage to an official extension project.

Appreciate your response. Looking forward to see the improvements.
By "tons of dependencies" I meant not only the top-level dependencies (these are indeed only a few), but also the sub-level.
For example, System.Net.WebSockets.Client brings in quite a few more dependencies which are likely never required in my particular project.
To shed some light: I want to receive messages, do some simple transformations on them, and pass them on via System.Threading.Channels. That's it.

Closing this for now because this change is planned but this will not happen soon.


setting dapr http-max-request-size in the tye yaml

Hi There
We are trying to get dapr to allow us to upload files to blob storage but there seems to be a default limit of 4mb
Now I have looked through the docs and it seems this can increased to 16mb by adding --dapr-http-max-request-size: "16" to the dapr run command I have tried to add it to the extensions but this does not seem to have any effect
Is there a way I can modify the dapr run command to include the extra argument
Thanks Christian

Here are the things I have tried, to no avail.
Using Tye with the DAPR extension, .NET6 Web API. We are trying to set the max to 2Gb.
extensions:
 - name: dapr
   http-max-request-size: 2000
   log-level: debug
   components-path: "./dapr/"

This populates at the top of the log from Tye UI
C:/dapr/dapr.exe run --app-id <truncated> --dapr-http-max-request-size 2000 --log-level debug
We have used the kestrel documentation to set the max size on the web api side. No matter what we try this is always the result.
Microsoft.AspNetCore.Server.Kestrel.Core.BadHttpRequestException: Request body too large. The max request body size is 30000000 bytes. or 30mb

@Ben-Carpenter, thanks, your code helps:
extensions:
 - name: dapr
   http-max-request-size: 2000


implement local rate_limit

Changes

🎁 Implement a local rate limit

/kind enhancement

Release Note

This will help to implement a local rate limit within envoy so we can protect the ingress from being unavailable to clients in. given time by accepting a limited ammount of requests per second
action required

Docs

[Other doc]: https://github.com/RamyChaabane/net-kourier/blob/local_rate_limit/docs/local_rate_limit.md

@RamyChaabane gentle ping

need some unit tests


test: update golden files for new release

Thanks ole I have to add an ignore or something for this label


the visualizer folder parameter has to end with "/"

[chengli@fiji11 pyramid-0.1.0]$ python visualization/visualizer.py /huge1/people/chengli/projects/pyramid/archives/app3/ohsumed_20000/8/reports/train_reports
Traceback (most recent call last):
File "visualization/visualizer.py", line 1624, in
main()
File "visualization/visualizer.py", line 1602, in main
f1 = open(directoryName + configName, 'r')
IOError: [Errno 2] No such file or directory: '/huge1/people/chengli/projects/pyramid/archives/app3/ohsumed_20000/8/reports/data_config.json'
[chengli@fiji11 pyramid-0.1.0]$ python visualization/visualizer.py /huge1/people/chengli/projects/pyramid/archives/app3/ohsumed_20000/8/reports/train_reports/
Json:/huge1/people/chengli/projects/pyramid/archives/app3/ohsumed_20000/8/reports/train_reports/report_1.json load successfully.

Solved?


Cleanup old compacted topic ledgers when a new one is available

When a broker is notified of a new compacted ledger for a topic, and
after it has been persisted to the compaction cursor, delete the old
compacted topic ledger. The deletion is best effort. If it fails it
doesn't affect reading from the newer ledger.
This isn't foolproof by any means. There's still plenty of
opportunities for ledgers to be orphaned. We'll need some sort of GC
based on ledger metadata to handle all cases, but that needs a newer
bookkeeper (for the metadata properties changes).

retest this please // seem to be flakes


Fix api to accept list of address spaces

What is being addressed
Fixes #2915
Fix api app to handle array of address spaces

/test-extended

/test-extended-aad

/test-extended-aad

/test-force-approve


Validate the schema of the files

Currently the only implemented validation of the files that are being opened is checking if the files are SQLite databases. The schema of those files should also be checked.

The app will be completely rewritten as a web app, thus all opened issues are being closed.


tests: Use freezegun for time mocking to fix pypy3 compatibility

Description of the changes being introduced by the pull request:
Use freezegun for time mocking instead of manually patching the datetime module, as it provides a more streamlined solution that works both on CPython and on PyPy.  Unfortunately, due to differences between the C datetime extension used by CPython, and the pure Python version of datetime (used by PyPy, and as a fallback on CPython), there does not seem to be a trivial way to mock time that would work with both versions.
Fixes #2708

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

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 96.805%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
1554

Relevant Lines:
1591

💛  - Coveralls


Properly remove all deprecations

This PR supersedes #220 and #222 by applying both fixes at once
Fixes #219
Fixes #221

any ETA on this ?

@Roms1383 I am pretty sure this will not get merged any time soon.
I also did not just want to steal the work from the other two PRs.
The reason for this PR is the following:
You can now use this action (and cargo) without deprecations by using a workflow similar to this:
on: [push]

name: build

jobs:
  check:
    name: Rust project
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - name: Install latest nightly
        uses: ThexXTURBOXx/toolchain@master
        with:
            toolchain: nightly
            override: true
            components: rustfmt, clippy

      # `cargo check` command here will use installed `nightly`
      # as it is set as an "override" for current directory

      - name: Run cargo check
        uses: https://github.com/richb-hanover/cargo@master
        with:
          command: check

any ETA on this ?

I do not believe this will get merge anytime soon - maybe not at all. :(
In the meantime, users of this action may consider using dtolnay/rust-toolchain instead. It is maintained and does not cause any deprecation warnings. Usage is similar to actions-rs/toolchain, so it can be used as a replacement:
name: test suite
on: [push, pull_request]

jobs:
  test:
    name: cargo test
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: dtolnay/rust-toolchain@master
        with:
          toolchain: stable
          components: clippy, rustfmt
      - run: cargo clippy
      - run: cargo fmt
      - run: cargo test --all-features


Add models

Looks good. Merging changes for disjoiner, disjoiner messaging, directory structure changes, etc.


Annotation is not working properly

HI,
I am checking cornerstoneTool package in local and tested examples. In Annotation example, it create first annotation with text displaying below the line. But when try to create another annotation and text entered is not displayed for new annotation draw, instead it update the value of first annotation line. Please guide me to solve this problem.
Thanks.

Hello @mpkumar87, please include a screenshot and code example? Bonus points for an example on codepen that demonstrates the issue.

Thanks for ur reply.. Example which is in github also not working for annotation.
https://rawgit.com/cornerstonejs/cornerstoneTools/master/examples/annotation/index.html
First annotation mark display value which i entered in dialog box. But further annotation mark not displaying value, it is getting update first annotation mark.
Thanks.

Looks like you're right. I wonder when that broke...

I have created a PR to fix this issue
https://github.com/cornerstonejs/cornerstoneTools/pull/557
@mpkumar87 Hey, I have analysed the issue and it was actually an issue only with example code that was not removing correctly the dialog events, so it was being called as many times as we have arrowAnnotations in the app.

Helle swederik,
First of all thanks for your great effort for medical software(Dicom Viewer). I configured your latest versions of  cornerstone library. cornerstone-wado-image-loader - 2.0.0, cornerstone-tools - 2.3.3,
Here i seen annotation is not working accurately. It is almost difference of hundreds in length. But when i tried with yous oldest library(Augast 2017 versions). It is working very fine. Let me know what is the bug in your latest library. It is working good for almost all dicom files. But there is some dicom on which not working fine. I have attached the Example dicom file of DX modality. Check and confirm..
DicomAnnotationIssue.zip

@param88singh can you create a new issue with this information? We try to avoid re-opening issues that have merged work.


Update hyperlink to avoid html-proofer error

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

An external resource was moved, see diff & https://travis-ci.org/github/opensource.guide/builds/375057138#L1325

You're welcome :-)


Missing LICENSE file

Hi Stephen,
this is really great!  Good job!
Is this a solution that has a License allowing us to use it?
Thanks

fixed with https://github.com/DigitalState/camunda-json-logging/commit/9aef4374e6b5798cfe582bac565ce70075d4e8d7


No ForeignKeyAction exists for the onUpdate trigger

I run:
flutter pub run build_runner build --delete-conflicting-outputs
and I get it with the latest Flutter, analyzer and build_runner:
No ForeignKeyAction with the value ForeignKeyAction (_name = String ('noAction'); index = int (0)) exists for the onUpdate trigger.
@TypeConverters([TrackingServiceTypeConverter])
@Entity(
  primaryKeys: ['key', 'serviceType'],
  foreignKeys: [
    ForeignKey(
      entity: TrackingServiceInfo,
      childColumns: ['serviceType'],
      parentColumns: ['type'],
      onDelete: ForeignKeyAction.cascade,
    ),
  ],
)
class AuthDataField {
  final String key;

  final String value;

  final TrackingServiceType serviceType;

  const AuthDataField({
    required this.key,
    required this.value,
    required this.serviceType,
  });
}

@vitusortner
I tested on the develop branch and the error is the same.

Does this issue occur with the latest version of Floor? I tried to reproduce it with the libertrack project, it works fine.

@dkaera
I couldn't upgrade to new versions of analyzer and build_runner because of this.

@dkaera I couldn't upgrade to new versions of analyzer and build_runner because of this. Maybe this is fixed, it will be clear after the update of dependencies

ok, no problem
FYI: tried to run the code generation without the override analyzer in your project, all works fine for me, so probably it's time to migrate to the latest version 😁


not able to use the component

tried to run your basic example. got this error
⨯ node_modules\date-picker-np\dist\index.umd.js (1:255) @ eval
⨯ ReferenceError: document is not defined

could you explain more ?
What version are you using and are you using nextjs?

I've identified the issue, which occurs only in Next.js, while it works fine in React, and I'm currently working on a solution.

@Kuldip-Bhutoria the issue has been resolved in latest release
try using the version 1.0.15


NameError: uninitialized constant Formtastic::ActiveSupport

So I'm currently studying a rspec & surveyor project.
Whenever I try to compile my project, using bundle exec rspec, I always get this error.
Failure/Error: require "surveyor"

NameError:
  uninitialized constant Formtastic::ActiveSupport
# ./.bundle/ruby/2.3.0/gems/formtastic-2.1.1/lib/formtastic.rb:5:in `<module:Formtastic>'
# ./.bundle/ruby/2.3.0/gems/formtastic-2.1.1/lib/formtastic.rb:4:in `<top (required)>'
# ./.bundle/ruby/2.3.0/gems/surveyor-1.4.0/lib/surveyor.rb:10:in `require'
# ./.bundle/ruby/2.3.0/gems/surveyor-1.4.0/lib/surveyor.rb:10:in `<top (required)>'
# ./lib/spec_helper.rb:4:in `require'
# ./lib/spec_helper.rb:4:in `<top (required)>'

My binaries and dependencies seems to be up-to-date (I already look up some issues here), but still didn't fixed it. Here's my bundle show result:
λ bundle show
Gems included by the bundle:
  * actionmailer (<IP_ADDRESS>)
  * actionpack (<IP_ADDRESS>)
  * activemodel (<IP_ADDRESS>)
  * activerecord (<IP_ADDRESS>)
  * activeresource (<IP_ADDRESS>)
  * activesupport (<IP_ADDRESS>)
  * arel (3.0.3)
  * builder (3.0.4)
  * bundler (1.15.4)
  * diff-lcs (1.3)
  * erubis (2.7.0)
  * fastercsv (1.5.5)
  * ffi (1.9.18)
  * formtastic (2.1.1)
  * haml (4.0.7)
  * hike (1.2.3)
  * i18n (0.8.6)
  * journey (1.0.4)
  * json (1.8.6)
  * mail (2.5.5)
  * mime-types (1.25.1)
  * multi_json (1.12.2)
  * mustache (0.99.4)
  * polyglot (0.3.5)
  * rabl (0.6.14)
  * rack (1.4.7)
  * rack-cache (1.7.1)
  * rack-ssl (1.3.4)
  * rack-test (0.6.3)
  * rails (<IP_ADDRESS>)
  * railties (<IP_ADDRESS>)
  * rake (12.1.0)
  * rb-fsevent (0.10.2)
  * rb-inotify (0.9.10)
  * rdoc (3.12.2)
  * rspec (3.6.0)
  * rspec-core (3.6.0)
  * rspec-expectations (3.6.0)
  * rspec-mocks (3.6.0)
  * rspec-support (3.6.0)
  * sass (3.5.1)
  * sass-listen (4.0.0)
  * sprockets (2.2.3)
  * surveyor (1.4.0)
  * thor (0.20.0)
  * tilt (1.4.1)
  * treetop (1.4.15)
  * tzinfo (0.3.53)
  * uuidtools (2.1.5)

The issue is going away if I remove the require "surveyor" line, though I obviously can't do that. I'm attaching the  project here.
rspecsurveyor.zip
Thanks!

Did you find any fix for this? I also caught with related issue i.e. uninitialized constant Formtastic::SemanticFormBuilder (NameError).


Does not list attributes that will be updated in output

$ terraform show -json tfplan | tf-summarize
+--------+----------------------------------------------------------------------------------------------------------------------------------------------------------+
| CHANGE |                                                                           NAME                                                                           |
+--------+----------------------------------------------------------------------------------------------------------------------------------------------------------+
| update | uptycs_alert_rule.AWS_ApiGateway_Rest_API_active_tracing_is_disabled                                                                                |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_certificates_are_not_rotated_periodically                                                                      |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_does_not_have_CloudWatch_logging_enabled                                                                       |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_does_not_have_CloudWatch_metrics_enabled                                                                       |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_does_not_have_Web_Application_Firewall                                                                         |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_does_not_have_content_encoding_enabled                                                                         |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_ApiGateway_Rest_API_is_not_using_client_certificates                                                                               |
+        +----------------------------------------------------------------------------------------------------------------------------------------------------------+
|        | uptycs_alert_rule.AWS_CloudTrail_is_not_enabled_in_a_region                                                                                |

Ubuntu 22.04 LTS
$ tf-summarize -v
Version: 0.2.2

Could you give some more details ? @myoung34
What is the output of terraform plan ? (you can mask the sensitive details and share the data)

Based on my understanding of your question. I am answering below
This tool will only show the summary of the terraform plan (like what is going to get updated/created/deleted/recreated). If we want to see the details of it, we should see the terraform plan or terraform show tfplan
Example usage:
terraform plan -out=tfplan # this is will print the detailed output
terraform show -json tfplan | tf-summarize   # See the summary, if needed see the details above

It's a very large plan, but its a lot of this
  # uptycs_alert_rule.Wbadminexe_execution_detected__T1490__Inhibit_System_Recovery__Windows will be updated in-place
  ~ resource "uptycs_alert_rule" "Wbadminexe_execution_detected__T1490__Inhibit_System_Recovery__Windows" {
      ~ event_tags     = [
            # (6 unchanged elements hidden)
            "process_events",
          + "RSIC_DISABLED",
        ]
      ~ id             = "ab6b5431-c8c7-4e5a-810a-331298fb6f15" -> (known after apply)
        name           = "Wbadmin.exe execution detected - T1490 - Inhibit System Recovery - Windows"
        # (9 unchanged attributes hidden)
    }

  # uptycs_alert_rule.WinRarexe_execution_detected__T1560001__Archive_via_Utility__Windows will be updated in-place
  ~ resource "uptycs_alert_rule" "WinRarexe_execution_detected__T1560001__Archive_via_Utility__Windows" {
      ~ event_tags     = [
            # (7 unchanged elements hidden)
            "process_events",
          + "RSIC_DISABLED",
        ]
      ~ id             = "2d843ba5-d771-4c87-8ec5-3e82ddbaaff6" -> (known after apply)
        name           = "WinRar.exe execution detected - T1560.001 - Archive via Utility - Windows"
        # (9 unchanged attributes hidden)
    }

What is the actual output of the tf-summerize ? and what is the expected output ?

That is the output from tf-summarize.
I'd expect it to show .event_tags in the row to show what attribute is changing

Oh okay. Today, it only supports just the resource name. We can add that feature in upcoming releases.
Initial thought, tf-summarize -detailed should show the changing properties.

I guess I'm confused. The gif seems to show that it would list the attributes in the plan

Okay cool. I will close the issue then. Thanks for trying out the look @myoung34


More debug logs

See https://vike.dev/debug#vite. We can implement more debug logs.

Done in vps 0.4.
https://vite-plugin-ssr.com/debug.
Add a comment here if you want more debug logs.

Massively appreciate the debug logs here, it'd be great to have timestamps on these, especially when using the stream debug log, as otherwise its very hard to tell the timing between the different flushes and where to optimize the code for a faster pageview.

Btw thank you for all the great work here, this + react-streaming are top notch :) We're looking at moving all of mixcloud.com over from our own old SSR systems to this, lots of testing/spikes going on right now, so sorry if I keep popping up with random questions. Our only gripe is we have our own custom router, so the interaction is a little weird between the two and took us a while to understand.

@matclayton Feel free to PM me. Is sponsoring on the table? You can then expect a very fast turnaround time ;-). Btw. mixcloud.com looks real neat. Big fan of democratizing access to financial sustainability.

Cool will DM
On Fri, 16 Dec 2022 at 14:13, Rom @.***> wrote:

@matclayton https://github.com/matclayton Feel free to PM me. Is
sponsoring on the table? You can then expect a very fast turnaround time
;-). Btw. mixcloud.com looks real neat. Big fan of democratizing access
to financial sustainability.
—
Reply to this email directly, view it on GitHub
https://github.com/brillout/vite-plugin-ssr/issues/249#issuecomment-1354911628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA62CURZN2BHPEBQM7LEGLWNR2G5ANCNFSM5M6EHVHQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Matthew Clayton | Co-Founder
Mixcloud Limited
mixcloud http://www.mixcloud.com/mat
twitter http://www.twitter.com/matclayton
email @.***
mobile +44<PHONE_NUMBER>

@sqs That would be great indeed, also telling when and what sets which pageContext value.
We will be focusing on such extra polishing features after the v1.0.0 release.


NOISSUE - Separate docker-push into function

Signed-off-by: drasko<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #386   +/-   ##
=======================================
  Coverage   89.14%   89.14%
=======================================
  Files          44       44
  Lines        1539     1539
=======================================
  Hits         1372     1372
  Misses        115      115
  Partials       52       52

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 845b69a...0266d0b. Read the comment docs.


Resolution L2

Related story
https://www.pivotaltracker.com/story/show/179325573
Changes

Moved old UNS implementation to UNSInternal which is now accessible only from the package
Added L2Resolver to implement concurrent resolution on two chains (see related story for resolution logic)
Created new UNS NamingService that uses the L2Resolver with two UNSInternal for each layer
Refactored Resolution, extracted Resolution.Builder into a separate class (the API to use the builder remains the same)
Updated Resolution.Builder to accept new L2 parameters
Added test cases for L2 resolution
Added tests for L2Resolver

I've added a temp RPC for unit tests that supports reading events from the whole chain just to make tests green for now. It should be changed once we figure out which RPC to use.

@DancingAxolotl I've created layer-two branch for keeping all L2 related updates there until release and updated PR base. Let't create all L2 pr's against layer-two branch instead of master.


Consider moving the Android Tracker into the snowplow-java-tracker repo

Consider: https://github.com/keenlabs/KeenClient-Java

Descheduling; we will come back to this much later...

These projects have diverged substantially, so closing.


Queue timeout doesn't respect delay

Laravel Version: 5.8.15
PHP Version: 7.3
Queue driver: Redis (Horizon)

Description:
We came across an issue today where we were staggering the delay of queue jobs into 5 min chunks over the course of 6 hours but any jobs that were delayed longer than the timeout failed instantly because the timeoutAt timestamp was before the delay time.
I was just wondering if this is the desired behavior? To me, it would make sense that the timeout should apply when the delayed job has been pulled from the queue to be run and not when it has been added to the queue. I.e. timeoutAt = timeout + delay.
Steps To Reproduce:
Here is a code sample:
// Split jobs into 5 min chunks over 6 hours
$chunkSize = (int) ceil(Model::count() / 72);

Model::chunk($chunkSize, function ($models, $page) {
    $delay = now()->addMinutes(($page - 1) * 5);

    foreach ($models as $model) {
        Job::dispatch($model)->delay($delay)->onQueue('my-queue');
    }
});

Is this also a problem in the latest release?

I've just tested this on a fresh Laravel install running Horizon and couldn't recreate the issue.

For the benefit of others, this turned out to be an issue related to job funneling and setting retryUntil to null which made the job fall back to attempts.


feat: add chunk split plugin in webpack-builder

PR Details
增加 webpack-builder 拆包功能。
Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

🎉🎉


Notification after a long running command finishes

Description of the new feature/enhancement

It would be awesome to add an option for a notification bubble or sound after a long running command. Long running command can be configurable but a sane default could be 10 seconds.
The scenario for this would to switch users from a pull to push model. Without notifications, a user would have to constantly "pull" their terminal to see if their command is done. Switching to push removes the cognitive overhead of having to constantly pull and instead be pushed only once the command is done running.
Current workarounds

Run a script that creates a notification after the command is done: long_command; notification.py

Extra asks
Beyond allowing for notifications, it would also be helpful if users were provided a hook. Here are some scenarios the hook can cover:

Hitting an API to report the run length of the command (Useful for engineering systems)
Changing the color of your smart LEDs (Think of all the things Makers could do!)
Context aware notifications (If the script name contains 'foo' then do then, otherwise do that

Other Terminal support

iTerm has this as a feature: https://www.iterm2.com/documentation-shell-integration.html#:~:text=Alert on next mark,in another window or tab.

Proposed technical implementation details (optional)

To try and tease this apart a bit:

#7718 is tracking the ability to send a notification from a commandline script. That's something that could easily be leveraged by notify.py (or whatever) to send a notification when the build finishes.
"Alert on next mark" is a really great idea. I think I want this thread to track that feature now.
#7955 is also tracking kinda the inverse of this thread.

This thread is "I want a notification after some time, when a command finishes"
That thread is "I want a notification if there's any output in an inactive tab"

I might want to spend a hot minute syncing these two designs with each other - we definitely don't want to send a notification for BOTH "there was new output" and "that output was the next mark". I suppose if both settings are enabled, we can defer to "there was output".


Remove MQ test from Safe health check

Changes
Fixes #257 and stops the safe health check adding a message to the test queue. As it doesn't send a message to the queue it also doesn't need to use the health check library and a static url should suffice
How to test

Check you can access /status
Check that it is not sending a message to the queue

Who can test
Anyone apart from @warrenbailey

LGTM 👍

Ready to merge - will rebase and merge once Travis completes tests.

LGTM 👍 - will merge once tests pass.


After active EL2, PCIe devices load failed

After active EL2, PCIe devices load failed
Known include WIFI, NVMe, GPU.
Directly boot from NVMe is impossible, so please use USB Disk.
dmesg
lspci -vv

Hi, Linux is generally out of scope for this repo, as mentioned in the readme.
One needs to fix Linux to be able to use pcie on sc8280xp; you can check @jglathe 's work though: https://github.com/jglathe/linux_ms_dev_kit/tree/jg/el2-hackery

It seems that the dtbhack just temporarily loads the dtb into the table, is there any permanent way to do it?

Have better firmware :) I.e. Lenovo provides "Linux mode" on x13s that loads the dtb on boot.
dtbhack is, as the name suggests, a hack. Long term it shouldn't exist and will probably be gone.

The jg/el2-iommu-v6.8 branch has video, too. Typing on this box rn.


Update my SSH key length to 4096 bits

As requested by @mattbostock in https://github.com/alphagov/govuk-puppet/pull/4194.

Thanks @tvararu!


Need help (again) to set up the youtube module

It's me again....
Before going further, I can see a small box (few pixels) in place of youtube videos but not thumnails.
Then I found the documentation here and I'm trying to add the js and scss files to the pipeline.
None of the following worked :

copy the files in assets/js/ and assets/scss and adding the 'import' and '@import' lines to my custom.js and custom.scss files.
adding mounts to hugo.toml under the module.imports :

[module]
  [[module.imports]]
    path = "github.com/davidsneighbour/hugo-modules/modules/youtube"
    disable = false
    [[module.imports.mounts]]
    source = "assets/"
    target = "assets/js/lite-yt-embed.js"
    [[module.imports.mounts]]
    source = "assets/scss/"
    target = "assets/scss/_lite-yt-embed.scss"

(Here I tried several things like source = "assets/js/ or target = assets/)
hugo server gives the following error :
TOCSS: failed to transform "/scss/custom.scss" (text/x-scss): "/home/ksdrummer/mywebsite/assets/scss/_lite-yt-embed.scss:53:11": The target selector was not found.
Use "@extend .object-fit-cover !optional" to avoid this error.
Built in 335 ms
Error: error building site: TOCSS: failed to transform "/scss/custom.scss" (text/x-scss): "/home/ksdrummer/mywebsite/assets/scss/_lite-yt-embed.scss:53:11": The target selector was not found.
Use "@extend .object-fit-cover !optional" to avoid this error.

the ! optionnal allows me to  build the site but I am in the same situation than before : small boxes in place of the youtube videos.
Could you help me out ? I'm really new to hugo moduling ...
And again, thank you so much for the work and for your time !

Here again, don't copy the configuration of the module. Just add it to your configuration:
[module]
  [[module.imports]]
    path = "github.com/davidsneighbour/hugo-modules/modules/youtube"

After that, when you run hugo or hugo server, the mounts are available in your assets folder.
Have a look at the way I am using them in my own website kollitsch.dev:

The stylesheet is imported into my sass pipeline here
The javascript is added here and initialised a line later.

Those "mounts" are only available when Hugo is running.
There are other mounts available that go into the static folder, so that you could add them via <script> and <link rel="stylesheet"> tags at /libs/liteyoutube/lite-yt-embed.css and /libs/liteyoutube/lite-yt-embed.js. I wouldn't do this though, because that leads to two more files being downloaded and you might loose the speed you gained again.

Let me add, the "paths" of these mounts are:

assets/js/lite-yt-embed.js
assets/scss/_lite-yt-embed.scss

and you have to use a relative path there, depending on where you import them.

Thank for your answer. But this is the first thing I tried... And I get the following error :
$ hugo server
Watching for changes in /home/ksdrummer/mywebsite/{archetypes,assets,content,data,i18n,layouts,static,themes}
Watching for config changes in /home/ksdrummer/mywebsite/config/_default, /home/ksdrummer/mywebsite/themes/hugo-coder/hugo.toml, /home/ksdrummer/mywebsite/go.mod
Start building sites …
hugo v0.134.3+extended linux/amd64 BuildDate=unknown

ERROR TOCSS: failed to transform "/scss/custom.scss" (text/x-scss): "/home/ksdrummer/.cache/hugo_cache/modules/filecache/modules/pkg/mod/github.com/davidsneighbour/hugo-modules/modules/youtube@v1.2024.6/assets/scss/_lite-yt-embed.scss:53:11": The target selector was not found.
Use "@extend .object-fit-cover !optional" to avoid this error.
Built in 294 ms
Error: error building site: TOCSS: failed to transform "/scss/custom.scss" (text/x-scss): "/home/ksdrummer/.cache/hugo_cache/modules/filecache/modules/pkg/mod/github.com/davidsneighbour/hugo-modules/modules/youtube@v1.2024.6/assets/scss/_lite-yt-embed.scss:53:11": The target selector was not found.
Use "@extend .object-fit-cover !optional" to avoid this error.

To be sure i didn't miss anything (I probably did) :

I added the module configuration in hugo.toml
I added the custom js and custom scss to hugo.toml and the following lines to layouts/Head/theme-styles.html (I already add a custom scss but no js before)

<!-- CSS STYLES -->
{{ range .Site.Params.custom_css -}}
    <link rel="stylesheet" href="{{ . | absURL }}">
{{- end }}
<!-- JS STYLES -->
{{ range .Site.Params.custom_js -}}
    <link rel="stylesheet" href="{{ . | absURL }}">
{{- end }}

I added @import "lite-yt-embed"; to assets/scss/custom.scss and

import LiteYTEmbed from "./lite-yt-embed.js";
customElements.define("lite-youtube", LiteYTEmbed);

to assets/js/custom.js
But the site won't build...

I see this in your module.toml
[[mounts]]
source = "assets"
target = "assets"

[[mounts]]
source = "hugo_stats.json"
target = "assets/watching/hugo_stats.json"

[[mounts]]
source = "i18n"
target = "i18n"

[[mounts]]
source = "layouts"
target = "layouts"

[[mounts]]
source = "static"
target = "static"

Tried to add this to my hugo.toml but same error... Might be something like this though

Oh I see... there is stuff in that SCSS file that comes from Bootstrap 5.
I will fix that today or tomorrow. To quickly fix it in your website copy the full stylesheet from here locally and replace that one line. Then include YOUR copy instead of the mounted one into your stylesheet.
Old line:
@extend .object-fit-cover;

Replace with
object-fit: cover;

The reason for this is that I am using BS5 everywhere... That should be taken out of the modules.

Thank you !
So  now the thumbnails are loading but nothing happens when I click on the videos (just youtube button stays red but that's it).

Open your browser developer console. Check if any error comes up in there. Sounds like SCSS is all fixed and Javscript might not be initialized yet. You need to initialize the component and not just import it.
// YouTube element
// @ts-ignore - this is loaded at runtime by GoHugo
import LiteYTEmbed from "./lite-yt-embed.js";

// Initiate custom elements
customElements.define("lite-youtube", LiteYTEmbed);

also the way your js is built is important. I have mine in a js.build pipeline.

Then include YOUR copy instead of the mounted one into your stylesheet.

But I don't know how to do this...

How is your assets/js/custom.js used in your layouts?

Not sure what you mean but I added customJS = ["js/custom.js"] to hugo.toml and
{{ range .Site.Params.custom_js -}}
    <link rel="stylesheet" href="{{ . | absURL }}">
{{- end }}

to layouts/partials/Head/thele-styles.html

I also have
{{ if hugo.IsServer }}
  {{ $cssOpts := (dict "targetPath" "css/coder.css" "enableSourceMap" true ) }}
  {{ $styles := resources.Get "scss/coder.scss" | resources.ExecuteAsTemplate "style.coder.css" . | toCSS $cssOpts }}
  <link rel="stylesheet" href="{{ $styles.RelPermalink }}" media="screen">
{{ else }}
  {{ $cssOpts := (dict "targetPath" "css/coder.css" ) }}
  {{ $styles := resources.Get "scss/coder.scss" | resources.ExecuteAsTemplate "style.coder.css" . | toCSS $cssOpts | minify | fingerprint }}
  <link rel="stylesheet" href="{{ $styles.RelPermalink }}" integrity="{{ $styles.Data.Integrity }}" crossorigin="anonymous" media="screen" />
{{ end }}

that I copied from the theme I'm using (hugo-coder)

from the console I have Uncaught SyntaxError: import declarations may only appear at top level of a module which seems to be it

Not sure what you mean but I added customJS = ["js/custom.js"] to hugo.toml and
{{ range .Site.Params.custom_js -}}
    <link rel="stylesheet" href="{{ . | absURL }}">
{{- end }}

to layouts/partials/Head/theme-styles.html

You are ranging JS files and put them into a stylesheet tag... That can't work. Let's go the other route. Remove what you added for my module in js/custom.js and then add the static file into your configuration instead:
customJS = ["/libs/liteyoutube/lite-yt-embed.js"]

Please don't remove the slash in the beginning. This file is mounted into static and will be copied into the website at runtime/build.

With this I can't build the site :
ERROR render of "page" failed: "/home/ksdrummer/mywebsite/themes/hugo-coder/layouts/_default/baseof.html:51:25": execute of template failed: template: _default/single.html:51:25: executing "_default/single.html" at <$script.RelPermalink>: nil pointer evaluating resource.Resource.RelPermalink
ERROR render: failed to render pages: render of "404" failed: "/home/ksdrummer/mywebsite/themes/hugo-coder/layouts/_default/baseof.html:51:25": execute of template failed: template: 404.html:51:25: executing "404.html" at <$script.RelPermalink>: nil pointer evaluating resource.Resource.RelPermalink

I tried to use js/lite-yt-embed.js from here as custom js but I still have the Uncaught SyntaxError: import declarations may only appear at top level of a module error from the console.

The website is hosted on Gitlab. Do you have an account ?

I uploaded the code to github and invited you to as contributor. I think something must interfere with the theme I'm using, which is also using modules...

I joined... give me a day or two to work on this. It's 9pm here; only daylight will bring good fixes ;) It's probably an issue with the order of modules (who overrides whom) and configs.

We are in the same timezone =)
There is no rush, the site is online and working, I can deal with the video page being slow for now.
Thank you so much for taking the time to help me with this. I would have given up a long time ago, I'm not a pro, just learned hugo few weeks ago because I needed a website.

I accidentally commited my changes to your main branch... If you need to remove that just run
git revert 42b65f6b73a819b12c8c453920895ec4e4593868

And it will create a commit undoing my changes.
I did the following:

removed custom.js content (that is code that needs to be compiled via js.Build. instead I included the direct already compiled script in footer.html)
removed custom scss and fixed the module.

You could delete the following files:

assets/js/custom.js and its configuration if ou don't have any other use for it
layouts/shortcodes/youtube-lite.html - not sure if that is related or another shortcode

The videos are loading now.

That was weird, it was not committed indeed. Sent it up now: https://github.com/ksdrummer/website/commit/2a9dc48fa10b401e0c385812022ee9070c85a46e

Yes ! It works perfectly now !
Again, thank you sooooooo much for your time and also for the module ! It's really appreciated !


Different loss function in ranking model in xgboost

I tried both rank:pairwise and rank:ndcg as loss function in XGboost, and found rank:pairwise is always better than rank:ndcg. rank:ndcg is based on lambdaMART which should have better performance than rank:pairwise. Can anyone  explain to me the difference between the two loss function?
thanks,
jd

rank:ndcg is based on LambdaRank.
rank:pairwise is based on LambdaMart.
In general LambdaMart performs better than LambdaRank.
In most cases, pairwise was working better than listwise approach, but it seems than with DL this is changing.
For more info:
https://www.microsoft.com/en-us/research/publication/from-ranknet-to-lambdarank-to-lambdamart-an-overview/


Cordova plugin market

Can you rebase this off of the current state of dev-next? and resubmit this PR?

There are branch conflicts; so you'll have to rebase and try again.  If I did it I wouldn't be able to get it in until next release.


Unify qg

We make sure that QG are defined uniformly across services

hm can't assign anyone, can you take this @kvalliyurnatt ?


ANRs when loading Cronet Provider

Sharing some of the ANRs encountered when calling CronetProviderInstaller and when calling build method of CronetEngine.Builder and CronetInterceptor
There is one ANR happening due to System.loadLibrary being called on main thread by Play Services.
Is it safe to call the CronetProviderInstaller.installProvider() on a background thread?
anr_load_library.txt
anr1.txt
anr2.txt
anr3.txt

Hi @adityameesho,
Initializing Cronet can be fairly expensive as it involves loading a non-trivial amount of Java code as well as the native library (which requires its own initialization as well). For this reason, it might indeed be a good idea to do it in a background thread to avoid blocking the UI on the loading process, which could potentially result in ANRs especially if the device is slow or starved of resources (e.g. during app startup).


@bentley/imodeljs-native 2.11.3

@evanpreslar-bentley one test is failing.
  ExtensionAdmin tests
    1) loads local extension
Error: iModel.js Shared Library @bentley/imodeljs-frontend is loaded, but is an incompatible version.
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:133:13
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:135:5)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/src/common/extensions/loadingTestExtension/loadingTestExtension.ts:6:1)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:91:18)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at http://localhost:6010/webpack/bootstrap:83:1
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:87:10

@evanpreslar-bentley one test is failing.
  ExtensionAdmin tests
    1) loads local extension
Error: iModel.js Shared Library @bentley/imodeljs-frontend is loaded, but is an incompatible version.
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:133:13
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:135:5)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/src/common/extensions/loadingTestExtension/loadingTestExtension.ts:6:1)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:91:18)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at http://localhost:6010/webpack/bootstrap:83:1
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:87:10

Closing in hopes that reopening will trigger a build...

Closing in hopes that reopening will trigger a build...

@evanpreslar-bentley one test is failing.
  ExtensionAdmin tests
    1) loads local extension
Error: iModel.js Shared Library @bentley/imodeljs-frontend is loaded, but is an incompatible version.
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:133:13
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:135:5)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/src/common/extensions/loadingTestExtension/loadingTestExtension.ts:6:1)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:91:18)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at http://localhost:6010/webpack/bootstrap:83:1
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:87:10

Just to round out the discussion, we discussed the issue offline and @pmconne is side-stepping it for now by not doing a full rush update in this PR and only updating the pnpm-lock.yaml file for the new native package version.
Separately, we will be working on a more general fix to the way version comparison is handled for Extensions in a separate PR that will resolve this issue and a few others.  That fix should be ready before the 2.11.0 release.

@evanpreslar-bentley one test is failing.
  ExtensionAdmin tests
    1) loads local extension
Error: iModel.js Shared Library @bentley/imodeljs-frontend is loaded, but is an incompatible version.
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:133:13
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:135:5)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/src/common/extensions/loadingTestExtension/loadingTestExtension.ts:6:1)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at Object.<anonymous> (http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:91:18)
    at __webpack_require__ (http://localhost:6010/webpack/bootstrap:19:1)
    at http://localhost:6010/webpack/bootstrap:83:1
    at http://localhost:6010/imjs_extensions/loadingTestExtension/index.js:87:10

Just to round out the discussion, we discussed the issue offline and @pmconne is side-stepping it for now by not doing a full rush update in this PR and only updating the pnpm-lock.yaml file for the new native package version.
Separately, we will be working on a more general fix to the way version comparison is handled for Extensions in a separate PR that will resolve this issue and a few others.  That fix should be ready before the 2.11.0 release.


Remove Beta label from Office365 SSO

https://github.com/mattermost/mattermost-webapp/pull/3072
PS: Sorry, I accidentally submitted against the master branch..

Can you help rebase to v5.14-documentation branch?

Resubmitted against 5.14: https://github.com/mattermost/docs/pull/2887


No RoundAnnounceMatchStarted event in one of my demos, and empty AdditionaInformations for players

Hello, I've been using this library to parse GOTV demos from 10mans me and my friends play on our own server. For the most part everything has worked fine with the exception of 1 match. All other demos always get the RoundAnnounceMatchStarted event and I use this to reset tracking information and mark when i need to start watching for other events. I tried getting around it missing by subscribing to just the MatchStarted event, but noticed after the demo is parsed, all players AdditionaInformations is 0 except for Ping. Any ideas why this would be the case and how I could avoid this being an issue in our other demos?
Demo File
Thanks for any help.

So although I'm not gettting the RoundAnnounceMatchStarted event, I can circumvent this by subscribing to MatchStarted, and the reason the stats were empty at the end of parsing is due to another MatchStarted event being triggered and re-clearing everything. I tried re-working my code to only account for a single start-to-finish match, but can't seem to find a good event to subscribe to on match end except for WinPanelMatch which doesn't seem to get called.

Realized that this match never had an actual end event as we force ended it early, need to work around this in my own code :)

Welcome to working with demos :)


Patch #151

Current coverage is 58.18%

Merging #186 into master will not affect coverage as of 0cf4ec0

@@            master    #186   diff @@
======================================
  Files           32      32
  Stmts          617     617
  Branches         0       0
  Methods         89      89
======================================
  Hit            359     359
  Partial          0       0
  Missed         258     258

Review entire Coverage Diff as of 0cf4ec0

Uncovered Suggestions

+1.94% via ...enViewController.php#124...135
+1.78% via ...ackendController.php#243...253
+1.62% via .../CloudController.php#33...42
See 7 more...

Powered by Codecov. Updated on successful CI builds.


add ctranslate2 models

Inspired by Hamel's benchmarks on fast transfomer inference, I propose adding the ctranslate2 library.
This almost works, but fails for multiple samples. The way to sample multiple sequences natively in ctranslate2 is to use
generator = ctranslate2.Generator("redpj", device="cuda")
ans = generator.generate_batch([tokens], beam_size=4, num_hypothesis=4)

First compile the HF model:
ct2-transformers-converter --model togethercomputer/RedPajama-INCITE-Instruct-3B-v1 --output_dir redpj --quantization bfloat16

This code works:
import outlines.models as models
model = models.ctranslate2("redpj", "togethercomputer/RedPajama-INCITE-Instruct-3B-v1", "cuda")

prompt = "Hey What's up? In the"
output = text.generate.continuation(model, max_tokens=250)(prompt)
print(output)

But this fails:
output = text.generate.continuation(model, max_tokens=250)(prompt, samples=5)

This is really cool, thank you for opening a PR. The slowness must come from the fact that we don't use the KV cache (to be fixed soon). Is there any way we can have the library return it at some point?

Ctranslate2 supports kv-caching in model.generate but doesn't look straightforward via the model.forward.

We can perhaps close this PR, or draft and park it.

If this works but the only issue is the KV cache we can still merge it once the style issues have been fixed.

Unable to understand how to fix the pre-commit error on github actions. On my local pre-commit passes completely.
GH Actions error:
outlines/models/ctranslate2.py:9: error: Cannot find implementation or library stub for module named "ctranslate2"  [import]

I also think that we should support MLC as more priority than ctranslate2, maybe because of more gh-stars or in general community support of MLC. See: https://github.com/mlc-ai/mlc-llm

I also think that we should support MLC as more priority than ctranslate2, maybe because of more gh-stars or in general community support of MLC.

Makes sense, but it is important that we merge this PR before opening any other one.


Cannot find module 'react-firebase-hooks' or its corresponding type declarations

I can't import the module neither in a CRA app nor in an empty TypeScript project. Steps to reproduce:
mkdir test-react-firebase-hooks
cd test-react-firebase-hooks
npm init -y
npm i -D typescript
npx tsc --init
npm i react-firebase-hooks
echo "import {} from 'react-firebase-hooks'" > index.ts
npx tsc

You shouldn't be importing from the root, it should be one of either react-firebase-hooks/auth, react-firebase-hooks/firestore, react-firebase-hooks/storage, or react-firebase-hooks/database.

@osilviotti is correct, in order to minimise application bundle size, you should import from the packages that you're interested in using.


Game lags second level.  Unplayable lag.

Hey So I got this mod installed it the game client works perfect and everything is perfect for the first stage and then going into the second stage the game client begins to lag and so badly so that you can't loot move or look in any direction. I minimize the game client to use deck top and all other apps and programs are running fine no temp issues with HDD, GPU, CPU. Just seems to be an issue with the add-on for sure. Any recommendations?

I have that problem too, my solution is that I downloaded Dnspy and use it to open Assamble-Csharp.dll and use the "Do it yourself" part codes to replace original codes. And then It works perfectly.


Img to Img crashing is regular and low Vram mode

This was using Version 978
Tried using Img to Img and will work for a few images but will then give the VK_ERROR_OUT_OF_DEVICE_MEMORY; error.
-Tried deleting VMFB, came back with same issue
-Tried Low VRAM with fresh VMFB and gave the issue in the .txt
Img to Img regular.txt
Img to Img low VRAM.txt

Same here, running a 5700XT, what video card are you on? Did they update torch?

Rx 590 Fatboy

Hey, it might help to use the new --device_allocator_heap_key option when launching the .exe.
if you have a .bat file wrapping the webui launch, or if invoking from the command line, you can try adding:
--device_allocator_heap_key='*;1gib'
Not sure if this will work with that exact key but essentially the memory allocations should be somewhat configurable.
Typically the low VRAM option would be the way to go for those cards -- I would make sure to use this option in any case, the error report above isn't giving much to go off of but it seems less like a memory issue and more like a compilaiton issue (which is surprising since the first error seems to show a successfully compiled unet failing at allocation time -- the same compiled .vmfb should be re-used when running with Low VRAM checked.


Task: Audit performance of About page

Describe the task
Audit performance of Clean and Green Philly's About This Project page using Google Lighthouse. Document audit results, including areas of improvement.
Acceptance Criteria

[ ] Generate Lighthouse report for mobile devices
[ ] Generate Lighthouse report for desktop devices
[ ] Post screenshots of results in this issue's comments section
[ ] Write brief summary of audit highlights, including Lighthouse's recommendations for improvement

Additional context

For instructions on using Lighthouse, see https://developer.chrome.com/docs/lighthouse/overview

Hi, can I be assigned this issue pls?

@zheyichn it's yours!

Lighthouse report on mobile devices

Performance score was 70/100. Score took a bit hit because the loading of the map image took 7.1 seconds.
Accessibility score was 95/100 due to <nav> element shoudn't have aria-passed attribute as mentioned in #727.
SEO score was 82/100. Penalty comes from links do not have descriptive text & links not crawlable.

Screenshoots

Lighthouse report for desktop devices

Performance score was 97/100. Large Contentful Paint took 1.3 seconds.
Accessibility score was 95/100 because of having aria-pressed attribute on <nav> elements.
SEO score was 82/100, since links do not have descriptive text & links being not crawlable.

Screenshoots

Summary

Performance got penalty from long image loading time, mianly for mobile devices.
Accessibility was good on both devices. #727 provided context for why this page did not get full score.
SEO suggests links to be crawlable and have descriptive text.

Improvement recommendations
For mobile devices

Explore ways to reduce the map image loading time. (Performance)
Make links crawlable to search engine and add descriptive text. (SEO)

For desktop devices

Make links crawlable to search engine and add descriptive text. (SEO)

@CodeWritingCow please add to your list to review


[Project] Add the Roslyn API reference to docs

The Roslyn APIs (aka the .NET Compiler Platform SDK) should be on docs.
See this project for sub-tasks.

Related issue on the Roslyn repo: https://github.com/dotnet/roslyn/issues/11557.

Somehow, I'm not surprised @svick wrote this some time ago.

The Roslyn APIs are now live on docs.
Thanks @dend and @meganbradley

Awesome! Next step: fill out documentation for the many members that don't have it from XML comments. 😄
Also, I can't find documentation for the Microsoft.CodeAnalysis.ISymbol interface. In fact, the whole Microsoft.CodeAnalysis namespace looks suspiciously empty. It seems the only type there is from the Microsoft.CodeAnalysis.CSharp assembly, but all types from the Microsoft.CodeAnalysis assembly are missing (at least for that namespace)

We can open issues for Roslyn API docs at https://github.com/dotnet/roslyn-api-docs/issues

@mairaw Didn't know about about that repo, thanks. Issue opened: https://github.com/dotnet/roslyn-api-docs/issues/18.

@svick it's new. 😄 Thanks!


Caching aspects

When working with pmtiles, especially remote, one may want to cache the directory access. I implemented it for pmtiles crate in https://github.com/stadiamaps/pmtiles-rs/pull/24 - you may want to add this capability too.
Basic idea -- the find_entry_rec recursive directory entry searcher always has a cache instance, and it tries to get a cached directory's entry.  That instance could be a noop (no caching, always returns cache miss), in which case it downloads directory and finds an entry in it before passing it off to cache.
This approach allows multiple pmtiles instances to share a common cache.  One extra field I added today was an ability to get the size of a cached directory - this way cache could tell how big it is and evict appropriately.

Thanks for the idea, but that wouldn't quite work the way this crate is currently implemented. Here, the directories are always to read in its entirety when opening the archive. The trade-off I'm opting for is that the initial "opening" of the PMTiles archive is more expensive, but retrieving the contents of an individual tile is less complicated as it is already known exactly where each tile is located in the archive. See the following code:
https://github.com/arma-place/pmtiles-rs/blob/a5e7f94e21830cf457a18b6f7925ff51edc9c57b/src/pmtiles.rs#L267-L283
There is a way to load only a specific range of tiles when opening an archive (which may allow skipping entire leaf directories when parsing), but this range cannot be extended at a later point. That case is only really helpful, if only a single tile / a couple of tiles are needed.
I'm not saying that is the best way to go, but just that's how it is implemented currently.


model.bc

I run the build_model.py and generated the model.bc file. How can I parse model.bc?

Please open a new thread in https://discuss.tvm.ai/ with more detailed descriptions.


$HOME with apostrophe fails to create local user

I created a new user with no apostrophe and it all works fine. Here is the log of the fail, followed by the $HOME directory name.
PS C:\WINDOWS\system32> lxrun /setdefaultuser roconnell
Enter new UNIX password:
Retype new UNIX password:
passwd: password updated successfully
/bin/bash: -c: line 0: unexpected EOF while looking for matching `''
/bin/bash: -c: line 1: syntax error: unexpected end of file
Creating UNIX user failed, this can be done later by running lxrun.exe /setdefaultuser
PS C:\WINDOWS\system32> dir c:\users
Directory: C:\users

Mode                LastWriteTime         Length Name

d-----        6/26/2016   8:56 PM                Default.migrated
d-r---        6/26/2016   8:54 PM                Public
d-----        6/28/2016   6:07 PM                Rob O'Connell

Looks like lxrun is not escaping the special-characters, ` is used in bash to redirect the output of one command to argv of another.

Thanks for reporting this.  You are correct, it looks like we need to escape ' and " characters when translating NT file paths to DriveFs.

If path being quoted with double quote, be aware of $ and `.

http://tldp.org/LDP/abs/html/special-chars.html should be a more or less complete overview over special-characters in bash

Since it seems bash works ok even if lxrun /setdefaultuser fails, would it be possible to describe exactly what the command is doing?
This could allow a possible workaround by just setting the default user manually.

Is there any progress on this?

Looking forward to a fix.

Sorry for the delay on fixing this.  I just now submitted a fix for this and two other related issues.  The fix will be available in an upcoming Windows Insider build.  Keep an eye out for the release notes!

I am using W10.1607 and the problem persists. It cannot escape spaces. Is there any update about where it might be available? Is there a way of getting the source and re-compile lxrun?

@aa6168 - can you share the output of the Windows "ver" command from cmd? I suspect you're still on anniversary update (14936).

@benhillis - You suspect wrong :) my case is worse than that. I am on update 14319. How can I get the latest update of lxrun?

@aa6168 - or waiting for Windows 10 Creator's update which I think is coming out soon.


Included instructions for installing on Gentoo

There was no package to install mininet using Gentoo's package manager.
I created an Ebuild, it is in the lemon-lime-overlay (not official). It is pretty basic at the moment. But, seems to get the job done!
I also found an Arch Linux "AUR" package.
PS: I do not know if the intention was to have per-distro instructions. Thought I would do this PR just to let you guys know.
Have a good day!

Note that I used setup.py to install it; I did not use install.sh.
I am getting the following error:
--------------------------------------------------------------------------------
Caught exception. Cleaning up...

FileNotFoundError: [Errno 2] No such file or directory: 'mnexec'
--------------------------------------------------------------------------------

I compiled the .c file manually and added to my path and it work.
In which file is the mnexec.c compilation handled?
Thanks in advance!

UPDATE: The ebuild show now be functional


mongoid search error

I couldn't find any documentation on how to use mongoid.
I added require 'puffer/orm_adapter/mongoid' and it started working (although I had to remove a bunch of my model fields). When I ran a search I got the following:
$where compile error

2:
<%= @title %>

3:
4:
5: <% if records.present? %>
6:   <div class="rui-paginator" data-paginator="{total: <%= records.num_pages %>, current: <%= params[:page] || 1 %>, link: {href: '<%= resource.collection_path(:page => '%{page}') %>'}}">

7:
8:   <%= render :layout => 'table', :locals => {:table_fields => index_fields} do |record| %>

mongo (1.3.1) lib/mongo/cursor.rb:101:in `next_document'
mongo (1.3.1) lib/mongo/collection.rb:251:in `find_one'
mongoid (2.2.0) lib/mongoid/collections/master.rb:25:in `block in find_one'
mongoid (2.2.0) lib/mongoid/collections/retry.rb:29:in `retry_on_connection_failure'
mongoid (2.2.0) lib/mongoid/collections/master.rb:24:in `find_one'
mongoid (2.2.0) lib/mongoid/collection.rb:60:in `find_one'
mongoid (2.2.0) lib/mongoid/contexts/mongo.rb:72:in `blank?'
mongoid (2.2.0) lib/mongoid/criteria.rb:44:in `blank?'
activesupport (3.1.0) lib/active_support/core_ext/object/blank.rb:18:in `present?'
puffer (0.0.29) app/views/puffer/base/index.html.erb:5:in `__home_gweber__rvm_gems_ruby_______p____gems_puffer________app_views_puffer_base_index_html_erb___2998686893773977758_47230160'
actionpack (3.1.0) lib/action_view/template.rb:144:in `block in render'
...

Now we have mongoid search support.

@gregwebs in what file did you add require 'puffer/orm_adapter/mongoid' to get puffer working with mongoid? Also, I tried using the mongoid fork of rails_admin, but it seems incompatible with rails 3.1. Any suggestions to make it work with the newest version of rails? Thanks for your help - I'm in desperate need of an admin interface that is compatible with mongoid.

Marcus,
I would recommend you first try the latest version of puffer.
I actually created a config/initializer/puffer.rb file with just that
require. Puffer does not have as many features as admin_assistant, so you
might try my fork, but it is kind of an ugly hack and may just fail on
features i don't use.
Thanks,
Greg
On Wed, Oct 19, 2011 at 12:20 PM, Marcus Gartner <
<EMAIL_ADDRESS>
@gregwebs in what file did you add require 'puffer/orm_adapter/mongoid' to get puffer working with mongoid? Also, I
tried using the mongoid fork of rails_admin, but it seems incompatible with
rails 3.1. Any suggestions to make it work with the newest version of rails?
Thanks for your help - I'm in desperate need of an admin interface that is
compatible with mongoid.
--
Reply to this email directly or view it on GitHub:
https://github.com/puffer/puffer/issues/12#issuecomment-2460010

Puffer master now has full mongoid support. So you can use it.
About features - puffer is a kind of framework. I try to make if as flexible as possible. You can create any kind of component you need. Components acts like apotomo, so they are self-updatable. I think powerfull components is a killer feature of puffer.

@gregwebs did you use https://github.com/apotonick/cells or https://github.com/apotonick/apotomo?
And i'm so sorry, i've just started my first really public project, so i'll l`rn how to write docs )
Maybe http://dl.dropbox.com/u/19461052/cast.mpeg - this cast could help you? It is unfinished right now, has no sound, but i hope, it'll help.

your docs are very good actually, it is just that components are a more
advanced concept. The screencast really needs some editing & audio, thanks
for sharing now then. I am looking at http://apotomo.de/ which has a lot of
docs.
BTW, This project has a lot of potential, largely due to orm_adapter &
Mongoid support which the others are falling behind on. Maybe components are
also the killer feature, or maybe others can already use cells/aptomo, not
sure.
On Thu, Oct 20, 2011 at 12:51 AM, pyromaniac <
<EMAIL_ADDRESS>
@gregwebs did you use https://github.com/apotonick/cells or
https://github.com/apotonick/apotomo?
And i'm so sorry, i've just started my first really public project, so i'll
l`rn how to write docs )
Maybe http://dl.dropbox.com/u/19461052/cast.mpeg - this cast could help
you? It is unfinished right now, has no sound, but i hope, it'll help.
--
Reply to this email directly or view it on GitHub:
https://github.com/puffer/puffer/issues/12#issuecomment-2465755

I've gotten a little bit further with Puffer, but I'm getting an error: Could not find table 'puffer_users'. What other setup steps do I need to perform that aren't mentioned in the README?

I'm also seeing and error: uninitialized constant ActiveRecord. I removed require rails/all and replaced it with
require 'action_mailer/railtie'
require 'active_resource/railtie'```

in application.rb to take out the ActiveRecord dependency because I shouldn't need it with mongoid, right?

Hm, i should share this screencast wider. After finishing )
In puffer you can use any auth system, but by default is uses AM secure_password with AR PufferUser model. I'll try to make this part of puffer more independant. Thanks for report.

Can you make the authentication wiki more clear on how to use your own authentication? Do i just need to add current_puffer_user and has_puffer_access? methods to my applications_controller.rb? Or is there more required?

I wrote an instruction for auth, but found there is docs already ) https://github.com/puffer/puffer/blob/master/lib/puffer/controller/auth.rb
Also please check master, it seems i've fixed uninitialized constant ActiveRecord issue

@mgartner: can you get your app (a simple hello world perhaps) without puffer to run first? Likely this error has been encountered by other Mongoid users.

I seem to have gotten by the ActiveRecord issue thanks to the latest fix. My next step is to setup the authentication system, either custom or getting the built-in one working with Mongoid. Currently puffer is trying to call an ActiveRecord method on the mongoid object:
undefined method `find_by_email' for PufferUser:Class

You can roll your own admin auth very easily. Add an admin field to your
users, check it in a before filter.
That code that is erroring should be changed to Rails3/Mongoid compatible
where(:email => email).first

On Fri, Oct 21, 2011 at 10:35 PM, Marcus Gartner <
<EMAIL_ADDRESS>
I seem to have gotten by the ActiveRecord issue thanks to the latest fix.
My next step is to setup the authentication system, either custom or getting
the built-in one working with Mongoid. Currently puffer is trying to call an
ActiveRecord method on the mongoid object:
undefined method `find_by_email' for PufferUser:Class
--
Reply to this email directly or view it on GitHub:
https://github.com/puffer/puffer/issues/12#issuecomment-2488539

Oh, my bad. I'll fix it soon. Also, basic own puffr auth system needs more docs.

Fixed, try master.

Ok I got my own auth system working. My next issue is that there seems to be no Mongoid-specific relationship components - ReferencesOne or References Many. I guess I may have to build my own?
BTW, I'm willing to document everything I've done to get up and running with Puffer and Mongoid if I can get to a comfortable stage where almost everything is working.

About ReferencesOne or References Many.
Puffer has own type-to-component mapping, https://github.com/puffer/puffer/blob/master/lib/puffer.rb#L62, so you even so i should add new mappings just like this:
map_component :references_one, :to => :ReferencesOneComponent
map_component :references_many, :to => :ReferencesManyComponent

Mongoid's referenceses api acs like AR's, so we can use the same components. On the other hand this two components are unfinished ) And they are next thing i should finish.

The ReferencesOneComponent and ReferencesManyComponent seem to break on Mongoid models because there is no 'foreign_key' field. That's the error I was getting when I tried to use them.

Then I'll write own mongoid components. Soon.