A 'Source model for source 4FGL J1228.7-0318 has no counts in ROI' error

Hello, I encountered an error while processing the light curve of 3C 279 using easyfermi on my Mac.
I was attempting to split the light curve spanning approximately 17 days into 20 bins and then split it using an adaptive method with a threshold of 50. I had successfully performed similar operations before (Also for 3C 279), but this time I encountered the error "WARNING lightcurve_408414660_408419015 GTAnalysis.profile_norm(): Source model for source 4FGL J1228.7-0318 has no counts in ROI." (I manually downloaded the photon and spacecraft files from https://fermi.gsfc.nasa.gov/cgi-bin/ssc/LAT/QueryResults.cgi?id=L240611223518B2F361C319, with a photon energy range of 100-1000 MeV and a search radius of 12 degrees.)
I tried removing 4FGL J1228.7-0318 from the source list in easyfermi and rerunning the analysis, but I still encountered the same error. Could you please advise me on how to resolve this issue? (Relevant information is included in the attached zip file.)

Package.zip

Thank you for your reply, I will try to use larger bins. By the way, how can I obtain a light curve with very small bins (for example, 3 hours or even smaller)? Should I manually use fermitools as shown in the FSSC tutorials instead of easyfermi?

Yes, for ~3 hours bins you should use the unbinned likelihood method available only in the fermitools.

Thank you for your patient response again, I got it.


bug: project_readonly ignored when meltano.yml lacks project_id

Meltano Version
2.15.3
Python Version
3.9
Bug scope
Configuration (settings parsing, validation, etc.)
Operating System
meltano/meltano:v2.15.3 x86_64 Docker image
Description
Context:

https://github.com/meltano/meltano/issues/7206#issuecomment-1430338308

Steps to reproduce:

meltano init example && cd example
meltano add --no-install extractor tap-github
Set project_readonly: true in meltano.yml
Delete project_id from meltano.yml
meltano install

project_id will have been re-added to meltano.yml, despite project_readonly being set to true.
Additionally, with the example in the reproduction steps above, meltano.yml was updated by meltano install, which should probably never edit meltano.yml.
Code
No response

Still relevant


Don't treat Google app measurement DBs as root files

A side effect of treating all files with su in name is preventing some Google APIs from work, like /data/user/0/<package>/databases/google_app_measurement.db. This can be used to detect the bypass or just make the app unstable or crash due to app database being unavailable.

@domyn  This issue has been resolved in my PR https://github.com/gauravssnl/BypassRootCheckPro/pull/20  .
I am closing this PR.

@domyn  This has been resolved in my PR https://github.com/gauravssnl/BypassRootCheckPro/pull/20 . So, I am closing this one.


remove hardcoded URLs

Will likely need additional exposed APIs in iron-router

https://github.com/iron/router/issues/109

Partially fixed. Need way to use url_for in templates.


Add Stax based xml parser and xml serializer

Purpose
$Subject. This is the first part of implementing new xml proposal.
https://github.com/ballerina-platform/ballerina-spec/blob/master/lang/proposals/xml/xml.md
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/19832
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Temporally closing this PR.


Enable security log

what would the security log be comprised of?

IMO, it should collect important events (sing-in event and etc) by collecting data such as username, time, ip-address, browser and other security-related request headers.

Check morgan https://github.com/expressjs/morgan


[FEATURE] Support for ARMv7 Architecture

Is your feature request related to a problem? Please describe.
ARMv8/ARM64 support was added in https://github.com/longhorn/longhorn/issues/6 but a lot of capable ARM devices are still in ARMv7.
An example is the Odroid HC2 which has a quite powerful CPU, 2GB of RAM and a SATA connector: https://www.hardkernel.com/shop/odroid-hc2-home-cloud-two/. A perfect device for running a replicated longhorn storage of hard drives or SSD at full speed.
Describe the solution you'd like
I would like to have a support for ARMv7 devices on Longhorn.
Additional context
If you need an ARMv7 device for testing:
Scaleway, a cloud provider, provides ARMv7 servers at a very low price (0.006€/hour): https://www.scaleway.com.
The server type is C1, it's not listed on their website anymore but it's still available on the control panel and using their API:

+1

I do believe, we can get many "+" here 👍

+1

+1

+1

+1

See #1847 for a tantalizing clue that it's possible to hack this together. @vsellier published some 32 bit arm images that are partially functioning.

It is a boomer not to have this yes :-(

Googling for longhorn support for my hc2 brought me here. It would make sense for homelab application

@innobead I'd really appreciate your suggestion, before I wipe out my working k3s/ganesha/glusterfs cluster

@ypyly We currently do not support ARMv7, as mentioned by @innobead there was some previous community work to make ARMv7 work, so if you want to go down that path we appreciate community contributions in this area. If you want an easier path, install a ARMv8 64bit operating system, for example https://ubuntu.com/download/server/arm

@joshimoo not sure I see how installing a different OS is going to change the CPU architecture of ARMv7.

i wish we could soon have support for armv7 so i could have a full working k3s cluster on my raspberry pis cluster <3

i wish we could soon have support for armv7 so i could have a full working k3s cluster on my raspberry pis cluster <3

You can already do that if you install armv8 on your rasp cluster

i wish we could soon have support for armv7 so i could have a full working k3s cluster on my raspberry pis cluster <3

You can already do that if you install armv8 on your rasp cluster

@unixfox but i could not find any armv8 images for longhorn? i can see only arm64 ad i tried to use arm64 and never worked

i wish we could soon have support for armv7 so i could have a full working k3s cluster on my raspberry pis cluster <3

You can already do that if you install armv8 on your rasp cluster

@unixfox but i could not find any armv8 images for longhorn? i can see only arm64 ad i tried to use arm64 and never worked. so i think i have to install 64bit OS to be able to use it

When I meant armv8, I also meant arm64. Just install an arm64/armv8 linux distribution on your raspi and you will be good to go.

Correct if you install the 64bit version of https://ubuntu.com/download/server/arm as an example you will be able to use longhorn on your raspi cluster.

+1

+1

+1


Milestone 9: Update README description

Milestone 9: Update README description
The second milestone in building my portfolio includes the following

README should be descriptive
Commit messages should be meaningful
PR should have a descriptive title and a short summary describing introduced changes in more detail

In the following task:
-import screenshot describing introduced changes in more detail
-create a meaningful README file

Thank you, I will do it


Just want to say this is amazing

Thank you so much for building this. I live in my calendar, but constantly forget to check the LEC schedule and miss games I want to watch because I'm busy.
This repo has made me so much more likely to catch the games I want to watch live — it's seriously a godsend. Thank you for building this!

And you're from Vienna as well?! What are the chances! 🇦🇹 represent 😍

thx all !!!!! @zlypher


Dev mode should support multiple modules

Use Case:
I have a maven reactor project with multiple modules.
The modules are the different components of the application.
One module compose the functionality and exposes it as REST API and Websocket API and only this module requires Open Liberty, so only this module has the liberty-maven-plugin installed.
Now i want to start liberty in dev mode and it should detect code changes in any module.
Problem:
Changes in other modules don't trigger reload of the deployed application.
I have to manually run mvn install to get the changes of all modules picked up by liberty.

A good number of our projects hit this scenario.  I see two main issues: 1) the Dev Mode must be started by a specific module, and 2) only that module's changes are monitored.  An additional wrinkle is that the developer must either invoke mvn liberty:dev from a leaf module, or configure skip=true for the root POM and skip=false for the module corresponding to the deployable artifact.

@Legion2 @rdean-csx Could you please elaborate on your project setup and ideally how you would use dev mode with it?  For example, from which module(s) would you expect to run dev mode, and what are the packaging types of each module (e.g. jar, war, ear)?

All of our projects use EAR packaging for consistency, and some of them use EJB modules or utility jars.  In the simplest case, the module layout would look like this:
myapp
+- myapp-ear
+- myapp-war

Sometimes there will be EJB modules or utility jars, but those cases are mainly legacy applications that have been migrated to Liberty.
In this setup, I would expect Dev Mode would operate in the module describing the artifact that is deployed to Liberty.  It should react to changes in that module, plus any module in the reactor pulled in as a dependency (e.g., in the example above, Dev Mode would operate on myapp-ear, and should react when I make changes to either myapp-ear or myapp-war).
Also, I would like Dev Mode to only operate in the deployed artifact's module, so if I run mvn liberty:dev from the root module, it will run in myapp-ear.

I have a couple of ideas of how to fix this.  Would you accept a code contribution here?  Are there any non-obvious constraints that I should be aware of?

Code contributions are welcome.  Please see the contributing guidelines.
You would also need to build a local snapshot of ci.common as a prerequisite to building ci.maven and include this commit.

OK.  I submitted the individual CLA and we're working on a corporate CLA.

Hi,

is there any update? If I can help reviewing or implementing this, please let me know.

Cheers,
Tom

