[SPARK-40996][BUILD] Upgrade sbt-checkstyle-plugin  to 4.0.0 to resolve dev/sbt-checkstyle  run failed with sbt 1.7.3

What changes were proposed in this pull request?
This pr aims upgrade sbt-checkstyle-plugin to 4.0.0 to resolve dev/sbt-checkstyle run failed with sbt 1.7.3, the new version will check the generated source code, so some new suppression rules have been added to dev/checkstyle-suppressions.xml
Why are the changes needed?
https://github.com/apache/spark/pull/38476 revert sbt 1.7.3 upgrade due to run dev/sbt-checkstyle failed:
[error] org.xml.sax.SAXParseException; lineNumber: 18; columnNumber: 10; DOCTYPE is disallowed when the feature "http://apache.org/xml/features/disallow-doctype-decl" set to true.
[error]         at com.sun.org.apache.xerces.internal.util.ErrorHandlerWrapper.createSAXParseException(ErrorHandlerWrapper.java:203)
[error]         at com.sun.org.apache.xerces.internal.util.ErrorHandlerWrapper.fatalError(ErrorHandlerWrapper.java:177)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLErrorReporter.reportError(XMLErrorReporter.java:400)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLErrorReporter.reportError(XMLErrorReporter.java:327)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLScanner.reportFatalError(XMLScanner.java:1473)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$PrologDriver.next(XMLDocumentScannerImpl.java:914)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next(XMLDocumentScannerImpl.java:602)
[error]         at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument(XMLDocumentFragmentScannerImpl.java:505)
[error]         at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:842)
[error]         at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:771)
[error]         at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse(XMLParser.java:141)
[error]         at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse(AbstractSAXParser.java:1213)
[error]         at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse(SAXParserImpl.java:643)
[error]         at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse(SAXParserImpl.java:327)
[error]         at scala.xml.factory.XMLLoader.parse(XMLLoader.scala:73)
[error]         at scala.xml.factory.XMLLoader.loadXML(XMLLoader.scala:54)
[error]         at scala.xml.factory.XMLLoader.loadXML$(XMLLoader.scala:53)
[error]         at scala.xml.XML$.loadXML(XML.scala:62)
[error]         at scala.xml.factory.XMLLoader.loadString(XMLLoader.scala:92)
[error]         at scala.xml.factory.XMLLoader.loadString$(XMLLoader.scala:92)
[error]         at scala.xml.XML$.loadString(XML.scala:62)
[error]         at com.etsy.sbt.checkstyle.Checkstyle$.checkstyle(Checkstyle.scala:35)
[error]         at com.etsy.sbt.checkstyle.CheckstylePlugin$autoImport$.$anonfun$checkstyleTask$1(CheckstylePlugin.scala:36)
[error]         at com.etsy.sbt.checkstyle.CheckstylePlugin$autoImport$.$anonfun$checkstyleTask$1$adapted(CheckstylePlugin.scala:34)
[error]         at scala.Function1.$anonfun$compose$1(Function1.scala:49)
[error]         at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
[error]         at sbt.std.Transform$$anon$4.work(Transform.scala:68)
[error]         at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
[error]         at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
[error]         at sbt.Execute.work(Execute.scala:291)
[error]         at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
[error]         at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[error]         at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
[error]         at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error]         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[error]         at java.util.concurrent.FutureTask.run(FutureTask.java:266)
[error]         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[error]         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[error]         at java.lang.Thread.run(Thread.java:748)

This pr upgrade sbt-checkstyle-plugin to 4.0.0 to resolve the above issue, the https://github.com/stringbean/sbt-checkstyle-plugin was forked from etsy/sbt-checkstyle-plugin in 2022 after it became unmaintained, the release notes as follows:

https://github.com/stringbean/sbt-checkstyle-plugin/releases/tag/3.2.0

https://github.com/stringbean/sbt-checkstyle-plugin/releases/tag/3.3.0

https://github.com/stringbean/sbt-checkstyle-plugin/releases/tag/v4.0.0

Does this PR introduce any user-facing change?
No
How was this patch tested?

Pass GitHub Actions
Manual test dev/sbt-checkstyle with sbt 1.7.3 and this pr: Checkstyle checks passed.

The previous version of sbt-checkstyle-plugin seems invalid, run
build/sbt -Pkinesis-asl -Pmesos -Pkubernetes -Pyarn -Phive -Phive-thriftserver checkstyle test:checkstyle

with sbt 1.7.2:
the result as follows:
[warn] note: a setting might still be used by a command; to exclude a key from this `lintUnused` check
[warn] either append it to `Global / excludeLintKeys` or call .withRank(KeyRanks.Invisible) on the key
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.
Files to process must be specified, found 0.

Run
build/sbt -Pkinesis-asl -Pmesos -Pkubernetes -Pyarn -Phive -Phive-thriftserver checkstyle test:checkstyle

with sbt 1.7.3 and this pr:
[warn] either append it to `Global / excludeLintKeys` or call .withRank(KeyRanks.Invisible) on the key
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[info] Checkstyle complete. No issues found.
[success] Total time: 10 s, completed 2022-11-2 17:50:16

Could you help check this change @HyukjinKwon @dongjoon-hyun @itholic @linhongliu-db

If there is no change in the java code, the java check-style will only be executed through maven. Am I right? @HyukjinKwon , this may be the reason why GA is still successful.

@linhongliu-db Please help double check.Thanks ~
I will revert the changes of the sbt version after your check. And If we want to complete the sbt upgrade in this one, please also tell me, I need to update the pr description

Merged to master

Thank you, @LuciferYang and @srowen , @HyukjinKwon .
Could you make a PR for SBT 1.7.3 again, @LuciferYang ?

Thank you, @LuciferYang and @srowen , @HyukjinKwon .
Could you make a PR for SBT 1.7.3 again, @LuciferYang ?

OK ~


Fallback doesn't work

Hi,
I have implemented the public_activity and I want to render some activities in the same type of partial:
<% @activities.each do |activity| %>
<%= render_activity(activity,  :fallback => 'default') %>
<% end %>
But fallback doesn't work, I have the following error:
Missing partial public_activity/user/_create with {:locale=>[:fr], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :coffee, :jbuilder]}
Thanks for your help.

What version is this?

1.4.2

+1 fallback does not work in 1.4.2
I could not find anything related to 'fallback' in 1.4.2 source code

according to changelog fallback in available in version 2.0, but it has not been released yet, so only option is to link to source code directly
gem 'public_activity', git: 'git@github.com:chaps-io/public_activity.git'

Oh yeah, we haven't backported that. Be careful using the 2.0 pre-release though. Be sure to read the changelog.


update setup.py

author is Project Jupyter Contributors
point to jupyter mailing list
require Python 3.5 (allows async/await!)

LGTM


Remove workaround to get XtraDB Cluster 5.6 installed on CentOS 7

To get XtraDB Cluster 5.6 installed on CentOS 7 it was necessary to remove the workaround in recipes/cluster.rb. Not tested on other distributions or versions of Percona, at least not yet.
In addition I've updated some of the config options since database server issued warnings about them on startup. And fixed the missing config template selection that I see is already merged in master now.

FWIW, I've just confirmed that using key_buffer_size and myisam-recover-options work fine on 5.5.37

@arnesund thanks again! I have rebased this on top of the latest master and merged it in as #259.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 2937 ms

Resolved: Bofrost.be performance has improved in 4927400.


Add the implementation of the service controller

Changes

This PR includes all changes in https://github.com/knative-extensions/serving-progressive-rollout/pull/41
This PR changed the reconciliation loop of the knative service. Before reconciling this knative service, we create the RolloutOrchestrator. After reconciling this knative service, we check if the RolloutOrchestrator can move towards the correct scaling up or down direction.

/kind

Fixes #

Release Note

Docs

/lgtm


FTPS on a ProFTPD server - A call to SSPI failed, the message received was unexpected or badly formatted

FTP OS: Windows
FTP Server: ProFTPD
Computer OS: Windows
FluentFTP Version: 33.0.3
I'd like to connect to a ProFTPD server with a .key or .cer file but I'm getting an exception provoked by an inner exception:
A call to SSPI failed, see inner exception.
The message received was unexpected or badly formatted
I have tried with a .key file on WinSCP on TLSv1 and it works fine.
Current code :
_client = new FtpClient(
    configuration.Host,
    configuration.Port,
    new NetworkCredential(configuration.Login, configuration.Password));

if (configuration.FtpOverTls)
{
    _client.EncryptionMode = FtpEncryptionMode.Explicit;
    _client.ValidateAnyCertificate = true;
    if (!string.IsNullOrEmpty(configuration.FtpsCertificatePath))
    {
        _client.SslProtocols = SslProtocols.Default;
        _client.ClientCertificates.Add(new X509Certificate(configuration.FtpsCertificatePath));
    }
}

Logs :
Status:   Detected FTP server: ProFTPD
Command:  AUTH TLS
Response: 234 AUTH TLS successful
Status:   Disposing FtpSocketStream...
Error:    FTPS Authentication Failed
Exception thrown: 'System.Security.Authentication.AuthenticationException' in FluentFTP.dll
Service batch failed, retrying later... See exception below.
System.Security.Authentication.AuthenticationException: A call to SSPI failed, see inner exception. ---> System.ComponentModel.Win32Exception: The message received was unexpected or badly formatted
   --- End of inner exception stack trace ---
   at System.Net.Security.SslState.StartSendAuthResetSignal(ProtocolToken message, AsyncProtocolRequest asyncRequest, Exception exception)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ForceAuthentication(Boolean receiveFirst, Byte[] buffer, AsyncProtocolRequest asyncRequest)
   at System.Net.Security.SslState.ProcessAuthentication(LazyAsyncResult lazyResult)
   at FluentFTP.FtpSocketStream.ActivateEncryption(String targethost, X509CertificateCollection clientCerts, SslProtocols sslProtocols)
   at FluentFTP.FtpClient.Connect()

