[BUG] Added datasources not visible in security analytics

What is the bug?
Added multiple datasources in opensearch for remote clusters. The added datasources shows up in dropdown in devtools but not in security analytics. In security analytics, datasource defaults to local cluster, although a newly added datasources is added as default.
How can one reproduce the bug?
Steps to reproduce the behavior:

Add datasources in opensearch
Go to security analytics and click on Local Cluster (written on top right corner)
Check the available datasources present in dropdown
It shows only 1 option i.e Local Cluster.

What is the expected behavior?
All the added data sources should be present in dropdown for selection in datasources in security analytics
What is your host/environment?

OS: linux
Version: 2.16.0
Plugins: security analytics

Do you have any screenshots?
All datasources available for selection in dev tools

Added datasources not showing up in security analytics

[Catch All Triage - 1, 2, 3]


Generate proper JSON Schema for nullable properties

Change Summary
Generate 'anyOf: [{'type': <field_type>}, {'type': 'null'}] for optional fields of a pydantic model.
Related issue number
#1270
Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

deferring this until v2.

Hello! I was working with #1270 and this came up:
quoting the specs

Note that there is no null type; instead, the nullable attribute is used as a modifier of the base type.

So instead of using anyOf we should simply add nullable: true to the field. What do you think?

OpenAPI 3.1 is now out and supports JSON Schema. It dropped "nullable: true" for type: "null". I reckon we should go with Json Schema and OpenAPI 3.1 support only and add "null" in the availble types.
I would be in favor of updating this PR and maybe avoid waiting for v2.
@samuelcolvin WDYT?
@ matpuk Would you be willing to rebase and update your PR?

OpenAPI 3.1 is now out and supports JSON Schema. It dropped "nullable: true" for type: "null". I reckon we should go with Json Schema and OpenAPI 3.1 support only and add "null" in the availble types as done in this PR

any update on this?


Can't Decorate Namespaces with Labels

I'm trying to build a Decorator that adds a given label testns = verified to any Namespace that contains annotation field.cattle.io/projectId.
In metacontroller logs I see the request being sent and a response received and as well metacontroller tries to update the namespace as well, but ultimately the label is not added. I've tried with and without passing attachments. I'm not sure what I'm doing wrong here (and I'm still building knowledge on MC):
decorator.yaml
`
apiVersion: metacontroller.k8s.io/v1alpha1
kind: DecoratorController
metadata:
name: namespace-labeler
spec:
resources:

apiVersion: v1
resource: namespaces
annotationSelector:
matchExpressions:

{key: field.cattle.io/projectId, operator: Exists}
hooks:
sync:
webhook:
url: http://namespace-labeler-service.metacontroller.svc.cluster.local:5000/sync
attachments:

apiVersion: v1
resource: namespaces
updateStrategy:
method: InPlace`
deployment.yaml
`
apiVersion: apps/v1
kind: Deployment
metadata:
name: namespace-labeler
spec:
replicas: 1
selector:
matchLabels:
app: namespace-labeler
template:
metadata:
labels:
app: namespace-labeler
spec:
containers:

name: namespace-labeler
image: python:3.9-alpine
command: ["/bin/sh", "-c"]
args:

|
echo "from flask import Flask, request, jsonify
import json
import logging
app = Flask(name)
logging.basicConfig(level=logging.DEBUG)
@app.route('/sync', methods=['POST'])
def sync_hook(request):
parent = request['object']
children = request['attachments']
namespace_labels = parent['metadata'].get('labels', {})
namespace_labels['testns'] = 'verified'
children['Namespace.v1'] = [{
    'apiVersion': 'v1',
    'kind': 'Namespace',
    'metadata': {
        'name': parent['metadata']['name'],
        'labels': namespace_labels
    }
}]

response = {
    "status": {},
    "attachments": children['Namespace.v1']
}

return flask.jsonify(response)

if name == 'main':
app.run(host='<IP_ADDRESS>', port=5000)" > /app/sync.py;
pip install flask;
python /app/sync.py
volumeMounts:

name: app
mountPath: /app
ports:

containerPort: 5000
volumes:

name: app
emptyDir: {}

apiVersion: v1
kind: Service
metadata:
name: namespace-labeler-service
spec:
selector:
app: namespace-labeler
ports:
- protocol: TCP
port: 5000
`

hi @wrkode , in case you want to add a label, it should be in a different way - just add a label/annotation to parent - as attachments are additional resources created. So in your case you do not need them

hmm wait, actually you are doing that, when you enable logging on v7, what do you get ?

@grzesuav ,
thanks for chiming in.
I've raised logging to v7 and created namespacefoo
here is the result:

@wrkode looking at https://metacontroller.github.io/metacontroller/api/decoratorcontroller.html#sync-hook-response
and  your snippet
namespace_labels = parent['metadata'].get('labels', {})
namespace_labels['testns'] = 'verified'
children['Namespace.v1'] = [{
    'apiVersion': 'v1',
    'kind': 'Namespace',
    'metadata': {
        'name': parent['metadata']['name'],
        'labels': namespace_labels
    }
}]

response = {
    "status": {},
    "attachments": children['Namespace.v1']
}

return flask.jsonify(response)

you are not setting labels in response,
there should be sth like
response = {
    "status": {},
    "labels": <set them here>
    "attachments": children['Namespace.v1']
}

@wrkode does it solved your issue ?

@grzesuav tomorrow I'll be trying again and I'll report back.

@wrkode how it went ?

@wrkode please re-open in case it still does not work for you

@grzesuav , sorry for not coming back to this. The issue was indeed solved. thank you


Fix enum import in repeated field

I am trying to fix https://github.com/envoyproxy/protoc-gen-validate/issues/511
Following the fix I noticed another bug in the go import statement generation for enums, go package names overlap. I tried to fix it but I am not sure to understand how harness works.
When executing the make task testcases, I am able to generate fixed pb files. Imports look like this:
	_go "github.com/envoyproxy/protoc-gen-validate/tests/harness/cases/other_package/go"

	_go1 "github.com/envoyproxy/protoc-gen-validate/tests/harness/cases/yet_another_package/go"

But when I am running harness, generated files are still broken, imports are not name correctly:
	_go "github.com/envoyproxy/protoc-gen-validate/tests/harness/cases/other_package/go"

	_go "github.com/envoyproxy/protoc-gen-validate/tests/harness/cases/yet_another_package/go"

Notice the overlapping import names.
I will keep on trying, but do you have any tips on this ?

@rodaine do you have any time to review this ?


Docker daemon failing to push (trying to write <IP_ADDRESS>:53?)

 Please upgrade.

This issue has been reported a long time ago against Docker 1.2.0. Since then, Go has been upgraded and many other changes have been made to Docker itself.
Errors such as this one are usually caused by network problems and there's not much to be done when it comes to Docker itself.
I'm going to close this issue. If anyone runs into this again, please comment on this issue and we'll open it again.


Inherited table aliases being clobbered in UNION

@Test
public void testQueryUnionJoinOnSameEntities() {
    Group group = new Group();
    group.setName("Hello!");
    data.insert(group);
    Person person1 = randomPerson();
    person1.setName("Carol");
    person1.getGroups().add(group);
    data.insert(person1);
    Person person2 = randomPerson();
    person2.getGroups().add(group);
    person2.setName("Bob");
    data.insert(person2);
    List<Tuple> result = data.select(Person.NAME.as("personName"), Group.NAME.as("groupName"))
            .where(Person.ID.eq(person1.getId()))
            .union()
            .select(Person.NAME.as("personName"), Group.NAME.as("groupName"))
            .where(Person.ID.eq(person2.getId()))
            .orderBy(Person.NAME.as("personName")).get().toList();
    System.err.println(result.size());
    System.err.println(result.size() == 2);
    assertTrue(result.size() == 2);
    assertTrue(result.get(0).get("personName").equals("Bob"));
    assertTrue(result.get(0).get("groupName").equals("Hello!"));
    assertTrue(result.get(1).get("personName").equals("Carol"));
    assertTrue(result.get(1).get("groupName").equals("Hello!"));
}

The generated SQL is correct for the first half of the UNION, but the second half switches from reusing the inherited aliases to using a new pair of aliases once it reaches the FROM clause.
io.requery.sql.StatementExecutionException: Exception executing statement: select a.name as name, b.name as group from Person a, Groups b where a.id = ? union select a.name as name, b.name as group from Person c, Groups d where c.id = ? order by name

It looks like this started happening after the fix for #482 (b7301d9).

Thanks for the test case! I've added it to the main set of test cases

Thanks Nikhil, that has fixed the test case.  Unfortunately there still seems to be some strangeness, if I manually perform a join I end up with a similar problem.
@Test
public void testQueryUnionExplicitJoinOnSameEntities() {
    Address address = randomAddress();
    Person person1 = randomPerson();
    person1.setName("Carol");
    person1.setAddress(address);
    data.insert(person1);
    Person person2 = randomPerson();
    person2.setName("Bob");
    data.insert(person2);
    List<Tuple> result = data.select(Person.NAME.as("personName"), Address.CITY.as("cityName"))
            .from(Person.class)
            .leftJoin(Address.class)
            .on(Person.ADDRESS_ID.eq(Address.ID))
            .where(Person.ID.eq(person1.getId()))
            .union()
            .select(Person.NAME.as("personName"), Address.CITY.as("cityName"))
            .from(Person.class)
            .leftJoin(Address.class)
            .on(Person.ADDRESS_ID.eq(Address.ID))
            .where(Person.ID.eq(person2.getId()))
            .orderBy(Person.NAME.as("personName")).get().toList();
    assertTrue(result.size() == 2);
    assertTrue(result.get(0).get("personName").equals("Bob"));
    assertNull(result.get(0).get("cityName"));
    assertTrue(result.get(1).get("personName").equals("Carol"));
    assertTrue(result.get(1).get("cityName").equals(address.getCity()));
}

io.requery.sql.StatementExecutionException: Exception executing statement: select a.name as personName, b.city as cityName from Person a left join Address b on a.address = b.id where a.id = ? union select a.name as personName, b.city as cityName from Person a left join Address c on a.address = c.id where a.id = ? order by personName

Thanks looks like this isn't a perfect fix

I've been looking at the DefaultOutput class trying to figure out the best way to fix this.  When appendQuery is called from SetOperatorGenerator, a new DefaultOutput is created which inherits the Aliases object from the original one.  The inherited aliases are reused by appendColumnForSelect and appendTables, but it goes wrong when appendJoin is called and the already-used alias for Address is removed and replaced by a new one.
I've been able to make this particular testcase pass in a couple of different ways.  If I change appendQuery to pass in a copy of Aliases with an empty appended set instead of the original object, the inherited alias gets reused by appendJoin instead of being deleted.  Alternatively if I don't pass in the Aliases at all when calling appendQuery from SetOperatorGenerator, the new output will happily generate its own aliases.
I'm not really sure if either approach is close to being correct, as I don't know how the alias inheritance is used.  If I remove it entirely the unit tests all still pass, so it doesn't appear to be exercised by the current test suite.