Gesendet: Montag, 27. Januar 2020 um 19:12 Uhr
Von: "rdean-csx"<EMAIL_ADDRESS>An: "OpenLiberty/ci.maven"<EMAIL_ADDRESS>Cc: "Thomas Schuettel"<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Betreff: Re: [OpenLiberty/ci.maven] Dev mode should support maven reactor projects (#697)
I've worked out an approach that I think will work. I'm going to code it this week.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I deleted that comment not too long after making it.  I'll have an ETA by next Monday.

This item is in plan as part of our roadmap, particularly for https://github.com/OpenLiberty/ci.maven/issues/727.  That said, we would welcome any contributions and are open to discussions on ideas that you may have on this.

Should consider the relevant Maven plugins (http://maven.apache.org/plugins/index.html) for the types of modules that will be supported.

I have another usecase for this: a WAR application which depends on JAR modules from the same reactor.
The code handling it will probably be the same, but this case is simpler in that there's only one deployable artifact (the WAR).
Btw, any news or updated ETA @rdean-csx?

All of our projects use EAR packaging for consistency. In the simplest case, the module layout would look like this:
myapp
+- myapp-ear
+- myapp-war

Sometimes there will also be EJB modules or utility jars, but those cases are mainly legacy applications that have been migrated to Liberty.
In this setup, I would expect Dev Mode would operate in the module describing the artifact that is deployed to Liberty. It should react to changes in that module, plus any module in the reactor pulled in as a dependency (e.g., in the example above, Dev Mode would operate on myapp-ear, and should react when I make changes to either myapp-ear or myapp-war).
Also, I would like Dev Mode to only operate in the deployed artifact's module, so if I run mvn liberty:dev from the root module, it will run in myapp-ear.

I use a slightly different layout:
myapp-moudule-list
+- myapp-ear (<type>ear</type>)
+- myapp-war (<type>war</type>)
+- myapp-impl (<type>jar</type>)
+- myapp-liberty (<type>pom</type>)

The idea is, to keep the "applications pom.xml" clean from the deployment configuration. In the *-liberty pom.xml the plugin is configured and as dependency, the ear module. mvn liberty:dev is executed in the myapp-liberty module.

Hi all, you can use the following workaround to enable dev mode with multi module projects.  This requires the latest development snapshot version 3.3.5-SNAPSHOT of liberty-maven-plugin which is available on the Sonatype snapshot repository.  It also requires an IDE or editor with Java support that can auto-rebuild to the Maven target directory of each module.

Enable the Sonatype snapshot repository by adding the following snapshots profile into your ~/.m2/settings.xml file:

<settings>
    <profiles>
        <profile>
            <id>snapshots</id>
            <pluginRepositories>
                <pluginRepository>
                    <id>sonatype-nexus-snapshots</id>
                    <name>Sonatype Nexus Snapshots</name>
                    <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
                    <snapshots>
                        <enabled>true</enabled>
                    </snapshots>
                    <releases>
                        <enabled>false</enabled>
                    </releases>
                </pluginRepository>
            </pluginRepositories>
        </profile>
    </profiles>
</settings>

Import your multi module project into an IDE or editor as a Maven project, and enable auto-builds in the IDE or editor's settings.  Ensure that the JDK used by the IDE or editor matches the Java version that is used by Maven.

From the directory that contains your multi module pom.xml file, run the following command (substituting <liberty-module> with the name of the module that contains your Liberty server configuration, which can be a module of type ear, war, liberty-assembly, or pom):

mvn -Psnapshots io.openliberty.tools:liberty-maven-plugin:3.3.5-SNAPSHOT:dev -pl <liberty-module> -am

Note that if your Liberty server configuration is located in a module of type liberty-assembly or pom which depends on a module of type ear, you will need to mvn install the ear project to the .m2 directory once before running the above command (the mvn install won't be needed on subsequent runs of dev mode).  For example:
mvn install -pl myapp-ear -am
mvn -Psnapshots io.openliberty.tools:liberty-maven-plugin:3.3.5-SNAPSHOT:dev -pl myapp-liberty -am

@Legion2 @rdean-csx @tschuettel @ggam @waldi5001  It would be great if you could give the above a try and let us know if this works for you or if you have any feedback. Thank you.

Thanks for your efforts here.  It fell off of my radar due to other priorities, but we still have many projects that could benefit from this.

This is great news - I will for sure give this a try and give my feedback.
Thanks a lot for your effort
On Thu, Apr 8, 2021, 15:55 Eric Lau @.***> wrote:

Hi all, you can use the following workaround to enable dev mode with multi
module projects. This requires the latest development snapshot version
3.3.5-SNAPSHOT of liberty-maven-plugin which is available on the Sonatype
snapshot repository. It also requires an IDE or editor with Java support
that can auto-rebuild to the Maven target directory of each module.

Enable the Sonatype snapshot repository by adding the following
snapshots profile into your ~/.m2/settings.xml file:

snapshots

sonatype-nexus-snapshots
Sonatype Nexus Snapshots
https://oss.sonatype.org/content/repositories/snapshots/

true

false

Import your multi module project into an IDE or editor as a Maven
project, and enable auto-builds in the IDE or editor's settings. Ensure
that the JDK used by the IDE or editor matches the Java version that is
used by Maven.
2.
From the directory that contains your multi module pom.xml file, run
the following command (substituting  with the name of
the module that contains your Liberty server configuration, which can be a
module of type ear, war, liberty-assembly, or pom):
mvn -Psnapshots io.openliberty.tools:liberty-maven-plugin:3.3.5-SNAPSHOT:dev -pl  -am
Note that if your Liberty server configuration is located in a module of
type liberty-assembly or pom which depends on a module of type ear, you
will need to mvn install the ear project to the .m2 directory once before
running the above command (the mvn install won't be needed on subsequent
runs of dev mode). For example:
mvn install -pl myapp-ear -am
mvn -Psnapshots io.openliberty.tools:liberty-maven-plugin:3.3.5-SNAPSHOT:dev -pl myapp-liberty -am
@Legion2 https://github.com/Legion2 @rdean-csx
https://github.com/rdean-csx @tschuettel https://github.com/tschuettel
@ggam https://github.com/ggam @waldi5001 https://github.com/waldi5001
It would be great if you could give the above a try and let us know if this
works for you or if you have any feedback. Thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenLiberty/ci.maven/issues/697#issuecomment-815845904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABY5K4WAJJ5BSJYJ3F6EP73THWYWZANCNFSM4JTGK4DQ
.

I've managed to, locally, test the 3.3.5-SNAPSHOT dev setup - and it works!! Wuhuuu :)
The app.war.xml was generated as expected with the correct inline  elements with sourceOnDisk, for the other modules
Unfortunately it is not easy to let the teams use it, just yet, due to the way we mirror maven dependency (and not central snapshots), but I am really looking forward to the 3.3.5 release version, hopefully it will not be long?

@lborupj Great to hear that it works for you!  We have made available a 3.3.5-M1 pre-release milestone on Maven Central and I've updated the steps in my comment above to use the new version.

I've testet too, and it works well. My toughs: In short term, with small and easy projects you will generate a great benefit. No questen. But with a big project and a long boot time because, you have to bootstrap hibernate and spring, it wouldn't be so easy.
If you trigger a change, the whole application restart and that can cause problems. I haven't time to test it, but i try to build a testcase and see whats happening.
Is a runtime class reload technically possible in conjunction with liberty?

@waldi5001 You can replace classes at runtime without application restart by disabling dynamic updates in Liberty and using your debugger's Hot Code Replace function instead.  Here are the steps using VS Code as an example:

Start dev mode according to the post above.

Specify the following in your server.xml file to disable dynamic updates in Liberty:

<applicationMonitor updateTrigger="disabled"/>

In VS Code, configure the debugger to connect to port 7777: go to the Run and Debug tab on the left, click create a launch.json file, click Java, replace the file contents with the below:

{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "java",
            "name": "Attach",
            "request": "attach",
            "hostName": "localhost",
            "port": 7777
        }
    ]
}

In VS Code Settings, search for "hot code replace", then set the Java › Debug › Settings: Hot Code Replace value to auto

Go to the Run and Debug tab and click Start Debugging (green arrow).

Then when you make changes to source files in the different modules, the code will be replaced without application restart.

Hi all, the design document for this feature (pending review) is at https://ibm.box.com/s/lziitxx4jpsmekddi8p0nev90sevqv3l
Please let us know if you have any comments on what is covered in the design.  Thanks.

This is the first UFO I've seen.  I like being able to see that as a consumer of the project.
The design looks good.  The only flag that went up for me is that on slide 14.  Given that class A in module A depends on class B in module B, when class B is changed, then class A should be recompiled if class A had a previous compile error or the change to class B had a signature change that impacts class A's usage.  (Ideally.... if that level of introspection is not practical, I would expect the users of class B to be recompiled any time its public signature changes).

@rdean-csx Thanks for the feedback.  Currently dev mode does not track public signatures so we will note this as a limitation when using plain text editors for refactoring (although the error would be apparent when exercising the code through manual testing or unit/integration tests.)  As you suggested, using an IDE to refactor would avoid this issue.

Here are the notes that I took from this mornings UFO socialization for this feature:
Feature Design / Inter-module compilation

Only rebuilding if there is a compile failure previously in a module seems too restrictive.  Look at rebuilding any depending modules when a module is changed.  If module B is changed, rebuild module A unconditionally if it depends on module B.

Feature Design / Tests

If a module is changed, depending modules' tests should also be run, i.e. rerun module A's tests if it depends on module B and module B changed.
Maybe give an example of how the flags are configured in pom, liberty maven plugin, or as a system property

Admin / Config / Command Line

Update the fully qualified maven coordinates bullet to be correct saying unless the maven plugin is configured in all modules and in the parent module.  See note in Serviceability slide below.
Maybe use a different word than liberty-module.  All module will be running in liberty so there is some confusion on the term.  In documenation it should be called something different.
Add clarification that -p1 is optional and if not specfied it will build all modules.
What happens with multiple liberty-modules when p1 is not specified?  Error condition?  Then p1 is required?
If possible see if we can detect the corner case and handle it under the covers to not require the user to run mvn install manually.  If do that the last error in the serviceability slide is not needed.

Serviceability

You can also put this in the settings.xml, you'll get some latest versoin:  io.openliberty.tools  to resolve the first error.

Updated the UFO at https://ibm.box.com/s/lziitxx4jpsmekddi8p0nev90sevqv3l to address the notes in the comment above.  The main changes are on Slides 17-23 and 29-30.
@rdean-csx Slide 17 has a change relating to recompiling other modules.  Please let me know if that addresses your previous comment, thanks.

A few small comments:

slide 12 - parent -> POM (this is just aggregation, not inheritance)
slide 41 - For the 2nd issue, it might just be you wrongly guessed how multi-module worked.. confusing the directory we get the server.xml from with the directory you need to run from.. you might just want to run from the top-level project and that's it, (and
not really also require a mvn -pl my-ear -am).
slide 14 - "Module where dev mode is run"  -  Earlier this is phrased as the  "module providing server config".  It also makes me wonder if maybe the feature design could use a diagram showing how the reactor will launch liberty:dev in each module, but the dev mojo will have logic to short-circuit and exit in every other module.  This also explains the first issue in slide 41.

