IStructuralEquatable on a non collection type causes schema generation to fail

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Inheriting System.Collections.IStructuralEquatable outside of a collection causes schema generation to fail.
Expected result would be the schema generated with the offending equality method ignored.
A rarer case in C# but F# records all implement this interface, see #2143 as a less specific duplicate. Adding the ReferenceEquality, or NoComparison + NoEquality attribute to a record results in the interface being omitted from the compiled code, hence the attributes being a workaround. Resolving this should indirectly add support for F# records (and ease further support).
Steps to reproduce
Simplest reproduction (anywhere in the schema)
public class Example : IStructuralEquatable
{
    public bool Equals(object? other, IEqualityComparer comparer) => throw new NotImplementedException();

    public int GetHashCode(IEqualityComparer comparer) => throw new NotImplementedException();
}

Alternate, using F#'s option type
using Microsoft.FSharp.Core;
public class Example
{
    public FSharpOption<int> Foo { get; set; }
}

F# example
type Example = {Foo: string}
// basically equivalent to
public sealed class Example : IEquatable<Example>, IStructuralEquatable, IComparable<Example>, IComparable, IStructuralComparable { ... }

Relevant log output
An unhandled exception has occurred while executing the request.
      HotChocolate.SchemaException: For more details look at the `Errors` property.

      1. Unable to infer or resolve a schema type from the type reference `Input: IEqualityComparer`.

         at HotChocolate.Configuration.TypeInitializer.DiscoverTypes()
         at HotChocolate.Configuration.TypeInitializer.Initialize()
         at HotChocolate.SchemaBuilder.Setup.InitializeTypes(SchemaBuilder builder, IDescriptorContext context, IReadOnlyList`1 types, LazySchema lazySchema)
         at HotChocolate.SchemaBuilder.Setup.Create(SchemaBuilder builder, LazySchema lazySchema, IDescriptorContext context)
         at HotChocolate.SchemaBuilder.Create(IDescriptorContext context)
         at HotChocolate.SchemaBuilder.HotChocolate.ISchemaBuilder.Create(IDescriptorContext context)
         at HotChocolate.Execution.RequestExecutorResolver.CreateSchemaAsync(NameString schemaName, RequestExecutorSetup options, RequestExecutorOptions executorOptions, IServiceProvider serviceProvider, TypeModuleChangeMonitor typeModuleChangeMonitor, CancellationToken cancellationToken)
         at HotChocolate.Execution.RequestExecutorResolver.CreateSchemaServicesAsync(NameString schemaName, RequestExecutorSetup options, CancellationToken cancellationToken)
         at HotChocolate.Execution.RequestExecutorResolver.GetRequestExecutorNoLockAsync(NameString schemaName, CancellationToken cancellationToken)
         at HotChocolate.Execution.RequestExecutorResolver.GetRequestExecutorAsync(NameString schemaName, CancellationToken cancellationToken)
         at HotChocolate.Execution.RequestExecutorProxy.GetRequestExecutorAsync(CancellationToken cancellationToken)
         at HotChocolate.AspNetCore.HttpPostMiddlewareBase.HandleRequestAsync(HttpContext context, AllowedContentType contentType)
         at HotChocolate.AspNetCore.HttpPostMiddlewareBase.InvokeAsync(HttpContext context)
         at Microsoft.AspNetCore.Builder.EndpointRouteBuilderExtensions.<>c__DisplayClass20_0.<<UseCancellation>b__1>d.MoveNext()
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
         at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

Additional Context?
No response
Product
Hot Chocolate
Version
12.10.0, 13.0.0-preview.24

I think this should not be the problem. The thing here is that the interface somehow was inferred. We actually only do that if we find a type exposing this interface on a prop

Can you create a small repro?

I am also open for creating an F# unit test project so that we can better cater to the F# community.

Here you go.
I'll take a look at the tasks outlined in #3670 and see if there's anything I can contribute to.

Cleaning some things up and Tested again and it's been resolved along the lines.


Content changes to be deployed on 1 Dec 21

Context
https://trello.com/c/AaqUSKcu
Changes proposed in this pull request
Guidance to review

 (DO NOT MERGE) Removal of content on 2 guidance pages for 1 December.


Having issues with proxy tags

Hi. When i try the example in the docs, I get this error:

(ack, the perils of pressing enter too soon- please hold on a sec)

Should be fixed now in v1.1.5 :slightly_smiling_face:
pip install -U pluralkit


No such file or directory: 'MarkupSafe-2.0.1.dist-info\METADATA'

Discussed in https://github.com/asreview/asreview/discussions/1512

Originally posted by JohnQuentin August 15, 2023
Hello,
After several weeks of not using ASReview, I attempted to restart it this morning as I require some data for my project. Upon startup, I encountered the following notification, as depicted below. I have already attempted to update ASReview, which was successful. However, I am still encountering this error. Could someone provide further assistance in resolving this issue?
(base) C:\Users\johnv>asreview lab
2023-08-15 10:02:54.288858: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'cudart64_110.dll'; dlerror: cudart64_110.dll not found
2023-08-15 10:02:54.289201: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Traceback (most recent call last):
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 3108, in _dep_map
    return self.__dep_map
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 2901, in __getattr__
    raise AttributeError(attr)
AttributeError: _DistInfoDistribution__dep_map

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 3099, in _parsed_pkg_info
    return self._pkg_info
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 2901, in __getattr__
    raise AttributeError(attr)
AttributeError: _pkg_info

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\johnv\anaconda3\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\johnv\anaconda3\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\johnv\anaconda3\Scripts\asreview.exe\__main__.py", line 7, in <module>
  File "C:\Users\johnv\anaconda3\lib\site-packages\asreview\__main__.py", line 48, in main
    entry.load()().execute(sys.argv[2:])
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 2516, in load
    self.require(*args, **kwargs)
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 2539, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 832, in resolve
    new_requirements = dist.requires(req.extras)[::-1]
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 2821, in requires
    dm = self._dep_map
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 3110, in _dep_map
    self.__dep_map = self._compute_dependencies()
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 3119, in _compute_dependencies
    for req in self._parsed_pkg_info.get_all('Requires-Dist') or []:
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 3101, in _parsed_pkg_info
    metadata = self.get_metadata(self.PKG_INFO)
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 1517, in get_metadata
    value = self._get(path)
  File "C:\Users\johnv\anaconda3\lib\site-packages\pkg_resources\__init__.py", line 1726, in _get
    with open(path, 'rb') as stream:
FileNotFoundError: [Errno 2] No such file or directory: 'c:\\users\\johnv\\anaconda3\\lib\\site-packages\\MarkupSafe-2.0.1.dist-info\\METADATA'

(base) C:\Users\johnv>
```</div>

Feel free to reopen or comment if this issue happens to you.


update from Node12 to Node16

The GitHub Actions workflow gives the following annotation while running the action.

[action_commit]
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>
So we need to update Node version from 12 to 16.

@jasondantuma
@nilzona
@epmallmann
Can we please get this merged?


Try/except blocks not working in numba

I just found out that try/except blocks don't work because SETUP_EXCEPT is not in the list of supported opcodes:
http://expertsoverflow.com/questions/25821596/raising-an-exception-while-using-numba
Could this be added please?

Closing this as duplicate of #2097


Fuelphp

How about it to make  a axon library integration (package) for the Fuelphp 1.8/ 1.9 dev framework ?

Hi,
There is no plan for a integration package for Fuelphp in our current milestone.
May be we could add it in the next one, but it will not be released before 6 to 8 weeks.
Meanwhile, you can use the Armada package (https://github.com/jaxon-php/jaxon-armada) which provides features that are nearly the same as in the integration packages.
For instance, you will be able to reuse the same Jaxon classes when migrating to the integration package.

Thanks for the answer.

You're welcome, @stvowi.
I am available to provide some help, in the case you want to create the plugin by youself.


Fix typos

Types

[x] 💄 Code Style Changes

Background or solution
Changelog
Fix typos

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Check for valid instance obj reference

By design ge_tts isn't supposed to provide any runtime type errors. Instead that's what the static type checking in Luanalysis is for.
Do you have an example of this being an issue in practice? If you're running into this then (casts aside) there's probably an issue with ge_tts's types (or perhaps Luanalysis itself) that ought to be addressed.

By design ge_tts isn't supposed to provide any runtime type errors. Instead that's what the static type checking in Luanalysis is for.
Do you have an example of this being an issue in practice? If you're running into this then (casts aside) there's probably an issue with ge_tts's types (or perhaps Luanalysis itself) that ought to be addressed.

Thanks

Thanks


Better error handling for fetch

@grabbou mentioned this in #592

PS. That maybeValidationError is a weird naming, should be invalidFieldName or something imo.

Anyway, this needs more work, because throwing throw response is not a good way to do the things. Plus here https://github.com/este/este/blob/master/src/common/auth/actions.js#L41-L42 you once throw response and once validation error. The type thrown should be unified and always be error or an object of known type.

Because there's no unified api service, we got 3 different cases:

no internet - error from fetch is thrown
bad data - 401 - error from validate
status code != 201 - response
Three different types. There should be unified api service that composes one, unified error when these happens.

And we need tests for all the 3 scenarios.

Since we switched to Babel 6 and fixed related errors, we can implement this issue.

@grabbou ?

I think it's over Este scope.


Create 0000-01-02-jbbfirstpull.md

committing per instructions in the GitHub training.
created a branch, created a file and made a commit, and opened a pull  request

reviewed and updated


c++ implementation of KKT solver + solvers unit-test [REPLACEMENT ISSUE]

The original issue
Id: 182
Title: c++ implementation of KKT solver + solvers unit-test

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @cmastalli on Jul 15, 2019, 08:44
assigned to @hammoudbilal

In GitLab by @cmastalli on Jul 16, 2019, 16:51
Are you planning to use sparse matrix (Eigen)? I would be great if this the case!

In GitLab by @cmastalli on Aug 9, 2019, 12:31
changed title from c++ implementation of KKT solver to c++ implementation of KKT solver{+ + solvers unit-test+}

In GitLab by @cmastalli on Aug 9, 2019, 12:31
changed the description

In GitLab by @proyan on Sep 20, 2019, 15:53
Hey Bilal
I can take this issue if you haven't started yet. Please let me know.

In GitLab by @hammoudbilal on Sep 20, 2019, 15:55
I actually finished it, finishing some unit tests and pushing tonight on the fork from @MaximilienNaveau

In GitLab by @proyan on Sep 20, 2019, 15:56
Ok, cool thanks.

In GitLab by @proyan on Sep 20, 2019, 15:56
The "solvers" in the title is misleading. Does this include unit-test for fddp solver as well?

In GitLab by @cmastalli on Sep 20, 2019, 17:56
We just need to merge it

In GitLab by @MaximilienNaveau on Sep 26, 2019, 10:54
I am working on the rebase the devel into this branch an checking the unit-tests pipeline

In GitLab by @cmastalli on Sep 26, 2019, 11:20
Great!

In GitLab by @cmastalli on Nov 12, 2019, 16:59
closed


Add support for default VPCs and subnets.

The first VPC created will be elected as default.
All subnets of the default VPC are considered default for their availability zone.

Great, thank you!


html2canvas with Angular6 - Object(...) is not a function

I am using the latest version of html2canvas (version 1.0.0-alpha.12) with Angular 6. My typescript function looks like below,
  public takeScreenshot(element: ElementRef) {
    console.log ('Take screen shot');
    html2canvas(element).then(function(canvas)  {
      const contentData = canvas.toDataURL('image/png');
      console.log('successfully took the byte array for screen shot');
    });
  }

However, i get the following error,
ERROR TypeError: Object(...) is not a function
    at ThumbnailService.push../src/services/thumbnail.service.ts.ThumbnailService.takeScreenshot (thumbnail.service.ts:19)
    at ScreenDesignerComponent.push../src/components/screen-designer/screen-designer.component.ts.ScreenDesignerComponent.takeScreenshot (screen-designer.component.ts:65)
    at Object.eval [as handleEvent] (ScreenDesignerComponent.html:7)
    at handleEvent (core.js:10258)
    at callWithDebugContext (core.js:11351)
    at Object.debugHandleEvent [as handleEvent] (core.js:11054)
    at dispatchEvent (core.js:7717)
    at core.js:8161
    at HTMLDivElement.<anonymous> (platform-browser.js:995)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:421)

Am I doing anything wrong?

how are you importing the html2canvas ? I am using it without issues in an angular 6 project importing it  like this import * as html2Canvas from 'html2canvas';
Also probably you would get faster answers to support questions on stackoverflow first.


fix(nebula_ros): remove calibration file argument from robosense launch

PR Type

Bug Fix

Related Links
Fixes: https://github.com/tier4/nebula/issues/115

Description

Since Robosense lidars provides calibration data via DIFOP packets, we don't require a calibration file. Every time the driver is started, calibration data is parsed from incoming DIFOP packets.
The calibration file argument was added during development process and removed after further developments. Currently, Robosense driver doesn't use them.
Review Procedure

The Robosense launch file should make the driver launch without errors.
Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (965b4d5) 6.54% compared to head (64909f5) 5.82%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #116      +/-   ##
========================================
- Coverage   6.54%   5.82%   -0.72%
========================================
  Files        136      28     -108
  Lines      10956    4049    -6907
  Branches     859     463     -396
========================================
- Hits         717     236     -481
+ Misses      9663    3401    -6262
+ Partials     576     412     -164

Flag
Coverage Δ

differential
5.82% <ø> (?)

total
?

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

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

@mebasoglu
Max is expected to rejoin us in March, so is it ok if I review approve it?

Hello @knzo25 , it is okay. Thank you for the information.


Dependabot alerts: optional comment with dismissal

Summary
Similarly to code scanning, with this change you will be able to add an optional comment when dismissing a Dependabot alert. Dismissal comments are recorded in the alert timeline.
Intended Outcome
You can record the context of an alert closure, which security teams can view when auditing resolved alerts.
How will it work?
When dismissing a Dependabot alert, you will be able to add a comment.

This has shipped to dotcom! https://github.blog/changelog/2022-08-22-dependabot-alerts-optional-dismissal-comment-2/ Leaving open to track release to GHES

This release has shipped to GHES 3.7🚢<EMAIL_ADDRESS>


why CrowdDet-master has not setup.py?thanks

This project bases on pytorch and torchvision. No other dependent module need to be installed.


buildPerlPackage: fix patchShebangs when cross-compiling

Motivation for this change
Cross-compiled perl.dev contains miniperl compiled for the build arch. When perl is added to buildInputs, perl.dev ends up in $HOST_PATH, causing patchShebangs to use ${perl.dev}/bin/perl as the shebang. A fix is to explicitly only include perl.out in buildInputs. This bug only occurs with Perl packages with executable scripts, for example XMLTwig.
This PR does not affect native builds, because native Perl does not have a dev output. I have only tested this PR with cross-compiled XMLTwig. I attempted to test git and hydra, but they both depend on ModuleBuild, which currently cannot be cross-compiled.
cc @alyssais @volth @worldofpeace
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

When perl is added to buildInputs, perl.dev ends up in $HOST_PATH, causing patchShebangs to use ${perl.dev}/bin/perl as the shebang.

As far as I remember, It is intentional, many perl packets run perl as part of build process.
So the change might break cross-compilation of other packages.
And there is patchShebangs --build and patchShebangs --host which could be explicitly run

Also, ModuleBuild (and packages which use it) is not crosscompilation-friendly.
I doubt that something so easy could fix it.

As far as I remember, It is intentional, many perl packets run perl as part of build process.

Yes, packages need miniperl at build time. perl.dev is added to nativeBuildInputs, which causes miniperl to be added to $PATH. $HOST_PATH, on the other hand, is only used by patchShebangs --host, and therefore must not have miniperl, which is what this PR fixes.

@ofborg build pkgsCross.aarch64-multiplatform.perlPackages.XMLTwig pkgsCross.aarch64-multiplatform.perlPackages.XMLParser pkgsCross.aarch64-multiplatform.perlPackages.NetDBus pkgsCross.aarch64-multiplatform.perlPackages.FileSlurp

As far as I remember, It is intentional, many perl packets run perl as part of build process.

Yes, packages need miniperl at build time. perl.dev is added to nativeBuildInputs, which causes miniperl to be added to $PATH. $HOST_PATH, on the other hand, is only used by patchShebangs --host, and therefore must not have miniperl, which is what this PR fixes.

Do you have a particular case?
A package which failed to cross-compile without this PR and succeeded with this PR?

@ofborg build pkgsCross.armv7l-hf-multiplatform.perlPackages.XMLTwig pkgsCross.armv7l-hf-multiplatform.perlPackages.XMLParser pkgsCross.armv7l-hf-multiplatform.perlPackages.NetDBus pkgsCross.armv7l-hf-multiplatform.perlPackages.FileSlurp

Do you have a particular case?
A package which failed to cross-compile without this PR and succeeded with this PR?

XMLTwig is affected by this PR. It successfully compiled, but the xml_grep script within the package had miniperl as the shebang, rather than perl built for the host. I found this while investigating unintentional build architecture references in a cross compiled NixOS system.

I  still do not understand how replacing perl with perl.out or perl does help.
When does perl have not .out ?
And when perl.out != perl ?

Yeah, the or perl part is not necessary. I guess I was thinking that native Perl did not use multiple outputs.
The important part is that plain perl adds both perl.out and perl.dev to $HOST_PATH, which we can avoid by explicitly specifying perl.out.

The important part is that plain perl adds both perl.out and perl.dev to $HOST_PATH, which we can avoid by explicitly specifying perl.out.

Ah, another implicit magic of using .dev output name.
Probably, renaming perl.dev to non-magic name (such as perl.mini), besides adding clarity, would solve the issue as well?

Yes, I think it would.

I updated this PR to rename the dev output to mini. This does not cause any rebuilds for native Perl, and fixes the shebang in XMLTwig.
@ofborg build pkgsCross.armv7l-hf-multiplatform.perlPackages.XMLTwig pkgsCross.armv7l-hf-multiplatform.perlPackages.XMLParser pkgsCross.armv7l-hf-multiplatform.perlPackages.NetDBus pkgsCross.armv7l-hf-multiplatform.perlPackages.FileSlurp

This situation is very confusing me. There is a tradition of putting build-platform binaries in the "dev" output, but it is generally the last resort.
Here are my questions:

Quick sanity check, does Perl have an target platform? It's an interpreter not compiler, so I don't think so.

Is the "mini" output only a native perl?

If it is just a build platform perl, why don't we use buildPackages.perl instead of perl.mini?

If it does also have some host platform things, we can use propgatedNativeBuildInputs to indicate those host platform things will need to be used with some other build platform things without polluting the HOST_PATH.

I've built XML::Twig on using pkgCross and QEMU and getting different results:
Building using QEMU gives a working xml_grep executable.
$ nix-build -A perlPackages.XMLTwig --argstr system aarch64-linux
...
$ head -1 result/bin/xml_grep
#!/nix/store/bpakd2xsfxv5gdpq4pshjir8l5di7knw-perl-5.30.3/bin/perl -w -I/nix/store/bpakd2xsfxv5gdpq4pshjir8l5di7knw-perl-5.30.3/lib/perl5/site_perl -I/nix/store/gqy8jqg0bilai2lhz1asr3dhdgmx0vfa-perl5.30.3-XML-Parser-2.44/lib/perl5/site_perl -I/nix/store/mrdhwviprylq0bxkn6x04mh01fkmq3h6-perl5.30.3-libwww-perl-6.45/lib/perl5/site_perl -I/nix/store/di9mpjrwx8wxbqqmkv24x1f14n51ypfd-perl5.30.3-File-Listing-6.04/lib/perl5/site_perl -I/nix/store/382d8gjy6cx9b8f8nmllk6fwn7ykwzfn-perl5.30.3-HTTP-Date-6.05/lib/perl5/site_perl [...]

However on pkgCross, all dependencies for xml_grep are not included in the shebang.
$ nix-build -A pkgsCross.aarch64-multiplatform.perlPackages.XMLTwig
..
$ head -1 result/bin/xml_grep
#!/nix/store/cpxrf7znnw2vx04bgp1l334z5x913dn3-perl-5.30.3-aarch64-unknown-linux-gnu/bin/perl -w -I/nix/store/bpg8nxy56lnbymr77b9bch08w1794p73-perl5.30.3-XML-Twig-3.52-aarch64-unknown-linux-gnu/lib/perl5/site_perl

The Perl cross infrastructure in nixpkgs is based on the perl-cross project which provides a custom build system and patches to make it possible to cross-compile Perl.
@Ericson2314

1. Quick sanity check, does Perl have an target platform? It's an interpreter not compiler, so I don't think so.

Perl has native (XS) modules, although I know very little about the build process.

2. Is the "mini" output only a native perl?

It also contains pure Perl stubs for native modules that are needed at build time.

3. If it is just a build platform perl, why don't we use `buildPackages.perl` instead of `perl.mini`?

miniperl is wrapped to add certain directories from the host perl to PERL5LIB. It might be possible to use buildPackages.perl instead of miniperl, but at the very least the search paths would have to overridden to match miniperl's.

4. If it does also have some host platform things, we can use `propgatedNativeBuildInputs` to indicate those host platform things will need to be used with some other build platform things without polluting the `HOST_PATH`.

I'm not sure what this would entail in this case.
@stigtsp

However on pkgCross, dependencies for xml_grep are missing from the shebang.

This is because the cross setup hook uses $targetOffset instead of $hostOffset. I don't know why this was done nor why this PR broke the existing behavior, but changing it to $hostOffset fixes the shebang, and doesn't obviously break any of the packages we have been testing.

However on pkgCross, dependencies for xml_grep are missing from the shebang.

This is because the cross setup hook uses $targetOffset instead of $hostOffset. I don't know why this was done nor why this PR broke the existing behavior, but changing it to $hostOffset fixes the shebang [..]

Tested that pkgsCross.aarch64-multiplatform.perlPackages.XMLTwig builds xml_grep with correct shebang, and that it executes OK.


Avoid all uses of async-std expect when spawning tasks

Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)

