feat: added name wallet functionality to settings and connection screen

Summary of Changes

Added the ability to change the wallet name from the settings screen
Added the ability to change the wallet name from the connection screen
Added functionality to update the agent label without restarting

https://github.com/hyperledger/aries-mobile-agent-react-native/assets/36937407/c8fe992f-4713-4dbf-bcec-d381302b6e7a
Related Issues
N/A
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

What happens if you change your wallet name after connecting? Does the label at any point get updated in the other wallet's contacts?

What happens if you change your wallet name after connecting? Does the label at any point get updated in the other wallet's contacts?

No, the connection label stays the same. Changing the wallet name will only change the label for future connections

No, the connection label stays the same. Changing the wallet name will only change the label for future connections

Ok, that's fine imo. Just let Kim know. Also the position of the wallet name in the settings menu looks like it should be aligned to the left like the rest of the headers. Looking at this wireframe: https://xd.adobe.com/view/0900b5b9-22d7-44aa-b9ee-e3693e33a5f3-5c53/screen/bcaa6e11-910e-42dd-96b1-c9b582182594/


Track LNY bonus luck via hunt_details

Restrict event mice check to only those done with event cheese (since Costumed ____ mice are not attracted to the stockpile's normal bait)
Track bonus luck from active lantern as a hunt detail

Some corner cases to consider:

when the lantern reaches 500, does the height keep increasing? Or is it fixed at 500? If it's fixed at 500, we'll need a way to determine if we should use luck = 50 or 49.
should we ignore all LNY hunts where the lantern is inactive and the count of unlit items is 0, i.e. "we just ran out of candles and that's why the lantern is inactive"
There's no way to know how much bonus luck was applied if we consume the final candle, unless we obtain a valid pre-hunt snapshot and use that instead of the post-hunt response.

Running out of candles generates a journal entry:

In journal_markup:
{
  "render_data":{
    "image":[],
    "entry_id":340169,
    "mouse_type":false,
    "css_class":"entry short misc custom chesla_trap_trigger",
    "entry_date":"9:07 am",
    "environment":"Cantera Quarry",
    "social_link_data":{
      ...
    },
    "entry_timestamp":1549638440,
    "text":"I have run out of <b>Lunar Lantern Candles<\/b>! I will no longer benefit from the magic of the Pig Lunar Lantern!"
  },
  "publish_data":{
    "type":"journal",
    "params":{
      "user_id":#######,
      "journal_id":340169
    },
    "quote":null,
    "params_string":"user_id=#######&journal_id=340169"
  },
}


feat: #681 Stateful tabs

@monsieurtanuki, I think I need some guidance here, so the tabs are now stateful, they kept rendered and get only hidden by the Offstage widget, but there is one problem, keeping the tabs rendered also keeps the camera open, this is of course not wanted behaviour but I am unsure how to get around this while keepin it stateful

Hi @M123-dev!
The root problem comes from the fact that there were (and maybe still are) 3 Widgets (profile, scan, history) created at the same time, with the tabs just deciding which one to display.
Which is problematic with one Widget (scan) that needs the camera on, even when not displayed.
My quick suggestion:

consider each tab as an actual page (Scaffold), and not a Widget
don't create the pages at the same time - there's just one current page at a time
if you leave the scan page, perhaps it's not enough to push a new page, camera-wise, because flutter may say "it's just the previous page, I keep the camera on, just in case we go back". Therefore you could either explicitly close the camera when you leave the scan page, or pop the scan page first then push the new page (something like Navigator.popAndPush probably exist)

I hope this helps.

Thanks for your answer @monsieurtanuki I decided to sticke with with closing the camera when I exit (other tab or continue navigating in the scanner) since poping and then pushing a new page wouldn't keep the state nor the next opend pages.
For the remaining points, we now do create all three pages at once where there is probably no good way around, but because of WillPopScope the tabs feel as if you have pushed them to the stack, as the back button switches back to the last tab.

@gspencergoog
@Piinks thank you for your hint on the AutomaticKeepAliveClientMixin I have also tested this out but as you have already suspected this unfortunately did not work with a own navigator.

PullRequest summary:

The navigation bar is now statefull, which means that it keeps not only things like the scroll position but also new opened pages.
If you push to another page and then press the tab you are currently on you will be redirected back to the inital tab page.
Because of WillPopScope the back buttons also work to get back to the inital tab.
The scanner/camera gets deactivated when you switch to another tab or open any page (Search, Product, Ranking, ...) directly from the scanner.

Hello @monsieurtanuki, thanks for your detailed review.
I don't know if I follow you on the onPause/onResume methods. And to the RouteObserver, this basically has nothing to do directly with the StatefullTabs but fits to this PR. It fixes #640, since the camera doesn't go out when you push a new page over it. Maybe we could save a few lines there by just rebuilding the scanpage when poping to it and not looking first if we could just reactivate the camera but for maybe 5 lines that's not worth it. I'll try to add a few more comments there maybe this makes it clearer.

Hi @M123-dev!
onPause/onResume is a reference to Android java dev: those are callbacks that you typically use when the page appears/disappears and you want to reload/save data (cf. incoming phone call). That is similar to your camera on/off issue: we need the camera on if and only if the scan page is displayed.
About putting in the same PR a code that solves several issues, I'm not against that in general, but here we're talking about a PR where you started stating "I think I need some guidance here". Currently we already exchanged 33 posts for this PR, and you committed 11 times. If you can implement in this PR only the "stateful tabs" part (even with a stupid camera always on), and then in a next PR the camera on/off system, I'd be grateful, as a code reviewer.

@monsieurtanuki I have done a bit of simplification, and documentation, I probably can't get this PR down to much less code

@M123-dev I'm currently in the hive code - I'll be able to review your code tomorrow at best.

No problem @monsieurtanuki, I don't want to distract you from it, the PR's shouldn't conflict each other either.


Get feedback from trusted external EAs

@michaltrzesimiech Something for you to think about when to do this and who to ask

No need to make a card for this as it isn't tech-related


Add ability to use replied message's content as .randomcase argument.

Closes #1074.
Description

As per title.
Users can now reply to a message to its contents as the text argument, thanks to a call to botcore.utils.commands.clean_text_or_reply.

Did you:

[x] Join the Python Discord Community?
[x] Read all the comments in this template?
[x] Ensure there is an issue open, or link relevant discord discussions?
[x] Read and agree to the contributing guidelines?

I think it would be nice if this supported randomcaseing replies to embeds, given this command already supports that through message links, although I don't really mind.

This isn't really possible to implement without making the code very message, since clean_text_or_reply doesn't have this feature. Users will at least still have the option of doing via message links (and said message link could be in the replied message).


Updated screenshots and text about edge browser

Current edge browser seems to not support service worker well.

It may makes students who learn PWA based on Visual Studio.
The update is under the following environment.
Microsoft Edge Beta Channel 79.0.309.54

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: flushpot1125 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Docs Build status updates of commit 533a2d5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps/get-started.md
:warning:Warning

Details

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-cache.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-sw-overview.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-offline-html.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/get-started.md

Line 14: [Warning] Link 'https://docs.microsoft.com/en-us/windows/uwp/get-started/whats-a-uwp' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 32: [Warning] Link 'https://docs.microsoft.com/en-us/visualstudio/nodejs/tutorial-nodejs' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 117: [Warning] Link 'https://docs.microsoft.com/en-us/legal/windows/agreements/store-policies' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://docs.microsoft.com/en-us/visualstudio/nodejs/tutorial-nodejs#optional-publish-to-azure-app-service' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://azure.microsoft.com/en-us/free/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://azure.microsoft.com/en-us/services/app-service/web/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 775921f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps/get-started.md
:warning:Warning

Details

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-cache.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-sw-overview.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-offline-html.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/get-started.md

Line 32: [Warning] Link 'https://docs.microsoft.com/en-us/visualstudio/nodejs/tutorial-nodejs' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 117: [Warning] Link 'https://docs.microsoft.com/en-us/legal/windows/agreements/store-policies' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://docs.microsoft.com/en-us/visualstudio/nodejs/tutorial-nodejs#optional-publish-to-azure-app-service' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://azure.microsoft.com/en-us/free/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 123: [Warning] Link 'https://azure.microsoft.com/en-us/services/app-service/web/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d67ac54:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps/get-started.md
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-cache.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-devtools-sw-overview.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps/media/chronium-based-edge-offline-html.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit ad9c18f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-edge/devtools-protocol/0.2/domains/css.md
:warning:Warning

Details

microsoft-edge/progressive-web-apps-chromium/images/chronium-based-edge-devtools-cache.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps-chromium/images/chronium-based-edge-devtools-sw-overview.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps-chromium/images/chronium-based-edge-offline-html.png
:white_check_mark:Succeeded

microsoft-edge/progressive-web-apps-edgehtml/get-started.md
:white_check_mark:Succeeded

microsoft-edge/devtools-protocol/0.2/domains/css.md

Line 449: [Warning] Cannot find bookmark '#stylesheetorigin' in 'devtools-protocol/0.2/domains/css.md', did you mean '#stylesheetid'?

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@flushpot1125
Thank you for taking the time to submit these changes.
Your suggestion has been evaluated and may be implemented in future.


Fix/use browserify over esbuild for e2e tests

Esbuild stopped working due to the watch command being removed in
v17.0.0. In theory we could roll back to v16.5.0 but this didn’t
work locally. More details here: https://github.com/badeball/cypress-cucumber-preprocessor/issues/929.
Previously we’ve had to add polyfills for certain Node APIs for esbuild too
(see https://github.com/ministryofjustice/hmpps-approved-premises-ui/pull/515).
Due to these two issues and the fact we’re not using esbuild anywhere
else in the codebase I think it’s best we use browserify for the time
being instead as it seems to work better ‘out of the box’.

Yes, that's right. If you're building a big app the difference can be huge but as we were just using it for the E2Es it's a few seconds... We can always revert once things are more stable but I think the costs outweight the benefits for now


Validate file/owner comments even if there are no rules

Codecov Report
Attention: Patch coverage is 97.70115% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.07%. Comparing base (0979306) to head (6d1c163).

Files with missing lines
Patch %
Lines

internal/comments/comments.go
71.42%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1223   +/-   ##
=======================================
  Coverage   95.07%   95.07%
=======================================
  Files         105      105
  Lines       12587    12637   +50
=======================================
+ Hits        11967    12015   +48
- Misses        471      473    +2
  Partials      149      149

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


Media armor causing crash

Media armor will sometimes cause my client to crash when it breaks.
crash-2024-07-02_04.36.35-server.txt

I can't seem to reproduce this, can you elaborate on "sometimes"?
I tried both with and without Fabrication (as well as in a Quilt instance), since the crash report refers to a method in one of its mixins, and I enabled the feature that the mixin is associated with, and couldn't get it to crash.

It seems like the HexxyCraft++ server is also getting then. The problem happens when someone takes off the armor, and the durability is set to -2147467263. This does let the armor be worn again, refreshing its durability to 16384 (or whatever its max is probably). Luckily, Neruina catches it before it crashes HexxyCraft++


component not found: climate.mi_acpartner and climate.xiaomi_miio

Home Assistant release with the issue:
Home Assistant 0.74.2
Last working Home Assistant release (if known):
I don't know.  0.74.2 is my first version.
Operating environment (Hass.io/Docker/Windows/etc.):
Raspberry Pi 3B +, Chinese Mossbian System(Based on Hassbian.)
Component/platform:
mi_acparter_v2 use platform "climate.xiaomi_miio" and "climate.mi_acpartner"
Description of problem:
Unable to find component climate.xiaomi_miio
下午10:45 loader.py (ERROR)
Unable to find component climate.mi_acpartner
下午10:45 loader.py (ERROR)
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
configuration.yaml
climate: !include devices/climates.yaml

devices/climates.yaml
- platform: mi_acpartner
  name: mi_acpartner1
  host: <IP_ADDRESS>
  token: mysecret_token_not_display_here
- platform: xiaomi_miio
  name: mi_acpartner2
  host: <IP_ADDRESS>
  token: mysecret_token_not_display_here
  target_sensor: sensor.temperature_158d0001f53706
  scan_interval: 60

Traceback (if applicable):
2018-08-09 22:45:55 ERROR (MainThread) [homeassistant.loader] Unable to find component climate.mi_acpartner
2018-08-09 22:45:55 ERROR (MainThread) [homeassistant.loader] Unable to find component climate.xiaomi_miio

Additional information:
I am a developer but new guy here about homeassistant, how can I solve this problem?

They are not home-assistant supported, please go to https://www.mossbian.com for support.

Sorry, but I don't see anything usable at mossbian.com. @steventong were you able to figure out what led to the issue or how you resolved it?

@sramanujan climate.mi_acpartner and climate.xiaomi_miio are custom components. you can take a look at https://github.com/mac-zhou/homeassistant-mi-acpartner


DOC: TLS compression is disabled by default

Checklist

[X] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Pushed to master, openssl-3.0 and OpenSSL_1_1_1-stable.


Feature: Self-Eviction

This feature adds an eviction sequence to ringpop to evict a member from the membership prior to shutting down. This can be used to prevent the suspicion period to kick in during rolling restarts, upgrades and downscaling of the number of running applications. Ringpop modules can hook into this system by implementing the SelfEvictHooks interface in swim and register the hook in Ringpop.

Coverage decreased (-0.5%) to 94.368% when pulling 9d8126aeee96c97d80888a4eaaa53209c6d39dc9 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.5%) to 94.417% when pulling d7751840c5d8c9a1427845bc247364f55f39ebd7 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.5%) to 94.417% when pulling d7751840c5d8c9a1427845bc247364f55f39ebd7 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.5%) to 94.417% when pulling d7751840c5d8c9a1427845bc247364f55f39ebd7 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.4%) to 94.522% when pulling 75b0eb603d8a6d7a704183e543c9b8b2e660a1d4 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.4%) to 94.522% when pulling 75b0eb603d8a6d7a704183e543c9b8b2e660a1d4 on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.2%) to 94.747% when pulling 1d6296624f8242567bdd433062a951ecd8b48d1c on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.

