Add a "verifierAction" reference to a Connector

This issue is a reminder, that we have to deal with the mismatch where the verification happens in the iPaaS and the Camel world:

In iPaaS the verification of a ConfiguredConnector (aka connection) is done by the expert developer when he enters the (technical) connection information. It's before a citizen developer can select an Action.
For Camel verification happens on the  camel-connectors ("twitter-mention"). This is an iPaaS Action which is not yet selected when the verification is supposed to happen.

One solution without touching Camel is to mark a specific Action as the verifierAction which is referenced by a Connector and use in the verification process. This of course works only if all Actions belonging to the same Connector have the same verification logic.
Alternatively, instead of reusing an existing Action, one could introduce a dedicated VerificationAction which has no business meaning except the verification of the connection parameters.
Are there other, better, solutions ?

Another idea could be to group all similar Camel connectors into single jar instead of having one jar per connector. The manifests could be added with a dir structure like
META-INF/
  services/
     org.camel.Verifier     <-- this is a "standard" Java ServiceLoader class used for the verification
  camel-connectors/
     http-get.json
     http-post.json

and then called http-connectors.jar. Might be a bit far fetched and might not fit into the Camel model (sorry, when I'm ignorant here, have too less of a clue ;-) but for iPaaS verification this would be awesome (since we have a single verifier for a group of similar connectors), and would also reduce a bit the granularity (i'm a bit afraid of Maven artefact and module 'explosion')
Just an idea ....

I do like the idea of putting all related connectors into a single jar.  Keep in mind we will need to eventually implement a UI which creates connectors and part of that is going to be defining actions, etc.  For all connectors I can think about, they are all based on the same camel component, it's just mostly extra meta data you need to add for each action that it supports.
And for projects like Salesforce where you need to generate a model, it makes sense to share that model across all the actions.  Yeah we could do multi module projects to handle those cases, but that seems much more complex than a single jar for a connector.


Ability to translate the column names and search input

Fixed in https://github.com/SpoonX/aurelia-datatable/pull/16


Fix SliverGrid garbage collection issue

The code doesn't consider indices of children may not be continuous after rebuild.
fixes https://github.com/flutter/flutter/issues/138749
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Would you mind reverting some of the dartfmt changes that snuck into this PR?

Would you mind reverting some of the dartfmt changes that snuck into this PR?

done.


All in this category K-LiteCodecPack is outdated

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
There is a newer version of CodecGuide.K-LiteCodecPack The Iobit Uninstaller will tell you the version.

This app is to check everything up to date. The clues to the new URL is: If there is not up to date, it must be in winget use that address replace to where support the older number and change it to a newer number which is the new URL.
Currently, <IP_ADDRESS> is unavailable because it could be waiting for future releases.  Also, the release note will be updated by the time.
Steps to reproduce
When winget searches mkv use it to support the MKV Video Player.
Actual behavior
When I use winget to search MKV, suddenly a bunch of K-Lite Codec is outdated. Because the Iobit Uninstaller tries to check the website on the server. It suddenly found another URL that can be the file name changed. If you are up to date, you might see 404 Unavailable. That means the <IP_ADDRESS> version is not yet released. The release note should be https://codecguide.com/changelogs_full.htm for the clues to the new URL.
Expected behavior
Install and Upgrade should work as expected
Environment
Windows Package Manager v1.4.10173
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.2673
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.19.10173.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

User Settings: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Screenshots and Logs

If I'm understanding correctly, K-LiteCodecPack version <IP_ADDRESS> has been released but our manifest's highest version is <IP_ADDRESS>. The solution would be to add an updated manifest with the new package version.

Yes. Right now, there is greater than <IP_ADDRESS> It is <IP_ADDRESS>

Close with reason: 17.5.0 and 17.5.2 versions have been added to the repo;


MSBuild throws 'Cannot modify an evaluated object'

