V1 Utility Module First Iteration - Followups

Objective
Address all of the discussion and implementation points left in the primary PR: https://github.com/pokt-network/pocket/pull/47
Origin Document
Issue #24: V1 Utility Module First Iteration
Goals
Deliverables

[ ] Check of all the boxes below
[ ] Add documentation or refactor code where applicable

Testing Methodology
$ make test_utility_types && make test_utility_module
Non-goals

Add any new functionality to the V1 utility module

Creator: @Olshansk
Co-Owners: @andrewnguyen22

Short-term refactor

[x] Use semantic variable naming for some of the constants. Related, we set ZeroInt when unpausing, but we do checks using magic value (e.g. “height == 0”)
[x] Given the “ethos” of basic types, why not use a uint64 to represent timestamps as an epoch timestamp instead of google.protobuf.Timestamp time?

Mid-term refactor

[x] Consider have an “Actor” interface with common functions (Pause, Unpause, Edit, Stake, Unstake, etc…)
[ ] Change all the tests to use testify

E.g. Change if err != nil { t.Fatal() } to require.NoError(t, err)
@Olshansk  can write a script to do this

[ ] Some small store related helper functions (e.g. InsertApplication) are only used in one place. For the purpose of verbosity, why not move the business logic inline? Alternatively, maybe we can just lowercase them?
[x] Some functions (like CalculateUnstakingHeight) are used across both the validator and fisherman. Should we have an "utility_context_actor_utils.go` type file (obviously with a different name?). This could potentially be a bigger refactor too.

Long-term refactor

[x] Avoid having isTesting (isDebug / isDevelop) if/else branches throughout the code, but try to use injection of handlers instead. Example: see persistence/pre_persistence/genesis.go.

Discussion - Utility

[x] *** Who is the signer? In different places we have the Address / PublicKey, the Output / OutputAddress, and an optional signer. What is the difference between these three?
[x] When / how are evidence and votes gossiped?
[ ] Discuss if we should use make / new / create for factory functions, just so we are all consistent
[x] Various protos / functions specify both an address and a public_key. The former can be derived from the latter, so specifying or passing in both is an entry point for potential errors / discrepancy. E.g. app.proto

@Olshansk believes the simpler interface and lower probably of error is better than @andrewnguyen22's argument related to compute cost

[ ] In service_node.go, we validate the relay chain count but not the actual values within them. Why not?
[x] Renaming output in various places to something clearer and more self-explanatory?

[x] Also, pick either output or output_address (e.g. used in UnstakingActor) and use the same one everywhere

[ ] Why is height not a uint64?
[x] What’s a pool?

Need a better way to explain this
Should we use an enum instead of strings?
Will this list be kept small or do we see it growing? Staking pools?
Source: https://github.com/pokt-network/pocket-network-protocol/tree/main/utility#36-account-protocol

[ ] Why not use enums for Code errors?
[x] Remove Begin from all functions that start with this?
[x] Percentage computations - why not create a helper for multiplication, division & sign check?
[ ] Should we update README or changeling to explain why amounts are strings and percentage are ints?
[ ] Constants in utility/types/session.go look more like errors, but mixed with business logic constants. How should we separate them?

Discussion - General

[ ] Each module should have a CHANGELOG
[ ] This is a good reference for go: https://dave.cheney.net/practical-go/presentations/gophercon-israel.html#_avoid_named_return_values. For example, the recommendation to avoid using named return types helped me catch a few issues.

AIs on @Olshansk :

[x] Go through the list of topics above with the team
[ ] Show an example of how gov.go can be refactored and made simpler using mappers but still keeping the verbosity in place
[ ] Write a script to refactor the tests to use testify
[x] Add changelog to consensus module
[ ] While everything is still fresh, add a README on “how to navigate this module”
[x] Consolidate errors into one file in consensus.go (similar to this)

@derrandz Feel free to add additional questions/comments related to the V1 utility iteration here (inline or link to it).

Note that all of the action & discussion items have been either addressed in https://github.com/pokt-network/pocket/pull/64 or discussed in internal team meetings. If you'd like access to the notes or recordings, please ping myself or someone else on the protocol team directly and we'd be happy to collaborate.
Leaving this open until the low prioriiry open-ended items are tended to or deemed not necessary.

Closing for housekeeping.


Run multiple dapr apps using dapr run -f documentation

What content needs to be created or modified?
Documentation for running multiple Dapr apps using dapr run -f
Describe the solution you'd like

Scenario - why is this useful? When should I use it?
How can I use this capability?

Where should the new material be placed?
Developing Applications > Local development > Dapr Run

@mukundansundar FYI I can help with this - I'll use info from the community call today and the proposal to get started and will reach out if I have questions or need a review


CFBundleIdentifier Error

When I launch the app using yarn run:ios I receive the error:
Failed to install the requested application An application bundle was not found at the provided path. Provide a valid path to the desired application bundle. Print: Entry, ":CFBundleIdentifier", Does Not Exist
Any idea?
Thank you for your great project,
Stefano

Hello @botsbuzz!
Thank you so much for being interested in our project. Actually, it's a common problem with RN and XCode 10. Have you tried this solution: https://stackoverflow.com/a/41854111/9582178 ?
Please, let me know if it worked for you.


PowershellStandard 7?

Powershell Standard 7.0.0 Preview was pushed to Nuget 5 days ago, but there are no related commits here. Has this open source development been moved or abandoned? What's the deal?
https://www.nuget.org/packages/PowerShellStandard.Library/7.0.0-preview.1
@joeyaiello @rjmholt @TylerLeonhardt

@JamesWTruher @SteveL-MSFT

@JustinGrote just running a bit behind, it's there now. I've merged the branch that was used to release the preview.


Implemented Filecoin.StateReadState API

Summary of changes

Changes introduced in this pull request:

Added support for Filecoin.StateReadState API

Reference issue to close (if applicable)

As part of https://github.com/ChainSafe/forest/issues/3639
Other information and links

Change checklist

[X] I have performed a self-review of my own code,
[X] I have made corresponding changes to the documentation,
[X] I have added tests that prove my fix is effective or that my feature works (if possible),
[X] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

@sudo-shashank please read this thread: https://github.com/ChainSafe/forest/pull/3719#discussion_r1395345666
It's not a blocker, but it would make the codebase more uniform. Basically stick most of the logic into state_manager and just call a couple of methods in the controller.
Approving this anyway.


[Event Request] Table 7302 - "Bin Content" - OnBeforeCalcQtyAvailToTakeUOM

Dear Support,
We would like to add a new event OnBeforeCalcQtyAvailToTakeUOM in the following Procedure below, using the IsHandled pattern. This would allow us to further customize the code and migrate existing customizations.
procedure CalcQtyAvailToTakeUOM(): Decimal
var
    IsHandled: Boolean;  //New  Code --++
begin
    //New  Code --
    IsHandled := false;
    OnBeforeCalcQtyAvailToTakeUOM(Rec, IsHandled);
    if IsHandled then
    exit;
    //New  Code ++
    GetItem("Item No.");
    if Item."No." <> '' then
        exit(
          Round(CalcQtyAvailToTake(0) / UOMMgt.GetQtyPerUnitOfMeasure(Item, "Unit of Measure Code"), UOMMgt.QtyRndPrecision()));