Coverage decreased (-0.2%) to 94.747% when pulling 1d6296624f8242567bdd433062a951ecd8b48d1c on feature/self-evict into ff0a2ac364a92622713e91eaf73aa79b684292bf on dev.


website: Fix broken LSP link

Motivation

Fixes #3189
Test plan

See included automated tests.

👍 I didn't realize that it was acceptable to write [text][url], is that docusaurus specific?

Nope, that's how markdown links work. These all work:
1. [foo]

2. [foo][foo]

3. [foo](https://example.com)

[foo]: https://example.com

foo

foo

foo


How do I print

When printing the page the slider is invisible. How to solve? Any clue?

Because web browsers does not print background-color (the green line is represented by background-color). So this is wont fix.
You may want to specify --webkit-print-color-adjust


No an issue but a question: How I can combine 978 and 1090 traffic in one ?

Hi,
I know there is a separate data extract for dump978 and 1090...using either dump1090/data/aircraft.json or dump978/data/aircraft.json.
My question is: how  can I have both data traffic in the same json? or how can I send the 978 traffic over the dump1090/data/aircraft.json so all 978 and 1090 traffic can be seen in just one http link ??
Thanks,

never mind...I think a got my answer,,,

dump978 data is fed to dump1090 so dump1090's aircraft.json already contains aircraft from both.

thanks...


VScode extension does not work

Describe the Bug
Installed and put a correct API key, no other changes, it just does not work
Please tell us if you have customized any of the extension settings or whether you are using the defaults.
I did not change the default settings, so gpt-3.5-turbo was used
Additional context

sorry, the issue with api key


Inconsistency in footer layout compare to other page footer layout on layer5

Description

On service-mesh-pattern website the footer layout is not consistent with other pages footer layout on layer5.
https://service-mesh-patterns.github.io/service-mesh-patterns/
Behavior

Expected Behavior

Environment:

Meshery Version:
Kubernetes Version:
Host OS:
Browser:

Assigned @Cvr421. All yours. Please also address this in this issue - https://layer5io.slack.com/archives/C015QJKUMPU/p1689864226229019?thread_ts=1689781477.372719&cid=C015QJKUMPU

Assigned @Cvr421. All yours. Please also address this in this issue - https://layer5io.slack.com/archives/C015QJKUMPU/p1689864226229019?thread_ts=1689781477.372719&cid=C015QJKUMPU
ok thanks @Chadha93

Hey @Chadha93 can i Work on this issue

Please see - https://github.com/service-mesh-patterns/service-mesh-patterns/pull/114#issuecomment-1646624448


⚠ Bofrost.de has degraded performance

In 3516da2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1423 ms

Resolved: Bofrost.de performance has improved in 4e4909b after 6 minutes.


Multiple errors 400 - The conditional request failed on Session Provider

Hi, this is a question I posted on StackOverflow last week and it seems like a bug. I'm sorry but I was not able to get any solid repro, but just some evidence regarding of it's existance. Here is the full question with minor edits.

We use AWS's DynamoDB Session Provider in our app to store session data.
I recently moved to an environment where I can have NewRelic monitoring my app and it started throwing alerts regarding Dynamo DB access. However, NewRelic is the only monitoring tool that is getting it. I cannot see anything related to this problem in my application logging (log4net) or the Windows event viewer.
I searched a lot and even went through the source code of the provider but came out empty.
I'm getting (400) Bad Request from what is seems to be all the calls made during a period of 1 or 2 minutes at a time happening 3 or 4 times per hour.
The stacktrace I could get is not promising:
at System.Net.HttpWebRequest.GetResponse()
at System.Net.HttpWebRequest.GetResponse()
at Amazon.Runtime.AmazonWebServiceClient.getResponseCallback(IAsyncResult result)

And the offending URL is:
dynamodb.us-east-1.amazonaws.com/Stream/GetResponse

By using Fiddler during a low usage period in the weekend on the production server I was able to capture some bad requests that were throw. I'm getting "The conditional request failed" which seems to happen because the value was updated while requesting and old value and therefore the value is not consistent to what was expected. Below is a full request/response as a sample.
Request:
POST https://dynamodb.us-east-1.amazonaws.com/ HTTP/1.1
X-Amz-Target: DynamoDB_20120810.UpdateItem
Content-Type: application/x-amz-json-1.0
User-Agent: aws-sdk-dotnet-35/<IP_ADDRESS> .NET Runtime/4.0 .NET Framework/4.0 OS/6.2.9200.0 SessionStateProvider TableSync
Host: dynamodb.us-east-1.amazonaws.com
X-Amz-Date: 20140510T153947Z
X-Amz-Content-SHA256: e7a4886acac6ccf16f0da9be962d3a68bd50e381c202277033d0d2bb3208aa8a
Authorization: AWS4-HMAC-SHA256 Credential=redacted/20140510/us-east-1/dynamodb/aws4_request, SignedHeaders=content-type;host;user-agent;x-amz-content-sha256;x-amz-date;x-amz-target, Signature=redacted
Accept: application/json
X-NewRelic-ID: redacted
X-NewRelic-Transaction: redacted
Content-Length: 399

{
    "TableName": "ASP.NET_SessionState",
    "Key": {
        "SessionId": {
            "S": "redacted"
        }
    },
    "AttributeUpdates": {
        "LockId": {
            "Value": {
                "S": "42a9ed29-7a92-4455-8733-2f56c7d974b3"
            },
            "Action": "PUT"
        },
        "Locked": {
            "Value": {
                "N": "1"
            },
            "Action": "PUT"
        },
        "LockDate": {
            "Value": {
                "S": "2014-05-10T15:39:47.324Z"
            },
            "Action": "PUT"
        }
    },
    "Expected": {
        "Locked": {
            "Value": {
                "N": "0"
            },
            "Exists": true
        }
    },
    "ReturnValues": "ALL_NEW"
}

Response:
HTTP/1.1 400 Bad Request
x-amzn-RequestId: redacted
x-amz-crc32: redacted
Content-Type: application/x-amz-json-1.0
Content-Length: 120
Date: Sat, 10 May 2014 15:33:17 GMT

{
    "__type": "com.amazonaws.dynamodb.v20120810#ConditionalCheckFailedException",
    "message": "The conditional request failed"
}

Graph 1

Graph 2

Any help is appreciated. Thanks!

Is it worth adding a Timer that will say every 30 minutes call the DeleteExpiredSessions static method?

That is an interesting thought. We would need it to be an opt in feature so that no one would be surprised about all the provision throughput being used up. Another thing that needs to be considered is if this is running in an environment with multiple instances we would probably not want this full table scan to run on every instance.


Conda env create fails on Windows/Mac

Hey,
conda can't solve the dependencies for non-linux platforms. This is because the packages in reinvent-shared.yml contain build strings.
The easiest fix is to use the --no-build flag when exporting the environment.
However, for long-term support, it might be beneficial to create minimal reinvent-minimal.yml, which only contains the explicitly needed dependencies and not the dependencies of our dependencies.
\Thomas

Hi,
Good point, although I'm not sure why would anyone run it under windows. Development maybe?
Anyway, I accepted the minimal environment but I fear there might be still packages missing. XGboost for instance. We had to reduce the code for the public release since there are features irrelevant to the broad audience and there are features that will be released later on. That is why there are so many packages in the environment. Likely some of those will be needed for future releases (OpenEye library for eg.).
Best,
Atanas


Feature request: Time

It would be nice to be able to also input time.

I thought to implement this feature after got suggestion from few other guys earlier. But my concern is i am using a popup for this component. If i make the text boxes to enter values, again the users will get another popup(key board). Does this look good? What do you suggest?

You could use the html5 input type 'time'. This would give you a native popup for the platform to enter the time I think. This prevents the keyboard from coming up.
Other options would be buttons you can touch to increase/decrease the hours/(tens of) minutes (so people dont have to tick 30 times to get to 30 minutes). example: http://plnkr.co/edit/QujX5A?p=preview
Or else perhaps a dropdown, like this: http://adamalbrecht.github.io/q-date/
I think the favoured solution would be either the first or the second, but that would depend on wheter or not the keyboard comes up. I agree with you on the fact that the keyboard popping up would not be the best thing that can happen.

May be its good have a drop down or 5 or 10 minute increment as you said. You can expect this functionality in the next release. Thanks for your suggestion. They really make some sense.

The drop down functionality is implemented. You can use the latest version to have drop downs. date is not supported in all the browsers. So i haven't implemented such feature.


Move panel topics to how to guides

What this PR does / why we need it:
#961 was getting a bit messy so started afresh with the proposed changes
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This is identical in intent to #961 except for the following changes:

doesn't rename custom panel options file (simpler)
uses "Read data from a data source" rather than "Read data frames" - this is up for debate, but wanted to show an alternative which led with the desired outcome rather than the mechanism
the error handling for panels document is identical to the one in extend a plugin as the content is all about panel plugins. We could remove the original or keep until we have ones per plugin type which are actually relevant.

When I say identical in intent that means I have changed other things as well:

fixed issues with dataframes and other files being deleted
fixed index files for how-to and panel-plugins to be relevant
fixed build issue from redirects

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for giving your attention to these issues. I think this looks good.


Is the contact email address active?

Which Cloudflare product does this pertain to?
Queues
Existing documentation URL(s)
https://developers.cloudflare.com/queues/platform/javascript-apis/
What changes are you suggesting?
According to the docs<EMAIL_ADDRESS>is a contact email address for Queues:

If you have any feedback about these APIs, please contact us and we would be happy to hear from you.

however, I've got a delivery error. Is this address active?

Delivery Status Notification (Failure) Address not found
Your message wasn't delivered to<EMAIL_ADDRESS>because the address couldn't be found, or is unable to receive mail.

FYI, my email address domain is @gmail.com.
Additional information
No response

Hello @moutend! Thank you for reporting this. I'll look into this with the team.

Instead of using the Queues email address, we recommend you to use the Discord channel and/or Community forum.
https://developers.cloudflare.com/queues/platform/javascript-apis/


Create Role, RoleBinding, ClusterRole, ClusterRoleBinding health check

What this PR does / why we need it:
Create Role, RoleBinding, ClusterRole, ClusterRoleBinding health check
Which issue(s) this PR fixes:
Fixes #1136
Does this PR introduce a user-facing change?:

NONE

I checked the definitions of structures of Role, but I couldn't figure out how to specify the criteria for determining healthy.
I felt that as long as the role is applied successfully, it will not become unhealthy in the middle of the process, so I implemented functions in this PR so that it always returns healthy.
The same goes for Rolebinding, ClusterRole, and ClusterRoleBinding.

I checked the definitions of structures of Role, but I couldn't figure out how to specify the criteria for determining healthy.
I felt that as long as the role is applied successfully, it will not become unhealthy in the middle of the process, so I implemented functions in this PR so that it always returns healthy.
The same goes for Rolebinding, ClusterRole, and ClusterRoleBinding.

@sanposhiho Thank you for your investigation! To be honest we've put off that check the whole time. As you mentioned, it looks like never getting unhealthy once applied. Let's deal with it later if we have to finely control for in the future 👍
/lgtm

@sanposhiho Thank you for your investigation! To be honest we've put off that check the whole time. As you mentioned, it looks like never getting unhealthy once applied. Let's deal with it later if we have to finely control for in the future 👍
/lgtm

thank you
/approve

thank you
/approve


code for running the rules

As far as I can tell, the code for running thru the rules and generate the json data files is not checked in. is that right? If someone clarifies that, I can stop looking for it.
I can tell snorkel.label_function is used. but the chemprot data has the chemical and genes highlighted with [entity_type] annotations, whereas, the other datasets, such as AGNEWS, do not.
My question is how are the type annotation generated?

The author answered that he used other scripts to do that. the issue is closed.


error page added

Fixes Issue
close  #34
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

It's giving error that the import route for ErrorPage doesn't exist, since you have changed the path of ErrorPage.js.

Create a new folder always for a new page / reusable component.
For example here, create a folder ErrorPage and have your files with the names ErrorPage.js and ErrorPage.css.
Once you raise a new PR, make sure your changes are working on the Netlify deploy-preview and if it fails, then check the deploy log and try resolving the issues present there.

oh i forget to commit app.js file

check now

Create a new folder always for a new page / reusable component.
For example here, create a folder ErrorPage and have your files with the names ErrorPage.js and ErrorPage.css.

This remains.

check again


Add update dataTree as an evalWorkerAction.

Done with https://github.com/appsmithorg/appsmith/pull/19756


NumberBox: Change StepFrequency to SmallChange and LargeChange

We got feedback that the "StepFrequency" name was confusing and also that folks wanted support for PageUp/PageDown to make large changes. This change adds SmallChange and LargeChange properties and gets rid of StepFrequency.

Should we ensure that largeChange > smallChange

Should we ensure that largeChange > smallChange

+@SavoySchuler for visibility but I would say no. In general i don't like enforcing things that don't strictly need to be enforced. Throwing exceptions is rude and coercing values in this case seems unnecessary.


Prefixed string/char literals

Unicode is good, unicode is great, but it has a few undesirable pitfalls sometimes:

Byte strings: low-level C users know what I'm talking about, a Nit Text object is necessarily Unicode, which leaves the guy wanting to use a string as a byte sequence sadly coping with invalid byte sequences (i.e. cannot do shellcoding in nit without expressing all, including text, in a hexdigest)
Chars used as their value as explained in #1718

For these uses, no real easy solution exists now in Nit.
As such, this issue proposes the introduction of prefixed strings and chars.
It will first be recognized grammatically, and some prefixes will be implemented in the compiler later.
Some candidate prefixes on strings are:

b"Byte\xfe" => for a byte string with potentially invalid unicode chars in it.
re"\w*?+" => for Regular Expressions
raw"\n\rEscaped" => For raw strings, where escape sequences are treated as normal characters

Some candidate prefixes on chars are:

a'\n' => ASCII value of char '\n' as a Byte
u'𐏓' => Unicode code-point of char '𐏓' as an Int

This issue is also a feature request for people wishing for more specific or exotic prefixes, this is the place and the moment to express your needs :)

