Is it possible to extend this to support dragging between lists?

Very clean library and eager to use it for my use case.
However, I am wondering if its possible to support multiple lists where I can drag items between lists, but also reorder the lists themselves.
So a structure like:

<SortableList {items} let:item direction="horizontal" on:sort={handleSort}>

    {item.id}
    <SortableList items={item.items} let:item direction="vertical">

        {item.id}

    </SortableList>

</SortableList>

Ideally, using <svelte:self /> it could be nested deeply. Do you think this is achievable?

Hi, @mattgi! :) My apologies for the delay, it’s been a busy week.
Well, I’ve got some good news for you, because I’m currently working on an important refactoring to improve the overall flexibility of the package, and I’m hoping it will help facilitate something like nested lists. I’m don’t think it will be possible to drag and drop items between lists out of the box, but nesting lists should be feasible.
It’s hard for me to give you an expected delivery date, but I’ll drop another comment when this is available ;)

Hey, @mattgi! That took a long while, but I’m back with some good news :)
I’ve finished the first and main stage of the major refactoring, which I just published an hour ago. It is now possible to create the kind of structure you were looking for, since I’ve extracted the <SortableItem> out of the <SortableList> component, allowing them to be imported separately.
I did a quick attempt to replicate your example, and even though the lists are displayed correctly, that’s about it. There are several bugs all around preventing it from working as desired. But I got really curious about it, so I will probably be working on it in the next few days to allow for proper nested lists :)
If you have the time, please, give it a try and let me know your thoughts.

Hey! @rodrigodagostino  i was trying to update my code with your new realease but i ve been facing an issue, my sortableItemData looks like this
let items: SortableItemData[] = [
	{
	  id: "list-item-1",
	  text: "List item 1",
	  files: ["file1.svg", "file2.svg", "file3.svg", "file4.svg"],
  }];

when i tied inside the SortableItem component to iterate through files array to create 1 component per file im getting this
Argument of type 'unknown' is not assignable to parameter of type 'ArrayLike<unknown> | Iterable<unknown>'.

i have a nested list but not a sortable a normal list, can you helped me pls, older code is this
 "@rodrigodagostino/svelte-sortable-list": 0.9.6",

	<SortableList {items} let:item on:sort={handleSort}>
		<SortIcon
			id="handle-icon"
			slot="handle"
		/>

		<Tooltip triggeredBy="#handle-icon" placement="left">Drag!</Tooltip>

		<Folder folder_name={item.trait}>
			{#each item.files as file}
				<File file_name={file} folder_name={item.trait} />
			{/each}
		</Folder>
	</SortableList>

Hi, @ManuelPuentes! Thanks for reaching out :)
I’m a little confused. I seems that you are using version v0.9.6, but the type definition for SortableItemData was not available back then. I reproduced your scenario with the latest available version (0.10.1), and even tried it in one of my own apps, but I’m not seeing any kind of errors.
If you’re working with version 0.9.6, try replacing SortableItemData[] with SortableListProps['items'] (check the README for v0.9.6).
If you need further assistance, I would appreciate it if you could open up a new issue. Since this comments are not related to the current issue, we’re derailing the conversation here :P

Hi, @ManuelPuentes! Thanks for reaching out :)
I’m a little confused. I seems that you are using version v0.9.6, but the type definition for SortableItemData was not available back then. I reproduced your scenario with the latest available version (0.10.1), and even tried it in one of my own apps, but I’m not seeing any kind of errors.
If you’re working with version 0.9.6, try replacing SortableItemData[] with SortableListProps['items'] (check the README for v0.9.6).
If you need further assistance, I would appreciate it if you could open up a new issue. Since this comments are not related to the current issue, we’re derailing the conversation here :P

no, i missexplained myself sorry, the older code version was using 0,9.6 but then new one is using 10.0, the problem mentioned above is for 10.0. the code  shared was the older version and  my question was how can i adappted to 10.0 ? my solution was to avoid using "SortableItemData" type on items array

Hey, @mattgi! just wanted to let you know that, due to lack of activity, I will be closing this issue. But feel free to open it back again in case there is something that we could still work on :)


Migration update fails in UWP with SQLite and EF7

I added a new filed in my model class and generate a new migration file with the command :
Add-Migration MyNewMigration
When I start the application, the following code :
public App()

        {
            this.InitializeComponent();
            this.Suspending += OnSuspending;

            try
            {
                using (var db = new BloggingContext())
                {
                    db.Database.Migrate();
                }
            }
            catch(Exception ex)
            {

            }
        }

throws exception : SQLite cannot support this migration operation.
Packages I have installed are : Install-Package EntityFramework.SQLite –Pre
and : Install-Package EntityFramework.Commands –Pre

Maybe your migration is hit by one of the many sqlite limitations: http://ef.readthedocs.io/en/latest/providers/sqlite/limitations.html

From http://sqlite.org/lang_altertable.html#otheralter

The only schema altering commands directly supported by SQLite are the "rename table" and "add column" commands.

This means to perform operations such as "DROP COLUMN", EF would need to do a table rebuild. We haven't implemented this yet.
Closing as a duplicate of https://github.com/aspnet/EntityFramework/issues/329

Opened https://github.com/aspnet/EntityFramework/issues/5389 to provide more help in this situation.


[Feature] Additional executable

Summary
Instead of custom start executable, launch other executable along side the game started by Starward. Keep the game timer working and able to play with something like SRMI injected.
Solution or Design
Start other executable after start the game itself

Thanks for the suggestion, but I can't implement it.
Starting game is the most important feature of Starward and it should be kept as brief and stable as possible. And starting additional executable is obviously not necessary for normal gameplay.
With the existing functionality you can define your startup process in a batch file .bat and then set it as the startup target.

Thank you for explanation.


🐞 Generate sources fails when dagger module is in .dagger directory

What is the issue?
With dagger v0.11.0, when running dagger develop and the module is in a folder called .dagger the package main/internal/telemetry is not added to the dagger.gen.go fille. The imports are as follows.
import (
	"context"
	"encoding/json"
	"fmt"
	"log/slog"
	"os"

	"main/internal/dagger"

	"go.opentelemetry.io/otel"
	"go.opentelemetry.io/otel/attribute"
	"go.opentelemetry.io/otel/sdk/resource"
	semconv "go.opentelemetry.io/otel/semconv/v1.4.0"
	"go.opentelemetry.io/otel/trace"
)

Dagger version
dagger v0.11.0
Steps to reproduce
The following repository contains a dagger module at .dagger running dagger develop on this folder will incorrectly generate the sources.
https://github.com/jumppad-labs/jumppad/tree/52c3ff885cf616650ef997de36768663e8a4048f
Log output
No response

Aha, this looks like the same issue I discovered while working on https://github.com/dagger/dagger/pull/7194 - the fix is part of that PR in 6fdd040 (#7194).

Not 100% sure why this only seems to appear in a small sub-section of cases, but it definitely needs fixing :scream:


builtin:swc-loader do not provide a type declaration

What problem does this feature solve?
we use builtin:swc-loader, but can't get  type declaration from rspack, only reference document.
https://www.rspack.dev/guide/loader.html#builtinswc-loader
What does the proposed API of configuration look like?
import { SwcLoaderOptions } from '@rspack/core';

/claim @9aoy

after a second thought and suggestion from @07akioni, we can put this type in @rspack/cli other than pollute @rspack/core.
import { defineSwcLoader }  from '@rspack/cli';
export default ({
  module: {
     rules: [{
       test: /\.js/,
       loader: defineSwcLoader.Loader,
       options: defineSwcLoader({
              jsc: {},
              transformer: {}
        })
     }]
  }
});

We can also use the @type jsdoc:
export default {
  module: {
    rules: [
      {
        test: /\.jsx?$/,
        use: [
          {
            loader: 'builtin:swc-loader',
            /** @type {import('@rspack/utils').SwcLoaderOptions} */
            options: {
              jsc: {},
            },
          },
        ],
      },
    ],
  },
};

From https://github.com/mdx-js/mdx/tree/main/packages/loader#use

I do not prefer the `@rspack/utils' solution, as users have to install an extra package just to type the SWC loader.

prefer RspackSwcLoader and defineRspackSwcLoaderOptions  exported by both @rspack/core and @rspack/cli just as same as other rspack/webpack builtin things like SwcJsMinimizerRspackPlugin and SwcJsMinimizerRspackPluginOptions

I will add the typing this week.


Lib update and .gitignore

Updated the mp3agic lib from a snapshot to the most recent release

added gitignore according to the linux, android, java and intellij addons

I put this pull request here, because it seems that you are more active than @arcus92 at the moment

Looks good, especially the .gitignore

I don't plan to do that much work on this project, so I added you as a collaborator. If you want to do anything else feel free.

Thank you ^^


GPIO support for Core on Appropriate Platforms

It was suggested that I post this here rather than in corefx or coreclr. I am intrigued by the support for Raspberry Pi in the core platform and getting that up and running on the nearest Pi is on my todo list.
What really intrigues me is the ability to access the GPIO system from within a .NET Core application so that I can read from sensors, control LEDs, and send I2C signals to other components chained up with the Pi.
I would love to know if this is supported and what the steps are to get it working.

@petermarcu suggested I create this issue here.

I did some searching and found a few good options from the Mono community that will probably work just fine on .NET Core.
https://github.com/raspberry-sharp/raspberry-sharp-system
https://github.com/AlexSartori/Raspberry-GPIO-Manager
https://github.com/andycb/PiSharp
@eerhardt any advice?

For general usage, https://github.com/raspberry-sharp/raspberry-sharp-system was the one I looked into briefly when prototyping/designing my drone/RF24 work.
However, I took a slightly different route.  Since I was only concerned about radio signals, I ended up wrapping the native RF24 library in a managed wrapper and used that to talk to the radio. My reasoning was that the RF24 library already had a good API for talking to the radio and I didn't want to rewrite it in managed code over a general purpose GPIO API. I think it worked out well for me, and if there was a native library that already exists and solved my needs, I would take the same approach.

That said, I would love to see https://github.com/raspberry-sharp/raspberry-sharp-io ported to .NET Core.  😉

Are we talking about general purpose and unified GPIO APIs (with abstracted support for universal protocols and interfaces such as I2C), that can then be specialized in hardware-specific libraries?

My core need is just to be able to detect pin status and to raise and lower pins. Obviously wrapper libraries above such a core would be useful for specialized tasks. But for now I'd love to be able to demo a microservice running on a Pi that controls an LED and reports the status of a temperature sensor, etc.

In other words, a demo for the maker community indicating that you're not stuck with Python or Node for all the things.

I like the idea of a blog post with code samples, using one of the available libraries then.

We need .NET in this space again!  I've used Netduino, and gadgeteer for years, but getting .NET Core with general purpose and unified GPIO APIs would be awesome!

I expect more will happen in this space after .NET Core 2.0 ships.
I'm not seeing anything immediately actionable in this issue. Closing.

I have a library you could use. It's .NET Standard 2.0, you can read/write to GPIO and PWM pins on any linux device.
https://github.com/utilitydelta-io/dotnet-gpio


fix: fix crash #297

@tcort This fixes an unnoticed regression from #278, the crash is quite common so likely best to merge and do a patch release. Sorry!


Scoping imported CSS w/ styled-components

I'm using babel-plugin-inline-import to import built css files for antd components. I import the CSS as strings & insert it via tagged template literal into a styled-component. This is to avoid requiring the entire CSS file on every page.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
When importing CSS as a string and passing it to a styled component, the imported styles should affect components in the proper scope.
Here's my .babelrc:
{
    "presets": ["next/babel"],
    "plugins": [
        [
            "styled-components",
            { "ssr": true, "displayName": true, "preprocess": false }
        ],
        [
            "inline-import",
            {
                "extensions": [".css"]
            }
        ]
    ]
}

Current Behavior
When I wrap the component in the regular styled function from styled-components, the styles are injected into the  and the component is given the correct classnames. However, no styles are applied.
Such an implementation looks like this:
import Button from 'antd/lib/button';
import styled from 'styled-components';
import Styles from 'antd/lib/button/style/index.css';

export default styled(Button)`
    ${Styles};
