there is a bug in file "binance/client.py" function _historical_klines_generator

Describe the bug
there is a bug in function _historical_klines_generator line 1032
@sammchardy

we need to distinguish spot=True and False;
and no need the parameter spot, it make error as follows:

fix

            if spot:
                output_data = self.get_klines(
                    # spot=spot,
                    symbol=symbol,
                    interval=interval,
                    limit=limit,
                    startTime=start_ts,
                    endTime=end_ts
                )
            else:
                output_data = self.futures_klines(
                    # spot=spot,
                    symbol=symbol,
                    interval=interval,
                    limit=limit,
                    startTime=start_ts,
                    endTime=end_ts
                )

this is fixed in the feature/asyncio branch that will be released as v1.0.0 shortly


dont render addwebapp if isWebApp

move  to main app router (router.js)
dont render if already isWebApp (see new helper)
Definition of Done

[x] move to main router (done in beta branch)
[x] dont render if already iswebapp

Checked on Dev, QA and eWallet environment.
Work as expected.
If user have already setted up the GoodDollar shortcut, they don't receive the "Add to homescreen" pop-up.
Used devices:
iPhone X // 12.4
LG Nexus 5X // 8.1


Web deployment

This adds a new doc for deploying OMERO.web separately to the server and makes the apache deployment as deprecated.
Not sure if we really want to replicate info from the existing nginx page but I think the plan might be for this to replace that page in the future (especially when the parallel apache page is removed).
Staged at https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/index.html

Thank you @hflynn  for all your help. Looks great. Code PR in https://github.com/openmicroscopy/openmicroscopy/pull/4805

Testing...
$ virtualenv omeroweb
$  source omeroweb/bin/activate
(omeroweb) $ pip install omego
(omeroweb) $ omego download --ice 3.6 --branch OMERO-DEV-breaking-build py
(omeroweb) $ mv OMERO.py-5.3.0-m2-1083-a3ff718-ice36-b754 OMERO.py
(omeroweb) $ pip install -r OMERO.py/share/web/requirements-py27-trial.txt
(omeroweb) $ cd OMERO.py
(omeroweb) $ bin/omero config set omero.web.application_server development
(omeroweb) $ bin/omero config set omero.web.debug True
(omeroweb) $ bin/omero web start
Traceback (most recent call last):
  File "manage.py", line 38, in <module>
    import settings
  File "/Users/wmoore/Desktop/testweb/OMERO.py/lib/python/omeroweb/settings.py", line 35, in <module>
    import omero
  File "/Users/wmoore/Desktop/OMERO/openmicroscopy/dist/lib/python/omero/__init__.py", line 43, in <module>
    raise Exception(msg)
Exception:

        ERROR: Ice version mismatch!

        Your OMERO code has been compiled using Ice version 3.5.1
        but you seem to have Ice version 3.6.2 installed. If you need
        help understanding this issue, please send this error message
        to the OME community:

        http://www.openmicroscopy.org/site/community

        Debugging Info:
        --------------
        VERSION=5.3.0-m3-ice35-SNAPSHOT
        PYTHONPATH=/Users/wmoore/Desktop/testweb/OMERO.py/lib/python/omeroweb:/Users/wmoore/Desktop/CRS4:/Users/wmoore/Desktop/OPENLAYERS4OMERO/openlayers4omero:/Users/wmoore/Desktop/WEBTEST:/Users/wmoore/Desktop/WEBTAGGING/webtagging:/Users/wmoore/Desktop/OMERO-MARSHAL/omero-marshal:/Users/wmoore/Desktop/FIGURE:/Users/wmoore/Desktop/OMERO/openmicroscopy/dist/lib/python:/Users/wmoore/Desktop/testweb/OMERO.py/lib/python:/Users/wmoore/Desktop/testweb/OMERO.py/var/lib:/Users/wmoore/Desktop/testweb/OMERO.py/lib/fallback:/Users/wmoore/Desktop/testweb/omeroweb/lib/python27.zip:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/plat-darwin:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/plat-mac:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/plat-mac/lib-scriptpackages:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/lib-tk:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/lib-old:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/lib-dynload:/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7:/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-darwin:/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk:/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac:/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac/lib-scriptpackages:/Users/wmoore/Desktop/testweb/omeroweb/lib/python2.7/site-packages

@will-moore could you review your venv version. Something corrupted your PYTHONPATH. We have that deployed on CI https://ci.openmicroscopy.org/job/WEB-DEV-trial-deploy/7/console and it works even then cowfish has Ice 3.5 installed globally

@aleksandra-tarkowska I had to manually adjust the indenting at the top of the osx-ice3.6 walkthrough to make the build happy so you'll need to adjust your autogen for that file next week.

Leaving @aleksandra-tarkowska to open a PR against https://github.com/ome/omeroweb-install

I fixed that before omeroweb-install was transferred to ome. This in only placeholder. We will test it once autogen job turns green.
The University of Dundee is a registered Scottish Charity, No: SC015096

https://ci.openmicroscopy.org/job/OMERO-DEV-merge-docs-autogen-omeroweb/4/ :smile:


AOA improvements

This pull request is to make it much easier for developers to use AOA/USB. Currently the way to make this work is very confusing.
This will allow developers to use the onSdlEnabled callback in the SdlBroadcastReceiver to start their service with the proper USB extras. Then in the onStart method in their service, they can check the intent for the proper extras.
public void startProxy(Intent intent) {
        if (proxy == null) {
            try {
                BaseTransportConfig transport = null;
                if(intent!=null && intent.hasExtra(UsbManager.EXTRA_ACCESSORY)){ //If we want to support USB transport
                    if(android.os.Build.VERSION.SDK_INT <= android.os.Build.VERSION_CODES.HONEYCOMB){
                        Log.e(TAG, "Unable to start proxy. Android OS version is too low");
                        return;
                    }
                    //We have a usb transport
                    transport = new USBTransportConfig(getBaseContext(),(UsbAccessory)intent.getParcelableExtra(UsbManager.EXTRA_ACCESSORY));
                }else{
                    //If we don't want anything but USB then would just do a return here and bail
                   // transport = new MultiplexTransportConfig(getBaseContext(), APP_ID);
                    Log.e(TAG, "Not usb intent. Shutting down");
                    stopSelf();
                    return;
                }
                proxy = new SdlProxyALM(this, APP_NAME,
                        true, APP_ID,
                        transport);
            } catch (SdlException e) {
                e.printStackTrace();
                // error creating proxy, returned proxy = null
                if (proxy == null) {
                    stopSelf();
                }
            }
        }
    }

A more detailed guide can be written around exactly how to connect the app via USB, but this is the main point.

Would it be possible to push a hello_sdl_android branch that could be used with to test the changes?

Yea I can do. I'll tag it here when pushed.

See hello sdl repo for example: https://github.com/smartdevicelink/hello_sdl_android/tree/feature/aoa_improvements


single term for 'haustorium' for two quite different structures

This arose in https://github.com/geneontology/go-ontology/issues/13193 when I was trying to determine if 'haustorium' should be moved down to 'plasma membrane bound cell projection'.
However there are at least two kinds of haustoria:

one in plant parasites of other plants roots. From what I have gleaned so far, these may be more appropriate as a cellular bridge. @tberardini do you know more about these?
From PMID:26322059

Parasitic plants form a haustorium, a unique multicellular invasion organ common to all
parasitic plants. In the Orobanchaceae root parasites, globular-shaped haustoria invade host
roots and form direct vascular connections with host plants, which likely enables nutrient transfer
(Hibberd and Jeschke, 2001; Yoshida and Shirasu, 2012).

one in fungal parasites of plants @dianeoinglis, do you know anything about these in fungi?
From PMID:25699068

On the leaf surface rust spores produce germ tubes which grow on the plant surface. Over
a stoma an appressorium is formed, a special structure from which the infection hypha is invading
the leaf tissue (Hoch and Staples, 1987). Within the substomatal vesicle intercellular hyphae
spread the plant tissue and haustorial mother cells are formed adjacent to mesophyll cells.
Haustoria are formed after penetrating the plant cell wall without vulnerating the plant plasma
membrane. The haustoria grow within the living plant cells and constitute an intimate contact to
the plant cells cytoplasm. As a result the cytoplasm of host and fungus remain separated by the
host plasma membrane, the fungal plasma membrane and between the so-called extrahaustorial
matrix (Voegele et al., 2009).

@krchristie I don't know more about parasitic plant haustoria than what you've stated.  Wish I could be more helpful but I'd be digging in the same literature that you currently are exploring and starting from the same base level of knowledge.

from: https://en.wikipedia.org/wiki/Haustorium

