Fatal: requested timeline 8 is not a child of this server's history

Did some startup and shutdown checks, eventually landed in the following state
LOG:  database system was shut down in recovery at 2015-08-27 11:58:44 CEST
WARNING:  recovery command file "recovery.conf" specified neither primary_conninfo nor restore_command
HINT:  The database server will regularly poll the pg_xlog subdirectory to check for files placed there.
LOG:  entering standby mode
FATAL:  requested timeline 8 is not a child of this server's history
DETAIL:  Latest checkpoint is at 0/15000028 on timeline 7, but in the history of the requested timeline, the server forked off from that timeline at 0/14000198.
LOG:  startup process (PID 2147) exited with exit code 1
LOG:  aborting startup due to startup process failure

Can we somehow come back from this?

Ok removed the files from pg_xlog/, base/ and global/. Governor is now trying to start again.
LOG:  database system was shut down in recovery at 2015-08-27 12:21:31 CEST
WARNING:  recovery command file "recovery.conf" specified neither primary_conninfo nor restore_command
HINT:  The database server will regularly poll the pg_xlog subdirectory to check for files placed there.
LOG:  entering standby mode
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
2015-08-27 12:24:18,544 ERROR: Error communicating with Postgresql.  Will try again.
2015-08-27 12:24:18,545 INFO: None
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
FATAL:  the database system is starting up
Traceback (most recent call last):
  File "./governor.py", line 64, in <module>
    if postgresql.is_leader():
  File "/var/lib/postgresql/governor/helpers/postgresql.py", line 84, in is_leader
    return not self.query("SELECT pg_is_in_recovery();").fetchone()[0]
  File "/var/lib/postgresql/governor/helpers/postgresql.py", line 49, in query
    raise e
psycopg2.OperationalError: FATAL:  the database system is starting up

LOG:  received fast shutdown request
waiting for server to shut down....LOG:  shutting down
LOG:  database system is shut down
 done
server stopped

