Can I suggest one more awesome Parallel Computing course?

It's a Udacity interactive course Intro to Parallel Programming.
It covers the fundamental of GPGPU parallel computing and contains very useful examples and exercises.
Here's the link:
https://www.udacity.com/course/viewer#!/c-cs344

@ydwu
this is an interesting course, for sure!
Let's see if there are more considerations from other students before we add or not this course into our curriculum.
:smile:

@ericdouglas we can also do a file in help with all the course we find and class them, for those who want to custom more deeply the curriculum. I thought about this after the discover of @hangtwentyy course about machine learning. I had already this one : https://www.coursera.org/course/hetero. It will open in january. Udacity is always open.

not sure if this makes sense with this course, but at some point more "in depth" courses might be better served on a "specialization" or "grad-level" list, which this OSS-CS list representing more of a undergrad CS sequence that  touches on every topic to some degree, but not into too much depth. Granted I am unable to judge what level makes sense. I just know as someone starting from course 1, if this CS list just keeps growing, then it begins to seem more and more unattainable. I understand lifelong learning is a goal, but achieving certain milestones is important to maintain momentum, so constantly moving the goal posts back is not ideal. I don't have a good solution to this, but there are other options to address this issue as well, like inserting certain miletones within the sequence that students achieve sequentially, etc.
Another idea would for this particular situation would be to have list "alternatives courses" that satisfy the same requirement. For example, perhaps it could be recommended that you should to take one of the two parallel computing courses, and the decision to take coursera vs udacity is up to the student, course, availability, etc.

@mseyne yeah we can/should do this. I just think that we can create another repository for such thing.
@crmackay you're totally right. I truly believe that we have a pretty solid/stable curriculum here, and I'm concerned about those things that you told.
Let's do a "win-win" solution here. We should add more courses, but in an optional list. :smile:


The default pose determination

When the screen can‘t track the arm or hand， we need the Avatar arm and hand stay in default pose， Looks normal.   when I put my arm down,  avator arm and hand is alse stay last position, don’t  looks normal

This needs some more work. Essentially, you will need to:

Add a low-pass filter to determine if a null result is because of a temporary frame drop (e.g. when we add the null check in #46), or is because of the posture change. (A better fix is actually to analyze if the null result can be a result of sending too many frames to MP and reduce the frame pool size to avoid this false positive.)
If it's because the posture, Kalman filter should make the movement smooth and natural (moving the arm down).


put .rodata stuff in the .rodata section; make reset_handler the entry point

Before this commit the .rodata sections of the inputs were placed in the output
.text section. As the vector table is in the .rodata of the cortex-m-rt crate
this placed the vector table first in the output .text section:
$ arm-none-eabi-objdump -Cd hello
Disassembly of section .text:

08000000 <_VECTOR_TABLE>:
 8000000:       10002000        .word   0x10002000

08000004 <cortex_m_rt::RESET_HANDLER>:
 8000004:       08000405                                ....
(..)

$ arm-none-eabi-readelf -h hello | grep Entry
  Entry point address:               0x8000000

With this memory layout using load and then step on a GDB session made the
processor execute the vector table (PC = 0x08000000).
This commit places the .rodata sections of the inputs into the output .rodata
section, and also moves the reset_handler code to the start the output .text
section. This turns the reset_handler into the entry point of the program.
$ arm-none-eabi-objdump -Cd hello
Disassembly of section .text:

08000410 <cortex_m_rt::reset_handler>:
(..)

$ arm-none-eabi-readelf -h hello | grep Entry
  Entry point address:               0x8000410

With these changes the GDB commands load and step work correctly: load
sets the PC = reset_handler.

r? @whitequark
With this we should be able to drop the monitor reset halt command from the
.gdbinit file in cortex-m-quickstart.

The underlying idea of this change looks good to me; I've hit this issue sometime ago (while investigating uploading code into SRAM) but never went to the bottom of it.
I'm far less certain about the section names. The name convention of .text.something or .rodata.something originates from the hosted ELF targets, and its purpose is a weakly structured form of communication between the various compilers and the system linker. E.g. in my system (try ld --verbose):
  .text           :
  {
    *(.text.unlikely .text.*_unlikely .text.unlikely.*)
    *(.text.exit .text.exit.*)
    *(.text.startup .text.startup.*)
    *(.text.hot .text.hot.*)
    *(.text .stub .text.* .gnu.linkonce.t.*)
    /* .gnu.warning sections are handled specially by elf32.em.  */
    *(.gnu.warning)
  }

Essentially, so long as there is a *(.text.*) somewhere, the compilers can introduce new conventions without hard breakage. This also incidentally allows -ffunction-sections to work (I think this convention predates the flag).
In case of cortex-m-rt we lack this compatibility hazard, and so I think there is no good reason to put something into a .text.something or .rodata.something section when we then explicitly specify that in the linker script. I would prefer to have an explicit .vectors section in the output, such that arm-none-eabi-size would be more useful on the final ELF product; the vector table shouldn't count against the data size, for one. This is what basically every other toolchain does with their linker scripts.
There are also a few minor issues for which I'll leave inline comments.

Couldn't leave inline comments so #9 and #10.

such that arm-none-eabi-size would be more useful on the final ELF product;

I should note that cortex-m-rt in its current form, this PR, and the modified version of this PR with an output .vectors section all produce the same arm-none-eabi-size output. That is the vector table is counted as "text", not as "data" or "bss".

I think there is no good reason to put something into a .text.something or .rodata.something section

If I understand you correctly we should not use, for example, #[linker_section = ".rodata.exceptions"] in Rust code because the toolchain / ELF standard may use use the ".rodata.exceptions" section in the future for some feature, or assign some other meaning to it. Instead we should use something like #[linker_section = ".vectors.exceptions"]. And you comment about an output .vectors* section was mainly due to the concern of arm-none-eabi-size not behaving correctly? The section .vectors.exceptionswouldn't make it to the output binary because the linker script will instruct the linker to produce a single.vectors` section in the output binary.

I should note that cortex-m-rt in its current form, this PR, and the modified version of this PR with an output .vectors section all produce the same arm-none-eabi-size output.

Try the -A option.

If I understand you correctly we should not use, for example, #[linker_section = ".rodata.exceptions"] in Rust code because the toolchain / ELF standard may use use the ".rodata.exceptions" section in the future for some feature, or assign some other meaning to it.

No, I meant there is no argument for using this name for the section. We control the entire toolchain (the linker doesn't make judgement of section names other than via the builtin script) so there's no argument against based on what the ELF standard or toolchain does.

Instead we should use something like #[linker_section = ".vectors.exceptions"].

I think having separate sections for vectors and not squishing them into .rodata is just handy for anyone working with the files on that level, which I had to do on quite a few occasions. The arm-none-eabi-size improvement is perhaps the most visible change.

The section .vectors.exceptions wouldn't make it to the output binary because the linker script will instruct the linker to produce a single .vectors section in the output binary.

Yeah sure, changing that name is just for consistency with the output product.

What is wrong with doing it this way, like any other linker script from C world? Is there a reason why we need to put all sections in one place (eg. in .text)?
SECTIONS
{
  .vectors ORIGIN(FLASH):
  {
    /* Vector table */
    _VECTOR_TABLE = .;
    LONG(_stack_start);

    KEEP(*(.vectors.reset_handler));
    KEEP(*(.vectors.exceptions));
    __exceptions = .;

    KEEP(*(.vectors.interrupts));
    __interrupts = .;
  } > FLASH

  .text : ALIGN(4)
  {
    /* Put reset handler first in .text section so it ends up as the entry */
    /* point of the program. */
    KEEP(*(.text.reset_handler));
    *(.text)
    *(.text*)
  } > FLASH

  .rodata : ALIGN(4)
  {
    *(.rodata)
    *(.rodata*)
  } > FLASH

  .data : ALIGN(4)
  {
    _sdata = .;
    *(.data .data.*);
    _edata = ALIGN(4);
  } > RAM AT > FLASH

  .bss : ALIGN(4)
  {
    _sbss = .;
    *(.bss .bss.*);
    _ebss = ALIGN(4);
  } > RAM

@pftbest this is exactly what I'm suggesting fwiw

Try the -A option.

TIL. I agree that's more helpful with the .vector split.
I have made the requested changes but I have picked .vector_table instead of .vectors. Does that rename seems good to you?

Also check my inline comments; I left a question there about .text.reset_handler.

looks good to me, i don't mind the rename. Also i agree that having .reset_handler instead of .text.reset_handler is less likely to break.

@japaric I prefer .vectors (after all we don't call .bss .uninit_data) but it's not a hill I will die on

I prefer .vectors

I chose .vector_table because that's how the ARM documentation calls it. Also not everything in .vector_table is a vector; the first thing on the vector table is the initial SP value.

I chose .vector_table because that's how the ARM documentation calls it. Also not everything in .vector_table is a vector; the first thing on the vector table is the initial SP value.

Ack.

Thanks for input @whitequark and @pftbest. I'll postpone the rename of the input sections .rodata.{interrupts,exceptions} to .vector_table.* as it would break crates like cortex-m-quickstart and cortex-m-rtfm. That would let us publish a new patch version with these changes today.


Don't create multiple Version objects for a version string

There are some cases in which the API may create multiple Version objects with the same version string. This causes problems with automated signing where there's a race condition and messes up the review queue when not in automated signing.

[ ] Change the Version creation check from "is this the most recent upload" to "does a Version already exist."
[ ] Wrap the version check code in a transaction to ensure we are looking at a stable snapshot and avoid the quick submission race condition.

Fixes https://github.com/mozilla/olympia/issues/996.


What is the bucket wellcomecollection-assets-archive-storage?

It's in the platform account, and looks like an early prototype of the storage service. Can we delete it?

@alexwlchan FWIW, there is no reference to this bucket in either the DDS or the DLCS configuration/tf/etc.

@tomcrane Yeah, it's a very incomplete copy of the archive which shouldn't be in use anywhere.

Per Jonathan in Slack, this is a test for the storage service, and can most likely be deleted.

It's gone!


Button not rendered with recent UI update from Chatgpt.

Easy fix, just add "items-stretch" to the following line of code:
inputActionNode = document.querySelector(
	"div[class*='relative flex h-full flex-1 md:flex-col']"
);

becomes
inputActionNode = document.querySelector(
	"div[class*='relative flex h-full flex-1 items-stretch md:flex-col']"
);

And remember to change both instances of that line (line 16 & line 62)

Implementing it right now + modded the parent container to remove the margin-top stylings which made the button look a bit weird on smaller screen


3.1 crashing at start on Win10

3.1 crashes:

G:\games\EliteDangerous\EDLaunch.exe /Epic -AUTH_LOGIN=unused -AUTH_PASSWORD=05d5d6d5a504472e8ddd35553d3605f1 -AUTH_TYPE=exchangecode -epicapp=9c203b6ed35846e8a4a9ff1e314f6593 -epicenv=Prod -EpicPortal /autorun /autoquit /EDH -epicusername=Filinka -epicuserid=a28f1a97902b47dea403d9dd481796ca -epiclocale=en
Unhandled Exception: EETypeRva:0x0011A0D0: TypeInitialization_Type_NoTypeAvailable
StackTrace_AtWordEDLaunch!+0x85874
StackTrace_AtWordEDLaunch!+0x754cd
StackTrace_AtWordEDLaunch!+0x755be
StackTrace_AtWordEDLaunch!+0xdf7e7
Exception_EndOfInnerExceptionStack
StackTrace_AtWordEDLaunch!+0xdf88c
StackTrace_AtWordEDLaunch!+0xdf6bd
StackTrace_AtWordEDLaunch!+0x75551
StackTrace_AtWordEDLaunch!+0x75f7b
StackTrace_AtWordEDLaunch!+0x760d8
StackTrace_AtWordEDLaunch!+0x7cff6
StackTrace_AtWordEDLaunch!+0xff505

3.0 works fine:

G:\games\EliteDangerous\EDLaunch.exe /Epic -AUTH_LOGIN=unused -AUTH_PASSWORD=05d5d6d5a504472e8ddd35553d3605f1 -AUTH_TYPE=exchangecode -epicapp=9c203b6ed35846e8a4a9ff1e314f6593 -epicenv=Prod -EpicPortal /autorun /autoquit /EDH -epicusername=Filinka -epicuserid=a28f1a97902b47dea403d9dd481796ca -epiclocale=en
[12:19:44 INF] Elite Dangerous - Minimal Launcher
Platform: Epic
OS: Win64
CobraBay Version: 0.4.6666.0
Launcher Version: 0.3.0
Products Dir: G:\games\EliteDangerous\Products
[12:19:45 ERR] Couldn't login: 400: Bad Request

This looks like an issue with the bootstrapper. Until I've figured it out, you should be able to use MinEdLauncher.Bootstrap.exe from the 3.0 release with the latest 3.1 MinEdLauncher.exe.

This is a possible issue with a dependency. I've reverted to a previous version for now until I can learn more about what's changed with NativeAOT. New releases should work with the bundled bootstraper exe.


Scrape accessions from downloaded summary data

When you download records from here, it's a file with a bunch of extra info, and we just need accessions to download genomes. I'm not sure if I'm getting the right accessions, so there might be duplicates, but there it is.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #243   +/-   ##
=======================================
  Coverage   25.25%   25.25%
=======================================
  Files          20       20
  Lines         681      681
=======================================
  Hits          172      172
  Misses        509      509

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 a41c231...1f412fe. Read the comment docs.

If the intention here is to get the accession number for genome collections, then you are getting the right accession numbers, if we want the RefSeq accessions or the accession numbers for individual sequences (ie: per chromosome) then those from the .ids files NCBI provides. You may know all of this already just making sure we're on the right track.

Also if you don't mind, I believe rewriting this to use pandas will be much simpler/faster. Pandas is pretty amazing for scientific data files that are in a standardised format. Let me know if you're keen to see a PR for that.

@thisisjaid no I didn't know that, so thank you! What's a .ids file and where does it come from?
Also re-writing with pandas would be very welcome, please do send a PR!

ftp://ftp.ncbi.nlm.nih.gov/genomes/GENOME_REPORTS/IDS/Eukaryota.ids
As far as I can tell that holds the various IDs for individual sequences (ie: chromosomes). Are these the accession numbers we want to be grabbing (the NC_031807 format column)?

PR is up, when you have a moment let me know if we should switch to reading the IDS files instead for RefSeq accession IDs. I haven't had a chance to fully read through the whole project yet to get an idea of how we use this info so I'll defer to you for the time being.

Oh yeah, the .ids file looks great. And yes, the accessions from the second column (NC_031807) are what we want.

I'm going to merge this anyhow, but @thisisjaid please feel free to submit a PR for the improved version.


What theme is it?

Hello, it caught my attention that in the values of an object it is shaded in a different color? Is it a theme or an extension that shadows a costom snippet?
Can you help me clear that doubt?

follow this to get inline hints,
https://ash12rai-weblearning.medium.com/visual-studio-code-vs-code-inlay-hints-32fee837e80
ie  add "typescript.inlayHints.parameterNames.enabled" : "all"; to .vscode > settings.json
its not part of a theme, but a typescript setting in vscode.
this is the theme though: https://marketplace.visualstudio.com/items?itemName=akmarnafi.midnight-code


Port CSS to SCSS

Hi there,
I did some work recently in porting the project's CSS files to SCSS. More details in this pull-request ->
https://github.com/diaspora/jsxc/pull/66
Is this stuff interesting to you?
Can you cherry-pick the changes, or should I rebase to one of your branches, and submit a pull request?

That looks very interesting. I will try to have a deeper look at it this week.

Your work looks great and I would be happy to see a pull request of your changes, because that should simplify many things. If you need help, I will assist.


Firmware Override

Getting update available for outlet, since it pulls the plugin version, can you add an option to override this?

nvm it pulls it. orphaned accessory I had.


[BUG] ReactiveUserControl is missing from the ReactiveUI namespace

Describe the bug
Class ReactiveUserControl<TViewModel> is missing from the latest (9.15.1) ReactiveUI nuget package.
Steps To Reproduce
Provide the steps to reproduce the behavior:

Create new UWP application
Install ReactiveUI v9.15.1 nuget package
Try to use ReactiveUI.ReactiveUserControl<TViewModel> class

Expected behavior
The ReactiveUserControl<TViewModel> class is found.
Screenshots

Environment

OS:       Windows10
Version   10.0.17763
Device:   Desktop

Additional context

    <TargetPlatformMinVersion>10.0.17763.0</TargetPlatformMinVersion>

Minimum version in 9.14 and above is 10.0.17763.0
Changed the release notes to make that more obvious now.
This change was required by Azure DevOps having that as a minimum on their servers.


[Issue]: clr build fails and issue being closed prematurely without possibility of opening

Problem Description
See here for details:
https://github.com/ROCm/clr/issues/39
https://github.com/ROCm/clr/issues/23
There are numerious problem with approaching this ticket being ended up closing prematurely without resolve:

closing without resolving a number of times. It asks you can re-open but that option is not available.
responder says, he can not reproduce and fails to provide log on his end, that makes it impossible back his claim, instead I see it is easily reproduce with just one command.
In overall, all these above with fickle approach from the responders to this is causing tremendous churning in trying to explain and reason to follow through logic toward resolution.
THEREFORE, IMPORTANT: <<-- STOP CLOSING TICKET UNTIL CONFIRMED BY SUBMITTER PLS >>__!!

Operating System
rhel 9
CPU
EPYC 7763
GPU
MI210
ROCm Version
ROCm 6.2.0
ROCm Component
No response
Steps to Reproduce
after checkout both clr and HIP in same dir.
from clr dir:
mkdir build ; cd build
sudo cmake .. -DCLR_BUILD_HIP=ON -DHIP_COMMON_DIR=../HIP
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Latest attempt after closing twice previous ticket prematurely without resolving:

 ; sudo cmake -DCLR_BUILD_HIP=ON -DHIP_COMMON_DIR=$HIP_COMMON_DIR=../../HIP  ..
-- The C compiler identification is GNU 11.5.0
-- The CXX compiler identification is GNU 11.5.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- HIPCC Binary Directory: /opt/rocm/bin
-- HIP Common Directory: =../../HIP
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Found AMD_OPENCL: /root/rocm/ROCm-6.2/clr/opencl/khronos/headers/opencl2.2/CL
CMake Error at rocclr/cmake/ROCclrLC.cmake:37 (file):
  file STRINGS file
  "/root/rocm/ROCm-6.2/clr/rocclr/=../../HIP/VERSION"
  cannot be read.
Call Stack (most recent call first):
  rocclr/cmake/ROCclr.cmake:139 (include)
  rocclr/CMakeLists.txt:30 (include)

-- Found NUMA: /usr/lib64/libnuma.so
-- Found OpenGL: /usr/lib64/libOpenGL.so
-- HIPCC_BIN_DIR found at /opt/rocm/bin
-- HIP_COMMON_DIR found at =../../HIP
-- HIPNV_DIR found at
-- Found Perl: /bin/perl (found version "5.32.1")
CMake Error at hipamd/CMakeLists.txt:88 (file):
  file STRINGS file
  "/root/rocm/ROCm-6.2/clr/hipamd/=../../HIP/VERSION"
  cannot be read.

-- Found Git: /bin/git (found version "2.43.5")
-- Using CPACK_DEBIAN_PACKAGE_RELEASE local
-- CPACK_RPM_PACKAGE_RELEASE: local%{?dist}
-- HIP Platform: amd
-- HIP Runtime: rocclr
-- HIP Compiler: clang
-- ROCM Installation path(ROCM_PATH): /opt/rocm
-- HIP will be installed in: /opt/rocm
-- Performing Test Terminfo_LINKABLE
-- Performing Test Terminfo_LINKABLE - Success
-- Found Terminfo: /usr/lib64/libtinfo.so
-- Found ZLIB: /usr/local/lib/libz.so (found version "1.3.#define ZLIB_VERSION "1.3"")
-- Found zstd: /usr/local/lib64/libzstd.so
-- Found LibXml2: /usr/lib64/libxml2.so (found version "2.9.13")
'sh' '-c' '/root/rocm/ROCm-6.2/clr/hipamd/src/hip_embed_pch.sh =../../HIP/include /root/rocm/ROCm-6.2/clr/build/hipamd/include /root/rocm/ROCm-6.2/clr/hipamd/include /opt/rocm/llvm/lib/cmake/llvm/../../..'
+ /opt/rocm/llvm/lib/cmake/llvm/../../../bin/clang -O3 --hip-path=/root/rocm/ROCm-6.2/clr/build/hipamd/include/.. -std=c++17 -nogpulib -isystem /root/rocm/ROCm-6.2/clr/build/hipamd/include -isystem =../../HIP/include -isystem /root/rocm/ROCm-6.2/clr/hipamd/include --cuda-device-only --cuda-gpu-arch=gfx1030 -x hip /tmp/hip_pch.80988/hip_pch.h -E
/tmp/hip_pch.80988/hip_pch.h:1:10: fatal error: 'hip/hip_runtime.h' file not found
    1 | #include "hip/hip_runtime.h"
      |          ^~~~~~~~~~~~~~~~~~~
1 error generated when compiling for gfx1030.
CMake Error at hipamd/src/CMakeLists.txt:200 (message):
  Failed to embed PCH

-- Configuring incomplete, errors occurred!
[nonroot@SMC-DH144-DC16-U27 build]$  sudo rm -rf ./* ; sudo cmake -DCLR_BUILD_HIP=ON -DHIP_COMMON_DIR=$HIP_COMMON_DIR=../../HIP  ..^C
[nonroot@SMC-DH144-DC16-U27 build]$ find ../../HIP -name hip_runtime.h
../../HIP/include/hip/hip_runtime.h
[nonroot@SMC-DH144-DC16-U27 build]$ ls -l ../../HIP
total 84
drwxr-xr-x.  2 root root    73 Nov  1 15:34 bin
drwxr-xr-x.  3 root root    54 Nov  1 15:34 cmake
-rw-r--r--.  1 root root    51 Nov  1 15:34 CODEOWNERS
-rw-r--r--.  1 root root     0 Nov  1 15:34 configure
-rw-r--r--.  1 root root  8992 Nov  1 15:34 CONTRIBUTING.md
drwxr-xr-x.  2 root root    90 Nov  1 15:34 docker
drwxr-xr-x. 10 root root  4096 Nov  1 15:34 docs
-rw-r--r--.  1 root root  5271 Nov  1 15:34 hip-lang-config.cmake.in
drwxr-xr-x.  3 root root    25 Nov  1 15:34 include
-rwxr-xr-x.  1 root root  1208 Nov  1 15:34 install.sh
-rw-r--r--.  1 root root 16645 Nov  1 15:34 Jenkinsfile
-rw-r--r--.  1 root root  1080 Nov  1 15:34 LICENSE.txt
-rw-r--r--.  1 root root 11115 Nov  1 15:34 README.md
-rw-r--r--.  1 root root  9833 Nov  1 15:34 RELEASE.md
drwxr-xr-x.  4 root root    42 Nov  1 15:34 util
-rw-r--r--.  1 root root    67 Nov  1 15:34 VERSION


CAMEL-12584:Allow seda producers to offer data to the queue with timeout

CAMEL-12584:Allow seda producers to offer data to the queue with timeout

Changes done

I'm merging

Merged


Fix host environment variable bug

While writing docs for #1300 I found a bug with the HOST environment variable. When I set this variable, it uses the value I passed to print the link to the app in the console, but it doesn't actually use it.
Steps to reproduce
Execute npm start.  The  console displays this message:
Compiled successfully!

The app is running at:

  http://localhost:3000/

The app is can be viewed from your browser, and also from all the devices connected to your local network such as your mobile phone. Once you find your computer's IP address, you can open your phone's browser at something like <IP_ADDRESS>:3000 to see the app. The localhost url is a bit misleading since the server is actually listening at <IP_ADDRESS> since that's the value that binds to all interfaces, not just the local loopback interface according to webpack-dev-server's docs. In this issue it is mentioned that binding to localhost doesn't let other devices in the same network connect to the dev server.
If you use <IP_ADDRESS> as host, then other devices  should not be able to connect to the dev server since it is binding to the local interface. Try executing HOST=<IP_ADDRESS> npm start. Mobile phones can still connect to it.  That's because the HOST environment variable isn't being properly used. It should be passed as an argument of dev server's listen function, along with the port.  This PR fixes it.
I also changed the default host from localhost to <IP_ADDRESS> since connections from other devices are currently allowed by default. <IP_ADDRESS> lets other devices connect to the server, localhost doesn't. Some user's may use this feature at work, so I don't want to break their setup.
This isn't a critical bug or anything like that, I don't think many users would worry about this. However it is still a bug. The other option is simply remove the HOST environment variable.

Hmm. So is HOST not used at all right now? How do people use it?

Tagging so we don't forget this.

I don't think many people use it anyway.
Here's how create-react-app sets up de dev server.
Here's how the webpack docs sets up the dev server.
They are not the same when it comes to the host variable.

Tagging as 0.9.5. We need to look into this.

Thanks! Backported to 0.9.x via 604519a741964d641f246396081f221ee5a48ca0, which binds to <IP_ADDRESS> by default. I modified this PR to bind to localhost by default in 0.10 for security reasons.
Sorry for the log backlog time, @gaearon and I have both been tied up recently. 😄


changed the prerequisits from undergraduate to following

The courses that had Undergraduate in them all called the same file. I changed the wording and now all classes that call that file have the change.

With the document naming, the files will no longer need to be updated by year. The only update to the fall file from now on will be the box link should it ever change to a different location in box, and the google sheet for the semester textbooks.


update icons using new logo

Which icons? Aren't they using the latest alreadt?

is there something still missing? if there is I can get it done

https://github.com/ShabadOS/desktop/tree/dev/resources is using an old icon. I have the svg and can make a png. Not sure about the rest.


Don't cache-bust output files that haven't changed.

This is a feature request.
I've noticed that Webpack adds cache-busting suffixes to all the compiled assets.  This is great because I can set cloudflare's edge-cache lifetime to be infinite and get a huge boost in load time.
The problem is that even when an asset isn't modified, webpack still appends a new suffix.  I can see this by looking at the git diffs which recognize it as a rename operation.  This defeats the optimization completely.
Can you change webpack's configuration so that it doesn't do this?  I suspect right now it's generating the suffix with a library like yeast, which goes by the date.  Is there any way for you to change it so that it bases it on, say, the hash of the file instead?  I know it would take longer but I don't really care.  This is the optimized production build, after all.

I suspect right now it's generating the suffix with a library like yeast, which goes by the date. Is there any way for you to change it so that it bases it on, say, the hash of the file instead?

I’m certain it uses hashes of the compiled bundles.

I can see this by looking at the git diffs which recognize it as a rename operation.

Can you provide a project reproducing this and an exact sequence of steps to reproduce?

I did further testing and it indeed doesn't create new files unless something has changed.  I guess I must have made some changes which I hadn't noticed.

Maybe dependencies.

That could be.  I'll re-open if it happens again and I'm able to find steps to reproduce it.

Well, I've again encountered what I thought caused it  I get git diffs like this:
renamed:    build/static/js/main.9a186559.js -> build/static/js/main.270acdd5.js

I originally I assumed this meant that main.js didn't change, but it did get a different name.  This time I know I did indeed change my source js, so there must be something I don't understand about git's diff algorithm.


Feature request: a way to disable admin API endpoint "/__admin"

We use wiremock in standalone mode with JSON files for configuration. Sometimes we need to have wiremock available on a public endpoint so for obvious security reasons we would like to have a way to disable the admin API endpoint completely.
For example, adding a --disable-admin-api command line argument.
Or is there another solution already available to achieve this?

Had a similar question. I couldn't find if there was a way to disable it but there appears to be an undocumented command-line option to set Basic Auth on the Admin API.
https://github.com/wiremock/wiremock/blob/3.2.0/src/main/java/com/github/tomakehurst/wiremock/standalone/CommandLineOptions.java#L99

+1

Hi, there isn't currently a way to disable the admin endpoints but as @owenkim mentioned, you can protect them via basic auth.  It looks like we missed adding that to the documentation so I will add that

Just as a quick example before I sort out the docs.  You can start WireMock with the following flag --admin-api-basic-auth passing you username and password in the standard format - username:passord:
java -jar wiremock-standalone-3.9.1.jar --admin-api-basic-auth my-username:my-super-secret-password

Any call made to the admin API after that will need the correct Authorization header passed or a 401 will be returned.

Hi, this has now been documented here - https://wiremock.org/docs/standalone/java-jar/#securing-the-wiremock-admin-api
I will close this issue if everyone is happy with this solution ?

Brilliant stuff - we've implemented in our internally hosted service. Thanks for the speedy response.

happy for you to close issue


Add cluster_cache_hit_rate metric

Address #13724
What changes are proposed in this pull request?
Add a gauge metric in MetricsStore.java, CLUSTER_CACHE_HIT_RATE = 1 - CLUSTER_BYTES_READ_UFS_ALL/(CLUSTER_BYTES_READ_DIRECT + CLUSTER_BYTES_READ_REMOTE + CLUSTER_BYTES_READ_DOMAIN)
Why are the changes needed?
We want to add a metric cluster_cache_hit_rate and now we can only calculate by ourselves
Does this PR introduce any user facing changes?
Users can get a new metric

Codecov Report

Merging #13725 (08c7789) into master (f47c6e6) will decrease coverage by 26.05%.
The diff coverage is 87.50%.

@@              Coverage Diff              @@
##             master   #13725       +/-   ##
=============================================
- Coverage     43.64%   17.59%   -26.06%
+ Complexity     9058     2666     -6392
=============================================
  Files          1384     1384
  Lines         80303    80318       +15
  Branches       9768     9769        +1
=============================================
- Hits          35050    14132    -20918
- Misses        42320    64935    +22615
+ Partials       2933     1251     -1682

Impacted Files
Coverage Δ

...main/java/alluxio/master/metrics/MetricsStore.java
62.50% <81.81%> (-21.91%)
:arrow_down:

...ommon/src/main/java/alluxio/metrics/MetricKey.java
97.20% <100.00%> (-0.42%)
:arrow_down:

...a/alluxio/master/metrics/DefaultMetricsMaster.java
81.57% <100.00%> (-11.85%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 657 more

Continue to review full report at Codecov.

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

alluxio-bot, merge this please


Add clojureArgs option for setting arguments for clojure executable

This should fix https://github.com/jasongilman/proto-repl/issues/313


feat: #138

Proposed changes
Documentation website improvements

fixed algolia search plugin
re-styling cookie banner
google analytics opt-out

Closes #138
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #140 (7f5ef64) into main (a7b2223) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #140   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          101       101
  Lines          884       884
  Branches       121       121
=========================================
  Hits           884       884

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 a7b2223...7f5ef64. Read the comment docs.


[FR] Optimize for collaborating with third-party plugin media-extended

For a common MD file, there is an option to open this file in a popover.
However, when opening a video in a new pane via media-extended, no corresponding option out there to put it in a popover or convert it into a popover.

Sure, I'll handle this. You can close your issue in the media extended repo.

@nothingislost Thanks, this would further impove my experience when taking video notes. I've closed the same issues in other repos.

This is available in 0.8.2 if you want to check it out and report back

Thanks for the enhancement, but currently when opening the video pane in a popover, it will lose the link to the note pane, so cannot add timestamps as expected.

I think it would be better if it could be further optimized like below. The video pane could directly convert into a popover, meanwhile, it would not lose the link to the note pane.

Ah, I'm not very familiar with the media extended plugin so I didn't test much beyond being able to show the menu item.
One option here is that you can manually relink that hover editor with your document using the context menu. Not ideal but at least the functionality is there.
I like the idea of having an option to convert a leaf to a hover editor. You can already do this but it's not as convenient as a "Convert pane to a Hover Editor" command. See the video for an example:
https://user-images.githubusercontent.com/89109712/163299960-68722dbd-d1c9-40a2-ac7c-1e3284c13f25.mp4

Well, my current workaround is to open the video pane in a popover, I think this one is better than the others.

Anyway, still looking forward to a new plugin feature that an existing pane to be converted to a popover. It would much benefit all workflows, not only when collaborating with media-extended.

If you're using the BRAT plugin, check for beta updates and you should get 0.8.4 which has the "Convert pane to Hover Editor" command and context menu item.

That's pretty awesome! And a little bit more suggestion is, for a popover that has been converted this way, to have a command and a context menu item to convert it back. Really appreciate your time and effort in this.

You can just drag the pane back into the workspace to accomplish that though right?
Assuming there's a need for the context menu item, where would you expect the pane to be placed within the workspace?

Sure, but sometimes I prefer to achieve this only by the keyboard.
I think it could be placed horizontally next to the active pane, just like the behavior when ctrl+cliking on a file to open it in a new pane.

Alright, 0.8.5 adds the "Dock Hover Editor to workspace" command and context menu item.


nixos/pretix: fix defaultText for celery options

Description of changes
It seems a mistake snuck in with the defaultTexts for services.pretix.settings.celery.*, probably copy-pasted from pretalx: they were referencing an option config.services.pretalx.celery.enable that doesn't exist.
The defaultTexts should now reflect the actual default.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This is not about it existing or not, but that it is not conditional.

No, actually it exists

Sorry, I mistyped: config.services.pretix.celery.enable (not Pretalx) was referenced and doesn't exist. And the conditional was wrong either way.
Anyway, thanks for merging!


fixed detection of type check

Codecov Report

Merging #114 into master will decrease coverage by 0.37%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   72.91%   72.54%   -0.38%
==========================================
  Files          39       39
  Lines        3234     3234
  Branches       51       51
==========================================
- Hits         2358     2346      -12
- Misses        842      854      +12
  Partials       34       34

Impacted Files
Coverage Δ

src/tokentree/utils/TokenTreeCheckUtils.hx
50.32% <100%> (-0.4%)
:arrow_down:

src/tokentree/walk/WalkFieldDef.hx
84.09% <0%> (-11.37%)
:arrow_down:

src/tokentree/walk/WalkTypedefBody.hx
85.05% <0%> (-2.3%)
:arrow_down:

src/tokentree/walk/WalkClass.hx
77.17% <0%> (-2.18%)
:arrow_down:

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 d785e4e...3987ab0. Read the comment docs.


Baseline Testing: Scale testing

Scaling the number of resources (HTTPRoutes, Gateways, Secrets, Pods )
Ensure NKG can process those resources and handle the traffic
Ensure a new version doesn't increase resource usage compared with the previous one.

Acceptance

A process is documented for deploying a large number of Gateway Resources resources with NKG installed to test for when NKG stop working correctly.
Traffic is generated and sent through NKG to an upstream as a part of this test.
The resource utilization (CPU, memory, disk) of NKG, errors from the NKG pod(s), and errors from the client, are measured and reported as a result of this test.
No element of the test is randomized.
The test is carried out manually and the results are reported in a static document within the NKG repo.

See #952 for potential overlap

All the scaling tests(incl. NGF, tea/coffee) should ideally be localised to this ticket so as no to have overlap with Longevity tests.
https://github.com/nginxinc/nginx-gateway-fabric/issues/955 can be a subcase of this


Make Num the default unit/length ?

Is there a reason (Num/Float) is not the default length?
This would get rid of all the (num 50), (mm 50), (px 50) and (em 50) all over the place.
I would prefer to only use them at the Svg level.
The inEm and inPX modules could then also be removed.
If you choose your unit at the SVG level, then all parameters on children inside that SVG element will have the same unit if you don't specify one.
Or is it a normal requirement to want different units inside a single SVG element?

I think I see but you might need to explain a bit more. Is it possible at the top-level  element to set up whether you want units in "num", "mm", "px" or "em"? And once you have done so, you then don't need any units within that  element.
This sounds ok to me. Do you think anyone would find this overly restrictive in practice? Or is this generally considered good practice when working with SVG to pick one kind of unit and stick with it?

Here is the Length type:
type Length
    = Cm Float
    | Em Float
    | Ex Float
    | In Float
    | Mm Float
    | Num Float
    | Pc Float
    | Percent Float
    | Pt Float
    | Px Float

One problematic one might be Percent. I could set up a grid in Px, but I still might want to give a Length in Percent for something, for example.

Yes, unitless values will follow whatever unit is set on the svg parent element.
https://stackoverflow.com/questions/12788422/svg-coordinate-system-points-vs-pixels?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa
I only use mm for everything.
So the idea was to make svg take length just like today, but change all the functions to take Float directly instead of a length.
It would simplify this library a bit.
But as you pointed out, I did not think of percent. I calculate  my SVG, so I do not need percent.  :/
Maybe wrapping in second svg will work, but may feel akward.

Also, when allowed to do mm/inch  length both on svg parent and on the children can cause srange behavior  if you also use a viewbox for zoom/scaling:
https://mpetroff.net/2013/08/analysis-of-svg-units/

I think that would work ok, assuming Percent is used less often.

I'm all for simplifying things, but I think this would actually throw out some useful design tools. Some of these units scale with the window, or the text-size, or don't scale at all, which is a useful tool in the hands of a skilled designer (which I'm not, but I've worked with them, and they're criminally underappreciated). So I'm all for having a consistent length unit (one of cm, mm, and in) but I think it would be a loss to not surface em, pt, px, etc.
I'm not sure if that's consistent with what's being proposed?

@neganp :
You still get all the todays length to use on svg elements:
    = Cm Float
    | Em Float
    | Ex Float
    | In Float
    | Mm Float
    | Num Float
    | Pc Float
    | Pt Float
    | Px Float

So an example with pixel units would be:
view model =
    svg
        [ width (px 200)
        , height (px 100)
        , viewBox 0 0 200 100
        ]
        [ circle
            [ cx 150
            , cy 150
            , r 30
            , fill Color.black
            , strokeWidth  2
            , stroke <| Color.rgba 90 60 60 0.5
            ]
            []
        ]

(The idea is that you only specify (px 50)/(mm 50)/(pt 50) +++  on the svg element.
But all the childrens width/height/cx/cy/r/stroke/fontSize is unitless)

Oh, nice. :slightly_smiling_face:
Thanks for your reassurance.


Various issues on osx

When I hit make from the src directory, the parser is rebuilt everytime
I would argue for fstar-mono.sh to add a --debug argument by default to the invocation of mono; it took me a while to figure out why I had backtraces without line numbers...

@beurdouche thoughts?

@protz: to me this isn't very problematic. Maybe @s-zanella will have a different opinion ?

Cannot reproduce the first issue anymore.
I would just remove bin/fstar-mono.sh. It calls fstar-mono.exe, which is never generated.

Closing via 2c647f017a18294a534242c29daed2796a05531e


gstkvssink.h:43:10: fatal error: 'gst/gst.h' file not found

Below with the error, while running install-script
mac$ ./install-script
===========================Starting to build ===========================
Kinesis Video Stream Producer SDK and Demo Application
========================================================================
Following open source libraries and tools will be downloaded and built
before building the producer sdk.
========================================================================
# ------------------------- BUILD-TOOLS -------------------------------#
autoconf   automake
bison      curl
flex       gettext
googletest iconv
libffi     libpcre
libtools   log4cplus
m4         openssl
zlib
    # ------------------------- GSTREAMER - for demo application --------#
    GStreamer, GStreamer plugins, yasm, gst-libav and x264.
    ------------------------------------------------------------------------
    *** NOTE *** for MacOS High Sierra: Use
        brew install autoconf
        brew install bison before running this script
    ------------------------------------------------------------------------
    ========================================================================
    This install script downloads and builds the dependent libraries from
    source in the ./kinesis-video-native-build/downloads directory only.
    ========================================================================

  Y - Confirm to proceed with Producer SDK build
  N - Exit Producer SDK build

Enter selection: Y
Include building GStreamer extra plugins= Y
Additional curl options=
cmake version 3.11.4 CMake suite maintained and supported by Kitware (kitware.com/cmake).
Checking openssl at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libssl.a
Checking curl at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libcurl.a
Checking log4cplus at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/liblog4cplus.dylib/.so
Checking gettext at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libgettextlib.dylib/.so
Checking libffi at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libffi.a
Checking pcre at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libpcre.dylib/.so
Checking zlib at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libz.dylib/.so
Checking iconv at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libiconv.dylib/.so
Checking glib at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libglib-2.0.dylib/.so
Checking googletest at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libgtest.a
Checking gstreamer at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libgstreamer-1.0.dylib/.so
Checking gst-plugin-base at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libgstvideo-1.0.dylib/.so
Checking gst-plugin-good at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/gstreamer-1.0/libgstavi.la
Checking gst-plugin-bad at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libgstbadvideo-1.0.dylib/.so
Checking for yasm
Checking gst-libav in /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/
Checking x264 at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libx264.dylib/.so
Checking gst-plugin-ugly at /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/gstreamer-1.0/libgstx264.dylib/.so
-- System name is Darwin
-- Kinesis Video PIC path is ../kinesis-video-pic
-- Kinesis Video Producer path is ../kinesis-video-producer
-- Kinesis Video JNI path is ../kinesis-video-producer-jni
-- Kinesis Video Gstreamer demp app path is ../kinesis-video-gst-demo
-- Kinesis Video Open Source path is /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads
-- Configuration Unix Makefiles
-- Bitness 64 bits
-- exe libraries Threads::Threads
-- Configuring done
-- Generating done
-- Build files have been written to: /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build
[  1%] Building CXX object CMakeFiles/gstkvsplugintest.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-gstreamer-plugin/plugin-src/gstkvstest.cpp.o
[  1%] Building CXX object CMakeFiles/producer.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/AwsV4Signer.cpp.o
In file included from /opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-gstreamer-plugin/plugin-src/gstkvstest.cpp:1:
/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-gstreamer-plugin/plugin-src/gstkvssink.h:43:10: fatal error: 'gst/gst.h' file not found
#include <gst/gst.h>
^~~~~~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/gstkvsplugintest.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-gstreamer-plugin/plugin-src/gstkvstest.cpp.o] Error 1
make[1]: *** [CMakeFiles/gstkvsplugintest.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  2%] Building CXX object CMakeFiles/producer.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/OpenSSLThreadCallbacks.cpp.o
/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/AwsV4Signer.cpp:4:10: fatal error: 'openssl/hmac.h' file not found
#include <openssl/hmac.h>
^~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/producer.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/AwsV4Signer.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/OpenSSLThreadCallbacks.cpp:3:10: fatal error: 'openssl/crypto.h' file not found
#include <openssl/crypto.h>
^~~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/producer.dir/opt/aws_demos/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-producer/src/OpenSSLThreadCallbacks.cpp.o] Error 1
make[1]: *** [CMakeFiles/producer.dir/all] Error 2
make: *** [all] Error 2

@arasub Any update on this issue?

Closing the issue as the question has been answered.

on ubuntu i am also not find found gst/gst.h file
it return fatal error: 'gst/gst.h' file not found

@rraghav476 did you follow the install steps? You need to install the following:
sudo apt-get install libssl-dev libcurl4-openssl-dev liblog4cplus-dev libgstreamer1.0-dev libgstreamer-plugins-base1.0-dev gstreamer1.0-plugins-base-apps gstreamer1.0-plugins-bad gstreamer1.0-plugins-good gstreamer1.0-plugins-ugly gstreamer1.0-tools


[client] deleting pastes from profile page

the issue here is that the paste card in the profile page is a link, so cant really have buttons inside a a tag...


PolyDeploy Client Watch Folder option

I'd like to have a CMD utility where I specify a folder, and the utility watches the folder indefinitely or until I stop it.
When a Module zip is dropped into the folder, the CMD utility uploads the zips via the DeployClient.
If it installs successfully, it's moved into a subfolder "Deployed", otherwise it goes in a subfolder "Failed".
I don't think this should be part of the DeployClient, but a separate cmd utility that interacts with the DeployClient.
If there's no interest to incorporate it into the PolyDeploy repo, I can create a separate repo to develop it for myself.

For me the appeal is that I can just drop a zip into the folder manually, or as part of my Visual Studio build process to automate the Module Deployments. I see this as a local dev tool, but I don't think its useful for CI/CD systems, or for dev/staging/prod environments


AirPressure Move Constructor is not marked noexcept

Original report (archived issue) by Michael Carroll (Bitbucket: Michael Carroll, GitHub: mjcarroll).

It should be marked noexcept like the rest of the move constructors, but would require an ABI break.

AirPressure uses IMPL_PTR from ign-utils starting from sdf12, which solves this issue.


Aehttpclient test is unstable

Sometimes Aehttpclient test fails, becouse of different transaction ordering in Pool (I think it depends on hash and that depends on generated addresses).
Example fail:
https://travis-ci.org/aeternity/elixir-node/builds/439842704

great use for 666 as issue number :joy:

:rofl: :rofl: :rofl:


Add 'wgu-geolocator-layer' to permalink ignore list

This adds the layer to show the geolocation on the map to the ignore layer list of the permalink module, since it makes no sense to have it in the permalink and the modules are (currently) not restored.
/cc @justb4 @fschmenger

@chrismayer For now we should also add 'wgu-drag-drop-layer' to the ignore list.
I think a better long term fix for permalink would be to ignore any layer added dynamically on a module level (not via config and factory) - since those most likely wont work.
An implementation would be fairly straightforward:

The layer factory will add a custom attribute 'usePermalink' to the OL layer list and default this flag to true (maybe with an option turn it off via config).
Permalink controller evaluates this flag (instead of using static ignore list).

Then application code can decide whether it makes sense to flag its own layers as usable for permalink.

Be aware, that 'wgu-drap-drop-layer' contains a random suffix, so it's not a one-liner fix.
I see no problem incorporating the PR. Should we convert the stuff stated above to an issue?

Yes, I vote for merging this and having an issue for the more general approach. Would you create such an issue @fschmenger ?

Agreed, merge this for now and I will open an issue.

Thanks for the issue tracking @fschmenger!


[Feature & Bug Repor] Add support for Scoped Enum (enum class)

Is your feature request related to a problem? Please describe.
Thanks to HLSL2021 I can share enumeration declarations between C++ and HLSL202x.
Bug Report: However whenever I declare the enum class HLSL does not complain until I actually try to use the scoped enum. Seems that enum class passes through your tokenizer unharmed, but then the rest of the logic is gone.
Describe the solution you'd like
For the compiler to support scope enum lookup enum_t:: without errors.
Describe alternatives you've considered
Not using scoped enums but a lot of my C++ code I want to expose to HLSL already does, so not fun.
Additional context
Here's a repro:
https://godbolt.org/z/1TdYh9E1a

Just my 2c, but I think enum class as a feature in C++ leaves much to be desired and don't know that the goal of future HLSL design should be driven primarily by the metric of "code shareability with C++."

Just my 2c, but I think enum class as a feature in C++ leaves much to be desired and don't know that the goal of future HLSL design should be driven primarily by the metric of "code shareability with C++."

would you care to elaborate on the "much to be desired" ?

Sure. As used currently, people often declare an enum class not specifically for strong-typing, but in order to get scoped enum field declarations. In exchange, you lose the ability to easily use enum class fields as array indexing operators, bit flags, or myriad other use cases. You can't provide a conversion operator for enum class types to circumvent this issue either, so you end up resorting to a soup of macros or nasty infectious templates to provide that behavior. The syntax itself is non-obvious (both enum class and enum struct map to the same type of declaration with no distinction, but my god what an awful naming scheme that is). I think we're better off making regular enum fields have proper lexical scoping first.

In exchange, you lose the ability to easily use enum class fields as array indexing operators, bit flags, or myriad other use cases.

you can use them as bit flags, you just need to declare the bitwise operators for them.
I think its great because it stops you "accidentally" bitwise-op'ing unrelated bitflags.

You can't provide a conversion operator for enum class types to circumvent this issue either, so you end up resorting to a soup of macros or nasty infectious templates to provide that behavior.

thats kinda why we have a core::bitflag<> in our engine to deal with all that.

HLSL 2017 and earlier all support C++11-style enum class declarations, and the accessing seems to work for me (Compiler Explorer Link).
Can you provide an example of what doesn't work for you? The link you provided doesn't actually have an enum class declaration.

yeah when you change the enum to class add scope the enums in the example I gave then it starts complaining about not being able to use a scoped enum as as integral expression (whereas it uses an unscoped one just fine)
https://godbolt.org/z/Pxd1zacr7


Clean up

Clean up some dust

[x] get the build going
[x] check if app runs on Android 13
[x] check if project can be updated to later sdk (?)

All resolved in latest dev branch tip

TODO:

Clean up commits
fix test
squash and merge into master


Nested OwnsMany entities 'VistLambda' error

I've multiple nested owns many relationships in my entity. Able to insert the data into cosmos db but facing below issue while retrieving data

Steps to reproduce
Same as issue https://github.com/aspnet/EntityFrameworkCore/issues/18265 that was marked as closed but still not working for me.
Entity configuration:

This same entity configuration worked on efcore 3.0 previewPlease let me know if I missed anything.
Thank you,
Further technical details
EF Core version: <IP_ADDRESS>
Database provider:Microsoft.EntityFrameworkCore.Cosmos
Target framework: (e.g. .NET Core 3.0)
Operating system:
IDE: (e.g. Visual Studio 2019 16.3)

Issue #18265 was fixed in 3.1.0-preview2. Try that version when it's out or a nightly.


Improve S3 Batch copy of Dataset to handle larger amounts of files

Enabler
follow on  from https://github.com/linz/geospatial-data-lake-team/issues/69
S3 batch copy was implemented. To do this a csv manifest file is generated to specify every asset and metadata file to be copied.
It was decided that the MVP datasets are small enough to write the file in one go in the lambda. However if we were to support significantly larger datasets in the future, this csv file may become too large to write in one go.
So we may need to think about implementing Batch Gets from the DynamoDB table  and creating a multipart upload to the s3 bucket  in order to process all of the files without the processing Lambda running out of memory.
Acceptance Criteria
S3 Batch copy of Dataset into storage bucket works for very large datasets.
Tasks
Definition of Ready
Definition of Done

Thanks @MitchellPaff.

Resolved elsewhere


GetObject() in Salesforce module conflicts with another Reference

Hello... I believe that the GetObject() in Salesforce module conflicts with another Reference, I believe OleAutomation object.  Since the GetObject() function in the Salesforce module is not used and not called anywhere else... I recommend simply renaming that function.  I have done so in my local copy.  Cheers!
Mark

Thanks @mwoollen should have figured such a common method name was used somewhere, I'll change it soon


__n() treats zero as singular

Given that we have the following function:
i18n.__n('%s movie', '%s movies', 0);

We get 0 movie back. Surely, this should be 0 movies? Is there a specific reason for this behaviour?
Cheers!

In fact; it depends a lot of the language. In English; 0 is considered plural; in French not, and so on.

Actually we'll see how https://github.com/jeresig/i18n-node-2/issues/78 resolves


[Feature] One combo, many snippets

Describe the solution you'd like
If one combo is attached to more than 1 snippet it would be neat if a searchable pull down of snippets would popup for me to choose the one I need. That would be useful for entering one of one's many email addressed with a single combo. I use this feature a lot in Phrase express as I have to remember less combos and loose little time.
Describe alternatives you've considered
A combo attached to group which would list all combos in the group for me to select from.
Additional context
Add any other context or screenshots about the feature request here.

Currently if a keyword is attached to more than one snippet, Beeftext will pick randomly.
However, you use the picker window. Invoke the picker window, type the keyword, and manually pick the entry you want using the arrows and return keys.

@xmichelo Is the shortcut the only way to open the Picker Window?
I'm also looking for what zrotar asked: getting the Picker window open without pressing the shortcut keys before, just by hitting the snippets.
The use case is near-identical snippets:
First example:
Snippet: Jon Doe, Keyword: jon
Snippet<EMAIL_ADDRESS>Keyword: jon
Snippet<EMAIL_ADDRESS>Keyword: jon
Second example:
Snippet: National id 1234567, Keyword: 123
Snippet: Tax id 1234567-TAX-ABCDE, Keyword: 123

Right now the shortcut is indeed the only way to trigger the picker window. But your suggestion is interesting. I could add an option to trigger display the picker window and prefill the search box with your keyword.


soc: arm: atmel_sam: Clock clean-up

Small clock clean-up. This moves all devicetree entries on SoC to atmel_sam_dt.h.
Signed-off-by: Gerson Fernando Budke<EMAIL_ADDRESS>

rebase

@galak pls, see if your fb is addressed

rebase and switch from DT_NODELABAL to DT_PATH following same approach on #24602.


Complete README update

Hey team,
Per ongoing convos with @indirect, we've revamped the README. I rewrote content and rearranged various sections.
Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.
What was the end-user problem that led to this PR?
The problem was...we wanted to make an easy-to-read README that included clear instructions for contributing to Bundler as well as providing several troubleshooting options up front.
What was your diagnosis of the problem?
My diagnosis was...to rewrite the things and rearrange sections!
What is your fix for the problem, implemented in this PR?
My fix...was to make copy and layout changes.
Why did you choose this fix out of the possible options?
I chose this fix because...@indirect and I discussed it beforehand :)

@colby-swandale @segiddins I made two small changes per your feedback. Once @indirect tells us more about bundle-report-bug and the graph re: paid contributors through Ruby Together, I'll make those changes (won't be for a few days, though).
Thx!

@indirect pinging about bundle-report-bug above; should I remove?

ping @indirect


fix(DataStore): Improve ObserveQuery for multiple sort input

Issue #, if available:
Description of changes:
Change the sort algorithm applied on an ObserveQuery subscription that contains sort inputs. Before the code was sorting the array based on the sort inputs. For 10k items, and more than one sort input, the performance was around 40s to generate a new snapshot. In this PR, we move over to an algorithm that maintains the list as sorted list by binary searching the sorted list for the index which the incoming model should be added at. Performance for this is in the range of sub 1 second
Check points:

[ ] Added new tests to cover change, if needed
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1464 (c2d12d0) into main (3bcd155) will decrease coverage by 5.49%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1464      +/-   ##
==========================================
- Coverage   58.28%   52.79%   -5.50%
==========================================
  Files         675      593      -82
  Lines       20565    15408    -5157
==========================================
- Hits        11987     8135    -3852
+ Misses       8578     7273    -1305

Flag
Coverage Δ

API_plugin_unit_test
61.41% <ø> (-0.29%)
:arrow_down:

AWSPluginsCore
72.86% <ø> (ø)

Amplify
46.02% <ø> (-0.03%)
:arrow_down:

Analytics_plugin_unit_test
71.76% <ø> (ø)

Auth_plugin_unit_test
78.76% <ø> (ø)

DataStore_plugin_unit_test
?

Predictions_plugin_unit_test
26.81% <ø> (ø)

Storage_plugin_unit_test
58.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...Configuration/AmplifyAWSServiceConfiguration.swift
82.60% <ø> (ø)

...CategoryPlugin/Operation/AWSGraphQLOperation.swift
50.00% <0.00%> (-4.66%)
:arrow_down:

...n/Operation/AWSGraphQLOperation+APIOperation.swift
41.81% <0.00%> (-3.64%)
:arrow_down:

.../DataStore/Model/Internal/Schema/ModelSchema.swift
62.50% <0.00%> (-2.09%)
:arrow_down:

...reCategoryPlugin/Storage/SQLite/Model+SQLite.swift

...tegoryPlugin/Storage/SQLite/QuerySort+SQLite.swift

...goryPlugin/Storage/SQLite/ModelSchema+SQLite.swift

...Plugin/Sync/InitialSync/InitialSyncOperation.swift

...eCategoryPlugin/Storage/ModelStorageBehavior.swift

...cileAndLocalSave/AWSModelReconciliationQueue.swift

... and 74 more

Continue to review full report at Codecov.

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


Disconnect bitness of DWARF and target architecture in libunwindstack

This change disconnects the bitness of DWARF (DWARF-32 or DWARF-64)
from the bitness of the target architecture, which were (incorrectly)
connected in libunwindstack when no encoding was given explicitly. The
main change is only a few lines of code in DwarfSection.cpp that set the
encoding size to the size of the target address size when needed (this
is potentially overridden later by explicit choice in the augmentation string
or by the address size parameter in DWARF version 4 or 5).
Relevant sections in the DWARF standard 4 and 5 (available
at https://dwarfstd.org/) are 7.4 and 7.5.2.
While the actual code change is small, multiple unit tests assumed the
original behavior and needed to be adapted. In detail:

Fixed PeCoffInterfaceTest due to incorrect CIE/FDE setup (wrong type
sizes for the target address).
Fixed DwarfEhFrameTest by enforcing encoding size using augmentation
string. The current code for .eh_frame parsing assumes a value size of
4 bytes for encoded target addresses. This is probably okay in practice,
as the encoding is typically (always?) given explicitly in the .eh_frame
case, but we need to work around this for unit testing.
Fixed DwarfEhFrameWithHdrTest by enforcing encoding size using
augmentation string, for the same reason as stated above.
Fixed DwarfDebugFrameTest by correcting the FDE setup and making it
depend on the chosen target address size.

@danielfenner @beckerhe Thanks for the review, I hope it looks good now!


refactor: Make types internal as users should only use interface

We don't need to make the implementation public

I will merge the PR - let me know if this was on purpose, then I will revert it.

It's ok, making them internal is fine.
The RunAt and IsOneTimeJob shouldn't be a part of that PR.
I was using those properties in a separate branch to deal with state management and they came along with my cherry picking.

Sorry that I deleted them - I thought they were a left-over as in the PR we discussed to delete that line.


feat: add nix flake support

Describe the solution
Nix is a package manager which, technically, can handle build in any language.
This flake.nix file is basically giving all the steps to build and import mask in their own flakes.
People can then, with the nix package manager, use mask right from this repository:
nix run "github:jacobdeichert/mask" -- --help

You can test it with my PR repository:
nix run "github:bachrc/mask" -- --help

Even if Mask is already available on the nixpkgs repository, but this flake remove any dependency on the nixpkgs repository system (time for the PR to be merged, etc.)
This flake also allows to have a dev environment with every needed dependency :
nix develop "github:jacobdeichert/mask"

The .envrc file enables integration with direnv, which basically triggers a nix develop when entering the folder (and then sets up the right build inputs versions)
I added a rust-toolchain.toml file which specifies the rust version needed to build the project. The nix flake is based on it in order to build it.
Your project is really great and I'm happy to contribute in any way to it! Please contact me if you need any changes

Hey, thanks for the PR! I don't have enough time to review it right now, but I expect to get to it within the next month or two.
It would be great if you could link to one or more popular Rust projects that have this exact same setup for me to reference. Since I'm not aware of how common this is, proof of other projects doing this might help me understand this better. Thanks!

Hey again, thanks for your interest in mask! However, I've decided not to merge this PR.
After looking at a few other CLI projects on GitHub, none of the ones I looked at have included flake support. And after reviewing the committed files, this is not something I feel comfortable maintaining.

Okay, I totally understand. Please have a great day. :)


Cannot sign in on Huawei

When I try to sign in, I get this error:

It will probably be difficult, but the option for a huawei login is necessary for huawei users like myself.
Like @LisoUseInAIKyrios mentioned here, sooner or later almost all users will be affected by this.
A (maybe) possible solution
A few months ago, I opened an isuue about huawei login, where the main dev gave us this explanation.
Another user provided a somewhat working solution that allows signing in microG, but which doesn't work when signing in actual gapps.
What if, alongside the main RV GMS Core release, there's a -hw one that allows signing in to ReVanced on huawei, also provided that the main -hw release adds proper support for more huawei devices in the future.
Another (maybe) possible solution could be doing this.

Because this is a fork that is only supposed to enable functionality with the GmsCore support patch, issues such as this would have to be resolved upstream. We can release the other build configurations, but upstream also doesn't, so we don't either. Why don't they?

I just tried what you recommended and I was able to sign-in
BUT
It doesn't function properly beyond that.
Since ReX switched to RV GMS Core in their latest release. I ran it, but it closed immediately upon launch.
So, it cannot be used to login and use any gapps, even modified. As oSum mentioned, it's an upstream issue and at this point, the RV team can't do much about it.

This is not a support channel for unofficial patches. Move to their respective places.

I just tried what you recommended and I was able to sign-in
BUT
It doesn't function properly beyond that.
Since ReX switched to RV GMS Core in their latest release, I ran it, but it closed immediately upon launch.
So, it cannot be used to login and use any gapps, even modified. As oSum mentioned, it's an upstream issue and at this point, the RV team can't do much about it.

download the YouTube app from Aurora store or anywhere it use it as a spoof. Then, the revanced work normal.


beware if unity project not in root directory nothing will work!

Fixed in https://github.com/3pjgames/unity-git-hooks/commit/8944e01c8eeae27c4e52ee6734caa6a6d48dfd67
Now the assets directory can be configured via git config


Disable handshake_verify_peer_options_test

Disable handshake_verify_peer_options_test as per b/178094682
@ZhenLian , should we also delete the API?

This change is

We will just need to put out a gRFC. The Core API is not stable API, so it should be fine to delete it in my opinion if no one needs it. If someone does need it, they will reply on the gRFC. As far as I am aware I doubt it is used by anyone since it is not exposed by any wrapped language.


Bug: serious lag at startup

Description
https://github.com/user-attachments/assets/17978a04-0dcc-4467-8a20-71a56d07a71e
Steps To Reproduce

Open the Files

Files Version
<IP_ADDRESS>
Windows Version
22631.4460
User ID
5f94dc44-df39-42c2-b6c0-13aed9a247ab
Log File
debug.log

Is this every launch, or just the first one?

Is this every launch, or just the first one?

Every launch,there is a second startup in the video.

You can work around this by enabling the option to keep Files open in the background, however, we're also tracking performance for a cold start in #4180.

You can work around this by enabling the option to keep Files open in the background, however, we're also tracking performance for a cold start in #4180.

There will still be problems

https://github.com/user-attachments/assets/06f59498-c906-443c-97a8-434613a0ef7c

The entire window seems to be frozen, and I can't even drag the window position.

It doesn't look like the option is working correctly, try running taskkill -f -im files.exe and testing again.

taskkill -f -im files.exe

thx


Added Anagram Calculator

Fixes Issue🛠

Closes #730
Description👨💻
Anagram Calculator is used to check for 2 strings whether one string can be rearranged to get another string and vice versa.
For example:
string 1: car
string 2: arc
We can rearrange the letters in string 1 to get string 2, and we can rearrange the letters in string 2 to get string 1
so the output will be ANAGRAM.
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅

Tested Locally

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳 @ImmidiSivani


[FLINK-18632][table-planner-blink] Assign row kind from input to outp…

What is the purpose of the change
This pull request assign row kind from input row to output row, otherwise every output row kind will be initiated by default type(insert type).
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Could you add a test to reproduce this problem?

Could you add a test to reproduce this problem?

OK，should i create a new java class named like PojoTypeTest under flink-table-planner-blink module parallel with class NullTypeTest?

I think you can add tests in StreamTableEnvironmentITCase and reuse the POJO classes there.

I think you can add tests in StreamTableEnvironmentITCase and reuse the POJO classes there.

OK. I have already add a test case.

I've simplified the test case, the last one has a little ambiguous.

It's my pleasure.


Get BOINC running on the PS4

 Issue by grctest
Sunday Apr 16, 2017 at 16:36 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/110

I've seen that some individuals have managed to get Linux running on a hacked PS4 and that they have been able to run Steam and applications.
What i'm very interested in finding out is if it's possible to run BOINC on a PS4 and successfully complete work units for any existing BOINC project?
If it's possible to run BOINC on a PS4 then we could create a PS4 supercomputer, or potentially run existing BOINC projects similar to running folding@home on the PS3 when it had linux capabilities.

https://www.reddit.com/r/ps4homebrew/comments/65mq4c/is_it_possible_to_run_boinc_within_linux_on_a/
If BOINC works within Linux on homebrew enabled PS4s, then this could be a great marketing opportunity for Gridcoin and BOINC as well as bring a large amount of computing power to BOINC.
There needs to be an updated homebrew entry point though, otherwise you either need to be in closed circles or have an older PS4 to run Linux.

 Comment by grctest
Friday Apr 21, 2017 at 19:15 GMT

Looks like firmware 4.5x has a working exploit, now we're just waiting for a kernel exploit to be ported then the whole thing to be made public & the PS4 supercomputer is a go!


capturePage quality

It would be cool if the quality of the screenshots were adjustable. At the moment, some screenshots have 0.5 - 1 MB, in some cases this is not that handy e.g. if you only want to provide previews or something like that.

In v0.21.3 there is a new API to do it:
https://github.com/atom/atom-shell/blob/master/docs/api/native-image.md#nativeimagetojpegquality


The shim is not surfacing variable provider errors

I'm running a Spin app that expects there to be a provided variable redis_endpoint. When I run this with the spintainer executor (basically just running a spin process in a container) I get the following error in the logs:
Error: provider error: no provider resolved required variable "redis_endpoint"

Caused by:
       no provider resolved required variable "redis_endpoint"

When I run it with the shim I see no errors in the logs and nothing in the k8s events. We should bubble up this error somewhere.

Duplicate of https://github.com/spinkube/containerd-shim-spin/issues/174?

Looks like it.


Release/2.1.0 merge to main

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[x] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Merges release/2.1.0 into main as part of Gitflow.
https://github.com/stellar/java-stellar-anchor-sdk/issues/953
Why
Gitflow
Known limitations
N/A

Reference Server Preview is available here:https://anchor-ref-pr954.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr954.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr954.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr954.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr954.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr954.previews.kube001.services.stellar-ops.com/


Fix linter warnings/errors in tests

What are these changes?
Fix linter warnings/errors introduced from using the jestPlugin.configs["flat/recommended"] ruleset.
Why are these changes being made?
Linter warnings and errors are bad.
Checklist before merging

[X] ./toolbox -- run clean-verify succeeds.

coverage: 90.728%. remained the same
when pulling 3afe133fd30e6813981effeeabb9145221c42a96 on test-fixes
into bcc14dcaff7e421e61cb804efa1469d0a4410dac on mainline.