Don't work with Expanded Storage?

Hello there!
I can't get it to work with the Expanded Storage mod.
It works perfect with regular vanilla chests but not the modded ones. I have enabled large chests in the config aswell.
In the log i found:
"EasierChests did not find mod Expanded Storage, not enabling support"
I tried to troubleshoot it myself but i dont know much about code, tho my suspicion is the recent renaming of the depency mod Expanded Storage require.
Ninjaphenix's Container Library have changed name to Ellemes' Container Library.
And i saw that your mod makes som api call(i think?) to something called like "ninjaphenix.container_library.api".
And i believe i saw that there was some api renaming on the Elleme's Container Library github page.
That is my amateur analysis
But as i said, i can't write or understand most of code and can be very wrong :)
Client Verisions:

Minecraft 1.18.2
Forge 40.1.31

Mods Verisions:

easierchests-1.0.2.jar
ellemes-container-library-1.4.0-beta.2+1.18.2-forge.jar
expandedstorage-7.4.0-beta.3+1.18.2-forge.jar

I am currently not home so i can't provide my log, but tell me if you need it and i will post it as soon as i get home.
Thanks in Advance
/R

I have just looked at the API and it seems like it's the same in 1.18.2 as it was before. the path hasn't changed. I will however take a deeper look to see why the mod fails to find it

Then my troubleshooting showed just how little i know about code, as i said haha! :)
I was thinking if it can have something to do with the order they are loaded in for som reason?
Because i noticed that the "not enabling support" line was pretty early in the log in comparision for expanded storage, it was mentioned for the first time waaay down in the log.
The log if it helps:
https://paste.ee/p/QavsJ
And btw, i REALLY REALLY appreciate that you are taking a look at it and responded so fast, and if i can help somehow just let me know.
This is easily my favorite mod of them all and when i saw that someone ported it to forge i was super happy to be able to use it even on my create server!

And btw, i REALLY REALLY appreciate that you are taking a look at it and responded so fast, and if i can help somehow just let me know.
easier to procrastinate on my finals ...

i'm trying to make modding a wel paying hobby, end-user satisfaction is a high priority for me.
I found out then while porting, the compat got commented out and dint make it back in.
currently talking with Ellemes to make it work again, given they did indeed change their API around a bunch

Oh! Does that mean that you have located the problem? :)
Looking forward to the solution! (No rush tho! Take your time!)
Love the mindset and priority from you!
When i browse through looking for new mods or updates i have noticed to many people who create mods that have little to no documentation/descriptions/wiki/screenshots for their creation, long response times/if any at all.
And some even have a really bad attitude when they bother to answer people. I get that it´s annoying to see 300 different "1.11.4 Please" e.g in the comments and so on, but come on :P

To show my appreciation and as a thanks, i did the only thing i can help with, and that is my language!

nice. is that giselbaer's translation website ?
i'll be releasing a beta for testing purposes. I don't have time right now to test it toroughly . changes have been made, it should work now with the beta latest release of the container lib. the other mods havent been added yet, but expanded storage should work.
i hope to hear from you again soon ! just gotta wait five minutes for the file to be accepted on curseforge and go trough mdoeration.

Oh, yeah i guess its his, i just clicked the link in the top of the description, didn´t think about that it´s a copy from the other one :) I am not really 100% on how stuff on github works with forks/pull request etc, it´s a jungle for a novice :)
Oh really? That was fast! :D
Thank you!! I will try it out right away!
I really only care about Expanded Storage for now so the beta is all i need hehe.

Tried it out now and it works like a charm! My life is now complete! Haha.
Just to mention, the arrows for the row underneath the inventory ends upp somewhere in middle of the inventory, easy just to disable in config so i will do that, but then you know atlest :)
Once again, THANK YOU!


Reinstate loadPlan multiple execution protection.

This addresses an issue that sometimes loadPlan is actually executed
way too often.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11142/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11258/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11259/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11273/

I have tested this with the arangosync test both in a single cluster and Vadim's load and with arangosync in two local clusters. It improves performance dramatically. I have not found any further problems.

Tests blue.


Cleanup lvgl_helpers

Cleanup of lvgl_helpers and fixes compilation errors.

Project builds successfully in my pc with ESP-IDF v4.4-dev-2740-gf65c8249af and ESP32 as target.
@tore-espressif Any idea on why it might be failing?

yep, there were some changes in esp_rom_*.h headers in IDF4.4. I'll fix it today or tomorrow

I can try to check your commits and look for the fixes.

well, I have these errors and warnings:
../components/lvgl_esp32_drivers/lvgl_helpers.c:330:13: warning: 'init_ft81x' defined but not used [-Wunused-function]
 static void init_ft81x(int dma_channel)
             ^~~~~~~~~~

../components/lvgl_esp32_drivers/lvgl_tft/ra8875.c: In function 'ra8875_init':
../components/lvgl_esp32_drivers/lvgl_tft/ra8875.c:186:5: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
     gpio_pad_select_gpio(RA8875_RST);
     ^~~~~~~~~~~~~~~~~~~~
     esp_rom_gpio_pad_select_gpio

../components/lvgl_esp32_drivers/lvgl_tft/GC9A01.c: In function 'GC9A01_init':
../components/lvgl_esp32_drivers/lvgl_tft/GC9A01.c:114:5: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
     gpio_pad_select_gpio(GC9A01_DC);
     ^~~~~~~~~~~~~~~~~~~~
     esp_rom_gpio_pad_select_gpio

I guess gpio_pad_select_gpio(GC9A01_RST) from lvgl_tft/GC9A01.c, gpio_pad_select_gpio(ignore) and gpio_pad_select_gpio(measure) from lvgl_touch/adcraw.c will also invoke these problems.

Single warning left during compiling, due to 'init_ft81x' defined but not used. But SPI configuration fails in my esp32c3:
I (327) lvgl_helpers: Initializing SPI master
I (347) lvgl_helpers: Configuring SPI host SPI1_HOST
I (347) lvgl_helpers: MISO pin: -1, MOSI pin: 1, SCLK pin: 2, IO2/WP pin: -1, IO3/HD pin: -1
I (347) lvgl_helpers: Max transfer size: 19200 (bytes)
I (367) lvgl_helpers: Initializing SPI bus...
E (367) spi: spi_bus_initialize(756): SPI bus already initialized.

assert failed: lvgl_spi_driver_init lvgl_helpers.c:301 (ret == ESP_OK)

It tries to use SPI1_HOST but I've set SPI2_HOST in sdkconfig: CONFIG_LV_TFT_DISPLAY_SPI2_HOST=y.
SPI bus already initialized message looks kind of weird to me or I've just forgotten how it should look like?

Thanks for the update, can you please attach the sdkconfig.h file?

Content of my ./build/config/sdkconfig.h

Hi all,
I couldn't follow all changes you have been doing recently, but I hope you find this useful:
@C47D This commit https://github.com/lvgl/lvgl_esp32_drivers/commit/5043946699f3e34d0f90d3fa9d31aee6ae081485 is fixing esp_lcd_backlight compilation issue (it is from august 2021), but apparently @arktrin is having a different problem.
This commit https://github.com/lvgl/lvgl_esp32_drivers/commit/8c7bc4214012b4b08854d07751a47836c5738d99 breaks compatibility  with IDF v4.1 and 4.2. The esp_rom_gpio.h header file is included from v4.3 onwards
@arktrin All your gpio missing declarations (gpio_pad_select_gpio, gpio_matrix_out) are included from driver/gpio.h. This works on all IDF version and all targets. So make sure all compilation units that use these function have driver/gpio.h included

@tore-espressif Not sure I understand you correctly. As you can see all of my errors associated with gpio_pad_select_gpio() function are the part of lvgl_esp32_drivers itself. Where should I include driver/gpio.h ?

I see, I thought that you are authoring the branch :)
So the branch is probably broken and misses the driver/gpio.h includes.

@tore-espressif Would you so kind to fix feat/new_driver_test branch of lv_port_esp32 project ? :)

Single warning left during compiling due to 'init_ft81x' defined but not used. But SPI configuration fails in my esp32c3:
I (327) lvgl_helpers: Initializing SPI master
I (347) lvgl_helpers: Configuring SPI host SPI1_HOST
I (347) lvgl_helpers: MISO pin: -1, MOSI pin: 1, SCLK pin: 2, IO2/WP pin: -1, IO3/HD pin: -1
I (347) lvgl_helpers: Max transfer size: 19200 (bytes)
I (367) lvgl_helpers: Initializing SPI bus...
E (367) spi: spi_bus_initialize(756): SPI bus already initialized.

assert failed: lvgl_spi_driver_init lvgl_helpers.c:301 (ret == ESP_OK)

It tries to use SPI1_HOST but I've set SPI2_HOST in sdkconfig: CONFIG_LV_TFT_DISPLAY_SPI2_HOST=y. SPI bus already initialized message looks kind of weird to me or I've just forgotten how it should look like?

Can you print the value of TFT_SPI_HOST? Based on your file the selected bus is SPI2, as you configured:

So the TFT_SPI_HOST should be set to SPI2_HOST

As a patch, can you add a return after this lines:
    lvgl_spi_driver_init(TFT_SPI_HOST, miso, DISP_SPI_MOSI, DISP_SPI_CLK,
        spi_max_transfer_size, dma_channel, DISP_SPI_IO2, DISP_SPI_IO3);

    disp_spi_add_device(TFT_SPI_HOST);

As a patch, can you add a return after this lines:
    lvgl_spi_driver_init(TFT_SPI_HOST, miso, DISP_SPI_MOSI, DISP_SPI_CLK,
        spi_max_transfer_size, dma_channel, DISP_SPI_IO2, DISP_SPI_IO3);

    disp_spi_add_device(TFT_SPI_HOST);

I have found these lines in lvgl_helpers.c, but I'm not sure what do mean by "add a return after these lines". Should I put return; after these lines ? Putting return; do not help. I am still getting the same SPI configuration error as before.

Were you able to print TFT_SPI_HOST? I don't remember if it's possible to increase the ESP logging level, but I didn't saw some logging information on your last message

Do this part have all corresponding endifs for proper working ?
/* Display controller initialization */
#if defined (CONFIG_LV_TFT_DISPLAY_PROTOCOL_SPI) || defined (SHARED_SPI_BUS)
    ESP_LOGI(TAG, "Initializing SPI master");

    int miso = DISP_SPI_MISO;
    int spi_max_transfer_size = calculate_spi_max_transfer_size(display_buffer_size);

    /* Set the miso signal to be the selected for the touch driver */
#if defined (SHARED_SPI_BUS)
    miso = TP_SPI_MISO;
#endif

    lvgl_spi_driver_init(TFT_SPI_HOST, miso, DISP_SPI_MOSI, DISP_SPI_CLK,
        spi_max_transfer_size, dma_channel, DISP_SPI_IO2, DISP_SPI_IO3);

    disp_spi_add_device(TFT_SPI_HOST);

    ESP_LOGI(TAG, "TFT_SPI_HOST is %d", TFT_SPI_HOST);

    /* Add device for touch driver */
