Data Structure for shared queue in level 8

What data structure is suitable for project level 8?
Self-checkout counters use a shared queue but we are not allowed to use any mutable data structure or variable without final keyword. Is there a good workaround for this?

Hi, for me I decided to keep a sharedQueue in my main that constantly updates itself when each Event is called (for self-check servers), then I update all the self-check servers' queue at the same time. So this means that I am still using the same queue field for the Server class and I only add in an additional field to check if the Server is a human / self-check.
Might have been q a lazy implementation but it somehow passed :p

For me i put the shared queue in shop

Ugly implementation, but personally didn't create a new shared queue. Just added logic in the execution of events such that if identified to be a counter, it would update the queue of that counter and simultaneously update all the other counter's queues as well

Ugly implementation, but personally didn't create a new shared queue. Just added logic in the execution of events such that if identified to be a counter, it would update the queue of that counter and simultaneously update all the other counter's queues as well

hi @SoSimplySean, can I check if you had to make any changes to your Simulate8's run() (compared to simulate7) with your implementation of selfcheckout? or was all the logic regarding selfcheckout handled in the events' execute() methods? thanks!

@mazchew Pretty much the same as Simulate7 - everything was managed inside the execute()'s


Testing workflow [DO NOT MERGE]

This is just a test for our workflow github action

Done testing for now :)


Support infinite_scroll_pagination 4.0.0

infinite_scroll_pagination 4.0.0 was just released a week ago, it would be great to get support for it in the long term.

I got an error trying to build for Flutter 3.22:
../.pub-cache/hosted/pub.dev/infinite_scroll_pagination-3.2.0/lib/src/ui/default_indicators/first_page_exception_indicator.dart:27:50: Error: The getter 'headline6' isn't defined for the class 'TextTheme'.
 - 'TextTheme' is from 'package:flutter/src/material/text_theme.dart' ('../development/flutter/packages/flutter/lib/src/material/text_theme.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'headline6'.
              style: Theme.of(context).textTheme.headline6,

infinite_scroll_pagination 4.0.0 resolves this issue so its critical to support it! otherwise this package is no longer useful..

Hi this should be resolved in https://github.com/ftognetto/riverpod_infinite_scroll/pull/45

@ftognetto
PR #45 doesn't seem to be in the latest version on pub.dev.
Also, I see the library's version after the change is the same 1.0.9: https://github.com/ftognetto/riverpod_infinite_scroll/blob/main/pubspec.yaml#L5
Should version 1.0.10 be published on pub.dev?

@ftognetto please update the pubdev package to include #45  ! so far the package is broken and doesn't support Flutter 3.22!!

Sorry all, the pr has been published now!
https://pub.dev/packages/riverpod_infinite_scroll/versions/1.0.10


Helm installing Logging Operator in k8s version 1.25 and above.

Describe the bug:
While trying to install logging-operator using helm, i am getting the error: UPGRADE FAILED: unable to build kubernetes objects from current release manifest: resource mapping not found for name: "psp.logging-operator" namespace: "logging" from "": no matches for kind "PodSecurityPolicy" in version "policy/v1beta1"
k8s version client side: 1.27
k8s version server side: 1.25
Expected behaviour:
logging-operator should be installed
Steps to reproduce the bug:
k8s version: 1.25
run: helm upgrade --install --wait --create-namespace --namespace logging logging-operator oci://ghcr.io/kube-logging/helm-charts/logging-operator --version=4.5.0
Additional context:
This can be related to k8s removing PodSecurityPolicy starting for k8s version 1.25 and above.
Environment details:

Kubernetes version: 1.25
Cloud-provider/provisioner (e.g. AKS, GKE, EKS, PKE etc): EKS
logging-operator version (e.g. 2.1.1): 4.5.0
Install method (e.g. helm or static manifests): helm
Logs from the misbehaving component (and any other relevant logs): helm.txt
Resource definition (possibly in YAML format) that caused the issue, without sensitive data: None

/kind bug

Hi @dipak140,
Thanks for submitting this issue.
I've tried to reproduce the issue, a clean installation on a K8s cluster (1.25+) worked fine.
Did you have a previous deployment of logging-operator on your cluster?
If you were actually upgrading a deployment, that was installed on an older K8s release that was upgraded at some point, the root cause can be the Helm upgrade process (https://github.com/helm/helm/issues/11287).

based on a community discussion this can be solved using https://ranchermanager.docs.rancher.com/how-to-guides/new-user-guides/authentication-permissions-and-global-configuration/pod-security-standards#cleaning-up-releases-after-a-kubernetes-v125-upgrade
https://discord.com/channels/1082023686028148877/1082624361963454545/1187721882439524412


Update auto-generated documentation

Repository: openmicroscopy/ome-documentation
Already up-to-date.
Generated by build OMERO-DEV-latest-docs-autogen#107.

--no-rebase

Content looks fine on staging

Merging to carry on with documentation changes.


Add tests

Just written the main script. Will write test soon.. Currently updating type annotations, testing with mypy, pylint extensions. Are you using this ?


if type is bool, return its value

working on duplicating this logic in kudu, and it looks like the code will try to expand isDisabledValue to an environment variable if we have "disabled":false
we can return its boolean value once we know its of JTokenType.Boolean

@watashiSHUN,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@fabiocav did you need to do something special on appveyor to enable C# 7? looks like those errors are appearing on the UT job

@mamaso yes, had to change the environment image to use VS 2017. I just updated the configuration for the UT project and kicked another build off.

@fabiocav thanks!

I'm seeing some failures unrelated to this change. Looks like we have some integration tests (needing secrets) incorrectly executing in the UT test suite. I'm flagging this PR to run the full set, but we need to have that fixed.


DHCP option 82 support

What would you like to be added:

https://www.juniper.net/documentation/us/en/software/junos/security-services/topics/concept/port-security-dhcp-option-82.html

https://github.com/yunionio/cloudpods/pull/13209


Please add Rocm support

WARNING:root:No GPU found. Conversion on CPU is very slow.
I would appreciate ROCM support. I don't have an nvidia gpu

(venv) (base) neuromancer :) > nougat
Traceback (most recent call last):
  File "/thearray/git/thegreatcaper/venv/bin/nougat", line 5, in <module>
    from predict import main
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/predict.py", line 18, in <module>
    from nougat import NougatModel
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/nougat/__init__.py", line 7, in <module>
    from .model import NougatConfig, NougatModel
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/nougat/model.py", line 17, in <module>
    import timm
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/timm/__init__.py", line 2, in <module>
    from .models import create_model, list_models, is_model, list_modules, model_entrypoint, \
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/timm/models/__init__.py", line 1, in <module>
    from .beit import *
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/timm/models/beit.py", line 29, in <module>
    from .helpers import build_model_with_cfg
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/timm/models/helpers.py", line 17, in <module>
    from .fx_features import FeatureGraphNet
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/timm/models/fx_features.py", line 10, in <module>
    from torchvision.models.feature_extraction import create_feature_extractor
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/torchvision/__init__.py", line 6, in <module>
    from torchvision import _meta_registrations, datasets, io, models, ops, transforms, utils
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/torchvision/_meta_registrations.py", line 163, in <module>
    @torch._custom_ops.impl_abstract("torchvision::nms")
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/torch/library.py", line 467, in inner
    handle = entry.abstract_impl.register(func_to_register, source)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/thearray/git/thegreatcaper/venv/lib/python3.12/site-packages/torch/_library/abstract_impl.py", line 30, in register
    if torch._C._dispatch_has_kernel_for_dispatch_key(self.qualname, "Meta"):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: operator torchvision::nms does not exist

(venv) (base) neuromancer :) > pip show torchvision
Name: torchvision
Version: 0.18.0
Summary: image and video datasets and models for torch deep learning
Home-page: https://github.com/pytorch/vision
Author: PyTorch Core Team
Author-email<EMAIL_ADDRESS>License: BSD
Location: /thearray/git/thegreatcaper/venv/lib/python3.12/site-packages
Requires: numpy, pillow, torch
Required-by: timm
(venv) (base) neuromancer :) > pip show torch
Name: torch
Version: 2.3.0+rocm6.0
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /thearray/git/thegreatcaper/venv/lib/python3.12/site-packages
Requires: filelock, fsspec, jinja2, networkx, sympy, typing-extensions
Required-by: lightning, pytorch-lightning, timm, torchaudio, torchmetrics, torchvision


[Detection/Instance-Segmentation] Adaptive Training Schedule Hook

Summary
This PR includes :

Add Adaptive Training Hook for Updating validation interval with LR scheduler & Early-Stop scheduler
Apply Adaptive Training Hook (validation interval only) to Instance-segmentation & Object-detection
EMA fix (https://github.com/openvinotoolkit/model_preparation_algorithm/pull/39) merged in this PR
Apply Adaptive EMA Momentum

First of all, we were able to set it to the corresponding value and obtain training time in accordance with detection and instance-segmentation. However, we will experiment with additional experiments + adaptive interval in this regard.

Hi @eunwoosh, This is the code for applying adaptive validation interval. This is applied in a small dataset and increases up to 5 interval. Please check if there is anything that affects HPO.

@harimkang I checked that HPO works well with this PR.

LGTM.


Move description of checkbox

Please move the description / title of a checkbox to the left side of it
Example:
[  ] Checkbox Title
Actually the title of the checkbox is on top of it.

we have to adjust the checkbox template to use following ht structure:

        <input name="nameForForm" id="checkbox1" type="checkbox" class="form-element">

    Checkbox 1

Oh, can you create a small info-description field on bottom of the checkbox?
Example:
[  ] finished contest
Check this, if the contest is finished

@berndhep do you have a design for something like this?

@wachterjohannes Yes of course… take a look at the Mothership ;-)

@berndhep then we could use the "info-text" of the template for that! I think that could be a good usecase for that.

There should exists a text label next to the checkbox. That's what they want, don´t they?

