Support build with Bazel

This is to achieve goals described in https://github.com/apache/rocketmq/issues/4864

hi li, why we need Bazel having Maven ?

@francisoliverlee See the issue description.

Codecov Report

Merging #4865 (4f0e667) into develop (58c1d1e) will decrease coverage by 1.67%.
The diff coverage is 54.38%.

:exclamation: Current head 4f0e667 differs from pull request most recent head 48a9d83. Consider uploading reports for the commit 48a9d83 to get more accurate results

@@              Coverage Diff              @@
##             develop    #4865      +/-   ##
=============================================
- Coverage      44.77%   43.09%   -1.68%
+ Complexity      7642     7626      -16
=============================================
  Files            982      991       +9
  Lines          68253    68507     +254
  Branches        9023     9075      +52
=============================================
- Hits           30561    29526    -1035
- Misses         33905    35248    +1343
+ Partials        3787     3733      -54

Impacted Files
Coverage Δ

...org/apache/rocketmq/namesrv/NamesrvController.java
63.02% <0.00%> (-1.08%)
:arrow_down:

...va/org/apache/rocketmq/namesrv/NamesrvStartup.java
16.66% <ø> (ø)

...a/org/apache/rocketmq/test/schema/SchemaTools.java
0.00% <0.00%> (ø)

...java/org/apache/rocketmq/test/util/RandomUtil.java
0.00% <0.00%> (-1.41%)
:arrow_down:

...he/rocketmq/tools/admin/DefaultMQAdminExtImpl.java
25.49% <ø> (-0.18%)
:arrow_down:

.../apache/rocketmq/tools/command/MQAdminStartup.java
0.00% <ø> (ø)

...and/acl/ClusterAclConfigVersionListSubCommand.java
20.40% <0.00%> (ø)

...s/command/consumer/ConsumerProgressSubCommand.java
0.00% <0.00%> (ø)

...cketmq/acl/plain/RemoteAddressStrategyFactory.java
89.51% <33.33%> (ø)

...e/rocketmq/remoting/netty/NettyRemotingServer.java
59.62% <45.45%> (-1.77%)
:arrow_down:

... and 108 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Merge to verify issue: https://issues.apache.org/jira/browse/INFRA-23612

Hi @lizhanhui I think this is a good idea to support build with Bazel, but I found that many tests were ignored. I think some tests need to be repaired and things need to be informed before implementation. In addition, DO NOT MERGE your own PR without anyone's approval. I will revert this PR first.

some test cases are ignored, will you open them later?
Only one or two UNSTABLE cases are ignored for now. I plan to fix them in separate pr.

@RongtongJin I think some tests need to be repaired and things need to be informed before implementation. In addition, DO NOT MERGE your own PR without anyone's approval. I will revert this PR first.
 It's advisable not to be so. Inspect the changes, first. Things are not what you  claimed.

Currently, the CI costs too much time, I also think changing the build system is a good attempt. If our members think more discussions are needed, let's do it. @RongtongJin
Also, let's follow the community guidelines, merge after some approvals is the best practice, if we need a merge event to trigger the GitHub action in the test phase, consider using a new branch. @lizhanhui
Unfortunately, I couldn't find the detailed guidelines about the merge conditions on our website, shall we discuss and clear it ASAP?

Actually, Here are some specifications on PR merging that have been voted by the community. I think it is better to follow these guidelines.

@duhenglucky Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

Currently, the CI costs too much time, I also think changing the build system is a good attempt. If our members think more discussions are needed, let's do it. @RongtongJin
Also, let's follow the community guidelines, merge after some approvals is the best practice, if we need a merge event to trigger the GitHub action in the test phase, consider using a new branch. @lizhanhui
Unfortunately, I couldn't find the detailed guidelines about the merge conditions on our website, shall we discuss and clear it ASAP?

Hi @zhouxinyu I agree with you that we need to reduce the time of CI and think it is necessary. But why did the author merge such a large PR by himself without any notice? In addition, he ignored some necessary tests.

Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

This convention has been discussed on the mailing list. https://lists.apache.org/thread/7onbv6yrw7q4ytbhy3w3f2d0v4rtf4dl

@duhenglucky Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

Sorry for my fault of forgot to update the status of this RIP, this is the link to vote at that time: https://lists.apache.org/thread/7onbv6yrw7q4ytbhy3w3f2d0v4rtf4dl, Now I have updated the status, thanks for the reminder ：）

@RongtongJin First, it is not a large PR, only a few lines of changes to main source files. Most of the changes are Bazel build files and fixes to test cases that are not hermetic or unstable, or not capable of running in parallel.  Only a very few tests are temporarily disabled, which will be enabled in a separate PR.  The focus of this PR is to enable Bazel, helping us build and run tests in parallel.
Second, Infra team is waiting to confirm if they configured secrets correctly.
As a member of the community, you, by no means, are supposed to revert contribution harshly, without even getting any response. This is not what a qualified PMC member should do, and can not make the community any way better in any perspecitve.

@duhenglucky Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

Sorry for my fault of forgetting to update the status of this RIP, this is the link to vote at that time: https://lists.apache.org/thread/7onbv6yrw7q4ytbhy3w3f2d0v4rtf4dl, Now I have updated the status, thanks for the reminder ：）

The RIP is obviously lack of thorough discussion. I suggest not to change it status in a haste. It's better to double review it since it would impact a lot in the community.

@RongtongJin First, it is not a large PR, only a few lines of changes to main source files. Most of the changes are Bazel build files and fixes to test cases that are not hermetic or unstable, or not capable of running in parallel. Only a very few tests are temporarily disabled, which will be enabled in a separate PR. The focus of this PR is to enable Bazel, helping us build and run tests in parallel.
Second, Infra team is waiting to confirm if they configured secrets correctly.
As a member of the community, you, by no means, are supposed to revert contribution harshly, without even getting any response. This is not what a qualified PMC member should do, and can not make the community any way better in any perspective.

I agree with your last paragraph. In addition, we should not treat yourself leniently and be strict with others

@duhenglucky Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

Sorry for my fault of forgetting to update the status of this RIP, this is the link to vote at that time: https://lists.apache.org/thread/7onbv6yrw7q4ytbhy3w3f2d0v4rtf4dl, Now I have updated the status, thanks for the reminder ：）

The RIP is obviously lack of thorough discussion. I suggest not to change it status in a haste. It's better to double review it since it would impact a lot in the community.

So why didn't you participate in the discussion when I sent the discussion email?

@RongtongJin First, it is not a large PR, only a few lines of changes to main source files. Most of the changes are Bazel build files and fixes to test cases that are not hermetic or unstable, or not capable of running in parallel. Only a very few tests are temporarily disabled, which will be enabled in a separate PR. The focus of this PR is to enable Bazel, helping us build and run tests in parallel.
Second, Infra team is waiting to confirm if they configured secrets correctly.
As a member of the community, you, by no means, are supposed to revert contribution harshly, without even getting any response. This is not what a qualified PMC member should do, and can not make the community any way better in any perspective.

I agree with your last paragraph. In addition, we should not treat yourself leniently and be strict with others

Read the context of that PR. You are purposely neglecting reviews, changing from one spell error to another. This is why the revert is necessary.

@duhenglucky Given the status of the guideline itself being DISCUSS, it does not make sense to assume it is something of a consensus.

Sorry for my fault of forgetting to update the status of this RIP, this is the link to vote at that time: https://lists.apache.org/thread/7onbv6yrw7q4ytbhy3w3f2d0v4rtf4dl, Now I have updated the status, thanks for the reminder ：）

The RIP is obviously lack of thorough discussion. I suggest not to change it status in a haste. It's better to double review it since it would impact a lot in the community.

So why didn't you participate in the discussion when I sent the discussion email?

This is not a normal RIP of specific features. If it would reflect continuous community consensus, it should be thoroughly reviewed and widely accepted.

Let's relax and focus on this PR, things should be moving forward, how about closing this issue in a new branch and then merging it to develop after every test case passed, I also want to be a volunteer to fix these unit tests. @lizhanhui
As for the merge convention, the community had a consensus, but that does not prevent us from discussing and polishing it continuously. For example, the so-called core contributor and by module rules are confusing and difficult to practice. IMO, we could have more discussions on the mailing list and apply the rules to .asf.yaml to reduce the labor burden once we have a new consensus.

Let's relax and focus on this PR, things should be moving forward, how about closing this issue in a new branch and then merging it to develop after every test case passed, I also want to be a volunteer to fix these unit tests. @lizhanhui
As for the merge convention, the community had a consensus, but that does not prevent us from discussing and polishing it continuously. For example, the so-called core contributor and by module rules are confusing and difficult to practice. IMO, we could have more discussions on the mailing list and apply the rules to .asf.yaml to reduce the labor burden once we have a new consensus.

I agree with that, and we could discuss more details about mergeability in this issue #4870 .

how about closing this issue in a new branch and then merging it to develop after every test case passed, I also want to be a volunteer to fix these unit tests.

This is the RocketMQ community it used to be and where the genuine Apache Way shines. It's okay to create a branch in the repository where we make every case pass and fix the rest flaky ones.

As for the merge convention, the community had a consensus, but that does not prevent us from discussing and polishing it continuously. For example, the so-called core contributor and by module rules are confusing and difficult to practice. IMO, we could have more discussions on the mailing list and apply the rules to .asf.yaml to reduce the labor burden once we have a new consensus.

Exactly. RIPs are not something fixed, if not deprecated.  They are consensus that helps the community.


Add Grafana to Observability

Grafana is missing in the observability panel.

Hi @seuf you are right, we will add to Observability/Monitoring in the upcoming release. Please provide a good logo to use (without background color).

Thanks @seuf, closing as this update was included in the most recent version https://github.com/cncf/landscape#current-version


Expanded indexing for RDMs.

