dns: Deadlock fix and performance improvements.

The pkg/dns/dns.go implementation is currently leaking timers, and has a channel send/receive deadlock race. The included commits address both the resource waste as well as eliminate the potential for a deadlock.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[X] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @nairb774. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


Fix some VFS retention tests to run with instant execution

And comment why the remaining ones don't work, yet.
See gradle/instant-execution#165 and #11689
Fixes #11835.

cc @gradle/build-cache

@lptr PTAL!


#860 - Add Code spliting

Fixes Issue
Closes #860
Changes proposed
Previously :

Now(After adding lazy loading):

@MonalikaPatnaik as you asked why i removed the preloader, that is because it was clashing with the Suspense fallback loader for the home page, hence I added the same loader to all the pages
No worries it is still the same
On my previous pr i made a typo in the import statement.

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.

@MonalikaPatnaik
Now if you can see the deployment is working fine


Increase queue limits.

1000 is far too low for streams to deal with bursts.

Yeh, I've seen this limit being hit in some of our internal load tests. I'll merge this change for now, but I'm also working on a rearchitect and I'll revisit this case to see if there something more intelligent we can do.

Thanks! I have also reached out to the maintainer of jctools to see if mpsc is worth while https://github.com/johnou/orbit/commit/f1a30bb1baebcd0332461d2ab78a7e3e98b1146d#commitcomment-18512047


Avoiding OpenSSL

Hi there, thanks for the excellent library.
I'm trying to statically link my tool using musl and getting OpenSSL errors. This is most likely due to attohttpc. I know with reqwest it's possible to use rustls instead of OpenSSL using features. Is there a similar option with attohttpc, and would it be possible for you to expose it to users of bigtools?

Figured out how to do this on my fork. Created #20 in case it's useful upstream. Thanks again!

Fixed in #20


Installing pow on Yosemite fails silently at launchctl bootstrap

Running the installer on OS X Yosemite 10.10 fails silently at the *** Starting the Pow server... step.
Error
Running the install script via sh -x (curl get.pow.cx | sh -x) reveals that the actual command which fails is the following:
launchctl bootstrap gui/501 /Users/dimitardimitrov/Library/LaunchAgents/cx.pow.powd.plist

The command fails with an exit status of 133. When I run it manually, I get the following output:
/Users/dimitardimitrov/Library/LaunchAgents/cx.pow.powd.plist: Operation already in progress

Port forwarding does not work after that.
Workaround
Restarting the machine activated the port forwarding and now Pow works. However, running the installer again after the restart still produces the same error.
Environment

I'm upgrading Pow from 0.4.3 to 0.5.0 using the installer script.
OS is 10.10 Yosemite, upgraded from Mavericks.
I'm using zsh and rbenv.
Regular Terminal app, no tmux or iTerm.

Install debug output
Here's an excerpt from running the installer with sh -x (here is the full log as well):
➜  ~  sh -x pow.sh
+ set -e
+ POW_ROOT='/Users/dimitardimitrov/Library/Application Support/Pow'
+ NODE_BIN='/Users/dimitardimitrov/Library/Application Support/Pow/Current/bin/node'
+ POW_BIN='/Users/dimitardimitrov/Library/Application Support/Pow/Current/bin/pow'
+ LATEST_VERSION=0.5.0
(...snip...)
+ NEEDS_ROOT=0
+ '[' 0 -eq 1 ']'
+ echo '*** Starting the Pow server...'
*** Starting the Pow server...
+ '[' 10 -ge 10 ']'
+ launchctl bootstrap gui/501 /Users/dimitardimitrov/Library/LaunchAgents/cx.pow.powd.plist
➜  ~  echo $?
133
➜  ~  launchctl bootstrap gui/501 /Users/dimitardimitrov/Library/LaunchAgents/cx.pow.powd.plist
/Users/dimitardimitrov/Library/LaunchAgents/cx.pow.powd.plist: Operation already in progress
➜  ~  echo $?
133
➜  ~

Any progress/update on this? I've encountered this same issue, with the difference that  running launchctl bootstrap gui/0 /Users/user/Library/LaunchAgents/cx.pow.powd.plist outputs "Could not find domain for"
Environment
Suddenly pow stopped working. Trying to install v0.5.0 using the installer script.
OS is 10.11.3 El Capitan
I'm using bash and rvm.
Tried in both iTerm and regular terminal


Require official 1.4.0 for min CocoaPods version

Now that 1.4.0 has been out for awhile, stop allowing beta usage

cc: @wilhuff @protocol86 @schmidt-sebastian


Send failure: Broken pipe uploading 4.6MB crate

I'm not sure what's going on, but I'm able to publish smaller crates to a docker container but larger crates are failing.
Docker command to run. Note that bumping KELLNR_REGISTRY__MAX_CRATE_SIZE doesn't appear to help:
docker run --rm -it     -p 8000:8000  -e KELLNR_REGISTRY__MAX_CRATE_SIZE=20 -e KELLNR_LOG__LEVEL=trace -e KELLNR_LOG__LEVEL_WEB_SERVER=trace  ghcr.io/kellnr/kellnr:5.0.0

It looks like there's something in the web stack that's dropping larger requests on the floor, but not leaving any trace in the logs.
    Packaged 96 files, 11.2MiB (4.6MiB compressed)
   Uploading deno_core v0.250.0 (/Users/matt/Documents/github/deno/deno_core/core)
error: failed to publish to registry at http://<IP_ADDRESS>:8000

Caused by:
  [55] Failed sending data to the peer (Send failure: Broken pipe)

There are no obvious errors in the logs:
2024-01-24T17:47:26.602933Z DEBUG request: tower_http::trace::on_response: finished processing request latency=1 ms status=200 method=GET uri=/api/v1/crates/se/rd/serde_v8 version=HTTP/1.1
2024-01-24T17:47:26.604148Z DEBUG request: tower_http::trace::on_response: finished processing request latency=2 ms status=200 method=GET uri=/api/v1/crates/de/no/deno_ops version=HTTP/1.1
2024-01-24T17:47:27.891855Z DEBUG request: tower_http::trace::on_request: started processing request method=PUT uri=/api/v1/crates/new version=HTTP/1.1
2024-01-24T17:47:27.928979Z DEBUG request: tower_http::trace::on_response: finished processing request latency=37 ms status=200 method=PUT uri=/api/v1/crates/new version=HTTP/1.1

According to Wireshark, it dies approx 2784kB into the stream with this error:
HTTP/1.1 100 Continue

HTTP/1.1 200 OK
content-type: application/json
content-length: 66
date: Wed, 24 Jan 2024 17:57:21 GMT

{"errors":[{"detail":"ERROR: Failed to buffer the request body"}]}

The max_crate_size was not respected. Fixed in the next Kellnr version 5.1.2 (comes in a few days)


Bluetooth: Gatt: Automatic discover of CCC when subscribing

Added a feature that allows an application to request
automatic discovery of the CCCD handle when subscriping
to a characteristic. In order to preserver RAM, the application
is expected to provide the discovery parameters, and it's up
to the application whether or not it wants to reuse the
discovery parameters or use one for each characteristic.
Signed-off-by: Emil Gydesen<EMAIL_ADDRESS>

I would have liked to avoid putting the struct bt_gatt_subscribe_params *sub_params; in the struct bt_gatt_discover_params, but I couldn't figure out any other way to get the bt_gatt_subscribe_params in gatt_ccc_discover_cb, except if the bt_gatt_subscribe_params contained the bt_gatt_discover_params rather than just having the pointer.


Theming: Menu

Subject
Menu
Description
Create a basic theming documentation for the Menu component that showcases how you can style it.
Theming file: Menu theming file

Hi @nikolovlazar, I would like to work on this issue. Would you like the format to be similar to https://chakra-ui.com/docs/components/input/theming?

Yes you can use this as a guideline :)

I'm still working on this. I plan to submit the pull request today. Sorry for the delay.


Not getting any debug commands

Testing #20982
Version 1.10.0-insider
Commit 256d76fed7947e7289f921df10658d7a69233b65
Date 2017-02-22T07:09:05.746Z
Linux

In the VSCode workspace, open VSCode on the 'extensions/html' folder.
press Ctrl + E, type: debug with space at the end
an empty dialog shows up
click the dialog away
error message shows up:
Unable to instantiate quick open handler vs/workbench/parts/debug/browser/debugQuickOpen - QuickOpenHandler: {"errorCode":"load","moduleId":"vs/workbench/parts/debug/browser/debugQuickOpen","neededBy":["===anonymous10==="],"detail":{"errno":-2,"code":"ENOENT","syscall":"open","path":"/usr/share/code-insiders/resources/app/out/vs/workbench/parts/debug/browser/debugQuickOpen.js"}}

This has been fixed and awaiting pull request review before merge in.


hiding command line space when empty ?

Is there a way to hide the bottom command line space, and show it only when commands are issued ? It would save space.

You can set the infobar to off.


add some extras

Checklist

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

cc @JacksonMaxfield @toloudis ... as discussed in https://github.com/AllenCellModeling/aicsimageio/issues/312, conda doesn't have the option for extras.  one just has to do things like conda install aicsimageio bioformats_jar imageio imageio-ffmpeg psutil etc...  Often, packages will just include all the extras in the conda install.  This starts that, lemme know what you think

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

@conda-forge-admin, please rerender

added readlif, since it was just merged as well.  will fail for now, but will rerun in an hour or so


Fix uneven batches in distributed dataloading

Fixes #233
This PR changes/fixes the implementation of how items are assigned to workers.
Before: Chunks are first assigned to ranks, then samples from ranks assigned to workers
Now: Assign samples directly across combined world size of all workers/ranks.
This allows us to correctly apply drop_last and ensure that each rank returns the same amount of data. However, this means this PR is a breaking change.
IMPORTANT:
This changes the order in which samples are batched and returned. A consequence of this also is resuming from checkpoints made prior to this PR are not going to be restored correctly.
TODOS