end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeCalcQtyAvailToTakeUOM(var BinContent: Record "Bin Content"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52890.


Add cover art to the API

Hi,
Thank you for making the LibriVox API available! This PR aims to add cover art support (PDF/JPG/thumbnail) to the API when ?extended=1. I'm still working on setting up the catalog locally, so I haven't been able to fully test it yet.
Thanks!
(Closes #245)

Hi,
Thanks for the details! I've installed the catalog locally and tested it with several books (including those without covers). It seems to be working properly – the coverart_jpg/coverart_pdf/coverart_thumbnail return the IA URL (or null if no cover is available). I've also squashed the changes into a single commit.
Thanks!

Thanks for the feedback! I've added a coverart parameter (?coverart=1). Please let me know if there's anything else that needs to be changed.

And, it appears the check has failed due to a temporary network error (can't download Wordpress). I can't clear it to run again, and I also won't be available to test this version until at least next week, but at least it's seen. 😅

Not quite - WordPress.com took over ACF and renamed it, changing its URL and making the one we were using to download it invalid. There was a whole kerfuffle about it, even made its way onto Hacker News [1]. https://github.com/LibriVox/librivox-ansible/pull/33 fixes it.
[1] https://news.ycombinator.com/item?id=41821336

@redrun45 as far as I know, the standard practice way of evolving APIs is to version them. In my OpenStack day job, we use a thing called "microversions". I believe the same overall idea applies elsewhere, even in web apps. The principle is that for every version of the API, the set of things that can be requested, as well as the possible responses, are well defined and fixed. Any chance to the API, be it in input or output, is a new version.
If we get to the point that we have enough changes to our API, we can move in that direction. I think that for now, what @fakerybakery has proposed is good enough.
[1] https://docs.openstack.org/api-guide/compute/microversions.html

@fakerybakery I'm fine with this as-is. Before merging, would you be willing to add tests for your new API? We have an old code base with a lot of gaps to cover, but at least for new things we're trying to move towards having tests where reasonable, and this is the perfect candidate for a test :) If not, either @redrun45 or myself can follow up with a separate PR.

I've attempted tests for this in https://github.com/LibriVox/librivox-catalog/pull/248
I probably got a whole bunch of stuff wrong, but I wanted to get something posted to prove it out. The main point of that PR is the DB seeder based on https://github.com/kenjis/ci-phpunit-test/blob/3.x/docs/HowToWriteTests.md#database-seeding that allows us to load up our scrubbed DB before every test.


RSDK-7258: change status to message type in board api

@Simberific do you have to merge this since it's breaking? If it's someone else let me know!


Order exact matches before word matches in command palette

Type: Bug

Open command palette and type open settings

Bug
The entry with the exact text open settings is shown after entries that match on open and settings

VS Code version: Code - Insiders 1.88.0-insider (Universal) (57028e3d274ebde851c7cc02123def372ca619a6, 2024-04-03T04:45:14.890Z)
OS version: Darwin arm64 23.4.0
Modes:

System Info

Item
Value

CPUs
Apple M2 Max (12 x 24)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: disabled_softwarevideo_encode: disabled_softwarewebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: unavailable_software

Load (avg)
3, 3, 2

Memory (System)
64.00GB (0.36GB free)

Process Argv
--crash-reporter-id 0fffb5da-9cd7-46fd-9e7f-a1564e8c5fda

Screen Reader
no

VM
0%

This works as designed, we intentionally show results in alphabetical order to group them together and make order predictive.

/duplicate https://github.com/microsoft/vscode/issues/173767

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Subledger batches

HI,
Partner contacted me regarding subledger transfer to General ledger. I shared this link but information seems not that clear on what should be done
Asynchronous transfer of subledger batches requires that the batch server be set up, online, and working. Otherwise, the Asynchronous transfer option won't work.
Can we bring some clarification on this part?
Where can customer setup batch?
how can customer setup batch (what are the option and best way to setup batches)?
....
Bug has been also created but not published
https://msdyneng.visualstudio.com/FinOps/_queries/edit/569698/?triage=true
Need to specify clearly that Asynchronous option is NO more an option and batch job is somehow the only option.
Kind regards
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c9043d35-d539-a24c-3802-55e4fc7ad04c
Version Independent ID: e6298f3e-7706-cd1f-79d8-88da48ae345a
Content: Subledger transfer to the general ledger - Finance | Dynamics 365
Content Source: articles/finance/general-ledger/subledger-transfer.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @rcarlson
Microsoft Alias: peakerbl

@anboudin - Thank you for the questions.
@Peakerbl - Can you please take a look and address these questions? Thanks!

created bug 642439 to track this issue.

https://github.com/MicrosoftDocs/Dynamics-365-Operations/pull/12750

This topic has been updated. The changes will be published in the next few hours. Thanks!


Interrupt wdt timeout when repeatedly enabling/disable BT controller and BLE adv (IDFGH-492)

Environment

Development Kit:      [ESP32-DevKitC]
Kit version (for WroverKit/PicoKit/DevKitC): [v1]
Module or chip used:  [ESP32-WROOM-32]
IDF version (run git describe --tags to find it): v3.3-dev-396-g8c7a6be
Build System:         [Make]
Compiler version (run xtensa-esp32-elf-gcc --version to find it): 1.22.0-80-g6c4433a
Operating System:     [Linux]
Power Supply:         [USB]

Problem Description
Recently when updating to latest ESP-IDF we started experiencing interrupt watchdog timeout crashes related to enabling/disabling BLE advertisements. I was able to reproduce the problem by patching the ble_adv example code. Bisected the problem and found it started when the following patch (in components/bt/lib) got in:
commit cd196645e29d0d7b050c1aadec850d285fbb1da6
Author: Tian Hao<EMAIL_ADDRESS>Date:   Tue Nov 13 17:28:03 2018 +0800
    fix controller disable cause some problem bug (ac67b340)

    1. Fix the bug that disable controller suddenly cause crash when ble is working. (585cfba1)
    2. Fix the bug that when disable controller, it will cause wifi cannot RX,  TX timeout,  buffer lack and etc. (f8546145)
    3. Update dport access as the newest version of IDF. (ac67b340)

Expected Behavior

Enable Bluetooth controller
Start BLE adv
Stop BLE adv
Disable Bluetooth controller
Repeat from 1.

No crash should be seen
Actual Behavior
Already second time BLE adv is started an interrupt wdt reset is seen.
Steps to repropduce

Apply the attached patch to esp-idf repo
Build and flash the ble_adv example code with default defconfig
Let example code run for just a few seconds
Notice the crash.

Code to reproduce this issue
Please see the attached patch file to be applied to esp-idf repo.
Debug Logs
Toolchain path: esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: crosstool-ng-1.22.0-80-g6c4433a
Compiler version: 5.2.0
Python requirements from esp/esp-idf/requirements.txt are satisfied.
MONITOR
--- idf_monitor on /dev/ttyUSB0 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57

rst:0x1 (POWERON_RESET),boot:0x3 (DOWNLOAD_BOOT(UART0/UART1/SDIO_REI_REO_V2))
waiting for download
ets Jun  8 2016 00:22:57

rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:6220
load:0x40078000,len:10248
load:0x40080400,len:6552
entry 0x40080764
I (29) boot: ESP-IDF v3.3-dev-397-ga03fd47-dirty 2nd stage bootloader
I (29) boot: compile time 11:23:13
I (30) boot: Enabling RNG early entropy source...
I (35) boot: SPI Speed      : 40MHz
I (39) boot: SPI Mode       : DIO
I (43) boot: SPI Flash Size : 4MB
I (47) boot: Partition Table:
I (51) boot: ## Label            Usage          Type ST Offset   Length
I (58) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (66) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (73) boot:  2 factory          factory app      00 00 00010000 00100000
I (81) boot: End of partition table
I (85) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x0ed4c ( 60748) map
I (115) esp_image: segment 1: paddr=0x0001ed74 vaddr=0x3ff80000 size=0x00000 (     0) load
I (115) esp_image: segment 2: paddr=0x0001ed7c vaddr=0x3ff80000 size=0x00000 (     0) load
I (122) esp_image: segment 3: paddr=0x0001ed84 vaddr=0x3ffbdb60 size=0x0128c (  4748) load
I (133) esp_image: segment 4: paddr=0x00020018 vaddr=0x400d0018 size=0x3a254 (238164) map
0x400d0018: _stext at ??:?

I (223) esp_image: segment 5: paddr=0x0005a274 vaddr=0x3ffbedec size=0x0155c (  5468) load
I (225) esp_image: segment 6: paddr=0x0005b7d8 vaddr=0x3ffc0348 size=0x00000 (     0) load
I (230) esp_image: segment 7: paddr=0x0005b7e0 vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at esp/esp-idf/components/freertos/xtensa_vectors.S:1685

I (239) esp_image: segment 8: paddr=0x0005bbe8 vaddr=0x40080400 size=0x11514 ( 70932) load
I (277) esp_image: segment 9: paddr=0x0006d104 vaddr=0x400c0000 size=0x00000 (     0) load
I (277) esp_image: segment 10: paddr=0x0006d10c vaddr=0x50000000 size=0x00000 (     0) load
I (284) esp_image: segment 11: paddr=0x0006d114 vaddr=0x50000000 size=0x00000 (     0) load
I (303) boot: Loaded app from partition at offset 0x10000
I (303) boot: Disabling RNG early entropy source...
I (305) cpu_start: Pro cpu up.
I (308) cpu_start: Application information:
I (313) cpu_start: Project name:     ble_adv
I (318) cpu_start: App version:      v3.3-dev-397-ga03fd47-dirty
I (325) cpu_start: Compile time:     11:23:12
I (330) cpu_start: Compile date:     Dec  6 2018
I (335) cpu_start: ESP-IDF:          v3.3-dev-397-ga03fd47-dirty
I (342) cpu_start: Starting app cpu, entry point is 0x40081058
0x40081058: call_start_cpu1 at esp/esp-idf/components/esp32/cpu_start.c:261

I (0) cpu_start: App cpu up.
I (352) heap_init: Initializing. RAM available for dynamic allocation:
I (359) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (365) heap_init: At 3FFB6388 len 00001C78 (7 KiB): DRAM
I (371) heap_init: At 3FFB9A20 len 00004108 (16 KiB): DRAM
I (377) heap_init: At 3FFBDB5C len 00000004 (0 KiB): DRAM
I (383) heap_init: At 3FFC2D78 len 0001D288 (116 KiB): DRAM
I (390) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (396) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (402) heap_init: At 40091914 len 0000E6EC (57 KiB): IRAM
I (409) cpu_start: Pro cpu start user code
I (91) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (177) BTDM_INIT: BT controller compile version [ac67b34]
I (177) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE
BLE advt task start
I (1257) phy: phy_version: 4000, b6198fa, Sep  3 2018, 15:11:06, 0, 0
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 1
host rcv pkt: 040e0405030c00
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 2
host rcv pkt: 040e0405062000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 3
host rcv pkt: 040e0405082000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 4
host rcv pkt: 040e04050a2000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 5
host rcv pkt:<PHONE_NUMBER>000000
host rcv pkt: 040e04050a2000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 6
BLE Advertise, flag_send_avail: 1, cmd_sent: 7
BLE Advertise, flag_send_avail: 1, cmd_sent: 1
host rcv pkt: 040e0405030c00
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 2
host rcv pkt: 040e0405062000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 3
host rcv pkt: 040e0405082000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 4
host rcv pkt: 040e04050a2000
controller rcv pkt ready
BLE Advertise, flag_send_avail: 1, cmd_sent: 5
ASSERT_PARAM(512 0), in rwble.c at line 222
Guru Meditation Error: Core  0 panic'ed (Interrupt wdt timeout on CPU0)
Core 0 register dump:
PC      : 0x400846d1  PS      : 0x00060034  A0      : 0x80087cee  A1      : 0x3ffbe620
0x400846d1: r_assert_param at ??:?

A2      : 0x00000001  A3      : 0x00000000  A4      : 0x00000000  A5      : 0x60008054
A6      : 0x3ffbdbbc  A7      : 0x60008050  A8      : 0x800846d1  A9      : 0x3ffbe600
A10     : 0x00000004  A11     : 0x00000000  A12     : 0x6000804c  A13     : 0xffffffff
A14     : 0x00000000  A15     : 0xfffffffc  SAR     : 0x00000004  EXCCAUSE: 0x00000005
EXCVADDR: 0x00000000  LBEG    : 0x40084609  LEND    : 0x40084610  LCOUNT  : 0x00000000
0x40084609: r_assert_param at ??:?

0x40084610: r_assert_param at ??:?

Core 0 was running in ISR context:
EPC1    : 0x40109bbe  EPC2    : 0x00000000  EPC3    : 0x00000000  EPC4    : 0x400846d1
0x40109bbe: esp_pm_impl_waiti at esp/esp-idf/components/esp32/pm_esp32.c:487

0x400846d1: r_assert_param at ??:?

Backtrace: 0x400846d1:0x3ffbe620 0x40087ceb:0x3ffbe640 0x400888b7:0x3ffbe660 0x400825e1:0x3ffbe680 0x40109bbb:0x00000000
0x400846d1: r_assert_param at ??:?

0x40087ceb: r_rwble_isr at ??:?

0x400888b7: r_rwbtdm_isr_wrapper at intc.c:?

0x400825e1: _xt_lowint1 at esp/esp-idf/components/freertos/xtensa_vectors.S:1105

0x40109bbb: esp_pm_impl_waiti at esp/esp-idf/components/esp32/pm_esp32.c:487

Core 1 register dump:
PC      : 0x40109bbe  PS      : 0x00060634  A0      : 0x800d187e  A1      : 0x3ffbc330
0x40109bbe: esp_pm_impl_waiti at esp/esp-idf/components/esp32/pm_esp32.c:487

A2      : 0x00000000  A3      : 0x80000001  A4      : 0x00000000  A5      : 0x00000001
A6      : 0x00000003  A7      : 0x00060023  A8      : 0x800d2892  A9      : 0x3ffbc300
A10     : 0x00000000  A11     : 0x00060623  A12     : 0x00060620  A13     : 0x00000001
A14     : 0x00060620  A15     : 0x3ffbd250  SAR     : 0x00000000  EXCCAUSE: 0x00000005
EXCVADDR: 0x00000000  LBEG    : 0x00000000  LEND    : 0x00000000  LCOUNT  : 0x00000000

Backtrace: 0x40109bbe:0x3ffbc330 0x400d187b:0x3ffbc350 0x4008fc11:0x3ffbc370 0x4008ec55:0x3ffbc390
0x40109bbe: esp_pm_impl_waiti at esp/esp-idf/components/esp32/pm_esp32.c:487

0x400d187b: esp_vApplicationIdleHook at esp/esp-idf/components/esp32/freertos_hooks.c:86

0x4008fc11: prvIdleTask at esp/esp-idf/components/freertos/tasks.c:3537

0x4008ec55: vPortTaskWrapper at esp/esp-idf/components/freertos/port.c:403

Rebooting...

Other items if possible
esp-idf patch file to apply for reproducing the problem
0001-Repeatedly-turn-on-off-bt-controller-and-ble-adverti.patch.txt

@Weijian-Espressif I saw that fix and tried with an updated IDF already but unfortunately that didn't solve the problem. Updated today to latest IDF (v3.3-beta1-44-gda2116f) but problem is still there.
The patch I attached to this case reproduces the problem within less than a minute. Can you please check if you're able to reproduce the problem with this?

@mikaelkanstrup, I reproduced this bug . We have fixed this bug and are doing stress testing. If the code is merged into IDF, I will notify you in time, thanks.

@Weijian-Espressif Ok great. Thanks!

I am still experiencing this exact problem on v3.3 right now.
I can disable BLE, sleep, wake up, enable BLE all with no issues. As soon as I try to advertise, the chip resets.
Was there a fix for this that just didn't make it into v3.3?

For anyone who may come here looking for a work-around like I did, here is my solution:
I completely deinit the bt/ble stack before going to sleep:
esp_bluedroid_disable();
esp_bluedroid_deinit();
esp_bt_controller_disable();
esp_bt_controller_deinit();

Then bring it all back up when waking:
esp_bt_controller_init(&bt_cfg);
esp_bt_controller_enable(ESP_BT_MODE_BLE);
esp_bluedroid_init();
esp_bluedroid_enable();

Doing it this way seems to work for me right now.

Hello @sowmimalu , we have known this issue and the solution is under test. For now, if you want to close the Bluetooth functions, we recommend you do the following things.
esp_bluedroid_disable();
esp_bluedroid_deinit();
esp_bt_controller_disable();
esp_bt_controller_deinit();

And you can restart the Bluetooth functions whit following things.
esp_bt_controller_init(&bt_cfg);
esp_bt_controller_enable(ESP_BT_MODE_BLE);
esp_bluedroid_init();
esp_bluedroid_enable();

And I will inform you here when the fix solution is release. Thank you.

This is fixed in version after release/v3.3


Cannot use custom certificate

I have attempted in many different forms to use a custom certificate, but did not succeed.
    if instance.ssl_certificate:
        cert = instance.ssl_certificate.open()
    instance._bot.setWebhook(webhook_url=url,
                             certificate=cert)

cert is None because FieldFile::open() returns None. I'm not sure how to work around this. It seems that Telegram library is expecting an instance of a file, as I've tried to pass the certificate as text but did not succeed.
Any suggestion?
Thanks!

Closing as fixed with #23.


Implement 3-coloring for triangulated polygon

https://www.geeksforgeeks.org/graph-coloring-applications/
https://en.wikipedia.org/wiki/Graph_coloring

complete with 464fb042283fd7780e4e6d42af9cce75aef57d0b


Support automatic sizing of the virtual-scroller element

First, thanks for this awesome scrolling library. It's still the best out there and provides solutions for almost every scenario. Hopefully this adds another useful solution. We are currently using a forked build with this patch in production.
For consistent positioning and sizing of parent elements, and elements below scroll-container:
This pull request allows the height or width of the virtual-scroller element to be set automatically to (roughly) the exact required pixels to display all items, accounting for wrapping in both horizontal and vertical directions. This does NOT change the number of virtual items that are rendered on screen.
It simply allows the ngx-virtual-scroller to be used in parent containers with fixed elements and sibling elements before and after the virtual scroller in the dom. Without this patch, fixed elements will eventually scroll off the screen as the scroll-container exceeds the height of the virtual scroller element (even when set to 100% height or width.)
This patch also removes the need to manually set height or width of the virtual-scroller element when this setting is enabled for the common case.
I'm sure it is not a perfect solution, but it seems like a decent compromise for those people dealing with situations like this.
CAVEAT: This feature requires the child heights to be of equal size.
NOTE: I disabled this feature by default to maintain backwards compatibility, but I do believe it is reasonable to default this to 'enabled' in reality. I think it is more intuitive than manually setting the height or width of the scroll container, in general, and provides a fairly safe function. It can always be disabled and height/width set manually if users wish to override the default.

@whyboris Do you have thoughts about this pull request? I'd love to hear them. This is currently running in production and working very well on a high-use site.

I'm honored to be a contributor to this repository, but I would want someone like @speige to review and merge.
As far as I can tell by looking at the code it doesn't interfere with anything unless calculateDimensionsAutomatically is set, and if it's set, it will do what you describe 👍

Awesome! Thanks. I was curious as I saw your reaction but wasn't sure what it meant :) Agreed. Would love a review.

Any news about this feature? :)

I wish I knew. We cloned the repo into our local codebase because it doesn't seem like this library is actively maintained, currently.

@lincolnthree is your solution currently working correctly ? we have massive problems with it in our messenger: https://mark6-demo.web.app/messenger
you will finde bugs like this: http://prntscr.com/vpz61q

@SvenBudak Yes it's working in production, currently for both single-column and multi-column layouts.

I think our problem is the unknown dynamic height of the items. If this is also working. Can you send me please your local version? I would be happy if I can try it in our messenger.

@SvenBudak I'd just clone & checkout the repository from my branch and copy it into your app. That would be the same thing as me sending you the local version.

It solves the exact problem you're having. It's the reason we made the change... so I think it will be worth your time.

Awesome! Where can I find the full file? I am sorry if you linked it already.. I looked already before my first post. But I am a bit confused which one is complete and final.

@SvenBudak This branch: https://github.com/lincolnthree/ngx-virtual-scroller/tree/automatic-element-sizing

@lincolnthree Thank you! I have sent the branch to my colleague. he will try it out.
maybe we should create a fork and actively develop it further? it's a pity, there seem to be a lot of people who are interested in further development.

@speige -- any thoughts on this?
ps - if you were to merge it, do we still need @rintoj to create a new release, or it's something you're able to do as well?

In worst case people can copy the .ts file manually to the project. better then nothing. There is no working alternative currently.

Sorry to do this, but... Bump.

Sorry to do this, but... Bump.

Sorry to do this, but... Bump. V2


PUSH in surgical theater gives generic reject

The description has "The table has various straps and contraptions to immobilize animals during surgery. Just next to the table is an anesthesia machine, with knobs and dials, and tubes leading to the ceiling." so maybe the game should say, you don't understand any of this, so best not meddle -- or with Dr Rambaud in the consciousness, you don't have anyone to operate on, so best not meddle.

Instead of simplePushing when the player is in the Bloc Opératoire:
if the consciousness of the player is less than 4:
say "You aren't sure what does what in here and don't want to risk accidently dissecting yourself by poking randomly about[one of]. That would be embarassing[or][stopping].";
otherwise:
say "You aren't scrubbed in and all the equipment in here is sterile; you don't want to contaminate the equipment."


Fixed documentation errors in CONTRIBUTING.md

Corrected a few grammatical errors and typos. Changed 'then' to 'after this' in line 22 to reduce the usage of 'then'.
Description

A few documentation errors were fixed to improve the quality of the code.
Issue Ticket Number
Fixes #(issue_number)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x ] This change requires a documentation update

Checklist:

[ x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[ x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[ x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes needed to the documentation

@Akash190104 Sorry but, We don't accept such PR where typo is fixed


Fix bug KEM tests that used incorrect key lengths for comparison

Issue # (if available): N/A
Description of changes: While making changes for SIKE round 2, we discovered that the KAT tests had been using incorrect key lengths for comparison.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (katfix@f21f9fb). Click here to learn what that means.
The diff coverage is n/a.


Review and update github actions and automated tests before 0.50  release

This is an org-wide issue affecting several repositories.
Github actions need to be reviewed and tested. Automated tests should be updated to test on all supported platforms. We should also consolidate some of the actions so they can be shared between repos and reduce code duplication

[ ] capability-providers (action for each provider)
[ ] control-interface
[ ] examples (each example)
[ ] interfaces (each interface, and integration tests)
[ ] project-templates (gen project for each template with wash, build, and run sanity checks)
[ ] wash
[ ] wasmcloud.com-dev (check for doc builds, bad links, and spelling)
[ ] wasmcloud-dev-site (check for doc builds, bad links, and spelling)
[ ] wasmcloud-js
[ ] wasmcloud-otp
[ ] wasmcloud-test
[ ] weld

we need to find ways to share code - common scripts, docker images, and/or custom github actions.
Also for more rapid iterations developing these, we might want to look into act

Because of #246 , some of this is duplicated. I suggest the following checklist:

[ ] examples (any tests needed)
[ ] interfaces (integration tests)
[ ] project-templates (gen project for each template with wash, build, and run sanity checks)
[ ] wasmcloud.com-dev (check for doc builds, bad links, and spelling)
[ ] wasmcloud-dev-site (check for doc builds, bad links, and spelling)
[ ] wasmcloud-js (integration tests)
[ ] wasmcloud-otp (touching up integration tests, inline documentation comments)

@stevelr just checking, is the intent here just a duplicate of #246?

In #246, I had meant "publish release distributables" to mean anything that pushes to crates.io or docker or a azurecr.
This issue could be read to encompass #246.


Loggingfixes

The Log.cs class contains some apparent copy/paste bugs. For example that the Warning methods aren't checking if the logLevel is actually warning or debug, but only debug.. Or that the same method checks for logLevel "information" rather than "debug" (there's a separate method for logLevel == Information)..
Also fixed a bug where one of the Error methods were incorrectly calling .Info() instead of .Error().

Hi @Zimmergren, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Very good catch! Thank you Tobias!


Ipad2 test for Molecules and Light

I recently reverted some workaround code for removing the 'Return Molecule' button from the observation window (see 672fb1d1bc555ed4f2ca043fcee9b3d85cf2ab9a and 2feff3f3b38e6da0965b8b6d666353b23f58bc72). For optimization purposes, we were setting the opacity in an effort to better optimize the performance.
Since https://github.com/phetsims/molecules-and-light/issues/98 and https://github.com/phetsims/scenery/issues/404 are both closed, I reverted back to explicitly setting the visibility on the button.
To test, break a molecule apart (UV + NO2 works well). The button will appear, and we can either click the button to return the molecule or select a new molecule which will remove the button and render the new molecule in the observation window.

@ariel-phet That's correct, though I'd be happy to give that a go in the office tomorrow.

@mbarlow12 when possible, always good to give a quick sanity check before handing to QA (unless urgent and no access to a device). Assigning back to you for the moment.

@mbarlow12 were you able to do this test?

Not yet—the question is about some notes on performance, and I currently don't know how to check that data.

@mbarlow12 Feel free to ask @phet-steele for guidance/help

Will do—I'll also start with JB and JO about how they found the issue in the first place. Thanks.

The original issue was https://github.com/phetsims/molecules-and-light/issues/91.  We expect it to no longer be a problem because https://github.com/phetsims/scenery/issues/404 (performance issues when adding and removing drawables with gradients) has been closed.  The workaround was removed in the workaround was removed in https://github.com/phetsims/molecules-and-light/commit/2feff3f3b38e6da0965b8b6d666353b23f58bc72.
I think the only thing we need to verify is that we don't see a significant drop in framerate when the "Return Molecule" button becomes visible on iPad2 and iPad3.

@jessegreenberg and I tested this afternoon and found no issues. Closing.


Consider Pint package for units & quantities

Consider using the Pint package for Units and Quantities
http://pint.readthedocs.org/en/0.6/
It seems well developed and tested.  I've used it in several projects and cross-checked it with Wolfram Mathematica.
EDIT: it's also deeply integrated with numpy and scipy

Thanks for the suggestion, I did not know this package yet. Could you explain in what sense you use it?
Compatibility with units and quantities would be great I think. So far no effort has really been put into this. There is a quantities module, of which the idea was to keep data regarding quantities together.
The pint package seems to focus only on units, and not quantities. For example, say I have a signal which represents sound pressure. I could multiply it with pints pascal unit (or comparable). Now, if I want to plot the signal I could easily check the unit, and convert if necessary. But now I create a plot (e.g. using a method from the Signal class). On the axes I want to see the units, which I can easily obtain now, but also what quantity it is. The quantity in this case is pressure.
Is there any way to obtain the quantity from a unit instance? I doubt that is included, since you can create an infinite amount of derived quantities with the same unit. I guess this aspect, meta data about the quantity, should be considered separately from support for (arithmetic with) units .


refactor(server): review classes and removed garbage files

Ticket
Title:
Notion Link:
Description

Removed files linked on FileProject and IsSandpackFiles decorator
Passed Errors responsibilities from Services to Resolvers
Alphabetic sort methods on services and resolvers classes.
Moved codenaute cli to utils

Guidelines
🔗 Link your PRs to their related Notion tickets
✍ Use the commit convention to your pull request title
📃 If your code affects the build, update README.md
🚫 Do not merge a PR until all comments are resolved
🗑 Remove your branch after merging

Did you make sure that this PR doesn't break anything? For example, if you change something in the graphQL schemas on the server side, please make sur that you don't need to modify something in the front, relanch codegen to update queries and mutation etc....

Usually, all methods follow the client structure and pass through all tests on the server side. But before merging it, we can test run all applications and test every method changed or removed.

Did you make sure that this PR doesn't break anything? For example, if you change something in the graphQL schemas on the server side, please make sur that you don't need to modify something in the front, relanch codegen to update queries and mutation etc....

Usually, all methods follow the client structure and pass through all tests on the server side. But before merging it, we can test run all applications and test every method changed or removed.

Yes but if you changed some naming, you need to update client GraphQL schema by running codegen... Sometime, before you make some changes i've got some broke feature because of it, don't forget to check it ! :)


[T6A4][T01-T3]Yogamurti Sutanto

@yogamurti Some comments added. Please close the PR after reading comments.
As we are now using StorageStub in LogicTest, we are no longer testing the correctness of StorageFile when we test the Logic class. assertEquals(addressBook, saveFile.load()); in LogicTest should be removed because the StorageStub does not save the AddressBook data any more. Saving of data should be tested in a different test class.


cupy.fuse() returns wrong (broadcasted) shape for fused function that returns multiple arrays

Description
I have an example of a function that returns two arrays of shape [2, 2, 2] and [2, 2]. When I apply cupy.fuse to this function and run it, the returned arrays are both shaped [2, 2, 2] with the second return broadcasted to match the first returned array. This is not correct.
I expect the cupy.fuse'd function to either return the correct size arrays, or error out if this is not supported.
To Reproduce
import cupy as cp

@cp.fuse
def foo(a, b):
  return 2.0*a, 3.0*b

a = cp.ones([2, 2, 2])
b = cp.ones([2, 2])

r1, r2 = foo(a, b)
print(f"r1.shape = {r1.shape}\n r1 = {r1}")
print(f"r2.shape = {r2.shape}\n r2 = {r2}")

Running this, you will get:
r1.shape = (2, 2, 2)
 r1 = [[[2. 2.]
  [2. 2.]]

 [[2. 2.]
  [2. 2.]]]
r2.shape = (2, 2, 2)
 r2 = [[[3. 3.]
  [3. 3.]]

 [[3. 3.]
  [3. 3.]]]

Installation
Wheel (pip install cupy-***)
Environment
OS                           : Linux-5.4.0-169-generic-x86_64-with-glibc2.35
Python Version               : 3.10.12
CuPy Version                 : 13.1.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.26.4
SciPy Version                : 1.13.0
Cython Build Version         : 0.29.36
Cython Runtime Version       : None
CUDA Root                    : /opt/nvidia/hpc_sdk/Linux_x86_64/24.1/compilers
nvcc PATH                    : /opt/nvidia/hpc_sdk/Linux_x86_64/24.1/compilers/bin/nvcc
CUDA Build Version           : 12040
CUDA Driver Version          : 12020
CUDA Runtime Version         : 12040 (linked to CuPy) / 12030 (locally installed)
cuBLAS Version               : (available)
cuFFT Version                : 11012
cuRAND Version               : 10304
cuSOLVER Version             : (11, 5, 4)
cuSPARSE Version             : (available)
NVRTC Version                : (12, 3)
Thrust Version               : 200200
CUB Build Version            : 200200
Jitify Build Version         : <unknown>
cuDNN Build Version          : (not loaded; try `import cupy.cuda.cudnn` first)
cuDNN Version                : (not loaded; try `import cupy.cuda.cudnn` first)
NCCL Build Version           : 21602
NCCL Runtime Version         : 21805
cuTENSOR Version             : 20000
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla V100-DGXS-16GB
Device 0 Compute Capability  : 70
Device 0 PCI Bus ID          : 0000:07:00.0
Device 1 Name                : Tesla V100-DGXS-16GB
Device 1 Compute Capability  : 70
Device 1 PCI Bus ID          : 0000:08:00.0
Device 2 Name                : Tesla V100-DGXS-16GB
Device 2 Compute Capability  : 70
Device 2 PCI Bus ID          : 0000:0E:00.0
Device 3 Name                : Tesla V100-DGXS-16GB
Device 3 Compute Capability  : 70
Device 3 PCI Bus ID          : 0000:0F:00.0

Additional Information
No response

Thanks for the feedback! This is currently a known limitation that needs to be documented - xref #8198.

Closing as it is an expected behavior.


refac: Remove use of Jetbrains.Annotation

Description
The purpose of this PR is to remove our uses of the JetBrains.Annotation DLL.
We are currently using it for [UsedImplicitly] and often misusing it for [NotNull].
It will be easier to spot if we dont use the DLL at all.
References

How do we now obtain a green R#?

If that is a goal for us, we should probably make an issue with analysing it. There are other things that it highlights at the moment that we should also address in that case, like our InlineAutoMoqData based tests which also gets a yellow remark at the moment.

How do we now obtain a green R#?

If that is a goal for us, we should probably make an issue with analysing it. There are other things that it highlights at the moment that we should also address in that case, like our InlineAutoMoqData based tests which also gets a yellow remark at the moment.

From my personal experience, a "green R#" requirement is very simple to understand and follow and often leads to a more homogenous code style across developers. Especially when combined with using the "clean up code" feature.


dev-cmd/typecheck: Update the Sorbet TODO file in --update-definitions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] ~Have you written new tests for your changes? Here's an example.~
[x] Have you successfully run brew style with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Sorbet's TODO file is useful to be periodically regenerated when we're trying to deal with typing errors. The result is an updated sorbet/rbi/todo.rbi file that defines missing constants. During GSoC, we did update this file on occasional Tapioca updates, according to the history. That suggests to me that it should be part of this command.

For some stats, regenerating this file takes us from 40 errors (some of which are about the spoom gem and will be fixed in the next Tapioca version) to 36 errors.

Looks good to me. Thank you @issyl0! 👏🏽


fix(docs): update paths in the icons getting started page

Since the snippet to set the meta for icons now uses the path /post-icons instead or /public/post-icons the postinstall script must be updated.

Preview environments are getting posted here, as soon as they are ready!


[Patch/News] Patch method can't update comment for more than 2500 characters

Environment:
macOS Sonoma Version 14.0
Postman
Reproducible: always.
Build found:
19.10.23
Preconditions
A User is logged in;
the token is added.
Steps to reproduce
eco-news-comment-controller
Actual result
the text is sent in parameters!

Expected result
The text needs to be sent in the body
comment body from 1 to 8000 characters
Postconditions
User story and test case links
E.g.: "User story #3210
Test case"
Labels to be added
"Bug", Priority ("medium"), Severity ("minor"), Type ("Functional"), "API" (for back-end bugs).

it's 500


Recode SNPchip MatrixTable sample names

Do so in Hail based on metadata provided by GWCCG.

Handled via sample-metadata setup.


Hello would ❓

[Enter feedback here]

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

ID: 3aa65838-922f-ab99-9604-1c8d592f875d
Version Independent ID: 3a6032ca-8315-c04f-168d-bfc4cd09ddb7
Content: TLS version supported by Azure Resource Manager - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/tls-support.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @mumian
Microsoft Alias: tomfitz

@ulfatufo
Could you please elaborate the query for better understanding. So, we can assist you further.

@ulfatufo
It would be great if you could elaborate the query for better understanding. This would help us redirect the issue to the appropriate team. Thanks!

@ulfatufo
We are going to close this thread if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


fix(link-with-icon): fixes layout styles

Related Ticket(s)
#6464
Description
Update component styles affecting icon sizes at smaller breakpoints.
Changelog
New

update link span styles to prevent squishing icons at smaller breakpoints

Removed

remove extra markup tags
redundant styling in web components link-list

@kennylam percy caught some changes with the icon shifting

@DragosRistici @annawen1 @kennylam you'd better take a look at this before merging this:

@DragosRistici megamenu style discrepancies have been fixed.

@DragosRistici megamenu style discrepancies have been fixed.

LGTM, @kennylam


Twig parser error when rendering page or directing to page

BitBag\SyliusCmsPlugin\Twig\Parser\ContentParser::__construct(): Argument #1 ($twigEnvironment) must be of type Twig_Environment, Twig\Environment given, called in /Users/tsolmonkhorloo/phpstormprojects/easystore-sylius-basic/var/cache/dev/ContainerZxvRmAw/getBitbagSyliusCmsPlugin_Twig_Runtime_RenderContentService.php on line 31
i got this error when i
{{ render(path('bitbag_sylius_cms_plugin_shop_page_show_link_by_code', {'code' : 'about', 'template' : '@BitBagSyliusCmsPlugin/Shop/Page/Show/_link.html.twig'})) }}
jump to the page through this code.

Fixed with this PR https://github.com/BitBagCommerce/SyliusCmsPlugin/pull/459


twikit.errors.BadRequest: status: 400, message: "{"errors":[{"code":366,"message":"flow name LoginFlow is currently not accessible"}]}"

twikit.errors.BadRequest: status: 400, message: "{"errors":[{"code":366,"message":"flow name LoginFlow is currently not accessible"}]}"
I get this error when trying to get my cookies

Duplicate of #169


[ALLUXIO-3164] Load ACL from HDFS when loading metadata

https://alluxio.atlassian.net/browse/ALLUXIO-3164
This PR:

adds getAcl to UnderFileSystem and implement it in HdfsUnderFileSystem.
when calling loadMetadata  from Alluxio master, if ACL is supported and enabled in under file system, loads the ACL to Alluxio Inode tree.

This PR extends the CreateFileOptions to include ACL entries, so when creating an Inode in loadMetadata, the ACL entries retrieved from UFS are set into the Inode's ACL.
Another design option is to extend the SetAttributeOptions to include ACL entries, then create Inode without ACL, after the creation is completed, use setAttribute to set the ACL entries into the Inode's ACL. But setAttribute contains complicated code paths like syncing with UFS, its overhead is not necessary for loadMetadata.
In a future PR, setAttribute will take ACL into account to support client and shell operations like setAcl.
There are two kinds of representations for ACL:

if ACL represents the entire access control list for a file or directory, then AccessControlList is used. For example, in UnderFileSystem.getAcl(String path), since the returned ACL is the complete access control list for path, its return type is AccessControlList.
if ACL represents part of a complete access control list for a file or directory, then List<AclEntry> is used. For example, in CreateFileOptions, the mAcl may be an extended ACL which only contains ACL entries for named users and named groups other than file owner and file group, so it is of type List<AclEntry>.

I'll add tests in another PR once the design in this PR is approved.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19358/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19359/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19360/Build result: FAILURE[...truncated 4379 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/pom.xml to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.8.0-SNAPSHOT/alluxio-core-common-1.8.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/target/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-worker/1.8.0-SNAPSHOT/alluxio-core-server-worker-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/target/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-logserver/1.8.0-SNAPSHOT/alluxio-logserver-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.8.0-SNAPSHOT/alluxio-core-client-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.8.0-SNAPSHOT/alluxio-examples-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.8.0-SNAPSHOT/alluxio-core-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/target/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-server-proxy/1.8.0-SNAPSHOT/alluxio-core-server-proxy-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.8.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.8.0-SNAPSHOT/alluxio-shell-1.8.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.8.0-SNAPSHOT/alluxio-tests-1.8.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19361/
Test PASSed.

@gpang @calvinjia PTAL

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19365/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19366/
Test PASSed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19371/Failed Tests: 15org.alluxio:alluxio-core-common: 1alluxio.security.authorization.AccessControlListTest.getModeorg.alluxio:alluxio-core-server-master: 7alluxio.master.file.FileSystemMasterTest.ttlDirectoryDeleteReplayalluxio.master.file.FileSystemMasterTest.setUfsFingerprintReplayalluxio.master.file.FileSystemMasterTest.propagatePersistedalluxio.master.file.FileSystemMasterTest.ttlFileDeleteReplayalluxio.master.file.FileSystemMasterTest.ttlDirectoryFreeReplayalluxio.master.file.FileSystemMasterTest.ttlFileFreeReplayalluxio.master.file.meta.InodeTreeTest.addInodeModeFromJournalWithEmptyOwnershiporg.alluxio:alluxio-tests: 7alluxio.master.file.FileSystemMasterIntegrationTest.concurrentCreateDeletealluxio.master.file.FileSystemMasterIntegrationTest.ufsModeReplayalluxio.master.file.FileSystemMasterIntegrationTest.unavailableUfsRecursiveCreatealluxio.master.file.FileSystemMasterIntegrationTest.syncDirReplayalluxio.master.file.FileSystemMasterIntegrationTest.syncReplayalluxio.security.ClusterInitializationIntegrationTest.recoverClusterFailalluxio.security.ClusterInitializationIntegrationTest.recoverClusterSuccess
Test FAILed.

Merged build finished. Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19372/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19373/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19383/
Test PASSed.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19401/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.zookeeper.ZookeeperFailureIntegrationTest.zkFailure
Test FAILed.


Biasdetection

Added functionality for debias tests, using Dbias package as the rationale and this paper https://arxiv.org/pdf/2208.05777.pdf
Primarily just added bias_classifier.py and debias.md along with any dependencies
Used same functionality as was in place for detoxify to keep consistent classifier functionality - also the paper seems to allow the 0.5 definition of bias in its description of incremental adversarial bias, so this makes sense.
Did not update test_utils.py or any other file since am not sure where this might fit in relation to other conversations of test cases against test datasets (and custom tests) - I just thought testing for bias was a cool addition here

Thanks for this - bias detection in LLMs is definitely important. Thanks for the pull request.
In future, I'll add a contributing.md but I'll clean it up for now and add tests.


Bug in the website: ColorGenerator throws an error and the whole page crashes

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
Description
The whole page crashes if we input an invalid color string in ColorGenerator.
Steps to reproduce

Access to Styling and Layout | Docusaurus and scroll down to the section "Styling your site with Infima".
Edit the input form "Primary Color". For example, add some additional character or delete the last character of the default color.
Then the whole page crashes.

Expected behavior
The page should not crash.
Actual behavior
The page crashes.

Solution
At the line 100 of website/src/components/ColorGenerator/index.tsx, replace Color(`#${baseColor}`) with Color(`#${colorValue}`).
Comments
This bug was introduced by the commit e0127c6, which includes the following changes in website/src/components/ColorGenerator/index.tsx.
// line 72
-  const color = Color('#' + baseColor);
+  const color = Color(`#${baseColor}`);

// line 99--104
            try {
-              Color('#' + colorValue);
+              Color(`#${baseColor}`);
              setBaseColor(colorValue);
            } catch {
              // Don't update for invalid colors.
            }

I guess that the author accidentally replaced colorValue at the line 100 with baseColor, since both of Color('#' + baseColor) and Color('#' + colorValue) appear here.
Self-service

[X] I'd be willing to address this documentation request myself.

Thank you for your very quick response!
I have made a PR #6344.
(I have submitted CLA and believe the status in PR will be updated in a few minutes)


Dark / Light mode confusion, unreadable docs, dark gray on black background.

This is about:

[ ] A typo
[ ] Innaccurate/misleading documentation (e.g. technically incorrect advice)
[ ] Undocumented code
[ ] Outdated documentation
[x] Other

Problem

Main page seems to have a dark mode / light mode issue.  Shown in image is chrome and safari on mac.  It's unreadable. Visible on: https://ribir.org/docs/introduction
I'd just like to read the docs.
Questionaire (Optional)

[ ] I'd like to write this documentation
[ ] I'd like to write this documentation but I'm not sure what's needed
[x] I don't have time to add this right now, but maybe later

Thank you for your issue @rseymour, it really affects reading. we'll fix it soon.

@rseymour We had set the mode fixed to light, hope you can comfortably reading.


Documentation – outdated JavaScript example "Tracking Via Proxy"

Tracking Via Proxy, api_host is not an option on Mixpanel.InitConfig. Seems to be replaced by protocol, host and path

Oops, wrong repo

FWIW api_host is absolutely still a valid config option: https://github.com/mixpanel/mixpanel-js/blob/8ac526e5cb8563d11e2206046ab986c6491ac6d7/src/mixpanel-core.js#L88


readthedocs: Add contribution guidelines

Added guidelines for creating issues, changes, PRs.

Codecov Report

Merging #3925 into master will increase coverage by 21.47%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #3925       +/-   ##
=============================================
+ Coverage     45.91%   67.39%   +21.47%
- Complexity        0        4        +4
=============================================
  Files          1032     1032
  Lines         50881    50881
  Branches       7108     7108
=============================================
+ Hits          23364    34289    +10925
+ Misses        25547    14249    -11298
- Partials       1970     2343      +373

Impacted Files
Coverage Δ
Complexity Δ

...a/manager/realtime/RealtimeSegmentDataManager.java
50% <0%> (-50%)
0% <0%> (ø)

...impl/dictionary/DoubleOnHeapMutableDictionary.java
68.88% <0%> (-6.67%)
0% <0%> (ø)

...e/operator/dociditerators/SortedDocIdIterator.java
55.55% <0%> (-2.78%)
0% <0%> (ø)

...not/broker/broker/helix/ClusterChangeMediator.java
66.66% <0%> (-2.57%)
0% <0%> (ø)

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
72.72% <0%> (-1.82%)
0% <0%> (ø)

.../broker/routing/HelixExternalViewBasedRouting.java
89.06% <0%> (+0.32%)
0% <0%> (ø)
:arrow_down:

...ava/org/apache/pinot/common/request/Selection.java
35.8% <0%> (+0.49%)
0% <0%> (ø)
:arrow_down:

...i/resources/PinotSegmentUploadRestletResource.java
67.82% <0%> (+0.49%)
0% <0%> (ø)
:arrow_down:

...re/realtime/impl/kafka/KafkaConnectionHandler.java
69.87% <0%> (+0.6%)
0% <0%> (ø)
:arrow_down:

...g/apache/pinot/common/request/AggregationInfo.java
32.91% <0%> (+0.62%)
0% <0%> (ø)
:arrow_down:

... and 520 more

Continue to review full report at Codecov.

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


Update intro.md

Updated intro.md

Sgtm, Great job! Can you possibly separate the Bullet points and make a new page for each?
Now we have an Intro page. adding DAOFIN, DAO Treasury, Voting Periods, Proposal Types, Make a new Proposal.

Noted.

Closing it. updated at #185


[FEATURE] Loadbalancer 600s timeout breaks idle 'kubectl exec' sessions

Problem
The Loadbalancer proxies K8s API on TCP 6443 with idle timeout of 600s. It's a problem when I run kubectl exec -ti <id> -- bash, I get disconnected after being idle for 10 minutes, the 'bash' and its children processes become detached.
/etc/nginx/nginx.conf in rancher/k3d-proxy:v4.4.7
server {
    listen        6443;
    proxy_pass    server_nodes_6443;
    proxy_timeout 600;
    proxy_connect_timeout 2s;
}

I need
I need a new flag for k3d cluster create that can overwrite the default 600s and make it 99999s for example.
Otherwise, just increase the default value in your Loadbalancer image.
Works fine with disabled Loadbalancer.

Hi @kochetov-dmitrij , thanks for opening this feature request!
May I ask, what's your use-case for doing this (long-running idle connections)?

Right now I cannot think of any large issue with increasing it to some super high value, but let's see what others say...
A new flag for such a niche action may be overkill though. But I'm thinking of a way to use the new proxy config file implementation in v5 to make the loadbalancer more configurable :+1:
It's a lower priority though :thinking:
In the meantime you may as well do something like docker exec k3d-mycluster-serverlb sh -c "sed -i 's/proxy_timeout 600/proxy_timeout 9999999/' /etc/nginx/nginx.conf && /usr/sbin/nginx -s reload" to increase it :+1:

We use k3d for development and EKS for production. Sometimes I exec with bash into containers for debugging and expect the connection to stay alive until I stop it myself. For example, I exec into a container, start a web app manually and observe the logs in the terminal. Go away for a cup of tea, come back and the connection is closed. The web server keeps running as a detached process, and if I want to restart it, I have to kill <pid> the process instead of just pressing ctrl+c. Neither can I see the logs anymore, unless the process stdout was redirected to the container stdout (>/proc/1/fd/1)
Thanks for the one-liner, I performed it manually last time

Thanks for explaining your use case @kochetov-dmitrij !
I'll leave the exposure of config options for the proxy on the list for v5.1.0 :+1:

The web app keeps running as a detached process, and if I want to restart it, I have to kill  the process instead of just pressing ctrl+c.

For this part you could also use a screen session if that can be made available in your container (image): https://linuxize.com/post/how-to-use-linux-screen/

@iwilltry42 This proxy setting has a highly unfortunate effect of breaking log tailing as well. I'm making a local dev environment using Skaffold and k3d and this env is comprised out of 7 micro-services. The main Skaffold workflow for local dev is skaffold dev at which point Skaffold would build everything in it's configuration, deploy it to configured k8s cluster, then rebuild/live sync changes as necessary all the while tailing logs for each service.  Right now at the 10 minute mark tailing breaks on all the services that did not have any log activity in that time frame. Normally you would be working on 1/2 service(s) at once so this is highly annoying behavior (especially since I did not find a way to make Skaffold reattach the logs again, I have to stop it and start again, which just wastes time).
Maybe a solution would be to increase the default to some higher value, like 1 day in next release, while deferring configurability to some later release?

Implemented configurability in #735 landing in the next pre-release of k3d v5.0.0.
v5.0.0 will be released after some more testing in about 1 week from now (latest, hopefully), after I had the time to revamp the documentation and release docs.


[UPDATE - ONE CLAN ADDED TO THE FAMILY] - Fedeltà Family

name: Fedeltà Family
key: fedeltà
clans:

name: Fedeltà♡
tag: RLPCRY
name: Fedeltà♡ Alpha
tag: 9CGG02RR
name: Fedeltà♡ Beta
tag: P0VL8J92
name: Fedeltà♡ Gamma
tag: PVJUPGQQ
name: Fedeltà♡ Delta
tag: Y2GUJORQ

Thanks!
Il giorno mer 28 ago 2019 alle ore 12:00 SML<EMAIL_ADDRESS>ha
scritto:

Hey there I realized that you had issues accessing the members page. It’s
been fixed and had to do with one of your clan tag is incorrect. should be
Y2GUJ0RQ (number 0 an not letter O)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RoyaleAPI/family-config/issues/84?email_source=notifications&email_token=AL2QTZZ4EUGWR42USEHVKUDQGZED3A5CNFSM4HZMZEN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5KSL5A#issuecomment-525673972,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL2QTZ6CQEE3UBEH2SA6VADQGZED3ANCNFSM4HZMZENQ
.


do you plan to create this for sd 1.5 too?

.

Hi, our code supports sd 1.5. The only change you need to make is the version of the CLIP model used for optimization. You may get more details here.

where do i have to change this please?: Hi, sorry for not including that important detail in the README file, but sure, you can use it for 1.5. SD 1.5 uses the text encoder of CLIP ViT-L/14 from OpenAI, so you can just simply change the argument by: args.clip_model = "ViT-L-14" and args.clip_pretrain = "openai". Also, if you are using the Jupyter notebook, you may also want to change model_id = "runwayml/stable-diffusion-v1-5" when loading the stable diffusion model.

Hi, the easiest way to do this is to change the corresponding values in sample_config.json.

THANKS :)


