_WindowOverflow4 and Failed to configure netif (IDFGH-2625)

Environment
Development Kit: ESP32-WROVER-B
Module or chip used: WINGONEER T-Koala ESP32
IDF version: v4.1-dev-2055-gf5b82c5b1
Build System: CMake
Compiler version: crosstool-NG esp-2019r2) 8.2.0
Operating System: Linux
Power Supply: USB
Problem Description
Can someone help me understand the following problems.
The following is the text printed in red from the full trace below:
0x40080000: _WindowOverflow4 at /home/surfskidude/esp/esp-idf/components/freertos/xtensa_vectors.S:1778
0x400d0020: _stext at ??:?
0x400870a8: lmacTxFrame at ??:?
0x400811c8: call_start_cpu1 at /home/surfskidude/esp/esp-idf/components/esp32/cpu_start.c:275
E (989) esp_netif_lwip: esp_netif_new: Failed to configure netif with config=0x3ffc0070 (config or if_key is NULL or duplicate key)
Full trace:
make -j8  monitor
Toolchain path: /home/surfskidude/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: esp-2019r2
Compiler version: 8.2.0
Python requirements from /home/surfskidude/esp/esp-idf/requirements.txt are satisfied.
Project is not inside a git repository, or git repository has no commits
will not use 'git describe' to determine PROJECT_VER.
App "file_server" version: 1
Python requirements from /home/surfskidude/esp/esp-idf/requirements.txt are satisfied.
MONITOR
--- idf_monitor on /dev/ttyUSB0 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57
rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4
load:0x3fff0034,len:7144
load:0x40078000,len:14864
ho 0 tail 12 room 4
load:0x40080400,len:4568
entry 0x400806f0
I (31) boot: ESP-IDF v4.1-dev-2055-gf5b82c5b1-dirty 2nd stage bootloader
I (31) boot: compile time 13:36:40
I (32) boot: chip revision: 1
I (36) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (43) boot.esp32: SPI Speed      : 40MHz
I (47) boot.esp32: SPI Mode       : DIO
I (52) boot.esp32: SPI Flash Size : 4MB
I (56) boot: Enabling RNG early entropy source...
I (62) boot: Partition Table:
I (65) boot: ## Label            Usage          Type ST Offset   Length
I (73) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (80) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (88) boot:  2 factory          factory app      00 00 00010000 00200000
I (95) boot: End of partition table
I (99) boot_comm: chip revision: 1, min. application chip revision: 0
I (106) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x45560 (284000) map
I (217) esp_image: segment 1: paddr=0x00055588 vaddr=0x3ffb0000 size=0x039d8 ( 14808) load
I (223) esp_image: segment 2: paddr=0x00058f68 vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at /home/surfskidude/esp/esp-idf/components/freertos/xtensa_vectors.S:1778
I (224) esp_image: segment 3: paddr=0x00059370 vaddr=0x40080400 size=0x06ca8 ( 27816) load
I (244) esp_image: segment 4: paddr=0x00060020 vaddr=0x400d0020 size=0xe637c (942972) map
0x400d0020: _stext at ??:?
I (582) esp_image: segment 5: paddr=0x001463a4 vaddr=0x400870a8 size=0x0ebe0 ( 60384) load
0x400870a8: lmacTxFrame at ??:?
I (621) boot: Loaded app from partition at offset 0x10000
I (621) boot: Disabling RNG early entropy source...
I (621) cpu_start: Pro cpu up.
I (625) cpu_start: Application information:
I (630) cpu_start: Project name:     file_server
I (635) cpu_start: App version:      1
I (639) cpu_start: Compile time:     Feb  2 2020 13:36:39
I (646) cpu_start: ELF file SHA256:  15c900c9c8236301...
I (652) cpu_start: ESP-IDF:          v4.1-dev-2055-gf5b82c5b1-dirty
I (658) cpu_start: Starting app cpu, entry point is 0x400811c8
0x400811c8: call_start_cpu1 at /home/surfskidude/esp/esp-idf/components/esp32/cpu_start.c:275
I (0) cpu_start: App cpu up.
I (669) heap_init: Initializing. RAM available for dynamic allocation:
I (676) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (682) heap_init: At 3FFBA100 len 00025F00 (151 KiB): DRAM
I (688) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (694) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (701) heap_init: At 40095C88 len 0000A378 (40 KiB): IRAM
I (707) cpu_start: Pro cpu start user code
I (725) spi_flash: detected chip: generic
I (726) spi_flash: flash io: dio
I (726) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (829) wifi: wifi driver task: 3ffc1a94, prio:23, stack:3584, core=0
I (829) system_api: Base MAC address is not set
I (829) system_api: read default base MAC address from EFUSE
I (849) wifi: wifi firmware version: 94efb94
I (849) wifi: config NVS flash: enabled
I (849) wifi: config nano formating: disabled
I (849) wifi: Init dynamic tx buffer num: 32
I (859) wifi: Init data frame dynamic rx buffer num: 32
I (859) wifi: Init management frame dynamic rx buffer num: 32
I (869) wifi: Init management short buffer num: 32
I (869) wifi: Init static rx buffer size: 1600
I (879) wifi: Init static rx buffer num: 10
I (879) wifi: Init dynamic rx buffer num: 32
I (889) example_connect: Connecting to DD-WRT...
I (979) phy: phy_version: 4180, cb3948e, Sep 12 2019, 16:39:13, 0, 0
I (989) wifi: mode : sta (b4:e6:2d:df:63:01)
E (989) esp_netif_lwip: esp_netif_new: Failed to configure netif with config=0x3ffc0070 (config or if_key is NULL or duplicate key)
I (999) example_connect: Waiting for IP
I (3049) example_connect: Wi-Fi disconnected, trying to reconnect...
I (5099) example_connect: Wi-Fi disconnected, trying to reconnect...
I (7149) example_connect: Wi-Fi disconnected, trying to reconnect...
I (9199) example_connect: Wi-Fi disconnected, trying to reconnect...
I (11259) example_connect: Wi-Fi disconnected, trying to reconnect...
I (13309) example_connect: Wi-Fi disconnected, trying to reconnect...
I (15359) example_connect: Wi-Fi disconnected, trying to reconnect...
I (17409) example_connect: Wi-Fi disconnected, trying to reconnect...
I (19459) example_connect: Wi-Fi disconnected, trying to reconnect...
I (21519) example_connect: Wi-Fi disconnected, trying to reconnect...
I (23569) example_connect: Wi-Fi disconnected, trying to reconnect...
I (25619) example_connect: Wi-Fi disconnected, trying to reconnect...
I (27669) example_connect: Wi-Fi disconnected, trying to reconnect...

Hi @surfskidude
Can you please share some portions of your initialisation code?
The Failed to configure netif error suggests that an incorrect configuration was supplied to esp_netif_new().
It looks like (from the logs ... Wi-Fi disconnected, trying to reconnect... ) a standard example based initialization was used calling esp_netif_create_default_wifi_sta(); -- Is it correct?
In that case, one must make sure that the function is not called multiple times (since it creates a netif instance with default configuration) -- Is it possible that the function gets called more than once?
Even if the above was true, then the function should abort if called incorrectly -- is not it possible that the asserts have been disabled in menuconfig?

Hi David,
I am not directly calling esp_netif_new()
As a (new) test, I copied the following example "as is":
esp-idf/examples/protocols/http_server/file_serving
My only modifications to the above is the wi-fi name and password; set with: make menuconfig.

This sounds very strange, as that example uses common example_connect() logic and creates wifi station from default params (so both config and key are not null) and the only issue could be duplicate interface key. Moreover the example-connect is called just once after the example startup.
Is not it possible that you've might have combined more examples in one application (like http-file-server plus wifi-simple-station)? This is very simple in make just forgotten *.c file somewhere in a main component? (seen that you've identified CMake as your build system, but typed make ... to the logs...)
Even so, if the the above failed as seen from the log, the program should abort() in the following assert (reset in the default config, perhaps) and not continue to fail as you describe. Could you please share the output of cat sdkconfig | grep ASSERT

