Could not find a version to edit for app 'AppName'

Issue Description
We suddenly have an error [!] Could not find a version to edit for app 'AppName' without changing our fastlane config.
We were wondering if this could be due to itunes connect recent update?
Complete output when running fastlane, including the stack trace and command used

[!] Could not find a version to edit for app 'AppName'

Environment

 fastlane 2.96.1

I got the same error . Have you solve it ? please...

How did you solved it? Without changing any settings, it seems to arise every now and then...

makesure there is an appstore version waitting   committed with correct version number


Tab icon color does not get highlighted when swiping between tabs.

My tabs have this indicator:
func indicatorInfo(for pagerTabStripController: PagerTabStripViewController) -> IndicatorInfo {
    let active = UIImage(named: "active")
    let passive = UIImage(named: "passive")
    return IndicatorInfo(image: passive, highlightedImage: active, userInfo: nil)
}

And it shows correct images only if you click on tabs. If you swipe between tabs, they are not updated. Is there something I am missing?

facing the same issue.

Any updates here?

Possible workaround:
Set cell's isSelected property in changeCurrentIndexProgressive callback
oldCell?.isSelected = false
newCell?.isSelected = true

Your workaround worked. @papipons

@papipons How exactly and where did you implement this callback?

@yigitserin
public override func viewDidLoad() {
    super.viewDidLoad()
    changeCurrentIndexProgressive = { [weak self] (oldCell: CategoryIconWithLabelCell?, newCell: CategoryIconWithLabelCell?, progressPercentage: CGFloat, changeCurrentIndex: Bool, animated: Bool) -> Void in
            guard changeCurrentIndex == true else { return }
            oldCell?.iconLabel.textColor = self?.unselectedIconColor
            newCell?.iconLabel.textColor = self?.selectedIconColor

            oldCell?.isSelected = false
            newCell?.isSelected = true
        }
}

neither work for me

same issue

@ajkerfaisa You implemented that code in which file?

@nowperson
I have implemented this in BaseButtonBarPagerTabStripViewController. It will make more sense if you look in this example: https://github.com/xmartlabs/XLPagerTabStrip/blob/master/Example/Example/Youtube/YoutubeExampleViewController.swift

@ajkerfaisal Thanks a lot for your response!


How To Accordion - Discover Page V2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.00%. Comparing base (0c5ee58) to head (31dac01).
Report is 55 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #1313      +/-   ##
==========================================
+ Coverage   42.72%   44.00%   +1.28%
==========================================
  Files          97       96       -1
  Lines       22896    22644     -252
==========================================
+ Hits         9782     9965     +183
+ Misses      13114    12679     -435

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Discussion: What shall we name the "low-level" API feature?

#139 Introduces a feature called "low-level API" which is the Pytorch Lightning-like API feature but I don't like that is sounds a bit obscure.
Potential options:

Low Level API (current)
Framework Agnostic API
???

Open to suggestions.

Since elegy is a lament for the dead, maybe we can call the low level API -inferno API-? In all seriousness, agnostic API?

melody ? I checked the formal definition of "elogy" and it mentioned "melody" 🎶

In all seriousness, agnostic API?

@Davidnet @cgarciae IMO, referring to it as "low-level" is probably more intuitive for new folks, since elegy supports haiku, optax and flax (some users may need to get used to what these library-frameworks mean and what they can do.)


Document how to handle path parameters.

There's no documentation on how to support path parameters required by our backend apis.

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

ID: c254cd61-0ff3-bd87-f088-6f9916e056c6
Version Independent ID: 1426c1ee-1979-f58c-8276-2afe8e6600b5
Content: Add an API manually using the Azure portal
Content Source: articles/api-management/add-api-manually.md
Service: api-management
GitHub Login: @mikebudzynski
Microsoft Alias: apimpm

@rabrahamOpti Thanks for the feedback! We are currently investigating and will update you shortly.

@rabrahamOpti You can parameterize the path of an operation as mentioned in this section of the doc.
By default, the same URI path is appended as is to your backend service URL.
You can override this behavior with these policies -

To override the backend service URL, use the set-backend-service policy
To override the URI path, use the rewrite-uri

Thank you @PramodValavala-MSFT Apologies, I was searching the documentation for "path parameters" and missed this section you referred me to. Feel free to close this.

@rabrahamOpti No problem at all. Glad we could help! 😄
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Connection to a ssl protected database

Hi,
i have the following configuration, with compreface run via docker-compse and an Azure Postgres server.
When running as single-container it somewhat went running, but only changing my profile did work, while every other action, like accessing an applicatoin resultet in a 502 bad gateway.  So i ran them as multi-container solution, with the compre-api always crashing.
In the logs i did find this

So i tried to set ssl settings via the conenction string, but that resulted jsut in the folowing

So finally i disabled ssl on the database server, and everything working fine. But running the connection unecrypted can't really be the solution, so I am wondering, if there is a way to configure compreface to use ssl with the database.

Hi @detresed
We are using two different datasource types in compre-api. SSL can be configured as usual for the first one, but it differs for the second one. In order to configure the second datasource properly as well, you can use the following guide:


Broken link to Azure portal

The link to deploy an Anomaly Detector resource is broken as this is no longer available via the marketplace.  The link needs to deploy an Azure AI service instead.

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

ID: 884b5e88-48eb-c3ad-1995-7cd87339783b
Version Independent ID: e06cfd58-46d0-200a-5967-c6a1a7276997
Content: Quickstart: Anomaly detection using the Anomaly Detector client library - Azure AI services
Content Source: articles/ai-services/Anomaly-Detector/quickstarts/client-libraries.md
Service: azure-ai-anomaly-detector
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@leehs-msft
Thanks for your feedback! We will investigate and update as appropriate.

@leehs-msft
I've delegated this to @mrbullwinkle, a content author, to review and share their valuable insights.

Thanks for the feedback. We'll go ahead and create an internal tracking work item and close this GitHub issue. We'll circle back here once we have a resolution.
#please-close


examples & without using cmd

I took a look at your bundles cause I think it's what I need as far as organizing resources goes, but I'm not sure.  the clone was severely lacking examples and use cases.
I'm building a language that dynamically generates MVCS/W based on data sets from multiple sources (aws rds/aurora, dynamo, mongo, postgre, etc.) and uses those to create ExtJS code, and Swagger APIs.   Anyhow, I like the idea of bundling apps together in a way where the resources are accessible and less disorganized.   You don't state your rationale in the Readme.md  ... if you have time hit me up on hc:irc(freenode:disruptive)  or<EMAIL_ADDRESS>https://www.hipchat.com/invite/513222/8ed949451ff5a195079c84e1bd1e633b?utm_campaign=company_room_link
Here's the project I've been barely starting, much of the code is crypto and/or written in go/php/ruby/bash and designed for docker/heroku with elastic beanstalk or EC2 AMI.
https://github.com/largesoft/skel
I'm still at odds as to whether or not to generate columns & fields from JSON and load them with the Ext.Loader, or if I should just have the server-side language create the models and corresponding stores, grids, etc.
For instance:  right now I can generate columns in JSON or array
http://a.badasscrm.com/skel/make.columns.php?c=contacts
http://a.badasscrm.com/skel/make.columns.php?c=contacts&f=array
It has become apparent to me that it's not common to pull the fields/columns/associations/validations from JSON, but rather to write the models.  So I guess I'm leaning more towards just creating the models dynamically and linking it to an api.  But I'm gonna make my own shorthand for it using functions like this:
function createModel(fields) {
return Ext.define('Rex.store.reading.Skel' + Ext.id(), {
        extend: 'Ext.data.Model',
        fields: fields,
        proxy: {
            type: 'memory',
            reader: {
                type: 'json',
                totalProperty: 'tc',
                root: 'Result'
            }
        }
    }
});

The end result would allow the user to create sophisticated apps, because the software is designed to allow them to manage fields/columns across a broad spectrum or applications.
https://www.lucidchart.com/documents/edit/4a9babed-a1be-46e6-8f18-aebefcbbca0e?shared=true&
I've been doing web a long time, and it's a very fun project cause of the end goal for me.  I love AWS and am devout ;)
.... So I'm not sure if this fits in with what you are doing... but yours is the closest thing I've found to a packaged solution that can be distributed for ExtJS open source with software like mine in container apps for the purpose of rapid deployment.
Cheers!
<< sorry for the long rant... I do that.... haha >>

also just fyi I am a long time tango dancer.  LOL


[Update] Use latest API signature

Description
Use the latest API for dynamic entity data source.
Linked Issue(s)

swift/pull/4393

How To Test
Build the app and there is no warning on this line.

Thank you.


the example failed the expected

Hi, I run your example and got the following result. It's mis-aligned, is there  anything setting i missed? what's your font family?