Closes #1947
Other information and links

Unresolved question: What to do about tide? It requires async_std. Can we use a different crate for the RPC server?

Unresolved question: What to do about tide? It requires async_std. Can we use a different crate for the RPC server?

There was an issue to potentially replace tide, I'd look into either warp or rocket.

Or maybe actix. The jsonrpc-v2 library has support for that crate but maybe that doesn't mean much.

actix was a bit rough the last time I used it; the API wasn't as high-level as it could be. Perhaps something changed though.

Nice! Could you update the branch with the new toolchain? (will be easier to switch branches).

Nice! Could you update the branch with the new toolchain? (will be easier to switch branches).

Done.

Did you try moving to tokio::test too? Or maybe we will just port the tests in a separate PR?

Did you try moving to tokio::test too? Or maybe we will just port the tests in a separate PR?

We can switch to tokio::test once we're not using async-std tasks anymore.


Outputdir not set correctly when -i only gets filename

If xlsx-conv is run from same dir as Excel file and only filename is provided as argument then outputdir is not set correctly and conversion will fail with permission denied error.
Workaround is to provide -o "." as additional argument.

What it did was write a forward slash before the filename, which resulted in the tool trying to write the output to the root of the drive. If the user has no permission to write output there then a Permission denied error will occur.
The use of os.path.realpath seems to able to solve this. Then we can get the os path to the filename, regardless of whether that is provided by fully by the user. Not sure if that has any side effects.


