Unable to properly type hint Django models when adding a Meta class

Hello,
I'm not sure if this is the good place to ask for this, but I'm wondering how to properly type hint a Meta class added inside a Django model.
I'm using VSCode 1.74.2 which includes pylance 2022-12-20 which includes django-stubs. I'm opening my issue here as the reported error is coming from the Pylance extension.
My question is quite similar to what's discussed in issue #927. When a Meta class is defined inside a Model, errors like these start to occur:

"Meta" overrides symbol of same name in class "Model"
"assessments.models.Species.Meta" is incompatible with "django.db.models.base.Model.Meta"
Type "Type[Meta]" cannot be assigned to type "Type[Meta]"

Here is how such a class is defined:
from django.db import models
class Species(models.Model):
    class Meta:
        verbose_name_plural = "species"

And here are the lines of django-stubs concerning this:
class Model(metaclass=ModelBase):
    class DoesNotExist(ObjectDoesNotExist): ...
    class MultipleObjectsReturned(BaseMultipleObjectsReturned): ...
    class Meta: ...

So to continue the parallel with #927, I define a Meta class which is only mentioned to exist in the stubs file...
Are there other options than disabling the reportIncompatibleVariableOverride rule in this case ?

tagging @erictraut

@sronveaux by the way, you can ignore the error on that specific line using this # pyright: ignore [reportIncompatibleVariableOverride]

The reportIncompatibleVariableOverride check is a strict-mode check. It's quite picky and opinionated, as many strict-mode checks are.
I think your options are:

Disable reportIncompatibleVariableOverride for your project, for the specific files where you use this approach, or on a per-line basis.
Use a cast as follows:

class Species(models.Model):
    class _Meta:
        verbose_name_plural = "species"

    Meta = cast(type[models.Model.Meta], _Meta)

Hi @heejaechang @erictraut
Thanks for the fast reply and thanks for the list of options here!
I think I'll personally stick with disabling the rule where the problem occurs then...
Best regards!

Another way of solving this problem, is changing the type of Meta on a BaseModel that every model on the project inherits from (I really common to have this on a Django project).
# pyright: strict

from typing import Any, ClassVar

from django.db import models

class BaseModel(models.Model):
    Meta: ClassVar[type[Any]]  # pyright: ignore [reportIncompatibleVariableOverride]

class MyModel(BaseModel):
    my_field = models.CharField(max_length=10)
    my_other_field = models.CharField(max_length=10)

    class Meta:
        verbose_name = "My Model"

My initial ideia was to change the stub files to include something like a DjangoMetaProtocol:
# typings/django/db/models/base.pyi
# ...
class DjangoMetaProtocol(Protocol):
    abstract: bool = ...
    verbose_name: str = ...
    verbose_name_plural: str = ...
    ...

class Model(metaclass=ModelBase):
    DoesNotExist: ClassVar[type[ObjectDoesNotExist]]
    MultipleObjectsReturned: ClassVar[type[BaseMultipleObjectsReturned]]

    Meta: ClassVar[type[DjangoMetaProtocol]]
    pk: Any = ...
    _state: ModelState
    _meta: Options[Any]
    ...

That was when I discovered that we cannot have optional protocols members according to PEP-544.
This means what the Meta class would need to inherit from the protocol to lie to the type-checker, like:
from django.db import models
from typing import TYPE_CHECKING

if TYPE_CHECKING:
    from django.db.models.base import DjangoMetaProtocol
else:
    DjangoMetaProtocol = object

class MyModel(BaseModel):
    my_field = models.CharField(max_length=10)
    my_other_field = models.CharField(max_length=10)

    class Meta(DjangoMetaProtocol):
        verbose_name = "My Model"

I didn't like the idea of having to add a base class to Meta.

According to https://github.com/microsoft/pyright/blob/main/docs/configuration.md#diagnostic-rule-defaults reportIncompatibleVariableOverride defaults to on, not just "strict".

According to https://github.com/microsoft/pyright/blob/main/docs/configuration.md#diagnostic-rule-defaults reportIncompatibleVariableOverride defaults to on, not just "strict".

@akvadrako, this was a recent change in pyright with the introduction of the new standard type checking mode. typeCheckingMode now defaults to standard in Pyright, but still defaults to off in Pylance.
Are you seeing some sort of problem?

Hey @sronveaux  , could:
class Meta(models.base.Model.Meta):
    ...

solve it? I had the same problem and chosen this way

@rayansostenes, I love your solution! Do you not have abstract = True inside your base model's Meta class already, though? This precludes me from declaring a type for Meta.

Wouldn't this just be solved with
from django.forms.forms import BaseForm
from django.forms.models import ModelForm

from .models import Subscriber

class SubscriberForm(ModelForm):

    class Meta(BaseForm.Meta):
        model = Subscriber
        fields = ['email', 'message']

?