`;

^^ Note that this implementation works in non-nextjs projects, at least in my experience.
Now, when I use styled-components injectGlobal, though, it works fine:
import Button from 'antd/lib/button';
import { injectGlobal } from 'styled-components';
import Styles from 'antd/lib/button/style/index.css';

injectGlobal`${Styles}`;
export default Button;

Using classnames to style descendants work fine in Styled Components/next in regular use cases, like this:
// index.js
const Wrapper = styled.div`
  .potato {
      background: brown;
      height: 100px;
      width: 100px;
  }
`
export default props => <Wrapper>
</Wrapper>

It seems the only broken case is when I'm importing the css.
Steps to Reproduce (for bugs)
I setup a small example repo based on the with-styled-components example in next's repo:
https://github.com/brandonmp/styled-components-next-scope
Context
I'm trying to port over a project from Gatsby, and I'd rather not pollute global with all of antd's styles.
Your Environment

Tech
Version

next
latest

node
8.6.0

OS
Win10 via WSL Bash

browser
Chrome 62

etc
My favorite food is pickles

I think the reason it works with injectGlobal vs putting it directly on the styled Button component is that the class scope is nested too deep when you put it on a Button.
Consider this:
You have a Button component and then import  the corresponding antd styles for that component.
Those styles now get nested as such:
const WrappedButton = styled(Button)`
  .ant-btn-primary {
    background: red;
  }