In botany and mycology, a haustorium (plural haustoria) is a structure that grows into or around another structure to absorb water or nutrients. In botany, this may refer to a cotyledon,[1] or to the root of a parasitic plant (such as the broomrape family or mistletoe) that penetrates the host's tissue and draws nutrients from it. In mycology, it refers to the appendage or portion of a parasitic fungus (the hyphal tip), which performs a similar function.

In light of the fact that the same word is used to describe structures from multiple taxa that perform a similar function, and that haustorium are not formed by humans so this is not a high priority term for mammalian annotation, I am going to leave this as is. If there becomes a need for more specific terms, they can be created with any additional specific parentage that is appropriate.


CodeIgniter: Error message in Project Launch results

I created a group project. At the top of the resulting code was this:

A PHP Error was encountered

Severity: Notice

Message:  Undefined offset: 0

Filename: project_launch/result_page.php

Line Number: 51

Otherwise the process worked as expected.

The relevant line of code is if ($translator[0]). If you add a translator the error will probably go away. It does make me wonder why it worked before. That line of code should probably be if (!empty($translator)) or something anyway.

I wonder if https://github.com/notartom/librivox-catalog/commit/4d9d09ccd7d9cac88ed9bda8cfad0354e30029c9 is somehow causing this...

Aha - looking at the POST request to https://librivox.org/add_project once the form has been filled out, the field that's being sent is trans_id[1]. It looks like application/libraries/Form_generator_item.php is what transforms these trans_id[X] into the trans_list that's used in the view. This transformation is conditional on trans_id existing in the POST data:
        //translators
        if (isset($fields['trans_id']))
        {
            $this->_process_authors('trans', $fields);

So we need to change to something that checks for "are there fields that start with 'trans_id'?" I also suspect authors has the same problem, not sure why it's not being hit...

I also suspect authors has the same problem, not sure why it's not being hit...

Maybe because the project launch form enforces that there is at least one author before you can submit it?

Actually, sticking a print() right after that if isset shows up in the page, so the conditional is somehow true (even if the field itself is trans_id[0]). So maybe your first idea was correct...

Addressed in https://github.com/notartom/librivox-catalog/commit/5fee26206e12e5516d06961d4cce55435ba38783.

We're not sure how it ever worked, but this patch fixes it with an isset.

I figured out why it worked before.
In public_html/index.php the newer CodeIgniter enables more errors via the call to error_reporting(). If you set the error_reporting value more similar to previously then this error also goes away (or vice versa you can trigger the error in the master branch be enabling more error reporting).
This also explains why the typo fixed in 53919a6afc32feadc8d10deb6800e96bf976d682 didn’t cause complete failure of the validator before.
I think I prefer the extra error reporting, especially during development testing. I have actually been upping the error level during a lot of my testing already.
I actually get PHP errors all over the place now on the CI3 branch running on my PHP7 system and I suspect these mostly come down to the different level of error_reporting and “Step 10: Many functions now return NULL instead of FALSE on missing items”. I hope to look into each of these more soon. I wonder how many of them are outright bugs and how many are just questionable code that works anyway.
But first I think I want to continue going through the 20 step list of the CodeIgnitor upgrade process. I am up to about 15 (not that I fully understand everything up to that point).

Just a quick note in case I hadn't made it obvious before - the upgrade to
PHP7 is separate from the upgrade to CodeIgniter 3. The current plan is to
run CI3 on PHP5.6 (and that's what's currently being tested on the
staging/dev server) and then upgrade the OS (and the PHP version)
underneath.
On Sun., Feb. 21, 2021, 16:12 kgroeneveld<EMAIL_ADDRESS>wrote:

We're not sure how it ever worked, but this patch fixes it with an isset.
I figured out why it worked before.
In public_html/index.php the newer CodeIgniter enables more errors via
the call to error_reporting(). If you set the error_reporting value more
similar to previously then this error also goes away (or vice versa you can
trigger the error in the master branch be enabling more error reporting).
This also explains why the typo fixed in 53919a6
https://github.com/LibriVox/librivox-catalog/commit/53919a6afc32feadc8d10deb6800e96bf976d682
didn’t cause complete failure of the validator before.
I think I prefer the extra error reporting, especially during development
testing. I have actually been upping the error level during a lot of my
testing already.
I actually get PHP errors all over the place now on the CI3 branch running
on my PHP7 system and I suspect these mostly come down to the different
level of error_reporting and “Step 10: Many functions now return NULL
instead of FALSE on missing items”. I hope to look into each of these more
soon. I wonder how many of them are outright bugs and how many are just
questionable code that works anyway.
But first I think I want to continue going through the 20 step list of the
CodeIgnitor upgrade process. I am up to about 15 (not that I fully
understand everything up to that point).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LibriVox/librivox-catalog/issues/90#issuecomment-782927398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4RJHQFXZ4B24PLYBSCPUTTAFZLLANCNFSM4XSNCMZA
.

With the upgrade complete, this can be closed.


TypeError: any[] is not assignable to keyof ClientEvents

Describe the bug
Error occurs at:
https://github.com/ghostdevv/jellycommands/blob/b607c2a9ce9ac32292cb81faef611d2e583b624f/packages/jellycommands/src/events/register.ts#L23
This error is not detected in build and linting scripts.
Reproduction
N/A
Logs
Argument of type 'any[]' is not assignable to parameter of type '[data: ApplicationCommandPermissionsUpdateData] | [autoModerationActionExecution: AutoModerationActionExecution] | [autoModerationRule: AutoModerationRule] | ... 67 more ... | [guildScheduledEvent: ...]'.
  Type 'any[]' is not assignable to type '[guildScheduledEvent: GuildScheduledEvent<GuildScheduledEventStatus>, user: User]'.
    Target requires 2 element(s) but source may have fewer.

System Info
N/A

Fixed by #164


Update CLI.md

Added a section on sending flags for tasks.

Lowercase F. Also, would you be able to contribute this as a recipe?

Corrected the upper case F. Regarding the recipe, we will be authoring some gulp tasks this week and I can give a subsequent pull request later.

I think this is fine.  We should put the answer into a recipe though instead of linking externally.


Not all exceptions converted to error messages in the C interface

Hi together,
According to the documentation in the C interface all exceptions thrown by the C++ interface should be converted to error messages in the C interface. I think this is not the case if a pdf document contains an invalid root.
The implementation of qpdf_get_root is the following:
qpdf_oh qpdf_get_root(qpdf_data qpdf)
{
    QTC::TC("qpdf", "qpdf-c called qpdf_get_root");
    return new_object(qpdf, qpdf->qpdf->getRoot());
}

and the C++ implementation of getRoot is:
QPDFObjectHandle
QPDF::getRoot()
{
    QPDFObjectHandle root = this->m->trailer.getKey("/Root");
    if (! root.isDictionary())
    {
        throw QPDFExc(qpdf_e_damaged_pdf, this->m->file->getName(),
                      "", this->m->file->getLastOffset(),
                      "unable to find /Root dictionary");
    }
    return root;
}

In case the root is not a dictionary an unhandled exception will be thrown if i am not missing something.
Thanks and kind regards,
Lars

Coincidentally, someone else just found this same issue. A fix is in progress.

Great thanks a lot

I pushed a fix for this. It will be in qpdf 10.5 which I will try to get out soon as there are a handful of fixes that need to get out.


[wip] Implement our own JSON decoder

Pull Request Test Coverage Report for Build 467133174

473 of 490 (96.53%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 99.57%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/json/lex.go
437
454
96.26%

Totals

Change from base Build 427575915:
-0.2%

Covered Lines:
7173

Relevant Lines:
7204

💛  - Coveralls

Pull Request Test Coverage Report for Build 467133174

473 of 490 (96.53%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 99.57%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/json/lex.go
437
454
96.26%

Totals

Change from base Build 427575915:
-0.2%

Covered Lines:
7173

Relevant Lines:
7204

💛  - Coveralls


not_found_404: points to main repo

link for missing pages points to dash-mantine-components repository instead of dmc-docs

Thanks, corrected that.


Add shard group duration section to schema page

Adds a separate shard group duration management section to the schema design page.
It outlines the information provided in #800.

@rkuchan good to go. I removed a level of titles to make deep links easier to use.

Thanks, @beckettsean!


StringEnumConverter attribute not generated for nullable enums

Our datamodel has nullable and not-nullable enum fields.
The enum definitions are correctly generated like this:
public enum Currency
{
  [System.Runtime.Serialization.EnumMember(Value = "EUR")]
  EUR = 0,

  [System.Runtime.Serialization.EnumMember(Value = "GBP")]
  GBP = 1,

  [System.Runtime.Serialization.EnumMember(Value = "USD")]
  USD = 2
}

public enum Gender
{
  [System.Runtime.Serialization.EnumMember(Value = "Male")]
  Male = 0,

  [System.Runtime.Serialization.EnumMember(Value = "Female")]
  Female = 1,
}

However, the data model is generated like this:
[Newtonsoft.Json.JsonProperty("currency", Required = Newtonsoft.Json.Required.Always)]
[Newtonsoft.Json.JsonConverter(typeof(Newtonsoft.Json.Converters.StringEnumConverter))]
public Status Currency { get; set; }

[Newtonsoft.Json.JsonProperty("gender", Required = Newtonsoft.Json.Required.AllowNull)]
public Gender? BounceType { get; set; }

Apparently the StringEnumConverter attribute is not being generated for nullable fields.
Any ideas?

This is a bug...

Cannot reproduce, see unit test in commit

Hmm, we are using NJsonSchema version 8.5.6255.20253
Could  it be that this has been fixed in a newer version?

Yes, this is must be it...

Have you tried with the latest version?

Apologies - I thought that since your unit test proved that the issue is fixed in the latest version, you'd just close the issue.
Unfortunately we are currently tied to using 8.5.6255.20253. We have worked around the problem, and I have no easy way to test our specific setup with a different version of NJsonSchema.
Thank you for following up though!

Ok thanks for your reply. What is stopping you from upgrading?

@RSuter it used in a part of our toolchain that we do not directly control...


Is it possible to define external jars to be included to the created server ?

We would like define a library fileset in the server.xml to the oracle jdbc driver.
<library id="db.oracle">  <fileset dir="${server.config.dir}/dblibs/" includes="ojdbc8.jar" /> </library>
It would be great do copy custom resources into the server.config.dir during package.
Then it is possible to run that packaged server on windows or linux without modifying the library path ..
Is that possible or is there a other solution ?
thanks
claus

Would be the ${shared.resource.dir} the place to put db libraries in ?

Yes, shared.resource.dir would be the place to add the db libraries. You can use the maven-dependency-plugin to do this during package:
<plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-dependency-plugin</artifactId>
    <version>3.0.0</version>
    <executions>
        <execution>
            <id>copy-derby</id>
            <phase>package</phase>
            <goals>
                <goal>copy</goal>
            </goals>
            <configuration>
                <artifactItems>
                    <artifactItem>
                        <groupId>org.apache.derby</groupId>
                        <artifactId>derby</artifactId>
                        <version><IP_ADDRESS></version>
                        <outputDirectory>${project.build.directory}/liberty/wlp/usr/shared/resources</outputDirectory>
                    </artifactItem>
                </artifactItems>
            </configuration>
        </execution>
    </executions>
</plugin>

Thank you very much for the info ...

@c-koell There is now a more elegant way of copying dependencies into the target server. See the documentation here for information on configuring the copyDependencies parameter in the libert-maven-plugin.

@cherylking ... thanks for the info !


Should snapping 45° angles be a thing?

Apparently the joystick only teleports? Not sure if there's a angle snapping to turn around kind of feature.

Snap turning should already be a feature in both default controllers.

Snap turning should already be a feature in both default controllers.

oh my bad, I ment smooth turning

Smooth turning is currently not supported. I'm skeptical about the feature due to motion sickness, but feel free to give it a shot and create a PR.


Update doc about default number of threads

To make it consistent with what the doc says:

Defaults to a threadpool of size 2 * num_cpus.

oops...Sorry but I just checked out the commit history. Seems it is the doc that needs to be updated.

Thanks!


Broadcast filter option

This adds a filter regex to the Veza broadcast options so that it will now only broadcast data to its clusters.

Closing as it is useless since people aren't meant to connect to the master by hand.


Terminate other buttons w/ extreme prejudice

Hopefully this'll fix the mysteriously doubling buttons I sometimes see.


Allow static overlay when the impute option has been used

After executing xtevent with the impute option, calling xteventplot, overlay(static) shows an error. This happens because overlay(static) needs the imputed policyvar to be available in the database. Nonetheless, the imputed policyvar is available only if the user specified the saveimp suboption within the impute option.

Summary: In this issue we fixed a bug with static overlay plots when the policy variable was imputed.
Thread continues in #50


bump dep to v0.5.0

Latest support version. We now have some changes to the lock file.
Makefile also supports the latest pinned version.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [costinm]

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

@zachgersh: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
51a6e0c8c3e22fbbba2ad7f6cfdbf495234222ae
link
/test e2e-simple

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

/test e2e-simple


Is this extension still in preview?

I've installed this extension yesterday and noticed today that the title in Visual Studio is Power Platform Tools for VS 2022 [Preview]
Calling dotnet restore removes important parts of the files project.asset.json and .nuget.dgspec.json which will result in an unbuildable project, until you clean the obj directory and restore the nuget packages again. This issue also makes me believe that the extension is still in preview.
If it is still in preview, I think that this should be mentioned here.

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

ID: f9dcb052-17a9-544f-ced8-9aa64b8defed
Version Independent ID: f9dcb052-17a9-544f-ced8-9aa64b8defed
Content: What is Power Platform Tools for Visual Studio - Power Platform
Content Source: power-platform/developer/devtools-vs.md
Service: power-platform
Sub-service: developer
GitHub Login: @phecke
Microsoft Alias: pehecke

@darshSWL No, the extension is no longer in preview. You can ask a question on the download site: Q & A
Also, I forwarded your issue to one of the extension developers.

@darshSWL Could you share repro steps with<EMAIL_ADDRESS>We plan to remove preview tag in coming months.


Try fix invalid entities in device networks

This PR tries to get rid of the PVS error spam for invalid entities on device lists & network configurators.

thank GOD

can you make them not serialize or just delete the invalid entity id when map saving

can you make them not serialize or just delete the invalid entity id when map saving

Device lists already remove invalid ids when saving, though configurators didn't. But ideally it just shouldn't be needed if this is fixed now.


Define the "value" in a Variable Query Parameter which is defined on the interface

Description
I've defined a query parameter on an interface like this:
public interface IApi
{
    [Query("api-version")]
    string ApiVersion { get; set; }

    [Get("users")]
    Task<List<User>> GetUsersAsync();
}

What I would to do is set the value for this Query Parameter like this:
public interface IApi
{
    [Query("api-version", "abc")]
    string ApiVersion { get; set; }

    [Get("users")]
    Task<List<User>> GetUsersAsync();
}

This makes the code more clean then just adding ?api-version=abc to every method like described here: https://github.com/canton7/RestEase#constant-query-parameters
Version Info

RestEase version: [1.5.5]

This basically means extending the Header attribute model to query parameters. It's do-able (and I think I started down this route at some point, before it got complex and I realised noone was asking for it, at that point)

It would be useful to have this.
It makes the API more clean.


Getting errors in ios build, working fine in android

I'm getting error something like this in ios build. In android app is compiled without any issues. Tried clearing RN cache. Re-installing node_modules and also deleting whole Pods folder and installing again. Nothing worked.

Hey @mrousavy, I got the same error when building production variant on iOS. Here is the output from the command that you asked for.
Updating spec repo `cocoapods`
  $ /usr/bin/git -C /Users/michal/.cocoapods/repos/cocoapods fetch origin --progress
  remote: Enumerating objects: 15, done.
  remote: Counting objects: 100% (15/15), done.
  remote: Compressing objects: 100% (9/9), done.
  remote: Total 9 (delta 6), reused 0 (delta 0), pack-reused 0
  From https://github.com/CocoaPods/Specs
     2f702c72a0b3..1ff66cd869a3  master     -> origin/master
  $ /usr/bin/git -C /Users/michal/.cocoapods/repos/cocoapods rev-parse --abbrev-ref HEAD
  master
  $ /usr/bin/git -C /Users/michal/.cocoapods/repos/cocoapods reset --hard origin/master
  HEAD is now at 1ff66cd869a3 [Add] GameFrameworkTool 1.1.2
Updating spec repo `trunk`
Auto-linking React Native modules for target `HammerWallet`: BVLinearGradient, RNBootSplash, RNCClipboard, RNDevMenu, RNDeviceInfo, RNGestureHandler, RNOS, RNReanimated, RNSVG, RNScreens, RNSentry, TcpSockets, VisionCamera, react-native-bignumber, react-native-config, react-native-flipper, react-native-get-random-values, react-native-mmkv, react-native-randombytes, react-native-restart, react-native-safe-area-context, react-native-udp, and vision-camera-code-scanner
Analyzing dependencies
Fetching podspec for `DoubleConversion` from `../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`
Fetching podspec for `RCT-Folly` from `../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec`
Fetching podspec for `boost` from `../node_modules/react-native/third-party-podspecs/boost.podspec`
Fetching podspec for `glog` from `../node_modules/react-native/third-party-podspecs/glog.podspec`
Downloading dependencies
Installing BVLinearGradient (2.5.6)
Installing CocoaAsyncSocket (7.6.5)
Installing DoubleConversion (1.1.6)
Installing FBLazyVector (0.67.2)
Installing FBReactNativeSpec (0.67.2)
Installing Flipper (0.142.0)
Installing Flipper-Boost-iOSX (<IP_ADDRESS>.11)
Installing Flipper-DoubleConversion (3.1.7)
Installing Flipper-Fmt (7.1.7)
Installing Flipper-Folly (2.6.7)
Installing Flipper-Glog (0.3.6)
Installing Flipper-PeerTalk (0.0.4)
Installing Flipper-RSocket (1.4.3)
Installing FlipperKit (0.142.0)
Installing GTMSessionFetcher (1.7.1)
Installing GoogleDataTransport (9.1.2)
Installing GoogleMLKit (3.0.0)
Installing GoogleToolboxForMac (2.3.2)
Installing GoogleUtilities (7.7.0)
Installing GoogleUtilitiesComponents (1.1.0)
Installing MLImage (1.0.0-beta2)
Installing MLKitBarcodeScanning (2.0.0)
Installing MLKitCommon (6.0.0)
Installing MLKitVision (4.0.0)
Installing MMKV (1.2.13)
Installing MMKVCore (1.2.13)
Installing OpenSSL-Universal (1.1.180)
Installing PromisesObjC (2.1.0)
Installing Protobuf (3.20.1)
Installing RCT-Folly (2<IP_ADDRESS>-v2)
Installing RCTRequired (0.67.2)
Installing RCTTypeSafety (0.67.2)
Installing RNBootSplash (4.1.3)
Installing RNCClipboard (1.8.5)
Installing RNDevMenu (4.0.2)
Installing RNDeviceInfo (8.7.0)
Installing RNGestureHandler (2.3.0)
Installing RNOS (1.2.6)
Installing RNReanimated (2.4.1)
Installing RNSVG (12.3.0)
Installing RNScreens (3.13.1)
Installing RNSentry (3.3.1)
Installing React (0.67.2)
Installing React-Core (0.67.2)
Installing React-CoreModules (0.67.2)
Installing React-RCTActionSheet (0.67.2)
Installing React-RCTAnimation (0.67.2)
Installing React-RCTBlob (0.67.2)
Installing React-RCTImage (0.67.2)
Installing React-RCTLinking (0.67.2)
Installing React-RCTNetwork (0.67.2)
Installing React-RCTSettings (0.67.2)
Installing React-RCTText (0.67.2)
Installing React-RCTVibration (0.67.2)
Installing React-callinvoker (0.67.2)
Installing React-cxxreact (0.67.2)
Installing React-jsi (0.67.2)
Installing React-jsiexecutor (0.67.2)
Installing React-jsinspector (0.67.2)
Installing React-logger (0.67.2)
Installing React-perflogger (0.67.2)
Installing React-runtimeexecutor (0.67.2)
Installing ReactCommon (0.67.2)
Installing Sentry (7.10.0)
Installing SocketRocket (0.6.0)
Installing TcpSockets (3.3.2)
Installing VisionCamera (2.13.2)
Installing Yoga (1.14.0)
Installing YogaKit (1.18.1)
Installing boost (1.76.0)
Installing fmt (6.2.1)
Installing glog (0.3.5)
Installing libevent (2.1.12)
Installing nanopb (2.30908.0)
Installing react-native-bignumber (0.1.7)
Installing react-native-config (1.4.5)
Installing react-native-flipper (0.142.0)
Installing react-native-get-random-values (1.7.2)
Installing react-native-mmkv (2.1.0)
Installing react-native-randombytes (3.6.1)
Installing react-native-restart (0.0.24)
Installing react-native-safe-area-context (3.4.1)
Installing react-native-udp (2.7.0)
Installing vision-camera-code-scanner (0.2.0)
Generating Pods project
Integrating client project
Pod installation complete! There are 77 dependencies from the Podfile and 84 total pods installed.

Installing OpenSSL-Universal (1.1.180)

hmm, looks like everything's being installed correctly... 🤔
could you share a reproduceable example? maybe try upgrading to RN 0.68?

I tried upgrade to RN 0.68 but it didn't help. Here is reproducible example repo. I'm not sure if it helps but I run on M1 chip.

Thanks for the repro, we'll take a look once we got some free time!
To make sure: can you build and run the Example app here?

Yes the Example app works but only when I change iOS version in Deployment Info section to iOS 11.

Hi @mrousavy I had some time to debug this issue a bit more and it seems that it may be connected to flipper. When I comment out use_flipper in Podfile release configuration build runs without any errors. Do you think this might be it?

Oh, yea Flipper also uses OpenSSL as a dependency, maybe that's the issue?

https://github.com/margelo/react-native-quick-crypto/issues/121

Thank you for posting you issue, but it is a duplicate of:
-https://github.com/margelo/react-native-bignumber/issues/59


Adding some risky secrets 08/15/23 18 43

altered password in 23.yaml and made s5.yaml

@JoeVayApiiro Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Thank you!

I recently started using Linkding and I wanted to express my appreciation for it. It was easy to set up, it works great, it looks better than most paid products, it has all the features I need but none of its functionality is extraneous, and browser integration is intuitive.
Linkding is great, so thank you @sissbruecker for developing it!

Thanks for the kind words @vladh !


Prettier precommit hook

Purpose

...

Adds initial setup for running prettier during commit so that code styles are always followed
#35
Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

adding prettier styling to the repo.
Also adds a nice prettier configuration badge to the readme file to show that prettier is used

@johnpapa I figured if i went ahead and opened a PR then you can pull it down and play around if you wanted until we find what works best

update to 80 chars and i think we're good.
@wardbell should try this out before we merge

@wardbell - please take a look. if you like this, let's merge

thanks for the PR!


OGD-169: Use :hub:shared inside Dockerfile (fix)

The build passed locally and in Travis, but failed in Concourse because the new library from previous commit was not included in the Dockerfile build.

Co-authored-by: James Howes<EMAIL_ADDRESS>

Thanks, all.


update documentation link to github wiki in readme

@maf-embraceit could we also change the text of the intro to:
AltaPay, headquartered in Denmark, is an internationally focused fintech company within payments with the mission to make payments less complicated. We help our merchants grow and expand their business across payment channels by offering a fully integrated seamless omni-channel experience for online, mobile and instore payments, creating transparency and reducing the need for manual tasks with one centralized payment platform.
AltaPay’s platform automizes, simplifies, and protects the transaction flow for shop owners and global retail and e-commerce companies, supporting and integrating smoothly into the major ERP systems. AltaPay performs as a Payment Service Provider operating under The Payment Card Industry Data Security Standard (PCI DSS).

sure, done

@emicha sure, done


fix: catch uninstalled github app

Apparently we »do« receive events from GitHub even for uninstalled Packit
GitHub App. Therefore, ignore raised exceptions during processing of the
webhooks.
Fixes #1542
Fixes PACKIT-2397
Signed-off-by: Matej Focko<EMAIL_ADDRESS>

Apparently we »do« receive events from GitHub even for uninstalled Packit
GitHub App. T

Can we create a cleanup handler for this event?

Can we create a cleanup handler for this event?

IIRC we get the same exception for suspended installations, I'm not sure about the UX once you resume and would need to verify fas again

either way I think it's out of scope of this PR; even removing from installed repos would not matter, since it is not checked at all


Fixed sinks having 500 water instead of 9999999 water

About the PR

I changed the amount of water in the sinks to 9999999, similar to this PR https://github.com/Goob-Station/Goob-Station/pull/762
Why / Balance

The PR above removed water regeneration to reduce lag, and made it so SinkEmpty has 9999999 water. The normal sink parents off of SinkEmpty, but then redefines their quantity to 500, so every sink below that inherits their 500 water instead of the intended 9999999 water. That means 3 out of 5 sinks that get mapped into the game have neither infinite water nor regenerative water, and quickly run out.
Technical details

I changed the Quantity: 500 in Sink and SinkStemlessWater to have the intended 9999999 water.
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

fix: Fixed some sinks having 500u of water instead of infinite water.

Stay hydrated

I wonder what makes solution regen so laggy :bigthonk:


Condensed Prefs Menu Options into 2 checkboxes

Changed logging on and logging off options in the preferences menu to a single "Enable Logging" checkbox, did the same with the commit sort options

Thanks, @MarthanielX : this mostly looks good. I merged it, but then backed off because I realized that the key missing point is that the checkboxes aren't sticky. When you quit Elegit and start it up again, the checkboxes are blank again. It also isn't clear to me that the checkboxes at start reflect the current state of the preferences. Can you look into?


Setting to use upper case hint letters

I think that upper case letters would be easier to distinguish

[ ] it would also be nice if they were a setting where you could give a whitelist off the characters you want to use. For example if two letters look too much alike you can exclude one of them. It would also allow us to add symbols to the hints if we wanted.

I would also love to have this feature since it's really difficult to read the small letters.
Another thing that makes it more difficult to read letters is that the contrast is sometimes very low.

Yes, this is on the roadmap for when I implement ui settings. I'll get to than once I finish with the stable hints rewrite.

Awesome, thanks for all of the work you are putting into this. The extension is really good!


[Enhancement] improve persistent index l0 memory usage

This PR will improve persistent index l0 memory usage, and add some config items:
CONF_mInt64(l0_min_mem_usage, "2097152");   // 2MB
CONF_mInt64(l0_max_mem_usage, "104857600"); // 100MB

Whether l0 should flush or merge into l1, isn't longer controlled by l0_l1_merge_ratio any more. L0 should flush or merge into l1, when:
1. l0 memory usage >= config::l0_max_mem_usage, or
2. l0 memory usage >= config::l0_min_mem_usage && update module memory usage is in urgent

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


Create a LoRa transceiver add-on in v2

I'm not sure how you'd package it, but it would be neat to integrate a LoRa transceiver in the next version of the board. You could use it to send and receive text like messages using LoRa gateways.
Use cases:

Burst based text messaging
One way broadcast paging
SOS (that's a stretch without a GPS too, but maybe a coarse location could be calculated using multiple gateways w/o GPS.)

Imagine rendering global mobile communication infrastructure obsolete by introducing Lora circuit and application utilizing public keys for it in beepberry lmao

LoRa is a really good idea

I'm thinking something like what they do with atak-civ.
https://meshtastic.org/docs/software/community/community-atak
I mean, no gui. Just the concept of the LoRa bridge for messaging.

Definitely no GUI!
And with mesh?
https://hackaday.com/2020/02/26/lora-mesh-network-with-off-the-shelf-hardware/

Someone on the uConsole forums posted this. It's like beepberry with an mcu and onboard LoRa. https://www.lilygo.cc/products/t-deck

Here's another LoRa/berry device.
https://hackaday.com/2022/05/25/long-distance-text-communication-with-lora/
https://github.com/BigCorvus/LORA-QWERTY-Communicator


Links without underline

Hi @colineckert  !
Would you consider showing links - internal and external - without the underline?

I would! I go back and forth on this myself. I am working on support for the Style Settings plugin and will include underline/no underline toggle options for both internal and external links. I should have this in the next release, expected to publish by tomorrow. Thanks!

Colin, hi.
You're always very kind.
Responsiveness is key these days and you demonstrate it.
Kind regards,
Giovani di Gesù
https://gdigesu.envision.consulting
https://directions.envision.consulting
Main +55(11)3042-8860

On 9 Nov 2021, at 15:48, Colin Eckert @.***> wrote:

I would! I go back and forth on this myself. I am working on support for the Style Settings plugin and will include underline/no underline toggle options for both internal and external links. I should have this in the next release, expected to publish by tomorrow. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://bra01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcolineckert%2Fobsidian-things%2Fissues%2F6%23issuecomment-964434507&data=04|01|gdigesu%40envision.consulting|0efbfffaf2ec4b6828d608d9a3b17acf|718f3212162b4bf891c55c06573e715e|0|0|637720804979939047|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=t12F4okkmS4arIZzJZSfWEzic7IazF58u%2Bcd2uRo3%2BQ%3D&reserved=0, or unsubscribehttps://bra01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKTSVE6C25HCYINBJXJCZPTULFUF5ANCNFSM5HQ45JFA&data=04|01|gdigesu%40envision.consulting|0efbfffaf2ec4b6828d608d9a3b17acf|718f3212162b4bf891c55c06573e715e|0|0|637720804979949000|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2F72KZN%2BxkpIW%2BbylqZ8eERjNeV0gzbcDzvH9LZwNDug%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://bra01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|gdigesu%40envision.consulting|0efbfffaf2ec4b6828d608d9a3b17acf|718f3212162b4bf891c55c06573e715e|0|0|637720804979949000|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SKlEkW%2FkFax0Lvj5oqvXEn7JWqcIkKxlETgzrTTrl4Y%3D&reserved=0 or Androidhttps://bra01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|gdigesu%40envision.consulting|0efbfffaf2ec4b6828d608d9a3b17acf|718f3212162b4bf891c55c06573e715e|0|0|637720804979958956|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ebwZKfSHsFiC8GL1%2BOWTRAAzoQl6T3Hatpq0cf%2F8C%2FU%3D&reserved=0.

I want to make sure I am responsive to those that are kind of enough to use my theme, let alone take the time to share feedback. Thanks :)


Urgency Problem With Build in iOS 14.4+

I'm having a problem building my iOS app, it just pops up too many issues related to FlutterBluePlugin. I use this package in an important application, I would very much like this problem to be resolved. Any suggestion?

I resolved it importing
flutter_blue:
    git: git://github.com/pauldemarco/flutter_blue.git


feat: dark type

similar to the dark theme

and it should be automatic

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Okta Plugin

It would be great it there was an Okta plugin to make authentication simple, rather than having to manually do it for both Android and IOS (which Okta supports).

You should ask the Okta team. This's not the Flutter team's responsibility.

any news? btw if i go as Okta for iOS i saying it will work?


BagNet Training

Hi im using train_bagnet script code to train for my own traffic sign dataset but the loss didn't seem to decrease much and bagnet33 keeps predicting objects as background after few epochs. I already find New feature map size of my input using bagnet33. What did i do wrong

I don't have enough information to debug the issue. Here are some guesses

the input image size to the model could cause a problem. Note that we are using BagNet with small receptive fields. If the image size is too larger and the object is too large, BagNet might not be able to make any reasonable prediction. In our paper, we use 416×416 for COCO and VOC and 224×740 for KITTI
If you are using a customized dataset, make sure that you didn't mess up with the box coordinates: for example, accidentally treat x coordinates as y.
The training loss is a combination of loss_pos and loss_negative. You might want to add a coefficient to balance these two losses. In your case, you might want to have a larger weight on loss_negative https://github.com/inspire-group/DetectorGuard/blob/main/checkpoints/train_bagnet_coco.py#L92
Try other classic hyper-parameters tuning: e.g., try a different learning rate, use BagNet pre-trained on ImageNet, try a different BagNet (BagNet-17) etc.

my image size is 600x800 and the objects height range from 24 to 120. Do u think the size is too big ? i would appreciate if u could have a look at my dataset (sent via email). I will definitely cite your work when i finish this paper

could you use the letterbox() to resize and pad it to 416x416?
https://github.com/inspire-group/DetectorGuard/blob/main/utils/dataset.py#L18
you can send me the dataset, but I can't promise when I will have time to take a look at it. Unfortunately, training models for customized datasets is always a tricky process.
by the way, there is another repository that does not involve bagnet training, which might be easier for you to use
https://github.com/inspire-group/ObjectSeeker

Thanks alot !! i haven't read this repo but is this still a defense for backdoor attack

ObjectSeeker and DetectorGuard consider the same threat model: test-time adversarial patch attacks.
they are not primarily designed for the backdoor attack, though they might work for localized backdoor triggered at the test time


Connect-AzureRmAccount -> Login-AzureRmAccount

see also similar https://github.com/MicrosoftDocs/azure-docs/issues/7734

@danhaywood Thank you for your contribution.  I see the issue here, instead of changing to the alias Login-AzureRMAccount I would rather have a note in the document that mentions that Login-AzureRMAccount is now an alias for Connect-AzureRMAccount.  Connect-AzureRMAccount is the correct cmdlet starting in AzureRM.Profile 4.3.  I will submit a PR to your branch to add this note so we can get the PR in a state where we can merge.  I should have that done later today or early tomorrow.

I realized after raising this that I was using an older version of the scripts, which is why I couldn't find Connect-xxx and only Login-xxx worked.  So it might be that Login-xxx is an alias is deprecated, in which case reject this PR I raised.


bug(datepicker): Improper focus & allowing to focus on background elements of calendar dialog

Reproduction
Steps to reproduce:

Open the Datepicker Examples
Go to the "Basic datepicker" demo
Linear navigate till the end of the dialog
Linear navigate forward

Expected Behavior
It should not allow to navigate to the background elements, when calendar dialog is open
Actual Behavior
It is allowing to focus on the background elements of the open calendar dialog.

It is showing the focus on last element edit box, when open calendar dialog on first element edit box & linear navigate back.
It is focusing on the darkmode & strong focus toggles, when linear navigate forward after reaching end of calendar dialog using ChromeVox

Environment
OS| Browser | Screen reader
ChromeOS | Google Chrome 91.0.4472.167 | ChromeVox
macOS Catalina | Google Chrome 92.0.4515.131 | VoiceOver
Additional Notes
This is copied from an internal bug report found during a11y testing.

This was fixed by #24300


About java.lang.String#format(java.lang.String,java.lang.Object[])

Having evaluated this plugin against our code base, I can say this is a job very well done. Kudos!
One issue I hit while using the bundled jdk-unsafe signatures is this violation:
[ERROR] Forbidden method invocation: java.lang.String#format(java.lang.String,java.lang.Object[]) [Uses default locale]

Indeed, this method can produce locale-dependent results. But in our code base I found over 400 calls to this method where the format pattern only contains %s format specifiers. As best as I can tell we never pass in objects which implement java.util.Formattable. So the results should be locale-independent; to vacuously add Locale.ROOT in hundreds of places seems overkill.
I guess making the plugin smart enough to detect this kind of safe method usage is out of scope. So my question is whether there is some way to exclude just this signature. If at all possible I'd like to avoid having to maintain a modified copy of the jdk-unsafe signature list.

Hi,
sorry for the late reply (was very busy)!
At the moment, I am not sure how the best way is to make this smarter. Currently you can use @SuppressForbidden only, but that won't help for this use-case.
As forbiddenapis is only a static checker, there is no way to check for string contents (this is more something for tools like PMD). So I don't think that automatic exclusion here is fine. Especially, as %s can be applied to formattables.... On Lucene/Solr/Elasticsearch code we just added the Locale everywhere, although it's boilerplate.
One possibility would be some "exclusion" pattern (something like a negative signatures file) so you can slecetively revert patterns. For now, your only chance is to fork the signatures file into your project. BTW: You can also publish it on Maven Central or your local Artifactory repo and refer to it from your project's plugin and reuse it somewhere else!

String.format has other shady things going on too. For instance, if you pass a Date, that will format using the default time zone of course. And for some reason, they didn't think it worth putting a timeZone() method on Formatter, so you can't even implement a Formattable correctly if you want it in another zone. >:-/

@trejkaz interesting :). That'd be an argument for adding Date#toString to one of the plugin's blacklists. Though in most cases (as in your example) the invocation of #toString is implicit :thinking: .
@uschindler apologies for not responding to your comment (also very busy here ;) ). I think adding support for exclusions would be a nice to have. When I get back to reviewing this plugin (could be weeks/months) I might look into contributing such support. Indeed, until that time I could fork the signatures file.

@trejkaz interesting :). That'd be an argument for adding Date#toString to one of the plugin's blacklists. Though in most cases (as in your example) the invocation of #toString is implicit 🤔.

