SQLite: Deletion is not cascaded to untracked entities

Observation
Even though foreign keys are defined adequately in code and database, deletion only works correctly for tracked entities, as well for one-to-zero/one relationships as for one-to-many relationships.
In case of untracked entities only the principal entities are deleted and no cascading in performed by the database, though documented differently at https://docs.microsoft.com/en-us/ef/core/saving/cascade-delete#cascading-to-untracked-entities.
EF database logging shows that PRAGMA foreign_keys=ON; is executed, nevertheless the contraints are ignored. Executing deletion manually (e.g. with DB Browser for SQLite), deletion is cascaded correctly.
Further technical details
EF Core version: 1.1.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite (1.1.0)
Operating system: Windows 10
IDE: Visual Studio 2015 (Update 3)

@volleyms I have been unable to reproduce this. Here is a console application showing cascade deletes working. Could you look at this and see what might need to be changed to reproduce what you are seeing?
public class Blog
{
    public int Id { get; set; }
    public string Title { get; set; }

    public ICollection<Post> Posts { get; set; }
}

public class Post
{
    public int Id { get; set; }
    public string Title { get; set; }

    public int BlogId { get; set; }
    public Blog Blog { get; set; }
}

public class BlogContext : DbContext
{
    public DbSet<Blog> Blogs { get; set; }
    public DbSet<Post> Posts { get; set; }

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        optionsBuilder.UseSqlite(@"Data Source=.db");
    }
}

public class Program
{
    private static void Main()
    {
        using (var context = new BlogContext())
        {
            context.Database.EnsureDeleted();
            context.Database.EnsureCreated();

            context.Add(new Blog
            {
                Posts = new List<Post> {  new Post { Title = "Old" } }
            });

            context.SaveChanges();
        }

        using (var context = new BlogContext())
        {
            Console.WriteLine(context.Posts.Count());
        }

        using (var context = new BlogContext())
        {
            context.Remove(context.Blogs.Single());

            context.SaveChanges();
        }

        using (var context = new BlogContext())
        {
            Console.WriteLine(context.Posts.Count());
        }
    }
}

Hello Arthur,
thanks for the quick reply.
Please find attached a project which shows the effect after the first
deletion (the first deletion works correctly).
In my productive code I had the problem even with
"database.RemoveRange(database.Users)".
Regards
2016-12-20 18:59 GMT+01:00 Arthur Vickers<EMAIL_ADDRESS>
@volleyms https://github.com/volleyms I have been unable to reproduce
this. Here is a console application showing cascade deletes working. Could
you look at this and see what might need to be changed to reproduce what
you are seeing?
public class Blog
{
public int Id { get; set; }
public string Title { get; set; }
public ICollection<Post> Posts { get; set; }

}
public class Post
{
public int Id { get; set; }
public string Title { get; set; }
public int BlogId { get; set; }
public Blog Blog { get; set; }

}
public class BlogContext : DbContext
{
public DbSet Blogs { get; set; }
public DbSet Posts { get; set; }
protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
{
    optionsBuilder.UseSqlite(@"Data Source=.db");
}

}
public class Program
{
private static void Main()
{
using (var context = new BlogContext())
{
context.Database.EnsureDeleted();
context.Database.EnsureCreated();
        context.Add(new Blog
        {
            Posts = new List<Post> {  new Post { Title = "Old" } }
        });

        context.SaveChanges();
    }

    using (var context = new BlogContext())
    {
        Console.WriteLine(context.Posts.Count());
    }

    using (var context = new BlogContext())
    {
        context.Remove(context.Blogs.Single());

        context.SaveChanges();
    }

    using (var context = new BlogContext())
    {
        Console.WriteLine(context.Posts.Count());
    }
}

}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/EntityFramework/issues/7278#issuecomment-268312218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXhzNmiGf3Cejnn_E8j45nm_4OtfIZDPks5rKBeLgaJpZM4LRviG
.

@volleyms The relationship between User and Setting is optional because the FK property UserId is nullable. By convention, EF will only configure cascade deletes for required relationships. You can make the relationship required by adding .IsRequired() to its configuration in OnModelCreating.

Hello Arthur,
I cannot agree:

UserId is defined as (part of) the primary key in both tables. The schema which is created by Add-Migration defines all primary key columns as non-nullable.

Even if all properties defined as keys are additionally defined as required, just like you suggested, the schema created by Add-Migration is identical, and the program behavior does not change.

The DeleteBehavior.OnCascade is defined in the schema as well, so it must be performed.

Finally, the behavior is buggy all the time: The first deletion is cascaded, the following deletions are not cascaded which is not OK as such, and leaves the database in an inconsistent/illegal state.

@volleyms I will investigate further.

Note for triage: I was able to reproduce this and it appears to be an issue with either SQLite or the provider. The database is created with the cascade delete and for the first entity deleted the cascade is applied, but for the next four it is not applied. The exact same code run against SQL Server results in all cascade deletes happening correctly.

Closing this as a dupe of #6968, which is fixed. This probably requires some explanation. What is happening here is that the connection count was being thrown off by the extra call to Close. This in turn meant that after the first delete all future deletes were being done without the pragma to turn on FKs, which meant that SQLite did not perform cascades. Also see #7527.


Cloning a selectize input.

I'm in the process of swapping out select2 with selectize on an application and have run in to a slight hiccup that I'm not sure how to handle.
I've written some javascript that allows me to easily add new rows of one or more inputs. So for example, I might have a row that contains first name, last name, and user type with a button on the far end that allows users to add multiple people. With select2, I simply cloned the row, discarded the initialized select2 instance/content, and then reinitialized the select element with select2. Since selectize doesn't keep the original select dom element in tact, this solution is no longer possible. Furthermore, I don't see any exposed methods that allow you to grab all the options or to clone an existing selectized input, so I'm not sure of the best way to go about getting this functionality back in place.
I know that selectize provides tagging support for the multiple selection use case, but that's not always desirable. I'm sure I can hack things together, but I was hoping that somebody might suggest a clean solution or a small addition to the API that would make this particular use case easier.
Thanks!

The method shown doesnt work entirely if the original combobox was already selectized which will is the case for me. What this method does is that it leaves the original code as is intact in the cloned (new) element. I am not able to figure out how to get rid of that. any suggestions


Bank-Account - Unable to use the log into your bank option

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to /bank-account/new
Select the log in to your bank account option

Expected Result:
Modal to select the bank account and log in should be displayed.
Actual Result:
Blank modal is displayed and an error can be seen in console.
Workaround:
None.
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.0.94-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Issue is not reproducible in production.

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation: https://github.com/Expensify/App/issues/5104#issuecomment-914650464
View all open jobs on GitHub

Triggered auto assignment to @MonilBhavsar (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

It's not reproducible on dev, but can see on staging for same version. Looking into it

Looks like we updated CSP and may be that caused this issue, Asked in this slack thread for confirmation from infra team

Confirmed that this is fixed on staging, checking it off on the checklist.


Select apartment type for pywinrt

When I import winrt, an MTA is immediately initialized. This has two problems:

when I import on a second thread, this code is not executed and the MTA is not initialized
if I want an STA I have no way to create one

It should be possible to explicitly initialize a single-threaded apartment, and if I have not, then a multi-threaded apartment should be automatically initialized on first use.

The Python tool is no longer under active development, but feel free to contribute the feature you're after.

Please reopen this issue if Python/WinRT is now under active development.


Remove misleading version metadata

The metadata in dependency specification was misleading because the version was not guaranteed to be the same anyway this was correctly linted but nobody so far cared to fix it. This change fixes it and adds a hint how to get the real version since some people seem still confused about how these things work.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.842%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
19651

Relevant Lines:
23721

💛  - Coveralls

Lol. concept ACK. Can you tone down the language, and point out that the "real version" is the "full version including a suffix indicating which version of Bitcoin Core is linked"? That part is the nonobvious part.

Can you tone down the language

What specifically are you referring to?

Can you tone down the language

What specifically are you referring to?

LOL, classic anglo vs slav misunderstanding, he means, I believe, this bit "... since some people seem still confused about how these things work." - which having talked with you in person before @Kixunil just made me laugh :)

Yes, "since some people still seem confused" comes off as pretty aggressive (what people? the ones who have explicitly complained about and ignored the warning?), as does "as you would normally with any other dependency" (this is the only dependency in this crate where cargo disregards part of the version, and anyway cargo tree's output is woefully incomplete and the actual way to view dependencies is to read lockfiles).
Tobin's suggested text sounds good except for the run-on sentence. His , should be a ..

It's a note justifying existence of seemingly unneeded comment. :man_shrugging:

what people?

Various people I've seen don't really know how Cargo.toml vs Cargo.lock work. This manifests by things like adding version metadata to Cargo.toml or pinning dependencies in Cargo.toml.

this is the only dependency in this crate where cargo disregards part of the version

Because it's not really a part of version, it's just weirdly placed documentation. :) But that's not the point anyway, for any crate, if you need to accurately know which is used you have to read Cargo.lock, not Cargo.toml.

cargo tree's output is woefully incomplete

It's not, you just need to pass in the correct flags.

Is this better?

It's not, you just need to pass in the correct flags.

Lol. Beyond --all-targets --all-features? Why can't cargo commands just do the right thing without increasingly arcane invocations?

Is this better?

Yep, thanks!

Beyond --all-targets --all-features?

I believe those should be enough but I haven't looked into it for a while. Funnily enough, I have the opposite problem: whenever I just want to check the tree it downloads all kind of crap that slows me down.

Various people I've seen don't really know how Cargo.toml vs Cargo.lock work. This manifests by things like adding version metadata to Cargo.toml or pinning dependencies in Cargo.toml.

I agree with this PR, although I think the justification being that people don't agree on the difference between .toml and .lock to be a weak argument.

people don't agree on the difference between .toml and .lock

