[Bug] Adding Shiny Core to Existing Project

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[X] iOS
[X] Android
[ ] UWP
[ ] watchOS
[ ] tvOS
[ ] Tizen
[ ] .NET Standard

Feature Library

[X] Core
[X] BluetoothLE
[ ] Beacons
[ ] Locations
[ ] HTTP Transfers
[ ] Notifications
[ ] Sensors

Version of Library
<IP_ADDRESS>4-beta
Version of OS(s) listed above with issue
iOS 9+
Android 6+
Steps to Reproduce

Have existing Visual Studio 2019 for Mac cross platform project for iOS and Android
Add Shiny NuGet package for BluetoothLE
Add example AndroidShinyHost.Init call from getting started
Create MyDelegateClass from getting started
Add services.UseBleCentral() to MyStarted Class from getting started
Note that there is an error the UseBleCentral doesn't exist. Checked all references and they seem to be there
Note that there is an error in thee Android project that AndroidShinyHost doesn't exist

Expected Behavior
I downloaded the samples github repo and they all seem to be defined. When I look in the Object Browser in the samples projects Shiny.Core has a lot more classes than when I look at it in mine and yet the nuget references look the same.
Actual Behavior
missing classes and extension methods.

Please supply a reproducible sample.

I created a brand new project and imported everything and made the classes and one of the problems is the same the other is different. I have attached a zip file with the solution in it. You can see the errors.
BLETest.zip

AndroidShinyHost.Init is required to be inside of an AndroidApplication like the sample and the readme.  You are also missing the permission wireup in the main activity.

2)There is no method services.UseCentral with a generic delegate yet.  There will be a in future beta.  Please take a look at the intellisense and the sample


[v6] [WIP] Added useActivePath hook

This is an example implementation of the feature mentioned in #7552
I am still not sure if useActivePath is a good hook name, thus I am very open to suggestions
Currently:

[x] Implementation

[ ] Tests

[ ] Documentation

There is a similar PR open already: #7142
For familiarity, please rename this useRouteMatch and match that functionality. That's what you're asking for anyways.

The useRouteMatch works in a different way but fulfils this need, thank you


ENH: GAT object should scorer strings as in sklearn.cross_val_score at all.

It seems the scoring API is a bit broken, the code does not allow for scorers that require the classifier as inputs. But this cannot even be fixed using partial function as the clf is copied inside.
cc @kingjr

I wasn't aware of this prerequisite. Can you make an example where the scorer needs the classifier?

In http://scikit-learn.org/stable/modules/classes.html#module-sklearn.metrics, I only see scorers that requires y_true and y_pred, or alike, but I guess I'm missing something.
Are you referring to cross_validation.cross_val_score(estimator, X)? In our case, the cross val is already implemented within the GAT.

@kingjr sklearn.metrics.SCORERS['roc_auc'] returns a function that would take the clf as input too.
I'm just saying we would need this kind of handling inside the gat.
Ultimately I would like to simply pass socring='roc_auc' as I do in cross_val_score.

Right... I need to think about this. But in the meantime, you can just use scorer=sklearn.metrics.roc_auc_score no?

Jumping on this with a related question - if we use roc_auc_score, then doesn't that require that the model we've fit use predict_proba rather than just predict? I looked through the GAT code briefly and couldn't find a flag that said something like "if clf.probability is True: do predict_proba instead of predict.

the new scoring API in sklearn does this magic for you.

Yes, we decided to keep the gat api simple for now. What I do to get proba
is create a class that inherits from SVC and change the predict behavior
into a predict_proba one
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Jumping on this with a related question - if we use roc_auc_score, then
doesn't that require that the model we've fit use predict_proba rather
than just predict? I looked through the GAT code briefly and couldn't
find a flag that said something like "if clf.probability is True: do
predict_proba instead of predict.
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118086019
.

ah ok - I wonder if one could just do
svm.predict = svm.predict_proba
and then just go from there
On Thu, Jul 2, 2015 at 2:18 PM, J-R King<EMAIL_ADDRESS>wrote:

Yes, we decided to keep the gat api simple for now. What I do to get proba
is create a class that inherits from SVC and change the predict behavior
into a predict_proba one
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Jumping on this with a related question - if we use roc_auc_score, then
doesn't that require that the model we've fit use predict_proba rather
than just predict? I looked through the GAT code briefly and couldn't
find a flag that said something like "if clf.probability is True: do
predict_proba instead of predict.
—
Reply to this email directly or view it on GitHub
<
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118086019

.

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118138808
.

Ah I can see why you guys wanted to keep the api simple for now...I was
running into "incorrect shape" errors so it seems like the simplest way to
get the code working is to set the predict_proba function to only output
the second column of its output probabilities (assuming that n_classes is
2).
@agramfort are you talking about the .17 release then? Or is this something
that's already implemented?
On Thu, Jul 2, 2015 at 3:33 PM, Chris Holdgraf<EMAIL_ADDRESS>wrote:

ah ok - I wonder if one could just do
svm.predict = svm.predict_proba
and then just go from there
On Thu, Jul 2, 2015 at 2:18 PM, J-R King<EMAIL_ADDRESS>wrote:

Yes, we decided to keep the gat api simple for now. What I do to get proba
is create a class that inherits from SVC and change the predict behavior
into a predict_proba one
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Jumping on this with a related question - if we use roc_auc_score, then
doesn't that require that the model we've fit use predict_proba rather
than just predict? I looked through the GAT code briefly and couldn't
find a flag that said something like "if clf.probability is True: do
predict_proba instead of predict.
—
Reply to this email directly or view it on GitHub
<
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118086019

.

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118138808
.

That s right. And predict proba is not the only outputting function out
there.... I sometimes use multidimensional prediction but you have to build
a scorer that knows what to do with it.
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Ah I can see why you guys wanted to keep the api simple for now...I was
running into "incorrect shape" errors so it seems like the simplest way to
get the code working is to set the predict_proba function to only output
the second column of its output probabilities (assuming that n_classes is
2).
@agramfort are you talking about the .17 release then? Or is this something
that's already implemented?
On Thu, Jul 2, 2015 at 3:33 PM, Chris Holdgraf<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','choldgraf@gmail.com');>> wrote:

ah ok - I wonder if one could just do
svm.predict = svm.predict_proba
and then just go from there
On Thu, Jul 2, 2015 at 2:18 PM, J-R King<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Yes, we decided to keep the gat api simple for now. What I do to get
proba
is create a class that inherits from SVC and change the predict behavior
into a predict_proba one
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
wrote:

Jumping on this with a related question - if we use roc_auc_score,
then
doesn't that require that the model we've fit use predict_proba rather
than just predict? I looked through the GAT code briefly and couldn't
find a flag that said something like "if clf.probability is True: do
predict_proba instead of predict.
—
Reply to this email directly or view it on GitHub
<

https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118086019

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118138808

.

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118166224
.

Cool - maybe I will write a post/example on getting this to work with AUC
for future reference, thanks for your clarification!
Chris
On Thu, Jul 2, 2015 at 5:45 PM, J-R King<EMAIL_ADDRESS>wrote:

That s right. And predict proba is not the only outputting function out
there.... I sometimes use multidimensional prediction but you have to build
a scorer that knows what to do with it.
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Ah I can see why you guys wanted to keep the api simple for now...I was
running into "incorrect shape" errors so it seems like the simplest way
to
get the code working is to set the predict_proba function to only
output
the second column of its output probabilities (assuming that n_classes is
2).
@agramfort are you talking about the .17 release then? Or is this
something
that's already implemented?
On Thu, Jul 2, 2015 at 3:33 PM, Chris Holdgraf<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','choldgraf@gmail.com');>> wrote:

ah ok - I wonder if one could just do
svm.predict = svm.predict_proba
and then just go from there
On Thu, Jul 2, 2015 at 2:18 PM, J-R King<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Yes, we decided to keep the gat api simple for now. What I do to get
proba
is create a class that inherits from SVC and change the predict
behavior
into a predict_proba one
On Thursday, 2 July 2015, Chris Holdgraf<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
wrote:

Jumping on this with a related question - if we use roc_auc_score,
then
doesn't that require that the model we've fit use predict_proba
rather
than just predict? I looked through the GAT code briefly and
couldn't
find a flag that said something like "if clf.probability is True: do
predict_proba instead of predict.
—
Reply to this email directly or view it on GitHub
<

https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118086019

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118138808

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118166224

.

—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118186896
.

@choldgraf that would be really useful. I just had to deal with this myself. I am calculating both the scores from crossval and I wanted to have the AUCs too. @kingjr, for clarification, are you saying that you can create a scorer that output both the classification score and the AUC?

Cool, maybe I can hash one out as I'm visiting Paris next week. I think it would be particularly useful for AUC, since the paper that "introduced" the time-generalization method explicitly recommended AUC as a metric they used (current debate about usefulness of AUC notwithstanding)

we should start tagging issues and PRs for the sprint.
Please go for it anyone.

I ve got some working examples already , I Lol put a ref here
On Friday, 3 July 2015, Chris Holdgraf<EMAIL_ADDRESS>wrote:

Cool, maybe I can hash one out as I'm visiting Paris next week. I think it
would be particularly useful for AUC, since the paper that "introduced" the
time-generalization method explicitly recommended AUC as a metric they used
(current debate about usefulness of AUC notwithstanding)
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2176#issuecomment-118393682
.