fend: 0.1.26 -> 0.1.27

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/printfn/fend/releases.
meta.description for fend is: "Arbitrary-precision unit-aware calculator"
meta.homepage for fend is: "https://github.com/printfn/fend"
meta.changelog for fend is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/1kp9p9insbbipqd3n9ky9m5s4qfqkk4j-fend-0.1.27/bin/fend passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 0.1.27 with grep in /nix/store/1kp9p9insbbipqd3n9ky9m5s4qfqkk4j-fend-0.1.27
found 0.1.27 in filename of file in /nix/store/1kp9p9insbbipqd3n9ky9m5s4qfqkk4j-fend-0.1.27

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
fend

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1kp9p9insbbipqd3n9ky9m5s4qfqkk4j-fend-0.1.27
ls -la /nix/store/1kp9p9insbbipqd3n9ky9m5s4qfqkk4j-fend-0.1.27/bin

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

1 package built:

fend

Maintainer pings
cc @djanatyn for testing.

Improved command-line argument parsing, including support for multiple arguments
The most recent calculation result is now stored in a special variable _ (or ans):

Basic functionality looks good to me


Is llama2 70b supported? Do you know minimal configuration?

FasterTransformer development has transitioned to TensorRT-LLM.
LLaMa 2 70B is supported in TensorRT-LLM. Please take a try.


