Double escape not working

I need to print number 9 into my input. Only 9. Not 8 or 7.
So my code is 'mask' => '+7 (\999) 999-99-99'
But it's useless:

However it works without yii: http://jsfiddle.net/7jhzf6oh/
Widget upgrade did not help.
https://github.com/RobinHerbots/jquery.inputmask/issues/1069

have you tried  'mask' => '+7 (\\999) 999-99-99'?

Are you sure widget was updated?

I tried double escape at first.
i cant print double escape because github moderates my double escape to one \
and my site do the same.
widget using js v.3.2.2

Well, then it is JS code thing as well... I've fixed version constraints so now it's all about plugin author. We can't do anything from Yii side.

no, no, no. this is not "inputmask.js'

error. something else destroy one of my escapes

:) That's PHP. Use 4 backslashes.

sorry, not working (:

But you've got 2 backslashes so the problem is about JavaScript plugin.

My first comment: http://jsfiddle.net/7jhzf6oh/
js without yii works perfect imho

Sorry about misleading you. The following works:
echo \yii\widgets\MaskedInput::widget([
        'name' => 'phone',
        'mask' => '+7 (\\999) 999-99-99',
]);

ok. i think it's a conflict beetween my scripts.
Your example works on one of my pages, but not working on another one.


Composer require with Symfony Flex fails.

Hi,
I am using Symfony 5 and I am unable to install this package using Symfony Flex.
Do you have an idea why I am getting below error message?
composer require sentry/sentry-symfony
Restricting packages listed in "symfony/symfony" to "5.0."
Using version ^3.5 for sentry/sentry-symfony
./composer.json has been updated
Loading composer repositories with package information
Restricting packages listed in "symfony/symfony" to "5.0."
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- The requested package symfony/polyfill-uuid could not be found in any version, there may be a typo in the package name.
Problem 2
- sentry/sentry 2.3.2 requires symfony/polyfill-uuid ^1.13.1 -> no matching package found.
- sentry/sentry 2.3.0 requires symfony/polyfill-uuid ^1.13.1 -> no matching package found.
- sentry/sentry 2.3.1 requires symfony/polyfill-uuid ^1.13.1 -> no matching package found.
- sentry/sentry-symfony 3.5.0 requires sentry/sdk ^2.1 -> satisfiable by sentry/sdk[2.1.0].
- sentry/sdk 2.1.0 requires sentry/sentry ^2.3 -> satisfiable by sentry/sentry[2.3.1, 2.3.0, 2.3.2].
- Installation request for sentry/sentry-symfony ^3.5 -> satisfiable by sentry/sentry-symfony[3.5.0].
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your minimum-stability setting
see https://getcomposer.org/doc/04-schema.md#minimum-stability for more details.
It's a private package and you forgot to add a custom repository to find it

Read https://getcomposer.org/doc/articles/troubleshooting.md for further common problems.
Installation failed, reverting ./composer.json to its original content.

symfony/polyfill-uuid 1.13.1 does actually exist: https://packagist.org/packages/symfony/polyfill-uuid#v1.13.1
It seems a local problem in your environment. Try updating Composer if you're not using the latest patch (1.10.6 as of now).

I've already tried to update composer.
Still having the same issue.
composer -V
Composer version 1.10.6 2020-05-06 10:28:10

Try composer clear-cache or composer why-not symfony/polyfill-uuid:^1.13.1

clear-cache didn't make a difference.
why-not:
composer why-not symfony/polyfill-uuid:^1.13.

  [InvalidArgumentException]
  Could not find package "symfony/polyfill-uuid" in your project

prohibits [-r|--recursive] [-t|--tree] [--] <package> [<constraint>]

You should report this to Composer, with your composer.json too.

@Jean85 yes I did, thanks for your help. I will close this ticket.


Only 19 of 21 landmarks in AFLW use? Why? Can not found settings in CPM.

hi~ I have found that you only use 19 of 21 landmarks in AFLW, and point out that it's follow CPW. But, I can't found any information about AFLW setting in CPM's paper. Could you please release the detail indices of 19 landmarks you have used in you paper?
I see the code in train.py:
points_flip = [6, 5, 4, 3, 2, 1, 12, 11, 10, 9, 8, 7, 15, 14, 13, 18, 17, 16, 19]  # no 0 and 20
points_flip = (np.array(points_flip)-1).tolist()  # then from 0~18, total 19 points

so, it means that you have exclude 0 and 20 in AFLW ?

Hi, in the paper, we did not refer to CPM, but this work: Zhu S, Li C, Loy CC, Tang X (2016) Unconstrained face alignment via cascaded compositional learning. In: CVPR. Please check again.

Thanks~ PIPNet seems can get good performance in my tests 👍👍👍

Thank you for your interests~


How about support encoding or decoding for key ?

[REQUIRED] Step 1: Describe your environment

Xcode version: 10.1
Pring version: 0.14.0

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
I declared variable as camel-case in model. But I have to save using underscore to firestore.
I found encoding or decoding for value but couldn't change for key. I try to change _properties but it's private variable.
How about support decodeKey, encodeKey ?
Relevant Code:
// TODO(you): code here to reproduce the problem

You can override ModelName.
https://github.com/1amageek/Pring/blob/master/Pring/Object.swift#L20

@1amageek I just said property name.

@YukSeungChan Sorry. not supported

@1amageek I think it's better to add a feature and I will send PR when I have it done. How is it sound?


Overhaul GitHub Actions workflow(s)

@aepfli @Michael1993 since this is still open, I incorporated some other GHA-related changes from Pioneer. Namely, using the new Gradle build action, updating Gradle to 7.3 and building against Java 17. Would you mind having a (second) look?


Ensure that Fact#command does not take global arguments

A recent commit c89e21142523b540ba8cf18dc8d54daf5113773c changed getting facts so that arguments to FactBase#command clashing with global arguments are filtered from the kwargs. This includes the often used argument name and facts like DebPackage broke as a result.
This PR introduces a check that subclasses of FactBase don't take any global arguments in their command method and updates all existing facts and tests accordingly. There is also a tiny bit of improved shell quoting in the deb, pacman and rpm facts.

A recent commit c89e211 changed getting facts so that arguments to FactBase#command clashing with global arguments are filtered from the kwargs. This includes the often used argument name and facts like DebPackage broke as a result.

heh, guess we know why the old handling existed! Lack of comments on that meant I had long forgotten the context which makes sense! This approach is cleaner, and although the global name argument clash is annoying it makes sense to keep things consistent between facts and operations.
Not quite got time to review code right now but thank you for pulling this one together @martenlienen 🙏


Allow link buttons inside another form when method/action set

Adding the below link inside a form doesn't work as expected even though both url and method are set.
Html::a(
   'Delete',
    ['item/delete', 'id' => $item->id],
    [
        'data' => [
            'method' => 'post',
            'confirm' => Yii::t('app', 'Are you sure you want to delete this?'),
        ]
    ]
)

Have you tried with code from master?

Yes, Just tried master,  Results using master...
HTML is
<form id="active-form" action="/model/update/1" method="post" role="form">
Delete
</form>

Request
Remote Address:<IP_ADDRESS>:80
Request URL:http://localhost/model/update/1
Request Method:POST
Status Code:302 Found

Seems like there are a few issues re:this. #8014 #6661 #8801

Hmm, should've been fixed by https://github.com/yiisoft/yii2/commit/926770b7d9249093737f32011b300c8e2af8c271

Verified that there's no issue with master code anymore using advanced project template:

In contact.php add the following right after submit button:

<?= Html::a(
   'Delete',
    ['site/index', 'delete' => 1],
    [
        'data' => [
            'method' => 'post',
            'confirm' => Yii::t('app', 'Are you sure you want to delete this?'),
        ]
    ]
) ?>

After submit it makes post request to index page with delete parameter as expected.


Consider removing unnecessary content from PR template and convert to embedded comment

Front matter
Why are we using this front matter which adds no value to the PR description, but confusion and additional lines to change while opening the PR?
https://github.com/eclipse-tractusx/.github/blob/3ba80ab1469dcd10c009888766a4787679d01aba/.github/pull_request_template.md?plain=1#L1-L5
If you want the PR authors to use a specific format, eg for the PR title use a PR linter. An example implementation can be found here: eclipse-tractusx/tractusx-quality-checks/.github/pullRequest-lint.yaml

Section: Description
I'd tend to remove the hole section as the complete input is considered as description. If a contributor is using extended commit messages to prepare the PR description, the commit message will be used as PR title, and the extended commit message will be used in the PR description which will be added at the top of the PR description (not under your Description section) → manual editing is required by PR author.
https://github.com/eclipse-tractusx/.github/blob/3ba80ab1469dcd10c009888766a4787679d01aba/.github/pull_request_template.md?plain=1#L7-L12
Convert section content into a embedded comment if you want to provide these content anyway, and remove heading.
Section: Additional information
This section makes the PR description bigger than needed an adds no value to the PR description itself. Add/convert these information as embedded comment into the PR template instead. The PR author will see these hints, but they won't appear in the PR itself.
https://github.com/eclipse-tractusx/.github/blob/3ba80ab1469dcd10c009888766a4787679d01aba/.github/pull_request_template.md?plain=1#L21-L27

@SebastianBezold thanks for merging, are there theese comments to make it even better anywhere stated or i can also find them, or should we discuss it in an open round?


Consistent way of writing

Hey,
first of all: lovely work. Thanks a lot.
I just thought, that it might be nice to be consistent in who you write the headlines for what mention it is. Either all lower case or all upper case. Easy to solve on my end of course, but I thought you maybe want to have a consistent way of doing this in the repo already. I attached screenshots of what I mean. Maybe you also had a reason for doing it this way … ;)

