Convert deprecated filter names to canonical one

Please provide a description of this PR:
Currently we can get a deprecated name in listener.  Below is an example
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: size-limit
  namespace: istio-system
spec:
  workloadSelector:
    labels:
      istio: ingressgateway
  configPatches:
    - applyTo: HTTP_FILTER
      match:
        context: GATEWAY
        listener:
          filterChain:
            filter:
              name: envoy.filters.network.http_connection_manager
      patch:
        operation: INSERT_BEFORE
        value:
          name: envoy.buffer
          typed_config:
            '@type': type.googleapis.com/udpa.type.v1.TypedStruct
            value:
              maxRequestBytes:<PHONE_NUMBER>

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test release-notes_istio

unit-tests_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test release-notes_istio

unit-tests_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test unit-tests_istio

integ-distroless_istio
fbd65a65c922160834a7988555883a995395ca89
link
true
/test integ-distroless_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

https://github.com/istio/istio/blob/da6178604559bdf2c707a57f452d16bee0de90c8/releasenotes/notes/envoy-filter.yaml - We mentioned we will not supported deprecated filter names going forward. I am wondering it is a continuous maintenance headache to covert back and forth.

Yes,  i totally agree. But we can not eliminate this immediately, otherwise could break backward compatibility if a name is removed from the envoy. So this is better to have

Should we add a feature flag to support old names so that the hit is taken by people who use it and we can default it to false and we can remove that flag couple of releases down the line?

I am not sure I understand the motivation here. There should be no expectation that EnvoyFilters are compatible on upgrade

NOTE 1: Some aspects of this API are deeply tied to the internal implementation in Istio networking subsystem as well as Envoy’s XDS API. While the EnvoyFilter API by itself will maintain backward compatibility, any envoy configuration provided through this mechanism should be carefully monitored across Istio proxy version upgrades, to ensure that deprecated fields are removed and replaced appropriately.

From https://istio.io/latest/docs/reference/config/networking/envoy-filter/
I am actually glad that we break compatiblity here; similar to the reasons in https://sre.google/sre-book/service-level-objectives/#xref_risk-management_global-chubby-planned-outage, it sets expectations that we will break EnvoyFilters. This is especially needed for the next few releases where I expect the internal XDS shape will change quite a bit.

Do both of you feel we should remove the canonical name convert since we have the notes?

Normally support deprecated name for some releases and removing it totally is ok. But as you can see, still many users are using the deprecated name.  This is the motivation here.
If both of you agree, I am ok with completely clean up all the temporary conversions.

Why not just keep things the way they are? Then we do not need to expand the workarounds but also do not break existing users?

I could not understand this philosophy.

With this theory, i doubt what's the effect of the deprecation release note.  Normally, we will announce the deprecation of something and then after some releases remove them, otherwise the deprecation stuff will become more and more. And eventually it would be more difficult to clean up


fix eventhubs java landing page

fix the link to EventHubs javadoc
remove references to exact java client versions. Versions are like moving targets :) - lets not add direct version numbers to documentation pages. Add maven central repository link of the project - so that developers can grab pom entry for the version they need.
add link to the main java samples.

cc: @sethmanheim, @ShubhaVijayasarathy, @JamesBirdsall, @sabeegrewal

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/eventhub.md
:white_check_mark:Succeeded

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


Add CMake option to not build tests

To enable builds without tests, I added an option. This is useful for packaged versions of yoga, see https://github.com/conan-io/conan-center-index/pull/23019#discussion_r1523276818

My company does not allowed me so sign the CLA. Sorry


Can this mod intercept packets?

This is a very interesting mod. May I ask what this mod specifically does? It intercepts MC packets first and then sends them? Or is it just a copy of the original data without affecting the original sending process?

yes, it just a copy of the original data. We can just peek networking of Minecraft.


Proposed Feature: AIBooru Tags System

AIBooru github:
https://github.com/aibooruorg/aibooru

Hi, I appreciate your creating an issue, however, in the interest off creating interest to people who would be reading, please add information on what specific feature you'd like and how it benefits you, and how it would be integrated into the system from your point of view.
Thanks.

woops

Hi, I appreciate your creating an issue, however, in the interest off creating interest to people who would be reading, please add information on what specific feature you'd like and how it benefits you, and how it can be integrated into the application from your point of view.
Thanks.


Does not seem to fix cases where generic parameters are missing

I'm refactoring an existing library and needing to introduce generic parameters to just about every type in the library.
There are hundreds of the following error message:
error[E0107]: missing generics for enum `Value`
  --> src/utils.rs:26:16
   |
26 |     args: &'a [Value],
   |                ^^^^^ expected 1 generic argument
   |
note: enum defined here, with 1 generic parameter: `State`
  --> src/model/value.rs:19:10
   |
19 | pub enum Value<State> {
   |          ^^^^^ -----
help: add missing generic argument
   |
26 |     args: &'a [Value<State>],
   |                ~~~~~~~~~~~~

However cargo fix doesn't seem to do anything about them when run with cargo fix --allow-dirty --broken-code. Am I missing something, or is this just not something cargo-fix does? Was hoping to automate a lot of this!

I believe that particular error has an applicability called "HasPlaceholders" which means it cannot be automatically applied since the suggestion may not be correct. #163 and #200 are open to provide support for applying suggestions that may not be correct.

Great, I'll subscribe to those and close this as a dupe :) Appreciate the quick response!


refactor: change the text on the job button to be start job

What does this pull request change?
Changes the button label on the create job button to be "start job"
Why is this pull request needed?
babak request
Issues related to this change

@collinlokken I'vre removed the create job component, please have another look 👍


MediaQuery.of() called with a context that does not contain a MediaQuery.

Done

Sorry ! I have fixed this problem

May I know what the solution was? I'm encountering the same issue.
Thank you in advance.


feat: add KMS key management param

fixes: https://github.com/firebase/extensions/issues/1693

Approved, may just missing an updated test case for the new configuration.


Add -FE0F-

Some environments (probably iPhone?) enter this emoji pattern as same as <emoji>-<emoji modifier>.
e.g. https://twitter.com/405yuna/status/799270360587739137

this should be present for all emoji sequences that require it. is there a particular emoji/modifier combo you think is incorrect?

https://docs.google.com/document/d/1pflCUy4NnVHoyZu9WEo9aZ1WfSH-QGrbJBhEvi5t6Xg/edit
Some app have incorrect output for "Gender Emoji ZWJ Sequences", like telegram.
For example 26f9-1f3ff-fe0f-200d-2640-fe0f is Woman Bouncing Ball: Dark Skin Tone

The qualified/non-qualified versions are generated from the Unicode source material, so contain only the official sequences.
To allow matching against badly implemented apps, you can explicitly filter out U+FE0F sequences and match against the non_qualified keys.

There is a bug here though:

#217


JiwJiw73

Sorry, we do not accept changes directly against this repository. Please see
CONTRIBUTING.md for information on where and how to contribute instead.
@CHOMSIN

/close


Check installed version of Dependency-Check before fetch new one

Stop fetching OWASP Dependency-Check from GitHub at each build when version parameter is specified to improve performance and reduce unnecessary network traffic.

It would be awesome to place the CLI in the tool cache of the agent. Prior to the scan update the data. I can pick this up in the next month or so

Does this increase the size of the vsix file? We ran into issues with this prior because the size exceed the default maximum size.

Not at all. You don't package the cli with the extension

The "localInstallPath" parameter can be used for on-premise build agents


#1065: No warnings when creating platform app

Bugfix for #1065. With these changes I can create a NetBeans Platform Application, open project properties and the libraries report no errors on JDK-11 as well as JDK8.
The fix works only when using NetBeans from a binary. There are still some problems with ant tryme test. Hopefully not that important.

Approved.


Rock overhaul - Reduex

Overhauls all of the rocks to have better UV scales as well as applies missing materials

This should be ready for testing and review

Requires #126 for improved Corsair Cove texture

Master / PR

Master / PR


Provide option for manual distanceFilter calcaultion

Previous iterations of the plugin did not auto-calculate the distance filter based on speed. While your distance filter calculation is a useful feature and works well on the move, it would be great to be able to control this calculation, tweak the parameters, or have no distance filter calculation at all.
For example, when a user is stationary it's useful to me to be picking up location still, but with a distance filter of '(round(speed, 5))^2 + distanceFilter' this will likely not trigger a location scan.
For me, it would be great to have the distance filter set as a static number and calculate the distance filter within the app. I've found distanceFilter = speed works quite nicely for this use case, as this allows for scans to take place even when stationary.
Perhaps it would be good to add a boolean variable, 'useIntelligentDistanceFilter' or something, to trigger either using your current formula or only the number entered into distanceFilter?

Ignore this, setting as 0 does work for stationary after all :)


⚠ Spryr has degraded performance

In 0eefef3, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3150 ms

Resolved: Spryr performance has improved in 51e1c14 after 12 minutes.


SSH access to Web App With Containers (VNET Integrated) not working

I am using a Web App service (Containers) in a Virtual Network and I cannot use the web interface to SSH onto a container. Based on the documentation, I should be able to just use env vars SSH_PORT but it does not work. In my Dockerfile, I have the following:
ENV PORT=80 ENV PORT_SSH=2222 EXPOSE $PORT $PORT_SSH
Also, SSH is configured to listen on port 2222, the Web App Configurations settings have PORT and SSH_PORT defined and I am still able to SSH onto my container for debugging purposes.
Any ideas?

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@andresbarcenas
Thanks for your Question! We are currently investigating and will get back on this.

We have noticed the same. For some reason App Service is not setting these environment variables. We set PORT and SSH_PORT with default values 80 and 2222, and last week everything worked fine as App Service overwrite those values when starting. Now when App Service starts, it will use ports 80 and 2222 when VNET integration is in use. Looking at environment variables from Kudu shows the same.

@andresbarcenas, and @PekkaJalonen, Apologies for the inconvenience with this issue.
Could you please the WebApp name directly here or indirectly for a deeper investigation.

We were able to get the instance working again. Looks like using AADDS as primary DNS broke the site as easyauth(using Azure authentication) logged errors of not able to resolve resource. After reverting DNS to Azure internal DNS, site started to work normally again. There was a health alert from friday on couple of our sites for need of restore of the site which probably was the trigger for this. Now the site does not overwrite PORT or SSH_PORT anymore, but uses default values 80 and 2222 which we have set and both are working even when VNET Integration is in use.

