Add lifecycle support

Imagine you need to do some preprocessing to your Swagger document prior to being processed into a SwaggerApi model.  You cannot do that right now and we should provide facilities for this.

After talking with some colleagues on this, we'll punt on this since we don't get anything out of this just yet.  We will reopen if necessary.


Update connectors to use the AMA agent and "Log Ingestion API"

Is your feature request related to a problem? Please describe.
OMSAgent → AMA
The old Log Analytics agent will stop working 31st of August 2024:

Deprecation notice on OMS agent: https://azure.microsoft.com/en-us/updates/were-retiring-the-log-analytics-agent-in-azure-monitor-on-31-august-2024/
Deprecation notice on data collector API: https://azure.microsoft.com/en-us/updates/azure-monitor-data-collection-api-retirement/
Doc on data collector API: https://learn.microsoft.com/en-us/azure/azure-monitor/logs/data-collector-api

Example of connectors that must be updated:

Cisco Meraki
VMWare vCenter

Example of already updated connectors:

Fortinet FortiGate Next-Generation Firewall connector for Microsoft Sentinel

HTTP Data Collector API → Log Ingestion API
The HTTP Data Collector API will stop working 14th of September 2026:

https://learn.microsoft.com/en-us/azure/azure-monitor/logs/data-collector-api
https://learn.microsoft.com/en-us/azure/azure-monitor/logs/custom-logs-migrate

Example of connectors that must be updated:

Cloudflare connector

Describe the solution you'd like

Update connectors to use AMA and "Log Ingestion API" long before August 2024 to give us time to migrate.
Create a roadmap to keep us updated on the progress for various affected connectors.

Describe alternatives you've considered
None.
Additional context
None.

Hi @o-l-a-v, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 06-03-2024. Thanks!

Hi @o-l-a-v,
We are checking with the respective team, we will provide you an update soon.
Thanks

Hi @o-l-a-v,
We got response from respective team. They will be working on these connectors soon, they don't provide any ETA.
If we got any update related to this, we will inform you. currently closing this issue.
Thanks

+1 for the meraki connector and also for the VMware one that also only support OMS at the moment

Any ETA yet? @v-rusraut

Hey, is there any way we can get this logs from data collector endpoints ?
MS link- https://learn.microsoft.com/en-us/azure/azure-monitor/logs/custom-logs-migrate

Stuck here too, the OMS agent goes EOL in 1 month and there is no solution that has been specified for bringing Cisco Meraki data into Sentinel via the AMA. I guess we'll have to go off the reservation and report back.

As an unsupported workaround, I have used logstash with the new DCR based connector to get syslogs into Sentinel. This works on both linux and windows.
https://learn.microsoft.com/en-us/azure/sentinel/connect-logstash-data-connection-rules

@JustinGrote yeah, we’re using Fluent Bit now with direct kind DCRs.
Any chance you’ve tackled ASA logs into CommonSecurityLog without OMS/MMA/OMS? I want to avoid recreating the wheel with whatever black box parsing MS is doing to get them into CEF

Not particularly, I'm mostly making things to match the ASIM schemas to make things easier on our hunters

I was informed today that a new collector just came out to support VMware, Meraki and many other log types, check the data connector named "Custom Logs AMA", I did not test it yet but it is looking to be the solution for AMA

@JustinGrote yeah, we’re using Fluent Bit now with direct kind DCRs.
Any chance you’ve tackled ASA logs into CommonSecurityLog without OMS/MMA/OMS? I want to avoid recreating the wheel with whatever black box parsing MS is doing to get them into CEF

Not particularly, I'm mostly making things to match the ASIM schemas to make things easier on our hunters

For ASA you can use the Common Event Format (CEF) via AMA connector as far as I know

I was informed today that a new connector just came out to support VMware, Meraki and many other log types, check the data connector named "Custom Logs AMA", I did not test it yet but it is looking to be the solution for AMA

I don't see the "Custom Logs AMA" connector you mentioned. Do you mean the "Common Event Format (CEF) via AMA" connector?

Maybe this one?

https://github.com/Azure/Azure-Sentinel/tree/master/Solutions%2FCustomLogsAma

I mean this: https://learn.microsoft.com/en-us/azure/sentinel/unified-connector-custom-device?tabs=rsyslog
I have it on my side on the sentinel content hub named "Custom Logs AMA"

Agreed, huge limitation of AMA, it requires Azure Arc to work. Lots of MSSPs have probe VMsfor syslog in other client tenants, and it's not supported to onboard an Azure VM to Arc in another tenant, making it a huge gap (apparently MS assumes noone runs things in Azure, funny enough)
For us, Logstash/Fluent Bit was the only solution to this (Fluent Bit being way less resource intensive)

Agreed, huge limitation of AMA, it requires Azure Arc to work. Lots of MSSPs have probe VMsfor syslog in other client tenants, and it's not supported to onboard an Azure VM to Arc in another tenant, making it a huge gap (apparently MS assumes noone runs things in Azure, funny enough)
For us, Logstash/Fluent Bit was the only solution to this (Fluent Bit being way less resource intensive). We originally tried "tunneling" client syslogs back to an azure collector on our side using dtlspipe, but the solution was too flaky and didn't allow for onsite log filtering to reduce traffic.

I m also working for a mssp but we are not getting the logs in our own sentinel. Each client has it own instance in their tenant and they grant us the access through an azure app.
For your case I would most likely use rsyslog to forward the logs to your own collector with ama. Rsyslog is pretty reliable, you just need a secure connection for the traffic.

" you just need a secure connection for the traffic."
Minor detail there, good luck setting up IPSec tunnels with every MSSP customer. That's why we use Fluent Bit, we just ingest directly into a DCR via HTTPS, no secure connection or overhead required (DCRs are free). You can even make DCRs direct now without a DCE, using kind: direct and the latest API

You can use secure syslog using tls

You can use secure syslog using tls
You're still saying a rsyslog vm at the customer site, then tls'd to some public ip of an azure vm on our side, just to then get it into the collector with AMA (which you would need 1 collector per customer and they couldn't run in azure because you couldn't tie it to the customer workspace)

vs.
a single windows/linux vm/container at the customer site that ingests directly to a DCR and can filter locally, minimizing bandwidth to both.
yeah I'll take the second options thanks.

But if you finally send the log to the customer workspace why not just use arc and ama in the customer collector hosted in their environment?
Anyway this is not the subject of the thread

I can understand, on our side we have nothing to hide on the collectors, they just have the ama agent with full standard configuration.


Azure Search (i.e. AI Search) doesn't support Private Endpoint Move

AI Search is added in list of resources supporting Private link move, however it doesn't support it.
{'message':'Resource move validation failed. Please see details. Diagnostic information: timestamp '20240515T141145Z', subscription id 'XXXX', tracking id 'XXXXX', request correlation id 'XXXXX'. (Code: ResourceMoveProviderValidationFailed) Moving Search Services to a different Resource Group/Subscription while Private Endpoints attached is not supported at this time. RequestId: XXXX (Code: BadRequest, Target: Microsoft.Search/searchServices)','code':'ResourceMoveProviderValidationFailed','name':'XXXXXX','status':409}

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

ID: d9962fb6-03f4-fd30-ab93-d2086713dc9b
Version Independent ID: 9106c368-72fe-d2d8-9ed5-ef7a46241c75
Content: Move Azure Networking resources to new subscription or resource group - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/move-limitations/networking-move-limitations.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @mumian
Microsoft Alias: tomfitz

@ambhatia
Thanks for your feedback! We will investigate and update as appropriate.

@eric-urban - can you take a look at this when you get a chance?

@HeidiSteen - Can you take a look at this?

@ambhatia, thanks for reporting the issue, we'll remove search from that list.


[Bug] Could not find a working python interpreter on Mac OS Ventura.

[REQUIRED] Please fill in the following fields:

Unity editor version: 2021 / 2022
Firebase Unity SDK version: 10
Source you installed the SDK: .unitypackage (.unitypackage or Unity Package Manager)
Problematic Firebase Component: All (Auth, Database, etc.)
Other Firebase Components in use: None (Auth, Database, etc.)
Additional SDKs you are using: None (Facebook, AdMob, etc.)
Platform you are using the Unity editor on: Mac (Mac, Windows, or Linux)
Platform you are targeting: Any (iOS, Android, and/or desktop)
Scripting Runtime: C# (Mono, and/or IL2CPP)
Pre-built SDK from the website or open-source from this repo: website

[REQUIRED] Please describe the issue here:
This error re-surfaced and is making Firebase unusable for development:
Could not find a working python interpreter. Please make sure one of the following is in your PATH: python3 python3.8 python3.7 python2.7 python2 python
Following this, and stdin error will appear in the console, preventing Unity Play mode from running.
Unable to use Firebase SDK in any Unity version on Mac Silicon 13.0.1.
Steps to reproduce:

Create Unity Project

Include google-services

Import a Firebase SDK unity package.

Hi @dev-deg
It seems like some versions of Mac OS, like Ventura, may not start with python installed.  Based on some documentation though, it should be when you set up Xcode command line tools, which you should be able to do by running xcode-select --install from the command line.
After python is set up, you can run which python3 or which python, which should give you the path to it.