When in reality, they should be this (note the &):
const WrappedButton = styled(Button)`
  &.ant-btn-primary {
    background: red;
  }

So this is why the injectGlobal works fine for you. I'd bring the issue up with the folks over @ https://github.com/styled-components to see if they have a good solution over using injectGlobal.

thanks @pruhstal. I don't believe the & is necessary in the way you've described it is.
There's an example in the repo I set up in OP demonstrating this, but here's a webpack bin with styled-components affecting a deeply nested div in a nested component with no &:
https://www.webpackbin.com/bins/-KywGV9ZD4mUavFIORe9
Here's code:
import React from 'react'
import styled from 'styled-components'

const Wrapper = styled.div`
  .potato {
    background: blue;
    height: 100px;
    width: 100px;
}
`

const SomethingElse = props => (
)

function Hello () {
  return (
    <Wrapper>

            <SomethingElse/>

      </Wrapper>
  )
}

export default Hello

FWIW it appears some are experiencing similar challenges w/ styled-jsx, though I can't be sure the problem is the same:
https://github.com/zeit/next.js/issues/544#issuecomment-337234098

I'm expecting this to be solved now that there's universal webpack and otherwise it sounds like more of a styled-components / babel plugins issue 🤔


Offset?

Hi !
Is there a way to set some x/y offset for the boss bar (maybe i'm missing it, but I looked everywhere)?
That would be super cool to prevent the bar from overlaping with UI stuff like the carousel combat tracker :)

Hello! It seems you haven't read the Before Posting an issue as indicated at the top of the Repository Readme The Issue has been closed.
If you are reporting a bug, feel free to follow the steps here (https://theripper93.com/info/issues) and reopen the issue with all the relevant information.


feat(modules): style vencord cards

before:

after:

@TakosThings hello! do you accept changes related to vencord?

I guess ignore means no


Crashes unexpectedly while typing with Fcitx5 in Chromium/Electron

Hyprland Version
Built from branch  at commit v0.25.0 (). Tag: ? flags: (if any): debug
Bug or Regression?
Bug
Description
I got Hyprland crashed randomly while typing with Fcitx5 using another input method than the default in Chromium and Electron applications (Brave and Discord/Webcord for more specific) on NixOS. Brave crashes regardless of using --gtk-version-4 or --enable-wayland-ime flag for it, Electron apps are obviously using later flag. It mostly crashes when I suddenly edit a text after a while, after switching from other tabs or windows.
How to reproduce

Open a website that has input box and type something
Switch to other tabs/windows for a while and go back
Getting crashed to the login manager

Crash reports, logs, images, videos
hyprlandCrashReport1665.txt
hyprlandCrashReport11664.txt

v0.25 ???

Haha, I'm a little lazy switching over flakes to get latest build than the one in NixOS's stable channel stucking at v0.25, but I did not see any fixes related to my issue from forward builds neither. Should be done upgrading to v0.28 in next few hours and report back if it's happen to be fixed.

I think my bug is caused by the combination of Hyprland + Waybar + Fcitx5 and there is already an opened issue, just in a different way, as when Waybar is not running the reproduced steps don't make it crash.


test: implement simple sugar test

I implemented some test, just to understand the project a little bit.
the expected_error probably will (should) improve? - but that can be done iteratively I guess
... and is this on purpose that resolve() changes the given variable in place?

Currently it does everything in-place though we will likely switch to copying the tree. If you use the return value of resolve in your test case it'll keep working after the change!


pdf-viewer does not work as of

Hi,
pdf-viewer does not work anymore as of version 1.56.0. After compiling my latex document, a tab opens but it is empty.  Downgrading to 1.55.0 fixes the problem and pdf-viewer shows normally.

Which pdf-viewer are you using?

Also raised at https://github.com/izuzak/atom-pdf-view/issues/238.

Thanks for contributing!
It appears that this is being caused by the atom-pdf-viewer package, not Atom itself. It may be that there is still a bug in Atom. If so, we'll need the maintainer of the package that is causing the problem to help track that down. You should file this issue on their repository so they can either fix the problem or tell us where the problem is coming from. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.


Avoid displaying over gitsigns.nvim

Instead of displaying over gitsigns.nvim, can the marks be displayed in the second column? Or alternatively, make them the same color as the gitsign.

The mark signs are somewhat clashing with my recent insight for automatically dropping a mark whenever I go into insert mode, e.g. nnoremap i mii, as shown in https://github.com/YodaEmbedding/dotfiles/commit/47f46bc6e4d3f0d1db7068e9bc3d7ecb7bf7d0eb .

You should use the 'sign_priority' option to ensure mark signs get displayed after gitsigns


Couldn't open file: cfg/mnist.dataset

Hi Mofan,
I execute 'darknetp classifier train -pp_start 4 -pp_end 10 cfg/mnist.dataset cfg/mnist_lenet.cfg' on Raspberry Pi 3b, but it shows 'Couldn't open file: cfg/mnist.dataset', I can't find a solution in the questions have asked.Looking forward to your reply.
thank you

Hi @HaoXuan-a
Were you able to resolve this?

This is because the data set was not placed in the folder shown in the tutorial, so the file was not found.


Workloads: Sidecars/multi-container UX could be improved

The UX for workloads with multiple containers needs to be improved  - team to review and discuss the issues and how to improve the UX.

A lot of the problems stem from the same page used to edit the workload and add/remove/edit containers
The tabs shown in the edit workloads page are a mix of deployment and container
From the workload list, or view workload page, adding a sidecar takes user to the edit workload page with a new container selected. Slightly confusing to change contexts

Addressed by 2.6.9/2.7.0 work


That's really an awesome project!

really awesome, makes it so easy to install OKD.

thanks! hope this repo can make your life easy for a cup of coffee :)


RemotePushIOS.requestPermissions not call when user click allow or not button

I install you component with npm , and follow you step in Examples, when i run code in a real device, the requestPermissions function is not called, so i can't get  the 'erro' or 'data'. how can i fixed it ? thanks a lot!

you have to add all the files to your xcode directory..

copy all of the files to your XCode directory, and add them to your project. Then, when you want to use the module, require the RemotePushIOS class, e.g.


can't use 'onRowClick' prop on customRowComponent

Should pass the  'onRowClick' prop to the 'CustomRowComponentContainer'  component and 'customRowComponent' component
griddle.jsx.js:
React.createElement(CustomRowComponentContainer, { data: data, columns: cols, metadataColumns: meta,
className: this.props.customRowComponentClassName, customComponent: this.props.customRowComponent,
style: this.getClearFixStyles(), onRowClick: this.props.onRowClick }),
customRowComponentContainer.jsx.js
var nodes = this.props.data.map(function (row, index) {
return React.createElement(that.props.customComponent, { data: row, metadataColumns: that.props.metadataColumns, key: index, onRowClick: that.props.onRowClick });
});

Hi there and sorry about the extreme delay in following up on this one! In the latest version of Griddle we've made it possible for just about every component to be passed in as a prop and would recommend that a Row component that overrides the default Griddle row be used instead of the CustomRowComponent from v0. The user defined Row component could handle events like onRowClick
Please see the customization docs for more information on this. Thanks 👍


Link protection ux is confusing

When clicking on "Configure Trusted Domains" you are sent to the picker which shows domain that you just clicked on in the list and checked:

This leaves you in a very confused state because it is now checked. So the natural thinking is since it's checked, why did it ask me to open the link? So I cancel the quickpick, click the link again and I get the same message. After inspecting it further, it appears that the "Configure Trusted Domains" actually adds the entry and you have to click on "Ok" to apply it. This is extremely confusing as the user has no idea this was added, this list is always treated as a static list.
Additionally, looking back on our notes from the last ux sync, I thought we all agreed the actions would be: Trust Once, Trust Always, Cancel. And the "Configure" part was going to be a setting? It appears like this isn't reflected at all.

That came out of discussion at https://github.com/microsoft/vscode/issues/78125#issuecomment-521689062

So the natural thinking is since it's checked, why did it ask me to open the link?

Would it make sense to make the message a bit different so it stands out? For example, everything below the separator -> already trusted domains. Top two options are:

Open all links without prompt
Trust new domain https://dev.azure.com and open link (and remove the faded details)

I don't think the message would improve the UX. Something that could help/improve is to have an action to add a domain when results are empty:

To me, "Configure Trusted Domains" does not equal "Add this domain" so we either should update the button text to indicate it's also adding, and then I'd auto-fill the input field to indicate that it's adding.

See See https://github.com/microsoft/vscode/issues/80595#issuecomment-531607624


RW Client - Modify KTOR to Add Credentials for Web Api cookie use case

Why

This is required for the cookies to work

How

Adding it in the HttpRequestData.toRaw extension method

Side effects

Unknown

Works without any issue on TV (kotlin native)


How to get current_time/now in C++ in 1.6

As system.h header will be removed when using C++ in v1.7, I'd like to use an alternative to now() and current_time() that is in a .hpp header.
Even when importing system.hpp now, it cannot find these functions:
#include <eosio/system.hpp>
error: use of undeclared identifier 'now'; did you mean 'new'?

From eosio.cdt v1.6 onwards
You have to #include <eosio/system.hpp> and then you can use:
eosio::current_time_point
eosio::current_block_time
to cast from time_point to uint32_t u can use like:
eosio::current_time_point().sec_since_epoch();

It is unclear what returning uint32_t means. Is the expectation that EOS will have been phased out by the year 2038?  Or does sec_since_epoch return a mod 2^32 of the actual time? If so, it seems an error prone API to include.
I checked the documentation already, but it didn't clarify.


fix(module:progress): fix nzFormat not shown status is exception

close #5130
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

It seems to me that this is adjustment rather than a bug fix.


Can I map arbitrary object as @Id of an entity?

Example:
@Entity
public class File
{
    @Id
    public FileKey fileKey;

    public String directory;

    public String filePath;

    public Long fileSizeBytes;
}

When I try to save, I get an error:

java.lang.ClassCastException: org.bson.types.ObjectId cannot be cast to com.mongodb.DBObject

Actually i'm also facing the same issue. My use case is somewhat different. If the client passes in an id then i need to use that as the _id. However if the client doesnt pass in an id then i need to use ObjectID can only have an ObjectID or the String id as the _id and not both.
For this i define my @Id as java.lang.Object and i get the same exception. After debugging i found that the error is caused here.
In normal flow if we defined the @Id as ObjectId, the expression mf.isTypeMongoCompatible() is true and uses a different Mapper (ValueMapper)
Is it possible to fix this based on the instance of the Object?

@seoj what happens when you put @Embedded on that field?
@nitinkcv mapping fields of type Object is not currently supported.  I'd suggest your ID field should be a String and when you generate an ObjectID just toString() it before assigning it to the ID field.  then your IDs will always be a String and there'd be no conflict.

Thanks. When do you think this can be supported? If you don't have time i can work on a PR when i get time.

There aren't plans currently to support that.  With Object, there's nothing we can track to map the serialization data properly.  In short, we can't tell how we're supposed to serialize it or how to read it out of the database.

@evanchooly I am unable to reproduce the issue for some reason. I did not need the @Embedded annotation on the field for it to work though.

@seoj OK.  I'll close this then.  If you run in to it again, please post to the mailing list and we'll take a look.

FWIW, to support a union type for an @Id field, I had success adding a custom TypeConverter (must implement SimpleValueConverter) that simply passes through Object values. I could definitely see this being error prone so obviously use with caution / add appropriate error handling.


Cached database doesn't take the mode into account

The cached database only uses the filename in its check. Checking the mode also would enable an application to cache separate Database objects for read and write access. In my application, I open the database first for checking the data has been ingested and if not, continue to write to the database. As a result I cant use the cached instance (which would be the one with read mode). If the API supported accounting for the mode, then the code could benefit from opening the database in its required mode without concerning if it was opened in any other mode by the other parts of the code, a cached instance will always be used when available.
I'd be happy to work on a patch.

Closing for now but would accept a PR 🙂


How to use html2pdf.js without NPM or any other package manager.

Hello
How to use How to use html2pdf.js without NPM or any other package manager. I am working on a legacy project and need to use html2pdf.js with jQuery or Javascript. Please confirm how can I do that.
I tried adding dependencies before html2pdf.js but it giving exception.
html2pdf.min.js:2 Uncaught TypeError: Cannot read properties of undefined (reading 'jsPDF')
    at ./src/plugin/jspdf-plugin.js (html2pdf.min.js:2:2378)
    at r (html2pdf.min.js:2:72332)
    at html2pdf.min.js:2:72910
    at html2pdf.min.js:2:73107
    at html2pdf.min.js:2:73120
    at html2pdf.min.js:2:303
    at html2pdf.min.js:2:328

Please confirm how can I use it without NPM.
Thanks

https://github.com/eKoopmans/html2pdf.js#cdn
<script src="https://cdnjs.cloudflare.com/ajax/libs/html2pdf.js/0.10.1/html2pdf.bundle.min.js" integrity="sha512-GsLlZN/3F2ErC5ifS5QtgpiJtWd43JWSuIgh7mbzZ8zBps+dvLusV+eNQATqgA/HdeKFVgA5v3S/cIrLF7QnIg==" crossorigin="anonymous" referrerpolicy="no-referrer"></script>

What if someone don't want to use CDN due to any reason?


lazyLoad: 'progressive' not working for slides added with slickAdd

http://jsfiddle.net/kdmk9p3a/9/
Can be fixed manually with
$('.slider').on('reInit', function(_event , slick) { slick.progressiveLazyLoad() });

+1
The manual fix works but if using lazyLoad: ondemand the additional slides are no longer laded on demand.


ArcChat.com added to showcase

Thanks! :+1:
Please could you add it to the bottom though? For now the showcase is currently on a first-come first-served basis. We can review that down the road when there are more entries.

We can review that down the road when there are more entries.

I agree, once we have enough examples, I think that we can follow React Native by adding a featured section.

reordered showcase items :+1:

@liesislukas Thanks! We usually ask to contributors to squash down their commits. But that's fine like this for your first contribution :+1:.


Refactor HTTP Query functionality into dedicated type

Refactor the HTTP Query functionality from the QueryServiceImpl type into dedicated type, as this will be needed in other places, too.

/CC @joluj

Duplicate of #90


Clean up backend.run(...) interface

The backend.run(...) interface has accumulated a lot of cruft over the years. We need to start paring down this interface to only things which are essential.
Some things that I think can go (should validate all of these):

remove Schedule and List[Schedule] from the allowed types of circuits. We nominally stopped supporting these types already in January 2022.
qubit_lo_freq
meas_lo_freq
schedule_los
noise_model - should be attached to the Backend, not to the call to run().
seed_simulator - again should be an argument on the simulator/backend

More controversial, but I think still worthwhile:

drop the memory argument and only expose the behavior of memory=True. Rationale: if you really want statistics, use the Sampler instead.
meas_return - if we only support memory=True, then it only makes sense for this to be 'single'

For future consideration:

meas_level - OpenQASM 3 allows expressing different measurement levels with separate measure commands in the OpenPulse grammar. I think it makes sense to develop equivalents in Qiskit and then read out the requested measurement level from the circuit itself

Closing since backend.run is deprecated.


MissingMemberHandling.Error should not exception when ExtensionData is present

This is a PR to make it not an error to put an element into ExtensionData when MissingMemberHandling.Error is set on the serializer.
Without this, catching the extra members in a parent with an expandable child (Scenario C) is not currently possible because otherwise valid objects deserialized into the expandable child exception (Scenario B throws MissingMemberHandling.Error Exception)
a) (TestMissingMemberHandlingForDirectObjects)
it is not an error to have extra fields on an entity with ExtensionData present.
b) (TestMissingMemberHandlingForChildObjects)
it is not an error to have extra fields on children when the children have ExtensionData present
c) (TestMissingMemberHandlingForChildObjects)
it is an error to have extra fields on the parent, even when the children have ExtensionData present.
This required changing 2 checks in the JsonSerializerInternalReader to only throw the MissingMemberHandling exception when the relevant ExtensionData was null.
As Written this is a mildly breaking change, as objects which previously failed to deserialize into their ExtensionData, will now begin to. I'm happy to alter this PR to have MissingMemberHandling have a new third state which allows ExtensionData, while trhowing on missing fields.
I implemented this as a breaking change, as when i explicitly say "put all other fields into this extension data dictionary" i do not expect an exception "nowhere to put this data".

This PR came about because I have strict deserialization on the objects i consume (MissingMemberHandling.Error set), however one of the objects i want to consume has a unknown and variable schema, as it has a generic key-value store exposed as an unpredictable set of json properties, which i wanted to store into the ExtensionData dictionary.
Without this PR, i have to turn of the MissingMemberHandler, add a ExtensionData  to the parent, and then have an onDeserialized check asserting that the dictionary is empty.

Please speak English.  Exactly what are you doing, and what have you done
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Andrew Hill<EMAIL_ADDRESS>Sent: Tuesday, February 19, 2019 10:35:09 PM
To: JamesNK/Newtonsoft.Json
Cc: Subscribed
Subject: Re: [JamesNK/Newtonsoft.Json] MissingMemberHandling.Error should not exception when ExtensionData is present (#1988)
This PR came about because I have strict deserialization on the objects i consume (MissingMemberHandling.Error set), however one of the objects i want to consume has a unknown and variable schema, as it has a generic key-value store exposed as an unpredictable set of json properties, which i wanted to store into the ExtensionData dictionary.
Without this PR, i have to turn of the MissingMemberHandler, add a ExtensionData to the parent, and then have an onDeserialized check asserting that the dictionary is empty.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/JamesNK/Newtonsoft.Json/pull/1988#issuecomment-465419557, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AmsNVlHLwmYAx6HBFBZDd3t_53w3mr0Tks5vPNB9gaJpZM4bEXk4.

@yatkimkaj, the report is in English. No morning coffee yet?

Yea something like that.    What is it u r trying to do?
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: elgonzo<EMAIL_ADDRESS>Sent: Wednesday, February 20, 2019 5:51:21 AM
To: JamesNK/Newtonsoft.Json
Cc: yatkimkaj; Mention
Subject: Re: [JamesNK/Newtonsoft.Json] MissingMemberHandling.Error should not exception when ExtensionData is present (#1988)
@yatkimkajhttps://github.com/yatkimkaj, the report is in English. No morning coffee yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/JamesNK/Newtonsoft.Json/pull/1988#issuecomment-465543197, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AmsNVhhSw_YWcjfxM_wPVfdGxFEWO0lpks5vPTa5gaJpZM4bEXk4.

Nobody is going to dumb down something for you here. This is an issue report about Newtonsoft.Json, not some place for tutoring someone who likes/needs stuff "dumbed down"

Hi
I've added MissingMemberHandling to JsonObjectAttribute. I prefer this because it is not a breaking change and it keeps each feature simpler. Also now people can choose to opt-in to error on some objects and no others.
https://github.com/JamesNK/Newtonsoft.Json/pull/2050


Track export options

Let's track export options directly in the repo. Then the initial workflow setup doesn't need to involve copying an unwieldy string.
Would look like this
    - script@1:
        inputs:
        - script_file_path:
        - content: |-
            #!/usr/bin/env bash
            envman add --key EXPORT_OPTIONS --value "$(cat Configuration/export_options.plist)"
        title: Make export options available
    - xcode-archive@5:
        inputs:
        - project_path: "$BITRISE_PROJECT_PATH"
        - scheme: "$BITRISE_SCHEME"
        - distribution_method: "$BITRISE_DISTRIBUTION_METHOD"
        - export_options_plist_content: "$EXPORT_OPTIONS"
        - cache_level: none

Haha, I also did the same and added the plist file: https://github.com/bitrise-io/BackyardBirds/commit/603770e33abd8b0dddd1076a1c5786d8b3eababd


Add Washington Post

They have a PWA! https://www.washingtonpost.com/pwa/

Tested, added, pushed to washington-post. Will merge and deploy once @andreasbovens confirms.

Added


My nginx proxy manager resets itself

Hi everyone,
I installed the Nginx proxy manager and it seems like every restart of the operating system which is proxmox lxc contianer the data of the container disappears.
every reboot its like a fresh install and all the things I changed and did are disappearing even the passwords and mail I did.
a little about the environment:
I ran proxmox lxc container that has docker inside it (yea I know its a little junky but its works for all the rest of the containers)
the container has an NFS disk mounted to it and the data are saved there for some reason the Nginx proxy manager deletes all the data and acts like it's a fresh install after every reboot.
and i checked multiple time there are no problems with the NFS mount
this is the compose file I use
version: '3.8'
services:
app:
image: 'jc21/nginx-proxy-manager:latest'
restart: unless-stopped
ports:
- '80:80'
- '81:81'
- '443:443'
volumes:
- /root/proxy-manager/data:/data
- /root/proxy-manager/letsencrypt:/etc/letsencrypt

Hi,
I have the same issue with nginx proxy manager running in docker on a VM.
I use the website docker composed as is (https://nginxproxymanager.com/setup/#running-the-app) and i have not defined a maria DB.
Everything is fine if I restart the docker, but as soon as i reboot the VM on witch docker is installed (ubuntu 22.04 LTS) it gets resset
How could I fixed so that it doesn't nuke everythink i did everytime I reboot my server / VM ?
Thanks

@lidoror I suspect that your issue is not with nginx proxy manager. You did mount /data to /root/proxy-manager/data, but /root isnt a hard drive in your context, because your Docker NPM is within a "proxmox lxc container". I dont know much about this tech so lets assume its a docker container (docker within docker). If THAT layer doesnt have a mount for /root (or /root/proxy-manager), then restarting that container will reset the content of /root, then your docker inside LXC will start with an empty folder.
Just restart the NPM container itself (docker restart my-nginx-proxy-manager-container, NOT your os or your lxc thing) and if data is still there, the problem is not with NPM.


Looking for Yarn 2.0 support? Look here:

Since this project appears abandoned, I have forked it and upgraded it to the Yarn 2.0 language.
But better than that, I have a wrapper around it called YarnBound, which has added features and a more user-friendly API.
This post is probably bad etiquette, but I think it could help people who find it (and who's around to stop me >:P ).

heya, its possible that @hylyh could give you access to maintain it if you ask for it :)
I dont have a lot of time to do it these days anyways

I wouldn't be opposed, but I'm not sure it makes sense without a hard commitment to 2.0, which might not be the wisest move given that every current project uses 1.x . Problems are:

I've made so many changes that "maintain" would basically have to mean "nuke and replace"
My fork is not backward-compatible with Yarn 1.x so existing users could be confused.
- In theory there is more work that could be done fixing 1.0 bugs and unfortunately I'm just not interested in digging back into those, which would leave the currently-used 1.x support unqualifiedly stranded...

I'm open to ideas; a note in the README pointing to my fork might be the way?

I think when we discussed this in the past, the consensus was that the old bondage 1/yarn 1 will be split into a bondage1 branch, while the yarn 2 version will become the master branch

OK, I opened a giant PR that merges YarnBound and my bondage changes into one repo. I know it's a pain haha, but it would be cool to consolidate!

I started a fork of YarnBound here before I realized that there was a 2.0 branch in this repo. Where is the proper place to be making changes and updates? Would it be this 2.0 branch? I am doing large updates for the purpose of integration into a game engine which includes:

Fixing issues that are out of alignment with the latest version of Yarn (system hashtags like #lastline, proper indenting rules, etc.).
Conversion to TypeScript.
Making runners poolable.
Events.

I should probably put in an MR to the 2.0 branch with my latest bugfixes; I'm sure it's a little out of date. At that point yeah it's kind of nebulous... I never had the motivation to fully merge yarn bound into bondage; my philosophy around that was: Make bondage fully compliant with the yarn 2.0 spec specifically and put qol features on top of that into yarn bound. I would be happy for my project to be subsumed by the main bondage repo but there's just questions of like how to name things lol and I'm not interested in doing the work
I don't personally intend on keeping up with the yarn language as it evolves, so I agree there should be a path forward there. I'm happy to talk through potential ideas

Yes, it's a common problem where maintenance tries to get passed off between hands as people have time and interest. I am a maintainer at the Poimandres code collective, one solution would be to move this project there with an eye toward modernizing the features. It would help it get visibility and a pool of maintainers who have a common interest in JS game dev. My work on this will eventually end up in a React Three Fiber game engine there.

Well it's not my place to say go ahead and move it there, but I'll just say that I personally have no protective inclinations and inasmuch as I can say, "do whatever you want with any code I've written (as of now at least) for free and no strings attached", I do.


[bug] Sensative Strips Guard 700 not responding on versions > 6.0.3

Version
Checklist:

[ x ] a developer has told me to come here.
[ x ] I have checked the troubleshooting section and my problem is not described there.
[ x ] I have read the changelog and my problem is not mentioned there.

Build/Run method

[ x ] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

Zwavejs2Mqtt version: 6.2.0
Z-Wave JS version: 8.9.1
Describe the bug
Sensative Strips Guard 700 window/door sensors unresponsive after installing any version above v6.0.3
To Reproduce
Steps to reproduce the behavior:

Install any release > 6.0.3
Open window/door with sensative strips guard 700
Device does not respond

Expected behavior
Device responds
v620__zwavejs2mqtt-store.zip
v603__zwavejs2mqtt-store.zip

In logs I see lot of

2021-12-29T13:16:49.280Z DRIVER « [Node 067] [REQ] [ApplicationCommand]
└─[Security2CCMessageEncapsulation] [INVALID]
error: Security2CC_CannotDecode
2021-12-29T13:16:49.281Z CNTRLR » [Node 067] Message authentication failed, cannot decode command. Requesting a
nonce...

Did you changed your security S2 keys?

No, but I will verify that the keys are the same between versions.

attempt_2.zip
Keys stayed the same between versions. Device responds to one "open/close" and then goes unresponsive.

I think the main problem is that the device keeps re-sending the same report for minutes. I can't say if this is the one:
2021-12-29T13:15:29.321Z DRIVER « [Node 067] [REQ] [ApplicationCommand]
                                  └─[Security2CCMessageEncapsulation]
                                    │ sequence number: 195
                                    │ extensions:
                                    │ · type: SPAN
                                    │   sender EI: 0x83d4dd8b1cdee93778c8a314bdaab62c
                                    └─[SupervisionCCGet]
                                      │ session id:      36
                                      │ request updates: false
                                      └─[NotificationCCReport]
                                          notification type:   Access Control
                                          notification status: 255
                                          notification state:  Window/door is open

but even minutes later you're having this in the logs:
2021-12-29T13:21:08.347Z DRIVER « [Node 067] [REQ] [ApplicationCommand]
                                  └─[Security2CCMessageEncapsulation] [INVALID]
                                      error: Duplicate command

Per the S2 specifications, the driver MUST discard duplicate commands (which are detected by looking at the sequence number).

Node 67 is one of about 10 such nodes — I’m curious if the duplicate message is interpreted by the software as any message from the same node with the same attributes — which would explain why the switch changes states once and never more than once on versions greater than 6.0.3.

Duplicate message detection is done on a per-node basis. In the case of S2-encapsulated messages, a message is a duplicate if its sequence number (which must be increased for each message) is in the list of the last 10 received sequence numbers from that node.

I find it interesting that I'm having issues with my Sensative Strips Guard 700 as well since first updating to an 8.9.x-beta (accidentally, using the HA addon). I'm not sure it's the same issue, because I don't seem to see those duplicate commands anymore, but it looks like a suspicious amount of coincidence.
For me it appears that open/close still triggers a battery state update, but no access control change. Reinterviewing affected nodes leaves the access control value as "undefined". Only a local reset seems to such nodes to respond normally again (at least once).
(If that sounds at all similar, I can elaborate what I tried etc., but I don't want to spam the ticket if it is only a coincidence after all.)

Just post a driver log highlighting your issues and we can check.

Sure. Here's today's log. Node 87 is one of the working ones, node 97 a non-working one:
zwavejs_2022-01-05 3.log
I got a new firmware from Sensative (10.28 instead of 8.26), but so far only managed to install it on a single device (not in this log, there's is only one unsuccessful attempt on node 87). In all other cases, I got "Duplicate command" errors early in the process.

Node 97 seems to send this notification instead:

notification state:  Magnetic field interference detected

I'm not sure what the implication of this is, but it could mean that it is unable to detect the open/close status. Maybe an issue mounting the sensor?

New 10.28 Sensative firmware initially appears to resolve the issue based on testing 2 devices in 6.2.0 after updating firmware via 6.0.3. Rolled back to 6.0.3 after initial tests to update the firmware on the rest of the devices. Will confirm once complete.

Yes, the magnetic interference is from manually waking the node with the magnet (to update its firmware, only to later realize that Z2M hadn't started the process - apparently only every second FW selection really triggers the process).
My current problem is that except for one device, all update attempts fail with lots of "Duplicate command" errors. I can't roll back easily because the node IDs have changed since I upgraded the HA addon. Since I do own a "second" controller now (the 700 series), I might set up a separate Pi with manually installed Z2M - is there a checklist somewhere for manual installation?

all update attempts fail with lots of "Duplicate command" errors

If you're able to, bring the node close to the controller, perform a node heal so it knows the new routes and then do the update.

Unfortunately, that's not really possible as the strips are permanently mounted. This also happens with nodes in the same room (direct line of sight to the controller).

I'm going to need some logs of these failed communication attempts then.

Here you are: zwavejs_2022-01-06 2.log

The two duplicate commands have the sequence numbers 123 and 124. 123 should have been received shortly before your snippet, 124 is the magnetic sensor report. Both seem to expect a SupervisionCCReport in return which have been sent and acknowledged by the node here:
2022-01-06T16:20:39.849Z DRIVER » [Node 087] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      13
                                  └─[Security2CCMessageEncapsulation]
                                    │ sequence number: 67
                                    └─[SupervisionCCReport]
                                        session id:          32
                                        more updates follow: false
                                        status:              Success
                                        duration:            0s
2022-01-06T16:20:39.861Z SERIAL « [ACK]                                                                   (0x06)
2022-01-06T16:20:39.865Z SERIAL « 0x0104011301e8                                                       (6 bytes)
2022-01-06T16:20:39.866Z SERIAL » [ACK]                                                                   (0x06)
2022-01-06T16:20:39.868Z DRIVER « [RES] [SendData]
                                    was sent: true
2022-01-06T16:20:39.878Z SERIAL « 0x011800130d00000200ad7f7f7f7f000003000000000301000057              (26 bytes)
2022-01-06T16:20:39.879Z SERIAL » [ACK]                                                                   (0x06)
2022-01-06T16:20:39.881Z DRIVER « [REQ] [SendData]
                                    callback id:            13
                                    transmit status:        OK, took 20 ms
                                    routing attempts:       1
                                    protocol & route speed: Z-Wave, 100 kbit/s
                                    ACK RSSI:               -83 dBm
                                    ACK channel no.:        0
                                    TX channel no.:         0
2022-01-06T16:20:39.901Z SERIAL » 0x0118001357119f03440072f894841825f8a377ee5af719250e90              (26 bytes)
2022-01-06T16:20:39.902Z DRIVER » [Node 087] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      14
                                  └─[Security2CCMessageEncapsulation]
                                    │ sequence number: 68
                                    └─[SupervisionCCReport]
                                        session id:          33
                                        more updates follow: false
                                        status:              Success
                                        duration:            0s
2022-01-06T16:20:39.907Z SERIAL « [ACK]                                                                   (0x06)
2022-01-06T16:20:39.912Z SERIAL « 0x0104011301e8                                                       (6 bytes)
2022-01-06T16:20:39.912Z SERIAL » [ACK]                                                                   (0x06)
2022-01-06T16:20:39.914Z DRIVER « [RES] [SendData]
                                    was sent: true
2022-01-06T16:20:39.935Z SERIAL « 0x011800130e00000200b07f7f7f7f000003000000000301000049              (26 bytes)
2022-01-06T16:20:39.937Z SERIAL » [ACK]                                                                   (0x06)
2022-01-06T16:20:39.939Z DRIVER « [REQ] [SendData]
                                    callback id:            14
                                    transmit status:        OK, took 20 ms
                                    routing attempts:       1
                                    protocol & route speed: Z-Wave, 100 kbit/s
                                    ACK RSSI:               -80 dBm
                                    ACK channel no.:        0
                                    TX channel no.:         0

I'm not sure why they are repeated despite being received.
Also, the sensor does not request any firmware update frames, although it has confirmed the command requesting it to do so:
2022-01-06T16:20:40.018Z DRIVER « [Node 087] [REQ] [ApplicationCommand]
                                  └─[Security2CCMessageEncapsulation]
                                    │ sequence number: 127
                                    └─[FirmwareUpdateMetaDataCCRequestReport]
                                        status: OK

Firmware updates are pulled by the nodes, so if they don't request fragments, there's nothing the driver can do.
You could try re-including the node insecurely and then try the update again - maybe there's something weird in the S2 implementation of that firmware.

Might be a firmware bug? Anyway, something has changed in how the driver interacts with that possibly buggy firmware, because I set up a separate Pi4 with Z2M 6.0.3 (using driver version 8.8.3) and moved the stick to that machine. Lo, and behold: FW updates were finished in half an hour for all remaining strips (whereas I had tried to update them using Z2M 6.2.0/driver 8.9.1 for two days in a row and only managed 2 out of 8 in that time).


Remove all elements from blacklists in source_code_test.py

We enforce some rules in source_code_test.py about what API we can't use in addons. (no private api, no tf.compat.v1, and no experimental API in user-facing code).
We didn't enforce it in some files because they didn't respect the rules before the check was put into place. Replacing the problematic code takes some time, so this is why there are blacklists now. To track the progress. Replacing some APIs by public ones can be very easy or very hard depending on what the code is about. We'll gladly take pull request removing some elements from the blacklists. When we remove all elements, we can remove the blacklists entirely.
It's possible to copy the code needed into addons, or to find another similar API as long as the effect is the same.

#1324
Rollback?

@fsx950223  I think we need to copy the code of the wrapper in Addons (we may already have some of it) and use that instead.

@fsx950223 I think we need to copy the code of the wrapper in Addons (we may already have some of it) and use that instead.

It's difficult if the API depends on lots of internal APIs.

We can limit the functionality to our use cases if that can help. For example, we don't support tf.compat.v1. The wrapper in TF seems fairly small, I think we should be able to port it.

TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


[New game]: Math quiz

🎮 Game Request
The game will generate random mathematical questions, the player has to give as many correct answer as possible in 30 sec.
Point down the features
To every correct ans 1 point is awarded.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @bhumika-1127 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

hi please assign me this task.

Hey @bhumika-1127 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


schedule dencun on holesky

see:

https://github.com/ethereum/EIPs/pull/8051

I think the cancun timestamp should be added to the EL genesis (genesis.json / besu.json / chainspec.json) too?
"cancunTime":<PHONE_NUMBER>

I think the cancun timestamp should be added to the EL genesis (genesis.json / besu.json / chainspec.json) too?
"cancunTime":<PHONE_NUMBER>

Ahh yes this has full custom config, makes sense will update 👍

this config's missing MAX_PER_EPOCH_ACTIVATION_CHURN_LIMIT: 8

made a PR for it here https://github.com/eth-clients/holesky/pull/95


Source maps in the npm package reference nonexistent files

Versions

0.3.4

Describe the bug
The source map comments included in the npm package point to source files that don't exist in the package. For example, build/module/index.js has the following contents:
import Plausible from './lib/tracker';
export default Plausible;
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoiaW5kZXguanMiLCJzb3VyY2VSb290IjoiIiwic291cmNlcyI6WyIuLi8uLi9zcmMvaW5kZXgudHMiXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6IkFBQUEsT0FBTyxTQUFTLE1BQU0sZUFBZSxDQUFDO0FBSXRDLGVBQWUsU0FBUyxDQUFDIn0=

sourceMappingURL decodes to this JSON:
{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,eAAe,CAAC;AAItC,eAAe,SAAS,CAAC"}

which indicates that the original TypeScript source can be found at ../../src/index.ts. But the src/ directory doesn't exist at all in the package! This causes a warning from source-map-loader when run from a downstream packages and makes this code harder to debug.
Expected behavior
The source maps should be well-formed: either they should refer to files that exist or they should embed the TypeScript source directly using a "sourcesContent" key.
Steps to reproduce

Add plausible-tracker 0.3.4 as a dependency to a project that runs source-map-loader as part of its build.
Build that project.
Observe three warnings like the following are printed.

WARNING in ./node_modules/plausible-tracker/build/module/index.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '.../node_modules/plausible-tracker/src/index.ts' file: Error: ENOENT: no such file or directory, open '.../node_modules/plausible-tracker/src/index.ts'
 @ ./src/analytics/plausible.ts 3:0-42 11:29-38
 @ ./src/index.tsx 8:0-55 13:0-14

The issue is resolved for me with v0.3.5. Thanks!


Formalize tracepoint parser

What does this PR do?
Raw events weren't enabled in the ringbuffer during the tracepoint PR.  This was an omission.
Motivation

We already had the syntax for specifying the offset into a raw event to use
raw events are needed to surface contextual information out of tracepoints
when raw events are specified, but we lack support to process them, there's no way to maintain ringbuffer state (the events are variable-length) so we segfault

Additional Notes
How to test the change?
Test added to the perf ringbuffer ut

Should we consider this one in the upcoming release ?

I think this can go in the next release (also it makes my life easier if we merge it). Can we do a small session on this PR ?

I tried to update the build, let's see if this merge works!


Await timer does not work in a for loop

Godot version
4.0.2.stable.official [7a0977ce2]
System information
MacOS Ventura 13.3.1, 3.4 GHz Quad-Core Intel Core i5
Issue description
This is my first bug report so i hope its valid.
Im making a screenshake effect with a for-loop and the new await keyword but the debug window just freezes or just completely skips the code.
Steps to reproduce
func _ready():
for i in 10:
print(i)
await get_tree().create_timer(1).timeout
Minimal reproduction project
test project.zip

Can you confirm this happens only if it is in a loop? Does it happen if done in some other context than _ready?

@henryernice
There is a logic problem in your code:
await get_tree().create_timer(1.0/i).timeout

The first time it get's there, i == 0, resulting in "infinite float" because you cannot divide by zero, so the timeout's signal will never be called.

Didn't look in the MRP so didn't see the code was different from the issue report

Sorry i didnt know that was a problem. The real problem was here:
func screenshake(amount):
randomize()
var strenght = amount
var cam_pos = $Head/Camera3D.position
var xnoise = FastNoiseLite.new()
var ynoise = FastNoiseLite.new()
xnoise.seed = randi()
ynoise.seed = randi()
xnoise.frequency = 4
xnoise.fractal_octaves = 5
xnoise.fractal_lacunarity = 6
ynoise.frequency = 4
ynoise.fractal_octaves = 5
ynoise.fractal_lacunarity = 6

for i in range(int(amount)):
	$Head/Camera3D.position = cam_pos + (Vector3(xnoise.get_noise_2d(i, 0), ynoise.get_noise_2d(i, 0), 0) * amount*0.03/i)
	await get_tree().create_timer(1.0/amount).timeout

$Head/Camera3D.position = cam_pos

Sorry if this is just a waste of your time, next time i will ask on the Godot forums first


Improve and fix --platform help.

Fixes #997.

Help now renders on my machine as:
   --platform=PLATFORMS
                        The platform for which to build the PEX. This option
                        can be passed multiple times to create a multi-
                        platform pex. To use the platform corresponding to the
                        current interpreter you can pass `current`. To target
                        any other platform you pass a string composed of
                        fields: <platform>-<python impl abbr>-<python
                        version>-<abi>. These fields stem from wheel name
                        conventions as outlined in
                        https://www.python.org/dev/peps/pep-0427#file-name-
                        convention and influenced by
                        https://www.python.org/dev/peps/pep-0425. For the
                        current interpreter at /usr/bin/python3.8 the full
                        platform string is manylinux2014_x86_64-cp-38-cp38

Adding Benjy and Stu because this topic has confused users quite a bit in the past. Relates to the incomplete docs at https://pants.readme.io/docs/python-platforms.


[enhancement](jdbc catalog) Enhance function pushdown of Jdbc Oracle Catalog

Proposed changes
Issue Number: close #xxx
Enhanced function pushdown of Jdbc Oracle Catalog, currently supports nvl function
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


Using HTML in SFC results in parsing issue Unexpected scalar at node end

Reporting a bug?
I'm trying to use short bits of html for text formatting in Nuxt. However, the yaml-eslint-parser package that is used does not seem to allow that. It throws an error saying "Unexpected scalar token". I tried changing the formatting from yaml to json, but that does not seem to have any effect, since the yaml-eslint-parser seems to be used for both cases.
Expected behavior
Parsing the translation values regardless of its HTML content, when strictMessage is disabled.
Reproduction
https://github.com/casperboone/nuxt-i18n-sfc-html-bug-reproduction
Issue Package
vite-plugin-vue-i18n
System Info
System:
    OS: macOS 14.2.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 694.39 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.7.1 - ~/.nvm/versions/node/v21.7.1/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 10.5.0 - ~/.nvm/versions/node/v21.7.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 116.0.5845.140
    Safari: 17.2.1
  npmPackages:
    vue: ^3.4.21 => 3.4.21

Screenshot
No response
Additional context
Unexpected scalar token
at Context.throwError (<redacted>/node_modules/yaml-eslint-parser/lib/context.js:80:15)
at processDoc (<redacted>/node_modules/yaml-eslint-parser/lib/yaml-cst-parse.js:16:23)
at parseAllDocsToCST (<redacted>/node_modules/yaml-eslint-parser/lib/yaml-cst-parse.js:31:9)
at parseForESLint (<redacted>/node_modules/yaml-eslint-parser/lib/parser.js:13:57)
at parseYAML (<redacted>/node_modules/yaml-eslint-parser/lib/index.js:46:40)
at generate$1 (file://<redacted>/node_modules/@intlify/bundle-utils/lib/index.mjs:598:13)
at TransformContext.transform (file://<redacted>/node_modules/@intlify/unplugin-vue-i18n/lib/index.mjs:527:48)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async file://<redacted>/node_modules/vite-plugin-inspect/dist/index.mjs:377:17
at async file://<redacted>/node_modules/vite-plugin-inspect/dist/index.mjs:377:17 (x2)

Validations

[X] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

This is happening because of devtools
Debugging the parameters received here at yamlParse
found this in my local project
the value is equal to:
text: "My text with html <b data-v-inspector="components/exmaple/index.vue:20:33">{text}</b>"

Devtools parsed  as html template
This prevent the error until it be fixed
// nunxt.config.ts
...
devtools: {
  enabled: true,
  componentInspector: false,
}
...


Could not connect to development server. Using Android emulator.

I just started a new project, with latest version of node.js, react, react-native, react-native-cli. My old projects were using some older versions of those packages and were working fine.
react-native init
cd
react-native run-android
I see the app gets installed on the emulator. When the app launches on the emulator after some time it give the error "Could not connect to development server".
My package.json:
{
  "name": "MyProjectName",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "react": "15.2.1",
    "react-native": "0.30.0"
  }
}

Output of "adb devices":
List of devices attached
emulator-5554   device

My Android sdk's are fairly new, but maybe I will try to reinstall all that stuff per react-native docs.
Btw, the react-native docs don't specify how to start the emulator. I start the emulator by launching Android Studio, and opening the AVD manager. This approach seems to work with my old projects.
Please let me know what other information is needed.

Also, the packager server is running on port 8081.

I think typically there is a request sent by the emulator when it is "Fetching JS Bundle" and that request is logged in stdout of the packager server, but now I do not see any request hitting the server.

Turns out the TermService was using port 8081. Imo react native packager server should throw an error if another process is using the port it wants.
Ticket can be closed.

@joshg111 Could you explain what you mean by TermService - or how you ended up fixing this? I am experiencing the same issue but have little experience on Android.

I'm on windows so I ran some command to get used ports.
netstat -a -b > out
Then I opened "out" and found that the TermService was using port 8081. TermService is used for windows remote desktop protocol. Turns out I coincidentally used port 8081 for remote desktop setup and didn't realize it would conflict with react native. I changed the port for remote desktop back to the default in regedit, and restarted, then everything worked fine.

@joshg111 for me it was also the port... potgresSQL.exe, for months and months i have been working on emulator and downloading 'release' apk's to my phone because debugger couldn't connect. Thanks Josh!!!


Move TLS config out of server blocks

Hey,
I think it'd be nice to move all the TLS specific parameters out of /home/dokku/*/nginx.conf.
My reasoning is that I'd like to manage the TLS config myself without it getting overridden by dokku-alt.
That let's me react to security flaws more quickly .
I suggest moving
  keepalive_timeout   70;
  add_header Alternate-Protocol 443:npn-spdy/2;
  ssl on;
  ssl_ciphers CIPHERS...
  ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
  ssl_prefer_server_ciphers on;
  ssl_session_timeout 5m;
  ssl_session_cache shared:SSL:50m;

to /etc/nginx/conf.d/dokku-tls.conf or something
So you just specify the cert and key in every /home/dokku/*/nginx.conf
  ssl_certificate     $SSL_INUSE/server.crt;
  ssl_certificate_key $SSL_INUSE/server.key;

Further I suggest using the following configuration for TLS (as per istlsfastyet.com with the cipher suite from https://wiki.mozilla.org/Security/Server_Side_TLS)
spdy_keepalive_timeout 300; # up from 180 secs default
spdy_headers_comp 6;

ssl_protocols TLSv1.2 TLSv1.1 TLSv1;

ssl_session_cache shared:SSL:10m; # 10MB -> ~40,000 sessions.
ssl_session_timeout 24h;
ssl_buffer_size 1400; # 1400 bytes to fit in one MTU

ssl_ciphers ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384:DHE-RSA-AES128-GCM-SHA256:DHE-DSS-AES128-GCM-SHA256:kEDH+AESGCM:ECDHE-RSA-AES128-SHA256:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA:ECDHE-ECDSA-AES128-SHA:ECDHE-RSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA:ECDHE-ECDSA-AES256-SHA:DHE-RSA-AES128-SHA256:DHE-RSA-AES128-SHA:DHE-DSS-AES128-SHA256:DHE-RSA-AES256-SHA256:DHE-DSS-AES256-SHA:DHE-RSA-AES256-SHA:AES128-GCM-SHA256:AES256-GCM-SHA384:AES128-SHA256:AES256-SHA256:AES128-SHA:AES256-SHA:AES:CAMELLIA:DES-CBC3-SHA:!aNULL:!eNULL:!EXPORT:!DES:!RC4:!MD5:!PSK:!aECDH:!EDH-DSS-DES-CBC3-SHA:!EDH-RSA-DES-CBC3-SHA:!KRB5-DES-CBC3-SHA;
ssl_prefer_server_ciphers on;

ssl_session_tickets off;

ssl on; is not needed as it is already set in the listen directive. I'm not sure about add_header Alternate-Protocol 443:npn-spdy/2;? Maybe it's needed for older nginx versions?
What do you think about this?
Thank you for dokku-alt,
Clemens

I'll get to work on a PR for this. To clarify I'm going to pull the SSL config (besides the actual cert specification) out of the per site .conf files and put it into a separate .conf file in /etc/nginx/conf.d? Then that file can be overridden as necessary by whoever is managing the server?

I've taken a shot at this in the attached pull request, but I didn't have time tonight to sort out how to test a debian package. If you could point me in the right direction for testing that I'm happy to try tomorrow.


Organizing Broker's performance test benchmars

Related to #408 #612

Changes

🧹 Added performance tests directories for each kind of tests
🎁 Added results images for different prefetch values
🧹 Added procedure and link to the readme
🧹 Fixed typos
🧹 Bumped resources versions in readme

/kind performance

Fixes #

Release Note

Now performance graphs for the RabbitMQ Broker are going to be generated for every mayor version release inside our performance test directories

/approve


disappear create and update buttons

When i add item and want to add some new related items, after adding one, create and update button desappeared. Why? Am i doing smth wrong?

You can't open a new modal from a modal.
It lead to various JS issues and I couldn't maintain it.
Support may come back, but not right now.


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app/src/atom.js:53:11)
StatusBarSummaryView.render (/Users/marchon/.atom/packages/linter/lib/statusbar-summary-view.coffee:11:21)

This is fixed in the latest version of linter; please upgrade.


[paho-mqtt] Update to version 1.3.5

Update paho-mqtt to version 1.3.5 and fix old patches.
Related #13536

This looks great. Thanks Lily!


Alt + DownArrow problem

We have written the needed data into your clipboard because
Issue Type: Bug
Alt + DownArrow should move the selected lines downward. If need, intendation should be applied.
But I get intendation even if it is not necessary.
For example in a json file
"a": 'first line',
"b": "second line",
would become
"b": "second line",
"a": 'first line',
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz (4 x 3000)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.89GB (9.35GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

All-Autocomplete
Ati
0.0.18

typescript-angular-plugin
cyr
1.2.0

jshint
dba
0.10.20

vscode-eslint
dba
1.8.0

gitlens
eam
9.5.1

tslint
eg2
1.0.43

vscode-npm-script
eg2
0.3.5

vsc-material-theme
Equ
2.7.0

beautify
Hoo
1.4.9

vscode-power-mode
hoo
2.2.0

reg
ion
0.0.2

run-in-terminal
kor
0.0.1

angular1-code-snippets-johnpapastyle
lpe
0.1.0

theme-monokai-pro-vscode
mon
1.1.10

cpptools
ms-
0.21.0

material-icon-theme
PKi
3.6.3

markdown-preview-enhanced
shd
0.3.11

gitconfig
sid
2.0.0

vscode-smoothtype
spi
1.2.0

unique-window-colors
stu
1.0.23

sort-lines
Tyr
1.8.0

vscodeintellicode
Vis
1.1.3

JavaScriptSnippets
xab
1.7.2

it was too large to send. Please paste.

And the problem is gone. Don't know how. Reloaded the window. It went back to what it was supposed to do.


Remove relayer fee from expected price check

Looks good, we probably also want to test and document this in each modal's docs.


added § (paragraph) symbol

Hey @toitzi thanks so much for the contribution! Would you mind updating this PR to match the new, less merge-conflict-prone format introduced in #29 please? 🙏


Fix ambiguity of promote_rule in Julia 1.0 (< 1.3)

Codecov Report

Merging #31 into master will decrease coverage by 1.67%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   90.56%   88.88%   -1.68%
==========================================
  Files           2        2
  Lines          53       54       +1
==========================================
  Hits           48       48
- Misses          5        6       +1

Impacted Files
Coverage Δ

src/InitialValues.jl
84.61% <0.00%> (-2.23%)
:arrow_down:

Continue to review full report at Codecov.

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


[fastpages] API

Collegeboard Notes
Project Roadmap and Team Plan
RapidAPI --> Cities for potential colleges
Burger,Fries,Drink Challenge

Akshat did a great job of completing every task thoroughly, and his effort is seen through how thorough each task is. I grade him to a 2.7+


get-airnow xarray error

melodies-monet get-airnow --start-date 20240521 --end-date 20240522
results in an xarray error
Fetching data with monetio ...
Aggregating AIRNOW files...
Building AIRNOW URLs...
Adding in Meta-data
Fetching data with monetio succeeded in 0.619 seconds
Forming xarray Dataset ...
Forming xarray Dataset failed in 0.00887 seconds
Error message: single positional indexer is out-of-bounds

@rschwant What is the task here? Is it improving the error message, as Zach mentioned?

I'm not sure. I think this was resolved right @zmoon. I think David was looking for data that was not yet available as there is a delay for uploading the data to the amazon servers we use for downloading.

We have yet to improve the error message. It's the .iloc[0] here that causes this error when there is no data for a requested period.
https://github.com/NOAA-CSL/MELODIES-MONET/blob/b243d099e57376a6d591d8530063f2c67ec9eaae/melodies_monet/_cli.py#L451
So, just need to check if it's empty and print appropriate error before attempting to create the units dict.
There's another confusing error you can get it you request when the files on S3 are in the midst of being updated, Dask complaining about files not matching dtype/cols.


Roll engine ff49ca1c6e5b..708fc625d715 (10 commits)

git@github.com:flutter/engine.git/compare/ff49ca1c6e5b...708fc625d715
git log ff49ca1c6e5b..708fc625d715 --no-merges --oneline
2019-08-09<EMAIL_ADDRESS>rename stub_ui to web_ui (flutter/engine#10776)
2019-08-09<EMAIL_ADDRESS>[fuchsia] Migrate from custom FuchsiaFontManager to SkFontMgr_fuchsia (flutter/engine#10700)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 26cedb4c603b..b7b0b3ad538c (11 commits) (flutter/engine#10807)
2019-08-09<EMAIL_ADDRESS>Rename flutter_java.jar and flutter_engine.jar so they match the Maven comvention (flutter/engine#10797)
2019-08-09<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from 9xVXx... to HsuAH... (flutter/engine#10806)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/dart 3d9a356f6e..78ce916d82 (7 commits)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia b2151310068b..26cedb4c603b (1 commits) (flutter/engine#10804)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 059bf776c184..b2151310068b (3 commits) (flutter/engine#10803)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/dart f29f41f1a5..3d9a356f6e (65 commits)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 4d557e3df433..059bf776c184 (1 commits) (flutter/engine#10801)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jimgraham@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6287411800178688 , https://cirrus-ci.com/task/5724461846757376

Commit queue failed; closing this roll.


setTimeout() can disrupt future page loads

This is actually two related bugs, but the combination makes it extremely confusing what's going on. (I can split them if that would make it easier; but since the fix of the first bug makes the second go away, it's probably not necessary).
Tested on version 2.33
First bug
The basic problem of the first bug: When a heavy javascript function is triggered by setTimeout(), and a new page is loaded, this function will continue to run on the new page.
Example:
page1.html:
<!DOCTYPE html>

Page 1

Page 2

page1.js:
function loadExtraContent()
{
	for (var i = 0; i < 100; i++)
	{
		var p = document.createElement("p");
		p.innerHTML = "new content!";
		var body = document.querySelector("body");
		if (body)
			body.appendChild(p);
	}
}
setTimeout(loadExtraContent, 1);

page2.html:
<!DOCTYPE html>

Page 2

This is page 2

(ignore the js for now, it is only important for the second bug)
Test:
try (WebClient webClient = new WebClient(BrowserVersion.CHROME)) {
	// Load page 1. Has a setTimeout(...) function
	HtmlPage page = webClient.getPage("file:page1.html");

	// Immediately load page 2. Timeout function will be triggered already
	page = webClient.getPage("file:page2.html");

	// Wait for javascript
	webClient.waitForBackgroundJavaScriptStartingBefore(100);

	// Fails: return 98 (about) instead of 1
	assertEquals(1, page.querySelectorAll("p").size());
}

The "loadExtraContent" method is started upon loading page 1. When we load page 2, it is still running. The if (body) clause is there, because there is a tiny window in which the new body is not loaded yet.
It is important that the timeout function is started before the second page is loaded, because loading a new pages causes all queued jobs in the JavaScriptJobManagerImpl to be cancelled (side effect of WebWindowImpl.setEnclosedPage() -> destroyChildren(). Also, a javascript job holds a lock on the HtmlPage, so using WebClient to just click on the "Page 2" link, would cause that click to be blocked until the execution has ended.
Suggested fix: make WebWindowImpl.setEnclosedPage synchronized on the current enclosedPage, or some other mechanism to make it block on the current running HtmlPage.
Second bug
Basic problem of the second bug: when a script modifies the DOM tree, it prevents other scripts from loading.
Continued example:
page2.js:
function setHeader()
{
	document.querySelector("h1").textContent = 'Loaded by JS';
}
setTimeout(setHeader, 50);

Test:
try (WebClient webClient = new WebClient(BrowserVersion.CHROME)) {
	// Load page 1. Has a setTimeout(...) function
	HtmlPage page = webClient.getPage("file:page1.html");

	// Immediately load page 2. Timeout function will be triggered already
	page = webClient.getPage("file:page2.html");

	// Wait for javascript
	webClient.waitForBackgroundJavaScriptStartingBefore(100);

	HtmlHeading1 h1 = page.querySelector("h1");
	assertEquals("Loaded by JS", h1.getTextContent());
}

The problem is here that javascript is loaded by HtmlScript.onAllChildrenAddedToPage(), which runs executeScriptIfNeeded(), which checks isExecutionNeeded(). The 4th test here is htmlPage.isParsingHtmlSnippet(). This one is only triggered by DOM modifications like element.innerHTML = x. So if, due to the first bug, a background script is still running and modifying the DOM, this can cause some scripts to not load. This gives really confusing error messages, and sometimes even internal HtmlUnit errors (since two processes are modifying the DOM concurrently).
This problem is really hard to trigger. This specific example fails almost never, but on our development environment, it fails often, since we load a lot of scripts which depend on each other.
One way to specifically make this example fail by forcing a specific thread interleaving:

Set debugger breakpoints on HtmlScript.isExecutionNeeded() and HtmlPage.registerSnippetParsingEnd().
Start the test
First only the isExecutionNeeded will be triggered for file1.js. Continue.
Next both breakpoints will be triggered, for the loading of file2.js and the first p.innerHTML. By this point, p will probably still refer to the old document/page, so continue the javascript thread.
The breakpoint will be hit again for the second iteration. This time the p will refer to the new document/page. Continue the main thread.
The main thread will now be blocked on waitForBackgroundJavascriptStartingBefore(...), so remove the HtmlScript breakpoint and continue the javascript thread.
The test will now fail with org.junit.ComparisonFailure: expected:<[Loaded by JS]> but was:<[Page 2]>

Suggested fix: Same as the previous one.

Wow, cool finding and great description. Will work in this during the next days, but this requires some time to understand and find a proper fix.

Have done some research on this, but seems like the implementation will be tricky. The synchronization handling in HtmlUnit is a tricky monster already, adding some more sync might end in deadlocks. have to think and test a bit more....

Thanks for looking into it and I appreciate the compliment!
Luckily the workaround is not too hard (WebClient.waitForBackgroundJavascriptStartingBefore(...)) or just close all the windows before the next test.
The biggest problem was mostly finding out wth is going on. It took me quite a few hours before finding out the problem was a background job, and a few more hours to find out how it was affecting the new pages. Maybe something dumb as logging a warning when there is still a currentRunningJob during JavaScriptJobManagerImpl.removeAllJobs() or WebWindowImpl.destroyChildren() would already be helpful.


Flags for native OSX build of AU

https://github.com/DISTRHO/DPF/blob/a291737aef87bfbac914f004bcf3f36bea80affc/Makefile.plugins.mk#L355
This line wont do much.. it should be -isysroot ... not -I ... but I think it would break cross compiling.

Added $(REZ_FLAGS), issue closed


About Spatial Jitter

Thanks for the repo. In the supplemental material, you guys said to augment data by spatial jittering in the ground truth. Can you give a bit more details on this? Like how do you generate the extra ground truth samples? Also, you mentioned the human and object stream losses are calculated on the 16 positive triplets, what does mean by that?

Each GT H-O pair has H_box and O_box. We randomly jitter these bounding boxes, i.e. translation, changing the ratio, but making sure the IOU between these augmented boxes and the GT boxes is larger than a threshold. We then pair the augmented H_box and O_box, and regard these pairs as positive training data as well
To train the human stream, at each iteration we focus on one H-O pair. This gives us one H_box. We can also borrow 15 additional augmented H_boxes. We train the human stream with these (1+15=16) H_boxes. Thus the batch size is 16 (even though all 16 boxes are about the same H).
Please let me know if I explain the procedure clearly. Thanks.

Thanks for your response. I have another question related to your answer, did you train the human stream with detected bounding boxes from Fast RCNN+augmented_boxes or ground truth boxes+augmented_boxes?

Ground truth boxes+augmented boxes.

Thank you.


Design and architecture

Hello everyone!
Since it's a long time I don't contribute to this project, I kind lost the sense of it. I saw there are a lot of ideas but the status of works is not changed a lot. So here I am with some questions.

What is the final goal of EddieBotUI?
Do we have a design for core features?

@eddiejaoude

We should dedicate some time to formalising these things to have a clearer and common vision about this project - that I find really interesting.
Above you can find a small design I did, just to have an idea about the architecture and the code organisation.

Hi @rossanodan 😃, imo the final goal of EddieBotUI is to have a deployed website that displays all kinds of info about the community like bios, stats, badges, etc. We have some GitHub actions that collect data to the DB, so that info can be on this UI.
As for design for core features, I don't think we have it. We started the repository with React, CSS and Firebase Firestore, then we experimented with Semantic UI React and we were going to move to Bootstrap. But now there will be a DB switch I believe, and the community website uses Tailwind CSS. So I think we can re-think the stack choice and see what everyone else is also comfortable using 😃.
I'd also like to hear the opinion of anyone who is interested in contributing to a front-end project like this one, and @eddiejaoude's vision of EddieBotUI 😄.

Hi @rossanodan 😃, imo the final goal of EddieBotUI is to have a deployed website that displays all kinds of info about the community like bios, stats, badges, etc. We have some GitHub actions that collect data to the DB, so that info can be on this UI.
As for design for core features, I don't think we have it. We started the repository with React, CSS and Firebase Firestore, then we experimented with Semantic UI React and we were going to move to Bootstrap. But now there will be a DB switch I believe, and the community website uses Tailwind CSS. So I think we can re-think the stack choice and see what everyone else is also comfortable using 😃.
I'd also like to hear the opinion of anyone who is interested in contributing to a front-end project like this one, and @eddiejaoude's vision of EddieBotUI 😄.

@BOLT04 I updated the issue description. Please, take a look and tell us what you think :)

@rossanodan that's pretty cool 👍, one tweak I'd suggest is to instead of having a firebase folder, calling it dataAccess or data-access. The reason would be to abstract that implementation detail of it being Firebase, Mongo, a RESTful API, etc. The UI components shouldn't care where the data comes from.
I also have two notes about the design:

on that design drawing (which is dope btw 👍), is that it's not completely clear where the components get their data from, since there is no arrow pointing to the Firebase box
I'm assuming that folder for components are the re-usable components that can be used by any component, and then we have feature components - specific components for each use case/feature (in React's docs they have that). If that is correct, we could document that on the README so that is not an assumption on our design, and it becomes explicit in the documentation 😃.

Great explanation @BOLT04 💪  and great Issue with diagram too @rossanodan  😍
I am have not used React much, so I will go with what you both think is best. Let's avoid Redux though please.
Some of my thoughts though:

We use TailwindCSS on the website, I think it would be good to use that as it made it easier for others to contribute (I think there is a PR for this already, I am still going through my GitHub notifications)
I think I saw yarn mentioned in Discord, I think it would be good to stick to npm to keep the barrier to entry as low as possible
If the preference is React that is fine, I think on the website I might use Vue instead of Angular (and I will move the BotMap away from Angular I think too, as it seems to put people off) - but I dont see this to be an issue
The DB is currently Firebase (Firestore), but I really think we will move away from this to lower the barrier to entry for people. It makes it easier to run it locally, without an internet connection, without an account on firebase and also to write automated testing - not sure which DB yet, something like mongo/postgres/elastic, but we can discuss the DB we move to in another issue
With design, could we do a low fidelity approach, to add value to the community sooner? I mean have it look good, but doesn't need to be perfect and it can be improved on - as I will take your guidance on this

Did I miss anything, let me know 🤓

About the drawing
Thanks @eddiejaoude and @BOLT04! I hope it's useful. It's just a draft, to explain my concepts and to support the conversation. A proper design can be included in some .md file in the repo, of course.
About components and feature-homepage (feature-components)
Since we're going to use Tailwind CSS, it does not make sense having the "shared components" folder. Components we will use will come from the CSS framework.
The feature-xxxxx is a folder that contains a big chuck of the project - see home page, 'user profile`, etc - that makes sense to have grouped in folders besides CSS and tests. For example
./feature-homepage
  homepage.js //component
  homepage.css
  homepage.test.js