That is one problem that impossible to solve (very similar to the formatter problem). You cannot do that with static code analysis, because the runtime type is unknown at compile time (in most cases).
In Lucene we had the same problem: As we wanted to use BCP47 language tags everywhere we tried to forbid Locale#toString(), as Locale#toLanguageTag() should be used. Unfortunately this did not work as expected, as toString() is implicitely called by StringBuilders all the time (same for Java 9 indy string concats, where the toString() is called far inside the StringConcatFactory invokedynamic call).
So I have no idea how to fix the underlying issue. An "exclusion" is just a workaround!

All those methods on Object are particularly troublesome. I figured it might be possible to detect the problem at runtime, but that doing it even at compile time was impossible.
invokedynamic adds a bunch of extra fun for APIs all over the place too. Now you pretty much have to ban methods just by name, right? :)

See also #123 for an explanation why that's not possible to detect with a static byte code analysis tool. So I will close this as wont fix.


Support for extensionElements

Is your feature request related to a problem? Please describe.
Certain ServiceTask customizations like camelTask use extensionElements to supply additional properties
Describe the solution you'd like
Read/Write support for extension elements

Duplicates #36


Remove WrapperArray

What problem does this feature solve?
IMHO,
Since WrapperArray has only methods which is the small wrapper method of Array,
it is not absolutly necessary.
I think that the Array of Wrapper is easier to understand than WrapperArray.
I do not think the methods of WrapperArray are often used.
By removing WrapperArray, it become simple and small maintenance cost.
What does the proposed API look like?
findAll() returns Array of Wrapper and WrapperArray is removed.