They may not agree with them and in that case they should complain at Rust, not turn it into some frankentoml. The reality is they work how they work and they have to adapt to it. Pinning in toml is absolutely unacceptable to me because it destroys the ability to update the crates so I will always be against it here. Thus those people will have to adapt or kick me out of contributing here.

Pinning in toml is absolutely unacceptable to me because it destroys the ability to update the crates so I will always be against it here.

Is something being pinned here?  Why does it destroy the ability to update the crate?

Is something being pinned here?

No

Why does it destroy the ability to update the crate?

Because that's how pinning works

@Kixunil

No

Then I don't think it should be mentioned in this PR

Because that's how pinning works

Right.  We pin deps for MSRV in the lock file.  It's not approriate to pin the deps in the toml file since dep pinning is only neeeded for MSRV.  Maybe that's what you wanted to point out but it seems besides the point here.

It's on topic, what's your problem?

I just felt like you where firing shots with your comment:

Various people I've seen don't really know how Cargo.toml vs Cargo.lock work. This manifests by things like adding version metadata to Cargo.toml or pinning dependencies in Cargo.toml.

Anyway, no problems.  Let's just drop it and move on.

FTR it's not wrong to be confused about something but it's wrong to not admit it or refuse being corrected.

FTR it's not wrong to be confused about something but it's wrong to not admit it or refuse being corrected.

I don't understand where this is coming from. Can you elaborate?  If there's something I can work on improving I'd like to know.

I mean if you though I'm "firing shots" then you must've assumed I meant "confused" as offensive while I meant is as neutral.

hmm ok.  Well your statement saying "it's wrong to not admit it or refuse being corrected" in response to me seemed like you where trying to tell me something, otherwise why say it?

It was about those confused people. I was explaining why it's neutral.

Maybe it's just your avatar that made me think it was offensive and that shots where fired xD

not saying your avatar is offensive, it just doesn't look defensive either.


HuCard startup code

Add a minimal HuCard startup code. This means simple irq vectors.

[x] pceas
[ ] ca65

Currently working on the pceas version.

Add a flag to switch between standard and expert mode.
In standard mode, everything is setup to have a ready to use env (bat already set, font loaded, default irq vectors, etc...).
In expert mode, you'll have to write the startup code and set everything up.

Thanks a lot for your project! I barely understand what it is about...
I guess you are creating an SDK based on CC65 for the PCE.
Sorry to bother you.
I have not been able to figure out how to use CC65 for the PCE target other than building 8k roms.
I would like to understand how you do it but I cannot find any documentation.
I would like to use for my massively multi-system game:
https://github.com/Fabrizio-Caruso/CROSS-CHASE
Obviously I am only using ANSI C.


Host Boxes on Vagrant Cloud

Tracking this in #249

Had a successful build with the Vagrant hosted boxes: https://circleci.com/gh/clong/DetectionLab/224#artifacts/containers/0
Marking closed.


AppCode is available for Windows?

Appcode's system requirements:

macOS 10.12 or higher
Xcode 9.2-10
4 GB RAM recommended

I remember it used to be available! I'll remove it now. Thanks 💯


python311Packages.openllm-core: 0.3.4 -> 0.3.9; disable time-sensitive checks

Description of changes
Diff: https://github.com/bentoml/OpenLLM/compare/refs/tags/v0.3.4...v0.3.9
Changelog: https://github.com/bentoml/OpenLLM/blob/refs/tags/v0.3.9/CHANGELOG.md

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

[x] Tested, as applicable:

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

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

Result of nixpkgs-review pr 262692 run on x86_64-linux 1

12 packages built:

python310Packages.openllm
python310Packages.openllm-client
python310Packages.openllm-client.dist
python310Packages.openllm-core
python310Packages.openllm-core.dist
python310Packages.openllm.dist
python311Packages.openllm
python311Packages.openllm-client
python311Packages.openllm-client.dist
python311Packages.openllm-core
python311Packages.openllm-core.dist
python311Packages.openllm.dist


fix: send a single payload on no_route error

sg would broadcast 2 messages upon unroutable non-droppable requests. Removed the one in message_dispatcher.ex in favor of the more helpful payload sent by dispatch.ex.
Made it convert the opcode back to an integer, so the extra_data and be considered a copy of the sent payload rather than a plain dump of the internal object.
I also dropped the "Unroutable payload: " prefix, as it makes it harder to programatically analyze the error field, and know what's the actual problem, allowing to react accordingly

Thanks!


Redux Implementation

Hi, I really love your library it's superb but i am finding it hard to use, can you kindly give me some insights on how I would implement with redux. Thanks a bunch.

Hi @geekdaniels, thank you for the kind words.
I think you're talking about access to the toast store outside of the react tree. Whilst not technically supported from this lib, there has been some discussion about it on #40

Ref #142


Move instructional articles into the wiki and delete

I’m also moving a few “articles” into the wiki. They shouldn’t really be articles at all: they are “meta”.
Like:
https://github.com/MicrosoftDocs/microsoft-365-community/blob/master/Community/adding-content.md
into
https://github.com/MicrosoftDocs/microsoft-365-community/wiki/Adding-Content

Getting Started and Adding Content moved to the wiki and deleted from Community.


dbus is very slow/does not work

Describe the issue
dbus is very slow to start and any applications that use dbus (VS Code) seem to not work at all.
To Reproduce

Install an X server (I am using VcXsrv) on Windows 10.
Install visual-studio-code-bin from AUR (any app that uses X11/dbus should work as well?)
Open Arch WSL and run the following:   $ sudo mkdir /run/dbus -p
   $ sudo dbus-daemon --system  # this takes ~90 seconds to start on my machine
   $ DISPLAY=:0.0 code # It is normal for this command to exit right away,
   # check Task Manager/do `ps` to see process. It exits after ~5 minutes
   # on my machine never launching VS Code properly. It *should* spawn
   # ~5 `code` processes and start a window with VS Code.

Expected behavior
Starting dbus should be much faster. In the official Ubuntu WSL instance, it takes almost no time for it to start. Likewise, starting applications that use dbus should start and work. VS Code works fine in Ubuntu WSL and takes ~5 seconds to start on my machine. See https://github.com/Microsoft/WSL/issues/2293 for details about using VS Code in Ubuntu WSL.
Enviroment:

Windows build number: 10.0.17763.253
ArchWSL version: 19010200
Launcher version: 18122700

Additional context
Not sure what you need to help track down the issue, let me know if you need anything else.

I did the same, but I could not confirm the problem. . .

Are you able to actually VS Code to launch?
I still get the same issues with dbus when I try to run it on the system level (as above with sudo dbus-daemon --system), but I did see https://github.com/yuk7/ArchWSL/issues/73 which allows me to launch dbus in the user space and it run instantly, but VS Code still has the same problem where it never launches.

I reproduced this as well, with these commands:
   $ sudo mkdir /run/dbus -p
   $ sudo dbus-daemon --system

I confirm. I have release <IP_ADDRESS>. Tried with administrator command prompt.
       $ sudo mkdir /run/dbus -p
       $ sudo dbud-daemon --system

Took 1 minute after LXCORE.SYS failed.

Any update on this @yuk7? There are multiple repros in this thread, and currently this is preventing me from switching to ArchWSL.

Installed All from today and got same problem

Replacing libdbus with dbus-x11 may solve this problem.

I think this issue was fixed in newest in systemd-libs.

I got the same issue again

We can try to use dbus-x11aur.


Support a debug framework for TVM Runtime

A debug framework for TVM Runtime.
Refer this link for the user manual.
This PR supports the below features.

[x] Show fused graph summary
[x] Perform debug run and show node details(inputs/outputs)
[x] Run without debug
The below features I'm still  working on and will PR later
[ ] Call graph run-time n times from UI
[ ] Support check for NAN in data and break
[ ] Support check for INF in data and break
[ ] Support for step debug(debug node by node)
[ ] Support changing node input values and running a specific node
[ ] Support inject node input values from a dump file
[ ] Support dumping of node outputs to a file
[ ] Support comparison of node output with a dump output
[ ] Support profiler for performance debugging
[ ] Test framework for tvmdbg

Thanks for contributing to this. Because this is a big change to the framework, I would recommend starting an RFC issue first, so that various design aspects can be discussed among the contributors, before we start reviewing the PR.

One thing we want to avoid is to make sure that the image files are not stored in this repo, to minimize the size of the history

https://github.com/dmlc/tvm/issues/1315

this is closed for now, once the RFC discussion is finished and we have actionable items, we can reopen this PR to do the review


Rotating animation problem

Bug
Environment info
"react": "16.3.0",
"react-native": "0.54.4",
"react-native-svg": "6.4.1"

Library version: 6.4.1
Steps To Reproduce
After using the Svg custom control, the rotateZ animation is performed.  There are some problems in android. It is found that the edges of the controls displayed by redmi7 and honor 10 have a lot of jagged edges, and there should be some other mobile phones.
This is a screenshot after turning on the rotation animation.

What should I do to fix this problem, please help me
Reproducible sample code
This is TestView
render() {
        return (
            <View style={{
                flex: 1,
                backgroundColor: 'red',
            }}>
                <Animated.View style={{
                    width: circleRadius * 2,
                    height: circleRadius * 2,
                    alignItems: "center",
                    justifyContent: 'center',
                    marginLeft: 30,
                    marginTop: 30,
                    transform: [
                        {rotateZ: this.rotateCircleZ},
                        // {scale: }
                        ]
                }}>
                    <SvgCircle
                        svgStyle={{
                            width: circleRadius * 2,
                            height: circleRadius * 2,
                        }}
                        size={circleRadius * 2}
                        strokeWidth={10}
                        stroke='yellow'
                    />
                </Animated.View>
            </View>
        )
    }

SvgCircle
export class SvgCircle extends React.Component {