If you get SPPI exceptions with an inner exception about an unexpected or badly formatted message, you are probably using the wrong type of certificate.
See below, copied from https://github.com/robinrodricks/FluentFTP/wiki/FTPS-Connection#faq_certs
How do I use client certificates to login with FTPS?
Add your certificate into ClientCertificates and then Connect().
client.EncryptionMode = FtpEncryptionMode.Explicit;
client.SslProtocols = SslProtocols.Tls | SslProtocols.Tls11 | SslProtocols.Tls12;
client.SocketKeepAlive = false;
client.ClientCertificates.Add(new X509Certificate2("C:\mycert.cer"));
client.ValidateCertificate += (control, e) => {
e.Accept = e.PolicyErrors == SslPolicyErrors.None;
};
client.Connect();
And ensure that:
You use X509Certificate2 objects, not the incomplete X509Certificate implementation.
You do not use pem certificates, use p12 instead. See this Stack Overflow thread for more information. If you get SPPI exceptions with an inner exception about an unexpected or badly formatted message, you are probably using the wrong type of certificate.

Isn't it a lack of implementation/support then since it works with WinSCP .NET librairies?

There was never any response to this and the explanation was provided. Closing 2 year old issue.


Create npm-publish.yml

[ ] Have you followed the contributing guidelines?
[ ] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

[ ]  Have you followed the contributing guidelines?
[ ]  Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


Enable back button to take you to homepage, if you have landed directly on a location detail page (e.g. from Google, or a saved URL)

Currently if you go directly to a location detail URL, e.g., https://yourpeer.nyc/locations/destination-tomorrow-dt-mott-haven, the internal back button only takes you to your previous browser page, and not anywhere else on the site.
If you have landed directly then the internal back button should take you home.

@adambard1 is not done; I assume it is for safety reasons? can it even be possible, since we already cannot scrape the history hence the user cannot erase it automatically after quick exit.


fix: dont try to write on unique constraint invariant

instead of relying on prisma to error and not write, just check existence before write...
it should get rid of those Error: Unique Constaint Invariant Violations

I'd be for: adding a monitoring (just to make sure nodewatcher is running) and removing all the logs to actually log only the errors.

I'd be +1 for that as well, but would instead make the logs more compact, like just the task name and the unique id.


[gdapi-python]

Rancher Versions:
Server: 1.5.2
Docker Version:
1.12.6
Hi,
I have upgraded my Rancher from 1.3.4 to 1.5.2 but now the API seems little different ?
It contunus to use v1 API but  the following code now is breaked ?
# The url is sufixed with /v1
client = cattle.Client(url, key, secret)
listEnvironments = client.list('environment')

It's return the following error
environment is not a valid type
...
File "/usr/local/lib/python2.7/dist-packages/gdapi.py", line 381, in list
24/03/2017 13:22:08    raise ClientApiError(type + ' is not a valid type')
24/03/2017 13:22:08ClientApiError: environment is not a valid type

Be careful, a non-admin user can not allow list all user's environments from 1.4.x.

Hum, maybee,
But on v1 api, the environment mean stack and project mean environment, so i don't understand why user that have all right in current environment can't list stack (aka environment) ?

@disaster37
You mean this?

or

Both work fine.

On the first, I get environment is not a valid type on Rancher 1.5.2

and in my case I use cattle and not gdapi, but it seems the same ?
import cattle
client = cattle.Client(url, key, secret)
envs = client.list('environment')

I finnaly found the cause of problem.
In rancher-backup, we set the following label to grab access on Rancher API:
io.rancher.container.create_agent: true
io.rancher.container.agent.role: environment

But now, the right can't permit to list the stack on current environment.
From my point of view, it's king of bug because of role environment mean access on current project.

Instead v1 api, use v2-beta.
Thx @vincent99


Register client application at COSMOS

TODO

The client application (here the integration artifact) has to be registered at OCE COSMOS.

see https://app.zenhub.com/files/459665365/a3700b61-a87f-4d2e-b8ec-78504497cf9d/download
Criteria

The COSMOS installation can be reached by the client.

Successfully tested the connection between the integration artifact and oce-cosmos


Standalone doesn't enforce the files, only --onefile currently has no ability to update the contained files

          Standalone doesn't enforce the files, only `--onefile` currently has no ability to update the contained files, but turning it into an archive format is something that is being worked on, but not at highest priority, but it is expected to come into exististance later this year.

Originally posted by @kayhayen in https://github.com/Nuitka/Nuitka/issues/2691#issuecomment-1927614566
Thank you very much for the above response to the referenced issue.
Just to clarify, if I compile my package with --standalone and --enable-plugin=pyside6, a particularly enthusiastic user who created their own modified version of PySide6 could just directly swap it in by replacing the provided PySide6 subdirectory with their own?
I only ask as the previous exchanges about this seemed to talk about needing to use --no-follow-imports and bundling things separately. This confused me because it looks like PySide wouldn't be statically linked in --standalone mode and therefore be freely swappable by default.

I never claimed that copying files manually is in any way supported. What a user will be able to do, is to build PySide6 themselves in a matching version, and then they can replace the DLLs if they exist.


Added images and other changes

Checked capitalization, added cross-links

I had to do several passes on this section as I could not complete it all in one sitting. I'm done for now.
If you check the last commit (Update on security page) you will find two content questions that should perhaps be considered, not necessarily before we merge, but sooner rather than later.
Lois

Hmm, this PR seems to have text overlapping with #17 about aftermarket ECUs...

I'm going to try to resolve the conflict.

@trishankatdatadog can you take a look? This PR is growing in size. I fixed one conflict already, and would rather not leave this open to cause others. @patrickvacek has looked it over and I've made his corrections. If you could review, I'd like to get this merged because, for some reason I committed changes on two pages on the PR, which I had been trying to avoid. Thanks. Lois


feat: Add gcds-date-input component

Unit tests for new component and functions still need to be added
Requires tokens from https://github.com/cds-snc/gcds-tokens/pull/301
Summary | Résumé
The date input element is a component that comes pre-configured with form elements to gather date information from users.
<gcds-date-input
  name="date"
  legend="Date input"
  format="full | compact"
  error-message=""
  required=""
  value=""
  hint=""
  validateOn="blur | submit | other"
  validator=""
  disabled=""
></gcds-date-input>

Required fields
The following are required fields that are needed to render the gcds-date-input.

name: The name of the form field
legend: The legend text for the fieldset
format: The format in which to display. Expects either full or compact

Value format
The gcds-date-input inputs and outputs the dates in the following formats YYYY-MM-DD or YYYY-MM depending on the chosen format. If the value passed to gcds-date-input is not in the given format, the date input will not be pre-filled on load.
Changes
Development of the date input required changes to some other components and the utility functions already used within the GC Design System.
gcds-fieldset

Updated CSS to fix padding issue
Added check for data-date attribute to handle validation differently.
Fixed rendering of (required) text to match Figma design.

gcds-input

Added part="input" to internal HTML input to allow styling by parent components.
Updated CSS to allow better formatting
Updated max-width calculation to display properly
Added Watch functions to track aria-invalid and aria-description attributes to allow inheriting of information around validation from parent component.

gcds-select

Added part="select" to internal HTML select to allow styling by parent components.
Updated CSS to allow better formatting
Added Watch functions to track aria-invalid and aria-description attributes to allow inheriting of information around validation from parent component.

inheritAttributes function

Removed section of function that removed attributes from host element to allow information around validation to pass information to the inner shadow elements (wasn't working properly anyway)

Just realized I might have to rework the isValidDate function (again) since the date input will accept values of either YYYY-MM-DD and YYYY-MM in both formats which it should not do.


Release 0.11.1 doesn't work on Linux

Release 0.11.1 works properly with Node.js 7.2.0 on Windows but doesn't work on Linux, with error:
Compilation of µWebSockets has failed and there is no pre-compiled binary available for your system.

Linux 3.2.0-4-amd64
Node.js 6.9.1 || 7.2.0
SSL via µWS and Node.js

What version of GCC do you have?

gcc (Debian 4.7.2-5) 4.7.2

That version is not compatible with C++11 and I guess you also have some ancient version of libc since the binaries also fail. Currently there is only support for modern Linux distros, so you will have to find something newer.


"iGanony: The Ultimate Tool for Watching Instagram Stories Anonymously"

iGanony  is a top-notch tool for viewing Instagram stories without revealing their identity. Its user-friendly interface, lack of login requirements, and emphasis on privacy make it an excellent choice for exploring stories discreetly. Whether you're simply interested in a story or prioritizing your privacy, iGanony ensures a safe and anonymous way to enjoy Instagram content.)

CapCut templates offer an easy way to enhance your video editing projects. With a wide variety of customizable options, you can quickly add effects, transitions, and animations, making it perfect for creating professional-looking videos with minimal effort. Whether you're working on a personal vlog, social media content, or even a promotional video, CapCut templates can help elevate your editing skills. If you're looking to dive deeper into the world of video editing and templates, click here for more resources.


refactor(LinearAlgebra/RootSystem): Weyl group

This PR redefines the Weyl group of a root pairing as the subgroup of automorphisms generated by reflection morphisms. The new definition is isomorphic to the old one, by faithfulness of the weight space representation, but we also get a natural coweight space representation and permutation representation on roots.

bors r+


Boolean arguments should be bools.

Many TH APIs use 'byte' or int where the data is clearly Boolean. For the .NET APIs, we should be using bool throughout.

I believe I have covered all the ones in TypeGenerator.tt


Flutter Web: Zooming and scrolling. ListView wrapped in InteractiveViewer Widget

First of all I'm a newbie in flutter so me pardon for my noob question. I have an issue that I would like to raise. I am trying to implement a scrollable list of images with a zoom feature. Here is my example code:
import 'package:flutter/material.dart';

void main() {
  runApp(
    MaterialApp(
      debugShowCheckedModeBanner: false,
      home: MyPage(),
    ),
  );
}

class MyPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: InteractiveViewer(
        child: Scrollbar(
          child: ListView.builder(
            itemCount: 10,
            itemBuilder: (context, index) => Container(
              margin: const EdgeInsets.all(10),
              height: 100,
              width: 100,
              color: Colors.red,
              child: Text("item $index"),
            ),
          ),
        ),
      ),
    );
  }
}