Implement Passport.js Local Strategy

This commit for #617 integrates Passport's Local Strategy for user authentication.
The strategy has been set up to use email as the username field. If the email is not found or the password is incorrect, error messages are returned.
Was not sure how to implement tests on Users because users do not exist yet.
Co-authored-by: KonnyGuo<EMAIL_ADDRESS>

As for your first question, I agree that we should merge these couple branches into 1 since it originally was 1 Issue, but we broke it down with Konny to work on it separately.


Language selection does not work in IE11

Hi @volkanceylan ,
just a quick note: Under IE11, the language selection (switch language for currently logged in user on right, upper side) does not work in newest Serenity and also not on your demo page. With Chrome it works.
With Kind regards,
John

I'll did this same question.
Language selection works on firefox too. Can't check if this works on edge too, because I don't have windows 10.
The other problem is when I migrate my classes from old Serenity Version to actual version (of course, checking all tips from upgrade guide or create a new project from template and migrate) and run, I have and error "prototype can't be null"
And it point to language script.

late notes:
1 - restarted VS
2 - restarted PC
3 - updated packages (Update-Package Serenity.Web and Update-Package Serenity.CodeGenerator)
4 - done steps 1 and 2 again
errors still happens.

Sorry, can't reproduce, it works in my PCs :/

Prototype error has no relation to this, you probably miss some /// references

@volkanceylan you are using Edge, try in IE.

@volkanceylan
you tried it with MS Edge, not IE11. Look at your Screenshot - This is Edge. :-)
P.S: With Edge it works - with IE11 not.
As IE11 is still the most used Webbrowser within Companies internal IT infrastructure - at least with those customers I work with - this should be fixed.
With kind regards,
John

@JohnRanger I already tried to tell that to @volkanceylan but it no answer, perhaps to busy ;)

For yet another confusing data point, changing language in IE 11 using 3.7.7 (asp.net, not core) is working for me locally. On the demo site, the Language dropdown is empty:

Just upgraded to Serenity 3.7.7 on core and it does not work for IE either. Chrome is fine. Thanks.

As I needed to solve it, I tried to find where it is happening. And in IE, the x function is undefined.

My knowledge of javascript is not deep enough to solve this in an elegant way, so I changed the async method to sync:

Until it gets fixed.

Is this from AccountLogin.cshtml? If so, you just need to change arrow function to normal function, e.g. instead of x => { }  write, function(x) { ... }

Looks like in IE 11, Promises are not working as expected, so Async doesn't work properly.
You might need to load "~/Scripts/rsvp.js" before Serenity.CoreLib.js etc. scripts in LayoutHead.cshtml or ScriptBundles.json.
I won't be doing that as i don't want to load an extra script to please IE11, and its usage percentage is pretty low.

@Volkanceylan,
thanks for your feedback.
I think that Serenity Framework is still mostly used within Enterprises for - borrowed from your advertisement of it :-) - data-driven business apps.
So far, all of my large customers do still use IE11 as their standard in-house browser as it is the only browser which is enough configurable to be useful within large enterprises (>=50'000 users). Chrome has some GPOs but all IT guys which I am talking to do confirm that Chrome is not yet enterprise ready.
Firefox cannot at all be used in large enterprises because it is not centrally configurable.
So I think it would be still a good thing that Serenity Framework would fully work with IE11 at this point in time.
With kind regards,
John

And, if you are developing a software for an Enterprise, it wouldn't be that hard to add a line to ScriptBundles.json is it? And, you always say you use Serenity for personal projects, and customers are Enterprises with 50k+ users ;)

Hi @volkanceylan ,
I just saw your last comment right now :-) Yes - It is no problem to add the relevant line to _LayoutHead.cshtml. I was more concerned about your way of thinking that IE11 is not anymore relevant and you would not anymore make sure that the current version of SF would be tested/garantied to run with IE11.
Even today (End of June 2019), while IE11 has about 1'500 GPO settings to manage it within an enterprise, Edge has only about 20 (a good article about that is here: https://theitbros.com/config-microsoft-edge-settings-using-gpo/.) This is why bigger entrerprises simply cannot just switch to anything else and are forced to continue to use an outdated browser - because the rest of them are not enterprise-ready (thank you Microsoft to leave all your well-paying enterprise customers in this really bad situation - a good new browser (Edge in Chromium mode) but not enterprise-manageable).
And with regards to my Serenity Projects... :-) With my customers, I am never contracted as a developer. Most positions are IT architect, IT admin or IT operator - depending on the needs of the customers. In these positions I often encounter situations where a Serenity-based app can relieve / solve some real problems - but I would never get any development budget for it. So I develop beside my official work and when the app then does its job, some people with which I work there (or myself) can do the work more efficient or more structured / guided because of the app.
This as a little insight in what I am doing :-)
Wish you a nice evening.
With kind regards,
John

Hi All,
if you came here because you are still forced to support IE11 - and the language selection didn't work anymore with the most recent SF 3.9.9, then do this (mostly borrowed from @globtech1  but made it work with the most recent (at time of writing) SF 3.9.9.
Modify /Modules/Common/Navigation/LanguageSelection to look like this:
namespace <your name space>.Common {
    export class LanguageSelection extends Serenity.Widget<any> {

        constructor(select: JQuery, currentLanguage: string) {
            super(select);

            currentLanguage = Q.coalesce(currentLanguage, 'en');

            this.change(e => {
                $.cookie('LanguagePreference', select.val(), {
                    path: Q.Config.applicationPath,
                    expires: 365
                });
                window.location.reload(true);
            });

            <Your Prroject name>.Administration.LanguageService.List({
            }, response => {

                response.Entities.forEach( x => {
                    Q.addOption(select, x.LanguageId, x.LanguageName);
                });

                if (!Q.any(response.Entities, z => z.LanguageId === currentLanguage)) {
                    var idx = currentLanguage.lastIndexOf('-');
                    if (idx >= 0) {
                        currentLanguage = currentLanguage.substr(0, idx);
                        if (!Q.any(response.Entities, y => y.LanguageId === currentLanguage)) {
                            currentLanguage = 'en';
                        }
                    }
                    else {
                        currentLanguage = 'en';
                    }
                }

                select.val(currentLanguage);

            });

        }
    }
}

That's it. This pulls the neccesary data direct from the server endpoint instead of relying on the lookups (which at that point in time are not yet ready). DB access always is more reliable than lookup because of no timing problems. :-)
Hope this helps someone.
With kind regards,
John


First dialog box missing "Republic"

Describe the bug
When you load the program, you get the nice Cylon eye splash screen.  But then you get the first dialog to let a new user know that they must choose a data set, and the title of that dialog is "Terran Interstellar Plotting System".  It leaves out "Republic" and it was bugging me subconsciously that TIPS != TRIPS.  And I'm guessing that most people have clicked the "Don't show this at startup" and then don't see it again (I'm trying to avoid clicking anything to avoid showing me stuff so I'm getting the full user experience).
To Reproduce
Load program and not have checked the "don't show this at startup" previously
Expected behavior vs Actual
Not sure if we would like to add "Republic" to that screen (which will either make the title wider or split it onto two lines) or make some other change?
TRIPS version

[ ] 0.52
[ ] 0.53
[ ] 0.54
[ ] 0.55
[X] 0.6.7

Desktop (please complete the following information):

[ ] IOS
[ ] Windows 10
[ ] Windows Vista, 7 or 8
[ ] Windows XP
[ ] Linux (add distribution and version to this line)
[X] macOS Big Sur

**Additional info **

done


Adding polyfill to Headers.getAll

Fixes this bug https://github.com/zeit/next.js/issues/711

If it's been removed from the spec I'd rather not test for it at all

@rauchg that'd break the current chrome (old) implementation


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2023-08-24T23:03:22+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
39b82e028b246ee2004f51b9bc50b413811d3de7
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
39b82e028b246ee2004f51b9bc50b413811d3de7
link
true
/test unit-tests-arm64

integ-basic-arm64_istio
39b82e028b246ee2004f51b9bc50b413811d3de7
link
true
/test integ-basic-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Combining forward reference type annotation with new union operator throws JSII error - Python3.10

:bug: Bug Report
Affected Languages

