[ISSUE #374] add unit test for header class.

For #374 .

add unit test for PushMessageRequestHeader class.
add unit test for PushMessageResponseHeader class.
add unit test for ReplyMessageRequestHeader class.
add unit test for ReplyMessageResponseHeader class.

Codecov Report

Merging #472 (6883d21) into develop (737c0f0) will increase coverage by 1.37%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop     #472      +/-   ##
=============================================
+ Coverage       9.84%   11.21%   +1.37%
- Complexity       283      326      +43
=============================================
  Files            228      228
  Lines          10829    10829
  Branches         923      923
=============================================
+ Hits            1066     1215     +149
+ Misses          9666     9517     -149
  Partials          97       97

Impacted Files
Coverage Δ

...h/common/protocol/http/common/ProtocolVersion.java
45.00% <0.00%> (+10.00%)
:arrow_up:

...http/header/message/PushMessageResponseHeader.java
60.29% <0.00%> (+60.29%)
:arrow_up:

...ttp/header/message/ReplyMessageResponseHeader.java
65.78% <0.00%> (+65.78%)
:arrow_up:

.../http/header/message/PushMessageRequestHeader.java
66.00% <0.00%> (+66.00%)
:arrow_up:

...http/header/message/ReplyMessageRequestHeader.java
66.66% <0.00%> (+66.66%)
:arrow_up:

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 737c0f0...6883d21. Read the comment docs.


Add example for exposing Rust types to GDScript, using derive macros and/or wrapper classes

This question is asked pretty frequently, so it's good to have something to point users to.

Exposing value types through FromVariant and ToVariant.
Exposing operations on those types, e.g. Eq, through helper methods on a auto-load script.
Exposing opaque types / types with behaviors through wrapper classes.
Factory auto-load to work around the constructor argument problem.

Partially documented in https://godot-rust.github.io/book/rust-binding.html.
The points mentioned above (especially using a wrapper or auto-load script) still need documentation and/or an example.

Autoload singletons now seem to be documented here: https://godot-rust.github.io/book/gdnative/faq/code.html#can-i-implement-static-methods-in-gdnative
Closing as completed.


Fix support for setting TEMP_PATH in .env (fixes #8099)

Proposed fix for #8099. I’ve opted to rename the variable to SS_TEMP_PATH for two reasons:

It fits the convention, and TEMP_PATH as an environment variable is currently broken anyway
It won’t conflict with the TEMP_PATH PHP constant in Injector service definitions that use '`TEMP_PATH`'

Looks sensible. Probably need to target 4 I'd say?

Yeah I wasn’t sure about the target branch. It’s a documented feature and it doesn’t work properly, so I opted to treat it as a bugfix even though it’s technically introducing a new environment variable.

@emteknetnz what's your opinion here, please?

Target 4 since it's new configuration
@kinglozzer Would you be able to write a quick explanation for the 4.10 changelog
Include support for releative/absolute paths, a little rationale behind this decision, as well as any potential gotchas

I’ve added some more documentation and rebased against 4

@kinglozzer just noticed an issues with the travis shared config that was causing this build to fail, it should be fixed now, when you get a chance could you try re-pushing (retriggering the build in travis will reuse the old config, so it needs to be a push)

Having issues with the shared-config - I'll test it here using github actions


Add install doc for fedora 26

It seems the repo for fedora 26 does not exist yet.
ex. https://download.docker.com/linux/fedora/26
The repo added with the instructions will default to fedora/26. In a pinch, manually changing this to fedora/25 will work for installing docker.

Fedora 26 is not yet supported.


ScatterElements unsupported reduction modes, and duplicate index support

Refactor scatter_elements operator to be JIT compiled
Add min and max reduction modes
Implement duplicate index handling
Add corresponding reference, onnx, and verify tests.
Resolves migraphx-benchmark/AMDMIGraphX#73

@music-dino  can you address the file conflicts


fix: reformat RCTLegacyInteropComponents

Summary:
When Codegen generates interop components it reformats RCTLegacyInteropComponents.mm to have each component in a new line. This creates this unnecessary diff every time I run pod install. This PR aims to commit a formatted version of the codegen-generated file to avoid creating this diff every time. Here is a reference of Codegen function generating this: https://github.com/facebook/react-native/blob/44d6e4310cc9ad0d711d05e8dd5ee5220738e5b5/packages/react-native/scripts/codegen/generate-legacy-interop-components.js#L53

Changelog:
[INTERNAL] [CHANGED] - Reformat RCTLegacyInteropComponents to follow Codegen generated formatting
Test Plan:
CI Green

@okwasniewski so if I import this, the internal formatter will attempt to reformat it back to the original code

@cortinico Thanks for letting me know. I've modified the Codegen script to take into account Meta internal linter (I wasn't sure if the split should happen at 3 or 4 lines) but now the linter should be happy and it shouldn't generate unnecessary diff every time


Convert BuiltInScalarFunction::{NthValue}  to a user defined functions

Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/8709
There is now no difference between "built in" / "prepackaged"  scalar and aggregate functions in DataFusion, however there are still some "built in" window functions  -- see the current source for BuiltInWindowFunction for the up to date list of what remains
The problem with having two different kinds of window functions is

There are some features that may not be available to User Defined Window Functions that rely on built in
Users can not easily choose which window functions to include or override the behavior if they need something different

Describe the solution you'd like
I would like to remove the "built in" version of this function and convert it to a user defined function
Describe alternatives you've considered
At a high level the process is:

Add a new WindowUDFImpl in the functions-window crate
Port the code from the relevant existing implementation of the the built in functions in datafusion/physical-expr/src/window
Remove the BuiltInWindowFunction variant and then get everything to compile (the compiler will show you where the existing implementations are)

Additional context
Here are some good examples:

@jcsherin  ported RowNumber in https://github.com/apache/datafusion/pull/12030

I think this one is also relatively straightforward and would make a good first project for someone skilled in Rust who wanted to get experience with the Datafusion codebase and tests

take

Convert BuiltInScalarFunction::{NthValue} -> Convert BuiltInWindowFunction::{NthValue}

Convert BuiltInScalarFunction::{NthValue} -> Convert BuiltInWindowFunction::{NthValue}

🤦  -- fixed

Hi, @dharanad . If you are busy can I look into this !

Hi, @dharanad . If you are busy can I look into this !

Sure

take


Update lcars.app.js

Changed HRM as patch proposed in  #2395 I could not find the clockinfo module.
This change has not been tested.

Thanks - please could you update the ChangeLog and version number and I'll merge this? It'd be great if you could do the other affected clocks too.
Clock Info details are on http://www.espruino.com/Bangle.js+Clock+Info - but adding it is a much bigger deal than just changing this one line

Just applied this change to all 3 clocks and updated changelog myself


Minify uglify code

Huge changes to make the plugin better:

[x] Making tests run on travis
[x] Splitting directives, factories and controllers each one in one file
[x] Refactoring code to use a little bit of angular style guide
[x] Creating gulp tasks to concat and minify
[x] Reverting the pr #22 because it was breaking the player

My next PR I will configure bower of the project, then when the user installs with bower it will only download the files on dist folder not the whole project.

Hi @matheusrocha89, nice work!  I only find the code hard to follow with this structure. The minifying and uglifying tasks are great but I am having trouble following the code to make changes...seems I am not used to this syntax. Seems too verbose to me.  Any chances we can use the older syntax? Is it that bad?

The only problem I saw on the older syntax it's the idea of having one big file with everything. This approach let open chances to put all the code in just one big file and later it will be hard to maintain. The idea here is to isolate each directive and service in your specific file and your specific responsibility, like angular styleguide, but it's just a PR :smile: if you don't like this approach no problem :smile:

The idea of splitting files is fine, I see the benefit. Just the code style
using the $inject instead of Inline Array Annotation. In fact Angular docs
suggest inline is the preferred way. See here:
https://docs.angularjs.org/guide/di/#dependency-annotation. By using inline
and closures -instead of separate functions- the code seems easier to read.
What do you think?
On Thu, Dec 3, 2015 at 11:34 PM, Matheus Cruz Rocha <
<EMAIL_ADDRESS>wrote:

The only problem I saw on the older syntax it's the idea of having one big
file with everything. This approach let open chances to put all the code in
just one big file and later it will be hard to maintain. The idea here is
to isolate each directive and service in your specific file and your
specific responsibility, like angular styleguide, but it's just a PR [image:
:smile:] if you don't like this approach no problem [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/arielfaur/ionic-audio/pull/24#issuecomment-161854417.

For me no problem, if you want this style it's ok.

Just change the injection to inline?

I can do that myself :)

:+1: :smile: Thanks


Build fails to get dependencies on macOS 10.14.6

Details: https://github.com/EmbarkStudios/rust-gpu/issues/123
Device:

OS: macOS 10.14.6

Additional context
This is pulling the moltenvk-1.1.0 branch

Linking with PR #25 as this is not something that has been merged in yet, due to other problems as well with MoltenVK.

I have made some commits to the moltenvk-1.1.0 branch and the example compiles fine for me, but I'm on 10.15.7. Give it a go again and see if it will work for you now as well.

Still failing for me
error: failed to run custom build command for `ash-molten v0.6.0+43 (https://github.com/EmbarkStudios/ash-molten?branch=moltenvk-1.1.0#9b301046)`

