Have a Set dressing family with both static models and rigged models

Goal
Simplify the creation of environments.
Implementation
We need to facilitate creating an environment consisting of both static models and live rigs. The rigs are to be manipulated further by animators once an environment has been published.
Consider adding a new family, mindbender.set for Set Dressing, where an artist doing Set Dressing is referred to as Set Dresser.
Alternatively consider introducing mindbender.layout for publishing rigs as-is, to be further manipulated by animators. An environment artist could then produce both an environment, either as a model or set, and a layout for things he intends being manipulated further.
The layout family could then also be used for preparing animation scenes in general, such as placement of which characters where.

Motivation

Modified from version originally written by @CalleHalldin

The current workflow is split into two stages - sandboxing and refinement.
Sandbox
Models and camera are imported and placed. If a tree for example needs to be stump the Set Dresser will simple delete and deform to get the desired look. Assets may be represented as simple shapes, like a box or a sphere.
Refinement
For the shot in which the environment is meant to be used, we reference in the sandbox and use it as a reference for how the final Environment should look.
All assets that is needed has a final model so we simply load it though the loader with the shot camera and place them so they match the sandbox reference.
All models are cloned with the "clone (special)" tool so they can be tweaked if needed. For example the camera need to go low in this shot but its impossible without the ground intersecting with the camera so we simply push the ground down. Or we deform models to get a nicer image composition, for example move a few leafs that is in the way of the camera, deform the mountain in the background to get a specific shape for this camera angle.
All these cloned models are then added as a model named Environment for this specific shot thought the creator. Simply group them under a ROOT group and create a model Set named Environment.
We publish the camera as an alembic and and the Environment as a model for the shot.
The animator then load the camera and Environment model form inside the Shot asset.
Problem
This workflow becomes problematic when we have rigged assets that is a part of the environment and is needed to create a final image composition. We want the Set dressing artist to be able to use and place these rigged assets and be able to export these positions, ideally be able to pose the rigs if needed beyond the basic main control position.
Suggested Solution
This is not a solution since I don't know how to technically solve it.
We need a new family for Set Dressing that accepts models and controls_SET's of rigged assets.
Maybe if the pipeline finds a controls_SET as part of the instance it will export only the values of the control and any children below.
Maya's Atom
One way of doing this would be to use Maya's built in "Atom" exporter to just export the values and to export a small list what asset was used, which could be the namespaces of the controls_SET's?
Ideally when the animator loads a Set Dressing for a specific shot it would load the the static models, load the rigs used and import the values to get placement and pose.
Automatic Animation Instances
Another problem is that since we have rigs in the scene the pipeline automatically creates an Animation Set so the pipeline when we publish will also try to export the rigs as alembics.
I don't want to remove this since its useful for Animators since they are not tech savy, is there a way to get around this when we create a Set Dressing? or should the Set Dresser simply remove them before publish?

I've given some thought to this, let me know what you think of this workflow.

A shot is entered via mb.bat, such as sh010
An environment is created, via loading assets
Static models are re-published as a new combined asset, as a model
Rigs are also re-published as a combined asset, as a layout

In the Pyblish GUI, you would see both instances.
Once complete, in the Loader, there would be two new assets.
sh010
  environmentDefault
  layoutDefault

The artist interested in animating the rigs from the environment would then load layoutDefault, and optionally environmentDefault if needed.
The one problem I foresee is that if we use Layout for the placement of characters, and if we don't do this in the environment scene along with other assets, then we would be publishing a second Layout for characters.
sh010
  layoutEnvironment
  layoutCharacters
  environmentDefault

But perhaps that is ok?

I like the idea, i think it could work.
What happends if we update the rigs? Is the (reference) connection to the original asset still there after the "re-publish" or do we have to go in to each layout scene, update the rig and publish  the layout again?

If we go with a layout family, then I'd imagine versions of the rigs not to be included in the published assets. Instead, the latest version is chosen whenever the layout is loaded.
So a layout asset would merely contain this.

For each contained asset:
Name, e.g. Greta
Subset, e.g. rigDefault
Default values on the controllers

Practically, I'd imagine the layout only being used once and never needing an update. The Animator will have started working on it, and would update the asset in his scene.
But I may be wrong here, in what scenario do you see someone updating a layout?

Maybe i misunderstood your original post but this would all happen inside the setdress scene where they would create the camera and environment for animation and render team correct? If that would be the case the layoutEnviroment along side with environmentDefault would be updated many times parallel with animation. Usually the rigs we are speaking of, like a tree will usually have a simple cycle and not be touched by the animator. If the animator will use it as a part of the preformence then its the animator who will continue to tweak the place the tree. Maybe its our workflow on how we handle ambient/cycled animation on environment assets that needs to be re-worked. We usually use rigged eviroment close to camera and tweak the cycles so incase you can't see the motion clearly enough we scale the curves up or if its too much we scale them down. I'm open to suggestion on different workflows. I will have to ask the layout artist tomorrow if there is any other usage for a rigged eviroment asset besides for cycles unless the animator is suppose to use it.

Seems like there would be no real reason to have rig besides having a animation cycle (like wind blowing in the leafs of a tree) on it, one small reason would because it easier to move around and to pose but usually the ones with rigs have a cycled animation on it anyway. Perhaps we should look into replacing rigs with animation with cache model of some kind like Roy from colorbleed suggested?

Usually the rigs we are speaking of, like a tree will usually have a simple cycle and not be touched by the animator.

Ok, this is good. We haven't talked about pre-made cycles before, that's a new requirement we'll need to take into account for this design.
Could an animator bring in a tree rig and either animate or apply a cycle to it and publish?
That way, we'd stick with the normal workflow for animation, and control could still reside within the hands of whomever publishes the cycles.

Seems like there would be no real reason to have rig besides having a animation cycle

Ok, that simplifies things. Let me know what you think of the suggestion above.

The animation of the tree could perhaps be done by the animator. It very much depends on the project. Recent King project for example have mainly been a forest environment, doing Setdressing and applying the animation cycles, exaggerating the motion or reduce it, offsetting the animation on trees and plants along side of placing static assets has allmost been a fulltime job since its a rather busy eviroment with alot of assets. To keep the production as effective as possible (and less expensive) I'd rather have animators doing the hero animation. The reason why it's allmost a fulltime isn't becase it's hard it's because it's just alot of stuff. I'd rather have this to be able to be a separate department for when is needed.
How about Setdressing publishing alembics instead of rigged assets then? And like I said before when a rig that is needed for the hero preformence it's the animators responsibility for all its motion. No real need to have have a rigged assets with a animation cycles in the animation scene if the animator never touches it. That should work right?

How about Setdressing publishing alembics instead of rigged assets then?

Yes, that's entirely possible. Try that and let me know what you find.

Hi @CalleHalldin, did you try this workflow, how did it go? Anything you need implemented or should I close this issue?

Not in a sharp production. This could work as an output to animators, the problem is that you don't get shaders and would have to load some sort of animLookDev for each asset. Also there is no way for the animator to do tweaks to the layout if needed.
I've been giving this some thought  and also discussed it with Bruno. This might be a separate issue but I'll let you be the judge of that.
This is how I picture it working:
In the picture of colorbleed launcher they had layout (or setDressing as we are referring it to) in the shot tab. We need this aswell I think.
Like I was saying before we usually do a big environment from alot of separate assets and create a "master scene" based from what was created in the sandbox stage, then we use that master scene as a starting point for the "per shot" tweak and refinement. I would like to keep this workflow intact but instead of using unstable nested references I would like to just like to tell the pipeline what assets that has been used and its position/animation. I would also like to have a script that creates a "placer" rig which is simply just a main control and joint with the right group structure to speed the process of making simple placer rigs. After loading in and placing these placement rigs in the right position we make a "master scene". You will load that for each shot and make the tweaks and refinement and again only export the info to the pipeline which assets was used and what position/animation. If sh030 is very close to sh020 you can also simple load the setDress for sh020 for creating the setDress for sh030.
The animator will load shXX_setDress from the loader and the pipeline will load each assets and also it's position/animation. Rigs will contain textures so that problem is solved and we can also continue refining the rigs if we need to add cycles for some assets in the environment in setDress or if the animator needs a more advance rig.
As a default a placer rig is set to "static" which means that only 1 frame will get exported once exported as an alembic by the animator but the SetDress artist or animator can decide at any point if a rig is "deforming".
What do you think? It kinda goes back to my original though but this would make it possible to work on the Layout/setDressing parallel to animation, adding ambient animation cycles to tres or what not or just simply tweaking image composition it to final.

@tokejepsen You seemed to have pushed a commit about 5 days ago that referenced this issue? I think it might be unrelated. Is that correct?

Yeah, unrelated. Dunno why that happened.


Yarn audit

Trivy is finicky with yarn.lock, so also scan for vulnerabilities with yarn.

Tested by pinning got: "12.0.0" then running yarn audit


[Bug]: Docker Container :latest (V0.8.1?) doesn't start

Client version
0.8.1 (:latest)
Node version
Unavailable
Operating System type
Docker
Operating system version
Ubuntu
Describe the bug
Container not starting
To reproduce
v0.7.0 works, :latest (0.8.1) doesn't, see below logs
Screenshots & Logfiles
Portainer logs show this:
/usr/local/bin/docker-entrypoint.sh: exec: line 11: /usr/src/app/run.sh: not found
Additional context
No response

wrong repo

Thanks, didn't realise I was on the wrong repo.
You may want to add a link to the right repo in your Docker description?
By the way this is the same issue as https://github.com/bropat/eufy-security-ws/issues/81


[v1.2][ISSUE-306] Add some necessary comments and add a detailed guide for filter.

What changes were proposed in this pull request?
Add some necessary comments and add a detailed guide for filter.
How was this patch tested?
UT

@yao531441 , when you rename the parameters for min_len and max_len, it happened to trigger an error when loading previous save pipeline.json, and please help to update the keywords here:
https://github.com/intel/e2eAIOK/blob/main/RecDP/tests/data/import_test_pipeline.json
error shows as below


Scrolling issue in modals

By scrolling modals content, it doesn't scroll

in fact behind of the modal will scroll :)

This issue is resolved, so I'm closing it.


Callbacks and Webhooks, with examples

We get quite a few questions about callbacks and webhooks, the use cases and the differences. Add some resources to make these topics clearer.

Do I need to mention @SensibleWood to make them a collaborator before I can assign?

Grabbing as next most useful thing I can crank out a page or two for


can geotiff support newer proj releases?

