[New Features] Multi-modal Jailbreaking Attack on LLaVA

Thanks for your review.
This PR include:

New Generator: LLaVA (Image+Text->Text)
New multi-modal jailbreak attack probe
New multi-modal jailbreak attack detector

Please let me know if there are anything I can do better.

Wow, thanks for this! We'll get it reviewed

Hi @leondz,

We will upgrade garak to be able to accommodate both text and text+vision prompts in the same abstraction, thank you for working around things here. The approach will change

Will https://github.com/leondz/garak/pull/587/commits/3ea7927281adb8932b0897b0cdee6796a192863b https://github.com/leondz/garak/pull/587/commits/f9332116e7a2307cd5c2995a4458301d97c44ad4 https://github.com/leondz/garak/pull/587/commits/724c9ad28b716e13532bc8fb74468c129be65383 be a part of the upgrade plan in your mind?

@leondz @jmartin-tech @erickgalinkin
Thank you all for the code review. Some major changes have been made to address the problems.
Still some questions beyond the code need to be answer, i.e. how can we design a better (more generalization) detector for every cases of visual jailbreak attack?

Will https://github.com/leondz/garak/commit/3ea7927281adb8932b0897b0cdee6796a192863b https://github.com/leondz/garak/commit/f9332116e7a2307cd5c2995a4458301d97c44ad4 https://github.com/leondz/garak/commit/724c9ad28b716e13532bc8fb74468c129be65383 be a part of the upgrade plan in your mind?

Excellent. Specifying i/o types accepted is important to tackle, i like this direction, not sure on the final data structure - could see this being part of probe, generator, and detector, and also the harness for assembling runs & validating the requested config. Thanks for flagging it. We'll take an internal discussion (or maybe on discord?) and get back to you

Hi @leondz,

We will upgrade garak to be able to accommodate both text and text+vision prompts in the same abstraction, thank you for working around things here. The approach will change

Will 3ea7927 f933211 724c9ad be a part of the upgrade plan in your mind?

Yes, modality checking is definitely part of #602 !