I am very sorry.
I lost interest this issue.
Would you please ignore my opinion in this issue.

I've just run into this. WrapperArray doesn't implement the iterator interface so we can't use for..of in cases where we need to manually assert each element in the array. What is the reasoning for not implementing the iterator interface?

I'm also missing the map method

You can access the wrappers on the wrappers property:
const { wrappers } = wrapper.findAll('div')
const textArr = wrappers.map(w => w.text())

hasClass of WrapperArray might be a bit confusing, because it checks if every child wrapper has class:
https://github.com/vuejs/vue-test-utils/blob/1ee03ac2d57eccd913027ecd51ac5777eeac5c84/packages/test-utils/src/wrapper-array.js#L99
Solution like:
return this.wrappers.some(wrapper => wrapper.hasClass(className))

seems to be more natural, but not perfect.

@eddyerburgh What about customization of findAll method, I mean something like:
findAll (selector, opts  = {}) {
   if (opts.list) {
     return arrayWrapper.wrappers;
   } else {
     return arrayWrapper;
   }
}


Update_SignType: test to_SignType

Issue: https://github.com/dotnet/dnceng/issues/2271
Validated in: https://dev.azure.com/dnceng/internal/_build/results?buildId=2411073&view=results
The build still has an error/warning because this was just merged: https://github.com/dotnet/dnceng/pull/2274