Ok bacially I did a clean restart by cleaning up as you recommended [https://github.com/compose/governor/issues/13#issuecomment-107049640] (here).
But now I get:
2015-08-27 13:17:39,940 INFO: Lock owner: sql2; I am sql3
2015-08-27 13:17:39,941 INFO: does not have lock
2015-08-27 13:17:39,946 INFO: no action.  i am a secondary and i am following a leader
FATAL:  database system identifier differs between the primary and standby
DETAIL:  The primary's identifier is<PHONE_NUMBER>339472088, the standby's identifier is<PHONE_NUMBER>791371405.
FATAL:  database system identifier differs between the primary and standby

This happens when you do a pg_basebackup from a different Postgres cluster than you are trying to stream replication from.  This can also happen if you run initdb for the member instead of running pg_basebackup from the cluster's leader.
I'm assuming you ran initdb for this Postgres directory.  If you did, remove the data directory for that Postgres, and let governor run initialize the empty directory from the cluster's leader.  Basically, you should start governor with Postgres uninitialized.


Fix attribute merging on Chef 12

As of Chef 12, node.force_override does not for allow values to be set, or for it to be used as a target for a merge. Setting with node.force_override! appears to work.

@andytson I've run all the kitchen suites on this branch without errors.


[Polymorphic] Improve performance

It's really hard to know for sure if tweaks to polymorphic types are improving performance but I picked a huge component (ScrollArea) and recorded a video of me tweaking its types (with and without these changes) to show how TS is able to more quickly report issues now.
The main change here is to remove the extends keyof JSX.IntrinsicElements from the generic which would have been slowing down all components regardless of whether the as prop was being used.
Instead, I check to see if as extends an empty string and if so, I type it as keyof JSX.IntrinsicElements to enable intellisense when used.
INTELLISENSE
https://user-images.githubusercontent.com/175330/125993174-b89e107b-4caa-4485-af6a-55c18b1e9347.mp4
BEFORE (~8s)
https://user-images.githubusercontent.com/175330/125992071-b1aa176b-6e08-4717-9b84-d6bb2ec77b22.mp4
AFTER (~5s)
https://user-images.githubusercontent.com/175330/125992101-02282990-5ddf-470c-b721-6421b83c7a77.mp4
I'll continue to think of ways to improve it further but this definitely feels better.

Ran a quick test with --diagnostics and looks like this change is more than halving the check time 😮

Ran a quick test with --diagnostics and looks like this change is more than halving the check time

Never used that before, TIL 😅


if exceeds MaxSPDMmsgSize then shall respond with ErrorCode == ResponseTooLarge or silently discard the request.

If the size of a response message exceeds the size of the MaxSPDMmsgSize of the requesting SPDM endpoint, the responding
SPDM endpoint shall respond with ErrorCode == ResponseTooLarge or silently discard the request.

Refer to paragraph 239  in DSP0274_1.2.1.

If the size of a response message exceeds the size of the `MaxSPDMmsgSize` of the requesting SPDM endpoint, the responding
SPDM endpoint shall respond with `ErrorCode == ResponseTooLarge` or silently discard the request.

Refer to paragraph 450 Table 48 — Error code and error data in DSP0274_1.2.1.

I think:

a. the code at following line 622 :

if (my_response_size < large_buffer_size) {
may should be like:
if (my_response_size <= large_buffer_size) {.

b. the code at following line 665~666:

status = libspdm_generate_error_response(context,
                                         SPDM_ERROR_CODE_LARGE_RESPONSE,
may should be like:
status = libspdm_generate_extended_error_response(context,
                                                  SPDM_ERROR_CODE_RESPONSE_TOO_LARGE,
https://github.com/DMTF/libspdm/blob/0af7c1197cab8e99615fd9f062653aaf32552c03/library/spdm_responder_lib/libspdm_rsp_receive_send.c#L622-L669

large_buffer_size is the size of the Responder's scratch buffer, no? In that case this is an internal error to the Responder and it should be Unspecified.
I think libspdm_build_response needs a major refactoring. It's difficult to figure out what's going on in it.

According to our definition:
#if LIBSPDM_ENABLE_CAPABILITY_CHUNK_CAP
...
#define LIBSPDM_SCRATCH_BUFFER_LARGE_MESSAGE_CAPACITY (LIBSPDM_MAX_SPDM_MSG_SIZE)

large_buffer_size is LIBSPDM_MAX_SPDM_MSG_SIZE.
As such, I think we need to return ReponseTooLarge.

ResponseTooLarge
The response is greater than the MaxSPDMmsgSize of the requesting SPDM endpoint.

In this case the response message is too large for Responder's internal buffer, not the Requester's. There is no error for that other than Unspecified.

ResponseTooLarge
The response is greater than the MaxSPDMmsgSize of the requesting SPDM endpoint.

In this case the response message is too large for Responder's internal buffer, not the Requester's. There is no error for that other than Unspecified.

This is internal buffer for LIBSPDM_MAX_SPDM_MSG_SIZE. Why we cannot return ReponseTooLarge  ?
Or if that is the case, then when we should return ReponseTooLarge ?

agree

other related issue: https://github.com/DMTF/libspdm/issues/1865

Current code does not return ReponseTooLarge at all.

Well that's a problem!
The checking priority should be

Responder checks if the response is larger than the Requester's MaxSPDMmsgSize. If so then send ReponseTooLarge. (It looks like this check is missing.)
Responder checks if the response is larger than the Responder's internal scratch buffer. If so then send Unspecified.

Yes. We need add check for Requester's MaxSPDMmsgSize. And I have submit PR.
For  Responder's internal scratch buffer, the check may don't need, because The responder will acquire sender_buffer size before build response.
https://github.com/DMTF/libspdm/blob/d7700798944f077bbd502710f222ce7580360c15/library/spdm_responder_lib/libspdm_rsp_communication.c#L65-L79
The libspdm_acquire_sender_buffer will return the max response size the when chunk is enabled.
    *max_msg_size = libspdm_get_scratch_buffer_large_sender_receiver_capacity(spdm_context);
So, the build response size will not be larger than the Responder's internal scratch buffer.


Support for Intellij Community Edition

Hello @linux-china ,
Came across this plugin, looks promising and great.
However can this plugin be supported for IntelliJ community edition?
looks like currently its only available in Ultimate edition.

@anantagarwal  httpx plugin is based on https://www.jetbrains.com/help/idea/http-client-in-product-code-editor.html and HTTP Client is an Ultimate feature.


GKE/AKS Windows compatability

See if:
a) we can easily create clusters with Windows nodes
b) if your features deploy correctly on them. I am assuming there will be problems with taints and tolerations.

Additional fields were needed in the spec in order to create Windows nodes https://github.com/rancher/aks-operator/pull/5
Besides that, AKS requires at least one node pool to be a "System" type and "System" node pools are Linux-only, similar to the way GKE requires at least one non-windows node pool. So there's not an issue with taints, and actually none of the nodes in any node pools seem to have any taints. The cluster agent gets deployed successfully and the cluster explorer comes up fine.

tested on aks operator rc4

See if:
a) we can easily create clusters with Windows nodes
Works
b) if your features deploy correctly on them. I am assuming there will be problems with taints and tolerations.

monitoring v2 installs fine. However, when I go to Grafana, the monitoring data is populated only for linux nodes (not windows nodes)

Update: I haven't entirely gotten it to work yet, but I think it /should/ work without needing any changes in aks-operator.
What I've done so far:

followed this guide to set up an RDP connection to the windows nodes in the cluster (the az commands did not work, but you can reproduce most of it via the web UI).
installed wins on the nodes.
installed the monitoring chart through Rancher, setting global.windows.enabled=true in values.yaml. For some reason this never seems to take on the first install, I have to go upgrade it and change it again.
Eventually you should see a healthy rancher-monitoring-windows-exporter daemonset running on the nodes using the kubernetes.io/os=windows selector.

At that point things seem to look fine but there are still no metrics in grafana for those nodes. In the Prometheus "Targets" UI I see the error "Get "http://<IP_ADDRESS>:9796/metrics": context deadline exceeded" for the cattle-monitoring-system/rancher-monitoring-windows-exporter/0 (0/2 up) target, so I think it's something wrong with the VNet or something wrong with how I'm running wins.exe.

Hmm. We should talk about this on Monday. Don't think you need to put any more work on it for now. It's mostly a question of what Rancher typically does for windows nodes, and how to align expectations with that. But that last part about VNet we may need to dive into at some point.

@prabalsharma can you test fleet and logging on mixed node GKE and AKS clusters, we think those will work by default, though you might have to enable the global.cattle.windows.enabled flag.
Monitoring I don't think will work on either. But can you also test that and open up a bug issue if it doesn't work. We can use this issue solely for whats going live in 2.6.

For posterity, one other thing I tried for monitoring was to add an inbound and outbound rule for port 9796 to the security group for the vnet the node pool is in. Should also note that that IP address <IP_ADDRESS> is the IP address of the windows exporter pod and is in the same subnet as the node range.

https://github.com/rancher/aks-operator/pull/17 and https://github.com/rancher/rancher/pull/32869 are merged so it should now not be possible to create node pools with os type "Windows".

tested on master-3d465ee
Added to https://github.com/rancher/rancher/issues/33183


Add Documentation for new developers

We have been doing it in https://github.com/serlo/documentation/wiki
Please open a new issue there if you have a specific request, or even better create a card in our product board


Is it possible to utilize ROCM to run this?

This isn't an issue, more-so a question. But I haven't had much luck running an instance of dreambooth with rocm

This Colab only supports the hardware offered by Google Colab


Dockerize the application

To have a deployment ready application, it would be great to have it Dockerized. To do so, we should have, basically 5 things:

Create a Dockerfile
A docker-compose.yml with build directive to run locally
README with instructions on how to ruin and build locally
A configured CI that will be triggered to build the image in DockerHub as new releases are created here in the repo
README with a docker-compose.yml instructions (including the use of stop-analyzing-api)

For this issue, we are building the first 3 items. Is that ok?

@tiagostutz  can I try this?

Sure, @giteshk11 !
Just assigned to you. If you need any help, let me know.
Just a suggestion, there's this light web server that you could consider: https://github.com/lwsjs/lws/, but feel free to bring your preferences.

@tiagostutz just a quick question, do we need any OS as a base in the container?

In most cases we need an OS, prefering the smaller ones like alpine or debian stretch-slim, but if you have any option that brings up a web server from scratch, that's ok also. Also, you could search for a ready-to-go Docker imagem with a web server.
What web server are you planning to use?

I'm thinking of Nginx or the link you shared before we can use that too.

Well, I think we can use this opportunity to increment the open source experience a little bit. What about opening a issue on https://github.com/lwsjs/local-web-server asking whether there is an official Docker image and offering to build it if there isn't one? What do you think?

Sounds good to me.

Hi @tiagostutz, I'm having some issues with docker on my machine and I'm not able to figure out. Can you assign this to someone else?

Sure! I'll create another issue and assign it to you.

Hey guys, I would like to try. Can I work on this?
:grin:

You got this, @lucasravy .!
Wanna try to open the issue in the local-web-server project?

@tiagostutz nice! Thanks for the challenge.
I'm working on it right now. As soon as I get the issue's number from local-web-server repository, I'll let you know here.
:nerd_face: :computer:

The issue has just been opened!
Quick question: If the official image docker proposal don't work, we're gonna create our own using the same solution, right? So: alpine as S.O base + local-web-server as web server. Is it good for a start?

The issue has just been opened!
Quick question: If the official image docker proposal don't work, we're gonna create our own using the same solution, right? So: alpine as S.O base + local-web-server as web server. Is it good for a start?

If it doesn't work, I suggest you create the image on your own user and then use it to Dockerize this application here. Sounds good?

Hi @lucasravy !
I think we waited enough time for the reply of local-web-server maintainers, right?
What do you think of opening a repo on your own GitHub user for this Docker image with a great README, publishig it to https://hub.docker.com and referencing it in the issue you opened?
Tip for the README: https://codarium.substack.com/p/my-secret-sauce-to-make-github-repositories

Hey @tiagostutz!
I've already done that (except the great README tips, thanks btw :grinning:) and I'm testing the image on our project. I'm having some troubles with the image size but I think I have the solution and I'll be able to open a PR in the next couple of days.

Fantastic, @lucasravy !
Can't wait to check it out. 🤓


Fix empty delimiter

Even if the delimiter is empty, the min length should be 1. This works on OSX and BSD.
Fixes #558

Works on my FreeBSD system currently.

Thanks for the quick bug fix, @dritter, and thanks for the review, @docwhat.
@awensaunders - Happy to hear it! Once we get the review comments sorted out, we'll proceed :)

Thanks for the review responses, @dritter! All looks well =)