Updated slides 12, 41, 14 based on above comment.  Module behaviour is out of scope for the document since that can be considered implementation detail.

Follow On Review

Slides 23, 30 recompileDependencies flag.  Currently this is a -D command line only setting.  Consider allowing this option to be set dynamically.  Look to add this to the current UFO instead of having a separate UFO to consider this.

Updated slide 17 to mention the above and opened https://github.com/OpenLiberty/ci.maven/issues/1206 as an enhancement.

@waldi5001 You can replace classes at runtime without application restart by disabling dynamic updates in Liberty and using your debugger's Hot Code Replace function instead. Here are the steps using VS Code as an example:
1. Specify the following in your `server.xml` file to disable dynamic updates in Liberty:

<applicationMonitor updateTrigger="disabled"/>

1. In VS Code's Settings, search for "hot code replace", then set the `Java › Debug › Settings: Hot Code Replace` value to `auto`

2. Configure debugger to connect to port 7777: go to VS Code's Run and Debug tab, click `create a launch.json file`, click `Java`, replace the file contents with the below:

{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "java",
            "name": "Attach",
            "request": "attach",
            "hostName": "localhost",
            "port": 7777
        }
    ]
}

1. Start dev mode according to the [post](https://github.com/OpenLiberty/ci.maven/issues/697#issuecomment-815845904) above.

2. After dev mode is fully started, click Start Debugging (green arrow) in the Run and Debug tab.

Then when you make changes to source files in the different modules, the code will be replaced in the running JVM without application restart.

Hi, I'm having the same issue as described by @waldi5001. This works perfectly, thanks for the info. I think it would be great if that could end up in the documentation of the multi-module dev mode support.

@ericglau one design detail we might just want to think through again is the statement:

If skipTests, skipUTs, or skipITs flag is enabled in a module, skip the corresponding tests in that module only.  If those flags are enabled from command line, skip corresponding tests in all modules.

In general, in Maven, plugin configuration will take precedence over -Dx=y properties (see:    https://issues.apache.org/jira/browse/MNG-4979).
While in dev mode, perhaps we are inventing something custom at the aggregate/reactor level, combining and executing other goals (UT/IT/etc.) from the dev mode goal... I wonder if there's really any need to depart from the Maven convention here?
I'd think most of the time, individual projects won't have the test skip config anyway, and the -Dx=y properties will take effect.

@scottkurz Updated slide 18 and opened issue https://github.com/OpenLiberty/ci.maven/issues/1227 to handle the ordering priority of those flags.

@NottyCode Please review the UFO https://ibm.box.com/s/lziitxx4jpsmekddi8p0nev90sevqv3l for design approval.

Support for multiple module Maven projects in dev mode is now available in version 3.4 of the Liberty Maven Plugin https://github.com/OpenLiberty/ci.maven/releases/tag/liberty-maven-3.4
For a general overview of the feature, see the blog post.
For details on how to run dev mode with a multi-module Maven project, see the dev goal's documentation.


mixed_instances_policy is not working

Description
i am trying to create a self managed node group with the eks module latest version 18.2.2
i am using mixed_instances_policy to create
      mixed_instances_policy = {         instances_distribution = {           on_demand_base_capacity = "1"           on_demand_percentage_above_base_capacity = "100"           on_demand_allocation_strategy = "prioritized"         }         override = [           {             instance_type = "m5.2xlarge"           },           {             instance_type = "c5.2xlarge"           },         ]       }
along with
use_mixed_instances_policy = true
unfortunatly the plan does not show that the correct policy was created.
looking at the code in node_groups.tf in module "self_managed_node_group"
i see that the variable mixed_instances_policy is not passed to the module so that why it cant work.
if we added a line
mixed_instances_policy = try(each.value.mixed_instances_policy, var.self_managed_node_group_defaults.mixed_instances_policy, null)
to the node_groups.tf this should fix the issue
Versions

Terraform: 1.1.4

Provider(s):

Module: eks 18.2.2

Reproduction
Expected behavior

Actual behavior

Terminal Output Screenshot(s)

Additional context

please past a terraform module configuration which you are using

I seem to have the same issue.
Following terraform config
module "eks" {
  source          = "terraform-aws-modules/eks/aws"
  version         = "18.2.1"
  cluster_version = local.cluster_version
  cluster_name    = local.name
  vpc_id          = var.vpc_id
  subnet_ids      = var.subnets

  iam_role_name = "test"

  enable_irsa = true

  self_managed_node_group_defaults = local.self_managed_node_group_defaults
  self_managed_node_groups         = local.self_managed_node_group

this is a copy from the given example in github repo
  self_managed_node_group = {
    one = {
      name = "spot-1"

      public_ip    = true
      max_size     = 5
      desired_size = 2

      use_mixed_instances_policy = true
      mixed_instances_policy = {
        instances_distribution = {
          on_demand_base_capacity                  = 0
          on_demand_percentage_above_base_capacity = 10
          spot_allocation_strategy                 = "capacity-optimized"
        }

        override = [
          {
            instance_type     = "m5.large"
            weighted_capacity = "1"
          },
          {
            instance_type     = "m6i.large"
            weighted_capacity = "2"
          },
        ]
      }

      pre_bootstrap_user_data = <<-EOT
      echo "foo"
      export FOO=bar
      EOT

      bootstrap_extra_args = "--kubelet-extra-args '--node-labels=node.kubernetes.io/lifecycle=spot'"

      post_bootstrap_user_data = <<-EOT
      cd /tmp
      sudo yum install -y https://s3.amazonaws.com/ec2-downloads-windows/SSMAgent/latest/linux_amd64/amazon-ssm-agent.rpm
      sudo systemctl enable amazon-ssm-agent
      sudo systemctl start amazon-ssm-agent
      EOT
    }
  }

output seems to show that input should be fine according to examples.

self_managed_node_groups = {
  "one" = {
    "bootstrap_extra_args" = "--kubelet-extra-args '--node-labels=node.kubernetes.io/lifecycle=spot'"
    "desired_size" = 2
    "max_size" = 5
    "mixed_instances_policy" = {
      "instances_distribution" = {
        "on_demand_base_capacity" = 0
        "on_demand_percentage_above_base_capacity" = 10
        "spot_allocation_strategy" = "capacity-optimized"
      }
      "override" = [
        {
          "instance_type" = "m5.large"
          "weighted_capacity" = "1"
        },
        {
          "instance_type" = "m6i.large"
          "weighted_capacity" = "2"
        },
      ]
    }
    "name" = "spot-1"
    "post_bootstrap_user_data" = <<-EOT
    cd /tmp
    sudo yum install -y https://s3.amazonaws.com/ec2-downloads-windows/SSMAgent/latest/linux_amd64/amazon-ssm-agent.rpm
    sudo systemctl enable amazon-ssm-agent
    sudo systemctl start amazon-ssm-agent

    EOT
    "pre_bootstrap_user_data" = <<-EOT
    echo "foo"
    export FOO=bar

    EOT
    "public_ip" = true
    "use_mixed_instances_policy" = true
  }
}
self_managed_node_groups_defaults = {
  "ami_id" = "ami-02b3f04ab50ffd9f1"
  "block_device_mappings" = {
    "xvda" = {
      "device_name" = "/dev/xvda"
      "ebs" = {
        "encrypted" = true
        "iops" = "3000"
        "volume_size" = 25
        "volume_type" = "gp3"
      }
    }
  }
  "bootstrap_extra_args" = "--kubelet-extra-args '--node-labels=node.kubernetes.io/lifecycle=spot'"
  "cluster_name" = "poc-kajuuc0e"
  "create_launch_template" = true
  "desired_size" = "1"
  "force_delete" = true
  "iam_role_additional_policies" = [
    "arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore",
  ]
  "launch_template_name" = "poc-kajuuc0e"
  "launch_template_use_name_prefix" = true
  "max_size" = "5"
  "min_size" = "1"
  "name" = "poc"
  "pre_bootstrap_user_data" = <<-EOT
  export CONTAINER_RUNTIME="containerd"
  export USE_MAX_PODS=false

  EOT
  "propagate_tags" = [
    {
      "key" = "k8s.io/cluster-autoscaler/enabled"
      "propagate_at_launch" = "true"
      "value" = "true"
    },
    {
      "key" = "k8s.io/cluster-autoscaler/poc-kajuuc0e"
      "propagate_at_launch" = "true"
      "value" = "poc-kajuuc0e"
    },
  ]
  "protect_from_scale_in" = true
  "update_default_version" = true
  "use_mixed_instances_policy" = true
  "use_name_prefix" = true
}

Tho deployed in AWS i see the following ASG.
As you can see there is 0% spot
And also the spot_allocation_strategy is not set.

I could be that i am missing something? not sure but it sure seems like @shaibs3 has a point

yes @shaibs3  is correct - I've opened a PR to fix and am just going to double check the rest of the variables between the root module and the sub-module. thanks for reporting @shaibs3 !

ok looks good with the fix in

This issue has been resolved in version 18.2.3 :tada:


feat: add YS-L/csvlens

YS-L/csvlens: Command line csv viewer
$ aqua g -i YS-L/csvlens

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ csvlens -h
Usage: csvlens [OPTIONS] [FILENAME]

Arguments:
  [FILENAME]  CSV filename

Options:
  -d, --delimiter <DELIMITER>      Delimiter character (comma by default) or "auto" to auto-detect the delimiter
  -t, --tab-separated              Use tab separation. Shortcut for -d '\t'
  -i, --ignore-case                Searches ignore case. Ignored if any uppercase letters are present in the search string
      --echo-column <ECHO_COLUMN>  Print the value of this column to stdout for the selected row
      --debug                      Show stats for debugging
  -h, --help                       Print help
  -V, --version                    Print version

Thank you for your contribution!

v4.130.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.130.0


[semver:minor] Add support for toggling tests with ginkgo labels

Background
The current go-mod-ginkgo-v2 test orb can run two types of ginkgo tests, unit and acceptance. These test types are great and cover a wide range of use cases but it can cause confusion when trying to write functional and/or integration tests. This is because it is not very clear whether a functional and/or integration test should be binned into the unit or acceptance test category.
Furthermore extending the go-mod-ginkgo-v2 job to support new test types is difficult because the job identifies acceptance tests by the directory name acceptance/ and unit tests as everything else. In order for this job to add new tests a new test directory would need to be added for each test type, eg. integration/, functional/, etc, and this isn't very practical when the scope of the test is not project wide.
What
This PR adds a ginkgo-v2-labels job that leverages the ginkgo label filtering system to select tests based on label filters. This solves the problem described in background by decoupling the test's location from how the test is run and instead tightly couples how tests are run to a label. This lets integration and functional tests to be located near the code they test and makes it easier to add new test types without having to change the project's directory structure.
Summary

Adds a ginkgo-v2-labels job and command
Adds scripts to

determine the go env
install ginkgo v2
execute ginkgo with label filters

⚠ auto-merge enabled ⚠


transfers: on internal transfers check Receiver Depository exists in GL

Codecov Report

Merging #111 into master will decrease coverage by 0.08%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   51.11%   51.03%   -0.09%
==========================================
  Files          23       23
  Lines        3097     3102       +5
==========================================
  Hits         1583     1583
- Misses       1187     1192       +5
  Partials      327      327


Search profiles in admin interface by country.

Coverage decreased (-0.01%) to 91.94% when pulling fd8e2a75f319ffb1ea6346e2f8c007a06731a8cc on akatsoulas:profiles-admin into 3a6e171ecb8f91842622514f5c6428714f9387ea on mozilla:master.

r+


➕ [RESOURCE] - Programming Language "GraphQL" Complete Resources

Idea Contribution

[X] I have read all the resource request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Where do you want to add the resource? Specify the category and section.
hey @jfmartinz,
I would like to add "GraphQL" Resources in Programming Language.

Introduction to GraphQL
GraphQL Basics
Core Concepts
Defining a Schema
Advanced GraphQL Concepts
GraphQL Clients
Authentication and Authorization
Validation
Execution
Books
Courses
Youtube Tutorials

Could you please assign me this under GSSOC'24?
Provide the URLs of the resources you are adding
No response
Any alternative approaches/features
No response
Additional Context
No response
Code of Conduct

[X] I follow the CONTRIBUTING GUIDELINE of this project.

@jfmartinz Could you please take a look and assign me the issue.

@jfmartinz


Selecting SessionDuration does not work

Once I change the session duration to Yes it stops prompting for saving the credential file and my AWS CLI session keeps expiring in 30 mins.

I don't think 'yes' should be the default, as it was when I installed, I needed to switch to 'no' to make it work at all.


Add pull request guide

In order to help engineers work better together, add etiquette and advice for giving and receiving pull request feedback as well as maintaining and merging pull requests.
Fixes #35.
👉 View rendered version 👈

I've tried to include some thoughts on what I consider to be good practices, but I also recognize that other people may have different experiences than I do. Open to opinions!
Also, feel free to make changes to or merge this while I'm gone. I'm not going to be offended :)

I'll merge this now, and address my two suggestions in a follow-up PR


Table of contents for configuration docs

This is from our feedback session with the Anaconda team (packit/packit-service#1694):
The configuration documentation page https://packit.dev/docs/configuration/ has become quite long and it may be hard to find the needed info in there. Therefore, create a table of contents that would link logically grouped different parts/options of the configuration. In the right panel of the page, there are already links to the sections, but they may not be that visible and we could still create more sections to be able to find information faster.

I'd go even a few steps further and completely revamp the page.

Duplicate of https://github.com/packit/packit.dev/issues/467


Content updated

I've done the translation of the video descriptions (4c393b0) and added some text with emojis to the highlights section (6a89318). Please give some feedback about them.

Errors corrected in f8c2de2.


Include ledger specifier when writing keys in a JSON file

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

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

Comparison is base (d12acd2) 90.25% compared to head (128e768) 90.26%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #711      +/-   ##
==========================================
+ Coverage   90.25%   90.26%   +0.01%
==========================================
  Files         371      371
  Lines       29816    29817       +1
==========================================
+ Hits        26910    26914       +4
+ Misses       2906     2903       -3

Flag
Coverage Δ

unittests
90.26% <100.00%> (+0.01%)
:arrow_up:

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

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


Dagster sensors

Created 2 dagster sensors to do the following :

to trigger the move_current job once clean job completes
to trigger the concatenate job once move_current job completes

Pull request created in wrong location. Created new pull request so closing this one.


Potential Multiplayer Issues

For your awareness; I'm sorry this isn't a well tested report.
When RiskyMod v2.0.1 dropped, my friend and I hopped in multiplayer. The game was extremely laggy - getting into each other's lobbies were extremely delayed, and once we were in game, clicking to shoot practically paused the game for a few seconds (enemies could still hit and kill us). Additionally, Commando's new "Phase Lightning" was just Captain's M1 charging shot, as if it charged instantly. If there were more issues, I was not able to find them due to the unplayable lag/freezing effects. Restarting the game or disabling other mods did not fix the issues; internet connections were just fine and didn't explain lag/freeze.
If both my friend and I disabled RiskyMod then the issues went away. Unfortunately I didn't have a chance to test further or with your most recent version of the mod.
Did not have the other "Risky" mods at the time. Only other mods were EliteReworks and Actually_Faster (and their dependencies)
I'll update here next time I get a chance to further test multiplayer.

A possible explanation - I think the issue is that the standard procedure "update all mods" in Thunderstore updated RiskyMod but did not update the dependencies, so Thunderstore did not download the RiskyFixes or RiskyTweaks. I didn't know these other two mods existed until I read the RiskyMod changelog.

Versions 2.0.0 to 2.0.2 had multiplayer syncing issues due to an unregistered skill, and was fixed in 2.0.3.
If the issues persist on the latest version let me know and have you and your friend upload your LogOutput.log file from r2modman -> Settings -> Locations -> Browse Profile Folder -> BepInEx/LogOutput.log, and I'll take a look at it.

had a chance to test and can confirm no more issues with multiplayer. Only changes made since initial post was updating currently installed mods.
Thanks for this mod! Psyched to keep playing it post-DLC.


Provide a more clear error message for odo login when authentication fails

[kind/Enhancement]
First of all, I'm very excited to see the new support for login and logout directly supported in odo!
When using odo login, if you enter an incorrect password, you get the following output:
challenger chose not to retry the request

It may not be clear to users that that message means login failed due to authorization issues. Could the message be clarified to be something more like what oc login displays, where it tells what went wrong and what to do to fix it?
example:
Login failed (401 Unauthorized)
Verify you have provided correct credentials.

@jankleinert , that is completely handled by openshift, we don't have control over it

@jankleinert , can you please verify this issue & close if resolved ?

/close


lets refactor the helm-index goal so we can also add a manifests-index goal to create an index of all kubernetes and openshift manifests

LTGM [merge]

[merge]


How to reduce the output size with to_netcdf?

Hello,
The options of the ds.to_netcdf method allow several netcdf modes and a control of several options (missing values, format...).
Our current options are as follows:
ds.to_netcdf(filenam,
'w',
'NETCDF4',
'netcdf4',
encoding={ vkey2:{'dtype': float32,'_FillValue':0}})
Whereas the input size is 6 GiB / variable, the output size reaches 23 GiB / variable.
What is the best strategy in the options to reduce the netcdf output size, other than changing the format (in my exercice we have float32)?
In particular, is there some interest in modifying the deprecated "missing_values" attribute?
Thank you in advance,

NetCDF and xarray support lossy compression (extremely efficient and fast with the cost of numerical precision loss) or gzip compression (without precision loss but with slower I/O - especially when reading chunks of data). You can have a look at the documentation about NetCDF I/O here.

How do I get lossy compression? I could not find it on the documentation :(

Please read the documentation

Thanks, I thought there were some methods to choose from or something like that. For future readers, scale_factor seems to be used to control compression loss.

You can also use zlib=True

You can also use zlib and complevel

Just making it clear: those would configure lossless compression of netcdf4 lib, not lossy compression.


[Removal]: Hotspot in the real location but purchased second hand and still in the deny-list

Hotspot Name
Late Opal Spider
Hotspot b58 Address
11LnLmscEmmAqjDRZFXKKAWW3yv75uf2a4sLZGLe975XFiDWMaX
Discord Handle
No response
Hotspot Manufacturer
Bobcat
Removal Reason
Hello, I purchased 5 Sensecap miners from a secondhand seller thru Alibaba. it's a real deploy. Could you please remove it from denylist? Thank you in advance. Regards
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
There's no first hand devices in my country can be purchased right now, so I bought one bobcat miner on the secondary market but it is in the deny-list.
I have already bought, and actual arrangement，But there's no way to complete the transfer.

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Unable to read the node text if it's very long

If the node text is very large, can we have an option to show ellipsis after some text and on click show the complete name in a popover. For now I have to get the short text and full text from server as name and fullname respectively. But I can only show the short text. Could you plz suggest a way to show the full text as well (like in a popover on click of the node)

I have this problem myself but have been too lazy to fix it. Now that there is another person who cares about it, let me try to come up with a fix. Thanks for trying this library :-)

Thanks for the reply. Also this library saved my life and I am more grateful to you than you'll ever know. Because other than this the only fishbone I found was with GoJS and apparently they are charging for it. So thanks a lot for trying to solve the issue. Hope this library will stay free. ( ͡° ͜ʖ ͡°)


Tests failing

https://travis-ci.org/senecajs/seneca-redis-cache/builds/127457203

Tests are failing due to use of Seneca.pin. This should be removed since it will be deprecated shortly.


C API todo list

[x] chd_precache_progress
[x] Shims for https://github.com/rtissera/libchdr/pull/78

[ ] Since we can not guarantee ABI-compatibility for chd_file* (since libc::FILE* also does not have a stable ABI) the only thing this involves is defining chd_open_core_file to be the same as chd_open_file

[ ] Verify correctness of bindings

https://github.com/SnowflakePowered/chd-rs/commit/3f2b3749e3082e52bed612bff4d842af8e369025 takes care of chd_precache_progress/chd_precache

pushed docs in https://docs.rs/chd-capi/0.1.3/chd_capi/


chore: Use async functions where reasonable

Might be missing a couple places but I wanted to be careful to avoid stuff that is used by the signal-exit callback.

Coverage increased (+0.7%) to 99.359% when pulling bf55c3c2908853ed56bdf8e39999e9600be0bdc5 on coreyfarrell:async into c18fb0a9a8eae5904298d87c62d9934243de8674 on istanbuljs:master.


Adds a bunch of new options, dox

Adds:

bundle list name filtering
bundle list caching which provides speed improvements;
skipping existing files unless zero-size or optionally md5 mismatches;

Awesome work 👍 I'm merging these changes into the feature/rewrite branch, to include them in the rewrite that I'm currently doing


[x] update guppy + move more code over to camino paths

LEC dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

LSR dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

This PR contains dependency updates. However, Depdive update review report exceeds max. characters allowed in a GitHub comment. Please, run depdive locally or check actions.

LEC dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

LSR dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

This PR contains dependency updates. However, Depdive update review report exceeds max. characters allowed in a GitHub comment. Please, run depdive locally or check actions.

:exclamation: Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR.

/land

@sunshowers :exclamation: This PR is still missing approvals, unable to queue for landing

LSR dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

LEC dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

:exclamation: Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR.

LSR dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

LEC dependency change summary:
target packages:
  M itertools 0.10.1 (direct third-party, crates.io)
    * version upgraded from 0.10.0
    * (unchanged features: default, use_alloc, use_std)
  M serde 1.0.130 (direct third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default, derive, rc, serde_derive, std)
  M serde_json 1.0.67 (direct third-party, crates.io)
    * version upgraded from 1.0.64
    * (unchanged features: default, std)
  M globset 0.4.8 (transitive third-party, crates.io)
    * version upgraded from 0.4.6
    * (unchanged features: [none])

host packages:
  M serde_derive 1.0.130 (transitive third-party, crates.io)
    * version upgraded from 1.0.127
    * (unchanged features: default)

This PR contains dependency updates. However, Depdive update review report exceeds max. characters allowed in a GitHub comment. Please, run depdive locally or check actions.

This PR contains dependency updates. However, Depdive update review report exceeds max. characters allowed in a GitHub comment. Please, run depdive locally or check actions.


There is no document formatter for 'sql'-files installed.

when I use react-monaco-editor, I want to click a button and then format the value.
The click function code is this:
handleFormat = ()=>{
const { editor } = this.state
setTimeout(function() {
editor.getAction('editor.action.formatDocument').run();
}, 300);
}
and here I set editor to state:
editorDidMount (editor, monaco) {
this.setState({
monaco,
editor
})
editor.focus()
}
The result is I got ''There is no document formatter for 'sql'-files installed.'',
So I want to ask how can I add formatter document for 'sql'  and Implement this format function?
Thank you very much!

This is a monaco question, not an issue with this repo.


Add fn_kwargs to map and filter of IterableDataset and IterableDatasetDict

Overview
I've added an argumentfn_kwargs for map and filter methods of IterableDataset and IterableDatasetDict classes.
Details
Currently, the map and filter methods of some classes related to IterableDataset do not allow specifing the arguments passed to the function. This pull request adds fn_kwargs to pass arguments to the mapping function. This allows users to preprocess data more flexibly.
Added fn_kwargs to the following classes and methods (description of the argument is also added).

class FilteredExamplesIterable
method filter of class IterableDataset
method map of class IterableDatasetDict
method filter of class IterableDatasetDict

Example of changes
Here's an example of how to use the new functionality:
from datasets import IterableDatasetDict

def preprocess_function(example, a=None, b=None):
    # do something
    return example

dataset = IterableDatasetDict(...)
dataset = dataset.map(preprocess_function, fn_kwargs={"a": 1, "b": 2})

Related Issues
This pull request is related to the following issue:
https://github.com/huggingface/datasets/issues/3444 .
Testing
I have added unit tests to test the new functionality.
In test_iterable_dataset.py

Added test_filtered_examples_iterable_with_fn_kwargs for 1.
Added test_iterable_dataset_filter for 2.
Added test_iterable_dataset_map_with_fn_kwargs. This is not a newly added feature, but was added because it was not tested.

In test_dataset_dict.py

Added _create_dummy_iterable_dataset for 3 and 4.
Added _create_dummy_iterable_dataset_dict for 3 and 4.
Added test_iterable_map for 3.
Added test_iterable_filter for 4.

Note that, there is no test for IterableDatasetDict at the current main branch. I thought about writing tests for IterableDatasetDict in a new file, but I decided to add them in the test file for DatasetDict (test_dataset_dict.py).
Checklist

[x] Format the code.
[x] Added tests.
[x] Passed tests locally.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Sorry, the local test passed because it was inadvertently testing the main branch. I am currently fixing where the test failed.

I have fixed the bug and addressed the above two points.
I have tested locally and confirmed that the test passes.

Please check the contents. @lhoestq
5715a7e64bdd2951e6705aee58d592392e1538d6

Cool ! You can run make style to fix code formatting to fix the ci

I had forgotten about it. I did it.
00248926a37c6f1387614aa388c36fdc105a59f5

Thanks for putting this together @yuukicammy ! Looking forward to using this new addition ASAP.
@lhoestq - sorry to bother you with this, but if this looks good to you, any chance we could get this merged in?
Thanks again to you both!

Yup there's just one test to remove and we can merge

Sorry for my understanding wrong! Correspondence has been addressed.  @lhoestq
ca511b7b29fdde51ffd69b58bda79220472e9e94
Thanks for your comment! @brianhill11


Update conftest.py to ignore pip installed files

When we do pip install -r requirements.txt, it creates a directory src that contains the source files downloaded from magma/mantle (this because we haven't released them on PYPI yet, so pip has to fetch from the repositories directly).
This creates an issue where pytest will discover the tests inside magma/mantle and try to run them, so we just add them to the collect_ignore variable in conftest.py. Also ignores src in .gitignore so it isn't accidentally checked into git.

Pull Request Test Coverage Report for Build 168

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.857%

Totals

Change from base Build 155:
0.0%

Covered Lines:
143

Relevant Lines:
154

💛  - Coveralls


Error occured at conclusion of long running program

Error deleting file '/home/tabcomputing/Projects/keyvolve2/lib/keyvolve2/.crystal/crystal-run-main.tmp': No such file or directory
*Errno::new<String>:Errno +293 [0]
*File::delete<String>:Nil +172 [0]
*Crystal::Command::execute<String, Array(String)>:NoReturn +658 [0]
*Crystal::Command::run_command<Array(String)>:Nil +179 [0]
*Crystal::Command::run<Array(String)>:(Nil | Bool | FileDescriptorIO+ | Crystal::Compiler::Result | Array(Crystal::ImplementationTrace) | Array(Crystal::Init::View+:Class)) +194 [0]
main +16923 [0]
__libc_start_main +240 [0]
_start +41 [0]

Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/manastech/crystal/issues

Without code to reproduce this we have no way to fix it. Could you provide it?

@trans are you still experiencing this issue?

Sorry I had forgotten about this. My code has since been re-factored and I have not experienced the issue since. So I will close this issue.


Remote SSH always disconnects after a few minutes

Without fail, my SSH session disconnects and fails to reconnect, until I manually close and re-open the connection.

Can you share the log from the "Remote - SSH" output channel, and any errors from the developer tools?
Can you explain what you do when you "close and reopen the connection"?
You could also check whether the server is crashing on the host like ps ax | grep server.sh

Let me give more details.
I'm using vscode remote to SSH into a machine on a VPN. It mostly works fine and I have no trouble with the initial connection.
Then sometimes the vscode terminal freezes, and I can't open new terminals, but I can still edit files.
Then usually within 10 minutes, I get the "attempting to reconnect..." message, which never completes, so I "dismiss" it. It then says "attempting to reconnect..." in the bottom right bubble. It never reconnects. It says "Disconnected from SSH: desktop" in the bottom left, and I click on it and select "close remote connection". Then I reopen the connection the same way I initially opened it.
I'll post the logs when I get a repro. All I can say is that the vscode-server is not running when I grep ps on the remote machine after I lose my connection. I also noticed that the "Remote - SSH" output channel is not listed after I lose the connection, and I couldn't find a file corresponding to that stream in the remote ".vscode-server" directory.

One thing in the "Log (Remote Server)" that does seem problematic (not sure this is related):
[2019-08-15 15:20:44.169] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:21:22.318] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:22:03.349] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:22:44.634] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:23:30.321] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:25:03.515] [remoteagent] [info] New EH opened, aborting shutdown
[2019-08-15 15:25:07.701] [remoteagent] [error] [File Watcher (chokidar)] terminated unexpectedly and is restarted again...
[2019-08-15 15:25:47.605] [remoteagent] [error] [File Watcher (chokidar)] failed to start after retrying for some time, giving up. Please report this as a bug report!