Just to make sure I did nothing wrong, I completely removed my installation and followed:
https://docs.espressif.com/projects/esp-idf/en/latest/get-started/index.html
Here is the installation log, including copying example, compiling, and running. As you can see, I still get the same error.
cd ~/esp
git clone --recursive https://github.com/espressif/esp-idf.git
cd ~/esp/esp-idf
./install.sh
source export.sh
cd ..
cp -r ~/esp/esp-idf/examples/protocols/http_server/file_serving .
cd file_serving
make menuconfig
make -j8 flash monitor
Toolchain path: /home/surfskidude/.espressif/tools/xtensa-esp32-elf/esp-2019r2-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: esp-2019r2
Compiler version: 8.2.0
Python requirements from /home/surfskidude/esp/esp-idf/requirements.txt are satisfied.
Project is not inside a git repository, or git repository has no commits
will not use 'git describe' to determine PROJECT_VER.
App "file_server" version: 1
Flashing binaries to serial port /dev/ttyUSB0 (app at offset 0x10000)...
esptool.py v3.0-dev
Serial port /dev/ttyUSB0
Connecting.......
Chip is ESP32D0WDQ5 (revision 1)
Features: WiFi, BT, Dual Core, 240MHz, VRef calibration in efuse, Coding Scheme None
Crystal is 40MHz
MAC: b4:e6:2d:df:63:01
Uploading stub...
Running stub...
Stub running...
Configuring flash size...
Auto-detected Flash size: 4MB
Flash params set to 0x0220
Compressed 26672 bytes to 15928...
Wrote 26672 bytes (15928 compressed) at 0x00001000 in 1.4 seconds (effective 151.4 kbit/s)...
Hash of data verified.
Compressed 766224 bytes to 480470...
Wrote 766224 bytes (480470 compressed) at 0x00010000 in 42.6 seconds (effective 143.8 kbit/s)...
Hash of data verified.
Compressed 3072 bytes to 119...
Wrote 3072 bytes (119 compressed) at 0x00008000 in 0.0 seconds (effective 1391.9 kbit/s)...
Hash of data verified.
Leaving...
Hard resetting via RTS pin...
MONITOR
--- idf_monitor on /dev/ttyUSB0 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ta        01 02 00009�ets Jun  8 2016 00:22:57
rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4
load:0x3fff0034,len:7144
load:0x40078000,len:14864
ho 0 tail 12 room 4
load:0x40080400,len:4568
entry 0x400806f0
I (31) boot: ESP-IDF v4.1-dev-2055-gf5b82c5b1-dirty 2nd stage bootloader
I (31) boot: compile time 08:31:04
I (32) boot: chip revision: 1
I (36) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (43) boot.esp32: SPI Speed      : 40MHz
I (47) boot.esp32: SPI Mode       : DIO
I (52) boot.esp32: SPI Flash Size : 4MB
I (56) boot: Enabling RNG early entropy source...
I (62) boot: Partition Table:
I (65) boot: ## Label            Usage          Type ST Offset   Length
I (73) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (80) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (88) boot:  2 factory          factory app      00 00 00010000 00100000
I (95) boot:  3 storage          Unknown data     01 82 00110000 000f0000
I (103) boot: End of partition table
I (107) boot_comm: chip revision: 1, min. application chip revision: 0
I (114) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x1d8d0 (121040) map
I (166) esp_image: segment 1: paddr=0x0002d8f8 vaddr=0x3ffb0000 size=0x02720 ( 10016) load
I (171) esp_image: segment 2: paddr=0x00030020 vaddr=0x400d0020 size=0x8452c (541996) map
0x400d0020: _stext at ??:?
I (367) esp_image: segment 3: paddr=0x000b4554 vaddr=0x3ffb2720 size=0x01178 (  4472) load
I (369) esp_image: segment 4: paddr=0x000b56d4 vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at /home/surfskidude/esp/esp-idf/components/freertos/xtensa_vectors.S:1778
I (374) esp_image: segment 5: paddr=0x000b5adc vaddr=0x40080400 size=0x1560c ( 87564) load
I (432) boot: Loaded app from partition at offset 0x10000
I (433) boot: Disabling RNG early entropy source...
I (433) cpu_start: Pro cpu up.
I (437) cpu_start: Application information:
I (441) cpu_start: Project name:     file_server
I (447) cpu_start: App version:      1
I (451) cpu_start: Compile time:     Feb  3 2020 08:31:02
I (457) cpu_start: ELF file SHA256:  e2dc788e1e8adb21...
I (463) cpu_start: ESP-IDF:          v4.1-dev-2055-gf5b82c5b1-dirty
I (470) cpu_start: Starting app cpu, entry point is 0x400811bc
0x400811bc: call_start_cpu1 at /home/surfskidude/esp/esp-idf/components/esp32/cpu_start.c:275
I (0) cpu_start: App cpu up.
I (481) heap_init: Initializing. RAM available for dynamic allocation:
I (487) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (493) heap_init: At 3FFB91D8 len 00026E28 (155 KiB): DRAM
I (500) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (506) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (513) heap_init: At 40095A0C len 0000A5F4 (41 KiB): IRAM
I (519) cpu_start: Pro cpu start user code
I (537) spi_flash: detected chip: generic
I (538) spi_flash: flash io: dio
I (538) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (641) wifi: wifi driver task: 3ffc0a60, prio:23, stack:3584, core=0
I (641) system_api: Base MAC address is not set
I (641) system_api: read default base MAC address from EFUSE
I (661) wifi: wifi firmware version: 94efb94
I (661) wifi: config NVS flash: enabled
I (661) wifi: config nano formating: disabled
I (661) wifi: Init dynamic tx buffer num: 32
I (671) wifi: Init data frame dynamic rx buffer num: 32
I (671) wifi: Init management frame dynamic rx buffer num: 32
I (681) wifi: Init management short buffer num: 32
I (681) wifi: Init static rx buffer size: 1600
I (691) wifi: Init static rx buffer num: 10
I (691) wifi: Init dynamic rx buffer num: 32
I (701) example_connect: Connecting to DD-WRT...
I (791) phy: phy_version: 4180, cb3948e, Sep 12 2019, 16:39:13, 0, 0
I (801) wifi: mode : sta (b4:e6:2d:df:63:01)
I (801) example_connect: Waiting for IP
I (2851) example_connect: Wi-Fi disconnected, trying to reconnect...
I (4901) example_connect: Wi-Fi disconnected, trying to reconnect...
I (6951) example_connect: Wi-Fi disconnected, trying to reconnect...
I (9001) example_connect: Wi-Fi disconnected, trying to reconnect...
I (11051) example_connect: Wi-Fi disconnected, trying to reconnect...
I (13111) example_connect: Wi-Fi disconnected, trying to reconnect...

Thanks for starting over from the clean repository. Now it looks a bit different. No error related to netif configuration, example is just trying to connect  but for some reason cannot. Is the password correct? You can switch to debug level (components -> log) to see more details about connecting.
As for the _WindowOverflow4, it's not an exception nor any error. This is just the way IDF monitor interprets addresses, documented here and very useful if program crashes, but might be confusing too if numbers starting with 0x4... appear in the log.

Yes, I had a typo in the password and it is now working.
As for the original project: I made a fork of the "file_serving" example and used it as a base for a different server. I then upgraded esp-idf and it failed to work as initially stated. I now see that several API calls are different in the new "file_serving" example. After updating my project based on the new "file_serving" example, everything appears to be working, however, I have a question:
Will you continue to change APIs and break backward compatibility in future releases?

@surfskidude glad to hear that it works now.

Will you continue to change APIs and break backward compatibility in future releases?

IDF releases follow the versioning scheme documented here, so I am afraid the incompatibilities may appear when switching between major release versions. Sorry for the inconvenience.

@surfskidude Any other trouble or could this issue be closed?


People & Partners

Who are the people we are creating for?
Do we want John Baez and Paul Bourke to be looking in?
Or are we monitoring the chat of high school seniors?
Are we here to help the learners or to help the teachers?
Do we want to help other besides the Exploratorium?
Perhaps Imaginary.org? MSRI.Org? http://physicslearning.colorado.edu/ (PIRO)
Do we want to partner with or join in existing efforts?
In both cases, would it be a nice thing to record somewhere the people and organizations that we find interesting?

Our primary target group should be coders of high school age, undergraduates and educators who teach either level. The exhibits should be simple enough that the code remains central. Research-level visualizations should only appear if they don't distract from that focus.

Links will be sent to Imaginary.org when there is more content, which will eventually happen.
This issue can certainly be used to record interesting organizations as they are identified.


Improve JDBC Metadata support

Improve Drill JDBC metadata support by upgrading Avatica version, adding types filter to getTable Metadata API, and letting the JDBC driver using new Metadata API instead of SQL queries.

Closing: same as #613


Fixes link to supported features and docs

Thank you, I appreciate the contribution! However, this is already handled fully in #455, so I'm going to centralize the work there 🙏


[SignIn] Cache the access token

Signing in often takes a few seconds, so to speed up debugging time and not waste users' time consider persisting the access token locally. (Is there a security risk to persisting it in plaintext? Need to use crypto?)

Done in https://github.com/jamesqo/Repository/commit/47e1427a41eb22eaf9c717e4f778ce19b3b7ec27.


backup/sql: reset Attr in load for ACL

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


test/support windows

it would be nice to have more testers on windows to debbug why the tool isnt working properly on it

I'll take a look into this.. I'm running Windows 10


Fix error in docs: rootSaga pattern code example

Fixing a small typo in the code example for rootSaga patterns. Also pointed out by another user here. The array of sagas needs to be passed to an all which is missing in the docs.
Without the all, none of the saga watchers are running, but there are no errors either, so a beginner wouldn't know what's going wrong.

Thansk!


[137] Location-related features for investments predictions

Closes #137

Adds a script to create a csv file with Crunchbase organisations and R&D indicators  of their locations.
Also, adds a utility function to geolocate any address using Bing API (not used in the final script)

Usage:
python innovation_sweet_spots/pipeline/pilot/investment_predictions/create_dataset/create_beis_indicators.py --help

The R&D indicators are sourced from the Nesta/BEIS R&D dashboard. The script will download the data if it doesn't exist locally.
This script makes use of a previously prepared dictionary mapping Crunchbase location_id to NUTS2 codes. This allows to map 99% of the UK companies in the Crunchbase data to NUTS2. You'll need to download the file cb_geos.json from s3 and place it into inputs/data/misc/geo
The remaining 1% could be mapped via postcodes or addresses, to longitude and latitude, and then to NUTS2 (not implemented in the final script; some early work is in the prototyping notebook which is included in this PR for completeness).
No need to review the prototyping notebook innovation_sweet_spots/analysis/prototyping/2022_04_08_beis_indicators.py - I've just left it in this PR as there is some code to geolocate companies via their postcodes and addresses, that might become useful later down the line.
@Jack-0-0: please feel free to adjust the script so that it fits with the rest of your pipeline.
In terms of which features to actually use in the final model: I guess we could first try all of them?
I think there might be some interesting issues with the fact that London area appears to be divided into smaller NUTS2 regions that can have considerably different values (eg the number of STEM professionals). Presumably, a London startup in a NUTS2 region with a smaller RnD value will still benefit from the RnD happening in the rest of the London regions.

Checklist:

[x] I have refactored my code out from notebooks/
[x] I have checked the code runs
[x] I have tested the code
[x] I have run pre-commit and addressed any issues not automatically fixed
[x] I have merged any new changes from dev
[x] I have documented the code

[x] Major functions have docstrings
[ ] Appropriate information has been added to READMEs

[x] I have explained the feature in this PR or (better) in output/reports/
[x] I have requested a code review

Thank you very much for implementing this @beingkk.
I had an issue when running python innovation_sweet_spots/pipeline/pilot/investment_predictions/create_dataset/create_beis_indicators.py
as it tried to load a bing API key from .env when it runs the utils/geo file.
Locally, I just commented out the geolocate_address function and the bing/env constants at the top to allow create_beis_indicators.py. Maybe this functionality could be moved to another file as it is not being used currently?
I made a couple of minor changes to remove an unused f string and int().

Thank you for checking and making the adjustments! I've added a quick fix by commenting out the lines and added an issue to deal with this properly  a bit later. Hope that works for now!

It looks good to merge 👍
I think it's nice to keep create_beis_indicators.py on its own rather than integrating into investment_predictions/create_dataset.py as it could be used on it own for different analysis.

Thank you @Jack-0-0 !


arguments are required: -y/--yolo

usage: mask-detector-video.py [-h] -y YOLO [-i INPUT] [-o OUTPUT] [-d DISPLAY]
[-c CONFIDENCE] [-t THRESHOLD] [-u USE_GPU]
mask-detector-video.py: error: the following arguments are required: -y/--yolo
Is there any help where i am wrong

@nanoakhi
-y is a required argument.
You can refer Quick Guide, to get the ideal command.
Link : https://github.com/adityap27/face-mask-detector#Quick-Start

Done successfully


chore(sqlsmith): disable FILTER with correlated input reference

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Workaround: https://github.com/risingwavelabs/risingwave/issues/4762
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features).
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