Encourage files in hosted workflows to be renamed

Is your feature request related to a problem? Please describe.
Hosted Workflows should somehow encourage people to change the name of the files.
Describe the solution you'd like
Maybe make the default name "untitled.cwl", etc
Describe alternatives you've considered
Make the default in the frontend ".cwl", which won't allow the user to save version at all until it's changed
Additional context
https://github.com/dockstore/dockstore/pull/3417#discussion_r413225789
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Sprint: Sprint 34 Isopods
┆Issue Number: DOCK-1328

I don't think you can even change the name of the main descriptor for hosted workflows. Or if you can, I don't understand how.

Barring major changes, it would be changing it before they manage they can save.

After some discussion in storytime, the priority of this ticket will be bumped out of 1.9.0.
Discussion summary:

The backend and UI depend on the primary descriptor being named Dockstore.x
It is a good feature to have. Users should be able to name the primary descriptor what they want, like they can for remote entries.
You can create a hosted tool version with both a wdl and cwl primary descriptor file
This ticket might be made easier if we don't support a single tool version to have both a cwl and wdl.
Currently we do have hosted tools by users other than our team members, but should check whether they make use of a version having both descriptor types before dropping support.


Add fulfillment cancelled webhook

I want to merge this change because it adds a new webhook - FULFILLMENT_CANCELLED.
⚠ This is a PR to fearture branch.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3845 Add webhook FULIFLLMENT_CANCELLED