[ ] TypeScript or Javascript
[X] Python
[ ] Java
[ ] .NET (C#, F#, ...)
[ ] Go

General Information

Python Version: 3.10.2
Platform:   MacOS Monterey (12.1)

What is the problem?

With Python3.10, comes an alternative syntax for expressing type annonations that must be one of a list of types:
Explicitly, Union[Type1, Type2] should now be written Type1 | Type2. This along with the fact that types can be expressed as strings (for instance, to reference the type of an object inside functions of that same class) is causing issues with compliation.
Within an example, it looks like this:
class L2SecurityGroup:
    def __init__(self, scope: Construct, *,
                 # some other typed arguments
                 security_groups: Sequence["L2SecurityGroup" | SecurityGroup] = None,
                 ): pass

SecurityGroup is a cdktf provided object, generated by jsii from the AWS provider. I assume it implements JSIIMeta.
This example fails to run properly as Python is unable to handle the | between a JSII object and a python string. Log is below.
As a example that shows that this interaction is due to only these two features interacting, this example works as expected:
class L2SecurityGroup:
    def __init__(self, scope: Construct, *,
                 # some other typed arguments
                 security_groups: Sequence[Union["L2SecurityGroup", SecurityGroup]] = None,
                 ): pass

Verbose Log

   class L2SecurityGroup:
      File "/Users/someuser/aws_resources/security_groups/common.py", line 124, in L2SecurityGroup
        security_groups: Sequence["L2SecurityGroup" | SecurityGroup] = None,
    TypeError: unsupported operand type(s) for |: 'str' and 'JSIIMeta'

I'm unsure if this is something that can be resolved by appending dunder methods to JSIIMeta or something simliar. I would assume the pattern that causes this error will become more common as more people start adopting Python 3.10 and its typing recommendations.

Never mind, this is an issue with Python itself. Currently 3.10.2 is unable to combine a forward ref specified by string and a type object by the union operator.
I found that including
from __future__ import annotations

lets me use typing as expected. Closing for now.


Create WGPU widget similar to Gtk.GLArea

I've been wanting to create a GUI app using WGPU and use GTK for window decorations on Linux/BSD. Using OpenGL this can be done using GLArea covering the entire window with the widget. I have seen requests for this but not seen any implementation.

That's certainly possible but it would probably make sense for this to either a) be just an example in this repo, or b) an external crate that provides such a widget in a general way.
Do you intend to work on this?

It would be possible to use the gles backend quite easily for this (wgpu-hal has a way to create an Adapter from a glGetProcAddress like function). But there are limitations with the gles backend and the Vulkan backend would be preferred. However the Vulkan backend would either force gtk to initialize with Vulkan or share memory objects between apis.
For a minimum viable product using the gles backend would be fine, but long term some work will be needed in wgpu (or an auxillary crate) to import memory objects from other apis.

If someone wants to work on this, we could potentially have a crate that has useful "integration" widgets with the rest of the rust ecosystem. See also #1163

I'm interested in this for Linux and Windows.

I was thinking that using the gtk webkit would get us some of the way there (even if not necessarily ideal).
Epiphany (gnome's web browser) is using gtk4 as the ui toolkit and some of the demos of wgpu.rs work (only webgl2 though, not WebGPU).
So using WebKitGTK inside a gtk4 app would allow for webgl2-based wgpu applications to run.

I think there's a working implementation (at least on the architecture side) here : https://github.com/BartBM/wgpu-dma-buf
that we could adapt for gtk.
What it does is
 wgpu -> dma-buf -> glcontext -> slint window
or
wgpu -> dma-buf -> glutin -> winit window
Using dma-buf to go from wgpu to gl.
As gtk 4.14 and gtk-rs 0.9 exposes the dma-buf graphics offloading on linux, it should be possible to make the same thing work on linux with wgpu -> dma-buf -> gtk using the graphics offload feature.
Though that's linux-only for the moment (although it may come to windows in the future : https://gitlab.gnome.org/GNOME/gtk/-/issues/6869)
Maybe it's possible to adapt the other part and do
wgpu -> dma-buf -> glutin -> glcontext (gtk) -> gtk window.
Now there are probably some things to adapt between OS to have this work on all platforms (with or without graphics offload) but it seems doable given that the same trick works with slint and glutin and it doesn't seem like there's too much conceptual difference between these and gtk.


Change comments from using # to using //

What was wrong?
It seems like we have consensus that we don't want to continue to use # for comments but rather use // instead. This PR changes that.
How was it fixed?

Tiny change in the parser
Updated code files
Updated examples + docs
Updated highlight.js on the website to use the custom fe highlighting that the docs already use

This actually needed to be rebased. It's now causing a CI failure on master. I'll send a follow up.

This actually needed to be rebased. It's now causing a CI failure on master. I'll send a follow up.

I pushed a fix directly to master :hand_over_mouth:  https://github.com/ethereum/fe/commit/9300ef4c9c9d4327a4551b036006cad2803e4abe

Yeah, I missed that. Thanks for following up.


ci: upload artifacts

Once Bindings are created, the CI will upload Android/iOS bindings to Github artefacts.
Signed-off-by: Rolson Quadras<EMAIL_ADDRESS>

Should be artifacts in the commit description for consistency


dummy pr for FPET-1021

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] the enable_keep_helm label has been added, if the helm release should be persisted after a successful build

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/FPET-1021
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Closing this as not needed any more


How to hide suffixes in chat display?

Hi,
I'm using Luckperm 4.4.28 as permissions plugins and EssentialsChat as ChatFormater.
In the EssentialsChatconfig, I can't control the display of the suffixes and prefixes:
format: '{factions_relcolor}§l{factions_roleprefix}§r{factions_relcolor}{factions_name|rp} &7{DISPLAYNAME} &7:&r {MESSAGE}'
I guess the prefixes and suffixes are a part of the "DISPLAYNAME" placeholder. I need suffixes to show it in the TAB, but I don't want to keep it in the chat formater.
Thanks you in advance! ;)

Try something like &7{PREFIX}{USERNAME} or &7{PREFIX}{PLAYERNAME} instead of &7{DISPLAYNAME}.
Also, please note, that your issue isn't actually relating to LuckPerms, but anyways, I hope my solution works for you.


Datafusion Example

I tried to use DataFusion programmatically. There is also the option to use a loader and provide a batched stream.
The output is
[operators/tests/datafusion.rs:201] &result[0] = RecordBatch {
    schema: Schema {
        fields: [
            Field {
                name: "a",
                data_type: Int64,
                nullable: false,
                dict_id: 0,
                dict_is_ordered: false,
            },
            Field {
                name: "b",
                data_type: Int64,
                nullable: false,
                dict_id: 0,
                dict_is_ordered: false,
            },
        ],
        metadata: {},
    },
    columns: [
        PrimitiveArray<Int64>
        [
          0,
          1,
          2,
        ],
        PrimitiveArray<Int64>
        [
          1,
          1,
          2,
        ],
    ],
}

Process finished with exit code 0

 Data Table to other Geo Join Operator

maybe we should create a playground repository for code like this which has no touching points with geo engine code

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Rename: case_results ->  case_text

Fixes a minor typo we found yesterday.

Sorry didn't see this, it got patched in another commit, thanks!


Update ember-cli, ember-data and dependencies

Updates most dependencies.
Uses new Ember Data API.
Adds Ember Suave!

Closes #18

@duggiefresh this is failing, looks like ember-try is blowing up

Closed in favor of #21


Display correct engagement count for staff users on home dashboard

Currently the total number of active engagements rendered on the home page dashboard is restricted to either superuser or engagement_lead.
My understanding is that "staff" level access should return the total count of engagements similar to superuser.
Something similar to this seems to correct the issue for me. May or may not be missing other updates.
Diff not rendering.
Submitting PR:
https://github.com/DefectDojo/django-DefectDojo/pull/3826

Can you make a PR?

Closing. resolved'ish in #3826 :)


[Snyk] Upgrade html-snapshots from 2.14.0 to 2.15.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.14.0 to 2.15.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 4 days ago, on 2024-01-26.

Release notes

Package name: html-snapshots

2.15.0 -<EMAIL_ADDRESS>

2.14.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

f916ee9 Merge pull request #612 from localnerve/rc-2.15.0
a6b4327 @ 2.15.0 -<EMAIL_ADDRESS>6ee7edc Merge pull request #610 from localnerve/rc-doc-update-2.14.0
70698a6 fix: upgrade html-snapshots from 2.13.3 to 2.14.0
e7997e6 Merge pull request #609 from localnerve/snyk-upgrade-e8b0c5d03a03d34c2de1aa8fec39a318
cf46e97 fix: upgrade html-snapshots from 2.13.2 to 2.14.0
553dc15 Merge pull request #608 from localnerve/snyk-upgrade-82132accc2c6b2b92b077df34fc3591b
e758a9c fix: upgrade html-snapshots from 2.13.3 to 2.14.0
45297de Merge pull request #607 from localnerve/snyk-upgrade-bbbc6003f2e0c6b50cb9c34d1558145e
de2b88b fix: upgrade html-snapshots from 2.13.2 to 2.14.0
1a7e349 Merge pull request #606 from localnerve/snyk-upgrade-17c875c3924cefaf83f4cc61c29e4823
d353eae fix: upgrade html-snapshots from 2.13.3 to 2.14.0
ba84b91 Merge pull request #605 from localnerve/snyk-upgrade-8ec379eeaff98b8cc28a5b1784218ee5
0e09f95 fix: upgrade html-snapshots from 2.13.3 to 2.14.0
fb3861e Merge pull request #604 from localnerve/snyk-upgrade-dde08a1422b801ae4c2cdf1351404f61
e91813f fix: upgrade html-snapshots from 2.13.3 to 2.14.0
df4e3c4 Merge pull request #603 from localnerve/snyk-upgrade-16c75b1c15f1368a0c7360056c36bbae
eaf5078 fix: upgrade html-snapshots from 2.13.2 to 2.14.0
0dc00af Merge pull request #602 from localnerve/snyk-upgrade-436aabbed45edaf851102294f33b14d5
043f6f8 fix: upgrade html-snapshots from 2.13.2 to 2.13.3

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same
when pulling c9f95e46289487fc13dab227a23ae7c164beefc2 on snyk-upgrade-cdefe55c5a200913f7de36b561edcd72
into bc2654273dc5c12b07f1e04d6d0a479202e92e81 on master.


Issue in converting the instance segmentation mask encoding from bdd100k to coco

Hello,
I am trying to convert the bdd100k instance segmentation using this command:
python3 -m bdd100k.label.to_coco -m ins_seg --only-mask -i ./bdd100k/labels/ins_seg/bitmasks/val -o ./ins_seg_val_cocofmt_v2.json
Also, tried this:
python3 -m bdd100k.label.to_coco -m ins_seg -i ./bdd100k/labels/ins_seg/polygons/ins_seg_val.json -o ./ins_seg_val_cocofmt_v3.json -mb ./bdd100k/labels/ins_seg/bitmasks/val
The conversion is successful in both cases and the annotation looks like this

** that's not how coco annotations are.
Now, if you see the segmentation field above there's string encoding of the masks. Now, I am unsure if that's expected or not.
Further, assuming it's correct,  I tried to load the annotations using loader from DETR https://github.com/facebookresearch/detr/blob/091a817eca74b8b97e35e4531c1c39f89fbe38eb/datasets/coco.py#L36
The line I have mentioned above is supposed to do the conversion but I am getting an error from the pycocotools that it's not expecting a string in the mask.

So, I am unsure where the problem is? Is the conversion correct to coco then the loader should work?
Note: I tried to convert the detections and they worked fine.
Thank you for any help you can provide.

Responded in BDD100K repo here: https://github.com/bdd100k/bdd100k/issues/206#issuecomment-1007629548. Closing this.


TabView: Setting SelectedIndex on init crashed on RS4 and below

Describe the bug
Setting SelectedIndex on the TabView on init will crash on RS4 and below. (I think there must have been some bug fix in Selector that fixed this in later versions.)
Steps to reproduce the bug

Version Info
This only happens on RS4 and below.

Yeah, I've noticed this too.  There was definitely a change in RS4 that fixed this for ListView, ComboBox, etc...
I had filed an issue about the error message being really bad here with .NET.


Bazzite Installer is rotated and dual screen on Steam Deck OLED

Describe the bug
When I boot into the "Install Bazzite Deck Deck - 39" on Steam Deck OLED, using Dock with HDMI monitor.
Half the screen is visible half is not.
It is still possible to start the installation, but some prompts are not visible.
Install Bazzite Deck Deck - 39

What did you expect to happen?
I didn't expect to have to use the monitor at all.   I expected both to be on the same screen.  Ideally it would be rotated in Landscape mode.
Hardware
Steam Deck OLED
Extra information or context
You have a comment at: https://universal-blue.discourse.group/t/bazzite-initial-setup-and-installation-guide/30#installer-requirements-15