What is the use of the Byte string with invalid Unicode chars?  I would prefer an ASCII string to go with the ASCII Byte.

I'm not sure I understood what you meant by "ASCII string to go with the ASCII Byte", could you clarify ?
Since an 0xFE byte is invalid in a string, the only use for such byte strings is to ease the writing of byte sequences with lots of text in it but with occasional delimiters or non-valid characters.
This is relatively common when dealing with protocols, shellcodes, and such.

About the ASCII string, I would have expected a a"ASCII string" alongside the ASCII byte a'\n'. But it's pretty useless, the literal unicode char with only ASCII characters is the same. So it's OK as proposed.
I had misinterpreted the goal of b"", it looked illogical to write invalid unicode string literals for a unicode parser, but now I see the escape sequences. So what would be the byte string, a Bytes?
The re prefix is great. You should also prepare for additional pre/suffixes such as i for ignore_case, m for newline and maybe something for not extented.

Bytes is a good idea for the return of the byte strings yeah.
The idea of this PR is to submit idea for prefixes and suffixes so the i, m and not extended (whatever the suffix could be, I'll take any input into consideration at the moment) can be added to the list of what to implement :)
I'll do this tomorrow maybe, be ready for the PR then !

I agree the the intention and the need.
Two (detail i.e. devil) things:

What is the real/fundamental difference between re"(a|b)*" and "(a|b)*".to_re? Shouldn't we expect, by the philosophy of POLA, that they are strictly equivalent, thus, shouldn't we assume, by the philosophy of KISS, that one is superfluous?
Should we go with prefixes, suffixes or both? Should we try to be coherent with literal integers that have both, the prefix being the representation and the suffix being the type (eg. 0x7Fu8)?

Following the reasoning, suffixes like u8 for Bytes should be suppressed since we can do 0x4F.to_b :)
I don't think that adding a layer of syntactic sugar is a bad idea for common cases like these, and I do believe that we should add both prefixes and suffixes as it might be useful.