Cheers,
/marc

Oh, I actually don't use that part of the code! This comes from the original kirby-webmentions plugin by Bastian Allgeier. I have my own snippets in site/snippets/webmentions, with Dutch translations of the above, but also with a few changes regarding presentation.
See examples here:
https://indieweb.org/facepile#Sebastiaan_Andeweg
I don't know if I want to ship the plugin with those icons, but I will make some changes in the default snippets to solve this better :)
(Also, make sure you got the latest version of either my seblog-kirby-webmentions or Bastian's kirby-webmentions, because it fixes a nasty vulnerability. Haven't blogged about it yet, I will soon.)

Ah ok. Thanks for clarifying. I don't think Bastian is maintaining the plugin right now. Will look deeper into your other things as well. The toolkit looks very nice.

Nope, that's why I started my own fork. First just for myself, but I think it's nice to have a version that is useful to others also. So: thanks for using it! Feedback is appreciated :)

Great. Again: your work is highly appreciated ;)
I still ave a lot to integrate such as the possibility to reply from within my blog. I wish I could attend the IndieWebCamp I organise in Düsseldorf once, to find the time. Are you coming?

Well, if you don't mind doing some manual work, you can actually just add:

In reply to Sebastiaan

... :)

... to the text field of your entry, and a  to the h-card in your footer.
The plugin should pick that up and send a mention. Or real manual: you can put the URL in the form underneath my reply, then my site sends the mention to itself by itself :)
I use a separate field for it, because it makes sense to do so in Kirby, but it's not required!
I plan on coming to Düsseldorf, really looking forward to that!