    render() {
        let {size, strokeWidth, startColor, endColor, gradient, svgStyle} = this.props;
        let width = size, height = size;
        if (svgStyle) {
            width = svgStyle.width;
            height = svgStyle.height;
        }
        let center = width / 2, radius = size / 2.0;
        if (strokeWidth) {
            radius -= strokeWidth / 2;
        }
        radius -= 1;
        let hasGradient = gradient && startColor && endColor;
        let strokeColor = hasGradient ? "url(#grad)" : (this.props.stroke || "none");
        let fill = this.props.fill || 'none';

        return (
            <AnimatedSvg
                width={width}
                height={height}
                style={svgStyle}
            >
                <Defs>
                    {hasGradient && (
                        <LinearGradient id="grad" x1={gradient.x1} y1={gradient.y1} x2={gradient.x2} y2={gradient.y2}>
                            <Stop offset="0" stopColor={startColor} stopOpacity="1"/>
                            <Stop offset="1" stopColor={endColor} stopOpacity="1"/>
                        </LinearGradient>)
                    }
                </Defs>

                <AnimatedCircle
                    {...this.props}
                    r={radius}
                    cx={center}
                    cy={center}
                    originX={center}
                    originY={center}
                    stroke={strokeColor}
                    fill={fill}
                    ref={(ref) => this.circle = ref}
                />
            </AnimatedSvg>)
    }
}

@msand  I tried to find out the problem, but nothing happened. If you don't have the above phone, you can try other comprehensive screens, thanks

@msand  I found this to be a problem with the resolution of the android phone. I tried to rotate the View, set the background color, and there would be a jagged problem.
 <View style={{
                flex: 1,
                backgroundColor: 'red',
            }}>
                <Animated.View
                    style={{
                        width: circleRadius * 2,
                        height: circleRadius * 2,
                        alignItems: "center",
                        justifyContent: 'center',
                        backgroundColor: 'blue',
                        marginLeft: 30,
                        marginTop: 30,
                        transform: [
                            {rotateZ: this.rotateCircleZ},
                        ]
                    }}/>
</View>

Hi there,
I am facing the same issue on Google Pixel 3a.
Did you find any workaround for this problem ?


Efficient directory import

The js-IPFS glob source should be split out of js-ipfs-utils and published for use with Helia.

Do we still need to do this considering globSource was used successfully in https://github.com/ipfs/helia/pull/171 ?

This is available in<EMAIL_ADDRESS>and later - https://github.com/ipfs/helia-unixfs/releases/tag/%40helia%2Funixfs-v1.4.0


sta: ensure heatmap and renderer is installed

Changes:

make sure the power density heatmap is available when invoked via gui::show

@gadfort This breaks my version of openroad when you don't build the GUI? I need to make sure that the register renderer is gated. How can we make sure both of our use cases work?

@QuantamHD I suspect the issue is that stub.cpp isn't handling things correctly.

What breakage are you seeing?

Yeah I'm seeing a segfault, because GUI::GUI::get() returns nullptr

Do you have a stacktrace for that?

I think you will need to be more specific about the segfault, I'm not able to reproduce it


[BUG] Error making request