Vertical dot container remove

I could not find an option to remove the vertical dot container which is a lot of times redundant and makes the report look ugly

Hi @ankurv2017 - can you provide more context here? I'm not sure what you're referring to.


Creating link references

What are the plans for representing links?
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.1.md#linkObject
With some input for alignment.. I'm happy to contribute code and PR.

This feature seems hard to provide. We should extend r200, object to write a code liking this:
class Link:
    operationId = "GetUserAddress"
    parameters = {
        "userId":  "$request.path.id"
    }

class GetUser(PydanticView):
    async def get(self, id: UUID, /) -> r200[UserSchema, Link]:
        ...

class GetUserAddress(PydanticView):
    async def get(self, id: UUID, /) -> r200[str]:
        ...

But currently we cannot write r200[type, other_data_such_as_link] and the code can become hard to read.
Perhaps is better to define the links outside the http handler annotations.
oas.define_link('/users/id', ...)

A Link needs an operationId and the operationId can be easily added in the generated OAS

operation_id property id is added here:
https://github.com/Maillol/aiohttp-pydantic/blob/4ff9739293a28a1cbfe728521769cd4dcd682689/aiohttp_pydantic/oas/struct.py#L167

The oas_operation.operation_id is set  with handler.__qualname__ here:

https://github.com/Maillol/aiohttp-pydantic/blob/4ff9739293a28a1cbfe728521769cd4dcd682689/aiohttp_pydantic/oas/view.py#L105
Could you create a PR to add the operation_id?
I need time to plan how to define links but proposals are welcome.
Thank you and happy new year 2021!

This feature seems hard to provide. We should extend r200, object to write a code liking this:
class Link:
    operationId = "GetUserAddress"
    parameters = {
        "userId":  "$request.path.id"
    }

class GetUser(PydanticView):
    async def get(self, id: UUID, /) -> r200[UserSchema, Link]:
        ...

class GetUserAddress(PydanticView):
    async def get(self, id: UUID, /) -> r200[str]:
        ...

But currently we cannot write r200[type, other_data_such_as_link] and the code can become hard to read.
Perhaps is better to define the links outside the http handler annotations.
oas.define_link('/users/id', ...)

A Link needs an operationId and the operationId can be easily added in the generated OAS

operation_id property id is added here:
https://github.com/Maillol/aiohttp-pydantic/blob/4ff9739293a28a1cbfe728521769cd4dcd682689/aiohttp_pydantic/oas/struct.py#L167

The oas_operation.operation_id is set  with handler.__qualname__ here:

https://github.com/Maillol/aiohttp-pydantic/blob/4ff9739293a28a1cbfe728521769cd4dcd682689/aiohttp_pydantic/oas/view.py#L105
Could you create a PR to add the operation_id?
I need time to plan how to define links but proposals are welcome.
Thank you and happy new year 2021!

I would like use typing.NewType to define which parameters can be used as input in other method.
https://docs.python.org/3.8/library/typing.html#typing.NewType
UserIdType = NewType('UserIdType', UUID)  # All parameters typed UserIdType will be linked.

class UserToCreate(BaseModel)
    name: str
    address: str

class ReturnedUser(BaseModel):
    id: UserIdType
    name: str
    address: str

class PostUser(PydanticView):
    async def post(self, user: UserToCreate) -> r201[ReturnedUser]:
        ...

class GetUser(PydanticView):
    async def get(self, id: UserIdType, /) -> r200[ReturnedUser]:
        ...

I would like use typing.NewType to define which parameters can be used as input in other method.
https://docs.python.org/3.8/library/typing.html#typing.NewType
UserIdType = NewType('UserIdType', UUID)  # All parameters typed UserIdType will be linked.

class UserToCreate(BaseModel)
    name: str
    address: str

class ReturnedUser(BaseModel):
    id: UserIdType
    name: str
    address: str

class PostUser(PydanticView):
    async def post(self, user: UserToCreate) -> r201[ReturnedUser]:
        ...

class GetUser(PydanticView):
    async def get(self, id: UserIdType, /) -> r200[ReturnedUser]:
        ...

Hello @Bengreen
I started a MR to represente the links.
https://github.com/Maillol/aiohttp-pydantic/compare/add_oas_link_references
Do you want continue ?

Hello @Bengreen
I started a MR to represente the links.
https://github.com/Maillol/aiohttp-pydantic/compare/add_oas_link_references
Do you want continue ?


[Snyk] Upgrade html-snapshots from 1.15.0 to 1.16.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 1.15.0 to 1.16.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 2 days ago, on 2023-05-06.

Release notes

Package name: html-snapshots

1.16.1 -<EMAIL_ADDRESS>

1.16.0 -<EMAIL_ADDRESS>puppeteer default headless=new

1.15.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

66f3026 Merge pull request #356 from localnerve/rc-1.16.1
404e1d9 Merge branch 'master' into rc-1.16.1
f610e81 Merge pull request #357 from localnerve/snyk-upgrade-685374b0fb822045c3411dde63383c45
f58ccb2 fix: upgrade html-snapshots from 1.15.0 to 1.16.0
a817ebc @ 1.16.1 -<EMAIL_ADDRESS>d8d4b1c Merge pull request #354 from localnerve/snyk-upgrade-dca1fa802906d79795cc8e4520387206
3e49c5a fix: upgrade html-snapshots from 1.14.1 to 1.15.0
d47d321 Merge pull request #353 from localnerve/snyk-upgrade-6a27638c2a7505c0c4242ecb4ad60af5
e58aeab fix: upgrade html-snapshots from 1.14.1 to 1.15.0
d0c32c5 Merge pull request #352 from localnerve/snyk-upgrade-079f5ac914033f8643382741d126ec9b
db34e8f fix: upgrade html-snapshots from 1.14.1 to 1.15.0
ee1d243 Merge pull request #350 from localnerve/rc-1.16.0
3ac9fbd Merge branch 'master' into rc-1.16.0
b83a35b Merge pull request #351 from localnerve/snyk-upgrade-ada81c0d693984560bccd9214bc45e63
e69a5b4 fix: upgrade html-snapshots from 1.14.1 to 1.15.0
02919a3 @ 1.16.0 - puppeteer 20.1.0
5b0d0b4 @ 1.16.0 - expand initial timeout
3e317b3 @ 1.16.0 - default to headless=new
d4a933a Merge pull request #349 from localnerve/snyk-upgrade-f412e51a891ed1260448085fbe1991e3
940cca7 fix: upgrade html-snapshots from 1.14.1 to 1.15.0
49231c7 Merge pull request #348 from localnerve/snyk-upgrade-f8b33c64f6109a20073183bab60edb2c
9b20c96 fix: upgrade html-snapshots from 1.14.1 to 1.15.0
e0341fc Merge pull request #347 from localnerve/snyk-upgrade-332d5266185f87a0436b89bad12cb008
e127d48 fix: upgrade html-snapshots from 1.14.1 to 1.15.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Coverage: 99.245%. Remained the same when pulling 56caff6b63046a71cd82bb8cd2b01fe3c1fb49bd on snyk-upgrade-bf41bf14b7457586226c6692ca30f3d4 into 2df82dc19293f98400fca1f7b5144f778cca10e5 on master.


Add custom node.js scripts repo

