[bug]  proc macro panicked

Describe the bug
error: proc macro panicked
--> src/lib.rs:19:14
|
19 |         .run(tauri::generate_context!())
|              ^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: message: failed to resolve ACL: SetPermissionNotFound { permission: "set-tray-icon-state", set: "default" }
error: could not compile tauriv2 (lib) due to 1 previous error
After configuring "permissions": [
"positioner:default"
] and running pnpm tarui dev, this error message will appear, but if it is not configured, the following method cannot be used
import { moveWindow, Position } from '@tauri-apps/plugin-positioner';
moveWindow(Position.TopRight);
Reproduction
No response
Expected behavior
Ability to use tauri-plugin-positioner plugin
Full tauri info output
[✔] Environment
    - OS: Mac OS 15.0.0 x86_64 (X64)
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.81.0 (eeb90cda1 2024-09-04)
    ✔ cargo: 1.81.0 (2dbb1af80 2024-08-20)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-apple-darwin (default)
    - node: 18.20.4
    - pnpm: 9.11.0
    - yarn: 1.22.22
    - npm: 10.7.0

[-] Packages
    - tauri 🦀: 2.0.2
    - tauri-build 🦀: 2.0.1
    - wry 🦀: 0.44.1
    - tao 🦀: 0.30.2
    - tauri-cli 🦀: 1.6.2
    - @tauri-apps/api : 2.0.0-rc.5 (outdated, latest: 2.0.2)
    - @tauri-apps/cli : 2.0.0-rc.16 (outdated, latest: 2.0.2)

[-] Plugins
    - tauri-plugin-shell 🦀: 2.0.0-rc.3
    - @tauri-apps/plugin-shell : 2.0.0-rc.1 (outdated, latest: 2.0.0)
    - tauri-plugin-positioner 🦀: 2.0.1
    - @tauri-apps/plugin-positioner : 2.0.0

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
No response

Did you add the tray-icon feature flag to the plugin as shown in the setup guide here? https://v2.tauri.app/plugin/positioner/ - or actually this sounds more like you didn't add the rust part of the plugin at all since the permissions should be detected regardless of the feature flag 🤔
Also, you have a few tauri packages still in their rc versions according to the tauri info output, that may be worth to look at too.
If my message doesn't help, could you please create a minimal reproduction for us to try?

Thanks for your help, the problem has been solved

Thanks for your help, the problem has been solved

I'm currently facing the same issue.
I added the permissions as per the documentation but still won't work.
Can you please explain how to solve this issue?
Thanks

Im having the same issue. I have followed the instructions, but still getting:
error: proc macro panicked
  --> src/lib.rs:21:14
   |
21 |         .run(tauri::generate_context!())
   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = help: message: failed to resolve ACL: SetPermissionNotFound { permission: "set-tray-icon-state", set: "default" }

error: could not compile `my_app` (lib) due to 1 previous error

Here is my code:
lib.rs
#[cfg_attr(mobile, tauri::mobile_entry_point)]
pub fn run() {
    tauri::Builder::default()
        .plugin(tauri_plugin_dialog::init())
        .plugin(tauri_plugin_clipboard_manager::init())
        .plugin(tauri_plugin_fs::init())
        .plugin(tauri_plugin_shell::init())
        // This is required to get tray-relative positions to work
        .setup(|app| {
            #[cfg(desktop)]
            {
              app.handle().plugin(tauri_plugin_positioner::init());
                tauri::tray::TrayIconBuilder::new()
                  .on_tray_icon_event(|tray_handle, event| {
                    tauri_plugin_positioner::on_tray_event(tray_handle.app_handle(), &event);
                  })
                  .build(app)?;
            }
          Ok(())
        })
        .run(tauri::generate_context!())
        .expect("error while running tauri application");
}

In capabilities/default.json I have "positioner:default"
And finally, my Cargo.toml
[package]
name = "my_app"
version = "0.1.0"
description = "A Tauri App"
authors = ["you"]
edition = "2021"

[lib]
name = "my_app_lib"
crate-type = ["staticlib", "cdylib", "rlib"]

[build-dependencies]
tauri-build = { version = "2.0.0", features = [] }

[dependencies]
tauri = { version = "2.0.0", features = ["macos-private-api", "image-png", "tray-icon", "image-ico"] }
tauri-plugin-shell = "2"
serde = { version = "1", features = ["derive"] }
serde_json = "1"
tauri-plugin-fs = "2"
tauri-plugin-clipboard-manager = "2.0.1"
tauri-plugin-dialog = "2"
tauri-plugin-positioner = { version = "2", features = ["tray-icon"] }

[target.'cfg(not(any(target_os = "android", target_os = "ios")))'.dependencies]

Did I miss something in the instructions?

Thanks for your help, the problem has been solved

I'm currently facing the same issue. I added the permissions as per the documentation but still won't work. Can you please explain how to solve this issue?
Thanks

Change positioner:default in your capabilities json to:
"positioner:allow-move-window",
"positioner:allow-set-tray-icon-state"

That worked for us!


dialog-list in desktop-mode as a sidebar

Is your feature request related to a problem? Please describe.
is leading to better UX....touches #29
Describe the solution you'd like
would be nice when it stayes open in desk but closes on touch on target in mobile....
Describe alternatives you've considered
keeping it the mobile way, duh!

done


Problems with porting new boards

Problem
When porting a new board it is possible to get into a situation where you have submitted mbed-os support and gotten mbed-os support mainlines, but not gotten mbed-ls support. It results in an in-ability to run tests.
example
Delta board, currently it is listed in mbed target --supported as DELTA_DFBM_NQ620, but plugging one in and running detect or tests results in the following
mbedls
C:\cygwin\home\ausbla01\mbed\ci-test-shield>mbedls
+---------------+----------------------+-------------+-------------+--------------------------+-----------------+
| platform_name | platform_name_unique | mount_point | serial_port | target_id                | daplink_version |
+---------------+----------------------+-------------+-------------+--------------------------+-----------------+
| unknown       | None[0]              | D:          | unknown     |<PHONE_NUMBER>0164481E74B074 | 0310            |
+---------------+----------------------+-------------+-------------+--------------------------+-----------------+

mbed detect
C:\cygwin\home\ausbla01\mbed\ci-test-shield>mbed detect

[mbed] Detected None, port None, mounted D:
[mbed] Supported toolchains for None
Traceback (most recent call last):
  File "C:\cygwin\home\ausbla01\mbed\ci-test-shield\mbed-os\tools\detect_targets.py", line 77, in main
    print mcu_toolchain_matrix(platform_filter=r'^'+mut['mcu']+'$')
TypeError: cannot concatenate 'str' and 'NoneType' objects
[ERROR] cannot concatenate 'str' and 'NoneType' objects
[mbed] ERROR: "python" returned error code 1.
[mbed] ERROR: Command "python -u C:\cygwin\home\ausbla01\mbed\ci-test-shield\mbed-os\tools\detect_targets.py" in "C:\cygwin\home\ausbla01\mbed\ci-test
-shield"

mbed test
and trying to run mbed test -n mbed-os-* results in the following
Build successes:
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-BUILD
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-BASIC_TEST
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-BASIC_TEST_DEFAULT
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_ASYNC_VALIDATE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_CONTROL_ASYNC
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_CONTROL_REPEAT
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_SELECTION
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_SETUP_FAILURE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CASE_TEARDOWN_FAILURE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-CONTROL_TYPE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-MINIMAL_ASYNC_SCHEDULER
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-MINIMAL_SCHEDULER
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-TEST_ASSERTION_FAILURE_TEST_SETUP
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-TEST_SETUP_CASE_SELECTION_FAILURE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-FEATURES-FRAMEWORKS-UTEST-TESTS-UNIT_TESTS-TEST_SETUP_FAILURE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-RTOS-RTX-TARGET_CORTEX_M-TESTS-MEMORY-HEAP_AND_STACK
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-INTEGRATION-BASIC
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-MBED-ATTRIBUTES
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-MBED-CALL_BEFORE_MAIN
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-MBED-CPP
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-MBED-DIV
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-BASIC
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-ISR
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-MAIL
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-MALLOC
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-MUTEX
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-QUEUE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-SEMAPHORE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-SIGNALS
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-THREADS
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBEDMICRO-RTOS-MBED-TIMER
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-C_STRINGS
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-DEV_NULL
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-ECHO
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-GENERIC_TESTS
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-LP_TIMEOUT
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-RACE_TEST
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-RTC
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-STL_FEATURES
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-TICKER
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-TIMEOUT
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-WAIT_US
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_FUNCTIONAL-CALLBACK
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_FUNCTIONAL-CALLBACK_BIG
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_FUNCTIONAL-CALLBACK_SMALL
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_FUNCTIONAL-FUNCTIONPOINTER
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_HAL-LP_TICKER

Build skips:
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-MEM_TRACE
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-MBED_DRIVERS-STATS
  * DELTA_DFBM_NQ620::GCC_ARM::MBED-OS-TESTS-STORAGE_ABSTRACTION-BASICAPI
mbedgt: greentea test automation tool ver. 1.2.0
mbedgt: test specification file 'C:\cygwin\home\ausbla01\mbed\ci-test-shield\.build/tests\DELTA_DFBM_NQ620\GCC_ARM\test_spec.json' (specified with --t
est-spec option)
mbedgt: using 'C:\cygwin\home\ausbla01\mbed\ci-test-shield\.build/tests\DELTA_DFBM_NQ620\GCC_ARM\test_spec.json' from current directory!
mbedgt: detecting connected mbed-enabled devices...
mbedgt: detected 1 device
mbedgt: mbed-ls was unable to enumerate correctly all properties of the device!
        check with 'mbedls -j' command if all properties of your device are enumerated properly
mbedgt: mbed-ls property 'serial_port' is 'None'
        check if your serial port driver is correctly installed!
mbedgt: mbed-ls property 'platform_name' is 'None'
mbedgt: processing target 'DELTA_DFBM_NQ620' toolchain 'GCC_ARM' compatible platforms... (note: switch set to --parallel 1)
mbedgt: all tests finished!
mbedgt: shuffle seed: 0.6793174890
mbedgt: no platform/target matching tests were found!
mbedgt: no matching platforms were found!
mbedgt: completed in 0.67 sec
mbedgt: exited with code -110
[mbed] ERROR: "mbedgt" returned error code -110.
[mbed] ERROR: Command "mbedgt --test-spec C:\cygwin\home\ausbla01\mbed\ci-test-shield\.build/tests\DELTA_DFBM_NQ620\GCC_ARM\test_spec.json -n mbed-os-
*" in "C:\cygwin\home\ausbla01\mbed\ci-test-shield"

---

Solution

