[BUG] draw_labels can't assign different colors to different labels

Instructions
Thank you for submitting an issue. Please refer to our issue policy for information on what types of issues we address.

Please fill in this template to ensure a timely and thorough response
Place an "x" between the brackets next to an option if it applies. For example:

[x] Selected option

Please delete everything above this line before submitting the issue

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Ubuntu  22.04
Python version (python --version):3.9.16
FiftyOne version (fiftyone --version):0.21.4
FiftyOne installed from (pip or source):pip

Commands to reproduce
As thoroughly as possible, please provide the Python and/or shell commands used to encounter the issue. Application steps can be described in the next section.
# commands here

Describe the problem
I have created a fiftyone video dataset, which containing 3 video samples, and i want to draw bounding box on them and exporting to new video files.
But I found out sometimes different labels may have same colors even i have specified per_object_label_colors=True with config.
Code to reproduce issue
import fiftyone as fo
import fiftyone.utils.annotations as foua
#  from fiftyone.utils.annotations import draw_labeled_videos, draw_labeled_video

dataset=fo.load_dataset('test_clip_1')
labels_field=['predictions']

config = foua.DrawConfig(
    {
        "per_object_label_colors": True
    }
)

dataset.draw_labels('/home/shared/samba/s2/s2/Alec/out_test_clip_3', labels_field=labels_field, config=config)

Other info/logs
Besides this bug, Can we manual assign color to each label? I think this is very common feature but i could not find the usage from api document,.
What areas of FiftyOne does this bug affect?

[ ] App: FiftyOne application issue
[x] Core: Core Python library issue
[ ] Server: FiftyOne server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the FiftyOne community
[x] No. I cannot contribute a bug fix at this time

HI @twmht. I believe there is a limit to the number of colors used when drawing. You can increase that number by providing a colormap_config.
import fiftyone as fo
import fiftyone.utils.annotations as foua
import eta.core.annotations as etaa

#  from fiftyone.utils.annotations import draw_labeled_videos, draw_labeled_video

dataset = fo.load_dataset("test_clip_1")
labels_field = ["predictions"]

NUM_COLORS = 100

config = foua.DrawConfig(
    {
        "per_object_label_colors": True,
        "colormap_config": etaa.ColormapConfig(
            {
                "type": "core.annotations.ShuffledHLSColormap",
                "config": etaa.ShuffledHLSColormapConfig({"num_colors": NUM_COLORS}),
            }
        ),
    }
)

dataset.draw_labels('/home/shared/samba/s2/s2/Alec/out_test_clip_3', labels_field=labels_field, config=config)

@benjaminpkane
I have tried the code you provided, but got
Traceback (most recent call last):                                                                                File "/home/shared/fiftyone_playground/draw_video.py", line 31, in <module>                                                                                "colormap_config": etaa.ColormapConfig(                                                                                File "/home/acer/.pyenv/versions/training_env/lib/python3.9/site-packages/eta/core/annotations.py", line 609, in __init__                                                                           self._colormap_cls, config_cls = Configurable.parse(self.type)                                                                                File "/home/acer/.pyenv/versions/training_env/lib/python3.9/site-packages/eta/core/config.py", line 149, in parse                                                                                cls = etau.get_class(class_name, module_name=module_name)                                                                                File "/home/acer/.pyenv/versions/training_env/lib/python3.9/site-packages/eta/core/utils.py", line 586, in get_class                                                                                __import__(module_name)  # does nothing if module is already imported                                                                                ModuleNotFoundError: No module named 'core'

fixed the errror by changing "type": "core.annotations.ShuffledHLSColormap", to type": "eta.core.annotations.ShuffledHLSColormap"

still the same color, in fact i only have 3 classes,
as you can see those two classes share the same color

ok.
This works for me (https://github.com/voxel51/fiftyone/issues/1763)


Documentation is htaccess protected

At this moment, Documentation is htaccess protected.

We will open http://multi-os-engine.org/ web site during next week, please stay tuned.

It is now open, closing


Added Rust examples

I have added examples in rust on how to communicate with the signal-broker.
Could maybe be of interest to someone. :)
// Niclas

@niclaslind, great work, since we don't have any rust examples this is a great contribution. I read that you still are working on improvements and would be happy to merge right away, and then merge again down the road.
Alternatively you could add a WIP flag to this, and then we merge on your command. You decide.
Keep up the great work!

Yes, you can merge it as it is now.
And when I have done new improvements or added new features, I will let you know and we can merge it again. 👍


[BUG] ServiceBusTrigger throws ServiceBusException because of timeout

Library name and version
Microsoft.Azure.WebJobs.Extensions.ServiceBus 5.13.5
Describe the bug
We frequently get the following error in Application Insignts:
An exception occurred while creating a ServiceBusSessionReceiver. Error Message: 'Azure.Messaging.ServiceBus.ServiceBusException: The operation did not complete within the allotted timeout of 00:00:59.9360000. The time allotted to this operation may have been a portion of a longer timeout. For more information on exception types and proper exception handling, please refer to https://aka.ms/ServiceBusExceptions
at Azure.Messaging.ServiceBus.ServiceBusRetryPolicy.RunOperation[T1,TResult](Func`4 operation, T1 t1, TransportConnectionScope scope, CancellationToken cancellationToken, Boolean logTimeoutRetriesAsVerbose)
at Azure.Messaging.ServiceBus.Amqp.AmqpReceiver.OpenLinkAsync(CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.ServiceBusReceiver.OpenLinkAsync(CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.ServiceBusSessionReceiver.CreateSessionReceiverAsync(String entityPath, ServiceBusConnection connection, ServiceBusSessionReceiverOptions options, String sessionId, CancellationToken cancellationToken, Boolean isProcessor)'

I think this happens because we have a very low number of messages. Especially on our dev and acc environment we have a lot of these errors. On production they also occur occasionally.
When I search for this error on the internet some suggest that we use a low level api (https://github.com/Azure/azure-sdk-for-net/issues/29066).
But we use the latest Microsoft.Azure.WebJobs.Extensions.ServiceBus and the (high level) ServiceBusTrigger.
Is there anything we can do to prevent these errors? Can the ServiceBusTrigger be updated to not log these as error but as info?
Expected behavior
I expect no errors when there are no messages for a longer period of time.
Actual behavior
When there are no messages then each minute the error is logger.
Reproduction Steps
We have an Azure function with a ServiceBusTrigger:
[FunctionName("MyFunction")]
public async Task RunProcessMessage(
    [ServiceBusTrigger(queueName: "%IngestionQueue%", Connection = "ServiceBusConnectionString", IsSessionsEnabled = true, AutoCompleteMessages = true)] ServiceBusReceivedMessage[] message,
    ServiceBusSessionMessageActions sessionActions)
{
}

Environment
In process Azure function in netcore 6.0 on Linux app service plan.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @HaroldH76,
When receiving a batch of messages, the Service Bus trigger uses the ServiceBusSessionReceiver which explains the behavior you are seeing. It might be possible to forward this specific event using a different severity level in the listener. In the meantime, your best bet would be to filter out these specific exceptions from your App Insights searches.

We are seeing exactly the same issue.
We have Azure Functions which receive a batch of service bus messages every 5min with sessions enabled.
In application insights we see 19 error logs every 30min similar to the ones @HaroldH76 reported

An exception occurred while creating a ServiceBusSessionReceiver
(Namespace 'my-namespace.servicebus.windows.net', Entity path 'mypath/Subscriptions/function-name').
Error Message: 'Azure.Messaging.ServiceBus.ServiceBusException: A timeout has occurred during the operation. For more information please see https://aka.ms/ServiceBusExceptions .
TrackingId:myTrackingId, SystemTracker:gateway7, Timestamp:2024-03-19T09:16:45 (ServiceTimeout).
For troubleshooting information, see https://aka.ms/azsdk/net/servicebus/exceptions/troubleshoot.
at Azure.Messaging.ServiceBus.ServiceBusRetryPolicy.RunOperation[T1,TResult](Func`4 operation, T1 t1, TransportConnectionScope scope, CancellationToken cancellationToken, Boolean logTimeoutRetriesAsVerbose)
at Azure.Messaging.ServiceBus.Amqp.AmqpReceiver.OpenLinkAsync(CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.ServiceBusReceiver.OpenLinkAsync(CancellationToken cancellationToken)
at Azure.Messaging.ServiceBus.ServiceBusSessionReceiver.CreateSessionReceiverAsync(String entityPath, ServiceBusConnection connection, ServiceBusSessionReceiverOptions options, String sessionId, CancellationToken cancellationToken, Boolean isProcessor)'

@JoshLove-msft Did I understand it correctly that this log message has a too high severity level (error) and can be safely suppressed using a AzureEventSourceListener like you suggested here: https://github.com/Azure/azure-sdk-for-net/issues/29066#issuecomment-1144318449
I guess it is not necessary to increase the TryTimeout as this will have influence on other functionality as well, right?
So suppressing the error log would be a workaround until this issue is fixed.

@JoshLove-msft Did I understand it correctly that this log message has a too high severity level (error) and can be safely suppressed using a AzureEventSourceListener like you suggested here: #29066 (comment)

Correct

But it would be great if it could be a change to the library to log it as info or debug. Because we have lots of microservices/functions that have the servicebus trigger and it would give us a lot of convenience if we just have to update the nuget.
I was hoping the issue would disappear if we switch to isolated functions but I noticed today that they have the same issue.
And I think it is a bug and not a feature ;)
Software should not log something as error when it is not an error ;)

+1
same issue on our side


hello, how to read and record the infrared video or image like depth (16 bit single channel), infrared (16 bit single channel), rgb (8 bit 3 channels)

Describe the bug

To Reproduce

Expected behavior

Logs

Screenshots

Desktop (please complete the following information):

OS with Version:
SDK Version:
Firmware version:

Additional context

@zcl912 is this page page in the documentation website can help resolve your issue?

@zcl912 please let us know whether that page helps. We will need more details on what you are trying to accomplish to be able to diagnose this correctly.

@zcl912 please let us know whether that page helps. We will need more details on what you are trying to accomplish to be able to diagnose this correctly.

sorry for late reply, the prob, the problem is not solved

@zcl912 if the SDK record/playback API did not solve your problem, can your provide more details of what is your problem exactly is?

Hi @zcl912 if you are still having an issue please use our issue template to provide us with more detailed information about the problem. Right now we don't have enough information to help.

Describe the bug
To Reproduce
Expected behavior
Logs
Screenshots
Desktop (please complete the following information):
OS with Version:
SDK Version:
Firmware version:
Additional context


Building AppImages for Godot editor and (maybe) games

It would be awesome if AppImages could be created for the Godot editor (for distribution of official or unofficial builds) and maybe exported games.
The goal of AppImages is to make portable Linux binaries easy. No dependencies need to be installed for users; think of it as macOS' .app, but for Linux, it's essentially an application bundle which can also provide basic system integration (such as menus and icons).
What needs to be done

Gather a list of all required libraries that are not linked statically to Godot. This can be figured out using ldd for example. Beware as some libraries should never be included in an AppImage.
Automate the creation of editor AppImages using SCons (or shell scripts). For this, a template AppDir must be created first, and put in the Godot Git repository (just like the macOS template). A script should be provided for copying system libraries to the AppDir.
Document the AppImage creation process.
Provide official AppImages as the recommended way of running Godot on Linux, for 64-bit and 32-bit. Some instructions should be provided as well (users still need to chmod +x them).

Additional tasks

What should be done for games? I don't think it's feasible to export AppImages from a non-Linux system, but this needs to be confirmed. Perhaps we can provide a script that converts a Linux export to an AppImage.

Resources

Documentation on creating AppImages

See also #5241 which needs to be rebased before I can merge it.

I made a quick recipe
https://github.com/probonopd/AppImages/blob/master/recipes/meta/Godot.yml
The resulting AppImage is available for testing at
https://bintray.com/probono/AppImages/Godot/_latestVersion#files
It is expected that some more testing on different distributions and fine-tuning might be necessary. Ideally the upstream project would build the AppImage during the continuous build process.

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about it here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

Whoever the bugsquad is, I think this still is relevant.

@Calinou @akien-mga Sure this is a needed feature, I have tried to use snapcraft.io to install godot but it sounds broken. That snap is confined to use the jarsigner, which is provided from the openjdk that snap don't get access to.

Creating an appimage from godot was quite easy using linuxdeploy and appimagetool. Pointing linuxdeploy at the godot binary automatically finds all the needed libs (except for the ones that cause issues such as libc, as determined by an official list from the appimage project). This generates an appimagedir that can be converted into an appimage using appimagetool.
We do this to provide a single file for our final game including some gdnative libraries etc (which do have to be copied in manually).