Pekka, Glad to know it is working now. Thanks for the update and sharing additional details with the community.
@andresbarcenas, Kindly review the ‘Service Health’ notification on this as Pekka mentioned.
Just to highlight, Service Health provides you with a customizable dashboard which tracks the health of your Azure services in the regions where you use them. In this dashboard, you can track active events like ongoing service issues, upcoming planned maintenance, or relevant health advisories. When events become inactive, they get placed in your health history for up to 90 days.
Again, I understand the frustration with this issue.  Apologies for all the inconvenience.

@andresbarcenas, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion.


NIFI-12854: Add S3 user metadata via flowfile attributes

Summary
NIFI-12854 The PutS3Object processor currently supports adding user metadata to objects via dynamic properties.  This PR adds support for using FlowFile attributes to generate S3 user metadata.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check
[x] JDK 21

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

Hey @gforeman02 thanks for putting this forward. I'm not sure that I agree that it's a good idea to add this, though. In general, we should not support multiple different ways to do the same thing. Doing so just leads to confusion. Is there a reason that the existing method does not work for you?

Hello @markap14 , thanks for posting.  Yes, the existing method (dynamic properties) requires knowing what the user metadata keys will be in advance.  My client has a scenario where one flow has metadata key A and another that has key B and this is only known by reading the flowfile attributes.  It would be great for the processor to work in the same way it does for tags.

@markap14 one other approach could be to have a special dynamic property that the processor uses to read key/value pairs from the dynamic property value.  Maybe something like a key "metadata.json" with a value of { "a": "a_val", "b": "b_val" .... }.

@markap14 i created a custom s3 nar with the following mod:
final Map<String, String> userMetadata = new HashMap<>();
for (final Entry<PropertyDescriptor, String> entry : context.getProperties().entrySet()) {
    if (entry.getKey().isDynamic()) {
        final String value = context.getProperty(
                entry.getKey()).evaluateAttributeExpressions(ff).getValue();
        if( "metadata.json".equals(entry.getKey().getName())){
            ObjectReader reader = new ObjectMapper().readerFor(Map.class);
            Map<String, String> map = reader.readValue(value);
            userMetadata.putAll(map);
        } else {
            userMetadata.put(entry.getKey().getName(), value);
        }
    }
}

I am happy to modify this PR and resubmit it for consideration.

@exceptionfactory thanks for the idea on an approach.  I guess a decision needs to be made:
Should the PutS3Object processor support dynamic metadata keys?
+1 here for supporting dynamic metadata keys since I have a scenario where this is used.  @markap14 if the concern is unique to me then I have a workaround in place.

Thanks for the reply @gforeman02, if you willing to rework the PR along the lines of using dynamic properties with wildcards as a way to support dynamic metadata keys, that sounds like a good way forward, without introducing new properties.

@gforeman02 Given the current conflicts and the different direction, I'm closing this pull request for now. Feel free to reopen when you are ready with a different approach, or open a new PR if you prefer. Thanks!


how to set time label?

hi :
how to set axis-x time label

Thank you very much !That's great !

@bill-bil  你好：
我们发现你的GitHub上的项目 ebank_enterprise 是我们公司的商务项目，我是本项目的创建人之一，请您尽快在GitHub上删除此项目，谢谢！
否则，我司将走法律途径！
有问题，可随时联系我。
我的联系方式：
电话：13006610162
微信号：per_lsy
@bill-bil  Hello:
We found your project ebank_enterprise on GitHub is a business project of our company, I am one of the creators of this project, please delete this project on GitHub as soon as possible, thank you!
Otherwise, our company will take legal way!
If you have any questions, you can contact me anytime.
my contact information:
Tel:<PHONE_NUMBER>2
WeChat ID: per_lsy


Packet IN via Http request

Hi,
I would like to know oftee sends the whole OFmessage or just its data part?
Thanks,
Adib

I am getting only data part not OF headers.

TeeRawPackets defaults to true and when this is true only the packets are sents, not the OF headers. perhaps this should default to false

And actually, there is a context sent in front of the OF packet. It is a context and consists of the DPID and port for the Packet In message. The DPID is 64 bits and the port is 32bits

just committed a change to the default. so now, by default, the context + of header + packet is sent/


Julia implementation using LibCURL

Is your feature request related to a problem? Please describe.
There is no production ready native Julia implementation of gRPC yet.
Describe the solution you'd like
In order to have a higher chance of a stable Julia native gRPC implementation, it would be helpful to have support for LibCURL as the HTTP/2 layer in gRPC.
LibCURL.jl will provide HTTP/2 support  starting with Julia 1.6 and is part of the stdlib in Julia then.
Describe alternatives you've considered
Wrapping the C core library with https://github.com/JuliaInterop/Clang.jl and starting with #15460
Additional context
We managed to rebuild the gRPC.jl mentioned here #24786, but we don't know in which state it is yet, because we need help with testing.
Related Julia talks are:
https://github.com/Humans-of-Julia/GraknClient.jl/issues/19
https://github.com/JuliaWeb/HTTP.jl/issues/328

Meanwhile there is a great solution here, it uses native LibCURL: https://github.com/JuliaComputing/gRPCClient.jl


Chat - Technical Spike

Within the application, we need to establish a chat function in which users can communicate with one another. For example, task volunteers should be able to reach informally to the task creator to ask for further details.
For this, we can incorporate already existing technology such as matrix: https://matrix.org/
Some documentation:
https://spec.matrix.org/latest/
https://matrix.org/docs/guides/usage-of-the-matrix-js-sdk
Matrix already has a React sdk, so we can use it to integrate with it. https://github.com/matrix-org/matrix-react-sdk
Tasks:

[ ] Integrate Matrix-react-sdk into the current application
[ ] Make sure it matches closely in style to the existing application
[ ] A chat dialog should be able to be opened with anyone who has a known public key
[ ] Anyone with an AccountID, should be able to connect with another AccountID

TASK: 100 Euros

I have created a branch for you. Please use it to commit all your changes...
git fetch origin
git checkout 60-chat-technical-spike


Android - Could not resolve project :UnityExport

Unity: 2019.4.23f1
React-native: 0.63.4
I run application in ios is ok.
but on android i face this issue.

my build.gradle

settings.gradle

UnityExport/build.gradle

Any help, please.

try this
include ":UnityExport"
project(":UnityExport").projectDir = new File('..\\android\\UnityExport\\unityLibrary')

flatDir {
   dirs "${project(':UnityExport').projectDir}/libs"
}


Add a device uuid filter for Onvif DH

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Each Onvif device comes with an uuid that can be used to unique identify an Onvif device on a network.  We should consider add a filter that uses the device uuid to filter discovered Onvif devices.
Is your feature request related to a way you would like Akri extended? Please describe.
A clear and concise description of what cool feature Akri lacks. Ex. I'd love Akri to support discovery of Bluetooth devices.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

PR merged


WER for Egyptian Arabic

Hi, I was wondering the WER for Egyptian Arabic, since I don’t see a score on this page? https://huggingface.co/Zaid/wav2vec2-large-xlsr-53-arabic-egyptian

If I remember correctly I think it was around 0.5 .

Thanks.


f key on YouTube will make video full screen

Describe the bug
The Youtube keyboard shortcut to make the video full screen is f. Pressing f to show link hints makes the video full screen and show the hints at the same time.

To Reproduce
Steps to reproduce the behavior:

Youtube
Press f key

Expected behavior
I think the plugin should block YouTube from getting the key press. Or the plugin should allow us to choose different keys that do not interfere with sites we use.
<!-- You can collect the versions by pasting the following script at the terminal

MACOS=10.15.6 (19G73)
SAFARI=Version 13.1.2 (156<IP_ADDRESS>.3)
VIMARI=2.0.3

@kenthinson Thank you for your report. This behaviour is actually fixed in the master branch, but has not been released on the App Store. For more info on how it is fixed see #176.


Update security.md

As per the issue #1441
Fixes:  #1441
Description
I changes the file with thw symbols and tried to make it interactive. So that it becomes easy to read it fastly and easily.
If any further changes required please let me know.
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[ x] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[ x] I have made this change from my own.
[ x] I have taken help from some online resources.
[ x] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] I have made corresponding changes to the documentation.
[ x] My changes generate no new warnings.
[ x] I have tested the changes thoroughly before submitting this pull request.
[ x] I have provided relevant issue numbers and screenshots after making the changes.

If possible please add the labels of a level 2 or level 3 , it will be a great help from your side.
Thank you for giving me an opportunity.


Remove Logging from library

Hi, for better performance of your library will be better to delete all of logging code, because it spends time and allocates a lot of objects, and also it uses reflection api for getting class name and etc, usually this does lazily putting arguments and formatted/concatenated on demand.
LoggerLevel settings nohow to helps here, because message will creates even with LoggerLevel.NONE
for example:
StartupLogUtils.d("${startup::class.java.simpleName} being dispatching, onMainThread ${startup.callCreateOnMainThread()}.")

and somewhere I saw buildString with huge tables)

Yes, you are right. But I won't delete the logging code because it's useful for development debugging.
However, I will modify the log output mode to solve this problem.
Thank you ~

It has been updated in version 1.1.0,  details


Failing TestLoad_RunWithCALLT/loaddeployed on Windows