[x] Resuming logic
[x] Chunk deletion logic
[x] Apply fix for no-shuffle
[x] Tests

Awesome work @awaelchli !


Broadcast::auth($request); gives error in pusher

Laravel Version
8.83.27
PHP Version
8.2.7
Database Driver & Version
MySql
Description
i want to authenticate the a channel to subscribe to that perticular channel.
below is the App\Events code
class SendMessage implements ShouldBroadcast
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

    public $message;
    public $chatId;

    /**
     * Create a new event instance.
     *
     * @return void
     */
    public function __construct(array $input)
    {
        $this->message = $input['message];
         $this->chatId = $input['chatId'];
    }

    /**
     * Get the channels the event should broadcast on.
     *
     * @return \Illuminate\Broadcasting\Channel|array
     */
    public function broadcastOn()
    {
        return new PrivateChannel('chatId_'.$this->chatId);
    }
}

below code is from routes/channels.php
Broadcast::channel('chatId_{chatId}', function () {
    return true;
});

this is the rout
Route::post('/broadcasting/auth', [Websocket::class, 'authorizePrivateChannel']);

it works fine when
class Websocket extends Controller
{
    public function authorizePrivateChannel(Request $request)
    {
        $response = $pusher->authorizePresenceChannel(
            $channelName,
            $socketId,
            $userId,
            $presence_data
        );

        return response($response);
    }
}

Steps To Reproduce
when i replace
$response = $pusher->authorizePresenceChannel(
            $channelName,
            $socketId,
            $userId,
            $presence_data
        );

        return response($response);

with
return Broadcast::auth($request);

i get the error
"error": {
        "code": "SERVER_ERROR",
        "description": "",
        "data": {
            "internal_message": ""
        },
        "stack_trace": "#0 C:\\Users\\dell\\Desktop\\chat\\karavali-karimani-backend\\vendor\\laravel\\framework\\src\\Illuminate\\Broadcasting\\BroadcastManager.php(384): Illuminate\\Broadcasting\\Broadcasters\\PusherBroadcaster->auth()\n#1 C:\\Users\\dell\\Desktop\\chat\\karavali-karimani-backend\\vendor\\laravel\\framework\\src\\Illuminate\\Support\\Facades\\Facade.php(261): Illuminate\\Broadcasting\\BroadcastManager->__call()\n#2

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so, please open up a new issue and we'll help you out.
Thanks!


[UIK-2684][docs] Updated Include/Exclude filter example

Motivation and Context
A11y

pressing "Clear all" button must invoke "Keywords cleared" live message
long text "Enter keywords..." must be aria-describedby for textarea, remove <label> semantics
textarea aria-label must be "Keywords"
combobox value must be "x keyword/s"
trigger role must be combobox
dialog name & trigger name must be both "Include keywords" ("Exclude" is incorrect)

UX

count keywords correctly, separators: new line, comma, semicolon
Clear all must clear the textarea, not apply
textarea height must be 7 lines of text
textarea must have autoFocus
fix paddings

How has this been tested?
Manually.
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 not work as expected).
[x] Nice improve.

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly or it's not required.
[ ] Unit tests are not broken.
[ ] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.
[ ] I have added new unit tests on added of fixed functionality.

@ilyabrower review the code please
@j-mnizhek review the docs please, and if you want - the example behavior/styles
I didn't go into fixing the doc page too deeply, it has a looot of translation problems, I fixed only the most critical

it has a looot of translation problems

Unfortunately. Filter pattern section is the one that still has all the pages with outsourced translations. We have a task somewhere to check and update it, so I guess we can leave them as is for now (unfrotunately).


Changed issue templated to yml based templateds and removed MD templates

What this PR does / why we need it
Changing issue templates from Markdown Based to yml based
Which issue(s) this PR fixes
Fixes:  https://github.com/vmware-tanzu/community-edition/issues/4417
Describe testing done for PR
This can not be tested against fork repo
Special notes for your reviewer

We had discussed the idea of using issue forms early on and the decision was we did NOT want to force things into a form.
We can see what others think now, but -1 from me on making this change.

After thinking about this and understanding what this would look like, the freeform MD is sufficient and there isn't a need to put a lot of enforcible structure around this. -1 for me as well

Thank you for proposing this Mohit, but I think we have consensus that this is not something we want to do. I'm going to close this PR and Issue.


operator: update CiliumNode in kvstore without lease

The first three commits are cleanups and refactoring in preparation for the 4th commit which contains the actual change:
Under normal circumstances, the agents should keep their own CiliumNode
up to date in the kvstore. In case of an agent restarting or otherwise
failing to renew the lease, the operator's sync logic might take over
and update the key with its own lease. This could lead to problems when
the respective agent comes back up and tries to renew the lease for its
own CiliumNode entry. To prevent this situation, let the operator
k8s->kvstore sync logic for CiliumNodes update the entries without
taking a lease.

/test


CI properly working

Ongoing for the first week, @KaiSforza will be working on this.
Started in #5

I'd call this done.


Correlation Key property added

Added additional property - CorrelationKey to alerts in specification/security/resource-manager/

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5404
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5404/commits/38e3cf4b483c8ef89d13ef71f7debda56ca1d33d

@orparnes is added to the review. #Closed


Feedback on the Common web application architectures article

Type of issue
Other (describe below)
Description
This issue has been moved from a ticket on Developer Community.

To whom it may concern,
I am writing to express my concern and dissatisfaction with the article titled “Common web application architectures” that is published on the Microsoft Learn website. In particular, I have an issue with the section that describes the Clean Architecture.
The article claims that the Clean Architecture is a generic term that has gone by many names over the years, such as Hexagonal Architecture, Ports-and-Adapters, Onion Architecture, etc. It then proceeds to describe a custom architecture that has nothing to do with the original Clean Architecture as defined by Robert C. Martin in his book of the same name.
This is misleading, inaccurate, and disrespectful to the author and his work. The Clean Architecture is not a generic term, but a specific architecture that has a clear definition and structure. It is based on a set of principles and rules that aim to make software systems more understandable, maintainable, testable, and independent of frameworks, databases, or external agencies. It consists of four main layers:

Enterprise Business Rules (Entities);
Application Business Rules (Use Cases);
Interface Adapters;
Frameworks and Drivers.

The architecture that the article describes introduces new terms and layers that are not part of the Clean Architecture, such as Application Core, Infrastructure, and UI. It also mixes up different architectures and patterns that are not equivalent or interchangeable with the Clean Architecture, such as Hexagonal Architecture, Ports-and-Adapters, Onion Architecture, etc.
These architectures and patterns may have some similarities or overlaps with the Clean Architecture, but they are not the same thing. They have different origins, definitions, structures, and goals. They may also have different advantages and disadvantages depending on the context and requirements of each project. To lump them together under one name and present them as one architecture is confusing and misleading for the readers.
Therefore, I urge you to change the title and content of the section that describes the Clean Architecture. You should either describe the Clean Architecture as it is defined by Robert C. Martin in his book, without making up new terms and layers, or you should not mention the Clean Architecture at all and use a different name for your custom architecture. This way, you will avoid misinforming and misguiding your readers and respect the author and his work.
Thank you for your attention and consideration.
Sincerely,
Dmytro Turskyi

Original Comments
Tracy Wang [MSFT] on 6/18/2023, 05:34 PM:
We have converted this feedback item to a suggestion. This change was done to better reflect the feedback’s nature. It will allow other developers to easily find it and engage on it.
Feedback Bot on 6/18/2023, 07:34 PM:
Thank you for taking the time to provide your suggestion. We will do some preliminary checks to make sure we can proceed further. You will hear from us in about a week on our next steps.
Ziyad Carter on 11/29/2023, 03:10 PM:
I found the Common Web Application Architectures article insightful, offering a comprehensive overview of various architectures. The detailed explanations enhanced my understanding of best practices. Additionally, the inclusion of real-world examples made it more practical. For those looking for expert guidance, https://www.nursingpaper.com/our-services/nursing-care-plan-writing-service/ offers valuable insights on nursing care plan writing, complementing the broader web application architecture discussion.
Page URL
https://learn.microsoft.com/en-us/dotnet/architecture/modern-web-apps-azure/common-web-application-architectures
Content source URL
https://github.com/dotnet/docs/blob/main/docs/architecture/modern-web-apps-azure/common-web-application-architectures.md
Document Version Independent Id
b227c667-82ae-97ef-c175-7e17051c5ab2
Article author
@ardalis
Related Issues

@Rick-Anderson
I found this issue floating around. I converted it to github feedback with all the data. But it was originally written by ardalis and I don't think he owns the content anymore. Is there someone on your team that should own this?


Dart and analyzer produces different types for typedef G<X, Y extends X> = X Function(Y)

Dart SDK version: 2.11.0-161.0.dev (dev) (Thu Sep 24 17:00:45 2020 -0700) on "windows_x64"
Here is a test example:
typedef F<X> = void Function<Y extends X>();
F<X> toF<X>(X x) => <Y extends X>() {};

typedef G<X, Y extends X> = X Function(Y);

test(G source) {
  var fsource = toF(source);

  F<G<dynamic, Never>> target = fsource;
  F<G<dynamic, dynamic>> target3 = fsource;
}

main() {}

Both analyzer and dart throw a compile error here, but for the different lines:  dart treats G<dynamic, Never> as a correct declaration and throws a compile error for G<dynamic, dynamic> whereas analyzer produces opposite result, all checked with nnbd turned on.
Both tools should behave in the similar ways.
Sample output is:

$> dartanalyzer --enable-experiment=non-nullable test.dart
Analyzing test.dart...
error - A value of type 'void Function<Y extends dynamic Function(dynamic)>()' can't be assigned to a variable of type 'void Function<Y extends dynamic Function(Never)>()'. - test.dart:9:33 - invalid_assignment
hint - The value of the local variable 'target' isn't used. - test.dart:9:24 - unused_local_variable
hint - The value of the local variable 'target3' isn't used. - test.dart:10:26 - unused_local_variable
1 error and 2 hints found.
$> dart --enable-experiment=non-nullable test.dart
test.dart:10:36: Error: A value of type 'void Function<Y extends dynamic Function(Never)>()' can't be assigned to a variable of type 'void Function<Y extends dynamic Function(dynamic)>()'.
F<G<dynamic, dynamic>> target3 = fsource;

I believe the discrepancy has been eliminated by https://dart-review.googlesource.com/c/sdk/+/164860, where the analyzer starts using the variance of type parameters during i2b. Did you try to run it with a fresh dartanalyzer? I get an error on line 10 from the analyzer (as well as the CFE) with a dartanalyzer from 8b567447dc93a08603a273d610edcfa8cefd6539.


I can't get any further and have no idea what it is! The installation process always stops when installing MagicMirror if «npm» is to be installed. An update from V. 8.5.5 to 8.10.0. All attempts to get this somehow failed, so I would like to ask for an explanation of how I can do the update.

Hello and thank you for opening an issue.
Please make sure that you have read the following lines before submitting your Issue:
I'm not sure if this is a bug
If you're not sure if it's a real bug or if it's just you, please open a topic on the forum: https://forum.magicmirror.builders/category/15/bug-hunt
I'm having troubles installing or configuring MagicMirror
Problems installing or configuring your MagicMirror? Check out: https://forum.magicmirror.builders/category/10/troubleshooting
A common problem is that your config file could be invalid. Please run in your MagicMirror² directory: npm run config:check and see if it reports an error.
I found a bug in the MagicMirror² installer
If you are facing an issue or found a bug while trying to install MagicMirror² via the installer please report it in the respective GitHub repository:
https://github.com/sdetweil/MagicMirror_scripts
I found a bug in the MagicMirror² Docker image
If you are facing an issue or found a bug while running MagicMirror² inside a Docker container please create an issue in the corresponding repository:

karsten13/magicmirror: https://gitlab.com/khassel/magicmirror
(deprecated) bastilimbach/docker-magicmirror: https://github.com/bastilimbach/docker-MagicMirror

I found a bug in MagicMirror
Please make sure to only submit reproducible issues. You can safely remove everything above the dividing line.
When submitting a new issue, please supply the following information:
Platform: Place your platform here... give us your web browser/Electron version and your hardware (Raspberry Pi 2/3/4, Windows, Mac, Linux, System V UNIX).
Node Version: Make sure it's version 14 or later (recommended is 16).
MagicMirror² Version: Please let us know which version of MagicMirror² you are running. It can be found in the package.json file.
Description: Provide a detailed description about the issue and include specific details to help us understand the problem. Adding screenshots will help describing the problem.
Steps to Reproduce: List the step by step process to reproduce the issue.
Expected Results: Describe what you expected to see.
Actual Results: Describe what you actually saw.
Configuration: What does the used config.js file look like? Don't forget to remove any sensitive information!
Additional Notes: Provide any other relevant notes not previously mentioned. This is optional.

how were you installing?
manual , npm install, or my scooted install?
after you flashed Theo's iange, what else did you do before running mm install?

Zuerst habe ich das „Automatic Installation Scripts»
Verwendedet, danach „ Manual Installation“ und zu guter Letzt, Versucht das npm, vorab zu installieren!
Ich bin ein absoluter Noub und lese mich einfach durch und Versuche.

reflash
setup ssh
and then use the install script from here
https://github.com/sdetweil/MagicMirror_scripts
does everything for MagicMirror

I got it to work on an 8GB micro SD,
but not on the 4GB micro SD!
Could it be the micro SD?
I tried different ones and none worked.

no idea. all my SD cards are 16gb or above
df -k
will tell I how much disk is used

It was probably due to the too small MicroSD 4GB.
After I installed everything on a larger (16 GB) it went without any problems.

we can close this


Fixed the issue with pool_size with sqllite

We use sqllite for unit testing out code we observed the following error
with flask_sqlalchmey 2.1
TypeError: Invalid argument(s) 'pool_size' sent to create_engine(), using
configuration SQLiteDialect_pysqlite/StaticPool/Engine.  Please check that
the keyword arguments are appropriate for this combination of components.
After investigating a bit, I've discovered that we were using wrong pooling
mechanism for sqllite in-memory dbs (StaticPool). You can get more details
http://docs.sqlalchemy.org/en/latest/core/pooling.html
Basically, for sqllite there only 2 pooling mechanisms i.e.,

SingletonThreadPool (in memory)
NullPool (file based)

So I have switched pooling mechanism to use SingletonThreadPool instead of
StaticPool to fix the issue.

Please let me know if you have questions or concerns. All our builds are broken due to 2.1 release and we were hoping if you can patch 2.1 release with this commit and release it as soon as possible.
Thanks.

Thanks for the PR. Interestingly, this contradicts other SQLAlchemy advice regarding using a memory database with multiple threads http://docs.sqlalchemy.org/en/latest/dialects/sqlite.html#using-a-memory-database-in-multiple-threads and http://docs.sqlalchemy.org/en/latest/core/pooling.html#sqlalchemy.pool.SingletonThreadPool
There was a change since 2.0 to 2.1 in this commit which introduced the default configuration of StaticPool as the poolclass for SQLite :memory: connections https://github.com/mitsuhiko/flask-sqlalchemy/commit/9b5bd9427fb3562926b3f9d9f9401ea6159a1fb2
Could you give me some more detail about which version of SQLAlchemy you're using?

In the first link you have posted, it also says that
"""
SingletonThreadPool is used by the SQLite dialect automatically when a memory-based database is used. See SQLite.
""""
The problem with StaticPool is that it's constructor does not take any pool_size argument
https://github.com/zzzeek/sqlalchemy/blob/master/lib/sqlalchemy/pool.py.
But apply_pool_defaults method in SQLAlchemy class adds the pool_size to the configuration (so pool size is present by default). Also, apply_driver_hacks method does the following
""""
if pool_size == 0:
raise RuntimeError('SQLite in memory database with an '
'empty queue not possible due to data '
'loss.')
""""
to ensure that pool_size is present for sqllite driver name and database being memory. This contradicts the StaticPool as it does not have pool_size argument in the constructor so during the create_engine call any configuration that is not poped out would be thrown as exception in this case it is pool_size with the following message
"""
TypeError: Invalid argument(s) 'pool_size' sent to create_engine(), using
configuration SQLiteDialect_pysqlite/StaticPool/Engine. Please check that
the keyword arguments are appropriate for this combination of components.
""""
BTW, I am using latest sqlalchemy.

I've tried this out and I can only replicate when SQLALCHEMY_POOL_SIZE is explicitly set or is not None, can you please confirm that's not in your config anywhere?

Test case:
from flask import Flask; from flask.ext.sqlalchemy import SQLAlchemy
app = Flask(__name__)
app.config['SQLALCHEMY_DATABASE_URI'] = 'sqlite://'
db = SQLAlchemy(app)
db.create_all()
app.config['SQLALCHEMY_POOL_SIZE'] = 5
db = SQLAlchemy(app)
db.create_all()
[...] TypeError: Invalid argument(s) 'pool_size' sent to create_engine(), using configuration SQLiteDialect_pysqlite/StaticPool/Engine.  Please check that the keyword arguments are appropriate for this combination of components.

Aha, I see it now ignore my PR sorry for the false alarm.

No problem. Glad this is working for you now, got a bit of a fright there ;)


Fix Drozd and C-20r unwielding on use-in-hand

About the PR

Makes the Drozd and C-20r rack instead of unwielding when used in hand while wielded.
Why / Balance

Racking rather than unwielding is the intended behavior for when these guns are used while wielded. A tangentially related PR accidentally changed this behavior (details below).
Technical details

No C#, just added unwieldOnUse: false to both guns.
#27907 removed BaseGunWieldable as a parent for the Drozd and C-20r, which previously provided this behavior to them. This PR re-adds the behavior directly to the guns' prototypes.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

No breaking changes.
Changelog

:cl:

fix: The Drozd and C-20r do not unwield on use again.

Thanks for the PR. When I apply the patch I don't seem to see any difference; could you explain what the change is?

Thank you for reviewing my PR!
To clarify what the change is, here is a sequence of events with and without the patch:
Without the patch:

Load in, then spawn and pick up a C-20r
Press use-in-hand (default z) -> You wield the C-20r
Press use-in-hand (default z) again -> You unwield the C-20r

With the patch:

Load in, then spawn and pick up a C-20r
Press use-in-hand (default z) -> You wield the C-20r
Press use-in-hand (default z) again -> Closed bolt

