Consider adding reduce(S initial, BiFunction<S state, T element> reducer) method to all AbstractArrayAssert instances

Feature summary
So let's assume we have a huge array (for example of float numbers). It is too verbose to check all items there, but pretty easy to check a hash code or sum of all items.
So I'd like to do something like:
float[] values = ...

// check minimum (or maximum)
assertThat(values)
    .reduce(Float.MAX_VALUE, Math::min)
    .isCloseTo(...)

// check sum
assertThat(values)
    .reduce(0f, Float::sum)
    .isCloseTo(...)

// check hash code
assertThat(values)
    .reduce(0f, (s, n) -> 37.0f * s + n)
    .isCloseTo(...)

Or it could be just a business requirement:
float[] values = ...

assertThat(values)
    .reduce(0, Float::sum)
    .describedAs("Total sum of all factors must be 1.0")
    .isEqualTo(1.0f);

``

A mixed type scenario: array of string, and state is HashSet<String> (to collect unique string):
String[] values = ...

assertThat(values)
    .reduce(new HashSet<String>(), (s, i) -> {
        s.add(i);
        return s;
    })
    .containsExactly("TRUE", "FALSE")

Reduce functions can't be described automatically, so adding reduces into assertj does not change much.
At the same time, .extracting(Function) already exists: https://github.com/assertj/assertj/blob/main/assertj-core/src/main/java/org/assertj/core/api/AbstractObjectAssert.java#L961

Thanks, we will look into this !

At the same time, .extracting(Function) already exists: https://github.com/assertj/assertj/blob/main/assertj-core/src/main/java/org/assertj/core/api/AbstractObjectAssert.java#L961

Not in AbstractArrayAssert

To my personal taste I would prefer:
double[] result = ...

assertThat(result)
    .isNotNull()
    .reduce(Double.MIN_VALUE, Double::max)
    .isCloseTo(1.000, Offset.offset(0.001));

over this:
double[] result = ...

assertThat(result)
   .isNotNull();

assertThat(Doubles.max(result))
   .isCloseTo(1.000, Offset.offset(0.001));

Unfortunately there is no .extracting(Function) for arrays in AssertJ. Even if we have it sometimes .reduce() is more handy, for example there is no sum(float[] values) in any of common libraries (Guava, Apache Common, Java util/lang), but there is Float::sum available in Java lang.
Reduce is a standard and widely-used concept in functional programming.

Reduce is a standard and widely-used concept in functional programming.

If you use that as an excuse, then you should consider adding all the rest: map, flatMap, fold, lfold, and so on.
Why adding reduce only?

for example there is no sum(float[] values) in any of common libraries

Anyone can write such a function, especially if you operate with float.
For double[] there's DoubleStream.of(...).sum().

    .reduce(Double.MIN_VALUE, Double::max)
...
    .reduce(Float.MAX_VALUE, Math::min)
...
    .reduce(new HashSet<String>(), (s, i) -> {
        s.add(i);
        return s;
    })