Go 1.20, Windows, likely caused by improper DB/Blockchain finalisation in the test itself, firstly discovered at https://github.com/nspcc-dev/neo-go/pull/3110, https://github.com/nspcc-dev/neo-go/actions/runs/6467920919/job/17558942705?pr=3110.
2023-10-10T10:25:31.4419707Z === RUN   TestLoad_RunWithCALLT
2023-10-10T10:25:31.4420116Z === RUN   TestLoad_RunWithCALLT/loadgo
2023-10-10T10:25:31.4420555Z === RUN   TestLoad_RunWithCALLT/loadnef
2023-10-10T10:25:31.4420971Z === RUN   TestLoad_RunWithCALLT/loaddeployed
2023-10-10T10:25:31.4422669Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	initial gas supply is not set or wrong, setting default value	{"InitialGASSupply": "52000000"}
2023-10-10T10:25:31.4423769Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	mempool size is not set or wrong, setting default value	{"MemPoolSize": 50000}
2023-10-10T10:25:31.4425018Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	P2PNotaryRequestPayloadPool size is not set or wrong, setting default value	{"P2PNotaryRequestPayloadPoolSize": 1000}
2023-10-10T10:25:31.4426163Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	MaxBlockSize is not set or wrong, setting default value	{"MaxBlockSize": 262144}
2023-10-10T10:25:31.4427295Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	MaxBlockSystemFee is not set or wrong, setting default value	{"MaxBlockSystemFee":<PHONE_NUMBER>00}
2023-10-10T10:25:31.4428485Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	MaxTransactionsPerBlock is not set or wrong, using default value	{"MaxTransactionsPerBlock": 512}
2023-10-10T10:25:31.4429786Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	MaxValidUntilBlockIncrement is not set or wrong, using default value	{"MaxValidUntilBlockIncrement": 86400}
2023-10-10T10:25:31.4430842Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	NativeActivations are not set, using default values
2023-10-10T10:25:31.4431732Z     logger.go:130: 2023-10-10T10:25:19.739Z	INFO	Hardforks are not set, using default value
2023-10-10T10:25:31.4432619Z     logger.go:130: 2023-10-10T10:25:19.746Z	INFO	no storage version found! creating genesis block
2023-10-10T10:25:31.4433291Z     basic.go:300: contract (..\..\examples\storage\storage.go):
2023-10-10T10:25:31.4433965Z         	Hash: 0f825b050eb8ce9eaa82993e90615025ab798016
2023-10-10T10:25:31.4436049Z         	AVM: VgIQv2AMB2RlZmF1bHRK2TAkBNswYUGb9mfOYEBXAAJYeHlTQeY/GIR4QFcAAVhZeFNB5j8YhFlAVwABWHhQQZJd6DFAWFlQQZJd6DFAVwABWHhQQS9Yxe0IQFcDAVh4EFNB3zC4mnAQwHFoQZwI7ZwmJ2hB81S/HUrZQCQE20ByaUrYJgRFwkpqEM4MATqLahHOi89xItVpQFcBAVh4EFNB3zC4mnBoQA==
2023-10-10T10:25:31.4438077Z     logger.go:130: 2023-10-10T10:25:20.895Z	DEBUG	done processing headers	{"headerIndex": 1, "blockHeight": 0, "took": "0s"}
2023-10-10T10:25:31.4439046Z     logger.go:130: 2023-10-10T10:25:20.895Z	DEBUG	done processing headers	{"headerIndex": 2, "blockHeight": 1, "took": "0s"}
2023-10-10T10:25:31.4440002Z     logger.go:130: 2023-10-10T10:25:20.895Z	DEBUG	done processing headers	{"headerIndex": 3, "blockHeight": 2, "took": "0s"}
2023-10-10T10:25:31.4441090Z     logger.go:130: 2023-10-10T10:25:20.911Z	DEBUG	done processing headers	{"headerIndex": 4, "blockHeight": 3, "took": "0s"}
2023-10-10T10:25:31.4442944Z     logger.go:130: 2023-10-10T10:25:20.975Z	INFO	persisted to disk	{"blocks": 0, "keys": 127, "headerHeight": 0, "blockHeight": 0, "took": "120.2739ms"}
2023-10-10T10:25:31.4443809Z     basic.go:300: contract (..\..\examples\runtime\runtime.go):
2023-10-10T10:25:31.4444481Z         	Hash: 24253e67469322dd404a95e594b8b9f96445f24a
2023-10-10T10:25:31.4447944Z         	AVM: VgEMFK7Y2ztdZkygS9177TeZL/lYVgtBStkwJATbMGBB6X04oAAgKBQMC2luaXQgY2FsbGVkQc/nR5ZAVwACeSYyDCtfZGVwbG95IG1ldGhvZCBjYWxsZWQgYWZ0ZXIgY29udHJhY3QgdXBkYXRlNFZADC1fZGVwbG95IG1ldGhvZCBjYWxsZWQgYWZ0ZXIgY29udHJhY3QgY3JlYXRpb240JEBYQfgn7IwmGQwOVmVyaWZpZWQgT3duZXJBz+dHlghACUBXAAF4Qc/nR5ZAVwEBeBHAcAwFRXZlbnRoUEGVAW9hQEHpfTigACAoBAlANLRANLGqJhsMFm9ubHkgb3duZXIgY2FuIGRlc3Ryb3k6NwAAQFcAAjWM////qiYaDBVvbmx5IG93bmVyIGNhbiB1cGRhdGU6eHlQNwEAQA==
2023-10-10T10:25:31.4450831Z     logger.go:130: 2023-10-10T10:25:21.398Z	DEBUG	done processing headers	{"headerIndex": 5, "blockHeight": 4, "took": "0s"}
2023-10-10T10:25:31.4452479Z     logger.go:130: 2023-10-10T10:25:21.398Z	INFO	runtime log	{"tx": "30566a82783bda00679c708906962ad8855153a2a72c5db9c91c037cb58b642a", "script": "24253e67469322dd404a95e594b8b9f96445f24a", "msg": "init called"}
2023-10-10T10:25:31.4453996Z     logger.go:130: 2023-10-10T10:25:21.398Z	INFO	runtime log	{"tx": "30566a82783bda00679c708906962ad8855153a2a72c5db9c91c037cb58b642a", "script": "24253e67469322dd404a95e594b8b9f96445f24a", "msg": "_deploy method called after contract creation"}
2023-10-10T10:25:31.4455665Z     logger.go:130: 2023-10-10T10:25:21.434Z	INFO	persisted to disk	{"blocks": 5, "keys": 139, "headerHeight": 5, "blockHeight": 5, "took": "36.5208ms"}
2023-10-10T10:25:31.4457011Z 2023-10-10T10:25:21.680Z	INFO	runtime log	{"tx": "1138bdffe0e8975d15e7e07d96d1a65f2093a0f09a24ffd81da7c318438abc22", "script": "24253e67469322dd404a95e594b8b9f96445f24a", "msg": "init called"}
2023-10-10T10:25:31.4458248Z 2023-10-10T10:25:21.680Z	INFO	runtime log	{"tx": "1138bdffe0e8975d15e7e07d96d1a65f2093a0f09a24ffd81da7c318438abc22", "script": "24253e67469322dd404a95e594b8b9f96445f24a", "msg": "Verified Owner"}
2023-10-10T10:25:31.4459658Z     testing.go:1206: TempDir RemoveAll cleanup: remove C:\Users\RUNNER~1\AppData\Local\Temp\TestLoad_RunWithCALLTloaddeployed2213489902\001\000002.ldb: The process cannot access the file because it is being used by another process.
2023-10-10T10:25:31.4460812Z --- FAIL: TestLoad_RunWithCALLT (5.20s)
2023-10-10T10:25:31.4461858Z     --- PASS: TestLoad_RunWithCALLT/loadgo (0.56s)
2023-10-10T10:25:31.4462894Z     --- PASS: TestLoad_RunWithCALLT/loadnef (0.66s)
2023-10-10T10:25:31.4463526Z     --- FAIL: TestLoad_RunWithCALLT/loaddeployed (3.98s)

???


Use homu to manage merging pull requests

We've got enough reviewers that I think this makes sense. It'll also ensure we don't have a red build as we do right now... :-)

@carllerche any objections if I do this? I feel weird modifying the settings of a repo under someone else's namespace.

Ah I don't have access to setting as a collaborator anyway.

The repo will be moving to the newly formed nix-rust org, which will hopefully make this easier.

We should have @homu now! I'm going to polish up a change that's wip and see if it all works. I've also protected master branch against force push.

Migrating the repo to the nix-rust org seems to have borked Travis, so we're kind of one step forward one step back right now. I'm going to give it a couple of hours to sort itself out, and then try and contact them.

#266 was merged good via @homu. Closing this!


Fix multi.exec with empty queue and previous watch

Description
When calling exec on a multi instance which you did not use, no command is sent currently.
This is a problem for watched keys, because no EXEC means no unwatch, which might cause hard-to-debug problems.
Proposed Fix: Sending UNWATCH

Checklist

[ ] Does npm test pass with this change (including linting)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Not tested yet, no documentation update needed

@Mik13 good catch!

IMO executing MULTI EXEC (instead of UNWATCH) is cleaner.
I don't like executing commands when not needed (if there is no WATCH, there is no reason to UNWATCH/MULTI EXEC).

I need to think about that more...

@Mik13 good catch!

IMO executing MULTI EXEC (instead of UNWATCH) is cleaner.
I don't like executing commands when not needed (if there is no WATCH, there is no reason to UNWATCH/MULTI EXEC).

I need to think about that more...

Hi @leibale!
To 1: Agree
To 2: Same, thats why I am using one O(1) command instead of O(1) + "Depends on commands in the transaction"
I do not understand the project good enough how we could possibly catch the watch inside of the multi-command when issued outside of course. That would be perfect, if it does not add too much complexity.

@guyroyse we need your opinion here.. :)

My thought is that we should call what the user told us to call. If I call MULTI and then call EXEC without doing anything else, then that's what I asked for and that's what I should get. It's cleaner and, more importantly, provides unsurprising behavior to the user.

@Mik13 @guyroyse review my changes please? :pray:

Codecov Report
Base: 95.60% // Head: 91.68% // Decreases project coverage by -3.93% :warning:

Coverage data is based on head (ba6485a) compared to base (1be8422).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head ba6485a differs from pull request most recent head 48259a7. Consider uploading reports for the commit 48259a7 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2423      +/-   ##
==========================================
- Coverage   95.60%   91.68%   -3.93%
==========================================
  Files         455      455
  Lines        4550     4544       -6
  Branches      520      518       -2
==========================================
- Hits         4350     4166     -184
- Misses        128      311     +183
+ Partials       72       67       -5

Impacted Files
Coverage Δ

packages/client/lib/cluster/multi-command.ts
66.66% <ø> (+1.04%)
:arrow_up:

packages/client/lib/multi-command.ts
80.00% <ø> (-1.82%)
:arrow_down:

packages/client/lib/client/index.ts
89.65% <100.00%> (-2.66%)
:arrow_down:

packages/client/lib/client/multi-command.ts
90.00% <100.00%> (+1.53%)
:arrow_up:

packages/client/lib/commands/FUNCTION_LOAD.ts
16.66% <0.00%> (-83.34%)
:arrow_down:

packages/client/lib/commands/FUNCTION_FLUSH.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/FUNCTION_RESTORE.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/CLIENT_NO-EVICT.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/FUNCTION_LIST.ts
37.50% <0.00%> (-62.50%)
:arrow_down:

packages/client/lib/commands/FUNCTION_STATS.ts
25.00% <0.00%> (-62.50%)
:arrow_down:

... and 35 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


chore: release 1.2.0

:robot: I have created a release *beep* *boop*
1.2.0 (2019-09-06)
Features