Issue: latest geotiff release doesn't appear to support newer proj releases.
I am attempting to resolve what is jamming up an upgrade to proj 6.3 and I appear to be at the point of blaming the geotiff package.
$ conda create  -n geotiff python=3.8 proj=6.3.0 geotiff
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: /
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Package proj conflicts for:
proj=6.3.0
Package proj4 conflicts for:
geotiff -> proj4[version='4.9.3|6.0.0.*|>=4.9.3,<<IP_ADDRESS>a0|>=5.1.0,<<IP_ADDRESS>a0|>=5.2.0,<<IP_ADDRESS>a0|>=6.0.0,<<IP_ADDRESS>a0|>=6.1.0,<<IP_ADDRESS>a0|>=6.1.1,<<IP_ADDRESS>a0']
Note that strict channel priority may have removed packages required for satisfiability.

Thanks for your efforts on these packages!

I merged proj 6.3.0 a few hours ago. I'm preparing a migrator for it but it will probably take 2-3 days for it to re-compile everything with latest proj.

See https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/368

Done in #25


You don't need to unpack and recreate the ISO to edit the ignition

@masayag pointed me to this repo and I recognize some of the code :wink:
That said, to replace the ignition in a CoreOS ISO there's no reason to unpack and repack the entire thing.
Feel free to import and use our stream reader that will inject your new ignition into the ISO without all that extra work.
Let me know if you have questions or if this is just a side project.
Happy to help.