Thanks. For the moment, I find the description too broad and your README not explicit enough. You should try to answer to the question: how can your repo be helpful to someone?


TypeError: unhashable type: 'DeviceArray'

Hi, thanks for sharing the code.
When I ran code python trajectory_experiments.py, there is an error as follows,

File "/home/xyz/code/SCoNe_GCN/trajectory_analysis/trajectory_experiments.py", line 277, in
n_nbrs = onp.array([len(nbrhoods_dict[n]) for n in last_nodes])
TypeError: unhashable type: 'DeviceArray'

more info:

type(last_nodes)
<class 'jaxlib.xla_extension.DeviceArray'>
last_nodes.shape
(1000,)

Sorry about that, should be resolved now -- caused by a typo (np instead of onp on line 236)


Changing how we set empty defaults

Removed applyDefaults in favor of using the existing empty spec. I don't think the intended purpose of applyDefaults is really to hydrate the object with empty structures just to support the components. I think it's really intended to provide the default values we want to display when creating a new resource.
Fixed an issues with parameters not resizing due to a misplaced &.

rancher/dashboard#1194

I actually think that this issue is present in other resources that have nested specs and components. They'll likely break as well in VIEW and EDIT if someone created a resource external to the UI and omitted some of the nested structures that aren't required by the backend.
As I explained in the description I don't think the intended purpose of applyDefaults() is really to hydrate the object with empty structures just to support the components. I think it's really intended to provide the default values we want to display when creating a new resource. I think this largely because of the mode === _CREATE. Simply removing the mode === _CREATE is problematic because then the defaults can override purposefully empty values or overwrite set values in the case of persistentvolumeclaim and catalog.cattle.io.app
Do we maybe want to also add a applyEmptyDefaults() that gets called for all modes and just merges into the existing object?

I see your point, it was intended for setting defaults on create.
You can't really apply any changes like this on _VIEW though because you're working with a live copy of the model from the store. a) I'd say it shouldn't be randomly tweaking the model to differ from the backend in general b) it could be overwritten 1 second later by a change event.  So we could chase that farther down and say it's applied every time data is loaded from the server, but I'd rather the thing using the model just be responsible for not blowing up if fields it's expecting aren't set.
On create/edit it's fine to add whatever trees of fields you need to store options the form could be setting.  There it's a fixed clone of the model and it's ok to tweak whatever we want when you're going to push it back to the server.
So maybe just calling it it if mode !== _VIEW and passing in the mode to applyDefaults? Then you can decide which things you want to add only if missing vs overwrite.  Actually mode is already passed in like that (despite only being invoked on create) but only one place in Rio ever looks at the argument (Service varies based on mode being create vs clone vs stage)

So maybe just calling it it if mode !== _VIEW and passing in the mode to applyDefaults? Then you can decide which things you want to add only if missing vs overwrite. Actually mode is already passed in like that (despite only being invoked on create) but only one place in Rio ever looks at the argument (Service varies based on mode being create vs clone vs stage)

So I considered this option as well but when we're using the same components for _VIEW as we do _EDIT/_CREATE it will still blow up in _VIEW.
One other things I toyed with was computed props with get/set (get returns a default when not found, set creates the empty structures before setting) to handle this. I think it works in each of the cases but it adds boilerplate and may be finicky with arrays (haven't thought that one through).


@Scheduled start_ts & duration is null in db

@Scheduled start_ts & duration is null in db

I'm closing this issue since it has no useful information. If you create an issue it provides you all details how to create a proper issue.


License GPL-3

Hi,
You package is in a copyleft license (GPL v3), and so we legally can't use it for a commercial usage (a web agency can't sell a closed source project with your package in it).
Is it a real assumed choice for you, or is it a mistake?
Thanks.
Matt'

Hi @mathieutu . Happy to switch to MIT license.


feat: add k1LoW/octocov

#5565 k1LoW/octocov: octocov is a toolkit for collecting code metrics (code coverage, code to test ratio and test execution time)
$ aqua g -i k1LoW/octocov

Thanks.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.32.0


Hi, works in spanish?

Hi, I change all sentences but I do not know if it will work in other languages, doesn't work to me. It's because they're in spanish?

What means doesnt work? you dont hear a voice? if is that, then you have a problm with Artyom, if you hear but is using english pronunciation just change the language here:
					zoombot.initialize({
					    lang: "en-GB", // GreatBritain english
					    continuous: true, // Listen forever
					    soundex: true,// Use the soundex algorithm to increase accuracy
					    debug: true, // Show messages in the console
					    listen: true, // Start to listen commands !
					    // If provided, you can only trigger a command if you say its name
					    // e.g to trigger "Hello", you need to say "Zoombot Hello"
					    //name: "zoombot"
					}).then(() => {
					    console.log("Artyom has been succesfully initialized");
					}).catch((err) => {
					    console.error("Artyom couldn't be initialized: ", err);
					});

We are currently working on translation capabilities


Update run-queries.md

Description
Update title
Issues Resolved
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kolchfa-aws Will you review this PR? I created it in GitHub following the process we discussed to resolve the TOC merge conflicts. If I missed any steps, please let me know so that I can correct my process before moving forward to resolve the other merge isses. Thanks!


Remove obsolete _MAC fields

The three fields at the end of WIN32_FIND_DATA are under an #ifdef _MAC.  This was used 25 years ago to support Visual C++ Cross Development Edition For Macintosh, which was not carried forward into OS X so has been obsolete for at least 20 years.  Anyone targeting Windows will not have these fields present in the structure, and suggesting they do is misleading.
This also removes the second definition of reparse tags from WIN32_FIND_DATAW.  Note the reparse tags are still included in the text above, and this second definition is not present in WIN32_FIND_DATAA, so it looks to be mistakenly included.

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

Hi @malxau - These fields are still in the header and, as such, still need to be included in the documentation. There's also the corresponding W struct that needs the same update. I'll close this PR and spin up a private one with the suggested info. Thanks.


webpagetest private instance not worked

hello,
i installed webpagetest on centos 7 using npm install.
also installed webpagetest webbased private instance.
i want to test webpagetest using command line on centos 7.
with private instance webpagetest.
server is located at china.
tested below command and gives issue:
webpagetest test https://twitter.com/marcelduran --key A.cb529269f4e9b03XXXXXXXX --location ec2-ap-southeast-2 -s myhostname
gives below output:
{
"statusCode": 400,
"statusText": "Invalid Location, please try submitting your test request again."
}
i also checked and tried some locations from
webpagetest locations.
but still same output for all.
please let me know whatv is issue here and how can i get output.
thanks.

@sachinbanugariya Even I have also faced the same issue...
Check whether your requested location is present or not : urdomain/getLocations.php
In my private instance, I can't find this location "ec2-ap-southeast-2"

ok, i also want to know that:
webpagetest test https://twitter.com/marcelduran --key A.cb529269f4e9b03XXXXXXXX --location ec2-ap-southeast-2 -s myhostname
if this command work then results seems like below:
http://myhostname.com/result.php ?
or seems like:
http://webpagetest.com/result.php.
and the other thing is that is if i use below command:
webpagetest test https://facebook.com --key A.cb529954098eed --location ec2-us-east-1:Chrome
then it gives results like:
{
"statusCode": 200,
"statusText": "Ok",
"data": {
"testId": "170831_6T_1e69fcb2b3613772814647a3b20a390b",
"ownerKey": "65ba099bab3a229306fd42727846816df6c9fb08",
"jsonUrl": "https://www.webpagetest.org/jsonResult.php?test=170831_6T_1e69fcb2b3613772814647a3b20a390b",
"xmlUrl": "https://www.webpagetest.org/xmlResult/170831_6T_1e69fcb2b3613772814647a3b20a390b/",
"userUrl": "https://www.webpagetest.org/result/170831_6T_1e69fcb2b3613772814647a3b20a390b/",
"summaryCSV": "https://www.webpagetest.org/result/170831_6T_1e69fcb2b3613772814647a3b20a390b/page_data.csv",
"detailCSV": "https://www.webpagetest.org/result/170831_6T_1e69fcb2b3613772814647a3b20a390b/requests.csv"
}
}
so the thing is command is not work if i use  --server https://hostname
that is webpagetest test https://facebook.com --key A.cb1b1954098eed --location ec2-us-east-1:Chrome --server https://hostname
please suggest how can i go ahead.
thanks


Issues setting up the project on AWS Amplify

Expected behavior and actual behavior
I can't make the demo app work. I have deployed to AWS Amplify and got it successfully built and deployed but when I try to create or join a call I get the error 'err in handleJoin: API meetingApi does not exist.'
Steps to reproduce the problem

Fork the repo.
Configure and deploy it on AWS Amplify
Create a User
Connect to the application
Try to start a meeting

Specifications like the version of the project, operating system, or hardware
Latest Version

Can you send me the error logs?


New router

Huge thanks to @bencompton

@nolimits4web - by the way, I still have it on my to-do list to update the F7 + Vue docs with this new routing functionality. My hope was to find some time next week to work on the framework7-react docs and also work on the docs for this as well.


Undefined variables cause uninitialized values

In for_do:
(print (reduce y acc cur (do
        (let x (* acc 98))
        (/ x curr)
    ) 1 ))

cur != curr. When division reads curr, it gets an unitialized value instead.
The simple fix would be to add something along the lines of the following:
                fprintf(stderr, "Undefined variable: %s\n", ast->content.data.str.body);
                exit(0);

at lines 88 / 89 of execute.c. In other words, if it couldn't find the variable, error out instead of just ignoring it.

Fixed :-)


Configuration option external is ignored

Hello,
I tried defining pg-native as an external dependency but it seems the option is ignored, i.e., not being merged with the computed externals here:
https://github.com/folke/esbuild-runner/blob/5f005db17becd14cdfe2025c41b82f2bc9d19d9e/src/esbuild.ts#L94
My esbuild-runner.config.js:
module.exports = {
  type: 'bundle',
  esbuild: {
    external: ['pg-native'],
  },
}

and the error that is produced when running:
 > ../../node_modules/pg/lib/native/client.js:4:21: error: Could not resolve "pg-native" (mark it as external to exclude it from the bundle, or surround it with try/catch to handle the failure at run-time)
    4 │ var Native = require('pg-native')
      ╵                      ~~~~~~~~~~~