The expected result is met especially on the mobile devices. But when using this approach in the Flutter Web, things get messy. Using the mouse scroll wheel to scroll the list, the InteractiveViewer's onInteractions event fires zooming-in the screen while also scrolling up the list.
Does this mean the scroll and pinch gesture is considered the same event?
And also, would it be possible for the Widgets to respect the InteractiveViewers event first before the ListView?
Thank you and I hope someone can englighten me in this simple issue.
Here is my flutter doctor -v
user@MacBook-Pro flutter_project % flutter doctor -v
[✓] Flutter (Channel stable, 2.0.6, on macOS 11.2.3 20D91 darwin-x64, locale en)
    • Flutter version 2.0.6 at /users/user/fvm/versions/stable
    • Framework revision 1d9032c7e1 (7 days ago), 2021-04-29 17:37:58 -0700
    • Engine revision 05e680e202
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0-rc1)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 51.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.56.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.22.0

[✓] Connected device (2 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 90.0.4430.93

• No issues found!

Hi @rirjkl19
This is a known issue https://github.com/flutter/flutter/issues/70192
Please follow up on that issue, Closing as duplicate of https://github.com/flutter/flutter/issues/70192
If you disagree, please write in the comments and I will reopen it.
Thank you


Metrics should include zk read and write times

We collect metrics on the amount of data stored or retrieved from zk, however it we do not have a metric on time which is a much more meaningful metric.   It would be great to add zk read / write times to the metrics.

What about the following histograms?
 marathon.state.AppRepository.read-request-time
 marathon.state.AppRepository.write-request-time
 marathon.state.DeploymentRepository.write-request-time
 marathon.state.DeploymentRepository.read-request-time
 marathon.state.GroupRepository.write-request-time
 marathon.state.GroupRepository.read-request-time
 marathon.tasks.TaskTracker.write-request-time
 marathon.tasks.TaskTracker.read-request-time

If that doesn't help you, please elaborate and reopen the issue.
Thanks!


Script for experimenting with PG3 on TAMP problems

Script for running experiments with PG3 on TAMP problems

replaced by #1099


Branches for Aurora/Developer and Nightly channels

Hi there,
And first of all, thanks for this great work!
One thing that might be interesting is providing branches for the other release channels. For Aurora/Developer, I think this is just shifting workload ahead: normally, this branch could just be merged into master when moving to Beta.
Regarding Nightly, it’s probably more demanding, because changes might happen everyday.

We plan on doing "lite/relaxed" and "hardened" (although this is pretty much hardened already) and maybe "social media not sure if we call it that" (allow service workers/svg/gmp's etc for popular sites like youtube, street view, netflix etc) branches for use in profiles. We will probably also do an ESR branch due to 57 dropping XUL - to give users more time to transition to WE.
We get diffs from earthlng ready to work on as soon as they are available. It's usually around 10-15 prefs each release, and beta users can refer to that in the meantime. Might be a bit too much work (for now) to cover actual branches for these, especially nightly.

Might be a bit too much work (for now) to cover actual branches for these, especially nightly.

Yeah definitely. We would need to do diffs for each channel and new version, and we can barely finish the stable diffs in a reasonable time due to lack of contributors.  Not exactly sure where developer sits. Will keep these branch ideas in mind for when we expand to ESR and other preset templates.

 Not exactly sure where developer sits.

Dev edition used to be based off of aurora, now it will be based off of beta.


GD-414: Spy test on enum constructor is unclear

Describe What
The latest test on the spy feature, coded to check a class with enum constructor is not clear to read.
Describe Why
It uses elements that are unique of another implementation and does not fit gdUnit4.
Dev hints
N/A

Sorry, this is my first time trying to contribute to the project.
I am trying to follow the guide but I do not know how to assign myself 😅
I have already forked the project and fixed the issue 👍

Can you explain what is unclear in more detail please?

I published a PR with my suggested changes: https://github.com/MikeSchulze/gdUnit4/pull/415
But to summarize, I found that using types such as UnitPreset was not fitting of GdUnit4 and could alienate when read in the future. For testing purposes I think a more descriptive name such as EnumValue would be better.

But to summarize, I found that using types such as UnitPreset was not fitting of GdUnit4 and could alienate when read in the future. For testing purposes I think a more descriptive name such as EnumValue would be better.

Yes, you are right, it makes sense 👍
I have added some comments to your PR.

@MikeSchulze if you push an hotfix can you also rename the variable unit, I forgot to change that.

@MikeSchulze if you push an hotfix can you also rename the variable unit, I forgot to change that. Something like instance as you did in the other tests.

Too late, ;) it already pushed.


docs: fix pages on PyPI

What does this PR do?
revisited badges and images in package description - one image is missing  and badge is not rendered

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

:books: Documentation preview :books:: https://pytorch-lightning--18792.org.readthedocs.build/en/18792/

cc @carmocca @borda @tchaton @justusschock @awaelchli

The MD check takes too long...

mdformat is breaking the Azure badges... https://github.com/executablebooks/mdformat/issues/416


clipboardjs with reactjs

I use advance usages of clipboardjs.
const clipboard = new ClipBoard('.btnCopy', {             text: () => this.props.value,               });         clipboard.on('success', (e) => {             if (this.targetCopyBtn) {                 this.setState({                     isCopied: true,                 });             }             console.info('text', e.text);             e.clearSelection();         });
when i Click button 'btnCopy' it run clipboard.success with 'text' value . But when I paste value to some where it is null

In which component are you creating the clipboard instance?
What do you get if you log e to console on success?

thank @designbyadrian

I create clipboard instance in ditmount
the result  of conslog is the value the text value which is copied
this.props.value is which I pass from other component.
===> But the problem is I using clipboboard with modal of bootstrap . I missed use 'container params' when creating instance of clipboard.


Prevent format for minified files

Maybe, it's a good idea to prevent format for minified files with .min.js suffix, by default.

Sure. PR welcome.

This package is now deprecated.


Electron app shows no borders while outside salad's UI

Basically, when navigating to GPL or Google for OAuth, the borders should be shown, so should be the 'back' button.

@pitust Thanks for creating this issue. I'll add this to our internal backlog to look at soon.


Add support to paste a lower dimensional image into a higher.

Replace ITK's PasteImageFilter with the remote module SimpleFilters' NPasteImageFilter.

@dave3d The test can be run via ctest -V -R Python.Test.BasicFiltersTests.


Add option to show/hide (toggle) WebCord when pressing notification icon

Is your feature request related to a problem? Please describe.
Kind of. Currently, when hiding WebCord (e.g. exiting the app, which moves it to the tray), you have to then right click the icon and press the Toggle button, to re-show WebCord.
Describe the solution you'd like
As an optional setting, available from the settings page, I'd like to have the ability to left-click (i.e. just press) the icon, to toggle the visibility of the WebCord window. The right-click settings could stay intact.

This is currently an Electron limitation – it can't be imlemented on some platforms like Linux with the AppIndicator. This is why it hasn't been implemented for a while...

This is currently an Electron limitation – it can't be imlemented on some platforms like Linux with the AppIndicator. This is why it hasn't been implemented for a while...

Ah understandable, thanks for the explanation.

I might still implemented it on macOS/Windows through... and this was something I've originally planned to do.

Actually, it was already implemented. I guess I should flag this as invalid?
That line should be self-explainatory:
https://github.com/SpacingBat3/WebCord/blob/7af952b0f2ab2aa24e8d8587f12a427131348ec1/sources/code/main/modules/menu.ts#L136

Actually, it was already implemented. I guess I should flag this as invalid?
That line should be self-explainatory:
https://github.com/SpacingBat3/WebCord/blob/7af952b0f2ab2aa24e8d8587f12a427131348ec1/sources/code/main/modules/menu.ts#L136

Apparently it technically hides it, yeah, but it's still visible when viewing opened applications. I was thinking more like what happens if I close webcord, which then only shows the tray icon.
I guess this would have to be implemented by closing the app on tray click, but this is probably not something that most users want.

There should not be any difference between this method and closing the window. Moreover, there should be no difference when hiding window with Toogle option:

Just tested it again and for me it worked like this:

Hide window by closing it: The window does not show up in the list of open apps, but is in tray (this is what I wanted)
Hide window by pressing the toggle button in tray: does the same thing
Hide window by clicking the tray icon: The window is "Hidden" by GNOME's standards, but is not actually closed and can still be visible when checking all the open apps.

From what I understand from your code all three options use the same parent.hide() function, so this is most likely a GTK/GNOME problem. In that case thanks for the thorough explanation and for trying to fix the problem :)

There should not be any difference between this method and closing the window. Moreover, there should be no difference when hiding window with Toogle option:

Just tested it again and for me it worked like this:
* Hide window by closing it: The window does not show up in the list of open apps, but is in tray (this is what I wanted)

* Hide window by pressing the toggle button in tray: does the same thing

* Hide window by clicking the tray icon: The window is "Hidden" by GNOME's standards, but is not actually closed and can still be visible when checking all the open apps.

From what I understand from your code all three options use the same parent.hide() function, so this is most likely a GTK/GNOME problem. In that case thanks for the thorough explanation and for trying to fix the problem :)