Hope this clarifies it! If either of these sequences produces a different result for you, please let me know so I can investigate further. Also note that this only affects the C-20r and Drozd, not any other guns (that part probably wasn't in question, but I just wanted to be sure).

Why is WieldableComponent not on the parent SMG to begin with?

WieldableComponent is not on the BaseSMG because not all SMGs can be wielded. In particular, the WT550 is an exclusively one-handed SMG. If we choose to put WieldableComponent on BaseSMG and define all SMGs to be wieldable, the WT550 will need to be either given a wielded sprite and accuracy modifier, or removed/deprecated. Either of these routes are doable but both seem like they warrant further discussion.

Got it, thanks for the explanation. The reason this was really confusing to me might have been related to the mispredict issue that you already fixed and merged.

Thanks for the fix!

Thanks for the fix!


Gemnasium badge is pointing to the wrong repo

I think this is related to #26 . You are pointing the badge to https://github.com/NateBaldwinDesign/aggregator
Ps.: Which redirects to Dragoman (Did you change the name of the repo?), but apparently Gemnasium see as two different things

Updated account & url for gemnasium (https://github.com/NateBaldwinDesign/Dragoman/commit/8058de276a0f0b672e5a31540c6de9022cb4d519)


require Julia v1.8

Recent Ci failures such as https://github.com/SKopecz/PositiveIntegrators.jl/actions/runs/8415189625/job/23039967577 are caused by issues with LinearSolve.jl. The problems ar fixed in newer versions of LinearSolve.jl, but these are not compatible with Julia v1.7. Thus, we bump the required Julia version of PositiveIntegrators.jl, too.
Edit: Julia v1.8 does not work either...

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.14%. Comparing base (8ab284e) to head (ddfc893).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   98.14%   98.14%
=======================================
  Files           4        4
  Lines         378      378
=======================================
  Hits          371      371
  Misses          7        7

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


ES5 clients

rspack-dev-server's dist code is transpiled to JS with modern features like spread, optional chaining... which fail on older devices (we support TV devices sporting Chrome 38+/Safari 6 equivalent browsers).
As the code is already transpiled, I couldn't find a way to use SWC in RSpack to convert those features to ES5, and have resorted to disable all hot features.
Ideally this library would emit maximally compatible code that will run on the oldest devices.

Seems to be a bug, the dist/ code shoud run in Node, but the ansiHTML.js actually run on the client. It should be placed in the client/. I will try to fix it.


Build APK does not produce native libraries for x86 ABI

Steps to Reproduce

Create a sample flutter app.
Build an apk with flutter build apk --obfuscate --split-debug-info=/some/dir/path

Expected results:
The generated APK contains all the compiled libraries for all supported ABIs.
Actual results:
The generated APK is missing some libraries for x86 ABI as shown below (it is missing specially the main flutter *.so library):

Logs

[✓] Flutter (Channel stable, 1.22.6, on Linux, locale en_US.UTF-8)
    • Flutter version 1.22.6 at /home/svprdga/opt/flutter
    • Framework revision 9b2d32b605 (9 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/svprdga/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.10+9-Ubuntu-0ubuntu1.20.04)
    • All Android licenses accepted.

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup
      for detailed instructions).

[✓] VS Code (version 1.54.3)
    • VS Code at /usr/share/code
    • Flutter extension version 3.20.0

[✓] Connected device (1 available)
    • SM N975F (mobile) • RF8N10PV53N • android-arm64 • Android 11 (API 30)

! Doctor found issues in 1 category.

x86 is not a supported architecture,  please see https://flutter.dev/docs/deployment/android#what-are-the-supported-target-architectures

Oopss... I see, @jonahwilliams do you know if there is some way to explicitely state that the app does not support x86? Something like a  in AndroidManifest, so the app is not listed in Google Play searches done with x86 devices.
Thanks.

Flutter will only build the supported ABIs by default, unless you requested an additional ABI wtih an ABI filter or something (though there are so many ways to do this on Gradle it could easily be an accident).


Support for linux-musl in JvmIndex

Currently when using Alpine, when doing cs java-home --jvm liberica:17 coursier will resolve the OS id to linux instead of linux-musl, which means there is no way to use java-home with the linux-musl version of liberica from the JvmIndex.

I see that linux-musl is included in the JVM index:
https://github.com/coursier/jvm-index/blob/5df5946359c7b6c68f5d3c9b52e7f75e8cd07819/index.json#L6620
Is this problem still happening?


Add simple request concurrency tests to S3 integration test suite.

Motivation and Context

#1304
Description

This PR adds 3 concurrency tests to our S3 integration test suite.

test_concurrency_on_multi_thread
This tests that S3 PutObject request futures can be run concurrently on a multi-threaded tokio executor. The requests are run against an in-memory service that always returns 200 OK. This runs 10k tasks in total with a concurrency limit of 100.
test_concurrency_on_single_thread
This tests that S3 PutObject request futures can be run concurrently on a single-threaded tokio executor. The requests are run against an in-memory service that always returns 200 OK. This runs 10k tasks in total with a concurrency limit of 100.
test_concurrency_put_object_against_live This test is ignored because it runs against S3.
When run, this test will upload 10k objects to S3, each with a size of 100kb. Concurrency is limited to 100 in these tests but can be pushed higher depending on the payload size.

Weirdly, when running the tests that hit the dummy server, it appears to be limited to 129 concurrent tasks before hyper starts return errors. Running against S3 doesn't have this issue although S3 will throttle requests and we don't implement adaptive retries so properly handling this throttling is not possible without significant effort.

Testing

This PR only contains tests.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This is currently failing due to including a dependency that's only used in an ignored test:
 unused dependencies:
`aws-sdk-s3 v0.0.0-local (/home/build/workspace/aws-sdk-smoketest/sdk/s3)`
└─── dependencies
     └─── "hdrhistogram"
Note: These dependencies might be used by other targets.
      To find dependencies that are not used by any target, enable `--all-targets`.
Note: They might be false-positive.
      For example, `cargo-udeps` cannot detect usage of crates that are only used in doc-tests.
      To ignore some dependencies, write `package.metadata.cargo-udeps.ignore` in Cargo.toml.

I'd say that we should allowlist this case or maybe run udeps with the --all-targets flag as suggested.

This is currently failing due to including a dependency that's only used in an ignored test:
 unused dependencies:
`aws-sdk-s3 v0.0.0-local (/home/build/workspace/aws-sdk-smoketest/sdk/s3)`
└─── dependencies
     └─── "hdrhistogram"
Note: These dependencies might be used by other targets.
      To find dependencies that are not used by any target, enable `--all-targets`.
Note: They might be false-positive.
      For example, `cargo-udeps` cannot detect usage of crates that are only used in doc-tests.
      To ignore some dependencies, write `package.metadata.cargo-udeps.ignore` in Cargo.toml.

I'd say that we should allowlist this case or maybe run udeps with the --all-targets flag as suggested.

This is currently failing due to including a dependency that's only used in an ignored test:
 unused dependencies:
`aws-sdk-s3 v0.0.0-local (/home/build/workspace/aws-sdk-smoketest/sdk/s3)`
└─── dependencies
     └─── "hdrhistogram"
Note: These dependencies might be used by other targets.
      To find dependencies that are not used by any target, enable `--all-targets`.
Note: They might be false-positive.
      For example, `cargo-udeps` cannot detect usage of crates that are only used in doc-tests.
      To ignore some dependencies, write `package.metadata.cargo-udeps.ignore` in Cargo.toml.

I'd say that we should allowlist this case or maybe run udeps with the --all-targets flag as suggested.

Nevermind, I misinterpreted this. I believe I need to set the dependency as a dev dependency in Codegen


Be explicit about waitUntil return value

Attempt to address dotnet/core-eng#1808

@alexperovich,
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

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please


Fix up arrow symbol in User Guide

github.io markdown does not accurately reflect the up arrow symbol with the values given. Lets update the values such that github.io will accurately reflect these symbols

The changes have been tested on github.io and works, this PR is ready to merge


nvc: 1.6.2 -> 1.7.0

Description of changes
Supersedes #185994.
https://github.com/nickg/nvc/releases/tag/r1.7.0
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 185995 run on x86_64-darwin 1

1 package built:

nvc

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1092


grep -P not available

workaround change grep -Po to grep -o

So, I did this a while ago, but I made the following change as a workaround. I'm not 100% sure this is exactly the same, but it has worked well enough for me:
keys)
    #declare | grep -Po "(?<=${FUNCNAME}_DATA_)\w+((?=\=))"
    declare | grep -E "${FUNCNAME}_DATA_[[:alnum:]_]+=" | sed -E "s/${FUNCNAME}_DATA_([[:alnum:]_]+)=.*/\1/g"
;;

declare | grep -o "${FUNCNAME}_DATA_[^=]*" | sed "s/^${FUNCNAME}_DATA_//"
Works for me tested on Ubuntu 22.04:
./test_shell_map.sh  All tests were successful


ESMValTool conda installation fails because of some udunits2 make error

When applying the installation instructions , the final warning is

Warning message:
In library(package_name, character.only = TRUE, logical.return = TRUE) :
there is no package called ‘udunits2’
2021-04-16 12:16:23 PM INFO : Failed to install packages: udunits2

Looking back in the install output, one can read :

udunits2:                  <IP_ADDRESS>-h975c496_1         conda-forge

...
udunits2-2.2.2 100% || Time: 0:00:00  33.24 MB/s
...
2021-04-16 10:33:47 AM INFO : Installing packages from CRAN: abind, akima, climdex.pcic, ClimProjDiags, docopt, dotCall64, functional, ggplot2, gridExtra, lintr, logging, mapproj, maps, multiApply, ncdf4, ncdf4.helpers, PCICt, plyr, RColorBrewer, s2dverification, snow, SPEI, styler, udunits2
...
trying URL 'https://cloud.r-project.org/src/contrib/udunits2_0.13.tar.gz'
...
begin installing package ‘udunits2’
...
make: *** [Makefile:180: udunits2.ts] Error 1
...
installing source package ‘udunits2’ ...
package ‘udunits2’ successfully unpacked and MD5 sums checked
...
checking udunits2.h usability... yes
checking udunits2.h presence... yes
checking for udunits2.h... yes
checking for ut_read_xml in -ludunits2... no
-----Error: libudunits2.a not found-----
If the udunits2 library is installed in a non-standard location,
use --configure-args='--with-udunits2-lib=/usr/local/lib' for example,
or --configure-args='--with-udunits2-include=/usr/include/udunits2'
You can alternatively use the UDUNITS2_INCLUDE and UDUNITS2_LIB
If udunits2 is not installed, please install it.
ERROR: configuration failed for package ‘udunits2’
removing ‘/data/ssenesi/conda/esmvaltool1/lib/R/library/udunits2’