I also noticed that the "Remote - SSH" output channel is not listed after I lose the connection

It ends up in a local folder, but that's ok. I don't think it will be useful. It sounds like the remote extension host is crashing. The log file at ~/.vscode-server/.<hash>.log would be helpful to see. Also is this an extremely large repo, or a VM with very little RAM, or is there anything else to note about your workspace or remote?

It is a large C/Python workspace (6000 C files, 7000 Python files) running on a desktop with 12 cores and 32 GB RAM.
Looks like the file watcher is running out of heap: https://gist.github.com/bonsairobo/8d2ddd3a402c647ce464db13a91908de

I think same as https://github.com/microsoft/vscode-remote-release/issues/735. I wonder whether this would help: https://code.visualstudio.com/docs/setup/linux#_visual-studio-code-is-unable-to-watch-for-file-changes-in-this-large-workspace-error-enospc. You could also exclude some files with "files.watcherExclude". I'll find out whether we have a way to increase the heap size of that process...


task52641

iteration 1 - updates according to CATS report

:warning: Validation status: warnings

File
Status
Preview URL
Details

security-docs/SDL/MS-SDL-Crypto-Recmd.md
:warning:Warning
View
Details

security-docs/Securing-the-Future-of-AI-ML-at-MS.md
:warning:Warning
View
Details