These features components are used by the router to show pages. More details can be discussed while implementing. This is just a general idea about the structure and code organisation.
About the data source
I notice @eddiejaoude said something about moving away from Firebase. But this is not a problem for the front end. If the backend can expose an endpoint with some APIs.. well, we're fine 😄  We can simply fetch data via REST or GraphQL. @eddiejaoude please take a look at this.

this is not a problem for the front end. If the backend can expose an endpoint with some APIs.

Yes there would be an API that all our Apps / Bots talk to

Consider this as a preliminary design. I'm going to create a new issue to be addressed for creating an official guide to design and architecture of EddieBotUI.
@eddiejaoude @BOLT04


Autofix.ci App Not Installed

Hi! I'm the autofix.ci troubleshooter bot.
It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix the main branch. If you are the repository owner, please install the app and then restart the CI workflow! 😃

Switched to husky, I close the issue.


Migration that getting bigger

I just wondering how we write migration strategy when schema version is getting bigger. Can anyone give me an example how to handle that situation?

@MuhammadDicky  This is what I do. It has been working great for me.

In my moor database file

onUpgrade: (Migrator m, int from, int to) async {
  print("UPGRADING $from   $to");

  if (to < from) {
    print("Trying to downgrade from $from to $to");
    throw Exception("Can't downgrade database");
  }

  await customStatement('PRAGMA foreign_keys = OFF');

  await transaction(
    () async {
      for (var i = from; i < to; i++) {
        await migrationsMap[i](m, this);
      }
    },
  );

  await customStatement('PRAGMA foreign_keys = OFF');
},