I did a separate install of udunits2 in the same conda environment, and did export the corresponding UDUNITS2_INCLUDE and UDUNITS2_LIB before launching a further 'conda install' of esmvaltool in the environment. This failed.
I made a search in the email list archive for 'uduints2' : no match. I made a search in the GitHub issues : three matches, which are for other udunits issues. Is there another place to look for ? (I am a newbie to EsmValTool.)
I see no way to further investigate the error as tmp build directories are cleaned on error
My only departures from the install procedure are :

I use a target conda environment install dir (with option -p, so discarding option --name)
I use conda 4.3.34, which is not inline with recommendation (> 4.9); but as far as I understand here, it is rather a make issue in building R than a conda issue.

My machine :

Linux pavillion-stephane 2.6.32-754.35.1.el6.x86_64 #1 SMP Wed Oct 7 03:47:54 CDT 2020 x86_64 x86_64 x86_64 GNU/Linux

Thanks for your help

Using conda  4.10.1 solves the issue
Thanks for your help


dbplyr 2.5.0 forward compatibility

dbplyr continues to get stricter about what is interpolated into SQL

We're planning to release dbplyr to CRAN on March 15.


Babel plugin crashes with type object spread

This is a:

[x] Bug Report
[ ] Feature Request
[ ] Question
[ ] Other

Which concerns:

[ ] flow-runtime
[x] babel-plugin-flow-runtime
[ ] flow-runtime-validators
[ ] flow-runtime-mobx
[ ] flow-config-parser
[ ] The documentation website

What is the current behaviour?

The babel plugin crashes if type spreading is used:
type A = { a: any };
type B = { ...A, b: any };

TypeError: /home/edo/test.js: Cannot read property 'name' of undefined

  at node_modules/babel-plugin-flow-runtime/lib/convert.js:664:40
  at Array.reduce (native)
  at converters.ObjectTypeAnnotation (node_modules/babel-plugin-flow-runtime/lib/convert.js:642:49)
  at convert (node_modules/babel-plugin-flow-runtime/lib/convert.js:49:16)
  at converters.TypeAlias (node_modules/babel-plugin-flow-runtime/lib/convert.js:401:14)
  at convert (node_modules/babel-plugin-flow-runtime/lib/convert.js:49:16)
  at TypeAlias (node_modules/babel-plugin-flow-runtime/lib/transformVisitors.js:165:47)
  at NodePath._call (node_modules/babel-traverse/lib/path/context.js:76:18)
  at NodePath.call (node_modules/babel-traverse/lib/path/context.js:48:17)
  at NodePath.visit (node_modules/babel-traverse/lib/path/context.js:105:12)

What is the expected behaviour?

It should not crash. I guess it should support type spreading, as that is now supported by Flow.
Which package versions are you using?

├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>node: v7.9.0

fixed in 0.11.1


Azure Pipelines agent Node V10 support design doc

The previous PR was targeting my own forks master branch, instead of the Microsoft repo. Oops!


<IP_ADDRESS> Ensure mounting of cramfs filesystems is disabled

Profile Applicability:
 Level 1 - Server
 Level 1 - Workstation
