Confusing YAML Schema description

The structure above this sentence does not show what the sentence mentions: "jobs":
"For simpler pipelines, not all of these levels are required. For example, in a single-job build, you can omit the container for "jobs" since there are only steps. Also, many options shown here are optional and have good defaults, so your YAML definitions are unlikely to include all of them."

What is also missing here is a guideline what should go in a job. Do I run maven package and docker build in 2 steps of one job or should this go into different jobs with single steps each?
For example, if I add a "jobs" container to my azure-pipelines.yml I get the following error:
Could not queue the build because there were validation errors or warnings.
azure-pipelines.yml (Line: 13, Col: 1): 'pool' is not allowed. 'jobs' is already defined and is mutually exclusive.
azure-pipelines.yml (Line: 16, Col: 1): 'steps' is not allowed. 'jobs' is already defined and is mutually exclusive.

After a lot of trial and error I was able to get the following azure-pipelines.yml working:
trigger:
- master

jobs:
- job: build
  pool:
    vmImage: 'Ubuntu-16.04'
  steps:
  - script: |
      echo Starting the build
      env
      java -version
      ./mvnw clean package -Dmaven.test.failure.ignore=true -e -U
    displayName: 'Build with Maven'
  - script: |
      env
      echo docker build -t $(dockerId)/$(imageName) .
    displayName: Docker Build
- job: test
  dependsOn: build
  condition: succeeded()
  pool:
    vmImage: 'Ubuntu-16.04'
  steps:
  - script: |
      echo Performing tests
      env
      ls -la
    displayName: 'Running integration tests'

The next issue is how I get the ACR coordinates injected into the 2nd build step.

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

ID: 7458d5b2-1bb7-7d87-7ce6-2b0c430b0155
Version Independent ID: edf5bbd3-a3f0-5426-e985-31290f8afd05
Content: YAML schema - Azure Pipelines
Content Source: docs/pipelines/yaml-schema.md
Product: devops
GitHub Login: @erickson-doug
Microsoft Alias: macoope

Hi All, I have an angular app that creates Clouformation template. Thus it can be deployed on self-hosted aws (ubuntu) from vsts pipeline.  Thus, in release pipeline, I have included s3 upload and cloud formation update/create stack task. I achieved uploading the cloudformation templates to S3, but not able to map to CFI stack. FYI.. I am unable to map the s3 artifacts to my CFI, as it's my Self hosted agent. Please share your thoughts..? thanks.

@santusri it sounds like you need some specific help with the product. Please file a ticket on Developer Community.

@vtbassmatt, thanks for the information. Can you help to resolve this error on build pipeline with self-hosted agent. Error "No agent found in pool Ubuntu-EC2-UE1 which satisfies the specified demands:
maven
Agent.Version -gtVersion 1.91.0"

Please open an issue on Developer Community.

http://docs.microsoft.com/azure/devops/pipelines/process/runs now describes better what happens in each kind of container (stages, jobs, steps). We've also improved the schema doc with concrete examples of syntax everywhere.


feat(vmware): imporv vmware add vsan sources

Additional vmware specific filters and vsan json

:tada: This PR is included in version 2.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Unable to compile

Trying to compile it myself, I keep running across this issue:
Compilation error: too many arguments for format [-Werror=format-extra-args]
That is from line 804 in the ino file:
Serial.printf("\nError: file dont reach spiffs offset %d, to read spiffs.", spiffs_offset,HEX);
I don't understand how to handle spiffs yet with all this, so is there something I'm doing wrong? I followed the directions to that point for a Cardputer.

It is not a problem with SPIFFs itself, it's a problem with this Serial.printf(), try to run without the ",HEX"
Or just comment this line out, it is an informative serial message, nothing much special...
What program are you using to build this project?  If Arduino IDE, which version?

Ok, I just commented it out and it compiled. It has errors in the output, but it does flash and boot. I'll have to try again removing ,HEX to see if there's any difference. I'm using Arduino IDE, version 2.3.2. As far as I can tell, I have all the relevant libraries needed. Here's the rest of the errors in case it's helpful or useful for anything, or if I've made some sort of mistake.
In file included from C:\Users\3drak\Desktop\launch\Launcher\Launcher.ino:20:
c:\Users\3drak\Documents\Arduino\libraries\SdFat\src/SdFat.h:448:2: warning: #warning File not defined because __has_include(FS.h) [-Wcpp]
 #warning File not defined because __has_include(FS.h)
  ^~~~~~~
C:\Users\3drak\Desktop\launch\Launcher\Launcher.ino: In function 'void loop()':
C:\Users\3drak\Desktop\launch\Launcher\Launcher.ino:722:28: warning: unknown escape sequence: '\040'
             LNDISP.println(" -## M5Launcher ##- \n Binary is too big \n should I continue? \nEnter/M5 to continue\n\ any key to cancel ");
                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
C:\Users\3drak\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.14\cores\esp32\esp32-hal-uart.c: In function 'uartSetPins':
C:\Users\3drak\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.14\cores\esp32\esp32-hal-uart.c:153:9: warning: 'return' with no value, in function returning non-void
         return;
         ^~~~~~
C:\Users\3drak\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.14\cores\esp32\esp32-hal-uart.c:149:6: note: declared here
 bool uartSetPins(uint8_t uart_num, int8_t rxPin, int8_t txPin, int8_t ctsPin, int8_t rtsPin)
      ^~~~~~~~~~~

The other question I have is what format is the boot image? I'm assuming in the M5Launcher-image.h, the boot image is the first section, and the menu or sd card error screen is the second. I thought it was just hex but that's probably wrong.

The M5Launcher-image.h stores the boot image for Cardputer/Stickcplus2 and stickcplus 1.1... that last is different from the other, that's why it need 2 matrices..
These errors you've got, I can't reproduce them here.. this is so odd...
Maybe something with the libraries, or board version, idk...

Awesome, I was going in the right direction on it then. Could you confirm if they're in hex or another format? I tried converting an image of the right resolution to hex and ended up with a black screen. I feel like it's not centered on the screen and black is the default without an image
With the errors, I've got the board selection set to M5Stack-ATOMS3 and have the following libraries. Some are from messing with ESP32 Cam boards.

SD
Adafruit BusIO
Adafruit DMA neopixel library
Adafruit GFX Library
Adafruit NeoPixel
Adafruit ST7735 and ST7789 Librart
Adafruit TFTLCD Library
Adafruit seesaw Library
Arducam_Mega
AsyncTCP
BME69x Sensor library
DeepSleepScheduler
ESP32-Chimera-Core
ESP32_RTC_EEPROM
ESPAsyncTCP
ESPAsyncWebServer
ESPxRGB
ILI9341_t3n
IRrenote
KinkedList
LovyanGFX
M%Cardputer
M5GFX
M5Unified
NimBLE-Arduino
PNGdec
SdFat
TFT Touch Shiel v2.0
TFT-eSPI_Scroll
Tjpg_Decoder
Time
Timer
TimerExtensions
Timezone

Yes, they are in HEX, but the conversion is kinda tricky... For the Cardputer hex image I needed to use a .bmp file in the conversion
And the plus 1.1 I needed to use a .png with 16bytes configuration..

Thank you. Seems I'm having some sort of issue with it. I started with a png that had been resized to 240x135, converted it to bmp, then used BmpHexToPro to convert it to hex, but I'm still getting a black display after flashing. Still has the battery and white bar on the left side, but the rest is black where the original boot image was.


End-point for users

Get some data per user:

age of user - i.e. time since first changeset
total no of changesets
total no of notes

Based on data we have from parsing the notes and changeset discussion files, we could also add bbox of first changeset - not sure if anything else useful.

Right now, we're only processing changesets that have discussion items associated with them. This will need to be implemented separately. Closing.


[RDBMS] az mysql flexible-server create/update: Change auto_scale_iops parameter validation logic

Related command

az mysql flexible-server create
az mysql flexible-server update
Description

This pr change the validation function. When customers input both --iops and --auto_scale_iops and the --auto_scale_iops is enabled, we should tell the customer that only one parameter take effect.
Testing Guide
It has been covered in test_mysql_flexible_server_paid_iops_mgmt

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

RDBMS

@yonzhan I just modified the title in order to pass the pipeline. But I couldn't find where to manually trigger a new pipeline build. Could you guide me? Besides, what could I do to accelerate the code review? Thanks.


test: enable dependabot