Hi @a-maurice thank you for your kind reply.
The issue was when I updated to Mac OS Ventura, XCode CLI tools where somehow uninstalled. Running xcode-select --install fixed the issue.
I had python 3 installed and it was done so using homebrew as you can see from the output below:
which python3
/opt/homebrew/bin/python3
which python
python not found
Thank you very much for your help and support.
Kind regards,
Dave


Instructions

Hello,
Could you share the instructions for deploy Contrail and integrate with Kubernetes.
As per your examples, should we have  to install docker engine before our deploy
<IP_ADDRESS>:
ansible_ssh_pass: contrail123
<IP_ADDRESS>:
ansible_ssh_pass: contrail123
<IP_ADDRESS>:
ansible_ssh_pass: contrail

the procedure is currently being reworked. New extensive readme will be online Monday/Tuesday

I am eagerly looking your updates.

@coolpalani
https://github.com/Juniper/contrail-ansible-deployer/wiki/Contrail-microservice-installation-with-kubernetes

@coolpalani is this issue fixed? please close it or comment.

I haven't tested yet. I will updated if anything else require


how to get caller information on mac os x?

Hi
I'm trying to profile a program and ran a command like:
sudo cargo profile cpu per-fn <target> <binary>
Everything runs great and I get a table that looks like:
Tot Time | Own Time | Function name
...
...
It's a little difficult to use this table without having the caller information - is there a way to get that? i would be able to diagnose where the time is coming from then.  So, I want a table like:
Tot Time | Own Time | Function name | Caller name
and I want each pair of function name/caller name listed separately.

Currently, it's not supported.


Update store to set status on click to pending (in advanced view)

Checklist:

[X] Have you successfully deployed to an AWS account with your changes?
[X] Have you successfully tested with your changes locally?

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

Reviewed in office hours, 12/05/2023.


AI activity name cutoff

Not sure what went wrong but it seems that the generated name was to long and that some of it was cut off.
It seems to be cutoff on Strautomator side I am guessing, in Strava UI I could manually edit the activity and add more characters.

It is in Dutch 😅

Hi @Glennmen could you send me the activity ID so I can check on the logs?
I've added a few mitigations to retry the prompt with a higher token count if the result was "abruptly cut", but clearly that hasn't worked out in this case. I'll have to investigate it further.

@igoramadas Sure, I have however manually edited the activity and removed the cutoff part because it looked weird 😅
https://www.strava.com/activities/11510590765

https://github.com/strautomator/core/commit/373163dab6ea01264d0274d815488ee5aa988e3c

Awesome! I will close the issue, will be hard for me to validate if it is fixed anyways 😅  and I trust you.


Updating embedded objects

Hi and thanks for open sourcing a cool configuration management database solution!
I'm having problems with modifying embedded objects via YiDB API. Conceptually I have a Server object that contains references to Interface as embedded objects (one-to-many relationship). So that when the server is deleted, interfaces are also deleted automatically.
But when trying to modify interfaces by modifying the Server object, modifications just won't apply, no errors are produced.
So far I could only delete a new Server object with updated interfaces and delete the old one.
Is there any better way of doing this with YiDB API?
Thank you.

Hi,
Thank you for using YiDB.
For embedded objects, YiDB doesn't allow to modify Interfaces by modifying the server object.
If you need to update/delete Interface objects, you can use normal update/delete api. If you delete an Interface, their reference with Server will be deleted too.
If you want to create an interface object, you need to paas "path" attribute in create api. For instance, the "path" for create interface object will be like
Server!5365e8d34f5af8c5c355aeeb!Interface
Btw, for your use case, I think you can use Inner reference instead.
Here is the introduction of inner reference
An inner class could not be created standalone. Its life-cycle is managed by its host class.YiDB supports cascading deletion when deleting host class object. E.g. The inner class objects should be deleted recursively.
Please let me know if you have any questions.
Thanks & Regards
Al Zhang


Unable to launch Linux app that depends on FluentAvalonia NuGet

Describe the bug
So this is actually my bad, but as a result of #153 when a Linux app that depends on the FluentAvalonia NuGet is built and ran, it exits with the following exception:

Unhandled exception. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
---> System.IO.FileLoadException: Could not load file or assembly 'Avalonia.Controls.DataGrid, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=c8d484a7012f9a8b'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'Avalonia.Controls.DataGrid, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=c8d484a7012f9a8b'
at CompiledAvaloniaXaml.!AvaloniaResources.Populate:/Styling/BasicControls/DataGridStyles.axaml(IServiceProvider , Styles )
at CompiledAvaloniaXaml.!AvaloniaResources.Build:/Styling/BasicControls/DataGridStyles.axaml(IServiceProvider )
at CompiledAvaloniaXaml.!XamlLoader.TryLoad(String )

Oddly this doesn't happen for Windows builds, and it doesn't happen for Linux builds that depend on the FluentAvalonia project directly instead of the NuGet. I'm unsure about OSX as I do not have an OSX environment to test with.
Unless you have an idea of what the cause of this is, I think #153 should probably be reverted until a better solution is found, though this does kinda suck as it adds like 30MB to my app's output when Diagnostics is included. (This is assuming you can reproduce the behavior and there isn't just some other issue on my end causing this behavior)
Desktop/Platform (please complete the following information):

OS: Linux (via Windows WSL)
FluentAvalonia Version: 1.4.0
Avalonia Version 0.10.15

EDIT: It seems like just including a reference to Avalonia.Controls.DataGrid in the project that is referencing FluentAvalonia is enough to fix this, so maybe just noting this somewhere is enough and reverting the change isn't necessary, but of course you would expect simply referencing FluentAvalonia to also include everything that it requires.

Diagnostics package should only be referenced during Debug, this should not be a problem, I have an application that used fork this repository to perform this change and it worked fine for a long time, so I don't think we should restore #153.
Maybe if you add a reference to the package Avalonia.Controls.DataGrid in your project and try generating it again, or delete the bin and obj folders and run msbuild -t:restore again, you might be able to fix it.

Diagnostics package should only be referenced during Debug

Correct, but DataGrids is referenced in release as well.

Maybe if you add a reference to the package Avalonia.Controls.DataGrid in your project

As I said in the post above, that does solve the issue, but this should not he necessary as you would expect all needed dependencies to be pulled in merely by referencing the FluentAvalonia nuget, and this is the case prior to #153 (e.g. in 1.3.5)

I just tested this in a blank project with my nuget and couldn't repro. I wonder if maybe the Avalonia.Controls.DataGrid dll was still an older version (0.10.14) for some reason and caused a conflict when it tried to load? That's the only thing I can think of


Javascript space before paren

Description
The results of beautification are not what I expect. I'm using Atom Beautify together with https://github.com/ricardofbarros/linter-js-standard with the [JavaScript Semi-Standard Style (semistandard)](https://github.com/Flet/semistandard) style. I could not find any setting in these two to make it work as the linter expects a space in front of the parentheses.
Maybe an additional checkbox in the settings?
I'm using JS Beautify-Beautifier but i tried every other one as well, to be sure.
Input Before Beautification
This is what the code looked like before:
order  (starterIndex, mainIndex)
  {
    return [
      this.starterMenu[starterIndex], this.mainMenu[mainIndex]
    ];
  },

Expected Output
The beautified code should have looked like this:
  order (starterIndex, mainIndex) {
    return [
      this.starterMenu[starterIndex], this.mainMenu[mainIndex]
    ];
  },

Actual Output
The beautified code actually looked like this:
  order(starterIndex, mainIndex) {
    return [
      this.starterMenu[starterIndex], this.mainMenu[mainIndex]
    ];
  },

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[ ] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

I'm wondering if i'm missing something as nobody seems to have had this problem before....

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


Is there any guideline to move from 0.3.4 to 0.4?

Is there any guideline to move from 0.3.4 to 0.4?
Or any advice.
I mean, what breaks backwards compatibility?
Thanks a lot.

Have you seen the releases page? Think most notably if you were creating your own errors, they look a bit different now.


[Safari 8.0.3] The Swiss cities demo jumps when changing zooming level

http://bl.ocks.org/xEviL/4921fff1d70f5601d159
OS X 10.10.x, Safari 8.0.3
(seems to be okay in Firefox 34.0.5 and Chrome 40.0)
The current (0.8.x) Leaflet master has "linear scaling", which could be used by the D3 SVG overlay as well. It would animate the zoom transformation instead of the current glitchy behaviour.

Hi all,
just wanting you to know that I've crafted a generic solution to using Leaflet and SVG libraries (including D3) together. Does not have issues with the zooming level. https://github.com/akauppi/Leaflet/tree/activeOverlay
It has the Swiss cities code as a demo in debug/map/swiss-cities.html. Is that okay for you? I'll take it away if needed. (did not see an email address to send this to, so thought to place here).

@akauppi sorry for the late response, when I have the time I'll have a look. Would be great to port your changes to D3SvgOverlay or maybe extract your code into another plugin like L.SvgOverlay and then D3SvgOverlay can depend on it.