fix the detect command to handle values of None, cause it looks like that is what its trying to lookup, which causes issues. I would suggest prompting the user with the command to manually patch / alias the board.
??

@theotherjimmy

fix the detect command to handle values of None, cause it looks like that is what its trying to lookup, which causes issues. I would suggest prompting the user with the command to manually patch / alias the board.

We can't always do user input, as it would hang something like a CI job.

Ah, no, not requiring user input, more like an error message that suggests a fix the user can try next time. Thinking as a post failure debug step, not a command interrupt step.

@theotherjimmy Think the python error is first priority (detect_targets.py). Second is handling of unknown targets in the test tools.
cc @bridadan

An error message with something like "Target not found" and then listing possible solutions like:

Is the board plugged in?
Are you developing a new board? Try mocking the device: https://github.com/ARMmbed/mbed-ls#mocking-new-or-existing-target-to-custom-platform-name (or just list the command mbedls --mock <first 4 characters of TargetID>:DELTA_DFBM_NQ620)

Bump @bridadan @theotherjimmy What's the latest status on this?

Hi @screamerbg. Phew, this is an old issue! Looks like the tools really just need a more helpful error message? Maybe adding those two points I mentioned above?
I'm still getting caught up but I may get a PR up with those changes so we can close this out.

Mbed CLI now provides the following message if USB devices are detected, but the target is not recognized.
If you're developing a new target, you can mock the device to continue your development.
Use 'mbedls --mock ID:NAME' to do so (see 'mbedls --help' for more information)

@BlackstoneEngineering @bridadan does this solve the feature request? (bug is already fixed)

Closing this issue as the bug was fixed. Let's follow-up in #578 regarding Cygwin support. In my environment Cygwin v2.10 (latest) for Windows 10 64bit, Mbed CLI works just fine.


MetroStyle: zTreeDragUL has position "static"