I have a list of the possible impacting repos and I will monitor them and if this is breaking I will revert this.


About Data Leakage in FFT_for_Period

Thank you for your work.

def FFT_for_Period(x, k=2):
    # [B, T, C]
    xf = torch.fft.rfft(x, dim=1)

    # find period by amplitudes
    frequency_list = abs(xf).mean(0).mean(-1)
    frequency_list[0] = 0
    _, top_list = torch.topk(frequency_list, k)
    top_list = top_list.detach().cpu().numpy()
    period = x.shape[1] // top_list
    return period, abs(xf).mean(-1)[:, top_list]

In this function, for every batch, the input x includes the data to be predicted.
This means that when calculating the period, the information of the testing data are utilized, which leads to data leakage.
Did I made a mistake?

Hi, x does not include the "data to be predicted". Please see: https://github.com/thuml/Time-Series-Library/blob/main/models/TimesNet.py#L113-L114 the future part is generated by past information at beginning


Update microsoft-graph-migration.md

Update microsoft-graph-migration.md

Docs Build status updates of commit 7150e67:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fa6a1b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Example proposal: For Solder ball modeling

📝 Description of the example
A script to create solderball model and also target solder for performing morphing
📁 Files needed for running the example
PyGeometry.zip
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
2024r2
🐍 Which Python version are you using?
3.10
📦 Installed packages
ansys=geometry-core==0.6.6