That looks very wrong, yes :(
I've tested this on the Alacritty and iTerm2 terminals, and I tried it with the Menlo, Input, and Azeret Mono fonts.  I have not seen output like your screenshot.  What terminal/font are you using?

Mac, iterm2 Build 3.4.15,

I tried several monospace fonts, and can't get the aligned version as yours in README.

I was able to reproduce this: it's the Ambiguous characters are double-width option.  Disabling that shows the proper layout.
I think it's the ─ character in particular.  There may be some other unicode characters that would work better here, but that character set was intended for box drawing so if this is broken then I'm not optimistic others will work.  I'll experiment more with this when I get some time.

Do you mind if I rename this bug to Default borders misalign when using double width for ambiguous characters for clarity?

https://github.com/mattn/go-runewidth/issues/14 may be related

https://github.com/charmbracelet/lipgloss/issues/40 also may be related

Disable Ambiguous Characters are Double Width and try setting export RUNEWIDTH_EASTASIAN=0 environment variable before running.


Search - "You appear to be offline" shown briefly before expenses appear when internet is off

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.37-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause Internal Team
Action Performed:

Open the staging.new.expensify.com website.

Navigate to search section.
Verifiy the expenses list is automatically displayed.

Expected Result:
When navigating to search section, the expenses list should be automatically displayed, despite the user being in offline mode.
Actual Result:
"You appear to be offline. This feature requires an active internet connection" message is displayed for a second, when navigating to search after turning internet connection off.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

https://github.com/user-attachments/assets/05c4929a-33c0-439b-9d9f-d978b1df54d8

View all open jobs on GitHub

@kadiealexander  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

This is the intended behavior. Closing.


How to access websocket connections in GET, POST,..etc routes.

Please this isn't a bug and can be labeled FAQ.
The uWebSockets manual strongly disapproves of using std::map to store websocket connections, that's why there's a userData structure available.
But assuming we have 3 websocket connections with unique ids (assigned via userData struct), how can we access the websocket connection outside of the websocket routes (i.e inside get, post...etc routes).
Thanks.

That's not up to me. You're supposed to make your own decisions. I don't know what's best in your case and it seems you've read the manual so do whatever you feel like

Hi @alexhultman Thanks for your response. I understand that use-cases differ per application but can you just do a small example showing how you would go about sending a websocket message to a user with an active websocket connection with a unique id (via userData) in a GET route.
Thanks.

Your question has nothing at all to do with this library.
If you cannot figure the pieces out then you need to go back to the basics first. You can't run before you know how to crawl and I don't crawl with people.


Don't rename doc.go since it's expected by k8s code generator

Currently fake client-go client sets are broken, e.g. we have[1]
var gatewaysResource = schema.GroupVersionResource{Group: "networking", Version: "v1alpha3", Resource: "gateways"}
which should really be
var gatewaysResource = schema.GroupVersionResource{Group: "networking.istio.io", Version: "v1alpha3", Resource: "gateways"}
This is because

kubetype-gen generated apis are missing groupName comment tag (addressed by istio/tools#549)
k8s code generator expects groupName comment tag to be in doc.go but it is currently getting renamed to doc.gen.go

This change fixes 2.
[1] https://github.com/istio/client-go/blob/21751e6cf0fea131872230940d95c40508aa0a4f/pkg/clientset/versioned/typed/networking/v1alpha3/fake/fake_gateway.gen.go#L21

Hi @skaslev. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/assign @jwendell

@skaslev do you a test/example you can add that verifies fake client, we can make sure that code compiles as part of the CI process so this doesn't break again.
Please also cherry-pick this to release-1.4 branch. Thanks!

/ok-to-test

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
c5238ec42698f78ed44341303071ee7aa936a070
link
/test gencheck_client-go

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

@skaslev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
c5238ec42698f78ed44341303071ee7aa936a070
link
/test gencheck_client-go

lint_client-go
1311251d9616283b3d4fd9ab7aff10dc3e81b26d
link
/test lint_client-go

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

@skaslev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go
1311251d9616283b3d4fd9ab7aff10dc3e81b26d
link
/test lint_client-go

gencheck_client-go
1311251d9616283b3d4fd9ab7aff10dc3e81b26d
link
/test gencheck_client-go

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

@skaslev do you a test/example you can add that verifies fake client, we can make sure that code compiles as part of the CI process so this doesn't break again.

I don't have one atm. This was noticed while migrating Knative to use istio.io/client-go instead of its own apis for istio, see knative/serving#6041. Some of Knative tests were hanging and the informer was logging errors when using the fake clientset.
I'll try to think of a good test for this.

Please also cherry-pick this to release-1.4 branch.

Will do. Is there a documentation of the process how to do that?

Will do. Is there a documentation of the process how to do that?

Yes, you just need to add the label "cherry-pick/release-1.4" to your PRs. I'm not sure if you're allowed to do that if you're not a member, give it a try. If it doesn't work I can do it for you.

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
8b0920022172098f99a0aef56b4711b7eb1afed9
link
/test gencheck_client-go

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

/hold
There are two issues with this PR:

gencheck_client-go is failing because its using kubetype-gen before istio/tools#549 got merged. Probably some container image should be updated before merging this?
lint_client-go is failing because new kubetype-gen generated doc.go have no license header and doc.gen.go were ignored by lint-copyright-banner. I can either make target lint-copyright-banner in Makefile.common.mk from https://github.com/istio/common-files ignore doc.go files, or have kubetype-gen output copyright header for doc.go files. Suggestions?

cc @jwendell @nrjpoddar

@skaslev you would need to manually run make update-common to grab the latest common files., until we automate this process. @geeknoid FYI.

@skaslev you would need to manually run make update-common to grab the latest common files., until we automate this process. @geeknoid FYI.

Sure, waiting on istio/common-files#149 to get merged first.

@skaslev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
8b0920022172098f99a0aef56b4711b7eb1afed9
link
/test gencheck_client-go

lint_client-go
abd50007e9b5997faa5bfbdc4ef353a445d5578d
link
/test lint_client-go

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

@skaslev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go
abd50007e9b5997faa5bfbdc4ef353a445d5578d
link
/test lint_client-go

gencheck_client-go
abd50007e9b5997faa5bfbdc4ef353a445d5578d
link
/test gencheck_client-go

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

The image in test-infra also need to be changed.

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
cf3adeadea42d0b069156eb97fe8deae12cdf8b6
link
/test gencheck_client-go

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

Since istio/common-files#149 is being pushed back, I changed header.go.txt so now all k8s client generated files have a copyright notice.
As @jasonwzm mentioned, for gencheck_client-go to pass the image in test-infra needs to be pushed and updated but I don't think I have permissions for this.

Since istio/common-files#149 is being pushed back, I updated header.go.txt so now all k8s client generated files have copyright banners.
As @jasonwzm mentioned, for gencheck_client-go to pass the image in test-infra needs to be pushed and updated with latest istio/tools but I don't think I have permissions for this.

@skaslev please submit a PR to test-infra to make the change. Anyone has permission to submit a PR.  If the PR is acceptable to the T&R reviewers, the PR will be merged pretty quickly.
I would help out here, but I am boarding a plane in a few hours and really busy for the next few days.
Modify this line: https://github.com/istio/test-infra/blob/master/prow/config/jobs/client-go.yaml#L4
with the latest image from:
https://console.cloud.google.com/gcr/images/istio-testing/GLOBAL/build-tools?gcrImageListsize=30
which happens to have the tag:
master-2019-11-15T13-58-33
After that change to istio/test-infra, in the TLD run make gen.  This will convert the client-go.yaml prow human-readable config into prow-less-human-readable parsed output.
You should end up with two files changed in your PR.  Push when ready.
I will review if you like.  Pass the information along 😀
Cheers
-steve

Thank for the help @sdake! Just sent istio/test-infra#2108 for review.

/retest

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
cf3adeadea42d0b069156eb97fe8deae12cdf8b6
link
/test gencheck_client-go

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

/hold cancel

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
820f8f934548d22c4cc4b92212b998fe5807123d
link
/test gencheck_client-go

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

@skaslev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go
d884deceff940d07c29b7dff945df5f75d8a62f2
link
/test gencheck_client-go

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

@geeknoid can you review this PR?
The only functional change is in this commit, everything else is auto generated.

@skaslev Thanks for making the change! A manual cherrypick is probably needed to 1.4.


Error with visual studio when I rebuild the project

I have many problems when I rebuild the plugins project
I did the Bleeding Edge Builds method to update pkhex and ALM (your branch), I put the pkhexcore.dll into the pkhexcore's nuget folder but when I rebuild I have the errors (image), before I never had a problem when I rebuild but now I have these problems

The branch that has the fixes for the latest pkhex commits has not been merged as there are still bugs in pkhex that need to be addressed.

I merged, it should work as expected now. Fair warning there are some sets that are still failing to legalize, these will be addressed in future  commits


formStore.element.submit() causes page reload at Qwik

I would like the form to submit programmatically equivalent to the manual submit.
But I couldn't find the recommended way to do that in document.
So I call the formStore.element.submit() but it reload the page, although the form doesn't reload when to submit manually.
How we can correctly submit the form programmatically with validation and without to reload?

Did you check if the reload can be prevented by an argument or event.preventDefault()?

Sorry for less information.
I made a runnable example here
https://stackblitz.com/edit/modular-forms-qwik-p4noyg?file=src%2Froutes%2Flogin%2Findex.tsx

Fill the form and click submit button then the dialog will appear.
Focus on the textarea and hit the Command+Enter or Ctrl+Enter then the page will reload.

I would like to simulate the first behaviour for the second case. Is there a way to do that?

I don't know. Probably I would not use the "submit" function of the <form /> element and call a function directly instead.

I see. If I call the function directly, how can I validate the form?

For example, this comment form here is possible to submit by hitting Command+Enter with validation.
I would like to have this function for the modular-forms.

You can call validate: https://modularforms.dev/qwik/api/validate

I see, I will use it. Thank you :)

@fabian-hiller I have tried to call the validate() but found an issue that the setFieldErrors() and setErrorResponse() are not exported so that we can't set the issues to the form.
How about to make the submitForm(formStore) in order to simulate the equivalent action of the manual form submit.

Oh, yes that's right. I will think about it. Until then, I recommend using validate with setResponse.

Thanks :)
I've made this for workaround.
https://gist.github.com/genki/632d57ffc53a136521f486867756469b

