How to do USB stdio output in vga-test3 dma example?

Hi, I wonder is it possible to somehow use usb printf (or anything more heavy than led-blinking to that matter?) in vga-test3 dma example?
I tried using usb printf in vga-test3 simple vgaimage.c example, and all works fine, but with vgaimage-dma.c I'm unable to make it work. I tried to use second core in both ways - to print tick string on it with 1 second delay, and vice versa - to print tick on main core, and moving vga routine to second core:
#include <stdio.h>
#include "pico/stdlib.h"
#include "pico/multicore.h"
#include "pico.h"
#include "pico/scanvideo.h"
#include "pico/scanvideo/composable_scanline.h"
#include "hardware/structs/dma.h"
#include "hardware/structs/ssi.h"

#include "image-vga.h"
//#define VGA_MODE vga_mode_320x240_60
#define VGA_MODE vga_mode_640x480_60
#define MIN_RUN 3

// flash_bulk_read() is from pco-playground/scanvideo/flashstream

// Use direct SSI DMA for maximum transfer speed (but cannot execute from
// flash at the same time)
void __no_inline_not_in_flash_func(flash_bulk_read)(uint32_t *rxbuf, uint32_t flash_offs, size_t len, uint dma_chan) {
    ssi_hw->ssienr = 0;
    ssi_hw->ctrlr1 = len - 1; // NDF, number of data frames (32b each)
    ssi_hw->dmacr = SSI_DMACR_TDMAE_BITS | SSI_DMACR_RDMAE_BITS;
    ssi_hw->ssienr = 1;

    dma_hw->ch[dma_chan].read_addr = (uint32_t) &ssi_hw->dr0;
    dma_hw->ch[dma_chan].write_addr = (uint32_t) rxbuf;
    dma_hw->ch[dma_chan].transfer_count = len;
    // Must enable DMA byteswap because non-XIP 32-bit flash transfers are
    // big-endian on SSI (we added a hardware tweak to make XIP sensible)
    dma_hw->ch[dma_chan].ctrl_trig =
            DMA_CH0_CTRL_TRIG_BSWAP_BITS |
            DREQ_XIP_SSIRX << DMA_CH0_CTRL_TRIG_TREQ_SEL_LSB |
            dma_chan << DMA_CH0_CTRL_TRIG_CHAIN_TO_LSB |
            DMA_CH0_CTRL_TRIG_INCR_WRITE_BITS |
            DMA_CH0_CTRL_TRIG_DATA_SIZE_VALUE_SIZE_WORD << DMA_CH0_CTRL_TRIG_DATA_SIZE_LSB |
            DMA_CH0_CTRL_TRIG_EN_BITS;

    // Now DMA is waiting, kick off the SSI transfer (mode continuation bits in LSBs)
    ssi_hw->dr0 = (flash_offs << 8) | 0xa0;

    while (dma_hw->ch[dma_chan].ctrl_trig & DMA_CH0_CTRL_TRIG_BUSY_BITS)
        tight_loop_contents();

    ssi_hw->ssienr = 0;
    ssi_hw->ctrlr1 = 0;
    ssi_hw->dmacr = 0;
    ssi_hw->ssienr = 1;
}

// RAW_RUN |color 1| n-3 |..|color n| 0 | EOL
int num_token = 2 + image_width + 1 + 1;

// render pixels
int32_t __time_critical_func(single_scanline)(uint32_t *buf, size_t buf_length, const uint16_t *data) {
    assert(buf_length >= num_words);

    uint16_t *p16 = (uint16_t *) buf;

    flash_bulk_read(&buf[1], (uint32_t) data, image_width / 2, 11);
    p16[0] = COMPOSABLE_RAW_RUN;
    p16[1] = p16[2];
    //    p16[2] = (num_token - 3) - MIN_RUN;
    p16[2] = (image_width + 1) - MIN_RUN;
    buf[num_token / 2 - 1] = 0 | COMPOSABLE_EOL_ALIGN << 16;

    return (num_token + 1) / 2;
}

static void inline render_scanline(struct scanvideo_scanline_buffer *dest) {
    uint32_t *buf = dest->data;
    size_t buf_length = dest->data_max;
    int line_num = scanvideo_scanline_number(dest->scanline_id);

    dest->data_used = single_scanline(buf, buf_length, &image[line_num * image_width]);

    dest->status = SCANLINE_OK;
}

// main loop
void __time_critical_func(render_loop)() {
    static uint32_t last_frame_num = 0;

    while (true) {
        struct scanvideo_scanline_buffer *scanline_buffer = scanvideo_begin_scanline_generation(true);

        render_scanline(scanline_buffer);

        scanvideo_end_scanline_generation(scanline_buffer);
    }
}

void core1_entry() {
  while(true) {
    printf("tick\n");
    sleep_ms(1000);
  }
}

void render_core1() {
  scanvideo_setup(&VGA_MODE);
  scanvideo_timing_enable(true);
  render_loop();
}

Second core vga routine main:
int main(void) {
    set_sys_clock_khz(200000, true);
    stdio_init_all();

    multicore_launch_core1(render_core1);

    while(true) {
      printf("tick\n");
      sleep_ms(1000);
    }

    return 0;
}

Second core ticking main:
int main(void) {
    set_sys_clock_khz(200000, true);
    stdio_init_all();

    multicore_launch_core1(core1_entry);

    scanvideo_setup(&VGA_MODE);
    scanvideo_timing_enable(true);
    render_loop();

    return 0;
}

If I move vga routine to second core - it draws without a problem but port is not opened and just no communication appears. If I move ticking printing to second core - it didn't draw at all, and no port is opened - looks just halt.
I debugged a bit, and it looks like everything works fine up until __no_inline_not_in_flash_func(flash_bulk_read) function gets executed. I tried to remove everything but first line with ssi_hw->ssienr = 0; - and it immidiately breaks. I'm a newbie into pi and not quite sure what this code does.
I also tried to do the same with flash-stream dma example in similar pico-playground examle, and created issue in its repository - https://github.com/raspberrypi/pico-playground/issues/36
So my question is - do you tried to output usb printout and what am I doing wrong here?
I want to draw 640x320 display while simultaneously checking capacitive touch panel every ~5ms via i2c, do you know if its possible at all and if all resources are taken by pico just to draw screen so it will halt? Thx

I also not sure how to use even uart though - since vga uses all pins with uart buses using gpio0-gpio17..

I managed to use UART with 20 and 21 pins, but turns out that even smth like led blinking loop simply hangs, so I think it just unusable with rendering to do anything else.

@sandric Have you tried SoftwareSerial?

@Andy2No no, but as I said - 20 and 21 gpio works fine for UART, but when I start vga routine it doesn't work. So the thing is not in usb or uart - I can not work simple loop of blinking led on second core, it hangs, you can try this code commenting vga code in and out to see:
void core1_blink() {
  while(true) {
    gpio_put(LED_PIN, 1);
    sleep_ms(1000);
    gpio_put(LED_PIN, 0);
    sleep_ms(1000);
  }

int main() {
  ...
  multicore_launch_core1(core1_entry);
  ...
}
}

@sandric vgaimage-dma.c employs DMA (direct memory access), a dedicated hardware mechanism for transferring data between devices, such as RAM and flash memory. The image data is stored in the flash memory and needs to be transferred to RAM. The DMA hardware performs this task because using the CPU would require two data transfer sessions: reading to the CPU and writing to RAM.
The issue is that the CPU requires access to flash memory where the code and data are located, but the flash memory is frequently occupied because the DMA hardware locks it. This is why printf() in your code does not work. printf() and other library code/data are located in the flash memory, and the CPU cannot access them.
To solve this problem, you should place your entire code in RAM by specifying the following option:
cmake -DPICO_NO_FLASH=1 -B build

However, this does not work with my code because the image data is part of the code, and the linker will try to place it in RAM, but there is insufficient space. A possible solution is to write the image data directly to the flash memory and access it from the code using a pointer that points to the image data's address in the flash memory. Please refer to
https://github.com/raspberrypi/pico-playground/tree/master/scanvideo/flash_stream
as this code does exactly that.

@boochow Thanks a lot for great explanation, I also came to this conclusion by trying to read pico datasheet, but its above my expertise level unfortunately. I tried the same but setting this variable in cmake itself:
set(PICO_NO_FLASH 1)

but didn't end up resolving the issue. But thanks for a clue about flash_stream - I know how to build it too, bud didn't though about picotool as a way to get away with it. I will try it and write back.

@boochow ok, I tried, but no success still. I tried both starting blinking on secondary core, and starting vga routine on secondary core. Both times - smth different is happening when PICO_NO_FLASH=1 is used: display got splitted on half with "blue/black" colors on sides, and blinking/printf CDC routine actually working, but not vga though.
I also found out that for some reason flashed code works only before restarting pico, and fallback to previously loaded without PICO_NO_FLASH=1, not sure but mb that mean that you need to flash it each time you power device if you want to use RAM loaded code?
I was playing also a bit with dma bulk loading, I found out experimentally that if I reduce buffer to read from ssi/dma to 32 bytes - it actually works, just renders like 1/20 of screen width.
I wonder if it is somehow means possible to reduce bulk loading times by reading picture only 1 time, instead of each time reading from RAM 60 times a second? The thing is that I need to display only 1 picture, not video etc. I mean mb there is typing for 1 hz in vga modes - so that rest of the second I could do anything else apart rendering? Or vga requires you to give 60hz all the times?

@sandric thank you for sharing this; it seems something still conflicts with the DMA... but I'm afraid I can't handle this issue with my knowledge. How about posting your question at https://forums.raspberrypi.com/viewforum.php?f=145 ?  kilograham may answer it.

@boochow sure, didn't have big luck before with rp forum, but I'll try there, thanks for your help.


AppointmentParticipantResponse: add UserId

feature/5836

@jay-lee00 Why are these values needed?

Mobile app needs to display details of the doctor's info.  Additional API calls require UserID as parameter.


Operator pod: don't go ready until the webhook is patched

When we deploy through manifests there's no way to understand when we can deploy the metallb cr, because the corresponding validating webhook might not have the ca injected.
Here I am proposing to mirror the same logic we have in metallb, and enable the readiness endpoint only when the cert-rotator signalled us that it has patched the webhooks.

@fedepaol I think we have this fixed under https://github.com/metallb/metallb-operator/commit/588bcd438c06718bc7a5f8c849b0b40cdeba38b0.

Yep. closing this


Add option for dynamic custom prompt

The custom prompt piece only allowed a static text
via BULLETTRAIN_CUSTOM_MSG. Now another variable is added
called BULLETTRAIN_CUSTOM_FN that should store the name of a custom
function that will be invoked each time the prompt is showed.
E.g. if you define a function my_custom_prompt { ... }
then set BULLETTRAIN_CUSTOM_FN=my_custom_prompt