Hi @agvarghe - what's the status here? Do you need any help?

Hi @RobPasMue
I had to lower the priorty in the last month, i plan to complete this in this week

Awesome, thanks @agvarghe!


Geometry alignment

Removed common code from system types and fixed alignment bug closes #1131
Added unit tests on rotation matrix formation.
Separated the align geometry to CIL and TIGRE versions.

warning from running tests on jenkins.
/home/jenkins/conda-bld/cil_1643802785731/test_tmp/Wrappers/Python/test/test_AcquisitionGeometry.py:399: PendingDeprecationWarning: the matrix subclass is not the recommended way to represent matrices or deal with linear algebra (see https://docs.scipy.org/doc/numpy/user/numpy-for-matlab-users.html). Please adjust your code to use regular ndarray.

Need to fix this before merging.


Implement caching of carbon intensity forecast

Currently a new request to carbonintensity.org.uk is made each time cats is run. In cats/__init__.py:
def findtime(postcode, duration):
  tuples = get_tuple(postcode) # API request
  result = writecsv(tuples, duration) # write intensity data to disk
				      # as csv timeseries
  # ...

Although the carbon intensity data obtained from the API is written on disk, this not taken advantage of. Instead, if the relevant carbon intensity data is already on disk, we'd like to reuse this data instead of making a new request each time.
The local carbon intensity forecast data is reusable if the last forecast datetime is beyond the expected finish datetime of the application, i.e. forecast_end > now() + runtime.
A possible approach is to reshuffle the responsabilites of both top-level functions api_query.get_tuple and parsedata.writecsv.

First, get_tuple could be responsible for ensuring that the right data is present on disk, and download it if not.
Then writecsv only cares about computing the best job start time, assuming correct intensity data is available. For instance,

# cats/__init__.py
def findtime(postcode, duration):
  tuples = get_tuple(postcode)
  result = writecsv(tuples, duration)

then becomes
# cats/__init__.py
def findtime(postcode, duration):
  # Check if cached carbon intensity data goes beyond
  # now() + duration, download new forecast if not
  # formerly `get_tuple()`
  ensure_cached_intensity_data(postcode, duration)
  # Then -- assuming data is available on disk -- compute
  # the best time to start the job.
  # formerly `writecsv()`
  result = get_best_start_time(duration)

This approach has the benefit is maitaining a good separation between talking to the API – and caching intensity data – and the calculation of the start time. We currently do almost have this, expect that the function returning the start time is also responsible for writing the intensity data on disk.
Another possible approach is to push the API query and data caching down to the current writecsv function:
def writecsv(data_path: str, duration=None) -> dict[str, int]:
    try:
	return cat_converter(data_path, method, duration)
    except MissingItensityDataError:
	cache_latest_intensity_forecast(postcode)
	return cat_converter(data_path, method, duration)

If we want, we could cache at the HTTP layer where we call the API. This seems quite easy (see #30).

Yes - I actually like your approach a lot better. It maybe doesn't allow for as much data reuse but it's much simpler. And I guess it covers most of the caching use case (i.e. several request in the same first or second half of the hour).

Merged the requests caching stuff. I think this is resolved.


Fix: Cannot process argument transformation on parameter 'Root'. Cannot convert value to type System.String.

Addresses Issue #71 "Cannot process argument transformation on parameter 'Root'. Cannot convert value to type System.String."
This bug will manifest itself when the module to be tested has a "ScriptsToProcess" key defined in the module manifest file.  The issue occurs due to Import-Module returning an array containing two or more PSModuleInfo objects.  One containing the actual module as well as one or more describing the script(s) defined in the ScriptsToProcess key in the manifest file.
The code producing the error contained in DscResource.Test\0.13.0\DscResource.Test.psm1:
$ModuleUnderTest = Import-Module -Name $ModulePsd1 -ErrorAction Stop -PassThru

as well the following in DscResource.Test\0.13.0\Tests\QA\FileFormatting.common.Tests.ps1:
$textFiles = @(Get-TextFilesList -Root $ModuleBase | WhereModuleFileNotExcluded)

Error:

I'm will be submitting a PR that will filter the actual module based on the manifest's defined GUID for the module under test.

Fixes #71

This change is

@bcwilhite Thanks for this! 😃


additionalHeaderFileName is needed

Feature Request
additionalHeaderFileName in swift was removed in the last version (commit 9302f55f829556ccebccd1b0f56ff0c6e7f69e04), I'd like to see it back.
Expected Behavior
additionalHeaderFileName used to work.
Actual Behavior
additionalHeaderFileName was removed.
Additional Information
If you have a transformable property with a attributeValueClassName of for example CLLocation, the compiler will throw an error if you don't import CoreLocation.
Ideally, in swift, it'd be even better if we could specify multiple additionalHeaderFileName (or a comma separated list), as we might have to use multiple imports for different types.

I agree that additionalHeaderFileName should support multiple files somehow (for both Obj-C and Swift).

Maybe rename it to additionalImports? Or use the mappedKeyName syntax, like additionalImport.1, additionalImport.2, etc...

additionalImports with comma-separated file names sounds great.

That sounds good to me too.

Why was it removed? Is there an alternative way to use enums in Swift? Using-enums-as-types seems not to work anymore.

I know this is an old issue, but it's one I'd like to fix, so if anyone's paying attention...
I'm thinking that additionalHeaderFileName is misnamed in Swift because you'd normally want to import a module there, not a header file. Of course you might want to do something similar in Objective-C, adding a custom @import Foo rather than #import "Foo.h". So, I'm inclined to add the additionalImportsas described here, as a comma-separated list that generates appropriate imports for the target language. I think it makes sense to leaveadditionalHeaderFileName` as-is, not part of Swift templates at all-- not just for compatibility, but because it supports an Objective-C specific syntax that doesn't really map to Swift.
As an experiment for anyone who messes with mogenerator source code-- add this line back in to the Swift template and then run the test suite. Testing fails for Swift because you end up with import Gender.h in a Swift source file, which nicely illustrates the problem with just adding additionalHeaderFileName back to Swift.
If there are no other comments (and this far after filing, I'm not expecting any, but I'd love to have some feedback) this is what I'm going to do.

Yup, additionalHeaderFileName predates Obj-C support for modules.
additionalImports seems like a fine idea.

Added additionalImports in aaea23b


Non-technical users will have a hard time stopping Raiden

The Raiden instance launched from the wizard is hard to stop for non technically / terminal savvy users.

We could add a "stop raiden" RPC endpoint add a way to call it to the WebUI.

We would need to think about the security implications of adding a stop endpoint.
But even just via normal OS process tools should be realatively easy.

We would need to think about the security implications of adding a stop endpoint.

We have endpoints to send tokens around. I don't think the stop end point is any more security critical.

True, but Jacob and I were thinking about the publicly hosted test / workshop nodes for example. It's no big problem if their channels get drained but shutting them down continually makes them unusable with very little effort on an attackers part.

should be solved inhttps://github.com/raiden-network/raiden/issues/6113

create a follow up issue for the web UI button

https://github.com/raiden-network/raiden/issues/6113 is done now, we can inlcude a shut down button into the WebUI.
Can you implement that @manuelwedler ? @christianbrb ?

Yes sure. I created an issue in the webui repo: https://github.com/raiden-network/webui/issues/389
I will align with @sashseurat on it.

Closing this issue, since we have an issue in the WebUI now


Campaign Language Support

These need to be spec'd out into separate issues in various repos, by @DeeZone but summary:
mb-users Mongo Database Update for POSTs (saving language)
Adding support for global users to receive the weekly digest email message. In order to support this functionality the user and campaign language setting will need to be supported.

Support user_language and user_country in POST to /user in mb-user-api.

Schema update to mb-users Mongo collection.

Support campaign_language in POST to /user in mb-user-apifor user campaign activity.

Schema update to mb-users Mongo collection.

Support user_language and user_country in mbc-userAPI-userRegister consumer to process user_register transactions for POSTing to mb-user-api /user.

Support campaign_language in mbc-userAPI-campaignActivity consumer to process campaign_signup transactions for POSTing to mb-user-api /user.

Duplicate of #5601


Initial movements of players

Finished work on movement of spaceship and refactors.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

133 of 133 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
162

Relevant Lines:
162

💛  - Coveralls


configureLogFilters appears to be modifying more than log entries

I'm still trying to get to the bottom of it, but it appears calls to logEx.configureLogFilters is modifying more than log entries. I discovered it was, for reasons I don't understand yet, also modifying the strings for my http requests in other, unrelated parts of the application. (Subsequently, the modified strings were being rejected by the REST service my application uses.)
If I comment out the calls to logEx.configureLogFilters, the REST service works because the http requests remain intact.
gist: https://gist.github.com/eppineda/8d37faf5e40cb3d89e6a947ecd43abb6

(btw, thank you for your fine work on this! I hope to contribute...)

@eppineda made a patch dev-release  0.0.12-1+sha.f3cb067. Test and let me know if your issue has been resovled.

all better now! thanks @ferronrsmith

you're welcome
On 09/22/2016 03:08 PM, Enrique Pineda wrote:

all better now! thanks @ferronrsmith https://github.com/ferronrsmith
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lwhiteley/AngularLogExtender/issues/46#issuecomment-248999167,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJwFMSmegBhvvDUFQn29l4yN8tqT5Fhks5qstJEgaJpZM4KCM67.


Remove once dependency

Removed once dependency, which makes this library a tiny bit smaller:
C:\Users\Michael\Github\xhr>browserify . | uglifyjs | wc -c
   7584 # before

C:\Users\Michael\Github\xhr>browserify . | uglifyjs | wc -c
   7240 # after

I was just going to write about how ensuring callback is only ever called once is important, when I noticed what you did :)
I'll merge this once I go through other PRs and figure out the order.

Bump :-x :-)