Hi, @xEviL
I don't know if this is the best place to discuss the ActiveOverlay thingy. Let's come up with some other media if we are to do something about it.
This PR in the Leaflet proper has most information about the patch. The author's opinion was that it could be done as a plugin. I have little motivation to start maintaining a plugin (I just want to use the thing :) ). So it's currently not heading anywhere. If someone wants to adapt it and start maintaining, I'm very happy about that.
As I see it - without remembering your D3 overlay very closely, I'm afraid - the ActiveOverlay patch should pretty much render SVG library-specific bindings unneeded. It lets Leaflet run in its world, and anything SVG run in its. It makes peace between these two domains so that e.g. zooming in Leaflet does not affect anything in the SVG domain.
I think this is how things should be. I also think Leaflet would indeed benefit from such functionality being built-in.
My own project is currently using Snap.svg but likely moving to svg.js later this year.
The best way to see what the patch is about is the above mentioned PR. Also - and this was one of the reasons I didn't want to start maintain it as a plugin - it would benefit from someone knowing the mindset of Leaflet better than I do. I can see the (existing) ImageOverlay and this should actually derive from a joint base class (but obviously this is beyond a plugin's architectural freedoms). I tried to follow the idea of the ImageOverlay very closely - it's just that the "picture" happens to be changing and interactive.
Any ideas - how would this benefit your D3 + Leaflet usage?

Seems that the best way to address the issue and also to simplify the plugin code is to use Leaflet-provided classes for animation.
For Leaflet 0.7.x the following example shows how to inject D3 generated elements into the Leaflet's SVG layer that handles animations and scaling on it's own:
http://bl.ocks.org/monfera/11100987
The only problem here is projection that rounds the pixel coordinates and this is why in this example you see "jumps" when zooming.
Basicly, calling map._initPathRoot() creates the svg layer in Leaflet and then you just use it to inject your elements.
For Leaflet 1.x there's a concept of a renderer: L.Renderer class, that abstracts zoom animation, scaling and shifting. There this should be used by L.D3SvgOverlay.


CH 9.1 missing ")" in the Hamburger

In Section 9.1 the Hamburger  element code block is missing a parenthesis

Fixed
https://github.com/EdCharbeneau/kendo-quickstart-mvc/commit/fa75f5eff29669c37c3c235ea7e3ee41fda9a5e2?short_path=9f248b5#diff-9f248b5b3d077c1f935fc7a42c962b04