#if defined (SHARED_SPI_BUS)
    tp_spi_add_device(TOUCH_SPI_HOST);
    touch_driver_init();

    return;
#endif

#elif defined (CONFIG_LV_I2C_DISPLAY)
#else
#error "No protocol defined for display controller"
#endif

Do this part have all corresponding endifs for proper working ?
/* Display controller initialization */
#if defined (CONFIG_LV_TFT_DISPLAY_PROTOCOL_SPI) || defined (SHARED_SPI_BUS)
    ESP_LOGI(TAG, "Initializing SPI master");

    int miso = DISP_SPI_MISO;
    int spi_max_transfer_size = calculate_spi_max_transfer_size(display_buffer_size);

    /* Set the miso signal to be the selected for the touch driver */
#if defined (SHARED_SPI_BUS)
    miso = TP_SPI_MISO;
#endif

    lvgl_spi_driver_init(TFT_SPI_HOST, miso, DISP_SPI_MOSI, DISP_SPI_CLK,
        spi_max_transfer_size, dma_channel, DISP_SPI_IO2, DISP_SPI_IO3);

    disp_spi_add_device(TFT_SPI_HOST);

    ESP_LOGI(TAG, "TFT_SPI_HOST is %d", TFT_SPI_HOST);

    /* Add device for touch driver */
#if defined (SHARED_SPI_BUS)
    tp_spi_add_device(TOUCH_SPI_HOST);
    touch_driver_init();

    return;
#endif

#elif defined (CONFIG_LV_I2C_DISPLAY)
#else
#error "No protocol defined for display controller"
#endif

Yes, it does.

Set this:
/* Display controller initialization */
#if defined (CONFIG_LV_TFT_DISPLAY_PROTOCOL_SPI) || defined (SHARED_SPI_BUS)
    ESP_LOGI(TAG, "Initializing SPI master");
    ESP_LOGI(TAG, "1.TFT_SPI_HOST is %d", TFT_SPI_HOST);
    int miso = DISP_SPI_MISO;
    int spi_max_transfer_size = calculate_spi_max_transfer_size(display_buffer_size);

    /* Set the miso signal to be the selected for the touch driver */
#if defined (SHARED_SPI_BUS)
    miso = TP_SPI_MISO;
#endif
    ESP_LOGI(TAG, "2.TFT_SPI_HOST is %d", TFT_SPI_HOST);
    lvgl_spi_driver_init(TFT_SPI_HOST, miso, DISP_SPI_MOSI, DISP_SPI_CLK,
        spi_max_transfer_size, dma_channel, DISP_SPI_IO2, DISP_SPI_IO3);
    ESP_LOGI(TAG, "3.TFT_SPI_HOST is %d", TFT_SPI_HOST);
    disp_spi_add_device(TFT_SPI_HOST);
    ESP_LOGI(TAG, "4.TFT_SPI_HOST is %d", TFT_SPI_HOST);

Got this:
I (332) lvgl_helpers: Initializing SPI master
I (352) lvgl_helpers: 1.TFT_SPI_HOST is 0
I (352) lvgl_helpers: 2.TFT_SPI_HOST is 0
I (352) lvgl_helpers: Configuring SPI host SPI1_HOST
I (352) lvgl_helpers: MISO pin: -1, MOSI pin: 1, SCLK pin: 2, IO2/WP pin: -1, IO3/HD pin: -1
I (372) lvgl_helpers: Max transfer size: 19200 (bytes)
I (372) lvgl_helpers: Initializing SPI bus...
E (372) spi: spi_bus_initialize(756): SPI bus already initialized.

assert failed: lvgl_spi_driver_init lvgl_helpers.c:304 (ret == ESP_OK)

One question, is it possible to have multiple versions of ESP-IDF in the same environment? I'm working with WSL (on Windows 10) and only have ESP-IDF v4 installed. I'm wondering if cloning other versions, and exporting one at the time, would allow me to have "multiple" versions of ESP-IDF.

Not sure about WSL particularly but in linux you can have different ESP-IDFs and export different ESP-IDF version in different session. Current session will use the IDF_PATH that was exported in it.

Yep, TFT_SPI_HOST is not being set correctly, is set to 0.

With latest ff7b380 and 9b66665 commits I'm getting gpio_pad_select_gpio and gpio_matrix_out related stuff again :(
../components/lvgl_esp32_drivers/lvgl_tft/esp_lcd_backlight.c: In function 'disp_backlight_new':
../components/lvgl_esp32_drivers/lvgl_tft/esp_lcd_backlight.c:59:9: error: implicit declaration of function 'gpio_matrix_out'; did you mean 'gpio_iomux_out'? [-Werror=implicit-function-declaration]
         gpio_matrix_out(config->gpio_num, ledc_periph_signal[LEDC_LOW_SPEED_MODE].sig_out0_idx + config->channel_idx, config->output_invert, 0);
         ^~~~~~~~~~~~~~~
         gpio_iomux_out
../components/lvgl_esp32_drivers/lvgl_tft/esp_lcd_backlight.c:65:9: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
         gpio_pad_select_gpio(config->gpio_num);
         ^~~~~~~~~~~~~~~~~~~~
         esp_rom_gpio_pad_select_gpio
../components/lvgl_esp32_drivers/lvgl_tft/esp_lcd_backlight.c:67:43: error: 'SIG_GPIO_OUT_IDX' undeclared (first use in this function); did you mean 'GPIO_NUM_MAX'?
         gpio_matrix_out(config->gpio_num, SIG_GPIO_OUT_IDX, config->output_invert, false);
                                           ^~~~~~~~~~~~~~~~
                                           GPIO_NUM_MAX
../components/lvgl_esp32_drivers/lvgl_tft/esp_lcd_backlight.c:67:43: note: each undeclared identifier is reported only once for each function it appears in

../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c: In function 'FT81x_init':
../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c:266:2: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
  gpio_pad_select_gpio(EVE_PDN);
  ^~~~~~~~~~~~~~~~~~~~
  esp_rom_gpio_pad_select_gpio

I have been able to replicate the TFT_SPI_BUS issue, it only appears when flashing the program. Working on it.

With latest ff7b380 and 9b66665 commits I'm getting gpio_pad_select_gpio and gpio_matrix_out related stuff again :(

I'wlll push a very hacky patch.

with latest 2f70856 and 7722f17 :
../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c: In function 'FT81x_init':
../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c:266:2: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
  gpio_pad_select_gpio(EVE_PDN);
  ^~~~~~~~~~~~~~~~~~~~
  esp_rom_gpio_pad_select_gpio

There are no errors with the latest release/v4.4 branch of the esp-idf. I've hardcoded TFT_SPI_HOST to work with SPI2_HOST, everything looks fine but i have nothing on the screen :(

Hi @arktrin , sorry for the late reply, I was offline all weekend.

There are no errors with the latest release/v4.4 branch of the esp-idf. I've hardcoded TFT_SPI_HOST to work with SPI2_HOST, everything looks fine but i have nothing on the screen :(

I see the same issue, also tested develop (without this PR) and still have the same issue. I will try to bisect it later today.

with latest 2f70856 and 7722f17 :
../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c: In function 'FT81x_init':
../components/lvgl_esp32_drivers/lvgl_tft/FT81x.c:266:2: error: implicit declaration of function 'gpio_pad_select_gpio'; did you mean 'esp_rom_gpio_pad_select_gpio'? [-Werror=implicit-function-declaration]
  gpio_pad_select_gpio(EVE_PDN);
  ^~~~~~~~~~~~~~~~~~~~
  esp_rom_gpio_pad_select_gpio

EDIT This is true for the latest master branch of the esp-idf.

Thanks for pointing out the compilation error is present on the master branch, I don't think we added it into the CI.

One question, is it possible to have multiple versions of ESP-IDF in the same environment? I'm working with WSL (on Windows 10) and only have ESP-IDF v4 installed. I'm wondering if cloning other versions, and exporting one at the time, would allow me to have "multiple" versions of ESP-IDF.

@C47D
You should have only one IDF version in single environment. But you can have multiple environments.
I'm on Windows 10 and work with Windows Terminal. I have multiple versions of idf cloned so my folder tree look like this:
C:/esp_repos
├───esp-idf-v4.1
├───esp-idf-v4.2
├───esp-idf-v4.3
├───esp-idf-v4.4
├───esp-idf-master

The windows terminal JSON file contains (among other things):
            {
                "altGrAliasing": true,
                "antialiasingMode": "grayscale",
                "closeOnExit": "graceful",
                "colorScheme": "Campbell",
                "commandline": "powershell.exe -NoExit C:\\esp_repos\\esp-idf-v4.2\\export.ps1",
                "cursorShape": "bar",
                "font":
                {
                    "face": "Cascadia Mono",
                    "size": 12
                },
                "guid": "{d6bd6b46-f392-494d-bf17-e9c670cc8e12}",
                "historySize": 9001,
                "name": "IDF v4.2",
                "padding": "8, 8, 8, 8",
                "snapOnInput": true,
                "startingDirectory": "C:\\esp_repos\\",
                "suppressApplicationTitle": true,
                "tabTitle": "IDF v4.2",
                "useAcrylic": false
            },

            ... and so on...

The shell uses Powershell, but it should be easy to use WSL too :)

There are no errors with the latest release/v4.4 branch of the esp-idf. I've hardcoded TFT_SPI_HOST to work with SPI2_HOST, everything looks fine but i have nothing on my st7789 display :(

I've just pushed a fix to the lv_port_esp32 repo.

@tore-espressif Thanks for the explanation, I'm working from a borrowed pc, will try to setup multiple environments new week, I will also try to get github actions running locally.

TFT_SPI_HOST issue seems to be fixed, but I'm still have nothing on my st7789 display :(

I've just pushed a fix to the lv_port_esp32 repo. The disp_drv registration was commented out

Can't see any new commits in lv_port_esp32 repo.

@arktrin thanks for letting me know, I wasn't pushing to a public remote. Now you should be able to see the fix in the feat/new_driver_test branch.

Now you should be able to see the fix in the feat/new_driver_test branch.
No warning at all! And finally now I see my app on the screen! Yay! This is definitely may be merged from my perspective.

I will add a resume so @tore-espressif can review it as well. Thanks a lot for testing it @arktrin , it took a lot longer than I was expecting it.

The open topics are not implemented in this PR, they are meant to be done later.

As you might have already noticed, I'm not a big fan of huge #ifdefs 🗡  :D
But apart from that, if we want to make the drivers platform agnostic we will have to deal with them, because not all platforms use Kconfig.

As you might have already noticed, I'm not a big fan of huge #ifdefs 🗡 :D
But apart from that, if we want to make the drivers platform agnostic we will have to deal with them, because not all platforms use Kconfig.