Feat:Profile page options get hidden under the book, scroll, translate icons on the right

Describe the feature
Profile page options get hidden under the book, scroll, translate icons on the right which can be transferred to the left hand side of the page to improve the page structure and visibility. Meanwhile social media icon could be placed at the bottom in the footer section.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 kindly assign this issue to me. Iam sure i will be able to solve this.

please assign  me this isue

/assign


written the program for converting celsius to fahrenheit

written the program for converting a given temp in degree celsius to degree Fahrenheit ,
here is the output :

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


RandSpatialCropSamplesd and CopyItemsd problem

When I compose
RandSpatialCropSamplesd(keys="image", roi_size=(128, 128, 128), random_size=False, num_samples=2,),
CopyItemsd(keys=["image"], times=1, names=["image2"], allow_missing_keys=False), it will make mistakes in CopyItemsd. My guess is that RandSpatialCropSamplesd returns the value as a list. Is there any way to solve this problem? This is code from the self-supervision tutorial

I couldn't replicate your issue:
import numpy as np

from monai.transforms import Compose, CopyItemsd, RandSpatialCropSamplesd

xform = Compose(
    [
        RandSpatialCropSamplesd(keys="image", roi_size=(128, 128, 128), random_size=False, num_samples=2),
        CopyItemsd(keys=["image"], times=1, names=["image2"], allow_missing_keys=False),
    ]
)
output = xform({"image": np.zeros((1, 200, 200, 200))})
print(output[0]["image"].shape)

please create a separate issue with detailed information so that we can help here.


Add files via upload

the school official website URL
https://www.unifi.it/
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
https://www.unifi.it/vp-10306-schools.html
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
Please check attached a file.

@cuntu I am sorry, this request has signs of fraud. I have to close it without merging. If you are a student of this school, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


Add support for breadcrumbs in .ion files.

Issue #, if available:
Fixes #38
Description of changes:
Adds breadcrumb support for .ion files.

Position within a struct is given by the field name
Position within a list is given by "[" $index "]"
Position within a sexp is given by "(" $index ")"
No information is given for position of a top-level value within the file

I thought it would be convenient to disambiguate sexp and list in the breadcrumb info, hence the [] and ().
E.g.:
{
  exprs: [
    (
      aaa
      +
      bbb
    ),
    (
      ccc
      +
      ddd
    ),
  ]
}

