Linter message: "not compatible with rangeIncludes"

Hello,
I am constructing a structured data bibliography with JSON-LD.
When I use the linter to evaluate a list of CreativeWorks that includes itemListElements with IRI identifiers (for example, the DOI of an article: "@id":"http://dx.doi.org/10.3917/psy.031.0093"), I receive the following message for some but not all of the IRIs:
property schema:itemListElement: Object <IRI> not compatible with rangeIncludes (schema:ListItem,schema:Text,schema:Thing)
I am confused by this message because each itemListElement is further identified as a type of CreativeWork (for example, "@type":"Chapter" or "@type":"ScholarlyArticle"). I would have thought that the identifier would apply to the CreativeWork rather than to the higher-level itemListElement container. If need be, I could certainly change the DOIs to simple "url" properties, but that does not address the cases in which I would like to use an "@id" to identify a periodical in which several articles have appeared.
The JSON-LD passes Google's structured data testing tool, and it is considered valid by the tools at http://jsonviewer.stack.hu/ and http://jsonlint.com/.
Are these error messages? Should I be worried about them?
Here's a more complete example, with the DOIs changed from "@id" to "url" but the "@id" for periodical retained:
<script type="application/ld+json">
{
  "@context": "http://schema.org",
  "@type": "ItemList",
  "@id": "https://example.com/en/writing.html#publications",
  "itemListElement": [
    {
      "@type": "ScholarlyArticle",
      "url": "http://dx.doi.org/10.3917/psy.031.0093",
      "name": "Entretien avec Frédéric Diart, peintre classique",
      "datePublished": "2014",
      "pageStart": "95",
      "pageEnd": "104",
      "author": [
        {
          "@type": "Person",
          "@id": "http://orcid.org/0000-0002-8518-7999"
        },
        {
          "@type": "Person",
          "name": "Véronique Sidoit"
        }
      ],
      "inLanguage": "fr",
      "isPartOf": {
        "@type": "PublicationIssue",
        "issueNumber": "31",
        "isPartOf": {
          "@type": "Periodical",
          "@id": "https://www.worldcat.org/title/psychanalyse/oclc/464588080"
        }
      }
    },
    {
      "@type": "ScholarlyArticle",
      "name": "La fin du monde",
      "datePublished": "2013",
      "pageStart": "61",
      "pageEnd": "76",
      "url": "http://dx.doi.org/10.3917/psy.028.0059",
      "author": {
        "@type": "Person",
        "@id": "http://orcid.org/0000-0002-8518-7999"
      },
      "inLanguage": "fr",
      "isPartOf": {
        "@type": "PublicationIssue",
        "issueNumber": "28",
        "isPartOf": {
          "@type": "Periodical",
          "@id": "https://www.worldcat.org/title/psychanalyse/oclc/464588080"
        }
      }
    },
    {
      "@type": "Chapter",
      "name": "Caring for Knowledge: Transmission in 'The Figure in the Carpet' and 'Nona Vincent'",
      "datePublished": "2013",
      "pageStart": "70",
      "pageEnd": "78",
      "author": {
        "@type": "Person",
        "@id": "http://orcid.org/0000-0002-8518-7999"
      },
      "inLanguage": "en",
      "isPartOf": {
        "@type": "Book",
        "name": "Henry James and the Poetics of Duplicity",
        "isbn": "9781443844178",
        "@id": "http://www.worldcat.org/isbn/9781443844178",
        "url": "http://www.cambridgescholars.com/henry-james-and-the-poetics-of-duplicity-14",
        "sameAs": "http://www.sudoc.fr/170467511",
        "editor": [
          "Annick Duperray",
          "Adrian Harding",
          "Dennis Tredy"
        ],
        "publisher": {
          "@type": "Organization",
          "name": "Cambridge Scholars Publishing"
        }
      }
    },
    {
      "@type": "Periodical",
      "@id": "https://www.worldcat.org/title/psychanalyse/oclc/464588080",
      "name": "Psychanalyse",
      "url": "http://www.editions-eres.com/collection/156/psychanalyse",
      "sameAs": "http://www.sudoc.fr/085440868",
      "issn": "1874-9062"
    }
  ]
}
</script>

The linter returns the following message for that code:
property schema:itemListElement: Object <https://www.worldcat.org/title/psychanalyse/oclc/464588080> not compatible with rangeIncludes (schema:ListItem,schema:Text,schema:Thing)
For some reason, the "@id" for the "Book" in the "Chapter" item above does not get flagged. It is true that this "@id" is not cited again on the same page, but then again the DOI "@id"s for the "ScholarlyArticle" items did get flagged even though they only appeared once.
Thanks in advance for any clarifications you can provide about this, and thanks also for this excellent tool, which has really helped my attempt to learn to use schema.org.

The Linter uses a reasoner with special rules for schema.org. I think one of these rules is too restrictive.
itemListElement has the range of ListItem, Text, and Thing. Periodical is (via inference) a kind of Thing, which should be allowed. However, there's a check that specifically says if any range is Text, and the value as an object (has @id), then it can't also have a type. This is too restrictive, and it should first see if the range includes a super-class of any type actually used.
There is no formalized description of Schema.org semantics, so they've sort have been ad-hoc. I'll take a look and see how to accommodate this without making the reasoner overly complex.
(FYI, you can find the rules for range checks here).

Thank you very much for the quick response. Your comments have given me a better sense of what ranges imply, something I hadn't really thought about before in my own ad-hoc approach as a user.
I agree that this particular rule seems a bit too restrictive, since it effectively excludes lists of different types of objects that very often do "ultimately" refer to texts in some sense of the term (although DOIs are also used for video content).
Thanks again!


Support ESLint v8

Description
Fixes #300
Updates @shopify/eslint-plugin to support eslint v8.

Increase peer dependency on eslint from ^7.30.0 to ^8.3.0
Increase node version to run on CI to 12.22, as that is the minimum version that eslint requires.
Replace usage of the long-deprecated babel-eslint and eslint-plugin-babel with @babel/eslint-parser and @babel/eslint-plugin.