you i have understand and see it (= i had only written that we could use the info-text (which is already configurable) for that (=

Togglers and Checkboxes can be used without a label and only a text right next to it in Sulu 2.0.


Do not rebundle unchanged files

Here's a challenge. Currently it's painful to develop based on bundled builds (which you may want to do for debugging issues that only occur in the bundled version) because the entire app needs to be rebundled for every change. This could take several minutes (or longer; depending on the size of your app). It would be really awesome if the bundler were smart enough to only bundle changes in the tree.

Eh, that gets really tricky.  Sure you could do this semi-easily with a gulp watch task but the problem is if you add/remove a file, the tree needs to be rebuilt entirely since the change could have cascading effects.

Yeah it's certainly a challenge. I just wanted to post an issue here for the backlog. Maybe someone will want to pick it up.


Minor revision as new macOS Monterey version is coming up with early …

Minor revision as new macOS Monterey version is coming up with early builds out.
Instead of three latest released versions. Why not site the exact versions?
Microsoft Defender for Endpoint on macOS offers antivirus, endpoint detection and response (EDR), and vulnerability management capabilities for three released versions of macOS: Big Sur, Catalina, and Mojave.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


Installation fails at Lubuntu 20.04 and Julia 1.4.1

Hello
When I tried to install GLMakie ((@v1.4) pkg> add GLMakie), I got error "ERROR: Unable to automatically install 'Xorg_libpthread_stubs' from '/home/mikko/.julia/packages/Xorg_libpthread_stubs_jll/zqOOX/Artifacts.toml". What could be problem and how to fix it? All necessary packages are installed.

Looks like a Julia issue with artifacts. Maybe ask on discourse or slack!


[MenuItem] "&.Mui-selected" style override not working properly

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
The following theme config does not override the &.Mui-selected items style on initial render:
createTheme({
  palette: { primary: { main: "blue" } },
  components: { MuiMenuItem: { styleOverrides:  { root: { "&.Mui-selected": { backgroundColor: "red" } } } } }
})

The color changes after we interact with the page/form, but not before (if we have an initial value that is)
I have done a quick example in this codesandbox: https://codesandbox.io/s/mui-v5-bug-selected-theme-override-xbb1c
Expected behavior 🤔
You would expect this to be the initial color when we click the select button.
Steps to reproduce 🕹
Steps:

Set a palette.primary.main color on the theme config
Set a components.MuiMenuItem.styleOverrides.root."&.Mui-selected".backgroundColor color (different to primary main)
Create a Select input with a list of menu items (MenuItem)
The end result is the background color remains as 80% transparency of the primary.main color (assume this is the default), and ignores the override if an initial value is given

Example: https://codesandbox.io/s/mui-v5-bug-selected-theme-override-xbb1c
Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Those are the styles applied when the menu item is both selected and focused. You need to add that selector too. See https://codesandbox.io/s/mui-v5-bug-selected-theme-override-forked-cfy8f?file=/src/App.tsx (the orange background)

Those are the styles applied when the menu item is both selected and focused. You need to add that selector too. See https://codesandbox.io/s/mui-v5-bug-selected-theme-override-forked-cfy8f?file=/src/App.tsx (the orange background)

Brilliant, thank you!


Crypto: message-replay mitigation

There is however an issue: it breaks permalinks or reloading events from a store.
The reason is that a permalink is opened in a separate window with its own timeline. And this timeline decodes events for a second time.

LGTM


AspNetCore RegisterTypes Use Exported Types

When scanning for types in FluentValidationMvcExtensions.RegisterTypes we should use Assembly.GetExportedTypes instead of Assembly.GetTypes.This would prevent fluent validation from installing private/internal validators that are implementation details.
Change
private static void RegisterTypes(IEnumerable<Assembly> assembliesToRegister, IServiceCollection services)
        {
            var openGenericType = typeof(IValidator<>);

            var query = from a in assembliesToRegister.Distinct()
                        from type in a.GetTypes().Where(c => !(c.GetTypeInfo().IsAbstract || c.GetTypeInfo().IsGenericTypeDefinition))

to
private static void RegisterTypes(IEnumerable<Assembly> assembliesToRegister, IServiceCollection services)
        {
            var openGenericType = typeof(IValidator<>);

            var query = from a in assembliesToRegister.Distinct()
                        from type in a.GetExportedTypes().Where(c => !(c.GetTypeInfo().IsAbstract || c.GetTypeInfo().IsGenericTypeDefinition))

Thanks

This has been in place for quite a while now but I forgot to update the issue.


set yScaleBottomValue if minY was given

Set yScaleBottomValue correctly if a minY value > 0 is given

@ajdani thank you for finding the bug.
I've applied a bit different solution but with the same idea in this commit


Dev/thla/chore cleaning

GitHub Issue: #
PR Type
What kind of change does this PR introduce?

Bugfix
Feature
Code style update (formatting)
Refactoring (no functional changes, no api changes)
Build or CI related changes
Documentation content changes
Other... Please describe:

Description

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Interface members are XML documented
[ ] Documentation (XML or comments) has been added and/or existing documentation has been updated
[ ] Architecture documents have been updated
[ ] Tested on all relevant platforms

Other information

Internal Issue (If applicable):

Do we need to do this refactoring? Is it the right moment?

Fixed some annoying warning and it was fast.


RSS does not compile

Hi,
after multiple cleanups, remakes and complete build from scratch I cannot get rid of the error.
"sensor.rss not found, please recompile with Python.rss" can someone please explain why this is happening and how I can fix it?
Ubuntu: 19
Carla: 0.9.8
Python: 3.6
Much Regards,
Ali

Bump

The error code is pretty self-explanatory, especially if you have read the docs for the intel rss portions of CARLA. Intel's rss features in CARLA are distributed under LGPL. CARLA is distributed under a MIT license. This causes a conflict due to the copy-left of the LGPL. Intel only wants RSS to be used for research, so thats why it is LGPLed.
Now, in order to get the RSS features in CARLA, you are going to need to switch away from Ubuntu 19 down to Ubuntu 16 (with a possible migration to 18 coming soon), go and take a look at https://github.com/intel/ad-rss-lib, build all that stuff, and then, once it is built, build CARLA from source with the RSS features enabled. then it should all work nice and dandy.


Client Allocation Request for: HELIOS

Client Allocation Questions
Core Information

Name: HELIOS
Website / Social Media: https://www.huilianyi.com/
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f1n7jfmtls4l3gqy23d3nhjgvxjca6kxkt4dkb5wq
Notary Requested: IPFSUnion

@shary980506 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Thank you for your request. I'd like to know more about your business. Can you kindly answer my following questions?

individual and organization

Please introduce yourself and your team briefly, and show me some your social media accounts such as GitHub, email address, WeChat official account etc.
For your organization, Please describe its time since inception, registration country，scale, named officers, project, industry reputation etc.

track record

Regarding your data you intend to store, What’s the original source of those data, and do you have any proof of ownership or authorization?
Has your organization obtained DataCap before?
Please list any of your honor and disputes in the field of Filecoin.

use plan

Please describe your plan to use DataCap briefly? (such as use case, miner etc.)

Dear,IPFSUnion

individual and organization
1.Please introduce yourself and your team briefly, and show me some your social media accounts such as GitHub, email address, WeChat official account etc.
Helios is the leading full-process expense management platform in China. It intensively cultivates the vertical fields of enterprise reimbursement and expense control, and continues to promote the electronic reimbursement process. Cost control consulting methodology practiced by leading companies.
Wechat：汇联易
2.For your organization, Please describe its time since inception, registration country，scale, named officers, project, industry reputation etc.
Helios was founded in 2016, we are located in Shanghai, China, since then we have more than 300 employees, serving many famous companies, Tencent, Dior, Fosun, BYD, etc.

track record
1.Regarding your data you intend to store, What’s the original source of those data, and do you have any proof of ownership or authorization?
We intend to store some historical image data for the development of photo recognition systems. The data comes from public collection.
2.Has your organization obtained DataCap before?
None.
3.Please list any of your honor and disputes in the field of Filecoin.
None.

use plan
1.Please describe your plan to use DataCap briefly? (such as use case, miner etc.)
We will abide by the relevant rules to use. We plan to find three miners in China to cooperate, mainly depending on the scale of the miners and the stability of the service.

Hi Helios. I decide to approve 2TiB for you in the first time. Based on your subsequent performance, I may allocate more next time.

Ok, thank you for your approval.


Fixed server tests memory leak

There has been a slow bleed memory leak in server unit tests.  It recently tipped from "annoying" to "blocking", so I dug in.
The root cause is that express-rate-limit starts a setInterval which is never stopped.  Each one holds a reference to the graph, which increases memory usage by about 50mb per test.
Unfortunately, the fix is not that simple.  express-rate-limit does not provide any stop or clear or shutdown utilities.  In fact, that handle to the timer is discarded as soon as it is created.  That is the reason for the new ratelimit.ts file.  The MemoryStore is almost 100% copied from the express-rate-limit implementation, except the memory leak is fixed.  I'll submit a PR to see if we can get it fixed upstream.
Along the way, I did some minor refactoring, because finding this leak was obnoxiously tedious.

Removed 2 index.ts files that did nothing be re-export.

Before, you could import { Repository } from './fhir';
Now you need to import { Repository } from './fhir/repo';
This helps keep the dependency graph tighter, and makes it easier to isolate problems

Added initAppServices() and shutdownApp()

Once upon a time, there was relatively little setup/shutdown code for unit tests, so it was copy/pasted in many tests
Over time, we have accumulated more setup/shutdown code, and it keeps getting copy/pasted
Rather than yet again copy/pasting shutdownRateLimiter, I added helper methods, which leads to a net decrease in code

Coverage decreased (-0.05%) to 94.384% when pulling 21c674a7d28e5ce140c2c39ba462babf98fa8fc4 on cody-server-tests-mem-leak into 9ebeebb7c35e4bfbd495c164c3d5c105ee4907b0 on main.

Submitted issue with express-rate-limit: https://github.com/nfriedly/express-rate-limit/issues/320


Issue 1659 table value alignment

Github Issue
https://github.com/h2oai/wave/issues/1659
Description

Added an option to the ui.table component to allow users to specify alignment per column.
Alignment values could be one of ['left', 'center', 'right']
Added new example table_column_alignment.py so that original table.py remains unchanged
Added this new feature to table.md
Ran make generate before raising PR

Successful local testing

@mturoci thanks a lot for these suggestions! I have updated the PR as per your comments, kindly review and let me know if the changes look good

Thanks for your review @mturoci! I have addressed your review comments, please let me know if the changes look good.


Add a spec to document /messages URL validation

Previously the URL validation for messages wasn't clear for someone testing the API. This makes the validation explicit and may help future developers.

Oops - finally added that to my global ignore file!


Clarify instructions for octomments server

I deployed an installation of octomments server for my website but I had some issues following the instructions in the README file for the server part of octomments. This PR clarifies those instructions according to my understanding of how octomments server should be set up.
The instructions seemed to suggest creating both a Github App and a Github OAuth App. I have changed this to creating just a Github App (since they are preferred nowadays over Github OAuth Apps). However for the Github App to work, it has to be installed in the repo to which the comments feature is to be added (my website in this case). I have added extra instructions for this part of the process.
Thanks for the work on this repo!

Thanks! 🚀

By the way, I'm thinking of adding a section "Who is using Octomments".


megawifi: add tasking support

My first patch for the MegaWiFi support. This adds multitasking support (with a supervisor task and an user task and a basic locking mechanism).
As I am not familiar with SGDK sources, I might have done something weird. Please if you see something strange let me know.
I have made the compilation of the introduced code conditional as cleanly as I could. But adding too many #if is not very elegant, so if you have suggestions about doing this in a different manner, I can try.

Added all the remaining sources, adapted to work with SGDK, without using newlib (I had to implement memcmp, strchr and workaround some problems due to some stuff from stdint.h and stdbool.h missing.

And added an example. With this, I think all I wanted to add is there.
BTW, is there a problem with all this Codacity nonsense? It seems most (all?) the problems it finds are due to problems with headers (things like not finding definitions for uint8_t, etc.).

And added an example. With this, I think all I wanted to add is there.
BTW, is there a problem with all this Codacity nonsense? It seems most (all?) the problems it finds are due to problems with headers (things like not finding definitions for uint8_t, etc.).

Thanks the PR @doragasu !
To be honest there is more code than expected but i've to say it's a really nice framework you made there 👍
I saw you even implemented a cooperative multitasking system through the v-int callback. It's really useful and something that i always wanted to add in SGDK. Do you think you can separate it from the MegaWifi framework itself so we can enable it separately ? I would even say that it could probably be always enable if the overhead isn't too heavy (we probably need to test is there is an active user task to not yield switch to user mode if not required).
Also i think it would be nice to have a sub folder like src/ext to separate base SGDK code from extension (or external) modules. So the MegaWifi related sources could be located in this src/ext sub folder.
And thanks for the LD script fix, i forgot to update the stack location after having changed the RAM location !

Also glad you were able to get rid of the newlib requirement ! Can i borrow your memcpm and strchr functions so they will be directly included in SGDK mem / string unit ?

Thanks for the review!
Sure, no problem, I can separate the multitasking from all the other MegaWiFi stuff. BTW the multitasking is preemptive (i.e. non cooperative). My previous approach (the one I implemented using setjmp()/longjmp() from newlib) used cooperative multitasking (using loop functions that had to return after doing the work), but most devs I talked to found that approach confusing, so I hope the preemptive multitasking is easier to use.
About the performance penalty of the multitasking when not being actively used, I could compute an estimate. I tried the Sonic example with it enabled and didn't see any difference in the CPU usage marker, but that's not a very objective test :-).
Also no problem moving my source files under src/ext. For the include files, do you want them to be in inc/ext? or maybe inc/mw (for MegaWifi) would be OK?
About my memcmp() and strchr(), they are not optimized, but sure, you can grab them if you want (I think memcmp() could be heavily optimized when buffers are aligned by subtracting long words instead of bytes) . If I can make a suggestion, these functions along with should be better in string.h, along with memset() and memcpy() to increase portability (they are defined in string.h in the standard C library). But if I can make even a bigger suggestion, for optimal portability I still think it would be better embracing newlib (yeah, yeah, I had to try :-)

Added a multitasking example.
I have also found a bug in my multitasking implementation :-|. If you press the reset button in the console while the user task is running, you will get an access violation exception (presumably when the startup code tries to move 0x2700 to the status register). I think I should be able to fix it using a trap during startup to go back to privileged mode. But that will be tomorrow :-)

Hum, about the bug, it is not happening in real hardware, so maybe it is a bug in Blastem.

Last commit moves MegaWiFi sources to src/ext/mw and includes to inc/ext/mw. It also enables multitasking by default (but MegaWiFi continues disabled by default). Tested sonic, megawifi and multitask examples and everything builds and runs nice.
BTW the bug in my previous comments was in blastem, it was already fixed in nightly build.

I did the cost calculation of the multitasking. If I did not screw the cycle counts, this is the result:

The interrupt code adds 362 cycles:

The tsk_user_yield() function adds 344 cycles (plus the jsr and maybe also clobberable registers save by caller):

Total is 706 CPU cycles (about a scanline and half). Not huge but not negligible either.
I will add the check in vdp.c to only call tsk_user_yield() if tasking is enabled. This should skip the call to tsk_user_yield() and lower the cycle count in VINT to just 18 (the btst and the bne.s). This way we should be able to keep ENABLE_MULTITASK always on with a negligible CPU penalty when not using it.

Wow, thanks for all the modifications ! That sounds perfect :) I think it's indeed better to add a check as you did, indeed the tsk_user_yield() is still a bit taxing for the vblank handler that we want to keep as fast as possible.
About the memcpy / memcmp.. functions, i know they should be located in string.h but honestly for SGDK i don't necessary want to mimic too much how standard C organize stuffs, specially here with these memxxx functions which for me shouldn't really stand in the string.h unit. Also when you use SGDK, you're supposed to just include "genesis.h" so for portability i think it's better to have something like :
#if TARGET == MEGA_DRIVE
  #include <genesis.h>
#else
  #include <string.h>
  #include ..
#endif

Anyway you will need that switch in your code in different place as you will have specific code for MEGA_DRIVE target.
And that should be enough to make the portable portions of code really portable as SGDK still try to use standard C function names (or alias for it) when it can.
Another point is that i really don't want SGDK to rely on any external library, i would even get rid of libgcc if I can but that would require quite a bit of work. There are already a switch to allow newlib compatibility but from what i read it doesn't work really well.

Done, now yielding to user task during VBlank wait must be explicitly enabled by calling VDP_setVBlankUserYield(TRUE).
I think with this all requested changes have been completed. @Stephane-D if you agree, now that CPU performance penalty when not using multitasking is negligible, I would like to remove the ENABLE_MULTITASK define to leave it always available and make the code cleaner (without all the ifdefs).

Done, now yielding to user task during VBlank wait must be explicitly enabled by calling VDP_setVBlankUserYield(TRUE).
I think with this all requested changes have been completed. @Stephane-D if you agree, now that CPU performance penalty when not using multitasking is negligible, I would like to remove the ENABLE_MULTITASK define to leave it always available and make the code cleaner (without all the ifdefs).

I think we can indeed remove the ENABLE_MULTITASK define but don't worry I think I will merge as it is now and do not requires more changes from your side :) I will probably make some minor modifications to make it more SGDK conventions friendly if you don't mind (move the tsk_xx methods to a separate unit for instance and maybe rename them using TSK_xx as the others units).

Thanks for the merge!
I have finished rebasing the branch to master, but it seems I arrived late ^_^U.
I had a look to the new master and I think it will not be able to build sources in src/ext as is. I will open another PR with the changes.

This time it's me which is late, i'm currently doing a bit of refactoring and i fixed the src/ext stuff.
I will push my changes as soon it will be complete (probably in 1 hour so i can test everything is ok)

No problem. Thanks!!!

Ok i just pushed the changes. I hope i didn't break anything but it looks like to be ok (tested with partial Megawifi support from BlastEm) =)

Great, I'll give it a look. Thanks!


Next/prev paging of entrants in order

The next/prev buttons for pagin through entrants needs to be sensitive to the order of the original list. This involves changing the relationship test of the SQL.

Now restricted to lists in EntrantID order only.


Applying the AppExtension fix to all NuGets + version bumps

Need to consider iOS App extensions as well for Xamarin.Build.Download.
Check for the IsAppExtension property.

Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message.

Hello! I'm the build bot for the Mono project.
I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.
Contributors can ignore this message.


doesn't work for airflow from the AZ market place.

This doesn't appear to work on Airflow for Ubuntu 20.04.02 LTS from the azure market place. After installation, the sas provider does not appear in the connections drop-down list.
Are there any pre-reqs for this airflow provider i.e. a certain version of Python being used for example?
Any suggested troubleshooting notes to review for novices new to cloud/python/airflow?

Required python version is >= 3.7.


[docs] Fix grammar on the Breakpoints page

Signed-off-by: Dustin-Digitar<EMAIL_ADDRESS>

[x] I have followed (at least) the PR section of the contributing guide.

"The" really isn't necessary in this sentence—it reads just fine as is. That said, "Material Design" should be capitalized here. Would you mind making that change instead @Dustin-Digitar ?


Using TAB to close HTML editor doesn't work

Hi guys,
Thanks for giving us this great lib!!!
The pattern of closing the editor when pressing TAB does not work in the HTML editor, which brings unpredictable errors, such as when pressing TAB in the example of the site:
https://bossanova.uk/jspreadsheet/v4/examples/richtext-html-editor
The editor remains open as shown in the print of the page even after leaving the worksheet and going to the end of the page:

Does anyone know how to fix this?
Thanks a lot!
Marcos

Hi,
In waiting fix on jspreadsheet, you can set this on event oneditionstart
oneditionstart: function() {
    document.querySelector(".jexcel_richtext").editor.options.onblur = function() {
       jspreadsheet.current.closeEditor(jspreadsheet.current.getCellFromCoords(...jspreadsheet.current.selectedCell), true);
   }
}

Is not a good solution, but you can close on tab editor

Hi @GBonnaire,
Thanks for reply.
I've tried that fix, but
document.querySelector(".jexcel_richtext")
is returnning "null".
I've tried others ways to get ".jexcel_richtext" like using getElementsByClassName's method but in all this cases, it return null.
Do you have any idea what is wrong?

this editor is accessible only if cell is on edit mode

Hi @GBonnaire,
I looked it up here and indeed the oneditionstart event is called before the creation of the jexcel_richtext component.
https://github.com/jspreadsheet/ce/blob/fbcc95aaf18e96b831e0770717b05210be4a297c/dist/index.js#L7836
https://github.com/jspreadsheet/ce/blob/fbcc95aaf18e96b831e0770717b05210be4a297c/dist/index.js#L7842
https://github.com/jspreadsheet/ce/blob/fbcc95aaf18e96b831e0770717b05210be4a297c/dist/index.js#L7961
So inside the oneditionstart event,
document.querySelector(".jexcel_richtext") will always return null.
So I would have to fix this issue in another event after creating the jexcel_richtext component (line 7961).
Would there be any other event that could fit this?
Thanks

Absolutely.
try this on your configuration instead of previous event configuration
oncreateeditor: function(el, cell, x, y, editor) {
        setTimeout(function() {
            editor.querySelector(".jexcel_richtext").editor.options.onblur = function() {
                jspreadsheet.current.closeEditor(jspreadsheet.current.getCellFromCoords(...jspreadsheet.current.selectedCell), true);
            }
        }, 1);
    }

Hi @GBonnaire,
Thanks for your reply.
This actually works in the oncreateeditor event.
Unfortunately it generates a side effect tough:
The ESC key also generates the unblur and thus it loses the effect of canceling the change in the HTML editor. :(
Anyway, I found how to solve this in jspreadsheet ce code.
I'm going to send a pull request.

@GBonnaire
Like you describe with your code, "jspreadsheet.current.closeEditor" have to trigged when TAB key is pressed for HTML editors.
So, it is the same behavior implemented for calendars components.
I found it in to the code of calendar component and fix it in my commit:
https://github.com/jspreadsheet/ce/commit/9fa636d054bcd0068d219509a149bdb5aee6ba48


Update Composer dependencies (2022-12-28-00-10)

Visual regression test passed!
View the visual regression test report


Allow easier customization of login redirect URL

It would be super useful if the backend or strategy could have methods to implement to get login, new user, error, etc. redirect URL's. Right now they are pulled from settings (like Django's LOGIN_REDIRECT_URL), but I would like to be able to use session data and/or any other request data to build the redirects.

Technically that's doable by overriding the strategy get_setting method.


Definening integrated development environment and libraries #SoftwareDesign (Onat Çipli - 010517 - PDM)

Libraries of the system.txt


make: *** No rule to make target 'obj/yolt.o', needed by 'darknet'.  Stop

Hi, thank you for your sharing firstly. I have deployed your code in docker but when i try to compile the Makefile it makes mistake like that:
make: *** No rule to make target 'obj/yolt.o', needed by 'darknet'.  Stop
And I note that the line 24 in Makefile "COMMON=-I/Users/avanetten/anaconda/pkgs/libgdal-2.1.0-0/include" , I just use python2.7 of system and already add PPA for "pip install gdal",  but still make failed.
So how should I fix it?

I ran into the same trouble when I tried to compile the codes. You need to modify the Makefile and corresponding C files. Since that the author upgraded "yolt" to "yolt2". You can try my forked version if you like.
Moreover, the Dockfile did not include "imagemagick" used in the python scripts.

@yghlc Yes, I just installed anaconda2 for "conda install gdal" and imagemagick for "convert" command after compiled the Dockerfile. And I found the "pip install tensorflow-gpu"  in Dockfile may install the newest tensorflow which may not be compatible with CUDA8, so I specified the version of keras==2.1.5 and tensorflow-gpu==1.4.0, and the docker works well.  I will try your forked version to correct the Makefile and corresponding C files, thank you!

The latest commit should address these issues.


Bump identity API server version to 1.2.62

$Subject
Related PRs:

https://github.com/wso2/identity-api-server/pull/463

Related Issue:

https://github.com/wso2/product-is/issues/15998

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5275176194


Add hours of operation

Please add hours of operation on the contact page.
Monday to Friday
11 a.m. to 11 p.m.

Good job!


Improve "system requirements" in installation docs

https://flutter.io/setup-macos/#system-requirements
Does not tell how to install the tools.
Even if there is no single approved way, it would be great to have one concrete suggestion, for example using Homebrew (https://apple.stackexchange.com/a/149088/238346)

Issue moved to flutter/website #1971 via ZenHub


Project status?

I've used Figaro for years on Heroku, and am even using it at present with Capistrano. I'm not a fan of Rails encrypted secrets, and even with the Rails 6 enhancements it is not as easy to use and full-featured as Figaro!
What is the status of this project? I see there is a v2 branch but it has been quiet for a while.
Do you need some help to get this released @laserlemon ?

Looks like @laserlemon is offline ... is anyone maintaining a fork of this project?

We have a fork with a couple of changes (looks like 255 other people do too!). It may mean someone (...) will have to make a statement of intent to hard fork, rename and release as a new gem if we can't get hold him. I see he actually works at Github. I will try contacting him there.

Hello and sorry for dropping off the map! I have all of the excuses you'd expect: family and full-time job. I still care very much about Figaro but have had difficulty finding and prioritizing time for its developemnt and gauging its continued value in the community. I'm attempting to provide some solution to both of those problems by opening my sponsorship profile and I've added an UPDATE section to the README with this information as well.
Sorry this answer doesn't provide more immediate results, but I hope that you'll stick with me as I try to navigate back to active Figaro development!

@laserlemon are you open to taking on additional maintainers?

Yes, I'll offer too. Its a great gem, we'd be very keen to keep it maintained.

Yes, I'll offer too. Its a great gem, we'd be very keen to keep it maintained.

What is the status of the version 2 branch? If it's far from releasable, it may be best to release a new 1.x version with updated dependencies.

Rails is pushing for using encrypted secrets now, but I still prefer the approach of this Gem. I have mostly worked with Heroku and VPS / Capistrano setups. Not sure if there is still a place for with Kubernetes...

Not everyone is moving to Kubernetes and this gem works fine with Heroku. I'd like to see a new release simply because the current one is preventing me from keeping my dependencies up-to-date. If that's not going to happen I'll try to switch to dotenv-rails.

Yes, agree, if we can just get a simple 1.x update with newer dependency ranges, that would be helpful.

+1 for a dependency update!

+2 for a dependency update!

@dotconde version 1.2.0 was released some time ago and should be fully up-to-date regarding dependencies.

Thank you @mvz

Note, figaro has now been forked to a new gem called figjam.
It can be found here:
https://github.com/hlascelles/figjam
https://rubygems.org/gems/figjam
It is initially a direct fork of figaro (without the heroku functionality), even replicating the Figaro module namespace. . PRs from the figaro repo will be ported over in the coming weeks/months.
Do submit PRs there!


Network Threat Analytics - Throughput Anomaly Detector

Detect throughput anomalies in Antrea flows using well-known algorithms.
The feature should be able to do detection for individual flows, or for a given pod traffic (source and destination)
The feature should provide:

Spark Application for Network Flow Analysis
Theia manager integration
CLI support

Append next steps in v0.6 release after PR #152 merged:

Expand the scope of throughput anomaly detection beyond each endpoint-to-endpoint flow, for example:

Monitor the aggregated in/out throughput of a given Pod
Monitor the aggregated in/out throughput of a Pod sets (Pod label)
Monitor the aggregated in/out throughput of a given Service

Develop CLI commands and Theia manager integration for the Spark Streaming version job: https://github.com/antrea-io/theia/blob/b1bcc1a2b48b1d2617afbd6f5fca53ac716f08f5/plugins/anomaly-detection/SparkStreaming.py


Twitter feed not showing anything

Hi,
using the last version of the theme and customized the folder sidebar within user/pages to just match the correct twitter account. But the sidebar with the Twitter card seems broken:

and this is the content of the sidebar/default.md file:
---
title: Sidebar
routable: false
visible: false
position: top
---

[twitter url="https://twitter.com/FaunaliaGIS" text="Tweets by @FaunaliaGIS" height="600"]

Hi @ghtmtt , I can't seem to replicate this issue. My online demo site (https://demo.hibbittsdesign.org/grav-open-publishing-quark/) is still showing the Twitter feed as is my local test site.
Are you still seeing this issue? What version of Grav are you using?
Paul

Hi @paulhibbitts weird...
Grav version is 1.6.22
I'm debugging it locally but also in production

If you would like you could send me your user folder on Discord (https://discord.gg/5VhYVkR) and I could test on my local setup. Maybe another Plugin is causing a conflict?


Fixed typo on default routes (uninstalled)

Just heard about this project, Artem. Would love to spread the word with the rest of the Atlassian dev community. 👍

@mansilladev great, thanks!


Update previous() method.

If previous URL is same as current URL then it should be return to the fallback URL otherwise return /

Seems to break tests and styleci


Documentation improvements

Embed StackBlitz component examples directly in our documentation using StackBlitz SDK.
https://stackblitz.com/docs#embed-options
Allow user to expand and collapse API documentation segments.
Show code snippets using highlightjs for syntax highlighting.

Open to suggestions from community? What would help you?

Hi,
I think it would be very useful to have code snippets (HTML and TypeScript) directly into the documentation, without having to open StackBlitz.
I'm skeptical about the usefulness of embedding StackBlitz into the documentation instead, because as far as I've seen it's very heavy (the dev server can easily take two minutes to start).

@daniloarcidiacono Completely agree. I've been researching since posting this issue, and I'm also not sold on embedding StackBlitz examples.
Using highlightjs or some variant should get us exactly what we're looking for..
Thanks for the feedback!

Tackling improved documentation next..

This is coming along nicely. Implemented lazy loaded modules, abstracted documentation generation into shared component, and making use of mdc-tab for separating api and examples.

Sample of the redesign..

@daniloarcidiacono Thanks for the suggestion! 💎
Implemented tabs to display HTML/Sass/TS example code when available. I'll post another sample from dialog, since it's probably the most complex.

@daniloarcidiacono Here's the largest of the bunch, dialog..

Nice, the documentation now looks much more quick to consult!

Thanks, working to get this deployed today.

Planning to phase out individual StackBlitz examples. Instead a single StackBlitz starter project for playground, and duplicating issues.


Use arguments in dameon_windows.go install method

This fork fix the install method for Windows that didn't use the args parameters in the method.
It just creates the list with the original parameters and append the arguments into that list.

thanks


Add handling of url:s using regexp on http

fixed by https://github.com/bjornregnell/new-in-Scala3/commit/6773931b5894e9d357b29dda0072bc0cf08bdae4


script should use rasa methods

https://github.com/RasaHQ/NLU-training-data/blob/f99c3ad15df7bdd98788062f2ac4d36f5dac5175/how-to-remove-entities/entity-remove-script#L12
@JEM-Mosig (@EmmaWightman said you wrote this script, right?)
this should use rasa 's built in methods, rather than a regex. that way this also works for the json format, here's a rough example:
from rasa.nlu.training_data import load_data

data = load_data('test.md')

for e in data.training_examples:
    e.set("entities", [])

with open('no_entities.md', 'w') as f:
    f.write(data.nlu_as_markdown())

Thank you for taking a look at this @amn41

Since we resolved by adding @JEM-Mosig as a collaborator and refreshing the PR, will close this issue. Thank you for the support! :)


Skills sample does not work with teams channel

Sample information

Sample type: samples/javascript_nodejs/80.skills-simple-bot-to-bot
Sample language: nodejs
Sample name: 80.skills-simple-bot-to-bot

Describe the bug
Bot to Bot conversations using the MS Teams channel fails with "/api/skills/vs/converations/msteams:{converationsId}/activities/{activityId} doesnt exist
To Reproduce
Steps to reproduce the behavior:
Simply run the sample using MS Teams channel instead of bot emulator

@anusharavindrar I dont have a screenshot of the error as I have resolved the issue. The issue should be apparent simply running sample using a Teams channel. The issue was due to the way conversation ids are formatted on teams cause the parent bot look up of the conversation id.

@01 Thank you for reporting this and glad that you could resolve your issue. I will have someone look into this . If you have any further issues, please feel free to open a new issue.


testing if this works with a PR

...'cause it's not working for PRs in mocha

surprise surprise ;)

I will take a look.

does a PR have access to the secrets?


All tests pass with purescript-aff 0.16.1

I have a project that chokes on bower install because it requires both purescript-aff 0.13.0 (via purescript-express) and 0.16.1 (via another dependency).
Adding "purescript-aff": "^0.16.1" to bower resolutions has not fixed the issue.
All tests pass with 0.16.1. Do you mind updating?

Hi @jedahu,
Nope, I don't mind. Dependencies are updated in purescript-express 0.3.3.

Thanks @dancingrobot84.


Question about Zombie processes

Thank you for this Docker image, I wrote up how to use it with OpenFaaS, and gave a credit / link back to yourselves.
Web scraping that just works with OpenFaaS with Puppeteer
Just today an openfaas user reported seeing zombie chrome processes, so I built a test container to reproduce it:
docker run --name puppets -t -i -p 8080:8080 alexellis2/puppets:latest

curl localhost:8080
curl localhost:8080
curl localhost:8080
curl localhost:8080

[alex@nuc openfaas-puppeteer-template]$ docker exec -ti puppets ps -ef
UID          PID    PPID  C STIME TTY          TIME CMD
app            1       0  1 14:58 pts/0    00:00:00 fwatchdog
app           12       1  2 14:58 pts/0    00:00:00 node index.js
app           39       1  0 14:58 ?        00:00:00 [chrome] <defunct>
app           40       1  0 14:58 ?        00:00:00 [chrome] <defunct>
app           57       1  0 14:58 ?        00:00:00 [chrome] <defunct>
app           61       1  0 14:58 ?        00:00:00 [chrome] <defunct>
app           66       1  4 14:58 ?        00:00:00 [chrome] <defunct>
app          103       0 43 14:58 pts/1    00:00:00 ps -ef
[alex@nuc openfaas-puppeteer-template]$

This issue looks to be related:
https://github.com/puppeteer/puppeteer/issues/1825
'use strict'
const assert = require('assert')
const puppeteer = require('puppeteer')

module.exports = async (event, context) => {
  let browser
  let page

  browser = await puppeteer.launch({
    args: [
      // Required for Docker version of Puppeteer
      '--no-sandbox',
      '--disable-setuid-sandbox',
      // This will write shared memory files into /tmp instead of /dev/shm,
      // because Docker’s default for /dev/shm is 64MB
      '--disable-dev-shm-usage'
    ]
  })

  const browserVersion = await browser.version()
  console.log(`Started ${browserVersion}`)
  page = await browser.newPage()
  let uri = "https://inlets.dev/blog/"
  if(event.body && event.body.uri) {
    uri = event.body.uri
  }

  const response = await page.goto(uri)
  console.log("OK","for",uri,response.ok())

  let title = await page.title()
  const result = {
    "title": title
  }

  await browser.close()

  // added recently, didn't change behaviour
  await browser.disconnect()

  return context
    .status(200)
    .succeed(result)
}

Would you have any suggestions for us? @toolmantim
Have you noticed similar issues yourself? (I didn't see anything in the issue tracker here, but Puppeteer appears to have many issues about it upstream)

I personally added lines on top of the --no-zygote and --single-process flags to my DOCKERFILE to have the ENTRYPOINT be ["/tini", "--"] and then actually running node was my CMD.


Why is the official website accurate to the minute, and only accurate to the day when I set it myself

You can always pass --date-spec=min, e.g.,
# goaccess access.log --log-format=COMBINED --date-spec=min --keep-last=15

Let me know if that helps.

Can it be accurate to the second？

to the minute.

So how do I know the queries per second for a web development service bu using it

Are you asking about how many visitors or hits you get per second, or do you want to know how to set up GoAccess for real-time updates? If it’s the first one, there’s no way to show hits or visitors per second. But if you’re looking for real-time updates, just use the --real-time-html option, and it’ll refresh the report as data comes in, e.g., every second.


BGE Grid - Unpaid payments from Closed Opportunities showing on review donations modal

Unpaid payments belonging to Closed Opportunities are currently being incorrectly shown in the Review Donations modal from the BGE grid.
Review donations modal should display only the following:
For Payments:
Unpaid Payments from open opportunities with multiple unpaid payments
Unpaid Payments from open opportunities with both paid and unpaid payments
Unpaid Payments from open opportunities with a single unpaid payment
For Opportunities:
Opportunities in any open stage with no payments
Opportunities in any open stage with multiple paid payments
Preconditions:
Having a closed opportunity with an unpaid payment
Being in BGE grid
Steps to reproduce:
Go to BGE and open access any batch
Use the lookup field to look for the Contact/Account for the closed opportunity
Click on Review Donations
Expected results:
A spinner will briefly load but no Review Donations link will be shown since there are no valid Payments or Opportunities to be shown in the modal.
Actual results:
Review Donations link appears and the modal displays the Payments belonging to a closed opportunity.

**lurch: attach W-031021

Tracking W-031021

Included in beta release 3.156 (Beta 11)


Adding custom keycloak theme for chart

FEATURE REQUEST
I think we need the possibility to add a configurable PVC in order to be able to provide custom keyloack themes to the deployments.
At least I don't know a better way for providing a custom theme. Don't want to build a custom keycloak image in order to do so. :/
Which chart:
Keycloak
What happened:
Theme not configurable as theres no optional mount point in the chart

@unguiculus thanks for making it possible to have extra volumes. Providing the theme directory as volume seems still tricky though. Apart from manually creating the volume and cp, there should be an easier way to configure through helm from a local directory.  sadly configmap can't be recursive.

I'll try and look into this. Will need it myself.

We actually switched to a custom derived theme as suggested by keycloak for the time being as setting up volumes is too much effort for us in comparison. Recursive configmaps would be perfect..

@Flowkap Could you provide more details on your current solution? Also, I'm not sure what you mean by "recursive configmaps."

I played around with this and have a solution using an init container as theme provider. The statefulset gets an option to specify arbitrary init containers. The theme has to be packaged up into a Docker image.
Dockerfile
FROM busybox
COPY my_theme /my_theme

Custom values.yaml
keycloak:
  extraInitContainers:
    - name: theme-provider
      image: myuser/mytheme:1
      imagePullPolicy: IfNotPresent
      command:
        - sh
      args:
        - -c
        - |
          echo "Copying theme..."
          cp -R /mytheme/* /theme
      volumeMounts:
        - name: theme
          mountPath: /theme

  extraVolumeMounts:
    - name: theme
      mountPath: /opt/jboss/keycloak/themes/mytheme

  extraVolumes:
    - name: theme
      emptyDir: {}

I'll create a PR for this. Feedback welcome.

Hi, thx for this ability, but how I can pass pullSecrets or imagePullSecrets if my theme image under private repo?
I have tried:
extraInitContainers: |
    - name: theme-provider
      image: myuser/mytheme:1
      imagePullPolicy: IfNotPresent
      imagePullSecrets:
        - docker-secret
      command:
        - sh
      args:
        - -c
        - |
          echo "Copying theme..."
          cp -R /mytheme/* /theme
      volumeMounts:
        - name: theme
          mountPath: /theme

or
extraInitContainers: |
    - name: theme-provider
      image:
        ......
        pullSecrets:
          - docker-secret
      command:
        - sh
      args:
        - -c
        - |
          echo "Copying theme..."
          cp -R /mytheme/* /theme
      volumeMounts:
        - name: theme
          mountPath: /theme

and no luck

This fix is not working for me. I can see only empty mounted directory in container even though I've populated some files in init container.

I tried using this example (to mount providers instead of themes), but I think I have a permissions problem so keycloak doesn't see my jars.  The directory that is mounted has permissions root jboss instead of jboss root like the others:
[jboss@keycloak-id-dev-0 keycloak]$ ls -l /opt/jboss/keycloak/
total 528
drwxrwx--x 3 jboss root    4096 Apr 23 14:09 bin
drwxrwxr-x 7 jboss root      92 Apr 23 14:09 docs
drwxrwx--x 5 jboss root      50 Apr 17 06:24 domain
-rw-rw-r-- 1 jboss root  488551 Apr 17 06:24 jboss-modules.jar
-rw-rw-r-- 1 jboss root   10637 Apr 17 06:24 License.html
-rw-rw-r-- 1 jboss root   26530 Apr 17 06:24 LICENSE.txt
drwxrwxr-x 3 jboss root      39 Apr 23 14:09 modules
drwxrwsrwx 2 root  jboss     44 Jun 25 19:56 providers
drwxrwx--x 1 jboss root      61 Apr 23 14:09 standalone
drwxrwxr-x 5 jboss root      76 Apr 23 14:08 themes
-rw-rw-r-- 1 jboss root      25 Mar  4 07:16 version.txt
drwxrwxr-x 2 jboss root      42 Apr 23 14:09 welcome-content

Any idea how to fix that using the instructions above?
(Note I am now using codecentric/keycloak chart 4.16.0)

@bentatham
You can solve the permissions issue by using this config:
extraInitContainers: |
    - name: theme-provider
      image: gcr.io/my-project/keycloak-my-theme:v0.1.0
      imagePullPolicy: IfNotPresent
      resources:
        limits:
          cpu: "20m"
          memory: "64Mi"
        requests:
          cpu: "10m"
          memory: "32Mi"
      command:
        - sh
      args:
        - -c
        - |
          echo "Copying My theme..."
          cp -R /my-theme/* /theme
      volumeMounts:
        - name: theme
          mountPath: /theme

  ## Custom script that is run before Keycloak is started.
  preStartScript: mkdir -p /opt/jboss/keycloak/themes/my-theme && cp -R /theme/* /opt/jboss/keycloak/themes/my-theme/

  extraVolumes: |
    - name: theme
      mountPath: /theme

  extraVolumeMounts: |
    - name: theme
      mountPath: /theme
      emptyDir: {}

My Dockerfile contains only:
FROM alpine:latest

COPY my-theme /my-theme
CMD ["sh"]

Hope this helps.

It works for me in the sense that I can see the my_theme folder in /opt/jboss/keycloak/themes but if I try to actually use the custom themes I am unable to select them in the Admin Console.
Any suggestions?


Help wanted with turbolinks capybara test

Any volunteer for writing this test case:

Be sure turbolinks is running in spec/dummy app
Visit root url, then a child link. (one case with server rendered page, one without)
Hit back button. The react input fields disappear due to current turoblinks issue.
Hit forward button. The react input fields disappeared.

This issue relates to #105

Added in https://github.com/shakacode/react_on_rails/commit/648b71c9e094e41f41b2980ea835469b04564b27

Too late, merged in https://github.com/shakacode/react_on_rails/pull/68


Fix crash in tcp_proxy

Closing the upstream connection is not safe from the Filter destructor,
because it triggers events back into the downstream connection, which
is partially destructed.
Ensure that the upstream connection is closed before the destructor is called.
Fixes #4310
Signed-off-by: Greg Greenway<EMAIL_ADDRESS>Risk Level: Medium
Testing: Unit and integration tests

Still working on this, and figuring out some tests, and expected behavior.
Another possible fix in tcp_proxy is to allow the destructor to close the upstream connection, but set a flag so that onUpstreamEvent() doesn't do anything.

@zuercher I think this is ready for review.
I wasn't able to test specifically for the case that I think caused the crash, because I think it requires the downstream connection to get closed, and the upstream connection to get established, in the same loop of the event loop. I couldn't think of any way to make that happen in an integration test.

re: testing -- I think there's a path where TransportSocket::doWrite is invoked via onWriteReady that might be testable? I think this must be the actual path for the crash anyway.

Following up on my previous comment, I think this reproduces the crash in a test:
https://github.com/envoyproxy/envoy/compare/master...turbinelabs:stephan/tcp_proxy_crash_repro?expand=1

Besides getting the tests nailed down, this looks correct to me. Thanks for picking this up.

It looks like that test reproduces the failure, and ran against this PR it crashes when the test is destructing. I'm not sure how to finish the test so that it shuts down cleanly though. And I think a test like this will always be racy.
I think I'd rather not add a racy test if we can avoid it. What do you think about merging this as-is and iterating on testing more later?

Thanks for fixing this, @ggreenway!


   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers.

@kazum @Huyuwei please review this PR

Also @yongwww & @ashutoshparkhi welcome to review

@srkreddy1238 please send another push to re-trigger CI. @yongwww please https://docs.tvm.ai/contribute/code_review.html#approve-and-request-changes-explicitly


Allow updating project versions via API

To automate the release process of the Spring Data release trains, I'd like to automate updating the project version for the modules contained in a train. I think being able to submit the form containing the available versions alongside API and documentation links as JSON would be sufficient.

Fixed by #702


wait_for_everyone() did not work

System Info
all is the latest

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
I have same code on different two servers where the only difference resources are gpu : (A server) A6000 x 8 and  (B server) RTX 3090 x 8. (Note that all versions are same cause same install package scripts are used)
When I operated same code, A server works very well with no stuck but B server stucks and freezes and cannot pass it at the right time accelerator.wait_for_everyone() is encountered. (How can guarantee the codes are same? this is because I've controlled the two server by git source, therefore I can always check the difference in vs code github tracking function)
What may be the candidates of the reasons for this problems?
I used DDP.
Is it hardware problem? how to solve...
Expected behavior
I want to run the code in B server

oh it was nvidia-driver problem. how would this be problem... Well, solved!


fix: always show the create insight page button

Description
This PR extract the create insight page button from within the repositories cart to outside, this way the user will be able to see the button always and know that they can create a page. In the old way, the user would not get to see the button until they play around with the page.
Added screenshots and added the second option for discussion purposes.

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

fixes #1425
Mobile & Desktop Screenshots/Recordings
Before
When no repo added:

After adding repos:

After (always)

Second option: (not applied)

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@a0m0rajab I think the button should remain where it is, but just be always visible

I felt having action items (page visibility) after the create button is a bit anti-pattern. Forms usually have the primary button at the end. Here is an example for primer design system:
https://primer.style/design/ui-patterns/forms/overview#structure
I am still fond of having this form as one column 😄

I felt having action items (page visibility) after the create button is a bit anti-pattern. Forms usually have the primary button at the end. Here is an example for primer design system:
https://primer.style/design/ui-patterns/forms/overview#structure
I am still fond of having this form as one column 😄

The inspiration for this button was taken from Shopify, and the experience was meant to feel like a shopping cart. I don't think we can accept a major design update in this PR.
This PR is a bit of scope creep and a low priority change because we will eventually invest in redesigning the experience.
I think this is context valuable for the issue and not in the active PR. I will close this PR for now, so we can keep the discussion in sync there.


Add project files to the repo

Summary

Add the following resources

[x] text in README

[x] Code of Conduct

[x] Contributing guideline

[x] Meeting templates

[x] Meeting notes from previous calls

[ ] Resources used for meetings, like presentations, resources etc. (this can be added in a separate PR by Fede and Leontien)

[ ] Anything else?

@malvikasharan hi! I just realised I was adding edits to the wrong branch! All looks good, I just added a couple of links - I'll merge it now ;-) thanks a lot!


use latest checkout action with safe.dir fix

Update release action to use latest checkout action

The recent release in checkout actions solves the directory permission issue upstream.
Related Issues

Addresses #21

The tag for the new version wasn't pushed for this PR. And because of that, the new release hasn't been published. (release action wasn't triggered) @Dragotic do you mind taking a look to see what happened with the tag push/release? Thank you!


Video returned by YouTube isn't what was requested

WARNING: This PR does not solve the underlying problem. Please test or reference only instead of merging this
Two mornings ago, YouTube was patched to redirect all video requests playing on older Android players to restricted channels. To fix that I did a patch like this PR and for now it seems to be working fine. I don't know how to fix the root problem, but I hope my fix will help resolve this pressing issue.
https://github.com/lavalink-devs/Lavalink/issues/1030

Compiled Lavalink from source with this as a dependency, can confirm Youtube playback works again.

I don't know anything about Lavalink, and I've only used Lavaplayer myself. But if that helps, I'll try that!

Compiled Lavalink from source with this as a dependency, can confirm Youtube playback works again.

I'm sorry if this is uncouth, can you give a small tutorial or summary of the steps you took to achieve this?

I'm sorry if this is uncouth, can you give a small tutorial or summary of the steps you took to achieve this?

It looks like this will help you!
repositories {
  maven { url "https://jitpack.io" }
}
dependencies {
  implementation 'com.github.kamilake:lavaplayer:main-SNAPSHOT'
}

Compiled Lavalink from source with this as a dependency, can confirm Youtube playback works again.

I'm sorry if this is uncouth, can you give a small tutorial or summary of the steps you took to achieve this?

i have a patch file for the Lavalink repo here
https://cdn.discordapp.com/attachments/1211003213248069632/1223485274340986920/usecomm.patch?ex=661a0674&is=66079174&hm=7559ca70ac270e3e0429475847ec63b47caa9f866d1385af82c68b47d5acb730&
Apply this then run gradlew.bat build (windows) or ./gradlew build (the rest of the world) from a terminal to compile

Somebody also just made a repo with that patch applied.
https://github.com/ZeyoYT/Lavalink

there's probably a youtube rep in here just jottin' down notes...

https://github.com/lavalink-devs/lavaplayer/pull/86
Some people here used iOS (like yt-dlp) instead of WEB. It looks better this way

Did you test this at all with playlists? Because during my testing, I'm getting playlists with zero tracks and a null title

I'll be closing this regardless as I'm working on a robust implementation with retry mechanisms in place to reduce error rates. Looking to have a test build out later.

Did you test this at all with playlists? Because during my testing, I'm getting playlists with zero tracks and a null title

When requesting a playlist it returns
{
  loadType: 'PLAYLIST_LOADED',
  exception: null,
  pluginInfo: {},
  tracks: [
    //list of all tracks with all info correctly displayed
   ],
   playlist: {
    name: 'Newest Playlist',
    selectedTrack: null,
    author: null,
    thumbnail: null,
    uri: null,
    duration: 105171375
  }

Did you test this at all with playlists? Because during my testing, I'm getting playlists with zero tracks and a null title

When requesting a playlist it returns
...

Cannot reproduce.

I got this error while trying to request a spotify url/playlist from the lavalink server (using lavalink v3)

Somebody also just made a repo with that patch applied. https://github.com/ZeyoYT/Lavalink

How to use this repo on maven project?


Publishing datasets using Python script fails or is very unstable after IPT was updated to v2.7.2

We have more than 1,000 datasets registered in our IPT. To add new datasets or update the existing ones, we use a python script to automate the processes. The script worked fine for the previous version of IPT but now it's updated to 2.7.2 and the publishing through the script is very unstable. The script updates EML and resource files, uploads them to the server and makes a request to publish it.
Sometimes, the process is done successfully but IPT does not recognize or stops loading datasets (the home page shows much less datasets than there should be (e.g.  "400 resources" instead of "1,113 resources") until I go to "Administration"-"IPT Setting". Without clicking [Save] or [Cancel], just go back to the home. Then, IPT lists all the datasets. It is, though, more often that IPT fails to recognize the updated & uploaded EML file and stops loading datasets. In this case, when I go to "Administration"-"IPT Setting", it shows error logs which usually say "Failed to reconstruct resource: /var/lib/ipt/resources/zd_1931/eml-1.9.xml not found!" where zd_1931 is the dataset name that I was trying to update (the previous version was 1.8 and new version is supposed to be 1.9).
In other occasions, it seems the update is successful but the change summary is missing (of course, the script includes the change summary text). Then, I have to manually update it.
So, the outcome differs case by case and it's very hard to solve this. Please investigate this issue. If there is an "official" way (e.g. APIs) to add or update datasets through scripts, please teach me.
Updating and publishing datasets manually is too overloaded (sometimes I have to update ~100 datasets), so using the script is critical.
The excerpt of the script that sends a publish request:
# Cookie-based authentication
hostUrl = 'https://%s/' % (ipt_server)
loginFormUrl = hostUrl + 'login.do'
loginUrl = hostUrl + 'login.do'
publishUrl = hostUrl + 'manage/publish.do'
# Input parameters we are going to send
payload = {
      'email': '....',
      'password': '....',
      'csrfToken': ''
}

s = requests.Session()
response = s.post(loginFormUrl, verify = True)
if response.status_code != 200:
	print "Failed to initiate login process: %s" % (response.status_code)
	sys.exit()

payload['csrfToken'] = s.cookies.get('CSRFtoken')
response = s.post(loginUrl, data = payload)
if response.status_code != 200:	# Could get a 502 Bad Gateway error.
	print "Failed to login: %s" % (response.status_code)
	sys.exit()
else:
	print "Sucessfully logged in to %s" % (hostUrl)

# Send a publish request
resource = "(dataset name)"
params = {'r' : resource,
        'autopublish': '',
          'currPubMode' : 'AUTO_PUBLISH_OFF',
          'pubMode': '',
          'currPubFreq': '',
          'pubFreq': '',
       'publish': 'Publish',
       'summary': "change summary text..."

contents = s.post(publishUrl, data = params)
# if successful, contents = <200>. But it is often <404>

Thanks for the password Ei :/

@fujiokae Thank you for contacting us
I'll see what we can do. Have you spotted any errors in the logs or something?

@fujiokae would you mind sharing that Python script? (without exposed passwords)

Yes, the EML is generated from scratch on every run of the script, as metadata for any dataset could have been changed in our meta-database (IMIS)
Yes, all datasets are updated and re-published. We ignore versions, by default IPT always generates 1.0. Not much use for versions in an automated script

@fujiokae I have tried your script on our UAT IPT and it went fine.
What exactly you update/upload please? Could it be isues with read/write rights?
During publication one of the steps is EML publishing when the IPT crates a versioned EML file and copies data from the eml.xml in there. If that fails, the IPT alerts user on UI about the exception and reverts EML version, but does not log an error. This can be your issue, we need to have additional error logging here

Thanks for testing the code and sorry I didn't include in the sample code that the publishing request is in a loop. I wanted it to be simple so you could understand well.
Yes, it seems that sending a publishing request for just one dataset usually succeeds but when it is in a loop for many datasets, the second request gets <404> from the IPT. I had a similar issue with the previous version and thought IPT needed more time to process. So, I added time.sleep(duration) where duration was 20 seconds or so. Then, this approach fails with <404> error for IPT 2.7.2
Here is the code with loop (only # Send a publish request part)
Send a publish request
duration = 20
for resource in resourcesToUpdate:
try:
params = {'r' : resource,		# resource = dataset name
'autopublish': '',
'currPubMode' : 'AUTO_PUBLISH_OFF',
'pubMode': '',
'currPubFreq': '',
'pubFreq': '',
'publish': 'Publish',
'summary': "change summary text..."
	contents = s.post(publishUrl, data = params)
	# The first dataset tends to be successful with <200>. The ones from the second on tend to be an error with <404>

	time.sleep(duration)
except urllib2.HTTPError, e:
	print 'HTTPError = ' + str(e.code)
except urllib2.URLError, e:
	print 'URLError = ' + str(e.reason)
except Exception:
	print 'generic exception'

@fujiokae Thank you, I understand that it works in a loop. I tried it with a bunch of resources and it went just fine. I can create you an account in one of our test IPT and you can try to run your script there if you like

Thanks, @mike-podolskiy90 If you can create a test account for me, that would be great! I will also consult with the IT person in my lab to see if there are server or network settings that may cause the problem...

@fujiokae No problem! Just give me your email address please, so I can send you credentials

Mine is<EMAIL_ADDRESS>Thanks.

Thanks, @mike-podolskiy90 for taking care of. Really appreciate it.
I mostly solved the issue. My script stopped IPT server (> systemctl stop ipt) before uploading the updated eml.xml and resource.xml. Then, restarted IPT (> systemctl stop ipt) before sending publish requests. It seems IPT needs a lot of time to get ready to accept requests. I modified the script so it does not stop & restart IPT. Then, the workflow went well without any issues!
Only remaining issue is that it seems IPT does not recognize a new dataset when the script creates a new folder and uploads new eml.xml and resource.xml while IPT is running. Is that true? Does IPT search through folders at an interval (e.g. every 30 minutes or so) to find new entries? Is there a request for IPT to look for new entries on-demand?

Glad to hear you figured it out!
No, IPT does not recognize externally created resources until you restart it, because it loads the resources on startup.

Not to hijack this thread, but @fujiokae i'm interested in why you do data publication through the IPT at all when it sounds like you are generating your own EML + ready mapped dwc files.


Evaluate performance of a bitmap.NextMany() functionality

It appears that we are getting good performance in Go with a bitmap.NextMany() function, to iterate in bulk over set values.
https://github.com/RoaringBitmap/roaring/pull/150

c.c. @richardstartin

I took a look. Is the idea to write the next n elements onto a fixed length int[], instead of one call to next() for each value? I think that would be quick to prototype and should be extremely fast if you just need to get hold of the integers quickly. But there's an alternative aspect to this which is what the calling code does. If the next n integers are scattered, some code may not profit from optimisation opportunities available with iterators which can transfer control to the caller for entire runs.

@richardstartin

Is the idea to write the next n elements onto a fixed length int[], instead of one call to next() for each value?

Yes.

But there's an alternative aspect (...)

Even if it works fast, it still not going to be appropriate for all applications. It would be an option the developer can take.

@richardstartin Great.


various: eliminate the usage of any?

any? is not the opposite of empty?. Besides the case that
[false, nil].any? will return false, any?(O(n)) has much worse
performance than empty?(O(1)).
cc @UniqMartin

:+1: except for a minor nit, but that's unsurprising given I once raised the issue that any? might not be a good choice (despite its convenience and brevity) and that empty? is to be preferred.

[…], any?(O(n)) has much worse performance than empty?(O(1)).

In practice that's of course only an issue in pathological cases with long arrays containing mostly nils. Otherwise any? will short-circuit the check as soon as it encounters a non-nil value.

Otherwise any? will short-circuit the check as soon as it encounters a non-nil value.

😉 just want to add that even with short-circuit the complexity is still the same as O(n).


Support JobSet in Kueue

What type of PR is this?
/kind feature
What this PR does / why we need it:
The ability to control JobSet startup through Kueue.
Which issue(s) this PR fixes:

Fixes #738

/ok-to-test

/assign

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/test pull-kueue-test-e2e-main-1-24

/lgtm

/hold
it depends on kubernetes-sigs/jobset#181

/hold
it depends on kubernetes-sigs/jobset#181

/hold cancel
As the other PR is merged already.
Do we expect code changes here related to the work in the other PR?

Do we expect code changes here related to the work in the other PR?

Nothing major, IsActive should be reworked.

/retest

/hold cancel

/retest

/retest

/lgtm

/lgtm

/approve


Useless topic parameter in GCP Pub/Sub metadata description

The description of yaml file for GCP Pub/Sub component creation contains useless parameter topic. The metadata of component does not contains such parameter (see https://github.com/dapr/components-contrib/blob/4d3e1be105ef2c10f64d6b80f87341244a6751e0/pubsub/gcp/pubsub/metadata.go#L4) so this parameter can be removed.
https://docs.dapr.io/operations/components/setup-pubsub/supported-pubsub/setup-gcp/
Expected content: without topic parameter

This was resolved

This was resolved


how best to share existing tarsnap setup with Tarsnap GUI?

It's not clear how best to re-use an existing tarsnap configuration yet.
This should cover:

cache dir permissions chown $USER:wheel /usr/local/tarsnap-cache
sharing or re-using machine key(s), perhaps regenerated with different permissions
sharing the tarsnap.conf -- atm I see no way of referring to this in the GUI so it's not clear if  /usr/local/etc/tarsnap.conf and/or ~/.tarsnaprc are being applied. I assume they will be used as usual...

What I do now for an existing cache spec is check if it's a dir and if it is writable for the current EUID. If not the Wizard will refuse it. In the Settings page I set the LineEdit to red color to signal an error. What else could I do is a bit unclear, maybe improve some specific error cases or phrases?
This is again an unclear problem. Currently one can only set an existing key either via the Wizard's file browse dialog, which implies read access to it. The other case being in the Settings pane.
This one is handled as follows:

thanks for the clarifications, how about I update README or some other file with more info?

Sure, do a PR. README is a good choice for the time being.

@dch Do you still plan to do something in regards to this? Alternatively it could be a FAQ entry in this wiki page https://github.com/Tarsnap/tarsnap-gui/wiki/Tarsnap.
I'm not sure if you can make PR for wiki pages on Github.

Added a clarification as FAQ entry to https://github.com/Tarsnap/tarsnap-gui/wiki/Tarsnap. Closed.

@shinnok I'm wondering if this should be highlighted to the user on startup, I spent a bunch of time trying to work out why it couldn't see my existing files.

Yes, there's more that we can do here.  I'll revisit this in the next few weeks.

Would be happy to go over areas which I think of as confusing in the UI if you want?

Hi @Folcon!  Thanks for the offer, but not yet.  Currently I'm focusing on tests and adding doxygen comments so that I know how the code is organized.  I'm going to look at the UI once there's a stable foundation so that I don't accidentally break anything if and when I change something.

@gperciva, no problem. Feel free to msg me if you want some feedback :)...


ValueError: No gradients provided for any variable

I'm trying to optimize only a specific subset of my variables like so:
my_train = tf.train.RMSPropOptimizer(0.001).minimize(my_loss, var_list=my_variables)

This gives me an error message
ValueError: No gradients provided for any variable: ((None, <tensorflow.python.ops.variables.Variable object at 0x7f30317a5f10>), (None, <tensorflow.python.ops.variables.Variable object at 0x7f303146fa10>), (None, <tensorflow.python.ops.variables.Variable object at 0x7f3031522390>), (None, <tensorflow.python.ops.variables.Variable object at 0x7f30314e1dd0>), (None, <tensorflow.python.ops.variables.Variable object at 0x7f30314a5fd0>), (None, <tensorflow.python.ops.variables.Variable object at 0x7f303146ae90>))

There is no error if I leave out the var_list, but this obviously isn't what I want, and from the error message it isn't clear to me what the problem is.

Turns out that my_loss didn't depend on my_variables due to a bug in my code. I was able to find the bug with the help of tensorboard and adding lots of name scopes. I suggest improving the error message to indicate what the problem really is (objective doesn't depend on variables).


Keep generation scripts in macOS and Ubuntu images

Description
Currently generation scripts are removed from image in one of the last stages of image generation. We need them to run Pester tests on runners provisioned in real environment.
The alternative solution would be to checkout this repo each time we want to run tests. But in may cause unexpected failures when testing previously built image after tests or toolset in repo have been updated.
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,winsows2022,macos11,macos12

/azp run windows2022


High CPU usage due to log being flooded

Describe the bug
From time to time I have seen CPU spikes in my Home Assistant instance. I have tracked the usage to AMR2MQTT addon.
During these "high CPU usage interval" the log is being flooded with these messages (hundreds of messages per second):
Error reading samples: EOF
To Reproduce
It's hard to reproduce as it happens at random intervals.
Logs
Error reading samples: EOF
(I will update with more info when it happens again)
Environment (please complete the following information):

Add-on version: 2.1.2]
Supervisor version: (No idea how to get this)
Operating system: Home assistant OS  2022.12.8 (running inside a VM)

This is a sample of the log
10:19:18.871674 main.go:145: Error reading samples:  EOF
10:19:18.871729 main.go:145: Error reading samples:  EOF
10:19:18.871781 main.go:145: Error reading samples:  EOF
10:19:18.871833 main.go:145: Error reading samples:  EOF
10:19:18.871886 main.go:145: Error reading samples:  EOF
10:19:18.871938 main.go:145: Error reading samples:  EOF
10:19:18.871990 main.go:145: Error reading samples:  EOF
10:19:18.872042 main.go:145: Error reading samples:  EOF
10:19:18.872093 main.go:145: Error reading samples:  EOF
10:19:18.872144 main.go:145: Error reading samples:  EOF
10:19:18.872197 main.go:145: Error reading samples:  EOF
10:19:18.872250 main.go:145: Error reading samples:  EOF
10:19:18.872304 main.go:145: Error reading samples:  EOF
10:19:18.872356 main.go:145: Error reading samples:  EOF
10:19:18.872490 main.go:145: Error reading samples:  EOF
10:19:18.872599 main.go:145: Error reading samples:  EOF
10:19:18.872652 main.go:145: Error reading samples:  EOF
10:19:18.872706 main.go:145: Error reading samples:  EOF
10:19:18.872759 main.go:145: Error reading samples:  EOF
10:19:18.872811 main.go:145: Error reading samples:  EOF

Similar issue for me happening today and for a few weeks.
Environment (please complete the following information):
Add-on version: 2.1.2
Supervisor version: 2023.04.1
Operating system: Home Assistant OS 2023.4.6
I have a Node-Red automation that looks at the Supervisor's AMR2MQTT CPU percent.  When it is greater than 3.5% for ten minutes (which I have determined is the CPU percentage when this issue is occurring), I have Node-Red restart the Add-on and log an alert.  It will not perform this automation again for one hour (throttled).  Today, it happened 6 times and would have continued until I stopped the Add-on, unplugged my Nooelec USB radio, plugged it back in and restarted the Add-On.


chore: gateway-api v0.5.1

Type of change:

[ ] Bugfix
[ ] New feature provided
[x] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1445 (c459e74) into master (6f83da5) will increase coverage by 0.10%.
The diff coverage is 20.83%.

@@            Coverage Diff             @@
##           master    #1445      +/-   ##
==========================================
+ Coverage   41.25%   41.35%   +0.10%
==========================================
  Files          82       82
  Lines        7279     7261      -18
==========================================
  Hits         3003     3003
+ Misses       3929     3911      -18
  Partials      347      347

Impacted Files
Coverage Δ

pkg/providers/gateway/translation/gateway.go
0.00% <0.00%> (ø)

pkg/providers/gateway/translation/translator.go
0.00% <ø> (ø)

...providers/gateway/translation/gateway_httproute.go
52.50% <52.63%> (ø)

pkg/providers/apisix/translation/apisix_route.go
31.77% <0.00%> (+0.35%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fixed GSAP SvelteKit Issue

Added SSR safeguard check.
Hi this is Rodrigo from GSAP. This should solve the issue you're seeing when creating a live example with this template.
Please let me know if there's anything else you need for this.
Thanks!
Cheers,
Rodrigo.

Works like a charm!
Thanks, Rodrigo! 🙌


DASB - Discrete Audio and Speech Benchmark

Hi @poonehmousavi, thank you for this PR! Here are my comments:

I propose to move the code in this PR to the main SpeechBrain repo (unstable v0.6 branch). In the benchmark section, we should only import the pretrained models that provide the discrete representations and write the code for training the probing heads of the downstream tasks. In the main repo, we can include interfaces to these models or push all the necessary steps for clustering them (in the case of models like Hubert, WavLM, Wav2vect, etc.)
Specifically, the Hubert Discrete Interface should be placed in the main repo's unstable-v06 branch under https://github.com/speechbrain/speechbrain/tree/unstable-v0.6/speechbrain/lobes/models/huggingface_transformers.
I think the Hubert Discrete Interface has to return both the indexes of the representations and their corresponding centroids. When we use the discrete representations for our downstream tasks, it seems important to use the centroid vectors corresponding to each index, right?
The DiscreteHuBERT's docstring currently resembles that of Hubert. It's crucial to update it to reflect DiscreteHuBERT, including describing important arguments like kmeans and providing a proper example.
I would suggest putting the model on HuggingFace, making it easy to import for us.  We can mark it in the ReADME as a work in progress.
If we decide to move this to the main repo, "LibriSpeech_prepare.py" should be a symbolic link.
I suggest changing the name from "cluster" to "clustering" for clarity.
Regarding "train_splits: ["train-clean-100"]," is there a specific reason for limiting us to "train-clean-100"? Have you seen this limitation mentioned in any research papers?
The value of "n_clusters: 10" might not be sufficient. Have you observed this number being used in the literature? Some research papers use larger vocabulary sizes for discrete representations, such as 1024.


[tests-only][full-ci] copy test dependencies from the ownCloud/core

Description
Copy test infrastructure for the oC/core repository
Related Issue

Fixes https://github.com/owncloud/ocis/issues/5094

Motivation and Context
To remove the oC/core dependency for the acceptance tests run
How Has This Been Tested?

:robot:

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

I think we can remove reva related setups, though AFAIK local ocis tests won't run for Reva.

I think we can remove reva related setups, AFAIK local ocis tests won't run for Reva.

@individual-it @phil-davis do we make a separate issue for this?

I think we can remove reva related setups, AFAIK local ocis tests won't run for Reva.

@individual-it @phil-davis do we make a separate issue for this?

AFAIK we will still need to be able to run the "regular" API acceptance test suites in reva. This PR copies the acceptance test code into the ocis repo, so that we can customize it here directly in ocis without needing to get mixed-up with the existing behavior that is tested in oC10 core repo. So we will want to be able to run CI in reva that executes the code here.
At the moment, we have not copied all the used oC10 core feature files into ocis - but that can happen in a future PR. After that happens, then we will need to run CI in reva referring to all the acceptance test code and feature files that are stored in the ocis repo.
For now, let's keep reva specific logic (code, setups, whatever) here. We can remove them in future after discussion.
@micbar ^

Should these changes be made to the stable branch first and then ported master?

@kiranparajuli589 Should these changes be made to the stable-2.0 branch first and then ported master?

Order would'nt make that difference...but yes we have to backport it to the stable branch too.

I made PR #5306 to stable-2.0 branch. Let's see that pass CI before we merge anything.

Note: we will leave this here until next week. Also leave #5306
I will switch it to draft for now.
I would like to double-check that all other test changes have been ported the same into both master and stable-2.0 branches, and get any recent test code changes that have happened in core. And then merge when we are happy that everything is "in sync".

codacy isn't happy with the run.sh script, if that does not make CI in other PRs fail I think we can merge this and fir the run.sh in a separate PR Or we fix the run.sh in stable and forward port both PRs at the same time to master

strange, run.sh is excluded for the codacy check in .codacy.yml

Codacy is not a required status, so it will not stop merging.