closed by #1851 #1991 and #1992


Block import notifications are not fired for blocks that are imported manually

ImportQueueService::import_blocks calls do not result in either import_notification_stream or every_import_notification_stream notifications.
This seems like a bug that should be fixed, my assumption was that every_import_notification_stream is fired for literally every imported block, no matter the source and whether it is on canonical chain or not.
Originally posted by @nazar-pc in https://github.com/paritytech/substrate/issues/14442#issuecomment-1604233068

Hmm, I checked the code and not sure why there is no notification send. You are sure that the blocks are really imported?
https://github.com/paritytech/substrate/blob/8dc42cf7b27bfa0290d5db847fbeac74ce8d810e/client/service/src/client/client.rs#L822-L830 could you add here some logs verifying that this is really not called when you have registered a listener?

Double checked and you're right, it works as expected.


want to show single page

Horizontal Reading mood...
when updated to new version it doesn't select a single page...
that's why it's annoying ...
is there any way to select a single page...

Duplicate of #70


chore: remove replay protection service

Purpose
Fixes #1740
How does the solution address the problem

Changes made

prnet cleaned-up successfully


Implement ZExt, SExt, Trunc, and Bitcast instructions.

These are needed to support memory loads and stores for non-integer values.
Note that bitcast is different from LLVM's bitcast. It's meant for bitwise casting between floating point and integer values. (We don't have pointers as part of our IR so LLVM's bitcast wouldn't make sense)

I've implemented ZExt, SExt, Trunc. Can you please add more info on bitcasting?

@mishazharov Basically bitcast is a bitwise cast between integers and equivalently-sized floating point types (and the other way as well).
It should have equivalent behaviour to doing
float x;
uint32_t y;
std::memcpy(&y, &x, sizeof(x));

that is to say that it should just move the bits from one type to another.

Some more notes:

The bitcast operation in LLVM is just used to convert between different pointer types (to the best of my knowledge). We only have one pointer type so this doesn't translate to anything on our end. In the interpreter it can probably be implemented as a pass-through.
The bitcast expression is used as part of reading and writing memory. You can see what it's used for by looking at Allocation::read and Allocation::write in MemHeap.cpp.

Feel free to rename the instruction to a clearer name if you think bitcast is too confusing given that LLVM's bitcast has different semantics.

Awesome, thanks. I'll take another look at this when it gets less crazy


Correct Link to Wallet on Website

https://electriccoin.co/wallet-resources/
This page links to the old wallet app.  Once this project is ready, the link should change to this repo.

Blocked temporarily by a decision over whether or not we will be sharing the new wallet's styleguide with the public / opensource questions.

(Additional content request from growth/dev)

Corrected!


Does this have the same issue like the original PushPlugin on phonegap-build?

On the PushPlugin, i.e. https://github.com/phonegap-build/PushPlugin, I had a really strange issue on Android Phones. iOS does not have this issue, android does. The issue is as follows:
I used to call the register() function to register with GCM.
var pageHome = {
    pushNotification: window.plugins.pushNotification,
    initPush: function() {
        pageHome.pushNotification.register(pageHome.successHandler, pageHome.errorHandler, {
            "senderID" : "1234567890",
            "ecb" : "pageHome.onNotificationGCM"
        });
    },
    onNotificationGCM: function(e) {
        switch(e.event) {
            case 'registered':
                // Just a random check for the regid.
                if (e.regid.length > 0) {
                    // Get the pushToken from localStorage.
                    var pushToken = localStorage.getItem("pushToken");
                    // Check if the registration has already happened and whether the already stored pushToken is the same as the new registration id (e.regid) coming as a result of re-registration with the GCM server
                    if(pushToken == null || pushToken != e.regid) {
                        // If the device has NOT registered or the device id has changed then only register again.
                        $.ajax({
                            url: 'https://xyz.com/register',
                            type: 'POST',
                            dataType: 'json',
                            data: {regId : e.regid},
                            success: function(data) {
                                // If stored on my own server, then only save it to localStorage.
                                localStorage.setItem("pushToken", e.regid);
                            },
                            error: function(error) {
                            }
                        });
                    }
                }
                break;

            case 'message':
                navigator.notification.alert(e.payload.message, app.alertDismissed, e.payload.title, 'Dismiss');
                break;
        }
    }
}

The problem I have been facing for long is that more than one android device has the same registration id. I mean the registration id is not unique for different devices. Multiple devices have the same registration id. This is quite weird and I have been facing this issue with that plugin. Will this plugin solve this error? Or is this error due to Google GCM servers? In all my apps I have this issue..

@incorelabs multiple devices should have different registration ID's. I have never heard of this happening before. Please try reproducing it with this plugin but again, this should never happen.

Thank-you @macdonst for looking into it. This problem has plagued all my apps and till date I do not know what I am doing wrong. Coz how can two devices have the same id? Moreover, I've seen 35 devices have the same id. Which was hilarious.
Is anything wrong with the code above? I hope the error is not due to my code. I've written down the code I use to register with GCM and store the received registration id on our third party server. For the "Sender ID" I use the one I have on my Google Cloud Messaging Project and the corresponding API is on our server.

@incorelabs I don't see anything wrong with the code above. Are all the devices the same HW type?

I really doubt, that all devices have the same HW type. We do not keep track of the device details other than the registration id of the device. Apart from the HW type reason could there be any other possible reason why this happen in the first place?

@incorelabs No, I can't think of a reason why this would be the case. Can you reproduce the issue with this plugin? If so can you provide me with the reproduction scenario?

I'll try reproducing the issue with this new plugin and revert back to you with the reproduction. But the reproduction will take time, I'm off work these days. Will get back with the results by mid-August.

Closing, no feedback.


"Local" form fields cannot be accessed (e.g. for validation)

When building an email form, for example, and would like to validate it, there is no way to access the "local" form fields. The fields should be validated before a "Send" button is clicked, by which time the modal should not yet be closed, but instead wait for an $http response. But ng-required has no impact at all, and trying to get something like $scope.emailAddress (a form control inside the modal window that has ng-model="emailAdresse" as attribute) outside of the close button only yields undefined.
Any change you could provide a simple email form example?

I'd also like to know please

There was a bug with form fields in the controller, it is resolved now


remove "Optional OAuth2 security" examples

I wonder why these "Optional OAuth2 security" examples were created in the first place.
I agree that an API can have several options how the API client authenticates but having oauth2 security or alternatively none seems not a good security practice.
e.g. here https://github.com/OAI/OpenAPI-Specification/blob/v3.2.0-dev/versions/3.0.3.md#optional-oauth2-security
If the example was intended to show multiple alternative security requirements or the combination of an oauth2 security object with another non-oauth2 security object, then the second one should not be empty.
Maybe the API is migrating from apiKey to oauth2 and is deprecating api_key.
{
  "security": [
    {
      "api_key": []
    },
    {
      "petstore_auth": [
        "write:pets",
        "read:pets"
      ]
    }
  ]
}

If the example was intended to show multiple alternative security requirements or combining an oauth2 security object with another non-oauth2 security object, then the second one should not be empty.
Maybe the API is migrating from apiKey to oauth2 and is deprecating api_key.

I can understand your interpretation of this example. However, while I do not recall the nature of the example what it wanted to convey originally, what if it the intent was to say "hey, the entire (or some resource of the ) API had no security, and now we can secure it with oauth2" and thus the two entries?    I think really this example is not intended to convey an overall good practice but simply that you have a list of choices you can apply to all or some surfaces of the API. More mechanical than overall good security.

I can understand your interpretation of this example. However, while I do not recall the nature of the example what it wanted to convey originally, what if it the intent was to say "hey, the entire (or some resource of the ) API had no security, and now we can secure it with oauth2" and thus the two entries? I think really this example is not intended to convey an overall good practice but simply that you have a list of choices you can apply to all or some surfaces of the API. More mechanical than overall good security.

So, you are agreeing that "no security" is not a good choice when we want to show multiple security options,  right?!


RemovedInDjango110Warning: OptionParser

Just opening this as a heads up that dbbackup may stop working in Django 1.10 if we don't update this.
/home/mjs7231/.virtualenvs/pk/lib/python3.5/site-packages/django/core/management/base.py:265: RemovedInDjango110Warning: OptionParser usage for Django management commands is deprecated, use ArgumentParser instead
  RemovedInDjango110Warning)