Let me know if you like this feature. I needed to add something dynamic into my prompt so I thought I'll just drop a quick PR :wink: The first commit had a typo and after fixing would show an empty segment if the functions returned nothing, so I fixed that in the next commit.

Please look at #125. I think 125 is superset of this functionality - it allows you to reorder segments and add you own.

That is true.
This PR can be interpreted as a subset of #125
Maybe we should add a section to Tips on how to add a custom dynamic segment?

With #125 I had added Tips section demonstrating how to add custom segment. It is using static text. We can modify it but I guess it is quite straightforward how to use dynamic text.

@dawikur sure, if that gets merged it allows more freedom :) The code seems nice and easy so I hope you guys merge it soon :+1:

@xificurC It is merged :smile: Can we close this PR?

Absolutely ;)


Make jcoin port configurable and add missing item to example config

Learn proper Git practices. Know the format for commit titles and messages in the repo before comitting yourself.


How to mark structure field / variable as volatile?

I have a function which, simplified, is as follows:
void func(emmc_t *emmc) {
  // some stuff
  emmc->reg->INT_STATUS = 1;
  while (emmc->reg->INT_STATUS) {}
  // more stuff
}

However, Ghidra doesn't know that INT_STATUS can change externally and instead decompiles it to an infinite loop (and kills everything after it). Is there a way to mark a structure field as volatile so that the reads don't get optimized away?

I think you either have to mark the entire region as volatile in the memory map (Windows > Memory Map), or untick "Eliminate unreachable code" (Right click in the decompiler text > Properties > Decompiler > Analysis). I think you can also change whatever type INT_STATUS is, to volatile.

Marking all memory regions as volatile doesn't change the output at all; unticking "Eliminate unreachable code" does make it a little better, but it still generates loops that look like do {} while (true). If I try to add a "volatile" keyword to a type, like volatile ushort, it just says it's "unrecognized data type".

I can confirm that "Eliminate unreachable code" had no effect.
Similarly, marking the entire region as volatile hamstrings the decompiler.  Ideally, volatile could be included as part of the type.

This problem is common in embedded applications (ie non-os) like on the AVR and ARM Cortex M architecture. An example which occurs frequently is something like:
ISR(TIMER_OVERFLOW) {
  some_register |= 0x01;
}

int main() {
  ...
  while ((some_register & 0x01) == 0) { // wait for the timer to overflow
    do_some_background_work();
  }
  ...
}

Found a similar condition in PSX code (MIPS processor):
                             LAB_80056408                                    XREF[1]:     80056420 (j)
        80056408                 lw         v0,0x0 (a1)=>counter1                            =
        8005640c                 nop
        80056410                 sw         v0,local_30 (sp)
        80056414                 lw         v1,local_30 (sp)
        80056418                 lw         v0,0x0 (a1)=>counter1                            =
        8005641c                 nop
        80056420                 bne        v1,v0,LAB_80056408

a1 = counter1 = 1f801110 (Horizontal retrace counter)
The decompiled code is:
  do {
  } while (*ptrCounterHorRetrace != *ptrCounterHorRetrace);

Where ptrCounterHorRetrace = counter1
I believe the code should be more along the lines of:
  int counter1;
  int counter2;
  do {
    counter1 = *ptrCounterHorRetrace;
    counter2 = *ptrCounterHorRetrace;
  } while (counter1 != counter2);

I've got the whole block ("hardware_registers" 1f801000-1f802fff) set to volatile.


Block Route Library Refactored for v2

v2 refactor of block.ts route library. Includes passing integration and unit tests. Execute tests with npm run test-v2
This PR should be merge after PR #109 as it builds on top of that work. Here is a diff of the two branches.

Pull Request Test Coverage Report for Build 669

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

Totals

Change from base Build 663:
0.0%

Covered Lines:
1352

Relevant Lines:
2110

💛  - Coveralls

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


[release/1.7] Prepare v1.7.6

the purpose of v1.7.x is to maintain a mapping version to latest containerd 1.7.x ?

the purpose of nerdctl v1.7.x is to maintain a mapping version to latest containerd 1.7.x ?

No, coincidence


One-handed Start gesture doesn't list that users need to be on current build.

This is a new feature that only works after updating. I believe there should be something that indicates to users that if this interaction isn't working that they should try updating their device.

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

ID: aeb17525-36a9-d993-b774-24fdd800fa0e
Version Independent ID: c3ec4304-6217-5341-47cf-774070496c73
Content: Getting around HoloLens 2
Content Source: devices/hololens/hololens2-basic-usage.md
Product: hololens
Technology: windows
GitHub Login: @scooley
Microsoft Alias: scooley

@e0i - can this be resolved fixed?
@DaniHalfin - what can I do to become a maintainer so we can use this for issue tracking too?