/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1466
  let error = new Error(`${text}${summary}`);
              ^
Error: Build failed with 1 error:
../../node_modules/pg/lib/native/client.js:4:21: error: Could not resolve "pg-native" (mark it as external to exclude it from the bundle, or surround it with try/catch to handle the failure at run-time)
    at failureErrorWithLog (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1466:15)
    at /Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1148:28
    at runOnEndCallbacks (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1066:65)
    at buildResponseToResult (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1146:7)
    at /Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1253:14
    at /Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:629:9
    at handleIncomingPacket (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:726:9)
    at readFromStdout (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:596:7)
    at runServiceSync (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1831:3)
    at Object.buildSync (/Users/vieira/Code/f1/node_modules/esbuild/lib/main.js:1651:3)

I should note that I am using npm7 workspaces and that pg-native is a dependency of a dependency of another package in the workspace, if I add the dependency directly to the package.json of the package I am trying to run it works (the externals computation finds it).

The reason is that we take all deps and devdeps from your package.json and put those as externals. I'll change it so you can additionally pass in any other externals as you do right now.

I just released a new version where your external config should now work as expected. Thank you for reporting!


Some linear control measures are not compliant with the 2525/APP-6 standards

This is related to issue #108 (Using Multipoint Geometry to draw Control Measures with unique draw rules/control points)
Several control measures have the same general issue (a list of these control measures needs to be added to this issue), where our current implementation of rendering a symbol using the vertices of a feature doesn't strictly follow the 2525/APP-6 standard.
For example, a Disrupt task, as defined in the standard, requires three points, two to define its length and a third to define its width.

Our current implementation is overly simplified, with no way to adjust the width of the symbol.

Army doctrine requires users have the ability to change the width of this symbol, to accurately depict the breadth of the disrupt task.  Military messages implementing 2525 will transmit/send three vertices from one system to a receiving system.  A message handler sitting on top of a geodatabase has a difficult choice.  Pass the three points to the geodatabase which would then be drawn as a hooked disrupt symbol, or pass just two points to the geodatabase which would then be drawn without the anticipated width.  Either decision results in ArcGIS drawing a different symbol then the one that was sent in the original message.

Thanks for writing these issues up. The control-points-from-the-standard usage does work in Runtime, but we haven't completely ironed out this usage in Pro yet. But this usage will be important when communicating these graphics over messages so it is good we are tracking it now.

Just an FYI here.  Over time I plan to work with the SSMC and JSP to simplify control measures like this.  For example, the Disrupt Task could be defined as two points that specify the length and direction of the symbol and a separate amplifier/attribute to define its width, rather than the current kind of odd way it uses three points.  Major changes like this though won't occur until the control measures are reviewed and overhauled for 2525E.

Changed milestone - this will be addressed when we can address #108 in Pro

Closing. Point order is being tracked in https://github.com/Esri/military-symbology/issues/267.


Update Project.toml

Add EarthSciMLBase v0.12

preempted by https://github.com/EarthSciML/GasChem.jl/commit/0bfaa6d2e2582f0e3d088b4a43bb14e1d5d157a1


maxLineLength not recognized by VSCode

Describe the bug
When I went to change the line length the extension considered, it was documented fine in the Wiki, but I first didn't see the option in the UI Settings in VSCode, then when I manually put the setting in the settings.json, the change took affect, but VSCode didn't recognize the setting.
Expected behavior
I expect VSCode to recognize this as a correct configuration. In fact, even though VSCode reports it as an unknown configuration, the configuration still takes affect appropriately.
Beyond the bug, it would seem appropriate to have this configuration show up in the UI Settings as well.
Actual behavior
VSCode reports the configuration setting as unknown.
Screenshots

OS (please complete the following information):

OS: Windows
Version: 11

Editor (please complete the following information):

Editor: VSCode
Version: 1.66.2

@kevin-induro Thanks for the feedback!
This is not a bug but a result of config source vs. UI settings. Config in source is recommended because it's global and under version control. You'll find more fields in Configuration but not UI settings, e.g. insertFinalNewline.
Another point is that most of the fields are derived from other plugins, e.g. Prettier, EditorConfig, ESLint. Ideally you shouldn't need to set them again if you have them defined already.


Support java worker resources management.

What do these changes do?

This change support resources management in java worker.
i. Specify the static resources of a node.
ii. Specify the required resources of a task.
[How to use?]
1. Specify node static resources:
(1)
Overwrite the static_resources filed in ray.config.ini file,
then use this ./run.sh start --head  command to start node manager.
or
use ./run.sh start --head --overwrite=Resource1:10,Resouces2:100 to start node manager directly.
(2)
Specify the required resources of your remote method or remote actor.
  @RayRemote(resources = {@ResourceItem(name = "Resource1", value = 10),
                          @ResourceItem(name = "Resource2", value = 2)})
  public static String sayHello() {
    String ret = "he";
    ret += "llo";
    return ret;
  }

Related issue number
N/A

@zhijunfu  @raulchen
I have addressed comments. Could u plz tak a look again?


Create default _realtime.tenant on helm install.

What kind of change does this PR introduce?
Bug fix
What is the current behavior?
the current behavior is that the superbase/realtime image seeds a realtime-dev tenant in _realtime.tenant. In order to make the values.example.yaml work out of the box realtime.fullnameOverride must be set to realtime-dev. If the fullnameOverride isn't set, then realtime never works with the default settings in values.example.yaml
What is the new behavior?
The new behavior updates the realtime pods /priv/repo/seeds.exs to get the TENANT_NAME from environment variables and then defaults to realtime-dev if the environment variable isn't set.
Additional context
I am going to submit a PR to the supabase/realtime seeds.exs file, in the meantime this should help with getting running out of the box.

re-submitting this from a different branch


verbose = false in s3Upload doesn't suppress the upload output

When we use s3Upload to upload bunch of files, there is a line of output for each file uploaded, which is little noisy. Then we set verbose: false in s3Upload, but it still prints out the output, one line for each uploaded file. Is verbose not working or is there something else we need to set up?

doesn't work also for me


[Doc-gae-java] Adding documentation for GAE Java SDK

What do these changes do?

Remove dummy comments in gae java sdk.
add GAE java SDK documentation generation as a part of make graphscope-docs, and referring to Java SDk docs in Docs index page.

Related issue number
Fixes #970

Thanks for the preview pictures.
OK to merge once the CI becomes green.


No method to override use of interactive visualizations

What is the expected enhancement?
When running visualizations that have an interactive mode if you're system meets the requirements we unconditionally use the interactive visualizations. We need to provide an option to let users override this and use the non-interactive visualizations.

@jaygambetta is this what you're looking for?

I think these are the only three functions we have
plot_bloch_vector
plot_state
plot_histogram

and i think we also want to have all aspects of them documented.

One thing I just thought of is do we want to deprecate the current default behavior for plot state and plot_histogram where they use the interactive mode if in jupyter and have network connectivity? The fix for this #1229 will add a mode kwarg that will let users select mpl or interactive.

@mtreinish is this for 0.7

Yes, this is going to be fixed in 0.7, it's part of #1229


could not find param file

hi , im new here , i download dsmapstudio when i open , it said could not find param file and crash , i tried to delete and extract a new one but still get "not find param file and crash" , sorry for my english

Sounds like it's trying to open a previous project and that project's setup is missing files.
Try looking in appdata/local/dsmapstudio/config.json and checking the last project. From there, make sure the vanilla game it's referencing has its param files

I do exactly what you said and it working again , thanks you sir .


Deprecate use with React 0.13

Curious on the other maintainers thoughts on deprecating use of this library with React 0.13 moving forward. Of course the previous versions are still usable. But it seems that the conditional requirement of test-utils causes plenty of confusion.
Deprecating 0.13 would allow us to clean up some code, and reduce confusion.
Thoughts?
cc @Aweary, @nfcampos, @ljharb, @lelandrichardson

I prefer to support the latest major, plus the previous 2. When v16 comes out, I'd be happy to immediately ship a breaking change that drops support for 0.13.
That said, "deprecation" is not breaking, and I think it's reasonably to support N, N-1, and N-2, but to have N - 2 be officially deprecated.

@blainekasten yep removing support for 0.13 would remove most (all?) of the conditional requires, which would be great to reduce confusion. would it actually remove all of them?

Now that enzyme v3 is out, and the 0.13 stuff is in its own adapter, there's not really a reason to ever drop support for it.


Debugger doesn’t work properly

Before reporting, I have confirmed that

[X] This bug does not appear to be reported on GitHub Issues before.
[X] This bug still exists in the latest version of the Blitz3D TSS.

Describe the Bug
When I use the debugger for IDEal and BlitzIDE, it closes automatically.
Related codes
None
Reproduction Steps

Open Blitz3D
Use the debugger

Caught Exception
None
Operating System
Windows 10
Extra Information
No response

This issue does not reproduce successfully. Requires further validation.


seo improvements: add custom plugin for page title generation

Also did a few cleanups of the templates I am going to clean up more though

This change is

When checking how the documentation now renders in staging, it looks like the page titles are now formed off the directory structure. See this page for an example: http://deploy-preview-1444.kubernetes-io-vnext-staging.netlify.com/docs/user-guide/pods/multi-container/
I don't think we want to push that live.
What exactly is the plug-in doing?

yeah thats what it's doing, if the page has no title it tries to make one from the directory structure
this was an idea to work around the problems with some pages, for example kubectl, not having titles and messing with the seo
but open to other ideas

I would actually prefer to have page authors explicitly title their pages. Let's hold onto this PR for now; I'd like to discuss the titling strategy with a wider audience (specifically Jared Bhatti and Steve Perry).

makes sense!


gobject-introspection: enable building doctool

Description of changes
Required for GIMP developer docs in #67576.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Make sure cross still works

@ofborg build pkgsCross.aarch64-multiplatform.gobject-introspection

Cross seems to work now.

thanks


Miri hangs on running with miri-compare-exchange-weak-failure-rate=1

To reproduce:
cargo init miri-test
cd miri-test
echo "#[test] fn test() {}" >> src/main.rs
MIRIFLAGS="-Zmiri-compare-exchange-weak-failure-rate=1" cargo miri test

The process hangs on "Running unittests src/main.rs...", without actually running the test.
It works fine with any other valid value of miri-compare-exchange-weak-failure-rate.
The problem is not reproducible with the nightly-2022-05-22 toolchain.