[Storage[DataMovement] Record some tests that could not be previously recorded

This issue preventing these tests from being recorded was resolved a while back so these tests can now be recorded. This change simply removes the LiveOnly and adds the new recordings (approximately 450recordings).
TODO: There are still some more tests to record but they need to be converted to use test base classes first. Will do those in a follow-up PR.

API change check
API changes are not detected in this pull request.


get profiling back to perceived "magical"

Profiling has gotten much slower since May. I think there are some easy solutions to this.

it would be nice if we re-ordered the queue a bit. Right now, we enqueue a bunch of stuff that the user doesn’t need to see immediately (such as top ns). I’m noticing that profiling feels a lot. slower with the new numeric histogram rug plot. It would feel faster if that was the lowest-priority item in the priority queue.
So I’m thinking of the following priorities:

SummaryProfile – anything that is directly visible in the profiling, e.g. the initial histograms, null counts, cardinality
EssentialProfileDetails  – top Ns, timestamp viz
DeeperProfileDetails – things like the rug plot, which seem to take a bit longer to run.

Along with that, we could prefetch a detail if the user mouses over the profile. This would amount to re-prioritizing the details to be "UserRequestedProfile", which should be above SummaryProfile here. @domoritz suggests also taking a look at premonish, which uses some simple heuristics to determine if a user's mouse is moving toward a DOM element. This would buy us another 200ms to reorder the priority queue, which would likely make it so that the detail views are already calculated.

We're going to stick to keeping to (1) for this sprint.

Makes sense. Doing the magic prefetching only gives you marginal benefits and everything else needs to be right first.


Delete New Relic config and dependency

We no longer use New Relic, even in development mode, so the gem and its config can be removed.

Nobody's objected, so I'm going to merge this. I'll revert and update the docs if it turns out that people are using New Relic in dev.


subsys: modbus: Add modbus_iface_get_by_dev

Add modbus_iface_get_by_dev() to get the modbus interface from a
struct device.
This is to allow removal of DT_LABEL.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

@jfischer-no need to cleanup users of modbus_iface_get_by_name, but wanted to get your take on this before I progressed further.

@jfischer-no ping

@jfischer-no ping

pong https://github.com/zephyrproject-rtos/zephyr/pull/47866


customizing toolbar marker for multiple choice of marker

Hi there,
I want to create a toolbar with different options of markers (ex. A marker for restaurant, a gas stations, hospital).
Right now I only have available one marker button, how do I create one buttons and can select the multiple choice of marker such restaurant, a gas stations, hospital and etc
here is sample function that i want to create... any ideas how to create it?

I would also be keen to get an understanding of how this is done with the selecting markers.  I presume there is an event which listens for the the marker click and then it sets the marker to use?


matchLabels is insufficient for legacy uses. Can we support matchExpressions too?

This syntax requires me to label all resources that belong to an app with the same label (both key and value). This might be hard for external pieces, that I don't want can't change easily.
spec:
  selector:
    matchLabels:
     app.kubernetes.io/name: "wordpress-01"

Why can't we also support this:
spec:
  selector:
    matchExpressions:
      key: app
      operator: in
      values:  ["foo", "bar", "baz"]

The selector is a metav1.LabelSelector so it, in general, supports SetBased selection. The app label is a best practice decided on by the app definition wg.


The application does not minimize when you click on its icon in the taskbar using pyqt qml

Describe the bug
Hello! I'm having trouble figuring out why my application isn't minimizing when I use your library. I've noticed that when I combine your library with QQmlApplicationEngine, the application doesn't minimize when I click the icon in the taskbar. This only happens when I use this combination, though, as other PyQt classes work fine.
To Reproduce
Run this code and see if you can minimize the application by using the taskbar icon.
import sys
from PyQt5.QtCore import Qt
from PyQt5.QtGui import QGuiApplication
from PyQt5.QtQml import QQmlApplicationEngine
import PyTaskbar

if __name__ == '__main__':
    app = QGuiApplication(sys.argv)
    engine = QQmlApplicationEngine()
    engine.load('interface.qml')
    main_window = engine.rootObjects()[0]
    taskbar_progress = PyTaskbar.Progress(hwnd=int(main_window.winId()))
    taskbar_progress.init()
    if not engine.rootObjects():
        sys.exit(-1)
    sys.exit(app.exec_())

import QtQuick 2.15
import QtQuick.Controls 2.15
ApplicationWindow {
    visible: true
    width: 400
    height: 300
    title: "Hello World"
    Rectangle {
        anchors.fill: parent
        color: "#ffffff"
        Text {
            text: "Hello, World!"
            anchors.centerIn: parent
            font.pixelSize: 32
            color: "#000000"
        }
    }
}

Expected behavior
The app should minimize when the icon is clicked in the taskbar.

I have no idea how this works, but adding a 5-second delay seems to solve the problem. However, having to wait 5 seconds each time before launching the app is a bit frustrating, to be honest. So, this isn't really a solution to the issue.
import sys
from PyQt5.QtCore import Qt
from PyQt5.QtGui import QGuiApplication
from PyQt5.QtQml import QQmlApplicationEngine
import PyTaskbar
from time import sleep

if __name__ == '__main__':
    app = QGuiApplication(sys.argv)
    engine = QQmlApplicationEngine()
    engine.load('interface.qml')
    main_window = engine.rootObjects()[0]
    sleep(5)
    taskbar_progress = PyTaskbar.Progress(hwnd=int(main_window.winId()))
    taskbar_progress.init()
    if not engine.rootObjects():
        sys.exit(-1)
    sys.exit(app.exec_())


Do we need a style for stylepedia?

Is your feature request related to a problem? Please describe.
According to FAQs: CCS style guides and term conventions, "The Red Hat Technical Publications Writing Style Guide (stylepedia.net) is no longer being used in CCS. It is still in use outside CCS, but it is no longer official for our group."
HOWEVER, Ingrid Towey states that stylepedia is still used by some people upstream (we're not sure who) and a technical docs group under Sales.

Who are these users?
Do their tools and file formats support Vale style linter?
Do they want or need a Vale style?
Do we want this Vale style for Red Hat project to include non-CCS use cases?
What is the priority of such a project?

Describe the solution you'd like
Depending on our conversations and the answers to the preceding questions, it might make sense to include stylepedia within the scope of this project. If so, we would create a stylepedia subdirectory in the Red Hat style and begin creating rules from it.
Describe alternatives you've considered
If the tools and file format of stylepedia users do not support Vale, they might consider using other style-checker tools such as Acrolink, Grammarly, and so on.
Additional context
Add any other context or screenshots about the feature request here.

I believe it is safe to leave Stylepedia out from the RedHat  style guide that we implement here. I don't see the point in implementing rules that we have deprecated.

I agree with @themr0c. These styles are deprecated. It seems like a waste of time to implement them if they are being phased out.

Stylepedia may useful as a reference on some points, but that doesn't mean that its content should be implemented in Vale.

We have voted to close this issue because stylepedia is obsolete and not relevant to the target user group.


9anime download does this to my command prompt, help

When I try to watch or install any anime with 9anime provider, this happens

anime dl --provider 9anime 'inazuma-eleven'
anime: anime-downloader 3.6.3
anime: 'NoneType' object has no attribute 'find_all'

Help?

Those are escape codes. They turn into color instructuions on a Linux terminal - but display as complete garbage on Windows. Dont mind them - you may use an alternative terminal if they bug you though...
Now, this is not related to this program, though. :)


How do I sync this with my server?

Currently I have:
isTokenSet(state) { 		if (state.token) { 			return true; 		} else { return false}
But, on the server, it lists the token as false, which is causing rendering errors. How do I sync this with the server?

Yeah I'd like my server to also have access to the persisted store on server-side render (from nuxtServerInit for example).

I can't remember this exactly, but I think it was caused by the wrong Authentication. I was trying to save the token returned from the API to the browser's local storage as well as to Vuex Store. The same way I do it for regular VueJS.
The solution for Nuxt: I needed to use the following module: https://auth.nuxtjs.org/

If you access to persisted store by browser storage with server side methods, It logically means that client side rendering must be performed before the server method can be executed.
To make this possible, it means that encrypted data from the browser storage must be passed to the nuxt server before the nuxt page is rendered, and there is no way to handle it at the nuxt module level.
So if you really need it, you have to make the server page plugins for it.
Server side router -> serve not rendered page for take browser storage data to server side -> make persisted store with the data -> render nuxt page with the storage


Support noParent aliases with name

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Currently this case is not representable in the SDK, although the protobuf model for aliases allows it. It looks like a small omission from the SDK, which only allows Alias.noParent() or Alias.builder().build() that never sets noParent=true.
Alias(name="foo", noParent=true)

See also https://github.com/pulumi/pulumi/blob/master/proto/pulumi/alias.proto
Affected area/feature

CC @Zaid-Ajaj

@t0yv0 this can be implemented in the Alias.Builder class. However, now I am reading the code and something's not right:
public Builder parent(@Nullable Resource parent) {
    requireNullState(name, () -> "Alias should not specify Alias#parent when Alias#parentUrn is  already.");
    this.parent = parent;
    return this;
}

public Builder parentUrn(@Nullable Output<String> parentUrn) {
    requireNullState(name, () -> "Alias should not specify Alias#parentUrn when Alias#parent is  already.");
    this.parentUrn = parentUrn;
    return this;
}

Notice requireNullState calls checks the wrong field (only checking name is null) should probably be
public Builder parent(@Nullable Resource parent) {
-   requireNullState(name, () -> "Alias should not specify Alias#parent when Alias#parentUrn is  already.");
+   requireNullState(parentUrn, () -> "Alias should not specify parent when parentUrn is specified.");
    this.parent = parent;
    return this;
}

public Builder parentUrn(@Nullable Output<String> parentUrn) {
-   requireNullState(name, () -> "Alias should not specify Alias#parentUrn when Alias#parent is  already.");
+   requireNullState(parent, () -> "Alias should not specify parentUrn when parent is specified.");
    this.parentUrn = parentUrn;
    return this;
}


ux: add a new expandStorage handler

update the ux backend server to use new expandstorage handler
For end to end UI testing image used for backend server: http://quay.io/paarora/rook-ceph:v637
UI will send the form
type AttachStorageForm = {
  storageClassForOSDs: string
  enableEncryption: boolean
  storage: "1"
  replica: int
  count: int
  strogaeClusterName: string
  poolDetails: {
    volumeType: 'Filesystem' | 'Block'
    poolName: string
    dataProtectionPolicy: '2Way' | '3Way'
    enableCompression: boolean
    filesystemName: string
    failureDomain: string
  }
  storageClassDetails: {
    reclaimPolicy: 'Retain' | 'Delete'
    name: string
    volumeBindingMode: 'Immediate' | 'WaitForFirstConsumer'
    enableStorageClassEncryption : bool
    encryptionKMSID: string
  }
}

@rchikatw @travisn @malayparida2000 @SanjalKatiyar @rewantsoni ready for the approval?

LGTM.

cc @iamniting PTAL

It looks like this design is more specific to LSO, shouldnt we try to make the API or code more generic instead of targeting only LSO (ocs-operator/backend server)?

Thanks, Removed any specific instance of LSO

@Madhu-1 @iamniting @rewantsoni can we get a final approval,
We would  like to test the end-to-end changes with UI using the build,  https://github.com/red-hat-storage/odf-console/pull/1662,
Currently, we are just able to test it by manually editing csv's and images

Can you pls separate manual changes and generated changes?

Added a separate commit

Also, if the pool and storage class already exist, should we not send a response indicating that the request has already been fulfilled or something similar? So that the requester/UI already knows that things are in place already.

This endpoint is only called when there is new LSO storageclass is used, that means we always need to create the pools and storageclass
PLus UI currently checks the ok status with 200 status code

Can you pls separate manual changes and generated changes?

Added a separate commit

Also, if the pool and storage class already exist, should we not send a response indicating that the request has already been fulfilled or something similar? So that the requester/UI already knows that things are in place already.

This endpoint is only called when there is new LSO storageclass is used, that means we always need to create the pools and storageclass PLus UI currently checks the ok status with 200 status code

The names are passed by the UI. If the same names exist we should send succeed msg IMO. But let's hear from @SanjalKatiyar

The names are passed by the UI. If the same names exist we should send succeed msg IMO. But let's hear from @SanjalKatiyar

UI has validations in place so this should not happen for happy-path, but in case it happens IMO it's okay to show error.
K8s API server also returns error if we try to create an existing CR (same name/namespace). Also, it can happen that specs/parameters of already existing pool/class are different than what user wants now in new pool/class, so better to show error and let user use new name instead.

The names are passed by the UI. If the same names exist we should send succeed msg IMO. But let's hear from @SanjalKatiyar

UI has validations in place so this should not happen for happy-path, but in case it happens IMO it's okay to show error. K8s API server also returns error if we try to create an existing CR (same name/namespace). Also, it can happen that specs/parameters of already existing pool/class are different than what user wants now in new pool/class, so better to show error and let user use new name instead.

As UI is sending a unique name, I don't this so we need to re-validate,
It's a single endpoint call which already loaded with three/four API operation,
We should not overload it.
If the blockpool wasn't created anyway user will see the error in the UI.

/unhold


win: fix compilation on mingw

Libuv compilation fails under MinGW (gist). This PR adds missing define for UNLEN from iphlpapi.h and MSDN.

I thought that UNLEN came from lmcons.h, but other than that, LGTM if the CI is happy.

LGTM.  CI: https://ci.nodejs.org/job/libuv-test-commit/81/

LGTM if the CI is happy.

Cheers @bzoz, landed in fc8cc42.


issue81 upgrade smart contract to cosmwasm provwasm 1.0.0 beta

Upgrade to 1.0.0-beta of cosmwasm/provwasm

tested the endpoints against the old instance of localnet smart contract from before upgrading and everything seems to work fine


Fe homework 12

https://video.drift.com/v/abOzShPT1cR8UHdlo7aMmqo5sWKxyHWYMM49AfZOh731A/
Added Items

Searching

Removing Item

Component structure

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Print Issues on Firefox

Previewed printout for both Chrome and Firefox Browsers. Chrome prints out fine, however Firefox has breaks between headers, step and tip (see screen shots)

@fjumawan thanks for mentioning this! I'll look into it.

Related to issue #72

@parkjc can we meet on this when you have the chance?

@fjumawan sure thing. i can meet today before 1PM or after 3PM

Unable to fix Firefox since there is no universal CSS to have all 3 browsers to print perfectly. Added a notification to print through Chrome or Safari.

Do we know anything more about this @fjumawan @parkjc? Does Firefox ignore the print.css that was created, or do special rules needed to be applied to help it work?

@billymeinke i'll have to do a bit more research on how firefox print preview works, but it doesnt seem like its recognizing the custom css file. Either that or page breaks in firefox are interpreted differently

Checked in on this, @parkjc is working on it.
May have to do with "media=print/screen"


Encryption domain network support

Is your feature request related to a problem? Please describe.
I would like support for encryption domains networks
Describe the solution you'd like
It would be good to have encryption domains fields related to networks in the ipam database.
Describe alternatives you've considered
The fields should be multiple choice items (cause you can have multiple networks for each encryption domain).
Additional context

This is basically what i would like to have. A built-in relationship between tunnels and ipam prefixes instead of static custom fields.

I don’t know if this falls into the model or not.
How I envisioned it is you’d have an inner IP tied to an interface and that interface is tied to the tunnel on side A or side B. The inner IP belongs to a VRF and that VRF is what is allowed between the tunnels. If you need something more strict, the custom fields allows you to do that.
I am open to other comments.

Hi,
thanks for your reply.
Actually I would like to store information about the private networks advertised/routed across the tunnel. I also don't want the data to be free-text but I want it to be related to a ipam prefix item.
The same could be said about the tunnel interface IP (something I did not think about before your reply), it would be nice to map it to a prefix item as well.

Would a custom field of type "Multiple Objects" not do what you want?
You can choose prefixes as the object and then it would allow you to directly link them to the tunnel.

I am closing this. I agree with @p-rintz. The best thing to do is to create a custom field of type Multiple Objects for dcim.prefixes. You then can do a multi-select of your prefixes.


UX Study to examine the utility of selecting samples with a checkbox

Closing, lets do a more general review as part of #953


KOGITO-9109: Jobs service complementary integration tests for KOGITO-9007

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[X] You have read the contributors guide
[X] Your code is properly formatted according to this configuration
[X] Pull Request title is properly formatted: KOGITO-XYZ Subject
[X] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[X] Pull Request contains link to the JIRA issue
[X] Pull Request contains link to any dependent or related Pull Request
[X] Pull Request contains description of the issue
[X] Pull Request does not include fixes for issues other than the main ticket

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request checks
Please add comment: Jenkins retest this

for a specific pull request check
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] tests

for quarkus branch checks
Run checks against Quarkus current used branch
Please add comment: Jenkins run quarkus-branch

for a quarkus branch specific check
Run checks against Quarkus current used branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] quarkus-branch