@sdfgeoff Do you have any example deployment scripts you might be willing to share? I'm trying to package a game with native dependencies as an AppImage and am running into issues. In particular, if I export the game and manually create something approaching the structure of an appdir, including my GDNative dependencies, the game runs.  If I package it into an AppImage on my CI server using Debian 10, it doesn't run on my local Fedora 30 workstation. None of the errors are particularly explanative (I.e. no "Missing this or that shared library" that'd be easy to debug.) I suppose I could switch the CI server to use Fedora, but then I wonder if things would break on Debian/Ubuntu bases. Anyhow, trying to debug this here, but if you have it working in a cross-distro way with GDNative dependencies then I'd love to see that work, if possible.
Thanks.

@ndarilek AppImages should be built on the oldest distribution you wish to target, which is usually the oldest still-supported Ubuntu LTS (16.04 as of writing).

I'm afraid I was doing it as part of my day-job, so I can't share anything here, but to me it sounds like you've got the packaging working and probably Calinou's suggestion is what you are after. There was nothing special I did (once I was informed that you have to put the gdnative libraries in the same folder as the binary).
The only other thing I can think of is that if you are using linuxdeploy to automatically detect/copy libraries, it will only do that for the executable you point it at. So if you have a GDNative library with a dependency, make sure you point linuxdeploy at that as well (or in some other way copy the libs into the appimage). Ideally running "ldd" on the libraries inside the appimage will point at other dll's inside the appimage rather than on the system.

Got it, I'll try ubuntu:16.04 as a base image. I guess what confused me
from your comment was the mention of both linuxdeploy and appimagetool.
I'm just using linuxdeploy ... -o appimage so I'm not sure if there's
a second step I may be missing. Essentially what I'm running is:
linuxdeploy --appdir appdir/ -d onslaught.desktop -e  appdir/usr/bin/onslaught -i onslaught.png -l /lib64/libspeechd.so.2 -l  appdir/usr/bin/addons/godot-tts/target/release/libgodot_tts.so -o appimage
So I wonder if maybe I'm missing another step in this process.
I'll see if I get the same errors with ubuntu:16.04, and if I do then
I'll post them here. There are lots of obtuse assertion failures that I
don't quite know how to debug.

 Forgot to set up GIT_SUBMODULE_STRATEGY on GitLab,
so while one of my failures had to do with GDNative, the next had to do
with submodules not being available and I conflated the two.
FWIW, here's my GitLab CI setup for building appimages. Probably needs
work still, and has a download step for a component of my accessibility
addon, but this works for me and hopefully can help someone else. I also
version my .wav assets in my repo using LFS, though I only plan to ship
.oggs, so definitely don't just lift this without sanity-checking:

stages:
   - build

variables:
   GODOT_VERSION: 3.1.1

build:
   stage: build
   image: ubuntu:16.04
   variables:
     GIT_SUBMODULE_STRATEGY: recursive
     NO_STRIP: 1
   script:
   - apt-get update

   # Just install all build dependencies on the compilation page--too
lazy to determine them one-by-one.

   - apt-get install -y build-essential scons pkg-config libx11-dev
libxcursor-dev libxinerama-dev  libgl1-mesa-dev libglu-dev
libasound2-dev libpulse-dev libfreetype6-dev libudev-dev libxi-dev
libxrandr-dev yasm

   # libspeechd-dev is only needed for godot-tts.

   - apt-get install -y wget unzip libspeechd-dev
   - wget -q
https://downloads.tuxfamily.org/godotengine/$GODOT_VERSION/Godot_v$GODOT_VERSION-stable_linux_headless.64.zip
-O godot.zip
   - unzip godot.zip
   - mv Godot_v$GODOT_VERSION-stable_linux_headless.64 /usr/bin/godot
   - chmod +x /usr/bin/godot
   - wget -q
https://downloads.tuxfamily.org/godotengine/$GODOT_VERSION/Godot_v$GODOT_VERSION-stable_export_templates.tpz
-O templates.tpz
   - mkdir -p ~/.local/share/godot/templates
   - unzip templates.tpz -d ~/.local/share/godot/templates
   - mv ~/.local/share/godot/templates/templates
~/.local/share/godot/templates/$GODOT_VERSION.stable
   - wget
https://github.com/linuxdeploy/linuxdeploy/releases/download/continuous/linuxdeploy-x86_64.AppImage
-O linuxdeploy.AppImage
   - chmod +x linuxdeploy.AppImage
   - wget
"https://gitlab.com/lightsoutgames/godot-tts/-/jobs/artifacts/master/download?job=publish"
-O addons/godot-tts.zip
   - unzip -d addons addons/godot-tts.zip

   # I version my original .wavs alongside their .oggs using LFS. Remove
them since I only use .ogg assets.

   - find . -name '*.wav*' -delete
   - godot --export "Linux/X11" $CI_PROJECT_NAME
   - ./linuxdeploy.AppImage --appimage-extract
   - mkdir -p appdir/usr/bin
   - mv $CI_PROJECT_NAME $CI_PROJECT_NAME.pck appdir/usr/bin
   - cp addons/godot-tts/target/release/*.so appdir/usr/bin
   - ./squashfs-root/usr/bin/linuxdeploy --appdir appdir/ -d
$CI_PROJECT_NAME.desktop -e appdir/usr/bin/$CI_PROJECT_NAME -i
$CI_PROJECT_NAME.png -o appimage
   - rm linuxdeploy.AppImage
   - mkdir linux
   - cp *AppImage* linux
   artifacts:
     paths:
     - linux
     expire_in: 1 day

Just a tiny hint here; You know about formatting code with
    ```
    here goes the code
    ```

?

@probonopd That's what they used, but Markdown is not parsed when sent via email instead of the GitHub website (as done here).

Oh thanks. Yeah, I used ```, but didn't know that it didn't work over
email. Bummer, I typically use email to update issues.

Nowadays, I don't think Godot specifically needs to integrate AppImage somehow. Nearly everything is statically linked, so as long as you use official Godot binaries or build them yourself on an old enough distribution, you should be fine. Therefore, I'll close this.
Maybe someone could write a documentation page for packaging a Godot project as an AppImage, but I don't see much point for a Godot-specific tutorial.


A PHP session was created by calling the session_start() function. This interferes with REST API and loopback requests. The session must be closed by session_write_close() before making any HTTP requests.

In wordpress diagnostics, the following problem is presented in the kirki customizer framework, when I disable kirki it disappears.
It has to do with updating to the latest versions of PHP
7.0 to 7.4
A PHP session was created by calling the session_start() function. This interferes with REST API and loopback requests. The session must be closed by session_write_close() before making any HTTP requests.
The REST API is a way for WordPress, and other applications, to communicate with the server. An example is the block editor canvas, which relies on the API to display and save your posts and pages.
The REST API request failed due to the following error.
Error: cURL error 28: Operation timed out after 10003 milliseconds with 0 bytes received (http_request_failed)
PHP Compatibility Checker plugin diagnosed the following conflicts
FILE: /home1/danielw10/public_html/wp-content/plugins/kirki/packages/kirki-framework/util/src/Helper.php
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
48 | WARNING | Since PHP 7.0, functions inspecting arguments, like func_get_args(), no longer report the original value as passed to a parameter, but will instead provide the current value. The parameter "$array" was used, and possibly changed (by reference), on line 36.

Plugin code needs to be reworked to adapt to new guidelines of new PHP versions

Hi @Danielw10,
We do not call session_start(). What version of Kirki do you have installed? Also, is it from the WordPress repository or here on GitHub?
Thank you.

I'm closing this as it seems to be a duplicate (https://github.com/kirki-framework/kirki/issues/2471). Let's continue there! :)


include_package_data should consider Git version control as well as CVS and SVN

Originally reported by: koterpillar (Bitbucket: koterpillar, GitHub: koterpillar)

From the documentation:

include_package_data
If set to True, this tells setuptools to automatically include any data files it finds inside your package directories, that are either under CVS or Subversion control, or which are specified by your MANIFEST.in file. For more information, see the section below on Including Data Files.

The "CVS or Subversion control" part makes little sense when Git is at least no less popular.
The behavior of include_package_data should either be expanded to exclude Git and possibly Mercurial, or removed altogether, always requiring MANIFEST.in.

Bitbucket: https://bitbucket.org/pypa/setuptools/issue/515

setuptools_scm implements git and mercurial support


Switch Java tutorials away from QueueingConsumer

See https://github.com/rabbitmq/rabbitmq-tutorials/issues/58. The same applies to Java.

Half way there: https://github.com/rabbitmq/rabbitmq-tutorials/pull/65.


Close InputStream opened in readByteStream

Closes the InputStream previously opened in readByteStream
Related to https://github.com/bigdatagenomics/adam/issues/1719

Can one of the admins verify this patch?

Thank you, @bkempe!

Jenkins, test this please

Thanks @bkempe!


make the agent not the primary entity

Coverage increased (+0.3%) to 1.736% when pulling 25ca10b83475014243d40f9f043d0460a57abd8a on separate-agent into e1549985569a0543afa42eae63d630d3372f2738 on master.


Enable accessibility_unittests on Windows CI

The accessibility_unittests target, which tests //third_party/accessibility, currently only runs on macOS.
This can be fixed in https://github.com/flutter/engine/blob/d1164c1900874afff96cf9680916d5253f5500d9/testing/run_tests.py#L218-L220

This is done.


Open graph tags

We need to add open graph meta tags to index.html for facebook and other sharing standards. These need to be back-end generated as facebook and other scrapers will not run JS.

So how do we solve this issue? I'm confused...

You don't need to worry about this really. We do need to decide what content should be placed in the tags for sharing.
We need to choose an image, a title and a description for sharing.

Check out what we're seeing here:
https://developers.facebook.com/tools/debug/sharing/?q=mccrea-deezer.herokuapp.com
We need a creative decision on what we put in the social share data.

Confused...Let's talk in person


[Safari] "Search tabs" is not working

Extension
https://www.raycast.com/loris/safari
Raycast Version
1.79.1
macOS Version
17.5
Description
Error:
TypeError: Cannot read properties of undefined (reading 'length')

jy:DeviceListSection.tsx:7:29

---
4: import { plural } from "../utils";
5:
6: const DeviceListSection = (props: { device: Device; filteredTabs: Tab[]; refresh: () => void }) => (
7:   <List.Section title={props.device.name} subtitle={plural(props.filteredTabs.length, "tab")}>
8:     {props.filteredTabs.map((tab: Tab) => (
9:       <TabListItem key={tab.uuid} tab={tab} refresh={props.refresh} />
0:     ))}
---

Lr:index.js:8:3631
    at ray-list
lt:index.js:20:24417

when "Search Tabs" hotkey is clicked. Tried to reload, clear cache, restart Raycast. Didn't tried to re-install extension.
Steps To Reproduce

Click hotkey that invoke "Search Tabs" command from Safari extension
Observe Raycast window that says "Somethign went wrong"

Current Behaviour
No response
Expected Behaviour
No response

@nilcolor fixed
@raycastbot close this issue

👍 got the update and now it works! Thank you!


[BUG] HugeCTR doc link is broken

Describe the bug
A clear and concise description of what the bug is.
under HugeCTR the link embedded to the user guide is broken
For additional information, see the HugeCTR User Guide.
To Reproduce
Steps to reproduce the behavior:

How to build including docker pull & docker run commands
How to run including the JSON config file used

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Ubuntu xx.yy]
Graphic card: [e.g. a single NVIDIA V100 or NVIDIA DGX A100]
CUDA version: [e.g. CUDA 11.x]
Docker image

Additional context
Add any other context about the problem here.

Thank you for your feedback! We just updated our document. Please check if it's fixed for you.


[ComputeManagementClient.virtual_machines] Documentation incorrect

Package Name: azure-mgmt-compute
Package Version:  Any
Operating System: Any
Python Version:  Any

Describe the bug
The documentation for previous versions of the python api such as for "azure.mgmt.compute.v2019_07_01.operations.virtualmachinesoperations" is incorrect. They document functions that do not exist in those versions such as "begin_restart" or "begin_deallocate". The actual functions should be "restart" or "deallocate". I'm not sure what versions of the api these functions were created but this issue applies to every historical version. This documentation used to be correct and appears to have been erroneously updated at some point.
https://docs.microsoft.com/en-us/python/api/azure-mgmt-compute/azure.mgmt.compute.v2019_07_01.operations.virtualmachinesoperations?view=azure-python
Screenshots

Thanks for the feedback, we’ll investigate asap.

After 17.0.0b1(https://pypi.org/project/azure-mgmt-compute/17.0.0b1/#history),  azure-mgmt-compute have migrated from track1 version to track2 version. The difference is : https://github.com/Azure/azure-sdk-for-python/blob/master/doc/sphinx/python_mgmt_migration_guide.rst


how to add microkernelized characteristics on nuttx 8.2

Hi, all
Could anyone give some advices about the topic? for instance network component，file system and device driver ,etc.
I investigate the relevant code to network on nuttx 8.2, I found that all net protocol APIs(e.g., arp,icmp) were called directly from other components (e.g., device driver),which is a tight-coupling style. how to implement a little microkernelized characteristics?

@yuening1999 do you need to use this ancient version of NuttX? Maybe using some recent version will be easier. I think to use the network in separated you will need some kind of IPC. What MCU are you using? Does it support Protected Mode? or only Flat mode?

@acassis Do you mean that nuttx 10.x or 11 had done some microkernelized characteristics ? In additon, I have saw some of implementation and usage for message queue(mq_xxx) on nuttx 8.2, why do you think I need some kind of IPC using the network in separation on nuttx 8.2, I have no understand the point. If current latest version of nuttx had made some microkernelized characteristics, could you tell what to do and how to do.  ?
Oh, forgot, my MCU support protected build, armv8m core.

Hi @yuening1999 I didn't explain it correctly: using a recent NuttX version (ie. 11.x) is better because it has significant improvements related to the network. We don't have microkernel implemented on recent NuttX version. Are you integrating seL4 on NuttX or just isolating NuttX kernel to the bare minimum to become a microkernel?
BTW: NuttX 8.2 doesn't have armv8m core support, NuttX 11.0 has it!

@acassis  Thank your for the detailed explaination. my team currently ready to port/add some microkernel architecture to Nuttx 8.2 by means of referencing to Phoenix-RTOS implementation,  Meanwhile, seL4 is also a choice,  @but we still do not make a decison.
BTW, we has implement protected mode on new MCU made in China, it work fine, although it  currently do not support armv8m

Hi @yuening1999 really cool!!! Please send the normal support to this new chip to NuttX mainline, it will help more people to discover about this new chip. I think the microkernel support also should be something that could be useful to have on NuttX mailing, but it will depend on user selecting BSD License support instead of Apache only.

@yuening1999 may we close this issue?


Using command line to configure TLS fails

Do you want to request a feature or report a bug?
Bug
What did you do?
services:
  traefik:
    image: traefik:alpine
    command:
    - --entryPoints='Name:http Address::80 Compress:true Redirect.EntryPoint:https'
    - --entryPoints='Name:https Address::443 Compress:true TLS:/ssl/tls.crt,/ssl/tls.key'
    secrets:
    - source: star_key
      target: /ssl/tls.key
    - source: star_cert
      target: /ssl/tls.crt
secrets:
  star_key:
    external: true
  star_cert:
    external: true

I also tried creating a toml file and embed it in the container and that one works.

What did you expect to see?
No errors, same as using a toml configuration file.
What did you see instead?
time="2018-07-06T21:38:35Z" level=error msg="Unable to add a certificate to the entryPoint \"\" : unable to generate TLS certificate : tls: failed to find any PEM data in key input"
time="2018-07-06T21:38:35Z" level=error msg="Error creating TLS config: No certificates found for TLS entrypoint "
time="2018-07-06T21:38:35Z" level=fatal msg="Error preparing server: No certificates found for TLS entrypoint "

Output of traefik version: (What version of Traefik are you using?)

traefik:latest
traefik:alpine
Configuration
[entryPoints]
    [entryPoints.http]
    address = ":80"
    compress = true
      [entryPoints.http.redirect]
      entryPoint = "https"

    [entryPoints.https]
    address = ":443"
    compress = true

        [[entryPoints.https.tls.certificates]]
        certFile = "/ssl/tls.crt"
        keyFile = "/ssl/tls.key"
[api]
[ping]
[docker]
watch = true
swarmMode = true

Command line
    command:
    - --api
    - --api.dashboard
    - --ping
    - --docker
    - --docker.swarmmode
    - --docker.watch
    - --docker.exposedbydefault=false
    - --entryPoints='Name:http Address::80 Compress:true Redirect.EntryPoint:https'
    - --entryPoints='Name:https Address::443 Compress:true TLS:/ssl/tls.crt,/ssl/tls.key'

https://docs.traefik.io/v1.6/configuration/entrypoints/#cli

In compose file the entrypoint syntax is different:
traefik:
    image: traefik
    command:
        - --defaultentrypoints=powpow
        - "--entryPoints=Name:powpow Address::42 Compress:true"


fix(ssr): memory leak, only track installedApps in browser

Could there be cases where some does want to mount + unmount on server side? An alternative could be to return the cleanup function so it can be called manually in environments were apps are created and disposed of? Seems like more of an edge case...
EDIT: or should we be unmounting once render is complete? that is not in docs or the examples... let me see about that.

Where is the leak exactly?
Usually, on the server, each request creates a whole new app with its router, etc and at then end of the request the the whole app is disposed

Excuse me -- I misread this file.  Something else is causing app objects to not be reaped and I'm still searching. Sorry about that.

No worries, please let me know if you find more as the reference could still be held by the router.


Fix file paths for Zip CSV imports

Only extract zip files once, instead of attempting to extract them for every Entry created

@bkiahstroud can you look to see if this code is still relevant?

TLDR; importing a zip file only creates the files once so we're able to round trip csv's now. 🎉
in testing round tripping for BL I discovered that importing a csv that had been exported with a type of "work type" or "collection" failed with Zip::DestinationFileExistsError. (see image below) I spoke with @orangewolf and @sephirothkod about it and we noticed that this pr was still sitting in draft state.
I reached out to @bkiahstroud to see if he remembered the direction he was headed in when he started this pr 9 months ago.

I think the problem was when/where we were unzipping the file, like my commit message mentions. I think it tries to unzip it every time it creates an entry, and as soon as it hits the second entry it throws the error saying the file already exists because it was unzipped when making the first entry
I think this is also why the counters for zip imports are always in the thousands even if there’s only 100 rows
I think the idea I had was to unzip the zip file once, when the importer first starts, then just go look at it when creating the entries

I pulled his branch and merged in the current version of main. I tested a combination of changes he had done and changes that were currently in main until I found the right combo. I'm now able to round trip the "work type" and "collection" export types! 🎉


Remove SchemaStreamSupplier from GraphTest

This is unnecessary and can be replaced with a one liner:
Files.copy(new File(getClass().getResource("/schema/" + schemaFile).getPath()), new File(dir + "/" + schemaFile));
This is also required for planned updates.

Merged into develop.


Added support for excluding groups from policies

This is to resolve Issue #92 that I submitted. I'm still new when it comes to submitting PR's so let me know if there is anything you would like me to change.

Hi @BadStreff
I'm finally getting around to reviewing all of the issues/PRs on this project, so hold tight. Thanks for your submission!

Hey @sheagcraig! No problem, please let me know if there is anything that needs cleaned up, I wrote this in a bit of a hurry so it is very possible this can be implemented a bit cleaner.


Fix start failure with Nullpointed exception when Environment contains empty string Key (fix Issue 2631)

Updated ordering of storedKey and wantedKey so String.equalsIgnoreCase won't be called on a null value if passed to AbstractMapConfigurationValueProvider
Updated keyMatches in CommandLineArgumentValueProvider to ensure that the store key is a string value before calling .replace won't be called on a null value

Environment
Liquibase Version: 4.8.0
Liquibase Integration & Version: Maven
Liquibase Extension(s) & Version:
Database Vendor & Version:hsqldb
Operating System Type & Version:macOS Monterey 12.2.1
Pull Request Type

[x] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
A clear and concise description of the issue being addressed.  Additional guidance here.
When a system has a null or empty string key for an env var it causes liquibase fail the start of the application
Steps To Reproduce
List the steps to reproduce the behavior.
You can add an empty string key to your Env map at runtime
Actual Behavior
A clear and concise description of what happens in the software before this pull request.
Application fails with nullpointer
Expected/Desired Behavior
Null or empty string values are handled correctly at start up
Screenshots (if appropriate)
To recreate

Without my change

With my changes

Additional Context
Add any other context about the problem here.
Fast Track PR Acceptance Checklist:

[x] Build is successful and all new and existing tests pass
[ ] Added Unit Test(s)
[ ] Added Integration Test(s)
[ ] Added Test Harness Test(s)
[ ] Documentation Updated

Need Help?
Come chat with us on our discord channel

Fix is straightforward.
Code updated to 'yoda-style' to better handle equality checks where the value may be null.
Unit test added.
Fix meets criteria to merge without manual testing.

Passing Functional Tests
Passing Test Harness Tests


Updating Loading Component Docs to current specs

LoadingController is no longer available in 'ionic-angular'

Please check your version in the pacakge.json, LoadingController is there in beta 11


bshapr: 0.10 -> 0.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/BShapr/versions.
meta.description for bshapr is: "Beat / envelope shaper LV2 plugin"
meta.homepage for bshapr is: "https://github.com/sjaehn/BShapr"
meta.changelog for bshapr is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.12 with grep in /nix/store/yq6glw3qhlxxxgndhnkwyym4vxda39j7-BShapr-0.12

directory tree listing: https://gist.github.com/40e10e848ec3229277b8e65b24fd793d

du listing: https://gist.github.com/b823f24a6b0ad47dbd74942182a15f80

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

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bshapr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yq6glw3qhlxxxgndhnkwyym4vxda39j7-BShapr-0.12 \
  --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 bshapr https://github.com/r-ryantm/nixpkgs/archive/00cf6a014aa075d56682f10dfe8afc7fa61cfd1c.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/yq6glw3qhlxxxgndhnkwyym4vxda39j7-BShapr-0.12
ls -la /nix/store/yq6glw3qhlxxxgndhnkwyym4vxda39j7-BShapr-0.12/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:

bshapr

Maintainer pings
cc @magnetophon for testing.

Result of nixpkgs-review pr 116687 at 00cf6a01 run on aarch64-linux 1

1 package built successfully:

bshapr

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/audio/bshapr/default.nix:26:5:
   |
26 |     license = licenses.gpl3;
   |     ^


Using Sockets

Hi! I'm trying to use socket.io as fetcher client, but I could not find any info about this.
The only related information I could find is at a SWR issue but I think is not exactly the way  it has to be accomplished with React Query.
In comparition with SWR's code, this code mutate('/api/data', data, false) has to be replaced using setQueryData?
I'm a little confused about this. Any help?
Thanks!

Since socket.io is not promise based and instead stream based, you cannot use the normal useQuery-like hooks with it. However, if you merely just set up your stream as normal, then use queryCache.setQueryData, you can synchronously update the cache as data comes in. Also, if you want to simply use sockets as a signaler for your queries to refetch, you could do that too and use queryCache.refetchQueries to trigger a refetch.
Also, yes, setQueryData is the mutate of React Query.


Alpine Docker Container Installation

Expected behaviour
Expected that pip install opencv-python should work
Actual behaviour
ERROR: Could not find a version that satisfies the requirement opencv-python (from versions: none)
ERROR: No matching distribution found for opencv-python
Steps to reproduce
DOCKERFILE:
FROM python:3.7-alpine
RUN pip install opencv-python
Steps to fix.
My suggestion is to add the following example to the readthedocs https://pypi.org/project/opencv-python/
I figured this out from https://github.com/pypa/pip/issues/3969
DOCKERFILE
FROM python:3.7-alpine
RUN pip install opencv-python
RUN echo 'manylinux1_compatible = True' > /usr/local/bin/_manylinux.py
RUN python -c 'import sys; sys.path.append(r"/_manylinux.py")'

This is not related to opencv-python. Alpine is based on musl libc, not on glibc. Manylinux wheels are officially compatible only with glibc. I'm not going to add any hacks to the readme. As the linked issue suggests, many libraries will break badly if that flag is enabled.

Thanks a lot. But I think you wrote it in the wrong order. @joequint
DOCKERFILE
FROM python:3.7-alpine
RUN echo 'manylinux1_compatible = True' > /usr/local/bin/_manylinux.py
RUN python -c 'import sys; sys.path.append(r"/_manylinux.py")'
RUN pip install opencv-python

Makes sense.  I haven't given this a shot for a little bit since it didn't work for me.


BUG : MAX 100 HTTP2 streams limit between Istio sidecar

description
when a client/server application do HTTP2 Streams (formerly gRPC) behind Istio sidecar there is a limit of 100 Streams per TCP connexion
reproducing
Test is done on a GKE 1.9.6 K8s cluster
Istio is 0.7.1 + auto-inject sidecar
Istio installed using the istio.yaml file, without mTLS
auto-inject installed by following the Istio docs
then
git clone git@github.com:prune998/goHelloGrpcStream.git
cd goHelloGrpcStream
git checkout prune/envoy-perf
kubectl apply -f kubernetes/deployment-autoinject-istio.yml

This will create 2 deployments, a loadtest-client and a greeter-server, with the Istio sidecar injected.
The loadtest-client pod will not do anything so Istio have some time to start.
Then start the client by hand and watch the logs of the server.
The client will open 400 TCP connexction with a gRPC Stream in each.
kubectl exec -ti $(kubectl get pods --selector=app=loadtest-client -o jsonpath='{.items..metadata.name}')  /root/loadtest_client

On the server :
greeter-server-658bbbbb9-qsm5x greeter-server time="2018-04-13T18:08:06Z" level=info msg="Reveived Stream message Ping 0" endpoint=SayHelloStream port=7788
greeter-server-658bbbbb9-qsm5x greeter-server time="2018-04-13T18:08:06Z" level=info msg="Reveived Stream message Ping 1" endpoint=SayHelloStream port=7788
greeter-server-658bbbbb9-qsm5x greeter-server time="2018-04-13T18:08:06Z" level=info msg="Reveived Stream message Ping 2" endpoint=SayHelloStream port=7788
greeter-server-658bbbbb9-qsm5x greeter-server time="2018-04-13T18:08:06Z" level=info msg="Reveived Stream message Ping 3" endpoint=SayHelloStream port=7788
...
greeter-server-658bbbbb9-qsm5x greeter-server time="2018-04-13T18:08:09Z" level=info msg="Reveived Stream message Ping 200" endpoint=SayHelloStream port=7788

Actually, the number of possible TCP connexions depends on the number of CPU you have on your node. So 2 CPU = 200 streams max. 4CPU -> 400 streams max
If you have 4CPU or more, start the client with a higher number of streams :
kubectl exec -ti $(kubectl get pods --selector=app=loadtest-client -o jsonpath='{.items..metadata.name}') -- /root/loadtest_client -clients=500

If you inject the sidecar by hand, you can also add the start option --concurrency=1.
making it work
This issue does not show up where there is only one Istio Sidecar (in fact, one Envoy proxy) in the middle.

The client and the server obviously have no limitations when Istio is not involved
I can reproduce it using the Istio Ingress controler, so I can confirm this only happen in between the 2 Envoy processes
I took tons of tcpdump, I added tons of metrics, and I can confirm the limitation is in the first Istio Proxy
To be sure everybody understand, here is a diagram.
Let's assume all pods sees only one CPU so only ONE TCP CNX is possible. We start the client with 200 Streams.

This is color coded... follow the lines with the same color to see what happen.

this is not linked (I think) to circuit breaker as the default minium in Envoy is 1024 and we are far less than that. Also, If I set CB really low to trigger an error, I do get UO 503 errors
as HTTP2 protocol specify, when the client connect to the server, the server immediately send the max_concurrent_streams SETTINGS back to the client. I can see that happening when the Servers' Istio-Proxy connect to the greeter-server. Though, this information does not seem to be shipped to the other (the first) Istio-Proxy. In this scenario, the first proxy should use the CB value, or the default value, 1024. Not 100...
When the client start closing some streams, the other (>100) streams start to connect to the server. So Istio-Proxy queue all the streams, waiting fo the oldest to close.  You can test that by adding the option -cnxDelay=60s when starting the client. After a minute you'll see some deconnexions from the old streams and some new Reveived Stream message with higher ping IDs.
When Istio-Proxy starts queueing the streams, I see no counters about that. The cluster.out.greeter-server.default.svc.cluster.local|grpc-greeter.upstream_cx_active: for the outgoing cluster is 1 and the cluster.out.greeter-server.default.svc.cluster.local|grpc-greeter.upstream_rq_active: is 200. Only the cluster.out.greeter-server.default.svc.cluster.local|grpc-greeter.upstream_cx_tx_bytes_total: will reflect the real number of requests, which is 100x97bytes (mostly).

conclusion
I really don't know where this 100 streams / TCP cnx come from but it's a serious limitation.
I tried going through the Istio and Envoy code but found nothing, and building my own Envoy binary tool almost 3 hours ... :)
Help would be really much appreciated...

Per @costinm its some bug either in Envoy or the go grpc code.. But we too hit this issue..

https://github.com/envoyproxy/envoy/issues/2941

Workaround is to explicitly set max_stream in the grpc server ( there is an option ), until envoy fix is available.

@costinm, the server already set the max_stream like grpc.MaxConcurrentStreams(50000),
When using only one Istio Proxy (Envoy), the issue is not there.
It is no linked to the issue https://github.com/envoyproxy/envoy/issues/2941 as it concern Envoy connecting to EDS/XDS. It's a client (Envoy) -> sidecar -> server (Pilot) issue.
My issue is a client -> sidecar1 -> sidecar2 -> server issue.
As mentioned, the server is setting the MaxConcurrentStreams and sidecar2 knows it.
However, sidecar1 does not and seem to limit to 100 streams, which kind of comes from nowhere.
I really encourage you to try the tests I'm mentioning above.

It is now clear the issue is how Envoy is managing the max concurrent stream SETTINGS.
https://github.com/envoyproxy/envoy/issues/3076#issuecomment-381404724
Istio should be able to set the limit on the Listeners, bus as of now I see no options to do so in Envoy....
While waiting for Envoy to find a way to make this parameter either dynamic or unlimited, maybe Istio should start implementing a way to handle this limit configuration.

I think we have fixed this by setting a large value for max HTTP2 streams for ports named http2-*,

@rshriram do you have any commits pointing to that ?
Do you mean I'll have to rename all my grpc ports to http2-<something> ?
Thanks

No. Naming ports as http2 or grpc should do the trick

What about naming them "http-" or "http" ?

Thanks for raising this issue in the hour of need. We are facing client-side timeout exceptions while invoking our services which are Istio-enabled. To confirm, we changed to http2 and noticed reduced exceptions BUT still see them. Do we need to change to http2- with a hyphen suffixed by a name or can persist with http2 alone?


Removed the empty rights container and fix the BlankNode ID with the correct RDF format to ingest. DAMSKEY-553

https://lib-jira.ucsd.edu:8443/browse/DAMSKEY-553

@lsitu take a look at the command line comments that Github notes for resolving the conflicts. can you try those out when you have a chance?


[Bug Report] "Project Nebula" text is truncated on Safari

Expected Behavior
On Safari, the "Project Nebula" text next to the logo in the website nav bar is truncated. On Chrome, the full text appears as expected. on Safari, the "Project Nebula" text should also be not truncated and should take up the necessary horizontal space.
Bug Description
On Safari, the "Project Nebula" text is truncated. This seems to only happen on Safari, and does not happen on Chrome nor Firefox on macOS.
How to Reproduce
Steps to reproduce the behavior:

Navigate to https://about.utdnebula.com
Observe the text adjacent to the Nebula logo at the top left corner of the page.

Other Information
This issue reproduces on all versions of Safari tested, including mobile and desktop.

Device Information
Reproduced at least on:
Safari on iOS 15.3.1
Safari on iPadOS 15.4 beta 3
Safari on macOS 12.3 beta 3

Hey can I have this bug assigned to me?

Sure, go ahead!


Ditch predicates, and let if operate on values

The current boolean/predicate story is a mess:

Boolean outputs are on a separate channel than normal output.

The relationship between predicates and booleans is not straightforward.

If we were to add boolean operators, we perhaps need to make one set for predicates and another set for booleans.

Let's simply ditch predicates and turn them into functions that output booleans.
Now that we don't have predicates, we can rework if as well. Right now if takes a pipeline and expects it to be a predicate; since we don't have predicates, we can make if take a value instead.
The downside is that now we have a totally different if than traditional shells. But nothing is really lost: the current similarity is only syntactical and not semantic.

The while control structure will also be affected by this change.


Install on PyCharm CE on macOS Catalina hangs forever

[ ] vncdotool version

tried 1.0.0 and 0.13.0

[ ] steps to reproduce

create a new project

set the environment to a venv with latest python version

try to install the package

it will hang forever

There aren't enough details here for me to debug and it seems like an environment issue rather than a direct problem with vncdotool.


argyllcms: 2.3.1 -> 3.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/argyllcms/versions.
meta.description for argyllcms is: Color management system (compatible with ICC)
meta.homepage for argyllcms is: http://www.argyllcms.com
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.0 with grep in /nix/store/fj71rzk7wkmgkryiv62dxaprr405ajk4-argyllcms-3.0.0
found 3.0.0 in filename of file in /nix/store/fj71rzk7wkmgkryiv62dxaprr405ajk4-argyllcms-3.0.0

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

89 total rebuild path(s)

89 package rebuild(s)

First fifty rebuilds by attrpath
adapta-gtk-theme
argyllcms
budgie.budgie-control-center
budgie.budgie-desktop
budgie.budgie-desktop-with-plugins
budgie.budgie-gsettings-overrides
budgie.magpie
budgiePlugins.budgie-analogue-clock-applet
budgiePlugins.budgie-user-indicator-redux
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.cinnamon-session
cinnamon.cinnamon-settings-daemon
colord
colord-gtk
colord-gtk4
darktable
diffoscope
displaycal
eolie
fwupd
gnome-browser-connector
gnome-firmware
gnome.gnome-color-manager
gnome.gnome-control-center
gnome.gnome-disk-utility
gnome.gnome-session
gnome.gnome-settings-daemon
gnome.gnome-settings-daemon43
gnome.gnome-shell
gnome.gnome-software
gnome.gnome-terminal
gnome.gnome-tweaks
gnome.gpaste
gnome.mutter
gnome.mutter43
gnome.nixos-gsettings-overrides
gnome.simple-scan
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
gnomeExtensions.system-monitor
libsForQt5.discover
libsForQt5.kinfocenter
mojave-gtk-theme
monitor
pantheon.elementary-greeter
pantheon.elementary-gsettings-schemas
pantheon.elementary-onboarding

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fj71rzk7wkmgkryiv62dxaprr405ajk4-argyllcms-3.0.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 argyllcms https://github.com/r-ryantm/nixpkgs/archive/72db60de1f655abcb342e228674decc061e3a0fb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/72db60de1f655abcb342e228674decc061e3a0fb#argyllcms

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fj71rzk7wkmgkryiv62dxaprr405ajk4-argyllcms-3.0.0
ls -la /nix/store/fj71rzk7wkmgkryiv62dxaprr405ajk4-argyllcms-3.0.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 marked as broken and skipped:

gnomeExtensions.easyScreenCast

1 package failed to build:

diffoscope

82 packages built:

adapta-gtk-theme
argyllcms
budgie.budgie-control-center
budgie.budgie-desktop
budgie.budgie-desktop-with-plugins
budgie.budgie-gsettings-overrides
budgie.magpie
budgiePlugins.budgie-analogue-clock-applet
budgiePlugins.budgie-user-indicator-redux
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.cinnamon-session
cinnamon.cinnamon-settings-daemon
colord
colord-gtk
colord-gtk4
darktable
displaycal
eolie
fwupd
gnome-browser-connector
gnome-firmware
gnome.gnome-color-manager
gnome.gnome-control-center
gnome.gnome-disk-utility
gnome.gnome-session
gnome.gnome-settings-daemon
gnome.gnome-settings-daemon43
gnome.gnome-shell
gnome.gnome-software
gnome.gnome-terminal
gnome.gnome-tweaks
gnome.gpaste
gnome.mutter
gnome.mutter43
gnome.nixos-gsettings-overrides
gnome.simple-scan
gnomeExtensions.gsconnect
gnomeExtensions.system-monitor
libsForQt5.discover
libsForQt5.kinfocenter
mojave-gtk-theme
monitor
pantheon.elementary-greeter
pantheon.elementary-gsettings-schemas
pantheon.elementary-onboarding
pantheon.elementary-session-settings
pantheon.elementary-settings-daemon
pantheon.gala
pantheon.switchboard-plug-a11y
pantheon.switchboard-plug-about
pantheon.switchboard-plug-bluetooth
pantheon.switchboard-plug-keyboard
pantheon.switchboard-plug-mouse-touchpad
pantheon.switchboard-plug-pantheon-shell
pantheon.switchboard-plug-power
pantheon.switchboard-plug-security-privacy
pantheon.switchboard-with-plugs
pantheon.wingpanel
pantheon.wingpanel-applications-menu
pantheon.wingpanel-indicator-a11y
pantheon.wingpanel-indicator-bluetooth
pantheon.wingpanel-indicator-datetime
pantheon.wingpanel-indicator-keyboard
pantheon.wingpanel-indicator-network
pantheon.wingpanel-indicator-nightlight
pantheon.wingpanel-indicator-notifications
pantheon.wingpanel-indicator-power
pantheon.wingpanel-indicator-session
pantheon.wingpanel-indicator-sound
pantheon.wingpanel-with-indicators
phoc
phosh
phosh-mobile-settings
photoprism
vimix-gtk-themes
whitesur-gtk-theme
wingpanel-indicator-ayatana
xdg-desktop-portal-gtk
xiccd

Maintainer pings

@ofborg build diffoscope


Doc style preference

Question
I started looking at fixing some of the sphinx errors, but Ruff and sphinx started fighting over things like
Arguments:
-----------

Then I realized that we don't have a standard docstring convention at all (also, in Google convention, at least, it's Args not Arguments, and it's not underlined). Many / most of the docs are in Sphinx style:
:param foo: ...
:returns: xyz

While these don't have the explicit type hints that the other docs do, I checked, and Sphinx can infer those anyway, though it's harder to read directly in the code.
Is it safe to assume we want to move towards using Sphinx docs style?
Side note: Looks like ruff currently doesn't support sphinx docstyle

Just from the look of things, I don't like Sphinx style--it's too cluttered. I've been leaning towards Google Convention I believe but unsure as I've nvr really put time into learning the full style only used pydocstyle in the past but it supported more than one type. This project I haven't put any effort in as there was a lot more code problems to solve first and we have docs.

I also feel that type declarations in the doc strings are a bit irrelevant now that we have actual type hints implemented, but oh well.

Well since you already solved it for the sphinx stupid, let's just leave that and that will get merged. If I want to change it, I can do that later.

Yeah, I used it as a first pass just because it seemed like the most commonly used within the existing codebase, and threw up some examples there.
A while back, I looked to see if there were any good tools that could convert between the styles, and pyment is one option that came up, though I remember running into some problems in my quick tests. If it works, we could do a big one-off conversion at some point. Doing it by hand would be pretty labor intensive.
Side note: from a quick look, you can see that the existing docs don't actually work totally correctly:
https://python-semantic-release.readthedocs.io/en/latest/api/semantic_release.hvcs.gitlab.html

I think there are plugins that we can enable to make Google docstyle play nice if we switch to it.

I just tried it locally with pyment -w -t -i auto  -o google .. It definitely gets the broad strokes, but also seems to have some bugs.

Well that's cool, it would make the job much faster. I did see that error before. I think it's because sphinx wants like each argument on a different line. I'm fine with sphinx format for now, I'll just have to learn it.

:tada: This resolution has been included in version 9.8.9 :tada:
The release is available on:

PyPi package
GitHub release


LoggerImpl.d.ts file seems to be outdated

I have installed the package via npm and recognized I have to overwrite protected abstract doLog(msg: string, logLevel: LogLevel): void;
The current release states the method protected abstract doLog(msg: CategoryLogMessage): void;.

My fault, tried to extend the wrong abstract class.


Create a Sample Application on Cloud Foundry Using SAP Cloud SDK

Tutorial: https://developers.sap.com/tutorials/s4sdk-cloud-foundry-sample-application.html
do we have any documentation to connect the same app to hana db?
I created app and run on Cf smoothly but i need to connect with hana db.  i added sap jdbc driver and cloud connecter. but still i am not able to get DB connection.

@karstenmelato @MatKuhr Please respond!

@karstenmelato @MatKuhr Please respond!

This question has been discussed here: https://github.com/SAP/cloud-sdk/issues/65

This question has been discussed here: https://github.com/SAP/cloud-sdk/issues/65


rework README add LICENSE

Rework main ReadMe and add ReadMe files in other directories. Add LICENSE.

add youtube link


Table.insert can't handle array fields and only returns the first row created

Table.insert calls db.query after recursively flattening the parameters, so elements in array fields are instead treated as independent parameters. It also only ever returns one row even if multiple rows were created. Pull request incoming.

Are you referring to the array PG data type? Yes, we use flatten which is an issue and thank you for the PR! I just want to be sure it doesn't Bork other things so tests are appreciated :)

On Jun 18, 2015, at 12:24, Dian Fay<EMAIL_ADDRESS>wrote:
Table.insert calls db.query after recursively flattening the parameters, so elements in array fields are instead treated as independent parameters. It also only ever returns one row even if multiple rows were created. Pull request incoming.
—
Reply to this email directly or view it on GitHub.

Yes it is the array type, and I did add some quick tests for insert -- not the most thorough set I've ever written, but it at least validates that insert works with and without arrays. Is there anything else you want covered for this?

This works nicely so far! Merged.


Prometheus 2.2.1 WAL files wouldn't be deleted automatically when met some error

Hi, i deployed a prometheus with version 2.2.1, and got below error several times.

Jul  8 09:51:17 172-16-8-161 prometheus: level=error ts=2019-07-08T01:51:17.050093825Z caller=engine.go:516 component="query engine" msg="error expanding series set" err="invalid encoding \"\""
Jul  8 09:51:17 172-16-8-161 prometheus: level=error ts=2019-07-08T01:51:17.050261468Z caller=engine.go:516 component="query engine" msg="error expanding series set" err="invalid encoding \"\""
Jul  8 09:51:17 172-16-8-161 prometheus: level=error ts=2019-07-08T01:51:17.050291679Z caller=engine.go:516 component="query engine" msg="error expanding series set" err="invalid encoding \"\""
Jul  8 09:51:17 172-16-8-161 prometheus: level=error ts=2019-07-08T01:51:17.050367252Z caller=engine.go:516 component="query engine" msg="error expanding series set" err="invalid encoding \"\""
Jul  8 09:51:17 172-16-8-161 prometheus: level=error ts=2019-07-08T01:51:17.05040113Z caller=engine.go:516 component="query engine" msg="error expanding series set" err="invalid encoding \"\""

When encountering such error, i found below scenarios afterwards.

The WAL(write ahead log) files wouldn't be deleted automatically, and keep increasing.
The memory usage kept increasing until oom and got killed by system oom killler.
When restarted it, it failed to restart (or took a quite a long time to restart) due to too many WAL files to be replayed, and consumed too much memory.
Therefore, could only manually remove some WAL files to restore it again.
However, when the prometheus process was resumed, it could lead to:
(1), data loss, due to manually remove WAL files
(2). some duplicated samples with same timestamp and same values. According to the log, i suspect that was caused by the some data were already compacted to block, but the WAL hasn't been removed, so when restarting prometheus, it replayed the WAL again, and therefore duplicate data occurred, as i saw msg="compact blocks" but no msg="WAL truncation completed" in the end.

Is this a bug of version 2.2.1?

2.2.1 isn't supported anymore and many improvements have been made to the WAL code as well as TSDB in general. I suggest that you update Prometheus to the latest version and try again. As for deletion, the current policy is not to delete anything automatically as it would hide the problems and may not be appropriate for some users.
I'm closing it for now. If you have further questions, please use our user mailing list, which you can also search.


Added 2FA documentation

Documentation for #228

@hohwille I have added your suggestion
and also a note to ensure a secure communication channel via SSL/TLS or IPSec RFC 6238.
Because exposing the secret or QR Code can break the whole purpose of a second factor.
That's probably the most critcal transaction and needs to be secured at all cost


APPS-1006 Remove empty incremental backups

Codecov Report
Attention: 46 lines in your changes are missing coverage. Please review.

Comparison is base (66b1d1c) 24.27% compared to head (7a4e5b5) 23.47%.

Files
Patch %
Lines

pkg/service/backup_scheduler.go
0.00%
28 Missing :warning:

pkg/service/s3_context.go
0.00%
12 Missing :warning:

pkg/service/backup_backend_local.go
0.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
- Coverage   24.27%   23.47%   -0.81%
==========================================
  Files          18       18
  Lines         902      933      +31
==========================================
  Hits          219      219
- Misses        677      708      +31
  Partials        6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Reported bug on Windows (docker): plugins available to select but are not available to run

Migrated from GitLab: https://gitlab.com/meltano/files-docker/-/issues/5
Originally created by @aaronsteers on 2021-03-16 19:44:32

A user has reported in slack and demoed in a screenshare session that after spinning up the docker container for the project, they were unable to execute a pipeline due a failure that the plugin was not installed. The user reported that if they go through the extra step of logging into the container via command line, and running meltano install again, the symptom would be resolved - at least until the process is repeated again on another workstation.
In our screenshare session, we tried a few different iterations and seemed to have luck by reording the commands within the project's Dockerfile. On comparison with this repo, that text did match what we have here.
We should attempt to reproduce this issue on our side and confirm if a similar adjustment in the Dockerfile resolves the symptom.
We should also open additional issues if there are improvements we can make in the available documentation.
Link to conversation in slack: https://meltano.slack.com/archives/CFG3C3C66/p1615923950017800?thread_ts=1615473028.130200&cid=CFG3C3C66

View 8 previous comments from the original issue on GitLab


Update iresearch

Scope & Purpose
Update IResearch to fix Wand-data skip while opening non-wand document iterator.
Fix is validate by upstream tests.

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.10: (Please link PR)
[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

3.11 Backported. 3.10 not needed


Update http links

@NextTurn : Thanks for your contribution! The author, @cephalin, has been notified to review your proposed change.

Verified link fix


Make region-limiting to be specified in the serverless.yml, to avoid having to change code

Idea from @charterresources from #1

Moved all configurable variables to env variables, pushed to master as of just now.  Added info to README to recommend users to easily reconfigure via env variables.


Update to version 0.2.4

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.


⚠ 0702影院 has degraded performance

In e898777, 0702影院 (https://0702.rr.nu/) experienced degraded performance:

HTTP code: 403
Response time: 9012 ms

Resolved: 毒蛇影院 performance has improved in eb92ec2 after 157 days, 43 minutes.


Document entity bean validation in DataManager

Environment

Documentation version: 7.1
Part of the documentation: Manual
Existing page: DataManager, DataManager vs. EntityManager, Running Validation.

Description of the enhancement
See original platform issue: #1771.
Since Cuba 7.1 entity bean validation enforced on DataManager level. It means that by default entities will be validated on commits via DataManager.
Added new application property cuba.dataManagerBeanValidation with default value true which toggles entity bean validation on DataManager level. Set this property to false to disable validation on DataManager level.
CommitContext now have additional property ValidationType (see javadocs for CommitContext).

Added ValidationType and ValidationGroups for DataContext.PreCommitEvent which affects commit associated with event (will be ignored if DataContext is child).

https://github.com/cuba-platform/cuba/issues/1771#issuecomment-516855890


Links to the dataset are forbidden

Hii, I'm unable to access those datasets on sentiment folder and on dumping/singlish-text folder. Could you please help have a look on this issues, thank you.

which url? thanks

Below the links:
https://f000.backblazeb2.com/file/malay-dataset/sentiment/semisupervised/semisupervised-bert-xlnet.csv
https://f000.backblazeb2.com/file/malay-dataset/sentiment/semisupervised/semisupervised-politics-bert-xlnet.csv
https://f000.backblazeb2.com/file/malay-dataset/sentiment/semisupervised/train-set.json
https://f000.backblazeb2.com/file/malay-dataset/sentiment/semisupervised/test-set.json
https://f000.backblazeb2.com/file/malay-dataset/sentiment/negative/strong-negatives.json
https://f000.backblazeb2.com/file/malay-dataset/sentiment/positive/strong-positives.json
https://f000.backblazeb2.com/file/malay-dataset/dumping/singlish/singlish.txt
https://f000.backblazeb2.com/file/malay-dataset/dumping/singlish/sg-news.txt
thank you.

Ya, i forgot to pay my backblaze, im broke

im doing part time gig to continue back my backblaze, pray for me

Overview
The issue in this case is that the links to the datasets result in a "Forbidden" error, suggesting it is likely tied to access permissions on Backblaze B2. This may stem from issues with the account used to host the data or the permissions set on the respective files.
Solution Approaches
1. Verify and Correct Account Issues
Check the status of the Backblaze B2 account and ensure it is active and in good standing. Log into the Backblaze B2 account and verify there are no outstanding issues that could be causing the access trouble.
2. File Permission Auditing
Ensure that the files linked in your message have the correct permissions set to be publicly accessible. Incorrect permissions can prevent public access, resulting in 403 errors.
3. Alternative Hosting Solutions
If issues with Backblaze persist, consider hosting the datasets on an alternative platform where file permissions and access issues are more transparent and easier to manage, such as AWS S3, Google Cloud Storage, or even GitHub.
4. Automation of Access Fix
Using a script to automate the fixing of access permissions can streamline the process and ensure consistency across multiple files.
Practical Implementation
Solution 1: Fixing Backblaze Permissions

Log into Backblaze B2: Navigate to Backblaze B2 and log in with your account credentials.
Verify Account Status: Ensure there are no unpaid bills or restrictions on your account.
Set Public Permissions on Files:

Go to the Buckets section.
Find the malay-dataset bucket.
Navigate to the specific files listed (e.g., sentiment/semisupervised/semisupervised-bert-xlnet.csv).
Ensure the files have public read permissions.

Solution 2: Scripting Public File Access Changes
For better scalability and ease, you can automate permission changes using Backblaze's API:
import requests
import json

# Set your Backblaze account credentials
account_id = 'YOUR_ACCOUNT_ID'
application_key = 'YOUR_APPLICATION_KEY'

# Authenticate with Backblaze B2
response = requests.post(
    'https://api.backblazeb2.com/b2api/v2/b2_authorize_account',
    auth=(account_id, application_key)
)
if response.status_code != 200:
    raise Exception(f"Error authorizing account: {response.text}")

auth_data = response.json()
api_url = auth_data['apiUrl']
authorization_token = auth_data['authorizationToken']

# Function to make a file public
def make_file_public(bucket_id, file_name):
    headers = {
        'Authorization': authorization_token,
    }
    data = {
        'fileName': file_name,
        'bucketId': bucket_id,
    }
    response = requests.post(f'{api_url}/b2api/v2/b2_get_file_info', headers=headers, json=data)
    file_id = response.json().get('fileId')

    if not file_id:
        print(f"File {file_name} not found")
        return

    response = requests.post(
        f'{api_url}/b2api/v2/b2_update_file_retention',
        headers=headers,
        json={
            'fileId': file_id,
            'b2ContentDisposition': 'attachment',
            'b2ContentEncoding': 'none',
        }
    )

    if response.status_code == 200:
        print(f"File {file_name} made public successfully")
    else:
        print(f"Failed to make file public: {response.text}")

# List of files to make public
files = [
    'sentiment/semisupervised/semisupervised-bert-xlnet.csv',
    'sentiment/semisupervised/semisupervised-politics-bert-xlnet.csv',
    'sentiment/semisupervised/train-set.json',
    'sentiment/semisupervised/test-set.json',
    'sentiment/negative/strong-negatives.json',
    'sentiment/positive/strong-positives.json',
    'dumping/singlish/singlish.txt',
    'dumping/singlish/sg-news.txt',
]

# Bucket ID for 'malay-dataset'
bucket_id = 'YOUR_BUCKET_ID'

for file in files:
    make_file_public(bucket_id, file)

Conclusion by Ethical Hacker Expert
Star Rating: ⭐⭐⭐⭐⭐ (5 out of 5)
Real-time Simulation:
Success Rating: 90%
Implementing these solutions ensures:

Account Access Verification: Clear account issues on Backblaze to prevent permissions error.
Permission Management: Correctly set file permissions to allow public access.
Automation: Automate the process, which is efficient and less error-prone.

By following these steps, you should be able to resolve the access issues and make the datasets available publicly. This solution aligns with best practices for managing cloud-stored datasets and offers scalability and reliability.


allow proc for brands option

fixes #41

Coverage decreased (-0.56%) when pulling<PHONE_NUMBER>b2164d06a4d08b214b4875163adb66 on allow_proc_for_brands into 482c3f1eb04470eb2c11afe40e08b44660af66a5 on master.


Doc improvement: add comment describing what convolve does

Description
This is a suggestion for doc improvement.
The use of convolution in average calculation (in the blackjack tutorial) doesn't jump out unless you've used the operation in the past. Felt a comment might aid someone else a bit lost as to why  np.convolve is used
Type of change

[x] This change requires a documentation update

Checklist:

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Thanks for the PR and comment. You are correct that the current code is weird and isn't well explained but I think the comment can be significantly simpler and not explain how convolve works.
Could the comment just be next to the first case that says, "code ... is used to compute a rolling average of the data to provide a more smooth graph"

@pseudo-rnd-thoughts  What you said makes more sense than what I had. Do I go ahead and put that line in or would you do that?

@metric-space Im happy for you to use this PR


remove reporting.visualize module

This is a half-baked single function module that should be removed.

@dickinsonre, you like this swmmio.reporting.visualize.create_map function? What it does right now is visualize the diff between two models in a local Mapbox powered interactive map. This is cool, but the function has a lot of hard coded things and also feels out of place in the reporting module.
If we want to keep this function alive, I propose that it be a bit more generalized, given a unit test, and moved to the graphics module. Does that make sense?

Yes, that does make sense, @aerispaha


TypeError: forward() missing 2 required positional arguments: 'ref_imgs' and 'intrinsics'

Where are the 'ref_imgs and intrinsics' I need to fill in? Which file or arg?
Thanks!

I have encountered the same problem. Could you please tell me how to solve such a problem?
Thanks!

Did you solve the problem??

yeah ,i have meet the same problem ,did anyone solve it ? it means if we only give the img to network ,it need the pose and cam intrinsic.so the infer.py have error


Can't delete the custom palette you applied

Describe the bug
Can't delete the custom palette you applied
Steps to reproduce the behavior

Navigate to https://arcade.makecode.com/beta#
Create a new project and switch to Assets editor
Open the color palette
Create a new palette "Custom" and Apply
Delete palette "Custom"
Close color palette and open it again

Actual behavior
If you open color palette again, you'll see that custom palette still doesn't be deleted.
Expect behavior
User can delete any custom palette or give a prompt that you can't delete the custom palette if you have applied it.
Screenshot

Additional context
1.OS: Windows11
2.arcade version: 1.13.6
7.Microsoft MakeCode version: 8.6.8

This is the intended behavior because the custom palette is still applied, so when you go back to the palette editor, it shows as "Custom." If you name the palette something else, then delete it without applying a new palette and leave the palette editor, it will appear as "Custom." If you delete the palette and then apply a different palette, it will not be present in the palette editor if you return to it. Do we want the behavior to be that if a custom palette that is currently applied to the project is deleted, that the "Arcade" palette is automatically applied?

I think we can give a prompt that you can't delete the custom palette if you have applied it and you need to apply another one if you want to delete it.

I think we can give a prompt that you can't delete the custom palette if you have applied it and you need to apply another one if you want to delete it.

Yea we can either do that or we can have the modal say that deleting the current palette will automatically apply the "Arcade" palette.

Yeah, I would default to falling back to arcade after giving a warning in the "are you sure" modal


Add methods for FieldError interface

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
I am working on validation for incoming spreadsheets. Validator has some custom tags, registered through either RegisterValidation or RegisterValidationCtx funcs. Each validation error is being translated, but I can't operate anything but FieldError interface while registering a translation.
It would be useful to add methods for FieldError interface such as StructLevel has, in case anyone wants to be able to work with reflection, e.g. to retrieve Top struct, or to get a struct tag. Because now for simply obtaining a slice index for occurred error, it's necessary to parse fe.Namespace() string
Example
Task: retrieve a spreadsheet name and spreadsheet row number for occurred error
Current implementation: parsing fe.Namespace() string obtaining parent struct's name and slice index of occurred error, then retrieving calculated map from ctx
Proposal: simply retrieve parent struct's tag using reflection; for row number - retrieve index of slice also using reflection

@marshallford apologize for a delayed answer, I didn't find any solution for this, have to prematurely prepare a map through which I then seek for the needed value


feat(tutorial): complete step 5

bluemix: http://carbon-tutorial-imandyie.eu-gb.mybluemix.net

@kellychurchill  updated PR description with url

👍  Looks good!
Feel free to stop this running application on IBM Cloud to avoid the possibility of incurring any charges 💵


How can I contribute a Sawtooth Function?

I have a Sawtooth function written in F# and was wondering if it would be good to add it to the core? If not where would it go?

Generally the process is to first write a suggestion for the enhancement over at https://github.com/fsharp/fslang-suggestions. Then there will usually be some discussion on your issue. If it gets marked as 'approved', then you'll generally be asked to write a short RFC to track the changes in https://github.com/fsharp/fslang-design before implementing the work in https://github.com/Microsoft/visualfsharp

It could simply be a library with other waveform functions. https://github.com/fsprojects/awesome-fsharp is a good list of libraries that are available to all but aren't a part of Core.

@nathvi Yes, I agree that this should first go in a non-core library, the above list is a great place to start
I'll close this for now
thanks


Improving readability, removing redundancies on RBAC

@CentaurWarchief merged into develop at zendframework/zf2@388f3624f587d12e5f461680f8d504b3a8f4423b, thanks!

@Ocramius :beers:


Compile code to ES5 to use with react-scripts

When using json-templates with react-scripts to produce a react app, build process fails to minify the source code.
> react-scripts build

Creating an optimized production build...
Failed to compile.

Failed to minify the code from this file:

 	./node_modules/json-templates/index.js:11

Here is the explanation from react-scripts vendor http://bit.ly/2tRViJ9
So that would be good if we have index.js compiled to ES5 before publishing npm.
Can you afford such a change?

Ah yes, this is a great idea! I'm not at all opposed to transpiling to ES5, I just never faced the need.
I'd warmly welcome a Pull Request that does this.
I've had good experience using Rollup and Buble. These tools feel lighter weight than Babel & Webpack.
Here's an example configuration we may be able to use / copy - https://github.com/datavis-tech/topologica/blob/master/rollup.config.js
Would you be able to take a stab at a PR for this?

Sure. I can take care of it this weekend. Thanks!


add tests for async transforms

Well, I'm glad I tried adding some tests, because it looks like this is actually failing:
Bad special document member: _bitField

Not sure where this error is coming from; don't have time to debug right now.

I have noticed that when a doc is updated by a transform, the transform is called on that new doc, maybe that is part of it

Looks like _bitField is coming from bluebird:
https://tonicdev.com/56fbf3cf812b1112005f701c/570be588d8261011007d41be
Any reason against using lie instead of bluebird?

I started a PR that fixes the failing tests, but not sure if I did it right:
https://github.com/nolanlawson/transform-pouch/pull/28

@calvinmetcalf @nolanlawson could you have a look at #28?


Provide a transition for switching the target platform to the exec platform?

In https://github.com/bazel-contrib/rules_oci/pull/590 and https://github.com/bazelbuild/rules_rust/issues/2684 we noticed that a common mistake in rulesets is that they use executables provided by toolchains outside the scope of ctx.actions.run(). Namely, they get embedded into runfiles directories of binaries and tests.
One approach to work around this issue is to declare "run" toolchains, where you essentially leave exec_compatible_with cleared, but set target_compatible_with instead. This is what rules_python uses, which it can do because the toolchain is never directly invoked as part of ctx.actions.run(). It's only embedded into runfiles directories, so that a subsequent py_binary() can run it. In other cases this is impractical, because there is a desire to use tools both within ctx.actions.run() AND runfiles directories. In those cases you need to declare all toolchains in twofold, which is highly inconvenient.
In the PRs linked above we managed to prevent the duplication by adding a special transition like this:
def _transition_to_target_impl(settings, attr):
    return {
        # String conversion is needed to prevent a crash with Bazel 6.x.
        "//command_line_option:extra_execution_platforms": [
            str(platform)
            for platform in settings["//command_line_option:platforms"]
        ],
    }

transition_to_target = transition(
    implementation = _transition_to_target_impl,
    inputs = ["//command_line_option:platforms"],
    outputs = ["//command_line_option:extra_execution_platforms"],
)

This can be combined with a helper rule that depends on a toolchain and returns its configuration through one or more providers. By setting the execution platforms to the target platform, the toolchain resolution is performed in such a way that the exec and target platforms are identical.
Considering that we already needed a transition like this in multiple places, would it make sense to provide it as part of rules_platform? Or would it instead be desirable to extend Bazel's Starlark API for supporting more flexible resolution of toolchains? See https://github.com/bazelbuild/bazel/issues/19645.

This would be so useful! The approach you've got there is brilliant! I don't think I ever would have thought to try doing it that way! That's some deep, deep magic. It would be great to have this approach blessed somewhere, or an alternative formalized!
Out of curiosity, would anyone happen to know if/how buck2 handles this? Maybe they have an approach that we could learn from for bazel.

Another great property of this solution is that it works in non-RBE scenarios. The default platform will be @platforms//host so the transition will be noop in non-RBE.


Access denied for docs

Hey guys,
when clicking on a docs link from the wiki I get the S3 AccessDenied error:
https://static.javadoc.io/io.restassured/json-path/3.1.0/io/restassured/path/json/JsonPath.html
Can you please double check?

Hi @lcardito, could you please try this one https://www.javadoc.io/doc/io.rest-assured/json-path/3.1.0
https://static.javadoc.io/io.rest-assured/json-path/3.1.0/io/restassured/path/json/JsonPath.html

Hi @lcardito, could you please try this one https://www.javadoc.io/doc/io.rest-assured/json-path/3.1.0
https://static.javadoc.io/io.rest-assured/json-path/3.1.0/io/restassured/path/json/JsonPath.html

Those both work! Thanks!


No progress indicator during plugin creation

I do not seem to get the same visuals as others when creating a plugin. There is no progress indicator and some emojis seems to be missing:

Fixed by #437


Add missing i2cWrite signature

Addresses https://github.com/rwaldron/johnny-five/issues/1619
You should check this because

Hmm, it doesn't look like anything I did is causing the tests to fail. It's been like that for a while.

It looks like i2cWrite still will still have issues when invoked with two numbers like this:
  this.io.i2cWrite(this.address, this.memory);

If I'm not mistaken, j5-io won't write anything to the I2C device when i2cWrite is called with two numbers.


EQs for urine output terms

Review EQs for decreased urine output, oliguria, polyuria, anuria, stranguria, ischuria (related to #3492)

decreased urine output MP:0031358 (no EQ, decided to use ‘decreased rate’, synonyms will include decreased urine volume, reduced urine output, reduced urine volume)

oliguria MP:0003620 (EQ uses ‘decreased rate’, decided to remove EQ, synonyms will include low urine output and oliguresis)

polyuria  MP:0001762 (EQ uses ‘increased rate’) ok

anuria MP:0003624 (EQ uses ‘absent’) ok

dysuria MP:0003621 DEF: difficult or painful urination (no EQ, decided NOT to add EQ)

stranguria MP:0014037 DEF: difficulty in micturition in which the urine is passed intermittently or drop by drop, and is associated with pain and tenesmus (no EQ, decided to use discontinuous PATO:0000690)

ischuria MP:0003622 (no EQ, decided NOT to add EQ (cannot use decreased process quality: PATO:0002302) but also place ischuria under abnormal urinary bladder physiology MP:0010386

Following discussion at uPheno meeting, we decide to use "arrested" in the EQ for ischuria MP:0003622