Inconsistency in handling instantiate to bounds in analyzer and CFE

For code like
class C<X> {}
typedef A<X extends FutureOr<C<X>>> = C<X>;
class D<X extends FutureOr<D<X>>> {}

Analyzer infers
class C<covariant X> {}
notSimplyBounded typedef A<covariant X extends dynamic> = C<X>;
notSimplyBounded class D<covariant X extends dynamic> {}

(with a pending CL) it also reports errors for both A() and D() instantiation.
main() {
  var a = A();
  var d = D();
}

error: Couldn't infer type parameter 'X'.

Tried to infer 'FutureOr<C<Object?>>' for 'X' which doesn't work:
  Type parameter 'X' is declared to extend 'FutureOr<C<X>>' producing 'FutureOr<C<FutureOr<C<Object?>>>>'.

Consider passing explicit type argument(s) to the generic.

 (could_not_infer at [my_test] bin/test.dart:9)
error: 'FutureOr<D<Object?>>' doesn't conform to the bound 'FutureOr<D<FutureOr<D<Object?>>>>' of the type parameter 'X'. (type_argument_not_matching_bounds at [my_test] bin/test.dart:10)
error: Couldn't infer type parameter 'X'.

Tried to infer 'FutureOr<D<Object?>>' for 'X' which doesn't work:
  Type parameter 'X' is declared to extend 'FutureOr<D<X>>' producing 'FutureOr<D<FutureOr<D<Object?>>>>'.

Consider passing explicit type argument(s) to the generic.

 (could_not_infer at [my_test] bin/test.dart:10)

CFE seems to produce the same inference:
static method main() → dynamic {
  self::C<FutureOr<self::C<core::Object?>>> a = new self::C::•<FutureOr<self::C<core::Object?>>>();
  self::D<FutureOr<self::D<core::Object?>>> d = new self::D::•<FutureOr<self::D<core::Object?>>>();
}

But I see the error only in one place:
../../../../dart/test/bin/test.dart:10:11: Error: Inferred type argument 'FutureOr<D<Object?>>' doesn't conform to the bound 'FutureOr<D<X>>' of the type variable 'X' on 'D'.
 - 'D' is from '../../../../dart/test/bin/test.dart'.
 - 'Object' is from 'dart:core'.
Try specifying type arguments explicitly so that they conform to the bounds.
  var d = D();
          ^
../../../../dart/test/bin/test.dart:6:9: Context: This is the type variable whose bound isn't conformed to.
class D<X extends FutureOr<D<X>>> {}
        ^

And the corresponding test co19/src/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_FutureOr_l1_t04.dart does not have an error expectation for
class C<X> {}
typedef A<X extends FutureOr<C<X>>> = C<X?>;

main() {
  A();
}

@leafpetersen @eernstg Could you please clarify?

@scheglov That looks like a bad test to me.  To be well-bounded, it requires that FutureOr<C<Object?>> <: FutureOr<C<FutureOr<C<Object?>>> be true.  This in turn requires that both:

`Future<C<Object?>> <: FutureOr<C<FutureOr<C<Object?>>> and
C<Object?> <: FutureOr<C<FutureOr<C<Object?>>>

Neither of those look to be provable to me.  I'll mark this as a bad test.
cc @eernstg

Yes, I'd say that neither A nor D can be used raw, and that's because i2b yields A<FutureOr<C<dynamic>>> respectively D<FutureOr<C<dynamic>>>, and those types are not well-bounded. I believe there are other tests covering the use of such types as type annotations (cf. https://github.com/dart-lang/co19/issues/1050).
However, the main functions mentioned here contain instance creations, and they would receive their type arguments from type inference (hence Object? rather than dynamic). The instance creations are not well-bounded (and hence not regular-bounded) so the test must expect them to be compile-time errors.

@eernstg There are more co19/ tests like this that started failing with this CL.
Is this something to cover in https://github.com/dart-lang/co19/issues/1050?
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_FutureOr_l1_t02
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_FutureOr_l1_t03
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_FutureOr_l1_t04
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l1_t01
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l1_t02
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l1_t05
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l1_t06
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l1_t07
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l2_t01
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l2_t02
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l2_t03
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l2_t09
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l2_t10
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_l4_t01
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l1_t01
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l1_t02
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l2_t01
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l2_t02
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l2_t05
FAILED: dart2analyzer-none release_x64 co19/LanguageFeatures/Instantiate-to-bound/nonfunction_typedef/static/nonfunction_typedef_l2_t06

Created https://github.com/dart-lang/co19/issues/1057 for this.


Debug Your Java App

It's not possible to validate de "step 1"  in the page : https://developers.sap.com/tutorials/cp-cf-java-debug.html.
The checkmark could not be green and so you can't have 100%.

Hi @ericrosfelder ,
could you please describe your problem again? Step 1 doesn't have any validation.  And could you also please retry validating the steps anyway? All validations work for me so far.
Cheers,
Maxi

Hi @maxstreifeneder ,
Exact, step 1 doesn't have any validation but step 1 taken into account for the overall rating.
So I have 67% because I can just validate step 2 and step 3.

@thecodester Could you please have a look at this problem? eric is right, I was able to reproduce the problem.

The rules file has no correct answer specified, so the question did not get created :-(
Do you know what the answer should be?
Whose tutorial is this?

issue should be fixed


Performance Testing

Description
Acceptance Criteria

[ ] Acceptance Criteria

Additional Context and Related Items

Closing: stale and empty


Setting the window icon

Original: https://github.com/tomaka/glutin/issues/771

Hi!
I am wondering, has anything been made in this matter? Is there at least an API people could start implementing?

Perhaps the best way to handle it would be a feature flag that uses a crate like image and abstracts the icon assignment for whatever window manager you're using. Of course, anything that requires an external image file would require saving to the file at runtime or a build.rs for it, but it should be possible.

I'd probably make some sort of system that just takes a simple RGBA bitmap in the form of a &[u8] and turns it into whatever platform-specific format is necessary.  It's a bit of a hack I expect but hopefully a very simple hack...

It's too fiddly to make Windows also take ARGB or something like that? (I
don't know much about Windows.)
On Thu, Apr 26, 2018, 22:17 Francesca Frangipane<EMAIL_ADDRESS>wrote:

I plan on using WindowBuilderExt and WindowExt for this, since 1) it only
seems relevant on Windows and X11, and 2) X11 wants ARGB while Windows
wants paths (and is picky about what it accepts).
[image: window-icon]
https://user-images.githubusercontent.com/2096580/39340530-a2e0de82-499c-11e8-8bfb-3d06a1e25f03.png
Implementing this on X11 will most likely be trivial.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tomaka/winit/issues/69#issuecomment-384843598, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABRfXUJlVyl8BlGwpuQnovpMI4sdWH1pks5tsn_TgaJpZM4KoFdd
.

To be honest, my main rationale was that the docs for CreateIcon looked intimidating. Since you asked, I gave it a shot today, and it was actually pretty easy.
So, now I can provide a consistent API that expects RGBA. I'll also include an example program that demonstrates how to use image to set the icon using any image file.

PR is up: #497


getting error while running ns-export suddenly

ns-export command throwing error with video dataset but not while running image dataset.

Earlier ns-export command was running successfully but suddenly it started throwing error. Also single dataset is running at a time.
Seems like bug in "rich" library, still need help to solve it.

Can you see if this change fixes your error?
https://github.com/nerfstudio-project/nerfstudio/pull/1600/files

Same issue here, any progress on this so far?
/workspace$ ns-export poisson --load-config outputs/unnamed/nerfacto/2023-04-24_171122/config.yml --output-dir /workspace/2023-04-24_15-08-11/exports/mesh/ --num-points 10000000 --target-num-faces 500000                            [17:31:09] Auto image downscale factor of 1                                                 nerfstudio_dataparser.py:314            Skipping 0 files in dataset split train.                                         nerfstudio_dataparser.py:165            Skipping 0 files in dataset split test.                                          nerfstudio_dataparser.py:165 Setting up training dataset... Caching 250 out of 1119 images, without resampling. Setting up evaluation dataset... Caching all 124 images. Loading latest checkpoint from load_dir ✅ Done loading checkpoint from outputs/unnamed/nerfacto/2023-04-24_171122/nerfstudio_models/step-000029999.ckpt Checking that the pipeline has a normal output. ☁ Computing Point Cloud ☁ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% --:-- Traceback (most recent call last):   File "/home/user/.local/bin/ns-export", line 8, in <module>     sys.exit(entrypoint())   File "/home/user/nerfstudio/scripts/exporter.py", line 368, in entrypoint     tyro.cli(Commands).main()   File "/home/user/nerfstudio/scripts/exporter.py", line 262, in main     pcd = generate_point_cloud(   File "/home/user/nerfstudio/nerfstudio/exporter/exporter_utils.py", line 133, in generate_point_cloud     ray_bundle, _ = pipeline.datamanager.next_train(0)   File "/home/user/nerfstudio/nerfstudio/data/datamanagers/base_datamanager.py", line 478, in next_train     image_batch = next(self.iter_train_image_dataloader)   File "/home/user/nerfstudio/nerfstudio/data/utils/dataloaders.py", line 131, in __iter__     collated_batch = self._get_collated_batch()   File "/home/user/nerfstudio/nerfstudio/data/utils/dataloaders.py", line 117, in _get_collated_batch     batch_list = self._get_batch_list()   File "/home/user/nerfstudio/nerfstudio/data/utils/dataloaders.py", line 110, in _get_batch_list     for res in track(results, description="Loading data batch", transient=True):   File "/home/user/.local/lib/python3.10/site-packages/rich/progress.py", line 167, in track     with progress:   File "/home/user/.local/lib/python3.10/site-packages/rich/progress.py", line 1169, in __enter__     self.start()   File "/home/user/.local/lib/python3.10/site-packages/rich/progress.py", line 1160, in start     self.live.start(refresh=True)   File "/home/user/.local/lib/python3.10/site-packages/rich/live.py", line 113, in start     self.console.set_live(self)   File "/home/user/.local/lib/python3.10/site-packages/rich/console.py", line 835, in set_live     raise errors.LiveError("Only one live display may be active at once") rich.errors.LiveError: Only one live display may be active at once

Can you try this change to see if it makes a difference. https://github.com/nerfstudio-project/nerfstudio/pull/1600/files


useTabGroupChoice no longer provided by docusaurus-theme-common

Describe the bug
When using docusaurus-openapi-docs to generate OpenAPI 3.0-based documentation, viewing the page shows an error: "This page crashed. useTabGroupChoice is not a function".
Seems like they removed that function in this commit on Dec. 20, 2022, which seems to have gone out as part of the 2.3.0 (2023-01-26) release.
Expected behavior
Normal page rendering.
Current behavior
Page crashes.
Possible solution
It seems like the useTabGroupChoice function has been deleted from docusaurus-theme-common, so I suppose an alternative may have to be used. Sorry, that's not a ton of help, I haven't dived into what the function used to do and if they've introduced an alternative.
Steps to reproduce

Generate a basic, classic Docusaurus project (as per their installation instructions).
Install/configure docusaurus-openapi-docs as per this repo's README instructions.
Copy the example Petstore yaml from OpenAPI into the project and make sure the configuration points at it.
Generate the MDX using this plugin and npm run start.
Navigate to one of the generated pages.

Screenshots

Context
Just trying out the plugin for a future documentation project, since it looks amazing!
Your Environment

docusaurus-plugin-openapi-docs version: 1.5.2
docusaurus-theme-openapi-docs: 1.5.2
Resolved @docusaurus/theme-common version in package-lock.json: 2.3.1
Chrome version: Version 105.0.5195.125 (Official Build) unknown (64-bit)

I think this is a duplicate of https://github.com/PaloAltoNetworks/docusaurus-openapi-docs/issues/396.  @sserrata maybe it's worth pinning that issue for now?


aws-adfs: 1.19.1 -> 1.20.0; fix build

Motivation for this change

Build is broken on Hydra for quite some time (https://hydra.nixos.org/build/107872384)
update available

Things done

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

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

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

Notify maintainers
cc @bhipple

@GrahamcOfBorg build pythonPackages.aws-adfs

@jonringer I fixed the commit message. I think the failing builds for Python  2.7 can be ignored as it will be EOL at the end of the year anyway - they relate to dependency versions not matching.

I would either disable python2, or freeze lxml to <4.4.0 for python2, disabling for python2 would be easier
builder for '/nix/store/sc2slb55b18ac0q9x8sina3brla6sf7p-python2.7-aws-adfs-1.20.0.drv' failed with exit code 1; last 10 log lines:
  adding 'aws_adfs-1.20.0.dist-info/RECORD'
  removing build/bdist.linux-x86_64/wheel
  Finished executing setuptoolsBuildPhase
  installing
  Executing pipInstallPhase
  /build/aws-adfs-1.20.0/dist /build/aws-adfs-1.20.0
  DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
  Processing ./aws_adfs-1.20.0-py2-none-any.whl
  ERROR: Could not find a version that satisfies the requirement lxml<4.4.0; python_version < "3.5" (from aws-adfs==1.20.0) (from versions: none)
  ERROR: No matching distribution found for lxml<4.4.0; python_version < "3.5" (from aws-adfs==1.20.0)

@jonringer I fixed it.

@GrahamcOfBorg build aws-adfs


Remove project dependency between unit tests

Copied the shared code so that each project has their own copy of the methods in use, so that the project references between test projects are removed (except Stratis.Bitcoin.Tests, that is the next task)

@dangershony I think it could be a good idea create a single test utilities project at some point with some of these commonly used methods if they can be shared. What do you think?

@Neurosploit I tend to agree, I just changed this PR to do that.  The new assembly is Stratis.Tests.Common - this could later be nugetted and shared across other projects too (where relevant)

I personally dont have an issue with duplicating test helpers in test assemblies as its not production code. But I do understand that its not very good practise with regards to maintaining the code.
If you feel strongly about not duplicating the code then we need to move the wallet helpers to Tests.Common.
The problem with having unit test projects referencing each other, is that Visual Studio and VSTS executes teste multiple times cause the referenced test assemblies gets copied into bin folders as well which then gets picked up the runner and executes them.
E.g. If unit test project A references B and C, then VSTS for example will run tests in A,B and C. If unit project D references A,B itll run D,A and B. So in this scenario B gets run twice.

I've never seen this happening in Visual Studio but maybe in VSTS, execution is different.
How do you run the tests there?

 That, or we have a Wallet.Tests.Common assembly but that seems like overkill. At the end of the day we can't have unit test assemblies referencing each other.
That's my favorite option. Thanks @majikandy for pointing it out, I'd missed it.

Good stuff, thanks!

@bokobza @fassadlr I've also never seen that happen. Usually you don't put any unit tests in projects you want have for shared code so you can reduce maintenance and repeat yourself less often. Only with class inheritance I've seen tests show up in the test runner multiple times.


Allow emitter to be placed on the barrel's sides

Suggestion
Allow the emitter to be placed on any sides of the storage block (or barrel) and have it rotate. One reason is top side of storage blocks is needed by mods like "create" for the chutes and has no alternative, also some build designs might want the emitter hidden but the barrel shown.
+++

Implemented in 66f11886c08318098d1c2d12bc245c4d972ceb61. All three sculk item transporters can now be placed on all sides.
We will release an update with this and other fixes & improvements in the very near future.


GitHub: Fixes to service tests and try.html

So I am confused, my PR was merged (in different commits) but still some code was not merged as-is.
Maybe a merge conflict went wrong ?
This PR includes:

Restructured/improved service tests
Updated try.html to include proper label tests (using an actively used label)

Huh. That was #1020, which was merged via 26093d and 8046a4, followed by 3d39cd, which this partly undoes. If you want to change the behavior we should let @espadrine weigh in on that. I'm not sure I follow what he was doing.
Fixing try.html and straight-up test improvements are no-brainers to merge though, so if you'd like to trim this back – or move those things to a different PR – I'd be happy to review it.

@dagwieers Thoughts? I'd like to merge a fix for the flaky Github tests.

@paulmelnikow I don't understand. This is not undoing anything from PR #2010 AFAICT, it's actually re-doing what should have been done if it would have been merged.
This is how it should have been after merging PR #2010:
https://github.com/dagwieers/shields/blob/62c470d56d230d378e595433e51baf72a1dae65c/server.js#L3373-L3378
This is how it is now after the broken merge:
https://github.com/badges/shields/blob/master/server.js#L3470-L3478
This is how this PR will make it:
https://github.com/dagwieers/shields/blob/1c9e994beea89b3f9f54fe3199c9684ec5062c53/server.js#L3401-L3406

Hey @espadrine, your merge of #1020 brought tests and try.html out of sync with the code. I'm not sure I understand your intent in changing the code as it was in the PR. Could you weigh in on this PR?

Huh. As I did further into the test failures, it seems they are due to our token handling logic and custom rate limits in the Search API. That probably needs a separate fix.

I moved the example badge fix to #1141.
@dagwieers If you still want to change the behavior back to what it was, could you merge master + clean up the diff so the change is clearer? If @espadrine doesn't weigh in I'll take a closer look myself.


Import map not capturing multiple versions of a library listed as a peer dependency in multiple packages

After adding a library, and getting the following error on page load:
client.js:24 Uncaught TypeError: e.hydrateRoot is not a function
    at r.hydrateRoot (client.js:24:16)
    at m (main-b1c1d123.js:22:123)
    at index.forecast.js:55:1

I found that when two libraries specify different versions for peer dependencies, that only one peer dependency version is included in the import map https://ga.jspm.io/ scope.
@mui/material
"peerDependencies": {
  ...
  "react": "^17.0.0 || ^18.0.0",
  "react-dom": "^17.0.0 || ^18.0.0"
},

react-data-grid
"peerDependencies": {
  "react": "^16.14 || ^17.0",
  "react-dom": "^16.14 || ^17.0"
}

The import map
<script type="importmap">
  {
    "imports": {
      "@mui/material/Paper": "https://ga.jspm.io/npm<EMAIL_ADDRESS>      "react"<EMAIL_ADDRESS>      "react-data-grid"<EMAIL_ADDRESS>      "react-dom"<EMAIL_ADDRESS>      "react-dom/client"<EMAIL_ADDRESS>      ...
      "react/jsx-runtime"<EMAIL_ADDRESS>    },
    "scopes": {
      "../": {
        ...
      },
      "https://ga.jspm.io/": {
        "react"<EMAIL_ADDRESS>        "react-dom"<EMAIL_ADDRESS>        "react/jsx-runtime"<EMAIL_ADDRESS>      },
      ...
<EMAIL_ADDRESS>{
        "scheduler"<EMAIL_ADDRESS>      }
    }
  }
</script>

I was expecting React 17 to be included in a separate scope -<EMAIL_ADDRESS>or in general to be able to reference multiple versions of react in the import map if that is what the libraries specify peer dependencies. Is this expected?
If I manually change the react versions in the https://ga.jspm.io/ scope to v18, then everything works.
Please let me know if there is a way to overwrite peer dependency versions so that I can force the react imports in the ga.jspm.io scope to be v18.

Looking at the react-data-grid GitHub repository I'm not sure why I got the @7.0.0-canary.49 version actually. In my package.json file the entry is:
"react-data-grid": "^7.0.0-beta.28",

Which specifies React peer dependency as v18. And fixing the entry to that version "react-data-grid": "7.0.0-beta.28", resolves my immediate problem.

Thank you for the responses @Bubblyworld and @guybedford.
In terms of treating peer dependencies as top level vs actual dependencies. I would have thought that most libraries bundle their dependencies into the bundle.js or equivalent. When I import a library that has bundled deps into it, I was just expecting to get multiple copies of those dependencies through the JSPM CDN. Is this correct?
I've never really understood the scopes part of import maps actually. In the import map (re-pasted below) I've got imports and scopes. My understanding is that direct import (i.e. if I have import @mui/material/Paper in the index.js file) are registered as imports. And that indirect imports are added as scopes. Is this correct?
In this import map I have 3 scopes:

../
https://ga.jspm.io
<EMAIL_ADDRESS>
What do each of these mean?

In the case of ../ is this a relative path? If so, what it relative to and from?
Why would my imports be scoped to https://ga.jspm.io/? This seems similar to (1) in that the imports are scoped to a location
What could cause there to be a scope of<EMAIL_ADDRESS>I can see this is also a location

Now that I'm looking at this... Is it correct to say that imports are scoped to locations in the import map? If so why?
I would have expected the keys of the scopes object to be library versions. I.e. if react-data-grid imports react-dom, then I would have expected a lookup of scopes['react-data-grid']['react-dom'] which would resolve to<EMAIL_ADDRESS>that would be a key in the imports object that I would have to ensure existed.
  {
    "imports": {
      "@mui/material/Paper": "https://ga.jspm.io/npm<EMAIL_ADDRESS>      "react"<EMAIL_ADDRESS>      "react-data-grid"<EMAIL_ADDRESS>      "react-dom"<EMAIL_ADDRESS>      "react-dom/client"<EMAIL_ADDRESS>      ...
      "react/jsx-runtime"<EMAIL_ADDRESS>    },
    "scopes": {
      "../": {
        ...
      },
      "https://ga.jspm.io/": {
        "react"<EMAIL_ADDRESS>        "react-dom"<EMAIL_ADDRESS>        "react/jsx-runtime"<EMAIL_ADDRESS>      },
      ...
<EMAIL_ADDRESS>{
        "scheduler"<EMAIL_ADDRESS>      }
    }
  }

Hey, there's quite a bit to unpack:

I've never really understood the scopes part of import maps actually. In the import map (re-pasted below) I've got imports and scopes. My understanding is that direct import (i.e. if I have import @mui/material/Paper in the index.js file) are registered as imports. And that indirect imports are added as scopes. Is this correct?

There are two different things going on here. First, for import maps in general (not just the ones JSPM produces), "imports" are used to map bare specifiers like react to actual module URLs, and "scopes" are used to change those mappings in particular contexts. For instance, in your map above, you have an "imports" mapping for react to<EMAIL_ADDRESS>This means that if you were to call import("react") in a web page with this map installed, it would fetch that module from ga.jspm.io.
However, you also have a "scopes" mapping in https://ga.jspm.io/ for react to<EMAIL_ADDRESS>This means that if any module in that scope (i.e. anything fetched from ga.jspm.io) imports the bare specifier "react", it'll get that version of react instead. Scopes apply to any module that is fetched from a subpath of the scope key, with more specific scopes taking precedence.
In theory you can organise imports and scopes however you like. JSPM takes the following approach:

top level dependencies in modules that are traced or installed go in the "imports"
secondary dependencies go in "scopes"
scopes are flattened as much as possible, i.e. pushed down to the least-common-URL, similarly to how npm flattens the node_modules folder as much as possible. This is why most of the secondaries are under the https://ga.jspm.io scope.

In the case of ../ is this a relative path? If so, what it relative to and from?

Yes, it's relative to the URL of the HTML page that installs the import map. So if the import map was installed into https://foo.bar/my/page.html, the mappings in this scope will apply to all javascript modules under https://foo.bar/my/.

Why would my imports be scoped to https://ga.jspm.io/?

Because JSPM flattens scopes to save space in the import map whenever it can do so (i.e. when doing so doesn't violate a dependency constraint).

What could cause there to be a scope of<EMAIL_ADDRESS>
This is because the scope couldn't be flattened for some reason - probably because one of your other dependencies requires a different version of scheduler, and flattening the scope would have violated that constraint.
Does that make sense? There's a lot going on - you could try reading the import maps spec for some more details, it's not super technical.

Thank you very much for the helpful response - I realize that I was essentially asking how import maps work, and that your answer went well beyond the scope of the JSPM generator. The spec may not be super technical, but it's still a lot to take in! It's a little easier to read now that I have some more context (as in, I'm actively working with import maps)
I'm pretty happy to finally understand how scoping is handled in the import maps! It's surprising to me that it has to do with the origin of a particular library rather than by library version.
Is it possible to have multiple module providers currently?
i.e. if I want to publish a package to consume via an import map that is configured to use JSPM as the provider. Do I have to publish that module to NPM so that it gets transpiled and republished via JSPM? What if I want that particular module to be made available via a different CDN?
Is this possible? One example that comes to mind is that ESRI publishes their JS library in ESM format via their own CDN: https://developers.arcgis.com/javascript/latest/es-modules/#using-the-esm-cdn.
I think there is a possibility that library providers (such as ESRI) stop publishing their JS via NPM so as to force people to use their own module CDNs. This could be mechanism to prevent public disclosure of their library usage metrics, but also so that they have access to usage stats.
... this is a bit on a tangent now, but the reason I thought of it was that, prior to my corrected understanding of import maps I would have assume that if an ESRI JavaScript module tries to import, for example, scheduler, the a single scheduler import mapping would be useable by both modules imported from ga.jspm.io and js.arcgis.com. But now I know that there would (in this theoretical case, if it's possible to support multiple providers in the same map) be two scope entries: ga.jspm.io and js.arcgis.com, and that the scheduler, when imported by libraries served from JSPM, would be fetched from JSPM. And when imported by libraries served via js.arcgis.com, scheduler would be served from their instead.

Actually, thinking about this... I think it makes sense that import map resolution would not support CORS requests for imports from imported modules.
Would that have security implications?

Yeah, the generator is pretty geared towards the single-provider use case at the moment, although there is some support for dividing up package space into different providers, using the providers option. And you can register custom CDNs quite easily with custom providers. (we've been testing the CDN builder like this, for instance - build packages to the filesystem, write a custom provider against them, use jspm to cook up import maps and test the builds work)
I think there are some edge cases to work through still though 👀
The policy is something like this at the moment, for a mapping in the import map:

If the mapped URL corresponds to a CDN that jspm knows about, then it assumes "ownership" of the mapping, and will translate it to other providers if you do a reinstall or update, like jspm install -p esm.sh etc.
If the mapped URL is something custom, it's left as-is.

Thanks - I hadn't seen that documentation before. It's very comprehensive. If I find some edge cases I'll be happy to post them!


[BUG?] @iconify/utils getIconCss does not respect transformations

Unlike iconToSVG, getIconCss seems do not respect transformations in IconifyIcon like vFlip or rotate.
I wonder is this intentional, or a bug. If it is intentional, what is the correct way to create stylesheet for an icon with transformations, like a variation?
I tested this under<EMAIL_ADDRESS>import { IconifyIcon } from '@iconify/types';
import { getIconCSS, iconToSVG } from '@iconify/utils';

const arrowRight: IconifyIcon = {
  body: `<path fill="none" stroke="currentColor" stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M5 12h14m-7-7 7 7-7 7"/>`,
  height: 24,
  width: 24,
};