load protos from JSON, grpc-fallback support (d3e8ac6)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-scheduler/releases/tag/v1.2.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


[CT-2828] Fix pre-commit failures when cutting a new release branch

pre-commit fails on the new branch that gets created by cut-release-branch.yml.  Before committing into the temp branch we need to fix the newlines and trailing spaces.
I tried to implement something similar to what we do for the release and it failed.
We were trying to release at the time so moved on without fixing this to unblock cutting release branches.  Still needs to be fixed as it requires manually running this fix instead of just merging in the PRs.
Note: when testing this, be sure to not create a new branch that falls into our branch protection rules and ends in .latest so you can modify and delete the new branch.

As part of this we should really be sure to test out the changes and make sure we're not missing any bigs.  Also ensure adapters are able to cut the branch.

Currently blocked on bug with merging.  Work is done.


Event Log Export

When you click on an Event Log entry and click the "Export" button it comes up saying downloading "Application" and then successfully downloaded "Application", however it does not download anything nor prompt for download in the browser.  I don't know where the log was exported to if it was indeed exported at all.

Turns out this works in Chrome for exporting things like Certs, but you can't connect to Event Logs from Chrome.  I CAN connect to Event Logs from Firefox but downloads don't work in Firefox.

Can you please confirm that the download works from a remote Windows 10 machine using Edge? Is the machine part of a workgroup or domain? Chrome has issues in a workgroup.

Closing due to inactivity.  Please reopen to pursue further.


Add Minecraft to emote filter

add Minecraft to the blacklist

+1

This should really not take this long to merge.

Thank god! xoxo @Jiiks


Wrong icons in file grid

Steps

Go to the project organizer.  Look at the icons.

Expected
Icons between each of those grids would be the same for the same items.
Actual
Components and projects in the file grid have folder icons inside of their category icons.

@GaryKriebel this could be a good intern task as well.

Is this issue talking about the difference between the icon that appears next to project/component names in different views? If so, would you like the cubes/component icons (first image below) to be used instead of the folder icons (second image below)? @brianjgeiger

That is all correct, @leb2dg. Use the icons from the first image in all places.

Wonderful, I'll work on this.

Closed by #2871


Trying to edit Custom Names and descriptions on large number of devices

Hi,
I am trying to customize my list of devices returned from the integration of network_scanner with the hints shown on other issues and documentation.
I have managed to get some of the devices to show the custom name assigned from the configuration.yaml:

However, since I have more than 25 devices in place, the devices beyond the 25th one dont seem to have picked up the custom name and description I intended for them:

When I go to the integration and click on the "Add entry", I see that there have been listed only the first 25 elements that I added on the configuration.yaml, but nothing beyond.
If it is not an issue on the integration, and it is actually me not being able to sort things out, could I get some assistance on how to proceed??
Thank you!

If I remember correctly, the custom list is limited to 25 entries.
(your configuration seems correct btw: the network scanner integrations stays in the UI and at the same time, you add in the configuration the lines in which you define de mac-addresses)

is there a way I can modify that limit of 25 ?

The limit of 25 is only in the popup screen, not in the actual code the applies the rules. To workaround this I edited the code on line 21 to hardcode my device list without limitation.
self.mac_mapping = self.parse_mac_mapping(""" FF:FF:FF:FF:FF:01;Nest WiFi;\n FF:FF:FF:FF:FF:02;Upstairs Nest Protect;\n """)

I have updated to support unlimited devices through configuration.yaml


Scouts stuck when there is no way to pass a room

Issue summary
Description of issue:

When scouts try to pass a room that is impassable due to obstacles, they would be stuck and hang around at the same place. Probably due to continuously recomputing the path, CPU usage skyrockets about 25% (shard 3) during the process and eventually consumes all the bucket. (I have to kill the scouts to prevent this manually)
Steps to reproduce:

The room replay: https://screeps.com/a/#!/history/shard3/E17S38?t=8174724
The issue occurs on the upper left corner of the room when the creep cannot move to the exit at the lower right of the room

Error message:

// Console output:

// Offending line:

Suggested fix (optional):

Maybe scout should stop recomputing the path and exit the room (or stand still) once it finds it cannot pass the room
Other information:

Overmind version: 0.5.2

Commit hash: {optional, but helpful: include the commit hash (if known)}

i had the same error, when i set the log level to 4 i got this:
[1:51:39 AM][shard3]DEBUG   12288587 Movement: incomplete path for [scout_4]! ([E7N19, 43, 4] ⟶ [E8N20, 25, 25])
[1:51:41 AM][shard3]DEBUG   12288588 Movement: incomplete path for [scout_0]! ([E7N19, 45, 5] ⟶ [E8N20, 25, 25])

I have bisected the problem to https://github.com/bencbartlett/Overmind/blob/5eca49a0d988a1f810a11b9c73d4d8961efca889/src/overlords/scouting/randomWalker.ts#L43
which it tries to travel to a room where no path could be found.
dirty hack fix this issue, but it breaks the assumption of 2D random walk, so it might not drift away from origin and have ability to reach anywhere
if (Game.map.isRoomAvailable(roomName) && scout.room.findExitTo(roomName)>0)

p.s. it also solve high cpu usage of scouts when no path is find


Added "eslint.useESLintClass": true causing eslint to fail to format ts files

你使用的是哪个版本的 ESLint。
最好能给我提供一个 GitHub 存储库，我可以克隆那个演示。否则我可能会开始使用不同的配置和版本，然后你会这样做。

v2.2.6

Which version of ESLint are you using.
It would be best if you could provide me with a GitHub repository, I can clone that demos that. Otherwise I might start using different configs and versions then you do.

v2.2.6

It is not about the extension. It is about you eslint npm config and all the configuration files around it, including the eslint plugins. That is why I asked you to please provide me with a GitHub repository I can clone since these configurations can be very sophisticated.

I created a new project and found that this configuration does not affect the normal use of eslint, so close this issue first, and will continue to pay attention to related issues in the future.


Additional docstrings changes for 0.9

Mostly updates BigchainDB class's docstrings to match current behaviour.
Contains some other small typo fixes / adjustments.

Current coverage is 98.40% (diff: 100%)

Merging #232 into master will not change coverage

@@             master       #232   diff @@
==========================================
  Files             9          9
  Lines           188        188
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            185        185
  Misses            3          3
  Partials          0          0

Powered by Codecov. Last update 776cb0e...a4fe3b8


Organize files, update package.json, and fix CommonJS bug

This repo is now a github fork from the author's repo.
The old repo was a copy/paste from MDN, and can still be found here:
https://github.com/yola/classlist-polyfill-old
This pull restores the file organization and the few changes found in the old repo.

:+1:


how to update pdfmake version?

Looks like pdfmake 0.1.25 is on NPM. Could you update this project to the latest version? The 0.1.18 does not seem to support configuration of borders in tables, which i need for a layout. Happy to submit a pull request if you can explain how to update it.

I've forked it and run the build with 0.1.25 but getting an error when using it in the project
ReferenceError: fs is not defined at Object.fontkit.openSync (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:365:14599) at Function.t.open (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:424:236) at o.font (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:463:525) at FontProvider.provideFont (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:506:734) at eval (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:533:1717) at Array.forEach (native) at measure (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:533:1247) at TextTools.buildInlines (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:533:2325) at DocMeasure.measureLeaf (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:500:2836) at eval (webpack:///../~/pdfmake-browserified/build/pdfmake-browserified.min.js?:500:1889)
my fork is here
https://github.com/jandetlefsen/pdfmake-browserified

@jandetlefsen: I see that you fixed the error in your fork. Great! :)
@xErik: could you please pull the changes from Jan and push a new release?
Or Jan, could you create a release from your fork?

Please ignore my last comment. I just saw that Jan did not fix the problem. Sorry, my fault.
So, is there any solution to this problem?

i've used this project, solves my problem https://github.com/EcoLogicAG/pdfmake-browserify

Yes, I noticed it, and already forked it to use 0.1.25 instead of 0.1.24. :)

Whoever has this fork should submit a PR.


Move external paramenters and resolved dependencies logic out of v2alpha3

Related with: #1065
This is a refactor in preparation for v2alpha4 which uses in its majority the same logic as v2alpha3.
Logic and tests moved to chains/formats/slsa/internal to be able to use it from different versions:

chains/formats/slsa/v2alpha3/internal/external_parameters/external_parameters.go => chains/formats/slsa/internal/external_parameters/external_parameters.go
chains/formats/slsa/v2alpha3/internal/resolved_dependencies/resolved_dependencies.go => chains/formats/slsa/internal/resolved_dependencies/resolved_dependencies.go

Current behavior shouldn't change with these changes.
This refactor will be use as base to enable v2alpha4 for TaskRuns, diff can be preview here
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

/ok-to-test

Thanks @renzodavid9! If there are still nits that need to be resolved, we can do that in the next PR.
Let's unblock your work.
/lgtm

/retest


wildcard_constraints not honored by glob_wildcards

Snakemake version
5.19.3+20.g0a55f335
Describe the bug
Wildcards values returned by glob_wildcards() include values that do not match the global wildcard_constraints.
Minimal example
Snakefile:
wildcard_constraints:
    filename=r'file_.+'

filenames, = glob_wildcards('{filename}.txt')

rule all:
    input: expand("{filename}.wc", filename=filenames)
    output: 'report.txt'
    shell: 'cat {input} > {output}'

rule word_count:
    input: '{filename}.txt'
    output: '{filename}.wc'
    shell: 'wc {input} > {output}'

Shell:
 $ touch file_1.txt file_2.txt file3.txt
 $ snakemake -j 1
 $ snakemake -j 1
Building DAG of jobs...
MissingInputException in line 6 of /mnt/data0/jmeppley/projects/snakemake.480/tests/test_glob_wildcard_constraints/Snakefile:
Missing input files for rule all:
file3.wc

Hi!
I just started looking into this. It seems that workflow.modifier.wildcard_constraints can be used to access the global wildcard constraints and snakemake.io.update_wildcard_constraints can be used to apply the constraint to the pattern. This means one could replace the wildcard pattern passed to glob_wildcards by snakemake.io.update_wildcard_constraints("/path/to/files/{somewildcard}", {}, workflow.modifier.wildcard_constraints), for example (this is a bit verbose but if your wildcard regex is complicated maybe it is a nice temporary solution).
What I do not know is how to access the workflow object from within glob_wildcards, which would enable doing the substitution inside glob_wildcards automatically. Or is there another way to access the global wildcard constraints @johanneskoester ?