I tried to simplify the workaround a bit 😅, here is mine:
// Old-school solution:
// "cancelable" has to be true. It allows modular-forms to preventDefault that "submit".
form.element.dispatchEvent(new Event('submit', { cancelable: true, bubbles:true }));

// Or for modern browsers
form.element.requestSubmit();

@fabian-hiller Let me know if you are interested in a PR for a submit helper with one/both of these patterns.
Related discussion:
https://github.com/final-form/react-final-form/issues/878#issuecomment-745364350
requestSubmit Browsersupport: https://caniuse.com/mdn-api_htmlformelement_requestsubmit

form.element.requestSubmit();

Does this work without specifying a submitter as the first argument?

Ok, feel free to create a PR for a submit method. Contact me on GitHub or Discord if you have any questions. Currently I prefer to use .requestSubmit(), but I am open to other options.


Add set_children and swap_coldkey/hotkey to proxy settings.

Is your feature request related to a problem? Please describe.
The current proxy settings are not up to date, and new features are needed to enable safe proxy use for validators. I would like to see either the set_children extrinsic added to the Staking proxy or a new proxy type. Currently, the most secure option for a set_children proxy would be NonFungibile, however, this would still allow the swap_coldkey/hotkey functions, making my coldkey vulnerable to attacks by using this proxy type.
Describe the solution you'd like

Add new proxy type for set_children or add set_children to Staking proxy.
Add swap_coldkey/hotkey to NonFungibile as these functions could be used indirectly to get control over a cold/hot key

Describe alternatives you've considered
No response
Additional context
No response

In mainnet from #1016


Plex & Chromecast?

This is more of a question!
I was looking at #50 and just out of curiosity, not directly pointed at @wind-rider, what do Plex command have to do with Chomecast?
Wouldn't this pollute the project? I myself am a fan of having projects do one thing, and do it good... Having al kinds of features usually makes it hard to maintain, and e.g. a plex change would make other people upgrade the library without the need. Would the Plex stuff work when it was removed from the project into a different one?
Nothing against the functionality... keep up the good work!

The plex stuff could be extracted out of the SDK and it would still work fine.
But I want to have 3-5 different application inside of the SDK. That way you can see how custom commands can be implemented by SDK. Most of the applications almost never update their Chromecast commands so I would not see that as a big risk.
When we go further with the project I might create a structure where you extend the SDK with nuget packages. So you could have SharpCaster.Plex to support plex. But currently I would see that as too much work to justify the gains

Thanks for the quick answer!
P.S.
To make refactoring possible in the future, maybe it's at least an idea to make it in separate namespaces?

It is pretty easy to change namespaces with ReSharper so for now I will just keep them like they are.


KFImage.renderingMode(.original) not working as expected.

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[X] I have read the wiki page and cheat sheet, but there is no information I need.
[X] I have searched in existing issues, but did not find a same one.
[X] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
It's my understanding that adding .renderingMode(.original) to a KFImage should remove the highlight effect when used inside a NavigationLink. I still see the highlight flicker. Using .buttonStyle(.plain) on Text views works fine for removing the highlight.
Reproduce
Add a few HStack with a KFImage().renderingMode(.original) and a Text("Title").buttonStyle(.plain) inside a LazyVStack. The HStacks should be wrapped in NavigationLink.
The images will have highlight effects but the texts will not.
Other Comment
[Add anything else here]

@phedlund
Seems I cannot reproduce it. Am I doing the right thing? Or can you paste some code snippet which we can use to reproduce it?

And if I remembered correctly, the default behavior of an image/text/button in the navigation link was changing all the versions from iOS 13 to 15. The screenshot above was taken in iOS 15. So if you are trying another version of iOS, it is also grateful if you can mention it too.


Enable skipping of rvm tasks

When executing capistrano task which does not execute any ruby code at target host, checking of rvm and ruby versions is unnecessary. This commit enables skipping of rvm hooks by specifying :skip_rvm_for_tasks variable, containing an array of strings/regexes against which currently executing top-level capistrano task is matched.
Related to #45

Hello
I am running capistrano in an environment that has a heterogenous set of servers/roles, some of which have rvm, and others that do not. Currently this gem is testing rvm on all cap tasks, including non-deploy tasks that access non-rvm machines.
This is a serious bug in this gem
Can PR #68 (this) or #64 be merged and published?
Kind Regards


Uncaught error in async endpoint crashes express

Bug Report
Any uncaught errors in an async handler function of an endpoint will crash the underlying express instance (however not the process as it seems). I understand that this is something that is flawed in express and should be fixed upstream however the Payload docs themselves use an async endpoint in the docs without any mention of this issue so I feel like it should be handled until express 5 becomes stable.
Workaround in the form of a plugin is included below.
Steps to Reproduce

Add these two innocently looking endpoints to the Payload config:

endpoints: [
  {
    path: "/poof",
    method: "get",
    handler: (req, res, next) => {
      throw new Error("Poof")
    },
  },
  {
    path: "/crash",
    method: "get",
    handler: async (req, res, next) => {
      throw new Error("Crash")
    },
  },
],

Open http://localhost:3000/api/poof -> The result is a 500 error saying Poof
Open http://localhost:3000/api/crash -> There is no response and any further requests don't produce a response anymore either.

Other Details
After some digging, it turns out that Express 4 does not handle errors in async route handlers which will be fixed in Express 5 however that's been in beta for a year now (related issue with maintainer updates: https://github.com/expressjs/express/issues/4920).
My two ideas for resolution of this would be:

Use express 5 beta (although I understand that this is probably not really an option for a project of this scale)
When mounting the handlers, wrap them in an async handler that takes care of catching the error and handling it by passing it to next with something like this:

const safeAsyncHandler = (handler: PayloadHandler): PayloadHandler => async (req, res, next) => {
  try {
    await handler(req, res, next);
  } catch (e) {
    next(e);
  }
};