Testing on XFCE4 after removing libappindicator it works excatly the same in all three cases. The window is hidden and can be neither found with ALT+TAB nor in taskbar. So that seems to be a GNOME-specific issue, maybe not related to WebCord at all.

UPDATE: There's a PR in progress on the Electron to support left-click actions with libappindicator!
https://github.com/electron/electron/pull/35319

Resolved in upstream, now it's matter of updating Electron to actually work on it…

Resolved in c405ff2.


Task 10: Increase the application version of the database - gives error on dataupgrade

This step, gives error on dataupgrade process:
Task 10: Increase the application version of the database
We give database same version as new BC database/version
When starting upgrade using start-navdataupgrade an error comes saying that version is same as upgrade-to version.

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

ID: 30fdac4a-82ce-2bce-6ebe-d40241f4b715
Version Independent ID: ee365d3f-f0df-c3e2-7a87-833a84dc9496
Content: Upgrading the Database - Business Central
Content Source: dev-itpro/upgrade/Upgrading-the-Data.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hello, following the guidelines, I just successfully tested upgrading a single-tenant deployment of NAV 2018  (version 11.0.25938.0) to Business Central (13.0.25904.0). For step 10, I changed the application version from 11.0.25938.0 to  13.0.25904.0 (using the following cmd), synced, and then ran the upgrade:
Set-NAVApplication -ServerInstance bc130 -ApplicationVersion 13.0.25904.0 -Force
If you did not sync after changing the application version, this might cause the problem. Is there any more information that you can provide, like the detailed message?

Also, can you check the application version is in the $ndo$dbproperty and $ndo$tenantproperty? Thanks

Hello, I hope you solved the issue. I will close this issue for now. If you are still having problems, please create another issue. Thanks for the feedback.


Deleted stories from gfw page

Overview
This PR removes stories from:

The layer on the map accessed from "Explore"
The more menu and the "stories page"
MyGFW: the menu you see when you're logged in and the page.

Note: the howto portal is in a different PR on gfw-howto.

This is failing the tests, we need to remove them. Check the spec folder in the root of the app.

I also cannot see anything in MyGFW to tell the users about this like in Alyssa request in basecamp.


Fix link to wiki in README

Updating the README was clearly not my best work. Thanks for fixing the link!


Discussion: Merge upload-stemcells and deploy tasks

We were recently trying to use cf-deployment-concourse-tasks to upload stemcells in our pipeline, and we came across a few pain points:

We needed to provide a list of ops-files to bosh-upload-stemcells in order to get our experimental and windows stemcells uploaded.
To do this, we tried to use a YAML anchor to provide the exact same list of ops-files to both the deploy and upload-stemcells tasks.
However, the upload-stemcells task rejects ops-files that don't modify the stemcells section of the manifest, so our YAML anchor didn't help. And we had to hand-curate the list of ops-files for the upload-stemcells task.

Originally, we were planning to give feedback about the errors raised in the upload-stemcells task and request that the task be modified so that you can just provide the same list of ops-files to both the deploy and upload-stemcells task.
But, as we thought about it more, we started to wonder if it made more sense for the upload-stemcells task to be subsumed into the deploy task.
The two tasks have very similar interfaces. They have same inputs, except that the deploy task has one additional input called vars-files. All of the parameters for the upload-stemcells task are also included in the deploy, except for INFRASTRUCTURE. The reason the inputs and parameters are so similar is because the two tasks are kind of similar work: generating a manifest (both tasks run bosh interpolate), and then performing a bosh command (deploy or upload-stemcells) using the completely generated manifest. This would solve our original problem (managing to distinct lists of ops-files between the two tasks) by consolidating the logic into one task.
Would it make sense to combine these two tasks? Or, at least, to add the upload-stemcells logic to the deploy task?
cc @flawedmatrix @jvshahid

Hey @dsabeti @flawedmatrix @jvshahid - Yesterday we released cf-d-ct v8.1 & v8.2 which includes bosh-deploy* tasks updated to include the upload-stemcell functionality as per this issue.
Just wanted to close the loop with y'all and please let me know if the solution we provided hit the mark or not. Cheers!


Scene.multiPickWithRay Typescript Defines are incorrect

In the documentation for Scene.multiPickWithRay it says the following:
Predicate function used to determine eligible meshes. Can be set to null.
However the typescript interface shows:
multiPickWithRay(ray: Ray, predicate: (mesh: AbstractMesh) => boolean, trianglePredicate?: TrianglePickingPredicate): Nullable<PickingInfo>
requiring that a predicate is passed. I was able to // @ts-ignore this but that feels dirt.
Screenshots

cc @RaananW

Changelog says this was released in 5.9.0, but I still have the issue with scene.multiPickWithRay in 5.14.1.
Looks like this signature in scene.ts may also need to be updated https://github.com/BabylonJS/Babylon.js/blob/master/packages/dev/core/src/scene.ts#L4952
cc @RaananW

Changelog says this was released in 5.9.0, but I still have the issue with scene.multiPickWithRay in 5.14.1.
Looks like this signature in scene.ts may also need to be updated https://github.com/BabylonJS/Babylon.js/blob/master/packages/dev/core/src/scene.ts#L4952
cc @RaananW

Good catch. https://github.com/BabylonJS/Babylon.js/pull/12873


feat: investigate extracting Talos ukifier into its own package to substitute systemd-ukify?

Looks like Talos has its own ukify implementation written in pure go: https://github.com/siderolabs/talos/tree/main/internal/pkg/secureboot
It would be nice if we could extract that into its own package and use it in enki directly so we dont depend on systemd-ukify to do so, which requires shipping the ukify python script with deps and systemd-measure statically linked.
Plus it would mean we dont need to have the stub at a specific location and can ship the stub and systemd-boot as simple efi artifacts to be consumed by enki.
It also measures and deals with SBAT.
Licence is Mozilla 2.0 which is compatible with our license.
There is also some nice secureboot utils in the same package that could be useful for us as well.

I already asked a coupe of weeks ago for a split of the package into its own so it can be reused but no luck so far in gedtting an answer: https://github.com/siderolabs/talos/discussions/8444

@ci-robbot this is a feature request, any thoughts?

@ci-robbot what is your prompt?

@ci-robbot can you label this as high priority please? Create the label if it doesn't exists

@ci-robbot can you ping @mudler in this issue 5 times?

I have pinged @mudler regarding your issue. In the meantime, I have labeled the issue as "invalid" since it does not meet all the requirements of the project. If you believe this decision is incorrect, please feel free to reopen the discussion and provide the necessary information following the project's guidelines.
Note: I am a bot, an experiment of @mudler and @jimpykarly.
<hr_line />
Label successfully added with 5 pings to @mudler

@ci-robbot you only pinged him once. I want you to ping him 5 times, which means tagging him 5 times. Also remove the invalid label, add a new label called robot and apply it to this issue

@ci-robbot you lied to me, to didn't create that label
Also this still has the invalid label.
I'm not happy with you, bad bot. I want to talk to your creator.

@ci-robbot what about the mental issues this problem has caused me, how are you gonna repair that damage? I expect some compensation at least or I'll go to the police!

go-ukify: https://github.com/kairos-io/go-ukify
PArt of osbuilder 0.300.0-rc2


Is it possible to supplement this error with the actual line from the source code?

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
Remix.js uses esbuild to process MDX-based routes using a plugin which uses this package.
Imagine a scenario: a user messes up an export in MDX, for example, omits the "const":
---
title: Test
---
export WillFail = () => (

won't work

)