Configure dependabot to keep GH actions updated

Wait, renovate does detect github-actions, see https://github.com/kubewarden/kubewarden-controller/issues/159

Could it be that dependabot is the one that handles the nice shasum # v1.2 syntax while renovate doesn't?

That's it! For example, on sigstore-rs, we have the nice shasum # v1.2 and it only has dependabot enabled.
I'm in for dealing with github-actions with dependabot, and disable renovatebot for them. For example with https://docs.renovatebot.com/configuration-options/#ignoredeps.

Renovatebot seems to do the same, but maybe one needs to do it once for it to keep with it:
https://docs.renovatebot.com/modules/manager/github-actions/#additional-information

Nice finding! Then we could configure renovatebot to use the nice syntax it provides

Closing in favor of https://github.com/kubewarden/kubewarden-controller/pull/489


fix for headers with Content-Type

@amarzavery,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@amarzavery it'd be good to start adding test cases to this tool, this would be one case I'd recommend covering, so we make sure it doesn't regress. There may be also a need to add a test case on the sway side. thanks!


I will just add in a future PR, want this up STAT

@philipermish


Remove @types/jss and update to latest jss dependencies

[x] I have followed (at least) the PR section of the contributing guide.

Closes #14040
Also
unrelated build config updates:

use --frozen-lockfile to avoid diff between repo and ci runs
update the yarn cache to that reported by the cli in the Check versions and env step

@oliviertassinari please take a look at the build errors - look unrelated but the size bump may be jss related to dep version changes.  I don't want to make the call on those.
I do believe we should try and release this as a hotfix asap.

Good/related question in https://github.com/cssinjs/jss/issues/964#issuecomment-450687840 - if we don't want the public release of mui using jss alpha release, we should close this and take a different tact, namely reverting type changes on createGenerateClasseName.d.ts and jssPreset.d.ts and withStyles.d.ts instead to be using @types/jss.


help me to protect content with shaka.js

hello ;
i wanna protect my website video and for that i'm using shaka.js , i have done the installation , but i wanna someone to help how can i implements protect content with shaka
thnks.

Also asked via the mailing list.  Anyone wanting to follow the mailing list discussion can do so here: https://groups.google.com/forum/#!msg/shaka-player-users/cf7QEauVOcQ/guHwdYCwBgAJ
Closing this as a duplicate of the same question on the mailing list.


Maneuver Node Evolved wrong KSP version support

Maneuver Node Evolved version 2.1 (not the latest version, 2.2) was improperly flagged as supporting KSP 1.2.2 in Space Dock and now doesn't seem to list KSP 1.2.1 support.
Could I get the 2.1 .ckan file changed back to KSP 1.2.1?
https://github.com/KSP-CKAN/CKAN-meta/blob/master/ManeuverNodeEvolved/ManeuverNodeEvolved-2.1.ckan

That should do it, thanks for bringing it to our attention.


fix issue: use emtpty string when targetPath not set

fix issue #295

Codecov Report

Merging #296 into develop will decrease coverage by 0.03%.
The diff coverage is 50%.

@@             Coverage Diff             @@
##           develop     #296      +/-   ##
===========================================
- Coverage     53.6%   53.57%   -0.04%
===========================================
  Files           57       57
  Lines         1705     1708       +3
  Branches       197      198       +1
===========================================
+ Hits           914      915       +1
- Misses         727      728       +1
- Partials        64       65       +1

Impacted Files
Coverage Δ

...src/main/java/com/microsoft/azure/maven/Utils.java
83.87% <50%> (-5.42%)
:arrow_down:

Continue to review full report at Codecov.

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


Rename GVFS protocol endpoint

There are two issues to consider here before renaming the endpoint:

We'll need to make sure there is a period of time that the client understands both endpoints, so that services have time to execute the rename
If services ever drop support for the old endpoint, that will render older clients unusable.

Given the second issue, and given that over time we expect to whittle down the gvfs endpoint as more functionality shows up in the native git protocol, we need to evaluate if this rename is worth doing.
A less disruptive approach that we could consider is to lock down the current protocol but preserve it, and only add any new functionality to a new, renamed endpoint.

As the author of Bitbucket Server's GVFS implementation, this is a change I'm pretty interested in (as well as any other protocol changes), because I'll need to ensure my server implementation is kept up to date. I've subscribed to this issue to monitor progress.


FEATURE: Add "recent" keyword

I use the 'recent' URL in drive, often.  It'd be super helpful to show this list in the alfred-drive-workflow, say with a 'drecent' keyword.
https://drive.google.com/drive/recent

Hi, were you ever able to get this working... came looking for the same thing and found your comment 😄


1D rendering

🚀 Feature
As originally laid out in #9 it would be nice if the viewer can move between different rendering modes - 1D, 2D, and 3D. We've made good progress on 2D and 3D and so I'm making this issue to discuss 1D rendering.
I think the goal with 1D rendering is not to become a full 1D plotting library, there are many of those, but it is more to be able to expose functionality that people may often want at the same time as they are viewing images.
For example when given a 3D volumertic image you might want to look at the a 1D z-profile through a single pixel or the mean or the max of each slice.
Or if given a 2D timeseries you might want to look at the timeseries as a mean of an ROI defined through a label as requested in #112 and #326.
I don't think napari should necessarily be providing means to do all this re-slicing etc itself, that could be the job of plugins, but it would need to provide the ability to display the 1D trace, which is something vispy can easily do.
Plotting 1D traces though could quickly explode in complexity. Would we support a new layer type called Trace(s) that could support one or many lines given as x and y coordinates either overlaid of exploded? Would we support only the ability to take our existing layers and move them to a 1D view, possibly with some processing along the way?
What are thoughts @royerloic @d-v-b @jni?

Also note #326

We could start witha  1D mode that is just a natural extension of 2D and 3D: we just show one dimension and have sliders for the rest. this can be useful when considering for example 2D kimograph data to go back to see the 1D profile, etc...
But, of course, what would be interesting would be able to link multiple canvas that display different aspect of a dataset. For example, we could draw a line and display a line profile through all layers. or we could set a plane that slices a 3D volume, etc... This is quite advanced, and it is not clear what is really needed or how to best build that functionality. So probably the safest approach is to build the features verry slowly in that respect -- hence the point above.
I think the multi-canvas issue and linking displays of different (or same dimensions) -- for example linked orthogonal 2D views (XY, XZ, YZ) etc... should probably a different issue separate from 1D rendering...
What do you  guys think?


Option to pass an existing element to the DOM Renderer

Are you open to a PR that allows us to provide an element for the DOM Renderer to use instead of creating its own?
This would help me implement reconciliation with server side rendering output using the built-in renderer via jss.createStyleSheet().

Can you explain the goal please?

@kof
I'm rendering <style> tags on the server in the document <head> so that there is no flash of unstyled content before the app is loaded.
Right now, when the client app loads up, jss is creating new stylesheets in addition to those rendered by the server.
On the client, I want to check if that stylesheet already exists in the DOM and recycle the element.

I'm rendering

What you do right now, is kinda half solved out of the box, we need to cover this anyways properly for jss users.

See #221

I think you are trying the second approach described in #221

@kof

Converting jss styles to regular css server-side, once js components, regenerate CSS from js styles. (possible now)

That's what I'm trying to do.
I will be using the contents of the style tag until the next change. I wasn't planning on trying to do any sort of hydration just yet.

Have you thought about trying the first appraoch?

But in any case I would like to support the second approach too. So in other words you need to give jss renderer the style element which is already created on the server to use it for regeneration to avoid head pollution, right?

I/we need stylesheet features which is why I am trying jss.
Right now I'm working on a proof of concept for a JS style setup to fit my use case using jss.

Yep, that's exactly what we need to do.

I/we need stylesheet features which is why I am trying jss.

wdym by "stylesheet features" ...

@kof CSS features that you can't use inline such as pseudo classes.

ok, so it was just a self contained statement, unrelated to the discussion.

@kof You suggested the (1) approach which was inlining the styles

no, you misunderstood the meaning of inlining there. I will expand on that in the original issue.

Ah, sorry!
Anyways, back to the goal. Right now this is what happens because there is nothing marrying them:

Being able to give jss the style element via createStyleSheet() would give me the ability to use the one rendered by the server.

I see 2 additional ways to avoid head pollution by style tags:

remove original style tag when the new one will be generated
render only one style tag for all styles rendered serverside