Yes, me neither, but I think cleaning up the existing code so later it can be refactored later is a good first step. I will address a couple of comments with leftover code, and wait for your reply on all the others. Thanks for taking the time to review this :)
I will also integrate the latest develop changes 😄

Hi @tore-espressif, I did the requested changes. Is it OK if we merge this?

Version checks removed and the CI is now passing @tore-espressif


HDDS-5364. EC: Adopt EC related utility from Hadoop source repository

What changes were proposed in this pull request?
It seems only a few key classes are required from Apache Hadoop to be reused. The goal here is investigating the build process and check how they can be re-used.
One option is just copy the files with some minor updates (e.g. using ConfigurationSource interface instead of o.a.hadoop.Configuration) which would help to reduce dependencies.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5364
How was this patch tested?
Full CI

@sodonnel @umamaheswararao Can you please review it?

Thanks @elek for the patch. Sorry for the late on this. I am thinking that, we should copy the code as it is in JIRA and then modify in next JIRA? SInce it's large patch it may be hard to follow the changes against hadoop code here.
Thoughts?
CC: @sodonnel @fapifta

Thanks the answer @umamaheswararao
However, I am not sure how is it possible. Just copying the code wouldn't compile. We have different dependency structure, package names and configuration objects.
But here I didn't change any logic, just copied the code and updated the used configuration interfaces. So technicaly this patch is the copy without any significant change.

I agree there is no logic change. Only worry is about the patch size and it's easy to miss things. For example, above conflict changes are not necessarily in this patch I guess. They are about using the new EcReplicationConfig with codec option. Currently they are already have default values and we can easily use this patch ECReplicationConfig in next patch. That way we can reduce modifications in this patch.

@elek did you get chance to check my comments? Thanks


Transforms internal links in Link components

Summary
gatsby-plugin-mdx doesn't seem to transform internal markdown links into Gatsby <Link /> components resulting in page being refreshed.
 The first link, while internal, doesn't use Gatsby <Link />.

After
The first link is internal and is rightfully rendered using Gatsby <Link />. Clicking on the link doesn't trigger a page reload.

There's actually a plugin for this: https://www.gatsbyjs.com/plugins/gatsby-plugin-catch-links/ so I'm closing.
This can either be part of the theme, or left to the user to install.

Hello @dbismut, I think we should add the plugin built-in.

@dbismut do you see any case when it is better to have raw link instead of Gatsby link ?

@gregberge Gatsby links should only be used for internal link navigation. Gatsby Link doesn't handle external links or anchor links. Nor download links! So I guess proper check should be made. The plugin is actually way more complete than my naive implementation, as it checks for the location host... It also works across the whole site and not just MDX.
I can of course upgrade this PR but I might as well open another one with the updated package.json / gatsby-config.js files if you'd like.

I can of course upgrade this PR but I might as well open another one with the updated package.json / gatsby-config.js files if you'd like.

Yeah if you could add the plugin in another PR it would be awesome! Thanks!

Fixed in #28


Query dataframe

add query_dataframe: call of a function with a string (which is the own query) and get dataframe with datetimeindex back as response

ok, now it is your task @karlTGA (may be , idea was to simple and there are a few hacks)

feat(main): add query dataframe


Use 01-index by default

According to https://www.well-typed.com/blog/2015/08/hackage-security-beta/
01-index.tar should be used instead of the old 01-index.tar.

Looks like I'm using an old cabal config/cache. Closing.


support writing unicode strings

Hello, I have a problem with the letters

Ə
Ğ
İ

did u check the fonts? on my environment it show as ?


Fixed content not rendered on save.

Fix for issue https://github.com/FlusherDock1/Gutenberg/issues/25.

is it possible to build this merge?

Your solution is not correct. Here is the fix #30


Add event when blog decrypted

PR
Issue Fixed #
Proposed Changes
在文章成功解密之后，抛出自定义事件，以支持主题内部监听解密状态，进行初始化页面等操作。
After successfully decrypting the article, a custom event is triggered to support internal theme listeners for initializing the page and other operations.
https://github.com/f-dong/hexo-theme-minimalism/issues/21

Nice job!


Improve keyboard support

Some form components have no or poor keyboard support. They cannot be focussed by using the keyboard and/or their values cannot be changed with the keyboard only.
Those are some components which have a lacking keyboard support:

checkbox
radio
select
spinner
switch
calendar
calendar picker
date picker
time picker
dialogs
info box

In addition, the focus styling of certain components should be improved as it is barely visible (for example in the slider component or <button class="button link">).
Reference: https://github.com/troxler/awesome-css-frameworks/pull/30

If you need additional component functionality, you can create one issue for each component with a detailed description of component behavior, user interactions or style changes.

I'm not involved in this project and I don't plan to do so. So I won't create individual issues as my only intention was to mention that currently this framework is a little lacking in that area.
There are already (pseudo) standards and proven patterns for many different widgets. You can find some here: https://www.w3.org/TR/wai-aria-practices/#aria_ex
For example, the checkbox:

When the checkbox has focus, pressing the Space key changes the state of the checkbox.

In Metro 4 chackbox, radio, switch can change state when user press Space

Oh, yes you are right. But I do not see which widget is currently focussed which makes it hard to actually use the keyboard.

Thank you, I will review all input elements and add an attribute that, when activated (example: data-focus="true"), will display the outline with focus.

Is the issue finally solved?


Use FaceAware cause Memory to Large

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
11.0

SDWebImage Version
4.2.0

Integration Method
cocoapods

Xcode Version
Xcode 9

Repro rate
100%

Repro with our demo prj
NO

Demo project link
I will try to create a Project later.

Issue Description and Steps
I use the Third-party libraries

FaceAware  ( https://github.com/BeauNouvelle/FaceAware )

when the ImageView set image Completed,  I Use FaceAware to deal with ImageView like The Code:
[self sd_setImageWithURL:[NSURL URLWithString:url]
        placeholderImage:nil
                 options:SDWebImageScaleDownLargeImages
               completed:^(UIImage * _Nullable image, NSError * _Nullable error, SDImageCacheType cacheType, NSURL * _Nullable imageURL) {
    weakself.focusOnFaces = YES;
    weakself.debugFaceAware = YES;
}];

When I scroll TableView, The memory growth too large,   Maybe it cause A circular reference Or Something not destruction, How can I release the  Memory?
Thanks !

@CocoaML thanks for reporting back to us.
First of all, I have re-added the issue template info you have removed from the issue description. We need you to fill in the info in this table to we can better understand the conditions of your issue.
Secondly, you are using another 3rd party code, so I think the best thing to do is just use Xcode profiler to see what is causing the memory grow.
One idea is to disable the FaceAware code and see if the issue still reproduces.
I see you have set SDWebImageScaleDownLargeImages. If you are dealing with large images, that might cause the memory growth.
You can also try disabling the decompression and see if that takes care of the mem issue
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];
[[[SDImageCache sharedImageCache] config] setShouldDecompressImages:NO];

New Issue Checklist

[x]  I have read and understood the CONTRIBUTING guide
[x]  I have read the Documentation
[x]  I have searched for a similar issue in the project and found none

Issue Info
Info	Value
Platform Name	 iOS
Platform Version	11.0
SDWebImage Version	. 4.2.0
Integration Method	 cocoapods
Xcode Version	 Xcode 9
Repro rate	e.g. all the time (100%)
Repro with our demo prj	 does it happen with our demo project? NO.
Demo project link	I will try to create a Project later.
Issue Description and Steps

I use the Third-party libraries