@evmill
This issue has been addressed with the following commit and content has been updated according to your feedback.
[c3d09ab ] CAT Auto Pulish for Windows Release Messages -<PHONE_NUMBER>1333 (#1564)
We are thus closing this issue since it seems to be resolved.
Thank you for taking the time to provide feedback.

@scooley
"You must update to the November 2019 update (build 18363) or later."
would be better identified as
"You must update to the November 2019 update (build 10.0.18363.1039) or later."
because the current ARC build could also be identified using the current statement and would not have this feature.


How to show the new button above the empty grid

Description:
Hi.
When my grid is empty, the new button will not appear at the top of the grid screen.

Laravel Version: 5.5.36
PHP Version: 7.1.9
Laravel-admin: 1.5.6

plz refs to :
http://laravel-admin.org/docs/#/en/model-grid-custom-tools

tanks a lot yang5664

tanks a lot yang5664.
but new button is default button in grid page. By default there is a new button on top gird page but if grid is empty, not display new button on top grid.
please help me!


w2sidebar scrollIntoView is not always scrolling far enough

In a sidebar with many items, scrollIntoView doesn't always scroll far enough.
see http://jsfiddle.net/mtnsutso/

click on "goto 0", selects node0
click on "goto 10", selects and scrolls to node10
click on "goto 99", selects node99 but only scrolls about 10 items
click on "goto 0", selects node0 but only scrolls back to around node10

This was using w2ui 1.4.2, with either Chrome or nw.js 0.12.0
In scrollIntoView: function (id) is
if (offset + item.height() > body.height()) {
    body.animate({ 'scrollTop': body.scrollTop() + body.height() / 1.3 }, 250, 'linear');
}
if (offset <= 0) {
    body.animate({ 'scrollTop': body.scrollTop() - body.height() / 1.3 }, 250, 'linear');
}

I think this should be
if (offset + item.height() > body.height() || offset <= 0) {
    body.animate({ 'scrollTop': body.scrollTop() + offset - body.height() / 2 }, 250, 'linear');
}

as in either case, the offset from the current scrollTop is already calculated.
I'm unsure what body.height() / 1.3 was intended to do, I'm thinking it was to not just move the item to the top of the tree when scrolling forward; by using + offset - body.height() / 2 it moves the item approximately to the centre.
Thanks for a terrific UI library, it fit well into our nw.js project.

You are right, there is a bug. I have fixed it with your suggesting, just added item height into equation. The reason for 1.3 was is when you use keyboard to go down, it would scroll 70% of the height. Same when going up. A bit more then half.


writing sequences

Hello, I need to develop a function now. Input Chinese characters to get the stroke order coordinates of this Chinese character. For example, input "十". It is composed of horizontal (一) and vertical (｜). The start coordinate of horizontal (一) is (0, 5) end coordinate (10, 5), and the start coordinate of vertical (｜) (5, 0) end coordinate (5, 10),  How to use Hanzi writer to get the coordinates of these writing sequences？

For your case, you probably don't need to use the HanziWriter library - you can just use the JSON data for each character. All the data is at: https://github.com/chanind/hanzi-writer-data. For instance, the data for 中 is at: https://github.com/chanind/hanzi-writer-data/blob/master/data/中.json. You probably just need the medians array from the data, which draws a line along the center of each stroke. So, the first point in medians for each stroke is what you'd be looking for. the only tricky thing is that the points coordinates vary from x between 0 and 1024, and y between 900 and -124, so you'll need to do some math to transform the coordinates into whatever grid you're using.

Thank you


[Chore #159647010] fix code structure and documentation

What does this PR do?
fixes code structure and documentation
Tasks

Fixes eslint errors
Updates README.md
Updates My404Component alert to a page content
Removes node modules from remote repository
Adds .gitignore file

Relevant pivotal tracker stories
#159647010

Pull Request Test Coverage Report for Build 87

4 of 19 (21.05%)  changed or added relevant lines in 7 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+1.1%) to 32.01%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Components/Login.js
0
2
0.0%

src/Components/Businesses.js
0
2
0.0%

src/index.js
0
4
0.0%

src/Components/Business.js
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/index.js
1
0.0%

src/Components/Business.js
11
2.0%

Totals

Change from base Build 70:
1.1%

Covered Lines:
110

Relevant Lines:
305

💛  - Coveralls

@kzyGit Good work on the 404 page, it would be better if you added a snapshot of the page on the PR

Thank you @sam-karis . I have added the snapshot


Add additional component types

In addition to the existing component types, the proposal is to add the following as new types:

image (a vector or raster image)
font
container

One thing of possible confusion is the word 'container'. By container, this is meant to be equal to a Docker image or running container regardless of deployment (could be running in a standalone container or in a pod).

Hi Steve, great to see the "container" type coming. One question I have is how you are thinking about linking containers to libraries and such? I.e., let's say I have a library and then I containerize it -- do you see a way to represent that it's the same library but now containerized?

CycloneDX has two different ways to represent relationships, dependency graphs, and nested components. You can also describe the final deliverable itself.
A nested component represents a system -> subsystem assembly. In other words, a component contains another component.
In the case of a container that is component of some bigger deliverable, you would basically have something like:
<component type="container">
    <name>my-custom-container</name>
    <version>1.0.0</version>
    <purl>pkg:docker/my-custom-container@sha256:cb8a924......</purl>
    <component type="library">
        <group>foo</group>
        <name>bar</name>
        <version>1.0.0</version>
<EMAIL_ADDRESS>    </component>
</component>

However, if what you're describing IS the container, then you would simply do something like this:
<component type="library">
    <group>foo</group>
    <name>bar</name>
    <version>1.0.0</version>
<EMAIL_ADDRESS></component>
<bd:metadata>
    <bd:component type="container">
        <name>my-custom-container</name>
        <version>1.0.0</version>
        <purl>pkg:docker/my-custom-container@sha256:cb8a924......</purl>
    </bd:component>
</bd:metadata>

Note to self. Instead of describing individual types of files (image, font, etc), it may make sense to add support for mime type instead as an optional attribute on a component.

firmware will also be a component type in v1.2


gutentags changes to home directory on BufWritePost

At af7132f1bf89f18a69481b9fe5103c96c4248b9b gutentags#pwd uses getcwd(0, 0) if haslocaldir(). This call always returns an empty string, meaning that directory changes to the home directory when trying to restore it: tab page number zero does not have the expected meaning. You should remove the second argument completely, or supply tabpagenr() there instead.

Using vim-8.0.348.

@ZyX-I I was having a similar issue and seems that #117 fixes it, please try it.

I used this fork to see if it fixed the cd issue and it did not. Vim config:
    Plug 'hkupty/vim-gutentags', { 'branch': 'new-fix-tcd' }

When I save it still cds to $HOME... sometimes.

@mikedfunk I'll investigate that.. I'm also having problems with cd/tcd and I think vim-fzf has something to do with it.

The simple fix is to just replace getcwd(0, 0) with getcwd().  haslocaldir() (with no arguments) reports whether the current window has a local directory and getcwd() also acts on the current window.

getcwd(0, 0) works for neovim, so I think the proposed change is the following, right @jamessan
   if has('nvim')
     return haslocaldir() ? getcwd(0, 0) : haslocaldir(-1, 0) ? getcwd(-1, 0) : getcwd()
   else
-    return haslocaldir() ? getcwd(0, 0) : getcwd()
+    return getcwd()
   endif
 endfunction

I can add that on #117

getcwd() works for both vim and nvim, so you could probably just simplify it to
let wd = getcwd()
if has('nvim') && !haslocaldir() && haslocaldir(-1, 0)
  let wd = getcwd(-1, 0)
endif
return wd

I'm not sure about the NeoVim version, but wouldn't a simple fix be to change getcwd(0, 0) with getcwd(winnr(), tabpagenr())? It looks like passing 0 is the wrong thing here.

Actually, all of this is overthinking things.  getcwd() always returns the effective working directory.  If there's a window-local working directory, it will be returned.  If not and you're using nvim, then a tab-local working directory will be returned.  Otherwise the global working directory is returned.
So the simple answer is to just use getcwd().

@jamessan that might make sense.. However, we still need to take care not to override buffer/tab/global cwd settings.

I think the merging of PR https://github.com/ludovicchabant/vim-gutentags/pull/117 would fix this since, among a couple other things, it reverted back to using getcwd()?

Closing old issues -- I think this one was fixed anyway.


Missing GaugeWithExemplar

I want to have a GaugeWithExemplar that adds an exemplar whenever gauge derivates from zero. The actual use-case is making a flamegraph and putting it into the exemplar label, so that it can be easily found.
It seems like it would mostly be a copy of CounterWithExemplar, so I'm happy to make a PR if this makes sense.

Apparently there's https://github.com/OpenObservability/OpenMetrics/issues/241.

Thanks for the input. No opinion here, i.e. not opposed to merging a patch. Just want to be consistent with Prometheus and/or OpenMetrics.


fix: network change bug

:tada: This PR is included in version 1.0.0-semantic-release.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update MD file on new branch 2017-11-26_23-34-43

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Move exceptions.py to utils/exceptions.py

I didn't notice the path while reviewing the PR yesterday :(

I can expose exceptions at root level yes.
About having guidelines and consistency, let's try to do our best but it's not really in the essence of HF to formalize stuff in libraries :unamused:

Better late than never, we now have all the exceptions defined in huggingface_hub.errors! See https://github.com/huggingface/huggingface_hub/issues/2069.

Thanks for taking care, @Wauplin. I am closing this PR.


[BP-1.14][FLINK-24631][k8s] Use a stable subset of labels to select jobManager and taskManager pods

Backport #17685 to release-1.14.

@flinkbot run azure


Document mapping between function arguments and equations

In the documentation for RMTS, we see the equation
$$
\min_{w}  \frac{1}{2} \mathbf{w}^{T}\mathbf{H}\mathbf{w} +  \frac{1}{2}\beta \mathbf{w}^{T}\mathbf{w} + \frac{1}{2}\frac{1}{\alpha} \sum_{i}^{nt} [F(x_i) -  y_{t_i}]^2
$$
where "α and β are regularization coefficients".  In the usage section, we see the example code
sm = RMTC(
    xlimits=xlimits,
    num_elements=20,
    energy_weight=1e-15,
    regularization_weight=0.0,
)

It would appear that energy_weight is β, and that the regularization weight is 1/α? Nonetheless, this relation is not made explicit. Would it be sensible to update the docs thusly?
sm = RMTC(
    xlimits=xlimits,
    num_elements=20,
    energy_weight=1e-15, # corresponds to β in  equation (1)
    regularization_weight=0.0, # corresponds to α in equation (1)
)

I agree the mapping between the documentation and the API is not clear.
Actually, we should mention explicitely in the documentation the section 3.5 - Summary and implementation of the paper. It defines α'=α and β'=αβ which corresponds respectively to the energy_weight and the regularization_weight arguments (so the opposite of what is stated above).


WIP initial local function spike

need to double check the async/await lack of

After reading a bit of this not sure we save anything in terms of perf https://blogs.msdn.microsoft.com/seteplia/2017/10/03/dissecting-the-local-functions-in-c-7/

However, saying that we're not capturing arguments

You are capturing all the arguments; but I assume that's the point?
e.g. handler and statusCodeHandlers are used in the local function myFunc without being passed -however its not capturing any more than the closure (lambda) was previously.

Local functions are allocation-free if no conversion to delegates is happening

Your newly added local functions are simply acting as a replacement for the RequestDelegate, as a result these is a conversion from the local function to the delegate. This means that there are allocations, and it is not "allocation-free".

Its only creating them on start up though, so allocations shouldn't matter too much?

No not too much. We also have local functions here https://github.com/jchannon/Botwin/blob/master/src/BotwinModule.cs#L32 but again not sure we've gained by having them.

No captures in the example you just linked; a local function would be better - or moving the lambda to a readonly static; as while the lambda will be initalized only once it will still be in the method body which will make the method less likely to be an inline candidate due to its size https://github.com/dotnet/roslyn/issues/13624.
A local function will be better than a readonly static lambda as it will be created at compile time, while the readonly static will be done at runtime.
Doing it at runtime has the issue that the first method calling it will have an initalization check inserted by the Jit to see whether it has already been created; and everything else calling it will be free of that - though the check is pretty minor in the scheme of things.

I might have been wrong: handler :-/

wrong about what?

protected void Get(string path, Func<HttpRequest, HttpResponse, RouteData, Task> handler)
{
    Task RequestDelegate(HttpContext httpContext) => handler(httpContext.Request, httpContext.Response, httpContext.GetRouteData());
    this.Get(path, RequestDelegate);
}

The lambda is capturing handler; so you couldn't even move it to a readonly static

Brill. Thanks for the help guys. Will close the PR then as it adds no value.


enable loading hf llama checkpoints for training

[x] enable loading llama checkpoint in HF format
[x] enable loading safetensor format checkpoint stored locally (this does not introduce new deps, since safetensors is a dep of transformers)
[x] test llama forward pass with checkpoint loaded from HF + safetensor format
[x] fix typing-related lints in llama files
[x] light refactor of tests/models/test_llama.py

pasting some logs of the tests
test_llama_optimized with checkpoint_format=huggingface and model_name=7B
Output max diff: 0.5215804576873779
Output mean diff: 0.0007660071132704616
HF fp16 max diff: 0.6984603404998779
HF fp16 mean diff: 0.0012651692377403378
Logits max diff: 0.21308326721191406
Logits mean diff: 0.0019804914481937885
HF fp16 max diff: 0.21308326721191406
HF fp16 mean diff: 0.0028059757314622402

test_llama_parallel with checkpoint_format=huggingface, model_name=13B, and world_size=2
Output max diff: 0.4620845317840576
Output mean diff: 0.0006382844294421375
HF fp16 max diff: 0.7977371215820312
HF fp16 mean diff: 0.0013001899933442473
Logits max diff: 0.15855121612548828
Logits mean diff: 0.0018728922586888075
HF fp16 max diff: 0.1822366714477539
HF fp16 mean diff: 0.003323471173644066

Do these numbers look reasonable to @tridao ?

@tridao This PR is ready to be reviewed. Aside, I have two questions:

Have you tested the backward pass for llama with TP? I would like to run training soon.
How did you decide on some of the constants in the numerical tests? E.g., why is 3 is a reasonable number in this test(logits - logits_ref).abs().max().item() < 3 * (logits_hf - logits_ref).abs().max().item()?

I'm asking mostly to learn about setting these tests, as I have become a little frustrated with numerical loss in half precision.

Do these numbers look reasonable to @tridao ?

Yes they look reasonable.

Have you tested the backward pass for llama with TP? I would like to run training soon.

I didn't test the backward pass of llama specifically. I did test the backward pass of a generic GPT model with TP, to make sure the gradient is the same as if we didn't use TP (up to numerical precision).

How did you decide on some of the constants in the numerical tests? E.g., why is 3 is a reasonable number in this test(logits - logits_ref).abs().max().item() < 3 * (logits_hf - logits_ref).abs().max().item()?

No particular reason, I usually set this to 2 or 3. E.g. the definition of "reasonable numerical difference" is somewhat subjective. In my experience if two different implementations have numerical difference within a factor of 2 or 3 then they're most likely both doing correct things.

Have you tested the backward pass for llama with TP? I would like to run training soon.

I didn't test the backward pass of llama specifically. I did test the backward pass of a generic GPT model with TP, to make sure the gradient is the same as if we didn't use TP (up to numerical precision).

Thanks. Sounds like I should run a quick sanity check with llama just to make sure things are alright.

How did you decide on some of the constants in the numerical tests? E.g., why is 3 is a reasonable number in this test(logits - logits_ref).abs().max().item() < 3 * (logits_hf - logits_ref).abs().max().item()?

No particular reason, I usually set this to 2 or 3. E.g. the definition of "reasonable numerical difference" is somewhat subjective. In my experience if two different implementations have numerical difference within a factor of 2 or 3 then they're most likely both doing correct things.

For context all tests with the hugging face checkpoint format pass. I’m running a final sweep with the meta format checkpoints.

For context all tests with the hugging face checkpoint format pass. I’m running a final sweep with the meta format checkpoints.

I installed all the fused kernels, and now all tests pass.

Thanks so much @lxuechen!


OffsetManager, ability to mark previous offset

SHA: a23cf43069c4a88e12887ac68250bb69e0fe8c84
in https://github.com/Shopify/sarama/blob/master/offset_manager.go#L321-L325
Offset manager only commit the offset if it's larger than current offset.
I'm having a problem in this one particular use-case where my program accept an option to read kafka from beginning. If for example current recorded offset is 10000 and I start from the beginning but stop at 500, next time I run the process again it will start at 10000 instead.
What do you think if we could add an extra function / parameter to skip this check ?

Interesting. It's a legitimate use case to "reset" the offset, but that currently cannot be done with the offset manager. You well have to hand-craft a CommitOffset request to do this (for an example, see here).
I am not necessarily against removing the if statement. Maybe adding a ResetOffset method would be a better idea, to make it explicit that you're trying to commit a lower offset? Unsure.

another option is a SetOffset method to make it explicit that you're committing an arbitrary offset, instead of doing ResetOffset followed by a MarkOffset

Can someone confirm this - In order to "hand-craft a CommitOffset request" (to reset offset), it needs access to the latest generationID, without which resetting an offset doesn't seem to work, if consumers are running already.

Hi guys. I'm trying to do the same thing.
I tried to remove the if just to test, and the program just blocked...
So now I'm trying to manually commit the offset but it doesn't look like it's working reliably either.
Here is an example of what I'm trying to achieve:
package main

import (
	"fmt"
	"github.com/Shopify/sarama"
	"os"
)

func main() {
	config := sarama.NewConfig()
	config.ClientID, _ = os.Hostname()
	config.Version = sarama.V0_10_1_0
	kafkaClient, _ := sarama.NewClient([]string{"<IP_ADDRESS>:9092"}, config)

	request := &sarama.OffsetCommitRequest{}
	request.ConsumerGroup = "blah"
	request.ConsumerGroupGeneration = sarama.GroupGenerationUndefined
	request.RetentionTime = 1000 // not sure what to put here
	request.Version = 2
	request.AddBlock("blah", 0, 5, 0, "Reseting offset to 5")
	broker, _ := kafkaClient.Leader("blah", 0)
	response, _ := broker.CommitOffset(request)
	fmt.Println(response)
	broker.Close()
	kafkaClient.Close()
}

And I end up with this error:
&{map[blah:map[0:kafka server: The provided member is not known in the current generation.]]}

Is there any way to commit a completely arbitrary offset without having to create a consumer?
Thanks!
PS: And I agree that ultimately, the PartitionManager should be able to handle that case. Plus, it's really not clear that committing an offset that's earlier that the current one doesn't do anything if you don't look at the code. A log message would be welcome I think :-)