In a migrations.dart file

typedef MigrateFunction = Future<void> Function(Migrator, AppDatabase);

final Map<int, MigrateFunction> migrationsMap = {
  1: migrate1,
  2: migrate2,
  3: migrate3,
  4: migrate4,
  ...
};

Future<void> migrate1(Migrator m, AppDatabase db) async {
  print("Migration 1 to 2");
  ...
}

Future<void> migrate2(Migrator m, AppDatabase db) async {
  print("Migration 2 to 3");
  ...
}

That great, thanks for the input.

@MuhammadDicky Anytime!

What if someone going from 1 to 3?
version 1.0 of the app have a schema version 1
version 2.0 of the app have a schema version 2
version 3.0 of the app have a schema version 3
and someone has version 1.0 of the app and then he doesn't update the app until version 3.0 of the app comes
now he is updating from version 1.0 to 3.0 it means schema from 1 to 3.
please explain

@muhammadfaheem-pycom This is covered by the snippet David posted:
      for (var i = from; i < to; i++) {
        await migrationsMap[i](m, this);
      }

You can just apply the migration for 1 -> 2 and 2 -> 3 in sequence.


Added devise

Pull Request: Implement Devise Authentication
Description
This pull request addresses the project requirements to integrate Devise for user authentication in our blog app. The outlined tasks have been completed, and the implementation adheres to best practices and styling guidelines.
Changes Made