Currently the RDMs object supports indexing by integers and bools.
It would be possible to expand this to support indexing by slices (e.g. rdms[2:]), as well as setting by index (e.g. rdms[2:2] = RDMs(…) or something), as well as indexing by descriptors (e.g. rdms['type'], or something pandas-like rdms[rdms['type'] == 'animate']).  Also related magic methods such as __delitem__.
Which of these would be useful?
Which of these would violate the principle of least astonishment?
When should we return copies of RDM objects versus references?

I finally got around to thinking about this a bit.
I think the slicing stuff seems helpful.
For the ones on descriptors I am not so sure. We already have subsetting & sampling methods by descriptors and given that we always have at least two dimensions (over rdms and over patterns) I think it is clearer without indexing directly and does not add too much burden. rdms.rdm_descriptors['type'] should work for getting a descriptor for example. And the code for the subset would be rdms.subset('type', 'animate')
I think the reference vs copy part is interesting. I am in favor of copies throughout, but I did not test carefully that this is true everywhere.

I think for the copy vs reference thing what could help is a convention on the wording:
If it returns a copy, use as or the perfect form: as_temporal_rdm(), sorted()
if it operates in places and returns a reference for chaining operations, use the infinitive verb:
sort(), align() etc

With references vs copies I had the following hypothetical example in mind:
One could imagine a temporal RDM with a lot of data (e.g. a 50x50 RDM for each ms of a minutes-long EEG recording session).  Then one might want to keep that in memory just once, and be able to slice out a time window (e.g. everything except the first pre-stimulus 1000ms) by reference, rather than by copy, which would require the entire thing to fit in memory twice.
This is perhaps an edge case, but other large datastructures (e.g. pandas.DataFrame, numpy.array, scipy.matrix) apply slicing and indexing by reference, so this may be the expectation:
import numpy
a = numpy.array([1, 2, 3, 4, 5])
a[2:4] = numpy.array([9,0])
a  # array([1, 2, 9, 0, 5])

import pandas
df = pandas.DataFrame({'a': [1, 2, 3], 'b': [4, 5, 6]})
df['a'] = [9, 8, 7]
df  #    a  b
    # 0  9  4
    # 1  8  5
    # 2  7  6

import numpy
import pyrsa.rdm
rdms = pyrsa.rdm.RDMs(dissimilarities=numpy.zeros((8, 10)),
                      pattern_descriptors={'type': numpy.array([0, 1, 2, 2, 4])}, dissimilarity_measure="Euclidean", descriptors={'subj': 0})
rdms[0]  # … dissimilarities = [[0. 0. 0. 0. 0. 0. 0. 0. 0. 0.]]
rdms[0].dissimilarities = numpy.array([[1, 1, 1, 1, 1, 1, 1, 1, 1, 1]])
rdms[0]  # … dissimilarities = [[0. 0. 0. 0. 0. 0. 0. 0. 0. 0.]]  <-- it didn't work

Would it make sense to base our data classes off something like xarray.DataArray instead of coding our own implementation from raw numpy and up? Keeping track of meta data while supporting a reasonable range of indexing operations is quite hard. Maybe it's not something we have to solve ourselves.

sorry for taking so long to come back to this.
The example you give illustrates really nicely why we need to get our policy on copy vs reference straight. I would think that directly setting parts of the dissimilarities array should be discouraged. At the same time I definitely understand the expectation that rdms[0].dissimilarities = numpy.array([[1, 1, 1, 1, 1, 1, 1, 1, 1, 1]]) could work. Especially not throwing an error while not doing the change is bad. In some way, the longer function form subsetting methods are less suggestive in this direction.


Design issue with dependency resolution and avoiding the service locator pattern

First of all, thanks to everyone who participated in this extendable and reliable project. I really am grateful for it but I also feel there's a stone in my shoe.
In my ASP.NET Core application, I provided my own implementation for a few components such as the connection validator, the subscription interceptor and the retained message manager. The ConfigureServices method of my Startup class looks like this:
services
  .AddHostedMqttServerWithServices(mqttServerOptions =>
  {
    mqttServerOptions
      .WithConnectionValidator(new CustomMqttConnectionValidator())
      .WithSubscriptionInterceptor(new CustomMqttSubscriptionInterceptor())
      .WithRetainedMessagesManager(new CustomMqttRetainedMessageManager())
      .WithoutDefaultEndpoint();
  });

As you can see, MQTTnet forces me to provide my own custom components instances but that's not ideal because they, in turn, they depend on other services I've registered with the ASP.NET Core dependency container. So, in order to resolve those dependencies, my components are forced to use the ugly service locator pattern which I'd like to avoid. This is not just a code style issue: I'm having a hard time testing my components because the service locator pattern makes it harder to mock their dependencies.
As suggested in this other issue by @ovidiaconescu, I think it would be better if we could register our own implementations with the dependency container, an approach used by other projects, such as Microsoft's ASP.NET Core Identity.
services.AddSingleton<IMqttServerConnectionValidator, CustomMqttConnectionValidator>();
services.AddSingleton<IMqttServerSubscriptionInterceptor, CustomMqttSubscriptionInterceptor>();
services.AddSingleton<IMqttRetainedMessagesManager, CustomMqttRetainedMessageManager>();

I'm suggesting the MqttServer should leverage the IServiceProvider for its dependency resolutions.
Thanks.

@BrightSoul I've got the exact same stone in my shoe too. I'm using a EFCore DBContext to validate clients and subscriptions to topics.
I'll have a look on how this could be achieved.
However, ideas and hints are always welcome - i haven't worked with this codebase before ;)

What i have found out by now is that you can use the following to inject the validators to the mqtt server component:
Create your validators as singleton dependencies with a dependency to an IServiceScopeFactory.
In the validation tasks you can create a scope and resolve the services required.
public class MqttConnectionValidator : IMqttServerConnectionValidator
{
    private readonly IServiceScopeFactory _serviceScopeFactory;

    public MqttConnectionValidator(IServiceScopeFactory serviceScopeFactory)
    {
        _serviceScopeFactory = serviceScopeFactory;
    }

    Task IMqttServerConnectionValidator.ValidateConnectionAsync(MqttConnectionValidatorContext context)
    {
        using (var scope = _serviceScopeFactory.CreateScope())
        {
            var dbContext = scope.ServiceProvider.GetRequiredService<ApplicationDbContext>();
            context.ReasonCode = MqttConnectReasonCode.Success;
        }
        return Task.CompletedTask;
    }
}

In your Startup.cs register the validator components as singletons as follows:
services.AddSingleton<MqttConnectionValidator>();

Then add your hosted MQTT service with services.AddHostedMqttServerWithServices(). There you can use the ServiceProvider from the AspNetMqttServerOptionsBuilder to resolve the validators.
services.AddHostedMqttServerWithServices(options =>
{
    var scope = options.ServiceProvider.CreateScope();
    options.WithDefaultEndpoint().WithDefaultEndpointPort(1883);
    options.WithConnectionValidator(scope.ServiceProvider.GetRequiredService<MqttConnectionValidator>());
});

Let me know if this works out for you!

@mKenfenheuer

Let me know if this works out for you!

YES! Indeed, thanks a ton for your solution! I implemented it right away in my project. At least I can now provide a mock implementation of IServiceScopeFactory to my components' constructor (and this eases unit testing which was my immediate concern).

Edit: Unfortunately we cannot use anything else than a singleton as the validators are singletons by design.

No problem since, as you said, I could always create a scope. I was already doing it in order to get a reference to ther scoped dependencies such as a DbContext and IHttpContextAccessor.
Thanks again! I'd like to keep the issue open in case project maintaners wanted to consider/implement proper dependency resolution for a future release.

@BrightSoul I will close the issue. I integrated microsoft.extensions.dependencyinjection 3 years ago and it was met by backlash and had to be removed again.


⚠ Bofrost.fr has degraded performance

In 39dc861, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 892 ms

Resolved: Bofrost.fr performance has improved in 66965cf.


DRAFT Dougwilson/cad 3875 integrate lmdb into consensus

Work in progress.

Work in progress.

Would it make sense to simply mark the PR as draft in github?

I think we can close this one as it was superseeded by #3710
@dnadales @jorisdral


Optimize leaves animation.

The difference is most visible on mobile devices :)

Use deltatime so the animation doesn't go slower if you don't reach 60FPS
Do not create Image objects every frame.
Change the page container to its own compositing layer so the page isn't repainted every frame.

Tested in local. For firefox there is a big improvement. CPU usage is still very high tho.
@casassg can you merge into master?

:+1:

kudos


Silently fails when bundler is not loaded

I've been seeing this issue pop up for people. They don't realize that the bundler gem is not installed in their environment. The reason the continue down the wrong path is because the generated Rakefile is rescuing from LoadError without providing any kind of error message. https://github.com/HipByte/Flow/blob/master/template/flow/files/Rakefile#L6
Exhibit A: http://community.rubymotion.com/t/cant-get-basic-flow-project-to-work/1846/3

Fixed by #54


unhandled exception