Warn or require an expression after defined Access Policy is incompatible with existing data

If an Access Policy is introduced (say on Insert and Update Write) that a User of type A can only define a and of type B can only define b properties, however a data exists in the DB that a User of type A has their field b defined, then trying to update that user to either a or b will result in Access Policy violation, which is often hard to debug, since one has to set them at the same time to figure out that initial data was incompatible. A warning in the migration may appear if something similar to fill_expr is unfeasible

I don't think this makes sense to do in general, since in general, access policies can (and probably usually will) refer to global variables.
I think that you're using them as a generalized constraint, and in that case this sort of warning could be useful, but I think that's probably too niche of a use case here.

I see. This is my first time using Access Policies so maybe I will come learn that


Update 20532C_LAB_AK_02.md

Removed the trailing '.' from Task 2, Step 7 e from the password field. Students commonly copy-paste the password, as the special characters ($) are a hassle for many international keyboards. This resulted in multiple students not being able to log in to a VM, as the password was incorrect when typed manually (but copy-pasted with the trailing dot succeeded).

This is a great idea!


Shebangs being overwritten

When compiling a file that includes a shebang, then running ef-tspm, the shebang is overwritten.
#!/usr/bin/env node

becomes
///usr/bin/env node

My workaround is just to follow up with a script to overwrite ///user/bin/env node with #!/usr/bin/env node in my bin folder. There's probably a more clever way, but this works at least:
import { resolve } from 'path'