Hello @pkkid
I made a piece of code about that: https://github.com/django-dbbackup/django-dbbackup/blob/master/dbbackup/management/commands/_base.py#L57
optparse is only used with Django < 1.11.

Cool, sorry for the noise.


bug (Revit) Missing structural foundations

Structural foundations with complex outline are not sent. This might be an issue for other types. See more in Notion.
https://www.notion.so/speckle/Missing-structural-foundations-cb4f5f4966004fde920d62d6a1eefe07?pvs=4

Closing duplicated issue, see https://github.com/specklesystems/speckle-sharp/issues/2931


L&S - Red Rook art assets

Another one that's almost done
(glasz edit)
model, texture done, skeleton done, onewalking anim done, needs more anims, needs tools/weapons

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/486253-l-s-red-rook-art-assets?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

Moved to https://github.com/Terasology/LightAndShadow/issues/28.


Communication Ticket with NPC Details

Description
This ticket is mainly for detailed information communication of chapter outlines and play scripts between team 5 and team 9.
Dependencies
Storyline (#64)
Milestones
Discussion of any conflicts and details

[ ] Traitor Info
[ ] Objects
[ ] Collection things inside game
[ ] Clues
[ ] Any other details

Completion Deadline:
Documentation

Storyline
Chapter Script
Traitor Clues
NPC Info
...

Members

JOJO (@ShokuJO) (Guanhao Qiao)
Jingyi Li (@AlynnLi) (Jingyi Li)
Yifei Liu (@liu8868) (Yifei Liu)

Hi @ShokuJO I noticed that you did not put the voice clue and body shape clue in your traitor clues (which you mentioned in Chap2 outline) so we made it up by ourselves. The traitor's voice is deep and the traitor is tall and thin. Let us know it you guys want to change it so we can stick to your clues. Cheers!

Thanks a lot, we will keep your clues and update on the wiki!


Prevent Passcode

Hello,
I'm using your plugin and it works really great.
But I want to prevent the user from entering his passcode if fingerprint scanning failed.
How can I do this ?

From the readme:

If the onSuccess handler was called, you can scan the fingerprint. There are two options: verifyFingerprint and verifyFingerprintWithCustomPasswordFallback. The first method will offer a fallback option called 'enter passcode' which shows the default passcode UI when pressed. The second method will offer a fallback option called 'enter password' (not passcode) which allows you to provide your own password dialog.

I'm sorry I wasn't clear enough,
I don't want to apply any other option than fingerprint scanning
(No password, passcode, pincode , only fingerprint scanning)
Does the function verifyFingerprintWithCustomPasswordFallback does this ?

Yes.

Okay thanks a lot :)