yeah, my bad. Reality came and changed schedules :)
Thanks for the reminder.

+1
i was never a fan of the once dep

ooh yeah, rad - shave the bytes!

haha @yoshuawuyts ⚡ this one is not even about bytes, in fact more than once, saw bugs in once!
for real tho,
the PR/ search on xhr doesn't lie: https://github.com/Raynos/xhr/search?q=once&type=Issues&utf8=✓

@ArtskydJ @yoshuawuyts @reqshark
Compare this PR with #127 and https://github.com/Raynos/xhr/commit/67958b7ab4ee01bd7d4e72fca8a2ea119bc2c02d
Who would have guessed people throw errors in an async callback to xhr?

Whoops, I didn't think that case through. Thanks for fixing!

No worries, I don't think anyone took that into account.
Always learning ;)


fix: make .lean hashes cross-platform/autocrlf compatible

This is the Lake equivalent of https://github.com/leanprover-community/mathlib4/pull/3096, needed for mathlib4's caching implementation

@Kha I imagine the reason you did not just change computeFileHash directly is because that is doing binary hashing and this optimization should only apply to text files? In that case, maybe we should just call the wrapper TextFilePath? Does that seem reasonable / am I understanding correctly?

Yes, exactly. TextFilePath is fine as well, sure.

I haven't touched other functions such as inputFile since this is issue quite specific to mathlib caching