data.http.available_versions: Reading...
data.http.available_versions: Still reading... [10s elapsed]
╷
│ Error: Error making request
│
│   with data.http.available_versions,
│   on main.tf line 18, in data "http" "available_versions":
│   18: data "http" "available_versions" {
│
│ timeout error: GET https://api.github.com/repos/rhythmictech/lambda-aws-secure-ssh-key/releases giving up
│ after 1 attempt(s): Get "https://api.github.com/repos/rhythmictech/lambda-aws-secure-ssh-key/releases":
│ read tcp <IP_ADDRESS>:64044-><IP_ADDRESS>:443: read: operation timed out

Fixed by https://github.com/rhythmictech/terraform-github-find-release-by-semver/pull/7


Tox and Travis

On Travis we test several environments and so we can do with Tox. However, Tox configuration hasn't been updated in a while.
I just read the following post:
http://jsatt.com/blog/using-tox-with-travis-ci-to-test-django-apps/
It is shown there how to specify your test matrix in Tox, and have Travis run Tox.
Perhaps it is a good idea if we eventually also go in this direction? Right now I don't really bother with testing locally, and just do a PR and see whether it passes or not, but of course that's not the right way.

I redid Travis and it now uses Nix. Testing currently only with Python 3.6.


Training mAP = 0.00 Help needed

As I'm not getting anywhere with this I will ask you guys for help again. I ran a model train for 50 epocs using the code bellow. When I try to evaluate I get 0 on all epocs. What am I doing wrong? My project gave a turn and changed concept so now I'm trying to detect bibnumber(marathon number plates) I got ~600 pictures of runners on each I annotated (made a square over the plate) using Imglabel as described. Upload to the folder bibnumber/train/img or annotation and bibnumber/validation/annotation. Installed tensorflow 1.13 and upgrade imageai (running on gcp notebook with gpu)
Everything seems to be normal this is my code for training:
`from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="bibnumber")
trainer.setTrainConfig(object_names_array=["bibnumber"], batch_size=4, num_experiments=50, train_from_pretrained_model="pretrained-model.h5")
trainer.trainModel()`
Got this output with 50 different loss for each epoc:
Using TensorFlow backend. /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint8 = np.dtype([("qint8", np.int8, 1)]) /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint8 = np.dtype([("quint8", np.uint8, 1)]) /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:528: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint16 = np.dtype([("qint16", np.int16, 1)]) /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:529: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint16 = np.dtype([("quint16", np.uint16, 1)]) /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:530: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint32 = np.dtype([("qint32", np.int32, 1)]) /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/dtypes.py:535: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   np_resource = np.dtype([("resource", np.ubyte, 1)]) Generating anchor boxes for training images and annotation... not well-formed (invalid token): line 1, column 0 Ignore this bad annotation: img/train/annotations/.DS_Store Average IOU for 9 anchors: 0.84 Anchor Boxes generated. Detection configuration saved in  img/json/detection_config.json Training on: 	['bibnumber'] Training with Batch Size:  4 Number of Experiments:  50 WARNING:tensorflow:From /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/framework/op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version. Instructions for updating: Colocations handled automatically by placer. WARNING:tensorflow:From /home/jupyter/.local/lib/python3.5/site-packages/imageai/Detection/Custom/yolo.py:24: to_float (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version. Instructions for updating: Use tf.cast instead. Training with transfer learning from pretrained Model /usr/local/lib/python3.5/dist-packages/keras/callbacks.py:1065: UserWarning: epsilonargument is deprecated and will be removed, usemin_delta instead.   warnings.warn('epsilon` argument is deprecated and '
WARNING:tensorflow:From /home/jupyter/.local/lib/python3.5/site-packages/tensorflow/python/ops/math_ops.py:3066: to_int32 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.
Epoch 1/50

1398s - loss: 24.7616 - yolo_layer_1_loss: 3.1134 - yolo_layer_2_loss: 6.0476 - yolo_layer_3_loss: 15.6006`

this is my code for the evaluation part
`from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="bibnumber")
trainer.evaluateModel(model_path="bibnumber/models", json_path="img/json/detection_config.json", iou_threshold=0.5, object_threshold=0.3, nms_threshold=0.5)`
got exactly the same output for every epoc in the list
`Starting Model evaluation....
/usr/local/lib/python3.5/dist-packages/keras/engine/saving.py:292: UserWarning: No training configuration found in save file: the model was not compiled. Compile it manually.
warnings.warn('No training configuration found in save file: '
Model File:  img/models/detection_model-ex-40--loss-7.47.h5
Using IoU :  0.5
Using Object Threshold :  0.3
Using Non-Maximum Suppression :  0.5
bibnumber: 0.0000
mAP: 0.0000
Model File:  img/models/detection_model-ex-18--loss-11.13.h5 `
I thought it could be some problem in the library (it's not) I decided to download the model with less loss, the json and try it:
`from imageai.Detection.Custom import CustomObjectDetection
detector = CustomObjectDetection()
detector.setModelTypeAsYOLOv3()
detector.setModelPath("detection_model-ex-50--loss-5.33.h5")
detector.setJsonPath("/detection_config.json")
detector.loadModel()
detections = detector.detectObjectsFromImage(input_image="Move_for_cancer7861.JPG", output_image_path="Move_for_cancer7861fix.JPG")
for detection in detections:
print(detection["name"], " : ", detection["percentage_probability"], " : ", detection["box_points"])`
All I get back it's a empty list. I really don't know what am I doing wrong can you please point me in the right direction? another thing... few very newbie question. What is the difference over a detection and a prediction? I saw there is a way to train a model from scratch like not use a pretrained model. It's this applicable for my scenario? is it better or worst some how?
Thank you guys. Sorry for flooding the post. I tried to fix the format of the post but looks like some characters in the code are messing with it

hi
Will try to help you.
in your very first piece of code, when you write:
train_from_pretrained_model="pretrained-model.h5")
where did you take pretrained-model.h5 file?
with respect to your last question (a more "conceptual" one):

What is the difference over a detection and a prediction? I saw there is a way to train a model from scratch like not use a pretrained model. It's this applicable for my scenario? is it better or worst some how?

ImageAI makes a distinction between those two tasks: detect and predict. It's not the standard way to distinguish between them, and is a bit conffusing for me to name those in this way.
detection refers to the so called object detection task: that is, to say where an object is located on a image, which may contain several objects and therefore we would like to know where is each of those objects and what is it.
prediction refer to the well known problem of image classification. You can think this as saying which is the main object in image. But it also works when you have different kind/categories of images and you would like to know, given an image, what category does it belongs to. For instance: does this image belongs to "cats" category or "dogs"? or maybe you want to know if the image is in the set of useful images or not useful (according to some criteria of usefulness).
Consider that the first problem is far harder than the second, so my advise is to try always to face your problem as a classification problem.  In addition to this, as the first problem is harder, involves to train bigger models, and therefore you may need more data to train them from scratch, and more computational power to train them.
checkout this small article which resumes and compares those task very well.

Hi there @rola93
Thx for your help I will read the article as soon as I get out for lunch.
I followed this article
more specifically download it from here
this is my type of image sometimes there will be more persons so one annotation per person in the same file. Used LabelImg to annotate them in pascal voc format and they look like this(one file per image)
<annotation> 	<folder>img</folder>         <filename>Move_for_cancer13834.JPG</filename> 	<path>/corridas/img/Move_for_cancer13834.JPG</path> 	<source> 		<database>Unknown</database> 	</source> 	<size> 		<width>5184</width> 		<height>3456</height> 		<depth>3</depth> 	</size> 	<segmented>0</segmented> 	<object> 		<name>bibnumber</name> 		<pose>Unspecified</pose> 		<truncated>0</truncated> 		<difficult>0</difficult> 		<bndbox> 			<xmin>3212</xmin> 			<ymin>1133</ymin> 			<xmax>3637</xmax> 			<ymax>1678</ymax> 		</bndbox> 	</object> </annotation>
I would be really glad if you can put me on the right direction. What is more wierd is that as sson as I started doing this I trained a model with 50 images fro 5 cycles and it work (only for one race not for all after all I didn't have too many images for training by then)
thx in advance

I was thinking... after reading with attention... is the problem the path on my annotations files?

Ok just changed the paths... same result

this is my type of image sometimes there will be more persons so one annotation per person in the same file. Used LabelImg to annotate them in pascal voc format and they look like this(one file per image)

I think your approach is correct. But what you should do is a first stage in which you try just to recognize the white surface with the number, with a object detection approach, and then try to extract the number with OCR/image classification or some other technique.
Just to clarify, my point is: do not try to extract the white surface AND its number on a single stage with object detection

@acleitao76  Although a custom detection  model might work for a OCR based task,

you will need not just as many images as possible, but also has possible instances of each number annotated
Also the more the number of unique items, the more the number of images of each instance you need to have for the training.

So, as @rola93 as suggested, it is better you detect the overall bib numbers and then apply an OCR based model to extract the number values, using a 2 stage process.

@OlafenwaMoses @rola93 that's just the case. I'm just trying to find the tag then I cut it off and apply a ocr(now google soon tesseract) the problem is I'm not getting high accuracy finding and cutting of the tag. if the tag just change(every race has different colors) it already don't find it. This weekend I will get my images and label them again to try to train. I'm still getting mAP 0

Oh hoy it.
Yes, if they are different each time you will need either train one model with all of those variations (or as much as possible) or one model per race.
If the amount of different samples is too big you could also train a big model with as many of those as possible and then fine tune a version of it for each race.
I don't know your use case :/
El 7 sept. 2019 9:48 AM, acleitao76<EMAIL_ADDRESS>escribió:
@OlafenwaMoseshttps://github.com/OlafenwaMoses @rola93https://github.com/rola93 that's just the case. I'm just trying to find the tag then I cut it off and apply a ocr(now google soon tesseract) the problem is I'm not getting high accuracy finding and cutting of the tag. if the tag just change(every race has different colors) it already don't find it. This weekend I will get my images and label them again to try to train. I'm still getting mAP 0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/OlafenwaMoses/ImageAI/issues/326?email_source=notifications&email_token=AGDWONDNJWCRUKRV2XY5OG3QIOPJFA5CNFSM4ISW23F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6EYDZA#issuecomment-529105380, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGDWONFXA43G2BBXQHTF6X3QIOPJFANCNFSM4ISW23FQ.


WIP Subquery related changes

Enabling support for subqquery which are enclosed inside round braces.
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Please provide a better explanation on what this is fixing and its intended usage. Also a test to ensure it's working correctly and docs if needed.

Yes. It's under WIP. I am building the test cases and documentation

Sorry. Missed that!

Travis ci build is completed, then too showing in progress.

So - no code changes needed to support this?
Don't know why GitHub is showing the tests as pending... but if you're done I can merge.

Yes. i am done with this PR. You can proceed with the merge


Update LimitClause doesn't use proper Postgres "OFFSET" syntax

Proper Postgres syntax is LIMIT 10 OFFSET 0
This is after updating to v1.10.0
Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42601]: Syntax error: 7 ERROR: LIMIT #,# syntax is not supported LINE 10: ) = $1 LIMIT 0 , 10 ^ HINT: Use separate LIMIT and OFFSET clauses.' in /vendor/slim/pdo/src/PDO/Statement/StatementContainer.php:429 Stack trace: #0 /vendor/slim/pdo/src/PDO/Statement/StatementContainer.php(429): PDOStatement->execute(Array) #1 /vendor/slim/pdo/src/PDO/Statement/SelectStatement.php(411): Slim\PDO\Statement\StatementContainer->execute() #2 /user/file.php(382): Slim\PDO\Statement\SelectStatement->execute() #3 /user/file2.php(76): UserFunction() #4 {main} thrown in /vendor/slim/pdo/src/PDO/Statement/StatementContainer.php on line 429

Please see #49

Ok, I have created https://github.com/kwhat/PDO-SqlSrv for experimental Microsoft support.  I will do the same for Postgres shortly.

This has been updated in 2.2.  Postgres to get its own driver in 3.0.


Positional arguments missing error

I am trying to use validictory package in python 2.6 and the validator failed due to the following error.
File "build/bdist.linux-x86_64/egg/validictory/validator.py", line 275, in validate_items
'{}[{}]'.format(path, index))
ValueError: zero length field name in format
Suggestions online:
In Python 2.6, you need indices in the format specs, like this:
"{0} {1}\n".format(x,y)

sorry, Python 2.6 is not supported by the latest versions of Validictory.  Python 2.6 hit its end of life over a year ago (October 2013)

Ok Thank you.


feat(icon): update visual studio icon(s)

Fixes #3170
Changes proposed:

[x] Add
[ ] Fix

Still needs to be saved as optimized SVG

Even with the inkscape optimization the icon size is greater than 8kb. We'll need to work on these icons before merging this.

We already have a vs icon in the vs folder svg that we could reuse for this one.

Is it the updated one?

Also svgo won't get it down to anything close to 2kb (csproj: 13.92k → 6.19k = 44.46%).


Identify tool or process for document schema validation

Knowing early what is or is-not valid OSCAL schema will be important for extending the OSCAL component files.
Provide documentation or automation for validating OSCAL schema.

Moving to post-POC for DD objectives

Currently no validation around the oscal files being ingested. Implementation might look like a separate command that is utilized to help document developers target problems (line/object etc) and resolve them.

https://github.com/GoComply/oscalkit does provide this validation logic. The validation looks to occur by supplying a schema  to validate a given document against.
Investigation needs to happen around how we might handle multiple versions of documents being used with a single version of the tool and what schemas we have within the compiled binary.

Clarifying context around this issue should be separated into two categories:

Lula needs to test (Likely E2E) that any expectations for input AND reports generated for output are valid to the OSCAL schema
How Lula will respond to invalid oscal. Error handling should be first priority - helper commands secondary as there are existing schema validation tools available.

Tested OSCAL-CLI github action. Seems to be solid.
Pros:
OSCAL-CLI and associated GitHub Action is maintained by NIST.
Can be used to validate and convert each part of OSCAL
Cons:
TBD

The GitHub Action, oscal-cli-action, is maintained by the OSCAL Club community not NIST, but let us know how we can help (I am the maintainer of that action).

@xee5ch thanks for the call out, I updated my Pros/Cons to reflect that! Still in the early phases of testing but everything seems to be straight forward. Thanks for the hard work on it!

No worries, thanks for testing it, we have not got much feedback on it yet. You did remind me to release the updated copy defaulting to v1.0.2 of the CLI out of the oven, so hopefully it keeps delivering value. :-)

Found issue with OSCAL-CLI validation command not being able to find json schema file: https://github.com/usnistgov/oscal-cli/issues/200
Fix should be in next release.

https://github.com/defenseunicorns/go-oscal/pull/50

document schema validation implemented in #136


minor fixes

Fix pack - Logo file is optional.
For debugging, don't overwrite args if provided.

Thanks Mike!


Error in running the app

Fatal error: Uncaught Slydepay\Exception\ProcessPaymentException: SoapFault exception: [soap:Server] Index: 0, Size: 0 in /Applications/XAMPP/xamppfiles/htdocs/slydePay/vendor/slydepay/slydepay-soap/src/Slydepay.php:61 Stack trace: #0 /Applications/XAMPP/xamppfiles/htdocs/slydePay/vendor/slydepay/slydepay-soap/src/Slydepay.php(61): SoapClient->__call('ProcessPaymentO...', Array) #1 /Applications/XAMPP/xamppfiles/htdocs/slydePay/index.php(39): Slydepay\Slydepay->processPaymentOrder(Object(Slydepay\Order\Order)) #2 {main} in /Applications/XAMPP/xamppfiles/htdocs/slydePay/vendor/slydepay/slydepay-soap/src/Slydepay.php:68 Stack trace: #0 /Applications/XAMPP/xamppfiles/htdocs/slydePay/index.php(39): Slydepay\Slydepay->processPaymentOrder(Object(Slydepay\Order\Order)) #1 {main} thrown in /Applications/XAMPP/xamppfiles/htdocs/slydePay/vendor/slydepay/slydepay-soap/src/Slydepay.php on line 68

@1988gadocansey i know the fix its simple u made a mistake where the items array comes u forgot to make it an array. example most people forget the [ ] before the { } so instead of items: [ { "id" : "", "name" : "", "price" : "", "quantity" : 1}] and do this rather items: { "id" : "", "name" : "", "price" : "", "quantity" : 1 } without the [ ] but that part takes an array so it wont work make sure make it an array even if u are using xml u see its telling u theres a problem with your array....

@anupamverma932 your problem is u are sending the same order id again after u used it to receive a token make sure its always unique try doing that check always at ur end before u send orderid out with the other info to the Slydepay class.

try {
    // Make request to Slydepay and get the response object for the redirect url
    $response = $slydepay->processPaymentOrder($order);
    return $response->redirectUrl();
} catch (Slydepay\Exception\ProcessPayment $e) {
    echo $e->getMessage();
}

pay_token URL will be generated from this and just want to view the generated URL but am getting error
"Call to a member function redirectUrl() on array"
the error is coming from this line return $response->redirectUrl();
Am using laravel.

https://github.com/DreamOval/slydepay-php-connector/issues/14#issuecomment-826667524


[Class hierarchy] 'vasa recta cell' is a capillary endothelial cell and a vein endothelial cell at the same time

CL term
vasa recta cell
Suggested revision of class hierarchy

[ ] This needs changes in Uberon: #2914
[ ] Replace kidney capillary endothelial cell for microvascular endothelial cell

@aleixpuigb  please see related uberon ticket: https://github.com/obophenotype/uberon/issues/2914


Instantiating a provider, error. segmentProvider not being recognized

Hi folks,
just trying to get started and get an error in my configuring angular-segment. The error I get is
Failed to instantiate module app due to:
Error: [$injector:modulerr] Failed to instantiate module SegmentConfigProvider due to:
Error: [$injector:unpr] Unknown provider: segmentProvider

this is my code
angular.module('SegmentConfigProvider', []).config(function (segmentProvider) {
    segmentProvider
        .setKey('xxxxxxxxxxx')
        .setCondition(function ($rootScope) {
            return $rootScope.isProduction;
        })
        .setDebug(true)
});

if i omit the empty params passed to my SegmentConfigProvider I get an error that it failed to load and was mispelled.
Uncaught Error: [$injector:modulerr] Failed to instantiate module app due to:
Error: [$injector:modulerr] Failed to instantiate module SegmentConfigProvider due to:
Error: [$injector:nomod] Module 'SegmentConfigProvider' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

had to pass ngSegment to the provider module. silly me

I'm glad you figured it out! In case others find this issue later - I'm guessing the line you fixed was this:
angular.module('SegmentConfigProvider', ['ngSegment']).config(function (segmentProvider) {


feat: adding a modal for fee on quote component

Summary
We needed a modal to show the fee details when clicking on the fee icon on Quote.
How did you test this change?
When you click on the gas station in the widget after getting the root, the fee details are displayed (also check the Figma)
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

Please examine all the paddings and spacings in this modal, as it diverges from the UI design in multiple places. Also, for a cleaner DOM, opt for using padding instead of the Divider component.

Height of this element doesn't match with UI design

The button's hover state has been overlooked

Kindly review the build error associated with the widget-embedded.


add support for new pod_format 0.0.17 features

I've tried to go through and make this converter use the new tools added in pod5 0.0.17 release - to be out shortly.

Thanks George, I appreciate that. I'll merge when then new version comes out and update the README.
Will the fields remain in pod5 when output from minknow on the future?


Start/Stop RtcCall when user goes to background or blocks the phone

Task

JIRA

          1 Message

:book:
:link: BM-546

JaCoCO Code Coverage 2.76% :white_check_mark:

Class
Covered
Meta
Status

co/netguru/baby/monitor/client/feature/client/home/livecamera/ClientLiveCameraFragment
0%
0%
:white_check_mark:

co/netguru/baby/monitor/client/feature/client/home/livecamera/ClientLiveCameraFragmentViewModel
0%
0%
:white_check_mark:

co/netguru/baby/monitor/client/common/view/CustomSurfaceViewRenderer
0%
0%
:white_check_mark:

APK file size

size

21.6MB

Number of method references

Dex file
count

classes.dex
34385

  Generated by :no_entry_sign: Danger


Design a new logo for OneStopHelp

Task
You are supposed to design a new logo (apart from the readme.md file one). Look for some awesome designs for cool ideas.
(Include logo designs of both - text type and icon type)
Implementation
Tools used: Adobe Photoshop, Adobe Illustrator, Canva or any other tool convenient to you.
Create designs and share screenshots in the comment section
Once finalized create PR
Procedure
Claim this issue: Comment below that you want to work on this issue
Write relevant commit message while making a PR

I would like to work on this issue.

I would like to work on this issue.

I would like to work on this issue.

I would like to work on the logo

I would like to work on this issue.

Go for it !!

I would like to work on this issue.

Go for it!!

I would like to work on this issue.

Go for it!!

Guys go for it. Design a new and cool logo for OneStopHelp. All the best!!😉

I would like to work on it too.

Pls check the following logos that i designed !!

Pls check the following logos that i designed !!

2nd one looks nice. We will wait for all the assignees to make logo and then we will finalise one. 😉

I have designed 2 logos so far and would add more if required.

I have designed 2 logos so far and would add more if required.

Looks nice. We will wait for all the assignees to make logo and then we will finalise one. 😉Till then feel free to make more designs if you want.

I have designed a few also. Please give them a look.

@nporia and @Mitali-laroia, Your designed logos look really really cool. 😁 We will finalise one soon. Thanks for contributing.

I have made these logos so far and I'm working on more options. Please have a look^

I have made these logos so far and I'm working on more options. Please have a look^

@nandini-412 Logos look really good. Thanks for your contribution.😉

I would like to work on this issue.

please review this. I will make more soon

please review this. I will make more soon

This one looks cool. But few changes that you can do are firstly write One-Stop Help straight in middle and also the red colour is not going good with pink and white, see if you can change it to another colour. 😊

ok, I will do the necessary changes.


redis.status is still ready after redis.quit() resolves

I'm not certain that this is a bug, but it definitely surprised me while migrating from node_redis. I would not expect the second console log to still be ready in this example:
const Redis = require('ioredis');
const client = new Redis();

async function run() {
  await new Promise(resolve => setTimeout(resolve, 100)); // make sure we're connected
  console.log('initial', client.status); // 'ready'
  client.quit().then(() => {
    console.log('quit resolve', client.status); // still 'ready' :/
    setTimeout(() => console.log('timeout', client.status), 0); // but now 'end'
  });
}
run();

In node_redis the client.ready boolean (which ioredis' client.status seems to parallel) will be false by the same point, so I would expect here that client.status would no longer be ready.
Full debug output:
$ DEBUG=ioredis:* node test.js
  ioredis:redis status[localhost:6379]: [empty] -> connecting +0ms
  ioredis:redis status[<IP_ADDRESS>:6379]: connecting -> connect +7ms
  ioredis:redis write command[0] -> info() +1ms
  ioredis:redis status[<IP_ADDRESS>:6379]: connect -> ready +4ms
initial ready
  ioredis:redis write command[0] -> quit() +97ms
quit resolve ready
  ioredis:redis status[<IP_ADDRESS>:6379]: ready -> close +2ms
  ioredis:connection skip reconnecting since the connection is manually closed. +0ms
  ioredis:redis status[<IP_ADDRESS>:6379]: close -> end +0ms
timeout end

After redis.quit() resolves, how would you recommend inspecting that the client is, in fact, in a non-ready state? Do I need to quit() and then also wait for a redis.on('close') event, or should the quit() command instead not resolve until redis.status has been updated?

One workaround, if you're looking to determine whether the connection is still good-to-use (synchronously - not by issuing a command), is to check the private manuallyClosing field. It would be really nice if this was accurately and consistently reflected in the status field; the cluster implementation sets the status to closing.


Fixing pub/sub link in resource manager.

Link currently points to a non-existent page in the pub/sub docs.

LGTM


TV state is regularly changing, even without any actions

Describe the bug
I have NU8042 and I found some weird stuff going on with the state of my TV.
My logs show that every ~25 mins the TV's state changes to "on" and after around 30s back to the "off" state. It happens without any human action, for example during the night.
(Figured this out when I had set an automation to turn off my main light after the TV is turned on - my light was randomly switching off during the day without touching the TV :D)
Expected behavior
TV state should change only when I actually turn on/off my TV.
Screenshots

Environment details:

Rasbian, Home Assistant as a standalone
Home Assistant version installed: 0.109.1
Component version installed: v0.3.2 (was happening also before)
Last know working version: none
TV model: NU8042

Output of HA logs
Check the screen above
My config
samsungtv_smart:
  -  host: <IP_ADDRESS>
     update_method: ping
     source_list: '{"ENTER": "KEY_ENTER", "TV": "KEY_TV", "HDMI": "KEY_HDMI1" }'

Do you use SmartThings API with your configuration?
TV during the night enable Internet connection, probably to check firmware update, this cause the integration to report the TV as on. May be enabling SmartThings (and also removing the line "update_method" in configuration) can make the check a little bit more reliable, but not sure about this.
My TV appear as "ON" once for night for about 30 sec., not more than this. How often happen to you?

@ollo69 it happens a lot -

Yes, i have smarthings API enabled.
I'll remove the update_method line and check how this works out. Will keep you informed.

@ollo69 unfortunately this hasn't changed anything -

These two occured after removing the update_method line and restarting the HA :(

Could you try to ping your IP TV from the RaspBerry for 5 minutes in one windows where the TV is reported as On and let me know how long the TV answer to the ping?

@ollo69 it responds for 32 seconds. It's actually the same window as it shows in the Home Assistant logs

So at least components detect well TV network availability :)
This is complex situation, I do not have other way to detect TV status.
What I can do is increase the time required to determine the TV is On, but this will cause a longer On detect time in case that the TV is really turned on.
I will think about this...

I released a new version with a possible fix for this issue. Eventualy you can try increase the configuration option "Seconds to delay power ON status" if default value (30 sec) is too low.
Please let me know if this solve.

@ollo69 Great, it works as expected! Thank you for your effort :)

Fine, I close this issue. Reopen if you still have problem.

@tadeusz-niemiec and whoever faced this problem. I got Samsung Frame TV 2020, and was facing the same behavior with both the generic HASS "Samsung Smart TV" component and this one ("SamsungTV Smart"). My TV would go online every 20 mins or so. That triggered the automation I had around state change from off to on and obviously it was a mess.
I figured that this TV intent to connect the network seems like some software update checks. And indeed, after I turned off in the TV settings "Automatic Check for Updates" the problem was remediated. I guess it is a reasonable trade to keep state change reporting sane.

This thread is closed, so please open a new one.
Anyway did you try to increase the configuration option "Seconds to delay power ON status"? This option was created to work-around this issue.

@ollo69 you might misunderstand my comment. I found out that if I turn off Samsung TV automatic check for updates, the issue disappears, the TV does not go online every 25 minutes. So this is just an alternative solution to this threat problem.

Clear, I understood. Just wondering if the component option also solve the problem in case you don't want to disable the automatic check for update option.

@ollo69 yes, it might solve it in my case. But 30+ sec delay was not in option in my case. I configured my speakers to turn on after TV was on, and having such a long delay was killing a good user experience.

@tadeusz-niemiec and whoever faced this problem. I got Samsung Frame TV 2020, and was facing the same behavior with both the generic HASS "Samsung Smart TV" component and this one ("SamsungTV Smart"). My TV would go online every 20 mins or so. That triggered the automation I had around state change from off to on and obviously it was a mess.
I figured that this TV intent to connect the network seems like some software update checks. And indeed, after I turned off in the TV settings "Automatic Check for Updates" the problem was remediated. I guess it is a reasonable trade to keep state change reporting sane.

Thanks for sharing this! This was so simple and I haven't thought about it... I also decided to sacrifice automatic updates for reliable TV state detection :)

Hello, @ollo69 I understand this is a very old issue but the trick to disable automatic updates is still relevant today - at least it was for me.
Are you open to a PR for the readme file? I can add a bit of insight on this topic where people get a chance to read it. To be fair, this is such a promiment issue with an otherwise perfect integration that I would mention the workaround in the setup flow - but that might be too far fetched.
Keep in mind I am using smart things and "Use SmartThings TV status information" option is enabled. I believe the integration prefers the ping to smart things information as it is faster, but this leads to a lot of false positives.
Let me know your thoughts, and thanks for the great work.


Put method for incidents not in api documentation

The documentation at https://docs.cachethq.io/v1.0/docs/  is lacking an entry for PUT for the /incidents endpoint although it is there in the code.

Good spot, I hadn't published the page: https://docs.cachethq.io/v1.0/docs/update-an-incident

Have you checked the metrics api as well? There is no PUT method and I havent verified that is is implemented in the sources.

We do indeed have PUT methods on metrics but no documentation - that's an oversight on my part. I'll fix this today.


Infinite recursion bug when calling generate_number

In the main.py file, when jsonformer fails to generate a number, it continues calling generate number, but without incrementing the iterations variable, causing the "Failed to generate a valid number" to never get called and we get max recursion depth exceeded error.

I don't know why sometimes it fails to generate any numbers which caused my experiments to fail. That's why I have to make a workaround by returning a default number when it fails after 3 iterations.


[Playground] Helm+kubernetes+deployment

New solution for Playground backend - with Kubernetes and Helm
New pipeline for Backend deployment: using HELM and Kubernetes

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

lgtm


SNAP-3015: Put thousands separators for Tables > Rows Count column in Dashboard.

What changes were proposed in this pull request?

Adding thousands separators for table row count as per locale.

How was this patch tested?

Tested manually.

Please review http://spark.apache.org/contributing.html before opening a pull request.

Review is not required as it is a minor and cosmetic change.

Merging.


add rpc documentation

This PR aims to add documentation for the RPC code, and make it more clear what calls are available, and how they work. This should also make it easier in the future to write clients in other languages.
I'm not sure if this initial state is good enough, please collaborate and give feedback where needed :)
I am not sure if I should omit the (incomplete) type info or not, for now it's there, but if you rather have me remote that part, I'll do so.

Does this require more changes?

yeah, I have some minor comments for the sections I didn't get to previously. So far it's looking good though

Cool @lukechampine, feel free to push those changes yourself to this branch though, I think as a owner of this repo you can do that. Might be quicker than going back and forward :) And makes this a real collaborative PR 👍

I think only you can push to the branch since it's on your repo. We sometimes push to each other's PRs on Sia because the branches are all part of the NebulousLabs repo.

@lukechampine ah, didn't know that. Somehow I thought that you were also able to push to it. Might have misread that at some point, a couple of months ago.
I added the changes you requested.

Removed comma as requested.

Woo


Allow plugins to run for pull requests

As a starting use case: it is often convenient to "stage" a change introduced by a PR for manual inspection. For instance, a staging version of a website could be published at a unique URL, in addition to regular build process:

Currently, plugins are not being run, unconditionally. To make the behavior backward compatible, the when filter would default to when: { event: push } if none specified.

This may or may not be relevant to this particular discussion, but I really like how the Kubernetes team is handling this with their Jenkins setup:

Here is the issue I pulled this from. I really dig this as an extra option on top of whatever Drone core protections we might add.

I propose we should only execute plugins if the yaml is signed. Otherwise there are a couple possible attack vectors that could be exploited. Right now these vectors are prevented by excluding plugins from executing during pull requests.
First example might be the ssh or rsync plugin where a malicious pull requests modifies the yaml to execute the plugin step, but changes the commands to do something very bad, such as removing directories of even altering the authorized_keys file to give themselves access to the server:
deploy:
  ssh:
    addr: <IP_ADDRESS>
-   commands: [ ./restart.sh ]
+   commands: [ sudo rm -rf /root/ ]
+   when: { event: pull_request }

Second example might be to try to exploit something like the git or pypi plugin. Let's say a malicious pull request add a post-commit hook to the .git directory (or alters the setup.py for pypi). The hook will run inside the plugin, and will be able to query the /proc filesystem to see what data the plugin was invoked with. The git plugin is passed you ssh private key, for example.
publish:
  git:
    push: git@github.com:foo/bar.git

I believe signing the yaml is the best way to prevent these sort of attacks. If a developer chooses to enable (and sign the yaml) to execute the pypi step when testing a pull request they may make themselves vulnerable. So how do we feel about "giving people enough rope to hang themselves with" (as the phrase goes)?
cc @benschumacher

Run plugins in PRs only when yaml is signed sgtm.

Just a note: all the attack vectors @bradrydzewski mentions generally come from environments where an arbitrary command can be injected bypassing yaml signature verification.
@gtaylor k8s has a load of shell scripts executed during testing (which brings you to the previous paragraph). but yeah, it would be a nice another feature for drone, although much more complex than this.

What I meant by first paragraph in https://github.com/drone/drone/issues/1476#issuecomment-182638745 is @bradrydzewski's first example can be exploited even if a yaml is signed: it is sufficient to insert sudo rm -rf /root/ in ./restart.sh w/o modifying the yaml file.
Users just need to know this is dangerous. Maybe some plugins could be manually marked as safe as an additional measure.

... first example can be exploited even if a yaml is signed: it is sufficient to insert sudo rm -rf /root/ in ./restart.sh w/o modifying the yaml file.

Yes, we need to warm people to use their better judgement, probably with gigantic warning messages in the documentation. Since people like to skip / skim the documentation they might miss the warnings. We could possible add something to the yaml to acknowledge that it might be a bad idea (unsafe: true?)

Users just need to know this is dangerous. Maybe some plugins could be manually marked as safe as an additional measure.

agreed, there are a few ideas we are kicking around such as the plugin requesting permissions to sensitive data. The pypi plugin could expose the ssh key or netrc file, even though it doesn't need this sensitive data to do its job.
Plugins could request sensitive data (sort of like android apps) by storing that meta-data in the Dockerfile. Drone could inspect the image prior to starting the plugin:
LABEL permissions=PRIVATE_KEY,NETRC

This still isn't full proof since the git plugins (which would be subject to evil scripts in .git/hooks) still need the ssh key, however, we could harden the git plugins to remove the .git/hooks directory at runtime.
This of course still means that we need to identify all the possible exploits in each plugin requesting secrets, which may is subject to me making mistakes :)

Maybe a PLUGIN_SAFE could be introduced as a first iteration, similar to what PLUGIN_FILTER does, which would include only a subset of official plugins by default.

There was a comment made in drone-dev on the topic highlighted in https://github.com/drone/drone/issues/1476#issuecomment-182639764. One potential mechanism to address this potential attack is to (optionally) allow the signing mechanism used on .drone.yml to be extended to other artifacts.
I'm not proposing a syntax, but if a change to an artifacts such that it doesn't match an expected signature, then it could fail the build early. This could already be implemented by a user by building these tests directly into .drone.yml. As I type this, I could imagine writing this logic directly in a custom before plugin based on #1443. Obviously this still requires users to adopt these security tools, but that's always the case.
I'm a fan of opt-in for unsafe behaviors, so I like the idea of having a configuration callout for unsafe: true, but is the YAML the correct location for that?

Noodling this a bit, there are varying degrees of trust already baked into Drone. With a "Trusted" repo setting you are already acknowledging some degree of "allow me to be vulnerable".  With that setting a user can enable things like privileged mode on a container, which from a security stand point is game over.
In the context of a trusted build, with conditionals on plugins, you have some granularity on when to run things particularly with event types, tags and branches.
Would it be reasonable that if you have a signed yml, trusted build setting enabled, and conditionals for event: pull_request that it be honored.
I also really like the suggestion of @gtaylor based on whitelisting users. Or org members default to some behavior vs non. For public projects, this could go a long way into defining how/where secrets are allowed.

@cloudnautique I think as long as the yaml is signed and event: [ pull_request ] is explicitly configured for each plugin we should have enough assurance to proceed.
The attack vector I'm most concerned with is altering the yaml to run a deployment step that shouldn't otherwise be executed. If the yaml is signed this attack vector cannot be exploited.
The exception to the above rule are plugins that don't require secrets to execute such as the ssh and rsync plugin. This is because drone provides a per-repository key pair that can be used instead. This has me thinking that drone should stop generating and using key pairs, and these plugins should instead source the keypair from the .drone.sec file.

What was the outcome of this thread?
Is this functionality available in the current 0.4 builds? Or do we need to wait for 0.5 / 0.6 ?
Thanks

@sl1pm4t this is fixed in 0.5. It required breaking changes and therefore won't get backported to 0.4

Wanted to ask if this should be closed or not as this is the only issue holding the v0.5.0 milestone back. Stoked for the 0.5 official release 🎉5⃣

closing since this is implemented in 0.5
there are some agent <> server communication issues blocking an official 0.5 tag, but since many people are already using 0.5 and taking advantage of this feature, I think we can close :)


docs: fix indent

Commit Message: corrected indentation for list items
Additional Description: N/A
Risk Level: Low
Testing: N/A
Docs Changes: fix indentation
Release Notes: N/A
Platform Specific Features: N/A

@shou1dwe You will need to fix the DCO issue in order to land this change. You can click "Details" next to the DCO check for more instructions, but in any case here is the direct link.
https://github.com/envoyproxy/envoy/pull/19090/checks?check_run_id=4310135596

/assign @phlax

/wait


JTAG debug of blink due to:  Error: Too large number of threads<PHONE_NUMBER>!

----------------------------- Delete below -----------------------------
If your issue is a general question, starts similar to "How do I..", or is related to 3rd party development kits/libs, please discuss this on our community forum at esp32.com instead.
INSTRUCTIONS
Before submitting a new issue, please follow the checklist and try to find the answer.

[x] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[x] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[x] I have searched the issue tracker for a similar issue and not found a similar issue.

If the issue cannot be solved after the steps before, please follow these instructions so we can get the needed information to help you in a quick and effective fashion.

Fill in all the fields under Environment marked with [ ] by picking the correct option for you in each case and deleting the others.
Describe your problem.
Include debug logs on the monitor or the coredump.
Provide more items under Other items if possible can help us better locate your problem.
Use markup (buttons above) and the Preview tab to check what the issue will look like.
Delete these instructions from the above to the below marker lines before submitting this issue.

----------------------------- Delete above -----------------------------
Environment

Development Kit:      [ESP32-Wrover-Kit|ESP32-DevKitC|ESP32-PICO-Kit|ESP32-LyraT|ESP32-LyraTD-MSC|none]
Kit version (for WroverKit/PicoKit/DevKitC): [v1|v2|v3|v4]
Core (if using chip or module): [ESP32-Wrover-I|ESP32-Wrover|ESP32-PICO-D4|ESP32-SOLO-I|ESP32-WROOM-32U|ESP-WROOM-32D|ESP-WROOM32|ESP32]
IDF version (git rev-parse --short HEAD to get the commit id.):
//bd6ea4393c7d2f059fc4decc70f1ec3eb3597268
Development Env:		[Arduino IDE|Make|Eclipse|other]
Operating System:		[Windows|Ubuntu|MacOS]
Power Supply:         [USB|external 5V|external 3.3V|Battery]

Problem Description
//Detailed problem description goes here.
Expected Behavior
Actual Behavior
Steps to repropduce

step1
...

// It helps if you attach a picture of your setup/wiring here.
Code to reproduce this issue
// the code should be wrapped in the ```cpp tag so that it will be displayed better.
#include "esp_log.h"

void app_main()
{

}

// If your code is longer than 30 lines, GIST is preferred.
Debug Logs
Debug log goes here, should contain the backtrace, as well as the reset source if it is a crash.
Please copy the plain text here for us to search the error log. Or attach the complete logs but leave the main part here if the log is *too* long.

Other items if possible

[ ] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

I'm using the wrover kit V3 with the ILI9341 LCD running under ubuntu.
1 - Connecting the pins as seen in the documentation gave execution errors.  Looking at the schematic I noticed that the pin above the gnd, TD was incorrect.  I removed it and it executed correctly but with further problems.   A
I copied blink to another library for testing (jjf) and  issued the make menuconfig.  I changed the gpio pin to pin 15 as pin 5 caused the LCD to blink and not the diode.
See **************** below for error message.

Execution of openocd in terminal 1
jjjf@jjjf-Z97X-SLI:~/esp/openocd-esp32$ bin/openocd -s share/openocd/scripts -f interface/ftdi/esp32_devkitj_v1.cfg -f board/esp32-wrover.cfg -c "program_esp32 $HOME/jjf/blink/build/blink.bin 0x10000 verify"
Open On-Chip Debugger 0.10.0-dev (2018-04-18-12:03)
Licensed under GNU GPL v2
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
none separate
adapter speed: 20000 kHz
esp32 interrupt mask on
force hard breakpoints
Info : ftdi: if you experience problems at higher adapter clocks, try the command "ftdi_tdo_sample_edge falling"
Info : clock speed 20000 kHz
Info : JTAG tap: esp32.cpu0 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
Info : JTAG tap: esp32.cpu1 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
Info : esp32: Debug controller was reset (pwrstat=0x5F, after clear 0x0F).
Info : esp32: Core was reset (pwrstat=0x5F, after clear 0x0F).
cpu0: Current bits set: BreakIn BreakOut RunStallIn
cpu1: Current bits set: none
Info : JTAG tap: esp32.cpu0 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
Info : JTAG tap: esp32.cpu1 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
cpu0: Current bits set: BreakIn BreakOut RunStallIn
cpu1: Current bits set: none
Info : Target halted. PRO_CPU: PC=0x40009081 (active)    APP_CPU: PC=0x00000000
esp32: target state: halted
Info : esp32: Debug controller was reset (pwrstat=0x5F, after clear 0x0F).
Info : esp32: Core was reset (pwrstat=0x5F, after clear 0x0F).
Info : Target halted. PRO_CPU: PC=0x5000004B (active)    APP_CPU: PC=0x00000000
esp32: target state: halted
Info : esp32: Core was reset (pwrstat=0x1F, after clear 0x0F).
Info : Target halted. PRO_CPU: PC=0x40000400 (active)    APP_CPU: PC=0x40000400
esp32: target state: halted
** Programming Started **
auto erase enabled
Info : Use core0 of target 'esp32'
Info : Target halted. PRO_CPU: PC=0x40091D07 (active)    APP_CPU: PC=0x400076DD
Info : Auto-detected flash size 4096 KB
Info : Using flash size 4096 KB
Info : Use core0 of target 'esp32'
Info : Target halted. PRO_CPU: PC=0x40091D07 (active)    APP_CPU: PC=0x400076E2
Info : Use core0 of target 'esp32'
Info : Target halted. PRO_CPU: PC=0x40091D07 (active)    APP_CPU: PC=0x400076E0
wrote 139264 bytes from file /home/jjjf/jjf/blink/build/blink.bin in 3.099145s (43.883 KiB/s)
** Programming Finished **
** Verify Started **
Info : Use core0 of target 'esp32'
Info : Target halted. PRO_CPU: PC=0x40091D07 (active)    APP_CPU: PC=0x400076DD
read 138640 bytes from file /home/jjjf/jjf/blink/build/blink.bin and flash bank 0 at offset 0x00010000 in 0.694212s (195.028 KiB/s)
contents match
** Verified OK **

executiion of blink in terminal 2
jjjf@jjjf-Z97X-SLI:~/jjf/blink$ xtensa-esp32-elf-gdb -x gdbinit build/blink.elf
GNU gdb (crosstool-NG crosstool-ng-1.22.0-80-g6c4433a) 7.10
Copyright (C) 2015 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "--host=x86_64-build_pc-linux-gnu --target=xtensa-esp32-elf".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
http://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from build/blink.elf...done.
0x40000400 in ?? ()
JTAG tap: esp32.cpu0 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
JTAG tap: esp32.cpu1 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
cpu0: Current bits set: BreakIn BreakOut RunStallIn
cpu1: Current bits set: BreakIn BreakOut RunStallIn
esp32: Debug controller was reset (pwrstat=0x5F, after clear 0x0F).
esp32: Core was reset (pwrstat=0x5F, after clear 0x0F).
Target halted. PRO_CPU: PC=0x5000004B (active)    APP_CPU: PC=0x00000000
esp32: target state: halted
esp32: Core was reset (pwrstat=0x1F, after clear 0x0F).
Target halted. PRO_CPU: PC=0x40000400 (active)    APP_CPU: PC=0x40000400
esp32: target state: halted
Hardware assisted breakpoint 1 at 0x400d2043: file /home/jjjf/jjf/blink/main/./blink.c, line 45.
0x0:	0x00000000

output on terminal 1 after starting blink in terminal 2.
** Verified OK **   (from terminal 1)
Info : accepting 'gdb' connection on tcp/3333

Error: Too large number of threads<PHONE_NUMBER>!

Info : JTAG tap: esp32.cpu0 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
Info : JTAG tap: esp32.cpu1 tap/device found: 0x120034e5 (mfg: 0x272 (Tensilica), part: 0x2003, ver: 0x1)
cpu0: Current bits set: BreakIn BreakOut RunStallIn
cpu1: Current bits set: BreakIn BreakOut RunStallIn
Info : esp32: Debug controller was reset (pwrstat=0x5F, after clear 0x0F).
Info : esp32: Core was reset (pwrstat=0x5F, after clear 0x0F).
Info : Target halted. PRO_CPU: PC=0x5000004B (active)    APP_CPU: PC=0x00000000
esp32: target state: halted
Info : esp32: Core was reset (pwrstat=0x1F, after clear 0x0F).
Info : Target halted. PRO_CPU: PC=0x40000400 (act

GPIO15 is one of the JTAG pins (others are 12-14). If you reassign it as GPIO, JTAG will no longer work. On wrover-kit, you can use IOs 0, 2, 4 which are connected to RGB LED.

Changed the GPIO to io2 and I did get the gdb prompt and entered n.  But, Target halted and got the output belo the ----------- line..
Other times I change the GPIO to IO19 and it hangs after
Hardware assisted breakpoint 1 at 0x400d1f8f: file /home/jjjf/jjf/blink/main/blink.c, line 45.
0x0:	0x00000000.  (no gdb prompt).

PRO_CPU: PC=0x400D1F8F (active)    APP_CPU: PC=0x400D1AB2
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[New Thread<PHONE_NUMBER>]
[Switching to Thread<PHONE_NUMBER>]
Temporary breakpoint 1, app_main () at /home/jjjf/jjf/blink/main/blink.c:45
45	    xTaskCreate(&blink_task, "blink_task", configMINIMAL_STACK_SIZE, NULL, 5, NULL);
(gdb) n
Target halted. PRO_CPU: PC=0x400D1F92 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1F95 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1F98 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1F9B (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1F9D (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1FA0 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1FA3 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1FA6 (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400851DC (active)    APP_CPU: PC=0x400D1AB2
Target halted. PRO_CPU: PC=0x400D1FA9 (active)    APP_CPU: PC=0x40080340
Target halted. PRO_CPU: PC=0x400800C3 (active)    APP_CPU: PC=0x40080340
/builds/idf/crosstool-NG/.build/src/gdb-7.10/gdb/regcache.c:697: internal-error: regcache_raw_read_unsigned: Assertion `regnum >= 0 && regnum < regcache->descr->nr_raw_registers' failed.
A problem internal to GDB has been detected,
further debugging may prove unreliable.
Quit this debugging session? (y or n)

@jjfretired I'm going to close this one because the same issue is already tracked in https://github.com/espressif/openocd-esp32/issues/37. We have some patches to GDB on the way to fix this. Please go to the linked issue and click "Subscribe" to get notified when this happens.


renewal_server_url ignored

Have you read the FAQ and checked for duplicate open issues?
Yes
What link can we use to reproduce this?
https://shaka-player-demo.appspot.com/demo/#audiolang=en-US;textlang=en-US;uilang=en-US;asset=https://wvm.ezdrm.com/demo/dash/BigBuckBunny_320x180.mpd;license=https://widevine-dash.ezdrm.com/clientdebug.aspx?pX=B03B45&customdata=original;panel=CUSTOM CONTENT;build=uncompiled
What version of Shaka Player are you using?
v3.2.0-uncompiled
What browser and OS are you using?
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.164 Safari/537.36
What did you do?

Played the test stream
What did you expect to happen?
Player should read the renewal_server_url and then call that URL for the renewal license,
https://widevine-dash.ezdrm.com/clientdebug.aspx?cd=renew1&rand=58H
What actually happened?
it is calling the base WV Proxy URL and ignoring the renewal_server_url value being passed to the player
calling https://widevine-dash.ezdrm.com/clientdebug.aspx?pX=B03B45&customdata=original

Player should read the renewal_server_url and then call that URL for the renewal license

Where should the player read it from?

This is in the license response,

The license response is not consumed by the player, but by the CDM.  We don't have access to the fields of a license response.

That feature of the Widevine protocol is not available to JavaScript applications running in web browsers.  We apologize for any inconvenience.

But the player can read the response in the HTTP traffic just like the chrome browser is doing, and parse it out. Or are you saying this can only be done in the EXOplayer native app?

No, these open-source players do not have the ability to parse the Widevine license protocol.  Android has a native API that exposes these details from the CDM, but browsers do not.

Thank you,


Limit verbose from new versions of ibmcloud

Signed-off-by: Yussuf Shaikh<EMAIL_ADDRESS>

/cc @aishwaryabk @Prajyot-Parab


Optimize: reduce unnecessary codes in RemoteBrokerOffsetStore

FEATURE REQUEST

Please describe the feature you are requesting.
reduce unnecessary codes in RemoteBrokerOffsetStore

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks involved, use -[] for each sub-task and create a corresponding issue to map to the sub-task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

close issue first due to relative PR closed.


Added test for replicationTaskExecutor: execute

Description:
Contains unit tests for the domainReplicationTaskExecutorImpl's Execute method, covering domain creation, update, name/UUID collision, and handling of unsupported domain operations.
The tests utilize mocking to simulate DomainManager interactions.
Why?:
Improved test coverage for domain replication task execution ensures robust handling of domain operations across different scenarios.
How did you test it?:
The added unit tests were executed locally, with all tests passing. Mocks were employed to simulate the DomainManager interactions.
Potential risks:
Minimal risk is associated with this PR as it primarily adds unit tests without altering the core logic of domain replication task execution.

Pull Request Test Coverage Report for Build 018e1abf-8f13-46ec-b35d-4d03c02e8324
Details

0 of 0   changed or added relevant lines in 0 files are covered.
36 unchanged lines in 10 files lost coverage.
Overall coverage increased (+0.02%) to 63.953%

Files with Coverage Reduction
New Missed Lines
%

service/history/queue/timer_queue_processor_base.go
1
77.62%

common/peerprovider/ringpopprovider/config.go
2
81.58%

service/history/task/timer_task_executor_base.go
2
78.38%

service/matching/taskListManager.go
2
79.7%

service/matching/taskReader.go
2
84.88%

common/membership/hashring.go
2
82.23%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
71.41%

service/history/task/fetcher.go
3
85.57%

service/history/task/transfer_active_task_executor.go
5
72.19%

common/persistence/nosql/nosqlplugin/cassandra/workflow_parsing_utils.go
14
79.13%

Totals

Change from base Build 018e1aa9-b112-4b5e-a137-732317ef6b1b:
0.02%

Covered Lines:
93755

Relevant Lines:
146599

💛  - Coveralls


Validation Error

Anyone know how to fix the following error?

Recently, this code stopped working. Perhaps some kind of update from C.AI .
I get the same error.

kramcat replied to me
pip install pydantic==2.7.0
But I got a new error.
2 validation errors for NewChat
participants.0.user.account.mobile_onboarding_complete
  Input should be a valid integer [type=int_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.7/v/int_type
speech
  Input should be a valid string [type=string_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.7/v/string_type


"Castelu", "Nisipari" are listed in the drop-down menu

Description:
When user writes "Castelu", "Nisipari" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Castelu"

Expected results
The location appears in the drop-down menu as "Sat Castelu, CONSTANȚA, (Castelu)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Nisipari"

Expected results
The location appears in the drop-down menu as "Sat Nisipari, CONSTANȚA, (Castelu)"


How to add the cache_annotations keyword in YoloDarknetFormatDetectionDataset

🐛 Describe the bug
data_set = YoloDarknetFormatDetectionDataset(data_dir='/data/praful_yolonas_dataset', images_dir="train/images", labels_dir="train/labels", classes=["Fingerprint"], cache_annotations=False)
TypeError: init() got an unexpected keyword argument 'cache_annotations
Versions
I tried to add the cache annotation keyword in  YoloDarknetFormatDetectionDataset but it's giving the keyword error.
Actually i don't want create the Caching annotations of training images and labels.

YoloDarknetFormatDetectionDataset class do not have this argument, so simply do not pass it to constructor.

Alright, thank you. However, could you guide me on running the training source code without generating cache annotations?


Local blob container not opening

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.24.3
Regression From
No response
Architecture
x64
Storage Explorer Build Number
20220617.2
Platform
Linux (snap)
OS Version
ubuntu 18.04
Bug Description
Error Details
If you cannot resolve this issue on your own, you may open an issue on the Storage Explorer GitHub site. Please include the details below to help us diagnose your issue. Make sure to redact any personal information you do not wish to make public before posting.
Unable to retrieve child resources.
Details:
{
"name": "FetchError",
"message": "request to https://<IP_ADDRESS>:10000/devstoreaccount1?comp=list&include=metadata failed, reason: write EPROTO<PHONE_NUMBER>9144:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:242:\n",
"stack": "FetchError: request to https://<IP_ADDRESS>:10000/devstoreaccount1?comp=list&include=metadata failed, reason: write EPROTO<PHONE_NUMBER>9144:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:242:\n\n    at ClientRequest. (/snap/storage-explorer/38/resources/app/node_modules/node-fetch/lib/index.js:1491:11)\n    at ClientRequest.emit (node:events:390:28)\n    at TLSSocket.socketErrorListener (node:_http_client:447:9)\n    at TLSSocket.emit (node:events:390:28)\n    at emitErrorNT (node:internal/streams/destroy:157:8)\n    at emitErrorCloseNT (node:internal/streams/destroy:122:3)\n    at processTicksAndRejections (node:internal/process/task_queues:83:21)",
"type": "system",
"errno": "EPROTO",
"code": "EPROTO"
}
Steps to Reproduce
open and view contents of container
Actual Experience
popup with title Error details
Expected Experience
No response
Additional Context
No response

Since you're using https with Azurite please either:

Try using system proxy (can be found under Settings (gear on the left hand side -> application -> proxy)
Following what this user did: https://github.com/microsoft/AzureStorageExplorer/issues/2598#issuecomment-616765993

Closing due to lack of response. If you're still running into this problem we recommend opening an Azure support ticket via the Azure Portal. Alternatively you can open a new issue here.

in the storage explorer there is a option to disable proxy for copy. selecting that solved this issue.


[owners] Fix team @ mentions

App-authenticated comments appear to come from the app user (ie. amp-owners-bot) but can't tag teams. The workaround is doing a fake update to the comment with a user-authenticated client, which enables the @ mention but leaves the comment author as the app name.

Unless you're in a rush to get this running, have you tried contacting GitHub support and asking about this? I feel like this should work from the app, instead of going through a hack

@danielrozenberg I definitely wanted it fixed because it was preventing the bot from notifying wg-caching about validator changes. I haven't reached out to support directly, but this is a well-established challenge that has a lot of discussions on the web and the end result is always using a personal access token. I gave the app every possible permission and nothing made the tag work when using app auth. I can reach out to support directly, but in the interim I'd like to make sure the team mentions are working