The script for returning the ship to orbit does not work

When I disabled this mod, everything worked as expected.

018dd089-1093-390a-350d-a5792fb89f02
I checked it alone and with the guys. On our build, an error is written in the console, something did not pass the test and the objection does not work.
For the error to work, in this build, turn on Late_Game and try to fly solo to any planet - die and monitor the behavior of the autopilot

Resolved in 3.2.2


Feature Layer - Expose URLs and service info

Looking at using the Gallery for the service catalog, but a Feature Layers url is not visible

closing - app will be moved to mature support at 6.1. Use Minimal Gallery app instead.


Configure Application Preferences via Web Interface

A feature to (optional) configure the wiki via a web interface is on tis way:

yesss amazing!

Done in 525c26d174233d98873430450fce07e21cb5c7a1


Microsoft Entra Authentication integration with azure-monitor-opentelemetry to send logs/traces does not work.

Package Name:  azure-monitor-opentelemetry
Package Version: 1.6.2
Operating System: Linux
Python Version: 3.11

Describe the bug
Unable to authenticate post requests to App-insights using user assigned Managed Identity for my app-service deployed in Azure. Managed Identity has been assigned the role of Monitoring Metrics Publisher
Further, I checked the documentation entra authentication in case of App-Insights (https://learn.microsoft.com/en-us/azure/azure-monitor/app/azure-ad-authentication?tabs=python#unsupported-scenarios), there are some Contradicting statements here:

It says, ingestion for AutoInstrumentation for Python on Azure App Service is unsupported.
However, here it says it is supported along with the configuration :Enable Microsoft Entra ID (formerly Azure AD) authentication.

To Reproduce
Steps to reproduce the behavior:

Create an app-service in Azure.
Create a user-assigned managed identity for the above app-service.
Enable app-insights for the app-service. Disable localAuth to stop unauthorized telemetry Ingestion
Configure Azure monitor for the Webapp.
Pass the managed identity along with the client ID for Managed Identity, when initializing Azure Monitor.
All the post requests to Application Insights instance fail with 401 error (unauthorized). Here is an example log from the Web-app:

DEBUG    urllib3.connectionpool
[https://eastus2-0.in.applicationinsights.azure.com:443](https://eastus2-0.in.applicationinsights.azure.com/)
"POST //v2.1/track HTTP/1.1" 401 None

Expected behavior
Telemetry with Managed Identity should be authorized and sent across to Application Insights.
Screenshots
Web app with Managed identity configured :

Role assigned to Managed Identity :

Logs:

Azure monitor config

def configure_basic_azure_monitor():

    conn_string = settings.APPLICATIONINSIGHTS_CONNECTION_STRING

    docgen_logger = logging.getLogger("test")

    managed_credential = ManagedIdentityCredential(client_id=settings.AZURE_CLIENT_ID)

    configure_azure_monitor(
        connection_string= conn_string,
        logger_name=docgen_logger.name,
        credentials=managed_credential
    )

Additional context
Webapp : Fast API with uvicorn

Thank you for the feedback @summahto . We will investigate and get back to you asap.

Hi @jeremydvoss ,
Any Update on this ?

@summahto
Apologies for the late reply.

Further, I checked the documentation entra authentication in case of App-Insights (https://learn.microsoft.com/en-us/azure/azure-monitor/app/azure-ad-authentication?tabs=python#unsupported-scenarios), there are some Contradicting statements here:

It says, ingestion for AutoInstrumentation for Python on Azure App Service is unsupported. However, here it says it is supported along with the configuration :Enable Microsoft Entra ID (formerly Azure AD) authentication.

These are not conflicting statements. The first link is for AUTOINSTRUMENTATION via Azure App service attach (which is currently unsupported and is planning to be released this year). The second is MANUAL instrumentation (the one you are interested in) via the azure monitor distro.
With that being said, we need to clear up a few things regarding your setup/configuration. If you deploy a Python application into AppService, and you choose to ENABLE appinsights through the portal (upon creation), this is using AUTOINSTRUMENTATION. This means that you DO NOT have to change your actual Python application to use the Azure monitor sdk (configure_azure_monitor() call) and your telemetry should flow automatically. It looks like from your setup that you are using BOTH, so it is difficult to know which telemetry is generating those 401 errors. Please follow the below guidance:
If you want to keep using AppService with entraID auth, you can either:

Wait until this feature is supported via AUTO instrumentation in a couple of months. When it is supported (via the checkbox on the UX create page), you can remove the call to configure_azure_monitor in your code and your telemetry should flow automatically.
REMOVE telemetry collection via AUTO instrumentation by setting ApplicationInsightsAgent_EXTENSION_VERSION=disabled in your environment variable and KEEP your code as is with configure_azure_monitor() call. This is effectively now using MANUAL instrumentation.

Hi @lzchen ,
Thank you so much for clarifying this. It makes much more sense now. Let me try the 2nd solution you mentioned and come back with my results.

@lzchen ,
Yes, we had both AUTOINSTRUMENTATION and Manual instrumentation enabled for our application.
I tried the 2nd solution which you provided i.e. disabling auto-instrumentation using the setting ApplicationInsightsAgent_EXTENSION_VERSION=disabled. However, even now the telemetry does not go through with Entra Authentication, that is the original problem remains.
Here is the stacktrace after enabling Debug logs for the web-app:

If you notice App-insights returns 401 unauthorized whereas request to livediagnostics https://eastus2.livediagnostics.monitor.azure.com/ got through.
All the other configurations are same as above.

@summahto
A couple of questions:

Did you turn on live metrics feature via the sdk?

configure_azure_monitor(enable_live_metrics=True)

Can you pip freeze your dependencies and paste that here?

@lzchen ,
Here are the answers :

Yes, I have enabled live metrics feature via the sdk.
Following are the dependencies :

azure-core==1.32.0
azure-core-tracing-opentelemetry==1.0.0b11
azure-cosmos==4.7.0
azure-identity==1.17.1
azure-monitor-opentelemetry==1.6.2
azure-monitor-opentelemetry-exporter==1.0.0b31
azure-purview-catalog==1.0.0b4
azure-storage-blob==12.23.1
opentelemetry-api==1.27.0
opentelemetry-distro==0.48b0
opentelemetry-instrumentation==0.48b0
opentelemetry-instrumentation-aiohttp-client==0.48b0
opentelemetry-instrumentation-asgi==0.48b0
opentelemetry-instrumentation-asyncio==0.48b0
opentelemetry-instrumentation-dbapi==0.48b0
opentelemetry-instrumentation-django==0.48b0
opentelemetry-instrumentation-fastapi==0.48b0
opentelemetry-instrumentation-flask==0.48b0
opentelemetry-instrumentation-httpx==0.48b0
opentelemetry-instrumentation-logging==0.48b0
opentelemetry-instrumentation-psycopg2==0.48b0
opentelemetry-instrumentation-requests==0.48b0
opentelemetry-instrumentation-urllib==0.48b0
opentelemetry-instrumentation-urllib3==0.48b0
opentelemetry-instrumentation-wsgi==0.48b0
opentelemetry-resource-detector-azure==0.1.5
opentelemetry-sdk==1.27.0
opentelemetry-semantic-conventions==0.48b0
opentelemetry-util-http==0.48b0

Yes, I am looking at the traces table in Application Insights resource :

The opened log does not have the sdkVersion field in it. Here is the screenshot.

@summahto
It does appear that these logs are being generated by an SDK in AppService and not from the Azure monitor distro (I see the type is AppServiceConsoleLogs). The distro is not responsible for sending those logs.
Try the following below:

First disable live metrics and see if you are still getting 401s. It could be that the issue is isolated to sending to live metrics endpoint.
Without disabling live metrics, try to run your code NOT deployed as an Appservice application. This will confirm whether or not your appservice instance is not configured to send to your appinsights resource which needs entraID auth. Instead of looking at your AppServiceLogs (which exists in LogAnalytics), go to your specific ApplicationInsights resource logs, under the traces table.

@lzchen ,

It does appear that these logs are being generated by an SDK in AppService and not from the Azure monitor distro (I see the type is AppServiceConsoleLogs). The distro is not responsible for sending those logs.

I enabled Debug logs for the app-service.
We do have logs from the azure monitor distro in the AppServiceConsoleLogs. as you can see in the following screenshot.
One small observation here,
We initially had these variables defined in App-Service.

After removing all of the above except the APPLICATIONINSIGHTS_CONNECTION_STRING, following logs were seen.
I am not sure, why app-service was trying to resolve 'westus-0.in.applicationinsights.azure.com' ? we have given 'eastus-0.in.applicationinsights.azure.com' in connection string.  One point to note here is that all our app-insights resources and LAW are behind AMPLS.

Our cloud resources are behind a firewall, so currently connecting to Application insights from local system is not possible.

@summahto
We need to step back a bit. In order to identify what logs are sent by the distro, you need to look at Applicaiton Insights logs and not AppServiceConsoleLogs. It is too difficult right now to decipher whether appservice is responsible for creating those logs or if the distro is.  It is also difficult to diagnose your original issue of not seeing logs when entra id is enabled if we don't look at the actual raw logs that the distro is responsible for, and not be distracted by all the noise that is being added by changing configurations. First, focus more attention on Application Insights Logs by going to "Logs" section under your ApplicationInsights resource. These logs + log levels are configured ONLY by in code configuration. Please modify your code snippet that you supplied:
def configure_basic_azure_monitor():

    conn_string = settings.APPLICATIONINSIGHTS_CONNECTION_STRING

    docgen_logger = logging.getLogger("test")
    docgen_logger .setLevel(logger.DEBUG)

    managed_credential = ManagedIdentityCredential(client_id=settings.AZURE_CLIENT_ID)

    configure_azure_monitor(
        connection_string= conn_string,
        logger_name=docgen_logger.name,
        credentials=managed_credential,
        enable_live_metrics=False,
    )

Only after we have seen that your logs in your AppInsights resource looks correct we can begin ruling out what is going on with your AppService logs.
SEPARATELY IN ADDITION TO ABOVE, please re-add all the variables that you had defined previously in AppService. They are all needed in setting up your application to be the most bare bones, and removing them may cause unnecessary noise/telemetry that may affect our investigation.

I am not sure, why app-service was trying to resolve 'westus-0.in.applicationinsights.azure.com' ? we have given 'eastus-0.in.applicationinsights.azure.com' in connection string.

In debug mode, you will get logs related to retriable scenarios in which telemetry attempted to send but couldn't, so we retry for later. These are legitimate use cases. You are seeing westus.... logs because you removed APPLICATIONINSIGHTS_DISABLED_STATSBEAT_ALL environment variable. If you do not want to see those, you can re-add the env var back.


Mock-SAS handling of X.509 certificates with Winnforum extensions

Mock-SAS is based on Python 2.7. Python 2.7 has its own SSL implementation which is Python27\Lib\ssl.py.
Winnforum has speciifc requirements for X.509 certificates for SAS and CBSD which include new OIDs <IP_ADDRESS>.4.1.46609.X.
In https://github.com/Wireless-Innovation-Forum/Citizens-Broadband-Radio-Service-Device/issues/new it is explained how to implement this OIDs in Linux openssl.cnf so the Linux will recognize these OIDs.
Currently in the Python SSL implementation we did not find a way to implement these new OIDs so Mock-SAS fails to establish the TLS connection if the <IP_ADDRESS>.4.1.46609.X are present in the X.509 certificates.

@idanrazisr i'm assigning this to you; may be as verification for pull #44 you can verify this?

The code has been modified in the Critical_Extension_Branch and sent sent for review to be merged in master branch.

Following the readme_file_x509_RSA_certs_test_labs file in the SAS Harness release, there is no need for Mock-SAS to handle X.509 certificates with Winnforum extensions as critical OIDs. Closing issue.


Test Failure server_uses_https

Refactoring tls support on another PR

Do you have any other information on the PR we could link the issue to? @nopestack
---- server_uses_https stdout ----
thread 'server_uses_https' panicked at 'called `Result::unwrap()` on an `Err` value: reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Ipv4(<IP_ADDRESS>)), port: Some(52212), path: "/health", query: None, fragment: None }, source: hyper::Error(Connect, Error { code: -9833, message: "decoding error" }) }', crates/vrrb_rpc/tests/http.rs:78:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This part of the codebase is deprecated and can be safely removed.


Feature: add TodayHeaderComponent

Fixes #104
Add TodayHeaderComponent prop, to allow highlighting today in the header.
I was not sure on how to visually highlight today in the header, so with this prop the developer can fully customize how it looks. Examples: bold the font, add a rounded box, change the background color, etc.

Thanks for the nice improvement, LGTM.


Unicode not working - non Latin languages not supported

Currently we cannot see non-latin characters displayed on the badge. Please add support for Unicode.

@mariobehling can i work on this issue

@refactor-droidyy go ahead


Convert to module (ESM).

Convert to a ESM module that generates CommonJS.
Simplify away unused webpack .jsonld loader.  Was used in previous context package incarnations.
Simplify away rollup. Only using webpack to convert module to CommonJS.

This is an attempt at designing the more up-to-date context package.  None of the others are ESM-first yet and are likely always loaded as CJS in node.  Even if this isn't used, it's likely the direction to head for simplification.


Add information about different types of licences

Added information that describes the two different types of licences available as of September 28th within Data Hub

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


NEXT-34181 - Partial check in EntityReader fetchAssociations

1. Why is this change necessary?
Search criteria will return an error if fields are set, but many-to-many association is set and fields are missing
2. What does this change do, exactly?
Checks if search criteria is partial. If so, check if association should be fetched
3. Describe each step to reproduce the issue or behaviour.
/api/search/product
{
    "limit": 1,
    "fields": [
        "name"
    ],
    "associations": {
        "properties": {}
    }
}

4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-34181
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Thanks for the PR @En0Ma1259 it's merged and also backported to 6.5 :)


Does not support firefox

Describe the bug
When trying to install the extension from Firefox, it gives an error that is is corrupt and so cannot install it. I've ran the following commands:
git clone https://github.com/ndom91/briefkasten-extension
cd briefkasten-extension
npm install
npm run build
cd dist
zip -r -FS ../my-extension.zip *

And then gone to the extension manager => install from file => my-extension.zip
From debugging, it seems that it's an invalid manifest version, so I changed that to 2 and then tried it again. Now it gives the following error:
Extension is invalid

Reading manifest: Error processing background: Value must either: contain the required "page" property, contain the required "scripts" property, or not contain an unexpected "type" property

Reproduction
View above
System Info
System:
    OS: Linux 6.0 Pop!_OS 22.04 LTS
    CPU: (6) x64 Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz
    Memory: 10.54 GB / 15.56 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 12.22.9 - /usr/bin/node
    npm: 8.5.1 - /usr/bin/npm
  Browsers:
    Firefox: 107.0

Used Package Manager
npm
Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I have compiled it and submitted to Mozilla for signing. After it's been signed, I will post it here so others can just install it right away.

I have successfully compiled it and submitted it to Mozilla for signing. After it's been signed, I will post it here so others can just install it right away.

Were you able to get the extension signed?

Unfortunately not, Mozilla has been less than lenient with my lack in
extension packing and assuming that I know how that works :D
--
Moritz Poldrack
https://moritz.sh

For those waiting, I was able to get Mozilla's approval.
You can download the addon here: https://addons.mozilla.org/addon/briefkasten-unofficial/


skrooge: remove QtWebKit dependency

Motivation for this change
Remove QtWebkit dependency introduced in
https://github.com/NixOS/nixpkgs/commit/a1b8fe8c8fa59024ed87024b54ddba38811010cf
It is no longer needed after upstream fix
https://cgit.kde.org/skrooge.git/commit/?id=cb8b56fa729d5f6ee4652e79de354721f4742ee7
Corresponding check in https://github.com/NixOS/nixpkgs/issues/53079 could be updated after this PR is merged.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @jokogr
@GrahamcOfBorg build skrooge


Removed WRITE_EXTERNAL_STORAGE and READ_EXTERNAL_STORAGE permissions (Android)

Is your feature request related to a problem? Please describe.
My assumption is that this plugin does not need to read or write to the external storage therefore WRITE_EXTERNAL_STORAGE and READ_EXTERNAL_STORAGE permissions are not required - see
https://developer.android.com/reference/android/Manifest.permission#WRITE_EXTERNAL_STORAGE and https://developer.android.com/reference/android/Manifest.permission#READ_EXTERNAL_STORAGE
Describe the solution you'd like
Removed WRITE_EXTERNAL_STORAGE and READ_EXTERNAL_STORAGE permissions from the manifest file
Describe alternatives you've considered
N/A
Additional context
The Google Play informs that starting May 5th, you must let us know why your app requires broad storage access
We've detected that your app contains the requestLegacyExternalStorage flag in the manifest file of 1 or more of your app bundles or APKs.

Developers with apps on devices running Android 11+ must use Scoped Storage to give users better access control over their device storage. To release your app on Android 11 or newer after May 5th, you must either:

Update your app to use more privacy friendly best practices, such as the Storage Access Framework or Media Store API
Update your app to declare the All files access (MANAGE_EXTERNAL_STORAGE) permission in the manifest file, and complete the All files access permission declaration in Play Console from May 5th
Remove the All files access permission from your app entirely
For apps targeting Android 11, the requestLegacyExternalStorage flag will be ignored. You must use the All files access permission to retain broad access.

Apps requesting access to the All files access permission without a permitted use will be removed from Google Play, and you won't be able to publish updates.

requestLegacyExternalStorage is often used with WRITE_EXTERNAL_STORAGE so it might require removing the both (if they are not required)
Please also see https://support.google.com/googleplay/android-developer/answer/10467955

I added the appropriate PR - see https://github.com/capacitor-community/camera-preview/pull/123

Closing as this was merged. Thanks for your contributions 😄


Tweak sidebar padding for consistency

Is your feature request related to a problem?
The sidebar in files is very nice, but there are two inconsistencies between the sidebar and right-hand pane that bother me: the sidebar header (hamburger icon and "Files" text) is not aligned with the tabs (see picture below), and the padding of sidebar items is almost-but-not-quite-the-same as padding between folder items in the right-hand-pane list view (see picture below).
Describe the solution
Slightly decreasing the vertical padding between sidebar elements to match the right-hand pane and slightly lowering the hamburger button and "Files" text would solve this. This would have the additional benefit of allowing more sidebar items to be seen at a time without scrolling!
Describe alternatives you've considered
Padding in the right-hand-pane could also be increased to match the sidebar, but I don't think that would look very nice.
Additional comment
App version <IP_ADDRESS>

This issue should be resolved in main.


chore: move files into Analysis.Normed.Lp

part of my organization of Analysis.NormedSpace

bors merge


RDP Authentication Artifacts

Type : Artifacts
OS/Protocol/Tool : Windows RDP
MindMap Topics : 159
Description : This Mindmap represents different artifacts related to RDP authentication with NLA enabled or disabled to help collect and analyze forensic artifacts during DFIR engagements.

Really great stuff! Thank you for your work!

Happy I can help


Add the ability to await for event happening more than once

It would be nice to have a built-in feature that will allow awaiting for some event happening more than once. For example, I'm sending three messages to the queue and expecting that they all three will be received at some point in time.
for (Message message : messages) {
    sendMessageToTheQueue(message);
}

List<Message> messages = await().until(receiveMessage(), hasMyId(), times(3));

...
Callable<Message> receiveMessage();
Matcher<Message> hasMyId();

That is a good idea but I don't think it belongs in the library. You could quite easily write a custom Hamcrest matcher for this already. Perhaps there's not even a need to write a customer matcher. Maybe you can just compose some of the existing ones such as:
List<Message> messages = await().until(receiveMessage(), allOf(hasMyId(), hasSize(3)))

So I'll close this issue. Please tell me if you disagree.

Unfortunately it's imporssible in my case. Let me explain.
I'm using awaitility for writing system tests of our product. It's a JMS based processing service. The case that I'm frequently have is such that if I post single message to the inbound queue of this service it may produce more than one messages to it's outbound topic.
The API for listening JMS topic is limited and all I can get is just one message at a time. And to check whether it's the message that I'm looking for, I need to match it with my Hamcrest cryteria.
Right now for this taks I made my own Callable function that aggregates the matching results inside. But think that it would probably be a nice-to-have feature built-in awaitility.

Oh so you mean that you want to wait for just ONE specific message but there may be other messages that arrive as well (before or after the message you're looking for)? We're using Awaitility for exactly this in my current project. We do something like this (rabbitmq):
MyEvent event = (MyEvent) await().until(() -> rabbitTemplate.receiveAndConvert(queue.getName()), instanceOf(MyEvent.class));

Using this approach we get the first event of the MyEvent. If you have several events of the same type you should be able to do something like:
MyEvent event = (MyEvent) await().until(() -> rabbitTemplate.receiveAndConvert(queue.getName()), allOf(instanceOf(MyEvent.class), hasProperty("id", equalTo("123-454-ABC")));

No no, I want to catch more than one message that matches cryteria. Something like:
 MyEvent event = (MyEvent) await().until(() -> rabbitTemplate.receiveAndConvert(queue.getName()), anyOf(hasProperty("id", equalTo("123-454-ABC"), hasProperty("id", equalTo("456-789-CDE")));

But this case will return only the first event that matches one of listed cryteria. But I want to catch, for example, exatcly three events from the stream.

Ah I see. Then I would probably create a custom Hamcrest matcher for this (perhaps that's what you're already doing?). I'm a bit hesitant to extending the API since you should be able to do it from a Hamcrest matcher. However if more users requests something like this I might change my mind. Or perhaps you just need to convince me even more? :)

Hamcrest matcher can match only something that would Callable return. So for the case when you may want to catch three events from the stream their aggregation must happen inside of the Callable class.
What I'm trying to do right now is wrapping the Callable into some sort of the aggregating Callable with it's own matcher. Following our previous RabbitMQ exmaple, I propably would look something like this:
 await().until(receiveAllOf(() -> rabbitTemplate.receiveAndConvert(queue.getName()), anyOf(hasProperty("id", equalTo("123-454-ABC"), hasProperty("id", equalTo("456-789-CDE"))), hasSize(3));

Here we have two matchers. One for the actual event filtering. And the other for size of the aggregating collection.

Here's a simple patch that implements aggregational feature

Sorry for not getting back to you earlier. I see what you mean, I'll have to think about it a bit though. The way I see this is essentially as a reduce of what Awaitility returns. So perhaps it should be generalized and the reduce should be made explicit. Something like:
await().until(reduce( (list, x)  -> list.add(x), new ArrayList<>()), () -> somethingThatReturnXs());

That would save all values to the list. Perhaps a context should be supplied so that you can capture all intermediate values as well? Something like this (if we pretend that list.add returns a new list):
await().until(reduce((list, ctx) -> ctx.isMatching() ? list.add(ctx.getEntity()) : list), new ArrayList<>()), () -> somethingThatReturnXs());

My only consern is: please, make it coincise for pour souls who still use java 7 (like me).

I suppose it would be possible to supply some connivence methods.


Use ibig

I think this does indeed provide a 20 to 50% speedup.

ibig on top of the evaulation cache does not improve the fixed column generation much, but it does improve our evaluation benchmark, so I think it is still good to have.


Watch and re-build automatically during unit test runs

Currently, if you want the tests to update, you have to run "ng build" on one tab and karma on the other.  The test run should be watching for updates and rebuilding.

Done via CLI


Jjc hunch

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
What are the significant changes in functionality due to this change request?
Testing a hunch

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test qsubs sawtooth on 63ba103 : invalidated by @aalfonsi

Job Test mac on 63ba103 : invalidated by @aalfonsi

Has been incorporated into feature selection branch.


coverPrefs

crop
filter
overlay

Need anything else for this @jonnor @bergie ?

Tests?

Like what?

Like one example of invalid data, and one of valid data in a examples/metadata.yaml file would do the trick. Following the pattern of the other example files.

examples/metadata.yaml doesn't exist. I added a good example to examples/contentblock.yml but don't understand what kind of invalid example would be helpful.

OK, added a fail.
Ready.

Thanks.
In an attempt to clarify: The purpose of invalid type tests is primarily to know what the limitations specified in schema is (correctly) applied. The purpose of valid type tests is primarily to ensure that the data we intend to accept is actually accepted and that we continue to accept it as schema evolves (not breaking compatibility).

Yeah. Was heartening to first make the expected fail fail, then pass.

Mo points!!
On Apr 28, 2015 7:52 PM, "Forrest Oliphant"<EMAIL_ADDRESS>wrote:

Yeah. Was heartening to first make the expected fail fail, then pass.
—
Reply to this email directly or view it on GitHub
https://github.com/the-grid/apidocs/pull/6#issuecomment-97266775.


Selected photos picker when user chooses select photos permission on iOS 14 and above

Feature Request
Motivation Behind Feature
When using the camera plugin on iOS in combination with cordova.plugins.diagnostic to request camera permissions as mandated by the platform, the camera plugin does not seem to differentiate between the regular Allow Access to All Photos and the new Select photos permission added in iOS 14. It always opens the camera roll and shows / allows selecting from all photos.
Note: I've searched a while for info about this feature but have not found it, either in plugin docs or open/closed issues.
Feature Description
Introduced in iOS 14. Gives user the option to only allow an app access to selected photos, which the user can then select from (instead of the full camera roll). Article describing the feature.
Alternatives or Workarounds
None that I know of at this moment.

The picker provided by Apple doesn't allow to filter and show only the images that the user selected, it always shows all and that's what we use.
When the user select Select photos instead of Allow Access to All Photos, the library permission is limited instead of granted. Limited means the access to the photo info is limited (i.e. you can't get the photo exif), but you can still get a copy of the photo, which is what the plugin returns.

yeah, those are custom pickers, there is a native API to get all the limited photos and you can use that to present a custom picker to the user, but you have to build the picker UI


[BUG] Cython uses PyObject_GenericSetAttr() on types in Limited API

Describe the bug
This code now hits an assertion in dev-builds of the CPython main branch:
https://github.com/cython/cython/blob/7e8a172d21b0c723be14685e25ae854bbc6cdaec/Cython/Utility/ExtensionTypes.c#L694-L699
https://github.com/python/cpython/blob/e6fad7a0e3d824f4a3c9cd71a48208880606d705/Objects/object.c#L1714
    assert(!PyType_IsSubtype(tp, &PyType_Type));

This is used in the auto-reduce setup code to fix up the reduce methods after creating the type. The assertion was added in
https://github.com/python/cpython/pull/118454
Not sure if there's a way to work around this – or whether we should. It seems to be a hack on our side to update the type dict like this (even after adding the type to the module dict).
Code to reproduce the behaviour:
No response
Expected behaviour
No response
OS
No response
Python version
3.14
Cython version
master
Additional context
No response

It seems to be a hack on our side to update the type dict like this (even after adding the type to the module dict).

It is a hack - yes :)
I've asked for (and will probably get) a better way of dealing with this in future with https://github.com/python/cpython/issues/121654. But that doesn't help now really.

This is used in the auto-reduce setup code to fix up the reduce methods after creating the type.

It's also used in adding CyFunctions to the class dict, which is probably the more important use.
The easiest option would be not to set the immutable flag on these classes, although I don't really like that.

I think the better solution might be to use a custom metaclass which isn't flagged as immutable, but has a custom __setattr__ that acts immutable but provides a backdoor to use in construction.
If we do that then it precludes custom metaclasses on cdef class as a future feature though.

I think I can see a more robust hack... will try it and make a PR if successful


Documentation: orphaned page (without page type)

when page type gets removed from site, pages with that particular page type are left on site. That breaks migration constraint - only page types assigned to site are migrated, therefore page migration fails.
How to resolve: assign page type back to migrated site
Category: troubleshooting
Symptoms: migration toolkit will not find page class

Documented. Pull request created:
https://github.com/Kentico/xperience-migration-toolkit/pull/104


Bug: Owner gets removed after safe creation and added with UNKNOWN name

This PR:

Fixes the bug by updating the logic used to compare owners in checkAndUpdateSafe function

Travis automatic deployment:
https://pr379--safereact.review.gnosisdev.com


Trojan detected in NavConainerHelper

Just updated today NavContainerHelper to <IP_ADDRESS>.
After, when I run my script to create New-NavContainer I have this message :
Exception lors de l'appel de « ReadAllBytes » avec « 1 » argument(s) : « Impossible de terminer l’opération, car le fichier contient un virus ou un logiciel potentiellement indésirable.
Au caractère C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\New-NavContainer.ps1:1343 : 9
+         New-DesktopShortcut -Name "$containerName Command Prompt" -Ta ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

When I look in Windows Defender :

Scripts used to create container and cause the issue
New-NavContainer arguments : -Accept_Eula -Accept_Outdated -doNotCheckHealth -containerName CAGPG160-Dev -imageName mcr.microsoft.com/businesscentral/onprem:2004-fr -auth UserPassword -updateHosts -includeCSide:False -memoryLimit 8G -doNotExportObjectsToText:True -enableSymbolLoading:False -includeTestToolkit:True -includeTestLibrariesOnly:True -doNotUseRuntimePackages:True -includeAL:True -assignPremiumPlan:True -isolation process -useBestContainerOS -licensefile "D:\Sources\DevOps\Git\D365BC CALLIOPACKS\CAGPG\Configs\NAV-DEV-License.flf" -additionalParameters @("--volume "D:\Sources\DevOps\Git\D365BC CALLIOPACKS\CAGPG:C:\Source"")

Full output of scripts
Removing container CAGPG160-Dev
Removing CAGPG160-Dev from host hosts file
Removing C:\ProgramData\NavContainerHelper\Extensions\CAGPG160-Dev
Creating container CAGPG160-Dev...
NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Microsoft  - 1909
Docker Client Version is 19.03.8
Docker Server Version is 19.03.8
Using image mcr.microsoft.com/businesscentral/onprem:2004-fr-ltsc2019
Disabling Health Check (always report healthy)
Creating Container CAGPG160-Dev
Version: 16.0.11240.12076-fr
Style: onprem
Platform: 16.0.11233.12061
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.973 (ltsc2019)
Host OS Version: 10.0.18363.815 (1909)
A better Generic Container OS exists for your host (mcr.microsoft.com/dynamicsnav:10.0.18363.778-generic)
Using generic image mcr.microsoft.com/dynamicsnav:10.0.18363.778-generic
Generic Container OS Version: 10.0.18363.778 (1909)
Generic Tag of better generic: <IP_ADDRESS>
Using locale fr-FR
Using process isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file D:\Sources\DevOps\Git\D365BC CALLIOPACKS\CAGPG\Configs\NAV-DEV-License.flf
Files in C:\ProgramData\NavContainerHelper\Extensions\CAGPG160-Dev\my:
- AdditionalOutput.ps1
- license.flf
- MainLoop.ps1
- SetupNavUsers.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container CAGPG160-Dev from image mcr.microsoft.com/dynamicsnav:10.0.18363.778-generic
c8c3a269b78131e2bbf09c271e94ebf68e7ce66a09fd24c1413e689f586d0776
Waiting for container CAGPG160-Dev to be ready
Installing Business Central
Installing Url Rewrite
Installing OpenXML
Installing DotNetCore
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying Web Client Files
Copying Client Files
Copying ModernDev Files
Copying PowerShell Scripts
Copying Test Assemblies
Copying Applications
Copying ReportBuilder
Changing Database Server Collation to French_100_CI_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the product. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Copying Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Installation took 206 seconds
Installation complete
Stopping Business Central Service Tier
Initializing...
Setting host.docker.internal to  in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to  in container hosts file (copy from host hosts file)
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is CAGPG160-Dev
PublicDnsName is CAGPG160-Dev
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint EDECD2A327D7B20AEBFA0093761FD2EEACEA57F1
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating sa as SQL User and add to sysadmin
Creating SUPER user
WARNING: The password that you entered does not meet the minimum requirements.
It should be at least 8 characters long and contain at least one uppercase
letter, one lowercase letter, and one number.
Assign Premium plan for SA
Container IP Address: <IP_ADDRESS>
Container Hostname  : CAGPG160-Dev
Container Dns Name  : CAGPG160-Dev
Web Client          : http://CAGPG160-Dev/BC/
Dev. Server         : http://CAGPG160-Dev
Dev. ServerInstance : BC
Setting CAGPG160-Dev to <IP_ADDRESS> in host hosts file

Files:
http://CAGPG160-Dev:8080/ALLanguage.vsix

Initialization took 64 seconds
Ready for connections!
Reading CustomSettings.config from CAGPG160-Dev
Creating Desktop Shortcuts for CAGPG160-Dev
Exception lors de l'appel de « ReadAllBytes » avec « 1 » argument(s) : « Impossible de terminer l’opération, car le fichier contient un virus ou un logiciel potentiellement indésirable.
 »
Au caractère C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\New-NavContainer.ps1:1343 : 9
+         New-DesktopShortcut -Name "$containerName Command Prompt" -Ta ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Additional context

does it happen all the time? Yes
did it use to work? Yes

it looks like this happens when creating the shortcuts on the desktop.
There is definitely no trojan in containerhelper and the script to create the shortcut just creates a shortcut.
If you uninstall <IP_ADDRESS> and install <IP_ADDRESS> you will probably see exactly the same problem, this hasn't changed.
Somehow your virus scanner detects this and gives the error. You can add -shortcuts None to avoid the shortcuts and the error.

Thank you for your rapid feedback!
Would like to be sure that package was not infected on NuGet.
Maybe its the result of Windows Defender Security update which interpret a bad new trojan signature in the temp link generated by NavContainerHelper in C:\ProgramData\NavContainerHelper

The build machines, which builds and publishes navcontatinerhelper are not accessible by anybody (also not me) - there has never been a login on that machine.
Also - PowerShellGallery performs checks when we publish.
Third - I have run this on several machines with updated defender bits, which doesn't give me any issues.
I cannot say that you don't have Win32/Conteban.A!ml - but if you have, I cannot imagine that it is caused by NavContainerHelper.
The way containerhelper creates the links on the desktop is to create a .lnk file in the containerhelper folder and move it onto the desktop.

Same warning with me today allthough different virus. Hopefully Defender false positive of sorts but annoying.

I have a repro now, from old versions of containerhelper as well.
Apparently, newest Windows Defender will react on the fact that I add an IconLocation to an .ico file inside the navcontainerhelper - I will have to fix that in the containerhelper to use a different icon.
In the meantime, you can modify
C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<your version>\ContainerHandling\New-NavContainer.ps1

Search for -IconLocation $dockerIco - and remove that parameter (both -IconLocation and $dockerIco)
Restart the PowerShell session.

Worked great, thanks Freddy! I wonder why it started failing for us in the evening when was building fine all day...

I see you already fixed and merged to master. I'am just curious how long does it normally take until its available in the psgallery?

Looking at the pipeline - another 15 minutes (unless something fails)

Publishing went wrong - will be another 2-3 hours, sorry

NavContainerHelper <IP_ADDRESS> is released with a fix for this.


GEODE-7852: test SNI client agin a geode (2-node) cluster

Automated tests for GEODE-7852: this PR sets up a dockerized 2-node Geode cluster and uses groups to target content to one node or the other.

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

All checks were green except rat. I had missed a license file. Just added it so I believe CI will be green on this now.

I added comments @upthewaterspout. I had to rebase the branch so once the tests run I'll merge.


Kanban Board Setup

The following is a breakdown of how we have distributed the Kanban board activities:
Group Members:

@Graciano1997 (STUDENT A)
@Bettenoch (STUDENT B)

Find the github project link HERE

Status: Approved :tada: :heavy_check_mark:
Hi @team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Kanban board setup properly. :heavy_check_mark:
Tasks shared fairly equal. :heavy_check_mark:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[Impeller] Implement shadows

See https://github.com/flutter/engine/blob/4a519a80e13d74e1df9a7d5606162cbd2433c351/impeller/display_list/display_list_dispatcher.cc#L735

This is blocked on https://bugs.chromium.org/p/skia/issues/detail?id=12125, which makes the DisplayListCanvasRecorder unable to record shadows.

In-progress implementation of the box shadow case here: https://github.com/flutter/engine/compare/main...bdero:bdero/is-rectangle
Still need to wire it up in the DrawShadows path and test against gallery.


Identity and update to latest NServiceBus

The NServiceBus library code in this article was very outdated, including referencing outdated versions of the linked sample project, so these have been updated to reflect latest. Additionally, references have been made to connecting using Entra Id credentials and the preferred function type has been switched to isolated worker.

@microsoft-github-policy-service agree company="Particular"

@spelluru
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@PhilBastian - usually, I have access to make changes in branches, but I can't with yours for some reason. If you will make the suggested changes, we can merge this. thanks.

@PhilBastian - usually, I have access to make changes in branches, but I can't with yours for some reason. If you will make the suggested changes, we can merge this. thanks.
Make sure to tag me when ready.

@Jak-MS @spelluru I have made the suggested changes

@spelluru
Can you review the proposed changes, specifically this new commit?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@Court72 - Please go and merge the PR. Thanks.


Redirect error in http://elm-lang.org/docs/syntax

In http://elm-lang.org/docs/syntax there is a link in the “Mapping” section to http://elm-lang.org/blog/announce/0.7.elm#do-you-even-lift Following this link in Chrome results in a Too Many Redirects error screen.
Doing a raw GET /blog/announce/0.7.elm#do-you-even-lift HTTP/1.1  results in a 200 and a reasonable looking HTML document,
but GET /blog/announce/0.7.elm HTTP/1.1 results in :
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Server: Snap/<IP_ADDRESS>
Date: Thu, 18 Jun 2015 14:13:08 GMT
Location: /blog/announce/0.7.elm

Confirmed in Chrome, Windows 7.

This has been fixed by this commit, so the issue can be closed.


Fix radial fourier template test

Dominant order produces different output, might
be triggered by many cores/ many workers.
Thx @sk1p @uellue
Fixes #895
Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if
changes were made to the GUI)

Merged, thank you!


bump apollo dependency

migrate all the current data fetching to be like this https://tianpan.co/onefx#fetching-data and use apollo client like this https://github.com/puncsky/web-onefx-boilerplate/blob/master/package.json#L83

use data/quries.ts for queries and data/mutations.ts for mutations


Adds  test cases for DateTime extensions

I have added a few test cases for Date time extensions. Followings are the methods.

DateTimeExtensions.ToMMDDYY

DateTimeExtensions.ToDDMMYY

This looks good. Merging with dev. Please go ahead and add other test cases as well @divyarbehera :)


[Feature Request] Ability to save some data from request body to environment variables

Good day
I'm seeking for an ability to perform some request, parse its JSON response, and save one field from JSON object to some environment variable to perform another requests with this value.
For example. There is a REST API server that requires clients to be authorized to access some sensitive information. To authorize, clients must to send a POST request to specific path (/auth) and then they will receive a JSON object with an access token like {"token": "aabbccddeeffgghh"}. Then this token must be put into Authorization header and carried with each further request.
For now, to fill Authorization header automatically, I need to use Request Chaining (and, as result, send one auth request before each GET request to the sensitive data). Or to copy a received access token to each request template manually.
Desired behavior: Auth request is performed only once. Then it's dispatched, token value extracted and saved to environment variables. Environment variable which contains token value then is inserted in each request template

Or I just misunderstood something

OK. Misunderstood. In request chaining auth request will be performed once and then its response is saved. To user saved value in other responses, response filter (tag... but why 'tag'?) must be used.
This issue may be closed


Doesn't appear to work with 1.2.8 murmur server.

!roll help works, but any other command are ignored. I've installed all pre-reqs.

If you're still following this, I've forked it and made a version that does work, but uses a different method for the actual rolling. I'm hoping it doesn't just work for me.

@moreisee Sorry for getting back to you that late. That sounds odd, did you get any error message? This works for me:
[8:18 AM] To Root: !roll d6
[8:18 AM] (Channel) dicebot: vaxr: 5
If !roll help works, the server version should not matter.


Fix SSL redirect for LB type NodePort

fixes #967
https://serverfault.com/questions/831065/i-need-to-configure-haproxy-with-multiple-ssl-ports

Is this merged , because i cant get ssl-redirect to work

Yes. Please open a new issue with Ingress yaml and the what you expect to see and what you see. Also, what version of Voyager you are using.


Background update on Windows steals focus

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: any
OS Version: Windows 10/11 any

Steps to Reproduce:

Have background update option enabled.
Launch VSCode, start doing something.
VSCode detects an update, puts the timer overlay on Settings icon on the left rail, focus goes away from VSCode window.

This happens all the time across any VSCode and Windows 10/11 version. Great indicator of an update that had arrived, even though I specifically want it to happen in background.

Another potential symptom of this bug is with the preview of a markdown file opened.
Switching to another tab and switching back to the preview causes the preview to reload/update which causes keyboard to lose focus (e.g. pushing up arrow does not cause preview to scroll up)


Fix fake diode name

The recent update to the fake diode resulted in a name change, which I think is unintended.
It looks like a couple of commits haven't hit github yet, I can rebase when they do.

I'm now seeing:
Cell sky130_ef_sc_hd__fakediode_2 couldn't be read

Is that because the gds file has the incorrect name:
strings ./sky130/custom/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fakediode_2.gds
sky130_fd_sc_hd
sky130_fd_sc_hd__diode_2

I'm now seeing:
Cell sky130_ef_sc_hd__fakediode_2 couldn't be read

Is that because the gds file has the incorrect name:
strings ./sky130/custom/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fakediode_2.gds
sky130_fd_sc_hd
sky130_fd_sc_hd__diode_2

Thanks; yes, it was an unintended copy-paste error. https://github.com/RTimothyEdwards/open_pdks/pull/102 fixes both of those issues now.

Thanks; yes, it was an unintended copy-paste error. https://github.com/RTimothyEdwards/open_pdks/pull/102 fixes both of those issues now.

Pulled and merged.

Pulled and merged.


[BUG] VNC and Serial Console is blank when accessed through Rancher Manager

Describe the bug
After adding my Harvester environment to Rancher Manager, cannot access a Virtual Machine's VNC and serial console. Both just show a blank screen. Works fine if accessing Harvester directly. These screen captures show Harvester on the left and Rancher Manager on the right:

To Reproduce
Steps to reproduce the behavior:

Add Harvester to Rancher Manager
Create a Virtual Machine
In the Rancher Manager UI, navigate to Harvester.
Once VM is running, select the Console, Open in Web VNC or Open in Serial Console options.

Expected behavior
VM consoles are accessible when accessed through Rancher Manager.
Support bundle
Environment

Harvester ISO version: v1.1.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Equinix m3.small.x86 (2 nodes)

Additional context
None

Hi @dnoland1  I imported harvester v1.1.0 into rancher 2.7.0 and both open VNC and open Serial Console worked fine.

What is the image used by your VM ?
I noticed that you're using Serial Console in rancher and you're opening web VNC in harvester,  I want to confirm, can you check in rancher to see if web VNC is working fine ？
What version of rancher are you using?

1. What is the image used by your VM ?

https://cloud-images.ubuntu.com/lunar/current/lunar-server-cloudimg-amd64.img

2. I noticed that you're using Serial Console in rancher and you're opening web VNC in harvester,  I want to confirm, can you check in rancher to see if web VNC is working fine ？

Both give me a blank screen in Rancher.

3. What version of rancher are you using?

v2.6.9

@dnoland1  I used the information you provided to reproduce this problem, but everything is working as expected.

Does this problem occur 100%?
Maybe it was just a broken link from rancher's websocket at the time ？
Can you check if the websocket link works?

@dnoland1  I used the information you provided to reproduce this problem, but everything is working as expected.

Does this problem occur 100%?
Maybe it was just a broken link from rancher's websocket at the time ？
Can you check if the websocket link works?

@WuJun2016 I'm not able to reproduce it. If it happens again, I'll check the websocket in developer tools and reopen the issue if needed. Thanks for looking into it.


Change AddressBook class to ExecutiveProDb

Change all instances of AddressBook to ExecutivePro
Update test cases with new class name
Fixes #62

LGTM


UI: Integrate Tungsten Fabric

Description
Related https://github.com/apache/cloudstack/pull/4205
This PR for adding new UI of Tungsten Fabric

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@utchoang tungsten fabric hasn't been merged yet, is this still draft?


pkgconfig not recognised

With tmux, spec-cleaner does following:
# osc diff
...
 BuildRequires:  automake
+BuildRequires:  pkg-config
 BuildRequires:  pkgconfig

It seems that either pkgconfig should be recognised, and pkg-config not added in this case or pkgconfig should be replaced by pkg-config.

Actually lest take it the other way around and use pkgconfig because it works on fedoras too.


[submit queue] should process older PRs sooner

Currently, submit queue sorts the PRs based on the PR number. This could be very misleading because many PRs stay open for a long time and becomes ready to merge only very recently. We should have a better metrics to gauge how long the PR has stayed in the queue and respect the order when processing.
One idea is to sort the PRs by the lgtm label timestamps, so that the PR that has been ready the longest time gets to be picked up first.

while #408 does not address this request, it creates a framework to do so if we so choose.

while #408 does not address this request, it creates a framework to do so if we so choose.

#408 seems like a great start. Thanks!


Date Sorting

Hello, I am using the 'react table' and 'moment' to format dates in the column I am trying to sort. However, although I have set the accessor like so:
{
Header: 'Date',
id: 'date',
accessor: row => moment(row.start).format('lll'),
},
The column is still sorting as a literal string. I have been looking through the many solutions suggested on this repo, but none have seemed to work for me. I welcome any suggestions to sorting date. Many thanks in advance!

Please repost this question in the spectrum.chat/react-table forum. Thanks!

@mBromige did you receive anxy useful answer?

This is mrBromige anwser on spectrum:
I have found a solution, if experiencing the same issues:
4:06pm
{
Header: 'Date',
id: 'date',
accessor: row => moment(row.start).format('x'),
Cell: row => moment(row.original.start).format('lll'),
},
3
4:07pm
Sort the accessor by timestamp and then format it to your desired moment date style


[BUG] Android 11 crash - Unable to get provider tech.linjiang.pandora.Pandora

I followed all the basic steps to setup the library, but on android 11 devices I get this error.
With android 9 all works perfectly

Hi @Phoenix986, thanks a lot for using Iadt and for reporting the first external issue to this project :1st_place_medal: :+1:
Unfortunately I can not reproduce this crash following the basic steps over a standard app project from template on an Android 11 device. But this exception is well known for me and I've been working on a similar crash happening for React Native apps. This changes are not published already but it will be done this weekend under a brand new version (0.0.58). If you don't mind, wait to test your issue against the next version.
If 0.0.58 don't fix your issue, it would be very helpful for me to have the full log of the Iadt startup. To do so, please add 'debug=true' and 'injectEventsOnLogcat=true' to the inappdevtools gradle configuration closure, rebuild your app into your Android 11 device and send me the logcat output. Also let me know if your app is using more than one Android process or if your app is doing any activity redirection on the startup?
Cheers!

Hi @Phoenix986, thanks a lot for using Iadt and for reporting the first external issue to this project :1st_place_medal: :+1:
Unfortunately I can not reproduce this crash following the basic steps over a standard app project from template on an Android 11 device. But this exception is well known for me and I've been working on a similar crash happening for React Native apps. This changes are not published already but it will be done this weekend under a brand new version (0.0.58). If you don't mind, wait to test your issue against the next version.
If 0.0.58 don't fix your issue, it would be very helpful for me to have the full log of the Iadt startup. To do so, please add 'debug=true' and 'injectEventsOnLogcat=true' to the inappdevtools gradle configuration closure, rebuild your app into your Android 11 device and send me the logcat output. Also let me know if your app is using more than one Android process or if your app is doing any activity redirection on the startup?
Cheers!

Hi, I've just published Iadt v0.0.58, please test if it fix your issue and let me know.
A couple of considerations:

As part of the startup process changes, we are now detecting crashes on Pandora initialisation (like the one from this issue) and auto disabling related features when it happens. If you can now reach our overlay interface without a crash, please also check that any of them are enabled. To do so just check if you can navigate to network, view or storage inspectors from our home screen.
This release introduce some new error logs on startup but they don't cause a crash. I will fix it with a small release in few days.

Hi, I've just published Iadt v0.0.58, please test if it fix your issue and let me know.
A couple of considerations:

As part of the startup process changes, we are now detecting crashes on Pandora initialisation (like the one from this issue) and auto disabling related features when it happens. If you can now reach our overlay interface without a crash, please also check that any of them are enabled. To do so just check if you can navigate to network, view or storage inspectors from our home screen.
This release introduce some new error logs on startup but they don't cause a crash. I will fix it with a small release in few days.

hi @rafaco, thanks for the update, i will try it asap, and i will let you know if the problem is gone and if all Pandora's features work

hi @rafaco, thanks for the update, i will try it asap, and i will let you know if the problem is gone and if all Pandora's features work

This issue was fixed a year ago


Adopt opentelemetry

related https://github.com/open-telemetry/opentelemetry-go/issues/4779

Seems like a great idea, I would be good with that. 👍

/kind feature

/remove-lifecycle stale


Why redirect install.fisherman.sh to a gist?

You could also use a git.io for url shortening.

The reason is, a friend asked me so I implemented this feature as soon as I could.
In the future, instead of a Gist, I think this should redirect to either a file in this repo, or a new repo created for model installs. This way we could have opinionated installs outside the realm of this project, but still recognized under the organization.

Oh, thanks a lot for the answer.
Good luck and have a nice day!

Not a problem, keep them coming! :smile:

@bucaran I had emailed you some problem too, should I open an issue for them?

@pickfire Absolutely, feel free to open as many issues as you like!
:anchor:


latest buf schema registry

Description
after merging the proto file changes for renaming processors' name to type, a new schema for conduit was pushed to the Buf schema registry, and this PR uses the latest version.
Fixes #498
Quick checks:

[ ] I have followed the Code Guidelines.
[ ] There is no other pull request for the same update/change.
[ ] I have written unit tests.
[ ] I have made sure that the PR is of reasonable size and can be easily reviewed.

Oh, I almost missed it - @maha-hajja can you please run make generate to update the OpenAPI definition? 🙏


Prevent resizes reducing IOPs/Throughput

We've seen an issue with resizing disks that have the "ebs.csi.aws.com/reconcile" tag set, where it reduces the IOPs and throughput for a volume using an enhanced storage class. This PR just adds an additional check to make sure that we aren't reducing those values when resizing, if the disk already has values greater than the generic scaling.

SDC this prevents in future: https://github.com/timescale/Support-Dev-Collab/issues/1202#issuecomment-1678693309


Use SvgBuilder (#30)

From now, SvgBuilder.MsBuild replaces the manual image conversion.
Closes #30.

@gsomix, could you please review?

We'll probably need to bump Mono version or something like this.

Travis seems to have some small troubles, but one of the builds has finally been fixed, so I state that the build is ok.
@gsomix, review please!


[SPARK-15453] [SQL] FileSourceScanExec to extract outputOrdering information

What changes were proposed in this pull request?
Jira : https://issues.apache.org/jira/browse/SPARK-15453
Extracting sort ordering information in FileSourceScanExec so that planner can make use of it. My motivation to make this change was to get Sort Merge join in par with Hive's Sort-Merge-Bucket join when the source tables are bucketed + sorted.
Query:
val df = (0 until 16).map(i => (i % 8, i * 2, i.toString)).toDF("i", "j", "k").coalesce(1)
df.write.bucketBy(8, "j", "k").sortBy("j", "k").saveAsTable("table8")
df.write.bucketBy(8, "j", "k").sortBy("j", "k").saveAsTable("table9")
context.sql("SELECT * FROM table8 a JOIN table9 b ON a.j=b.j AND a.k=b.k").explain(true)

Before:
== Physical Plan ==
*SortMergeJoin [j#120, k#121], [j#123, k#124], Inner
:- *Sort [j#120 ASC, k#121 ASC], false, 0
:  +- *Project [i#119, j#120, k#121]
:     +- *Filter (isnotnull(k#121) && isnotnull(j#120))
:        +- *FileScan orc default.table8[i#119,j#120,k#121] Batched: false, Format: ORC, InputPaths: file:/Users/tejasp/Desktop/dev/tp-spark/spark-warehouse/table8, PartitionFilters: [], PushedFilters: [IsNotNull(k), IsNotNull(j)], ReadSchema: struct<i:int,j:int,k:string>
+- *Sort [j#123 ASC, k#124 ASC], false, 0
+- *Project [i#122, j#123, k#124]
+- *Filter (isnotnull(k#124) && isnotnull(j#123))
 +- *FileScan orc default.table9[i#122,j#123,k#124] Batched: false, Format: ORC, InputPaths: file:/Users/tejasp/Desktop/dev/tp-spark/spark-warehouse/table9, PartitionFilters: [], PushedFilters: [IsNotNull(k), IsNotNull(j)], ReadSchema: struct<i:int,j:int,k:string>

After:  (note that the Sort step is no longer there)
== Physical Plan ==
*SortMergeJoin [j#49, k#50], [j#52, k#53], Inner
:- *Project [i#48, j#49, k#50]
:  +- *Filter (isnotnull(k#50) && isnotnull(j#49))
:     +- *FileScan orc default.table8[i#48,j#49,k#50] Batched: false, Format: ORC, InputPaths: file:/Users/tejasp/Desktop/dev/tp-spark/spark-warehouse/table8, PartitionFilters: [], PushedFilters: [IsNotNull(k), IsNotNull(j)], ReadSchema: struct<i:int,j:int,k:string>
+- *Project [i#51, j#52, k#53]
   +- *Filter (isnotnull(j#52) && isnotnull(k#53))
      +- *FileScan orc default.table9[i#51,j#52,k#53] Batched: false, Format: ORC, InputPaths: file:/Users/tejasp/Desktop/dev/tp-spark/spark-warehouse/table9, PartitionFilters: [], PushedFilters: [IsNotNull(j), IsNotNull(k)], ReadSchema: struct<i:int,j:int,k:string>

How was this patch tested?
Added a test case in JoinSuite. Ran all other tests in JoinSuite

ok to test

Test build #64552 has started for PR 14864 at commit 07196a8.

Test build #64552 has finished for PR 14864 at commit 07196a8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64552/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins test this please. The last run had JVM crash

Test build #64592 has started for PR 14864 at commit 07196a8.

Test build #64592 has finished for PR 14864 at commit 07196a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64592/
Test PASSed.

cc @rxin , @cloud-fan for review

I'm not sure it's safe to do so. A bucket may have more than one files(this can happen if we append data into bucketed table), so for each file, it's sorted, but for the whole bucket, it is NOT, because currently Spark SQL doesn't guarantee the records are sorted across all files of a bucket.

@cloud-fan : I have taken care of that case in the PR (see L175 to L185). The sort ordering will only be used when all the buckets have single file. In subsequent PRs I plan to extend this so that it will benefit Hive tables where there is strict guarantee about single file per bucket.

hi @tejasapatil , I think your second proposal(always preserve bucketing across all operations) makes sense, but need more discussion to make the decision. For now, I think it's ok to use your current approach, e.g. only expose the outputOrdering if a bucket only has a single file, but we should avoid listing files twice.
We can add a lazy val in FileSourceScanExec, to store the files after partition prunning, and use it to calculate outputOrdering and inputRDD.
What do you think?

@cloud-fan : Sounds good to me.
I tried doing that but got a Task not serializable: java.io.NotSerializableException: org.apache.hadoop.fs.LocatedFileStatus. This is because the new lazy val introduced is a sequence of Partition and Partition comprises of FileStatus which is hadoop internal [0]. LocatedFileStatus is not serializable. Any suggestion on how to work around that ?
[0] : https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/fileSourceInterfaces.scala#L324

mark it as @transient?

@cloud-fan : Thanks !! Did the change.
Jenkins, test this please

Test build #64898 has started for PR 14864 at commit c60afd6.

Test build #64898 has finished for PR 14864 at commit c60afd6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64898/
Test PASSed.

Test build #64901 has started for PR 14864 at commit 568b742.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64901/
Test FAILed.

Test build #65136 has started for PR 14864 at commit 445549b.

Test build #65136 has finished for PR 14864 at commit 445549b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65136/
Test PASSed.

LGTM except some minor comments about tests, thanks for working on it!

Test build #65155 has started for PR 14864 at commit 7db6c10.

Test build #65159 has started for PR 14864 at commit 070c249.

Test build #65155 has finished for PR 14864 at commit 7db6c10.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65155/
Test PASSed.

Merged build finished. Test PASSed.

Test build #65159 has finished for PR 14864 at commit 070c249.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65159/
Test PASSed.

thanks, merging to master!

Was this patch merged in spark2.0.1 release? If so, how can it be enforced/turned on?

This is an optimization and we usually don't backport optimizations.


feat: Add a feature for sharing information with a WhatsApp account

This feature can be used by anyone to share information with a WhatsApp account.
I am working now on a basic feature that can demonstrate how it can be used by anyone.

Good work!
I'll probably decide to leave this PR opened until we'll have a way to integrate it into the frontend :)


Does this still work?

Been a while since any activity... I'm just wondering whether this library still works.

Yes, it does work. I am working on a new version with some changes, the most critical of which is adding Firebase compatibility. Currently if you have Firebase Crash Reporting in your app, CustomActivityOnCrash will not invoke Firebase's UncaughtExceptionHandler and you will get no crash reports in Firebase. That will be fixed on the next version.

Hi @AdityaAnand1 , the latest version is already released on Maven Central.

@Ereza Thanks for the reply! I'll go and check out the new version now :)


PHP 8.3 Deprecation Notice: trim(): Passing null to parameter #1 ($string)

Description
When using WPCS with PHP 8.3, the I18n and PrefixAllGlobals sniffs are causing deprecation notices due to trim() being called with null values.
Error Message
trim(): Passing null to parameter #1 ($string) of type string is deprecated
The error occurs in:

/WordPress/Sniffs/WP/I18nSniff.php on line 194
/WordPress/Sniffs/NamingConventions/PrefixAllGlobalsSniff.php on line 280

Environment

PHP Version: 8.3.11
WPCS Version: 2.3.0
Project: Working with _s (Underscores) WordPress starter theme

Impact
This affects all PHP files being checked by these sniffs, making it difficult to use WPCS with PHP 8.3. The error prevents proper code analysis as the checking is aborted when the deprecation notice is encountered.
Temporary Workaround
Currently using a local PHPCS configuration that excludes these sniffs to allow development to continue, but this means losing important WordPress-specific checks.
Suggested Solution
Update the relevant sniffs to ensure trim() is only called on string values, possibly by adding type checking before the trim() calls.

Closing as invalid. This was fixed years ago and included in the WPCS 3.0.0 release.


Support systemd and fix the build on Ubuntu 16.04

Call for volunteers.  Would someone like to test this branch?
$ git clone -b ubuntu-16.04 https://github.com/metalefty/X11RDP-o-Matic.git
$ cd X11RDP-o-Matic
# ./X11RDP-o-Matic.sh --justdoit --branch devel  # master branch will fail, not a o-Matic bug.

If you have time, results of with --nox11rdp are welcome.
# ./X11RDP-o-Matic.sh --justdoit --branch devel --nox11rdp

At least build should be succeed on:

Ubuntu 16.04, 15.10, 15.04, 14.04, 12.04
Debian GNU/Linux 8.0, 7.0

At least these two points should be checked:

build successfully finished with no errors
xrdp is running when o-Matic successfully finished

If no problem, I'll merge this and close #60.

On Ubuntu 16.04 failed, here is the output:
config.status: creating config_ac.h
config.status: executing depfiles commands
config.status: executing libtool commands
| dh_make --single --copyright apache --createorig
./X11rdp-o-matic.sh: line 520: cd: debian: No such file or directory

Run X11RDP-o-Matic.sh again, it will succeed.
But this is indeed a bug. I'll fix it later.

@nerdknight  Fixed. Deinstall dh-make package and could you try again?

xubuntu 16.04 - everything ok

It works ok now on Ubuntu 16.04

Thanks, still waiting for testing on older Ubuntu and Debian.

Works ok on Ubuntu 14.04

Works on Debian 8.4.

I'll merge develop into master in a few days.

This looks like it was merged back into the devel branch, but I still get same error on debian 8.7.  I clone the repo and ran "./X11RDP-o-matic.sh --justdoit --branch devel".  I suspect I'm doing something wrong though.  Any ideas?

run "git clone -b ubuntu-16.04 https://github.com/metalefty/XllRDP-o-Matic.git"
Cloning into "XllRDP-o-Matic'...
fatal: Remote branch ubuntu-16.04 not found in upstream origin
I still did not understand.
Is there a support for ubuntu-16 x64 or not?


KAFKA-7895: Fix stream-time reckoning for suppress

Add suppress to system tests
Move stream-time reckoning from Task into Processor

Even within a Task, different Processors have different perceptions
of time, due to record caching on stores and in suppression itself,
and in general, due to any processor logic that may hold onto
records arbitrarily and emit them later. Thanks to this, we can't rely
on the whole task existing in the same "instant" of stream-time. The
solution is for each processor node that cares about stream-time to
track it independently.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman @mjsax @bbejeck @guozhangwang ,
Please take a look at this fix for suppression.
As I mentioned in the description, it was incorrect for a processor to use the task's view of stream time, when a processor within a task may have an arbitrarily delayed local view of time, when it has an upstream record cache or suppression.
This was actually always true (since I introduced stream time into the context), but it didn't manifest as a bug until suppression, which depends very strictly on the stream time.
The correct thing to do is track observed stream time locally in each processor node. Since not every processor needs stream time, and since it was never in the ProcessorContext public API, I opted to remove it from the ProcessorContext and just explicitly implement it in the nodes that need it.
If we wish to expose "stream time" to processors in general, we can instead track stream time in ProcessorNode and then wrap the ProcessorContext to supply the local stream time when injecting the context to Processor.init. But I that can wait until the need arises. The current change is sufficient for now.
Ideally, we can merge this fix into 2.2 as well.

I guess it should be noted that the same condition could cause segments to be prematurely expired and records to be prematurely rejected from windowed aggregations due to grace period restrictions, if the windowed aggregation were downstream of a cache or suppression.
It just happens that suppression was the first thing that people noticed (probably due to the long default retentions and grace period).

Java 11 failure was a kafka admin test. Apparently, gradle crashed in the Java 8 build!
Retest this, please.

https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/2402/
java.lang.AssertionError: The consumer group deletion did not work as expected
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.assertTrue(Assert.java:42)
	at kafka.admin.DeleteConsumerGroupsTest.testDeleteCmdWithShortInitialization(DeleteConsumerGroupsTest.scala:211)

Java8 timed out (cf https://issues.apache.org/jira/browse/KAFKA-7933

Thanks for the review, @mjsax . I think I've addressed your comments.

I think that's a good idea, @bbejeck .
I'm thinking this could be an integration test, to avoid adding too much on to the smoke test, do you agree?

Failures unrelated:
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/19515/
kafka.api.ConsumerBounceTest.testRollingBrokerRestartsWithSmallerMaxGroupSizeConfigDisruptsBigGroup
org.apache.kafka.common.errors.GroupMaxSizeReachedException: Consumer group group2 already has the configured maximum number of members.

https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/2414/
kafka.api.PlaintextConsumerTest.testConsumeMessagesWithCreateTime

java.lang.AssertionError: Partition [__consumer_offsets,0] metadata not propagated after 15000 ms

Retest this, please,

Java 11 had one unrelated failure:

Java 11 had one unrelated failure:
https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/2424/
kafka.api.PlaintextConsumerTest.testMultiConsumerRoundRobinAssignment
java.lang.AssertionError: Did not get valid initial assignment for partitions [topic2-3, topic1-2, topic2-0, topic2-5, topic1-4, topic2-7, topic2-1, topic2-6, topic1-0, topic1-3, topic2-4, topic1-1, topic2-2]
	at kafka.utils.TestUtils$.fail(TestUtils.scala:356)
	at kafka.utils.TestUtils$.waitUntilTrue(TestUtils.scala:766)
	at kafka.api.PlaintextConsumerTest.validateGroupAssignment(PlaintextConsumerTest.scala:1772)
	at kafka.api.PlaintextConsumerTest.testMultiConsumerRoundRobinAssignment(PlaintextConsumerTest.scala:864)

That plaintext  consumer test was the only failure. I'm not sure if there's value in repeating the tests at this point.

https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/2424/
java.lang.AssertionError: Did not get valid initial assignment for partitions [topic2-3, topic1-2, topic2-0, topic2-5, topic1-4, topic2-7, topic2-1, topic2-6, topic1-0, topic1-3, topic2-4, topic1-1, topic2-2]
	at kafka.utils.TestUtils$.fail(TestUtils.scala:356)
	at kafka.utils.TestUtils$.waitUntilTrue(TestUtils.scala:766)
	at kafka.api.PlaintextConsumerTest.validateGroupAssignment(PlaintextConsumerTest.scala:1772)
	at kafka.api.PlaintextConsumerTest.testMultiConsumerRoundRobinAssignment(PlaintextConsumerTest.scala:864)

Retest this please.

Hey, it passed that time! Thanks, @mjsax .

I'm thinking this could be an integration test, to avoid adding too much on to the smoke test, do you agree?

Yes, I actually was thinking of an integration test and just wasn't being clear about it.  I

Actually, after wrestling with this for a bit, it's more complicated than I thought. What do you think about tackling that in a separate PR? I went ahead and added it to the acceptance criteria of KAFKA-7895. There's also a task there to add system test coverage with caching disabled, so this PR wouldn't have resolved the ticket anyway.

That's fine.

Looks good, thanks @vvcephei

Thanks, @bbejeck ,
Yes, for the processors that track stream time, the definition of time is essentially the same as it is for the record queue. It's always the maximum record timestamp ever observed, so, as you say, it will always increase or stay the same.
It used to be true that (when there's one or more upstream caches or suppressions in the same task), we'd always see records arriving from the past. E.g., the clock would say 10:00, but I'd be getting records from 09:55. Now, we've made stream time tighter, so the clock will say 09:55 when we're getting records for 09:55.
Another change is that, for the processors tracking stream time, time only advances for records that actually flow through the processor, so we won't see the clock continuing to "tick" as a result of records that actually get filtered out before they get to us. I think this is good for predictability. That is, it's a simpler mental model, so it's easier to predict how the windowed aggs and suppression will behave wrt stream time.

Thanks @mbragg02!

@vvcephei This PR has conflicts with 2.2 branch and I could not cherry-pick it cleanly. Can you please do a follow up PR against 2.2 for this? Thanks.

Merged to trunk.

Thanks, @mjsax ! I will do so now.


[Tests-Only] Add api trashbin tests

Description
This PR adds apiTrashbin bug verification scenarios.
The same as PR https://github.com/owncloud/ocis/pull/489 and PR https://github.com/owncloud/ocis-reva/pull/453
This enables future developers to be able to see the behavior in tests while they are understanding and fixing.

for some reasons some of these tests have been marked as passing in my new PR: https://github.com/cs3org/reva/pull/1125

for some reasons some of these tests have been reported unexpectedly passing in my new PR: #1125

This PR is adding local bug-demo scenarios. It does not adjust expected-failures from core.

@ishank011 @labkode these scenarios might work better or differently after #1125 - I will rebase this...

Local API tests: https://cloud.drone.io/cs3org/reva/2402/4/7
136 scenarios (136 passed)
1078 steps (1078 passed)
2m36.41s (17.07Mb)
runsh: Total 136 scenarios (136 passed, 0 failed)
runsh: Exit code of main run: 0
runsh: There were no unexpected failures.
runsh: There were no unexpected success.

oC10 core API tests:
runsh: Total 1112 scenarios (434 passed, 678 failed)
runsh: Exit code of main run: 1
runsh: Exit code after checking expected failures: 0
runsh: There were no unexpected failures.
runsh: There were no unexpected success.

Looks good.
@labkode @ishank011 please review-approve-merge.


Testcase for OpenPower Gard Interfaces.

This testcase just copies the fake.gard file from test_binaries
into the test machine and flashes into the Host PNOR.
Then it verifies this with opal-gard tool for list, show and clearing
gard records.
Signed-off-by: Pridhiviraj Paidipeddi<EMAIL_ADDRESS>

This change is

@stewart-ibm Can you please look into this one? This one is tested against AMI and SMC BMC systems, working fine. Need a test on OpenBMC systems.

@stewart-ibm Fixed the hard coding of GARD partition offset and added pflash tool check to be done before test starts. Please have a look at it. Thanks.

Thanks! Merged to master as of b0d147dbc548

Thanks @stewart-ibm .


feat: ADR-015: Improve chain link signature support

Description
Closes: #871

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch (see PR Targeting)
[x] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

@dadamu @bragaz If you can review this it would be great. I will take care of updating the docs once everything is fine for you

Awesome work! So this supports also the signatures made with Metamask right?

Yes, it does. It supports ETH personal_sign signature. The tests case present is actually testing against a signature generated with MetaMask


fix: minor fix for golangci-lint

fix make fmt under macos
add gofmt -s options for aligning golang-ci's

Signed-off-by: Ji Bin<EMAIL_ADDRESS>Summary

Does this close any open issues?
Not. Related: #2099
Screenshots
N/A
Other Information
N/A

Thank you so much Bin!


Unified cover image sizes to 1280x720 res
Changed all image format to .png
Changed readme files

Nice images. Thanks!


DOC: Manually updating the state causes a bad request error

Issue with current documentation:
In this section: https://langchain-ai.github.io/langgraph/tutorials/introduction/#what-if-you-want-to-overwrite-existing-messages
events = graph.stream(None, config)
for event in events:
    for value in event.values():
        if isinstance(value["messages"][-1], BaseMessage):
            print("Assistant:", value["messages"][-1].content)

The tutorial is using Claude 3 haiku but, using any of the claude 3 models, Claud 3 returns a bad request:
"name": "BadRequestError",
"message": "Error code: 400 - {'type': 'error', 'error': {'type': 'invalid_request_error', 'message': 'messages.2: Did not find 1 tool_result block(s) at the beginning of this message. Messages following tool_use blocks must begin with a matching number of tool_result blocks.'}}"
gpt-4 is working as expected though.
Idea or request for content:
gpt-4 is doing it correctly but is less verbose so the first part of the tutorial gives less interesting feedback so it should probably be updated so that the expectations of Claude are met.

Should be fixed by this update to serialization: https://github.com/langchain-ai/langchain/pull/21342

@TVScoundrel how did you solve it ?

i'd also know to fix it!


apply_neighborhood: What to return when an overlap is used?

Process ID: apply_neighborhood
Describe the issue:
What I’m not clear about when working with the process is: If you specify overlap, what is expected to be returned from the callback? the data with or without overlap?
Proposed solution:
Actually, how easy is it to detect what the overlap and what the actual data is? In the end, it's probably easier to return the data with overlap, but the back-end discards any changes to the overlap and only replaces the values for the "actual data" without overlap.
Additional context:
https://discuss.eodc.eu/t/moving-average-with-low-pass-filter/428/11

Dev Teclo: Return with overlap, back-ends handles the rest...


[8.0] Reset select bindings for count query

Fixes #1600

This one is breaking my table, see: https://github.com/yajra/laravel-datatables/issues/1983


About the moudle of TVM

When I set the value of use_tvm from False to True in line 270 of Single_step_main.py , Hierarchical_mm_tpm.py line 130 contains an error 'No module named 'tvm' '. When I tried to install the tvm package, it was not successfully installed. Are there any existing tvm packages available for windows system, and what should I do to solve this problem?

You can compile TVM on windows as described in the TVM documentation (https://tvm.apache.org/docs/install/from_source.html#building-on-windows). Note that this repo uses TVM 0.8.0.
If you feel too troubled to compile, as an alternative, you can find a compiled TVM docker image from https://tvm.apache.org/docs/install/docker.html#docker-source. Then delete files under 'pyraformer/lib' and run the code again.


feat: publish instanceType, capacityType, availabilityZone in a k8s event when we receive a IsUnfulfillableCapacity error while trying to create fleet

Fixes #
Related: https://github.com/aws/karpenter/pull/3955/files#r1217432191
Description
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

feat: Append k8s event for instanceType, capacityType and availabilityZone when we get an IsUnfulfillableCapacity error while launching Instance

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

Seeing this after the merge: what objects do we associate the event with?
I'd expect to report against the relevant Provisioner and, for AWS, any associated AWSNodeTemplate.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 7c61cd6, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 13530 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 648d1ff.


Forms: input has placeholder value color

Reproduce:

Add random item to cart
Start checkout to arrive at shipping step
Provide data

Issue: Note the color of the user's input has the same value as the color of the placeholder texts. This shouldn't be the case, actual input should display in a much more prominent manner. I recommend to use black here.
Global issue, applies to all input fields accross the site.

Text color is now more prominent in poster version 2.1.


help: Remove description of transition from self-hosted billing page.

Notes:

I also removed the server version banners from the new customers section, since new customers are expected to be on 8.0+ in any case.

Current: https://zulip.com/help/self-hosted-billing#upgrades-for-legacy-customers

Updated

This change is great; I'll have a few additional tweaks to suggest orally on top of this.


Update _content/hu from Lokalise

Merging this should launch Hungarian translation. Before deploying, we should check the preview deployment for the following things:

hungarian translation is offered as option
markdown structure matches English (e.g. header levels match etc)
images on covid-19 page are not broken and they are translated (this easily gets broken)

Latest preview available at: https://guide-preview-kjqnfwjzn.now.sh/hu/

language is offered in the menu
images on covid-19 are broken, they do not point to /hu/images (needs attention)
markdown structure not checked

There is one header problem in the keep-the-main-thing-the-main-thing section but other than that, structure looks good and images are correct as well.

apart from an open discussion regarding label on the COVID-19 image we are mostly good to launch this. The conversation about this happens on #translation-hungarian slack channel.

This is good to go.


Error on templates

I get a huge amount of errors on my website, since I upgraded from paypal 3.10.10 to 3.16.2.
They are all like :
ErrorException
htmlspecialchars(): charset `utf8' not supported, assuming utf-8

Please use the correct spelling in your templates !

This has been fixed here on this commit https://github.com/202-ecommerce/paypal/commit/fbce9bb06adbc09c3be2b8eaed288ecce125c60f


M3-1339 2FA QR code border invisible in dark mode

Adding a white border to QR code so it is scannable in dark theme. See ticket for details on reproducing.

@alioso updated to just be a one-off color! Thanks everyone for the reviews!


Rendering fails - 'Vertex shader failed to compile'

Installed the workshop to see if I can use it for teaching Cesium, but found this error when starting up from both npm and python:
An error occurred while rendering. Rendering has stopped.
RuntimeError: Vertex shader failed to compile.  Compile log: ERROR: 0:50: 'w' : vector field selection out of range

Error
    at new HS (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:77:4591)
    at $Se (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:896:122)
    at gQ (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:897:1409)
    at H3 (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:897:1361)
    at zm._bind (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:897:2210)
    at NQe (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:8352:41422)
    at Au.draw (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:8352:42192)
    at kv.execute (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:894:36194)
    at Ca (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:11738:17586)
    at UY (https://cesium.com/downloads/cesiumjs/releases/1.103/Build/Cesium/Cesium.js:11738:22446)

Console errors also included.
console error.txt
I am on Kubuntu 22.04. I tried installing WebGL drivers to fix this, but it didn't (and Cesium works in general for me as well, so it's not something across every Cesium application)

I've figured out that if you rename the CesiumDrone.gltf file, this error doesn't happen. I suspect something is wrong with the model provided.


prediction package

https://bbolker.r-universe.dev/prediction

unclear if we should address prediction in this guide


F-Friend

Description
In this folder I want to discuss about friend function and friend class in C++.
Domain
CPP
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


New pattern: aws-applicationloadbalancer-lambda

Alternative to using API gateway in front of lambda
Use Case

Proposed Solution

Other

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

@hnishar I can help with this pattern, you could assign this to me

thanks @hnishar, can we rename the construct to aws-applicationloadbalancer-lambda? Makes it inline with AWS Application Load Balancer service name.

Would it make sense to call this aws-alb-lambda just like aws-sns-lambda, aws-sqs-lambda?


Convergence issue when importing torch on linux

Hi, I recently noticed that when I run a simple dft calculation on linux and import torch into the same script, the dft fails to convergence. Upon removal of the torch import,
the code worked as expected.
Has anyone else also encounter this?

Probably related to this issue https://github.com/pytorch/pytorch/issues/109446

Hi @mforstenhaeusler , thanks for posting this issue. Has it been resolved?


Can't upload files

Hello,
I am not being able to upload files, I can create folders but when i try to upload a file it just shows uploading and doesn't upload.

can you share any error that you are facing? please check the network tab on chrome console.  and check every request is resolved or not. if any error then share with me.


launchpad with price discovery

comparable to https://app.streamswap.io/

idea: a platform co-op to promote cooperation with less vulnerability using capability security
Is the advogato trust metric consistent with #2 Democratic Member Control?
perhaps start with a "fair launch" and then exit to community?