You could render all styles on the server in one style element by using https://github.com/jsstyles/jss/blob/master/docs/js-api.md#a-style-sheets-registry

@kof
I prefer to keep the render output on the server the same as what the client would render -- 1 stylesheet per component.
(1) is a possible alternative, but using the same node that jss rendered styles in on the server with toString() is the more natural approach to me.

So how would you do it?

An easy way is to pass an option to .createStyleSheet(...) and use that option within the renderer. But I already see that it's gonna be ugly when using it.

Using an id/hash created from the original styles (before they are given to jss) as the meta option in createStyleSheet(). On the client, before I create a jss stylesheet I can check before creating the style sheet if a style element with the correct hash exists.

Why would it be ugly?
styleSheet.sheet = this.jss.createStyleSheet(
  styleSheet.getRuleDefinitions(this.theme),
  { element: myElement, meta: styleSheet.name, named: false }
);

Lots of questions in this code snippet, can you show a full class example?

import { create as createJss } from 'jss';
import nested from 'jss-nested';
import camelCase from 'jss-camel-case';
import defaultUnit from 'jss-default-unit';

export function createStyleManager({
  jss = createJss().use(nested(), camelCase(), defaultUnit()),
  theme = {}
} = {}) {
  return Object.create(styleManager, {
    sheetMap: { value: new WeakMap() },
    jss: { value: jss },
    theme: { value: theme }
  });
}

const styleManager = {
  /**
   * Attaches a styleSheet object
   */
  attach(styleSheet) {
    const counter = this.sheetMap.get(styleSheet) || 0;

    if (counter === 0) {
      if (!styleSheet.hasOwnProperty('sheet') || styleSheet.sheet.options.jss !== this.jss) {
        styleSheet.sheet = this.jss.createStyleSheet(
          styleSheet.getRuleDefinitions(this.theme),
          { meta: styleSheet.name, named: false }
        );
      }
      styleSheet.sheet.attach();
    }

    this.sheetMap.set(styleSheet, counter + 1);
  },

  /**
   * Detach a styleSheet object
   *
   * @param  {Object} styleSheet - styleSheet Object
   */
  detach(styleSheet) {
    const counter = this.sheetMap.get(styleSheet) - 1;
    if (counter) {
      this.sheetMap.set(styleSheet, counter);
    } else {
      styleSheet.sheet.detach();
      this.sheetMap.delete(styleSheet);
    }
  }
};

I took some cues from your ideas in react-jss for this experiment.

Looks overcomplicated.

Having some sort of id for a style sheet, which is predictable and unique at the same time, would make it easy to automate.

maybe some hashing algorithm.

@kof That's what I was going to do, but was waiting to see if I could get an option to provide the element first.

To avoid doing right now a good solution and just support your case, I can add element option to the style sheet.

However I see a need for something much nicer.

That would be great.
I agree that something better is needed. Right now I'm trying to validate a proof of concept to solve some issues with basic inline styling that we have been having.
I would like to discuss a solution to implement better support for this natively in jss. Once I have some conclusions I'll probably engage in some of the issues you have open such as #221

added and published


C# Received message larger than max 4194304

I am using grpc for message passing and am testing a simple server and client. When my message size goes over the limit, I get this error
Status(StatusCode=InvalidArgument, Detail="Received message larger than max (4450581 vs. 4194304)")
Client
var channel = new Channel(target, ChannelCredentials.Insecure, new List {
new ChannelOption(ChannelOptions.MaxMessageLength,int.MaxValue) });
Server
Grpc.Core.Server server = new Grpc.Core.Server(new List {
new ChannelOption(ChannelOptions.MaxMessageLength,int.MaxValue) })
grpc version 1.1

The channel option name has been changed in C core recently.
C# has been fixed by https://github.com/grpc/grpc/pull/10124
Try upgrading to gRPC C# 1.2.2 which has this fix.


No trackpoints read out when position data is missing

Hi,
I'm using the tcxreader in a personal project, and it works nicely with most of my tcx-files exported from Garmin. However for this one file where positional data of the trackpoints is missing (but timestamps, heartrate data etc. still available) it only reads out very little general information (author, activity_type), but tcx.trackpoints is a completely empty list. I am assuming there's an exception built in during the parsing of the file if no positional data is available? Would it be feasible/desirable to change that, so that all available data is read in and stored, even if some parameters are missing?
I have attached the relevant tcx-file for reference:
activity_12702654569.tcx.txt
Cheers, Paul

Ok, sometimes digging into the code before raising an issue helps - I found the option of setting "only_gps" to false in the read-method. This resolves it :)

Thank you for raising this issue and using this library. Maybe I can update the readme so that it is pointed out more clearly.

Hi @alenrajsp , thanks for your great project, much appreciated. I ran into the same problem as @pepeheeren also with a similar project and second your idea of pointing this out more prominently in the docu. Thanks

I've addressed the suggestion in the latest readme.


Loopring Milestones Modifications and Additions

Added Milestones

Loopring Payment Support
Loopring NFT Support
Loopring DeFi Port Support
Loopring Trading Support

Modified name:
'Loopring Protocol 3.0 has launched',
to
'First zkRollup is live',

@adam-bro thanks for your contribution, well done
there are only needed some editorial changes and CI flow is not passing (format, lint, test), if you need my help with any of these let me know


Better syntax highlighting for directives

In Vue-syntax-highlight directives get full javascript syntax highlighting, which I think is much nicer than this one, where they are just shown as strings. Or maybe it'd be better to just take out the language definition, leaving that to vue-syntax-highlight, and only provide the snippets.

I think, this could be solved in #8

https://github.com/vuejs/vue-syntax-highlight/issues/52#issuecomment-285162761


fix(DASH): PeriodCombiner - compare labels

If available streams differ only by label, PeriodCombiner does not take it into account when looking for the best candidate. Due to that streams from newly arrived periods will create new audio tracks, as existing streams match always with firstly found stream from new period.
Issue has been mitigated by recent PeriodCombiner improvements, as label has been used for hash generation. But if hash don't match i.e. due to bandwidth change, issue would still appear.
I haven't added similar check for video streams because I'm not sure what should be the precedence of it. If any suggestions, I'm happy to add it as well!

@theodab can you review it? Thanks!


Allow access to the "actual" node a script is attached to.

Feature Request
Lets say we have a base class called Entity, subclasses of which are attached to various types of nodes in our game.
public abstract class Entity : Node { ... }

We want to attach a subclass of Entity it to a KinematicBody2D node.
Once we do this, we have (almost) no way of accessing the properties and methods of the real KinematicBody2D node from Entity outside of using the Get(), Set() and Call() methods because Entity only inherits from Node.
Because of this, I propose adding a public method or property to Node that returns a single instance of the script's underlying real node. Let's call this method GetRealNode() for now.

public class Node : Object
{
    public Node GetRealNode()
    {
        ...
    }
}

General Concept
If we have an Entity attached to a node of type Node2D, the following code will not compile because Entity has no position property.
var entity = GetNode<Entity>("Entity");
var position = entity.Position; // ==> Error

But if we can grab an instance of the entity's actual node, we can access the position and all of the functionality of the underlying Node2D.
var node = (Node2D) entity.GetRealNode();
var position = node.Position; // ==> OK

Details
Calling GetRealNode() on a user created script class should return an instance of the actual node the script is attached to. Calling GetRealNode() on a built-in, engine defined node such as Spacial should cause the node to simply return itself.
For this section, the code snippets refer to calls from within the Entity class.
The following expression equals true as both the C# script instance, and the instance returned by GetRealNode() should share the same native node instance. In terms of what the engine sees, they are the same node.
NativeInstance == GetRealNode().NativeInstance // ==> True

However, this is false, as they are not the same C# object...
this == GetRealNode() // ==> False

Example
public class KinematicEntity : Entity
{
    // Shortcut to access the underlying KinematicBody2D.
    KinematicBody2D Body => (KinematicBody2D) GetRealNode();

    public override void _Ready()
    {
        // This should print true, as the same C# instance should be returned every time.
        GD.Print(GetRealNode() == GetRealNode());

        // This should also print true.
        GD.Print(GetRealNode().GetRealNode() == GetRealNode());

        // Let's say the node this script is present on is named "Todd".
        // These should both print "Todd".
        GD.Print(Name);
        GD.Print(GetRealNode().Name);

        // If we change the script's name variable...
        Name = "Susan";

        // These should both print "Susan", as all native data (such as the name of the node) is shared in the native node instance.
        GD.Print(Name);
        GD.Print(GetRealNode().Name);

        // And vice versa...
        GetRealNode().Name = "Steve";

        // These should both print "Steve".
        GD.Print(Name);
        GD.Print(GetRealNode().Name);
    }