Here's an example:
import mne
from mne.datasets import spm_face
from mne.decoding import GeneralizationAcrossTime
from sklearn.linear_model import LogisticRegression
from sklearn.pipeline import Pipeline
from sklearn.preprocessing import StandardScaler
# Preprocess data
data_path = spm_face.data_path()
# Load and filter data, set up epochs
raw_fname = data_path + '/MEG/spm/SPM_CTF_MEG_example_faces%d_3D_raw.fif'
events_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw-eve.fif'
raw = mne.io.Raw(raw_fname % 1, preload=True)  # Take first run
picks = mne.pick_types(raw.info, meg=True, exclude='bads')
raw.filter(1, 45, method='iir')
events = mne.find_events(raw, stim_channel='UPPT001')
event_id = {"faces": 1, "scrambled": 2}
tmin, tmax = -0.1, 0.5
decim = 4  # decimate to make the example faster to run
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, proj=True,
                    picks=picks, baseline=None, preload=True,
                    reject=dict(mag=1.5e-12), decim=decim, verbose=False)

class clf_proba(LogisticRegression):
    def predict(self, X):
        self.y_pred_ = super(clf_proba, self).predict(X)
        return self.y_pred_

def scorer_auc(y_true, y_pred):
    from sklearn.metrics import roc_auc_score
    from sklearn.preprocessing import LabelBinarizer
    le = LabelBinarizer()
    y_true = le.fit_transform(y_true)
    return roc_auc_score(y_true, y_pred)

clf = Pipeline([('scaler', StandardScaler()), ('clf', clf_proba())])

gat = GeneralizationAcrossTime(clf=clf, n_jobs=2)
gat.fit(epochs)
gat.score(epochs)
gat.plot()

"current debate about usefulness of AUC notwithstanding"

Could you give a ref? Do you have alternative proposals we should use by default?

@kingjr, for clarification, are you saying that you can create a scorer that output both the classification score and the AUC?

You could if you wanted to. The dimensionality of the scorer isn't restricted if I recall correctly. However, I think the easiest is to run the scorer twice, without passing the epochs.
Using the example above you'd have something like this:
gat = GeneralizationAcrossTime(clf=clf, n_jobs=2, scorer=scorer_auc)
gat.fit(epochs)
gat.predict(epochs)

gat.scorer_ = scorer_auc
scores_auc = gat.score()
gat.scorer_ = accuracy
scores_acc = gat.score()

Could you give a ref? Do you have alternative proposals we should use by default?

Seems like the wikipedia entry [1] has some good refs...here's the relevant paragraph:

The machine learning community most often uses the ROC AUC statistic for model comparison.[10] However, this practice has recently been questioned based upon new machine learning research that shows that the AUC is quite noisy as a classification measure[11] and has some other significant problems in model comparison.[12][13] A reliable and valid AUC estimate can be interpreted as the probability that the classifier will assign a higher score to a randomly chosen positive example than to a randomly chosen negative example. However, the critical research[11][12] suggests frequent failures in obtaining reliable and valid AUC estimates. Thus, the practical value of the AUC measure has been called into question,[13] raising the possibility that the AUC may actually introduce more uncertainty into machine learning classification accuracy comparisons than resolution. Nonetheless, the coherence of AUC as a measure of aggregated classification performance has been vindicated, in terms of a uniform rate distribution,[14] and AUC has been linked to a number of other performance metrics such as the Brier score.[15]

I don't have the statistical training to really know the ins and outs of the debate, but it seems like there is a debate anyway (I suppose there always is for anything in academia).
Thanks for the example, btw...very useful!
[1] https://en.wikipedia.org/wiki/Receiver_operating_characteristic#Area_under_curve

Also per your example, couldn't you even implement a convenience mixin in case you wanted to try a different classifier, e.g.:
class PredictMixin(object):
    def predict(self, X):
        self.y_pred_ = super(clf_proba, self).predict_proba(X)
        return self.y_pred_
class myRF(PredictMixin, RandomForestClassifier):
    pass
class mySVM(PredictMixin, LinearSVC):
    pass

or is that just super, super lazy? :)

Seems like the wikipedia entry [1] has some good refs...here's the relevant paragraph:

Mmmh, I wasn't aware at this at all. My impression so far, is that AUC was much better than accuracy, at least in my datasets and simulations: both in terms of stability and usability (accuracy is just not very good for GAT analyses; although it's true that the AUC is actually more computationnally expensive).
If anyone wants to give some practical feedbacks here that'd be much appreciated.

couldn't you even implement a convenience mixin in case you wanted to try a different classifier

Sure! I didn't know this syntax #StillLearning. In my case, I actually have a done something like this for generic usecases:
class force_predict(object):
    def __init__(self, clf, mode='predict_proba', axis=0):
        self._mode = mode
        self._axis = axis
        self._clf = clf

    def fit(self, X, y, **kwargs):
        self._clf.fit(X, y, **kwargs)
        self._copyattr()

    def predict(self, X):
        if self._mode == 'predict_proba':
            return self._clf.predict_proba(X)[:, self._axis]
        elif self._mode == 'decision_function':
            distances = self._clf.decision_function(X)
            if len(distances.shape) > 1:
                return distances[:, self._axis]
            else:
                return distances
        else:
            return self._clf.predict(X)

    def get_params(self, deep=True):
        return dict(clf=self._clf, mode=self._mode, axis=self._axis)

    def _copyattr(self):
        for key, value in self._clf.__dict__.iteritems():
            self.__setattr__(key, value)

ah that's a cool way of going about it!
For the AUC stuff, I haven't done extensive testing but generally have found similar things to yourself. I think that the main layperson's argument about the drawbacks of AUC is that a benefit of ROC is it's ability to tell you about how the TP/FP ratio changes as a function of your decision cutoff. However, by reducing that information to a single point estimate (AUC) you remove this information and make it about as valuable as accuracy. I have no way of confirming this one way or the other though :)

. However, by reducing that information to a single point estimate (AUC) you remove this information and make it about as valuable as accuracy.

I hope it's more than this, because the main point is that AUC is 1) non parametric and 2) robust to non-balanced datasets and biases, whereas accuracy isn't necessarily. The classic cases is when you train A versus B, and test on C versus D. If, par malheur, C and D really looks like A, they will be discretely predict as A, and accuracy drops to chance, whereas the AUC will be resistant to such bias. But I guess other metrics have this advantage too.
Another intereting point is that AUC is ~ to the U of the Mann-Whtiney U test, the established non param 2 sample test, so that's a strong bridge; it's not just about TP/FP rates.

this sounds reasonable to me - I'm just playing devil's (uninformed) advocate :D

And you should, if AUC turned out not to be great, we should know it ASAP before it's too late.

@kingjr I am hoping to solve this issue. if I understand this correctly our main aim is to integrate as much sklearn into gat as possible including the make_scorer function. However for the time being, keeping ourself limited to this PR we have to make roc_auc_score work properly without having users to write the custom functions.
Please let me know if I am understanding correctly.
Thank you

Some parts of the discussion is outdated. The GeneralizationAcrossTime object can use different predict methods (e.g. predict_method='predict_proba').
However, the scoring necessarily takes a callable; it should be able to take a string and evaluates it with sklearn (@dengemann suggested sklearn.metrics.SCORERS['roc_auc']).


Update coffeescript dependency

Coffeescript moved on npm. coffee-script ➡ coffeescript (no hyphen).

@nfischer Hi Nate. I'd like to take care of this. I just cloned the repo and will look at the document on contributing.

I'm not sure the process for making a PR. I guess I need to be able to push my branch up to the repo.

Hey Josh, you can fork ShellJS, push your branch to your fork, and submit a pull request from that branch.


Always validate date and date-time strings before converting to...

...a JavaScript Date object. Added unit test coverage for a wider sampling of valid and invalid dates and date-times.
Relates to:
https://github.com/apigee-127/swagger-tools/pull/361

Thanks @SandyChapman.


Set up database and models

User Stories

As a user, I need a database that can store my account, posts, comments, or upvotes

Finished; connected to MySQL and established /models files


WIP: Docs/updates in line with wiki

Updates to repo in line with changes to wiki as part of #1913

@Parbhat What do you think of these changes, can they go in?
Trying to cleanup the PR:s a bit.


Upgrade 5.6.6 and other bumps

Upgrades:

server 5.6.6
web 5.16.0
figure
iviewer
omero-py 5.13.1

Further:

adding logic to each playbook to upgrade to latest omero-py when server is upgraded - please check this one @sbesson

This was deployed on

[x] ome-training-4
[x] ome-training-3
[x] outreach
[x] workshop
[x] demo
[x] learning (in the time 4 - 6 January 2023 - clarified with the appropriate person)
[x] send ns users email
[x] nightshade (in the time 5 January 2023 at 12.00 - 12.30)

cc @jburel

I do not see any problems with the training servers where this was deployed already, with web, figure, iviewer work fine, omero-py version in server and web envs is correct

Nothing that worries me in this set of upgrades

Thanks and sorry, pushed https://github.com/ome/prod-playbooks/pull/366/commits/9f18486109aa614b16830678fc5d8be8f505ea5e as I forgot to add the omero-py addon on dundeeomero.

Upgraded demo as well, updated description.

Everything upgraded now, see description list
Maybe merge ?

Yes! (since otherwise it could lead to a downgrade :smile:)


Defunct hyperlink

Hello,
The documentation is great but there is a defunct hyperlink which could be rectified.
Under "Step 1", there is a point "Enable modern authentication: You need to enable modern authentication on your tenant for this feature to work." -> The hyper-link for 'modern authentication' takes you to 'Page Not Found'

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