import { sync as globSync } from 'glob'

export const fixShebangsInDist = (): void => {
  const binaries = globSync('dist/bin/**/*.js')
  binaries.forEach(binFilePath => {
    const binFileContents = readFileSync(resolve(binFilePath)).toString()
    writeFileSync(
      resolve(binFilePath),
      binFileContents.replace('///usr/bin/env node', '#!/usr/bin/env node')
    )
  })
}


App fails to install due to execution failure

Thank you for forking the original project.
Your Environment

Plugin version: 0.6.6
Platform: Android
OS version: macOS 11.4
Device manufacturer and model: Apple

Running in Simulator: Pixel 4 Api 29
React Native version: 0.64.2
Plugin configuration options:
Link to your project: Private repo

Context
I can build and install the app to a simulator using npm run android, but when I configure this library I get
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugAarMetadata'.
> Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
   > Could not find ::.
     Required by:
         project :app

Expected Behavior
App builds, installs, and runs successfully.
Actual Behavior
Errors before installing.
Steps to Reproduce

Create a brand new react native project
Add this library to it
Try to run npm run android

Context
Trying to replace the project that this forked.
Debug logs

I guess you have to input which command you use to install this library.

same problem here


Fix error RN 0.70: No member named 'throwJSError'

fixes https://github.com/Shopify/react-native-skia/issues/885