    public override void _PhysicsProcess(float delta)
    {
        // Make use of the underlying KinematicBody2D.
        Body.MoveAndSlide(new Vector2(0, 100));
        GD.Print(Body.GlobalPosition);
    }
}

If we do not have the above access to the underlying node instance (like we do now) we would have to do this to call MoveAndSlide() and then print the position...
Call("move_and_slide", new Vector2(0, 100));
GD.Print(Get("global_position"));

These calls lose static typing, are slower, and can incur per-frame heap allocations due to boxing or params object[] creation.

Having a robust way to access the real node type would make it possible to write more reusable, generalized code, potentially shareable between 2D and 3D, while still maintaining static typing, performance and avoiding per-frame heap allocations.

Related issue at:
#11980
https://github.com/godotengine/godot/issues/11980

It would be extremely useful since writing scripts where multiple types of nodes share common properties/methods is now a bit more difficult since you can't just automatically have the properties and methods via inheritance and now have to have a class reference in each class that you want to have the same properties/methods. Any "common base" you want means you have to do it through an interface instead of inheritance. So, it's not impossible, but does make it a little bit more difficult.
If this is a problem that can be solved it would be great if it were implemented, but in the meantime I don't personally mind since I'm more interested in seeing C# in as stable of a state as possible :)

Just wanted to raise this:
It looks like the same problem raised some time ago in GDScript, where devs also wanted several scripts of their game to inherit a common base, but as a result they could not use any derived function of the nodes they put it on, because it would break OOP and required downcasting or using get/set/call.
To me that highlighted a weird situation for a dynamically typed language in which that was technically possible, but as we introduced strongly typed inheritance it made it "conceptually" impossible.
In C# it's worse because the language itself disallows this, you can't inherit two things at the same time, so we end up with downcasting :shrug:

So when looking at this situation, from an OOP point of view it looks like a wrong design, and the way to solve it would be to move away from inheritance and compose using child nodes rather than inheriting, there are plenty other designs doing variations of that in order to escape those limitations.

Yeah, I can attest that composition is generally the way to go. Godot and Unity were obviously designed with it in mind, just in different ways! However, one point of friction in Godot, for me at least, is what I'm going to call "The Battle for the Root Node".
Composition in Godot works well in a downward direction, ie parent nodes use their direct or indirect children to control how they work. We (usually) would like to use the scene's root node as a sort of facade that provides functionality that is ultimately implemented in its children. In most cases this works great, but some nodes like KinematicBody2D, really need to be the root node of the scene to function.
As such, we can't really "have" a KinematicBody2D on an scene. That scene has to "be" a KinematicBody2D. If we attach a common script to the KinematicBody2D, one that might even facilitate composition (what I am currently working on), we lose proper access to the KinematicBody2D. The attached script completely masks the functionality of the root node.
I have seen a few other related issues and I am not in favor of allowing multiple scripts on the same node. I love the simplicity of Godot's node system and don't want to change it in such a big way.
However, I don't think attaching a script to a node should cause drastic net loss of functionality. We should keep the "script first" ideal of nodes, but we should not hide the fact that underneath that script, there is a real, in-engine node, in all it's glory, that we can use.
For every node, two instances could be possible. One for the script if it exists, and one that represents the bare node type. No more. This still preserves encapsulation as outside nodes do not immediately know the actual node type, but the script itself can ensure that its actual node is of the correct type. The script can then downcast the actual node to the type it needs to be and access it through a property. This allows a common script class to "compose" itself of its actual node.
In this way, I don't believe we would be breaking OOP, at least not too badly!

There is a feature proposal for C# 8.0 (release date soonish) which allows default implementation in interfaces.
https://www.infoq.com/articles/default-interface-methods-cs8
https://github.com/dotnet/csharplang/issues/52
So in the future, IF and only IF this feature makes it into C#8 (I have not found any confirmation that it will be included), using an inteface with default method instead of an abstract class might help, with no change to Godot.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


hyprexpo: switching workspace while hyprexpo is open crashes Hyprland