Caused by:
  process didn't exit successfully: `/Users/gray/Code/rust-gpu/target/debug/build/ash-molten-47b38550fede2767/build-script-build` (exit code: 101)
  --- stdout
  Already up to date.

  ========== Retrieving MoltenVK dependencies into External ==========

  ========== Cereal ==========

  cereal repo: https://github.com/USCiLab/cereal.git
  cereal revision: 51cbda5f30e56c801c07fe3d3aba5d7fb9e6cca4
  Fetching origin

  ========== Vulkan-Headers ==========

  Vulkan-Headers repo: https://github.com/KhronosGroup/Vulkan-Headers.git
  Vulkan-Headers revision: 7f9879b1b1fab53f719a9ed5e6e29533b10972b2
  Fetching origin

  ========== SPIRV-Cross ==========

  SPIRV-Cross repo: https://github.com/KhronosGroup/SPIRV-Cross.git
  SPIRV-Cross revision: e6f5ce6b8998f551f3400ad743b77be51bbe3019
  Fetching origin

  ========== glslang and SPIRV-Tools ==========

  glslang repo: https://github.com/KhronosGroup/glslang.git
  glslang revision: bacaef3237c515e40d1a24722be48c0a0b30f75f
  Fetching origin
  0a1fb588cd365f7737cb121fdd64553923e0cef6
  060627f0b0d2fa8581b5acb939f46e3b9e500593
  Change directory to .
  Get spirv-tools

  In External/spirv-tools: ['git', 'remote']
  known-good
  origin

  In External/spirv-tools: ['git', 'checkout', u'0a1fb588cd365f7737cb121fdd64553923e0cef6']

  Get spirv-tools/external/spirv-headers

  In External/spirv-tools/external/spirv-headers: ['git', 'remote']
  known-good
  origin

  In External/spirv-tools/external/spirv-headers: ['git', 'checkout', u'060627f0b0d2fa8581b5acb939f46e3b9e500593']

  ========== Vulkan-Tools ==========

  Vulkan-Tools repo: https://github.com/KhronosGroup/Vulkan-Tools.git
  Vulkan-Tools revision: 2bfd6a752dc6cb26ceda4ec9d56e7dce98527680
  Fetching origin

  ========== VulkanSamples ==========

  VulkanSamples repo: https://github.com/LunarG/VulkanSamples.git
  VulkanSamples revision: a1fe1176690ec2574deb4dd3be6c4e62d1f5d186
  Fetching origin

  ========== Started building dependency libraries at 10:46:26 AM ==========
  Please be patient on first build
  Building external libraries for macOS
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  Completed building external libraries for macOS
  xcframework successfully written out to: /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/External/build/Release/SPIRVCross.xcframework
  xcframework successfully written out to: /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/External/build/Release/SPIRVTools.xcframework
  xcframework successfully written out to: /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/External/build/Release/glslang.xcframework
  ========== Finished at 11:02:31 AM ==========
  xcodebuild build -quiet -project "MoltenVKPackaging.xcodeproj" -scheme "MoltenVK Package (macOS only)"
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  still building MoltenVK
  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1283:42: warning: undeclared selector 'addPresentedHandler:' [-Wundeclared-selector]
                  if ([_mtlDrawable respondsToSelector: @selector(addPresentedHandler:)]) {
                                                         ^
  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1287:52: error: property 'presentedTime' not found on object of type 'id<MTLDrawable>'
                                  CFTimeInterval presentedTimeSeconds = drawable.presentedTime;
                                                                                ^
  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1285:18: warning: instance method '-addPresentedHandler:' not found (return type defaults to 'id') [-Wobjc-method-access]
                          [_mtlDrawable addPresentedHandler: ^(id<MTLDrawable> drawable) {
                                        ^~~~~~~~~~~~~~~~~~~
  2 warnings and 1 error generated.
  note: Using new build system
  note: Planning build
  note: Constructing build description

  --- stderr
  From https://github.com/USCiLab/cereal
     3e4d1b84..562321c3  master     -> origin/master
  HEAD is now at 51cbda5f Merge branch 'develop' for release 1.2.2
  From https://github.com/KhronosGroup/Vulkan-Headers
     320af06..87451c5  master     -> origin/master
   * [new tag]         v1.2.159   -> v1.2.159
  HEAD is now at 7f9879b Update for Vulkan-Docs 1.2.154
  From https://github.com/KhronosGroup/SPIRV-Cross
     a57b4b1b..fc644b50  master     -> origin/master
  HEAD is now at e6f5ce6b Merge pull request #1471 from KhronosGroup/fix-1467
  From https://github.com/KhronosGroup/glslang
     3b334b2b..d550bebe  master     -> origin/master
   * [new tag]           10-11.0.0  -> 10-11.0.0
  HEAD is now at bacaef32 Update spirv-tools and spirv-headers known goods (#2401)
  HEAD is now at 0a1fb588 Update CHANGES
  HEAD is now at 060627f Register the ANGLE compiler (#168)
  From https://github.com/KhronosGroup/Vulkan-Tools
     ee0de094..51c1c5ab  master     -> origin/master
   * [new tag]           v1.2.158   -> v1.2.158
  HEAD is now at 2bfd6a75 vulkaninfo: Add json output for portability subset
  HEAD is now at a1fe11766 Update to 1.2.154 headers
  ** BUILD FAILED **

  make: *** [macos] Error 65
  thread 'main' panicked at 'failed to fetchDependencies', /Users/gray/.cargo/git/checkouts/ash-molten-2e22393c7b19b606/9b30104/build.rs:107:9
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
warning: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: /Users/gray/Code/rust-gpu/target/debug/build/spirv-tools-sys-dedf52f62e2aa81b/out/libspirv-tools.a(pch_source_opt.o) has no symbols

  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1283:42: warning: undeclared selector 'addPresentedHandler:' [-Wundeclared-selector]
                  if ([_mtlDrawable respondsToSelector: @selector(addPresentedHandler:)]) {
                                                         ^
  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1287:52: error: property 'presentedTime' not found on object of type 'id<MTLDrawable>'
                                  CFTimeInterval presentedTimeSeconds = drawable.presentedTime;
                                                                                ^
  /Users/gray/Code/rust-gpu/target/debug/build/ash-molten-86dfc2a28855154e/out/MoltenVK-v1.1.0/MoltenVK/MoltenVK/GPUObjects/MVKImage.mm:1285:18: warning: instance method '-addPresentedHandler:' not found (return type defaults to 'id') [-Wobjc-method-access]
                          [_mtlDrawable addPresentedHandler: ^(id<MTLDrawable> drawable) {

The problem is your macOS version MoltenVK depends on api's that Metal provides only in 10.15+. Maybe it is possible to just upgrade the Metal libs although I doubt that is possible without upgrading your OS

One thing you can also try is to see if you can force TARGET_OS_SIMULATOR and see if that allows you to build on macOS 10.14

Ah right Catalina is required right, would be nice to detect Mac OS version in build.rs and fail on it there instead of in the C++ compilation. Probably is way to do that reasonably easily? Will help us when Big Sur later on becomes required.
@termhn any potential issues for you to upgrade to Catalina (or Big Sur soon)?

I think I could... my macbook's getting a little old but I don't think they're unsupporting it or anything at least through Big Sur's release cycle. But yeah would be nice to error or at least document that 10.15 is required.

Filed #30


bugfix/1226/fixing-reply-comment-translation-bug

Fixed bug with error message translation when you change language of the app
Screenshots

Original
Updated

Checklist

[x] 🔽 My branch is up-to-date with "development" branch
[x] ✅All tests passed locally
[x] ✨My changes working with up-to-date admin and back-end part locally, like charm
[x] 🔗 Link pull request to issue #1226

Task is done already


Openapi schema/deprecated annotation

The existing code LGTM and is great work, but we are missing a test and error message for disallowing the @schema/deprecated on a schema document.
The error should mention that annotating a document as deprecated is disallowed, and instead encourage users to annotate individual keys.

Thanks for the comments, Cari! I have added the test case and error message for disallowing @schema/deprecated on a schema document.


Reservation index page: add option to filter by due date

The reservation index page automatically filters by start date; there should be an option to toggle by due date. We should also consider adding sane defaults for each category; eg filter by due date when viewing "Returned Overdue" though I'm personally wary of adding in non-intuitive/well documented variations in behavior.
See discussion in #1197

I think this is superseded by #1432, where we're including any overlap with those dates, not just start date overlap. Closing.


Fix implicit fallthrough in pd_translate_event

See https://github.com/goToMain/libosdp/actions/runs/3650442394/jobs/6166438430 failing on master. I'm not exactly sure why the other failure is happening.

I have a fix for this. I haven't pushed it yet because the test failure needs to be resolved. Will close this one, thanks for the PR.

Pushed 21a56246d204c34e433dc8714e69bfdfa189d089


⚠ Bofrost.be has degraded performance

In 00926cd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1063 ms

Resolved: Bofrost.be performance has improved in a4ca5b7.


Django version with patches

Codecov Report

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

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   75.63%   75.63%
=======================================
  Files          16       16
  Lines         238      238
=======================================
  Hits          180      180
  Misses         58       58

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 8a65740...2528553. Read the comment docs.


mypy plugin reports an unsafe variance error with no line number when extending class where the error is suppressed

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
i'm attempting to work around #9580 by using a type:ignore comment. however attempting to extend the class causes another unsafe variance error that cannot be suppressed because it does not have a line number:
>mypy .\test2.py
test2.py: error: Cannot use a covariant type variable as a parameter  [misc]
Found 1 error in 1 file (checked 1 source file)

Example Code
from typing import Generic, TypeVar

from pydantic import BaseModel

T = TypeVar("T", covariant=True)

class Foo(BaseModel, Generic[T]):
    value: T  # type:ignore[misc]

class Bar(Foo[T]): ... # this line causes the additional error that cannot be suppressed

Python, Pydantic & OS Version
pydantic version: 2.7.3
        pydantic-core version: 2.18.4
          pydantic-core build: profile=release pgo=true
                 install path: C:\Users\user\project\.venv\Lib\site-packages\pydantic
               python version: 3.12.0 (tags/v3.12.0:0fb18b0, Oct  2 2023, 13:03:39) [MSC v.1935 64 bit (AMD64)]
                     platform: Windows-10-10.0.19045-SP0
             related packages: typing_extensions-4.12.1
                       commit: unknown

Which version of mypy are you using? (Just to help reproduce the issue.)

I was able to reproduce this issue and think I have a fix in #9606, if you can test whether that branch fixes your issue that would be much appreciated. In particular, I have a slight concern that it could introduce other issues, so it would be good to run it on a "real" codebase using these covariant generic models and confirm that it doesn't break new things.

Marking as resolved. Feel free to ping us if that change doesn't fix your issue!


npiperelay creates terminal window every time it is invoked from WSL2

The Problem
On the new version of the Windows Insider Preview for Windows 11 (10.0.22518.1000), WSL2 is fully installed and maintained via the windows store app. Something about this change (either the version of windows or the new WSL version) causes the npiperelay application to launch new windows every time it is invoked.
This occurs regardless of if the default Terminal app is set to "Windows Console Host" or "Windows Terminal".
Windows, WSL2/g, & Kernel version info
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.29
Windows version: 10.0.22518.1000
WIndows Terminal version: 1.11.2921.0
Backstory and use case
I'm using this as directed by the WSL2 workaround specified at: https://github.com/rupor-github/wsl-ssh-agent
My shell rc file contains:
# Set up SSH Agent Compatibility with Windows
export SSH_AUTH_SOCK=$HOME/.ssh/agent.sock
ss -a | grep -q $SSH_AUTH_SOCK
if [ $? -ne 0   ]; then
    rm -f $SSH_AUTH_SOCK
    ( setsid socat UNIX-LISTEN:$SSH_AUTH_SOCK,fork EXEC:"$HOME/winhome/.wsl/npiperelay.exe -ei -s //./pipe/openssh-ssh-agent",nofork & ) >/dev/null 2>&1
fi

When SSH activates, it uses this named pipe to interact with the Windows SSH Agent. I don't believe that the issue of a new terminal window appearing each time npiperelay is activated is related to this specific use-case, so I figured it was more appropriate to submit the issue here rather than on the wsl-ssh-agent repo.

Can you check to see if this is the same issue you're describing?  https://github.com/microsoft/WSL/issues/7568

This may be related. When running the example command referenced in your linked issue, a console window does open.
Weirdly enough, after updating to the latest Windows 11 build, running npiperelay from WSL as described in my original description doesn't seem to open a new window anymore, even though I do get a console window opened when running the example command at microsoft/WSL#7568
So... Maybe my issue is fixed by the newer version of Windows 11? There's nothing about terminal, console, WSL, or background calls being fixed in the release notes though, so I'm not entirely sure.
Updated version info:
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.29
Windows Version: 10.0.22526.1000
Windows Terminal Version: 1.11.3471.0

As an addendum to my above comment, it looks like I was incorrect in saying that npiperelay didn't open a new window any more. It seems to be a somewhat sporadic issue. That day, it wasn't opening a new window, but every day since, it has behaved as originally described. I'd wager that the issue is likely the same as microsoft/WSL#7568 and we can wait until that has some movement.


feature request: Please integrate support for S/MINE encryption and decryption

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.804
Problem you are trying to solve
no support now.
Suggested solution
feature request: Please integrate support for S/MINE encryption and decryption
I cannot say whether this is already planned. I have searched for S/MINE in the issues, but found nothing.
The integrated PGP encryption is certainly the better or more free option for many people. However, many mail clients support S/MINE. Since S/MINE requires less preparatory initiative on the part of the recipient, a signed e-mail can be sent to practically anyone with S/MINE. I would therefore be very pleased if K9 Mail offered both encryption methods similar to Thunderbird for desktop computers.
Screenshots / Drawings / Technical details
I consider signing, encryption and decryption as one feature, since only the entire package is usable together. Please split the tasks if this is useful for the implementation.

Duplicate of https://github.com/thunderbird/thunderbird-android/issues/1003

yes it is.


[0][Easy][App] Remove Ollama warning

Find out why the below warning still pops up, and remove it! We don't use Ollama anymore.

On it

Problem is in the file pearai-app/extensions/pearai-submodule/extensions/vscode/src/util/loadAutocompleteModel.ts the default model is used, since none are defined in the config. The default model is hardcoded to Ollama, which is why this warning pops up. I also noticed you guys have code for the "autocomplete" feature in core.js but that code is never called, as the "autocomplete" feature is directly handled in extensions/vscode/src/extension/VsCodeExtension.ts with the ContinueCompletionProvider class. It seems someone coded everything for the "new" autocomplete I guess (without Ollama) and didn't plug it to vscode.
I can fix this and plug back into core.js for the autocomplete. Or if the feature is not ready I can just remove the warning, but would make autocomplete not work.

it should be undefined. fixed.


No way to get Subscriptions?

I can't seem to find any method to get subscribed subreddits (preferably returning a List of sub names). Does one exist?
Thanks!

https://thatjavanerd.github.io/JRAW/docs/git/net/dean/jraw/paginators/UserSubredditsPaginator.html
You have to use the SUBREDDITS_MINE_SUBSCRIBER endpoint

Wow didn't see that page. Thanks!


in paperspace there is no active area in the settings of the machine

Good afternoon! When I go to paperspace in the settings of the machine is not able to click on anything, neither to select a video card, nor to see what are currently available. Please tell me who knows what it is. Attached a screenshot

Just click on "Start machine" and when there is a free GPU available, it will automatically clone this repository to your account and you will be able to use it right after.

But it uses any available video card, and I would like to choose which video card to use

a more interactive template will be added soon

Thank you very much TheLastBen, Gamemaster2930. I'll try it tomorrow.


Do not expose Guava's ToStringHelper from the public API

Motivation:
It's a bad idea to expose Guava classes from our public API because all
Guava classes are shaded into com.linecorp.armeria.internal.shaded.guava,
exposing the internal classes in the public API.
Modifications:

Fixed the broken test in javadoc/build.gradle.
Replaced DynamicEndpointGroup.toStringHelper() with toString(Consumer).

Result:

We do not leak internal classes in our public API anymore.

eckstyle] [ERROR] /home/ec2-user/actions-runner/_work/armeria/armeria/core/src/main/java/com/linecorp/armeria/client/endpoint/DynamicEndpointGroup.java:36:8: Unused import - com.google.common.base.MoreObjects.ToStringHelper. [UnusedImports]

Would you mind fixing the lint failure?

Thanks a lot for fixing the lint error, @jrhee17 :bow:


Reject incoming updates when the channel is in registering or later phases

Context
Currently,

For outgoing updates, updateGeneric calls machine.Update, which in turn checks if machine is in Acting phase

For incoming updates, CheckUpdate is called, which in turn calls validTransition.

Problem
I think, this means an incoming update will be passed onto the user even when the channel is in registering or later phases. It will be upto the user of go-perun to check the phase and then reject this update.
Possible solution
In CheckUpdate, return error if phase of the registering or later phases.

@matthiasgeihs


future plans?

This engine is awesome!
Are there any plans to separate the engine from the demo? Then this engine could be used to power general 3d games!

hi qntf,
there are currently no plans for that, no. Thanks for the compliments though :)


Devcon 2024 talk proposal

What is it?
Due today
https://devcon.org/en/speaker-applications/

Submitted
https://docs.google.com/document/d/1DVuh4BenGd_88FCiB6NnApk_I42in0WAOMKLis3KGEY/edit


Android banner position bottom center not working

I have tried to implement admob for both ios and android but it is not working on android. Ios is working perfectly fine.
if (window.AdMob ){
                var admobid = ( /(android)/i.test(navigator.userAgent) ) ? adMobHelper.getAndroid() : adMobHelper.getIos();
                AdMob = window.AdMob;

                AdMob.createBanner({
                        adId: admobid.banner,
                        adSize:AdMob.SMART_BANNER,
                        position:AdMob.AD_POSITION.BOTTOM_CENTER,
                        autoShow:true
                    },
                    function(){console.log("Success Ad");},
                    function(error){console.log("Error ad: "+error);}
                )
            }

For android, ad is showing up in the middle of the screen. I have tried on galaxy s3 and nexus 5, same behaviour on both devices.

@floatinghotpot
Is it possible to use AD_POSITION.BOTTOM_CENTER without setting overlap: true? Doesn't seem to work for me.

cordova and plugin version?

Cordova CLI: 5.0.0
Installed platforms: android 4.0.0

com.google.cordova.admob 2.7.7 "AdMob Plugin Pro"
com.google.playservices 21.0.0 "Google Play Services for Android"
com.rjfun.cordova.extension 1.1.4 "Cordova Plugin Extension"

I have
Cordova 5.0.0
android 4.0.0
com.google.cordova.admob 2.7.7 "AdMob Plugin Pro"
com.google.playservices 19.0.0 "Google Play Services for Android"
com.rjfun.cordova.extension 1.1.4 "Cordova Plugin Extension"
But the overlap does not reserve the space in screen, do I need to fix the height now?

To come back to this one, instead of AD_SIZE.BANNER, I've switched to AD_SIZE.SMART_BANNER, so the banner no longer needs centring.

i'm using SMART_BANNER also

Which plugin is better for wp8, i tried cordova-plugin-admob its working fine, but got error in admobpro, I dont know how to give position for ads, its only available in admobpro. How to set x,y position in admob ,please reply me

admobpro is recommended, if same feature works in admob but not working in admobpro, it will be fixed with high priority. What error do you get in admobpro? If you problem is in wp8 only, please find a issue tracker for wp8 and post there.

I got a workaround to fix the Center-Issue in android. In my AdMob.js I changed the admobExport.showBanner-function from
admobExport.showBanner = function(position, successCallback, failureCallback) { 	if(typeof position === 'undefined') position = 0; 	cordova.exec( successCallback, failureCallback, 'AdMob', 'showBanner', [ position ] ); };
to
admobExport.showBanner = function(position, successCallback, failureCallback) { 	if(typeof position === 'undefined') position = 5; 	cordova.exec( successCallback, failureCallback, 'AdMob', 'showBanner', [ position ] ); };
After that change it works fine with MEDIUM_RECTANGLE and CENTER.


Does GCL need an ECCN and if so, what is it?

We would like to bundle GCL in an Apache Software Foundation product.  Someone has raised concerns about the files in closure/goog/crypt.  Do any of these files require an Export Control Classification Number?

I don't know how I would find out.

I would be surprised if Google does not have experts in ECCN rules and regulations.  If you read up on ECCN, it seems like serious stuff.  Apache Software Foundation projects are expected to conform.  I would think Google would want its open source projects to conform as well.

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


raftstore-v2: fix error handling during merge trim check

What is changed and how it works?
Issue Number: Ref #12842 #15262 #15242 #15289
What's Changed:
- Restore merge status when there's an error between trim check and propose.
- Fix a case where snapshot just before prepare merge can cause rollback.
- Mute an assertion that fails in v1 bulk_insert testing.
- Use SkipUntil in compaction filter, together with the newly introduced
  UnsafeFilter API, this should remove any tombstones regardless of the context.

Related changes
https://github.com/tikv/rocksdb/pull/344
https://github.com/tikv/rust-rocksdb/pull/757
Check List
Tests

Unit test [TODO]

Release note
None

/release

@tabokie why "Revert "check snapshot before trim""?
The version QA tested does not have this commit and it seems OK.

/merge


make the go get compatible with go modules

right now if one types out go get github.com/grafeas/voucher it will get v1.2.2 of Voucher. this is because it's the latest v1 version of Voucher and since we're using go modules, we need to adopt a new module path to be able to make go get correctly fetch the latest version (and also any other v2 version; v2.5.3 is the latest version). more information about this can be found in this official blog post
for now though, if one wants to fetch any v2 version of this package they have to reference it by commit hash. for example:
➜ go get github.com/grafeas/voucher@fef6b9c35e24e71ba48804ed96a3e4117da95e91
go: github.com/grafeas/voucher fef6b9c35e24e71ba48804ed96a3e4117da95e91 => v1.2.3-0.20210303185942-fef6b9c35e24
go: downloading github.com/grafeas/voucher v1.2.3-0.20210303185942-fef6b9c35e24

Technically, does this have be to be updated for every v2 version released?


Improve documentation for writing plugins

The current doc for writing plugins is really sparse.
Feedback on what needs to be improved
Include examples for how to format groups output:
{
  "value": [
    {
      "name": "standard",
      "targets": [
        {
          "uri": "bb8531de599e"
        }
      ]
    }
  ]
}

basically, what is the expected output with examples when a grouped response is returned.  Also, can the returned group response also have other config data?
{
  "value": [
    {
      "name": "standard",
       "transport": "ssh",
       "ssh": {
          "tmpdir": "/root/tmp"
       },
      "targets": [
        {
          "uri": "bb8531de599e"
        }
      ]
    }
  ]
}

Are there other types of responses that can be returned?


feat: import.meta.glob support ?raw

Description
#5304 #3222
Add ?raw support for import.meta.globEager()
For example,
const files = import.meta.globEager('../assets/svgIcon/*.svg?raw')
console.log(files)
// Expected results
/*
{
  '../assets/svgIcon/github.svg':
  <svg t="1634195145119" class="icon" viewBox="0 0 1024 1024" version="1.1" ...',
  '../assets/svgIcon/logo.svg':
    '<svg t="1634195051463" class="icon" viewBox="0 0 1024 1024" version="1.1" ...'
}
*/

Additional context

Use cleanUrl() to remove ?raw so it will not affect glob.sync(pattern).
Whether need to add ?url support can be discussed

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

I have implemented a version compatible with the standard. If it feels appropriate, I can continue to implement support such as url and json.

Thanks! Lets wait til next week meeting so we check with Evan that this is the way to go

We talked about it in the last team meeting, and we think it is good to align with the standard as this PR is now implemented. Thanks for the refactor @ygj6
For reference, these are notes from Evan showcasing the alignment. We could later add options like inline, exclude and others
import './foo.js' assert { type: "worker", inline: true }
import('./foo.svg', { assert: { type: "raw" }})
import.meta.glob('./*.svg', { assert: { type: "raw" }, exclude: '...' })

I'm worried about including json5 to do the parsing. But it is already in two of our dependencies (we will probably be using another version though). @bluwy @ygj6 what do you think?
Let's target the 2.8 beta to include this feature.

I think json5 may be the most reliable choice for now, since the spec doesn't seem to be explicit on its syntax, so it would likely be parsing any kind of JS objects. +1 for import assertions though!

Thanks for your replies, I agree with @bluwy 's https://github.com/vitejs/vite/pull/5545#issuecomment-986036037 , I can continue to implement the rest later.

We could later add options like inline, exclude and others

@patak-js Where can I find reference docs about it to help me understand it more? thx

Yes, let's merge this PR only with raw support in 2.8
@ygj6 the other types and options would resolve to the current suffixes (url, raw, worker, inline, etc). But this API isn't yet defined, maybe we could wait a bit here.

@patak-js Oh, I see, looking forward to it coming. If there is anything I can help, please call me :)
In addition, I found some minor bugs in the code of lexGlobPattern

when lexing the pattern, it may contain quote strings, e.g. '/Tom's files/**', results in '/Tom'
simply find index of ) to get the endIndex is fragile, the options code may contain annotation, e.g.

import.meta.globEager('/dir/*.json', {
    // (set type)
    assert: { type: 'raw' }
  })

get the wrong pos next to type)<--
So, I plan to use acorn lexer to further refactor the code of lexGlobPattern and push it, see if you think the new approach is acceptable.

@ygj6 If we go with acorn, it might be the first time Vite does parsing in dev mode, which have been avoided for performance reasons. But given it only runs for globEager and glob, maybe it's not too bad. Otherwise we can accept this bugs upfront and document them instead to preserve performance. I'm slightly leaning on accepting the bugs though, as they can be easily worked around.

Yes, as @bluwy said, we also discussed this with the team. We are accepting a few edge cases in exchange for speed with Vite. There are a lot of places where it will be better to remove regex and replace them with an AST but DX would suffer if we do that. It is a tradeoff that at this point is working well for us. So let's keep the current scheme and see if we can polish that edge cases or document the limitations if we can't.

Reasonable, more understanding of vite’s pursuit of speed and certain tradeoffs, thanks for your patient explanation!

I think this is safe to merge?

I am using vite 2.8.0-beta.1 but with below code I am unable to get it working 🤔
const pageLayoutsGlob = import.meta.globEager('../../../views/generator/sneat/page-layouts/*.html?raw')
console.log('pageLayouts :>> ', pageLayoutsGlob)

Does anyone know why?

Same if I use import.meta.glob()

@jd-0001 you should use
const pageLayoutsGlob = import.meta.globEager('../../../views/generator/sneat/page-layouts/*.html',{ assert: { type: 'raw' }})
console.log('pageLayouts :>> ', pageLayoutsGlob)

Thanks :)
Just to make vite robust I have something to share:
const pageLayoutsGlob = import.meta.globEager('../../../views/generator/sneat/page-layouts/*.html',{ assert: { type: 'raw' }})
console.log('pageLayouts :>> ', pageLayoutsGlob)

won't work because I guess it is taking path from root dir of project.
Hence, I updated it to:
const pageLayoutsGlob = import.meta.globEager('/src/views/generator/sneat/page-layouts/*.html', {
  assert: { type: 'raw' },
})
console.log('pageLayouts :>> ', pageLayoutsGlob)

Also, I am getting linting errors:

I guess function signature isn't updated yet 🤔

I am using vite<EMAIL_ADDRESS>

I assume #6433 is not released yet

request support code splitting
return like this
{
  'path1': () => import('path1'),
  'path2': () => import('path2')
}

@LiuJiangshan Would you like to open a new issue and describe the request more clearly in it?


MINIFICPP-502 Add validation to config parser to validate required properties

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file?
[x] If applicable, have you updated the NOTICE file?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

501 is a subset of 502. Is that correct? If so I'll review 502. The same comments apply here


private components indented too far in file tree

In the file tree, private components are all listed together at the bottom, and are indented slightly more than other components, making them appear to be children.

They should be as equally indented as sibling components.

+1. @caneruguz Please take note to fix this in Treebeard/Fangorn.

In fangorn this is not an issue, private folders appear aligned with their siblings as expected. How can we tag this issue to revisit after 0.24?

Marked with "Hold" label, pending merge of treebeard/fangorn

This is what's happening now. Time to close this issue? @sloria @lbanner

Yes @caneruguz -- Got it -- Thank you!


EnableVirtualizationBasedSecurity Licensing Inconsistency

When attempting to apply the CSP EnableVirtualizationBasedSecurity to a computer running Windows 11 Pro, Intune returns an error. In the event log on the computers where this is occuring, the following error is present: "MDM PolicyManager: Policy is rejected by licensing, Policy: (EnableVirtualizationBasedSecurity), Area: (DeviceGuard), Result:(0x82B00006) Unknown Win32 Error code: 0x82b00006."
Normally, when a policy does not apply due to licensing constraints, Intune shows it as "not applicable". The page linked below indicates this policy should work with Pro editions of Windows.
Should the policy work with Pro editions of Windows or not? If it does not, this article should be updated to reflect that.

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

ID: 329b43eb-db67-4458-730a-65e21a5d95c2
Version Independent ID: 3bb2c84f-60db-0512-c44c-e6c011c36285
Content: DeviceGuard Policy CSP - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-deviceguard.md
Product: windows-client
Technology: itpro-manage
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

Thanks @arourke for the feedback, but we auto-generate that content from the DDF XML, which is supposedly authoritatively declaring that applicability.
So two possible next steps for you:

Go through Intune Support to help troubleshoot the issue, perhaps there's something else causing the issue. Like you said, it usually returns a different error when the policy isn't applicable.

If it truly doesn't apply to Pro edition and the DDF is wrong, then it needs to go to the Windows engineering team via the Feedback Hub. I think it’s important that you submit it directly, because that allows you to receive notifications and more closely monitor the progress.

I'm closing this issue as there's nothing actionable for docs at this time.


[Tensor] Add transpose with out

[Tensor] Add transpose_i

This patch creates transpose_i, transposes(out) to manipulate given tensor
With exhaustive list of test

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1150. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @zhoonit, nntrainer/tensor/tensor.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

Please consider rebasing this PR directly to main, as IMO, it does not depend on other PRs and can be merged faster.

Closing this to try automated merge pr.


Download button breaks on certain mobile width

Seems like our Download button is getting out of the bound on certain device width.
Hint

Add/ Edit media query properties for download button.

Current Situation

Is it Resolved already or I have to do any work ?

@Kkotiswar  If you want to , you can.


health text changes

woohoo now everything looks awesome
ascendeds1on with the supa hot fiya

WE HAVE ALIGN_REVERSE WHY DONT YOU USE IT???

WE HAVE ALIGN_REVERSE WHY DONT YOU USE IT???

cuz im cool


[ToDo] Testing & CI/CD & Workflows

Title: Implement Testing Phase
Description
This issue covers the implementation of the Testing Phase for the Gradebook Management System project. The Testing Phase involves creating unit tests, integration tests, and performing manual testing.
Tasks

[ ] Write unit tests for all functions
[ ] Automate unit tests
[ ] Write integration tests
[ ] Perform manual testing
[ ] Fix identified bugs and issues
[ ] Use Valgrind to fix/find memory leaks
[ ] Create GitHub Actions/Workflow to build and test

Acceptance Criteria

All functions have corresponding unit tests
Unit tests are automated and run successfully
Integration tests cover all main functionalities
Manual testing is performed and all issues are fixed

Notes

Use CTest and/or Google Test
Ensure test coverage for all critical paths in the code
Document the purpose and expected outcome of each test

See: https://github.com/gbowne1/gradebook-management-system/discussions/23


Character limits should only be presented for relevant form elements

Description
A number of form elements that are selectable rather than inputs have character limits that can be specified, which doesn't really apply.
For example: number might benefit from character limit but does not have that feature, while radio button cannot and has it.
Steps to reproduce

Go to GC Forms > Edit
Click on Add a form element
Add all types of form element
Go to "More" for each one
Notice how they have character limits even when that is irrelevant

Details

Browser: Edge
Operating system: Mac
Environment: Staging
Language: French
Viewport: Tablet

Expected behaviour
Only show character limit if/when relevant, when a text input is expected by the form filler.
Screenshots or videos

Issue resolved in 4175


Quantile plots fail when horizon is 1

Quantile plots fail when horizon is 1 on ozone dataset (regression introduced with #158)
Traceback (most recent call last):
  File "/home/dev/python/packages/timeseries/pyaf/tests/func/test_ozone_horizon_one.py", line 32, in <module>
    lEngine.standardPlots("outputs/my_ozone_horizon_one");
  File "/home/antoine/dev/python/packages/timeseries/pyaf/pyaf/ForecastEngine.py", line 48, in standardPlots
    self.mSignalDecomposition.standardPlots(name, format);
  File "/home/antoine/dev/python/packages/timeseries/pyaf/pyaf/TS/SignalDecomposition.py", line 763, in standardPlots
    self.mBestModels[lSignal].standardPlots(lName, format);
  File "/home/antoine/dev/python/packages/timeseries/pyaf/pyaf/TS/TimeSeriesModel.py", line 379, in standardPlots
    tsplot.quantiles_plot(lOutput,
  File "/home/antoine/dev/python/packages/timeseries/pyaf/pyaf/TS/Plots.py", line 176, in quantiles_plot
    _, bins1, patches = axs[h].hist(q_values, bins = q_values, weights=[1]*len(lQuantileNames), density = True)
TypeError: 'AxesSubplot' object is not subscriptable

Closing


interactive map

link map with tabulator like in https://nestroyca.acdh-dev.oeaw.ac.at/nestroyvienna/
map should go to all list views for classes with coordinates (districts, neighbourhoods, karies), but start with districts

Adjusting the script for our data has proven to be a bit complicated: we just use markers, not circles indicating frequency, we have a lot of entries that don't have coordinates, no layers (at least not at this point), the data displayed in the table is quite different and the whole process of getting the json files was also different. It works now, just some cosmetics left (shadows are still visible after the marker becomes invisible, map focus when no markers are visible etc.)

well, just checked and looks good enough for now; please feel free to merge your code into the main branch, but please make table and map the same height before
thank your!!!

I already did that yesterday, just haven't commited it yet because I was trying to fix the clicking after filtering issue


perf(): column formatter

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 99.724%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
219

Relevant Lines:
219

💛  - Coveralls


Add CoinCollect (CC)

Currency name: CoinCollect
Symbol: CC
Website: https://twitter.com/CoinCollectHQ
Theme color: #36b0f3
Logo (SVG): cc-logo.svg.zip
I could only find a PNG of their logo, so I vectorized it.

cc-logo_1.svg.gz
I have attached a native SVG icon. Can you please use that?

Yup


Feature request for WSConvTranspose2d

Describe the solution you'd like
I would like to have transposed version.
Additional context
Want to test how it works for gan.

Added here https://github.com/vballoli/nfnets-pytorch/commit/26d6bd89c71e31e19e862df431bd721e3225f5ee! Do let me know how it works out for you! It's not in the latest release yet, you'll have to install it as pip install git+ .

It's added to the new release. pip install nfnets-pytorch directly to access the WSConvTranspose2d

Sound like it doesnt work in gan setup.
Thanks for your efforts.

Have you tried only WSConv2d on the discriminator and the normal generator framework?


Finish PayPal

PayPal has code but I don't think it works. Feel free to use a new package. Maybe old one doesn't support Laravel 9. Support coupons. Get list of plans from PayPal API, I'll copy from Stripe to the PayPal account. Support 7 day trials. (https://developer.paypal.com/docs/subscriptions/customize/trial-period/)
Change role plan to purchased package using webhook/api.

Cancelled.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.16.3 to before v5.16.5",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=215a90ce3754fe509efbce6b73a4bb643c7e7528",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=fa4ad064a6bd49208221df5e62adf27b426d1720"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "215a90ce3754fe509efbce6b73a4bb643c7e7528",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "fa4ad064a6bd49208221df5e62adf27b426d1720",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "powerpc/perf: Fix power_pmu_disable to call clear_pmi_irq_pending only if PMI is pending\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.5 by commit fa4ad064a6bd49208221df5e62adf27b426d1720, it was introduced in version v5.16.3 by commit 215a90ce3754fe509efbce6b73a4bb643c7e7528. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000629


The image cannot be displayed error

I have the following code and the image popup throws the error: 'the image could not be loaded". Also, the href attribute of the popup is 'undefined'. What am I doing wrong?
$('span.confluence-embedded-file-wrapper').magnificPopup({type: 'image'});
Popup snippet of the error:

The image could not be loaded.

And the HTML code:

It came out to two things.

Missing href. For this I added the below:

Instead of before or after the magnific call, I had to add it at the end of page after the body tag. It works now.

Is there a way to make it work work without inserting the href attribute? src attribute is already there which is what I am pushing to href.

All seems working now.


Archive to separate file does not honour date format

Plugin version: 0.23.2
Obsidian version: v1.3.5 (windows desktop)
Windows 11 22H2 (OS Build 22621.1848)

When using "Archive to a separate file" option, with File name spec 2023/Daily/{{date}} and under "Configure variables" both "Date format" and "obsidian-taks completed date format" set to YYYY-MM-DD_ddd the created filename is in format YYYY-MM-DD
Example screen grab, with expected existing filename 2023-06-24_Sat, archived tasks are put in newly created file 2023-06-24.  (Same happend if expected file does not exist:

BTW Thanks for the great plugin, exactly what I needed.

Same issue here.

Hello, @Kolossi, please update the plugin and re-check.

Yup confirm fixed.  Thanks. @ivan-lednev


Jogging not working

When jogging the panel adds $J=G91G20X0.1 F50 and we get an error:Bad number format.
How do I correct this?

Grbl version? Is Spindle speed set?

We are using it on the Inventables X-carve.
Grbl Version 1.0c
Spindle speed is set.
It happens on all axis.
No indication for Idle,Jog only Alarm.

1.0c is ancient, lots of changes since then. I recommend that you upgrade.
I should probably add a pop up indicating unsupported for anything less than 0.9h or 1.1.


Double new lines not being displayed correctly in WebAssembly as module

Running the following code...
package main

import (
	"fmt"
)

func main() {
	fmt.Print("Should be on 1st line\n\n")
	fmt.Print("Should be on 3rd line\n")
}

In Go Server; output:
Should be on 1st line

Should be on 3rd line

In WebAssembly (as module); output:
Should be on 1st line
Should be on 3rd line

After 2 new lines, every other works fine.


Provide a way to flag crops as duplicates

#106 I think was done for site admins and so forth; but as a non admin user it'd be nice to indicate some kind of problem.
Specifically things like: https://openfarm.cc/en/en/crop_search?q=Cucurbita+pepo
The majority of those are just aliases, and the bionomial name should be a strong indicator for consolidation.

I don't think #106 was ever actually implemented. We moved things to Loomio but I don't think anyone cared about that (downside of spreading across different platforms).
I agree that flagging those kinds of crops would be a good idea. How do you see that happening?

Whenever I am on OF editing crop info/adding photos/etc I usually find duplicates and fix/cleanup stuff. It's not too difficult, but there are a lot of them and it gets tedious quickly. I think if we invite people to flag duplicates then we should first have a way for admins to quickly/easily merge crops.
As far as the flagging feature, I think we could have a flag dropdown on all crop pages that gives the users a few flagging options:

Inappropriate
Duplicate
Inaccurate
Needs photos
Needs description

Admins could have extra options:

Remove flags (after review)
Flag as featured crop (for the homepage hero images mentioned in #724)


Options for quick non-reapack install

Some scripts no longer work if you just use the source file - due to the use of provide and the use of find → create a default option?

Done for rec arm in folder and selected folder. Check other scripts


Special case chunk encoding for dict chunk store

This removes the special case copying of chunks in commit ( https://github.com/zarr-developers/zarr/commit/3c00d52356fb59968d05b8764b05a66c50457850 ). Instead this checks to see if a dict instance is being used to back chunk storage. If it is, this coerces the chunk data to bytes (if it is not already bytes). This avoids a copy for well-defined backends that already protect their data against unwanted tampering. Also continues to avoid copies when compressors and/or filters already return the data as bytes. However this will copy in any case where a non-bytes object (e.g. an ndarray) is placed in the chunk store (e.g. no filters/compressors are used or some specific filter is used at the end). This behavior for dict-based chunk stores is analogous to what DictStore does currently for stored values.
Note: This also extends to subclasses of dict, which include defaultdict and OrderedDict. It does not include ChainMap or UserDict; however, it's unclear whether we should be special casing those two as ChainMap is meant to provide a unified interface onto many MutableMappings of various kinds and UserDict is intended as a clean and simple alternative for subclassing dict.
TODO:

[x] Add unit tests and/or doctests in docstrings
[x] Add docstrings and API docs for any new/modified user-facing classes and functions
[x] New/modified features documented in docs/tutorial.rst
[x] Changes documented in docs/release.rst
[x] Docs build locally (e.g., run tox -e docs)
[x] AppVeyor and Travis CI passes
[x] Test coverage is 100% (Coveralls passes)

This is ready for review. Would be curious to hear thoughts on this. Basically this is meant to make working with dict objects as an Array chunk store a bit better behaved.

Would be great to hear your thoughts on this when you have a chance, @alimanfoo.

Adding to the v2.3 milestone so we can keep track of it. Happy to change that as needed.

Thoughts @zarr-developers/core-devs?

Hello @jakirkham! Thanks for updating the PR.

In the file zarr/core.py, following are the PEP8 issues :

Line 14:80: E501 line too long (82 > 79 characters)
Line 15:80: E501 line too long (81 > 79 characters)
Line 24:80: E501 line too long (84 > 79 characters)
Line 242:80: E501 line too long (81 > 79 characters)
Line 387:80: E501 line too long (85 > 79 characters)
Line 399:80: E501 line too long (93 > 79 characters)
Line 405:80: E501 line too long (90 > 79 characters)
Line 440:80: E501 line too long (82 > 79 characters)
Line 532:80: E501 line too long (84 > 79 characters)
Line 537:80: E501 line too long (81 > 79 characters)
Line 545:80: E501 line too long (82 > 79 characters)
Line 548:80: E501 line too long (85 > 79 characters)
Line 553:80: E501 line too long (89 > 79 characters)
Line 554:80: E501 line too long (85 > 79 characters)
Line 568:80: E501 line too long (85 > 79 characters)
Line 569:80: E501 line too long (90 > 79 characters)
Line 573:80: E501 line too long (86 > 79 characters)
Line 639:80: E501 line too long (83 > 79 characters)
Line 646:80: E501 line too long (84 > 79 characters)
Line 651:80: E501 line too long (81 > 79 characters)
Line 659:80: E501 line too long (82 > 79 characters)
Line 661:80: E501 line too long (84 > 79 characters)
Line 662:80: E501 line too long (83 > 79 characters)
Line 668:80: E501 line too long (85 > 79 characters)
Line 723:80: E501 line too long (81 > 79 characters)
Line 731:80: E501 line too long (83 > 79 characters)
Line 732:80: E501 line too long (84 > 79 characters)
Line 733:80: E501 line too long (86 > 79 characters)
Line 734:80: E501 line too long (82 > 79 characters)
Line 740:80: E501 line too long (87 > 79 characters)
Line 745:80: E501 line too long (86 > 79 characters)
Line 761:80: E501 line too long (89 > 79 characters)
Line 788:80: E501 line too long (89 > 79 characters)
Line 819:80: E501 line too long (82 > 79 characters)
Line 823:80: E501 line too long (89 > 79 characters)
Line 824:80: E501 line too long (85 > 79 characters)
Line 852:80: E501 line too long (86 > 79 characters)
Line 873:80: E501 line too long (89 > 79 characters)
Line 881:80: E501 line too long (89 > 79 characters)
Line 884:80: E501 line too long (87 > 79 characters)
Line 887:80: E501 line too long (86 > 79 characters)
Line 888:80: E501 line too long (84 > 79 characters)
Line 889:80: E501 line too long (86 > 79 characters)
Line 894:80: E501 line too long (89 > 79 characters)
Line 895:80: E501 line too long (85 > 79 characters)
Line 922:80: E501 line too long (88 > 79 characters)
Line 923:80: E501 line too long (85 > 79 characters)
Line 929:80: E501 line too long (89 > 79 characters)
Line 934:80: E501 line too long (86 > 79 characters)
Line 958:80: E501 line too long (83 > 79 characters)
Line 966:80: E501 line too long (85 > 79 characters)
Line 967:80: E501 line too long (81 > 79 characters)
Line 973:80: E501 line too long (85 > 79 characters)
Line 992:80: E501 line too long (86 > 79 characters)
Line 993:80: E501 line too long (85 > 79 characters)
Line 994:80: E501 line too long (83 > 79 characters)
Line 996:80: E501 line too long (87 > 79 characters)
Line 997:80: E501 line too long (86 > 79 characters)
Line 1015:80: E501 line too long (82 > 79 characters)
Line 1029:80: E501 line too long (84 > 79 characters)
Line 1073:80: E501 line too long (83 > 79 characters)
Line 1078:80: E501 line too long (81 > 79 characters)
Line 1087:80: E501 line too long (82 > 79 characters)
Line 1090:80: E501 line too long (82 > 79 characters)
Line 1091:80: E501 line too long (80 > 79 characters)
Line 1096:80: E501 line too long (89 > 79 characters)
Line 1097:80: E501 line too long (85 > 79 characters)
Line 1111:80: E501 line too long (84 > 79 characters)
Line 1116:80: E501 line too long (90 > 79 characters)
Line 1159:80: E501 line too long (85 > 79 characters)
Line 1165:80: E501 line too long (81 > 79 characters)
Line 1174:80: E501 line too long (88 > 79 characters)
Line 1175:80: E501 line too long (83 > 79 characters)
Line 1181:80: E501 line too long (85 > 79 characters)
Line 1196:80: E501 line too long (80 > 79 characters)
Line 1198:80: E501 line too long (80 > 79 characters)
Line 1206:80: E501 line too long (87 > 79 characters)
Line 1211:80: E501 line too long (90 > 79 characters)
Line 1252:80: E501 line too long (88 > 79 characters)
Line 1267:80: E501 line too long (82 > 79 characters)
Line 1271:80: E501 line too long (89 > 79 characters)
Line 1272:80: E501 line too long (85 > 79 characters)
Line 1291:80: E501 line too long (89 > 79 characters)
Line 1301:80: E501 line too long (90 > 79 characters)
Line 1322:80: E501 line too long (88 > 79 characters)
Line 1335:80: E501 line too long (89 > 79 characters)
Line 1338:80: E501 line too long (87 > 79 characters)
Line 1343:80: E501 line too long (89 > 79 characters)
Line 1344:80: E501 line too long (85 > 79 characters)
Line 1369:80: E501 line too long (86 > 79 characters)
Line 1370:80: E501 line too long (85 > 79 characters)
Line 1376:80: E501 line too long (89 > 79 characters)
Line 1381:80: E501 line too long (90 > 79 characters)
Line 1405:80: E501 line too long (88 > 79 characters)
Line 1418:80: E501 line too long (85 > 79 characters)
Line 1419:80: E501 line too long (81 > 79 characters)
Line 1425:80: E501 line too long (85 > 79 characters)
Line 1493:80: E501 line too long (86 > 79 characters)
Line 1494:80: E501 line too long (84 > 79 characters)
Line 1497:80: E501 line too long (87 > 79 characters)
Line 1498:80: E501 line too long (87 > 79 characters)
Line 1538:80: E501 line too long (90 > 79 characters)
Line 1598:80: E501 line too long (83 > 79 characters)
Line 1623:80: E501 line too long (80 > 79 characters)
Line 1649:80: E501 line too long (87 > 79 characters)
Line 1663:80: E501 line too long (84 > 79 characters)
Line 1686:80: E501 line too long (88 > 79 characters)
Line 1689:80: E501 line too long (88 > 79 characters)
Line 1691:80: E501 line too long (88 > 79 characters)
Line 1693:80: E501 line too long (88 > 79 characters)
Line 1704:80: E501 line too long (86 > 79 characters)
Line 1739:80: E501 line too long (82 > 79 characters)
Line 1741:80: E501 line too long (81 > 79 characters)
Line 1742:80: E501 line too long (82 > 79 characters)
Line 1761:80: E501 line too long (80 > 79 characters)
Line 1802:80: E501 line too long (87 > 79 characters)
Line 1863:80: E501 line too long (86 > 79 characters)
Line 1882:80: E501 line too long (81 > 79 characters)
Line 1913:80: E501 line too long (81 > 79 characters)
Line 2064:80: E501 line too long (89 > 79 characters)
Line 2065:80: E501 line too long (88 > 79 characters)
Line 2207:80: E501 line too long (87 > 79 characters)
Line 2233:80: E501 line too long (86 > 79 characters)

In the file zarr/tests/test_core.py, following are the PEP8 issues :

Line 17:80: E501 line too long (87 > 79 characters)
Line 24:80: E501 line too long (82 > 79 characters)
Line 93:80: E501 line too long (80 > 79 characters)
Line 233:80: E501 line too long (81 > 79 characters)
Line 375:80: E501 line too long (89 > 79 characters)
Line 669:80: E501 line too long (86 > 79 characters)
Line 743:80: E501 line too long (88 > 79 characters)
Line 776:80: E501 line too long (88 > 79 characters)
Line 872:80: E501 line too long (100 > 79 characters)
Line 882:80: E501 line too long (100 > 79 characters)
Line 891:80: E501 line too long (97 > 79 characters)
Line 938:80: E501 line too long (87 > 79 characters)
Line 985:80: E501 line too long (85 > 79 characters)
Line 988:80: E501 line too long (84 > 79 characters)
Line 991:80: E501 line too long (87 > 79 characters)
Line 1006:80: E501 line too long (86 > 79 characters)
Line 1021:80: E501 line too long (84 > 79 characters)
Line 1039:80: E501 line too long (86 > 79 characters)
Line 1058:80: E501 line too long (85 > 79 characters)
Line 1062:80: E501 line too long (82 > 79 characters)
Line 1066:80: E501 line too long (84 > 79 characters)
Line 1080:80: E501 line too long (87 > 79 characters)
Line 1099:80: E501 line too long (84 > 79 characters)
Line 1120:80: E501 line too long (85 > 79 characters)
Line 1130:80: E501 line too long (87 > 79 characters)
Line 1139:80: E501 line too long (86 > 79 characters)
Line 1165:80: E501 line too long (82 > 79 characters)
Line 1732:80: E501 line too long (86 > 79 characters)
Line 1772:80: E501 line too long (83 > 79 characters)


Maybe need more smart snapping or routing?

Not so long ago find some "not so good" thing that may be improved (i hope, sure :)), this thing is working snapping near cross points where 2 road one under another - example here:
https://graphhopper.com/maps/?point=52.522318%2C13.370871&point=52.523252%2C13.369986&vehicle=car
and now question - is we need to snap this point to upper road or mean lower road?
if we have next points like this:
https://graphhopper.com/maps/?point=52.523252%2C13.369986&point=52.52366%2C13.368891&vehicle=car
in 99% of cases we mean lower road
if next point like this:
https://graphhopper.com/maps/?point=52.523252%2C13.369986&point=52.523484%2C13.371139&vehicle=car
in 99% of cases we mean that point must snap to upper road
So that's why maybe improve snap logic, coz if point comes not so close to needed road routing result may be so bad with long detour for example :)

What is the use case behind this requirement? Do you want to find out the taken route from some point or for navigation? Then probably the map matching component is more appropriate?

not for navigations, for route geometry from array of points, if some point like in sample has some coordinate lapse then route may calc wrong

@FallenKain, you could also have a look at the PointHint feature. https://graphhopper.com/maps/?point=52.522318%2C13.370871&point=52.523252%2C13.369986&vehicle=car&point_hint=""&point_hint=Tunnel Tiergarten Spreebogen

@boldtrn not so good if sender didn't know is there a tunnel, brigde or simple road

for example we have a car that travels in city , some db has gps tracks of this car, we want to reproduce how it travels, we don't know all of info, only array of points with some lapse of gps tracker, that's all, in this situation we can get wrong routing result coz point snapped to wrong road

for example we have a car that travels in city , some db has gps tracks of this car, we want to reproduce how it travels,

This is a perfect example for map matching. See @karussell comment. You can input a gpx and get the travelled route.


A strange bug in fonts generation, since 1.8.0

Hi guys,
I have created a simple project that shows this bug.
Please, clone it:
git clone https://github.com/serhii-k/test.git
It contains the ttf-file that contains only one character 'm', for the simplicity of this test.
When I generate a font, specifying:
parameter.characters = "m";
Then
font.draw(batch, "m", 300, 300);
displays this result:

A workaround is to specify the characters parameter as "mm":
parameter.characters = "mm";
Then, the font is renderred correctly:

This issue was first seen since libGDX 1.8.0.
Kind regards,
Serhii

The described workaround doesn't work well on every platform. E.g. on some iOS simulators that workaround doesn't give consistent results. On some, 3 same characters are required, on some, 2 characters are too much... You can't guarantee which characters need this workaround. Etc.
At this stage, instead of using FreeTypeFontGenerator.generateFont() I'm using FreeTypeFontGenerator.generateGlyphAndBitmap() for each separate character (I'm using this technique to generate app icons from a ttf-font). Then I'm packing pixmaps obtained from bitmaps with PixmapPacker to get a TextureAtlas with icons.
It looks that the generateGlyphAndBitmap method works consistently on different platforms.

Creating example code as described here instead of whole repo will make it easier for others to help.
That said, looking at your code: does this happen with smaller glyph sizes? 150 is pretty big.

Darkyenus I can confirm that this bug exists.
This bug does not occur, when FreeType is in incremental mode.
Here is my reproduction case:
public class FreeTypeOneCharTest extends GdxTest {

	SpriteBatch batch;
	BitmapFont font;
	FreeTypeFontGenerator generator;
	FreeTypeBitmapFontData data;

	public void create () {
		batch = new SpriteBatch();

		generator = new FreeTypeFontGenerator(Gdx.files.internal("data/arial.ttf"));

		FreeTypeFontParameter param = new FreeTypeFontParameter();
		param.incremental = false; //Switch to true and it will magically work
		param.size = 20;
		param.color = Color.GREEN;
		param.shadowColor = Color.YELLOW;
		param.shadowOffsetX = 6;
		param.shadowOffsetY = 6;
		param.characters = "s";

		data = new FreeTypeBitmapFontData();

		font = generator.generateFont(param, data);
	}

	public void render () {
		Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);

		batch.begin();
		font.draw(batch, "s", 10, 300);
		batch.end();
	}

	public void resize (int width, int height) {
		batch.getProjectionMatrix().setToOrtho2D(0, 0, width, height);
	}

	@Override
	public void dispose () {
		generator.dispose();
		font.dispose();
		batch.dispose();
	}

}

Try viewing generated textures to check if the problem is in the rasterization or in rendering. This may help with that. Just put it in a if(Gdx.input.justTouched()) block in the render method.

Thanks for the snippet, will try debugging this issue tomorrow.

Oh wow, @intrigus I can confirm that setting incremental to true resolves the issue in my example too.
I'm sorry @Darkyenus I sould have definitely added a simple barebones class.
In my code I was taking textures from the generated font with icons and rendered them. So I suppose the problem is in the rasterization. I will check this assumption a little later.

I can confirm that FreeType itself renders the glyphs correctly. (The pixmaps returned by https://github.com/libgdx/libgdx/blob/master/extensions/gdx-freetype/src/com/badlogic/gdx/graphics/g2d/freetype/FreeType.java#L607 are the same for incremental=true and false)
So the problem seems to be in the packaging of the glyphs to the texture, will look into this later.

So the problem seems to be in the packaging of the glyphs to the texture, will look into this later.

Yes, this is absolutely true. In this post I will provide some observed examples.
Here's a barebones version of my code. Please, download the test-font here:
https://github.com/serhii-k/test/blob/master/android/assets/test-font.ttf
import com.badlogic.gdx.ApplicationAdapter;
import com.badlogic.gdx.Gdx;
import com.badlogic.gdx.backends.lwjgl.LwjglApplication;
import com.badlogic.gdx.graphics.Color;
import com.badlogic.gdx.graphics.GL20;
import com.badlogic.gdx.graphics.g2d.BitmapFont;
import com.badlogic.gdx.graphics.g2d.SpriteBatch;
import com.badlogic.gdx.graphics.g2d.TextureRegion;
import com.badlogic.gdx.graphics.g2d.freetype.FreeTypeFontGenerator;
import com.badlogic.gdx.utils.Array;

public class TestFont extends ApplicationAdapter {

    SpriteBatch batch;
    BitmapFont font;

    Array<TextureRegion> regions;

    @Override
    public void create() {
        batch = new SpriteBatch();

        FreeTypeFontGenerator generator = new FreeTypeFontGenerator(Gdx.files.internal("test-font.ttf"));
        FreeTypeFontGenerator.FreeTypeFontParameter parameter = new FreeTypeFontGenerator.FreeTypeFontParameter();
        parameter.characters = "mm";
        parameter.size = 150;
        parameter.gamma = 0.5f;
        font = generator.generateFont(parameter);
        generator.dispose();

        regions = font.getRegions();
        Gdx.app.log("getRegions.size", regions.size + "");
    }

    @Override
    public void render() {
        Gdx.gl.glClearColor(0.2f, 0.2f, 0.2f, 1f);
        Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);
        batch.begin();

        batch.setColor(Color.WHITE);
        font.draw(batch, "m", 100, 450);

        batch.setColor(Color.BLUE);
        batch.draw(regions.get(0), 10, 10, regions.get(0).getRegionWidth(), regions.get(0).getRegionHeight());

        if (regions.size > 1) {
            batch.setColor(Color.RED);
            batch.draw(regions.get(1), 10 + regions.get(0).getRegionWidth() + 10, 10,
                    regions.get(1).getRegionWidth(), regions.get(1).getRegionHeight());
        }

        batch.end();
    }

    public static void main (String[] args) throws Exception {
        new LwjglApplication(new TestFont());
    }
}

if parameter.characters = "m"; only one 256x256 region is generated. But, look, it has some awkward square... Strange, ha? The actual character is placed below this square and partially is outside the region...

If parameter.characters = "mm"; two 256x256 regions are generated. The first one is same as in the first case, the second one has a correctly placed character.

So, the issue is really in the packaging of the glyphs to the texture as @intrigus has noted. In the example, the strange square appears.
If the font size is lower, let's say 32 pixels instead of 150, the issue remains the same. Only the generated regions are smaller.

Incremental mode uses different pixmap packer implementation, so that explains it.

Hi. Any progress on this? I have been having this issue since upgrading to 1.9.4 from 1.6.5, when using the ttf from font awesome.
Some observations:

Sometimes the icons are rendered correctly on the desktop version and not on the android version. The android version has a higher dpi so the icons will have more pixels.
Sometimes when I add just 2 to the parameter.size, they render correctly.

Hi. Has there been any progress on this? I'm packing my images using PixmapPackers SkylineStrategy and its giving me this problem. I can't use GuillotineStrategy because the packing isn't tight enough, making images overflow.


New version: Microsoft.PowerShell.Preview version <IP_ADDRESS>

Result: Installation Successful

Manifest
Add/Remove Programs

Name
PowerShell
PowerShell 7-preview-x64

Version
<IP_ADDRESS>
<IP_ADDRESS>

Publisher
Microsoft Corporation
Microsoft Corporation

ProductCode

{3AC4078C-B562-4575-B4C9-ABDAD8F6A760}

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 4716
Microsoft Reviewers: Open in CodeFlow

Close with reason: Superseded by #91947;


NestJS E2E tests failing when using vitest (when using sequelize query)

Describe the bug
I have a project using NestJS, Sequelize, Sequelize-Typescript, and Testcontainers. The E2E test is responsible for setting up some data and testing an API.
By relying on the sequelize-typescript project, when querying some data, the result should be an object populated. It works as desired when using jest, but after converting my tests to vitest, it's not working as expected.
My query result is returned with some empty fields (although the object brings the data I want in some hidden property).
{
  subject: Subject {
    dataValues: {
      id: '59a59521-665c-4a21-86ea-8044da9c33ac',
      description: 'test',
      orderId: '56b6f0aa-c592-44d6-b0b7-e02dc1910a03',
      createdAt: 2024-08-19T18:38:18.962Z,
      updatedAt: 2024-08-19T18:38:18.962Z,
      observations: [Array]
    },
    _previousDataValues: {
      id: '59a59521-665c-4a21-86ea-8044da9c33ac',
      description: 'test',
      orderId: '56b6f0aa-c592-44d6-b0b7-e02dc1910a03',
      createdAt: 2024-08-19T18:38:18.962Z,
      updatedAt: 2024-08-19T18:38:18.962Z,
      observations: [Array]
    },
    uniqno: 1,
    _changed: Set(0) {},
    _options: {
      isNewRecord: false,
      _schema: null,
      _schemaDelimiter: '',
      include: [Array],
      includeNames: [Array],
      includeMap: [Object],
      includeValidated: true,
      attributes: [Array],
      raw: true
    },
    isNewRecord: false,
    observations: undefined,
    id: undefined,
    description: undefined,
    orderId: undefined
  }
}

If I try to get subject.id, subject.description or anything, it returns undefined.
What I am missing here? Why did it work with jest?
Reproduction
To reproduce the bug, here's the repo: https://github.com/studiojms/nest-vitest-fhir-sample
The E2E test is here: https://github.com/studiojms/nest-vitest-fhir-sample/blob/main/test/app.e2e-spec.ts
This is the repo with jest setup for E2E (working): https://github.com/studiojms/nest-vitest-fhir-sample/tree/2042a1c32bf5c9e74e352b1bf2f67b07986d9afb
The problem happens at this line when it tries to check the query result: https://github.com/studiojms/nest-vitest-fhir-sample/blob/main/src/subject/subject.service.ts#L30
To execute the project, run:
npm run test:e2e

or
yarn test:e2e

System Info
System:
    OS: macOS 14.6.1
    CPU: (8) arm64 Apple M3
    Memory: 68.89 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.15.1 - ~/.nvm/versions/node/v20.15.1/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.15.1/bin/yarn
    npm: 10.7.0 - ~/.nvm/versions/node/v20.15.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 127.0.6533.120
    Safari: 17.6
  npmPackages:
    @vitest/coverage-v8: ^2.0.5 => 2.0.5
    vitest: ^2.0.5 => 2.0.5

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

How is this different from https://github.com/vitest-dev/vitest/issues/6342? If the underlying issue is suspected to be same fhir, then I think you can add details there.

@hi-ogawa this issue has nothing to do with #6342 , it's only related to the E2E tests, not fhir package. I'm just using the same project to reproduce the error

Thanks for closing the other issue :+1:
Now looking at your vitest config https://github.com/studiojms/nest-vitest-fhir-sample/blob/2353826c6866e40c016c3b3be860a007f267f189/test/vitest.config.mts#L16, it might be necessary to enable swc's typescript decorator transpilation. There was examples/nest in the past (now it's removed though) and it looked like this:
https://github.com/vitest-dev/vitest/blob/d68a73908ef00f1b7c6584ef015657b3a11416bf/examples/nestjs/vitest.config.ts#L6-L18

@hi-ogawa thanks for the tip, I tried adding the params to my vitest.config file, but unfortunately, it didn't produce the expected output, tests keep failing with the same error...

Hmm, I just noticed Nest.js has documentation for Vitest https://docs.nestjs.com/recipes/swc#vitest and it looks like just using swc.vite is meant to work.
Then I would suspect sequenlize-typescript is having some issues with decorator on Vitest. It looks like there's an open issue there https://github.com/sequelize/sequelize-typescript/issues/1711 Probably tacking that reproduction might be an easier way to understand the issue.

I had the same issue and I fixed it by adding useDefineForClassFields: false to the jsc.transform, my full Swc config in vitest is:
swc.vite({
  module: { type: 'es6' },
  jsc: {
    transform: {
      useDefineForClassFields: false,
    },
  },
}),

As a note, useDefineForClassFields to false is the default swc config for @nestjs/cli

Thank you so much @Davide-Gheri , that solved the problem


Help wanted, how do I install the 3mf plugin

Thank you very much for this program it is very handy!
I got  f3d working on my debian based system and the files render with a 3d model but I would like to get 3mf files to do the same thing. I see that there are plugins available, but I have no idea how to install them. There is one that I saw on here that add 3mf support. Could someone walk me through on how to install the plugin and get the plugin it working?

How did you install F3D on Debian? (Repository, .deb file, manual install?)
Can you copy the output of f3d --version?

I did the manual install.
`$ f3d --version
F3D 2.5.0
F3D - A fast and minimalist 3D viewer
Version: 2.5.0.
Build date: 2024-07-01 20:59:05.
Build system: Linux 64-bits.
Compiler: GNU 8.4.0.
External rendering module: ON.
Raytracing module: OFF.
VTK version: 9.3.0-3457-gecf3646d64 (date: 20240620).
Copyright (C) 2019-2021 Kitware SAS.
Copyright (C) 2021-2024 Michael Migliore, Mathieu Westphal.
License BSD-3-Clause.
By Michael Migliore, Mathieu Westphal and Joachim Pouderoux.
`

Ok then you should have 3MF support.
Can you confirm the command f3d --readers-list | grep 3mf returns something?

~$ f3d --readers-list | grep 3mf 3MF        assimp  3D Manufacturing Format                 3mf   model/3mf
With 3mf in red letters.

Then you have 3mf support, the plugin is already loaded.
What happens when you try to open the file?

So it works, you are just missing the tumbnail, is that correct ?

What do you get if you run:
mimetype path/to/file.3mf ?

So it works, you are just missing the tumbnail, is that correct ?
Correct

I think it's related to a crash issue at exit in the assimp library that has been fixed since then.
@mwestphal do you confirm the fix is not included in F3D 2.5?

Related: https://github.com/f3d-app/f3d/issues/1301
@mitchngwen can you try the nightly build?

Sure, I'll give it a try and let you know.

Not related to #1301 , @mitchngwen is missing the mime type for some reason . I'll need to check which package provides mimetypes in debian

No difference.

Can you share the failing file @mitchngwen ?

I think the package you need is media-types. apt install media-types.

I can not install it.
Package media-types is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

Gridfinity 2x4x3 Container.zip
All 3mf files are like this.

You need to find out how to install mimetypes. Please share your exact distribution if you want me to find the name of the package.

TLDR: apt install shared-mime-info
Longer version:
Using debian12 docker image, here is what I get:
apt update
apt install libfile-mimeinfo-perl wget unzip
wget https://github.com/user-attachments/files/16685834/Gridfinity.2x4x3.Container.zip
unzip Gridfinity.2x4x3.Container.zip
mimetype mimetype Gridfinity\ 2x4x3\ Container/Gridfinity\ 2x4x3\ Container.3mf

Gridfinity 2x4x3 Container/Gridfinity 2x4x3 Container.3mf: model/3mf

If I check /usr/share/mime  I find that there is actually two definition of mime types for 3mf:
packages/freedesktop.org.xml and  model/3mf.xml.
These have been installed by the packages:
shared-mime-info
But this package should be installed already.

@Meakk maybe we should add that info in the DESKTOP_INTEGRATION.md ?

I think the package you need is media-types. apt install media-types.

I can not install media types.

Feren OS 2023.04
I seem to be missing  model/3mf.xml

This has been fixed. I had to manually add 3mf mime type. What a pain. Some weirdness with my system I guess. Thank you for all your help, guys.

It is possible the 3mf was added quite recently in freedesktop.org.xml and is not present in your version. We could add a f3d-3mf-formats.xml but we are trying to avoid duplicating mime types with the one provided by the system.

Ah, perhaps make it available in case someone else runs into this issue? Thank you guys again for being so helpful.


Add Chroma Palette

Chroma Palette 🎨 is a chrome extension to extract, pick and manage the dominant color palette from the visible area of any webpage or pick and manage specific color palettes using an uploaded image.

@markodenic can you please review and merge?

@bymayanksingh We already have a lot of color palette resources. Please, consider contributing to some other category with fewer resources.


Add newline to output if stdout/err is a tty

If stdout/stderr is a TTY, emit a newline so that running the CLI from the prompt still looks nice. Piping into another process (head, less, etc.) will still omit the newline.

Ah, excellent. This explains why I added the \n and @victorquinn removed it again in 7e912c6558a3887ed81245a2bdd637b9f217031e ;D

Indeed, this is an ideal solution, thanks @codeaholics

You're welcome :-)


Potential readability improvements

Looking though, and felt like some could be worded slightly different.

Only meant to submit https://github.com/Yellow-Dog-Man/Locale/pull/586/commits/0891e113e8a4fabf52136e499e280d0cae10fa85


PICO_MALLOC_TRACK_PEAK, peak allocation tracking for malloc

This PR implements a peak allocated bytes tracking option for malloc.
To enable:
target_compile_definitions(my_project PRIVATE PICO_MALLOC_TRACK_PEAK=1)

Usage: determining the peak allocated bytes during a block of code / call-stack:
malloc_reset_peak();
unsigned char* m1 = malloc(1000);
unsigned char* m2 = malloc(2000);
unsigned char* m3= malloc(3000);
// this simply prevents the compiler from optimizing out the malloc calls:
if (m1 == NULL || m2 == NULL || m3 == NULL) {
    printf("malloc failed");
}
free(m1);
free(m2);
free(m3);
printf("malloc peak: %u bytes", malloc_peak_bytes);

which will print:
malloc peak: 6024 bytes

Looks interesting

Some additional info about mallinfo, for folks interested in the subject of heap usage: https://forums.raspberrypi.com/viewtopic.php?p=2071251#p2071251

The fields that appear to matter are arena and either of uordblks or fordblks. arena does not cover all of RAM, and in fact appears to be initially 0, increased as necessary by invocations of malloc(). From what I've seen, the value of arena can be viewed as a high watermark for heap usage during the lifetime of the app -- but I don't know if that is always true after an app has done a lot of malloc()-ing and free()-ing over a long period of time.

uordblks is the amount of heap currently allocated -- space that has been malloc()-ed but not yet free()-ed. fordblks is the size of free space in the arena. I haven't always checked, but every time I have looked, it's been true that arena == uordblks + fordblks.

Some additional info about mallinfo, for folks interested in the subject of heap usage:
https://forums.raspberrypi.com/viewtopic.php?p=2071251#p2071251

It should be pointed out that the author of that forum post (that would be me) was not sure if mallinfo is always guaranteed to work that way. Said author is still not sure, it all depends on how malloc() and mallinfo() work. If an app does a lot of malloc()ing and free()ing, it is not hard to imagine that non-contiguous allocated and freed portions of the heap could result in such a way that the mallinfo numbers don't add up so neatly.
It's worth looking into, because malloc_peak_bytes would be valuable. The truth is in the newlib source code. It would also help to test an app that causes some heap fragmentation with malloc() and free(), and then check what mallinfo says.


Programs: update dashboard display for programs

As a learner, I would like to see the programs I've enrolled in on the dashboard (whether they are direct enrollments in the program, or enrollments in a course that belongs to a program).
This is blocked since the API needs to be amended first to return program enrollments properly - see #719 . - It made more sense to pull the program enrollments out into a separate API, so the dashboard code will need to call that API instead.
Designs and Mockups
Programs on the dashboard: https://projects.invisionapp.com/d/main?origin=v7#/console/21535972/468431706/preview?scrollOffset=0
Program drawer: https://projects.invisionapp.com/d/main?origin=v7#/console/21535972/468431707/preview?scrollOffset=0
Acceptance Criteria:

[ ] Use the updated API to display program enrollments alongside course enrollments.
[ ] Courses that belong to a program should be displayed in the drawer.
[ ] Other courses in the program should not be displayed in the drawer - just the things that the learner is enrolled in.
[ ] Programs should include tags as appropriate (see mockups).
[ ] Standalone courses should display as they were.

Related Issues
#704

Two minor questions (probably for @pdpinch):

Where should the program image come from? My plan currently is to use the image from the first course - is there somewhere else this should be set?
What does the hamburger menu do? For courses, this gives you a menu to adjust email settings and unenroll from the course. I can work those out so that they make changes to all the associated course enrollments for the program (which may be a bit dangerous for the unenroll option). Alternatively, we can skip that menu, or have it pull up the course drawer.

Where should the program image come from? My plan currently is to use the image from the first course - is there somewhere else this should be set?

These change very seldom. If it's not too much trouble, I would pull it from the CMS, like we do in xPRO.

What does the hamburger menu do? For courses, this gives you a menu to adjust email settings and unenroll from the course. I can work those out so that they make changes to all the associated course enrollments for the program (which may be a bit dangerous for the unenroll option). Alternatively, we can skip that menu, or have it pull up the course drawer.

I think it should pull up the drawer.


Use literal types to allow discriminated unions

See https://basarat.gitbooks.io/typescript/docs/types/discriminated-unions.html - this will allow TypeScript to narrow types when checking equality on the type field.

@jackbearheart Would be awesome if you could merge this!


replace hard code content with dynamic market data, update list template

@mannickutd @norm @TheUKDave

👍  waiting for successful build.


Add org-mode support for internal links (incl. update documentation)

Similar to the problem with image url's, the url's of internal link are also not handled well by ox-html for correct use with nikola. I copied and slightly the solution for handling image urls so that internal urls will also be handled correctly now with org-mode.
In addition to changing the implementing the code I updated the documentation in the README file for the plugin.

I have implemented your suggestions/requirements for accepting the pull request

I don’t know anything about org-mode, but could you show a more specific case where this style of linking would actually be useful? Can’t you just use links like 'foo.jpg' and get the correct output? And as for the part with .org specifically, why would you want to use that?


Add shortcut to switching to sovereign cloud within the tenant view

This allows users to quickly change their sovereign cloud setting without having to go into the user settings.
The shortcut lives here under the gear icon:

There is also a command palette command titiled: Azure: Configure Sovereign Cloud
This brings up the below quick pick. The names correspond with the names on the setting.  When an item is selected the setting microsoft-sovereign-cloud.environment is changed to the corresponding value.
(I included the generic azure one so user could switch back easily)

does this bring up a quick pick or something

Can you put a screenshot of the quick pick please

Yup I'll edit the description to add that

just some quick feedback: Maybe we can put (Default) in the description of the "Azure" normal option?

just some quick feedback: Maybe we can put (Default) in the description of the "Azure" normal option?

Yup I'll do that!

Also I think we'll merge this after we merge the tenant view PR. There might be other small features like this on top of the tenant view that we want to add.

Also I think we'll merge this after we merge the tenant view PR. There might be other small features like this on top of the tenant view that we want to add.

Sounds good I'll change the branch to main before we merge this.


better RegEx which finds error/warning kinds of lines

    regex = (
        r"^(?P<filename>.+?):(?P<line>\d+):(?P<col>\d+):\s"
        r"(?P<message>((?P<error>error:)|(?P<warning>warning:)).+)\s\[(?P<code>\w+)]"
    )

this way SublimeLinter can detect lines 'kinds'.
also at and, content of ?P<code>, changed to \w+.

Reasonable.  I was very defensive here because quick-lint-js is very young, and thought maybe everything changes here rather quickly.  But
        r"((?P<error>error:)|(?P<warning>warning:)\s)?(?P<message>.+)\s"

is probably better.  Because if we "detect" or parse for the kind, we should not also leave it in the message but extract it, no?

Why the w+ though?

https://regex101.com/r/U8y6xs/1

Why \w+ . To make it safer if future version appears with output 'Error line [some text here]'.