security-docs/Solving-TLS1-Problem.md
:warning:Warning
View
Details

security-docs/MSRC/FY18-Strategy-Brief.md
:white_check_mark:Succeeded
View

security-docs/SDL/Security-Bug-Bar-sample.md
:white_check_mark:Succeeded
View

security-docs/SDL/MS-SDL-Crypto-Recmd.md

Line 0: [Warning] (Line: 5, Col: 1, Idx: 187) - (Line: 5, Col: 1, Idx: 187): While scanning a simple key, could not find expected ':'.

security-docs/Securing-the-Future-of-AI-ML-at-MS.md

Line 0: [Warning] (Line: 5, Col: 1, Idx: 215) - (Line: 5, Col: 1, Idx: 215): While scanning a simple key, could not find expected ':'.

security-docs/Solving-TLS1-Problem.md

Line 0: [Warning] (Line: 5, Col: 1, Idx: 204) - (Line: 5, Col: 1, Idx: 204): While scanning a simple key, could not find expected ':'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

security-docs/MSRC/FY18-Strategy-Brief.md
:white_check_mark:Succeeded
View

security-docs/SDL/MS-SDL-Crypto-Recmd.md
:white_check_mark:Succeeded
View

security-docs/SDL/Security-Bug-Bar-sample.md
:white_check_mark:Succeeded
View