Did u get the solution how to hide enter passcode with this method verifyFingerprintWithCustomPasswordFallback?

@harshvardhan12 yes, use the method "verifyFingerprintWithCustomPasswordFallbackAndEnterPasswordLabel" and set the enter password label to empty string

Thanks @mabdelwahab

Hi @mabdelwahab for iOS ver 9 , if i use "verifyFingerprintWithCustomPasswordFallbackAndEnterPasswordLabel" and set the enter password label to empty string it will come as, the cancel button will align left  and in the right empty String button is there which is clickable. Please help on this regard
Its working for IOS ver 10.

@harshvardhan12 I apologize but I'm a developer like you and I tried all possibilities till it worked with me, so we can ask the creator of the plugin.
@EddyVerbruggen can you help us with the above case ?

@mabdelwahab , @harshvardhan12 as per your comments I have used verifyFingerprintWithCustomPasswordFallbackAndEnterPasswordLabel to hide passcode button, but now i have a issue, if user has unsuccessful 5 attempts the touchid get disabled/locked for the device, is there a way to unlock without passcode, so that we can implement within the app


Update all the things

Closing in favor of #59 as Symfony 4.0 is not possible for us thanks to our Slack client :cry:


[Engine]QAT Stable Diffusiomn Enabling

Type of Change
Feature Add
Description
Add QAT INT8 stable diffusion example.
Expected Behavior & Potential Risk
N/A
How has this PR been tested?
test ut-engine
Dependency Change?
N/A

https://inteltf-jenk.sh.intel.com/view/nlp-toolkit-validation/job/nlp-toolkit-validation-top-mr-extension/1270/


Add function and state variable visibilities

Not a ground breaking change but it is good to promote contract standards. Hope it helps even just a small amount.


Update for the latest Skyrim AE version

Compass Navigation Overhaul Infinity UI cause CTD for the latest Skyrim AE version.
A fix has been posted on reddit.

Thanks! I'm on it ;)

Done :)


abbr tag display [object object] as title

Please describe your issue. This is so vague.

@Kiarash-Z
here is what happened when user hover on calendar WeekDays (e.g. ش/ی/د/س/چ/پ/ج):

Oh, you're right. Thanks for the bug report. Will be fixed in the next version.

Fixed on the latest release(3.0.3) 🎉


[SPARK-27001][SQL][FOLLOW-UP][WIP] Drop Serializable in WalkedTypePath

What changes were proposed in this pull request?
This pr tried to drop Serializable in WalkedTypePath.
How was this patch tested?
Pass Jenkins.

This is a follow-up baed on the comment: https://github.com/apache/spark/pull/23908#discussion_r261501611

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8499/
Test PASSed.

Test build #103036 has started for PR 23973 at commit 9145e48.