If you have your cursor in bbb, the resulting breadcrumb is exprs > [0] > (3).
For comparison, Intellij's breadcrumbs for .json files provides property name for elements in an object, and unadorned index for position within an array.
In order to make this work, I had to refactor the plugin's bnf grammar because otherwise operator symbols were not parsed but not made available as a PsiElement—that's why the diff is so large. You can ignore all files with gen in the path since they are automatically generated from them contents of the .bnf file.
Finally, I also make a few adjustments to the workflow so that it wouldn't try to publish to the beta channels from a fork.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is there a particular reason the generated files are tracked in git?

Is there a particular reason the generated files are tracked in git?

Yes. The project was not set up so that the files are generated by the build process—the get generated by running the IntelliJ language-kit plugin on the .bnf file, so we have to commit them to keep them around.
It looks like it's (now) possible to use a gradle plugin instead, but that's a little beyond the effort I want to put into this right now.


disable failing tests on fpdf2

Description of changes
Fixes/ works around https://github.com/NixOS/nixpkgs/issues/308941 - two tests are disabled which try to download a file, as well as an additional JPEG2000 test which is failing for unclear reasons.
Part of Zero Hydra Failures #309482
Still has some build failures on packages that were failing prior to this patch as well - these appear to be broken upstream, including:

docformatter
certomancer
comicon: marked as broken for pypdf>4.0.0

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[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

[x] 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

Note: some packages (python3.12-docformatter-1.7.5, python3.12-certomancer-0.11.0, and python3.12-comicon-1.0.1/python3.11-comicon-1.0.1) broken, but they appear to have been broken before this change as well.

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

Full nixpkgs-review log:
...
┣━ Dependency Graph:
┃             ┌─ ⚠ python3.12-docformatter-1.7.5 failed with exit code 1 after ⏱ 2s in pytestCheckPhase
┃          ┌─ ⏸ python3.12-xsdata-24.2.1
┃          ├─ ⚠ python3.12-certomancer-0.11.0 failed with exit code 1 after ⏱ 9s in pytestCheckPhase
┃       ┌─ ⏸ python3.12-pyhanko-0.21.0
┃    ┌─ ⏸ python3.12-xhtml2pdf-0.2.15
┃    ├─ ⏸ python3.12-mandown-1.7.0 waiting for 1 ⚠
┃    ├─ ⏸ python3.11-mandown-1.7.0 waiting for 1 ⚠
┃    ├─ ⏸ python3.11-mandown-1.7.0 waiting for 1 ⚠
┃    ├─ ⚠ python3.12-comicon-1.0.1 failed with exit code 1 after ⏱ 0s in pythonRuntimeDepsCheckHook
┃    ├─ ⚠ python3.11-comicon-1.0.1 failed with exit code 1 after ⏱ 0s in pythonRuntimeDepsCheckHook
┃ ┌─ ⏸ env
┃ ⏸ review-shell
┣━━━ Builds
┗━ ∑ ⏵ 0 │ ✔ 0 │ ⏸ 8 │ ⚠ Exited after 4 build failures at 09:19:40 after 16s
12 packages failed to build:
comic-mandown comic-mandown.dist python311Packages.comicon python311Packages.comicon.dist python311Packages.mandown python311Packages.mandown.dist python312Packages.comicon python312Packages.comicon.dist python312Packages.mandown python312Packages.mandown.dist python312Packages.xhtml2pdf python312Packages.xhtml2pdf.dist

48 packages built:
calibre-web calibre-web.dist diffoscope diffoscope.dist diffoscope.man diffoscopeMinimal diffoscopeMinimal.dist diffoscopeMinimal.man maigret maigret.dist mdbook-pdf-outline mdbook-pdf-outline.dist pretix pretix.dist python311Packages.camelot python311Packages.camelot.dist python311Packages.clarifai python311Packages.clarifai.dist python311Packages.fpdf2 python311Packages.fpdf2.dist python311Packages.llama-index python311Packages.llama-index-readers-file python311Packages.llama-index-readers-file.dist python311Packages.llama-index-readers-s3 python311Packages.llama-index-readers-s3.dist python311Packages.llama-index.dist python311Packages.pypdf python311Packages.pypdf.dist python311Packages.pypdf.doc python311Packages.xhtml2pdf python311Packages.xhtml2pdf.dist python312Packages.camelot python312Packages.camelot.dist python312Packages.clarifai python312Packages.clarifai.dist python312Packages.fpdf2 python312Packages.fpdf2.dist python312Packages.llama-index python312Packages.llama-index-readers-file python312Packages.llama-index-readers-file.dist python312Packages.llama-index-readers-s3 python312Packages.llama-index-readers-s3.dist python312Packages.llama-index.dist python312Packages.pypdf python312Packages.pypdf.dist python312Packages.pypdf.doc sasview sasview.dist

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

Result of nixpkgs-review pr 309846 run on x86_64-linux 1

12 packages failed to build:

comic-mandown
comic-mandown.dist
python311Packages.comicon
python311Packages.comicon.dist
python311Packages.mandown
python311Packages.mandown.dist
python312Packages.comicon
python312Packages.comicon.dist
python312Packages.mandown
python312Packages.mandown.dist
python312Packages.xhtml2pdf
python312Packages.xhtml2pdf.dist

48 packages built:

calibre-web
calibre-web.dist
diffoscope
diffoscope.dist
diffoscope.man
diffoscopeMinimal
diffoscopeMinimal.dist
diffoscopeMinimal.man
maigret
maigret.dist
mdbook-pdf-outline
mdbook-pdf-outline.dist
pretix
pretix.dist
python311Packages.camelot
python311Packages.camelot.dist
python311Packages.clarifai
python311Packages.clarifai.dist
python311Packages.fpdf2
python311Packages.fpdf2.dist
python311Packages.llama-index
python311Packages.llama-index-readers-file
python311Packages.llama-index-readers-file.dist
python311Packages.llama-index-readers-s3
python311Packages.llama-index-readers-s3.dist
python311Packages.llama-index.dist
python311Packages.pypdf
python311Packages.pypdf.dist
python311Packages.pypdf.doc
python311Packages.xhtml2pdf
python311Packages.xhtml2pdf.dist
python312Packages.camelot
python312Packages.camelot.dist
python312Packages.clarifai
python312Packages.clarifai.dist
python312Packages.fpdf2
python312Packages.fpdf2.dist
python312Packages.llama-index
python312Packages.llama-index-readers-file
python312Packages.llama-index-readers-file.dist
python312Packages.llama-index-readers-s3
python312Packages.llama-index-readers-s3.dist
python312Packages.llama-index.dist
python312Packages.pypdf
python312Packages.pypdf.dist
python312Packages.pypdf.doc
sasview
sasview.dist


Meta data for NFTs that an account holds

Problem
User has 20 of one NFT in the token balances section for account 0.0.326181. There is no way to drill down and see the details of the NFTs they are holding for each individual NFT.
Solution
There should be a way to review the details of each individual NFT the user holds.
Alternatives
No response

duplicate #525


Account wide tasks

The ability to add tasks to all characters

Variables moved from per character to account wide

Ready for deploy


Start adding document types to index

The "document supertype" is a grouping for document types. It will allow us to segment pages by more types, like "Publication" or "Announcement" (used in Whitehall).
With this we'll be able to search like this:
https://www.gov.uk/api/search.json?filter_navigation_document_supertype=guidance
https://www.gov.uk/api/search.json?filter_whitehall_document_supertype=publication

See https://github.com/alphagov/govuk_document_types/pull/1.
https://trello.com/c/07lAcDtC

I'm fine with the rummager changes, just wondering if we need a gem for this or we could add this into publishing api somehow.
Would it work have a document type of document_type_group and make each of these groupings a separate content item? (This may be a really terrible idea)

Good point, @carvil and I talked to the architects about this, but we didn't write down the outcome of that (which was to use this method). I'll write up some of the considerations today.


[Feature Request] Improve Backgrid accessibility

I'm evaluating using Backgrid for a current project, and I love how it integrates with my PageableCollection.
However, the HTML rendered by Backgrid.Grid, Backgrid.Extension.ServerSideFilter, and Backgrid.Extension.Paginator could be much more accessible. There could be more issues than the following, but these are what I've found so far from the components that I'm using.

The table is missing a <caption> element.
The table is missing 'scope' attributes on the <th> elements
The table uses <a> tags for sorting, but they should be buttons. (<a> tags initiate navigation while <button> tags take an action
The filter/search <input> is missing an accompanying <label>
The filter/search <input> is missing an 'aria-labeledby' or 'aria-label' attribute
The search clear button is an <a>, but it should be a <button>

Regarding the <a> vs <button> issue above, I suppose it depends on how users are using Backgrid. If it updates the URL, which deep-links to a particular table state, then I think the <a> is appropriate (though they are missing valid 'href' attributes). However, for all other cases, I think the <button> is more semantically appropriate.
I may discover more issues as I continue evaluating Backgrid. I can fix them in my own project, but I'd much rather merge my fixes into this repository.
Fixing these issues will have no visual impact and benefit all users with disabilities.

I too am facing these same problems. There is also no tabbing to the grid/table, as tabbing skips straight over the grid to the paginator. I really need to be able to tab through each row and each cell, like a disabled person would want.

@dan-f  I'd very much love to see an accessibility PR. I currently have no bandwidth to deal with this, so I'd definitely appreciate any help I can get in this area.

Hey @wyuenho. Sorry for the lack of an update on my part. I also have little bandwidth for this at the moment, though I'm happy to take on this work when I get a moment. I'm realistically not likely to have time to work on this for a while. Would you prefer that I leave the issue open or closed?

No worries! We can just keep this open

On 9 Mar 2016, at 8:43 PM, Daniel Friedman<EMAIL_ADDRESS>wrote:
Hey @wyuenho. Sorry for the lack of an update on my part. I also have little bandwidth for this at the moment, though I'm happy to take on this work when I get a moment. I'm realistically not likely to have time to work on this for a while. Would you prefer that I leave the issue open or closed?
—
Reply to this email directly or view it on GitHub.

Another accessibility issue we are facing is that the grid does not render until AFTER document.ready completes. This causes screenreaders to ignore Backgrid all together. We may not be able to use it, unless we figure out a fix.

Also (one more thing), to fix the tabbing problems we faced, we have to change the 'UriCell' extension method:
render: function () {
this.$el.empty();
var rawValue = this.model.get(this.column.get("name"));
var formattedValue = this.formatter.fromRaw(rawValue, this.model);
this.$el.append($("", {
tabIndex: -1,
href: rawValue,
title: this.title || formattedValue,
target: this.target
}).text(formattedValue));
this.delegateEvents();
return this;
}
** simply remove tabIndex = -1 and your grid is now tab friendly....

@dulldave you mind sending over a PR for the accessibility improvements?

Hey @wyuenho, sure thing. I'll have to find some time as I'm bogged down, but I'd love to help out :)

Another thing: it doesn't look like there is a way to create a "HeaderColumn" so that you can have table with two headers.
This is important for screen readers because it would give additional context to a focused cell (e.g. "Wednesday Closed for 11:00 - 13:00" instead of just "Wednesday Closed").


Fixed typos

Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>

Thanks for fixing all my typos!


HL_WCO2 Works fine

Describe The Bug:
HL_WC02 not supported submit a request
Message keeps popping up in logs.
To Reproduce:
As said above
Expected behavior:
It is support so the error message should not appear

However, it does not work on the latest alpha version not sure why. When I downgrade one version down it works fine except for the message.

38 fixes the 403 error however like 37 alpha it doesn’t show the on/off switches for my Wyze outdoor 2 cams for some reason. It worked with version 36 before the 403 error any insights would be great. Interestingly it shows 5 devices. 2 Wyze cam v3 which work and 2 outdoor v2 cam and bridge that do not work.

Ok I see what is going on. You have excluded that device for some reason. I have unexcluded the device to test from inside the enums.js file and it loads fine. Thanks.

https://github.com/jfarmer08/homebridge-wyze-smart-home/releases/tag/v0.5.39

https://github.com/jfarmer08/homebridge-wyze-smart-home/releases/tag/v0.5.39

I think you typo’s the string for the outdoor v2.
This is what I have that works.

Looks like I had a zero instead of a O. I just pushed a new release to correct this.


login SP graph permission required

if login sp does not have access to graph permission, the command will result in error like:
Operation failed with status: 'Forbidden'. Details: 403 Client Error: Forbidden for url: https://graph.windows.net/xxxxxxxx-0000-0000-0000-000000000000/getObjectsByObjectIds?api-version=1.6
the login SP need to be configured to give permission on Microsoft Graph .

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thank you @yugangw-msft. Yes that is correct, two application permission needs to be configured. I have  updated the pull request, please check.

@afengli, I don't think  Microsoft Graph is needed; also unless the service principal needs to run commands under az ad, it doesn't really need to access any other resources

Please reopen when the change is ready


cli/sql: bump lib/pq to 1.8.0

PR: https://github.com/cockroachdb/cockroach/pull/51947
From release notes:

Changed the Kerberos URL connection string parameters to the standard krbsrvname (previously service) and krbspn (previously spn). [#51947][#51947] {% comment %}doc{% endcomment %}

Looks like this is yours, @Amruta-Ranade.


WebAssembly compatibility

I'm trying to use this crate with wasm-bindgen, and it seems that using u32 indices does not work. Specifically this line
https://github.com/sdd/kiddo/blob/4e2dbf471229266a88ce3ef8dcbea38ebc0b76b0/src/types.rs#L80
panics if the crate is built on wasm target. Here's a minimal example to reproduce the issue:
use kiddo::{float::kdtree::KdTree};
use wasm_bindgen::prelude::*;

#[wasm_bindgen]
pub struct KdTreeWasm {
    tree: KdTree<f64, u64, 2, 32, u32>,
}

#[wasm_bindgen]
impl KdTreeWasm {
    #[wasm_bindgen(constructor)]
    pub fn new() -> Self {
        let tree: KdTree<f64, u64, 2, 32, u32> = KdTree::with_capacity(5);
        Self { tree }
    }
}

and on the JavaScript side, simply just initialize:
import('../pkg/index.js').then({ KdTreeWasm } => {
    let tree = new KdTreeWasm();
});

and this'll panic at attempt to multiply with an overflow.
Is this behaviour expected? Am I missing something?
If I'm not mistaken, capacity_with_bucket_size only works with u32 if #[cfg(target_pointer_width = "64")].

Fix released in v4.1.1


Thread exceptions due to UI elements not being able to be accessed in multiple threads

Not able to access UI elements like Windows.Shape across multiple threads. This is apparently causing thread exceptions in Whiteboard and corresponding UX. A possible fix for this could be to create and use those UI elements in the same thread. Please fix accordingly @ak-303

Raised a PR for the same:  https://github.com/manassharma158/meet.me/pull/35
Tested and verified with @tripathiarpan20. Now its working.


No docs yet :cry:

This has turned complicated enough to need at least some documentation, if other people are gonna try to use it :sweat_smile:
So I'll need to get to that :D

I'll create a docs category for several branches to work on.

docs/website
docs/manual

and so (or maybe do that on another repo?)


Add mca XmlnsPrefix for http://schemas.microsoft.com/netfx/2009/xaml/…

…activities
This will fix #107 and improve compatibility with older projects

@lbargaoanu  check this out!

Look at the tests. The workflows run just fine.

I can't see the settings since the pr is closed

Ok, sorry I missing that XmlnsCompatibleWith solve the problem case expose the .dll as provider for that assembly


Test does not run via grunt/phantom, but OK in web browser

OS 10.8.2, using phantomjs 1.8.1 and grunt v0.3.17:

git clone --recursive git@github.com:fabien-d/require-qunit-test.git
grunt qunit (success: "1 assertions passed")
Edit test/index.html and use the ../vendor/require-2.1.4.js instead of the 2.0.6 script.
grunt qunit (failure: 0 assertions passed)

This was originally filed as jrburke/requirejs#618. I am filing a ticket here because the require-2.1.4.js version of the test runs fine in QUnit in the browser, just opening test/index.html in a web browser.
While I am open to this being an issue with requirejs, my first thought is it may be due to how the qunit grunt task binds to the running qunit tests and how it detects with QUnit is "done".
One guess at the problem may be that requirejs 2.1.4 waits a bit longer to start the actual qunit test, after whatever signal the grunt qunit driver uses to detect "done".
So, any insight into what I might look at in the grunt qunit code, or pointers is appreciated.

Found this related to the issue. http://stackoverflow.com/questions/18392463/grunt-not-running-qunit-tests-on-phantom
Is there a solution yet?

I've run into this issue for every requirejs-based grunt-contrib-qunit test I have, in several different projects.  Doesn't affect tests that don't use require.js.
Suspect it is realted this change: db075a2b0a5e0f00390f39538cf6d18ff098673a

@davidbau that change is a week old, this issue and most of the comments are much older.
For an example of using QUnit, grunt-contrib-qunit and AMD via requirejs, check out these two files.

Going to close this out, as @jzaefferer already pointed (https://github.com/gruntjs/grunt-contrib-qunit/issues/19#issuecomment-90588136) to an example of using QUnit, grunt-contrib-qunit and requirejs, which we can consider as a solution.
for a similar solution, see http://stackoverflow.com/a/34928952/3904791


Introducing IndraHttpClient

IndraHTTPClient can be used to communicate with the service endpoint via
HTTP.
Currently supports Relatedness requests.
Moving RelatednessRequest and RelatednessResponse to indra-common
to serve as the nexus between the indra-client and indra-service.
I incremented the PATCH version number, to avoid further breaking other projects. Unsure if you plan to use semver.

I've now seen the client operation in stargraph and realized that the existing restlet implementation might be the better option for future operation.
What do you think?

@blang for the contribution.
In fact it is planned to have a client to easy integration.  We have a couple of improvements on the API and we were just postponing a little bit. I will revise your PR soon.
About the semver. In fact is being used. In the beginning things changes too much hence we opted to make some release candidates before the oficial 1.0.0 version. You can see the tag 1.0-rc3 as the latest, but we kept the pom with the snapshot version.
Restlet was used before in Indra. As a matter of choice (licensing, popularity, support, transitive dependencies etc) we dropped the restlet library to stay with jersey ref implementation.
thanks for the feedback!

Ok, so you don't want any client class like IndraHttpClient but simply move the resources to common?
I will open another PR for it if you want.
In this case, just close this PR

Yes. We're using JAX-RS standards hence I think this makes sense to keep the solution cleaner and let the users chose whatever they want as the client framework. Nowadays there are so many options, restlet, jersey, okhttp, unirest etc.


Wrench BadRequestException: No origin header

Hi there,
I'm trying to run the builder command with the watch and auto reload server with this command:
docker run -t --rm --name my-running-script -v "$(pwd)":/usr/src/myapp -w /usr/src/myapp -p 7007:8002 php:5.6-cli php core/builder.php -w -r

It basically gets a PHP Docker to run my command. It seems to work, the output is:
configuring pattern lab...
your site has been generated...
site generation took 0.27731108665466 seconds and used 4MB of memory...
starting page auto-reload...
watching your site for changes...
info: Server initialized

But then when I try to connect to my site on the browser, I see just a blank page and the only error is a 404 for the favicon.
On the server side, I get this output:
info: Wrench\ConnectionManager: Wrench\Connection: <IP_ADDRESS>:65245 (2762c0c7d40637e8455ad4b9fdad42c0065a859da273d713eb5e7d6e73bb2177424d01a41f95f285c3190f16fc5ab5f23c411b58cfacad87522f5e67be297a4b): Connected
warning: Wrench\ConnectionManager: Error on client socket: exception 'Wrench\Exception\BadRequestException' with message 'No origin header' in /usr/src/myapp/core/lib/Wrench/Protocol/Protocol.php:455  Stack trace:#0 /usr/src/myapp/core/lib/Wrench/Connection.php(227): Wrench\Protocol\Protocol->validateRequestHandshake('GET / HTTP/1.1\r...')#1 /usr/src/myapp/core/lib/Wrench/Connection.php(210): Wrench\Connection->handshake('GET / HTTP/1.1\r...')#2 /usr/src/myapp/core/lib/Wrench/Connection.php(403): Wrench\Connection->onData('GET / HTTP/1.1\r...')#3 /usr/src/myapp/core/lib/Wrench/ConnectionManager.php(245): Wrench\Connection->process()#4 /usr/src/myapp/core/lib/Wrench/ConnectionManager.php(172): Wrench\ConnectionManager->processClientSocket(Resource id #146)#5 /usr/src/myapp/core/lib/Wrench/Server.php(201): Wrench\ConnectionManager->selectAndProcess()#6 /usr/src/myapp/core/autoReloadServer.php(45): Wrench\Server->run()#7 {main}

@felippenardi -
I'll be honest, I'm not in a good position to debug this nor am I really familiar with the Wrench stuff. I just plugged it together and stuff seemed to work. Sorry.

I see this is closed, but I am also having the same problem with files when trying to use the servers in core/scripts. Tried it with my old files, as well as a fresh new clone of pattern-lab/patternlab-php and I keep seeing 400 error when trying to load the page, and wrench related errors in the terminal.

Hey, im having this same issue. I tried moving my folders to default web server location but that didn't help. @dmolsen @felippenardi @linoleumtile
AFAIK, when starting the Wrench server, the Origin field in the header is sent blank, which is why it is throwing a BadRequestException (why not an InvalidOriginException instead?).
Any leads to fixing this?

@guile I've been just serving patternlab>public with jekyll and running the generate script when I need to. Not a fix, but it gets the job done for me.

This was happening with my Grunt setup and I think it's because I had a Shell task running 'php core/builder.php -wr' after Sass changes but 'php core/builder.php -wr' was already running because it was called in my default (concurrent) task. I was making it too complicated.
Now I just have a Sass watch task that runs Compass and I let Pattern Lab's native watch task ('php core/builder.php -wr') handle the rest. So maybe check if that's the case with your particular setup. I can share my Grunt setup if that would help anyone.


chore: add a dockerhub link to readme

The primary consumer for generated JSON files is aurora-relayer. Recommended way to use it would be to use standalone-rpc. Perhaps it is worthwhile to mention this in readme? WDYT? Something like:
Data Consumption and Further Usage
The primary consumer for the data generated by the Refiner application is the aurora-relayer. This component consumes refined data produced by Refiner to populate Relayer database and keep it in sync with NEAR chain.
Using Aurora Relayer with Standalone RPC
For optimal integration and usage, it is recommended to use standalone-rpc. It is a comprehensive script designed to facilitate the smooth rollout of both the Aurora relayer and the refiner. Moreover, it assists in downloading all necessary snapshots required. This approach simplifies the initial setup process, ensuring that users can quickly and efficiently start working with Aurora's blockchain data and services.

@spilin Actually, this is a great suggestion! ❤

@spilin Added your text. Looks very nice!