Steam Deck OLED users: The installer only displays properly in Basic Graphics Mode.

However you don't explain how to enter this mode.  ... Is this the issue?  If it is, could you let me know how to get into this mode please.  e.g. is it a kernel param?

Basic graphics mode is a selection you make at the very start of the install process. At this time we do not support the OLED deck due to numerous issues with refresh rate control and sound. Once those are resolved we will modify the installer to handle OLEDs gracefully.


⚠ ClamAV has degraded performance

In 9edce1c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in b7e2bec.


Issue with external scripts when in dev mode

Environment

Operating System: Darwin
Node Version:     v22.7.0
Nuxt Version:     3.13.2
CLI Version:      3.13.2
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules, css, compatibilityDate
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-1bvrus?file=app.vue
Describe the bug
Hello!
I'm trying to add a CRM form that is being provided via external scripts. When running in dev mode, the following error appears in the console:
TypeError: Cannot read properties of undefined (reading 'extend')
and the form won't render.
Here’s a minimal reproduction link.
The issue doesn’t occur when I write the same code using pure Vue or in a Vue SSR sandbox. Here’s the expected result using pure Vue SSR.
It also doesn’t appear after I run npx nuxi generate.
My guess is that it’s somehow connected to the Nuxt dev server, but unfortunately, I don’t have enough expertise yet to understand how everything works under the hood.
I’d appreciate any suggestions!
Additional context
No response
Logs
No response

Looks like this script

Doesnt have access to Vue instance and therefor it fails to execute code

Yes, but as this form is generated by SaaS, there’s only so much I can do. It’s also supposed to be separate from my site’s code and should work in most situations.
My biggest concerns with this issue are:
Why does it work in pure Vue then?
Why doesn’t it work after I generate a static page in Nuxt?
It seems like there might be differences in how the page is rendered in Vue vs Nuxt (even though Nuxt uses Vue), even with SSR turned off. That’s what I’m trying to understand) May be there's a bug or a room for improvement for Nuxt in this case.


Problems with building xds-2.60

Hello,
I tried building xds-2.60 on Windows 10 according to the instructions provided; however, I ran into a problem. The last successful step of the build was "====== POSIX C sym-files ======== OK!", and the failure occured in the step,
START QUOTE
xm src\m\xm =p -mode:=enduser -ts_compatible:+ -dbgfmt=HLL -decor=rt -woff+ -env_target=winnt -env_host=winnt 1>log\xm.out 2>&1
NMAKE : fatal error U1077: 'xm' : return code '0x1'
END QUOTE
On examination of xm.out I found that it ends in,
START QUOTE
Dirs.c
C:\Users\Bostjan\AppData\Local\xds-2.60\build\Win32\enduser\tmp\Dirs.h(7): fatal error C1083: Cannot open include file: 'X2C.h': No such file or directory
NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.34.31933\bin\HostX86\x86\cl.EXE"' : return code '0x2'
Stop.
external command fault 2:
nmake /nologo /f enduser\tmp\xm.mkf
END QUOTE
I looked in bin\xds\include\C and found that X2C.h is present, so I'm not sure why the build stopped.
Regards,
bbostjanv

Hello!
Could you please send the "enduser\tmp\xm.mkf" file?
It looks that it was generated with wrong "/I" instructions.
Thank you in advance,
Andrey

Hello,
Just to conclude this issue, it turned out all my problems resulted from using a newer VS. After appropriate changes were made the build proceeded as advertised. Perhaps the README should mention some of these problems. In case it might be
helpful I am including my version of the .bsc and setenv.bat files:
https://1drv.ms/u/s!AnO7wUiRDtNH63cDrDw7SV3U19xy?e=AsRBcg
https://1drv.ms/u/s!AnO7wUiRDtNH63iIZBq5VRXuwsql?e=O9P3NV
Regards,
bbostjanv


Getting false positive on various Galaxy models

I am using the latest version of your library but I have received more than 20 reports from my users telling that they get a root warning message, and most of them use Samsung Galaxy devices.
The devices which had the false positive are:
Galaxy S6 Edge
Galaxy S7 and S8
Galaxy Note 20
Galaxy S10
Galaxy S20

@scottyab !!!!!

@scottyab !!!!!

Hey @NotNotMarshall I've deleted your other comment as it's not helpful here. You'll need to provide us we some more details on the root check that's failing. Please reopen when you have some details to share. Thanks

Hey @NotNotMarshall I've deleted your other comment as it's not helpful here. You'll need to provide us we some more details on the root check that's failing. Please reopen when you have some details to share. Thanks


"Settings Sync: Select Service" not present in Insiders command pallet

Issue Type: Bug
I want to sync my settings from Insiders to VSCode, and I found this section of the docs:
https://code.visualstudio.com/docs/editor/settings-sync#_syncing-stable-versus-insiders
They refer to a "Settings Sync: Select Service" option. I searched for this option in the command pallet, but couldn't find it - it seems to be missing to me.
Thanks
VS Code version: Code - Insiders 1.64.0-insider<PHONE_NUMBER>8229accd11fdfe1cba36a908208dc9, 2021-12-09T05:14:18.136Z)
OS version: Darwin arm64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (0.12GB free)

Process Argv
--crash-reporter-id d6b55923-ea5b-4116-93ff-b1744deac02c

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

prettier-vscode
esb
9.0.0

codespaces
Git
1.4.1

copilot
Git
1.7.4016

go
gol
0.29.0

terraform
has
2.17.0

jsonnet-language
lia
1.0.0

rainbow-csv
mec
1.10.1

vscode-docker
ms-
1.18.0

remote-containers
ms-
0.209.3

remote-ssh
ms-
0.68.0

remote-ssh-edit
ms-
0.68.0

remote-wsl
ms-
0.63.4

vscode-remote-extensionpack
ms-
0.21.0

sublime-keybindings
ms-
4.0.10

vsliveshare
ms-
1.0.5196

vscode-commons
red
0.0.6

python
tht
0.2.3

markdown-all-in-one
yzh
3.4.0

vscode-proto3
zxh
0.5.5

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256197
pythontb:30258533
pythonvspyt551:30291412
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805cf:30301675
pythonvspyt200:30323110
bridge0708:30335490
bridge0723:30353136
pythonrunftest32:30365365
pythonf5test824:30361779
javagetstartedc:30350118
pythonvspyt187:30365360
vsaa593cf:30376535
vsc1dst:30396469
pythonvs932:30404738
vscop804:30404766
vs360cf:30404996
vsclayoutctrt:30405800
py55gd98:30408621
vscexrecpromp3t1:30407758
azactmsal:30409287

You can select service only while turning on settings sync, as written in the doc
You can sync your Insiders with Stable by selecting the Stable sync service while turning on Settings Sync. This option is only available in VS Code Insiders.

So, please turn off and turn on sync again.

@sandy081 ok thanks, you are right, that was written in the text of that help article - many thanks for your help.
I wonder if the docs could be improved with more imagery, or more context on the image which is already included, in order to make it clearer that you can't simply change that setting directly via the command pallet.
In combination with another issue which I reported on Zendesk: https://support.github.com/ticket/1421927 this problem caused me a lot of pain and time wasted.

Thanks for the feedback.
Definitely, I will wait until I listen more feedback before doing any changes.


ESP32: implement UART support

UART is not supported on ESP32 right now, it shoud be implemented using following APIs: https://docs.espressif.com/projects/esp-idf/en/release-v3.2/api-reference/peripherals/uart.html

Feature currently in progress.

UART is now implemented on ESP32.


Require 'rails_helper' consistently

Noticed that some specs here were requiring 'spec_helper' and some
weren't requiring anyting at all.  I changed them all to use
'rails_helper' as per standard practice.

Incidentally, there is a reason for not requiring rails_helper: if you don't need to load the whole Rails, it's much faster, which makes TDD more pleasant. require 'spec_helper' is always redundant because that's already specified in .rspec.


Getting a 400 Error on the DFP requests

I have 2 ad units on the page, and they don't load.
In safari, I see the follow error in console for each:
Failed to load resource: the server responded with a status of 400 (Bad Request)