ID: 7d9cfb97-d0f3-3cc1-a460-ff4e05530b92
Version Independent ID: 3db82b30-1d14-1613-e473-13310e855ab2
Content: Azure AD Connect: Seamless Single Sign-On - quick start
Content Source: articles/active-directory/connect/active-directory-aadconnect-sso-quick-start.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@aamer-s I am getting any error going to the link and it goes to the page - https://www.microsoft.com/en-us/microsoft-365/blog/2015/11/19/updated-office-365-modern-authentication-public-preview/

The behavior is still the same for me:
I'm located in Adelaide, Australia (if it makes any difference)
The main article is : https://docs.microsoft.com/en-us/azure/active-directory/connect/active-directory-aadconnect-sso-quick-start
This is the URL I'm clicking:

And this is where I get redirected:

The URL is : https://www.microsoft.com/en-au/microsoft-365/blog/2015/11/19/updated-office-365-modern-authentication-public-preview/
with difference being 'en-us' and 'en-au' in the URL.
So there is indeed a problem - it was just a matter of digging in deeper. To make it easier for you - you need to fix the 'en-au' URL.
Let me know if you need more details.


[CRYPTO-169] Add protection for the ExceptionInInitializerError scenario to the CryptoRandomFactory#getCryptoRandom(Properties) method

In version 1.2.0 of commons-crypto, the static initialization code block of o.a.c.crypto.random.OpenSslCryptoRandom may throw a GeneralSecurityException wrapped by IllegalStateException, which ultimately gets wrapped into j.l.ExceptionInInitializerError by Java's own mechanism.
This results in behavior differences when running the statement
org.apache.commons.crypto.random.CryptoRandomFactory#getCryptoRandom(new java.util.Properties());

on platforms that do not support OpenSslCryptoRandom compared to when using commons-crypto 1.1.0 (e.g. Apple Silicon)

commons-crypto 1.1.0

After OpenSslCryptoRandom initialization fails, it tries to initialize JavaCryptoRandom, and JavaCryptoRandom can be successfully initialized and return results.

commons-crypto 1.2.0

After OpenSslCryptoRandom initialization fails, it throws an ExceptionInInitializerError. Since the CryptoRandomFactory#getCryptoRandom method does not catch ExceptionInInitializerError and perform fault tolerance, ExceptionInInitializerError continues to be thrown upward, losing the opportunity to try to initialize JavaCryptoRandom.
Therefore, this PR adds the catch and fault tolerance for ExceptionInInitializerError in the CryptoRandomFactory#getCryptoRandom method to keep it behaving similarly to commons-crypto 1.1.0.

Codecov Report

Merging #258 (e0fd653) into master (c5d19d0) will decrease coverage by 0.25%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #258      +/-   ##
============================================
- Coverage     74.69%   74.44%   -0.25%
  Complexity      441      441
============================================
  Files            38       38
  Lines          1810     1816       +6
  Branches        176      177       +1
============================================
  Hits           1352     1352
- Misses          348      354       +6
  Partials        110      110

Files
Coverage Δ

...che/commons/crypto/random/CryptoRandomFactory.java
82.00% <0.00%> (-11.19%)
:arrow_down:

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

JIRA: https://issues.apache.org/jira/browse/CRYPTO-169

Thanks @garydgregory  ~

Thanks @garydgregory ~

YW! 😄


Nasty memory corruption bug

global::delta.time() is currently returning the correct value before and after the UpdateComponents() call in main.cpp, but within the method global::delta.time() returns 0! Highly suspect there's some corruption going on.

Unbelievably this bug is even as far back as 33b284e63ac897c0d123c548a36570bcb295d2ab. Suspect the ECS has something dodgy. This is a properly bizarre bug!

Ha, fixed it! For some foolish reason I made global::delta static, resulting in multiple copies of it cropping up wherever the delta.h file was #included. 🤦♂ So in other news I now understand why extern is a thing :P at least it wasn't a corruption bug, I suppose.


trait org.scalamock.matchers.Matchers without protected incapsulation

Hello, i don't understand why in trait org.scalamock.matchers.Matchers all methods encapsulate like protected. Can we reconsider encapsulation politics?

can you explain what problem or bug you are facing, or what additional usage you want to propose?
Just reworking traits for political reasons doesn't sound worthwhile.
Or, if this is a usage question, Matchers are explained here, please read: https://scalamock.org/user-guide/features/