Would gladly submit a PR to work on a solution for this. The two that I can think of are:

Dropping the guard that makes sure that we only increment offsets.
Adding another func on the partition offset manager to allow for resetting to lower values.

Would you be interested in either or another solution that you can think of?
PS. sorry for the noise my references cause.

I would be happy to take a PR for a separate function; I agree with Willem that it's better to be explicit in this kind of case.
I thought there was some sort of reason I hadn't looked at this, that it was more complicated than it seemed, but honestly I can't find anything too scary-looking here.


Implement logging of downloads with simple DynamoDB on AWS

Regarding #18
Concerns mostly:
@Emily-Jiang @cealsair @rdebusscher
Hi guys, so this is what I suggest as a solution. if it doesn't stir up any opposition I will go ahead and forge this little demo into an application scoped bean callable from GeneratorDataBean.java. Rudy might suggest a better integration though...
AmazonDynamoDB
We host the service on Amazon, so there is probably no harm in using it for a simple logging of loosely structured data rows. If we want to leave, we always can. Moving away is as easy as dumping that log into JSON and importing it into something else.
Demo of what we might log
(This is just a showcase, not integrated, it is hard coded, demo, etc.)
package org.eclipse.microprofile.starter;

import com.amazonaws.regions.Regions;
import com.amazonaws.services.dynamodbv2.AmazonDynamoDB;
import com.amazonaws.services.dynamodbv2.AmazonDynamoDBClientBuilder;
import com.amazonaws.services.dynamodbv2.document.DynamoDB;
import com.amazonaws.services.dynamodbv2.document.Item;
import com.amazonaws.services.dynamodbv2.document.PutItemOutcome;
import com.amazonaws.services.dynamodbv2.document.Table;
import org.junit.Test;

import javax.xml.bind.DatatypeConverter;
import java.security.MessageDigest;
import java.security.NoSuchAlgorithmException;
import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Set;
import java.util.stream.Collectors;
import java.util.stream.Stream;

/**
 * @author Michal Karm Babacek<EMAIL_ADDRESS> */
public class DownloadsLoggerDemoTest {

    @Test
    public void demoLoggingTest() throws NoSuchAlgorithmException {
        final AmazonDynamoDB client = AmazonDynamoDBClientBuilder.standard()
                //.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration("http://localhost:8000", "eu-west-1"))
                .withRegion(Regions.EU_WEST_1)
                .build();
        final DynamoDB dynamoDB = new DynamoDB(client);
        try {
            final Table table = dynamoDB.getTable("microprofile_starter_log");

            final Calendar calendar = Calendar.getInstance();
            final SimpleDateFormat formattedTime = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSSZ");
            final MessageDigest hash = MessageDigest.getInstance("SHA-1");

            // Logged
            final String mpVersion = "2.0";
            final String supportedServer = "liberty";
            final String beansxmlMode = "ANNOTATED";
            final String moduleStructure = "SINGLE";
            final Set<String> selectedSpecs = Stream.of("HEALTH_METRICS", "HEALTH_CHECKS", "REST_CLIENT").collect(Collectors.toSet());
            final String timestamp = formattedTime.format(calendar.getTime());

            // Just used to add entropy to unique stamp
            final String groupId = "biz.kamrs";
            final String artifacId = "demodemo";

            // This should uniquely mark log record row. World wouldn't end if there is a collision though
            final String logmark = Stream.concat(
                    Stream.of(mpVersion, supportedServer, beansxmlMode, moduleStructure, timestamp, groupId, artifacId),
                    selectedSpecs.stream()
            ).collect(Collectors.joining());

            hash.update(logmark.getBytes());

            final String hashedLogmark = DatatypeConverter.printHexBinary(hash.digest());

            System.out.println("logmark:" + logmark);
            System.out.println("logmark hashed:" + hashedLogmark);

            final Item item = new Item()
                    .withPrimaryKey("logmark", hashedLogmark)
                    .withString("mpVersion", mpVersion)
                    .withString("supportedServer", supportedServer)
                    .withString("beansxmlMode", beansxmlMode)
                    .withString("moduleStructure", moduleStructure)
                    .withStringSet("selectedSpecs", selectedSpecs)
                    .withString("timestamp", timestamp);

            PutItemOutcome outcome = table.putItem(item);

            System.out.println("Outcome:" + outcome.toString());

        } finally {
            client.shutdown();
        }
    }
}

Runtime
Once I have AWS_SECRET_ACCESS_KEY and  AWS_ACCESS_KEY_ID env vars set I can run it.  For testing it is possible to use local DynamoDB, but I configured the AWS DynamoDB for our account and tried it on "production" right away:
-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.eclipse.microprofile.starter.DownloadsLoggerDemoTest
logmark:2.0libertyANNOTATEDSINGLE2019-01-17T00:37:44.115+0100biz.kamrsdemodemoHEALTH_CHECKSREST_CLIENTHEALTH_METRICS
logmark hashed:6FB59BD3338342E6A34CA39CBDBC73012E021BB7
Outcome:{}
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.38 sec

AWS Console
I executed the "test" twice and what I can see are two log entries:

Command line
if you have AWS command line tool installed, you can easily use DynamoDB from your shell, either complex queries or scans, this dumps everything:
$ aws dynamodb scan --table-name microprofile_starter_log
{
    "Count": 2,
    "Items": [
        {
            "moduleStructure": {
                "S": "SINGLE"
            },
            "logmark": {
                "S": "4CBCD59B08945E9C0C84DCE262DE13E700DF34D5"
            },
            "selectedSpecs": {
                "SS": [
                    "HEALTH_CHECKS",
                    "HEALTH_METRICS",
                    "REST_CLIENT"
                ]
            },
            "timestamp": {
                "S": "2019-01-17T00:57:04.348+0100"
            },
            "beansxmlMode": {
                "S": "ANNOTATED"
            },
            "supportedServer": {
                "S": "liberty"
            },
            "mpVersion": {
                "S": "2.0"
            }
        },
        {
            "moduleStructure": {
                "S": "SINGLE"
            },
            "logmark": {
                "S": "6FB59BD3338342E6A34CA39CBDBC73012E021BB7"
            },
            "selectedSpecs": {
                "SS": [
                    "HEALTH_CHECKS",
                    "HEALTH_METRICS",
                    "REST_CLIENT"
                ]
            },
            "timestamp": {
                "S": "2019-01-17T00:37:44.115+0100"
            },
            "beansxmlMode": {
                "S": "ANNOTATED"
            },
            "supportedServer": {
                "S": "liberty"
            },
            "mpVersion": {
                "S": "2.0"
            }
        }
    ],
    "ConsumedCapacity": null,
    "ScannedCount": 2
}

Question
Good / no good?  IMHO it is just what we need for simple logging.

PR merged.


A space before the link should not be deleted

html like this

I need a space