security-docs/Securing-the-Future-of-AI-ML-at-MS.md
:white_check_mark:Succeeded
View

security-docs/Solving-TLS1-Problem.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Uncaught SyntaxError: Unexpected token  in JSON at position 0

[Enter steps to reproduce below:]

...
...

Atom Version: 1.12.6
Electron Version: 1.3.9
System: Windows 7 Pro 64bit

maybe,
http://schemastore.org/api/json/catalog.json
this json has UTF-8 BOM, so fail on parse

same here

Same

Same... Few weeks already and is damn annoying!

Indeed @ayapi there is something with the encoding of the http://schemastore.org/api/json/catalog.json.
Just tried with a local one and worked perfectly!


Syntax Error in enhance-image.sh example

Hey there,
thanks for keeping this nice plugin up-to-date. Great work! :)
I noticed a small typo during the installation of this plugin in our new pipeline. There is a syntax error in the enhance-image.sh example in the README.md:
requirements=(
-sentry-auth-ldap>=21.9.0
+'sentry-auth-ldap>=21.9.0'
# You can add other packages here, just like requirements.txt
)

The requirements must be quoted, otherwise bash complains about a syntax error.

Thank you. I've corrected it.


NotFittedError: This SGDClassifier instance is not fitted yet

Hey,
any ideas why I get this error?
clf = HungaBungaClassifier()
clf.fit(x_train, y_train)
clf.predict(x_test)
NotFittedError: This SGDClassifier instance is not fitted yet
I want to classify.
thanks :)

Hi @shelikohan
I think I need some more information...
I'm assuming you are referring to HungaBungaClassifier, which sits on top of scikit learn.
What exactly are we trying to do here, get HungaBunga summary into trains? or is it about logging all the different models?
Any chance you can provide a toy code sample?

Thanks, Martin, already fixes it myself.
The problem was I entered categort type y_train/y_test array and not integers...

Hi @shelikohan , I'm closing the issue for now, if it resurfaces feel free to reopen.


API - websocket ?

Thanks a lot for this, works flawlessly on my m2 mac with node-red.
I have not used the OpenAI API but I can imagine because they have computing power that responses are fast and a restAPI would suffice - it would be nice to have a websocket server available in addition to REST so that we can follow along with the response instead of waiting.

Hey, thanks :wave:
There is available support for SSE and tokens are streamed one-by-one - however it's available only on llama.cpp compatible models (doesn't work for instance, with gpt4all-j, yet).
This is issue is a dup of #109 where you can follow up progress, therefore I'm closing it.


[wip] fix overriding ctl and pid file paths

/lgtm !

Applied the patches manually.  Closing the PR.
Thanks


Question about dataloader item_name

I have a simple question about dataloader.
For example, item_name =  'data_syn/003759'
As I understand, this part should be changed int(item_name[5:9]) -> int(item_name[9:15])
Could you comment about this??
https://github.com/ethnhe/PVN3D/blob/3cfcbe33880ffacb37a9862067d303d27e9fcf82/pvn3d/datasets/ycb/ycb_dataset.py#L186

In the YCB_Video dataset, only real data (item_name[:8] != 'data_syn') with  int(item_name[5:9]) >= 60 use ycb_K2, all other data (synthethic data & reall data with   int(item_name[5:9]) < 60 use ycb_K1.


[spark] Fixes python tarslip security concern

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

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

Comparison is base (bb5073f) 72.08% compared to head (8f68289) 72.20%.
Report is 997 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2995      +/-   ##
============================================
+ Coverage     72.08%   72.20%   +0.11%
- Complexity     5126     7286    +2160
============================================
  Files           473      722     +249
  Lines         21970    32525   +10555
  Branches       2351     3395    +1044
============================================
+ Hits          15838    23485    +7647
- Misses         4925     7419    +2494
- Partials       1207     1621     +414

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

Hi @frankfliu @zachgk ,I am the security researcher who reported this vulnerability
can someone here validate the report on Huntr platform ? i would really appreciate it .
link to report


Update extract_frames.py

python extract_frames.py                                                     Traceback (most recent call last):
File "/home/relh/Code/AURORA/data/something/extract_frames.py", line 58, in
with open(json_file, 'r') as file:
^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'validation_2K.json'
This file doesn't exist however you have one called valid.json and changing the path here seems to fix extract_frames.py

Thank you for fixing this, Richard!


Client with number of connections to same server causes transports to error

Using websockets this doesn't exist, but when I use xhr/jsonp transport this happens when I open 10 tabs to the same page.
Node App / Socket.io fills with:
info  - handshake authorized<PHONE_NUMBER>1050415967
info  - transport end
warn  - client not handshaken client should reconnect
info  - transport end
A couple of these lines every second.  These lines quickly fill up the log files.
The client gets this trace during all this:
Uncaught TypeError: Property 'open' of object # is not a function
socket.io.0.8.5.js:1688
Socket.connect
socket.io.0.8.5.js:1688
XHRPolling.ready
socket.io.0.8.5.js:3474
I am able to reproduce this everytime by using xhr/jsonp and opening up a bunch of tabs.

By the way, this is totally expected with long polling/jsonp since browsers have a set limit on the number of http requests they can make per hostname.  Once you hit that limit timeouts are going to occur and certain tabs are going to 'lose' their connection to the socketio server.
Maybe having a short polling transport would help prevent (to some extent) these errors.  An ajax request that just checks the server for new messages every 10 or seconds and then disconnections right away when there are none.  The transport idea is not optimal, in fact it's pretty shitty, but it would prevent these errors on clients that like to open a few too many tabs.

why "open" variable is used as function and then is changed to boolean: XHRPolling .open -> onOpen -> this.open=true ?

the problem here is io.util.defer.
When self.transport.ready (in Socket.handshake function) is called, io.util.defer sets a timeout (100ms) but if another connection is done, the timeout is not clear so a new self.transport.ready call is done.
I don't know what's the best way to fix that, setting the timeout to 0 does the trick, but its really ugly.

I'm getting this same error
Uncaught TypeError: Property 'open' of object # is not a function
I'm using on Heroku.
I followed they recommendation http://devcenter.heroku.com/articles/using-socket-io-with-node-js-on-heroku
io.configure(function(){
io.set("transports", ["xhr-polling"]);
io.set("polling duration", 10);
io.set('log level', 1);
});
Localhost works very well, but on Heroku I receive this error.


Macro-provided body conflicts with body in extensions

Description
In our complex app, we have some pretty big features. Therefore, we have sometimes moved the reducer-body into a different file as an extension of the feature. This triggers the following bug:
If the body  of a Reducer is provided in an extension, the @Reducermacro does not see it and creates a conflicting empty body.
Example:
@Reducer
struct TestFeature {
    struct State: Equatable {
        var count: Int
    }

    enum Action {
        case increment, decrement
    }

// Generated by the macro:
//let body = ComposableArchitecture.EmptyReducer<State, Action>()`
}

// In a different file TestFeature+Reducer:
extension TestFeature {
    @ReducerBuilder<State, Action> // we have to explicitly add the ReducerBuilder, because the default-value from the protocol does not work with extensions. See also https://github.com/apple/swift/issues/65252
    var body: some Reducer<State, Action> {
        Reduce { state, action in
            switch action {
            case .increment:
                state.count += 1
            case .decrement:
                state.count -= 1
            }
            return .none
        }
    }
}

I am aware that a macro cannot see extensions and that there are other workarounds for splitting a large feature. Nevertheless, I think this is a source-breaking change that you should be aware of and that maybe can be mitigated.
Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
The code should compile.
Actual behavior
The following is generated by the macro:
let body = ComposableArchitecture.EmptyReducer<State, Action>()
This leads to the following error:

❌ Invalid redeclaration of 'body'

Steps to reproduce
No response
The Composable Architecture version information
1.8.0 / main
Destination operating system
iOS 17.3.1
Xcode version information
Version 15.2 (15C500b)
Swift Compiler version information
swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.56 clang-15<IP_ADDRESS>.5)
Target: arm64-apple-macosx14.0