Steps to reproduce
This happens when running dotnet-format combined with msbuild-sdk-extras.
For example, run it on https://github.com/wildgums/orc.controls
Expected  behavior
The project should be loaded successfully (it can build, load, etc, but when running dotnet format, it fails.
Actual behavior
The project should be loaded successfully (it can build, load, etc, but when running dotnet format, it fails.
I've been investigating the source code, and found that it's correctly splitting multi-targeted builds, but then needs to re-check for the target version which it doesn't.

Environment data
msbuild /version output:
OS info:
windows 10 (1903)
If applicable, version of the tool that invokes MSBuild (Visual Studio, dotnet CLI, etc):
dotnet-format 3.1.0-dev

So it's trying to remove a property that's been added while re-evaluating the project. Then the code should check for the value again (it might have been set by the re-evaluation).

And I think the fix is to re-check for the property existence just before removing the property.

Created ticket at Roslyn instead of MSBuild.


pytest: add "--skip-flash" option

The pytest is always trying to flash when running on real HW (--device-type=hardware).
On ARM64 Cortex-A platform like RPI5 there no way to implement flashing through pytest framework now: the Zephyr binary has to be stored on SD-card and loaded by bootloader (especially if no HW debugger available).
This PR adds "--skip-flash" option which still will allow to run pytest with shell interface manually, after platform is booted and shell is ready.
The related Discord discussion is [1].
With this feature the below test sequence become possible with ARM64 (in my case RPI5 platform):

boot RPI5 with Zephyr application which have SHELL interface enabled;
close tty, for example "/dev/ttyACM0"
enable access to tty "sudo chmod a+rw dev/ttyACM0" (if needed)
Setup Zephyr environment
go to Zephyr application folder
run pytest

Run pytest example:
cd <Zephyr proj>
source zephyr-env.sh
export PYTHONPATH=${ZEPHYR_BASE}/scripts/pylib/pytest-twister-harness/src:${PYTHONPATH}

cd <Zephyr application>
pytest --twister-harness --device-type=hardware \
    --device-serial=/dev/ttyACM0 \
    --build-dir=../build/rpi_5/zephyr-dom0-xt \
    -p twister_harness.plugin --log-level=DEBUG --log-cli-level=DEBUG

Note. pytest may not always detect SHELL cmd line, so just run
echo aa > /dev/ttyACM0
from another terminal while pytest is running and waiting for cmd line.
The Zephyr application which uses this feature can be found in [2] and pytest tests commit [3].
[1] https://discord.com/channels/720317445772017664/1233080559866089493/1239579773613899817
[2] https://github.com/xen-troops/zephyr-dom0-xt/tree/rpi5_dom0_dev
[3] https://github.com/xen-troops/zephyr-dom0-xt/commit/9a98afb17b6a23eed23f59403f2d6d64e2c9e68e

Thank you very much for your comments.

IMO this is to narrow case. It is not integrated with Twister. Pytest-harness was added to extend Twister testing, to run some scenarios automatically, without manual actions. Here you want to skip Twister, or build only, then prepare the board and run pytest directly, I don't know it it is good way.

It's only one way I've found now unfortunately to test at least smth. Sorry, not an expert in CI/twister, so could be missing smth.

If you have such a case it is better just to overwrite dut fixture in your test dir, mock flash method or rewrite this fixture to not call any flash method. You can add your own fixture in conftest.py or alongside your test scenarios. Here is an example to overwrite dut fixture with mocked hardware flashing:

Thank you for your solution with  conftest.py - I've tried it and it's working fine: i can run pytest.
I've updated PR with your comments fixed. I'd be appreciated if you advice please if it make sense to continue with this PR or should I close it?


HTTP 404 on nbextension installed as per readme

After following install instructions in readme (using pip -e /path/to/bgplot), I get HTTP 404 on attempts to create inline plots in jupyter-1.0.0:
Successfully installed bqplot-0.3.5
(test-bqplot) myuser /software/test-bqplot  % python -m bqplot.install --symlink --user --force
symlink /home/myuser/.local/share/jupyter/nbextensions/bqplot -> /software/bqplot/bqplot/nbextension

The symlink exists and points to the correct location:
% ls -al ~/.local/share/jupyter/nbextensions/bqplot
lrwxrwxrwx 1 myuser myuser 35 Sep 29 14:39 /home/myuser/.local/share/jupyter/nbextensions/bqplot -> /software/bqplot/bqplot/nbextension

Error output is:
[W 15:03:40.123 NotebookApp] 404 GET /nbextensions/bqplot/components/d3/d3.js?v=20150929150214 (::1) 2.72ms referer=http://localhost:8888/notebooks/test1.ipynb

Installed versions are:
% pip freeze
backports.ssl-match-hostname==<IP_ADDRESS>
-e git+https://github.com/bloomberg/bqplot.git@fe21b42c5a2c8f520bd4a95e2dd39637cb3c1e8c#egg=bqplot-master
certifi==2<IP_ADDRESS>
decorator==4.0.4
funcsigs==0.4
functools32==3.2.3.post2
ipykernel==4.0.3
ipython==4.0.0
ipython-genutils==0.1.0
ipywidgets==4.0.3
Jinja2==2.8
jsonschema==2.5.1
jupyter==1.0.0
jupyter-client==4.0.0
jupyter-console==4.0.2
jupyter-core==4.0.6
MarkupSafe==0.23
matplotlib==1.4.3
mistune==0.7.1
mock==1.3.0
nbconvert==4.0.0
nbformat==4.0.0
nose==1.3.7
notebook==4.0.5
numpy==1.9.3
pandas==0.16.2
path.py==8.1.1
pbr==1.8.0
pexpect==3.3
pickleshare==0.5
ptyprocess==0.5
Pygments==2.0.2
pyparsing==2.0.3
python-dateutil==2.4.2
pytz==2015.6
pyzmq==14.7.0
qtconsole==4.0.1
simplegeneric==0.8.1
six==1.9.0
terminado==0.5
tornado==4.2.1
traitlets==4.0.0
wheel==0.26.0

Browser is Google Chrome 47.0.2516.0 dev (64-bit)

@jfkw thank you for reporting.
To run bqplot from source, you also need to run bower install from the main bqplot directory in the terminal to install the JavaScript dependencies. I have updated the instructions in PR #12.

Thanks, confirmed this step fixes figure display. The visuals look great, thanks.


Could not retrieve user token. Please run 'az login' Azure CLI 2.33.1

Hi,
I have a CI pipeline which contains an Azure Cli Task to submit a python script. This is the inline script the task uses:
az logout

az login --service-principal --username $servicePrincipalId --password $servicePrincipalKey --tenant $tenantId

az ml run submit-script -g myRG-w myWS -e ExpName --ct clusterName -d conda_dependencies.yml -t metadata/run.json ml_service/pipelines/build_pipeline_score.py

It results in the error:
{'code': ExecutionFailed, 'message': [{"exit_code":1,"error_message":"Execution failed with error: [stderr]  File \"/azureml-envs/azureml_d8ac72e9b124a7fe0100c9a94b37cf8b/lib/python3.8/site-packages/azureml/core/authentication.py\", line 899, in _get_arm_token\n[stderr]    return _get_arm_token_with_refresh(\n[stderr]  File \"/azureml-envs/azureml_d8ac72e9b124a7fe0100c9a94b37cf8b/lib/python3.8/site-packages/azureml/core/authentication.py\", line 426, in connection_aborted_wrapper\n[stderr]    return function(*args, **kwargs)\n[stderr]  File \"/azureml-envs/azureml_d8ac72e9b124a7fe0100c9a94b37cf8b/lib/python3.8/site-packages/azureml/core/authentication.py\", line 2293, in _get_arm_token_with_refresh\n[stderr]    raise AuthenticationException(\"Could not retrieve user token. Please run 'az login'\", inner_exception=e)\n[stderr]azureml.exceptions._azureml_exception.AuthenticationException: AuthenticationException:\n[stderr]\tMessage: Could not retrieve user token. Please run 'az login'\n[stderr]\tInnerException Please run 'az login' to setup account.\n[stderr]\tErrorResponse \n[stderr]{\n[stderr]    \"error\": {\n[stderr]        \"code\": \"UserError\",\n[stderr]        \"inner_error\": {\n[stderr]            \"code\": \"Authentication\"\n[stderr]        },\n[stderr]        \"message\": \"Could not retrieve user token. Please run 'az login'\"\n[stderr]    }\n[stderr]}\n[stderr]\n","process_name":"/azureml-envs/azureml_d8ac72e9b124a7fe0100c9a94b37cf8b/bin/python","error_file":"user_logs/std_log.txt"}], 'target': , 'category': UserError, 'error_details': [{'key': exit_codes, 'value': 1}, ], 'inner_error': null}
I have tried downgrading the azure cli version to 2.29.2 (as stated in some old issues).
Here is the cli task configuration

route to CXP team

@obiii The Azure CLI task should already be logged in using the service connection defined in the task. You don't really have to login in code like so.
Either way, is the az login command running without any errors? You could run az account show to double check.


Added the import statement to the usage example.

Short description of what this resolves:

Problems in typescript where Storage is unknown or where you are directed to the wrong Storage interface by your IDE

Changes proposed in this pull request:

Added the import statement in the usage example

Ionic Version: 1.x / 2.x
Fixes: #
It is very useful for users to know where Storage comes from especially since e.g. WebStorm will autocomplete the search for Storage to a different interface.

Sorry but that path is not the recommended way for users to import ionic-angular.


Add services.fossil

Add a fossil server service to NixOS
Project description
Fossil is a VCS which can run as a standalone http server to publish repositories
Metadata

homepage URL: https://fossil-scm.org
source URL: https://fossil-scm.org
license: bsd
platforms: unix, linux, darwin

Fossil is a bit more than a VCS, as it supports other features useful for project management as well,

In addition to doing distributed version control like Git and Mercurial, Fossil also supports bug tracking, wiki, forum, email alerts, chat, and technotes. https://fossil-scm.org/home/doc/trunk/www/index.wiki

It's got a web UI and can be hosted on a $5/month VP.
So there is more value to supporting fossil in nixpkgs.

@srid And the best part is that the wiki, the bug database, the source code history, the forum etc. fit all nicely in a single SQLite database! Also, fossil all push is so great.
I refreshed the PR a bit, let's get it revieved

I also would like to see this happen and am willing to help.

It would be great if this were added! :D

services.fossil.enable = true;
would be great :)

I also got stumped by lacking fossil service, so I ended up throwing together initial version of such module.
I won't have time/will to get it into nixpkgs, so feel free to copy or get it upstreamed.
https://github.com/cipharius/systems/blob/262dfc835119654a7f0a1b1ff84d45ee5bfc5a9e/presets/server/fossil/fossil.nix
Personally I'm using it in SCGI mode hidden behind nginx.


Support folders dropped onto the droparea

Possibly entire compendiums and rolltables as well?

Folders and RollTables added to v10 release.


[BUG] Quantization of Qwen return garbage

OS
Linux
GPU Library
CUDA 12.x
Python version
3.10
Pytorch version
2.4.0
Model
No response
Describe the bug
I quantize my own qwen 7B model and the return token is always 60021. Here is config file of my qwen model
{
  "_name_or_path": "models/qwen2-7B",
  "architectures": [
    "Qwen2ForCausalLM"
  ],
  "attention_dropout": 0.0,
  "bos_token_id": 151643,
  "eos_token_id": 151645,
  "hidden_act": "silu",
  "hidden_size": 3584,
  "initializer_range": 0.02,
  "intermediate_size": 18944,
  "max_position_embeddings": 131072,
  "max_window_layers": 28,
  "model_type": "qwen2",
  "num_attention_heads": 28,
  "num_hidden_layers": 28,
  "num_key_value_heads": 4,
  "rms_norm_eps": 1e-06,
  "rope_theta": 1000000.0,
  "sliding_window": 131072,
  "tie_word_embeddings": false,
  "torch_dtype": "bfloat16",
  "transformers_version": "4.41.2",
  "use_cache": true,
  "use_sliding_window": false,
  "vocab_size": 152064
}

Reproduction steps
Here is my parameter to quantize
python convert.py -i models/myQwen-7B-HF -o models/myQwen-7B-EXL2/ -b 8 -hb 8 -l 16384 -ml 16384

Expected behavior
The generation after quantization is working.
Logs
No response
Additional context
No response
Acknowledgements

[X] I have looked for similar issues before submitting this one.
[X] I understand that the developers have lives and my issue will be answered when possible.
[X] I understand the developers of this program are human, and I will ask my questions politely.

Additional Note: After I tried to quantize qwen model made by the developer Qwen/Qwen2-7B-Instruct, the result is also garbage. So, the problem is not from my model.

I've been unable to reproduce this with v0.2.1 and the original Qwen2 model. It quantizes and inferences correctly here.
Are you using Q4 cache by any chance? Qwen2-7B specifically fails with Q4 cache, but Q6 and Q8 should be fine:

Model
Quant
Cache
pass@1
pass@10
Wikitext 5x1k

Qwen2-7B
FP16
Q4
19.74%
46.34%
40.72

Qwen2-7B
FP16
Q6
61.65%
81.70%
15.20

Qwen2-7B
FP16
Q8
62.37%
81.09%
15.18

Qwen2-7B
FP16
FP16
61.16%
82.31%
15.16

Could you test it using my exllamav2_hf here at #600 ? Because I kept getting garbage answers inferencing using that. But, when inferencing another model, it works just fine.

I looked into it and managed to reproduce the problem with the HF wrapper code.
It seems the issue is with attention. Since you're supplying a mask, that disables both the Flash Attention and SDPA code paths as they only support causal attention. ExLlama falls back on matmul attention, which works in half precision, which isn't an issue for most models, but for Qwen2-7B specifically you get occasional overflows on some layers and then inference breaks. I think it's to do with weird normalization of the keys/queries, related to why it doesn't like the Q4 cache mode.
Regardless, SDPA can use an arbitrary mask, Torch just won't use efficient kernels internally, but it should avoid the overflows anyway. I've enabled that in the latest commit on the dev branch and it seems to be working with your wrapper.

wait, do you mean that if I set the input_mask parameter, flash attention wouldn't be used? Then how to generate a batch of texts without input_mask?

Yes, flash-attn doesn't support input masks. And if you want to supply a rectangular input IDs tensor where the rows aren't the same length, the only way that can happen is with masking. Otherwise you'd have to start with the shortest input, generate at a batch size of 1 until it reaches the same length as the 2nd shortest input, then at bsz 2, etc. That way the input is always rectangular and you won't have to mask out any padding, but it's very inefficient.
flash-attn does have a "varlen" mode, but it's not efficient either since it requires the cache to be contiguous, so you have to constantly rebuild it (copy the whole thing in VRAM) to make space for new keys/values for every token generated.
The alternative is to use paged attention with a flat cache. This however is only compatible with flash-attn.
The SDPA approach at least allows for Torch to switch to a more efficient backend at some later time if flash-attn ever supports masking. There has been some work on this with https://github.com/Dao-AILab/flash-attention/pull/617, but it's not finished yet, apparently.

I've been unable to reproduce this with v0.2.1 and the original Qwen2 model. It quantizes and inferences correctly here.
Are you using Q4 cache by any chance? Qwen2-7B specifically fails with Q4 cache, but Q6 and Q8 should be fine:
Model 	Quant 	Cache 	pass@1 	pass@10 	Wikitext 5x1k
Qwen2-7B 	FP16 	Q4 	19.74% 	46.34% 	40.72
Qwen2-7B 	FP16 	Q6 	61.65% 	81.70% 	15.20
Qwen2-7B 	FP16 	Q8 	62.37% 	81.09% 	15.18
Qwen2-7B 	FP16 	FP16 	61.16% 	82.31% 	15.16

@turboderp, would you have similar metrics for other models?

I've been unable to reproduce this with v0.2.1 and the original Qwen2 model. It quantizes and inferences correctly here.
Are you using Q4 cache by any chance? Qwen2-7B specifically fails with Q4 cache, but Q6 and Q8 should be fine:
Model 	Quant 	Cache 	pass@1 	pass@10 	Wikitext 5x1k
Qwen2-7B 	FP16 	Q4 	19.74% 	46.34% 	40.72
Qwen2-7B 	FP16 	Q6 	61.65% 	81.70% 	15.20
Qwen2-7B 	FP16 	Q8 	62.37% 	81.09% 	15.18
Qwen2-7B 	FP16 	FP16 	61.16% 	82.31% 	15.16

@turboderp, would you have similar metrics for other models?

https://github.com/turboderp/exllamav2/blob/master/doc/qcache_eval.md

@Thireus I tested Command-R here:
https://old.reddit.com/r/LocalLLaMA/comments/1f6ijye/commandr_35b_q4q6q8_cache_perplexity_mmlu/
And this is a model with extremely  "compressed" attention where 110K of Q4 context only takes like 4GB. I think Qwen2 was just a crazy extreme, and the new Qwen 2.5 doesn't behave like that.


[FLINK-21745][tests] Sets the parallelism in tests to comply to the AdaptiveScheduler requirements

What is the purpose of the change
FLINK-21135 removed the code for setting the parallelism of each JobVertex to 1 if not properly set, yet. This caused IllegalStateExceptions in different tests having flink.tests.enable-adaptive-scheduler enabled where the parallelism was not set.
Brief change log

introduces a Precondition in AdaptiveScheduler checking the parallelism of each JobVertex
sets the parallelism for vertices in failing tests if the test failure was caused by the newly introduced IllegalStateException

Verifying this change

I ran affected tests and fixed the related test failures

Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Have you run this PR on Ci with the adaptive scheduler profile activated, to make sure the profile is passing now?

From this Ci run, it seems that there are still some instabilities: https://dev.azure.com/mapohl/flink/_build/results?buildId=356&view=results

This build failure is covered by FLINK-21954.

This build failure is covered by [FLINK-21963|https://issues.apache.org/jira/browse/FLINK-21963)

This build failure is covered by FLINK-21964.

This build failure is covered by FLINK-21965

I addressed all issues that appeared in the builds. It's ready to get a final review.


[HUDI-7041] Optimize the mem usage of partitionToFileGroupsMap during the cleaning

Change Logs
Mainly to solve the OOM when generating clean plan if there are too many partitions.

Clean allows grouping execution by setting cleanerParallelism.
After generating cleaning information for a partition, clean the metadata of that partition in TableFileSystemView.
Optimize the use of read and write locks in AbstractTableFileSystemView.

Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level (write none, low medium or high below)
If medium or high, explain what verification was done to mitigate the risks.
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Related issue for another cleaner OOM improvement: https://github.com/apache/hudi/pull/10116

@zhuanshenbsj1 I would recommened to separate the fs view locking from this PR. Let this PR only have the optimization for cleaner. Cleaner changes look good. You can still keep the cleanFileGroupsInPartitions method in fs view. But, raise another PR for fs view locking with details about the problem.

Nice advice~

Thanks for the contribution @zhuanshenbsj1 , I'm not sure why the read/write lock changes are introduced, can we just implement another interface getAllFileGroupsStatelessly(String partition) on fs view similar with getAllFileGroups(String partiton), in this method it does 2 steps to load a file group.

if the partition is already in the file group, just return it;
if it is not, load the file group on the fly, but we do not cache it in the file group mapping of the fs view.

The clean can just use this method for cleaning.

Thanks for the contribution @zhuanshenbsj1 , I'm not sure why the read/write lock changes are introduced, can we just implement another interface getAllFileGroupsStatelessly(String partition) on fs view similar with getAllFileGroups(String partiton), in this method it does 2 steps to load a file group.

if the partition is already in the file group, just return it;
if it is not, load the file group on the fly, but we do not cache it in the file group mapping of the fs view.

The clean can just use this method for cleaning.

Before getAllFileGroups is called, the partition metadata has been loaded (through getReplacedFileGroupsBefore | | getAllReplacedFileGroups) in fsView,  which is reused here.  If modified according to this approach, it will result in duplicate loading of metadata and very poor performance.

Thanks for the contribution, I have reviewed and created a patch:
7041.patch.zip

Rebase & add ut, cc @danny0405.

@hudi-bot run azure

Hi， Could you please merge the 0.14.1 （0.x） branch to support it? When will version 0.14.1 be released?
@zhuanshenbsj1 @danny0405

Hi， Could you please merge the 0.14.1 （0.x） branch to support it? When will version 0.14.1 be released?
@zhuanshenbsj1 @danny0405

Yes, it will merge to 0.14.1 and will be released soon.


What should I pay attention to if the backend cluster node are distributed in different area?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[X] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
What should I pay attention to if the backend cluster node are distributed in different area or city?
Is there any potential risk or unstabitily?

Bug

Which version of SkyWalking, OS and JRE?
8.1.0

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

We don't recommend that I think, unless. you have a great network across the cities. For me, multiple zones support usually is a part of a commercial product, but AFAIK, there is no one or company did that.


[ci skip] [cf admin skip] NO_CI adding bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I've added bot automerge as instructed in #44.
Merge this PR to enable bot automerging.
Fixes #44

@conda-forge/freeimage The repo has an history of successful migration that can be merged without problem (such as https://github.com/conda-forge/freeimage-feedstock/pull/42 or https://github.com/conda-forge/freeimage-feedstock/pull/41). I think it would be convenient to have the bot automerge enabled on this repo.

Makes sense. Thanks! 🙏


Machine readable warnings/errors

Eventually, this module might need to be used from "something else". However, the error and warning output is meant for human eyes on a command-line interface.
It might be useful adding an argument, something like --machine-readable, which causes the output to be in an easily parseable format.

Look into isatty()


Could you give me some guidance on how I can adapt this to vllm like llava?

Hello,
I’m currently exploring how to visualize the heatmap on LLAVA or other kinds of multimodal large language model to understand the model’s focus during text generation. I am familiar with using Grad-CAM for single-target classification tasks. However, with LLAVA generating complete sentences, I’m unsure how to obtain heatmaps for individual words. Could you provide any guidance or advice on how to approach this?

do you have answers? Have same issue.

do you have answers? Have same issue.


Built-in validation types

Hi there,
Is it possible to use antd built-in validation types with TextField component?

I think you have to make it in sync with redux form so you probably can't use them


[vite-plugin-components] has naming conflicts with other components, ignored.

src
--pages
   |- index.vue
--components
----role
      |- List.vue
----user
      |- List.vue

import components/user/List.vue  in  pages/index.vue
import components/role/List.vue  in  components/user/List.vue
npm run dev

then error goes here:

Also, as components/role/List.vue   is ignored, so it will not be rendered,
if file name is naming conflicts,  a good way to solve is do like this:
import RoleList from "src/components/role/List.vue";

add folder name as compoent name.
!important
after reading source code, I learned that parse role/List.vue as  '<Role-List/>`'。Maybe this should be reminded in the documentation
repo is here : https://github.com/fyeeme/vite-plugin-components-demo

And what about dynamic imports aka import('src/components/role/List.vue') and <component :is="component" />?

Sorry @fyeeme but didnt get that solution for this. Can you explain how did you solve it?

@pedropcruz  config is supported for solving this,default is false, you can have a try.
{
// Allow subdirectories as namespace prefix for components.
directoryAsNamespace: false,
}

@pedropcruz config is supported for solving this,default is false, you can have a try.
{ // Allow subdirectories as namespace prefix for components. directoryAsNamespace: false, }

i found even added directoryAsNamespace: true, below still warning:
[unplugin-vue-components] component "BarChart"(/overview/components/BarChart.vue) has naming conflicts with other components, ignored.

i have another component in /dashboard/components/BarChart.vue


Group edit refactor

This PR needs some cleaning and an update on the tests. It's basically a draft but since this is a big chunk of code I opened in for a first review.
@johngian a few things that are not ready yet:

Form errors in case that the data are not correct
Add by default a curator to a new group
Correctly add a new group

Tested it locally, and there are many things that need improvement. Basic functionality looks OK. Let's tackle the failing tests in an other PR.

Although this is not polished, I am merging for demo purposes.


added dom-events-delegation

Hi
dom-events-delegation is a JavaScript (delegation) event micro library (more info here http://brunoscopelliti.github.io/dom-events/).
I would  appreciate if you'll consider to add it to the micro.js collection.
Thank you.

Could you point the source url at your unminified code please?

Hi @sbrl ,
thank you for coming back to me...
i've just pushed the editing you suggested...
let me know if is there anything else i can do to help.
thank you

@brunoscopelliti Thanks! Could you squash your commits into a single commit too please?

Hi @sbrl
This is something I've never done before. (Really learning Git is on my todo list for too long.)
I searched on google... and I found I've to do a rebase.
I read about rebase here, and several other blog post.
So I executed the commands:
git rebase -i HEAD~2

I edited the commits history as follows:
p fa92d51 added dom-events-delegation
f bafb404 point the source url at the unminified code

Save, quit.
Brunos-iMac:microjs.com brunoscopelliti$ git rebase -i HEAD~2
Successfully rebased and updated refs/heads/master.

The result is that history didn't change at all, but the new file is created.
If I try:
git push origin -f

I get the "Everything up-to-date" message.
What have i missed?
Any help will be appreciated.

You want to change the f to squash, and then once the rebase is is complete, perform a git push origin --force. If that doesn't work, I'll take a look in my branch instead.

I've already tried with squash (or the less verbose s).
I get the same result...

@brunoscopelliti That should have worked. I think git is not working correcting - I'll take a look in my branch when I add it to my next commit bundle.
Thanks for trying though!

You're welcome.

Closed because changes were requested and no action was taken. Please feel welcome to create a new PR after resolving merge conflicts! You should place your added entry in data.js randomly and not at the very bottom, since this will help to prevent you from encountering another merge conflict.
Sorry this has been such a drawn-out process for you...


Fix a set of breaking BBE

Purpose
This PR fixes a set of breaking examples in BBE
Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #16995   +/-   ##
=======================================
  Coverage   68.85%   68.85%
=======================================
  Files           5        5
  Lines         122      122
  Branches       21       21
=======================================
  Hits           84       84
  Misses         38       38

Continue to review full report at Codecov.

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

@irshadnilam shall we resolve the conflicts?


MultiResponse with output queue leads to: Exception: System.ObjectDisposedException: IFeatureCollection has been disposed. Object name: 'Collection'

Description
Description
I'm experiencing issues with my Azure Functions while using the MultiResponse on HttpTriggers.
Once a single request it's working, once several requests are coming in, it leads to the following error:
'Exception while executing function: Functions.IqIngressFunction Result: Failure
Exception: System.ObjectDisposedException: IFeatureCollection has been disposed.
Object name: 'Collection'.'
Expected behavior
It should not throw that error
Actual behavior
It's working correctly locally on my machine, the issue appears only when it's deployed on Azure.
When I remove ASP.NET Core Integration and returning only the messages (string[]) in output it's working
When I remove the output queue it's also working (returning only IActionResult)
Only when we mix Httpresponse management with ASP.NET Core Integration + output queue it's crashing on the cloud only, not locally
Relevant source code snippets
public class MultiResponse
{
    [QueueOutput("%SmsIngressQueueName%", Connection = Constants.StorageAccount.ConnectionString)]
    public string[] Messages { get; set; }

    [HttpResult]
    public IActionResult HttpResult { get; set; }
}

public class IqIngressFunction
{
    private readonly IIqSmsService _iqSmsService;
    private readonly ILogger<IqIngressFunction> _logger;

    public IqIngressFunction(IIqSmsService iqSmsService, ILogger<IqIngressFunction> logger)
    {
        _iqSmsService = iqSmsService;
        _logger = logger;
    }

    [Function(nameof(IqIngressFunction))]
    public async Task<MultiResponse> Run([HttpTrigger(AuthorizationLevel.Anonymous, "post", Route = null)] HttpRequest req)
    {
        var output = new List<string>() { };
        var multiResponse = new MultiResponse();

        try
        {
            string requestBody = string.Empty;
            using (StreamReader streamReader = new StreamReader(req.Body))
            {
                requestBody = await streamReader.ReadToEndAsync();
            }

            _logger.LogInformation($"HTTP trigger function processed a request for IqIngressFunction, body: {requestBody}");

            var inbound = JsonConvert.DeserializeObject<IqInboundModel>(requestBody);
            if (inbound != null && inbound.deliveryReceipt)
            {
                multiResponse.Messages = new string[] { };
                multiResponse.HttpResult = new OkObjectResult("ok");
                return multiResponse;
            }
            var validMsg = _iqSmsService.ValidateIngressMessage(inbound);
            if (!string.IsNullOrEmpty(validMsg))
            {
                throw new Exception(validMsg);
            }

            var msg = new SmsMessage
            {
                Sender = inbound.from,
                Receiver = inbound.to[0],
                Message = inbound.text,
                Smscid = "IQ-x"
            };
            output.Add(JsonConvert.SerializeObject(msg));

            multiResponse.Messages = output.ToArray();
            multiResponse.HttpResult = new OkObjectResult("ok");
            return multiResponse;
        }
        catch (Exception ex)
        {
            _logger.LogError(ex, $"IqIngressFunction - An unexpected error occurred. Details: {ex.Message}");

            multiResponse.Messages = new string[] { };
            multiResponse.HttpResult = new BadRequestObjectResult(ex.Message);
            return multiResponse;
        }
    }
}

Known workarounds
Using a queue client (SDK required works)
App Details
<ItemGroup>
	<PackageReference Include="Azure.Identity" Version="1.12.0" />
  <PackageReference Include="Apache.NMS.ActiveMQ.NetStd" Version="1.8.0" />
  <PackageReference Include="Azure.Storage.Queues" Version="12.19.1" />
  <PackageReference Include="Microsoft.Azure.Cosmos" Version="3.42.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Extensions" Version="1.1.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore" Version="1.3.2" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Storage.Queues" Version="5.5.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.17.4" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.23.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Warmup" Version="4.0.2" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.EventGrid" Version="3.4.2" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.EventHubs" Version="6.3.6" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.OpenApi" Version="1.5.1" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Storage" Version="6.6.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http" Version="3.2.0" />
	<PackageReference Include="Microsoft.Azure.Functions.Worker.ApplicationInsights" Version="1.4.0" />
	<PackageReference Include="Microsoft.Azure.AppConfiguration.Functions.Worker" Version="7.3.0" />
	<PackageReference Include="Microsoft.ApplicationInsights.WorkerService" Version="2.22.0" />
	<PackageReference Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" Version="7.3.0" />
  <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
  <PackageReference Include="Twilio" Version="5.79.0" />

.NET 8 Isolated mode
Azure function with HttpTrigger
Azure Functions runtime version v4:
**C#*:

Screenshots

If deployed to Azure
see screenshot below

Timeframe issue observed::Happening right after the deployment happened, NOT HAPPENING ON LOCAL MACHIN

Steps to reproduce
Using MultiResponse as follows:
public class MultiResponse
{
    [QueueOutput("%SmsIngressQueueName%", Connection = Constants.StorageAccount.ConnectionString)]
    public string[] Messages { get; set; }

    [HttpResult]
    public IActionResult HttpResult { get; set; }
}

Then create an HttpClient and run 10 calls. 50 % of them will fail (InternlServerError 500), while others will work and return HttpStatus 200 OK

@satvu let's sync. This shouldn't be using the HttpResponseData path.

@AnthonyGiretti We have been unable to repro this issue with IActionResult - have you still seen this issue recently? Could you share a recent invocation and the app privately through these steps? Lastly, is there any chance the deployment payload was stale?
This is a known issue in MultiResponse with HttpResponseData under the new HTTP model, but the code path in the stack trace should not be hit when IActionResult is used.

@satvu  Hi, the issue is still happening. But only when deployed in the cloud. It's never happening on my local machine. The issue stopped when we rollbacked to .NET6 in process. I can't right away send what you are asking, it will take me some days.
We have removed the MultiResponse and replaced the queue attribute by the usage  of the azure storage queue sdk instead.
No issue when the HttpTrigger only returns IActionResult, it's only happening when it's coupled to the QueueOutput attribute in a MultiResponse. I suggest you to deploy on it on Azure, make a load test, sending 15 httprequest in a loop, the first call will work and all other will fail.
I'll get back to you in a few days

@satvu Can the following help?  I have been to AI to find the errors:
Region: WestUS 2

Operation Id | ddb04eac3c6193e217f504e8d5ac381d
Event time | 10/16/2024, 12:34:12.2047185 PM (Local time) --> Canada Eastern time
prop__EndTime | 2024-10-16T16:34:12.2040000Z
prop__StartTime | 2024-10-16T16:34:12.1640000Z
InvocationId | 91a5575b-3db2-4ea7-83eb-1133656c9d86

@AnthonyGiretti Thanks, we will take a look.

After extensive investigation, we were unable to repro this with a simple multi-output scenario which included a queue output and IActionResult. Load tests for it were unable to produce the same error, but we got a matching error when using HttpResponseData only. Internal tests that use multi-output have not reported this problem with IActionResult either (#2682 was discovered internally).
Closing this and tracking the fix for this error in #2682. I

@satvu Thanks for your researches. What your researches gave with the log info I provided? the goal was to investigate the internal issue from the logs, I was not expecting to hear that could not reproduce the issue.


Require bg-atlasapi and bg_space in setup.py

See https://github.com/brainglobe/napari-brainreg/issues/5

See https://github.com/brainglobe/napari-brainreg/issues/5


Changed fxns in OrderLibrary to be internal.

Pull Request Test Coverage Report for Build 1183

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

Totals

Change from base Build 1180:
0.0%

Covered Lines:
333

Relevant Lines:
333

💛  - Coveralls


Added LakeBTC exchange

Please remove README file changes from this Pull Request.

Thank you for your contribution!


throws : The super constructor to "inherits" must have a prototype

We have been trying to configure winstond with following simple conf :
 var server = winstond.nssocket.createServer({
  services: ['collect', 'query', 'stream'],
  port: 9003
});

But it throws an exception :
TypeError: The super constructor to "inherits" must have a prototype
at Object.inherits (util.js:973:11)
at Object. (/Users/shalinlk/Projects/test/log-server/dist/node_modules/winstond/lib/winstond/server.js:29:8)
at Module._compile (module.js:643:30)
at Object.Module._extensions..js (module.js:654:10)
at Module.load (module.js:556:32)
at tryModuleLoad (module.js:499:12)
at Function.Module._load (module.js:491:3)
at Module.require (module.js:587:17)
at require (internal/module.js:11:18)
at Object.Server (/Users/shalinlk/Projects/test/log-server/dist/node_modules/winstond/lib/winstond.js:8:10)
Also tried with http  :
var server = winstond.http.createServer({
  services: ['collect', 'query', 'stream'],
  port: 9003
});

returns the same error.

Been having this same issue as well with Winstond...

Same error here :(


Update sbt-tpolecat to 0.2.3

Updates io.github.davidgregory084:sbt-tpolecat from 0.2.2 to 0.2.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Codecov Report

Merging #284 (a1a4aec) into develop (36d9810) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #284   +/-   ##
========================================
  Coverage    50.58%   50.58%
========================================
  Files            5        5
  Lines           85       85
========================================
  Hits            43       43
  Misses          42       42

Continue to review full report at Codecov.

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


#1216 AddOcelot overload to support merging of configuration files to memory

Closes #1216
Implementation of feature suggested in issue:

#1216

Hi. I just realized that I've not linked this PR correctly to issue #1216, but it seems that I don't have permissions to fix it?

@ebjornset
Hi Eirik!
Thanks for the great PR!
Unfortunately It is hard to resolve merge conflicts using GitHub UI.
Could you resolve merge conflicts please?
You need to merge develop branch to feature branch with rebasing. See below! 👇
And I don't see develop branch in your fork!
Could you Sync fork please? So, new develop branch will occur with all top commits!
Let me know upgrading results please!

Eirik,
I thought you need some help in merge conflicts resolving.
There are no conflicts now.
The feature branch has been rebased onto ThreeMammals:develop.
And we can start code review...

@pablo0219
Could you review the code please?
Your opinion is prioritized because of past discussions.

@ebjornset Hi Eirik!
I've rebased the feature branch. In current release the AddOcelot extension methods were changed. Also we've refactored acceptance tests and especially Steps class.
I tried to merge develop and rebase the feature branch in the best logical manner. But now some unit tests are failing.
Could you look into these failed tests please?

Ready for delivery! ✅

Code review ✔✔
Team approvals ✔
Unit testing ✔
Acceptance testing ✔
Reviewed and updated docs ✔✔


gvim (vim-gtk3) has significant scroll-lag in Debian-11 Dolphin

Debian package maintainer<EMAIL_ADDRESS>said that this problem is an "upstream" issue, and to contact the gvim developers.
Describe the bug
In Dolphin, gvim scroll-up/down lags significantly and is choppy.
In Konsole, gvim scroll-up/down has no lag and is smooth.
It appears that the gvim configuration given to Dolphin needs an adjustment to match the performance of the Konsole configuration.
To Reproduce
rm ~/.vimrc                                           #remove all gvim configuration settings (clean tests)

Open Dolphin                                          #Example: Dolphin gvim has scroll lag
right-click on: /usr/share/vim/vim82/doc/version8.txt #open large file
select:         Open with->GVim                       #open large file
maximize gvim window                                  #maximum gvim widow size
Hold CTL-e for 10 seconds                             #scroll-down for 10 seconds
Release CTL-e                                         #>>>> SCROLL-DOWN CONTINUES FOR ANOTHER 5 SECONDS AFTER RELEASE

Open Konsole                                          #Example: Konsole gvim has no scroll lag
gvim /usr/share/vim/vim82/doc/version8.txt            #open large file
maximize gvim window                                  #maximum gvim widow size
Hold CTL-e for 10 seconds                             #scroll-down for 10 seconds
Release CTL-e                                         #>>>> SCROLL-DOWN STOPS IMMEDIATELY AFTER RELEASE

Expected behavior
No scroll-lag after releasing CTL-e.
Environment (please complete the following information):
Debian-11 (Bullseye), KDE
Konsole Version 20.12.3
Dolphin Version 20.12.2
vim-gtk3 (gvim) package version 2:8.2.2434-3
vim version 8.2
Edit:
Additional information:
Mistake: For Konsole, I stated: "SCROLL-DOWN STOPS IMMEDIATELY AFTER RELEASE".
Correction: For Konsole, should state: "SCROLL-DOWN STOPS 1-1.5 SECONDS AFTER RELEASE".
Online research indicates that this gvim scroll-lag issue is due to a difference in gtk2(fast) vs. gtk3(slow):
https://groups.google.com/g/vim_dev/c/1V8EC5lFcf0/m/VdHuTo_UDwAJ
https://wiki.pine64.org/index.php/Pinebook_Pro
This scroll-lag problem is NOT present in gvim (vim-gtk2) (gtk2 is available in Debian 10, but not available in Debian 11).
This scroll-lag problem is present in gvim (vim-gtk3).
Unfortunately, new Debian-11 dropped gvim (vim-gtk2); forcing the use gvim (vim-gtk3).
Speed check of nvim-qt (gvim equivalent) in Debian-11 showed that nvim-qt does not have this scroll-lag problem.
gvim is based on gtk3.
nvim-qt is based on Qt5 GUI.
So this scroll lag problem appears to be associated with gtk3.

With Dolphin do you mean the file manager?
Does that really matter, or is it just the difference between gtk2 and gtk3?

With Dolphin do you mean the file manager?
Yes, Dolphin is the file manager.
Does that really matter, or is it just the difference between gtk2 and gtk3?
The main problem is the difference between gtk2 and gtk3.  (Using Dolphin just makes the problem worse.)
The problem is not present in gvim (vim-gtk2); but is present in gvim (vim-gtk3) (I tested both using Debian 10).
Unfortunately, Debian-11 dropped gvim (vim-gtk2); forcing the use gvim (vim-gtk3).

As you stated when initially emailing me, you found https://github.com/vim/vim/issues/1199 which reports the same issue.  Not sure why a new issue needed to be created.

Because that issus was created in 2016, and still hadn't been addressed.

Even though that issue is old, it contains relevant information.  Therefore closing this one.
You might want to ask the GTK developers about this.  Creating a new issue doesn't help.


macOS | library that static links openssl still loaded in process memory after calling dlclose

Hello,
I have the following use case:

two dynamic libraries named libwaapi.dylib and libwautils.dylib

libwaapi.dylib has implicit-dynamic linking to libwautils.dylib

libwautils.dylib has static linking to openssl (libcrypto.a and libssl.a)

Below is the complete workflow of the issue:

void* v4_handle = (void*)dlopen( "libwaapi.dylib", RTLD_NOW )
At this time, the OS loads libwautils.dylib and libwaapi.dylib in the process memory.
function exposed by libwaapi.dylib is called, which uses openssl functions from libwautils.dylib
dlclose(v4_handle) is called.
After this call, the OS unloads only libwaapi.dylib from the process memory.
libwautils.dylib is still loaded in the process memory.

**We found a temporary workaround for this due to high priority of the issue ** but this is not sustainable in the long-term.
void * handle = (void*)dlopen( "libwautils.dylib", RTLD_NOLOAD );
if( handle != nullptr )
{
dlclose( handle );
dlclose( handle );
}
We are getting the handle to libwautils.dylib which is still loaded in the process memory.
Then we are calling dlclose twice, the first one for the dlopen operation we used with RTLD_NOLOAD flag, and the second one for the handle that existed in the process memory after Step 3 from the workflow.
Additional info about the issue:

The issue is reproducible with openssl 1_1_0x and 1_1_1x.

With openssl 1_0_2x, after Step 3 is performed, both libwaapi and libwautils libraries are successfully unloaded from the process memory.

We noticed this issue when we upgraded from 1.0.2x to 1.1.1x.

We tested on macOS Sierra 10.12 and 10.12.6. The issue is reproducible on both.

We performed a lot of additional tests and all the findings are taking us to openssl

Please let me know if you need additional information.
Thank you in advance for your help!

Compile OpenSSL with the "no-pinshared" option (available from OpenSSL 1.1.1b and above).
See the documentation here:
https://github.com/openssl/openssl/blob/5ded1ca6dade13e093086573f4cebe400bff9f4d/INSTALL#L446-L462

Thank you @mattcaswell , it is working!


We need to install both istioctl and asmctl to deploy Kubeflow on GKE

In the Install the required tools section, from the Deploy using kubectl and kpt documentation, we need to install both binaries (istioctl and asmctl) from the Preparing to install Anthos Service Mesh instructions, not only istioctl. It would be great to add this information to the documentation.

Thank you @kelvins. Yeah, Anthos Service Mesh is a Google Cloud flavor of Istio Mesh (AFAIK) and part of this larger Anthos platform.
And, as you mentioned, the official GCP instructions state that users have to download istio-X.X.X-asm...tar.gz and extract it, which "returns" istioctl and asmctl.

Assigning team Google Cloud from docs/gke:
/assign @joeliedtke @rmgogogo @Bobgy

Should we just add a few words - "and asmctl" - to Step 7:
- 1.  Follow the instructions from [Preparing to install Anthos Service
-     Mesh](https://cloud.google.com/service-mesh/docs/archive/1.4/docs/- gke-install-new-cluster#preparing_to_install_anthos_service_mesh)
-     to install `istioctl`.
+ 1.  Follow the instructions from [Preparing to install Anthos Service
+     Mesh](https://cloud.google.com/service-mesh/docs/archive/1.4/docs/gke-install-new-cluster#preparing_to_install_anthos_service_mesh)
+     to install `istioctl` and `asmctl`.

    Note, the `istioctl` downloaded from above instructions is specific to Anthos
    Service Mesh. It is different from the `istioctl` you can download on
    https://istio.io/.

## Prepare your environment

1. Log in. You only need to run this command once:
...

WDYT? PTAL. Thank you all

Thanks @8bitmp3. Adding "and asmctl" to Step 6 is enough, IMO.

@Bobgy SG from multi-user case perspective?

From the doc, it seems asmctl is used for validation.

asmctl: You use asmctl to help validate your security configuration after installing Anthos Service Mesh. (Currently, asmctl isn't supported on GKE on-prem.)

Is it necessary?

I could not find any reference to asmctl in the Makefiles (management and kubeflow clusters). I suppose it is being used internally by istioctl. When trying to deploy Kubeflow without asmctl installed, we were having a problem related to an unsupported env key in one of the manifests (unfortunately I don't remember exactly which one). We spent a few minutes debugging it and found that asmctl was missing, but the error message was unclear about that.

I am not sure about

When trying to deploy Kubeflow without asmctl installed, we were having a problem related to an unsupported env key in one of the manifests (unfortunately I don't remember exactly which one)

because there were previous reports of missing the env key and that's when people downloaded istioctl from opensource website, not from Google. I successfully deployed with just istioctl downloaded from Google documentation.

I'm not sure, but maybe that's what happened to us. If that is the case, feel free to close this issue, please.


Avoid unintentional memory overwrite in IF operator

IF operator copies its input to the active branch subgraph's input and copy the active subgraph's output's to IF's output. Because IF's tensors and the active branch subgraph's tensors are separately planned and can have overlap.  This PR fix the unintentionl overwrite due to those overlap by using an intermediate buffer.
Bug=https://b/214244865

Thanks for the suggestion, I also agree with the need for two buffers and this new PR reflect that.
For those functions, I am slightly biased towards saving some code space by using enum and have a common underlying function instead of have copy inside each function.  But please feel free to let me know if we should further discuss this

I think I found some edge case where this new approach causes an hard error. Imagine the case where one of the input tensors lies in the read-only memory space (like a kernel for Conv2D). After executing the IF Operation this new implementation tries to write the saved input from intermediate back into the read-only memory space. This causes some kind of hard crash.
I will craft a minimal example later on but this is the log I can provide for now. The Model uses 2 nested IF where the outer one gets a read only Conv2D kernel as input. It crashes after finishing the outer IF and executing this line https://github.com/tensorflow/tflite-micro/blob/746f880a405aeca2db6d005fd9fb8e451f3cea34/tensorflow/lite/micro/kernels/if.cc#L316

Does this make sense to you @deqiangc?

Hi @deqiangc,
I agree on the point that these "read only" inputs should be mapped as input to the subgraphs instead to save on copy operations. The use case is in fact unusual. I am currently trying to implement a research paper which needs to slice Conv2D kernels based on some width (0.0 to 1.0). As TFLM doesn't support dynamic shapes I went with an approach using some IF ops to slice with some threshold levels (like 0.25, 0.5...). The different IF branches slice according to the width threshold. More info can be found in #867.
I know that TFLM is not really made to support these operations 😄 I can provide you with the code for this specific use case (new issue maybe?) if that helps to figure out why these go through the IF instead of being mapped directly.

@ScholliYT
Hello, Tom,
Thanks for the explanation of the background!   It would be useful if you can elaborate how you end up with a read-only tensor as input IF operator. I guess I don't really get the part that using slide_slice would lead to a "read-only" tensor.


How to improve the algorithm?

How it work, how can i improve the algorithm, learner.js, i need to study genome mutation?
Where i find more docs about this kind of programming?

Hi @fernandoGuisso , you will find lots of resources on the Internet about Genetic Algorithms, Neural Networks and so on.
There isn't a perfect place to look for it, just Google ;)


Making some array extensions, generic collection extensions

Making keep(while: ), take(while: ) and skip(while:) array extensions, collection extensions.
Just want to use them with other types 😅

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 540 tests, with 0 failures (0 unexpected) in 19.747 (24.036) seconds

:book:
tvOS: Executed 506 tests, with 0 failures (0 unexpected) in 7.150 (9.714) seconds

:book:
macOS: Executed 370 tests, with 0 failures (0 unexpected) in 1.290 (1.638) seconds

  Generated by :no_entry_sign: Danger


Improving the website name in header

No description provided.

@Mansi168 @MonalikaPatnaik  I can add suitable animation to website to enhance its performance

Heyy sure..you can work on this!
Thanks :-)

Heyy @raiankitsrare u working on changing the U of Umatter...if u don't just lmk??

Yeah, I can do that.
On Sun, 28 May 2023, 10:41 am Monalika Patnaik, @.***>
wrote:

Heyy @raiankitsrare u working on changing the U of Umatter...if u don't
just lmk??
—
Reply to this email directly, view it on GitHub
https://github.com/MonalikaPatnaik/UMatter/issues/81#issuecomment-1565890345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2O2YKLUK3M5J7ZIWADEEMLXILNBTANCNFSM6AAAAAAYIWN5SM
.
You are receiving this because you were assigned.Message ID:
@.***>

Thanks..The current name in header is looking as normal text..you can try improving the font size and style or adding something that distinguishes it in the header beside the logo!

Yes, I can do that
But I am not able to fetch code into my local repository,  ( I.e = Code is
not shown properly ) . That why there are delay in updating those things .
Please provide me direct code .
Thanks & Regards
Ankit Rai
On Sun, 28 May 2023, 11:24 am Monalika Patnaik, @.***>
wrote:

Thanks..The current name in header is looking as normal text..you can try
improving the font size and style or adding something that distinguishes it
in the header beside the logo!
—
Reply to this email directly, view it on GitHub
https://github.com/MonalikaPatnaik/UMatter/issues/81#issuecomment-1565911587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2O2YKL2GPR3IKWE52GGCPLXILR73ANCNFSM6AAAAAAYIWN5SM
.
You are receiving this because you were assigned.Message ID:
@.***>

Hey, u can head over src/components/navbar/index.js around line 81

for fetching the repo you can clone it and run npm start to see the site in your localhost

I am talking about code!!
On Sun, 28 May 2023, 11:46 am Monalika Patnaik, @.***>
wrote:

for fetching the repo you can clone it and run npm start to see the site
in your localhost
—
Reply to this email directly, view it on GitHub
https://github.com/MonalikaPatnaik/UMatter/issues/81#issuecomment-1565923559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2O2YKI63XUIXSA5BCPFEU3XILUSLANCNFSM6AAAAAAYIWN5SM
.
You are receiving this because you were assigned.Message ID:
@.***>

Hey, u can head over src/components/navbar/index.js around line 81

Read this..told you the path for the desired code only..you can look contributing.md file in the repo for the steps to contribute!

Thanks, I'll check it out.
On Sun, 28 May 2023, 12:05 pm Monalika Patnaik, @.***>
wrote:

Hey, u can head over src/components/navbar/index.js around line 81
Read this..told you the path for the desired code only..you can look
contributing.md file in the repo for the steps to contribute!
—
Reply to this email directly, view it on GitHub
https://github.com/MonalikaPatnaik/UMatter/issues/81#issuecomment-1565935218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2O2YKJMLZSES2WX5XMR6ULXILWZZANCNFSM6AAAAAAYIWN5SM
.
You are receiving this because you were assigned.Message ID:
@.***>


Allows menus to list tags instead of children.

This addition allows tags to be listed in menus instead of the element's children.
There is a field for each item in the menu called Tags property. When it is filled, the item will gather the tags from the children and list them.
This fulfills a request for Royal Crescent.

@barryokane @ondrejpialek the ShowNestedContentMenu method only knows about the home page and has no knowledge of the current page it is rendering on. I circumvented this by using UmbracoContext.  Still I'm not able to get the full link with appended anchor such as http://royalcrescent.80d-stage.com/offers/#seasonal.
What do you guys think about deprecating this method and adding a new one that gets the current page as a parameter?

I think there are a couple of different things going on here:

highlighting (adding "current" class): this makes sense to me.  I also think using umbracoContext is fine here.  To solve the issue re full URLs: perhaps refactor so we have an overloaded method that accepts the current page and uses the umbracoContext otherwise?

a "menu" of tags : this adds too much complexity to this method.  I think it would be better to separate the child pages menu functionality from the tags menu functionality.

Also this will change the output HTML (empty  tags will be inserted in some cases), which might break sites when this package is updated.

@filipesousa20 I think it makes more sense to reject this PR and create one for the highlight change on its own.  For the 2nd change : is this something that we 100% know we need for multiple sites?  Perhaps just create a TagMenu method on the specific site project (that can be moved if needed for future sites)?

@barryokane

I thought of this too. I just not sure it was a good option.
I added the tags functionality because I think this will be used again. How do you suppose we should separate tags from regular links? Why do you think empty tags will be rendered?

@filipesousa20 I also cannot find the empty tags reference either. @barryokane did you perhaps mean an empty <ul> tag? The linkContent.Children.Any() has been moved inside the <ul> element, so that will now be rendered for menu items that have no children. Could have a side-effect on sites that style the <ul>.

Yeah, it would need refactored to avoid code duplication - and not a breaking change.  This is a good change though, so we should work out how to include it.

Yes, sorry ( @filipesousa20 @ondrejpialek ) I meant empty  tags when there are no child nodes.

re separate : as the tags is a different functionality it seems to me that a separate extension method makes more sense.  @filipesousa20 we haven't needed this on other sites yet, but if you are sure it will be useful then maybe an extension in the existing Tags class?

Just asked the client about this who replied "Yes, this would definitely be something for the pattern library."
Should I create a separate similar method?

I think it is a good idea.


Support fluids at different temperatures

Right now the mod transports all fluids at the same temperature which works for the base mod but makes hard to work with advanced mods like Pyanodons. This issue is to improve this.

Implemented!


combobox option issue with auto selecting option and issues with virtualization

Description of the bug
when searching combobox, if value property is lowercase and matches one of the options in combobox it will auto select, this becomes an issue when we have some of our datasets value where we will have to select 01 before we can search for 01.1 01.2 and so on, i'd asume this would also be an issue if value is regular numbers where you cannot search for key 1000 before you've selected 1, 10 and 100. this is not an issue with keys that use uppercase.
virtualization still seems to have an issue with our largest dataset, if i search for a code that does not exist it will crash. same crash happens in example storybook thousands of options https://storybook.designsystemet.no/?path=/story/komponenter-combobox--thousands-of-options
and
https://next.storybook.designsystemet.no/?path=/story/komponenter-combobox--thousands-of-options
virtualization seems to have been fixed for our medium-large datasets that also crashed previously
(in codesandbox example below it also crashes with a different error: "Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops")
https://codesandbox.io/p/sandbox/broken-night-tk7wjl?file=%2Fsrc%2FApp.tsx
with filter and multiple virtual sets if you start typing a search that has results but then end up typing something that does not have results, then backspace it will behave weird. in codesandbox if i search NDI i get option 42 and a couple on 400+ but if i then add another character so i get no results like NDII and backspace to NDI i have 0 results and comboboxempty is not showed either, if i then backspace again to ND i only have results over 400 and option 42 does not reappear until the entire field is cleared
Steps To Reproduce
filter needs to be added to avoid first bug when searching for values otherwise it would auto select the first perfect match of value
https://codesandbox.io/p/sandbox/broken-night-tk7wjl?file=%2Fsrc%2FApp.tsx
write: NDI in search, observe 42 being in the set and add another I so search is NDII with 0 results
backspace to NDI and first bug appears with blank box
backspace again to ND and some results appear but option 42 or any other option below 400 is missing
Additional Information
"dependencies": {
"@digdir/designsystemet-css": "0.11.0-alpha.2",
"@digdir/designsystemet-react": "1.0.0-rc.6",
"@digdir/designsystemet-theme": "1.0.0-rc.6",
No response

Thanks for reporting this!
We'll look into this soon hopefully.

In this commit, I fixed one of out objects lying about how many children were being rendered - and it seems like this has fixed the issue.
I'll let you know when we publish another next version, so you can test :)
As for the other issue, it was comparing what you typed in the input to the value, not the label. Should have been resolved in this commit.

I have published a new next version which includes these fixes so we can test the fix "downstream" :)
 - @digdir/designsystemet-css => 0.11.0-alpha.4
 - @digdir/designsystemet-react => 1.0.0-rc.8
 - @digdir/designsystemet-theme => 1.0.0-rc.8

looks good so far 😎 👍

This will be released in #2130


utf8cpp: 3.1.1 -> 3.1.2

Motivation for this change
just a simple update
tested execution of zcash after nixpkgs-review wip
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 96495 run on x86_64-linux:
2 packages built:

utf8cpp
zcash


Introduce RsSmartNotEmpty #952

#952 suggests to change the status code from RsEmpty from 200 to 204. This breaks ~60 test cases which rely on RsEmpty returning 200.
I had to edit a few classes that add content to the Response so that the status code is the expected (and correct) 200. I also had to edit a few tests that expected a 200 status code while 204 is the correct one.

Job #971 is now in scope, role is REV

This pull request #971 is assigned to @fabriciofx/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

Codecov Report

Merging #971 into master will not change coverage.
The diff coverage is 61.53%.

@@            Coverage Diff            @@
##             master     #971   +/-   ##
=========================================
  Coverage     72.29%   72.29%
  Complexity      971      971
=========================================
  Files           224      224
  Lines          4905     4905
  Branches        364      364
=========================================
  Hits           3546     3546
  Misses         1209     1209
  Partials        150      150

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/takes/rs/RsEmpty.java
80% <100%> (ø)
4 <1> (ø)
:arrow_down:

src/main/java/org/takes/rs/RsText.java
54.54% <50%> (ø)
6 <2> (ø)
:arrow_down:

src/main/java/org/takes/rs/RsHtml.java
45.45% <50%> (ø)
5 <2> (ø)
:arrow_down:

src/main/java/org/takes/rs/RsWithBody.java
72.41% <75%> (ø)
8 <3> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@paulodamaso reminder that the PR is not reviewed since 16 days

@paulodamaso I addressed your comment

@Serranya CI checks are failing, please take a look

@paulodamaso I fixed the problems.

@Serranya Thank you very much!

@rultor merge

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 26min)

Quality is low, no payment, see §31:

Code review was too long (23 days), architects (@paulodamaso) were penalized, see §55

The job #971 is now out of scope

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @paulodamaso/z


Document default section and library configuration

It is talked around but not explicitly stated in one part of the
documentation that you should put library configuration lines at the
start of the configuration file.

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

This came up in a troubleshooting session on ##openssl. While default section is obvious for people who worked with openssl configuration files before. See here for examples of end user confusion that the person I was helping eventually found to understand what they were doing wrong:
http://blog.fkraiem.org/2013/03/13/linux-smart-card-authentication-openssl/ - after in the comments
Please let me know if this is not a minimal change.

We need you to either sign our CLA, or edit/push the commit so that "CLA: trivial" appears in the text message (not the commit title).
I agree this is trivial.

So @xemdetia if you update your commit, we can merge this; thanks!

Sorry- I ended up getting put on the road after submitting this PR. I will be updating the commit soon.

I just pushed an update. Let me know if I did it in a fashion that won't break the universe.

Removing needs-cla label (and adding ready), as this is agreed to be trivial.
@richsalz or @levitte to merge?

Reconfirm and I agree it's still trivial.

merged; closing


Use package.json as an .npmrc alternative

As an alternative to .nvmrc, it could be nice to have a custom nvm field in package.json:
{
  "nvm" : {
     "use" : "8.10.0"
   }
}

with aliases
{
  "nvm" : {
     "use" : "lts"
   }
}

Many popular tools like babel, eslint, jest, etc. allows the configuration to be embeded in package.json.
That allows a simpler repository structure without a lot of hidden/rc files (.editorconfig, .nvmrc, .npmrc, .gitignore, .mailmap, etc..) 🤷♂
PS: Related to https://github.com/creationix/nvm/pull/1535#issuecomment-379805161

This is related to the original issue, #651.
The reason that using anything in JSON is difficult is the same; #1535 is an attempt to address it.
I think that using "engines" makes sense because it has an existing use; i don't think I'd want to further pollute package.json with an "nvm" field.


feat: Add argument to control caching, block explorer fetching, forced disk cache updates [APE-913]

What I did
Implement booleans with default values to control:

if retrieved storage types will be cached to disk (useful when working with minimal ABIs)
if the get method will attempt to retrieve a contract type from the block explorer
if a forced disk cache update is performed after contract type is retrieved

fixes: #1419
How I did it
Add three boolean args (cache_to_disk: bool = True, fetch_from_explorer: bool = True, force_disk_cache_update: bool = False) to two ContractCache methods defined in ape / managers / chain.py:

get
instance_at

Add if checks before calls to:

_cache_contract_to_disk
_cache_proxy_info_to_disk

Move the caching logic out of _get_contract_type_from_explorer into the parent get method so it can be managed separately (_get_contract_type_from_explorer should be a single-concern method, and it is only called by get so this should not break any other functionality).
Refactor the contract type logic inside get:

Track types retrieved from memory, disk cache, and from the block explorer separately
After retrieving, determine the 'canonical' contract type by comparing against None and default contract type
Then manage updates to the cache

Treat calls to instance_at with a value for contract_type with special behavior:

If a contract type was provided, assume it was done with the intention of skipping the block explorer fetch, so fetch_from_explorer is overridden to False.

Wherever contract types are compared to default values, explicitly check against contract_type is None instead of not contract_type.
Backwards Compatibility
A default call to instance_at (no contract_type, cache_to_disk: bool = True, fetch_from_explorer: bool = True) must preserve the expected behavior of earlier Ape releases (check memory, check the block explorer, cache the result to disk).
How to verify it
Simple script to verify behavior against a known Uniswap V3 LP address, can observe the caching behavior in a separate terminal:
from ethpm_types import ContractType
import os

os.environ['ETHERSCAN_API_KEY'] = 'edit me'

POOL_ADDRESS = "0xCBCDF9626BC03E24F779434178A73A0B4BAD62ED"
V2_FILE_PATH = "./v2lp.json"
V3_FILE_PATH = "./v3lp.json"

with ape.networks.ethereum.mainnet.use_provider("geth"):

    print("Loading contract with UniswapV2 LP ABI (cache disabled)")
    contract_v2_no_cache = ape.Contract(
        address=POOL_ADDRESS,
        contract_type=ContractType.parse_file(V2_FILE_PATH),
        cache_to_disk=False,
    )

    print("Loading contract with UniswapV3 LP ABI (cache disabled)")
    contract_v3_no_cache = ape.Contract(
        address=POOL_ADDRESS,
        contract_type=ContractType.parse_file(V3_FILE_PATH),
        cache_to_disk=False,
    )

    print("Loading contract with UniswapV2 LP ABI (cache enabled)")
    contract_v2_cached = ape.Contract(
        address=POOL_ADDRESS,
        contract_type=ContractType.parse_file(V2_FILE_PATH),
    )

    print("Loading contract with UniswapV3 LP ABI (cache enabled)")
    contract_v3_cached = ape.Contract(
        address=POOL_ADDRESS,
        contract_type=ContractType.parse_file(V3_FILE_PATH),
    )

    print("Loading contract without ABI (cache enabled, block explorer fetch enabled)")
    contract_v3_from_explorer = ape.Contract(
        address=POOL_ADDRESS,
    )

    print("Loading contract with UniswapV2 LP ABI (forced disk cache update)")
    contract_v2_forced = ape.Contract(
        address=POOL_ADDRESS,
        contract_type=ContractType.parse_file(V2_FILE_PATH),
        force_disk_cache_update=True,
    )

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

Great job! Will need to give it some QA from the team to double check the new features.
Also needs some tests grin

Of course! I'll review some old PRs to see how y'all deal with tests and make updates to mine 👌


Readme update

Corrected M2 to M3 and added details of grub screw for pre loading the spring.

Uh oh, I think you included all the other commits from your two other PRs into here. You should isolate 733f5a5 into a separate branch from the HEAD on the main branch before you attempt to push to your PR.

It's not a good idea to add commits to revert the other ones, it'll just jumble up the revision history. I'm going to close this PR. Sorry. Isolate your changes by reverting back to the HEAD and then commit just this change to a separate branch.


Named Values and some policies not created in upper environment

Release version
4.5.0
Describe the bug
The extractor and publisher ran without any error. The APIs are created but a couple of items are not (indicated by red arrows in the picture)

The policies in the Post method (which is extracted in a subfolder of the API). These are Logic Apps and the operations are extracted in subfolders.
The Named Values
What can be the issue here?

Some more background:
The extraction used a configuration file to limit only certain APIs but everything else is extract all (i.e. no specification under the other headings). The named values are extracted in the "named values" folder properly. Same for all other objects. Earlier I didn't set these secret values in the apim_automation variable set and the publisher pipeline failed with this warning "Named value 64c057c8d442a43b1eaf2629 is secret, but no value or keyvault identifier was specified. Skipping it...". I don't see this warning anymore now after setting the variables in the pipeline library as secret.

Expected behavior
Named values with secret values (not in Key Vault) are not created in upper environment.
Policies for method in Logic App are not created
Actual behavior
Named values with secret values (not in Key Vault) are created in upper environment.
Policies for method in Logic App are created
Reproduction Steps

Run Extractor pipeline with config file (i.e. not Extract all)
Complete pull request of newly created branch

@guythetechie any thoughts?

Just noticed that I transposed the actual/expected behaviour, but the gist of the issue was in the description. Somehow named values and the policies for the operation were not created.

Can you post scrubbed pipeline logs?

@guythetechie The named values in question are 64C057C8D442A43B1EAF2629 and 64C1A4256473136A128D60DB representing logic app sig. You can see them being represented by the variables 64C057C8D442A43B1EAF2629_LOGICAPPSIG and 64C1A4256473136A128D60DB_LOGICAPPSIG respectively. The secrets are substituted in the rewrite-url policy of the POST operation. The logic app is just a hello world type with http trigger.
build72log-prod.txt
The policy.xml has the content. None of these was set in the method, i.e. not just the one with the secret in the named value, so not sure if these two things are completely related.
<policies> 	<inbound> 		<base /> 		<set-method id="apim-generated-policy">POST</set-method> 		<rewrite-uri id="apim-generated-policy" template="/manual/paths/invoke/?api-version=2016-06-01&sp=/triggers/manual/run&sv=1.0&sig={{covtltestapi_manual-invoke_64c1a4256473136a128d60db}}" /> 		<set-header id="apim-generated-policy" name="Ocp-Apim-Subscription-Key" exists-action="delete" /> 	</inbound> 	<backend> 		<base /> 	</backend> 	<outbound> 		<base /> 	</outbound> 	<on-error> 		<base /> 	</on-error> </policies>

I have this issue as well my Named Value are not published, am I missing something?

@UmairEjaz01 you need to override the secret in the configuration..yaml. Please make sure your read the docs and the wiki.

Thanks

Please remember to close the issue if your problem is resolved.

@waelkdouh I had the named values in the configuration.prod.yaml and the variable set in the apim-automation variables library of the pipeline. No warning anymore (used to get them before setting the two things mentioned) when running the publisher.
@UmairEjaz01 Were the secret named values created after you set those?

@waelkdouh I had the named values in the configuration.prod.yaml and the variable set in the apim-automation variables library of the pipeline. No warning anymore (used to get them before setting the two things mentioned) when running the publisher.
@UmairEjaz01 Were the secret named values created after you set those?

Nope I didnt set at all, though they would pass normal text.

I also tried (instead of using variable substitution) and have the "secret" as clear text as the value property in the configuration.prod.yaml. The secret named value wouldn't create. The key vault one works. We expect to use kv for all secrets anyway so this particular bug is not a killer. The Logic App policies for the operations have to be created so that one for me is a critical issue.

@waelkdouh Tested through different combinations of named values set up. It appears that the Name and Display Name of the named values have to be the same for apiops to work. I had them different. So, part of the mystery is solved. Now I noticed another issue, the keyvault url is not updating from the configuration.prod.yaml. Need to see why it is not picking that up.

The key vault url is the same thing. The Name and Display Name of the named value have to be the same for the substitution in the config file to match and take effect.

@covterence that is not true. I just ran a test where the name and displayname are different and it still worked.

@waelkdouh I tried it again and it works as you showed. I backtracked and noted what was the problem. Because I was just updating the config file after the first commit and there is no change in the named value from the dev env (I added another named value and thought that would trigger all of them), the changes were picked up by the diff and publisher didn't see the need to deploy the named value again and pick up the updated config file. Basically if there is any change in the config file relating to a particular artifact item, the source artifact has to be changed somehow in order for config to take effect. This might not be apparent to everyone. For this particular issue, I'm going to close this issue. Thanks for your time.

We give you the option to recommit everything for these scenarios. It's not perfect but it's the nature of the beast as the publisher is driven by git diffing. Glad to see your issue was resolved. Enjoy APIMing!

We give you the option to recommit everything for these scenarios. It's not perfect but it's the nature of the beast as the publisher is driven by git diffing. Glad to see your issue was resolved. Enjoy APIMing!

Is there a way to either not publish or extract policies or put then in exception in extractor? As I only want to extract API and only publish API as rest is configured in upper environment.

You can use the configuration. Extractor.yaml file to extract certain artifacts. More information can be found here. You can't filter by policies but you can use a gitignore file to force the publisher to ignore any artifacts you don't want to promote to higher environments.


AutoCompleteField missing props

Misses isAsync prop and input event handler. (Both passing on/from control)

Fixed


Numpy changes for aten::index converter

This PR addresses the changes for the cases where indices are list of numpy arrays or numpy for aten::index. #2394

Yes, there is. You would just need to pass the indices as inputs to the forward function, instead of having them as constants in the model/graph


Recognition of QR code performs poorly in dark mode

This happens when we use the computer's camera (and this library, in browser) to scan the QR code displayed on a mobile phone's screen, when the phone is set to dark mode, such that even though the code itself is black and white as usual (with a short white border), the rest of the screen is black / dark grey.
An example use case scenario of this is to scan the QR code exported by Google Authenticator on iOS. The more accounts selected, the more complex the QR code and the harder it is for the library to recognize anything. In my experiment using various parameters (varying number of accounts, light conditions, background environment, glare and reflections, different camera resolutions, ability to auto-focus, etc.), I found the library is generally unable to recognize anything even with only 1 account exported (i.e. a rather simple QR code). The library only successfully recognized a QR code displayed in dark mode using a camera with auto-focus ability (Logitech C920, even though it is only 480p-720p). The library fails completely on standard cameras such as Mac's default camera, FaceTime 720p/1080p.
In contrast,  https://github.com/JodusNodus/react-qr-reader can generally recognize the QR codes displayed in dark mode, until the code gets fairly complex (>=3 accounts) and the image received is quite blurred / disturbed (by reflections, etc.)

@polymorpher This component ist just a React wrapper for @zxing/browser, so probably you want to raise an upstream issue unless it is really a problem with the wrapper.
Maybe you can cross-check your scenario with their online demo: https://zxing-js.github.io/library/


Improve logging of unknown extensions

Add missing newlines, and log all unknown extensions instead of logging only the first unknown extension found.

Note: this test has too many issues to deal with in one go.  My hope is to initially decrease the amount of duplication in the C code used here with trivially-safe changes, and once that is done, rewrite it to use std::vector, std::string etc.


imagePullSecrets option to clabverter

Closes #124

cloned by #126 (ci vars)
closing

cloned by https://github.com/srl-labs/clabernetes/pull/126 (ci vars)
closing


Make the alt buffer inherit cursor state from the main buffer

When switching to the alt buffer, the starting cursor position, style,
and visibility is meant to be inherited from the main buffer. Similarly,
when returning to the main buffer, any changes made to those attributes
should be copied back (with the exception of the cursor position, which
is restored to its original state). This PR makes sure we handle that
cursor state correctly.
At some point I'd like to move the cursor state out of the
SCREEN_INFORMATION class, which would make this inheritance problem a
non-issue. For now, though, I've just made it copy the state from the
main buffer when creating the alt buffer, and copy it back when
returning to the main buffer.
Validation Steps Performed
I've added some unit tests to verify the cursor state is inherited
correctly when switching to the alt buffer and back again. I also had to
make a small change to one of the existing alt buffer test that relied
on the initial cursor position being at 0;0, which is no longer the
case.
I've verified that the test case in issue #3545 is now working
correctly. I've also confirmed that this fixes a problem in the
notcurses demo, where the cursor was showing when it should have been
hidden.
Closes #3545

As always, thank you. We're ever more compliant thanks to your continued efforts :smile:

@j4james are the other elements of #3545 already fixed? That issue also mentions things about the tab stops, the margins, etc. I wouldn't want to pre-emptively close that issue if there's still actionable work it's tracking 😝

@j4james are the other elements of #3545 already fixed? That issue also mentions things about the tab stops, the margins, etc. I wouldn't want to pre-emptively close that issue if there's still actionable work it's tracking 😝

Sorry, I mean to address that. The tab stops were fixed in PR #5173 and I think the margin issue was probably fixed by PR #3628. All aspects of the #3545 test case are definitely passing now.

Awesome, thanks for following up!

I've rebased one of my branches on top of main and this PR seems to have a flaw unfortunately.
If you run cmatrix or cacafire (which I assume use the alt buffer?) and press Ctrl+C it'll crash conhost.


while hit terraform apply plan.out got error

while i run terrform apply i got below error
Error: Error creating instance: googleapi: Error 400: Invalid value for field 'resource.networkInterfaces[0].network': 'projects/anthos-lb/global/networks/microk8s-network'. The referenced network resource cannot be found., invalid
with module.microk8s.google_compute_instance.microk8s_instances["microk8s-us-central1-b-01"],
on modules/microk8s/main.tf line 84, in resource "google_compute_instance" "microk8s_instances":
84: resource "google_compute_instance" "microk8s_instances" {

I changed line 76 to use the "default" network.  I didn't debug the terraform script, but suspect that the vpc is not correctly provisioned for the referenced network; /microk8s-network.
This was done at the root level of the project containing the terraform.tfvars.
Look for:
instanceInfo = {
microk8s-us-central1-b = {
name          = "microk8s-us-central1-b"
network       = "default"
zone          = "us-central1-b"
machineType   = "e2-small"
imageProject  = "ubuntu-os-cloud"
imageFamily   = "ubuntu-2004-lts"
instanceCount = 1
}
}


Install Java programs as Windows services without 3rd party libraries

As reported on Reddit: https://www.reddit.com/r/java/comments/jxllez/java_development_on_windows_request_for/gcxqmr7/

It could be done by contributing support for Windows services to jpackage. This would also make Windows a more attractive deployment target while not requiring any changes to Windows itself.


http -> https

added https in site url

What this PR does / why we need it:
This PR adds https in site url on readme file
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Hi @vyaspranjal33 please sign your commits

@utkarsh2102 can i sign commits after push

@utkarshmani1997 commit is verified by github so i dont think i need to sign


Study improval of feedback

Actually we have fery few (no?)  feeback on usage. As we have choosen not to include statistic tools it is even harder to hear about usage of packlist.
The idea would be to make feedback more easy by including a feedback button on any page.
Here are potential libs for helping implementing this :
https://github.com/rm3l/maoni

That looks actually really good. :)

@natrius Would you like to implement it in a pull request?

I would love to but unfortunately i'm not able to yet. Starting now programming with learning C, will switch to Java in something about 2 years. I don't think it should be waited so long g


[loc][Query][TypeScript] Localizability of"0; 0; "FormatCompletedBlockOnLeftCurlyBrace"

Hi,
please help localization team to answer question from translator.
In case translator is not sure about context/meaning/localizability of string we raise a question to Core team.
Please re-assign back to use in case you are not the right contact person or you can´t assign to a proper person.
Can you help please?
Thanks!
Source:
Format _opened block on {
File Name:
LanguageService\TypeScriptLanguageService\bin\Release\localize\ENU\Microsoft.CodeAnalysis.TypeScript.EditorFeatures.dll
Resource ID:
0; "FormatCompletedBlockOnLeftCurlyBrace"
Query:
Is this a code except?Should we keep it in English or is it translatable?

@aozgaa, do you know how we've handled these in the past?  I remember you saying there were some localization wrinkles for new settings checkboxes.  In the absence of other information, I'd say it should be handled the same as Format closed block on }.

i.e. FormatCompletedBlockOnLeftCurlyBrace should be handled in the same way as FormatCompletedBlockOnRightCurlyBrace.

@PDostalek does this address your question?


[Monitoring] Negative submission time report

From time to time, we receive an alert claiming that we did not submit -T minutes.
Questions to answer

Why is it happening?
Is it actual warning?

TODO

Fix the computation of time delay.

planned to use different code base for monitoring


[Config] Capture body

Classic agent link

https://www.elastic.co/guide/en/apm/agent/java/current/config-core.html#config-capture-body

What it does
For transactions that are HTTP requests, the Java agent can optionally capture the request body (e.g. POST variables). For transactions that are initiated by receiving a message from a message broker, the agent can capture the textual message body.
Implementation difficulty, location, and dynamism

Already implemented
https://opentelemetry.io/docs/zero-code/java/agent/configuration/#capturing-servlet-request-parameters
https://opentelemetry.io/docs/zero-code/java/agent/configuration/#capturing-consumer-message-receive-telemetry-in-messaging-instrumentations
Not dynamic

@akhileshpok please feedback on whether this really needs to be supported as a dynamic capability

@jackshirazi - I have not heard that this parameter needs to be dynamic. Let me check with the customer!

@jackshirazi - I have been informed by the WF account team that this parameter needs to be dynamic.