Update babel/* rules to @babel/*
Remove babel/camelcase and babel/valid-typeof rules as they do not exist in @babel/eslint-plugin.
Update setting parsers from babel-eslint to @babel/eslint-parser
Update usage of ClassProperty ast type to also check for PropertyDefinition

Update versions of all plugins. Note major version bumps to eslint-plugin-jest and typescript-eslint/* and eslint-plugin-prettier which caused the following changes:
Remove jest/prefer-inline-snapshots rule
Replace jest/prefer-to-be-null and jest/prefer-to-be-undefined and jest/no-expect-resolves  rules with jest/prefer-to-be and jest/no-restricted-matchers
Rename jest/valid-describe rule to jest/valid-describe-callback
Rename jest/lowercase-name rule to jest/prefer-lowercase-title

Package
Old version
New version

typescript-eslint/eslint-plugin
^4.28.2
^5.4.0

typescript-eslint/parser
^4.28.2
^5.4.0

eslint-plugin-import
^2.23.4
^2.25.3

eslint-plugin-jest
^24.3.6
^25.3.0

eslint-plugin-jest-formatting
^3.0.0
^3.1.0

eslint-plugin-jsx-a11y
^6.4.1
^6.5.0

eslint-plugin-prettier
^3.4.0
^4.0.0

eslint-plugin-promise
^5.1.0
^5.1.1

eslint-plugin-react
^7.24.0
^7.27.1

eslint-plugin-react-hooks
^4.2.0
^4.3.0

eslint-plugin-sort-class-members
^1.11.0
^1.14.0

Type of change

eslint-plugin: Major

Hey @BPScott - is there a possibility of a new release, so we can start using this? 🙏


Update metabrowse-server to 0.2.7

Updates org.scalameta:metabrowse-server from 0.2.3 to 0.2.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.2.3).
You might want to review and update them manually.
docs/website/package-lock.json
examples/requirements.txt

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "metabrowse-server" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scalameta", artifactId = "metabrowse-server" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #970.


Checking it out since every one was doing the same.

Another place that I have to fill.

#310

Duplicate of #301

@Satyam52


quickemu: 4.8 -> 4.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/quickemu/versions.
meta.description for quickemu is: Quickly create and run optimised Windows, macOS and Linux desktop virtual machines
meta.homepage for quickemu is: https://github.com/quickemu-project/quickemu
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 4.9 with grep in /nix/store/a3m7nhbxlh6bm2zz708rp51rk150hxmr-quickemu-4.9
found 4.9 in filename of file in /nix/store/a3m7nhbxlh6bm2zz708rp51rk150hxmr-quickemu-4.9

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
quickemu

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a3m7nhbxlh6bm2zz708rp51rk150hxmr-quickemu-4.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/81774f226a2790f010f2e5ebdd9c9f2840a5683b#quickemu

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a3m7nhbxlh6bm2zz708rp51rk150hxmr-quickemu-4.9
ls -la /nix/store/a3m7nhbxlh6bm2zz708rp51rk150hxmr-quickemu-4.9/bin

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

1 package built:

quickemu

Maintainer pings
cc @FedX-sudo for testing.

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

1 package built:

quickemu

/nix/store/6vdkh7mira9sm0il5bihkc0bqbhnykd0-quickemu-4.9/bin/quickemu --version
4.8

@wegank why the draft?

@wegank why the draft?

The version seems to be incorrect. Is there an upstream patch for this?

Right. I'll handle it.

new pr #266371 . please review


Change height of slide out window?

Is there a way to adjust the height on the slide out? Would be awesome if we can just have a simple setting or a config route to do that. I like being able to just mouse over the taskbar icon and quickly flip a switch or check on the doorbell camera. I just find the height in a massive 4k monitor to be too short. Thanks!

%appdata%\homeassistant-desktop\config.json
This will let you adjust the window width and height


When exporting Markdown to HTML, all footnotes after the first are missing

I have a Markdown document that declares multiple blocks of footnotes using the [^1]: ... syntax. In the live preview side-panel I can see that the the footnotes collect correctly at the bottom of the document, as one would expect.
However; when exporting the document as HTML, all footnotes but the first disappear from the exported document. This happens with both of the HTML-related export options ("As HTML" and "Using Template").
[EDIT]
This seems to happen when the document itself ends with a fenced code block. Adding something as simple as an HTML comment block at the bottom of the Markdown document triggers the notes to appear correctly.
[EDIT]
Seems this is a problem with partial updates of the document when exporting to HTML
Forcing a full reload/reparse of the document before executing an HTML export fixes the problem.
A very similar issue with out-of-date content occurs with the [TOC] tag, which is also fixed by forcing a full reload/reparse before export.

Right. That's a regression. Will try to fix this tonight.

Same thing happens to me--had to disable MathJax. Also, the old problem with disappearing MathJax has reared its head again. 4.3.9 jumped to 4.3.11. Could the user specify which version to use?


Remove extraneous notebook container

Fixes #666.

Looks good to me, and some user testing seemed to be fine too. Thanks!


[DOC] object_detection and motion_detection intervals

Two questions:

interval is how often to run the specified detection. Does that mean that all the frames between the last run and this run will be analyzed, or only a few frames around this point in time will be analyzed? e.g. if I have motion_detection:interval: 1.0 and my camera is running at 30fps then the motion detection interval will look at the last 30 frames every second to find motion, or only the last frames: x number of frames to see if any motion occurred?

I believe that I understand clearly that with motion_detection:trigger: true then object_detection will not run unless motion is detected. Otherwise, object detection will run every interval seconds (with the same question about how many frames are analyzed). Is that the case?

The note under interval for object_detection states "For optimal performance this should the same as motion detection interval." Is this only true when motion_detection:trigger is false?

Detection runs only on one frame, so if you have a camera with 30 fps and interval: 1, every 30th frame will be scanned.

If either timeout or trigger under motion detection is true, the intervals should match up (or atleast be divisible by eachother, gonna make a note in the README for that). That is because a lot of processing has to be done on each frame before its ready to run either motion or object detection on. If these two are in sync, that processing only happens once for the frame.

Regarding 1: Ok, the interval: 1 is easy to understand.  Let's try with 30fps and interval: 10.   On the tenth frame motion detection will be run: does it compare every tenth frame to see if anything moved (e.g. essentially looking at a 3fps video in my example) or does it look at the last ten frames to see if anything moved?  I guess my question is does interval set the motion detector's video stream fps, or does it simply set how many frames are "batched up" and compared at a time? I'm pretty sure it's the first one (i.e. it essentially provides a "camera fps / interval" video stream to the motion detector).
question 2's clear now. Thank you for clearing that up.

Only one frame is scanned, they are not bunched up.
interval is a value in seconds, and its not tied to the fps in any way. interval: 10 will run ever 10 seconds which is probably not what you want.
If you have a camera with 30fps and you want to run detection on every frame(might be a bit too CPU intensive) you would set interval to 1/30=0.333. Or say you want it to run every other third frame you would set it to 1/(30/3)=0.1.
This might be a bit confusing and maybe having interval be basically FPS instead would make more sense

No, I like the interval as it is and I understand how it operates;  if the camera outputs 30fps and I want object detection 10 times per second I set interval: 0.1 to achieve that (basically taking every third frame from the stream and essentially doing a "diff" of the two images to see how much has changed. If the number of pixels that has changed is more than area * width * height of the motion_detected image, then we call that a motion event. If trigger is true, then we run the object detector (on which frames, that's something I asked in #22).


API Replace Page Delete with new Page Archive action

The major distinction between archiving and deleting is that "deleting" only deletes from stage (which is practically pretty useless), but archiving removes both live and stage, but informs the user that the page is safely recorded in the archive.
"delete" gives the false impression that the page is lost forever, which has never been the case.
This PR must be merged and reviewed alongside a CMS pull request incoming.
Lots of old code has been updated and fixed; Some which hasn't worked since 3.0 (e.g. batch action confirmations).

Ok the CMS pr is at https://github.com/silverstripe/silverstripe-cms/pull/1203

This is a semi-major change, so please ensure that this PR is left open with plenty of time for users to review and give feedback.

/me stops hovering over the merge button...

Update to this; I've re-tested and fixed some of the CSS rules in the sitetree, and fixed some issues with recent work to make it work with the updated flags / styles.

A few updates:

Restoring an archived pages now restores its parents
Some more CSS cleanup / fixes to make sure that disabled / unfiltered styles are properly inherited

So excited this is finally happening, great work guys! I've just checked it out locally. Long-term I think we need to make it more obvious on how to find archived pages in the site tree again - at the moment they stay in the tree until you reload, which might come as a surprise to users. You can only get them back through the filter panel.
On that note: You should update the userhelp ;)

I've updated the other PR (https://github.com/silverstripe/silverstripe-cms/pull/1203) based on feedback:
Instead of automatically restoring the parent page, the user is given the option to restore any orphaned pages to the root. The text / description / alert on the action will change depending on the parent page.

More bugfixes to https://github.com/silverstripe/silverstripe-cms/pull/1203:

Side by side preview was not working on archive action, so ContentController has been fixed for checking permissions of the page in the current stage
Fixed some typos in the new alerts
Changed "Un-publish" to "Unpublish"
Changed "Restore" and "Restore at top level" to "Restore draft" and "Restore draft at top level"

Rebased with new colours / hover behaviours. :D

Rebased again

This looks nice, although I think that applicablePages() might need a test, and I'd also be keen to see how this interacts with advancedworkflow, as a smoke-test of how complicated modules might be broken by this change.

There aren't any tests anywhere for batch actions... so... I'll just have to make all new ones I guess.

@sminnee I've added tests but they are on the https://github.com/silverstripe/silverstripe-cms/pull/1203 PR, since framework doesn't actually implement that API anywhere.

Rebased again

It seems that this breaks the build on postgres, although I can't understand why. :-(

https://github.com/silverstripe/silverstripe-framework/pull/4189/files#diff-69e8356e095b9501b3b000dbed87a13dR276 maybe?

Nothing to do with postgres; It's that one test that only sometimes passes. I restarted the build for postgres.

Fixed :D

Yay thanks

Related: https://github.com/silverstripe/silverstripe-framework/issues/6202


Feature Request: Copy link to Post directly to Clipboard

On the Archive listing page or the post detail page, it would be great to have a button to copy the Reddit link directly to clipboard. This would make it easier to share links with other people.

Hi @jamilnyc! Thank you for the feature request. This has been added and will be included in the next released build.
On the footer of each Post on the Archive listings page and on the footer within a detail page view, there is now a Copy button that copies the link of the Reddit URL for the content in question upon clicking.
Usage example:


[Dev][TL] Merge Hopper and Pipeline Modifications

This pull request includes a small change to the 3rdparty/tvm submodule. The commit reference for the submodule has been updated.

3rdparty/tvm: Updated submodule commit reference from f1ad5c1c57c15485d5da1362621f40749ddfa9a1 to cd230c5cb374d7b6b7c51b3f34dbd4c0e598bf65.

local test pass, merged.


Integration of Sphinx

Hey guys,
is it planned to integrate sphinx?
It would be really fine, so it can speed up massive without any big changes of queries.
regards

Sphinx, the open source search server?

hi, yes right - http://sphinxsearch.com/. this makes the implementation with mariadb, which includes by default the plugin very easy do implement.

Added an example for sphinx for building documentation inside Makefile:
https://github.com/webdevops/Dockerfile/blob/develop/Makefile#L72

As we provide Solr, you might not want to use Sphinx-Search.


(Not yet ready for merge) Add create overview csv functionality

@ErikKassubek
I basically just:

added a new command line parameter to toggle the functionality
in case of csv creation create different csv files for program/trace (if needed Times and results can be easily added too)
refactor the file creation a bit, since stat files needs header while csv doesn't
and then basically just copied the parse/write functions into more specific new functions (if necessary this can be refactored like with the file creation functions. I just found it to be more readable this way)

any thoughts?

@yellow-tshirt
At first glance it looks good, but I will look at it in detail and try it out tomorrow.
One thing I have noticed, at the moment it seems you can either create the csv file or create the overview file, but not both at the same time.
Maybe you can change the flags such that this is also possible, e.g. -c for the csv, -o for the overview and either -c -o or no flags for both

Looks good


New in 3.1.0: ERROR [worker reload] [worker init] Cannot find package 'sitemap' imported from myprojectpath.nuxt\dev\index.mjs`

Environment
- Operating System: `Windows_NT`
- Node Version:     `v16.15.0`
- Nuxt Version:     `3.1.1`
- Nitro Version:    `2.1.0`
- Package Manager<EMAIL_ADDRESS>- Builder:          `vite`
- User Config:      `css`, `hooks`, `modules`
- Runtime Modules:  `<EMAIL_ADDRESS>`<EMAIL_ADDRESS>`<EMAIL_ADDRESS>- Build Modules:    `-`

Reproduction
npm run dev with this nuxt.config.ts:
export default defineNuxtConfig({
        modules:
            [
                [
                    "@funken-studio/sitemap-nuxt-3",
                    {
                        hostname: "https://example.com"
                    }
                ],
                "@nuxtjs/tailwindcss",
                [
                    "@storyblok/nuxt",
                    {
                        accessToken: process.env.API_TOKEN,
                    }
                ]
            ]
    }
);

Describe the bug
Nuxi 3.1.1
Nuxt 3.1.1 with Nitro 2.1.0

  > Local:    http://localhost:3000/
  > Network:  http://...

i Using Tailwind CSS from ~/assets/css/tailwind.css
i Tailwind Viewer: http://localhost:3000/_tailwind/
i Vite client warmed up in 6419ms
√ Nitro built in 1329 ms

 ERROR  [worker reload] [worker init] Cannot find package 'sitemap' imported from myprojectpath\.nuxt\dev\index.mjs

  at new NodeError (node:internal/errors:372:5)
  at packageResolve (node:internal/modules/esm/resolve:954:9)
  at moduleResolve (node:internal/modules/esm/resolve:1003:20)
  at defaultResolve (node:internal/modules/esm/resolve:1218:11)
  at ESMLoader.resolve (node:internal/modules/esm/loader:580:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:294:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)
  at link (node:internal/modules/esm/module_job:78:36)

Additional context
Works fine reproducibly with nuxt 3.0.0
Error occurs with nuxt 3.1.0 and 3.1.1
Logs
No response

This is not reproducible for me: https://stackblitz.com/edit/github-xyg9vo. I would confirm you actually have the sitemap package installed.

I tried to make a reproducible stackblitz but did not succeed.
Some more investigation:
nuxt 3.0.0 uses nitro 1.0.0, nuxt 3.1.0 uses nitro 2.0.0.
The error mentions index.mjs.
This file indeed looks very different depending on which version of nuxt is used:
nuxt 3.0.0
import { createRoutesCache, validHttpCache } from<EMAIL_ADDRESS>import { createSitemap } from<EMAIL_ADDRESS>import { excludeRoutes } from<EMAIL_ADDRESS>
nuxt 3.1.0
import * as sm from 'sitemap';

This is not reproducible for me: https://stackblitz.com/edit/github-xyg9vo. I would confirm you actually have the sitemap package installed.

sitemap was not installed and it worked without in nuxt 3.0.0.
Now it seems necessary to use it.
I added "sitemap": "7.1.1" to the dependencies and now it works.
Is this now a bug or regression?

As I said, I can't reproduce it, so I'm assume it's an issue with your package manager, lockfile or node_modules folder rather than a regression in Nuxt.

Same with "nuxt-proxy": "0.3.12"
 [worker reload] [worker init] Cannot find package 'http-proxy-middleware' imported from ...

cc: @pi0 in case this is a regression of the node_modules hoisting


Deepspeech

Project description
DeepSpeech is an open source Speech-To-Text engine, using a model trained by machine learning techniques based on Baidu's Deep Speech research paper. Project DeepSpeech uses Google's TensorFlow to make the implementation easier.
Metadata

homepage URL: https://github.com/mozilla/DeepSpeech
source URL: https://github.com/mozilla/DeepSpeech
license: Mozilla Public License https://github.com/mozilla/DeepSpeech/blob/master/LICENSE
platforms: Unsure

I already tried this with a local clone of deepspeech:
let
  pkgs = import <nixpkgs> {};

  deepspeechPythonPackages = python-packages: with python-packages; [
  ];
  pythonPackages = pkgs.python3.withPackages deepspeechPythonPackages;

  deepspeech = pkgs.python3Packages.buildPythonApplication {
    version = "0.8.2";
    pname = "deepspeech";
    src = ./DeepSpeech;
    propagatedBuildInputs = with pkgs; [ sox ];
    buildInputs = with pkgs; [ sox ];
    checkInputs = with pkgs; [ sox ];
  };
in deepspeech

But I get this error for unknown reasons:
Finished executing setuptoolsBuildPhase
installing
Executing pipInstallPhase
/build/DeepSpeech/dist /build/DeepSpeech
Processing ./deepspeech_training-0.9.0a10-py3-none-any.whl
ERROR: Could not find a version that satisfies the requirement sox (from deepspeech-training==0.9.0a10) (from versions: none)
ERROR: No matching distribution found for sox (from deepspeech-training==0.9.0a10)

I guess pip is searching for a python package named sox.
https://pypi.org/project/sox/1.4.0/

@davidak That one doesn't seem to be in nixpkgs, is that right?

@davidak That one doesn't seem to be in nixpkgs, is that right?

Doesn't look like it, you might want to package that as well

looks like this would need packaged as well http://sox.sourceforge.net/

There is sox in nixpkgs.

:(

@NorfairKing I added binary package in https://github.com/NixOS/nixpkgs/pull/117416, which works fine for my use case (linking to it from Rust). The binary itself works fine as well


can’t make $request->getBody(); to work.

First, thank you for this wonderful framework. You save me a lot of time with this! 😊
But I have small problem, I can’t make $request->getBody(); to work. I need to get some JSON data but each time, I get this error :
Fatal error: Call to a member function getBody() on null
I have to manually get the data each time like this:
$data = file_get_contents("php://input");
Could you help me with this?
Thank you very much!
Kind regards,
George.

Possible bug, need to be verified first


MGH dataset - How to deal with combined Cord + CSF masks  & creating json files from nifti hdr

Hello everyone,
MGH site kindly shared their 7T fMRI datasets with us.
Two points:

As they did NOT use dcm2niix, we do not have accompanying .json files. See what @jcohenadad suggested on this issue below.

`maybe you want to check existing functions to get a 'proper' JSON from nifti's header-- i'm sure it exists somewhere-- @rohanbanerjee

`

Their manually drawn masks combine CSF and cord.

Again, see Julien's suggestion:

yes, but i suggest we also keep CSF masks because training a 2-class nnUnet to segment the cord and CSF would be very useful. Several people need the CSF mask for fMRI studies. I suggest you or Rohan open an issue to discuss best strategies for these masks (file name, should the CSF mask include SC or should both masks be mutually exclusive, etc)

@rohanbanerjee

Do you have any suggestions of a function that create a proper json from nifti header?
What do you think about the 2. point?

Thank you!!
Merve

@MerveKaptan you described two distinct issues which could (should) be discussed and solved asynchronously. Could you please split this issue into two separate issues. Thx

Sorry!!! Of coruse!


Enhance CPE generation

Closes #396
Closes #471

rebased / force pushed to incorporate https://github.com/anchore/syft/pull/474


Why do not use LinkedList for GetStream LightSubscription?

Hi Sir, I am big fan of GetX.
Just wondering why do not use LinkedList in GetStream Class for LightSubscription? cos i saw the result from benchmark Unit test  and it shows that the MiniStream which implemented with LinkedList got higher performance than GetStream that using List.

Hi @yunze-ola
Previously, it was a hypothesis, but the next version of GetX will use notifiers (check the master branch), as fast as GetStream/MiniStream/ValueNotifier


add additional filter huntstep to fix #6

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Xiaokui Shu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


FStar.Buffer does not have a wrapper for C's memcmp

There are blit and fill for memcpy and memset, but no wrapper for memcmp.
One design choice is what type to give it. On the one hand memcmp should work for any t if it's only being used for equality comparisons, whereas for a buffer U8.t it can also promise to return positive/negative numbers appropriately. My preference would be to start with just equality comparisons for any type:
// corresponds to memcmp, but without the sign comparison
assume val compare: #t:Type -> b1:buffer t -> b2:buffer t ->
                    len:UInt32.t{v len <= length b1 /\ v len <= length b2} -> Stack Int32.t
 (requires (fun h0 -> live h0 b1 /\ live h0 b2))
 (ensures (fun h0 r h1 -> live h1 b1 /\
                       live h1 b2 /\
                       modifies_none h0 h1 /\
                       (Int32.v r == 0 <==>
                        Seq.slice (as_seq h1 b1) 0 (v len) ==
                        Seq.slice (as_seq h1 b2) 0 (v len))))

which extracts to something like memcmp(b1, b2, len*sizeof(t)).

https://github.com/FStarLang/kremlin/blob/master/kremlib/kremlib.h#L41 ?

I think you could add it and use the same trick in kremlib.h as above for the implementation of it, provided that the signature of Buffer.eqb is not what you want. Or, perhaps fix Buffer.eqb to give you a strong post-condition?

Seems like eqb is going to foot the bill... closing.


Make OK button the submit button in all forms

... so hitting enter will submit the form 💯

Could you please provide a (check)list with all forms which need to be adjusted?

I guess the best way is a project-wide look for btn-success or btn-primary, depending on what you used for buttons that shall indicate submit.

Should work, yes. Could anyone please provide a list with all forms/files? @eScienceCenter/spacialists

CAN SOMEONE PLEASE PROVIDE ME A LIST WITH ALL FORMS, AS REQUESTED SEVERAL TIMES NOW....?!

[ ] main form
[ ] additional information form (confidence & reference)
[ ] add new literature
[ ] add new user
[ ] modify user
[ ] add new role
[ ] modify role
[ ] create new attribute
[ ] create new context-type
[ ] modify context-type
[ ] add attribute to context
[ ] layer editor

additional information form (confidence & reference)

Textareas do not support submitting a form with enter or shift+enter. It is possible to add a listener in js and check if the current key is enter and if shift is pressed.
IMO this is unnecessary for this form.

layer editor

The properties form on the right side already has a submit button. Do you refer to another form?


Update realistic-app-benchmark.md

This is the contents I added for this section.
I am suggesting to change the name to 'customer-like application'.
I am not confident yet that the included pictures will render correctly, so we will see.

if you look at https://deploy-preview-1568--preliminary-istio.netlify.com/docs/performance-and-scalability/realistic-app-benchmark/ the images are squished, probably because you didn't set the aspect ratio (#1460)

Deploy preview for preliminary-istio failed.
Built with commit 13c5b989240c4477b83c922fc82d01004fd68af4
https://app.netlify.com/sites/preliminary-istio/deploys/5b2d7f0bc96592102cecd188

you need to add entries to the dictionary file I mentionned for the build to pass

@ldemailly I have updated the dictionary.

you forgot git push ? (or pull first as I fixed a typo in your fork)

Your commit is already of my patch-1 branch which is in the PR. When I try push from my local git it says it up-to-date. So, I am confused about what is missing. I think I started in wrong way this. Maybe it is better to start over. The problem I am leaving pretty soon. Can this wait until I come back?  If not, could you pls get the content and create a new PR yourself?

your branch (pr) only has 1 file changed, you need to change the dictionary file I mentioned above

Hi, I did some refactoring and the file you edited has moved in the repo. Please rebase and push again. Sorry for the inconvenience.

addresses (partially) #1377

@jnativio: PR needs rebase.

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

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

Needs approval from an approver in each of these files:

OWNERS

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

/assign @geeknoid

Hi there,
To get this PR through, you will need to sync your tree, resolve any conflicts, and then update the PR. Once you update the PR, it'll run the current set of tests and hopefully they'll pass and then this PR can go in.


Un-hardcode Tools in ReActHelper

Currently, the list and descriptions of tools in ReActHelper are hard-coded. They really ought to be drawn dynamically somehow, so that when the new tool is added to the graph, I don't have to change the code of the helper.

https://github.com/google/labs-prototypes/commit/0730102abf6c00fd17f428d86177cebfaf6572e4 introduced JSONata support, which reduces the list/describe functions of ReActHelper to just providing an object.

[ ] Introduce reflection node


Fixed function documents for GIRFApplier and GIRFEssential

Fixed function documents for GIRFApplier and GIRFEssential

apply_girf still needs documentation. I'll add tomorrow morning


Don't crash when bit for 8 is set in prefix length

Using & 8 on the prefix_len will produce 8 if it has thwe bit for 8 set and 0 otherwise. So las will sometimes be 8 and sometimes be 0, and later whjen we shift a byte by las, if it is 8, it fails:
thread 'tokio-runtime-worker' panicked at 'attempt to shift right with overflow', src/proxy.rs:121:30
stack backtrace:
   0: rust_begin_unwind
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/panicking.rs:142:14
   2: core::panicking::panic
             at /rustc/a55dd71d5fb0ec5a6a3a9e8c27b2127ba491ce52/library/core/src/panicking.rs:48:5
   3: http_proxy_ipv6_pool::proxy::get_rand_ipv6
             at ./src/proxy.rs:121:30
   4: http_proxy_ipv6_pool::proxy::Proxy::process_request::{{closure}}
             at ./src/proxy.rs:64:25
...

This PR changes the behavior so that las can be 0 through 7, but never 8. I'm not certain it ends up having the right value to make the algorithm correct, but it is at least always in range for the shift.

Sorry for that, it's a typo.

Thanks

I'm very sorry that my use case didn't cover this, I didn't realize that the problem had been lurking for so long.
I have just implemented a simpler and easier to understand version with better bench performance.


Multiline TextInput + Initials avatar support

To enable multiline textinputs, set multiline to true as a property to GiftedMessenger. set maxInputHeight to specify the maximum height the input can grow too. If you need the change event for when it changes height, use onTextInputHeightChanged. You can pass additional props to the input now with textInputProps.
IMPLEMENTATION NOTES:
The autogrowing part was easy. What was very time-consuming was getting the positioning compensation for the ListView so the correct part was always placed properly (similar to the iPhone text message app). I had to animate the textinput separately (well, its container). So similar to the ListView container, I animate its height. They are both done in parallel, but could likely be optimized to interpolate. The reason it had to be done separately--and believe me, I tried to use the initial mechanism to move the input up--is because when the placement of the textinput is dependent on the height of the ListView, then you can't move the ListView (in relation to the TextInput) without also moving the TextInput. I spent a long time trying to do that via various ways and things just got tangled, so I did what I probably should have originally did which was isolate the TextInput so it moves in an unrelated fashion. Absolute positioning and a bottom property are used to have the textinput stick to the bottom and grow up from the bottom. Aside from the listview placement compensation, the bottom-based absolute positioning is its secret sauce.

good job!

@FaridSafi Any indication when this can be merged/will be reviewed?

@faceyspacey, This could be cleaned up a little bit, but I would like to see this merged as well. Thanks!

@faceyspacey while you're at it, i'd also suggest to submit multiple pulls with the features split up. Let me know if you're going to do a refactoring. If not I'll take care of some of the features pulled for here.

Hey guys, I've reached the stage of my project where we're about to launch and I only have time for urgent stuff. I can't forecast when I'll have time for this.
....as far as the AutoGrow repo required, its creator basically made the requested changes. So we should be able to just use his repo now. There may be one modification we want to make to our code though.


Block if sectigo is the provider for autossl domains

Case RE-54:
When CentOS 7 is elevated to AlmaLinux 8, the "nobody" user will retain its uid/gid of 99 where as on AlmaLinux 8 the expected uid/gid for "nobody" is 65534. This will cause issues with the Sectigo AutoSSL provider.
Changelog: Block if Sectigo is the AutoSSL provider.
By submitting pull requests to this repo, I agree to the Contributor License Agreement which can be found at: https://github.com/cpanel/elevate/blob/main/docs/cPanel-CLA.pdf

I'm declining this for now.


Dependency on ojdbc

The only package that should depend on ojdbc is eurekaclinical-package. The i2b2-export-package package also does, but we should just deprecate it because we don't use it any longer.

This is removed in version 3.


Support reload over SSE instead of websockets

There are a number of Rails projects that don't use ActionCable, but would benefit from this gem. I'm interested in supporting live reload via ActionController::Live and server-side events to establish a connection over http for live reload.
I'd be happy to take a crack at this if there's interest/alignment.

I'm open to it. It would be great to have more universal alternative to ActionCable!


Implement nozzle geometry simulation

Want to move the code from this repository: https://github.com/aesirkth/nozzle_design


Fixing spacing issues and add missing article

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

Looks good; thanks.

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


Method 'hash()' overrides Objective-C method 'hash' from superclass 'NSOjbect'

Hello, just upgraded to latest XCode (6.3 beta 2) and it appears that they brought back the hash method to NSObject. As a result the app won't compile because the JSQMessageData requires the implementation of this method that has the same name.
Any workarounds?

hey @attheodo - hash never went away. You probably just need an override. I'm assuming you're using Swift?

I am not really sure what happened, it worked properly before today that I updated to latest XCode. This is what used to work fine:
func hash() -> UInt {         return UInt(self._uuid.hash) ^ Int(self._timetoken)     }
Adding 'override' creates a complaint that Method does not override any method from its superclass which kind of makes sense because in the docs hash is a computed property (https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Protocols/NSObject_Protocol/index.html#//apple_ref/occ/intfm/NSObject/hash)
Any suggestions?

@jessesquires yes I am on Swift 1.2... any workarounds this issue?

Looks like a Swift bug to me.. :(

Actually it's not a Swift bug, the made some new changes in the way Swift allows overloading/overriding. Please check http://stackoverflow.com/questions/28718577/issue-with-conforming-to-objective-c-protocol-from-swift-nsobject-subclass/28718947?noredirect=1#comment45724845_28718947
I don't see any way this could be worked around, other than changing hash method name in the JSQMessageData protocol to something else.
What do you suggest? This totally breaks our build and I am suspecting that because I commented out the hash requirement from the protocol I am also getting the errors mentioned in #819

Hey @attheodo - yes, the hash method is a known issue. #684

Do you have an ETA for a fix?

@jessesquires I can see this is scheduled for the 7.0.0 milestone. Can you give me quick directions on where the hash method needs to be renamed apart from the protocol?

@attheodo - I'm probably going to push out 7.0 sooner rather than later to get this fixed.
(and the 6.x milestones will become 7.x)


Disable the initial telemetry notification

VSCode Version: Any recent version (>= 1.37.0)
OS Version: openSUSE Tumbleweed

Steps to Reproduce:

Launch a fresh instance of VSCode
wait a few seconds
a popup with "Help improve VSCode ..." appears in the notification area, even when telemetry is disabled

Does this issue occur when all extensions are disabled?: Yes
I'd like to disable this notification for running automated test on an extension that I am developing, as the telemetry notification is interfering with those displayed from my extension. I assume that this notification is displayed only once, since I haven't seen it in my development instance of vscode. What is the appropriate setting/flag for that?

Any updates on this? This notification is really annoying as it results in my automated tests being extremely flaky.

This should no longer happen. Thus closing


https://plugins.traefik.io rewrites GitHub badges markdown to invalid URL

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I put a GitHub badge in my README.md in my plugin repo:
[![Go](https://github.com/Brainnwave/jwt-middleware/actions/workflows/go.yml/badge.svg)](https://github.com/Brainnwave/jwt-middleware/actions/workflows/go.yml)

What did you see instead?
https://plugins.traefik.io is serving it with the URL rewritten to use the raw.githubusercontent.com domain, which results in a 404 for the image and a broken link on the page:

What version of Traefik are you using?
n/a
What is your environment & configuration?
n/a
If applicable, please paste the log output in DEBUG level
No response

hello,
we were forced to do that to display images from the repository itself (ex: from the doc folder).


[feature] Passing an additional parameter not to be used in the URL

I'm trying to call a dynamic route and I have this working really well but what i really need is to be able to pass an extra parameter to the template so that I can use that to specify a page type
for example:
v-link="{ path:  "pagename" , params : (pageType : "page"}
is this possible?
I've done a work around using named routes but this is not good for me as I don't want the name of the route appearing in the URL I only want the page name.

Alias can be a solution for your example.
Here is a comment on another issue.
https://github.com/vuejs/vue-router/issues/235#issuecomment-201734131

Technically any parameter must be passed inside url.
For example, if clicking a link goes to /pagename whose 'type' is 'foo'.
What happens if the user refreshes the page after clicking? Now he hasn't clicked any link, there's no 'type' specified for the page.
If you do want to pass such 'invisible' parameter, use props.
<router-view :type="yourType">
And cache the type value using localStorage afterwards.

What @fnlctrl said: what you are trying to "pass along" is not a URL param, but rather application state. A URL param is by definition part of the URL. You are actually trying to do two things: 1. navigate to the route 2. mutate the application state (pageType).

Thank you guys.. I really couldn't see a way around the problem with out
passing a parameter. For most web apps using the named routes or parameters
in the url would be sufficient but I wanted clean simple URLs.
I will give the props a go and see how I fair with it and take this across
to support if I run into issues.
Thanks for the replies.
On Sat, 2 Apr 2016 at 07:36, Evan You<EMAIL_ADDRESS>wrote:

Closed #444 https://github.com/vuejs/vue-router/issues/444.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/vuejs/vue-router/issues/444#event-612078703


weaviate draft

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

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

**NOTE:

i got multiple flake8 warnings and errors that previously would pass the CI workflow but i didn't know if should run autopep8 on the extension to fix it. please advise if i should run it to fix the linting errors on all azext_containerapp files. (running azdev style containerapp returns no error but running flake8 . anywhere does.)
the failing integration tests are unrelated to my change.**

Hi @zhoxing-ms @wangzelin007
Could you help to review this PR?
The azdev style check got lots of flake8 warnings, I will handle them in other PR. Thanks.

@zhoxing-ms is the new change ok?

@Greedygre @zhoxing-ms, i have a milvus add-on PR that will be out tomorrow. i will make the switch of expiry to 2.57.0, then. thank you all!


Moving gl-matrix into smaller npm modules

Hey @toji – awesome work with gl-matrix, I've been using it in all of my graphics projects since I got started with WebGL.
My only gripe, coming from the browserify community, is that it's all kept in a single package: it'd be great to be able to pull in the one or two bits of it piecewise for when we're trying to trim down a script.
Would you be open to @mikolalysenko and myself forking the project to take this approach? More than happy to credit you as the author and link back to this repository from each package. Thanks!

errr... what's wrong with
import { vec3 } from 'gl-matrix';
And let your webpack/rollup do the tree shaking?

@andrevenancio iirc this was before tree-shaking was a thing :)


Replace OpenSSL With rbsecp25k1

Overview
This change replaces the current OpenSSL secp256k1 ECDSA with rbsecp256k1, a native extension gem binding to libsecp256k1. It replaces all key generation, signing, and key recovery methods with their rbsecp256k1 equivalents.
Directly using libsecp256k1 will provide performance and security benefits over the current hand-rolled OpenSSL version.

[x] Replacing OpenSSL

[X] Replace OpenSSL usage with rbsecp256k1.
[x] Remove OpenSSL dependency and Secp256k1 module.

[x] Clean up and refactoring

[x] Factor out recoverable signature functionality into a single interface.

@etscrivner unfortunately, your solution still doesn't work for me. When I'm trying to run rails console I get this error and nothing else:
Process finished with exit code 134 (interrupted by signal 6: SIGABRT)

Any ideas?

@se3000 do you need someone to take over this PR? I'm getting an issue with OpenSSL on Mac (it pretends to be OpenSSL but it's really LibreSSL) and this PR could make my life a lot easier.

would love to have this working!

I implemented this in https://github.com/q9f/eth.rb/pull/4 and https://github.com/q9f/eth.rb/pull/24 and plan to release it as 0.5.0 soon. Thanks for your pioneering work @etscrivner 🎉


A way to use standard font as default font?

Issue description:
I'm trying to set "Sans Serif" standard font as a default font for the Typography field, but can't make it work. The "Font family" field is empty. It populates after selecting and saving/publishing the field. Works smoothly for Google fonts though. Tried to use put both label and stack as a "font-family" value, but with no luck. Any ideas on how to solve it?
Version used:
<IP_ADDRESS> && develop
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
Kirki::add_field( 'theme_config', array(
	'type'        => 'typography',
	'settings'    => 'body_font',
	'label'       => esc_attr__( 'Control Label', 'textdomain' ),
	'section'     => 'body_typography_1',
	'default'     => array(
		'font-family'    => 'Sans Serif',
		'font-size'      => '18px',
		'color'          => '#333333',
	),
	'priority'    => 10,
	'output'      => array(
		array(
			'element' => 'body',
		),
	),
) );

The default needs to be the full value of 'Sans Serif', which is:
		'font-family'    => '-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Oxygen-Sans, Ubuntu, Cantarell, "Helvetica Neue", sans-serif',

Or sans-serif, that also works :wink:


FLAC Support

According to a TestFlight report, FLAC is not currently working. Let's check out what's going wrong there.

Can confirm as well, in the test flight, no flac support present.

This is fixed with the new build (4) that should be out on TestFlight any moment now.


Select component md-menu-class

First off thank you for this amazing project.
Now I've been trying everything and its been great, but I can't seem to to make the the "md-menu-class" property of Select component to work. What I was trying to achieve is to use md-align-trigger and md-size on the select options. An example implementation will be much appreciated.
Thank you in advance and again amazing job on this project.

Hi @filpineda
the md-menu-class only apply a different class to the select container. What do you want to do with your select?

Hi @marcosmoura
I was trying to align the top of the select options container to the bottom of the select container by using md-menu-class="md-align-trigger" and to adjust the width of the select options container by using using md-size prop. I thought it would work since the md-select component uses md-menu as container of the select options iirc...
Is that possible to achieve using md-menu-class?


bump up postgres operator

What can we help you?

trying to use postgres-operator:v1.8.0, and spilo-14:2.1-p5
but failed to start pod postgresql-cluster-sample-ns-harborcluster-sample-0

it looks like the spec is changed

the secret name is changed from something like harbor.postgresql-cluster-sample-ns-harborcluster-sample.credentials to  harbor.postgresql-cluster-sample-ns-harborcluster-sample.credentials.postgresql.acid.zalan.do


[APP-6685] - group pending messagse with successful messages

Description
chainTop is a property associated to messages that determines whether a message header is rendered. It is based on whether adjacent messages are part of the same group. The first of such a group is given a value of false, the rest a value of true.
Messages are grouped if according to sender id, timestamp, sent status, and read status.
We want to group pending messages with successfully sent messages as long as they are:

from the same sender
sent within the same minute

Test Plan

tbh I'm kinda confused why they split according to read status
https://github.com/gathertown/sendbird-uikit-react/blob/fc0dd8080bfdba591ed5fd684f061de9583a1d62/src/utils/messages.ts#L79
I guess if you are above the fold, they split the group in two?


Use only a single TransactionStatus enum

We currently have TransactionStatus and TransactionStatusType which are almost the same. The only difference is that the former also wraps a block hash with two of its variants. TransactionStatus is used only in a single place, as the response type for get_transaction_status.
In my opinion, this is quite confusing. Both types do pretty much the same job. I'm suggesting to introduce a different type to replace the current TransactionStatus, something like:
pub struct ShortTransactionStatus { // name is of course open for discussion
    pub block_hash: Option<H256>,
    pub tx_status: TransactionStatusType
}

It would be used as the return type for get_transaction_status. That way, we could remove TransactionStatus and also TransactionBlockHash, because that would just be "merged" into this new struct and have a cleaner API.

As I was writing the tests and digging through the types deeper, I came across TransactionWithStatus which could be refactored in sync with the proposed changes. An example (I don't have it fully fleshed out yet):
pub enum Transaction {
    pub Short(ShortTransaction)
    pub Full(FullTransaction)
}

pub struct ShortTransaction {
    pub block_hash: Option<H256>,
    pub tx_status: TransactionStatusType
}

pub struct FullTransaction { // just a new name for the current TransactionWithStatus
    // ...
}

We could probably use the Transaction for both /get_transaction and /get_transaction_status calls.
Does it make sense to you @xJonathanLEI?

Hmm you're right that the types around transactions are a bit messy, thanks to the inconsistency of the API schema...
I'm not making a stance here, but for the sake of explaining the original design decision, I was wrapping the block hash inside an enum variant so that users can directly extract the block hash idiomatically inside a match arm, for example, without calling unwrap().

If we decide to diverge from gateway response shape, we'll need to move the deserialization tests from core to providers, since they'll no longer be the same representation.

Yeah, I understand the original design decision, the API isn't the cleanest to work with.
By giving up the idiomatic handling with match we gain much more understandable types. I think it's a worthwhile tradeoff to make. Users of the library will be accustomed to calling unwrap since it's a common occurrence in Rust, but will have to learn the types it exposes and wonder what's the difference between TransactionStatus and TransactionStatusType.

My point on unwrap() was more on idiomaticness, since unwrap() potentially panics. It's more of a peace of mind thing. But I agree that this is a tradeoff.

Resolved in #60.


BUD: Fix time out

A timeout that is too short can cause unnecessary errors due to server or network problems.


Make it possible to enable/disable chains without rebooting app UI #6035

Closes #6035

should be good for testing

Sometimes the "x" button to close and save doesn't work. I haven't figured out why and if it's just with this PR:

Would you help create an issue if you can't reproduce right away?

Sometimes the "x" button to close and save doesn't work. I haven't figured out why and if it's just with this PR:

yep catched it on starting of development of this feature, but then its gone, thought its fixed, can't catch it for now

updated pr with this flow:
5. Tap Enable button 6. Automatically enable Goerli and Optimistic Goerli (and not needing to save manually)

@oa-s looks great! Can you help modify the "Enable" button to "Enable and Connect" (or "Enable and Accept" or whatever is appropriate and fits), since it also accepts the new session?


How to disable camera

Hi all, I want to keep photo gallery and remove live camera in Chatto. Anyone help me?
Thanks...

@huynhphuc185 I'm afraid you can't do that out of the box. You need either:

Add ability to hide live camera to PhotosChatInputItem
Create you own input item that conforms to ChatInputItemProtocol.


feat: list uncompleted tasks

PR for Issue #15
UI for empty todo tasks

UI for listed todo tasks
Updated HTML. Added separate columns for task name and options for future updates

Will re-create into a different branch rather than the main branch


Upgrade NearApiJS to v4

There was only one snag in the way we convert wallet-selector.Action into nearApiJs.Action where we were previously relying on the wallet-selector to do the conversion (it is outdated) but nearApiJs has a supportive constructor.

Closed in favour of #532


Save extensions from field options in message props

This is first step to implement plugins. Protobuf allows to declare and use custom options for fields too (currently only file options is supported), but this library omits that info, so I think neat place to put that info is __message_props__/0:
message Foo {
  optional string a = 1;
  optional string b = 2 [(mypkg.myopt_bool)=true];
  optional string c = 3 [(mypkg.myopt_string)="test"];
}

iex(1)> Protobuf.Protoc.ExtTest.Foo.__message_props__()
%Protobuf.MessageProps{
  ...
  field_props: %{
    1 => %Protobuf.FieldProps{
      default: nil,
      deprecated?: false,
      embedded?: false,
      encoded_fnum: "\n",
      enum?: false,
      extensions: %{},
      fnum: 1,
      map?: false,
      name: "a",
      name_atom: :a,
      oneof: nil,
      optional?: true,
      packed?: false,
      repeated?: false,
      required?: false,
      type: :string,
      wire_type: 2
    },
    2 => %Protobuf.FieldProps{
      default: nil,
      deprecated?: false,
      embedded?: false,
      encoded_fnum: <<18>>,
      enum?: false,
      extensions: %{{Mypkg.PbExtension, :myopt_bool} => true},
      fnum: 2,
      map?: false,
      name: "b",
      name_atom: :b,
      oneof: nil,
      optional?: true,
      packed?: false,
      repeated?: false,
      required?: false,
      type: :string,
      wire_type: 2
    },
    3 => %Protobuf.FieldProps{
      ...
      extensions: %{{Mypkg.PbExtension, :myopt_string} => "test"},
      ...
    }
  },
  ...
}

I thought a lot about how Plugins interface should looks like to support wide area of features, but I get stuck :(
In my project I have a few different types, that requires pre- and post-processing, like UUID that should be transformed from string representation to binary, or enum constants that should be converted from human readable to protobuf-acceptable (accordingly to protobuf style guide).
Keep that meta information in options looks like a solution (but it looks like workaround without plugins system), with that meta I can write Elixir's Protocol that process protobuf structs and automatically generate encoders/decoders for each struct/field.
The most important thing — this is not a duct tape for this library, my PR just add able to hold custom options :)

I'm also added handy mix task to compile proto schema: mix protobuf, no longer need to install escript globally in system, it's per-project now. Create protoc-gen-elixir bash script with contents:
#!/bin/sh
mix protobuf

and run protoc like: protoc --elixir_out=./lib --plugin=./protoc-gen-elixir *.proto.

Added config :protobuf, extension_paths: [path1, glob2] to support custom extensions placed in project (not in this library), protobuf-elixir will look in this paths and compile everything before generating protobuf schema, eg:
config :protobuf,
  extensions: :enabled,
  extension_paths: ["lib/my_app/myext.pb.ex"]

I feel the correct direction for plugins should be writing a separated protoc plugin and generate separated files. For example, these plugins https://github.com/envoyproxy/protoc-gen-validate & https://github.com/grpc-ecosystem/grpc-gateway work like this. And we can add some functions
to simplify writing a protoc plugin in Elixir, for example this project https://github.com/lyft/protoc-gen-star.


Support jnifuse on MacOS

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9955/
Test FAILed.


Link to Github-Pages gem on the pages tag so people can see their dependencies.

@parkr is there a page I can link to that lists them in a nice format?

/cc @parkr

/cc @benbalter

Yep: https://pages.github.com/versions/
Also https://pages.github.com/versions.json for the machines in us.


Support non-text files too

Is there a way to enable the same view for things like images without having to right click -> properties on them?

Thought this was impossible, but found an undocumented API that contains the URI for non-text files.

Should be fixed in v.1.2.0 (ab1417409bd747593477c375ef19d9cfbb9e90eb).


Edit activity diagram for add feature to include start node

Edit diagram which did not include starting node.

Codecov Report

Merging #103 (a12cf33) into master (5e080df) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #103   +/-   ##
=========================================
  Coverage     70.66%   70.66%
  Complexity      504      504
=========================================
  Files            81       81
  Lines          1718     1718
  Branches        182      182
=========================================
  Hits           1214     1214
  Misses          454      454
  Partials         50       50

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


A minor change in CONTRIBUTING.md

When a user forks a repository then the nomenclature of the name of the forked repo basically looks like this:
'username/repo-name'
That's why instead of using:
git clone https://github.com/ChiragAgg5k/cal-buddy
We can use this:
git clone https://github.com/'your username'/cal-buddy
It would be much easier for the users when they fork this repository, this would clear the confusion for them if they encounter any while forking.
That said, i can be wrong just wanted to make it clear for the users. At the end its up to you @ChiragAgg5k

Thanks for pointing this out. Good work 👍


M5 Competition Communication

hello, friend, in M5, i saw your sharing notebook. well. thumbs-up.
here https://www.kaggle.com/yassinealouini/seq2seq   in comment:
When it works (which isn't the case for now ;)), it is around 1.97.
This isn't a great score for sure and I am investigating what can be improved.
I am working on replacing OHE with embeddings.
Stay tuned for updates (hopefully).

I'd like to know if you followed up with this method of optimization and what the results were.
my email is<EMAIL_ADDRESS>expecting your reply ^_^

Hello @wanpingDou. Unfortunately, I was unable to add the embedding layer and I won't have time to do it :(
If things change, I will let you know.


GIF Image Support in UIKit

@junhaiyang  hm... is there anything AFNetworking can do about it? can you elaborate a bit?

Let's track this feature request in #3243


f# search

VSCode Version: Code 1.5.2 (66f37fd2a99eb9d628dd374d81d78835b410c39b, 2016-09-12T13:30:03.456Z)
OS Version: Windows_NT ia32 10.0.14926

Steps to Reproduce:

OPENED BY ACCIDENT. please excuse my mess.


Tailwindcss classes

really love the library but not having tailwind support is tough, most projects nowadays use tailwind as their css I think it would be very nice for the extension to support that
if you ever plan on adding it here is a blog from tailwind with most of the checks needed
https://tailwindcss.com/blog/tailwindcss-v3-3

Tailwind is outside of the scope of what I am able to work on right now and, I believe, would require re-thinking quite a bit of the current implementation.
If you feel inclined feel free to create a PR.


Data

Hi, could you please clarify your question regarding the data? It would help us provide a more accurate response.


Fix Ci Slowdown

Checklist before submitting

[x] Did you read the contributor guide?

Description
CI had slowed down because validation was looping through the env files, but still validating all of them each time.
Also combined all variants to be able to parallelize them. Removes the ability to report a failure for a specific variant, but is a little faster.
Here is a run using these changes:
https://github.com/open-ce/open-ce/pull/436/checks?check_run_id=2823628863

Codecov Report

Merging #65 (2f541b7) into main (4fda5f7) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
- Coverage   96.70%   96.69%   -0.01%
==========================================
  Files          23       23
  Lines        1971     1965       -6
==========================================
- Hits         1906     1900       -6
  Misses         65       65

Flag
Coverage Δ

unittests
96.69% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open_ce/errors.py
100.00% <100.00%> (ø)

open_ce/validate_config.py
100.00% <100.00%> (ø)

open_ce/conda_utils.py
98.75% <0.00%> (-0.02%)
:arrow_down:

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 4fda5f7...2f541b7. Read the comment docs.

Codecov Report

Merging #65 (2f541b7) into main (4fda5f7) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
- Coverage   96.70%   96.69%   -0.01%
==========================================
  Files          23       23
  Lines        1971     1966       -5
==========================================
- Hits         1906     1901       -5
  Misses         65       65

Flag
Coverage Δ

unittests
96.69% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open_ce/errors.py
100.00% <100.00%> (ø)

open_ce/validate_config.py
100.00% <100.00%> (ø)

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 4fda5f7...2f541b7. Read the comment docs.


cache

How exactly does the "caching" system work? What is the use of it? How would I use it within my code?

For eample I have this code:
try {
$data = $riotCache->getSummonerId($summoner_name);
$currentGame2 = $riotCache->getCurrentGame($data, "NA1");
print_r($currentGame2);
} catch(Exception $e) {
echo "Error: " . $e->getMessage();
};
try {
$summonerID = $riot->getSummonerId($summoner_name);
$currentGame = $riot->getCurrentGame($summonerID, "NA1");
print_r($currentGame);
} catch(Exception $e) {
echo "Error: " . $e->getMessage();
};
Does this mean that if I put print_r($currentGame2); it will post the json data that is stored within /cache/ instead of grabbing data from riot again? Won't it be like this:
{"createdAt":1447367558,"ttl":3600,"data":"{"gusu":{"id":64887680,"name":"GUSU","profileIconId":750,"summonerLevel":30,"revisionDate":1447366190000}}"}
That's not the same as regular json data from riot... Like this:
{"gusu":{"id":64887680,"name":"GUSU","profileIconId":750,"summonerLevel":30,"revisionDate":1447366190000}}
Is there any simple way to transform the cached data into what is above? I want to be able to make my script check the cache before making an ajax request.

The caching works by checking to see if it has saved it before and returning data from cache if it has. If it hasn't, it goes out and grabs it like normal and returns that. It also saves that call to cache in case it's requested again.

So when I use $riot = new riotapi(new filesystemcache) it will check the cache before? I also created a new function and was wondering how to upload it
Sent from my iPhone

On Nov 12, 2015, at 6:15 PM, Kevin Ohashi<EMAIL_ADDRESS>wrote:
The caching works by checking to see if it has saved it before and returning data from cache if it has. If it hasn't, it goes out and grabs it like normal and returns that. It also saves that call to cache in case it's requested again.
―
Reply to this email directly or view it on GitHub.

It should. I'm not sure what you mean by created a new function and upload it?

Oh yeah I created a new function for the api and I was wondering if you could add it to the api. I had to create this function for my site.
Sent from my iPhone

On Nov 13, 2015, at 12:40 PM, Kevin Ohashi<EMAIL_ADDRESS>wrote:
It should. I'm not sure what you mean by created a new function and upload it?
―
Reply to this email directly or view it on GitHub.

You're welcome to submit a pull request so it can be reviewed for possible inclusion. What does the function do?

It just gets the champion By it's ID. getChampion() returns all champion
information while my function "getChampionByID" allows someone to get a
champion based on it's ID. How do I submit a pull request?
On Fri, Nov 13, 2015 at 11:32 AM, Kevin Ohashi<EMAIL_ADDRESS>wrote:

You're welcome to submit a pull request so it can be reviewed for possible
inclusion. What does the function do?
—
Reply to this email directly or view it on GitHub
https://github.com/kevinohashi/php-riot-api/issues/54#issuecomment-156535980
.

https://help.github.com/articles/using-pull-requests/

I just realized that getMatchHistory() seems to be deprecated.  I think riot uses this now:
https://na.api.pvp.net/api/lol/na/v1.3/game/by-summoner/{summoner_id}/recent?api_key={api_key}

Just found it on the forums: https://developer.riotgames.com/discussion/community-discussion/show/iz34oFti
Yeah they removed it 2 months ago. You should remove it from the project


quaternion rotations are ignored on iOS/XCode

Since iOS and XCode currently do not support quaternion xform rotations, add an option to convert quaternion rotations to baked linear Euler rotations:
https://github.com/kcoley/gltf2usd/issues/50

I added a new flag called "--use-euler-rotation".  Using this should get the rotations to work by converting the rotation quaternions to euler angles.  Note that this won't be smooth for all animations.

@kcoley I was trying this out today, and basically if you rotate more than 180 degrees in one direction it jumps a little. Is that because of gimbal lock when using euler rotations? i'm assuming quaternion rotations don't suffer the issue.

@wave-electron yes, I think it is gimbal lock related.  I am currently using the USD apis to directly convert the quaternions to euler angles.  I'm hoping an iOS update would support quaternions, but in the meantime, I may try to come up with a better algorithm to prevent the jumps.


docs: auto-deployment of versioned docs

On tag of a new version, get the version and build the docs for that version.

Closed in #20


/scss/ in preprocessorIgnorePatterns has no effect

We're using CSS modules in our React components, which are importing .scss files, like this:
import styles from './ReviewFeedItem.scss';

To ignore them when running jest, I've excluded /scss/ from the module loader (or so I think):
// package.json

"preprocessorIgnorePatterns": [
  "/scss/"
],

However, when running the tests, the module loader still tries (and fails) to parse the .scss file as javascript and yields this error:
Runtime Error
SyntaxError: Unexpected token ILLEGAL in file 'src/components/navigation/Navigation.scss'.

Make sure your preprocessor is set up correctly and ensure your 'preprocessorIgnorePatterns' configuration is correct: http://facebook.github.io/jest/docs/api.html#config-preprocessorignorepatterns-array-string
If you are currently setting up Jest or modifying your preprocessor, try `jest --no-cache`.
Preprocessor: jest-preprocessor.js.
Jest tried to the execute the following preprocessed code:
body { height: 0; }

Any variation of the exclude Pattern, such as \\.scss, scss and so forth has no effect.
We're using babel-jest as the scriptPreprocessor (so no specific entry for that in package.json)

Any updates on this?

This doesn't work because ignoring scss from the preprocessor just means that it will not be run through babel-jest. Jest will then try to require the scss file directly and then fail because it isn't valid JavaScript.
There are two solutions:
Use the moduleNameMapper option and a stub file:
Jest config:
"moduleNameMapper": {
  "\.scss$": "path/to/SCSSStub.js"
}

SCSSStub.js:
module.exports = {}; // Or other stub data for all SCSS modules.

Build your own preprocessor using node-sass.
I'm not entirely sure what webpack does to export scss files as JavaScript, but basically you'll want to wrap babel-jest and do something like this:
module.exports = function(src, filename) {
  if (fileName.endsWith('.scss')) {
    return processSass(src);
  }
  return babelJest.process(src, filename);
}

where the processSass would use node-sass and whatever the webpack loaders do :)
I hope one of these is a satisfying solution. If you do build a preprocessor that wraps babel-jest, I'd be happy to add it to Jest as an officially supported package.

The moduleNameMapper solution worked great for me. Here's what I used (took me a while to figure out the magic that would make all imports for SCSS files load the same module):
    "moduleNameMapper": {
      "^.*\\.scss$": "./src/shared/test/SCSSStub.js"
    }

that's great!

preprocessorIgnorePatterns does not need slashes around the entries. It should work with .scss$

@cpojer I just tried upgrading to v16.0.1 (from v14.1.0) and all our tests now fail because it complains about syntax errors related scss files. We have moduleNameMapper set up and the setup worked fine with v14.1.0. Wondering if you have any thoughts on what might be the issue?
jest setup:
"jest": {
    "moduleFileExtensions": [
      "",
      "js",
      "json"
    ],
    "modulePaths": [
      "./src"
    ],
    "moduleDirectories": [
      "node_modules"
    ],
    "testRegex": "/.*\\-snapshot\\.js$",
    "moduleNameMapper": {
      "^testUtils": "./src/utils/tests.js",
      "^.+\\.(s?css|less)$": "./src/utils/styleMock.js",
      "^.+\\.(gif|ttf|eot|svg)$": "./src/utils/fileMock.js"
    },
    "setupFiles": [
      "./src/utils/setupJest.js"
    ]
  },

styleMock.js
// Return an object to emulate css modules
import idObj from 'identity-obj-proxy';

export default idObj;

Example of an error:

@iam-peekay I was getting the same error due to sass imports. Following the instructions in the Jest webpack tutorial resolved the issue for me - specifically adding the moduleNameMapper for files and styles, and the mocks they reference.
I also had to add md to the mapped file extensions as I'm importing markdown content in the project.

Took me a while to get this to work for me because I thought the path was relative to the project.
Below works for me:
    "modulePaths": [
      "v2/app"
    ],
    "moduleNameMapper": {
      ".scss$": "SassStub.js" // <-- lives in v2/app
    },

SassStub.js
module.exports = {}

Hi guys, I'm running into this problem too. What is a SassStub.js file? What goes inside it? Can't find anything about it.

@leongaban Try this solution: https://stackoverflow.com/questions/41040269/syntax-error-when-test-component-with-sass-file-imported

@leongaban See https://github.com/facebook/jest/issues/870#issuecomment-286575674:
SassStub.js
module.exports = {}

Struggling to get this to work, I have:
"jest": {
  "transform": {
    "^.+\\.jsx?$": "babel-jest"
  },
  "collectCoverageFrom": [
    "src/**/*.{js,jsx}"
  ],
  "moduleNameMapper": {
    "^.*[.](jpe?g|gif|png|less|css)$": "./emptyTestModule.js"
  },
  "setupFiles": [
    "<rootDir>/setupTests.js"
  ]
}