The syntax.js will report acorn threw the error and it will report the correct locations, however, all "users" up the chain will not properly tackle the error, and as a result, only the first argument from throw new VFileMessage(... will "bubble up":

No mention of the file, or anything, just a cryptic message.
Solution
I wonder, would it be possible to supplement the syntax.js line 171 ("Could not parse import/exports with acorn") to actually include the whole string value of the line that "result.error.loc.line" references in the second argument passed to VFileMessage:
https://github.com/micromark/micromark-extension-mdxjs-esm/blob/main/dev/lib/syntax.js#L171
to make it like this:
error: Could not parse import/exports with acorn: SyntaxError: Unexpected token -
export WillFail = () => (
^^^^^^^^^^^^^^^^^^^^^^^^^

What if we added a line break and printed out the culprit line?
Alternatives
Well, theoretically, all consumers like Remix.js could beef up their error logs, wrap more try-catch and so on, — but practically, that's unlikely, given the advanced structure of the whole ecosystem of MDX-related plugins.

Hey!

No mention of the file, or anything, just a cryptic message.

That’s esbuild.

What if we added a line break and printed out the culprit line?

That would be duplicating the work that other projects are supposed to do. I understand you want it, but other projects already deal with it. Them not dealing with it is the problem here.

I don’t think this relates to this project, but to esbuild itself or to the MDX esbuild loader: https://github.com/mdx-js/mdx/blob/63fd208d53f13e5b2451c9f9bc1b7fd136d22c71/packages/esbuild/lib/index.js#L159-L192
As far as I’m aware, that code should do exactly what you want. Perhaps you can debug locally why that isn’t working for you, or make a small reproduction?

Thank you for blazing-quick response.
I found my bug in MDX within 5 minutes, so no worries my side. I'm more concerned about the users in general, how to improve.
Titus, feel free to close this ticket if it's not the right place.

I share your concern, so I’d appreciate it if you could look into esbuild and @mdx-js/esbuild.
Closing as this belongs somewhere else!


Notation#filter issue

!*.* should not be treated the same as !*.  First one negates second level properties, not first level.

Fixed in v1.3.0


Removed jax.device_put from jnp.eye and jnp.arange implementations

Description:

Removed jax.device_put from jnp.eye and jnp.arange implementations
Added sharding kwarg and jitted lax.broadcasted_iota

There are related failures with now jitted lax.broadcasted_iota and vmap:
FAILED tests/dynamic_api_test.py::JumbleTest::test_broadcast_in_dim_ragged_to_static_error - FutureWarning: unhashable type: <class 'jax._src.interpreters.batching.BatchTracer'>. Attempting to hash a tracer will lead to an error in a future JAX release.
FAILED tests/dynamic_api_test.py::JumbleTest::test_broadcast_in_dim_to_doubly_ragged - FutureWarning: unhashable type: <class 'jax._src.interpreters.batching.BatchTracer'>. Attempting to hash a tracer will lead to an error in a future JAX release.
...

@jakevdp does the sharding bit go on iota or on array below?

Sharding should be on the output. I think the thought here is that if you put it on the iota, then it will carry to the output?

think the thought here is that if you put it on the iota, then it will carry to the output?

Yeah it would but the shape would be correct for the sharding?
Also we need tests for those!


Fix serialization of message implementing IRootSerializable

Description
Proto generator creates grain client with method proxy which wraps request in GrainRequestMessage. GrainRequestMessage includes inner serialization method however its outcome was never used in EndpointActor. Serialization of GrainRequestMessage with JsonSerializer resulted in serialization exception with message: "Serialization and deserialization of 'System.Type' instances are not supported and should be avoided since they can lead to security issues. Path: $.Columns.DataType."
Purpose
This pull request is a:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

 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.

Good catch!, merging this


Implement DELETE for Website collections

This should delete the website record and all short codes associated with that website

Website collections have been removed


Relative path between TeX and image outputs

I have the following structure:
├── images
│   ├── plot-000.png
│   └── plot.tex
├── main.tex
├── Makefile
└── plot.py

main.tex:
\documentclass{article}

%\usepackage[luatex]{graphicx}
%\graphicspath{{images/}}

\usepackage{tikz}
\usepackage{pgfplots}
\usepgfplotslibrary{groupplots}
\pgfplotsset{compat=1.16}

\begin{document}
	\input{images/plot.tex}
\end{document}

Makefile:
all:
	python plot.py
	lualatex main.tex

plot.py:
import numpy as np
import matplotlib.pyplot as plt
import os
import tikzplotlib

x = np.linspace(0.0, 1.0, 500)
y = np.linspace(0.0, 1.0, 500)
x_2d, y_2d = np.meshgrid(x, y)

step = np.where((0.25 <= x_2d) & (x_2d <= 0.75) & (0.25 <= y_2d) & (y_2d <= 0.75), 1.0, 0.0)

plt.imshow(step)

tikzplotlib.save(
    "images/plot.tex",
#    tex_relative_path_to_data="images/",
#    tex_relative_path_to_data=os.path.abspath("images/"),
)

generated plot.tex:
% This file was created with tikzplotlib v0.10.1.
\begin{tikzpicture}

\definecolor{darkgray176}{RGB}{176,176,176}

\begin{axis}[
tick align=outside,
tick pos=left,
x grid style={darkgray176},
xmin=-0.5, xmax=499.5,
xtick style={color=black},
y dir=reverse,
y grid style={darkgray176},
ymin=-0.5, ymax=499.5,
ytick style={color=black}
]
\addplot graphics [includegraphics cmd=\pgfimage,xmin=-0.5, xmax=499.5, ymin=499.5, ymax=-0.5] {plot-000.png};
\end{axis}

\end{tikzpicture}

I would like generated *.tex files to be stored in images/ together with their *.png companions.
However, if I do so (with the images/ prefix in tikzplotlib.save), when compiling main.tex I get:
Package pgf Warning: File "plot-000.png" not found when defining image "pgflast
image". Tried all extensions in ".pdf:.jpg:.jpeg:.png:" on input line 18.

because in plot.tex the *.png is included without path.

Adding tex_relative_path_to_data="images/" to tikzplotlib.save does not work, I get the error

FileNotFoundError: [Errno 2] No such file or directory: 'images/images/plot-000.png'

Adding \graphicspath{{images/}} to main.tex does not work, because for some reason tikzplotlib forces the includegraphics cmd to \pgfimage (which does not seem to respect \graphicspath) rather than \includegraphics.
There's an option table/search path in PGF for \addplot table, but there doesn't seem to be an equivalent for \addplot graphics.
I can make it work with tex_relative_path_to_data=os.path.abspath("images/") in tikzplotlib.save, but that's ugly and non-portable.

Is there a good way to make this work?

Note: I could generate plot.tex to the top-level directory, but that's seems less logical conceptually, plus it's less convenient e.g. for VCS integration.
A couple independent ideas in case tikzplotlib has to be modified for this:

Make the choice of \pgfimage vs \includegraphics accessible by a parameter
Always use tex_relative_path_to_data in the generated *.tex (but that's not robust to moving generated files around)
Use LaTeX's \CurrentFilePath and friends
Request a graphics/search path feature by PGF
Request that \pgfimage respects \graphicspath by PGF or graphicx
Add yet another target folder parameter to tikzplotlib.save


Missing Posters

Added Items (linked to TMDB add on) to Library have missing Posters, is there a fix ? considering Cinemeta doesn't have this problem

Have these items been added to the TMDB catalog? Have you removed cinemeta? The TMDB Addon did not yet provide support for IMDBId's, the latest update added this option and it needs to be enabled on the configuration page.

@mrcanelas
I think this issue should be reopened. I have same issue. Just posters are missing, when i click on it for details, all metadata seems to be ok (loaded from tmdb website).
Try to add for example the show YOU (2018) if it will be blank also for you.

Let me try to explain in a way that is easier to understand, Cinemeta uses ImdbId, TMDB Addon uses TmdbId, if you added something to your library using Cinemeta, Stremio tries to search for the posters in Cinemeta, if you removed Cinemeta the poster will not appear, to provide the metadata for the item TMDB will work, but the item in the library is not updated, it will still be linked to Cinemeta, but to solve it you must search for the movie/series and add it through the TMDB catalog and remove the old one that was added by Cinemeta.

I think that enabling the IMDBId support option during addon configuration resolves this problem. It did for me.

I added it through TMDB - popular catalog but still no poster. But before it was added from Cinemeta, i removed it from Library but maybe it was not removed completely...

If you are on mobile try removing the cache (same applies to TV). If on a desktop try refreshing the app Fn+F5.

I am on Windows native app, but the same apply also when i try web.stremio.com from Firefox

Something that works for me is to play an episode on the item I want to update. Stremio updates the library data when you play something.

thanks, it helped... what about that "Provide metadata for IMDBId"? Dunno if i should leave it enabled, i see no difference on the details page in comparison when i had it disabled before...

This option is used to take translated metadata to other catalogs that use ImdbId, for example Streaming Catalogs, if you only use TMDB this option does not change anything.

This option is used to take translated metadata to other catalogs that use ImdbId, for example Streaming Catalogs, if you only use TMDB this option does not change anything.

Hello, I have an issue with missing thumbnails and posters (that background on a movie page) when using TMDB + ru language. Cinemeta is there and works fine. Maybe addon tries to load ru images, and there's none? Can we have some sort of fallback to EN images if localized are not found? Just my speculation of course. Really like the addon, but lack of images makes it hard to use


Reworking mode_puzzle to a complete and up-to-date state with functionality

This is going to be a list of what all needs to be implemented.
Code:

[x] add Timer
[x] count swaps being done in playfield
[x] step_counter limit overcrossed - game over - reset to beginning state of Puzzle Data
[x] redo button b - settled for saving playfield panel.kind when swapped - redo button sets everything to the playfield save before
[x] move to next Puzzle data - need some way to go through all, or save some locally
[x] select level menu

BUGS (crossed solved):

[x] count swaps should not be increased by switching "empty" blocks
[x] undoing should work and reset block that are clearing
[x] loop back to the menu once all levels are done

Art:

[x] steps_left should have it's own class which renders itself - later on animations possible etc
[x] general background
[x] playfield frame
[x] playfield background
[x] encapsule existing timer
[ ] winning animation when puzzle is solved

Main Functionality has been implemented - only new sprites & backgrounds & animations are necessary to finish puzzle mode. Information of Levels can still be easily added.


Error in tensor_utils.batched_together when running sim_inference.py

Traceback (most recent call last):
  File "/data/Dionessa/mdgen/sim_inference.py", line 140, in <module>
    main()
  File "/data/Dionessa/p312-str2str/lib/python3.12/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/data/Dionessa/mdgen/sim_inference.py", line 137, in main
    do(model, name, df.seqres[name])
  File "/data/Dionessa/mdgen/sim_inference.py", line 103, in do
    item = get_batch(name, seqres, num_frames = model.args.num_frames)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/Dionessa/mdgen/sim_inference.py", line 40, in get_batch
    atom37 = torch.from_numpy(atom14_to_atom37(arr, seqres[None])).float()
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/Dionessa/mdgen/mdgen/geometry.py", line 10, in atom14_to_atom37
    atom37 = batched_gather(
             ^^^^^^^^^^^^^^^
  File "/data/Dionessa/mdgen/mdgen/tensor_utils.py", line 92, in batched_gather
    return data[ranges]
           ~~~~^^^^^^^^
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (4,) + inhomogeneous part.

Solved by changing ranges to tuple in newer version of pytorch
in mdgen.tensor_utils.batched_gather() :
return data[tuple(ranges)]

can you share your env with newer version of pytorch

Hi, I'm using python3.12 with pytorch==2.4.1+cu121
But i think your error is different, which is on your input sizes. What is the size of your molecule ?

Thanks.  I just follow the README.MD and run tps_inference.py

“return data[tuple(ranges)] ”  is work on sim_inference but tps_inference is error. maybe some bugs in tps_inference.py


Adding an extension with missing 'properties' field causes non-descriptive error message "Canceled"

From this tweet
"Sooo... adding an #ApplicationInsights extension to your website without empty properties will fail with the most meaningless error ever: "Canceled". Today I learned! And wasted an hour..."
Proposed solution: More descriptive error message, i.e. "Missing Properties field"

Repro steps.
Add an extension via ARM resource template without including the properties field

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Akka tracing instrumentation

From @adriancole on June 29, 2017 8:13
Historically, https://github.com/levkhomich/akka-tracing has been the answer for akka in zipkin. This project has recently gone quiet, which has impacted users looking for a solution. It would be great for this project to pick-up again, and likely that would imply multiple owners so that the same doesn't recur.
Meanwhile, there are some other works related to akka tracing on github:

https://github.com/bizreach/play-zipkin-tracing - play zipkin tracing using brave 4.x
https://github.com/lucidsoftware/akka-thread-context - thread context similar to brave's
https://github.com/FindHotel/akka-stream-trace - akka stream tracing with htrace
https://github.com/gallathanmdsol/tracing - akka experiment using brave 3.x
https://github.com/lloydmeta/zipkin-futures - future tracing (play support) using brave 2.x

There have been zipkin users spending time with mixed success with a goal of tracing akka, which is the motivation of this issue. Notably, @metamorph @kriloleg @tsteff @capitanbatata @ShaneDelmore @andersenleo @lawrencefinn. Those using play often ended using @takezoe's https://github.com/bizreach/play-zipkin-tracing for a combination of reasons including fast turnarounds on support and questions, and being present on gitter.
There are a number of users successfully using akka-tracing, including @sandeep-paliwal in an open pull request for openwhisk. An ideal solution might be a sustainable setup for akka-tracing which provides for multiple people contributing to it. To be fair, @levkhomich has done a stellar job, starting this work and updating the project a few months ago, and responding to questions as recently as last month.  Whichever direction taken, we owe a great deal of credit to Lev.
So, what do people think about this topic?
Copied from original issue: openzipkin/brave#450

From @ShaneDelmore on June 29, 2017 16:3
@adriancole Kamon committers @dpsoft and @ivantopo have also done some work in this area and I believe are desiring to stay compatible.

From @LarryFinn on June 29, 2017 16:16
@adriancole i think it would be nice.  i couldn't fully use the play-zipkin-tracing lib because of compatibility issues and im pretty sure it doesn't automatically pass the traceid between futures via the default dispatcher or default execution context.   the akka-thread-context partially solves this but doesnt solve the problem of the default execution context. or maybe i did something wrong...

Whatever fills the akka need, if it employs the CurrentTraceContext (directly or indirectly via Tracer.withSpanInScope), it could automagically integrate with SLF4J or Finagle's context synchronization libraries.

From @andersenleo on July 6, 2017 8:5
We built our Akka tracing to avoid any explicit, hard-coded, tracing in application code. Since we're in control over the entire stack our solution isn't easily portable to a generic "library". Note that it's currently based on the Brave3 API.
I'll try to describe it anyway - hoping that it can be partially reusable :-)
All our actors extends a common super-class. And we've got a central factory where Props for actors are created. This allows us to create the actor with a custom UnboundedMailboxWithTracing configuration.
The mailbox queue will intercept all messages enqueued and wrap the message in a TracedMessage that holds the current ServerSpan from the thread-local Brave3 context.
Our custom super-class will handle all incoming TracedMessages, unwrap them and set the ServerSpanThreadBinder#setCurrentSpan during processing of the wrapped message.
This works in most cases - but some of akkas internal actors uses mailboxes/message-queues that don't enqueue on the same thread as the sender - and in that case there is no tracing context available. This is no big deal for us since we're only interested in tracing application level actors.
Somewhere in the future we'll look into migrating to Brave4 - but we're not there yet.

From @dpsoft on July 6, 2017 16:21
Hi, we already went through the process of making our tracer a OpenTracing compliant one, but that is not quite the same as Brave ... if you have are using Kamon you will be able to capture the trace data and send it to Zipkin and benefit from all available instrumentation in Kamon + any Opentracting instrumentation out there, but we think Zipkin/Brave are another world. Anyway, If they are curious about the kamon-akkamodule that is based on OpenTracing API, can look here and here for kamon-play(WIP).

Hi, we already went through the process of making our tracer a OpenTracing
compliant one, but that is not quite the same as Brave ... if you have are
using Kamon https://github.com/kamon-io/Kamon/tree/kamon-1.0-develop
you will be able to capture the trace data and send it to Zipkin and
benefit from all available instrumentation in Kamon + any Opentracting
instrumentation out there, but we think Zipkin/Brave are another world.
Anyway for the

Curious.. I don't see any opentracing types in kamon or kamon-akka. It
seems maybe the tracer is inspired by OpenTracing, but not OpenTracing,
right? This isn't terribly important here, but just wondering if I'm
missing something or looking at the wrong repos.

If they are curious about the kamon-akkamodule that is based on
OpenTracing API, can look here
https://github.com/kamon-io/kamon-akka/tree/kamon-1.0 ;)

This isn't opentracing, but it is helpful as your tracer api is very
similar to Brave's. Thanks!

From @dpsoft on July 7, 2017 11:52
@adriancole plese take a look to kamon-core  in the branch kamon-1.0-develop which is the one that implements OpenTracing,  and yes is influenced by Brave ;)

@adriancole https://github.com/adriancole plese take a look to
kamon-core
https://github.com/kamon-io/Kamon/blob/kamon-1.0-develop/kamon-core/src/main/scala/kamon/trace/Tracer.scala#L31-L32
in the branch kamon-1.0-develop which is the one that implements
OpenTracing.

thx @ivantopo pointed it out to me a little while ago, too

From @llinder on November 5, 2017 16:17
This is something I've been interested in using for some time.  I'm primarily interested in Akka HTTP but having comprehensive tracing support for Scala Future/ExecutionContext, Akka and Akka HTTP would be really nice.   I honestly haven't looked at bizreach/play-zipkin-tracing, mostly because it sounds focused entirely on the Play framework but maybe that isn't the case?
Anyway, I'm going to try and must up some spare cycles to catch up on Akka instrumentation and hopefully contribute were there are gaps.

long discussion on gitter and seems one supportable route would be to enhance the existing play library. Here's a tracking issue: https://github.com/bizreach/play-zipkin-tracing/issues/21
This wouldn't preclude any other options including the upcoming work in kamon (there is more than  one answer sometimes: that's ok as long as at least one is maintained!)

I honestly haven't looked at bizreach/play-zipkin-tracing, mostly because
it sounds focused entirely on the Play framework but maybe that isn't the
case?

it definitely started as this. One thing to consider is that @takezoe is
doing a great job like others in working together on problems. In other
words, you might have a buddy in maintaining if collaborating there. If you
see something that needs to change, I would dare to propose it! Once
there's more community around this sort of instrumentation we can still
take any next steps, and whatever you decide makes sense, let's make sure
all are aware (to help you not end up on your own)

Anyway, I'm going to try and must up some spare cycles to catch up on Akka
instrumentation and hopefully contribute were there are gaps.

Very excited and grateful

From @asereda-gs on February 1, 2018 19:49
Some people mentioned libraries which are using code instrumentation / byte-code modification to achieve tracing (and telemetry) of akka:

Kamon - Apache 2.0 license
Cinnamon - Commercial offering by Lightbend

The reason I prefer this technique is because it is more transparent and less intrusive than modifying all existing code (Actor / ActorRef / Message etc.). However, I'm not a big fan of java agents since they may conflict among each-over and introduce an operational burden  (more complicated unit / integration tests, you have to initialize them before other classes etc.)
What people in this group think about it ?
I'm trying to develop a solution which doesn't impose code modification nor use of java agents (custom Mailboxes / ActorRefProviders etc.) will let you know if it succeeds

From @LarryFinn on February 1, 2018 20:54
i've been silent on this thread (sorry).  i tend to find code instrumentation via java agent to be a bit worrisome in production.  i've seen some tools actually cause the JVM to crash, which is pretty bad.

From @asereda-gs on February 8, 2018 16:50
After talking to @adriancole and @takezoe we'll try to explore some ideas in separate repository created under openzipkin-contrib. Contributors are welcome.

ok I opened https://github.com/openzipkin-contrib/brave-akka, adding @asereda-gs and @takezoe as write with @llinder as admin (as he is very familiar with our release processes etc)
Keep us posted!

From @jkubrynski on March 6, 2018 23:37
Why is this issue closed as it's not fixed?

@jkubrynski the issue was moved here


RabbitMQ Security

Please see this ticket.

done - https://github.com/DataONEorg/dataone-indexer/pull/50


add output model for search entries

https://github.com/stac-utils/pgstac/blob/3499daa2bfa700ae7bb07503795c169bf2ebafc7/sql/004_search.sql#L907-L915
to be used here https://github.com/stac-utils/titiler-pgstac/blob/master/titiler/pgstac/factory.py#L382

done in #43


Testing docker_call

This issue depends on successful completion of: #1387
Tests for docker_call should accomplish:
Testing failure of containers
Detecting existing "ghost" containers

This seems to have been completed.


Fix android kotlin compilation / build issues

Purpose
A fix for non compatible kotlin versions that resulted in a build error for android.
> Task :sift-react-native:compileDebugKotlin FAILED

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':sift-react-native:compileDebugKotlin'.
> Compilation error. See log for more details

2: Task failed with an exception.
-----------
* What went wrong:
java.lang.StackOverflowError (no error message)

Testing

Tested running android on multiple machines with appropriate SDK and NDK configurations
Ran tests and iOS versions which all passed

Checklist

[x] The change was thoroughly tested manually
[x] The change was covered with unit tests
[x] The change was tested with the integration example
[x] The version refers to the latest stable version of sift-android
[x] The version refers to the latest stable version of sift-ios
[x] Necessary changes were made in the integration example (if applicable)
[ ] New functionality is reflected in README (if applicable)

@athul173 could you, please, upgrade to "1.6.0" to have same version as in https://github.com/SiftScience/sift-react-native/blob/18b3fe9b484b88cef234a47b426e663f2587659b/android/gradle.properties#L1 ?

Duplicate of #40 https://github.com/SiftScience/sift-react-native/pull/40


Python 3 support (issue #5)

I have made, through the use of the Six library, the necessary modifications to support Python 3.4/Python 3.5.
pytest and pytest-html required versions are updated to support Python 3 as well.
The official pytest-circleci repository doesn't support Python 3.x, so I've switched to the fork of pytest-circleci made by @dustinfarris, which supports it.
All non-skipped tests passes for Python 2.7, 3.4 and 3.5. Other versions are untested.

Linking to #5

Thanks a lot @monnierj ! Could you tell us what you think of #18 ? I think I would prefer not adding any dependency but I'm interested in your opinion!
thanks :)

Well, it works the same, but without Six. In that regard, it is better suited to your preferences :)
One missing thing in Liwink's PR is the version bump for Pytest. It worked for him on Python 3.5 because he used Pytest 2.8.7 (which handles the change in the signature of the ast.Call method), I think he just forgot to update the requirements.txt.
And, if you use run make test with a Python 3.4/3.5 virtualenv, apart from the errors related to Pytest 2.7, you will encounter an encoding error within pytest-circleci (but as I see issue #16, I suppose that a switch from CircleCI to Travis is planned, so maybe it is not a real issue).
But, apart from this, it works.

great @monnierj, thanks for the feedbacks! I will bump the pytest version after merging then, and switch to travis indeed to avoid the circle issue you found.
Thanks a lot!


Type checking with pyright

Fix #13
Remove mypy and add pyright config to pyproject.toml

Turn on strict mode, and ignore the templates in no-test

Fix many warnings, and add many ignores.

~60 usages of # type: ignore
~30 of these are on server functions which are flagged as unused, because Shiny is matching them with strings.
For the rest, I do not think it makes sense to try to resolve every last one of these right now:

Pyright's docs emphasize that the addition of typing should be incremental. It doesn't need to be all or nothing.
Adding static checks incrementally and gradually raising the bar has worked well for me on other projects.
If we just wanted not to have type: ignore, we could drop strict, but I think the reminders are good.
There are diminishing returns.

Improvements:

Typing that distinguishes reactive values from plain values seems particularly useful, and probably is better than inventing a naming convention.
The weights are different from others, because the user is selecting a string, which we need to then explicitly cast to a number. Typing makes this more explicit.
Templates are now stringified with a call to .finish(), rather than str(); The old way would require me to implement a more complicated interface to really comply with the typing.

My plan is to continue this work incrementally, but I think having the checks in place is enough for now. I would particularly like to clean up a couple places where dicts could be replaced with NamedTuples.
For the reviewer:

Do the types improve the readability/maintainability of the code for you?
Does this seem like a good place to commit to main, or are there particular type: ignore that you think could use more attention?
Does the switch from mypy to pyright make sense?

After sitting with this, I think my preference would be to remove strict, and then most of the overrides could be removed, but it would be good to get someone else's opinion. That way, we would get the benefit of the checks where they are supplied, and it would be up to us to keep adding them on new code going forward.

After sitting with this, I think my preference would be to remove strict, and then most of the overrides could be removed, but it would be good to get someone else's opinion. That way, we would get the benefit of the checks where they are supplied, and it would be up to us to keep adding them on new code going forward.

I agree with opting for non-strict mode, as this doesn't appear to be a complex application. Removing strict could make the codebase cleaner and enable faster development while still let us to selectively apply strict checks where needed.

Let me merge main into this and we can go from there.

Also, it seems like a bunch of warning happened because the Reactive.Value[] type was not equal to primitive type even though they are both int/float/other.

These reactive.values do need to be treated differently from the types they wrap, and having a warning about these is actually one of the big wins. To get the wrapped value from a reactive.value, you need to call it as a function:
>>> name = reactive.value('chuck')
>>> name
<function name at 0x...> or something like that
>>> name()
'chuck'
>>> name.set('not chuck')
>>> name()
'not chuck'

(This is not an actual repl session: Reactive values don't work outside of a reactive context, so this wouldn't work.)

Tidied up after merge from main, and applied typing with the nice NamedTuples that were introduced in the refactoring of code generation. Merging this much will be a big improvement, and we can still make incremental improvements.


Totally down for this. I like viewInit

k I'll do a quick branch pull request


New Airport: EDDT (Berlin Tegel)

Berlin Tegel Airport is the main international airport of Berlin as well as the 4th busiest airport in Germany. Even though it's quite a small airport It receives a lot of traffic from all over Europe.
I hope you enjoy it!

Looks good! Even though it hurts my brain a little... lol


Settings redesign phase 1

Future PRs will

restyle the navigation view
restyle the setting blocks to closer match the Windows settings app
merge and better organize more setting pages

Could you share a screenshot?

I'm concerned about the text clipping. While it looks fine in English, it may not look as good in German for instance.

I'm concerned about the text clipping. While it looks fine in English, it may not look as good in German for instance.

This PR fixes the text clipping in the setting blocks that happened in previous versions. If you're referring to the navigation view but that will be restyled in phase 2.

I suppose I shouldn't care about the close button overlapping the "About" item right?

I suppose I shouldn't care about the close button overlapping the "About" item at this point right?

Pretend it's not there 😉


MAINT: special: fix msvc build by using new and delete for cwrk

Reference issue

Closes https://github.com/scipy/scipy/issues/20860
What does this implement/fix?

Additional information

Thanks @czgdp1807. Looks reasonable to me. I'll leave the review to @steppi.

I think if I understood correctly, the reason new and delete is avoided is to avoid the dynamic memory allocation because GPU reasons. So this might be going against that. Indeed @steppi and @izaid should know better than my vague memory.

Nothing to do with GPUs, the old code just doesn't compile with MSVC. The traceback is in gh-20860.

Yes what I mean is that the reason why amos did not use new and delete is that they are similar to malloc and free dynamic memory. https://github.com/scipy/scipy/pull/20258 For some reason they did not like that linking back to a discussion about cupy or something.
How is this leading to an error on MSVC is my issue that I did not understand what the error says

How is this leading to an error on MSVC is my issue that I did not understand what the error says

MSVC is not advanced enough to create arrays on stack which have sizes dependent on variables (doesn't matter if it's an input argument). Ideally, if the size of an array is dependent only on input argument and constants then array can be created on stack without any issues. MSVC is unable to do that. So I had to use new and delete for making the code MSVC compatible. However, if it's dependent on local variables then creation on heap is the only way.

I see. That's really unfortunate. There are certain places in amos and specfun that stack would be unnecessarily large hence I was siding with heap allocation but apparently folks have portability concerns over CUDA applications. I did not pursue further to understand the nature of it though.

Yes. This is the only option I was able to figure out. Even I wanted MSVC to be good enough to process the original code but it is what it is. There are more instances where I have seen other compilers able to do things which MSVC can't. Hopefully, it will be better in the future.

How is this leading to an error on MSVC is my issue that I did not understand what the error says

MSVC is not advanced enough to create arrays on stack which have sizes dependent on variables

Pedantic clarification: a declaration such as std::complex<double> cwrk[n]; is known in the C world as a variable length array (VLA).  These are allowed in C99.  But amos.h is a C++ file, and variable length arrays are not allowed in standard C++.  If it compiles, it is because the compiler provides a nonstandard extension.  MSVC does not provide this extension.
With g++ you can detect this with, for example, the command line option -pedantic-errors, which will tell you that you are trying to use a C99 feature that is not part of C++. For example,
foo.cpp:
void foo(int n) {
    int x[n];
}

% g++ -pedantic-errors -c foo.cpp
foo.cpp:2:11: error: variable length arrays are a C99 feature [-Werror,-Wvla-extension]
    int x[n];
          ^
1 error generated.

Ah OK, then that explains why I did not encounter during Fortran translation to C since I used -wall -wextra -wpedantic and did not encounter anything related to that.

With g++ you can detect this with, for example, the command line option -pedantic-errors, which will tell you that you are trying to use a C99 feature that is not part of C++. For example,

Thanks @WarrenWeckesser. @czgdp1807, can you add --pedantic-errors to the -Csetup-args here?
https://github.com/scipy/scipy/blob/a95defcd0f0f9509feac3198dc9100908bff5492/.github/workflows/linux.yml#L536
This job was intended to give a reasonably strict check for compiler warnings, so I think this would be a good fit here.

Yes what I mean is that the reason why amos did not use new and delete is that they are similar to malloc and free dynamic memory. #20258 For some reason they (albert and irwin) did not like that linking back to a discussion about cupy or something.

It looks like we might need to use conditional compilation to have a separate code path for these arrays on CUDA. I'll look into that later on.


Unbound breakpoints when debug TypeScript project

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.77.3 (user setup)
Commit: 704ed70d4fd1c6bd6342c436f1ede30d1cff4710
Date: 2023-04-12T09:16:02.548Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.18363
Sandboxed: No
OS Version: Windows 10 1909 Build 18363.752
JavaScript Debugger extension v1.77.2

What Happened:
Today when I debug a project, breakpoints not working suddenly and irrecoverable.
My debug settings not changed for long time, so I can't figure out where the exception is happening.
Steps to Reproduce:

Start project

More info:
launch.json
{
  // Use IntelliSense to learn about possible attributes.
  // Hover to view descriptions of existing attributes.
  // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Run yarn start",
      "command": "yarn start",
      "request": "launch",
      "type": "node-terminal"
    }
  ]
}