for quarkus main checks
Run checks against Quarkus main branch
Please add comment: Jenkins run quarkus-main

for a specific quarkus main check
Run checks against Quarkus main branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] quarkus-main

for quarkus lts checks
Run checks against Quarkus lts branch
Please add comment: Jenkins run quarkus-lts

for a specific quarkus lts check
Run checks against Quarkus lts branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] quarkus-lts

for native checks
Run native checks
Please add comment: Jenkins run native

for a specific native check
Run native checks
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] native

for native lts checks
Run native checks against quarkus lts branch
Please add comment: Jenkins run native-lts

for a specific native lts check
Run native checks against quarkus lts branch
Please add comment: Jenkins (re)run [kogito-runtimes|kogito-apps|kogito-examples] native-lts

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

Quarkus-3 PR check is failing ... what to do ?

The Quarkus 3 check is applying patches from the `.ci/environments/quarkus-3/patches`.
The first patch, called 0001_before_sh.patch, is generated from Openrewrite .ci/environments/quarkus-3/quarkus3.yml recipe. The patch is created to speed up the check. But it may be that some changes in the PR broke this patch.
No panic, there is an easy way to regenerate it. You just need to comment on the PR:
jenkins rewrite quarkus-3

and it should, after some minutes (~20/30min) apply a commit on the PR with the patch regenerated.
Other patches were generated manually. If any of it fails, you will need to manually update it... and push your changes.

related to: https://github.com/kiegroup/kogito-apps/pull/1714

(tests) - kogito-quarkus-examples job #871 was: FAILURE
Possible explanation: Pipeline failure or project build failure

Reproducer
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-examples -u https://github.com/kiegroup/kogito-runtimes/pull/2975 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-quarkus-examples/871/display/redirect
See console log:

Console Logs
[Pipeline] withEnv[Pipeline] {[Pipeline] script[Pipeline] {[Pipeline] echo[2023-05-05T14:34:25.929Z] [INFO] Getting build-chain version from composite action file[Pipeline] readFile[Pipeline] echo[2023-05-05T14:34:26.004Z] [INFO] build-chain version recovered '^v3'[Pipeline] sh[2023-05-05T14:34:26.287Z] + npm install -g '@kie/build-chain-action@^v3' -registry=https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/[2023-05-05T14:35:47.825Z] npm ERR! code E503[2023-05-05T14:35:47.825Z] npm ERR! 503 Service Unavailable - GET https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/@kie%2fbuild-chain-action[2023-05-05T14:35:47.825Z] [2023-05-05T14:35:47.825Z] npm ERR! A complete log of this run can be found in:[2023-05-05T14:35:47.825Z] npm ERR!     /home/jenkins/.npm/_logs/2023-05-05T14_35_35_808Z-debug.log[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Build projects)Stage "Build projects" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Sonar analysis)Stage "Sonar analysis" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Upload artifacts to repository)Stage "Upload artifacts to repository" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (check space after build)Stage "check space after build" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh[2023-05-05T14:35:48.759Z] + wget --no-check-certificate -qO - https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-quarkus-examples/871/api/json[Pipeline] readJSON[Pipeline] sh

(tests) - kogito-apps job #2510 was: FAILURE
Possible explanation: Pipeline failure or project build failure

Reproducer
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-apps -u https://github.com/kiegroup/kogito-runtimes/pull/2975 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-apps/2510/display/redirect
See console log:

Console Logs
[Pipeline] withEnv[Pipeline] {[Pipeline] script[Pipeline] {[Pipeline] echo[2023-05-05T14:35:05.318Z] [INFO] Getting build-chain version from composite action file[Pipeline] readFile[Pipeline] echo[2023-05-05T14:35:05.387Z] [INFO] build-chain version recovered '^v3'[Pipeline] sh[2023-05-05T14:35:05.673Z] + npm install -g '@kie/build-chain-action@^v3' -registry=https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/[2023-05-05T14:36:27.641Z] npm ERR! code E503[2023-05-05T14:36:27.641Z] npm ERR! 503 Service Unavailable - GET https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/@kie%2fbuild-chain-action[2023-05-05T14:36:27.641Z] [2023-05-05T14:36:27.641Z] npm ERR! A complete log of this run can be found in:[2023-05-05T14:36:27.641Z] npm ERR!     /home/jenkins/.npm/_logs/2023-05-05T14_36_15_430Z-debug.log[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Build projects)Stage "Build projects" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Sonar analysis)Stage "Sonar analysis" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Upload artifacts to repository)Stage "Upload artifacts to repository" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (check space after build)Stage "check space after build" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh[2023-05-05T14:36:28.499Z] + wget --no-check-certificate -qO - https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-apps/2510/api/json[Pipeline] readJSON[Pipeline] sh

(tests) - kogito-springboot-examples job #886 was: FAILURE
Possible explanation: Pipeline failure or project build failure

Reproducer
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-examples -u https://github.com/kiegroup/kogito-runtimes/pull/2975 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-springboot-examples/886/display/redirect
See console log:

Console Logs
[Pipeline] withEnv[Pipeline] {[Pipeline] script[Pipeline] {[Pipeline] echo[2023-05-05T14:35:25.461Z] [INFO] Getting build-chain version from composite action file[Pipeline] readFile[Pipeline] echo[2023-05-05T14:35:25.530Z] [INFO] build-chain version recovered '^v3'[Pipeline] sh[2023-05-05T14:35:25.815Z] + npm install -g '@kie/build-chain-action@^v3' -registry=https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/[2023-05-05T14:36:47.170Z] npm ERR! code E503[2023-05-05T14:36:47.170Z] npm ERR! 503 Service Unavailable - GET https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/@kie%2fbuild-chain-action[2023-05-05T14:36:47.170Z] [2023-05-05T14:36:47.170Z] npm ERR! A complete log of this run can be found in:[2023-05-05T14:36:47.170Z] npm ERR!     /home/jenkins/.npm/_logs/2023-05-05T14_36_35_064Z-debug.log[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Build projects)Stage "Build projects" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Sonar analysis)Stage "Sonar analysis" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Upload artifacts to repository)Stage "Upload artifacts to repository" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (check space after build)Stage "check space after build" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh[2023-05-05T14:36:48.102Z] + wget --no-check-certificate -qO - https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-springboot-examples/886/api/json[Pipeline] readJSON[Pipeline] sh

(tests) - kogito-runtimes job #2447 was: FAILURE
Possible explanation: Pipeline failure or project build failure

Reproducer
export BUILD_MVN_OPTS_CURRENT=-T 1C -Dvalidate-formatting
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-runtimes -u https://github.com/kiegroup/kogito-runtimes/pull/2975 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/2447/display/redirect
See console log:

Console Logs
[Pipeline] withEnv[Pipeline] {[Pipeline] script[Pipeline] {[Pipeline] echo[2023-05-05T14:35:35.607Z] [INFO] Getting build-chain version from composite action file[Pipeline] readFile[Pipeline] echo[2023-05-05T14:35:35.678Z] [INFO] build-chain version recovered '^v3'[Pipeline] sh[2023-05-05T14:35:35.964Z] + npm install -g '@kie/build-chain-action@^v3' -registry=https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/[2023-05-05T14:36:57.318Z] npm ERR! code E503[2023-05-05T14:36:57.318Z] npm ERR! 503 Service Unavailable - GET https://repository.engineering.redhat.com/nexus/repository/registry.npmjs.org/@kie%2fbuild-chain-action[2023-05-05T14:36:57.318Z] [2023-05-05T14:36:57.318Z] npm ERR! A complete log of this run can be found in:[2023-05-05T14:36:57.318Z] npm ERR!     /home/jenkins/.npm/_logs/2023-05-05T14_36_45_279Z-debug.log[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Build projects)Stage "Build projects" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Sonar analysis)Stage "Sonar analysis" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Upload artifacts to repository)Stage "Upload artifacts to repository" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (check space after build)Stage "check space after build" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh[2023-05-05T14:36:58.172Z] + wget --no-check-certificate -qO - https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/2447/api/json[Pipeline] readJSON[Pipeline] sh

jenkins retest this

@tiagodolphine suggestions where applied


Meta: Improve the app's design

Right now it looks like a working demo. Lets improve a bit the design, add footer and other expected elements to give it a more realistic look.
This is a meta issue, smaller issues should be opened.

[ ] Footer (#42)

The design can be similar to the one you have created in your Simplify Bootstrap post


37 update client testing

Alrighty this was more painful than expected.
So we're using the Jest + Enzyme setup - right now I just added snapshot testing for our two components. That was simple. Once there is some more behavior and jawn added, I'll add render & behavior tests (following this nice article)
To improve our code coverage, I am ignoring the main index.js file, this can be changed in the future but it's literally just rendering the App to the root element so there's nothing to test (everything that gets rendered is a component that has its own). If we add actual logic or setup to it I'll add tests.
I also disabled our serviceWorker for now. In the future we can reenable it and test, but it's not necessary right now and after reading this thread it might give us more headaches than benefits for initial development.

Oh and this also resolves #35 although I saw some weird behavior in there where when a test wouldn't work the requestAnimationFrame warning would appear 🤔 the updated react scripts 1.0.15 had this as resolved in the changelog, so if it pops back up again we can try to track it down and reopen that issue.


Add more details to ModelAttributes

This change adds more information to ModelAttributes, so that metadata
providers can look at the attributes on the property and type separately
if so desired

/cc @dougbu

:watch:

@dougbu updated

:shipit:


construction wrech

http://em.geosci.xyz/content/geophysical_surveys/fundamentals/survey_design.html
The material is ok but this important topic is just beginning.  Put a construction wrench at the top but leave all the material.

Done!

closed with #233


measurement.yaml not created

I added this addon to a second HA instance, but the measurement.yaml file is not created. could that be because nothing is measured yet? The s0pcm reader hasn't received any pulses yet since installation of the addon.

Yes that makes sense, let's see what it does after some data is measured..

this morning the file was created.

Thanks for the feedback :-)


Could not find a declaration file for module 'valtio/utils'.

Using valtio/utils with typescript like
import { proxyWithComputed } from "valtio/utils"
will result in error
Could not find a declaration file for module 'valtio/utils'. '/Users/me/valtioProject/node_modules/valtio/utils.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/valtio` if it exists or add a new declaration (.d.ts) file containing `declare module 'valtio/utils';`ts(7016)

And @types/valtio doesn't exist.
Could you please add utils.d.ts? Thank!

Which version has this issue please?

the same with proxyWithHistory.
Version 1.1.0 @dai-shi

Does this fix with v1.0.6?
#184 is probably something different, but you can try the "Local install instructions" in https://ci.codesandbox.io/status/pmndrs/valtio/pr/184

Can anyone help?
@barelyhuman Do you have any insight?

@dai-shi  the utils.d.ts is never added to the dist folder that's the reason for the issue, it's for both the esm exports and the cjs exports, want me to raise a PR for the same?

Whoa! Didn't know that. How could that happen?
Hmm, v1.0.7 has utils.d.ts. So, it's the only issue in v1.1.0.

@dai-shi it's a small change, the utils.ts uses a type from vanilla.ts but vanilla.ts doesn't export it and so the file never gets generated due to that error.
the type is AsRef
you just need to export it from vanilla.ts and build, check if utils.d.ts is built, if not force import the type in utils.ts in the import type line.
the build should then work

So,
https://github.com/pmndrs/valtio/commit/40de10439ace8dff1a383a9b9453aa856bd9171a#diff-39b2554fd18da165b59a6351b1aafff3714e2a80c1435f2de9706355b4d32351R2
this caused the issue...


Direct DStream shall support eventhubs.filter.offset

eventhubs.filter.offset

2.0.x fixed by https://github.com/hdinsight/spark-eventhubs/pull/74

fixed by https://github.com/hdinsight/spark-eventhubs/pull/84


"1GB" becomes "1 GB"

I'm not sure about "an Installation Media". Maybe "Creating Installation Media" makes more sense there? It might be worth some discussion if we really want "Media" instead of "Disk"
In either case, I think the ID should be updated to match.

@danrabbit Oh woops, I'm really sorry the commit got attached to the "1GB" pull request. That was unintended.
I'm pretty sure you don't want "Disk", since a USB stick is not a disk. I agree "a Media" in English must sound wrong. How about "an Installation Medium"?

Well a "disk" with a "k" I feel like can be a USB or just any sort of storage media really. "Disc" with a "c" no. I'm not sure "an installation medium" is better than "an install disk" :p But maybe @cassidyjames has some thoughts on that.

@danrabbit I don't think so: https://en.wikipedia.org/wiki/Disk_storage
"[D]ata are recorded by various electronic, magnetic, optical, or mechanical changes to a surface layer of one or more rotating disks." That excludes USB flash drives.

This needs a merge or rebase from master, I can't merge it automatically.

@lewisgoddard Right, working on it.

@lewisgoddard Meh, I couldn't automatically rebase it from master either. I created another pull request in another branch: https://github.com/elementary/mvp/pull/180


Add unit tests for JSONRPC

Further to #43440, this PR contains several unit tests for the JSONRPC module.

Various invalid (i.e. not Dictionary or Array) derived classes of Variant produce a well-formatted error when given to JSONRPC::process_action
Likewise for JSONRPC::process_string
Using the given TestClassJSONRPC class, and its public method something (that appends ", please" to the input String):

A Dictionary works correctly
An Array has each of its inner Dictionary elements iterated over
Giving a bad method name produces a well-formatted error.

Similarly to #89021, DOCTEST_CONFIG_NO_EXCEPTIONS_BUT_WITH_ALL_ASSERTS was added to the CPPDEFINES of the JSONRPC module, if both tests are enabled and exceptions are disabled, so that the REQUIRE assert could be used.

Hi all 👋
Having just noticed that all of my test cases are running twice, does anyone know why this might be?
It could be that I'm including the new header file in too many places.
Will investigate later!

Thanks!


Feature: Pre Netherite Backtank

Content
Adds a new Backtank to the game. Possibly with explosive behavior.


Help: Add a chapter for the printf command

Please fill out this template
Add a chapter for the pstree command:
https://github.com/bobbyiliev/101-linux-commands-ebook/blob/main/ebook/en/content/097-the-printf-command.md
Additional context
Make sure to check out the Guidelines.md file on how to contribute!

It looks like that it has already been contributed, but feel free to make improvements to the existing one or choose another command from the list!


Implement CANivore-based sampling thread for odometry

Once we are using all Krakens for drive, we'll want to implement a new version of the signal sampling thread which uses time syncing.

Finished with PR #49


Exception thrown randomly when testing.

Im using xUnit and WebApplicationFactory to create a test server. I wrote some test. When I run all one or two randomly fails. Run again and they succeed.
Here is my exception:
Message:
System.InvalidOperationException : Collection was modified; enumeration operation may not execute.

  Stack Trace:
Enumerator.MoveNextRare()
Enumerable.Any[TSource](IEnumerable`1 source, Func`2 predicate)
EndpointDefinition.<PostProcessors>b__139_0(IGlobalPostProcessor p)
WhereArrayIterator`1.MoveNext()
List`1.InsertRange(Int32 index, IEnumerable`1 collection)
EndpointDefinition.PostProcessors(IGlobalPostProcessor[] postProcessors)
<>c__DisplayClass0_0.<<Main>$>b__7(EndpointDefinition ep) line 58
MainExtensions.MapFastEndpoints(IEndpointRouteBuilder app, Action`1 configAction)
MainExtensions.UseFastEndpoints(WebApplication app, Action`1 configAction)
Program.<Main>$(String[] args) line 51
Program.<Main>(String[] args)
InvokeStub_Program.<Main>(Object, Object, IntPtr*)
MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)
--- End of stack trace from previous location ---
DeferredHost.StartAsync(CancellationToken cancellationToken)
HostingAbstractionsHostBuilderExtensions.StartAsync(IHostBuilder hostBuilder, CancellationToken cancellationToken)
HostingAbstractionsHostBuilderExtensions.Start(IHostBuilder hostBuilder)
TraffiCaptureFactoryBase.CreateHost(IHostBuilder builder) line 47
WebApplicationFactory`1.ConfigureHostBuilder(IHostBuilder hostBuilder)
WebApplicationFactory`1.EnsureServer()
WebApplicationFactory`1.get_Services()

Any idea whats going wrong?

can you pls try v5.1.1-beta1
should be fixed if the assumed cause is correct ;-)

Updated to latest version. Still fails occasionally. Is is possible im doing something wrong? And if yes, do you have any idea what am i doing wrong?

huh... interesting...
i'm afraid the only way to get to the bottom of this is for you to reference the fastendpoints source code instead of the nuget package and run the tests in debug mode.
then it should break on the exact location where the exeption occurs.
i'm gonna need a file name and line number of the location where the problem occurs.
are you by any chance using rider+xunit combo?
people have reported similar issues with rider+xunit before.
there has been no incidents with visual studio so far.

Im using visual studio with xunit right know. But the problem has occured in rider too, if i remember correctly. Later im gonna pull de source, but right know i have a lot of think to do. The biggest problem is that its only happens 2-3 times in about 100. So im not sure how to reproduce it deterministicly.

@szekelymatyas can it be that you run tests in parallel? I also have sometimes an error when i run tests in parallel because the test server is being configured multiple times and (as far as i understood) FE uses many static things during registration..
i don't consider it as FE problem (i just need to correct my tests) but maybe it is also relevant to you..

i've tried to replicate it by creating 100 web application factories simultaneously where each factory also creates 100 http clients parallelly in the attached solution. works without any issues. pls give it a try and see if you guys can replicate any static vars being messed up due to parallelism. this leads me to believe the testing libraries may be doing something weird under the hood. need to know how to reliably replicate it to be able to fix it :-(
Parallel-Repro.zip

i've found the error from the pipeline (if you're curious):

System.InvalidOperationException : Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
Stack Trace:
at System.Collections.Generic.Dictionary2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)    at FastEndpoints.MainExtensions.MapFastEndpoints(IEndpointRouteBuilder app, Action1 configAction)
at FastEndpoints.MainExtensions.UseFastEndpoints(WebApplication app, Action`1 configAction)
at MYAPP.EndpointsConfiguration.UseEndpoints(WebApplication app)

I will try to dig into the problem if it happens next time by me 😄 It happened only 2 times from 130 runs..

@laya-alsh that stack trace is actually not that helpful as it doesn't offer any clues as to which dictionary got corrupted. the MapFastEndpoints() method only has 1 dictionary and it's already a concurrent dictionary. https://github.com/FastEndpoints/Library/blob/fc55e7f870daa3a3716549cba80f9224c3fe8ac6/Src/Library/Extensions/MainExtensions.cs#L75
it must be something deep within that's throwing. no idea how to debug this.

i've tried to replicate it by creating 100 web application factories simultaneously where each factory also creates 100 http clients parallelly in the attached solution. works without any issues. pls give it a try and see if you guys can replicate any static vars being messed up due to parallelism. this leads me to believe the testing libraries may be doing something weird under the hood. need to know how to reliably replicate it to be able to fix it :-( Parallel-Repro.zip

I added some request to the sample. I got an exception every second try. Here is my example:
Parallel-Repro.zip
The exception:
Message:
System.InvalidOperationException : Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.

  Stack Trace:
Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
MainExtensions.MapFastEndpoints(IEndpointRouteBuilder app, Action`1 configAction)
MainExtensions.UseFastEndpoints(WebApplication app, Action`1 configAction)
Program.Main() line 16
--- End of stack trace from previous location ---
DeferredHost.StartAsync(CancellationToken cancellationToken)
HostingAbstractionsHostExtensions.Start(IHost host)
WebApplicationFactory`1.CreateHost(IHostBuilder builder)
WebApplicationFactory`1.ConfigureHostBuilder(IHostBuilder hostBuilder)
WebApplicationFactory`1.EnsureServer()
<4 more frames...>
<<Test1>b__1>d.MoveNext() line 21
--- End of stack trace from previous location ---
<<ForEachAsync>b__50_0>d.MoveNext()
--- End of stack trace from previous location ---
<<Test1>b__0_0>d.MoveNext() line 19
--- End of stack trace from previous location ---
<<ForEachAsync>b__50_0>d.MoveNext()
--- End of stack trace from previous location ---
Test.Test1() line 16
--- End of stack trace from previous location ---

@szekelymatyas that was actually helpful to find the culprit. thank you very much!
fixed in v5.2.0-beta10


Moved non-psr7 functions to a separate package

This will fix #44. See https://github.com/Nyholm/psr7-server
This builds on top of #54 (my bad). I will rebase when #54 is merged.

(doesn't it feel great to remove code 😄 )

I originally put server request separate because of all this code. But since we are removing this code now, I think the next pr should merge the serverrequest to psr17factory

Change my mind. You are very correct. It is easier to do it now.
Feel free to merge if you like it.

(I just rebased this PR.)

@zegnat, could you review please?

Since HTTPlug supports PHP5 we cannot use the same class for HTTPlug's UriFactory and PSR17's UriFactory.

Aah, I was looking at the use statements up top and missed the namespace on the implements! In that case: LGTM :+1:

Thank you for the review


SciMedia dual camera importing

The file format from SciMedia for MiCam 05 / Ultima supports dual camera recording, see Rhythm-3.0.
I don't have any files to test if optimap's recording importer works for these recordings. I'm unsure how we should handle this case for the load_video() function, give back a tuple with two numpy recordings?

Tested with MiCam Ultima dual imaging file, importer seems to work correctly


Update boox packages paths to follow composer 2.0 requirements

https://jira.24i.com/browse/BPLAT-14263

https://github.com/booxmedialtd/json-api/releases/tag/v1.2.2


cleaner SSL interface

instead of forcing the user to construct the SSL cert, take the most
common options and create an SSLContext from them.
This sidesteps a lot of potential security worries, and means our users don't need to read the docs for SSLContext.

:heart:


Proposal: Allow hosting of UWP apps inside UWP apps

Proposal: Allow hosting of UWP apps inside UWP apps

Summary
It would be awesome to be able to host a UWP app insider of another one.
This would be similar to what both My People and Xbox game bar do.
Rationale
I am one of the developers of Quarrel which is an unofficial UWP discord client.
The developer of MyTube (an unofficial UWP YouTube client) kindly provided us with a dll that contains large parts of the MyTube code that allows us to host an instance of MyTube inside Quarrel to provide youtube embeds in a way that does not require us to pull in a webview.
However, it would be much better if we could host the store version of MyTube inside Quarrel similarly to how Xbox game bar can host UWP apps.
I imagine there are many other similar cases in which other developers would find this useful.

Scope

Capability
Priority

This proposal will allow UWP apps to host other UWP apps
Must

This proposal will allow win32 apps to host UWP apps
Could

Open Questions

As this is not related directly to WinUI I'm not certain if this is the correct place to ask?
Could this have broader security implications?
Should any UWP app be able to host any other or should it require the hosted UWP app support it?

You can already host another app using Launcher.LaunchForResults.

This is super interesting, thank you!
At least in theory there's support for this through a combination of App Extensions and App Services that would let you put an iframe-like thing in your app from another app.

The hosting app declares a app extension where the "public" folder is expected to contain some HTML+JScript.  It finds extension implementations and loads their HTML into a WebView.
The hosted app that included the HTML then can use App Services to call over from the hosting app to the hosted app's background task.

There are security concerns for the hosting app, as they're effectively running code (JavaScript) from another app under their own identity. There's ways to control access from the JavaScript runtime to the host, and the WebView can use the "out of process" model to further isolate it.  But it is possible.  And It's nowhere near the model proposed on the thread so far, but it's something that works today.  We understand the challenges associated with (a) writing part of your app in HTML and another in XAML (b) using AppServices efficiently as part of an interactive application.
If we do something more here, we'd follow the same model that app extensions use - the hosted app would say "this is the part of me that I'm OK to have hosted elsewhere" and use capabilities to verify that the hosting app is who they want to be.  We'd also want app A's code to run inside app A's process and app B's code to run in app B's, while getting an input/output pipe between them.
A core issue for all apps is lifecycle - if A hosts B hosts C then the system needs to keep all of them alive.  These would also necessarily be optional and somewhat ephemeral - the user can choose to uninstall B or A at any time, and the hosting app needs to be OK with a "this child thing disappeared" message.
So yes - I've got a lot of ideas here, but we're looking to the community to know whether this is something we should spend additional resources on, either at the platform layer or in Project Reunion.  For now, please pile onto this thread:

Suggested API shapes
Scenarios you'd like to enable (either with multiple of your own apps, or apps to whom you're related)
Non-scenarios
Security, lifecycle, and identity concerns

This idea has great potential, may be in future the whole windows shell could be a UWP app and other developers can just make different shells as UWP app just like how Android handles Launchers.

Seems like the Calender is able to show content from the People app when you click on a specific event member even though the People app is in a separate app package.

Seems like the Calender is able to show content from the People app when you click on a specific event member even though the People app is now in a separate app package.

Mail, Calendar is a combined app.  And I think the People app provides a "picker" interface.  So you can use one app to load content from another UWP app - if that app provides the support.

@mdtauk I know Mail and Calender are in the same package but how UI from People from Calender? I only know about launching for results which would show a modal window but this seems something only Microsoft can do

@mdtauk I know Mail and Calender are in the same package but how UI from People from Calender? I only know about launching for results which would show a modal window but this seems something only Microsoft can do

I imagine its through this API
Windows.Storage.Pickers.Provider
https://docs.microsoft.com/en-us/uwp/api/windows.storage.pickers.provider?view=winrt-19041

No those APIs for adding yourself to the File/Folder Picker. I think this is some undocumented stuff that only Microsoft knows

Subset of the feature already in Game Bar Widget Overview, The Xbox Game Bar Widget can display XAML view for "GameBar" project through UI Extension host. Anyone know that UI Extension host available in SDK to let us mirrow two UWP Apps?
Xbox Game Bar Design

I did something horrifying in https://github.com/microsoft/terminal/compare/main...dev/migrie/fhl/dyndep that let an "extension host" app package load WinRT runtimeclasses from another package. This just used the public SDK, and dynamic dependencies. It's all in-proc though, so reliability is suspect at best.

I didn't get it loading a xbf from the extension package (but XAML generated in code-behind worked fine), nor did I try C#.
I'm pretty sure @jonwis has explicitly warned me not to try this in the past (again, reliability issues), but I had to see if it was possible 😄

I'm pretty sure @jonwis has explicitly warned me not to try this in the past (again, reliability issues), but I had to see if it was possible 😄

Ow, my eyes.
The main problem here is that UWP code is not usually designed to be loaded into other apps' appcontainers. That is, when you use dynamic dependency to bring in otherapp.main.dll code in that binary is now running inside the yourapp_8wkey... package family appcontainer and not the otherpublisher.otherapp_2k3fm... container. If the code in the DLL is a "pure function" (meaning it doesn't read-write any state and only uses its parameters) then sure, that probably works. If it produces any state, that state goes into your yourapp_8wkey... container.
Which means it's also something of a security risk - if your app manages credentials, and you load a DLL from another app, that other app now has access to your app's credentials.  Normally this would be blocked by UWPs and appcontainers, as they can't reach-across container boundaries directly.  This somewhat breaks the user's mental notion of "store apps" and UWPs - apps are isolated from each other and the system, and they have to directly approve when two apps want to work together.
So if you are doing this in terminal, @zadjii-msft , please make sure there's a user-consent prompt.
I won't way "dyndep allows app 1 to load code from app 2 is a bug" as it's something app1 decided to do.  But it's a sharp tool to be used carefully.
In another world entirely we'd fully support cross-app-package composition islands, so you host the visuals of another app without loading its code into your process.  As other have noted above, this is how the Xbox GameBar as well as various Picker types work.  We just never enabled it for UWPs.


