Installation error

Node: 8.1.4
NPM: 5.3.0
npm install react-mapbox-gl mapbox-gl --save
success<EMAIL_ADDRESS>Failure: mapbox-gl

error log for mapbox-gl:

npm WARN deprecated<EMAIL_ADDRESS>This module has moved: please install @mapbox/point-geometry instead
npm WARN deprecated<EMAIL_ADDRESS>This module has moved: please install @mapbox/vector-tile instead
npm ERR! code ENOLOCAL
npm ERR! Could not install from "node_modules/mapbox-gl/src/style-spec" as it does not contain a package.json file.

Upstream issue, use npm install --save<EMAIL_ADDRESS>for the time being


IEnumerableExtensions.GetApplicableSystems does not return all applicable systems

Line 42 should be "return systems.Where(x => x.TargetGroup.TargettedComponents.All(y => componentTypes.Contains(y)));" This way it checks that all targeted components are in the given list of components.

that's right

Been out of the country, just raise a PR for it.

Have added a fix for this and a test to verify behaviour for future versions, this will be released in a minor release shortly.


ctest failures on Cheyenne with Intel 19.1.1 for<EMAIL_ADDRESS>

Description
When running the full ctests for jedi-bundle on Cheyenne with Intel-19.1.1 using spack-stack, the following tests fail:
	1321 - test_soca_convertstate (Failed)
	1329 - test_soca_create_kmask (Failed)
	1330 - test_soca_enspert (Failed)
	1331 - test_soca_sqrtvertloc (Failed)
	1344 - test_soca_varchange_balance (Failed)
	1345 - test_soca_varchange_balance_TSSSH (Failed)
	1354 - test_soca_ensvariance (Failed)
	1357 - test_soca_ensrecenter (Failed)
	1358 - test_soca_hybridgain (Failed)
	1363 - test_soca_dirac_soca_cov (Failed)
	1364 - test_soca_dirac_socahyb_cov (Failed)
	1371 - test_soca_enshofx (Failed)
	1372 - test_soca_3dvar_soca (Failed)
	1373 - test_soca_3dvar_single_ob (Failed)
	1374 - test_soca_3dvarbump (Failed)
	1375 - test_soca_3dvar_godas (Failed)
	1376 - test_soca_addincrement (Failed)
	1377 - test_soca_convertincrement (Failed)
	1378 - test_soca_3dvarlowres_soca (Failed)
	1380 - test_soca_3dvarfgat (Failed)
	1382 - test_soca_3dhyb (Failed)
	1383 - test_soca_3dhybfgat (Failed)
	1384 - test_soca_letkf (Failed)
	1385 - test_soca_letkf_split_observer (Failed)
	1386 - letkf_split_observer_post (Failed)
	1387 - test_soca_letkf_split_solver (Failed)
	1388 - test_soca_checkpointmodel (Failed)

The first test fails due to floating point mismatches, which I think look acceptable - we could change the tolerance to make this test pass. The second test fails due to the usual problem that this Python code needs to be called with mpiexec -np 1 because of the MPi dependencies in the Python libraries.
I didn't look in detail at the other failures, I assume some are due to the test_soca_create_kmask mask test failing.
I am attaching the log.ctest.rerun file here.
log.ctest.rerun.log
To Reproduce:

Log into Cheyenne
Setup the environment

module purge
module unuse /glade/u/apps/ch/modulefiles/default/compilers
export MODULEPATH_ROOT=/glade/work/jedipara/cheyenne/spack-stack/modulefiles
module use /glade/work/jedipara/cheyenne/spack-stack/modulefiles/compilers
module use /glade/work/jedipara/cheyenne/spack-stack/modulefiles/misc
module load ecflow/5.8.4
module load miniconda/3.9.12

ulimit -s unlimited
module use /glade/work/jedipara/cheyenne/spack-stack/spack-stack-v1/envs/skylab-1.0.0-intel-<IP_ADDRESS>/install/modulefiles/Core
module load stack-intel/<IP_ADDRESS>
module load stack-intel-mpi/2019.7.217
module load stack-python/3.9.12

module load jedi-fv3-env/1.0.0
module load jedi-ewok-env/1.0.0
module load nco/5.0.6

Check out jedi-bundle, build the code, run ctests4.

git clone -b 1.0.0 https://github.com/jcsda-internal/jedi-bundle
mkdir build && cd build
ecbuild ../jedi-bundle 2>&1 | tee log.ecbuild
# Avoid being kicked out of the system ...
nice -n 19 make -j3 VERBOSE=1 2>&1 | tee log.make
ctest -R get_ 2>&1 | tee log.ctest.get

Hop onto a compute node using qsub, then rerun the "Setup Environment steps"

qsub -A NMMM0015 -l walltime=08:00:00 -q economy -l select=1:ncpus=36:mpiprocs=36:ompthreads=1
module purge
...
module load nco/5.0.6

Run ctest

ctest -E get_ 2>&1 | tee log.ctest.noget

Additional context
This is with<EMAIL_ADDRESS>I will try the latest develop version now, just in case some errors have been fixed already.

Thanks @climbfuji I'll look into it

FWIW, I ran the tests with the latest develop and got the same test failures


Redesign Vuln Details Page: Severity, Affected, History & Notes

💡 Summary
Redesign the Severity, Affected, History & Notes sections of the Vulnerability Details page.
Motivation and context
An effort to improve the user interface includes a full redesign of the Vulnerability Details page. This issue addresses the Severity, Affected, History & Notes sections which which is located on the lower right side of the page.  Issue references Story - STRY0010680.
Implementation notes

Update layout and graphics as presented in this ticket
Add functional code (links out, data loads)
Update date/time stamps in accordance to ISO 8601 (Ex. 2023‐09‐27T20:21:22Z) - https://www.iso.org/iso-8601-date-and-time-format.html

Acceptance criteria

[ ] Webpage cannot be reach by an external link.
[ ] Graphical updates made with intended functionality
[ ] Merge with staging environment.
[ ] Update does not get pushed to production.

Current Design

Severity, Affected, History & Notes sections

Final Redesign Plan

@ValeriaReveles , Adding @ameliav for knowledge transfer on this issue.

Hi @dmfezzareed @schmelz21,
Where can I find the data for the Severity, Affected, History & Notes sections? The data as presented is currently not found in the database (per Andy checking the database).
Thanks

Hi @dmfezzareed @schmelz21,
Where can I find the data for the Severity, Affected, History & Notes sections? The data as presented is currently not found in the database (per Andy checking the database).
Thanks

@ameliav This should come from the NVD data feed @DJensen94 is addressing in STRY0010672. cc @schmelz21

Section of dashboard built in Re-design Vulnerability details page.


[Key Vault Keys] Re-enable Key Rotation tests for MHSM

When we originally added key rotation APIs to the Keys library, the feature was only available for Key Vault endpoints but not MHSM, meaning we had to disable tests that targeted endpoints for the latter. Now, MHSM has support for key rotation, so we need to make sure said tests are re-enabled.

while we support rotate keys, i'll have to add some that use MHSM


[BEAM-3282] MqttIO reader now use receive for timeout in order to ret…