I want more flexibility in writing my test. I want create all mocks in place where i want not only under trait MockFactory trait. And i don't understand why this encapsulate politics have been chosen, may be u can explain me. And it's for me strange that Matchers has dependence in MockContext but not using it at all (mock, stub methods take implicit parametrs).
I'm really straggling with this for my humble opinion unnecessary encapsulate politic.
MockContext private too(((, but for spec2 it's not private.

The Matchers trait you refer to requires a MockContext to work.
As you can see in the code, it creates FunctionAdapter instances for the various mock setup/expectation calls.
These FunctionAdaptors then do some more  work based on the arguments they were given to abstract from a concrete call into a more generic way of handling mock invocations.
As such, mixing in Matchers into your code separately doesn't give you an advantage.
You will need to mix in MockFactory into your test suite.
What you describe in your last message - creating all mocks in one place - is also already possible by using a fixture. How that can be done is described in the user guide as well.
if that's what you want to do then you should probably create a common base class for all your tests that manages the fixture and then you can subclass and share your mocks throughout your code.
We really don't need to change ScalaMock for that or cherry-pick which internal traits to mix in.


How to make Select2 with Search work in Livewire?

Hi all,
I have implemented Select2 with Search in Livewire component. But it does not work. What is the problem.
The following is my code.
Component's Controller
<?php

namespace Modules\ArSys\Http\Livewire\Option\Research;

use Livewire\Component;
use Modules\ArSys\Entities\ResearchType;

class Type extends Component
{
    public function render()
    {
        $config = [
            "placeholder" => "Select multiple options...",
            "allowClear" => true,
            "liveSearch" => true,
            "liveSearchPlaceholder" => "Search...",
        ];

        $researchType = ResearchType::all();

        return view('arsys::livewire.option.research.type', ['config' => $config, 'researchType' => $researchType]);
    }
}

Component's View
<div>

    <x-adminlte-select2 id="sel2Category" name="sel2Category" label="Categories"
        label-class="text-danger" igroup-size="sm" :config="$config">

        <x-slot name="appendSlot">
            <x-adminlte-button theme="outline-dark" label="Clear" icon="fas fa-lg fa-ban text-danger"/>
        </x-slot>

        @foreach ($researchType as $type)
            <option>{{$type->code}}-{{$type->description}}</option>
        @endforeach
    </x-adminlte-select2>
</div>

@desyashasyi There are no liveSearch option, neither liveSearchPlaceholder option for the select2 plugin. Always give a read to the plugin documentation: Select2 Options
Also, take a look to this link: https://stackoverflow.com/questions/36188356/enable-select2-multi-select-search-box

The answer is that the select2 input element becomes the search box for multiple selects without back end data

if you start typing , your results will start filtering the options

if you have it set to load remote ajax data, it actually does retain a search box, but for multiple selects without a data source, the input is the search bar, which is fairly intuitive

Closed because no feedback, reopen in case you need more help...


navi: 2.12.0 -> 2.12.1

Motivation for this change
https://github.com/denisidoro/navi/releases/tag/v2.12.1
Things done

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

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

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

@ofborg eval

LGTM.
Result of nixpkgs-review pr 100889 1

1 package built:

navi


FYI Upstream Issue with Flask-caching > 1.10.1 (1.11.1 as of writing)

Error below encountered when using ServersideOutput:
get key '__wz_cache_count' -> invalid load key, '\x00'
Turns out it's an issue with Flask-caching. Hope this helps anyone else with the same issue.
Original issue: https://github.com/pallets-eco/flask-caching/issues/221

Same here!
Moving back to flask-caching=1.10.1 is working fine.
flask-caching has been upgraded to 2.0.0 an 2022-06-26. Incorporating this upgrade into dash-extensions might solve the issue. Please test.

Same here!
Using flask-caching==1.10.1 works as a temporary workaround, however, flask-caching<1.11 is considered a security risk:
https://nvd.nist.gov/vuln/detail/CVE-2021-33026

The underlying issue was indeed a breaking change in flask-caching. If have changed the implementation to adhere to the new implementation, and bumped flask-caching to 2.0.0 as part of the 0.1.4 release. Please let me know, if you are still seeing any issues after upgrading.

Thank you very much @emilhe! Works on our side no issues so far.


do we need a document that sets our motivations and keeps track of coding philosophy?

The repo wiki? Seems we need to centralize our individual assumptions.

I think this would be very good, we seem to fluctuate on how much we want to be like base R plotting, when we want to strike out and be different, and when we want to be like ggplot2.

Absolutely


[FLINK-31617] Upgrade the SnakeYaml Maven dependency

What is the purpose of the change
Upgrade the SnakeYaml Maven dependency from 1.33 to 2.0 to fix the container vulnerability CVE-2022-1471
Brief change log
I modified some files in the project.

Upgrade SnakeYaml version from 1.33 to 2.0.
Updated the SnakeYaml Maven dependency in the pom.xml.
Updated the "flink-kubernetes/src/main/resources/META-INF/NOTICE" file in the project.

Verifying this change
This change is a just maven dependency update doesn't make impact.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (no)

Hi,  @wuchong please take a look in your free time,thank you very much!

@MartijnVisser Thanks for your review,  and i will update as soon as possible.

@MartijnVisser I think we should upgrade snakeyaml and kubernetes.client at the same time, don't you think?The new version solves the compatibility problem.

I will fix it.

@flinkbot run azure

Okay, I'll take care of it as soon as possible.


TF controller pod is throwing an error  crashloopbackoff after the installation

I am getting the following error when I install the terraform controller on flux v2.
How to fix this issue?
tf-controller-7ff4f9d84b-s59wf                0/1     CrashLoopBackOff   5          5m20
k logs -f tf-controller-7ff4f9d84b-s59wf -n flux-system --previous

kubectl logs -f tf-controller-7ff4f9d84b-s59wf -n flux-system --previous
unknown flag: --runner-creation-timeout
Usage of tf-controller:
--ca-cert-validity-duration duration        The duration that the ca certificate certificates should be valid for. Default is 1 week. (default 168h0m0s)
--cert-rotation-check-frequency duration    The interval that the mTLS certificate rotator should check the certificate validity. (default 30m0s)
--cert-validity-duration duration           The duration that the mTLS certificate that the runner pod should be valid for. (default 6h0m0s)
--concurrent int                            The number of concurrent terraform reconciles. (default 4)
--enable-leader-election                    Enable leader election for controller manager. Enabling this will ensure there is only one active controller manager.
--events-addr string                        The address of the events receiver.
--health-addr string                        The address the health endpoint binds to. (default ":9440")
--http-retry int                            The maximum number of retries when failing to fetch artifacts over HTTP. (default 9)
--kube-api-burst int                        The maximum burst queries-per-second of requests sent to the Kubernetes API. (default 50)
--kube-api-qps float32                      The maximum queries-per-second of requests sent to the Kubernetes API. (default 20)
--leader-election-lease-duration duration   Interval at which non-leader candidates will wait to force acquire leadership (duration string). (default 35s)
--leader-election-release-on-cancel         Defines if the leader should step down voluntarily on controller manager shutdown. (default true)
--leader-election-renew-deadline duration   Duration that the leading controller manager will retry refreshing leadership before giving up (duration string). (default 30s)
--leader-election-retry-period duration     Duration the LeaderElector clients should wait between tries of actions (duration string). (default 5s)
--log-encoding string                       Log encoding format. Can be 'json' or 'console'. (default "json")
--log-level string                          Log verbosity level. Can be one of 'trace', 'debug', 'info', 'error'. (default "info")
--metrics-addr string                       The address the metric endpoint binds to. (default ":8080")
--requeue-dependency duration               The interval at which failing dependencies are reevaluated. (default 30s)
--runner-grpc-port int                      The port which will be exposed on the runner pod for gRPC connections. (default 30000)
--watch-all-namespaces                      Watch for custom resources in all namespaces, if set to false it will only watch the runtime namespace. (default true)
unknown flag: --runner-creation-timeout

This repository is for the Flux Terraform Provider which is not the same with tf-controller. Please open an issue on https://github.com/weaveworks/tf-controller


Best way to accomplish sum()?

I'm transitioning my codebase from custom objects + Lodash and so far Immutable has replaced both perfectly. But there's ont thing I can't seem to do easily which is to get the sum of a List/Map. Currently doing this:
let total = 0;
entries.forEach(entry => total += entry.get('price'));

Is there anything similar to Lodash's sum like entries.sum('price') or entries.sum(entry => entry.get('price')) or something that I missed in the documentation?

I couldn't find any methods like sum but I guess the closest way to accomplish something like sum is to use the reduce method.
var entries = Immutable.fromJS([
    {price: 1},
	{price: 2},
    {price: 3},
]);

var sum = entries.map(entry => entry.get('price')).reduce((prev, current) => prev + current);
console.log(sum); // 6

reduce is pretty much the standard way to do these aggregation operations in fp

Closing out. reduce is the way to go!


[FLS] UI fixes

Updated SettingsControls to 0.7.

Removed the wrapping of the End Task button on resizing the window.

Icon fixes to fix: #21638

[ ] Closes: #21638

[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected

[ ] Tests: Added/updated and all pass

[ ] Localization: All end user facing strings can be localized

[ ] Dev docs: Added/updated

[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@niels9001
The margins at the bottom are missing. There is no space between process listview and window border.

@niels9001 , do you know if the glyph mentioned here can be used safely? https://github.com/microsoft/PowerToys/pull/21688#issuecomment-1302412689

@niels9001 , do you know if the glyph mentioned here can be used safely? #21688 (comment)

Yep. Everything above the E500 range can be used: https://learn.microsoft.com/en-us/windows/apps/design/style/segoe-fluent-icons-font#icon-list

THanks for opening the PR.
There are lots of conflicts, though, and most of them seem to be caused just by changes in white spaces and xaml code indentation on this PR. Were the changes you did just the icon changes or was there something else?

THanks for opening the PR. There are lots of conflicts, though, and most of them seem to be caused just by changes in white spaces and xaml code indentation on this PR. Were the changes you did just the icon changes or was there something else?

I'll resolve the conflicts! Might be good to start adopting a XAML Styler config file so XAML formatting is consistent for everyone?

Is that possible? :)
(I do XAML manually, though, perhaps there's a linter? )

For reference, these were the changes that made it in recently that are responsible for the conflict: https://github.com/microsoft/PowerToys/pull/21688/files#diff-3bc04002f00a182e5028f397568efda091f8895fa8ae3ee5fe66d7a1e0587e38

Is that possible? :) (I do XAML manually, though, perhaps there's a linter? )

Yep! XAML Styler is a commonly used plugin to format XAML. You can export the (default) config and add it to the root of the project folder. For everyone that has XAML Styler installed, would then use the same formatting.
In Labs, we also have a GH Action that checks on the correct formatting. But that is maybe a bit too much :-)
https://github.com/Xavalon/XamlStyler/wiki/XAML-Styler-Configuration

In Labs, we also have a GH Action that checks on the correct formatting. But that is maybe a bit too much :-)
https://github.com/Xavalon/XamlStyler/wiki/XAML-Styler-Configuration

Definitely something to consider, though. These files end up with weird identations / different spaces/tabs usage across projects.

@niels9001
The margins at the bottom are missing. There is no space between process listview and window border.

Are you going to fix this in this PR? Otherwise I will open an issue tomorrow.

This pr doesn't include (...) yet. Can it?
https://github.com/microsoft/PowerToys/blob/49007369cf45aa192f025eb79c72f9fe7babf3c2/src/modules/Hosts/Hosts/Views/MainPage.xaml#L52

@niels9001
The margins at the bottom are missing. There is no space between process listview and window border.

Are you going to fix this in this PR? Otherwise I will open an issue tomorrow.

Fixed!

@jaimecbernardo Conflicts should be resolved now!

Hi @niels9001 ,
Thanks a lot for the contribution!
I've added two commits:

I've uncommented the text block that was commented but was needed, otherwise it'd be an empty scrollviewer which was causing crashes when trying to render.
I've also removed the ItemsPanelTemplate that was added, because that was turning off virtualization and causing crashes when trying to list a big number of items (such as using File Locksmith on the Windows folder running as admin).

PTAL. Are these changes OK?
Otherwise, please let's make this PR only about the icon changes and leave other UI fixes for another PR, because we're introducing crashes otherwise on what could go in the hotfix.


Generated code with field "type" conflicts with "type.CustomType" during compilation phase

This error occurs in compileDebugJavaWithJavac phase – after Apollo code generation. Here is a sample of failing generated code:
@Generated("Apollo GraphQL")
public final class Schema_SchemaInput {
...
  private final Input<String> type;
...
  public InputFieldMarshaller marshaller() {
    return new InputFieldMarshaller() {
      @Override
      public void marshal(InputFieldWriter writer) throws IOException {
...
        if (multipleOf.defined) {
          writer.writeCustom("multipleOf", type.CustomType.BIGDECIMAL, multipleOf.value != null ? multipleOf.value : null);
        }

And the compiler error:
/Users/dpisoni/sandbox/gitapps/gp-android/QBPayments/build/generated/source/apollo/type/Schema_SchemaInput.java:677: error: cannot find symbol
          writer.writeCustom("multipleOf", type.CustomType.BIGDECIMAL, multipleOf.value != null ? multipleOf.value : null);
                                               ^
  symbol:   variable CustomType
  location: variable type of type Input<String>

It seems strange that no one else has encountered this yet, as "type" seems like a pretty common field name to have in schema.
Naturally, this is a blocking issue since it results in a compilation failure.
Gradle: 4.4
Grandle Android Plugin: 3.1.2
Android SDK: 27
apollo-android: 0.5.1-SNAPSHOT
N.B., I am using SNAPSHOT because I need the benefits of an unreleased bug fix. Using 0.5.0 my schema fails to compile because there is a field contained within called "class", a Java reserved keyword.

You should put the graphql files under some package structure

My graphql files are in a package structure, about 7 levels deep! The code generator however generates the "type" directory flat.

what about the schema file?

Ah, so I was not aware moving that around had any effect. Not obvious from the documentation, though I see the sample app does this. I will try this now...
That worked! Or at least, compilation isn't failing for this reason any more. (It now appears I've bumped into #949)
Thank you!


Missing data chunks when capturing white objects (D435)

Required Info

Camera Model
{ D400 }

Firmware Version
(<IP_ADDRESS>)

Operating System & Version
{Win (10))

Platform
PC

SDK Version
{ legacy / 2.50.0 }

Language
{Python/  Pyrealsnese2}

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I am having trouble capturing depth information on white-colored objects. My specific problem is when trying to read depth data from cows that have ambient lighting on one side (the picture below should help) One side of the cow is shaded and the other side receives ambient light. The day this was taken was overcast, but it still seemed to throw the data accuracy off. I am using a USB 3.1, 30 fps, and experience the problem with both videos and pictures. My program uses auto-exposure and if possible I would like to continue to use auto-exposure because the lighting in the environment is constantly changing. Through random testing, I have found that the camera D435 seems to struggle to get readings off any white objects. Any help or suggestions on how to improve reading would be appreciated.
Thanks in advance!

Hi @codeandstuf  In the areas of the cow where the ambient light is at its most intense, it blots out the skin texture of the cow and effectively makes it a low-texture or textureless surface that is difficult for the camera to read depth detail from (like a plain door, wall or table).  In these circumstances, you can use the camera's infrared dot pattern projection to read depth detail, as the dots act as an analyzable texture source in place of the detail on the object / surface itself.
The visibility of the dots to the camera can be increased by maximizing the Laser Power setting to '360' (its default value is '150').  In the RealSense Viewer, you can maximize Laser Power by using the slider beside the option under the Stereo Module > Controls section of the Viewer's options side-panel.
Normally the camera would also have difficulty reading depth detail from dark grey / black surfaces.  This is because these color shades absorb light.  This is not a problem in this particular scene though, as projecting a strong light source onto the scene can bring out texture detail from such colors.  So the strong ambient light is an advantage in this particular situation.  And the infrared dot-pattern projection should enable depth detail to still be read from the black areas of the cow if the ambient light weakens.
Where there are black empty areas with no depth detail on the image, you could fill these in by enabling the Hole-Filling Filter post-processing filter under the Post-Processing Filters section of the Viewer side-panel.
If you have the budget to purchase a physical optical filter product to overlay over the lenses on the outside of the camera, a linear polarization filter can significantly negate glare from reflections.  Section 4.4  When to use polarizers and waveplates of Intel's white-paper document about optical filters provides further information about this subject.
https://dev.intelrealsense.com/docs/optical-filters-for-intel-realsense-depth-cameras-d400#section-4-the-use-of-optical-filters
The image below demonstrates the glare negation that a linear polarization filter can provide.

Thank you @MartyG-RealSense! I have tried turing the lasers power up, but I attempted that on site and it's very possible I did not do it correctly. I will give that another try next time I am data collecting. The use of a linear polarization might be my best bet. Cost is a limitation for this project (its a research project and the real world use would be for farmers who are trying to save every penny they can), but  if the addition of a optical filter improves the image quality enough I belive it would be worth it. Do you have any recommendations on linear polarization filters? It might not be for a few months but I will update this post with the results of any solution I attempt to implement.
Lastly I want to say a huge thank you @MartyG-RealSense. I have read many of your previous posts and your advice has been extremely helpful to me in the past.

You are very welcome. :)
A RealSense team member advised about linear polarization filters in a past case that "any thin film polarizer will do that is linear.  The circular polarizers used most often in 3D glasses won’t work".
You could see some budget-priced examples of linear polarization filters by searching Amazon for thin film linear polarization
Information about a 3D printing blueprint for a clip to hold filters to the outside of a D435 camera model is at https://github.com/IntelRealSense/librealsense/issues/7384#issuecomment-881687933


iroh-net: reduce size of crypto keys

We have a lot of places where we are dragging iroh_net:: key ::PublicKey (size 224) around just to use it as keys.
The reason these things are so fat is that they contain some information that is expensive to compute. But I think in a lot of places this extra information is not needed and is just taking up space. This is probably costing us something wherever we use message passing because we end up with giant enums, but this is hard to track down in perf traces. Also every single key -> whatever map suffers.
How about this: we use a global map / intern table for deduplication, and e.g. a PublicKey becomes just a newtype for an [u8; 32]?

Ok, I tried removing Copy from PublicKey so we can do some fancy stuff in the background. After changing every single file of our code, I gave up. Will have to be something else...

Second attempt is having a cache but keeping the thing Copy. Not so nice internally, but you should never really see it.


fix: Issue #2890, Klondike Step5, re-work drags and taps to treat some short drags as taps.

Description

The objective of this fix is to make a tap on a card more positive-feeling for the player and not to disappear silently if it is interpreted as a drag.
It adds a Base Card to make an empty Stock Pile behave as a Card and use the tap and drag logic of the Card class. Any attempted drag on a Stock Pile card, including the Base Card, is now changed to a tap in onTapCancel() and the drag is not followed. The Base Card is rendered in outline only and does not take part in gameplay.
In other Piles a short drag is either treated as a tap or ignored. Only the Waste and Tableau Piles allow such taps. As before in Klondike Step5, they result in the tapped card moving automatically to its Foundation Pile if it is eligible to "go out".
As before in Klondike Step4 and Step5, all piles except the Stock Pile allow drags to start on them and they can finish on a Foundation Pile or a Tableau Pile.

Checklist

[x] I have followed the Contributor Guide when preparing my PR.
[-] I have updated/added tests for ALL new/updated/fixed functionality.
[-] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[-] I have updated/added relevant examples in examples or docs.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[x] No, this PR is not a breaking change.

Related Issues

Closes #2890.
When testing, try sliding the finger or mouse slightly while making a tap.

Thank you for your prompt review, Spydon. This PR #2894 is ready to be merged now.


A loss becomes a nan during training. The code is basically unchanged.

Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
在训练时，loss和decode.loss_ce变成了nan。源码除了数据地址部分，其他没有修改过。
Reproduction

What command or script did you run?

python Samba/tools/train.py Samba/configs/samba/samba_upernet-15k_potsdam-512x512_6e4.py --work-dir Samba/output/p --amp

Did you make any modifications on the code or config? Did you understand what you have modified?
源码除了数据地址部分，没有更改。
What dataset did you use?
使用的potsdam，目前只使用过这个数据集。
Environment

sys.platform: linux
Python: 3.9.19 (main, Mar 21 2024, 17:11:28) [GCC 11.2.0]
CUDA available: True
MUSA available: False
numpy_random_seed:<PHONE_NUMBER>
GPU 0: NVIDIA GeForce RTX 4090
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.8, V11.8.89
GCC: gcc (Ubuntu 11.3.0-1ubuntu1~22.04.1) 11.3.0
PyTorch: 2.0.0+cu118
PyTorch compiling details: PyTorch built with:
  - GCC 9.3
  - C++ Version: 201703
  - Intel(R) oneAPI Math Kernel Library Version 2022.2-Product Build 20220804 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.7.3 (Git Hash 6dbeffbae1f23cbbeae17adb7b5b13f1f37c080e)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 11.8
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_90,code=sm_90
  - CuDNN 8.7
  - Magma 2.6.1
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.8, CUDNN_VERSION=8.7.0, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wunused-local-typedefs -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_DISABLE_GPU_ASSERTS=ON, TORCH_VERSION=2.0.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=1, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.15.1+cu118
OpenCV: 4.9.0
MMEngine: 0.10.3
MMSegmentation: 1.2.2+997fe71

Error traceback
If applicable, paste the error trackback here.
2024/04/18 18:15:37 - mmengine - INFO - Iter(train) [  650/15000]  lr: 2.5977e-04  eta: 0:36:53  time: 0.1487  data_time: 0.0087  memory: 4364  loss: 1.4168  decode.loss_ce: 1.0096  decode.acc_seg: 41.0303  aux.loss_ce: 0.4072  aux.acc_seg: 35.1656
2024/04/18 18:15:45 - mmengine - INFO - Iter(train) [  700/15000]  lr: 2.7979e-04  eta: 0:36:41  time: 0.1509  data_time: 0.0097  memory: 4364  loss: 1.4327  decode.loss_ce: 1.0229  decode.acc_seg: 14.2394  aux.loss_ce: 0.4098  aux.acc_seg: 14.0891
2024/04/18 18:15:52 - mmengine - INFO - Iter(train) [  750/15000]  lr: 2.9980e-04  eta: 0:36:26  time: 0.1454  data_time: 0.0087  memory: 4364  loss: 1.3953  decode.loss_ce: 0.9931  decode.acc_seg: 61.1958  aux.loss_ce: 0.4022  aux.acc_seg: 57.2154
2024/04/18 18:15:59 - mmengine - INFO - Iter(train) [  800/15000]  lr: 3.1981e-04  eta: 0:36:06  time: 0.1341  data_time: 0.0080  memory: 4363  loss: nan  decode.loss_ce: nan  decode.acc_seg: 58.5070  aux.loss_ce: 0.4762  aux.acc_seg: 53.7278
2024/04/18 18:16:06 - mmengine - INFO - Iter(train) [  850/15000]  lr: 3.3983e-04  eta: 0:35:44  time: 0.1358  data_time: 0.0076  memory: 4365  loss: nan  decode.loss_ce: nan  decode.acc_seg: 42.6550  aux.loss_ce: 0.4075  aux.acc_seg: 56.2516
2024/04/18 18:16:08 - mmengine - INFO - Exp name: samba_upernet-15k_potsdam-512x512_6e4_20240418_181350
2024/04/18 18:16:13 - mmengine - INFO - Iter(train) [  900/15000]  lr: 3.5984e-04  eta: 0:35:26  time: 0.1427  data_time: 0.0112  memory: 4364  loss: nan  decode.loss_ce: nan  decode.acc_seg: 27.4019  aux.loss_ce: 0.3872  aux.acc_seg: 38.4336
2024/04/18 18:16:20 - mmengine - INFO - Iter(train) [  950/15000]  lr: 3.7985e-04  eta: 0:35:10  time: 0.1369  data_time: 0.0083  memory: 4363  loss: nan  decode.loss_ce: nan  decode.acc_seg: 32.8315  aux.loss_ce: 0.4617  aux.acc_seg: 45.6420
2024/04/18 18:16:27 - mmengine - INFO - Exp name: samba_upernet-15k_potsdam-512x512_6e4_20240418_181350
2024/04/18 18:16:27 - mmengine - INFO - Iter(train) [ 1000/15000]  lr: 3.9987e-04  eta: 0:34:54  time: 0.1326  data_time: 0.0076  memory: 4364  loss: nan  decode.loss_ce: nan  decode.acc_seg: 39.9736  aux.loss_ce: 0.4192  aux.acc_seg: 55.7047

希望能得到解答，感谢

Hello. Thank you for your interest and detailed information about the issue you're encountering with the Mamba framework.  It appears that the issue with loss and decode. loss_ce turning into NaN during training may be due to insufficient support for Automatic Mixed Precision (AMP) in Mamba.  You could try disabling AMP by removing the --amp flag from your training script to see if this resolves the problem.
Please feel free to communicate with me if you have further issues or need more assistance.

Yes, that solves the problem. Thank you very much. However, I also found a problem that without modifying the parameters, the results obtained were relatively low, far lower than the results of the paper, and the previous mmseg classical model (FCN, etc.). (In the vaihingen dataset)

Hi. Please kindly check if you exclude "clutter" class when calculating the mIoU metric.
In both ISPRS data sets, the clutter category needs to be excluded when calculating metrics, which I have already pointed out in the article. If you check the benchmarks for these two ISPRS datasets, you will find that everyone's experimental setups are similar.
Please kindly check if your experimental setup is the same as mine.

Thank you again for your attention, and feel free to communicate with me.

Thanks for your answer.Climb your mountain of success!

Thank you for your kind words. I wish you smooth sailing in your academic endeavors as well.!


Add missing integrity algorithm

Q
A

Bug fix?
yes

Improvement?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Docs PR
missing

I just pushed a small addition, so that we reuse the constant when generating the hash, as well as simplifying the code (we can make assumptions, if we just document them).


fix(runtime-dom): v-model can update correctly when the element is an input of type number

close: #7003

Wow, look at those merges.. really appreciated that you kept it in sync with the main branch throughout the year
If Vue had auto-publish (npm install vue-prs@fix-vmodel) on PR branches that would be so helpful, not just for the maintainer but also for Vue users
Here is a proposal for PRs

Explain PR and related issues
What is excepted and what's not
How to monkey test it

In this way, everyone could help and review/test the PRs
@sodatea sry for mention WDYT? I know it's hard but at least I said my opinion xd

The PRs being merged recently are largely bug fixes.
Feature PRs that get merged directly are also usually ones that are self-explanatory.
Big features / changes like Svelte runes will have dedicated RFCs at https://github.com/vuejs/rfcs.


Use UuidProvider instead of direct Ramsey\Uuid\Uuid call

https://github.com/Sylius/PayPalPlugin/blob/c1c5d4586c3e792d0e062f672b830840583018a1/src/Api/CacheAuthorizeClientApi.php#L66

On separate note, PayPalCredentials can use factory
https://github.com/Sylius/PayPalPlugin/blob/c1c5d4586c3e792d0e062f672b830840583018a1/src/Api/CacheAuthorizeClientApi.php#L65


[BUG] Always redirect to signin page.

@Ns2Kracy thanks for trying out gitness.
I see that you are running it under a custom domain - I've seen before that a collision of the auth cookie with other existing cookies can cause such a behavior.
To see if that's the case, can you start it with the following environment variable and see if that solves the issue?
GITNESS_TOKEN_COOKIE_NAME=gitness-token
In case you still experience the same behavior, it would be good if you can include the gitness logs from the container (and possibly the request that caused the 401)

GITNESS_TOKEN_COOKIE_NAME=gitness-token

Fixed! Thanks.

GITNESS_TOKEN_COOKIE_NAME=gitness-token

Not work, still redirect to login page.


TV/EPG tab missing

Bug report
Describe the bug
Here is a clear and concise description of what the problem is:

Kore Android app version 3.0.0 is missing the TV navigation tab. This is quite esential since it has the TV channels and EPG.
Expected Behavior
Here is a clear and concise description of what was expected to happen:

Show all tabs just like native Kodi does.
Actual Behavior

TV tab missing.
Possible Fix

According to https://github.com/xbmc/Kore/issues/23 it was in 2.0.0
To Reproduce
Steps to reproduce the behavior:

Open Kore app on Android smarthphone.
Notice tere is no TV tab the navigation menu.
Be disappointed.

Debuglog

The debuglog can be found here:
Not applicable
Screenshots
Here are some links or screenshots to help explain the problem:

https://ibb.co/8Mw0jLw
Additional context or screenshots (if appropriate)
Here is some additional context or explanation that might help:

Your Environment
Used Operating system:

Mobile device: Samsung Galaxy A32 5G
Android version: 13
Kodi version and platform: 20.3
Kore version: 3.0.0

note: Once the issue is made we require you to update it with new information or Kodi versions should that be required.
Team Kodi will consider your problem report however, we will not make any promises the problem will be solved.

PVR is TV, with EPG.

I see.
Why would you name it different from Kodi? Other tab names are consistent with Kodi (Movies, TV Shows, Music).

I recall it was 'PVR and TV' once, no?


Ubuntu Support

The module on Ubuntu seems to be broken, mainly because the default database provider for Ubuntu is (right now) hdb.
The recent change for Debian 8+ to have mdb as default cause all Ubuntu versions to get their default changed to mdb as well, because all of the current Ubuntu versions (10.04,12.04,14.*,15.*) have a major version number bigger than 8.
This means that when the module tries to remove the default olcDatabase={1}mdb.ldif file it can't find it, as the file is called olcDatabase={1}hdb.ldif.
A fix for that would be to introduce an additional switch statement on $::operatingsystem when the $::osfamily is Debian.

I'll work on a patch that works in our environment and submit it as a PR.


check styling inconsistency across browsers

There is non-scrollable horizontal scroll bar on firefox, but not on chrome.

I modified the outer-table class in /common/styles/app.css to use overflow-x: auto; instead of overflow-x: scroll; so that the scroll bar only appears when needed.


DataFile and RepertoireGroup modifications

Per #581, DataFile changes:

Remove AlleleDescription, DataProcessing, and Tree
Add GenotypeSet

Per the August 8, 2022 call:

Rename RepertoireGroup to RepertoireSet for consistency with other schema.

hopefully you can merge this and #581 without too much hassle

Yes, I hope so. Once this is in, then I can finish the R package and #581 should be good to go.  Everything else is done with it. (As near as I can remember anyway - I'm sure I forgot something and we'll need to patch release.)

I don't think I understand why Clone is a top-level object but Tree isn't. Neither is an object which is "within" another object (at least if I am reading the schema correctly).

Good point. It doesn't look like it's setup that way in the schema. Should it be?
Though, my motivation for cutting is more so I wouldn't have to figure out how to parse Tree.properties.nodes.additionalProperties.

Also, @schristley and I were on the fence about cutting Clone, so I went with "no".

Should it be?

I.... need to think more. Let's leave it for now and revisit for 2.0?

I.... need to think more. Let's leave it for now and revisit for 2.0?

Seems wise.

Should it be?

I.... need to think more. Let's leave it for now and revisit for 2.0?

It's that way now because there was the use case that multiple trees could be defined for a clone, so instead of embedding a single tree into the clone object, Tree allows multiple to be defined.


Question on psf error map

psfr version: latest
Python version: 3.9
Operating System: Linux

Description
In the psf error map calculation, the expected noise level (i.e. sqrt(error_map_list[i])) is subtracted from the residuals (lines 358-360).
Here are the lines of code I am talking about:
        # subtract expected noise level
        if error_map_list[i] is not None:
            residuals -= np.sqrt(error_map_list[i])

If I understand correctly, the goal is to avoid double counting the errors, i.e. separate the "expected gaussian+poisson noise" to the one which comes from a true difference between the star and the estimated psf, the latter being the only one which should be taken into account in the psf_error_map. Is it correct ? If so, why are we subtracting simply instead of quadratically substracting ? I am not sure of the right answer, I just wanted to be sure we are consistent.

Hi @LyneVdV ,
thanks for the question. Indeed, quadratically subtracting would be the better/proper way. The question is just how that can be achieved when at the end we want the median (or mean) absolute difference of the model relative to the data for a set of multiple point sources. The way I've currently implemented it is more conservative in the sense that it suppresses the error map a bit.


"ADD  /dir" copies the file instead of untarring it

In this context, https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar is tarball to a minimal container.

This works:
docker import https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar test1
docker run -t -i test1 sh

This works too:
curl -L https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar | tar x
cat >Dockerfile <<EOF
FROM scratch
ADD . /
EOF
docker build -t test2 .
docker run -t -i test2 sh

This doesn't work:
cat >Dockerfile <<EOF
FROM scratch
ADD https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar /
EOF
docker build -t test3 .
docker run -t -i test3 sh

This copies root.tar to /testroot:
cat >Dockerfile <<EOF
FROM test1
ADD https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar /testroot
EOF
docker build -t test4 .
docker run test4 ls -l /testroot

This copies root.tar to /testroot/root.tar:
cat >Dockerfile <<EOF
FROM test1
ADD https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar /testroot/
EOF
docker build -t test5 .
docker run test5 ls -l /testroot

This works:
curl -L https://gist.github.com/jpetazzo/4353387a2688f22af217/raw/fd305c52ee29e148f6c021abeb1766a408d8b9a1/root.tar > root.tar
cat >Dockerfile <<EOF
FROM scratch
ADD root.tar /
EOF
docker build -t test6 .
docker run -t -i test6 sh

Initial discussion on docker-user: https://groups.google.com/forum/#!topic/docker-user/TQpAtr-d4eo

@jfrazelle just to aknowledge I already read about your policy to prevent Dockerfile syntax changes on #3050.
Personally I think this issues falls apart from what you all were discussing on #3050. Seems that here people agreed that ADD syntax splitting could solve this issue, and pointed there but as it is closed, this should be now waiting for another solution, instead its closed.
Now, IMHO and backing up to the OP question, since ADD command extracts local packed-files instead of just copying them, makes sense to do the same with remote packed-files . Before achieving this opinion, I had to think about if this is or not a patch, and if it is, if it could be an BC issue that forces docker team to don't implement.
After reading initial discussion and since docker import could be comparable with ADD here, and its behavior with remote packed-files is to extract them, this seems to be a bug that requires a patch that will cause a large BC, just because people had to include .tar on their repos and/or external files to Dockerfile to download packed-files so docker could import them.
Unfortunately, it also seems this was kept and documented as "normal" behavior.
Quoting semver docs:

What if I inadvertently alter the public API in a way that is not compliant with the version number change (i.e. the code incorrectly introduces a major breaking change in a patch release)

Use your best judgment. If you have a huge audience that will be drastically impacted by changing the behavior back to what the public API intended, then it may be best to perform a major version release, even though the fix could strictly be considered a patch release. Remember, Semantic Versioning is all about conveying meaning by how the version number changes. If these changes are important to your users, use the version number to inform them.

So, again IMHO, I think this needs to be opened, discussed and fixed on next major.
--
José Moreira


Where is the new operator's manual?

Please answer these questions before submitting your issue.

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

Question

What do you want to know?

Bug

Which version of SkyWalking, OS, and JRE?

Which company or project?

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

Requirement or improvement

Please describe your requirements or improvement suggestions.

Please read the website page before you submit this kind of issue.
https://github.com/apache/skywalking-swck
https://skywalking.apache.org/docs/

Please read the website page before you submit this kind of issue.
https://github.com/apache/skywalking-swck
https://skywalking.apache.org/docs/

提交此类问题之前，请阅读网站页面。
https://github.com/apache/skywalking-swck
https://skywalking.apache.org/docs/

There used to be principle pictures, but now it's just a brief introduction. Are there no configuration items in the project?
Is this the so-called new operator's manual? It is recommended to refer to Seata's rookie Handbook.

提交此类问题之前，请阅读网站页面。
https://github.com/apache/skywalking-swck
https://skywalking.apache.org/docs/

There used to be principle pictures, but now it's just a brief introduction. Are there no configuration items in the project?
Is this the so-called new operator's manual? It is recommended to refer to Seata's rookie Handbook.

You are asking questions from wrong perspective. We trust our PMC and committer team. They think it is enough, then, it is.
If you want to contribute more details, we are welcome.

You are asking questions from wrong perspective. We trust our PMC and committer team. They think it is enough, then, it is.
If you want to contribute more details, we are welcome.

I am a new user. If I want to use a service, I will first describe how to operate in the specific configuration items of the server and project side, rather than the general description. Therefore, there is no wrong point of view in my question.
If the founder of the service thinks that it is enough, then as a user, I have nothing to say.

I am a new user. If I want to use a service, I will first describe how to operate in the specific configuration items of the server and project side, rather than the general description. Therefore, there is no wrong point of view in my question.
If the founder of the service thinks that it is enough, then as a user, I have nothing to say.


[Doc Improvement] [Provision cloud resources]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/provision.md
91
56
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 63a1f02:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/toolkit/provision.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/toolkit/provision.md
91
56
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 39cd7c5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/toolkit/provision.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Clsing this PR


Responder Creation Issue

Responder Creation Issue
Request Type
I am currently experiencing an odd issue which I believe might be a bug with a responder I am creating for thehive that requires data to be pulled from thehive and returned - I receive a proxy error upon attempting this. I have had numerous people review the code and suggest alternative things for this, but no success as of yet.
Work Environment

Question
Answer

OS version (server)
Cortex docker 3.0.1-1

OS version (client)
TheHive docker 4.0.0-1

TheHive4py version / git hash
TheHive4py 1.8.0

Problem Description
The responder performs the simple task of extracting a case observable ('url') and importing it into an email to be sent. However, when I run the responder, I get the below error - we run our instance in front of an Nginx proxy:

Steps to Reproduce
    from cortexutils.responder import Responder
    import smtplib
    from email.mime.text import MIMEText
    from email.mime.multipart import MIMEMultipart
    from thehive4py.api import TheHiveApi
    from thehive4py.models import CaseObservable
    from thehive4py.query import Eq, And
    import ssl
    import sys
    import json
    import requests

    #Configure Mail Settings
    class email_phish_alert(Responder):
        def __init__(self):
        Responder.__init__(self)
        proxies = {'https': '***', 'http': '***'}
        api = TheHiveApi('*TheHive Address/Port*', '*API KEY*, proxies=proxies)

    #Build query
    query = And(Eq('dataType', 'url'), Eq('ioc', True))

    #Call the API to search all case's URL observables
    response = api.get_case_observables('case_id', query=query)

    #Display the result
    if response.status_code == 200:
        #Get response data
        list = response.json()

    else:
        print('Failure: {}/{}'.format(response.status_code, response.text))

I have removed the parts of the code relating to the emailing side of things, as that has been tested and is fine. The only issue remains with getting the data and pulling it to display. I have a feeling it could be linked to bug reported #201 however the format used in that code has not been successful for myself.
Complementary information

Does not look to me like responder creation issue. What you describe and what logs are showing (403) is that you have a proxy that requires authentication and you did not authorised your request properly.

Does not look to me like responder creation issue. What you describe and what logs are showing (403) is that you have a proxy that requires authentication and you did not authorised your request properly.

Unfortunately this is something I've tried - our proxy requires no authentication, and I've even tried specify that it doesn't use a username/password when passing through it, however the same issue persists.


Image not showing up in Ubuntu 16.04

I followed every step as described in the installation process, but nothing happened. The is installed, I have looked into the directory, but the desktop background didn't change.

yup, this is true. the background image did not change automatically with crontab configuration, but only worked with manual run on terminal. why?

cron only has access to a very restricted set of environment variables - the python script here makes use of one that is not available in that case.
see SO answer here:
https://stackoverflow.com/questions/10374520/gsettings-with-cron/19666729#19666729

Hey,
I tried and experienced the problem myself as well, although I didn't encounter it before (not sure if they changed the behaviour of services [if not, then how did it work for so long?]).
Anyway, not only the script makes use of environment variables but also gsettinngs (on $DISPLAY) which himawaripy heavily relies on. I'm trying to find a way without hard-coding environment variables, but it might be our last resort.

i have managed to fix this issue as a walk-around approach. basically, the link that @ngrande  shared above helped a lot.
that is, i have created a shell script as the below, and added in my crontab as the same period as the himawaripy executable file. this seemed to work very well so far.
#!/bin/bash
# http://bit.ly/2ines3u
# TODO: At night only dark wallpapers.

# Wallpaper's directory.
dir="${HOME}/.cache/himawaripy/"

# export DBUS_SESSION_BUS_ADDRESS environment variable
PID=$(pgrep gnome-session)
export DBUS_SESSION_BUS_ADDRESS=$(grep -z DBUS_SESSION_BUS_ADDRESS /proc/$PID/environ | cut -d= -f2-)

# Random wallpaper.
wallpaper=`find "${dir}" -type f | shuf -n1`

# Change wallpaper.
# http://bit.ly/HYEU9H
gsettings set org.gnome.desktop.background picture-options "spanned"
gsettings set org.gnome.desktop.background picture-uri "file://${wallpaper}"

@soohyunc answer should be in README. Thank you!

Fixed! You should now be able to use it as described in the README without any additional scripts. Let me know if you have any further questions. =)

OR PID=$(pgrep gnome-session | head -n1)


refactor(oxlint): remove extra if check in Walkdir

#7989  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.


Parallel performance on CPU

thrust::sort is not executed in parallel on CPU. Tried running with flags thrust::system::cpp::par and thrust::host.
Attached sample to compare GPU, CPU using Thrust, and CPU parallel implementations.
CUDA_sort.zip

We chatted about this by telecon today; the issue here is that the default Thrust "host" system is a sequential backend. To use one of the parallel host systems, you need to set a pound define:
#define THRUST_HOST_SYSTEM THRUST_HOST_SYSTEM_OMP

Or set it on the command line.
There are three options for host systems: OMP (which requires OpenMP), TBB (which requires the TBB library), and CPP (which always executes sequentially).


"<" related errors in Decompiler References (Find Uses of) Window

Describe the bug
The "<" token results in unexpected output in the Decompiler References (Find Uses of) table, in particular, in the Context column
To Reproduce

Find an expression of the form if (a->b < 0x__) or if (0x__ < a->b)
Use the Find Uses Of feature on the b field.
In the Context Column, the "<" token is invisible.
Searching for "b < 0x" or similar in the filters yields no results
Using the Create Column Filter, select "Context", "Contains", and type "<" into the textbox.
html> blah blah blah results popup in the preview.

Expected behavior
"<" tokens should display properly and be searchable in the Context Column. The html should not appear in the search window.


Reformat CLI command name using kebab case

Cosmos SDK and Tendermint use kebab case (word-word-word) for name of commands for the CLI but in Chainlink module we are using camel case (wordWordWord).
In order to have consistent commands name with the rest of the ecosystem, we should rename commands.
E.g:
addFeed -> add-feed
addDataProvider -> add-data-provider
addModuleOwner -> add-module-owner

We should also fix the different scripts and CLI tests with the new commands name

yeah we definitely need to do it!


Added imagecounter component ie counter script issue#4

hey @ta0a2000t  Here is my PR please review and do give me a feedback if there is any required change needed

hey @ta0a2000t  i have done the commit here but that sync thing you asked me to do for that i have created another PR Check that if it is what is required and if not then please let me know I'll try again


test(dtslint): add infer tests to distinctUntilKeyChanged

Description:
This PR adds infer tests to distinctUntilKeyChanged.
Related issue (if exists): #4093

Coverage remained the same at 96.809% when pulling 4e04773811fc51a5f5a4c86f83c5ab827453ae10 on timdeschryver:pr/dtslint-distinct-until-key-changed into d032804c0b0ef1967bc9da2a2b4202c41fb1121e on ReactiveX:master.


[Question] Is there any way to manage application body classes with Helmet?

Title ^

No, sorry, Helmet is meant to manage your document head.  I assume you are referring to classes on the body tag? - <body class="application test"> ...

Bummer. Helmet would be perfect for such a thing.

Actually this could be something we consider, we did open up attributes for the html tag.  We could extend it to support body attributes.  Great idea.  Curious, do you have a particular use case?

Lots! Body classes are an awesome way to control core layout changes of an application. For example, I have a few pages of my app that need a full-screen header treatment instead of a singular centered-column. For example, the user can switch from a search results list to a search results map view - a much more full-screen layout.
The page component I wrote is extended by each page, and part of the constructor involves calling super(props, ['fullscreen-header']) where the Page base class applies these classes to body during ComponentDidMount (amongst many other things).
It's a workable solution, but I'd love to get these body classes integrated into my server-side rendering to avoid the split second where the body class isn't applied on load. Doing so would require me to rework a solution that would ideally be similar to how we're using Helmet for meta/title information, but I'd hate to reinvent the wheel there.
I suppose I could use  attributes to achieve this instead of body classes though

@mmmeff I think you are correct about using attributes or potentially serving a different style sheet per the layout that you want.
For now, we'd like to keep this library focused mainly on the document head.  Thanks.

Just thought I'd mention that I was interested in this feature as well for the same reason (applying classes to the body element to swap to a full screen mode).

I'll add myself to the queue of people interested in setting body classNames.

Helmet now supports classes to <html>, does that work for you? @codepunkt

Amazing, that should work! Thanks @cwelch5!
How do you do it, using the htmlAttributes?

👍

Was looking for something for this and came across this: https://www.npmjs.com/package/react-body-classname
Might be helpful. :)

@stefvhuynh If i absolutely had to use body classNames, that would probably be a good option. As i was using react-helmet anyway, i switched to setting html classNames using htmlAttributes so that i wouldn't need another dependency.

@codepunkt, yeah that makes sense. In my situation, the body class was necessary as it was a transition project, so I ended up using my own version of the package I pointed to with some modifications.

Was looking for something for this and came across this: https://www.npmjs.com/package/react-body-classname

@stefvhuynh Careful with that one, make sure you call rewind, otherwise you'll get a BIG memory leak:

Important: This component keeps track of mounted instances, so if you don't call BodyClassName.rewind() you'll get a memory leak.

There is at this point (for quite a while now but apparently not at the time this thread was active) a bodyAttributes attribute to the main Helmet object, so you can add classes to the body for a particular page. https://github.com/nfl/react-helmet/blob/2aecac5d0df4bc208fb11394e867b1b1ddcd5a5f/src/Helmet.js#L35

And strangely enough, i found that for it to work right when running my site with gatsby develop, i needed to pass it in like:
bodyAttributes={{ class: 'landing-page-body' }}

instead of saying className as you'd expect to do based on the comments in react-helmet. And using class instead also works when the site is built by Gatsby, so that's what i'm going with


Pylint docs and removing wildcarding

Signed-off-by: Nadia Bahrami<EMAIL_ADDRESS>

Please don't change files outside the scope of your PR.
If Michael or Ned approve, wait for another maintainer.

Sorry I asked for them to review.  I didn't know we had requirements of maintainers.  Who are the maintainers just so I know in the future?

Try not to touch files modified in other pending PRs.
tests/rbac/common/role/role_manager_test.py was deleted in PR #398
https://github.com/hyperledger/sawtooth-next-directory/blob/master/MAINTAINERS.md


OTP is not getting validated

Running VacCowin...
Enter the Registered Mobile Number: ********
Successfully Requested OTP for the Mobile Number ******* at 2021-05-25 11:44:33.155353..
Enter OTP (If you don't recieve OTP in 2 minutes, Press Enter to Retry): ******
Validating OTP. Please Wait...
Unable to Validate OTP...
Response: {"message": "Endpoint request timed out"}
Want to Retry with the<PHONE_NUMBER> ? (y/n Default y):

@Anwit I've updated this issue in the last commit. Please try again. Should work now.


InconsistentClusterIdException: The Cluster ID doesn't match stored clusterId

Hello,
I am using the single kafka, single Zookeeper configuration here: https://github.com/conduktor/kafka-stack-docker-compose/blob/v6.2.1/zk-single-kafka-single.yml
For the most part it works fine, except sometimes when I restart the containers I am getting this:

kafka.common.InconsistentClusterIdException: The Cluster ID xxx doesn't match stored clusterId Some(yyy) in meta.properties. The broker is trying to join the wrong cluster. Configured zookeeper.connect may be wrong.

I am guessing the old cluster ID yyy remained there on docker compose down/stop and was not flushed.
The proper fix advised on StackOverflow is supposedly to persist Zookeeper/Kafka data, but I think it's already done with the volumes.
My only workaround at the moment is to prune all the volumes to start over from a clean state, but surely there is a better way to resolve this?
Edit: it seems that the latest version removed volumes support. Was it removed because of this issue, or because of something else?
Thank you!

I ended up removing the Kafka and Zookeeper volumes, since I did not need to persist them. This effectively resets them every time I run the project but it's not an issue for me and it solved this issue.


resource linking fix

It looks like line endings changed?  You may want to pick a consistent line ending settings for the repository:
https://github.com/Microsoft/BotBuilder/blob/master/.gitattributes#L4


Use opaque composition if transparency is disabled

The Vulkan driver used transparent composition always when possible, which led Wayland to show all alpha data as transparent even if per pixel transparency was disabled.
I have no idea if it's possible to easily control the buffer type like on OpenGL (which is what is done on X11 and also on the WIP Wayland backend), but this looked like the easiest/cleanest change.
Feel free to close this PR if it's not a proper fix and sorry for not making a proper issue, but I'm not sure where to start.

Looks good, thanks!


Feature request: EDNS EXPIRE (RFC 7314)

Hi. Long chains of XFR servers can lead to a situation where a zone's expiry is extended well beyond what's in the SOA record. We have recently had this situation with some of our zones, where the secondary kept serving a zone with expired RRSIGs.
Would you consider implementing RFC 7314 in Knot DNS NSD, both when providing XFR as well as requesting XFR, and honouring the expiry from the EDNS EXPIRE option instead of the SOA record?

I copy+pasted from the same request to the Knot DNS folk... oops, haha.
s/Knot DNS/NSD/ :)

It turns out that Knot DNS has implemented this since version 3.2. It has been in BIND for even longer. I would love to see this in NSD, so that we can make use of it uniformly. Other than in XFR, it also helps when you issue a SOA query, because you can quickly know how far a zone is from expiry, and could use it as a monitoring aid.

Hi @k0ekk0ek. Any update on this issue?

Hi @anandb-ripencc. I'm sorry, not yet. It got sidetracked. I'll see if I can start work on this again soon (#278, or simdzone, is keeping me busy, but a first release is close).

@anandb-ripencc, #278 turned out to take (way) more time. I'm sorry it took so long. I'll get started on this feature later this week.

I've been working out how to fit this into NSD. The problem is that the processes serving the data do not keep track of zone administration as that is done by xfrd. The initial idea was to use a shared memory segment containing an expire timer per zone. As multiple versions might be served (current version, plus version after reload), that is not as straightforward as I hoped it'd be. Anyway, just a quick update to indicate this is top of my list.

Thanks for this update Jeroen. Perhaps you can try to solve this issue in 2 parts. It would already be useful if XFRD were to ask for, and honour the EXPIRE option in XFR queries. This would solve the problem where we have multiple chains of XFR servers, and the zone expiry time is extended beyond the operator's intention. This is my main motivation for wanting EXPIRE support in NSD, because we have actually faced this issue.
Later, you can try to figure out a way of passing on the expiry timer information to the child processes.

That's most certainly less complicated, I'll see if I can split it up.


Update Readme

Fixes since type mandatoriness

Codecov Report

Merging #836 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #836      +/-   ##
============================================
+ Coverage     66.48%   66.51%   +0.03%
- Complexity     1749     1750       +1
============================================
  Files           286      286
  Lines          6081     6081
  Branches        700      700
============================================
+ Hits           4043     4045       +2
+ Misses         1789     1788       -1
+ Partials        249      248       -1

Impacted Files
Coverage Δ
Complexity Δ

.../com/scottlogic/deg/generator/utils/IsinUtils.java
87.03% <0%> (+3.7%)
26% <0%> (+1%)
:arrow_up:

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 6c60821...86631fc. Read the comment docs.


(fix)registry: fix configmap registry server liveness probe timeouts

Description of the change:
In #1378, the readiness probe initial delay and timeout were bumped
to account for node resource constraints during pod startup. This
PR bumps the liveness probe delay and timeout values too, to account
for scenarios where node resources continue to be overloaded causing
the liveness probe to fail temporarily.
Motivation for the change:
Failing readiness/liveness probe fires prometheus alerts for
CatalogSources backed by configmap registry servers.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive

/lgtm

/lgtm

/approve

Rerun the unit test due to flakes.

/cherry-pick release-4.8