[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: ClientException with SocketException: Connection timed out (OS Error: Connection timed out, errno = 110), address = email-otp.api.rohitchouhan.com, port = 51578, uri=https://email-otp.api.rohitchouhan.com/v2/

Issue fixed


Currently there is no way to check if a cluster has been drained successfully and workload has been moved completely.

Describe the bug
Currently there is no way for user to check if a cluster has been drained successfully and workload has been moved completely to another synctarget
[knarra@knarra ~]$ kubectl version -o yaml
clientVersion:
buildDate: "2022-07-29T17:25:01Z"
compiler: gc
gitCommit: 70750898e45ff4a349995b08e1d64a359e4c4880
gitTreeState: clean
gitVersion: v1.24.1
goVersion: go1.18.4
major: "1"
minor: "24"
platform: linux/amd64
kustomizeVersion: v4.5.4
serverVersion:
buildDate: "2022-08-02T22:50:38Z"
compiler: gc
gitCommit: 350f7e9
gitTreeState: dirty
gitVersion: v1.24.3+kcp-v0.0.0-350f7e9
goVersion: go1.17.13
major: "1"
minor: "24"
platform: linux/amd64
To Reproduce
Steps to reproduce the behavior:

Add a synctarget by running the command ‘kubectl kcp workload sync knarracluster1 --syncer-image ghcr.io/kcp-dev/kcp/syncer:cf540bb -o - | KUBECONFIG=/home/knarra/Downloads/kubeconfig_411 kubectl apply -f -’
Create deployment using the command ‘kubectl create deployment kuard --image gcr.io/kuar-demo/kuard-amd64:blue --dry-run=client -o yaml > kuard.yaml & kubectl apply -f kuard.yaml`
Now make sure that deployment has been successfully synced
Run command ‘kubectl kcp workload drain knarracluster1`
There is no user facing message which indicates that the cluster has been actually drained and all the workload has been moved.

Expected behavior
There should be a way for user to check if cluster has been drained and all the workload has been moved successfully.
Additional context
Add any other context about the problem here.

/transfer-issue contrib-tmc


✨[feature Request]: Implementing a search bar for multi-language support option

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This feature will allow users to type the name of their desired language in the input field, making it easier and faster to find and select the language they need
Use Case
The search functionality will filter the available languages in real-time as the user types, displaying relevant matches even when only part of the language name is entered. For example, typing "Ge" should show both "Georgian" and "German." (and other matching languages). The filtering will be case-insensitive.
Benefits
No response
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

Subject: PR for Multi-language Search Bar
Hi @Anjaliavv51
I’ve submitted a pull request that implements the multi-language search bar, resolving issue #622. Please review it search bar .html file and consider merging it into the main branch.

@rajatsingh535 did you raise an issue for this feature?

nope @LNischala Issue already exits i didn't raise

Before submitting a pull request you need to either raise an issue or comment under an existing issue that you want to work on it. Only after the PA assigns you the issue you can start working on it and submit the pull request

@LNischala  Now i already worked on it @Anjaliavv51  please assign this to me and review merge it

@Anjaliavv51 Since I have raised the issue and followed a proper procedure for it and am waiting for your reply to start working on it, I would like you to assign this to me (according to the first come first serve basis)
Thank you


fix(lit-core): LIT-4016 - Enhance error handling during epoch changes

Description
Updated error handling and logic flow for handling epoch change events, and signalling errors during epoch change processing to consumers/listeners

LitCore is now an event emitter

Used EventEmitter3 which is extremely widely used, ESM compatible, and written in ES3 for maximum compatibility (!)

Events for disconnected, connected and error are emitted
We no longer call _stopListeningForNewEpoch() during _connect(); concurrent calls to connect() are chained automatically already.

This fixes a corner case where failures to connect during event handling would cause LitCore to stop listening for future epoch changes.

Error handling / execution flow updated in _handleStakingContractStateChange()

Errors encountered anywhere in the process will emit error/disconnected events appropriately
this.ready is now set to false when epoch change events are not processed correctly

We now throw an explicit NotReady error from getSessionSigs() if we get to the point we're going to map across this.connectedNodes, but the LitCore instance is not ready.

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

How Has This Been Tested?

[X] Ran all local-tests

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Currently the loading of modules in the connect() is coupled with the node handshake? it's possible that the handshake fails for any reason so the this.ready=false but we'll still try to reload the WASM modules even though they've been loaded in the gloabalThis? Maybe multiple loads isn't that bad or shall we revisit this?

Are we able to successfully & repeatedly reproduce this error @MaximusHaximus ? How has the fix been tested?

Can you please confirm that in the _connect() even if we're unable to handshake to a single node we never set this.ready=true? @MaximusHaximus @Ansonhkg

Yes, that's right -- if any handshakes fail, the entire connect() chain rejects immediately

Currently the loading of modules in the connect() is coupled with the node handshake? it's possible that the handshake fails for any reason so the this.ready=false but we'll still try to reload the WASM modules even though they've been loaded in the gloabalThis? Maybe multiple loads isn't that bad or shall we revisit this?

Our crypto module code checks to see if globalThis. has been set, and if it has, it does nothing at all. I agree it's a bit confusing -- the good news is that in v7+, we no longer keep global state around and there is no loadModules() method -- so this is temporary logic, thankfully :)

Are we able to successfully & repeatedly reproduce this error @MaximusHaximus ? How has the fix been tested?

Unfortunately this is a very corner-case issue that is caused by failures in entirely internal code.  It also requires that we trigger epoch changes to actually verify that the fix is working :(
With very creative Jest mocks, and long-running Shiva tests, I can write a test case that will reproduce it consistently and verify this fix is always present, but our current local-tests don't really facilitate this degree of testing, and we don't have live epoch-change tests yet :(.  I was, however, able to test the fix by adding manual throw Error() calls into specific spots in the SDK callstack and verifying the SDK followed the right code paths :+1:


Add tests for column names with dots

Part of #8666.  Adds Parquet and ORC tests for column names with dots in them.  This also fixes a minor bug in the assert code where column names were not being escaped as needed when column names contain a dot.

build


[RFC]: Upgrade OpenBLAS dependency

Description
OpenBLAS 0.2.19 is almost eight years out of date.
As a result, I ran into issues running make install-deps-openblas. See truncated error logs below:

...
Makefile:488: warning: undefined variable `CCBLASOBJS'
Makefile:488: warning: undefined variable `CCBLASOBJS_P'
Makefile:488: warning: undefined variable `CZBLASOBJS'
Makefile:488: warning: undefined variable `CZBLASOBJS_P'
Makefile:488: warning: undefined variable `CXBLASOBJS'
Makefile:488: warning: undefined variable `CXBLASOBJS_P'

<output truncated>

gemm.c:403:103: error: use of undeclared identifier 'GEMM_DEFAULT_OFFSET_B'
  sb = (XFLOAT *)(((BLASLONG)sa + ((GEMM_P * GEMM_Q * COMPSIZE * SIZE + GEMM_ALIGN) & ~GEMM_ALIGN)) + GEMM_OFFSET_B);
                                                                                ^
../common_param.h:972:23: note: expanded from macro 'GEMM_OFFSET_B'
#define GEMM_OFFSET_B   GEMM_DEFAULT_OFFSET_B
                        ^
6 errors generated.
make[2]: *** [sgemm.o] Error 1
make[1]: *** [libs] Error 1
make: *** [deps-install-openblas] Error 2

Upgrading to 0.3.27 resulted in successful installation. I propose to upgrade the repo URL, checksum, and version to result in successful OpenBLAS installation.
Related Issues
No response
Questions

Do we need to lock down more build flags along with this? See:
https://github.com/stdlib-js/stdlib/blob/466b5aa2ef6075528418c8fb86bf91566242f95a/tools/make/common.mk#L406-L480

Are there other compatibility issues/concerns/validations which would need to accompany this change?

Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

@rreusser This would be a welcome addition. And indeed, upgrading OpenBLAS has been a TODO.

@rreusser I originally based everything on how Julia manages OpenBLAS: https://github.com/JuliaLang/julia/blob/master/deps/openblas.mk

You can probably cross-reference there to see if we are missing anything in terms of build flags.

...and no backward compatibility concerns we need to work about. Previous OpenBLAS support via stdlib was always experimental.


LD SP nn implementation and test

Coverage increased (+0.007%) to 99.151% when pulling 30523dea35b157e903ead7d72b1fb28e8c882725 on ldspnn-implementation into e5592ca3683d66014d92aa673d93d600efdb3576 on master.


Dependency fetching seems incredibly slow; one dependency adds 20+ seconds to 'terragrunt plan' execution time.

First of all: thanks for creating terragrunt! Both this tool and the articles from @brikis98 have significantly contributed to the enjoyment and success of my DevOps vocation.
That said, I (finally) just started using module dependencies and they seem to work as intended, yay! However, the time it adds to running terragrunt plan is making my workflow sad. Before setting up this module dependency, the time delta between executing terragrunt plan and getting to the point where terragrunt shows Running command: terraform plan was ~2 seconds. Now it's 20+ seconds! Is this expected?
We have what is (I think?) a simple dependency setup. Service/module names changes to protect the innocent my pride.
├─ service-one/
│  ├─ terragrunt.hcl
│  └─ main.tf
├─ service-two/
│  ├─ terragrunt.hcl
│  └─ main.tf
└─ modules/
   └─ a-module/
   │  └─ main.tf
   └─ b-module/
      └─ main.tf

service-one pulls in a-module.
service-two pulls in b-module.
service-two takes input variables from service-two via this terragrunt.hcl stanza:

dependency "service-one" {
  config_path = "../service-one"
}

inputs = {
  foo = dependency.service-one.outputs.foo
  bar = dependency.service-one.outputs.bar
  baz = dependency.service-one.outputs.baz
}

Somehow this takes 20+ (sometimes over 30) seconds:
$ terragrunt plan -out plan.out
[terragrunt] [/Users/myname/git/infrastructure/terraform/service-two] 2020/11/24 13:13:38 Running command: terraform --version
[terragrunt] 2020/11/24 13:13:38 Terraform version: 0.13.5
[terragrunt] 2020/11/24 13:13:38 Reading Terragrunt config file at /Users/myname/git/infrastructure/terraform/service-two/terragrunt.hcl
[terragrunt] [/Users/myname/git/infrastructure/terraform/service-one] 2020/11/24 13:13:38 Generated file /Users/myname/git/infrastructure/terraform/service-one/.terragrunt-cache/016418505/backend.tf.
[terragrunt] [/Users/myname/git/infrastructure/terraform/service-one] 2020/11/24 13:13:38 Running command: terraform init -get=false -get-plugins=false
[terragrunt] [/Users/myname/git/infrastructure/terraform/service-one] 2020/11/24 13:14:01 Running command: terraform output -json
[terragrunt] 2020/11/24 13:14:05 Running command: terraform plan -out plan.out

Based on this output, it looks like the terraform init -get=false -get-plugins=false command is what's taking so long. However, when I manually run this same command in the service-one directory, it takes < 5 seconds.

This is essentially a duplicate of https://github.com/gruntwork-io/terragrunt/issues/1322. You can follow that ticket for updates on when we improve this.
As far as workarounds go, depending on how deep your dependencies are, it may be faster to go through the standard path (set disable_dependency_optimization = false on your remote state block).

I've subscribed to #1322, though I will add here that setting disable_dependency_optimization = false does not appear to affect the timings at all in my setup.


[wip] PMTiles and UN Smart Maps Bazaar to demonstrate a possible future KSJ operation

background

@mapconcierge suggested considering a new cloud-native format for KSJ - Kokudo Suchi Joho (国土数値情報) . See https://speakerdeck.com/fullfull/guo-tu-shu-zhi-qing-bao-jian-tao-hui-yoriopunnaguo-tu-shu-zhi-qing-bao-tonarutamenoti-an-gu-qiao-zi-liao?slide=39
@hfu believes that prototyping is important for technology decision. @hfu decided to create a prototype on the idea to share geospatial information in PMTiles.
@hfu chose UN Smart Maps Bazaar for hosting these prototype PMTiles.

what is wip
https://github.com/optgeo/pmtiles-ksj

#274


Indeterminate (Looping) Progress Bars do not get color styling

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior

Consider this code:

<progress></progress>

Progress bar is expected to be red (danger styled).
Actual behavior

Progress bar is blue as by default.
Reproduction of behavior

Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

https://plnkr.co/edit/efcWXsvKTAUHekK6P8g2?p=preview
Environment details

Angular version: 4.x.x

Clarity version: Latest

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

@a-student
I cannot get your plunkr to work but I have verified that this is an issue on the website.


Please update Paragraph with a note to indicate that it should not be placed on a SelfAsserted TP with Required=True

When a paragraph User Input Type is included in the Output claims for a self-asserted TP and that output claim is set to Required=true, the TP will always indicate that input has not been provided. Please include a note indicating that this is an unsupported configuration.

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

ID: 3e31c230-4294-6b21-91c2-b829852f80a3
Version Independent ID: ea251507-2a65-6ceb-0ccf-fa4f75133fa3
Content: ClaimsSchema  - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/claimsschema.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@dotnetgator Thank you for your feedback . We will check with the content team on how best the documentation can be improved as per your recommendations at this point.


how to display image when setHtml contains path

Hi !
I found a solution, but I don't know if this will help you or if you have found another solution : https://github.com/wasabeef/richeditor-android/issues/289
Bye and good luck for the future.
Clerclem


fix(query-devtools): Make devtools internal fields private

Fixes #6153
This change makes all the internal fields of the core query devtools private to prevent users from accidentally mutating internal values. We have implemented getters and setters to allow controlled access to select private fields. Should there be a need to expose additional fields from the core devtools in the future, we can address that as necessary.

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

packages/query-devtools/src/utils.tsx
49.20% <74.64%> (ø)

... and 80 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


chore: setup code owners

Proposed changes
Setup code owners to restrict who can approve changes.
Types of changes

[x] Other

Checklist

[x] I have read the CONTRIBUTING documentation
[x] Lint and unit tests pass locally with my changes

@air1one - please review this in the next few days. Be sure to explicitly select labels so I know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   77.67%   77.67%
=======================================
  Files          22       22
  Lines         318      318
=======================================
  Hits          247      247
  Misses         71       71

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 cfc9de4...700323b. Read the comment docs.


Why only three parameters were output

I try to run stage1 of the image captioning task, but encounter an issue. I follow the instructions
I download ofa_large.pt in /home/szf/OFA/checkpoints/

Then I unzip the datesets caption_data.zip in /home/szf/OFA/dataset/caption_data/

When I run the code and only three parameters are output.

Looking forward to your response, thank you.

@zhongfansun try modifying the script to
for max_epoch in 2 do
  echo "max_epoch "${max_epoch}
  for warmup_ratio in 0.06 do
    echo "warmup_ratio "${warmup_ratio}
    for drop_worst_after in 2500 do
      echo "drop_worst_after "${drop_worst_after}

If you use nohup, the log will be saved in train_stage1.out

Hi @zhongfansun, I encountered the same issue. Have you solved it?


Fix link to blogpost in changelog

cc @jospoortvliet

Oh, looks like I had not pushed that commit :( but this was already fixed, just deployed it, too.
Sorry. Good idea to link to the upgrade blog, I added that.

https://github.com/owncloud/owncloud.org/pull/909


Precision on Date Time Utils

@shtanganet commented on Fri Apr 02 2021
we have events in next inserted order (generated by command handler line by line of course):
{
    "_id" : ObjectId("6058ba8e760a705e7d51436c"),
    "aggregateIdentifier" : "121eb099-3503-4521-80fa-2e0d24a8ce86",
    "type" : "FinanceOrderAggregate",
    "sequenceNumber" : NumberLong(66),
    "timestamp" : "2021-03-22T15:41:02.1019Z",
    "payloadType" : "...data.access.persistence.axon.event.order.FinanceOrderTaxDeletedEvent",
},
...
{
    "_id" : ObjectId("6058ba8e760a705e7d514372"),
    "aggregateIdentifier" : "121eb099-3503-4521-80fa-2e0d24a8ce86",
    "type" : "FinanceOrderAggregate",
    "sequenceNumber" : NumberLong(72),
    "timestamp" : "2021-03-22T15:41:02.101944Z",
    "payloadType" : "....data.access.persistence.axon.event.order.FinanceOrderTaxCreatedEvent",
}

and during findTrackedEvents we have next problem:
as ordering of received events by timestamp, sequenceNumber and timestamp is string, value "2021-03-22T15:41:02.1019Z" will be greater then "2021-03-22T15:41:02.101944Z".
as result we have handled events in order [72, 66] , not in [66, 72] and got wrong cqrs state.

@shtanganet commented on Mon Apr 05 2021
after investigation, the problem is found in org.axonframework.common.DateTimeUtils.formatInstant
little test:
sout("from util: " + DateTimeUtils.formatInstant(Instant.parse("2021-03-22T15:41:02.101900Z")));
sout("from util: " + DateTimeUtils.formatInstant(Instant.parse("2021-03-22T15:41:02.101911Z")));
sout("from util: " + DateTimeUtils.formatInstant(Instant.parse("2021-03-22T15:41:02.1019Z")));
sout("simple toString: " +  Instant.parse("2021-03-22T15:41:02.101900Z").toString());
sout("simple toString: " + Instant.parse("2021-03-22T15:41:02.101911Z").toString());
sout("simple toString  " + Instant.parse("2021-03-22T15:41:02.1019Z").toString());
from util: 2021-03-22T15:41:02.1019Z
from util: 2021-03-22T15:41:02.101911Z
from util: 2021-03-22T15:41:02.1019Z
simple toString: 2021-03-22T15:41:02.101900Z
simple toString: 2021-03-22T15:41:02.101911Z
simple toString  2021-03-22T15:41:02.101900Z

@smcvb commented on Thu Apr 08 2021
Firstly, thanks for filing this issue with us, and spotting the likely culprit.
The original post you shared already points out there's a discrepancy with the formated time between the two events, which causes the ordering issue as you've noted.
I am sorry for any inconvenience this might have caused on your end, @shtanganet.
For completion sake, would you be able to also provide the Axon Framework (AF) and Mongo Extension (ME) versions you are using?
This might help others to uncover the problem if they encounter this, as well as that it better guides us on where (e.g. branch or version) to fix this.
For the time being, I assume you are using the latest versions of AF and ME.
As the DateTimeUtils belongs to AF, I will add the provided test case to the DateTimeUtils class.
What you might have noticed, is that the precision lies with the used DateTimeFormatter present as a static field in the DateTimeUtils.
The addFraction method there ensure that at least 3 digits are present; all trailing zeroes are removed per definition of the DateTimeFormatter implementation.
This can be changed towards a higher value, thus resolving the problem you noticed.
Lastly, I will move the issue to the Axon Framework repository, as it's a AF utility causing the issue and not the Mongo Extension.

The issue has been resolved in commit de6db21.
The precision of the date-time strings has been adjusted to only include milliseconds, instead of potentially trailing numbers after this.
This is done to overcome the potential problem that a provided Instant has trailing zeroes after the third fraction position.
By design, the DateTimeFormatter removes such trailing zeroes.
Although completely reasonable from the formatter's perspective, this can cause issues if equals checks are performed on the resulting String, as is apparent from the above issue description.
Adjusting the position to only contain milliseconds will be sufficient, and is in line with the provided documentation of the DateTimeUtil class.

i'm using axon.version 4.3.5 and axon.mongo 4.3. I'm firstly had checked the latest version before creation this issue of course.
Yes I had fixed in our project it by the same way. I had created custom implementation AbstractMongoEventStorageStrategy and own EventDateTimeUtils for now. and migrated all my events.timestamp to same length (because we are all time using events reply).
Thanks a lot guys!

Great to hear you have resolved the problem on your end for the time being too @shtanganet!
And as stated on the Mongo issue too, thanks for filing this issue with us too.
Any contribution is always much appreciated.


Doesn't Seem to be Working on Windows 10

On Windows 10, The command line tool wasn't able to translate text.

The interpreter is not working too I guess.

https://github.com/ssut/py-googletrans/pull/78

Hi, all you gotta do is edit this into the gtoken.py file
RE_TKK = re.compile(r'TKK=eval\(\'\(\(function\(\)\{(.+?)\}\)\(\)\)\'\);',
                    re.DOTALL)
RE_RAWTKK = re.compile(r'TKK=\'([^\']*)\';',re.DOTALL)

def __init__(self, tkk='0', session=None, host='translate.google.com'):
    self.session = session or requests.Session()
    self.tkk = tkk
    self.host = host if 'http' in host else 'https://' + host

def _update(self):
    """update tkk
    """
    # we don't need to update the base TKK value when it is still valid
    now = math.floor(int(time.time() * 1000) / 3600000.0)
    if self.tkk and int(self.tkk.split('.')[0]) == now:
        return

    r = self.session.get(self.host)

    rawtkk = self.RE_RAWTKK.search(r.text)
    if rawtkk:
        self.tkk = rawtkk.group(1)
        return

Hi, all you gotta do is edit this into the gtoken.py file
RE_TKK = re.compile(r'TKK=eval\(\'\(\(function\(\)\{(.+?)\}\)\(\)\)\'\);',
                    re.DOTALL)
RE_RAWTKK = re.compile(r'TKK=\'([^\']*)\';',re.DOTALL)

def __init__(self, tkk='0', session=None, host='translate.google.com'):
    self.session = session or requests.Session()
    self.tkk = tkk
    self.host = host if 'http' in host else 'https://' + host

def _update(self):
    """update tkk
    """
    # we don't need to update the base TKK value when it is still valid
    now = math.floor(int(time.time() * 1000) / 3600000.0)
    if self.tkk and int(self.tkk.split('.')[0]) == now:
        return

    r = self.session.get(self.host)

    rawtkk = self.RE_RAWTKK.search(r.text)
    if rawtkk:
        self.tkk = rawtkk.group(1)
        return

This helps me so much!

hey i did the same. However, now i am getting the json decode error. Anything on that?

Nah i don't think there is a fix for that yet.

so it keeps happening? no alternative?


[Mass flow meter, ALICAT, and CODA Coriolis mass flow meter]

Device Description
Mass flow meter for high pressures and low flow rate.
Device Access
Do you have access to this device? Will you be able to test the integration?
Yes, this device has been ordered and will be arriving very soon and will be available for testing.
Communication Protocols
Specify the communication protocols that the device supports. E.g. Ethernet, RS485, RS232, CAN, DIO, etc.
This device supports Modbus RTU and ASCII over RS-485 communication.
Documentation
Include publicly available links to the device documentation. If there are no public links available, please attach the documentation file to the issue.
Manual and data sheet attached.
DOC-MANUAL-CODA.pdf
DOC-SPECS-K-2.pdf
Questions and Ideas
If you have any integration questions or implementation ideas that will ensure a high-quality integration, please suggest them here.

Hi @tjzay! We have Alicat Scientific Mass Flow Meters Models K · KC · KF · KG blueprint. Could you check it out and tell if it works for you?


Adding events to a template that is replacing another one

When adding an event to a new template that is replacing an existing template, I have to use the existing (to be replaced) templates name.
Template.loginFormSignUpView.events({
I would expect to define the new event on my new template and not on the one that is replaced.
But this doesn't work. The event is not triggered.
Template.loginFormSignUpViewMarketplace.events({
My code:
Template.loginFormSignUpViewMarketplace.inheritsHelpersFrom(["loginFormSignUpView"]); // necessary or else helpers are missing
//Template.loginFormSignUpViewMarketplace.inheritsEventsFrom(["loginFormSignUpView"]);
Template.loginFormSignUpViewMarketplace.inheritsHooksFrom(["loginFormSignUpView"]); // necessary or else events don't work right

Template.loginFormSignUpViewMarketplace.replaces("loginFormSignUpView");

Template.loginFormSignUpView.clearEventMaps(); // this is necessary to prevent the core "submit form" event from running

/**
 * Events: Login form sign up view
 */
Template.loginFormSignUpView.events({ // for some strange reason our custom event needs to be speficied on the template that we override. doesn't work with our new template name.
  /**
   * Submit sign up form
   * @param  {Event} event - jQuery Event
   * @param  {Template} template - Blaze Template
   * @return {void}
   */
  "submit form": function (event, template) {

    // some code here ...

  }
});

Closing old issues. The problem here seems to be using replaces instead of copyAs. replaces only replaces the render function, continuing to use the original events, hooks, and helpers.


Inaccurate HTTPRequest documentation node

Removing the note that says that request data is ignored when using the GET method.
I was able to send request data with the code:
extends HTTPRequest

func _ready() -> void:
	set_use_threads(true)
	var body = {"email": "gm", "password": "pw"}
	request(
		"http://localhost:5000/login",
		["Content-Type: application/json"],
		false,
		HTTPClient.METHOD_GET,
		to_json(body)
	)

Then, in my Python server (using Flask), I was able to recover the data with the code:
from flask import Flask, request

app = Flask(__name__)

@app.route("/login", methods=["GET"])
def login():
    request_data = request.get_json()
    email = request_data["email"]
    password = request_data["password"]

I think the note should be modified instead of removed. It sounds like Godot is going to ignore the data, but it should be the server.
Sending a GET request with data payload is not recommended according to the HTTP specification.

A payload within a GET request message has no defined semantics; sending a payload body on a GET request might cause some existing implementations to reject the request.

i.e. The behavior depends on the server. The server may accept it, ignore the data, or even reject the request with 4XX. In your case, Flask accepts it.

I think the note should be modified instead of removed. It sounds like Godot is going to ignore the data, but it should be the server.
Sending a GET request with data payload is not recommended according to the HTTP specification.

A payload within a GET request message has no defined semantics; sending a payload body on a GET request might cause some existing implementations to reject the request.

i.e. The behavior depends on the server. The server may accept it, ignore the data, or even reject the request with 4XX. In your case, Flask accepts it.

Amazing. Thank you very much!
Will improve the PR.

PR changed! LMK if it needs more tweaking.

Should be good to merge after applying suggestions and squashing commits together.

Done! I had thought about that but I wasn't sure if I should add external links on the Godot documentation.

Cherry-picked for 3.4.

Cherry-picked for 3.3.4.


Single Logout passes empty NameId from IS-KM to the external IdP

Description
Single Logout passes empty NameId from IS-KM to the external IdP when log out proceed as follows.

Access Publisher Portal, Login there
Access Store Portal (logged automatically thanks to SSO)
Logout from Store Portal (which is not the login entrypoint, Publisher portal was)

or

Access Store Portal, Login there
Access Publisher Portal (logged automatically thanks to SSO)
Logout from Publisher Portal (which is not the login entrypoint, Store portal was)

Steps to reproduce

Configure External IDP through Identity Server (IS-KM) for SSO with SLO
Access Store Portal, Login there
Access Publisher Portal (logged automatically thanks to SSO)
Logout from Publisher Portal (which is not the login entrypoint, Store portal was)
Observe 500 internal server error
Using saml trace, observe the empty nameID from IS-KM to external IDP

Effected versions
IS-KM 5.7.0

@JKAUSHALYA , @mefarazath ,
Is this fixed

This is fixed with https://github.com/wso2/carbon-identity-framework/pull/2137


linux capnpc -o js xxx.capnp  Cannot create property 'pos' on string 'TestView'

capnpc -o js test.capnp
DeprecationWarning: 'createTypeReferenceNode' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createIdentifier' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createToken' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createThis' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createNamespaceImport' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createImportClause' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createLiteral' has been deprecated since v4.0.0. Use factory.createStringLiteral, factory.createStringLiteralFromNode, factory.createNumericLiteral, factory.createBigIntLiteral, factory.createTrue, factory.createFalse, or the factory supplied by your transformation context instead.
DeprecationWarning: 'createImportDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createExpressionStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createVariableDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createVariableDeclarationList' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createVariableStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createPropertyAssignment' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createNumericLiteral' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createNewExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createObjectLiteralExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createPropertyDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: '' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createCallExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createReturnStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createBlock' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createMethodDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createParameterDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createLogicalNot' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createBinaryExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createExpressionWithTypeArguments' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createHeritageClause' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createClassDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
TypeError: Cannot create property 'pos' on string 'TestView'
at setTextRangePos (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:21342:19)
at Object.setTextRangePosEnd (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:21360:32)
at createNodeArray (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:22985:16)
at asNodeArray (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:27581:28)
at createBaseGenericNamedDeclaration (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:23037:35)
at createBaseInterfaceOrClassLikeDeclaration (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:23072:24)
at createBaseClassLikeDeclaration (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:23078:24)
at Object.createClassDeclaration (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:25474:24)
at Object.createClassDeclaration (/usr/local/lib/node_modules/capnpc-js/node_modules/typescript/lib/typescript.js:3138:29)
at generateStructNode (/usr/local/lib/node_modules/capnpc-js/node_modules/capnpc-ts/src/generators.js:494:36)
js: plugin failed: exit code 1

See my comment here, might help: https://github.com/jdiaz5513/capnp-ts/issues/178#issuecomment-2431579443


Is it possible to customise image editing menu?

Is there an existing issue for this?

[X] I have searched the existing issues

The question
I would like to remove resize option, change icons, etc. of image editing menu. Is it possible?

The ImageOptionsMenu widget itself isn't customizable however you can have your custom onImageClicked:
QuillEditorImageEmbedConfigurations(
  onImageClicked: (imageSource) {},
)

and show a different menu.

@EchoEllet should we add a new way to customize the image menu?

@EchoEllet should we add a new way to customize the image menu?

We probably should document on how to show a different dialog with an example.


Document Weight predicate

Hi , i am trying to customize or use any available predicate to use for A/B testing - like for the same route send 10% traffic to destination B and remaining 90% to A

There is a Weight predicate (while trying to find the docs, I couldn't, so I'll reuse this issue for that.)
Under spring.cloud.gateway.routes
      - id: weight_high_test
        uri: ${test.uri}
        predicates:
        - Host=**.weighthigh.org
        - Weight=group1, 8
      - id: weight_low_test
        uri: ${test.uri}
        predicates:
        - Host=**.weightlow.org
        - Weight=group1, 2

The weights are calculated per group.

Thank you Spencer, @spencergibb , I saw something about ThrottleRoutePredicateFactory - is that old? i dont find it in the master branch anymore.

It's only a sample.

@spencergibb - Thank you - I think this works, but I was wondering how this uses the weight - does it take the total # of requests in a day? or an in hour? and splits it into X % to service A and Y% to service B.  does it reset the count if I restart the gateway application?
I am using a config server to hold the application.yml and using actuator refresh for gateway to pick it up. I wanted to know how the gateway changes - does it consider the apps in a day or an hour and split?

@PadmaGit2 : My use case is similar to yours. Can you tell me about your experience with weight based routing implementation in production? Did you find answers for this part of your question:
I was wondering how this uses the weight - does it take the total # of requests in a day? or an in hour? and splits it into X % to service A and Y% to service B. does it reset the count if I restart the gateway application?
Thanks in advance.

I also have a quite similar use case @hariharakumar @PadmaGit2 , can u please share more about your experience with weight based routing implementation,
also i was facing difficulties in implementing it in configuration java class, i want to make the uri and split percentages dynamic.
Thanks


Completely suppresses channel registration in packets

Due to how the brand packet is overwritten, resources such as Axiom cannot communicate properly between a server plugin and client-side mods.
This creates incompatibilities with mods compatible with paper that communicate in this way.
This behavior also prevents plugin-triggered server transfers, such as Skript's Connect Player to Proxy Server effect and similar backend-server behaviors that communicate with the proxy in this way.
I'm unaware if there's a workaround for this that retains the F3 brand rewriting behavior, mainly wanted this documented in case others encounter a similar issue with mod-plugin communication channels simply not existing while using this resource.

Experienced the same issue, tried to trace down why my backend servers weren't communicating with the proxy for about a week before being told to removed any F3 plugins I may have, sure enough this was the only plugin I disabled which brought back all plugin messaging functions.

First of all, sorry for a late response, i've been very busy with my main job.
As stated in the plugin description the velocity version in in alpha because not using that proxy i've been able to do only some simple tests on the plugin so thank you for your help.
I don't know if there is something i could do for client side mods that looks for a specific brand on the server.
Other plugin messages instead should be forwarded without any problems, so if it's not happening it surely a bug that i'll try to fix.

I've released 1.2.1 version that should fix some bug, let me know if it solves your problems

I cannot recreate any of these issues as of version 1.2.1, such as Axiom's packet registration on login or Skript connecting players between servers.
Very cool to see, I will continue testing and report if any other behavior arises as I re-implement it into my network.  Thanks for your work!


Mixin for operationmode conversion

As an idea for a future implementation, It would be awesome to create a Mixin class for the OperationModes to map from operation factor to power and viceversa.

Cool idea!
I think I could work on this as I already have some code that converts fill rate into operation factor:
https://github.com/FlexMeasures/flexmeasures-client/blob/55088c53667b4bbd17221edc191aa2e8e61eca85/src/flexmeasures_client/s2/control_types/FRBC/utils.py#L15

Hahaha it was your idea :P : https://github.com/flexiblepower/s2-ws-json-python/pull/5#discussion_r1239063345
Happy to hear that you still like the idea. I do as well. Feel free to assign it to yourself and pick up of course. Any contribution is appreciated!


MNT: Brain refactoring

This PR modifies Brain so that it uses the internal _renderer as much as possible instead of importing from _pyvista or vtk.
No feature is added.

This is 100% copy-paste + fix calls more or less, right?

Yes. I also got rid of self._iren and self.remove_picking_callbacks since we don't really need it.

Also will this _silhouette refactor help with the (potential) upcoming PR to have edges shown for the glass brain to improve visibility?

I don't know yet but at least it's more DRY.

This is 100% copy-paste + fix calls more or less, right?

Yes. I also got rid of self._iren and self.remove_picking_callbacks since we don't really need it.

Also will this _silhouette refactor help with the (potential) upcoming PR to have edges shown for the glass brain to improve visibility?

I don't know yet but at least it's more DRY.


Plugins error using this module with latest webpack 4.0.0-alpha.0

I have added to my wepback.common manifest in a typical webpack build using different configurations of loaders, but I think the issue has to do with webpack not reading this plugin properly
  plugins: [
    new ExtractTextPlugin("main.css") //also as {filename:"main.css", allChunks: true},
  ],

The webpack error is

Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.plugins[0] should be one of these:
   object { apply, ... } | function
   -> Plugin of type object or instanceof Function
   Details:
    * configuration.plugins[0] misses the property 'apply'.
      function
      -> The run point of the plugin, required method.
    * configuration.plugins[0] misses the property 'apply'.
      function
      -> The run point of the plugin, required method.
    * configuration.plugins[0] should be an instance of function
      -> Function acting as plugin

when logging the const that I import using require I see the extractText function when I log it. It would appear this import does not satisfy, Function acting as plugin, though it does have a new instance on it

@Kielan - This will be taken care of in #676

https://github.com/Kielan/testWebpackErr
Example repo just for reference, switch webpack versions back and forth and notice the different console logs.


Fix save_weights_only flag in ModelCheckpoint

ModelCheckpoint then passes self.save_weights_only to the save function.
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[x] Did you write any new necessary tests?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes #427.
PR review
Anyone in the community is free to review the PR once the tests have passed.

I patched the dump_checkpoint and save_checkpoint methods in TrainerIOMixin to include a weights_only flag, which defaults to False and so should be backwards compatible.
One alternative would be to read the flag from self.checkpoint_callback inside dump_checkpoint, but I find it sub-optimal (for a few reasons).
What's missing is verifying the loading side of things, regarding whether some error handling is necessary.

@kepler thanks, this is great!


How restart slugify after edit column slug

That slugify is in modal, after I edit column slugify and I open again modal, that slugify not automatically again. So, how to restart that slugify again?
My code is like this
$('body').on('click', '#add-bloggerUser', function(e){
            $('#blogger_slug').slugify('#blogger_name');
            $('#popup_blogger-form').modal('show');
            e.preventDefault();
});

$('body').on('click', '.update-bloggerUser', function(e){
            $('#blogger_slug').slugify('#blogger_name');
            $('#popup_blogger-form').modal('show');
            e.preventDefault();
});

Thank you

Sorry - I do not quite understand the use case here. Can you create a plunkr (http://plnkr.co/) with a minimal use case?

Sorry my bad english
can you check this video please
https://s.nimbus.everhelper.me/share/1162347/9p661q3bl8f03n1ad5pa

@Iyongbudiarso Thanks for the cool video 👍 I blindly fixed the issue - since I assumed that it was due to the fact, that the content of your modal is not destroyed when you click away when the modal closes.
Whenever you change the slugified field yourself - altering the source will have no effect on the slugified field.
This is done by setting a .data field like $target.data('locked', true); . This stays the same - since the DOM does not change. After that - your changes will have no effect when you re-open the modal.
I commited a test and a fix into master branch. I just asserted that a target field can never be "locked" when it is empty and the source field is changed...
Since I am not sure how you include the script: You can use the latest master for testing or:
https://cdn.rawgit.com/madflow/jquery-slugify/master/dist/slugify.min.js
Please test my changes if this resolves your issue. Maybe I missed something...

Thats perfect. ok @madflow thanks for the advice :1st_place_medal: My problem is solved.
Thank you


No Command Line Feedback

A detailed description of the proposed feature
brew update provides no command line feedback on initialization. There should be a log like... brew update initiated
The motivation for the feature
Users don't know if the process has kicked off or not. I was in a low-bandwidth area & had no feedback for at least a minute.
How the feature would be relevant to at least 90% of Homebrew users
90% of Homebrew users use Homebrew sparsely.
What alternatives to the feature have been considered

You can run brew update --verbose which should give you more feedback.


Set layout after adding each new pane

Especially when using tiled layout, creating more panes ends up with message:
"pane too small". For example:
tabs:
  - shells:
      layout: tiled
      panes:
        - echo 1
        - echo 2
        - echo 3
        - echo 4
        - echo 5
        - echo 6

It's because the layout is set after creating all the panes. Setting layout
after creating each pane fixes this issue.

sorry to exhume this issue but this is still happening to me with tmuxinator 0.11.1 and tmux 2.7.
here is what I have
name: apomux
tmux_options: -f .apomux.tmux.conf

windows:
    - core:
        layout: tiled
        panes:
          - squall:
            - clear
          - midgard:
            - clear
          - tidus:
            - clear
          - zack:
            - clear
          - wutai-internal:
            - clear
          - vince:
            - clear
          - barret:
            - clear
          - wutai:
            - clear
          - jenova:
            - clear
          - highwind:
            - clear
          - nanaki:
            - clear
          - yuffie:
            - clear
          - sephiroth:
            - clear
          - tagle:
            - clear
          - marlene:
            - clear
          - tifa:
            - clear
          - meister:
            - clear
          - apotests:
            - clear
          - apoctl:
            - clear
          - empty:
            - clear

This says this
create pane failed: pane too small
can't find pane 18
can't find pane 18
can't find pane 18
create pane failed: pane too small
can't find pane 19
can't find pane 19
can't find pane 19
create pane failed: pane too small
can't find pane 20
can't find pane 20
can't find pane 20

and the window looks like this

This is on iTerm2 with tmux mode, but on Apple's Terminal it's the same problem (minus the weird gray bottom bar)

@primalmotion I'm pretty sure the culprit is split-window's default behavior.
From the manpage:

split-window:
Create a new pane by splitting target-pane: -h does a horizontal split and -v a vertical split; if neither is specified, -v is assumed.

As things stand, tmuxinator splits the pane vertically and then sets the layout. So, in your case it's running out of vertical space and the command is failing before it even tries to use the available space.
I just did a quick experiment which used tmux splitw -h ... to explicitly split the previous pane horizontally and your use case works -- as you're expecting it to. This can't just be wantonly implemented, though, because it will likely break other peoples' currently "working" layouts. I'll have to think on this and revisit.

I have 32 panes (8 (row) x 4 (columns)) in one of my tmux windows. All the panes are equal in size.
After upgrading to tmuxinator 0.8.1 (from whatever the version in Ubuntu 16.04), I started getting this error for almost the half of the panes in the window.
Then I generated the shell script from tmuxinator (tmuxinator debug ), and tried;

Saved the output to a file and ran it.
Result: I saw the same error.
2.a. Manually executed the initial commands that creates tmux session.
2.b. Attached to the session from another terminal.
2.c. Ran the rest commands.
Result: I didn't see any error. Panes were created successfully.
In the script, I copied the attach command to the line after session creation. So, the script creates session and attaches to it first. Then I manually detach from the session. The script continues and creates windows and panes, and then it attaches back.
Result: I didn't see any error. Panes were created successfully.

So, it looks like the panes are not correctly (re)sized when we don't attach. The resolution of terminal or the monitor actually doesn't matter in my case. Possibly the default size before attaching is too small?

Hi, @mutlubicer. I believe you're seeing the issue covered by #651. If you haven't already, please have a look at this issue and the suggested fix.


Schedule use redux thunk

Followed this http://redux.js.org/docs/advanced/AsyncActions.html to refactor fetching code into actions with thunk.
Also tweaked animation return logic.
@ayc92 Check the variable and constants for actions.js

Also extract animation function into a utility, and added code for waiting until scroll is finished to load map.

Ship it!


How to fix that ? Doesnt work autocompletion

gz#6333

Hi @smolyakov26 can you try to unsintall jupyter-tabnine and install jupyterlab-tabnine instead?
https://github.com/codota/tabnine-jupyterlab

Ive tried reinstallation, no use of it (
сб, 11 сент. 2021 г. в 11:37, Amir Bilu @.***>:

Hi @smolyakov26 https://github.com/smolyakov26 can you try to unsintall
jupyter-tabnine and install jupyterlab-tabnine instead?
https://github.com/codota/tabnine-jupyterlab
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codota/jupyter-tabnine/issues/76#issuecomment-917368382,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKXUTEVZZAS3GLSXXRM3R6LUBMIOLANCNFSM5D2V7MDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

No reinstall. Note that you're using Jupyter extension, you need to install the JupyterLab extension (link above)

No use , ive even installed ubuntu again than conda , than jupyter with all
extentions , so tabnine doesnt work , kite is working well
сб, 11 сент. 2021 г., 17:17 Amir Bilu @.***>:

No reinstall. Note that you're using Jupyter extension, you need to
install the JupyterLab extension (link above)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codota/jupyter-tabnine/issues/76#issuecomment-917414953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKXUTEUT5APIW72N5NJAXZDUBNQF7ANCNFSM5D2V7MDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Which plugin did you install?
jupyterlab-tabnine is for JupyterLab
and jupyter-tabnine is for Jupyter Notebook
If you use JupyterLab, you need to have only jupyterlab-tabnine installed

Both of them , so i installed last update nodejs  now i have my  notebook
completion is working. But i dont have in jupyter lab
вс, 12 сент. 2021 г., 17:54 maya-tabnine @.***>:

Which plugin did you install?
jupyterlab-tabnine is for JupyterLab
and jupyter-tabnine is for Jupyter Notebook
If you use JupyterLab, you need to have only jupyterlab-tabnine installed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codota/jupyter-tabnine/issues/76#issuecomment-917651727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKXUTETQH53HBNODU3XE7TLUBS5JHANCNFSM5D2V7MDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Can you please try to install **only** jupyterlab-tabnine on fresh conda environment and see if it works?
Keep us updated
Amir
°°°


etc: DependencyInstaller no need to use sudo

The script should be run with sudo or with root permissions, using sudo inside it is redundant and might break on systems that do not have sudo installed.
Signed-off-by: Vitor Bandeira<EMAIL_ADDRESS>

Mention the use of sudo be mentioned in https://github.com/The-OpenROAD-Project/OpenROAD#install-dependencies


Deprecate --recompile-scripts flag

The --recompile-scripts flag is ignored when the daemon is used: scripts won't get recompiled.
Expected Behavior
Build scripts should be recompiled if the user requested it through command line, independently of the use of the daemon.
Current Behavior
The --recompile-scripts flag only works when the daemon is off, or that a new daemon is spawned. The reason is the presence of the [CrossBuildInMemoryCachingScriptClassCache])https://github.com/gradle/gradle/blob/659eda9e9c4651d0374b25e0d9407723275d3a5b/subprojects/core/src/main/java/org/gradle/groovy/scripts/internal/CrossBuildInMemoryCachingScriptClassCache.java#L29-L29). If it contains scripts (that is to say, it's not empty), which always happens after the first build, then the flag is ignored.
Steps to Reproduce (for bugs)
Run a build:
gradle tasks
then run:
gradle -i tasks
Output should contain something like:

Compiling build file '/tmp/groovylibrary/build.gradle' using SubsetScriptTransformer.
Compiling build file '/tmp/groovylibrary/build.gradle' using BuildScriptTransformer.

But it doesn't. Add the following to the build script:
gradle.buildFinished {
 gradle.services.get(org.gradle.groovy.scripts.internal.CrossBuildInMemoryCachingScriptClassCache).cachedCompiledScripts.clear()
}

and to the same: now that we clear the cache at the end of the build, the script is recompiled, which indicates the cross-build cache gets into the way (because it finds a script with the same name, hash and timestamps, it doesn't bother recompiling).
Your Environment
This is a low-impact bug.

@melix Is this a recent regression?

I don't think so. It has probably been introduced when cross-build script caching was added, which was, if I recall properly, around 2.4.

Since nobody has complained about that in such a long time, this makes me doubt the value of this switch.

We should remove it and just fix whatever caching issues still affect script compilation (if any). There would always be rm -rf as a workaround for these situations.

+1 to what @oehme and @adammurdoch said

If this is a low-impact bug, what's the workaround for when I've edited a script but its changes aren't reflected in the build? adammurdoch mentioned "rm-rf", but what directory are we supposed to use that on?

Do you have a reproducible example of when changes aren't reflected in the build?
I think we don't have any known problems with caching of script compilation, so we would be interested in fixing that.
The workaround is to stop the daemon and re-run the build or edit the build script again.

I don't have anything easily reproducible outside of my development environment. TBH, the problem is probably the way my script is structured. I'm new to Gradle and have been having a lot of trouble understanding the right way to do things. Things such as the way that code inside tasks runs before the task actually runs have been tripping me up. Thanks though.

I've edited a script but its changes aren't reflected in the build

That should never happen, as any change to the script will trigger a recompile. You are probably running into a different problem that just looks like the change didn't take effect. We'll need more details to get to the bottom of that though.

The --recompile-scripts flag does work correctly for the Kotlin DSL when using the daemon, see kotlin-dsl/../ScriptCache.kt. So I guess this bug only affects the Groovy DSL.
We've been using this flag during development of the Kotlin DSL. Doing rm -rf on the whole script cache is not strictly equivalent, --recompile-scripts triggers recompiling scripts for the current build only. I suppose we could live without this flag but it felt pretty convenient at times.
/cc @bamboo

We've been using this flag during development of the Kotlin DSL.

For what use case?

@oehme debugging the Kotlin DSL implementation, when working on script compilation, so not really a "valid use case". We can definitely live without it.

--configure-on-demand worked for me with newer versions of gradle.

@Ethan1983 It returns Configuration on demand is an incubating feature. for me, what does this mean?
Please bring --recompile-scripts back, I need to sync (but not build) using the terminal.


Removed unfurling of same links

closes #21

Ask the maintainer as they cannot see this PR but in spite they make their own PR and merge it.

Nope not relevant


fatal error when build the application use make

Got this error:
splitter.cpp:13:10: fatal error: 'osmium.hpp' file not found
#include <osmium.hpp>
^
1 error generated.
Do you know what is the problem?

Please read https://github.com/MaZderMind/osm-history-splitter/blob/master/CONTRIBUTING.md#the-issue-tracker-is-for-issues-with-the-software


fix: Keyboard interrupt errors

This fixes remaining Keyboard interrupt errors which occasionally happen after pressing :
E5108: Error executing lua Keyboard interrupt
stack traceback:
        [C]: in function 'getchar'
        ...al/share/nvim/lazy/which-key.nvim/lua/which-key/view.lua:96: in function 'read_pending'
        ...al/share/nvim/lazy/which-key.nvim/lua/which-key/view.lua:281: in function 'on_keys'
        ...al/share/nvim/lazy/which-key.nvim/lua/which-key/view.lua:255: in function 'open'
        ...al/share/nvim/lazy/which-key.nvim/lua/which-key/init.lua:49: in function 'show'
        [string ":lua"]:1: in main chunk

This relates to the remaining issues from #33

fixed in v3


Blog post proposal about zend-log

@froschdesign I incorporated your feedback in my review prior to merging; thanks!


为什么熔断策略、隔离策略无法全局配置，必须每个接口对应自己的熔断、隔离策略。设计的初衷是什么？

为什么熔断策略、隔离策略无法全局配置，必须每个接口对应自己的熔断、隔离策略。设计的初衷是什么？

Can you please post an English translation?

@bestLibra
You Can config either globally or for just per hystrixcommand.
doc: https://github.com/Netflix/Hystrix/wiki/Configuration#thread-or-semaphore
eg.
If you Want to set isolation strategy to semaphore globally, try :
hystrix.command.default.execution.isolation.strategy = semaphore
If you want to config fine grain, try:
hystrix.command.<HystrixCommandKey>.execution.isolation.strategy = semaphore
You Can find HystrixCommandKey on Hystrix Dashboard.

@spencergibb I've translated for him.
Why we cannot config the circuit strategy or isolation strategy globally?
Do we must set the strategy for each interface?
What's the original intention of the design?

I dont understand you can configure the isolation strategy globally or on a per circuit breaker basis


Add support for nested menu

Apparently active_material does not support nested submenus for the active admin, Is this by choice or we have not updated the package to support it?
For example, when I try to create a menu item with nested submenu, it messes up the layout:

As shown in the image the submenu is overlapping the main menu and some of the menu items are being hidden
Nested menu support was added a while back in active admin https://github.com/activeadmin/activeadmin/pull/5994
If you remove the styling provided by active_material and just use the core styling by active admin it works fine:

Hi @dkniffin any plans on having this implemented?

@IshmeetSingh06 I'd welcome a PR for adding support for this, but I'm not sure when I'll get to adding support for it myself.


What are the limits of this API?

What are the connections limit by api keys?
If im making a platform to connect multiple users with their apikeys.
Is there a limit of connections that i can use?

No, you shouldn't have issues doing that. Keep in mind Bittrex does have ratelimits, but those issues are outside the scope of this library.


[markdown-language-features / markdownEngine.ts / getMarkdownOptions] Expose (export) the highlight.js object used for preview highlighting to allow adding new languages easily (registerLanguage)

Hi,
The internal markdown-language-features extension provides highlighting of fenced blocks associated with a language when previewing a Markdown file.
To do this, it imports the highlight.js component and uses its highlight function, but this import is local to the getMarkdownOptions function of the  markdownEngine.ts file.
This does not allow the highlight.js component to be enriched with new languages (or even removed).
By declaring the import of the highlight.js component at the global level, and exposing it (export) outside the markdown-language-features extension, possibly through a function or an interface, this would allow it to be enriched or its behavior adapted from another extension.
It is still necessary to have a highlight.js plugin supporting the language to implement, and possibly a textmate grammar for highlighting when editing the Markdown file itself, but the work would be simplified if the highlight.js object used for the Markdown preview was public and modifiable.
FYI, solution found for implementing additional language highlighting both when editing a Markdown file and previewing it:

https://github.com/IBM/zopeneditor-about/issues/428#issuecomment-2250038933

Thanks.

#91279 is a better solution for this

Open since 24, Feb 2020…
Solution I suggest seem to be more easy to implement and can be temporary.

Exposing an entire 3rd party library as api is not going to be good long term (or short term)


Add terraform state inventory source

Note: this is a resubmission of PR 1868 from a branch instead of a fork so checks can run. See previous review comments there.
Summary:
A new Terraform state inventory source was added to AWX in PR 14840. This PR updates the UI to include the new source type. It also adds Execution Environment selection to the inventory source add/edit forms, as an EE selection is required for the Terraform source -- users must provide and select an EE containing a Terraform binary, as we cannot ship one with the Terraform binary for licensing reasons.
In addition, this PR fixes a couple of related bugs: when trying to edit an inventory source that is not sourced from a project, the API returns a 400: Bad request error with detail "Cannot set source_path if not SCM type." due to a default value in the form for the source_path field. And the cypress deleteAwxExecutionEnvironment command had the wrong endpoint so this fixes that as well.

We should be able to link to docs on AH now: https://console.redhat.com/ansible/automation-hub/repo/published/cloud/terraform/content/inventory/terraform_state/


Adding Aspect Fit/Fill to Image Resizing Processor

@BrikerMan Could you see my comment and do some refactoring to the existing resize(to size: CGSize, for contentMode: UIViewContentMode), making it something like resize(to size: CGSize, for contentMode: ContentMode) so we could use it in both iOS and macOS?
Thanks!

Codecov Report

Merging #599 into master will decrease coverage by -0.26%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##           master     #599      +/-   ##
==========================================
- Coverage   74.72%   74.46%   -0.26%
==========================================
  Files          21       21
  Lines        2247     2256       +9
==========================================
+ Hits         1679     1680       +1
- Misses        568      576       +8

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 9f576cc...547b7f2. Read the comment docs.

@BrikerMan Good to see this.
Since a similar p-r #600 was merged, I am going to close this one.
Thanks for your time and considering to contributing! 👍


jmeter: 5.4.2 -> 5.4.3

Motivation for this change
Update log4j to 2.17.0 (fix CVE-2021-45105)
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Same rodeo as https://github.com/NixOS/nixpkgs/pull/151080


[Bug]: [Mobile] CheckoutConflictError: Your local changes to following files would be overwritten by checkout

Describe the bug
Happens when trying to clone an existing repo on mobile. I got it to work on mac and windows but IOS is proving troublesome. I have a fresh install of obsidian mobile so not sure why this would be happening. It happens with both config setups (.obsidian and .obsidian.mobile). Please assist if possible. Thanks!

Relevant errors (if available)
No response
Steps to reproduce
Use Obsidian Git to clone an existing repo and follow the steps.
Expected Behavior
Cloning the repo and syncing the Obsidian vault.
Addition context
No response
Operating system
iOS
Installation Method
None
Plugin version
2.19.1

When it asks you the question "Does your repository have an .obsidian directory?" are you hitting "Yes"?
It then warns you if you hit yes you can then either "abort clone" or "delete all local settings, appearances, etc", which deletes the iOS .obsidian folder so that you can properly clone your repo.
alternatively, add a .gitignore file and stop the entire folder from being sync'd. add this line to the .gitignore; ".obsidian/*"
personally i have found syncing the .obsidian folder to be tricky if you like to use multiple devices semi-simultaneously. i do not sync it and use git for the vault, but not settings. additionally, i have several more plugins on desktop than mobile.

When it asks you the question "Does your repository have an .obsidian directory?" are you hitting "Yes"?
It then warns you if you hit yes you can then either "abort clone" or "delete all local settings, appearances, etc", which deletes the iOS .obsidian folder so that you can properly clone your repo.
alternatively, add a .gitignore file and stop the entire folder from being sync'd. add this line to the .gitignore; ".obsidian/*"
personally i have found syncing the .obsidian folder to be tricky if you like to use multiple devices semi-simultaneously. i do not sync it and use git for the vault, but not settings. additionally, i have several more plugins on desktop than mobile.

I did the .gitignore update and also hit yes and it worked! Thank you!


Add Support for Azure Synapse Analytics terratest module

As Azure Synapse Analytics module support is not currently available in this terratest, i would like to make contribution to add this Enhancement.

Included in release https://github.com/gruntwork-io/terratest/releases/tag/v0.43.2


Object Detection reduce false positives.

HI, I'm using an object detector model and it works well and correctly identifies the desired object in most cases.
However it also wrongly identifies similar but different objects often with a high confidence.
I'm wondering what is the best strategy to improve this? Can I include images with no tags that don't have any of the desired object in them? A negative sample? Or should I start to tag similar objects so hopefully they will be identified as that and not the desired object?
Thanks.

@tagy - I think both solutions you mention should improve things. Just adding images of the similar objects without any annotation is probably easier. I would probably start with that approach.
Let me know how it goes. Also let me know if you have any more questions

Thanks for your reply Toby, I will try as you say and let you know.
Can I ask about the confidence of an observation, when I get a VNClassificationObservation and the label I can inspect the  VNConfidence which is normally around 0.9 - 1.0
I'm confused how this confidence score is related to the confidence threshold you can set in the model with a MLFeatureProvider eg:
open var values = [             "iouThreshold": MLFeatureValue(double: 0.45),             "confidenceThreshold": MLFeatureValue(double: 0.5)         ]
Increasing the confidenceThreshold here does make the model stricter, but the VNConfidence is always close to 1.0
How do these number relate? is the confidenceThreshold something that happens before Non-maximum suppression?
Presumably I can set the confidenceThreshold low and apply a filter once I have the VNConfidence? Or I can set a higher confidenceThreshold and not filter using the VNConfidence? Would both give similar results? Thanks.

@tagy - A neural network returns fixed number of predictions which are first filtered by the confidence threshold and then only the boxes left after this filtering with confidence threshold are subjected to Non Maximum Suppression (NMS). Essentially setting higher threshold would affect what the input to Non maximum suppression. So depending on how different your input to NMS is based on your threshold, the output of the NMS could vary.
I think adding more data for the classes which are similar looking as well as more negative data would be great.

Just to confirm I have tried adding images without any annotation and it does help in reducing false positives thanks.
But also I wasn't understanding the confidence levels, the sample Apple live preview app shows a confidence but it is the confidence of the top label (and nearly always 0.9+). Not the confidence of the observation. Getting the confidence of the actual VNRecognizedObjectObservation now matches what I'm getting in the predict function, and so I can more easily set a threshold. Someone else mentioned it here:
https://github.com/apple/turicreate/issues/1314#issuecomment-536052657

@tagy - I'm glad both those things worked out for you. I'm going to close this issue, but let us know if you have any more questions.


[run_sk_stress_test] Update xfails after SR-11911 was fixed

@swift-ci test and merge

@swift-ci please test


Updated types of PTRACE requests for musl to be consistent.

For some reason the type of ptrace request constants in musl change for the register based requests. As they are used in the same way and this doesn't mimic musl this has been changed for consistency and ease of use.
First raised in issue #744

@bors: r+

:pushpin: Commit 86b1213 has been approved by alexcrichton

:hourglass: Testing commit 86b1213196d7fb4befa3485697d3cac89fdd4d23 with merge 1c1413e9615c92ace322b62ba35ee610008d4756...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 1c1413e9615c92ace322b62ba35ee610008d4756 to master...


GEODE-6804: add tab completion for deploy --jar= and --dir=

Before, no tab completion was available, in contrast to other commands like start server --properties-file=
After, filesystem-based completion is available, with the following rules:

--jars supports a comma-separated list of .jar files, although you must type the comma to start the next completion cycle
--dir will avoid suggesting directories that don't contain any .jar files

During tab completion of either jars or dirs, it will look ahead one directory level to see if it can filter out directories that contain neither jars nor further subdirectories.

@jmelchio


Invalid content /en/shelley/get-started/installing-and-running-the-cardano-node/building-the-node-from-source/

Where does the issue occur?
./scripts/shelley-testnet-live.sh
Describe the issue
The script does not work
Expected changes
Fix or remove
Additional context

this has been fixed, thank you