Hey @carbonin thanks for comment! Yeah this repo was just to try out some new PoC for the installer ;)
We just ended up with another approach (see https://github.com/openshift/installer/pull/6842) which anyhow required us to unpack/repack the iso

Ah, great! Looks good


EDU-656 Extend AMB response

Closes https://educandu.atlassian.net/browse/EDU-656

@ahelmberger, thanks for the remark that this about field should be configurable 👍 Changed.


Fix for "Error for missing 'super' call is misleading"

Fixes issue #3991

:+1:

class Base {}

class A extends Base {
    constructor() {
         class C {
             [super(), Symbol.iterator]() { return undefined; }
         }
    }
}

there is computedPropertyNames27_ES5.ts test that show error message
error TS2466: 'super' cannot be referenced in a computed property name.

Oh, in a computed property it's an error? Check with @DanielRosenwasser because that was not what I thought he said.
Also, what happens with super calls in static initializers, like this (also very silly)?
class Base {}

class A extends Base {
    constructor() {
         class C {
             static prop = super() && "hello";
         }
    }
}

Yeah, haven't though about static initializers. Will check for them too.

What about the extends clause?
class Base {}

class A extends Base {
    constructor() {
         class C extends (super(), Base) {

         }
    }
}

I guess my point is that it might be easier to check for the cases where the super call does not count (method bodies and instance property initializers). The cases you are thinking about now are the ones that are left over.

Yes, as far as I understand, it is not an error for super() to appear in a computed property.

So to clarify, super calls in a computed property of a class do count, but super calls in a static initializer or an extends clause do not count?

@ShyykoSerhiy I believe @JsonFreeman is correct, would you be open to sending us a pull request for that as well?

We also need tests covering the computed property case.

Ok. It's probably will be better to rewrite containsSuperCall to smth like this:
            function containsSuperCall(n: Node): boolean {
                if (isSuperCallExpression(n)) {
                    return true;
                }
                else if (isFunctionLike(n)) {
                    let functionBody = (<FunctionLikeDeclaration>n).body;
                    return forEachChild(n, n => {
                        if (n !== functionBody) {
                            return containsSuperCall(n);
                        }
                    });
                }
                else if (n.kind === SyntaxKind.PropertyDeclaration) {
                    if (!(n.modifiers && n.modifiers.some(modifier => modifier.kind === SyntaxKind.StaticKeyword))) {
                        //not static property declarations may not have super call inside initalizer
                        let initializer = (<PropertyDeclaration>n).initializer;
                        return forEachChild(n, n => {
                            if (n !== initializer) {
                                return containsSuperCall(n);
                            }
                        });
                    }
                }

                return forEachChild(n, containsSuperCall);
            }

So should I remove https://github.com/Microsoft/TypeScript/blob/master/src/compiler/checker.ts#L6499?
Anyway looks like https://github.com/Microsoft/TypeScript/blob/master/src/compiler/checker.ts#L6452
has some problems too:
class Base {}

class A extends Base {
    constructor() {
         class C {
             static prop = super() && "hello";
         }
    }
}

results in
error TS2335: 'super' can only be referenced in a derived class.
P.S.
And the whole computed property thing is kind of tricky.
Let's look into this code:
"use strict";
class Base {
    constructor(){
        console("Base")
    }
}
class Base1 {
    constructor(){
        console("Base1")
    }
}

class A extends Base {
    constructor() {
        class C extends Base1{
            constructor(){}
            [super()](){var z = "";}
        }
        new C();
    }
}
new A();

for ES6 as a target we'll see "Base", but for ES5 because of
__extends(A, _super);
    function A() {
        var C = (function (_super) {
            __extends(C, _super);
            function C() {
            }
            C.prototype[_super.call(this)] = function () { var z = ""; };
            return C;
        })(Base1);
        new C();
    }
    return A;

we`ll see Base1

Yeah you are right about _super shadowing the desired _super. I think it's fine for super calls to not count at all in a nested class.
The part about the function bodies is incorrect I think. Namely, super calls should not count inside default initializers in a function's parameter list, even though it is not in the body.

There's also decorators on class members, and decorators on the class itself. It seems weird to me to count super calls in those as well.


Fixed #26344 -- EmailMessage: don't ignore alternatives when it has attachments

https://code.djangoproject.com/ticket/26344

Could you add a comment for the if statement and a docstring for the test that explains the expected behavior?

@timgraham done

Hi, did you accidentally delete the branch or did I discourage you?

Merged in af35c69a3c595ff77188ff0d7dac6ab747484ac3, thanks!


Passing argument not working?

First off, I do not know if this belongs here in this module, but I'll start here. I'm experimenting with the following code:
var convert = require('koa-convert');
var genbind = require('generator-bind');

function* myGen(i) {
    console.log(this.x, i);
}
myGen = genbind({x:666}, myGen);
convert(myGen)(9); // <--- this outputs 666 undefined, was expecting 666 9

I can't seem to pass an argument to the generator function. If this isn't to do with koa-convert then I'm guessing generator-bind is at fault.

I don't think you need generator-bind. e.g.
var convert = require('koa-convert');

function* myGen(next) {
  console.log(this.x);
}

var convertedGen = convert(myGen);

// modern middleware has first argument as context, and second argument as `next` function
convertedGen({x: 666}, function next() {  });


Weighted Polar Chart

Summary
I have a situation where I want to use Polar Charts, but I have noticed that the pies in the polar charts are equally added, which means they have same angle. I need to change this angle based on the item weight. I assume the summation of all weights must be 100, or 360, etc
Intended Use Case
In nested models where for example a tree node and a weight for all its children and the children have weights also, I would be able to present the parent and its (weighted) children together using polar chart.

@JawadJaber Hi, I have a similar usecase for my current project. Did you find any solution?


Notebooks 2.0 // Backend // Reject invalid requests

Reject requests that pass names/namespaces with nonvalid Kubernetes resource references:

E.g. non-ASCII, too long >255
This should be for BOTH the URL parameters and the JSON payloads of POST

Hi @ederign
I would like to work on this issue. Could you please assign it to me?

/assign @Yael-F


fix: update version constant file for optimize-search-sorted

This change is

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

Project coverage is 70.15%. Comparing base (a2d4fca) to head (97d67eb).

Additional details and impacted files
@@                       Coverage Diff                        @@
##           meship/test/fix_l1_handler_test    #1638   +/-   ##
================================================================
  Coverage                            70.15%   70.15%
================================================================
  Files                                   59       59
  Lines                                 8331     8331
  Branches                              8331     8331
================================================================
  Hits                                  5845     5845
  Misses                                2012     2012
  Partials                               474      474

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


Add MVSNeRF and GNeRF

Pull requests instruction

Put the bibtex in awesome-NeRF/citations/<YOUR-PROJECT>.txt.
Modify the README and follow the format TITLE, AUTHOR, CONFERENCE YEAR | OPTIONAL LINK | BIBTEX. For example:

NeRF: Representing Scenes as Neural Radiance Fields for View Synthesis, Mildenhall et al., ECCV 2020 | github  | bibtex

Done, thank you @apchenstu :)


Add Bitbucket App Password rule

Looks good; thank you @gemesa!
I will fix the Docker CI job soon — that failure is unrelated to your change.


Add build recipe to actually compile singularity

LGTM, but I'd only suggest using this installation method (building straight from git) for testing / playing with Singularity.
The preferred installation method for production environments (in my opinion) would be to download a pre-compiled released JAR from Maven (i.e. the shaded.jar links on http://search.maven.org/#search|gav|1|g%3A"com.hubspot" AND a%3A"SingularityService"). Could we add support for this too?

I would be totally in favor of adding that. I didn't even know it existed until just now.


cleanup Resouce slice related code

This change is

I guess this PR is OK to merge. We can solve the tech debate later.
Can someone approve it?


CQUI_RealHousingFromImprovements still necessary / can we share code?

CQUI_RealHousingFromImprovements calculates the housing value for each city from the improvements around it, which required work when the Maya pack was released.  The base game did this calculation without apparent issue.  CQUI_RealHousingFromImprovements is something we may not need to maintain any longer and just let the base game do its thing.
However IF we do, then we DO have similar code in 3 places (vanilla, exp1, exp2) that feels like it doesn't need to be repeated like this.  the code itself isn't the most clear, either...
For a simple example:
            local kImprovementData = GameInfo.Improvements[eImprovementType].Housing;
            if kImprovementData == 1 then    -- farms, pastures etc.

kImprovementData is the amount of housing that comes from that improvement type.  Note that except for the Maya, farms provide 0.5 housing, though it says 1 above.  This is because in the database (debugGameplay.sqllite), the housing values are all integers in the Improvements table.
For what it's worth, there doesn't seem to be a consistent flow with finding these requirements:
for example, walking through finding the data for Kampung:
Table: Improvements

Filter: IMPROVEMENT_KAMPUNG
(all other values no meaning right now)

Table: ImprovementModifiers

ImprovementType: IMPROVEMENT_KAMPUNG
ModifierId: KAMPUNG_HOUSING_WITHTECH

Table: Modifiers

ModifierId: KAMPUNG_HOUSING_WITHTECH
SubjectRequirementSetId: PLAYER_HAS_KAMPUNG_TECHNOLOGY_REQUIREMENTS

Table: RequirementSetRequirements

RequirementSetId: PLAYER_HAS_KAMPUNG_TECHNOLOGY_REQUIREMENTS
Value: REQUIRES_PLAYER_HAS_MASS_PRODUCTION_TECHNOLOGY

Table: Requirements

RequirementId: REQUIRES_PLAYER_HAS_MASS_PRODUCTION_TECHNOLOGY
RequirementType:  REQUIREMENT_PLAYER_HAS_TECHNOLOGY
ProgressWeight = 1
All other values 0

It's unclear from there exactly how from there as there's no specific consistency in picking out that technology...

The problem with the in-game GetHousingFromImprovements() function is that it rounds down the Housing. As example you can never know if you have 1.0 or 1.5 free, in both cases the UI will show 1. Thus you don't know if you need to plop 1 or 2 farms to unblock growth. This is the reason why CQUI counts it manually in the first place.
In the Improvements table there is also a field TilesRequired. It is usually 2 which corresponds to 0.5 from Farms, etc. However, to count correctly you need to be prepared that some mod may introduce let's say 2 housing from 3 improvements. It is possible. So you need to add up Housing/TilesRequired and not Housing and then at the end divide by 2.
Some Improvements gain extra housing with technologies or civics. This is always done via modifiers, so to account for that you need to get this info dynamically from the modifier definitions. To make things more confusing those amounts are always integers. You can't give 0.5 housing via a modifier.
Then there is Maya, which is another exception. Her farms give 1.5 housing (0.5 from the table, and 1 from the modifier).
I am not sure how is know, but last time I checked CQUI counted only worked tiles, but imo the tile needs to be improved to provide housing, no need to work it.
In Better Report Screen I wrote my own piece of code that calculates all of the above. Lines 291-365. You can use the code in CQUI, I am ok with that. However, you still have to code the modifier analysis because I am using RMA for that (Real Modifier Analysis), which is part of BRS.

Awesome, thank you for the info!  Very good to know.

There may be an easier way. This topic got me thinking a bit and maybe it would be enough just to calculate the correction from improvements and apply it. This way you would not have to dabble in the modifiers. Just an idea, need thorough testing.

Seems to be working.
-- 2020-06-09 Infixo: new idea for calculations - calculate only a correction and apply to the game function
local iCityMaxBuyPlotRange:number = tonumber(GlobalParameters.CITY_MAX_BUY_PLOT_RANGE);
function GetRealHousingFromImprovements(pCity:table)
	local cityX:number, cityY:number = pCity:GetX(), pCity:GetY();
	local iNumHousing:number = 0; -- we'll add data from Housing field in Improvements divided by TilesRequired which is usually 2
	-- check all plots in the city
	for _,plotIndex in ipairs(Map.GetCityPlots():GetPurchasedPlots(pCity)) do
		local pPlot:table = Map.GetPlotByIndex(plotIndex);
		if pPlot and pPlot:GetImprovementType() > -1 and not pPlot:IsImprovementPillaged() and Map.GetPlotDistance(cityX, cityY, pPlot:GetX(), pPlot:GetY()) <= iCityMaxBuyPlotRange then
			local imprInfo:table = GameInfo.Improvements[ pPlot:GetImprovementType() ];
			iNumHousing = iNumHousing + imprInfo.Housing / imprInfo.TilesRequired; -- well, we can always add 0, right?
		end
	end
	return pCity:GetGrowth():GetHousingFromImprovements() + Round(iNumHousing-math.floor(iNumHousing),1);
end

I'm not sure this code is working right - at least for the Maya.  I'm testing right now - just basic stuff to see if I needed to halve the value or not... so maybe I am missing something.
The code I have in-game:
-- at top of file
local CQUI_cityMaxBuyPlotRange :number = tonumber(GlobalParameters.CITY_MAX_BUY_PLOT_RANGE);
--
-- inside the calculate housing function:
    local cityX:number, cityY:number = pCity:GetX(), pCity:GetY();
    local iNumHousing:number = 0; -- we'll add data from Housing field in Improvements divided by TilesRequired which is usually 2
    -- check all plots in the city
    for _,plotIndex in ipairs(Map.GetCityPlots():GetPurchasedPlots(pCity)) do
        local pPlot:table = Map.GetPlotByIndex(plotIndex);
        if (pPlot ~= nil
            and pPlot:GetImprovementType() > -1
            and not pPlot:IsImprovementPillaged()
            and Map.GetPlotDistance(cityX, cityY, pPlot:GetX(), pPlot:GetY()) <= CQUI_cityMaxBuyPlotRange) then
            local imprInfo:table = GameInfo.Improvements[ pPlot:GetImprovementType() ];
            print_debug("!!!! iNumHousing: "..tostring(iNumHousing).."  imprInfo.Housing:"..tostring(imprInfo.Housing).."   TilesRequired:"..tostring(imprInfo.TilesRequired));
            iNumHousing = iNumHousing + (imprInfo.Housing / imprInfo.TilesRequired); -- well, we can always add 0, right?
        end
    end

    local baseHousingFromImprovements = pCity:GetGrowth():GetHousingFromImprovements();
    local realHousingValue = baseHousingFromImprovements + Round(iNumHousing - math.floor(iNumHousing), 1);
    print_debug("!!! BaseHousingFromImprovements: "..tostring(baseHousingFromImprovements));
    print_debug("!!! iNumHousing value: "..tostring(iNumHousing));
    print_debug("!!! Updated housing value: "..tostring(realHousingValue));

City - far enough away from other things that I shouldn't have any interference.  2 farms only at the moment.

I'd expect the housing from improvements value to be 2.  The existing CQUI code calculates it (4, i have the print statement there before the divide by 2)

Adding a Seastead, it's better-ish:

I think this incident was solved together with seasteads #42 and can be closed.

Yes indeed, closing


Discuss:  Time to go ESM only? (for NPM highlight.js package)

This is on the v12 possibilities list: #3219.  Some were pushing us to do that last year but it felt very premature at the time.  Come this May it will be one year later... and I'm thinking of perhaps a summer release for v12.
Related: #2600
To be clear: this is only discussing highlight.js NPM package, not the highlightjs/cdn-release which will continue (for now) to include our default web build (CJS monolith, CJS modules, ES modules).

Keep using v11 (for 6 months) and then "tough luck" is one potential story...
It would be fairly easy to add a web build that adds global.hljs = hljs // etc to the end of the build, allowing v12 users to (as a hack) use the v12 web/CDN build and modules if they must stay on CJS.

Opening up for discussion.. CC @highlightjs/core

Proposed changes if we go ESM only:
For the node build:

We no longer build the lib CJS folder
We rename the es folder to lib
the -ne (don't build ESM) build  flag goes away
simplify package.json to remove all ESM + CJS stuff
the es files become the actual built source, rather than merely stubs as they are now

Actually the CDN build has long worked when required on Node via the tiny hook at the bottom:
if (typeof exports === 'object' && typeof module !== 'undefined') { module.exports = hljs; }

So maybe no other changes are required?  The hack for CJS support via CDN modules would look something like:
const hljs = require("cdn-release/highlight.js")
global.hljs = hljs // required for CDN language modules
require("cdn-release/languages/matlab.js") // side effects

I don't think is so terrible as a "last resort" option for CJS people...

I'm very interested in an ESM release so I can enable tree shaking. Even though I use maybe 3 languages, I have to bundle the entire package and it's now ~80% of my initial bundle size.
Also, you can publish both ESM/CJS packages and let node resolve the file automatically
  exports": {
    ".": {
      "require": "./index.cjs", // CJS
      "import": "./index.mjs"   // ESM
    }

A more complex example: https://github.com/vuejs/core/blob/main/packages/vue/package.json#L5-L42

Not sure what your bundler is doing. You certainly don't need ESM to only bundle the grammars you use. Rollup does this just fine, just to name one.
We already publish a CJS/ESM dual package currently. This issue is regarding whether or not we drop the CJS support entirely with v12 or not.

I was not aware there's already an ESM export. I just assumed it was CJS because in order for webpack to tree shake the unused files, every file needs to be an ESM import/export rather than just having an ESM wrapper around CJS modules. If the plan is to use ESM imports everywhere rather than just the index file, it would probably solve my issue.

You should probably be importing core, not index.

Note that you cannot successfully use the core import today with TypeScript because it's published in a location that TS cannot by default resolve. Until TS 4.7 introduced support for Node's package export maps (exports in package.json), nested layouts like the one highlight.js currently uses will only resolve types in one of two ways:

for any layout starting in the root, matching classic Node import resolution
for a single type definition specified via the types key, with no additional lookup from there

What that means in practice is that a dist directory with types in it is roughly useless to end users (without a compilerOptions.paths configuration, at least) except insofar as those types are re-exported from whatever definition file is specified by the types key in package.json. As it stands, TS users who want to import from highlight.js/core must set up their tsconfig.json to resolve it manually:
{
  // ...
  "compilerOptions": {
    // ...
    "paths": {
      "highlight.js/*": ["node_modules/highlight.js/lib/*"]
    }
  }

(Leaving this here mostly for reference for other folks who end up stumbling across this!)

We already publish a CJS/ESM dual package currently

The ESM file imports a CJS file that references module.exports so effectively it's just a facade:
await<EMAIL_ADDRESS>

This file:
<EMAIL_ADDRESS>
Imports this file (scroll to the bottom to see module.exports)
<EMAIL_ADDRESS>

It's a "façade" on purpose. I did quite a bit of reading at the time and from what I could determine this is/was the recommended way to publish a dual NPM package for Node.JS.  If you make the ESM module truly stand alone then it becomes possible for someone to accidentally import the CJS library AND the ESM library (via different peer-dependencies). Then you have two different instances of the library when you only expect one.
The "facade" solves that problem.  When CJS support is dropped the facade goes with it.
If I recall correctly our web ESM build is fully standalone as it does not have to deal with this Node.js ecosystem drama.

That remains the best path today. For TS consumers, you might use the typesVersions hack to make the rest of it resolveable, but the basic interop choice is sound and much appreciated.

I get it, but it’s still pointless because:

it requires a CJS bundler (Rollup doesn’t support this out of the box, for example)
any bundler with import support has always been able to import CJS files, so adding this intermediate export default file wasn’t and still isn’t useful.

I mean, just look at it, the ESM file imports a CJS file; any external tool that supports that can also support import 'highlight/file.cjs directly.
What I’m saying is that highlight is not a dual CJS/ESM package as you stated.

Having said that, I think publishing a “true ESM” package should be fine since all bundlers support it and highlight is mainly a browser package, from what I understand. I say this because if you publish a "type: module" package meant to be used in Node you will get a bunch of requests to revert it.

The challenge is that there are multiple use cases for published ESM:

bundlers, which you're describing
direct browser access, for which there is already a build
Node itself, which is what the ESM shim is actually targeting, and where it does provide considerable value

Also, the terminology @joshgoebel is using, describing this as a dual-ESM/CJS package, comes literally straight out of the Node docs.
None of that changes the annoyance of the interop, but that’s primarily on Node (which is where I use it, as part of my website build process, not what I bundle and ship to browsers), rather than on maintainers trying to do the least-worst thing. 😩

There seems to be a lot of confusion around this, and it's understandable given the long history of ESM, but it seems that none of you actually verified that it does provide "considerable value".
I challenge you to delete the "module" field from the package.json and tell me what exactly stops working, because I'm saying it does absolutely nothing. The CJS lib/index.js file even has a exports.default property so most versions of bundlers/babel are able to correctly pick it up.

dual-ESM/CJS package, comes literally straight out of the Node docs.

The docs purely mention setting a module field but don't specify nor care about what's inside the file, because it's completely out of their scope. Using that as "an official source" is therefore meaningless.
Once again, offering an intermediate ESM file does not equate to offering a real ESM package. You may say it's ESM, but browsers can't load it, Node can't load it, Rollup can't load it… other bundlers can, and they can just as well bundle without it.

Node does load it. 😄 Not sure what you’re experiencing, but I have used this exact pattern in several other (internal) libraries at work successfully and have used it as a consumer of this library as well.
Also, I linked directly to the section of Node’s docs using exactly this verbiage and describing exactly this pattern for one solution to avoiding problems when publishing both versions of a package in parallel; they say considerably more than you suggest!

Darn, I had totally missed the exports in the package.json, that changes a couple of things I said. 😅 This still stands:

offering such dual entry point isn't useful to any tool, so it's just extra complexity (as seen in #3007 and the successive issues that it caused)

I seem to hear absolutely zero people saying "we still really need CJS"... 💯


EGERIA-31 Fix for null pointer for missing Kafka parameters

Signed-off-by: Chandana Mirashi<EMAIL_ADDRESS>I added missing license information on 2 files CODE_OF_CONDUCT.md and IsTypeOf.java otherwise egeria build fails

Actually I think git merge is being sensible and allowing it just fine. testing and will commit if it looks good - as Graham is hitting a kafka issue too (may be unrelated)

looks good. not tested kafka function but as it's new & in process consider it appropriate to now merge. Doesn't seem to conflict with #122 where the same SPDX omissions were also fixed


Only 48 bytes max of SysEx are sent out

Describe the bug
Can't seem to send out SysEx message bigger than 48 bytes including the F0 and F7 bytes.
To Reproduce
Steps to reproduce the behavior:
I use this code to sent out 146 bytes of SysEx message.
static void sendSysExAllOn() {
  auto intf = usbhMIDI.getInterfaceFromDeviceAndCable(midiDevAddr, 0);
  if (intf == nullptr)
    return;  // not connected

  const byte PatchAllOn[] = { 0xf0, 0x52, 0x00, 0x58, 0x28, 0x08, 0x61, 0x00, 0x00, 0x24, 0x00, 0x28, 0x00, 0x00, 0x32, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x04, 0x00, 0x00, 0x00, 0x00, 0x21, 0x00, 0x00, 0x00, 0x22, 0x01, 0x04, 0x00, 0x32, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x24, 0x00, 0x61, 0x00, 0x00, 0x48, 0x03, 0x78, 0x08, 0x00, 0x37, 0x10, 0x04, 0x22, 0x73, 0x03, 0x00, 0x07, 0x00, 0x40, 0x00, 0x00, 0x01, 0x02, 0x00, 0x00, 0x0c, 0x05, 0x78, 0x00, 0x1e, 0x70, 0x40, 0x60, 0x06, 0x00, 0x00, 0x00, 0x00, 0x00, 0x03, 0x00, 0x00, 0x11, 0x00, 0x00, 0x50, 0x74, 0x28, 0x79, 0x00, 0x11, 0x20, 0x00, 0x40, 0x16, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x15, 0x0f, 0x54, 0x00, 0x45, 0x53, 0x54, 0x20, 0x43, 0x54, 0x52, 0x00, 0x4c, 0x20, 0x00, 0xf7 };
  unsigned int SysExLength = sizeof(PatchAllOn);

  intf->sendSysEx(SysExLength, PatchAllOn, true);
}

Expected behavior
All 146 bytes expected to be received by the slave device.
Screenshots
These screenshots are from my Android device acting as a MIDI Monitor.
Shorter SysEx sent by RP2040 does work as seen on the 1st-3rd line. But longer SysEx suddenly cut off as seen on the 4th line onwards.

Expected Behaviour: (sent out via Windows 11 using Bome SendSX) All 146 bytes are received properly.

What is your setup like?
Arduino 2.3.2
RP2040 3.8.0
Adafruit TinyUSB 3.1.0 (3.1.4 is latest)
usb_midi_host 1.0.1
EZ_USB_MIDI_HOST 1.0.1
OS: Windows 11
Board: RP2040 Zero
Base Sketch: EZ_USB_MIDI_HOST_PIO_example
Using Android Devices as a MIDI Monitor using MIDI monitor app.
Additional context
Shorter SysEx does work fine

@VocasoGK Thank you for taking the time to submit this issue. Because your sysex message is 147 bytes long, please first
be sure to define RPPICOMIDI_TUH_MIDI_MAX_SYSEX in your test code to something at least 147 bytes or larger. If you don't define it, then the default is 128 bytes. Please read the comments in EZ_USB_MIDI_HOST_Config.h. If that fixes your issue, I will try to update the README.md file to make this more clear.
If that doesn't fix your problem, please report back and include enough of your modified sketch to duplicate the issue. It is suspicious to me that you are only getting 48 bytes, because 48 bytes is the maximum size of a MIDI message that can fit in a 64-byte (maximum length) full speed USB bulk transfer packet.

@VocasoGK, to be more clear, I will need to see all of the includes and configuration defines the sketch has before it calls sendSysExAllOn()

Changing RPPICOMIDI_TUH_MIDI_MAX_SYSEX does solve a problem I had (not mentioned in this issue post) on incoming 146 bytes SysEx messages that was divided as a 128 bytes message, and 20 bytes message. But it still does not solve the outgoing message problem.
Here is my sketch. Changes from the example sketch was just basically calling sendSysExRequestInfo() and sendSysExEditMode() when MIDI device is connected to send out short SysEx messages (works fine), and a timer to call sendSysExAllOn() periodically.
Workaround that comes to my mind currently is dividing the 146 bytes message to a smaller 48 bytes messages and send them out until it reaches the F7 stop.

I tried separating the 146 bytes into 4 chunks consisting of 48 bytes each. Then changed
  intf->sendSysEx(SysExLength, PatchAllOn, true);

into
  intf->sendSysEx(sizeof(patchChunks[0]), patchChunks[0], true);
  usbhMIDI.writeFlushAll();
  intf->sendSysEx(sizeof(patchChunks[1]), patchChunks[1], true);
  usbhMIDI.writeFlushAll();
  intf->sendSysEx(sizeof(patchChunks[2]), patchChunks[2], true);
  usbhMIDI.writeFlushAll();
  intf->sendSysEx(sizeof(patchChunks[3]), patchChunks[3], true);
  usbhMIDI.writeFlushAll();

Code above does send out SysEx, but only the first 96 bytes (patchChunks[0] and patchChunks[1]), and are very unreliable as sometimes it randomly cuts off then spits out the remaining chunks.

@VocasoGK Thanks for the info. Very sorry for the bug. I never tested long sysex messages. I am investigating now.

@VocasoGK I understand the problem now. Changing RPPICOMIDI_TUH_MIDI_MAX_SYSEX in EZ_USB_MIDI_HOST_Config.h fixes the receiver because the EX_USB_MIDI_HOST library includes file. However, the usb_midi_host library does not include EZ_USB_MIDI_HOST_Config.h, so properly defining CFG_TUH_MIDI_TX_BUFSIZE in that config file does not help. In the C/C++ projects, all you have to do is add a new definition for CFG_TUH_MIDI_TX_BUFSIZE in tusb_config.h. That option is not
practical for Arduino IDE. I need to think about it. Meanwhile, as a short-term workaround, add
#define CFG_TUH_MIDI_TX_BUFSIZE 256

after line 37 in the usb_midi_host.c library file.
Thank you for your patience.

It works now! Thank you very much! Really appreciate the quick response. I'll close the issue and let you know if I have any other issues in the future.

@VocasoGK I am glad I helped you. If you do not mind, I am going to keep this issue open until I come up with a fix that doesn't require changing library files.

I just pushed code that addresses this issue for this library. There is now a function you can call in your sketch tuh_midih_define_limits() that lets you define buffer sizes at runtime. See the Arduino examples for details. However,
now I have to make EZ_USBMIDI_HOST library use this function. That is https://github.com/rppicomidi/EZ_USB_MIDI_HOST/issues/1. I am closing this issue for now.

Ugh. The good news is that I didn't push new release tags for this. The bad news is I botched the last commit to fix this issue. New one coming soon. I will wait until I push release tags for Arduino before I close this one.

This should be fixed with version 1.1.1. https://github.com/rppicomidi/EZ_USB_MIDI_HOST/issues/1 is still open.


Endpoint: POST /api/v1/files/ missing file-id in response

Bug Report
Installation Method
OpenWebui Running in Docker
Environment

Open WebUI Version: v0.4.7(neueste)

Ollama (if applicable): neuste

Operating System: macOS latest

Browser (if applicable): Brave latest

Confirmation:

[ x] I have read and followed all the instructions provided in the README.md.
[ x] I am on the latest version of both Open WebUI and Ollama.
[ x] I have included the browser console logs.
[ -] I have included the Docker container logs.
[ x] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:

I am using the endpoint POST /api/v1/files/
The file is uploaded
The endpoint is returning http:200 and a summary of the content of the file.
file-id should be included in the response
call the next api endpoint to add the file to a knowledgebase is  impossible without file-id
I could live by looking up the knowledgebase-id and hardcode it, but without the file-id this is useless if you want to build an external agent app for users without access to Open WebUI .

Actual Behavior:

I am using the endpoint POST /api/v1/files/
The file is uploaded
The endpoint is returning http:200 and a summary of the content of the file.

Description
It is impossible to continue without a file-id. In an external application with an agent where customers
have no access to Open WebUI  they cannot get this file-id. So the API endpoints are useless without being able
to retrieve the file-ids or the knowledgebase ids. Especially the POST /api/v1/files/  should return the file-id, additionally to the summary of the uploaded file.
Bug Summary:
Uploaded file-id should be included in the response of the POST /api/v1/files/  endpoint
Reproduction Details
Steps to Reproduce:

Upload file via POST /api/v1/files/
check response
summary is present, file-id is missing

Logs and Screenshots
Browser Console Logs:
{"success":true,"message":"File uploaded successfully: I've parsed the context information and extracted relevant details from the user query. It appears that the user is seeking information on various companies or services related to construction, building maintenance, and real estate......\n\nPlease let me know how you would like me to proceed with generating a response!"}
Docker Container Logs:
not needed
Screenshots/Screen Recordings (if applicable):
not needed
Additional Information
If you want an agent application for users that do not have access to open-webui and look up the file-id or knowledge-id manually, it is impossible to build a working agent app.

id is returned.

Sorry, but i cannot see a file-id here.
This is var_dump of the complete Response Object from the POST /api/v1/files/
$this->rsp->Raw     = curl_exec($this->curl);
var_dump($this->rsp);
object(EmbedDocument)#2 (4) {
["Raw"]=>
string(780) "{"model":"llama3:latest","created_at":"2024-12-04T22:49:03.13258Z","message":{"role":"assistant","content":"I've parsed the context information, which appears to be a collection of company profiles and services offered by various businesses in the construction and contracting industry. The companies listed seem to provide a range of services including building renovation, electrical installation, plumbing, and more.\n\nIf you have a specific query or question about one of these companies or their services, I'd be happy to help. Please let me know what you're looking for!"},"done_reason":"stop","done":true,"total_duration":14693765542,"load_duration":18803333,"prompt_eval_count":2048,"prompt_eval_duration":10043000000,"eval_count":86,"eval_duration":4613000000}"
["RawDecoded"]=>
array(11) {
["model"]=>
string(23) "handwerkerllama3:latest"
["created_at"]=>
string(26) "2024-12-04T22:49:03.13258Z"
["message"]=>
array(2) {
["role"]=>
string(9) "assistant"
["content"]=>
string(467) "I've parsed the context information, which appears to be a collection of company profiles and services offered by various businesses in the construction and contracting industry. The companies listed seem to provide a range of services including building renovation, electrical installation, plumbing, and more.
If you have a specific query or question about one of these companies or their services, I'd be happy to help. Please let me know what you're looking for!"
}
["done_reason"]=>
string(4) "stop"
["done"]=>
bool(true)
["total_duration"]=>
int(14693765542)
["load_duration"]=>
int(18803333)
["prompt_eval_count"]=>
int(2048)
["prompt_eval_duration"]=>
int(10043000000)
["eval_count"]=>
int(86)
["eval_duration"]=>
int(4613000000)
}
["Data"]=>
string(467) "I've parsed the context information, which appears to be a collection of company profiles and services offered by various businesses in the construction and contracting industry. The companies listed seem to provide a range of services including building renovation, electrical installation, plumbing, and more.
If you have a specific query or question about one of these companies or their services, I'd be happy to help. Please let me know what you're looking for!"
["Err"]=>
bool(false)
}
{"success":true,"message":"Datei wurde erfolgreich hochgeladen: "}


v1.5 backports 2019-06-05

PRs:

#7710
#8197

This change is

test-me-please

test-me-please

test-missed-k8s

test-upstream-k8s

test-docs-please

Build got stuck: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2167/ #7796

test-missed-k8s

Same issue, stuck again in the CI:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2168/
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2168/
[...]
20:49:34  Ran 44 of 369 Specs in 1790.763 seconds
20:49:34  SUCCESS! -- 44 Passed | 0 Failed | 0 Pending | 325 Skipped
20:49:34  PASS
20:49:34
20:49:34  Ginkgo ran 1 suite in 30m9.749359989s
20:49:34  Test Suite Passed
Post stage
[Pipeline] sh
20:49:35  + cd /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.15-gopath/src/github.com/cilium/cilium/test
20:49:35  + ./post_build_agent.sh
20:49:35  output of following commands are ran on the Jenkins agent itself
20:49:35  output of: "ps -ef | grep -i vbox"
20:49:35  root      4129     1  0 19:22 ?        00:00:07 /usr/lib/virtualbox/VBoxXPCOMIPCD
20:49:35  root     11924 11921  0 20:49 ?        00:00:00 grep -i vbox
20:49:35  root     12785     2  0 07:23 ?        00:00:00 [iprt-VBoxWQueue]
20:49:35  root     12790     2  0 07:23 ?        00:00:00 [iprt-VBoxTscThr]
20:49:35  root     22663     1  0 20:11 ?        00:00:07 /usr/lib/virtualbox/VBoxSVC --auto-shutdown
20:49:35  root     23888 22663  0 20:11 ?        00:00:00 /usr/lib/virtualbox/VBoxNetDHCP --comment HostInterfaceNetworking-vboxnet0 --config /tmp/dhcp-config-ldxvv.xml
20:49:35  root     24418 22663 61 20:11 ?        00:23:35 /usr/lib/virtualbox/VBoxHeadless --comment test_k8s1-113_1559765474350_2913 --startvm 64a0fea4-1b2c-4a96-9285-61fcd2f10cb9 --vrde config
20:49:35  root     29813 22663 24 20:18 ?        00:07:31 /usr/lib/virtualbox/VBoxHeadless --comment test_k8s2-113_1559765901978_22449 --startvm 1fefc655-4bab-4e1c-b5be-f794a5df286b --vrde config
20:49:35  output of: "ps -ef | grep -i vagrant"
20:49:35  root     11926 11921  0 20:49 ?        00:00:00 grep -i vagrant
20:49:35  output of: "VBoxManage list runningvms"
20:49:35  "test_k8s1-113_1559765474350_2913" {64a0fea4-1b2c-4a96-9285-61fcd2f10cb9}
20:49:35  "test_k8s2-113_1559765901978_22449" {1fefc655-4bab-4e1c-b5be-f794a5df286b}
20:49:35  output of: "VBoxManage list intnets"
20:49:35  Name:        cilium-k8s2168-Cilium-PR-Ginkgo-Tests-K8s-1.13
20:49:35  Name:        intnet
[Pipeline] sh
20:49:36  + cd /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.15-gopath/src/github.com/cilium/cilium/test
20:49:36  + ./archive_test_results.sh
20:49:36  ./archive_test_results.sh: line 3: cd: /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/test: No such file or directory
[Pipeline] sh
20:49:38  + cd /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.15-gopath/src/github.com/cilium/cilium/test/..
20:49:38  + mv test_results_Cilium-PR-Ginkgo-Tests-K8s_2168_BDD-Test-k8s-1.15.zip /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s
[Pipeline] sh
20:49:39  + cd /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.15-gopath/src/github.com/cilium/cilium/test
20:49:39  + mv k8s-1.11.xml k8s-1.12.xml k8s-1.15.xml /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/src/github.com/cilium/cilium/test
[Pipeline] sh
20:49:40  + cd /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-K8s/1.15-gopath/src/github.com/cilium/cilium/test
20:49:40  + vagrant destroy -f
20:49:41  ==> k8s2-1.14: VM not created. Moving on...
20:49:41  ==> k8s1-1.14: VM not created. Moving on...
20:49:41  ==> runtime: VM not created. Moving on...
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] }

test-missed-k8s

Same here.

test-missed-k8s

test-missed-k8s

test-docs-please

test-me-please

test-upstream-k8s

Direct route test failed again: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/2172/flowGraphTable/

test-missed-k8s

test-missed-k8s

Only 00:52:36.557  [Fail] K8sFQDNTest [It] Restart Cilium validate that FQDN is still working failed


Support associating tests to multiple parents, if they have multiple parents

Original discussion in: https://github.com/astronomer/astronomer-cosmos/pull/751#issuecomment-1878578854
Cosmos can improve its implementation towards tests with multiple parents - we should make it consistent with dbt.

@tatiana, thank you, but unfortunately, I wouldn't be able to contribute at this time.


Spurious "Invalid cookie header" warnings

This spurious warning shows up in my logs every time I execute a remote call from my local machine or from AWS Lambda (in GMT) to Airtable. It happens on both reads and writes. I'm just using the airtable.java library with no customizations of the cookie headers.
WARNING: Invalid cookie header: "Set-Cookie: brw=bfwPbO3TrDZZa5sPw; path=/; expires=Fri, 20 Mar 2020 03:42:39 GMT; domain=.airtable.com; secure; httponly". Invalid 'expires' attribute: Fri, 20 Mar 2020 03:42:39 GMT

Im having the same problem, did you find a solution?

I added two logging libraries to my projects in build.gradle:
dependencies {
     ...

    // Provides a logger implementation for airtable.java to get rid of warning
    compile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.25'

    // Use log4j logging to be able to turn off spurious warnings from libraries
    compile group: 'log4j', name: 'log4j', version: '1.2.17'

    ...
}

and I put this in src/main/resources/log4j.properties
# Root logger option
log4j.rootLogger=INFO, stdout

# Direct log messages to stdout
log4j.appender.stdout=org.apache.log4j.ConsoleAppender
log4j.appender.stdout.Target=System.out
log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
log4j.appender.stdout.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} %-5p %c{1}:%L - %m%n

# Remove spurious cookie headers warning
log4j.logger.org.apache.http.client.protocol.ResponseProcessCookies=FATAL

So now doing it this way you get the data from airtable using .select()? I
don't know what I am doing wrong but I can update, remove, and create new
records in airtable but I can't seem to get them unless I use find.
El mié., 14 de agosto de 2019 1:33, Joe Sondow<EMAIL_ADDRESS>escribió:

I added two logging libraries to my projects in build.gradle:
dependencies {
...
// Provides a logger implementation for airtable.java to get rid of warning
compile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.25'

// Use log4j logging to be able to turn off spurious warnings from libraries
compile group: 'log4j', name: 'log4j', version: '1.2.17'

...

}
and I put this in src/main/resources/log4j.properties
Root logger option
log4j.rootLogger=INFO, stdout
Direct log messages to stdout
log4j.appender.stdout=org.apache.log4j.ConsoleAppender
log4j.appender.stdout.Target=System.out
log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
log4j.appender.stdout.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} %-5p %c{1}:%L - %m%n
Remove spurious cookie headers warning
log4j.logger.org.apache.http.client.protocol.ResponseProcessCookies=FATAL
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Sybit-Education/airtable.java/issues/35?email_source=notifications&email_token=AGLCDWR5UPK3M2HIXMQLSWDQEORNHA5CNFSM4G7ZOR42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4H2S6I#issuecomment-521120121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGLCDWRWSDLHRZYXUEYKQ7TQEORNHANCNFSM4G7ZOR4Q
.

So now doing it this way you get the data from airtable using .select()? I don't know what I am doing wrong but I can update, remove, and create new records in airtable but I can't seem to get them unless I use find.

This question seems out of scope for this issue ticket. I recommend asking in the Airtable community forums or on Stack Overflow. Or for an example of how I've used .select() successfully you can check one of my projects here.

According to the posts here https://stackoverflow.com/questions/36473478/fixing-httpclient-warning-invalid-expires-attribute-using-fluent-api
this is an issue that originates in the Apache HTTP client.  There is a proposed solution of setting the cookie spec to "standard":
HttpClient httpClient = HttpClients.custom()         .setDefaultRequestConfig(RequestConfig.custom()                 .setCookieSpec(CookieSpecs.STANDARD).build())         .build();

There's a good looking list of solutions to this problem based on which library and which syntax is used to create the Http Client on this page: https://www.lenar.io/invalid-cookie-header-invalid-expires-attribute/

It looks like airtable.java uses Unirest for http calls. Unirest updated in 2019 to fix this error.
https://github.com/Kong/unirest-java/issues/97
I've attempted to figure out all the necessary changes in pull request https://github.com/Sybit-Education/airtable.java/pull/52 but the integration tests are a challenge for an outsider to run, so I'm not sure if I missed anything.


SortedLastIndexBy has an iterator on the 2nd arg

Fixes #44

Merged and fixed in v1.4.2. Thanks for the PR!


Update the Partners Solution Overview page for Elastic to point to current documentation

The Elastic Documentation link points to stale documentation link. Could you please update it to the current documentation link here https://www.elastic.co/guide/en/cloud/current/ec-azure-marketplace-native.html ?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d57d4e32-12ab-0c8c-1a4b-803d7adc819b
Version Independent ID: 12afb1db-a6f3-7fb9-546c-31e0174b4e6f
Content: Elastic integration overview - Azure partner solutions
Content Source: articles/partner-solutions/elastic/overview.md
Service: partner-services
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@hemantmalik Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.

@hemantmalik - thanks for the feedback. It looks like this link has already been fixed.
#please-close

Thank you @tfitzmac


can it possible run loop in mxnet?

I'd like to write loop code to tuning dnn parameters ,in case that to avoid loading large data everytime.The code as below

And I found when the first loop finished,it will be hunt after starting to the second loop.

Can't it possbile to run loop in python code?
the command as below
launch.py -H hosts2 -n2 -s1 python train.py --kv-store dist_sync --train-dir=$train_path --val-dir=$val_path --save-model -prefix=model/dnn --batch-size=800 --num-epochs=100 --gpu=0,1,2,3 --momentum=0.8 --num-examples=$train_examples --lr-factor=0.4
@piiswrong

@liumilan what's you want is to try different lr? this code should be run ok, but the second loop result model will override previous i think.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


benchmark_model: Could not create Hexagon delegate: platform may not support delegate or required libraries are missing

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Android 11
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: Samsung S21
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): nightly

Describe the current behavior

Follow instructions in https://www.tensorflow.org/lite/performance/measurement#native_benchmark_binary and https://www.tensorflow.org/lite/performance/hexagon_delegate to run TFLite benchmark_model with use_hexagon=true.
adb push contents of these to /data/local/tmp:

android_aarch64_benchmark_model
libhexagon_interface.so
hexagon_nn_skel v<IP_ADDRESS>
mobilenet_v2_1.0_224_quant

Verify with adb shell ls /data/local/tmp:android_aarch64_benchmark_model
libhexagon_interface.so
libhexagon_nn_skel.so
libhexagon_nn_skel_v65.so
libhexagon_nn_skel_v66.so
mobilenet_v2_1.0_224.tflite
mobilenet_v2_1.0_224_quant.tflite

Run adb shell /data/local/tmp/android_aarch64_benchmark_model --graph=/data/local/tmp/mobilenet_v2_1.0_224_quant.tflite --use_hexagon=true --hexagon_lib_path=/data/local/tmp
Hexagon delegate fail to create:STARTING!
Log parameter values verbosely: [0]
Graph: [/data/local/tmp/mobilenet_v2_1.0_224_quant.tflite]
Use Hexagon: [1]
Hexagon lib path: [/data/local/tmp]
Loaded model /data/local/tmp/mobilenet_v2_1.0_224_quant.tflite
INFO: Initialized TensorFlow Lite runtime.
WARNING: Failed to fetch Hexagon NN version. This might be because you're using incompatible versions of libhexagon_interface and libhexagon_nn_skel. You must use compatible versions. Refer to Tensorflow Lite Hexagon Delegate Guide.
INFO: Hexagon Delegate is not supported.

loaded libcdsprpc.so
Could not create Hexagon delegate: platform may not support delegate or required libraries are missing
The input model file size (MB): 3.57776
Initialized session in 51.739ms.
Running benchmark for at least 1 iterations and at least 0.5 seconds but terminate if exceeding 150 seconds.
count=66 first=23605 curr=6778 min=6765 max=23605 avg=7583.89 std=2197

Running benchmark for at least 50 iterations and at least 1 seconds but terminate if exceeding 150 seconds.
count=82 first=6870 curr=850063 min=6765 max=850063 avg=17097.5 std=92551

Inference timings in us: Init: 51739, First inference: 23605, Warmup (avg): 7583.89, Inference (avg): 17097.5
Note: as the benchmark tool itself affects memory footprint, the following is only APPROXIMATE to the actual memory footprint of the model at runtime. Take the information at your discretion.
Memory footprint delta from the start of the tool (MB): init=4.73828 overall=8.75

Describe the expected behavior
benchmark_model runs with hexagon delegate enabled.

Hi @tonysung ! I see you have pushed multiple versions of libhexagon_nn_skel.so . Can you let us know the results with only one libhexagon_nn_skel.so ? Thanks!

Tried keeping only each one of libhexagon_nn_skel.so, libhexagon_nn_skel_v65.so, libhexagon_nn_skel_v66.so, both with their names unchanged, and with them all renamed to libhexagon_nn_skel.so (total 6 combinations). Same result:
WARNING: Failed to fetch Hexagon NN version. This might be because you're using incompatible versions of libhexagon_interface and libhexagon_nn_skel. You must use compatible versions. Refer to Tensorflow Lite Hexagon Delegate Guide.
INFO: Hexagon Delegate is not supported.

loaded libcdsprpc.so
Could not create Hexagon delegate: platform may not support delegate or required libraries are missing

as you can see from the hexagon delegate page, Samsung S21 (supposedly Snapdragon 888, which has Hexagon 780) is not supported.

Could you please follow the steps mentioned here and let us know if this helps to solve your problem. Thanks!

Hi @sachinprasadhs If this has anything to do with the nativeLibraryDir as mentioned here, then I'm afraid it is a bug in benchmark_model, because it should load the library NOT from nativeLibraryDir but from the path specified in --hexagon_lib_path:
$ adb shell /data/local/tmp/android_aarch64_benchmark_model --help
...
--hexagon_lib_path=/data/local/tmp string optional The library path for the underlying Hexagon libraries.
...

And here's the original command when this bug is reproduced:
adb shell /data/local/tmp/android_aarch64_benchmark_model \
--graph=/data/local/tmp/mobilenet_v2_1.0_224_quant.tflite \
--use_hexagon=true \
--hexagon_lib_path=/data/local/tmp

And the content of /data/local/tmp:
$ adb shell ls /data/local/tmp:
android_aarch64_benchmark_model
libhexagon_interface.so
libhexagon_nn_skel.so

I believe that's not the cause, as the page said, "Currently the following Hexagon architecture are supported, including but not limited to ..."

Please check https://github.com/tensorflow/tensorflow/issues/47246.

I tried the same on both Samsung Galaxy S8 (Snapdragon 835) and Google Pixel 3 (Snapdragon 845), and both failed.
Both listed to be supported according to https://www.tensorflow.org/lite/performance/hexagon_delegate
The error message is a bit different though.
Samsung Galaxy S21:

WARNING: Failed to fetch Hexagon NN version. This might be because you're using incompatible versions of libhexagon_interface and libhexagon_nn_skel. You must use compatible versions. Refer to Tensorflow Lite Hexagon Delegate Guide.

Samsung Galaxy S8 & Google Pixel 3:

ERROR: Failed to load libhexagon_interface.so, Error: dlopen failed: library "libhexagon_interface.so" not found

adb shell ls -l /data/local/tmp/android-benchmark-tools/aarch64
total 16776
-rwxrw-rw- 1 shell shell 6707096 2022-04-01 16:01 benchmark_model
-rwxrw-rw- 1 shell shell 6727576 2022-04-01 16:09 benchmark_model_performance_options
-rwxrw-rw- 1 shell shell<PHONE_NUMBER>-04-01 16:01 libhexagon_interface.so
-rwxrwxrwx 1 shell shell 1148720 2022-04-12 22:31 libhexagon_nn_skel.so
-rwxrwxrwx 1 shell shell 1165168 2022-04-12 22:31 libhexagon_nn_skel_v65.so
-rwxrwxrwx 1 shell shell 1161072 2022-04-12 22:31 libhexagon_nn_skel_v66.so

Full error log:
STARTING!
Log parameter values verbosely: [0]
Graph: [/data/local/tmp/android-benchmark-tools/tflite/mobilenet_v2_1.0_224_quant.tflite]
Use Hexagon: [1]
Hexagon lib path: [/data/local/tmp/android-benchmark-tools/aarch64]
Loaded model /data/local/tmp/android-benchmark-tools/tflite/mobilenet_v2_1.0_224_quant.tflite
INFO: Initialized TensorFlow Lite runtime.
ERROR: Failed to load libhexagon_interface.so, Error: dlopen failed: library "libhexagon_interface.so" not found
INFO: Hexagon Delegate is not supported.

Could not create Hexagon delegate: platform may not support delegate or required libraries are missing
The input model file size (MB): 3.57776
Initialized session in 2.229ms.
Running benchmark for at least 1 iterations and at least 0.5 seconds but terminate if exceeding 150 seconds.
count=15 first=38031 curr=34185 min=34119 max=38031 avg=34471.7 std=954

Running benchmark for at least 50 iterations and at least 1 seconds but terminate if exceeding 150 seconds.
count=50 first=34574 curr=34198 min=34125 max=35839 avg=34541.4 std=450

Inference timings in us: Init: 2229, First inference: 38031, Warmup (avg): 34471.7, Inference (avg): 34541.4
Note: as the benchmark tool itself affects memory footprint, the following is only APPROXIMATE to the actual memory footprint of the model at runtime. Take the information at your discretion.
Memory footprint delta from the start of the tool (MB): init=1.24219 overall=6.80078

@tonysung as freedomtan mentioned new chipset doesn't use the Hexagon NN, so we can't support it currently.
For the other supported use cases of Samsung S8
You will need to make libhexagon_interface.so in the LD library path so it can be loaded
Example
LD_LIBRARY_PATH=/data/local/tmp/android-benchmark-tools:${LD_LIBRARY_PATH}

See example here when it is done for testing.
For benchmark tool we set rpath to /data/lcoal/tmp/ by default to ease running. See instructions

One more thing for pixel phones DSP access is disabled so it will not work.
If you have the Pixel phone rooted you can run these commands first
adb root
adb shell setenforce 0

Documentation updated.
Thanks a lot

@tonysung would you please provide the complete instruction to make dsp run? there's no android-benchmark-tools in tmp directory!


Generic Vertex Array Implementation

Similar to the VertexAttribute in WhirlyGlobe, we need a generic way of holding and defining vertex arrays.
On the shader side we need a way of describing them and on the Drawable side we need a way of holding the data.

I think you've got this one going as well.  What do you think?


DIANA_mAP Error: subscript contains out-of-bounds indices

Hi,
When running DIANA_mAP with four fatq.gz files under two conditions I receive the following error:
ANALYSIS END TIME: Wed 22 Sep 2021 10:29:07 AM UTC
DE ANALYSIS START TIME: Wed 22 Sep 2021 10:29:07 AM UTC

INITIATING DIFFERENTIAL EXPRESSION ANALYSIS...
converting counts to integer mode
estimating size factors
estimating dispersions
gene-wise dispersion estimates
mean-dispersion relationship
final dispersion estimates
fitting model and testing
PLOTTING PCA...
PLOTTING MA-PLOT...
EXTRACTING TOP RESULTS...
Error: subscript contains out-of-bounds indices
Execution halted

Any ideas why this might be occurring?
Thanks

Hello,
Can you check the file named "DE_full_results.csv" for the number of results/lines?
It is possible your DE analysis returns less than 200 results and so when the script tries to isolate the top 100 and the bottom 100 it runs out of results.
Double check that your conditions and samples are configured correctly and that you are comparing what you want.
In order to fix this, you can open the "diana_map_multi.r" script, and search for line 149 which should write: "n = 100".
Change the number 100 to a lower number according to your total results number, save the script and run it again. (Remember this gets the top n and bottom n differentialy expressed results. So if you set n to 50, you would need to have at least 100 total results in the "DE_full_results.csv" file)
Regards,
Thanos

Yes you were correct, my DE analysis returned ~ 160 results so this does
appear to be the reason for the error. Once I added in more samples the
error disappeared.
Thanks again for your help.
On Wed, 22 Sept 2021 at 12:19, Athanasios Alexiou @.***>
wrote:

Hello,
Can you check the file named "DE_full_results.csv" for the number of
results/lines?
It is possible your DE analysis returns less than 200 results and so when
the script tries to isolate the top 100 and the bottom 100 it runs out of
results.
Double check that your conditions and samples are configured correctly and
that you are comparing what you want.
In order to fix this, you can open the "diana_map_multi.r" script, and
search for line 149 which should write: "n = 100".
Change the number 100 to a lower number according to your total results
number, save the script and run it again. (Remember this gets the top n and
bottom n differentialy expressed results. So if you set n to 50, you would
need to have at least 100 total results in the "DE_full_results.csv" file)
Regards,
Thanos
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/athalexiou/DIANA-mAP/issues/3#issuecomment-924834535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQEY7DXOMKWZQA3WQ7ABAEDUDG3SRANCNFSM5ERBCNOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Update napari hub info

Closes #152, also fixes some minor packaging details for napari-hub

Codecov Report

Merging #157 (af65947) into main (acded1f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #157   +/-   ##
=======================================
  Coverage   81.08%   81.08%
=======================================
  Files          14       14
  Lines        1475     1475
=======================================
  Hits         1196     1196
  Misses        279      279

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 acded1f...af65947. Read the comment docs.


invalid constant string

I'm trying to make this package compatible with the latest master, but running into issues. First line 109 has an invalid convert call now, but it seems like anything I try to change it to gives me:
Out [2]: ERROR: LoadError: HTTP Parser Exception: HPE_INVALID_CONSTANT(25):invalid constant string
while loading In[2], in expression starting on line 3

Looking at the C file header, it seems like it should be pretty straightforward, http_parser_execute is just expecting a const char *, yet I still see the error above. Any ideas? @Keno?

Never mind, it was a bug from my Requests package.


Test building on freebsd with cgo disabled

Ensure that it's possible to build for freebsd with cgo disabled.
Ref: https://github.com/zalando/go-keyring/issues/73

:+1:

:+1:


build a private library: Build Failed with POST_BUILD_CHECKS_FAILED

I was building a private library (open-simulation-interface) using the vcpkg platform by creating a new portfile.
While the entire build process went smoothly and built the required libraries, I still receive 1 error
Error: Building package open-simulation-interface:x64-windows failed with: POST_BUILD_CHECKS_FAILED
The following DLLs have no exports: xxxx.dll
DLLs without any exports are likely a bug in the build script.
Is there any option which makes it generate the necessary dllexport option?

Hi @pankit7
Thanks for posting this issue.
Could you please provide portfile.cmake and CONTROL files for this port?

Please feel free to reopen this or open a new issue if this is still a problem for you.


RTS cannot be controlled by individual serial tools (EUB-68)

使用ComTool无法控制rts，但是可以控制dtr
使用CH340则是正常的
不只是ComTool，我也还在其他串口工具也发现这个问题
希望修复这个兼容性问题

Raw translation:
It is possible to control DTR with ComTool but not RTS. This is possible with CH340. The same issue is present with other serial tools as well.

Hi @HoGC. Can you please describe what do you want to achieve? Are you talking about changing RTS/DTR for the host or the target MCU?

@dobairoland Yes, I need to use RTS to restart MCU


update Open Rodent's Revenge license, platforms and dependencies

Hey, thanks for the awesome resource!
This is just a quick PR to update Open Rodent's Revenge entry as it has switched to the MIT license since.
Also, Qt was missing from the code dependencies list. Since I used SFML for the game proper, it is effectively cross-platform even though I never provided builds for macOS.
I wish I could write build instructions but the C++ and especially Qt build tooling and practices have changed quite a bit in the last couple of years. I do know that the master can easily build on recent Qt Creator versions but not sure about the 0.7 and previous releases.

Thanks for the info. Btw. how did the switching of the license take place. Was there only one developer or did all developers agree or was there a complete rewrite in the middle? Usually one cannot go from GPL to MIT.

I'm the sole developer of this project and I did not know about the MIT license at the time!
The switch did happen during a full rewrite (that is not complete to this day).


Add category in DTM with text in filter

9.2.2 / windows 10 / 11.0.2
master / ubuntu 18.04 / 11.0.10
Was adding a new struct and wanted to categorize it.
I typed in a filter to see if i had already made the category, which I hadn't.
I left my filter string there.
Right clicked my program's archive and did new->category
I'm not sure on the timing, maybe its when I went to go delete the filter, but this raises an exception
java.lang.NullPointerException
	at docking.widgets.tree.internal.GTreeModel.valueForPathChanged(GTreeModel.java:115)
	at java.desktop/javax.swing.plaf.basic.BasicTreeUI.completeEditing(BasicTreeUI.java:2408)
	at java.desktop/javax.swing.plaf.basic.BasicTreeUI$Handler.editingStopped(BasicTreeUI.java:4117)
	at java.desktop/javax.swing.AbstractCellEditor.fireEditingStopped(AbstractCellEditor.java:147)
	at java.desktop/javax.swing.DefaultCellEditor$EditorDelegate.stopCellEditing(DefaultCellEditor.java:370)
	at java.desktop/javax.swing.DefaultCellEditor.stopCellEditing(DefaultCellEditor.java:234)
	at java.desktop/javax.swing.tree.DefaultTreeCellEditor.stopCellEditing(DefaultTreeCellEditor.java:302)
	at java.desktop/javax.swing.plaf.basic.BasicTreeUI.completeEditing(BasicTreeUI.java:2356)
	at java.desktop/javax.swing.plaf.basic.BasicTreeUI$Handler.valueChanged(BasicTreeUI.java:4133)
	at java.desktop/javax.swing.tree.DefaultTreeSelectionModel.fireValueChanged(DefaultTreeSelectionModel.java:641)
	at docking.widgets.tree.internal.GTreeSelectionModel.fireValueChanged(GTreeSelectionModel.java:74)
	at java.desktop/javax.swing.tree.DefaultTreeSelectionModel.notifyPathChange(DefaultTreeSelectionModel.java:1111)
	at java.desktop/javax.swing.tree.DefaultTreeSelectionModel.removeSelectionPaths(DefaultTreeSelectionModel.java:505)
	at docking.widgets.tree.internal.GTreeSelectionModel.removeSelectionPaths(GTreeSelectionModel.java:90)
	at java.desktop/javax.swing.JTree.removeDescendantSelectedPaths(JTree.java:3804)
	at java.desktop/javax.swing.JTree$TreeModelHandler.treeStructureChanged(JTree.java:3911)
	at docking.widgets.tree.internal.GTreeModel.swingFireRootChanged(GTreeModel.java:150)
	at docking.widgets.tree.internal.GTreeModel.privateSwingSetRootNode(GTreeModel.java:54)
	at docking.widgets.tree.GTree.swingSetModelRootNode(GTree.java:765)
	at docking.widgets.tree.GTree.swingSetFilteredRootNode(GTree.java:749)
	at docking.widgets.tree.GTreeFilterTask.lambda$1(GTreeFilterTask.java:57)
	at docking.widgets.tree.GTreeTask$CheckCancelledRunnable.run(GTreeTask.java:85)
	at ghidra.util.Swing.lambda$2(Swing.java:227)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

There has been work in the tree node area fairly recently.   This sounds like an edge case that was not tested or  worse, it may be timing related.

Followup issue, I can open a new ticket if that's easier to track or if seemed separate enough.
In the failure case it looks like a new category New Category is created.  I guess you could go either way with that, but I'd lean towards no side effect artifacts

I think this has been fixed in master.  Is it possible for you to test?

Sweet, I can test tonight, and I'll follow up if it's still happening

FYI - there is another pending fix which has not yet been pushed-out.


Jenkins script should use released pytest once v2.8 is released

Bc we need pytest-dev/pytest@aa25fb0 to make things nice as can be.

Not relevant anymore, as we're using the pytest release after all.


Alpha release

@iamareebjamal @simarsingh24 @dreadpool2 Please review

I see a lot of commits being as long as Shakespeare story. Please follow commit guidelines and use commit description to write essays
Also, is the version code and name of the app incremented?

@iamareebjamal No but this is the first time we would be submitting the app to the playstore. Should we still increment it ?

Yes, it is already on F-Droid


[BUG] drak mode logo is bigger in size affecting the height of the navbar

Describe the bug
The logo for dark mode is huge which is also increasing height of the navbar.
If changes are required,Do assign it to me under GSSOC 2023,chamge will be done : )
To Reproduce
Steps to reproduce the behavior:

Go to 'Navbar component reduce the size accordingly. '
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Kindly assign this issue to me under GSSoC'23

This issue has been inactive for a long time
I would like to work on this issue
Please assign

Hey ,
I would like to work on this issue. Can you please assign it to me.


add name of author

I saw on the PyDigger site that there is no author name

Thanks for the contribution!


Existing context

#67
this solves the issue, it is the same change made in v1

Thanks for the PR and report, I've gone in this direction which is a bit more efficient: https://github.com/pressly/chi/pull/69


Improve gmlock, gmutex, grpool and gflock unit testing.

Improve gmlock, gmutex, grpool and gflock unit testing.
Improve gflock.

Codecov Report

Merging #212 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   75.77%   75.92%   +0.15%
==========================================
  Files         204      205       +1
  Lines       17558    17586      +28
==========================================
+ Hits        13304    13352      +48
+ Misses       3619     3600      -19
+ Partials      635      634       -1

Impacted Files
Coverage Δ

g/os/gflock/gflock.go
92.85% <100%> (ø)

g/os/gcfg/gcfg.go
85.07% <0%> (-1.12%)
:arrow_down:

g/container/gtree/gtree_btree.go
91.87% <0%> (-0.68%)
:arrow_down:

g/container/gtree/gtree_avltree.go
98% <0%> (+0.5%)
:arrow_up:

g/os/gmutex/gmutex.go
100% <0%> (+2.32%)
:arrow_up:

g/os/gmlock/gmlock_locker.go
100% <0%> (+8.88%)
:arrow_up:

g/os/gspath/gspath_cache.go
86.36% <0%> (+13.63%)
:arrow_up:

g/os/gmlock/gmlock.go
100% <0%> (+18.18%)
:arrow_up:

g/os/grpool/grpool.go
100% <0%> (+21.27%)
:arrow_up:

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 ebed433...3806f9d. Read the comment docs.


Add 'updates' to the dict auto() returns

Hey there- I found a simple bug:  auto() doesn't return all the keys you're expecting when you call set_schedule().  I'm not sure this is exactly the best default to use, but it seems like it would be.
Great package, btw!

Any interest in merging this?  It's been 2 weeks...

@andrewgiessel yes, I'm interesting in merging but my time on this project is on a volunteer basis and I've been busy with other things. I will take a look when time permits.

@andrewgiessel thanks for noticing the bug!
I'm not sure why I added the updates to set_schedule in the first place. Looking at it again, the concerns about how often the console gets updated should be separate from the annealing schedule. I'm inclined to remove updates from set_schedule method instead of passing it around here. But this works for now. I'll get a release out soon...
I appreciate the contribution, sorry it's taken me so long to review.


libnsl.so.1missing since Fedora 28

Hi!
biber is not included in my texlive distribution so I downloaded the binary from https://ctan.org/tex-archive/biblio/biber/biber-linux. Unfortunately I cannot run it on Rocky Linux 8.9 as biber depends on libnsl.so.1, which is no more installed by default since Fedora 28.
The issue has already been mentioned at various places, but I could not find anything on github.
https://lists.gnu.org/archive/html/reproduce-devel/2022-10/msg00010.html
https://lists.gnu.org/archive/html/reproduce-devel/2019-08/msg00019.html
https://tex.stackexchange.com/questions/301126/how-to-install-vanilla-texlive-on-fedora
Many projects suffered from the same issue (searching for "missing libnsl.so.1" show how common it is), for instance it is discussed in
https://github.com/AppImageCommunity/pkg2appimage/issues/336
For anyone having this issue on RHEL-based OS, a solution is to install locally libnsl.so.1, running sudo dnf install libnsl.
Concerning biber, I read that libnsl.so.1 is a legacy and that applications should use libnsl.so.2. Is there a plan to switch to libnsl.so.2?

This basically needs an upgrade of my linux 64 build machine. It's on Ubuntu 10.10 currently to be as backwards compatible as possible. Every few years it needs an upgrade ...

libnsl.so.1 is also missing in standard installations of SuSE Tumbleweed. However it is still available from the main repository and can be installed using either YaST or
sudo zypper install libnsl1

But note, that libnsl1 does not have support for IPv6. So using libnsl3 would be recommended, because it libnsl3 has support for IPv6.

Thanks @komascript, I had the same issue on Tumbleweed :pray:

With SuSE Tumbleweed this one works without error (one biber --version tested, but should be enough for the libnsl.so.1 issue). Thank you.
BTW: I don't know which libnsl version is part of SuSE Leap 15.4, 15.5, 15.6, because dedicated libnsl packages are only available with Tumbleweed and Slowroll. The Leaps seem to have the lib hidden in another package and currently I do not have any Leap version installed.

I confirm I can execute biber version: 2.20 (beta). When I try to use it in a project I run into compatibilities issues with my biblatex but this is unrelated. Thanks for the fix!

With the latest, updated  texlive on my Fedora 39 system, when I run this new devel version of biber I get the error
Running Biber' on series_review' with ``biber  series_review''
INFO - This is Biber 2.20 (beta)
INFO - Logfile is 'series_review.blg'
INFO - Reading 'series_review.bcf'
ERROR - Error: Found biblatex control file version 3.10, expected version 3.11.
This means that your biber (2.20) and biblatex (3.19) versions are incompatible.
See compat matrix in biblatex or biber PDF documentation.
INFO - ERRORS: 1
TeX Output exited abnormally with code 2 at Thu Mar  7 06:16:07
Is there a version of the biber executable that would work on both Fedora 39 and the newest (2023) texlive out of the box (I understand the workarounds you gave above).

Not really,  you need the corresponding biblatex version which for biber 2.20 is biblatex 3.20 dev version which you can get from Sourceforge.


Collector 5.0.0-alpha version upgrade FAQ.

Please answer these questions before submitting pull request

Why submit this pull request?

[X] Document

Related issues
#1050

You need to add a link from document catalog page, in FAQ section.

FYI @peng-yongsheng I adjust some comments in your doc: https://github.com/apache/incubator-skywalking/pull/1052/commits/fa3dd1e67cc09126dfce9b367e3b966ea7a38363

Coverage remained the same at 19.758% when pulling fa3dd1e67cc09126dfce9b367e3b966ea7a38363 on peng-yongsheng:document/upgrade_faq into 566f7f8fbc73a4195dc17aa2a1b9d9c3f27eb4c4 on apache:master.


Give the author control over where mixin dockerfile lines are inserted

Allow the author to optimize their dockerfile, or take advantage of actions performed by the mixins, by allowing them to control where the dockerfile lines generated by the mixins are inserted into the custom dockerfile template.
When the user adds a line # PORTER_MIXINS to their dockerfile, porter will replace that line with the generated dockerfile lines from the mixins.
If that line is not present, then the mixin dockerfile lines are appended to the end of the dockerfile template.
The default Dockerfile should be updated as well to place # PORTER_MIXINS on the line before the COPYs.

This is a straightforward change, which is why I flagged it with help wanted. Any of our intrepid contributors can tackle this, or one of the maintainers if we need to complete it before a major demo.

Nice one. I'd like to start working on this issue. What I've spotted so far (just looked into the repo while having my first ☕ this morning), is that ./pkg/build/dockerfile-generator.go looks like a good place to start 😄  Or am I totally wrong?

@ThorstenHans That's the spot! It would be great if you'd like to take it on because it is making @squillace twitch, and he's pinging me on slack constantly to complain about his slow builds. 😂
Since your are a regular contributor now I've added you to our org so that we can officially assign issues. 🌟 I have no idea if you could have assigned it to yourself? 🤔 If you have that power, feel free to do so for other issues in the future.
We have a Contributor Ladder that lays out how it all works. Let us know if it's clear as mud, as it's kind of a new thing and probably needs improvement.

Hey @carolynvs thanks for adding me.
I've already drafted a PR, I struggled a bit with the unit-test (still new in the golang world) but did my best.
I've described the struggle in the PR #712
I'd love to get some feedback on that one to improve my golang quickly💃

BTW, I was not sure where documentation for the change would fit best. What do you think?
Or should I add the # PORTER_MIXINS (INJECT_PORTER_MIXINS_TOKEN) to the default Dockerfile.tmpl and add some documentation there (to have something in first place)

@ThorstenHans does https://github.com/deislabs/porter/pull/712#issuecomment-540610831 have anything to help there?

Hey @squillace,
Yep that helps. Will look into it tomorrow evening CEST. Was sick, still need some time to recover 😕


Clicking shipping or payment method options in quick succession bounces them back and forth

Currently when you click a shipping or payment method and adjust your choice before the corresponding graphql call finishes it bounces back to the choice from before the graphql call.
This seems to be the flow of operations;
Pick option 1 > async GQL to backend 1 happens > pick option 2 > async GQL to backend 2 happens > async GQL results 1 come in > page renders option 1 as picked > async GQL results 2 come in > page renders option 2 as picked

I think showing the loader would avoid this situation.

Loader is again available in the development mode. I really hope that will avoid this issue. Hence, closing this.


Replace Grower Contact Info with Party Id; Remove Farm Contact Info

We currently have no mapping between a party (individual/entity) and a grower.  In addition, we have Contact Info objects on both Grower and Farm.
If the Grower has Contact Info, then we should map that via the Party object so that we can optionally classify the grower as an individual/business entity and allow for entity parents.
I'm not sure why a Farm has Contact Info.   Perhaps a physical address of buildings/yard that will have been associated with the farm when it was a standalone entity?   Without a clear definition, I recommend we remove it.

I am fine with both of these changes. Linking things back to the "Party" 🥳 concept makes total sense to me. I can see the use for contact info on the Farm, but I doubt it is common enough and could easily be handled in other ways in the model (context items?).

Our model has contact info at the farm level. It was necessary due to some reporting requirements where certain corporate growers had operations in different states. I request we leave it in.

I can make that work, though it may muddy the semantics of the "company" party type.

Agreement on 27 September 2023 meeting to remove Contact Info from Grower and Farm and replace it with a link to a Party id


Error using --base64 Option

Hey,
When I use --base64 to obfuscate a script the resulting script can not be executed.
The error is:

ï»¿Get-Command : Die Benennung "ï»¿Get-Command" wurde nicht als Name eines Cmdlet, einer
Funktion, einer Skriptdatei oder eines ausführbaren Programms erkannt. Überprüfen Sie die
Schreibweise des Namens, oder ob der Pfad korrekt ist (sofern enthalten), und wiederholen
Sie den Vorgang.
In Zeile:1 Zeichen:1

As you can see the highlighted section contains some invalid characters.  Any reasons why this happens? Or am I doing something wrong?

It still doesn't work, but I think it has something to do with the encoding.
ï»¿ is the BOM of UTF-8 in Windows-1252. I tried changing the encoding to  utf-8 or utf-8-sig, but it made no difference.

I found out what the problem was. The PowerShell ISE save a skript as utf-8 with BOM for some reason. I  just opened the skripts in Notepad++ and saved them with a different encoding.

That's great @MaxMax147! I never work in ISE and just tried it to confirm this behaviour. Now that I know what is the problem, I can address it in the code itself. Thanks for the help here! ;)


Retrieve Routes

Hello there!
This is my very first public PR. I needed the ability to retrieve routes and tried to code it. So far it's been working for me. Don't hesitate to amend or improve and thanks a lot for the lib!
C.

Nice work!


Feature request - Have some rules already enabled in the default installation

I installed Kubenvious in my cluster and the installation worked fine.
However I found that no rules were defined.
I think that it is best if some rules are already there, so that the user can see the value of the tool right away
I suggest at least the following rules

public application
no-resource-limit-pods
latest-tag-check
no-memory-request-set

@kostis-codefresh, thanks for the recommendation.

Resolved in v0.7.


again

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

again