Code License

Hi @jafermarq,
unrelated to the other issue I filed: Would you mind specifying a license for this repo so that I know to what extent I can experiment/work with your examples?

Hi @lbhm ! Many thanks for asking about this. I just added a MIT license!
Also, let me know if you are particularly interested in new examples. And see you in the next Flower Monthly!


Feature/select add items

What's preventing you from merging this? I'd love to use Choices, but I need to be able to add options to a select field or provide choices for a text input.

This is on hold for the time being - it was initially merged into a release but caused an issue where you couldn't select more than one option on a select input. It needs investigating.

Is that release still tagged? That's what I need it for on this project, so if that's the only issue, it'd work for me for now.

@curtisblackwell - https://github.com/jshjohnson/Choices/releases/tag/v2.8.0

Awesome, thank you. I'll give this a go today/Wednesday.

@jshjohnson that means that the v2.8.0 supports addItem again?

@dgt41 Yes. I used 2.8.0 and it was exactly what I needed.

@curtisblackwell thanks for the update! I guess I'll have to reopen this then 😀

Closing this - the codebase has moved on a lot since this was opened causing major conflicts.

@jshjohnson Do you have an idea/plan (how) to make this possible? I exactly need a combination of both, being able to search for already existing entries, but also adding custom items to a single or multi select field.
Any help is appreciated!


[bug] Android dev build error - Activity class does not exist

Describe the bug
I'm trying to integrate Android in my tauri existing app, but I've reproduced the issue with a clean project.
When running npm run tauri android dev at the end of the run process I'm getting this error:
Performing Streamed Install
Success
Starting: Intent { cmp=com.tauri_app.app/.MainActivity }
Error type 3
Error: Activity class {com.tauri_app.app/com.tauri_app.app.MainActivity} does not exist.

Reproduction
Run this command:  to create a new project:  irm https://create.tauri.app/ps | iex and select these options:

Project name: tauri-app
Identifier: com.tauri-app.app
Typescript / Javascript
npm
React (or another platform)
Typescript

Then run these commands:

cd tauri-app
npm install
npm run tauri android init
npm run tauri dev

At the end of the build you'll get the error
...
Performing Streamed Install
Success
Starting: Intent { cmp=com.tauri_app.app/.MainActivity }
Error type 3
Error: Activity class {com.tauri_app.app/com.tauri_app.app.MainActivity} does not exist.

Expected behavior
The apps installs and starts on the emulator
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.26100 x86_64 (X64)
    ✔ WebView2: 131.0.2903.99
    ✔ MSVC:
        - Visual Studio Build Tools 2022
        - Visual Studio Community 2022
    ✔ rustc: 1.82.0 (f6e511eec 2024-10-15)
    ✔ cargo: 1.82.0 (8f40fc59f 2024-08-21)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 18.20.3
    - npm: 10.7.0

[-] Packages
    - tauri 🦀: 2.1.1
    - tauri-build 🦀: 2.0.3
    - wry 🦀: 0.47.2
    - tao 🦀: 0.30.8
    - @tauri-apps/api : 2.1.1
    - @tauri-apps/cli : 2.1.0

[-] Plugins

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
JAVA
openjdk version "21.0.5" 2024-10-15 LTS
OpenJDK Runtime Environment Temurin-21.0.5+11 (build 21.0.5+11-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.5+11 (build 21.0.5+11-LTS, mixed mode, sharing)
Android

SDK: Build-Tools 36-rc-1
NDK: 28.0.126474087

Is it the java version? This is my version that can run.

I've tested and it's owrking. Probably there was also an issue with the android emulator


OCM-4459 | feature: CRUD for external auth providers

https://issues.redhat.com/browse/OCM-4459

nitpick:
Change the dir names "externalauthproviders" to singular form to match command "externalauthprovider"

/lgtm

@chenz4027 Just a minor comment from me on moving a final command over to the runWithRuntime pattern. It also needs a rebase by the looks of things.
Great work 🚀
cheers, Rob

@robpblake Refactored and rebased, could you take another look 🤠


[hotfix][scala] Let type analysis work on some Java types

This is mostly to allow Vertex, which is a Java Tuple2<> to be used in
the Scala Gelly API.

Seems to contain a second fix, to skip static fields in data types.

Does the Scala API not treat Java classes the same way as its own classes?
That sounds absolutely like the right thing to do. Why not do as much compile time analysis as possible...
To understand this better, does the Scala type analysis now generate

Java Tuple TypeInformation?
Does it treat Java POJOs as POJO types?
...

It does not treat Java classes the same. For some strange reason, the Scala Type analysis in Scala 2.10 can sometimes not "see" the fields of classes defined in Java.
The Scala Type analysis treats Java Tuples as what they are, POJOs. We could maybe add special case handling to make it treat Java tuples as Java tuples.

How hard is it to add Java Tuple support to the Scala API? May be also interesting for people that want to use mutable tuples in Scala...

See here: https://github.com/apache/flink/pull/669


ediscovery/content search is able to get edited messages without a hold, not just the last edited message

Hi,
the statement:
Edited messages | If the user is on hold, previous versions of edited messages are also preserved.
is not too accurate as when performing a Content Search without a Hold, all the edited messages in a channel/chat are able to be found , and not just the last edited message.
feedback is to provide better written clarity in this doc, of what is able to be discovered in Content Search without the Hold for edited/deleted chat/channel messages.

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

ID: 840dd36b-8c29-9738-4adc-ea9a7d277fd0
Version Independent ID: b12c43c3-ed81-ace5-dfeb-77cac0f97fe9
Content: Conduct an eDiscovery investigation of content - Microsoft Teams
Content Source: Teams/eDiscovery-investigation.md
Service: msteams
GitHub Login: @v-tophillips
Microsoft Alias: v-tophillips

Hi @Teamsgineer, thank you for your feedback.
When you edit or delete a message in Teams and there is no retention policy to preserve it, it is permanently deleted typically between 1-7 days, not immediately. That's why you can find it using eDiscovery. But after 7 days it's permanently deleted.
For more information:
https://learn.microsoft.com/microsoft-365/compliance/retention-policies-teams


script to generate strong password

Description
This PR includes py script to gen password with python script.
Fixes #(issue_no)
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]

[ ] Bug fix (non-breaking change which fixes an issue)

[X] New feature (non-breaking change which adds functionality)

[X] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[X] I have named my files and folder, according to this project's guidelines.
[X] My code follows the style guidelines of this project.
[X] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[X] I have commented on my code, particularly in hard-to-understand areas.
[X] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[ ] I have included a requirements.txt file (if external libraries are required.)
[X] My changes do not produce any warnings.
[X] I have added a working sample/screenshot of the script.

We already have a similar script in our sister repo python-mini-projects


Find weekend days from number of years

Description
Weekend means Saturday & Sunday together. In total, we have 52 weeks in a year. So there are 52 weekends in a year. So, in normal we have 104 Weekend Days.
Input  : 1
Output : 104

Program added successfully :tada:
Thanks for your contribution :hugs:


Update to Space Dock

So what's wrong?

I know this one.  Because Olympic1 just told me.  File extension needs to change to .netkan from .kerbalstuff

Fixed in #3156


