Major release notifications

Question
Would it be possible to setup major release notifications so that people integrating w/ your releases downstream know when to add them?
Context
I am one of the maintainers of the official Clojure Docker image and would love to be able to get a notification when there is a new major release available of Eclipse Temurin so I can add it to our build matrix. In this case a notification of the new official eclipse-temurin Docker image would be ideal. A GitHub release would be ideal as the notification source as I can easily subscribe to those and that's how I monitor for new releases of some of the build tools we package as well.
I discussed this in Slack w/ @sxa.

@cap10morgan How would you expect to get that notification?

@cap10morgan How would you expect to get that notification?

A GitHub release notification is sent to any subscribers when you create a “Release” from a tag. So you’d need a GitHub repo in which you could create tags and releases corresponding to the JDK release versions. But it’s fine if a different approach works better for you all.

We could tag our containers repo I suppose, challenge is that our images are actually built and maintained by Docker Inc at DockerHub.  Have you looked at seeing if you can get notified via them?

That may be tricky since we don't have our own repository for temurin in dockerhub - it's a generic one (Our files are at e.g. https://github.com/docker-library/official-images/blob/master/library/eclipse-temurin)

That may be tricky since we don't have our own repository for temurin in dockerhub - it's a generic one (Our files are at e.g. https://github.com/docker-library/official-images/blob/master/library/eclipse-temurin)

Yeah, same for us in the official Clojure image. It's not critical that the notification be immediately after the image becomes available on Docker Hub. If it's more of a "heads up, this is imminent" that's OK too.

@cap10morgan our release schedule is quite predictable - did you notice the "Release Roadmap" on our website? HTH


feat: support optimism

@hai-rise  This is a proof-of-concept implementation. I am gonna break it down to smaller PRs:
Targetting main branch:

Add definition enum ChainSpec
Rewrite calculate_receipt_root by manually encoding each field
Refactor get_tx_envs into get_tx_env
Extract to a separated function fn get_rewards(...): HashMap<Address, U256> in vm.rs

Targetting optimism branch:

Add arm to enum: ChainSpec::Optimism { chain_id: ChainId }
Customize fn get_rewards(...)


fhir check likely won't ever fail without specifying the --fail flag

In https://github.com/FirelyTeam/firely-terminal-pipeline/blob/main/action.yml#L241, fhir check is ran without specifying the --fail flag. This means that even if validation fails, the action still completes successfully, which is most likely not the expected behavior.
See:
$ fhir validate patient-bundle.json
warning: Evaluation of FhirPath for constraint 'GeburtsdatumRundenAufQuartal' failed: Invocation of function 'matches' failed: Cannot cast from 'Date' to '.NET type String'
error: Instance failed constraint GeburtsdatumRundenAufQuartal "Geburtsdatum darf nicht tagesgenau angegben werden. Das Geburtsdatum MUSS quartalsweise auf-/abgerundet werden."
Result: INVALID

$ echo $?
0

(I can't run check since its a licensed feature, but I believe the commands behave identically).

Thanks for the report. This is now fixed in the next version of the validator pipeline v0.4.7 (once the final version is released). The pipeline will continue to run if VALIDATION_CONFORMANCE_DOTNET is set in the EXPECTED_FAILS variable.


Unskip backend tests

Unskip/Fix backend tests where possible.
Fixes are near the top of the files changed, then there's a bunch of fixture updates, then test updates are at the bottom.
Pull requests into Bonnie require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[x] This pull request describes why these changes were made.
[x] This PR is into the correct branch.
[x] JIRA ticket for this PR: https://jira.mitre.org/browse/BONNIE-2053
[x] JIRA ticket links to this PR
[x] Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] If UI changes have been made, google WAVE plug-in has been executed to ensure no 508 issues were introduced. N/A
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Test fixtures updated and documented as necessary ( see internal wiki ) N/A
[x] Code coverage has not gone down and all code touched or added is covered.
[x] Automated regression test(s) pass N/A

Reviewer 1:
Name: @jbradl11

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code

Reviewer 2:
Name: @hackrm

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Needs a rebasing but otherwise looks good.


create INDArray from String

For debug purposes, it can be useful to have the possibility to be able to create INDArray from their output string.

@rubenfiszel ya, closing this


Use tippy to position typeaheads

No visual changes, just used tippy to position elements.
Next steps here would to use start using the new styles while renaming the clases for typeahead and dropdown-menu to ensure we are not using any bootstrap styles.

Fun whac-a-mole session with puppeteer today.

It looks broken in light theme when I tested (dark theme doesn't have this issue).

Which typeaheads are affected by this PR, so that I know what to test?

Sorry, forgot to check for light theme.
Here is a list of all typeaheads:

Compose box textarea with various triggers - @,#, /, ....
Compose box topic / DM
Message edit box
Search
Playground settings > Language - http://localhost:9991/#organization/playground-settings
Profile > Mentor - http://localhost:9991/#settings/profile
Stream settings > Add subscribers - http://localhost:9991/#channels/21/Denmark/subscribers

Works for me in manual testing!

This is not a regression, but I did notice while playing with this that we don't place the typeahead anywhere near the actual mention in the compose box. How hard would it be to fix that?

There's a regression where the mentions typeahead get hidden when the compose box is full-screen. On main, it gets scooted down.

we don't place the typeahead anywhere near the actual mention in the compose box. How hard would it be to fix that?

I think we tried that before and it had some significant downsides, some of which might now be easier to avoid on Tippy; let's keep that separate but could be worth prototyping again.

Test-deployed on chat.zulip.org despite the one bug since I think this will need a lot of bake time to feel confident in.

There's a regression where the mentions typeahead get hidden when the compose box is full-screen. On main, it gets scooted down.

Fixed (not sure we can avoid it overlapping text here without placing at around the text that triggered it, which has no obvious solution)

I think we tried that before and it had some significant downsides, some of which might now be easier to avoid on Tippy; let's keep that separate but could be worth prototyping again.

What would that look like? I can image we can set offset property of tippy based on cursor position but yeah, likely needs to be a separate PR after this is merged.

Fixed (not sure we can avoid it overlapping text here without placing at around the text that triggered it, which has no obvious solution)

I think it'd be better for it to be at the bottom of the screen (or something like that) than overlapping the text you've typed, since it's pretty hard to work on a topic mention in particular if you can't see what you've got. But probably ideal would be if we can just have it pop below the cursor rather than above if there isn't space above but there is below?
I should also mention that I have managed to once, on chat.zulip.org, have the typeahead popover get leaked when jumping around the compose box ... but I don't know how I did it or whether it's new. Seems pretty low rate but something to be on the look out for.

I think it'd be better for it to be at the bottom of the screen

Okay, pushed a version that does just that. I know it would be better to show the typeahead above the control buttons but I think better investment would just be to use a library like textarea-caret to get the current caret position and show typeahead based on that.

I filed #29974 for the placement follow-up.

First commit is same as #29983

So I notice one bug with this, which is that if you are looking at the combined feed (or maybe other views) and open the @-mention typeahead and then scroll up and down a bunch especially near the bottom, the typeahead widget has a bunch of flickering/bouncing that looks ugly; compare that with main where it just stays cleanly in place.

So I notice one bug with this, which is that if you are looking at the combined feed (or maybe other views) and open the @-mention typeahead and then scroll up and down a bunch especially near the bottom, the typeahead widget has a bunch of flickering/bouncing that looks ugly; compare that with main where it just stays cleanly in place.

Pushed a commit to fix that.

Yeah it looks like the only one we had not using fixed is search_typeahead = new Typeahead, and that one used parentElement instead. I'll test that one a bit just to be sure, but otherwise this looks good.

OK, looks great, merged, thanks @amanagr!


How do install Inertia in laravel app starting fresh project,

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

i am using breeze install for inertia setup in laravel (SSR),  what version from there i need to install inertia so that it runs only from php artisan serve,
i am able to render  the vue templates from laravel controller but the issue is i have to  maintain two terminal running constantly for
php artisan serve
and
npm run dev
but in the demo project thats not the case, only php artisan serve sufffice the render


twister: Include coverage support for device testing

Include coverage support for device testing. It works by switching
capture_coverage to True (as it would meet coverage data start
pattern). Then we continue tot read coverage data until we get
coverage data end pattern. Otherwise, after receiving test result
pattern, we close serial console and do not get time to find start
coverage data pattern.

@finikorg  please rebase

tested on ehl board, it could get coverage data, but for some testcases, the default timeout is not enough to get test results and all coverage data, it will timeout like below, the testcase is passed, but have not got all coverage data.
DEBUG   - Timed out while monitoring serial output on ehl_crb
DEBUG   - run status: ehl_crb/tests/kernel/threads/thread_apis/kernel.threads.apis passed
INFO    - 1/1 ehl_crb                   tests/kernel/threads/thread_apis/kernel.threads.apis PASSED (device 77.924s)

tested on ehl board, it could get coverage data, but for some testcases, the default timeout is not enough to get test results and all coverage data, it will timeout like below, the testcase is passed, but have not got all coverage data.
DEBUG - Timed out while monitoring serial output on ehl_crb
DEBUG - run status: ehl_crb/tests/kernel/threads/thread_apis/kernel.threads.apis passed
INFO - 1/1 ehl_crb tests/kernel/threads/thread_apis/kernel.threads.apis PASSED (device 77.924s)

Yes, I found that for some tests we need to increase timeout.

Yes, I found that for some tests we need to increase timeout.

How much increase is needed, and for which tests? It sounds like this patch introduces this timeout issue. In the past it was important to address root cause of any significant increase to timeout needed for tests to complete, rather than make a significant change for only 1 HW platform that will effect all platforms and arches. If it is minimal increase in timeout, is that a separate PR to address this?

How much increase is needed, and for which tests? It sounds like this patch introduces this timeout issue. In the past it was important to address root cause of any significant increase to timeout needed for tests to complete, rather than make a significant change for only 1 HW platform that will effect all platforms and arches. If it is minimal increase in timeout, is that a separate PR to address this?

where is the timeout being changed?

Yes, I found that for some tests we need to increase timeout.

How much increase is needed, and for which tests? It sounds like this patch introduces this timeout issue. In the past it was important to address root cause of any significant increase to timeout needed for tests to complete, rather than make a significant change for only 1 HW platform that will effect all platforms and arches. If it is minimal increase in timeout, is that a separate PR to address this?

What I mean is that:

By default nothing actually changes.
If you enable --coverage then after test execution we get coverage data on console, in my case pseudo-console over network
I tested for ibecc tag and I am able to get coverage data for my tests but I believe if there are lots of coverage data it may take some time to receive them over console / network so we need to check timeout


Server not permision

This should be only on Xiaomi and Oppo devices.

for miui: "USB Debugging (Security settings)" in "developer option".
for oppo: "disable permission monitoring" in "developer option".


[WFLY-13471]:Warn the ws deployment packaged with JAXP implementation

https://issues.redhat.com/browse/WFLY-13471

@pferraro OK now?
@jimma Can you squash the commits?

@bstansberry @pferraro  Done.
Fail the ws deployment change breaks XercesUsageTestCase and looks previous change isn't backward compatible. I changed this to print warning message instead of failing the deployment.


Accessibility keyboard shortcuts not working on mat-tab-nav-bar

Bug, feature request, or proposal:
Question
What is the expected behavior?
Right arrow, left arrow, Enter and Space keyboard shortcuts should work on mat-tab-nav-bar
What is the current behavior?
None of the above seems to work in my implementation. (The Tab key does navigate between the tabs.)
I do not see an example for mat-tab-nav-bar on the Examples page on the material website here. It would help to have such an example.
What are the steps to reproduce?
html:

    My Tab 1
    My Tab 2
    My Tab 3

<router-outlet class="content-background"></router-outlet>

and the TS function:
 select(tab: number) {
        if (tab === this.selectedTab){
           return;
        }
        this.selectedTab = tab;
        this.activeRoute=this.routes[tab];
        this.router.navigate([this.routes[tab]]);
    }

What is the use-case or motivation for changing an existing behavior?
On the site here there is a list of keyboard shortcuts that should work for Accessibility.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Tested on Chrome Version 68.0.3440.106 (Official Build) (64-bit).
Here are the app dependencies:
  "dependencies": {
    "@angular/animations": "^6.1.0",
    "@angular/cdk": "~6.2.1",
    "@angular/common": "^6.1.0",
    "@angular/compiler": "^6.1.0",
    "@angular/core": "^6.1.0",
    "@angular/flex-layout": "6.0.0-beta.15",
    "@angular/forms": "^6.1.0",
    "@angular/http": "^6.1.0",
    "@angular/material": "~6.2.1",
    "@angular/platform-browser": "^6.1.0",
    "@angular/platform-browser-dynamic": "^6.1.0",
    "@angular/router": "^6.1.0",
    "@types/crypto-js": "^3.1.40",
    "angular-logger": "^0.1.8",
    "basscss": "^8.0.2",
    "core-js": "^2.5.4",
    "crypto-js": "^3.1.9-1",
    "ngx-img-cropper": "^0.10.4",
    "rxjs": "6.2.0",
    "rxjs-compat": "^6.2.0",
    "zone.js": "~0.8.26"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~0.7.0",
    "@angular-devkit/core": "~0.7.0",
    "@angular/cli": "~6.1.2",
    "@angular/compiler-cli": "^6.1.0",
    "@types/jasmine": "^2.8.7",
    "@types/node": "^6.0.112",
    "autoprefixer": "^7.2.3",
    "circular-dependency-plugin": "^5.0.0",
    "codelyzer": "~4.3.0",
    "copy-webpack-plugin": "~4.5.0",
    "file-loader": "^1.1.5",
    "html-webpack-plugin": "^3.2.0",
    "istanbul-instrumenter-loader": "^3.0.0",
    "jasmine-core": "~2.5.2",
    "jasmine-spec-reporter": "~3.2.0",
    "karma": "~1.4.1",
    "karma-chrome-launcher": "~2.0.0",
    "karma-coverage-istanbul-reporter": "^0.2.0",
    "karma-jasmine": "~1.1.0",
    "karma-jasmine-html-reporter": "^0.2.2",
    "less-loader": "^4.0.5",
    "postcss-import": "^11.0.0",
    "postcss-loader": "^2.0.10",
    "postcss-url": "^7.1.2",
    "protractor": "~5.1.0",
    "raw-loader": "^0.5.1",
    "sass-loader": "^6.0.6",
    "style-loader": "^0.19.1",
    "stylus-loader": "^3.0.1",
    "ts-node": "~2.0.0",
    "tslint": "~4.5.0",
    "typescript": "~2.9.2",
    "uglifyjs-webpack-plugin": "^1.1.8",
    "url-loader": "^0.6.2",
    "webpack": "^4.11.0",
    "webpack-cli": "^3.0.2",
    "webpack-dev-server": "^3.1.0"
  }

This is working as intended. As you can see in the template, the nav tabs present themselves as a  element with anchors instead of using the tab-related ARIA roles. This keeps the experience accessible since it aligns with expectations that any interactions that update the URL are done via  element.


runtime for python

Is there a plan for building runtime for python. there is already a project for skia-python.
Is it even possible?
This will enable python users to render rive animations for desktop applications,

I'm also waiting for this.


Add hotkey for Visual Diffs

This PR adds a new toggle hotkey (g) that highlights all unsaved edits on the screen.
Short video narrated by yours truly:
https://www.youtube.com/watch?v=Vo4q4D_Uo4o
It has support for lines, vertices, and areas, and when the 'G' key is pressed, green accent is drawn over/in the features in question.  Generally tag edit highlights are lighter /lower opacity than geometry changes.
Light green denotes a tagging-only change, darker green indicates a geometry change.
Geometry changes supersede tagging changes, so if both are true for a particular entity, the geometry style will prevail.

I'm interested in feedback on the visual styles I employed to communicate the diff information as well as the color choice!

haha cool.. I love this, and can't wait to try it out 👍

@Bonkles Thanks so much for this feature! It looks pretty good so I'm just merging as-is. I will tweak it somewhat but I just wanted to get it in there first 👍

So I iterated on the original design somewhat:

Include standalone points
Render the diff as a halo behind the features
Render line and area diffs the same way
Green for added, yellow for tag-only changes, orange for geometry changes

I think the results are good enough for now but I'm open to feedback and ideas!

I added a toggle button for this in the Map Data pane. This should make the feature more discoverable, escapable, and mobile-friendly.


Observed vs. expected behavior

Steps to reproduce

Environment

Querydsl version:
Querydsl module:
Database:
JDK:
Additional details

Robots is all the activity you gonna get here


Trying to resize FBLoginMock button - fallback to FBLoginMock neither worked

I have tried using FBLogin and resize the width of the button to the size of the screen by setting alignSelf: 'stretch' that did nothing and the size stayed the same, - However when clicking the button it function correctly.
I then tried the same with FBLoginMock which worked i did manage to make the button as big as i wanted it however clicking the button does nothing
Whats going on ? why is FBLoginMock does nothing?

Did you also copy over the FBLoginManager functionality from the LoginMock example? You might want to share your code because the example button should work as is from the example project.


Search specific text in Column

Hello,
I've been using this tool for translation. So in the Main tab I have the column for the source language (English) and in the column to the right the target language translation. There are some cases in which I need to translate concrete words, e.g., "Name", so as it is expected when I type "name" in the column filter, ResX RM shows me all text results that have the word "name" on it. This means sentences like "First Name" "A name must have 3 characters" and so on. What I need is to find texts that have the word "Name" and only that word. I tried used quotation marks, but it doesn't work. So I was wondering if there is any kind of filter I could use to find cells or text that only contain the specific text I'm searching. I don't know if this has been asked in the past, if so, apologies. I tried to read all threads and info related to this tool to see if I could find an answer before posting here, but with no luck.
Thanks

You can use regular expressions, e.g. ^name$

Thanks, that's exactly what I needed.


Higher resolutions?

Hey guys thanks for this really cool software it's really nice to be able to use the CPU as it lets me make much higher resolutions even though it's slower (I max out at 640x640 on my GPU, but here I can go all the way to 1024x1024). When I use the highest resolution my CPU still never goes higher than 60% and I have plenty of RAM over so I'm wondering if there is a way to generate even higher resolution images?
Also, a stable diffusion upscaler would be really nice as well!

@DiffusionAllusion now supports upscale https://github.com/rupeshs/fastsdcpu/releases/tag/v1.0.0-beta.24


zfs: import loop-backed storage pools + bugfixes

Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

Addresses https://github.com/lxc/lxd/issues/2990.

Includes test if import works correctly after an export.

Closes https://github.com/lxc/lxd/issues/2997.

jenkins: test this please

jenkins: test this please

jenkins: test this please

jenkins: test this please

jenkins: test this please

jenkins: test this please


Proxy for IDP + README update

1.) Fix for https://github.com/gambol99/keycloak-proxy/issues/271
New config option for proxy, which will be used for communication with IDP (that's common use case in the enterprise environment)
2.) A few typos in the README

lgtm


Docker build working; GitHub action scaffolding

do I need to do any config with this?

And removing this: cd .. - matters. If you remove that, you can't run the build script from the directory it's in, you have to run it from the one above

No, when you run the script, it runs in its own bash interpreter, right? The script can change to wherever it wants, and when the script exits, you're right back where you started.

do I need to do any config with this?

I don't think so.


Cannot deal with registry-based lookups of nixpkgs

We're getting a failure in https://github.com/DeterminateSystems/nix-to-kubernetes/actions/runs/5552539679/jobs/10140009561?pr=2 because nixpkgs is specified as nixpkgs.url = "nixpkgs";. Registry-based references like this should be looked up, and maybe also warned against -- in my opinion they're bad style but I'm not sure we all agree on this.

The problem happened before the flake update which introduced the local path reference too ;)

(and it's still the case after your change: https://github.com/DeterminateSystems/nix-to-kubernetes/actions/runs/5579412902/jobs/10195017057 )

@lheckemann The thing is, I have no idea why this is happening. Running the Flake Checker in that repo works fine and I even ran that flake.lock as a test case in the Flake Checker repo. I think it's a spurious error having something or other to do with Actions.

I'm closing this, as it hasn't been a Flake Checker issue specifically for a while. Once the new release (v0.1.12) is made available via the Action (PR pending), the original issue should be resolved.


Server crashes if mine URI is not resolved

Just had a server crash because mines.legumeinfo.org didn't resolve (for whatever reason, DNS work being done). This is unacceptable and should be fixed.

To be clear, this wasn't at startup. It was a running graphql-server. I'm guessing it's the web-properties polling that crashed it but I was a dork and didn't copy the error before I lost the screen.

Is this effectively a duplicate of #103?

It MAY be if it's the same solution. But this isn't a data source outage per se (like a mine being down), it's a DNS issue. Could be it's effectively a repeat but I thought I'd enter the DNS-caused version and we can delete it if it's the same resolution, which it probably is.

It MAY be if it's the same solution. But this isn't a data source outage per se (like a mine being down), it's a DNS issue. Could be it's effectively a repeat but I thought I'd enter the DNS-caused version and we can delete it if it's the same resolution, which it probably is.

It'll be the same solution, i.e. the Apollo server doesn't catch any non-graphql errors. Closing this issue as a duplicate.


Update architect orb

Switch build method to abs

This is missing a .abs/main.yaml file.
I recommend the following content:
replace-chart-version-with-git: true
generate-metadata: true
chart-dir: ./helm/jaeger-operator-app
destination: ./build

This is missing a .abs/main.yaml file.
I recommend the following content:
replace-chart-version-with-git: true
generate-metadata: true
chart-dir: ./helm/jaeger-operator-app
destination: ./build

oh no sorry :O accidental clicking around

oh no sorry :O accidental clicking around

Oh dang, sorry for making such a mess :( Follow up is #23

Oh dang, sorry for making such a mess :( Follow up is #23


Update zone redundancy to state that three instead of nine instances …

…are the minimum instance count
The Portal states the minimum instance count is three:

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

@GeorgDangl There is a difference between minimal instance count and minimal instance charge. A zone redundant plan has a minimal count of three instances. A zone redundant ASE has a minimal charge of nine instances. The documentation around pricing is correct.
#please-close


Is there a way to add the collectionView menu to another view?

I'm trying to remove the collectionView from its superView to add it to another view above the container which holds the pages, but when I try to, it also removes the pageView.
Anyway, thanks for sharing this awesome tool!

For those who wants this:
Add new constraints to pagingViewController?.pageViewController.view and its superview.


audit_log: Update audit log entries created while creating a group.

Earlier a extra audit log entry of type USER_GROUP_GROUP_BASED_SETTING_CHANGED was
made when a new user group is created. This commit updates the code to not create that audit log entry.
There is no need to create these entry as we would still have the required data from the "OLD_VALUE" field in the audit log entry created when changing the setting and this also makes it consistent with the entries created for other operations like stream creation.
Taken from #26355, since this was pending from a long time.

Fixes:  https://chat.zulip.org/#narrow/stream/101-design/topic/New.20permissions.20model/near/1612949

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @sahil839 and @Ujjawal3!


Question about displaying and editing settings.

Questions

What is the best way to display the timelapse in home assistant?
If I wanted to change a setting, what is the best way to do that? If I click on the 'Integration entries' in the mjpeg-timelapse, I just get the entity and can't get to the settings (ie frame rate etc).
Thanks

Probably a camera entity with live replay.
You can change the yaml settings otherwise if you've done it via the UI unfortunately you cannot update it after it's created. You'll need to re-add the entry.


feat: upload support presign

upload preferentially uses presign, and if the stroage does not support presign using stream upload instead.

the PUT statement should reuse this change.

the PUT statement should reuse this change.
 self.conn.put_files(&args[1], &args[2]).await?

Looks like a duplicate with #204 . This PR can be closed.


FW: ConcurrentModificationException in Client.java

@PrabithaK reported an issue in the JS-Helper  but it doesn't look like an issue from the JS Helper. It happened that this was related to the android client (not the java one).

Stacktrace from Issue #166 on algoliasearch-client-java:
Caused by java.util.ConcurrentModificationException
java.util.ArrayList$ArrayListIterator.next (ArrayList.java:573)
com.algolia.search.saas.Client.multipleQueries (Client.java:362)
com.algolia.search.saas.Client$5.run (Client.java:270)
com.algolia.search.saas.AbstractClient$AsyncTaskRequest$1.doInBackground (AbstractClient.java:692)
com.algolia.search.saas.AbstractClient$AsyncTaskRequest$1.doInBackground (AbstractClient.java:688)
android.os.AsyncTask$2.call (AsyncTask.java:288)
java.lang.Thread.run (Thread.java:818)

@PrabithaK, thank you for reporting this issue!
An ArrayList will throw a ConcurrentModificationException if you try to modify its content while iterating on it, see the documentation's section on iterators:

The iterators returned by this class's iterator and listIterator methods are fail-fast: if the list is structurally modified at any time after the iterator is created, in any way except through the iterator's own remove or add methods, the iterator will throw a ConcurrentModificationException.

When calling Client#multipleQueries, we iterate on queries and post a request for each one ; modifying the queries at the same time is thus forbidden.
@PrabithaK, can you check in your code what is modifying the queries after you call multipleQueries?

@PrabithaK, thank you for reporting this issue!
An ArrayList will throw a ConcurrentModificationException if you try to modify its content while iterating on it, see the documentation's section on iterators:

The iterators returned by this class's iterator and listIterator methods are fail-fast: if the list is structurally modified at any time after the iterator is created, in any way except through the iterator's own remove or add methods, the iterator will throw a ConcurrentModificationException.

When calling Client#multipleQueries, we iterate on queries and post a request for each one ; modifying the queries at the same time is thus forbidden.
@PrabithaK, can you check in your code what is modifying the queries after you call multipleQueries?

I have declared the List (queries) globally and this is cleared before calling the multipleQueries on changing the input in Search. multipleSearch is called everytime when user enters something.
Please suggest some soln.

The list that you pass to multipleQueries should not be modified between the call to multipleQueries and its return. As you tell me you declared this List globally, it is likely that you update it before the end of a call to multipleQueries: I suggest you to modify the queries before the call to multipleQueries or after, but not during.
If you cannot avoid modifying this list while the method executes, I advise you to call multipleQueries with a copy of the original List:
client.multipleQueries(new ArrayList<>(queries), strategy);


Help needed with context, context-file, context-array parameters

Hi,
I am trying to figure out how to correctly use context-file and context-array in API call. So far I can use context parameter and I get the desired/correct output of ModerMT:

But when I try to use context-file or array I get simple output:

Documentation states that output should be the same as context output.
My API call  looks like:
curl "http://localhost:8045/translate?q=Patients with unresolved thrombus at the lesion site\u002E&context-file=beckman" | python -mjson.tool >> TEST.txt
Should the path to context-file be full path /home/matecat/ModernMT/Data/beckman?(tried all the variations with no avail)
Could you please provide 2 correct samples of context-file and context-array usage in translate API call
For time being  I am using this: echo "This is an example" | extras/translate --context-weights medtronics:1 as suggested in #137
Thanks

Hi @wsuphappyhour
here's some examples that work with the data in examples/data/train/:
CONTEXT ARRAY
$ curl "http://localhost:8045/translate?q=world&context_array=%5B%7B%22id%22%3A%20%22europarl%22%2C%22score%22%3A1.0%7D%5D" | python -mjson.tool

The array sent is [{"id": "europarl","score":1.0}]. The current version of the API does not output back the context passed, this is a bad design choice and it will change in future release.
CONTEXT FILE
In order to get the context weights out of a text file you have to use the GET context API, you cannot use directly the Translate one. Here's an example:
curl "http://localhost:8045/context?local_file=/home/ubuntu/mmt/examples/data/train/europarl.en" | python -mjson.tool

{
    "data": [
        {
            "id": "europarl",
            "score": 0.99998164
        },
        {
            "id": "ibm",
            "score": 0.19798715
        },
        {
            "id": "microsoft",
            "score": 0.18271142
        }
    ],
    "status": 200
}

You can then reuse the context array obtained in the Translated call as previously illustrated.


Sentinel authentication failed when using the pingBeforeActivateConnection parameter

Bug Report
When the parameter
Sentinel authentication failed when using the pingBeforeActivateConnection parameter is reported to Sentinel the error below occurs:
2020-08-28 10:35:32,129 [lettuce-nioEventLoop-4-1] WARN  io.lettuce.core.RedisClient - Cannot connect Redis Sentinel at RedisURI [host='<IP_ADDRESS>', port=32193]: java.util.concurrent.CompletionException: io.lettuce.core.RedisCommandExecutionException: NOAUTH Authentication required.

Java 11.0.7 OpenJDK
Spring Boot 2.3.2.RELEASE
Maven 3.6.3

Input Code
Connection
@Configuration
public class RedisConnectionFactory {

	@Bean
	public StatefulRedisConnection<String, String> test()
	{
		RedisURI redisUri = RedisURI.create("redis-sentinel://<IP_ADDRESS>:32193,<IP_ADDRESS>:32194,<IP_ADDRESS>:32195?sentinelMasterId=mymaster");
		redisUri.setPassword("passwordFake");
		RedisClient client = RedisClient.create(redisUri);
		redisUri.getSentinels().forEach(it -> it.setPassword("passwordFake"));
		client.setOptions(ClientOptions.builder()
                .autoReconnect(true)
                .pingBeforeActivateConnection(true)
                .cancelCommandsOnReconnectFailure(true)
                .disconnectedBehavior(DisconnectedBehavior.REJECT_COMMANDS)
                .build());
		return client.connect();
	}
}

Set Key to test
@Configuration
public class ServiceCache {

	@Autowired
	StatefulRedisConnection redisConnection;

	@PostConstruct
	public void getName()
	{
		redisConnection.async().set("key", "value");
	}
}

Expected behavior/code
When starting my application a key with the name "key" should be added to my sentinel.
Environment

Lettuce version(s): Lettuce 5.3.1.RELEASE
Redis/Sentinel version: 6.0.6

Additional context
If I simply comment out the pingBeforeActivateConnection parameter, the connection will be successfully established and the expected behavior of adding a key to redis will occur

Good catch, that's fixed now.


Differences with ReRoPE

Differences with ReRoPE

Self-Extend (window_size = training_size/2) does not work at all
ReRoPE is Self-Extend (group PE) with window_size = training_size - 1

| Self-Extend | ReRoPE
-- | -- | --
  | tensor([[0, 0, 0, 0, 0, 0, 0, 0, 0, 0],        [1, 0, 0, 0, 0, 0, 0, 0, 0, 0],        [2, 1, 0, 0, 0, 0, 0, 0, 0, 0],        [3, 2, 1, 0, 0, 0, 0, 0, 0, 0],        [4, 3, 2, 1, 0, 0, 0, 0, 0, 0],        [4, 4, 3, 2, 1, 0, 0, 0, 0, 0],        [5, 5, 4, 3, 2, 1, 0, 0, 0, 0],        [5, 5, 4, 4, 3, 2, 1, 0, 0, 0],        [6, 6, 5, 5, 4, 3, 2, 1, 0, 0],        [6, 6, 5, 5, 4, 4, 3, 2, 1, 0]]) | tensor([[0, 0, 0, 0, 0, 0, 0, 0, 0, 0],        [1, 0, 0, 0, 0, 0, 0, 0, 0, 0],        [2, 1, 0, 0, 0, 0, 0, 0, 0, 0],        [3, 2, 1, 0, 0, 0, 0, 0, 0, 0],        [4, 3, 2, 1, 0, 0, 0, 0, 0, 0],        [5, 4, 3, 2, 1, 0, 0, 0, 0, 0],        [6, 5, 4, 3, 2, 1, 0, 0, 0, 0],        [6, 6, 5, 4, 3, 2, 1, 0, 0, 0],        [6, 6, 6, 5, 4, 3, 2, 1, 0, 0],        [6, 6, 6, 6, 5, 4, 3, 2, 1, 0]])
base | 256  : 1.0464 512  : 2.2302 1024 : 2.4575 | 256  : 1.0464 512  : 0.8970 1024 : 2.1020
+logn | 256  : 1.0464 512  : 2.2816 1024 : 2.8442 | 256  : 1.0464 512  : 0.8832 1024 : 1.8993
+keynorm | 256  : 2.1632 512  : 2.5168 1024 : 3.2525 | 256  : 2.1632 512  : 3.1556 1024 : 4.6036

You may check this section How to choose the group_size and neighbor_window. It's about how to select the two hyperparameters: group size / window size. Different models may have different empirical rules. But anyway, window_size = training_size/2 is too large.
Rerope is a special case of SelfExtend when the group_size = +∞ (or any  large enough value) rather than with window_size = training_size - 1 . Also, You may refer How to choose the group_size and neighbor_window for more results. Some of settings in this section is close to rerope. SelfExtend has superiority. We also have discussion in our paper about the relationship of SelfExtend and existing methods such as T5, iRPE and rerope. You can take a look at it for more details.


Send a notification to the inviter when the invitee redeems

Expected Behavior
Send a push notification to the inviter when the person they invited as redeemed the invite.
Current Behavior
The inviter has no way of knowing if the invitee has successfully redeemed the invite.

This is already implemented :)


wayland backend error in<EMAIL_ADDRESS>

I have a semi-reliable way of triggering a crash in the wayland backend and Wayfire, I have not noticed this before so it might be a recent regression. What I am doing: resize a client from SSD (so, the compositor sets custom cursor from via wlr_xcursor) inside the Wayland backend. I am seeing the following errors:
EE 04-01-21 12:12:07.382 - [render/swapchain.c:97] No free output buffer slot
EE 04-01-21 12:12:07.982 - [render/swapchain.c:97] No free output buffer slot
EE 04-01-21 12:12:14.520 - [render/swapchain.c:97] No free output buffer slot
wl_display@1: error 1: invalid arguments for<EMAIL_ADDRESS>EE 04-01-21 12:12:16.977 - [backend/wayland/backend.c:64] Failed to dispatch remote Wayland display
EE 04-01-21 12:12:16.977 - [backend/wayland/backend.c:64] Failed to dispatch remote Wayland display

After that, the compositor either goes into an endless loop, or crashes on shutdown.

Attaching the full log with debug output from wlroots.

Can you collect WAYLAND_DEBUG=client logs?

Maybe this helps?
diff --git a/backend/wayland/output.c b/backend/wayland/output.c
index 6bc84dcba1a8..633e9a594213 100644
--- a/backend/wayland/output.c
+++ b/backend/wayland/output.c
@@ -450,7 +450,7 @@ static bool output_set_cursor(struct wlr_output *wlr_output,
                wlr_egl_unset_current(&output->backend->egl);

                struct wlr_wl_buffer *buffer =
-                       create_wl_buffer(output->backend, wlr_buffer);
+                       get_or_create_wl_buffer(output->backend, wlr_buffer);
                if (buffer == NULL) {
                        return false;
                }

Maybe this helps?
diff --git a/backend/wayland/output.c b/backend/wayland/output.c
index 6bc84dcba1a8..633e9a594213 100644
--- a/backend/wayland/output.c
+++ b/backend/wayland/output.c
@@ -450,7 +450,7 @@ static bool output_set_cursor(struct wlr_output *wlr_output,
                wlr_egl_unset_current(&output->backend->egl);

                struct wlr_wl_buffer *buffer =
-                       create_wl_buffer(output->backend, wlr_buffer);
+                       get_or_create_wl_buffer(output->backend, wlr_buffer);
                if (buffer == NULL) {
                        return false;
                }

Much longer, but here it is:
client.txt

Much longer, but here it is:
client.txt

Maybe this helps?

Doesn't seem like it.

Maybe this helps?

Doesn't seem like it.

OK, cool. We were probably running out of FDs in the parent compositor because we were creating many wl_buffer objects.
I submitted the change as a PR: https://github.com/swaywm/wlroots/pull/2595

OK, cool. We were probably running out of FDs in the parent compositor because we were creating many wl_buffer objects.
I submitted the change as a PR: https://github.com/swaywm/wlroots/pull/2595

Fixed in https://github.com/swaywm/wlroots/pull/2595

Fixed in https://github.com/swaywm/wlroots/pull/2595


UI fails to run when launched with sudo in Linux enviroment

When launching a UI with the sudo command (ex. "sudo dotnet myapp.dll" or "sudo -s" "dotnet myapp.dll") the build seemingly fails.
This can be reproduced with any basic application, in my particular case it requires superuser access. Running as a user runs as expected (aside from permission issues from my client dll which isn't related to any UI issues).
Desktop (please complete the following information):

Ubuntu 62 bit
Ubuntu 22.04.1 LTS

Unhandled exception. System.Exception: XOpenDisplay failed
   at Avalonia.X11.AvaloniaX11Platform.Initialize(X11PlatformOptions options) in /_/src/Avalonia.X11/X11Platform.cs:line 57
   at Avalonia.AvaloniaX11PlatformExtensions.<>c__0`1.<UseX11>b__0_0() in /_/src/Avalonia.X11/X11Platform.cs:line 280
   at Avalonia.Controls.AppBuilderBase`1.Setup() in /_/src/Avalonia.Controls/AppBuilderBase.cs:line 303
   at Avalonia.Controls.AppBuilderBase`1.SetupWithLifetime(IApplicationLifetime lifetime) in /_/src/Avalonia.Controls/AppBuilderBase.cs:line 179
   at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime[T](T builder, String[] args, ShutdownMode shutdownMode) in /_/src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime.cs:line 208
   at myapp.Program.Main(String[] args) in C:\git\myapp\Program.cs:line 14

Check if xterm works with sudo on your Linux machine.

XOpenDisplay means a failure to connect to X11, so it's an environment issue


fix for a segfault

Description
I added a fix to a segfault that is happening when calling .text() on an empty menu.

Thank you 🙏


STEMM Role Models

[ Project Lead ] @KirstieJane
[ GitHub Repo ] https://github.com/KirstieJane/STEMMRoleModels
[ Track ] Tools: you are building a tool others can help you build
[ Level ] Beginner
Description
Inspire future generations by providing the most exciting and diverse speakers for your conference.

Note to the Project Lead
Congrats, @KirstieJane! This is your official project listing for the Mozilla Science Global Sprint 2016. Please comment on this issue to confirm your project submission and complete the following:

[x] Provide a GitHub repository for work and discussion on your project
[ ] Confirm in a comment that at least one person will be available to review and answer questions on this project from 9-5 in their timezone on both June 2 & 3.

Confirmed @acabunoc! We'll host a site in Berkeley CA :smiley:


GC test failure on 32-core on high abort

C++ exception of vector index access out of bounds reported by student. Because the access was UINT64_MAX into a 0-size array, overflow is the likely cause.

Shouldn't this change get merged into the project branch? Also the BwTree test is reliably failing. @tli2 @malin1993ml

Yes, we need to merge this to the project branch.
@yeshengm Can you post a separate issue for the failing bw-tree test?

We have #264 open. I've still never seen it myself. If we can get logs I can probably enlist @wangziqi2013's help to look at it.

There's already one open issue #264 for BwTree.

@mbutrovich Run the test cases in debug mode and the bug appears. Otherwise in release mode the test cases just can't catch the bug.


How to do visualization using PAN?

@aswa123 you can visualize using opencv cv2.polylines() function.
with torch.no_grad():
outputs = model(**data)
if cfg.report_speed:
report_speed(outputs, speed_meters)
bboxes = outputs['bboxes']
img_name=os.path.basename(image_path)
filename, file_extension = os.path.splitext(img_name)
with open(os.path.join(result_path,filename+'.txt'), 'w') as f:
for i, bbox in enumerate(bboxes):
poly = np.array(bbox).astype(np.int32).reshape((-1))
strResult = '\t'.join([str(p) for p in poly]) + '\r\n'
f.write(strResult)
poly = poly.reshape(-1, 2)
cv2.polylines(ori_img, [poly.reshape((-1, 1, 2))], True, color=(0, 0, 255), thickness=2)
result_name='res_%s.jpg' % filename
cv2.imwrite(os.path.join(result_path,result_name), ori_img)

@bharatsubedi Thank you very much for your attention.
sorry to bother you but where I need to place this code in test.py file

@aswa123您可以在 test.py 文件中使用 opencv cv2.polylines() 函数进行可视化。
with torch.no_grad():
        outputs = model(**data)

bboxes = outputs['bboxes']
img_name=os.path.basename(image_path)
filename, file_extension = os.path.splitext(img_name)
for i, bbox in enumerate(bboxes):
      poly = np.array(bbox).astype(np.int32).reshape((-1))
      poly = poly.reshape(-1, 2)
     cv2.polylines(ori_img, [poly.reshape((-1, 1, 2))], True, color=(0, 0, 255), thickness=2)
result_name='res_%s.jpg' % filename
cv2.imwrite(os.path.join(result_path,result_name), ori_img)

did you prepare a code for testing with single images? I also want to test with a single image. Could you please share your code? Thank you !


Argon uses wrong project.json when using a multi-place setup

While testing Argon 2 for usability for my team, I ran into an issue when setting up more than one .project.json file. Our two files are Town.project.json and Track.project.json, pictured here.

When running the Town project with argon run Town.project.json, everything works fine. When running the Track project file, however, everything appears to work correctly, except that it mirrors what's in the Town file instead. At first, I thought it was due to Argon just not picking up the correct file, but the Argon Roblox plugin was picking up the "Track" project name. So I went back and ran argon sourcemap Track.project.json and found out what is going on. The filePaths property generated by the sourcemap refers to the Town project file instead of the Track project file, shown below.

It is probably worth noting two things - first, the Town project file was created before the Track project file. Second, I'm fairly certain we ran Argon once when the Track file did not yet exist. Not sure why, but something seems to have caused it to latch onto the Town project as the "default", and I suspect it could have something to do with either of these steps we took while testing.

I just saw this after posting this issue; seems likely that it's the same bug?
https://github.com/argon-rbx/argon/discussions/22#discussioncomment-8866550

Yep that's the same bug. I thought I fixed it but it is slightly more complicated.
Argon middleware processes "child" instances diffrently as it gets first occurence of matching SyncRule in the parent directory of that instance. And the project is "child" instance as its .json file is located under the main directory and because its file also contains name: *.project.json it makes the method I mentioned not work properly.
Even though it is the only exception I'm significantly modifying the middleware right now but as I said the update will be available today.

Fixed in d38ef21f841cd48cb94ca240c911fbfe7fb18b4c. Available in 2.0.0-pre4.
The fix however caused another bug - sourcemap does not include .src and init files. I'm going to fix this tomorrow.


XP Globes show inaccurate timer for Agility

Type
Incorrect behavior
Operating System
None
OS Version
MacOS Monterey
Bug description
Needing 21 more laps, the calculator says it should only take me 2m13s left.
I've done nothing else since logging in other than than 9 laps recorded, so no other activities are influencing the XP per hour.
(My guess: could it possibly be taking the xp related to the large xp drop at the end of completing the course and multiplying THAT by the number of actions taken, rather than the rolling average? that would explain why it thinks i'm making XP BANK rather than just 240 a lap)
Currently at Canifis.
Screenshots or videos

(photo at 11:42AM)
RuneLite version
No response
Logs
No response

updated photo as I go.

duplicate of #10099 most likely

Agreed, looks like #10099 strikes again.


Add .NET 5 support

Describe the feature

Now .NET 5 is released and I want to migrate the target framework of my WPF project(used MahApps.Metro) to .NET 5. So I need this library to support .NET 5.
Additional context

This will be done after doing this with ControlzEx.

DateTimePicker no 24-hour system

DateTimePicker no 24-hour system

@ourplan9000 ?

Can someone give an update on the .Net 5 compatibility/state at the moment? As I see, ControlzEx should be compatible - Is there an .Net 5 preview of MahApps?

Hi @auxua
as you cleary can see it is marked with the 3.0.0-Milestone. So there is a plan, but as this is a free-time project it will be done when there was enough free time.
Asking frequently will not make the release happen earlier.
Thank you for your understanding & happy coding
Tim

@auxua It's coming... ControlzEx will be available in next weeks, after that the 3.0 of MahApps will start, after the fixes for 2.4.4

Will compatibility come for .net 5?

@crippledfaith it is in work. 🚧 Please be a bit patient because it is a fee time project and it's done when it's done. Give it a bit more time.
Happy coding
Tim

Hello, just to give a small feedback on that topic...
First of all, I love this project, I use it for years and it's beautiful!
Congrats to the dev team, you're doing an amazing job!
About .Net 5, I did the migration (.Net Framework to .Net 5) of one of my personal projects and everything seem to work fine with MahApps, so that's really great news!  🥇
During the build I just have a few warning:
warning CA1416: This call site is reachable on all platforms. 'ThemeManager.Current.get' is only supported on: 'Windows' 7.0 and later.
warning CA1416: This call site is reachable on all platforms. 'ThemeManager.SyncTheme()' is only supported on: 'Windows' 7.0 and later.
warning CA1416: This call site is reachable on all platforms. 'ThemeSyncMode.SyncAll' is only supported on: 'Windows' 7.0 and later.
According to Microsoft (about CA1416):
Cause
Violations are reported if a platform-specific API is used in the context of a different platform or if the platform isn't verified (platform-neutral). Violations are also reported if an API that's not supported for the target platform of the project is used.
This rule is enabled by default only for projects targeting .NET 5.0 or later. However, you can enable it for projects that target other frameworks.

We are waiting patiently


[Port v2.8] Skip clusterdeploy handler if downstream cluster is disconnected

This is a port of https://github.com/rancher/rancher/pull/43542
Ported issue is https://github.com/rancher/rancher/issues/43570
All commits cherry-picked without any changes.

@snasovich I need you or somebody from your team to take a look at this PR, can you point me to the most appropriate person? Thanks in advance

Note Drone CI is failing on a known flaky test:
https://drone-pr.rancher.io/rancher/rancher/34606/3/2
https://github.com/orgs/rancher/projects/44/views/1?pane=issue&itemId=44014987


Problem with mask

So i have
{{ phone }}
		<label for="phone">Phone</label>
		<masked-input
			id="phone"
			class="form-control string"
			v-model="phone"
			mask="\+\7 111 111-11-11"
			placeholder=""
			type="tel" />
	</div>

And i have phone: '72731241241',
Every time i open this modal i will get +7 727 312-41-24
Next try +7 772 731-24-12
And next +7 777 273-12-41
And next +7 777 727-31-24...

It' look like mask work wrong, when send model data back to input.
Example I have mask ="+\9\6\2 111-11-11"   and enter number  + 962 333-33-33.   After save correct number, and then try edit, and put data back to input  - result is + 962 962 333-33-33.  It's not cut static numbers at beginning.
Solution can be to save raw value@input="rawVal = arguments[1]"  and use it.
Or in my case, I just cut static start of  mask
<masked-input  v-model="userPhone" mask="\+\9\6\2 / 11 / 111/ 1111" placeholder="Phone"/>
data() {
		return {
			phone: '',
},
computed: {
		userPhone: {
			set(val) {
				this.phone = val;
			},
			get() {
				return  this.phone.substring(4);
			},
		},
}

Same problem


Specification1.1

JSON Feed version 1 specified a singular author field instead of the authors array used in version 1.1. New feeds should use authors, even if only 1 author is needed. Existing feeds can include both author and authors for compatibility with existing feed readers. Feed readers should always prefer authors if present.
Deprecated items remain valid forever, but you should move to the new fields when you can. A feed using fields from JSON Feed 1.0 is still a valid feed for version 1.1 and future versions of JSON Feed.
ToDo: Extensions. Publishers can use custom objects in JSON Feeds. Names must start with an _ character followed by a letter.

@RouR apologies for not seeing this sooner. Thanks for the PR!


Simple Image Classification using ANN

Required Changes:

outputlayer/last layer must have number of units equal number of classes in case of multiclass classification or 1 in case binary classification.
is this description required: Model Tuning Parameters¶
n_estimators -> Number of Dense layer.
output of sample data is not there but code exists

for example:
model.add(Dense(1024, input_shape=(X_train.shape[1],), activation="relu"))   # first hidden layer has 1024 nodes
model.add(Dense(, activation="sigmoid"))

Above changes done, but i am facing error in last three codes.
In Model Accuracy
AttributeError: 'Sequential' object has no attribute 'score'
In Confusion Matrix
ValueError: plot_confusion_matrix only supports classifiers
In Classification Report
ValueError: Classification metrics can't handle a mix of binary and continuous-multioutput targets

How to solve them ?

This data is used.
https://drive.google.com/open?id=1dbcWabr3Xrr4JvuG0VxTiweGzHn-YYvW

To Debug this issue.
first you must understand how te prediction looks like

Tensorflow library doesnt have score function for testing accuracy : utilize model.evaluate(test)
for confusion matrix use confusion_matrix(Y_test,y_pred)
for classification report also you have first understand and manipulate the predicted output

For instance your resulting neural network has wrong architecture in terms of output layer
Reason : number of neurons in last layer should be either equal to number of classes in target  or 1 in case of binary classification
at same time activation function influence result outputs. for binary problem it preferred to use sigmoid (returns either 1 or 0)
while for multiclass classification problem softmax(returns probability of each available classes) activation function is preferred
the reason why the classification report throws error is due to mismatch in y_test and predicted data
your y_test is binary in nature while your y_pred is not.

i did not understand the exact issue with output layer.
What to change exactly ? and again how ?

https://colab.research.google.com/drive/1gnATlBplxD2QYtXN5xZ9CTYCF9fQbBeq?usp=sharing


2 embedding matrices

Hey,
why do you have a separate embedding matrix for the questions and contexts? Why not just use the same embedding matrix for each?

Because the authors of relation network use separate LSTM for the questions and contexts. :)

I'm not as familiar with NLP. Why don't you use the separate LSTMs with the same embedding matrix?

Sorry, the above answer was unkind... In the original paper of the Relation Network, embedding question and context separately, reminded me of a Memory Network. What was mentioned in this paper was that when question and context were embedded via different embedding matrices, they performed better than when using the same embedding matrices (They left the reason as a future work). So I tried to separate the question and context as much as possible, by using different embedding matrices.
However, as I was also curious about this question, I ran a model with the same embedding matrix, and found out that the performance was slightly better. Thanks for catching this. I have to open this to the issue for a while and check again after fixing model to handle multi-class task.

This is the result. I don't get why task 2 accuracy much lower than others.
task 1 acc: 0.9980
task 2 acc: 0.4362
task 3 acc: 0.9990
task 4 acc: 1.0
task 5 acc: 0.9910
task 6 acc: 0.9950
task 7 acc: 0.9146
task 8 acc: 1.0
task 9 acc: 0.9337
task 10 acc: 0.9367
task 11 acc: 0.8291
task 12 acc: 0.9950
task 13 acc: 0.9980
task 14 acc: 0.8523
task 15 acc: 1.0
task 16 acc: 1.0
task 17 acc: 0.9980
task 18 acc: 1.0
task 19 acc: 0.9960
task 20 acc: 1.0

Sure thing. Thanks for clarifying!

Those are very high results. The authors of the original paper didn't
succeed on either task 2 or 3. Why do you think you succeeded on the more
difficult of the two (3)?
On Mon, Aug 7, 2017 at 10:15 AM Jihyung Moon<EMAIL_ADDRESS>wrote:

This is the result. I don't get why task 2 accuracy much lower than others.
task 1 acc: 0.9980
task 2 acc: 0.4362
task 3 acc: 0.9990
task 4 acc: 1.0
task 5 acc: 0.9910
task 6 acc: 0.9950
task 7 acc: 0.9146
task 8 acc: 1.0
task 9 acc: 0.9337
task 10 acc: 0.9367
task 11 acc: 0.8291
task 12 acc: 0.9950
task 13 acc: 0.9980
task 14 acc: 0.8523
task 15 acc: 1.0
task 16 acc: 1.0
task 17 acc: 0.9980
task 18 acc: 1.0
task 19 acc: 0.9960
task 20 acc: 1.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/juung/Relation-Network/issues/2#issuecomment-320724081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF7pvo6C8duQFoCzu-PnWc226CGvAAXCks5sV0YfgaJpZM4OtP70
.

--
Wilka Carvalho,
https://wcarvalho.github.io/ https://wcarvalho.github.io/


Include a mermaid architecture diagram

This PR includes an architecture diagram created with mermaid.js.
Additionally, this PR cleans up some of the top level README.
Related: #35
Signed-off-by: Steve Martinelli<EMAIL_ADDRESS>

@meganmajewski pushed a new revision.

looks good!


Replace babel require hook with transformFileSync

Start of discussion - https://github.com/sindresorhus/ava/commit/52b0ccb33ca30bba5185056361f1c2976b7cf56f#commitcomment-13971327
If you have babel and want to use require hook in application code and want to run tests with ava - you can get syntax errors (SyntaxError: Unexpected reserved word) because babel will get options from ava.

Does the caching still work with this solution? Since Babel is relatively slow, the caching is very important.

@sindresorhus I'm afraid, cache should be implemented by hands.
co must be replaced (or patched) to use Promise polyfill, since babel now will not do this for us.

@floatdrop in order to support the stuff that "generator community" is used to, co is required. Babel's built-in implementation is very limited.

@floatdrop Maybe you could either extract this into a separate module we can depend on or do a PR on Babel to expose it?

@vdemedes yeah, I know, but you can't use co in Node.js 0.10 without some transpiler (and I don't think, that co will adapt to it soon https://github.com/tj/co/pull/244). So we need to come up with something. I believe, that transpiling only test code will improve performance compared to patched require function.
@sindresorhus okay, but need to do some performance tests - I wonder, how fast cache file will fill up and parsing time eat all the benefit.

@floatdrop generators are only being used in node 0.11, iojs and node v4.x projects, so we don't have to worry about that.

@vdemedes co uses global Promise object - so you can't use it right away in Node.js 0.10.

@floatdrop Hmm, did not understand the point. I am just saying that nobody uses generators on the old node.js versions.

@vdemedes generators are not a problem, yes. But ava does require co at the top of test.js and this leads to errors in 0.10 - https://travis-ci.org/sindresorhus/ava/jobs/87312158

JFYI: seems like babel require hook have some polyfill side-effect - https://github.com/floatdrop/ava-in-node-0.10

Just to keep track - waiting for https://github.com/tj/co/pull/250

Doesn't look like co has much activity, so I don't think we should wait on that tbh.

Why do we even depend on co again? Can't Babel+Regenerator do this for us?

Because co implements some very nice features, which babel's regenerator does not implement. Babel's implementation only makes Promises yieldable, but nothing else.
Why don't we use @floatdrop's fixed fork for now, until they merge his fix into master?

Why don't we use @floatdrop's fixed fork for now, until they merge his fix into master?

:+1:

okay, but need to do some performance tests - I wonder, how fast cache file will fill up and parsing time eat all the benefit.

@floatdrop I'm ok with merging this now if you commit to looking into caching later. Compilation is a lot larger overhead than just reading in a cached file from disk. I do think we can do caching better than Babel itself, though. Mainly using separate cache files for each test file (Babel uses one large file) and a LRU cache.

@sindresorhus sure. I'm started to use ava in some work related projects, so I will hang around for some time :)

I'm started to use ava in some work related projects

:heart_eyes:

This looks good to me. @vdemedes Can you take a final look?

Good to go! 👍

Thanks again @floatdrop :)


kvm/ceph: When create a new RBD image use modern flags/features for t…

…he image
These additional RBD features allow for faster lookups of how much space a RBD
image is using, but with the exclusive locking we prevent two VMs from writing
to the same RBD image at the same time.
These are the default features used by Ceph for any new RBD image.
Signed-off-by: Wido den Hollander<EMAIL_ADDRESS>Description

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2397

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3136)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 23708 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2985-t3136-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Smoke tests completed. 66 look OK, 3 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_nic_secondaryip_add_remove
Error
42.02
test_multipleips_per_nic.py

test_04_rvpc_network_garbage_collector_nics
Failure
461.52
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Failure
319.77
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
343.88
test_vpc_redundant.py

test_02_cancel_host_maintenace_with_migration_jobs
Error
2.32
test_host_maintenance.py

Thanks for the feedback! See the updated code

Yes, I trusted my IDE to help with this, but fixed it now! Compiles :)

@blueorangutan package

Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2423

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@wido @DaanHoogland @rhtyd merging it to the master branch, based on code review (3 LGTMs) and that all checks have passed.

Trillian test result (tid-3163)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24282 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2985-t3163-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 66 look OK, 3 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_nic_secondaryip_add_remove
Error
32.93
test_multipleips_per_nic.py

test_04_rvpc_network_garbage_collector_nics
Failure
484.29
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Failure
305.61
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
327.72
test_vpc_redundant.py

test_01_vpc_site2site_vpn_multiple_options
Failure
230.04
test_vpc_vpn.py


HDDS-9933. Recon datanode 'Last Heartbeat' should print relative values

What changes were proposed in this pull request?
Recon's frontend receives a timestamp number from the backend and then formats the number as a date. It's more useful to the cluster admin and practical, to view the Last Heartbeat as a relative value than an entire date.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9933
How was this patch tested?
The patch was tested manually, see the attached screenshot.

Also,  because there are no unit tests for Recon's frontend, the method was tested externally. Here are some sample values
[LOG]: "31s ago"
[LOG]: "50s ago"
[LOG]: "1m 0s ago"
[LOG]: "1m 2s ago"
[LOG]: "1m 15s ago"
[LOG]: "1m 16s ago"
[LOG]: "1m 33s ago"
[LOG]: "2m 54s ago"
[LOG]: "7m 17s ago"

BTW, this what 'Last Heartbeat' looks like in master

cc @devmadhuu

@devmadhuu Thanks for the review. Do you know anyone else that I can ping, who's working on Recon's frontend?

@devmadhuu Thanks for the review. Do you know anyone else that I can ping, who's working on Recon's frontend?

@devabhishekpal @smitajoshi12 could you pls review

@xBis7
We have implemneted similar functionality uisng one line code  using moment library. Can you check associated code so we can apply similar logic on column.

moment(lastUpdatedOMDBDelta).fromNow()

@smitajoshi12 That was my first approach to this but moment().fromNow() gives pretty vague results like a few seconds ago, a minute ago, ...
In the last heartbeat scenario default values are up to 30/60 seconds and we need them to be more precise than a few seconds ago.
https://github.com/apache/ozone/blob/master/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/HddsConfigKeys.java#L25-L32

Can You check values with DB.json manually editing old values and UI  some negative test cases also and cluster data

@xBis7, fromNow() is expected to give vague descriptions like this.
Here is the breakdown of what is displayed for what time range.

Can You check values with DB.json manually editing old values and UI some negative test cases also and cluster data

@smitajoshi12 Yes, sure. I'll post some screenshots. When saying negative test cases, you mean like provide values from dates later than the current? Can you explain more?

@xBis7, fromNow() is expected to give vague descriptions like this.
Here is the breakdown of what is displayed for what time range.

@abhishekaypurohit Thanks for the link and the clarification. That's what I'm saying, that moment().fromNow() might be convenient but the values are pretty vague for this use case. Our operations team wants to see precise values, 2s ago, 3s ago, ...

Thanks @devabhishekpal, your code snippet looks good. I'll push the changes shortly.
@smitajoshi12 Here is a screenshot from pnpm run dev with some more recent values in db.json. Let me know if this isn't what you meant.

@smitajoshi12 Any comments? Do you want to another look? Otherwise, I'll merge it once there is a green CI.

@smitajoshi12 Any comments? Do you want to another look? Otherwise, I'll merge it once there is a green CI.

@xBis7
Thanks Before substring can check with && otherwise substring will fail UI will break. If We get 0 value LastHeartbeat or if there is no data then check UI is failing or not. Apart from It looks good to me. Thanks for this patch

If We get 0 value LastHeartbeat or if there is no data then check UI is failing or not.

@smitajoshi12 If heartbeat is anything other than a positive number, we get NA.
return heartbeat > 0 ? getTimeDiffFromTimestamp(heartbeat) : 'NA';

The values I provided in db.json were

asd for localhost2
-1 for localhost3
0 for localhost4

Check screenshot. We get NA in all cases.

If We get 0 value LastHeartbeat or if there is no data then check UI is failing or not.

@smitajoshi12 If heartbeat is anything other than a positive number, we get NA.
return heartbeat > 0 ? getTimeDiffFromTimestamp(heartbeat) : 'NA';

The values I provided in db.json were

asd for localhost2
-1 for localhost3
0 for localhost4

Check screenshot. We get NA in all cases.

@xBis7
Thanks It looks good to me

Thanks @devmadhuu @smitajoshi12 @devabhishekpal for the review.


fix(alert): added missing 'disabled' attribute from input

Short description of what this resolves:
Alert inputs can be disabled.
Ionic Version: 3.x
Fixes: #13488

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


File Picker won't pick file on iOS XCode 11.1

It worked in past versions until I upgraded XCode to 11.1 and VS for Mac to 8.3.4
Expected Behavior
File Picker is openned,
the user selects a file
File Picker is closed and the file data is returned
Actual Behavior
The File Picker is not closed after a user selects the file. Actually file selection doesn't seem to work at all. It is possible to rename the file, delete, etc. but not to select it.
Steps to Reproduce the Problem
using Plugin.FilePicker;
using Plugin.FilePicker.Abstractions;
data = await CrossFilePicker.Current.PickFile();
if (data!=null)
{
}
Specifications

NuGet package version: 2.1.34
Platform(s):
Visual Studio for Mac 8.3.4 (Xamarin.iOS <IP_ADDRESS>)
XCode 11.1

I guess this is on an iOS 13 device? Does it work on iOS 12?

It does
On Mon, Oct 21, 2019, 1:13 PM vividos<EMAIL_ADDRESS>wrote:

I guess this is on an iOS 13 device? Does it work on iOS 12?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jfversluis/FilePicker-Plugin-for-Xamarin-and-Windows/issues/156?email_source=notifications&email_token=AIOTFRKTN5FTP3BXYBOBR7LQPV6FJA5CNFSM4JCTGDYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZZ3GA#issuecomment-544447896,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOTFRJFUYPECSRDWKZXVQDQPV6FJANCNFSM4JCTGDYA
.

I just tested this with the FilePicker sample project on an iOS 13.1.3 and it works without problems. The file picker is presented as a page sheet, not as a new view controller, which is new in iOS 13. Can you reproduce the error with the sample project? If not, can you share more of your project code, best would be a compilable project. Thanks!

Hi, Was this ever addressed? I am having same issue. Works fine on all versions except iOS 13.

There was no code fix or something. If you can reproduce it, why to try to fix it? I don't have a Mac to debug this at the moment. We are happily accepting PRs!

Closing this since this project will retire, please check out Xamarin.Essentials which now has a FilePicker based on this one.


The "expand" transform costs too much time when meets large image

When I train my own dataset , I found sometime GPU-util became 0 , and expand transform costed too much time.
This is my test, you can find np.full  function in "expand"  is slow.

So , is there any way to solve this?

use np.full((int(h * ratio), int(w * ratio), 3), 1.2, dtype=np.float32)


implement OGC API - Features - Part 4

This PR implements support for the OGC API - Features - Part 4: Create, Replace, Update and Delete draft specification.

Coverage increased (+0.1%) to 59.472% when pulling d348f9aeaec9d1b6777cb12a4d19469223fd1268 on tomkralidis:ogcapi-transactions into 08c41d18b7a26a15298f5c34c3d7a2b9f7276239 on geopython:master.


chore(explore): Medium font weight for section headers

SUMMARY
Set font-weight: 500 for section headers in Explore's control panel.
This is one of a few incoming PRs to improve the visual balance in the control panel + fix readability after uppercase changes in control panel.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I believe there's a conflict of interest and @kgabryje should not be able to approve @kasiazjc's PRs 😛 😍

I believe there's a conflict of interest and @kgabryje should not be able to approve @kasiazjc's PRs 😛 😍

I had to, no dinner otherwise

I believe there's a conflict of interest and @kgabryje should not be able to approve @kasiazjc's PRs 😛 😍

I had to, no dinner otherwise

a lie, obviously, he's just keeps an eye on new prs 🤷


LangFlow can't run in the same environment than LangGraph SDK

Feature Request
Increase orjson minimum version.
langflow-base 0.0.94 requires orjson==3.10.0, but you have orjson 3.10.1 which is incompatible.
langgraph-sdk 0.1.29 requires orjson>=3.10.1, but you have orjson 3.10.0 which is incompatible.
Motivation
I have a environment running with LangFlow and trying to install LangGraph in this same environment to make some more complex workflows (hope in the future to create some custom component in LangFlow to use this LangGraph agent).
However, I can't install both libs in the same environment because LangGraph SDK requires >= 3.10.1. But in LangFlow is required 3.10.0 version.
Is there any reason to maintain this support for "3.10.0" version instead of newer versions?
Your Contribution
orjson releases doesn’t seem to have any breaking change.
https://github.com/ijl/orjson/releases
Probably only change version in this file:
https://github.com/langflow-ai/langflow/blob/59b6d8acc083dccf63dfd735b0a9d27e9a458b49/src/backend/base/pyproject.toml#L48
And run testing suite, to make sure it does not break anything.

Hi there! How are you?
Could you provide more details about your environment and specific needs? Based on your response, I believe it might be possible to update the library version manually.


Thread email notification example

Description

Adds an example to send an email using the threadEmailNotification event.
Adds the threadEmailNotification documentation.
Adds threadEmailNotification event.

Instructions for running

Setup ngrok to point to locahost:3000 to have the webhook endpoint available.
Setup webhook endpoint in dev

Docs look good 👍 I was going to to turn this into a guide, but I'm not going to have a chance before it's merged @FlowFlorent.

I have more time now, so I'm going to look at this properly. I'm going to build an example in another branch.


Allow functions marked @LLDBDebuggerFunction to bypass actor isolatio…

…n checks
In order to allow LLDB to evaluate expressions inside an actor without spawining
async functions and potentially continue all threads, this relaxes the actor
isolation checks in @LLDBDebuggerFunction functions to allow a synchronous call
into an extension method.
rdar://75905336
(cherry picked from commit 93449728b9b7aaf84ad69ed1d0d7785cd825e0d6)

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 2a01301632ca47db43268679a9cbdf4f3cd3b62a

@swift-ci Please test macOS


Taxes - System message for tax amount edit shows USD first, then changes to correct currency

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: 1.4.60-1
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

The Collect workspace has taxes.
The default workspace currency is not USD.

Create a manual request with tax in non-USD currency.
Go to transaction thread.
Click Tax amount.
Edit the amount and save it.

Expected Result:
The system message for tax amount edit will show the correct tax currency.
Actual Result:
The system message for tax amount edit shows the tax in USD first, then changes to the correct currency.
Workaround:
n/a
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/a98dff0f-5a98-4bc9-9a16-95b9db5a86fe
View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 1

This can be demoted from deploy blocker. Not a major issue. Feel free to assign it to me. Happy to handle it.
We can handle as a regression from https://github.com/Expensify/App/pull/33927

PR is merged! No further action required here


Unable to install/progress to "Configuration" stage

I am not able to install the integration under the "Configurations - Integrations" menu on HA. I get the following (possibly related) errors in the log:
2021-02-28 19:45:12 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.
2021-02-28 19:45:13 WARNING (MainThread) [slixmpp.stringprep] Using slower stringprep, consider compiling the faster cython/libidn one.
Any suggestions you have would be greatly appreciated.

Nevermind. It showed up finally. I think I had to install Pyscript via HACS. That's the only change I made. Thanks.


IGAPP-1145: Locally deploy rasa widget

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

We should check if the errors @f1sh1918 got are fixed after the Salt PR is merged.

Implementation looks fine, but i get an invalid certificate error

Cannot reproduce. I do not get this error. Also this seems to be an error of the certificate twb created, so we cannot change this anyways. But it seems they have fixed this already.


Change of Currency

Another cool feature once the USD values are imported automatically via the API, could be to change how they are displayed in the app, by converting the USD value to the GBP value via the latest exchange rate or something similar!?

@PhillWalker I've got this one in the works.  Should have a pre-release for you to review in a day or two.


feat(pipettes): add GPIO EXTI interrupt support for the tip sense GPIOs

Overview
We want to decide that a tip has been detected based on an EXTI interrupt. This PR adds the piping in HAL to support an EXTI interrupt. While making these changes, I also edited the code on the pressure sensor to support two data ready lines for the 96 channel. We'll also want to do this for DVT 8 channel pipettes since there are now two separate data ready lines.
Test Plan

[x] Test that the pressure sensor still functions normally on the single/eight channel

Yeah I needed to test this on the firmware to see if that would actually work. I know you can combine GPIOs but it wasn't clear to me in the NVIC documentation whether the same thing would work or not. I can just change the implementation (and probably make the code more readable by doing so.)

Codecov Report

Merging #601 (ab3420a) into main (2680171) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #601   +/-   ##
=======================================
  Coverage   83.47%   83.47%
=======================================
  Files          84       84
  Lines        3056     3056
=======================================
  Hits         2551     2551
  Misses        505      505

Impacted Files
Coverage Δ

include/can/core/messages.hpp
99.21% <ø> (ø)

include/sensors/core/utils.hpp
100.00% <ø> (ø)

Tested liquid sensing, auto calibration and the new tip sense functionality on a robot. Everything is functioning as expected ✔


add user-analysis

what kind of users,
what do they expect (goals, expectations),
what do they require,
what issued do they encounter

functional issues
quality issues
layout/design issues

For me this has implicitly been part of the stakeholder analysis so far

UA has more focus on UI stuff...,
Includes Useability, layout, design...
I want to make these more explicit than in the more general stakeholder analysis...

For me this has implicitly been part of the stakeholder analysis so far
—
Reply to this email directly or view it on GitHub.

So the idea is not only to analyze (and name) the users of the system but to also find issues that those users have when they (try to) utilize the system to fulfill their tasks/ workflow?!


Update playbook-settings.md

changed details of quiet mode, per notes from Meir

Status
Ready/In Progress/In Hold (Reason for hold)
Related Issues
fixes: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer

@meirwah

@meirwah looks good.
the only thing i'm not sure is about auto-extract.
@itayronen quiet-mode is impact on auto extract?

@meirwah looks good.
the only thing i'm not sure is about auto-extract.
@itayronen quiet-mode is impact on auto extract?

Never tested until now.
Tested on a task that prints an indicator, and on a task that explicitly calls extractIndicators.
Both tasks did on quiet mode, did NOT create any indicators on the incident.

@meirwah

can someone squash/merge? i'm not supposed to.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs.

can someone squash/merge? i'm not supposed to.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs.

@tschanfeld I will merge. But in general you can feel free to "Squash and merge" once a PR is approved and it is ready to be merged.


⚠ Processors has degraded performance

In c041af1, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in 0229880.


Multiple labels & service pick&ship

Hello, I should have the following question. I send a PrepareLabels () request to the GLS, and then when the payment passes, I send GetPrintedLabels () to download the labels. The problem is when the customer chooses the pick & ship service on one package and not on the other, or when he wants to send more packages with the pick & ship service. The request with this service is not stored in gls as prepare and only one label response is returned. with the pick & ship service and other labels will not be printed. Am I making a mistake somewhere?

Hello!
I think it is a GLS bug, in case you send multiple ID-s with GetPrintedLabels request and one of them is a pick&ship parcel, it will return only the pick&ship label (receipt). If you make it in separate calls, it will return with the correct pdf.
I will report it to GLS.


[Selection] Reimplement SelectableText with SelectionArea

SelectableText is currently a read only EditableText which has several issues, such as the selection is bounded in a single selectable text, and it may be costly in terms of memory usage.
The idea is reimplement SelectableText with SelectionArea + text, but we should make sure they are feature on par before we do that.
several issue with SelectionArea:
https://github.com/flutter/flutter/issues/111370
https://github.com/flutter/flutter/issues/111021 （fixed）
https://github.com/flutter/flutter/issues/110594 (won't fix)
https://github.com/flutter/flutter/issues/104703 (fixed)
https://github.com/flutter/flutter/issues/104603 (fixed)
https://github.com/flutter/flutter/issues/104594
https://github.com/flutter/flutter/issues/126652 (fixed)

Hey @chunhtai. Thanks for filing this issue. This looks like technical debt. Marking as such

Instead of deprecating, maybe we should reimplement it with the new selection stack. It can still be useful for apps that only want to make certain pieces selectable.

Wouldn't that make it redundant? If you do want to make certain pieces selectable, you can wrap those pieces in a SelectionArea, can't you?

@goderbauer
Is there a reason that selectable_text_test.dart is under widgets while selectable_text.dart is under material?
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/selectable_text.dart
https://github.com/flutter/flutter/blob/master/packages/flutter/test/widgets/selectable_text_test.dart

Is there a reason that selectable_text_test.dart is under widgets while selectable_text.dart is under material?

Looks like an oversight. The test should be under the same directory as the implementation.

All the mentioned issues have now been fixed or completed.

Is there an update on this? At the moment it is impossible for me to select multiple lines with setting selectable: true. Would be great if this would be possible by using SelectionArea. Thank you for your great work!


feat: add drools-lsp support

This PR adds support for Red Hat's Drools Rule Language (ft=drools, ext=.drl). The drools-lsp server is still under development, but it does have basic working capabilities, and I have successfully tested it (with this PR) in a minimal/vanilla neovim init config. Having this server configuration in nvim-lspconfig would enable developers to more easily develope DRL, and furthering the development and testing of the drools-lsp server itself.
Note: As soon as this PR gets merged, I will be submitting PRs to the mason.nvim repo (diff) and the mason-lspconfig repo (diff) - however this PR needs to get merged fist.
Thank you so much!
Full disclosure: I work for Red Hat on the Drools team itself. :)

Do I have to squash the commits so that there is a single commit comment that has the expected commit message (like my first one had)? Otherwise I don't know why the commit_lint is failing...

because the other commit not match the commit head like fix: feat: docs etc. it doesn't matter .but if you can squash then that's very nice.

Squashed.

@glepnir I'm sorry, at this point aren't these just programming style choices? I feel like the way I have it is more readable. Your recent suggestions might make for less lines of codes - but not many - and it doesn't really change anything. The previous things I fixed were indeed real fixes, but these latest suggestions really don't seem necessary. Again, just a matter of programming style. Will this PR not get merged unless I make these changes? Thanks again for your time.

Not all. It is indeed readable but the code is redundant. And calling a function frequently. Although insignificant in terms of time. But Lua will look for insert in the global table. This process will have a hash lookup. We have built-in functions vim.list_extend to help you simplify the process.
You insert after initializing the table. This is very strange. You can put the value in the table at initialization instead of calling the insert function
        config.cmd = {}
        table.insert(config.cmd, java_bin)

@glepnir Okay I've made your changes.

LGTM then. did you tested ? if works fine I will merge then.

Yes, I re-tested, and with a few different config variations to be safe.

oh nice. thanks for your contribution :)

Thank you!


Add commandline options and shorthands

Allow the use of:

--shell [shell] / -s [shell]
--distro [distro] / -d [distro]
--login / -l
etc

This is a follow-up to https://github.com/arkane-systems/RunInGenie/pull/5#issuecomment-782603747

I've been thinking about this a little in the interim.
It's seeming to me that there are some downsides to allowing arbitrary options, insofar as there's no rule in Linux preventing options, etc. from having filenames starting with - or --, meaning there's a chance and a greater one with every new options that it's going to cause a name collision with something for someone somewhere.
It also requires options whose parameters, if any, are all strictly defined enough that there won't be ambiguity between what's an option and what's a command.
I'm thinking it may be better to make one breaking change now to head off future problems. Specifically, introduce a mandatory -- separator now when options are supplied; if found in the argument list, everything before it gets parsed as options to RunInGenie, and everything after it gets passed as the inside-WSL command line. If not found in the argument list, everything gets passed as the inside-WSL command-line as is, and RunInGenie doesn't try to parse it for options.
Not sure what to do with -h/--help in this model, since people aren't going to understand to use -h -- unless they've already read the -h output. Must think on this more.


Why GP app not aviliable in my country?

I want try your app in Russia, but i can't do it. Why?

I made this app as an experiment and was just testing it around with friends.
I've released it and opened it up to other countries, it should be available in the Play store soon.
I don't image I'll update it much though, feel free to fork this repo and play around with it, or install the app from the store at this link: https://play.google.com/store/apps/details?id=com.danielcswain.nearbychat
Cheers


Fixed swissranger dependency

Now swissranger_camera will not be compiled if the swiss_ranger driver does not exist

I just tried to install the master branch on a clean machine, I am receiving a cmake error now; I reran the ./openptrack_install.sh script (3) times:
CMake Error at         /home/remap/workspace/ros/catkin/devel/share/swissranger_camera/cmake/swissranger_cameraConfig.cmake:141 (message):
Project 'opt_calibration' tried to find library 'swissranger_utility'.  The
library is neither a target nor built/installed properly.  Did you compile
project 'swissranger_camera'? Did you find_package() it before the
subdirectory containing its code is included?
Call Stack (most recent call first):
/opt/ros/indigo/share/catkin/cmake/catkinConfig.cmake:75 (find_package)
open_ptrack/opt_calibration/CMakeLists.txt:8 (find_package)
-- Configuring incomplete, errors occurred!
See also "/home/remap/workspace/ros/catkin/build/CMakeFiles/CMakeOutput.log".
See also "/home/remap/workspace/ros/catkin/build/CMakeFiles/CMakeError.log".
Invoking "cmake" failed

The problem should be fixed with #51


Add pseudo_level_type to entries in STASH_TRANS

This is a bit speculative - I'm very happy to take comments and revise or drop.  Thanks to @marqh for helping put this one together. (@marqh I'm working from home so can not see if you have emailed me on this today - but the sparql query appears to be working
The pull request adds the pseudo level type to the information recorded against stash code in the STASH_TRANS dictionary.  The motivation for doing this is to allow applications to understand what the pseudo levels mean for a particular stash code.  This implementation is pretty minimalist - it does not attempt to assign the pseudo level type to any part of the cube or its pseudo level coordinate meta data.
The particular case that prompted this was a callback that needed to take action depending on the pseudo level type - in order to help interpret what the pseudo level values mean.
I'm a bit unsure of how the production of '_ff_cross_references' is managed.  I don't think its part of the setup.py like standard names as the resulting file is revision controlled in the repository.  So I'm guessing you just run it every now and again when you want to make changes to the content of STASH_TRANS. Sorry if I have this wrong.

Hello, I've added a gist to demonstrate a use of this pull request: https://gist.github.com/jkettleb/9ad83aba80dac288c377       I hope that gives a little more context.
any comments on this pull request?
Thanks, Jamie

thank you for the input @jkettleb
Adding the pseudo_level type clearly has application for your case, I can see your use in the example gist
I expect that we would like to see some richer pseudo level comprehension over time, but this is a good first step.
the code to collect the cross references looks fine to me.
please may you add a new text file into
https://github.com/jkettleb/iris/tree/pseudoT/docs/iris/src/whatsnew/contributions_1.9
briefly describing the change, for our documentation?
once you've done this, you can 'squash' your commits into 1 and then push the result to this PR (let me know if you're not sure about this step)
then I think this should be ready to commit
cheers
mark

thanks @marqh - I agree this is only a small next step for clearer pseudo level handling - there should be more to follow.
I'll add to that text file and squash.  Its likely to be next week now though.

:+1: with @marqh's suggested change.

Thanks @jkettleb :+1:


Tests: add test for zero candidates

Codecov Report
Patch coverage has no change and project coverage change: +0.44 :tada:

Comparison is base (d57099c) 82.12% compared to head (0ef79b7) 82.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
+ Coverage   82.12%   82.56%   +0.44%
==========================================
  Files          22       22
  Lines         895      895
  Branches      211      211
==========================================
+ Hits          735      739       +4
+ Misses        139      137       -2
+ Partials       21       19       -2

Impacted Files
Coverage Δ

src/pymovements/utils/filters.py
100.00% <0.00%> (+12.90%)
: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.


MGDOBR-990: Displaying smart event instance details, while the drawer is open

for more details:
https://issues.redhat.com/browse/MGDOBR-990

I have tried manually that the changes work. I am approving. However it would be good to add some unit test probably.

@jomarko InstanceListPage is a "container" component dealing with APIs, I would keep it covered only with cypress tests. We could add a test there to check that the drawer content is properly refreshed over time for example.
But sure we should improve unit test coverage too. The InstanceDetails component for example lacks unit tests completely.

@kelvah thank you, I have created a separate ticket for cypress test https://issues.redhat.com/browse/MGDOBR-1087


knife upload doesnt work because  method 'make_child_entry' is missing

Version:
in rpm  chefdk-0.9.0-1.el6.x86_6 at gem : chef-12.5.1
[What you are trying to achieve and you can't?]
I am trying to upload resources from chef_repo_path to chef server.
Steps to Reproduce:
I am trying to upload resources from my chef_repo_path to chef server using command:
knife upload / -c /root/.chef/knife-jenkins.rb -VV
Expected Result:
It should not return a failure message.
Actual Result:
Command succeeds but returns failure message with error. :
=========================================
ERROR: knife encountered an unexpected error
This may be a bug in the 'upload' knife command or plugin
Please collect the output of this command with the -VV option before filing a bug report.
Exception: NoMethodError: undefined method `make_child_entry' for #Chef::ChefFS::FileSystem::CookbookSubdir:0x0000000415d958
Verbose output is available in attachment :
ErrorDuringKnifeUpload.txt
========================================

dup of #4070