update userguide matrix via element

IOS screenshots are missing
will close #2361

its nice to see inline srcset however what are we doing here? future proofing -site for a time when jpgxl's are more widely supported when its going to be replaced shortly? if we had translations set up we would be presenting translators with alot of raw html. replacing the screenshots with new avifs is perfectly acceptable.
I'd much rather see your efforts and knowledge put into how we should be displaying images in astro / discussing fallbacks/image types with the new designer/dev.

future proofing -site for a time when jpgxl's are more widely supported when its going to be replaced shortly?

yes, JPEG XL has more features and looks more freer format*, that's why I preferred over AVIF.
In my testings:
AVIF performing better for more lossy pictures
JPEG XL performing better for less (or lossless) pictures
there's more detailed comparison: https://cloudinary.com/blog/time_for_next_gen_codecs_to_dethrone_jpeg

converting to ready for review, IOS screenshots can be fixed with different PR

NACK - remove the srcset busy work.


Onboarding Animation Broken

Describe the issue
Please provide a general summary of the issue you're experiencing
The onboarding badge animation no longer animates the badge position.
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:

Expected behaviour
Tell us what should happen
The badge should animate to the new position along with the onboarding circle.
Actual behaviour + errors
Tell us what happens instead including any noticeable error output (any messages
displayed on-screen when e.g. a crash occurred)

App Version:
iOS Version:
Device: (if applies)

Any extra information that might be useful in the debugging process.

After some investigation I've been able to identify what is causing the badge to be broken.
Previously the badge images were defined:
var image: Image {
        switch self {
        case .shield: return Asset.Assets.Icons.shield.image
        case .list: return Asset.Assets.Icons.list.image
        case .person: return Asset.Assets.Icons.profile.image
        }
    }

somebody changed it to
 @ViewBuilder var image: some View {
        switch self {
        case .shield:
            Asset.Assets.Icons.shield.image
                .resizable()
                .renderingMode(.none)
        case .list:
            Asset.Assets.Icons.list.image
                .resizable()
                .renderingMode(.none)
        case .person:
            Asset.Assets.Icons.profile.image
                .resizable()
                .renderingMode(.none)
        case .error:
            ErrorBadge()
        }
    }

and that is the root cause of the issue. I'm pretty sure the issue was present even before onboarding enhancements task was done. At this point I don't know why it's broken as well as how to solve it other than getting back to previous version and setting
.resizable()
.renderingMode(.none)

manually like before


Bug: Allow IPython to work with Contract methods / names

Overview
Allow IPython to work with Contract methods / names
Specification
When using ape console, I should be able to tab-complete on a contract to see the methods and such. This feature is currently missing.

See https://ipython.readthedocs.io/en/stable/config/integrating.html

This works.

Would be useful to do the same for method args. Not sure how to do that with IPython though


Use xdg-open but not Google Chrome in view-cover

Let's npm i opn --save-dev and use the opn binary as that's cross platform :)

Close in favor of #145


MaintenanceController doesn't check if currently used container runtime is supported by new image

How to categorize this issue?

/kind bug
/priority 3
What happened:
The maintenance controller selected a new MachineImageVersion during maintenance which doesn't list the cluster's container runtime in the CloudProfile
What you expected to happen:
When picking a new version, support for the currently used container runtime is taken into account. It could happen, for example, that future versions of an image drop support for the docker runtime. In this case, these versions must not be selected for clusters which use the docker container runtime.
How to reproduce it (as minimally and precisely as possible):

Deploy a Shoot using containerd using gardenLinux v184.0
adapt the CloudProfile:

set gardenLinux v184.0 to deprecated with an expirationDate in the past (e.g. 2021-06-30T23:59:59Z)
set gardenLinux v318.8.0 to supported

trigger shoot maintenance immediately with ./hack/usage/shoot-operation <shoot-name> <namespace-name> maintain

Anything else we need to know?:
Right now this is probably not that big of an issue, as we haven't seen events of container runtime support being removed. This is likely to happen in the future, though, if we e.g. decide to drop support for docker at some point in time.
Environment:

Gardener version: 1.26.0
Kubernetes version (use kubectl version): 1.21.0
Cloud provider or hardware configuration: gcp
Others:

Related: I can also do this change manually! Updating the Shoot spec, such that my worker's MachineImageVersion refers to a version which doesn't support the currently selected runtime.
For example:

Create your shoot with gardenlinux and container runtime containerd
Then update it to use suse-chost (which doesn't support containerd yet and therefore doesn't have it in its list of supported runtimes in the CloudProfile
This update should be rejected during validation, but it is not!

/assign
/assign voelzmo


[SPARK-40166][SPARK-40167][PYTHON][R][SQL] Expose array_sort(column, comparator) in Python and R

What changes were proposed in this pull request?
This PR exposes array_sort(column, comparator) in Python and R.
Why are the changes needed?
Feature parity.
Does this PR introduce any user-facing change?
New signature in Python and R APIs.
How was this patch tested?

New doctest, manual testing (Python)
New unit test (R)

cc @HyukjinKwon FYI

Merged to master.
Thanks @HyukjinKwon and @zhengruifeng!


Customizable VC Metadata

As a VC issuer, I want to be able to customize the VC document headers to add optional name/value pairs such as "title", "description", "term-of-use", etc.
The VC issuer can already add any parameter to the encrypted content of a credential. This request is to add values to the cleartext envelope of the credential.
When resolving the VC, any member of the public would be able to see the new "title", "description", "terms", etc. contained in the headers.

I'll look into DIF, W3C and other practitioners to see how credential headers are used in the industry.


Contributing improvements

Description
As someone trying to get the test suite running on macOS, the steps in the contributing guide and the test-utils and suite itself could be improved to make contribution easier.
This mainly aims to improve the ease of contributing to the repo on macOS. Whilst Docker on mac does support host networking Docker on mac doesn't support host networks, docker itself is run in a virtualised environment which means the host network is actually to that VM and not to the local host machine. This means that the ports do not actually get exposed as expected.
This PR does:

Documents the need for npm v7+ in the contributing guidelines
Adds titles to the before hooks that bring up docker containers (now if these fail during the test run it's easier to see what failed)
Improve the docker provisioning in the test suite so port forwarding works on macOS.

Outstanding items:

[ ] The docker cluster tests do not work locally on a mac still (investigating)

Checklist

[x] Does npm test pass with this change (including linting)? Not locally
[x] Is the new or changed code fully tested? N/A
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)? N/A

The clusters aren't working locally on a mac for me. Presumably because each instance of the redis docker can't communicate with the others when they are using traditional port-forwarding and the host network option doesn't work on mac either.
I'm no Docker expert and I've spent a bit of time looking into this and I'm not sure there's a good way to get it working as currently built. I'd have thought using docker-compose would have been a better solution (though admittedly more rigid) but at least it would work cross-platform rather than linux only as is the case.

@dhensby It should work now, wanna test it?

Codecov Report

Merging #1792 (ccf5011) into master (24f3e3f) will decrease coverage by 4.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1792      +/-   ##
==========================================
- Coverage   95.04%   90.84%   -4.20%
==========================================
  Files         353      353
  Lines        3311     3311
  Branches      407      407
==========================================
- Hits         3147     3008     -139
- Misses         79      216     +137
- Partials       85       87       +2

Impacted Files
Coverage Δ

packages/client/lib/commands/HELLO.ts
18.18% <0.00%> (-81.82%)
:arrow_down:

packages/client/lib/commands/ACL_CAT.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/ACL_GENPASS.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/ZRANGESTORE.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/GETEX.ts
21.42% <0.00%> (-78.58%)
:arrow_down:

packages/client/lib/commands/LPOS_COUNT.ts
22.22% <0.00%> (-77.78%)
:arrow_down:

packages/client/lib/commands/ACL_LOG.ts
28.57% <0.00%> (-71.43%)
:arrow_down:

packages/client/lib/commands/COPY.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/LPOS.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/ZINTER.ts
36.36% <0.00%> (-63.64%)
:arrow_down:

... and 38 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24f3e3f...ccf5011. Read the comment docs.

Hi @leibale - thanks so much for looking into this. I'm afraid that the testWithClient tests are still failing, presumably because the docker containers still can't communicate with each other :(

I've tried adding macos (and windows too) to the build matrix as that might make things easier to see if it's working/fixed


Adding new product-centric writing rule

Based on this How to put the customer first in your sentences article and a slack message from Ingrid, this rule flags "product-centric" sentence constructions suggesting that you rewrite the sentence in a more active and direct style.

@rolfedh @michelle-purcell Any objections or thoughts?


crowdin github action fails due to folder permission

Describe the bug
Hi all, thanks for your action. It seems a change done somewhere may be in Github runner make my action fails due to to fatal: unsafe repository ('/github/workspace' is owned by someone else)`
this job was working until yesterday
To Reproduce
action is configured with
    - name: Crowdin sync
      uses<EMAIL_ADDRESS>      with:
        upload_sources: true
        upload_translations: true
        download_translations: true
        localization_branch_name: 'l10n_main'
        create_pull_request: true
        project_id: ${{ secrets.CROWDIN_PROJECT_ID }}
        token: ${{ secrets.CROWDIN_PERSONAL_TOKEN }}
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        CROWDIN_PROJECT_ID: ${{ secrets.CROWDIN_PROJECT_ID }}
        CROWDIN_PERSONAL_TOKEN: ${{ secrets.CROWDIN_PERSONAL_TOKEN }}

my action log is
Run<EMAIL_ADDRESS>/usr/bin/docker run --name bcf090af149a5ed5c4402b5b32f[26](https://github.com/bonitasoft-presales/presales-localization/runs/6002923363?check_suite_focus=true#step:4:26)78004ace_baa940 --label 2bcf09 --workdir /github/workspace --rm -e GITHUB_TOKEN -e CROWDIN_PROJECT_ID -e CROWDIN_PERSONAL_TOKEN -e INPUT_UPLOAD_SOURCES -e INPUT_UPLOAD_TRANSLATIONS -e INPUT_DOWNLOAD_TRANSLATIONS -e INPUT_LOCALIZATION_BRANCH_NAME -e INPUT_CREATE_PULL_REQUEST -e INPUT_PROJECT_ID -e INPUT_TOKEN -e INPUT_UPLOAD_SOURCES_ARGS -e INPUT_UPLOAD_LANGUAGE -e INPUT_AUTO_APPROVE_IMPORTED -e INPUT_IMPORT_EQ_SUGGESTIONS -e INPUT_UPLOAD_TRANSLATIONS_ARGS -e INPUT_DOWNLOAD_LANGUAGE -e INPUT_SKIP_UNTRANSLATED_STRINGS -e INPUT_SKIP_UNTRANSLATED_FILES -e INPUT_EXPORT_ONLY_APPROVED -e INPUT_PUSH_TRANSLATIONS -e INPUT_COMMIT_MESSAGE -e INPUT_PULL_REQUEST_TITLE -e INPUT_PULL_REQUEST_BODY -e INPUT_PULL_REQUEST_LABELS -e INPUT_PULL_REQUEST_BASE_BRANCH_NAME -e INPUT_DOWNLOAD_TRANSLATIONS_ARGS -e INPUT_ADD_CROWDIN_BRANCH -e INPUT_NEW_BRANCH_TITLE -e INPUT_NEW_BRANCH_EXPORT_PATTERN -e INPUT_NEW_BRANCH_PRIORITY -e INPUT_DELETE_CROWDIN_BRANCH -e INPUT_CROWDIN_BRANCH_NAME -e INPUT_IDENTITY -e INPUT_CONFIG -e INPUT_DRYRUN_ACTION -e INPUT_GITHUB_BASE_URL -e INPUT_GITHUB_API_BASE_URL -e INPUT_GITHUB_USER_NAME -e INPUT_GITHUB_USER_EMAIL -e INPUT_GPG_PRIVATE_KEY -e INPUT_BASE_URL -e INPUT_BASE_PATH -e INPUT_SOURCE -e INPUT_TRANSLATION -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_RUN_ATTEMPT -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_REF_NAME -e GITHUB_REF_PROTECTED -e GITHUB_REF_TYPE -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e GITHUB_STEP_SUMMARY -e RUNNER_OS -e RUNNER_ARCH -e RUNNER_NAME -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/presales-localization/presales-localization":"/github/workspace" 2bcf09:0af149a5ed5c4402b5b[32](https://github.com/bonitasoft-presales/presales-localization/runs/6002923363?check_suite_focus=true#step:4:32)f2678004ace
STARTING CROWDIN ACTION
UPLOAD SOURCES
✔  Fetching project info
✔  File 'app-Ticketing/localization.json'
✔  File 'app-HRManagement/localization.json'
✔  File 'showroom-cloud/localization.json'
✔  File 'app-InvoiceManagement/localization.json'
UPLOAD TRANSLATIONS
✔  Fetching project info
✔  Translation file 'app-HRManagement/localization_es-ES_crowdin.json' has been uploaded
✔  Translation file 'app-Ticketing/localization_fr-FR_crowdin.json' has been uploaded
✔  Translation file 'showroom-cloud/localization_es-ES_crowdin.json' has been uploaded
✔  Translation file 'app-Ticketing/localization_es-ES_crowdin.json' has been uploaded
✔  Translation file 'app-InvoiceManagement/localization_es-ES_crowdin.json' has been uploaded
✔  Translation file 'app-HRManagement/localization_fr-FR_crowdin.json' has been uploaded
✔  Translation file 'app-InvoiceManagement/localization_fr-FR_crowdin.json' has been uploaded
✔  Translation file 'showroom-cloud/localization_fr-FR_crowdin.json' has been uploaded
DOWNLOAD TRANSLATIONS
✔  Fetching project info
✔  Building ZIP archive with the latest translations
✔  Building translation (100%)
✔  Downloading translations
✔  Extracting archive
✔  Extracted: 'app-HRManagement/localization_es-ES_crowdin.json'
✔  Extracted: 'app-HRManagement/localization_fr-FR_crowdin.json'
✔  Extracted: 'app-InvoiceManagement/localization_es-ES_crowdin.json'
✔  Extracted: 'app-InvoiceManagement/localization_fr-FR_crowdin.json'
✔  Extracted: 'app-Ticketing/localization_es-ES_crowdin.json'
✔  Extracted: 'app-Ticketing/localization_fr-FR_crowdin.json'
✔  Extracted: 'showroom-cloud/localization_es-ES_crowdin.json'
✔  Extracted: 'showroom-cloud/localization_fr-FR_crowdin.json'
CONFIGURATION GIT USER
fatal: unsafe repository ('/github/workspace' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /github/workspace

Possibly related to https://github.blog/2022-04-12-git-security-vulnerability-announced/
Similar issue in the checkout action https://github.com/actions/checkout/issues/760

Thank you very mush for this quick fix.
version 1.4.8 works also with git v2.35.1


fix discrepency between two sets of heading lists when updating scroll state

Preview environment created!
The preview environment has been deployed to the cluster.
Client URL: https://pr-1122--epp.elifesciences.org
Storybook URL: https://pr-1122--epp-storybook.elifesciences.org
Note: If this is a new environment, it can take a few minutes for DNS to update.

Preview environment images deployed!
The preview environment has been updated:
client image tag: preview-1122-4ede98aeb0baab20ed1d021727cea511972dac3e-20230905.1455
storybook image tag: preview-1122-4ede98aeb0baab20ed1d021727cea511972dac3e-20230905.1455
Client URL: https://pr-1122--epp.elifesciences.org
Storybook URL: https://pr-1122--epp-storybook.elifesciences.org
Note: Updated images can take a few minutes to roll out.

Worked on in mobbing, preview env shows the fixes, so merging.


Rare assertion failure in tests

While spelunking in our tests and trying things, I've spotted the following assertion failure locally:
00:00:17 thread 'main' panicked at /home/dpc/lab/fedimint-wrk-1/devimint/src/tests.rs:1473:17:
00:00:17 assertion `left == right` failed

https://pastebin.com/U8Apii3B
You probably need to copy&paste to find the difference so to make it easy:

I don't think it's my chances that caused it, but hard to be certain.

Not so rare anymore. https://github.com/fedimint/fedimint/actions/runs/8367742212/job/22910691287?pr=4624

Here's a nice diff https://www.diffchecker.com/7xjCDzEQ/


Sliding testimonials component

This section will keep on rotating on its own improving the UX of the page.

I can work on it if you assign it to me with the Hacktoberfest tag

@shreyansh1410 Can you please explain what you are trying to do.

It will move like a horizontal slideshow


Update the stale action configuration

The idea is to mark issues as stale only if they haven't been triaged for 30 days. Once an issue is triaged, we have to remove its needs-triage label.
We consider an issue as valid (and not monitored by the stale action):

if it doesn't have the needs-triage label
it was not closed while it was triaged.

The following PR:

Adds an action that labels automatically any opened/reopened issue with the needs-triage label
Updates the stale action to monitor only issues labeled with the needs-triage label.
Bumps the time for closing to 30 days

Alternatives:

Another option we were thinking about is having a tag which exempts issues from this action.
List of available options - https://github.com/marketplace/actions/close-stale-issues#all-options

I wasn't particularly sure what is the best approach for us here, so suggestions are welcomed!
Fixes https://github.com/stacklok/minder/issues/1721

@rdimitrov I was thinking of a different strategy, let me know if it makes sense.
In the spring team we had something similar, but it was a "waiting for feedback" bot, rather than a stale bot.
If someone submitted an issue that we needed more information on, then we would an a "waiting-for-feedback" label.
Then, if an issue had that label for more than X days, we would send a feedback reminder, and then after Y days we would close it.
I think we can reuse what we have here, and just rename the label and change the message. The bot would only apply to issues with the label "waiting-for-feedback". And the message would warn the owner that their issue will be closed if they don't provide feedback.
I prefer the "waiting for feedback" approach because with the stale bot we might close an issue that a contributor opened, just because we haven't gotten around to triaging it, which would look bad on us.

I like @eleftherias suggestion; the messaging also sounds friendlier for contributors.

@eleftherias @JAORMX - updated the PR based on your feedback 👍


berkeley-schema-fy24: Restore missing derived files (e.g. nmdc_schema/nmdc_materialized_patterns.yaml)

This is a recurrence of https://github.com/microbiomedata/nmdc-schema/issues/2150

This was added to the sprint which ends today. I'm moving this to the next sprint.  @turbomam @eecavanna

This one also hasn't been touched in 2 weeks
Removing from sprint, adding backlog label
@turbomam @eecavanna

@eecavanna resolved this issue for the most recent release.
@turbomam thing to sort out is these derived files shouldn't need to be created manually.
Requesting help to decide if this needs tweeks.
@eecavanna . didn't need tweeks last time and is in favor of closing this issue.

When I did that most recent release of berkeley-schema-fy24, I did not have to include any special options to $ git add when adding the derived files. The derived files are not currently listed in .gitignore. So, I don't expect this issue to recur (in that fork). Closing.


chore: pin shared composite gha versions

close this pr for this:
https://github.com/smartcontractkit/chainlink-functions/pull/4


Use<EMAIL_ADDRESS>from npm

What does this change?
Test PR for<EMAIL_ADDRESS>

Closing in favour of https://github.com/guardian/commercial/pull/1241


How do I filled up form auto

When asking a question please also include where you looked for an answer (so we can update the documentation if needed).

Sorry, I don't know what you said. Please provide more details.

@mnsshawnbd-hub  We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


⚠ Bofrost.be has degraded performance

In a4fe23d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2012 ms

Resolved: Bofrost.be performance has improved in a886302.


Request: Inline text highlighting

Outlook partners are asking for a way to highlight a portion of a TextBlock. There is currently no way to do so.

Asks

Outlook partners

Shipped in 1.2!

Does the highlighting color change when the user is in high contrast mode? or even dark mode? and how do I control which color it changes to?


Automatic Group name extraction from mzML/h5 file names

Currently, the process for LCMS data analysis is something like this:

mzML files are uploaded to NERSC
files are converted to HDF5 and added to the MySQL DB
a file list is dumped out in CSV format, and provided to the researcher
the researcher goes through the file list, and fills in the "group names"
the file is fed back into the database, and the data is retrieved to generate chromatogram plots

As the process for this consists of the user going through the list of files, and just grouping them based on the file name, this process could be easily automated, which would remove steps 3 and 4, and allow step 5 to happen automatically following step 2.

@benbowen: This idea came from my discussion last week with Leslie about her workflow.


Upgrade Dataflow worker

R: @tgroh

Run Dataflow RunnableOnService

LGTM pending green

retest this please

retest this please

Dataflow runnable on service passed, and the Maven clean install failed only after everything succeeded but uploading to coveralls failed (s3 outage?).


Fixes the draw clip bounds issue visible on ScrollView.

Fixes #1713.

May be this commit https://github.com/migueldeicaza/gui.cs/pull/1714/commits/c8db4ff88b98e25678ca4e8598ce3a49ecade41a fixes #1712.


Increase indent when aligning multiline keywords in *** Settings ***

This code:
*** Settings ***
Suite Setup    Some Keyword
...            ${arg}
...            ${arg2}

could become
Suite Setup    Some Keyword
...              ${arg}
...              ${arg2}

It could be by default or enabled through special flag.

Configurable through argument_indent (default: 4). To disable this feature it should be enough to set argument_indent to 0.


Could not find 'phpcbf'. The program may not be installed. using WordPress Coding standards

Description
Hi, as a part of setting up my development environment for WordPress developing, I installed atom-beautify to reformat my code automatically.
I have installed everything correctly according to the instructions at the WordPress coding standards github page: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards
with composer which also bundled phpcs and phpcbf and configured everything correctly..
After i went and attempted to configure atom-beautify to use the phpcbf, i get the following Error:

Could not find 'phpcbf'. The program may not be installed.
See phpcbf installation instructions at https://github.com/Glavin001/atom-beautify#beautifiers or go to https://github.com/squizlabs/PHP_CodeSniffer#installation
You can configure Atom Beautify with the absolute path to 'phpcbf' by setting 'Executable - PHPCBF - Path' in the Atom Beautify package settings.
Your program is properly installed if running 'where.exe phpcbf' in your CMD prompt returns an absolute path to the executable.

I have added the path to the file to the environmental variable and I have set it under Extensions > phpcbf.
I have also set the phpcbf to be the default coding beautifier and the  WordPress as the standard under PHP section.
I have made sure phpcbf installed correctly by running it from the command prompt (Windows 10)
Input Before Beautification
This is what the code looked like before:
<?php
if ( !defined( 'ABSPATH' ) ) exit;

if ( !function_exists( 'hestia_child_parent_css' ) ):
    function hestia_child_parent_css() {
        wp_enqueue_style( 'hestia_child_parent', trailingslashit( get_template_directory_uri() ) . 'style.css', array( 'bootstrap' ) );
    }
endif;
add_action( 'wp_enqueue_scripts', 'hestia_child_parent_css', 10 );

Expected Output
The beautified code should have looked like this:
<?php
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}
if ( ! function_exists( 'hestia_child_parent_css' ) ) :
	function hestia_child_parent_css() {
		wp_enqueue_style( 'hestia_child_parent', trailingslashit( get_template_directory_uri() ) . 'style.css', array( 'bootstrap' ) );
	}
endif;
add_action( 'wp_enqueue_scripts', 'hestia_child_parent_css', 10 );

Actual Output
did not make any changes
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom and the computer to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues, all of them were about php-cs-fixer and none had the right solution
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.

anyone has some insights?

@szeck87  Well Thank you for making me check that again.. after running debugger tools i managed to understand I was lacking the .bat extension at the end of the path, which i thought was unnecesary..
So for me the solution was to add phpcbf.bat instead of just phpcbf at the end of the path.. only took me 4 days to figure that out :)

Ah, yea for Windows I think you have to.  Sorry I missed that part at the end of your description.

Hi guys, I have the same problem. I'm trying to make Atom Beautify work with WordPress files, because otherwise working on it is unthinkable, but the Atom gives me the same notice it gave to Vertigoooo.
So far the steps I have done are:

Atom Beautify installation (which works correctly with other formats);
installation of PHP_CodeSniffer;
selected as Default Beautifier PHPCBF in Atom Beautify.

Unfortunately I'm a beginner so I do not know if the problem is a missed passage or setting, etc. ..
Can someone help me? Tell me what I can do to make you understand the problem better.
Thank you so much.

@meshuu which way did you download/install PHPCBF?

@meshuu those two files you curl'ed should be somewhere on your Mac now, depending on which directory you were in when you did so.  Copy the path to the phpcbf.phar file and paste it into Atom Beautify's settings.  It would be under Executables -> PHPCBF -> Binary/Script Path.  Try using Atom Beautify against a PHP file after doing that.

I did as you suggested:

but the program still gives me the same error.
Maybe something is still missing?

  Under settings, there is an Executables heading just below General.

Under that is PHPCBF where you want to put that path.  Also I don't think you need the ~ at the beginning, and should be .phar at the end, without the y.
Make sure it's the full path to phar file and that both phar files you downloaded are in the same directory.

You're right, I realized after making the screen that there were errors in the syntax. I proceeded to correct and I entered the path in the right field.
The previous error message no longer appears, but another one appears:

As you suggested, this file is also in the same folder as the file: "phpcs.phar".
However @ szeck87 thanks again for your interest, I'm grateful. I hope we can solve.

@meshuu please run the debug command in Atom for Atom Beautify.  See here for instructions:  https://github.com/Glavin001/atom-beautify/blob/master/docs/troubleshooting.md#how-to-create-debugmd-gist.  Paste the link to the gist here in this issue.

@szeck87 Here is the link: https://gist.github.com/meshuu/633921ba8793287a22aa32821682c433

@meshuu please do the debug command against a php file.

@szeck87 I corrected the gist.

Hmm, looks like it is running only the /path/to/file/phpcbf.phar file when it should be running php /path/to/file/phpcbf.phar.  Likely due to an update to phpcbf that was never accounted for by atom-beautify.
Since you're on a Mac, I would recommend just using the Homebrew version.  If you don't have Homebrew installed, directions are here:  https://brew.sh/.  Then run brew install php-code-sniffer.  Remove whatever you pasted into the path in Atom Beautify settings and it should work after all that.

@szeck87 I cleaned up all the paths and installed php-code-sniffer via Brew.
Everything seems to be successful because when I try to perform the beautification no error message appears, but not after the appearance of the analysis pop-up nothing happens, so it does not perform any beautification.
Around I read that you need other tools to make the beautification of a WordPress file, like WordPress-Coding-Standards that I do not understand how to install.
So it may depend on the fact that essential packages are missing that I have not installed?
Also when opening a WordPress file this is not recognized as such, at least it would seem from here:

However I did not think it was so complicated to make the beaufication work with a WordPress file.


Libc++.so.1 (llvmtools) requires libatomic.so.1

llvmtools' libc++.so.1 from libcxx should NOT rely on libatomic.so.1, which is provided by GCC.
$ ldd /llvmtools/lib/libc++.so.1
        ldd (0x7fb67cb49000)
        libc.so => ldd (0x7fb67cb49000)
Error loading shared library libatomic.so.1: No such file or directory (needed by /llvmtools/lib/libc++.so.1)
        libunwind.so.1 => /llvmtools/lib/../lib/libunwind.so.1 (0x7fb67ca6b000)
        libc++abi.so.1 => /llvmtools/lib/../lib/libc++abi.so.1 (0x7fb67ca0f000)

This will require to build LLVM for llvmtools with a LD_LIBARARY_PATH hack

https://reviews.llvm.org/D43509?id=135044 ?

hmmmmmm
http://llvmweekly.org/issue/336

Re-built llvmtools and made sure libcxx did not require libatomic.
now llvm/clang for llvmtools fails to build:
FAILED: include/llvm/IR/Attributes.inc
cd /mnt/cmlfs/sources/llvm/build && /mnt/cmlfs/sources/llvm/build/bin/llvm-tblgen -gen-attrs -I /mnt/cmlfs/sources/llvm/include/llvm/IR -I /mnt/cmlfs/sources/llvm/build/include -I /mnt/cmlfs/sources/llvm/include /mnt/cmlfs/sources/llvm/include/llvm/IR/Attributes.td --write-if-changed -o include/llvm/IR/Attributes.inc -d include/llvm/IR/Attributes.inc.d
/bin/sh: /mnt/cmlfs/sources/llvm/build/bin/llvm-tblgen: No such file or directory
[220/3316] Building CXX object lib/BinaryFormat/CMakeFiles/LLVMBinaryFormat.dir/MsgPackDocument.cpp.o
ninja: build stopped: subcommand failed.

hmmmmmm
http://llvmweekly.org/issue/336

Looks like for libcxx, I’ll have to set a flag to disable libatomic

hmmmmmm
http://llvmweekly.org/issue/336

Looks like for libcxx, I’ll have to set a flag to disable libatomic

How?

Looks like I’ll have to patch libcxx/cmake/Modules/CheckLibcxxAtomic.cmake

Although it’s strange that after rebuilding llvmtools... llvmtools’ Libc++.so.1 no longer requires libatomic.
not sure if it matters that PATH=/cgnutools/bin:/llvmtools/bin:/bin:/usr/bin

Commenting this will disable libatomic detection for Linux.
projects/libcxx/cmake/config-ix.cmake
107 #  check_library_exists(atomic __atomic_fetch_add_8 "" LIBCXX_HAS_ATOMIC_LIB)

I think atomic detection is not from libcxx for the first time. Basically from libcxxabi? which produces atomic header.

I wonder how atomic detected when building libcxx

I probably will include a check for libunwind, libcxx, and libcxxabi to make sure there are no GCC dependencies. It’s annoying to finally build LLVM for llvmtools only to realize llvmtools has GCC dependencies.

Success
[ 99%] Built target clangStaticAnalyzerFrontend
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
[ 99%] Building C object tools/clang/tools/c-index-test/CMakeFiles/c-index-test.dir/c-index-test.c.o
[ 99%] Building CXX object tools/clang/tools/c-index-test/CMakeFiles/c-index-test.dir/core_main.cpp.o
[100%] Linking CXX executable ../../../../bin/c-arcmt-test
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
[100%] Built target c-arcmt-test
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
[100%] Linking CXX static library ../../../../lib/libclangFrontendTool.a
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
[100%] Built target clangFrontendTool
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
make[2]: Entering directory '/mnt/cmlfs/sources/llvm/build'
[100%] Linking CXX static library ../../../../../lib/libclangDependencyScanning.a
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
[100%] Built target clangDependencyScanning
[100%] Linking CXX executable ../../../../bin/c-index-test
make[2]: Leaving directory '/mnt/cmlfs/sources/llvm/build'
[100%] Built target c-index-test

Commenting this will disable libatomic detection for Linux when configure libcxx (cmake).
check_library_exists(atomic __atomic_fetch_add_8 "" LIBCXX_HAS_ATOMIC_LIB)

projects/libcxx/cmake/config-ix.cmake
102 else()
103   check_library_exists(pthread pthread_create "" LIBCXX_HAS_PTHREAD_LIB)
104   check_library_exists(m ccos "" LIBCXX_HAS_M_LIB)
105   check_library_exists(rt clock_gettime "" LIBCXX_HAS_RT_LIB)
106   set(LIBCXX_HAS_SYSTEM_LIB NO)
107 #  check_library_exists(atomic __atomic_fetch_add_8 "" LIBCXX_HAS_ATOMIC_LIB)
108 endif()

Closing issue as @owl4ce fixed issue


[BUG] The new secret webhook may cause CI to fail

Describe the bug

The CI may fail from time to time.
https://github.com/harvester/harvester/actions/runs/11008796171/job/30578651252
make ci log..
...
requested resource (get virtualmachines.meta.k8s.io)
Add/Modify event for harvester-pre-delete: MODIFIED
Starting delete for "harvester-pre-delete" Job
  uninstall: given cascade value: , defaulting to delete propagation background
Starting delete for "harvester-webhook" Service
Starting delete for "harvester-load-balancer-webhook" Service
Starting delete for "harvester-network-webhook" Service
Starting delete for "harvester-node-manager-webhook" Service
Starting delete for "harvester-snapshot-validation-webhook" Service
Starting delete for "harvester" Service
Starting delete for "harvester-network-webhook" Deployment
Starting delete for "harvester-snapshot-validation-webhook" Deployment
Starting delete for "harvester-node-manager-webhook" Deployment
Starting delete for "virt-operator" Deployment
Starting delete for "harvester" Deployment
Starting delete for "harvester-network-controller-manager" Deployment
Starting delete for "harvester-webhook" Deployment
...

To Reproduce
Steps to reproduce the behavior:

The new feature added a webhook on secret deletion https://github.com/harvester/harvester/blob/0bde44193ce1c2b2e2787237a2b31eb2263fc756/pkg/webhook/resources/secret/validator.go#L36
The make ci will run as following, which inturn deletes all resources from the chart

func uninstallHarvesterCharts(ctx context.Context, kubeConfig *restclient.Config) error {
	// uninstall chart
	_, err := helm.UninstallChart(testChartReleaseName, testHarvesterNamespace)
	if err != nil {
		return fmt.Errorf("failed to uninstall harvester chart: %v", err)
	}

// UninstallChart uninstalls the chart.
func UninstallChart(releaseName string, namespace string) (*release.UninstallReleaseResponse, error) {
...
	return action.NewUninstall(helmCfg).Run(releaseName)
}

harvester-webhook is removed at the beginning
There happens to have a secret resources

Starting delete for "snapshot-validation-webhook-tls" Secret

The webhook will be called but failed

Expected behavior

No CI failure.
Follow https://github.com/harvester/harvester/blob/0bde44193ce1c2b2e2787237a2b31eb2263fc756/deploy/charts/harvester/templates/job.yaml#L1 to add a pre-delete job which removes the secret webhook
Support bundle

Environment

Harvester ISO version: v1.4.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context

cc @Yu-Jack @bk201

I'm curious why helm uninstallation doesn't delete webhook, so I did some investigation and provided some contexts.
I checked these two actions.

Succeed - https://github.com/harvester/harvester/actions/runs/11008796171/job/30583327089
Failed - https://github.com/harvester/harvester/actions/runs/11008796171/job/30578651252

I noticed that helm uninstallation only processed this deletion Starting delete for "harvester-snapshot-validation-webhook" ValidatingWebhookConfiguration. However, it processed this deletion Starting delete for "harvester-webhook" Service  in the beginning.
Then, I found out they are managed by different tools.

@bk201 regarding your question: https://github.com/harvester/harvester/pull/6629#pullrequestreview-2326916250
@Yu-Jack
This is a sequence issue, it does not happen 100%.
from
Starting delete for "harvester-webhook" Service
to
Starting delete for "snapshot-validation-webhook-tls" Secret

it depends on how fast k8s
deletes the harvester-webhook service and related pods
VS
deletes the secret,
the secret has a delete webhook, if harvester-webhook is gone, it will report error, otherwise it can pass.

The explanation sounds sane to me, thanks!

PRs merged for both master and v1.4 branches. We can observe if the issue happens again.

Observed a new fail on https://github.com/harvester/harvester/actions/runs/11031062806/job/30637017434, I will continue to check it.


chore(deps): bump androidx.compose:compose-bom from 2024.10.01 to 2024.11.00

Apply compose version bump that only include bug fixes changes to help lbc upgrade their version quicker

+ androidx.compose.material:material-android:1.7.5
+ androidx.compose.material:material:1.7.5

Thanks I had an issue with my gpg program once again that blocked me from pushing the change 😮💨


Add magic ranking to trending page

Adds "magic" Hacker News-style ranking to the trending page and uses it by default.
This ranking applies a time-based exponential decay to the weight of the number of mirrors a post has. This means that a post remains at the top if it has a lot of mirrors and continues to get mirrors, but begins to move down as time passes and mirroring slows down. The goal is to keep fresh, interesting content on top.

This is really cool! I'm worried about the dataset though, since you're not only running the algo across all posts but either the most recent 50 or the most collected 50.

Haha, just saw that something like this was submitted in #3 . Feel free to chose your preferred implementation and close this if necessary.

@m1guelpf: I see what you mean. The difference between HN and Refract is that HN does all the ranking in the backend and here, it's all happening on the frontend.
Right now you're pulling 50 posts, but how many do you think you could get away with and remain performant?


Styles for the preview button are broken on a page with more than one preview mode

Issue Summary
Styles for the preview button are broken on a page with more than one preview mode (on form page, for example).
Steps to Reproduce

Define a form page model using wagtail.wagtailforms
Go to the Wagtail admin UI to create a new page.

Result

Technical details

Django version: 1.11.3
Wagtail version: 1.12 (Can't reproduce on 1.11.1)
OS: MacOS 10.12.5 (16F73)
Browser versions:

Google Chrome 60.0.3112.101
Safari 10.1.1 (12603.2.4)
FireFox Developer Edition 56.0b2

I think I know the cause for this, will release a PR fixing it ASAP.


Support Wine registry

It turns out Wine registry format differs somehow from Windows one.

interesting, can you provide an example?

For now no sample files (I guess I should generate them, not just dump the files from my system), but
https://source.winehq.org/source/dlls/kernelbase/registry.c
and
https://source.winehq.org/source/include/winreg.h
should be useful.

I think the loading of the registry files is done here: https://source.winehq.org/source/server/registry.c#1717
notably, it appears the format is a simple textual one, almost like what is used by *.reg files and not like a typical registry hive. i can see that there might be some interest in being able to programmatically query this data, and the interface to do so might be like python-registry, but im not sure if this is the right place.
given that this is a textual format and not much like a transactional database, i don't think there's likely to be any slack space or other interesting metadata to inspect. i think it might already be human readable, but i'm only guessing based on the source code.

here's an example of a wine registry file: https://www.virustotal.com/gui/file/6693cae85214a0f40b7f6f850e916cefc49978a073f814d43576c01aab9860c9
(found using signature "WINE REGISTRY Version 2")
WINE REGISTRY Version 2
;; All keys relative to \\User\\S-1-5-21-0-0-0-1000

#arch=win32

[AppEvents\\Schemes\\Apps\\Explorer\\Navigating\\.Current]<PHONE_NUMBER>
@=""

[Control Panel\\Accessibility\\Blind Access]<PHONE_NUMBER>
"On"="0"

[Control Panel\\Accessibility\\Keyboard Preference]<PHONE_NUMBER>
"On"="1"

[Control Panel\\Accessibility\\ShowSounds]<PHONE_NUMBER>
"On"="0"

[Control Panel\\Colors]<PHONE_NUMBER>
"background"="0 0 0"

[Control Panel\\Desktop]<PHONE_NUMBER>
"ActiveWndTrackTimeout"=dword:00000000
"BlockSendInputResets"="0"
"CaretWidth"=dword:00000001
"ClickLockTime"=dword:000004b0
"DoubleClickHeight"="4"
"DoubleClickWidth"="4"
"DragFullWindows"="0"
"DragHeight"="4"
"DragWidth"="4"
"FocusBorderHeight"=dword:00000001
"FocusBorderWidth"=dword:00000001
"FontSmoothing"="1"
"FontSmoothingGamma"=dword:00000000
"FontSmoothingOrientation"=dword:00000001
"FontSmoothingType"=dword:00000001
...

the number next to the key path is a timestamp:
 path                                                      timestamp
[AppEvents\\Schemes\\Apps\\Explorer\\Navigating\\.Current]<PHONE_NUMBER>

https://source.winehq.org/source/server/registry.c#1468

Thanks for your research. I was surprised to know that the format is a textual one and wastly different from the one used on Windows for the files with the same names. I also think, that since that format is not a variant of Windows format (I had been assumming it given the exceptions I got from this and the another lib for parsing hives) it may be out of scope of this project to parse that text format.
So it makes more sense to embed it into some .reg parser libs. Unfortunately I have found no Pythkn package for that, but it seems there is some related code:

https://github.com/andrey-malets/dg/blob/master/win7/filter_reg.py
https://github.com/saiwolf/reg-file-parser
https://github.com/dnso86/reg2bin/blob/master/reg2bin.py


[Bug]: XAML update failed

The problem
Same update at every run. It always fails. Xaml 2.8 is already installed.

What version of WAU has the issue?
latest
What version of Windows are you using (ex. Windows 11 22H2)?
11 22h2
What version of winget are you using?
latest
Log information
No response
Additional information
No response

I added it to my blacklist, everything works as it should anyway.
This is what I have installed before:


fix(resource): check if resource description breaks json.encode and remove checking a specified dev

Discussed with @tabarra on discord here:
https://discord.com/channels/577993482761928734/1140025605740564581/1196835709978411039
Basicly i removed the checking for just one developer that seems to have that broken resource description and instead add a general checking if the resource description breaks json.encode.
@tabarra said that the issue with the broken unicode was probably fixxed on cfx side and it may not be a problem at all anymore that resource description breaks it. But decided to do a pr anyways.

Thanks so much for the research and PR, but I have tested and indeed the issue has been solved, so that hacky patch is not needed, neither your PR.
Test fxmanifest.lua:
author 'ok\xE2\x96\x93'
version 'broken\xE2\x96'
description 'broken2\xE2'

txAdmin resources page:

if the issue hadn't been solved, the entire post of resources would have failed.


Chore/pme middleware types

This PR merges beta into alpha and upgrades the middleware types to reflect the latest PME gql instance

:tada: This PR is included in version 6.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.0.0-beta.10 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Factor's Raylib bindings mention being at v4.5 now.

I looked at the linked Factor binding page for Raylib and found that the file summary.txt has the following text in it:
Bindings for Raylib 4.5

The bindings have also been updated within the last week.
This implies that Factor's Raylib bindings are either already up to date with v4.5 or will be soon.
Thus, I have submitted this likely correction to Raylib's bindings table.
(PS: There are other Factor Raylib bindings on the internet that are less up-to-date, so don't confuse those with this.)

@WraithGlade Thanks for the update!


aws-controllers-k8s/rds-controller aarch64/arm64 support

Is your feature request related to a problem?
Support rds-controller on Graviton instances
Describe the solution you'd like
1/ Run the rds-controller on aarch64 instances
2/ Provision graviton-based RDS database instances
Describe alternatives you've considered
Use CDK e.g.,
https://github.com/aws-samples/amazon-aurora-call-to-amazon-sagemaker-sample/tree/master/multiplayer-matchmaker/aurora-pg-cdk

We're tracking this feature in this issue: https://github.com/aws-controllers-k8s/community/issues/138
Currently @santyamzn is working on a PoC

1/ Run the rds-controller on aarch64 instances

As @A-Hilaly pointed out, we're working on this in issue #138.

2/ Provision graviton-based RDS database instances
Please note that you can do this today. The EC2 instance type that an RDS DB instance is housed on does not have anything to do with where the ACK controller for RDS is placed.

I will close this as it is already tracked. Created https://github.com/aws-controllers-k8s/code-generator/pull/358 for "Run the rds-controller on aarch64 instances"


Added Provider->getLastTouchedOfEachRow() and tests

@mpdude please have a look.

Added a few comments, otherwise ❤  – especially the test part!


Allow api responses to be ordered

addresses the first half of #36
Adds a new query parameter to each api endpoint called 'ordering'.
Also fixes some typos in the descriptions of other query parameters.

@jhsimpson is the d-{order_field} a convention you've seen or used elsewhere? I've been having a quick dig around how other APIs do this and can't see an obvious consensus. I've seen sort, sortBy, orderBy among others, so I don't have a particularly strong opinion on "ordering", but I've not seen the d- elsewhere. I've seen just a negation (e.g. ?sort_by=-field), specific second parameter for direction (e.g. ?sort_by=field&order/direction=asc/desc), and combination using a separator (e.g. ?sort_by=field:asc).
Given that there doesn't seem to be a particularly strong convention, I'm mostly happy as long as we're consistent, just wondering where the d- came from?

It is a fair question. The 'd-' is not a common convention.  It is a workaround to a bug that used to exist in the openapi-generator that couldn't handle keys in an enum that begin with + or -.  I added a commit just now to change this to using a simple '-' instead of 'd-'.
I agree there isn't a single dominant convention, and I do not personally have a strong inclination to one over the other.  Of the 3 examples you gave, I prefer using a '-' to indicate descending order (?ordering=foo,-bar) over the use of a separator (?ordering=foo,bar:desc) or the use of a 2nd query parameter for direction.

closing this PR, it is superceded by #79


Develop

I have updated the library to take advantage of the WebDriver portion of selenium (along with accounting for the Selenium-RC portion).  The Read Me will still need to be updated, but I've added unit tests for the new features and fixed some issues with the previous unit tests.  I've added some comments along the way to try to assist others with issues I ran into when using selenium v1 code.  I've also added the latest selenium jar file and updated all references.

Is this going to merged into master?

this looks pretty awesome. im a huge fan of webdriver.

conflicts need to be resolved before merging, who want's to be a super hero ?

I may take a crack at it this weekend.

Resolved conflicts in https://github.com/teamcfadvance/CFSelenium/pull/25
@Lampei, could you have a look to see if this is in line with your original intent?

Excellent! @jamiejackson glad someone got around to be able to merge it in. I'd started to get the CI portion going but ran out of time.


Fix padding in combineGltfs

I noticed some byte offset issues when loading the GTModels tileset and tracked it down to incorrect padding in combineGltfs. I did a quick search for other areas that handle padding and it seems like only this area has the problem.


showTableSuggestionValueHelp does not work in IE

Hello,
when I set the parameter showTableSuggestionValueHelp=false it has no effect in IE.
According the documentation (https://sapui5.hana.ondemand.com/#/api/sap.m.Input):
For tabular suggestions, this flag will show/hide the button at the end of the suggestion table that triggers the event "valueHelpRequest" when pressed. The default value is true.
NOTE: If suggestions are not tabular or no suggestions are used, the button will not be displayed and this flag is without effect.
Here is an example:
https://jsbin.com/ramutumuxo/1/edit?html,output
This is the result in Chrome:

This is the result in IE:

I expected that the IE gives me the same result as the Chrome.

Thanks for your report! This is now handled internally as<PHONE_NUMBER>


Please add guidance for unit testing

How can I inject my mocks?

Thanks for comment. Can you confirm if your feedback is related to Azure/Cloud or On-premise scenario? From the comment above, this seems unrelated to a cloud solution. Can you share the link to the documentation you are referencing above?

any update please?

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Fail trying to run (building from sources) in Mageia 8

[ x] I think my problem is NOT with youtube-dl
[x ] I've verified and i assure that I'm running yt-dlg 1.6.0
[x ] I assure that i am using the latest version of youtube-dl
[ x] Searched bugtracker
[x ] I've read the FAQs file

What is the purpose of your issue?

[x ] Bug report
[ ] Feature request (request for a new functionality)
[x ] Question
[ ] Other

Bug report
What operating system do you use ?
Mageia Linux 8
List of actions to perform to reproduce the problem:

.. Download source tarball
.. Build a rpm
.. Install the resulting rpm
.. Run yt-dlg
.. Get this error

Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/youtube_dl_gui/app.py", line 65, in OnInit
    wx.Locale.AddCatalogLookupPathPrefix(locale_dir)
TypeError: Locale.AddCatalogLookupPathPrefix(): argument 1 has unexpected type 'NoneType'
OnInit returned false, exiting...

What is the expected behaviour ?
yt-dlg is launched correctly
What happens instead ?
Fail with the error mentioned
This is the spec file i use to build the rpm,  my distribution don't provide python3-altgraph so i make it and install
youtube-dl-gui.spec.txt
python3.8

When you can try re-build from Source (.zip).
Remember before build you need generate and add the .mo files from locale folder for translations
Thanks for the report! 😉

Build with 1.7.0 source, bow i get this
yt-dlg
Traceback (most recent call last):
  File "/usr/bin/yt-dlg", line 33, in <module>
    sys.exit(load_entry_point('yt-dlg==1.7.0', 'console_scripts', 'yt-dlg')())
  File "/usr/bin/yt-dlg", line 25, in importlib_load_entry_point
    return next(matches).load()
StopIteration

Build with 1.7.0 sources, now i get this
yt-dlg
Traceback (most recent call last):
  File "/usr/bin/yt-dlg", line 33, in <module>
    sys.exit(load_entry_point('yt-dlg==1.7.0', 'console_scripts', 'yt-dlg')())
  File "/usr/bin/yt-dlg", line 25, in importlib_load_entry_point
    return next(matches).load()
StopIteration

You can open other issue for the second error (in the quotes) and describe the workflow/steps before to until you get this error. Thanks one more time!

I think i miss some requires, i am working on that, when i finish i will comment my result.

I think i miss some requires, i am working on that, when i finish i will comment my result.

Ok any type of information will help fix the issue.

I get it work but i still have to clean the spec file to see what is really necessary
For the moment i get it works with the system wxpython so i don't have to build again as with the recommended method (that fail me anyway :( )
I don't sure if must open other issue or comment in this one but
at first run it downloads youtube-dl and install it on .config/yt-dlg/youtube-dl but without execute rights
log-yt-dlg.txt
Once i chmod a+x .config/yt-dlg/youtube-dl the application works as must :)

Once i chmod a+x .config/yt-dlg/youtube-dl the application works as must :)

That issue will fix in the PR #36 set the permission execution on Linux. The next release will add that 😉

Once i chmod a+x .config/yt-dlg/youtube-dl the application works as must :)

That issue will fix in the PR #36 set the permission execution on Linux. The next release will add that wink

Test the fix, works if the youtube-dl is not previously in the folder but if a youtube-dl with wrong rights is in the folder the bad behaviour persists ;)

Once i chmod a+x .config/yt-dlg/youtube-dl the application works as must :)

That issue will fix in the PR #36 set the permission execution on Linux. The next release will add that wink

Test the fix, works if the youtube-dl is not previously in the folder but if a youtube-dl with wrong rights is in the folder the bad behaviour persists ;)

Easy fix press on update so i release a new package for mageia that make the warning.
Thanks for all

Easy fix press on update so i release a new package for mageia that make the warning.

I’m glad for that. Have planned make oficial release for Mageia?. Share the link here the last Release updated if is possible.
Thanks too, and will in contact!

I’m glad for that. Have planned make oficial release for Mageia?. Share the link here the last Release updated if is possible.
Thanks too, and will in contact!

Not for the Official repositories but for the Repository of BlogDrake (https://blogdrake.net/), the community for Spanish Speaking users.
I put the instructions to add the repository and the application tomorrow (i hope)


Stop using babel-plugin-add-module-exports

Using babel-plugin-add-module-exports appends module.exports setting line like this:
exports.default = 'foo';
module.exports = exports['default']; // Added!

which disables exports.default and causes problems when using transpilers (like TypeScript) that does not have same interop rules to Babel. (Related issue: DefinitelyTyped/DefinitelyTyped/issues/8333).
Indeed there is no one-to-one correspondence between CommonJS and es6 modules, and export default foo in es6 is not equal to module.exports = foo in CommonJS. As material-ui's source code are using es6 module format, how about stopping using babel-plugin-add-module-exports?
Staying in CommonJS way with babel-plugin-add-module-exports might reduce commonjs -> es6 transition cost (like #2802), but I think Babel 6, which does not natively support module.exports setting, is spreading and people are getting to know they need to access exports.default (or require('foo').default) to access es6's default exported modules.

This sounds good. Just a little history on this, babel-plugin-add-module-exports, was added after a regression was introduced during a patch release after we upgraded from babel 5 to 6.
We plan on reorganizing the directories and simplifying the import paths. That might be a good opportunity to drop the plugin since users will have to update the import paths anyway.

@pawsong Thanks this is definitely something we consider important when we get the opportunity.

@newoga @nathanmarks I am looking forward to 0.15 release. Thank you for your answer!

Thanks for reminding us! :+1:


Add option of using ChatGPT (or any other LLM) for step 1 of getting started

A potentially faster alternative way of creating/generating a JSON resume is to use ChatGPT or something like it, to convert an existing resume from text to the JSON Resume schema
The prompt, in pseudocode is:
Resume:
{resume}

JSON Resume Schema example:
{
  "basics": {
    "name": "John Doe",
    "label": "Programmer",
    "image": "",
    "email"<EMAIL_ADDRESS> ...
}

Provide JSON data structure of the resume, formatted according to the JSON Resume Schema. Output json, no yapping

And that outputs the original resume in JSON format, according the JSON Resume Schema

Created a PR for it: https://github.com/alexpovel/ancv/pull/197
Here's a recording of the process in Loom (using ChatGPT to get the JSON format of a resume, and also then creating the PR for the README):


Simplify sasl user iteration.

Part of the idea is that we don't need conditionals for iterating over a sequence that defaults to empty.


[help] question on embedded grpc-web and http service

Hey! I'm having a difficult time trying to replicate something I've written in Go. Hoping someone with a bit more understanding of the tonic package can help.
In general what I'm trying to do is have normal http requests go to a normal http handler and grpc/grpc-web requests go to a grpc handler. This allows me to embed the frontend and the backend into a single binary.
In Go, I've achieved this using:
// WrapServer turns normal grpc server object into one capable of handling grpc-web requests
wrappedGrpc := grpcweb.WrapServer(server)

router := mux.NewRouter()

frontend := frontend.NewFrontend()
frontend.RegisterUIRoutes(router)

combinedHandler := http.HandlerFunc(func(resp http.ResponseWriter, req *http.Request) {
if strings.Contains(req.Header.Get("Content-Type"), "application/grpc") || wrappedGrpc.IsGrpcWebRequest(req) {
	wrappedGrpc.ServeHTTP(resp, req) // If we think it's a grpc request we just pass it to be handled by the grpc server
	return
}
router.ServeHTTP(resp, req) // else we pass it to the regular service
})

      httpServer := http.Server{
	      Addr:         config.URL,
	      Handler:      combinedHandler,
	      WriteTimeout: 15 * time.Second,
	      ReadTimeout:  15 * time.Second,
      }

httpServer.ListenAndServeTLS(config.TLSCertPath, config.TLSKeyPath))
...

Go's http.HandlerFunc interface makes this a bit easy to grok how the pieces fit together, but I'm having trouble understanding what I would need for the equivalent using tonic.
What I have so far is just a normal tonic grpc-web service. Looking through the docs though I'm unable to understand if tonic-web is capable of being called as a handler within some other http router:
let reflection = Builder::configure()
    .register_encoded_file_descriptor_set(tonic::include_file_descriptor_set!("reflection"))
    .build()
    .expect("could not build reflection server");

info!("started grpc-web service"; "address" => &self.conf.server.url);

Server::builder()
    .accept_http1(true)
    .add_service(reflection)
    .add_service(BasecoatServer::new(self.clone()))
    .serve_with_incoming(incoming)

    .serve(self.conf.server.url.parse().unwrap())
    .await
    .unwrap();

In axum (a web framework with the same foundations as tonic) we're working on an example of this exact thing actually. See https://github.com/tokio-rs/axum/pull/967

Oh wow that PR is exactly spot on for what I needed. Thanks @davidpdrsn, saved me hours of stumbling around in the dark.


Add an example for batch normalization.

Paired with mila-udem/blocks#941.
Attn: @rizar

@cooijmanstim Mind looking at this too?

Other than my two inline comments, LGTM.

Sorry, I forgot about this PR. I agree with @vdumoulin and have nothing else to add.

Home come it had not been merged?! :)


Format code using 'cargo fmt'

Thanks for this, but I'm going to close. I'm not actively developing the master branch, development is currently in "sort_middle" and "sorta." Locally I have a lot of branches with various experiments, some of which are quite valuable (one example is generating trace data which can then be visualized with chromium tracing). Having formatting changes would make it more difficult to juggle these. I might create an archive version with all these local branches.
When I am no longer in experiment mode and am reasonably satisfied with the code, I'll do a cleanup which will include rustfmt. Again, thanks for your interest.


Tu1ly/swagger json

https://github.com/apiaryio/apiary-client/issues/137

 Apiary CLI should support JSON files (?)

I was thinking that there could also be a warning if you try to use JSON without --json

@JackuB  yup  - that was the idea but then i realized that we need more complex approach for that - detect format in client. it would be confusing otherwise

no --json option

invalid yaml passed - Apiary service responded with an error  || invalid yaml client message (?)
invalid bp passed - Apiary service responded with an error
invalid json passed - Apiary service responded with an error || invalid json client message (?)

valid json passed -  no  json param client message (?)

valid json/yaml but not valid as ADD

@tu1ly my point is that, if you forget --json, and you will get a parsing error back - can you also get You forgot the --json flag? kind of message?

@JackuB added - but i don't see any added value

It could help with user experience - easy to forget these flags


Zoom-to-task button on Multi-select view goes to previous task

STR

Complete a task
In the multi-select view, click the zoom-to-task button

ER: Zooms to current task's location
AR: Zooms to previous task's location

Thanks for reporting this, @jgscherber! I've submitted PR #1482 to address it.

Thanks for reporting this, @jgscherber! I've submitted PR #1482 to address it.

This should now be fixed on maproulette.org

This should now be fixed on maproulette.org


/en-US/docs/Learn/CSS/Building_blocks/Overflowing_content

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
It may be necessary to mention the difference in how overflow: scroll looks depending on the OS.
There are the following sentences in the article.
https://developer.mozilla.org/en-US/docs/Learn/CSS/Building_blocks/Overflowing_content

Perhaps you would instead like to add scrollbars when content overflows? If you use overflow: scroll then your browser will always display scrollbars — even if there is not enough content to overflow. You may want this, as it prevents scrollbars appearing and disappearing depending on content.

However, as shown in the capture below, this is true on Windows, but not necessarily on macOS.
In the first place, there is no implementation that scroll bar is always displayed in macOS, and I think that it is displayed only while scrolling.

I wanted to modify the article directly, but I wasn't confident that I fully understood the intention of this sentence, so I first sent a request.

I've tested this on macOS, and found that with overflow: scroll; set, the scrollbars don't remain when there is nothing to scroll, but the scrollbar tracks still remain (this was the case on Firefox and Chrome).
I've therefore ammended the text to
"If you remove some of the content from the box below, you'll see that the scrollbars still remain even with nothing to scroll (or at least just the scrollbar tracks)."

Thank you for quick response:)
I noticed this in the Japanese version, so I will correct it.


[BEAM-2051] Remove all methods from PCollectionView

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Cast to SimplePCollectionView everywhere a method is used.
PCollectionView is a marker interface. Runners can expect to
always recieve a subclass of SimplePCollectionView, and cast to it when
methods are required.

R: @kennknowles

https://builds.apache.org/job/beam_PostCommit_Java_MavenInstall/3451/ is a postcommit oneshot of this pR

https://builds.apache.org/job/beam_PostCommit_Java_MavenInstall/3453/ now

And this is just a forceful way of saying "these methods are just for runner authors"?

yes.
On Tue, Apr 25, 2017 at 3:26 PM, Kenn Knowles<EMAIL_ADDRESS>wrote:

And this is just a forceful way of saying "these methods are just for
runner authors"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/2680#issuecomment-297183261, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAaPKg03yCKpDRVqTBAEmzLBhem3L-rDks5rznMZgaJpZM4NICjl
.

Coverage decreased (-0.05%) to 69.526% when pulling f52254b272fc89b89f9f3a979525e731b84ca6c9 on tgroh:view_as_marker into 99cf72e7d0b3ef79e691655dbc23bf3032e28a0b on apache:master.

Coverage increased (+0.1%) to 69.727% when pulling f52254b272fc89b89f9f3a979525e731b84ca6c9 on tgroh:view_as_marker into 99cf72e7d0b3ef79e691655dbc23bf3032e28a0b on apache:master.

run dataflow validatesrunner

run flink validatesrunner

run spark validatesrunner

run apex validatesrunner

run flink validatesrunner

run apex validatesrunner

run spark validatesrunner

Coverage increased (+0.2%) to 69.738% when pulling 7d8b71491606e83f31dc5cc4fe04fad984663045 on tgroh:view_as_marker into 99cf72e7d0b3ef79e691655dbc23bf3032e28a0b on apache:master.

run dataflow validatesrunner

LGTM, though there are a lot more places in the SDK where we don't use this technique. I'd say it should be only temporary.

Just noting the alternative of leaving the methods and heavily documenting that they are internal only, which is what we've done most places.

Coverage increased (+0.06%) to 69.638% when pulling 1dec53e6dd46fa66f36a846797fe4eec3d90673d on tgroh:view_as_marker into 99cf72e7d0b3ef79e691655dbc23bf3032e28a0b on apache:master.

retest this please

Changes Unknown when pulling b48c9d3b41771849a6daf64077c3528398ce7add on tgroh:view_as_marker into ** on apache:master**.

With the @Internal annotation we don't actually want this.


Consider changing how you get logonUserIdentity

I suggest you to change how you get your logonUserIdentity’. It is only available in ‘PostAuthenticate’ OWIN pipeline stage events. If you protect your resources with OAuth2/Open ID connect Identity provider you would be running in ‘Authenticate’ event and ‘logonUserIdentity’ is not  available (((System.Web.HttpContextBase)Request.Environment["System.Web.HttpContextBase"]).Request.LogonUserIdentity) , but if you change to (Response.Context).Authentication).User it would work in both OWIN pipeline stage events.
I implemented something similar and it is working.

Not sure how its working for you? as I'm interested in getting the WindowsIdentity !

Usage Step 3

3 . Use MixedAuth in Startup.Auth.cs
//Enable Mixed Authentication
//As we are using LogonUserIdentity, its required to run in PipelineStage.PostAuthenticate
//Register this after any middleware that uses stage marker PipelineStage.Authenticate

app.UseMixedAuth(cookieOptions);

Important! MixedAuth is required to run in PipelineStage.PostAuthenticate, make sure the use statement is after any other middleware that uses PipelineStage.Authenticate. See OWIN Middleware in the IIS integrated pipeline.

As long as you configure MixedAuth after any provider that uses  PipelineStage.Authenticate, you'll run in PipelineStage.PostAuthenticate.


Fix some issues

According to  suggestions in this issue, I made some changes in related files. See in details in the above page.

Please remove the preview block of audio in the speech recognition section


[tool_crash] FileSystemException: Cannot copy file to 'C:\Users\HP\Desktop\nkflutter\projects\test_1\android\app\src\main\res\mipmap-mdpi\ic_launcher.png', path = 'C:\Users\HP\Desktop\nkflutter\flutter\packages\flutter_tools\templates\app\android.tmpl\app\src\main\res\mipmap-mdpi\ic_launcher.png' (OS Error: Access is denied., errno = 5)

Command
flutter create test_1

Steps to Reproduce

...
...
...

Logs
FileSystemException: FileSystemException: Cannot copy file to 'C:\Users\HP\Desktop\nkflutter\projects\test_1\android\app\src\main\res\mipmap-mdpi\ic_launcher.png', path = 'C:\Users\HP\Desktop\nkflutter\flutter\packages\flutter_tools\templates\app\android.tmpl\app\src\main\res\mipmap-mdpi\ic_launcher.png' (OS Error: Access is denied.
, errno = 5)
#0      _File.throwIfError (dart:io/file_impl.dart:645:7)
#1      _File.copySync (dart:io/file_impl.dart:350:5)
#2      ForwardingFile.copySync (package:file/src/forwarding/forwarding_file.dart:30:50)
#3      ForwardingFile.copySync (package:file/src/forwarding/forwarding_file.dart:30:50)
#4      Template.render.<anonymous closure> (package:flutter_tools/src/template.dart:227:18)
#5      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:379:8)
#6      Template.render (package:flutter_tools/src/template.dart:153:24)
#7      CreateCommand._renderTemplate (package:flutter_tools/src/commands/create.dart:630:21)
#8      _rootRunUnary (dart:async/zone.dart:1192:38)
#9      _CustomZone.runUnary (dart:async/zone.dart:1085:19)
#10     _FutureListener.handleValue (dart:async/future_impl.dart:141:18)
#11     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:682:45)
#12     Future._propagateToListeners (dart:async/future_impl.dart:711:32)
#13     Future._completeWithValue (dart:async/future_impl.dart:526:5)
#14     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:36:15)
#15     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:298:13)
#16     Template.fromName (package:flutter_tools/src/template.dart)
#17     _rootRunUnary (dart:async/zone.dart:1192:38)
#18     _CustomZone.runUnary (dart:async/zone.dart:1085:19)
#19     _FutureListener.handleValue (dart:async/future_impl.dart:141:18)

[✓] Flutter (Channel stable, v1.17.5, on Microsoft Windows [Version 10.0.18362.592], locale en-IN)
    • Flutter version 1.17.5 at C:\Users\HP\Desktop\nkflutter\flutter
    • Framework revision 8af6b2f038 (3 weeks ago), 2020-06-30 12:53:55 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\HP\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_SDK_ROOT = C:\Users\HP\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.47.2)
    • VS Code at C:\Users\HP\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.12.2

[✓] Connected device (1 available)
    • AOSP on IA Emulator • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

Flutter Application Metadata
No pubspec in working directory.

That error simply means that Windows can't copy the file to your target directory. Could be because you already have it open in a different application, or because you don't have permissions to that directory.

Hi @vyshnavi7120,
I see there's a similar open issue describing your case, 50880.
Please follow-up there for updates.
Closing this duplicate. If you disagree, write in comments and I'll reopen.
Thanks.

I solved this error. Thanks for your advices.


feat: add type restrictions rfc

Description

This RFC introduces support for type restrictions on relations. Restricting what types of users can be assigned to specific relations
References

Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

@rhamzeh I don't see the RFC mention restrictions on specific relations of an object type.
e.g. group#member
Can we please include this in the scope of the RFC?

I don't see the RFC mention restrictions on specific relations of an object type.
e.g. group#member
Can we please include this in the scope of the RFC?

@jon-whit can you elaborate please? This is definitely mentioned in the RFC


Fix JS and Go tour documents to use new Database/Dataset API

Whoops, shoulda fixed these with the patches that updated the
APIs.

PTAL :-)
Also, FYI @frick, @sandalwing

👍


fix: resolve undefined searchInput reference in Alpine.js component

Hi everyone,
in this PR I am fixing a small bug I noticed when creating a new resource in my project.
I am looking forward to a review and feedback, and thank you very much for your work!
Best regards,
Lukas
Changes

The searchInput reference was not accessible due to nested x-data scopes. I moved sticky logic into searchResources() to ensure proper reference access.

Issues
Uncaught TypeError: this.$refs.searchInput is undefined
    loadResources http://localhost:8000/project/c80wsccws0wswcs40okcoogk/staging/new:2262
    nextTick http://localhost:8000/livewire/livewire.js?id=38dc8241:1516
    releaseNextTicks http://localhost:8000/livewire/livewire.js?id=38dc8241:1524
    nextTick http://localhost:8000/livewire/livewire.js?id=38dc8241:1511
    setTimeout handler*nextTick/< http://localhost:8000/livewire/livewire.js?id=38dc8241:1510
    nextTick http://localhost:8000/livewire/livewire.js?id=38dc8241:1509
    loadResources http://localhost:8000/project/c80wsccws0wswcs40okcoogk/staging/new:2261
    runIfTypeOfFunction http://localhost:8000/livewire/livewire.js?id=38dc8241:1207
    generateEvaluatorFromString http://localhost:8000/livewire/livewire.js?id=38dc8241:1195
    tryCatch http://localhost:8000/livewire/livewire.js?id=38dc8241:1113
    evaluate http://localhost:8000/livewire/livewire.js?id=38dc8241:1137
    <anonymous> http://localhost:8000/livewire/livewire.js?id=38dc8241:3479
    skipDuringClone http://localhost:8000/livewire/livewire.js?id=38dc8241:1898
    flushHandlers http://localhost:8000/livewire/livewire.js?id=38dc8241:1278
    stopDeferring http://localhost:8000/livewire/livewire.js?id=38dc8241:1283
    deferHandlingDirectives http://localhost:8000/livewire/livewire.js?id=38dc8241:1286
    initTree http://localhost:8000/livewire/livewire.js?id=38dc8241:1473
    start http://localhost:8000/livewire/livewire.js?id=38dc8241:1425
    start http://localhost:8000/livewire/livewire.js?id=38dc8241:1424
    start2 http://localhost:8000/livewire/livewire.js?id=38dc8241:8739
    <anonymous> http://localhost:8000/livewire/livewire.js?id=38dc8241:9999
    EventListener.handleEvent* http://localhost:8000/livewire/livewire.js?id=38dc8241:9995
    <anonymous> http://localhost:8000/livewire/livewire.js?id=38dc8241:10002

Thank you for the PR!


[CSS] things aren't lined up/ style inconsistencies

Subscriptions latest only page

Not horizontally lined up

Subscriptions overview page

Also not lined up
Not bold like the other view

Subscriptions download toggle

Not lined up

Working on this

resolved in #2450


Api Compliance Rework

### Tasks
- [ ] #473
- [ ] https://github.com/OpenAssetIO/OpenAssetIO/issues/1242
- [ ] https://github.com/OpenAssetIO/OpenAssetIO/issues/1243
- [ ] https://github.com/OpenAssetIO/OpenAssetIO/issues/1040
- [ ] https://github.com/OpenAssetIO/OpenAssetIO/issues/604

Havn't had time to refine this beyond a list of scattered tasks, but the thing here really is that API compliance has been ignored for a long time, and we need to do these things.

Reduce the burden of implementing API compliance tests going forward.
Backfill all the missing test assertions that should be there but arn't because we've been ignoring it.
Reduce the burden of providing fixtures for manager authors.
Make API compliance suite the premium implementation aid/guide it could be to manager authors.


Invalid link to global FFZ emote detail

Describe your issue
Using the Open FFZ emote links action on a global FFZ emote currently opens a link without the emote ID, leading to a 404.
E.g.: Attempting to open the emote link for the AndKnuckles emote opens
https://www.frankerfacez.com/emoticon/-AndKnuckles - instead of  https://www.frankerfacez.com/emoticon/24999-AndKnuckles.
OS and Chatterino Version
Windows 10, 2.2.3

i think this could be the problem:
https://github.com/Chatterino/chatterino2/blob/14e8a2bf6c163b1a8f09478a7e2bd8a27fcf8911/src/providers/ffz/FfzEmotes.cpp#L70
compared to how the id is parsed for channel emotes:
https://github.com/Chatterino/chatterino2/blob/14e8a2bf6c163b1a8f09478a7e2bd8a27fcf8911/src/providers/ffz/FfzEmotes.cpp#L135
though i'm not proficient enough in C++ to be sure

I'll look into this bug.


ZIP Download doesn't work anymore (Some Browsers)

I tried to decrypt a bunch of Files for someone  but I saw that my Firefox (on Fedora 26) freezes if I try to Download the Files as ZIP... So I have to investigate why

I use now FileSaver.js to Download the ZIP-File. I tested it with 150 Files and my Browser (Firefox) just downloaded it perfect after that change! - See: https://github.com/Petschko/RPG-Maker-MV-Decrypter/commit/a58367a709e67e59645fe72019bc9479114512a7
Can please someone test Opera?

Test with 700+ files with Opera 60.0.3255.109 and it works well.

Thank you!^^


Robot-2023 Makefile broken for Windows venv

Description
The makefile does not properly upgrade pip setuptools when setting up the windows virtual environment, causing the make to fail.
Steps to Reproduce

Clone Robot-2023 Repo in Windows PS
Run make within Repo

Expected behavior: virtual environment is created and simulation opens
Actual behavior: Error Code 2, file not found
Required Information

What tag/branch/release of Robot code are you using?
Robot-2023 main branch as of 1/28/2023

Useful Information
Python 3.10.6 on Windows 11

Should close upon merging #53


please use FALSE and TRUE exclusively in the .tsv files

Hi,
It is easier, when reading these into R if you use FALSE and TRUE, rather than False and True, as the former are keys
that are used to make the column a logical vector rather than a character vector.
You do use these in the UseConstraints column, but for some reason use False and True in the IsPhenotype and OntologyMapped columns.
In some sense, no matter what you should always use the same character strings for true and false, throughout a table.
thanks

This should be fixed now in v3.9.1.


freqz() output for fifth order butterworth bandpass  (low cut off frequencies, high sampling rate)

I am using a slightly modified code form scipy-cookbook here. The parameters are the default ones in the code below (fs = 200,lowcut = 1.0,highcut = 2.0) . I am wondering if the "order-5" curve is correct?!
Reproducing code example:
from scipy.signal import butter, lfilter

def butter_bandpass(lowcut, highcut, fs, order=5):
    nyq = 0.5 * fs
    low = lowcut / nyq
    high = highcut / nyq
    b, a = butter(order, [low, high], btype='band')
    return b, a

def butter_bandpass_filter(data, lowcut, highcut, fs, order=5):
    b, a = butter_bandpass(lowcut, highcut, fs, order=order)
    y = lfilter(b, a, data)
    return y

def get_freqz(fs = 200,lowcut = 1.0,highcut = 2.0, order = [1,2,3,4,5]):
    import numpy as np
    import matplotlib.pyplot as plt
    from scipy.signal import freqz,freqs
    plt.style.use('bmh')

    plt.figure(figsize=(4,4),dpi = 150)

    for o in order:
        b, a = butter_bandpass(lowcut, highcut, fs, order=o)
        w, h = freqz(b, a, worN=2000)

        plt.plot((fs * 0.5 / np.pi) * w, abs(h), linewidth=1,label="order = %d" % o)
        plt.xlim((0,3))

        plt.xlabel('Frequency (Hz)')
        plt.ylabel('Gain')
        plt.legend(loc='best')

    plt.show()

get_freqz()

Scipy/Numpy/Python version information:

scipy.__version__ = '0.19.1'
numpy.__version__ = '1.13.1'
sys.version_info = sys.version_info(major=2, minor=7, micro=13, releaselevel='final', serial=0)

Yes it might actually be correct, b, a / TF representations of high order IIR filters can be unstable. Can you check if the second-order sections representation gives you what you'd expect?

Well, I wouldn't call it correct.   For those parameter values, butter appears to produce an unstable filter.  But that is because of floating point numerical errors that occurs in scipy's implementation; it is not a "correct" filter.  If the filter was implemented with much higher precision floating point values, e.g. withmpmath, the frequency response would work as expected, even using the transfer function representation.
In this case, @larsoner's suggestion to use the SOS format is a good one.  It is less susceptible to numerical errors.
Here's a variation of your script.  I modified it to use the SOS format for the filters (along with a few other tweaks):
import numpy as np
import matplotlib.pyplot as plt
from scipy.signal import butter, sosfilt, sosfreqz

def butter_bandpass(lowcut, highcut, fs, order=5):
    nyq = 0.5 * fs
    low = lowcut / nyq
    high = highcut / nyq
    sos = butter(order, [low, high], btype='band', output='sos')
    return sos

def butter_bandpass_filter(data, lowcut, highcut, fs, order=5):
    sos = butter_bandpass(lowcut, highcut, fs, order=order)
    y = sosfilt(b, a, data)
    return y

def get_freqz(fs=200, lowcut=1.0, highcut=2.0, order=[1, 2, 3, 4, 5]):
    plt.style.use('bmh')
    plt.figure(figsize=(4, 4), dpi=150)

    for o in order:
        sos = butter_bandpass(lowcut, highcut, fs, order=o)
        w, h = sosfreqz(sos, worN=5000)

        plt.plot((fs * 0.5 / np.pi) * w, abs(h),
                 linewidth=1,
                 label="order = %d" % o)
        plt.xlim((0, 3))
        plt.xlabel('Frequency (Hz)')
        plt.ylabel('Gain')
        plt.legend(loc='best')

    plt.axhline(np.sqrt(2)/2,
                linestyle='--', linewidth=1, color='k', alpha=0.5)
    plt.show()

get_freqz()

Here's the plot that it generates:

@abdelwahed okay to close? Or is there somewhere we could have warned about this limitation of b, a representations that would have helped?

@WarrenWeckesser  Thanks for your updating the code/methodology .
@larsoner a warning about this limitation would really help (what could be general rule for issuing this warning ? ). I investigated into freqz() output visually only after I observed that filtering results doesn't seem to be correct.

what could be general rule for issuing this warning ?

We could try to inspect the outputs to see if they are reasonable, but we might still miss some cases. I'm more thinking that maybe a documentation update to the Notes or the output keyword argument would be appropriate. Would something like this have helped?
.. warning:: Designing high-order IIR filters in TF form can result in unstable
    or incorrect filtering due to numerical precision issues. Consider inspecting
    output filter characteristics `freqz` or designing the filters with second-order
    sections via ``output='sos'``.

We might want to add something like this to our IIR design functions. It might even be common enough a problem to add a Sphinx directive to take care of it :)

In the warning, can we be more precise about what constitutes "high-order"? I would have naively not expected 5th-order to be considered high. Is anything larger than second-order considered numerically unstable?

@u55: That's a great idea.  Unfortunately, the numerical sensitivity depends on the width of the passband of the requested filter.   A filter with order 9 and a wide passband might have no problems, while one with order 5 and a very narrow passband (like the one in this issue) might result in instability.

Is it possible that I can work on this issue and open a PR?

Sure, a documentation update here would be good!

Thank you for reply. Could I start to work on this from now on?

@musteng feel free - please check out our contributing guide and feel free to reach out if you have questions 😄

Thank you so much @melissawm, I think I should open a pull request and link this issue and work 😁. If anything comes up I will reach out!

@musteng I believe the suggestion above was to add this comment to the docstring of the butter function. This is defined here: https://github.com/scipy/scipy/blob/e6c27e50f3f98a68768291af274d783749ffdf15/scipy/signal/_filter_design.py#L2980


Sort repositories by type

Maybe a combination of both could be done:

Adding a search
Sort repositories by type and then by name:

provider
repository enabled
source repository
fork

This would allow the list to be much more friendly when like me you have 152 repositories but only one enabled

This should partially have been fixed, as right now we split the enabled and disabled repositories. The searchbar will be done in #891.
We could potentially tackle a lot with the searchbar so I'm thinking sorting is not needed (but do tell if you disagree 👍)


generateTocListDOM() returns useless DOM tree, please help.

About few months ago, someone told us how to get TOC when open a book. As following:

Now our web app has launched, but we found that for some books, readium returns a whole document DOM containing no useful TOC data as below:

Normally, a book should return a DOM structure like this:

We have tried the books that returns "Bad" DOM structure in Readium Chrome App, the TOC function works pretty well.
We don't know how to solve the problem. Maybe we are using the wrong way to get TOC data？
Please help, Thanks.

hello, the first thing that comes to mind is the fact that your webapp / cloud reader example (looking at your debugger screenshots) seems to involve opening a zipped / packed publication container, rather than an exploded / unzipped / unpacked EPUB file.
So, could you please try extracting the EPUB resources on the server, then point your web reader URL to the root folder?
Let me know is how it goes.
daniel

secondly, have you tried loading the same EPUB with the standard build of the ReadiumJS cloud reader?
eg.
https://readium.firebaseapp.com/?epub=YOUR_EPUB_URL

Hi, Daniel. Thanks for your reply.
Yes, we did load the zipped epub book directly from the browser, we load the book as a File object, most book works just fine. Our app dosen't use a server to do any work related to book rendering. So we can not rely on any server-side code.
Yesterday, I have looked into the readium-reader-lite's code and try to use the code found there to rewrite our Toc component. Now I can successfully loaded the toc dom to a special DIV, all toc items showed up as expected.
The code we are using now shows below:

Now the Toc items shows up as expected, the only different is that: we didn't put append the dom given by generateTocListDOM(). However, we got another problem. The href attribute of each  element is not ok for openConetentUrl().  As you can see in the following screenshot. The correct href should have one more folder structure before it. I have tested the "Text/xxxx.xhtml", it did works. However, this book toc works pretty well in readium chrome app as well.

The sturcture of the epub book when unzipped

So we are stucked again now. We don't know how to solve this problem.
I have tried to look into the readium code in EpubReader.js, but nothing helpful was found.
Can you please help us?
Thanks.

BTW, we want to keep the url clean, for example, no matter what book user opens, the url seen on the browser will always be http://ourapp.com. So we don't want to rely this to redirect user to specific part.

@danielweck Please help us, sir.

I am colsing it because we totally copied the code in readium example and it works. We don't know the reason yet, but this is not a issue anymore.
Thanks.