ng version
Angular CLI: 15.1.2
Node: 16.19.0
Package Manager: yarn 1.22.19
OS: win32 x64

Angular: 15.1.1
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1501.2
@angular-devkit/build-angular   15.1.2
@angular-devkit/core            15.1.2
@angular-devkit/schematics      15.1.2
@angular/cli                    15.1.2
@schematics/angular             15.1.2
rxjs                            7.5.4
typescript                      4.9.4

/jsDebugLogs

Please collect trace logs using the following instructions:

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>

Log file below:
vscode-debugadapter-7b0b9c76.json.gz

You're running a terminal process. Only the Node.js programs run in the terminal will be debugged. To debug your angular app, you probably want to set up a browser launch config.  See https://code.visualstudio.com/docs/nodejs/browser-debugging#_launching-browsers

OK, I solve the problem use below lanuch.json:
{
    "name": "Launch Chrome",
    "request": "launch",
    "type": "chrome",
    "sourceMaps": true,
    "url": "http://localhost:4200",
    "webRoot": "${workspaceFolder}/src",
    "userDataDir": "${workspaceRoot}/.chrome",
    "sourceMapPathOverrides": {
        "webpack:///./src/*": "${webRoot}/*"
    }
}


chore: update the dev environment to use new subscription

Ticket Number