signed

@chrfalc I already signed CLA


useControl strange behaviour

Describe the bug and the expected behavior
I am using useControl in a DateInput component and mapping the value type from Date to string as "DD/MM/YYYY" to match the string type of formData. However, when I submit, it always gets converted to a ISO String. But useInputControl, on the other hand, does not have this issue in this case.
Please check the StackBlitz example:
https://stackblitz.com/edit/stackblitz-starters-hymcsl?file=app%2Fform.tsx
I console.log the value of inputControl.value and control.value both in onSubmit and in the server action.
Conform version
v1.0.2
Steps to Reproduce the Bug or Issue

Open the StackBlitz example
Select dates and Submit
Check the terminal logging, the control value is a ISO String while the inputControl value is not.

What browsers are you seeing the problem on?
Chrome
Screenshots or Videos

Additional context
In fact, the same thing happened in useInputControl, but I can't seem to reproduce it.

This is because you are leaving it to mantine to decide how the input is rendered when you provide a name prop to it. If you inspect the element, you will find this html:
<input type="hidden" name="control" value="2026-02-12T23:00:00.000Z">

If you wanna have full control over the value on the input, it would be better to create your own hidden input like this:
<input
  type="hidden
  ref={control.register}
  name={fields.control.name}
  defaultValue={control.value}
/>
<DatePickerInput
  label="useControl"
  placeholder="Pick date"
  valueFormat="DD/MM/YYYY"
  name={fields.control.name}
  value={control.value ? stringToDate(control.value) : null}
  onChange={(value) => control.change(value ? dateToString(value) : '')}
  error={fields.control.errors}
/>


Datasets challenge import error

Hi,
I encountered an error when trying to import the datasets_challenge.py file (see screenshot). Could anyone else confirm if they have experienced a similar problem?

Thanks!

Hi Gorka,
you're right, I'm aware python 10 is now needed, but I used the code on a computer with python 3.9 by mistake. Sorry!
Thanks


Balls stuck to my left leg

📝 Describe the Bug
A clear and concise description of what the bug is.
🚶♀ Steps to Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

🎯 Expected Behavior
A clear and concise description of what you expected to happen.
🖼 Screenshots
If applicable, add screenshots to help explain your problem.
💻 Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

📚 Additional Context
Add any other context about the problem here.

I'm glad we resolved this issue


Expose PopupProperties in PopupMenu

PopupMenu internally uses Popup, but uses a hardcoded PopupPropterties object. This exposes it as a param and uses the old hard coded values as the default parameter value.

@francisconoriega you need to run apiDump :)

@francisconoriega you need to run apiDump :)