Hi @andtie, thanks for bringing this up. I think the only way to support this is to provide a new argument to @Reducer that suppresses the synthesized requirements:
@Reduer(synthesizeRequirements: false)
struct Feature {
  …
}

We'll think about it a bit more.

Hi @andtie, we discussed this today and we have decided to leave it as-is. If you want to do something specialized like move State, Action or body to a different file, then you will just need to forgo using @Reducer and implement the details manually. That's just the reality of dealing with macros.
Sorry that this breaking change snuck into a minor release. It's very subtle and we didn't realize it at the time.
I am going to close this issue now, but feel free to open a discussion if you want to discuss further.


[style]: Redesigned Testimonials section on about us page #355

Related Issue
This PR closes issue #355

Description
Implemented Design with TailwindCSS, Vanilla CSS and Slideshow with SwiperJs react library.
Files modified

public/index.html
src/components/About/Testimonial.css
src/components/About/Testimonial.js

Screenshots
Before this PR

After this PR
Informatician.webm
Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Please review this PR
Thanks

Hi @k-deepak04. Can you please add GSSoC23 label to PR. Otherwise my PR won't count.

sure


Missing @nteract/presentational-components on fresh install

Description
Hydrogen fails after installation due to missing node package @nteract/presentational-components
Steps to Reproduce

rm -rf ~/.atom
atom
install hydrogen (2.12.0)
try to start a kernel

Versions
Hydrogen version: 2.12.0
Expected behavior
Expected to start Hydrogen.
Workaround:
add "@nteract/presentational-components":"^3.0.6", to .atom/packages/Hydrogen/package.json in the dependencies section and run cd .atom/packages/Hydrogen && apm install

I just found another workaround installing the module myself through npm:
sudo npm install --save @nteract/presentational-components
and then
npm install<EMAIL_ADDRESS>(cause I didn't have it)
Is it alright or it can cause some problems doing it this way?

Most installation methods of @nteract/presentational-components and its dependencies should be fine.
@atteggiani

I just found another workaround installing the module myself through npm:
npm install --save @nteract/presentational-components
and then
npm install<EMAIL_ADDRESS>(cause I didn't have it)
Is it alright or it can cause some problems doing it this way?

I have tried these but now I have this error: mobx-react-lite requires react with hooks support
What can I do? Since the 2.12 update hydrogen doesn't work anymore on my MacOS :/

@rastafrange I would undo whatever you have done, then do a fresh install and use the op solution not the second as it can lead to confusing results without knowledge of npm and how it works.

@wadethestealth Ok thanks. If I sum up I uninstall Hydrogen and reinstall it? what is the "op solution"? thanks

@rastafrange

Workaround:
add
"@nteract/presentational-components": "^3.0.6",

to .atom/packages/Hydrogen/package.json and run cd .atom/packages/Hydrogen && apm install

Thank you it worked!

Make sure you put "@nteract/presentational-components": "^3.0.6", in the dependencies section. Also running apm install gave my errors with husky but still seemed to fix issue.

@rastafrange

Workaround:
add
"@nteract/presentational-components": "^3.0.6",

to .atom/packages/Hydrogen/package.json and run cd .atom/packages/Hydrogen && apm install

You should see other similar entries of other nteract libraries under the dependencies section of package.json

How do I add this?
I opened the file on notepad and there are many code blocks. Didn't want to mess anything up.
Newbie here.

There is a section titled dependencies, you should see lines very similar to what you need to add @kettlecorner

There is a section titled dependencies, you should see lines very similar to what you need to add @kettlecorner

Thank you!
Ran apm install on conda prompt in the Hydrogen folder and opening atom doesn't throw an error anymore!

Hi, i have added "@nteract/presentational-components":"^3.0.6" to the code, but when i run the cd&apm install command, terminal gives me the following error:

@yinzhema This seems to be a completely unrelated issue. Can you please open a new issue explaining what steps you took to install hydrogen along with your versions?

I have opened a new issue here. Thanks!
https://github.com/nteract/hydrogen/issues/1774

Hi Guys,
I have tried to work around below Workaround but it still does not work for me. Any suggestions or things I can checked? I am a newbie in this field.
Thanks

10 days and it's still present? Fresh windows install gives this error. Workarounds are useless when I can simply open VS Code and use it's Python extension.

@yinzhema it has been fixed but not published. Unfortunately I have no control of this. @captiansafia Can we get a publish of nteract/markdown

This should be officially fixed by now. If you see this issue, please upgrade @nteract/markdown from version 4.0.0 to something greater.

@wadethestealth, In atom I don't use @nteract/markdown and not one package is showing to be in need of an upgrade. I manually forced a check. I tried to bring up a list of nteract packages https://atom.io/users/nteract and only hydrogen is listed. When you say "upgrade" do you mean a npm cmd like npm  install @nteract/markdown  or something else? It seems you advised against using the npm workaround a few days ago, so I must be off track. Could you explain for the uninitiated? Thanks

If you install hydrogen again you should not have this error, but I am referring to the npm package as our dependency so instead of using presentational-components in the work around use markdown

Uninstall + install hydrogen from atom pref did it. Thanks

I've been having many issues getting hydrogen working again and none of the myriad workarounds have fixed my problem. I had this issue on Ubuntu 19.04 and 19.10. I've done, among several other things, the following tasks in many permutations: reinstalling atom, (re)installing hydrogen using the atom GUI, (re)installing hydrogen via apm, (re)installing hydrogen directly from the .atom/packages/Hydrogen directory, adding the presentational-components line (as done above) to the package.json file, manually downgrading Hydrogen to 2.12 and 2.11, attempting the various fixes associated with getting apm/npm to find python2 (rather than python3), including modifying python-interceptor.sh and updating my apm config to use python2.7 explicitly.
The current status of the problem on my end is that I can get Hydrogen to install, but attempting to start a kernel gives me the stack trace given below.
Atom: 1.41.0 x64
Electron: 4.2.7
OS: Ubuntu 19.10
Thrown From: Hydrogen package 2.13.1
Stack Trace
Failed to activate the Hydrogen package
At Unexpected token export

/home/sseyler/.atom/packages/Hydrogen/node_modules/react-syntax-highlighter/dist/esm/styles/prism/vs.js:1
(function (exports, require, module, __filename, __dirname, process, global, Buffer) { return function (exports, require, module, __filename, __dirname) { export default {
                                                                                ^^^^^^

SyntaxError: Unexpected token export
    at new Script (vm.js:79:7)
    at NativeCompileCache.runInThisContext (/usr/share/atom/resources/app/static/<embedded>:11:146322)
    at Module.get_Module._compile (/usr/share/atom/resources/app/static/<embedded>:11:147206)
    at Object.value [as .js] (/usr/share/atom/resources/app/static/<embedded>:11:150977)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/usr/share/atom/resources/app/static/<embedded>:11:146745)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/syntax-highlighter/index.js:15:30)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/syntax-highlighter/index.js:46:3)
    at Module.get_Module._compile (/usr/share/atom/resources/app/static/<embedded>:11:147429)
    at Object.value [as .js] (/usr/share/atom/resources/app/static/<embedded>:11:150977)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/usr/share/atom/resources/app/static/<embedded>:11:146745)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/components/source.js:15:46)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/components/source.js:51:3)
    at Module.get_Module._compile (/usr/share/atom/resources/app/static/<embedded>:11:147429)
    at Object.value [as .js] (/usr/share/atom/resources/app/static/<embedded>:11:150977)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/usr/share/atom/resources/app/static/<embedded>:11:146745)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/components/input.js:16:18)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/components/input.js:52:3)
    at Module.get_Module._compile (/usr/share/atom/resources/app/static/<embedded>:11:147429)
    at Object.value [as .js] (/usr/share/atom/resources/app/static/<embedded>:11:150977)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/usr/share/atom/resources/app/static/<embedded>:11:146745)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/index.js:10:17)
    at /packages/Hydrogen/node_modules/@nteract/presentational-components/lib/index.js:23:3)
    at Module.get_Module._compile (/usr/share/atom/resources/app/static/<embedded>:11:147429)
    at Object.value [as .js] (/usr/share/atom/resources/app/static/<embedded>:11:150977)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)