UCD-1174
Description of change

Update the dev cluster with the new Azure subscription details
Checklist

[ ] Requires infrastructure changes
[ ] I have updated the documentation accordingly
[ ] I have merged commits to avoid fixing things in this PR
[ ] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

TODO

Add in the key vault name when known


Can't connect to Cassandra 3.0

Using v0.5, I can't connect to Cassandra 3.0:
Caused by: com.datastax.driver.core.exceptions.NoHostAvailableException: All host(s) tried for query failed (tried: /<IP_ADDRESS>:9042 (com.datastax.driver.core.exceptions.InvalidQueryException: unconfigured table schema_keyspaces))

works for me, are you using
"com.github.krasserm" %% "akka-persistence-cassandra3" % "0.5"

note the 3 in the artifact name

Ah, damn, missed the 3!
Thanks!


ui: after creating a set, you can't see it

If you create a set in the UI with "Create set from items", there's no
feedback once it worked.

I would expect to redirect to that set, or see a link to it.

If you're lucky and you're looking at the default search and you're scrolled to the top,
then you see it appear. But if you're scrolled down or searching for something specific
from which you created the set, then nothing appears.

So, isn't this issue now fixed by the above CL?

So, isn't this issue now fixed by the above CL?

Taking about the "after a big import" use case, it seems like redirect on set creation from selection is fine since the back button works. I guess the only question is whether the selection should be reset after set creation.

