Updating cron-send-usage-pv to support SI units

This adds unit handling to this check so that the proper usage values are reported. This will fix some of the ops-runner alerts for this check that are currently on the zabbix board.

[test]

:-1:
Rather than doing unit coversions in the script, I'd prefer to see the raw number submitted to zabbix and use the item's Units, Custom Multiplier, or Store Value As configuration settings to tell Zabbix how it should be transformed, whichever is appropriate.

Chatted with @blrm and he made a convincing argument. So, :+1: for the change given that we're responding to unfortunate behavior from the underlying call we're making to collect data.


add Marcelo Amaral to reviewers

Signed-off-by: Marcelo Amaral<EMAIL_ADDRESS>Add @marceloamaral as a reviewer.
I have been working with the KubeVirt project for several months now, mainly contributing to the kubevirt and project-infra subprojects. In addition,I have been also reviewing and approving PRs. I will be happy to continue contributing as a reviewer of the project.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @zcahana @davidvossel @stu-gott
Release note:

NONE

/lgtm
/approve
@marceloamaral great to see you expanding your engagement even more with additional code reviews! :+1:

/lgtm cancel

/lgtm


Player notes system

Description

Admins need the ability to leave notes on players. These notes should be stored in the database and carry across to servers in the same "cluster". (For example if an admin leaves a note for someone on SS14 USWest it would show to an admin on EUW but not on say Hyperion Station).
Notes will expire after a set amount of time. You should be able to enter this at note creation but there should also be a default of a year that applies if you leave nothing in the box.
In the future players should be allowed to see their notes, ideally by logging into a web-panel but displaying them in-game would be acceptable for now since we don't have much of a player web-presence.
This is a big project so i've left made rough mockups I made for what i'd like the UI to look like. I'd like to avoid admins opening a cascade of windows when making a note so try and handle stuff in the window itself instead of opening new "textbox entry" windows.
Screenshots

Currently working on this.

Currently working on this.

K I assigned you.

Done in https://github.com/space-wizards/space-station-14/pull/7259


Fix output handling in RenderProcessor

Ensure that output modifications by subscribers to onOutputGenerated are picked up.
In contrast to 1.2.4, 1.3.0-rc.2 no longer recognizes output modifications applied by subscribers to onOutputGenerated.
In 1.2.4, output was rendered in Grav.php via echo $this->output; after processing onOutputGenerated inside RenderProcessor.

You're right.


Add shaders

Shaders from https://github.com/hyprland-community/awesome-hyprland/issues/22
also closes #22
todo:

[ ] move to subdir
[ ] get images of the shaders, prob with capture card or vm
[ ] get more

I need screenies, if anyone here wants to help. That'd be sick

Only missing three: blur and bloom, which I couldn't load, and drugs which didn't look good on my screenshot.

Bluelight

CRT

Monochrome

Vibrance

Sick thanks so much!


Chapter 3 - Classification

this error happens when trying to view the instance

Having a similar issue on the first code example of Chapter 3 (2nd Ed Page 86). Instead of setting index, I had to change it to X.iloc[idx] and some_digit.values.reshape() to get it work.
``
There are 70,000 images, eadh image has 784 features (28x28 pixel)
import matplotlib as mpl
import matplotlib.pyplot as plt
some_digit = X.iloc[0]
some_digit_image = some_digit.values.reshape(28,28)
plt.imshow(some_digit_image, cmap="binary")
plt.axis("off")
plt.show()

Thanks for your feedback. This issue was caused by the fact that Scikit-Learn changed fetch_openml() in version 0.24 to return Pandas DataFrames by default instead of NumPy arrays. I fixed this by setting as_frame=False when calling fetch_openml(). Please git pull to get the latest code, and try again, everything should work fine now.
Closing this issue, but feel free to reopen this issue if the problem persists.


set nwchem env vars upon activation

fix #6
don't automatically copy the .nwchemrc file

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Does this look good to you @edoapra ?

@ltalirz it looks good to me.


Add Dockerfile and docker-compose.yml for developing locally

Added Dockerfile and docker-compose.yml files for developing locally.
You may need to add missing .env variables.
To run the app locally, you'll first need to run the following command.
docker compose up conferece-app

After that, open http://localhost/ on your machine and you'll see something like below.

Click the conference-app (80:3000) link to view the application!

Thank you for the review! I fixed some codes and added some more details on how to start the application.

Sorry, I need to tweak some more to allow sidekiq to be running 🙏
I'll change this to a draft for now, and bring this back once everything seems working 🙇

I didn't intend actually to call the API request in the test.... It's my bad....
https://github.com/kaigionrails/conference-app/actions/runs/6668363377/job/18123776972


Dealing with synonyms in the text

Sometimes, the authors introduce a term for concept together with synonyms. There needs to be a possibility to capture these synonyms in SNIK-Tag.

We typically use a set of alternative labels for that purpose in SNIK. For ease of use, we could also just use a single list for the labels and the first one becomes the label at the end.

Decision meeting 2019-07-19: use multiple labels.

[ ] classes have a set of labels
[ ] the set of labels is shown appropriately in the class table
[ ] the set of labels can be edited in the class table
[ ] the labels are properly converted to RDF
[ ] the labels are properly handled in the class text area
[ ] the labels are properly shown in the relation pane (adapt Clazz::toString())

Note: Multiple labels is not the usual SNIK style. Until now, we had at least one label, one at most per language, and the rest is modelled as skos:altLabel. We could still convert it differently at the end but then we have to identify the label against the altLabel. Probably the first one is the label. However if we have a set we don't preserve order in Java.


fixed incorrect command

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9398

#sign-off

@greg-lindsay PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


A pull request for my linkfree contribution

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thanks for the approval and the support.
Please which others contributions can i make?
i'm still trying to understand a large codebase, how to understand an
issues that needs to be fixed etc
On Sat, Feb 4, 2023 at 3:51 PM Priyankar Pal @.***>
wrote:

@.**** approved this pull request.
LGTM 👍🏻
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/LinkFree/pull/4472#pullrequestreview-1284077006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWLW3VOD32FFTLNGJV6RXFLWVZUHFANCNFSM6AAAAAAURF2W5A
.
You are receiving this because you authored the thread.Message ID:
@.***>

@Ewaaugustine you can check issues or you can join the discord.

Hey Augustine, you can check out the other issues on LinkFree you can contribute to.
Here is a great article by Pradumna explaining other 5 ways you can find open-source projects you can contribute to: https://blog.pradumnasaraf.dev/5-ways-to-find-projects-in-open-source

i'm still trying to understand a large codebase, how to understand an issues that needs to be fixed etc

You can be good at these by practicing, but one thing I wanted to add is that you can look at the past closed and merged PRs. It really helps you to understand the codebase of a project easily when you took reference to an issue.
For any issue you feel like you can fix, you can always ask the maintainers if you have any confusion about what changes needed to be made and where, or if you need more info about that. 🤝

Hope this helps!
Happy Open-Sourcing! 🚀


Config option for whether teleportChangesGravity/teleportationChangesScale affect portals by default

Currently portals change scale by default, but changing gravity has to be explicitly specified via a command. Some sort of option in the config which controls whether they're present by default or not would be useful.

It's being considered


Panel visiblity status is not reflected in the "Window" menu

Our tool has multiple panes that can have their visibility status toggled by clicking on their menu item in the "Window" menu.  The issue is that those menu items always have a check mark next to them, no matter the visibility status.  Repro steps:

Add a OutputService to your program
Launch program and notice the "Output" menu item in the "Window" menu is checked and the panel is visible.
Click the "Output" menu item.
Notice that the panel has been hidden.
Click on the "Window" menu again and notice that the "Output" menu item is still checked.

Expected that the "Output" menu item is no longer checked because the panel is not visible.
I believe I have identified that the issue is that the CommandInfo is never updated after the program is initialized.  I came up with a solution by modifying CommandService.item_Click :

ICommandClient client = GetClientOrActiveClient(tag);
if (client != null && client.CanDoCommand(tag))
{
client.DoCommand(tag);
UpdateCommand(tag); //Force the command status to be updated
}

My question is if this is the correct fix or if there is a better way to accomplish the desired behavior.

Please keep your local fix as long as it works.
I will try to reproduce this bug and update this issue when I get a chance.
So let us keep this issue open until then.
Alan

Hello Alan,
Any updates on this?

Sorry I don't any update at this time.
Currently  I don't have any time to work on ATF because of internal projects.
I will update ATF when I get a chance but I don't any time estimate.
Thanks
Alan

I couldn't reproduce the issue with the steps you provided using master branch.
if you are using anything other than master branch, please use master branch instead.
Due to lack of time my current plan to update only master branch.
If anyone else experiencing this issue please let me, I will keep this tracker open.


update proxy of master to raftstore-proxy by proxy pr 202

What problem does this PR solve?
Issue Number: ref #4982
Problem Summary:
update proxy to raftstore-proxy
Proxy PR:
https://github.com/pingcap/tidb-engine-ext/pull/202
Including:\nSubmodule contrib/tiflash-proxy 9f34e1bc8...010f92e98:

Save pending_delete_ssts in TiFlashObserver to avoid potential leak of file (#202)
Enable Multi SST Snapshot (#201)
Merge TiKV about 6.4(b448214b8) (#194)
< Merge branch 'merge-6.4' of github.com:calvinneo/tidb-engine-ext into merge-6.4
< re order
< fmt
< remove snap test since we don't support full leader logic
< fix tests and format
< Merge remote-tracking branch 'tikv/master' into merge-6.4

What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/merge


Add "View Liked songs" section

Description
This PR adds a "View Liked Songs" section for users to be able to search their liked songs and play any they choose.
The changes include adding a new scope to be able to view the user's library. This will require re-authenticating with Spotify.
Screencast
https://user-images.githubusercontent.com/9905640/228963528-6ec8e79b-20e4-4a34-9b32-259cd636731f.mp4

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thank you for your PR, but I believe this is superseded by PR #5602 which is a v2 of the extension and includes a comprehensive overhaul of the Spotify extension in general. I'd kindly suggest awaiting the release of the updated extension to see if it suits your needs here.

Hey, thanks for taking the time to create this PR. As @andreaselia said, a V2 is underway. More info #5602
Thanks


Hi everyone ！ i can't copy a new level from a  existing level

I tried using a derived method to copy a new level, but it was constantly being read and could not generate a new level
i need your help ！

Did you use the Fork action in the top right of the level editor? If so, what error did you get?


Starlight's sidebar visibility issues with view transitions

Describe the Bug
First of all, thank you for these great tools and tricks!
I am encountering a couple of issues with the view transitions implementation on my Starlight site. These issues include:
Sidebar initially invisible: When landing on a page with active view transitions, the sidebar is initially invisible. However, after reloading the page, the sidebar becomes visible and remains so for subsequent navigation.
Sidebar showing up on inappropriate pages: When navigating to certain pages, such as the docs home page, the sidebar is displayed even though it shouldn't be. Reloading the page seems to resolve this issue temporarily.
Steps to Reproduce:

Navigate to the docs site (link is in the footer).
Click on Get started button.
Observe that the sidebar is initially invisible.
Reload the page.
Note that the sidebar becomes visible after reloading.
Navigate to a page where the sidebar should not be displayed (e.g., docs home page).
Observe that the sidebar is still visible on the page.
Reload the page.
Note that the sidebar disappears after reloading.

What's the expected result?
Display  a sidebar without the need to reload the page.
Link to Minimal Reproducible Example
https://github.com/mearashadowfax/ScrewFast/pull/40
Participation

[ ] I am willing to submit a pull request for this issue.

Hey @mearashadowfax! Thank you for test driving the Starlight support!
I'm sorry for the issues but try to fix them asap ;-)


let the room manager delete from router and store

room manager deletes local room before deleting room state and stored room - potentially introduces a bug if we remove from router and store which room manager still thinks it's there


Unable To Find Entry Point in DLL 'libLLVM'

Hi.
I'm using LLVMSharp 5.0, and I've been able to replicate on LibLLVM 5.01, 6.01, and 7.01 (from nuget).
I'm seeing the following error:
System.EntryPointNotFoundException: 'Unable to find an entry point named 'LLVMInitializeAllTargetInfos' in DLL 'libLLVM'.'
My code is:
            LLVM.InitializeAllTargetInfos();
            LLVM.InitializeAllTargets();
            LLVM.InitializeAllTargetMCs();
            LLVM.InitializeAllAsmParsers();
            LLVM.InitializeAllAsmPrinters();

            LLVM.GetTargetFromTriple("x86_64-pc-windows", out LLVMTargetRef target, out string error);

I'm trying to retrieve the target so that I can get the target data.
Going through the first block and commenting out, each of them caused the same error (with the entry point name changed).
I'm not seeing this for any other LLVM methods, including GetTargetFromTriple.
Updating beyond LibLLVM 7.01 causes the following error:
System.DllNotFoundException: 'Unable to load DLL 'libLLVM': The specified module could not be found. (Exception from HRESULT: 0x8007007E)'
Thanks.

Going down to LLVMSharp 3.8.0 and libLLVM 3.9.1 fixed the issue

According to dumpbin, the function names (like LLVMInitializeAllTargetInfos) are actually suffixed with an underscore (like LLVMInitializeAllTargetInfos_). However, attempting to import the function with underscore-suffixed name still failed similarly. I was able to get past those linkage errors by specifying the ordinal (also obtained from dumpbin) instead of the name, like below.
[DllImport("libLLVM", EntryPoint = "#530", CallingConvention = CallingConvention.Cdecl)]
public static extern void LLVMInitializeAllTargetInfos();

[DllImport("libLLVM", EntryPoint = "#532", CallingConvention = CallingConvention.Cdecl)]
public static extern void LLVMInitializeAllTargets();

[DllImport("libLLVM", EntryPoint = "#531", CallingConvention = CallingConvention.Cdecl)]
public static extern void LLVMInitializeAllTargetMCs();

[DllImport("libLLVM", EntryPoint = "#527", CallingConvention = CallingConvention.Cdecl)]
public static extern void LLVMInitializeAllAsmParsers();

[DllImport("libLLVM", EntryPoint = "#528", CallingConvention = CallingConvention.Cdecl)]
public static extern void LLVMInitializeAllAsmPrinters();

But calling these function still doesn't seem to work. After doing so, I get either an error that no LLVM targets are registered, or an SEHException: "(0x80004005): External component has thrown an exception."

Downgrading LLVMSharp gave me all sorts of problems, so I just uninstalled and made my own wrapper for the functions I needed.

LLVM requires you to exactly match versions or all kinds of bugs and other bad behaviors can occur. They likewise take breaking changes each and every release, which can lead to another class of issues.
It's likewise worth noting LLVMSharp 5.0 is a very old build and had many known bugs. LLVMSharp 8.0.0-beta and newer are recommended for use instead. The newer versions of the bindings are 1-to-1 and blittable and don't have the same sorts of issues or other regressions that you could otherwise run into. Because of this even though they are marked beta, they are effectively more stable than 5.0.0 and prior.
A new stable version is planned, but there needs to be some more validation on the latest bits before that can happen.


Consider removing the flattened CallOptions overload from C# GRPC generated client service code?

While considering altering the code structure of C# GRPC generated service code in #5371 might we also consider removing the flattened CallOptions overloads from the client code?
So instead of calling an RPC method like this:
client.Method1Async(request,
    cancellationToken: myCancellationToken,
    deadline: myDeadline);

The call would need to use the explicit CallOptions method like this, which is functionally equivalent:
client.Method1Async(request, new CallOptions(
    cancellationToken: myCancellationToken,
    deadline: myDeadline));

A few of reasons for suggesting this:

GRPC-generated code is expected to be low-level and simple; providing functionally-equivalent overloads goes against this somewhat.
For many APIs we would expect users to target a higher-level API than this, so providing simplicity at this level isn't required.
Removing this overload reduces the API surface area, making client-side testing/mocking/faking simpler.

Result of discussion: whether we move to abstract classes or not, extension methods would be better here, for both testing and simple expression of capabilities. So let's keep the overloads "logically", but express them via extension methods instead of being part of the interface/abstract-class.

Isn't this one basically the same topic as https://github.com/grpc/grpc/issues/5323? Let's keep just one of these issues alive to track the progress.

I think the resolution is the keep those overloads and make them call the CallOptions based overload (as described in #5323)


max-tests option in config.yml not working as documented

Hello 👋
I added a max-tests setting to my config.yml under the retries key as documented here but when I try to run the suite with captain run ...  I get a parsing error saying field max-tests not found in type cli.SuiteConfigRetries

I get the same error both locally on v1.14.3 and in CI on v1.15.0
Thanks!

Hi Christian,
Thanks for reporting this - it looks like our parsing code is indeed not handling this correctly. We'll have a fix for this shortly - as a temporary workaround, you might be able to set the option by spelling the field maxTests.

Hi Pierre, thanks a lot for the prompt response! 🙌

Alright, we just published v1.15.1 which fixes this issue. Thanks again for the report!


Always uses default search query (0)

No matter what is passed as an argument, it returns search results for 0.
$ search foo bar
 0:  https://twitter.com/0?lang=en
 1:  https://en.wikipedia.org/wiki/0
 2:  https://en.wikipedia.org/wiki/0_(number)
 3:  https://www.ixl.com/math/grade-3/multiply-by-0
 4:  https://www.theguardian.com/football/live/2016/jun/05/sweden-v-wales-pre-euro-2016-international-friendly-live
 5:  http://www.bbc.com/sport/football/36407575
 6:  http://mathforum.org/dr.math/faq/faq.0.to.0.power.html
 7:  https://play.google.com/store/apps/details?id=com.clearskyapps.fitnessfamily.PushupsPro&hl=en
 8:  https://www.youtube.com/user/Apple
prompt: Update selection (blank to accept)>>:

On investigation, the issue has been fixed in 1.2.3, but package.json is hard-coded to install version 1.2.2. 1.2.2 should be removed from the repository and 1.2.3 should be renamed just "node-cli-google.js." package.json should also be updated accordingly.

hi @nloveladyallen,
thanks for the contribution. will be checking the bug.
Take care.


Fix repo check

This fixes an issue whereby even though the link to the repo data is created the apply fails first time on occasions


Fix API calling with different parameters

Summary
Fetch 20 releases from Open/Closed state separately
https://app.zenhub.com/workspaces/ocm-team-space-5db37ce273e8f70001d9e2b1/issues/gh/bcgov-registries/ocm/1035

/gcbrun

Temporary Url for review: https://release-notes-dev--pr-22-k0c8etr9.web.app

/gcbrun

Temporary Url for review: https://release-notes-dev--pr-22-k0c8etr9.web.app


Remove pandas as dependency

Can we please remove pandas from the requirements?
Using this in the wild requires using pandas which is not needed by the core utils AFAIK

Closed by #227


Historic Match History Download

What problem is this solving
The ability to download historic match demo's, outside of the current system that's available; Such as GCPD.
Proposed solution
Valve provides "GCPD" to view historic match information and obtain demo's for those matches; A way to download via GCPD along with aligning the date of the match with the correct date provided on the page...
Describe alternatives you've considered
Thus far I've manually downloaded matches and altered metadata to provide the correct date of the match; However this is extremely cumbersome and time consuming.. I'm aware of another project that /tries/ to automate this, however it doesn't seem to be maintained.

Hi!
Unfortunately, there is no API to fetch these data without being logged on Steam.
It means it can't be automated without asking for Steam credentials or copying/pasting Steam auth cookies.
I don't like both ideas, and the app will never ask for Steam credentials.
While it could be an external project idea, I don't plan to work on a tool that scraps data from the Steam page myself.
I didn't try it, but maybe this project would work as it seems to be recent https://github.com/claabs/cs-demo-downloader
I'm closing this issue as there is no actionable action.
That would be possible if the Steam GC ever exposes unranked/wingman matches.


Thank you for indexing these packages

nixpkgs-channels: Enumerating objects: 1014819
nixpkgs-channels: Enumerating objects: 1014819
nixpkgs-channels: Enumerating objects: 49583, done

while evaluating the attribute 'config.system.build.toplevel' at /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/top-level.nix:275:5: while evaluating 'foldr' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/lists.nix:52:20, called from /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/top-level.nix:139:12: while evaluating 'fold'' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/lists.nix:55:15, called from /nix/var/nix/profiles/per-user/root/channels/nixos/lib/lists.nix:59:8:`
boot.loader.grub.devices

=IIf(DCount("*";"[0]

=IIf(DCount("*";"[0]


Template error on kubelet config file

Is this a BUG REPORT or FEATURE REQUEST?: Bug report

I am running kargo against my cluster with a new node added (to increase the cluster's size). This is the only change between this run and the previous successful run.
Environment:

Cloud provider or hardware configuration: Openstack/CoreOS 1235

OS (printf "$(uname -srm)\n$(cat /etc/os-release)\n"):

Version of Ansible (ansible --version):

ansible <IP_ADDRESS>
  config file = /home/ubuntu/itsec-kube/kargo/ansible.cfg
  configured module search path = Default w/o overrides

Kargo version (commit) (git rev-parse --short HEAD): I am using a fork checked in to another git repo and lost the version
Network plugin used: Flannel
Copy of your inventory file:
kube-etcd-01   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-etcd-02   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-etcd-03   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-master-01 ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-master-02 ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-master-03 ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-01   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-02   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-03   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-04   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-05   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-06   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-07   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-08   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-09   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true
kube-node-10   ansible_ssh_host=<IP_ADDRESS>  ansible_become=true

[kube-master]
kube-master-01
kube-master-02
kube-master-03

[etcd]
kube-etcd-01
kube-etcd-02
kube-etcd-03

[kube-node]
kube-node-01
kube-node-02
kube-node-03
kube-node-04
kube-node-05
kube-node-06
kube-node-07
kube-node-08
kube-node-09
kube-node-10

[k8s-cluster:children]
kube-node
kube-master

[all:vars]
ansible_python_interpreter=/opt/bin/python

Command used to invoke ansible:
ansible -i inventory/inventory cluster.yaml
Output of ansible run:
fatal: [kube-node-01]: FAILED! => {"changed": false, "failed": true, "msg": "AnsibleError: template error while templating string: expected token '=', got 'end of statement block'. String: # logging to stderr means we get it in the systemd journal\nKUBE_LOGGING=\"--logtostderr=true\"\nKUBE_LOG_LEVEL=\"--v={{ kube_log_level }}\"\n# The address for the info server to serve on (set to <IP_ADDRESS> or \"\" for all interfaces)\nKUBELET_ADDRESS=\"--address={{ ip | default(\"<IP_ADDRESS>\") }}\"\n# The port for the info server to serve on\n# KUBELET_PORT=\"--port=10250\"\n# You may leave this blank to use the actual hostname\nKUBELET_HOSTNAME=\"--hostname-override={{ ansible_hostname }}\"\n\n{# Base kubelet args #}\n{% set kubelet_args_base %}--pod-manifest-path={{ kube_manifest_dir }} --pod-infra-container-image={{ pod_infra_image_repo }}:{{ pod_infra_image_tag }}{% endset %}\n\n{# DNS settings for kubelet #}\n{% if dns_mode == 'kubedns' %}\n{% set kubelet_args_cluster_dns %}--cluster_dns={{ skydns_server }}{% endset %}\n{% elif dns_mode == 'dnsmasq_kubedns' %}\n{% set kubelet_args_cluster_dns %}--cluster_dns={{ dns_server }}{% endset %}\n{% else %}\n{% set kubelet_args_cluster_dns %}{% endset %}\n{% endif %}\n{% set kubelet_args_dns %}{{ kubelet_args_cluster_dns }} --cluster_domain={{ dns_domain }} --resolv-conf={{ kube_resolv_conf }}{% endset %}\n\n{# Location of the apiserver #}\n{% set kubelet_args_kubeconfig %}--kubeconfig={{ kube_config_dir}}/node-kubeconfig.yaml --require-kubeconfig{% endset %}\n{% if standalone_kubelet|bool %}\n{# We are on a master-only host. Make the master unschedulable in this case. #}\n{% set kubelet_args_kubeconfig %}{{ kubelet_args_kubeconfig }} --register-schedulable=false{% endset %}\n{% endif %}\n\nKUBELET_ARGS=\"{{ kubelet_args_base }} {{ kubelet_args_dns }} {{ kubelet_args_kubeconfig }}\"\n{% if kube_network_plugin is defined and kube_network_plugin in [\"calico\", \"weave\", \"canal\"] %}\nKUBELET_NETWORK_PLUGIN=\"--network-plugin=cni --network-plugin-dir=/etc/cni/net.d\"\n{% elif kube_network_plugin is defined and kube_network_plugin == \"weave\" %}\nDOCKER_SOCKET=\"--docker-endpoint=unix:/var/run/weave/weave.sock\"\n{% elif kube_network_plugin is defined and kube_network_plugin == \"cloud\" %}\n# Please note that --reconcile-cidr is deprecated and a no-op in Kubernetes 1.5 but still required in 1.4\nKUBELET_NETWORK_PLUGIN=\"--hairpin-mode=promiscuous-bridge --network-plugin=kubenet --reconcile-cidr=true\"\n{% endif %}\n# Should this cluster be allowed to run privileged docker containers\nKUBE_ALLOW_PRIV=\"--allow-privileged=true\"\n{% if cloud_provider is defined and cloud_provider in [\"openstack\", \"azure\"] %}\nKUBELET_CLOUDPROVIDER=\"--cloud-provider={{ cloud_provider }} --cloud-config={{ kube_config_dir }}/cloud_config\"\n{% elif cloud_provider is defined and cloud_provider == \"aws\" %}\nKUBELET_CLOUDPROVIDER=\"--cloud-provider={{ cloud_provider }}\"\n{% else %}\nKUBELET_CLOUDPROVIDER=\"\"\n{% endif %}\n"}

I backported my changes in the master from this repo and things are working again. Not going to spend any more time debugging. Closing ticket.

i have the same issue.
i just make changes to some lines into group_vars/all.yml like following

any help ?

this is my all.yml file
all.txt

I've run into this issue also, it seems like whatever is rendering the kubelet.j2 jinja2 file doesn't like the {% set xxxxx %} yyyyy {% endset %} lines. {% set xxxxx = "yyyyy" %} seems OK.
Need to look into this further myself, will update if I find out exactly why it doesn't render set/endset.

I had the same issue. Upgrading jinja2 worked for me. (1 was using stock version from Debian 8)

@PatxD what version did you upgrade to?  I used pip and am running Jinja2 2.9.5 which should support set/endset. This is for python 2.7, unless kargo uses a venv on another python version or something

python 2.7.9

pip show jinja2
Name: Jinja2
Version: 2.8.1
Location: /usr/local/lib/python2.7/dist-packages
Requires: MarkupSafe

it's a dup of https://github.com/kubernetes-incubator/kargo/issues/844 . Try with the new jinja

If kargo is supposed to be supporting CentOS7 it should be using jinja techniques that are supported in the jinja package that is installed with that distribution or document in the 'Requirements' section what a user is supposed to do manually before running the ansible playbook.. tho the ansible playbook could take care of this for the user probably since it's already heavily distro agnostic .. perhaps using the side effects of rendering a template as a way to set imperative variables could be handled in a better way?

@skyscooby The os support are for the target host on which you deploy kubernetes.
The deploy host on which you install ansible/jinja must be prepared by the user (install ansible version and dependencies).

I had the same issue:
I just updated Jinja2: pip install --upgrade Jinja2

I have instructed users to jinja2==2.8.1 in the requirements.txt. I'm not sure how this isn't a problem for others but that's my work around -- probably something to do with my brew setup. Closing for now.

I am running it with jinja=2.10 and facing the same issue


Import symbols used in version check (through macro)

This fixes an issue for users without use gdnative::prelude::*, which could run into an error regarding symbols not found (e.g. Variant).
bors try

bors retry

bors try-
bors try

bors r+


bug: ionic-starter-slidemenu 404

Type: bug
Platform: desktop browser
Hi,
I'm running ionic on windows (conEmu / Git Bash)
After running ionic start [APP NAME] slidemenu, I'm getting:
Downloading: https://github.com/driftyco/ionic-app-base/archive/master.zip
[=============================] 100% 0.0s
Downloading: https://github.com/driftyco/ionic-starter-slidemenu/archive/master.zip
✗ Not found: https://github.com/driftyco/ionic-starter-slidemenu/archive/master.zip (404)
✗ Please verify the url and try again.
✗ More info available at:
http://ionicframework.com/getting-started/
https://github.com/driftyco/ionic-cli

Greetings @heldrida!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

sorry typo


fix: should get runtime name from container info

when use ctr t checkpoint, runtime name should get from container
info, but not passed by flag, since task has already running with
specified runtime
remove flag ctr task checkpoint --runtime
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>

Codecov Report

Merging #2825 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2825   +/-   ##
=======================================
  Coverage   43.79%   43.79%
=======================================
  Files         100      100
  Lines       10741    10741
=======================================
  Hits         4704     4704
  Misses       5307     5307
  Partials      730      730

Flag
Coverage Δ

#linux
47.45% <ø> (ø)
:arrow_up:

#windows
40.96% <ø> (ø)
:arrow_up:

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 32aa0cd...461222d. Read the comment docs.

👍

LGTM


Add section about managed transfers to the readme

Going to merge, we can revise more in the future if needed.


Confusing / wrong error message

When sending from Winden, if the receiver says "n" in the Python CLI then sending user is presented a confusing error message:
"Connection time out: it looks like the connection between you and the sender was briefly lost".
In fact, the receiving side will have sent an {"error": ...} message back with "transfer rejected" or similar as the error message, so we should be able to tell the user here that (instead of the wrong "connection lost" message).

It won't let me attach the screenshot, but doesn't say why. I tried both PNG and JPEG.

Adding a screenshot:

We receive a message and then even close mailbox with mood error.
We get this messages, as it is default error messages, when we don't match to anything else:
https://github.com/LeastAuthority/winden/blob/7eeba68acedafb5e4180aaa51f34de8c876f0406/client/src/app/util/errors.tsx#L31
However as we receive error: Error details:  SendErr: TransferError: transfer rejected , we should be able to map this error to more proper message.

Fixed by https://github.com/LeastAuthority/winden/pull/104 and https://github.com/LeastAuthority/winden/pull/114


#33 Added Armenian translation of Sergey

hi, can you please resolve the conflict?

Hi, I resolved the conflict.

cool, thanks!
merged #33


”Dognecea”, ”Calina” are listed in the drop-down menu

Description:
When user writes ”Dognecea”, ”Calina” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Dognecea"

Expected results
The location appears in the drop-down menu as "Dognecea, Caraș-Severin(Dognecea)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Calina"

Expected results
The location appears in the drop-down menu as "Calina, Caraș-Severin(Dognecea)".


Make this into an R package

This is clearly a novel and useful Shiny app but this would really be a good R package. That would allow things like tests to be run as well as control which packages are loaded into R. For example, the number of conflicts when loading the Shiny app isn't really good form.

done.


Update upper bounds + compile tweaks

some new upper bounds, patch for newer Aeson, compile fix to exclude whatever weird QC generator for Values was emitting strings with embedded NULs, and a minor tweak to the roundtrip property so counterexamples get printed

let me know if you'd like me to squash the commits

I've pulled your changes into https://github.com/goodlyrottenapple/quickjs-hs/pull/4 and adjusted CI.


Fix MacOS assemble workflows

Description
Fix MacOS assemble workflows
Related Issues
See please https://opensearch.slack.com/archives/C0561HRK961/p1725526478827459

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

FYI its possible to set some CI checks to make them not required through settings by updating Branch protection rules


int to float convertion produced 0.

Normally I used doubles for all interfaces that are taking a float value from python, but recently I was porting code that had to use a explicit float.  In one of the conversions I ended up with a 0 value where an integer was passed into a float.  It appears that one of the conversion routines missed a case and gave canConvert but did not actually perform the conversion.  It wasn't caught in testing so appears we have an incomplete testing of this section.
This may be a recent issue.  I will investigate when I can.

See pull request #300.

Resolved.

It appears to still be broken on 0.6.3 when I call a function that accepted a float.


IGNITE-12747 Correlated queries support

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

The patch fixes the test subquery/test_neumann.test_ignore.
Please unmute the test and close the issue: https://issues.apache.org/jira/browse/IGNITE-14537

@korlov42 i made some code changes, can u take a look plz ?


Pairing fails when providing custom userid and key

Hello,
I am trying to switch from meross app (and cloud) to a local integration. When trying to pair a smartplug mss310 via custom pairer app, it does not pair when providing custom userid and key. If I do not provide it, the pairing works.
What I am doing wrong ?
Thanks for any help!

Hi @Creat1veM1nd ,
the usage of custom userid and key attributes is reserved when pairing against third-party brokers, which might require specific configurations. When using the local-addon you don't need to customize those attributes.
Are you pairing against the local-addon or against other systems?

I paired against a local-addon. But I did not got it working, the problem might be on my end aka misunderstanding the mechanism behind it, I guess. I am using now another integration for my meross because of lack of time.
Since no one else seems to have this issue (at least I did not find anything related), I  guess it can be closed for now. When finding time to dig into it again I will open up a new one, if stumbling across it again. (I am still learning all that stuff,).
I hope, this is ok and thanks for your time!


Add FT81x support to display

The lvgl_esp32_drivers repo contains FT81x support in addition to the current display support.
Would someone fancy migrating the driver to this repo?
Or I could do that if I have time.

I am considering porting the code here.
FT81x chipset has both display and touch support. Where should I contain the code for it? display or indev? Or create a new folder like sdl named ft81x?

Could you consider rather adding it to the LVGL repository directly? For v9 (under development) we add the driver directly to the LVGL repo. See https://github.com/lvgl/lvgl/tree/master/src/dev

I am using the v8.3 stable release for production so I would port the code to v8.3 first and then migrate to v9.
As I am not quite sure when the release date for v9 will be, I would adhere to v8.3 for now.

Note that in this repo is not really maintained. Only a few high level drivers are being fixed (SDL, Wayland, etc).
However if adding the driver here is required step to migrate it to v9 later, I'm ok with that :slightly_smiling_face:


STORM-3476 don't query remote files on cleanup if target size is acce…

…ptable
I'm seeing supervisors querying files over and over on the namenode from this method and never deleting them, adding namenode load.  If we're below the cleanup target size, it seems we should just not bother doing these checks.

+1, I guess there's no harm to not cleaning up unused blobs unless we're over the size limit.


improved script for fetching images from routes

Description

Made the fetch_image_from_route.py script easier to use by:

(optionally) allowing the selection of a specific camera
making authentication optional

Verification

Ran the provided example and various other examples.  Ie.

~/openpilot$ tools/scripts/fetch_image_from_route-modified.py "0c7f0c7f0c7f0c7f|2021-10-13--13-00-00" 3 500 driver


Bug/#113631741

'service/models.py' should now conform with existing database structure.
Correct migrate/upgrade procedure can be followed again but was redundant in this case as no changes to be applied to database.

Agreed to these changes after our findings.

lr-check-code executed successfully, after deleting extra blank line at end of service/models.py


Add fp16 support of Qwen1.5MoE models (A2.7B) to DeepSpeed-FastGen

This PR adds support for Qwen1.5MoE-A2.7B models.
support for https://github.com/microsoft/DeepSpeed-MII/issues/457
Test Code
for mii pipeline:
import mii

pipe = mii.pipeline("/data/zonepg/models/Qwen/Qwen1.5-MoE-A2.7B")
responses = pipe("DeepSpeed is", max_new_tokens=128, do_sample=False)
if pipe.is_rank_0:
    print(responses[0])

for huggingface:
import mii

from transformers import AutoModelForCausalLM, AutoTokenizer
from transformers.generation import GenerationConfig
import torch
tokenizer = AutoTokenizer.from_pretrained("/data/zonepg/models/Qwen/Qwen1.5-MoE-A2.7B")
model = AutoModelForCausalLM.from_pretrained("/data/zonepg/models/Qwen/Qwen1.5-MoE-A2.7B", device_map="auto", torch_dtype=torch.float16, trust_remote_code=True).eval()
print(model)
inputs = tokenizer('DeepSpeed is', return_tensors='pt')
inputs = inputs.to(model.device)
pred = model.generate(**inputs, max_new_tokens=128, do_sample=False, repetition_penalty=1.0)
test = tokenizer.decode(pred.cpu()[0], skip_special_tokens=False)
print(test)

Qwen1.5-MoE-A2.7B
Huggingface output with prompt "DeepSpeed is":
 a deep learning framework that is designed to accelerate the training of large-scale neural networks. It is built on top of PyTorch and provides a set of tools and techniques for optimizing the performance of deep learning models.

DeepSpeed supports a variety of hardware accelerators, including GPUs, TPUs, and FPGAs, and can be used to train models on distributed systems, such as clusters of GPUs or TPUs.

One of the key features of DeepSpeed is its ability to automatically parallelize the training of deep learning models across multiple GPUs or TPUs. This can significantly reduce the time required to train large models, as it allows the

DeepSpeed-FastGen output with prompt "DeepSpeed is":
 a deep learning framework that is designed to accelerate the training of large-scale neural networks. It is built on top of PyTorch and provides a set of tools and techniques for optimizing the performance of deep learning models.

DeepSpeed supports a variety of hardware accelerators, including GPUs, TPUs, and FPGAs, and can be used to train models on distributed systems, such as clusters of GPUs or TPUs.

One of the key features of DeepSpeed is its ability to automatically parallelize the training of deep learning models across multiple GPUs or TPUs. This can significantly reduce the time required to train large models, as it allows the

DeepSpeed-FastGen output with prompt "DeepSpeed is" with 8-way sharding:
 a deep learning framework that is designed to accelerate the training of large-scale neural networks. It is built on top of PyTorch and provides a set of tools and techniques for optimizing the performance of deep learning models.

DeepSpeed supports a variety of hardware accelerators, including GPUs, TPUs, and FPGAs, and can be used to train models on distributed systems, such as clusters of GPUs or TPUs.

One of the key features of DeepSpeed is its ability to automatically parallelize the training of deep learning models across multiple GPUs or TPUs. This can significantly reduce the time required to train large models, as it allows the

When I use your source code to build DeepSpeed，and run “for mii pipeline” code， the process is blocked and no error。How should I identify the problem? I use 4090 GPU and transformer is 4.41.0.dev0, torch is 2.2.1, cuda version is 11.8.
BTW  the transformer code runs well , just runs very slow.

Hi, @heiseon  I just created a new conda environment and built it from my deepspeed code and deepspeed-mii offical source code, and it’s ok without any issues.
maybe you can delete ~/.cache/torch_extensions/pyxxx_cuxxx and try it again.
my path is /data/zonepg/.cache/torch_extensions/py311_cu121.

Hi, @heiseon I just created a new conda environment and built it from my deepspeed code and deepspeed-mii offical source code, and it’s ok without any issues.
maybe you can delete ~/.cache/torch_extensions/pyxxx_cuxxx and try it again.
my path is /data/zonepg/.cache/torch_extensions/py311_cu121.
delete ~/.cache/torch_extensions/pyxxx_cuxxx  is worked for me.
I have another question. When using the Qwen1.5-MoE-A2.7B-Chat-GPTQ-Int4 quantized version, an error occurs: 'Could not find a mapping for dependency "mlp.experts.18.gate_proj.bias"'. Does it mean that the GPTQ quantized version of the model is not supported?"
Would you like assistance with addressing this issue or understanding more about GPTQ quantization compatibility?

Hi, @heiseon It does not support quantized Qwen models currently. Supporting this would likely require a big effort, so it might not be considered in the short term.


The pixel damage somewhat still activates when in creative mode or dont have fall damage on

It think it damages you and does the effect but i am in creative

Can you tell the game and mod version please?


Display multiple spaces properly in Confirm and Validate dialog

Issue
https://github.com/gravitee-io/issues/issues/8867
https://gravitee.atlassian.net/browse/APIM-779
Description
Display multiple spaces properly in Confirm and Validate dialog

🧪  Gravitee.io Automatic Prerelease
npm install<EMAIL_ADDRESS>
yarn add<EMAIL_ADDRESS>

📚  View the storybook of this branch here

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request]: Sugarcane Leaf Disease Detection

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This project focuses on classifying sugarcane leaf diseases using different machine learning models. The dataset contains images of sugarcane leaves classified into various categories. Image preprocessing, data augmentation, and ensemble learning methods are applied to boost the model’s accuracy.
Use Case

Early Disease Detection:
•	Farmers can detect diseases in sugarcane crops at an early stage, enabling timely intervention and minimizing crop damage
Automated Crop Monitoring:
•	The system can be integrated with drones or mobile apps to automatically monitor large sugarcane fields for disease symptoms, reducing the need for manual inspections.
Improved Crop Management:
•	With real-time disease detection, farmers can optimize pesticide use, applying treatments only when necessary, which reduces costs and minimizes environmental impact.
Yield Improvement:
•	Early and accurate disease detection helps farmers take proactive measures, leading to healthier crops and potentially increasing overall yield.
Disease Pattern Analysis:
•	Researchers and agronomists can use the classification system to study the prevalence and spread of different sugarcane diseases, helping in developing resistant crop varieties.
Decision Support for Farmers:
•	The model can be part of a decision support system that provides recommendations on disease management strategies based on the detected disease type.

Benefits
The integration of sugarcane leaf disease classification into agrotech AI systems can offer several key benefits:

Enhanced Crop Monitoring Capabilities:
•	AgroTech AI systems equipped with disease classification models can provide real-time, automated crop health assessments, increasing efficiency in monitoring large-scale farms without the need for manual inspections.
Cost Reduction:
•	The early identification of diseases through AI reduces the need for blanket pesticide applications. Farmers can save costs by applying treatments only when necessary, leading to more targeted and efficient use of resources.

Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Pls assign me under gesso-extd and hacktoberfest

Can you please assign me this issue under gssoc-ext

hi @Sana1902 thank you for your interest.
as of now @IkkiOcean is working very hard on this issue..
according to gssoc rules.admins cannot be able to assign one issue to more than one person at a time.
you can raise a new issue after exploring agrotech ai..you come up with new ideas
thank you @Sana1902 .
its great to have you in our contributors family here in agrotech ai..expecting more from you..
all the best.. @Sana1902

@manikumarreddyu I have my PR ready with a Ensemble model consisting of Vgg16, Conv2D, ResNet152 and InceptionV3 with accuracy of .
I'm putting the notebook in /notebooks directory with a model file (.h5). Is there anything else I should be aware of?

dataset link in readme file along with project ->sugarcane leaf disease  detection description..

dataset link in readme file along with project ->sugarcane leaf disease detection description..

I don't think I will be able to upload .h5 /.keras model files since the size exceeds its 100MB limit


mock: change use of strncat_s to strcat_s

safestr does not currently include strncat_s so this changes mock.c to
use strcat_s instead.

Coverage remained the same at 69.332% when pulling ab24fb6b24d746404361c567b394f0afe8545880 on rrojo/mock-safestr into 00f144208b9120b7877bfa3fc67e886e0effca79 on master.


Multiple stores doc

PR for issue: https://github.com/solid/userguide/issues/31

Thank you @TallTed and @bourgeoa for the feedback. I consider this PR properly reviewed.


linear_composite needs blacklist

the function needs a blacklist argument similar to others, such as Basket init:
    :param <dict> blacklist: cross-sections with date ranges that should be excluded
        from the DataFrame. If one cross-section has several blacklist periods append
        numbers to the cross-section code.

To be closed by #891


Usage in composable

Hi thanks, for the module!
Quick question: the module works fine in components, using this.$notify, but I'm trying to use it in a composable, in this way:
const notification = useNotification();
notification.notify({ type: "error", text: "test" });

but it doesn't show any notification, no error. Am I doing something wrong?

@mfrascati Hi. I'm glad it was helpful to you
It's a bug @kyvg/vue-notification3, i think...
After reload page notification is work fine)
You tried?

@mfrascati I will try to deal with this problem. thank you very much for issue!

Hi. Please reopen this, the notification is still not working.

I'm unable to get this to work as well.
Here is a Stackblitz project: https://stackblitz.com/edit/nuxt-starter-tjg71j
I had to disable ssr as well, because that generated a lot of hydration mismatch errors.

I'm unable to get this to work as well.
Here is a Stackblitz project: https://stackblitz.com/edit/nuxt-starter-tjg71j
I had to disable ssr as well, because that generated a lot of hydration mismatch errors.
UPDATE with solution:
put your <NuxtNotifications /> component in the root of your app.vue and right before the closing </template> tag

Hi.
Thank you for detailed reproduce.
I see that this doesn't work and I'll try to figure it out.

Extra option to fix this: switch ssr to true in you nuxt.config.ts

@BramEsposito Hi!
Can u try clean .nuxt and node_modules/.cache and run dev build again?

Hi everybody!
Put your hands up 😻
Can anyone check that this problem is resolved in the new version? v1.2.2
@BramEsposito @daviddomkar
Playground
Thank you very match!

Close because inactive

Sorry, I couldn't make time before today.
I have reproduced the resolution and can confirm this works now with these versions:

nuxt: 3.12.2
nuxt3-notifications: 1.2.2

as you can see here: https://stackblitz.com/edit/nuxt-starter-tjg71j
thanks for your awesome job!

Sorry, I couldn't make time before today.
I have reproduced the resolution and can confirm this works now with these versions:

nuxt: 3.12.2
nuxt3-notifications: 1.2.2

as you can see here: https://stackblitz.com/edit/nuxt-starter-tjg71j
thanks for your awesome job!

Hi!
Thanks for reproduce!
And big thanks for idea use this code
const app = useNuxtApp();

const notify = app.vueApp.config.globalProperties.$notify;


ndarray 0.13 + build_dim_array

Fix #58

Update to ndarray 0.13 and required fixes (from_iter, all_close and approx). Version 0.13 is now required for nifiti-rs.
As discussed, I moved the approx crate from "dependencies" to "dev-dependencies".
There some autoformat in this PR. I added #[rustfmt::skip] 3 times to avoid matrix formatting. I think formatting in Rust is great but not for matrices! I don't like using skip, but I consider it essential for matrices.
The Create build_dim_array function commit doesn't fit in this PR, sorry. I'm not a fan of creating PR for tiny changes like that, but maybe you prefer it cleanly separated. Just tell me and I won't do it again :)

As there's no specific PR or issues that we're currently working on, would it be possible to get a release with ndarray 0.13? Maybe updating num-derive before, if you do a release.
nifti-rs is the only missing lib for ndarray 0.13 in our project and we're kind of excited about it :)

Yes, it's a nice time for another release, but we might as well merge #61 and think a bit about #60 first. :+1:


Create README for release process

I think the closest we have on this is in CONTRIBUTING.rst on gcloud-python? https://github.com/GoogleCloudPlatform/gcloud-python/blob/master/CONTRIBUTING.rst
maybe we call this RELEASING.md ?

I'm going to document the release process in this issue for now.  We can comment on it, update it, and let the documentation live elsewhere once we've made the necessary changes.

Overview
Most of the release process is handled by the after_success.sh script, triggered after Travis CI successfully completes a non-PR build.  A new artifact will be released to Maven Central Repository via Travis CI when "-SNAPSHOT" is not included in the version (as listed in the base directory's pom.xml).  The website and README files will also be updated automatically in this case.  When "-SNAPSHOT" is included in the version, Travis only updates the artifact in the snapshot repository.
To push a release version

Run utilities/update_pom_version.sh from the repository's base directory.
This script takes an optional argument denoting the new version.  By default, if the current version is X.Y.Z-SNAPSHOT, the script will update the version in all the pom.xml files to X.Y.Z.  If desired, another version can be supplied via command line argument instead.

Create a PR to update the pom.xml version.
After this PR is merged into GoogleCloudPlatform/gcloud-java, Travis CI will push a new website to GoogleCloudPlatform/gh-pages and push a new artifact to the Maven Central Repository.

Run utilities/update_pom_version.sh again (to include "-SNAPSHOT" in the project version).
As mentioned before, there is an optional version argument.  By default, the script will update the version from "X.Y.Z" to "X.Y.Z+1-SNAPSHOT".  Suppose a different version is desired, for example X+1.0.0-SNAPSHOT.  Then the appropriate command to run would be utilities/update_pom_version.sh X+1.0.0-SNAPSHOT.

Create and merge in another PR to reflect the updated project version.

To push a snapshot version
Pushing a snapshot is completely automated.  If "-SNAPSHOT" is included in the version denoted by the base directory's pom.xml, then an updated artifact will be pushed to the snapshot repository when Travis CI successfully completes a non-PR build.
Improvements
Tagging is not currently implemented, though it was discussed in #119.  If the version updates continue to be manual, a one-line git tag command can be added to after_success.sh to correctly tag releases.  However, if the release process becomes fully automated, tagging becomes a harder problem, as mentioned in that issue.

Looks good and I am fine with having it in RELEASING.md.
As for the Improvements section. In the manual process how can we add the tag to  after_sucess.sh
if that one is invoked by Travis?
Is the tag needed to be included in the local git before pushed or should we add another manual step after (2) to create a gitHub release?

I was thinking that we could add a lightweight tag (git tag v${SITE_VERSION}), then push it to the remote.  However, it'd probably be nice to have some sort of annotation denoting what the release contains, and that's hard to automate.  Also, since tags have to be unique, if something goes wrong with the release (say the build that triggers a github push passes, but the builds for java 8 fails because of javadoc errors), then we'd be in the business of deleting remote tags to fix the release, which is something we want to avoid.

@ajkannan I think for the moment we can tag the commit at the end of after_success.sh (after publishing the artifact, once we know that everything worked as expected). We can then manually add annotations to the tag (changelog, acknowledgements, etc...)

After thinking about it a bit more, I think adding the lightweight tag, even at the end of after_success.sh, adds only a little while introducing extra risk.  It's very possible to accidentally merge in other commits between the two pom.xml version change PRs, and if one build (say the java8 build) were to fail for platform-specific reasons and we accidentally merge the commit in anyways, then we'd have to spend time and effort to delete/move tags around.
If we're going to anyways manually add annotations to the tag, there's a option to "create tag on publish," when drafting the release, so I now support creating tags manually (for now at least).

@ajkannan Sounds good to me. Even though we could assume we are careful enough not to merge other commits in between:)


Services are sorted by create date

Hello,
Today I discovered that services are sorted by create date:

The same on a booking page. Should it be that way? I think sorting by name is more intuitive.
(Sorry if I posted it in a wrong issue tracker)
Greetings

Thanks @Chotimir , from an API perspective, you can ask the results to be sorted by a particular property.
For example:
GET https://graph.microsoft.com/beta/bookingBusinesses/{businessId}/services?$orderby=displayName

For the UI itself, use the feedback channel to vote on this idea:
https://outlook.uservoice.com/forums/314907-microsoft-bookings/suggestions/16782892-more-sorting-options-for-services


Create Search_in_Rotated_Sorted_Array.java

To Do: Given the array nums after the possible rotation and an integer target, return the index of target if it is in nums, or -1 if it is not in nums.
Describe the solution : Write an algorithm with O(log n) runtime complexity
Solution using Binary Search
Please assign this to me under ssoc'23. I have submitted the PR

Add line comments so its easier to follow

It's done please let me know if it's alright. @akgmage


Issue with color and Markdown formatting in rc.46

Trying out the tldr extension and I am  getting the following code showing for cat
   [1m[39mcat [22m[39m [39mPrint and concatenate files.[22m[39m [39mMore information: https://keith.github.io/xcode-man-
   pages/cat.1.html.[22m[39m [32m

   • Print the contents of a file to  stdout :[22m[39m
   [31m    cat [22m[39m[34mpath/to/file[39m[31m[22m[39m
   [32m
   • Concatenate several files into an output file:[22m[39m
   [31m    cat [22m[39m[34mpath/to/file1 path/to/file2 ...[39m[31m > [22m[39m[34mpath/to/output_file[39m[31m[22m[39m
   [32m
   • Append several files to an output file:[22m[39m
   [31m    cat [22m[39m[34mpath/to/file1 path/to/file2 ...[39m[31m >> [22m[39m[34mpath/to/output_file[39m[31m[22m[39m
   [32m
   • Copy the contents of a file into an output file without buffering:[22m[39m
   [31m    cat -u [22m[39m[34m/dev/tty12[39m[31m > [22m[39m[34m/dev/tty13[39m[31m[22m[39m
   [32m
   • Write  stdin  to a file:[22m[39m
   [31m    cat - > [22m[39m[34mpath/to/file[39m[31m[22m[39m
   [32m
   • Number all output lines:[22m[39m
   [31m    cat -n [22m[39m[34mpath/to/file[39m[31m[22m[39m
   [32m
   • Display non-printable and whitespace characters (with  M-  prefix if non-ASCII):[22m[39m
   [31m    cat -v -t -e [22m[39m[34mpath/to/file[39m[31m[22m[39m

I tried it both in the Warp terminal and in the VSCode one

strange I'm not able to reproduce the bug. Does it only happen in the cat command ?

It was for all commands. I managed to get to the bottom of it though :-)
My issue was that I had tldr the https://tldr.sh/ version and not tldr the tealdeer rust version. The first version doesn't have a --raw option and was returning colored text.
Long story short, I fixed it with brew remove tldr and brew install tealdeer

I should probably just strip color codes from the command output, thanks for the explanation.


Support multiple targetFields (or locale support)

I'm working on a project with i18n, where it would be useful to provide a locale to the preview URL in addition to a slug.
I wonder if it might be helpful for others if the targetField option could support more than one field which is included for the front-end query - in my case it would be slug and locale - so that a config such as this:
'preview-button': {
  enabled: true,
  config: {
    contentTypes: [
      {
        uid: 'api::page.page',
        targetField: ['slug', 'locale'],
      },
    ],
  },
},

would result in a URL:
http://example.com/api/preview?slug=your-page&locale=en&secret=YOURSECRET

@muhzain Please see my comments above. If I'm understanding correctly, it looks like you are wanting every param of the config to be dynamic. Can you provide more details on this scenario?

@mattmilburn Your plan makes sense to me. And at least for my use case, the query string isn't needed for the published URLs.

Hi,
I've extended the plugin for our company's needs (really easy by overriding the findOne function) and could create a PR if you are interested.
The path configuration accepts placeholders with double curly braces like this /{{locale}}/c/{{slug}}/{{id}} and replaces them with the corresponding entity fields.

Hi,
I've extended the plugin for our company's needs (really easy by overriding the findOne function) and could create a PR if you are interested.
The path configuration accepts placeholders with double curly braces like this /{{locale}}/c/{{slug}}/{{id}} and replaces them with the corresponding entity fields.
The proposal above wouldn't work for us since we have the namespace after the locale in our urls.

This would be useful for our use case, too. We are in the same situation currently.

Hi,
I've extended the plugin for our company's needs (really easy by overriding the findOne function) and could create a PR if you are interested.
The path configuration accepts placeholders with double curly braces like this /{{locale}}/c/{{slug}}/{{id}} and replaces them with the corresponding entity fields.
The proposal above wouldn't work for us since we have the namespace after the locale in our urls.

This would be really useful for us too. Would you be able to share your solution?

I fixed it by overriding the findOne function in the preview-button controller:
const pluginId = 'preview-button';

module.exports = (plugin) => {
  plugin.controllers['preview-button'].findOne = findOne;
  return plugin;
};

async function findOne(ctx) {
  const { uid, id } = ctx.request.params;

  const pluginService = getService('preview-button');
  const { contentTypes, baseUrl } = await pluginService.getConfig();
  const supportedType = contentTypes.find(type => type.uid === uid);

  // Collection types will find by the ID and single types do not.
  const findParams = id ? { where: { id } } : {};
  const entity = await strapi.query(uid).findOne(findParams);

  const urls = getPreviewUrls(baseUrl, entity, supportedType);

  ctx.send({ urls });
}

function getPreviewUrls(baseUrl, entity, contentType) {
  const { path } = contentType;
  const url = replacePlaceholders(baseUrl + path, entity);

  return {
    draftUrl: url,
    publishedUrl: url
  };
}

function replacePlaceholders(url, entity) {
  function replace(_, propertyName) {
    return entity[propertyName];
  }

  // Matches everything between double curly braces (https://stackoverflow.com/a/57117346/8059242)
  return url.replace(/{{(([^}][^}]?|[^}]}?)*)}}/g, replace);
}

const getService = name => {
  return strapi.plugin(pluginId).service(name);
};

(save the code to src/extensions/preview-button/strapi-server.js)
In our case draftUrl equals publishedUrl, if you need separate urls you have to adapt the getPreviewUrls function.
My plugin-config.js looks like this:
module.exports = ({ env }) => ({
  'preview-button': {
    enabled: true,
    config: {
      baseUrl: 'https://www.oktav.com',
      contentTypes: [
        {
          uid: 'api::entity.entity',
          path: '/{{locale}}/asdf/{{slug}}?preview=' + env('PREVIEW_SECRET'),
        }
      ],
    },
  }
});

Hi @zrothauser, been a while! The Strapi preview button plugin has been updated with breaking features and now supports localization in the URLs as well as other new features 👍🏻


I'm unable to login

Very Annoying unable to login.
https://streamable.com/654ayw

Very Annoying unable to login.
https://streamable.com/654ayw

Hi, thanks for reaching out, the link you attached seems to be broken, we tested it out again, the features work for us, could you make sure that the link you attached is working correctly so that we can guide you through the issue you are facing,
Also can you confirm that you had confirmed you email via the code received either in your spam/inbox,
I guess the video service you've used for the screen record seems to be broken, you can also use alternatives like shareX, recordit.

copy paste the link in web browser
https://streamable.com/654ayw

Can you embed the video here? @procodervit69?
Or use Vimeo or something 😄

No.
Security issues for me.
Codechef people will threaten me.

Could you give us steps to recreate your problem and what error you’re facing?
We don’t want to open a malicious link either 😃

@procodervit69 please let us know if the bug you mentioned is now fixed.
Thanks for your help 😄


Misleading Request.Depth documentation

Currently, the documentation for Request.Depth claims "Depth is the number of the parents of the request".
However, the very first request has a Depth of 1, even though it clearly has no parents.

Indeed. Off by one.. =]


tag protection via rest api

Feature Description
The tag protection function is available since version 1.15.0 (https://github.com/go-gitea/gitea/issues/15545).
It would be helpful to manage tag protection for a lot of repositories when a api like branch protection would be available.
/repos/{owner}/{repo}/branch_protections --> /repos/{owner}/{repo}/tag_protections
Screenshots
No response

I follow this request, it would be useful for those who have many repositories to manage.

Guys, I need this feature too!


Exception handling?

in Calculate method, randomly throw exception or a deadlock or sleep for next 10 years. what happens main thread reach WaitHandle.WaitAll(doneEvents)?

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

ID: 96a63b62-dd9b-aea0-87e8-2219dc37df69
Version Independent ID: 364eb267-f008-89cf-4fb1-157fdf80fa15
Content: How to: Use a Thread Pool (C#)
Content Source: docs/csharp/programming-guide/concepts/threading/how-to-use-a-thread-pool.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

NOTE: the referenced page was removed by PR #6739 and redirects to the ThreadPool.QueueUserWorkItem API page. The code example was also moved to that API page (QueueUserWorkItem(WaitCallback, Object) example). So, the original question stays actual.

Thanks for writing this @bobsort and adding the clarifying comments @pkulikov
These are important questions that should be answered as part of this sample.
I've added this issue to our backlog, and to our project on async / parallel and related programming concepts.


File highlight start/end line number option

I incorporated the data-start attribute from the line-numbs plugin and added a corresponding data-end attribute. If either is present, the download file is split into lines and slice'd to the correct subset before highlighting.
Plays nice with line-numbers plugin, and treats negative data-start's as 0.
I included an example on the page.

I like the original idea. But IMO the regexp option would only solve some uncommon use cases and so is not really nedded.

@Golmote hah, I got pretty much the exact opposite response from a couple of my coworkers - they thought that line numbers were relatively annoying and would much prefer being able to add a comment to the code to mark the start and end of the snippet to show.
However, I'll admit that it might be a little bit uncommon, and that's part of why I hadn't included it yet. (Also, I was hoping to hear from @LeaVerou before putting much more work into this.)

Hey, we should probably just close this. I don't work at the same employer anymore and I don't think they're still using any of this :/


One Drive doesn't works

Hi Yang,
One Drive link is dead

Sorry. The link in our school can only exist for 3 months. The following is a new one.
https://uofr-my.sharepoint.com/:f:/g/personal/yfeng23_ur_rochester_edu/EgDosCuY5t9HmlBfFsVyxdAB4xGf6aTJ0DmQlYWASdjYsw?e=GsuSkD

Thanks


Missing examples

The examples are missing: https://trpc-nuxt.vercel.app/examples/basic
See image below

Closing this. I will be updating the docs site to use Vitepress instead since docus is not maintained anymore.


[Feature] self-cleaning

Implement trash note
now you can move note to trash after 28 will auto delete also you can delete and restore+
Uploading bandicam 2023-10-17 00-33-33-747.mp4…

Can you please fix the ktlint checks @salmanA169.


chore(main): release cyclistsofmsn-client 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-04-03)
Features

show versions in client ui (302bf39)

Bug Fixes

test failure due to global vitest import issue (965249b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/madisonbikes/cyclistsofmsn-client/releases/tag/cyclistsofmsn-client-v1.1.0 :sunflower:


Using XML-JS with EnyoJS

Hey,
I apologize if this is posted in the wrong place, but looking at the XML-JS's Quick Start example we have
var convert = require('xml-js');
            var xml =
                '<?xml version="1.0" encoding="utf-8"?>' +
                '<note importance="high" logged="true">' +
                '    <title>Happy</title>' +
                '    <todo>Work</todo>' +
                '    <todo>Play</todo>' +
                '</note>';
            var op = { compact: true, spaces: 4 };
            var result1 = convert.xml2json(xml, op);
            console.log(result1);

Converting back,

        console.log(convert. json2xml(result1, op));

Which gives the following error:
json2xml.js:4 Uncaught ReferenceError: Buffer is not defined
    at Object.module.exports [as json2xml] (json2xml.js:4)
    at enyoConstructor.handleExport (DataControl.js:357)
    at enyoConstructor.dispatch (Component.js:779)
    at enyoConstructor.dispatchEvent (Component.js:687)
    at enyoConstructor.dispatchEvent (Control.js:1311)
    at enyoConstructor.dispatchBubble (Component.js:734)
    at enyoConstructor.bubbleUp (Component.js:645)
    at enyoConstructor.dispatchEvent (Component.js:716)
    at enyoConstructor.dispatchEvent (Control.js:1311)
    at enyoConstructor.dispatchBubble (Component.js:734)
    at enyoConstructor.bubble (Component.js:617)
    at Object.dispatchBubble (dispatcher.js:161)
    at Object.dispatch (dispatcher.js:115)
    at Object.sendTap (gesture.js:142)
    at Object.up (gesture.js:102)
    at Object.mouseup (gesture.js:295)

Other than what's up there, I grabbed xml-js through
npm install --save xml-js

Thoughts?

Are you running this on an older version of IE by any chance?  You may need to require Enyo's Buffer shim (or, apply your own).

I'm using Chrome.
How do I access Enyo's Buffer?

Sorry, turns out I was having a bit of a brain fart.  It was Blob that we had polyfilled for older browsers.  I think given that it's trying to access Buffer that it is getting confused about being packed by our node build tools.  Is there some way to force the library to use the browser version?  Buffer is a node concept.


Uploading Messages via IoTMessage fails sometimes

This issue is for a: (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
When deploying the IoTMetricsCollector via IoTHub, you can set the environment variable UploadTarget = IoTMessage .
It worked for 12 hours after deployment then stopped. Worked 10 mins again during the night and then stopped working. Changed deployment to UploadTarget = AzureMetrics (works) and then reverted to UploadTarget = IoTMessage. I did this a few times. Reverting to UploadTarget = IoTMessage works only sometimes, not always
Any log messages given by the failure
N/A
OS and Version?
Linux Ubuntu Server 18
Versions
edgeHub and edgeAgent 1.2.3
IoTEdgeMetricsCollector 1.0
I Have no specific logs that might help. It seems that the device, log analytics and function app work, but it is then not possible to visualize the data on the IoTHub Workbook.

@robertokcanale your failure is probably related to the problem described in this PR.
You can either wait for the merge or just update your current Function App Configuration by adding WEBSITE_LOAD_USER_PROFILE  with value 1. If you try it, please let me know if the failure is fixed.

@MagdaPaj thank you for your comment and directives.
I just made the appropriate changes and it worked! IoTMessages started coming in in less than two minutes.
To me, the failure seems fixed (I'll monitor it in the next 48 hrs and will let you know if something goes wrong).
Thanks


feat(SPV-815): refactor how the outputs in draft transaction are created

Pull Request Checklist

[ ] 📖 I created my PR using provided  : CODE_STANDARDS
[ ] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
[ ] 🏠 I tested my changes locally.
[ ] ✅ I have provided tests for my changes.
[ ] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[ ] 💾 PR was issued based on the Github or Jira issue.

Codecov Report
Attention: Patch coverage is 66.99029% with 34 lines in your changes missing coverage. Please review.

Project coverage is 48.92%. Comparing base (a53cf1e) to head (a810d05).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #608      +/-   ##
==========================================
+ Coverage   48.86%   48.92%   +0.05%
==========================================
  Files         259      259
  Lines       12137    12162      +25
==========================================
+ Hits         5931     5950      +19
- Misses       5612     5616       +4
- Partials      594      596       +2

Flag
Coverage Δ

unittests
48.92% <66.99%> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

engine/action_transaction.go
36.27% <0.00%> (-0.36%)
:arrow_down:

engine/model_draft_transactions.go
76.06% <69.00%> (+0.70%)
:arrow_up:

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


RSA generate endpoint does not support key size of 256

Plugin version
v1.0.16
netbox-plugin-extensions version
None
NetBox version
v3.2.1
Python version
3.8
Steps to Reproduce
Send a request to http://localhost:8000/api/plugins/netbox_secretstore/generate-rsa-key-pair/?key_size=256 and inspect the response
Expected Behavior
The response should contain the data generated using 256 key size
Observed Behavior
The key size used is 2048

Did you post or get?

I did GET request

The response should contain the data generated using 256 key size

256 is not a valid value for RSA, modulus must be at least 1024.


Inject BrPageService for each br-page component separately.

Your suggestion was implemented in the release 19.0.2 of the SDK. Thanks again @Meightem.


Installation error with python3.5

After installing (presumably) correctly on python3.5, I get an ImportError:
ImportErrror: No module named 'mie._mie'

Did you start the your script from within the py-mie directory?

@thisch Yes. It appears to install correctly, which I find strange if it truly can't locate the extensions...

I fixed this by adding support for setuptools installation rather than numpy.distutils.core
I'm not sure why that worked, but it did..

Building for Python 3.5 works on Travis, so more investigation is needed here.


Garmin Instinct 2

Please add support for Garmin Instinct 2.
Thank You :)

Is there anything missing not working? I get most of the known values from Garmin Connect site.


feat: exposed installation method in debug mode

JIRA card: INSTA-778

[x] test in local with --import flag: node --import<EMAIL_ADDRESS>./index-esm.mjs
[x] test in local with --require flag: node --require ./node_modules/@instana/collector/src/immediate ./index.js
[x] test in local with no flags: node ./index.js

commit messge to main:
feat: exposed installation method in debug mode
refs INSTA-778

For the normal commonJS instrumentation using require('@instana/collector')(); in the code level currently considered as default and the debug log will print like

The app isnstrumented instana using: no additional flags

commit messge to main:
feat: exposed installation method in debug mode
refs INSTA-778

For me, this isn't a feature but rather a fix, which is useful for troubleshooting. It doesn’t add any new functionality for the customer.

I'm done with the review. Added one more comment.

LGTM!


Local variable is assigned but never used (F841) cannot be ignored with dummy-variable-rgx in except block

Example:
try:
    1 + 1
except Exception as _err:
    print("Problem")

Ruff always triggers F841 on the except Exception as _err line:
F841 [*] Local variable `_` is assigned to but never used

Ruff version 0.0.282

Flake8 doesn't respect the dummy-variable-rgx here either -- but what's the use-case? Did this occur in a real example, where you want to assign the exception but not use it?

Flake8 doesn't respect the dummy-variable-rgx here either.

Hm, that is interesting.
Originally I had written this in a project:
try:
    1 + 1
except:
    logger.exception("Cannot add numbers :(")

And then I got feedback that it would be nice to capture the caught exception in a variable for easy access in pdb and other debuggers. So I thought it would be simple to rewrite as:
try:
    1 + 1
except Exception as _err:
    logger.exception("Cannot add numbers :(")

and was surprised with as _err got rewritten by the autofixer. What I ended up shipping was this:
try:
    1 + 1
except Exception as err:
    logger.exception("Cannot add numbers :(", exc_info=err)

Maybe there is a slight benefit in this final solution, as logger.exception(msg) is equivalent to logger.error(msg, exc_info=True) which under the covers gets the original exception with sys.exc_info().
So it's an easy workaround and maybe worthwhile to keep the exact same behavior as flake8 in this scenario.

it would be nice to be able to capture the caught exception in a variable for easy access in pdb and other debuggers
This is the tail wagging the dog, IMHO. My team complained about RET504 removing the variable they introduced for the sole purpose of debugging. I held firm, as I don't believe we should allow the code to become messier for the sake of debugging practices.

Yeah, hmm... IMO it's strange to use that as _err pattern, but it's also surprising not to respect the dummy-variable-rgx here.

removing the variable they introduced for the sole purpose of debugging

@tylerlaprade if it makes you feel any better for holding firm, there are other ways to expose that return value.
In Pycharm you can check the 'Show Return Values' option

In pdb, you can use __return__, detailed tip from SO

I have a similar use case by for walrus assignment:
    while _has_not_timed_out := (last_try_epoch_sec - start_epoch_sec) < timeout_sec:
        if _reached_retry_interval := (time() - last_try_epoch_sec) >= retry_interval_sec:

is this resolved or has been any possible workaround this issue? couldn't get it working with walrus operator either. below is my code sample
def _strides_as_shape(
    shape: t.Sequence[int],
    itemsize: int,
) -> tuple[int, ...]:
    """Computes the memory strides for an array based on its shape and
    element size.
    """
    stride = itemsize
    return tuple((stride := stride * dim) // dim for dim in reversed(shape))[  # F841 local variable 'stride' is assigned to but never used
        ::-1
    ]


[RFC] From Releases to Deployments

What's the problem? 🤔
Releases are a too high level concept for basic users. We should support the simplest workflow to promote your app to production. For pro plan we can think of more elaborate use cases
What are the requirements? ❓
A user must be able to "save" the state of their application to share with stakeholders without locking themselve into being blocked from making any changes.
What are our options? 💡
We currently have the releases, where we let users create snapshots and have them switch freely between them to designate one production snapshot.
Proposed solution 🟢
We would focus less on releases and more on the "deployment". This would simplify the way users think about the app lifecycle. Working in the editor would "stage" the changes, and a deploy button would promote the state to production. Concretely:

Instead of the "release" button, we provide a "deploy" button, after confirmation it will promote the current state to production, the underlying release management should be transparent for (CE) users.
Instead of the releases tab, we provide a "deployments" tab, that lists all the previously made deployments, with the most recent one up top. Optionally, but not necessarily for CE we could provide for each deployment:

a permalink to view the app in that state
a rollback button to create a new deployment based on a older one

Relevant resources/benchmarks 🔗
Alternative to https://github.com/mui/mui-toolpad/issues/795#issuecomment-1229171634

How is this different from releases:

releases currently are about versioning, not about deploying. The list of releases is basically the list of versions through time, the latest one isn't necessarily what's deployed. This RFC instead proposes to make the list of deployments be a chronological record of the use of the deploy button, and therefore be less about "versioning" and more about tracking a history of pushing changes to rpoduction.

The direction this RFC is going is interesting. What I hear is that our "Releases" currently merge two different notions, each equivalent to: git commits and Netlify deployments. We could focus on deployments. A deployment would create a commit, only that there can't be a commit that is not deployed.
Now, I don't think that this directly closes #795, since this issue stated from https://www.notion.so/Toolpad-Pricing-50df8f336b3d4120b8df8a8cef0b7fa9?d=0af1ff513e5b4bdf9bfe3f748d10035d#de48ce63c7c34c149c2aa68a57299e9d. If we consider that the community plan is about spinning the contributions flywheel, we have to have enough features for the basic use cases. What does a user with a basic use case need? I think that it needs to allow you to undo your mistakes, there wouldn't be a worse DX if it was not the case. But to deploy an older version? Personally, I didn't need so far with Pipedream, outside of a git reset of my working environment, same with Toolpad.

The direction this RFC is going is interesting. What I hear is that our "Releases" currently merge two different notions, each equivalent to: git commits and Netlify deployments. We could focus on deployments. A deployment would create a commit, only that there can't be a commit that is not deployed. Sounds simpler 👍

Hitting the nail on the head. Currently the underlying datamodel already separates both concepts. Practically it's a matter of

Shifting the UI from visualizing the "commits" towards visualizing the "deployments".
Creating a release and deploy at the same time

This won't be too big of a change


boost stacktrace and fiber libraries are missing when building 1.78.0 from source

Hi,
I am trying to build Boost 1.78.0 from source in Linux but have noticed that the stacktrace and fiber libraries are missing. I am using the following command to build boost.
bootstrap.sh
b2 -j8 --build-dir=/opt/disk1/boost/boost_build --prefix=/opt/disk1/boost_install/ --without-python address-model=64 toolset=gcc threading=multi variant=release install
I couldn't see any error in the log as well. So other libraries apart from these are building fine. I am using GCC 7.2.0 compiler, I tried with intel 2022 compiler as well but no luck.
Please suggest if I am missing something here.
Thanks.

Even better: https://github.com/bfgroup/b2/issues/104

Ok, I will try 1.79.0
Thanks.


Request for advice for the Audubon Core MG about ac:associatedXReference terms

I think that it would be good as a part of the upcoming discussion of proposals related to #332 #331 #330 #324 and #283 to consider two Audubon Core terms: ac:associatedObservationReference and ac:associatedSpecimenReference. I was not part of the discussion when these terms were created, so I'm not sure why they didn't just create a single ac:associatedOccurrenceReference property.  These terms are essentially inverse properties for dwc:associatedMedia, which is organized in the Occurrence category of terms.
It is not clear to me what the advantage is of drawing a distinction between the current two terms. It seems like a common use case would be to link the media item back to a GBIF occurrence record, so to require a user to figure out whether that occurrence was based on an observation or a specimen might be kind of silly. How would one even make a reference to an observation without an occurrence ID, since I don't think people typically issue "observation IDs" in the way museums issue identifiers for specimens.
Possible courses of action for the AC MG could be:

adding ac:associatedOccurrenceReference and leaving the two existing terms.
adding ac:associatedOccurrenceReference and deprecating the two existing terms.
doing nothing and keeping the status quo.

Any advice would be appreciated. I don't actually know how or even if these terms are used "in the wild", so that would be useful information if anyone can provide that.

I have just added the even more directly related Issue #337 Change term - associatedMedia.


n8n: 0.213.0 -> 0.214.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for n8n is: Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.213.0 with grep in /nix/store/w5jk4vqc26mndangbviqpl7fiacn2wmf-n8n-0.213.0
found 0.213.0 in filename of file in /nix/store/w5jk4vqc26mndangbviqpl7fiacn2wmf-n8n-0.213.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
n8n

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w5jk4vqc26mndangbviqpl7fiacn2wmf-n8n-0.213.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A n8n https://github.com/r-ryantm/nixpkgs/archive/8a1518e7e7c4ea2e7d193ca252f532b24e585a74.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w5jk4vqc26mndangbviqpl7fiacn2wmf-n8n-0.213.0
ls -la /nix/store/w5jk4vqc26mndangbviqpl7fiacn2wmf-n8n-0.213.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

n8n

Maintainer pings
cc @freezeboy @K900 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for n8n is: Free and source-available fair-code licensed workflow automation tool. Easily automate tasks across different services.
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.214.0 with grep in /nix/store/afaw467z1kgz7bsdgjldi65dql3jgwx5-n8n-0.214.0
found 0.214.0 in filename of file in /nix/store/afaw467z1kgz7bsdgjldi65dql3jgwx5-n8n-0.214.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
n8n

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/afaw467z1kgz7bsdgjldi65dql3jgwx5-n8n-0.214.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A n8n https://github.com/r-ryantm/nixpkgs/archive/805ee2b9f6a35a7153bdd4c12696a16f789c4c1b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/afaw467z1kgz7bsdgjldi65dql3jgwx5-n8n-0.214.0
ls -la /nix/store/afaw467z1kgz7bsdgjldi65dql3jgwx5-n8n-0.214.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

n8n

Maintainer pings
cc @freezeboy @K900 for testing.


Not always possible to edit dynamic content

Describe the issue
Please look at the screen recording below.  There are three cases:

Changing the dynamic content of the condition is not possible
Changing the dynamic content in the HTTP action URL is not possibe
Changing the dynamic content in the Set Variable is possibe

It would be great to always be able to edit dynamic content and function expressions.
Are you using new designer or old designer
New Designer (With action panels)
Screenshots or Videos
No response
Browser
VS Code
Additional context
No response

There is no screen recording attached?

Sorry forgot, updated right now

I see now, this is by design with dynamic content tokens because there's nothing to edit. They're simple insertions you can delete and add another. Function Expressions however should be editable when you click on them, those are much more complex.

I have quite some situations where I need to change dynamic content or maybe I have to wrap it inside a function expression (e.g. string()).  In such cases, I always need to go first to the code view, copy paste the dynamic content and use it in a new function expression.
Being able to change dynamic content would be a great UX improvement

@your-azure-coach - As travis mentioned, only function expressions are editable because you can add/remove/update the expression and also add outputs from previous actions within function expressions.
In your scenario you will have to add the function expression string() switch to dynamic tab and then pick the output from other tab in the picker.
Outputs are not editable and function expressions are, if you really want use the function expression and then add the dynamic content