Failing because skipped query threshold reached in deterministic test. But should probably ignore error threshold in deterministic test / give more leeway, since rng will not be random.


Fallout 2 Community Edition

Project Link
https://github.com/alexbatalov/fallout2-ce
Project License
https://github.com/alexbatalov/fallout2-ce/blob/main/LICENSE.md
Steam App Ids
https://steamdb.info/app/38410/
Additional Context
No response

@KawaiiDinosaur Thanks for the package request!
Do you know what the current state of this project is, in terms of how playable it is?
Also, is this a reverse engineering project, or a re-implementation?
It looks like with the license, I'd probably consider this a non-free license if implemented, similar to a few other engines that I've done.

@d10sfan the game is playable from start to finish, yes.
The reverse engineered project by the same individual is in a separate repository (https://github.com/alexbatalov/fallout2-re). I understand that the "Community Edition" is a reimplementation of the former.


error connecting form W10 to ubuntu 17.10

All PCs are in my lan, here is the rdp error.
I have an headless server with ubuntu
i cannot connect in any way
SSH works well
sudo sudo tail /var/log/xrdp-sesman.log
[20171207-13:10:55] [DEBUG] Closed socket 9 (AF_INET6 :: port 6210)
[20171207-13:10:55] [DEBUG] Closed socket 8 (AF_INET6 ::1 port 3350)
[20171207-13:10:55] [DEBUG] Closed socket 7 (AF_INET6 ::1 port 3350)
[20171207-13:10:55] [INFO ] Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp
[20171207-13:11:05] [ERROR] X server for display 10 startup timeout
[20171207-13:11:05] [INFO ] starting xrdp-sessvc - xpid=1436 - wmpid=1435
[20171207-13:11:05] [ERROR] X server for display 10 startup timeout
[20171207-13:11:05] [ERROR] another Xserver might already be active on display 10 - see log
[20171207-13:11:05] [DEBUG] aborting connection...
[20171207-13:11:05] [INFO ] ++ terminated session:  username lospo, display :10.0, session_pid 1434, ip <IP_ADDRESS>:59574 - socket: 12

You should read /usr/share/doc/xrdp/README.Debian if you haven't read it.


hardhat test riemann

Gas costs without Riemann approx : 26174
Gas costs with Riemann approx : 12173

Converted to draft to keep #6 major


Allow fallback when no node is found

I've looked through the readme, and it looks like right now it's only possible to add a fallback that will be used for buffers where treesitter is not present. However, I'd like to also have a fallback in cases where I do have treesitter, but no node is found.
The main use case for this would be literal strings that can't be parsed by tresitter, but still contain code that can be split/joined.
I'd be up for implementing this as well, what do you think about this feature?

This would be a useful improvement, thanks for the idea! If you are willing to do the implementation - feel free.
Please note that currently a recursive search for a suitable node is implemented, which only ends at the root node. Therefore, it makes sense to make the "fallback" field in the settings of each node, rather than a common one for all, to avoid false positives.

Done #125 and #124


Update User Guide for Edit Person function.

Codecov Report

Merging #22 (754f08a) into master (b14fff2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #22   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 b14fff2...754f08a. Read the comment docs.


[Task]: SOQL and SOSL querying in Salesforce

Connector Name
module/salesforce (Salesforce)
Task Description

SOQL (Salesforce Object Query Language) is a query language specific to Salesforce which could be used to  read information stored in your org’s database. SOQL is syntactically similar to SQL (Structured Query Language).

SOSL (Salesforce Object Search Language) is a language that performs text searches in records. Unlike SOQL, SOSL can query multiple types of objects at the same time. SOSL can also use a word match to match fields, while SOQL needs the exact phrase.

Deliverables

A summary of concepts of SOQL and SOSL queriying, Governer limits and best practices
Milestone plan for next steps

Summarized document prepared https://docs.google.com/document/d/1j-l42WTShpAKvlYh7fOziv2NnOTudGdg42t5Enx7Ft0/edit?usp=sharing


some updates fail on windows xp

KB2565057 Visual Studio 2010 Service Pack 1 MFC Security Update

(fails only offline installer, succeses from LegacyUpdate)
https://www.microsoft.com/en-US/download/details.aspx?id=27049

KB2687455 Service Pack 2 for Microsoft Office 2010

https://www.catalog.update.microsoft.com/Search.aspx?q=KB2687455

KB2598241 Input Method Editor 2010 update: July 10, 2012

https://www.catalog.update.microsoft.com/Search.aspx?q=KB2598241

KB2553347 July 14, 2015, update for Office 2010

https://www.catalog.update.microsoft.com/Search.aspx?q=KB2553347

after installing newer root certificates from below site, resolved this probrem.
http://win2k.org/wlu/wluen.htm
search query: KB3097966

im getting this now with Security Update for Windows XP Embedded (KB2686509) & Windows Live Essentials which fails to install w/ this Error

this is (still from my report that was on 1.5's Parameter bug,) a Mid 2010 "Unibody" Mac Mini , but XP was installed via "XOMPIE" ( XP on Mac) not via Bootcamp since XP was never Offically supported on this Mini
Incidently Microsoft Security Essentials cannot update and spews this:

Live Essentials 2011 installer and Security Essentials definition updates won’t work because they directly use Microsoft Update, bypassing the WSUS server configured by Legacy Update. I believe the final definitions update for MSE on XP neutered its functionality anyway, so not really worth trying to fix that. There is an offline Live Essentials 2011 update on Windows Update that gets offered if you have WLE 2009 installed, been hoping to override so that gets offered even without WLE 2009 installed.

It's no longer reproduced after reinstalling windows and legacyupdate. thanks.


[Bug]: ReferenceError: exports is not defined when using --experimental-vm-modules

Version
29.0.3
Steps to reproduce
I followed the documentation ( https://kulshekhar.github.io/ts-jest/docs/next/guides/esm-support/ ) to enable the support for Typescript and ESM on Jest.
jest.config.ts
import { JestConfigWithTsJest } from "ts-jest";

const jestConfig: JestConfigWithTsJest = {
  preset: "ts-jest",
  testEnvironment: "node",
  extensionsToTreatAsEsm: [".ts"],
  moduleNameMapper: {
    "#ansi-styles": "ansi-styles/index.js",
    "#supports-color": "supports-color/index.js",
    "^(\\.{1,2}/.*)\\.js$": "$1",
  },
  transform: {
    // '^.+\\.[tj]sx?$' to process js/ts with `ts-jest`
    // '^.+\\.m?[tj]sx?$' to process js/ts/mjs/mts with `ts-jest`
    "^.+\\.m?[tj]sx?$": [
      "ts-jest",
      {
        tsconfig: "./tsconfig.json",
        useESM: true,
        isolatedModules: true,
      },
    ],
  },
  transformIgnorePatterns: ["/node_modules/(?!(chalk))"],
};

export default jestConfig;

tsconfig.json
{
  "extends"<EMAIL_ADDRESS>  "include": [
    "./**/*"
  ],
  "compilerOptions": {
    "baseUrl": ".",
    "outDir": "../../dist",
    "module": "Node16",
    "moduleResolution": "Node16",
    "sourceMap": true,
    "target": "es6",
    "allowJs": true,
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "lib": [
      "dom",
      "dom.iterable",
      "es6",
      "scripthost",
      "es2015",
      "es2016",
      "es2017",
      "es2021.string",
      "esnext.intl"
    ],
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "strict": true
  }
}

src/bin/tests/Cunningham.spec.ts
describe("Cunningham Bin", () => {
  test("Run", async () => {
    expect(1).toEqual(1);
  });
});

package.json
{
  ...
  "type": "module",
  ...
}

Expected behavior
Adding --experimental-vm-modules should not execute my tests as CJS.
Actual behavior
When I run node_modules/.bin/jest --config ./src/bin/jest.config.ts --no-cache everything works fine:
 PASS  src/bin/tests/Cunningham.spec.ts
  Cunningham Bin
    ✓ Run (1 ms)

But when I add the --experimental-vm-modules flag here is what I get:
NODE_OPTIONS=--experimental-vm-modules node_modules/.bin/jest --config ./src/bin/jest.config.ts --no-cache
 FAIL  src/bin/tests/Cunningham.spec.ts
  ● Test suite failed to run

    ReferenceError: exports is not defined

This error leads me to think that enabling  NODE_OPTIONS=--experimental-vm-modules makes the test to be executed as CommonJS .. and not ESM.
NB: "You could juste not set the NODE_OPTIONS option if it works without" -> Nope, I can't because. Here I simplified this example, but in my real tests I need it You need to run with a version of node that supports ES Modules in the VM API. See https://jestjs.io/docs/ecmascript-modules )
I spent the afternoon on this problem and I can find a solution .. thank you in advance
Debug log
 FAIL  src/bin/tests/Cunningham.spec.ts
  ● Test suite failed to run

    ReferenceError: exports is not defined

      at tests/Cunningham.spec.ts:11:23

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.328 s
Ran all test suites.
(node:32070) ExperimentalWarning: VM Modules is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)

Additional context
No response
Environment
System:
    OS: macOS 13.0.1
    CPU: (10) arm64 Apple M1 Pro
  Binaries:
    Node: 16.15.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.11.0 - /usr/local/bin/npm

This is related to configuration. You can check example app https://github.com/kulshekhar/ts-jest/tree/main/examples to find the correct config. I hope it helps.


Fix Py3 issue - hasattr raises exception from getattr instead of swallowing

In Py2 and Py3, hasattr() works by trying to get the attribute.  But (I think) in Py2 any exception gets re-raised as AttributeError while in Py3 the actual exception gets raised.  If it isn't AttributeError then hasattr() just raises an exception instead of giving False.
See the Hacker News thread "hasattr considered harmful" for background.

BTW this was causing a test to fail, so no updates to tests are needed.

I don't completely understand the attribute handling in Time.py to understand the fix, but don't see a problem with it.
:+1:


[Android] Make shared mode prompt more clearly

Change the wording on the positive button from “Continue” to
“Get Crosswalk”
Change "application store" to "app store"
The term “runtime” has no meaning to an end user, it is a highly
technical term. Change it to "Add-on"

BUG=XWALK-4801

@baleboy @rakuco @halton Please help to review

LGTM after check with the bug discussion. I'll merge it after @baleboy 's lgtm.

There was some further discussion about the name "Add-on" and finally "Service" was considered a better option. LGTM if you can replace "Add-on" with "Service".

Done!


try pyo3 main

See if https://github.com/PyO3/pyo3/pull/2881 improves performance.

Codecov Report

Merging #614 (28fcc97) into main (c818c3e) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #614      +/-   ##
==========================================
- Coverage   94.19%   94.19%   -0.01%
==========================================
  Files          98       98
  Lines       13189    13189
  Branches       25       25
==========================================
- Hits        12424    12423       -1
- Misses        759      760       +1
  Partials        6        6

Impacted Files
Coverage Δ

src/input/input_python.rs
98.72% <100.00%> (ø)

... and 3 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

Done in #658.


Add proper documentation for website builder

Change-type: patch

I think that simplification would result in extra friction being added to the process. I don't see any use for a randomly generated URL as compared to the name-based generated one. What it would do is add extra code for automating more of Cloudflare.

@vipulgupta2048  @Bucknalla here's a thought - why would someone care about where it is hosted? Cloudflare should be an alien concept to someone who just wants a pretty looking docs site

Agreed, tbh the user shouldn't even need to know that it's cloudflare, they should just specify that they want a website and what subdomain it should have. Thus we abstract and hide all of that from the in a 'website builder' rather than having two slightly conflated workflows that overlap.

@Bucknalla @vipulgupta2048 to add more clarity - I am now proposing that we rename the flowzone input for docs site to just docusaurus_project and then let ops fill that in flowzone.yml for anyone requesting a website. Because there are a couple of steps that ops has to do and just one step that the user has to do - let's just eliminate the user step, and keep it all for ops.

What about removing the fact that it's docusaurus under the hood? website: autokit-usermanual for example. If we can error handle it better for when there isn't a cloudflare site set up, to suggest they should ask ops, that would be good but not necessary critical.

I'm not sure that we can remove the Docusaurus details. This was my big realisation from my docusaurus-101 session with @vipulgupta2048 a while back - as soon as cloudflare_website is added, (part of) the repo becomes docusaurus content that will be pre-processed by docusaurus-builder, processed by docusaurus and then published. So the repo's maintainers need to be familiar with (and have access to) the requirements of those tools.
So in my experience the user's tasks are:

Pick a project name and check it's not already taken <-- maybe in consultation if we want central coordination of these names - pending outcome of Zulip thread.
As ops to create it / add redirects.
Prepare the repo to suit - eg. take advantage of the separate READMEs for GitHub and for Docusaurus, add any frontmatter tags to suit our theme, update absolute/relative media URLs, consider the specialisations afforded by docusaurus-builder.
If the URL is public/known/linked, then preview locally.
Add maintainer documentation that explains that the repo contains docusaurus content, and how to work with it, referencing docusaurus-builder.
When ready, throw the switch and edit flowzone.yml.

We've already set the precedent that the repo maintainer is in charge of the flowzone.yml when we made the switch in the first place. We centrally administer Flowzone, but expect maintainers to manage their own flowzone.yml. I think it's a good precedent to maintain, since being aware of and having control over of your CI/CD is important when crafting PR's to suit.

As far as the file in this PR is concerned, I prefer the original content (but with a corrected URL!) since docs on using docusaurus-builder ought to live with docusaurus-builder, which also avoids redundancy. I expect the Flowzone docs just to say Flowzone supports it and alternatives, and to define each "input" (funny name, btw).
On the separation of cloudflare/docusaurus, I agree cloudflare_website is a poor name because Cloudflare should be a fungible implementation detail, unlike docusaurus which the user must design for. But again, perhaps slightly out of scope for this PR. I don't mind "Cloudflare" appearing in the docs here, since the detail is important to know and this is a reasonable place to mention it.


Not enough buzzwords

This project does not contain enough buzz words. In particular I'm missing blockchain, VR, and AR.
Please fix this as soon as possible!

you can provide some code samples to show how something of your mentioned buzz words is working ;-)


Refer to other packages from same pkgsDirectory

Packages that try to use other packages in the same pkgsDirectory fail with the error:
$ nix build .#b
error:
       … while evaluating the attribute 'optionalValue.value'
         at /nix/store/x3d704r2qzq7h0m7i04liyzrvnng1pi2-source/modules.nix:868:5:
          867|
          868|     optionalValue =
             |     ^
          869|       if isDefined then { value = mergedValue; }

       … while evaluating a branch condition
         at /nix/store/x3d704r2qzq7h0m7i04liyzrvnng1pi2-source/modules.nix:869:7:
          868|     optionalValue =
          869|       if isDefined then { value = mergedValue; }
             |       ^
          870|       else {};

       … while evaluating definitions from `/nix/store/714g6clapigzz83jh4gb8r4q9zzndk10-source/modules/transposition.nix':

       … while evaluating definitions from `/nix/store/smkvydkb12kwq35jxf1kh9g7h49z1cnq-source/flake-module.nix, via option perSystem':

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: evaluation aborted with the following error message: 'lib.customisation.callPackageWith: Function called without required argument "a" at /nix/store/p18pwjzr1bywzb84md8ira3z3c08ssdf-source/pkgs-by-name/b/package.nix:1, did you mean "R", "a4" or "ao"?'

With the repo set up like this:
└── pkgs-by-name
    ├── a
    │   └── package.nix
    └── b
        └── package.nix

a/package.nix

{ hello }: hello

b/package.nix

{ a }: a

Possibly the issue is it isn't getting callPackage from itself like https://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/bsd/freebsd/package-set.nix#L13-L18

Let me know if the proposed solution fits for you.

No feedback so far, feel free to reopen if this is still an issue for you.

Yep that works for me 👍


Implement View Resume

Under each cohort member's quote on their card, a button with a link to their resume can now be found.

the change you made look great! just fix the conflicts since i updated the classmate images while you were working


Add jit request API and schema.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6185
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6185/commits/a5e64370baf2986d86db33db077398b7312b62a6

Automation for azure-sdk-for-java

The new swagger isn't referenced in any README.md file so it won't get included in any SDK. Is this expected?

Suppressions are added only if there's a linter error that can't be fixed for some legitimate reason, so no need to add anything there at present.

There are model validation failures that need to be fixed, please see the log here.

There are model validation failures that need to be fixed, please see the log here.

Actually none of those warnings were add my new API or schema. Or is there anything needs to be added for existing apis in new version?

They will need to be cleaned up at some point, if you don't want to clean them up now (e.g. tight deadline) we can skip it however can you please open an issue to track fixing them?

Requested for description .. updating with old and new apis contents to see the diff easily.

@dandanwang0320 hello!  Any update here?  Also there's a merge conflict that needs to be resolved.

@dandanwang0320 - please let me know when this is ready for re-review.

@KrisBash Hi, would you have time to take another look?

@openapibot sdkautomation rebuild

@dandanwang0320 hello is this ready for review by ARM?

@dandanwang0320 can we close the PR for now since you're not ready for review, and the PR is over several months. Pls create new PR when you're ready for review and merge.

Closing due to inactivity.


Identify initial controls for Istio Demonstration

Source: https://repo1.dso.mil/platform-one/big-bang/apps/core/istio-controlplane/-/blob/main/oscal-component.yaml
Identify and document controls from the above that can be queried in a meaningful way.
What does the query look like?
Can we demonstrate injecting a change that validates/invalidates this control? (its fine if the answer is no - this is still valuable information)

[ac-4](https://repo1.dso.mil/platform-one/big-bang/apps/core/istio-controlplane/-/blob/main/oscal-component.yaml#L139 looks like a probable target for creation of an initial query.
Provenance does not have to be %100 for Proof of Concept. Instead targeting the following should be sufficient:

Check that all pods are istio-injected (excluding kube-system namespace)
Ensure each namespace contains a PeerAuthentication resource with mtls.mode: STRICT

Implemented in this pr


Help use BingMapService

how to use BingMapService in an application, more exactly, how can you explicitly use the method this.mapService.GetCenter () or LocationToPoint

Hy
1.Wait for the  (MapService)="_processMapService($event)" event to emit from the map, so the service is initialised
2. You can call the mentioned fuction on the service which was emitted


Fix GpuCast losing the timezoneId during canonialization

Fixes https://github.com/NVIDIA/spark-rapids/issues/5670
The timezoneid setting was being lost during canonicalization on GpuCast which ultimately caused a parse error to occur because during plan canonicalization we end up trying to access a canoncialized Alias which will end up trying to do a parse if it is not resolved.  The GpuCast wasn't resolved because it didn't have the timezone id.
The fix is straight forward and we had a comment there to add in the needsTimeZone check.  Basically we were intentionally nulling it out when we shouldn't have been because we weren't checking needsTimeZone
I want to try to add a test case to the integration tests but need to find minimal repro cast, putting up here for early review and run tests and get in asap.  I can always followup with test.
Signed-off-by: Thomas Graves<EMAIL_ADDRESS>

build


Follow the patterns especified in the backlog

The API must follow the patterns  given by the proffesor in this URL: https://docs.google.com/document/d/1rJS79hEAhGGXx-tz2ThcriZ8knUj4QUGfeuwDoVNFn0/edit


Increase test coverage

Unit tests should be added or modified based on the following analysis of the attached coverage report.
If you would like to contribute, you do not have to increase coverage for all files. Feel free to add a comment to this issue stating which one you would like to pick up and add changes for. When the changes are done, make a pull request with those changes and reference this issue in the pull request comments.
This issue will be closed once coverage on all files are completed.
Coverage Report
To get the coverage report run the following commands from the root directory of the packageless project on git branch main:
go test --coverprofile=coverage.out ./...
go tool cover --html=coverage.out

Analysis of Coverage Report
github.com/everettraven/packageless/main.go does not need modification
github.com/everettraven/packageless/subcommands/install_sc.go

If possible, tests to test the logic based on different GOOS values should be created.

github.com/everettraven/packageless/subcommands/run_sc.go

Tests should be added to test that the current working directory is being added to the volumes array if a volume path is equivalent to ""

github.com/everettraven/packageless/subcommands/subcommand.go

Test should be added to test that the error is being throw properly if the length of the args variable is < 1

github.com/everettraven/packageless/subcommands/uninstall_sc.go

If possible, tests to test the logic based on different GOOS values should be created.

github.com/everettraven/packageless/subcommands/update_sc.go does not need modification
github.com/everettraven/packageless/subcommands/upgrade_sc.go

Test should be added to test that the error is being thrown properly if the pim config file does not exist

github.com/everettraven/packageless/subcommands/version_sc.go does not need modification
github.com/everettraven/packageless/utils/alias_unix.go does not need modification
github.com/everettraven/packageless/utils/alias_win.go does not need modification
github.com/everettraven/packageless/utils/docker.go

Test should be added to cover the case when the length of variable splitVol is != 3

github.com/everettraven/packageless/utils/hcl_parse.go

Test should be added to cover the case when an unexpected type is passed to the ParseBody function

github.com/everettraven/packageless/utils/mocks.go does not need modification
github.com/everettraven/packageless/utils/utils.go does not need modification

I would like to give this a shot!

@kastolars Go for it! Are you interested in taking on the whole issue or an individual file that needs coverage increased?

@everettraven Hi yes I will review the code and post here which file(s) I think I can cover tomorrow.

@everettraven Hi yes I will review the code and post here which file(s) I think I can cover tomorrow.

@kastolars Sounds good. Let me know if you have any questions

I have "Test should be added to test that the error is being throw properly if the length of the args variable is < 1" for /subcommand.go complete

Next: Test should be added to test that the error is being thrown properly if the pim config file does not exist in upgrade_sc.go

Next: Tests should be added to test that the current working directory is being added to the volumes array if a volume path is equivalent to "" in run_sc.go

Haven't started ^ yet, got kind of busy. Will work on it soon as I can.

@kastolars I merged in your PR, great work on the test changes! Just wanted to mention you are welcome to take as long as you need on any contributions!

Hi @everettraven.
After the merged PRs from @kastolars , is the Analysis of Coverage Report section above still up to date?

Hi @everettraven.
After the merged PRs from @kastolars , is the Analysis of Coverage Report section above still up to date?

Hi @neverbeenthisweeb here is an updated one:
Updated Analysis of Coverage Report
github.com/everettraven/packageless/subcommands/install_sc.go

If possible, tests to test the logic based on different GOOS values should be created.

github.com/everettraven/packageless/subcommands/uninstall_sc.go

If possible, tests to test the logic based on different GOOS values should be created.

github.com/everettraven/packageless/utils/docker.go

Test should be added to cover the case when the length of variable volSplit is != 3

github.com/everettraven/packageless/utils/hcl_parse.go

Test should be added to cover the case when an unexpected type is passed to the ParseBody function

@everettraven
Got it.
Btw for the docker.go part, I think that should be splitVol instead of volSplit because I can't find any volSplit in the code base (but I can find the splitVol in the RunContainer method).
Wdyt?

@neverbeenthisweeb Oh, thanks for pointing that out! You are correct, going to fix the typo now.

Typo fixed in my previous comments

Got it, thank you @everettraven.
Anyway, I've just read the Docker documentation related to volumes.
It looks like, referring to our code base, Docker only accept either 2 or 3 splitVol:

The first one is source
The second one is target
The third one, which is optional, is an option

In our code base, docker.go#114, it is still possible for user to provide a volume with 1 or >3 splitVols.
Example of 1 splitVol case: /a/path
Example of >3 splitVol case: /a/path:/b/path:abc:def
My question is, should we validate the value volumes passed to RunContainer args to have only either 2 or 3 splitVol? Or it is not the responsibility of RunContainer method? (so perhaps our user should be aware of this case of only volumes of 2 or 3 splitVol that is valid).
This is the error that I have when passing volumes with splitVol value of 1 ([]string{"/another/path"}) of to RunContainer:
panic: runtime error: index out of range [1] with length 1 [recovered]
	panic: runtime error: index out of range [1] with length 1

Maybe we could throw a more user friendly error like: Error: Invalid volumes, should at least have source and target.
Wdyt?
Thank you.

@everettraven Actually I just realized the case of splitVols value of 1 is still considered as a valid cmd by Docker.
I ran some command similar to this, and Docker still executes this.
$ docker run -d \
--name devtest \
-v myvol2 \
nginx:latest

However, I don't know the mounting behavior of this case (still trying to understand).

@neverbeenthisweeb
First of all, thanks so much for all the research and thought you have put into this!
I definitely like the idea of throwing a much more friendly error.
I'm not entirely sure if we will ever hit a case where the volume isn't controlled by the logic within packageless since the volumes are dictated by the pim configuration files (can be found at packageless-pims). As of now it only accepts a source and target field and not the extra volume options, which is something I am now thinking should be allowed.
The volumes defined in the pim configuration files is what gets parsed and passed to the RunContainer function when running something like packageless run python

@neverbeenthisweeb You should be able to see the logic that goes into creating the volume based on the parsed pim configuration file here:
https://github.com/everettraven/packageless/blob/main/subcommands/run_sc.go#151

@everettraven
I see, sorry I haven't tried packageless app fully, so I don't know about that packageless-pims.
If that's the case, I think we can just assert, whether it will be an error or not for each of splitVols != 3 cases.
Something like:
_, err := util.RunContainer(image, ports, volumes, cName, args)
if err == nil {
    t.Fatalf("RunContainer: Expected to receive an error, but did not receive one.")
}

Wdyt?
If you're ok, I can create the PR and can continue the discussion there.
Thanks.

@neverbeenthisweeb No worries on not having tried the app, I'm always happy to help answer any questions about it.
I think that is a good approach so  open up a PR whenever you have those changes and I can review it!

@everettraven
Got it, thanks for the explanation.
Anyway, I've created the PR in #65.
But I forgot to assign you as the reviewer.
Can we add reviewer when the PR is already created here in Github? Sorry I am new to contributing in this platform.
Thank you.

@neverbeenthisweeb Yes, you should be able to request my review but I  can go ahead and add myself on there

Hi @everettraven
Anyway, I tried to resolve this issue:
github.com/everettraven/packageless/utils/hcl_parse.go

Test should be added to cover the case when an unexpected type is passed to the ParseBody function

in #66.
Please LMK if there is anything I can help with that PR.
Thanks.
Btw, I just read an article in stackoverflow, and it looks like only collaborators that are able to assign reviewers. Maybe this is the reason why I can't assign you as reviewers previously :grin:.

@neverbeenthisweeb Oh gotcha, that makes sense. I will take a look at the PR now!

Most of the tests in the original issue have been changed to increase coverage so I will be closing this issue. I may open new issues regarding coverage on certain tests in the future.
A huge thank you to both @kastolars  and @neverbeenthisweeb for all your help with this issue!


Fix for "GeographyDirectory" file path parameter for IPlocation

Added a fix to convert "geographyDirectory" parameter value to absolute path when necessary. This parameter now can take both relative and absolute path.

The 'geographyDirectory' parameter definitely needs to handle both relative and absolute pathnames, but I'm not sure this is quite the right way to do it:
  if(filepath.is_relative()) {
    filepath = streams_boost::filesystem::absolute(filepath, getPE().getDataDirectory());
  }

The 'data' directory is optional in Streams release 4, isn't it? If so, what does getDataDirectory() return if the application was compiled without the '--dataDirectory' parameter, and the directory does not exist?
Relative pathnames should certainly be resolved into absolute pathnames at runtime, but I think they should be relative to the current working directory. That could be the 'data' directory, the Eclipse project directory, somewhere in /tmp, or someplace else entirely. I think this code will accomplish that:
  if(filepath.is_relative()) {
    filepath = streams_boost::filesystem::absolute(filepath);
  }

but you should verify that ...
-- Ed

I think the question here is, what are the geography files? Are they data files that the user may want/need to update? Or are they static files (similar to a configuration file) that will never change during the lifetime of the operator?
If they are data files, then making them relative to the data directory by default is standard practice. If they are configuration files, then they should be relative to the toolkit directory. In the later case, the files should placed in the 'etc' folder so they get added to the application bundle.

Customers must download geography data from Maxmind Inc. (see http://dev.maxmind.com/geoip/geoip2/geolite2/ for the free version, which is updated monthly, and https://www.maxmind.com/en/geoip2-city/ for the priced version, which is updated more frequently).
I'm not sure I understand your comment about application bundles and the 'etc' directory. Geography data should be included in application bundles, whatever directory its in, since its required by the operator at initialization.

If the geography data is included in the application bundle, that implies that the data is static and will never change. In order to update the geography data, the user would need to download that data into their application project, recompile the application in order to generate a new bundle and then relaunch the application. Or to put it another way, a user would need to take their application offline in order to update the geography data stored in the application bundle.
By putting the geography data in the 'data' directory, this makes it possible for the user to update that data without interrupting their application (assuming of course that the operator is capable of supporting dynamically updating this data). The caveat here is that you need to ensure that the application is compiled with a data directory.
My point about the 'etc' directory is, if it's decided that the geography data should be included in the application bundle, then this data should be placed in the 'etc' directory, which is included in the application bundle by default.

OK, now I understand the distinction between the 'data' and 'etc' directories. That does create an ambiguity for geography data, though. For most customers, this sort of data is static -- the developer will download the free data from Maxmind once and never change it. But I can imagine applications that might need the latest updates to the Maxmind data, and customers that would be willing to pay for it.
So I think we should recommend that geography data be placed in the 'data' directory (or some other directory outside the Eclipse workspace) and referenced with an absolute pathname. That's what all of the sample applications in the toolkit do.
I think we should also have an example of an 'info.xml' file that bundles the data directory into the application bundle for situations that need that, like cloud deployments.

We don't necessarily need to provide an example of how to bundle the data directory. The issue here is which directory are relative paths relative to. I agree that making the data directory the default relative path makes sense for these files.
For cloud deployments or cases where the user wants to ensure the geography data is included in the application bundle, they can still put the files in the 'etc' directory (for example) and refer to it using the 'getThisToolkitDir()' function, which returns the absolute path to the root of the application directory within the bundle.
For example, if the geography data was stored in the 'etc' directory, a user can add that path to their application using the following code snippet:
  param
    geographyDirectory : getThisToolkitDir() + "/etc/" ;

Hi Edward
I tested the following code change you suggested but it did work . If we use a relative path for the geography directory such as "../dir1/dir2/<geodata-file.csv>, the operator will not be able to find it and an exception will be thrown.
if(filepath.is_relative()) {
filepath = streams_boost::filesystem::absolute(filepath);
}
filepath = streams_boost::filesystem::absolute(filepath, getPE().getDataDirectory()); This code change works for both absolute and relative path  for example : (../../<geodata.csv> or /dir/dir/<geodata.csv>)
Thanks

OK -- if that works, then let's go with it!

Sounds like we are all in agreement on this. If there are no concerns, I will merge in this pull request.
@kanduibm Please open an issue for this problem and reply back with the issue #. I will merge this request once I get the issue #.

Fixes issue #37


Stacked Area Chart Weirdness in 2.1

I'm using a stacked area chart with just a single data source. This has always worked in all versions prior to 2.1 — I like this over the line chart because it provides a nice background fill. After upgrading to 2.1, the chart now renders a bit of weirdness.
Expected Behavior
This is how the chart rendered on 2.0, before upgrading, and what I was expecting:

Current Behavior
This is how the chart now renders on 2.1, after the upgrade:

Possible Solution
I'm not sure what changed with the stacked area chart rendering, but something seems a little off? I don't have any suggestions for a solution.
Steps to Reproduce (for bugs)
I'm not doing anything differently from what the documentation says to do, and my dataset matches the expected dataset format. The only change that caused this bug was upgrading from 2.0 to 2.1 😕
Your Environment

Version used: 2.1
Browser Name and version: Latest Safari/Chrome
Operating System and version (desktop or mobile): macOS Sierra
Link to your project: [private]

Hi @rpheath , thanks for the issue!
It looks like a CSS problem. In this release we are adding a new element on the stacked area, so maybe that is what is failing.
Try to update the CSS styles and let me know if that fixed it.

@Golodhros thanks, yes that was it! I updated the CSS from the release but didn't realize there was a linting issue and it didn't build my scss... Sorry about the false alarm! All's well now, thanks a lot 👍

No problem @rpheath, happy to help!


POS remote terraform context and housekeeping

Background
POS currently uses Terraform for deploying the VMs that will be used for building the Clickhouse and Elastic Search images.
The current approach uses a local Terraform context, which may result in the loss of this information, and the need for manually cleaning up its resources.
As an operator that uses POS for several tasks within the context of an Open Targets release, I’d like POS to use a remote terraform context and implement house keeping automations on this context and all possible contexts that may have been left unattended.
Tasks

Setup the hosting bucket for the different POS contexts
Write context automatic use / creation
Write context housekeeping actions (current context, bucket folders, etc.)

This is now implemented in opentargets/platform-output-support#18


DJV Requests Ignored

I made numerous requests for version 2.x of DJV view but it appears they were all ignored.

Hi, what were you exactly requesting and where?

I wrote it under Issues.

Hi, it certainly wasn't intentional; a lot of work went into the new version and there is still a lot more to do. Can you link to the requests you made? Was it on GitHub or SourceForge?
Darby

It was on SouceForge.

What is the link to the DJV on Source Forge ?

https://sourceforge.net/p/djv/_list/tickets

I also made copies on GitHub of all the open issues on SourceForge so they may also be here:
https://github.com/darbyjohnston/DJV/issues

Here, these are the issues I was hoping would be resolved in 2.x.

Also the old scrubber from version 1.0; I'd like to see brought back, it was more effective then current version.

Could you be more specific? I just compared scrubbing between version 1.3 (ctrl+left mouse) and 2.0.5 (right mouse) on a 700 frame sequence and it felt similar. Or do you mean the buttons for scrubbing that were on the timeline in 1.3?

How do I write an annotation; I can create the Annotation window, but I can't write anything ?

Annotations are still a work in progress for 2.0, they will hopefully be added in the next version.

There is no option of loading a custom LUT in DJV ?

DJV supports loading LUTs through OpenColorIO; I have not tried adding a custom LUT to OCIO, there might be information about that here:
https://opencolorio.org/
Once you have created a custom OCIO configuration this is how you can add it to DJV:
https://darbyjohnston.github.io/DJV/djv_color.html

Also the split window issue I mentioned here;

Window snapping would be useful, I'll add a GitHub issue for it but cannot promise when it will be implemented:
https://github.com/darbyjohnston/DJV/issues/168

Plugging into DCC applications, yes; I mean, Maya, Houdini, Blender, Max

I've created issues for each of these, the only one I have access to currently is Blender so again I can't promise when they will be implemented:
https://github.com/darbyjohnston/DJV/issues/30
https://github.com/darbyjohnston/DJV/issues/169
https://github.com/darbyjohnston/DJV/issues/170
https://github.com/darbyjohnston/DJV/issues/171

Or do you mean the buttons for scrubbing that were on the timeline in 1.3?

The buttons which were on the timeline in 1.3; I would like to see them brought back.

Annotations are still a work in progress for 2.0, they will hopefully be added in the next version.

I look forward to this. As well as Issue #168.

I created an issue for adding scrubbing buttons to the timeline:
https://github.com/darbyjohnston/DJV/issues/175
I'm going to close this issue since it is overly broad and all the specific feature requests now have issues associated with them.


Lots of errors trying on FreeBSD 13.0-RELEASE-p4

zig build
zig build-simple
both don't produce something usefull.

Zero for effort. Closing.

I have some logs :),
zig build returns:
zig build                                                                                0
Warning: error.FileNotFound. The cfltk library will be grabbed and built from source!
-- Configuration Summary for FLTK 1.4.0 generated by CMake 3.21.3 --
-- The following OPTIONAL packages have been found:

Doxygen
LATEX

-- Static libraries will be built in /home/x/Dirs/Src/Zig/fltk/zfltk/vendor/bin/fltk/lib
-- Shared libraries will not be built (set OPTION_BUILD_SHARED_LIBS=ON to build)
-- Example programs will not be built (set FLTK_BUILD_EXAMPLES=ON to build)
-- End of Configuration Summary --
-- Configuring done
-- Generating done
-- Build files have been written to: /home/x/Dirs/Src/Zig/fltk/zfltk/vendor/bin
Consolidate compiler generated dependencies of target fltk_z
Consolidate compiler generated dependencies of target cfltk
[  0%] Building CXX object CMakeFiles/cfltk.dir/src/cfl.cpp.o
[  2%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_window.cpp.o
[  5%] Built target fltk_z
[  5%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_dialog.cpp.o
[  6%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_draw.cpp.o
[  6%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_menu.cpp.o
[  7%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_font.cpp.o
[  7%] Building CXX object CMakeFiles/cfltk.dir/src/cfl_surface.cpp.o
/home/x/Dirs/Src/Zig/fltk/zfltk/vendor/cfltk/src/cfl_window.cpp:10:10: fatal error: 'X11/Xatom.h' file not found

zig build run-simple returns the same error

Xatom.h can be found in freebsd in /usr/local/include/X11

If you're using zig as a C compiler, it might not be searching /usr/local/include. It's also better to use the system's default compiler.
Searching online, Freebsd also seems to have this issue in general:
https://wiki.freebsd.org/WarnerLosh/UsrLocal
Anyways, you can try:
$ export CPATH=/usr/local/include
$ export LIBRARY_PATH=/usr/local/lib

And see if this resolves the cfltk build step.


GATConv with segment fault

❓ Questions & Help

I run the benchmark/citation/gat.py, and I get segment fault, what's wrong?
What I get is as follows, can somebody help me?
 Can you re-install it and see whether an error occurred?


[Lecture] Mythology, Alchemy, and Philosophy in Video game lore and plot: Final Fantasy series

[Lecture] Mythology, Alchemy, and Philosophy in Video game lore and plot: Final Fantasy series
If you want to attend this lecture, please vote by reacting with 👍 in the issue.

Goals: Help people understand and create deeper and more complex videogame stories
Tools Required: None
Knowledge Required: None

Timeline Structure
Introduction (5 minutes)

Introduction to Final Fantasy
Introduction to Videogame story creation

Final Fantasy (30 minutes)

Review of the stories of all mainline games

Meaning Stories and Writing (30 minutes)

How to apply what makes Final Fantasy stories great into your own stories

Closing (5 minutes)

Thank everyone for participating
Remind them of the next session or other opportunities to practice and continue learning.

👍


how to use it in datagrip?

couldn't find where to add a Mongo Server in datagrip, pls help

@JeffcottLu when u installed the plugin ,and restart datagrip ,then u can see the button 'Mongo Explorer' on the right side

@JeffcottLu when u installed the plugin ,and restart datagrip ,then u can see the button 'Mongo Explorer' on the right side

thx, i've found this


User-defined functions with resource-provider types

Is your feature request related to a problem? Please describe.
User-defined functions do not allow resource-provider types in parameters or as the return type. Currently I can only reference user-defined types or built-in primitives (object, string, int, bool, array) in this context.
Using Bicep 2.7.0 PS module and ms-azuretools.vscode-bicep version 0.32.4.
Describe the solution you'd like
I want to be able to define functions like these and benefit from real-time intellisense support in VS code as well as compile-time schema checks (instead of waiting for ARM deployments to fail):
func urlRewriteRules(...) ApplicationGatewayRewriteRule[] => [ ... ] or
func firewallRule(...) WebApplicationFirewallCustomRule[] => [ ... ], etc.
The referenced ApplicationGatewayRoutingRule and WebApplicationFirewallCustomRule types.

There's an experimental feature that allows this called resource-derived types. It can be enabled by adding a bicepconfig.json file to a project that looks like the following:
{
    "experimentalFeaturesEnabled": {
        "resourceDerivedTypes": true
    }
}

In the bicep file, you can use a resource-derived type anywhere a type expression is expected. These types are communicated in terms of where they appear in resource declarations:
func urlRewriteRules<EMAIL_ADDRESS>=> [ ... ]
func firewallRule<EMAIL_ADDRESS>=> [ ... ]


feat (ui): Add Native Histogram rendering to new UI

Adds native histogram to new ui
Some changes:

Bucket table underneath the histogram is scrollable
Now hovering the entire DataTable.tsx component doesnt highlight it

https://github.com/prometheus/prometheus/assets/14011425/a8b7b8db-b3ca-4fcb-b40e-9dcb77bb683e

I've applied all the suggested changes except the one with the floating weird line

Nice, I think this is mostly ready to merge for now, we just have to deal with this CI lint check failing:
/__w/prometheus/prometheus/web/ui/mantine-ui/src/pages/query/DataTable.tsx
  205:14  warning  Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components  react-refresh/only-export-components
ESLint found too many warnings (maximum: 0).

Maybe just remove the export, since the histogramTable function isn't used outside of that file anyway?

Maybe just remove the export, since the histogramTable function isn't used outside of that file anyway?

done!

Yay, thanks! :) :+1:


Adding gits branch workflow

Same as 'git branch'. Basically list all the branches in the repository

Implemented and move to Done


sql/index: auto delete corrupted indexes

Closes #220

Sounds good to me

we can reuse an empty bolt db file. moreover bolt set a lock till is opened

@ajnavarro an issue I see with this approach:
We can know if the index was created or not by checking the CREATED file.
But if it's not created, the index is corrupt and we have to delete it. Then we need to read the config, which may fail. What does it happen if reading the config fails? If we don't read the config we can't delete the index because we need that data for removing the pilosa stuff.
After all, it's the same logic as before, adding the extra step of having to check for the CREATED file, which is overcomplicating the thing. The only part that is simplified is the part of writing the file at the end of the index save.
I mean, we can use CREATE, but we still need to think what to do with config read files and it's basically the same logic with extra files.
Create should not be idempotent because an index cannot be overwritten, only deleted and recreated, right? Or am I missing something?

@erizocosmico why do you need to read the config file if the index is corrupt? Just ignore it on LoadAll method, and when someone call again Create, override all the files.

So you mean we should leave everything there and wait for recreation? Wouldn't it be better to delete all the garbage that a partial index may have written (which potentially can be a lot)?

So that's why I wanted to reuse boltDB file as a lock. It can give us double-check.

if file exists
can you open (if not most likely lock is still set)

Changed the logic

@kuba-- PTAL


Question: is --- in prompts.txt for one conversation many prompts, or many conversations one prompt?

Please forgive me for a noobish question, but I am questioning my understanding of something.
Within prompts.txt, is --- designed for:

Delimiting within one conversation (one conversation, many prompts)
Delimiting across conversations (many conversations of one prompt)

To reword, from https://promptfoo.dev/docs/configuration/parameters#prompt-files:
Translate the following text to French: "{{name}}: {{text}}"
---
Translate the following text to German: "{{name}}: {{text}}"

Is this like having a two chat conversation, or two separate one chat conversation?

It's split into multiple prompts.  It's as though you had two prompt files containing each line.

Thank you!  Opened a docs PR to clarify this.


[FEATURE]:  improvement in the flash message

Need to implement the custom flash message to the user after doing some thing like login and logout etc.

Hey @venurockss ,
Custom flash messages are already available. Could you please elaborate on what specific messages you'd like to display and when you'd want them to appear?
Thanks!

I mean that custom flash message is not designed properly

@venurockss , you can proceed. I'm assigning this to you.
please specify whether you're part of any open source program. so that I can apply necessary labels

@Vignesh025 add  appropriate labels gssoc-ext

@Vignesh025 add appropriate labels for the issue
Program is gssoc-ext


NaN cannot be used as a key in Map

The primary motivation forMap existance is implementation of Vector.distinct. To make such implementation easy/fast/possible we don't want errors from Map.insert. On contrary, we want every object to provide hash code - so it can be used as a key for the Map. Constraints:

it is desirable for Map to always satisfy: m.insert k v . contains_key k == True
For Number we are going to use JavaScript's Equality comparisons and sameness to ensure NaN works as a key
Custom types - shall have a forced reflexivity - e.g. Meta.is_same_object implies that they are Any.== which in turn satisfies the first point

Following code shows the expected behavior for custom types that violate reflexivity:
type My_Nan
    Value comment:Text

...comparator such that it always returns Nothing

k = My_Nan.Value "foo"
k2 = My_Nan.Value "foo"
(k==k).should_be_true
(k==k2).should_be_false
m = Map.empty.insert k 10
m.contains_key k . should_be_true
m.get k . should_equal 10
m.contains_key k2 . should_be_false

m2 = m.insert k2 20
m2.get k . should_equal 10
m2.get k2 . should_equal 20
m2.size . should_equal 2

m3 = m2.insert k 30
m3.size . should_equal 2
m3.get k . should_equal 30

e.g. , we will do a best effort solution. Thanks to Any.== relying on Meta.is_same_object, even if compare implementation is always incomparable, x==x will be True. Multiple separate instances of incomparable atoms will however not be.
Follow up of

#5771
#5818

### Tasks
- [ ] Make sure `NaN` can be a key in `Map` using the JavaScript's [Equality comparisons and sameness](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Equality_comparisons_and_sameness#same-value-zero_equality)
- [ ] Make sure `x == x` via `Meta.is_same_object`
- [ ] Update tests
- [ ] Update documentation

Probably also worth adding a test for Vector.distinct:
[Number.nan, 1, Number.nan, 2, 1].distinct.to_text . should_equal "[NaN, 1, 2]"

(using to_text as regular equality may actually yield False due to the NaNs...)

Let's be sure to test the custom types properly, i.e. have tests for a custom type like My_Nan above.
As @Akirathan mentioned, this may be blocked by #5709


PostgreSQL Array type search keywords ANY/ALL not working

Hi, I am trying to generate a query for array search which requires the literal to be on left hand side of the expression and I am getting below error
no column reference found
Is this not supported or I am making some mistake.

@dharmjit could you include a sample query and table that causes the issue? It's difficult to debug this issues without it.

Hi @kyleconroy , Below is the table structure and query I want to generate
CREATE TABLE IF NOT EXISTS applications (
    application_id VARCHAR NOT NULL PRIMARY KEY,
    application_name VARCHAR NOT NULL,
    application_version VARCHAR NOT NULL,
    application_owner VARCHAR,
    scope VARCHAR NOT NULL,
    created_on TIMESTAMP NOT NULL DEFAULT NOW()
);

And I am using query with dynamic filter as below
-- name: GetApplicationsView :many
SELECT a.application_id,a.application_name,a.application_owner
FROM applications a LEFT JOIN applications_instances ai ON a.application_id = ai.application_id
WHERE (CASE WHEN @is_product_id::bool THEN @product_id = ANY (ai.products) ELSE TRUE END)

Thanks for the SQL. I was able to reproduce the result. To get this to work, you need to cast @product_id to text. Here's a link to a playground for your example.
-- name: GetApplicationsView :many
SELECT
  a.application_id,
  a.application_name,
  a.application_owner
FROM applications a
LEFT JOIN applications_instances ai
  ON a.application_id = ai.application_id
WHERE (
  CASE WHEN @is_product_id::bool
       THEN @product_id::text = ANY (ai.products)
       ELSE TRUE
  END
);

The reason you need to add the cast is that sqlc's type inference is still very limited. It doesn't know the type of any(ai.products) should be text. I'm working on those improvements, but they won't land for a few more releases.


feat: decode_in_slice()

For memory constrained enviroments and network protocols being able to use memory without extra copies is a benefit. E.g. in my ZMODEM2 crate I need to do this for hex messages:
if encoding == Encoding::ZHEX {
    let mut tmp = [0; (FrameHeader::unescaped_size(Encoding::ZHEX) - 1) / 2];
    match hex::decode_to_slice(out, &mut tmp) {
        Ok(x) => x,
        _ => {
            log::error!("from_hex error");
            return Err(ErrorKind::InvalidData.into());
        }
    }
    out.clear();
    out.extend_from_slice(&tmp);
}

It is good example since the protocol frames can come also in binary format in addition to hex. It would be nice to do this preprocessing within the same buffer as where the data landed to make a transparent zerocopy flow.
Link: https://github.com/jarkkojs/zmodem2/

Asymmetry is actually cool here. There's much less use for encode counter-part as you don't end up doing extra copies with current functions, as you build your buffer from zero. So I'd consider this without encode_in_slice. It is also less nice in the sense that it needs more space than the original.

I adapted it to my zmodem2 crate:
        if encoding == Encoding::ZHEX {
            hex::decode_in_slice(&mut out).or::<io::Error>(Err(ErrorKind::InvalidData.into()))?;
            out.truncate(out.len() / 2);
        }

I'd like to make this scale from microcontroller to xeon so thus I'm proactively interested on saving in this case 7 byte of stack space.


Does placement take PortLocation constraints into account?

PortLocation constraint could be leveraged during placement as it impacts the wire length for the specified port.

It is not considered now. We can add them. We can create a PR if needed.

Yes, I will create that PR shortly. Let's first fix the ordering issue and finish hardening symmetry.


Attribute translate permission not working

What I'm trying to achieve
Translate an attribute using attributeTranslate mutation using app token (app with full permissions). The mutation returns don't have permission error.
Steps to reproduce the problem

Create an app with full permissions and activate it.
Create an attribute using dashboard (or whatever).
Try to translate the created attribute using app token in graghql playground using attributeTranslate mutation.
The mutation returns error: you don't have permission

What I expected to happen
The translation to be added
Screenshots and logs

System information

Saleor version:

[ ] dev (current master)
[ ] 3.0
[✓] 2.11
[ ] 2.10

Operating system:

[ ] Windows
[✓] Linux
[ ] MacOS
[ ] Other

Hi @Mowafy, you give me more information, because I cannot reproduce it.
When I use app which has full rights then everything works fine.

Hello @Mowafy this issue has been corrected in version 3.0 and we advise to use this version. If you need more information please let me know.


Update the Tink dependency to 1.14.1

This is the newest Tink.
We do this upgrade because I want to limit usage of certain methods of Tink within Google. These methods will afterwards be deprecated open source, and then maybe eventually removed.

This change is


Investigate why HPC Jupyter Server isn't working

The Jupyter Forward and Custom Server Script options for launching compute environments do not work for me. We need to investigate why these might not work for a user and find a resolution. I'm happy to get on the phone and troubleshoot with you about this if you cannot replicate.

This is not "urgent" but fairly high priority, as we want users to be able to run these notebooks from Denali if they need to.

can you add more detail about what's working, what isn't, etc...


docs(clean): 'clean userId amount' is invalid

This PR adds a warning to the description of the clean command making it clear (excuse the pun) that the combination of clean userId mention is invalid. This has been a source of confusion for many in the support server who have encountered the issue, especially as the auto-generated usage implies this combination is valid, when in fact it is not.

Clean <User:Mention/ID> <Num:Whole number>

It is arguable whether this is a good way to resolve the issue - should the code be changed to handle this specific combination instead? In any case, this needs to be fixed or made clear in some way, hence the PR.

don't rush your PRs to master branch

Oops, forgot about that. Will open a new PR.


New holds a strong reference to instances created with events/property change handlers/bound state if they're not explicitly Destroyed

I was originally planning to solve this issue pre-initial release, but my initial assumptions about it were false. Luckily, as long as end users always explicitly destroy their instances when they're done with them, this is a non-issue.
I already tried implementing an experimental solution to get gc to work correctly with New (toggled by ENABLE_EXPERIMENTAL_GC_MODE) however this causes some instances to get collected too early. I'm planning to simply redo a lot of this code at some point since it's gotten a bit messy as the requirements have piled on.
I'll return to this issue when I'm not running on low sleep lol.

Note to self: this could possibly be caused by children not having references kept to them, which could maybe cause problems where New calls are used directly as children and nothing else? Needs further investigation.

This is fixed by #105 - closing!


Output Modes

I choose the output Style "compressed" but every time that a make a change in the .scss file, the output renders a random between compressed, nested, compact and extended.
I would like to have the same style in every save, any ideas about this issue?
tks

@fredcerdeira @impressivewebs @winghouchan
Scout-App is being maintained again and has a new version.
Issue is resolved. Closing issue. Download the latest version of Scout-App.

Scout-App 2 (2016)

Good to know, thanks for letting us know. Glad it's being worked on again.

@impressivewebs oh hey, you're quoted on the site

http://scout-app.io/#quotes

That section is actually currently being redesigned by @kimmihack

http://i.imgur.com/gpEGWaK.jpg

@TheJaredWilcurt Nice, thanks for letting me know about that, I hadn't seen that.
FYI - I do a newsletter called Web Tools Weekly, so I'll be sure to include the new version in an upcoming issue.

@impressivewebs Awesome, let me know when it comes out, I've been checking the archive on the site.

yay! http://webtoolsweekly.com/archives/issue-163

Haha, yeah, I just happened to have enough links for a "Sass" list, so it worked out. :)


Distribute parameter data files as separate conda package

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Failed to even lint the recipe, probably because of a conda-smithy bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint . from the recipe directory.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

It looks like the 'cp2k-potentials' output doesn't have any tests.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin restart ci

@conda-forge-admin restart ci

@conda-forge-admin restart ci

@conda-forge-admin, please rerender

Hehe, enter the bot police!


Kindly remove W3Schools from the list.

W3Schools has a great bad reputation, so please consider removing W3Schools.

ok sir

I was curious as why "W3Schools has a great bad reputation" @praveenscience


How to run?

None of the instructions I find on how to use this promising tool work. When i install with pipx on Windows 11, I CD into my project with a pipeline and try pipeline-runner list, I get The term 'pipeline-runner' is not recognized as a name of a cmdlet etc.....
When I clone this repo and follow your instructions, on pip install -r requirements.txt, pip can't find any requirements.txt. I'm interested in adopting this at enterprise level, so a pointer as to how to proceed would be appreciated, please.

I have to start with a fair warning here, I have never tested my software on Windows. In theory it should work, but some features may not.

None of the instructions I find on how to use this promising tool work. When i install with pipx on Windows 11, I CD into my project with a pipeline and try pipeline-runner list, I get The term 'pipeline-runner' is not recognized as a name of a cmdlet etc.....

It looks like the directory where pipx installs apps is not in your PATH. You can run pipx ensurepath as described here to fix the issue.

When I clone this repo and follow your instructions, on pip install -r requirements.txt, pip can't find any requirements.txt. I'm interested in adopting this at enterprise level, so a pointer as to how to proceed would be appreciated, please.

This project uses poetry to manage dependencies. If you want to run from source, you need to install poetry then you can run poetry install to create a virtual env and install all dependencies

Thank you for a prompt reply. You don't have to test this on Windows, I'm here for that, hehe. A second try with pipx seems to have worked, I must be getting old.
I'll learn some poetry a bit later today, because I'm very interested in how you parse the yaml for a pipeline.


libnd4j: deconv2d op failures (MKLDNN?)

On CPU, built with MKLDNN:
        DynamicCustomOp op = DynamicCustomOp.builder("deconv2d")
                .addInputs(
                        Nd4j.create(DataType.FLOAT, 1, 3, 3, 2),
                        Nd4j.create(DataType.FLOAT, 2, 2, 3, 2),
                        Nd4j.create(DataType.FLOAT, 3)
                )
                .addOutputs(Nd4j.create(DataType.FLOAT, 1, 5, 5, 3))
                .addIntegerArguments(2, 2, 1, 1, 0, 0, 2, 2, 0, 1)
                .build();

        Nd4j.exec(op);

o.n.l.f.Nd4jBackend - Loaded [CpuBackend] backend
o.n.n.NativeOpsHolder - Number of threads used for OpenMP: 8
o.n.n.Nd4jBlas - Number of threads used for OpenMP BLAS: 8
o.n.l.a.o.e.DefaultOpExecutioner - Backend used: [CPU]; OS: [Windows 10]
o.n.l.a.o.e.DefaultOpExecutioner - Cores: [16]; Memory: [7.1GB];
o.n.l.a.o.e.DefaultOpExecutioner - Blas vendor: [OPENBLAS]
o.n.l.c.n.o.NativeOpExecutioner - Failed to execute op deconv2d. Attempted to execute with 3 inputs, 1 outputs, 0 targs,0 bargs and 10 iargs. Inputs: [(FLOAT,[1,3,3,2],c), (FLOAT,[2,2,3,2],c), (FLOAT,[3],c)]. Outputs: [(FLOAT,[1,5,5,3],c)]. tArgs: -. iArgs: [2, 2, 1, 1, 0, 0, 2, 2, 0, 1]. bArgs: -. Op own name: "c3d254b7-c926-4013-9009-06f59aa791a2" - Please see above message (printed out from c++) for a possible cause of error.

java.lang.RuntimeException: Op [deconv2d] execution failed

	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:1710)
	at org.nd4j.linalg.factory.Nd4j.exec(Nd4j.java:6560)
	at org.nd4j.Temp.test(Temp.java:23)
...
Caused by: java.lang.RuntimeException: could not create a dilated deconvolution forward descriptor
	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:2006)
	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:1700)
	... 24 more

Note this was pulled out of this test case (also reproducible there)
https://github.com/eclipse/deeplearning4j/blob/master/nd4j/nd4j-backends/nd4j-tests/src/test/java/org/nd4j/linalg/convolution/DeconvTests.java

A separate failure: it's failing with FP16 also (not sure if we should be using mkldnn or not for this case). This is pulled out of a test that was previously passing...
Float32 is fine for this case.
    @Test
    public void testDeconv2dHalf(){
        for(DataType dt : new DataType[]{DataType.FLOAT, DataType.HALF}) {
            System.out.println("STARTING: " + dt);

            INDArray in = Nd4j.create(dt, 2, 3, 4, 4);
            INDArray w = Nd4j.create(dt, 2, 2, 3, 3);
            INDArray b = Nd4j.create(dt, 3);

            INDArray out = Nd4j.create(dt, 2, 3, 8, 8);

            DynamicCustomOp op = DynamicCustomOp.builder("deconv2d")
                    .addInputs(in, w, b)
                    .addOutputs(out)
                    .addIntegerArguments(2, 2, 2, 2, 6, 6, 1, 1, 1, 0)
                    .build();
            Nd4j.exec(op);
            System.out.println("FINISHED: " + dt);
        }
    }

o.n.l.f.Nd4jBackend - Loaded [CpuBackend] backend
o.n.n.NativeOpsHolder - Number of threads used for OpenMP: 8
o.n.n.Nd4jBlas - Number of threads used for OpenMP BLAS: 8
o.n.l.a.o.e.DefaultOpExecutioner - Backend used: [CPU]; OS: [Windows 10]
o.n.l.a.o.e.DefaultOpExecutioner - Cores: [16]; Memory: [7.1GB];
o.n.l.a.o.e.DefaultOpExecutioner - Blas vendor: [OPENBLAS]
STARTING: FLOAT
FINISHED: FLOAT
STARTING: HALF
o.n.l.c.n.o.NativeOpExecutioner - Failed to execute op deconv2d. Attempted to execute with 3 inputs, 1 outputs, 0 targs,0 bargs and 10 iargs. Inputs: [(HALF,[2,3,4,4],c), (HALF,[2,2,3,3],c), (HALF,[3],c)]. Outputs: [(HALF,[2,3,8,8],c)]. tArgs: -. iArgs: [2, 2, 2, 2, 6, 6, 1, 1, 1, 0]. bArgs: -. Op own name: "e0d62a99-9f8d-492c-8e84-e353e5a7a236" - Please see above message (printed out from c++) for a possible cause of error.

java.lang.RuntimeException: Op [deconv2d] execution failed

	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:1710)
	at org.nd4j.linalg.factory.Nd4j.exec(Nd4j.java:6560)
	at org.deeplearning4j.Temp.testDeconv2dHalf(Temp.java:42)
...
Caused by: java.lang.RuntimeException: could not create a primitive descriptor iterator
	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:2006)
	at org.nd4j.linalg.cpu.nativecpu.ops.NativeOpExecutioner.exec(NativeOpExecutioner.java:1700)
	... 24 more