Anisotropy Plots Are Missing the Header and Footer

What command was run:
https://earthref.org/MagIC/16436 and clicked on plots thumbnail
What happened:
The anisotropy plots (e.g. the first two plots) are missing the header and footer:

What I expected to happen:
To see the header and footer like the equal area plots:

PmagPy version:
Latest on PmagPy Server (plots made on 2/13/2019)

So do histogram plots (e.g. https://earthref.org/MagIC/16433):

Working on this.  Small data integrity note:  the contributions table lacks the id column in this download.
Here's a first pass at some fixes.  There are some inconsistencies with how the program is set up to format these -- i.e., the tcdf plot doesn't have a secondary label, and the other two have 'bootstrap' and McMurdo respectively.

How do we want to actually title these plots?  What else should be changed/added?

https://earthref.org/MagIC/16433 is also missing contribution.id.  Additional weirdness -- the names are not consistent between table levels, so there is not a full hierarchy from measurements -> locations.
Also, I noticed on MagIC there are only 3 plots for this contribution, but when I run it locally, I get additional hundreds of plots.

@rminnett The lack of contribution IDs is a problem for magic_geomagia.py also. I have added the issue to https://github.com/earthref/MagIC/issues/374 but maybe should have its own?

First attempt with histplot, MagIC borders added:

How should this plot be titled?  Also, is it a problem that it is a different size/shape from most of the other plots?

Maybe "Intensity Histogram" might be a bit more clear (although "Intensity" is in the x axis label)? It would be nice if the area under the histograms was colored in (light grey?) to make it more clear where most of the data are. I'll let @njarboe and @ltauxe decide about plot titles for the other types above.

@rminnett The lack of contribution IDs is a problem for magic_geomagia.py also. I have added the issue to earthref/MagIC#374 but maybe should have its own?

We have an old issue for this (https://github.com/earthref/MagIC/issues/291), so I elevated it back to P1 and I'll check for and insert the contribution.id field and upload to S3

@moonshoes87 the aspect ratio isn't critical, although as close to square as possible is better, as long as the axes bounding box is consistent relative to the final figure for each plot type to make creating nice thumbnails possible (#518).

@rminnett Are you going to make changes to the contribution files and keep the same magic id. It is pretty nice that all files named magic_contribution_16544.txt are always the same and one does not need to consider if one is working with the most updated version of a data file. I have data files in many areas and having to download a new copy from the website each time to make sure it is not an old one would be a pain. Maybe this one time is OK, but I would prefer a different route.
Maybe magic_contribution_16544-2.txt for the second version of a contribution although of course we already have a versioning system in place.

@moonshoes87 @Lisa
I think it would also be nice to have an easy way to know how many for something is in each bin. I like having the frequency axis but maybe you could put a number axis on the other side. Lots of plots are like this and would be pretty neat.

This might not be the best contribution to use to comment on other plot features as the id and hierarchy are missing. A (sub)title with the location:site:sample/specimen displayed is needed but maybe that already happens when the data is not corrupt.

@rminnett, that's good to know.  I am hesitant to mess with the aspect ratios as @ltauxe set them up, however it is very easy to change them if we agree that the visual representation of the data isn't undermined.

@moonshoes87 @lisa
I think it would also be nice to have an easy way to know how many for something is in each bin. I like having the frequency axis but maybe you could put a number axis on the other side. Lots of plots are like this and would be pretty neat.

I'll try adding this as well.

i can re-do the histogram plots with square plots, and even a kernel density curve (which is nicer than a histogram anyway).

On Feb 13, 2019, at 12:32 PM, Rupert<EMAIL_ADDRESS>wrote:
Maybe "Intensity Histogram" might be a bit more clear (although "Intensity" is in the x axis label)? It would be nice if the area under the histograms was colored in (light grey?) to make it more clear where most of the data are. I'll let @njarboe https://github.com/njarboe and @ltauxe https://github.com/ltauxe decide about plot titles for the other types above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/514#issuecomment-463357147, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDV7xyUubxW-4rIEJDkjyQLeVmnn0ks5vNHZggaJpZM4a59B8.

@njarboe, adding a second y axis is turning out to be a bit tricky.  I will return to this as it is lower priority than some of our other issues.

@moonshoes87 Ok. Save for later.

I think the anisotropy plots are completely meaningless for this study as they are all unoriented and the data were for correcting the paleointensity measurements.  is there some way to suppress making silly plots?  maybe all equal area projections should be in either geographic or stratigraphic coordinates  for a start?   - this goes for equal area projections  or directions for nything other than the specimen level for unoriented specimens (like many paleointensity studies)

regarding the histogram plot, making a second y-axis on the right with the numbers is pretty easy (twinx command).  I'll give it a go

I tried twinx, but couldn’t get the units right.  Good luck!

On Feb 13, 2019, at 3:25 PM, Lisa Tauxe<EMAIL_ADDRESS>wrote:
regarding the histogram plot, making a second y-axis on the right with the numbers is pretty easy (twinx command). I'll give it a go
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/514#issuecomment-463417540, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIkcTI8PrXQBeEADOQzzWgUD7PO6Mks5vNJ7MgaJpZM4a59B8.

It is plt.twiny().  One could be normalized (density set to True) and one not normalized.
but histplot.py already has the option to plot things that are not normalized (the -N option).  Should I take out the option and just do it both ways for everybody?

it’s twiny sorry I can have a look, but we already have a -N option for un-normalized plots.  should i change it so that it plots both ways always?
lisa

On Feb 13, 2019, at 3:26 PM, Lori J<EMAIL_ADDRESS>wrote:
I tried twinx, but couldn’t get the units right. Good luck!

On Feb 13, 2019, at 3:25 PM, Lisa Tauxe<EMAIL_ADDRESS>wrote:
regarding the histogram plot, making a second y-axis on the right with the numbers is pretty easy (twinx command). I'll give it a go
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/514#issuecomment-463417540, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIkcTI8PrXQBeEADOQzzWgUD7PO6Mks5vNJ7MgaJpZM4a59B8.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/514#issuecomment-463417892, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDWKzrJGnaivs9V_9CsNvBoDTDJ28ks5vNJ8igaJpZM4a59B8.

actually - it WAS twinx (not sure why), but anyway, i added an option to plot both to histplot.py (-twin) and changed make_magic_plots.py to do it that way.


[FEATURE] adding support for aliyun Maxcompute(aka. ODPS) catalog

Describe the feature
Add support for MaxCompute catalogs
Motivation
Since Aliyun(aka. Alicloud) is very popular in China, Maxcompute, which is a in-house cloud datawarehouse product of Aliyun is also being used by a lot of companies in China. I think it's Graviton should support Maxcompute catalogs
For more info, see its official website:https://www.alibabacloud.com/help/en/maxcompute/product-overview/what-is-maxcompute
Describe the solution
Like catalog-hive,  develop a catalog plugin for Maxcompute using the official SDK
Additional context
No response

Hi @haitaoyao will you work on this, or just a wish for the community?

@jerryshao I want to work on this if the community has the plan to support MaxCompute.

According to the Official Doc of Maxcompute by Alicloud, we connect to MaxCompute  with your username & password(NOT AK/SK) in PostgreSQL connector

I think it make sense to add the support to MaxCompute given a large user community there.
@haitaoyao you want to get this issue assigned to you?

I submitted a basic module. Can it be merged?#5048


Error when rendering an AngularJS page

Hello,
we have a strange issue with prerenderer for different pages we get every 3rd time following error:
PMphantom stdout: Error: [$rootScope:infdig] http://errors.angularjs.org/1.3.15/$rootScope/infdig?p0=10&p1=%5B%5D

Does anybody know how to solve the issue?

This can happen in PhantomJS 1.9.8 when there are spaces in the URL on an AngularJS page. There isn't a good way to solve this issue besides upgrading PhantomJS to 2.x (which there are currently issues with unless you use a slightly older version of PhantomJS 2.0)

Does prerender work with PhantomJS 2.0?

Yes. When rendering that directly against our service, I'm seeing the correct HTML returned.