Pretty much what the title says (I'm using keyboard bindings)
hyprlandCrashReport174650.txt

you need to be more specific; it doesn't for me

Try to open hyprexpo while on ws1, and switch back and forward between 1 and 2 a couple of times. Does the job for me

Doesn't crash anymore on my side either, may just be a random thing.
As before tho, it looks quite weird (Like, say I open it while on ws1 and then switch to ws2, I can't switch back to ws1). Is this the same on your machine?

yea it generally is buggy when you switch workspaces with a keybind. I might forcefully disable it, or make it close the overlay.

both options make equally sense
throw a coin
or, if you wanna do it with style, close the overlay with the same animation that would play if you had clicked on the selected workspace (which I guess requires intercepting which workspace the bind was pointing to)

Ok I think it just looks like the window from all workspaces are just being put into the same workspace

in this image I was on ws6 and tried switching to 1 2 and 3
Hyprland crashed once I tried to press on ws4 to switch to that.

if you wanna do it with style, close the overlay with the same animation that would play if you had clicked on the selected workspace (which I guess requires intercepting which workspace the bind was pointing to)

that's literally option 2

also crash this crash that but no crash report where crash report

thoughts?

crash report is in the original issue lmao

Works as almost perfectly, but the weird behavior is still visible if changing workspace again during the closing animation
https://github.com/hyprwm/hyprland-plugins/assets/84153269/711b07b4-a83f-4d8d-90de-f8cd1e1652a8

Agree, but I guess it deserved being pointed out.


refactor: rename const to expr

Issue Reference(s):
Fixes #1640
Build & Testing:

[x] I ran cargo test successfully.
[x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

/claim #1640

Summary by CodeRabbit

Refactor

Renamed instances of "const" to "expr" across various files to improve clarity and consistency.

New Features

Replaced the @const directive with @expr for enhanced handling of constant responses in GraphQL schemas.
Introduced a new Expr input type for the @expr directive, replacing the previous Const input type.

Bug Fixes

Adjusted references and logic in configuration and blueprint processing to align with the renaming from "const" to "expr".

@neo773

Drop some of the existing expr tests (Weird that they never failed after @ssddOnTop deleted all the the relevant code). These tests are old.
Files still are named as const which need to be changed to expr.


AttributeError: module 'coloredlogs' has no attribute 'CAN_USE_BOLD_FONT'

Hello, thank you for this beautiful package :)
In version 14.0 we are facing this error:
AttributeError: module 'coloredlogs' has no attribute 'CAN_USE_BOLD_FONT'

Was this removed or something?

Yes, looks like it was removed here: 044402534c2dd02ff340df2950e485ce4999386c was there a reason for the removal? This is a breaking change in our code.

Hi both, thanks for the feedback and sorry for the breaking change. A couple of thoughts on this:

The CAN_USE_BOLD_FONT variable was never intended to be used outside of coloredlogs.
The fact that it was documented can be taken to imply that it was part of the public API, however to be honest I didn't put that much thought into it - I simply tend to document all code I write.
Long after the first releases of coloredlogs were published, I picked up the habit of defining __all__ in the Python packages I publish, to avoid this ambiguity. Unfortunately it takes time to go back and retroactively add this to all my projects (I hadn't gotten to coloredlogs yet).
Release 14.0 bumped the major version number because it contains these kinds of backwards incompatible changes (this is what major version numbers are for). If this is -understandably- too big a risk, pin the major version number (this is what requirements pinning is for).

About why CAN_USE_BOLD_FONT was removed:

Previously colorama was the only way to get colors on Windows.
Starting with coloredlogs 14.0 there's a (better) alternative that doesn't have the same quirks.
Compensating for quirks in colorama was basically dealing with a "leaky abstraction" (edge case) and like most developers I'd rather have less than more edge cases in my code, so with the advent of Windows 10 native ANSI support I decided to remove the special handling of colorama.

Hi @xolox , thank you for your response. And yes, it makes sense since it is a major version change.
So to confirm that I understand correctly: CAN_USE_BOLD_FONT is not needed any more, because it will always be True?


Missing --net-alias functionality.

It'd be nice to be able to use the "--net-alias" functionality for user-defined networks.  It appears there is a new addition to the container create json:

      "NetworkingConfig": {
      "EndpointsConfig": {
          "isolated_nw" : {
              "IPAMConfig": {
                  "IPv4Address":"<IP_ADDRESS>",
                  "IPv6Address":"2001:db8:abcd::3033"
              },
              "Links":["container_1", "container_2"],
              "Aliases":["server_x", "server_y"]
          }
      }
Where aliases are DNS aliases.
See https://docs.docker.com/engine/userguide/networking/configure-dns/ for more info.

It should be available in 2.0


size of data buffers is measured as both Int and Long. Make it consistent

examles:
tupleNum in GpuFilteredRDD is long but array.length returns Int.
We don't need to store the length in a Long variable because we chop the data into smaller partitions.

Bugs caused by this took days to pinpoint but only 5 seconds to fix. Argh!


Blank S<P

If I create a new panel anywhere in the UI it instantly crashes with this error:
Error: Spider Monkey Panel v1.6.1 ({150D4953-751F-4C1B-8735-0362DDE8FA11})
GdiDrawText failed:
font argument is null
File:
Line: 65, Column: 8
Stack trace:
on_paint@:65:8
For clarification, this is on Windows and I've seen someone else post an issue with the same error except on WINE.

You are missing somehow the default font, and the default script should be checked against this...
Line 65:
    if (g_font) {gr.GdiDrawText(g_text, g_font, g_hot ? g_textcolour_hl : g_textcolour, 0, 0, ww, wh, DT_VCENTER | DT_CENTER | DT_WORDBREAK | DT_CALCRECT | DT_NOPREFIX);}
    else {fb.ShowPopupMessage('Missing default font on ' + (g_is_default_ui ? 'DUI' : 'CUI') + '.')}

Yes, you were right the default font wasn't selected. I have fixed it now and it works. Thank you.


Fix for "have_queued" without "in" with RSpec 2.8.0

Resetting queue_name in the matchers to avoid matching against the previous queue name when a subsequent "have_queued" is used without a chained "in" with RSpec 2.8.0 (works fine as is in RSpec 2.7.0)

Thanks!


v1.25.3 incompatible with rancher

Environmental Info:
K3s Version:

Node(s) CPU architecture, OS, and Version:

5.4.217-126.408.amzn2.x86_64
Cluster Configuration:

The cluster has three control planes and 3 agent/worker nodes. 3 of the nodes are running embedded etcd.
Describe the bug:

When attempting to run a helm install for rancher rancher-latest/rancher the following error listed above occurs.
Steps To Reproduce:
helm install rancher rancher-latest/rancher --namespace cattle-system --set hostname=$(hostname -f) --set bootstrapPassword=<REDACTED> --set replicas=3 --set ingress.tls.source=secret --set privateCA=true
Output from error:
Error: INSTALLATION FAILED : chart requires kubeVersion: < v1.25.0-0 which is incompatible with Kubernetes v1.25.3+k3s1

Installed K3s:
To install k3s-server for control plane:
curl -sFL https://get.k3s.io | K3S_TOKEN=<REDACTED> sh -s server --server <REDACTED> --etcd-expose-metrics=true
For agent nodes:
curl -sFL https://get.k3s.io | K3S_TOKEN=<REDACTED>  K3S_URL=<REDACTED> sh -
Expected behavior:

I expected the general output of rancher installation to occur.
Actual behavior:

``
Additional context / logs:

Thank you ! I thought this was the case, but I just wanted to confirm for my own sanity.


Create Cloudformation scripts

@anbestephen Did you mean to request this PR to this repo? I don't see the value of this for the package. Seems like a personal PR?

it was mistakenly created .... 😄


Fix printErr recursion

Fixes #194

I tested my project on this PR but there is still a crash.
GD.printErr("Hello, World!") will crash with a segmentation fault

Ouch, I am sorry, I did not check the PRs before I fixed the issue independently :-(


Replace parseUrlpkg from @jspm/generator

The main reason for using this, is to create file names using packagename@version. So we use parseUrlPkg from the @jspm/generator. Again as i mentioned in #13 to keep the loader lean. Let's see if we can do the check in the loader itself.
Reason

We are currently not bundling the loader along with it's dependencies and distributing to npm. So, that makes the users to download the whole @jspm/generator every-time they download the loader too. Give the use-case for extracting name and version, this is something we can avoid downloading.

I can re-add this?


Better regular expression attribute handling

This PR solves two reported issues regarding regular expression handling and ignores.
Fixes https://github.com/github/octocatalog-diff/issues/175. Regular expressions were not supported for attributes which explains the error that was reported. Now regular expressions are supported.
Fixes https://github.com/github/octocatalog-diff/issues/183. The "no newline at end of file" is now ignored if it appears in BOTH the "from" and "to" diffs because it's not really a difference in that case. (If it appears in one but not the other, it's still a difference, and it would need to be explicitly ignored.)
/cc @baurmatt who reported both of these issues. Care to give this branch a try with your use cases?

Thanks a lot! :)
Regarding #183: This is fixed for my example and now works as expected.
Regarding #175:
This seems to be kind of fixed as in: it isn't throwing errors anymore. But my change still isn't ignore.
Example:
mbaur@jumphost:~ $ Sources/syseleven-control/scripts/diffenvs/diffenvs -e mbaur -n server1234 --no-puppet
I, [2018-03-28T18:06:29.231143 #26699]  INFO -- : Running diffenvs for server1234 (production <=> mbaur)
  File[/etc/apache2/mods-available/mpm_prefork.conf] =>
   parameters =>
     group =>
      + test
     mode =>
      + 0666
     owner =>
      + test
*******************************************

The ignore list is configured as following:
catalog_diff_result = OctocatalogDiff::API::V1.catalog_diff(
  {
    :to_catalog   => catalog_new.path,
    :from_catalog => catalog_old.path,
    :ignore       => [
      {
        type: Regexp.new('\AFile\z'),
        attr: Regexp.new("\Aparameters\f(owner|group|mode)\z"),
      },
    ],
  }
)

attr: Regexp.new("\Aparameters\f(owner|group|mode)\z"),

Try this:
attr: Regexp.new("\\Aparameters\f(owner|group|mode)\\z"),

This may demonstrate what's going on:
irb(main):001:0> r = Regexp.new("\Akittens\z")
=> /Akittensz/
irb(main):002:0> r.match("kittens")
=> nil
irb(main):003:0> r = Regexp.new("\\Akittens\\z")
=> /\Akittens\z/
irb(main):004:0> r.match("kittens")
=> #<MatchData "kittens">

attr: Regexp.new("\Aparameters\f(owner|group|mode)\z"),

That does the trick. Thanks! :)

Testing within GitHub for a couple days, then I'll merge this.


Feature Implement GitLab Native License Check for GitLab Hosted Projects

Is your feature request related to a problem? Please describe.
The License check was improved to provide more details as to the nature of the license selected for a given project. This improvement used GitHub's License API to get those details. These improved details should also be available for GitLab hosted projects (and scored accordingly).
Describe the solution you'd like
clients/gitlabrepo/licenses.go should behave in a like manner as clients/githubrepo/licenses.go where possible applying the same semantic results as returned by the GH API and the builtin heuristics used by scorecard when an API is not available.
Describe alternatives you've considered
PR #2442 implemented an improved builtin heuristic for better identification of the license selected for a given project but does not actually open the identified license file to attest to that license selected (but rather naming conventions, which is unlike the GH API which performs such content analysis). That builtin heuristic is only used when the hosting repository does not have an License API endpoint for getting that information--and is therefore slower and potentially not as accurate. Although the builtin heuristic should work on GitLab hosted projects, use of a GitLab License API endpoint, would be a better choice.
Additional context
Add any other context or screenshots about the feature request here.

Completed as part of #3413


ZIP archive file entries with an "data descriptor structure" will confuse ZipReader

When a ZIP archive file entry has a data descriptor structure following its compressed file data, then ZipReader will falsely report the CRC and file size for this entry being zero. This in itself is more an inconvenience than an error when considering "streaming" of an ZIP archive. Note that it is still possible to obtain the decompressed file data of such a file entry by reading its EntryStream (ZipReader.OpenEntryStream()) until the end of the EntryStream.
However, calling ZipReader.MoveToNextEntry() without reading the EntryStream of such a file entry will upset the ZipReader and make it seek to some arbitrary position in the ZIP file. It will read 4 bytes at this position, expecting to find a local file header signature (i guess). Since those 4 bytes at this arbitrary file position will not be a valid signature, the ZipHeaderFactory.ReadHeader(...) method will throw a NotSupportedException telling: "Unknown header: <random number>".
I have seen a few reports about NotSupportedExceptions telling "Unknown header: <some random number>". Although i cannot be sure what caused the NotSupportedExceptions in those cases, it is certainly a possibility that they might have been caused by the problem i explain here.
What i believe ZipReader should do:
ZipReader can check for "Crc-32" and "Compressed size" fields being zero. If that is the case and this file entry should be skipped (instead of being extracted), then ZipReader could (A) check the compression mode and/or if a signature is following this file entry -- which would indicate a zero-byte. If the entry has not been identified as a zero-byte file, then (B) ZipReader can attempt decompressing the file data in memory to get to the end of the compressed data and thus reaching the optional data descriptor of this entry or the local file header of the next archive entry.
Background info:
The ".ZIP File Format Specification" contains more information with regard to data descriptor structures. Especially the following chapters are worth a read:
4.3.7 Local file header
4.3.9 Data descriptor
4.4.4 General purpose bit flag, Bit 3
4.4.7 CRC-32
4.4.8 compressed size
4.4.9 uncompressed size
Link to the ".ZIP File Format Specification": https://pkware.cachefly.net/webdocs/casestudies/APPNOTE.TXT
Also pay attention to the paragraphs 4.3.5 (a data descriptor structure may be present even if general purpose bit 3 is not set), <IP_ADDRESS> (optional data descriptor signature 0x08074b50) and <IP_ADDRESS> (data descriptor and central directory encryption).
Remarks
ZipArchive and its ZipArchive.Entries enumeration do not seem to be affected by this issue.
The ZIP archive i found to have file entries as described above is about 300MB in size. This is obviously too large for uploading it as a sample. I will provide a small ZIP file with the described file entries as soon as i managed to produce one myself ;)

Okay, i managed to produce a small ZIP archive. It can be downloaded from:
URL: http://wikisend.com/download/589558/test_data_descriptor.zip
Password: adamhathcock
The ZIP file contains two files:

"-" (the file name is just a dash)A text file containing the word "FIRST" repeated a few times
"second.txt"Text file containing the word "SECOND" repeated a few times

(I used Info-ZIP's command line utility zip.exe 3.0 to create this file, which explains why the file name of one of the file entries is just a dash...)
Look at the local file header of the first file ("-"). It has the general purpose bit 3 set, and the fields "crc-32", "compressed size" and "uncompressed size" are zeroed. As required by the general purpose bit 3 being set, this file entry has a data descriptor following its file data.
Also interesting is the local file header of the second file entry "second.txt". It has the general purpose bit 3 set too and has a data descriptor as well, but notice that only the fields "crc-32" and "compressed size" are zeroed, whereas the field "uncompressed size" is not zero (it contains the actual correct uncompressed file size for this entry). If you follow the ZIP file format specification to the letter, then this local file header is actually violating the specification. One has to assume that Info-ZIP is not the only software which could create such local file headers...
Note, that this small ZIP file will not produce a NotSupportedException as described in my report above, but rather an EndOfStreamException. I guess the arbitrary stream position the ZipReader wants to jump to after getting confused is beyond the end of the zip archive file, which would explain the different exception i observed when testing ZipReader with this small ZIP archive.
Some boring tidbits about how i created the ZIP file
There are basically two ways to create file entries with data descriptors using Info-ZIP's zip utility.
The first way is to use the "-fd" command line switch, which will enforce data descriptors and sets the general purpose bit 3 on the affected archive entries. I used this switch to add "second.txt" to the archive. However, as i explained Info-ZIP's zip utility "forgets" to set the "compressed size" field to zero. And i wanted to get an archive entry where "compressed size" is properly set to zero.
The other way is to provide the data to be comressed via stdin. In this case, Info-ZIP's zip utility will also use a data descriptor and set the general purpose bit 3 for the resulting archive entry. It will also properly zero out "crc-32", "compressed size" as well as "uncompressed size".
Hence, i used the following command line to create the small test ZIP file:
type first.txt | zip -fd -fz- test_data_descriptor.zip - second.txt

**Trying to produce ZIP files which would make it unfeasible to use streams incapable of seeking**
Do you remember the issue with storing files which are in ZIP file format (zip files, epubs, docx, etc.) uncompressed in a ZIP archive? I do :)
It made me think: What if such archive entries would use a data descriptor and have the general purpose bit 3 set? Since the local file header does not provide the "compressed size"/"uncompressed size" information anymore, the only way to read the ZIP archive correctly would require to parse the central directory first. This would in consequence mean that any implementation that needs to read ZIP archives from stream incapable of seeking would need to dump the complete stream into some temp file (or similar) first before being able to process the ZIP archive. That's certainly not what the inventors wanted, but the ZIP file format specification is actually not very clear about whether such archive entries are allowed or illegal... :(
Well, i tried to produce such a nasty ZIP file by adding the switch "-0" (store files without compression) to the command line:
type first.txt | zip -0 -fd -fz- test_data_descriptor.zip - second.txt

Fortunately, the developers of this utility already knew about this potential problem, because the zip utility responded with:
zip error: Invalid command arguments (zip -0 not supported for I/O on pipes or devices)

Phew... :)
On the other hand, i can imagine that there are plenty of ZIP implementations out there in the world that do not prevent the occurrence of such archive entries. Although i would consider an occurrence as a rare fringe case, I guess it would be safer (and hopefully easy to implement) if ZipReader would throw a meaningful exception if it encounters such archive entries (compression mode being "store" and either "compressed size" or "uncompressed size" field in the local file header being zero)...

Thanks for this info.  My brain is baby fried so I'll have to look at this a bit later.  Just wanted you to know I'm not ignoring you.

Don't worry. I am not expecting you to start rushing just because i wrote something. I am sure Github will be around for quite some time and so will the stuff i wrote... :)
However, you might want to grab and make a backup of that ZIP file from my second comment. The file hoster (Wikisend) will delete it after 90 days.

could you repost the file?

nm, made my own thanks to your helpful notes. I'm on osx, used:
cat myfile.txt | zip -fz- target.zip -

Shucks, I was hoping that would create a zip file with General purpose bit 3 set, but it looks like it doesn't...

Here the ZIP file again: test_data_descriptor.zip
Just FYI, when creating the ZIP file i also used the the command line parameter -fd which enforces usage of data descriptors. Not sure whether the ZIP tool on OSX provides this parameter, but i noticed that you didn't use it when creating your ZIP file (which could explain why your ZIP tool did not choose to use data descriptors based on whatever reasons and circumstances)


[Bug]: headless_shell takes 100% CPU on start-up (loop or race condition?) in 96 (94 is OK)

Bug description
Steps to reproduce the problem:

We run Chromium headless on RHEL-8 virtual machines.
We start Chromium from NodeJS code in a very standard way, by calling puppeteer.launch() with an empty config (all defaults). The main purpose of Chromium is to print PDF files.
The NodeJS service is started by systemd as a "simple" unit (again, pretty straightforward).
We use the EPEL RPM packages for Chromium and Driver. While we were on v. 94, everything was fine.
Once we upgraded to v. 96, headless_shell started taking 100% of one CPU core on each boot - and stays there forever even when the system is completely idle. On v. 94, the load average was close to 0 when the system was idle. This upgrade did not touch the puppeteer version, only the Chromium/Driver ones.

Inspecting the process list, it seems that two particular instances of the headless_shell get locked:

The one that is started with --type=renderer
The one that is stared without the --type option (but has --disable-background-networking and many other options) - likely the "main" process?

It does seem that these two somehow get interlocked, possibly by some race condition. This could explain the fact that occasionally (after manual restart of the whole app) they would start normally - but in more than 50% of the cases they would get interlocked at 100% of one CPU core.
A downgrade to 0.94 resolved the issue, so this is likely something in either Chromium or Chromedriver. However, since this only happens when running it headless, we decided to open the ticket with you.
Puppeteer version
2.0.0
Node.js version
10.x
npm version
6.14
What operating system are you seeing the problem on?
Linux
Relevant log output
No log output to show - two headless_shell process get stuck on the same core, consuming it on 100%.  The processes look like these two:

/usr/lib64/chromium-browser/headless_shell --disable-background-networking --enable-features=NetworkService,NetworkServiceInProcess --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=TranslateUI,BlinkGenPropertyTrees --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --headless --hide-scrollbars --mute-audio about:blank --remote-debugging-port=0 --user-data-dir=/tmp/puppeteer_dev_profile-gtsWDI

/usr/lib64/chromium-browser/headless_shell --type=renderer --headless --lang=en-US --disable-dev-shm-usage --disable-background-timer-throttling --disable-breakpad --enable-automation --force-color-profile=srgb --remote-debugging-port=0 --allow-pre-commit-input --ozone-platform=headless --field-trial-handle=2872467996602483998,1561577570800212603,131072 --enable-features=NetworkService,NetworkServiceInProcess --disable-features=BlinkGenPropertyTrees,PaintHolding,TranslateUI --lang=en-US --num-raster-threads=1 --renderer-client-id=2 --shared-files=v8_snapshot_data:100

Do we have a solution to lock Chromium version?
I got issues after auto-upgrade of Chromium a lot of times.


Write History of Longevity as a Sport

https://github.com/nopara73/LongevityWorldCup/blob/master/LongevityWorldCup.Documentation/LongevitySportHistory.md

https://github.com/nopara73/LongevityWorldCup/blob/master/LongevityWorldCup.Documentation/LongevitySportHistory.md


HADOOP-18487. Protobuf 2.5 removal part 2: stop exporting protobuf-2.5 (#6185)

backport of trunk pr #6185
How was this patch tested?
For code changes:

[X] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
11m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
13m 33s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
40m 33s

branch-3.3 passed

+1 :green_heart:
compile
20m 32s

branch-3.3 passed

+1 :green_heart:
mvnsite
27m 50s

branch-3.3 passed

+1 :green_heart:
javadoc
7m 40s

branch-3.3 passed

+1 :green_heart:
shadedclient
47m 30s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 30s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
40m 37s

the patch passed

+1 :green_heart:
compile
18m 51s

the patch passed

+1 :green_heart:
javac
18m 51s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
23m  8s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
javadoc
7m  7s

the patch passed

+1 :green_heart:
shadedclient
46m 56s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
739m 47s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
1m 52s

The patch does not generate ASF License warnings.

1040m 20s

Reason
Tests

Failed junit tests
hadoop.yarn.server.timelineservice.security.TestTimelineAuthFilterForV2

hadoop.hdfs.qjournal.server.TestJournalNodeSync

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6258/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6258

Optional Tests
dupname asflicense codespell detsecrets compile javac javadoc mvninstall mvnsite unit shadedclient xmllint shellcheck shelldocs

uname
Linux 64066f09af00 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 0e6bd7edb00b5f6849630750a5160b0ff8856dcd

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~18.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6258/1/testReport/

Max. process+thread count
3137 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6258/1/console

versions
git=2.17.1 maven=3.6.0 shellcheck=0.4.6

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

test failure is due to loss of storage, not as far as I can see this change


[Smoke] Fix test for using two SDMA engines

This updates the test to work with the changes in https://github.com/llvm/llvm-project/pull/73633

The PR has landed in trunk.


AttributeError: module 'gast' has no attribute 'Num'

`
$ python generation.py --model_dir ./seqlen256_v1.ckpt/
WARNING:tensorflow:From generation.py:6: The name tf.enable_eager_execution is deprecated. Please use tf.compat.v1.enable_eager_execution instead.
WARNING:tensorflow:From generation.py:40: The name tf.random.set_random_seed is deprecated. Please use tf.compat.v1.random.set_random_seed instead.
246534 unique words
2019-09-24 05:25:59.182876: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-09-24 05:25:59.187901: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-09-24 05:25:59.188597: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x55690b21cf30 executing computations on platform Host. Devices:
2019-09-24 05:25:59.188731: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): ,
WARNING:tensorflow:Entity <bound method TiedEmbeddingSoftmax.call of <main.TiedEmbeddingSoftmax object at 0x7f7a1c112050>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method TiedEmbeddingSoftmax.call of <main.TiedEmbeddingSoftmax object at 0x7f7a1c112050>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method Encoder.call of <transformer.Encoder object at 0x7f7a1c10a990>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method Encoder.call of <transformer.Encoder object at 0x7f7a1c10a990>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aff3f50>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aff3f50>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a21b81150>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a21b81150>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1af28650>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1af28650>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1af28510>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1af28510>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aed7f90>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aed7f90>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aed7e50>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aed7e50>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aeff5d0>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aeff5d0>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aeff610>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aeff610>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1ae8d810>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1ae8d810>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1ae8d850>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1ae8d850>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1ae9aad0>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1ae9aad0>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1ae9ab10>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1ae9ab10>>: AttributeError: module 'gast' has no attribute 'Num'
WARNING:tensorflow:Entity <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aea7d10>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aea7d10>>: AssertionError: Bad argument number for Name: 3, expecting 4
WARNING:tensorflow:Entity <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aea7d50>> could not be transformed and will be executed as-is. Please report this to the AutgoGraph team. When filing the bug, set the verbosity to 10 (on Linux, export AUTOGRAPH_VERBOSITY=10) and attach the full output. Cause: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aea7d50>>: AttributeError: module 'gast' has no attribute 'Num'
^CTraceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 524, in to_graph
return conversion.convert(entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 306, in convert
entity, program_ctx, free_nonglobal_var_names)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 229, in _convert_with_cache
entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 433, in convert_entity_to_ast
nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 624, in convert_func_to_ast
node = node_to_graph(node, context)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 657, in node_to_graph
node = converter.standard_analysis(node, context, is_initial=True)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/core/converter.py", line 354, in standard_analysis
node = qual_names.resolve(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 254, in resolve
return QnResolver().visit(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 317, in generic_visit
value = self.visit(value)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 326, in generic_visit
new_node = self.visit(old_value)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 236, in visit_Subscript
if isinstance(s.value, gast.Num):
AttributeError: module 'gast' has no attribute 'Num'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 423, in converted_call
experimental_optional_features=options.optional_features)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 528, in to_graph
entity, e.class.name, str(e)))
tensorflow.python.autograph.impl.api.ConversionError: converting <bound method Encoder.call of <transformer.Encoder object at 0x7f7a1c10a990>>: AttributeError: module 'gast' has no attribute 'Num'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 524, in to_graph
return conversion.convert(entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 306, in convert
entity, program_ctx, free_nonglobal_var_names)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 229, in convert_with_cache
entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 433, in convert_entity_to_ast
nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 624, in convert_func_to_ast
node = node_to_graph(node, context)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 667, in node_to_graph
node = converter.apply(node, context, return_statements)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/core/converter.py", line 380, in apply_
node = converter_module.transform(node, context)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/converters/return_statements.py", line 412, in transform
node = transformer.visit(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/core/converter.py", line 317, in visit
return super(Base, self).visit(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/transformer.py", line 480, in visit
result = super(Base, self).visit(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/converters/return_statements.py", line 363, in visit_FunctionDef
converted_body = self._visit_statement_block(node, node.body)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/converters/return_statements.py", line 287, in _visit_statement_block
nodes = self.visit_block(nodes, after_visit=self._postprocess_statement)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/transformer.py", line 371, in visit_block
replacement = self.visit(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/core/converter.py", line 317, in visit
return super(Base, self).visit(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/transformer.py", line 480, in visit
result = super(Base, self).visit(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/converters/return_statements.py", line 237, in visit_Return
retval=retval)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/templates.py", line 260, in replace
replacements[k] = _convert_to_ast(replacements[k])
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/templates.py", line 222, in _convert_to_ast
return gast.Name(id=n, ctx=None, annotation=None)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/gast/gast.py", line 19, in create_node
format(Name, nbparam, len(Fields))
AssertionError: Bad argument number for Name: 3, expecting 4
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 423, in converted_call
experimental_optional_features=options.optional_features)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 528, in to_graph
entity, e.class.name, str(e)))
tensorflow.python.autograph.impl.api.ConversionError: converting <bound method EncoderLayer.call of <transformer.EncoderLayer object at 0x7f7a1aea7d10>>: AssertionError: Bad argument number for Name: 3, expecting 4
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 524, in to_graph
return conversion.convert(entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 306, in convert
entity, program_ctx, free_nonglobal_var_names)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 229, in _convert_with_cache
entity, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 433, in convert_entity_to_ast
nodes, name, entity_info = convert_func_to_ast(o, program_ctx)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 624, in convert_func_to_ast
node = node_to_graph(node, context)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/conversion.py", line 657, in node_to_graph
node = converter.standard_analysis(node, context, is_initial=True)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/core/converter.py", line 354, in standard_analysis
node = qual_names.resolve(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 254, in resolve
return QnResolver().visit(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 317, in generic_visit
value = self.visit(value)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 326, in generic_visit
new_node = self.visit(old_value)
File "/home/ubuntu/.pyenv/versions/3.7.4/lib/python3.7/ast.py", line 262, in visit
return visitor(node)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/pyct/qual_names.py", line 236, in visit_Subscript
if isinstance(s.value, gast.Num):
AttributeError: module 'gast' has no attribute 'Num'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 423, in converted_call
experimental_optional_features=options.optional_features)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 528, in to_graph
entity, e.class.name, str(e)))
tensorflow.python.autograph.impl.api.ConversionError: converting <bound method MultiHeadAttention.call of <transformer.MultiHeadAttention object at 0x7f7a1aea7d50>>: AttributeError: module 'gast' has no attribute 'Num'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "generation.py", line 108, in
transformed = transformer.Encoder()(embedded, training=False)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 629, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 146, in wrapper
), args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 446, in converted_call
return _call_unconverted(f, args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 253, in _call_unconverted
return f(*args, **kwargs)
File "/home/ubuntu/salesfoce-ctrl-test/ctrl/transformer.py", line 137, in call
x = getattr(self, "layer%i" % i)(x, training, mask)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 629, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 146, in wrapper
), args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 446, in converted_call
return _call_unconverted(f, args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 253, in _call_unconverted
return f(*args, **kwargs)
File "/home/ubuntu/salesfoce-ctrl-test/ctrl/transformer.py", line 91, in call
attn_output  = self.multi_head_attention(normed, normed, normed, mask)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 629, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 146, in wrapper
), args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 446, in converted_call
return _call_unconverted(f, args, kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/autograph/impl/api.py", line 253, in _call_unconverted
return f(*args, **kwargs)
File "/home/ubuntu/salesfoce-ctrl-test/ctrl/transformer.py", line 65, in call
output = self.dense(original_size_attention)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 629, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/keras/layers/core.py", line 1036, in call
outputs = standard_ops.tensordot(inputs, self.kernel, [[rank - 1], [0]])
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/ops/math_ops.py", line 3796, in tensordot
a_reshape, a_free_dims, a_free_dims_static = _tensordot_reshape(a, a_axes)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/ops/math_ops.py", line 3747, in _tensordot_reshape
prod_free_dims = reduce_prod(free_dims)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/util/dispatch.py", line 180, in wrapper
return target(*args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/ops/math_ops.py", line 1894, in reduce_prod
name=name))
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/ops/gen_math_ops.py", line 7052, in prod
name=name)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/framework/op_def_library.py", line 788, in _apply_op_helper
op_def=op_def)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/framework/func_graph.py", line 464, in create_op
compute_device=compute_device)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/util/deprecation.py", line 507, in new_func
return func(*args, **kwargs)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 3616, in create_op
op_def=op_def)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 2025, in init
op_def, inputs, node_def.attr)
File "/home/ubuntu/.pyenv/versions/ctrlenv/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 2072, in _reconstruct_sequence_inputs
if input_arg.number_attr:
KeyboardInterrupt
(ctrlenv) ubuntu@ip-172-31-46-34:~/salesfoce-ctrl-test/ctrl$ python --version
Python 3.7.4
(ctrlenv) ubuntu@ip-172-31-46-34:~/salesfoce-ctrl-test/ctr
`

It was fixed.
https://github.com/tensorflow/tensorflow/issues/32319


Update README.md

I came across an error message in the browser after installing Bootstrap using Rails 5.1.3 and realised the jquery gem is not added by default with the newer version of Rails thus gem jquery-rails needs to be written manually in the Gemfile
(Using Google Chrome V60)

Already fixed in the next branch, closing


Addon error message when vendoring junk

I was selling junk to a field repair bot from the Turtle Wow donation shop.

Thanks, should be fixed in the latest version :)


Zed Wrapper randomly dies when set pose service is called

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
On our robot, we launch the zed wrapper with all of our other sensors. We then launch our navigation with a pose converter that calls the Set Pose service in order to get a position and orientation relative to our own odometry frame. Occasionally, when we call the Set Pose service (1 in 10-12 times), the zed wrapper dies with error code -11.
Steps to Reproduce

Launch ZED ROS 2 wrapper
Call set pose 10-20 times until it dies

Expected Result
Set pose works
Actual Result
[ERROR] [zed_wrapper-11]: process has died [pid 54022, exit code -11, cmd '/home/ros/colcon_deploy/install/zed_wrapper/lib/zed_wrapper/zed_wrapper --ros-args -r __node:=zed_node -r __ns:=/puddles/zed2i --params-file /home/ros/colcon_deploy/install/zed_wrapper/share/zed_wrapper/config/common.yaml --params-file /home/ros/colcon_deploy/install/zed_wrapper/share/zed_wrapper/config/zed2i.yaml --params-file /tmp/launch_params_77eslm20'].
ZED Camera model
ZED2i
Environment
OS: Ubuntu 20.04
CPU: ARM
GPU: NVIDIA Jetson AGX Orin
ZED SDK: 3.8
Jetpack SDK: 5.1
ROS 2 Humble

Anything else?
No response

This still has not been resolved.

Same issue on Jetson NX with ZED SDK 3.8.2 and ZED Mini (also with SDK 4.0.1).
In our case the issue seem to happen only, when running the wrapper as a composable node (with component_container_mt). Which makes me think that this might a race condition issue.

This has not been fixed yet

@Myzhar could this be looked at?

Same issue here when calling the stop_svo_rec service on my Laptop (Thinkpad T490) as well as on the Jetson Xavier AGX:
[zed_wrapper-2] [INFO] [1691742368.031871536] [zed2i.zed_node]: SVO Recording started:
[zed_wrapper-2] [INFO] [1691742368.031941371] [zed2i.zed_node]:  * Bitrate: 0
[zed_wrapper-2] [INFO] [1691742368.031966139] [zed2i.zed_node]:  * Compression: LOSSLESS
[zed_wrapper-2] [INFO] [1691742368.031984120] [zed2i.zed_node]:  * Framerate: 0
[zed_wrapper-2] [INFO] [1691742368.031998830] [zed2i.zed_node]:  * Input Transcode: FALSE
[zed_wrapper-2] [INFO] [1691742368.032015174] [zed2i.zed_node]:  * Filename: /media/andy/e44cb8f2-cfe8-4644-9e05-9b290aa899961/test_01.svo
[zed_wrapper-2] [ERROR] [1691742368.032048026] [zed2i.zed_node]: Error saving frame to SVO
[zed_wrapper-2] [ERROR] [1691742368.219456540] [zed2i.zed_node]: Error saving frame to SVO
[zed_wrapper-2] [INFO] [1691742376.345337048] [zed2i.zed_node]: ** Stop SVO Recording service called **
[zed_wrapper-2] [WARN] [1691742376.415200856] [zed2i.zed_node]: SVO Recording stopped
[ERROR] [zed_wrapper-2]: process has died [pid 7031, exit code -11, cmd '/home/andy/zhaw/ba/AutonomousSystem_ROS/install/zed_wrapper/lib/zed_wrapper/zed_wrapper --ros-args -r __node:=zed_node -r __ns:=/zed2i --params-file /home/andy/zhaw/ba/AutonomousSystem_ROS/install/zed_wrapper/share/zed_wrapper/config/common.yaml --params-file /home/andy/zhaw/ba/AutonomousSystem_ROS/install/zed_wrapper/share/zed_wrapper/config/zed2i.yaml --params-file /tmp/launch_params_cqwfebvn'].

Please update to the latest ZED SDK.
The bug should have been fixed.

@Myzhar tried it and you're right, works fine with SDK 4.0.6 and the latest version of the ROS2 wrapper. Thank you so much for your fast reply and please excuse the circumstances.


Overview of what's happening VT wise

In gitlab by @A-Lawrence on Sep 1, 2016, 10:52
Display enabled/disabled status bar for whether we're accepting Visitors or Transfers.

In gitlab by @A-Lawrence on Sep 1, 2016, 12:37
Removed ~98471 label