translates into this
I[need](http://google.com) a space

I think that space before the link should not be deleted.

Thanks for spotting that! I’d be happy to accept a fix for this.
I believe it has something to do with: https://github.com/domchristie/to-markdown/blob/master/index.js#L122
Thanks again.

(might require a bit of trial and error!)

I using this and I'm seeing html removing the first space so:
something strange happens

becomes
something**strange** happens

WOW! It was so fast! Thanks @domchristie!

I have bad news.
html like this

I need more spaces!

translates into this
I [need](http://google.com)[more](http://yahoo.com) spaces!

As you can see there is no space between links.

Ugh, sorry about that. It’s pretty difficult to get the right amount of trimming!

That latest fix seems to have resolved the collapsed spacing issue we were experiencing, where:
The average cost for overseas medical treatment is £2,040
Would become:
The average cost for overseas medical treatment is**£2,040**

@ourmaninamsterdam unfortunately not for consecutive inline elements though :(

Released in 1.0.0-rc.4

:+1:  @domchristie

Thanks. :) Just need to sort out those minor trailing whitespace issues…

@grabus @ourmaninamsterdam Should be pretty solid now in 1.0.0-rc.6

(or 1.0.0-rc.7!)

Works good :+1: Thanks @domchristie .

@ourmaninamsterdam you’re welcome! Sorry for messing about with frequent updates. There are so many weird cases!


bump-formula-pr: Determine Linux-only formulae with depends_on :linux

[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?

We used to use # tag "linux" in Homebrew/linuxbrew-core for the ~50 Linux-only formulae. As of https://github.com/Homebrew/linuxbrew-core/commit/6578a4aa86b80816cf0553f1d45828c0072197a4, we're using depends_on :linux to have a consistent syntax between Linux and macOS.
Therefore, we have to change the search string for Linux-only formulae that determines whether or not brew bump-formula-pr deletes the bottle line.

Successfully tested with https://github.com/Homebrew/linuxbrew-core/pull/19623.

Nice work 👍


ci: pull trivy image from ECR vs github due to rate limiting

This update is to use the ECR AWS registry for the trivy db vs Github due to rate limiting. This may not fix the issue, but will look at caching the db or an alternative replacement.

I swapped to ECR AWS and still the same issue. I'm closing this PR. will research either caching the db or going a different provider.


Use clippy and fmt rules from rust stable

refactor(fmt): use fmt style from rust stable

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

refactor(clippy): use rust stable and remove exceptions

Switch to rust-stable and remove most clippy exceptions from linter script.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors try

bors cancel

bors try

bors merge

bors merge


Consistenly rebind loop variable in tests

Summary
Ref: https://github.com/cosmos/ibc-go/pull/4413#discussion_r1301257466
Though in many cases not necessary, this is a nice defensive rebinding to do in order to guard against the possibility of running things in parallel. A quick grep showed me that there's other instances where this is omitted, would be nice to get them consistent.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

I think in this case, we should also wrap the actual test logic inside a suite.Run like this:
	for _, tc := range testCases {
		tc := tc
		suite.Run(tc.name, func() {
			suite.SetupTest()

			// ...
		})
	}


Easy way of changing the color of the selected item's text

Hi, I'm wondering if there is an easy way of changing the color of the selected item's text (without touching the items in the dropdown itself), besides the selectedItemBuilder, which requires a List<Widget>? It would be useful to do this via a property, which accepts a Color.

What's the problem with selectedItemBuilder ?
It's familiar and been used at flutter's DropdownButton.

The only problem with it - it's not user friendly for this simple case. Any chances of adding the selectedItemTextStyle (or something similar) property soon? Or maybe to make it a part of the buttonStyleData... Thanks.

The thing is: making selectedItemTextStyle will not just update TextStyle of the items but will also ignore the other characteristics applied to items like padding, alignment, etc.. so, you'll have to use selectedItemBuilder anyway.

If you don't want to duplicate items.map(...).toList() part at both items and selectedItemBuilder properties, you can make a helper function that take your ItemTextStyle as a parameter and return that list. There's no way to do that internally at the package.

I'm just wondering, why selectedItemBuilder always returns a List<Widget>, including the case, where there is only one selected item? What should I do in this case?

It allows more flexibility in specifying how a certain selected item is displayed, same as items, i.e:
selectedItemBuilder: (BuildContext context) {
  return items.map<Widget>((String item) {
    if (item == "something")
      return Text(
        item,
        style: TextStyle(color: Colors.red),
      );
    return Text(item);
  }).toList();
},

That'll show text in red only when something item is selected.
If you don't care and want to show the same for all items then just return the same widget for all items.

Thank you anyway)


minor: with_inputs() can use original schema.

Which issue does this PR close?

Closes #.
Rationale for this change
following #6820
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

``


Extend relationalBone to hook-able getEntity function

I need this to override the way how an entity is retrieved: It may also be referenced by an unique îdentifier that needs to be looked-up. Here's an example implementation:
class appBone(relationalBone):
	"""
	appBones is a relationalBone that accepts also the app identifier as valid key...
	"""

	def __init__(self, descr="App", kind="app", indexed=True, *args, **kwargs):
		super().__init__(kind=kind, descr=descr, indexed=indexed, *args, **kwargs)

	def getEntity(self, key):
		entity = super().getEntity(key)

		if not entity:
			entity = db.Query(self.kind).filter("identifier", key).getEntry()

		return entity

Can we push this to the next ViUR meeting? There no comments yet and i'm also quite unsure if that's a good idea...


Move all synchronization out of GameNode

Just like how it was done with GameUsernameSync.
I believe (I could be wrong) that it would be much better if the all game sychronization code were at the same place. They would all be children of GameSync (a child of the Main node). These nodes would all automatically spawn when a new game is created, and despawn when the game is discarded. If you want to synchronize a game, all you'd have to do is create a GameSync node for it. Also, GameNode is just an implementation detail of the game's visuals. We should be able to make and use different scenes for the game visuals. The visuals shouldn't be responsible for synchronization; even if they were, they would always have the sync nodes, breaking the DRY principle.
Alright, that's enough overthinking.
(More detailed tasks TBD)

[ ] AutoArrowSync
[ ] ActionSynchronizer

Oh yeah also it would make it possible to synchronize a game without having to create visuals for it... Kind of a big deal!


Partly accept in JetBrains IDE

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
Does JetBrains IDE support partly accept?
Solution
No response

@facelezzzz Currently this is possible with option+command+right

It works, thank you


Add ldn-channel-2023 ED

This ED defines the LDNChannel2023 Notification Channel Type. This specification deprecates LinkedDataNotifications Subscription 2021.

Preview

Updated to follow the structure of Solid Notifications Protocol Version 0.2.0.
Introduced activityType as a feature to indicate the desired notification activity type, e.g., messages can be about particular activities, like when a resource is created or added to a container (a general requirement as suggested in https://github.com/solid/notifications-panel/issues/1#issue-489650894 ). I suggest making this a core feature of Solid Notifications Protocol.
Subscription Server and Notification Receiver is a specialisation of LDN Receiver. (See also https://www.w3.org/TR/ldn/#subscribing-to-notifications )
Subscription resource acts like an LDN inbox. POSTing to it returns a 201 to a resource where it can be modified or deleted (aka unsubscribing). Whether DELETE is used to unsubscribe or POSTing again to the created resource indicating the changes to be determined.
Introduced receiver (analogous to sender) to identify the party (Notification Receiver) that controls the sendTo resource.
Described an outline for exchanging public keys and towards signing messages. (Copied to issue https://github.com/solid/notifications/issues/148 ).

Merging as per https://github.com/solid/notifications-panel/blob/main/meetings/2023-02-02.md#add-ldn-channel-2023-ed .


Print UI URL for step-functions trigger and argo-workflows trigger

          Can we file an issue for a follow up PR that prints the UI URL for `step-functions trigger` and `argo-workflows trigger`?

Originally posted by @savingoyal in https://github.com/Netflix/metaflow/issues/1290#issuecomment-1456677430

I would like to contribute on this issue, is it available?

That would be great.  I would recommend waiting for the parent PR to close first (#1290)

Thank you, also I'm new to resolving issues. I have set up the repository, if I have questions to make should I post them here?

guess this is closed?


Issue(s) with release 2.2.1 (e.g: custom color doesn't work)

Hi,
custom color
While I am testing NGL 2.2.1 for nglview (https://github.com/nglviewer/nglview/pull/1086) , I find that the custom color doesn't work anymore

Thanks for reporting this. I've noticed the same issue in a separate branch and did not realize it was still in production.

Thanks @ppillot. I look forward for the fix.

@hainm fix has been published to npm as v2.2.2

@ppillot I confirm the fix does work. Thank you so much, Paul.


Add small object tracking demo using SAHI

This PR implements a demo using SAHI: Slicing Aided Hyper Inference to be able to track small objects in a video.

@fcakyon maybe you have any comments or suggestions here :)

Wow @JuanFKurucz, thanks for this awesome demo!
@dekked added some comments. In overall, it looks good 💯


Adds support for Number Arrays as Body Parameters

See #22.

This could probably be expanded to support Number Arrays in other places.
Also, playing around with this was interesting 😛


Java command line options do not get considered when 'bal run' is used for running a Ballerina file or a package

Description:

If the java heap size needs to be increased to run a Ballerina program, changing the java command line options -Xms -Xmx in the BAL_HOME/bin/bal file have no effect in the Ballerina runtime jvm heap size,  if bal run is used inside a ballerina package or to run a file.bal Currently it works only when using bal run with a file.jar.
Describe your problem(s)
Describe your solution(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

The -Xms and -Xmx can be set with JAVA_OPTS environment variable.


Removes BUILD_CPR_TESTS from cache

The project's CMakeLists.txt file has the BUILD_CPR_TESTS to ON by default, but when added as submodule most probably this is not be desirable (it is my case).
The problem is that most of the times (especially on the first checkout) one forgets about this and simply executes the cmake command, which produces a cache of values and on that cache the value of BUILD_CPR_TESTS is set to ON. After you realize about the tests being built and change it on the CMakeLists.txt file, there is no effect on test build since cmake will use the cached value as long as it is there (workarounds are always to edit/delete the cache and re-run cmake).
Unsetting the value of BUILD_CPR_TESTS right before the end of the file will force cmake on the next execution to read again the value on the CMakeLists.txt file instead of using the cached one, which will have no effect on building time but it will on personal time spent until one realizes about the cmake cache smashing the clearly written value on the file (again, this has happened to me in several occasions).
This can also be done with any other parameters that are to be changed constantly or by the project using cpr as submodule mantainer (and default test building is most surely one of them).

Just figured you can add set(BUILD_CPR_TESTS OFF CACHE BOOL) right before add_subdirectory(cpr) to force tests not being build without having to ever touch cpr CMakeLists.txt.


Added Manual Installation guide

fixes #3

Ok, looks good. Thanks!

Awesome. Happy to help your lib! 😄


Line read command mechanics

Determine if line read without any arguments works as follows:

reads current line
shifts the pointer to the next line

If so, implement the overload for this kind of stuff

No it doesn't.


Linking Info.plist to main framework target

For some reason this was unlinked, causing some code signing issues.

@facebook-github-bot shipit


Spawn world as a moon makes an inaccessible main planet

I have my spawn world as a moon around a planet.
The central planet (around which the spawn world/moon orbits):

appears very large
you fly through that large outer surface (and looks like clouds from behind)
hides a tiny inner planet
if you land on that planet you get trapped and cant move or jump/get off that planet

It only seems to happen to the spawn planet as other planets in the system with moons are fine.
It has happened to two of my games of late.

I've got this too. Any luck so far?

I've only had this issue if I set my start as moon of gas giant, no problem if starting as moon of regular planet


Fix typos in README. Add hint to speed up HIE compilation

Hi!
I'd like to offer two improvements to the installation instructions for Arch Linux:

Typo fixes
Add hint to reduce the number of supported GHC versions thus speeding up HIE compilation

Thank you for catching this!


fix: fullscreen button not working for videos

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #7744
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

https://github.com/kodadot/nft-gallery/assets/16473062/0ab9af9e-c6fa-4e90-857d-5e7b36a3edfc
Copilot Summary

🤖 Generated by Copilot at e8b726c
Improved the layout and responsiveness of the gallery item modal on different screen sizes. Fixed a bug where the modal close button was hidden by the media.

🤖 Generated by Copilot at e8b726c

Sing, O Muse, of the skillful coder who refined the gallery
And made the modal window fit the screens of every size
He added append-to-body and max-height to b-modal
And gave gallery-item-modal-content full width to please the eyes

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 4.15 USD/DOT ~ 7.229 $DOT
🧗 13rv1SWoLg9Gb3tmvHPZxb7JbVy51BtMziX7k9WQGSJ7Kp3A
🔗 0xf676f022e35cd8a50813817994ec1a7ec5e8b9587cc7c22fa9b1c41dc60a6226
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Allow a single notebook to export to multiple .py modules

Use the same #|export modname syntax that was in nbdev1

The directives docs suggest this issue can be closed, no?

Maybe you could give it a try and see if it works? I don't recall where we got to with this.

It doesn't work in my simple example. The export directive silently fails, not exporting the function anywhere:
nbdev-hello-world-test.zip

@hugetim it works in my example with nbdev Version: 2.3.13
01_two.ipynb:
# | default_exp two

#| export core
def foo_core_mixin(): # is exported to core.py
    pass

#| export
def foo_two(): # is exported to two.py
    pass

However, I don't like the fact that it appears that you can't have two notebooks with the same default_exp (as it says here, it gets overwritten. However, i can't find anything in the docs about this)


Music Preferecences:  Social Media Share Text

The current text for sharing on social media is showing points award , it does not really fit the context of Music Preferences Experiment. It would be nice to present a more informative summary that highlights participants' actions in the experiment. So, the text provides a clear and concise summary of the information to be shared on social media.
Proposed Text:
Your top 3 favorite songs: 'XX,' 'YY,' 'ZZ.' You know N'out of 64 songs. All players' top 3 songs: 'A,' 'B,' 'C.'
In Chinese:
您最喜欢的三首歌曲：'XX,' 'YY,' 'ZZ‘；您知道64首歌曲中的N首；其他人最喜爱的三首歌曲：'A,' 'B,' 'C'。

Considering that people will share this, wouldn't it be better to have this read "My top 3 favorite songs: ..." It would also be good to include the URL, to invite other people to play, too. So start with "I rated songs on {{url}}! My top 3 favorite songs..." @XHuang218 what do you think?

Yes, it'd be better to have the text read "My top 3 favorite songs ..." and it makes sense to include the URL.
But I want the text to be direct to the point at the beginning, so it would have this read "Explored musical preferences together at {{url}}! My top 3 favorites songs: 'XX,' 'YY,' 'ZZ.'；I know N out of 64 songs；All players' top 3 songs: 'A,' 'B,' 'C.'
In this case, the new Chinese translation is:
一起探索音乐喜好!{{url}}  我最喜欢的三首歌曲：'XX,' 'YY,' 'ZZ‘；我知道64首歌曲中的N首；其他人最喜爱的三首歌曲：'A,' 'B,' 'C'。

This has already been addressed, closing.


Many errors when compiling

Describe your problem here.
First I wanna say sorry ahead of time, because I do not use GitHub myself too often.
Now, when compiling after fixing SScript, I receive a few errors that seem to all be of the same type, like so:
source/flxanimate/PsychFlxAnimate.hx:92: characters 11-21

  92 |  function _removeBOM(str:String) //Removes BOM byte order indicator
     |           ^^^^^^^^^^
     | Field _removeBOM should be declared with 'override' since it is inherited from superclass flxanimate.FlxAnimate

Or this one:
source/states/stages/Tank.hx:332: characters 24-27

 332 |   pico.anim.onComplete.add(picoStressCycle);
     |                        ^^^
     | () -> Void has no field add

There are a few more, though I don't know if posting them all here would be very efficient or not. Though I can. (I am not sure if this helps, but I am compiling in Visual Studio Code)
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

haxelib remove flxanimate
then:
haxelib git flxanimate https://github.com/Dot-Stuff/flxanimate 768740a56b26aa0c072720e0d1236b94afe68e3e


Use parameter=value for all dragen 4 workflows

Will make parameter querying much easier in future
Resolves #264

@alexiswl - can we run a test in development on any one of the tools to make sure this woks as expected?

Yup, done here trn.3070763aab584c619bd868322333417a, uses = for boolean values, updated and using trn.461f285439f148dfbd35a77361dd4a34 for all values!

@alexiswl - also wondering is there any specific reason for only updating syntax for boolean arguments? Would it not work for others?

@alexiswl - also wondering is there any specific reason for only updating syntax for boolean arguments? Would it not work for others?

Not just boolean arguments - see umccr/cwl-ica@41663dc (#267)


The 100k virtual scrolling table - it actually uses 10k records

I've looked at the virtual.ts sample which read the "100k.json" file and sends to the datatable. The web sample says it displays 100k rows, while there're only 10k business objects in json. The file itself has 100k rows, but not 100k objects :)

LOL Thanks for letting me know.


IndexError: pop from empty list on request finished signal handler

Raven-Python 6.0.0 on Django 1.9.13
All affected requests are HEAD. But not all HEAD requests are affected. Neither all of the exact same requests(Uptime tracking system) are affected.
IndexError: pop from empty list
  File "raven/middleware.py", line 20, in common_exception_handling
    yield
  File "raven/middleware.py", line 71, in close
    self._close()
  File "django/http/response.py", line 252, in close
    signals.request_finished.send(sender=self._handler_class)
  File "raven/contrib/django/middleware/__init__.py", line 111, in request_finished
    client.transaction.pop(self._txid)
  File "raven/utils/transaction.py", line 47, in pop
    return self.stack.pop()

Could this be caused by a request_finished sent without having the prerequisite request_started ? I suspect a premature close of the TCP/HTTP connection causing the request cycle behave in a weird manner.
On a 7days range, this error happens on few candidates (<10) over a very large sample (100,000+) of identical requests spread across tens of dockers hosts running the same codebase.

We ran into this too.  With raven 6.7.0 and Django 2.0.4


[Bug] can't resolve next/head in storybook in nx monorepo

Bug Report
Describe the bug
We have a nx monorepo, with storybook in a separate app from the next instance. I have a storybook story about a component using the CldImage component, and upon start storybook it throws:
ModuleNotFoundError: Module not found: Error: Can't resolve 'next/head' in '/Users/sjanmaat/code/[...]/node_modules/next-cloudinary/dist'
Did you mean 'head.js'?
BREAKING CHANGE: The request 'next/head' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '*.mjs' file, or a '*.js' file where the package.json contains '"type": "module"').

Storybook itself can find next/head, I can import it in preview.jsx no problem.
The next-cloudinary package can be found in the root node_modules, just like next.
Is this a regression?
I don't think so. I'm new to this package.
Steps To Reproduce the error
Expected behaviour
I expect it to find next/head.
Your environment
next-cloudinary 3.4.0
Storybook 6.5.16 built with webpack5
nx 14.1.5
yarn 3.3.1

OS: MacOS Ventura
Node version: 16
Npm version: 8.15.0, but we use yarn 3.3.1
Browser name and version: Chrome

Additional context
I have set up a next instance with create-next-app, with storybook 7 beta and CldImage and then it does work fine.
Next I'll try a clean nx monorepo setup and see if that works.

hey @stijnjanmaat do you have a public repo available i could spin up and poke at?
i haven't used nx before and haven't tried adding any of the components to storybook yet

In the mean time, I have found a workaround for the error for Storybook 6.5 by adding this to webpackFinal in .storybook/main.js:
  webpackFinal: async (config) => {
    const newConfig = config;

    newConfig.resolve.alias = {
      ...newConfig.resolve.alias,
      'next/head': false,
      'next/script': false,
    };

    [...]

    return newConfig;
  },

Then i ran into the next error: Uncaught You must supply a cloudName when initializing the asset
But that was fixed by setting NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME="my-cloud-name" in .env of storybook.
So in short, now have a working CldImage in Storybook albeit with a workaround.

I don't really know to be honest... but i'm okay with this work around until Storybook 7 is released, which should be very soon, March 14 2023: https://www.thedroptimes.com/27692/storybook-will-hit-version-70-march#:~:text=Those who are familiar with,about to hit version 7.0.

Ah soon then. Okay. Gonna close for now. Thanks for looking into this and the info


addQuantize and addDequantize in th_final_initialize of anamoly detection submitter_implemented.cpp

Would it be possible to receive an explanation on towards why these functions are being used in the th_final_initialize function. From what I can see, they only run a check and are never used anymore. I ask because I am having some trouble importing the files needed for them to work onto my project. The issue being that adding all the source files needed for those functions makes me go over my memory requirements.

Basically, are these functions/checks essential to the project or would I be able to leave them out?

Those are the functions that add the required operations to the TFLite Micro interpreter. Without them, I don't think the model will be able to successfully execute because it wouldn't be able to resolve calls to any of the layers.
micro_op_resolver.AddFullyConnected() is a function that adds the fully connect operation to the op resolver. It returns kTfLiteOk if the function executes successfully.
What are your memory requirements? I may be able to help optimize the memory.

Right now I am trying to report all the source files needed to get those functions running again; so that I can show you what happens. However, it is taking longer than expected so I will give an update as soon as possible
The device has 352kB of flash and 80kB of RAM. Before adding all the source files needed for the AddQuant and AddDequant I am roughly using 95%-97% of the flash already. I already had all the files needed for the AddFullyConnected functions. Last time I tried to add all the files I began to go over the memory requirements.
Also, I am not quite sure if this file is needed. It shows up in a few files but it is not a source or header file. Also, when I go into the file it is just a single line definition.

Looking at the model (below) it seems the int8 model we provide doesn't need the quantize/dequantize operations so you should be able to remove those.
What do you mean by "had all the files needed for the AddFullyConnected functions". You should probably keep micro_op_resolver.AddFullyConnected().

Sorry, I just meant that AddFullyConnected() was working with no problem. It was the quantize/dequantize giving me the issues.
So I should be fine just keeping micro_op_resolver.AddFullyConnected() while removine micro_op_resolver.AddQuantize() and micro_op_resolver.AddFullyDeQuantize().

If you are using the int8 model we provide then yet I think that should work.
Let me know how it goes!

Hello,
I tried running the project with the micro_op_resolver.AddFullyConnected() function, but it caused the an overflow of FLASH by 377 bytes. Is there any way of optimizing the memory?
Thank you!

Update: The flash overflow is at 10805 bytes now


Missing basic information about how to get started with App.

[Enter feedback here]
Hi
I tired to use the Power Bi App function.
But the GUI require a "CapacityID" ??
Where can I found this Id?
Why is the ID not present in a Power BI report?
Also this setup information are missing from the webpage about Power BI App.

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

ID: fe16182b-c80c-4cc3-da85-bf6fb0fe152f
Version Independent ID: 6b92afe7-dfda-39b3-455c-e886b27e3d70
Content: Apps in Power BI - Power BI
Content Source: powerbi-docs/consumer/end-user-apps.md
Service: powerbi
Sub-service: pbi-explore
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @GGBMLC. This issue has been assigned to the doc author for review.

Hi @GGBMLC ,
Thank you for submitting your feedback.  However, I am unable to duplicate your error.
This looks like a support request or product question rather than documentation feedback. You might find a solution in the Power BI Community, at https://community.powerbi.com. You may also be able to contact Support (refer to this article: https://docs.microsoft.com/en-us/power-bi/admin/service-support-options to review available Support options.)


Chromecast Playing Youtube shows up as Off

Home Assistant release (hass --version):
0.20.3
Python release (python3 --version):
Python 3.4.2
Component/platform:
media_player/Google Cast
Description of problem:
Chromecast device reports youtube content as 'off' status.
Expected:
When youtube content is played on chromecast, should show content playing, or at least youtube, on the media player widget.

Link Chromecast to dashboard via the media_player component and the Google Cast platform
Play content via youtube on the Chromecast
Note result on Dashboard.

This works with Plex as well as Netflix, but not YouTube. i assume it's outdated code in combination with the recent youtube cast protocol changes. If someone is willing to point out where to start working, I can contribute to getting this issue fixed. Thanks!

I've started to dive into this. First reports I have are just that Youtube is accurately reporting an App_ID and a display_name when HASS is showing it as off.
This is a log from when HASS was playing hulu, then switching to youtube and playing a video. On each status update I was logging the app_id and app_display_name.
Going to keep working on this tonight.
http://pastebin.com/raw/H9yb6UpS

Okay so this is, as expected, a pychromecast issue. I've posted an issue report there:
https://github.com/balloob/pychromecast/issues/97
Before I close this, is it against our best practices to add support for declaring any youtube chromecast status 'playing', just so playing events will at least trigger, and maybe have volume/play/pause support for youtube videos? As that could be done in just a few lines, but it could be considered a band-aid for a dependency's issue.

@Aubron I've been playing youtube videos all day and they all show up for me as playing on my chromecast.

I'll try with the web browser later today, but I should have latest everything, unless the iOS version is screwing this up. I've even tried compiling a version with a newer version of pychromecast with no luck.

Does this still occur? Is it limited to when iOS devices are doing the casting?

@aubron I just tested this with iOS and android and it worked. Can this be closed?

Almost certainly. It's been a while since I tried, whatever it was is
probably not an issue any longer.
On Sun, Dec 4, 2016 at 1:27 PM William Scanlon<EMAIL_ADDRESS>wrote:

@aubron https://github.com/aubron I just tested this with iOS and
android and it worked. Can this be closed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/2219#issuecomment-264721020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHehCd0C4Ns3EyGzmG1XIgWBnI_QPZ5ks5rEwYkgaJpZM4ItOX-
.

--
[image: Immersion Media] https://aubronwood.com/
[image: Twitter]  https://twitter.com/aubron [image: Facebook]
https://facebook.com/aubron [image: Google +]
https://plus.google.com/+AubronWood [image: LinkedIn]
https://www.linkedin.com/in/aubron [image: Github]
https://github.com/aubron
Aubron Wood / Head of Development
Lead Web Developer
<EMAIL_ADDRESS>aubronwood.com
Immersion Media
(910) 868-8441
Fayetteville, NC
imsports.com
https://imsports.com/

Awesome, closing.


Consider adding descriptions to entities

The only one that needed it was Client -- did I miss any? // @scottbrady91
Done on dev/2.0


RRE 7.5: "warning message: cannot decrease memory memory: ignored"

build 2016-02-24.3
machine: VM with 4G memory
RRE 7.5
The first command (ls()) shows warning message. I seems like from setting memory limit initially.

ls()
character(0)
Warning message:
cannot decrease memory limit: ignored

7.5 is no longer relevant


Add warnings for experimental features

ref: https://github.com/lima-vm/lima/blob/master/docs/experimental.md

How does one remove the warnings ? When are features graduated to non-experimental ?

Not saying that these features are not experimental or that the warning is not warranted :-)

How does one remove the warnings ? When are features graduated to non-experimental ?

Yes


Current Search Bar Hotkey

Feature Requested
A command through surfing can be used to edit the current search bar using a command slash hotkey. Editing the search bar is done often and should be as quick and easy as possible.
Relevant Screenshot
No response
Checklist

[X] The feature would be useful to more users than just me.

press / why you are in surfing view.

press / why you are in surfing view.

Oh, you mean specific hotkey.

Oh, you mean specific hotkey.

Done, will be available while pr is merged

Done, will be available while pr is merged

Thanks! That was so quick! Nice work! Do you plan on expanding t?

Thanks! That was so quick! Nice work! Do you plan on expanding t?


LineChart doesn't track mouse events if the terminal size increases

It sets the mouse state machine once with the initial canvas size and never updates the area.

Fixed directly in the devel branch.


🐛 Bug Report: SonarQube plugin does not compile

📜 Description
Updating to v1.11.0 breaks compilation in plugin-sonarqube-react
👍 Expected behavior
tsc compiles correctly
👎 Actual Behavior with Screenshots
yarn run v1.22.19
$ tsc --skipLibCheck false --incremental false
node_modules/@backstage/plugin-sonarqube/dist/index.d.ts:13:10 - error TS2305: Module '"@backstage/plugin-sonarqube-react"' has no exported member 'FindingSummary'.
13 import { FindingSummary } from '@backstage/plugin-sonarqube-react';
~~~~~~~~~~~~~~
node_modules/@backstage/plugin-sonarqube/dist/index.d.ts:16:10 - error TS2305: Module '"@backstage/plugin-sonarqube-react"' has no exported member 'SonarQubeApi'.
16 import { SonarQubeApi } from '@backstage/plugin-sonarqube-react';
~~~~~~~~~~~~
Found 2 errors in the same file, starting at: node_modules/@backstage/plugin-sonarqube/dist/index.d.ts:13
error Command failed with exit code 1.
👟 Reproduction steps
Update to backstage v1.11.0
Run tsc --skipLibCheck false --incremental false
📃 Provide the context for the Bug.

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

What's going on with the plugin? Checking the<EMAIL_ADDRESS>/* Excluded from this release type: FindingSummary */

/** @public */
export declare const isSonarQubeAvailable: (entity: Entity) => boolean;

/* Excluded from this release type: MetricKey */

/* Excluded from this release type: Metrics */

/** @public */
export declare const SONARQUBE_PROJECT_KEY_ANNOTATION = "sonarqube.org/project-key";

/* Excluded from this release type: SonarQubeApi */

/* Excluded from this release type: sonarQubeApiRef */

/* Excluded from this release type: SonarUrlProcessorFunc */

/* Excluded from this release type: useProjectInfo */

export { }

@drodil looks like these are labeled as @alpha.
https://github.com/backstage/backstage/blob/0870577bd4ac6922ff999a0207c2aed71512d499/plugins/sonarqube-react/src/hooks/useProjectInfo.ts#L22-L36
So they would need to be imported like this:
import { useProjectInfo, MetricKey } from '@backstage/plugin-sonarqube-react/alpha`

I think that this is a problem here as we don't have #15816 yet, and these imports are not being re-written in the main codebase.
@Rugvip is this an oversight or a bug, should this have worked?

@benjdlambert it worked before 1.11.0

Yep it's basically this commit: https://github.com/backstage/backstage/commit/6310eacc11 which is because it's using /alpha imports without actually using the /alpha path because it's not supported in the monorepo yet.

You should be able to roll-back the package.json bump for @backstage/plugin-sonarqube at least to get you unblocked from the bump.

OK, so to clarify a little bit, this is the issue what's happening here: https://github.com/backstage/backstage/pull/10128
It's currently not possible to re-export /alpha types in the current monorepo setup, and we have no linting or anything around this, just e2e tests, which unfortunately don't catch it for this plugin as it's not installed in the create-app bundle, so there's an action for us to maybe add better linting around this.
We have some options here, we either move some imports around and move them from /alpha and make sure that we're not re-exporting any /alpha types. Maybe it doesn't make sense to even have the /alpha imports for a package that isn't 1.x anways?
Or we revert the original PR, and create a new release, so that upgrading to 1.11 is easier as you don't have to lock the sonarqube dependency.
We haven't had time to test the https://github.com/backstage/backstage/pull/15816 subpath exports properly yet, so I think rushing that through is probably the last thing we want to do.

downgrading to @backstage/plugin-sonarqube 0.6.2 will resolve the issue for now (even still keeping @backstage/plugin-sonarqube-react 0.1.2)

Chatted with the other @backstage/maintainers, and I think we're going to opt for removing the /alpha types from the package for now. It's really tricky to fix right now until we have the subpath exports which we don't want to rush through as it's quite a lot of tooling changes and we want to test to make sure this works as expected first. I will create a PR to move these types now, and get the fix out today.

Closing as got released part of the bonus 1.11.1 release.


Diary: submitting a post that uses an indented bullet - Invalid content for node bulletList: <bulletList(listItem(paragraph("This works the oth

The string referenced here "This works the oth [...]" is an indented bullet point in a larger list of bullet points.
RangeError: Invalid content for node bulletList: <bulletList(listItem(paragraph("This works the oth
    at nh.check (https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:68:57649)
    at https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:68:57997
    at Qe.forEach (https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:68:44143)
    at nh.check (https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:68:57984)
    at https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:168:5580
    at Array.forEach (<anonymous>)
    at https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:168:5566
    at Object.insertContentAt (https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:165:8054)
    at https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:168:4392
    at Object.insertContent (https://sogrum-savluc.me/apps/groups/assets/index.e70a0985.js:165:7432)

Looks like a duplicate of #1598 but thank you for the additional context (indented bullet-point).


[Type checker] Fix ranking of _OptionalNilComparisonType overloads of ==/!=

Always prefer them to Optional's ==/!=, because they are @transparent and
more efficient.

@swift-ci please test and merge


Check for broken G-Cloud filters.

relates to https://github.com/alphagov/digitalmarketplace-buyer-frontend/pull/468

Is this still pertinent or has it been superseded by more recent changes now?

Will review in https://trello.com/c/5t96gz0Z/400-functional-tests


Submit package to Package Control

For ease of installation and updates, this package should be added to their channel.

Submitted pull request (https://github.com/wbond/package_control_channel/pull/6785). Awaiting update.

Merged


Issue running librespot via raspotify

Issue running librespot via raspotify
Mar 10 16:31:31 raspberrypi pulseaudio[5099]: [pulseaudio] server-lookup.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11
Mar 10 16:31:31 raspberrypi pulseaudio[5099]: [pulseaudio] main.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11
Mar 10 16:31:31 raspberrypi pulseaudio[5099]: [pulseaudio] bluez4-util.c: org.bluez.Manager.GetProperties() failed: org.freedesktop.DBus.Error.UnknownMethod: Method "GetProperties" with signature "" on interface "org.bluez.Manager" doesn't exist
Mar 10 16:34:08 raspberrypi librespot[1707]: INFO:librespot_playback::player: Loading track "White Shadows" with Spotify URI "spotify:track:0WWz2AaqxLoO0fa9ou6Fqc"
Mar 10 16:34:09 raspberrypi librespot[1707]: INFO:librespot_playback::player: Track "White Shadows" loaded
Mar 10 16:39:38 raspberrypi librespot[1707]: thread 'main' panicked at 'Box<Any>', core/src/session.rs:79:43
Mar 10 16:39:38 raspberrypi librespot[1707]: stack backtrace:
Mar 10 16:39:38 raspberrypi librespot[1707]: 0:   0x87e63f - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 1:   0x88889b - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 2:   0x876ec7 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 3:   0x8769a3 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 4:   0x877373 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 5:   0x588deb - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 6:   0x601d7f - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 7:   0x85861f - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 8:   0x856ecf - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 9:   0x85bbab - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 10:   0x472ec3 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 11:   0x463aa3 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 12:   0x473a13 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 13:   0x87701b - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 14:   0x89dc33 - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 15:   0x87e2cb - <unknown>
Mar 10 16:39:38 raspberrypi librespot[1707]: 16:   0x464583 - main
Mar 10 16:39:38 raspberrypi librespot[1707]: 17: 0x76d5c293 - __libc_start_main

Is there a version number in here somewhere?

INFO:librespot: librespot (raspotify) 8f8eb8c (2018-02-28). Built on 2018-03-07. Build ID: yb9b1Vkp

Don't know if that is librespot's version or Rapotify's

8f8eb8c is our git hash so that's what we need.

Thanks for the report. We have this issue on our radar, and we're tracking it here: https://github.com/librespot-org/librespot/issues/134. The crash you're seeing is due to your internet connection dropping out, and librespot not handling reconnection at the moment. Will close this in favour of tracking in #134


Enumerating all classes in the JDK

Sadly, RuntimeMXBean.getBootClassPath() is not supported in Temurin 17. Is there an alternative? I need to know the names of all classes in the current JDK. Thanks

Well... I hacked something together :o)
classNamesInLoadedJigsawModules


Custom generator as a service produces PHPStan error

Since Doctrine Bundle 2.3 we can pass a service id to the CustomIdGenerator annotation. However this causes Internal error: Internal error: Can't instantiate custom generator : app.doctrine.user_id_generator in file /app/src/Entity/User.php.
BTW. Can this even be fixed? It would require phpstan/phpstan-doctrine to have access to the Symfony's DI container.

we can pass a service id to the CustomIdGenerator annotation

How does an annotation like that look like? I need something to reproduce that.
Also you might solve that by providing objectManagerLoader: https://github.com/phpstan/phpstan-doctrine#configuration

* @ORM\CustomIdGenerator("<service_id_here>")
Will take a look at this objectManagerLoader feature.

I just ran into the same issue, but there seems to be a solution that is cleaner anyway.
This will trigger the error:
#[ORM\CustomIdGenerator(class: 'doctrine.uuid_generator')]
...and this will not:
use Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator;
[...]
#[ORM\CustomIdGenerator(class: UuidGenerator::class)]

objectManagerLoader (see README) will AFAIK solve it.

I know, I'm just pointing out that this would be a solution and, let's be honest, much cleaner code than using magic strings.

I just ran into the same issue, but there seems to be a solution that is cleaner anyway.
This will trigger the error: #[ORM\CustomIdGenerator(class: 'doctrine.uuid_generator')]
...and this will not:
use Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator;
[...]
#[ORM\CustomIdGenerator(class: UuidGenerator::class)]

Thanks this worked!
use Symfony\Component\Uid\Uuid;
use Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator;

    /**
     * @ORM\Id
     * @ORM\Column(type="uuid", unique=true)
     * @ORM\GeneratedValue(strategy="CUSTOM")
     * @ORM\CustomIdGenerator(class=UuidGenerator::class)
     */
    private Uuid $uuid;

As @nicolas-grekas pointed out in https://github.com/symfony/symfony-docs/pull/18427#pullrequestreview-1486527166, UuidGenerator::class is currently not a service id.
In many cases, UuidGenerator::class will work as expected, but one should be aware, that, in fact, in regard of the property, it replaces the doctrine.uuid_generator service by a new instance, created by doctrine ORM itself. The service might use a custom uuid factory, while the instance, created by doctrine for the particular property, will always use the default factory.
If you don't want to change your application code, to solve the Can't instantiate custom generator issue, and want to keep using the service, I can confirm that using the objectManagerLoader feature, as suggested by @ondrejmirtes, works:
# phpstan.neon
parameters:
	doctrine:
		objectManagerLoader: tests/object-manager.php # needs to be created according to README.md

Maybe the README.md could be extended by something like:

If you are using this extension in a Symfony project with Doctrine bundle and you are using a CustomIdGenerator, you might want to set up the objectManagerLoader, to allow service ids to be resolved.

Currently it sounds as the objectManagerLoader is relevant for "more advanced analysis" and "DQL validation" only. What do you think?
After all, this seems not to be an issue with phpstan-doctrine to me. Instead the issue seems to be caused by Doctrine bundle, by intentionally using the class attribute of the ORM's CustomIdGenerator attribute/annotation in a way which is not supported/expected by ORM. Doctrine ORM expects "name of the class…", while the Doctrine Bundle allows a service id to be used. This seems to cause issues for tools like phpstan-doctrine, which try to gather information about entities without Doctrine bundle. Maybe I will raise this topic in doctrine/DoctrineBundle, after further research.


Shrinkwrap is unstable

Current coverage is 100% (diff: 100%)

No coverage report found for develop at 02d9c8c.

Powered by Codecov. Last update 02d9c8c...fb82d50


issue Profiler Argument

Hello,
After installing SoclozMonitoringBundle, I encountered this error:
ContextErrorException: Catchable Fatal Error: Argument 2 passed to Socloz\MonitoringBundle\Listener\Profiler::__construct() must be an instance of Socloz\MonitoringBundle\Notify\StatsD\StatsDInterface, null given, called in /project/app/cache/dev/appDevDebugProjectContainer.php on line 5127 and defined in /project/vendor/socloz/monitoring-bundle/Listener/Profiler.php line 58
app/config/config/yml
socloz_monitoring:
    exceptions:
        enable: true
    profiler:
        enable: true
        request: true
    mailer:
        enable: true
        from<EMAIL_ADDRESS>        to<EMAIL_ADDRESS>    statsd:
        enable: false
        host: nowhere
        port: 42
        merge_packets: true

Did I forget something ?
I would like to know execution time of my rest api in production. And other stats of functions.
Thanks in advance.
Tanguy

Hi @tanguybernard could you try to replace (https://github.com/SoCloz/SoclozMonitoringBundle/blob/master/Listener/Profiler.php#L58):
public function __construct(Xhprof $profiler, StatsDInterface $statsd, Logger $logger = null, $sampling = 100)

by:
public function __construct(Xhprof $profiler, StatsDInterface $statsd = null, Logger $logger = null, $sampling = 100)

If it's solve your problem could you do a pull request.

Thanks @mhor !
It's solved my problem.
One more question :) I expected a new element in my profiler symfony with this configuration:
profiler:
    enable: true
    sampling: 50
    request: true

But nothing appears, did I miss something ?


os: corrupted error from Open on plan9-arm

--- FAIL: TestTransform (4.49s)
    transform_test.go:96: unexpected error from Transform: open /tmp/TestTransform2759707054/blob.bin: '/tmp/TestTransform2759707054/blob.bin'
FAIL
FAIL	cmd/go/internal/lockedfile	5.123s

The error text here looks truncated. @millerresearch, is this the same sort of corruption you were referring to in https://github.com/golang/go/issues/50009#issuecomment-988276192?

greplogs --dashboard -md -l -e '\Aplan9-arm.*(?:.*\n)*.* open [^:]+: [^ ]+\s*$'
2022-01-10T17:54:35-933f668/plan9-arm

The error text here looks truncated. @millerresearch, is this the same sort of corruption you were referring to in #50009 (comment)?

I think so. As far as I can tell, every Plan 9 filesystem error message should have some explanatory text, not just a bare filename. And there's a space at the end of the line, suggesting there should have been more.

Looks like these are happening roughly once a month — at least the ones that match the open regexp above.
greplogs -l -e '\Aplan9-arm.*(?:.*\n)*.* open [^:]+: [^ ]+\s*$' --since=2022-01-11
2022-04-06T20:46:47-81ae993/plan9-arm
2022-04-04T21:06:49-5c4ed73/plan9-arm
2022-03-02T17:03:04-4a13f6f/plan9-arm
2022-02-21T21:28:40-c9fe126/plan9-arm


'Screen' does not contain a definition for 'width'

Plugin Version or Commit ID
v0.11.0
Unity Version
2021.3.18f1
Your Host OS
Windows10
Target Platform
Windows Standalone
Target Device
No response
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
No response
[iOS Only] XCode Version
No response
Build Command
MediaPipeUnityPlugin-all
Bug Description
'Screen' does not contain a definition for 'width'
Steps to Reproduce the Bug

insert int width = Screen.width; statement into FaceMeshSolution.sc
run FaceMeshSolution.sc

Log
Assets\MediaPipeUnity\Samples\Scenes\Face Mesh\FaceMeshSolution.cs(28,24): error CS0117: 'Screen' does not contain a definition for 'width'
Screenshot/Video
No response
Additional Context
No response

self-solving


total number of mask identities

Hi, great work! How to get the total number of mask identities? In your train.json file, you pre-define the num_classes as 256. The num_classes should vary over different 3D scenes, right? Why you define it as a fixed number?

Hi, thanks for watching our work. 256 is the max number of identities here. We set it to 256 since it can be easily stored in one gray image with 8 bits per pixel. Usually, a scene has less than 256 identities. If a scene has more than 256 we can also switch to a 16-bit gray image.

Thank you so much.

Hello, my scene is more complex, and the number of objects that need to be encoded more than 256. So, as you mentioned above,  I am trying to switch to 16-bit gray image. So can you tell me what needs to be modified in the code when switching to a 16-bit gray image?


Source generator - AddHeaderRow from property names

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (4d4581c) 87.76% compared to head (5210ada) 87.76%.

Files
Patch %
Lines

...adCheetah.SourceGenerator/WorksheetRowGenerator.cs
92.30%
1 Missing and 5 partials :warning:

...eetah/SourceGeneration/EmptyWorksheetRowContext.cs
84.21%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   87.76%   87.76%   -0.01%
==========================================
  Files         127      129       +2
  Lines        4513     4561      +48
  Branches      878      879       +1
==========================================
+ Hits         3961     4003      +42
- Misses        247      250       +3
- Partials      305      308       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


cant upload

:+1:

No upload button when right clicking in Project tree view (context menu).

Also, when I save a file, it generates matching css.
But even if the css folder and its content is watched, those files are not uploaded =/

upload & download links are missing in tree view. Please fix asap makes the tool unusable.

I've the same issue here...

I'm running into the missing upload option as well

I´m using filezilla again.
On Thu, May 18, 2017 at 11:58 AM, Michael Mason<EMAIL_ADDRESS>wrote:

I'm running into the missing upload option as well
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mgrenier/remote-ftp/issues/797#issuecomment-302470925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWIMbt14eeZB2cykldU4MD01dntn0fh3ks5r7HjNgaJpZM4Ndh6Y
.

#798 Check this answer.