Yeah, I guess removing WalkedTypePath from logical plans would change the needs on Serializable. (Before that I clearly saw test failure when WalkedTypePath doesn't extend Serializable.)
LGTM once the full build succeeds.

Test build #103036 has finished for PR 23973 at commit 9145e48.

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

Merged build finished. Test PASSed.

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

thanks, merging to master!


Improve flow coverage

#### Problem/Rationale

Fusion.js packages, with the exception of `fusion-core` do not export [libdef](https://flow.org/en/docs/libdefs/) and consumers rely on the Flow type definitions embedded in the source code.  Given this, it is a pain point for consumers when type definitions are missing or incomplete.

We should also strive for full Flow coverage to minimize issues within our own packages.

#### Solution/Change/Deliverable

Two-fold:
* Reach 100% Flow coverage on exported type definitions.
* Strive for 100% Flow coverage internally for each package.

(compare to before)

!merge


[WIP] Update the handler to match style of golang-http-template

Description

Update the template to match the (event, context) style of the golang-http-template.
Add dev build options
Update README with instructions on how to use the template
Add waitress as the production server

Motivation and Context

This will allow users to return custom responses for their functions.

[x] I have raised an issue to propose this change (required)

Which issue(s) this PR fixes

Fixes #8
Fixes #15
Resolves #12
How Has This Been Tested?

[x] All examples in README were built and tested for expected responses for python-flask
[x] All examples in README were built and tested for expected responses for python3-flask
[x] All examples in README were built and tested for expected responses for python3-flask-armhf

Types of changes

[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Signed-off-by: Kevin Turcios<EMAIL_ADDRESS>

Hi, is this still WIP?

It should be ready now

@kturcios I'm going to merge this and would like the comments followed-up in a separate PR.
Please also upgrade the watchdogs to the latest possible version.
Thank you for working on this template 💯


Citation Assignment

@alanarnholt
For the citation assignment, are we just supposed to use Zotero to cite the sources, or do we do the second part (below) as well?
"Describe findings and interesting themes that
you have uncovered and list a tentative research question or two
that you hope to pursue. Be brief and use bullets to cover these
details."

Nevermind, I understand what we have to do after watching the videos.


Dependencies almost working

There is something wrong with twitter-typeahead-components-xdomainrequest that has to be included in the main project (the one including donejs-typeahead)
You'll need to run this for a while in order to make things work:
npm install twitter-typeahead-components-xdomainrequest --save
you can also remove it as a dependency of donejs-typeahead:
    "meta": {
      "twitter-typeahead-components-bundle": {
        "deps": [
          "twitter-typeahead-components-handlebars",
          "jquery",
          "twitter-typeahead-components-xdomainrequest" // ===>>>>> remove this
        ],
        "exports": "Bloodhound"
      }
    }

NOTE: twitter-typeahead-components-handlebars works but twitter-typeahead-components-xdomainrequest why?

you can also remove it as a dependency of donejs-typeahead:
"meta": {
      "twitter-typeahead-components-bundle": {
        "deps": [
          "twitter-typeahead-components-handlebars",
          "jquery",
          "twitter-typeahead-components-xdomainrequest" // ===>>>>> remove this
        ],
        "exports": "Bloodhound"
      }
    }

I'll remove the dependency for a while


Landing page overhaul

Currently, I'd say our landing page is a tad bit lackluster with regards to expressing who we are. As we're broadening our reach to general audiences such as PINs, volunteers, partners, potential donors and non-technical contributors, we need to have a professional-looking central website that conveys our mission, our team, our partners, our achievements, etc. to let the world gain a holistic view of our project.
Before the actual frontend development, the beginning stages are gonna be more in line with graphic design and UX flow before the actual frontend dev bit. Ideally, we want a few iterations that the UX and User Testing teams can provide more detailed feedback on.
Starting with design, Miranda's already made some amazing stuff through our brief and marketing materials which are a fantastic start. I've identified a few landing pages that work as good models such as VolunteerMatch and Webflow. I also found a few cool looking templates here that could be useful for designing.

Webflow looks good, as do some of the templates. The Volunteer Match though, I don't like it.
We should open up a poll in the slack channel about this

Although I do agree that the visual design of VolunteerMatch is a bit dated, it does seem like a very inviting and nice social environment that you'd want to support. Also before we present options to the whole team, we should probably come up with some options of our own as well after marketing design discuss this a bit.


When placed on another vertical slab, ender campfire becomes soul campfire

https://user-images.githubusercontent.com/24257045/102688184-dd8c9f80-41ec-11eb-8e7c-7eccbb7ab339.mp4

Thank you for the bug report and sorry for the delayed response. What mod is the ender campfire from please?

It's from Endergetic Expansion. No worries about the response time, there's 41 issues after all!


Remove FSDP restriction from PyTorch 1.13

What does this PR do?
Remove FSDP restriction from PyTorch 1.13. We added this to give a friendlier error for users when PyTorch had a bug, but the corresponding issue is closed and fixed and has been patched for a while since PyTorch 2.

lgtm, but can you include a run name that successfully runs with torch 1.3 and fp32 param_dtype?

It does not run with 1.13, but we've moved off from that and upgraded


OneManage Federated Support

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Support OneManage Federated Multi Cluster
https://www.cisco.com/c/en/us/td/docs/dcn/ndfc/1222/articles/ndfc-one-manage/one-manage.html#_creating_and_managing_multi_cluster_fabrics_using_one_manage
New or Affected Role(s):

valdiate, create, deploy, remove

Cisco NDFC version

V x.x.x

Cisco NX-OS version

V x.x.x

Potential Ansible Task Config

# Copy-paste your ansible playbook

References

Additional context
Add any other context or screenshots about the feature request here.

https://github.com/netascode/ansible-dc-vxlan/pull/205 includes the first pass at this


Reorg metrics graph

Describe the feature
Currently we have some reorg metrics:
https://github.com/paradigmxyz/reth/pull/3860
But they are not surfaced in the reth grafana dashboard. These metrics should be added to the blockchain tree section of the dashboard.
Additional context
No response

I'll take this


Relying on PKEXEC_UID might not be correct?

If swhkd is started in a systemd service PKEXEC_UID won't be correct. How will swhkd then communicate with the user process?

If swhkd is started in a systemd service PKEXEC_UID won't be correct. How will swhkd then communicate with the user process?

I believe you can run processes as the desired user with systemd if I'm not mistaken?
But either way I generally agree with your stance, privilege escalation with PKEXEC has caused a lot of minor issues here and there. Once again, if you would like to send patches for a better privilege escalation model, I'd be more than happy to collaborate and merge the patch.

I believe you can run processes as the desired user with systemd if I'm not mistaken?

Then you would have to specify a password every time you startup? You would want to run swhks as the desired user but swhkd system-wide?

Then you would have to specify a password every time you startup? You would want to run swhks as the desired user but swhkd system-wide.

Apologies, I should've mentioned that I use runit and I have no knowledge of systemd at the moment.

The premise should be the same though. If you run swhkd as a user process (in systemd, runit, etc), you have to type in your password for pkexec everytime you startup, right?

The premise should be the same though. If you run swhkd as a user process (in systemd, runit, etc), you have to type in your password for pkexec everytime you startup, right?

This is incorrect! we package a pkexec policy file so that you don't need to enter the password!

I doesn't seem to be working for me. pkexec swhkd as a normal user asks for my password every time.

@Shinyzenith I got it to work! You have to give pkexec the full path. pkexec /usr/bin/swhkd works. Is this supposed to happen? The readme says to just run pkexec swhkd.

@Shinyzenith I got it to work! You have to give pkexec the full path. pkexec /usr/bin/swhkd works. Is this supposed to happen? The readme says to just run pkexec swhkd.

Hmm odd, on my system it works fine...I wonder why just swhkd didn't work.
Can you run which swhkdm

/sbin/swhkd

Ah that's the reason, the policy file is only for /usr/bin/swhkd

Probably best to remove it from skin, closing

I don't think that is the solution. My /sbin seems to be symlinked to /usr/bin or something. Removing something from /sbin removes it from /usr/bin.

I don't think that is the solution. My /sbin seems to be symlinked to /usr/bin or something. Removing something from /sbin removes it from /usr/bin.

That's not how symlinks work ❤. You can however bump up the precedence of /usr/bin but that might break stuff. Easiest is to either:

Edit the pkexec file to accept /sbin
Or remove the sbin symlink

That's not how symlinks work :heart:.

What do you mean? From the arch linux faq

Arch Linux follows the file system hierarchy for operating systems using the systemd service manager. See file-hierarchy(7) for an explanation of each directory along with their designations. In particular, /bin, /sbin, and /usr/sbin are symbolic links to /usr/bin, and /lib and /lib64 are symbolic links to /usr/lib.

That's not how symlinks work heart.

What do you mean? From the arch linux faq

Arch Linux follows the file system hierarchy for operating systems using the systemd service manager. See file-hierarchy(7) for an explanation of each directory along with their designations. In particular, /bin, /sbin, and /usr/sbin are symbolic links to /usr/bin, and /lib and /lib64 are symbolic links to /usr/lib.

yes but removing a binary from sbin shouldn't cause catastrophe, that was what I was aiming for haha

Anywho, I think your best bet is to run the following:
sudoedit /usr/share/polkit-1/actions/com.github.swhkd.pkexec.policy
and edit the usr/bin/swhkd into  /sbin/swhkd

What do you mean by "cause catastrophe"?

What do you mean by "cause catastrophe"?

Deleting soft/hard link files don't remove the source if I'm not mistaken. ( note: had a coworker check haha )

I'm pretty sure the directory /sbin itself is linked to /usr/bin, not /sbin/swhkd linking to /usr/bin/swhkd. Then, deleting something from /sbin should delete something from /usr/bin

okay got it, in that case editing the policy file is probably the best


Pass locator argument to matplotlib when calling plot.contourf

Is your feature request related to a problem? Please describe.
Everytime I have to do a contourf I need to call it from matplotlib directly because locator argument is not passed from xarray.plot.contourf to matplotlib.
Describe the solution you'd like
Being ds a xarray.DataArray, I want to have the behaviour described here when passing locator =  ticker.LogLocator() to ds.plot.contourf.

I can tackle this one, if no one does that before, I just have to separate a time in my week schedule for open-source projects.

I can tackle this one, if no one does that before, I just have to separate a time in my week schedule for open-source projects.

@iuryt you could also create levels manually using the locator you like and pass that. That seems better than adding yet another kwarg to our plotting functions.

@iuryt you could also create levels manually using the locator you like and pass that. That seems better than adding yet another kwarg to our plotting functions.


chore: bdd-sbhg-1581943431 to 0.0.1

chore: Promote bdd-sbhg-1581943431 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Keyboard or focusNode

Is there any way to use nextFocus without a context? Because when my TextField in a widget that is opened via Get my keyboard is going to be crazy from this line:
onSubmitted: (_) => FocusScope.of(context).nextFocus(), // move focus to next
),

Is there any way to use nextFocus without a context? Because when my TextField in a widget that is opened via Get my keyboard is going to be crazy from this line:
onSubmitted: (_) => FocusScope.of(context).nextFocus(), // move focus to next
),

You can retrieve the context of the last builder widget using Get.key.currentWidget from anywhere in your code.
onSubmitted: (_) => FocusScope.of(Get.key.currentContext).nextFocus(), // move focus to next
),

Sorry but this is not working at all:
next Focus never call when I use with Get.key.currentContext
Also when I use this:
TextField(
        textInputAction: TextInputAction.next,
        controller: controller,
 onSubmitted: (s) {
          FocusScope.of(context).nextFocus();
          //FocusScope.of(Get.key.currentContext).nextFocus();
        },

then the probeleme is:
First time iot is working like a charm, But after I write something in the field and submit called setState or changeNotifier, the second time when click in the TextField my keyboard just pop up and immediatelly pop down (show and hide)
Also I have to generate on the fly a lot of TextField and controllers for that and perhaps listeners to that...
Can you help me how to fix this?
Basically what I want:
Row1 [Textfield][Textfield][Textfield]
Row2 [Textfield][Textfield][Textfield]
Row3 [Button to add new Row]
ADD ->
Row1 [Textfield][Textfield][Textfield]
Row2 [Textfield][Textfield][Textfield]
Row3 [Textfield][Textfield][Textfield]
Row4 [Button to add new Row]
DEL ->
Row1 [Textfield][Textfield][Textfield]
Row2 [Textfield][Textfield][Textfield]
Row3 [Button to add new Row]
Also I have to save those values in somehow and refresh the UI, via setState or changeNotifier
I also find this BUG in the flutter app:
https://github.com/flutter/flutter/issues/52342#issuecomment-599456423

ere is my textfield code
TextField(
        textInputAction: TextInputAction.next,
        controller: controller,
        maxLength: 3,
        maxLines: 1,
        /*
        onChanged: (s) {
          if (onChanged != null) onChanged(s);
        },*/
        /*
        onSubmitted: (s) {
          //FocusScope.of(Get.key.currentContext).nextFocus();
          FocusScope.of(context).nextFocus(); //EZ BASSZA EL?!!!!!!  // minden of(context) !!!
          //if (onSubmitted != null) onSubmitted(s);
        },*/
        textAlign: TextAlign.center,
        enableInteractiveSelection: !disableCopyPaste,
        keyboardType: TextInputType.number,
        inputFormatters: <TextInputFormatter>[WhitelistingTextInputFormatter.digitsOnly],
        cursorColor: Colors.black,
        decoration: InputDecoration(
          counterText: "",
          border: InputBorder.none,
        ),
        style: textStyleNormal,
      ))