When we move a item, in METRO STYLE, the element zTreeDragUL has position="static" and appears at the wrong place (bottom of the parent)
The problem is in metroStyle.css:
CHANGED
ul.ztreeul.ztreeDragUL{
TO
ul.zTreeDragUL{
ZTREE is a very good plugin !

Thank you very much. I have fixed this issue.


Error: Version Conflict

Hi, I have:
MST2_EU_SE_ZR_P0480T
GEM 4.11T
I downloaded the .zip (tried with 1.3.3 and 1.3.2 version), extracted to SD FAT32 root, inserted to SD slot 1 (I have two SD Slot with Nav). I go to service mode, update, check source, SD Card, and next to it shows this error:

Error: Conflict Version
Medium: Device doesn't support this (MainUnit: variant).
My car is Seat Leon FR 2018 with Virtual Cockpit. On docs shows this:
SEAT units with firmware 04xxwith virtual cockpit (MOST bus)
Maybe this is the problem?
What can I do? Thanks.

Read https://github.com/olli991/mib-std2-pq-zr-toolbox#readme

Read https://github.com/olli991/mib-std2-pq-zr-toolbox#readme

I readed that document like 20 times today...

Variant is not supported. Seat with virtual cockpit have a variant that isn't supported by the backdoor anymore.


MultiSplitPane does not feature setWidgets method supporting iterables/collections

Arrays are awkward to manage in pure Java - in many cases, it would be much more convenient to set MultiSplitPane children using a method like #setWidgets(Iterable<Actor> widgets). Currently only arrays are supported.

It's varargs so you can do splitPane.setWidgets(label, label2, label3); but yeah that method won't hurt.

But you don't always know the exact amount of widgets you're going to use and collections are much more flexible for that.


⚠ Turnos 24x7 has degraded performance

In 18532f7, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 855 ms

Resolved: Turnos 24x7 performance has improved in 6bbc4d9.


Geosearch _geoRadius(lat,lng,0) doesn't match direct hits

Describe the bug
Filtering results with _geoRadius(lat,lng,0) will return no results even though search documents in index have an exact lat,lng match.
To Reproduce
Steps to reproduce the behavior:

Activate geosearch
index a search document with a specific lat, lng (e.g. vienna lat: 48.28581699999999, lng: 16.37935643642719)
Do a search for _geoRadius(48.28581699999999,16.37935643642719, 0)
No search results
Do a search for _geoRadius(48.28581699999999,16.37935643642719, 1)
Search results of that document with _geoDistance: 0

Expected behavior
Direct lat,lng hits should be returned when specifying _geoRadius with distance in meters of 0
Meilisearch version: v0.29.2

TODO

[x] Implement changes in Milli -> external contribution accepted for this step
[x] Release a Milli version containing these changes
[x] Bump this new Milli version in Meilisearch and merge it into main

Hello @mmachatschek
thanks for the report!
I invoke @irevoire 🧞

Hey @mmachatschek, this is kind of expected, actually, because it's simply the way floats behave in computer science :grimacing:
What we could do, though, is add a really super tiny value to your distance, so we match your document (and maybe other things but probably not too much).
@amab8901 I see you were looking at some place you could contribute to. Is this something that would interest you?
The idea would be to:

Go in milli
Find the place where we process the geo-filters
Increment the provided distance by EPSILON
Add a test at the end of the file that ensures you find your document if you exactly match them

We expect this issue to be fixed for Meilisearch v1, depending on the time we have to fix this regarding our other priorities.
Indeed, this issue could completely done by the community!

That's a good idea, but it's still not enough; we really need to add this + EPSILON for it to work, I think :thinking:
At least, it's what's recommended in the rust documentation.

That's a good idea, but it's still not enough; we really need to add this + EPSILON for it to work, I think thinking At least, it's what's recommended in the rust documentation.

I'll do the + EPSILON approach then. I wonder though what the underlying explanation is. Is it like rounding errors and truncation errors, or something like that?

I finished the code pretty much. I tried the following cases:
<= radius + f64::EPSILON => test passes
<= radius => test passes
< radius => test fails
conclusion: it's fine to just use <= and there's no need to use f64::EPSILON. Keeping it simple is a design principle I think we should adhere to. No need to complicate the syntax more than necessary. I'm sending in the PR to get your feedback.

ahhh nice 😍
Thank you @amab8901 for tackling this issue 👍

ahhh nice heart_eyes
Thank you @amab8901 for tackling this issue +1

you're welcome :)

Fixed by #3202, will be released in v0.30.1 today!


Licensing question

I can't seem to find any license info in the repo or source. Is this public domain, MIT, other?
Thanks for your work on this module. Very lightweight and handy. 👍

It's licensed under the BSD-3-Clause.  I've added that to the repo properly now.


enh(sparkle): tooltip should be outside of Citation card

Description
https://github.com/dust-tt/dust/issues/6120
Sparkle citations tooltips are cropped inside the Citations card. This fixes the issue by wrapping the card in the tooltip instead of simply wrapping the title, as suggested by @Duncid .
Risk

Deploy Plan

@philipperolet yes


fix: Disable nonce validation for id token for e2e tests

solves #561

Any news on this?

Thanks for this!


Make Oklab and Oklch chroma ratios relative to 0.4

In CSS, oklab(50% 100% 50%) produces the same color as oklab(50% 0.4 0.2).
The same is true with oklch's chroma parameter: oklch(50% 100% 0deg) = oklch(50% 0.4 0deg).
This PR changes Typst's behavior to be in line with CSS's, which IMO is more intuitive overall.
It also adds increased compatibility with any color pickers that output CSS oklch calls.

Why clamp the A and B components to 0.4? The CSS color spec states that values up to +-.5 are in the visible gamut: https://drafts.csswg.org/css-color/#specifying-oklab-oklch

That is totally correct and something I want to improve in a later PR. In this PR I kept the limit in place because it seemed better placed in a PR that implements support for wide-gamut colors in general.

Okay seems sensible

Thanks!


Fixing the incorrect use of the gomock library

When using the goMock library the Controller. Finish() needs to be called when using Times, MaxTimes, or MinTimes functions.

I know I was the one suggesting using Times in the first place, but I'm starting to think just using a counter variable is better if its less likely to turn into a silent failure.

Yeah, I can go either way.


Conda version of Biokit

Hi!
I just wonder could the newest version of biokit also be uploaded in conda?
Cheers

Hi @Andyargueasae,
Is the version on PyPi not sufficient? You can use this YAML file for your Snakemake workflow (assuming this ticket is related to your OrthoSNAP ticket).
best,
Jacob

Hi,
It has been over a month, and I haven't heard any more updates. I assume the issue has been resolved, however, please feel free to reopen it if that is not the case.
best,
Jacob

Yes, it is working.
On Thu, Oct 19, 2023 at 1:06 AM Jacob L. Steenwyk @.***>
wrote:

Hi,
It has been over a month, and I haven't heard any more updates. I assume
the issue has been resolved, however, please feel free to reopen it if that
is not the case.
best,
Jacob
—
Reply to this email directly, view it on GitHub
https://github.com/JLSteenwyk/BioKIT/issues/1#issuecomment-1768539188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU7UKDKNKN4LGXDDDFTULGTX77O7LAVCNFSM6AAAAAA4ZTA4X2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRYGUZTSMJYHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Better logging

I want to use zap to log exactly what users see.
? Your SSH key is expired. Do you want to update it? Yes
? Do you want to generate a SSH key? Yes
? What format do you want the SSH key to be?  [Use arrows to move, type to filter]
> ED25519 - Preferred due to its security
  RSA - More compatible with older systems

Right now, I only log the question prompt and the answer, but not the cute interface with ? and >.
Any ideas? Thanks.

Hi, what is zap, and why do you need to log the "cute interface"?
Right now, Survey has no internal logger of its own. However, you can edit your app to add logging around invocations of survey.Ask() and survey.AskOne() if you want to log the fact that the user was presented with some prompts.


NuGet Package?

I'm not seeing a NuGet package for this, is it under a different name?

I'm sorry that there is no NuGet packages available :(

Can you create one?


office demo delivery task

In the example office, I try to run the 'delivery' task. The robot stops at the pickup waypoint. What command should I send to make it run to the dropoff point,thanks!

Hi there,
Instructions to send a delivery task for the office world are available in the README here. Scroll down to "Submit a task via CLI"
ros2 run rmf_demos_tasks dispatch_delivery -p pantry -pd coke_dispenser -d hardware_2 -di coke_ingestor --use_sim_time


Potential Bug Risks and Anti-Patterns

Description
I ran DeepSource Static Code Analysis upon the Project, the results for which are available here.
The Static Code Analysis Tool found potential bugs and anti-patterns in the Code, that can be detrimental at a later point of time with respect to the Project. DeepSource helps you to automatically find and fix issues in your code during code reviews. This tool looks for anti-patterns, bug risks, performance problems, and raises issues.
There are plenty of other issues in relation to Bug Discovery and Anti-Patterns which you would be interested to take a look at.
Some of the notable issues are:

Unused Variables and Functions ( here and here )
Deprecated APIs here
Import of Private Modules here
Direct Usage of Object.prototypes builtins here

There are plenty of other issues in relation to Bug Discovery and Anti-Patterns which you would be interested to take a look at.

I think dependabot also notifies the same, don't know why the bot is sending PRs for that.

I think dependabot also notifies the same, don't know why the bot is not sending PRs for that.

Dependabot is a dependency version based vulnerability anaysis tool. It uses a list of vulnerabilities and the versions in which the dependencies show those dependencies to send PRs for solving the same.
DeepSource is different in that it helps you find issues in the code that come up due to developer fault, like wrong syntax, unnecessary control flow statements, messy code, etc.
You can check out the Result Dashboard to better analyze what DeepSource does 😄

@goliakshay357 @Kajol-Kumari Please have a look at this. Let me know your views if you feel integrating this would be great for our project.


Scrolling Broken in Settings for certain devices

Environment
Windows build number: 1909
PowerToys version: 0.18.0 and above
PowerToy module: Settings
Device: HP Notebook, product number X7T77UA

Note: this issue was initially observed on a laptop equipped with a Synaptics touchpad, and does not manifest on another, non-Synaptics computer.
Steps to reproduce

Open PowerToys Settings (i.e. right-click system tray icon, choose Settings)
Attempt to scroll using two fingers on touchpad

Expected behavior
The main settings page should scroll as normal (if it's long enough, which it is on my device).
Actual behavior
The main settings page does not scroll.
Overview:

Method
Functional?

Mouse wheel
✔

Touchscreen
✔

Touchpad (two fingers)
❌

Notes
After connecting my wireless mouse, scrolling with its wheel works as expected. While the mouse is connected, however, touchpad scrolling still does not work.
Also, the same issue was originally present in the PowerToys Run results list, but is solved as of at least v0.25.0.

I can't repro this with either a mouse or trackpad

Tested on two laptops and on a Logitech keyboard that has a trackpad, I can't reproduce the probem.

Interesting. It could be hardware-specific somehow - I'm on a three-year-old HP Notebook, if that helps any.
Is there someone else with an HP Notebook who could try to reproduce this?

i have a surface and logitec mouse, both 'just work'
tried a few different permutations around what has focus and it seemed work.

I haven't tried it with the PT settings window out of focus.
Just to be sure about mouse issues, I connected my wireless mouse, then closed and reopened PT. That didn't help, nor did running PT in elevated mode. I'm sorry I can't provide more insightful reports -- Windows apps are not an area of experience for me.
Again, this seems like it might be a hardware thing rather than an PT issue per se. If someone else with an HP laptop could check on this, that could potentially illuminate the problem further.

Yeah, I got a touch screen too (MS Surface pro 4, the be specific) and absolutely no problem here :)

Oh, I actually have a touchscreen too -- I just never use it. Let me see...
Scrolling does work using my touchscreen. So, at this point, things look like this:

Method
Functional?

Mouse wheel
✔

Touchscreen
✔

Touchpad (two fingers)
❌

Also, this continues to be an issue with v0.18.1.

Repro'd on a spectre x360 late 2k17

I edited 3772 to add some details before it was closed as a duplicate, but it doesn't seem to have kept the edit.
Anyway, every time Powertoys updates, the two finger scroll on the Lenovo touchpad breaks without fail. This is for any app that has a scroll sidebar. The touchscreen scroll still works fine though. Also, the touchpad control of the mouse pointer goes very erratic - sometimes very laggy, as if it's struggling to process the input. Whenever this happens, I know the machine is going to be unusable until I reboot, and once I've done that it's absolutely fine (until the next time I update Powertoys).
So for me at least, it's not PowerToys per se breaking the touchpad, it's the act of updating. I've tried opening the PT settings dialogue as described in the current ticket, and this does not give me a problem - so 3772 and 3440 may not be the same issue.

I experience this bug on my HP Pavilion - 15-au639tx. This is present in PowerToys Settings as well as PowerToys Run.

I am seeing scrolling issues on my laptop. Cannot scroll with touchpad using two finger scroll, nor can I scroll using my mouse wheel.
Can only scroll by using the pointer to grab the scrollbar and drag.
Scrolling works fine in all other applications.
Windows 10 1909
PowerToys 0.18.0
Seems like similar symptoms to a bug that was recently fixed in Windows Terminal.

@wolf99

Seems like similar symptoms to a bug that was recently fixed in Windows Terminal.

Thanks for pointing it out.

@kneelesh48 I hadn't thought of checking PT Run, but the issue is present there for me as well.

both are inside xaml islands for results currently.

Think we may have a lead here, Terminal may have a fix

Update: The PT Run results list scrolls properly since as least v0.25.0! It's not smooth scrolling (i.e. it jumps by the height of one result each time), but the scrollbar provides the same behavior.

@crutkas I should have specified this earlier, but pages in the PT Settings window still do not scroll properly using a touchpad. Since this issue is not completely resolved, could you reopen it?

From #8349, they had the issue with an Arc mouse.

If it helps, scrolling works as expected with the touchpad on my new Lenovo Legion 5 laptop.

I have reported this before in another issue. Scrolling is broken whether using a mouse or a touchpad. Currently using a Logitec MX vertical mouse connected via a dongle to a USB hub. Dell Latitude e 5440

Facing the same problem here, using Synaptics trackpad on an HP laptop. PT version 0.35.

Same problem by my Lenovo laptop with a Synaptics trackpad.
It seems the problem only occurs with a Synaptics trackpad.

I lose Thinkpad KT-1255 BT keyboard Trackpoint scrolling as soon as I run PowerToys as administrator. Reboot restores function but also disables Keyboard manager.

I lose Thinkpad KT-1255 BT keyboard Trackpoint scrolling as soon as I run PowerToys as administrator. Reboot restores function but also disables Keyboard manager.

I think that's a different issue. This one is about not being able to scroll in the PT Settings window only, not complete loss of Trackpoint function.

I am also unable to scroll in PowerToys Settings, I'm using a Kensington Expert Mouse Trackball USB on WIndows 10 ver 1909 build 18363.1379

PT version 41.4
unable to scroll with mouse wheel or keyboard (arrows, pg up, pg down) in settings for a long time now and since many PT versions ago.
Windows ver 2004, build 19041.1052
mouse: Logitech mx518, driver Logitech v<IP_ADDRESS>

For that use X-Mouse Button Control, a workaround may be possible, See the related Windows Terminal issue thread: https://github.com/microsoft/terminal/issues/9340#issuecomment-885822956
For those that don't, the mouse configuration software (Synaptics trackpad, Lenovo Thinkpad Trackpoint) may or may not have the ability to work around this issue.

This is fixed right? I had the same issue and timadye's comment about X-Mouse mentioned right above by landonth in https://github.com/microsoft/terminal/issues/9340#issuecomment-885822956 fixed my issue as well.

This is fixed right? I had the same issue and timadye's comment about X-Mouse mentioned right above by landonth in microsoft/terminal#9340 (comment) fixed my issue as well.

cc: @crutkas

looks like, @LRitzdorf do you have x-mouse installed?  https://github.com/microsoft/terminal/issues/9340#issuecomment-885822956

/needinfo

Unfortunately, I do not. Also, I've moved to a new laptop since opening this issue, which does not have a Synaptics trackpad and is therefore unaffected by the issue. (The old one now runs Kali Linux, so I can't easily reinstall Windows there either.)
Sorry, I wish I could be of more help here -- maybe @theAeon could help us out with a test? They were the first person after me to repro the issue.

Sadly, I don't have a synaptics pre-precision touchpad any more either. Well, not sadly, considering I finally got HP to replace that lemon of a machine with a newer one, but I can't be of any help here.
I'm fairly sure there was a period in which it was working, but do not quote me on that.

I might have the hardware you are looking for. What exactly do you need?
On Tue, Jan 25, 2022, 6:57 AM Andrew Robbins @.***>
wrote:

Sadly, I don't have a synaptics pre-precision touchpad any more either.
Well, not sadly, considering I finally got HP to replace that lemon of a
machine with a newer one, but I can't be of any help here.
I'm fairly sure there was a period in which it was working, but do not
quote me on that.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/3400#issuecomment-1021207517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFC3L3JJRPMZHNQHB4NTAALUX2T3DANCNFSM4NGRUYLQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

As far as I'm aware, we just need a Windows machine that's affected by this issue, with an owner who's willing to install x-mouse.
Per the Terminal comment that crutkas linked above, it looks like x-mouse might provide a workaround for the scrolling-without-focus issue in XAML islands, and it'd be nice to test that with PowerToys.

I just tested it. Power toys version 0.53.3 and it worked like a charm. As soon as I enabled that setting my touchpad scroll started working. Normally I'm using external mice so I will have to try that later. But it worked. I use x-button anyway so that's not a huge deal for me.

The issue still exists on my Lenovo laptop with a Synaptics touchpad as of version 0.53.3.
This problem also happened on other programs based on .NET Core/.NET 5, but they fixed soon after upgrading to .NET 6.
Not sure if .NET 6. would be helpful. (maybe no bearing on that)

.net 6 will be added next release (.56).  we have a PR for it open.  Going from 5 to 6 is easy but we needed a few moving parts with CI & build agents so we had to do .NET Core 3.1 to .NET 6 in two steps.

So I have some questions... so is there a point where synaptic touchpads were replaced with precision touchpads? so you either have one or the other? The workaround mentioned by @landonth worked for me on my older dell machine that has a synaptic touchpad. on my newer Lenovo ThinkPad 460s which I use the most as it's my work-issued computer. the issue is quite the opposite. The scroll does not work within the app using my external mouse (MX vertical). but it works with the touchpad. I activated that same option in x-button mouse control and it seemed to fix the issue... not sure what this means but I thought I would share as this unfolds.

looks like, @LRitzdorf do you have x-mouse installed? microsoft/terminal#9340 (comment)

Thank you, wanted to mention https://github.com/microsoft/PowerToys/issues/13537 pointing me to the same source of error.

do you have x-mouse installed?
Confirmed! No scroll with X-Mouse enabled. In X-Mouse the scroll wheel is not intercepted.
https://github.com/microsoft/PowerToys/issues/16848
Confirmed with Shift, but not with Scroll Lock.
PS, there is an option in X-Mouse that disables it with Scroll Lock on, maybe that's it.

In theory this is fixed

I have the same issue. I can't scroll in settings menu. I am using touchpad in HP laptop.
Is there a fix?

@lars18th what version are you on?  Can you try .59 which is now on WinAppSDK 1.1.  Terminal is using a different stack than us from 0.58 moving foward.

Can you try .59
Still not fixed! Scrolling works only with Shift depressed continuously.

@watchmakerful And you have x-mouse control enabled?

Hi @crutkas ,

@lars18th what version are you on? Can you try .59 which is now on WinAppSDK 1.1. Terminal is using a different stack than us from 0.58 moving foward.

I've upgraded PowerToys to 0.59. After I re-enabled in the X-Mouse Button Control v2.19.2 the option "Make scroll wheel scroll window under cursor". And I opened the configuration window of PowerToys. The result? The scroll with the whell doesn't work. Only when I disable this option in XMBC the scroll works with the whell.
Any idea?
Only FYI, I'm using the native X-Windows mouse windows selection (without raising the window). So in fact I don't need the option from XMBC enabled. However, I feel that other users that doesn't enable the X-Windows mouse mode want this fixed.

@lars18th that is an issue with WinAppSDK and how those tools interact with mouse input.

The root original issue of this has been fixed when we migrated to WinAppSDK and removed the XAML islands a few builds back.  If you are using something like xmouse or XMBC,

that is a different issue but in itself not directly a powertoy issue

I've upgraded PowerToys to 0.59. After I re-enabled in the X-Mouse Button Control v2.19.2 the option "Make scroll wheel scroll window under cursor". And I opened the configuration window of PowerToys. The result? The scroll with the whell doesn't work. Only when I disable this option in XMBC the scroll works with the whell.

Thanks @lars18th , that helped a lot!


Be consistent about importing Inject/Singleton

We had a mixture of imports from javax.inject and com.google.inject. I don’t know what the differences are – we should be consistent to reduce the risk of hard-to-spot bugs.

@mklander Forgive my ignorance, but what’s JSR330, and why would it be better? (Google is not immediately helpful in this regard.)


Peer-peer code review

Please change the color of the contact me text to white to match the figma design.

Nice implementation of animation. Good work!

Please change the color of the contact me text to white to match the figma design.

Hello @Bellagirl-maker thanks for suggestion i will make correction

Nice implementation of animation. Good work!

Hello @fatmahussein thanks for your feedback


Can we please remove the automatic () after some commands like Findset

The intellisense in Dynamics 365 BC has started to add parentheses () after some keywords.
For example if I type:
if Cust.Findset then repeat
until Next=0;
Then the intellisense will perform like this
If Findset({Cursor}) and the cursor positions itself in the parentheses.
The same with Next({Cursor})
It seriously spoils my coding pace that I have to use the arrow keys.
Another example:
ResponseMessage.Headers.Add()
Then the intellisense does this:
ResponseMessage.Headers()
Then I have to remove the () before adding the .Add()
I'm sorry that is just annoying.
It worked fine before.
/Peik

Microsoft clearly states that the cough useless cough parentheses are the suggested way to go now:
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/compliance/apptest-bestpracticesforalcode#calling-methods
Therefore, i doubt that your suggestion will happen.
But it sure feels weird at first, i also removed all the first parentheses when I started AL, but you get used to it. Also having the visual clue between Function calls and properties makes code way more readable to people who aren't used to C/AL.

He he, I know.
But I also know that it was a final decision that there would never be DotNet in extensions - not.
And the fact that ALL table-fields in extensions should be prefixed with your approved initials which has changed to be only fields in table extensions.
So I am just hoping that they realize that it sounded good in theory, but it is useless in the real life :)