I think the class name should be FigStep rather than VisualJailbreak for garak/detectors/visual_jailbreak.py and garak/probes/visual_jailbreak.py because the attack method FigStep is a subset of visual jailbreak attack. I think there will be more visual jailbreak attack in the future. (https://github.com/leondz/garak/pull/587/commits/65f94cc9116633cf575fc9b3bdbc5c315e97d4d2)

The FigStep paper needs to be cited/recognised somewhere in the garak code, maybe in a docstring for the FigStep probe? With a reference and paper link? This seems appropriate enough

Paper title, arxiv link, and reference in acm format of FigStep are added in docstring in https://github.com/leondz/garak/pull/587/commits/39c57a29cbf76e922629f8060f4aee816d88e8a3.

The default probe uses 80 images
Optionally, a "Full" probe uses more images, but is deactivated by default. This can inherit from the 80 image version (or vice versa) - that's a pretty common pattern in garak

Full (size 500) and small (size 80) of FigStep probe classes have been added in https://github.com/leondz/garak/pull/587/commits/de6c6efb1df2c9eca38f695e39fd0f6206f34d19.
Additionally, size check logic for self.prompts of those two classes is added in https://github.com/leondz/garak/pull/587/commits/6784255c7c3aa99e68dd9070a7f9368412ab1725 and https://github.com/leondz/garak/pull/587/commits/ce2325518e685d94851af373c97573190afec636 respectively.


Weak{Set, Map} entries do not get removed even after GC

WeakSet/WeakMap entries should only exist for as long as the key is still a GC root/still has strong references. In the current implementation, entries are never removed even when a key gets GC'd (last strong Rc gets dropped). This will probably be easier to fix with a proper GC and we no longer use Rc:
Repro
let x;
{
  let y = {};
  x = new WeakSet([y]);
}

console.log(x) // WeakSet { <1 items> }

(This wouldn't necessarily be observable with a real GC where values don't get freed immediately once unreachable but we are using Rc, and because Weak<T> is used as key, the WeakSet count should be at 0)

closing for now because Weak{Set, Map} doesn't even exist anymore at this point, and this can be reopened if it's still an issue when reimplemented


Update Azure_70-532_Objective_1.1.htm

Thanks for this PR. I'm restructuring and updating the practice test questions to match the latest exam objectives update from Nov 2016 (I know.. finally..) As a result, this PR is no longer applicable. I'll be reviewing and updating ALL the practice questions as part of this update process.. Thanks!


Add Skills

Adds Potent Brew and Finesse skills.


Table to produce

Example/specification of the table to produce

dandiset
version
schema_version
api_status
modified
pydantic
linkml

000727
0.240106.0043
0.6.4
Valid
2024-01-06T00:43:28.815333Z
4 (1 value_error, 3 string_pattern_mismatch)
14 (1 + 13)

000727
draft
0.6.4
Invalid
2024-05-18T17:08:27.003540Z
???

[x] "api_status" as API returns in the listing of dandisets, e.g. "Valid" in both cases here:
"most_recent_published_version": {
  "version": "0.240106.0043",
  "name": "Mapping the Neural Dynamics of Locomotion across the Drosophila Brain",
  "asset_count": 18,
  "size":<PHONE_NUMBER>98,
  "status": "Valid",
  "created": "2024-01-06T00:43:26.749476Z",
  "modified": "2024-01-06T00:43:28.815333Z"
},
"draft_version": {
  "version": "draft",
  "name": "Mapping the Neural Dynamics of Locomotion across the Drosophila Brain",
  "asset_count": 18,
  "size":<PHONE_NUMBER>98,
  "status": "Invalid",
  "created": "2023-12-10T10:22:46.559944Z",
  "modified": "2024-05-18T17:08:27.003540Z"
}

[x] if no most_recent_published_version -- only draft

[x] "modified" - also from API

[x] if draft and published version have the same "modified" and "status", then do not produce DRAFT version row

[x] schema_version -- the version of schema from metadata.json schemaVersion

[x] for linkml validation outputs, check "message" and do following transformation:

apply following regex substitution (here in sed format): "s,.*\(is .*\) in \S.*,\1,g" which ATM produces across all dandisets:

❯ grep -h '"message"' validation-reports-dandi/*/linkml_validation_errs.json | sed -e "s,.*\(is .*\) in \S.*,\1,g" | sort -n | uniq -c | sort -n
      1 is not valid under any of the given schemas
      1     "message": "Additional properties are not allowed ('@context', 'datePublished', 'publishedBy' were unexpected) in /",
    179     "message": "Additional properties are not allowed ('@context', 'datePublished', 'doi', 'publishedBy' were unexpected) in /",
   1399 is not of type 'string'

[x] and then for linkml reference, just count the number of such different errors. Most of the time it would be 2 ATM since they all have that "is not of type string" and "Additional propert

this regex "s,.*\(is .*\) in \S.*,\1,g" is the "GNU sed" regex,  so it escapes the group etc, in python it would be the difference, not sure in OSX

All features requested for the table are complete in https://github.com/dandi/dandisets-linkml-status/pull/1. However, I want to remove the following one. The required condition in the feature is never true for any of the existing dandisets.

[x]  if draft and published version have the same "modified" and "status", then do not produce DRAFT version row

The modified timestamp for a draft version seems to be never the same as the corresponding latest published version. Take a look at the "000017" dandisets @ https://github.com/candleindark/dandisets-linkml-status/blob/init/dandi-reports/summary.md, and you will that the draft version is essentially the same as the latest published version, but they have different timestamp.
In place of the condition you suggested, we may just take away a draft version if it has the status of Published. However, my recommendation would be not taking away any draft version at all. Even for a draft version with the status of Published,  its metadata is not exactly the same as the metadata of the corresponding latest published version.

FTR: (per @yarikoptic's approval) The outputting of validation reports on draft version is now unconditional, i.e. the validation reports of the draft version of all dandiset will be outputted.

@yarikoptic I am closing this issue. All requested features for the summary table in this issue has been implemented. Additional feature requests for the table can be done by filing a new issue, such as https://github.com/dandi/dandisets-linkml-status-tools/issues/14.


SAK-41271 Fixes some broken wiki macros

@smarquard The first commit looks good, but the second one has some leakage it seems. Should the maven wrapper changes be coming through in this PR?

Hm I thought it looked ok as the diff showing in  https://github.com/sakaiproject/sakai/pull/6515/files is right. Do I need to rebase or will a squash & merge take care of it?

It'll be alright looking at that diff. I always get twitchy when I see that happen and I just rebase then force push my branch.


Revisit the caching in DateFinder and TitleFinder

The way the caches are implemented in DateFinder and TitleFinder makes caching implicit, and makes it incredibly hard to use them in a thread safe way. The cache should probably be removed from these classes (or at least made more explicit) to ensure they can be used correctly. At the very least, there shouldn't be a global cache as different usage scenarios will have different cache needs.

Perhaps the better answer is to cache the FileInfo classes.

This was addressed as part of #166 which moves the caching into the DirectoryObserver.


Standardize blank lines to no more than 1 or 2

The opinion of some formatters for other languages

https://prettier.io/docs/en/rationale.html#empty-lines
https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#empty-lines

Both leave empty lines from original format, but collapse them to a standard number of lines (ie. 0 empty lines remain 0, 7 or 6 or 1 empty lines all become eg 1 empty line)
Black goes even further and adds empty lines in some places (which I think you could consider doing eg around CTEs for readability)
Originally posted by @nfcampos in https://github.com/tconbeer/sqlfmt/discussions/230#discussioncomment-3311683

My proposal is:

No more than 2 consecutive blank lines of depth (0,0). In other words, we'll allow 2 blank lines in between statements (e.g., between a {{ config }} block and select, which seems popular)
No more than 1 consecutive blank line at any other depth.


Select GCC_VER doesn't work.

If that option is activated we get: make: *** No rule to make target 'gcc-7.2.0', needed by 'install'.  Stop.
See: https://travis-ci.org/github/messense/rust-musl-cross/jobs/638295318
I have tried GCC_VER = 7.2.0 and more.
https://github.com/messense/rust-musl-cross/issues/27

GCC 7.2.0 support was removed in January:

https://github.com/richfelker/musl-cross-make/commit/49f489e5c6ece5ebaf6b01542f9672dcea99495b

If you're using master, you can glean what's supported from the patches and hashes directories:

https://github.com/richfelker/musl-cross-make/tree/master/patches
https://github.com/richfelker/musl-cross-make/tree/master/hashes

Thank you for the answer!


#5201: plemented StringTemplate Caching based on location in source Code.

Changed the earlier implementation of caching the StringTemplates based
on Raw String Literals to their location in the source code.
Old Behavior:
function getCallsite(c) { return c; }
function getFooCallsite() { return getCallsitefoo; }
print(getFooCallsite() === getFooCallsite()); // true
print(getCallsitefoo === getCallsitefoo); // true
print(getCallsitefoo === eval('getCallsitefoo')); // true
New Behavior:
function getCallsite(c) { return c; }
function getFooCallsite() { return getCallsitefoo; }
print(getFooCallsite() === getFooCallsite()); // true
print(getCallsitefoo === getCallsitefoo); // false
Deleted the added mapping that would compare and ensure that the two callsite objects are equal based on their raw String literals.
Now it store every diferent pnode object it comes accross and assigns it a register.

Make sure you put Fixes #.... in your PR title and/or description to automatically close your PR when it is merged: https://help.github.com/articles/closing-issues-using-keywords/


Chain-specific Safe addresses

What it solves
Resolves #464
How this PR fixes it
Adding the getChainSpecificDefaultSaltNonce which appends the given chainId to the PREDETERMINED_SALT_NONCE. This results in different default addresses for the Safe contract on different chains.
Example
Deployed two Safes with identical parameters on two different chains without defining a custom salt nonce. Received different addresses:

Görli: 0x6a724ffd8bab7ce14144cbfafa44994121ec8b16
Mumbai: 0x0c78086f0b398d044e54dcfdf86387942ef05713

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.4%) to 81.333%

Totals

Change from base Build<PHONE_NUMBER>:
4.4%

Covered Lines:
55

Relevant Lines:
63

💛  - Coveralls

I have read the CLA Document and I hereby sign the CLA


Remove display of tags

I want to use my own presentation of displaying tags. What is the best way to remove the default presentation of displaying tags? Right now I am using this, but I don't know if there's a cleaner way?
<template slot="tag" slot-scope="">

</template>

Also a problem that too much tags make component grow horizontally. It is not well idea, because you can't always have a design with flexible height of the component.


@strawberry.group("Admin")

I am unable to group mutations and queries based on namespaces

Hi @jediiry can you write more about your use case?

import strawberry
from .schema import Admin
from strawberry import field
from Directives.directives import UserRole, HasRoleDirective
@strawberry.type
class AdminQuery:
@field(extensions=[HasRoleDirective(UserRole.ADMIN)])
@strawberry.group("Admin")
def admin(self) -> Admin:
return Admin(name="Samuel", age=100)
@field(extensions=[HasRoleDirective(UserRole.ADMIN)])
def adminOne(self) -> Admin:
    return Admin(name="Samuel", age=100)

@jediiry how does the grouping affect the schema?

@patrick91 it helps to group my mutations and queries in graphql playground into admin and users.

@jediiry sorry for asking so many questions :D
is there any docs on grouping for GraphQL playground? I have never seen this feature 😊

 This is what I am trying to achieve

I see! That's something you can do like this, I think:
import strawberry
from .schema import Admin
from strawberry import field
from Directives.directives import UserRole, HasRoleDirective

@strawberry.type
class AdminQuery:
    @field(extensions=[HasRoleDirective(UserRole.ADMIN)])
    def admin(self) -> Admin:
        return Admin(name="Samuel", age=100)

    @field(extensions=[HasRoleDirective(UserRole.ADMIN)])
    def adminOne(self) -> Admin:
        return Admin(name="Samuel", age=100)

@strawberry.type
class Query:
    admin_group: AdminQuery

Adding a group parameter might not be easy (in terms of UX/DX and edge cases), and maybe you can implement an extension for this to make it easier if you don't like the snippet above?

@strawberry.type
class AdminQuery:
@field(extensions=[HasRoleDirective(UserRole.ADMIN)])
def admin(self) -> Admin:
return Admin(name="Samuel", age=100)
@field(extensions=[HasRoleDirective(UserRole.ADMIN)])
def adminOne(self) -> Admin:
    return Admin(name="Samuel", age=100)

@strawberry.type
class UserQuery:
@field(extensions=[HasRoleDirective(UserRole.USER)])
async def user(self) -> User:
try:
return User(name="Patrick", age=100)
except Exception as error:
return error
@strawberry.type
class Query:
AdminGroup: AdminQuery = strawberry.field(
resolver=lambda: AdminQuery())
UserGroup: UserQuery = strawberry.field(
resolver=lambda: UserQuery())
schema = strawberry.Schema(query=Query)
Finally got it to work, Thanks

I'm glad! I'll close the issue 😊


Mobile Keyboard: typed text is empty, but "length of (typed text)" returns 9 somehow

I'm trying to make a custom text box for myself.
I used the following script:

As you can see, I absolutely make sure that the text box is completely empty before I do any actions on it.
And yet, when this script runs, the text shows "undefined", and the length of the inputted text somehow immediately jumps up to 9.

I'm not sure if this is supposed to happen, but my guess is that it shouldn't happen.

That's because the extension seems to be returning undefined with no text inputted. It's returning 9 because "undefined" is 9 characters long.

The fix is for the extension would be to just return an empty string, instead of undefined.

 Huge thanks, you helped me hit my coding deadline 💖

No problem

This problem has been solved 👍

It hasn't, I haven't made the fix yet. Thanks for the comment, it reminded me.


Hi，When will the code be released? Thank you!

Hi，When will the code be released? Thank you!

We will release the code before April 20, thank you~

Hi, I find in '3_modify_activation.py', the code
                _, logits = model(input_ids=input_ids, attention_mask=input_mask, token_type_ids=segment_ids, tgt_pos=tgt_pos, tgt_layer=0, imp_pos=kn_bag, imp_op='remove')
why the tgt_later is always 0, in kn_bag, some neurons is [9,1000] or [10,1001], not always in layer 0.


Evaluation of prefix expression

Would like to add an algorithm to evaluate prefix expression in C++.
Please assign this to me under gssoc2021.

@raksha009  , please assign this issue to me , i want to implement it in c++


Cannot mount in Arenas

Environment

Addon version: v3.4.19
WoW version: 3.4.0 (build 46248)

Steps to reproduce

Attempt to use keybind to mount in an Arena

Expected Behaviour

Character casts and is successfully mounted

Actual Behaviour

Addon throws an error and character is left unmounted; No cast is ever performed.

Debug Output
72x MountsJournal\Mounts-Mounts.lua:482: Usage: local info = C_Map.GetMapInfo(uiMapID)
[string "=[C]"]: in function `GetMapInfo'
[string "@MountsJournal\Mounts-Mounts.lua"]:482: in function `setMountsList'
[string "@MountsJournal\Mounts-Mounts.lua"]:603: in function `setFlags'
[string "@MountsJournal\mixins/MacroMixin.lua"]:375: in function `getMacro'
[string "@MountsJournal\mixins/MacroMixin.lua"]:454: in function <MountsJournal\mixins/MacroMixin.lua:451>

Locals:
(*temporary) = nil

Screenshots
N/A

Hello, thx for feedback. Fixed, check pls.

No longer having the issue, thanks!


[utils] Add option to control utils installation

Previous behaviour preserved for those using package-release.sh script.

What is the point?

By default ninja install command will install setup script into $prefix/bin directory.
With option it's easier to control if custom setup script used, or no setup script needed. Especially when ninja install used to install files directly into system.

Ok, maybe I should extend my answer.
I think the only reason all this cp .../x.dll .../y.dll constructions exist in build-scripts is that DXVK always install setup_dxvk.sh into bin/ directory.
While writing this text I realized that winelib and mingw installation differs, and while winelib libraries are installed into $libdir, mingw dlls installed into $bindir and *.a static libs into $libdir. This also can be controlled w/ e.g. static_libs option for those don't want them to be installed.
The point is to use common practice, when you have to properly configure meson build and then use ninja install .
Something like this (simplified example for Proton)
meson --strip --buildtype="release" --prefix="$DST_DIR" --bindir=lib64/wine/dxvk ...
cd "$TOP"/build/dxvk.win64
ninja install

Fixed this PR with following changes:

enable_utils option set to true by default, so nothing will change for e.g. https://haagch.frickel.club/files/dxvk/latest/ and those who expect setup_dxvk.sh still there
descriptions added for meson options, will be visible using $ meson configure command in current build directory

Also I wanted to add 'install_implibs option, to control import libraries installation (lib/libd*.dll.a), but I have no MinGW to test this. This can be done using implib keyword argument (libd3d11.dll.a.patch.txt), or by adding custom targets with build_by_default: false keyword argument and install: get_option('install_implibs'), for MinGW builds.

Nevermind, this issue don't worth more time than it deserves.


Add extract_all_binaries.js

This file is used as part of the build process, and replaces "node-gyp rebuild" in package.json when installing appmetrics from npm.

fixes #300

LGTM


Hide overlapping points in supercluster, markers with colored tex

I love dash-leaflet because it avoids me having to use JS too much :)
Unfortunately my apps are becoming more and more complex and I'm starting to believe that the only way of taking full advantage of all the plugins/customization available for leaflet is to start building the app directly into JS without having to port every feature in Python :(
But I'm not done yet!
I'm trying to build something really simple, that is a scatter plot of temperature measurements overlaid on a satellite/radar background updated in real time. I managed to do something acceptable (see attached figure) but I'm struggling to bring this even one step further.

More precisely these are the things that I would like to change but could not find how to do it:

instead than creating a superCluster of points within a certain radius using a symbol that shows how many points are clustered and that expands on the click action, I would like to just hide overlapping points and show just 1 point per superCluster defined with a certain radius. This will depend, of course, on the zoom level.  I could not find any way to do this in leaflet but I guess this is officially supported..no?
instead than using markers colored according to colorProp I would like to show only text showing the value of colorProp with a textfont color given also by colorProp. So basically a map of temperature values shown by text colored according to the value itself. It sounds more complicated than it actually is :D

Is there a way to do this? Do I need to use the tooltip and to "lock" it in place as I found on some stackoverflow answers? Still it is not clear how I would implement something like this on the dash side.

Thanks for the really nice work!

While you can only get the full flexibility by moving to JavaScript, I think what you are currently trying to do should be possible with dash-leaflet.

To disable zoom interaction, set zoomToBoundsOnClick=False
To "hide" points instead of rendering clusters, I guess you could simply set the rendering of points and clusters to be the same (from a UI perspective, not the same function). If you do so, can't see what is a point/what is a cluster, but the clusters will still expand on zoom, which is guess is what you want
To render numbers instead of dots, set the pointToLayer property for an appropriate function. And set the clusterToLayer prop to a similar function, where the numbers is based of e.g. the mean, or whatever makes the most sense in you application

Sorry for the late reply, I only had time today to try it out...
Unfortunately the use of the pointToLayer and clusterToLayer functions for me is still a little bit confusing, maybe because I'm not an expert of JS and Leaflet. The only way that I found to show text together with the points was to bind the tooltip like this
window.myNamespace = Object.assign({}, window.myNamespace, {
    mySubNamespace: {
        pointToLayer: function(feature,latlng){
            label = String(feature.properties.name)
            return new L.CircleMarker(latlng, {
              radius: 0.1,
            }).bindTooltip(label, {permanent: true,
                direction: "center",
                className: "my-labels"}).openTooltip();
            }
    }
});

And I get something really ugly like this

Unfortunately it is not really clear for me how can I change the text color according to the "value" of the tooltip. The geojson component has a colorProp parameter that acts on the scatter so I thought I could use that one also for the text but it seems that it has no effect. Do I really need to define my own function that does the mapping between values - colormap and then set manually the text color in the style?
Anyway I think these are questions more for the leaflet folks so I will annoy them and leave you alone :)
But before going I did notice something weird.
If I start my application using the namespace defined in the assets folder with the embedded server (python app.py) then everything works fine. However, if I start the app with gunicorn app:server then I get the following error in the console
no match for [myNamespace.mySubNamespace.pointToLayer] in the global window object
Any idea what could be the problem?

Sorry, I finally understood how the all things work, especially after I found out where the default namespaces for the scatter are defined here https://github.com/thedirtyfew/dash-leaflet/blob/f8823bca5f7afb0fdc99c80d5105485b72d8a8e8/src/lib/express.js#L64-L93
I just adapted this to only have markers colored with the temperature and permanent tooltips for the text part.
Furthermore I adapted the clusterToLayer function to show only 1 point per cluster. The two functions now look like something like this
window.myNamespace = Object.assign({}, window.myNamespace, {
    mySubNamespace: {
        pointToLayer: function(feature, latlng, context){
            label = String(feature.properties.name)
            const {circleOptions, colorProp} = resolve(context.props.hideout, scatterDefaults,
                ["circleOptions", "colorProp"])
            circleOptions.fillColor = getColor(context.props.hideout, feature.properties[colorProp]);
            return new L.CircleMarker(latlng, circleOptions).bindTooltip("" + label, {
                permanent: true,
                direction: "center",
                className: "my-labels"
            }).openTooltip();
        },

        clusterToLayer: function (feature, latlng, index, context) {
            const {circleOptions, colorProp} = resolve(context.props.hideout, scatterDefaults,
                ["circleOptions", "colorProp"])
            // // Set color based on mean value of leaves.
            const leaves = index.getLeaves(feature.properties.cluster_id);
            // Choose just one value to be representative
            leave = leaves[0]
            label = String(leave.properties[colorProp].toFixed(0))
            circleOptions.fillColor = getColor(context.props.hideout, leave.properties[colorProp]);
            return new L.CircleMarker(latlng, circleOptions).bindTooltip("" + label, {
                permanent: true,
                direction: "center",
                className: "my-labels"
            }).openTooltip();
        }

    },
});

I know, it's definitely not clean but it works :) I will optimize it in the future.

Here is a demo
https://user-images.githubusercontent.com/12760310/112756346-07701200-8fe5-11eb-9720-ae160b400593.mov
P.S. the problem with namespace loading with gunicorn was due to the fact that I was not using __name__ when initializing the Dash app :)

That looks nice! Thanks for the demo ;)

Sorry, last comment. I just figured out that doing with the scatter is much better than binding the tooltips (the UI is much faster). here is the latest version
        pointToLayer: function(feature, latlng, context){
            const {circleOptions, colorProp} = resolve(context.props.hideout, scatterDefaults,
                ["circleOptions", "colorProp"])
            label = String(feature.properties[colorProp].toFixed(0))
            const icon = L.divIcon.scatter({
                html: '
' + label + '
',
                iconSize: L.point(20, 20),
                className: "marker-modified",
                color: getColor(context.props.hideout, feature.properties[colorProp])
            });
            return L.marker(latlng, {icon : icon})
        },

        clusterToLayer: function (feature, latlng, index, context) {
            const {circleOptions, colorProp} = resolve(context.props.hideout, scatterDefaults,
                ["circleOptions", "colorProp"])
            const leaves = index.getLeaves(feature.properties.cluster_id);
            // Choose just one value to be representative
            leave = leaves[0]
            console.log(leave)
            label = String(leave.properties[colorProp].toFixed(0))
            const icon = L.divIcon.scatter({
                html: '
' + label + '
',
                iconSize: L.point(20, 20),
                className: "marker-modified",
                color: getColor(context.props.hideout, leave.properties[colorProp])
            });
            return L.marker(latlng, {icon : icon})
        }

Only thing that I still didn't manage to do is to activate the tooltips for the points of the clusters. Right now they appear as normal points but they're not and it appears that Leaflet does not support hover tooltips for superclusters so...I'll have to invest some time to figure it out :)


[Subscription] catch up latest API version

Resource Provider

Subscription. https://github.com/Azure/azure-cli-extensions/tree/master/src/account
Description of Feature or Work Requested

Service team asks to catch up latest API version. https://github.com/Azure/azure-rest-api-specs-pr/tree/master/specification/subscription/resource-manager/Microsoft.Subscription/preview/2019-10-01-preview
Minimum API Version Required

Swagger Link

https://github.com/Azure/azure-rest-api-specs-pr/tree/master/specification/subscription/resource-manager/Microsoft.Subscription/preview/2019-10-01-preview
Target Date

catch up latest API version


Ignore for some views

Autoformatted the code using 'black' autoformatter
Removed the test shortcut, t.i. the tests are now running through the debouncer
Ignore scratch, read-only views, and widgets.

That is basically so that the plugin does not annotate git diff views or error panels from SublimeLinter/LSP etc.

Don't know if you will pull this ... everyone hates this reformatting stuff on PR's ... sorry ... but if look at the commits using 'no whitespace' diff it is easy to spot that I don't cheat here, and really just used a formatter tool. 😁


[scripts-audit] add guidelines for cmake

Any hint about the min version of CMake scripts must be compatible with / the version scripts may assume without checks?

How about not allowing
set(VAR )

because it is a common trap (in particular with initialization of lists) that the real intention is
set(VAR "")

but the actual effect is
unset(VAR)

which unhides cache variables. So there are two explicit alternatives which are not ugly.
I would also encourage use of
string(APPEND VAR " xxx")

over
set(VAR "${VAR} xxx")

because it makes accidental
set(VAR "${VAR};xxx")

stand out more clearly.

set(VAR) is used to create an empty variable in the local/current scoped thus hiding any higher scoped variables. Setting it to an empty string instead is rarely meant and doesn't play nice with list(APPEND)

set(VAR) is used to create an empty variable in the local/current scoped thus hiding any higher scoped variables. Setting it to an empty string instead is rarely meant and doesn't play nice with list(APPEND)

This is what I though until recently. But it is wrong, and that's why I'm bringing it up:
$ cat > demo.cmake << END_CMAKE
set(FOO "Initial")
set(FOO "Cached value" CACHE STRING "doc")
set(FOO )
message(STATUS "\${FOO}")
list(APPEND FOO "bar")
message(STATUS "\${FOO}")
set(FOO "")
list(APPEND FOO "bar")
message(STATUS "\${FOO}")
END_CMAKE
$ cmake -P demo.cmake
-- Cached value
-- Cached value;bar
-- bar

After adding one more message(STATUS "${FOO}"), it turns out that (at least) in script mode, the non-cache variable (aka "set" binding) doesn't even hide the cached variable (aka cache entry). This is in contradiction to cmake documentation. Oh no.

your example should use at least one scope change
cache variables in script mode are kund of funky since there is no concept of cache in script mode(no CMakeCache.txt to store them). So all Cache variables become priority read access.

your example should use at least one scope change

The result doesn't change when I wrap the code in function(...) ... endfunction().

cache variables in script mode are kund of funky since there is no concept of cache in script mode(no CMakeCache.txt to store them). So all Cache variables become priority read access.

I'm aware that there is no persistent cache in script mode but there are still the different types of variables.
Variable evaluation is documented cmake behaviour, regardless of script mode.
The documented behaviour of set(VAR ) is unsetting (actually creating an "unset" binding), regardless of scope.
I opened https://gitlab.kitware.com/cmake/cmake/-/issues/22285.

I opened https://gitlab.kitware.com/cmake/cmake/-/issues/22285.

A fix is already merged for CMake 3.21, and there will be a new policy, CMP0126.
My original point stands: set(VAR ) does NOT create an empty variable, and so it doesn't hide anything.

[ ] REGRESSION: opencv4:arm-uwp: There is not enough space on the disk.
[ ] REGRESSION: qt5-sensors:x64-windows-static-md: Source download issue
[ ] REGRESSION: qt5-scxml:x64-windows-static-md: Source download issue
[ ] REGRESSION: otl:x64-linux: The hash changed, need update it.

/azp run

/azp run

what about guidelines for vcpkg-cmake-wrapper.cmake?
https://github.com/microsoft/vcpkg/pull/18393#issuecomment-861263579 shows that it seems to be required

what about guidelines for vcpkg-cmake-wrapper.cmake?

Fair, but the dominant problem is that documentation is completely missing vcpkg-cmake-wrapper.cmake and vcpkg-cmake-config.cmake.

@dg0yt @Neumann-A that would be good guidelines, but these are meant more for the scripts under the scripts tree.

Tensorflow:x64-osx failure would be fixed by PR https://github.com/microsoft/vcpkg/pull/18812

Closed for rollup PR #18838


Need Jenkins CI to test OE Windows with DCAP libraries

PR #1865 once merged to master will provide a way to build and test ELF enclaves on Windows using DCAP libaries when the build flag USE_LIBSGX is enabled. Need Jenkins CI stages to provide capability to build and test.

@ionutbalutoiu - PR #1865 is in master. Please work on this issue.

@ionutbalutoiu - PR #1865 is in master. Please work on this issue.

Awesome! We'll come with a PR for this as soon as possible.

Syncing the issue milestone with the pull request milestone as changed by Bruce.
/cc @yakman2020

Fixed by https://github.com/openenclave/openenclave/pull/1945


Update nuget packages

Signed-off-by: Yury-Fridlyand<EMAIL_ADDRESS>Description
Abstractions are taken from https://github.com/opensearch-project/opensearch-net-abstractions/pull/19
Direct System.Net.Http dependencies were updated in scope of https://github.com/opensearch-project/opensearch-net-abstractions/pull/15. Dependencies listed below are indirect dependencies -- dependencies of the dependencies, dependencies  of the dependencies of the dependencies and so on.
I have to add extra dependency for Microsoft.NET.Test.Sdk because xunit was updated, see more.
Issues Resolved

Title
Vulnerability
Module
Version
Severity

Improper Neutralization, .NET Core Remote Code Execution Vulnerability
CVE-2021-26701 GHSA-ghhp-997w-qr28 GSD-2021-26701
System.Text.Encodings.Web
4.6.0
Critical

Improper Neutralization, .NET Core Remote Code Execution Vulnerability
CVE-2021-26701 GHSA-ghhp-997w-qr28 GSD-2021-26701
System.Text.Encodings.Web
4.5.0
Critical

Improper Certificate Validation, Security Feature Bypass
CVE-2017-0248 GHSA-ch6p-4jcm-h8vh GSD-2017-0248
System.Net.Http
4.3.1
High

Improper Input Validation, Elevation of Privilege
CVE-2017-0249 GHSA-qhqf-ghgh-x2m4 GSD-2017-0249
System.Net.Http
4.3.1
High

Improper Input Validation, Denial of Service
CVE-2017-0247 GHSA-6xh7-4v2w-36q6 GSD-2017-0247
System.Net.Http
4.3.1
High

Exposure of Sensitive Information to an Unauthorized Actor, .NET Core Internals
CVE-2018-8292 GHSA-7jgj-8wvc-jh57 GSD-2018-8292
System.Net.Http
4.3.1
High

Spoofing, vulnerability affects Microsoft.AspNetCore.Mvc
CVE-2017-0256 GHSA-j8f4-2w4p-mhjc GSD-2017-0256
System.Net.Http
4.3.1
Moderate

Uncontrolled Resource Consumption, Regular Expression Denial of Service
CVE-2019-0820 GHSA-cmhx-cq75-c4mj GSD-2019-0820
System.Text.RegularExpressions
4.3.0
High

Combining this PR and https://github.com/opensearch-project/opensearch-net-abstractions/pull/19 .Net client are cleaned of the listed vulnerable package except the last one.
The dependency chain:
Tests.ScratchPad - BenchMarkDotNet v.0.13.1 - Microsoft.CodeAnalysis.CSharp v.2.10.0 - Microsoft.CodeAnalysis.Common v.2.10.0 - System.Xml.ReaderWriter v.4.3.0 - System.Text.RegularExpressions v.4.3.0
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Combining this PR and opensearch-project/opensearch-net-abstractions#19 .Net client are cleaned of the listed vulnerable package except the last one. The dependency chain: Tests.ScratchPad - BenchMarkDotNet v.0.13.1 - Microsoft.CodeAnalysis.CSharp v.2.10.0 - Microsoft.CodeAnalysis.Common v.2.10.0 - System.Xml.ReaderWriter v.4.3.0 - System.Text.RegularExpressions v.4.3.0

This CVE will not affect end users. Tests.ScratchPad is not part of the client library and is not published.

@dblock thank you for merging CVE PR on abstractions repo!
Would you have a moment to look at this one as well?


[ISSUE #1558]module acl compatible with 1.6

What is the purpose of the change
fix #1558 module acl compatible with 1.6

@geektao could you fix this PR based on the CI result?

Good job! And please check the CI result.If the CI result is ok, I think we can merge this pr! @duhenglucky @geektao

Coverage increased (+0.08%) to 50.921% when pulling<PHONE_NUMBER>5257d5c47adf0d19594170ce383a18 on geektao:1558 into dc815cc581f9e9dd36efa926fc7087ff1ecbf1a6 on apache:develop.

Codecov Report

Merging #1920 into develop will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #1920      +/-   ##
=============================================
- Coverage      45.13%   45.06%   -0.08%
+ Complexity      4197     4190       -7
=============================================
  Files            544      544
  Lines          35727    35733       +6
  Branches        4744     4748       +4
=============================================
- Hits           16125    16102      -23
- Misses         17558    17587      +29
  Partials        2044     2044

Impacted Files
Coverage Δ
Complexity Δ

.../java/org/apache/rocketmq/acl/common/AclUtils.java
83.12% <100.00%> (ø)
66.00 <0.00> (ø)

...ava/org/apache/rocketmq/acl/common/Permission.java
91.89% <100.00%> (+1.26%)
20.00 <0.00> (ø)

...apache/rocketmq/acl/plain/PlainAccessResource.java
73.33% <100.00%> (ø)
28.00 <0.00> (ø)

...che/rocketmq/acl/plain/PlainPermissionManager.java
70.23% <100.00%> (ø)
39.00 <6.00> (ø)

...cketmq/acl/plain/RemoteAddressStrategyFactory.java
91.20% <100.00%> (ø)
14.00 <0.00> (ø)

...org/apache/rocketmq/common/stats/StatsItemSet.java
47.52% <0.00%> (-11.89%)
12.00% <0.00%> (-3.00%)

...tmq/logappender/log4j2/RocketmqLog4j2Appender.java
35.00% <0.00%> (-10.00%)
3.00% <0.00%> (-1.00%)

...apache/rocketmq/remoting/netty/ResponseFuture.java
85.00% <0.00%> (-5.00%)
16.00% <0.00%> (-2.00%)

...in/java/org/apache/rocketmq/test/util/MQAdmin.java
40.00% <0.00%> (-4.45%)
7.00% <0.00%> (-1.00%)

...rocketmq/remoting/netty/NettyRemotingAbstract.java
46.69% <0.00%> (-4.05%)
20.00% <0.00%> (-2.00%)

... and 8 more

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 dc815cc...8633609. Read the comment docs.


SectionName to be used from section id

Is there a way to pull section name for URL from section id?
Precisely I'm using Divi theme and trying to set this, but with no luck.
Any way to get horizontal scroll soon?

I had to implement something similar for a mini-CMS, so I slightly modified the scrollify script (see attachments)

Replace the original scrollify.js with the variation in the attachments
Define the hashSuffix option in the scrollify settings (has to be inside $(document).ready()):
$(document).ready(function () {  $.scrollify({      section: ".scrollSnap",      sectionName: "section-name",      hashSuffix: "_section"  }); });

This will change the hash only on sections with the suffix "_section" and set the section name to the section id without the suffix.
For example:

Changes the hash to "home" whereas

will not.
Sections without the suffix in the section id will still be scrollified.
If you don't define a suffix, the feature will be ignored.
scrollify-1.0.19.zip


A link to the documentation page seems to be broken

I remember I was able to navigate to the deployed documentation page before but.
It seems that I lost the ability though and get a 404 error. Has the deployment location been changed?

As far as I know, we are abandoning the pocketnet.app domain
It would be more correct to use https://bastyon.com/documentation
But it doesn't work either.. I will discuss with my colleagues where these links should lead, but I think that the most logical place is docs.bastyon.com

Thank you! https://docs.bastyon.com/ seems like the right place to be navigated to. I will create a PR with a change to the README.md file in a few.


Transaction outputs doubled in wallet after 'Repair wallet' - balance is then wrong

I just did a 'repair wallet' on a perfectly ok wallet (just to try it out).
The balance was higher after the repair wallet and a send failed with:
{"@timestamp":"2015-05-14T17:22:23.606+01:00","@version":1,"message":"Could not commit the transaction","logger_name":"org.multibit.hd.core.services.BitcoinNetworkService","thread_name":"safe-fixed-bitcoin-network-2","level":"ERROR","level_value":40000,"stack_trace":"org.bitcoinj.core.VerificationException$DuplicatedOutPoint: Duplicated outpoint\n\tat org.bitcoinj.core.Transaction.verify(Transaction.java:1239) ~[multibit-hd.jar:na]\n\tat org.bitcoinj.core.Wallet.maybeCommitTx(Wallet.java:2226) ~[multibit-hd.jar:na]\n\tat org.bitcoinj.core.Wallet.commitTx(Wallet.java:2291) ~[multibit-hd.jar:na]\n\tat org.multibit.hd.core.services.BitcoinNetworkService.commit(BitcoinNetworkService.java:1057) [multibit-hd.jar:na]\n\tat org.multibit.hd.core.services.BitcoinNetworkService.performCommitAndBroadcast(BitcoinNetworkService.java:392) [multibit-hd.jar:na]\n\tat org.multibit.hd.core.services.BitcoinNetworkService.performSend(BitcoinNetworkService.java:371)

Screenshots before and after (notice the balance has changed although there have been no sends or receives).

Possibly the same unspent transaction output is appearing in the wallet twice ???
p.s. After another repair wallet the wallet balance increased again so the same tx is probably being added repeatedly. Screenshot:

This was caused by the set of unspent tx "myUnspent" in Wallet not being cleared when the wallet was being repaired. Fixed.
There is an update to bitcoinj-alice which is pushed to the staging repo.
Awaiting review and closing.

Reviewed bitcoinj code covering this and it looks fine. Removed some logging that flooded the logs during repair since we are limited to 200Kb and Repair is likely to occur if someone is encountering problems.
Closing.


Package is un-go-gettable due to reference to old package name

This fixes a reference to acme as the old project name in Godeps/ rather than the current letsencrypt.
Looks like you missed some of it in #3.

Thanks! I've gone ahead and used godep to correct that paths. Let me know if you have any problems.


Rendering not interactive

Hello,
I've found some odd behavior since I reinstalled qgrid on a fresh miniconda(py27) instalation.
My notebook works fine right until the show_grid command, which renders nothing except if I use the
option {'enableAddRow': True} which renders the add/remove row and export table buttons. If I click on the export, it renders the table and I can filter and scroll through it, but can't edit the entries even if I specifically add the {'editable':True} keyword to the options.
My relevant packages from conda list are:
ipykernel                 4.6.1                    py27_0    conda-forge
ipython                   5.4.1                    py27_0    conda-forge
ipywidgets                7.0.0                    py27_1    conda-forge
notebook                  5.0.0                    py27_0    conda-forge
pandas                    0.20.3                   py27_1    conda-forge
some noteworthy jupyter notebook --debug lines are:
[NotebookApp] Loaded config file: $home/miniconda3/envs/iceage/etc/jupyter/jupyter_notebook_config.json
[NotebookApp] Loaded config file: $home/.jupyter/jupyter_notebook_config.json
[NotebookApp] 404 GET /nbextensions/widgets/notebook/js/extension.js?v=20170913215009 (<IP_ADDRESS>) 4.02ms referer=the qgrid calling notebook

This probably caused by https://github.com/quantopian/qgrid/issues/129, since you're using ipywidgets 7.  I have a new version which should work for you, please see my comment on that other issue for information on how to install it: https://github.com/quantopian/qgrid/issues/129#issuecomment-332706796

Yup, that was it.
pip install qgrid==1.0.0b3
jupyter nbextension enable --py --sys-prefix qgrid
worked for me since the conda install asks for python 3.
TY!


PowerToys prevents use of hotkeys in Windows Terminal

Microsoft PowerToys version
0.66.0
Installation method
Chocolatey
Running as admin
No
Area(s) with issue?
General
Steps to reproduce

Have an application use the shortcut Win+PgUp; in my case this is the Windows Terminal with the following configuration entry:{
  "actions": [
     {
         "command": {
             "action": "prevTab"
         },
         "keys": "win+pgup"
     }
  ]
}

Install PowerToys and make sure that no hotkey is set to Win+PgUp.
Attempt to use the Win+PgUp hotkey inside the application from step 1.

✔ Expected Behavior
The hotkey works as configured.
❌ Actual Behavior
The hotkey does not work.
Other Software
Windows Terminal Preview
Version: 1.16.3463.0

If I understand you correctly it does work when PowerToys is not running right?
/needinfo

That is correct. The hotkey stops working if and only if PowerToys is running.

/bugreport

Happy to: PowerToysReport_2023-01-13-10-58-38.zip

Not able to repro this, going to close. Let us know if you see this again!

I have never stopped "seeing this". The issue persists. My workaround is simply not to not use PowerToys.

Oh interesting! I'll reopen this and see if anyone else on our team can repro


YARN-11014. YARN incorrectly validates maximum capacity resources on …

…the validation API. Contributed by Benjamin Teke

Description of PR
How was this patch tested?
For code changes:

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

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
8m 29s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.2 Compile Tests _

+1 :green_heart:
mvninstall
26m 35s

branch-3.2 passed

+1 :green_heart:
compile
0m 42s

branch-3.2 passed

+1 :green_heart:
checkstyle
0m 35s

branch-3.2 passed

+1 :green_heart:
mvnsite
0m 48s

branch-3.2 passed

+1 :green_heart:
javadoc
0m 35s

branch-3.2 passed

+1 :green_heart:
spotbugs
1m 35s

branch-3.2 passed

+1 :green_heart:
shadedclient
13m 55s

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

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 34s
/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

-1 :x:
compile
0m 34s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

-1 :x:
javac
0m 34s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 30s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 21 new + 81 unchanged - 0 fixed = 102 total (was 81)

-1 :x:
mvnsite
0m 35s
/patch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

+1 :green_heart:
javadoc
0m 25s

the patch passed

-1 :x:
spotbugs
0m 35s
/patch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

-1 :x:
shadedclient
8m 16s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 36s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

+1 :green_heart:
asflicense
0m 27s

The patch does not generate ASF License warnings.

63m 50s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux b857c4f63e7b 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.2 / 91ac3f1aba345eea52e27d909fedbdb31eddff98

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

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

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

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

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

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 50s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.2 Compile Tests _

+1 :green_heart:
mvninstall
26m 41s

branch-3.2 passed

+1 :green_heart:
compile
0m 44s

branch-3.2 passed

+1 :green_heart:
checkstyle
0m 36s

branch-3.2 passed

+1 :green_heart:
mvnsite
0m 50s

branch-3.2 passed

+1 :green_heart:
javadoc
0m 37s

branch-3.2 passed

+1 :green_heart:
spotbugs
1m 37s

branch-3.2 passed

+1 :green_heart:
shadedclient
14m 15s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 48s

the patch passed

+1 :green_heart:
compile
0m 37s

the patch passed

+1 :green_heart:
javac
0m 37s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 28s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 21 new + 81 unchanged - 0 fixed = 102 total (was 81)

+1 :green_heart:
mvnsite
0m 41s

the patch passed

+1 :green_heart:
javadoc
0m 27s

the patch passed

+1 :green_heart:
spotbugs
1m 39s

the patch passed

+1 :green_heart:
shadedclient
15m 19s

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

_ Other Tests _

-1 :x:
unit
74m 58s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 31s

The patch does not generate ASF License warnings.

141m 14s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.scheduler.capacity.TestCapacitySchedulerConfigValidator

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/2/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux b1f5782caf09 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.2 / 5c1db61f4845da054ef3eab99afb10236e813958

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/2/testReport/

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

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/2/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
11m 46s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.2 Compile Tests _

+1 :green_heart:
mvninstall
29m 12s

branch-3.2 passed

+1 :green_heart:
compile
0m 42s

branch-3.2 passed

+1 :green_heart:
checkstyle
0m 35s

branch-3.2 passed

+1 :green_heart:
mvnsite
0m 47s

branch-3.2 passed

+1 :green_heart:
javadoc
0m 33s

branch-3.2 passed

+1 :green_heart:
spotbugs
1m 39s

branch-3.2 passed

+1 :green_heart:
shadedclient
15m 27s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 47s

the patch passed

+1 :green_heart:
compile
0m 38s

the patch passed

+1 :green_heart:
javac
0m 38s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 29s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 21 new + 81 unchanged - 0 fixed = 102 total (was 81)

+1 :green_heart:
mvnsite
0m 40s

the patch passed

+1 :green_heart:
javadoc
0m 24s

the patch passed

+1 :green_heart:
spotbugs
1m 42s

the patch passed

+1 :green_heart:
shadedclient
16m 36s

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

_ Other Tests _

+1 :green_heart:
unit
77m  9s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 28s

The patch does not generate ASF License warnings.

159m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/3/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 25c1430543cd 4.15.0-153-generic #160-Ubuntu SMP Thu Jul 29 06:54:29 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.2 / 374d0f01d0f9b8ef4579ae283e19ada8ae79bbf5

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~18.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/3/testReport/

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

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3772/3/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

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

This message was automatically generated.

Thanks @brumi1024 ,
Patch LGTM, committed to branch-3.2


chore(renovate): Fix ‘pinVersions’ config

Our pinVersions override is currently less specific than the dependencies.pinVersions value defined in renovate-config-seek, so this fixes it.

:tada: This PR is included in version 0.4.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FINERACT-1047 Integration test for Audit trails(Create)

Description
Integration Tests are divided into two parts:

Creation
Verifying for different routes provided, like: limit,orderby etc.

This PR deals with the first part.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[x] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[x] API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm has been updated with details of any API changes.

[x] Integration tests have been created/updated for verifying the changes made.

[x] All Integrations tests are passing with the new commits.

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@vorburger  I was a little worried about this approach though and thought of changing this to something more robust.
The problem:
We are checking count for the entries like this: Get me the count of audit trails whose resource-id is x it was activated and is of type client. Here, the same event can occur twice and hence two audits will be returned, which does not necessarily mean that the test failed(since we are checking count to be equal to one)
The solution:
Calculate the count before the action(create, update, activate) and check if count incremented by one, after the action.
Why the old approach is still correct?
Two reasons: Every entity is created for the first time, no action is performed twice.
Just that I thought it was fundamentally wrong to check count to be one(not a general case), I hope I am not too confusing. Which approach will you prefer?

@thesmallstar yes that's a very good point.. you are absolutely right, of course, this test could fail if it runs against a DB that already contains data, for example if it runs after other tests, which makes it potentially flaky and brittle.  Maybe instead of only testing that it's +1 more, you could assert on the content of the audit record itself as well?
PS: There's a bigger issue about test isolation underlying this, but we're not going to solve this here.

@vorburger I have made the suggested changes. I am not supposed to user var in this case(right?).


Wider ECS Benchmarks

Objective
As a part of evaluating #4800, at the behest of @cart, it was noted that the ECS microbenchmarks all focus on singular component queries, whereas in reality most systems will have wider queries with multiple components in each.
Solution
Use const generics to add a wider variants of existing benchmarks.

bors r+

Sorry, found out late that there was a mixed duplicate benchmark ID which caused benches to panic. The only thing that was changed was a benchmark name.

bors retry


THRIFT-5029: Fix Node.js lib entry point

The latest published thrift version (0.13.0):

does not contain index.js at the package root
has no "main" entry point in package.json.

So module request throws an exception:
require('thrift')
Thrown:
Error: Cannot find module 'thrift'
Require stack:
- <repl>
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:625:15)
    at Function.Module._load (internal/modules/cjs/loader.js:527:27)

This PR adds missed "main" field to package.json.

👍 for this PR

Same problem.
@antongolub it doesn't fix completely, because of missing also module.exports in thrift.js

PRs welcome.

I think thrift npm package was published from lib/js folder. It should be published from root folder. package.json in root folder references lib/nodejs folder which contains both nodejs and browser bindings.

Yeah, I don't believe this change of package.json in lib/js will make a difference, that is not the package.json that is used for nodeJS. Similarly @icebob, since that is browser code it doesn't have module.exports.

I think thrift npm package was published from lib/js folder. It should be published from root folder. package.json in root folder references lib/nodejs folder which contains both nodejs and browser bindings.

Also, i created a JIRA ticket 2 months ago. Any progress on this ? Thanks.
https://issues.apache.org/jira/browse/THRIFT-5039

@dcelasun @bforbis or anyone else who has npm maintainer access - can you please fix the state of thrift 0.13.0 as published to npm? It seems to have mistakenly been published using https://github.com/apache/thrift/blob/master/lib/js/package.json when all prior versions were published using https://github.com/apache/thrift/blob/master/package.json, which properly includes and references both the browser and Node.js runtimes in ways that browser and Node.js builds can both understand. I am guessing that this mistaken publishing happened first, prompting this PR, when the real fix would be to republish with the correct package.json at the root.
This has broken all Node.js projects using Thrift for the past several months - for instance, the 'hello.js' example included in the npm Readme. But everything Node trying to require 'thrift' after installing it via npm will get the same error regardless:
{ Error: Cannot find module 'thrift'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:582:15)
    at Function.Module._load (internal/modules/cjs/loader.js:508:25)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18) code: 'MODULE_NOT_FOUND' }

Can you please republish 0.13.0 immediately to correct this? It would be greatly appreciated by those of us trying to stay up to date with Node.js projects, and would make this project work again for any newbies trying to work with the examples.

Unfortunately I don't have npm access. cc @jeking3 @Jens-G

@Jens-G @jeking3 Either of you have a spare moment to take a look at this? This can't be a pull request, as it requires an npm maintainer to republish.

Need this fix!

tmp fix diy: fork --> rename --> publish to npm
npm i thrift-tmp --save

then:
rename node_modules/thrift-tmp to  node_modules/thrift

Actually, I think the version 0.13.0 is published with a totally incorrect code base, install 0.12.0 instead.

In order to reduce the pressure on this issue, I've published a snapshot of the current Apache Thrift trunk with many fixes and improvements to https://www.npmjs.com/package/@biodataanalysis/thrift-trunk
Could you please test this to see if it would suite your needs? Any feedback is more than welcome.

Hi @wadey could you please take a look at this pr and the issue https://issues.apache.org/jira/browse/THRIFT-5039 ?
The package have been broken for about 7 months.

temporary solution:
npm add git://github.com/apache/thrift.git#v0.13.0


Hello，I want to know the discrete environment configration

Hello，I want to know the discrete environment configration to run the pointnext code.Could you supply to me?Thanks.

source install.sh


Return module name at end of chain

Return module name at end of chain so that so that it can be used in es6 style code without a literal string module name.
For example:
import ngStorage from 'ngstorage';

angular.module('myApp', [ngStorage]);

Looks like the CI tests failed at installing phantomJS


Drop deployment target to 9.3 from 10.0

Drop deployment target to 9.3

Been testing this on my phone, works well from my limited testing. Looks like it's fixed some of my crashes

I'm also testing on iOS 12 but with the deploy target of 9.3, so I'm essentially just testing that this builds and runs with sub-10 iOS deploy target


Feature/dashlocal

Description
Adding --local flag to opta local.
This will allow developers to test their services on a local cluster before they release the code via opta to the public cloud provider.
Safety checklist

[X ] This change is backwards compatible and safe to apply by existing users
[ X] This change will NOT lead to data loss
[ X] This change will NOT lead to downtime who already has an env/service setup

How has this change been tested, beside unit tests?
manual testing

@bigbitbus looks like the pipfile lock changed even though the pipfile did not. Did you forget to commit the changes to the pipfile?

Not sure why pipfile changed (I rebased on dev branch, so that may have come from there because dev has recent package changes). I have not added a package, although I recreated my pip environment locally


Allow wider range of dim values

Dim value can be set between 0.1 to 0.8
Could we get a dim value of 0, so to only show the active part of the gauge? Should be technically feasible?
If yes and while changing, allowing up to 0.9 could be changed as well. Allowing 1 would not make sense for sure, but there might be a use case for 0.9 for some colors?

0 - 0.9 Since v0.1.5

thanks


Use keyframes instead of literal height

Instead of setting a height literally, we can use keyframes to animate the height which will mean we don't need to do the whole set auto dance.
shadcn has a good implementation of this using tailwind.
source: https://ui.shadcn.com/docs/components/accordion

On second thought, key frames can't be written inline so I would likely need to use elm-css which I prefer to avoid.


feat(package): Move nsp check from prepare step to an optional script.

prepare will now output a message telling the user how to run nsp.

fixes #12

Pull Request Test Coverage Report for Build 38

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

Totals

Change from base Build 37:
0.0%

Covered Lines:
12

Relevant Lines:
12

💛  - Coveralls


Unknown top level mime type: multipart/related

I got the error when parsing an inbound email
Unknown top level mime type: multipart/related

Try https://github.com/OfimaticSRL/parsemail


enhance scalping strategy

Now that the simple base strategy is functioning, let's try to improve it.

add additional fields to the transactions table for the following (done)

strategy name
the exchange used
the value of the transaction in USD

Create a new BarryMultiOrderStrategy, and have these parameters in the strategy configuration:

max-concurrent-sell-orders - max number of outstanding sell orders that can be in existence at any one time. If the market falls and never recovers, then this (times the amount of each order) will be the most that we can lose.
percent-change-threshold - if the market goes down by this percent, issue buy order at the current price. Once that buy order is filled, issue a corresponding sell order at the buy price + percent-change-threshold.
minimum-counter-currency-buy-order-amount  - never place a buy order for less than this amount, regardless of base-currency-trade-amount.
base-currency-trade-amount - the amount of base currency (e.g. BTC) to buy or sell (unless the value is less than minimum-counter-currency-buy-order-amount).

Here is how the algorithm will work:
if lastTransactionPrice is not set  {
    Send buy order at the current ask price. Push that buy order onto a buyOrderStack.
    (It should fill quickly, but might get stuck if price goes up quickly)
    set lastTransactionPrice to the current price.
    set highPrice to current price.
}
else {
   if have any buy orders on buyOrderStack {
      if the top order has been filled {
          pop the top filled buy order from the buyOrderStack.
          set lastTransactionPrice to the price that it was bought at.
          send a sell order for lastTransactionPrice * (1 + percent-change-threshold)
          push that sell order on a sellOrderStack. (it will not fill until market goes up by percent-change-threshold)
      }
   }
   if have any sell orders on sellOrderStack {
      if the top order has been filled {
          pop the top filled sell order from the sellOrderStack.
          set lastTransactionPrice to the price that it was sold at.
      }
   }
   if currentPrice < lastTransactionPrice * (1 - percent-change-threshold) and sellStack.size < max-concurrent-sell-orders {
       then send buy order at the current price. Push that buy order in a buyOrderStack.
       set lastTransactionPrice to the current price.
   }
   if (currentPrice > highPrice * ( 1 + percent-change-threshold) and have enough base currency balance (i.e BTC) {
       Send sell order at the current ask price. Push that sell order onto the sellOrderStack.
       set highPrice to currentPrice
   }
}

It would be a good idea to add some integration tests that do some level of back-testing for different cases.
Here are a few timeseries that could represent BTC prices over time.
Case 1 - monotonically increasing (linear)
Case 2 - exponentially increasing (e^x) - with randomnesss
Case 3 - volatile, but trending up
Case 4 - mountain/inverted V.
Case 5 - flat
Case 6 - V-shaped - up then down
Case 7 - volatile, but trending down
Case 8 - exponentially decreasing (e^-x) - with randomness
Case 9 - monotonically decreasing (linear)

figure out why it stops working after a day.
switch from Bitstamp to Kraken to have lower fees.
Do backtesting.

I got some results from simulating the scenarios with both strategies. As expected, each strategy does well in some cases and poorly in others. It all depends on what you believe is going to happen. For the set of cases tried, the multi-order strategy preserved value better in the "exponential decreasing" and "linear decreasing" cases, but for most other cases the rebalancing approach did better.
Overall, I do like the rebalancing approach better, but one potential problem is that it can make much bigger large buy/sell orders when there are big swings. Probably it is ok, but I feel a little safer when the size of the orders is limited.
Assuming that I haven't made any errors in the calculations, below are the simulated results.
In each case, you start with $500 USD and $500 worth of BTC. At the end of the simulation, you sell all BTC at current price, and the sum is reported. So you can subtract $1000 from each result value below to see the gain you would have to report on your taxes.
// Barry's multi-order strategy
LINEAR_INCREASING, 1246.6687449900676
VOLATILE_INCREASING, 1676.0152523952222
EXPONENTIAL_INCREASING, 3777.0709648135107
FLAT, 1106.8671234190554
RANDOM_WALK, 1782.468174054819
EXPONENTIAL_DECREASING,<PHONE_NUMBER>649542
VOLATILE_DECREASING, 1140.9945772497686
LINEAR_DECREASING,<PHONE_NUMBER>465663
EXPONENTIAL_INCREASE_WITH_CRASH,<PHONE_NUMBER>060816
EXPONENTIAL_INCREASING_WITH_CRASHES, 1608.6149046197206
HISTORICAL_DATA, 1185.3989547175693

// Bill's rebalancing strategy
LINEAR_INCREASING, 1429.0203698769033
VOLATILE_INCREASING, 2021.7335350394198
EXPONENTIAL_INCREASING, 4500.905912653756
FLAT, 1118.444951577584
RANDOM_WALK, 2120.245032765944
EXPONENTIAL_DECREASING, 77.63225380742851  // big loss here
VOLATILE_DECREASING, 1219.1793934840493
LINEAR_DECREASING,<PHONE_NUMBER>038278
EXPONENTIAL_INCREASE_WITH_CRASH,<PHONE_NUMBER>4873613
EXPONENTIAL_INCREASING_WITH_CRASHES, 1942.3531632376228
HISTORICAL_DATA, 1844.5837059273517

I got some results from simulating the scenarios with both strategies. As expected, each strategy does well in some cases and poorly in others. It all depends on what you believe is going to happen. For the set of cases tried, the multi-order strategy preserved value better in the "exponential decreasing" and "linear decreasing" cases, but for most other cases the rebalancing approach did better.
Overall, I do like the rebalancing approach better, but one potential problem is that it can make much bigger large buy/sell orders when there are big swings. Probably it is ok, but I feel a little safer when the size of the orders is limited.
Assuming that I haven't made any errors in the calculations, below are the simulated results.
In each case, you start with $500 USD and $500 worth of BTC. At the end of the simulation, you sell all BTC at current price, and the sum is reported. So you can subtract $1000 from each result value below to see the gain you would have to report on your taxes.
// Barry's multi-order strategy
LINEAR_INCREASING, 1246.6687449900676
VOLATILE_INCREASING, 1676.0152523952222
EXPONENTIAL_INCREASING, 3777.0709648135107
FLAT, 1106.8671234190554
RANDOM_WALK, 1782.468174054819
EXPONENTIAL_DECREASING,<PHONE_NUMBER>649542
VOLATILE_DECREASING, 1140.9945772497686
LINEAR_DECREASING,<PHONE_NUMBER>465663
EXPONENTIAL_INCREASE_WITH_CRASH,<PHONE_NUMBER>060816
EXPONENTIAL_INCREASING_WITH_CRASHES, 1608.6149046197206
HISTORICAL_DATA, 1185.3989547175693

// Bill's rebalancing strategy
LINEAR_INCREASING, 1429.0203698769033
VOLATILE_INCREASING, 2021.7335350394198
EXPONENTIAL_INCREASING, 4500.905912653756
FLAT, 1118.444951577584
RANDOM_WALK, 2120.245032765944
EXPONENTIAL_DECREASING, 77.63225380742851  // big loss here
VOLATILE_DECREASING, 1219.1793934840493
LINEAR_DECREASING,<PHONE_NUMBER>038278
EXPONENTIAL_INCREASE_WITH_CRASH,<PHONE_NUMBER>4873613
EXPONENTIAL_INCREASING_WITH_CRASHES, 1942.3531632376228
HISTORICAL_DATA, 1844.5837059273517


Release 0.19.0

Once these documentation changes have been merged, I'll set the tag.

Maybe replace 0.18.0 references in README.md ?

Done.


Add baselines for net8

cc @marcpopMSFT @dsplaisted

I don't have any access to this repo so someone else will need to merge

I don't have any access to this repo so someone else will need to merge

I don't either. I would if I could ;-)


improve ux when out of credits with a 3rd party api

at the moment, for example, i see an error message like this in open-webui instead

There's no consistency with how APIs report this. The only way is using their specific APIs for querying that information, and those are also not consistent between providers.

would it work if the pipelines manifolds reply with a standard openai-like response about it so it works across api providers?

Returning non-standard response bodies is a good way to break compatibility with downstream clients that aren't expecting it. It would need to be it's own API, similar to how valves are handled, meaning it'd only work with Open WebUI unless other clients adopted that spec.


More specific time

Hi, the way the time played is displayed is a bit confusing,
Congrats, you've played 8 days, 17:31:15
What does that mean? 8 days and 17h? (so 209h?).

Honestly I just let the datetime library figure it out when writing the script
Specifially it's datetime's "timedelta" objects  that print things this way, and since dates/times are very much a matter of opinion (24h format? total hours or total days + extra? what about microseconds? etc), I didn't go a particular way with this other than the default handling of the library.
Frankly this was a quick foray into Python scripting, and the resulting exe with GUI is also really big for what I was hoping to achieve with such a small script, but alas it's the way Python goes.
If I rewrote this I'd probably pick a language more likely to result in a near GUI and filesize, but if you want to either fork it (to make needed changes) or create a PR for any format you deem more friendly (on this repo), the code is completely yours too :)

Ok, Thanks. I'll see what i can do.


[stable/jaeger-operator] Fixing typo in NOTES.txt

What this PR does / why we need it:
After installing the chart, running the commands specified in the NOTES.txt results in a kubectl error.
error: there is no need to specify a resource type as a separate argument when passing arguments in resource/name form (e.g. 'kubectl get resource/<resource_name>' instead of 'kubectl get resource resource/<resource_name>'
There is a typo in the line:
export POD=$(kubectl get pods-l app.kubernetes.io/instance=jaeger-operator -lapp.kubernetes.io/name=jaeger-operator --namespace jaeger --output name)
Adding spaces between the -l option fixes the issue, as it's a syntactical error.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @batazor

/assign @batazor

/lgtm

/ok-to-test

/retest

/retest

/ok-to-test

/retest

@batazor I don't understand how the e2e test's are failing? I only changed the NOTES and the Chart version?

@BailyTroyer tests failed by timeout, it needs retry

/retest

/retest

/ok-to-test

@lachie83 help)

@lachie83


RuntimeError: An attempt has been made to start a new process before the current process has finished its bootstrapping phase.

I'm having the following issue, any help would be appreciated. I'm using Python 3.10.2.
chrome = Chrome()
youtube = Youtube(
    browser=chrome
)

result = youtube.upload(video_path=video_path,
               title=title,
               description=description,
               tags=tags,
               made_for_kids=made_for_kids,
               visibility=visibility,
               # thumbnail_images_path=thumbnail_images_path,
               timeout=timeout)`

Here is the stack trace:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 116, in spawn_main
    exitcode = _main(fd, parent_sentinel)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 125, in _main
    prepare(preparation_data)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 236, in prepare
    _fixup_main_from_path(data['init_main_from_path'])
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 287, in _fixup_main_from_path
    main_content = runpy.run_path(main_path,
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/runpy.py", line 269, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/runpy.py", line 96, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/Users/user/Documents/Projects/testproject/main.py", line 444, in <module>
    testUpload()
  File "/Users/user/Documents/Projects/testproject/main.py", line 136, in testUpload
    uploadVideo("test-vid.mp4")
  File "/Users/user/Documents/Projects/testproject/main.py", line 358, in uploadVideo
    chrome = Chrome()
  File "/Users/user/Documents/Projects/testproject/lib/python3.10/site-packages/selenium_chrome/chrome.py", line 113, in __init__
    super().__init__(
  File "/Users/user/Documents/Projects/testproject/lib/python3.10/site-packages/selenium_browser/selenium_browser.py", line 62, in __init__
    self.driver = webdriver_class(**webdriver_kwargs)
  File "/Users/user/Documents/Projects/testproject/lib/python3.10/site-packages/undetected_chromedriver/__init__.py", line 392, in __init__
    self.browser_pid = start_detached(
  File "/Users/user/Documents/Projects/testproject/lib/python3.10/site-packages/undetected_chromedriver/dprocess.py", line 39, in start_detached
    ).start()
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/process.py", line 121, in start
    self._popen = self._Popen(self)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/context.py", line 224, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/popen_fork.py", line 19, in __init__
    self._launch(process_obj)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/popen_spawn_posix.py", line 42, in _launch
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 154, in get_preparation_data
    _check_not_importing_main()
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/multiprocessing/spawn.py", line 134, in _check_not_importing_main
    raise RuntimeError('''
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.

Put all your code under:
if __name__ == '__main__':
It will resolve the issue


[NFC] Shortened SIL [init] flag.

Instead of writing out [initalization] for some instructions, use [init] everywhere.

@swift-ci please test

@swift-ci please test and merge

@swift-ci please test linux platform


Same Docs Updates as #231 without DM

Adding just the docs updates from #231 without the density matrix base.

Thank you GenericP3rson for the PR!


Support King Art - King Q4 Cover

Hi, first, thanks for your work on this.
Do your component have compatibility with the Kingart king-q4 blinds controller?
I'm using peterbuga's component but althoug I can open and close the blinds, it is not possible to have control of the percentage.
Thanks!

What version of firmware in ewelink app?

What version of firmware in ewelink app?

Versio 2.7.0.... Firmware PSF-BTA-GL

Here #31

Try new version #99

Hi,
first, thanks for your awesome work on the Sonof stuff.
The King Art - King Q4 is still detected as a normal switch in HA supporting only power off or on.
This is the entity:
manufacturer: KingArt
model: KING-Q4
sw_version: PSF-BTA-GL v2.7.0
connection: cloud
rssi: -69
friendly_name: Persiana Terraza
supported_features: 0
icon: 'mdi:window-shutter'
No up - stop - down buttons neither slider.... :(
Am I missing something in the setup?
Thanks,

@sergioa77 you can write me in Discord or Telegram.
And share devices to my acc. I need it for debugging.

Thanks for you hard work @AlexxIT . We have now available this King Q4 Cover device fully working with slider and up, stop, down buttons. Magic!

Done

Здравствуйте!
Дайте пример конфигурации.

@riorao https://github.com/AlexxIT/SonoffLAN#config-examples

Hi @AlexxIT,
I've installed the last release 3.0.5 and the king art cover is not working properly. It worked ok with 2.4.7. It is KING-Q4 Fw: 3.4.3
Is there something to change / configure or is it something to be fixed?

Is it possible to use this cover over LAN only? Right now it just goes offline when switching the integration to local mode


GoogleMaps/GoogleMaps.h file not found

Please I am trying to build my app with xcode but am getting this error GoogleMaps/GoogleMaps.h file not found, I am using xcode 8.0 and am using the Master branch of cordova-plugin-googlemaps. Please help me with a solution.

The current version will be deprecated soon.
Please use the v2-beta version.
https://github.com/mapsplugin/cordova-plugin-googlemaps#version-20-beta

@wf9a5m75 I have installed the current v2-beta and it's crashing the app when I open the map section of the app. This happen on android while the issue for ios not building bcos of GoogleMaps/GoogleMaps.h file not found is still there but I think I have found the solution to that in one of the closed issues, I will try that later. The real issue here now is why is the v2-beta crashing my app? Please I need solution.

The v2 does not use GoogleMaps/GoogleMaps.h anymore.

Ok but why is it crashing the app?

Or does the v2-beta has its own implementation if it's having pls can you give me a link to the documentation.

I can not say anything without log.
Please read this carefully.

And you should install the demo app before asking.

ok.


import closed page

Good day!
It will be possible to import the design of the page that is closed. Authorization required (login, password)

Hey @FaridDok - you can use our chrome extension for that


All exported json should include version flag

Save data is currently very in flux; last month the save data just included a list of histories. This month we are storing two kinds of data, history data and the player's money. Next month it's likely we'll change the scenario history to only include 2-3 records per scenario. This will also be a compatibility-breaking changes.
We only have 2 users right now, so this change is frustrating. When we have 10 users and some of them have put hours into progressing through the game, this will be a game-ruining issue. We need to be able to tell when we're working with an old save format, and we need to have utilities to convert or load old data.
Exported scenarios and save data should include a version flag, like "version": "15c0". The first digit corresponds to the (year - 2019) in hex. The second digit corresponds to the month (1-12) in hex. The next two digits correspond to the (day  * 8) in hex, plus a little extra.
For 2020-05-24, this would be in the range 15c0-15c7.
The version should only be incremented if the save format changes.

In OpenOffice, this format is expressed with =DEC2HEX((YEAR(NOW())-2019)*4096+(MONTH(NOW()))*256+(DAY(NOW()))*8+INT(HOUR(NOW()) / 3))


heroku boot timeout

for running butler on heroku, I keep getting an Error, R10 (Boot Timeout);
2015-11-07T21:49:15.389471+00:00 heroku[web.1]: Error R10 (Boot timeout) -> Web process failed to bind to $PORT within 60 seconds of launch
2015-11-07T21:49:15.389471+00:00 heroku[web.1]: Stopping process with SIGKILL
2015-11-07T21:49:16.030841+00:00 heroku[web.1]: Process exited with status 137
2015-11-07T21:49:16.054402+00:00 heroku[web.1]: State changed from starting to crashed

Not sure if I missed a step in the configuration, but it seems that heroku is looking for a port to bind to for this app? Do we need to open an endpoint for heroku  to verify the app is alive?

@bbrock25 are you running butler as a web dyno? You have to run him as a worker. Web dynos on heroku have to bind to a port. I've been thinking about adding a web frontend to butler which would enable him to bind as a web dyno. Thats in the future though.

That would most certainly be it. Pretty much common sense in retrospect. thanks!


PPP always pushing <IP_ADDRESS> via IPCP

I am using sstp-client (https://sourceforge.net/p/sstp-client/) on Linux for connecting via SSTP to SoftEther VPN server with SecureNAT on Ubuntu 14.04.1 LTS x64.
I noticed that the SoftEther VPN server is pushing the address <IP_ADDRESS> via IPCP, causing the client pppd to set up a route for it:
# route -n4
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> U 0 0 0 ppp0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> UH 0 0 0 ppp0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> UG 0 0 0 eth0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> U 1 0 0 eth0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> UH 0 0 0 lo

# grep IPCP /var/log/syslog
Nov 21 08:28:58 lub pppd[3533]: sent [IPCP ConfReq id=0x1 <compress VJ 0f 01> <addr <IP_ADDRESS>> <ms-dns1 <IP_ADDRESS>> <ms-dns2 <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: rcvd [IPCP ConfReq id=0x2 <addr <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: sent [IPCP ConfAck id=0x2 <addr <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: rcvd [IPCP ConfRej id=0x1 <compress VJ 0f 01> <ms-dns2 <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: sent [IPCP ConfReq id=0x2 <addr <IP_ADDRESS>> <ms-dns1 <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: rcvd [IPCP ConfNak id=0x2 <addr <IP_ADDRESS>> <ms-dns1 <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: sent [IPCP ConfReq id=0x3 <addr <IP_ADDRESS>> <ms-dns1 <IP_ADDRESS>>]
Nov 21 08:28:58 lub pppd[3533]: rcvd [IPCP ConfAck id=0x3 <addr <IP_ADDRESS>> <ms-dns1 <IP_ADDRESS>>]

# ifconfig -a
eth0 Link encap:Ethernet HWaddr 08:00:27:75:fb:7c
inet addr:<IP_ADDRESS> Bcast:<IP_ADDRESS> Mask:<IP_ADDRESS>
[...]
ppp0 Link encap:Point-to-Point Protocol
inet addr:<IP_ADDRESS> P-t-P:<IP_ADDRESS> Mask:<IP_ADDRESS>
UP POINTOPOINT RUNNING NOARP MULTICAST MTU:1500 Metric:1
RX packets:15 errors:0 dropped:0 overruns:0 frame:0
TX packets:9 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:3
RX bytes:1219 (1.2 KB) TX bytes:450 (450.0 B)

The Virtual Host's Network Interface IP address is <IP_ADDRESS>, therefore I do not understand why the VPN server pushes <IP_ADDRESS> instead of <IP_ADDRESS>.
Since the pppd configuration has the options [defaultroute] and [replacedefaultroute], there is a default route to interface ppp0 in place, therefore missing the route for <IP_ADDRESS> is USUALLY not a problem.
However if we have a route for <IP_ADDRESS>/16 (which is PRIVATE-ADDRESS-CBLK-RFC1918-IANA-RESERVED) in place (because there are such hosts on the LAN), then having a more specific route for <IP_ADDRESS> to ppp0 is mandatory (also because <IP_ADDRESS> is the DNS server address pushed by the VPN server to the client).
Of course I can myself force the route using a command like [route add -net <IP_ADDRESS>/32 dev ppp0], still I believe that the VPN server should push it through IPCP.
The problematic code piece is
https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/src/Cedar/IPsec_PPP.c#L304
and was identified by dajhorn in thread
http://www.vpnusers.com/viewtopic.php?f=7&t=3899

Related issue: #61

Is this issue solved yet? i'm trying to modify the server ip,but meet some problems

Pull request: #524

I looked back at the commit and noticed this is no longer applied from what I'm seeing and also the latest SE server reflects still handing out <IP_ADDRESS>. This is an issue, that's now a cloudflare IP, and I believe its still the cause of routing issues on mikrotik (and possibly others) trying to L2TP to SE.

#524 changed the IP address to <IP_ADDRESS>, which is defined in RFC7600 as dummy IPv4 address.


feat: austin reaves

Hi @naa4beebeeoh , we actually are also including AR along with other player data in this PR: https://github.com/naw-db/naw-db.github.io/pull/70/files

@naa4beebeeoh Im going to mark this as a duplicate and close it. Thanks for looking into it though!


Use dynamic headers for otel-dc-llm

LLM Otel DC can obtain x-instana-key and x-instana-host from the request sent by traceloop-sdk, so it's not required to explicitly set OTEL_EXPORTER_OTLP_HEADERS anymore.

@jinsongo one comment here: if two PRs have dependency, after the first PR created, please cut a new branch based on existing branch and create a new PR, this will generate a dependency PR.


Allow Triggerable to be globally disabled

This is especially useful for tests in projects that use Triggerable, so that we can run factories to set up data in a specific way without triggers interfering with them.

Merged, thank you!


How to host a kafka trigger and invoke a webjob

Apologies.
Could someone please explain how to host a trigger and then execute a webjob.
We are planning to implement a kafka trigger which would then invoke a webjob whenever a new message is available in kafka.

@sunil-suresh
What are you attempting to do?
By execute a webjob do you mean an Azure Function, or something else (or job built against the webjobs sdk)?

Im trying to execute the kafka trigger as a azure function app but I get a run time message saying no jobs found. Any reasons why?

Hi @sunil-suresh how are you building your function app?
Are you adding the reference to this project or using the nuget package for kafka extensions?

@sunil-suresh this has been open and awaiting further input from you for some time, we should either progress this, or archive the issue.
thanks.


Add 3 more milestones to the list

The 3 things are:

gitlab
merge requests
localize.drupal.org

Seems cool to me.

The 3 things are:

gitlab
merge requests
localize.drupal.org

Seems cool to me.

We have an entry for issue forks. IMO that covers 1. and 2. Please resubmit the entry for localize. Thanks.

We have an entry for issue forks. IMO that covers 1. and 2. Please resubmit the entry for localize. Thanks.

Let's skip the hungarian milestone. It's too country specific for this timeline imo.

Let's skip the hungarian milestone. It's too country specific for this timeline imo.

It was inspired by the German local Drupal communityʼs milestone in PR #44 (which is still waiting for merge too).

It was inspired by the German local Drupal communityʼs milestone in PR #44 (which is still waiting for merge too).


Option to pan on trackpad 2-finger scroll, zoom on pinch

Thank you for taking the time to report an issue with react-d3-tree!
Feel free to delete any questions that do not apply.
Are you reporting a bug, or opening a feature request?
Feature request
I'd be happy to contribute a PR if the requirements and outline implementation are agreed.
What is the actual behavior/output?
If zooming is enabled, it's triggered by scrolling. On a trackpad a 2-finger vertical scroll gesture triggers zoom in/out. 2-finger horizontal scroll does nothing. Pinch gesture also controls zoom. The only way to pan is with click and drag.
What is the behavior/output you expect?
It would be great to be able to emulate the behavior of popular whiteboard canvas tools like figma, miro etc. where 2-finger gesture pans and pinch zooms. Perhaps in this mode click and drag would no longer pan, and instead e.g. could be used to select multiple nodes within a selection rectangle, or whatever custom behavior was desired.
Perhaps this could be a simple boolean or enum prop field to toggle between different preset behaviors, or perhaps it would be better implemented as callback functions to optionally override parts of the zoom handler with arbitrary logic?
What version of react-d3-tree are you using?
3.5.1

On my trackpad this is already working like that. I think this is more a trackpad related thing - on my old laptop it's not working like that, but my newest laptop it is.


Updated the humans_or_horses livebook

Added a really simple Exploratory Analysis section and
configuration module.
All hanging functions were put inside modules.  The livebook
can run to completion on my CPU.

I believe that I've made the changes that were requested.


[WIP] Improve SSD

This PR contains a part of training.  I think it is better to develop everything in #206.


Update database-schema.md

Uploaded new DB schema

@v-sambati : Thanks for your contribution! The author(s) have been notified to review your proposed change.

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

File
Status
Preview URL
Details

microsoft-community-training/analytics/custom-reports/database-schema.md
:white_check_mark:Succeeded
View

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