And I'm not exactly sure where to put the emptyTestModule, I've tried both the root and src/ but I get:
 Test suite failed to run

    Configuration error:

    Could not locate module ag-grid/dist/styles/ag-grid.css (mapped as emptyTestModule.js)

    Please check:

    "moduleNameMapper": {
      "/^.*[.](jpe?g|gif|png|less|css)$/": "emptyTestModule.js"
    },
    "resolver": undefined

95% of projects will import non JS stuff if using webpack so I'm surprised this is a pain point!


[$250] TextField(Composer) is not focused again when group chat participants Details drawer is closed - Reported by @aneequeahmad

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any group chat or create a new group chat from + icon on LHN
Click on top header to open chat participants details.
Click on cross icon of Details.

Expected Result:
TextInput should be focused
Actual Result:
TextInput isn't focused
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.96-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
potentially related to https://github.com/Expensify/App/issues/10414
https://user-images.githubusercontent.com/43995119/189034190-22c26871-5599-4157-b8ab-97d3f99b6bf0.mov
Expensify/Expensify Issue URL:
Issue reported by: @aneequeahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661285952608309
View all open jobs on GitHub

Internal upwork job: https://www.upwork.com/ab/applicants/1567811708523073536/job-details
External upwork job: https://www.upwork.com/jobs/~01f274e1ebbcbe68a6