Removed the current_user method in ApplicationController as Devise provides its own.
Installed and configured Devise for user registration and login.
Implemented user registration functionality.
Enabled user login using a combination of email and password.
Ensured hashed passwords are stored securely in the database.
Implemented email confirmation and password reset functionalities.
Modified Devise views for registration and login to match the wireframes and styling.

Additional Notes

All linter errors have been addressed.
Gitflow has been followed for branch management.
Documentation has been updated to reflect the changes made.

BlogPress Demo Test after Devise authentication
Screencast from 2024-01-31 20-03-36.webm
Checklist

[x] All linter errors resolved.
[x] Gitflow followed correctly.
[x] Documentation updated.
[x] User registration and login functionalities tested.
[x] Devise views match wireframes and styling.

Hi @AlierPM and @Eccb7
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a pair programming project, that  does not have balanced commits from you and your partner,   as partners you are supposed to have balanced commit, @Eccb7 has only one commit on this pull request on the readme
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification @prolajumokeoni.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


[2.0] Allow Symfony 4 & Diactoros 2

This PR allows Symfony 4 by allowing 4.x versions of the only component required by us, which is symfony/options-resolver. I also took the liberty of switching from ~ to ^ in all the constraints, because it's the right one to use when following SemVer:

This is the recommended operator for maximum interoperability when writing library code.

I've also added support for Diactoros 2, since #693 is not feasible for now.