URL requested is like (I have replaced identifying info and replaced the DFPid with XXXXXXX:
https://securepubads.g.doubleclick.net/gampad/ads?gdfp_req=1&correlator=3779269110005760&output=json_html&callback=callbackProxy&impl=fif&eid=108809080%2C108809049&sc=0&sfv=1-0-2&iu=%2FXXXXXXX%2FAd_Unit_Name&sz=960x0&cust_params=UrlHost%3Ddomain.com%252Cdomain.com%26UrlPath%3D%252Ffile-name.php%252C%252Ffile-name.php%26UrlQuery%3D&cookie=ID%3Df6c5a6b44c82b8f6%3AT%3D1415104988%3AS%3DALNI_MZZvAlvhI1KblUjxGWRbm3zKiTufw&lmt=1443721122&dt=1443706722796&cc=100&frm=20&biw=1383&bih=884&oid=3&adx=212&ady=2690&adk=1055041421&gut=v2&ifi=4&u_tz=-240&u_his=3&u_java=true&u_h=1080&u_w=1920&u_ah=1057&u_aw=1885&u_cd=24&u_nplug=16&u_nmime=105&u_sd=1&flash=19.0.0&url=http%3A%2F%2Fdomain.com%2Ffile-name.php&dssz=98&icsg=559776&std=11&vrg=73&vrp=73&ga_vid=107901648.1245432157&ga_sid=1443706720&ga_hid=1713498263&ga_fc=true
Not sure if it is relevant, but the ads being served in these units are custom creatives that contain javascript blocks.

are you certain about the ID? I had the same error but after checking the ID and matched it with a correct one I got it working!

Sorry was wrong, I'm having the same issue...

I'm also having this issue. I think I had it previously and fixed it by removing other google adsense code on the page (from an old ad setup) and using jquery dfp exclusively. However I think i'm having this problem again on a different site and I think I've removed all the old adsense code.

If any of you have an example URL where this is happening that I can view can you please share it here... or email it to me at<EMAIL_ADDRESS>if you don't want to have it in public

@coop182 Actually I just fixed it. (disregard my email). The problem was I hadn't set the data dimensions.

Cool... I just had a look and it was working so that was confusing haha.... glad you got it working! :smile:
@markcarey or @Preen if you want me to take a look at your URL's just let me know.

thanks for your help!

@markcarey — did you ever resolve this?
We are running into the same issue, have data-dimensions set correctly, but still get the 400 (Bad Request) error occasionally coming back with all with blank ads.


[CWS] skip raw packet event when no process context

What does this PR do?
Do not forward rawpacket event to userspace when there is no process context. This should limit the performance impact. We'll remove this limitation later.
Motivation
Describe how to test/QA your changes
Functional tests should still pass
Possible Drawbacks / Trade-offs
Additional Notes

/merge


ch3-simulation minor

labelfix: https://app.reviewnb.com/gabors-data-analysis/da_case_studies/blob/master/ch03-simulations%2Fch03-distributions.ipynb/file/#comment-nb-9543625e

b8aed59


Shared notes activity suggestion

--

You're wrong, a user who join see the current set of notes.

Oh I see, I expected changing of text to be reflected reflected as the typing is being done rather than once the typing is completed and the box is clicked.
I apologize for such an embarrasing mess I made here. Be assured this is not gonna happen again.
I close this issue without delay.


Do not use in-memory root for citadel

If Citadel fails to update the CA-root stored in K8s secret, it uses an in-memory root for the cluster. This was introduced in very early stages to support testing Citadel in special environments (non-K8s). Now we should not support in-memory root, because the in-memory root is not sustainable for production usage. It may also cause issues in special scenarios. For example, when Citadel fails to read the existing CA-root secret, it may choose to generate a separate root cert and use it to issue certificates for the cluster, which conflicts with the existing root cert.
@wenchenglu @wattli @lei-tang

The fix went out in 1.1.8 and 1.0.8.  Is there any reason to keep this issue open?

We should keep this open until the change gets into 1.2 probably (#14505)

@myidpt this is a 1.2 release blocker

Closing. Changes merged to 1.0, 1.1, 1.2 and master branches.


fix(swagger):Update docs.go - fix typo

What type of PR is this?

Update docs.go - fix typo

Which issue does this PR fix:

swagger.json has typo.

What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:

swagger.json open by swagger-ui

Testing done on this change:

None.

Automation added to e2e:

None.

Will this break upgrades or downgrades?

No.

Does this PR introduce any user-facing change?:

No.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@arukiidou may i know why this PR was closed?

@rchincha
sorry. closed because it failed to create a commit log, and CI failed.

retrying it https://github.com/project-zot/zot/pull/1055


Add support for indexing collection Transfer command

Description
As a part of #1, we have to update streamer project to support the "Transfer" command of the collection module.
Acceptance criteria
Service should be able to
Index transactions of collection:transfer
List collections with the latest updates
Filter collections by
    collectionID
    creatorAddress
    offset
    count
The above functionality must be tested (automated).

It's been done, reviewed.


PIL OSError when font is missing

PIL throws an OSError when it can't load a font

Traceback (most recent call last):
  File "enhance-ir.py", line 338, in <module>
    init()
  File "enhance-ir.py", line 82, in init
    handle_input(args.INPUT)
  File "enhance-ir.py", line 184, in handle_input
    process(input)
  File "enhance-ir.py", line 244, in process
    fnt = ImageFont.truetype("arialbd.ttf", size=38)
  File "/usr/lib/python3/dist-packages/PIL/ImageFont.py", line 280, in truetype
    return FreeTypeFont(font, size, index, encoding, layout_engine)
  File "/usr/lib/python3/dist-packages/PIL/ImageFont.py", line 145, in __init__
    layout_engine=layout_engine)
OSError: cannot open resource

Possibly caused by non-Unicode characters in font path

https://github.com/python-pillow/Pillow/issues/3480
https://github.com/python-pillow/Pillow/pull/3785
https://github.com/python-pillow/Pillow/issues/3145
pil-issue-oserror-cannot-open-resource

Same issue after updating to Pillow 7.0.0

Added try/catch for missing fonts in 92df164

I have the same issue when using Windows 10 with  Japanese and trying to open the font ARIAL.TTF
I'm using version 7.0.0 and handle dependencies with Poetry
Traceback (most recent call last):
  File "render_annotations.py", line 240, in <module>
    main(file_path)
  File "render_annotations.py", line 216, in main
    write_image_with_annotations(image_annotations, url, item["key"])
  File "render_annotations.py", line 182, in write_image_with_annotations
    newly_drawn_text_box = draw_annotation_to_image(annotation, image, drawn_text_boxes)
  File "render_annotations.py", line 154, in draw_annotation_to_image
    return draw_text(annotation, image, drawn_text_boxes)
  File "render_annotations.py", line 112, in draw_text
    fnt = ImageFont.truetype("ARIAL.TTF", ANNOTATION_FONT_SIZE)
  File "C:\Users\Man\AppData\Local\pypoetry\Cache\virtualenvs\qa-show-all-annotations-gvzPHQ0M-py3.7\lib\site-packages\PIL\ImageFont.py", line 642, in truetype
    return freetype(font)
  File "C:\Users\Man\AppData\Local\pypoetry\Cache\virtualenvs\qa-show-all-annotations-gvzPHQ0M-py3.7\lib\site-packages\PIL\ImageFont.py", line 639, in freetype
    return FreeTypeFont(font, size, index, encoding, layout_engine)
  File "C:\Users\Man\AppData\Local\pypoetry\Cache\virtualenvs\qa-show-all-annotations-gvzPHQ0M-py3.7\lib\site-packages\PIL\ImageFont.py", line 188, in __init__
    font, size, index, encoding, layout_engine=layout_engine
OSError: cannot open resource
PS C:\Users\Man\Documents\GitHub\crowd-ops\Projects\Add-Annotations>

I was able to find a work-around copying the file to the same folder as the script and then changing:
fnt = ImageFont.truetype("ARIAL.TTF", ANNOTATION_FONT_SIZE)

to
fnt = ImageFont.truetype("./ARIAL.TTF", ANNOTATION_FONT_SIZE)

Ok interesting, this seems to be a bit tedious to get going cross-platform. I'll investigate.


Prefered way to force redraw of rows/items.

Currently when options like item sizes are changed I have to somehow force the component to redraw. It seems that emitting the options itself is not enough to trigger any sort of redraw.
If I only emit options they seem to only take effect as I scroll through the items and the rows are updated.
For now I'm emitting an empty array and then right after that the data array, this seems to make it redraw but it feels messy.
 subs.push(this.debounceResize$.subscribe(() => {
        observer.next([]);
        observer.next(curData);
      }));

Hi @ktersius, yeah, this is an issue I am aware of and I will look into it. The reason I did not implement it in the first place is that I was unsure, whether it could harm the performance or not.
But as there seems to be a use case, I will evaluate the practicability.
Thanks!

Yup, I met the same problem today, but I solved it by this tricky way.
In templates
<od-virtualscroll [vsData]="data$" [vsOptions]="options$">
...
</od-virtualscroll>

In components
data$ = new Subject<number[]>();

change() {
    // use this to force redraw
    this.data$.next([]);
    this.data$.next(new Array(100).fill(0).map(x => Math.floor(Math.random() * 1000)));
}

Have you had a chance to look into this? I'm facing a similar issue.

Not yet, but I (or someone else :+1:) will!


Edit and Post Recording from 10/28/2019 Community Call

Edit zoom recording from 10/28 call and post to YouTube channel for PresQT.

Video on OSF. Ready for YouTube


Can't play the macro

Hi everyone!
For the first time, i downloaded the zip and tried to use it.
I can launch the program, i can record macro but i can't play them to do the same action again.
Just to know if i'm doing good :

I press the "play" button
Then, i press the record button
I see the number of frame going higher and higher
I press the stop then the record button.

I can save the macro in a xml file, see inside the macro what is recorded but i can't play this macro.
When i press "play", i see the frame going from 0 to the end but nothing happen :/
Am i doing something wrong?
An other thing that maybe can help, i don't have the picture for the "stop" and "record" button

Thx for the help!

Hey there, I know you asked this a long time ago, so I hope you've got it going now. If not you're doing the steps slightly out of order.
You'll need a dualshock 4 plugged into the pc or have the emulation set to true.
You'll first hit record and the line at the bottom right should turn red.
You'll then hit play and do the inputs on the controller.
You'll hit stop, then make sure to also toggle off the record button. It should go back to black.
Once all that is one you should be able to hit play and your macro will run on a loop.
You'll know it has worked because where the line in the corner is that will be replaced with number/numbers.
One thing to keep in mind is under playback there's the record on touch selected by default. That's the touchpad I usually will toggle that off to keep the app from going back into recording mode.

Hey there, I know you asked this a long time ago, so I hope you've got it going now. If not you're doing the steps slightly out of order.
You'll need a dualshock 4 plugged into the pc or have the emulation set to true.
You'll first hit record and the line at the bottom right should turn red.
You'll then hit play and do the inputs on the controller.
You'll hit stop, then make sure to also toggle off the record button. It should go back to black.
Once all that is one you should be able to hit play and your macro will run on a loop.
You'll know it has worked because where the line in the corner is that will be replaced with number/numbers.
One thing to keep in mind is under playback there's the record on touch selected by default. That's the touchpad I usually will toggle that off to keep the app from going back into recording mode.


Advisory board logos should be black and white and get color when hover upon.

Acceptance Criteria:

[x] Advisory board logos should be black and white and get color when hover upon.

[ ] Add the hand instead mouse when you are inside icons. For more details refer the image below.

@partini Please see this detail.

@georgeslabreche @dafinaolluri Should the mouse be displays when hovering the logos, like shown picture attach ?
https://cloud.githubusercontent.com/assets/21961919/22020634/8a32b384-dcb9-11e6-94a8-5e9403091791.png


[DS-6891][MasterServer] reduce db operation when process instance running

#6891
Brief change log

use workflow memory cache instead of selecting processInstance and taskInstance from DB each time
reduce the repeat query
remove the old design TaskInstanceCacheManager

Codecov Report

Merging #6904 (688ddfb) into dev (f5e7da3) will decrease coverage by 0.18%.
The diff coverage is 23.17%.

@@             Coverage Diff              @@
##                dev    #6904      +/-   ##
============================================
- Coverage     33.07%   32.88%   -0.19%
+ Complexity     1618     1617       -1
============================================
  Files           433      432       -1
  Lines         14292    14322      +30
  Branches       1427     1437      +10
============================================
- Hits           4727     4710      -17
- Misses         9111     9160      +49
+ Partials        454      452       -2

Impacted Files
Coverage Δ

...ver/master/consumer/TaskPriorityQueueConsumer.java
0.00% <0.00%> (ø)

...uler/server/master/processor/TaskAckProcessor.java
23.52% <ø> (-2.79%)
:arrow_down:

...server/master/processor/TaskResponseProcessor.java
0.00% <ø> (ø)

...ter/processor/queue/StateEventResponseService.java
0.00% <0.00%> (ø)

...r/server/master/runner/task/BaseTaskProcessor.java
5.22% <0.00%> (+0.07%)
:arrow_up:

...server/master/runner/task/CommonTaskProcessor.java
6.45% <0.00%> (ø)

...ver/master/runner/task/ConditionTaskProcessor.java
0.00% <0.00%> (ø)

...ver/master/runner/task/DependentTaskProcessor.java
0.00% <0.00%> (ø)

...er/server/master/runner/task/SubTaskProcessor.java
0.00% <0.00%> (ø)

...server/master/runner/task/SwitchTaskProcessor.java
0.00% <0.00%> (ø)

... and 18 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 f5e7da3...688ddfb. Read the comment docs.


Discussion: API to peek if a transfer is complete?

One of the key operations in the FireFly core aggregator, is checking if a message is complete.
In the case of a pinned private message, there are three layers that all have to be complete:

The blockchain pin transaction
The message batch containing the metadata for any blobs, along with the message itself
The blobs

So we need a quick way to check if these are all available without downloading the large blob into the core.
I don't think there's an API to do this currently on the HTTP data exchange, or a way to know the difference between a partially completed write vs. a completed write. Or to know what the hash is of a transfer to know if it's valid.
The event that's emitted when the transfer is complete is fantastic for waking up the logic in core, to prompt it to go back and check if the message is ready from all the other counts (including all previous messages on the topic being ready). However, that's fact that the blob is ready is not persisted anywhere.
Seems like there's two major options (assuming we don't put an indexed store into dx-https itself):
Option 1: A change to dx-https such as