The grid moves up and down

When the browser does not display a vertical scrollbar the grid goes into an infinite loop of moving up and down.
Details:
The grid has the auto-resize directive set.
The grid has the row-select directive set.
The grid has both a header and footer.
Grid+Page Content does not fill the page.
When the browser window is resized the grid resize event fires.
If the new grid size is smaller than the new window size, but the grid is temporarily larger than the              window size, the browser resize bar triggers and infinite loop of resize events.
Forcing the vertical scrollbar to appear on the page at all times is a work-around for preventing this issue from occurring.
This issue reproducible in IE 9 and Chrome Version 39.0.2171.71 m.  It does not seem to be an issue in later versions of IE.

I've also seen this intermittently in IE 11 and it seems to be fairly consistent in Chrome. I think it plays into how chrome and IE handle the dynamic scroll bar, the only place I haven't been able to reproduce the error is in Firefox. The issue may also have something to do with having multiple elements on a page that can be expanded/collapsed dynamically which would also affect the dynamic scroll bar.

Clarification: IE11 only causes the dancing grid resize issue when it is set to IE9 or lower mode.

I'm hoping that the resizing changes I've done may fix this, although it's hard to say as I don't have a reliable recreate.  Could someone create a plunker showing the problem (then I'll work to fix it), or alternatively confirm that it's no longer occurring?

Here's a plunker demonstrating the issue. If you click the button a few times you'll see it.
This happens to me in IE11, Chrome, and FF.

This seems to be fixed in the current version.


Update and align Python with TS

Closes #329
Changes made

Implemented ConfigError and replaced usage of error_message
Adjusted EvalCommand to use the new ConfigSchema, fixed _eval_previous and pipe
Adjusted ExportCommand param for pipe
Implemented pipe functionality in UpsertCommand
Implemented the new ConfigSchema
Aligned ConfigSet with TS
Installed jsonschema package
Fixed is_valid_name regex and added is_valid_json_schema
Fixed tests to use the new ConfigSchema

@jamesL92 I would greatly appreciate it if you could conduct a CR


Adding Pytest framework and unit tests to dss-sdk. Converted dss_target.py to be python3 compatible

Code coverage is about 19 % for repository based off of this PR


Redirect stdin to the stubbed command to allow users to verify stub inputs

This PR adds a feature that I needed for my tests.
Basically, we save the content of stdin early in the binstub process to be able to redirect it to the stub command. This way, we can verify what input was given to our stub.
STDIN is only read if it is not a tty, or test will wait forever for some user input.

Hiya! I totally missed this.

Thanks for the contribution!


[Docs] Link not working

Current State
Link navigating to Meshery UI docs not working in Contributing to Meshery UI - Notification Center page https://docs.meshery.io/project/contributing/contributing-ui-notification-center#contributing-to-meshery-ui-notification-center

Desired State
Contribution to Meshery UI text should navigate to that page (i.e. https://docs.meshery.io/project/contributing/contributing-ui) on clicking.

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Muhammed-Moinuddin Thank you for your Cotribution.
Let's discuss this in Meshery Dev meeting. Adding this as an agenda item....
Meeting minutes: https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit?pli=1#
Meeting link: https://meet.layer5.io/meshery

@Muhammed-Moinuddin can you assign this issue to me?

@ashish0kumar Thanks for your initiative! Also, please note that we move in chronological order with issue assignments. Kindly wait to be assigned before working on an issue next time. We'll review your PR shortly. 🚀

Hey @Muhammed-Moinuddin, i can see no one is currently assigned on this issue. I would like to work on it, could you please assign it to me?

Hey @Muhammed-Moinuddin, i can see no one is currently assigned on this issue. I would like to work on it, could you please assign it to me?

It's just a small thing. Kindly do it ASAP. If u need any help do let me know...

Hey @Muhammed-Moinuddin, I have raised a PR.  Kindly have a look at it.


Remove duplicate jobs from github workflow

We now run pre-commit job in prow which covers most of go-lang tasks.
This also adds a github action that checks for go.mod replace, which was added to several operator repos.

/test glance-operator-build-deploy-tempest


Different numbers of ranking groups between aggregate and subgroup pattern for computing ranking trend distance

For example, in house election data, the aggregate pattern in a state always has all parties, but in different districts the number of party varies.

this is related to #183


[5.8] Return fake objects from facades

For consistency in facade fakes I have added the fakes a return type. That way I can create a property for the fake in the test's setUp method.
Current
public function setUp()
{
    parent::setUp();

    Bus::fake();
    $this->eventBus = $this->app->make(Dispatcher::class);
}

New
public function setUp()
{
    parent::setUp();

    $this->eventBus = Bus::fake();
}

All return doc blocks should be fully qualified to match the rest of the framework.


FindFirstFile fails to list a directory if the parent is not accessible

Describe the bug
FindFirstFile will not be able to list a directory if the parent directory cannot be opened. For example, on Android we can redirect an SD card (the path will vary but in my case is /storage/emulated/0). FindFirstFile tries to call opendir on the parent directory (in this case /storage/emulated) which is not accessible and causes the file listing to fail. Mapping a subdirectory (for example /storage/emulated/0/Download) does work successfully.
To Reproduce
Redirect a drive mapping a directory whose parent directory is not accessible.
Expected behavior
We should be able to list the directory.
Application details

FreeRDP version: master
OS version connecting to (server side): Windows 11

Environment (please complete the following information):

OS: Android
Version/Distribution: Android 13
Architecture: arm64

The issue was introduced in #9197, however that corrected an earlier bug that prevented this scenario working (trying to find files by the "." pattern, where "." and ".." files don't exist on Android external storage).

While this behaviour seems to be on-spec for FindFirstFile:

If the string ends with a wildcard, period (.), or directory name, the user must have access permissions to the root and all subdirectories on the path.

However, using this in the drive redirection does break certain scenarios (like the original issue(

@thenextman does it work with redirecting /storage/emulated/0/. for android? since you have access to /storage/emulated/0 the opendir should work and the fix would be contained to the android client components

@akallabeth My colleague tells me that does not work, I will need to debug and see exactly why not. But it's important to note that the external storage doesn't seem to use "." and ".." files...


[docs] Split changelog into current and old

Size of CHANGELOG.md is now 262kB (was 820kB).
We discussed splitting the changelog into major versions but I don't think we need to do this since the rationale for reducing the size of the changelog doesn't apply:

The old changelog is rarely edited (if at all)
The old changelog isn't published.

@mnajdova We may want to do this before https://github.com/mui-org/material-ui/pull/27938 to allow feedback for the reduced changelog. I think we can fast-track this change since we can always split the old one later if we disagree. The old one isn't published manually so any change that is merged, is immediately visible to everybody.

@mnajdova We may want to do this before #27938 to allow feedback for the reduced changelog.

Sure why not, I anyway plan to do the release later today, so no hurry from my side.

Should we update https://next.material-ui.com/discover-more/changelog/#main-content? Not sure as the old Changelog is already linked in the current.
I am 👍 for the changes.


After selecting a player, highlight the [field] position that's been in the longest

Actually, leaving the field-position with the longest shift in orange all the time now as of https://github.com/jmarantz/soccersub/commit/426bd384660d8364f6ecbe03b07a5c8425fee76f


Flag is not present when showDialCode

When adding property showDialCode in InputOptions the flag disappears when is in the initial state. If I insert 2 numbers in the input field then the flag appears.  I met this problem for version 5.1.0, I downgraded it to v4.4.2 and it works fine with the same code.
inputOptions: {
showDialCode: true
}

@valeriu9 Can you update to the latest release to see if it still happens? If it does, please re-open the issue.

Similar problem, when I change the route in nuxt. The flag appears only after refreshing the page. 😕 @iamstevendao
Using: Nuxt 2, vue-tel-input ^5.11.0


Fix shift graphql to use post data

See https://github.com/silverstripe/silverstripe-graphql/issues/48 for more details

some linting issues


Remove outdated MD5 files

Previously os.remove was called before HumbleHash.remove_md5file.
However, remove_md5file does nothing if the file (full_filename) does not exist:
def remove_md5file(full_filename):
...
        if full_filename is None or not os.path.exists(full_filename):
            return

So the md5 file was never actually removed and the call to remove_md5file did nothing.
This lead to the problem that corrupt files were always re-downloaded again and again without updating the corresponding md5 file. (The md5 file was never deleted and it is assumed that existing md5 files are correct and up-to-date.)
With this pull request, the md5 file is actually deleted (and recreated when necessary).
This way there are no more out-of-date md5 files.

Pull Request Test Coverage Report for Build 40

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 40.682%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

humble_download.py
0
1
0.0%

Totals

Change from base Build 11:
0.0%

Covered Lines:
334

Relevant Lines:
821

💛  - Coveralls


Elements in args-list does not update when restarting debug session

Issue Type: Bug

Create a launch.js for a node.js script.

Run the debug-session, let VS Code stop at the breakpoint.
Now, make a change to the update the args-list:
"args": ["--update", "someOtherFile.json", "filename2.json"]
Restart the debug-session (Ctrl+Shift+F5)
The script still uses the old parameter-list.

The issue is that the js-debug supportsRestartRequest. So we let the js-debug handle the restart. Even if there are changes in the configuration.
@connor4312 @weinand I am not sure what is the right behaviour here.
If a change happened to the launch configuration should VS Code disconnect and start the debug session again ignoring the supportsRestartRequest? Or should js-debug handle configuration changes as part of its restart?
Let me know what you think.

Today the spec of the DAP restart request doesn't say anything about changed launch configurations.
But we could add an optional argument to the restart request so that an implementation has easy access to it.
I suggest that we create a DAP feature request in the CAP's repo.
@connor4312 @isidorn what do you think?

@weinand this sounds good to me
Let me know if you would like me to create a DAP feature request in the DAP repo.
Also let's see what @connor4312 thinks about this

That sounds good to me

@isidorn could you please create a DAP feature request in https://github.com/Microsoft/debug-adapter-protocol/issues

@isidorn Now the restart request takes an argument for the "launch" or "attach" configuration.
Please update VS Code accordingly.

@weinand great, I can look into this.

I have pushed a change such that VS Code will send the configuration as part of the restart request. I send a resolved configuration, not the exact one that is in the launch configuration.
I can easily change that I send the exact one in launch.json.
Here's an example of what I would send
Assigning this item to you @connor4312 so you can look into adopting it on the js-debug side.
Also let me know if you would like me to change something on the client side. Thanks

@isidorn yes, sending the resolved configuration is correct (since that is the same what a DA sees in the "launch" or "attach" requests when it does not implement DAP's "restart" request).
(and it goes without saying that the resolved configuration must be based on the most recent launch config on disk...)

It looks like for js-debug, restart is always being dispatched to the child session, so there's no configuration to re-read. Should restarting via the toolbar go to the root session like stop/detach does?

@connor4312 yes this makes sense, I have pushed a change to tackle this.
So now restart, stop and disconnect are aligned, they always send to the parent session.

Ok, great. I'll tackle this next iteration.

Following the steps from the description, I still see the old parameters after the restart.

index.js: console.log(JSON.stringify(process.argv));
launch config

{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "pwa-node",
            "request": "launch",
            "name": "Launch Program",
            "args": ["--update", "someFile.json", "filename2.json"],
            "skipFiles": [
                "<node_internals>/**"
            ],
            "program": "${workspaceFolder}\\index.js"
        }
    ]
}

set breakpoint in index.js, hit the breakpoint, change the args, restart (Ctrl+Shift+F5), finish the program, the first args are printed.

It looks like ctrl+shift+f5 sends the restart to the child session which does not cause the launch.json to be re-evaluated

@connor4312 aha because we do not have a correlation  between the child session and the launch.json configuration. Is this something that could be handled on the js-debug side, or we have to handle it in VS Code?

Maybe it would be more sustainable to introduce a new concept in DAP, like supportsForwardParentLifecycle that would instruct the editor to send all stop/disconnect/restarts to the parent session?

Sounds a bit funky 😊
@weinand thoughts?

Yea, it's not really a 'capability', but some mechanism to request the client to forward lifecycle requests to the parent session. Remember for this case VS Code must initiate the restart on the parent since it needs to re-read the debug configuration for us. Would have to think about naming for it.

Thinking some more, I don't believe supportsLifecycle would be a terrible flag, as a default-true capability to indicate that a session doesn't independently manage its own lifecycle, implying it is instead a component of its parent..

@connor4312 @isidorn since parent/child relations between debug sessions are not part of the DAP, I think the supportsLifecycle flag is more an option on the startDebugging extension API, right?
Name suggestion: manageByParent

Yeah I think it would belong under the DebugSessionOptions in the startDebugging API since we already have parent child session flags there like parentSession and compact.

Sounds good to me

I'll add the manageByParent property to the DebugSessionOptions in July.

@isidorn a new flag lifecycleManagedByParent is now available on IDebugSessionOptions passed into DebugService.startDebugging.

@weinand thanks for adding the flag.
Since the flag's name if lifecycleManagedByParent I think it means that all the disconnect and restart requests are actually sent to the parent session? I do not think this flag would affect any other requests, correct?

@isidorn yes, only if lifecycleManagedByParent is true, disconnect and restart requests are sent to the parent session.
@connor4312 that's OK with you?

I pushed a change that the debugSession now respects lifecycleManagedByParent for disconnect, terminate and restart requests.
Leaving this issue open so @connor4312 can pass the lifecycleManagedByParent flag when calling the startDebugging API.

@connor4312 are you planning to add support for lifecycleManagedByParent to js-debug in this milestone?
If yes, I would create a test item for #128058.

Fixed in https://github.com/microsoft/vscode-js-debug/commit/f6635d9f7dd43671a17524b2f208e132527d601a

@connor4312 yes, please mark https://github.com/microsoft/vscode/issues/128058 as verified.

Verified that this works fine now.


Deblurring code

Thank you for your fantastic work.
When will you be sharing your deblurring and SR code/configs?

Thank you!
We will try to release all codes next month. However, you can easily apply the current code to deblurring by just replacing the dataset.
For SR, you can upsample all LR images to be the same sizes as HR images, and then replace the data paths in the config file. We almost use the same settings for all tasks.

@Algolzw got it. Thank you for the quick reply!


POSIX struct siginfo does not have si_stime or si_utime

Signed-off-by: Thiago Macieira<EMAIL_ADDRESS>

+1


[Urgent] Alternate for logmgr (GoLang logging framework)

Reference: https://github.com/lf-edge/edge-home-orchestration-go/issues/173#issuecomment-730330067
The current logmgr imports a 3rd party library for its GoLang logging framework, but it does not have any license statement. We need to apply its alternate library which has been released under a permissive license in open source communities.
You can refer to the current logmgr with this topic as follows:
https://github.com/lf-edge/edge-home-orchestration-go/blob/a18e10ea178cc4148307b521e7b6dadfd6d6984f/src/common/logmgr/logmgr.go#L25

I found homeedge(config.go) uses logrus as well. so I'd like to suggest to use logrus as a logging framework.

@t25kim Thank you for the prompt suggestion! It is under MIT License and it seems good to me.

I found the licese statement from 3rd-party logger library. It may be added a few hours ago^^.
However, I'd like to say we'd better use well-known logger  ather than somone's persnal logger. ( If I miss-understood, please let mt know. ) I don't know what loggers are mostly used for golang, so I've just searched and found some keywords such as Logrus, glog. Please consider them as alternative logger. Just my two cents.

@mgjeong Thank you, MJ. We will check out your proposal. As you said (and some codes in our edge-home-orchestration-go), we found Logrus used already in our codes.

Great. ^^

The current license issue is resolved since the current logger is now under Apache 2.0 license. Then the remainder is :

Consider to go / no-go : Adopt more popular logging framework as suggested by @mgjeong.


Add link to each spec's Git URL

This commit adds a a link from each Gauge spec published
in Jira by the plugin back to the spec's source file in Git (on
one of e.g. GitHub, Gitlab, Bitbucket etc).
This is a breaking change, as it requires a new mandatory
SPECS_GIT_URLconfiguration variable to be provided.

Thanks for the review @eadragomir.  An example of use is a great idea, I've been thinking the same.  Are you thinking to have a separate example repo, showing how to configure and invoke the plugin?
In terms of having one of your colleagues try it, that sounds good too.  There are some things that I want to improve on the plugin in the next release which I'll do over the coming couple of days - see #23.  I think it would be good to wait until that's done before introducing your colleagues to it (as the changes should simplify the setup).  Sound ok?

Sound OK to wait for #23. yes I am thinking for a separate repo as you mentioned.


Fix failing e2e tests due to slow animations

I had problems running the e2e tests, they appeared to randomly fail,
copied from
https://github.com/angular/angular.js/blob/master/protractor-shared-conf.js

Thanks, this worked for me.
And it makes the tests go faster too.
Interestingly, this was not necessary in the version of the phone cat that used protractor 1.0.

The whole tutorial (including all dependencies and tools) has been updated recently (#326) and this is hopefully not an issue any more. This PR is now obsolete.
If anyone is still having problems, please open a new issue/PR.


Adds client remote address getter to ConnectionDescriptor class.

Being able to retrieve client remote address would be very useful to implement ip black list or to use it in the user interceptors  for logging / debug or any specific bespoke behaviour.

If you want to blacklist a client probably you have to close the connection before creating the ConnectionDescriptor, but the IP could be useful in logging


Add new templates to devphase

Description
Add support for more templates to be created in devphase.

http_client
phat_hello
phat_storage
signing
use_cache
vrf

Currently all tests end up getting downloaded during the devphase init command. I did not want to mess with the functionality and cause anything to break, but this may need to be changed to only copy the flipper test. Then other tests can be added during the devphase contract create command.
Tests work locally when pointing to the locally built @devphase/service with the added templates.

I have improved few things. Now merged. thx


Replace zip/jar artifacts with source.

This PR removes zip/jar files that are not referenced in this repo. It may be there are references to them from other repos (I checked some repos but not all). In any case, it makes sense to move any such artifacts to where they are used. For example the python virtual env related zip files are used in the python runtime only as far as I can tell.
For the remaining test artifacts, I committed the sources and a build script to construct the zipped artifacts at setup time.

Description

Related issue and scope

[x] I opened an issue to propose and discuss this change (#4630)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[x] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4763 into master will decrease coverage by 6.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4763      +/-   ##
==========================================
- Coverage   84.85%   78.42%   -6.44%
==========================================
  Files         198      198
  Lines        8837     8837
  Branches      606      606
==========================================
- Hits         7499     6930     -569
- Misses       1338     1907     +569

Impacted Files
Coverage Δ

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0% <0%> (-100%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.85%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-93.85%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-92.31%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-77.78%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-74.08%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
7.69% <0%> (-50%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6126a94...5c8cef6. Read the comment docs.

@dgrove-oss ok to merge? once committed, we should move ahead with a release - are we ready?

@dgrove-oss - i think this broke the openwhisk cli travis...  a few of the removed files are used by the cli tests.
@steven0711dong fyi

@mdeuser https://github.com/apache/openwhisk-cli/pull/477 adds the target to build them


Extend parse rules to accept a range of frames in c++ version

This feature is used to convert a large number of frames using one command. I do not see the benefit of this feature in Python version due to its low performance so it will only be implemented in the c++ version. But be noted that each frame will output its own vtu file even though they share the same cells and nodes. So, if you have 50000 frames, odb2vtk will generate 50000 vtu files in your directory. Be prepared for that.

Implemented by the commit #965432cbc730baf583ed23ba4f6678f0158b365e


Multiclass Confmat calculation throws different device exception

🐛 Bug
When you try to calculate the confmat with GPU tensors for multiclass it throws the different device exception. Problems arises due to below operation:
self.confmat += conftmat
Where confmat defined in the GPU, and self.conftmat defined in CPU. Moving self.conftmat to GPU using self.conftmat.cuda() solves this issue.
To Reproduce
Try to calculate multiclass conftmat using GPU tensors. Pred dims (N,C,H,W), Label dim (N,H,W)

Code sample

Expected behavior

Environment

TorchMetrics version (and how you installed TM, e.g. conda, pip, build from source):
Python & PyTorch Version (e.g., 1.0):
Any other relevant information such as OS (e.g., Linux):

Additional context

Hi @onurtore, thanks for raising this issue.
I assume the problem here is that you are not moving the metric to GPU before you try to use it?
Metrics are just simple nn.Module thus they need to be moved to the device where calculations are going to happen. You can read more about this here:
https://torchmetrics.readthedocs.io/en/stable/pages/overview.html#metrics-and-devices
And here is a simple example:
import torch
from torchmetrics.classification import MulticlassConfusionMatrix

num_classes = 10

preds = torch.randn(100, num_classes, 28, 28).softmax(dim=1)
target = torch.randint(num_classes, (100, 28, 28))

metric = MulticlassConfusionMatrix(num_classes=num_classes)
assert metric.confmat.device == torch.device("cpu")

# move to GPU
metric = metric.cuda()
assert metric.confmat.device == torch.device("cuda:0")

result = metric(preds.cuda(), target.cuda())
assert result.device == torch.device("cuda:0")

:]

Sorry if I did not understand the question. The short answer is that you should not be able to do it for F1 and Accuracy either, that is an error. Locally it also works as expected for me, namely an error telling me that my accuracy metric has not been moved to the correct device.
Running:
import torch
from torchmetrics.classification import MulticlassAccuracy

num_classes = 10

preds = torch.randn(100, num_classes, 28, 28).softmax(dim=1)
target = torch.randint(num_classes, (100, 28, 28))

metric = MulticlassAccuracy(num_classes=num_classes)
result = metric(preds.cuda(), target.cuda())

gives me


Thumbnailing

This is currently (2024-06-20) does basic thumbnailing, with extremely little client-side changes.
Remaining work is to:

announce to the client the list of supported size/format/animation possibilities
transcode the full-side images for lightboxes, so clicking on .heic images shows them full-res
have them detect their capabilities (e.g. AVIF / WEBP support)
have them rewrite the thumbnailed paths accordingly
render the spinner in a more reasonable manner (one that works on both light and dark themes, and ideally is all in CSS)
for animated thumbnails, render the still version, with a mouseover to swap in the animated one

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I reviewed the half of the commits that seemed most safe and created #30516 to merge those, with the goal of locking in the early requirements changes and PROVISION_VERSION updates. That PR succeeds in replacing cairosvg with libvips while doing nothing that I consider high risk, which seems like a nice checkpoint to help shrink this down.

OK I finished reviewing everything before this commit starts to do more serious migrations:

avatars: Encode version into the filename

I expect we should be able to merge those too, after addressing the above minor comments and maybe doing a bit of extra testing of the emoji re-encoding on our data set -- I think that should be the main source of outstanding risk in removing Pillow, which we've never been particularly happy with. We can also test the test case from https://github.com/zulip/zulip/issues/21618 and likely mark the commit replacing Pillow with libvips as fixing that issue.

I've extracted 10 commits to #30584, since the first 11 looked good aside from my being unsure about https://github.com/zulip/zulip/pull/30477#discussion_r1653761464.

I did a straight rebase of this, dropping the commits already merged via #30584.

I think everything but the last 3 commits on this is merged through the PRs we split out from here.

OK finished my read, all my comments are above.

Rebased to re-insert the previous version that preserved the colorized string diff feature, and to add node testing for rendered_markdown.ts. @roanster007 FYI, as the tests that I did here verify more of the logic than what you added in #31017, and also note a TODO that I think would be well worth addressing, through just stuffing the object that I have in the test into state_data in test setup and calling thumbnail.initialize; doing so should result in it picking the 300x200 sizes rather than the first couple.


gdcp support

gdcp: copy files or folders.

82d7f33b9aefc71b67dc4cdc2f97506c6e306742: able to copy file.
Need more implementation for folder copy.


Blob expr udf

Adding support for BLOB type, registering and  retrieving  User defined functions in MyriaX and support for Python UDF expressions.

I'm not sure whether you are waiting on a code review yet.
If so, can you fix the tests first?

Coverage decreased (-0.05%) to 91.831% when pulling 1c437b833ed50ebf98264b0aa28152a2e2076233 on blob_expr_UDF into 04ad4953d1e6f01d2fd45cafc60568e34ad7854d on master.

Brandon, the tests are passing now, sorry about the delay. If you could please take a look, that woudl be great!


[New Feature]: Refactoring to have output filename generation be a method

Checked for duplicates
Yes - I've already checked
Alternatives considered
No - I haven't considered
Related problems
Right now there are 5 locations where the output path is built up in an f-string from variables as {burst_id}_{date_str}_{pol}.slc
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/s1_geocode_slc.py#L88-L89
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/s1_rdr2geo.py#L72-L74
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/s1_resample.py#L84-L90
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/utils/geo_metadata.py#L101
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/utils/range_split_spectrum.py#L90-L91
Besides needing to make sure these all match if we ever change them, there are now 5 places to change the output extension once we finish off the COG PR: https://github.com/opera-adt/COMPASS/pull/28
Describe the feature request
I'd like a single source of truth to make the output filename, which would make it easier to adjust to accomodate GTiff/HDF5/ENVI formats.
I'm picturing something like how the output path is made as a property of the GeoRunconfig:
made by the GeoRunconfig object:
https://github.com/opera-adt/COMPASS/blob/45d325b4399e37c72d2d307eee8b3549c44a47f8/src/compass/utils/geo_runconfig.py#L113-L117
Does it seems like a good idea to have another property to make an output string? It might have to go into the s1reader repo, but I was picturing something like
@property
def output_name(self):
    date_str = self.sensing_start.strftime("%Y%m%d")
    burst_id = self.burst_id
    pol = self.polarization
    return f'{burst_id}_{date_str}_{pol}'

Alternatively, it could just be a utility function in in this repository to take a burst and output that string:
# maybe in utils.helpers
def get_output_name(burst, ext=".tiff"):
    date_str = burst.sensing_start.strftime("%Y%m%d")
    burst_id = burst.burst_id
    pol = burst.polarization
    return f'{burst_id}_{date_str}_{pol}{ext}'

@LiangJYu have we address this? If yes, let's close this issue

@LiangJYu have we address this? If yes, let's close this issue

This is partially solved in geocode_slc with this but not so much elsewhere.

@LiangJYu I think we have addressed this entirely. Can you confirm?

@LiangJYu I think we have addressed this entirely. Can you confirm?

It is not entirely addressed. I will fix and submit a PR.

When #194 is merged, I think issue can be closed.

Closing as #194 has been merged!


⚠ Bofrost.lu has degraded performance

In e6d2eaa, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2462 ms

Resolved: Bofrost.lu performance has improved in 3dac624 after 6 minutes.


Screen Sharing

It would be nice if we could get a screensharing feature.

no message?

@benawad doesn't want this feature.

Ah okay wasn't clear on that


Stylecop custom rules from 4.7 does not work on 5.0

Hi,
My app was working well with stylecop 4.7. Stylecop 4.7 was installed as separate application. Custom rules - StyleCop.CSharp.Rules.MyProj.dll, was copied directly to Stylecop 4.7 folder.
Now I want to migrate to 5.0.
Stylecop 5.0 installed as VS2017 extension. Also Stylecop.MSBuild 5.0 NuGet package was installed for my solution's projects. My own Settings.StyleCop was used successfully.
But custom rules was not.
StyleCop.CSharp.Rules.MyProj.dll was copied into $SolutionDir$\packages\StyleCop.MSBuild.5.0.6\tools directory.
I need my rules also to be copied to StyleCop 5.0 extension directory. How am I suppose to do that? I can't find any way to know the exact directory, since the name is random.
Or is there any other way to import my custom rules?

Just copy custom rules to  project dir, same way as Settings.StyleCop


v2.3.0-beta.5 Climate Card Shows Multiple Set Temperature Groups

Describe the bug
Climate card shows set temperature, target_temp_high, and target_temp_low in heat mode.
Thermostat is an Ecobee using the Homekit integration. This also happens when using the (cloud based) Ecobee integration.
To Reproduce
Steps to reproduce the behavior:

Create a bubble climate card
Choose an Ecobee thermostat (either homekit or cloud integration)
Card shows 3 temperature sub-button groups

Expected behavior

In heat or cool mode, I expect to only show 1 temperature sub-button group
In heat/cool mode, I expect to see 2 temperature sub-button groups

Screenshots

YAML
If applicable, add any relevant YAML code.
type: custom:bubble-card
card_type: climate
entity: climate.ecobee
sub_button:
  - name: HVAC modes menu
    select_attribute: hvac_modes
    state_background: false
    show_arrow: false

Informations (please complete the following information):

OS: Windows, Android
Browser/App: Chrome, Home Assistant Companion App
Bubble Card version: v2.3.0-beta.5
Home Assistant version: 2024.11.1

Additional context
Thermostat state and attributes
state:
heat
(homekit) attributes:
hvac_modes: off, heat, cool, heat_cool
min_temp: 45
max_temp: 92
min_humidity: 20
max_humidity: 50
fan_modes: on, auto
current_temperature: 68
temperature: 68
target_temp_high: null
target_temp_low: null
current_humidity: 34
humidity: 36
fan_mode: auto
hvac_action: fan
friendly_name: Ecobee
supported_features: 399

(cloud) attributes:
hvac_modes: heat_cool, heat, cool, off
min_temp: 44.6
max_temp: 95
target_temp_step: 0.5
fan_modes: auto, on
preset_modes: home, away, sleep, away_indefinitely
current_temperature: 68.2
temperature: 68
target_temp_high: null
target_temp_low: null
current_humidity: 34
fan_mode: auto
hvac_action: fan
preset_mode: home
fan: on
climate_mode: Home
equipment_running: fan
fan_min_on_time: 30

Hi! I don't have any smart thermostat to test it, so your feedback is really welcome!
I will fix that in the next release 👌

This should be fixed, could you to try this version? 🤞
To install it, you need to remove Bubble Card from HACS first, then install it with the "Without HACS" method.
bubble-card.js.zip
Once installed, clear your cache then check in the editor if you see v2.3.0-beta.6.
I can't wait for your feedback because this is probably the last issue with the climate card 😁

It seems to work now. It's only showing the 1 group when in heat or cool mode. And 2 groups show when in heat/cool mode.
Adjusting the temperature works as well.

Yeah, finally! Thank you for testing it that fast! 😃

Also, confirming this works for me as well. Thanks for your responsiveness!


Update branch with QBILab code

I want to update my SI_to_Conc testing branch with the QBILab code so I can write a test for it.  I think I was able to do this automatically last time so I'm not sure why it's different this time....?


added TypeSafeMiddleware aliases

Type Aliases used with TypeSafeMiddleware. This is needed before the TypeSafeMiddleware pull request will pass.

Adding type aliases into Kitura contracts would require importing Kitura which we don't want to do so we are closing this PR.


Can't launch flake8 in atom or term.

fixed,
updated flake8 from git source, to 3.5.0
updated pycodestyle to 2.4.0
issues resolved.


Update pipeline and triggers package

What type of PR is this?

/kind cleanup

What does this PR do / why we need it:
Update pipelines and triggers packages to their latest versions.
Pipeline https://github.com/tektoncd/pipeline  - v0.28.0
Triggers https://github.com/tektoncd/triggers - v0.16.0
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

/test v4.8-integration-e2e

Thanks @chetan-rns for the PR. Tested the changes locally, works fine for me.
@wtam2018 PTAL

/lgtm
/approve

/test v4.8-integration-e2e

/test v4.8-integration-e2e

/test v4.8-integration-e2e

/test v4.8-integration-e2e

/test v4.8-integration-e2e

/test v4.8-integration-e2e

/test v4.8-integration-e2e


Error: Packets out of order. Got: 80 Expected: 0

I am trying to connect to our mysql server using the following code:

var mysql = require('mysql');
var db = mysql.createConnection({
  host: "XXX.XXX.XXX.XXX",
  port: XXXX,
  database: "XXX",
  user: "XX",
  password:" XXX",
  debug:true
});

db.connect(function(err) {
  if (err) console.error('error connecting: ' + err.stack);
  console.log("Connected!");
});

However I receive the following error message:
error connecting: Error: Packets out of order. Got: 80 Expected: 0
    at Parser.write (some-folder/code/node_modules/mysql/lib/protocol/Parser.js:42:19)
    at Protocol.write (some-folder/code/node_modules/mysql/lib/protocol/Protocol.js:39:16)
    at Socket.<anonymous> (some-folder/code/node_modules/mysql/lib/Connection.js:103:28)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at TCP.onread (net.js:547:20)
    --------------------
    at Protocol._enqueue (some-folder/code/node_modules/mysql/lib/protocol/Protocol.js:145:48)
    at Protocol.handshake (some-folder/code/node_modules/mysql/lib/protocol/Protocol.js:52:23)
    at Connection.connect (some-folder/code/node_modules/mysql/lib/Connection.js:130:18)
    at Object.<anonymous> (some-folder/code/src/db/index.js:21:4)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)

Node version: 6.11.1
mySql Version: 5.6.38
Any idea what I am missing here?

Nothing you're doing wrong; that usually means something is really wrong. Can you provide the version of this module you are using as well? I'll work on setting up a MySQL server version of exactly 5.6.38. I assume you're using the community version? Did you get it from a package manager or compile from source? Also would you be willing to use Wireshark to capture the packets during this on your machine?

@dougwilson I think I was using the wrong port and don't believe its an issue with your package. Thanks for the quick response though. If you still need anything from my end, let me know.

Hey! Sorry to comment on an older thread (I'm a bit new to GitHub and Node.js...) but has anyone found a solution to this? I'm running into the same problem myself. If I can provide any info, I'll be happy to help. I'm using Node version 8.9.4 at the moment!

Hi @TheRockstarr33 it's OK. Please feel free to open a new issue regarding what you're encountering as this issue has been resolved (connecting to wrong port). Please include in your report as much as possible, including the exact error you're getting, version of this module, and perhaps even code to reproduce the issue and we're happy to help.
Sorry I didn't see your post until I went to clean up this issue. Opening a new issue brings it to attention quicker.

Hi @dougwilson, crash occurs i.e. when you stop MySQL service while mysqljs connection (single or via pool) is open, server sends to clients "Connection was killed" which eventually kills whole node app.
Server on this error sets "Packet Number" to 0 (zero), that makes fatal exception on mysqljs side.

Hi @kac- very interesting. I wonder why then would OP get the message that mysqljs expected the packet number to be zero but the server sent packet number 80 if that's the case here?

Ah, interesting @kac- . Are you sure you've having the same issue here or do you want to create a new issue to describe your issue so we can take a deeper look?

It definitely looks like separate issue.


Use updated conda recipes

add Makefile and use 'master' branch of cdat/cdat since 'validateNightlyNew' branch has been merged to 'master'.

@lee1043 https://circleci.com/gh/PCMDI  all test runs passed. those circleci job like linux_pmp_* macos_pmp_* jobs that are listed above that still say "Required" are old circleci jobs.
The build above picked up the genutil that has @jasonb5 's fix for https://github.com/CDAT/genutil/issues/38.
NOTE that I added py36, py37, py38 for both macos and linux.  As a result, each commit triggers builds and test runs that take a long time to finish. It is up to you if you want to include all those configs.

@muryanto1 thank you for the update. Once all tests passes on circle ci, which is pending now, I will merge this.

@lee1043 all tests passed, please see the list of tests under "Some checks haven't completed yet".  build-linux, build-macos, test-linux-3.6-mpi_mpich, test-linux-3.6-mpi_openmpi, test-linux-3.6-nompi, etc...they are all jobs from .circleci/config.yml, and they all passed. For some reason the old jobs linux_pmp_py2, linux_pmp_py3 and macos_pmp_py2, macos_pmp_py3 are still listed up there under "Some checks haven't completed yet". I do not know why.

@muryanto1 I see, thanks for clarifying. Because of the "Some checks haven't completed yet" showing up, I thought it was still processing. I am merging it now.