Description:
The cramfs filesystem type is a compressed read-only Linux filesystem embedded in
small footprint systems. A cramfs image can be used without having to first decompress
the image.
Rationale:
Removing support for unneeded filesystem types reduces the local attack surface of the
system. If this filesystem type is not needed, disable it.
Audit:
Run the following script to verify cramfs is disabled:
#!/usr/bin/env bash
{
l_output="" l_output2=""
l_mname="cramfs" # set module name
# Check how module will be loaded
l_loadable="$(modprobe -n -v "$l_mname")"
if grep -Pq -- '^\h*install \/bin\/(true|false)' <<< "$l_loadable"; then
l_output="$l_output\n - module: \"$l_mname\" is not loadable:
\"$l_loadable\""
else
l_output2="$l_output2\n - module: \"$l_mname\" is loadable:
\"$l_loadable\""
fi
# Check is the module currently loaded
if ! lsmod | grep "$l_mname" > /dev/null 2>&1; then
l_output="$l_output\n - module: \"$l_mname\" is not loaded"
else
l_output2="$l_output2\n - module: \"$l_mname\" is loaded"
fi
# Check if the module is deny listed
if grep -Pq -- "^\h*blacklist\h+$l_mname\b" /etc/modprobe.d/*; then
l_output="$l_output\n - module: \"$l_mname\" is deny listed in:
\"$(grep -Pl -- "^\h*blacklist\h+$l_mname\b" /etc/modprobe.d/*)\""
else
l_output2="$l_output2\n - module: \"$l_mname\" is not deny listed"
fi
# Report results. If no failures output in l_output2, we pass
if [ -z "$l_output2" ]; then
echo -e "\n- Audit Result:\n ** PASS **\n$l_output\n"
else
echo -e "\n- Audit Result:\n ** FAIL **\n - Reason(s) for audit
failure:\n$l_output2\n"
[ -n "$l_output" ] && echo -e "\n- Correctly set:\n$l_output\n"
fi
}

This audit must be done on the actual machine, virtualization or tools like Docker will fail. This is because modprobe and lsmod may not be loaded. You can't load kernel modules in a Docker container. You need to load them on the host.


App route goes to "/" after refresh

🐛 Bug Report
After page refresh app route goes to "/".
Also it slightly harms developer experience (impossible to use hot reload feature).
To Reproduce

Go to any route except "/", for example "/patients/new".
Try to refresh a page

Expected behavior

Route should stay same after page refreshing.

Your Environment

node version: 12.6.1
os:  Linux

P.S. I want to help with solving this issue. I've already fixed it and I could make PR if you don't mind.

Sure, you can open up the PR fix. Thanks!


Adding another touch gesture to slider-view does not work for Android

We have a use case where we need to have a tap event on the entire slide.
I've added this code to the demo and while it works with iOS, it does not work on Android:
slider-view.xml
<StackLayout loaded="onLoaded" horizontalAlignment="center" padding="12" backgroundColor="yellow" tap="onTap" width="100%" height="100%">
    <Label text="{{ first }}" css="label"/>
    <Label text="{{ last }}"/>
    <Button text="Remove me" tap="buttonTap"/>
</StackLayout>

slider-view.ts
export function onTap(args: EventData) {
    console.log("Hello World!" + Math.random());
}

Similar issue to https://github.com/TheOriginalJosh/nativescript-slides/issues/46 but in this case it still allows the slide to pan but the touch gesture is not registered.

I think i found the issue.

Releasing the fix.

Awesome thanks Alex!


User profile page should show the user their username

I like this. I just tried it out and found a bug: if the user hasn't made any annotations, it crashes trying to find the last annotation.

@joefutrelle Bug Fixed - Any other items we could add here?


Fix: 158 changed label

Description
Changed the label as described in the issue #158
Related Issue

issue goes here

Fixes #158

[x] Code

Code/Quality Assurance Only

Bug fix (non-breaking change which fixes an issue)
This change requires a documentation update (software upgrade on readme file)

How Has This Been Tested?

Additional Info (OPTIONAL)

Checklist

[x] My code follows the code style of this project.
[x] My UI is responsive
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

@soham4abc make PR of these changes on develop branch so we can test and merge in master later after mannual testing and make the change as * What you want to be called ?

@soham4abc make PR of these changes on develop branch so we can test and merge in master later after mannual testing and make the change as * What you want to be called ?

Done as per request Please do have a look!


Prisma SDK's getDMMF function fails to parse and return instances of @@index

Problem
Prisma SDK's getDMMF function fails to parse and return instances of @@index defined on the model. The same function does return instances of @@unique, so missing @@index seems like missing functionality.
Suggested solution
Extend the getDMMF parser to include @@index defined as part of a Prisma schema.
Additional context
Came across this while writing a generator / plugin that can generate supplemental models and add fields to existing models defined in a Prisma schema file.
Here's a quick test

const datamodel = `
  datasource db {
    provider = "postgresql"
    url      = env("DATABASE_URL")
  }

  model User {
    id              Int       @id @default(autoincrement())
    email           String    @unique
    firstName       String
    lastName        String
    fooVersions     UserFooVersion[]

    @@index([email], name: "idx_users_email")
    @@unique([firstName, lastName])
  }
`;

const result = await getDMMF({ datamodel });
console.log('result should include index info', result)

If this is the case, reasoning is probably that indexes usually do not influence the Client functionality. But it sounds reasonable to add the information anyway - so this is a reasonable feature request.

Closing this as a duplicate of https://github.com/prisma/prisma/issues/3998 for now.


support info window

With issues:

Use the search tool before opening a new issue.
Please provide source code and commit sha if you found a bug.
Review existing issues and provide feedback or react to them.

Description

How to reproduce
Steps to reproduce
Expectations
Actual result
Environment

idea version:
operating system:

show  metrics


X509 tool suggested in this article is marked as archived

If you're using a DevKit device, this command-line tool generates a CA certificate that you can add to your IoT Central application to verify the certificates.

this command line tools links to https://github.com/azure/iot-central-firmware/tree/master/tools/dice that is archived.
Is this tool still the recommended option?

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

ID: 703425a8-a444-1c4e-7e2b-152f6856bbf4
Version Independent ID: 5df42622-30a0-cb07-f31d-51fc6efd6199
Content: Device connectivity in Azure IoT Central
Content Source: articles/iot-central/core/concepts-get-connected.md
Service: iot-central
GitHub Login: @dominicbetts
Microsoft Alias: dobett

Hello @rido-min Thanks for the feedback.
I have assigned the issue to the content author for further review.

I'll remove this link - you can use either of the other two tools mentioned to create the CA certs and chains.
#please-close


Theme ignored

Describe the bug
The current documentation for using themes is incorrect, there are certain values that Spotify-Player is looking for, including capitalization, which it will complain about.
However, even after make the necessary changes so the theme is accepted, it ignores the theme values and continues to display with the default colors.
To Reproduce
create a theme, adjust colors, use RGB or Hex codes, run spotify-player -t [theme name], no display / color changes are applied
Expected behavior
theme should change
Log and backtrace
Reference log and backtrace, which are usually located in $HOME/.cache/spotify-player/, of a run reproducing the bug.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

OS: NixOS
Application version: 23.05
Application features: if applicable, provide cargo features used to build spotify_player
player in distro apps

Additional context
Add any other context about the problem here.

create a theme, adjust colors, use RGB or Hex codes, run spotify-player -t [theme name], no display / color changes are applied

Theme selection still works fine for me. Can you provide the corresponding theme configs? Without that, it's impossible to know the problem.

The current documentation for using themes is incorrect, there are certain values that Spotify-Player is looking for, including capitalization, which it will complain about.

Can you elaborate on which part is incorrect or outdated?

Hello Aome510, certainly.
Here is an excerpt from the documentation in the configuration setting:
bright_blue bright_cyan bright_green bright_magenta bright_red bright_white bright_yellow
here is what it requests when I run spotify-player with the theme. as you see below it wants camel case
 BrightBlack = "#555555" BrightRed = "#ff5555" BrightGreen = "#50fa7b" BrightYellow = "#f1fa8c" BrightBlue = "#bd93f9" BrightMagenta = "#ff79c6" BrightCyan = "#8be9fd" BrightWhite = "#ffffff"
here is an config where I took the existing theme and changed the values of Red and Magenta
`[[themes]]
name = "default"
[themes.palette]
background = "#1e1f29"
foreground = "#f8f8f2"
Black = "#000000"
Red = "#d21a75"
Green = "#50fa7b"
Yellow = "#f1fa8c"
Blue = "#bd93f9"
Magenta = "#972050"
Cyan = "#8be9fd"
White = "#bbbbbb"
BrightBlack = "#555555"
BrightRed = "#ff5555"
BrightGreen = "#50fa7b"
BrightYellow = "#f1fa8c"
BrightBlue = "#bd93f9"
BrightMagenta = "#ff79c6"
BrightCyan = "#8be9fd"
BrightWhite = "#ffffff"
[themes.component_style]
block_title = {fg="Red"}
border = {}
playback_track = { fg="Red", modifiers = ["Bold"] }
playback_artists = { fg="Red", modifiers = ["Bold"] }
playback_album = { fg="Red" }
playback_metadata = { fg = "BrightBlack" }
playback_progress_bar = { bg="Magenta", fg="Red" }
current_playing = { fg="Red", modifiers = ["Bold"] }
page_desc = { fg = "Cyan", modifiers = ["Bold"] }
table_header = { fg = "Blue" }
selection = { modifiers = ["Bold", "Reversed"] }`
after loading the theme name 'default'  the spotify-player shows no color changes.
any help you can provide is greatly appreciated!

Here is an excerpt from the documentation in the configuration setting:
bright_blue bright_cyan bright_green bright_magenta bright_red bright_white bright_yellow
here is what it requests when I run spotify-player with the theme. as you see below it wants camel case
 BrightBlack = "#555555" BrightRed = "#ff5555" BrightGreen = "#50fa7b" BrightYellow = "#f1fa8c" BrightBlue = "#bd93f9" BrightMagenta = "#ff79c6" BrightCyan = "#8be9fd" BrightWhite = "#ffffff"

Ah thanks for pointing this out. Will update the document accordingly.

here is an config where I took the existing theme and changed the values of Red and Magenta

It's not possible to override the default configuration. You can name it something else and set the theme config variable in app.toml. Sorry for the confusion.

Thank you! Ill give it a go. I should have known the "default" name was probably in use. Not sure what I was thinking.
Thanks again!

Works!


[csrng/dv] Add DV for the force fips bit functionality

At some point we should probably add some coverage for the new registers. I can do so once I'm back.
Rarely there are still some failures I wasn't able to investigate yet. When I ran a 10% regression I found 1 failure due to the added DV in the csrng_cmds test.
This is the DV for #23324

Update: https://github.com/lowRISC/opentitan/pull/23324 got merged, I've rebased this PR now.


CI/CD

This work is great. I'm loving this!. Can we invoke this UI tests to CI/CD scripts?

Thank you for your interests and suggestion. And sorry for the super late response. GAN Lab is more like a prototype developed from a research project, so we haven't thought much about CI/CD, but it sounds like a good idea.
Cheers,
Minsuk


Generalized display function again

In the past the hv.ipython.display function simply extended the IPython version to add support for displaying HoloViews objects. When refactoring the MIME types that generality was lost and non-holoviews objects were simply rendered as their repr, which disrupted various usages of the function inside the extension, e.g. to adjust the width or display various warnings.

Makes sense. Happy to merge when the tests pass.


Fix publishing SDKs to include required attribute for Pulumi.Automation

Currently when publishing the Pulumi.Automation SDK there is an expected property PulumiSdkVersion which should be set when building the project
This property is set during buildSdk and testPulumiAutomationSdk but not the publishing targets which provide the resulting nuget package.
This means when a consumer references the Automation Api nuget package they have a version that does not have the required property set causing LocalPulumiCommand.Install to fail.
Refactored the F# code to ensure the version is set during publish as well
Fixes #392

Hey @JasonWhall if you rebase this on latest main I think we should be able to get the tests passing and get this merged in.

@Frassle - Thanks! Re-based


Resuming download does not work

Examples of dataset where it does not work:

https://github.com/metmuseum/openaccess/blob/master/MetObjects.csv

See if requests can handle it alone.

Done


Not able to Copy from WebView html

Copy is disable or not working, not even right click contextmenu working in the webview. Tried to figure out by example but not found Copy Enable Option.

Copy feature is not implemented currently.

Previously, copying was not possible because keyboard events were not passed to CEF, but now it should be supported. @addieindustry


Fallback pagination

I don't think it's a good idea actually. This is error-prone and may lead to confusing debugging sessions when you changed name of param and yet rom defaults it for you and you wonder why things don't work.
IMO every more complex query that relies on user input where you can use default values should be handled by some query object. Typically I use virtus for that.

Then you think everybody should use everywhere they use pagination smt like
PaginationValidation.call(PaginationParams.new(params))

Yeah as soon as the relation needs parameterized input I go for a query object. In fact I'm planning to build a rom add-on with such functionality.

The only disadvantage is to manually write and add it every time, but maybe its better solution, ok I will close those PR

This would make rom-sql responsible for handling user input in web-apps, it's a similar mistake as input sanitization happening on an ORM-level.


Fix specs for ruby head.

WIP to resolve #3123 I can't currently install head locally and it works fine on preview2

Fixes #3125 #3123


Replace token query with header attribute

See : https://developer.hashicorp.com/consul/api-docs/api-structure
The "?token=" query parameter is deprecated and will be removed in Consul 1.17.

rebase only, comments haven't been processed


Call trading options on init for digit stats

Coverage decreased (-0.006%) to 28.93% when pulling 4091a28b30cf3fa9434d37a3e7049bfe1033ae40 on qingweibinary:qingwei/fix_digit_stats into 797a6816e741e78d13c0c65f5a6d946f3ebca4bd on binary-com:master.


a piece of "point cloud"?

I just get a piece of "point cloud" after running process_exr.py,what happened?

I also use the code [process_exr.py] and [render_depth.py] to produce pcd file.

It may be used to project part of the point cloud to be used as a training set


bumped grpc and python version to build on macos

Build was failing on macos ventura and python 3.11 docker image.
This PR updates grpc to version 1.51.1 and the docker image to use python 3.11. Confirmed it builds on macos ventura and in docker.

lgtm!
fixes #5


Include full generated documentation in repository

Including the doxygen generated documentation in the repository prevents developers having to generate this themselves. But it also allows viewing the full documentation online using the link https://htmlpreview.github.io/?https://raw.githubusercontent.com/stcorp/coda/master/doc/html/index.html

Implemented in<PHONE_NUMBER>bf736c95c47d086184673c8782f5e2


fix(ci): add Cloudfront Distribution ID

What type of PR is this?
/kind bug
/area CI

What this PR does / why we need it:
Add the CF Distribution ID to the workflows so we'll be able to invalidate cache entries if needed.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 63012d336f832e5d3c092f1786ede8210f9b2d8b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Refs #2501


fizzbuzz written in golfscript

I didn't invent the solution to this, but thought it might be a nice addition to the repo.
To run: ruby golfscript.rb fizzbuzz.gs

This is nice and all but we do search self-made solutions :/
Thank you for being honest tho! Some people just blatantly copy pasted stuff from StackOverflow,RossettaStone and co and hoped they'd get away with it...


Fix typo: updated README.md

Hi @fijijavis
Thanks for the great work!
While I was following the steps from README.md, I have found a small typo.
I think this my help who are relying on the docs rigorously 😅

Thanks for catching this!


Fix retain cycle in EasyTipView

Task/Issue URL: https://github.com/duckduckgo/iOS/issues/621
CC: @bwaresiak
Description:
This clears the observer token at the end of the one-shot block. Against all
reason, the token appears to be holding a strong reference back to the block
which creates a memory leak.
Steps to test this PR:

Perform a clean install of the app
Navigate through three URLs and click away the three tip views
Start the memory-graph debugger
With the fix applied no instances of TouchView should occur

Internal references:
Software Engineering Expectations
Technical Design Template

@bwaresiak: I feel a little cheap to send the PR since you figured out the fix but maybe you find it helpful. 🙂

@Johennes thanks :)
Internal reference: https://app.asana.com/0/856498667320406/1172088068983666

Merged, thank you @Johennes :)


⚠ Bofrost.de has degraded performance

In 1ce44ae, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1036 ms

Resolved: Bofrost.de performance has improved in 1535260 after 7 minutes.


Add subteam_created And subteam_updated Events

@mitchellwrosen any thoughts here?

Sorry, didn't see this :)


fix sm2 encryption/decryption implementation bug.

fix #23209

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Test exempted as I assume it would be very hard if not impossible to create a test case.

Test exempted as I assume it would be very hard if not impossible to create a test case.

yes, it's very hard to create a test.

Is this a orignal bug of master branch?
(copyed from build logs of  CIFuzz)
2024-01-08T04:12:34.0379548Z /usr/bin/ld: /usr/bin/ld: DWARF error: invalid or unhandled FORM value: 0x25
2024-01-08T04:12:34.0381035Z ./libcrypto.a(libdefault-lib-cipher_sm4_ccm.o): in function `sm4128ccm_dupctx':
2024-01-08T04:12:34.0382368Z cipher_sm4_ccm.c:(.text.sm4128ccm_dupctx[sm4128ccm_dupctx]+0x23): undefined reference to `sm4_ccm_dupctx'
2024-01-08T04:12:34.0383908Z /usr/bin/ld: cipher_sm4_ccm.c:(.text.sm4128ccm_dupctx[sm4128ccm_dupctx]+0x32): undefined reference to `sm4_ccm_dupctx'
2024-01-08T04:12:34.0385279Z /usr/bin/ld: /usr/bin/ld: DWARF error: invalid or unhandled FORM value: 0x25
2024-01-08T04:12:34.0386538Z ./libcrypto.a(libdefault-lib-cipher_sm4_gcm.o): in function `sm4128gcm_dupctx':
2024-01-08T04:12:34.0388161Z cipher_sm4_gcm.c:(.text.sm4128gcm_dupctx[sm4128gcm_dupctx]+0x23): undefined reference to `sm4_gcm_dupctx'
2024-01-08T04:12:34.0389699Z /usr/bin/ld: cipher_sm4_gcm.c:(.text.sm4128gcm_dupctx[sm4128gcm_dupctx]+0x32): undefined reference to `sm4_gcm_dupctx'
2024-01-08T04:12:34.2165433Z clang-15: error: linker command failed with exit code 1 (use -v to see invocation)
2024-01-08T04:12:34.2176892Z make[1]: *** [Makefile:13589: fuzz/asn1parse] Error 1

@t8m

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 123 days ago

This pull request is ready to merge

Merged to all the active branches. Thank you for your contribution.


Variable IP address in configuration?

My inverter regularly changes IP address (and MAC address, meaning I can't assign a static IP), so I have to manually update 'Inverter_host' in the config frequently so that the addon can find it again.
I can find the correct IP by pinging and can record this as an entity in HASS, but not sure if I can use this value in the addon configuration?
In short, is there a way that 'Inverter_host' in the config/yaml can point to a HASS entity with the current IP, rather than using just a manually-entered IP address?

Config your router to assign inverter fix IP. You can also try to use inverter's name instead of inverter's IP.


Button - default background color not working (beta9)

After update to b9, all buttons no longer have their background color working on the default settings.
When the entity state is on, the button does not change color.  Revert to b8 and this works again as expected.
Using the new styles functionality for the button in b9 allows me to change the background, however, it's not responding to state ternary operator per the example.

In this example, using styles, the "Front Door Snapshots" button should be 'blue' with entity on.
"Front Door Detect Motion" is showing a default button behavior.  Entity is on, but color isn't changing.

This is indeed due to my code rewrite and I forgot to fix this 😅 I will fix this as soon as I can!

This is fixed in beta 10 :
https://github.com/Clooos/Bubble-Card/releases/tag/v0.0.1-beta.10

Thanks so much for the quick fix!  Works great.
Thanks for your hard work on this project - I'm really enjoying the possibilities it opens up!

I'm seeing this issue on the 2.0.0-beta 1

What do you mean precisely? The button color is correctly changing on my side when the state is on.

Sorry, I should have been more specific and provide useful info. It's not quite the same issue.
The issue is I can't change the color of the "on" state from the default orange. Using this style code on a button:
.switch-button {
  background-color: ${state === 'on' ? 'green':'red'} !important;
}

Grid card with embedded bubble cards:

State is off (first button in list correctly shows specified color):

State is on (first button in list is incorrectly orange instead of green):

I'm a new user so could be making a simple mistake. Thanks for your work on this addition, I'm having a blast with it otherwise!


[BUG] Dockerfile builds containerd from source unnecessarily

Describe the bug
docker build . and make integration both download and compile the containerd source as part of creating the soci image, which significantly increases testing time.
Steps To Reproduce
make integration
Expected behavior
Download containerd from a release, like we already do for nerdctl

#390 contains my local changes which take a brute force approach to speeding up local integration testing. Parts of that may be informative in the implementation of a real solution to this issue.


Clippy fixes and small improvements

In order to better understand nano and rs-nano in particular, I cleaned up some code and removed technical debt.
I mostly fixed clippy warnings that are split into two commits:
410975a for things like redundant references and e2f750c for some fixes that are a little bit more complex.
Furthermore there are many let _ = statements (e.g. ledger_stats.rs). I left them untouched for now - should those be removed as well?
Let me know, if there are any changes missing or unwanted! Maybe we can set the right clippy rules then.

A cargo_clippy job was also added to the Static Analyzers GitHub workflow. This requires further verification, because I'm not 100% sure what I am doing

Merged with commit 09f76e88de36681fdbc98a6fff42963b2fad47b2.
I undid the changes to fan.rs because removing the Box makes the data sequentially aligned in memory and the purpose of Fan is to spread the data over the heap - so removing the box would make Fan less secure. Everything else was ok, so great job and thank you!


Fix out of bound array access in ssl_session_reuse plugin (#6235)

This is an update for PR #6218 .

[approve ci autest]

Cherry-picked to v9.0.x branch.


Android:Wallet fixed network failure fiat balance not displayed

What does this implement/fix? Explain your changes.

Fixed network failure fiat balance not displayed

Does this close any currently open issues?
If it fixes a bug or resolves a feature request, be sure to link to that issue.
None
Pull request type

Put an x in the boxes that apply

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Where has this been tested?
…
Any other comments?
…

@Mergifyio refresh

@Mergifyio refresh


Update italian language - Small review

@Klocman
Small Italian language review
More deep language check
Fix some typos
Please check and merge.
What's the matter to leave "Unescape" untranslated?

Unescape is a programming term, I think it's much clearer if left untranslated since you can easily google it for answers if you're unfamiliar with the concept.

@Klocman
Could you pelase merge PR? Thanks.


Add tests for cached results

Caching results was added in #97 by @AkashD-Developer, it would be nice if CLI tests were added for it.
The details of implementation is up to you, here's some ideas:

Checking for the cache json
Running same command again and time differences

Hey @hedythedev, I have linked the pull request for this issue. Please have a look


Serial crystallography binning

Add a cli for ssx fixed target collections

As things stand, the current code will correctly bin the first image but return empty ones after that. The issue seems to come from the use of valid_events with arrays instead of a single value for start ans stop.
Going to try and find a solution.

Having a very quick look at the output, rather than the change set. @noemifrisina you were concerned that the same frame was simply repeated 400 times: this is not teh case

Derived from:
import numpy
import hdf5plugin
import h5py
f = h5py.File(filename, "r")
d = f["/entry/data/data"]
for j in range(400):
  print(j, numpy.sum(d[j,:,:]))

Looking at the very constant spot count per image output I am going to guess there are some unmasked bad pixels here.

Thanks for checking.
About the almost constant spot count: unmasked pixels are entirely possible. But as this is a corner block, it might also be that there was nothing in most windows.
Btw, right now there's no mask in that dataset (as it's from last year, the file needs to be copied in the same directory as the binned images for it to work - newer collections won't have this issue).

OK, lack of mask would probably do it, thank you


ExtraSounds crashes with Twigs on Creative Inventory

Minecraft: 1.19
Mod: ExtraSounds 2.3.1
Details
Only mods installed are ExtraSounds alongside Twigs and its dependencies.
As the title suggests, when you open the creative inventory tab of Twigs with these two mods installed, the game immediately crashes.
latest.log
crash-2022-07-06_00.23.07-client.txt

Solved by https://github.com/moddingplayground/frame-fabric/issues/4.


Fix #2434: Support reflective calls to default methods.

There were two problems.
First, the Analyzer did not make any effort regarding default methods when looking for targets of reflective calls. Depending on the order of lookups, the algorithm could find existing default bridges, or fail. Now, default bridges are ignored when looking for potential targets, but implemented interfaces are searched for default methods as potential targets. If a default method is found, the creation of the reflective proxy will (automatically) subsequently trigger a lookup for the target method, which will synthesize a default bridge too.
Second, the BaseLinker crashed if the lookup resolved (by chance) to a default bridge, as it tried to find an actual MethodDef for the bridge in the class' Tree. Now, if the target of a reflective proxy is a default bridge, the bridge is "dereferenced" to its original default method to find a MethodDef.

Review by @gzm0
/cc @lrytz This fixes the AssertionError. There are still 4 tests failing due to #2437.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3174/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4118/
Test PASSed.

Updated.

I think this needs a round of explanation. Because either I got this wrong completely and have to spend much more time understanding the analyzer and the linker, or something is seriously wrong here... :-/

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3175/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4119/
Test PASSed.

LGTM


installation seemingly broken

followed recommended pipx installation on haldane; result:
[adf@haldane ~]$bionorm
Traceback (most recent call last):
File "/home/localhost/adf/.local/bin/bionorm", line 5, in
from bionorm import cli
File "/home/localhost/adf/.local/pipx/venvs/bionorm/lib/python3.8/site-packages/bionorm/init.py", line 84, in
from .installer import install  # isort:skip
File "/home/localhost/adf/.local/pipx/venvs/bionorm/lib/python3.8/site-packages/bionorm/installer.py", line 13, in
from packaging import version
ModuleNotFoundError: No module named 'packaging'

Ah, I missed a non-developer dependency.  Fixed in version 0.3.2, which I just pushed.

Fixed by 55c9bd3


Traceback when invoked as «panda list»

Method 'chars' not found for invocant of class 'Any'
in method dispatch: at /home/william/share/perl6/runtime/CORE.setting.moarvm:1
in sub listprojects at /home/william/share/perl6/lib/Panda/App.pm:53
in sub MAIN at /home/william/share/perl6/site/bin/panda:34
in block  at /home/william/share/perl6/site/bin/panda:87

I think this can be fixed with something like:
my $max-ver   = @projects».version».grep({$_}).chars.max;

but there's still a couple of "undefined value" warnings from «panda --verbose list» and it's possible someone might specify version as "0" ...

I don't get that locally. Can you send me your copy of projects.json from panda's directory on which it occurs to you?

Sure I can ... how would you like it? I can't see an email address here anywhere; and the Gist interface doesn't seem to have anywhere to attach a file ... (but I can probably paste it in there if need be).

You can gist it, it's plaintext with no metadata, should be fine :)

will-crawford/70336d37b94284b7c936

I just tried it with fresh panda and I don't get any errors. Is it possible that it was fixed since it was reported? Do you still get it on updated panda?

I've built MoarVM and nqp fresh from git, and get the error from both a fresh build of rakudo star (July) and after building rakudo from git but installing the modules from star. It still happens after doing "panda install panda". The projects.json appears to be installed by star modules-install-m target. Can I substitute a different one to see what happens?

panda update should update projects.json to the list from http://ecosystem-api.p6c.org/projects.json

... then panda install panda fails with:
==> Fetching panda
fetch stage failed for panda: source-url meta info missing
  in method install at /home/william/share/perl6/lib/Panda.pm:121
  in method resolve at /home/william/share/perl6/lib/Panda.pm:218
  in sub MAIN at /home/william/bin/panda:20
  in block <unit> at /home/william/bin/panda:87

I'm giving up for now :)

Although I don't think I meant to actually close it ...

Oh, yeah, I might have broken that recently :/ Should be fixed now though.


Appinspect is generating failures for Splunklib 1.7.4

Describe the bug
Running Appinspect in precert mode with cloud tags enabled on an app containing splunklib results in a failure for lack of support for v2 API. This seems likely an issue with appinspect's check but I'm opening an issue here to try and get visibility on it.
To Reproduce

Grab an app that has splunklib.py in it
splunk-appinspect inspect ./<the app>.spl --mode precert --included-tags cloud

Expected behavior
No failed checks, since this is a Splunk module, or an update to splunklib.
Logs or Screenshots
            FAILURE: Some 'search/*' endpoints has been deprecated in Splunk
            9.0.1 and replaced by new v2 APIs.They might be removed entirely in
            a future release. An alternative could be found at
            https://docs.splunk.com/Documentation/Splunk/9.0.1/RESTREF/RESTsearch#Semantic_API_versioning
            File: lib/splunklib/client.py Line Number: 578

Splunk (please complete the following information):
N/A
SDK (please complete the following information):

Version: 1.7.4
Language Runtime Version: 3.11
OS: Mac OS 13.6

Additional context
N/A

Good morning,
Could you provide AppInspect CLI version?
Could you also try to submit the same app via AppInspect API and let us know whether this behaviour still occurs?
Thanks

I can confirm i am seeing this behaviour with Splunklib 2.0.0 and the AppInspect API as well.


Converting to Github Actions CI

fixes https://github.com/fsharplang-ru/pulsar-client-dotnet/issues/74
Added 2 workflows (for different badges):

dotnetcore-windows.yml
dotnetcore-ubuntu.yml

As part of this PR also included:

FAKE removal (nobody needs it, it's just redundant wrapper on top of dotnet CLI)
Other CI removal (Azure-pipelines and Travis)
Updated Readme

Fixed flaky tests by adding optional parameter to both Trackers to control when TickTime happens.
If this argument is not presented (which is normal), Tracker will create the good old Timer without any overhead.
If passed (unit test case) we could come up with our own object to tick (and this object is ManualInvokeScheduler)


Add onboarding hint that the cards can be tapped

Why

It's not intuitive that cards can be tapped

What

Add 1-time onboarding hint that the cards can be tapped

Mockup

Part of

#561

I added a little "tooltip" in front of the page, but two points need to be confirmed concerning the behavior of showing/dismissing this little popup:
The hint will popped up:

only one time on the page sample heath card if user starts(or resumes) the onboarding process from this page or any page before(no hint any more even back the the page).
only one time on the page sample eco card if user resumes the onboarding process from this page (no hint any more when back the the page).
in the middle of the screen

The hint will be dismissed if the user taps anywhere on the screen
If those behaviors need to be changed or are ok for you, please let me know

Sounds reasonable @cli1005 👍


Host regex volume chooser for WALs

There is a need to allow one to tie the volumes for sets of tservers to a specific volume.  This is especially useful if one wants to tie the WALs to a specific volume for performance or space purposes.  I suggest that we create a HostRegexVolumeChooser that can choose a volume based on the tserver hostname.  So given the following configuration:
general.volume.chooser=org.apache.accumulo.server.fs.PerTableVolumeChooser
general.custom.volume.chooser.logger=org.apache.accumulo.server.fs.HostRegexVolumeChooser

We might create the following configuration
general.custom.volume.chooser.hostgroup.A=host[0-4]
general.custom.volume.chooser.hostgroup.A.volumes=hdfs://cluster1/accumulo
general.custom.volume.chooser.hostgroup.B=host[5-9]
general.custom.volume.chooser.hostgroup.B.volumes=hdfs://cluster2/accumulo

This will tie the WALs for host group A to volume1 and the WALs for host group B to volume2

FWIW, the PerTableVolumeChooser property for delegating the logger scope is: general.custom.volume.chooser.logger, rather than volume.chooser.logger, and the other properties would also have the general.custom. prefix.
Would such a chooser be useful for other scopes, other than the logger scope? (the answer may drive the name and when the implementation is more general).

Yes, such a chooser would be useful for other scopes as well.  If we have a set of tservers running on top of a particular HDFS cluster, then it might be beneficial to only choose that cluster for storing RFiles.  The implementation would work for mote table and "non-table" volume choosing.

After thinking a little more about this, I think this chooser should extend the PreferredVolumeChooser and hence the properties and defaults should follow the same scheme.  Here is the class javadoc I am starting with.  Please tell me if this makes sense:
/**
 * A {@link PreferredVolumeChooser} that limits its choices from a given set of
 * options to the subset of those options preferred for a particular table (or
 * non-table) and then for the tserver host making the choice.  If no
 * configuration for the tserver host is defined, then this class delegates to
 * the PreferredVolumeChooser to make the choice.
 *
 * For tables, the configuration of a hostgroup regex is defined as follows
 * {properties later in the list override properties earlier in the list}:
 *
 * general.custom.volume.hostregex.{group}
 * table.custom.volume.hostregex.{group}
 *
 * Then the volumes for a given hostgroup are defined using the following
 * properties:
 *
 * general.custom.volume.preferred.default.{group}
 * table.custom.volume.preferred.{group}
 *
 * For non-tables (e.g. the logger scope), the configuration of a hostgroup
 * regex is defined using the following:
 *
 * general.custom.volume.hostregex.{group}
 * general.custom.volume.hostregex.{scope}.{group}
 *
 * Then the volumes for a given hostgroup are defined using the following:
 *
 * general.custom.volume.preferred.default.{group}
 * general.custom.volume.preferred.{scope}.{group}
 *
 */

I was thinking more about the use case for configuring different servers to choose different volumes and realized that we don't need a new class to achieve this. This can already be easily accomplished by setting PreferredVolumeChooser with different config on different hosts.
There is some convenience in deploying the same config globally, but still behaving differently from server to server, but I think that convenience might be marginal. What do you think?

I was a little scared to do that because of the configuration comparison mechanism that tries to ensure configurations are consistent across hosts.  Are you saying that the volume chooser configurations to not get factored into that comparison?

I was a little scared to do that because of the configuration comparison mechanism that tries to ensure configurations are consistent across hosts. Are you saying that the volume chooser configurations to not get factored into that comparison? I will try to find that code. In any case when handling large systems, having inconsistent configurations on different sets of nodes is usually asking for trouble.

Only the properties that begin with instance have to be the same across an entire cluster.

Well, then we will try doing this with local configurations and see how that goes.  If that goes well, then  I will not need to create this chooser unless you feel it is still worthwhile.

The experiment worked well.  We simply have the PerTableVolumeChooser being used at the top, and then the PreferredVolumeChooser for the logger.  The preferred volumes for the logger are then set differently in the accumulo-site.xml depending on the host.  This has the desired effect.  That being said I still think this might be a useful chooser.  It will require getting the host and port available through the parameters but that does not look too hard.  This is however lower priority now given our system is working as desired.


Migrate MentionArea to styled component

From #532
I'd like to do this.

❤  Let's try to keep all discussions about this in #532. Comment there and you'll be added to the list!