const arrowDown: IconifyIcon = {
  ...arrowRight,
  rotate: 1,
};

const arrowRightSvg = iconToSVG(arrowRight);
const arrowDownSvg = iconToSVG(arrowDown);

// This is expected to be false
console.log(arrowRightSvg.body === arrowDownSvg.body); // => false, as expected

const arrowRightCss = getIconCSS(arrowRight);
const arrowDownCss = getIconCSS(arrowDown);

// I expect this to be false, too
console.log(arrowRightCss === arrowDownCss); // => true

Thanks a lot for noticing and reporting!
Fixed in 2.1.32


Many images have just one bounding box for a class that in reality has multiple instances on that image

For example this image:

It has multiple chairs, but only one of them has bounding box.
Is it how it was classified or was it done intentionally?

@samihaija Sami, can you please comment on this?

Ivan, thank you for the mention!
The above is intentional, assuming the image came from the training partition. We only have one box per image per entity, for the training set. However, for the validation set, we tried to have all instances boxed.

@samihaija @rkrasin  thank you.
@samihaija I'm not a big expert in object detection algorithms, but my first guess would be that a neural network that is learning on this data will be penalized while training. It predicts multiple chairs, but the training data has only one and my guess would be that when it predicts YES and the training data says it NO, when in reality it's YES, the network will be penalized for such predictions.
It much more a question than statement, as I am not sure.
What do you think, could be a problem?

I looked at the loss function of YOLO

And it seems like absence of bounding box for image when in reality it should be will affect the loss function on line 4. And it's interesting what was the reasoning behind making only one bounding box for openimages.

I am curious about this method also. @lamerman @samihaija any updates about this?

Does someone has an explanation concerning the fact that training data contains only one bounding box instead of all boxes ?
As a result, this dataset cannot be used to train object detection algorithm ?

because of this, Yolo does not train well with this dataset.

@dashesy hi, do you try any other detection algorithm ( like faster rcnn, ssd ) with openimages dataset? I try to train faster rcnn with mxnet with openimages, but I have many problems when preprocessing the dataset.

@dashesy hi, do you try any other detection algorithm ( like faster rcnn, ssd ) with openimages dataset? I try to train faster rcnn with mxnet with openimages, but I have many problems when preprocessing the dataset.

Further, a lot of images have missing labels. Is this problem fixable?


Migrate apps to CloudFormation scheduler

This is my initial thought on how we might migrate to the new CloudFormation based scheduler:

All existing apps will continue to use the old scheduler.
New apps will use the new scheduler.
When you want to migrate an app to the new scheduler, you set an environment variable.

TODO

[x] Delete from scheduler_migration when destroyed.
[x] Docs about upgrading Empire to the CloudFormation backend.
[x] Add existing apps to scheduler_migration table.

The process of switching over new apps to cloudformation looks great.
For migrating an existing app, my ideal would be:

bring up new stacks
test they're working (maybe with some temporary dns record?)
swap dns record from old stacks to new stacks
check to make sure everything is working again
remove old stacks

my hesitation around the 10 minute wait time as well as waiting to remove the old app before bringing up the new one is just issues i've had previously where you can't delete a resource because something else depends on it or bringing up a stack takes longer than you expect. i don't think the removal of something with a dependency is an issue if you assume empire has managed the whole process, but that would be my worst case scenario thinking.

Yep, agreed. I think making it a 3 step manual process like that would be ideal.

Ok, I added some tests for the different migration steps and scenarios and addressed all the comments. I'll probably hold off on merging this until I have a chance to test migrating a couple of apps in our staging environment to see if any issues pop up.

👍

Also, been thinking about how the scheduler changes will play out in new releases, and when to deprecate the old scheduler. My initial plan is in https://github.com/remind101/empire/wiki/Scheduler-Deprecation-Plan.

Discovered an issue with this when moving one of our larger apps that generates a (very) large CloudFormation template (~830kb), which goes ways over the 460kb limit when using s3 templates: http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/cloudformation-limits.html
So, this means for large stacks, we'll need to generate multiple templates, then combine them into a single stack using nested templates. I think that should be pretty easy to do, and shouldn't have too many downsides. We can just generate a separate stack file for each process, then submit a really small template to CloudFormation.

Actually, nested stacks may not work well since they actually create separate cloudformation stacks, which complicates how we return the running tasks for an app. Not impossible if we have to go down that route, but then it makes it easier to hit the cloudformation stack limit.
This is a pretty exception case, so maybe there's something that we can do to reduce the size of templates. Most of it is duplication in task definitions anyway.

