Error store selection/resending/performance issues

Describe the bug
When something goes into error with an errorstorage configured, the incoming message is saved and can be retried/deleted from error storage.
When deleting no tracing is possible, for this a different ticket is already created.
When retrying it's possible to process items more than once and select all works wierd. Besides that performance is below expectations.
In case you have messages being retries and you refresh the page, it's possible to retry the same message.
In case you use multiple nodes, it's possible that all nodes process the same message.
In case multiple users retry messages, it's possible that the same message is processed multiple times.
This causes duplicate output which is highly unwanted.
When clicking select all, it does not only select all visible records, but also selects all earlier accessed records. When for example opening error storage page 1, then page 3 and then page 5 and click 'select all' and 'retry selected'. Then first all page 1 messages are retried, then page 3 and then page 5.
When messages are being retried, it's being done with 1 thread leading to 1 message at a time. For the engageone process we use 15 threads for speed. 1 thread is not enough to process a daily load. Therefore is not usable to recover 1 day of errors.
Reporter
Bonno Smeele for Jeri Tjalsma
Category
Use following Tags to categorize your bug report

Frank Console
-Frank Framework

To Reproduce
Should be reproducable by sending messages to the error store
Expected behavior

prevent duplicate messages being sent from error store during recovery
fix bugs with selection in console
enable multiple threads on error store resending to speed up processing

Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

Websphere
any browser
FF! 7.6-20200722.094609

Additional context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

Currently messages are 'retried' as described above: by retrying them individually. I suggest this to change the way of retrying, for Listeners that support it, by moving the messages from the 'errorStorage' to the 'input' queue. This can be implemented for all FileSystem based Listeners (DirectoryListener, ExchangeMailListener, ...) by moving from 'errorFolder' to 'inputFolder'. It can also be implemented for the JdbcTableListener, by updating the status from 'statusError' to 'statusAvailable'.

In the case of a jdbc table listener, would it be an idea to create a retryQuery property in the jdsbclistener that can be used to reset errors entries in the table? This query could be executed when the retry button is pressed.
For instance, at the moment we have this error query in the listener:
updateStatusToErrorQuery="UPDATE OUTPUTSET SET OUTPUTSET_STATUS = 4 WHERE OUTPUTSET_ID = ?"
with this key field:
keyField="OUTPUTSET_ID"
This could correlate all the IDs from the selected messages in the error store with the JDBC listener tabler and set them to a "ready" status.
Apart from this suggestion there is also the problem of the UI selecting messages for retrying from pages that have been visited manually in the error store. This is not intuitive:
"When clicking select all, it does not only select all visible records, but also selects all earlier accessed records. When for example opening error storage page 1, then page 3 and then page 5 and click 'select all' and 'retry selected'. Then first all page 1 messages are retried, then page 3 and then page 5."

Made separate ticket #1355 for the 'Select All' issue


Not able to view new screenshot directly in the Cypress UI.

Hello FRSOURCE
Firstly I just want to thank you for creating this plugin
I've followed the installation instructions and nearly got the plugin working however when I click on 'see comparison' in the cypress test runner

it doesn't allow me to preview old & new screenshot directly in the Cypress UI.
Also when I open link in a new tab I get the following

Cypress version: v10.2.0
please can you assist and happy to provide more detail if needed

Hey @JavedAubdoolQA
Thanks for reaching out, I'll be more than happy to assist!
I'm trying to reproduce it on my private repo, but without any luck.
Can you provide me with a simple reproduction scenario of this issue? You can use the example in this repository as a starting point.

Hey @JavedAubdoolQA - have you got any updates about this issue?
I've tried to reproduce it once again, but without any luck.
Regarding opening link in the new tab: the screenshot comparison is being opened in a modal, it's not a separate page. So opening in a new tab is not supposed to be working. Unfortunately Cypress does not make it easy to customize anchor tags in the test UI - so I think no further action can be done here (I'm open if anyone find a solution though!) 😞
Regarding the original question - as I'm not able to reproduce for 20 days now I'm marking this issue as "stale". If nothing happens in this thread for next week I'll need to close it

Hello there! I've seen a similar behavior, but using cypress-mochawesome-reports plugin. I can see the diff on the Cypress UI, but when I run the tests on headless mode (the default on the CI) the images are not being added to the report. It creates a set of empty folders (where the image should be).
What I'm expecting to see on the report is the original screenshot in case there's no diff. If there is any diff, I expect to see the original screenshot, the diff and the actual.
I'm using cypress-mochawesome-reports default config, and visual regression plugin defaults. The issue becomes more evident when I turn on embeddedScreenshots on for the reporter:
reporter: 'cypress-mochawesome-reporter',
reporterOptions: {
   embeddedScreenshots: true,
},

I'm getting the following error:
Start generate report process
Read and merge jsons from "{root_path}/cypress/reports/html/.jsons"
Enhance report
An error was thrown in your plugins file while executing the handler for the after:run event.

The error we received was:

Error: ENOENT: no such file or directory, open '{root_path}/cypress/screenshots{root_path}/cypress/integration/__image_snapshots__/Error page should validate there are no visual changes #0.actual.png'
    at Object.openSync (node:fs:585:3)
    at Object.readFileSync (node:fs:453:35)
    at convertImageToBase64 ({root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:111:52)
    at {root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:104:11
    at Array.map (<anonymous>)
    at createScreenshotsContextList ({root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:100:22)
    at attachScreenshotsToTestContext ({root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:76:9)
    at {root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:34:24
    at Array.forEach (<anonymous>)
    at {root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:32:19
    at Array.forEach (<anonymous>)
    at attachScreenshotsToSuiteTestsContext ({root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:31:12)
    at {root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:16:7
    at Array.forEach (<anonymous>)
    at enhanceReport ({root_path}/node_modules/cypress-mochawesome-reporter/lib/enhanceReport.js:6:18)
    at mergeAndCreate ({root_path}/node_modules/cypress-mochawesome-reporter/lib/generateReport.js:19:3)
    at async Promise.all (index 0)
    at async generateReport ({root_path}/node_modules/cypress-mochawesome-reporter/lib/generateReport.js:62:22)
    at async afterRunHook ({root_path}/node_modules/cypress-mochawesome-reporter/lib/index.js:35:3)
    at async Object.handler ({root_path}/node_modules/cypress-mochawesome-reporter/plugin.js:9:5)

This is what I see when I disable embeddedScreenshots:

The url of the file is pointing to file:///{root_path}/cypress/reports/html/screenshots{root_path}/cypress/integration/__image_snapshots__/Error%20page%20should%20validate%20there%20are%20no%20visual%20changes%20#0.actual.png
Note: {root_path} is the full path to the project folder. I'm using linux.
I believe that the issue is related to the path of the image being sent to the test context.
Is this enough detail for a follow up?
Thank you!

Hey @Pablo-Quiroga-Globant 👋
Thanks for the detailed response, it's great!
This thread is about @JavedAubdoolQA not being able to see the screenshot when using "See comparison" button (which comes from this package) while your comment seems to be focusing on the case when the package is used together with some reporting plugin. Could you please open new issue for your scenario and fill out the bug template? 🙏

Regarding the original info - @JavedAubdoolQA due to lack of response I need to close this issue. Please open up a new issue if you believe that issue still exists and have an example!
And once again - thanks for reaching out 🎉
@Pablo-Quiroga-Globant please open up a new issue ticket about your scenario, cheers!


How do the campaign finance Committee and core Organization models overlap?

I'm working on filling out @aepton's implementation of the Campaign Finance enhancement proposal, and I've uncovered an issue @jamesturk might need to consider sooner rather than later.
In the campaign finance OCDEP, Committee is described as a "Subclass of Popolo Organization", and adds a couple of attributes not expected on the base class. The existing OCD Organization class is a non-abstract model, and my understanding is that, as a general rule, we don't want to subclass non-abstract models.
This is similar to the Event/Election situation we got into previously. The solution then was to partially copy the definition of the Event model and create a new Election model. We also copied the definitions of the EventIdentifier and EventSource models to make new ElectionIdentifier and ElectionSource models.
I'm following this same approach for Committee and it's related models, but a few things give me pause:

Whereas, Event was in the legislative module and was tailored to those use cases, Organization is in the core module and is defined rather generically.
Organization has a couple of other related model that Committee also needs:

OrganizationIdentifier for storing alternate ids for the campaign finance committee
OrganizationName for storing the committee's various different names
Membership and Post for storing relationships committees and their treasurers, officers, staff, etc.
Maybe also OrganizationSource and OrganizationContactDetails

In the campaign finance spec, each Transaction has a .sender and .recipient each of which will have either a .person or .organization attribute populated, depending on the entity type. Under the current pattern, I think we also need a .committee attribute, since these can also send and receive transactions.
While I don't foresee the need to represent an election as any other OCD data type, I can foresee needing to represent a single organization as a campaign finance committee as well as other types of organizations (e.g., political parties).

If 4 is a real need, then it feels like exactly what Django's multi-table inheritance was designed for. Even if it isn't, multi-table inheritance would also address the issues raised in 2 and 3.

1 (and also 2, I guess): So Organization is in the core module and defined generically, but it's non-abstract? I'm far out of my depth re: Popolo and OCD classifications, but those seem like reasons it'd be a fair candidate for subclassing.
3: Yes, for sure. Though, since it's not always clear from a disclosure whether a donor is a person, company or even committee, would it be better to have a generic donor entity with flags for person, committee or organization? Not sure how to distinguish between the latter two;  companies are clearly organizations and not committees; on the other hand, the entity doing the donating on behalf of a company is often a committee set up by that company. Some disclosures in some jurisdictions, filed by some campaigns, make it (reasonably) clear which is which, but many do not.
4: Yeah, there can be fairly intricate relationships between different types of committees and organizations. In WA, the state D and R parties have specific committees who fundraise and make donations to candidates in each chamber of the legislature (Kennedy Fund, Truman Fund, etc). Those committees are essentially controlled by the state parties, but nominally different entities.


Edge Case Bug

Describe The Problem
A clear and concise description of what the bug is.
https://github.com/user-attachments/assets/79a14f27-32ba-442a-8194-99a885377cc4
Expected Error / Traceback
Please provide the error traceback here

Sample of your .conf file
[Interface]
Address = <IP_ADDRESS>/24, fe80::638b:df05:dd54:65d1/64, fe80::3060:9c6d:41e7:13e4/64, fe80::b004:ebe5:df7:86a5/64, fe80::fdc4:33c4:dbe2:3437/64, fe80::1dcd:6d91:349:6b77/64
SaveConfig = true
PostUp = /opt/wireguarddashboard/src/iptable-rules/Admins/tor-postup.sh
PreDown = /opt/wireguarddashboard/src/iptable-rules/Admins/tor-postdown.sh
ListenPort = 4430
PrivateKey = EOeVZ+333333333333333333333333333=
Jc = 5
Jmin = 500
Jmax = 550
S1 = 30
S2 = 40
H1 = 123456
H2 = 67543
H3 = 32345
H4 = 123123

[Peer]
PublicKey =<PHONE_NUMBER>3333333333333333332HXgnxuVFWQ=
PresharedKey =sssssssssssssssssssssssssssssssssssssssnDTPc=
AllowedIPs = <IP_ADDRESS>/32

[Peer]
PublicKey = bbjwbejhbdwbdbwO11gggggfsbsbxxibbbbixbebxibTAU=
AllowedIPs = <IP_ADDRESS>/32
Endpoint = <IP_ADDRESS>:60547

@donaldzou Nope, Wireguard works fine and doesnt cause this issue turns out. The AmneziaWG-go is based off of wireguard-go, which doesnt natively handle ipv6 causing a new one to be generated upon interface startup. Youll run into this and other issues trying to intigrate AmneziaWG-go, im still working on the kernel module image, which wouldnt need patching. The only issue sould now be IPV6 Allowed Ip Assignment.
def toggleConfiguration(self) -> [bool, str]:
        self.getStatus()
        interface_address = self.get_awg_iface_address()

        config_file_path = os.path.join(DashboardConfig.GetConfig("Server", "wg_conf_path")[1], f"{self.Name}.conf")

        if self.Status:
            try:
                check = subprocess.check_output(f"wg-quick down {self.Name}",
                                                shell=True, stderr=subprocess.STDOUT)
            except subprocess.CalledProcessError as exc:
                return False, str(exc.output.strip().decode("utf-8"))

            # Write the interface address after bringing it down
            write_error = self.patch_awg_iface_address(interface_address)
            if write_error:
                return write_error
        else:
            try:
                # Extract IPv6 address from the WireGuard configuration file
                with open(config_file_path, 'r') as f:
                    config_data = f.read()

                # Extract the IPv6 address from the Address line
                ipv6_address = None
                for line in config_data.splitlines():
                    if line.strip().startswith("Address"):
                        parts = line.split("=")[1].strip().split(", ")
                        for part in parts:
                            if ":" in part:  # Check if the part looks like an IPv6 address
                                ipv6_address = part.strip()
                                break
                        if ipv6_address:
                            break

                # Modify the logic to continue without IPv6 if not found
                if ipv6_address:
                    # Bring the WireGuard interface up
                    check = subprocess.check_output(f"wg-quick up {self.Name}",
                                                    shell=True, stderr=subprocess.STDOUT)

                    try:
                        # Remove any existing IPv6 addresses for the interface
                        remove_ipv6_cmd = f"ip -6 addr flush dev {self.Name}"
                        subprocess.check_output(remove_ipv6_cmd, shell=True, stderr=subprocess.STDOUT)

                        # Add the new IPv6 address with the desired parameters
                        add_ipv6_cmd = f"ip -6 addr add {ipv6_address} dev {self.Name}"
                        subprocess.check_output(add_ipv6_cmd, shell=True, stderr=subprocess.STDOUT)
                    except subprocess.CalledProcessError as exc:
                        return False, str(exc.output.strip().decode("utf-8"))
                else:
                    # No IPv6 address found, just bring the interface up without modifying IPv6
                    check = subprocess.check_output(f"wg-quick up {self.Name}",
                                                    shell=True, stderr=subprocess.STDOUT)
            except subprocess.CalledProcessError as exc:
                return False, str(exc.output.strip().decode("utf-8"))

        self.getStatus()
        return True, None

WireGate using Static WGDashboard Binary with AmneziaWG Support
https://github.com/user-attachments/assets/921afd53-39c5-4d08-ae44-340d499e9292


blocked when building project

goto conductor folder and run: ./gradlew build -x test, then got the blocking below:
<==-----------> 18% EXECUTING [1m 5s]
> :conductor-es6-persistence:shadowJar

@andyyumiao This gradle task is a slow-running task (~6-7 mins). It is used to shade the ES6 dependency so that ES6 can be used without having the rebuild the project.

@apanicker-nflx The task is taking more than 10 mins even after multiple attempts.
C:\Users\conductor\server>..\gradlew clean build -x test
Starting a Gradle Daemon (subsequent builds will be faster)

Configure project :
Inferred project: conductor, version: 2.31.0-SNAPSHOT
Publication nebula not found in project :.
<====---------> 34% EXECUTING [14m 35s]
:conductor-es6-persistence:shadowJar

Also what should be done regarding the Nebula publication ?

@apanicker-nflx The task is taking more than 10 mins even after multiple attempts.
C:\Users\conductor\server>..\gradlew clean build -x test
Starting a Gradle Daemon (subsequent builds will be faster)

Configure project :
Inferred project: conductor, version: 2.31.0-SNAPSHOT
Publication nebula not found in project :.
<====---------> 34% EXECUTING [14m 35s]
:conductor-es6-persistence:shadowJar

Also what should be done regarding the Nebula publication ?


Updating pom.xml to have variables of hadoop and antlr versions and to remove tailing spaces.

Updating pom.xml to have variables for versions of hadoop and antlr and to remove tailing spaces.
The versions of Hadoop and antlr are used in multiple sections manually: this PR fixes it and changes to use variable - hadoop.version and antlr.version.
We can also pass the version as arguments of maven command like mvn clean package -Dhadoop.verion=2.4.1.

Test build started here.

Hi, thanks for contributing this!  As usual, I've started a test build, and pending the results of that, this looks good to me.

@dusenberrymw thank you for the review. The test looks to be passed?

Testing successful.

Merged.  Thanks, @oza!

Thank you for committing :-)


added spfx v1.11 check to 'm365 spfx doctor' command

Extend 'spfx doctor' to support v1.11

Adding spfx v1.11 check to 'm365 spfx doctor' command
Added a few spec tests
Did not see any major dependencies updates so currently matches v1.10 versions pre-req

Closes #1721

@waldekmastykarz looks like there is another test case that was failing in list-webhook-add.spec.ts. That appears existing issue when i cut the topic branch from master.  Should I patch that up?

This issue should be fixed in upstream now, unless there is another instance of it. Before you patch it, could you please have a look if you have latest changes in your branch?

@waldekmastykarz should I just close this on my own?

No, please don't. I ran out of time yesterday and haven't complete the process. Sorry for the confusion.

Merged manually. Thank you! 👏


outlier_detector: accept large base_ejection_time when max_ejection_time not specified

Commit Message:
accept large base_ejection_time when max_ejection_time not specified
Additional Description:
PR #14235 broke compatibility with istio. Istio sets large base_ejection_time and configs where max_jection_time was not specified were rejected. That scenario happened during upgrade from istio 1.8 to 1.9. See https://github.com/envoyproxy/envoy/pull/14235#issuecomment-772156254.
This PR changes the logic to following:

if both base_ejection_time and max_ejection_time are specified in the config, they are compared and config is rejected when max_ejection_time is smaller than base_ejection_time.
if max_ejection_time is missing in the config, the default value of 300s is applied. If base_ejection_time is larger than 300s, then max_ejection_time is initialized to base_ejection_time.

Risk Level: Low
Testing: Added unit test to simulate receiving config without max_ejection_time and with large base_ejection_time.
Docs Changes: No
Release Notes: No
Platform Specific Features:

@lizan Can we backport this to 1.17 once it is merged?

/retest

@markdroth Would you mind approving api? Syntax was not changed - just updated the comment. Thanks!


Pipfiles

Switch to pipfiles because we're standardizing on pipenv!

@melinath Oh right, removing requirements.txt breaks our RTD builds. Did we come up with a way of addressing that?

Pipenv has figured it out, so should be doable.

Looks like RTD has an option to install via setup.py, maybe that?

We are already using that option. Maybe if we add the Sphinx requirements to setup.py? I'm not sure how all of that works.

There's always this option which involves a manual step: https://github.com/gsemet/pipenv-to-requirements

Oh, pipenv has a separate requirements.txt in the docs folder. We should probably just do that.


Fetch URLs should follow redirections

Apparently, HTTP redirections are not followed by the URL fetcher: they probably should.

I'm already looking at the URL fetching as part of #1217 so I can look at this as well

Related question on the mailing list.


Update README layout section

Signed-off-by: David Cassany<EMAIL_ADDRESS>

Just realized I missed to update the README file in the previous PR, sorry for the noise.


Updating the developer guide and my ppp

could this be merged

@Sameow @windrichie @JKupoluyi

Can you clear the tests first


KeyError: "name='csrftoken', domain=None, path=None"

I used instaloader for two days and it worked fine but today every time that I tried to use it I got this error:
Traceback (most recent call last):   File "/usr/local/bin/instaloader", line 11, in <module>     sys.exit(main())   File "/home/pi/.local/lib/python3.5/site-packages/instaloader/__main__.py", line 324, in main     storyitem_filter_str=args.storyitem_filter)   File "/home/pi/.local/lib/python3.5/site-packages/instaloader/__main__.py", line 87, in _main     instaloader.login(username, password)   File "/home/pi/.local/lib/python3.5/site-packages/instaloader/instaloader.py", line 317, in login     self.context.login(user, passwd)   File "/home/pi/.local/lib/python3.5/site-packages/instaloader/instaloadercontext.py", line 169, in login     session.headers.update({'X-CSRFToken': resp.cookies['csrftoken']})   File "/home/pi/.local/lib/python3.5/site-packages/requests/cookies.py", line 328, in __getitem__     return self._find_no_duplicates(name)   File "/home/pi/.local/lib/python3.5/site-packages/requests/cookies.py", line 399, in _find_no_duplicates     raise KeyError('name=%r, domain=%r, path=%r' % (name, domain, path)) KeyError: "name='csrftoken', domain=None, path=None"
Edit: when only instaloader is installed I get the "Too many queries in the last time" error and if I use:
pip3 install --upgrade git+https://github.com/instaloader/instaloader@issue_112_fix
I get the error above.

Any updates on this? I still get this error :/


English translation

I think English translation required for the community. Thanks.

Hi @EgoistDeveloper ， sorry for my late response,   the repo's chart is generated base by roadmap that I added commented in README,   get more information yourself . About the other articles, I can't spare time for translation and other things .


refactor: remove unnecessary lazy_static

Signed-off-by: TennyZhuang<EMAIL_ADDRESS>What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Remove some unnecessary lazy_static
Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: TennyZhuang:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

@mergifyio requeu

@merfifyio requeue

@mergifyio requeue


raftstore: use KvEngine trait instead of implementations

What problem does this PR solve?
Replace KvEngine implementations with KvEngine itself in raftstore, so that other applications can use raftstore directly.
Related changes
Check List
Tests

Unit test
Integration test

Side effects
Release note

Use KvEngine trait in raftstore

/run-all-tests

rest LGTM


using group.children instead of the query selector

Unfortunately, there isn't a querySelectorAll that works for getting the immediate children of an element. :scope would work but it has limited support and I dont want to add a polyfill. Using .children and then filtering out the things that don't have a pfe-heading attribute should do the trick.
Fixes #314

👍 Lemmings going toward mountains

Any reason why this can't be merged?

@starryeyez024 No, it's ready and approved!  It was just pending AT passing and it has. w00t.


rpcs3: 0.0.33-17070-8b8396b94 -> 0.0.34-17206-9d4ff13c2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rpcs3 is: PS3 emulator/debugger
meta.homepage for rpcs3 is: https://rpcs3.net/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.34-17206-9d4ff13c2 in filename of file in /nix/store/s69v91fmmn71zq4bka58zam4fq2s28dk-rpcs3-0.0.34-17206-9d4ff13c2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rpcs3

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s69v91fmmn71zq4bka58zam4fq2s28dk-rpcs3-0.0.34-17206-9d4ff13c2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rpcs3 https://github.com/r-ryantm/nixpkgs/archive/49fe76b7290f95eac9238af5a7521dc3d84ad568.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/49fe76b7290f95eac9238af5a7521dc3d84ad568#rpcs3

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s69v91fmmn71zq4bka58zam4fq2s28dk-rpcs3-0.0.34-17206-9d4ff13c2
ls -la /nix/store/s69v91fmmn71zq4bka58zam4fq2s28dk-rpcs3-0.0.34-17206-9d4ff13c2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

rpcs3

Maintainer pings
cc @abbradar @neonfuz @ilian @vs49688 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

error: hash mismatch in fixed-output derivation '/nix/store/4f3azm436mg094lpzwpi2g10dgv5xjdj-source.drv':
        likely URL: https://github.com/RPCS3/rpcs3.git
         specified: sha256-FZItMeClPxwzwLVH5YPLV7NQZ5kFn6SjdduMI9P2kWQ=
            got:    sha256-Ai1JWjvo1GBO7ewVtdX7wQg/gwvFtyinUae4Btu0Hok=
error: 1 dependencies of derivation '/nix/store/x8krxxwxm1bchmfpbak21f3ic5azslja-rpcs3-0.0.34-17206-9d4ff13c2.drv' failed to build
error: 1 dependencies of derivation '/nix/store/k9csm7y4j27z9czfzpfdwzchlk1cvmfc-system-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/9nad1kzi1zrg27y725sr6hbgdkhrc2cr-nixos-system-blahaj-25.05.20241219.4d982ce.drv' failed to build

weird


Use of Vue SFC in the UTabs #item slot

I'm trying to insert an SFC inside of a UTabs panel, but nothing shows up.
<template>

         À L'AFFICHE :

      <UTabs :items="items" class="w-full">
         <template #default="{ item, index, selected }">

               {{ item.label }}

         </template>
         <template #item="{ item }">
            <NewsArticle :info="item" :is-on-front-page="true" />

         </template>
      </UTabs>

</template>

1st: I'm transforming my response data into something compliant with the UTabs props. Just guessing based on the docs.
I can pass the items into the slot correctly because when using a "pre" tag, all my data is displayed. Switching between tabs works as well, the data is correctly matching the Tab title (#default slot)

I'm using this rather complex SFC in other parts of my app and I don't want to repeat a few 100s lines.
Is there something that I'm missing?
Thank you
PS: I have a TS error on UTabs that I don't know how to fix,
Argument of type '{ items: { key: number; label: string; content: Info; }[] | undefined; class: string; }' is not assignable to parameter of type '{ ui?: any; modelValue?: number | undefined; items?: TabItem[] | undefined; orientation?: "horizontal" | "vertical" | undefined; defaultIndex?: number | undefined; ref?: VNodeRef | undefined; ... 12 more ...; "onUpdate:modelValue"?: ((...args: any[]) => any) | undefined; } & Record<...>'.

Would you mind providing a reproduction link on StackBlitz, it would be easier to help. There is an example of what you're trying to achieve here: https://github.com/nuxt/ui/blob/dev/docs/components/content/examples/TabsExampleItemSlot.vue

Hi @benjamincanac,
I have dug much deeper on this recently.
1st, I sorted my stuff out, and now it works.
I am able to pass an entire SFC inside the body of the tab through the template #item="{ item }".
There is a hiccup though:
The Tab SFC accepts items as TabItem[]
and TabItem only has these properties:
label: string
slot?: string
disabled?: boolean
content?: string

Why do we only limit content to be a string, where it can handle Objects?
This works perfectly:
<UTabs :items="items">
    <template #default="{ item, index, selected }">

            {{ item.label }}

    </template>
    <template #item="{ item }">
        <NewsArticle :info="item.content" :is-on-front-page="true" /> // This is a Nuxt SFC!!
    </template>
</UTabs>

I have a single TS error in this file:
Type '{ label: string; content: Info; }[]' is not assignable to type 'TabItem[]'
Which can be fixed by enhancing TabItem with Generics instead, for example.

Update:
I wanted to propose a PR with this, but I saw that @Haythamfpco pushed a commit to allow other cutom KV pairs of type 'any'.
Here's from a git clone of nuxt/ui:
export interface TabItem {
  label: string
  slot?: string
  disabled?: boolean
  content?: string
  [key: string]: any
}

I could use this custom key to pass my object and get rid of the error, but as of now with @nuxt/ui v.2.8.1, this change is not available even after removing the package and reinstalling the latest version!
Can you please bump the version and release the unpublished changes?

Would you mind providing a reproduction link on StackBlitz, it would be easier to help. There is an example of what you're trying to achieve here: https://github.com/nuxt/ui/blob/dev/docs/components/content/examples/TabsExampleItemSlot.vue

I am able to pass any other prop into the tab item with the recent addition of '[key: string]: any' in the nuxt/ui local clone playground but this change is not live in the latest release v2.8.1. and doesn't work in my project depending on the official release.
Can you please look at my proposal above (to keep type inference for 'content')?
If yes, I'll commit and open a PR.

Can this be closed? If the remaining subject is about generic components, it will be done in #1289.


keepassx is outdated

Issue description
keepassx in nixpkgs is 0.4.4 whereas latest release is 2.0.2
Requires an update. I'll take this up this week, this issue is to keep track of it.

It is already available as keepassx2. :/ Closing.


slim option

Hi
I would like to know what the 'slim' option is for. I searched everywhere on the site but couldn't find anything.
Thanks

Hi, thank you for your enquiry. Slims are selected parts of the ontologies where you can choose certain terms of your choice within the ontology. However, there isn't any standardised way within the ontologies community about the definition of slims or what constitutes a slim therefore it's not implemented in the OLS. You can further read about slims here: https://www.ebi.ac.uk/QuickGO/help/slims
I hope this helps.

Thanks for your answer.


Enable Sellers To Decline Offers

Per this week's sprint meeting, a listing will be considered "sold out" at the DApp level if there is one valid, outstanding offer (whether accepted or awaiting acceptance). And buyers will not be able to dictate their own offer terms. So in order to enable sellers to free up listings for other buyers, they will need to have the ability to decline offers.

This has been added as part of #401


Balance not updating cancelling open orders in resolved market

f I am a market creator and have open orders on the sportsbook (offers) after the market finalizes, cancelling those open orders (works through the modal) but my balance never updates...refreshing doesnt fix either

going to push this one week due to potentially being resolved by updates on the backend/pulling out the gsn.

I would like to help with the issues.
Is there anyone I can discuss for the work?

looks correct


iscoroutinecheck check is more restrictive than documented types

Steps to reproduce
https://github.com/hikari-py/hikari/blob/c956f6c70ac9f6b12c34e0ab26d143b42f7c66fe/hikari/impl/event_manager_base.py#L278
This line errors if an object with an async __call__ is passed or if a normal function which returns a coroutine is passed even though the documented types allow both cases.

Would changing that line to iscoroutine be better? Seems to allow async def __call__ like this:
import inspect

class Foo:
    async def __call__(self):
        print("hi")

inspect.iscoroutine(Foo()()) # True

What use case would you have for this?

import inspect

class Foo:
    async def __call__(self):
        print("hi")

inspect.iscoroutine(Foo()()) # True

there's no guarantee that a function matching Callable[[Event], Coroutine[Any, Any, None] won't do stuff before yielding or have any side effects when called but not awaited though like
async def _foo_async(event: Event) -> None:
    # Do async stuff

def foo(event: Event) -> Coroutine[Any, Any, None]
    # do_time_sensitive_stuff ...
    return _foo_async(event)

(plus you'd get warnings about unawaited coroutines)

Think the best way to go around this would be to just have people pass the coroutine to call (__call__ in the case of the class).
For the other example you said, they could just do this:
async def _foo_async(event: Event) -> None:
    # Time sensitive stuff can be done in another non-async function or just here avoiding awaits
    #
    # Do async stuff

I unfortunately dont see any way of doing this, as the way inspect.iscoroutinefunction work is by checking an internal flag, nothing to do with the public facing API

Looking back at these old issues, I dont think this is something we can support nor encourage. Documentation change might be needed, but we do already error warning that the subscribed function is not a coroutine function

Ngl I still don't see why the check has to be there in the first place tbh, kinda goes against the library's normal approach of don't make the interface more restrictive or worse just for the sake of babying ppl


build 2.3.2 pushes up screen for iOS Intel XDK builds

In the latest build 2.3.2, the screen gets pushed up (the amount of space the statusbar uses)
For comparison, the previous build I was using 2.0.1 does not have this issue (attached screenshot)
tested on a iPhone 6 8.1 Intel XDK iOS Build

works for me! Thanks


Start process with ClockUtil time set by user

Needs to be reconsidered in the light of two Camunda time related bugs I work around by setting process time to a half second.


No detection or inverse results

Hi Nikki,
I am trying to replicate your model for brain tumour segmentation to explore image analysis tools and algorithms. After carefully implementing your model, I got either completely blank predictions or negated results (image had slices made in the area where tumour is absent) . I am using your pre-trained model with available weights  downloaded from this repository. Could you please help me improve my result?
Few changes I made are listed here:
1.The BasicModel() used in SegmentationModels.py seems undefined, I replaced it with SegmentationModel().

I downloaded the BRATS2015 dataset as it is. Are there any changes made to the same before using?

Thanks in advance.

@JRevati  Same problem. Once the model is saved in models/examples.json and we test we get IO error
IOError: cannot identify image file <open file '/home/adminsters/Documents/Training/HGG/brats_tcia_pat165_0001/VSD.Brain.XX.O.MR_T1c.40873/VSD.Brain.XX.O.MR_T1c.40873.mha', mode 'rb' at 0x7ff4cae90660>
But the file brats_tcia_pat165_0001/VSD.Brain.XX.O.MR_T1c.40873/VSD.Brain.XX.O.MR_T1c.40873.mha is at correct location

@build2create Can you specify where exactly do you get this error, is it in brain_pipeline.py ? . In my case the T1c file name gets a suffix "*_n" before .mha extension (after running the script n4_bias_correction.py). So while creating basically, it searches for VSD.Brain.XX.O.MR_T1c.36175**_n**.mha file instead as per the code ( t1_n4 = glob(self.path + '/*T1*/*_n.mha') in brain_pipeline ). Did you run and check the output of the same script before?

@JRevati  @naldeborgh7575  Basically this is what I did in steps
Step 1. Rann4_bias_correction(modified specifying the arguments as in brain_pipeline namely path etc and saved to respective folder with _n.mha for all T1 t1 c type)
Step 2. Ran brain_pipeline.py (uncommenting the code for slices) with norm=n4 that saved slices to n4_PNG (see the comments in the code). I did not do other 2 normalisation. Next commented that and ran the code to save the labels for ground truth## (# Doubt 1 where is that Label/ * folder used ahead?)
Step 3. Ran the Segmentation_Models.py. I replaced this     train_data = glob('train_data/**') by train_data=glob('n4_PNG/*').# (Doubt 2: Is this ok? n4_PNG contains all the n4 normalised images) The sequential model ran for 10 epochs each taking appoximately 1800 seconds
Step 4 :Swap the comment and uncommented portion for running the Testing phase. Now here i replaced first for entire folder(i.e replace tests = glob('test_data/2_*') by my path for testing folder) unfortunately that didn,t work out.So tried for single image, now I get Value error on reshape for mha image (5,240,240) and it runs for png image though. # (Doubt 3: What must be the input to tests=glob(?))
The biggest doubt after all this is use of Label/* ground truth images( I believe that this path Original_Data/Training/HGG/**/*more*/**.mha is path to ground truth) Another big doubt is if we go for loading two-path CNN Graph() is deprecated according to latest keras documentation. Simply helpless at this point. Please help.

@build2create This is what I think,
Last question first and Doubt2 : As you have mentioned, Original_Data/Training/HGG/**/*more*/**.mha is the path to ground truth, moreover, this will be used to generate ground truth which will be appended to the strip created using other 4 scanned forms of images, ex. scans = [flair[0], t1[0], t1[1], t2[0], gt[0]] in brain_pipeline . Now if you haven't used the any other normalized forms I don't think that will cause any trouble. The labels are saved in a path that you provide in save_labels() method. Also, Graph is depricated in later versions, so I used Keras 1.1.1 for the same reason. From 1.2.1 onwards, Graph() will not work. If you are planning to train the model on your own, then you need to either use the alternatives of downgrade the keras version.
Doubt 3 : input to test glob should be the folder path where you have saved preprocessed test images (similar to the training ones) from downloaded BRATS_testing folder. In my case this works. Note that you will have to use all the pipelined methoods for testing images to get the same effect and dimensions. In patch_library.py  you will find a comment where it is explicitly mentioned that the images should have shape (5*240,240) for training images which also applies to test images.
Doubt 1 :  the labels are used to feed y_train (ref. find_patches() where you provide labels folder path ) and later in calculating dice co-efficient etc.
I haven't used the two-path model, hence I am unable to comment on it, but hope the rest helps a bit.

@JRevati  Thanks for the reply. Just confirming as you said " Note that you will have to use all the pipelined methoods for testing images to get the same effect and dimensions. In patch_library.py you will find a comment where it is explicitly mentioned that the images should have shape (5*240,240) for training images which also applies to test images." This means we have to convert test images in BRATS Training set to PNG
Also another point current version of brain_pipeline.py generates n4_PNG( as folder of n4ITK normalized image see comments ). Here the dimension of each image is 1200X240 was that also in your case? Did you use modified code given by @umanghome in the Pull request section.
One last thing, you said you used pre trained model  "I am using your pre-trained model with available weights downloaded from this repository." How did you do that what are steps for directly doing testing? Or do we need to train everytime before we test?

@JRevati Please give confirmation for above question. I am really stuck here.

@JRevati I got the same problem when I corrected and ran the code. Black images. I think @naldeborgh7575 did not upload the correct model. Because this is the same problem I faced.

Okay, seems like there might be a problem with your normalization if your network is not learning

Can you post which .py file we need to run first?

@JRevati I am running the code with BRATS2013,but some problems happen to me,so I want to try the BRATS2015 dataset.I can't download it successfully, sincerely hope you can help me for the datasets

www.smir.chdownload it from here
On Friday, December 22, 2017 7:24 AM, tiantian-li<EMAIL_ADDRESS>wrote:

@JRevati I am running the code with BRATS2013,but some problems happen to me,so I want to try the BRATS2015 dataset.I can't download it successfully, sincerely hope you can help me for the datasets—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Allow new outgoing messages to be anonymized

The Phantom Remailer currently only allows incoming messages that originate from another user to be anonymized.
The flow is like this currently: User->remailer->anonymous person->remailer->User
The proposed change will allow: anonymous person->remailer->User->remailer->anonymous person
To allow for new messages to be relayed through the remailer it will need another authorization token which is separate from the signing key.
New messages could be addressed to box+token.email_address@domain, then the remailer will remove the sensitive content before remailing.
The email address needs to be encoded in a way that the @ is replaced, the URL encoded variant %40 could be used in place of the @ in outgoing messages, or the phonetic version _at_.  Both could be supported.
An example To address would be:
<EMAIL_ADDRESS>or
<EMAIL_ADDRESS>This format has the advantage that it's easy for a human to generate the correct address, the token can be a simple word or phrase, anything but the word 'anonymize'.

Relay capability is implemented as described above except how the escapes are implemented.
Here are the following escapes available:
For @: %40, _at_, and =40
For .: %2e, '%2E, =2E, and dot`
This means the following addresses are valid as the To address:
            'foo_at_example_dot_com',
            'foo=40example=2Ecom',
            'foo%40example%2ecom',
            'foo%40example%2Ecom'

The full format is like: box+<token>.foo_at_example_dot_com@domain


Use SSL without verifying certificates

I have a self-signed SSL etcd cluster running, and passing around ca.crt file is tiresome and prune to errors.
I trust my own etcd server, so I wonder is there a way to skip SSL verification which using python-etcd?

I'm sorry but allowing to skip SSL verification of cert validity is against the whole idea of how encryption works.
I personally will never, ever, add the ability to skip a check to my own library (while you have your means to do it anyways, if you really want to shoot yourself in the foot, given we use urllib3).


prevent showing scroll when not needs

Hi @JoeyAndres,
I found two extra scroll.
One in content of all page like this:

And another in SideMenu like this:

@cyclops24 There have been lots of changes involving ionSideMenu lately.

This only exist in desktop mode imo. Can you check and close this one?


Update @types/unist to latest version

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)
[X] If applicable, I’ve added docs and tests

Description of changes
Updates to latest version of @types/unist.
cc @wooorm @ChristianMurphy

version is already in range https://nodesource.com/blog/semver-tilde-and-caret/
no update is needed


installShellFiles: ensure shell completion has more than 256 bytes

Description of changes
I found an error in the completion script of tmuxp which were outputting
an error message instead of a proper script so a sanity check in the
installShellCompletion script is helpful to avoid this to happen in the
future.
After looking at possible ways to detect this, the only valid check I
found was based on the generated file size as any completion script I
could find had more than 256 bytes.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Running review to find other examples like terraform

can't nixpkgs-review it
it's not going to fit

Status?

@Artturin it is ready, I'd need a full build log to fix any breakage of packages.


[hsmtool] Add RSA operations

Support the following RSA operations:

Key generation
Key import
Key export
Encryption
Decription
Sign
Verify

Signed-off-by: Chris Frantz<EMAIL_ADDRESS>

You added me as a review but this PR is still a draft. Are you looking for a preliminary review, or did you intend to mark this PR as ready?


[changelog] Identify commit labels by authors or committers

Add an option to label commits by their authors/contributors/committers such as
release:
  github:
    overwrite: true
    changelog:
      formatted: ALWAYS
      preset: conventional-commits
      labelers:
        - label: hideme
          contributor: aalmiray

🎉 This issue has been resolved in v1.10.0 (Release Notes)


Generate LICENSES, make --branch required

Couple of items in this PR:

make --branch mandatory so that LICENSES URLs point to license text on correct branch.
initial output of LICENSES for 1.0 release

License notes:

github.com/signalfx/com_signalfx_metrics_protobuf is only used in testing. In general we need some automated mechanism to ignore anything that doesn't end up in a binary (https://github.com/istio/istio/issues/7288).
siphash is used in open-policy-agent/opa. The version we have in 1.0 uses 0.7.1 which still points to the old siphash which doesn't have a license. The new one is 0.8.2 which does have a license (https://github.com/istio/istio/issues/7287)
github.com/BurntSushi/toml has WTFPL but this is only used in helm and is transitively included through k8s.

Proposal: will delete 1 & 3 manually and fix 2 to get the right version.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ostromart
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt 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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
afe88ccb6f51a16261d305b3e153d403827eb709
link
/test istio-unit-tests

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

Do we need to include this file? its 5.3MB!

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
afe88ccb6f51a16261d305b3e153d403827eb709
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
afe88ccb6f51a16261d305b3e153d403827eb709
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

I think we should generate it during the release and not commit it in our repo. Its huge and unnecessary. Out of curiosity, are there any GPL style licenses in there?

@rshriram I think we are ok with the file size. It has huge redundancy so it zips to ~97k. git will compress it automatically per https://gist.github.com/matthewmccullough/2695758.


Typo in the Document.

Source is misspelled.

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

ID: e87d5c9d-85f5-8809-8f60-0031c59b0c97
Version Independent ID: ecc0bce6-d784-a570-e9b2-bfc96c531db5
Content: Troubleshoot outbound connections in Azure Load Balancer
Content Source: articles/load-balancer/troubleshoot-outbound-connection.md
Service: load-balancer
GitHub Login: @anavinahar
Microsoft Alias: anavin

Made a PR to address this issue.


CookieService not found when used with TestBed

Not sure what I'm doing wrong, but this fails with the angular2/systemjs/karma setup:
class mockCookieService {
}
describe('AppComponent with TCB', function () {
beforeEach(() => {
TestBed.configureTestingModule(
{
declarations: [AppComponent],
providers: [
{ provide: CookieService, useClass: mockCookieService }
]
});
});
The error I get:
26 10 2016 11:35:57.473:WARN [web-server]: 404: /base/node_modules/angular2-cookie/services/cookies.service.js
The setup was based on angular2 quick-start project: https://github.com/angular/quickstart

The issue was with karma configuration. So what's needed is:
Add to karma.conf.js:
files: [       // angular2-cookie       { pattern: 'node_modules/angular2-cookie/**/*.js', included: false, watched: false },       { pattern: 'node_modules/angular2-cookie/**/*.js.map', included: false, watched: false },
Add to karma-test-shim.js:
System.config({      packages: {         'angular2-cookie': {                   main: './core.js',                   defaultExtension: 'js'          }      },      map: {              'angular2-cookie': 'npm:angular2-cookie'      } });


Move container rename and remove to engine-api

Addresses #1879, follow up to #1983
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

Ping @vieux @dongluochen

LGTM

LGTM


[Tabs][iOS] RadListView inside TabContentItem breaks navigation

When a RadListView is put into a MDTabContentItem and the tab is opened, the user loses the ability to navigate to other tabs. The selectedIndexChanged event stops firing, and tapping other tabs only results in the ripple animation showing.
Which platform(s) does your issue occur on?

iOS
iOS 12.4, 14.4, possibly others.
Emulator

Please, provide the following version numbers that your issue occurs with:

CLI: 8.0.1
Runtime(s): `8.0.0
Plugin(s):         "nativescript-ui-listview": "^9.1.0",

Please, tell us how to recreate the issue in as much detail as possible.

In the latest copy of this repo, install the nativescript-ui-listview package, and put a RadListView in a MDTabContentItem
Use the tab bar to navigate to the tab containing the RadListView

Is there any code involved?
I've put an example in an up to date fork of this repo, under demo-ng.

@cityinspain does it work with other listviews?

@farfromrefug this specific behavior does not seem to occur with the stock N listview, but there have been other issues using the N listview with tabs, such as the animation getting stuck halfway through when switching between tabs with listviews.

I am seeing the same behavior on iOS, though in my case if I swipe to the tab with the RadListView, it fixes the issue. I can then navigate back and forth by swiping or tapping the other tabs. But if I try to navigate by tapping first, I cannot tap to the other tabs. Can still swipe though.

But if I try to navigate by tapping first

Same issue here. I might move to @nativescript-community/ui-collectionview if there aren't any issues with that one.
Did anybody try?


ARMv8-a SSWAP/DSWAP/CAXPY/ZAXPY/ZSWAP test failed.

 **[Post a bounty on it!](https://www.bountysource.com/issues/10187674-armv8-a-sswap-dswap-caxpy-zaxpy-zswap-test-failed?utm_campaign=plugin&utm_content=tracker%2F69691&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F69691&utm_medium=issues&utm_source=github).

@huazhuaz , thank you for the report.
I cannot access ARMv8 machine now. Thus, I afraid I cannot debug this issue.

Based on the above and the following msgse, it mean that these issues are caused by multi-thread?
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat1
Real BLAS Test Program Results
Test of subprogram number  1             SDOT
----- PASS -----
Test of subprogram number  2            SAXPY
----- PASS -----
Test of subprogram number  3            SROTG
----- PASS -----
Test of subprogram number  4             SROT
----- PASS -----
Test of subprogram number  5            SCOPY
----- PASS -----
Test of subprogram number  6            SSWAP
----- PASS -----
Test of subprogram number  7            SNRM2
----- PASS -----
Test of subprogram number  8            SASUM
----- PASS -----
Test of subprogram number  9            SSCAL
----- PASS -----
Test of subprogram number 10            ISAMAX
----- PASS -----
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./dblat1
Real BLAS Test Program Results
Test of subprogram number  1             DDOT
----- PASS -----
Test of subprogram number  2            DAXPY
----- PASS -----
Test of subprogram number  3            DROTG
----- PASS -----
Test of subprogram number  4             DROT
----- PASS -----
Test of subprogram number  5            DCOPY
----- PASS -----
Test of subprogram number  6            DSWAP
----- PASS -----
Test of subprogram number  7            DNRM2
----- PASS -----
Test of subprogram number  8            DASUM
----- PASS -----
Test of subprogram number  9            DSCAL
----- PASS -----
Test of subprogram number 10            IDAMAX
----- PASS -----
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./cblat1
Complex BLAS Test Program Results
Test of subprogram number  1            CDOTC
----- PASS -----
Test of subprogram number  2            CDOTU
----- PASS -----
Test of subprogram number  3            CAXPY
----- PASS -----
Test of subprogram number  4            CCOPY
----- PASS -----
Test of subprogram number  5            CSWAP
----- PASS -----
Test of subprogram number  6            SCNRM2
----- PASS -----
Test of subprogram number  7            SCASUM
----- PASS -----
Test of subprogram number  8            CSCAL
----- PASS -----
Test of subprogram number  9            CSSCAL
----- PASS -----
Test of subprogram number 10            ICAMAX
----- PASS -----
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./zblat1
Complex BLAS Test Program Results
Test of subprogram number  1            ZDOTC
----- PASS -----
Test of subprogram number  2            ZDOTU
----- PASS -----
Test of subprogram number  3            ZAXPY
----- PASS -----
Test of subprogram number  4            ZCOPY
----- PASS -----
Test of subprogram number  5            ZSWAP
----- PASS -----
Test of subprogram number  6            DZNRM2
----- PASS -----
Test of subprogram number  7            DZASUM
----- PASS -----
Test of subprogram number  8            ZSCAL
----- PASS -----
Test of subprogram number  9            ZDSCAL
----- PASS -----
Test of subprogram number 10            IZAMAX
----- PASS -----

@huazhuaz ，I am not sure. How could you compile the library? Could you try make USE_OPENMP=1?


Editing YAML

This is not exactly an issue, mostly a question. Which tools are you using to edit YAML files? So far I've been really struggling to make a valid YAML, especially when resource description grows. There is always a case of mistyped space or extra delimiter and so on. Both VS and VS Code is lacking a good yaml editing experience too. I understand that any valid JSON document is a valid YAML file, so I guess I could revert to JSON in descriptions. Unless I'm really missing something here?

I use visual studio code with this extension - https://marketplace.visualstudio.com/items?itemName=adamvoss.yaml

True, i had several problems with YAML, formatting is not one of its good points.

I use VS code too..

For VSCode this one is really good: https://marketplace.visualstudio.com/items?itemName=redhat.vscode-yaml


MagritteElementBuilder does not honor beHidden

FYI fixed in Magritte proper https://github.com/magritte-metamodel/magritte/pull/343

Thanks! We pulled in those changes last week :)


Example1

a=[11,22]
b=a
The ids of the two will be the same.
Now,
a[0]=1
The ids of the two will still be the same as both a and b are referring to the same location, changes made to a are made to b as well.
Hence,
a=[1,22]
b=[1,22]
Now if we do
import copy
c=copy.deepcopy(a)
id(a) will not be equal to id(c)

Let me know if I've missed anything?

deepcopy has not been used explicitly.


Workspace wrong PHP version

Description:
I've set up to use the PHP version 8.1 in the .env file but the workspace is updating it to 8.2 so somewhere is doing an update to the last version of PHP which is wrong.
Expected Behavior:
I expect to have the PHP version I select in the .env file.
Context information:
I'm trying to rebuild the workspace with the PHP version 8.1 but instead the PHP version installed is 8.2 hence the build is failing at the 93rd step.

Most likely you set the wrong PHP_VERSION value, not  8.1.  .

yes I did setup the correct PHP version 8.1
RUN set -xe; php -v | head -n 1 | grep -q "PHP ${LARADOCK_PHP_VERSION}."

UPDATE:
Mailparse is causing this.
apt-get install -yqq php-mailparse
When is being installed, it upgrade the PHP version. I don't know why. Since I'm not using this extension I disabled it from .env file
WORKSPACE_INSTALL_MAILPARSE=false

UPDATE: Mailparse is causing this. apt-get install -yqq php-mailparse  When is being installed, it upgrade the PHP version. I don't know why. Since I'm not using this extension I disabled it from .env file WORKSPACE_INSTALL_MAILPARSE=false

same issue here. This problem occurs since PHP 8.2 was introduced. I believe that is some problem with mailparse library install step.


Seed<PHONE_NUMBER>613, Librarian has no Relic.

Other relics of note were
Cube of Zoe Location: Tooth of Vlad
Fairy Orb Location: Sword Familiar
Fairy Scroll Location: Vanilla item
Elevator in Outer Wall: Power of Mist
No way to proceed any further on this seed unless I'm missing something?

Can you retry this? I'm not sure when you created the seed, but the shop relic glitch should be fixed now.


Merge 05-10-Refactored_index_ts_and_extracted_functions into 05-10-Installed_pretty-quick

Current dependencies on/for this PR:

main

PR #93

PR #94   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: cjoQCnypZ9JkBJPvVwMd

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: MbXiflwBblcjFMMtdGei

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: wZMVzgbgOdBD7itEYE5l


Is the goal to be able to build the docs from the main Semantic UI repo?

Is the goal to be able to build the docs from the main Semantic UI repo? Or should you only be able to build the docs from:
https://github.com/Semantic-Org/Semantic-UI-Docs
See below when gulp build-docs from root of main Semantic UI rep:
npm install
gulp build-docs

users-MacBook-Pro:Semantic-UI carlf$ gulp build-docs
[22:42:29] Using gulpfile ~/Documents/dev/open source/Semantic-UI/gulpfile.js
[22:42:29] Starting 'build-docs'...
Building Metadata
Copying examples
Copying LESS source
Building Semantic for docs
fs.js:77
      throw err;  // Forgot a callback but don't know where? Use NODE_DEBUG=fs
            ^
Error: ENOENT, open '/Users/carlf/Documents/dev/open source/docs/out/metadata.json'
    at Error (native)
users-MacBook-Pro:Semantic-UI carlf$

It's expecting two sibling folders one with each repo's contents, ui/ and docs/

So to clarify. To build the docs one must clone this repo, then clone Semantic-UI-Docs into this repo's docs/ directory?


Comment 2

This line doesn't seem to do anything.
(maybe implementation is not finished)
https://github.com/enes-usta/bgp-filter/blob/2c313a7fc6a74d74a2f16f8fc09432915f235afd/bin/BGPFilter.py#L246

Used to check if a country code is valid


Show organizations repos

add the ability to show public organizations repositories on gitprofile

It might get messy if the user has many organizations. As the user has little control over which repos to display, we are considering a new feature where users can specify the project list including organization projects.

@arifszn Any updates on this?


Test-Connection shows unwanted output when DNS not resolved on Powershell 7.0.0

Test-Connection -Quiet -Count 1 Hostname
gives output
Test-Connection: Testing connection to computer 'Hostname' failed: Cannot resolve the target name.
Expected behaviour would be
False
In a code block like
 $comps | ForEach-Object -Process { if (Test-Connection $_.Name -Quiet -Count 1) {Write-Output "$($_.Name) on"} else {Write-Output "$($_.Name) off"} }
this gives output
Test-Connection: Testing connection to computer 'Hostname' failed: Cannot resolve the target name. Hostname off
Again, the extra output is unwanted, given the Quiet flag.

🤔 This is kinda weird. We shouldn't be surfacing any hostname resolution errors...
This code path handles the hostname resolution:
https://github.com/PowerShell/PowerShell/blob/43e0e1a1c54144ae351e1541dd0965b7792d338b/src/Microsoft.PowerShell.Commands.Management/commands/management/TestConnectionCommand.cs#L640-L733
As you can see, we're catching everything in there. Not sure why you'd be getting hostname resolution errors here, unless you're using a different Test-Connection command. There are only two cases you should be getting errors from this cmdlet -- when you provide a $null hostname, or when you provide an IP address that doesn't match the requested switch (i.e., providing an IPv6 address and specifying -IPv4 or vice versa).
Just to be sure, can you grab the output from Get-Command Test-Connection so we can see what it's calling?

 Get-Command Test-Connection
CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Cmdlet          Test-Connection                                    <IP_ADDRESS>    Microsoft.PowerShell.Management

Can you post the result from $PSVersionTable as well, please, so we have any necessary OS / other info? Thanks! 😊

Oh, I see it.
https://github.com/PowerShell/PowerShell/blob/43e0e1a1c54144ae351e1541dd0965b7792d338b/src/Microsoft.PowerShell.Commands.Management/commands/management/TestConnectionCommand.cs#L680-L704
We should definitely be just returning false here.

Issue is here
https://github.com/PowerShell/PowerShell/blob/43e0e1a1c54144ae351e1541dd0965b7792d338b/src/Microsoft.PowerShell.Commands.Management/commands/management/TestConnectionCommand.cs#L575-L577

We catch the hostname resolution error in the method and call WriteError() -- there's nothing that can be done outside the method call itself other than writing false to output. The issue here is that the error still shows in -Quiet mode, so we'd need to handle that in the method.

Quiet parameter can be used in interactive session too. If user don't want to get error SilentlyContinue exists.

-Quiet should do what it says on the tin, in my opinion. 🤷♂
If it's erroring, it's not quiet.

@vexx32 btw the get host entry call is part of why the command sometimes doesn't respond to stop processing requests.

That's a very good point... Is there an async/cancellable method for that one you know of?

@vexx32 Since there's no cancel, all you can really do is something like:
private CancellationTokenSource _cancel;

// ...

private IPHostEntry GetHostEntryWithCancel(string hostNameOrAddress)
{
    var task = Dns.GetHostEntryAsync(hostNameOrAddress);
    var waitHandles = new[] { ((IAsyncResult)task).AsyncWaitHandle, _cancel.Token.WaitHandle };
    if (WaitHandle.WaitAny(waitHandles) == 1)
    {
        throw new OperationCanceledException();
    }

    return task.GetAwaiter().GetResult();
}


Localhost studio doesn't load.

Support Question
Hi,
I installed using the instructions for the docker compose on a Windows 2019 server. When I go to http://localhost:9000/studio it just keeps loading. Also, I am trying to install Query but pulling of image is going on for few hours and still can't see anything.
Please can someone help?
Thanks,
Mohit

seems to be an issue with Windows server VM. Works on Win 10 VM.


Profiles: Not clear the tree pane is vertically resizable in the Show Contents View

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:

Version: 1.75.0-insider
Commit: 727d2d32aea02baf7d13fa38787e8ed64dd858d0
Date: 2023-01-06T05:21:04.561Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0
Sandboxed: Yes

OS Version:
Mac BigSur 11.7.2

Steps to Reproduce:

Use the Default Dark theme
Create a new empty profile
Open the Show Profile Contents view
Since there is little/nothing in the Profile, the Export/Close buttons are located close to the top of the view

Install the Python extension (which installs a bunch of extensions as well)
Open the Show Profile Contents view again
Note that the visible area of the tree is the same size and the scrollbar isn't visible by default
It looks like there are only 3 extensions installed (when there are really about 9)

Now scroll the tree down with the mouse and you can see the rest of the extensions

Ideally there is a visible sash between the area where the tree is and the area that contains the Export and Close buttons, which makes it possible to expand the area with the tree.

Good catch. But in this case it seems like a bug is that the view has a wrong initial size. It takes the previous size, instead of the larger one that is needed.
But having a visible scrollbar would also help.

Yes it's a bug to resize the initial size when contents are updated. IMO invisible scrollbar is a general issue across our UX.

I can only reproduce it if I manually click on refresh button in the view after installing new extensions. Fixed this case.

@sandy081 thanks!
@chrisdias let us know if you see it again.


vscode: Add libsecret as run time dependency

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Added libsecret as runtime dependency because of recent API changes to VSCode in 1.81.0, more specific details at microsoft/vscode#190062

@BlankParticle in future, please follow the commit message convention described in https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md, especially if you're going to check the box in the template :)
@NobbZ please check this before merging :)

I simply didn't expect that the commits could been have messed up for a 10 letter or so change...

I got it successfully using gnome-keyring with this patch. But can't do it with Insiders. It still complains, that OS keyring is not available.
programs.vscode = {
    enable = true;

    package = (pkgs.vscode.override { isInsiders = true; }).overrideAttrs (oldAttrs: {
        src = (builtins.fetchTarball {
            url = "https://code.visualstudio.com/sha/download?build=insider&os=linux-x64";
            sha256 = "sha256:1725d4fzq1jnjx6ins3i6d770nyqnnjn8cdrk9gjnim2qfixhhz7";
            });
        runtimeDependencies = oldAttrs.runtimeDependencies ++ [ pkgs.libsecret ];
        version = "latest";

        buildInputs = oldAttrs.buildInputs ++ [ pkgs.krb5 ];
    });


fix: block sync on big blocks

When big blocks are received during sync, sometimes deserialization can take very long. Doing this much work in a spawn_blocking task is not ideal, so we switched to using rayon for the deserialization.
Addionally, nodes handling big BlockRequests stop responding to Ping and PeerRequest as the inbound messages are handled sequentially. This can lead to the syncing node disconnecting the peer before the blocks are deserialized. Then all outstanding BlockRequests are flushed from this peer, and when the blocks are finally ready to be processed they are thrown away as now it seems we didn't even request these blocks. Leading to an infinite loop of requesting the big blocks, dropping the peer, dropping the blocks and re-requesting.
Testing
This was tested on isonet snarkos-client-val8-client0. Ledger was cleared and node got initially stuck on blocks 189886 where a series of big (50+ MB) blocks was located. The rayon change was not enough to fix as the peer got disconnected and so the blocks were always thrown away. I had to patch 4 nodes (validator 8 and his 3 clients) to test (so they would answer to the Ping/PeerRequest messages).

A little extra context here: Tokio's blocking thread pool is convenient for shorter cpu-bound work but it isn't well suited to really heavy compute because the pool is large relative to the number of cores. Offloading heavy work to rayon will result in much better performance, especially if the workload can be parallelised (which we should be able to do for ser/de).
With that in mind, we're still investigating why deserialisation of large block responses take a long time by using an OS thread to isolate the work from any runtime  funkiness but the approach here should already improve things. Parallelisation will take a little tinkering but is also something we should consider, especially as we've seen up to a minute of deserialisation time for the largest blocks.

Well, up to a minute on our dev machines, over 2 minutes and probably more on the AWS machines.

Patch was installed on all stuck nodes on isonet today, unstuck another 11 nodes. All nodes are now progressing again towards tip.

This can be clearly seen on the graph of since I started working today:

Closing and opening on AleoNet instead


Add a grunt task that publishes directly to Launchpad.

BuildingASourcePackage
Uploading

Seems like it's a matter of having the right key installed on the right box, and running
debuild -S -sa
dput ppa:atomeditor/ppa <filename>.changes

Relevant, of course is the Ubuntu Packaging Guide

Ok. I have confirmed that the launchpad builders have no internet access. I would pre-install the node modules first, but I don't think I can do that because atom needs to build native architecture dependent packages, right? Any advice on how to approach this? Would it be possible to build atom without intenet access

Would also love to see this

Its really awesome if we have this in place. It will really help most of Ubuntu and debian members. Waiting on it. :)


Rename coredb-pg-base -> tembo-pq-slim

Rename coredb-pg-base -> tembo-pg-slim
Delete current coredb-pg-slim
Delete current coredb-pg

I think OK to merge even though ML image failure


compile / decompile emojis

I like using emojis for examples sometimes but it gets bad when you compile (just some combination of _ and numbers), so this would let allow emojis to persist between compile -> decompile:

Obviously the generated names aren't great (they're just "E" + emoji's codepoint in base 16 + "X", maybe could use $ to surround instead as that's also valid / uncommonly used anyways) but I don't know it's any worse than just _ / _2 / etc; starting as a draft though as I don't know if we'd want to take it necessarily for this reason
test build with a sample project: https://arcade.makecode.com/app/0610dda95754bed7e6df6ce7485f7d128dd0f4ea-1375337f35#pub:_XLqH792iMbks
Most of the diff is from https://github.com/microsoft/pxt/pull/7861/files#diff-b3b92c2632ab9b3508e5ea8cd2f72f4d5829fbfb8ef35ab61303cc85d61b93aeR30, which I needed to change here as it produces invalid encodings for most emojis (unicode 'astral plane' codepoints) / breaks anything above 0xffff; as far as I can tell html / xml with utf8 encoding should only ever need to encode <>"'& special anyway though so this seems fine / better anyways? e.g. https://www.w3.org/International/questions/qa-escapes#use & https://hsivonen.fi/producing-xml/#noescaping
I messed up switching between branches after my local version of the branch broke and accidentally pulled when I should have checkout'd, so I accidentally merged this into master / reverted it already D:. The commits from when I wrote it are in https://github.com/microsoft/pxt/compare/8e3181e...bb4281a, and I reverted that in https://github.com/microsoft/pxt/commit/596e90c79336f81c1a086ec05149ec8bdc544ff5, so this pr would... revert that.
edit added support for debugger view as well, and made that show unescaped names (e.g. a variable text list currently shows as text_list in arcade/live, but with this shows without the space escaped):

also fixes https://github.com/microsoft/pxt-arcade/issues/2774

i wonder if we could do a mapping to the unicode names instead? Might look a bit nicer (but would definitely be more complicated).

For using the unicode names, I believe the values should be the same in general -- that is, it's currently encoding U+1f4da as E1f4daX in the name just to make it easier to escape / not run into a case where it tries to swap a users actual intended variable name with an emoji that happens to match. I can make it U1f4da$ or something instead (just want to keep a marker on each side so it's definite which things are part of emoji and which are just letters that are valid hexchars) -- only needs changing like this: https://github.com/microsoft/pxt/pull/7861/commits/db107f0cc42c068a613aee5c8d3b0b8688586ea4

@jwunderl I meant the name of the emoji. e.g. "grinningFace" and "personInSuitLevitating"

Okay finally brought this up to date, I'm gonna try and get this in for next arcade release :) here's a test build, everything appears to still be working but I'll poke around a bit more
https://arcade.makecode.com/app/ef36c7fc259238e12d680c626d4af6c6bbf1b771-418c08602e#pub:_Rd0UJmiP0RWq

Is this PR still valid?


Charging Battery Behavior

After testing the linear_battery_demo.sdf, I noticed that the command to charge the battery actually depletes the battery even more.  Additionally, when I stop the vehicle after driving it for some time, the power_supply_status registers as FULL even though the percentage of the battery is sub 100.

Running the same example (in both Edifice and Dome), I check the ../battery/linear_battery/state topic and the percentage is 100% before sending /cmd_vel to the robot. After giving a /cmd_vel to the robot for some time and letting the percentage deplete, I'll stop the robot and start charging it. The percentage only increases for ~ 1% then stops


Weird behavior of file-loader with hash as a query string in development

Bug report
Describe the bug
Consider a custom file-loader in next.config.js:
module.exports = {
  webpack: config => {
    config.module.rules.push(
      {
        test: /\.(?:woff2?)$/,
        include: path.join(__dirname, 'assets'),
        use: [
          {
            loader: 'file-loader',
            options: {
              name: 'static/[path][name].[ext]?h=[hash:8]',
              publicPath: `${assetPrefix}/_next/`
            }
          }
        ]
      }
    )

    return config
  }
}

Note the hash of a file appended as a query string parameter, an allowed way to use it as per https://github.com/webpack-contrib/file-loader.
Now, in production, everything is fine, the file is generated correctly:
$ ls -lah ./.next/static/assets/fonts/inter
[...] inter-regular.woff
[...] inter-regular.woff2

But in development I was getting 404s for requests to these files, and I found out it was generating wrong files:
$ ls -lah ./.next/static/assets/fonts/inter
[...] inter-regular.woff2?h=bffaed79
[...] inter-regular.woff?h=aebfbb3c

I have to mention that I'm using a setup with a custom server (Koa).
To Reproduce

Set up a basic custom server,
Use the mentioned above next.config.js,
Import static files to trigger the file-loader,
Run your app in development.

Expected behavior
Correct files are generated both in development and production.
Screenshots
N/a.
System information

OS: macOS 10.14.3 (18D109)
Version of Next.js: 8.0.3

Additional context
I think the problem may be in using a custom server, although not really sure about it.

I think this is a bug in the write-file behavior of webpack-dev-middleware 🤔


removing hard-coded locales from URLs

These are causing problems for our localization process.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/antipatterns/chatty-io/index.md
:white_check_mark:Succeeded
View

docs/aws-professional/index.md
:white_check_mark:Succeeded
View

docs/aws-professional/services.md
:white_check_mark:Succeeded
View

docs/best-practices/auto-scaling.md
:white_check_mark:Succeeded
View

docs/best-practices/retry-service-specific.md
:white_check_mark:Succeeded
View

docs/checklist/dev-ops.md
:white_check_mark:Succeeded
View

docs/cloud-adoption-guide/adoption-intro/subscription-explainer.md
:white_check_mark:Succeeded
View

docs/guide/architecture-styles/event-driven.md
:white_check_mark:Succeeded
View

docs/patterns/cache-aside.md
:white_check_mark:Succeeded
View

docs/patterns/retry.md
:white_check_mark:Succeeded
View

docs/reference-architectures/sharepoint/index.md
:white_check_mark:Succeeded
View

docs/reference-architectures/virtual-machines-windows/single-vm.md
:white_check_mark:Succeeded
View

docs/resiliency/disaster-recovery-azure-applications.md
:white_check_mark:Succeeded
View

docs/resources/diagrams.md
:white_check_mark:Succeeded
View

docs/service-fabric/migrate-from-cloud-services.md
:white_check_mark:Succeeded
View

docs/service-fabric/refactor-migrated-app.md
:white_check_mark:Succeeded
View

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.

If we have 2/3 approvals, feel free to merge.

Looks good to me!


Fix typos

Issue #, if available:
Description of changes:
Fix typos found via typos --format brief
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you!

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


What should I do if I just want to parse the value

For example the following code I just want to parse the hashmap:
let v:HashMap<String, String>  = toml::from_str(r#"{biz= "b1", type= "t1"}")

Whats the use case for parsing a specific data type from within the toml spec?
Supporting something like this would require a specific way to opt-in as it would be an invalid toml document and compliance requires we error on a case like this.
toml_edit uses more specific types and we could implement FromStr for them as a way to opt-in.

main supports the following

string.parse::<toml_edit::Value>()
string.parse::<toml_edit::de::ValueDeserializer>()
toml::ValueDeserializer::new(string)`

As this seems to now be covered, I'm closing this out.  If there is something I missed, let me know and we can re-open.


become_user vault_user when modifying files in vault_home

Fixes #184

To fix #185 a tag could be introduced.

Would be nice if it was either possible to skip the Insert http(s) export in dotfile task,

To fix #185 a tag could be introduced.

Would be nice if it was either possible to skip the Insert http(s) export in dotfile task,


feat: add sell_all for route executor

Fixes https://github.com/galacticcouncil/hydration-node/issues/883

[x] test out on chopsticks

I assume the native balance test is also a fee payment asset and in this case LGTM

@jak-pan  Although it is true that the fee payment asset is indeed HDX in the native balance test, it does not have any impact as in integration tests the tx fees are not deducted directly. Anyway, I tested out sell_all on chopsticks, and working fine, first the fee is deducated then the rest is sold in sell_all

https://github.com/galacticcouncil/hydration-ui/pull/1042


The Mobs of the Living Light

About the PR
A line of mobs composed of light, invulnerable to firearms, and extremely durable to melee attacks. However, they are very vulnerable to laser weapons.
Two mobs for close combat and one for long-range fire.
Also added one of the mobs to Artifact Polymorphism.
Why / Balance
Right now, there are few mobs of one type in the game that can be used in expeditions or events. These mobs will provide another choice, perhaps in the future they can be used in night expeditions (they glow brightly, so it is more beautiful).
Media
https://github.com/space-wizards/space-station-14/assets/128169402/5a3abbf7-57b2-4176-b40e-f7ca095ab752

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

:cl: Nimfar11

add: Adds three new mobs from the Living Light family

these are rly cool, i like the work put into these

something about the sprite is odd. Not sure if its the clarity, color or shape but yeah idk.

I like it
But sprite pallete is strange

I'll think about changing the colour scheme.


chore: bdd-gh-1584443103 to 0.0.1

chore: Promote bdd-gh-1584443103 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


Add resource conflict Kuttl tests

Add Kuttl tests to check for resource conflicts with knative service, deployment and statefulsets.

Related issue https://github.com/WASdev/websphere-liberty-operator/issues/442

Delivered to common test repo by https://github.ibm.com/websphere/operators/pull/50


#20988 move foreign table to separate folder in the navigator tree

A separate folder was created for PG (starting version 8.4) and other forks that support foreign data wrappers.
You will only see Foreign Tables in the tree if you see foreign wrappers and servers.
Cockroach, Redshift, and Yugabyte - do not support it.
Also, external tables in Greenplum 7 will be in the Foreign Tables folder if the view pg_catalog.pg_exttable is not readable.
External tables in Greenplum 7 will be in the External Tables folder if the view pg_catalog.pg_exttable is readable.
Also, Foreign tables can have only columns, constraints, and dependencies. It is expected behavior.
So, please check that new folders are visible (for databases support it) and that tables inside Foreign or External folders do not have duplicates in the "Tables" folder.

verified


Telekinesis balancing

If you can find a way to get hitbox size, making mobs harder to restrain based on their size and other stats, like their melee damage or their speed, could balance it out and prevent you from doing really OP things like pinning bosses.
Oh, and you'll probably want to make it possible to break the containment with a teleport.

Player, endermen, modded mobs...
Oh, and when you find out how to pull hitbox size, mind letting me know? I need to do that for my own ballistics damage calculations.

Remember to tell Nianor! xD

That'll come with telekinesis levels.

That'll come with telekinesis levels.


Multiple Directives with Same Type on Same Element

Having multiple directives on the same element that have the same type, for example:

The only prop will be m-on:"click:otherAction", because the other property is overwritten.
A solution would be to store props as an array inside, only for directives, and execute for all of them.

Update: The new syntax with an argument (like m-on:click="method:) will be added in v0.7.0


non-system libraries in linker flags: -lboost_program_options

Android NDK: WARNING:/project/ai/scratch01/aargao01/android_simulator/AIPU_simulator_design/jni/Android.mk:simulator: non-system libraries in linker flags: -lboost_program_options     Android NDK:     This is likely to result in incorrect builds. Try using LOCAL_STATIC_LIBRARIES     Android NDK:     or LOCAL_SHARED_LIBRARIES instead to list the library dependencies of the     Android NDK:     current module
In my Android.mk, I try to link this library, but it gives me this warning, the executable file can be generated. Someone can give me some  tips? thanks

anyone faces this before?


Integrated

Fully integrated system in docker compose.
To setup: docker compose up
To access the site: visit localhost:5000

lgtm!


kvdb: concurrency issue tracker

After merging https://github.com/lightningnetwork/lnd/pull/5484 we'll need to make sure that any DB code is aware of the reset/retry mechanism.
This issue is meant to keep track of all the possible problems that need to be solved before 0.14.0 can be released.

[ ] failIdx in batch/batch.go (https://github.com/lightningnetwork/lnd/pull/5484#discussion_r674900458)

Linking https://github.com/lightningnetwork/lnd/pull/5643#discussion_r692703535
"I had an idea that could help with that: modify the code to artificially always run the code inside each transaction twice. It is then as if everything needs a retry. Itests, bench, etc, should still work."

The other issue that came up in #5643 is that callers of query functions with a callback such as ChannelGraph.ForEachChannel (and probably several others) should be prepared to receive duplicates if the read transaction is retried.
Linking https://github.com/lightningnetwork/lnd/pull/5484#discussion_r682943762 about retries of a read-only tx.


Sub-issues support

Creating Linear sub-issues in a parent issue should be mapped to a parent GH issue with children sub-issues linked by a task list

Hey @leog, great idea! I don't personally have the bandwidth to add this (would require some work ensuring task list doesn't show up in Linear) - would you want to give it a try? Otherwise I might close it as stale

@tedspare sure, I can take it as a side project 👍


Drop dependency on httpoison, use hackney directly

This removes one dependency from the project, and I don't think it complicates it in any significant way.

I looked at this quickly on my phone and it seems good to me! I'll have to take a closer look on Friday when I have a bit more time but I think it'll be good to merge as-is. Thanks!

Just one question about using a list as the URI. Other than that, this looks great!

@paulcsmith updated

Thank you! :D Yay for one less dependency


squid:S1155, squid:S2786 - Collection.isEmpty() should be used to tes…

This pull request is focused on resolving occurrences of Sonar rule squid:S1155, squid:S2786 - Collection.isEmpty() should be used to test for emptiness, Nested "enum"s should not be declared static
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1155
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2786
Please let me know if you have any questions.
M-Ezzat

Sweet!