Added tpp as c++ extension

Fixes #946

C++ sure does have a lot of extensions. :)
Thanks for the PR.


NameError: name 'string_types' is not defined

Hey,
You forgot to define 'string_types', I guess you meant to str (relate to python 3)

Thanks.
Solved in: https://github.com/pltrdy/rouge/pull/12.


Routing event support for request/response

Wanted to implement http basic auth checking as a filter and I'm not able to using routing events.
Request and response objects are not available to routing events and according to the comments in the routing events code, we should not return anything from a routing event function.

@erikhoward What part of the documentation are you referring to? Here's a quick sample:
  var director = require('director');
  var router = new director.http.Router();

  router.before('/', function (cb) {
    //
    // Do auth
    //
    return cb(new director.http.NotAuthorized('youre not allowed');
  });

  router.get('/this/is/behind/auth', function () {
    res.writeHead(200);
    res.end();
  });

Oh, also, request and response objects are available to routing events. e.g.
var director = require('director');
var router = new director.http.Router();
router.before('/', function (cb) {
//
// this.req and this.res are the objects
//
console.dir(this.req);
console.dir(this.res);
//
// Do auth
//
return cb(new director.http.NotAuthorized('youre not allowed');

});
router.get('/this/is/behind/auth', function () {
res.writeHead(200);
res.end();
});

Wasn't aware that req/res was available to me in the routing events. Thanks for clearing that up.

@erikhoward We are considering a route method signature more consistent with http.Server request handlers, but it hasn't been fully fleshed out yet.


[WEB] Can't run a transaction inside exclusively

Describe the bug
I'm trying out the new exclusively feature and it seems to be failing on web when running a drift transaction inside a exclusively block.
On a VM environment it works correctly.
await db.exclusively(() async {
  await db.transaction(() async {
    ...
  });
});

Thanks for the report! I've fixed this in drift version 2.19.1, you'll need a new worker though.


Bugs in Movemem_cpu.cpp?

Dear Cyten developers,
In dev-master branch, I somehow was not able to compile with -DUSE_OMP=ON.
From the error message, the vector shifter_old has not been properly initialized before use in some places.
After adding shifter_old as follows,
      std::vector<cytnx_int64> shifter_new(old_shape.size());
      std::vector<cytnx_int64> shifter_old(old_shape.size());

      for (cytnx_int64 i = old_shape.size() - 1; i >= 0; i--) {
        shifter_old[i] = accu_old;
        shifter_new[i] = accu_new;
        accu_old *= old_shape[i];
        accu_new *= newshape[i];
      }

I was able to get it to compile.
But, I don't really know what's going on..
If I'm doing something incorrect, let me know.
Best,
Jheng-Wei Li

It’ll be fixed as soon as possible, thanks!


Implemented Fix https://java.net/jira/browse/JAX_WS_COMMONS-137

As reported from https://java.net/jira/secure/ViewProfile.jspa?name=boerh1, I simply implemented his idea. Now, also paths with a "%2F" in them are working with a bindings.xml

This problem occurs e.g. with the new Jenkins Pipeline-Plugin, using the Multi-Branch-Project-Job-Style. If you use Atlassian Bitbucket und follow the guidance to use Branch-Names like feature/xyz you get into trouble. Is there any chance to take this fix into the next version? Is there any roadmap? Thanks in advance!

Unfortunately this PR contains some undesirable changes other than the fix. I've created #51  to track this fix request.


Update remaining RecordingInterfaces to use SpikeInterface backend

[ ] Intan
[ ] Blackrock
[ ] CED
[ ] Cell Explorer
[ ] Neuralynx
[ ] NeuroscopeSortingExtractor
[ ] OpenEphys
[ ] PhySortingInterface
[ ] SpikeGadgetsRecordingInterface

@h-mayorquin Please add the others, thanks! (Done!)

I edited your comment @CodyCBakerPhD  to add the rest.
Also I think missing in spikeinterface are:

[ ] Axona

And that's it. Maybe for good reasons.
And a bunch of extractors more to add as needed.

I edited your comment @CodyCBakerPhD to add the rest.

Thanks!
We'll have to decide on what to do with Axona as a team - maybe bring it up in the next team meeting if you can remember?


We need a gs-keyboard element

We keep building custom keyboards for people who are using a mobile platform with a barcode scanner. When you have a barcode scanner attached, the built-in keyboard stops appearing.
There would be two elements: the gs-keyboard and the gs-key. The keyboard gs-keyboard will listen for clicks on it's child gs-key elements. What this does is allow us to define the structure and placing of the keys within the keyboard, but we don't have to do any javascript.
The gs-keyboard will update the document.activeElement, which is the currently focused element. gs-keyboard is going to have to trigger key events on the element whose value it updates.
Numpad example code:
<gs-keyboard>

            <gs-key key="7">7</gs-key>
            <gs-key key="8">8</gs-key>
            <gs-key key="9">9</gs-key>

            <gs-key key="4">4</gs-key>
            <gs-key key="5">5</gs-key>
            <gs-key key="6">6</gs-key>

            <gs-key key="1">1</gs-key>
            <gs-key key="2">2</gs-key>
            <gs-key key="3">3</gs-key>

            <gs-key key="0">0</gs-key>
            <gs-key key="backspace">Delete</gs-key>

</gs-keyboard>

A few features:

We'll have a format attribute for the gs-keyboard, which will just be a shortcut for some standard keyboards (numpad, qwerty, arrows, qwerty,arrows,numpad).
If a gs-key doesn't have a key attribute defined, we'll just use the text of the key.
You'll be able to force a target for the keyboard by using a selector in a target attribute on the gs-keyboard.

A few special keys:

backspace will delete the character behind the cursor.
delete/del will delete the character in front of the cursor.
left-arrow
right-arrow
down-arrow
up-arrow
home will move the cursor to the beginning of the line (moves to the very beginning if you're already at the start of a line).
end will move the cursor to the end of the line (moves to the very end if you're already at the end of a line).
next-control will move the focus. Usually called tab on your keyboard.
previous-control will move the focus. Usually shift-tab on your keyboard.
tab will add a tab character.
tab-spaces will add up to four spaces (depending on how far you're from a position that's divisible by four on the line).

A few of things to figure out:

How are we going to handle a shift, control, option and command keys?
What's the styling going to be?

[Shift] should be implemented, but the other modifier keys should not be implemented and should never be shown. You could spend a lot of time implementing crazy stuff like [ctrl]-[c], but I advise against it. An entry keyboard is not an operating system keyboard and does not need the same features. If someone has a physical keyboard, they won't see this keyboard, so you don't need these keys.
Think strictly about implementing the [shift] key as it works on other platforms and don't worry about anything else.
Regarding styling, keep it plain so people can set their own CSS without trouble.

Regarding special keys:
backspace is typically provided
delete is typically not provided but i'm not against it.
arrow keys are not necessary. no mobile platform supplies these because they have touch.
home-end are not necessary, mobile platforms have touch.
tab is not necessary, touch
tab-spaces... i don't think you need it but if you make gs-key allow for input of an arbitrary string and not just one letter then anyone can do stuff like this. It's possible someone may use it.

This element has been added and documented.


PreprocessorSymbolNames are missing for .NET 5+ projects

As mentioned on this page https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/preprocessor-directives I can use new symobols in preprocessor if like NETCOREAPP3_1_OR_GREATER which should be added if the targetframework is greater than 3.1.
If I use this in a project and build with Visual Studio the code is included. But when I analyse that same project with buildalyzer the preprocessor symbols are not added. Only the default symbols.
Reproduction
Reference .NET 6 and add to a file
    public int Bit()
    {
#if NETCOREAPP3_1_OR_GREATER
        return 1 ^ 2;
#endif
    }

This will compile in VS and dotnet build, but not in buildalyzer
Context
I'm not sure yet why they are missing in Buildalyzer. They should be added on SDK style projects. But there is a message saying: For traditional, non-SDK-style projects, you have to manually configure the conditional compilation symbols for the different target frameworks in Visual Studio via the project's properties pages.
So it might just be the case some SDK assembly is not loaded? Or if we have bad luck this is custom logic in MSBuild thats used in VS and the dotnet CLI.
Either way I can't compile the project that builalyzer returns... I also see no workaround as the symbols are added at parse time and can't be changed once the syntax tree has been parsed.
Issue was found in https://github.com/stryker-mutator/stryker-net/issues/1828

I think I found where the preprocessor symbols are added in MSbuild: https://github.com/dotnet/sdk/blob/997376c6bc798af22633a8eb36074a0acec8a200/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.BeforeCommon.targets

I have found the issue. We create our syntax trees with ParseOptions based on what Buildalyzer tells us. I see Builalyzer does the same internally here https://github.com/daveaglick/Buildalyzer/blob/b2bd62140b2073d8a1ef4b30cecd1a2ce34038b5/src/Buildalyzer.Workspaces/AnalyzerResultExtensions.cs#L174
The issue is that the DefineConstants property is lacking the *_OR_GREATER constants. But I have found a property that contains all constants we need. It's private so we can't access it.

I'll try to create a PR that makes this public and will also fix the internal usage so Buildalyzer support the new preprocessor symbols from now on.


specs: add rightPadder

Fixes: https://github.com/moov-io/iso8583/issues/155

Codecov Report

Merging #156 (2a7815c) into master (605da49) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #156      +/-   ##
==========================================
- Coverage   70.54%   70.37%   -0.18%
==========================================
  Files          37       37
  Lines        1633     1637       +4
==========================================
  Hits         1152     1152
- Misses        308      312       +4
  Partials      173      173

Impacted Files
Coverage Δ

specs/builder.go
68.96% <0.00%> (-1.63%)
: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 605da49...2a7815c. Read the comment docs.


Solution to issue #2

I have added code to solve the issue #2 in the code and documented 2 methods in the primary file

Looks like this issue has been fixed. Thank you!


Doesn't support keyfile as set in ssh_options

So I just converted a repository that I use capistrano-rails-console on from a staging to production situation. As part of this we're using a hard linked ssh key, as opposed to inheriting one from the ssh-agent/user keys.
We just spent a couple of hours trying to figure out what was going on, but basically the gem is not accepting whatever is set under:
ssh_options, {..., keys: <code to produce keypath here>, ...}
What actually seems to be happening is that if the local capistrano users keychain can't get into a system (such as in our test) it tries to use this setting, but for some reason only trys to send whatever is the first character of the path as a key.  So if you have:
keys: File.expand_path('~')<<"/.ssh/my_special_key"
Returns: Load key "/": Is a directory
But if you send: keys: 'config/my_special_key then you get:
Warning: Identity file c not accessible: No such file or directory.
And if you send keys: './config/my_special_key
You get: Load key ".": bad permissions
In the first example it's reading the "/" in the Dir.file_expand and trying to use it as the key.  In the second it's reading the 'c' in the string, where the line begins 'config' and in the third it's reading the first '.' in the string as the argument and calling bad permissions on it....
Very confusing.  I took a look at the code for the gem but it seems that in version 1.0.2 a lot of the code was reworked, and I'm thinking perhaps the author has some insight as to what's happening here.
I have to say this is also pretty mysterious, because if you're using a config/deploy/.rb configuration that's also your users ssh key, it appears as if it's using the config to work... but it's not.
Hope this makes sense - truly baffling, any ideas?

I guess it is a problem in conjunction with the used sshkit version. And it really sound very weird and confusing.
I was planning to extract the interactive part of the gem to a more generic gem that can be used for all kinds of things. That's why the code from master was very different from the released version.
Can you please try again using the latest version (2.0.0) and use the latest revision of sshkit-interactive which is now used for the ssh connection:
gem 'sshkit-interactive', git: 'https://github.com/afeld/sshkit-interactive.git'

I tested it by renaming my default key according to your example and it worked
set :ssh_options, { keys: [File.expand_path('~')<<"/.ssh/my_special_key"] }


Shiny Dataset Download

After Download the shiny datasets through one-drive link, I can't decompress the zip file.
To extract the file, I repaired the compressed file through following command.
zip -FF my_zip --out my_zip_ver2.zip
But, the file still have some problems after repairing.
When I decompress the repaired file, there is the log file which notice the error logs.

many files does not exist.

1062 files does not exist.
Is there any problem with the uploaded file?
Or how to download and extract the complete dataset?
I tried download on OSX and Ubuntu, but both failed.
Do I have to use window to download the file?

Maybe the zip file is too large. you can try to download each subfolder instead or download it via the google drive link.

@pureexe
Thanks a lot!!
Solved.


Allow function type annotations params to have names

The following doesn't parse properly:
declare let mag: (p: Point) => number

I think this should probably be required since TypeScript requires function params to be named in type annotations.

This was implemented a while ago when we switched to using tree-sitter for the parser.


Makefile: add riscv32imac target in setup

The hifive1 board needs the 'a' atomic operations as well. Install that
target during setup.
Signed-off-by: Dylan Reid<EMAIL_ADDRESS>

The code of the setup script part was intended to be used by travis CI, not by library testers.
I don't know if it makes sense to install every possible target in the setup script. Any opinions from others on that? Maybe the setup target should be renamed to setup-ci?

Ah, understood. Would it be better to change the quick start readme to explain the boards not covered by CI need to manually install the toolchain? It currently instructs users to run make setup.

I think it is better to just keep it simple and have make setup work for all boards.

bors r+


Linux OS tuning for InfluxDB

This is an epic for different system tuning parameters we have noticed in real installations, and making InfluxDB easier to configure correctly.
Mitigations for bad parameters could include documentation, or InfluxDB doing a self test and logging / stopping / failing to start when misconfigured.
Mitigations for Enterprise and OSS 2 should be considered.

[ ] vm.max_map_count - if the number of active mmap calls is too high, InfluxDB fails to open shards and can even mark database files as corrupt / unqueryable (.tsm.bad files will be generated in the data directory). See https://unix.stackexchange.com/questions/640233/vm-max-map-count-for-a-running-process and https://www.elastic.co/guide/en/elasticsearch/reference/current/vm-max-map-count.html for prior art
[ ] vm.min_free_kbytes - based on internal experimentation, we recommend setting vm.min_free_kbytes to 4.5% of total RAM on machines with more than 50GB of RAM. Compare https://docs.aerospike.com/operations/install/linux/bestpractices . Setting too high can cause OS instability (see https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7/html/performance_tuning_guide/sect-red_hat_enterprise_linux-performance_tuning_guide-configuration_tools-configuring_system_memory_capacity). Setting too low on machines with fast disks and large amounts of RAM appears to cause the OS to trigger the OOM killer even when InfluxDB is using much less than the total available RAM.
[ ] ulimit -a - TSI indexes use many open files (TODO: get a good estimate for open files per shard with TSI indexes). So databases with very large numbers of shards may refuse to start when converted from inmem indexing to TSI indexing. Note that InfluxDB OSS 2 only supports TSI. there are limits on files, number of mmaps, and virtual memory sizes - most of these should be set quite high for InfluxDB.

Places to check
cat /proc/{influxd-pid}/limits

cat /proc/sys/fs/nr_open

Some notes:
If they need lower default limits, prlimit can be used to increase the limit of a specific pid.
prlimit -n500000 -p your-pid-here
fs.file-max should be increased to around half a million or so.

complete summary of things to check when mmap goes bad:
First, check free -h to see if influxd is actually using too much memory.
mmap is used to both map a file into memory (in order to query it), and to allocate 'anonymous' memory (normal memory not backed by a file). InfluxDB does both of these extensively.
For a full list of errors that the mmap system call on linux can experience, see: https://man7.org/linux/man-pages/man2/mmap.2.html#ERRORS
Many of these are applicable only to file-backed mappings.
For non-file-backed mappings, generally the map will happen in the go runtime, via sysAlloc: https://go.dev/src/runtime/mem_linux.go#:~:text=func-,sysAlloc,-(n%20uintptr%2C%20sysStat . This will exit for two common errors, _EACCESS (unlikely) and _EAGAIN (indicates one should look at ulimit -l).
Other errors would result in a nil returned from sysAlloc, often caught by the go runtime with code similar to:
result = (*gcBitsArena)(sysAlloc(gcBitsChunkBytes, &memstats.gcMiscSys))
if result == nil {
	throw("runtime: cannot allocate memory")
}

Other errors could be caused by:
ENFILE:
Check cat /proc/sys/fs/file-nr - this gives 3 numbers. The first is the total file handles on the system, and the last is the maximum. If close to a maximum, consider increasing the limit.
Check cat /proc/$(pidof influxd)/limits for file handle limits, and ls /proc/$(pidof influxd)/fd | wc -l for the total number of file handles used by influxd. If close to a maximum, consider increasing the limit.
ENOMEM:
Check cat /proc/sys/vm/max_map_count for the per-process limit on memory maps, and cat /proc/$(pidof influxd)/maps | wc -l for the total number of maps used by influxd. Increasing this is fairly safe, SUSE has some docs on side effects - basically 128 bytes per memory map, and some extra lookups when resolving a memory address. There is no performance penalty unless the maps are actually used.
EINVAL:
This would indicate a bug where an invalid size or type of memory region was requested.

https://github.com/influxdata/influxdb/issues/23439 - specific issue for adding warnings about these to the logs.

We ran into the problem of  vm.max_map_count being too low for our dataset.
The primary server had to be restarted due to a memory leak in a secondary application which caused the machine to start killing processes.
This would also mark the TSM files as bad, even though this was not the case. If we renamed them from *.tsm.bad back to *.tsm and opened them in a second influx process, the data would show.
We got errors in the logs while restarting the service indicating: failed cannot allocate memory.
The influx process would crash during startup while reading in our 68TB database. It would crash around 63.8TB of virtual memory being used. Interestingly enough, our back up server which was running at the same time did not encounter this problem during run and is using more than 67TB of virtual memory.
We worked around the problem by increasing the vm.max_map_count limit from 65530 to 262144. The process now starts normally. We suspect this will have increased the limit to 256TB of max virtual memory per process.
We started to discover where the issue was coming from through this issue: https://github.com/prometheus/prometheus/issues/4392

Not as needed as before.  These problems are mostly addressed.


fix: Fix old flutter version for CI/CD

We always used flutter 2.5.0 eventhough 2.8.1 is latest in stable

Post-Submit is currently failing because of that


IsList

Also, all of these types should be given IsList  instances.

There are two ways to write the IsList instance. One would be to naively fold over the elements, turning them into singleton sets and unioning them as you go. Alternatively, you could load them into a buffer in O(n) time, do an in-place stable sort like mergesort in O(n log n) time, and then do something like what the unix tool uniq does, removing adjacent duplicates of a sorted list in O(n) times. This would bring the total complexity down from O(n^2) down to O(n log n). Also, you can do something similar with mconcat, avoiding the default definition in favor of something like this (it will be a little messier though since you want to take advantage of the fact that your working with sets that have already been sorted).

I've got a repo where I implemented a concurrent variant of mergesort some time ago. You could dump the concurrency and switch to PrimArray instead of ByteArray, and this would work.

WE SHOULD KEEP THE CONCURRENCY. GOTTA GO FAST. CC @mckeankylej @taktoa

What about PrimArrays make it so you cant use concurrency?

IsList seems done for everything except Diet-Sets/Maps now


Cannot import ApifyWrapper

Hello! I am attempting to run the snippet of code linked here locally on my
Macbook.
Specifically, I'm attempting to execute the import statement:
from langchain.utilities import ApifyWrapper

I am running this within a conda environment, where the Python version is 3.10.9 and the langchain version is 0.0.134. I have double checked these settings, but I keep getting the above error. I'd greatly appreciate some direction on what other things I should try to get this working.

Nevermind, resolved this by reinstalling! Thanks for creating langchain, really enjoying exploring this package.


fix:修复了set_schema当中会因为多次调用open而添加重复数据的问题

What problem were solved in this pull request?
Issue Number: close #xxx
Problem:NLJ里面会遍历多次右表，就会调用很多次open，在open当中就会调用set_schema，然后join的时候就会添加大量重复的field和value的tuple，后续的话会被select的字段给过滤掉，不影响正确性，但是进行了冗余存储
What is changed and how it works?
在set_schema当中，每次先将原本的数据clear掉
Other information

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


Invalid validation for multiple ports

I can't reproduce this.
Which UI version do you use?
Could you show me how to reproduce this?

Create an app with ports "0". Than afterwards edit the app and change it to something like "1xxxx,0".

I think that it is a service level bug.The API call returns
{"message":"Bean is not valid","errors":[{"attribute":"ports","error":"Elements must be unique"}]}


709 improve charge accuracy

price can be fully compensated by over-restoring

@zxcat yes, and we discussed it a few weeks ago in Slack. there are pros and cons. the behavior is quite counterintuitive, but due to the fact that the charge can be fully compensated, we can only store the values for active users (zero values are not stored in the table)


QDM 4.3 cannot resolve "birth datetime"

I have the following cql
using QDM version '4.3'

context Patient

define "test1":
        AgeInYearsAt(@2017-01-01T00:00:00.000)

I attempted to compile it and got the following:
================================================================================
TRANSLATE ./Test.cql
Translation failed due to errors:
Error:[6:2, 6:39] Member birth datetime not found for type QDM.Patient.

I then fixed this by modifying line 5 of Src/java/qdm/src/main/resources/gov/healthit/qdm/qdm-modelinfo-4.3.xml
from
        <ns4:element name="birthDatetime" type="System.DateTime"/>
to
        <ns4:element name="birth datetime" type="System.DateTime"/>
then rebuilt the compiler and ran it again and it was successful.
I'm not sure if the solution is just to submit this as a PR so I posted here just to make sure it was brought to your attention.

Hi @Caskman , yes, it looks like this is just a bug in the modelinfo for QDM 4.3. If you can submit a pull request, that would be fantastic.


Create CONTRIBUTING.md

Hello,
I create the file "CONTRIBUTING.md", it's the first version and it will be improve in the future.
(Issue #255)
Thanks.

Looks like a pretty good start to me, any additional thoughts @garyttierney @ryleykimmel?

@Major- have some changes I want to add. Going to pull this branch and make them.

wow i really like this thanks :)

Closed, duplicate of #269


Handle API methods with empty response types

Render empty response types as java.lang.Void in places like UnaryCallable<RequestT, Void>.
Create a new TypeModel, EmptyTypeModel as the in-memory placeholder for the null response types.
Minor fixes:

render Java float and double initial values correctly
fix duplicate naming of getters/setters for fields that have the same underlying reference schema

This PR can be reviewed on its own, but it would be helpful to have https://github.com/googleapis/toolkit/pull/1890 checked in first. The other PR contains test resource changes, and this PR would have more helpful baseline changes once the test resources get updated.

Codecov Report

Merging #1892 into master will decrease coverage by 0.15%.
The diff coverage is 27.27%.

@@             Coverage Diff              @@
##             master    #1892      +/-   ##
============================================
- Coverage     85.22%   85.06%   -0.16%
  Complexity     4744     4744
============================================
  Files           439      440       +1
  Lines         18800    18834      +34
  Branches       2001     2006       +5
============================================
- Hits          16023    16022       -1
- Misses         2073     2102      +29
- Partials        704      710       +6

Impacted Files
Coverage Δ
Complexity Δ

.../configgen/transformer/DiscoConfigTransformer.java
98.82% <ø> (ø)
16 <0> (ø)
:arrow_down:

...er/java/JavaDiscoGapicSchemaToViewTransformer.java
95.87% <ø> (ø)
19 <0> (ø)
:arrow_down:

.../google/api/codegen/discogapic/EmptyTypeModel.java
0% <0%> (ø)
0 <0> (?)

.../transformer/java/JavaSchemaTypeNameConverter.java
51.75% <0%> (-5.39%)
28 <10> (ø)

...oogle/api/codegen/config/DiscoveryMethodModel.java
81.05% <0%> (ø)
37 <1> (ø)
:arrow_down:

...degen/discogapic/transformer/DiscoGapicParser.java
91.93% <100%> (-0.26%)
23 <0> (ø)

.../com/google/api/codegen/config/DiscoveryField.java
50% <100%> (-0.8%)
46 <2> (ø)

...i/codegen/transformer/SchemaTypeNameConverter.java
64.7% <66.66%> (-1.97%)
5 <0> (ø)

...oogle/api/codegen/transformer/SchemaTypeTable.java
72.05% <77.77%> (-0.96%)
28 <1> (ø)

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 71f4579...c9cad64. Read the comment docs.

PTAL @garrettjonesgoogle

Removed some instanceof checks. PTAL


Owl Carousel Border

There should not be any gray border around the carousel. See the mockup. Also there shouldn't be a white space above it.

I have removed the extra space. But there is still a thin grey line above the carousel which is due to a class on the iframe. Since iframe is a part of sdk,we cannot make changes to that.
Also, the border below and above the carousel is due to the height of the image. Lets say I have 3 images in my carousel and 2 have smaller height and one have larger height. Then carousel will have to take height of the longest image. Or else we would have to force user to upload images with same height.


SILGen: do not emit hop_to_executor in some objc thunks

This is sort of a reimplementation of https://github.com/apple/swift/commit/e39dca4dbf26526e03fc91e5ddb7ef08da89bd24
An @objc async thunk does not need to perform a
hop_to_executor if the native method the thunk is
wrapping is async, since the native method will do
its own hop.
This redundant hop was causing a crash in SILGen
with async actor-instance isolated @objc methods
declared in an actor, because SILGen was not
prepared to add the hop to  to self within
the @objc thunk.
resolves rdar://80130628

@swift-ci please smoke test

@swift-ci please smoke test macOS

@swift-ci please smoke test and merge


Added Help Link

Motivation and Context

Provides easy access to the wiki from MarkUs.
Your Changes

Description:
Added help link to the main MarkUs dropdown.

Type of change (select all that apply):

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

Testing

Tested the change on the web interface.
Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have verified that the pre-commit.ci checks have passed.
[x] I have verified that the CI tests have passed.
[x] I have reviewed the test coverage changes reported on Coveralls.
[x] I have updated the Changelog.md file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 91.852%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
37920

Relevant Lines:
40743

💛  - Coveralls


Make public on npm

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Not creating/updating a field in firestore document when path is long

Describe the bug
I am trying to create/update myTimestamp field in inouts_id2 document but it's showing as failed without any error message only when document path is long. Otherwise when document path is short, it's working fine.
To Reproduce

String documentPath = "school inout/west bengal/howrah/711101/wb0001/2020-2021/students/inouts_id2";
Firebase Client v2.3.5

Token info: type = id token, status = on request
Token info: type = id token, status = ready
Update a document...
FAILED
REASON:
------------------------------------

String documentPath = "aa/bb";
Firebase Client v2.3.5

Token info: type = id token, status = on request
Token info: type = id token, status = ready
Update a document...
PASSED
JSON DATA:
------------------------------------

ok
{
  "name": "projects/electronics-f66ec/databases/(default)/documents/aa/bb",
  "fields": {
    "myTimestamp": {
      "timestampValue": "2021-07-04T08:56:23Z"
    }
  },
  "createTime": "2021-07-03T14:22:13.620444Z",
  "updateTime": "2021-07-04T09:09:07.522699Z"
}

Expected behavior
String documentPath = "school inout/west bengal/howrah/711101/wb0001/2020-2021/students/inouts_id2";
Firebase Client v2.3.5

Token info: type = id token, status = on request
Token info: type = id token, status = ready
Update a document...
PASSED
JSON DATA:
------------------------------------

ok
{
  "name": "projects/electronics-f66ec/databases/(default)/documents/school inout/west bengal/howrah/711101/wb0001/2020-2021/students/inouts_id2",
  "fields": {
    "myTimestamp": {
      "timestampValue": "2021-07-04T08:56:23Z"
    }
  },
  "createTime": "2021-07-03T14:22:13.620444Z",
  "updateTime": "2021-07-04T09:09:07.522699Z"
}

IDE and its version:

Arduino
Version 1.8.15

ESP8266 Arduino Core SDK version

Version 3.0.1

Additional context
Below code snippet for the reference:
    String content;
    FirebaseJson js;

    String documentPath = "school inout/west bengal/howrah/711101/wb0001/2020-2021/students/inouts_id2";

    js.set("fields/myTimestamp/timestampValue", "2021-07-04T14:41:23+05:30");
    js.toString(content);

    Serial.println("Update a document... ");

    /** if updateMask contains the field name that exists in the remote document and
       this field name does not exist in the document (content), that field will be delete from remote document
    */

    if (Firebase.Firestore.patchDocument(&fbdo, FIREBASE_PROJECT_ID, "" /* databaseId can be (default) or empty */, documentPath.c_str(), content.c_str(), "myTimestamp" /* updateMask */))
    {
      Serial.println("PASSED");
      Serial.println("JSON DATA: ");
      Serial.println("------------------------------------");
      Serial.println();
      Serial.printf("ok\n%s\n\n", fbdo.payload().c_str());
    }
    else
    {
      Serial.println("FAILED");
      Serial.println("REASON: " + fbdo.errorReason());
      Serial.println("------------------------------------");
      Serial.println();
    }

It's not bug.
You are not show which device you use but I guess ESP8266.
You should increase the BearSSL buffer for rx and tx, like this
fbdo.setBSSLBufferSize(2048, 1024); //2048 bytes for rx and 1024 bytes for tx.
The default rx and tx buffers are  1024 and 512 by default and may be too small for large payload.

Yes, the device is ESP8266.
Even after increasing BearSSL buffer size as suggested. Still, it's getting failed without error message.

You document path contains spaces, which should be avoid.
The space should be encoded to %20 string.
Library doesn't do extra works for path encoding for you and you should do it yourself.


Update config-infra-prod.yml

Added sub-folder to make sure tf-ado-deploy-infra.yml runs properly.
Adapted to folder structure in config-infra-prod.yml.
Base Image in Dockerfile (cv/aml-cli-v2/data-science/environment/Dockerfile) must be changed to "golden image" (mcr.microsoft.com/mirror/nvcr/nvidia/pytorch:22.04-py3) in order to avoid container security violations and errors in ADO pipelines.
PR into Azure/mlops-v2
Checklist
I have:

[x] read and followed the contributing guidelines

Changes

Updated folder path

fixes #

@yvonnebarthp this patch isn't necessary because of the way the sparse checkout and moving of directories happens in this initiating bash script: https://github.com/Azure/mlops-v2/blob/main/sparse_checkout.sh
We move the terraform folder contents into the "infrastructure" folder directly and therefore there is no subfolder in the sparse checkout initiated repo.


support greater than a second resolution for the rate limiter

if i specify .filter(rateLimiter.apply(TupleBuilder.tuple().of(REPLENISH_RATE_KEY, "1", BURST_CAPACITY_KEY, "2"))) then that implies it supports 1 request per second. If i want to do less than a single request per second, it won't work as that REPLENISH_RATE_KEY eventually gets stored in an int. So, 0.1 - which should mean one request every 10 seconds - doesn't work.

This is an implementation detail of the redis lua script. Not much I can do.


Transaction scheduler should use CheckTx

As of #1555 the runtime exposes a CheckTx API for verifying incoming transactions but this is currently unused. The transaction scheduler should use the API to reject incoming transactions early instead of submitting them through the processing pipeline.
The runtime must ensure that any such checks are cheap enough that they can be performed quickly and not become a bottleneck of the whole pipeline.
Estimated cost: 1 sprint

Performing CheckTx should be a node configuration parameter so it can be disabled if needed.


Slow Undeployment

I am trying to deploy war file with ShrinkWrap in managed/remote Wildfly 8.2. So far, everything is working just fine, but undeployment after tests is very slow like around 45 - 60 seconds. In addition, the war file is quite small, manual deploy/undeploy through Administrator console takes couple seconds only.
    @Deployment(name = "core")
    public static WebArchive createDeployment() {
        File file = Maven.resolver().resolve("org.test:core:war:1.0.0").withoutTransitivity().asSingleFile();
        ZipImporter zipImporter = ShrinkWrap.create(ZipImporter.class, "core.war");
        WebArchive webArchive = zipImporter.importFrom(file).as(WebArchive.class);
        return webArchive.addAsResource("persistence.xml", "META-INF/persistence.xml");
    }

Here are logs;
19:03:43,390 INFO  [org.jboss.weld.deployer] (MSC service thread 1-3) JBAS016009: Stopping weld service for deployment core.war
19:03:43,404 INFO  [org.jboss.as.jpa] (ServerService Thread Pool -- 13) JBAS011410: Stopping Persistence Unit (phase 1 of 2) Service 'core.war#default'
19:03:43,407 INFO  [org.jboss.as.clustering.infinispan] (MSC service thread 1-15) JBAS010282: Stopped local-query cache from hibernate container
**19:03:43,409** INFO  [org.jboss.as.clustering.infinispan] (MSC service thread 1-15) JBAS010282: Stopped pending-puts cache from hibernate container
**19:04:27,028** INFO  [org.jboss.as.server.deployment] (MSC service thread 1-11) JBAS015877: Stopped deployment core.war (runtime-name: core.war) in 43722ms
19:04:27,046 INFO  [org.jboss.as.repository] (management-handler-thread - 3) JBAS014901: Content removed from location C:\wildfly\standalone\data\content\ef\4ae2aa3fec52551884fb440d6d1927294203f8\content
19:04:27,046 INFO  [org.jboss.as.server] (management-handler-thread - 3) JBAS018558: Undeployed "core.war" (runtime-name: "core.war")

See highlighted time difference above.
pom.xml
    <dependency>
        <groupId>org.wildfly.arquillian</groupId>
        <artifactId>wildfly-arquillian-container-remote</artifactId>
        <version>1.0.1.Final</version>
        <scope>test</scope>
    </dependency>

Well that has noting to do with wildfly arquillian adapter but with your deployment.
Looks like cache is shutting down for most of the time of deployment.

I see your point, but it should have some sort of connection with the adapter because undeploying with CLI like jboss-cli.sh --connect --command="undeploy core.war" is instant. In addition, slowness starts right after it stops pending-puts cache;
Stopped deployment core.war (runtime-name: core.war) in 43722ms


Request use of botocore.config for to handle API throttling when archiving findings in SecurityHub

Steps to Reproduce
As a security engineer running prowler with the below command:
prowler AWS --log-level ERROR -M csv html json-asff --ignore-exit-code-3 --security-hub --status FAIL --log-file /foobar/logs.txt --output-directory /foobar
I expect prowler to archive the findings.
Expected behavior
I expect prowler to archive the findings in securityhub that were no longer detected in the latest scan.
Actual Result with Screenshots or Logs
2024-09-11 18:53:36,513 [File: security_hub.py:247]     [Module: security_hub]   ERROR: Client error - - [227]:An error occurred (TooManyRequestsException) when calling the GetFindings operation: Too Many Requests in region us-gov-west-1
Sending findings to AWS Security Hub, please wait...

Archiving previous findings in AWS Security Hub, please wait...
0 findings archived in AWS Security Hub!
How did you install Prowler?
From pip package (pip install prowler)
Environment Resource
Gitlab runner docker container.
OS used
Ubuntu:latest
Prowler version
4.3.5
Pip version
24.0
Context
Request that prowler make use of 'botocore.config' to handle backoff and retries with the SecurityHub API.'
I believe this change would need to occur at the following location:
https://github.com/prowler-cloud/prowler/blob/master/prowler/providers/aws/lib/security_hub/security_hub.py#L140-L142
from botocore.config import Config

# Custom configuration for retries and backoff
config = Config(
    retries={
        'max_attempts': 10,  # Max number of retry attempts
        'mode': 'adaptive'  # or 'standard' for standard exponential backoff
    }
)

# Initialize the client with custom config
enabled_regions[region] = self._session.client(
    "securityhub", region_name=region, config=config
)

Hey @mlmerchant, thanks for reporting this.
I've addressed your suggestion internally and we will take care of it as soon as we can. I will leave a note so the team can update with a message here when we fix it.

Hi @mlmerchant, have you tried using the flag --aws-retries-max-attempts? You have more information in our docs here. Let us know if that works for you, thanks!

I will close this issue due to lack of activity, feel free to reopen it if you are still encountering this problem, thanks!


Have release.sh handle updating CHANGELOG when only some fixes are being released

There's an assumption in release.sh that the process will work cleanly from beginning to end.
This is true so long as

CHANGELOG has changed while the script is being run (probable)
You are releasing from a commit that is the most up to date CHANGELOG

If the latter isn't the case, you will run into issues when merging the CHANGELOG to updates to master and adding a new unreleased section.
My idea to handle this would be to provide a way (perhaps through a prompt) to not automatically do that last step and instead make it optional.  If we allow for that part to be done manually, we could add a section to the RELEASE_PROCESS.md doc for how to handle manually should the need arise.
@Theodus thoughts?

@SeanTAllen are you suggesting that commit changelog and push to master section should become optional?

@Theodus yes. it would allow for smooth handling of the edge case will arise at some point.

@SeanTAllen Ok, I'll get on that.

I wasn't suggesting you do it @Theodus. Just wanted to discuss with you. Thank you though for taking it on.

Closed by #1675


Connectors not able to switch language whose name consists more than one word (e.g. Chinese Traditional)

We can switch the bot's output language according to http://docs.botframework.com/connector/bot-options/#navtitle

User: I would like to speak spanish
Bot: Voy a usar el español. Déjeme saber si usted quiere cambiar hacia atrás.

While this works with languages like Spanish, languages that consists of more than one word like Chinese Tradition, Serbian Cyrillic doesn't not work. I suspect this is a simple string parsing issue but since the code for this is not open I can't really contribute.

User: I would like to speak Chinese Traditional
Bot: I can use any of the following languages:
Arabic

Bosnian
Bulgarian
Catalan
Chinese Simplified
Chinese Traditional
Croatian
Czech
Danish
Dutch
English
Estonian
Finnish
French
German
Greek
Haitian Creole
Hebrew
Hindi
Hmong Daw
Hungarian
Indonesian
Italian
Japanese
Kiswahili
Korean
Klingon
Latvian
Lithuanian
Malay
Maltese
Norwegian
Persian
Polish
Portuguese
Querétaro Otomi
Romanian
Russian
Serbian Cyrillic
Serbian Latin
Slovak
Slovenian
Spanish
Swedish
Thai
Turkish
Ukrainian
Urdu
Vietnamese
Welsh
Yucatec Maya

This is now fixed.


more details for error code 4xx

more details and resolutions added for error 4xx

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.

This PR was created in public repo by mistake - should be closed


[Breaking change]: Stop skipping configuration binding a dictionary with a key that has empty value

Description
In previous versions, when binding configuration to a dictionary type, any keys listed in the configuration without corresponding values would be skipped, resulting in their non-creation within the dictionary. However, a recent breaking change alters this behavior, whereby such keys will no longer be skipped but instead will be automatically created with their default values. This change ensures that all keys listed in the configuration will be present within the dictionary, streamlining the process and avoiding potential issues with missing keys. Developers should be aware of this update and adjust their code accordingly to accommodate the new behavior.
Version
.NET 8 Preview 5
Previous behavior
Consider the following example:
Configuration
var json = @"{
    ""Queues"": {
        ""q1"": {
            ""V"": 1
        },
        ""q2"": {
            ""V"": 2
        },
        ""q3"": {
        }
    }
}";

Code
public class Q
{
    public Dictionary<string, QueueValue> Queues { get; set; } = new();
}

public class QueueValue
{
    public int V { get; set; }
}

var configuration = new ConfigurationBuilder()
    .AddJsonStream(StringToStream(json))
    .Build();

Q options = new Q();
configuration.Bind(options);
foreach (var kvp in options.Queues)
{
    Console.WriteLine($"{kvp.Key}: {kvp.Value.V}");
}

Output
q1: 1
q2: 2

Please take note that the third key, q3, is currently being skipped, resulting in no corresponding dictionary entry being created for it.
New behavior
With the new behavior, the resulting output will now include the q3 key, which was previously skipped. This key will be created and initialized with the default value within the generated dictionary. This change ensures that all listed keys are accounted for, eliminating any potential omissions, and promoting consistency in the dictionary structure.
q1: 1
q2: 2
q3: 0

Type of breaking change

[ ] Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[ ] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code may require source changes to compile successfully.
[X] Behavioral change: Existing binaries may behave differently at run time.

Reason for change
This change ensures that all keys listed in the configuration will be present within the dictionary, streamlining the process and avoiding potential issues with missing keys. Developers should be aware of this update and adjust their code accordingly to accommodate the new behavior. The users request this new behavior. https://github.com/dotnet/runtime/issues/66229
Recommended action
Users should be aware of the new behavior, which involves the automatic population of dictionary entries with empty values listed in the configuration during the binding process. This change necessitates that users verify and adapt their application logic to accommodate the presence of these newly created entries.  If the new behavior is undesirable, the workaround involves removing the empty value entries from the configuration. By eliminating these entries, users can revert to the previous behavior and avoid automatic population of dictionary entries with empty values during the binding process. This allows users to maintain the desired configuration structure according to their specific needs and preferences.
Feature area
Extensions
Affected APIs
ConfigurationBinder
IConfigurationRoot  extension methods
OptionsConfigurationServiceCollectionExtensions

Associated WorkItem - 143185

https://learn.microsoft.com/en-us/dotnet/core/compatibility/extensions/8.0/dictionary-configuration-binding