Need to fire text focus again when we close drawer on on drawer close event

Hello, I just explored about your company Expensify and the work you people are doing is great. I visited this github link and watched the video and understand that you want this text field to stay focused even when we click on the details of a user. From my initial understanding, you are having some issues with your CSS Psuedo Classes such as focus and active. And if you require further edition in the website on ReactJS then I'll be right person to do so.
For editing, I would first of all trace down its component where this is written and then afterwards as I have mentioned above that as per my understanding there is an issue with the CSS Pseudo CLasses then I'll change that class to the right one. However I feel that there's the similar issue on every text field so I'll trace their components and edit them as per your requirement. Furthermore I'll edit this code as per your requirement which is mentioned below.
Looking forward to hearing from you.
Regards,
Hashir
https://www.upwork.com/ab/proposals/1567856788281503745?success This is my upwork proposal

Proposal
Root cause
To focus the composer it depends on modal.isVisible. But currently when open ReportParticipantsPage, the Onyx key of modal.isVisible is not set to true and instead has a false value even after the page is closed.
https://github.com/Expensify/App/blob/1718572434eafd9c3a816653c6eb8a95054e958c/src/pages/home/report/ReportActionCompose.js#L162-L174
There's a modal listener to set the modal.isVisible when the modal is focused or before closing, but the screen definition with not set the modal screen listener to the ReportParticipantsPage so that's why the modal.isVisible value will still to be a false.
https://github.com/Expensify/App/blob/1718572434eafd9c3a816653c6eb8a95054e958c/src/libs/Navigation/AppNavigator/AuthScreens.js#L260-L264
Solution
Adding the modal screen listeners to the screen option.
diff --git a/src/libs/Navigation/AppNavigator/AuthScreens.js b/src/libs/Navigation/AppNavigator/AuthScreens.js
index 3454a6847..335adbec4 100644
--- a/src/libs/Navigation/AppNavigator/AuthScreens.js
+++ b/src/libs/Navigation/AppNavigator/AuthScreens.js
@@ -261,6 +261,7 @@ class AuthScreens extends React.Component {
                     name="Participants"
                     options={modalScreenOptions}
                     component={ModalStackNavigators.ReportParticipantsModalStackNavigator}
+                    listeners={modalScreenListeners}
                 />
                 <RootStack.Screen
                     name="IOU_Request"

Result
https://user-images.githubusercontent.com/25520267/189139092-a19c9fd2-a982-4814-85e8-c9016c8cea45.mov

@mollfpr very clear proposal thanks. Let's do it!

Applied to Upwork and PR raised, thanks!

Thanks! @aneequeahmad can you please also apply to the Upwork job?

@puneetlath I have applied to the Upwork job. Thanks

All paid. Thanks everyone!


fix: remix icon color and overlapping bgColor in framework link in dark mode

Closes #359
📝 Description
Sets the color of the Remix SVG to black and fixes the overlapping of FrameworkLinks bgColor in the bottom corners in dark mode.
⛳ Current behavior (updates)

invisible color of Remix SVG due to fill set to currentColor
overlapping pixels of the bgColor in the bottom corners

🚀 New behavior

color of Remix SVG is set to black, similar to the Next.js SVG
no white bgColor visible in the bottom corners by increasing the borderRadius

💣 Is this a breaking change (Yes/No):
No
📝 Additional Information

@m4x3d Thanks for taking care of this. Looks good to me :) 🚀