And then using that function in here (maybe somewhere else, I haven't looked far and wide but this seemed like a reasonable guess):
https://github.com/payloadcms/payload/blob/4d578f1bfd05efab5cc8db95895eabb776b2d9d1/src/express/mountEndpoints.ts#L4-L12
Payload version: 1.6.17
Workaround (Payload plugin)
Using the function from above, it's possible to write a plugin as workaround that handles async errors gracefully:
import {type Config, Endpoint, PayloadHandler} from "payload/config";

const safeAsyncHandler = (handler: PayloadHandler): PayloadHandler => async (req, res, next) => {
  try {
    await handler(req, res, next);
  } catch (e) {
    next(e);
  }
};

const patchEndpoint = (endpoint: Endpoint): Endpoint => {
  const handler = Array.isArray(endpoint.handler)
    ? endpoint.handler.map(safeAsyncHandler)
    : safeAsyncHandler(endpoint.handler);

  return {...endpoint, handler};
}

export const safeAsyncEndpoints = (config: Config) => {
  config.endpoints = config.endpoints?.map(patchEndpoint) ?? [];
  for (const c of (config.collections ?? [])) {
    c.endpoints = c.endpoints?.map(patchEndpoint) ?? [];
  }

  return config;
}

// Usage: plugins: [safeAsyncEndpoints] in payload.config.ts

Hey there! This is a great topic and the amount of information you've provided here is awesome.
I think personally the way to go would be your Option 2:

When mounting the handlers, wrap them in an async handler that takes care of catching the error and handling it

But I wonder, because having a try / catch in your own handler is typically always going to be required, is this duplicative? I think best practice would be to handle your own errors (thus controlling logs, resulting REST responses, etc.) It's an interesting thought.
Would certainly clean up custom handlers if you don't have to wrap every single one in a top-level try / catch.
I would love to get more feedback on this idea. I'll take it to my team and share it with Discord.
I will also convert this to a Discussion and tag it as a Feature Request.
Thanks for all of the notes. The plugin is great too!


Migrations Failed

Hi, I would to try this system, but when I try to migrate the database, I have an error like this, can you help me?

Can you post an image of your db settings?
The most likely reason is you accidentally added the wrong values.

this is my database.php file:

this is my .env file:

That seems ok. U assume you're using a local database?
Did you check your db name and are you Sure you haven't set a password?
Hope this helps!

I didn't set my local database password, so I set my password for the database setting to empty.
I was did composer update to this, everything is fine, here you can look at the result:

Is something wrong and what should I do?


Added Longest Palindromic Subsequence Under Dynamic Programming in C-Plus-Plus

…-Plus

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Find Longest Palindromic Subsequence
Can be solved in O(N^2) by using Bottom-Up Dynamic Programming
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

@HarshCasper I had made the required changes, can you please check it.

@atarax665 I had made the changes, can you please check it 😊😊

Hey @BullHoN Are you still working on this?


Preserving filters in the admin interface to match django.

Is your proposal related to a problem?

In the django admin interface, if you have a filter applied in the list view, when going to the detail view these filters will be preserved so that, upon returning to the list view, they're still in place. We've had some questions from partners about this feature in the wagtail interface, and maybe its time to explore whether this is possible?
Describe the solution you'd like

The cleanest solution would likely to be referencing the current django implementation:

https://github.com/django/django/blob/f3cd252cfc46c0c7d66e765818dd3dadf60d4d0e/django/contrib/admin/options.py#L1161-L1179 is used to get any filters
And then when generating urls, these preserved filters are then applied to them;

Describe alternatives you've considered

Internally, we have used a session to store the filters of a given admin model, and then reapply these when a model admin is resaved. Im not sure this is the best solution for wagtail, though.

Hi, can you elaborate a little bit more how did you did this? I am trying to keep the filters but I have no ideia where to start.


QouJS and Windows Phone

I see, this problem have a older issue, and it's not solved https://github.com/soyjavi/QuoJS/issues/97
I have a same problem in my project, and I want to know, when you want to implement Pointer Events API to QuoJS ? Or if you don't need it, let me a know.
Thank you in advance!!!

@psrafo i too am curious about this...Though I think this project maybe abandoned

@karneaud no, project not abandoned. And this is not QuoJS bug, later I will write here details of this problem.

@psrafo
Thanks for the response. I was beginning to wonder. Are you also responsible for https://github.com/tapquo/appnima.docs
I have a project that i am interesting in doing and Appnima sounds PERFECT. However, there are concerns ...i sent an email but no response.

Windows Phone WebView by default rendering by IE10 or EdgeHTML's older versions. And those browsers don't support Pointer Events API. For solve this problem, we need make our WP WebView with EdgeHTML's latest version.
No, I am not responsible for that, sorry :)

+1

Your project's problem too in it ? In WebView render core ?

@psrafo Yes. It doesn't work on Lumina 530 Windows Phone 8.1 IE10


Typo "Azure functions"→"Azure Functions"

https://docs.microsoft.com/en-us/azure/iot-edge/tutorial-deploy-function?view=iotedge-2018-06

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kgremban - Could you review the proposed update to your article?
For reference, the internal cloud style guide says this:
"Use Azure Functions as necessary to distinguish it as an Azure service. It's OK to use just Functions when the context makes it clear you're referring to the Azure service.
To refer to something built using Functions or in a generic reference, use lowercase function app."
Thanks.

#sign-off


Method onClick's problem

What if I want to execute a custom method when the player clicks on something in the book?
How can I achieve that?
I tried searching and stuff but I couldn't figure it out.
I want that when a player clicks it'll run a method within my plugin that does stuff, not necessarily a command.
By the way you got a great API and it helped me a ton, I'd love if I could have your discord or something to better communicate with you about the problem and some more stuff.
My discord is Kyren223#8030

Thanks! We have a discord but it's not used much.
As far as I know there is no way to execute a function directly from a book click (book clicks are client-side while functions are server-side), So the easiest thing to do is to create a command and map your clicks to commands,
This also comes in handy for power users of your plugin, it's faster to type a command than to skim and click a book

The problem is I am using a ton of very specific values per book,
The book that is being created is very specific, per item (that triggers the book to open) and even with the same item the values are constantly changing, when you click something on a book I want it to run functions with those specific values that change other values
The example is clicking a morph button, there are 3 of those in the book, each activates a different morph
And when a morph is activated it changes the values of the specific morph chosen to empty
I also don't want players to have access to those commands (that will activate the functions)