Are these two tests still failing?

The two original test cases (above) are both passing now.
I tested master with the simple dilation_mkldnn = dilation_ours - 1, based on MKLDNN's docs
That's instead of this equation here:
https://github.com/KonduitAI/deeplearning4j/blob/master/libnd4j/include/ops/declarable/helpers/convolutions.h#L205
with that "subtract 1" adjustment, everything passes (libnd4j and nd4j) except for 3 of 28 cases from DeconvTests.compareKeras
The 3 cases are:
mb3_k3_sz3_s1_same_d2_nchw
mb3_k3_sz8_s1_same_d2_nhwc
mb3_k4_sz3_s1_same_d2_nchw
that's minibatch / kernel / input size / stride / mode / dilation / format
note that these 3 fail on master also - which is expected, the equation I replaced is for valid mode only.
so dilation 2 + same mode fails
d2 + valid is fine; d1 + same is also fine
Essentially, yes, we should just use dH = dH - 1 everywhere for MKLDNN, and fix whatever this other issue is...
As for what's going on:
We have an effective kernel size of k+(k-1)(d-1)
mb3_k3_sz3_s1_same_d2_nchw  -  3+(3-1)(2-1) = 5
mb3_k3_sz8_s1_same_d2_nhwc  -  3+(3-1)(2-1) = 5
mb3_k4_sz3_s1_same_d2_nchw  -  4+(4-1)(2-1) = 7
Which gives me 3 hypotheses:

MKLDNN doesn't support dilated deconv with same mode (could explain all 3)
MKLDNN doesn't support dilated deconv when effective kernel size is greater than input (can only explain first and 3rd)

Unless we can find another explanation, I suggest that we just don't use mkldnn for dilated deconv + same mode. It's a rare case anyway.
cc @shyrma

Confirmed passing now


Add NPM instructions

There should be instructions for adding AppInsights via NPM and/or with React.

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

ID: d56ecb14-9e66-5313-a381-d2824ec3d1e2
Version Independent ID: ff056ac8-ae76-be10-6895-757333a28e05
Content: Azure Application Insights for JavaScript web apps
Content Source: articles/azure-monitor/app/javascript.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@MisinformedDNA thank you for your feedback. I have assigned this to the doc author to decide whether or not to add the requested information.

The change to address this PR is now merged internally it will go live between 3-4 PM (UTC-7). Thanks again for bringing this to our attention.

#sign-off

#please-close


Remove attachmentId char limit

Internal task

OPS Build status updates of commit 10d94d2:
:clock10: Pending: waiting for processors (14 builds ahead of you)
:warning: OPS Build is busy, click here to report any potential performance downgrade issues, or you could view OPS Portal for trend of build queue length.

  This is the expected behavior of the build system.

OPS Build status updates of commit 10d94d2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.1/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.item.md
:white_check_mark:Succeeded

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