Alright, the vast majority of the size was just because we were using json.MarshalIndent. Getting rid of that cuts the file size in about half. May be some more that can be done to minify templates further, but for the 99%, this will be fine I think.

Ok, used this to migrate 7 applications in our staging environment, and I'm pretty happy with the flow. For future reference, there's a Scheduler Migration Guide in the wiki that walks through the steps that this implements.


Switch to more efficient reducer

We have a (we think!) better version of the reducer in branch reducer_worked_well.  For the release we should make a judgement call about whether to switch to it.

I am benchmarking the new reducer compared with the old reducer.
Due to it being a hard and discontinuous problem it's still not clear to me which is better.

Fixed in #348


[CT-1248] dbt docs serve locks dbt.log file on windows

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Unders Windows 10 environment. Using dbt v1.2.1
run dbt docs serve as one process. Running dbt run as another process at the same time.
once the dbt.log file gets large enough to be moved to .1 this process fails as the file is locked by dbt docs serve and error messages are generated.
If I close the dbt docs serve process the error messages stop and dbt run continues per normal.
Expected Behavior
Under Windows dbt run should be able to execute an interact with log file regardless of whether dbt docs serve is running.
Note this is the behaviour on linux environment.
Steps To Reproduce

windows 10
dbt-core/snowflake (I've not tried with other target databases).
dbt run (possibly mulitple times until log file reaches size limit)

Relevant log output
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: '...\\logs\\dbt.log' -> '...\\logs\\dbt.log.1'

Environment
- OS: Windows 10
- Python: Python 3.9.10
- dbt:
Core:
  - installed: 1.2.1
  - latest:    1.2.1 - Up to date!

Plugins:
  - snowflake: 1.2.0 - Up to date!

Which database adapter are you using with dbt?
snowflake
Additional Context
No response

hi @jaredchurch, are you confident this is specific to Windows?
is this blocking any real use, or just something you noticed? I agree it's something we should fix but it's a bit unclear if this will affect normal usage given it requires running multiple dbt docs serve locally

Hi @lostmygithubaccount,
In regards to windows specific/other platforms, I also run dbt in a docker container on Ubuntu & do not face the same issue in that environment. I cannot comment on any other platform.
On the real usage question, it is only a single instance of dbt docs serve that I'm running when I observe this behaviour. My normal set up is to have dbt docs serve running as a background task, that way I don't have to restart it to see changes that I'm making

@jaredchurch got it, I think I misread. we work on Macs so this might take some effort to reproduce, but sounds like a real bug

If there is something more I can do to help with that I'm happy to.

It is quite common for CLI users to be running docs serve concurrently with other commands, in separate processes. That's not true for most CLI tasks.
It sounds like this error crops up when dbt.log gets too large, and dbt tries to cycle it (using Python's built-in RotatingFileHandler):
https://github.com/dbt-labs/dbt-core/blob/0e60fc1078cd337859ac78ce74a06e5b8bc2e431/core/dbt/events/functions.py#L81-L83
Potential resolutions?

Prevent dbt's logging system from locking dbt.log on Windows
Prevent file logging during the serve task — the task itself fires a few events, plus standard fare during initialization

I've found a work-around for this. The work-around requires at least dbt v1.2.
The work-around is to set $env:DBT_LOG_PATH in the environment where the docs server is to be run to a different path from what is used by other dbt processes.
This works by segregating the log file that the web server uses from the log file used by other processes.


Added support for storage infrastructure encryption

This change adds an additional column for storage accounts, indicating whether infrastructure encryption is enabled.
Microsoft Reviewers: Open in CodeFlow

Hello there,
Thanks for the PR, I just released the version 3.5, and since it changes a lot of the files, this PR has some conflicts.
Can you solve the conflicts so we can approve the PR?

Hello there,
Thanks for the PR, I just released the version 3.5, and since it changes a lot of the files, this PR has some conflicts.
Can you solve the conflicts so we can approve the PR?

Hi Claudio
I'll close this PR. I'm going to test and resubmit the changes with the new version 3.5.


[CMake] CMake Gardening Project: Part 4

This PR is yet another entry in the CMake Gardening Project (TM), which cleans up the SwiftWindowsSupport.cmake module to conform to proper CMake styling conventions and use the module import guard pattern (similar to header guards in C). This PR is a non-functional change.
Related to #16982
rdar://40764844

@swift-ci please test


Fix SSL Redis cache.

https://github.com/oddbird/Metecho/pull/1867

I'd like to merge this; could you first commit the suggestion that Peter made?

Done ✔


Make Target Settable in FeedbacksPlayerPlayFeedback

Hey! Me again. :D
I have a dynamic state machine transition where I'm basically running a Feedback for hiding one menu and then showing another. Rather than create feedbacks for every possible transition, I was hoping to create generic "HideInventory" and "ShowMap" transitions which play one after another in a feedback player play script. The problem I ran into was that FeedbacksPlayerPlayFeedback.Target is just a lambda to the private FeedbacksPlayerPlayFeedback.target field with no setter.
Obviously, that makes me cry. Haha.
Can we make the target field settable at runtime, either by making it public or switching Target to be a Getter/Setter?
For my local versions, I've turned it to a Getter/Setter since that seemed to be in line with how duration is being set:
public FeedbacksPlayer Target
{
	get => target;
	set => target = value;
}

Let me know if there's any reason this value shouldn't be changed.

That's actually a bug! For some reason I forgot to add the set to some feedbacks.
You can see how UIAlphaFeedback has the proper implementation D:

I will fix this, thanks for bringing up this big issue!

Closing this. Fixes are available on a new release.
Thanks again for the help :D 👍

The new release also contains a new FeedbacksPlayerKillAndResetFeedback, to reset target feedbacks.


Analyst conv 9, modelling with method of data collection for 4 months then predicting conversion in subsequent 2 months

No conflicts with base branch


Replace (int, float, complex) and permutations with numbers.Number

Instances of np.int64 are not instances of int, leading to unhandled cases. For instance,
import cirq
import numpy as np

a = np.arange(1)
print(isinstance(a[0], int))

q = cirq.LineQubit(0)
p = cirq.X(q)

print(0 * p)
print(a[0] * p)

gives
False
0j*X(0)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-17-72d730548027> in <module>
      9
     10 print(0 * p)
---> 11 print(a[0] * p)

TypeError: operand type(s) all returned NotImplemented from __array_ufunc__(<ufunc 'multiply'>, '__call__', 0, cirq.X.on(cirq.LineQubit(0))): 'int64', 'SingleQubitPauliStringGateOperation'

Urgh, this is probably all over the code base.

Indeed, it is a numbers.Number. That is the way to go then.

In many instances it may be more appropriate to check against numbers.Integral, numbers.Real, and numbers.Complex to verify that the code will behave as expected w.r.t. integer vs. floating point arithmetic or other similar concerns.

For the specific problem given by the example test, replacing our number checks with these other checks won't fix the problem. We also need to add cases for multiplication and for each other operation we want to support to the referenced method. Otherwise, numpy will have no idea what to do with our Cirq objects and the same error message will result. For these other operations, we will need to cast the numpy numbers into python numbers to avoid an infinite recursive loop. Adding support for numpy data types in dunder methods will have similar complexity anywhere else in the code base we want to include them. Similar complications also exists for sympy symbols that are numbers.
https://github.com/quantumlib/Cirq/blob/c572eb7fd98be8390858c0edf9a865a89e33940b/cirq/ops/pauli_string.py#L527-L531

Unfortunately numpy 1.20 type hints doesn't seem to like numbers.Complex as coercible to numpy array.


Enhancement to remote desktop

I think it would be helpful to have more functionality as far as mouse input goes.  Currently we are only able to click/double click/right click.  If we want to move a window to somewhere else on the screen we aren't able to drag the window to another location.  Also, if there are a cluster of files we are wanting to highlight, it becomes a difficult and overwhelming task.  This would also make a great addition to when LaPanthere finishes keyboard input :)
If this is something acceptable, I can have this added shortly.  (Already have the logic implemented in an older build)

Related Issues:
1. Issue 126
    - Keyboard input from the remote desktop and remote combinations handling.
2. Issue 112
    - Mouse input from the remote desktop to the client.

@MaxXor, Do you think this is acceptable?  I can implement this easily.
My idea behind this is to make the Remote Desktop feature similar to something like TeamViewer (as far as mouse input goes)

Yes, we should improve the Remote Desktop to make it more user friendly and bug-free. :)

Great :D I will improve the mouse interaction for now, and will likely focus on the keyboard part this upcoming week.

#299

Will also implement Delta handling (MWHEEL) :D


foreground service automatically stops after some time

https://stackoverflow.com/questions/37751823/how-to-use-firebase-eventlistener-as-a-background-service-in-android

https://stackoverflow.com/questions/37751823/how-to-use-firebase-eventlistener-as-a-background-service-in-android

I don't think this fits into our usecase. Here the discussion is about starting a background service based on the change in Firebase database.

https://medium.com/androiddevelopers/who-lives-and-who-dies-process-priorities-on-android-cb151f39044f

Using AlarmManager for checking 'if service is live or not'.
https://developer.android.com/reference/android/app/AlarmManager (Please look)


Overlay is in the way and should be closable

Most of the time there is an overlay in the upper right corner. Especially on smaller laptop screens or smaller browser windows it is in the way when coding and it should be closable:

We made a fix already and is deployed on our most current test version here
http://yann300.github.io/remix/
It will soon be deployed also on remix.ethereum.org
On Sat, Oct 14, 2017 at 9:48 AM, Sebastian C. Bürgel <
<EMAIL_ADDRESS>wrote:

Most of the time there is an overlay in the upper right corner. Especially
on smaller laptop screens or smaller browser windows it is in the way when
coding and it should be closable:
[image: screen shot 2017-10-14 at 10 45 27]
https://user-images.githubusercontent.com/11379673/31574307-1ae8d344-b0cd-11e7-8cac-a60f1677dfcb.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ethereum/remix/issues/593, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBsyyBPQLlzgC65rb7V1qsobNouXSP6ks5ssHVBgaJpZM4P5T3V
.

--
Nina Breznik
*Vision baker at *FairyDust.agency
T: @ninabreznik
M: +11<PHONE_NUMBER>
G: gitter.im/ninabreznik

Check out our work:
www.wizardamigos.com
www.refugeeswork.com
www.codingamigos.com
www.zweidesign.co
http://www.zweidesign.co


fix a few warnings

@loosebazooka

I'm in the middle of this code at the moment, this should be fixed by #502


Display problems - display overlap

check these 2
1: on the dashboard

2: on the multisig setting

I had the same display problem in multisig.

I have the same problem with the search field being covered up.
Before I created a multisig account, it is fine and looks normal.
But after I create a multisig account, there is another dropdown and not enough space to accommodate it.

@trestoJP should be fixed now. If yes, please close.

Closing.


build(codecov): add codecov configuration

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added/updated (for bugfix/feature)
[x] Docs have been added/updated (for bugfix/feature)

PR Type
What kind of change does this PR introduce?

[x] Feature (a new feature)
[ ] Bugfix (a bug fix)
[ ] Style (changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc))
[ ] Refactor (a code change that neither fixes a bug nor adds a feature)
[ ] Perf (a code change that improves performance)
[ ] Test (adding missing tests or correcting existing tests)
[x] Build (changes that affect the build system, CI configuration or external dependencies)
[ ] Chore (anything else), please describe:

What is the current behavior?

What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@vvalembois I will merge this one without approval.
I need it for the coverage I will add soon.
You can still check it post-merge to acknowledge.


How to prevent PDF from splitting over multiple pages using JSPDF

I am using JSPDF to generate PDF. I am passing html div reference to jspdf. My html contains text and tables. The text is coming fine but the table header is splitting to the second page if the table heading starts from the end of the page. Any help would be appreciated. Below is my code

`
function btnDownload_click()
{
	var doc = new jsPDF('p', 'mm', [340, 340]);
	var output="";

	var elementHandler = {
	  '#tobeIgnored1': function (element, renderer) {
		return true;
	  },

	  '#tobeIgnored2': function (element, renderer) {
		return true;
	  }

	};
	var source = document.querySelectorAll("#SecondaryStatements");
	for(var i=0 ; i<source.length;i++)
	{
		output = output + source[i].innerHTML;

	}
	var  disclaimerText = document.getElementById("Disclaimer").innerHTML;
	output= output + disclaimerText;
	var margins = { top: 30, bottom: 30, left: 40, width: 522 };
	doc.fromHTML(output,15,15,{'pagesplit': false,'width': 300,'elementHandlers':     elementHandler},function (dispose) {
	doc.save("SearchResults.pdf");},margins);

}`

We are closing this issue, because we will not support any longer fromHTML and addHTML.
Explaination:
We are working on a new html2pdf plugin, which will be based on html2canvas and our context2d plugin. This should lead to more reliable results for your projects. And it will give us the time to focus on the core functionality of pdf-generation because we will not use our energy for writing/supporting/extending 2 html plugins. If you still want to use addHTML or fromHTML you can still use jsPDF 1.4.1.
Best Regards


When connected via high latency over the Internet, tapping the screen becomes a long press behavior

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows
scrcpy version: 2.4
installation method: Windows release
device model:pixel 4 XL
Android version: 13

Describe the bug
When I controlled my device over the internet, tapping the screen turned into a long-press behavior, making it nearly unusable. The same applies even if parameters such as "max-size=720" and "video-bit-rate=1M" are added.
However, I don't have this behavior when I use my local LAN to control my device.
I also use rustdesk to remotely control my device through the Internet. There is a high delay, but the clicking behavior is normal.
The latency of connecting my device from the Internet is relatively high, with a "ping" of more than 200 milliseconds, and the actual performance is estimated to have a delay of 2 to 3 seconds. I don’t know if this phenomenon is caused by high latency. If so, is there any way to resolve it? Or will it be solved in the future?

tapping the screen becomes a long press behavior