Write the files with a .part extension while the transfer is in-flight
Write the hash of the file as calculated by dx-https to a .hash once the transfer completes
Then rename the file to a deterministic location (losing the details of where it came from - think this is the challenge)
Provide an API that queries the existence of the file, that just does a direct file name check (no expensive ls etc.)

Option 2: Add an extra collection in core for blobs

Updated only as a result of transfer events
Stores a mapping from a transfer event (namespace, ID, hash), to a path to query it from

@gabriel-indik - would like to discuss this in the morning if that timing works for you

Looking forward to having the conversation. Some information to consider ahead of it:
The blob-received WebSocket event which is sent to the recipient once the file has been stored includes the hash of the file. This way the core does not have to download it. The event also includes the sender and the blob path. In other words, at the time the event code was being written the 2nd option you described was the one I had in mind. Having said that, we can definitely look into expanding the API of data exchange to allow querying for files using senders/hashes. We can probably use something like levelDB to create the entries after the transfers are complete. This way we would avoid the need to store in-flight files with a different name.

The following changes will be included in data exchange:

A metadata file will be stored for each blob with its calculated hash
A new API endpoint will be added to query the metadata
In transfers, the metadata will only be written after the transferred file has been successfully stored

So, it looks like core will still need to keep its own index.
The reason is that in FireFly core the data object is (very deliberately) not tied to a particular transfer. You can take that same object, and pass it many times, to many different parties. So there's no direct coupling of a peer to a data id, that would allow core to make a query to DX to ask for "data in namespace X with ID Y, received from peer Z". Only X and Y are known.
However, I do think the way the HTTPS DX is structured makes perfect sense - it tracks what's been sent, what's been received etc.
So I think there needs to be an index in core that tracks the mappings from DX blob references (which can be anything) to hashes. Then it's perfectly acceptable for DX to contain multiple copies of a piece of data, and core simply needs one that matches the hash to be able to process a message.


.so loading ModuleLoader

I've created a (mostly untested) ModuleLoader that loads .so file, and imports functions that don't start with '_' as system functions. The relevant code is in src/dyn_loader.rs.
This is unsafe and easy to cause segfaults with so I would recommend against bringing it in tree, but I thought you might like to know about it ;)
link

Loading all symbols is definitely an interesting way of doing this. Some other method that's a bit saner and safer would be preferred and may be implemented eventually. I'm not sure about the logistics of building Rust programs that load plugin libraries.

Saner and safer would certainly be nice. I don't think it's possible at present, but hopefully the ecosystem (and compiler) will develop in this area. There has been pretty much no work on type safe plugin loading in rust that I'm aware of.


Jobs page should allow users to expand their search if it returns no results

We're currently searching for USFWS jobs in the southeast region by State name.  If we get zero results on the initially search we should update the page to show USFWS nation wide.  We should also provide a button to toggle between R4 and national job openings.

Closed via d7ee2e43cb4d6ccc0dbe7cf6389733b2d4450901.


Transitioning from "node-red-contrib-azure-iot-hub" palette

Thank you for developing the Azure-IoT-Device palette. The "direct" node works great for my publishing and direct method needs. I have tested my application offline and am trying to incrementally incorporate the new "direct" node into a production system that includes 25 flows, each flow communicating with various subsystems on various communication protocols (Modbus, SNMP, OPC, etc.). My existing production system uses the "node-red-contrib-azure-iot-hub" palette for just publishing telemetry in each of the 25 flows.
When I edited the first flow to include the "direct" node I noticed all IoT nodes from the legacy palette stopped publishing telemetry to the IoTHub. I was hoping to replace one flow at a time in the production system but this result creates a problem. Is this a know problem? Is there any work around? Thank you.

Related to the above I've tried everything I could think of:  export/reimport, edit node one at a time and redeploy, restart the controller, etc. All "direct" nodes are publishing duplicate payloads to IoT Hub but everything is very stable and reliable. I just don't want to be publishing double the payloads than what is necessary. Thank you.

same issue here, all the payloads (twins) are being published even to nodes without those 'properties'

@mgKeeKrog @BC-josgreeve thanks for reporting. I haven't tested using my node nect to the "node-red-contrib-azure-iot-hub" palette. I will check it out. Will also check the duplicate messages and twins. Each Azure IoT Device created with "node-red-contrib-azure-iot-device" should be individually addressable as twin.

Closing issue. Couldn't reproduce.


0.4.0

Automatically generated by python-semantic-release

@PaulHax I see now, the ci job did not trigger with the push of the github action. Ill check it out. Approving this. Thanks for pointing this out


Linting error  AnimatedCircularProgress not found in 'react-native-circular-progress'

import {  AnimatedCircularProgress } from 'react-native-circular-progress';
gives me an error when I run linting on my machine.
error AnimatedCircularProgress not found in 'react-native-circular-progress'
Versions:
"react-native-circular-progress": "^0.1.2",
"react": "16.0.0",
"react-native": "0.51.0",
lint: version 26.0.2
My .eslintrc rules:
 "rules": {
      "react/display-name": 0,
      "import/default": 0,
      "no-console": 1,
      "comma-dangle": 0,
      "import/extensions": 0,
      "import/no-absolute-path": 0,
      "import/no-unresolved": [2, {
        "ignore": [ "^/src", "/", ]
      }],
      "import/no-extraneous-dependencies": ["error", { "devDependencies": true }]
    }

This doesn't seem like an issue with the library but rather your linting setup. To ignore the wrong error, you could use // eslint-disable-line or something similar.


Allow to configure window open on web

Is your feature request related to a problem? Please describe.
When using flutter_web_auth_2 on web, it is not possible to configure how the auth window will open.
Describe the solution you'd like
I would like to configure the window.open call based on a custom configuration.
For example I would like to use a popup instead of a new window.
Describe alternatives you've considered
Currently there are no configuration options.
A custom implementation on a fork is the only possible option.
Additional context
This part
https://github.com/ThexXTURBOXx/flutter_web_auth_2/blob/a0a870280a8b37a25bbcc8bc2ca1231815ca2d49/flutter_web_auth_2/lib/src/flutter_web_auth_2_web.dart#L49
should be more configurable, so that e.g.
context.callMethod('open', [url, "popup", "popup=true"]);

can be achieved.

Thanks for the suggestion! This seems like a quick enhancement that I should be able to add very soon :)


Nestjs core instruction instrumentation with Jaeger exporter

Hi, I am trying to integrate the Nestjs application with jaeger using open telemetry and nest instrumentation by using the below configuration.
const provider = new NodeTracerProvider({
resource: new Resource({
[SemanticResourceAttributes.SERVICE_NAME]: 'bff-trace-test',
}),
});
const jaegarConfig: ExporterConfig = {
host: 'localhost',
port: 6832,
};
const exporter = new JaegerExporter(jaegarConfig);
provider.addSpanProcessor(new SimpleSpanProcessor(exporter));
provider.register();
registerInstrumentations({
instrumentations: [new NestInstrumentation()],
});
But i don't see any traces in a jaeger. Is there anything I am missing here?
As per my understanding, if we are using instrumentation, I don't have to create and end span's in order to see the trace correct?
Below are the versions I am using.
"@nestjs/common": "^8.0.0",
"@nestjs/core": "^8.0.0",
"@nestjs/platform-express": "^8.0.0",
"@opentelemetry/api": "^1.0.3",
"@opentelemetry/exporter-jaeger": "^1.0.1",
"@opentelemetry/instrumentation": "^0.26.0",
"@opentelemetry/instrumentation-nestjs-core": "^0.26.0",
"@opentelemetry/resources": "^1.0.1",
"@opentelemetry/sdk-trace-base": "^1.0.1",
"@opentelemetry/sdk-trace-node": "^1.0.1",
"@opentelemetry/semantic-conventions": "^1.0.1",

@dyladan  can you help me on this one?

@SunilManthenaG01 You can try using
import { diag, DiagConsoleLogger, DiagLogLevel } from "@opentelemetry/api";
diag.setLogger(new DiagConsoleLogger(), DiagLogLevel.DEBUG);

To make sure that spans get created, so that issue isn't with Jaeger exporter.
Also, one thing to keep in mind, you should do New NestInstrumentation() before you import anything from @nestjs/core. For example:

import { NestInstrumentation } from '@opentelemetry/instrumentation-nestjs-core';

... opentelemetry stuff ...

import { NestFactory } from '@nestjs/core';
import { AppModule } from './app.module';

async function bootstrap(): Promise<void> {
  ... nestjs stuff ...
}

@mentos1386  thank you for your response.
After some debugging, I see the issue is because I am using spdy listening on h2 protocol. If I switch that to HTTP 1.1 I can see the traces in jaeger but if the server protocol is h2 I can not see the traces.
Do we need to change anything if we need to make the trace work for h2 protocol?


[BUG - Release] dwmapi.dll is causing crashes in Palworld