and my listeners generating in realtime after build something like this:
 final textWeightControllerListener = () {
          final newText = textWeightController.text;
          textWeightController.value = textWeightController.value.copyWith(
            text: newText,
            selection: TextSelection(baseOffset: newText.length, extentOffset: newText.length),
            composing: TextRange.empty,
          );
          _updateSetFor(exercises[i], j, weight: textWeightController.text);
        };
        textWeightController.addListener(textWeightControllerListener);

You are probably swapping some StatefulWidget for StatelessWidget, or inserting controllers inside build methods instead of inserting them at the beginning of a Stateful class.

So basically I have to create all the TextEditingControllers in initState instead of build? That will work?

So basically I have to create all the TextEditingControllers in initState instead of build? That will work?

Just like I told you in that comment:
https://github.com/jonataslaw/get/issues/20#issuecomment-597416705
Controllers can NEVER be inside builders, no matter if it’s the main build (context) widget, or inside a StreamBuilder, FutureBuilder, BottomSheet, because at each setState a new controller will be created.
You can never do this:
class Example extends StatelessWidget {
  const Example({Key key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final _editText = TextEditingController(); // NEVER DO THAT. Controllers should never be within builder methods.
    return TextField(
      controller: _editText,
    );
  }
}

You should ALWAYS do this:
class Example extends StatefulWidget {
  // If you have any controller it is mandatory to use StatefulWidget.
  Example({Key key}) : super(key: key);

  @override
  _ExampleState createState() => _ExampleState();
}

class _ExampleState extends State<Example> {
 final TextEditingController _editText = TextEditingController(); // HERE

  @override
  Widget build(BuildContext context) {
    return TextField(
      controller: _editText,
    );
  }
}

You can also create the controller in the mentioned place, and instantiate it in your initState, so it will only be called once, however, for code security reasons, I recommend doing it as above.
Note that this is not related to this library, and as much as I want to help you, it can mess up people who are looking for something in issues.
If you want to ask questions about the framework not related to this library, you can ask me on the messenger
https://www.facebook.com/jonny.borges.982
Reserve this field only for issues directly related to Get.


[Bug] [ui] moving a task definition gets http code 405

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
moving a task definition gets 405 code, the api is be removed #8190
What you expected to happen
no error
How to reproduce
move a task definition

Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Fixed.


Part files can't reside a directory level above the main file

I have a part file located at example/part.dart which contains:
`part of `hello_world/main.dart`

I have the main file located at example/hello_world/main.dart which contains:
part `../part.dart`

This returns the invalid error:
Expected this library to be part of 'file:///usr/local/google/home/grouma/Projects/webdev/dwds/example/hello_world/main.dart', not 'file:///usr/local/google/home/grouma/Projects/webdev/dwds/example/hello_word/main.dart'.

I see /hello_word/ in the error message. Is there an l missing in world in the part of declaration?
(The part of  declaration included here is not syntactically valid, so it's probably not the actual one).

Well I feel silly. Looks like this was just a typo. Fixing the typo resolves the issue with part files.


Error: The component text has been already registered. Check that you are not loading two versions of the same component or two different components of the same name.

When requiring kframe with aframe already on the page (also tried k-frame). We tried 0.3.0 and 0.3.2-beta2.
Using Nightly 55.0a1 (2017-05-22) (64-bit) on Windows 10.
Is this conflicting with the built in aframe "text" component?

Include the components individually. The kframe package is gone


rPackages.glmnet: Add missing dependency

Motivation for this change
Fixes the error:
ld: library not found for -liconv
I've followed the convention of other packages relying on iconv.
Things done

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

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

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

Notify maintainers
cc @

I've done a bit more playing with the rPackages on Darwin, and I've realised this fix is only necessary there. rPackages.glmnet works fine on linux using Nixpkgs master.
I can't see any "prior art" which adds iconv for Darwin only, so not sure if that's the right thing to do.
This kind of fix is also needed for rPackages.lars as well, and so there might be a bunch more.


OCP4 Migration

The services need to be migrated to OCP4, they are currently hosted in the devex-von-* namespace set in OCP3.

the target namespace in OCP4 is 4a9599
The contents of the folder /opt/app-root/src/data on the prod instance of dsc-chat-server will need to be migrated to the new deployment. It is just a couple of text files, so I would just copy their contents over with vim rather than setting up a PVC migrator, but see what works best for you.
The client app uses *.vonx.io vanity URLs: please coordinate the certificate and DNS entry settings with @WadeBarnes

I'm assuming the only NSP policies that need to be implemented are client -> server and server -> client? @esune

I'm assuming the only NSP policies that need to be implemented are client -> server and server -> client? @esune

NSPs are typically defined in a single direction client -> server.  This allows the server to respond to the client requests, while blocking the server from making a request to the client.  Which is typically the desired behaviour.


Global constraints and compiler

Codecov Report
Patch coverage: 80.00% and project coverage change: +0.04 :tada:

Comparison is base (5444391) 86.96% compared to head (08a508c) 87.01%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #394      +/-   ##
==========================================
+ Coverage   86.96%   87.01%   +0.04%
==========================================
  Files         186      189       +3
  Lines       23852    24119     +267
==========================================
+ Hits        20743    20987     +244
- Misses       3109     3132      +23

Impacted Files
Coverage Δ

unified_planning/engines/factory.py
78.69% <ø> (ø)

unified_planning/grpc/proto_writer.py
86.11% <ø> (ø)

unified_planning/model/mixins/objects_set.py
87.71% <ø> (ø)

unified_planning/model/problem_kind.py
96.99% <ø> (ø)

unified_planning/test/test_pddl_io.py
99.56% <ø> (ø)

...ed_planning/grpc/generated/unified_planning_pb2.py
62.95% <6.00%> (+0.22%)
:arrow_up:

...g/engines/compilers/negative_conditions_remover.py
95.54% <50.00%> (-0.93%)
:arrow_down:

..._planning/engines/compilers/quantifiers_remover.py
94.07% <50.00%> (-1.35%)
:arrow_down:

unified_planning/io/anml_writer.py
87.10% <50.00%> (-0.30%)
:arrow_down:

...ngines/compilers/trajectory_constraints_remover.py
77.44% <57.14%> (-0.59%)
:arrow_down:

... and 19 more

... and 1 file with indirect coverage changes

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


GROM 自定义中间表的关联字段  (Grom customizes the associated fields of the intermediate table)

Your Question
GROM 多对多关联修改中间表的关联字段,  默认的关联字段是 xxx_model_id这样的 , 我想去掉 _model 这个前缀
Grom many to many association modifies the associated fields of the intermediate table. The default associated field is xxx_model_id,  I want to get rid of this _model prefix
The document you expected this should be explained

Expected answer

https://gorm.io/docs/gorm_config.html#naming_strategy


fix(robots.txt): Remove robots.txt to enable indexing by search machines

Issue
https://docs.cloudfoundry.org/robots.txt is
User-agent: *
Disallow: /

preventing indexing by search machines.
Fix
Looking at https://web.archive.org/web/20240000000000*/https://docs.cloudfoundry.org/robots.txt it sems like since 2014 until July 15th 2024 trying to download robots.txt would give you a 404 and starting August 4th 2024 the current robots.txt appeared.
As previously robots.txt was a 404 this PR removed it, restoring the previous state that worked for years.

Note: I was not able to run this repo due to issues with Ruby, but I would hope that a simple removal of a file should work.


use http transport to send event from manager to agent

Current coverage is 44.57% (diff: 0.00%)

Merging #272 into master will increase coverage by 2.39%

@@             master       #272   diff @@
==========================================
  Files            11         12     +1
  Lines          9696       9806   +110
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4090       4371   +281
+ Misses         4517       4310   -207
- Partials       1089       1125    +36

Powered by Codecov. Last update a745699...3501154