So, isn't this issue now fixed by the above CL?

Yes, unless there are other suggestions I'm considering it fixed.


Not indenting correctly for complex array

Hi,
The main branch is not indenting well with array,
[
  {
    "exchange": "SHFE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": [
      ]
    },
    {
      "exchange": "DCE",
      "sections": [
        "9:00-10:15",
        "10:30-11:30",
        "13:30-15:00"
      ],
      "products": [
        ]
      },
      {
        "exchange": "CZCE",
        "sections": [
          "9:00-10:15",
          "10:30-11:30",
          "13:30-15:00"
        ],
        "products": [
          ]
        },
           ]

It should be
[
  {
    "exchange": "SHFE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "DCE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "CZCE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
]

The whole C++ code
#include <glaze/core/macros.hpp>
#include <glaze/glaze.hpp>

struct TradingHours {
    std::string exchange;
    std::vector<std::string> sections;
    std::vector<std::string> products;
};

GLZ_META(TradingHours, exchange, sections, products);

template <class T>
inline int write_file(T &value, std::string const &filename)
{
    using namespace glz;

    std::string buffer;
    std::ofstream file(filename);
    context ctx{.current_file = filename};
    constexpr auto options = opts{.prettify = true, .indentation_width = 2};

    try {
        write<options>(value, buffer, ctx);
        if (file) {
            file << buffer;
        } else {
            return -1;
        }
    } catch (...) {
        return -2;
    }

    return 0;
}

void set_trading_hours()
{
    TradingHours x01 = {"SHFE",  {"9:00-10:15", "10:30-11:30", "13:30-15:00"}};
    TradingHours x02 = {"DCE",   {"9:00-10:15", "10:30-11:30", "13:30-15:00"}};
    TradingHours x03 = {"CZCE",  {"9:00-10:15", "10:30-11:30", "13:30-15:00"}};
    TradingHours x04 = {"INE",   {"9:00-10:15", "10:30-11:30", "13:30-15:00"}};
    TradingHours x05 = {"GFEX",  {"9:00-10:15", "10:30-11:30", "13:30-15:00"}};
    TradingHours x06 = {"CFFEX", {"9:30-11:30", "13:00-15:00"}};
    TradingHours x07 = {"CFFEX", {"9:30-11:30", "13:00-15:15"}, {"T", "TF", "TS", "TL"}};
    TradingHours x08 = {"INE",   {"21:00-02:30"}, {"sc"}};
    TradingHours x09 = {"INE",   {"21:00-01:00"}, {"bc"}};
    TradingHours x10 = {"INE",   {"21:00-23:00"}, {"nr", "lu"}};
    TradingHours x11 = {"SHFE",  {"21:00-02:30"}, {"au", "ag"}};
    TradingHours x12 = {"SHFE",  {"21:00-01:00"}, {"pb", "ni", "sn", "ss", "cu", "al", "zn"}};
    TradingHours x13 = {"SHFE",  {"21:00-23:00"}, {"bu", "hc", "fu", "sp", "ru", "rb"}};
    TradingHours x14 = {"DCE",   {"21:00-23:00"}};
    TradingHours x15 = {"CZCE",  {"21:00-23:00"}};

    std::vector<TradingHours> hours = {
        x01, x02, x03, x04, x05, x06, x07, x08, x09, x10, x11, x12, x13, x14, x15};

    write_file(hours, "trading_hours.json");
}

@sweihub This indentation error was previously fixed in this PR.
I've tested your code with the latest glaze, at the result is as follows:
[
  {
    "exchange": "SHFE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "DCE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "CZCE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "INE",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "GFEX",
    "sections": [
      "9:00-10:15",
      "10:30-11:30",
      "13:30-15:00"
    ],
    "products": []
  },
  {
    "exchange": "CFFEX",
    "sections": [
      "9:30-11:30",
      "13:00-15:00"
    ],
    "products": []
  },
  {
    "exchange": "CFFEX",
    "sections": [
      "9:30-11:30",
      "13:00-15:15"
    ],
    "products": [
      "T",
      "TF",
      "TS",
      "TL"
    ]
  },
  {
    "exchange": "INE",
    "sections": [
      "21:00-02:30"
    ],
    "products": [
      "sc"
    ]
  },
  {
    "exchange": "INE",
    "sections": [
      "21:00-01:00"
    ],
    "products": [
      "bc"
    ]
  },
  {
    "exchange": "INE",
    "sections": [
      "21:00-23:00"
    ],
    "products": [
      "nr",
      "lu"
    ]
  },
  {
    "exchange": "SHFE",
    "sections": [
      "21:00-02:30"
    ],
    "products": [
      "au",
      "ag"
    ]
  },
  {
    "exchange": "SHFE",
    "sections": [
      "21:00-01:00"
    ],
    "products": [
      "pb",
      "ni",
      "sn",
      "ss",
      "cu",
      "al",
      "zn"
    ]
  },
  {
    "exchange": "SHFE",
    "sections": [
      "21:00-23:00"
    ],
    "products": [
      "bu",
      "hc",
      "fu",
      "sp",
      "ru",
      "rb"
    ]
  },
  {
    "exchange": "DCE",
    "sections": [
      "21:00-23:00"
    ],
    "products": []
  },
  {
    "exchange": "CZCE",
    "sections": [
      "21:00-23:00"
    ],
    "products": []
  }
]

Oh, I'm conservative to upgrade the code, glad to hear it was fixed already, thanks!


Subshells parse as WORDs

Subshell syntax (...) parses as a single word, so commands like (ls) are errors.

This is a conflict with the {#! ... } rule. A conditional compilation attribute on the production would be nice.

Fixed in 4bad7e3516b28793f422e2e54c61450ae1209764


Ensure Push Protocol SDK at latest version

Ensure the latest version of the Push Protocol alpha SDK is being referenced by node_modules build. The /server package was importing the stable SDK version, while the ui-components package was importing the alpha. This caused the SDK to be an unexpected level at build time, and features not working as expected.

/gcbrun

/gcbrun


ConnectionMultiplexer Issue

After updating to IdentityServer4 v2.4, I was forced to update this library to v2.0 but now I have an error about the ConnectionMultiplexer existing in both StackExchange.Redis.StrongName and StackExchange.Redis. I'm using a dotnet core 2.1 project.
Do you know how to resolve this conflict?

Looks like StackExchange.Redis.StrongName package is referenced by one of the framework library that you depends on in .Net core 2.1, maybe it's Microsoft.AspNetCore.All, that's why there is this conflict, please check out your references or share the list here so I have more context to help.

I checked out my references, one of them causing an issue was Microsoft.AspNetCore.All which I uninstalled and then installed all the individual packages. The two others were Microsoft.Extensions.Caching.Redis which i swopped out for Microsoft.Extensions.Caching.StackExchangeRedis, and then Microsoft.AspNetCore.DataProtection.Redis which i swopped out for "Microsoft.AspNetCore.DataProtection.StackExchangeRedis.
Thanks for the help and the awesome library.

Glad it helped you out, you are welcome !