I thought it was an automatic CI step :P.
Would it make sense to run the klint, api-dump, and other tasks as as pre-commit hooks?

@francisconoriega it's not automatic because we want to ensure people are aware they're changing the APIs and do it knowingly. When we reach 1.0 we won't allow breaking changes anymore, unless it's a major change.

Would it make sense to run the klint, api-dump, and other tasks as as pre-commit hooks?
I have a pre-push hook locally that runs ktlintCheck, we should eventually add it as a built-in. I looked into ways to do that "properly" in the past but never settled on one solution. Do you know any Gradle plugin to manage and install hooks?

I don't love the idea of running it as pre-commit as it's very slow. Plus, since we squash, we only care about the formatting when we push.


PKG -- [fcl] Get custom window size in popup src

This change allows developers to change the size of the popup window by using the query string width and height in the challenge handshake URL. For example, you can set url query string like http://localhost/flow/auth?width=500&height=700 to adjust popup window size.
I think this will make wallet provider has more design flexibility.

Thank you for this PR! Our sincere apologies for the long delay in response. While we appreciate you contribution efforts, we unfortunately cannot accept this PR because we must not create any undue assumptions that FCL wallets can control browser size on the FCL client. We want to avoid wallets making assumptions of what the client browser size is or will be, since it is out of the wallets control.


Cards custom image issue

When using a custom image i don't get the same hover effect. Tried a couple of things, including the changing of z index, but with no luck. When i change back to the original, everything works fine.

@Doppelganger113 are your images slightly semi-transparent? The ones in the example are and that's why the color change is visible on hover.

@Magnus-G
Then i might have tried with faulty ones from google images. Anyway, if that is the case, one would always have to change their pictures to be semi-transparent. Wouldn't it be easier to put a semi transparent layer in front of the picture and toggle him ? Not the other way around.

@Doppelganger113 you are probably right! If you get it working, feel free to open a PR here and we might turn it into that instead!

Merged


Others: use defined constant to detect conflicted entries in rebase dashboard

Use the constant MERGE_CONFLICT_PORCELAIN_STATUSES to detect conflicted entries in rebase dashboard

LGTM


Feature Request: Availability of Event Callback(Delegates, Signal/Slots)

Proposal for adding signal/slots to Biogears for event handling.
Goals

Ease the amount of code needed to detect common physiology events.
Support functional programming paradigms for downstream integrator.
Document best practices for reacting to physiology changes

Who benefits

C++ Integrators
All GUI developers using BioGears
Real-time BioGears embedded systems.

Implementation

Work with modeling team to identify 2-5 events of interest
Determine on Option 1 or Option 2 based API signatures
Implement hooks and provide community howto’s on new features
Implement use in BioGears Visualizer as proof of concept.

Completed


Add LogWriter for iOS OSLogStore

https://developer.apple.com/documentation/oslog/oslogstore

It looks like that's for reading, not writing: https://steipete.com/posts/logging-in-swift/. Unless I'm reading that wrong.

Ah that's right, it's for reading only. The problem that I ran into is that platformLogWriter() didn't seem to write iOS console logs in a way where they would end up in the OSLogStore. Haven't tested with Kermit 2.0 yet though!

This bug is still present as of Kermit v2.0.2. And it's a critical bug.
If the OSLogWriter is functioning properly, the log entries will be visible/available in the following locations:

Xcode's debug console
Console.app
OSLogStore API

The first 2 are working fine currently, but the last isn't.
The OSLogStore is an API for reading log entries that have been sent to OSLog system. For example, your application might want to export a log file, and send it to the development team for debugging. To do this, you MUST call the OSLogStore API within your code. If you do this now, with the current codebase, all log entries sent from Kermit result in a log message of <compose failure [UUID]>.
Unfortunately this renders Kermit unsuitable for use with many iOS applications, because it means:

any log entries sent thru Kermit are LOST
any kotlin-multiplatform libary that uses Kermit won't have readable log entries

To be clear: Kermit works fine if you're debugging on your own device when plugged into Xcode. But if you need to debug problems from actual users of your app, then you MUST use OSLogStore, and Kermit has unfortunately broken all log entries within OSLogStore.

Totally agree with @robbiehanson .
My current solution is to create a LogWriter in the app and pass it as a parameter to the kmm library.
`final class MyLogger: LogWriter {
private static let subsystem = Bundle.main.bundleIdentifier ?? "app_system"
private let myLogger = Logger(subsystem: subsystem, category: "MyLogger")
override func log(severity: Severity, message: String, tag: String, throwable: KotlinThrowable?) {
    #if APP_ENV_PROD
    myLogger("\(Date()) [MYLOGGER] [\(severity)] \(message, privacy: .private)")
    #else
    myLogger.debug("\(Date()) [MYLOGGER] [\(severity)] \(message, privacy: .public)")
    #endif
}

}`

Any updates on this? It is frustrating to not be able to see logs on a device without Xcode attached.

The relative PR has been merged in. It should be available in the next release.


test: restore hybrid vaadin-router test module

The hybrid test module with vaadin-router needs the following workaround to prevent issues with flow-react dependency:

exclude flow-react (this is not enough because the ReactAdapterComponent class is indexed in vaadin-core-jandex and undertow tries to load it at runtime because DevModeStartupListener declares Component class in @HandlesTypes
exclude vaadin-core-jandex (not enough because then Vaadin classes are not found by undertow extension)
Re-index Vaadin deps in application.properties (quarkus.index-dependency."all-vaadin".group-id=com.vaadin)

With the three changes above, the module can run and tests pass.


Feature Request: Mirror DOM hierarchy for layers hierarchy / groups

Ah yes, I have experimented with this. I have found it makes a looooooooot of groups, but I can add some kind of "advanced" options that can have this in it in an upcoming release

This is a feature that you think you want, but you really don't. I'm talking from experience here (I built a html to css script back in the day). Like @steve8708 mentioned just the amount of noise it adds is over 9000. Also, as a design tool you want the least number of things colliding.
The objective of html-to-figma is to just do some scaffolding not 1:1.

Another thing to consider - is that Figma has a very different model. Figma wants you to use frames more than groups. So that's something to keep in mind.

Good points @MarioRicalde  This gave me an idea for my own recent plugin actually. It's called Super Select, I think I'm going to add Select Intersecting, and Select Contained as commands then they can be grouped manually.

@alexpineda Oh, so that's where I know your name from!
I sent you an email not long ago about your plugin hoping you'd rename your Super Select menu items so they are easier to use with CMD+/. Never got an answer what gives?! 😠😠😠😠
Just kidding (about me being angry ;) did send the email, reply someday please!) Great job with the plugin!

 I did send a response. You must have never received it. It was a good suggestion and you should checkout the update :P

Haha! I've already replied. Kudos!

Oo select interesting and select contained for grouping is a great idea for Super Select! I've had those use cases, particularly for grouping elements containing/overlaying others many times and that'll work great with html-to-figma

We really need to group those :)
Header
Main
Footer
etc :)


Data analysts should be able to use Text.to_case to change the case of Text values

Pull Request Description

Important Notes

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the Scala, Java, and Rust style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[ ] If GUI codebase was changed: Enso GUI was tested when built using BOTH ./run dist and ./run watch.

One minor change for a ToDo and as discussed lets add the Title defintion from UCharacter.
Otherwise LGTM


Error about annotation path when fine-tuning.

Dear author, thanks for your great work. I successfully train the pertained model, but when I fine-tuning, it reports such error:
FileNotFoundError: [Errno 2] No such file or directory: 'datasets/vocsplit/box_10shot_aeroplane_train.txt'.
I have generate the shots by running python datasets/prepare_voc_few_shot.py.

prepare_voc_few_shot.pyb will generate files of 10 seeds, these files is put in 'datasets/vocsplit/seed*' (e.g.  'datasets/vocsplit/seed1' ). You need to choose the file set you want and put them in a correct address.
You can also get them from http://dl.yf.io/fs-det/datasets/cocosplit/

@Retiina
prepare_voc_few_shot.py by default generate 10 seeds (suffix 0-9). The default seed used in literature is seed0. The author of FsDet implement it such that, when a seed is not specified, it will look for box_[k]shot_*.txt in datasets/vocsplit instead of datasets/vocsplit/seed0.
We follow the same design. So you can try something like cp datasets/vocsplit/seed0/* datasets/vocsplit to move the desired shots data to datasets/vocsplit directory
And for COCO, you may also need coco/datasplit which can be downloaded from http://dl.yf.io/fs-det/datasets/

@Chauncy-Cai , but when I run prepare_voc_few_shot.py, it generates suffix 1-10

seed0 is under this path: http://dl.yf.io/fs-det/datasets/vocsplit/
(all the file box_[k]shot_*.txt under this path is few-shot annotation of seed0)

Thank you!

@Chauncy-Cai , thanks for your help I successfully run the experiment of base training and 1 shot fine-tuning. But surprising, I got 43.456 nAP50 for split1 shot0 using 1shot_baseline.py, is it normal, here is the result, which is higher than it reported in the paper (39.8).
[03/21 23:11:04] fsdet.evaluation.pascal_voc_evaluation INFO: Evaluate overall bbox:

AP
AP50
AP75
bAP
bAP50
bAP75
nAP
nAP50
nAP75

41.377
69.769
43.106
47.264
78.540
49.770
23.714
43.456
23.113

39.8 is reported in TFA, however, our so-called "baseline" is not the same as TFA, please refer to our paper for details. BTW, your reproduced result is reasonable.


Add Socials @nishant0708

https://recodehive.github.io/awesome-github-profiles/index.html

I can work on this if @nishant0708 has no issues with that