That's an intresting problem you have there,
The only way to communicate things back to the server is with a command, so you could make a command with all of the specific values, but it will also be writable by a user (altough not so easily).
If you want to be extra-sure that the users don't type the command themselves you can borrow some concepts from web-development (maybe you could pack the parameters in a json and encrypt it with a private key?).
If you want you can create a custom framework to use functions on clicks by registering functions on a hashmap, ex:
When the book is open, the book has 3 on-click functions, so we will register 0 -> function 1, 1 -> function 2 , 2 -> function 3.
When we click the first text the user will run a command like /customcommand 1 (for function 2).
The command will search the hashmap of that user for that number, and if it's present it will execute that function.
After the book is closed the hashmap should be cleaned (so that the user doesn't execute the function by themselves).
A lightweight version of this is specific for your use case is to have a custom command that sets the morph:
If you have morphs A, B, C, you could have a command /setmorph <A|B|C> that when invoked will change the values of the morph internally.
You should also make checks on the command though: (if user hasn't bought morph C /setmorph C should not work)
I'm sorry, it's quite hard but minecraft isn't designed for extensions, so we have to work around a lot of stuff.

Another path you may take is with inventory-based GUIs, servers can know when a user interacts with the inventory so we don't have to do commands to work around it,
I've been out of bukkit development since some years now, so there should be better solutions, but we've made a custom GUI library in our uppercore lib (it's not supported now, there should be other ones out there),

Chiming in here as I had to solve this for myself and under the context of unique books for players.  The end result is I generate tokens (a short string of numbers and letters) for each command and save each in a hashmap.  The plugin will then listen for its generic command and the token as a param.  The tokens are all generated on-the-fly each time a player opens a book and all are destroyed when they execute a command.  This makes it so a player is very unlikely to guess a token and there are no "shared" books - player will only ever see their book when they open it.  I am not claiming to be good at coding but I offer the work for you to rip or draw inspiration:

Yea I have now came up with kind of a solution that will be really hard for a player to guess
As commands don't have autocomplete by default they can't really know what parameters they should put
And for the commands I need the specific variables for I am using the book's key, the key id (generated uuid)
Which is used to link the item to a custom object which stores the data and now also to execute a command on that specific thing
I am facing now another problem thought, when reloading/restarting the server the book becomes invalid for some reason?
I am checking an interact event with a custom item (that has the id (key) stored in PDC) and using that PDC getting the custom object and with that object opening a new book each time, so it shouldn't get outdated by restarting as it's each time using an object to generate a specific book?

I don't know what is a PDC, but are you using a HashMap? You might want to persist that hashmap

PDC is persistent data container, it's a property of an ItemStack object
It lets you store values, in my case a string which is the ID that determines the linked object that stores the data
Then I am using the object to open the book with specific values
I have done another item that just uses the item and object and it works file, what's breaking is the book
The book's values doesn't update for some reason

I can't debug it without having the code, it doesn't seem to be related to be related to this library though

Yep, I have fixed it, a dumb problem, forgot to call the saving and loading method for the morph object that stores the data lol

Closing the issue to keep this repo clean, if you have any other questions ask them!


Hanging on startup

Spigot 1.12.2
LP 4.1.40
Only change to system was updating LP. Nothing in console/log after the following:
[11:08:53 INFO]: [LuckPerms] Enabling LuckPerms v4.1.40
[11:08:53 INFO]:                __        __   ___  __         __
[11:08:53 INFO]:     |    |  | /  ` |__/ |__) |__  |__)  |\/| /__`
[11:08:53 INFO]:     |___ \__/ \__, |  \ |    |___ |  \  |  | .__/
[11:08:53 INFO]:
[11:08:53 INFO]:   Loading version v4.1.40 on Bukkit - CraftBukkit
[11:08:53 INFO]:   Running on server version git-Spigot-c3093ef-7e94e65 (MC: 1.12.2) - 1.12.2-R0.1-SNAPSHOT
[11:08:53 INFO]:
[11:08:53 INFO]: [LP] Loading configuration...
[11:08:53 INFO]: [LP] Found lang.yml - loading messages...
[11:08:53 INFO]: [LP] Loading storage provider... [YAML]

Waited 5 minutes. Killed everything off and reverted to LP 4.1.37. All OK.

Yeah, seems to be an issue after the latest set of changes.
I can't work out why the hell it's happening though. Seems to be an issue with log4j.
https://paste.ubuntu.com/p/p78wWGP9dV/

Okay, fixed in 0f99a6c5caef480e6db58240c89a5296a8dc8a87

That was a pretty fast turnaround for "can't work out ..."


surface: don't destroy surface while compositor has locks

References: https://github.com/swaywm/wlroots/issues/2957

Breaking change: wlr_surface.resource may now be NULL if the client has destroyed it but the compositor still has locks.
TODO: check all surface->resource usage throughout wlroots, make sure we won't hit a NULL dereference.

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:
https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3024


build(cmake/modules): update driver version to 33c00f

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area build

What this PR does / why we need it:
This driver version, among other awesome things (like userspace instrumentation
support), includes a fix for building the eBPF driver on CentOS 8 machines too.
Which issue(s) this PR fixes:
Fixes #1129
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
update: driver version is now 33c00f0063d3bc3f35082f2b99d5418180841efe
fix: make the eBPF driver build work on CentOS 8

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.24.0

LGTM label has been added.  Git tree hash: e4418494495f4c9e49d4b4eee9e72379bf83a57f

LGTM label has been added.  Git tree hash: e4418494495f4c9e49d4b4eee9e72379bf83a57f

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
We should update the driver version again before merging (see https://github.com/draios/sysdig/pull/1635)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold cancel


Ship a 1.0 release

Part of https://github.com/rust-embedded/wg/issues/383
Blockers:

[x] remove const-fn now it's in Rust
[x] specify our MSRV
[x] Review and merge https://github.com/rust-embedded/bare-metal/pull/20 to fix https://github.com/rust-embedded/bare-metal/issues/7
[ ] Review against https://rust-lang.github.io/api-guidelines/ – happening in https://github.com/rust-embedded/bare-metal/issues/26
[ ] ~Mutex is not safe on multi-core systems - https://github.com/rust-embedded/bare-metal/issues/12~
[x] Should Peripheral be removed?
[x] What to do with the Nr trait? (it's gone)

Some simple improvements to be made before 1.0:

Remove the const-fn feature, and the build.rs logic related to const fn support
Review and merge https://github.com/rust-embedded/bare-metal/pull/20 to fix https://github.com/rust-embedded/bare-metal/issues/7

Other blockers:

Resolve https://github.com/rust-embedded/bare-metal/issues/12 via https://github.com/rust-embedded/wg/pull/388

Unresolved questions:

Where is bare_metal::Peripheral used? Does it pull its weight? Needs more documentation.
Same goes for bare_metal::Nr. Would it be better to move it outside this crate? Also: Why return u8?

As agreed in the 2020-03-10 meeting we're taking off the Mutex from the 1.0 blocker list. We're aiming to have the new Mutex based on mutex-trait implemented by the arch crates instead.

Maybe Nr can be used as a parameter to configure vendor specific interrupt controller. However in this way Nr::nr should not always return a u8 (there could be more than 256 interrupts). Maybe necessary to provide as Nr { type Count; fn nr(&self) -> Self::Count } (still working on it to adapt to Bumblebee's ECLIC controller)

IMO Nr logically belongs in architecture-specific core crates like cortex-m or cortex-m-rt.
We should check what uses it, and whether the users would see any benefit of having a traits that's shared between architectures. Maybe come up with potential use cases.

Ticked MSRV, as it's in the README.

Closed by #35.


chore: Release v.1.9.36

Release v.1.9.36

/build-deploy-preview

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


Dark Mode in GNOME?

Please add dark mode in GNOME.

Should be as easy as running the command
echo -e "[Settings]\ngtk-application-prefer-dark-theme=true" | tee ~/.config/gtk-{3,4}.0/settings.ini

and rebooting afterwards

Thanks, that worked.


Add more dartdoc to material.dart

The dartdoc will continue until morale improves.

@Hixie @HansMuller

LGTM


Constexpr resolving

#1
Research the ability to resolve values from a hana::tuple in constexpr context.

Currently, is seems achieving constexpr behavior would require too much hacking. A lot of the code makes use of std::forward, which is not marked as constexpr.
Even if there was a technique to branch code at compile-time based on whether a function is called in a constexpr context or not, I would consider this a too-much-hacking for what currently seems just a cool feature.


THRIFT-4932: Using a default string on a binary field results in invalid Java code.

When a binary field has a default value the value is treated as a String. However you cannot set a string to a bytebuffer so this fails to compile.

[x ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, add  [skip ci] at the end of your pull request to free up build resources.

AppVeyor build error is from setting up jdk8 environment:
Chocolatey v0.10.15
Nothing to change. Config already set.
Chocolatey v0.10.15
Installing the following packages:
jdk8
By installing you accept licenses for the packages.
jdk8 not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable
jdk8 package files install completed. Performing other installation steps.
The install of jdk8 was NOT successful.
jdk8 not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable
Chocolatey installed 0/1 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Travis failure was due to Haskell's package manager failing:
Making all in hs
make[3]: Entering directory '/thrift/src/lib/hs'
/usr/bin/cabal update
Config file path source is default config file.
Config file /root/.cabal/config not found.
Writing default configuration to /root/.cabal/config
Downloading the latest package list from hackage.haskell.org
<socket: 10>: hGetBufSome: resource vanished (Connection reset by peer)
Makefile:643: recipe for target 'all-local' failed
make[3]: *** [all-local] Error 1
make[3]: Leaving directory '/thrift/src/lib/hs'
Makefile:585: recipe for target 'all-recursive' failed

LGTM. Can you add a test as well so it doesn't regress in the future?

A separate Travis build failed due to Quicklisp:
debugger invoked on a QL-HTTP:UNEXPECTED-HTTP-STATUS in thread
#<THREAD "main thread" RUNNING {10005205B3}>:
  Unexpected HTTP status for #<URL "http://beta.quicklisp.org/archive/trivial-utf-8/2011-10-01/trivial-utf-8-20111001-darcs.tgz">: 500

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [CONTINUE] Ignore runtime option --eval #<(SIMPLE-ARRAY CHARACTER (204)) (quicklisp:bundle-systems '(#:puri #:usocket #:closer-mop #:trivial-utf-8 #:ieee-floats #:trivial-gray-streams #:alexandria #:bordeaux-threads #:cl-ppcre #:fiasco #:net.didierverna.clon) :to "external... {100173CF9F}>.
  1: [ABORT   ] Skip rest of --eval and --load options.
  2:            Skip to toplevel READ/EVAL/PRINT loop.
  3: [EXIT    ] Exit SBCL (calling #'EXIT, killing the process).

((LAMBDA (QL-HTTP::CONNECTION) :IN QL-HTTP:HTTP-FETCH) #<SB-SYS:FD-STREAM for "socket <IP_ADDRESS>:35108, peer: <IP_ADDRESS>:80" {1003FA6263}>)
   source: (ERROR 'UNEXPECTED-HTTP-STATUS :URL URL :STATUS-CODE (STATUS HEADER))
0]

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

LGTM. Can you add a test as well so it doesn't regress in the future?

@dcelasun I've pushed a fix in the same commit (the contrib guidelines say to squash everything).
I am having trouble running the tests locally but I think the updates to the thrift files should demonstrate the issue.

Yeah that looks fine. As you've noticed, we have some flaky Haskell and Lisp tests so if everything else passes I'll go ahead and merge this.

Had a dumb mistake where I had optional binary var default "" but it should have been optional binary binvar = "".
I've pushed a fix.

Tests are failing:
/thrift/src/lib/java/build/gen-java/thrift/test/StringAndBinary.java:102: error: cannot find symbol
    this.binval = ByteBuffer.wrap("".getBytes());
                  ^
  symbol:   variable ByteBuffer
  location: class StringAndBinary

That should have been fully qualified as java.nio.ByteBuffer.


HBASE-25657 Fix spotbugs warnings after upgrading spotbugs to 4.x

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 27s
the patch passed

+1 :green_heart:
compile
3m 51s
the patch passed

+1 :green_heart:
javac
3m 51s
the patch passed

-0 :warning:
checkstyle
0m 16s
hbase-rest: The patch generated 1 new + 28 unchanged - 0 fixed = 29 total (was 28)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
17m 23s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 14s
hbase-server generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)

+1 :green_heart:
spotbugs
0m 50s
hbase-rest generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)

_ Other Tests _

+1 :green_heart:
asflicense
0m 26s
The patch does not generate ASF License warnings.

50m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3041

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux fb7813bdda58 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d79019ba3b

Default Java
AdoptOpenJDK-1.8.0_282-b08

spotbugs
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-general-check/output/branch-spotbugs-hbase-rest-warnings.html

spotbugs
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-general-check/output/branch-spotbugs-hbase-server-warnings.html

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-rest.txt

Max. process+thread count
96 (vs. ulimit of 30000)

modules
C: hbase-server hbase-rest U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 16s
master passed

+1 :green_heart:
compile
1m 32s
master passed

+1 :green_heart:
shadedjars
6m 45s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  2s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  4s
the patch passed

+1 :green_heart:
compile
1m 32s
the patch passed

+1 :green_heart:
javac
1m 32s
the patch passed

+1 :green_heart:
shadedjars
6m 42s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  2s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
141m 45s
hbase-server in the patch passed.

+1 :green_heart:
unit
3m 56s
hbase-rest in the patch passed.

176m  5s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3041

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e8078772e7b6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d79019ba3b

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/testReport/

Max. process+thread count
4195 (vs. ulimit of 30000)

modules
C: hbase-server hbase-rest U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 28s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 32s
master passed

+1 :green_heart:
compile
1m 43s
master passed

+1 :green_heart:
shadedjars
8m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  6s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 25s
the patch passed

+1 :green_heart:
compile
1m 30s
the patch passed

+1 :green_heart:
javac
1m 30s
the patch passed

+1 :green_heart:
shadedjars
8m 53s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  1s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
162m 36s
hbase-server in the patch passed.

+1 :green_heart:
unit
3m 33s
hbase-rest in the patch passed.

202m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3041

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 610d166f708b 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d79019ba3b

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/testReport/

Max. process+thread count
3977 (vs. ulimit of 30000)

modules
C: hbase-server hbase-rest U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3041/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Enhancement#106

Adding changes to incorporate the review comments.
Updating XSS_Clean will change the behaviour of certain other functions like the URL's. I think this is a more systematic approach, let me know if you think otherwise.
Regards,
Shivam Rastogi

Hi Sam, This pull request is for saving the group permission with permission name instead of permission IDs. The previous old request can be closed.
Regards,
Shivam

Hi Shivam,
This looks good. I'll merge once i get into the Git Workflow, it can be a pain sometimes.

Just realized you pushed to the master branch, can change that to develop. thanks


Running pronto results with segmentation fault

Hi!
I came across an issue that makes the pronto run task fail miserably - with segmentation fault.
The error message can be seen in this gist: https://gist.github.com/simicic/60f84a1f6187c0ec6e1c
Checking it in deep I saw it comes from the rugged gem - I can report an issue there as well, but let me first ask, how come that a single rename of a class (class name and file name) breaks pronto?
The task is running after the test suit on Circle CI and I could detect what actually caused the failure - or at least I like to think so. After the commit that changes the file name - things started to go wrong.
I tried to clear the cache for running the task, update both pronto and rugged to the latest version but nothing seem to be helping...
(Running Ruby 2.0.0-p353, also tried with 2.1.0-p95, Pronto: 0.4.1, Rugged: 0.21.4, Github Formatter )
I would appreciate any help - maybe I am doing something wrong?

@simicic hi, thanks for reporting!
Could you create a test repository with which anyone else could reproduce it too?

@simicic maybe it's this issue: https://github.com/mmozuras/pronto/issues/103 ?

Assuming that it was #103 and closing.


requirements.txt

thanks for your work, can you add requirements.txt

thanks for your work, can you add requirements.txt

Requirements.txt is available now, or you can just follow diffusers controlnet and SPIGA , some simple packages just pip install.


Details sometimes get "lost"

Sometimes editing existing details for action-/info-items removes them instead of saving the changes.

Reprodicible in Safari browser:

Add: a Topic => with Info Item => with one Detail (enter some text)
then add second detail (enter some text) - keep the cursor flashing inside the detail in "edit mode"
Press "Cmd+Tab" to swap to a different application, or optionally: change to a different browser tab
Switch back to 4Minitz - the second details is visible on screen, but not in edit mode anymore (seems like it is stored)
now klick on the second details text => BAAMM it's gone!

We avoid this problem by introducing a confirmation dialog which will be showed before details will be deleted.
Take care: Adding details and leaving the input field without entering any text should go silently.

Today this happend again... to an end-user.
(No "security question poped up).
With current "v0.8.0" - we will try to get "steps to reproduce".

Not seen anymore... Closing


chore(main): release 0.32.0

:robot: I have created a release beep boop
0.32.0 (2024-11-23)
Features

Updates supported resource types (14) (#814) (db6aecc)
Updates supported resource types (2) (#805) (4a7a700)

This PR was generated with Release Please. See documentation.

Summary by CodeRabbit

New Features

Updated supported resource types, increasing the count from 1 to 14.
Additional update to supported resource types, now totaling 2.

Version Updates

Version incremented from 0.31.0 to 0.32.0 across multiple files.

Changelog

New version entry for 0.32.0 added to the changelog, detailing the recent updates.

:robot: Release is at https://github.com/udondan/cfn-teleport/releases/tag/v0.32.0 :sunflower:


Use Node12 and install yarn

I've alresady setup the build for the master branch. I went with including the node version in the image name.
We took the same approach with the jre version.
https://jenkins.swinfra.net/job/SEPG/job/UXAspects/view/Developer/job/UXAspects~buildenv-node12-image~initial~CI/

CI Build Link:
https://jenkins.swinfra.net/job/SEPG/job/UXAspects/job/UXAspects~buildenv-node12-image~initial~CI


Investigate if we can override or customise the generated API client READMEs

The generated README for each API client is very unclear and convoluted - it is aimed at an API client which is generated inside your own code - not as an externally consumed package.
Documentation for API Endpoints onwards is useful so this is not just a case of not generating the README and writing our own.
OpenAPI Generator has customization options through templates so this is a possible approach we should investigate further.

Sounds like we need custom templates with ascidocs.


Versioning database tables

If we want to support database upgrades ( and we really should ), we need to store an additional table in the database, that will contain a version number of each table, so we can lookup proper 'upgrade' scripts to run. Something like this:

ID
Table_Name
Version number

1
"accounts"
1

Then in database management tool we can query current database for the table list and lookup their versions, and run apropriate update scripts.

Perhaps we can utilize SQLite's PRAGMA user_version to store our own schema version information. There is also PRAGMA schema_version but it seems that SQLite auto increments that for every CREATE or DROP command, but it's not recommended for manual use as it may be changed by SQLite itself.
http://www.sqlite.org/pragma.html#pragma_user_version
Then, as you said, we could check this version info upon authserver startup to see if it matches the version in our default files under the default_dbs folder. If not, qFault() and recommended running dbtool to update the databases.
We can either store the update scripts in a default_dbs subfolder and call them in order until we reach version parity, or hard code those queries within the dbtool. This is similar to how Drupal runs dbase updates in PHP/MySQL.
Or worst case, the dbtool can be used to overwrite the current database completely with a new (updated) one.
I'll start to work on this issue. If you have any specific template or roadmap for the database schema in mind please let me know.

So definitely no schema_version

SQLite automatically increments the schema-version whenever the schema changes. As each SQL statement runs, the schema version is checked to ensure that the schema has not changed since the SQL statement was prepared. Subverting this mechanism by using "PRAGMA schema_version" may cause SQL statement to run using an obsolete schema, which can lead to incorrect answers and/or database corruption.

http://www.sqlite.org/pragma.html#pragma_schema_version

My thoughts:

Add a table_versions table to database
initialize it with all table names that are already in our database
set all versions to 0.

Using sqlite-only pragma's is not the way to go, since it will prevent using any other DB engine ( and since we use Qt, we get a few of those 'for free' )

As for storing updates, I'd prefer to store them in tool, since some updates could need to perform some logical operations, for example if we had a table with firstname_lastname column, and the update wanted to split that column into two firstname and lastname, then writing a standalone sql script that would do such a thing could become complicated. We could even go one step further towards the standard way of doing such things, and call them migrations :P

Ah yes that makes perfect sense.
And tool code makes sense too, I had overlooked the possibility of complex actions that would be overly difficult with queries.
I'll work towards this goal and submit schema changes for review using dbdesigner.net

Proposed schema changes for segs_game. GitHub wont let me attach the sqlite script file courtesy http://dbdesigner.net:

Looks good :+1:

This can be closed.

Ayup 👍


cyw43_arch_threadsafe_background bugs preventing deinit() / init()

I have code which is linked using the CMake cyw43_arch_threadsafe_background link_library.
The code can call cyw43_arch_init(), cyw43_arch_deinit() and later cyw43_arch_init() again.
The second call to cyw43_arch_init() hangs.
I have found at least two issues which are responsible for this:

In cyw43_arch_threadsafe_background.c routine cyw43_arch_init() includes the line:

    gpio_add_raw_irq_handler_with_order_priority(IO_IRQ_BANK0, gpio_irq_handler, CYW43_GPIO_IRQ_HANDLER_PRIORITY);
However cyw43_arch_deinit() is missing the corresponding:
    gpio_remove_raw_irq_handler (IO_IRQ_BANK0, gpio_irq_handler);

Secondly, in file cyw43_arch_poll.c, the call to lwip_init() is wrapped in code to ensure that it is called only on the first call to cyw43_arch_init():

    static bool done_lwip_init;
    if (!done_lwip_init) {
        lwip_init();
        done_lwip_init = true;
    }

However, the call to lwip_init() in cyw43_arch_threadsafe_background.c is missing a similar wrapper.

See also #968 and #969 ?

should be fixed by #1177; please give it a try if you can - i haven't yet tested this particular issue

#1177 merged into develop


ByteBufUtil.prettyHexDump may have a problem when calculating the initial capacity of StringBuilder

i use netty4.1.34
source code in netty4.1.34
private static String prettyHexDump(ByteBuf buffer, int offset, int length) {
            if (length == 0) {
              return StringUtil.EMPTY_STRING;
            } else {
                int rows = length / 16 + (length % 15 == 0? 0 : 1) + 4;
                StringBuilder buf = new StringBuilder(rows * 80);
                appendPrettyHexDump(buf, buffer, offset, length);
                return buf.toString();
            }
        }

in my opinion, rows = length / 16 + (length & 15 == 0? 0 : 1) + 4;
if  int rows = length / 16 + (length % 15 == 0? 0 : 1) + 4;
when length = 0 or length = 15, length % 15 always 0，this is wrong.

This should be an issue. The way to find the remainder is either length % 16 or length & 15. length & 15 performs better.

@qeesung yes，i think so.

PR welcome

Am 29.06.2019 um 08:15 schrieb xiaoheng1<EMAIL_ADDRESS>@qeesung yes，i think so.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@normanmaurer Ok, I will submit a PR.

@qeesung

length & 15 performs better.

I'm not that sure: last time I've checked length % 16, the JIT, given that 16 is constant, is able to perform the same optimization you'll perform by hand. Please check the assembly or perform a benchmark to be sure of it

@franz1981 I will check it more deeply. :-)

I am pretty sure that HashMap uses &. As the HashMap is very optimized I think & might be faster

I am pretty sure that HashMap uses &. As the HashMap is very optimized I think & might be faster

I found the code in the JDK7 source code
https://github.com/openjdk-mirror/jdk7u-jdk/blob/f4d80957e89a19a29bb9f9807d2a28351ed7f7df/src/share/classes/java/util/HashMap.java#L272-L277

Maby they also used this method because length may not be constant?

@SplotyCode @qeesung The JVM doesn't trust final (non static) fields unless for specific internal java util (invoke) classes ie length cannot be assumed (from the JIR) as constant on both ArrayDeque and IdentityHashMap or other impls need to use the "clever" mask bit-trick.
prettyHexDump is hardcoding as constant 16 so this kind of trick is not needed and should be avoided as "premature" optimization unless there are strong proof that the JVM has failed to optimize it.


Filter expressions and deep path traversal

Hey,
I just tried to upgrade to get the awesome support for deep paths on filter expressions and found a bug that was caught by one of our unit tests.
One level down after a filter expression still works as expected:
iex(10)> Warpath.query([%{"id" => 1, "foo" => %{"bar" => "baz"}}], "$[?(@.id==1)].foo")
{:ok, [%{"bar" => "baz"}]}
iex(11)> Warpath.query([%{"id" => 1, "foo" => %{"bar" => "baz"}}], "$[?(@.id==1)].foo", result_type: :path)
{:ok, ["$[0]['foo']"]}

But trying to traverse one level deeper gives an unexpected result:
iex(12)> Warpath.query([%{"id" => 1, "foo" => %{"bar" => "baz"}}], "$[?(@.id==1)].foo.bar")
{:ok, nil}
iex(13)> Warpath.query([%{"id" => 1, "foo" => %{"bar" => "baz"}}], "$[?(@.id==1)].foo.bar", result_type: :path)
{:ok, "['bar']"}

Hi @jonasschmidt , thanks for report and the use of the master branch version!
I have been fixed the issue, please given another try if you have any issue feel free to open up a new one.

Yes, that fixed the issue. Thank you for the quick response! 🚀
I have another smaller (maybe related?) issue that I will open separately. Maybe it's just something that needs clarification: https://github.com/Cleidiano/warpath/issues/84


EBU subtitles regression in 4.1.3 - subtitles lost: not added to region

Have you read the FAQ and checked for duplicate open issues?
Yes
Note the coincidental relationship with #4631 on the same stream and same release.
What link can we use to reproduce this?
https://v4-1-3-dot-shaka-player-demo.appspot.com/demo/#audiolang=en;textlang=en;uilang=en;asset=https://otvplayer.nagra.com/demo/content/bbc/bbc_ebu-tt-d_subtitle_fragmented/dash/ondemand/elephants_dream/1/client_manifest-all-minus-1080p.mpd;panel=CUSTOM CONTENT;build=uncompiled
What version of Shaka Player are you using?
v4.1.3-uncompiled
What browser and OS are you using?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
What did you do?

Play this stream with captions enabled
Observe the captions at 2:52 to 2:57

What did you expect to happen?
See the captions as follows:
You have to learn to listen.
then
This is not some game.
i.e. from the following TTML snippet.
...

                You have to learn to listen.

                This is not some game.

...

What actually happened?

Neither of these were shown, nothing was in this time range.
Note: this can only be reprod by playing the stream through this range uninterruted, if you pause in that region, they can be shown without issue.
Note: I believe this was introduced in #4412

I see you're using v4.1.3.  Please try the latest release in that branch (v4.1.7) and overall (v4.2.4 as of now, though v4.3.0 will be out later today).
(We ask similar questions in the issue template, but you seem to have deleted them.  Filling out the issue template saves a lot of time for everyone involved, especially across many timezones.)
If you're still seeing the issue in those more recent versions, that would be useful to know.  Thanks!

I see you're using v4.1.3. Please try the latest release in that branch (v4.1.7) and overall (v4.2.4 as of now, though v4.3.0 will be out later today).
(We ask similar questions in the issue template, but you seem to have deleted them. Filling out the issue template saves a lot of time for everyone involved, especially across many timezones.)
If you're still seeing the issue in those more recent versions, that would be useful to know. Thanks!

Sorry, I cloned the similar issue rather than started afresh from the template and lost some stuff.  Refreshed now with the template.
Confirmed still present in 4.3.0.

After some time of debugging this, some clues emerging.
Just before the cue (say N) that is not shown, the region element is uprooted when we remove the cue N-1.
Then it is not reinstated when N or N+1 is handled.
Only reinstated with N+2.


add readme badges

I want to add readme badges.Please assign this issue to me under GSSOC

Sure, assigning this task to you.


End to end integration tests for Dashboard URL output.

We need a series of end to end integration tests which make sure we don't regress logic around emitting Now listening on text to make sure we don't accidentally break dotnet watch scenarios that depend on that text.

@eerhardt @radical  do we have a good way to test this with the new testing infrastructure?

Ping @radical

We have the infrastructure to test this now. cc @adamint and @JamesNK who have been adding dashboard tests.

The test here would be verifying apphost logs output

There is an existing test which happens to test the string - https://github.com/radical/aspire/blob/f3569ce824ca777c460a028116800086328229e7/tests/Aspire.Dashboard.Tests/Integration/StartupTests.cs#L373-L453
.. but a dedicated test for this would be good.

Hmm, this might be good enough.


Use interfaces on publicly visible classes and structs to support dependency injection

For instance SgfInitializationContext and SgfSourceProductionContext just to name the two I'm working around right now. Likely want to expose the original IncrementalGeneratorInitializationContext and ILogger objects as well, maybe with note that these are convenience accessors for advanced scenarios. That would allow either using the new interfaces, or just rolling your own if more convient.

Expose interfaces on publicly visible classes and structs to support dependency injection. #1


multiple choice cleanups for frontend

Some stuff that is coming up as I work on the frontend (there may be more, but want to get this in)

fix a couple return type annotations
simplify proposal.msgs type
allow multiple choice options to have their own titles
increase the max number of options

Codecov Report
Base: 93.95% // Head: 93.98% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (3711a13) compared to base (48d9629).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #532      +/-   ##
==========================================
+ Coverage   93.95%   93.98%   +0.02%
==========================================
  Files          37       37
  Lines        3589     3589
==========================================
+ Hits         3372     3373       +1
+ Misses        217      216       -1

Impacted Files
Coverage Δ

...cts/proposal/cwd-proposal-multiple/src/proposal.rs
90.24% <ø> (ø)

...cts/proposal/cwd-proposal-multiple/src/contract.rs
92.03% <100.00%> (+0.18%)
:arrow_up:

packages/cwd-voting/src/multiple_choice.rs
94.87% <100.00%> (+0.42%)
:arrow_up:

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

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


Laravel Mix in XAMPP not working!

Laravel Mix Version: (6.0.6)
Node Version (node -v14.15.0):
NPM Version (npm -v6.14.8):
OS: Ubuntu 18.04

Description:
Hi,
Currently i am working on a project with Laravel Mix with Vue
i have installed and configured all the dependencies. when i serve through artisan (php artisan serve) the application works fine.
But i am facing issue when i place my project to localhost inside htdocs directory (http://localhost/myProject) does not worked!
in console:
Uncaught SyntaxError: expected expression, got '<' app.js:1
My webpack.mix.js
mix.js('resources/js/app.js', 'public/js')      .vue()      .sass('resources/sass/app.scss', 'public/css');
Please help me to resolve this issue.

check your path to your app.js


sentence-generator: introduce a new, faster sampling strategy

Our previous sampling strategy consisted of enumerating, for each template, all possible expansions of that template, and then
sampling a portion of those expansions to call the semantic function with. This is not efficient. Instead, we can sample directly one candidate expansion without enumerating, if we are careful with setting up the probabilities correctly.
It's worth noting that previously we were doing sampling without replacement. The new algorithm samples with replacement for speed. Practically, I don't think it makes a difference in terms of the quality of the generated datasets.
In dialogues, this gave me a 40% increase in generation throughput (turns/sec) with a small pruning size. I expect with a large pruning size it will yield even higher gains.

Actually, this fails full size generation:
AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value:
assert_1.default(from.length > 0)
at expandRuleSample (/opt/genie-toolkit/dist/lib/sentence-generator/generator.js:1237:33)
at expandRule (/opt/genie-toolkit/dist/lib/sentence-generator/generator.js:1319:42)
at SentenceGenerator.generate (/opt/genie-toolkit/dist/lib/sentence-generator/generator.js:800:25)
at MinibatchDialogueGenerator._generateUser (/opt/genie-toolkit/dist/lib/sentence-generator/batch.js:219:29)
at MinibatchDialogueGenerator.nextTurn (/opt/genie-toolkit/dist/lib/sentence-generator/batch.js:284:14)
at DialogueGenerator._generateMinibatch (/opt/genie-toolkit/dist/lib/sentence-generator/batch.js:391:33)

I need to investigate why.

Another bit of measurement: on the MultiWOZ experiment, this cuts the time to generate the dataset from 10h to 5h. So it's a big win!


Fixed logic of SEMI in clustering

Directly based on the https://github.com/cqfn/veniq/pull/18

@Vitaly-Protasov which part of the SEMI pipeline does this PR implement?


What is the difference between pyvesync vs pyvesync_v2 ?

bump.

This is basically the same as pyvesync but I recently updated after mark stopped development. The only differences at the moment are the air purifier display toggle, CI and some formatting. Pyvesync will have the latest stable version.


Terminal plugin

a terminal for hackers, using a plugin
make some fancy additions like cd displays the sections on the page, etc

seems unnecessary now


More vistir dropping

Moving to Python version 3.7+ allows us to remove more usage of vistir.
Eventually we can completely drop it, but I prefer doing this gradually instead of a giant PR.
Also, I am not done yet. Will add 2-3 more commits, and we'll continue chopping this vistir with b[i,y]te size steps.
Pun intended :-)

@matteius when (and if) merging, please do not squash the commits. Doing so will lead to losing of alot of information found in the commit messages.


HDFS-16957. RBF: Exit status of dfsrouteradmin -rm should be non-zero for unsuccessful attempt

DFS router admin returns non-zero status code for unsuccessful attempt to add or update mount point. However, same is not the case with removal of mount point.
For instance,
bin/hdfs dfsrouteradmin -add /data4 ns1 /data4
..
..

Cannot add destination at ns1 /data4

echo $?
255

bin/hdfs dfsrouteradmin -rm /data4
..
..
Cannot remove mount point /data4

echo $?
0

Removal of mount point should stay consistent with other options and return non-zero (unsuccessful) status code.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 36s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m 12s

trunk passed

+1 :green_heart:
compile
0m 45s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 40s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 35s

trunk passed

+1 :green_heart:
mvnsite
0m 45s

trunk passed

+1 :green_heart:
javadoc
0m 52s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 59s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 35s

trunk passed

+1 :green_heart:
shadedclient
20m 34s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 34s

the patch passed

+1 :green_heart:
compile
0m 36s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 36s

the patch passed

+1 :green_heart:
compile
0m 32s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 32s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 18s

the patch passed

+1 :green_heart:
mvnsite
0m 35s

the patch passed

+1 :green_heart:
javadoc
0m 33s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 52s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 21s

the patch passed

+1 :green_heart:
shadedclient
20m 30s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
20m 44s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt
hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

117m 40s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterRPCMultipleDestinationMountTableResolver

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5487

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux ac3115c14446 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0bc696c88a85edf36827a25f44c5ff0ce0ef2ecf

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/1/testReport/

Max. process+thread count
2544 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-rbf U: hadoop-hdfs-project/hadoop-hdfs-rbf

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

All tests in TestRouterRPCMultipleDestinationMountTableResolver are passing locally

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m  5s

trunk passed

+1 :green_heart:
compile
0m 48s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 36s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 31s

trunk passed

+1 :green_heart:
mvnsite
0m 42s

trunk passed

+1 :green_heart:
javadoc
0m 47s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 56s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 33s

trunk passed

+1 :green_heart:
shadedclient
24m 21s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 32s

the patch passed

+1 :green_heart:
compile
0m 36s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 36s

the patch passed

+1 :green_heart:
compile
0m 30s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 30s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 16s

the patch passed

+1 :green_heart:
mvnsite
0m 33s

the patch passed

+1 :green_heart:
javadoc
0m 32s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 21s

the patch passed

+1 :green_heart:
shadedclient
24m  9s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 46s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

126m 59s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5487

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux c30fbbcd3c76 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / b5b3c234b865bb494bd2372c094ab59f7e244dbf

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/2/testReport/

Max. process+thread count
2070 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-rbf U: hadoop-hdfs-project/hadoop-hdfs-rbf

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5487/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

I think this was raised somewhere in past as well but not done considering incompatible

I thought so too, that's why wanted to check on the Jira :)

@ayushtkn @goiri
I just looked at the function where the changes are made in the PR and realized that other options -setQuota, -setStorageTypeQuota, -clrQuota and -clrStorageTypeQuota also have the same problem and they are also not consistent. I have not tested these options yet but with a quick glance, I feel they also are likely having the same issue.
Would you recommend fixing them all once and for all on trunk (3.4.0) only with this PR?

yep, that is something known to me, it is there with almost all commands, and we had a discussion around it long back and it was kind of: 'Changing exit code is incompatible', remember discussing internally long back as well and the conclusion was people who have internal test scripts and code they heavily rely on these exit code(we were one of those then), so, it got dropped.

Would you recommend fixing them all once and for all on trunk (3.4.0)

I won't say yes, have always been part of the gang or say I come from that era which was strong believer and follower of compat guidelines. Not sure how people are dealing nowadays though :(
see this ex., just a minor CLI change which in all ways is good very useful got reverted just because of compat from all branches. that is some fun to follow
https://issues.apache.org/jira/browse/HDFS-13732

Ah yes, not having compatibility is a pain for sure :(
The only problem though is, on one hand we have compatibility to maintain, and on the other, inconsistent behavior among different options of dfsrouteradmin.
Any decision is fine though :)

Do we have this exit code output being 0 in 3.3.X?
If it hasn't been released it is fine but I suspect it has.

It is there since very starting, even in branch-2.9
https://github.com/apache/hadoop/blob/branch-2.9/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java#L182-L185
There ain't any release line which has not released this behaviour :)

Do we have this exit code output being 0 in 3.3.X?
If it hasn't been released it is fine but I suspect it has.

It is there since very starting, even in branch-2.9 https://github.com/apache/hadoop/blob/branch-2.9/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java#L182-L185
There ain't any release line which has not released this behaviour :)

On the above reference of branch-2.9, though even -add and -update are of same behavior. That means all commands are exiting with status code 0 even for failures, so while not intended behavior, at least they all are consistent in that sense.
Whereas right now on 3.3, -add and -update have different behavior (which is correct with outside world) than -rm etc, which is inconsistent already. It's like some of router admin CLI options have correct behavior from outside world viewpoint but others don't.

Looks like this is the commit that has changed behavior for -add and -update https://github.com/apache/hadoop/commit/9315db5f5da09c2ef86be168465c16932afa2d85
(even though HDFS-13815 title doesn't reflect that)

Hmm, that changed something wrapped with the bug. There failure was claimed as successful or something like that….
But this behaviour including that also is there since branch-3.1, should be in almost all 3.2.x and 3.3.x
no strong opinions here, slightly more inclined to stay with compat, for one reason 3.3.x has this behaviour


While auto importing widgets the suggestion getting illegal characters error every time

I recently shifted from windows to fedora 36 and I installed flutter and started a new project.
Flutter provides an auto-importing tool for widgets when I try to auto-import a widget with closed brackets ().
It is showing an illegal character error. example:-
If I try to import a text widget  it gives me Text(...) which decreases productivity and please help me to solve this error

Hi @iamkranthi
It looks like you are working with VSCode. Following the indication of VS Code Flutter Extension, the correct way to address the problem is to open an issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you!