Thanks for the report!
Setting the failure rate to 1 means it always fails, as it says in the docs. It is expected then that programs hang, since their weak compare exchanges never succeed, so they never can make progress.
We recently fixed a bug here with https://github.com/rust-lang/miri/pull/2105; that's why you did not see this behavior before.

Oh, that makes sence, thanks, @RalfJung!


New package: PowerModelsGMD v1.0.0

Registering package: PowerModelsGMD
Repository: https://github.com/lanl-ansi/PowerModelsGMD.jl
Created by: @bluejuniper
Version: v1.0.0
Commit: c025b1ed236e4e45e44e9b594b7f0c01b787d226
Reviewed by: @bluejuniper
Reference: https://github.com/lanl-ansi/PowerModelsGMD.jl/issues/82#issuecomment-2415269902
Description: GMD problem formulation for PowerModels.jl

[noblock]
Is "GMD" a well known acronym? If not I would recommend spelling it out as "GeomagneticDisturbances" as in "PowerModelsGeomagneticDisturbances". GMD is also an acronym for the Geoscientific Model Development journal and there may be other things called GMD.

I think the name is fine. This package builds on PowerModels.jl, and that should be sufficient context. Pretty much PowerModelsAnything should receive very little scrutiny as far as the package name is concerned. [noblock]

[noblock] Oh I was unaware of PowerModels.jl. Feel free to ignore my earlier comment :)