On second thought (and testing), it wouldn't since in the runtime environment there is no Model.Meta.
This is why casting Meta (mentioned in https://github.com/microsoft/pylance-release/issues/3814#issuecomment-1376276168) will also yield a runtime error.
The simplest option is to disable reportIncompatibleVariableOverride for the meta lines.


No ToListAsync() on the ITableQuery method

Describe the bug
We want the ability to do .ToListAsync() on the table query.
i.e.
var list = await table.Where(f => foo).ToListAsync();

The .ToListAsync() method is on the System.Linq.Async package of the IAsyncEnumerable<T> package, so you have to use .ToAsyncEnumerable() first.

Added the following (code samples):
// Note that table can be swapped for table.{some set of LINQ} or even query.
var resultset = await table.ToArrayAsync();
var resultset = await table.ToDictionaryAsync(keySelector: t => t.Id, comparer: null);
var enumerable = table.ToEnumerable(); /* Don't use this - blocking threads */
var resultset = await table.ToHashSetAsync();
var resultset = await table.ToListAsync();

Not added yet, but desirable,
var resultset = await table.ToObservableCollectionAsync();
await table.UpdateObservableCollectionAsync(collection);


ERROR: Can't roll back mmdet; was not uninstalled

How uninstall mmdet?

just remove the whole directory if you installed it following the documentation.


lltex-ls-plus: init at 17.0.1; vscode-extensions.ltex-plus.vscode-ltex-plus: init at 14.0.2

Description of changes
This adds ltex-ls-plus and vscode-extensions.ltex-plus.vscode-ltex-plus, see https://ltex-plus.github.io/ltex-plus/ and https://github.com/ltex-plus/vscode-ltex-plus
It is a fork of ltex-ls (see https://github.com/valentjn/vscode-ltex) which seems to be unmaintained.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Shouldn't this also add the vscode extension as well?

Shouldn't this also add the vscode extension as well?

I was thinking that the extension (vscode-ltex-plus) could be a separate pull request, if anyone is interested. Personally, I am using github:nix-community/nix-vscode-extensions to get the extension, and then configure it so that it uses the binary from this package (by setting the vscode option ltex.ltex-ls.path and also ltex.java.path).
Note that if we want to package the extension vscode-ltex-plus, we'd probably want to use the "offline" source, which already contains the binary from this package. Otherwise, the extension will just try to download the binary into the store path, which is doomed to fail. So these two packages don't really go together.

Shouldn't this also add the vscode extension as well?

I was thinking that the extension (vscode-ltex-plus) could be a separate pull request, if anyone is interested. Personally, I am using github:nix-community/nix-vscode-extensions to get the extension, and then configure it so that it uses the binary from this package (by setting the vscode option ltex.ltex-ls.path and also ltex.java.path).
Note that if we want to package the extension vscode-ltex-plus, we'd probably want to use the "offline" source, which already contains the binary from this package. Otherwise, the extension will just try to download the binary into the store path, which is doomed to fail. So these two packages don't really go together.

It could be the same PR, but has to be a different commit :3

It could be the same PR, but has to be a different commit :3

I added the extension. Not sure what the naming convention for the pull request is, though...?

Please squash your commits. Then it is good to go.

FYI there's also two other open PRs for ltex-ls-plus:

https://github.com/NixOS/nixpkgs/pull/357814
https://github.com/NixOS/nixpkgs/pull/363410

Plus one already merged for the VS Code extension: https://github.com/NixOS/nixpkgs/pull/362085


ESLint v6

Based on a quick check this plugin doesn't appear to be working with the new ESLint. Can look into it further later, just opening an issue to track it.
https://eslint.org/docs/6.0.0/user-guide/migrating-to-6.0.0

$ eslint .

Oops! Something went wrong! :(

ESLint: 6.0.1.

ESLint couldn't find the plugin "eslint-plugin-fp".

(The package "eslint-plugin-fp" was not found when loaded as a Node module from the directory "/home/ronan/repos/drivesheet".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

    npm install eslint-plugin-fp@latest --save-dev

The plugin "eslint-plugin-fp" was referenced from the config file in "PersonalConfig".

If you still can't figure out the problem, please stop by https://gitter.im/eslint/eslint to chat with the team.

I've gotten it semi working so will close for now, may reopen.


Fix issue that could lead to a muted actor being run

A small logical flaw was discovered in the Pony runtime backpressure system that could lead to an actor that has been muted to prevent it from overloading other actors to be run despite a rule that says muted actors shouldn't be run.
The series of events that need to happen are exceedingly unlikely but we do seem them from time to time in our Arm64 runtime stress tests. In the event that a muted actor was run, if an even more unlikely confluence of events was to occur, then "very bad things" could happen in the Pony runtime where "very bad things" means "we the Pony developers are unable to reason about what might happen".
Fixes #3901

When this is squashed for merging, the PR comment at the top should be used as the body of the squashed commit.


A11y_AzureDataStudio_New deployment: SQL Server 2019 Big Data Cluster_Screen Reader: Narrator is not announcing the current status of checkbox after changing it as checked/unchecked.

GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #BM-AzureDataStudio-Win32-Feb2022; #Azure Data Studio; #WCAG1.3.1; #Win32; #AINotInScope; #Info & Relationships; #Narrator; #NVDA;
Environment Details:
Windows Version :Windows 11 Enterprise
Application name :Azure Data studio
Repro Steps:
1.Turn on  the Narrator.
2.Launch Azure Data Studio.
3.Select the . Select the Connections icon (if not already selected) from left bar.
4. Click on the ellipses '...' and select "New Deployment".
5. In the Deployment options, select "SQL Server Big Data Cluster".
6. In Step-1 of Deployment pre-requisites , there is a Accept terms of use checkbox.
7.Verify whether that the narrator is announcing the current status of checkbox after changing it or not.
Note:
Similar issue can be observed in throughout the application whether the checkboxes are there.
Actual Result:
Narrator is not announcing the current status of checkbox after changing it as checked/unchecked.
Expected Result:
Narrator should announced the current status of checkbox after changing it as checked/unchecked.
Attachment:
Narrator Attachment:

NVDA Attachment:

Closing this as an issue with Narrator - the state is announced correctly in NVDA and I can repro this myself in Chrome.

External bug ID: https://microsoft.visualstudio.com/OS/_workitems/edit/40438048
We will take action on this bug once the external bug is fixed and closed.
And also, please confirm that this bug should close this bug only by Truster tester.  Thanks in Advance.


Implement a node script to codegen CoinType.ts

Fixes #75

It will be cool to have some description at Readme file about how to run codegen for CoinType.ts generation.


skipOptionalDependencies doesn't appear to do anything?

Building chokidar on Linux fails because fsevents is Darwin only. Adding fsevents to the skipOptionalDependencies doesn't change the behavior at all. Furthermore the code doesn't do anything with that binding except remove it from the final attribute set.
Am I misunderstanding how skipOptionalDependencies is intended to be used?

Looking at the nix definition, this option doesn't do anything. It probably did at one point. I can take a look at it.

I should add that when I build with node-4_x it seems to skip without being told to. If I build with node-6_x it throws a hissy fit.

Also, I'm very happy to help tweak this if there's more context about how it was intended or used to work that you can knowledge-share here. :smile:

I think we can close this guy now. Feel free to reopen if needed


remove unhandled promise on bad creds

Hey, I just made a Pull Request!
remove unhandled promise on bad creds
The unhandled promise had been bringing down the whole application when the creds were bad.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

My main worry is the fact that whatever happened led to the entire application going down. Do you know why that is, instead of just failing the request with a 500?
I gave a quick look at the code and it does seem to use express-promise-router right?
I'm just afraid that this stopped one exception  from being thrown, but some underlying issue remaining.

Yeah perhaps there is some underlying issue. I am fairly sure this specific case is caught. But let me dig a little deeper. I think it makes sense to write a regression test here too.

Closing this for the moment while I can get together a good argument for needing this change.

@freben Ive reduced this issue with the crashing backend to its essence https://github.com/backstage/backstage/issues/16841

Now that we can see the cases where a backend plugin can crash the whole backend, I am reopening this pull request.

fixes #16846

@punkle Note the ask to add a test around this as well.

yep. Working on it. :)

@freben that tests is done now.

lgtm let's wait for @freben to ok


Add Starknet signature

Adds Starknet Signature Authenticator
Adds corresponding tests

Implem details:

metadataURI had to be split into the IntsSequence type
This authenticator does not support Argent / OZ (by that I mean, regular account contracts) signatures (indeed it does not call Account.is_valid_signature). We need to decide on whether we want a different authenticator for those accounts, or if we wish to simply pass an argument like is_wallet to the auth

Closes #282

LGTM


Timeline for upadating Readme.md

I am puzzled by the difference between my reproduced results on v-coco and the results reported in the article. I use 2 4090 GPU with batchsize=32 for  reproducing.

Hi @ZhouGuangP,
The V-COCO code went through some major restructuring after the submission. I might have broken something in that process. I'm still trying fix it.
In the meantime, were you able to reproduce the performance on HICO-DET?
Fred.

Hi @ZhouGuangP,
The V-COCO code went through some major restructuring after the submission. I might have broken something in that process. I'm still trying fix it.
In the meantime, were you able to reproduce the performance on HICO-DET?
Fred.

Yes. I reproduced the performance on HICO-DET, it is next to the results reported in your paper. The performance is 35.44map on full,31.06map on rare 35.44 on none-rare. Thanks for your nice work.

Hello, when will your training and evaluation code on the V-COCO data set be updated? My recent work requires your code very much. I will be very grateful if you can update it! !

The documentation has now been updated.
Fred.


remove duplicate header in wrong spot of README

Just noticed  this duplicated text.  Hope this PR is helpful, but let me know if I misunderstood something.


Update JDBC driver to 2.0.9 (fix #89)

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   90.87%   90.87%
=======================================
  Files           8        8
  Lines         603      603
=======================================
  Hits          548      548
  Misses         55       55

Impacted Files
Coverage Δ

pyathenajdbc/__init__.py
86.48% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[4.0.x] Added missing destination check before processing message batch

We found a bug with message batching on the receiver side.
The bug leads to the new node silently ignoring messages from other nodes so it can not completely join the cluster.
Steps to reproduce:

Form a cluster of 2 nodes
node 1 has physical address ip1 and logical address A
node 2 has physical address ip2 and logical address B
Shutdown B
A still sends unicast messages to B (for example responses that B did not wait for).
These messages are not acknowledged and so UNICAST3 keeps them in its send table.
The messages are also retransmitted in an interval.
Start C on the same physical address ip2 as B while messages are still sent/retransmitted from A to B.
The messages to B will arrive at C.
TP.handleSingleMessage filters out single messages to the old destination B before calling MaxOneThreadPerSender.process via unicastDestMismatch.
TP.handleMessageBatch or TP.processBatch does not filter out message batches to the old destination B before calling MaxOneThreadPerSender.process.
If a message batch to the old destination B (without OOB and INTERNAL flags) is received, it starts a new batch in MaxOneThreadPerSender.MessageTable.Entry by calling MessageTable.get.
The destination of the batch is set to the old destination B.
If a message to the new destination C arrives quickly afterwards it is bundled into the same message batch inside MaxOneThreadPerSender.MessageTable.Entry.
A message batch must never bundle messages to different destinations, so here is the problem.
The whole batch is then ignored in SubmitToThreadPool.BatchHandler.run via the unicastDestMismatch check because the destination of the batch is set to B.
This includes ignoring the message to the new destination C.
Retransmissions retransmit the old message to B and the new message to C which then are again batched together and ignored.

Result: Messages arrive at the transport level but are never passed up the stack, so the new node is stuck although heartbeats work.
The fix we provide is very easy: the unicastDestMismatch check is performed for messages batches the same way it is performed for single messages before starting processing the message.

Thanks for the detailed bug report and PR! Applied

Thank you for the fast response and merging the PRs


Color: Darken() produces unexpected result

border is expected to be darker than fill but it's a different hue

@swharden I am not totally sure the used code to calculate Hue, Saturation and Luminosity is consistent. I my understanding of this value they should be limited between:

Hue: [0° .. 360°]
Saturation: [0.0 .. 1.0]
Luminosity: [0.0 .. 1.0]

That don't seems to be the case with current code:

var c = Colors.Red;
var rgb = c.ToStringRGB();
(float h, float s, float l) = c1.FillStyle.Color.ToHSL();
Debug.WriteLine($"RGB: {rgb} Hue: {h}, Saturation: {s}, Liminance: {l}");

Result is: RGB: #FF0000 Hue: 1, Saturation: -1,0079051, Liminance: 127,5

Maybe this has been fixed with #3390 ... the following looks good to me!
using ScottPlot;

namespace Sandbox.WinForms;

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();

        for (int i = 0; i < 10; i++)
        {
            double fraction = (double)i / 10;
            var circle = formsPlot1.Plot.Add.Circle(i, 0, 1);
            circle.LineWidth = 0;
            circle.FillColor = Colors.LightPink.Darken(fraction);
        }

        for (int i = 0; i < 10; i++)
        {
            double fraction = (double)i / 10;
            var circle = formsPlot1.Plot.Add.Circle(i, -2, 1);
            circle.LineWidth = 0;
            circle.FillColor = Colors.Navy.Lighten(fraction);
        }
    }
}


Music activated lights, multiple questions

Jeremy Blum just put up a vid on the tube about how he makes his addressible RGB lights activate with music.  It's a very interesting idea and would be a great thing to incorporate with the G35's.  Now I have a few questions.  I already have several of those lights in the G35's, icicles, and several yard sculptures.  I just wonder how complicated it's going to be to sync all of those together.  To sync them all together now with the stock remote is terrible at best.  Each set is off by a few m/seconds so after they run for a little while, it's way out of sync.  I know each node has an address and this could get very complicated couldn't it?  I was wondering if you guys have already experimented with the other strands/sculptures as well.  It would be a very neat idea to elaborate a bit on Jeremy's idea and not only capture the music but also the frequency and assign colors based on frequency ranges.  Just a few ideas.  I know I'm getting way way out in left field now.  I'm just glad I have a set to experiment with since the transformer went out on one set I have this year.

Thanks for the reply. You've done some awesome work. I was just excited when I found this information. I guess I should have started reading before I started asking questions which had already been addressed.
Keith N
On Dec 28, 2011, at 1:23 PM, Mike<EMAIL_ADDRESS>wrote:

Please check out the README and the example sketches, and you'll see that the library handles all the synchronization cases you describe. As for having external inputs drive the content, that's up to you; the library just provides a reliable way to talk to the lights, but it doesn't get into application-specific use cases such as what Jeremy's doing.

Reply to this email directly or view it on GitHub:
https://github.com/sowbug/G35Arduino/issues/4#issuecomment-3294192

Keith, welcome to the project. I just need to make sure that new issues filed against the project represent either real bugs in the software or else new feature requests. If you find that there are missing features that do belong in the project, please go ahead and file a new issue.
By the way, if you're interested in going farther with home light shows, visit http://doityourselfchristmas.com/. The site's a little under the weather at the moment, but it's an excellent resource for a potentially all-encompassing hobby.


TODO list

[ ] Web Crypto API  (https://developer.mozilla.org/en-US/docs/Web/API/Web_Crypto_API)
[ ] Uploads from URL to mirror stuff
[ ] Migrate config to YAML
[ ] Proper error and exception logging
[x] Migrate PUT uploads to Mojolicious
[x] add custom HTTP methods for file management to ease console access
[x] UA detection for console browsers so they are given text instead of HTML (not found pages etc)
[ ] Automatic low space detection - delete oldest files when free space is less than X MB (defined by config) until X MB amount of space becomes available

This topic is also open for new feature suggestions.

Anyone know if @somenonymous is alive and well?

Also note https://github.com/somenonymous/OshiUpload/issues/20 is propably part of said todo...


Wasted screen space

What's the Problem?
First and foremost: thanks for working on this tool!
The problem: I think there is a lot of wasted screen space on the application window. See screenshot below.
The Windows OS is trying to work well on both touch and non-touch screens and forces UI guidelines with a lot of empty space. It looks nice on the OS itself, but when I develop, I prefer to see more of my own content and minimize the tools' GUI. In addition I doubt there are that many developers who tap the screen while they are programming, and having such wide borders that are supposed to work well with fingers seems very wasteful.
Generally the application looks really nice, I don't want to suggest entirely changing the UI.

Solution/Idea
On the tools that convert or format text, I would prefer to see more from the text itself than the application margins or configuration GUI.
Alternatives
Have different margin size depending on whether tablet mode is enabled or not.
Priorities

Capability
Priority

This proposal will allow developers to see more from their content and less from the application UI
Must

The developers can work both in tablet and desktop mode
Could

DevToys Version
Version <IP_ADDRESS> | X64 | RELEASE | b949a18 | b949a18
Comments
No response

How about something like this? (early development of DevToys 2.0)

Added in 2.0 https://github.com/veler/DevToys/commit/71fdb24ca8ef374adeb27cbf1e53f0e4698e4555


New release any time soon?

Hello,
Thanks for the great product! We're running into https://github.com/opstree/druid-exporter/issues/100 in our system. That bug looks to have been fixed the day after the last release was made (v0.10.0). Any chance of a new one any time soon?
Thanks

Hi @klarose ,
We have a plan to release the next version by this week

Hi! Any news regarding this? Thanks!


Make gRPC ServerParameters configurable

Which problem is this PR solving
At the moment, there is no way to override the ServerParameters used in
the keepalive configuration of the gRPC server. As to @bdarfler's
comments in #263, issues can arise where new hosts will never be
connected to because clients will reconnect to already existing hosts
and by the time replacement hosts are available, all clients will have
reconnected to existing hosts with a
MaxConnectionAge/MaxConnectionAgeGrace of
infinity.
Short description of the changes
The changes made are to plumb through the configuration of the GRPC
ServerParameters from the viper configuration and into the instantiation
of the actual gRPC server. These changes kept the previous default
configuration of refinery as the defaults of the configurable settings,
so anyone who doesn't explicitly override any of these configurations
should have exactly the same behavior.
Closes #263

I think there might be a bug with GitHub. I tried to re-request reviews from both @kentquirk and @MikeGoldsmith but it seems to have removed @emilyashley and then says I need to re-request review from either Kent or Mike. And if I re-request one, the other gets shown as needing a re-request.

Whatever GitHub did, I updated the docs and commented about the min value in question!


create a model for person

Finished Person model in models/Person.js, updated the realtionship in the models/index.js file!


steam: update generated runtime dependencies

Motivation for this change
The referenced zenity URL wasn't valid anymore:
trying http://repo.steampowered.com/steamrt/pool/main/z/zenity/zenity_3.4.0-0ubuntu4+steamrt2+srt6_amd64.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404 Not Found

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Hmm, so they depub binaries published there? That's a pity, as it effectively breaks users - we now need to worry about backporting this to 20.03 as well I guess?
Can you send them an email, and ask them nicely if they can not do that? ;-)

Yup, I just ran into this while trying to upgrade 20.03.
Does it only affect zenity or are there other packages also affected?
I spot checked a few of 20.03's derivations and, while most of my samples weren't in Steam's repos anymore, they were still available on the abbradar mirror.
Is there any reason why we don't backport changes to runtime-generated.nix to stable in general? (8bd3cf22c2b7f42c4e30b81c5d9c7dcc57d43b92, 877baac95becce52d898642bda8d8d9ade3bd542 and 467a987f0be457683001c38c24c50aa71f58ae4b)

I ran a quick little script on all .debs in my Nix store to find out which ones can't be built anymore:
for deb in /nix/store/*.deb ; do nix show-derivation "$deb" | jq 'keys | .[0]' -r ; done | while read drv ; do nix-build --check "$drv" || echo "$drv" >> /tmp/broken ; done

"/nix/store/90fjshgnbdwf8zkirlsi0d40wslcwnza-libxcb-dri2-0.deb.drv"
"/nix/store/2qzhpphh8a18ql7klgp1cfbgnrga2xbk-libxcb-shape0.deb.drv"
"/nix/store/77alsjgckkhjh5rcrsc84dq57xjkampx-libxcb-xinerama0.deb.drv"
"/nix/store/5fwx5w634v7q9gf5h9qyvhci00yw5rnc-libpixman-1-0.deb.drv"
"/nix/store/61s9fqyvzidz6qn60psy9ddpdx4wgz5p-libldap-2.4-2.deb.drv"
"/nix/store/9zby5xjsh3qcjc6ygjbrk2qv65dzkwms-libhcrypto4-heimdal.deb.drv"
"/nix/store/0bfn9shya6q9x8izi19myzgih0ckiyqf-libgpg-error0.deb.drv"
"/nix/store/an2zg1s550vdahfvx5swj4fqdh4nq50g-libkrb5support0.deb.drv"
"/nix/store/4cbw1pzqz4k8wgqq3ahiajfh17rfr3n2-libxcb-xvmc0.deb.drv"
"/nix/store/5kjlg35c6xiwdsa9z1jz9bwj28rbkjgh-libcg.deb.drv"
"/nix/store/ddzb7knl08244km2h0nrm7w7kph4x8z3-libsdl2-mixer.deb.drv"
"/nix/store/10rc69xvrbmpd7jk0bkii43zllda31v7-libavahi-common3.deb.drv"
"/nix/store/09qd0v7dbp70wgy0lxy3kmvj5k0h7nqh-libasyncns0.deb.drv"
"/nix/store/8s16qbp3k19rjscsh7qgny4h9w28slsk-libasound2-plugins.deb.drv"
"/nix/store/61yh8l49qlih42wx2sq8d5bbx2widnp3-libspeexdsp1.deb.drv"
"/nix/store/f3rdxal48amisys413vfq5zf0yxzy2x9-libgnutls26.deb.drv"
"/nix/store/gngq8xjblivrxrwcyxfibfknxkgbswxj-libcomerr2.deb.drv"
"/nix/store/1z55hf9zmh05cd273gzbcfd25i80nd23-libsqlite3-0.deb.drv"
"/nix/store/6mhfdxw5yqq2w44zfvm1h3xy4riwc7vz-dconf-gsettings-backend.deb.drv"
"/nix/store/4kj1qkf8i0iigqbp29mhpl0675h6mhvn-libvorbis0a.deb.drv"
"/nix/store/1llf5z7a4xml03ls5razhi01h7l1pwmj-libnettle6.deb.drv"
"/nix/store/4dyn20v5rywav47w4yask1jczc6ycf3g-libgudev-1.0-0.deb.drv"
"/nix/store/39yv2qkzgi0jlvyaqg6mjzs94k6zszh3-libxcb-dri3-0.deb.drv"
"/nix/store/7ykzrsrh1pl9rg4rqzg4glv170dngjsw-libmikmod2.deb.drv"
"/nix/store/g9zg1ixklyrxwrgb7qfaz7qkmh7l3085-libtasn1-3.deb.drv"
"/nix/store/5rfv9kmqiv8d2rsn5inz66jsiwq10yw4-libxcb-dri2-0.deb.drv"

Is there any reason why we don't backport changes to runtime-generated.nix to stable in general? 8bd3cf2, 877baac and 467a987)

I don't know of any, better backport those and have a non-working package.

Was this backported to 20.03?

No, please do, I'm currently otherwise occupied.
On 5/2/20 7:42 PM, Pavol Rusnak wrote:

Was this backported to 20.03?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/86358#issuecomment-622989352, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABAED56TTHU36SDAI4A64DRPRLQDANCNFSM4MUCZ46A.

Backport to 20.03 in https://github.com/NixOS/nixpkgs/pull/86619

Why don't we use the steamrt-images snapshots hosted here: https://repo.steampowered.com/steamrt-images-scout/snapshots/
It should be a lot easier to maintain going forward, as we would just dump the image in a directory instead of needing to fetch each steamrt package individually.


SvgReader improvements

Ahh shoot, I went and made path parsing changes without first looking at this branch. Would it be possible for you to merge?

Done

Sorry it took so long to merge these in. Thank you so much for your effort!


Fix unavaiable sidecar link

Make it works.

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/kind bug

thanks @vincent-pli, verified the link 👍
/lgtm


Multi-project setup confusion after version 0.7.4 (and still in 0.7.5)

I have a (pretty big, but consistent) multi-project setup, that I needed help with setting up for the previous versions.
Now that things have changed again since version 0.7.4, and the documentation has not become more clear to me, I'd like to request help once again. I'm going to call the Android studio project the 'project' and each sub-project 'modules'.
Project structure
:app (Android)

:core:data (Android)
:core:domain (JVM)
:core:presentation (Android)

:featurex:data (Android)
:featurex:domain (JVM)
:featurex:presentation (Android)

... Other features...

The app, data, and presentation modules have different flavors, but this is pretty consistent:

Each presentation module has a release build flavor that I'd like to use for Kover tests coverage.
Each data module has a tstRelease build flavor that I'd like to use for Kover test coverage.
The app module also has a tstRelease build flavor.

Current configuration:

Each module (Android and JVM) has the Kover plugin applied
As suggested last time: the :app module contains the following

dependencies {
    forEachModuleLayer { // (All above-mentioned project modules)
        implementation(project(it))
        kover(project(it))
    }
   ...
}

koverReport {
    // filters for all report types of all build variants
    filters {
        ...
    }
    defaults {
        mergeWith("tstRelease")
        ...
    }
}

The pipeline runs this as:
./gradlew app:koverXmlReport app:koverHtmlReport
which generates the expected single coverage output for the entire project.
Now, after updating to Kover 0.7.5, I get the following error:
> Kover android variant 'tstRelease' was not matched with any variant from dependency ':core:domain' of project ':app'. Check that the Kover plugin is applied in the ':core:domain' project and there is a variant compatible with 'tstRelease' in it.

The Kover plugin has been applied to :core:domain, but adding this to the core/domain/build.gradle.kts also doesn't work:
koverReport {
    defaults{
        mergeWith("tstRelease") // Or "release" for that matter
    }
}

It gives me the error:
Build variant 'tstRelease' not found in project ':core:domain' - impossible to merge default reports with its measurements.
Available variations: []

In short, I'm stuck and do now know how to proceed from here. Could you guys point me in the right direction?

Hi,
this comment provides recommendations for creating a merged report for Android + JVM projects.
The first thing you need to change is to move the code with kover(project(it)) dependencies to a project where no Android plugin is used. Most often, the most appropriate place is the root project.
In this case, you can add a dependency by iterating through if allowed (if allowed).
Then you need to add it to all presentation projects
koverReport {
    defaults{
        mergeWith("release")
    }
}

also in all data and app projects
koverReport {
    defaults{
        mergeWith("tstRelease")
    }
}

Specify reports filters
koverReport {
    // filters for all report types of all build variants
    filters {
        ...
    }
}

only in the project in which you wrote kover(project(it)) (preferably in the root project)
So, if the combined report is located in the root project, then you can generate it with the :koverHtmlReport command

This worked :D
Thanks for the help!

Hey @JWvanV just curious what your forEachModuleLayer {} dsl is? Is it a custom gradle plugin?

Oh, just a function I wrote inside of BuildSrc.
Because of the project structure (see OP) and having about 9 features, all with 3 sub-modules (layers), I didn't want to have to write 27 'implementation' lines. (doubled if I would have to do both an implementation line and a kover line)
So that function just generates a module string for each of the 27 module-layers (combines a list of 9 feature names and a list of the 3 layer names) and then runs a foreach with an optional block.
So no magic DSL here, just a helper function for my project structure :)


Feature: Generic model tracking

Is your feature request related to a problem? Please describe.
run.log_model("model_type", ....)

is bad
Describe the solution you'd like
run.log_model(...)

low-priority: TF is sufficient for now


Draft: added GH actions for windows and Mac

This PR includes the GH to run the test on Mac and windows.
closes #111

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

Project coverage is 79.31%. Comparing base (48f3aa5) to head (9cab1f0).
Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
+ Coverage   79.21%   79.31%   +0.10%
==========================================
  Files         133      133
  Lines       11193    11223      +30
==========================================
+ Hits         8866     8902      +36
+ Misses       2327     2321       -6

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


passing the test !

this version only passing the test with the vcr
haven't done with refactor, test coverage
please help me checking with the code
flay tweet_spec.rb  => Total score = 0

I've added test coverage reporting feature in #10, could you merge master in first?

@Leoli0320  you might need to resolve the conflicts first, or you can close this one and do it again on another branch then open a new pull request.
It should be much more easier to open a new pull request since I've added simplecov on master and all you need to do is stubbing requests using  vcr and webmock.

@edsong1257 @Leoli0320 Just for the heads up, I updated the code:

The names of the cassettes are now automatically generated based on the spec description and the file name of the spec.
Remove unused files
Updated README.md

Test results:
$ bundle exec rake test
Run options: --seed 45136

# Running:

..

Finished in 0.079843s, 25.0491 runs/s, 25.0491 assertions/s.

2 runs, 2 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for RSpec to /Users/cyhsutw/Projects/temp/twitter-api/coverage. 34 / 34 LOC (100.0%) covered.

Quality metrics
$ bundle exec rake quality
bundle exec rubocop lib
Inspecting 2 files
..

2 files inspected, no offenses detected
bundle exec flay lib
Total score (lower is better) = 0
bundle exec flog lib
    23.7: flog total
     3.0: flog/method average

     7.3: Twitter::Client#search_tweets    lib/twitter/client.rb:15
     5.2: Twitter::Tweet#initialize        lib/twitter/tweet.rb:6
     3.6: Twitter::Tweet::search           lib/twitter/tweet.rb:12

If there's no further problems, I think it's ready to go.
Please merge if you'd like to.


Auto Reload actions on editing

Auto reloading actions server on edit:
Currently, every time I edit the actions file I have to stop the rasa actions server and restart it. It is kind of overkill since you already built your model and just tweaking with actions.
Overview of the Solution:
Sanic server comes with a DEBUG mode. We can incorporate it into rasa run actions command as an option to enable reloading server.
Definition of Done:
What needs to be there to consider this feature as done?

[ ] Tests are added
[ ] Feature is documented
[ ] rasa run actions --debug

I can work on this if we think this feature is important.

Thanks for the suggestion! what does the implementation for that look like? cc @wochinge

@bhavaniravi just my 2 cents, I think as we have --debug already mostly for increased logs, I wouldn't like this implementation to go under that flag, as it would be surprise new changed functionality. I'd prefer a new descriptive flag, for example --hot-reload or something like that. not sure what @wochinge thinks though

I am not hooked to what it's called. I think this feature would help developing actions much easier

I agree with @erohmensing . We shouldn't do this with --debug. How about using --auto-reload or --hot-reload and then setting auto_reload=True ( https://sanic.readthedocs.io/en/latest/sanic/debug_mode.html#manually-setting-auto-reload).

@bhavaniravi Did you try out whether this actually works when you change the actions.py?

I checked out rasa_sdk codebase and executor is written in such a way that the function references are held in a dict and called accordingly, which means the edit on those references will not reload the Flask or WsgiServer used.

that's what I thought 😞

I am not sure iti s relevant how the functions are stored as long as the modules are loaded.

The only thing we should need to do is "reload" the function in debug mode

@tmbo so it is possible then? In that case @bhavaniravi are you still interested in implementing this functionality?

yes, sanic already has this build in, so we should reuse it's hot reload functionality. not sure if we run into issues though, e.g. https://github.com/huge-success/sanic/issues/1248


no such file or directory "blank_code.pkl"

Hi I come back again! sorry to bother you again!
In training stage 1, when i run command, it show "no such file or directory "blank_code.pkl". but I see that  in eval step 0, it will generate blank_code.pkl, so i decide to run the command in eval step 0,but it still show "no such file or directory "blank_code.pkl". And i really don't  know how to solve this problem?

I uploaded a new version of code, please try this new one.

I uploaded a new version of code, please try this new one.
thank you ,that's help a lot


remove src/Compiler dependencies from src/{Builder,Transform,Conversion}

to make it easy to use each of the dialects and passes as libraries that people can link into their own compiler binaries without using the onnx-mlir binary, and to avoid unnecessary cyclical dependencies, we should avoid making the dialects and passes depend on anything in src/Compiler (with #include or cmake LINK_LIBS)

@jenkins-droid test this please

ok, I tried
ONNX_MLIR_FLAGS="-O3 -march=arm64" make check-onnx-backend

and it ran successfully for me on a M1 Mac

Jenkins Linux amd64 Build #12678 [push] remove src/Compiler depe... started at 16:37

Jenkins Linux ppc64le Build #11691 [push] remove src/Compiler depe... started at 17:46

Jenkins Linux s390x Build #12698 [push] remove src/Compiler depe... started at 17:37

Jenkins Linux s390x Build #12698 [push] remove src/Compiler depe... failed after 3 min 15 sec

Jenkins Linux amd64 Build #12678 [push] remove src/Compiler depe... failed after 1 hr 33 min

Jenkins Linux ppc64le Build #11691 [push] remove src/Compiler depe... passed after 1 hr 53 min


chatbot deploy error

i created a chatbot using bard api and streamlit when i deployed it, the chatbot gives a response error ? please tell a possible solution. Only in localhost, it is working correctly

If it functions properly on localhost but you're unsure about the exact server environment, there is a possibility that your server is behind a proxy server or that port forwarding is being used.
In such cases, please refer to the following Behind a proxy.
Due to the numerous possibilities and the lack of specific information, I'm unable to provide further assistance. Since it is not an issue with the package itself, I will close the issue.


Support Pandas 2.2

Currently the pandas version is limited to <2.2. Please support pandas 2.2.x as well.

Resolved by #4079


Can't setup in existing Vue project

I followed the instructions as specified (run get storybook). I get the following errors (related to dependencies it seems).
error.log.zip

It seems the project type was undetected, this means the storybook CLI was unable to determine what to add.
Try setup/add storybook manually (it's not that hard!)
I have an example here:
https://github.com/ndelangen/storybook-vue-example
You need to:

add the dependencies
add the .storybook folder.
add stories

I hope this helps you.

nom install wasn't working, yarn install did the job (+manual setup).


Output to a list of png instead of a single png

Hello,
How can I use image_gen.py to iterate through the RGB list and then produce every single colour from it?

Yes, just change build_image: https://github.com/mattbierner/One-Pixel-Cinema/blob/master/image_gen.py#L22
At that point you have the complete list of colors for further processing


feat: FAQ - Add Network and Nodes Chapter

Removes the old Network and Validator pages and sets up the new Network and Nodes folder, including the following pages:

Blockchain Architecture
Network Configuration
Peer Connections
Node Setup
Validators
Security
Staking

Thanks, Johann, for taking the time 🙏🏻😙 I applied your suggestions and clarified the answers. Also added redirects and modified the headings as in the other sections

sadly we now have a few conflicts bc we merged the other PRs :/
should be easy to fi anyway


Show OpenStack Server details in Jenkins WebUI computer page

The OpenStack plugin copes with configuration ambiguity by choosing the newest match.
In situations where new things are being added/changed while new servers are starting, it isn't clear which Jenkins slaves started from the "old" settings and which started from the "new" settings.
This enhancement

Adds metadata to the OpenStack VM so one can see "at a glance" what it was booted from
Displays that metadata, and other OpenStack VM settings, on the Jenkins .../computer/myAgentName page.

I like the idea, can you please attach a screenshot of how will it look like? Btw, I have a preference of using tables over dls, with pane and similar css classes, we can make them sortable and hopefully easier to consume.

See systemInfo page [1] for inspiration, it cuts the number of "lines" to half making it denser. Implementations comments coming soon...
[1] https://github.com/jenkinsci/jenkins/blob/d4737b2b60438ce0e3cb7a8a6f26675f7c880914/core/src/main/resources/jenkins/model/Jenkins/systemInfo.jelly#L38-L61

Jelly reworked in commit https://github.com/jenkinsci/openstack-cloud-plugin/pull/212/commits/f7eb28cd37b8a675dfb62defa9b5d13f5cdbc54a.
The details now looks like this:

I've added code to show the Server ID too, as that was an obvious omission in the previous implementation (see commit https://github.com/jenkinsci/openstack-cloud-plugin/pull/212/commits/5f064d554445a825492253ecee9443bb04ba4c93).
This screenshot is from a node that was booted from a VolumeSnapshot, so it includes the description of the VolumeSnapshot (implemented in https://github.com/jenkinsci/openstack-cloud-plugin/pull/212/commits/1cb26dd3092dcbf9b739dccb94eac339b6ad3d7e)

Hmm, ci.jenkins.io didn't trigger a new build after those commits.
I'm going to close this PR and re-open it in a bit to see if that triggers a build...

OK, code enhanced and refactored.
We now show "live" data from the OpenStack API but cache the results (for 20 seconds) to prevent us from spamming the API.
This also removes any need to persist additional data within the computer's config.xml.
Let me know what you think, both of the (new) code and the new appearance...
For a slave VM that's alive, we'd see:

For a slave VM that's been deleted by a 3rd-party e.g. via Horizon, we see less information but get told that all is not well:

Good, can we eliminate "jenkins-instance", "jenkins-scope", "jenkins-template-name" and perhaps even "jenkins-cloud-name" from the metadata, to limit the amount of the data somewhat? All that should be obvious from looking at the slave page.

Yes, we can do that.
I've also turned the flavor field back into a single line instead of having it split out into multiple fields.
...and I've switched to camelCase instead of PascalCase because that's what the OpenStack API uses (and I personally prefer the look of it).
This is the result:

@pjdarton, I intend to merge this with some more cleanup, some fields removed. Here is how it will look like:

That looks good to me.

Thanks!


Prevent building component that doens't exist.

Coverage increased (+0.4%) to 91.489% when pulling 766a122a0005106cda6fcf1dc4c56053c8f17ded on jansav:component-not-found into ed78d3cbd22ea6905ed4520bb05cbf69efea4254 on Houston-Inc:v1.0.4.


NearestCentroid method get_attr

Hello,
For NearestCentroid function, I have an error for get_attr method.
With this code :
from verticapy.learn.neighbors import NearestCentroid

drop(name = schema +".model_titanic_test", cursor = cur, method = "model")
model = NearestCentroid(name = schema +".model_titanic_test", cursor = cur, p = 20)
model.fit(titanic,["age","fare"], "survived")
model.get_attr()

I have the following error :

why?
Regards,
Alexandre

Hi Alexandre,
Thanks for reporting this bug. It will be corrected in the next release.
Badr


fix(deps): :arrow_up: update angular monorepo to v17.0.5

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@angular/animations (source)
17.0.4 -> 17.0.5

@angular/common (source)
^17.0.4 -> ^17.0.5

@angular/common (source)
17.0.4 -> 17.0.5

@angular/compiler (source)
17.0.4 -> 17.0.5

@angular/compiler-cli (source)
17.0.4 -> 17.0.5

@angular/elements (source)
17.0.4 -> 17.0.5

@angular/forms (source)
17.0.4 -> 17.0.5

@angular/language-service (source)
17.0.4 -> 17.0.5

@angular/platform-browser (source)
^17.0.4 -> ^17.0.5

@angular/platform-browser (source)
17.0.4 -> 17.0.5

@angular/platform-browser-dynamic (source)
^17.0.4 -> ^17.0.5

@angular/platform-browser-dynamic (source)
17.0.4 -> 17.0.5

@angular/router (source)
^17.0.4 -> ^17.0.5

@angular/router (source)
17.0.4 -> 17.0.5

@angular/service-worker (source)
17.0.4 -> 17.0.5

Release Notes

angular/angular (@angular/animations)
v17.0.5
Compare Source
core

Commit
Type
Description

6be88040d1
fix
avoid stale provider info when TestBed.overrideProvider is used (#52918)

dee50f1d78
fix
inherit host directives (#52992)

07920d96d4
fix
Reattached views that are dirty from a signal update should refresh (#53001)

migrations

Commit
Type
Description

aab7fb8654
fix
Add ngForTemplate support to control flow migration (#53076)

dbd6f386ea
fix
allows colons in ngIf else cases to migrate (#53076)

5b9f896009
fix
cf migration fix migrating empty switch default (#53237)

2b3d3b0fe1
fix
CF migration log warning when collection aliasing detected in @for (#53238)

dffeac8386
fix
cf migration removes unnecessary bound ngifelse attribute (#53236)

00cb3339ba
fix
control flow migration formatting fixes (#53076)

c22af72f75
fix
fix off by one issue with template removal in CF migration (#53255)

ba6d7fe018
fix
fixes CF migration i18n ng-template offsets (#53212)

8f6affdd64
fix
fixes control flow migration common module removal (#53076)

6ae408847c
fix
properly handle ngIfThen cases in CF migration (#53256)

0fcef65cea
fix
Update CF migration to skip templates with duplicate ng-template names (#53204)

router

Commit
Type
Description

91486aaf07
fix
Resolvers in different parts of the route tree should be able to execute together (#52934)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-12-02T15_20_20_332Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-12-02T15_20_20_332Z-debug-0.log


Building thrift fails when not building as root - some configuration missing

I'm following the README build instructions on a Debian-based distribution (Mint 18.2, which itself is based on Ubuntu 16.04).
Now, I've gotten to the part where I'm building thrift. Since I'm not root, here's how I configured it:
./configure     --with-lua=no     --with-python=no     --with-php=no     --with-ruby=no     --prefix=$HOME/opt/mapd-deps

That is, same as in your instructions but under a folder of my own, not the system-wise /usr/local/mapd-deps. Then, while makeing, it seems additional packages are being downloaded and built - but they are targeted at a system-wide path. Here's the relevant part of my console output:
mvn.ant.tasks.download:
      [get] Getting: http://repo1.maven.org/maven2/org/apache/maven/maven-ant-tasks/2.1.3/maven-ant-tasks-2.1.3.jar
      [get] To: /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/tools/maven-ant-tasks-2.1.3.jar
      [get] Not modified - so not downloaded

mvn.init:
[artifact:dependencies] [WARNING] Overriding profile: 'maven-ant-tasks-repo-profile' (source: pom) with new instance from source: pom

init:

compile:

dist:

javadoc:
  [javadoc] Generating Javadoc
  [javadoc] Javadoc execution
  [javadoc] Loading source files for package org.apache.thrift...
  [javadoc] Loading source files for package org.apache.thrift.async...
  [javadoc] Loading source files for package org.apache.thrift.meta_data...
  [javadoc] Loading source files for package org.apache.thrift.protocol...
  [javadoc] Loading source files for package org.apache.thrift.scheme...
  [javadoc] Loading source files for package org.apache.thrift.server...
  [javadoc] Loading source files for package org.apache.thrift.transport...
  [javadoc] Constructing Javadoc information...
  [javadoc] Creating destination directory: "/home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/javadoc/"
  [javadoc] Standard Doclet version 1.8.0_131
  [javadoc] Building tree for all the packages and classes...
  [javadoc] /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/src/org/apache/thrift/server/TExtensibleServlet.java:68: warning - @return tag has no arguments.
  [javadoc] /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/src/org/apache/thrift/server/TExtensibleServlet.java:76: warning - @return tag has no arguments.
  [javadoc] /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/src/org/apache/thrift/server/TExtensibleServlet.java:84: warning - @return tag has no arguments.
  [javadoc] Building index for all the packages and classes...
  [javadoc] Building index for all classes...
  [javadoc] Generating /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/javadoc/help-doc.html...
  [javadoc] 3 warnings
      [jar] Building jar: /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/libthrift-0.10.0-javadoc.jar

install:
     [copy] Copying 11 files to /usr/local/lib
Attempt to copy /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/libthrift-0.10.0-javadoc.jar to /usr/local/lib/libthrift-0.10.0-javadoc.jar using NIO Channels failed due to '/usr/local/lib/libthrift-0.10.0-javadoc.jar (Permission denied)'.  Falling back to streams.

BUILD FAILED
/home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build.xml:149: Failed to copy /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/libthrift-0.10.0-javadoc.jar to /usr/local/lib/libthrift-0.10.0-javadoc.jar due to java.io.FileNotFoundException /usr/local/lib/libthrift-0.10.0-javadoc.jar (Permission denied)

Total time: 2 seconds

So, it would be nice if the README.md actually included the appropriate setting, command or switch to get that to happen (or alternatively, a disclaimer saying that thrift installation effectively requires root privileges).

Thanks for your interest in MapD.
In this case (avoiding all elevated privs) it looks like the appropriate change would be to set the JAVA_PREFIX env var when running configure. I'll add this to the readme after verifying the change.
export PREFIX=$HOME/opt/mapd-deps
JAVA_PREFIX=$PREFIX/lib ./configure     --with-lua=no     --with-python=no     --with-php=no     --with-ruby=no     --prefix=$PREFIX

Also consider using an environment variable for the dependencies directory, i.e.
MAPD_DEPS_DIR=/usr/local/mapd-deps

and then using that in the rest of the instructions, so that when someone installs as a non-privileged user they'll just replace that line with, say
MAPD_DEPS_DIR="$HOME/opt/mapd-deps"

and will be able to copy-paste the rest of the instructions (except for the sudo ones which will have to go to their system administrators).

BUILD FAILED
/home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build.xml:149: Failed to copy /home/eyalroz/src/mapd-core/scripts/thrift-0.10.0/lib/java/build/libthrift-0.10.0-javadoc.jar to /usr/local/lib/libthrift-0.10.0-javadoc.jar due to java.io.FileNotFoundException /usr/local/lib/libthrift-0.10.0-javadoc.jar (Permission denied)
mabe Permission denied


Simplify batching

A re-think of the batch process. Simplified, better comments, a test case. The involvement of WGD in batching is removed as we're using smaller cohorts of samples, so we don't necessarily have the numbers required to batch on multiple different parameters.
This doesn't address the PCR +/- state as the data isn't currently available here. We could add this in at sample batching if we can draw from another source, or we could use PCR status in the custom cohort builder (when that functionality comes around) to ensure that this stage will only run with either PCR- or PCR+/unknown samples, meaning that the batch generation doesn't need to consider it.
Process:

split samples by sex (inferred from chrX ploidy)
order samples by median coverage
calculate appropriate number of batches to get the # samples in each batch within parameters
split each sex-list into equal parts, one for each batch
create batches by joining lowest cov. male &female samples, up to highest coverage male & female
annotate each batch with mf ratio, coverage medians

thinking here is that if we need to drop additional samples into a batch later, this metadata will help determine the best choice instead of requiring a complete re-run

Can you assume there is a metamist field: pcr_status [positive, negative, unknown] and add that to the batching logic. At the moment I think we should group the unknowns with the +ve's. I will talk to Eddie about populating the field in metamist.

assume there is a metamist field: pcr_status

Will this be in the Meta, or as a top-level attribute in the database? Should be easy enough to adjust, might have to immediately change when we shuffle to SequencingGroups... but so will everything else.
Will throw this on Slack for discussion


Add polyjuice to monorepo

About fuzz test, there is pending pr.
We can disable EVM FUZZTEST before it gets merged.

I see, the workflow is disabled, I can force merge the PR after we collected two approvals.


Examples don't compile

It looks like the examples are not compiling due to a change in the second generic argument to SdkError. This issue tracks fixing the examples, and also determining why this wasn't caught by CI or the release process.

Fixes in progress:
https://github.com/awslabs/smithy-rs/pull/2931
https://github.com/awslabs/smithy-rs/pull/2930

This has been fixed with the August 23rd release.


Not detecting files on Windows local folder

I've edited the config file and used the full path as "Y:\Compressed Downloads" using 2 back slashes as instructed on the manual, still it does not detect any .rar files put in there at the room or inside subfolders. I check the status log, and nothing comes up. Am I missing something?

Did ya figure this out?

I found out it was my mistake not uncommenting the [[folder]]


Program randomly decides it cant read the char/list xml

Error parsing char/list xml!
Error description: Start-end tags mismatch
Error offset: 65
Error:
<EMAIL_ADDRESS>| Error parsing char/list xml!
Error description: Start-end tags mismatch
Error offset: 65
Error loading config, can not continue program.
I never changed it between compiles but it just decides to stop working

You loaded too many accounts in a certain period of time.


Update to Node 10.1.0 + dependencies

Following up on https://github.com/Shopify/draggable/pull/218 and https://github.com/Shopify/draggable/issues/219 I have experimented with upgrading to the latest node + all dependencies.
As @om3n07 described, I encountered the error with upath, which appears to be the result of an indirect webpack dependency...
In order to solve it and get a successful yarn install, I had to add upath 1.1.0 in our package.json and completely delete the yarn.lock. Only then could I complete a dependency install and build the project.
@tsov if we decide to go ahead and merge this, I say we open an issue to remove upath as an explicit dependency once the issue has been resolved (will need to be done on `chokidar I believe).
While I was at it... I discovered two issues with the Examples, possibly introduced by dependency updates. I will point them out in code comments.
Finally, we now have some critical issue with eslint. I get the following when linting both the root and examples:
The comparison function must be either a function or undefined
TypeError: The comparison function must be either a function or undefined
    at Array.sort (native)
    at getAcceptableSlots (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint-plugin-sort-class-members/dist/rules/sort-class-members.js:276:3)
    at /Users/beefchimi/Development/Shopify/draggable/node_modules/eslint-plugin-sort-class-members/dist/rules/sort-class-members.js:178:25
    at Array.map (<anonymous>)
    at getClassMemberInfos (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint-plugin-sort-class-members/dist/rules/sort-class-members.js:176:5)
    at ClassDeclaration (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint-plugin-sort-class-members/dist/rules/sort-class-members.js:31:20)
    at listeners.(anonymous function).forEach.listener (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint/lib/util/safe-emitter.js:47:58)
    at Array.forEach (<anonymous>)
    at Object.emit (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint/lib/util/safe-emitter.js:47:38)
    at NodeEventGenerator.applySelector (/Users/beefchimi/Development/Shopify/draggable/node_modules/eslint/lib/util/node-event-generator.js:251:26)
error Command failed with exit code 1.

I will try to investigate this tomorrow.
Browsers:

[x] Chrome version
[x] Firefox version
[x] Safari version
[x] IE / Edge version
[x] iOS Browser version
[x] Android Browser version

@tsov

drag:move should be a cancelable event

Yea I thought so as well, as clearly at one point it was, but when it stopped working in this latest update, I went and looked at the README and it says "cancelable: false"
https://github.com/Shopify/draggable/tree/master/src/Draggable/DragEvent#dragmoveevent
That is really the only issue I'd want you to look into. The rest I can take care of, and if not, I'll reduce the scope of this PR. Might not get around to it until after vacation.


[BUG][Android][Chat -1.1.0-beta.3] Messages have a delay between time of sending and time of receiving

Describe the bug
When a host messages the participants, messages have a delay of 2-4 seconds between the time they were sent and the time the participants see them
To Reproduce
1.	Host a Teams conference
2.	Join with an Android device with Calling 1.3.0-beta.1 and Chat-1.1.0-beta.3
3.	Ask the host to attach some images
4.	See error
Code Snippet
private fun startMessagesPolling() {
        val runnable: Runnable = object : Runnable {
            override fun run() {
                Timber.d("fetch new messages")
                val newMessages = chatThreadClient?.listMessages(ListChatMessagesOptions(), null)
                formatMessages(newMessages)

                handler.postDelayed(this, MESSAGES_POLLING_INTERVAL)
            }
        }

        handler.post(runnable)
    }

Expected behavior
The messages should not have this delay as it can impact the user experience , especially if the user has an unstable internet connection
Setup (please complete the following information):

OS: Huawei P30 Pro
Version of the Library used
Calling 1.3.0-beta.1 and Chat-1.1.0-beta.3

Additional context
Behaviour is the same with Chat-1.1.0-beta.4

Closing this issue as we found that the delay was caused by our polling mechanism for new message. The issue is no present with the SDK


Getting started videos - menu overlap

What you did (step by step)
-resized getting started videos page
Where does this happen on the OSF?
-https://osf.io/getting-started
What you expected
-resizing to happen without menu overlap
What actually happened
-menu and video/text sections overlapped

Potential causes
Suggest a solution
What will it look like when this issue is resolved?
-overlap will not occur

This is also happening to me when browser window is small. However, the outline re-sizes when you scroll by the videos and then goes back to overlapping after.

@GaryKriebel I'm interested in taking a jab at this

OK @zkraime -- We can assign it directly in GH if you accept your invite to the COS org. -- Did you get one? Thanks.

@GaryKriebel I forgot to check my other email yesterday afternoon - just accepted!
Thanks

Done!

Fix has been merged. Will be released in 0.41.0


how can I see the graphs in tensorboard?

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

This looks like a simple user question. Please follow our tutorials, or reach out to stackoverflow.
If you suspect there is a bug, you will need to file an issue under tensorboard repository.


CellposeTraining: IndexError: list index out of range

Dear all,
I have been trying to use the command line to train Cellpose. However, I kept getting "IndeError: list index out of range". In addition, the flow images look weird too....
Here are my test images https://github.com/chenzhao2016/TestImages/blob/main/Test_images.rar
Here is the error message

I would appreciate it if someone could give me some suggestions. Thank you very much!

it looks like your _masks images might be empty (the diameter is nan). did you save the training masks as indices 0=no cell, 1=cell 1, 2=cell 2,... etc

We changed the mask images to instance segmentation and indexed each mask by grayscale.

I'm also having this issue... I'm on mac. Made the mask, saved NPY file... followed exactly online tutorial on here

What I figured out is if all your training images have fewer than 5 cells, then you'll produce this error. If you have at least 1 image with more than 5 cells segmented then its fine...


[bug] Fix Unnecessary re-rendering of Dashboard

Notes for Reviewers
It was occurring due to unwanted props being passed to dashboard component. Since these redux stats where being updated as a result of side-effects of other components, hence the dashboard component was re-rendering due to change in state of these inbound props.
This PR removes the unnecessary props and redux state vars.
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

Thanks @theBeginner86 , i was unaware of this section of props so couldn't get to the cause


Configurable Maximum Steam Output

I'd like to beable to configure in my pack the maximum steam output a reactor can make. I'd like to beable to configure above or below the current 50 B/t max.

I'm not clear exactly what you're requesting. Are you asking that the reactor be limited to a certain power output? Are you asking that the reactor only be allowed to pump out X amount of steam, regardless of how much it produces? Are you asking that, regardless of power output, the reactor be capped to produce X steam per tick?
In the first case, you can limit a reactor's size in the config and use the powerProductionMultiplier config setting to adjust the maximum theoretically-attainable power output.
The other two cases don't seem to make much sense. I'll need more explanation.

If i get it right, the variable you say is the thing what he needs, if it affects steam production.

On this line https://github.com/erogenousbeef/BigReactors/blob/d54085cd53d0604ae216d7e31b8c2fb9e1184b13/src/main/java/erogenousbeef/bigreactors/common/multiblock/MultiblockReactor.java#L1049
You've hardcoded the maximum steam output of a reactor to 50buckets i was wondering if this could be made a configurable value.

Ah, you mean the size of the steam container. Yes, that's pretty easy. I'll get it done for the next patch release.


build: upgrade dependencies further

From https://github.com/moov-io/paygate/pull/372

Codecov Report

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

@@           Coverage Diff           @@
##           master     #520   +/-   ##
=======================================
  Coverage   53.47%   53.47%
=======================================
  Files          97       97
  Lines        3680     3680
=======================================
  Hits         1968     1968
  Misses       1363     1363
  Partials      349      349

Continue to review full report at Codecov.

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


Plans for eventually using published pubgrub?

I noticed that pubgrub is one of two dependencies that uv currently needs to fork (https://github.com/astral-sh/pubgrub), with async_zip being the other. I also noticed that there seems to be ongoing work toward minimizing the changes in the fork, e.g. https://github.com/astral-sh/uv/pull/3713, https://github.com/astral-sh/pubgrub/commit/eeb63dd5fe524d4e3633d789da81e1d73dd574e5.
For the benefit of distribution packagers and others who need to treat git dependencies specially, could you please comment briefly on whether you have a plan for eventually using the published pubgrub crate?

We unfortunately require api-breaking changes to pubgrub and access to some of the internals that will not be exposed in the published version. While we have been and will be spending significant effort into upstreaming our changes (https://github.com/pubgrub-rs/pubgrub/pulls?q=is%3Apr+author%3Akonstin), we unfortunately do not have plans to switch to a crates.io version of pubgrub.

Thanks! Even if the fork will be maintained indefinitely, the statement and explanation is valuable.

We could vendor it within uv itself if it makes a big difference externally. We generally find it easier to maintain the fork in a separate repository though.

We could vendor it within uv itself if it makes a big difference externally. We generally find it easier to maintain the fork in a separate repository though.

In Fedora, we handle git crate dependencies by adding an additional source archive, like https://github.com/astral-sh/pubgrub/archive/0e684a874c9fb8f74738cd8875524c80e3d4820b/pubgrub-0e684a874c9fb8f74738cd8875524c80e3d4820b.tar.gz, to the source RPM. Then we extract it and patch the git crate dependency into a path one.
This is perhaps a little more effort than dealing with an “in-tree” vendored/bundled crate, but not much more, as long as the number of such crates is small. Plus, this method has the advantage that it’s explicit, with no chance of missing vendored/bundled code that we needed to document and justify to satisfy our packaging guidelines. So I can’t speak for other distributions, but I’m perfectly happy with the way these two forked crates are managed now.


Update GitHub Actions to versions using Node 20

Github will soon remove Node 16 from the Github Action; see blog post. In this PR, I want to update the actions used in this workflow to the latest version in order to get rid of the warning message on action launches.

@mattxwang I just saw that an identical PR #46 is present. Feel free to close one and merge the other.

Thanks for this @lamasfoker. @mattxwang I saw your reply a while back but have just been too smashed at work to get around to it. Thanks for merging it in and closing off my PR, sorry for the delay!

Thanks for this @lamasfoker. @mattxwang I saw your reply a while back but have just been too smashed at work to get around to it. Thanks for merging it in and closing off my PR, sorry for the delay!

No worries at all @IbrahimAH - I was also swamped with work, hence my late reply 😓 thank you for opening the PR that kickstarted this process!


Edit spelling of GitHub

Corrects a very common error when referring to GitHub. The middle 'H' should be in caps.

Great!


Allow a user to build UI's from the filters

While the primary entry point to perform the query is through an advanced search string, we would not expect that to be the primary way an end-user would interact with the library.
We would like to provide a means for a developer to generate various UI components that would be the primary way for an end-user to interact with the UI. These UI components would generate the appropriate search input. A UI component may be a dropdown menu, a segmented control, or another means by which an end-user would choose from an existing set of options to perform their query.
We should not be prescriptive in how someone builds these UI components. We should merely provide a means for someone to generate a UI component using the data the library knows about.
For example, if a user has defined an expression filter for filtering on a User's Role using the expressions is:admin or is:member, we should allow them to generate a menu that has options for all known roles.
This requires a few things

Naming/identifying the filter. Currently, we add all filters to an array in the Definition. We should likely change this array to a hash with identifier keys, so we can reference the filters later.
A way to enumerate all options. For the Role example, a user would need to define the list of options that should be presented. This would involve specifying a Name (Admin), the expression that should be added to the query string (is:admin), and any other data that may be needed to render the menu item (icon, alt text, etc).
A way to generate a new query string from the existing string and the selected menu option. For example, if the current query is is:published author:eebs and a user clicks a menu option in a Category menu titled "Ruby", we would expect the query string to now be is:published author:eebs category:ruby.

I would not expect the work to close this issue to be done all at once. There are many moving parts here that should be done iteratively.

The first iteration of this was completed in https://github.com/thoughtbot/yuri-ita/pull/37


can't access grap

I had start ./bin/kafka-monitor-start.sh config/kafka-monitor.properties and can using
curl localhost:8778/jolokia/read/kmf.services:type=produce-service,name=/produce-availability-avg
{"request":{"mbean":"kmf.services:name=,type=produce-service","attribute":"produce-availability-avg","type":"read"},"value":{"kmf.services:name=single-cluster-monitor,type=produce-service":{"produce-availability-avg":1.0}},"timestamp":1592473146,"status":200}.
but can't access to graphic. I'm checked port 8000 it's not exist

plz help me. thanks

Hello @letran3691 -- please review https://github.com/linkedin/kafka-monitor/pull/269.

Hello @letran3691 -- please review #269.
thanks you have reply.
I had config with https://github.com/linkedin/kafka-monitor/pull/269/files/8bb07357ee0a5a5190bbab37bc47708567cb5a0f
after compile ./gradlew jar then output error  "not override abstract method awaitShutdown(long,TimeUnit) in Service"

I don't know more code. Below is files I had edit and create before compile. You can see help me plz. Thanks!
build.gradle
https://paste.ofcode.org/VpEVpXrhmxxHE2gBF9kSjj
config/xinfra-monitor.properties
https://paste.ofcode.org/Z2rciSWNexBJnDzAN5RgKE
src/main/java/com/linkedin/kmf/services/JettyService.java
https://paste.ofcode.org/iymrvGAatRiHPdfetXMnnw
src/main/java/com/linkedin/kmf/services/JettyServiceFactory.java
https://paste.ofcode.org/wDbNn6msGX6QQw2SVFBcSb
src/main/java/com/linkedin/kmf/services/JolokiaService.java
https://paste.ofcode.org/eK82WE2X3wqhZVRnsfNiiG

Thanks for the files and screenshot.
I have made the changes in that branch (PR) and have conducted some merge conflict resolutions.
Please try again on the clean copy of that branch once again.
On my end I see:
 ~/kafka-monitor   andchoi_ultimateSoftwareInquiries ?  ./gradlew jar                                                                                 SIGINT(2) ↵  10851  22:22:05

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/4.10.3/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 1s
3 actionable tasks: 1 executed, 2 up-to-date

Thanks for the files and screenshot.
I have made the changes in that branch (PR) and have conducted some merge conflict resolutions.
Please try again on the clean copy of that branch once again.
On my end I see:
 ~/kafka-monitor  andchoi_ultimateSoftwareInquiries ? ./gradlew jar

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/4.10.3/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 1s
3 actionable tasks: 1 executed, 2 up-to-date

thanks for feedback.
I have clone brand andchoi_ultimateSoftwareInquiries and complie success, But when run ./bin/xinfra-monitor-start.sh config/xinfra-monitor.properties or ./bin/single-cluster-monitor.sh --topic test --broker-list localhost:9092 --zookeeper localhost:2181 .
I seen port 8778, but I don't see port 8000. I don't know why so.
You can help  this troubleshooting thanks

Thanks for the update.
Looks like the index HTML file in that PR was set to something else.
Please try again with the newest commit in that PR and run the application service once you have your brokers running.
After that, try the following URL in your browser.
http://localhost:8000/index.html

I have using PR "git clone -b andchoi_conusmerhotfix https://github.com/linkedin/kafka-monitor.git". After build success and run "./bin/single-cluster-monitor.sh --topic test --broker-list  <IP_ADDRESS>:9092 --zookeeper localhost:2181". I don't have see port 8000. I don't why, you can speak particular PR  had support graphics. Thanks

Hello Letran, instead of the andchoi_conusmerhotfix PR's branch (andchoi_ultimateSoftwareInquiries), could you try #269 and attempt running clean build as well as ./gradlew jar?


Cannot find base config file<EMAIL_ADDRESS>[tsconfig.json]

"dependencies": {
"vue": "^3.4.21"
},
"devDependencies": {
"@vitejs/plugin-vue": "^5.0.4",
"@vue/tsconfig": "^0.5.1",
"typescript": "^5.2.2",
"vite": "^5.2.0",
"vue-tsc": "^2.0.6"
}
help me, thanks!

Can't reproduce. If you still encounter this issue, please open a new issue with a link to a reproducible project.


[HOLD] Add collapsible "see more/fewer options" link to work type modal

Fixes #716
Why was this change made?
Includes:

Pull surrounding <div>s into the work type modal component to reduce copypasta
Allow work types to contain any subtypes including "more" types
Add WorkType.more_types to prevent collaborators from needing to know the internal structure of the class; prefer message-sending

Use the message-sending approach from within the work type modal ERB

Add optional param to WorkType.subtypes_for that allows adding "more" types to subtypes which are now valid subtypes
Add feature spec to test "See more options" link in work type modal
Adjust tests to reflect changes to work type and subtype validation

NOTE: Until #972 is done, the "more" types selected for a work type WILL NOT be visible in the work form. I'm taking that issue next.
SCREENCAST

How was this change tested?
CI, local browser
Which documentation and/or configurations were updated?
None

@sul-dlss/infrastructure-team I welcome reviews on this PR as is, but I also need to do some work to incorporate the changes from #905 into this PR.

@peetucket I think the latest commit in this branch subsumes all the work from #905 and extra bits that I found to be missing or misaligned when I wrote some new tests. Have a look-see? Or we can pair next week.


Why can't I write withInterface?

Hello! Firstly, thank you for this package - it's wonderful. Secondly, I've hit a wall, and I think it's because of asynchronous impurities. I wanted to write this little function, withInterface, to tidy up some of my logic:
withInterface :: forall a . (Interface -> Program a) -> Program a
withInterface f = do
  interface <- createConsoleInterface noCompletion
  result    <- f interface

  close interface
  pure result

The problem is that writing something like withInterface $ \interface -> prompt interface will show a prompt and immediately close it. Presumably, this is because close interface runs before I have time to type anything into the prompt. I'm pretty new to PureScript, but is this something that could be solved with the help of Aff?
Sorry, I think/hope I answered my own question, but I'm hoping I can work to build something to allow this :) Thanks!

You have indeed answered your own question; Aff is the asynchronous version of Eff. :)
If you were using Aff then f could await some response before returning the result, with no need to deal with explicit callbacks or anything like that.

Well, I hope I haven't jumped the gun, but I've written a little module that puts the focus on readline.question - is this worthwhile? Or am I being far too eager? https://github.com/i-am-tom/purescript-node-readline-question

Following @paf31's tweet, I've published the package! However, let me know if I'm missing some obvious way to recreate this behaviour and I'll remove it from Pursuit (assuming I can work out how!) - no point in me duplicating work :)
If, on the other hand, it does have some use, I would love to talk about transforming it into a PR for this library? Sorry, this thread's title is probably not so applicable anymore!

Closing as there is an Aff variant of this now.


Consider converting server to a language server

I should be using a standard protocol (language server protocol) for discovering and running tasks.
A language server for java exists here: https://github.com/redhat-developer/vscode-java I can build a custom eclipse.jdt.ls extension to integrate with the java language server, but that limits this extension to just java, where gradle supports many other languages.
No "gradle language server" exists, as far as I can tell, but that seems the right way to go.

Have decided the language server is outside of the scope of running build tasks. This protocol seems more appropriate: https://build-server-protocol.github.io/docs/specification.html but it's not a standard yet. Going with grpc for now.


[csharp] Current Min version are not supported for Xcode_14.1 fixing #32558

jtattermusch try if this fixes it. 14.3 do not support 10.109 anymore.
Lets see if this could fix https://github.com/grpc/grpc/issues/32558#issuecomment-1485027373
How do I get the tests to run?

@yashykt Maybe you can help, how do I get the tests to run?

ah, I didnt pay attention to this since this was marked as a draft


Sqlite Collation Support

Adds a method create_collation to SqliteConnection.
Adds a unit test confirming the collation works as expected.

Per the comments in #430, this adds the method directly on SqliteConnection.
The implementation is essentially a direct copy of what rusqlite did. I put the method in a separate module, collation, that seemed to be similar to the approach taken with other methods on SqliteConnection.
I had some trouble with the unit test, I'm not sure the why I coded it is best.

@mehcode @abonander I think this is ready to be merged. The CI failures appear to come from existing code in the master branch.

👍  Thanks for seeing this through

Is it possible to create a collation for a Sqlite Pool?  Or only a single connection?

Is it possible to create a collation for a Sqlite Pool?  Or only a single connection?

In the version we just released this week, you can configure collations on SqliteConnectOptions which when used with a pool will create those collations on all connections created by the pool: https://docs.rs/sqlx/latest/sqlx/sqlite/struct.SqliteConnectOptions.html#method.collation

In the version we just released this week, you can configure collations on SqliteConnectOptions which when used with a pool will create those collations on all connections created by the pool: https://docs.rs/sqlx/latest/sqlx/sqlite/struct.SqliteConnectOptions.html#method.collation

Perfect! Thanks so much!

Perfect! Thanks so much!


Enter-PSSession fails from Docker image

Steps to reproduce
From a RHEL server running docker, pull down latest release by running:
docker run -it microsoft/powershell
From PS command prompt, connect to a Windows 2012 R2 server by running:
PS /> $creds = Get-Credential
PS /> Enter-PSSession -ComputerName {Win2012-R2 Hostname} -Authentication Basic -Credential $creds
Expected behavior
PS connects to server
Actual behavior
Enter-PSSession : Connecting to remote server {Win2012-R2 Hostname} failed with the following error message : MI_RESULT_ACCESS_DENIED For more information, see the about_Remote_Troubleshooting Help topic.
At line:1 char:1

Enter-PSSession -ComputerName {Win2012-R2 Hostname} -Authentica ...

CategoryInfo          : InvalidArgument: ({Win2012-R2 Hostname}:String) [Enter-PSSession], PSRemotingTransportException
FullyQualifiedErrorId : CreateRemoteRunspaceFailed

Environment data

PS /> $psversiontable
Name                           Value

PSVersion                      6.0.3
PSEdition                      Core
GitCommitId                    v6.0.3
OS                             Linux 3.10.0-693.21.1.el7.x86_64 #1 SMP Fri Feb 23 18:54:16 UTC 2018
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@Stlouistechy
Question: Did you setup Winrm linux on RHEL Docker?
I know it's kind-of hard to setup but it's possible.  The best and fast route it to setup ssh on the Windows Server.  Docker IT/PowerShell image will not have the Winrm configured.
:)

@MaximoTrinidad
Thanks Maximo. I just tested this on my Container by installing pywinrm (pip install pywinrm) and running the steps again. Unfortunately no luck, I receive the same error.

@Stlouistechy
OpenSSH can be install in Windows so you can use SSH protocol for remoting.
By the way, the AD Module will be available (if is not already) for PSCore. Then again, all components to support AD Authentication need to be in place on the Linux system. I know the way to install these components does varies on different Linux distributions.
:)

hi Stlouistechy:
docker run -it microsoft / powershell ,then you get a linux powershell.
the only way is OVER SSHD from a docker linux powershell to windows,This is the by design.
so this issue is a  question

@chuanjiao10 Thanks. Is there a timeline when PSSession and ActiveDirectory modules may be available to PSCore on Linux?

Now you can.
the only way is OVER SSHD from a docker linux powershell to windows.

@Stlouistechy The microsoft/powershell image is based on Ubuntu, which does not bundle /etc/services, so you are likely being bit by this issue: https://github.com/Microsoft/omi/issues/623. The workaround for now is to build your own Docker image based on the PowerShell image that includes a command like:
RUN echo 'http            80/tcp          www www-http' > /etc/services

@Stlouistechy The microsoft/powershell image is based on Ubuntu, which does not bundle /etc/services, so you are likely being bit by this issue: microsoft/omi#623. The workaround for now is to build your own Docker image based on the PowerShell image that includes a command like:
RUN echo 'http 80/tcp www www-http' > /etc/services

...or apt-get install netbase
I just lost two hours of my life because of this.

Thanks @jimdigriz an apt update && apt install netbase did the trick for me as well.
However I can't get around saying how poor this looks to me that Microsoft is unable / unwilling to fix errors like this in their official images that are around for two years.
I only stumbled upon this issue because it seems not to be possible to do PowerShell remoting from Mac OS X (even though it is an officially supported platform) due to an incorrect hard linked dependency to an outdated OpenSSL library.
Searching the web tells that even after 3 years still a lot of users (like me) are wasting their time with no workaround or even a solution provided by Microsoft.
So I thought with their docker image at least it should work right out of the box. Double fail! What a shame!
Sorry but I'm just frustrated after wasting nearly three hours.

@apiening Agreed. I think it took me 1-2 days of work last year digging into the OMI library internals to identify the issue - and then when I did, the fix I proposed was promptly rejected. 🤷♂ 😖

It also fails for a non-docker image for me with netbase installed on Ubuntu 20.04. Don't kid yourself, the total lack of support of Microsoft for this and similar issues just shows clearly how committed they are to providing true cross platform solutions.
Kerberos support should work out of the box, it's trivial to make that work if it works on Windows. I would expect more from a so called commitment to open source and other platforms.

apt-get install netbase

Life savior!!

Please take a look into this threat - https://github.com/jborean93/omi/issues/29
Looks like the limitation comes from the libraries which ship with different version of Windows. The workaround is to set the CbtHardeningLevel to None so Windows doesn't try to validate the value at all.

apt-get install netbase

Life savior!!

I had a working image in .net core 3.1 where I could establish remote connections via powershell from linux containers to windows machines, but the same code did not succeed when upgrading to .net 5.
Simply adding netbase package installation to my container dockerfile did the job, I am able to connect from linux containers to windows machines via powershell in .net 5 using sdk 5.0 docker hub image.


Unable to submit transactions

A POST call to https://horizon.stellar.org/transactions with following tx (generated by official stellar account viewer):
tx=AAAAAIqVUeWsSCEJTGbts5UTv2%2BcgkvfHzQg6rlJG%2BAMsxJrAAAAZAAN1vkAAAACAAAAAAAAAAIAAAAAAG5xsgAAAAEAAAAAAAAAAQAAAACM2yhJ2o8VvZubPN%2BPdu4UNJV%2BBrArlu1kZ55D3mEZnwAAAAAAAASMJzlQAAAAAAAAAAABDLMSawAAAEDUe1HpEz1jjoEpG1SUZFJPq%2Fiyowrh9aL%2FRoyFXkug29BYVzguBPo7C5ibio3iFcvWVj8USLAt2FBmHj%2BuRwoD

returns the following error:
{
  "type": "https://stellar.org/horizon-errors/server_error",
  "title": "Internal Server Error",
  "status": 500,
  "detail": "An error occurred while processing this request.  This is usually due to a bug within the server software.  Trying this request again may succeed if the bug is transient, otherwise please report this issue to the issue tracker at: https://github.com/stellar/horizon/issues. Please include this response in your issue.",
  "instance": "horizon-live-001/YWy4c7JWVN-061296"
}

that tx envelope doesn't seem valid
check it out here: https://stellar.github.io/xdr-viewer/
was there an error pasting?

That's in form url encoded format. This is the parsed format:
AAAAAIqVUeWsSCEJTGbts5UTv2+cgkvfHzQg6rlJG+AMsxJrAAAAZAAN1vkAAAACAAAAAAAAAAIAAAAAAG5xsgAAAAEAAAAAAAAAAQAAAACM2yhJ2o8VvZubPN+Pdu4UNJV+BrArlu1kZ55D3mEZnwAAAAAAAASMJzlQAAAAAAAAAAABDLMSawAAAEDUe1HpEz1jjoEpG1SUZFJPq/iyowrh9aL/RoyFXkug29BYVzguBPo7C5ibio3iFcvWVj8USLAt2FBmHj+uRwoD

Ah thanks our live horizon was hung. It should be working now

It works


feat: wire-up lifetime rewards

Description
Note, since rewards haven't started yet, which would be the main issue as far as displaying lifetime rewards go - this means that for now, we're fetching from one month in the past instead of the starting date of rewards (which is in the future).
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

relates to https://github.com/shapeshift/web/issues/6948

Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

"Lifetime Rewards" properly loads, and eventually displays "0 RUNE"

Engineering

☝🏽

Operations

☝🏽

Screenshots (if applicable)
https://jam.dev/c/af4a0da0-d152-4c52-a50a-c5c798d08b14

Looks sane to me, the query is actually loading as expected
How do we plan to think about reverting this when the first rewards epoch hits?

https://github.com/shapeshift/web/issues/7224 here we go! 🙏🏽