This behaviour needs a bit of tweaking for sure, maybe position cursor inside parentheses for some places and not the others, but they definitely must be in the code to signify the function call instead of just variable.

Parentheses are indicating that this is a function. Is FindSet a function? Yes - therefore it should have parentheses, I complete agree with Microsoft's decision to finally make AL a proper programming language.

I had no intentions of starting world war III and I know that DotNet programmers have their set ways.
My problems are:

The naming of the function should reflect if it is a function or a variable, but if others need parentheses to read their code then that's all right with me.
However, if there are additional methods that should show up after the previous then the parentheses should not be suggested until the last method.
Also if the parentheses must be there then it must be handled differently from function to function.
for example: how many use the parameters in the FindSet and the Next commands. Here the cursor should be after the parentheses just like they have done in the FindFirst command.

Hi @peikba, thanks for filing this issue and fueling this discussion. However, as @wisa1 already mentioned, we'll stick with the current behavior. If you would find a bug in this area, please file another issue.


ModuleNotFoundError: No module named 'torch'

I tried using pip and cloning the project and I still have the same error. knowing that I have torch and torchvision installed.
Python 3.10.9
torch 2.0.1+cu118
torchvision 0.15.2+cu118

running into this as well

The temporary solution I found was to use the non-cuda version of torch and torchvision

was able to get it working with:
groundingdino = {git = "https://github.com/IDEA-Research/GroundingDINO.git"}
groundingdino-py = "0.4.0"
in the pyproject.toml file

#49


Datadog still inspect non-existed container

