SecurityAudit phase in the build is throwing InvalidAccessException: Account xxxx is not an administrator for this organization

Describe the bug
When I run the Accelerator pipeline, it fails under SecurityAudit with the following :

Stack AWSAccelerator-SecurityAuditStack-xxxxx-eu-west-3
logical Id failure SecurityHubMembersxxxxx
Status reason: Received response status [FAILED] from custom resource. Message returned: InvalidAccessException: Account xxxx is not an administrator for this organization at Object.extractError (/var/runtime/node_modules/aws-sdk/lib/protocol/json.js:61:27) at Request.extractError (/var/runtime/node_modules/aws-sdk/lib/protocol/rest_json.js:61:8) at Request.callListeners (/var/runtime/node_modules/aws-sdk/lib/sequential_executor.js:106:20) at Request.emit (/var/runtime/node_modules/aws-sdk/lib/sequential_executor.js:78:10) at Request.emit (/var/runtime/node_modules/aws-sdk/lib/request.js:686:14) at Request.transition (/var/runtime/node_modules/aws-sdk/lib/request.js:22:10) at AcceptorStateMachine.runTo (/var/runtime/node_modules/aws-sdk/lib/state_machine.js:14:12) at /var/runtime/node_modules/aws-sdk/lib/state_machine.js:26:10 at Request. (/var/runtime/node_modules/aws-sdk/lib/request.js:38:9) at Request. (/var/runtime/node_modules/aws-sdk/lib/request.js:688:12) (RequestId: d03045f4-4e51-42c5-95fe-32657237486e)

security-config.xml excerpt
homeRegion: &HOME_REGION eu-west-1
centralSecurityServices:
  ##################################################################################################################
  # Based upon AWS Security Reference Architecture (AWS SRA),                                                      #
  # Assigning delegated administrator to security tooling (Audit) account                                          #
  # https://docs.aws.amazon.com/prescriptive-guidance/latest/security-reference-architecture/security-tooling.html #
  ##################################################################################################################
  delegatedAdminAccount: Audit
  ebsDefaultVolumeEncryption:
    enable: true
    excludeRegions: []
  s3PublicAccessBlock:
    enable: true
    excludeAccounts: []
  scpRevertChangesConfig:
    enable: true
    snsTopicName: Security
  macie:
    enable: true
    excludeRegions: []
    policyFindingsPublishingFrequency: FIFTEEN_MINUTES
    publishSensitiveDataFindings: true
  guardduty:
    enable: true
    excludeAccounts:
      -<PHONE_NUMBER>39
    excludeRegions: []
    s3Protection:
      enable: true
      excludeRegions: []
    lambdaProtection:
      enable: true
      excludeRegions: []
    exportConfiguration:
      enable: true
      overrideExisting: true
      destinationType: S3
      exportFrequency: FIFTEEN_MINUTES
  auditManager:
    enable: true
    excludeRegions:
      - eu-west-3
    defaultReportsConfiguration:
      enable: true
      destinationType: S3
    lifecycleRules:
      - enabled: true
        abortIncompleteMultipartUpload: 7
        expiration: 1000
        noncurrentVersionExpiration: 1000

To Reproduce
I tried to run it few times and it keeps failing
Expected behavior
It should pass
Please complete the following information about the solution:

[ x] Version: [e.g. v1.6.1]

To get the version of the solution, you can look at the description of the created AWS CloudFormation stack used to install the LZA (AWSAccelerator-InstallerStack). For example, "(SO0199) Landing Zone Accelerator on AWS. Version 1.5.1.". If the description does not contain the version information, you can look at the Parameters of the stack for the RepositoryBranchName as that should contain the version number.

[x] Region: eu-west-3
[ ] Was the solution modified from the version published on this repository?
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the services this solution uses?
[ ] Were there any errors in the CloudWatch Logs?

Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).
Additional context
Add any other context about the problem here.

Hello Zaid! Thank you for using the Landing Zone Accelerator on AWS solution. We've seen this issue before when the delegated admin account had already been set for the AWS Organization. In your Management account, for SecurityHub, does it show the delegated admin account being set for the Audit account?

Spot on @bo1984 ! The Delegated Admin account for the Security Hub (in the management account, for the same region) was not set! I did set it manually to the Audit Account and everything worked perfectly afterwards, thank you! I am closing the issue


Update help window

Update help window to include missing commands:

help
list
schedule edit
schedule delete
clear
remark

Changed INDEX to SCHEDULE_INDEX for schedule edit and delete
Edit: Added schedule clear

@LeeEnEn to review the following?:

add schedule clear command
change NEW VALUE to NEW_VALUE
List all candidates... to Lists all candidates...
add ... behind [k/MORE_KEYWORDS] (so it looks like [k/MORE_KEYWORDS]...)

Thanks for pointing out the mistakes!
Will add them in the next commit 👍

@LeeEnEn en en im thinking dont expand the schedule button as such in the photo you uploaded. thoughts? i feel like it could just be centered (shown below):


Accept keyhashes for required signers

Required signers can now be specified via the --required-signer-hash cli option
Resolves: #3338

QA tested this with the following plutus script with both normal and extended keys, and with key files and key hashes. Works as expected :+1:
mkPolicy :: PubKeyHash -> ScriptContext -> Bool
mkPolicy pkh ctx = traceIfFalse "not signed by redeemer pubkeyhash" checkWitness
  where
    info :: TxInfo
    info  = scriptContextTxInfo ctx

    checkWitness :: Bool
    checkWitness  = txSignedBy info pkh

bors r+

bors merge


Trying to get property 'code' of non-object

Hello, i have a problem here.
I just add new Indonesian Rupiah currency and update default currency in my channel, but i got this problem.

Please help me :(

OMG, finally i realized that the currency option in channel : Channel->Currency should be all selected, @vivek-webkul i think you should change your User Interface for that, it seem old style.

can you give me some pictures example to solve this thing ?

@L200180024
If you are using latest version 1.3.1 then you will not face this error anymore as it has already resolved here https://github.com/bagisto/bagisto/pull/4014


nested subcommands declared programatically not listed in usage

When declaring subcommands programatically, the subcommands are not listed when printing the usage. This goes for subcommands of subcommands (nested). When I declare them using annotations, everything works as expected.
I have some requirements where I need to pass some specific objects to each command (via parameterized constructor), therefore I cannot use the annotations.
I feel like I am missing the approach here...as I am trying to avoid global objects. I could probably come up with a DI (using Dagger2) to inject the correct instance of the objects I need, but its getting pretty convoluted.
The usage works fine if I call it directly, but if pass in, say a bad parameter using the following
List<Object> results = commandLine.parseWithHandlers(
       new CommandLine.RunAll().useOut(System.out).useAnsi(CommandLine.Help.Ansi.ON),
       CommandLine.defaultExceptionHandler(), new String[]{"project"}
);

The automatic usage will only print subcommands of subcommand that are declared via annotation...

Away from pc now, but I thought there was a unit test for that. The intention is that the usage help prints all subcommands, regardless of whether they were added programmatically or declaratively.
Can you provide an example that demonstrates the issue?

By the way, the default exception handler will print the usage help for the subcommand where parsing failed. Not for the top-level command.

Awesome lib BTW...well thought out and designed...
Simple example...my simple commands take a PrintStream as a constructor param...
 CommandLine commandLine = new CommandLine(new CLICommand(System.out))
                .addSubcommand("project", new CommandLine(new ProjectCommand(System.out)).addSubcommand("list", new ProjectListCommand(System.out)) )
                .addSubcommand("test", new TestCommand());

and then call...
List<Object> results = commandLine.parseWithHandlers(new CommandLine.RunAll().useOut(System.out).useAnsi(CommandLine.Help.Ansi.ON),CommandLine.defaultExceptionHandler(), new String[]{"project"} );

so in the above, I am essentially just faking someone running the command with the parameter of project which is a subcommand, and project has a subcommand called list
So since no other arguments are passed, it does err, which is expected and prints the usage...
Usage: project [-h]
project command
  -h, --help   Help

above, it doesn't print the subcommands (list). If I provide via annotation (can have a custom constructor i don't think) it works as expected.

And for the record, I am implementing a console / socket solution, hence the reason to pass the in the PrintStream, as well, I will have other dependencies in the commands, as well as a custom error handler but for simplicity I've just used the default.

Looks like I need a bit more information to be able to reproduce the issue.
I don't understand this sentence in your comment:

So since no other arguments are passed, it does err, which is expected and prints the usage...

Why does the project subcommand have an error if no other arguments are passed? Does it have any required options or positional parameters? (I did not see any "missing required option..." error message...)
Here is my test code:
public class Issue427 {

    @Command
    static class CLICommand implements Runnable {
        public CLICommand(OutputStream out) {}
        public void run() {
            System.out.println("running cli");
        }
    }

    @Command
    static class ProjectCommand implements Runnable {
        public ProjectCommand(OutputStream out) {}
        public void run() {
            System.out.println("running project");
        }
    }

    @Command
    static class ProjectListCommand implements Runnable {
        public ProjectListCommand(OutputStream out) {}
        public void run() {
            System.out.println("running list");
        }
    }

    @Command
    static class TestCommand implements Runnable {
        public void run() {
            System.out.println("running test");
        }
    }

    public static void main(String[] args) {
        CommandLine commandLine = new CommandLine(new CLICommand(System.out))
                .addSubcommand("project", new CommandLine(new ProjectCommand(System.out))
                        .addSubcommand("list", new ProjectListCommand(System.out)))
                .addSubcommand("test", new TestCommand());

        List<Object> results = commandLine.parseWithHandlers(
                new CommandLine.RunAll()
                        .useOut(System.out)
                        .useAnsi(CommandLine.Help.Ansi.ON),
                CommandLine.defaultExceptionHandler(),
                new String[]{"project"} );
    }
}

When I execute this, it simply executes the run method of the top-level command and the project subcommand:
running cli
running project

Away from the code, I'll look and post more in the AM.
Sorry, err is the wrong word, but because no other commands are referenced,
ie, list ; it prints out the usage command for the subcommand. I don't
know, but I assume this is the desired behaviour? It is the behaviour I am
seeking.
What i am trying to achieve is to a have commands that have subcommands,
with no default action, but a requirement for the caller to pass in the
subcommmand. In other words, if a command (parent) is called with no
arguments, it should be considered invaild and display the usage for that
sub command. I am pretty sure this is happening for me by default, but your
example seems to contradict that. I am pretty sure that what I just
explained happens when I do your example but using annotations to define
the subcommands but I can't be positive at the moment.
Also, if the parent command is CLI, as an example, and the subcommand is
run, I wouldn't want the parent command to do anything. Is this acheived by
swapping RunAll() for RunLast() ?
My commands all have 1 option, defined for help, not required...if that
helps. Not sure how helpful this is at the moment, but I'll edit this with
a more concrete example. Thanks!
On Thu, Aug 2, 2018, 5:56 PM Remko Popma<EMAIL_ADDRESS>wrote:

Looks like I need a bit more information to be able to reproduce the issue.
I don't understand this sentence in your comment:
So since no other arguments are passed, it does err, which is expected and
prints the usage...
Why does the project subcommand have an error if no other arguments are
passed? Does it have any required options or positional parameters? (I did
not see any "missing required option..." error message...)
Here is my test code:
public class Issue427 {
@Command
static class CLICommand implements Runnable {
    public CLICommand(OutputStream out) {}
    public void run() {
        System.out.println("running cli");
    }
}

@Command
static class ProjectCommand implements Runnable {
    public ProjectCommand(OutputStream out) {}
    public void run() {
        System.out.println("running project");
    }
}

@Command
static class ProjectListCommand implements Runnable {
    public ProjectListCommand(OutputStream out) {}
    public void run() {
        System.out.println("running list");
    }
}

@Command
static class TestCommand implements Runnable {
    public void run() {
        System.out.println("running test");
    }
}

public static void main(String[] args) {
    CommandLine commandLine = new CommandLine(new CLICommand(System.out))
            .addSubcommand("project", new CommandLine(new ProjectCommand(System.out))
                    .addSubcommand("list", new ProjectListCommand(System.out)))
            .addSubcommand("test", new TestCommand());

    List<Object> results = commandLine.parseWithHandlers(
            new CommandLine.RunAll()
                    .useOut(System.out)
                    .useAnsi(CommandLine.Help.Ansi.ON),
            CommandLine.defaultExceptionHandler(),
            new String[]{"project"} );
}

}
When I execute this, it simply executes the run method of the top-level
command and the project subcommand:
running cli
running project
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/remkop/picocli/issues/427#issuecomment-410103111, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMqi-ea463b-DFayu4K1mYQLG8BVbNOvks5uM5HEgaJpZM4VtAe2
.

Interesting. So you want to make it invalid for a command to be invoked without a subcommand.
This can be accomplished by throwing a ParameterException from the run method. The default exception handler will catch ParameterException and treat it as invalid input.
Here is my updated example:
public class Issue427 {

    @Command(name = "cli", mixinStandardHelpOptions = true)
    static class CLICommand implements Runnable {
        public CLICommand(OutputStream out) {}
        public void run() {
            System.out.println("running cli");
        }
    }

    @Command(name = "project", mixinStandardHelpOptions = true, description = "manage projects")
    static class ProjectCommand implements Runnable {
        @Spec
        CommandLine.Model.CommandSpec spec; // injected by picocli

        public ProjectCommand(OutputStream out) {}
        public void run() {
            throw new CommandLine.ParameterException(spec.commandLine(),
                    "This command requires a subcommand");
        }
    }

    @Command(name = "list", mixinStandardHelpOptions = true,
            description = "shows a list of projects")
    static class ProjectListCommand implements Runnable {
        public ProjectListCommand(OutputStream out) {}
        public void run() {
            System.out.println("running list");
        }
    }

    @Command(name = "test", mixinStandardHelpOptions = true)
    static class TestCommand implements Runnable {
        public void run() {
            System.out.println("running test");
        }
    }

    public static void main(String[] args) {
        CommandLine commandLine = new CommandLine(new CLICommand(System.out))
                .addSubcommand("project", new CommandLine(new ProjectCommand(System.out))
                        .addSubcommand("list", new ProjectListCommand(System.out)))
                .addSubcommand("test", new TestCommand());

        List<Object> results = commandLine.parseWithHandlers(
                new CommandLine.RunAll()
                        .useOut(System.out)
                        .useAnsi(CommandLine.Help.Ansi.ON),
                CommandLine.defaultExceptionHandler(),
                new String[]{"project"} );
    }
}

When your run it, it gives the following output:
This command requires a subcommand
Usage: cli project [-hV] [COMMAND]
manage projects
  -h, --help      Show this help message and exit.
  -V, --version   Print version information and exit.
Commands:
  list  shows a list of projects

I believes this does what you have in mind.

Also, if the parent command is CLI, as an example, and the subcommand is
run, I wouldn't want the parent command to do anything. Is this acheived by
swapping RunAll() for RunLast() ?

Yes, that is correct. If you use RunLast, it will execute the most specific subcommand. This is the default behaviour and this is probably what you want.
As a side note, from the end-user point of view, it may be user-friendly if every command does something useful even without subcommand (if possible). For example, git branch --list shows a list of branches, but when invoked without any options, git branch will also show a list of branches.

Ok, so I copied your code and I noticed that the @spec annotation was not available, so silly me, I noticed I was on version 3.0.0 (my bad!).  And now....everything is working as expected. I think we can close this issue, but seeing your code leads me to think I need to really parse the documentation for more ways to customize.
Seeing that you can inject the @spec looks promising and convenient, so I have to ask if there are ways of getting other info.  So in my particular case, I am injecting the a PrintStream, so my commands can print out to the correct socket output stream when running. I can set the proper stream when using parseWithHandlers() for the automatic stuff, but when I need more custom control I need to either pass it in, OR return a Ansi.Text and have iterate through the results and print them to the socket.
One particular thing I struggled with, was in my parent command (CLI) I wanted to override the help options (display string), so I augmented the string builder code in the docs...but...is there a way to have a command automatically print MY help string vs me having to check when help should be displayed and return my own string? Should I be implementing a HelpCommand? Just wondering if there is a canonical or endorsed way of doing this.
Anyways, thanks for the support. Super awesome library! Looking forward to making it a solution for us.

Have you specified usageHelp = true on the custom help option?
See https://picocli.info/#_printing_help_automatically

You may also be interested in the @ParentCommand annotation to allow subcommands to access global options defined on the parent command.
Also check the “new and noteworthy” section of recent release notes: https://github.com/remkop/picocli/releases


chore(main): release iso-web 1.0.4

:robot: I have created a release beep boop
1.0.4 (2024-02-05)
Bug Fixes

update doh to new iso-kv ttl in seconds (6023226)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugomrdias/iso-repo/releases/tag/iso-web-v1.0.4 :sunflower:


[tx model] Store hash for asset and metadata

Highlights of the Proposal
Instead of storing the entire asset payload (for CREATE transactions) this issues proposes to store the hash of the asset data.
Similarly, for the metadata field, this issue proposes to store the hash of the metadata.
Pros & Cons
Pros

Simplifies the transaction model.
Makes it easier to specify a strict schema. No need to assume arbitrary fields and nesting under the asset and/or metadata fields. Instead it would be possible to require the asset and metadata fields to be (hex encoded) strings.
Could simplify binary serialization (for some formats at least). Related to #1932. As an example, specifying an ASN.1 data structure for the current tx model seems non-obvious as far as the asset and metadata fields are concerned.
Related to the above two points: could simplify validation of the schema.
Decouples the asset data and metadata from the transaction. Users are free to store the data wherever they wish and protect access to the data however they wish.
Could potentially allow interoperability with content addressing based systems such as IPFS.

Cons

A user cannot see the entire data in a transaction as the two fields, asset and metadata will point to cryptic strings. (It should be noted that as far as the asset field is concerned this inconvenience will only hold for CREATE transactions.)
Complicates queries. An API could however be easily provided that would allow the storage of the data. The hash of the data could be used as a unique identifier. It is interesting to note that different unrelated transactions could actually be referring to the same asset or metadata. There could perhaps be a registry of assets ...

More pros:

could reduce the transaction size
"bad data" could, in theory, be deleted if it's really bad, and the blockchain would still be okay (valid)

Things to consider:

Where to store the data/metadata?
Replicated storage? Fault tolerant? Byzantine fault tolerant?
How to index and query the data/metadata?
Why not use an existing blockchain to do this? (Maybe it's because BigchainDB has a higher transaction rate, or offers built-in indexing/querying, or something else.)


Enumeration Support for Jerkson

It would be a great addition to Jerkson if it supported ser/deser of Scala Enumerations.  Sample below:
object Day extends Enumeration {
val Mon, Tues, Wed, Thurs, Fri, Sat, Sun = Value
}

+1
Also the following construct doesn't seem to work:
sealed abstract class Status
case object ON extends Status
case object OFF extends Status
if one tries to serialize/deserialize the object containing such type, it is serialized as {}
if we do something like:
case object ON extends Status
{val strValue:String="ON"}
case object OFF extends Status
{val strValue:String="OFF"}
then it does put these values in when serializing, but crashes and returns null when tries to deserialize.

I would dearly love to be able to serdes arbitrary algebraic data types.  I have a lot of pairs of Foo and FooDTO classes. :(


layoutanimation on android not work

layoutanimation on android not work

UIManager.setLayoutAnimationEnabledExperimental &&   UIManager.setLayoutAnimationEnabledExperimental(true);

to enable LayoutAnimation on Android

This should be documented in LayoutAnimation.

+1 @amiuhle

This SHOULD be documented in LayoutAnimation!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Where do I put this code?

@JasonScopServ Hey I usually put it to the constructor:
constructor() {
    super();

    if (Platform.OS === 'android') {
      UIManager.setLayoutAnimationEnabledExperimental && UIManager.setLayoutAnimationEnabledExperimental(true);
    }
  }

This should really be documented, broke my head over this.

@janmarsicek Super helpful.

Hi @jasan-s! According the docs it should be ok to set the following flags like in the example in https://github.com/facebook/react-native/blob/master/docs/Animations.md#layoutanimation-api Sorry I don't have time to check if it works...

One more thing to import Platform.
import { Platform } from 'react-native';

still not getting this to work with an actual animation, whether i set the flag in the app.js file or ini a component directly.

I manage to fix this issue
https://github.com/ngohungphuc/Mobile-Development/issues/2

Could anyone answer:
If we use LayoutAnimation in several components - should we call
UIManager.setLayoutAnimationEnabledExperimental(true)
in each component, or we can just call it once?
Thanks


support Windows platform?

hi,is this support windows platform ?

We do not test / develop this for windows. If you'd like to and add a GitHub action to test it we will accept the PR, as long as it does not break using on POSIX operating systems.


To Bootstrap, or not Bootstrap?

In issue #2, you @masbicudo proposed a project structure that contemplates both Bootstrap and not Bootstrap components. I totally agree, given that we have the exact same components with the exact same API for both
The problem right now is that we don't have enough time. I'd rather have a rich set of components for Bootstrap  (or not) then supporting both and have them half-baked.
Then I propose start off having Bootstrap only components. Here's why:
Bootstrap con:
Bootstrap isn't that flexible. However, from my personal experience, I had a bad experience with bootstrap when I wanted to create websites with a visual identity.  But Bootstrap is very suitable for line-of-business apps because they don't need to be that unique and it offers good support for forms.
Bootstrap pro:
If we didn't go with Bootstrap, we'd have to implement our own presentation framework. I mean, we'd have to implement common CSS to make sure all the components share a concise look and feel. It's not just about styling each component. I's also about making sure they look consistent. And with Bootstrap we'd have that for free.
So, what do you think about going with Bootstrap for now and then trying to make BS independent component, if we feel the need?

One other HUGE advantage of bootstrap: it already has got tons of themes.

Right.. Then we agree to start on Bootstrap and then do whatever feels right.

We can start with bootstrap, but that means only using their CSS, not the JS part.
All components must be reactjs, using our gearz guidelines for components development.

I'm good with using only CSS..


Remove appended campaign id appended to new campaign name

Remove appended CX id to campaign name when creating new campaign. We are changing the way we parse exports and will reference by the format "_levelX" instead of CX identifiers. This gives us the ability to correlate levels of assessments with campaigns in GoPhish exports.
🗣 Description
This removes the old structure of appending CX to the campaign name. We no longer need this format and will be referencing by level_id (1-6).
💭 Motivation and context
We no longer need this format and will be referencing by level_id (1-6).
🧪 Testing
To test and validate that the name format change was implemented correctly, I did the following:

Build a new image that includes the changes in this branch and tag it for reference by running:
docker build -t gophish-tools-issue-43 .

Run the following to override the default image and reference the newly tagged image with the changes:
export GOPHISH_TOOLS_IMAGE="gophish-tools-issue-43" && eval "$(docker run gophish-tools-issue-43)"

The alias should be setup to drop into a bash instance on the gophish-tools container. You can run the following to access the container:
gophish-tools-bash

. In the contaner bash instance, create a new assessment as you normally would by running the alias:
pca-wizard

Input Campaign data and import into gophish. The campaign name should now exclude the appended _C1 that was originally there.

I also confirmed that the campaign went through all phases correctly sending email and generating click output as expected with the new name format.
✅ Checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.

You should remove the unchecked checkbox under "Checklist" in the PR description, unless you intend to add tests fro this change.
Also, I didn't see any documentation updated here, so I suggest removing the "All relevant repo and/or project documentation..." line as well (unless you intend to add it).

You should remove the unchecked checkbox under "Checklist" in the PR description, unless you intend to add tests fro this change.
Also, I didn't see any documentation updated here, so I suggest removing the "All relevant repo and/or project documentation..." line as well (unless you intend to add it).

Thanks for catching that. I meant to go back and remove them, but my memory failed me. 😆
I've updated the checklist! Thanks for looking it over!

This looks good to me. 👍
In the "Testing" section of your PR description, I'd like to see this written as "Here's what I did and how I confirmed that everything is correct", rather than what you currently have which comes across more like "Here's what I plan to test" with no indication of successful testing.

👍  Agreed. That provides more clarity. I've updated it to include more details on how it was tested. Thanks for looking it over!


Passes without changing code Perform a Subset Check on Two Sets of Data

Describe your problem and - if possible - how to reproduce it
This challenge can be passed without changing the code in the editor at all. Just click "Run the Test" and it passes.
Add a Link to the page with the problem
https://learn.freecodecamp.org/coding-interview-prep/data-structures/perform-a-subset-check-on-two-sets-of-data/
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: Version 67.0.3396.99 (Official Build) (64-bit)
Operating System: Ubuntu Linux 16.04

If possible, add a screenshot here

Pull request made

Closing this issue, because it has been fixed in beta. Viewable at https://www.freecodecamp.rocks/


kxcjk1013: code cleanup and update sensor description

Code cleanup and proper close iio device in destructor.
KXCJK-1013 is tri-axis accelerometer from Kionix.
This sensor can measure acceleration in metre per second squared or in
G-forces.
The library provided is libupm-kxcjk1013.so.0.4.0.
The example provided is kxcjk1013-example where it will print x,y,z axis when
trigger buffer data is ready.
Signed-off-by: Lay, Kuan Loon<EMAIL_ADDRESS>

Merged.  Thanks Kuan Loon.


assign style to table by row?

currently, I do this by something like
head = ['Name', 'Birthday', 'Phone No']
for i in range(len(head)):
    tab.cell(0, i).text = head[i]
    for par in tab.cell(0, i).paragraphs:
        par.style = 'my style'

is it possible to do something like?
head = ['Name', 'Birthday', 'Phone No']
for i in range(len(head)):
    tab.cell(0, i).text = head[i]

tab.rows[0].style = 'my style'

Thnaks

@retsyo did you get this to work?
I have a similar question. @scanny what are our options for conditionally changing row styles (e.g. background colour)? Or are they set at the table-level? In which case, is there a way to access background colour etc. of a table cell/row and change it dynamically?

@davidasboth there are no "row" styles, just a table style. If you want to do something like highlight a row you need to do it for each cell in that row individually.

Thanks, yeah I wasn't convinced you could have separate styles for separate parts of a table. I couldn't find the right interface for changing a cell's properties like background color, where should I be looking @scanny?

I don't believe any of that is implemented yet so you'd be on your own to develop or find someone who's worked out a workaround function that manipulates the underlying XML directly. I'd start with a Google of "python-docx cell shading" and see where that takes you.

I figured as much. I'll see what I can do with the XML. Thanks again for the quick replies.

The first result already looks promising (to give future readers a head start): https://stackoverflow.com/questions/26752856/python-docx-set-table-cell-background-and-text-color


offline video playback

As a user of an OCW mirror drive or otherwise offline site, I'd like to be able to play back locally stored archive videos while browsing said offline site
Acceptance Criteria:

[ ] Embedded videos and video pages load the offline version of the video in offline themes
[ ] Transcript and captions are also loaded from local resources in the offline themes
[ ] If the offline version of the video isn't available, try and load the video from YouTube instead

Progress update:

I have videos playing back locally in VideoJS locally
Since the mirror drive and the offline ZIP downloads both use the same theme, course-offline, if this change is made there then the sites in the ZIP downloads are going to expect the videos to be included. With this in mind and also the fact that there are videos missing from the original OCW mirror, I'm exploring the possibility of checking for the existence of the archive video during the offline build. If the video is not found, the YouTube player would be rendered instead
Regardless, the archive videos will need to be pulled into the Concourse worker at build time and I'm worried that this will significantly increase publish time because of the size of videos. I'm working on testing locally to get an idea about how much impact copying in the videos might have on publishing time.


Add @typescript-eslint/recommended-requiring-type-checking to eslint ruleset

The main purpose of this PR is to turn on the following rule: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/await-thenable.md (follow up from this).
The above rule requires type information to run.
I thought that it would be a good time to turn on plugin:@typescript-eslint/recommended-requiring-type-checking
Most of the changes are to appease no-floating-promises which was already on but not working.

Seems like this was greatly benefitial in finding bugs and probably will be in the future as well. Good job!
Should we turn '@typescript-eslint/no-unsafe-return' and the likes off for the webview? Seems like we're disabling the rule a lot.

I spent a really long time getting the new rules turned on and fixing the commented-out lines in the webview would have pushed me over the edge. '@typescript-eslint/no-unsafe-return' is a good rule and I think we should use it. If we turn it on now then we can avoid any further breakages going forwards. Hopefully, we can slowly remove the comments over time (haha).


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit e4fca28:
:clock10: Pending: waiting for processors (32 builds ahead of you)
:warning: Docs Build is busy, currently there are 32 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit e4fca28:
:clock10: Full build: the average full build time is 7 min(s) 58 sec(s), this is based on the last 30 build(s) for this repository.

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

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


feat(behavior_path_planner): plan shift pull over/out on curve

Description
https://user-images.githubusercontent.com/39142679/203253194-bd027d2b-d2cb-46a8-b036-0bb2fc932a94.mp4
https://user-images.githubusercontent.com/39142679/203253626-04b66217-dd4e-460b-b86b-2c5393c104ce.MOV
Related links

tier4 internal link
Tests performed

tier4 internal test 820/825
Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@rej55 I fixed them with your review, could you please check them?

@taikitanaka3
could you approve this as a code owner?


Copy doesn't work in the search editor

In the VS Code search editor, if you try to use cursorless to copy some text it doesn't get loaded to the clipboard. You don't get the same behaviour if you use knaus/talon.

This is somewhat improved, but still not fixed. Now if your focus is in the search editor you should be able to use Cursorless. The problem is that we have no reliable way for cursorless to move focus from the input box to the actual search editor.


I flashed but screen not seen

I flashed but screen not seen

bro do you have discord
if you have pls add me
my discord haji_x

Friend request sent👍


Print source/target branches on PR's list

Feature Description
It would be great if user can see source/target branches of each PR on PR's list like it shown in BitBucket (for example).
I'm already implemented these feature in my fork for my team and I'm thinking about creating PR to your project (from my fork). In this issue I want to discuss about these feature/design. I'm ready to make some changes (change design, make it optional and etc) to make it acceptable for your mainstream project. Example attached as screenshot
Screenshots

Closing this issue because PR was successfully merged


XGlow kicking players and throwing errors (1.19.3)

Paper Version: git-Paper-386 (MC: 1.19.3)
Plugin Version: XGlow version 1.1.0
ProtocolLib Version : 5.0.0-SNAPSHOT-b612
Console error message: https://gist.github.com/Chronoken/fd5e1739671b5961eb14328575661b31
Steps to reproduce:

Launch a paper 1.19.3 server with latest version of XGlow and ProtocolLib.
Disguise any entity or other player
Watch everyone get kicked out of the server with that error message appearing in the console/logs.

Just to let everyone know, tried reproducing it without the TAB plugin from NEZNAMY but the issue still exists :/

The players are getting kicked with this message: Internal Exception: io.netty.handler.codec.EncoderException: java.lang.ClassCastException: class net.minecraft.network.syncher.DataWatcher$Item cannot be cast to class net.minecraft.network.syncher.DataWatcher$b (net.minecraft.network.syncher.DataWatcher$Item and net.minecraft.network.syncher.DataWatcher$b are in unnamed module of loader java.net.URLClassLoader @61e717c2)

The plugin does not support 1.19.3 at the moment. The server protocol has been changed - you need to add a new implementation of the package.

Fixed this bug for 1.19.3 in #30


[Issue] In v0.3, there's 'NSRangeException'

Describe the bug
When printing about PCPU1's info, there's 'NSRangeException'
Operating System (please complete the following information):

Name: Ventura
Version 13.0

Device (please complete the following information):

Model: 2021 MacBook Pro / MacBookPro18,1
CPU: Apple M1 Pro

Additional context
2022-08-22 10:04:26.763 socpwrbud 2[77096:665102] *** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayM objectAtIndexedSubscript:]: index 2 beyond bounds [0 .. 1]'
*** First throw call stack:
(
0   CoreFoundation                      0x00000001b23bbe08 __exceptionPreprocess + 176
1   libobjc.A.dylib                     0x00000001b1f0aea8 objc_exception_throw + 60
2   CoreFoundation                      0x00000001b24a03b4 -[__NSCFString characterAtIndex:].cold.1 + 0
3   CoreFoundation                      0x00000001b232dc20 -[__NSCFString hasSuffix:] + 0
4   socpwrbud 2                         0x0000000104dca878 textOutput + 1232
5   socpwrbud 2                         0x0000000104dcf888 main + 6264
6   dyld                                0x00000001b1f3be38 start + 2520
)
libc++abi: terminating with uncaught exception of type NSException
[1]    77096 abort      ./socpwrbud\ 2

@op06072 Can you confirm whether this occurs on a machine not running the Ventura beta, and could you try running in Xcode to see at what exact line the failure comes from?

@op06072 Can you confirm whether this occurs on a machine not running the Ventura beta, and could you try running in Xcode to see at what exact line the failure comes from?

Oh... I don't have extra machine. But I can run this on Xcode. Ok. I'll do that

@op06072 Can you confirm whether this occurs on a machine not running the Ventura beta, and could you try running in Xcode to see at what exact line the failure comes from?

It can't build in Xcode. How can I build this?

@op06072 Can you confirm whether this occurs on a machine not running the Ventura beta, and could you try running in Xcode to see at what exact line the failure comes from?

There's error on 44th line in output.m file. In vd->sluster_instrcts_ret[i], i is 2 but vd->cluster_instrcts_ret's length is 2.

@op06072 can you share an output from the IOReport dumping tool, please?
This issue didn't occur in v0.2... v0.3 hasn't changed anything that would cause this problem, so I am assuming that the Ventura beta is messing with stuff. I need to check IOReport entries to verify...

@op06072 can you share an output from the IOReport dumping tool, please?
This issue didn't occur in v0.2... v0.3 hasn't changed anything that would cause this problem, so I am assuming that the Ventura beta is messing with stuff. I need to check IOReport entries to verify...

Here it is.
ioreport_dump_ventura.txt

@op06072 ooooh I see what the problem is. Okay, can you see if this works for you?
socpwrbud-v0.3.1.zip

@op06072 ooooh I see what the problem is. Okay, can you see if this works for you? socpwrbud-v0.3.1.zip

It works! Thanks!


SampledEfficientZeroModel does not pass lstm_hidden_size through DynamicsNetwork

In the __init__ method of SampledEfficientZeroModel, the argument lstm_hidden_size should be passed through the DynamicsNetwork. Otherwise the DynamicsNetwork will have the default lstm_hidden_size of 512, rather than what the user specifies. This leads to the following error:
Traceback (most recent call last):
  File "zoo/pooltool/sum_to_three/config/sum_to_three_image_config.py", line 122, in <module>
    train_muzero(
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/entry/train_muzero.py", line 160, in train_muzero
    new_data = collector.collect(train_iter=learner.train_iter, policy_kwargs=collect_kwargs)
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/worker/muzero_collector.py", line 411, in collect
    policy_output = self._policy.forward(stack_obs, action_mask, temperature, to_play, epsilon)
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/policy/sampled_efficientzero.py", line 867, in _forward_collect
    self._mcts_collect.search(
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/mcts/tree_search/mcts_ctree_sampled.py", line 162, in search
    network_output = model.recurrent_inference(
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/model/sampled_efficientzero_model.py", line 305, in recurrent_inference
    next_latent_state, reward_hidden_state, value_prefix = self._dynamics(latent_state, reward_hidden_state, action)
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/model/sampled_efficientzero_model.py", line 427, in _dynamics
    next_latent_state, next_reward_hidden_state, value_prefix = self.dynamics_network(
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/Users/evan/Software/pooltool_ml/LightZero/lzero/model/efficientzero_model.py", line 576, in forward
    value_prefix, next_reward_hidden_state = self.lstm(x, reward_hidden_state)
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 875, in forward
    self.check_forward_args(input, hx, batch_sizes)
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 791, in check_forward_args
    self.check_hidden_size(hidden[0], self.get_expected_hidden_size(input, batch_sizes),
  File "/Users/evan/anaconda3/envs/pooltool_ml/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 256, in check_hidden_size
    raise RuntimeError(msg.format(expected_hidden_size, list(hx.size())))
RuntimeError: Expected hidden[0] size (1, 6, 512), got [1, 6, 128]

Hello, thank you for your feedback. We have identified this bug and it has now been fixed in the latest commit https://github.com/opendilab/LightZero/commit/3823560044297ec34662589feac172210af4c4e6. Best wishes!

Thank you @puyuan1996. You guys at the LightZero team are so quick and helpful.


Training time on Kinetics400 and GPU?

Thanks for your great works!
I want to know that how much time did you take to train on Kinetics400 and what GPUs did you use?

Normally, it will take 3~4 days for me  to train TANet-50 with 8 * 1080Ti GPU on Kinetics400. In my opinion, it better to assure batch size >= 64 during training. Other training details can refer to our paper.
Hope this work for you.

Thanks, it helps me a lot.


What the lib "pointops" for

Hi,
I'm kind of a rookie here. Could someone please tell me something about the lib "pointops"? I searched it on google but got insufficient results. I guess it is some tool for point operations and saw "pointops_cuda" when searching, but I wonder where is its source?
Thanks a lot.

Sorry for not understanding you well. There are some pointclouds operations implemented on cuda in the pointops. What do you mean source?

Sorry for not express the question well. What I mean is, is the pointclouds operations implemented by 'pointops' defined by individuals (i.e. author yourself), or is is some kind of library that was predefined such that we only need to call them? If it is a library stuff, where is it originated?
By the way, I follow the guidelines and run the code, but I dont know why the output pointcloud by SAM3D is somehow lack of groups(or labels? colors?). The final visualization of scene0000_00 is shown below. The running feedback of the terminal is normal(showing 0.jpg to 5560.jpg and merging pointcloud by half the number until it goes to 1). Do you happen to know where could be wrong?

And the visualization code I use is:
`from util import *
import torch
Define the file paths
pcd_filepath = 'F:/sam3d/processed_pcd/train/scene0000_00.pth'
pcd_seg_filepath = 'F:/sam3d/results/pcd/scene0000_00.pth'
Load point cloud data
pcd_data = torch.load(pcd_filepath)
seg_data = torch.load(pcd_seg_filepath)
save_path = 'F:/sam3d/results/pcd/scene0000_00.ply'
visualize_partition(pcd_data['coord'], dict(group=seg_data)['group'], save_path)`


App Config Sample with Entra ID

Adds App Configuration sample that uses Entra ID

please also add this into the parent readme and pom


fly should offer shell completion

Shell completion is very handy. It would be nice if fly could offer it the same way kubectl does.

http://kubernetes.io/docs/user-guide/kubectl/kubectl_completion/
https://github.com/kubernetes/kubernetes/blob/8fd414537b5143ab039cb910590237cabf4af783/pkg/kubectl/cmd/completion.go

This should already work, but we should link to these docs (or document it ourselves): https://godoc.org/github.com/jessevdk/go-flags#hdr-Completion

It would be neat if fly knew how to set up bash/zsh completion without making users copy and paste a script. Apparently kingpin offers a --completion-script-bash option for doing this, but I didn't see a similar option in go-flags. Any interest in teaching that option to fly or go-flags? Or just link to the docs like you said.

Moved to concourse/concourse#1098


WindowCovering: YAML Add GoTos tests

Problem
Add the following YAML tests in the harness :

Add WNCV 3.4 3.5 4.3 4.4 for Linux chip tool
Add WNCV 4.3 4.4 for Darwin Chip tool
3.4 3.5 -> are not compiling yet with Darwin https://github.com/project-chip/connectedhomeip/issues/15192
Contains Todos added due to missing SDK implementation https://github.com/project-chip/connectedhomeip/issues/15190

Change overview

We activate some missing function in all cluster app
Creating Yaml TC-WNCV-4.3 TC-WNCV-4.4 TC-WNCV-3.5 TC-WNCV-3.4

Testing
How was this tested? (at least one bullet point required)

Tested on EFR32 demo app
Passed the CI test suite

/rebase


Using getMultiCapabilities() does not start tests with provided config

Bug report

Node Version: 6.2.1
Protractor Version: 3.3.0
Browser(s): chrome
Operating System and Version Mac OS
Your protractor configuration file:

getMultiCapabilities: function () {
//Example is simplified, but still does not work.
let cap = {
            'browserName': 'chrome',
            'chromeOptions': {
                // Allowing unsecure HTTPS connections.
                'args': ['disable-web-security', 'use-fake-device-for-media-stream', 'use-fake-ui-for-media-stream'],
                // Allowing all browser notifications by default - http://goo.gl/RM1Q3n
                prefs: {'profile.managed_default_content_settings.notifications': 1}
            },
            prefs: {
                'VideoCaptureAllowedUrls': ['url']
            },
            maxDuration: 3600,
};
return [cap]
},

A relevant example test
Any, tests does not start at all.

Output from running the test:

**************************************************
Process finished with exit code 0

Steps to reproduce the bug
Just try to use getMultiCapabilities in you protractor conf file

Hi! Trying to use getMultiCapabilities() function to provide customized multicapabilites, but it seems like protractor does not accepts capabilites from this function.
Tried a lot - to return array with capability, to return promise resolved to array of capabilities, no luck. Browser does not starts at all. Can someone please check if this function is still works?

Updated to protractor 4.0.3 - and it works there
Should i close it now?

Since it works on the recent release of Protractor, closing this issue. Thanks for doing the investigation.


CB-21798 Public subnet info contains only ENDPOINT_ACCESS_GATEWAY in …

…DeploymentRestrictions. In case of new network if the customer does not request private network then we do not apply just the LB restriciton but all of them.
See detailed description in the commit message.

retest this please

https://github.com/hortonworks/cloudbreak/pull/14896


Version bump <IP_ADDRESS> (DEV)

Already included in https://github.com/corona-warn-app/cwa-app-android/pull/2851/files#diff-3d103fc7c312a3e136f88e81cef592424b8af2464c468116545c4d22d6edcf19


Can't get LTS versions for Centos7, Fedora28

From here:
https://dotnet.microsoft.com/download/dotnet-core/2.1
If I am interested in LTS version (v2.1.10, SDK 2.1.506) for Centos7 or Fedora28 and I follow the Centos7 or Fedora28 instructions how to get SDK 2.1.506:
https://dotnet.microsoft.com/download/linux-package-manager/centos/sdk-2.1.506
https://dotnet.microsoft.com/download/linux-package-manager/fedora28/sdk-2.1.506
I do NOT get SDK 2.1.506:

================================================================================
 Package                  Arch    Version    Repository                    Size
================================================================================
Installing:
 dotnet-sdk-2.1           x86_64  2.1.603-1  packages-microsoft-com-prod  114 M
Installing for dependencies:
 aspnetcore-runtime-2.1   x86_64  2.1.10-1   packages-microsoft-com-prod   28 M
 dotnet-host              x86_64  2.2.4-1    packages-microsoft-com-prod   44 k
 dotnet-hostfxr-2.1       x86_64  2.1.10-1   packages-microsoft-com-prod  196 k
 dotnet-runtime-2.1       x86_64  2.1.10-1   packages-microsoft-com-prod   27 M
 dotnet-runtime-deps-2.1  x86_64  2.1.10-1   packages-microsoft-com-prod  2.8 k

Transaction Summary
================================================================================
Install  1 Package (+5 Dependent packages)

As you can see, I get SDK 2.1.603, which is NOT labeled an LTS version on the dotnet-core/2.1 website.
Could you please create a new rpm, or rpm "alias", call it "dotnet-sdk-2.1-lts", that actually points to the LTS packages Microsoft is advertising in its SDK download website please? (see screenshot):

This way there is always a way for people to pull down the LTS version via package manager tools (yum, dpkg, etc).
Thank you.

@jpfreyen, there are a few confusing things at play - sorry about that.
The toolset (SDK) is not LTS; the Runtime is. The latest SDK can be used to target and build for any version of the .NET Core runtime. With the announcements which happened at //BUILD last week, we're working on our LTS policy language and will make sure this part is clear.
Most package managers give you the ability to specify a specific package version to install. On Fedora, this would look like the following.
dnf --showduplicates list dotnet-sdk-2.1
Last metadata expiration check: 0:00:49 ago on Tue 14 May 2019 12:35:03 PM PDT.
Available Packages
[I shortened the list a bit]
dotnet-sdk-2.1.x86_64                    2.1.504-1                    packages-microsoft-com-prod
dotnet-sdk-2.1.x86_64                    2.1.505-1                    packages-microsoft-com-prod
dotnet-sdk-2.1.x86_64                    2.1.602-1                    packages-microsoft-com-prod
dotnet-sdk-2.1.x86_64                    2.1.603-1                    packages-microsoft-com-prod
dotnet-sdk-2.1.x86_64                    2.1.604-1                    packages-microsoft-com-prod

Then you can install using dnf install dotnet-sdk-2.1-2.1.504-1 (or whatever available version you want). It's by-design that 2.1.507 is not available on the feed because 2.1.604 is the correct choice for folks not developing in Visual Studio.

I do have a few notes here... First being that Fedora 28 is soon to be EOL'd and you should hop on 29 or even 30.
Then it's the whole question of using the right install sources. With Linux in mind, always use what's native to the distribution, it's repositories and things maintained by the people who know their distribution the best. In the case of Fedora the information is behind a handy alias fedoraloves.net which goes on to explain how to install dotnet on Fedora, for example to get the 2.1 (LTS) it would be:
$ sudo dnf copr enable @dotnet-sig/dotnet
$ sudo dnf install dotnet-sdk-2.1

@leecow that is more clear, thanks.
@RheaAyase, sometimes it's not practical for a shipping project to always use what is available to the distribution, especially if the distro keeps moving up packages and there isn't a good way to get the previous package, that is hard on QA.  The dotnet-sdk-2.1.-lts idea is a Ubuntu thing in which they have package "aliases" that will go and get the specific packages (like "build-essentials" for example, using that will go and get 'git', gcc', etc).

Well, 2.1 is LTS and it doesn't need an alias for that. As it is described above doesn't make much sense. It would make sense to have dotnet-lts though.
Especially in Fedora we package everything so that you can stick to any minor and even any feature version of the SDK.
Here is slightly reduced list for dnf list --showduplicates dotnet-sdk*
rjanek@RedHat:~/dev/qe$ dnf list --showduplicates dotnet-sdk* | grep xx.x86_64
dotnet-sdk-2.1.5xx.x86_64             2.1.506-1.fc30    @group_dotnet-sig-dotnet
dotnet-sdk-2.1.2xx.x86_64             2.0.9-1.fc29      group_dotnet-sig-dotnet
dotnet-sdk-2.1.3xx.x86_64             2.1.302-1.fc30    group_dotnet-sig-dotnet
dotnet-sdk-2.1.4xx.x86_64             2.1.403-3.fc30    group_dotnet-sig-dotnet
dotnet-sdk-2.1.5xx.x86_64             2.1.506-1.fc30    group_dotnet-sig-dotnet
dotnet-sdk-2.2.1xx.x86_64             2.2.103-1.fc30    group_dotnet-sig-dotnet

@RheaAyase I agree and think it's a great idea to have a "dotnet-lts" :-).

Is there still an outstanding question or follow up?

The initial problem is with unclear versioning :P
I guess that the real question is whether it's documented sufficiently or we need to improve that.

Were I last commented was the idea of having a dotnet-lts named package.  Is that going to happen?

Oh and a question of Fedora 28 EOL'ing brought up by @RheaAyase ...is .NETCore 2.1 LTS going to be updated for: RHEL8, Fedora 29, and Fedora 30 (and others like Linux Mint 19)?  Documentation doesn't mention any newer releases that 2.1 LTS supports:
https://github.com/dotnet/core/blob/master/release-notes/2.1/2.1-supported-os.md

Microsoft may be building and publishing the built product for some of the Linux distributions, that however isn't the preferred install source from Linux point of view. The release notes you link, or the packages that Microsoft publishes on the download website, are likely to not keep up with the life cycle that many Linux Distributions have.

RHEL7 has everything currently supported.
RHEL8 was released only with LTS 2.1 because dotnetcore 2.2 will EOL very soon and that wasn't enough to justify the heroic effort that would be required to also include 2.2 :P
Fedora 29/30/rawhide (and future versions) has everything currently supported. You can find specific builds, versions, dates, etc here: https://copr.fedorainfracloud.org/coprs/g/dotnet-sig/dotnet
Can't comment on other distributions as I don't know :)

Created PR#2855 to address the release-notes/supported-os issue.

@RheaAyase can we close now? Or are we still tracking some additional work?

Hi @karelz, I don't think that there is more work to be done, unless y'all want to somehow resolve the confusing inconsistency between the different SDK versions :/
(I know that this should be resolved in the future versions. Can't wait the day!)

@RheaAyase, yes, if the inconsistencey between the different SDK versions could be resolved sometime that would be great


GIFs over ~50mb do not show up

GIFs that are over ~50 mb do not show up on the actual webpage.
If the GIF is over ~50 mb, it will simply not show anything, it won't show the "It's a surprise" image for when there is no image associated for that particular day, just nothing.
GIFs that are below ~50 mb show up totally fine.

Hi @daylamtayari
I reproduced the same behaviour on my webserver.
However, this is a limitation of the PHP engine, not the application.
In my php.ini file I have the default configuration for the output_buffering parameter:
output_buffering = 4096

That means the file served can not be over 4096 bytes, the ~ 50 MB limit you noticed.
By changing it to Off my 99 MB GIF is well displayed.

Another solution would be to use another function that file_get_contents() to read and serve the file. Seems a bit complex for now.

https://stackoverflow.com/a/21936954/2337765
https://stackoverflow.com/a/54704668/2337765


Resource Health RP changes for new resource type /events and /impacte…

Resource Health RP changes for new resource type /events and /impactedResources.
This is a re-open from previous PR https://github.com/Azure/azure-rest-api-specs/pull/2127 which was closed due to inactivity around an year back. Starting a new one with first iteration for changes which were already signed off by ARM (please check label in previous PR). Will add another update/iteration with changes that have happened since then, should be minor additions of additional properties.
We want a new preview api-version 2018-01-01-preview.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-js

Automation for azure-sdk-for-node

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

@HimanshuMisra1 could you fix model validation errors https://travis-ci.org/Azure/azure-rest-api-specs/jobs/464693921 ?

There seems couple of model Validation errors still pending https://travis-ci.org/Azure/azure-rest-api-specs/jobs/466782277, please take a look.


is there a mac binary available?

is there a mac binary available?

Of xi-gtk? No there isn't. I wouldn't even know how to build it on macOS.

Ok what about Xi mac

From: eyelash<EMAIL_ADDRESS>Sent: Sunday, July 16, 2017 2:46:57 PM
To: eyelash/xi-gtk
Cc: markmark1; Author
Subject: Re: [eyelash/xi-gtk] is there a mac binary available? (#31)
Of xi-gtk? No there isn't. I wouldn't even know how to build it on macOS.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/eyelash/xi-gtk/issues/31#issuecomment-315639525, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGHomz-g1aG2fTkGvRvy6puYYZBgo9rkks5sOoTRgaJpZM4OZZiU.

What's xi

From: eyelash<EMAIL_ADDRESS>Sent: Sunday, July 16, 2017 2:46:57 PM
To: eyelash/xi-gtk
Cc: markmark1; Author
Subject: Re: [eyelash/xi-gtk] is there a mac binary available? (#31)
Of xi-gtk? No there isn't. I wouldn't even know how to build it on macOS.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/eyelash/xi-gtk/issues/31#issuecomment-315639525, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGHomz-g1aG2fTkGvRvy6puYYZBgo9rkks5sOoTRgaJpZM4OZZiU.

I don't think there are any mac binaries available at all, but xi-mac can easily be built with Xcode. See also google/xi-editor#373


Add pseudo fullscreen mode

Many sites have a "pseudo fullscreen mode"(网页全屏) for video player that fill the current browser window with the video other than the entire screen.

#194


introduce lite ipfs node for downloading results

A kubo node configured with low power and limited functionality for the purpose of downloading results from IPFS. In addition to that, the downloader now is reusing the node instead of spinning up a fresh node for each download request

it seems there are better options using https://github.com/ipfs/go-libipfs library
Thread: https://filecoinproject.slack.com/archives/C02RLM3JHUY/p1675705603221589?thread_ts=1675682783.855449&cid=C02RLM3JHUY


Return value is 0 when specify --fail-on-issue and found a issue

▶ infer analyze --fail-on-issue -a eradicate
Found 34 source files to analyze in /Foo/bar/xx/infer-out
Starting analysis...

legend:
  "F" analyzing a file
  "." analyzing a procedure

F..FF.app/src/main/java/com/xx/MainActivity.java:10 Error ERADICATE_NULL_METHOD_CALL in foo The value of `s` in the call to `length()` could be null. (Origin: null constant at line 67)
ERADICATE_NULL_METHOD_CALL: app/src/main/java/com/xx/MainActivity.java: void MainActivity$1.foo(AppBarLayout,int)
The value of `s` in the call to `length()` could be null. (Origin: null constant at line 9)
....F..F..........
Analysis finished in 0.543549s

foo@bar ✗
▶ echo $?
0

Thanks for the report. It appears that --fail-on-issue only does its job when you use infer run and not for infer analyze. Will fix.


catchError does not expose the status and body

dio.post("https://local.xxxx.com/api/native/v1/login/email", data: {"email": 12, "password": "wendu"})
.then((response){
print(response);
})
.catchError((onError){
print(onError);
});`

print(onError); prints:
DioError [DioErrorType.RESPONSE]: Http status error [422]
the server is sending:
status:422
message: {errors: [{field: "email", error: "incorrect format"]}
Do I need to regex DioError [DioErrorType.RESPONSE]: Http status error [422] to get 422?
How do i get the message?

you can get statusCode by DioError.response, for example:
 dio.get("https://wendux.github.io/xsddddd").catchError((e)=>print(e.response?.statusCode));

same here, did you get to the solution @ziasultan2

same here,

The body can be accessed with e.response.data


Add option to call tarsnap with a function

This is a great idea!

calltarsnap() feels a little obtuse to me. I feel like simply tarsnap() might be more fluent. Do you agree?

The tarsnap command itself, if it's in $PATH, already uses the name tarsnap. I don't feel comfortable overriding that.

Should this deprecate the dedicated pre/post script logic? It could but I'm not sure if it's a good idea. Thoughts?

I think stability is more important than a tight design (especially in a fire-and-forget system like acts) so I would not deprecate anything.

The tarsnap command itself, if it's in $PATH, already uses the name tarsnap. I don't feel comfortable overriding that.

Ah, of course. That is completely correct.
I am on vacation right now, but I’ll merge this when I get back.


Improve SSL settings in Nginx conf.

Set up strong SSL security on the Nginx webserver. This way we have a strong and future proof ssl configuration and we get an A+ on the Qually Labs SSL Test.
Example: https://www.ssllabs.com/ssltest/analyze.html?d=nightly.umbrella.apinf.io (A+)
Allow new config key dhparam in api-umbrella.yml with path to dhparam file.
Example:
nginx:
  server_names_hash_bucket_size: 128
  dhparam: /etc/ssl/private/dhparams.pem

I am only not sure about enable "HTTP Strict Transport Security" https://developer.mozilla.org/en-US/docs/Security/HTTP_Strict_Transport_Security, because after that changes host will be available only by HTTPS.
This request based on instructions and tutorials:

https://wiki.mozilla.org/Security/Server_Side_TLS
https://cipherli.st/ (https://raymii.org/s/tutorials/Strong_SSL_Security_On_nginx.html)
https://github.com/h5bp/server-configs-nginx

/cc @brylie

That changes related to https://github.com/NREL/api-umbrella/issues/24, https://github.com/NREL/api-umbrella/issues/234

Ping @GUI

Ping @GUI

Thanks so much for the contribution! And apologies for the delay in responding to this and the original SSL issue.
Also, thanks for tackling this, since providing better SSL defaults is important (we had previously been terminating SSL outside of API Umbrella in our own installations, so we admittedly hadn't been as on top of this as we should have, but we're beginning to switch to having API Umbrella handle SSL too).
The only issue I see is that I think we need to make some of these options more configurable for some of our other use-cases. While these do look like good default values, we have some legacy environments we need to support where Strict-Transport-Security can't be enabled yet, and I'd also like to be able to tune some of the other items like the SSL session cache memory (since 50MB may be high or low, depending on your environment). Ideally, I think also allowing things like the protocols and ciphers to be configurable via api-umbrella.yml config options would also be good so if future security issues come up, they can be resolved via more immediate config changes.
Does making some of these options more configurable in the api-umbrella.yml config make sense to you? I'm also happy to help out with those additional tweaks, but it will probably be next week before I can get to it.
More generally, there's two things on the SSL and configuration front that might be relevant:

At some point in the future (hopefully not too far off), we'll probably look to integrate lua-resty-auto-ssl with API Umbrella. This will allow for SSL certificates to automatically be registered/managed/renewed for free for any domain. We hope that will make it super-easy for SSL to be the default for any number of domains API Umbrella is handling. We're currently piloting this, and it's been working well, but we still need to do a bit of work to better integrate it with API Umbrella. So stay tuned!

Regarding making these items configurable, as a bigger picture idea, I've also periodically thought about allowing for the inclusion of other custom files directly into API Umbrella's nginx config. So for example, API Umbrella could include /etc/api-umbrella/custom_nginx.conf, and then administrators could manage that file to specify custom settings.
This might allow for a lot of flexibility in configuring things like these SSL settings without needing to patch API Umbrella's template files. However, for now, we've been steering clear of that approach, since I think it would quickly complicate support, and things would be more likely to break across upgrades if you customized that file at all. For example, we will need to tweak a various SSL settings if we rollout the lua-resty-auto-ssl support, so if you happened to have any conflicting options in your custom file, things would likely break in unexpected ways during API Umbrella upgrades.
So for now, I think we'll probably stick with only allowing more structured and specific configuration options to be set via api-umbrella.yml configuration changes which can then be leveraged in the template config files. While this approach does mean jumping through a couple of hoops when you want to implement something new, I think it gives us a bit more control to making the upgrade path more predictable. But I did want to mention this alternative idea, since it has come up. Let us know if you have any thoughts.

Thanks again!

@GUI Updated. Most useful options now configurable by api-umbrella.yml

@GUI BTW maybe will be interesting for that project Routing HTTP requests with Nginx+Mruby


Create resolve_issues_with_laptops_and_tablets.md

Context
Changes proposed in this pull request
Guidance to review

Closed in favour of https://github.com/DFE-Digital/get-help-with-tech/pull/1558


docs(charter): Declare new Intentional Artifacts as 'small' changes

What does this PR try to resolve?
The default stance for new "Intentional Artifact" crates is that an RFC is needed, see Crate ownership policy.
However, it gives room for team's to have their charter define the process.  As we don't generally need wide input for these decisions, I'm proposing we treat these as "small" changes and only require an FCP.
How should we test and review this PR?
Additional information

@rfcbot fcp merge

As this one is not too controversial, going to merge this. Thanks!
@bors r+

:pushpin: Commit c08b437e9d329dc01856a6947deda4a5bbcb5dc7 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit c08b437e9d329dc01856a6947deda4a5bbcb5dc7 with merge faf69dfe104eb0ae33630dbfeb493ce5db270aab...

:boom: Test timed out

@bors retry

:hourglass: Testing commit c08b437e9d329dc01856a6947deda4a5bbcb5dc7 with merge ac3de0b1479283fad9252a34b2dc0c5eb79ba5e0...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing ac3de0b1479283fad9252a34b2dc0c5eb79ba5e0 to master...


chore(): remove debug and log statements

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (npm run build) was run locally and any changes were pushed
[x] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #18190
What is the new behavior?

console.log and console.debug will no longer be allowed in production builds of Ionic

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Removing log/debug statements from prod builds of Ionic is resolved by this commit: https://github.com/ionic-team/stencil/commit/e676e40999a0ea53d3eced6ce757ed4457fc442a

@liamdebeasi Do we need Stencil 1 for this or will it be backported?

@brandyscarney I believe @manucorporat introduced this fix only for Stencil 1.


Transaction data requirement

Thanks again for the well-maintained document! I am a researcher who interested in the impact of the monetary incentive on OSS and the opencollective is a great dataset which deserves further study. My target is to provide useful insights for the backers, projects and the platform.
Following is the script I found in https://github.com/opencollective/opencollective-api/pull/1723/commits/f47c42eeed467f50295c7feb3857c7ab77544b06, which is used to export transaction data. Since I want to analyze the full transaction dump, I modified the startDate placeholder to '2015-11-01' and the endDate placeholder to '2019-02-01'. (btw, is the earliest creation date of projects on opencollective is "2015-11-01"?)
Since the rejected expense information is also important and valuable, how should I query about rejected expense information?
Query script
SELECT
t."createdAt" , c.slug as "collective slug", t.type as "transaction type", t.amount::float / 100,
t.currency, fc.slug as "from slug", fc.type as "from type", t.description, e.category as "expense category",
h.slug as "host slug", t."hostCurrency", t."hostCurrencyFxRate",
pm.service as "payment processor", pm.type as "payment method type",
t."paymentProcessorFeeInHostCurrency"::float / 100 as "paymentProcessorFeeInHostCurrency",
t."hostFeeInHostCurrency"::float / 100 as "hostFeeInHostCurrency",
t."platformFeeInHostCurrency"::float / 100 as "platformFeeInHostCurrency"
FROM "Transactions" t
LEFT JOIN "Collectives" fc ON fc.id=t."FromCollectiveId"
LEFT JOIN "Collectives" c ON c.id=t."CollectiveId"
LEFT JOIN "Collectives" h ON h.id=t."HostCollectiveId"
LEFT JOIN "PaymentMethods" pm ON pm.id=t."PaymentMethodId"
LEFT JOIN "Expenses" e ON e.id=t."ExpenseId"
WHERE t."createdAt" >= "2015-11-01" AND t."createdAt" < "2019-02-01"
  AND t."deletedAt" IS NULL
ORDER BY t.id ASC
  `

Thank you in advance!

Yeah we can’t export expenses like that because the receipts / invoices may contain sensitive information. Also note that this won’t pull up rejected expenses. This will only get the ones linked to a transaction, therefore the PAID ones.

Yeah we can’t export expenses like that because the receipts / invoices may contain sensitive information. Also note that this won’t pull up rejected expenses. This will only get the ones linked to a transaction, therefore the PAID ones.

Thanks for the explanation. I have already removed the e* in the main content. Btw how to query the rejected expense information? Should I raise a new request for this requirement?

It's ok I've adapted the query. Everything is in https://drive.opencollective.com
Have fun with the data and please share what you found! :-)

Great! Thanks for your help!!!!! Definitely, I will share the result and hope that will help this community as well!


Fix idsAvailable calls regarding recent React Native updates

Fixes https://github.com/geektimecoil/react-native-onesignal/issues/54

Thanks, Do you have any idea if this is working on previous versions of RN, rather than 0.28?

@avishayil just checked on RN 0.23.1 - working fine.


Added a useful option

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Congrats @TwistedxTempo on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


feat: index a model on multiple indexes at once

Adds a build_object/2 with the index name
Adds the possibility to index a same model on multiple indexes (applies for replicas as well)
Based on #56

Are we ready to merge this PR? @quaresc @StephaneRob
We're almost done with Jobs migration and we'd like to use the main branch in production 🙂

@quaresc can you rebase this PR?


Does the package need ios version > 9.0?

why the ios deploy target version of this package is 9.1 ?

@haibinyu - good point!
I lowered it to 8.0 (the lowest xcode will let me select) and pushed version 1.1.1


Recommend settings for exporting tiles quickly

Exporting even modest datasets to tiles or mbtiles currently takes a very long time (for me) -- typically 12+ hours for a Postgres database that is only around 100mb in size.
#688 recommends some Postgres config settings to improve performance. Are there other recommended settings for the JVM that would improve performance or parallel processing settings?
Would it be possible to put these in the baremaps documentation?

Would it be possible to get more information about the source data imported in postgresql, the size of each table in postgresql and the tilset.json?
For comparison the included example in examples/openstreetmap is exported to mbtiles in less than a minute for a postgresql database around 50MB.
select
  table_name,
  pg_size_pretty(pg_relation_size(quote_ident(table_name))),
  pg_relation_size(quote_ident(table_name))
from information_schema.tables
where table_catalog  = 'baremaps' and table_schema = 'public' and table_type = 'BASE TABLE'
order by pg_relation_size desc;

table_name     |pg_size_pretty|pg_relation_size|
---------------+--------------+----------------+
osm_nodes      |36 MB         |        37478400|
osm_ways       |15 MB         |        16039936|
spatial_ref_sys|6896 kB       |         7061504|
osm_relations  |904 kB        |          925696|
osm_headers    |8192 bytes    |            8192|

Thanks for the quick response. Here's a rundown of the setup, and then the output you requested.
I'm running baremaps 0.7.1 locally, connecting to a postgis database container running in docker. For export, I run the following command:
baremaps map export --tileset="./data/tileset.json" --database="jdbc:postgresql://localhost:9001/baremaps?&user=baremaps&password=baremaps" --repository="./data/tiles"

The input data is a subset of the OSM export for the state of Texas, limited to administrative boundaries and information about roads and airports. Almost all of the tileset definitions are of the form:
SELECT id, tags, geom FROM osm_ways WHERE tags ? 'aeroway' AND tags ->> 'aeroway' = 'runway'

There are indices on tags and geom for all the osm_* tables, like osm_ways.
When running the export command, you can see how long it takes:
[INFO ] 2023-07-20 17:57:27.967 [main] ExportVectorTiles - Exporting vector tiles from jdbc:postgresql://localhost:9001/baremaps?&user=baremaps&password=baremaps to /home/joe.polastre/git/basemap-tiles/./data/tiles
[INFO ] 2023-07-20 17:57:27.972 [main] HikariDataSource - HikariPool-1 - Starting...
[INFO ] 2023-07-20 17:57:28.126 [main] HikariPool - HikariPool-1 - Added connection org.postgresql.jdbc.PgConnection@7fc6de5b
[INFO ] 2023-07-20 17:57:28.127 [main] HikariDataSource - HikariPool-1 - Start completed.
[INFO ] 2023-07-20 17:57:33.280 [ForkJoinPool.commonPool-worker-20] ProgressLogger - 0.0%
[INFO ] 2023-07-20 17:57:38.329 [ForkJoinPool.commonPool-worker-8] ProgressLogger - 0.0%
[INFO ] 2023-07-20 17:57:43.350 [ForkJoinPool.commonPool-worker-33] ProgressLogger - 0.0%
[INFO ] 2023-07-20 17:57:48.360 [ForkJoinPool.commonPool-worker-28] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:57:53.374 [ForkJoinPool.commonPool-worker-17] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:57:58.399 [ForkJoinPool.commonPool-worker-27] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:03.414 [ForkJoinPool.commonPool-worker-10] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:08.415 [ForkJoinPool.commonPool-worker-14] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:13.423 [ForkJoinPool.commonPool-worker-14] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:18.485 [ForkJoinPool.commonPool-worker-19] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:23.500 [ForkJoinPool.commonPool-worker-29] ProgressLogger - 0.01%
[INFO ] 2023-07-20 17:58:28.506 [ForkJoinPool.commonPool-worker-20] ProgressLogger - 0.02%
[INFO ] 2023-07-20 17:58:33.530 [ForkJoinPool.commonPool-worker-24] ProgressLogger - 0.02%

Postgresql database:
baremaps=# select
  table_name,
  pg_size_pretty(pg_relation_size(quote_ident(table_name))),
  pg_relation_size(quote_ident(table_name))
from information_schema.tables
where table_catalog  = 'baremaps' and table_schema = 'public' and table_type = 'BASE TABLE'
order by pg_relation_size desc;
   table_name    | pg_size_pretty | pg_relation_size
-----------------+----------------+------------------
 osm_nodes       | 76 MB          |         80117760
 osm_ways        | 13 MB          |         13524992
 spatial_ref_sys | 6896 kB        |          7061504
 osm_relations   | 1384 kB        |          1417216
 osm_headers     | 8192 bytes     |             8192
(5 rows)

Hi, thanks for the information. I was not able to reproduce your issue.
What comes to mind are the following suggestions:

Make sure you have increased the docker limits for CPU, memory and disk (in settings>resources)
Try running Postgis without Docker
Tweaking the postgres.conf settings (this is specific to each machine)

@leonardcser I have run the workflow and export for examples/openstreetmap. The output area of those tiles is incredibly small, the size of Liechtenstein (160 sq km). When I export, of course it runs fast (a few seconds) since so few tiles actually need to be generated.
Regardless of how PostgreSQL is run (docker or not), the baremaps map export java process takes up 100% of every available CPU up to my 36 CPUs available. This is likely the bottleneck on export. I'm not sure what java is actually doing since most of the work is in PostgreSQL to generate the MVT tiles.
Have you tried exporting areas than Liechtenstein? Can you send me those examples and any benchmarks? How long did it take? What kind of machine did you use? Were you using v0.7.1 release or a branch?

Yes, the bounds must be set to the correct extent. The planet export currently takes about three days on a big server (16 cores, 128GB of Ram, SSD drives, etc.). That beeing said, the export process has not yet been optimized and we can probably find ways to improve it (e.g. with batches).

@polastre Do not hesitate to reopen this issue if the proposed fix does not address it correctly.


[NETBEANS-54] Module Review debugger.jpda.ui

no external library
checked Rat report: add the license header to genfiles.properties and README.txt
skimmed through the module, did not notice additional problems

For the genfiles.properties I would consider not adding a header.

I'll revert them. Thanks!


[RFC] Add LayerSkip to AO

Tracker issue for adding LayerSkip to AO.
This is a training and inference optimization that is similar to layer-wise pruning. It's particularly interesting for LLM inference because it combines very cleanly with speculative decoding to provide up to a 1.86x speedup.
@mostafaelhoushi is interested in adding this to torchtune and is interested in upstreaming a subset of the code to ao. See here for more details. In particular, he's interested in doing this without having to alter the module definition.
This is attractive because this part of LayerSkip is not unique to LLMs and can be used for other models. (@mostafaelhoushi to fill out with relevant results).
What is being proposed:
for LayerSkip there is a training recipe and there is an inference recipe:

training recipe:

layer dropout: This is skipping layers sotchastically during training. this is what I think we can get into torch.ao, because it could benefit all types of models, transformers, CNNs, vision, text, etc. It can speedup training and might improve accuracy.
early exit loss: This could also be added to torch.ao and it could help different modalities, but my require more time.

inference recipe:

speculative decoding: yes, this applies to LLM only, has been added to gpt-fast here: https://github.com/pytorch-labs/gpt-fast/compare/main...LayerSkip

Yup, the layer dropout aspect of layer skip is basically a version of stochastic depth, that's part of the reason why I'm interested in having it in AO, since a generic stochastic depth function / module would be useful outside of just LLMs.
IIRC when talking to mostafa he is faster when masking + rewriting but the speedups mostly come from the self-speculative decoding part of the technique.
@mostafaelhoushi can you share some benchmarks about the layer dropout implementation specifically when you update the issue? Thanks.

Sorry for the delay from my side.
Other Papers
I would like to mention other papers or models that used layer dropout (aka stochastic depth):

It was first explored in ResNets by Huang et al, 2016.
ConvNext uses it in its training recipe. It uses higher layer dropout rates for larger models 0.1/0.4/0.5/0.5 for ConvNeXt-T/S/B/L respectively trained on ImageNet. However, when training on the larger ImageNet-22K it uses smaller layer dropout rates: 0.0/0.0/0.1/0.1/0.2.
Layer dropout is also commonly used in vision transformers. Swin Transformers use higher layer dropout rates for larger models: i.e., 0.2, 0.3, 0.5 for Swin-T, Swin-S, and Swin-B, respectively.
Dinov2 also used layer dropout (Cc @danthe3rd) when training.

Other Implementations

Regarding per batch vs per sample dropout, we can implement both. However, I would like to mention that

when I tried measuring speedup for both I found them to be similar.
per batch dropout causes errors when training with FSDP. This is because FSDP likes to find gradients for all modules in a model. Per batch layer dropout will cause a module not to have a forward pass nor a backward pass at an iteration, and this causes FSDP to throw an error. There are mitigations for this, e.g., to overwrite the backward pass (as done here in fariseq2).

My understanding that the timm implementation doesn't lead to speedup as it replaces samples with zeros rather than skipping compute, right?

Benchmark Results
On TorchTune, I ran this command on a single A100 GPU
$ tune run --nproc_per_node 1 full_finetune_distributed --config llama3/8B_full output_dir=$CKPT_PATH checkpointer.checkpoint_dir=$CKPT_PATH/original checkpointer.output_dir=$CKPT_PATH tokenizer.path=$CKPT_PATH/original/tokenizer.model batch_size=16

and got these measurements:

Maximum Dropout
Dropout Scale Across Layers
Time to Reach 50 Iterations
Speedup

None

01 min 32 sec
1x

0.2
Uniform
01 min 23 sec
1.07x

0.3
Uniform
01 min 17 sec
1.19x

0.5
Uniform
01 min 05 sec
1.42x

0.5
Linear.
01 min Xy sec
Xx

0.2
Exponential
01 min 30 sec
1.02x

0.5
Exponential
01 min 22 sec
1.12x

I also want to tag @danthe3rd as he guided me to implement the per-sample layer dropout and he has implemented it for Dinov2.


Verify All Airflow DAGs Working

[ ] DAGs retrieving data from websites starting w/ ftp:/ not working. @brettpalmberg
[x] High Resolution Rapid Refresh (HRRR) downloads are failing with HTTP Status 403

HRRR requests and associated DAG are. All S3 requests (s3.amazonaws.com) are routed through a VPC Endpoint. Configuration has been modified to allow required actions on bucket `noaa-hrrr-bdp-pds through VPC endpoint.  Confirmed working:

DAGs retrieving data via ftp:// are now working in the environment.

All current Airflow DAGs should now work from this point forward in the environment. Root cause has been fixed, infrastructure-as-code updated.  Will open additional GH issues if we find any other DAGs that are not working.
cc: @willbreitkreutz @adamscarberry @jeffsuperglide

awesome.  thanks @brettpalmberg !

You bet! Happy to jump in, team effort all the way through! 👍🏻


Generated macro functions.

Following @OvermindDL1's idea:
Currently when you define a named pattern
   def var(name, meta, context) when is_atom(name) and is_list(meta) and is_atom(context)

The following macros are generated:
defmacro var()
defmacro var(bindings)
defmacro var(name, bindings)
defmacro var(name, meta, bindings)
defmacro var(name, meta, context, bindings)

var() will underscore all pattern variables not used in guard.
bindings is a Keyword of names to expressions, used to bind variables inside the pattern.
All other functions are just there for providing positional arguments (vs named ones via the last keyword)
Now, I'm thiking about adding a couple more:
defmacro var?(expr)
defmacro var!(bindings)

The first one will just check that expr conforms - that is it matches the pattern and all its guards are satisfied
and the second will raise an error unless all variables inside the patter are bound by name from bindings, it will be used for as data constructor, even for those patterns with guards.

Now I'm thinking of no generating more macros, but just have Expat.match!(pattern) and match? just like we have Kernel.match?


Track jumps (source and destination) in a stack

This will allow them to be reversed by rewinding, or if the writer wants to imitate Renpy's call function.

Completed as of 5602f546ae79fa27861856d53ae75221796433cd.


smartgithg: 18.2.9 -> 19.1.1

Motivation for this change
noticed it was broken (bad sha) while reviewing another package.
bumped version while at it
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @jraygauthier
[1 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/66692
1 package were build:
smartgithg

Looks like their sha256 keeps changing upon download.
  wanted: sha256:0hl0nypvkj1bnqn6352a0szhzhr6zd7b4p9f5wyaxa4dw565461p
  got:    sha256:0i0dvyy9d63f4hk8czlyk83ai0ywhqp7wbdkq3s87l7irwgs42jy

Couldn't it be that the expected sha256 is simply incorrect? I obtain the exact same sha as @mmahut:
$ nix-prefetch-url https://www.syntevo.com/downloads/smartgit/smartgit-linux-19_1_1.tar.gz
[47.7 MiB DL]
path is '/nix/store/ifbgm8klj9lzccg59n5pym0dicki5xzn-smartgit-linux-19_1_1.tar.gz'
0i0dvyy9d63f4hk8czlyk83ai0ywhqp7wbdkq3s87l7irwgs42jy

looks like i fixed the sha to get it to build, then bumped the version, then forgot to update the sha again. My bad :)

@GrahamcOfBorg build smartgithg


Fix flake8

When I follow the contributing guide and run flake8 tests pipreqs, I get many flake8 warnings, so i try to fix them.
Note still has following warnings:
tests/_data/test.py:2:80: E501 line too long (117 > 79 characters)
tests/_data/test.py:37:14: E401 multiple imports on one line
tests/_data/test.py:46:17: E401 multiple imports on one line
tests/_data/test.py:46:25: E901 SyntaxError: invalid syntax

The reason:

the first: I'm afraid that the modify may affect the pylint
the second and the third: is it for test purpose?
the forth: SyntaxError, I can't get the point.

maybe we should just ignore tests/_data/test.py? Cause we have many unused import.

tests/_data/test.py is syntactically wrong to test pipreqs for different cases.


Automation at 5kW flow to net

Iam looking for a way to switch on my waterboiler when production-consumption= >5kW and switch it back off when < 5kW.
Not sure if this is already possible, if so ill like to have some guidance. If not it would be great to have this implemented

please install 7.7.13-beta.8 then go to plugin settings Advanced Settings>>>Consumption>>>Net>>>Power  enable Power Peak Sensor  and set  Power Peak Detected to 5000W , save and restart plugin. Check and let me know.

i have pushed 7.8.0-beta where you can create multiple sensors for Power and Energy - Production, Consumption Total and Net (in the Net You can use negative values like -5000W, this is production-consumption = > 5kW)

added in 7.8.0


[5.7] improve heuristics for @main support

5.7 cherrypick of #5687

@swift-ci smoke test

These heuristics make sense to me.  FWIW, I cleaned up some of my earlier requests about a -parse-as-executable flag at https://github.com/apple/swift/issues/60277 so that build systems can declare the expected intent and don't need to be in the business of trying to conditionally defeat the compiler's heuristics.

But for now this looks like a definite improvement.

@tomerd Could you possibly pull https://github.com/apple/swift-package-manager/pull/5704 into this?

@swift-ci smoke test

@tomerd Could you possibly pull https://github.com/apple/swift-package-manager/pull/5704 into this?

@bitjammer done


Dynamic SchemaTransform wrapper provider

Creating a dynamic wrapper provider according to the design in https://s.apache.org/autogen-wrappers
Also created a GenerateSequence schematransform to use for testing (the reasoning is that this core transform is simple and will be relatively unchanged, making it a good candidate to check that the wrapper provider fundamentally works)

R: @chamikaramj
Thanks, PTAL!

Previous tests passed. Just added an Experimental comment and Python lint is passing. Merging now

This seems to be the source of spotbugs failure on main.

Danny has a fix in #29798


pandas-1.4-compatibility

These are leftovers from #296

Thank you! The staging seems to have disappeared, but the change looks like it will do the right thing. I am happy for this to be merged if the tests pass. Many thanks!

Codecov Report

Merging #297 (bb2427b) into master (8058ea3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #297   +/-   ##
=======================================
  Coverage   88.21%   88.21%
=======================================
  Files           5        5
  Lines        1035     1035
=======================================
  Hits          913      913
  Misses        122      122

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8058ea3...bb2427b. Read the comment docs.

Staging has appeared, and the bug is fixed. Well done - thank you!


feature show call start position at stack trace.

This feature solved one problem that is not clearly happened position.
This feature solved one problem that is not clearly happened position.
The problem is happen when using Ownable. actually, Problem source position is extended class method, but shown only Ownable method line.

@nakajo2011 Thanks!

Wow..! you do very fast work!!
Thanks for marge.


Release: Version 14

I haven't seen any check lists going around or a discussion on releasing the next branch so here we are.

[ ] Merge master bug fix pending PRs
[ ] Merge master changed into Next
[ ] Test Next
[x] RenderToNodeStream?

What else am I forgetting? We should get this up to date and out since the release of React 16 seems to be smooth.

Yes! We should also go through the ideas/suggestions for 'next' ticket and see which of those can be included: https://github.com/ctrlplusb/react-universally/issues/428

Hey all! Thanks for the activity here.
Please would all contributors get active on the gitter channel. Thanks 👍
I'd like us to start using a git rebase strategy for next and all pull requests in order to keep the merge noise down.  If you are unfamiliar with git rebasing then please let me know and I would be happy to take you through it. 👍
Please keep all changes targetting next rather than master. I know we have already let a few slip into master but moving forward it would be best to use next to stage up some changes before me merge in.
I'll review #428 and give some more of my ideas.
Also, has anyone tried react-loadable against this project?  James has added some cool APIs such as the preloadAll.  It may be worth trying a branch out and seeing how it flies.
I'd also like to start killing the feature branches.  Really keep them down to 2 or 3 at an absolute maximum.  I know we had an issue for this already.  I'll revisit it soon.

How does this work with css-in-js libraries, e.g. styled-components? Will this be a problem for branches?

@ctrlplusb which branches you think will be good to maintain?

@codepunkt you mean 'RenderToNodeStream' with css-in-js?

@sergiokopplin Yeah. Just noticed i've commented into the wrong issue. Sorry for that!

Another topic for v14:
https://github.com/ctrlplusb/react-universally/blob/83d533a9c780716d18f034f7fb52dbd3a1c4051b/internal/webpack/configFactory.js#L379-L383
I noticed in the configFactory here that we create client bundles in ES5. I was researching a little bit more and it seems like if we set up two bundles - one for modern browsers and one for non-modern browsers, we can make a significant dent on bundle sizes.
Still new to this topic but most of my understanding comes from this article: https://philipwalton.com/articles/deploying-es2015-code-in-production-today/
#514 and #531 seem to also be related to this.

I've been busy in React Native world, please let me know if you guys need help.
It's my favourite project so I'm looking forward to see in what direction we're going.

Hello,
Any updates on next version?
Best

next now has both webpack 4 and react 16
https://github.com/ctrlplusb/react-universally/compare/master...next
Question @ctrlplusb , are we using prettier-eslint or no? There seems to be an inconsistency between branches regarding prettier. All pull-requests are now including prettier junk, making it very hard to glance PR's.
Thanks

I am keen on vanilla prettier (i.e. without the prettier-eslint). We should also use eslint-preset-prettier alongside eslint-preset-airbnb in order to disable all the style rules. 👍

@birkir Thoughts on https://github.com/ctrlplusb/react-universally/issues/524#issuecomment-340304381 with webpack 4?