Commands
     -0:02.5.0 hydrogen:run-cell-and-move-down (input.hidden-input)

Non-Core Packages
atom-beautify 0.33.4
atom-ide-ui 0.13.0
atom-monokai 0.10.9
atom-notes 1.20.0
autocomplete 0.47.0
autocomplete-python 1.16.0
busy-signal 2.0.1
character-table 0.4.3
code-annotations 3.3.1
custom-folds 1.9.5
hard-wrap 1.7.1
hey-pane 1.2.0
Hydrogen 2.13.1
hydrogen-cell-separator 0.4.1
hydrogen-launcher 1.2.2
hydrogen-python 0.0.8
ide-fortran 0.4.0
ide-python 1.5.0
intentions 1.1.5
kite 0.174.0
language-csv 1.1.2
language-cython 0.3.0
language-fortran 2.1.9
language-lammps 3.1.0
language-latex 1.2.0
language-lua 0.9.11
language-markdown 0.37.0
language-restructuredtext 1.1.2
latex 0.50.2
latex-autocomplete 1.1.1
latex-completions 0.3.6
latex-wordcount 0.6.0
linter 2.3.1
linter-annotations 1.0.4
linter-spell 0.15.0
linter-spell-latex 0.11.0
linter-ui-default 1.8.0
MagicPython 1.1.1
markdown-pdf 2.2.0
markdown-writer 2.11.9
monokai-dark 1.5.0
monokai-seti 0.9.0
open-on-bitbucket 0.5.0
platformio-ide-terminal 2.10.0
python-tools 0.6.9
set-syntax 0.4.0
seti-icons 1.5.4
seti-syntax 1.2.0
seti-ui 1.11.0
special-symbols 0.1.0
split-diff 1.6.1
sync-settings 0.8.6
tablr 1.8.3
videoplayer 1.5.1
vim-mode-plus 1.36.2
wordcount 3.1.0

@sseyler the issue you are seeing now is #1804, which ironically is another unchecked pr for this package. Sorry for your troubles once #1804 is solved though you will likely be able to use hydrogen with no further issues.


Multiple manifests+kustomize releases

helmfile version v0.119.1
I'm trying to deploy two release based on raw manifests+kustomize:
helmDefaults:
  wait: true
  timeout: 300
  recreatePods: true

releases:
  - name: coredns
    namespace: kube-system
    chart: ./coredns
    version: v1.6.6

  - name: metrics-server
    namespace: kube-system
    chart: ./metrics-server
    version: v0.3.6

Next, I'm running helmfile template and result is sometimes ok, but another time - error
It seems helmfile reuses same temporary dir for all charts:
❯ helmfile template

I0708 10:53:06.924967   16716 chartify.go:241] using requirements.yaml:
in ./helmfile.yaml: [exit status 1

COMMAND:
  kustomize -o /var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/5498348699427315495/kustomized.yaml build --load_restrictor=none --enable_alpha_plugins /var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/5498348699427315495

OUTPUT:
  Error: accumulating resources: accumulateFile "accumulating resources from '../1245175871864360600/coredns': open /private/var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/1245175871864360600/coredns: no such file or directory", loader.New "Error loading ../1245175871864360600/coredns with git: url lacks host: ../1245175871864360600/coredns, dir: evalsymlink failure on '/private/var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/1245175871864360600/coredns' : lstat /private/var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/1245175871864360600/coredns: no such file or directory, get: invalid source string: ../1245175871864360600/coredns"]

❯ helmfile template

I0708 10:53:08.069165   16908 chartify.go:241] using requirements.yaml:
in ./helmfile.yaml: [exit status 1

COMMAND:
  helm fetch ./metrics-server --untar -d /var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/7458430504350378420 --version v0.3.6

OUTPUT:
  Error: repo . not found]

❯ helmfile template
Fetching ./metrics-server

I0708 10:53:09.040729   16965 chartify.go:241] using requirements.yaml:
in ./helmfile.yaml: [command "/Users/olfway/Workspace/aws/core/cardpay-core-k8s/bin/helm" exited with non-zero status:

PATH:
  /Users/olfway/Workspace/aws/core/cardpay-core-k8s/bin/helm

ARGS:
  0: helm (4 bytes)
  1: fetch (5 bytes)
  2: ./metrics-server (16 bytes)
  3: --version (9 bytes)
  4: v0.3.6 (6 bytes)
  5: --untar (7 bytes)
  6: --untardir (10 bytes)
  7: /var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/757875816/metrics-server/v0.3.6/metrics-server (95 bytes)

ERROR:
  exit status 1

EXIT STATUS
  1

STDERR:
  Error: repo . not found

COMBINED OUTPUT:
  Error: repo . not found]

❯ helmfile template

I0708 10:53:10.315520   17016 chartify.go:241] using requirements.yaml:
in ./helmfile.yaml: [mkdir /var/folders/v2/7pkc_4x952l8rv9rfgmyxkl80000gn/T/5613871196411981847/templates: file exists]

I can confirm these are race conditions as some times it works otherwise I've observed two distinct issues:

os.Chdir seems to be clashing because it's process wide (this is evident as the output kustomization.yaml files differ like so (note that the relative paths aren't at the same level):

❯ cat /var/folders/q7/01p0xp0d7n1bmbv39bw6xlqr0000gn/T/1233419843434499643/kustomization.yaml
bases:
- ../7076805948479984078/addons/kustomize/env/staging/kube-system/chart1

❯ cat /var/folders/q7/01p0xp0d7n1bmbv39bw6xlqr0000gn/T/8995533136524226211/kustomization.yaml
bases:
- ../../../../../../../Users/ash/code/redacted/redacted/kustomize/base/kube-system/chart2

Both charts sometimes have the same temporary directory as per predictable nanosecond randomness:

Thoughts @mumoshu?

@olfway @acaire Thanks for reporting / Sorry for the long silence! This does seem like a bug in helmfile/chartify. I'll try fixing it soon.


[firebase-crashlytics] Crashes hide exception information and have no stack trace

Steps to Reproduce

Set up firebase_crashlytics
Crash the app
Go to the console

Crashes appear, but they are pretty anemic:

Logs also don't actually show up, despite      Crashlytics.instance.log(message); being called:

@anaisbetts
The issue at https://github.com/flutter/flutter/issues/30147 has been closed and moved here. Future collaboration on this issue will be done here.

A new Crashlytics plugin has now landed on master which should solve this issue.
Checkout the migration guide to get started on upgrading to the new Crashlytics plugin: https://firebase.flutter.dev/docs/migration/


How to start training？

When I change num_classes=1, decode.acc_ seg=0.0000.

I also encountered the same problem, ask two categories how to reproduce this method?


Writing experience can be further enhanced

https://www.loom.com/share/3cb7866c344e467791c29a9621d26d68

Editor design documentation is being prepared by @vysaghsekharbb. Keeping this issue on-hold _t for now. All UI cleanup issues can be taken up together.
cc: @goutham-subramanyam

@neerajdotname @AbhayVAshokan
We need to match the styles of the editor and the actual page. The editor is identical to the page.
I will provide guidelines for spacing and styling.
Thank You.

@vysaghsekharbb we've updated neetoEditor to the new design system via #364. I've removed the on-hold label on this issue. This can be taken up.

@AbhayVAshokan Please refer this screen for the spacing.
https://www.figma.com/file/Osl7GBbBk0iwVMc57366nZ/neetoKB?node-id=4046%3A23038
Preview

This looks great. I continue to type every week on substack and I think their editor is pretty awesome. So we can check their editor for inspiration.

@neerajdotname While exploring Substack, I thought this feature is interesting.
'Secret draft link'
Please take a look at the screenshot below and let me know what you think of having it implemented in KB.

@vysaghsekharbb We have preview and our previews anyone can see. I mean the preview page is not restricted by login etc. So that's like "secret draft link".


Time and Frame node – how to see their effect?

It seems that QuiltiX has support for the "Time" and "Frame" Application Nodes (see ).
However, it's not clear to me how to actually see them have any effect in the UI – is there a way to set a time, advance the time, ...?
I would expect this to cycle the color from black to white and then from the start again:

Hey!
Unfortunately we don't really have anything to support the forth dimension just yet. It wouldn't need a lot and could be a good first issue to tackle.
If anyone ever stumbles upon this issue and would like to learn a bit about Python/Qt/USD and would like to tackle this, here are the things that I imagine would need to be done:

A timeline Qt widget including start & endframe, playbar with draggable slider, play/pause/end/start buttons
Some event on the widget that sends the current time in the StageView