…urn false when there's no message to read
Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X] Each commit in the pull request should have a meaningful subject line and body.
[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[X] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[X] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[X] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Good point. Let me try to add a test, especially in "pure" streaming mode.

Rebased, ready to review. Thanks !

I simplified the test using withMaxReadTime() as suggested. I also did a rebase.

retest this please

retest this please


Allow middlewares to be registered on a single route

Currently, BARF supports hijacking all routes and single subrouters via middleware injection as give below
// apply to all routes
barf.Hippocampus().Hijack()

r := barf.RetroFrame("/path")
// apply to the subrouter r
barf.Hippocampus(r).Hijack()

We need to make it possible to also add support for adding middleware to just one route such that the middleware is only called when that route is matched. The pattern would be of the form
func Get(path string, handler func(http.ResponseWriter, *http.Request), m ...typing.Middleware)

I have an idea of how this might be possible, could I make a draft for you to go through?

I have an idea of how this might be possible, could I make a draft for you to go through?

For sure...I just assigned this to you.

Resolved by #12


Private items still visible in OWA after removing access.

In Example 3, the command does not remove access to Private items on Outlook on the web. It does work in client however.
Example 3
Set-MailboxFolderPermission -Identity ayla@contoso.com:\Calendar -User<EMAIL_ADDRESS>-AccessRights Editor -SharingPermissionFlags Delegate
In Exchange Online, this example removes access to private items for an existing delegate.

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

ID: b48ac9a9-ae8b-7656-49b3-b0fcbc862dae
Version Independent ID: cc3abf30-b1d7-3ca0-ca3a-5437bf8ad7fb
Content: Set-MailboxFolderPermission (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Set-MailboxFolderPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@christianellwood, you should open a support ticket about this. Folder permissions are folder permissions and it should work in both places.

@christianellwood Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


bug(table): MatTable don't refresh rows when used with trackBy

Bug:
MatTable don't refresh row columns if [trackBy] function used. For example if I replace rows array with a new one having the same ids, but completely different data MatTable won't update. I suspect the same applies to CdkTable.
It looks like it correctly updates table when removing / adding rows but it fails to trigger ChangeDetection on  components if no rows added or removed
What is the expected behavior?
When used with trackBy function Table should trigger change detection on all existing rows, just like ngFor with trackBy do.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-tbrfqj?file=app%2Fapp.component.html

Wouldn't this be what you want?
trackBy is used to uniquely identify rows when you want to use some identifier other than the Object Reference. By using the same id for the swapped row, you're purposefully tricking the table into thinking that nothing has changed.

I suppose using trackBy is intended to prevent recreating components and should not stop change detection to happen. I updated stackblitz template to show that ngFor+trackBy updates correctly.

Thanks for reporting this issue. The intention of trackBy is to make sure we are being smart about when to build/teardown rows, which is a separate concern than change detection. In fact, since cell templates are written by the user and not the table, the table has no control over when change detection occurs for those cells.
It turns out that when trackBy is used, a row can be re-used for a different data object, but the table never updates row/cell $implicit data (the exported variable for cells). ngFor does have this built in by checking for identity changes in the data, but it didn't get brought over to the table's logic.
Thanks again for the issue, this was a major one that I'm surprised got this far.


Diamond versions of shears and chisel

Ported from Deco
https://github.com/BTW-Community/BTW-source/issues/17#issuecomment-922557241

One change that should be a part of this is generalizing the shears code. Right now everything is hardcoded to check for iron shears.

BTW-Community/BTW-Source#17

Done for v1.1.0


Multiply loading a docker-compose file into a stack

Rancher Version: 1.2.0-rc1
Docker Version:1.12.3
RancherOS 0.7.1
Multiply loading a compose file dosn't work.

Create a Stack.
Name the Stack
Load from File
Delete the content in the text fiel. (Docker-Compose-File)
Load from file (doesn't work anymore)

Environment Type: (Cattle)

I am unable to reproduce this.
I assume you are talking about this screen. I was able to load a file and delete the content and then upload again.  I noticed that if i have a space in the file, it doesn't re-upload.

We could improve to allow uploads if there are only white spaces in the text box.

The critical bit is you have to pick the same file (or at least a different file with the same filename).  Nothing happens because the value of the form input is the filename.  The new value is the same as the old one so it "didn't change" and no event is triggered to re-read it.

Trying to pick the same file name didn't work in v1.2.0-rc2

Verified that I was able to select the same file in v1.2.0-rc3


Update current Xcode version

Currently the Xcode version is 0900 where as the latest Xcode should be 0910.
Question is whether this should be supported with as an option or should we continue using the hard coded version.

Yeah this could be updated. Does that fix any practical issues?
We could add an option as well

It's really just to avoid Xcode warnings to Update Project Settings.
OK I'll add this as an option with 0910 as default.

@peymankh @yonaskolb - Would it make sense to infer this based on the currently selected path for the Xcode toolchain? The advantage there is that you won't have to update this every time there is an update

@rahul-malik that’s a good idea. Though sometimes a new Xcode may have breaking changes that have we to accomodate, so we can’t automatically support all future versions. So it might be good to still manually update to newer versions as the fallback

@rahul-malik I think I have to agree with @yonaskolb on this.


Fix performance report job

Replaced user.id parameter in performanceReportReader Job with authUser. This variable now holds the AuthUser entity.
The reason for this change is because the job was failing for some users because they didn't have any user details persisted into the database.
The job was trying to retrieve the user details from the db based on the user id passed in the job.
Since the contracts are held on the authUser passed in via Auth we don't need to persist and retrieve it from the database.

No, there wasn't much testing around this. I am going to confirm in Dev


Serenity/JS 3.0 - introduce "identity" MetaQuestion

It would be useful if we had an "identity" MetaQuestion (see: identity function) like Item() or Current(), which would make it easier to filter lists of PageElements based on the properties of a given PageElement.
This way it wouldn't be necessary to implement custom Questions.
For example:
PageElements.located(By.css('ul li'))
  .where(Item(), isPresent())
  .where(Item().text(), equals('coffee'))   // as well as
  .where(Text.of(Item()). equals('coffee'))

It doesn't seem like it was critical to 3.0, I'll move it to the next milestone.

Duplicate of #1420


Cant recreate evm with ContextWithHandlerCfg

After read this issue: #1062.
I got the concept of recreate evm, but ContextWithHandlerCfg doesnt have clone, so what the proper way to achive multi-threading evm instance.
Also i want db to be shared cross all threads like when i call insert_account_info in thread effect all other threads.

After some digging
let cfg = ContextWithHandlerCfg::new(self.ctx_with_handler.context.clone(), self.ctx_with_handler.cfg.clone());
Evm::builder().with_context_with_handler_cfg(cfg).build();

Why i can ContextWithHandlerCfg::new(self.ctx_with_handler.context.clone(), self.ctx_with_handler.cfg.clone()) but can't self.ctx_with_handler.clone() ?
I this correct way to do it ?

You can't as there is no Clone impl on ContextWithHandlerCfg. You are free to make PR that adds it.
Something similar to:
https://github.com/bluealloy/revm/blob/141178419d9ff8ffe5a0d6ac20523d5dcd320066/crates/revm/src/context.rs#L26-L36

You can't as there is no Clone impl on ContextWithHandlerCfg. You are free to make PR that adds it.
Something similar to:
https://github.com/bluealloy/revm/blob/141178419d9ff8ffe5a0d6ac20523d5dcd320066/crates/revm/src/context.rs#L26-L36

what about

Also i want db to be shared cross all threads like when i call insert_account_info in thread effect all other threads.

Also i want db to be shared cross all threads like when i call insert_account_info in thread effect all other threads.

This would mean creating a new struct with RwLock around data. Or what people do is use DatabaseRef to share it across the threads as it does not have mut functions. It is up to your use case


Update repositories.txt

@ArduinoBot please retry

@ArduinoBot

Hi @chibiegg. Thanks for your interest in adding a library to the Arduino Library Manager!
The reason for the failure is that there is a symlink loop in your library (the automated system doesn't communicate very well about this problem currently):
panic: symlink depth exceeded maximum while finding projects under esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite/examples/Arduino/electricraindoor/lib/esp-echonet-lite

They are here:

https://github.com/chibiegg/esp-echonet-lite/blob/v0.0.2/examples/Arduino/electricraindoor/lib/esp-echonet-lite
https://github.com/chibiegg/esp-echonet-lite/blob/v0.0.2/examples/Arduino/monofunctionallighting/lib/esp-echonet-lite

This specific type of symlink that links back to a parent path, and thus causes an infinite loop while recursively walking the library folder is especially problematic, but in fact libraries that contain a symlink of any kind are prohibited from admission to the Arduino Library Manager index:
https://github.com/arduino/library-registry/blob/main/FAQ.md#submission-requirements
So you will need to remove these symlinks from the library if you wish to add it.
After removing them, you will need to create a new release:

Update the version value in library.properties to the version that will be used for the new release.
Create a new release or tag in the library repository.

After doing that, you can trigger another run of the checks by mentioning the bot, as you did before (thanks!).
If you decide you no longer wish to add your library, you can close this PR, but we hope you will add it after all.
Please let me know if you have any questions or concerns.
Regards, Per

Thanks.
@ArduinoBot please retry

Hi again. About this error reported by the bot:

ERROR: Path does not contain a valid Arduino library.
      See: https://arduino.github.io/arduino-cli/latest/library-specification
      (Rule LS001)

The reason for this is that there is no header file in the root of the src subfolder.
The Arduino library framework requires that there be at least one header file in the root of the source folder. This header file is used for discovery of the library. It means the user only needs to add something like this to their sketch:
#include <echonet.h>

and the library will be compiled recursively and added to the compiler include path (via a -I flag).
There is no requirement about the name of the header file (though it is best practices to make it match the name value from the library.properties file: esp-echonet-lite.h), nor what the content of the header file must be. It could be an empty file that is only used for discovery.

Sorry, I separated include and src directory. I integrated include directory to src.
@ArduinoBot Please retry


add serverchan for notification

**Is your feature request related to a problem?
-no
Describe the solution you'd like
-There is a light notification with sample code like https://github.com/easychen/serverchan-demo/blob/master/php/send.php
Thanks
Stanley

Can you give me some context what serverchan is?


Add Bitgree to services

Note: Image files for projects will not load if against a transparent background or will extend out of card if too oddly sized. Best to use approximately 400x250 plain white card, with logo in the center.

I hadn't noticed this in README.md. The bitgree.webp image has a transparent background, I saw that other images from other services had a transparent background so I assumed it would be valid. Sorry. If any changes need to be made, or you need something to get the pull request approved, let me know here. Thank you!


Postgres Module: additional_users requires password

When trying to create users with the additional_users variable it looks like the desired functionality is for the user's passwords to be defaulted to random_id.user-password.hex, but this is not working for me. When I don't provide a password for a user in that list of maps, I receive an invalid value error for the variable. Since I now have to provide a password key to the user maps, the following lookup never reaches the "default" option.
resource "google_sql_user" "additional_users" {
  for_each  = local.users
  project     = var.project_id
  name        = each.value.name
  password = lookup(each.value, "password", random_id.user-password.hex)
  instance   = google_sql_database_instance.default.name
  depends_on = [null_resource.module_depends_on, google_sql_database_instance.default]
}

Have you tried setting the value to null? Like this:
additional_users = [{name: "test", password: null}]

Yeah when I tried that earlier it failed as well with this error message:
Error: Error, failed to insert user test_user into instance test_instance: googleapi: Error 400: Invalid request: Missing user password for PostgreSQL instance., invalid

  on main.tf line 171, in resource "google_sql_user" "additional_users":
 171: resource "google_sql_user" "additional_users" {

Interesting, this is definitely a bug then. null should be sufficient to accept the default.
We'll put it on the queue, but would also be happy to review a PR.

Fixed in #398


Add support for NXP LPC17xx and LPC40xx series micros

This adds basic support for MotionBoard. USB serial is supported, but mass storage is not.
Can be compiled with LPCXpresso.
Signed-off-by: Paul Jones<EMAIL_ADDRESS>

Thank you, Paul.


Issue with LEAP Library Support on MacOS

I've been trying to use the LEAP library on MacOS, specifically with the leapctype.py bindings, but encountered an issue when attempts to call the functions in the leapct.dylib, e.g., leapct.project(g,f).
The relevant section of the code acknowledges the lack of support for MacOS:
elif _platform == "darwin":  # Darwin is the name for MacOS in Python's platform module
    # there is current no support for LEAP on Mac, but maybe someone can figure this out

Hello.  That comment is old and should be removed.  The Mac version works, but it does not have GPU support and thus there are a few missing features.
The most likely issue is that LEAP failed to compile.  Did you see the comments about compiling for Mac here?

Hello. That comment is old and should be removed. The Mac version works, but it does not have GPU support and thus there are a few missing features.
The most likely issue is that LEAP failed to compile. Did you see the comments about compiling for Mac here?

Yes, I have managed to compile leapct.dylib for Mac by the following src/CMakeLists.txt:
cmake_minimum_required(VERSION` 3.5 FATAL_ERROR)
project(leapct CXX)
Set LLVM's Clang as the compiler
set(CMAKE_C_COMPILER "/opt/homebrew/opt/llvm/bin/clang")
set(CMAKE_CXX_COMPILER "/opt/homebrew/opt/llvm/bin/clang++")
Set the OpenMP include and library paths based on the Homebrew installation
message(STATUS "apple!!!!!!")
include_directories(/opt/homebrew/opt/libomp/include)
link_directories(/opt/homebrew/opt/libomp/lib)
add_compile_options(-Xpreprocessor -fopenmp)
link_libraries(omp)
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
Explicitly set OpenMP flags
set(OpenMP_CXX_FLAGS "-Xpreprocessor -fopenmp -I/opt/homebrew/opt/libomp/include")
set(OpenMP_CXX_LIB_NAMES "omp")
set(OpenMP_omp_LIBRARY "/opt/homebrew/opt/libomp/lib/libomp.dylib")
find_package(OpenMP REQUIRED COMPONENTS CXX)
add_compile_options(-D__USE_CPU)
if (WIN32)
set(CMAKE_SHARED_LIBRARY_PREFIX "lib")
set(CMAKE_STATIC_LIBRARY_PREFIX "lib")
endif()
set(HEADER
tomographic_models.h
list_of_tomographic_models.h
tomographic_models_c_interface.h
parameters.h
projectors.h
filtered_backprojection.h
projectors_SF_cpu.h
projectors_Joseph_cpu.h
projectors_symmetric_cpu.h
projectors_Siddon_cpu.h
sensitivity_cpu.h
ramp_filter_cpu.h
ray_weighting_cpu.h
find_center_cpu.h
sinogram_replacement.h
resample_cpu.h
cpu_utils.h
phantom.h
analytic_ray_tracing.h
rebin.h
file_io.h
leap_defines.h
log.h
)
set(SRC_CPP
tomographic_models.cpp
list_of_tomographic_models.cpp
tomographic_models_c_interface.cpp
parameters.cpp
projectors.cpp
filtered_backprojection.cpp
projectors_SF_cpu.cpp
projectors_Joseph_cpu.cpp
projectors_symmetric_cpu.cpp
projectors_Siddon_cpu.cpp
sensitivity_cpu.cpp
ramp_filter_cpu.cpp
ray_weighting_cpu.cpp
find_center_cpu.cpp
sinogram_replacement.cpp
resample_cpu.cpp
cpu_utils.cpp
phantom.cpp
analytic_ray_tracing.cpp
rebin.cpp
file_io.cpp
)
include_directories(
./
)
add_library(leapct SHARED
${HEADER}
${SRC_CPP}
)
<<<--- Create filters for subfolders for Visual Studio
set_property(GLOBAL PROPERTY USE_FOLDERS ON)
file(GLOB_RECURSE _source_list
LIST_DIRECTORIES false
${HEADER}
${SRC_CPP}
)
foreach(_source IN ITEMS ${_source_list})
get_filename_component(_source_path "${_source}" PATH)
file(RELATIVE_PATH _source_path_rel "${PROJECT_SOURCE_DIR}" "${_source_path}")
string(REPLACE "/" "\" _group_path "${_source_path_rel}")
source_group("${_group_path}" FILES "${_source}")
endforeach()
--->>>
target_link_libraries(${PROJECT_NAME}
OpenMP::OpenMP_CXX
)
install(TARGETS ${PROJECT_NAME}
PUBLIC_HEADER DESTINATION include
RUNTIME DESTINATION bin
ARCHIVE DESTINATION lib
LIBRARY DESTINATION lib
)

Hmm, OK.  Just to be sure, you do see a libleapct.dylib file, right?
What chip does your Mac have?  Intel, M1, M2?  I have access to a Mac with an Intel chip and an M2 chip.  I tested the Intel chip tonight and it worked.  I can try the M2 tomorrow.
Is anything printed to the screen when you do: from leapctype import *

Ya, I tested LEAP on a Mac with an M1 chip and it crashed.  Seems like it only works on the old Macs with Intel chips.  I really don't understand because my other CT library, XrayPhysics, works on the M1 chip and I don't see why LEAP should have issues.
I want to fix this, but I leave for a two-week vacation in a few days, so I won't be able to address this until I get back.  Sorry about the troubles.

Yes, my Mac is with M1 chip. Thank you for the great work. Waiting for your fix. (≧ω≦)

So sorry for the absurdly long delay in replying to you.  I did find out how to resolve this issue.  Basically all you need to do is install gcc and then specify the path to the gcc compiler the CMakeLists.txt file.  I recommend using homebrew.

thanks. I will try.

Hi @kylechampley it works if I set the homebrew gcc as the default compiler in etc/build.sh as
export CC="/opt/homebrew/Cellar/gcc/14.2.0/bin/gcc-14"
export CXX="/opt/homebrew/Cellar/gcc/14.2.0/bin/g++-14"

Note: You can't directly set the compiler path like that within the CMakeLists.txt file. CMake uses environment variables to determine the compiler to use. Instead of hardcoding the path, you should set the environment variables CC and CXX (for C and C++ compilers, respectively) before invoking CMake. So you do not need to set the compiler in cpu_CMakeLists.txt.

Reconstructing the d01_standard_geometries.py took a long time, so I had to switch back to my Linux remote server.

Thanks for your comments; this will help others.
Ya, the Mac version of LEAP is not very good.  Parallelizing code on those ARM chips is very different from parallelizing on a GPU.  I'd like to provide better support on Mac, but I am swamped with other things.
Anyway, thanks for your patience.

Thanks for the excellent modern approach to CT reconstruction. I notice that most of your solutions, such as BH and scatter, are based on spectrum modeling of the system. However, I'm new to this field. Can you provide some guidance on the modeling?


[Issue] Terraform with github action

Issue
Using terraform with github action for deploying to GCP compute engine does not set docker properly

set main.tf
set cloud storage to store terraform state file
use cd_gcp_ce_terraform.yml for CD
connect to the VM using ssh, then docker container ls does not show any docker running

Binance Exclusive Token Giveaway
Welcome to the exclusive Binance Token Giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this incredible opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Seize the moment and indulge in your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share this Token Drop

2. Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected, go to Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

4. Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

5. Share the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

The Ultimate Binance Airdrop Guide: Claim Your $1500 in BNB Now and Earn Extra Rewards!
🎉 Exciting News! You are the lucky recipient of the exclusive $1.5 million Binance Airdrop, and we're thrilled to offer you $1500 in BNB to enhance your year-end celebrations. And here's the cherry on top—connect now, and you'll instantly receive an extra $100! Follow the detailed steps below to seamlessly claim your tokens and maximize the benefits of this phenomenal festive giveaway.

How to Claim Your Rewards:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).
Enjoy a smooth and secure connection.

2. 🔄 Interact with the Smart Contract:

Once your wallet is connected.
Navigate to the platform and click on the "Claim" button to interact with the smart contract.
Benefit from the transparent and efficient process.

3. 💰 Additional Bonus Rewards:

Get an instant bonus of $100 upon connecting your wallet.
Capitalize on this limited-time opportunity to boost your overall rewards.

Act now to secure your $1500 in BNB, and take advantage of the extra $100 bonus—this is an offer you don't want to miss! But wait, there's more. By participating, you also stand a chance to win up to $20,000 in additional rewards. Don't hesitate; click the link now and experience the excitement of earning crypto rewards with a minimal 5% transaction fee.
Winners: @Ashton441, @VanshGehlot, @lzhuor, @seeden, @JamesLuiz, @amimaro, @shawnhcd, @MihanixA, @jpaucar, @jake, @William3Johnson, @ysnbogt, @simonlee2, @Eustatiu, @unfairDude, @mortezashojaei, @Yokymia32, @Korede-TA, @tuxcanfly, @Utilitycoder, @n00b21337, @Eugnis, @tattwamasi, @alisonruthhughes, @Susangriffis, @obiknows, @WoNd3RBoY, @Nikhil-Kulkarni, @ppoliani, @nelly81

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
ghiliweld, @vaibhavmuchandi, @yebaihedechuntian, @timmhuang, @iovody11, @0xgeert, @Sori1401, @Isaiah258, @mj-vn, @danghe405kap, @Symars, @codemilli, @AmirValeev, @taramakage, @sunbuhui, @AAAZZZR, @Dlyoungml, @chimerakang, @cruzdanilo, @gonzalolater

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
LucasBitt7, @mdyu1000, @zbraiterman, @NoorahSmith, @betterjun, @0x0108, @mattgstevens, @tangivis, @Kathi-cloud, @devinaconley, @markosneiders, @PwnSecc, @kazuki-hanai, @yueyi2013, @yashwantx, @avishkarabhishek786, @willzhuang, @y77cao, @ngmisl, @Balou9


Fix factory girl dependency for Solidus < 2.5

We need to load a factory_bot version that has factory_girl in it
to support Solidus versions < 2.5
This change also includes conditional logic for the database
interface gems.

@fastjames I believe if we merge this, #54 will pass.

@jacobherrington do you know why this one is still red?

@kennyadsl there are failing tests addressed by #54.


newLine switch is removed on Windows platforms

This optimization was buggy and remove the switch on any platform
I've tried to set newLine in tsconfig and as grunt-ts setting and the flag was removed always.
As workaround I've to enable passThrough (why is not set by default?)
        tsconfig: {
          passThrough: true
        }

Hi - thanks for submitting this.  When you say it was buggy, can you elaborate?

I was trying to use LF in a Windows platform with Grunt linefeed set to '\n'. Result is newLine switch is not set.
It's clear that optimization does not work. Also I don't understand why grunt-ts should be responsible of "remove" that switch.

Understood.
We have a lot of stuff in the codebase for legacy compatibility, and grunt-ts does need to know the intended newline style to handle things like transforms.
I hear you saying that you've set grunt.util.linefeed to '\n', but it's not passing through on Windows.  I'd like to investigate that issue before just removing this call because it did solve a problem for a user when we first implemented it.
Thanks for the PR - I'll look into it ASAP.

I had set grunt.util.linefeed in Gruntfile and newLine in tsconfig.

Hi @Maks3w,
Thanks for bringing this to my attention.  I believe I may have fixed the issue.  The trouble was that we were using grunt.util.linefeed as the default linefeed for the OS, but of course it's reasonable that someone might change that for use with other Grunt tasks.  I've changed the function to query the OS directly as this is what TSC does.  I've also added tests.
Can you please check out<EMAIL_ADDRESS>and let me know if it resolves your issue?  If not, please open an issue and let's discuss further.  Thanks again for your contribution!  I've added a credit for you to the Changelog file.

I still don't understand why grunt-ts should check if the parameter is redundant. Also I guess the only value redundant is CRLF without matter in which platform is executed.

I'm operating under the assumption that on Mac/Linux, the default for TS is to use LF and on Windows it's CRLF.  Is that not your observation?  That's how it was when we implemented this.


Add Holographic Sky Recipe

This is my suggestion for a crafting recipe for the Holo Sky. I believe the materials properly represent it, without making it too expensive, just my opinion though.

Works for me.


Fix part upload retry (during multipart upload)

Closes #403.

@oneklc reports he verified the patch by running an extensive load test again.


Separate comment header display from comments

Currently the comments page doesn't even show the header until all comments load which doesn't make much sense.

This might end up needing a larger refactor of how we handle navigation. compose-destinations is looking like an attractive choice.

Progress on the compose-destinations migration is rough but it should still be a net positive once done.


Handle unsetting "optimistic" accountIDs from Onyx data after report creation

cc @aldo-expensify
Coming from https://github.com/Expensify/Auth/pull/10745#issuecomment-2091521262
Problem
We are almost done migrating off of participantAccountIDs and on to participants. However, there are several places where we do something like:

Create some kind of chat with an "optimistic" user (either no account or not yet known)
Frontend sets {[optimisticAccountID]: Participant} in the report.participants
Backend doesn't do anything about them

In the past, overwriting the array of participants would solve this problem since they were an array value.
Solution
Only the requesting caller of some API needs to have this unset so we could:

Return a {[optimisticAccountID]: null} it in the PHP layer like we do for Group Chats. This update is only applied by the API caller so it doesn't need to happen in Auth.

OR

Do it in the Frontend in the successData

Kinda leaning towards the latter as I can't see a benefit to having it in the BE vs FE.

We ended up doing this in the front end PR to refactor participants so this can be closed now.


In-App purchases for baker android?

Is this possible to do yet? What backend do you need? I have an iOS using magrocket, and it would be awesome if my android app could use it too.

@tobiasstrebitzer What do you guys use for the In-App purchases?

@fcontreras has anyone been able to see this?

@codegeek511 we have this as a "next to" plan to verify the integration of Baker AndroidR in-app to the Baker console.. maybe in couple of weeks we can have a more solid answer to your question.

We need this feature also. Do you have any updates when this will be see daylight?

We really need this. I think that I need to call some friends and ask if them help you and if they can do this. I only need information when this possible is coming, then I know if I still wait or I choose that second option.

We need this stuff too. Maybe you could give us the callback function, so that we implement it by ourselves?

@fcontreras Is there any update on the  In-App purchases ?
It as been over 10 months since you last said you would have a more solid answer, and nothing still.
Can we assume that this project is dead ?

@omegah3 not much progress... our team is on hold on this project. I wouldn't say is dead but on hold.


docs(k8s): remove storage account requirement

PR Checklist

[x] Leverages formatting as provided by https://squidfunk.github.io/mkdocs-material/reference/abbreviations/
[ ] Have viewed and verified edits in rendered form ensuring proper formatting

What are you changing?

Anything the reviewer should know when reviewing this PR?

Merging this.  Local K8s and AKS have been verified.  Currently troubleshooting EKS with I believe unrelated problem to persistent volume change.


Fix command alias printing in help text

Closes #405

Doh, thanks for catching that!

Would you be open to a knob for disabling this (for only printing .Name instead), or should I just plan to roll our own help template when we get this change pulled in?  Or perhaps a new HiddenNames/HiddenAliases field (analogous to hidden flags)?


deepfilter-ladspa,nixos/deepfilter,nixos/rnnoise: init

Description of changes
This adds the deepfilter-ladspa package, as well as nixos modules for using it and/or rnnoise with pipewire.
Closes #236646.
I've tested the modules on my own nixos system.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Build fails since the hash of what fetchFromGitHub downloads does not match the one defined in the derivation. Was there a rebase or something?

Build fails since the hash of what fetchFromGitHub downloads does not match the one defined in the derivation. Was there a rebase or something?

not sure, but i have fixed it

@K900 do you have any opinions on where the module should go?

Just programs.deepfilter-ladspa or something?

@ralismark could you relocate the module?

happy to move it. i'm sharing a bunch of code between the rnnoise and deepfilter modules, how should i structure that in nixos/modules/programs?

Hmm... Maybe programs.pipewire-filters.{rnnoise,deepfilter}? It's not like the modules are usable with pulseaudio or alsa...

Sorry to bother everyone, is there something blocking this from being merged? Would really love to use this module

Hello,
Wanted to bump this again since easyeffects has added support for deepfilter as of October, and adding support in nixpkgs would require this package.

I would prefer to have this outside the services.pipewire module, as these are not official Pipewire tools.

I think this is fine to have there? It's where the new options are most relevant. services.pipewire is for configuration of the PipeWire installation, and plugins are effectively a part of the installation. I don't think moving these options out of services.pipewire will make NixOS clearer or easier to configure. The option documentation should already make clear that a plugin is configured, and that's sufficient IMO.
It would make sense to separate out these options if they were useful outside of PipeWire.


No selector for Recovery Checks (Death Saves)

Unable to interact with Dying Recovery Checks using Rules Elements, no selector exists.

See diehard for an example of changing a recovery DC


Jest.fn should return new function

what is the current behavior?
When call jest.fn(implementation) it evryTime returns link to one instance in memory;
for example we want to test this code
function memoize(f){
   return (...args)=>{
         var result = memo[f];
          if (!result){
                 result = f(...args);
                 momo[f] = result;
          }
          return result;
   }
}

const memo = {};

in this case link of passed function is important;
if you will pass to this function any mocked function created by jest.fn - you will see that memo containse one function;
expected behavior?
jest.fn should return new function

jest.fn creates a new function every time. It seems like you made a mistake somewhere with your memoization function. Note that this is not a help forum; please ask on stackoverflow or on our discord channel. You can also create a repro here: https://repl.it/languages/jest and share that to stackoverflow or discord.

@cpojer  i'm sorry!
My mistake.


Implemented integer initialize with base

Functionality for integer with give values in different numeral bases like binary, octal has been implemented and tests have been written.

@zeerorg could you rebase/merge with the latest version of master? Also it seems you did not set your git settings correctly, as the commits do not seem to be tied to your github profile.
If you need pointers, let us know :-)

Hi there! It looks like this PR might be dead, so we're closing it for now. Feel free to re-open it if you'd like to continue, or think about directing your efforts to https://github.com/beeware/briefcase or https://github.com/beeware/toga. Both of these have more active development right now. 😄


Adds the Head of Greytide stamp

About the PR

Adds the Head of Greytide stamp, a cruel stamp made out of a screwdriver and a bar of soap (you cannot make/handcraft this item). Stamps documents with a "Head of Greytide" label. The stamp icon on the paper is a colored in grey version of the HoS stamp.
Added the stamp to the maints lockers lootpool with a 5% chance, on par with the magicarp plushie.
A few notes on this PR:

I did not want to make the stamp on paper look like it was drawn with a No. 2 pencil. Coding that is a can of worms that I would not like to merge unless it's done right (spoiler alert: I can't do it right the first time, so I won't!)
The stamp sprite is drawn by yours truly. If you were wondering. But you could probably already tell.

Why / Balance

Feature requested jokingly on the SS14 discord, I thought I would jokingly make it a reality. It requires no code and I was looking for an excuse to contribute something to SS14, so here it is.
"""discussion""" on the SS14 discord about the HoG stamp: https://discord.com/channels/310555209753690112/900473545040207942/1262352310793797744
Technical details

Adds respective entries in rubber_stamp.yml, stamps.rsi/meta.json, bureaucracy.rsi/meta.json.
Adds a RubberStampGreytide with prob 0.05 to maints lockers and it's variant in Lockers/misc.yml.
Media

https://github.com/user-attachments/assets/a97424be-f4d1-4f9a-a545-965e09b4e0be

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

No breaking changes.
Changelog

:cl:

add: Added the greytide stamp. This stamp can be rarely found in maints lockers.

Having this as maints loot is cool and all but I think it would be better to have this as a Passenger loadout item

Having this as maints loot is cool and all but I think it would be better to have this as a Passenger loadout item

Takes the hype away.

1k hours prestige loadout item.
 A fun idea for a tider stamp, though the sprite is a bit hard to read as that.

1k hours prestige loadout item.

i would rather it be 100 hours if that was the case

I think it's a better idea to make them just about as rare as insuls as maints loot, I can imagine people just going around stamping everything they see. It also adds novelty and prestige if there's only very few and you have to go hunting for them

(I think only people actually involved in bureaucracy should be stamping anything and have access to stamps too)
Also probably change the description to describe what it is accurately

(I think only people actually involved in bureaucracy should be stamping anything and have access to stamps too) Also probably change the description to describe what it is accurately

Mime, Clown, Warden, Det, Chaps. Lawyer. I think you forget that these jobs also have stamps..

I think this might cheapen the steal objective a bit.

It's a fun idea but should definitely be rare. It would be silly to have 15 HoGs running around. Way cooler if there's just one.

Hi all! Thanks for the comments.

Having this as maints loot is cool and all but I think it would be better to have this as a Passenger loadout item

The problem with this is that any passenger with a set amount of hours gets this item. It's something you cannot control. Having this only in maints lockers is something you can control.

 A fun idea for a tider stamp, though the sprite is a bit hard to read as that.

Yeah... I know it's a bit hard to read. Not the best sprite, but I have very little space to work with, stamps are a very small sprite.

I think it's a better idea to make them just about as rare as insuls as maints loot, I can imagine people just going around stamping everything they see. It also adds novelty and prestige if there's only very few and you have to go hunting for them

They ARE as rare as insuls maints loot. Insuls (ClothingHandsGlovesColorYellow) have a prob of 0.05 or 5%, on par with the stamp. I used this prob value because I only saw about one magicarp every few rounds, from searching a lot of lockers. Also, the modular receiver, a component in making an improvised gun, is actually a 10% chance, and I only see 1 of those items every few rounds.
If we want, we could reduce the rarity to just 1%. It's no problem for me. But I would like to see this item.
The chances are supposed to make it so 99% of the time there is just 1 in a round. Having it be a passenger prestige makes it so everyone has it. And when everyone's special, no one is.

(I think only people actually involved in bureaucracy should be stamping anything and have access to stamps too) Also probably change the description to describe what it is accurately

Mime, Clown, Warden, Det, Chaps. Lawyer. I think you forget that these jobs also have stamps..

All of those have clear uses for their stamps, even the mime and clown who can use it for fun circus gimmicks and stuff phony paperwork

its a neat idea, but the sprite looks inconsistent with the other stamps. looks kinda out of place imo.

I"m planning to resprite the stamp to make it more in-line and discernable with all other stamps. I'm also thinking about halving the chance for it to appear in lockers, but I might not, we'll see.

Should probably map it into the greytide churches that are on box and core as well but that's another prs problem

Made some new stamp sprites with consistency in mind. Here are two versions compared to Det's stamp.
Let me know which one you guys would like to see merged!

I'm probably going to make the stamp a little rarer and add some misc stuff.

New changes. Halved the chances from prob: 0.05 (5%) to prob: 0.025 (2.5%). Resprited to the one that looks more like a screwdriver, not a retextured bulky head. Added a custom description. Let me know if it's too OOC or anything needs to be changed.

Dear lord this is actually getting developed.

I'll redo le shit and change the sprite to one graciously provided by UbaserB (thanks!!!!!!)
as well as changing the stamp to just "greytide" not "head of greytide", I think this fits the best.

Are you coming back to adding those changes?

Are you coming back to adding those changes?

yep! I told myself "oh i'm going to work on it later" and then never worked on it :)
making my changes now

tfw force push

This PR is now ready to merge, I have no intentions of modifying it further (unless I am told to)
New changes:
Used the sprite provided by @UbaserB because I suck at spriting
Changed the description because it no longer fit, let me know if it needs changing

Thanks for contributing!

WAIT
WAIT HOLD ON

Please don't tell me I turbofucked git up and you merged my fuckup

Please don't tell me I turbofucked git up and you merged my fuckup

no i just tapped the wrong thing and im new so i have no clue how to fix this

GOOD JOB DEVI!

GOOD JOB DEVI!


[WIP] Uses the enrollments contained in the genesis block to test pre-images revelation

I previously made a PR to add the validator's enrollment to Genesis, and it's merged.
Since then, there have been occasional errors in the unit test.
I have identified and corrected the cause of the unit test error.
I hope this will be approved as soon as possible and our work will proceed smoothly.

It's not really the correct fix. It's just a workaround.
Because with this PR we no longer have networking tests for the enrollValidator API endpoint.
But if it's blocking work, we can temporarily accept it. My tests in #845 will include re-enrolling, so that endpoint will be tested again.

Since then, there have been occasional errors in the unit test.

Could you post a link to the failure ?

It's not really the correct fix. It's just a workaround.
Because with this PR we no longer have networking tests for the enrollValidator API endpoint.
But if it's blocking work, we can temporarily accept it. My tests in #845 will include re-enrolling, so that endpoint will be tested again.

Yes, that's a very good point.
All of the validators' enrollment data are included in the Genesis block.
We cannot change a non-validator to a validator in a unit test.
So I didn't call the enrollValidator.
Therefore, in order to register further, we will need to allow re-enrollment
This is a departure from my issue.
I'll think about a better way.

Since then, there have been occasional errors in the unit test.

Could you post a link to the failure ?

@Geod24 @AndrejMitrovic I found errors in the unit test and recorded them here.
https://github.com/bpfkorea/agora/pull/887/checks?check_run_id=718978489
https://github.com/bpfkorea/agora/pull/887/checks?check_run_id=718978503
https://travis-ci.com/github/bpfkorea/agora/jobs/340796300

Ok, but this means the test is causing the Node to enter into an unexpected state.

Full error: core.exception.AssertError@source/agora/consensus/protocol/Nominator.d(372): Assertion failure

This assertion should never happen. It means there is a bug. We shouldn't just hide the bug, but we should find the root cause of the issue and fix that.

@Geod24 @AndrejMitrovic The genesis block contains the enrollment data of all validators.
Additional enrollment is not allowed before maturity.
I created a new network topology. NetworkTopology.AllValidatorOneNotEnroll
All nodes are validators and only the last one node doesn't have enrollment data in the genesis block.

Note that we still need to provide them with a frozen UTXO, or build a test node that creates one frozen UTXO.

I changed it to the way you suggested.
Most unit tests required the enrollment of a validator.
There were some tricky problems.

Information on which node is the validator is unclear.
Transactions from genesis block are consumed, and for further testing, a test transaction must be prepared. These may cause confusion in other tests.
Do you mind if we continue this way?

What this PR was trying to resolve has already been resolved.
If you have any other opinions or issues, please proceed with new issues or related PR.

I'm working on what @Geod24 suggested.
I need to check if this is related to my current issue or if it is something to proceed with a new one.
In order to solve this problem, I think @Geod24 and @AndrejMitrovic need to agree.
Therefore, I think it would be better to proceed with this part if you create a new issue.


Prototype an API for using variables in routes

Think /v1/{user_id}/test
Keep variables stringly typed and add methods for lazy interpretation in the handler

Closed by #40


Getting error after Mac OS Big Sur update

After I updated my system from Catalina to Big Sur, I am trying to install rbenv with no luck.
When I am trying following:
rbenv install 3.0.0
I am getting following error:
Downloading ruby-3.0.0.tar.gz...
-> https://cache.ruby-lang.org/pub/ruby/3.0/ruby-3.0.0.tar.gz
error: failed to download ruby-3.0.0.tar.gz
BUILD FAILED (macOS 11.1 using ruby-build 20210119)

Upon executing following command:
curl -fsSL https://github.com/rbenv/rbenv-installer/raw/master/bin/rbenv-doctor | zsh

I am getting following error:
curl: (35) error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version

I never reproduce that.

I guess it was issue with the Mac OS Big Sur update. I was able to fix it. Thank you @hsbt


How to Add Extra Configuration to Kubelet Config?

Is there a way to define some extra configuration to the kubelet config used?
For instance, I would like to add the following:
allowedUnsafeSysctls:
  - "net.core.rmem_max"
  - "net.core.rmem_default"

I noticed there is a kubernetes_kubelet_extra_args variable, so I updated my kubernetes.tf as follows:
kubernetes_kubelet_extra_args = {
  "allowed-unsafe-sysctls" = "net.core.rmem_max,net.core.rmem_default"
}

I applied the changes with Terraform.
After applying the change, I restarted my Cloudflared deployments:
$ kubectl rollout restart deployment/cloudflared
Warning: would violate PodSecurity "restricted:latest": forbidden sysctls (net.core.rmem_max, net.core.rmem_default)

No changes seem applied.
How can I properly configure kubelet to allow these unsafe sysctls? Is there a step I’m missing or a different approach to take?

I'm not sure whether the change is effectively applied or not. At least your args have been added to the kubelet:
talosctl get kubeletspec -o yaml | grep -A 9 "args:"
    args:
        - --allowed-unsafe-sysctls=net.core.rmem_max,net.core.rmem_default
        - --bootstrap-kubeconfig=/etc/kubernetes/bootstrap-kubeconfig
        - --cert-dir=/var/lib/kubelet/pki
        - --cloud-provider=external
        - --config=/etc/kubernetes/kubelet.yaml
        - --hostname-override=k8sa-p-control-nue-hnr-1
        - --kubeconfig=/etc/kubernetes/kubeconfig-kubelet
        - --node-ip=<IP_ADDRESS>
        - --rotate-server-certificates=true

The setting is marked as deprecated and should be applied via the kubelet config:
talosctl logs kubelet | grep allowed-unsafe-sysctls
<IP_ADDRESS>: Flag --allowed-unsafe-sysctls has been deprecated, This parameter should be set via the config file specified by the Kubelet's --config flag. See https://kubernetes.io/docs/tasks/administer-cluster/kubelet-config-file/ for more information.

I added a new variable called kubernetes_kubelet_extra_config to add extra configs to the kubelet in 0.7.1
kubernetes_kubelet_extra_config = {
  allowedUnsafeSysctls = [
    "net.core.rmem_max",
    "net.core.rmem_default"
  ]
}

This adds your config to the kubelet as follows:
talosctl get kubeletspec -o yaml | grep -A 2 "allowedUnsafeSysctls"
        allowedUnsafeSysctls:
            - net.core.rmem_max
            - net.core.rmem_default

I'm still not sure if this solves the root cause of your problem tho.


chore (Int.Cast.Defs): delete last line

The last line is open Nat which came from an incomplete deletion of content from this fild. We delete the line.

maintainer merge


Datepicker can't display error/required messages

The majority of the components are able to display error/required messages via the import mapError from './map' statement, except for poor old DatePicker. It's never able to display an error message as the props are never mapped.

This is true. There is no mention of errorText in the DatePicker docs, so I did not implement it, but it does, in fact, work. This will be fixed in the next version.

Fixed in v4.0.0.


Make clear how to use this as a site owner

Not entirely clear here (examples would help, maybe?) what one would do as a site owner to make this happen. Put it into contribute.json at the root of the site when it's deployed (what about projects that aren't deployed)? Or into the repository root?

It'd be great to see this updated.  I'm adding contribute.json files to all of the relengapi-related repos (https://github.com/mozilla/build-relengapi/pull/226) but I'm not sure how to make them discoverable..

A file named contribute.json at the root of the repo. If it's a website and you want web-based tools to also find it then serve it up as /contribute.json as well.

@pmclanahan That lovely short sentence you just wrote is what this issue is all about. We should make a new page just for that.


[system] Add API section

How to use useTheme while not using the deprecated @mui/styles package?

@SteplerPaulo To answer your direct question, you can use:
import { useTheme } from '@mui/material/styles';
// or
import { useTheme } from '@mui/system';

Now, this surfaces a much larger issue. We miss documentation for the modules we export.

ThemeProvider
useTheme
etc.


preserve namespaces for classes/definitions when using swaggerToTypeScriptClient code generator

Is there any way to preserve the individual namespaces around each definition/class when using the nswag swaggerToTypeScriptClient code generator?
Here is a sample of the Swagger .json:
"definitions": {
    "WebService.SysOp.Models.AdServiceAccountModel": {
      "required": [
        "Login",
        "Password",
        "ServiceAccountId",
        "EnvironmentId"
      ],
      "type": "object",
      "properties": {
        "Login": {
          "type": "string"
        },
        "Password": {
          "type": "string"
        },
        "ServiceAccountId": {
          "format": "int32",
          "type": "integer"
        },
        "EnvironmentId": {
          "format": "int32",
          "type": "integer"
        }
      }
    },
   "Shadow.BusinessActions.Framework.ActionResponse[System.Net.HttpStatusCode,Shadow.DataLayer.AdServiceAccount]": {
      "type": "object",
      "properties": {
        "Data": {
          "$ref": "#/definitions/Shadow.DataLayer.AdServiceAccount",
          "readOnly": true
        },
        "Status": {
          "enum": [
            "OK",
          ],
          "type": "string",
          "readOnly": true
        },
        "Message": {
          "type": "string",
          "readOnly": true
        },
        "Messages": {
          "type": "array",
          "items": {
            "$ref": "#/definitions/Shadow.BusinessActions.Framework.Message"
          },
          "readOnly": true
        }
      }
    },
    "Shadow.DataLayer.AdServiceAccount": {
      "required": [
        "Id",
        "ServiceAccountId",
        "EnvironmentId",
        "Login",
        "Password"
      ],
      "type": "object",
      "properties": {
        "Id": {
          "format": "int32",
          "type": "integer"
        },
        "ServiceAccountId": {
          "format": "int32",
          "type": "integer"
        },
        "EnvironmentId": {
          "format": "int32",
          "type": "integer"
        },
        "Login": {
          "type": "string"
        },
        "Password": {
          "type": "string"
        },
        "ServiceAccount": {
          "$ref": "#/definitions/Shadow.DataLayer.ServiceAccount"
        },
        "Environment": {
          "$ref": "#/definitions/Shadow.DataLayer.Environment"
        }
      }
    }
}

I would like it to produce TypeScript like this:
namespace WebService.SysOp.Models {
	export interface AdServiceAccountModel {
		Login: string;
		Password: string;
		ServiceAccountId: number;
		EnvironmentId: number;
	}
}
namespace Shadow.DataLayer {
	export interface AdServiceAccount {
		Id: number;
		ServiceAccountId: number;
		EnvironmentId: number;
		Login: string;
		Password: string;
		ServiceAccount: Shadow.DataLayer.ServiceAccount;
		Environment: Shadow.DataLayer.Environment;
	}
}

Instead it produces TypeScript like this:
export interface AdServiceAccountModel {
    Login: string;
    Password: string;
    ServiceAccountId: number;
    EnvironmentId: number;
}

export interface AdServiceAccount {
    Data?: AdServiceAccount2;
    Status?: AdServiceAccountStatus;
    Message?: string;
    Messages?: Message[];
}

export interface AdServiceAccount2 {
    Id: number;
    ServiceAccountId: number;
    EnvironmentId: number;
    Login: string;
    Password: string;
    ServiceAccount?: ServiceAccount;
    Environment?: Environment;
}

This is not yet possible...
My long term plan is to generate output artifacts (class/interface/enum) instead of generating a single file... Each of these artifacts will have its own namespace and other meta information. Then these artifacts can be merged into a single file (same output as now) or multiple files with different namespaces and the correct imports. But this is a very big change in the code base. Without being paid I don't have time for this now - I currently focus on consolidating the existing features... (also there is the huge task of migrating all templates to DotLiquid for much better customizability)...

See https://github.com/NSwag/NSwag/wiki/Roadmap

At least, the generated classes/intefaces generated from the definitions could be embedded in the namespace specified in the settings right ?

Yes, isn't everything wrapped in the namespace?

Yes, sorry.
I was mistaken.
Thanks.


TextBlock render wrong text in some situations.

Describe the bug
TextBlock will render a wrong text in some situations. It's Text property is "AAA" and it displays a "BBB"。
Situations

Open the popup and close it by 'IsLightDismissEnabled'.
The result should be 0.

Analysis

The render of MyTextBlock is called between the text updating by 'MediaContext.Instance.SyncDisposeCompositionTarget(this.CompositionTarget);' of class CompositingRenderer.
And the render() will make TextBlock's TextLayout be recreated, with a temp text, which will be updated later.

To Reproduce
I have made a demo to demonstrate it.
Avalonia.TextBlock.Render.Demo_v11.1.0.zip
Expected behavior
The MyTextBlock should display a "0" after the popup is closed in the demo.
Avalonia version
11.1.0
OS
Windows
Additional context
No response

So, could the TextBlock skip the render calling if it's IsMeasureValid is false? Or just destroy the Textlayout if the IsMeasureValid is false after rendering?

You sample causes a InvalidOperationException Visual was invalidated during the render pass

Visual was invalidated during the render pass is definitely an bug in the app.
Changing visual tree (exactly what LogService.ShowLog does internally) is not allowed from inside of Render call. That's by design. @liwuqingxin, you can add log messages on dispatcher call.

But  It's Text property is "AAA" and it displays a "BBB"。 part needs to be clarified by author then.

Also, why do you early return from OnPropertyChanged method?
Base TextBox.OnPropertyChanged is never called for other properties because of that, which might cause unwanted behavior.

Your sample causes an InvalidOperationException Visual was invalidated during the render pass This issue isn't related to text rendering

@Gillibald @maxkatz6  Sorry, I can not see that exception. But I have changed the code and removed the LogService, which is used to clarify the progress. Now without the LogService, you need to see the result by DevTools.

Also, why do you early return from OnPropertyChanged method?
Base TextBox.OnPropertyChanged is never called for other properties because of that, which might cause unwanted behavior.

You are right. That's wrong. I need to filter some debugger breakings when debugging. I just forget to delete it. And, the whole MyTextBlock's logical is not necessary. It's for debugging. I have removed it.

@maxkatz6 I don't think that the #16481 can fix this problem.
In the process of this problem, There are 4 changes to TextBlock's Text. After the 2nd change, The rendering, which is fired by MediaContext.Instance.SyncDisposeCompositionTarget, comes and rebuild the TextLayout by accessing TextBlock.TextLayout property.
In the Layoutable.InvalidateMeasure(), if (!this.IsMeasureValid) return; will skip the calling of this.OnMeasureInvalidated();. So the TextLayout will not be rebuilt anymore.

@maxkatz6 @Gillibald  In the process of this problem, There are 4 changes to TextBlock's Text. After the 2nd change, The rendering, which is fired by MediaContext.Instance.SyncDisposeCompositionTarget, comes and rebuild the TextLayout by accessing TextBlock.TextLayout property.
In the Layoutable.InvalidateMeasure(), if (!this.IsMeasureValid) return; will skip the calling of this.OnMeasureInvalidated();. So the TextLayout will not be rebuilt anymore.
So with the pr #16481, can the rendering between the changes be prevented?

So with the pr https://github.com/AvaloniaUI/Avalonia/pull/16481, can the rendering between the changes be prevented?

TextBlock will be invalidated on each Text change, but rendering is scheduled independently after. So, it won't render between text changes, if they happened right after each other.


launchEditor URI-option

Release 7.2.0 added the launchEditor option, but it doesn't work for my webstorm based environment because webstorm is not registered as global application (as it is managed by the JetBrains toolbox).
Luckily, Jetbrains does provide a URI-scheme that can be used to open a file directly, this looks like the following for webstorm.
jetbrains://web-storm/navigate/reference?project=&path=
Would it be possible to support this?

We use vite-plugin-vue-inspector as our launchEditor feature backend, PRs are welcome.


Obtain user data: followers_count, statuses_count, name, screen_name, description

Is it possible to obtain users/lookup.json?user_id=xxxxxxxxxx info?

is this not enough? https://github.com/JohnnyTheTank/apiNG-plugin-codebird#requests-by-user

@aberbenni if you want, I could give you live support via gitter: https://gitter.im/JohnnyTheTank/apiNG
there, we can discuss what you need

close for now


Use SecurityTokenDescriptor when creating tokens

Use SecurityTokenDescriptor when creating tokens.

Let's make sure there is an issue so we can track and include in release notes easier.

Let's make sure there is an issue so we can track and include in release notes easier.

@keegan-caruso Updated.

"Suscribe"
Luis Armando Robledo salazar

From: msbw2 @.>
Sent: Saturday, November 9, 2024 1:45:49 PM
To: AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet @.>
Cc: Subscribed @.***>
Subject: Re: [AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet] Use SecurityTokenDescriptor when creating tokens (PR #2991)
Let's make sure there is an issue so we can track and include in release notes easier.
@keegan-carusohttps://github.com/keegan-caruso Updated.
—
Reply to this email directly, view it on GitHubhttps://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/2991#issuecomment-2466426438, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFLCBZ6NMEPFIJUH37WZCQDZ7ZQ63AVCNFSM6AAAAABROUMYFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWGQZDMNBTHA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Compiled into single JS, refactored Renderer into a class

tsc -p . now compiles everything from entrypoint play.ts down into a single JS file. The renderer was giving me issues, so refactored it into a class, and it's now passed in form the board down. The board defines a single TrainRenderer, and this is used when spawning trains. This in turn is then used to render out each carriage, as it is passed down. Theoretically, you could derive an interface from the renderer, allowing for multiple types of trains and carriages to be drawn, they just need to be passed in at spawn time.

I haven't tested this 100%. Just a warning :P

Works on my machine!


.bashrc should be used by default on Linux

Ubuntu 16.04 - bash
Hyperterm appears to be sourcing .bash_profile out of the box, but not the .bashrc found in the home directory. I'd say that .bashrc should be the first shell config to be sourced.

bash appears to be getting launched as a login shell.

--login should probably be an option that's off by default https://github.com/zeit/hyperterm/blob/master/session.js#L23

Removing that flag works for me :+1:

#572 solves it. You can add -i as shellargs on current master.

I think this can be closed, right?


[Suggestion] Command line support

could you make something like
WEMCompilerTool.exe -source "C:\Folder with spaces\file with spaces.mp3" -output "C:\Folder with spaces\file with spaces.wem"
or
WEMCompilerTool.exe -source file with spaces.mp3 -output file with spaces.wem

Seconding this.
Bethesda changed to WEM for Starfield. Now that the CK is out there's a need for a tools that can be called at the command line that can take WAV created from any of a number of Text to Speech engines and output the same file as WEM.


Erorr

Piggy auction does not open, after which the inventory does not work.

please read here
https://github.com/DaPigGuy/PiggyAuctions#invcrashfix-is-still-needed

пожалуйста, прочитайте здесь
https://github.com/DaPigGuy/PiggyAuctions#invcrashfix-is-по-прежнему необходим

I know he's standing by me! But sernovno does not work, maybe it does not support  1.18.30?


Its not working with SSR nextjs.

I have tried following the examples in the readme.
its not working for nextjs.

@andrew-oko-odion It works for me with Next.js. Of course this highly depends on where you put the code. If you are still interested, just let me know and I'm glad to help you!

I am interested, can you show a working code in nextjs?

Sure! Currently, I am doing it like this (using next-redux-wrapper):

makeStore() (called by next-redux-wrapper)
and an express middleware that extracts the state for SSR. I would prefer a solution that keeps everything in makeStore(), but I could'nt figure out a way for that yet because makeStore() is not async (getStoredState returns a promise unfortunately).
I am currently looking into this and will consider creating a little package to make this hassle-free within the next weeks if I find a nice all-in-one solution...

I will be look out for that package, please keep me posted
thanks!!!

@andrew-oko-odion I had a little hackathon and finished the package :smile: Will release it on Monday :+1:

@andrew-oko-odion Here it is! https://github.com/bjoluc/next-redux-cookie-wrapper
Unfortunately, I can only publish it to npm tomorrow because I used npm unpublish on a test release. So if you want to use it before then, clone it and run npm install, npm build, and npm link. Then in your project, run npm link next-redux-cookie-wrapper and you should be ready to go.
Cheers!

Thanks!
On Nov 26, 2019 10:38 PM, "bjoluc"<EMAIL_ADDRESS>wrote:

@andrew-oko-odion https://github.com/andrew-oko-odion Here it is!
https://github.com/bjoluc/next-redux-cookie-wrapper
Unfortunately, I can only publish it to npm tomorrow because I used npm
unpublish on a test release. So if you want to use it before then, clone
it and run npm install, npm build, and npm link. Then in your project,
run npm link next-redux-cookie-wrapper and you should be ready to go.
Cheers!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/abersager/redux-persist-cookie-storage/issues/17?email_source=notifications&email_token=ABACDMEL44V5HGALXNQGKKDQVWJOXA5CNFSM4GHPO7FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFHQVYA#issuecomment-558828256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABACDMFNPD33L2QSYUZDH5LQVWJOXANCNFSM4GHPO7FA
.

It's on npm now :tada: https://www.npmjs.com/package/next-redux-cookie-wrapper
Let the advertising begin :grinning:
You're welcome!

Fantastic, would check it out and let you know how it goes... Good work

Fantastic, would check it out and let you know how it goes... Good work

can you include an example project, most people have a slightly different configuration of their nextjs projects, so that one can see make comparison and adapt legacy project accordingly

Let's discuss this in the next-redux-cookie-wrapper issues :+1:


Validation failure on GPU

I have a stencil in my code which validates for any CPU backend, but fails on GPU, either using gtcuda or gtc:gt:gpu. The stencil is rather lengthy, but it can be stripped down to
@gtscript.stencil(backend="gtcuda")
def test_stencil(in_field: gtscript.Field[float], out_field: gtscript.Field[float]):
    with computation(PARALLEL), interval(...):
        out_field = (
            in_field[0, +2, 0] + in_field[0, -2, 0]
            - 4.0 * (in_field[0, +1, 0] + in_field[0, -1, 0])
            + 6.0 * in_field[0, 0, 0]
        )

If in_field is uniform, the stencil should compute zeros. But validation fails if in_field is filled with 0.08333333333333529 (catched through Hypothesis).

I get something of order 1e-17, do you expect an exact 0?

I forgot to mention that out_field is scaled by a certain factor in the original stencil. So 1e-17 is surely acceptable per se, but can lead to a substantial discrepancy if the factor is large. I'm not surprised by this - common scenario with floating points. But curious (or maybe not so unexpected?) that it only occurs on GPU. CPU backends produce exactly zero.

Sorry, forget about this. I erroneously thought that this issue could affect my simulations too, but it doesn't actually. My bad. I will increase the tolerance in my tests to make them pass on GPU. I'm going to close if nobody objects.

Sure, go ahead and close it.


sky_tool needs to be smart about --install just like shelldb is

@collinjackson fixed shelldb to know how to autoinstall when needed.  We need to do the same to sky_tool if we're going to switch to it.

This is fixed.


Blank lines option

Is possible to have an option to add blank line rule(s) in the formatter?
Eg transform this
class Temp {
    @Dec1
    prop1! int;
    @Dec2
    prop2! string;
}

to this:
class Temp {
    @Dec1
    prop1! int;

    @Dec2
    prop2! string;
}

or this:
class Temp {

    @Dec1
    prop1! int;

    @Dec2
    prop2! string;

}

Relevant:
https://eslint.org/docs/rules/lines-between-class-members
https://eslint.org/docs/rules/padded-blocks
Might be of some help.


rulegen: support outdent and dedent tagged template expressions

Currently, we filter out any test cases which use the outdent and dedent functions, which are sometimes used for unicorn and typescriot-eslint rule tests. This means that we are likely missing many test cases from our rule testing. We should implement our own version of dedent and outdent so that we can import these test cases as well:
https://github.com/oxc-project/oxc/blob/92812349e5266b074d2cb050cc7a6f200e4ac914/tasks/rulegen/src/main.rs#L319

I would like to work on this issue. Can I try this?

It seems that there are still some places in rulegen that do not handle String.raw and dedent, outdent in Expression::TaggedTemplateExpression. I am thinking whether to deal with it in this issue, I would be grateful if you could give me some advice or guidance.
https://github.com/oxc-project/oxc/blob/4348eae1577bd72094d1ec1f6c2422c20288a3d9/tasks/rulegen/src/main.rs#L271
https://github.com/oxc-project/oxc/blob/4348eae1577bd72094d1ec1f6c2422c20288a3d9/tasks/rulegen/src/main.rs#L229

It seems that there are still some places in rulegen that do not handle String.raw and dedent, outdent in Expression::TaggedTemplateExpression. I am thinking whether to deal with it in this issue, I would be grateful if you could give me some advice or guidance.

I think we should try to handle outdent/dedent wherever it shows up, but I don't think we need to handle it all in one PR. It would probably help to break it up into a few smaller

https://github.com/oxc-project/oxc/blob/4348eae1577bd72094d1ec1f6c2422c20288a3d9/tasks/rulegen/src/main.rs#L271

This does seem like a case we should handle, outdent in output seems to be a case that is often used: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/0d91fe8530611f68a95261e2a8f5f80b12caee35/test/prefer-ternary.mjs#L107-L111. We don't support String.raw here yet, but that would be a good idea too, since there are examples of that: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/0d91fe8530611f68a95261e2a8f5f80b12caee35/test/string-content.mjs#L116

https://github.com/oxc-project/oxc/blob/4348eae1577bd72094d1ec1f6c2422c20288a3d9/tasks/rulegen/src/main.rs#L229

We already handle String.raw here, but I think we should handle outdent here as well, since it's also common to see outdent inside of a code property: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/0d91fe8530611f68a95261e2a8f5f80b12caee35/test/prefer-ternary.mjs#L98-L106

@camchenry Thank you very much for your reply. Considering the reusability of code, I have already handled String.raw and dedent, outdent in Expression::TaggedTemplateExpression. I plan to handle noFormat (in typescript-eslint) in the future, and I will PR again at that time. If possible, If possible, could you please help me review #6866? Thanks in advance for your suggestions on my code.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:41: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
ERROR: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:41: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_7bp3b3yv0p/clone/recipe/meta.yaml:41: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Adding a new profile or clicking the gear icon in "other profiles" causes Thorium AVX2 to crash

System Details

OS: Windows 11 23H2 build 22631.3155
Thorium Version 121.0.6167.204

Problem
Attempting to create a new profile or clicking the gear icon to manage profiles causes Thorium AVX2 to crash.
Screenshots
N/A
Additional Notes
Attempting to reinstall Thorium from scratch does not fix the problem, as well as deleting the whole User Data folder. This is a bug with the application.

Edit: Your repos are very confusing, I'm opening the issue in the right repo.

@SheMelody Fixed in latest M122 version


Update codicons

Add map and map-filled https://github.com/microsoft/vscode-codicons/commit/33bed29edb3afb2eef6a9aa56fb2d68238956267
Remap debug-stackframe-dot to circle-small-filled
Add circle-small https://github.com/microsoft/vscode-codicons/commit/2b98cb89ecb7b66cf6d41063aba4e246ee68e5be

cc @Tyriar @connor4312 @daviddossett

circle-small should be circle-small-outline for consistent naming?

@Tyriar I'll be updating all of our names and using the outline version as the default and then adding filled for the solid ones, since the outline style is our default. Tracking under https://github.com/microsoft/vscode-codicons/issues/117


v1.1.11

This repository contains GitHub workflows for managing releases automatically.
Please name this issue what you would like the new release to be called. E.x. "v1.0.0"
The following slash commands are available to use in comments on this issue:

Command
Description
Parameters

/release
Creates a new released based on the current state of the master branch
frontend=<version>backend=<version>gitapi=<version>status=<version>dispatcher=<version>agnosticv=<version>anarchy=<version>poolboy=<version>

/promote
Promotes the created release to the next environment
N/A

/cancel
Cancels the current release (and closes this issue)
N/A

/finish
Completes the current release (and closes this issue)
N/A

/release frontend=v1.1.5 backend=v1.0.10 gitapi=v1.0.8 status=v1.0.0 dispatcher=v1.0.12

/promote

/cancel


Remove un-needed target when upgrading to v2 BuildTools

We have done some change in the PR #3580 to change the license Url for packaging. because corert repo not using v2 of the buildtools we had to hack this by adding the temporary target OverrideLicenseUrl in the dir.targets files. This target should be removed after upgrading to v2 of the buildtools and need to ensure when removing it the license Url in the packages nuspec files is correct

@tarekgh FYI, the build tools have been updated to 2.0.0-prerelease-01702-02.


merge(E2ETF): Merge E2ETF branch into DTFS2

Introduction :pencil2:
This PR tracks the merge of E2ETF into DTFS2

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

Comparison is base (376dd67) 22.57% compared to head (601587c) 22.70%.

Files
Patch %
Lines

...ervice/put-utilisation-report-status.controller.ts
0.00%
38 Missing and 3 partials :warning:

...rt-service/upload-utilisation-report.controller.js
0.00%
31 Missing and 6 partials :warning:

dtfs-central-api/src/drivers/db-client.ts
37.50%
11 Missing and 4 partials :warning:

...port-service/get-utilisation-reports.controller.js
0.00%
12 Missing and 2 partials :warning:

...al-api/src/v1/routes/utilisation-reports-routes.js
0.00%
14 Missing :warning:

...eport-service/get-utilisation-report.controller.js
0.00%
11 Missing and 1 partial :warning:

...rs/tfm/amendments/tfm-get-amendments.controller.js
0.00%
10 Missing :warning:

...pi/src/v1/controllers/bank/get-banks.controller.js
0.00%
8 Missing :warning:

dtfs-central-api/src/v1/routes/bank-routes.js
0.00%
8 Missing :warning:

...c/v1/controllers/tfm/users/tfm-users.controller.js
0.00%
7 Missing :warning:

... and 53 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2595      +/-   ##
==========================================
+ Coverage   22.57%   22.70%   +0.12%
==========================================
  Files        1136     1156      +20
  Lines       27255    26623     -632
  Branches     4486     4297     -189
==========================================
- Hits         6153     6044     -109
+ Misses      18388    17986     -402
+ Partials     2714     2593     -121

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
64.60% <ø> (-1.07%)
:arrow_down:

unit-test
10.28% <51.71%> (+5.64%)
:arrow_up:

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

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


Need a feature to duplicate values under a new key

Hi,
We are using etcd to hold application configurations and sometimes we need to create new keys by modifying existing values.
So we would like to have a feature to duplicate existing values under a new key.
If the community agrees with this feature. We would happy to do this as a contribution.
Let me know your thoughts on this.
Thanks.

Welcome to submit new features, just hope that the UI interface maintains a simple and consistent style.

Hi @evildecay
We need an extra feature to copy a whole directory. the idea is to keep configurations as v1, v2 etc. So multiple applications can use different versions of config directories and we can keep the old config directories untouched.
Ex:
/root/myapp/v1/conf1
/root/myapp/v2/conf1
Please let us know your thoughts on this. We would be happy to take the development if you are OK with this.
Thanks.

Hi @srimaln91
I understand that the etcd address is configured. In fact, this is right. I personally don't have time to do these things. If it can be realized, I welcome contributions.


No search field on mobile navbar

None of the mobile responsive navbars include a search field as is present on the desktop versions.

Thanks for reporting this as issue. We actually have done it intentionally to make mobile experience cleaner. But, we will reconsider this during next update. Thanks 🙏


Cannot load file containing pickled data when allow_pickle=False

Sorry mate, been really busy lately. I've re-uploaded the data as csv, so hopefully that fixes this.


Add spread strategy and make it the default

Signed-off-by: Victor Vieux<EMAIL_ADDRESS>

@jhamrick & @aluzzardi  can you please take a look ?

:+1: thanks, this looks awesome!

@vieux is spread the replacement of the balanced proposed a while ago?
if so, great to see it's made as the default one !

:+1:. considering that this scheduling strategy has been proposed at least 3 times now it sounds like users want this feature :)

LGTM


IndexOutOfBoundsException in CommandHandler.channelRead

Bug Report
When used along with spring-boot default cache type, along with concurrent load throws
Redis exception; nested exception is io.lettuce.core.RedisException: java.lang.IndexOutOfBoundsException: writerIndex(2147483642) + minWritableBytes(1842) exceeds maxCapacity(2147483647): PooledUnsafeDirectByteBuf(ridx:<PHONE_NUMBER>, widx:<PHONE_NUMBER>, cap:<PHONE_NUMBER>)"
Current Behavior
With bit of debugging reliazed that that is a buffer created in CommandHandler object which lives till connection stays? and keeps increasing buffer size when there is a response from (in redis. io.lettuce.core.protocol.CommandHandler#channelRead ). Also monitored stack size, that stays with in 200.
Dumping writer index :
writerIndex:8387482
writerIndex:16777126
writerIndex:20971118
writerIndex:25165378
writerIndex:29356774
writerIndex:33554002
writerIndex:37712882
writerIndex:41942218
writerIndex:46136326
writerIndex:50330434
writerIndex:54525898
writerIndex:58720006
...
writerIndex:264241152
...

Let me know if you need any more info, or if you can provide some pointers to resolve/fix.

Stack trace
java.lang.IndexOutOfBoundsException: writerIndex(2147483644) + minWritableBytes(1842) exceeds maxCapacity(2147483647): PooledUnsafeDirectByteBuf(ridx:<PHONE_NUMBER>, widx:<PHONE_NUMBER>, cap:<PHONE_NUMBER>)
	at io.netty.buffer.AbstractByteBuf.ensureWritable0(AbstractByteBuf.java:276)
	at io.netty.buffer.AbstractByteBuf.ensureWritable(AbstractByteBuf.java:265)
	at io.netty.buffer.AbstractByteBuf.writeBytes(AbstractByteBuf.java:1079)
	at io.netty.buffer.AbstractByteBuf.writeBytes(AbstractByteBuf.java:1072)
	at io.netty.buffer.AbstractByteBuf.writeBytes(AbstractByteBuf.java:1062)
	at io.lettuce.core.protocol.CommandHandler.channelRead(CommandHandler.java:506)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
	at io.netty.channel.ChannelInboundHandlerAdapter.channelRead(ChannelInboundHandlerAdapter.java:86)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
	at io.netty.channel.ChannelInboundHandlerAdapter.channelRead(ChannelInboundHandlerAdapter.java:86)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1434)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:965)
	at io.netty.channel.epoll.AbstractEpollStreamChannel$EpollStreamUnsafe.epollInReady(AbstractEpollStreamChannel.java:808)
	at io.netty.channel.epoll.EpollEventLoop.processReady(EpollEventLoop.java:417)
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:317)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:884)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.lang.Thread.run(Thread.java:748)

Expected behavior/code

Environment

Lettuce version(s): 5.0.3.RELEASE, 5.1.7.RELEASE
Redis version: 5.0.3, 5.0.5

Possible Solution

Additional context

CommandHandler keeps an aggregation buffer that is used to aggregate multiple response chunks when reading from the transport channel. The buffer gets consolidated after decoding commands:
https://github.com/lettuce-io/lettuce-core/blob/5.1.x/src/main/java/io/lettuce/core/protocol/CommandHandler.java#L614-L616
Observing the output, you work with large chunks of data (8 MB increments). What worries me is that the stack size remains constant as this can be an indicator that commands are not decoded properly. Without decoding commands, the aggregation buffer keeps growing.
Can you provide a reproducer?

I will try to get a simple reproducer, however data size is not huge,  its generated json couple of hundreds of KB. However there is normalize allocation happening in buffer which rounds to nearest power of 2 or so. it takes many requests to go for next allocation.
Also is there possibility that it doesnt get decoded if command timed out. I saw await being used

I think I understand now why it never goes to buffer.discardReadBytes();
input buffer can have partial payload for bulk / byte response, for example, my payload is 1845 bytes so it reads long and gets count = 1845, however buffer has only 1200 bytes then decode simply returns false. next time when it tries to decode, another chunk would be added to buffer that also in half. so this continues until there are no calls to redis for a while.
Possible solution could be circular buffer? or  check for clean up when it goes for resize?

We've introduced in 5.2.0 a new method to discard read bytes also if the decode was unsuccessful. Care to upgrade to 5.2.0 snapshots? See #906.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


KOGITO-3058: Improve DEBUG option

JIRA: https://issues.redhat.com/browse/KOGITO-3058
Description: Debug log of test run is not appearing
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

I've tested these changes and now the debug traces from the steps are appearing. But when running in debug mode, the output is not in JSON, I can easily change this but it would imply changes in the operator framework as well and maybe this was done without json on purpose. What do you think @sutaakar ?

@sutaakar another question, the JIRA says that the command output should be displayed too: "+ set command execution in debug mode as well", but I think it was already working fine (the only problem was that the debug traces were hidden). Am I missing something here?

Debug output in plaintext is ok for me.

About command output - it seems to be implemented already, I think we can consider it as implemented.