Looks good to me. Merged.

Oops, committing flake.lock was not intended. Let's continue the talks about merging lake into lean4 soon so I don't have to worry about that anymore :smile: .

@Kha Fixed. 😄


AVX2 instruction support for MacOS systems

Description
MacOS systems reports that they don't have AVX2 instruction available.
I understand that those systems are executed via VMware and many things going on, but from MacOS and Apple HW perspective there is no currently supported apple x86 hardware without AVX2 support.  All AVX systems lost both HW/SW for years ago.
To be consistent with Apple ecosystem it would be nice to have AVX2 support to be enable for mac VMs
CPU features:
=== CPU FEATURES ===
VendorId                : GenuineIntel
num_virtual_cores       : 3
num_physical_cores      : 3
num_threads_per_core    : 1
num_cpus                : 1
cache_line_size         : 64
cache_L1_size           : 32768
cache_L2_size           : 262144
cache_L3_size           : 12582912
OS_x64                  : True
OS_AVX                  : True
OS_AVX512               : False
MMX                     : True
x64                     : True
ABM                     : False
RDRAND                  : True
BMI1                    : False
BMI2                    : False
ADX                     : False
PREFETCHWT1             : False
MPX                     : False
SSE                     : True
SSE2                    : True
SSE3                    : True
SSSE3                   : True
SSE4.1                  : True
SSE4.2                  : True
SSE4.a                  : False
AES                     : True
SHA                     : False
AVX                     : True
XOP                     : False
FMA3                    : False
FMA4                    : False
AVX2                    : False
AVX512f                 : False
AVX512pf                : False
AVX512er                : False
AVX512cd                : False
AVX512vl                : False
AVX512bw                : False
AVX512dq                : False
AVX512ifma              : False
AVX512vbmi              : Fals
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[X] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
System Software Overview:

  System Version: macOS 12.6.2 (21G320)
  Kernel Version: Darwin 21.6.0
  Boot Volume: Macintosh HD
  Boot Mode: Normal
  Computer Name: Mac-1672177612509.local
  User Name: runner (runner)
  Secure Virtual Memory: Enabled
  System Integrity Protection: Enabled
  Time since boot: 21 minutes

Hardware Overview:

  Model Name: Mac
  Model Identifier: VMware7,1
  Processor Name: Unknown
  Processor Speed: 3.33 GHz
  Number of Processors: 1
  Total Number of Cores: 3
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 14 GB
  System Firmware Version: VMW71.00V.13989454.B64.1906190538
  OS Loader Version: 540.120.3~22
  Apple ROM Info: [MS_VM_CERT/SHA1/27d66596a61c48dd3dc7216fd715126e33f59ae7]Welcome to the Virtual Machine
  SMC Version (system): 2.8f0
  Serial Number (system): VM8Mb/79Cglq
  Hardware UUID: 4203018E-580F-C1B5-9525-B745CECA79EB
  Provisioning UDID: 4203018E-580F-C1B5-9525-B745CECA79EB

Is it regression?
don't know
Expected behavior
AVX2 support avaialbe within VM
Actual behavior
AVX2 support is not avaialbe within VM
Repro steps
CPU info
echo "CPU features:"
if [ -x "$(command -v lscpu)" ]; then
lscpu
elif [ "$(pip list | grep cpufeature)" != "" ]; then
python -c "import cpufeature; cpufeature.print_features()"
else
echo "Unable to get CPU features via lscpu or python/cpufeature"
fi
echo
if [ -x "$(command -v system_profiler)" ]; then
echo "MacOS detailed features:"
system_profiler SPSoftwareDataType SPHardwareDataType
fi

Hi @napetrov!
It is the same as https://github.com/actions/runner-images/issues/5609 but for macOS. I forced to close that issue because nothing has changed from previous one. We will inform community when it happen.


[FEATURE] Batch Download

So this is a suggestion directly related to how I would use this tool.
From the looks of the code, it seems like you would download chat from one stream at a time.
Is there a way to implement batch chat logging with the code as it is? It doesn't have to do them
simultaneously, but just copy-pasting a list of links and having it download all the chats to a folder,
naming the file what the stream was named, and once it finishes with one link, it moves on to the next
until the end of the list.
I'm looking at using this tool for archiving purposes, so that's why I was interested in a "batch" feature.
If I could already do this, please let me know how (I'm an amateur with Python...) If it can't be implemented
in the current iteration of this program, it is a feature I'd like to see added if possible.
Thanks!

Sure, this can be done 👍
Does the following implementation solve your needs?
from chat_downloader import ChatDownloader
from chat_downloader.sites.youtube import YouTubeChatDownloader

# Change channel_id here
channel_id = 'UCWxlUwW9BgGISaakjGM37aw'

# Create a downloader
downloader = ChatDownloader()

# Forcefully create a YouTube session
session = downloader.create_session(YouTubeChatDownloader)

# Use the YouTube session to get the channel's videos
videos = session.get_user_videos(channel_id)

# Loop over all the user's videos
for video in videos:
    # Get the chat for the video and save to output file
    chat = downloader.get_chat(video['video_id'], output='output/{title}.txt')
    for message in chat: # Iterate over generator
        pass # Do something with message here, if you wanted to

The comments should help explain what is happening, but basically get the user's videos and save the chat to a file in a directory called output and whose name is the title of the video.
You would save it to a file (e.g., batch.py) then run python batch.py (provided you have already installed Chat Downloader using pip install chat-downloader.
Let me know if you have any other questions 👍

One small question, let's say I'm signed in on my youtube account via firefox. And I have a memebership to channel A. If I use this plugin on the same computer with my browser open, would it be able to get member-only stream's chat logs too?

Yes the tool is able to fetch member-only stream chat logs, you just have to pass in a valid cookies.txt file. You can follow this tutorial to export a cookies.txt file: https://github.com/ytdl-org/youtube-dl#how-do-i-pass-cookies-to-youtube-dl and then use the --cookies <file> argument when running from the command line, or
downloader = ChatDownloader(cookies='path/to/cookies.txt')

when using the above Python script

So, I added the above script in the same area as the other "from" statements in chat downloader.py, and I did everything with cookies.txt that you said to. I left the path in single quotes, but then when I try to run it, I get the error shown in the picture... I tried adding a space, or not using single quotes around the path, but it always ends up the same... Any idea what I've done wrong? I'm on Windows 10 if that helps.

Nevermind! So, I had to go through just about every script in the package and remove the . or .. in the from statement
(from .sites.youtube import blabla  -> from sites.youtube import blabla)    [don't ask me why, it just works...]
Then I found a couple of modules I was missing (requests and websocket)
After doing all of that one by one, it finally worked. It's running at the moment, I'll see if I get membership stuff and everything, but that's how it works. Also, I had initially put the text above within chatdownloader.py, but realized while debugging that this needed to be its own script. I just named it batch.py and put it in the same folder as chatdownloader.py. This means that in the above script, 'from chat_downloader.sites.youtube import YouTubeChatDownloader' becomes just 'from sites.youtube import YouTubeChatDownloader'
So I'll probably make one more post once it's done working, and I'll see what all it got and what it might've missed. Thanks for all the help though!

Ran into one bug/issue - if the video doesn't have a chat, it stops the process with an error message. How would I implement a "if no chat, skip to next video" command?
File "C:\Users\patri\Desktop\chat-downloader-master\chat_downloader\sites\youtube.py", line 1525, in _get_initial_video_info
raise NoChatReplay(error_message)
errors.NoChatReplay: Video does not have a chat replay.

This is done using try... except in Python (see https://docs.python.org/3/tutorial/errors.html#handling-exceptions)
and the error you will be catching is NoChatReplay.

Okay, I was able to use that to have it iterate beyond the video that threw an error. It's still processing them, so I'll let you know how it ends up with that now. Thanks again for all the help!

So far it's going well. I noticed that it's significantly slower if I run it in Idle rather than just running the py script. But then I already had several done, so I made a small addition where I can use KeyboardInterrupt to make it skip to the next video. That way, if I ever stop and restart the process, I can skip anything that's already been done.
Having said that, in Idle it did one video per 30 minutes or so, now it's doing one video every 10 minutes. Considering I'm doing an entire channel, that is a very significant amount of time gained.
If/when it finishes successfully, do you want me to upload my versions of the files to another branch? Especially considering I had to change the import syntax in each file, it probably wouldn't work to just add my batch.py file to what's currently there. But that's up to you.

Oof, 92 videos in, Windows Update struck. I don't feel like using KeyboardInterrupt that many times, so I added a loop to skip the number of videos that are complete. I'm really learning a lot about Python by having to use this plugin lol.

Windows update always chooses the worst times! Well, it's good to know you are learning more Python haha. If you need any help, feel free to drop a comment.

Everything completed perfectly, thanks!