The duration between input events is not guaranteed to be preserved, which can become visible under huge network jitter.
 The only way to avoid it would be to add buffering (i.e. additional latency on purpose) to compensate for jitter, the same way as the option --display-buffer (#2464) buffers video frames.
It is not plan to implement it for input events (scrcpy aims to mirror with high performance, so it assumes that the latency/jitter is low enough).


heroku blocked

heroku is blocked, am i able to use another website to deploy this?

replit or glitch are both free


Add Vertex AI support

Rename Generation#text to content and info to properties.
Make Generation extend the AbstractMessage and default to ASSISTANT message type.
Add vevertex-ai project with native api client for generation and embedding.
Add unit and IT tests for the vertex-ai native client.
Add AiClient and EmbeddingClient implementation for the the Vertex AI along with ITs.

Resolved by 69906db4f5cba8498e89713207ac5e901f4564b0 and  11e47c07ef7ef1cb2b4bd80e5565552dcf38d72a


How to change text color?

there is no api to change text color

Add the option to customize the chat bubble text styles by changing textStyle parameter (v0.8.0)


history state pushed twice

I am experiencing an issue where the history state is pushed twice in a single route when calling navigateAction. RouteStore handles both NAVIGATE_START and NAVIGATE_SUCCESS events by updating the store and calling emitChange. Since handleHistory listens to RouteStore, it's componentDidUpdate method ends up being called twice, once for each of those two events. There are no checks in componentDidUpdate to prevent the history state from being pushed twice.
Am I somehow misusing something here?
My proposed fix is to only push the history state when the navigation is complete and succeeds, probably by checking this.props.isNavigateComplete and this.props.currentNavigateError in componentDidUpdate. Does this sound like a reasonable strategy? I should note that many of handleHistory's tests will need to change since many of the tests test state after routeStore._handleNavigateStart is called.

We do check the url to see if it is the same before we call pushState/replaceState here: https://github.com/yahoo/fluxible-router/blob/master/lib/handleHistory.js#L158
Can you check to see why this check is not working for you?
The history change is expected to change immediately and not on completion of the action. The idea is that this is the same behavior that you get when you do a full HTTP load of a page: the url changes and then the actually HTTP call happens.

Ah, yes you are right - I missed that logic before. The behavior makes sense. In my case, nav.url was not url encoded and getUrl() was returning an encoded version during the second call.
Thanks for the help.


Update collectionHandlers.ts

This PR looks good to me, but it seems that #8558 made a very similar change, and that has been merged into main. Unless I'm missing something, I think this PR can now be closed.

Closing as #8558 has been merged.


Do we need support for a configurable autodiscovery.lock location?

We could easily do that by adding a config file with a sane default. Users of the package are able to publish the config file if they want to change the location. I can create a PR for you if you agree.

@dvdheiden that sounds great! I've created issue #23 for that.

will be implemented in #23


https://sonarwhal.com/scanner/ returns 500 result

Bug report:
After accessing https://sonarwhal.com/scanner I am getting 500 http status error:

The page cannot be displayed because an internal server error has occurred.

General headers:
Request URL: https://sonarwhal.com/scanner/
Request Method: GET
Status Code: 500 Internal Server Error
Remote Address: <IP_ADDRESS>:443
Referrer Policy: no-referrer-when-downgrade
Reposnse headers
Content-Length: 75
Content-Type: text/html
Date: Tue, 29 May 2018 13:02:20 GMT
Strict-Transport-Security: max-age=31536000
Request URL: https://sonarwhal.com/scanner/
Request Method: GET
Status Code: 500 Internal Server Error
Remote Address: <IP_ADDRESS>:443
Referrer Policy: no-referrer-when-downgrade
Content-Length: 75
Content-Type: text/html
Date: Tue, 29 May 2018 13:02:20 GMT
Strict-Transport-Security: max-age=31536000
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,/;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: en,cs;q=0.9,en-US;q=0.8
Cache-Control: max-age=0
Connection: keep-alive
Cookie: consent=on; _ga=GA1.2.836500726.1527598158; _gid=GA1.2.1077847205.1527598158; _gat=1
Host: sonarwhal.com
Referer: https://sonarwhal.com/
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.181 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,/;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: en,cs;q=0.9,en-US;q=0.8
Cache-Control: max-age=0
Connection: keep-alive
Cookie: consent=on; _ga=GA1.2.836500726.1527598158; _gid=GA1.2.1077847205.1527598158; _gat=1
Host: sonarwhal.com
Referer: https://sonarwhal.com/
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.181 Safari/537.36

@sarvaje ⬆

It looks like there's a problem with Azure. Some packages are not found and it fails to start. Trying to figure out why it broke and if it's in our side or Azure's side.

This is fixed now. There was a problem with the configured node version and a missing dependency (request) that for whatever reason was installed automatically previously


Error in Build with 2.59.0 SNAPSHOT: netscape\javascript\package-info;' cannot be represented in dex format

build.grade:
    dependencies {
        implementation 'net.sourceforge.htmlunit:htmlunit-android:2.59.0-SNAPSHOT'
        ...
    }

Error found under jetified-htmlunit-android-2.59.0-SNAPSHOT.jar:
    com.android.tools.r8.internal.r8: Class descriptor 'Lnetscape\javascript\package-info;' cannot be represented in dex format.

Running gradlew -q app:dependencies --configuration releaseRuntimeClasspath:
+--- net.sourceforge.htmlunit:htmlunit-android:2.59.0-SNAPSHOT
|    +--- xalan:xalan:2.7.2
|    |    \--- xalan:serializer:2.7.2
|    +--- org.apache.commons:commons-lang3:3.12.0
|    +--- org.apache.commons:commons-text:1.9
|    +--- commons-io:commons-io:2.10.0
|    +--- commons-logging:commons-logging:1.2
|    +--- commons-net:commons-net:3.8.0
|    +--- commons-codec:commons-codec:1.15
|    +--- org.brotli:dec:0.1.2
|    +--- com.shapesecurity:salvation2:3.0.0
|    \--- org.eclipse.jetty.websocket:websocket-client:9.4.45.v20220203
|         +--- org.eclipse.jetty:jetty-client:9.4.45.v20220203
|         |    +--- org.eclipse.jetty:jetty-http:9.4.45.v20220203
|         |    |    +--- org.eclipse.jetty:jetty-util:9.4.45.v20220203
|         |    |    \--- org.eclipse.jetty:jetty-io:9.4.45.v20220203
|         |    |         \--- org.eclipse.jetty:jetty-util:9.4.45.v20220203
|         |    \--- org.eclipse.jetty:jetty-io:9.4.45.v20220203 (*)
|         +--- org.eclipse.jetty:jetty-util:9.4.45.v20220203
|         +--- org.eclipse.jetty:jetty-io:9.4.45.v20220203 (*)
|         \--- org.eclipse.jetty.websocket:websocket-common:9.4.45.v20220203
|              +--- org.eclipse.jetty.websocket:websocket-api:9.4.45.v20220203
|              +--- org.eclipse.jetty:jetty-util:9.4.45.v20220203
|              \--- org.eclipse.jetty:jetty-io:9.4.45.v20220203 (*)

Earlier package-info issue: https://github.com/HtmlUnit/htmlunit/issues/133#issuecomment-1053674599

@sengsational
I directly download the jar package and import it
implementation files('libs/htmlunit-android-2.59.0.jar')
I also encountered this problem. How can I solve it

@sengsational
delete:
/javascript/package-info
/netscape/javascript/package-info
Now it has worked for me in Android studio
But I'm not sure if it works in practice

I try to add some content in pom.xml file
<exclude>netscape/package-info.class</exclude>
<exclude>netscape/javascript/package-info.class</exclude>

and delete lines 61~63
<exclude>commons-codec:commons-codec</exclude>
<exclude>commons-io:*</exclude>
<exclude>commons-logging:*</exclude>

Finally,  add in Android studio
implementation files('libs/htmlunit-android-2.59.0.jar')
implementation group: 'org.apache.commons', name: 'commons-lang3', version: '3.12.0'

Now it has worked for me in Android studio

@zerorooot @lqh995204185 Thank you for the suggestions.  I am unable to make progress.  I am afraid that because I do not have proper context to know where and how to make the suggested changes, I am getting a Transform's input file does not exist error.  Please advise me what I'm doing wrong and more detailed steps for me to make these changes.
Here are the steps I executed:

Locate htmlunit-android-2.59.0.jar (found in C:\Users\Owner\.gradle\caches\modules-2\files-2.1\net.sourceforge.htmlunit\htmlunit-android\2.59.0-SNAPSHOT\ed7b1a0371f6a0184a2cc63552086f0245295531\htmlunit-android-2.59.0.jar)

Rename  htmlunit-android-2.59.0.jar to  htmlunit-android-2.59.0.zip

Extract  htmlunit-android-2.59.0.zip to tmp

Edit 'tmp\htmlunit-android-2.59.0-SNAPSHOT\META-INF\maven\net.sourceforge.htmlunit\htmlunit-android\pom.xml'

Resulting in:

<build>
<plugins>
....
  <plugin>
    <artifactId>maven-shade-plugin</artifactId>
    <executions>
      <execution>
....
        </goals>
        <configuration>
          <artifactSet>
            <excludes>
              <exclude>org.apache.commons:*</exclude>
              <exclude>commons-net:*</exclude>
              <exclude>org.brotli:*</exclude>
              <exclude>com.shapesecurity:*</exclude>
              <exclude>xalan:*</exclude>
              <exclude>org.eclipse.jetty.websocket:*</exclude>
              <exclude>org.eclipse.jetty:*</exclude>
              <exclude>netscape/package-info.class</exclude>
              <exclude>netscape/javascript/package-info.class</exclude>
            </excludes>

Zip htmlunit-android-2.59.0.zip and rename to htmlunit-android-2.59.0.jar

Place the htmlunit-android-2.59.0.jar file in C:\Users\Owner\AndroidStudioProjects\MyApplication2\app\libs\ folder.

Edit build.gradle:

 dependencies {

    //implementation 'net.sourceforge.htmlunit:htmlunit-android:2.59.0-SNAPSHOT' (removed)
    implementation files('libs/htmlunit-android-2.59.0.jar')
    implementation group: 'org.apache.commons', name: 'commons-lang3', version: '3.12.0'

RESULTING ERROR:
Transform's input file does not exist: C:\Users\Owner\AndroidStudioProjects\MyApplication2\app\libs\htmlunit-android-2.59.0.jar. (See https://issuetracker.google.com/issues/158753935)

@zerorooot @lqh995204185 Thank you for the suggestions. I am unable to make progress. I am afraid that because I do not have proper context to know where and how to make the suggested changes, I am getting a Transform's input file does not exist error. Please advise me what I'm doing wrong and more detailed steps for me to make these changes.
Here are the steps I executed:

Locate htmlunit-android-2.59.0.jar (found in C:\Users\Owner\.gradle\caches\modules-2\files-2.1\net.sourceforge.htmlunit\htmlunit-android\2.59.0-SNAPSHOT\ed7b1a0371f6a0184a2cc63552086f0245295531\htmlunit-android-2.59.0.jar)
Rename  htmlunit-android-2.59.0.jar to  htmlunit-android-2.59.0.zip
Extract  htmlunit-android-2.59.0.zip to tmp
Edit 'tmp\htmlunit-android-2.59.0-SNAPSHOT\META-INF\maven\net.sourceforge.htmlunit\htmlunit-android\pom.xml'

Resulting in:

<build>
<plugins>
....
  <plugin>
    <artifactId>maven-shade-plugin</artifactId>
    <executions>
      <execution>
....
        </goals>
        <configuration>
          <artifactSet>
            <excludes>
              <exclude>org.apache.commons:*</exclude>
              <exclude>commons-net:*</exclude>
              <exclude>org.brotli:*</exclude>
              <exclude>com.shapesecurity:*</exclude>
              <exclude>xalan:*</exclude>
              <exclude>org.eclipse.jetty.websocket:*</exclude>
              <exclude>org.eclipse.jetty:*</exclude>
              <exclude>netscape/package-info.class</exclude>
              <exclude>netscape/javascript/package-info.class</exclude>
            </excludes>

Zip htmlunit-android-2.59.0.zip and rename to htmlunit-android-2.59.0.jar
Place the htmlunit-android-2.59.0.jar file in C:\Users\Owner\AndroidStudioProjects\MyApplication2\app\libs\ folder.
Edit build.gradle:

 dependencies {

    //implementation 'net.sourceforge.htmlunit:htmlunit-android:2.59.0-SNAPSHOT' (removed)
    implementation files('libs/htmlunit-android-2.59.0.jar')
    implementation group: 'org.apache.commons', name: 'commons-lang3', version: '3.12.0'

RESULTING ERROR:
Transform's input file does not exist: C:\Users\Owner\AndroidStudioProjects\MyApplication2\app\libs\htmlunit-android-2.59.0.jar. (See https://issuetracker.google.com/issues/158753935)

It needs to be recompiled according to the source code

there is an updated version of the compiler plugin, the latest snapshot (2.60.0-SNAPSHOT) is build using the fixed version. Please try and check if the problem is gone.

This version has resolved my problem.
Specifically, to build.gradle (:app), I simply added:
dependencies {
    implementation 'net.sourceforge.htmlunit:htmlunit-android:2.60.0-SNAPSHOT'
    ....

The above requires that settings.gradle (applicationName) include:
dependencyResolutionManagement {
    repositoriesMode.set(RepositoriesMode.PREFER_SETTINGS)
    repositories {
        ....
        maven { url 'https://oss.sonatype.org/content/repositories/snapshots'}
        ....

@rbri
When I exclude the package-info.class file, after importing and compiling, report errors
java.lang.NoClassDefFoundError

So i removed it.

@zerorooot can you please check with the latest snapshot build
And if you still het this classnotfound exception please include the name of the class from the exception

@rbri
Everything works fine with the latest  snapshot build, thank you very much

@zerorooot a release is planned for this weekend.
Thanks for the feedback.


Update TildeETranslation.java

Cahnged wrong http header name

@jnehring please merge also this one. This is for e-translation service.

merged


sceneform fd leak

when I load 3d model called this ModelRenderable.builder().setSource(context, resourceId) .build() method , I found the fd a lot of growth. Anybody see the same question？


fix: Mark deprecation on Channel & ChannelList modules

Add deprecation mark on Channel, ChannelProvider, ChannelList, and ChannelListProvider

Are there more places to add deprecation marks?

How about adding a migration guide link instead of providing an example code?

Looks good to me! Let's wait for Irene's review.


Add License and License file in the setup.cfg

Actually the License is not exported in the package.
https://files.pythonhosted.org/packages/77/da/a2448ccfe19d58c49a8bceda47dc960d8622f5dfa86ecf3b730ffa67b0e3/swagger_ui_bundle-0.0.9.tar.gz

Should be fixed in pyproject.toml.


Flaky test

https://github.com/opendifferentialprivacy/whitenoise-system/runs/1145105601
The above build failed  due to flakyness in test:
___ TestQuery.test_group_by_noisy_typed_order ___

Test flakyness for test_group_by_noisy_typed_order  has not been noticed for a few weeks now


Add public key pinning

Support for public key pinning is missing. curl has PINNEDPUBLICKEY.

Some more reference material: https://www.owasp.org/index.php/Certificate_and_Public_Key_Pinning
There’s an OpenSSL example in there. If given a pointer or two I’d be happy to try implement this.

Currently reqwest also does not work if a certificate store is not available on the system, even if the correct certificate has been added with .add_root_certificate() (or a public key is pinned). Running without a certificate store should probably also be supported as pinning methods get added.

any activity on this? I have the same prolem...

Doesn't #1150 fix this?
using add_root_certificate and tls_built_in_root_certs should create the desired behavior unless I misunderstood what the issue is about.

Hi, any news here?

Doesn't #1150 fix this? using add_root_certificate and tls_built_in_root_certs should create the desired behavior unless I misunderstood what the issue is about.

No, add_root_certificate is much more strict because it only allows adding a root certificate. I just tried to put only the leaf certificate there, and the connection got rejected.
Furthermore, I would like to supply the sha256 of the leaf certificate instead of providing the full certificate (chain)


 But they support suppression comment like // parasoft-begin-suppress and // parasoft-end-suppress.
cf. https://docs.parasoft.com/display/JTEST1040/Suppressing+the+Reporting+of+Findings

Wouldn't it make more sense to ask Parasoft JTest to allow for configurable Generated annotations?
It is possible to instruct delombok to insert @javax.annotation.Generated.

We have already report this problem to Parasoft in Aug 2017. But unfortunately, nothing has changed for now.

Hmm, sorry. We've never heard of this tool before. So far, all other static code analyzers either respect @javax.annotation.Generated, or have configurable suppression annotations.


Enable automatic parallel execution in pre-compiled code

Feature request

Follow up from this stack overflow querestion it seems that the pycc compiler is not able to recognize the method numba.prange and execute the code as it was a standard range method.
It will be great to have pycc able to recvognize numba.prange so to be able to execute  computations in parallel from a pre-compiled numba code.
A reproducible example:

File: numba_ray_tracing.py

from numba import jit, njit
from numba.pycc import CC
import numba
cc = CC('nbspatial')

import numpy as np

@cc.export('array_tracing', 'b1[:](f8[:,:], f8[:,:])')
@njit(parallel=True)
def array_tracing(xy, poly):
    D = np.empty(len(xy), dtype=numba.boolean)
    n = len(poly)
    for i in numba.prange(1, len(D) - 1):
        inside = False
        p2x = 0.0
        p2y = 0.0
        xints = 0.0
        p1x,p1y = poly[0]
        x = xy[i][0]
        y = xy[i][1]
        for i in range(n+1):
            p2x,p2y = poly[i % n]
            if y > min(p1y,p2y):
                if y <= max(p1y,p2y):
                    if x <= max(p1x,p2x):
                        if p1y != p2y:
                            xints = (y-p1y)*(p2x-p1x)/(p2y-p1y)+p1x
                        if p1x == p2x or x <= xints:
                            inside = not inside
            p1x,p1y = p2x,p2y
        D[i] = inside
    return D

if __name__ == "__main__":
    cc.compile()

Compile: python3.7 numba_ray_tracing.py

Test

import nbspatial
import numpy as np
# regular polygon for testing
lenpoly = 10000
polygon = np.array([[np.sin(x)+0.5,np.cos(x)+0.5] for x in np.linspace(0,2*np.pi,lenpoly)[:-1]])

# random points set of points to test
N = 1000000
points = np.array([np.random.random(N), np.random.random(N)]).reshape(N,2)

%%time
nbspatial.array_tracing(points, polygon)

CPU times: user 1min 31s, sys: 512 ms, total: 1min 32s
Wall time: 1min 31s

The code above doesn't run in parallel. The same code runs in parallel if used in an interactive python session.
Example code

Thanks for the report. At present (Numba 0.40.0) I do not think that the parallel CPU target is supported by pycc/AOT compilation, which would explain what you observe. Adding as a feature request.

Thanks for considering it!
I'm using numba from git:master branch. I will monitor this issue #3336 for any update. And possible testing.

Any plan to add this feature for next release?

HI! will this be part of the 2020 roadmap - perhaps fits in the """ Improve ahead of time compilation (for low powered devices) """ topic :) - Thanks for any update!

Do we have any update on this feature ?

@jdenos No updates I'm afraid.

From discussion in today's triage meeting, related to #7696: this is not likely to be supported as AOT code doesn't require Numba to be installed - this would mean a great deal of work and issues to overcome for packaging the code for the threading layers.

Would the devs expect the PIXIE work to make this more achievable?


Option for not displaying newly added cards immediately

We as a team like a short brainstorming phase at the start of the meeting where we are not influenced by the others.
So it would be great being able to start with the cards "turned around", i.e. with the content not visible. Then, after some time, the cards could be "turned around" and then consolidation and discussion could start.
This could be handled as an (optional) seperate phase, like Phase 0: "brainstorm"

HI @michaelamaura, we did consider this option for our release of a new tool version in https://github.com/inovex/scrumlr.io/projects/3


No mention of Unridable dragons

Attempting to mount a dragon while under the effects of Reduce Person to be small enough yields the error "Target Has Feature Not Available". This was tested with a Claw of the False Wyrm/Cavalier multiclass.
Also, barding is unavailable to select, but weapon proficiencies are.

I disabled mounting as there are no animations so the unit glitch into each other and cause problems, I will update the FAQ on nexusmods to reflect this.
Barding is disabled on purpose as drakes cannot wear barding per the tabletop rules.

Important correction; They can wear barding, but they won't. This is important because barding absolutely can be made to fit a drake or small dragon, but they will never wear it.


Add Phoenix to Model Monitoring

Add Phoenix to Model Monitoring

Hi @nate-mar. Thanks for your PR, much appreciated.
Something we've been deliberate about in this repo is only including projects whose license is approved by the Open Source Institute. Because 'open source' can mean lots of different things to different people, we feel that this was a good way to draw a line. Unfortunately, the Elastic license is not OSI-approved, so that means we aren't able to include Phoenix.

All good! Thanks for the follow up!


react 17 update

Dependency and jest setup update for react 17.
Uses unofficial @wojtekmaj/enzyme-adapter-react-17
https://github.com/VictorCazanave/react-svg-map/issues/53

[x] Fork the repository
[x] Develop your code changes
[x] Ensure eslint is happy: npm run lint
[x] Ensure the tests pass: npm test
[x] Commit your changes
[x] Push to your fork
[x] Submit a pull request

Thanks for your contribution @RamonDonnell ! 😃
Does it solve the issue https://github.com/VictorCazanave/react-svg-map/issues/53?
If yes, since the components aren't modified, can't we just update the peerDependencies to support both React 16 and 17 (^16.0.0 || ^17.0.0)? 🤔

Please resolve and merge this pull request.
I'd like to use react-svg-map with react-17, and I'm sure many other people would too.

Yes, other people too ;p

As mentioned in https://github.com/VictorCazanave/react-svg-map/issues/53, I didn't see any issue using these components with React 17.
So @sdnalloh @piotrgregor what do you need exactly? Only updating the peer dependencies to also support React 17?
If there is an issue with React 17, please provide a repository with a minimal example to reproduce it.

Thanks again for your contribution @RamonDonnell, but stopping React 16 support is a major change that will be done in the v3


Expert review: add comments to connective tissue terms

spreadsheet
ROBOT merge template ROBOT Reviewer-Curator Agree - Leave:
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vQwbBXJPMisOhwx9Pbr1ekXVxF_Q7WvfnJUNycgTmPwBW_FjksvYsyoTAwWwBflpQ/pub?gid=298440223&single=true&output=tsv" merge_template -B
ROBOT merge template ROBOT Reviewer-Leave Curator-Stay:
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vQwbBXJPMisOhwx9Pbr1ekXVxF_Q7WvfnJUNycgTmPwBW_FjksvYsyoTAwWwBflpQ/pub?gid=1295833230&single=true&output=tsv" merge_template -B

@nicolevasilevsky
The following to do is missing

[ ]  remove parents that are in the connective tissue disease branch ("Reviewer-Leave Curator-Stay" Tab)

Otherwise, the comments have been added correctly.
Thank you.

got it.
These terms are classified as 'connective tissue disorders' because of the parent 'autoinflammatory syndrome'.
I can exclude this parent from these terms (and in some cases, the terms will need to be reclassified) but I am not sure if this is the best approach, for example, MONDO:0957405	granulomatous autoinflammatory syndrome of childhood seems to make sense as a child of 'autoinflammatory syndrome'.
Rather, i wonder if we should reconsider the classification of 'autoinflammatory syndrome'. It is a child of 'rheumatic disorder', which is a child of 'connective tissue disorder'. I wonder if rheumatic disorder should be a child of 'connective tissue disorder'? This comes from DO and MESH.
The CDC definition.) of rheumatic disorder is more broad:
Rheumatic disease is an umbrella term that refers to arthritis and several other conditions that affect the joints, tendons, muscle, ligaments, bones, and muscles (arthritis refers to disorders that mainly affect the joints).
Or maybe 'autoinflammatory syndrome' should not be a child of 'rheumatic disorder'? This comes from Orphanet.

Addresses #6742

I didn't realize that 'autoinflammatory syndrome' is considered a 'connective tissue disorder'.
I agree, we should review the definition and classification of 'rheumatic disorder' instead, and this will solve this issue.
Let's create a new GH issue to review this, and disregard the removal of the parents for now.
Thank you for catching this!
auto


Cannot abort starting a new session

When the user clicks the Start Session button on the New Session screen, the Load Order window is displayed. There is no way for the user to cancel out of this window and abort starting a new session.

This is intended behavior.


Carousel: stop cycling if there are other carousels in page and are not visible

Say, one has 3 carousels in the same page with autoplay on. An example would be 3 tabs with a carousel in each tab.
We shouldn't cycle any carousel that is not currently visible.
/CC @Johann-S @mdo @bardiharborow for thoughts

Definitely need to mention it in the release notes.
BTW I haven't been adding the PRs I merge in the release notes. I will from now on, I promise :)
But if we could add everything that's missing that would help.


WPT: Replace invalid characters in test names

Remember kids, just say no to byte-order marks and surrogate pairs.

Internal PR passes


Make pip wheel with Dakota+Carolina

Resolves #27

Could you also change the README with updated install instructions? Currently on PyPi it looks a bit weird since the install instructions are essentially the "Development install instructions". Could you split it into two sections one installation where its essentially pip install carolina and another section with "developing" instructions? (similar to  ert) :slightly_smiling_face:


Make the inductive is_sorted the default

We currently have three different definitions of is_sorted. Two of those are essentially the same (is_sorted' and is_sorted''), but there is an important difference between those two and the third (is_sorted). See https://github.com/awalterschulze/regex-reexamined-coq/blob/36de430013b663b142db04360512751e7be5bf26/sort.v#L10-L39
Our main definition (the one called is_sorted, without any apostrophes), is a match-statement, where at the end of all the matches, you get either True or False. I think it is probably better to take one of the inductive definition as our standard definition, for the following reason: with the current definition of is_sorted, if you have a xs: List X and you also have  a hypothesis H : is_sorted xs, that does not actually mean that xs is sorted, because is_sorted xs could just as well be false. So I think it's more natural to take the inductive definition, and I also think it's probably easier to prove things with that definition.
For example, I find it quite difficult (or at least more difficult than it should be) to prove the following lemma
Lemma is_sorted_sufficient_cond {X: Set} {tc: comparable X} (x x' : X) (xs: list X):
  ((compare x x' = Lt) \/ (compare x x' = Eq))
  -> is_sorted (x'::xs)
  -> is_sorted (x::x'::xs).

What do you guys think?

Very happy to switch between which function has the tick and which doesn't.
And also happy to switch between which version we prove things about.
Please do what unblocks you and explain why in the pull request.
If others disagree and think it is easier to prove the old way, then that is a challenge for them to prove and us to learn from.  Either way we both win :)

We could prove that all the definitions are the same and then we can use
them interchangeably.
On Sun, 12 Apr 2020 at 13:33, Walter Schulze<EMAIL_ADDRESS>wrote:

Very happy to switch between which function has the tick and which doesn't.
And also happy to switch between which version we prove things about.
Please do what unblocks you and explain why in the pull request.
If others disagree and think it is easier to prove the old way, then that
is a challenge for them to prove and us to learn from. Either way we both
win :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awalterschulze/regex-reexamined-coq/issues/36#issuecomment-612607532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAWVTMHMPF7WUBFXG42R5TRMGYKZANCNFSM4MGL72QQ
.

I think this has been answered and can be closed?  Please reopen if I missed something.


Postgres cache issue

import knex from "knex";
import { LoggerFactory, SmartWeaveNodeFactory } from "redstone-smartweave";
import Client from "../utils/init";
import fs from 'fs'

const knexConfig = knex({
    client: 'pg',
    connection: 'postgres://redstone:redstone@localhost:5432/smartweave',
    searchPath: ['knex', 'public'],
    useNullAsDefault: true
})

const wallet = JSON.parse(fs.readFileSync("./wallet.json") as unknown as string);

async function KnexCached() {
    const smartweave = await SmartWeaveNodeFactory.knexCached(Client, knexConfig)
    LoggerFactory.INST.logLevel("debug", "Read contract interactions")

    const contractTxId = "-8A6RexFkpfWwuyVO98wzSFZh0d6VJuI-buTJvlwOJQ"
    const contract = smartweave.contract(contractTxId).connect(wallet)
    const { state, validity } = await contract.readState()

    console.log(state);
    console.log(validity);
}

KnexCached().catch(e => console.log(e))

I'm trying to read contract state using knex, postgres. I'm getting below error Can you please tell anything wrong in code or setup?
warning ../package.json: No license field
$ ts-node -r dotenv/config ./scripts/redstone/read.knex.cache.ts
error: select "contract_id", "height", "state", max("height") from "states" group by "contract_id" order by "height" desc - column "states.height" must appear in the GROUP BY clause or be used in an aggregate function
    at Parser.parseErrorMessage (ar-contracts/node_modules/pg-protocol/src/parser.ts:369:69)
    at Parser.handlePacket (ar-contracts/node_modules/pg-protocol/src/parser.ts:188:21)
    at Parser.parse (ar-contracts/node_modules/pg-protocol/src/parser.ts:103:30)
    at Socket.<anonymous> (ar-contracts/node_modules/pg-protocol/src/index.ts:7:48)
    at Socket.emit (node:events:390:28)
    at Socket.emit (node:domain:475:12)
    at addChunk (node:internal/streams/readable:315:12)
    at readableAddChunk (node:internal/streams/readable:289:9)
    at Socket.Readable.push (node:internal/streams/readable:228:10)
    at TCP.onStreamRead (node:internal/stream_base_commons:199:23) {
  length: 176,
  severity: 'ERROR',
  code: '42803',
  detail: undefined,
  hint: undefined,
  position: '23',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'parse_agg.c',
  line: '1419',
  routine: 'check_ungrouped_columns_walker'
}

duplicates https://github.com/redstone-finance/redstone-smartcontracts/issues/78


Get-DbaDbSpace command is returning all DBs including all system database

Before submitting a bug report:

[ ] Ensure you are able to reproduce it on the latest released version (we release often)
[ ] Verified this bug is not already reported in an issue
[ ] Verified errors are not related to permissions
[ ] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce
/*
    Any T-SQL commands involved or used to produce test objects/data.
*/

<#
    Provide exact (or sanitized) code to reproduce the error
#>

Expected Behavior

Actual Behavior

Environmental data

PowerShell:

SQL Server:

Per the message output when using the IncludeSystemDBs switch, this appears to be expected behavior; if filtering is needed, the list of DBs should be filtered before passing into this function.
Absent any other information, I don't think this is a bug. @TechieFriend, can you please update this issue with more information?

Sure, let me test and update you.
Thanks,
Pinakin Patel

On Feb 24, 2019, at 2:12 PM, Andy Levy<EMAIL_ADDRESS>wrote:
Per the message output when using the IncludeSystemDBs switch, this appears to be expected behavior; if filtering is needed, the list of DBs should be filtered before passing into this function.
Absent any other information, I don't think this is a bug. @TechieFriend, can you please update this issue with more information?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[Diagnostics] A couple of adjustments to FailureDiagnostic

Remove HasComplexLocator which is no longer useful
Rename getChoiceFor to getCalleeOverloadChoiceIfAvailable to make it consistent
Remove unsafe getRestrictionFor method
Remove obsolete getResolvedMemberRef

@swift-ci please smoke test

@swift-ci please smoke test Linux platform

@swift-ci please smoke test Linux platform


Restricting when the umbrella cover runs

First, swapping the cover envs. in the umbrella tox config:
isolated-cover --> cover
         cover --> umbrella-cover

and dropping the (now redunant) tox -e cover from .travis.yml. Finally, wrapping the after_success: Travis call to tox -e coveralls in a check to make sure it only runs on push builds (to speed up PR builds).
NOTE: This means that Coveralls will no longer be a check on PRs.

This looks good to me pending travis.

Unfortunately the build still took 8 minutes because just installing the lint environment took 80s (namespace packages!) and similarly the do-nothing system test envs took 45s and 70s. Finally the docs build took 200s.
Will try to get these down. I know Sphinx has support for incremental builds to it's worth taking a look. (I also know that tox disables Sphinx caching so we'd want a special hack there.)


New version: PeterPawlowski.foobar2000 version 2.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Shouldn't packgeVersion not have quotes? For some reason, maybe it's that, Winget is now picking up my foobar 2.1 as foobar 1.6.16


ZstdCompressCtx setLevel(int level) not working as intended

Hi @luben
The  ZstdCompressCtx setLevel(int level) API is not working
i set compression level using below API
cctx.setLevel(this.compressionLevel)
I load dictionary using cctx.loadDict()
then I use cctx.compressByteArray() API for compression.
I observed compression ratios, and compression speed to be the same even if I vary compression Levels as 1, 6, 12, and 22 which is weird.
Please help!

Thanks for your quick response! That helps!


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

closed on 7-2-23


EXILE MOD

I'm getting this error right before I spawn..
Script ExAdClient\XM8\Apps\CHUD\OnClose.sqf Not Found.
After I hit ok I can't use my map or XM8

CHUD or CHVD ?

CHVD

Check if you have that file in your mission PBO on that path or not.

Most likely wrongly installed


Find and save the distro version when extracting from debian and alpine

Previously, we ignored the distro version when querying for vulnerabilities. This adds in parsing for distro version.
To enable this for alpine, which doesn't encode their release version in their apk-installed file, I also implemented Open() function for image DepFile, and for alpine, open the /etc/os-version file to read the alpine version.
This hid the previous vulnerabilities that were showing up for alpine 3.10, since alpine 3.10 is no longer supported, the advisories weren't being published for those packages, even though those vulnerabilities probably do apply.

It is quite difficult to find packages that are actually vulnerable and haven't been updated in alpine, and older packages are removed from the repository, so we can't really test the positive case where a vulnerability is found.

Codecov Report
Attention: Patch coverage is 82.75862% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 61.25%. Comparing base (b28c1c8) to head (633de4e).
Report is 24 commits behind head on main.

Files
Patch %
Lines

pkg/lockfile/apk-installed.go
79.31%
3 Missing and 3 partials :warning:

internal/image/extractor.go
70.00%
2 Missing and 1 partial :warning:

internal/image/image.go
80.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #854      +/-   ##
==========================================
+ Coverage   59.78%   61.25%   +1.47%
==========================================
  Files         136      144       +8
  Lines       11268     9192    -2076
==========================================
- Hits         6737     5631    -1106
+ Misses       4102     3083    -1019
- Partials      429      478      +49

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

It is quite difficult to find packages that are actually vulnerable and haven't been updated in alpine, and older packages are removed from the repository, so we can't really test the positive case where a vulnerability is found.

I've not looked at the changes yet, but I don't follow this - why can't we just craft a alpine lockfile that lists a vulnerable package? even if we can't get alpine to generate one, I would think we know enough about the format to be able to hack a fake one together which'd be better than having no test at all...

Yeah we definitely can, I'll add a crafted test image that shows some vulns. I'm mainly referring to the difficulty of looking for a natural example to make sure we aren't missing anything.


GIF support?

First of all: thanks for a great library!
Would it be possible to add support for GIF image format? Or would this have a significant impact on the size of the .wasm file?
What's the reason GIF support is currently not included?

Looking at the configure script for imagemagick (https://imagemagick.org/script/advanced-linux-installation.php), I don't see a flag for GIF support specifically so maybe GIF support is builtin. Have you tried reading/writing a GIF image with this library?

Just realized there is a test for writing GIFs: https://github.com/jsscheller/imagemagick-wasm/blob/master/tests/index.js#L57. There isn't one for reading but I assume it works as well.

Thanks for your responses. I've done some more testing and I can indeed confirm that converting to GIF works. But it is incredibly slow. Converting a JPG image of 1024x768px to PNG or WebP takes 0.6 seconds on my Mac (in Chrome), converting it to GIF takes over 55 seconds. On Firefox it's less than 1 second vs 40 seconds.
If the source image gets larger, the differences get even bigger.
Here is my test: http://2img.webgear.nl/gif.html
worker.js
import createModule from<EMAIL_ADDRESS>
const getTime = () => {
   return new Date().getTime()/1000;
}

(async () => {
   let buf, blob, startTime;

   const magick = await createModule({
      // Tell Emscripten where the WASM file is located.
      locateFile: () =><EMAIL_ADDRESS>   });

   magick.FS.createLazyFile('/', 'koala.jpg', '/pics/koala.jpg', true, false);

   //
   // Convert a JPG image to PNG (fast)
   //
   console.log('convert koala.jpg -resize 50% -colorspace gray out.png');
   startTime = getTime();

   magick.callMain(['koala.jpg', '-resize', '50%', '-colorspace', 'gray', 'out.png']);

   buf = magick.FS.readFile('out.png');
   blob = new Blob([buf], { type: 'image/png' });
   postMessage({text:'PNG (' + (getTime() - startTime).toFixed(2) + 's)', blob:blob});
   console.log('Done JPG -> PNG!');

   //
   // Convert a JPG image to GIF (very slow)
   //
   console.log('convert koala.jpg -resize 50% -colorspace gray out.gif');
   startTime = getTime();
   magick.callMain(['koala.jpg', '-resize', '50%', '-colorspace', 'gray', 'out.gif']);

   buf = magick.FS.readFile('out.gif');
   blob = new Blob([buf], { type: 'image/gif' });
   postMessage({text:'GIF (' + (getTime() - startTime).toFixed(2) + 's)', blob:blob});
   console.log('Done JPG -> GIF!');
})();

Why is converting to GIF so slow?

Interesting. What happens when you just do the JPG to GIF conversion (or do that conversion first)? Some WASM engines will try to optimize the WASM if it is used more than once - wondering if that is the issue here.
If that isn't the issue I would guess the GIF encoder is just slow. Perhaps there is some command line argument which makes it faster?


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #31920

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Thanks for accepting