Branch or Release
version 3.0
Game and Engine Version
Palworld
app ID:1623730
build ID:13390747
Describe the bug
I recently installed this along with two mods: Gameplay Tweak and Admin Engine. All was fine at first, then I began getting more and more frequent memory errors where it claimed the game attempted to access a location it wasn't allowed to, or that didn't exist. After checking my memory to be sure the issue wasn't hardware, I found a suggestion to remove dwmapi.dll from the folder. Doing this made the game return to a working state. After playing for a bit, I tested if reinstalling it would work, and for a while, it did. Then the game went back to having memory issues. Today, I uninstalled it again as a test, and once again, the game returned to functioning. It seems like this file is somehow making the game crash by attempting to access restricted or non-existent memory locations.
Mods directory
Mods.zip
To Reproduce
Steps to reproduce the behavior:

Launch game
Game immediately crashes.

Expected behavior
I would hope to be able to play the game without crashing.
Desktop (please complete the following information):

OS: Win10

Same issue here. I removed the file and the game launched but the mods stopped working.

dwmapi.dll is what injects UE4SS into the game. So removing the file is basically removing UE4SS.
Does this issue still occur if you turn off all of your mods but keep dwmapi.dll (i.e. UE4SS is still injected)? Please download the latest release - 3.0.1 - and use that.
If it does, then there is indeed something wrong with UE4SS. If it does not, then it is an issue with one or more of your mods.

this is what I am getting for the crash report screen (I set all the mods in the mod.txt folder to 0 as I am assuming that turns them off)

I set all the mods in the mod.txt folder to 0 as I am assuming that turns them off

Setting a mod to to 0 in mods.txt doesn't turn off the mod if there's an enabled.txt file in that mods directory.
To turn off a mod, the mod must be set to 0, or not exist in mods.txt at all and there needs to be no enabled.txt file.

I set all the mods in the mod.txt folder to 0 as I am assuming that turns them off

Setting a mod to to 0 in mods.txt doesn't turn off the mod if there's an enabled.txt file in that mods directory. To turn off a mod, the mod must be set to 0, or not exist in mods.txt at all and there needs to be no enabled.txt file.

ah ok. Ya getting rid of everything in th mods.txt seems to allow me to load into a game now.
Posting screenshots of how everything looks like

Ok so downloaded a mod and had it enabled and its working perfectly fine, so I'm guessing the issue is with one of the mods that came with the Mods folder

dwmapi.dll is what injects UE4SS into the game. So removing the file is basically removing UE4SS.
Does this issue still occur if you turn off all of your mods but keep dwmapi.dll (i.e. UE4SS is still injected)? Please download the latest release - 3.0.1 - and use that.
If it does, then there is indeed something wrong with UE4SS. If it does not, then it is an issue with one or more of your mods.

I'll give this a try soon. I only just saw this comment.

Ok, so installed 3.0.1, cleared my mod file, updated the mod text file and reinstalled the mods I wanted to use. Am going to play a bit to see if everything is working properly or not. Last time I removed dwmapi.dll then restored it, the game ran well for a few hours, then began crashing constantly. Will update after a while of playing.

I played for about two hours without any issues. I will try one more time before closing this though; I did the same thing before opening this, playing for a few hours, only for the game to start having errors at the end.
It turns out I had a large number of unknown files in my mod folder. I did not install these myself, and I think I picked them up by playing on a friend's server; I plan to ask them about what mods they were using. I cleaned everything up though, and hopefully this will have fixed the problem permanently. I'll make one more post sometime tomorrow, and if all is well, I'll close this.


Date Picker reset

Summary
We're incorporating a date picker into our design and would like to provide the ability to "clear" or "reset" whatever a user has selected. Right now, we have a clear button show up below the date picker but this pushes the items down. We're wondering if this is the best solution or if there are other ways with icons or something inline with the date picker to better support this action.
Relevant information

Example

Upon browing through some components it looks like "file uploader" has the type of action that we are looking for. Would the other input components benefit from having this type of action built in?


Add CJS/Min Builds

Hold: This feature is on hold until some bleeding-edge features (ex class fields) are supported by RollupJS without transpilation
Add builds for publishing
Checklist

[x] build:cjs config
[x] cjs build
[ ] build:min config
[ ] minified build

CommonJS added in 1.0.10

Minified added in v1.0.11


chore: go-demo-6 to 1.0.407

chore: Promote go-demo-6 to version 1.0.407

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dariver: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

promotion-build
7a72c9ddee3463e0da4dd9004d2a436a6bf2c98a
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


HttpMessages implementing HttpContent types too aren't handled correctly

Changes due to #12709 have added a code path where HttpMessage types that implement HttpContent types too are handled in their separate code paths while handled as just HttpMessage right after too.
The fact that CI doesn't fail there means that there are no types like that (and it's not impacting existing users using default types too).

All subclasses of io.netty.handler.codec.http.FullHttpMessage was affected. In our case it is AggregatedFullHttpRequest.


provide conmon documentation

Description
Currently this is very little to none documentation available on conmon and what it actually does.
There are some vague statements at https://cri-o.io/ for example, such as "Each container is monitored by a separate conmon process. The conmon process holds the pty of the PID1 of the container process. It handles logging for the container and records the exit code for the container process."
This doesn't make it clear on what function it actually does.
To avoid confusion it would be helpful if there was some documentation available on it's function and what it means that "each container is monitored by a separate conmon process" for example. Can conmon take any action if it detects some error with the container?
What does "handles logging" mean?
The architecture image on https://cri-o.io/ is a good starting point but perhaps it can be better described what it then integrates with, when I quickly look through the code I can see journald for example being used.
This type of integration would be good to have documented so that I can better understand what it does.

We haven't added much documentation on Conmon because we treat it purely as an implementation detail; no one should be using Conmon except for Podman and CRI-O, and we don't even put it in the system's default $PATH as such.
In brief, Conmon is an extremely small C program with almost no memory footprint that wraps the OCI runtime (usually runc) that runs the actual container. Upon being launched by Podman, it double-forks to daemonize and detach from the parent that launched it, then launches the OCI runtime as its child. It will live for as long as the runtime (and container) live, and when the container dies it records its exit time and exit code for Podman or CRI-O. While the container is alive, Conmon provides a socket for attaching to the container, holding open the container's standard streams and forwarding them over the socket (and also writes the contents of the container's streams to a log file so podman logs and CRI-O can read them). And that's about it.
In short, it's an extremely minimal C binary that manages containers so that Podman processes can die in the foreground and the container will still be under our supervision, available to be attached to and recording exit information when it dies. Each container has a single Conmon process that acts as its parent until the container dies, and each Conmon process manages a single container.

A further note: "can conmon take any action if it detects some error with the container?"
No. Conmon is extremely minimal, and doesn't have the logic to detect or handle errors. If we wanted to add such logic, we'd do it in Podman and CRI-O, to keep Conmon simple and its memory footprint minimal.

@mheon What you wrote above should probably be added to the conmon readme. :^)

I agree. I asked @opuk on IRC to open the issue here.
Although conmon does not do much, the more curious users (and devs) are currently having a hard time finding out what conmon does. For some, it might seem suspicious that some additional but undocumented binary is installed.

We should also create a man page for it.

do folks agree this should be moved to the conmon repo?

Actually conmon has documentation now at github.com/containers/conmon. Closing this issue as such, reopen over there if you disagree


chore: remove unused metrics-exporter

This PR removes the metrics-exporter which was only exporting a test metric.
It also fixes a bug where lvm operator was not exposing controller runtime metrics.

/test all

Codecov Report

Merging #425 (728670e) into main (3b83b72) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #425   +/-   ##
=======================================
  Coverage   62.26%   62.26%
=======================================
  Files          27       27
  Lines        2335     2335
=======================================
  Hits         1454     1454
  Misses        749      749
  Partials      132      132

/test all

/test all


Adjust raspotify config to reflect its new volume control parameters

https://github.com/dtcooper/raspotify/commit/2594e1dff9a6b710a29673717a726bca014a82d8 introduced new volume control arguments. This PR adjusts the raspotify config to reflect these new volume control arguments.

In https://github.com/dtcooper/raspotify/pull/352, raspotify's README has also been updated accordingly to reflect the new parameters.

In https://github.com/dtcooper/raspotify/pull/352, raspotify's README has also been updated accordingly to reflect the new parameters.