What is the format for masks when using CustomDataset?

Hi,
Thank you for the great documentation regarding training on custom datasets. I was unable to find the correct format for specifying masks in the annotations. My guess is that for each bounding box there should be a corresponding binary array in annotation['ann']['masks'] with the same dimensions as the image.
For example, a file with 4 bounding boxes on a 1200 x 800 image would have binary mask annotations of size 4 x 1200 x 800, even if they are each of different classes.
Thanks for your help.

For COCO-style datasets, masks are annotated with polygons and stored in the json file after being encoded. In the dataset, these annotations will be decoded to binary masks, and you can refer to _parse_ann_info() for details.
For custom datasets with mask annotations, you can either (1) convert the mask annotation to the format compatible with COCO dataset, or (2) override the get_ann_info method as mentioned in the documentation. I prefer the second choice.

@hellock
Hi,
I have the same question as @i-pan , and I looked up the founction "get_ann_info" in coco.py. However, it said "masks are raw annotations and not decoded into binary masks". I can not find where the polygon annotations has been converted to binary masks, could you tell me about it?
Thanks!

It looks like native support for binary masks is no longer possible after a commit on July 4, 2019 (https://github.com/open-mmlab/mmdetection/commit/0d5233a32dac4288760f6d819ddfef9c1e020cf7).
Now it looks like annotations must be in COCO segmentation format. If you have binary masks, you can do a mask-to-COCO preprocessing step, or override get_ann_info to convert the mask to COCO dynamically.


Use eventId instead of eventid

Description
Use variable name eventId instead of eventid to follow the camel case naming convention.
Fixes #831
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Please move the CHANGELOG entry under the Unreleased section and fix any merge conflicts. Thanks.

Please fix the merge conflict. Thanks.

I tested this PR by adding a new sub-event and super event, editing and deleting these. I also viewed some old events. This generally looks fine. So, far I haven't come across any issues.


CQL expression with error is being reformatted

When a CQL expression has an error, it is being reformatted.
For example,
library test version '0.0.000'

using QDM version '5.3'

notvalueset "TestValueset" '1.1.1'

define "TestDef":
	["Encounter, Performed"] E
		wher E.code is not null

results in:
library test version '0.0.000'

using QDM version '5.3' notvalueset "TestValueset" '1.1.1'

define "TestDef":
	["Encounter, Performed"] E wher E.code is not null

Addressed in v1.2.12-SNAPSHOT.


Rename Servers to Server

@dkhamsing Sounds reasonable. But rename to Server or Server-Side? :stuck_out_tongue:

Haaha shorter is better?

Deal!

@victorgama thanks!


Fix link to documentation about Worldpay 3D Secure setup

Change link from https://govukpay-docs.cloudapps.digital/switching-to-production#worldpay-setup-for-3d-secure to https://govukpay-docs.cloudapps.digital/#worldpay-setup-for-3d-secure

But FYI, I just broke self service master, fixing acceptance tests now, but this is also needs to go in https://github.com/alphagov/pay-selfservice/pull/684


Display number of remaining threats in dialog

When selecting an item in the threat model, then "suggest threats for the selected element", a dialog pops up with a suggest threat. Accepting or Ignoring the dialog may cause further threats to show up. It would be useful if the title of the dialog could show a count. E.g. "Add this threat? (1 / 4)"

fixed in commit b8efc67


Add spatial scatterplot function to Galaxy Squidpy tools

Added a wrapper of Squidpy.pl.spatial_scatter() to the Galaxy Squidpy tool suite. This implementation of the function is limited to generating scatterplots of spatial data that can be colored by a categorical or continuous variable. This fulfills the need for Galaxy-ME to have static spatial scatterplots, and also has some nice features (coordinate cropping, scalebars, etc.)

This PR is related to

[x] Adding a new tool
[ ] Updating an existing tool to a newer version
[ ] Fixing a bug or updating just the Galaxy wrapper of an existing tool
[ ] Making a change to the tools-mti repo, CI, or other misc. change

If updating an existing tool to a newer major version

[ ] I have updated the TOOL_VERSION token in the tool's macros file
[ ] I have reset the VERSION_SUFFIX to 0 in the tool's macros file

Provide details here

Tool requirements have not changed
See Squidpy docs for more on the tool itself
This limited implementation does not include functionality for multiple images (library_id), or overlaying the actual image data with the scatterplot. These could be added in the future

It would be nice if we can convert the shed.yml into a suite based one. So that all tools in this folder get a separate repo.

Thanks @bgruening, that should be implemented now!


Problem accessign DOM in turbolinks:load event

Good morning/afternoon guys.
I've found what seems to be an issue. In a rails 5.1 project I've detected that sometimes the turbolinks:load event is triggered before the DOM is ready. As a simple example:
$(document).on('turbolinks:load', function() {
var dtpEventFilter = $('#dtp-event-filter');
console.log(dtpEventFilter.length);
});
The element with ID exists, but even there it prints '0' in the console.
Making this small change makes things start working again:
$(document).on('turbolinks:load', function() {
setTimeout(function() {
var dtpEventFilter = $('#dtp-event-filter');
console.log(dtpEventFilter.length);
}, 0);
});
Is it a but, or it's something I'm doing wrong?
I'm developing with Rails 5.1. Turbolinks versions are:
turbolinks (5.0.1)
turbolinks-source (5.0.3)
I've observed this behaviour under Windows 10 + Chrome Version 59.0.3071.115 (Build oficial) (64 bits)

Sorry guys, you can ignore this. One of my coleagues implemented the form with Knockout so the binding returns zero simply because it's not available when the event is triggered.
My fault!


Fix for https://github.com/mikebrock/mvel/issues/27

We came across the problem mentioned in https://github.com/mikebrock/mvel/issues/27
and unfortunately, we cannot move to latest version of MVEL2.  We tried with below change and it works for us with Java 8. Is this something we can release part of 2.1.X releases, may be as 2.1.10.Final?  And unfortunately, I am not finding any branch associated to 2.1. releases but can see tags. Please let me know if you are fine with this change but re-create PR against a different branch.

Thanks @mariofusco  for merging this Pull Request. Is this fix can be released part of 2.1.X.Final releases? Let me know in case you want me to create a separate PR for a specific 2.1.x branches.

@mariofusco We still use JBPM and Drools 5 and cannot upgrade due to version 6 because of dependencies. I have verified that @mkornipati 's fix solved our intermittent compilation issues with Java 8.  Can this fix be backmerged to 2.1.X?

@jscattergood @mkornipati  : Could you please tell me the steps involved to fix the issue ? We are facing the intermittent compilation error with JAVA 8. Any help would be much appreciated.

@arun2arunraj You can see the changed file in this PR.  If the fixed isn't merged into your version, then you need to create a fork of mvel repository, create a branch appropriate for yourself, merge the changes and build your own jar.

Code within org.mvel2.optimizers.impl.asm.ASMAccessorOptimizer should be fixed as shown in below article:
http://copyrightdev.tumblr.com/post/146315831773/getting-drools-5x-to-operate-smoothly-with-java-8

Hi sir I am new to gradle and I am using maven as dependency Management tool how i will be able to achieve the same please enlighten me on the same.

@shaileshsingh9214 you can follow same practice in Maven too - https://maven.apache.org/guides/introduction/introduction-to-optional-and-excludes-dependencies.html
exclude MVEL explictly for drools dependencies and include your patched MVEL

@mariofusco We still use JBPM and Drools 5 and cannot upgrade due to version 6 because of dependencies. I have verified that @mkornipati 's fix solved our intermittent compilation issues with Java 8. Can this fix be backmerged to 2.1.X?

I'm confused: Why the issues is interstitial. I'll be appreciate if you can explain.


ACR Auth with Service Principal script does not work anymore

The script to create a create a service principal and give access to ACR does not work anymore.
Here's the output from a freshly installed Azure CLI on Ubuntu.
> # Create the service principal with rights scoped to the registry.
> # Default permissions are for docker pull access. Modify the '--role'
> # argument value as desired:
> # acrpull:     pull only
> # acrpush:     push and pull
> # owner:       push, pull, and assign roles
> SP_PASSWD=$(az ad sp create-for-rbac --name http://$SERVICE_PRINCIPAL_NAME --scopes $ACR_REGISTRY_ID --role acrpull --query password --output tsv)

WARNING: Creating 'acrpull' role assignment under scope '/subscriptions/[redacted]/resourceGroups/DiscordBotRg/providers/Microsoft.ContainerRegistry/registries/DiscordBotAcr'
WARNING: The output includes credentials that you must protect. Be sure that you do not include these credentials in your code or check the credentials into your source control. For more information, see https://aka.ms/azadsp-cli
WARNING: 'name' property in the output is deprecated and will be removed in the future. Use 'appId' instead.
> SP_APP_ID=$(az ad sp show --id http://$SERVICE_PRINCIPAL_NAME --query appId --output tsv)
ERROR: Service principal 'http://acr-service-principal' doesn't exist
>
> # Output the service principal's credentials; use these in your services and
> # applications to authenticate to the container registry.
> echo "Service principal ID: $SP_APP_ID"
Service principal ID:
> echo "Service principal password: $SP_PASSWD"
Service principal password: [redacted]

It successfully created the SP, it prints the password, but the following line does not work:
SP_APP_ID=$(az ad sp show --id http://$SERVICE_PRINCIPAL_NAME --query appId --output tsv)

As a result, the output is Service principal ID:  without the ID.
If you use the list command to query service principals by display name, you can use http://$SERVICE_PRINCIPAL_NAME, but using the show command with the id argument, this does not work.

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

ID: 11bbaa44-52ed-33fd-d710-af73d3665ec9
Version Independent ID: b11deb17-311c-3aa3-78a7-1f8cdc7e9e5e
Content: Authenticate with service principal - Azure Container Registry
Content Source: articles/container-registry/container-registry-auth-service-principal.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

@swimbuger-  Thanks for brining this to our attention. We will investigate it further and update you shortly.

(Duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/77870)
@Swimburger - Thanks for reaching out and apologies that the script didn't work for you. A change was introduced in the az ad sp create-for-rbac command in Azure CLI v. 2.25. 0 that affects how ids are handled in this script. Instead of using this az ad sp show command:
SP_APP_ID=$(az ad sp show --id http://$SERVICE_PRINCIPAL_NAME --query appId --output tsv)

you should instead run az ad sp list as you suggested:
SP_APP_ID=$(az ad sp list --display-name http://$SERVICE_PRINCIPAL_NAME --query [].appId --output tsv)

Please let us know if this helps. We'll retest and publish a doc update shortly.

@Swimburger Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


we do not want to encourage chat using guest access. Further, the very next note talks about the right way of chatting and the word chat in this paragraph is likely to create confusion to readers.

@MicrosoftHeidi are you able to review and merge as needed?


Triton supports http streaming return？

Hello, currently I refer to decouple The entire service is already running

I would like to ask, if I want to stream the return, can I only use the grpc method? Can the http method also stream the return? Thank you~

No. We cannot use decoupled mode with http. More details and discussion are here https://github.com/triton-inference-server/server/issues/2168.


Adding a post about Power Query ETL Capabilities.

Category

[ ] Content fix
[x] New article

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

Contents of the Pull Request
In this article, I talked about the ETL (Extract, Transform, and Load) capabilities of Power Query. I also added some screenshot which i took and created myself.
Guidance

Thanks @LuiseFreese