I believe what you suggest are the anti-patterns: they make the error message obscure, and assertj won't really help there, because assertj won't see beyond the functions.
On contrary, a targeted assertions would be way better.
For instance, instead of reduce(HashSet one can already use
String[] values = ...

assertThat(values)
    .containsOnly("TRUE", "FALSE")

The code would be easier to follow, and assertj can prepare a much better message than the one when user calls .reduce(new HashSet...

IMHO we need to carefully evaluate if AssertJ should replicate the functional programming features already offered by the JDK via the Stream API.
extracting(Function) is kind of already belonging to this topic.

extracting(Function) in AbstractArrayAssert will solve 95% of my cases.
and the additional map() in AbstractArrayAssert which convert the items of the array/list (for example converts float[] to double[] or double[][] to double[] by aggregating rows) will solve the other 5%.
probably extracting() + map() with arrays/lists will be even better, probably reduce looks too scary/specific

The issue with extracting is it provides no information on what is extracted.
I believe, extracting should have a mendatory parameter that dercribes the extraction function

probably extracting() + map() with arrays/lists will be even better

@mazurkin aside from naming, aren't extracting(Function) and map(Function) the same?
Iterable assertions already have map(Function) and extracting(Function), one being the alias of the other.
Object[] assertions already have extracting(Function).
We could argue that all the subclasses of AbstractArrayAssert should be at least at the same functional level as AbstractObjectArrayAssert.

There are slightly different cases:

transform each element in the list one-to-one. List<T> -> List<U>. It would be list.map in Kotlin.
transform the list as a single object. It would be list.let in Kotlin.
flatMap: transform elements to their own lists and concatenate the result into a single list
assertj has 'property extracting' by name. I guess, naming it as map would be weird.

I do not mean all of them are needed in AssertJ.

A bit of history, extracting was introduced a long time ago  (java 6 time!) and the main use case was to extract some properties from the elements under test, ex the id property of a list of persistent objects.
Then came java 8 and functional programming, we added support for lambdas and map(lambda) as an alias of extracting(lambda).
In my view extracting is kinda a specialized case of map where you dig in the elements under test the values to ... extract.

first let me thank you for you job, AssertJ is great, I am using AssertJ instead of hamcrest.assertThat for the new project which contains a lot of math and so a lot of numerical arrays.

aside from naming, aren't extracting(Function) and map(Function) the same?

@scordio please check, @vlsi already explained: extracting(Function) converts the whole array into some other type, map converts T[] into S[] (I suspect there are might be type issues with such a conversion).
My main problems for me right now:

huge numerical arrays, I just don't need to check all items for some of them, hashsum could be enough
some numerical arrays, small ones, should have 1.0 as the sum (the sum of the probabilities). Of course I use containsExactly to check all elements but later if someone modifies the logic this invariant might be broken without checking the total sum.

Of course none of these are blockers, I can make one other extra check, I just want to combine everything into one single chained sequence:
assertThat(array)
   .isNotNull()
   .containsExactly(0.36, 0.42, 0.22)
   .extracting(StatUtils.sum) <-- here
   .isCloseTo(1.000, Offset.offset(0.000001));

// or

assertThat(array)
   .isNotNull()
   .containsExactly(0.36, 0.42, 0.22)
   .reduce(0.0, Double::sum) <-- here
   .isCloseTo(1.000, Offset.offset(0.000001));

// instead of

assertThat(array)
   .isNotNull()
   .containsExactly(0.36, 0.42, 0.22);

assertThat(StatUtils.sum(array))
   .isCloseTo(1.000, Offset.offset(0.000001));

You already provide extraction(function) but for some reason it is not available for arrays.

Thank you for the kind words, @mazurkin 🙂

@scordio please check, @vlsi already explained: extracting(Function) converts the whole array into some other type, map converts T[] into S[] (I suspect there are might be type issues with such a conversion).

This is not how extracting(Function) works for Iterable or Object[] assertions. I suggest to double-check its Javadoc and/or implementation.
Out of this conversation, I see at least two obvious takeaways:

It would be nice to bring all subclasses of AbstractArrayAssert at the same functional level as AbstractObjectArrayAssert.
extracting(Function) can be confusing as the provided Function is not necessarily "extracting" values from the object under test but could return anything.

These however do not address the original need of @mazurkin where a reduce API would seem more appropriate. I see adding that API as a bigger change that would require more careful design (https://github.com/assertj/assertj/issues/2920#issuecomment-1382799163).


The loss and metric suddenly became abnormal during training.

I was training SCNet-large (without using mixed precision training), and the loss and metric suddenly dropped in the middle of the training. How should I handle this situation?
2024-11-09 12:58:25,306 - INFO - ----------------------------------------------------------------------
2024-11-09 12:58:25,306 - INFO - Training Epoch 61 ...
2024-11-09 13:26:45,479 - INFO - Train Summary | Epoch 61 | Loss=0.0548 | Grad=0.0438
2024-11-09 13:26:45,479 - INFO - ----------------------------------------------------------------------
2024-11-09 13:26:45,479 - INFO - Cross validation...
2024-11-09 13:32:05,373 - INFO - Valid Summary | Epoch 61 | Loss=0.1420 | Nsdr=9.094
2024-11-09 13:32:06,420 - INFO - Learning rate adjusted to 0.0002657527142592
2024-11-09 13:32:06,421 - INFO - ----------------------------------------------------------------------
2024-11-09 13:32:06,421 - INFO - Training Epoch 62 ...
2024-11-09 14:02:04,930 - INFO - Train Summary | Epoch 62 | Loss=0.1112 | Grad=1.0411
2024-11-09 14:02:04,930 - INFO - ----------------------------------------------------------------------
2024-11-09 14:02:04,930 - INFO - Cross validation...
2024-11-09 14:07:13,337 - INFO - Valid Summary | Epoch 62 | Loss=0.1595 | Nsdr=7.740
2024-11-09 14:07:14,359 - INFO - Learning rate adjusted to 0.0002657527142592
2024-11-09 14:07:14,359 - INFO - ----------------------------------------------------------------------
2024-11-09 14:07:14,359 - INFO - Training Epoch 63 ...
2024-11-09 14:35:24,521 - INFO - Train Summary | Epoch 63 | Loss=0.1143 | Grad=0.1646
2024-11-09 14:35:24,521 - INFO - ----------------------------------------------------------------------
2024-11-09 14:35:24,521 - INFO - Cross validation...
2024-11-09 14:40:34,042 - INFO - Valid Summary | Epoch 63 | Loss=0.1615 | Nsdr=7.577
2024-11-09 14:40:35,062 - INFO - Learning rate adjusted to 0.0002657527142592
2024-11-09 14:40:35,063 - INFO - ----------------------------------------------------------------------
2024-11-09 14:40:35,063 - INFO - Training Epoch 64 ...
2024-11-09 15:07:52,362 - INFO - Train Summary | Epoch 64 | Loss=11.0862 | Grad=1651.3630
2024-11-09 15:07:52,362 - INFO - ----------------------------------------------------------------------
2024-11-09 15:07:52,362 - INFO - Cross validation...
2024-11-09 15:13:00,251 - INFO - Valid Summary | Epoch 64 | Loss=0.1682 | Nsdr=7.171
2024-11-09 15:13:01,288 - INFO - Learning rate adjusted to 0.0002657527142592
2024-11-09 15:13:01,289 - INFO - ----------------------------------------------------------------------
2024-11-09 15:13:01,289 - INFO - Training Epoch 65 ...
2024-11-09 15:15:03,930 - INFO - Total number of parameters: 42181232
2024-11-09 15:15:09,915 - INFO - train/valid set size: 43723 50
2024-11-09 15:15:10,236 - INFO - Learning rate adjusted to 0.0005
2024-11-09 15:15:10,237 - INFO - ----------------------------------------------------------------------

It looks like you are using a learning rate that is too high. The recommended learning rate for large models should not exceed 0.0003. You can refer to the config file I provided in the README.

I haven't tried training a large model without AMP. May I ask what batch size you used? Perhaps you could try further lowering the learning rate.

The batch size I used is 4, and your paper states that the learning rate is 5e-4 when using only the MUSDB18 dataset. But in the config file you provided, the learning rate is 3e-4. May I ask, what is the appropriate learning rate to adjust when training a large version of the model?

A learning rate of 5e-4 corresponds to the standard version of SCNet. In the configuration file I provided, a learning rate of 3e-4 corresponds to the large version. However, I trained with AMP  and if you don't use it, the learning rate might need to be further reduced.


fix(eslint-config): Ignore external modules with import/no-cycle

What does this change?

Configures the import/no-cycle rule to ignore external modules to decrease the duration of the lint check.
In @guardian/cdk we're seeing import/no-cycle take a significant amount of time:
➜ TIMING=1 npm run lint

><EMAIL_ADDRESS>lint
> eslint src --ext .ts

Rule                                    | Time (ms) | Relative
:---------------------------------------|----------:|--------:
import/no-cycle                         | 24660.951 |    81.0%
import/namespace                        |  3185.143 |    10.5%
prettier/prettier                       |  1226.525 |     4.0%
@typescript-eslint/no-misused-promises  |   283.835 |     0.9%
@typescript-eslint/no-floating-promises |   228.278 |     0.7%
@typescript-eslint/no-unsafe-assignment |   134.758 |     0.4%
@typescript-eslint/no-unsafe-argument   |   114.937 |     0.4%
@typescript-eslint/naming-convention    |   102.000 |     0.3%
import/no-unresolved                    |    76.968 |     0.3%
import/order                            |    69.877 |     0.2%

With import/no-cycle configured to ignore external modules, this is dramatically changed:
➜ TIMING=1 npm run lint

><EMAIL_ADDRESS>lint
> eslint src --ext .ts

Rule                                    | Time (ms) | Relative
:---------------------------------------|----------:|--------:
import/namespace                        |  8576.203 |    54.0%
import/no-cycle                         |  4863.588 |    30.6%
prettier/prettier                       |  1130.410 |     7.1%
@typescript-eslint/no-misused-promises  |   278.362 |     1.8%
@typescript-eslint/no-floating-promises |   230.481 |     1.5%
@typescript-eslint/no-unsafe-assignment |   127.062 |     0.8%
@typescript-eslint/no-unsafe-argument   |   115.256 |     0.7%
@typescript-eslint/naming-convention    |    99.910 |     0.6%
import/order                            |    60.633 |     0.4%
import/no-unresolved                    |    45.641 |     0.3%

See: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-cycle.md#ignoreexternal
How can we measure success?

Improved eslint performance.

This looks like a valuable improvement. If there are cyclical dependencies in external code, there’s not much we can do about it!

agreed, i can't think of a good reason to not do this?


Make file size query failure not fatal

Don't make it be an error if we fail to find the size of the file by querying the origin.

Sure.


phtml files

Does uncss work with phtml files? I'm using it with gulp and it doesn't seem to recognise any of my local files. I'm guessing it's not possible for uncss/PhantomJS to render files so it would be giving errors even if files where accepted
The problem is that when giving pages on a localhost:8080, uncss works fine but it's unable to track css classes used in, for example, a checkout page (as there's no page added to cart for PhantomJS).
So if phtml can't be used, would it be possible to use CasperJS with uncss/phnatomjs to create possible scenarios?

Hi
This is a question I am also interested about.
With frameworks and CMS these days, it is very rare to have a big project with just static HTML pages. It is common to use includes.
Every loaded CSS is in a file containing only the header and that file is loaded for all the pages. There is no CSS classes in the file containing the reference to the CSS files, even for a simple CMS like Wordpress.
I'd like to know if there is a solution in that situation. A need would not only be to remove unused classes but also duplicated code or what has been overwritten. Is that something uncss does?
Thanks in advance.

@phlorie you could use php2html to generate static HTML files for a specific set of routes in your local Wordpress project. You could then use the generated files as source for uncss

@phlorie would that work with phtml though? I've just tried php2html using gulp and it only gives error.

@cortopy @phlorie You are correct in your assumptions about UnCSS. It has no concept of being able to track a user's journey through a website, it can only load a page and calculate what styles are used at runtime. Therefore, for richer experiences which involve managing session data & logged in and logged out states, you have to feed UnCSS static files that contain all of that state.
You could do this in the case of a CMS by getting a spider to crawl your site and dump the HTML output into a directory, then have UnCSS analyse the files for anything unused. And in the case of some other dynamic content, it is possible to use something like CasperJS to interact with your site and write a HTML file at each state - load page, capture, do a form action, capture, etc.
I admit that this isn't ideal, and hope that once the unused reporter is finished, then people can be more aware of what classes UnCSS is removing, and then be able to make better decisions about how to manage their ignore lists.

@cortopy what kind of errors? Would you mind adding an issue over at gulp-php2html?

@mikelambert Any reason to keep open?

Just to followup with the original poster's question at the end. There is no integration with CasperJS inside uncss, but it would be possible to build something like that. Alternately, you could probably use CasperJS to dump out versions of the HTML after every modification, and feed those files to uncss, which would accomplish the same thing (but without needing to dig in and modify any of the uncss internals, or tie uncss to casperjs)


Add possibility to add application-defined configurations for permissions

Depends on https://github.com/vivid-planet/comet/pull/1654
I don't like the term 'configuration', ideas welcome.
Type-Safety is quite good:

(However, it's not perfect: when requiring more than one permission the configurations are merged, the same happens in getPermissionsForUser)
TODO (in next PR): apply this concept to the API-Generator.

@fraxachun is this PR still ready for review, or do you plan to change it? I can't remember what we discussed in last week's meeting...

How would this work for settings permissions on entity level? I believe that will be the main use case for permission configuration. Shouldn't the configuration then store an array of allowed IDs?


Don't ask for password in Google and Github registration

@timabbott, main code is done. I want to add some tests for this though.

@timabbott, added test for password less registration for remote user. Still need to add test for "google: Respect is_signup argument.".

Nice!  I posted a few quick comments; looking forward to having this completed :)

@timabbott, done.

Merged, after adding 1d10941712dbbcd38c4909a5f487a88c1c3b1f6c on top to avoid the prompt and test whether or not the user is prompted for a password.
Thanks @umairwaheed!


Sample project fails to build with Nuget package

I'm trying to build Sample project in VS2019. I've removed Twilio.Voice.iOS and I've added a nuget reference to version 6.3.0 of https://www.nuget.org/packages/Twilio.Voice.iOS.XamarinBinding
When building I'm getting the following errors
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSessionActivated. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioSessionActivated. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioDeviceWriteCaptureData. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioDeviceWriteCaptureData. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioDeviceFormatChanged. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioDeviceFormatChanged. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioDeviceReadRenderData. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioDeviceReadRenderData. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _kTVOErrorDomain. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.kTVOErrorDomain. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate48000. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate48000. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate44100. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate44100. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate16000. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate16000. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TwilioVoiceSDK. The symbol '_OBJC_CLASS_$_TwilioVoiceSDK' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOStatsReport. The symbol '_OBJC_CLASS_$_TVOStatsReport' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate8000. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate8000. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVORemoteTrackStats. The symbol '_OBJC_CLASS_$_TVORemoteTrackStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioDeviceExecuteWorkerBlock. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioDeviceExecuteWorkerBlock. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOOpusCodec. The symbol '_OBJC_CLASS_$_TVOOpusCodec' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOLocalTrackStats. The symbol '_OBJC_CLASS_$_TVOLocalTrackStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOIceOptionsBuilder. The symbol '_OBJC_CLASS_$_TVOIceOptionsBuilder' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCallInvite. The symbol '_OBJC_CLASS_$_TVOCallInvite' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOIceOptions. The symbol '_OBJC_CLASS_$_TVOIceOptions' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOIceCandidateStats. The symbol '_OBJC_CLASS_$_TVOIceCandidateStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOIceCandidatePairStats. The symbol '_OBJC_CLASS_$_TVOIceCandidatePairStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOConnectOptionsBuilder. The symbol '_OBJC_CLASS_$_TVOConnectOptionsBuilder' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOAudioCodec. The symbol '_OBJC_CLASS_$_TVOAudioCodec' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVODefaultAudioDevice. The symbol '_OBJC_CLASS_$_TVODefaultAudioDevice' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOAcceptOptions. The symbol '_OBJC_CLASS_$_TVOAcceptOptions' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioChannelsMono. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioChannelsMono. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCancelledCallInvite. The symbol '_OBJC_CLASS_$_TVOCancelledCallInvite' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate24000. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate24000. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVORemoteAudioTrackStats. The symbol '_OBJC_CLASS_$_TVORemoteAudioTrackStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCallOptions. The symbol '_OBJC_CLASS_$_TVOCallOptions' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCallerInfo. The symbol '_OBJC_CLASS_$_TVOCallerInfo' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOConnectOptions. The symbol '_OBJC_CLASS_$_TVOConnectOptions' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCall. The symbol '_OBJC_CLASS_$_TVOCall' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSampleRate32000. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioSampleRate32000. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOPcmuCodec. The symbol '_OBJC_CLASS_$_TVOPcmuCodec' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioSessionDeactivated. This symbol was referenced by the managed member Twilio.Voice.iOS.CFunctions.TVOAudioSessionDeactivated. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOLocalAudioTrackStats. The symbol '_OBJC_CLASS_$_TVOLocalAudioTrackStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOBaseTrackStats. The symbol '_OBJC_CLASS_$_TVOBaseTrackStats' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOIceServer. The symbol '_OBJC_CLASS_$_TVOIceServer' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5214: Native linking failed, undefined symbol: _TVOAudioChannelsStereo. This symbol was referenced by the managed member Twilio.Voice.iOS.Constants.TVOAudioChannelsStereo. Please verify that all the necessary frameworks have been referenced and native libraries linked.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOAudioFormat. The symbol '_OBJC_CLASS_$_TVOAudioFormat' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOAcceptOptionsBuilder. The symbol '_OBJC_CLASS_$_TVOAcceptOptionsBuilder' could not be found in any of the libraries or frameworks linked with your application.
MTOUCH : error MT5211: Native linking failed, undefined Objective-C class: TVOCallOptionsBuilder. The symbol '_OBJC_CLASS_$_TVOCallOptionsBuilder' could not be found in any of the libraries or frameworks linked with your application.

This can be be related to #4 .
Any ideas how to have nuget package working ?

Hi, I have the same issue were you able to resolve it?

Bump

Same issue when I updated the package from 6.0.2 to 6.3.0, any update on this?  @pogarek @bmarrelli05

Bump


1.2.0 (2019-10-14)

(bluefox) Used MQTT 5.0
(bluefox) Support of js-controller 2.0 was added

Thank you for merging iobroker.vis-justgauge. Can you add me to npm owners for this repo too?

merged, published and added.


"Cannot look up subprotocol when connection is closing"

Trinity version: 190ab8a7d23b8b02e4998e89fdcf48cb1ac776ca

What is wrong?
Trinity syncs, but crashes after about an hour of activity with this exception which causes a cancellation to propagate up through everything:
ESC[1mESC[33m WARNING  2019-12-07 15:31:04,328           ETHPeerPool  Task <coroutine object BasePeerPool._periodically_report_stats at 0x7f0f10b99648> finished unexpectedly: PeerConnectionLost('Cannot look up s
ubprotocol when connection is closing')ESC[0m
   DEBUG  2019-12-07 15:31:04,328           ETHPeerPool  Task failure traceback
Traceback (most recent call last):
  File "/home/brian/trinity/p2p/service.py", line 158, in _run_task_wrapper
    await awaitable
  File "/home/brian/trinity/p2p/service.py", line 179, in _run_daemon_task_wrapper
    await awaitable
  File "/home/brian/trinity/p2p/peer_pool.py", line 537, in _periodically_report_stats
    if not hasattr(peer, "eth_api"):
  File "/home/brian/.local/share/virtualenvs/trinity-dwQEboUK/lib/python3.7/site-packages/cached_property.py", line 35, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/home/brian/trinity/trinity/protocol/eth/peer.py", line 86, in eth_api
    return self.connection.get_logic(ETHAPI.name, ETHAPI)
  File "/home/brian/trinity/p2p/connection.py", line 228, in get_logic
    if not self.has_logic(name):
  File "/home/brian/trinity/p2p/connection.py", line 224, in has_logic
    raise PeerConnectionLost("Cannot look up subprotocol when connection is closing")
p2p.exceptions.PeerConnectionLost: Cannot look up subprotocol when connection is closing

I can put up a more detailed log if that would be helpful.

After running across this I tried using v0.1.0-alpha.31 and haven't run into any issues, so it might have been introduced by the 22 commits which have been committed since then?
brian@pop-os:~/trinity$ git log --oneline v0.1.0-alpha.31..190ab8a7d23b8b02e4998e89fdcf48cb1ac776ca
190ab8a7 (upstream/master, master) Merge pull request #1355 from carver/quiet-broken-pipe-errors
ed6663d5 Merge pull request #1365 from cburgdorf/christoph/feat/matomo-tracking
0471b6e1 Add matomo tracking to docs site
6a2c5be7 Merge pull request #1364 from carver/mutual-peer-connection
a82c4325 Merge pull request #1354 from carver/squash-already-cancelled-logs
dd440fae Merge pull request #1356 from carver/cleaner-txpool-shutdown
aba4d023 Suppress BrokenPipeError from previews on shutdown
d454f9c0 Add exception suppressor, and bugfix the async one
05ae1758 Show PeerConnection instead of UnknownAPI on close
2b20b4b1 Avoid double-connection when peers mutually dial
80e96f2b Add repr and str to p2p.Connection
af451b51 Merge pull request #1360 from ralexstokes/fork-choice-updates-for-v0.9.2-cherry-pick
39a9e49d PR feedback for #1357
ee27212d Add stack trace to logs, when service cancelled 2x
9ff9e63e Silence warnings of connection "already cancelled"
be619f12 Add release notes #1356
12a6a739 show warning when skipping ahead for beam sync
29b9aebc Cleaner TxPool shutdown
53b98510 Add fork choice spec updates from v0.9.2
08892c96 Convert DiscoveryService to use the trio-based Service
121e2ccf Merge pull request #1357 from carver/just-linter-fixups
602b91cf Mysterious unrelated linter fixups
brian@pop-os:~/trinity$ git log --oneline v0.1.0-alpha.31..190ab8a7d23b8b02e4998e89fdcf48cb1ac776ca | wc -l
22

If it wasn't fixed before, it was probably fixed by one of #1545 or #1548 . I'll optimistically close, until it comes back again.


Unable to bind to rpc

I am getting the EOF-1 errors when trying to run the main app, so I tried to re-index and am getting the below unable to bind to rpc
2018-06-17 13:25:53 Binding RPC on address ::1 port 1979 failed.
2018-06-17 13:25:53 Binding RPC on address <IP_ADDRESS> port 1979 failed.
2018-06-17 13:25:53 Unable to bind any endpoint for RPC server
2018-06-17 13:25:53 Error: Unable to start HTTP server. See debug log for details.
2018-06-17 13:25:53 Shutdown: In progress...
2018-06-17 13:25:53 scheduler thread interrupt
2018-06-17 13:25:53 StopRPC: waiting for async rpc workers to stop
2018-06-17 13:25:53 StopNode()
2018-06-17 13:25:53 Shutdown: done
How do you allow it to bind to that port?

Yes, after enough restarts it was able to bind, it may be useful, if there is a conflict listing what the conflict is in future versions, as it is still unknown what was preventing it from binding.


FRONTEND-33. Feature: fixed code smells in sonar cloud.

Codecov Report

Merging #245 (b1ba345) into develop (c8cf355) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #245   +/-   ##
========================================
  Coverage    17.50%   17.50%
========================================
  Files          226      226
  Lines        20309    20309
  Branches       107      107
========================================
  Hits          3555     3555
  Misses       16718    16718
  Partials        36       36

Flag
Coverage Δ

web
17.50% <ø> (ø)

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

Continue to review full report at Codecov.

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


Index Select operator should work on unstructured meshes

Index Select operator should work on unstructured meshes.
Pictured is a render of my unstructured mesh. I would like to use index select on it so that instead of seeing all 1200 lines, I see fewer, so it is easier to visually parse. On the right-hand side of the image is the message VisIt displays when I attempt to do this.

Yeah, I agree. An UNstructured mesh can be viewed as a 1D logical index space and so we should support a 1D index select on UNstructured mesh.
If we got really creative, I wonder if we could concoct an UNstructured mesh that has some 2D logical index space as well. Maybe in the case of a fully logicaly disconnected UNstructured mesh of hexahedra (for example) we could have the first dimension for the elements and the second dimension for the nodes on each element defining a 2D logical indexing of the nodes, N x 8.
All that said, does your ensemble of line meshes here form a multi-block mesh in VisIt? If so, can't you use subset controls to turn some on and off? Yes, for 1,200 objects, an interface which requires you to individually check and uncheck each item you want on and off is nowear near as workable as one that allows you to specify some sort of striding (start, stop, stride). But, maybe it helps?

All that said, does your ensemble of line meshes here form a multi-block mesh in VisIt? If so, can't you use subset controls to turn some on and off?

This is all one mesh. I'm not exactly sure what a multi-block mesh is. I played around with subset plots and the controls there and couldn't make it draw any fewer lines.

Yes, for 1,200 objects, an interface which requires you to individually check and uncheck each item you want on and off is nowear near as workable as one that allows you to specify some sort of striding (start, stop, stride). But, maybe it helps?

I should note that this is a toy example of some of the output from the x ray image query, and my guess is that typical outputs will have probably 100x as many lines if not more.


array.forEach is not a function

build failure due to array.forEach iteration of files in version 1.1.15

It seems like the jslint expects an array some where but it is not an array there, for instance in jslint directives globals, it use to take an object but now if it an array of global is given then it works fine.


Icon next to "Short URL" label on stats page broken

The icon next to the "Short URL" label on the stats page shows as a broken image.
The generated HTML shows an empty IMG src.

The bug appears to be in the yourls_favicon function in functions.php.  If the $favicon variable isn't null, it just returns the value ... ignoring the $echo parameter.

Also in 2.0 something wrong may be in https://github.com/YOURLS/YOURLS/blob/2.0/yourls-infos.php:255
I have double favicon image url in short link on statistics page.
Short URL

Technical details

YOURLS version: 2.0
PHP version: 7.0.7

@seventhsite thanks for spotting!

I don't want to start new issue for a little error, I post here, if you don't mind.
In 2.0 on refferers tab of stats page link to "details" don't work. It's lines 487-488 in https://github.com/YOURLS/YOURLS/blob/2.0/yourls-infos.php.
Maybe wrong ID's, or empty href='', or spoiler need ... Don't know now.


AppAvailability always resolves to FALSE on Android (Twitter link in the menu)

As I was told on Twitter by @breningham 👍

I was using the wrong URL Scheme for Android. I should read my own READMEs 😚


Fix some typos

Hi,
I found some typos in the comment section of the following code and fixed it.
Please merge if you like.

src/ciphers/morse_code.rs
src/ciphers/transposition.rs
src/dynamic_programming/edit_distance.rs
src/dynamic_programming/egg_dropping.rs
src/general/nqueens.rs

Hello! Thank you very much for investing time in this project! I hope you had a good read 😁


embedsMany not casting fields properly?

I updated to the latest version before sending this in,
"php": "^7.1.3",
    "fideloper/proxy": "^4.0",
    "guzzlehttp/guzzle": "^6.3",
    "jenssegers/mongodb": "^3.5",
    "laravel/framework": "5.8.*",
    "laravel/telescope": "^2.0",
    "laravel/tinker": "^1.0",
    "optimus/api-consumer": "0.2.*",
    "votong/laravel-mongodb-passport": "^1.1"

I tried using an embedsMany relationship and saved some items into the db. When i retrieve them they are not casting object Ids and dates properly? Is there  step I am missing or is this the result of a collection being returned?
parameters -> is the embedsMany relationship.

@bytewoopy, $oid is standard mongodb representation for ObjectId. See docs

I've got this working now, I was probably not loading the relation when listing the results.

i having the same issue. can you show 2 model file for me? thanks you


Increment versions for kusto releases

Increment package versions for kusto releases

API change check
API changes are not detected in this pull request.


Add Docker build image instructions to README

Add instructions, to the README, for building an Ubuntu Docker image that has all (and only) the needed build dependencies.
I personally think this is helpful. Even if you don't actually want to build in a Docker (which I usually do these days), it's still useful to know the exact packages required.

Thanks, sorry for the delay, have had some things come up lately.
-[Unknown]


cgroup: extract cgroup rest api handler from daemon

Currently, the cgroup rest api handler is implemented as part of the daemon itself.
To further minimize the number of dependencies on the daemon, this commit extracts the cgroup rest api handler into the cgroup manager hive cell in package pkg/cgroups/manager.

/test

/test

rebased to main to resolve conflicts

rebased to main to resolve conflicts

/test


failing on import React

🐛 Bug Report
I am trying to create a basic test importing a component but it seems to fail on line 1 of the test trying to import React
To Reproduce
clone repo  https://github.com/shorif2000/elucidat-javascript-test
run jest
Paste the results here:
 FAIL  test/lib/Seatmap.test.js
  ● Test suite failed to run

    /opt/react-seatmap/test/lib/Seatmap.test.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){import React from 'react';
                                                                                ^^^^^

    SyntaxError: Unexpected identifier

      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:537:17)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:579:25)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.22s
Ran all test suites.

i was missing the following
"plugins": [
    ["@babel/plugin-transform-modules-commonjs", {
      "allowTopLevelThis": true
    }]
  ]


Production Error "Uncaught TypeError: r.emit is not a function"

I'm currently seeing a production-only TypeError when initializing the SDK. Local doesn't have any issues but when transpiling and bundling using browserify (+ babelify + uglify), the error pops up.
Here is the type error that is being thrown in the console:
purecloud-webrtc-sdk.js:1 Uncaught TypeError: r.emit is not a function
    at XMLHttpRequest.t.onreadystatechange (purecloud-webrtc-sdk.js:1)
t.onreadystatechange @ purecloud-webrtc-sdk.js:1
XMLHttpRequest.send (async)
u._end @ purecloud-webrtc-sdk.js:1
u.end @ purecloud-webrtc-sdk.js:1
(anonymous) @ purecloud-webrtc-sdk.js:1
t @ es6-shim.js:1663
r.then @ purecloud-webrtc-sdk.js:1
value @ purecloud-webrtc-sdk.js:1
value @ index.js:36                        // The line here is _sdk.intitialize()_
commitLifeCycles @ react-dom.production.min.js:159
w @ react-dom.production.min.js:185
_ @ react-dom.production.min.js:182
y @ react-dom.production.min.js:181
b @ react-dom.production.min.js:181
h @ react-dom.production.min.js:180
c @ react-dom.production.min.js:179
t @ react-dom.production.min.js:190
updateContainer @ react-dom.production.min.js:191
nn.render @ react-dom.production.min.js:228
(anonymous) @ react-dom.production.min.js:242
unbatchedUpdates @ react-dom.production.min.js:188
sn @ react-dom.production.min.js:242
render @ react-dom.production.min.js:244
354.../platform/teams/u4b/utils/constants @ routes.jsx:252
o @ _prelude.js:1
(anonymous) @ _prelude.js:1
366../components/routes.jsx @ main.js:48971
o @ _prelude.js:1
e @ _prelude.js:1
(anonymous) @ _prelude.js:1

If I manually go into node_modules/purecloud-webrtc-sdk/web/purecloud-webrtc-sdk.js to format the file with Prettier for a better stack trace, it looks like the line that's being thrown is here:
        (_.prototype._end = function() {
          var t = this,
            e = (this.xhr = l.getXHR()),
            n = this._formData || this._data;
          this._setTimeouts(),
            (e.onreadystatechange = function() {
              var n = e.readyState;
              if (
                (n >= 2 &&
                  t._responseTimeoutTimer &&
                  clearTimeout(t._responseTimeoutTimer),
                4 == n)
              ) {
                var r;
                try {
                  r = e.status;
                } catch (t) {
                  r = 0;
                }
                if (!r) {
                  if (t.timedout || t._aborted) return;
                  return t.crossDomainError();
                }
                t.emit("end"); // LINE 831 This is where emit is not a function
              }
            });
          var r = function(e, n) {
            n.total > 0 && (n.percent = (n.loaded / n.total) * 100),
              (n.direction = e),
              t.emit("progress", n);
          };

I'm still debugging and trying to isolate the issue but was wondering if you had seen this before and knew what the issue is.

I'm not having luck reproducing this one so far. Do you have your browserify config set up to ignore uglifying this module, since it's already bundled? We could publish an unminified/uglified version if it's a problem with that, which I've seen before on other modules. Alternatively, you could import from source instead of the bundle. I'll keep working on a repro of this and get back to you if I find anything.

Hey @xdumaine I was able to repro using<EMAIL_ADDRESS>which is the currently what our build tools are using. I was also able to repro the issue with gulp-uglify and purecloud-webrtc-sdk on a clean project as well.
I haven't done any deeper digging to see if this is a bug with<EMAIL_ADDRESS>but thankfully we have a work around by importing the CDN library using the <script> tag.

Cool, thanks for the extra info, I'll work up a sample.

That was it. I did a sample with<EMAIL_ADDRESS>and got the error. With latest 2.x and 3.x branches of gulp-uglify, I do not get the error. I would look into upgrading gulp-uglify.

Thanks for testing the other versions! I appreciate the quick response and support. It is unfortunately time-prohibitive for us to upgrade at the moment. But I'll look into upgrading in the future. Thanks!

Sounds good.


Could not create SSL/TLS secure channel

1. Describe the bug
Error consuming external web service from BC SaaS.
The web service security protocol is TLS 1.2.
2. To Reproduce
When executing the call to the web service, the system always returns the error while sending the request:
"The request was aborted: Could not create SSL/TLS secure channel."
AL Code (invoke the web service):
local procedure SendToWS(RequestUri: Text; var IStream: InStream)
var
    vHttpClient: HttpClient;
    vHttpContent: HttpContent;
    vHttpHeaders: HttpHeaders;
    vHttpRequest: HttpRequestMessage;
    vHttpResponse: HttpResponseMessage;
    ResponseText: Text;
begin
    vHttpContent.WriteFrom(IStream);
    vHttpRequest.Content(vHttpContent);
    vHttpRequest.Method('POST');
    vHttpRequest.SetRequestUri(RequestUri);
    vHttpRequest.GetHeaders(vHttpHeaders);

    vHttpHeaders.Remove('ContentType');
    vHttpHeaders.Add('KeepAlive', 'TRUE');
    vHttpHeaders.Add('Method', 'POST');
    vHttpHeaders.Add('ContentType', 'text/xml; charset=utf-8');
    vHttpHeaders.Add('Timeout', '20000');

    vHttpClient.Send(vHttpRequest, vHttpResponse);

    if not vHttpResponse.IsSuccessStatusCode() then begin
        vHttpResponse.Content().ReadAs(ResponseText);
        Error(ResponseText);
    end else
        Message('ok');
end;

3. Expected behavior
I hope to get the response status: 200 OK, as with other tools used successfully (SoapUI and Postman) and also from Business Central OnPremise.
4. Actual behavior
The system returns the following error:

5. Versions:

AL Language: 7.2.475273
Visual Studio Code: 1.57.0
Business Central: 18.1 (Platform 18.0.24800.26638 + Application 18.1.24822.24906)

Similar duplicate issues: #5405, #1414.  Don't seem to be related to the development environment.
For quicker resolution on runtime issues the best is to open a support ticket.


[BUG] upgrade stuck on create upgrade VM

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Install Harvester with at least 4 nodes
Create an image for VM creation
Create VLAN
Import Harvester into Rancher 2.7.4
Create RKE1 and RKE2 cluster (1 node)
Install Harvester Cloud Provider and CSI Driver
Create DHCP load balancer service (only RKE1 cluster)
Create Nginx deployment with new Harvester PVC
Upgrade Rancher to 2.7.5
Upgrade Harvester

Expected behavior
Environment:

Harvester ISO version: v1.1.2 -> v1.2.0
ui-source Option: Auto
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Baremetal DL360 4 nodes
Rancher Version: v2.7.4 -> v2.7.5

Additional context
v1.2.0 Release testing for 4 nodes upgrade
Support bundle
supportbundle_eb8adc43-b57e-4e00-99b9-cb77dd4c07fb_2023-07-17T14-59-30Z.zip

The root cause can be found below:

We meet the known issue (https://github.com/longhorn/longhorn/issues/4305) for a stopped replica. (Backing image sync fail)
The engine needs to wait for whole replicas to be running, so that is why it is stuck on creating an upgrade VM (because one replica is stuck due to the above reason).

It looks the bug has been fixed in newer version, for users upgrading from old version, we might need document a workaround to continue the upgrade.

Doc merged: https://docs.harvesterhci.io/dev/upgrade/v1-1-2-to-v1-2-0#2-an-upgrade-is-stuck-in-creating-upgrade-repository


curl: 7.88.1 -> 8.0.1

https://curl.se/changes.html#8_0_0
https://daniel.haxx.se/blog/2023/03/20/curl-8-0-0-is-here/
https://daniel.haxx.se/blog/2023/03/20/curl-8-0-1-because-i-jinxed-it/
Fixes: CVE-2023-27533, CVE-2023-27534, CVE-2023-27535, CVE-2023-27536,
CVE-2023-27537, CVE-2023-27538
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.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

[x] Fits CONTRIBUTING.md.

As a birthday present, various version-detection got broken by the major bump.  (no surprise, really)
@fabaff (maintainer): cameradar vendors a dependency that breaks on this: https://hydra.nixos.org/log/3k7vm7bjh2g47b4bnyl1xlirb035f4zc-cameradar-5.0.2.drv

Btw I marked cameradar as broken with an explanation here: #235040


[deployer] Explicitly enable the cluster-autoscaler when generating the support chart for aws clusters

Fixes https://github.com/2i2c-org/infrastructure/issues/3881

Thanks @consideRatio ❤

I assume there may be related docs that is now outdated, but we can merge this as it is also as i expect no significant negative consequence of that.

So, actually there is no info in our docs about this anymore and this is why I had trouble when deploying the dandi hub 😅  I believe it was removed under the assumption it is already being autogenerated by the deployer, which is what this PR does.
We do have outdated info about the support chart, mainly about the need to manually generate its contents, but that is tracked in https://github.com/2i2c-org/infrastructure/issues/2187

I'd cancel the job to deploy on merge this PR btw

Good idea. I will try to also exclude the template files from the worflow too. Commit incoming.


Feat: Create GitHub Action

Description
It would be really helpful to add a GitHub action that will cover the following:

Check lint issues
Help with updating forked repo

#10 refers this issue too
Here are some of the links from that issue:
Enable Dependabot security updates: https://github.com/nhcarrigan/code-snippets/blob/main/optional-workflows/dependabot.yml
Enable Node.js CI: https://github.com/nhcarrigan/code-snippets/blob/main/optional-workflows/node-ci.yml

#54 added the node CI

Thanks for raising @Rahma-AlWadhahi 👍
PR for this merged in #54, I will close


Single quote in PR title breaks the get-pr-info variables

Scenario
Given PR title:

ASDF-111 don't use single quotes in PR title

When using step ghpr/get-pr-info and accessing $GITHUB_PR_BASE_BRANCH variable
Then variable is an empty line and error message is printed:
/tmp/.bash_env-5ff85b683890bc490b71ca94-0-build: line 4: unexpected EOF while looking for matching `''
/tmp/.bash_env-5ff85b683890bc490b71ca94-0-build: line 5: syntax error: unexpected end of file

Details
I've noticed that exporting variables in get-pr-info command is not escaping the contents:
PR_TITLE=$(echo $PR_RESPONSE | jq -e '.title' | tr -d '"')
echo "PR_TITLE: $PR_TITLE"
echo "export GITHUB_PR_TITLE='$PR_TITLE'" >> $BASH_ENV

In my case this resulted in a corrupted env file:
echo $BASH_ENV
/tmp/.bash_env-5ff86be9b2b77a7bed582e27-0-build
cat $BASH_ENV
export GITHUB_PR_NUMBER=1568
export GITHUB_PR_TITLE='ASDF-111 don't use single quotes in PR title'
export GITHUB_PR_BASE_BRANCH='develop'
export GITHUB_PR_AUTHOR_USERNAME='jakub'

I think single quotes should be escaped to avoid such issues.

Oh whoops, sorry I didn't see this earlier! I'll put in a fix shortly.

Hmm, so I'm not sure it's possible to do this actually. Or at least not without changing the outer enclosing ' to ", which I'd prefer not to do to prevent variable expansion.
From the GNU docs:

Enclosing characters in single quotes (‘'’) preserves the literal value of each character within the quotes. A single quote may not occur between single quotes, even when preceded by a backslash.

Open to suggestions though!

What about removing single quotes with sed?

Please have a look: https://github.com/NarrativeScience/circleci-orb-ghpr/pull/10

This is still a problem when multiple single quotes are passed to the PR title or commit messages.
I have a PR up that should fix the problem: #15


Record server CA during registration

This ensures a more secure and correct connection from the syncer to the workspace.
I ran into certificate issues when deploying pipeline-service into an existing kcp instance in OpenShift.

Couldn't test it completely as of yet. Cluster expired and ckcp CI would only be able to test it once we add openssl to it. Is it somehow needed for Openshift 4.11?

This was needed for OpenShift 4.10 in my case.

/ok-to-test

Failures appear to be unrelated to the change?

Shellcheck failure is related to the change and mentions the line that generate the error.

For the ckcp-test, please make sure you have rebased against main. If you have rebased and still see the issue, please contact @xinredhat to get support on the CI.

/test plnsvc-ckcp-test

@lcarva Here is the snippet from the logs. Please fix this.
In ./images/kcp-registrar/register.sh line 233:
    ca_cert="$(openssl s_client -showcerts -servername ${servername} -connect ${host} </dev/null | openssl x509 | base64 -w 0)"
                                                       ^-----------^ SC2086: Double quote to prevent globbing and word splitting.
                                                                              ^-----^ SC2086: Double quote to prevent globbing and word splitting.
Did you mean:
    ca_cert="$(openssl s_client -showcerts -servername "${servername}" -connect "${host}" </dev/null | openssl x509 | base64 -w 0)"

@lcarva Your PR now has conflicts, please rebase when you fix the shellcheck issue.


Cleaning up objects from S3 (Textract)

Not a direct issue, but after running into issues with the local PDF parser I quickly went on to use Textract. All works fine after the initial set up, but now I'm wondering what to do with the objects in the bucket. As they are used for a one time operation (Textract extract) maybe it would be nice if the objects get deleted after the operation. Not sure what you think. This can onviously also be achieved in other ways (cron, etc.).

Hmmm i guess we could do this in the package, however, setting up a lifecycle rule on the (textract-specific) bucket that automatically deletes them after 1 day might be more desirable.
As long as that is documented somewhere and people are aware of it, having to setup a lifecycle rule on the bucket seems to me like the better approach.
Needs some time to think about this.
Opinions and suggestions welcome.

Agreed on the lifecycle rule. Some people might actually have use cases where they want to keep the files as well. I’ve also been thinking about a way where you might want to reuse a file from the bucket. Again, not sure if this should be part of the package.

Currently the filename i think is randomly generated from an UUID, which might make it impractical to do the cleanup  manually, maybe providing a callback or flag to simply delete the file once it's done is fair enough.
Made a PR with a draft implementation, feedback welcome.

Merged in the PR


Autodeploy on tags using Travis CI

http://docs.travis-ci.com/user/deployment/npm/
This allow anyone with commit access to make releases, and ensures they are tested by Travis CI

Implemented as of https://github.com/the-grid/imgflo-url/commit/96a49dc451cb57081ae78611ad532bbfbb7e773e


extract device id blob and private key from android phone

plz can you help give me idea hint on how to extract device id blob and private key from android phone

@HackerBoy0412 i am not making jokes to laugh if you dont want help go away

The thing is, no one will give you a tutorial public.
So try somewhere else

@HackerBoy0412  so give can you guide me to the right place or give it to me you seem to have an idea

No i havent.
Its will not come public

@HackerBoy0412 where somewhere i google it for a lot of days and no info and i know they have a discord to share info but how to get there

they have a lot of fear give public , fear the method is down and they can't earn money any more.,
because they earn money a lot of money PIRATE MOVIES AND SERIES
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS
they are criminals who deserve to be reported, all in this github are CRIMINALS

https://www.youtube.com/watch?v=v_fRnpIJx9I
For example, here you have a group of discord, dedicated to pirating movies and series with the widevine method. and you have users who claim to have their own bat to download.
what you can do is identify them and find a lawyer, and threaten them, if they do not share the method then proceed with your lawyer and give explanations to a judge for what purpose they want and use the method to download movies and series in a way illegal. threaten them and they pay in justice


UCP: Support code highlight and auto format SQL

UCP: #98

hi @qxhy123 , can you attach the screenshot for the result so we can see how the effect looks like?

here is screenshot of the effect:

hi @qxhy123 , the rest LGTM, please help fix the format changes, you can run yarn format to do that.

Looks cool in detail page:

Looks like we can further change to use sql-formatter-plus so that keywords can be formatted as uppercase using uppercase: true. Do you have interests to open a PR?

@breeswish yea, sure


run bundle(-upgrade): configure service account and secret

Description of the change: This PR adds --service-account to run bundle and run bundle-upgrade to bind registry objects to a non-default service account. It also adds --secret-name to run bundle and run bundle-upgrade to configure the registry Pod with an in-cluster docker config Secret to pull bundle images from private registries.
Motivation for the change: private registries containing bundles should be accessible by opm within the ephemeral registry pod. The service account can be used to configure image pull secrets for private index images.
Closes #4650
/kind feature
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[x] Add or update relevant sections of the docs website in website/content/en/docs

This still needs testing, as I haven't had the time to set up a local docker registry and configure a k8s cluster to pull from it.
/hold

/ping @AlexNPavel

This has been tested and works, provided the service account attached to the registry Pod specifies the same image pull secret.
/hold cancel

/retest

Going to follow up with a CA file flag once opm registry add gets a corresponding flag.

/retest


cover jest import mock with a fake declarations

jest.mock('./bla/bla.module.ts')
The idea is to test how it behaves and to add a proper error message which would lead the developer to the proper solution.

v12.3.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


[Question] Additional imlpementations

Hey,
When will all the new or "old" stuff be re-added to the mod?
Is there any specific ETA or hopeful one right now?

The old stuff will be done through a different mod. https://github.com/BuiltBrokenModding/ICBM-Classic

It will actually be several mods, concrete and decorations will be military base decor.
Radar will be re-added as a basic functionaliy, but will be re-implemented in a large mod dedicated just towards that feature.
Redstone stuff will be re-added threw Addicted to red, which already has the detector block.
Guns will be added to the armory mod.
Everything else in its original way will be recreated in the classic version as dmod already said. This will be a complete replication of the 1.6.4 version.

Ah, how long would that be (I know modders hate being asked this).
I was hoping maybe sometime mid June?
I have to pump out a pack version with some requirements for ICBM soon.
-Power consumption.
-Larger Missile stable.
-ABMs or countermeasures.
-Radar/tracking ability.
-Maybe cooler textures?
Atleast i feel those things are necessary, im happy with the progress but those are critical to my pack.
Keep up the good work and try your best to possibly get some of those things done sooner 👍

no clue, since this is not a job I have no ETA's and have found estimating them never works. I can say that large missiles will not happen anytime soon. As these will require so many backend features in order to function correctly. As these missiles are 60+ blocks tall and can remove kilometers of the map in one go. The ABMs and power will be next on my list to work on, right after the overhaul of VoltzEngine which I just started this week.
Textures are almost 100% dependent on the asset team.
Radar is already implemented and I'm waiting on radar dish models from the asset team.


Google Play Store Description Shows incorrect required Server Version

The application page in the Google Play store for v1.1.0 says "To use this app, you need the URL for a v3.7+ Mattermost server."  The Changelog on github for v1.1 says "Server Versions Supported: Server v3.10+ is required".  Trying to connect to a v3.9 server with the v1.1 android app gives a server version error.  The Play Store description should be updated to reflect the new server version requirement for the latest version of the app.

Thanks!
I think this should be fixed now, turns out it was being overwritten by our automation for submitting to play store. I've added a note to our process to update this file to make sure it doesn't happen again.


Add support for countries' social media accounts

The feature
Maybe inside #102?
Why?
No response
Examples and inspiration
This is more something for the “sub-countries”/Federal subjects, but since they use the same template. 😉

I think you need to explain the feature a bit more, as I don't really understand what you want to do.


Support equal_nan parameter for unique

Part of https://github.com/cupy/cupy/issues/6821, following a change of NumPy 1.23.
https://numpy.org/devdocs/release/1.23.0-notes.html#new-parameter-equal-nan-added-to-np-unique
Note that numpy.unique seems to have a bug that it returns wrong results when it takes float16 ndarrays with one or more nan.

https://github.com/numpy/numpy/issues/21838

/test mini

I can't tell what's wrong with Jenkins here, but it'd be nice to get this PR merged so that I can fire another array-api change 🙂


Add user questions to docs pages owned by SIG UX

Sorry for the lack of context! The epic is https://github.com/giantswarm/giantswarm/issues/13270
We plan to have a special sort of search function in docs that helps users find _the_article answering their questions. That's what we need those for.


Updated legend in grafana Liqo dashboard

Updated description in some panels of the Grafana dashboard, to make it more clear.

/rebase


Update google/apiclient

@JoeDawson Needs update google analitycs package - and it requires google/apiclient newest version of requirements your package. How can resolve that?

Hey @aios,
I'm currently looking into rewriting this package ahead of the Laravel 5.5 release. google/apiclient will be updated to ^2.1 with this.

Hey @aios,
This package has now been updated to use<EMAIL_ADDRESS>

i still get
Call to undefined method Google_Client::setClassConfig()

@jimmyt2003 are you on the latest version? What version of Laravel are you using?

version 3.0.1 and laravel 5.4
seems to not like the line
$this->client->setClassConfig('Google_Http_Request', 'disable_gzip', true);

if i comment it out it gets to the google auth then errors again. Maybe it is something I am doing wrong i have used this library in the past about a year ago without issue. I will try uninstall everything and try again
Thanks!

Thanks @jimmyt2003 let me know how you get on.
I did a lot of testing over the weekend with 5.4 and the latest and for the most part everything went smoothly.
Look forward to hearing from you!

when I downgrade to 2.0.1 it works ok but on anything higher i still just get
Call to undefined method Google_Client::setClassConfig()


New Icons Request

We'd like to have Nigeria currency Icons please

Search "Nigeria" in new version 2.94

Thanks Richard  Sent from Mail for Windows 10 From: RichardSent: 02 April 2024 04:44 AMTo: Richard9394/MingCuteCc: Charles Usa; AuthorSubject: Re: [Richard9394/MingCute] New Icons Request (Issue #36) Search "Nigeria" in new version 2.94—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


fix django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet.

this PR contains:

fix for django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet.

@Andrew-Chen-Wang i created this PR to address your comments on https://github.com/SimpleJWT/django-rest-framework-simplejwt/pull/175

Thanks, @hishamkaram .  Looks good!

@Andrew-Chen-Wang @davesque how can i help to merge this PR?

Thanks for all the work on this, guys!


proxy: 429 Too many requests

Motivation
Control plane returns this error code when it's overloaded with requests. Proxy should listen to this and enable a global throttling of requests to get_auth_info and wake_compute.


[SPARK-17964][SparkR] Enable SparkR with Mesos client mode and cluster mode

What changes were proposed in this pull request?
Enabled SparkR with Mesos client mode and cluster mode. Just a few changes were required to get this working on Mesos: (1) removed the SparkR on Mesos error checks and (2) do not require "--class" to be specified for R apps. The logic to check spark.mesos.executor.home was already in there.
@sun-rui
How was this patch tested?

SparkSubmitSuite
On local mesos cluster (on laptop): ran SparkR shell, spark-submit client mode, and spark-submit cluster mode, with the "examples/src/main/R/dataframe.R" example application.
On multi-node mesos cluster: ran SparkR shell, spark-submit client mode, and spark-submit cluster mode, with the "examples/src/main/R/dataframe.R" example application. I tested with the following --conf values set: spark.mesos.executor.docker.image and spark.mesos.executor.home

This contribution is my original work and I license the work to the project under the project's open source license.

Can one of the admins verify this patch?

ok to test

@srowen LGTM

This might not handle packages the best it can on mesos but that could be a follow-up
@sun-rui

@felixcheung There are separate issues for packages that are not included in this PR:

Support using 3rd-party R packages on Mesos
Support Spark packages with R code on Mesos

Also, I could not find any documentation about packages. It would be helpful to fill in some information about requirements or testing instructions in the links above.

Test build #3407 has started for PR 15700 at commit 11b6568.

Test build #3407 has finished for PR 15700 at commit 11b6568.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

retest this please

Test build #3413 has started for PR 15700 at commit 11b6568.

Test build #3413 has finished for PR 15700 at commit 11b6568.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Jenkins add to whitelist

Jenkins retest this please

Test build #68200 has started for PR 15700 at commit 11b6568.

Test build #68202 has started for PR 15700 at commit 11b6568.

Test build #68200 has finished for PR 15700 at commit 11b6568.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68200/
Test FAILed.

Test build #68202 has finished for PR 15700 at commit 11b6568.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68202/
Test PASSed.

Merged build finished. Test PASSed.

Merged to master/2.1


go-redis v7 7.2.0 with redis 5.0.8 seems to hit a dead lock

We do not see this problem with 5.0.4 redis
Sample code:
// create cluster client
rdb := 	redis.NewClusterClient(clusterOptions)

// spawn of 10 concurrent writers
for i := 0; i < 10; i++ {
  go func() {
    defer wg.Done()
    ci.processEvents(rdb)
  }()
}

in 5.0.8 we see a dead lock. We see exactly 10 go-routines waiting on github.com/go-redis/redis/v7/internal.(Once).Do
go-redis-issue.log
Links to:  https://github.com/go-redis/redis/issues/1313

I don't think this is really a deadlock, nor do I think this is related to #1313. I think this is just that in the current implementation internal.Once could hold the lock for excessively long period time when there are problems talking to the cluster, making it look like a deadlock.
The rest is copied from the discussion on #1404 (cc @rbg @vmihailenco ):
We recently started to use go-redis to do some high volume requests, and observed occasional error/latency spikes. We tried redispipe and never observe any error/latency spike with the same usage. We do prefer go-redis overall feature wise (redispipe works better for this use case, but for it's lacking on some other, nor generic use cases), and want to avoid the situation of using two different redis libraries for different use cases in the same code base.
The retries on the errors here, is an example of go-redis code making a bad situation worse, as the reason I described above. I can understand the rational of "we only want to cache successes, not errors". But the current implementation could result in a lock being held for excessively long period of time. I don't know what's a good fix right now. Maybe we can make the timeout in ClusterClient.cmdsInfo more configurable? That's just a random idea.
Also we tried to apply this patch, and still observed the error/latency spikes (but probably less frequent), so this is not the only problem we hit.

I wonder if you use ReadTimeout and WriteTimeout options? They allow to configure how much time cmdsInfo spends waiting for a response from Redis. There should not be a deadlock, but ClusterClient won't process commands until commands info is available. That seems okay because ClusterClient can't properly route commands without that info.

I wonder if you use ReadTimeout and WriteTimeout options? They allow to configure how much time cmdsInfo spends waiting for a response from Redis. There should not be a deadlock, but ClusterClient won't process commands until commands info is available. That seems okay because ClusterClient can't properly route commands without that info.

We used to set read/write/connect timeouts, but we stopped doing that and switched to rely solely on context timeout.
For most of the operations, this is the function to actually decide which timeout to use: https://github.com/go-redis/redis/blob/f2645d373d0ec8b2bcb48417be8c835ce353b7b3/internal/pool/conn.go#L106
Basically that function pick the earlier one between read/write/connect timeout and the context timeout as the timeout to use. This is the perfect solution, as in reality, context timeout is set for the whole request (function call to the go-redis library), while read/write/connect is set for a single read/write/connect operation. For example, we give every request to take up to 500ms, we would set the context timeout to be 500ms to reflect that. If the read/write timeout is something lower than that (say 200ms), that poses a risk that we would abandon the request prematurely (for example, that read/write request actually takes 250ms to finish, which is still acceptable under the 500ms per request timeout, but would be overridden by the 200ms read/write timeout and cause timeout error). As a result, if we would need to set a global, read/write/connect timeout, that need to be longer than the longest pre-request timeout the whole code allows.
We could set a long read/write/connect timeout for that case to use, which would improve the situation, but since that's trying the requests in a for loop, for every failed requests, it would still cause the lock to be held for a long period of time (but would be shorter than no timeout set).

it would still cause the lock to be held for a long period of time

Without commands information we can't properly route commands - go-redis will send commands to random nodes. It is not the end of the world, but introduces another failure mode...
What do you think about trying 3 random nodes in https://github.com/go-redis/redis/blob/master/cluster.go#L1544-L1569 ? This will limit the time we wait for commands to load.

We used to set read/write/connect timeouts, but we stopped doing that and switched to rely solely on context timeout.

This explains the "deadlock" though you must be rather unlucky to reliable trigger it :)
You still have to set ReadTimeout and WriteTimeout to some relatively high value. go-redis will pick the lowest value between context and ReadTimeout. It is useful for case like this when context.Context does not have deadline.

What do you think about trying 3 random nodes in https://github.com/go-redis/redis/blob/master/cluster.go#L1544-L1569 ? This will limit the time we wait for commands to load.

Yea I think that would help.
To clarify we didn't really get "deadlock" (I'm not the OP reporting this issue). We got occasional error and latency spikes. I don't believe we actually hit this particular one, the one we hit are more likely the lock in the client pool. I was just going through the locks used in this library and noticed that this one could be held for long period of time. The lock in client pool might need some more work to fix, e.g. use channels instead of slice+lock to implement the pool.
But I agree that we still need a high value of read/write timeout, otherwise if we hit this one it will be very long.

@shrivardhan92 you should try v7.4.0 and make sure to set Read/Write timeouts properly.


Implement graph matrix using adjacency list

Finished implementation in commit https://github.com/Tomkay94/java-datastructures/commit/43baaaea193b4ef9566f9e70b4860ee4abaf0cd1


fixing suppression when margins is true

This is to fix the crosstab. An issue was raised that when suppression is true and margins is true the cells that violate the rules are not being suppressed.
The problem stemmed from the process of recalculating totals. Initially, the function applied a list of boolean conditions for each true cell in the suppression masks to the dataset and then generated the crosstab. However, it missed applying suppression after calculating the crosstab. This was added in this PR.

@mahaalbashir shouldn't there be a test case for this?


Chaid model result always return 0 accuracy

I'am using chefboost for CHAID algorithm. Dataset contains 10000 rows and 7 columns and fit object always return 0 accuracy. What could be causing this, can you help me?
I also want to visualize tree graph how can I do this?
config={"algorithm":"CHAID",'enableParallelism': False}
model=cb.fit(df.loc[:10000,independent_variable_columns],config,target_label='Decision')
CHAID  tree is going to be built...
finished in  6.883694887161255  seconds
Evaluate  train set
Accuracy:  0.0 % on  10001  instances
Labels:  [0 1]
Confusion matrix:  [[0, 0], [0, 0]]
Precision:  0.0 %, Recall:  0.0 %, F1:  0.0 %

Would you please share dataset?

oh if the column types are actually classes instead of real numeric values, then you should make them string.
e.g. instead of 0, make it class_0.
Otherwise, chaid will build trees with rules being greater or less than a threshold value as explained in this video: https://www.youtube.com/watch?v=YTC7sTXDmWk

The problem was solved, the model worked thank you so much :)
How can I visualize the tree ?

trees cannot be visualized with chefboost, you can just check the output .py files out.

This is so sad. Do you know any other lib for visualization using rules.py file?

nope


Feedback for Check application performance

Need more information about what we count as a record for the API Performance columns Count vs Record Count

One entity = one record, meaning one company, contact, project, sale with all corresponding rows like the address, phones, list items linked to it and so on.
Asking for a list however, will count each list item as one database record. So calling the country list will count each individual country in the list as one record.
Caching information that is rarely changes will have an impact, like all lists available in the settings and maintenance client..

Document Details
⚠ Do not edit this section. It is required for docs.superOffice.com ➟ Docs Team processing.

Content Source: dp-metric-performance

Reuse text from developer-portal/analytics/metered-services.


Fixes issues with running any kind of test with MacVim

What?
vim-test checks to see whether colours should be enabled in test output using the 'test#base#no_colors' function.
This function short-circuits when the current version of vim doesn't have the gui_running feature enabled (I imagine this is the majority of cases for users of vim (?))
function! test#base#no_colors() abort
  let strategy = get(g:, 'test#strategy', 'basic')
  return has('gui_running') && strategy ==# 'basic'
endfunction

I'm currently using MacVim for a trial and am hitting the second check (i.e. strategy ==# 'basic') since has('gui_running') is true.
However the strategy returned by the following call:
 let strategy = get(g:, 'test#strategy', 'basic')

is not a string and is in fact a dictionary object containing all of the strategies defined for the available run types (i.e. file, nearest and suite):
{'file':'iterm', 'nearest': 'iterm', 'suite': 'iterm'}

This PR fixes the failure arising from comparing a dictionary to a string:

Why?
So that I can use vim-test with MacVim without it exploding :).
A/C
Make sure these boxes are checked before submitting your pull request:

[ ] Add fixtures and spec when implementing or updating a test runner
[ ] Update the README accordingly
[ ] Update the Vim documentation in doc/test.txt

I wanted to do a couple of things:

Understand what 'basic' should be compared against (i.e. for which run type - nearest, file or suite) - I'd like there to be a strategy available for the current run if at all possible.
Mock out the has('gui_running') functionality so that I can check the behaviour of this method in a base spec file to avoid this happening again in future.

Does this behaviour only occur with MacVim?

For me, yeah. I don't think neovim/vim return 1 for has('gui_running') in any circumstance and I'm not familiar with other graphical vim implementations

I think I just answered question 1 above. I'm now passing the run strategy to the build_args for each runner which means the checks will work with MacVim now :).

I appreciate that it ended up needing an interface change so seems bigger than it actually is.

I've just downloaded MacVim to see if I can recreate the issue and when I run echo get(g:, 'test#strategy', 'basic') I just get a string not a dictionary. Are you assigning test#strategy anywhere?

This is my configuration, yeah:
"""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""
" => vim-test
"""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""
if has('gui_macvim')
  let test#strategy = {
        \ 'nearest': 'iterm',
        \ 'file':    'iterm',
        \ 'suite':   'iterm',
        \}
else
  let test#strategy = {
        \ 'nearest': 'neovim',
        \ 'file':    'dispatch',
        \ 'suite':   'dispatch',
        \}
endif

let test#ruby#rspec#executable = "bundle exec rspec"

nnoremap <silent> <Leader>x :TestNearest<CR>
nnoremap <silent> <Leader>t :TestFile<CR>
nnoremap <silent> <Leader>r :TestSuite<CR>
nnoremap <silent> <Leader>e :TestLast<CR>
nnoremap <silent> <Leader>l :TestVisit<CR>

I was following the documentation for using different strategies: https://github.com/janko/vim-test/commit/a8fc25c143d4b9417a744cdf3af81120d6390e22

Thank you for your work on this and your patience. Seems reasonably sound though I wonder if there is a way to avoid passing the strategy to every runner's build_args - @janko do you know if this is possible?

No worries codeinabox :). We could get more MacVim users using vim-test \0/

Any thoughts on merging this @janko? Is really just a bug fix for a certain combination of config and MacVim.
Doesn't break/change the interface in a way that hasn't been updated/validated by running the tests.

This change will break any custom-made runners that people have created locally (because the build_args function is now given two arguments). I also don't like the fact that this introduces coupling between runners and strategies, because now runners would be able to generate different parameters based on the strategy. I know runners are not doing it at the moment, but this change would allow it, and that would introduce a slipper slope.
It would be better if we first determined whether colors should be generated, and then called build_args with that boolean value as the second argument (but we would also handle build_args accepting only one argument for backwards compatibility). That way the runner knows only what it needs to know – whether it can use colors.

So AFAIK inside of the test#base#no_colors function we can do one of:

Pass in the run type and draw on a function that pulls out the strategy config for the specific run type (breaks custom runners)
Pass in the strategy (breaks custom runners)
Set/unset a variable that is in lexical scope of the entire run which specifies the type and/or strategy (doesn't break custom runners as no change in test#<language>#<runner>#build_args is required). I'm guessing this would be us using a global variable :scream: or equivalent?

How does the global variable approach sound @janko?

This was superseded by https://github.com/janko/vim-test/pull/429


auth-backend: revert jose to v1

Reverting jose back to v3 for the time being, since it has a minimum requirement of Node 14.
@backjo got no way of verify the alb changes ^_> are you able to have a look?
We do want this included in todays release so may end up having to break the alb auth in favor of introducing the Node v14 requirement without warning.

@Rugvip just seeing this change - will test the ALB provider and see what changes need to be made, if any.

@Rugvip just seeing this change - will test the ALB provider and see what changes need to be made, if any.

@Rugvip just curious - where was the issue with Node 12? Docs on the jose git repo seemed to suggest that it should support Node 12 LTS.

@Rugvip just curious - where was the issue with Node 12? Docs on the jose git repo seemed to suggest that it should support Node 12 LTS.

It's because using the esmodule import syntax like this import parseJwk, { JWK } from 'jose/jwk/parse' doesn't work on node12 yet.

It's because using the esmodule import syntax like this import parseJwk, { JWK } from 'jose/jwk/parse' doesn't work on node12 yet.

Mainly the jose/jwk/parse part doesn't resolve to anywhere on node12

Mainly the jose/jwk/parse part doesn't resolve to anywhere on node12

Specifically it's the exports in package.json not being picked up.

Specifically it's the exports in package.json not being picked up.


device_plugins: Add install_operator.yaml

Added the following in the PR:

install_operator.yaml to install operator via oc CLI
Updated the readme with this file's install command

Signed-off-by: vbedida79<EMAIL_ADDRESS>

@chaitanya1731 @uMartinXu Please review

Looks good to me


Navigation generated instead of hard coded

Probably won't work with Posts otherwise

Requires portfolio pages as posts


Navigation Bar issue,

Hello, I got some displaying issue in the navigation bar please check the image

Also My CollectionView was resizing when rotating but after adding SLPagingView it doesn't resize any more I tried to add this code to didLoadView method in SLpagingView but haven't helpd :
self.view.autoresizingMask = UIViewAutoresizingFlexibleWidth |
    UIViewAutoresizingFlexibleHeight;

I'm using storyboard as in tinder example (and I'haven't added any code) I just subclassed from SLPagingView and I add the costum Segue that's all

To slove the autoresize problem I've added this code :
self.scrollView.autoresizesSubviews = YES;
    [self.scrollView setAutoresizingMask:UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight];

@StefanLage  check this also :
before scrolling ( After app is lunched no interaction made yet)

after scrolling


"Group" property configuration for RabbitMQ is triggering job operator multiple times with same parameters, after consumer restart.

From @ravi28 on April 24, 2018 21:55
We have an application that is using spring-cloud-stream with RabbitMQ as binder. We are seeing issues during a specific scenario as described below

Scale down consumer instances to zero.
A request from producer application is initiated and is in the queue waiting to be picked up.
Scale up consumer instance to one and it processes the job in the queue after it starts.

To support scaling up of consumer instances we have added the "group" property to our configuration so that it now looks as shown below
spring.cloud:
  stream:
    bindings:
      # inputs
      sandboxOperationRequestInput:
        destination: "sandboxTaskRequests"
        content-type: "application/json"
        group: operator
        consumer:
          maxAttempts: 1
      # outputs
      sandboxStatusEventOutput:
        destination: "sandboxStatusEvents"
        content-type: "application/json"

Since we added the "group" property the above scenario has been failing. Once the consumer is scaled up to one instance we are noticing that Request Listener is triggering job operator multiple times with the same job parameters which is undesired. As a result the job is failing due to JobInstanceAlreadyExistsException. I've tried changing multiple configurations but in vain. Can someone please suggest if I'm missing something? Any leads to what is causing this will be very helpful.
You can access the full logs for both working and failing scenarios.
Copied from original issue: spring-cloud/spring-cloud-stream#1367

From @garyrussell on April 30, 2018 15:34
Sorry, I don't understand the issue. When there is no group, you get an anonymous (auto-delete) queue...

{"@timestamp":"2018-04-20T21:02:23.563+00:00","@version":1,"message":"declaring queue for inbound: sandboxTaskRequests.anonymous.-dE_tzueRIutSLvI826abQ, bound to: sandboxTaskRequests","logger_name":"org.springframework.cloud.stream.binder.rabbit.provisioning.RabbitExchangeQueueProvisioner","thread_name":"main","severity":"INFO","level_value":20000}

So when you scale down to zero, there is no queue present to hold the request; any request sent with zero consumers will be lost.
When there is a group, the queue is persistent and a message is processed once, and once only.
If you are getting JobInstanceAlreadyExistsException you must have multiple messages with the same parameters.

From @garyrussell on April 30, 2018 15:55
Perhaps you are sending the request multiple times and relying on the fact that they are discarded by RabbitMQ when there is no queue.

@garyrussell Thank you for your response. We are using persistence for RabbitMQ to ensure that the message is not lost when consumer is not up. That is why in the first scenario where we didn't have the "group" parameter the message was picked up to an anonymous queue and processed successfully.
As you advised I will look to see if somehow we end up having multiple messages which will then cause JobInstanceAlreadyExistsException

We are using persistence for RabbitMQ to ensure that the message is not lost when consumer is not up.

But if you are using anonymous consumers (no group), there is no queue in which to persist the message. Anonymous consumer queues are auto-deleted when the consumer is stopped.

Closing this due to inactivity.


Workaround docker-py dependency's failure to import six

repo2docker depends on docker, but docker==5.0.0 is failing to import its dependency six and isn't getting patched because of a lack of maintainers it seems.
This is a PR similar to one we have done for chartpress as well (https://github.com/jupyterhub/chartpress/commit/8b486732f4ea518f65f4538dbb4e334f5e8aa5c6). The reason for repo2docker tests succeeds anyhow without this workaround is because six is available due to other reasons in the environment where tests run apparently.
PS: The Dockerfile was updated as installing a six wheel would fail as it would uninstall an existing six first but run into a failure doing so, so I adjusted the Dockerfile to not try to install a six wheel from the previous Dockerfile build step.
Closes #1039

Could we set our requirement to docker-py<5 or docker-py!=5.0.0? It'll make future maintenance easier when the upstream bug is fixed, and also avoids installing a new dependency for all users of repo2docker.


test(setup-node): add shellcheck to nodes

Code LGTM.

Deferring to @mboersma on the feedback since he already ran these commands on the Jenkins nodes.

@bacongobbler is right, both those changes would be preferable.

Done!

Code LGTM++.

Lgtm


Table level 'row_name'

Extend the display annotation on tables with a way of naming the row. A row_name can also take the place of what we used to call the title. Meaning a row_name is a valid way of forming a title when a title is needed. A row_name can also be used in different contexts like in the selection control of a data entry form for picking a foreign key.

See this:
https://github.com/informatics-isi-edu/ermrest/blob/annotation-update/user-doc/annotation.md#2015-display

I merged this PR so not sure this needs further review.  I think a separate issue covers implementation of the feature.


feat(batch): clean up session and query related resources

Is your feature request related to a problem? Please describe.
While investigating #5724, we discovered that frontend's memory does not drop after queries are finished.

From the flamegraph, a large proportion of memory is occupied by QueryExecution, which is maintained by QueryManager of FrontendEnv. It seems that queries will never be removed from QueryManager.
In addition, the SessionImpls are not removed from SessionManagerImpl when a session closes.
Describe the solution you'd like

[ ] Clean up SessionImpl from SessionManagerImpl when a session closes.
[ ] Clean up QueryExecution when a query completes.

Describe alternatives you've considered
No response
Additional context
https://github.com/risingwavelabs/risingwave/blob/4e5619c962d5649afbf2479be1eacd17c19b46a6/src/frontend/src/scheduler/distributed/query_manager.rs#L158
https://github.com/risingwavelabs/risingwave/blob/4e5619c962d5649afbf2479be1eacd17c19b46a6/src/frontend/src/session.rs#L208

Even after we cleared QueryExecution, the memory still do not drop due to #5816


where .exe?

i can't find the TaskarGroups.exe file
why is that?

Can you elaborate on this?
It is located here but a more up to date fork with some pulls applied can be found here.

the taskbargroups.exe file is not contained in the .zip


Disable clipboard in Android

Im looking for a way to prevent android coping the scan result to clipboard. From what I have found it can be done via Zxing preferences, but the problem is with intents I dont have much access to it. Any ideas how to deal with this?

@McOffsky, have you tried to use preferences_copy_to_clipboard parameter?

Hi @daserge how can I use this parameter? I dont know where to put, and didn't found any example. Thanks in advance!! Regards

@jorgeram I suppose you need to follow the steps at https://github.com/EddyVerbruggen/barcodescanner-lib-aar#steps-to-build-a-new-aar.
There is preferences.xml containing preferences_copy_to_clipboard.
It would be great if you could write your findings here.

Hi @daserge my problem was because my development is an Ionic Framework project (cordova / phonegap) and I am not familiar with Android Studio to modify and compile a new aar. Luckily after some try i got working by this way:

Download and unzip barcodescanner-lib-aar from EddyVerbruggen
Open Android Studio and this AAR project
Change line 55 of src/main/res/xml/preferences.xml (true to false)
Build Menu -> Rebuild project
Get build/outputs/aar/barcodescanner-debug.apk and use it to replace the original library in your Android project (my_ionic_project/platforms/android/libs/barcodescanner.apk)
Rebuild the Android project with the new AAR from Android Studio.

Thank you very much!
Regards.

#197

Hi, i have the same issue, when i scan the QRCode, its content is copied to the clipboard.
Did you found a solution for this issue ?


Add collection of metrics for Linux VMs on Azure

Additionally, this commit has a performance benefit to all captures
since it adds the PartitionKey to the filter.
Follow on from #8565
@blomquisg @djberg96 Please review.

:+1: merge when green

Looks good to me. Let's give it a whirl. 👍


reverse history search with k ignores last character

julia> aa=1
1

julia> ab=2
2

then typing aa escape k takes you to the ab=2 line, not the aa=1 line.  if rather you type aa= then escape k ab=2 is skipped and you're taken to aa=1.

I was hoping to solve this one in #97, but wasn't able to.
For my own future reference, I believe the issue here is that the reverse history search includes content before the cursor, and we unintentionally exclude the last character because when moving into Normal mode the cursor is moved back by one.
One possible hacky solution may be to move the cursor forward by one character immediately prior to the reverse history search, and then immediately return the cursor to its original position; the question is whether this would cause any kind of flashing artifacts.


DOC: Add example to scipy.signal.hilbert

Add example showing how to determine the amplitude envelope and
instantaneous frequency of a signal.

Travis build failed because it stalled.

Thats okay; this is a documentation only change.

I think adding an example like this is a fine plan.  I think the example could be better.  For instance, the frequency_modulation variable is confusing since there is no frequency modulation in the waveform you create.  The dips in the plot frequency vs. time are not due to frequency modulation, but due to the amplitude dropping to zero at those points.
I think that using a wave with moderate-depth, non sinusoidal amplitude modulation and some moderate frequency modulation would make a better example.
It may also be worth noting that when dealing with real signals, np.diff is probably not what you want.

Thanks for the feedback.
It is indeed only amplitude-modulation, the name frequency_modulation is
confusing. Perhaps frequency_of_modulation would have been better. I've
changed the example, which is now an amplitude-modulated chirp.
On Wed, May 13, 2015 at 6:50 PM, Eric Moore<EMAIL_ADDRESS>wrote:

I think adding an example like this is a fine plan. I think the example
could be better. For instance, the frequency_modulation variable is
confusing since there is no frequency modulation in the waveform you
create. The dips in the plot frequency vs. time are not due to frequency
modulation, but due to the amplitude dropping to zero at those points.
I think that using a wave with moderate-depth, non sinusoidal amplitude
modulation and some moderate frequency modulation would make a better
example.
It may also be worth noting that when dealing with real signals, np.diff
is probably not what you want.
—
Reply to this email directly or view it on GitHub
https://github.com/scipy/scipy/pull/4874#issuecomment-101743510.

LGTM.  I canceled the CI run, since this is documentation only.

Thanks!