FaceAware  ( https://github.com/BeauNouvelle/FaceAware )

when the ImageView set image Completed, I Use FaceAware to deal with ImageView like The Code:
[self sd_setImageWithURL:[NSURL URLWithString:url]
        placeholderImage:nil
                 options:SDWebImageScaleDownLargeImages
               completed:^(UIImage * _Nullable image, NSError * _Nullable error, SDImageCacheType cacheType, NSURL * _Nullable imageURL) {
    weakself.focusOnFaces = YES;
    weakself.debugFaceAware = YES;
}];

When I scroll TableView, The memory growth too large, Maybe it cause A circular reference Or Something not destruction, How can I release the Memory?

@bpoplauschi Thanks for your quick reply. And I  I have added the issue info.  I set local Image and enable  FaceAware code, The memory is Fine, but when I use SDWebImage to load net Image, the memory is growing.  and I try to set the code.
`[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];
[[[SDImageCache sharedImageCache] config] setShouldDecompressImages:NO];`
but it does not work. and I think what cause A circular reference Or Something not destruction.
Thanks for your quick reply.

@CocoaML what do you mean by

I try to set the code. but it does not work.

What doesn't work? Doesn't compile? Doesn't run? The issue still happens with this code?

Sorry , I mean I use the code:
[[SDWebImageDownloader sharedDownloader] setShouldDecompressImages:NO];
[[[SDImageCache sharedImageCache] config] setShouldDecompressImages:NO];
and the memory also grows , not reduce my memory issue.

@CocoaML Hi.
 I guess it may because of the FaceAware code that open too much of bitmap context to detect the faces (I don't check their source code, you can have a try).
The best way to view your memory usage, firstlly try Xcode Memory Graph Debugger. You can see each alive object's retain chain. There may be a ❕mark in the object which is in retain cycle.

For more advanced debug, you need Instruments and using Leak scheme to check the status.

@CocoaML Hi.
Thanks, Now I have a try.

@dreampiggy  Hi.

Thanks, Now I have a try.

@dreampiggy @bpoplauschi Hi.
About The memory issue, I did a demo, link is Here,
https://github.com/CocoaML/SDWebImageAndFaceAwareDemo
The memory report shows memory used so high.
What can do to reduce memory ?

@CocoaML, I'll try to find the issue, but I need some extra informations, refer to your issue template, the issue only appear in iOS11? Which device can reproduce? Simulator can reproduce? How to define large memory? 300M? 1G?
I just run the demo you post on simulator iOS12 and memory works fine.


unsupported config type

So we have a policy that looks for cache-cluster resource types and reports the resources back. The policy runs just fine over c7n CLI without 'mode' defined but when its run with 'mode' defined it blows up with the following error message.
$ custodian run -s . policy.yml
name: policy-name
    description: Notify when elasticache-redis clusters have insecure configurations
    resource: cache-cluster
    filters:
       ......

When mode is defined, it blows up
name: policy-name
    description: Notify when elasticache-redis clusters have insecure configurations
    resource: cache-cluster
    mode:
        type: config-rule
        role: arn:aws:iam::<accountid>:role/<role>
   filters:
       ......

Here is the error message. May be I need to change the mode-type to something else, config-rule may not be the right choice but I couldn't find any help on other mode-types.
custodian.commands:ERROR Error while executing policy elasticache-redis-insecure, continuing
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/c7n/commands.py", line 227, in run
    policy()
  File "/usr/lib/python2.7/site-packages/c7n/policy.py", line 812, in __call__
    resources = mode.provision()
  File "/usr/lib/python2.7/site-packages/c7n/policy.py", line 511, in provision
    role=self.policy.options.assume_role)
  File "/usr/lib/python2.7/site-packages/c7n/mu.py", line 289, in publish
    if e.add(func):
  File "/usr/lib/python2.7/site-packages/c7n/mu.py", line 1444, in add
    params = self.get_rule_params(func)
  File "/usr/lib/python2.7/site-packages/c7n/mu.py", line 1408, in get_rule_params
    raise Exception("You may have attempted to deploy a config "
Exception: You may have attempted to deploy a config based lambda function with an unsupported config type. The most recent AWS config types are here: http://docs.aws.amazon.com/config/latest/developerguide/resource-config-reference.html.

Any insight would be appreciated.
Thank you

aws config don't support elasticache, so custodian can't generate a config rule policy for that resource type. the link in the error message shows you what types are supported by aws config. alternatively you can do a periodic, or cloudtrail mode to have a lambda policy that targets them, or run a non lambda policy.
see docs for more info https://capitalone.github.io/cloud-custodian/docs/policy/lambda.html

I see, so this is shortcoming from AWS side. I will dig some more into the alternatives and will close this then.
Thank you


"untilEndOfTurn" modifier

The nested effects apply until the start of next turn. Example:
"Give target unit +3/+0 until the end of turn."

Would it be useful if this worked similar to onStartOfTurn and onEndOfTurn? They allow for more variance:

Start of turn vs end of turn might matter when considering the order effects are applied.
The on* triggers optionally accept an argument specifying whose turn, e.g. onStartOfTurn('opponent').


remove $snapshotName from create image from managed disk

the variable $snapshotName is not used in this example of ## Create an image from a managed disk using PowerShell

@jrudley : Thanks for your contribution! The author, @cynthn, has been notified to review your proposed change.

#sign-off @jrudley Thank you for the contribution :)


libnvme: 1.11 -> 1.11.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/linux-nvme/libnvme/releases.
meta.description for libnvme is: C Library for NVM Express on Linux
meta.homepage for libnvme is: https://github.com/linux-nvme/libnvme
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.11.1 with grep in /nix/store/jdgxy3k55j722maf1y4hjahc7jg1yqrv-libnvme-1.11.1
found 1.11.1 in filename of file in /nix/store/jdgxy3k55j722maf1y4hjahc7jg1yqrv-libnvme-1.11.1

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

143 total rebuild path(s)

143 package rebuild(s)

First fifty rebuilds by attrpath

CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcsys
adapta-gtk-theme
almanah
ayatana-indicator-datetime
blivet-gui
bubblemail
budgie-control-center
calls
cantata
chatty
cheese
cinnamon-common
cinnamon-gsettings-overrides
cinnamon-screensaver
clementine
deepin.dde-device-formatter
deepin.dde-file-manager
deepin.dde-gsettings-schemas
deepin.deepin-album
deepin.deepin-anything
deepin.deepin-compressor
deepin.udisks2-qt5
deepin.util-dfm
endeavour
enlightenment.enlightenment
eos-installer
evolution
evolution-data-server
evolution-data-server-gtk4

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/6a05139118d67d2b1c91e6028f4d1f1bf4cb0f3e#libnvme

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

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

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


Questions about numbers in the Excel used to create cornstover biorefinery

I was under the impression that the "_humbird2011.xlsx" included in the cornstover package was based on Appendix A of the Humbird report so all sizes, costs, etc., should match. However, I noticed some differences between the numbers in the Excel and the numbers in Appendix A (those are just what I have noticed so far, I didn't do a full comparison).
I marked the differences in the attached Excel file using shading and added notes. The values in the attached file are what I think should be. Can you take a look and let me know if those are typos or intentional? Thanks!
_humbird2011_YL.xlsx

Thanks for catching these! The typos are probably negligible as the impact of those units combined is so small (some pumps and very small tanks). Since you are probably the only one working on this biorefinery, it is easy for you to make updates to the repository with out any conflicts! Could you give it a try to contribute back these bug fixes?
It is a little hard to use GitHub for the first time, but after that its a breeze. Here are some basic instructions (although you might have to google for a little more help):
https://biosteam.readthedocs.io/en/latest/developer's guide/Contributing to BioSTEAM.html

Also, FYI, some unit classes in the source code do not match the appendix table, but they do match the data given in the flowsheets (e.g. fermentors). The results on these units (as summarized in the original report) match what is found in the flowsheet... I believe some information was left out of the appendix regarding electricity consumption. Please let me know once you run into these for further explanation.

Yes I agree they are pretty minor issues, just want to make sure I'm understanding the codes correctly.
I forked the project for now but haven't committed anything - I'm still (slowly) looking at the code and may come up with questions later,  I'll compile them and push the changes together.

Sorry for leaving this unattended for so long, but I think many of the problems I was thinking have been fixed in the cornstover biorefinery now, the left issues are more of personal preference and not affecting the accuracy of the biorefinery, so I'll not pushing an update for the cornstover biorefinery.
But I do plan to push the new orgacids (portmanteau of organic acids) biorefinery in a month or so, that one is built on the cornstover biorefinery, and I left (I think) detailed notes on sources of all the numbers (e.g., which stream/table from the Humbird report), so that should help.


[SPARK-45481][SQL][FOLLOWUP] Add lowerCaseName for ParquetCompressionCodec.

What changes were proposed in this pull request?
https://github.com/apache/spark/pull/43308 introduces a mapper for parquet compression codecs.
There are many place call toLowerCase(Locale.ROOT) to get the lower case name of parquet compression codecs.
Why are the changes needed?
Add lowerCaseName for ParquetCompressionCodec.
Does this PR introduce any user-facing change?
'No'.
New class.
How was this patch tested?
Exists test cases.
Was this patch authored or co-authored using generative AI tooling?
'No'.

Merged to master. Thank you, @beliefer .

@dongjoon-hyun @LuciferYang Thank you!


New Main Loop

Don't worry, I kept the old loop behind a #define DO_OLD_MAIN_LOOP preprocessor directive.  Other than making the old loop more readable for me, I left it completely untouched.  The new loop processes all of the messages at once, then user input, then ticks the simulation as many times as needed to keep the simulation running at a realtime pace.  At least, that's what's supposed to happen.

Can you please explain what problem you are trying to solve with your edit? Note that Orbiter's fixed-timestep mode is not supposed to run in realtime. Its main point is to make the simulation independent of hardware by advancing the simulation by a defined time at each step, independent of how long it actually takes to compute. This is useful for debugging due to reproducibility independent of machine, or things like trajectory generation where you want to churn out data as fast as possible.
I am not convinced that the machine independence is preserved in your code, because the number of inner loop passes over the simulation steps is now dependent on system time, so the outer loop will be performed at different frequencies on different machines.

Fixed timestep mode has been very valuable to me in some of my NASSP systems stability testing.

I'm wanting a consistent simulation time step to run at realtime when a fixed time step is selected.  Also, I'm not sure I see what the problem with this code is.  I preserved the old loop behind a preprocessor directive, which allows fixed time steps to be used for debugging.
Here's what I'm trying to achieve:
https://www.gafferongames.com/post/fix_your_timestep/

Wouldn't it be better to impliment as a 3rd timestep type option, rather
than a compiler option that modifies how fixed-step works?
On Fri, Jul 30, 2021, 17:27 Timothy Chapman @.***>
wrote:

I'm wanting a consistent simulation time step to run at realtime when a
fixed time step is selected. Also, I'm not sure I see what the problem with
this code is. I preserved the old loop behind a preprocessor directive,
which allows fixed time steps to be used for debugging.
Here's what I'm trying to achieve:
https://www.gafferongames.com/post/fix_your_timestep/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mschweiger/orbiter/pull/17#issuecomment-890162833,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYP36XVD4V5H3BBCEJ6A3T2MKN5ANCNFSM5BJD7F3A
.

That should be relatively easy to do.

It is an interesting concept, but it probably needs more thought. Note that Orbiter already does timestep subsampling in various situations, so there is a danger that you effectively end up with two nested subsampling loops, which would likely be bad news.
I may pull your PR into a test branch to play with it. Did you actually compile and try it? What were your observations?

I'm getting about 2,000 compile errors right now, so I'm unable to test it out.

Also, I'm under the impression that the tick subdivisions happens either in BeginTimeStep() or EndTimeStep() or UpdateWorld().  I cannot find anything in the main loop itself that suggests subdivision.  So I do not believe that will be a problem.


Rock, Paper, or Scissors Project

Reference code was put at the comment below
Rock paper scissors Project is a classic two players game. Each player chooses either rock, paper, or scissors. The items are compared, and whichever player chooses the more powerful item wins.
The possible outcomes are:

Rock destroys scissors.
Scissors cut paper.
Paper covers rock.
If there’s a tie, then the game ends in a draw.

Our code will break the game into four parts:

Get the user’s choice.
Get the computer’s choice.
Compare the two choices and determine a winner.
Start the program and display the results.

// @Meow.Nalini98

const getUserChoice = (userInput) => {
  userInput=userInput.toLowerCase();
  if (userInput==="rock" || userInput==="paper" || userInput==="scissors") {
    return userInput;
  } else {
    console.log('Error: Please input again!')
  }
}

// console.log(getUserChoice('scissors'))

const getComputerChoice = () => {
  const randomNumber = Math.floor(Math.random()*3);
  switch(randomNumber) {
    case 0:
      return "rock"
    case 1:
      return "paper"
    case 2:
      return "scissors"
  }
};

// console.log(getComputerChoice())

const determineWinner = (userChoice, computerChoice) => {
  if (userChoice === computerChoice) {
    return 'The game is a tie!';
  }
  if (userChoice === 'rock') {
    if (computerChoice === 'paper') {
      return "That's a pity! The computer won!"
    } else {
      return "Congrats! You won!"
    }
  }

  if (userChoice === 'paper') {
    if (computerChoice === 'scissors') {
       return "That's a pity! The computer won!"
    } else {
      return "Congrats! You won!"
    }
  }

  if (userChoice === 'scissors') {
    if (computerChoice === 'rock') {
       return "That's a pity! The computer won!"
    } else {
      return "Congrats! You won!"
    }
  }
}

const playGame = () => {
  const userChoice = getUserChoice('rock');
  const computerChoice = getComputerChoice();
  console.log('You threw: ' + userChoice);
  console.log('The computer threw: ' + computerChoice);
  console.log(determineWinner(userChoice, computerChoice));
}

playGame()


Serving plain-text SVGs with Express.js

I posted this question on StackOverflow.  I wonder if there is a subtlety in Express that might help explain the behavior that I'm experiencing.

Hi @MCTaylor17 , there isn't fully complete code to really be able to tell, but some key questions I would have to track this down:

Can you show the complete response headers from that path when you use res.send vs res.sendFile?
Is your res.sendFile, which worked, actually using the exact same content as your res.send trial? Your example seems like it does not, since you are saying that res.sendFile(iconPath) works, though it looks like you are trying to alter the SVG in memory in some way. Is it possible to get the complete SVG contents before and after your svgProcessor call?

@dougwilson Thank you.  Your questions helped me to find the bug.  I posted the cause and solution on StackOverflow.

Awesome, @MCTaylor17 , I'm glad you got it working :) !


Import babel on-demand for faster startup when skipBabel: true

Issue

On my machine, importing "babel-core", "babel-plugin-istanbul", and "babel-preset-jest" takes 1.5 seconds.  TS takes only 0.2s.  If I set skipBabel: true ts-jest can skip importing babel and boot tests faster.
> [['jest'], ['typescript'], ['babel-core', 'babel-plugin-istanbul', 'babel-preset-jest']].forEach(a => {console.time(a); a.forEach(b => require(b)); console.timeEnd(a)})
jest: 24.603ms
typescript: 207.688ms
babel-core,babel-plugin-istanbul,babel-preset-jest: 1545.913ms

These imports can be executed conditionally, so that tests will start faster when "skipBabel" is enabled.
https://github.com/kulshekhar/ts-jest/blob/master/src/postprocess.ts#L5-L7

babel-core can be moved to optionalDependencies now no?


quay-enterprise: bump verisons to v2.2.0

@jzelinskie Close. There's also references to 2.1.0 in initial-setup.md and postges-additional-modules.md.
Other than that, this looks good.


opening a window "in the background" opens far away in alt+tab

Environment

Item
Value

OS, Version / Build
10.0.19042.0 Microsoft Windows NT 10.0.19042.0

Processor Architecture
AMD64

Processor Type & Model
intel

Memory
___

Storage Type, free / capacity (e.g. C: SSD 128GB / 512GB)
___

Relevant apps installed
___

Description

opening a window "in the background" by repeatedly clicking on another window while it opens. pretty easy to repro with chromium. i open a window or an app from the taskbar. i see this often with chromium but i don't repeatedly click on another window of course.
Steps to reproduce

see above. soz. can't be bothered filling this out properly. might later.
Expected behavior

to be next to my current window in alt+tab
Actual behavior

it opens far away. sometimes 6 windows away, sometimes it's the last window in alt+tab.

Unfortunately this issue is out of scope for this repo, so it won't be tracked here. BUT there's actually no need because this issue has been fixed in a recent build 🎉🙂 https://support.microsoft.com/en-us/topic/february-2-2021-kb4598291-os-builds-19041-789-and-19042-789-preview-6a766199-a4f1-616e-1f5c-58bdc3ca5e3b.

Only took several years. It really should be in scope of the repo. Rich said something like that.


refactor: cross margin math

removing unnecessary structs

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature/vector-of-options@8796749). Click here to learn what that means.
The diff coverage is 94.73%.

@@                     Coverage Diff                      @@
##             feature/vector-of-options     #157   +/-   ##
============================================================
  Coverage                             ?   38.43%
============================================================
  Files                                ?       39
  Lines                                ?     1837
  Branches                             ?      295
============================================================
  Hits                                 ?      706
  Misses                               ?     1071
  Partials                             ?       60

Impacted Files
Coverage Δ

src/core/engines/cross-margin/CrossMarginMath.sol
87.64% <94.73%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Mitigate Memory Leak in Libp2p Helper

Chainsafe gave some advice for how we can mitigate our libp2p helper memory leak. We should implement their suggestions and measure to see if there is a reduction in memory growth.
I will post the details of the advice Chainsafe gave us in this issue.

I am seeing this issue on my seed node:
process coda-libp2p_helper takes about 10-60 gigabytes of RAM (after a while)
Daemon_version: minaprotocol/mina-daemon-baked:1.1.7-d5ff5aa-mainnet

statistics for 7 days

Getting the same on our machine as a seed-node for Staker Space

Full-load CPU alert
High Ram alerts


All alcoholic beverages under one branch, merge various "distilled al…

…cohol" branches
Performed "nodeid" check on all files that exhibited changes.

Edited 'distilled spirit':

-Renamed 'distilled spirit' to 'alcoholic beverage (distilled)'
-Added 'SubClass Of' 'member of' some 'distilled spirits (us cfr)'
!-Added 'SubClass Of' 'has quality' some heat-treated
-Changed definition: 'An alcoholic drink produced by distillation of grains, fruit, or vegetables that have already gone through alcoholic fermentation. True distillation involves heating and cooling to seperate alcohol from the solution via evaporation and condensation.'
/-Definition source: https://en.wikipedia.org/wiki/Liquor
-Definition added to by term curator
/-Current synonyms: distilled drink, hard alcohol, liquor, hard alcohol, hard liquor, spirit
-Added synonyms: distilled alcohol, distilled spirit
-Changed ExactSynonym "distilled drink" to 'has broad synonym'
-Added additional term editor

Removed 'beverage (alcoholic, distilled)':

-No Subclasses to move
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason' Redundant entry
-Add annotation 'term replaced by'
Entity IRI> Classes> 'alcoholic beverage (distilled)'
-Added 'obsolete:' as prefix to label
-Added term editor

Removed 'beverage alcohol':

-No Subclasses to move
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason' Redundant entry
-Added annotation 'term replaced by' 'alcoholic beverage (distilled)'
-Added 'obsolete:' as prefix to label
-Added term editor

Removed 'plant derived distilled spirit'

-Moved Subclasses terms to 'alcoholic beverage (distilled)': absinthe, distilled spirits, grain based distilled spirit, liqueur food product, rum vodka
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason' Redundant entry
-Added annotation 'term replaced by' 'alcoholic beverage (distilled)'
-Added 'obsolete:' as prefix to label
-Added term editor

Removed 'spirits'

-No Subclasses to move
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason' Redundant entry
-Added annotation 'term replaced by' 'alcoholic beverage (distilled)'
-Added 'obsolete:' as prefix to label
-Added term editor

Removed 'plant based alcoholic beverage'

-Moved Subclasses terms to 'alcoholic beverage' (can reorganize further later): alcohol, barley malt beverage, fruit spirit drink, fruit spirits, grain based alcoholic beverage, herbal alcoholic beverage, manhatten cocktail mix (nutritively and nonnutritively sweetened), margarita mix, spirits (extract of), spirits with not more than 15% alcohol, sugar cane wine, tom collins mix, whiskey sour mix
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason': Redundant entry, already have 'alcoholic beverage' and both have the parent class 'plant derived fermented beverage'
-Added annotation 'term replaced by' 'alcoholic beverage'
-Added 'obsolete:' as prefix to label
-Added term editor

Removed 'distilled spirits'

-No Subclasses to move
-Changed 'SubClass Of' to 'Obsolete Class'
-Removed all other 'SubClasse Of's
-Added annotation 'owl:deprecated' true
-Added annotation 'has obsolescence reason': Redundant entry
-Added annotation 'term replaced by' 'alcoholic beverage (distilled)'
-Added 'obsolete:' as prefix to label
-Added term editor

Issue #104


ARTPayload: class review

Change payload property to data:

message.payload.payload --> message.payload.data

ARTPayload has an encoding property and ARTBaseMessage (that uses the ARTPayload class) has an encoding property as well. Confusing

Why is this not simply message.data like every other library?

Updated

@tcard Is this fixed, right? Can I close?

Yep, fixed!


Add Azul's Zulu JDK bundle

Zulu is a supported release of OpenJDK published by Azul systems.

Needs (at least) brew cask style zulu --fix
Not too familiar with the java-esque casks so will leave it for another maintainer to review.

Most if not all of the splits can be replaced with better version methods.

I can't figure out how to use homepage as referrer, so that value is duplicated. The version splits came from java.rb but it's much cleaner with the version methods.

Azul's download page isn't sorted, so the order of releases seems to vary. This now uses the _72 release.

Thank you for the contribution. It was merged directly as dc26dc02117b5b3f70dd45ecda154a27dcaa9623 to keep commit history cleaner. Your contribution is still credited to you.


README should note installation instructions

README says:
Just set your makeprg and errorformat as normal, and run:
:Neomake!

I just installed neovim and was told by a friend that I should use NeoMake instead of syntastic. I do not know how to "set my makeprg or errorformat as normal". I assume I need to actually download this repository to a particular directory on my machine.
I'll be happy to update the documentation to link to an external resource with instructions once I can figure out what I'm supposed to be doing.
Thanks for the help! 😸

I think I figured out an installation process that works:
$ git@github.com:neomake/neomake.git ~/.config/nvim/neomake
$ cd ~/.config/nvim/
$ ln -s neomake/autoload .
$ ln -s neomake/doc .
$ ln -s neomake/plugin .

The section is about usage, not installation (which is missing).
The recommended method is to use a plugin manager like https://github.com/junegunn/vim-plug (or Pathogen etc).  I agree that this should be added as a section.

Fixed by now: https://github.com/neomake/neomake#installation.


doubt about the loss computation in the train.py

loss = trainer(x_0).sum() / 1000.
why is 1000?

because 1000 can create a isotropic

could you pleasr tell me what does "isotropic" mean? Thanks!

because 1000 can create a isotropic

could you pleasr tell me what does "isotropic" mean? Thanks!

just make gradient desent faster

It's common to scale the loss by a constant factor to control the magnitude of gradients during training. Large gradients can lead to unstable training, so scaling the loss down can help stabilize the optimization process.


Fix #5927 [again] using solution from #5949

This passed all Unit Tests on Jasmine, and the lint test.
See #6935.

The #5927 was never fixed. Please change title and commit of the PR.

Originally the solution was submitted at #5949


System.InvalidOperationException: Unable to convert [Amazon.DynamoDBv2.DocumentModel.DynamoDBNull] of type Amazon.DynamoDBv2.DocumentModel.DynamoDBNull to System.String

Describe the bug
DynamoDBContext.QueryAsync has suddenly started throwing System.InvalidOperationException on query operation after upgrading Visual Studio from 2019 (16.11) to 2022 (17.4.3). We are using DynamoDB in our Xamarin.Forms App.
It seems kind of weird as we did not do any code changes or library updates. This was working fine in VS 2019.
Expected Behavior
DynamoDBContext.QueryAsync API call should succeed.
Current Behavior
DynamoDBContext.QueryAsync API throws the following exception
System.InvalidOperationException: Unable to convert [Amazon.DynamoDBv2.DocumentModel.DynamoDBNull] of type Amazon.DynamoDBv2.DocumentModel.DynamoDBNull to System.String
Reproduction Steps
After rebuilding the Xamarin Forms project in VS 2022. Our App throws the following exception for DynamoDBContext.QueryAsync API call
System.InvalidOperationException: Unable to convert [Amazon.DynamoDBv2.DocumentModel.DynamoDBNull] of type Amazon.DynamoDBv2.DocumentModel.DynamoDBNull to System.String
Possible Solution
No response
Additional Information/Context
Microsoft Visual Studio Community 2022
Version 17.4.3
VisualStudio.17.Release/17.4.3+33205.214
Microsoft .NET Framework
Version 4.8.04084
Installed Version: Community
ASP.NET and Web Tools   17.4.326.54890
ASP.NET and Web Tools
Azure App Service Tools v3.0.0   17.4.326.54890
Azure App Service Tools v3.0.0
C# Tools   4.4.0-6.22580.4+d7a61210a88b584ca0827585ec6e871c6b1c5a14
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Extensibility Message Bus   1.4.1 (main@2ee106a)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Mono Debugging for Visual Studio   17.4.19 (8c0a575)
Support for debugging Mono processes with Visual Studio.
Node.js Tools   1.5.40817.1 Commit Hash:66443775f9f3b1d8f8fee47af5002828b346688d
Adds support for developing and debugging Node.js apps in Visual Studio
NuGet Package Manager   6.4.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
Razor (ASP.NET Core)   <IP_ADDRESS>6202+61cc048d36a3fc9246d2f04625988b19a18ab8f0
Provides languages services for ASP.NET Core Razor.
TypeScript Tools   17.0.10921.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   4.4.0-6.22580.4+d7a61210a88b584ca0827585ec6e871c6b1c5a14
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools   17.4.0-beta.22512.4+525d5109e389341bb90b144c24e2ad1ceec91e7b
Microsoft Visual F# Tools
Visual Studio IntelliCode   2.2
AI-assisted development for Visual Studio.
VisualStudio.DeviceLog   1.0
Information about my package
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
VSPackage Extension   1.0
VSPackage Visual Studio Extension Detailed Info
Xamarin   <IP_ADDRESS>2 (d17-4@be7e8d1)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   <IP_ADDRESS> (remotes/origin/d17-4@d36bba3cc9)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin Templates   17.4.2 (c457c97)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.
Xamarin.Android SDK   <IP_ADDRESS> (d17-4/13ba222)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: a96bde9
Java.Interop: xamarin/java.interop/d17-4@fcc33ce2
SQLite: xamarin/sqlite/3.39.3@23e1ae7
Xamarin.Android Tools: xamarin/xamarin-android-tools/main@0be567a
Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (933c6c2c9)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.
AWS .NET SDK and/or Package version used
AWSSDK.DynamoDBv2 (<IP_ADDRESS>)
Targeted .NET Platform
.NET Standard 2.0
Operating System and version
Windows 10

@cjkarande Thanks for reporting the issue. The said error should not be a result of Visual Studio upgrade. Could you please share minimal sample code to reproduce the issue?
Thanks,
Ashish

@ashishdhingra . Below is the sample code
var records = await AWSServiceProvider.DynamoDBService.Context.QueryAsync(countryCode, QueryOperator.Equal, new List { mobileNumber }).GetNextSetAsync();
It throws the following exception.
System.InvalidOperationException: Unable to convert [Amazon.DynamoDBv2.DocumentModel.DynamoDBNull] of type Amazon.DynamoDBv2.DocumentModel.DynamoDBNull to System.String
PS: This code was working fine earlier when built in VS2019. Not sure if this has to do anything with the change in the .NET runtime & .NET framework versions, post VS 2022 upgrade

DynamoDBNull

@cjkarande Thanks for the reply. Could you please share the following so that I could reproduce the issue:

You mentioned you upgraded from Visual Studio from 2019 to 2022. This should not affect the code. Did you also update the AWSSDK.DynamoDBv2 package as well? If yes, from what version?
Details about the DynamoDB table schema (like HaskKey, SortKey).
Sample data. Also does the sample data has NULL value in any of the fields, such as in Sort Key?

Thanks,
Ashish

@ashishdhingra ,

AWSSDK.DynamoDBv2 version during VS2019 was <IP_ADDRESS> which I initially built with as-is post VS2022 upgrade and discovered this issue. Later I tried updating it to the latest version of <IP_ADDRESS>, however, the problem still persists
Table has a Hash key and Sort key both of type String and have valid string values (Not Null). The existing records in the table have been persisted earlier through App using DynamoDB APIs during VS2019 time.
As we cannot persist Null values for Keys they all have Not Null string values. Some of the other String attributes had Null values and they were working fine earlier. I even tried to replace these Null String attributes with empty strings ("") directly from the AWS console, but the issue still persists and the APIs like LoadAsync and QueryAsync throw the aforementioned exception.
Regards,
CP

@ashishdhingra ,

AWSSDK.DynamoDBv2 version during VS2019 was <IP_ADDRESS> which I initially built with as-is post VS2022 upgrade and discovered this issue. Later I tried updating it to the latest version of <IP_ADDRESS>, however, the problem still persists
Table has a Hash key and Sort key both of type String and have valid string values (Not Null). The existing records in the table have been persisted earlier through App using DynamoDB APIs during VS2019 time.
As we cannot persist Null values for Keys they all have Not Null string values. Some of the other String attributes had Null values and they were working fine earlier. I even tried to replace these Null String attributes with empty strings ("") directly from the AWS console, but the issue still persists and the APIs like LoadAsync and QueryAsync throw the aforementioned exception.
Regards,
CP

@cjkarande Thanks for the reply. As requested earlier,

Could you please share sample data (showing hash and sort key values, including rows with null values) and schema details?
Sample code project with model class. (as an example, refer https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/DynamoDBContext.QueryScan.html)

You may remove sensitive data and replace them with dummy values. I need these details to reproduce the issue. The scenario appears to be a basic one and should not just break with Visual Studio upgrade.
Thanks,
Ashish

@ashishdhingra, Sorry it was my bad. As other objects Load or Query were working fine, I could debug and isolate the issue. It was emanating from one of my model classes. The issue has now been fixed.
Apologies for soliciting your time on this issue, I appreciate your support though.
Thanks,


An option to disable "swim mode" (or sprinitng in general)

Is your feature request related to a problem? Please describe.
At this very moment i am trying to terraform underwater. Multiple times i accidentally entered "swim mode" which can interrupt instamining or make me mine on the wrong height.
Describe the solution you'd like
An option to just stay in "walk mode" like in 1.12 and before.

I'm not to sure if or how it's possible to implement that, but I'll try to implement it.


Emergency Message RX Callback

A feature which I've found lacking was the inability to take action when another node on the network transmits an emergency message.  These commits expand the emergency message callback to function similarly to the NMT callback.  Instead of this callback being called only on transmission of an emergency message it is now called on reception of one as well.
For example, we wanted our master (in this case canopend) to report any emergency messages on the bus by passing them through the command socket.
First we define a handler:
/* Handle CANopen emergency messages. */
void EM_report(const uint16_t ident,
               const uint16_t errorCode,
               const uint8_t errorRegister,
               const uint8_t errorBit,
               const uint32_t infoCode) {

    char buf[40];
    int len;

    len = sprintf(buf,
                  "EM: %04x %04x %02x %02x %08x\r\n",
                  ident,
                  errorCode,
                  errorRegister,
                  errorBit,
                  infoCode);

    CO_command_write(buf, len);
    taskMain_cbSignal();
}

Next we register it during setup:
CO_EM_initCallback(CO->em, EM_report);
Now whenever one of our nodes has a fault it is sent to the socket so action can be taken.

Any reason why this is not yet merged? I ran into the same issue and am currently using this code to handle incoming emergency messages.

Unfortunately I don't think anyone has heard from the owner since 2017, so the project is mostly abandoned.

Woops, didn't mean to close it.  Might as well keep it open just in case!

@flcapurso good news!  Looks like CANopenNode might be back in development soon!
see #105

I've changed that into two callbacks for compatibility of current software. please have a look as I don't use this.


Add Contributing.md file

Proposed Changes

Add Contributing.md file with all instructions for contributors
Instructions from forking the project to making a PR will be present
Additional Links to git tutorial would be present
Code for each of the above part would be present in case contributors want to avoid the hassle of reading the documentation of git.

someone is already doing it @Mitternachtsmond !!

There are neither any issues nor any PR regarding this
Anyways it's fine

Ohh Soryy...that was adding Contributors..coool...You can do thisss!! @Mitternachtsmond


Ability to sort Taxonomy terms via REST API

Related to #725 , you should be able to sort an arbitrary term's children via the REST API
The contract is:
/api/taxonomy/{uuid}/sortchildren?path={parentTermPath}

where the parentTermPath is optional.  If parentTermPath is not supplied the root terms of the taxonomy will be sorted.

@edalex-ian @cathfitz FYI

Is this included as part of #1225 @abidingotter ?

@edalex-ian Yes, I added it for the purposes of the autotest


Packages fail with Visual Studio 17.10

Describe the bug
vcpkg being used with Visual Studio 17.10 fail. Prior to 17.10 these packages were successful. Ella Zhang from Microsoft has reported that this is a vcpkg problem and not a Visual Studio 17.10 problem.
Environment

OS: Windows
Compiler: Visual Studio 17.10

To Reproduce

Install Visual Studio 17.10
Use vcpkg standalone (not integrated)
Attempt to install package from vcpkg (tested using multiple, but primarily fmt)
Package will fail
Roll back to previous Visual Studio 17.9.6
Attempt to install package from vcpkg
Package will fail

Expected behavior
vcpkg bootstraps and packages install
Failure logs
-- Running vcpkg install
error: in triplet x64-windows: Unable to find a valid Visual Studio instance
Could not locate a complete Visual Studio instance
The following paths were examined for Visual Studio instances:
  C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary/Build\vcvarsall.bat

-- Running vcpkg install - failed
CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:899 (message):
  vcpkg install failed.  See logs for more information:
  <source tree>\build\vcpkg-manifest-install.log
Call Stack (most recent call first):
  C:/Program Files/CMake/share/cmake-3.29/Modules/CMakeDetermineSystem.cmake:146 (include)
  CMakeLists.txt:3 (project)

-- Configuring incomplete, errors occurred!

Additional context
https://developercommunity.visualstudio.com/t/VCPKG-fails-with-VS-v17100/10663893

I upgraded visual studio 2022 to version 17.10, and then tested the installation of fmt. The results were normal and the problem you mentioned could not be reproduced.

I upgraded visual studio 2022 to version 17.10, and then tested the installation of fmt. The results were normal and the issue you mentioned could not be reproduced.

Thank you for this, it looks like it was an issue in vcpkg-cmake that was resolved in latest.  Will test and report back.


master: allow for hash-less job create

Validate job after generating the hash, if not present.
Add regression test.

Fixes #677, #610.

:+1: :100:

Client will fail here https://github.com/spotify/helios/blob/master/helios-client/src/main/java/com/spotify/helios/common/JobValidator.java#L263 as there's no hash.

helios inspect with data from shared.cloud works. helios create with old clients works.


X509: Unable Self-Sign Certs

Implementing: OpenSSL 3.0.7 1 Nov 2022 (Library: OpenSSL 3.0.7 1 Nov 2022) on Win64 OpenSSL and OpenSSL 3.0.2 15 Mar 2022 (Library: OpenSSL 3.0.2 15 Mar 2022) on Win WSL/Ubuntu VM
I am able to create CA Key/CA Cert/Target Key/Target CSR with no issues, but when I attempt to Self Sign the Target with x509 the command breaks.
Command:
openssl x509 -req -verbose -CA rootCA.crt -CAkey rootCA.key -in server.csr -out server.crt -days 365 -CAcreateserial
Error:
Certificate request self-signature ok
subject=C = US, ST = SC, L = Charleston, O = NIWCLANT, OU = TRSS, CN = TRSS-DEVCTR
A00F0000:error:0308010C:digital envelope routines:inner_evp_generic_fetch:unsupported:crypto\evp\evp_fetch.c:373:Global default library context, Algorithm (verbose : 0), Properties ()
A00F0000:error:03000086:digital envelope routines:do_sigver_init:initialization error:crypto\evp\m_sigver.c:253:
I have tried this on three different platforms: Win64 OpenSSL, WSL, and Ubuntu running in VM and I get same error on all three platforms.
All the help/error docs I have read through reference Node and React but not OpenSSL directly.
Thank You

It cant find the digest name in the provider. What is the name of the digest you are using e.g SHA256 ?

It cant find the digest name in the provider. What is the name of the digest you are using e.g SHA256 ?

I don't understand what that means.  Here is what I am doing:
openssl req -x509 -newkey rsa:4096 -days 365 -noenc -keyout rootCA.key -out rootCA.crt -verbose -subj "{my subject content}"
openssl req -newkey rsa:4096 -noenc -keyout server.key -out server.csr -verbose -subj "{my subject content"
openssl x509 -req -verbose -CA rootCA.crt -CAkey rootCA.key -in server.csr -out server.crt -days 365 -CAcreateserial
All the how to's and Stackoverflow articles follow this basic format.

Could you please show your openssl.cnf file?

openssl_UbntuVM.cnf.txt

Side note: I am able to self cert with 1.0.2K-fips, but I believe that is EOL.

There is no -verbose option with the x509 command. Just drop it from the x509 command line.

There is no -verbose option with the x509 command. Just drop it from the x509 command line.

Would be nice if the error message would have been a little descriptive like "Unknown option -verbose"?
Thanks for the help, though.

Closing since this is marked as resolved.


Create Model Objects

Create the model objects for the project.

Note: Given we are probably going to move away from Curated, we probably don't want to adhere too closely to what they are using, so we should just create a nice clean set of Model objects that suit our app.

Issues

Issue Number
Issue Date
Issue Description
List Curators

Articles

Article ID
Title
Author (Reference to an author object?)
URL
Description
Issue Published
Category

Author

Author ID
Author Name
Profile Photo URL
Github URL
WebPage URL
Twitch URL
YouTube URL
Twitter URL

Event

Event ID
Title
URL
Date
Event Type (Conference, User Group, Live Stream, Webinar)

Hi! I would be really happy to contribute to this project😃
About this issue specifically, I believe this would be relevant to clarify the type of every model's attributes.
Even if this is obvious for most of them, that'd allow us to get a better overview,
don't you think? ☺


Change wording on the landing page - ‘Coworkers’

Currently, on the landing page it reads:
‘Flow can help prevent bad rebases. Flow can help protect your carefully designed library from your coworkers.’ (Emphasis mine)
I do not want to protect my library from coworkers. I want them to be able to quickly and safely contribute.
In this PR I suggest ‘protect […] from errors’. However, I'd be happy with other changes.

wow just noticed it hahah. Yeah I can see this being change-worthy, thanks!

I think I'd prefer this be "Flow can help protect your carefully designed library from misuse and misinterpretation.", which is probably more on in spirit of the original text.
The "misuse" bit: JavaScript is very dynamic-- you can add and remove properties from pretty much any object at any point, browse up and down the prototype chain as you'd like, pass in objects that act like other objects or pretend to be functions/arrays... the way I read it is: if I meant to write a library that is moving an object around, and that shouldn't be mutated, I don't need to worry that someone is going to pass objects that mutate, or fake objects, or is going to inject their own methods into my library to handle their use-cases.
The "misinterpretation" bit: you can name types. If you're coding a big project it's not rare that things get bolted on the wrong places, because people understand things differently. For example, if I have a configuration object, it's well typed, defined in a single place everyone can see, and it's named type Configuration = ..., that helps me ensure people will understand if they want to pass in data this is not the object that should house it, but rather they should look somewhere else.
I think errors is a bit too much like the "bugs" sentence on "Code faster".
Thoughts?

@keithamus Sure! misuse and misinterpretation sounds much better. Thanks. Should I change that?

Yeah, can u? Ping me when done and I'll re-import 👍

@mrkev, I updated it 👍
(Yes, I typed ‘Kevin’ and blindly accepted the first suggestion 😄)


Fix: missing pod add events

#107 is missing pod add events, which ignores the pods running before merbridge
It was my bad, sorry!

Waiting for #113


export CUDA_VISIBLE_DEVICES=1 disables CUDA

Nice work!
I noticed though that on systems with 1 GPU, by default the scripts train on CPU. This is due to the bash files containing
export CUDA_VISIBLE_DEVICES=1
The issue is if you do this on a system with a single GPU the following returns false
torch.cuda.is_available()
Since the first GPU (normally) starts at 0

Hi, thanks for your interest.
You can change the bash script to export CUDA_VISIBLE_DEVICES=0 if the system only has 1 GPU.


Logo fix

Codecov Report

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

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   91.79%   91.79%
=======================================
  Files           2        2
  Lines         268      268
=======================================
  Hits          246      246
  Misses         22       22

Flag
Coverage Δ

#pytests
91.79% <ø> (ø)

Continue to review full report at Codecov.

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


rename "+New resource template" to "+New template"

Rename "+New resource template" button to "+New template" to be more straightforward.
Right now if your Sinopia window isn't wide enough, the button wraps, and at quick glance it looks like it says "+New resource"

now that templates are resources, there are probably other places we want to fix some naming. see also https://github.com/LD4P/sinopia_editor/issues/2378


Changes hashbang to bash.

The current is /bin/sh, but the used syntax is not posix shell compliant, e.g.: $() is a bash-ism.
It's unlikely to fail, as most systems link /bin/sh to a shell that would support $(), but it's better to be save ;)

Fine with me! Are there actually people without bash?

It's not always the default shell under /bin/sh, I believe the root user on ubuntu uses dash for instance, but it's almost always present

:-1: from me.
From prior experience writing cross-platform shell scripts, I recall that $() also runs fine on ash/busybox. I couldn't get ash to compile & run on OSX so I tested it with the equally minimalistic dash, and $() does work.
shellcheck(1) actually warns against using back ticks in POSIX compliant shells, which is what the original Bourne shell uses. I believe all modern POSIX shells now actually support $() so realistically won't cause any problems.
Unless forced to do so, I'm against using /bin/bash because it greatly reduces compatibility across systems.

:+1: last time I checked shellcheck did warn against backticks, but argued switching to $() and switching shells. Seems this has now been changed, so I stand corrected.


StrawberryShake subscription returns before ready - timing issue

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Context:
Topic: Automated API/integration tests.
Project: dotgraphee
Issue: StrawberryShake's subscription .Watch().Subscribe() seem to return before the system is ready to receive subscriptions.
(Cleanup: .Subscribe() returns an IDisposable which is disposed by the test fixture TearDown. We didn't forget it. 😉 )
Workaround: Thread.Sleep(1000); after subscription is started. 😢
Steps to reproduce

Set up a HotChocolate endpoint with subscriptions.
Set up a StrawberryShake client to subscribe and trigger that subscription with a mutation.
Create a test that calls the mutation immediately after starting the subscription, see image above. (dotgraphee is building a branch 'feature/strawberry-shake-client' that does step 1, 2 and 3.)
Test may pass randomly or when run in isolation. Create several or run in loop to reproduce.

Relevant log output
No response
Additional Context?
No response
Product
Strawberry Shake
Version
12.6.2

Do you have a small repro?

I had flaky test similar to what you describe but maybe my setup was wrong. To do subscribe->mutation->wait for subscription response->assert kind of test I was doing something like this.
var timeout = TimeSpan.FromSeconds(5);
var cancellationTokenSource = new CancellationTokenSource(timeout);
var value = 0;
using var _ = Client.OnSaleBid.Watch(saleId).Subscribe(response =>
{
  value = response.Data.OnValue.Value;
  cancellationTokenSource.Cancel();
});

await Client.Foo.ExecuteAsync(CancellationToken.Token);

await Task.Delay(-1, cancellationTokenSource.Token).ContinueWith(_ => { }, CancellationToken.None);
value.Should().BePositive();

It usually happened in CI where multiple tests run in parallel.

Delays are no good to deal with these kinds of tests.

@michaelstaib Same error happened when I rewrote my test using System.Reactive I guess there is race condition somewhere or something in my fixture setup which is more likely because I had to copy paste code from HC to make strawberry shake to use test server's web socket client. When I was asking for help on slack you said "Wait for the subscription stitching pr" which I assume would make it easier to do. Is the PR already part of some release? I could then update my fixture to see if the error still happens.


Portal no longer allows for storage accounts to be restored from another account

Updated documentation to reflect current restore procedure.
Removed unnessecary images and added screenshots
Changes to be committed:
deleted:    articles/storage/common/media/storage-account-recover/recover-account-portal.png
deleted:    articles/storage/common/media/storage-account-recover/recover-account-support.png
new file:   articles/storage/common/media/storage-account-recover/recover-storage-account-portal.png
new file:   articles/storage/common/media/storage-account-recover/recover-storage-account-support.png
modified:   articles/storage/common/storage-account-recover.md

@joshmccurry : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tamram,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.
#label:"aq-pr-triaged"

@joshmccurry - Thanks for making these updates! We updated the doc ourselves in the meantime. If you see anything else we're missing in the experience, please let us know.


[fix #303] Check MimeType uncompressed

To check that mimetype file in EPUB is not comressed, add a check in checkEpubHeader()
to avoid checking twice, ignore some checks in checkTrailingSpaces()

For EPUB files in https://github.com/IDPF/epubcheck/issues/303#issuecomment-260916592 , epubcheck with this patch finds 1 error in epub30-spec_compressed.epub , but no errors in epub30-spec_uncompressed.epub.

Thanks @takahashim! This has been on my list for months but I just didn't find the time for it 🙈


pip install (as per docs) fails with ModuleNotFoundError: No module named 'axolotl'

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When installing just using pip via the command in the docs:
pip3 install "axolotl[flash-attn,deepspeed] @ git+https://github.com/OpenAccess-AI-Collective/axolotl"

Current behaviour
/home/nlothian/axolotl/venv/bin/python3: Error while finding module specification for 'axolotl.cli.train' (ModuleNotFoundError: No module named 'axolotl')
Traceback (most recent call last):
  File "/home/nlothian/axolotl/venv/bin/accelerate", line 8, in <module>
    sys.exit(main())
  File "/home/nlothian/axolotl/venv/lib/python3.9/site-packages/accelerate/commands/accelerate_cli.py", line 47, in main
    args.func(args)
  File "/home/nlothian/axolotl/venv/lib/python3.9/site-packages/accelerate/commands/launch.py", line 994, in launch_command
    simple_launcher(args)
  File "/home/nlothian/axolotl/venv/lib/python3.9/site-packages/accelerate/commands/launch.py", line 636, in simple_launcher
    raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/home/nlothian/axolotl/venv/bin/python3', '-m', 'axolotl.cli.train', 'examples/openllama-3b/lora.yml']' returned non-zero exit status 1.

Others are seeing the same behavior: https://discord.com/channels/1104757954588196865/1111279858136383509/1164806862508666900
Steps to reproduce
Install via pip:
pip3 install "axolotl[flash-attn,deepspeed] @ git+https://github.com/OpenAccess-AI-Collective/axolotl"

However cloning the repo and installing that seems to get further:
git clone https://github.com/OpenAccess-AI-Collective/axolotl
cd axolotl

pip3 install packaging
pip3 install -e '.[flash-attn,deepspeed]'

Config yaml
No response
Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
main/7fabc4d
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

#910 ?

Duplicate of #910 ?

Probably the same underlying issue but the error message is subtly different:
(ModuleNotFoundError: No module named 'axolotl') vs (ModuleNotFoundError: No module named 'axolotl.cli')
Unclear why this is the case.

May I ask if docker works? Alternatively, could you try create a new conda/pip venv again?

Just running pip3 install -e '.[flash-attn,deepspeed]' once again in my venv solved the issue.

Actually the docs are wrong. You need to git clone the repo, then cd into the repo. Then do
pip install -e '.[flash-attn,deepspeed]'
and you will almost certainly need:
pip install --upgrade flash_attn

I am having this same issue. I looked at other issue reports and haven't found a solution yet

Axolotl needs to be installed from source by GitHub cloning the repository currently. We have dependencies that aren't packages currently so we can't push axolotl as a pypi package currently

Is there any plan to support installing Axolotl without cloning the repository or is that a fundamental limitation?

@fozziethebeat , you can run it like this.
pip3 install "axolotl[flash-attn,deepspeed] @ git+https://github.com/OpenAccess-AI-Collective/axolotl"

@NanoCode012 Interesting, that's what I'm doing (in a conda environment) but Axolotl isn't being found.  When I check the environments site-packages I just see the axolotl-0.4.0.dist-info directory but not the actual package.
Running python -m axolotl.cli.preprocess gives me the disappointing
/home/fozziethebeat/anaconda3/envs/planar-ax/bin/python: Error while finding module specification for 'axolotl.cli.preprocess' (ModuleNotFoundError: No module named 'axolotl')

Oh sorry, I did not read the earlier context. Hmm, it would seem that it's necessary to clone as per above discussion.


ci: cancel workflows when a new commit is pushed

Description
This should help with load on our test runners. Copied from https://stackoverflow.com/questions/66335225/how-to-cancel-previous-runs-in-the-pr-when-you-push-new-commitsupdate-the-curre
Notes & open questions
How fast will this cancel the jobs on our own boxes? No idea how that works really...
Change checklist

[x] Self-review.

Did a quick test, it seems to work with a small delay. Also, it triggers re-test if you remove some commits.

Nice, the delay is expected, even manually canceling jobs takes a bit of time due to the architecture of GH runners.
For those interested, there is no "push" mechanism to send updates to the runners, rather there's an endpoint that gets polled from the runners periodically. This means the poll loop if working correctly will take X amount of time to hit it again. Also under certain situations those runners get slower to poll (ie some heavy load/continuous tests) and then it takes a little bit longer.

Also let's just keep an eye out how this interacts with main and manually (comment based) runs.
ie multiple merged PRs on main or running CI for a PR + invoking /netsim


Stalling/freezing when using --model-path in CLI

Hi!
When using --model-path to a pre-downloaded model, there seems to be something that makes CLI calls stall:
./whisperkit-cli transcribe --verbose --skip-special-tokens --report --report-path '/Users/maxlund/whisper-reports' --model-path '/Users/maxlund/whisperkit-models/argmaxinc/whisperkit-coreml/distil-whisper_distil-large-v3_turbo' --audio-path '/Users/maxlund/audio/arthur-fx2.wav' --language 'en'
Task: Transcribe audio at ["/Users/maxlund/audio/arthur-fx2.wav"]
Initializing models...
Models initialized in 521.27 seconds
  - Encoder load time: 519.13 seconds
  - Decoder load time: 1.69 seconds
  - Tokenizer load time: 0.34 seconds

As you can see it took almost 9 minutes to initialise the model.
The second time we run the CLI call however, it's fast:
./whisperkit-cli transcribe --verbose --skip-special-tokens --report --report-path '/Users/maxlund/whisper-reports' --model-path '/Users/maxlund/whisperkit-models/argmaxinc/whisperkit-coreml/distil-whisper_distil-large-v3_turbo' --audio-path '/Users/maxlund/audio/arthur-fx2.wav' --language 'en'
Task: Transcribe audio at ["/Users/maxlund/audio/arthur-fx2.wav"]
Initializing models...
Models initialized in 1.25 seconds
  - Encoder load time: 0.83 seconds
  - Decoder load time: 0.06 seconds
  - Tokenizer load time: 0.27 seconds

It seems files are still being created in ~/Library/Caches/whisperkit-cli and it seems to be 1.5gb large even though I specified the --model-path. I suspect this has something to do with it?
Is there anything you can pre-download or files you can pre-populate in order to make startup faster (and possible) in a completely offline environment?

Hi @maxlund, this is unlikely to be related to the model being pre-downloaded. The additional file you observed is the system cache that Core ML has to generate so the generic model is specialized to your device's chip generation and the selected compute unit. When you first load the model, the cache is generated which takes a long time. We do have a solution coming up for this so stay tuned. This cache can not be pre-populated with publicly available APIs.

Thank you for the clarification!
Any idea on how different those files are from machine to machine? Could we pre-create the files for e.g. the distil large-v3 model, and bundle these files in an application to run on macOS?

Hi @atiorh ! Following on from Max's questions...

The additional file you observed is the system cache that Core ML has to generate so the generic model is specialized to your device's chip generation and the selected compute unit. When you first load the model, the cache is generated which takes a long time.

Are you able to provide give some more information/insight into this? Does this process happen faster on some machines than others? Does it only happen on macOS 15 and above?

We do have a solution coming up for this so stay tuned.

Are there any WIP branches I could play with? Is there a rough ETA of when this could be improved - weeks or months?

This cache can not be pre-populated with publicly available APIs.

Is this because it's proprietary CoreML magic that's happening under the macOS hood?
Thanks heaps for your time!

...and a dumb question... whilst the CoreML cache is being generated, would it be possible to just do WhisperKit processing on the CPU & GPU, until the CoreML cache is ready, then start using the Apple Neural Engine/NPU?

Playing around with @ZachNagengast's branches - it looks like MLX might solve this?

@atiorh I am unfortunately experiencing issues where this delay is happening more than once for the same model, even if it is located at the same path as previously etc. After a reboot we see ANECompilerService taking up a lot of resources and the model taking ~9min to produce a transcription result. Any guidance on possible ways to mitigate this would be much appreciated!

MLX definitely helps with this as long as you can spare the memory to load both models at the same time. We also rolled out some experimental changes in the latest testflight, try it out and see how the speeds compare, it's enabled in the experimental section in the settings.


Trying to docker container 2.5.10 but getting 2.4.20

I am creating a docker container for rhasspy on Unraid.  I got everything working correctly, except it is not pulling the correct container.  I am trying to pull the latest container from https://hub.docker.com/r/rhasspy/rhasspy.  That container should be 2.5.10 but when  pull up the web page for rhasspy it says version 2.4.20.  I tried forcing it to pull 2.5.10 by using rhasspy/rhasspy:2.5.10 and it still is using 2.4.20.  So I thought I would try pulling 2.5.9 by using rhasspy/rhasspy:2.5.9 but it is still using 2.4.20.  I am very confused.  Any help is greatly appreciated.

Configured on parameter in what was 2.4.20 and saved it and then the new 2.5.10 interface came up and it is working as expected now.


Issue with mutable_content

Hello ! I'm quite at a loss there, so I'm trying to write a message there.
So I want to display a rich notification with a nice image in iOS 10+.
For this I'm using FCM and an UNNotificationServiceExtension which, if I understood correctly, should get the Data payload, find the image URL and load it before modifying the UNNotificationContent.
I'm tried many many ways to send data in the payload of my call to https://fcm.googleapis.com/fcm/send and the issue I have is that I can't get a hold of the "data" I should handle in the UNNotificationServiceExtension.
My understanding is that it's the only way to do this kind of push, and make them work when the application is not launched.
I created a SO question some times ago with more details about what I'm doing and what is sent : using-data-in-rich-push-notification-on-ios-with-fcm
Could you help me pointing the issue there or tell me if I juste need to wait a little for this feature to be available?
Thanks a lot!

Based on your SO description and this one, I think you might be expecting that the rich media is automatically retrieved for you within the UNNotificationServiceExtensions. It looks like the payload you're receiving is as expected:

mutable-content is 1 within aps
mediaUrl is included outside of aps

It's the developer's job to take whatever data is in the payload, and convert that to a UNNotificationContent object, with media attachments.

How can I get the data that are outside of aps ? The object received 'UNNotificationRequest' doesn't contain anything that is outside of aps.


Add migration guide for v1.2.0

This is a first draft of the docs, migration guide, and release notes for v1.2.0. The date will need to be updated when we know the planned release date, but I figured we could start reviewing the docs now.
Resolves https://github.com/hyperledger/firefly/issues/1146
Resolves https://github.com/hyperledger/firefly/issues/1148

Codecov Report

Merging #1159 (5e3cdfd) into main (9ffba93) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              main    #1159      +/-   ##
===========================================
- Coverage   100.00%   99.98%   -0.02%
===========================================
  Files          305      305
  Lines        19967    19964       -3
===========================================
- Hits         19967    19962       -5
- Misses           0        2       +2

Impacted Files
Coverage Δ

internal/events/webhooks/webhooks.go
99.05% <0.00%> (-0.95%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


No detections is found

I tried a few actions like these:
~ # sh -c "/bin/grep 'x:0:' /etc/passwd > /tmp/passwords"
~ #
~ #
~ # sh -c whoami '[S];pwd;echo [E]'
root
$ cp /usr/bin/whoami ./whoami.rtf; ./whoami.rtf
app

When I go to Investigate > Detections
There's still no detections found.

Ok. I will contact the support again.
I was asked to post the issues here from the support ticket before.


Export Viewer component

Hello,
thank you for this useful component. In the current build, the viewer component does not seem to be exported, so you cannot use it at the moment.

Thanks for the pull request.