Docs update, prep for radio, from tbl-ftag(bzr#96)

bzr#86..bzr#96 diffs:
Updated instructions
radio optional, if module not there, fail soft
Improved print_stats format
print_stats also works with no parameters, so can call interactively on stop Verified the jpg received, it is intact
better progress monitoring - shows bitset of received blocks - allows restart doesn't terminate transfer unless metadata changes. Now display blockmap status when END appears
New bitset added into ftag, str() now shows 80 char status of fullness Fixed remainder bit issue
Fixed the phantom bit issue
Fixed bitset off by one error
created a tester for new bitset
Designed code for new BitSet.str
tested with uart, new verifying chk state still works add the CHK_COMPLETE state, independent of VERIFYING split verification from end detect, for resume
radio now in ftag_pico
Placeholders added for pin redirects
Fixed the off-by-one length on send
display received data in radio_test as hex
use '?' for out of range ASCII chars.
Improved received trace
new test_radio API integrated
Recoded test_radio to use Buffer
Introduced send() and recvinto() API in radio.py with Buffer dependency Tested new receive wait API with timeout int flag
Introduced recvinto(wait) polling in ftag_pico.UartLink New wait:int API deployed
changed all wait:bool=False into wait:int=0 for all recvinto() fns pass in '1' instead of True,
new wait:int API design, for recvinto
designed architecture: recvinto(wait)
ftag_pico now has a get_phy() question for Uart or Radio. Removed radio names from uart pipeline in ftag_pico radio now uses myboard GPN numbers
removed RSSIRDY and return None
introduced platdeps in radio.py for sleep_ms and ticks_ms(now_ms) added a trace:bool=False flag to the radio
removed MOCKING and host code from radio
added a counter so we don't get scroll effects on trace test_radio refactored to use myboard and RadioISM import preparation for moving this into ftag_pico
move radio.py and test_radio.py into the ftag folder Power calcs documented
Captured txpower calcs, in case any questions ever asked about them. Configured the radio for 0dBm mode (PA1 only) which allows any duty cycle/bw Pushed the baud rate and FDEV up to 300kbps - works well at 250kbps Frequency dev calculator in place - still works at 30kHz frequency deviation word calculator
bw values increased
Changed data shaping to whitening
Changed packet preamble, now longer to allow better RSSI Have reduced to minimum tx power for this PA setting Now will display RSSI against each rx packet trace Wrote code for RSSI measure
Captured some RSSI reg defaults and data
sorted regs into regions that align with data sheet ordering added default values in comments for all radio regs use a tuple for the radio config
time delays tweaked a bit
Fixed a hard coded 32 byte limit, now 66
Moved further work from README.md to github issues created a board definition file
mapped UART pins etc into that file
OPTION displayed when ftag is imported
platdeps introduced and working

Lots of architectural changes to allow introduction of radio.py - this works with the radio on my bench test, but I am not releasing the radio.py yet, until I have completed the validation process. However, all these changes in this PR will be required, in order to allow the new radio to slot in later.


Interactors don't work after updating to Cypress 12

Hi @ostapwd. Thanks for reporting this. Can you create a small repository that recreates this issue? This would be very helpful in tracking this down /cc @wKich

It seems Cypress did something with built-in commands, this might be related with https://github.com/cypress-io/cypress/issues/22733


Passing current configuration to the configure(function(config))

In configure(function(config: HttpClientConfiguration)) current configuration is passed to the function. This allows us to override existing configuration without loosing its defaults and baseUrl

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@micro-jumbo Can you remove all the dist files from this PR?

@EisenbergEffect Files removed

Thanks!


Downloading 320kpbs Maximum bitrate of songs using Premium Spotify Account

[x] Using latest version as provided on the master branch
[ ] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Script won't run
[ ] Encountered bug
[x] Feature request
[ ] Question
[ ] Other

System information

Your python version: python 3.x
Your operating system: Ubuntu 16.04

Description

Hello can we download the maximum bitrate of the music 320kbps using our Spotify Premium account
on your Spotify Downloader?

This tool only looks and download for matching songs on YouTube.
You won't get any special features out of this tool even if you own a premium account.

ohh i see, but its possible that converts from 320kbit/s right?
Thanks

No, that is not possible. YouTube only contains audio streams for 192k bitrate.


Exported properties not updating on inspector even via PropertyListChangedNotify()

Godot version
3.4.stable
System information
Windows 10 GLES 2
Issue description
Doing a minimal test with the script below (attached to Node2D) with:
-Debug>Synchronize Scene Changes ON.
-Debug>Synchronize Script Changes ON.
-Click on Play (F5)
Any idea what might be happening? It will print accordingly, but inspector stuck to 0.
Steps to reproduce
using Godot;
using System;

public class refresh_inspector_test : Node
{
    [Export] public int counter = 0;

    public override void _Process(float delta)
    {
        counter = counter + 1;
        GD.Print(counter);
        PropertyListChangedNotify();
    }

}

Seems like the changes are only displayed in the "remote" scene, even though I have been learning to tweak values using the local one.
Newcomer mistakes I guess, doubt I guess this might happen to more people, I would suggest switching to remote scene when in play mode perhaps. Or changing the name convection in here, external sounds like something running remotely or more like networking things to me.


How do you use Vuex with a nested structure?

The documentation says that you should use v-model with Vuex, but you're not allowed to use v-model on props passed down to nested components. How do we work around that limitation?

For such complex case, You can listen for @change event from VueDraggableNext


Captcha on API!!!

I trying to resolve an information about a wallet through API request https://insight.litecore.io/api/addr/LQFJLDNtScTn77BoWP8KpUr3taF1xTA5aw and I have the next page:

Could you explain me please, how should my software avoid it?

litecore.io is not related to this repo.
That's a Cloudflare captcha.  Try googling "cloudflare captcha"


[WIP][SPARK-34068][CORE][SQL][MLLIB][GRAPHX] Remove redundant collection conversion in Spark code

What changes were proposed in this pull request?
There are some redundant collection conversion can be removed, for version compatibility, clean up these with Scala-2.13 profile.
Why are the changes needed?
Remove redundant collection conversion
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass the Jenkins or GitHub Action

Test build #133918 has started for PR 31125 at commit a4e44c2.

Test build #133918 has started for PR 31125 at commit a4e44c2.

Now this pr is  WIP, need some local manual test

Now this pr is  WIP, need some local manual test

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38505/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38505/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38505/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38505/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38505/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38505/

Test build #133918 has finished for PR 31125 at commit a4e44c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133918 has finished for PR 31125 at commit a4e44c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133918/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133918/

Merged to master

Merged to master

thx~ @srowen

thx~ @srowen


Importing a package?

I've got Python4Delphi (P4D) and PythonEnvoronment working together in a very crude REPL fashion for testing purposes as an Embedded Python 3.9 (can't use higher - ONNX....)
If I try something simple like...
import numpy
Obviously this raises an import exception in P4D as the PythonEnvironment has hardly any libraries. What I'd like to therefore do is handle the exception and import numpy.
This repo appears to include the functionality to add a library, e.g. numpy (importing libraries in general via PIP are what I'm trying to get working)
Some guidance on how to get PIP to add any library (like numpy) to my Embeded Python would be very much appreaciated (I'm positive this is something other people will want to know as well)
Note: I know this project is very new so apologies for hassling you. I'm also aware numpy is probably not the best example as it'll raise a DivByZero - or maybe that does make it a good example - who knows?
Edit: Looking through the source it appears I'm after Import for an un-managed package

The progress will be implemented as a feature locally, it is not functional in the current implementation.
We are setting up a Discord channel. I will invite you to join us.

Look forward to the Discord - there are some examples of the output from my project in my own Discord - https://discord.gg/e2rGEEHrBd (you can PM me the invite there...)


would be nice having a flutter google maps plugin made with flutter

so google maps will works on every platform..

Hi @renanyoy
Google map is a service that supports various platforms and Flutter runs on those platforms, your proposal should Google maps platform itself
Closing, If you disagree, please write in the comments and I will reopen it.
Thank you

sorry, I though google maps addon was made by flutter team..

@renanyoy
The plugin (which basically runs the native google maps code)  is developed by the Flutter team, not the actual Google maps platform.


Deprecate #user_agent.

This closes #122.

Sorry for asking after 5 years, but WHY #user_agent was deprecated? What if need change User-Agent per request?

You still can, just set the User-Agent header.

Oops, "User-Agent" works, and "user-agent" - not. Ok


Websockets

I had to dig through discord and do a bunch of testing to get this working with their new url api.elections.kalshi.com/trade-api/v2 and apiKey auth. This is a sketchy and somewhat untested implementation. Could use a good polish pass from maintainer or others but figured it's worth merging in.
ApiKeys are only implemented for websockets btw someone else will need to do it for rest side but should be easy as I've basically outlined how to use the api key auth here.
Here's example of how to use websockets with this addition for anyone curious / referencing this:
pub mod common;

use std::{fs, path::PathBuf, str::FromStr, vec};

use kalshi::Kalshi;
use kalshi::KalshiChannel;
use serde::Deserialize;

#[tokio::main]
async fn main() {
    // Load config
    let file_str = fs::read_to_string("config.yaml").unwrap();
    let Config {
        kalshi_api_key_id,
        kalshi_api_private_key,
        ..
    } = serde_yaml::from_str(&file_str).unwrap();

    let mut kalshi_instance = Kalshi::new_with_api_key(
        kalshi::TradingEnvironment::LiveMarketMode,
        kalshi_api_key_id,
        kalshi_api_private_key,
    );

    let mut kalshi_ws = kalshi_instance.connect_ws().await.unwrap();

    let mut kalshi_stream = kalshi_ws.receiver();
    let listen_task = tokio::spawn(async move {
        while let Ok(res) = kalshi_stream.recv().await {
            match res {
                Ok(kalshi_res) => println!("From Kalshi {kalshi_res:?}"),
                Err(e) => {
                    eprintln!("{e:?}");
                    break;
                }
            }
        }
    });

    // Listen to all trades
    kalshi_ws
        .subscribe(vec![KalshiChannel::Trade], vec![])
        .await
        .unwrap();

    // Wait for listen task
    listen_task.await.unwrap();
}

#[derive(Deserialize, Debug, Clone)]
struct Config {
    kalshi_api_key_id: String,
    kalshi_api_private_key: String,
}

Hi Sage,
Thanks for the inclusion! I'll make a quick pass on this and include it when I have more time in the next few weeks. As for API keys, I took a look at the Kashi docs and it looks like they added this in since a year ago; I can add in an option for the rest side or generalize the WebSocket implementation sometime soon.
Once again, thank you for the contribution! I'll make sure to work to get this merged :)
Thanks,
David


/var/run/docker.sock is broken

Jenkins and plugins versions report
Can't, NDA.
What Operating System are you using (both controller, and any agents involved in the problem)?
Linux (Ubuntu)
Reproduction steps
Setup Docker cloud in Jenkins.
No special configuration needed.
Expected Results
Jenkins can access Docker via unix socket, just as the elder gods intended.
Actual Results
Jenkins can't access Docker via unix socket, elder gods ain't happy.
Anything else?
First screenshot — 5.2.2-1.1, broken.
Second screen shot — 5.2.1-1.1, works.

Are you interested in contributing a fix?
No response

I guess there is nothing that could be fixed in this plugin since it only provides the dependencies. Maybe there was a (breaking) change in the latest HttpComponents release? We should forwarded this issue to the Docker plugin to validate it and potentially find a fix.


Testing suite is running again

The testing suite is running again. You can simply merge the main branch into your forked repository.


SetMaxPacingRate or similar for a QUIC connection?

Is there a way to set the max pacing rate from a QUIC connection client side ? For example, any API similar to the one in Google's C++ implementation ?

https://github.com/cloudflare/quiche/pull/1476


Added script to download all of the tokenized fineweb100B data from huggingface

I created this to show an example of what it would look like to download all of the files from Huggingface, this is the first step and after all of the files are uploaded to Huggingface I will upload all of the compressed versions to the same location and then add the decompression steps.

Also added the compressed version too, it will take a while to finish uploading but its around halfway done

@ChrisDryden - for the compressed version: What about having all the shards compressed to a single file and downloading them as one file and then decompressing them? Not sure about the disk size impact.

I was thinking about having both options, but then I discovered that the biggest bottleneck with xz compression is that it only works on one thread. Once I figured out how to split each file request and decompression into multiple threads then I discovered that the download speeds are limited per file and by having many requests at the same time its a huge increase in download speeds.
AFAIK from my AWS knowledge huggingface is fronted by cloudfront cdn and they have caching limitations past 5gb for speed to it would be signifigantly faster to split into many files for both decompression and network speeds

Any long-term solutions like using bittorrent ? @karpathy

No Scripting: Everything is done within your BitTorrent client
Resilience: No single point of failure (like Hugging Face)
Scalability: More downloads = faster speeds for everyone
Cost-Effective: Reduces bandwidth costs for you as the model provider
Community-Driven: Embraces the decentralized ethos
DHT (Decentralized Goodness): BitTorrent clients can use DHT to find other peers even without a central tracker

@ChrisDryden  I noticed that fineweb100B here does not support parallel download. I can adjust the code ok I'm just curious why this was done? Is there any throttling or so?

I just hadn't gotten around to updating it, I haven't come across any throttling so far

pushed a version of this code to master. closing.


Update stylelint-prettier

There is a new version of stylelint-prettier and stylelint-config-sass-guidelines that includes a breaking change. Update the packages and fix the breaking changes. Renovate sent a PR that failed ().
### PRs that failed
- [ ] https://github.com/swisspost/design-system/pull/1186
- [ ] https://github.com/swisspost/design-system/pull/1269

@gfellerph is this still a thing?

@oliverschuerch, it's done, stylelint prettier is up to date.


Splitting on tags more than 1 level deep confuses xmlsplit

XML File
I have the following small XML file
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">
        1
    </Inner>
    <Inner otherattr="yyy">
        2-0
        2-1
        2-2
        2-3
    </Inner>
    <Inner>

            <C>
                <D Dattr="Value"/>
            </C>

    </Inner>
</Outer>

Program
And the following file
import fs from 'fs';

const XmlSplit = require('xmlsplit');

const xmlsplit = new XmlSplit(1, 'A'); // Splitting on Tag

const CHUNK_SIZE = 200; // bytes

const xmlfile = 'Test.xml';

async function start() {

    const stream = fs.createReadStream(xmlfile, { highWaterMark: CHUNK_SIZE});
    stream.pipe(xmlsplit).on('data', function(data: any) {
        const xmlDocument = data.toString();
        console.log(xmlDocument);
        console.log('--------------------------------------')
    });
}

start();

Expected output
You would expect different XML documents with A-tags, either
<Outer>
	<Inner>

			...

	<Inner>
</Outer

or an XML without the Inner tag.
Realized output
But XmlSplit return the following:
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">
        1</Outer>
--------------------------------------
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">

    </Inner>
    <Inner otherattr="yyy">
        2-0</Outer>
--------------------------------------
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">

        2-1</Outer>
--------------------------------------
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">

        2-2</Outer>
--------------------------------------
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">

        2-3</Outer>
--------------------------------------
<?xml version="1.0" encoding="utf-8"?>
<Outer>
    <Inner attr="xxx">

    </Inner>
    <Inner>

            <C>
                <D Dattr="Value"/>
            </C>
        </Outer>
--------------------------------------

If you look at the output returned you can see that in several instances the process gets confused.

Old question, and this is most likely this is not maintained but we actually ran into this error the other day and I set to Google'ing and found this...
I did a "dirty" fix as we don't really care about the nested elements, just to get it split up.
The problem is that the first dataChunk (index = 0) will retain the "parent", e.g.:
<Inner attr="xxx">
    1

While the following dataChunk parts will be "clean":
1

When this is pieced together again it'll include <Inner attr="xxx"> from the first dataChunk but never close it.
As we don't care about the <Inner attr="xxx"> element I just added a fix on the next line of this:
https://github.com/remuslazar/node-xmlsplit/blob/7a7e081c226ebe0577b35743fd40b22064f621ee/lib/xmlsplit.js#L83
By:
        dataChunks.forEach(function (data, index) {
          const tagChk = new RegExp(`^<${this._tagName}[\\S|>]{1}`);
          if (tagChk.test(data)) {
            // eslint-disable-next-line no-param-reassign
            data = data.slice(data.match(tagChk).at(0)?.length);
          }
          dataChunk += data;

This will strip away the element Inner from the first dataChunk data making the resulting XML valid.

@QAnders looks good to me. Could you please open a pull request with the changes above? Thanks!

Sure, I can do that, @remuslazar , but I need write access then to the repo... :)
I've reworked it a bit so that it does include the element as well now...

@QAnders you can fork this repo and create the PR which I can then merge later on (having write access).

Not sure I have time for that now... sorry... but here's the files changed...
./tests/test-xmlsplit.js
./tests/fixtures/nested-element.xml
./lib/xmlsplit.js
./.gitignore

node-xmlsplit.zip

PR open @remuslazar https://github.com/remuslazar/node-xmlsplit/pull/10


[1.19.2] Unable to modify modded armors

Issue description
I've been trying to change armor values of the modded armor sets available in the mod Iron's Spells and Spellbooks.
The config file is successfully loaded - its other changes do work as expected, though the armor values themselves do not change.
Config snippet
	#Set the values for armor.
	[items.armor]

		#Set the defense value of an armor item. May not work on modded armors, please file an issue if that's the case. Example (without the leading #):
		#"minecraft:golden_chestplate" = 10
		[items.armor.defense]
			"irons_spellbooks:wandering_magician_helmet" = 1
			"irons_spellbooks:pumpkin_helmet" = 1
			"irons_spellbooks:pyromancer_helmet" = 1
			"irons_spellbooks:electromancer_helmet" = 1
			"irons_spellbooks:archevoker_helmet" = 1
			"irons_spellbooks:cultist_helmet" = 1
			"irons_spellbooks:cryomancer_helmet" = 1
			"irons_spellbooks:shadowwalker_helmet" = 1
			"irons_spellbooks:priest_helmet" = 1
			"irons_spellbooks:plagued_helmet" = 1
			"irons_spellbooks:wandering_magician_chestplate" = 3
			"irons_spellbooks:pumpkin_chestplate" = 3
			"irons_spellbooks:pyromancer_chestplate" = 3
			"irons_spellbooks:electromancer_chestplate" = 3
			"irons_spellbooks:archevoker_chestplate" = 3
			"irons_spellbooks:cultist_chestplate" = 3
			"irons_spellbooks:cryomancer_chestplate" = 3
			"irons_spellbooks:shadowwalker_chestplate" = 3
			"irons_spellbooks:priest_chestplate" = 3
			"irons_spellbooks:plagued_chestplate" = 3
			"irons_spellbooks:wandering_magician_leggings" = 2
			"irons_spellbooks:pumpkin_leggings" = 2
			"irons_spellbooks:pyromancer_leggings" = 2
			"irons_spellbooks:electromancer_leggings" = 2
			"irons_spellbooks:archevoker_leggings" = 2
			"irons_spellbooks:cultist_leggings" = 2
			"irons_spellbooks:cryomancer_leggings" = 2
			"irons_spellbooks:shadowwalker_leggings" = 2
			"irons_spellbooks:priest_leggings" = 2
			"irons_spellbooks:plagued_leggings" = 2
			"irons_spellbooks:wandering_magician_boots" = 1
			"irons_spellbooks:pumpkin_boots" = 1
			"irons_spellbooks:pyromancer_boots" = 1
			"irons_spellbooks:electromancer_boots" = 1
			"irons_spellbooks:archevoker_boots" = 1
			"irons_spellbooks:cultist_boots" = 1
			"irons_spellbooks:cryomancer_boots" = 1
			"irons_spellbooks:shadowwalker_boots" = 1
			"irons_spellbooks:priest_boots" = 1
			"irons_spellbooks:plagued_boots" = 1

Screenshots

Minecraft version
1.19.2
Mod version
propertymodifier-1.19.2-0.3.3
Forge version
1.19.2-forge-43.2.3

I also found that the defense section uses the wrong syntax.
it says defense {} instead of [items.armor.defense], which prevents it from loading unless you change it.


China Lake Buff

About the PR
Screenshots

Changelog

:cl:

tweak: The China Lake's projectiles now have smaller explosions.
tweak: The China Lake bundle no longer comes with flash grenades.

You mean nerf

You mean nerf

you weren't able to use them indoors without killing yourself before so

Sounds like a skill issue

Pretty sure higher direct damage and less friendly fire capabilities qualifies as a buff

no flash nades
Life sucks, no more funzies.


Updated navbar on the shop page for responsive view.

Fixed the navbar on the shop page as per the issue #319
Now the navbar is properly aligned on the shop page and also added with other styling.

Fixes #319


Firestore for database + Chat SDK for chat in my App

Requirement
I have a requirement to integrate chat functionality in my android app. This app has firestore backend with several collections like - Users, cities, etc.

What I have done so far
User Signup / login :
I use firebase for Google login /signup and create user in Firestore database.
For Chat : I have used following from examples -

public void authenticateWithCurrentFirebaseLogin () {
       Disposable d = ChatSDK.auth().authenticate().subscribe(() -> {
           User currentUser = ChatSDK.currentUser();
           createThread("Test Chat",currentUser);
       }, throwable -> {
           Toast.makeText(this, "Error while authenticating with Chat SDK.",
                   Toast.LENGTH_SHORT).show();
       });
   }

This creates a User in Realtime database and starts a thread that results in opening a chat window. But, I think this is not the right way.

Android Version: 8.0.0

Questions:

How can I correctly execute chat screen between the current logged in user and selected user from list of Users(from firestore)?
How do I load previous chat each time I come to the same user again?
Thanks,
Kanak

Hi @kanaksony,
What you have done so far looks to be correct. The only issue is that you should be passing in another user to the conversation. As it is, you are creating a chat with only one user which isn't valid.
You will need a way to discover the other user's entity ID. The Chat SDK provides a method to do by searching for a user then adding them to your contacts. But you could also create your own way. Once you have the user ID, you can create a chat given that ID like this:
public Single<Thread> createPrivateChatWithUser (String userEntityID) {
        return ChatSDK.thread().createThread("", ChatSDK.db().fetchOrCreateEntityWithEntityID(User.class, userEntityID));
}

You can then open that thread like this:
public Completable startPrivateChatActivityWithUser (Context context, String userEntityID) {
    return createPrivateChatWithUser(userEntityID).doOnSuccess(thread -> {
        // Start the chat activity
        ChatSDK.ui().startChatActivityForID(context, userEntityID);
    }).ignoreElement();
}

@bensmiley This looks what I need. I will try it my morning and will update you. :)

Hi Ben, I used following functions -
For fetching the other User -
    public void getUserWithEntityID(String entityID) {
        chatUser = ChatSDK.db().fetchOrCreateEntityWithEntityID(User.class, entityID);
    }

For creating chat thread with above User -
 public Single<Thread> createPrivateChatWithUser () {
     return ChatSDK.thread().createThread(username, chatUser );
 }

Starting Private chat -
  public Completable startPrivateChatActivityWithUser () {
        return createPrivateChatWithUser()
                .doOnSuccess(thread -> {
            **// This code never calls
            ChatSDK.ui().startChatActivityForID(BusinessProfileDetailActivity.this, user_id);**
        }).ignoreElement();
    }

Please correct me if I am doing something wrong above.

@kanaksony This will only work if the ID of the Firestore user is the same as the Entity ID used by the Chat SDK. If they are not the same, when a user logs in, you would need to save their Chat SDK Entity ID to Firestore, and then read this value.

Thanks @bensmiley this worked.

Great!


Tooltip opens save dialogue

Version of Archi

v4.9.3
Archi Plug-ins

None
Operating System

GNU Linux Ubuntu 22.04.1 LTS
Expected Behaviour

Tooltips should pop-up when I hover over icons and objects
Actual Behaviour

A "save" window opens offering me to save the <icon-type>.html page to disk.
Steps to Reproduce the Behaviour

Download Archi-Linux64-4.9.3.tgz
Decompress to folder and run Archi application
Hover mouse over Palette icons

There are no error logs and this happens with all .archimate files.

Are you running Ubuntu under X11 or Wayland?

How would I tell?

I think the default for Ubuntu is Wayland. You need to run Archi with export GDK_BACKEND=x11 as in the Archi-Ubuntu.sh file included.

Do you have the "Hints" tab open or closed. Hints are stored in html files like the name in your screenshot.

Thanks for the quick feedback.
The hints tab was closed.
It was all tooltips, in the pallet, on the tree, on the diagram, etc.
Even though the command above suggested I was running under x11, changing my desktop shortcut to run the Archi-Ubuntu.sh shell script appears to have fixed the issue for tooltips.
But since you mentioned the hints tab, if I have the hints tab open the same issue occurs - this time you can see the tooltip and the save dialogue.

The Hints tab is displaying a local HTML page when you hover over the palette, or select a component in the Tree or in a diagram.
The Hints tab is basically an embedded WebKit browser. When it displays a hint it sets the embedded browser URL to something like file:///<pathto>/hintfile.html This is standard practice for displaying a local file URL.
As to why it is opening a save dialog I have no idea. If you can offer any further clues that would be helpful, at this stage there's nothing I can change in Archi's code.

FWIW, I installed a fresh install of Ubuntu <IP_ADDRESS> and didn't see this issue. Perhaps different versions of Webkit? Configuration? Permissions?

This was a fresh install of Ubuntu 20.04 about a month ago, although the /home partition was preserved from a previous install so $HOME/.local and other places configuration is kept will have been preserved from previous years of running various Ubuntus.
I'm new to Archi, but I have never seen the issue - downloading instead of displaying web pages - in other Java based GUI apps.
I don't know where to find any config which might change how webkit behaves, but is this useful in any way?
$ dpkg -l|grep webkit
ii  gir1.2-webkit2-4.0:amd64                   2.36.6-0ubuntu<IP_ADDRESS>                 amd64        Web content engine library for GTK - GObject introspection data
ii  libwebkit2gtk-4.0-37:amd64                 2.36.6-0ubuntu<IP_ADDRESS>                 amd64        Web content engine library for GTK

I have the same webkit versions.
I start Ubuntu when logging in with the option Ubuntu on XOrg available from the bottom right corner of the log-in screen. I tried Archi with this and using Wayland, with and without the Archi-Ubuntu.sh script and all is OK.
Did you try with the Hints tab closed? It shouldn't trigger that dialog.

I think I may have found what's happening here, assuming this only happens with Hints files:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=534113
It concerns the mime type of the html file which may be reported as application/xhtml+xml.  The solution is to fix that mime type association (I don't know how).

Actually, the above may not be the solution here. I was confusing Archi's help files (which use XHTML) with the Hints files which are normal HTML. But there may be a clue there.

You could try this command:
file --mime-type <pathto>/business_actor.html

<pathto> is the location of Archi's help plugin which is:
Archi/plugins/com.archimatetool.help_<IP_ADDRESS>203221030/hints

Currently:
With the hints tab closed and running the Archi-Ubuntu.sh script I see no issues.
With the hints tab open and running the Archi-Ubuntu.sh script I see this issue.
Tools -> Preview HTML Report main menu item causes four save dialogues to appear.
As far as I can see, no preview report tab opens.

Both the hints tab and the preview report use the webkit browser with a local url set to file:///path_to/somefile.html So this means that webkit is not seeing these files as HTML files. Might be a mime type association as mentioned above.

With the hints tab closed and running the Archi-Ubuntu.sh script I see no issues.

I don't think using Archi-Ubuntu.sh will make any difference. It's the Hints tab being open that causes this.

$ file --mime-type $HOME/Applications/Archi/plugins/com.archimatetool.help_<IP_ADDRESS>203221030/hints/business_actor.html
/home/mdsh/Applications/Archi/plugins/com.archimatetool.help_<IP_ADDRESS>203221030/hints/business_actor.html: text/html

Looking at the comments in https://bugs.eclipse.org/bugs/show_bug.cgi?id=534113 I traced the internal Eclipse WebKit code. It seems that a decision to either display the html file or show a download dialog is based solely on how that Eclipse code determines the mime type of the file. As this issue has not been reported before it's likely that a WebKit config of mime types is different on your system. I don't know how to investigate that further.

I'll close this one as there's nothing we can change in the Archi code.


haskellPackages.haskell-language-server: fix build

Required after the switch to GHC 9.2.4 in hackage2nix.
Note that the situation with Cabal flags is not optimal: since flags may add or remove dependencies (e.g. here the auto flag controls whether ghc-lib-parser is included if GHC 9.2 is used), we should tell hackage2nix that we want to disable the auto flag instead of doing it in configuration-common.nix. Then it could add the correct dependencies and also add the flag to the generated configureFlags itself.
EDIT: found https://github.com/NixOS/cabal2nix/blob/master/cabal2nix/src/Distribution/Nixpkgs/Haskell/FromCabal/Flags.hs, should we add | name == "ghc-lib-parser-ex" = [disable "auto"] in there?

Thank you! I combined this with another commit and pushed all of this to haskell-updates.