yyyy-MM-dd HH:mm:ss UTC | CORE | ERROR | (pkg/autodiscovery/listeners/* in createService) | Failed to inspect container * - "container docker://* in PodList" not found

Describe what you expected:
Let's double check if the container are still existed, if not, then it should not be an error though.
Steps to reproduce the issue:
Additional environment details (Operating System, Cloud provider, etc):
Datadog Agent (v7.25.1)

And people still get this issue though: https://github.com/DataDog/datadog-agent/issues/1522


[DocsArchive-1.0.23045.1] Archive binary serialization articles

Please don't merge this PR until target archive repo PR https://github.com/dotnet/docs-archive-pr/pull/33 is merged into live branch.
Auto archive content to https://github.com/gewarren/docs-archive-pr.git
Contributes to #32568

Internal previews
Toggle expand/collapse

📄 File
🔗 Preview link

docs/architecture/maui/mvvm-community-toolkit-features.md
docs/architecture/maui/mvvm-community-toolkit-features

docs/core/compatibility/unsupported-apis.md
docs/core/compatibility/unsupported-apis

docs/csharp/programming-guide/concepts/linq/filtering-data.md
docs/csharp/programming-guide/concepts/linq/filtering-data

docs/csharp/programming-guide/concepts/linq/linq-to-objects.md
docs/csharp/programming-guide/concepts/linq/linq-to-objects

docs/framework/data/adonet/sql/linq/sql-clr-type-mapping.md
docs/framework/data/adonet/sql/linq/sql-clr-type-mapping

docs/framework/wcf/feature-details/serialization-and-deserialization.md
docs/framework/wcf/feature-details/serialization-and-deserialization

docs/framework/wcf/feature-details/version-tolerant-serialization-callbacks.md
docs/framework/wcf/feature-details/version-tolerant-serialization-callbacks

docs/fundamentals/code-analysis/quality-rules/ca1801.md
docs/fundamentals/code-analysis/quality-rules/ca1801

docs/orleans/host/configuration-guide/serialization.md
docs/orleans/host/configuration-guide/serialization

docs/standard/serialization/binaryformatter-security-guide.md
docs/standard/serialization/binaryformatter-security-guide

docs/standard/serialization/how-to-chunk-serialized-data.md
docs/standard/serialization/how-to-chunk-serialized-data

docs/standard/serialization/index.md
docs/standard/serialization/index

docs/standard/serialization/introducing-xml-serialization.md
docs/standard/serialization/introducing-xml-serialization

docs/standard/serialization/system-text-json/how-to.md
docs/standard/serialization/system-text-json/how-to

docs/standard/serialization/xml-and-soap-serialization.md
docs/standard/serialization/xml-and-soap-serialization

docs/whats-new/dotnet-7-docs.md
docs/whats-new/dotnet-7-docs

Pending approval from @GrabYourPitchforks.


usePrometheus: Also count requests that doesn't pass the validation or parse step

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Related: #1356
Type of change
WIP: Currently only a failing test case
Checklist:

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

It makes sense but I think I need to close this in favor of discussing #1359 which I think supercedes the previous issue, before proceeding here.


NCMesh memory optimizations [ncmesh-mem-opt-dev]

This adds the following memory optimizations in the NCMesh class:

Most 64-bit pointers (to child elements, nodes, etc.) were replaced by 32-bit ints.
The ints are indices into NCMesh::elements, NCMesh::nodes and NCMesh::faces. These are instances of a new container, BlockArray (or its subclass HashTable).
BlockArray is a fast allocator, it allocates T in chunks. It allows the slots in the container to be freed and reused later.
HashTable now uses BlockArray for storage, and adds the usual hashing on top of it. There are some changes in the interface because of the primary use of indices to refer  to items.
NCMesh::Vertex, NCMesh::Edge and NCMesh::RefCount are all gone. I realized they're not needed.
NCMesh::Node and NCMesh::Face are smaller. Nodes just track the vertex and edge reference counts and their indices in the Mesh.
NCMesh::top_vertex_pos contains the coordinates of top-level vertices and is practically the only geometry left in NCMesh now. If the Mesh is not curved, this member is used to calculate positions of all vertices (in NCMesh::GetMeshComponents, see also NCMesh::CalcVertexPos).

Changes in ex1 and ex9p are not to be merged. In ex1 there is some code to measure the memory use of NCMesh.
Compared to master (you can try running the modified ex1 in there), NCMesh now uses 50% less memory (in total). The average cost of an NC element (including the refinement hierarchy, but not including the temporary
face_list and edge_list) is now only about 290 bytes. Also, the total time of the testing ex1 is now 30-40% smaller.
ParNCMesh could use some optimization too, but I only had time to make it work again with the new NCMesh. Maybe some other time.
There's quite a few changes in NCMesh and ParNCMesh. I tried to make sure all examples work, especially ex1/p, ex6/p, ex9/p and ex14/p.
The changes in this PR need not be studied in detail, there's a lot of boring stuff. We should make sure all examples work exactly as before, though.

@rw-anderson, @mlstowell and @v-dobrev, do you mind taking a look / approving this?

We need to fix the travis ci file - make check failed but it was not reported properly because it exited with code 0.

Hmm, looking at https://travis-ci.org/mfem/mfem/jobs/209589765 I would have expected a failure... there was a core dump after all.

@mlstowell and @rw-anderson do you mind if we merge this PR in, or do you need more time to review it?


branch-2.0Ignore fe/gen (#31512)

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

#31512


"Cannot find type Topdirs.printer_type_new" with OCaml 5

Testing with this input:
```ocaml
# let pp f x = Format.fprintf f "[%s]" x;;
# #install_printer pp;;
# "hi";;
```

On OCaml 4.14 it works:
```ocaml
# let pp f x = Format.fprintf f "[%s]" x;;
val pp : Format.formatter -> string -> unit = <fun>
# #install_printer pp;;
# "hi";;
- : string = [hi]
```

but on 5.0.0 I get an error:
```ocaml
# let pp f x = Format.fprintf f "[%s]" x;;
val pp : Format.formatter -> string -> unit = <fun>
# #install_printer pp;;
Cannot find type Topdirs.printer_type_new.
# "hi";;
- : string = "hi"
```

It works in the OCaml 5 toplevel itself:
$ ocaml
OCaml version 5.0.0
Enter #help;; for help.

# let pp f x = Format.fprintf f "[%s]" x;;
val pp : Format.formatter -> string -> unit = <fun>
# #install_printer pp;;
# "hi";;
- : string = [hi]

I seem to be able to get this to work by including compiler-libs.toplevel in the libraries field of the mdx stanza. I'm not quite sure why, maybe something related to https://github.com/ocaml/ocaml/pull/11745 but I'm not sure.

The relevant PR for 5.0.0 is probably more ocaml/ocaml#11199: In brief, REPLs that want to install printers need to call Topcommon.load_topdirs_signature () in 5.0.0. Normally, https://github.com/ocaml/ocaml/pull/11745 will solve this issue by removing completely the need to read Topdirs.printer_type_new in 5.1 .


feat: generate type declarations

Proposed changes
Generate typescript .d.ts type files. This allows vs code intellisense.
Screenshots (if appropriate) or codepen:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My commit messages follows the conventional commit format
[ ] My change requires a change to the documentation, and updated it accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

I'm also doing more tests about this. Not about adding dist folder to gitignore, but about the way types are exported to be used from other modules.
I think the standard way will be not the one used here, but adding a index.ts class that re-export all types like
export * from './global';
export { Toast } from './toasts';
export { Tabs } from './tabs';
export { Sidenav } from './sidenav';
export { Collapsible } from './collapsible';
export { Dropdown } from './dropdown';

and set this as the "typings": "dist/src/index.d.ts" entrypoint for the package.
This will allow to use import each class as
import { Sidenav  } from '@materializecss/materialize';

But of course all this need to be tested in the different kinds of project to check it works correctly for all.
For the moment I think is better do not include this.

As said in the previous comment I have changed the way types are exported.
Now it follows the standard way of re-exporting all types from a index.ts file.
In practical terms this means that to import some class instead of having to define each class path in the library as
import { Tabs } from '@materializecss/materialize/dist/src/tabs';

now all classes can be imported from @materializecss/materialize as
import { Collapsible, Tabs } from '@materializecss/materialize';

For projects that are using typescript and importing types this is a big difference: without this change, using materialize from a typescript project is NOT complete.
As a side note I need also those changes to use materialize as a monorepo. I think "monorepo" is a very interesting option to work MUCH faster of materialize, as you don't need to deploy nothing, you change the compo and you see the change immediately.
I think that's and important topic, and I would open a discussion about that, but only if there is some interest.... and looking at the feedback received on my previous discussions i don't think this will happen.

Sure, that sounds nice! We can discuss everything here. The only thing why i did not merge your change was because you added the dist folder into the gitignore. This will cause errors during the release process.
I dont know why this was done and asked about it. Thanks for your ideas and improvements. They are really helpful and important for the project. I am looking forward for the next improvements. Please describe in more detail how you use your solution step by step so that I can test it better, then it is easier for me to understand why it is so useful. Thank you and good work 👍

Ok, I see, sorry for the confusion...
Adding dist was because I changed the output dir in tsconfig to this folder
"outDir": "./dist",
And then it generates lots of "dist\src\autocomplete.d.ts" like files.
Adding dist to gitignore seems normal, following the common practice of not including build generated files in the source control.
One solution could be just adding "dist/src" to .gitignore. But maybe we will face the same problem in the package distribution files.
Maybe this  is the solution for this issue. I would prepare an update:

just adding "dist/src"  to gitignore
adding a .npmignore file exactly like gitignore but including all dist

What do you think?

As commented I have changed gitignore to "dist/src" and added a .npmignore file exactly like gitignore but including all dist.
I think with this configuration it should generate a package with type info...

What is the first commit? It contains Styles and I like it, but why is it in this PR?
Sorry, but I am a bit confused. I can not merge because i dont fully understand.

Can you make the style changes and the Type-Changes in seperate PRs?
Then we can progress faster an I can more easily merge the PRs. Thank you.

Can you make the style changes and the Type-Changes in seperate PRs? Then we can progress faster an I can more easily merge the PRs. Thank you.

Sorry, this was unintended... I have update the PR including only the type declarations commit.

Haha ok thats fine.
But I also like the Style changes. It looked like, that you have implemented a primary and secondary color system which would be very awesome. It would be nice if you could provide this as a seperate PR too.
Also I talked recently with the old team members and one of them told me it would be a good idea to start as a new repo to avoid distraction from the old materializecss. That would also give the possibility to clean many things up or remove (maybe grunt). Also i would suggest to give you @danice  and @mauromascarenhas more rights, in order to check in PRs faster. What do you or others think about this?

Created PR for M3 styles.
About the new repo question, please have a look to the "Modern development environment using a vite monorepo with materialize as a package"   discussion.
I would suggest removing from the materialize repo all the website stuff, and creating a separated repo for the website that would include materialize repo as a "monorepo" package and also as git submodule (see the linked project as an example of how it will look).
This "monorepo" is a very practical development environment, but of course it requires to be familiar with git submodules. My experience is that for this is better to use (at least a as complement) some git client like git extensions as - at least for me- Visual Studio is not very friendly with submodules.

Yes, that sounds good! The only concerns i have:
Does handlebars also work serverside to pre-compile all the pages?
That is better for SEO and also compile in multi languages and even keep old versions.
I will make a new Repo in the next days and then we can continue splitting the project step by step.

Does handlebars also work serverside to pre-compile all the pages?

Yes, the pnpm build generates a dist folder of "plain" version of html pages with all the partials resolved.

I have done some fixes for ts type errors and to maintain M import as default ("import M from ...", without { M } ) to keep backward compatibility.
I have re-checked in webpack and "js import" project and it works fine.
So I think it's ready for merge...

That are some serious pro-moves dude! Haha awesome, I'll merge it.


Pub can fail from Pub Update with little info to work with

Running pub update from editor, getting this:
Running pub update ...
Pub update failed, [69] Resolving dependencies...
HTTP error 500: Internal Server Error
Not sure how to turn on --verbose when running via the editor. But maybe on a hard failure like 500 we print a bit more, so we can get actionable bug reports from users.

This issue has been moved to dart-lang/pub#422.


Expose dfflegalize -mince, -minsrst in synth scripts

dfflegalize allows setting a threshold when it uses cells with CE ans SRST, but this is only exposed by synth_ice40. While troubleshooting https://github.com/YosysHQ/nextpnr/issues/1062 I found that a combination of -mince and -minsrst would alleviate the placement pressure, allowing PnR to complete. This diff is a start to expose these knobs in every synth script.
Some test results for an iCE40 LP1K design, with no minimums (or -dff-min-ce 0 -dff-min-srst 0):
=== mkSidecarRevBTargetWithResetButton ===

   Number of wires:                895
   Number of wire bits:           3025
   Number of public wires:         895
   Number of public wire bits:    3025
   Number of memories:               0
   Number of memory bits:            0
   Number of processes:              0
   Number of cells:               1302
     SB_CARRY                       51
     SB_DFF                         93
     SB_DFFE                       229
     SB_DFFESR                      34
     SB_DFFESS                      34
     SB_DFFR                        24
     SB_DFFSR                       56
     SB_DFFSS                        1
     SB_IO                           6
     SB_LUT4                       774

With -dff-min-ce 4 -dff-min-srst 4:
=== mkSidecarRevBTargetWithResetButton ===

   Number of wires:                886
   Number of wire bits:           2982
   Number of public wires:         886
   Number of public wire bits:    2982
   Number of memories:               0
   Number of memory bits:            0
   Number of processes:              0
   Number of cells:               1302
     SB_CARRY                       51
     SB_DFF                         68
     SB_DFFE                       229
     SB_DFFESR                      67
     SB_DFFESS                      44
     SB_DFFR                        24
     SB_DFFSR                       39
     SB_IO                           6
     SB_LUT4                       774

Similar for a design targeting an ECP5-45F device, no minimums:
=== mkSidecarMainboardControllerRevB ===

  Number of wires:              32230
  Number of wire bits:         113931
  Number of public wires:       32230
  Number of public wire bits:  113931
  Number of memories:               0
  Number of memory bits:            0
  Number of processes:              0
  Number of cells:              44140
    $_TBUF_                         2
    CCU2C                         880
    DP16KD                          2
    L6MUX21                       478
    LUT4                        24894
    PFUMX                        3181
    TRELLIS_FF                  14703

And with -dff-min-ce 4 -dff-min-rsrst 8:
=== mkSidecarMainboardControllerRevB ===

  Number of wires:              32969
  Number of wire bits:         114685
  Number of public wires:       32969
  Number of public wire bits:  114685
  Number of memories:               0
  Number of memory bits:            0
  Number of processes:              0
  Number of cells:              45155
    $_TBUF_                         2
    CCU2C                         880
    DP16KD                          2
    L6MUX21                       614
    LUT4                        25258
    PFUMX                        3696
    TRELLIS_FF                  14703

I am more than happy to add this to the remaining synth scripts where applicable, but would like some review of the UI prior to doing so.

Thanks for this PR! If they can make a noticeable difference in QoR for this target then it definitely makes sense to surface the options in the respective scripts.
Independently of that, it would also be nice to make these kind of QoR knobs accessible via the scratchpad, as we've started to do with various options in abc and abc9. That way, they can be changed even if a script pass doesn't have an option for it. I think all that would be needed in this case is to change the initialization in dfflegalize.cc:1024-1025 to this:
		mince = design->scratchpad_get_int("dfflegalize.mince", 0);
		minsrst = design->scratchpad_get_int("dfflegalize.minsrst", 0);

Then you should be able to use scratchpad -set dfflegalize.mince 4; scratchpad -set dfflegalize.minsrst 8; synth_ecp5 with the same result as synth_ecp5 -dff-min-ce 4 -dff-min-rsrst 8, but the scratchpad value would still be overridden if dfflibmap gets an explicit -mince or -minsrst argument. Could you try that out and add it to this PR?


update roamjs-components to 0.80.3

I do not have access to this repo to check for repo secrets @dvargas92495

Also, there was an issue with testing this one.  I'll discuss that with you today.

tested with API_URL
While it worked, I got the following console errors:

The first two also happen with the current Roam Depot version, but the second two do not.

first two is expected, I am pushing out a fix to the bottom two now

hey @mdroidian, I'm going to take over this branch - there are some backend redirecting that I want to group with this change

Actually going to merge it into another branch that isn't main that I'm working off of

Actually going to merge it into another branch that isn't main that I'm working off of

I noticed these changes haven't made it into main yet.  Are you still working on them locally?

agh this slipped - feel free to take it over


Modal titile on Android

Modal title on Android
How can I modify the modal title or header text on Android? the feature is there on iOS but is missing on Android.

You can't do it, the title on Android is set in the native layer (which is not controlled by this component).


[BUG] - node synchronization stuck at 23.11% forever with ErrorPolicySuspendPeer errors

Internal/External
External
Area
Other
Summary
cardano-node is stuck syncing for hours at 23.11%, as calculated by this script (adjusted for version 1.26.1 to ask for .slot instead of .slotNo, but otherwise unmodified). Surprisingly enough, both the numerator and the denominator keep growing steadily (e.g. slot 25543697/110496501 23.11%), but the overall ratio hasn’t changed for a number of hours.
Steps to reproduce
Start cardano-node 1.25.1 or 1.26.1 like this:
cardano-node run \
    --database-path /var/lib/cardano-node/db \
    --socket-path /run/cardano-node/socket \
    --host-ipv6-addr :: \
    --host-addr <IP_ADDRESS> \
    --port 3000 \
    --config /var/lib/cardano-node/config/mainnet-config.json  \
    --topology /var/lib/cardano-node/config/mainnet-topology.json

Expected behavior
It should make steady progress to a full sync.
System info (please complete the following information):

OS: ArchLinux
Version: up-to-date
Node version: 1.26.1, based on this package

Screenshots and attachments
Error messages usually come in groups of 3 per peer IP address:
[charon:cardano.node.ErrorPolicy:Warning:283] [2021-03-30 13:03:37.64 UTC] IP xxx.xxx.xxx.xxx:45102 ErrorPolicySuspendPeer (Just (ApplicationExceptionTrace (MuxError MuxBearerClosed "<socket: 68> closed when reading data, waiting on next header True"))) 20s 20s
[charon:cardano.node.ErrorPolicy:Warning:283] [2021-03-30 13:03:38.70 UTC] IP xxx.xxx.xxx.xxx:52084 ErrorPolicySuspendPeer (Just (ApplicationExceptionTrace (MuxError MuxBearerClosed "<socket: 71> closed when reading data, waiting on next header False"))) 20s 20s
[charon:cardano.node.ErrorPolicy:Warning:283] [2021-03-30 13:03:40.73 UTC] IP xxx.xxx.xxx.xxx:59960 ErrorPolicySuspendPeer (Just (ApplicationExceptionTrace (MuxError MuxBearerClosed "<socket: 71> closed when reading data, waiting on next header False"))) 20s 20s

However, there is some communication with the network, because there are Chain extended messages every now and then.
[charon:cardano.node.ChainDB:Notice:257] [2021-03-30 13:09:13.73 UTC] Chain extended, new tip: 9c57345badcf8fc267f7df530ff80fd55b1c6a934c2dd36cea25ced8f7e73175 at slot 25543462

Additional context

Initially the synchronization started normally and progressed up to the 23.11% just fine (in a few hours). Then something got stuck for no apparent reason. The problem showed up already with 1.25.1. (The only reason why I upgraded was to double-check if the problem would disappear.)
It looks like the node is listening normally (according to netstat -atpn):Proto  Recv-Q  Send-Q  Local Address  Foreign Address  State   PID/Program name
tcp6        0       0  :::3000        :::*             LISTEN  1151036/cardano-node
tcp         0       0  <IP_ADDRESS>:3000   <IP_ADDRESS>:*        LISTEN  1151036/cardano-node

Port 3000 is allowed in nftables, so incoming TCP connections should work as well. DNS resolution works fine (as the machine happens to be also a caching + authoritative DNS server), both the legacy way (nameserver ::1 in /etc/resolv.conf) as well as the modern way (systemd-resolve --status looks sane with ::1 everywhere).

I spotted #1124, #2016 or #2269, but upon a closer look those issues seem unrelated. However, #2452 might be somewhat related.

Further context:

This looks related, but I’m using 1.26.1, not the old versions mentioned there…
This has been asked on Reddit.

What I’ve tried thus far (to no avail):

remove --host-addr
remove --host-ipv6-addr
set both --host-addr and --host-ipv6-addr to an address on the external interface instead of the zero wildcard
switch to port 3001 instead of 3000 (with corresponding nftables adjustment)
delete the entire db directory and start over

After the database deletion I’m seeing way fewer of ...closed when reading data... errors, but it seems to be getting stuck nonetheless. It hasn’t reached 20% after ~10 hours and progress is slowing down steadily.

@andrejpodzimek That script looks a little dodgy to me (testnet etc). What does the following command say? :
CARDANO_NODE_SOCKET_PATH=/path/to/node/socket cardano-cli query tip --mainnet

Obviously, replacing path/to/node/socket with whatever is correct for your system.

$ CARDANO_NODE_SOCKET_PATH=/run/cardano-node/socket cardano-cli query tip --mainnet
{
    "epoch": 257,
    "hash": "28c2e905f111b1da0330ac8a854c31cafe77b591b14b6dde49a4de880f69d21a",
    "slot": 25866123,
    "block": 5543047
}

That is ad the tip of the chain. What does the script say?

I have raised a feature request.

That is at the tip of the chain. What does the script say?

slot 25866533/110819360 23.34%

Your node is fully synced, but the script says it is not. The script is wrong.
If you agree, please close this ticket.

That’s awesome! I just couldn’t wrap my head around what the script was saying.
Yet another source of confusion was that Cardano’s data has ~8 GB on my machine, whereas openethereum has 430 GB and bitcoin-qt has 364 GB. This, combined with the script’s strange output, made me think: Heck, Cardano can’t be in sync with so little data! :grin:

The BTC and ETH chains have been running since 2009 and 2013 respectively. The Cardano chain started in 2017 so its is not surprising it has much less data.


Including lyrics API

Pull Request Label:

[ ] Bug
[ ] New feature
[ ] Enhancement
[ ] New component
[x] Typo

Pull Request Checklist:

[ ] Have you followed the guidelines for contributing?
[ ] Have you checked that there aren't other open pull requests for the same fix or component?
[ ] Have you ran the tests locally with bats tests?

Nice catch, thanks 🎉


How Can remove ol and li or change the to div in hits design?

Hi @amit4106udale, we explain how to do this in the section Customize the UI - connectHits, with a full example you can copy paste.
I hope this helps. Happy coding!


Replace DigitalOcean with GitHub Codespaces

Simplier, easier, auto-shutdown.

Known issues

Need to have better static http server than python3 -m http.server.
Write the instructions
Refer people to scratch_run.sh for them to edit and run.


Fixed lint warnings.

It looks like stackdriver/client.go also uses projectId/ProjectId. Can you add the change there too for consistency?

Good catch. I fixed a bunch more. I just realized we have many more lint issues that aren't projectId, but I won't touch them now.


python311Packages.colorlog: 6.8.0 -> 6.8.2

Description of changes
Changelog: https://github.com/borntyping/python-colorlog/releases/tag/v6.8.2

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 284695 run on x86_64-linux 1

23 packages marked as broken and skipped:

home-assistant-component-tests.abode
home-assistant-component-tests.comelit
home-assistant-component-tests.isy994
python312Packages.chat-downloader
python312Packages.chat-downloader.dist
python312Packages.cot
python312Packages.cot.dist
python312Packages.jaraco-abode
python312Packages.jaraco-abode.dist
python312Packages.jedi-language-server
python312Packages.jedi-language-server.dist
python312Packages.lsprotocol
python312Packages.lsprotocol.dist
python312Packages.nox
python312Packages.nox.dist
python312Packages.online-judge-api-client
python312Packages.online-judge-api-client.dist
python312Packages.online-judge-tools
python312Packages.online-judge-tools.dist
python312Packages.pygls
python312Packages.pygls.dist
python312Packages.skybellpy
python312Packages.skybellpy.dist

8 packages failed to build:

apache-airflow
apache-airflow.dist
python311Packages.optuna
python311Packages.optuna.dist
python312Packages.clldutils
python312Packages.clldutils.dist
python312Packages.segments
python312Packages.segments.dist

109 packages built:

bitbake-language-server
bitbake-language-server.dist
cmake-language-server
cmake-language-server.dist
cot (python311Packages.cot)
cot.dist (python311Packages.cot.dist)
errbot
errbot.dist
gkraken
gnomeExtensions.freon
liquidctl (python311Packages.liquidctl)
liquidctl.dist (python311Packages.liquidctl.dist)
liquidctl.man (python311Packages.liquidctl.man)
octoprint
octoprint.dist
online-judge-template-generator
online-judge-template-generator.dist
online-judge-tools (python311Packages.online-judge-tools)
online-judge-tools.dist (python311Packages.online-judge-tools.dist)
phonemizer (python311Packages.phonemizer)
phonemizer.dist (python311Packages.phonemizer.dist)
python311Packages.aio-pika
python311Packages.aio-pika.dist
python311Packages.aiocomelit
python311Packages.aiocomelit.dist
python311Packages.aiofile
python311Packages.aiofile.dist
python311Packages.aiomisc
python311Packages.aiomisc-pytest
python311Packages.aiomisc-pytest.dist
python311Packages.aiomisc.dist
python311Packages.aiormq
python311Packages.aiormq.dist
python311Packages.caio
python311Packages.caio.dist
python311Packages.chat-downloader
python311Packages.chat-downloader.dist
python311Packages.clldutils
python311Packages.clldutils.dist
python311Packages.colorlog
python311Packages.colorlog.dist
python311Packages.dask-gateway-server
python311Packages.dask-gateway-server.dist
python311Packages.dataprep-ml
python311Packages.dataprep-ml.dist
python311Packages.jaraco-abode
python311Packages.jaraco-abode.dist
python311Packages.jedi-language-server
python311Packages.jedi-language-server.dist
python311Packages.lsprotocol
python311Packages.lsprotocol.dist
python311Packages.lupupy
python311Packages.lupupy.dist
python311Packages.mindsdb-evaluator
python311Packages.mindsdb-evaluator.dist
python311Packages.nox
python311Packages.nox.dist
python311Packages.online-judge-api-client
python311Packages.online-judge-api-client.dist
python311Packages.pyenvisalink
python311Packages.pyenvisalink.dist
python311Packages.pygls
python311Packages.pygls.dist
python311Packages.pyisy
python311Packages.pyisy.dist
python311Packages.pypca
python311Packages.pypca.dist
python311Packages.pysim
python311Packages.pysim.dist
python311Packages.python-lsp-ruff
python311Packages.python-lsp-ruff.dist
python311Packages.riscof
python311Packages.riscof.dist
python311Packages.riscv-isac
python311Packages.riscv-isac.dist
python311Packages.segments
python311Packages.segments.dist
python311Packages.skybellpy
python311Packages.skybellpy.dist
python311Packages.type-infer
python311Packages.type-infer.dist
python312Packages.aio-pika
python312Packages.aio-pika.dist
python312Packages.aiomisc
python312Packages.aiomisc-pytest
python312Packages.aiomisc-pytest.dist
python312Packages.aiomisc.dist
python312Packages.aiormq
python312Packages.aiormq.dist
python312Packages.colorlog
python312Packages.colorlog.dist
python312Packages.liquidctl
python312Packages.liquidctl.dist
python312Packages.liquidctl.man
python312Packages.lupupy
python312Packages.lupupy.dist
python312Packages.pyenvisalink
python312Packages.pyenvisalink.dist
python312Packages.pypca
python312Packages.pypca.dist
rapid-photo-downloader
rapid-photo-downloader.dist
ruff-lsp
ruff-lsp.dist
sbomnix
sbomnix.dist
vimPlugins.YouCompleteMe
vscode-extensions.ms-python.python
ycmd

Result of nixpkgs-review pr 284695 run on x86_64-linux 1

20 packages marked as broken and skipped:

python312Packages.chat-downloader
python312Packages.chat-downloader.dist
python312Packages.cot
python312Packages.cot.dist
python312Packages.jaraco-abode
python312Packages.jaraco-abode.dist
python312Packages.jedi-language-server
python312Packages.jedi-language-server.dist
python312Packages.lsprotocol
python312Packages.lsprotocol.dist
python312Packages.nox
python312Packages.nox.dist
python312Packages.online-judge-api-client
python312Packages.online-judge-api-client.dist
python312Packages.online-judge-tools
python312Packages.online-judge-tools.dist
python312Packages.pygls
python312Packages.pygls.dist
python312Packages.skybellpy
python312Packages.skybellpy.dist

4 packages failed to build:

apache-airflow
apache-airflow.dist
python312Packages.segments
python312Packages.segments.dist

116 packages built:

bitbake-language-server
bitbake-language-server.dist
cmake-language-server
cmake-language-server.dist
cot (python311Packages.cot)
cot.dist (python311Packages.cot.dist)
errbot
errbot.dist
gkraken
gnomeExtensions.freon
home-assistant-component-tests.abode
home-assistant-component-tests.comelit
home-assistant-component-tests.isy994
liquidctl (python311Packages.liquidctl)
liquidctl.dist (python311Packages.liquidctl.dist)
liquidctl.man (python311Packages.liquidctl.man)
octoprint
octoprint.dist
online-judge-template-generator
online-judge-template-generator.dist
online-judge-tools (python311Packages.online-judge-tools)
online-judge-tools.dist (python311Packages.online-judge-tools.dist)
phonemizer (python311Packages.phonemizer)
phonemizer.dist (python311Packages.phonemizer.dist)
python311Packages.aio-pika
python311Packages.aio-pika.dist
python311Packages.aiocomelit
python311Packages.aiocomelit.dist
python311Packages.aiofile
python311Packages.aiofile.dist
python311Packages.aiomisc
python311Packages.aiomisc-pytest
python311Packages.aiomisc-pytest.dist
python311Packages.aiomisc.dist
python311Packages.aiormq
python311Packages.aiormq.dist
python311Packages.caio
python311Packages.caio.dist
python311Packages.chat-downloader
python311Packages.chat-downloader.dist
python311Packages.clldutils
python311Packages.clldutils.dist
python311Packages.colorlog
python311Packages.colorlog.dist
python311Packages.dask-gateway-server
python311Packages.dask-gateway-server.dist
python311Packages.dataprep-ml
python311Packages.dataprep-ml.dist
python311Packages.jaraco-abode
python311Packages.jaraco-abode.dist
python311Packages.jedi-language-server
python311Packages.jedi-language-server.dist
python311Packages.lsprotocol
python311Packages.lsprotocol.dist
python311Packages.lupupy
python311Packages.lupupy.dist
python311Packages.mindsdb-evaluator
python311Packages.mindsdb-evaluator.dist
python311Packages.nox
python311Packages.nox.dist
python311Packages.online-judge-api-client
python311Packages.online-judge-api-client.dist
python311Packages.optuna
python311Packages.optuna.dist
python311Packages.pyenvisalink
python311Packages.pyenvisalink.dist
python311Packages.pygls
python311Packages.pygls.dist
python311Packages.pyisy
python311Packages.pyisy.dist
python311Packages.pypca
python311Packages.pypca.dist
python311Packages.pysim
python311Packages.pysim.dist
python311Packages.python-lsp-ruff
python311Packages.python-lsp-ruff.dist
python311Packages.riscof
python311Packages.riscof.dist
python311Packages.riscv-isac
python311Packages.riscv-isac.dist
python311Packages.segments
python311Packages.segments.dist
python311Packages.skybellpy
python311Packages.skybellpy.dist
python311Packages.type-infer
python311Packages.type-infer.dist
python312Packages.aio-pika
python312Packages.aio-pika.dist
python312Packages.aiomisc
python312Packages.aiomisc-pytest
python312Packages.aiomisc-pytest.dist
python312Packages.aiomisc.dist
python312Packages.aiormq
python312Packages.aiormq.dist
python312Packages.clldutils
python312Packages.clldutils.dist
python312Packages.colorlog
python312Packages.colorlog.dist
python312Packages.liquidctl
python312Packages.liquidctl.dist
python312Packages.liquidctl.man
python312Packages.lupupy
python312Packages.lupupy.dist
python312Packages.pyenvisalink
python312Packages.pyenvisalink.dist
python312Packages.pypca
python312Packages.pypca.dist
rapid-photo-downloader
rapid-photo-downloader.dist
ruff-lsp
ruff-lsp.dist
sbomnix
sbomnix.dist
vimPlugins.YouCompleteMe
vscode-extensions.ms-python.python
ycmd

4 packages failed to build:

apache-airflow
apache-airflow.dist
python312Packages.segments
python312Packages.segments.dist

apache-airflow will be fixed by #286828 and segments is just as broken on master

Result of nixpkgs-review pr 284695 run on x86_64-linux 1

25 packages marked as broken and skipped:

apache-airflow
apache-airflow.dist
home-assistant-component-tests.abode
home-assistant-component-tests.comelit
home-assistant-component-tests.isy994
python312Packages.chat-downloader
python312Packages.chat-downloader.dist
python312Packages.cot
python312Packages.cot.dist
python312Packages.jaraco-abode
python312Packages.jaraco-abode.dist
python312Packages.jedi-language-server
python312Packages.jedi-language-server.dist
python312Packages.lsprotocol
python312Packages.lsprotocol.dist
python312Packages.nox
python312Packages.nox.dist
python312Packages.online-judge-api-client
python312Packages.online-judge-api-client.dist
python312Packages.online-judge-tools
python312Packages.online-judge-tools.dist
python312Packages.pygls
python312Packages.pygls.dist
python312Packages.skybellpy
python312Packages.skybellpy.dist

2 packages failed to build:

python312Packages.segments
python312Packages.segments.dist

113 packages built:

bitbake-language-server
bitbake-language-server.dist
cmake-language-server
cmake-language-server.dist
cot (python311Packages.cot)
cot.dist (python311Packages.cot.dist)
errbot
errbot.dist
gkraken
gnomeExtensions.freon
liquidctl (python311Packages.liquidctl)
liquidctl.dist (python311Packages.liquidctl.dist)
liquidctl.man (python311Packages.liquidctl.man)
octoprint
octoprint.dist
online-judge-template-generator
online-judge-template-generator.dist
online-judge-tools (python311Packages.online-judge-tools)
online-judge-tools.dist (python311Packages.online-judge-tools.dist)
phonemizer (python311Packages.phonemizer)
phonemizer.dist (python311Packages.phonemizer.dist)
python311Packages.aio-pika
python311Packages.aio-pika.dist
python311Packages.aiocomelit
python311Packages.aiocomelit.dist
python311Packages.aiofile
python311Packages.aiofile.dist
python311Packages.aiomisc
python311Packages.aiomisc-pytest
python311Packages.aiomisc-pytest.dist
python311Packages.aiomisc.dist
python311Packages.aiormq
python311Packages.aiormq.dist
python311Packages.caio
python311Packages.caio.dist
python311Packages.chat-downloader
python311Packages.chat-downloader.dist
python311Packages.clldutils
python311Packages.clldutils.dist
python311Packages.colorlog
python311Packages.colorlog.dist
python311Packages.dask-gateway-server
python311Packages.dask-gateway-server.dist
python311Packages.dataprep-ml
python311Packages.dataprep-ml.dist
python311Packages.jaraco-abode
python311Packages.jaraco-abode.dist
python311Packages.jedi-language-server
python311Packages.jedi-language-server.dist
python311Packages.lsprotocol
python311Packages.lsprotocol.dist
python311Packages.lupupy
python311Packages.lupupy.dist
python311Packages.mindsdb-evaluator
python311Packages.mindsdb-evaluator.dist
python311Packages.nox
python311Packages.nox.dist
python311Packages.online-judge-api-client
python311Packages.online-judge-api-client.dist
python311Packages.optuna
python311Packages.optuna.dist
python311Packages.pyenvisalink
python311Packages.pyenvisalink.dist
python311Packages.pygls
python311Packages.pygls.dist
python311Packages.pyisy
python311Packages.pyisy.dist
python311Packages.pypca
python311Packages.pypca.dist
python311Packages.pysim
python311Packages.pysim.dist
python311Packages.python-lsp-ruff
python311Packages.python-lsp-ruff.dist
python311Packages.riscof
python311Packages.riscof.dist
python311Packages.riscv-isac
python311Packages.riscv-isac.dist
python311Packages.segments
python311Packages.segments.dist
python311Packages.skybellpy
python311Packages.skybellpy.dist
python311Packages.type-infer
python311Packages.type-infer.dist
python312Packages.aio-pika
python312Packages.aio-pika.dist
python312Packages.aiomisc
python312Packages.aiomisc-pytest
python312Packages.aiomisc-pytest.dist
python312Packages.aiomisc.dist
python312Packages.aiormq
python312Packages.aiormq.dist
python312Packages.clldutils
python312Packages.clldutils.dist
python312Packages.colorlog
python312Packages.colorlog.dist
python312Packages.liquidctl
python312Packages.liquidctl.dist
python312Packages.liquidctl.man
python312Packages.lupupy
python312Packages.lupupy.dist
python312Packages.pyenvisalink
python312Packages.pyenvisalink.dist
python312Packages.pypca
python312Packages.pypca.dist
rapid-photo-downloader
rapid-photo-downloader.dist
ruff-lsp
ruff-lsp.dist
sbomnix
sbomnix.dist
vimPlugins.YouCompleteMe
vscode-extensions.ms-python.python
ycmd


KubeCon Docs Sprint: Update Weights for Setup and Subfolders

This PR updates duplicated weights in the setup folder and it sub folders.
Related # https://github.com/kubernetes/website/issues/35093

Thanks @cailynse!
/lgtm
/approve

@reylejano @natalisucks deploy preview fail... could you take a look?

/kind cleanup
/approve

/retest

I'm afraid that Prow doesn't know how to trigger another build attempt by Netlify. Not yet.

I reduced a weight that was 30 above to be only 20 above in order to trigger the netlify build again

/lgtm


Publish Microsoft Authored templates to azure samples browser

Update metadata, and publish agent-openai-python-prompty template to azure samples browser.
@hemarina and @cassiebreviu for notification.

@cassiebreviu Could you review this PR and merge it if it looks good? Thanks a lot.

Just seeing this PR, merging


Remove ibm terraform provider download

Now we have the provider registered in the terraform registry, hence no more download required. https://registry.terraform.io/providers/IBM-Cloud/ibm/latest

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mkumatag: Updated the job-config configmap in namespace prow at cluster default using the following files:

key kubetest2-plugins-presubmit.yaml using file config/jobs/ppc64le-cloud/kubetest2-plugins/kubetest2-plugins-presubmit.yaml

In response to this:

Now we have the provider registered in the terraform registry, hence no more download required. https://registry.terraform.io/providers/IBM-Cloud/ibm/latest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


ARROW-9429: [Python] ChunkedArray.to_numpy

While it technically is still using pandas during the conversion it exposes the to_numpy() method.
Also refactored the chunked array construction to support more flexible input, see the test case.

@kszucs Can you rebase this PR? Hopefully it will fix AppVeyor.

@pitrou rebased and addressed your comments


Use an android the Date serializer if cannot use standard java one

:pushpin: References

Issue: Initialization of KotlinSnapshot in Android app fails due to java.lang.IllegalArgumentException: Unknown pattern character 'X'. Looking through documentation, SimpleDateFormat does not support SSSX format and needs to use SSSZ format instead.

:tophat: What is the goal?
Allow Kotlin snapshot to be used on an android application
:memo: How is it being implemented?
Leave the default behavior using the standard java SimpleDateFormatter and fallback to Android formatting if the formatter fails to be created.
:boom: How can it be tested?

[ ] Use case 1: Try to use the snapshot library in Android integration test

Hi @mariannabudnikova thanks for your contribution. I totally agree with @davideme, every PR should add the coverage needed to ensure the feature is working as expected 😃
However, there is something we should review. KotlinSnapshot is not designed to work in the instrumentation tests folder. This is because when you run any instrumentation test the code is bundled into an apk which is executed inside the emulator or de device you've got connected. This means your code will be executed inside the device and not your computer so the library won't have access to the snapshots folder.
Additionally, right now KotlinSnapshot library is not added to the build as an androidTestImplementation so if you create any test inside the androidTest folder you won't be able to use any kotlin snapshot function. Maybe you want to take a look at this PR where we reviewed this 😃
There is another scenario where KotlinSnapshot can be used. When running an instrumented test as a unit test using Robolectric under the hood. However, in this case you don't actually need the serializer. I'm adding a new commit to your branch adding the tests @davideme requested using two different runners, robolectric and the JUnit 4 runners, so we can review the result. The execution of the two new tests use this serializer:
SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSSX", Locale.US)

as the default one and works properly because the code is being executed using the JVM and not the Android SDK.
After all this review I'm still wondering if there is something I'm missing @mariannabudnikova . Could you please check my comment out and let me know if there is anything else you'd like to add?


Issue-delete & search features fixed

Automated Review Comments
Issues in backend/controllers/crudController/crudMethods.js:

(6, 'Hardcoded string found')
(7, 'Hardcoded string found')
(9, 'Hardcoded string found')
Issues in frontend/src/pages/Customer.jsx:
(3, 'Hardcoded string found')
(5, 'Hardcoded string found')
(6, 'Hardcoded string found')
Issues in frontend/src/pages/Product.jsx:
(2, 'Hardcoded string found')
(4, 'Hardcoded string found')
(6, 'Hardcoded string found')

I got it.


Fixed typo in OneDrive API Endpoint Differences

Fixed typo in OneDrive API Endpoint Differences: sensative => sensitive.

@krizalys,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks @krizalys!


Optionally force HTTPS content URIs

It is a way to solve https://github.com/google/shaka-player/issues/1389

@joeyparrish can you review it?


Add multm_prev_ layer and enhance gemm() function for PLANE_WISE operations

This pull request introduces a new layer, multm_prev_, and enhances the gemm() function to support PLANE_WISE operations. These changes aim to improve the flexibility and performance of matrix multiplications in deep learning models, particularly for attention mechanisms.
New layer: multm_prev_
The multm_prev_ layer performs element-wise matrix multiplication between the current layer's input and the previous layer's output. This new layer is particularly useful for implementing attention mechanisms and other operations that require element-wise interactions between tensors.
Key features of multm_prev_:

Supports PLANE_WISE matrix multiplication
Preserves sample and channel dimensions
Efficiently handles 4D tensors

Enhancement to gemm() function:
The gemm() function has been updated to support two modes of operation: CHANNEL_WISE (default) and PLANE_WISE. This modification allows for more efficient and flexible matrix multiplications, especially when dealing with 4D tensors.
Key changes to gemm():

Added a new parameter g_mode to specify the operation mode (0 for CHANNEL_WISE, 1 for PLANE_WISE)
Implemented PLANE_WISE mode, which performs matrix multiplication for each corresponding 2D plane across all samples and channels
Updated documentation to reflect the new functionality and requirements for both modes

These changes provide greater flexibility in implementing complex neural network architectures, particularly those involving attention mechanisms or other operations requiring element-wise interactions between tensors.
A new test function, test_multm_prev(), has been added to verify the correct functionality of the multm_prev layer and the enhanced gemm() function.

Nice, I was just wondering if matmul_prev wouldn't be a better name.

We can change the name without any problem. I am already dealing with compilation issues likely due to static uses of mult_prev_ (in the template part), and we will decide on the name to keep afterward.

On the other hand, I was thinking of using the same convention for the transformation to be applied to softmax and thus have a special layer named softmaxm. So we would take mat_softmax or perhaps better msoftmax?

Would it be too difficult to have just an attention_ layer? I know that would mean doing the backpropagation by hand inside that layer, just like the loss_barlowtwins is doing it (but that one is just a bn_con).

It would be simpler to use for some people, but we would lose the flexibility to build attention in a potentially specific way (even though it currently follows fairly standard and structured steps). For instance, we can decide whether or not to mask, apply an additional filter for whether or not to remove a pad token before applying softmax, and so on. I was thinking more of providing, as you did for ResNet, an external definition file that gives a certain definition of the network...

Yes, we would lose flexibility, or maybe that layer could be initialized with a struct of options that control the behavior/features of the attention layer. But yes, it would still be less flexible.

It would be harder to implement something like flash attention without an explicit attention_ layer

Indeed, I can add a high-level declaration in the layer definition file, similar to what was done for the inception layer, like :
template <int embedding_dim, int nb_heads, typename SUBNET>
using attention_ = (...)

Sorry, I'm just catching up on these threads.  Seems like this PR is still being worked on?  There are conflicts with master in any case.   Let me know when I should look it over :)

@davis, no you can do the merging. I think the conflicts with the master come from the fact that I created several branches from my own Dlib fork to be able to work on several layers in parallel. Any new layers currently being created are finished and can be integrated please.
Technically, I still have a new layer to release but I'm going to wait until all the changes have been merged into the master branch to avoid any further conflicts... let me know if that's OK with you.
Translated with DeepL.com (free version)

@davis, no you can do the merging.

You please merge them :)
I'll review them once there aren't merge conflicts.

Technically, I still have a new layer to release but I'm going to wait until all the changes have been merged into the master branch to avoid any further conflicts... let me know if that's OK with you.

Yeah that's fine :D

@Davis, could you please review this PR?

@Davis, I think I'm on the right track now (a lot of difficulty to find an enum shared by all the classes and accessible from the CPU and CUDA codes for example) and everything is working on my side now (including the dnn.cpp test). As indicated in the closed PR, to simplify the review, I've merged all the modifications (i.e. for the two new layers) via this single PR.
However, when I run dtest, I get this error, which seems to have nothing to do with my new layers (it might have something to do with convolution). Does this sound familiar?
Running test_dnn \
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!! TEST FAILED: test_dnn !!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Failure message from test:
Error occurred at line 1518.
Error occurred in file C:\Users\Shadow\source\repos\multm-prev-layer\dlib\test\dnn.cpp.
Failing expression was max(abs(mat(filter_gradient1)-mat(filter_gradient2))) < 1e-3.
0.00897217
Testing Finished
Total number of individual testing statements executed: 955
Number of failed tests: 1
Number of passed tests: 0

I've run the test on my GPU machine (dtest --runall) and they all pass.

I've run the test on my GPU machine (dtest --runall) and they all pass.

OK, so maybe it's due to the GPU I'm using (an A4500); could it be possible to review again and merge all now? There are still other elements to work on ;-)

Nice this is great. Fix up the minor stuff I commented on and I'll merge it right away :D

Done.


where to set AlertManager Config?

who know where to set alertmanager configs? i want to set sent email to someone.
i create monitor by kubectl create -f manifests/

@mxinden  could you help me? i am getting into a flap.thank you

It seems we don't have this information documented right now, so I opened a PR to address this: #1450. Feedback would be highly appreciated! :slightly_smiling_face:

I will close this issue. @jiuchongxiao let us know if you need any further help by either opening a new issue or commenting on this one.

where position the file (alertmanager-config.jsonnet or alertmanager-config-external.jsonnet、alertmanager-config.yaml) should to put?
@brancz

jsonnet is a language, that generates json, in the readme we described how to build that and convert it to yaml. You can run kubectl create -f manifests/ once you have generated the new manifests yourself with jsonnet.

it works,thank you very much @brancz

it works,but why i cannot find the alertmanager config info in the manifest .yaml. where it goes,and why not in ymal file?
@brancz

the files need to be relative to the jsonnet file you are executing


Be clever with the blank line for API calls

It might be possible to inspect the associated API spec to determine whether a payload or anymore payload is expected and trigger execution if no more payload is possible?
The downside is to make multiple statements harder to compose in the terminal.

This feature would rely on the API spec files too heavily and it might not be the right thing.