@nicokaiser please merge this when you get the chance, it fixes an issue where raspotify is installed in a broken state (also reported in #87)


Incredibuild support

For the record: Now that 4 component are built by IncrediBuild (ITK, NiftyReg, SuperElastix, and elastix), a complete VS2013 Debug build of SuperBuild (from a freshly generated build tree) takes me about 20 minutes.

Without IncrediBuild, a complete VS2013 Debug build of SuperBuild (from a freshly generated build tree) takes me about 52 minutes!

Works at my machine too! Closed.


Setting RazorPagesOptions.AllowAreas to false does not disable area pages

From @dougbu on February 10, 2018 21:26
Change Startup.ConfigureServices to include
            services
                .AddMvc()
                .SetCompatibilityVersion(CompatibilityVersion.Version_2_0)
                .AddRazorPagesOptions(options =>
                {
                    options.AllowAreas = false;
                });

Expected: All area-related pages result in 404s.
Actual: See the usual content of pages such as https://localhost:44312/Identity/Account/Manage or https://localhost:44312/MyArea
Note either CompatibilityVersion.Version_2_0 or AllowAreas == false should have been sufficient to disable areas.
Copied from original issue: aspnet/Mvc#7348

From @pranavkm on February 14, 2018 15:55
Is this specifically with pages from a precompiled class library? I can't reproduce this with files on disk.

No, problem wasn't specific to area pages in a library. I also cleaned and rebuilt my solution a few times to ensure I wasn't testing older code. Then again, my test app does use area pages from a library -- the Identity.UI pages in particular.
Which SDK build are you testing with? I'm using 2.1.300-preview1-008117.
I'll test this again with 2.1.300-preview1-008123 later today.

Turns out the problem was my Startup calls IdentityBuilder.AddDefaultUI(). This has the unexpected behaviour of enabling a IPostConfigureOptions<RazorPagesOptions> implementation (IdentityDefaultUIConfigureOptions<{IdentityUserType}>) which overrides this user's RazorPagesOptions.AllowAreas setting.
Side note: That configuration setup also overrides user StaticFileOptions.FileProvider settings. Fortunately, other options properties set in IdentityDefaultUIConfigureOptions<{IdentityUserType}> seem to be additive.
Will move this issue to the Identity repo.

From @mkArtakMSFT on February 14, 2018 18:1
@dougbu, can you please confirm this is not the case for usual (local) views? /cc @javiercn

@dougbu, can you please confirm this is not the case for usual (local) views?

Not sure of your exact question @mkArtakMSFT. But, the effect of setting RazorPagesOptions.AllowAreas to true unconditionally is global -- affecting area pages on disk as well as those in the Identity.UI library. And, it overrides the user's chosen compatibility version as well as any implicit or explicit change they've made to AllowAreas.

This is sort of expected, AddDefaultUI is really forcing some opinionated settings on MVC as well.
cc @davidfowl I think this is another point in favor of making one uber AddDefaultIdentityEntityFrameworkUI<TUser, TRole, ApplicationDbContext>() method since now its just not auth settings that identity will clobber, its also clobbering razor and static files

@HaoK how would that uber method help here? The issue is the unexpected clobbering, not the number of extension methods needed to thoroughly clobber anything.
My suggestion is for dentityDefaultUIConfigureOptions<{IdentityUserType}> to also implement IConfigureOptions<RazorPagesOptions> and IConfigureOptions<StaticFileOptions>. Those new methods would handle RazorPagesOptions.AllowAreas and StaticFileOptions.FileProvider while allowing later user overrides.
One alternative would be for the existing methods to throw if reached when the options aren't set as expected. That would give the user the knowledge to choose between setting AllowAreas==true or using the default Identity UI.
Finally, overriding StaticFileOptions.FileProvider unconditionally may just be a bug. Could add a null check there.

I think the default UI requires areas so I don't think its unreasonable for that to set allow areas = true.
If you want to override it, you can still turn it off again after you call AddDefaultUI right?
Basically my point is mostly just that identity sets up a bunch of opinionated defaults to make things 'work' by default.

Oh I see, so you are just complaining about the fact that the UI is doing the stomping in post configure?  So users would have to override by adding a PostConfigure(o = > o.AllowAreas = false)...  but its kind of nonsensical to use both DefaultUI and try to turn areas off....

it's kind of nonsensical to both use DefaultUI and try to turn areas off

Not sure I buy that. It's an internal implementation detail of the Identity UI that it uses areas. Why would a user think the two features are related?
Separately, we could also go at this by removing the global option's impact on area pages in libraries. Users could then set the option however they like. See also aspnet/Mvc#7346. (Note IdentityDefaultUIConfigureOptions<{IdentityUserType}> does not change the options mentioned in that issue. But, they still have the potential to break the Identity UI.)

So AllowAreas = false with AddDefaultUI would result in blocking areas except for ones added by libraries?
I fully agree that its a mystery what AddDefaultUI does...  I pushed strongly for naming it something explicit like AddRazor/MVCUI, but Dan/Damian liked a more generic name rather than mention any implementation details in the name...

So AllowAreas = false with AddDefaultUI would result in blocking areas except for ones added by libraries?

Yup, that's what I'm suggesting.

It's an internal implementation detail of the Identity UI that it uses areas

Areas are not an implementation detail of how the Identity UI works, it is part the contract that we choose to have at the MVC level with other parts of your application.
We don't consider the scenario of turning off areas to be a common case and we favor an experience in which AddDefaultUI always gives you a result that works. In other words, we prefer doing anything we need to do to make it work other than throw an exception if things are not setup correctly.
Regarding the changes on the file system part, it's a small bug in there. I've sent out a PR to fix it. I'll let Hao review it.
https://github.com/aspnet/Identity/pull/1629/files

it is part the contract that we choose to have at the MVC level with other parts of your application.

That contract (between Identity and MVC) doesn't equate to user understanding.

Is the issue here that the DefaultUI is modifying MVC settings to make sure that it will work? That seems reasonable to me.
Is there any other issue being discussed here?

@Eilon my concern is more about user expectations and their ability to configure the application as they wish. AddDefaultUI(...) isn't just changing MVC settings so it will work, it's also resetting changes users may make.
The more I think about this, the more I like the idea of splitting user decisions about their application from provider decisions about libraries they provide. That's more general than AllowAreas having only two possible values and affecting all area pages everywhere. But, if gave library authors control over their libraries (as we did in localization), they wouldn't have to clobber user settings.

ability to configure the application as they wish

They absolutely are. Last person to change an option wins. The user's customizations of options should come last.

The user's customizations of options should come last.

Not true in this case.
Because Identity uses IPostConfigureOptions<RazorPagesOptions>, the only way users can change things would be to also implement IPostConfigureOptions<RazorPagesOptions> and add their instance after calling AddDefaultUI(...). That's not particularly discoverable or friendly and is why I earlier suggested having IdentityDefaultUIConfigureOptions<{IdentityUserType}> set AllowAreas in a new IConfigureOptions<RazorPagesOptions> method (i.e. non-post).

Ah, fair point, I did not realize that.
However, if this option isn't set, the UI won't work, right? So why would you want to disable it and use the UI?

So its a little odd that its post configure instead of normal configure, but its not that hard for them to override, they just do servives.PostConfigure(o => o.AllowAreas = false)

We are not planning to change how this works. I chatted at the time with @danroth27 and @DamianEdwards about this and we want an experience in which AddDefaultUI makes everything it can to make ensure that the UI is plugged-in and functional.


macOS complains about an "unidentified developer"

Is there a way to mark this app as from an identified developer with macOS so I don't get the scary warning every time I launch this app after a restart of my computer?

I'm not familiar with this, but every release is signed with PGP so that shouldn't be an issue.
Is this an apple-specific thing that might need to be specified in the app bundle?

Yep!
Info here: https://stackoverflow.com/questions/30297665/how-to-sign-my-app-as-identified-apple-developer#30477278

On Dec 13, 2018, at 8:02 AM, Christian Duerr<EMAIL_ADDRESS>wrote:
I'm not familiar with this, but every release is signed with PGP so that shouldn't be an issue.
Is this an apple-specific thing that might need to be specified in the app bundle?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

If you set yourself up as a nonprofit who distributes free software, Apple will allow you to apply for an exception according to this article. You have to apply for a DUNS number, but can do that if you're a sole-prop. Here's the article apple has on their requirements.
 Setting up a nonprofit and applying is also not an option, since that's way too much work.
Apple users will just have to deal with the restrictions Apple imposes on them.

I respect y'alls choice of not signing things, it does come across as a bit hostile to osx platform users. Anyhow, I wanted to leave a couple notes about the process as I understand it today. If y'all wanted, I'd be happy to try to PR them into your Makefile (or similar).
I think the apple developer program is $99 USD/year.
You can codesign outside of the Xcode environment. You'd want to do something like:
codesign --force \
             --entitlements entitlements \
             --options runtime \
             --timestamp \
             -s "$CODESIGN_IDENTITY" \
             -v \
            target/release/osx/Alacritty.app

You may also need to notarize it (where apple is going for code signing). That's a similar xcrun altool but it's primarily for distribution packages.
Anyhow, happy to write shell code to help. Though I'm not sure there's interest.

I think the apple developer program is $99 USD/year.

This is the main problem here, and as you can read in above comments. The problem is not 99$/year it's 99$ a year to apple and strange hassle. Thank you for your instructions though.

it does come across as a bit hostile to osx platform users

I think it's a bit hostile of Apple to request a significant amount of money for developers to write free software for their platform.

I think the apple developer program is $99 USD/year.

Which would leave Alacritty at $-99.00/year, assuming that all the contributor's free time is worth nothing (which is obviously not correct).

The very emotional and unprofessional official response to this issue has put me off this app altogether.

The very reactionary and unprofessional official response to this issue has put me off this app altogether.

Couldn't agree more.

If you've made the mistake to purchase a macOS device and are now locked into your system, that's on you. But please don't request that other people spend $100 a year because you've made the mistake of siding with a shitty company.
I have no interest in supporting Apple and if you're looking for a solution to this problem, you can always install another operating system that doesn't hold developers hostage like this.


Add a postprocessing which allows to compare an arbitrary number of scenarios

Mockup of the code: https://github.com/allfed/pytradeshifts/blob/main/src/postprocessing.py
Mockup of usage: https://github.com/allfed/pytradeshifts/blob/main/scripts/scenario_comparison.ipynb

Things that could be included in the postprocessing:

Calculating the most important country for every community (e.g. by their centrality)
Stability/robustness comparison between the different scenarios, e.g. how dependent is the network on certain nodes, how stable is it against removal of nodes?

Papers that describe methods we could use:

https://www.mdpi.com/2304-8158/12/2/271
https://ieeexplore.ieee.org/document/10144664


[featurerequest] Add tocLastLevel option

or tocMaxLevel
In order to be able to define a maximum title level to takes into account for Table of Content generation

PR welcome :)

OK, I'll do a PR when I'll have the time to (probably in a week or two)


Switch from lazy_static to once_cell

This will ultimately facilitate migrating to the std solution, once the MSRV is high enough.

FWIW this was previously reverted in https://github.com/console-rs/console/pull/137 over MSRV and https://github.com/console-rs/console/issues/134 contains discussion.

Thanks -- I took care of this in #227 and released the result as 0.15.9.


Tests for the PST parser

The problem the component solves
Once mailbag is minimally functional, it will need automated tests to help us maintain it over time. There should be a test for the PST parser.
Relevant part of mailbag spec?
n/a
Type of component

[ ] Core
[ ] Input
[ ] Attachments
[ ] Derivatives conversion
[ ] Reporting/Exporting
[ ] GUI
[x] Testing
[ ] Distribution

Expected contribution

[x] Pull Request
[ ] Comment with proposed solution

Major challenges or things to keep in mind

addressed with #67


Feat: Server cost model

To add price matching in subfile exchanges,

Start with adding a CLI config --price-per-byte to ServerArgs.
The price is not stored in a database or subfile-specific at the moment. There may be a case where the bytes in a subfile is more valuable than another, then we can consider a more complex management.

Add a /cost endpoint that returns the price per byte for client/third-party price matching

When receiving paid queries, parse for receipts and condition on the receipt value where $\text{price per byte} * \text{bytes range} \leq \text{receipt value}$

Add a admin/cost endpoint for adjusting price_per_byte on the fly, taking a method set_price and a param price_per_byte to replace the previous price.

Additionally consider if indexer_framework can take care of cost models

Done 1,2,4 in 650b7410a9f6037f039a71ab665bd708d9ccf02c
3 left for future when receipt is constructed


Add Puppeteer support

Add latest puppeteer support by adding the required system dependencies
Credit: Darker's BetterAIO; because I want to use the latest puppeteer version in DBH
Haven't tested because I'm too poor to afford a VPS myself and test it, but it should theoretically work

Code looks good, if I have time tomorrow I'll test it out on a panel.

Code looks good, if I have time tomorrow I'll test it out on a panel.

As referenced in Darker's BetterAIO, you need these options to launch puppeteer without errors:
 puppeteer: {
        headless: true,
        args: [
            "--disable-gpu",
            "--disable-dev-shm-usage",
            "--disable-setuid-sandbox",
            "--no-sandbox",
        ]
    }
    ```