powershell GenerateResourcesAndImage script failed: "Provisioning step had errors: Running the cleanup provisioner, if present..." on packer.exe step

Describe the bug
We are trying to create a Windows2016 image by running this  script locally.
The script runs for few hours and at the end it fail on packer.exe with this error:
==> vhd: The resource group was created by Packer, deleting ...
==> vhd: Deleting the temporary OS disk ...
==> vhd:  -> OS Disk : 'https://aipdevtest2001.blob.core.windows.net/images/pkrosix6hu9iihy.vhd'
==> vhd: Deleting the temporary Additional disk ...
==> vhd: Provisioning step had errors: Running the cleanup provisioner, if present...
==> vhd: Removing the created Deployment object: 'pkrdpix6hu9iihy'
==> vhd: ERROR: -> ResourceGroupNotFound : Resource group 'packer-Resource-Group-ix6hu9iihy' could not be found.
==> vhd:
==> vhd: Removing the created Deployment object: 'kvpkrdpix6hu9iihy'
==> vhd: ERROR: -> ResourceGroupNotFound : Resource group 'packer-Resource-Group-ix6hu9iihy' could not be found.
==> vhd:
Build 'vhd' errored: Script exited with non-zero exit status: 1.Allowed exit codes are: [0]

==> Some builds didn't complete successfully and had errors:
--> vhd: Script exited with non-zero exit status: 1.Allowed exit codes are: [0]

==> Builds finished but no artifacts were created.

this is the command that the script calls:
 GenerateResourcesAndImage -SubscriptionId ******* -ResourceGroupName "AIPDevTest" -ImageGenerationRepositoryRoot "C:\mages" -ImageType Windows2016 -AzureLocation "East US" -GithubFeedToken *****************
Halfway through the script we checked and saw that the resource group was created and so is the VM. When the script failed, the resource group and the VM were both gone.
Area for Triage:

Question, Bug, or Feature?:
Bug
Virtual environments affected
Windows Server 2016
Expected behavior
Script finish successfully without errors.
Actual behavior
The script failed

The actual error is probably further up in the packer log and not actually when packer attempts to clean up after itself. I see these a lot when packer is used in a pipeline where there is no interaction with packer itself. I suspect you ran this via a pipeline?
Looking at the script you are using packer is run using packer.exe build -on-error=ask this is great for an interactive session but does not work well in an automated pipeline. If replace on-error=ask with on-error=abort packer will not attempt to clean up after itself and give you an opportunity to access the vm to triage further. The default is to clean up after itself. https://www.packer.io/docs/commands/build.html
The actual error will probably be further up in the packer output. Maybe if you can have a look or post the console output here?

Thanks for the quick response.
Actually, I didn't run it via pipeline, I ran it locally on my PC.
I looked at the packer output as you suggested and I found this error:
==> vhd: Provisioning with Powershell...
==> vhd: Provisioning with powershell script: c:\images\images\win/scripts/Installers/Install-WindowsUpdates.ps1
    vhd: Run windows updates
    vhd:
==> vhd: Get-WUInstall : Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))
    vhd: X ComputerName Result     KB          Size Title
    vhd: - ------------ ------     --          ---- -----
==> vhd: At C:\Windows\Temp\script-5e9f1a2e-92a1-91f0-9188-9815cca77108.ps1:9 char:1
==> vhd: + Get-WUInstall -WindowsUpdate -AcceptAll -Install -UpdateType Software ...
==> vhd: + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    vhd: 1 pkrvmix6h... Accepted   KB4537477   51MB 2020-02 Cumulative Update for .NET Framework 4.8 for Windows Server 2016 ...
==> vhd:     + CategoryInfo          : NotSpecified: (:) [Get-WindowsUpdate], UnauthorizedAccessException
    vhd: 1 pkrvmix6h... Accepted   KB4494175    2MB 2020-01 Update for Windows Server 2016 for x64-based Systems (KB4494175)
    vhd: 1 pkrvmix6h... Accepted   KB4052623    5MB Update for Windows Defender Antivirus antimalware platform - KB4052623 (V...
    vhd: 1 pkrvmix6h... Accepted   KB890830    31MB Windows Malicious Software Removal Tool x64 - March 2020 (KB890830)
    vhd: 1 pkrvmix6h... Accepted   KB4550947    1GB 2020-04 Cumulative Update for Windows Server 2016 for x64-based Systems (...
==> vhd:     + FullyQualifiedErrorId : System.UnauthorizedAccessException,PSWindowsUpdate.GetWindowsUpdate
==> vhd:
    vhd: 1 pkrvmix6h... Accepted   KB2467173   10MB Security Update for Microsoft Visual C++ 2010 Redistributable Package (KB...
    vhd: 1 pkrvmix6h... Accepted   KB4481252   13MB Microsoft Silverlight (KB4481252)
    vhd: 1 pkrvmix6h... Accepted   KB4537477   51MB 2020-02 Cumulative Update for .NET Framework 4.8 for Windows Server 2016 ...
    vhd: 1 pkrvmix6h... Accepted   KB4494175    2MB 2020-01 Update for Windows Server 2016 for x64-based Systems (KB4494175)
    vhd: 1 pkrvmix6h... Accepted   KB4052623    5MB Update for Windows Defender Antivirus antimalware platform - KB4052623 (V...
    vhd: 1 pkrvmix6h... Accepted   KB890830    31MB Windows Malicious Software Removal Tool x64 - March 2020 (KB890830)
    vhd: 1 pkrvmix6h... Accepted   KB4550947    1GB 2020-04 Cumulative Update for Windows Server 2016 for x64-based Systems (...
==> vhd: Get-WUInstall : Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))
==> vhd: At C:\Windows\Temp\script-5e9f1a2e-92a1-91f0-9188-9815cca77108.ps1:10 char:1
==> vhd: + Get-WUInstall -MicrosoftUpdate -AcceptAll -Install -IgnoreUserInput - ...
==> vhd: + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
==> vhd:     + CategoryInfo          : NotSpecified: (:) [Get-WindowsUpdate], UnauthorizedAccessException
==> vhd:     + FullyQualifiedErrorId : System.UnauthorizedAccessException,PSWindowsUpdate.GetWindowsUpdate

So I manually ran the WindowsUpdates.ps1 script and it passed successfully.
Then I reran the  GenerateResourcesAndImage script and it failed again.
This time it gave me the generated artifacts' uri. When I tried to view the template URI I got an error the the resource was not found.
==> Builds finished. The artifacts of successful builds are:
--> vhd: Azure.ResourceManagement.VMImage:

OSType: Windows
StorageAccountLocation: eastus
OSDiskUri: https://aipdevtest3001.blob.core.windows.net/system/Microsoft.Compute/Images/images/packer-osDisk.a43c0737-32de-4779-a0a4-b49532f6aea5.vhd
OSDiskUriReadOnlySas: https://aipdevtest3001.blob.core.windows.net/system/Microsoft.Compute/Images/images/packer-osDisk.a43c0737-32de-4779-a0a4-b49532f6aea5.vhd?se=2020-05-23T14%3A28%3A56Z&sig=uT0LAYBtcVtdWx3vYGxpm0JXQW4rkCq5MmARdTfZ2r0%3D&sp=r&sr=b&sv=2016-05-31
TemplateUri: https://aipdevtest3001.blob.core.windows.net/system/Microsoft.Compute/Images/images/packer-vmTemplate.a43c0737-32de-4779-a0a4-b49532f6aea5.json
TemplateUriReadOnlySas: https://aipdevtest3001.blob.core.windows.net/system/Microsoft.Compute/Images/images/packer-vmTemplate.a43c0737-32de-4779-a0a4-b49532f6aea5.json?se=2020-05-23T14%3A28%3A56Z&sig=yE7%2FS5jbiHS2ORNfLIk5PRpUb9WsI6H35nytAUahaLs%3D&sp=r&sr=b&sv=2016-05-31

I attached the console output and the packer log.
logs.zip

Hi @jmos5156
Did you have any idea what the issue is?

Hello, @yodrami ,

Script GenerateResourcesAndImage works correctly, it doesn't delete resource group, it just prepares environment for build and run packer.exe. Since your packer build has started, script works as expected.
it is expected that resource group is deleted when build doesn't finish successfully. Based on your logs, It is done by packer.

2020/04/23 17:26:21 ui: ==> vhd: Deleting resource group ...
2020/04/23 17:26:21 ui: ==> vhd:  -> ResourceGroupName : 'packer-Resource-Group-u3bm1l1ynl'
2020/04/23 17:26:21 ui: ==> vhd:

If you want to prevent it, you should use packer.exe build -on-error=ask option.

As for the errors during image-generation, could you please try to pull the latest master and try one more time? What customization do you have locally?
We have nightly builds on master branch and build works correctly for us. So if you don't change, add new scripts, it should pass successfully.

Closing this one.
Please let me know if it is still an issue for you.


Table exists but sometimes it returns error SqliteException(1): while preparing statement, no such table:

I haven't understood which are the conditions but in the same run i made a select from the table and it returns me 0 rows without throwing any error, but then i try to delete and batch insert values into it inside a transaction and it returns the error in the title.
The table exists because I have installed drift_db_viewer and I see it

It happens only on Android

I think I have resolved the problem: I had to programmatically delete the sqlite database (uninstall the app or clearing the data wasn't enough).
Maybe it was somehow corrupted

This problem can be encountered by the user if we update the table on new versions of the application? I just met this problem too in debug mode(development stage), I deleted and installed the application and everything is ok.

This problem can be encountered by the user if we update the table on new versions of the application?

After changing the schema of the database for users (e.g. by adding a new table or by altering an existing one), you need to write a migration to avoid this.
Deleting data during development is fine, but don't forget to write a migration before releasing an upgrade or your users will run into issues.


embed error

error: TS2740 [ERROR]: Type '{ title: string; }' is missing the following properties from type 'MessageEmbed': getTitle, setTitle, getType, setType, and 20 more.
embed: { title: 'hello' }

In the example I get this error

Could you provide the code that you worked with? Was it just from the example?

yes example

Got it. Just tried it now and it seems to be an error where it was type checking with a Message Embed class. Will fix this soon. For now you can just create a MessageEmbed the same way you would in Discord.JS v12.

Please change it quickly.

I've updated the README.md example, for the time being please use MessageEmbed class to send embeds. Thank you for raising this issue.


Why you removed me and changed my access in OSSU organization?

@waciumawanjohi today I received several emails from GitHub because I was removed from almost all OSSU teams.
Someone also changed my access to member, and as a founder of OSSU organization and OSSU-CS I think it was really disrespectful since I was not even contacted before.

I would like to have my access restored to owner again.
Thanks.
cc @hanjiexi

@ericdouglas I know that the OSSU cohort group was removed, which was planned, but it's absolutely wrong for your access to be taken away. 😞

@amorriscode thanks for your support, I really appreciate it.
@waciumawanjohi was you that changed my role in the organisation?
This situation is going out of control in the disrespect that is happening to me.
4 days without an answer, and activity are happening in the repository, people are being added to the organisation...
I'm extremely disappointed with all this. This is completely wrong.
Would someone watching this repo help me to change this? I would really appreciate because it is really affecting me.
Thanks again.

@ericdouglas dropped a message to @waciumawanjohi on the Discord, hopefully that brings visibility to this sooner.

Hopefully, it does. This is quite disrespectful.

Hi Eric,
You are correct, last week I was making changes to the Github project to match permissions to the roles that individuals currently have. Part of this was prompted by the number of repos that we have that are functionally abandoned. For example, earlier this year the 'Help' repo received an awesome Issue from a first time contributor laying out an almost complete math curriculum. It's something that I wish OSSU could have responded to within hours. Instead, it was months before we did. The root cause was that permissions and notifications had not been updated to match the individuals putting in daily work for OSSU.
You mentioned that you found it disrespectful that we didn't contact you. I apologize that you and other individuals whose GitHub roles were changed were not contacted beforehand. I realize now that failing to contact you beforehand likely made it a surprise. I want to assure you that regardless of what administrative changes we make, the leadership of OSSU is continuing to feature your role as the founder.
At the same time, you stated publicly that you were setting up others as the leaders of OSSU. In the years since that announcement, the installed leaders have made changes, including bringing on other new leaders. Some have even formally stepped down themselves. And in those 2 and a half years (half the life of this 5 year old project) you have not participated in any of the public discussions or modifications to OSSU.
We certainly want to honor the contributions that you made to OSSU. When you stepped down, I thanked you publicly for shepherding the project. I praised your judgement in choosing successors who had demonstrated leadership in the organization. You are listed as founder of OSSU in the CS repo, a page thats been seen by hundreds of thousands of people. We honor and respect your contribution.
I am closing this issue since it does not pertain to modifying the CS curriculum, but feel free to reach out to @hanjiexi or me with any questions or concerns.

@waciumawanjohi what you did is absolutely out of your scope to act, you are not allowed to change my permission in this organization.
There is absolutely no reason in doing this, and I want to have my access restored to owner again.
Serious, you are doing a lot of harm to me removing my access from the organization I created. This is absolutely insane.
I want to ask again for contributors of this project and students to manifest your thoughts about this absurd situation.

@Eric I feel you man
On Wed, 16 Sep 2020 at 19:18, Eric Douglas<EMAIL_ADDRESS>wrote:

@waciumawanjohi https://github.com/waciumawanjohi what you did is
absolutely out of your scope to act, you are not allowed to change my
permission in this organization.
There is absolutely no reason in doing this, and I want to have my access
restored to owner again.
Serious, you are doing a lot of harm to me removing my access from the
organization I created. This is absolutely insane.
I want to ask again for contributors of this project and students to
manifest your thoughts about this absurd situation.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ossu/computer-science/issues/784#issuecomment-693512169,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQCETIV7VZJVFCL7PMIZGDSGDQLTANCNFSM4RITGBIA
.

I have been doing OSSU's recommended CS courses for more less two years without commenting since I wanted to be more knowledgeable on the curriculum before criticizing. Yet I just recently started discussing and reading more on the curriculum instead of just following it blindly.
Reading @waciumawanjohi almost daily active helpful contributions to peoples comments and discussing with him and the community has definitely helped me in a great positive way on understanding my CS path. However I don't see why the founder of OSSU has to have his permissions removed, founding an organization its usually seen as the biggest contribution inside most organizations.
Btw @ericdouglas can you constructively criticize my curriculum draft posted on comment-660886018.
Thanks in advance!
Preview:

Intro CS
Knowledge Areas: SDF - Software Development Fundamentals
If you've never written a for-loop, start here. Students who have been introduced to programming should also take this course for a flavor of the material to come. If you finish the course wanting more, Computer Science is likely for you!

KA
Course
Hours
Optional Resources
Prerequisites

SDF
CS50's Understanding Tech. alt
24
Programming for Everyone
none

Core CS
Theory
Knowledge Areas: SDF - Software Development Fundamentals SP - Social Issues and Professional Practice DS - Discrete Structures PL - Programming Languages AL - Algorithms and Complexity CN - Computational Science

Objective: Develop effective ways to solve computing problems.

KA
Course
Hrs
Optional Resources
Prerequisites

SDF
Python for Everbody
102
Intro, Fundam., Basics, khan, Java, GT, 100+
none

SDF
Intro to CS - CS50
144
Missing, Code 1 2 Book, IT,  Law, Business
none

SP
Learning How to Learn
15
Mindshift
none

DS
1 Calculus A B C & Prob.
256
Calculus K B, Probability MIT K B
high school

DS
Mathematics for CS
65
Discrete, Logic, Effective, Thinking, Sets
Calculus A B C

PL
PL Part A Part B Part C
66
Rosalind, Book 1, Book 2, Projects, Web
algebra

AL
Algorithms
96
Khan, UCSD, Algo, freecodecamp, geeksforgeeks, Princeton 1 2 Book, MIT 1 2
SDF PL DS

CN
Computation, videos
80
Princeton, Intro Book
DS AL

...

Hi Eric,
Let me give you some context so that you can hopefully understand better what's going on: in the time since you've stepped away, OSSU has grown significantly larger. The volume of students and active contributors has been growing steadily since I became a maintainer, to the point where I couldn't do it myself and appointed Waciuma as co-maintainer two years ago; later, OSSU's activity exploded around the time the pandemic hit, since everyone started doing online courses. The people who are actively working on OSSU have become utterly indispensible in managing what is now a highly complex social structure with short-term needs and long-term expectations.
To manage this growth and protect it from current and future threats, OSSU is now governed by a board of directors. This board handles legal/administrative stuff behind the scenes, while the community-first approach remains in place for its daily operations. As an example of a threat, recently we were contacted by an organization that appears to be a diploma mill and may be committing fraud. On a few days notice, and without permission from the board, this organization created a website asking students to pay them for a degree in partnership with Open Source Society University. This puts OSSU and all of the work that contributors have done at risk. It's the responsibility of the board to handle such a situation, to research those who want to partner with OSSU and to stop organizations who want to profit crookedly from the OSSU name.
To bring this back to the issue, what the board has decided to do is standard practice in organizations: when a leader steps down, they relinquish their leadership rights, including any authority for making decisions for the organization or the ability to represent the organization, as well as any administrative access they may have had. The former prevents many an awkward situation; the latter is basic cybersecurity.
It has been a long time since you disengaged from OSSU. You will always be OSSU's founder — no one can take that away from you — but we've got to move on and forward. Therefore, I have to disagree that this is an absurd situation. I would instead argue that it would be exceedingly odd and irresponsible if the new management of an organization didn't shut down the former manager's administrative access. Notifying the former manager of such routine procedures is unnecessary; for instance, if I left OSSU, I wouldn't expect an email requesting permission to shut down my access. On the contrary, I think it would be quite strange if I did receive such an email.
You mentioned that this is causing you harm, but it's unclear to me what this harm is. The only change that has occurred is on the backend: you no longer have the ability to create/edit/delete repositories/teams and add/remove members from the GitHub organization/teams. I don't see why these would be of interest to you, and in any case, with the formalization of OSSU's governance and community-oriented processes, we can't allow people who aren't involved with OSSU to make major administrative changes without going through the proper procedures of board approval or community buy-in, as appropriate. Like anyone else, you can still contribute when you find the time and take part in discussions about the curriculums, virtually all of which are public. If you're interested in starting a new curriculum, nothing prevents you from going through the same process that we all go through: engagement with the community.
I hope you would agree that OSSU moving forward with a formal organizational governance is the best next step, and that such board needs to do what every responsible board does to put our affairs in order. Absolutely no disrespect was intended. We still admire you for making the brazen claim that you can get a free college-level education in computer science for free, and jumpstarting a community and curriculum to make this a reality. As founder, you are of course welcome to set up a meeting with us to discuss anything you wish. But I must emphasize that we, as the board, cannot reinstate your GitHub account to that of an Owner of the OSSU GitHub organization simply because you founded the repo some years ago. There would have to be a reason for it that can be formally justified to support the organization's operations or meet its administrative requirements.
I hope you can understand. Thank you for taking the time to read this long post.
Regards,
hanjiexi

@StevenKJo thanks for your support and it looks very interesting your draft indeed!

@hanjiexi when I give you the owner level access, it was to you manage OSSU-CS more comfortably, with ability to add people and create repositories and stuff like that.
It was never conceded to you permission to remove/change my access in the future. This is absolutely wrong. OSSU organization is very different from OSSU-CS. You removed my access to OSSU organization AND OSSU-CS without even contact me. This is not right at all.
I really regret to have conceded to you this access level, but I never imagined that something like this would occur. This is an act of treason.
I learned the lesson in the worst possible way.

Going to chime in here as a semi-neutral third party, since despite being an active member of the Discord/Gitter side of OSSU for the last few years, I didn't know who Eric was until this thread started.
Here's what it looks like from my perspective:

Eric handed over the reins for OSSU and then disappeared from it. Like I said, I didn't know who he was, and I doubt many of the currently active members in the community do either. There's even a Hacker News post from 2016 wondering where he went.

Nothing wrong with this at all, it's not only normal for changes in leadership to happen, it's necessary for the survival of an organization. People move on, but an organization still needs people to manage it and put in the work.

OSSU continued on without him with @hanjiexi and @waciumawanjohi carrying the torch.
Several years later, in accordance with the principle of least privilege, Waciuma removed admin access to the Github repo for anybody who didn't need it.
Eric shows up after years of absence angrily declaring that OSSU belongs to him and demands access back.

The way I see it, this is purely a political battle.
While I appreciate the contribution Eric made in getting this ball rolling, I'd definitely side with waciuma and hanjiexi on this one. Giving Eric his access back would send a clear message that OSSU values politics over security and good governance, which is not something that as an engineer I can get behind.

While I do this it's unfortunate how this came about, after hearing both sides and thinking it through further, I do think it makes sense. I'm extremely happy the OSSU is picking up steam. I'm glad there is an organization being developed. I think it could truly help shape how education is consumed in the future.
For security reasons, I think the admin access makes sense. How can an organization function when somebody could destroy all of their work in the middle of the night without a moments notice? Eric is still a member of the organization, just not an administrator.
I wish Eric was notified earlier but he has been absent and a lot has happened without it. The project itself likely looks very different than it did when Eric was last involved. I imagine that, if he chose to be involved again in the future, he would be welcomed with open arms.

There is more to this situation than people know.
My absence for this long period had several reasons (did someone thought about it?), and one of them is directly related to behaviour of current and past members in the organization along the time that made me loose motivation to keep contributing.
Despite all of this, I genuinely appreciate every single person that commented in this issue, for real.
I did not invite more people to give opinion here to necessarily do this in my favour, I wanted to hear what you thought about all this situation, independently which opinion is yours.
All this problem that happened to me was derived from my own action (give owner access to others), so I know I am responsible for all this.
I did not change my opinion about what happened, but that's it.
So I finish my participation in this issue.
Thanks again for every person that replied here, regardless of your opinion, you truly helped me a lot.


Rename providers to sockets in the dal (and clean it up)

Introduction
REST IN PEACE PROVIDERS! You've served us well back when we first added you in ~Spring 2022. You provided values well to the rest of the system and now you can go the way of the dinosaur.

Description
This PR renames providers to sockets in the dal and in immediately affected crates. This is not a holistic naming conversion, but a first pass to prevent future conflicts and create a massive PR that is untenable to rebase and review.
Essentially, only in active modules of lib/dal were providers renamed to sockets. What does that mean in practice?

Wherever providers were renamed to sockets, doc comments, code comments and string literals were changed too
All queries using old sockets or old providers were deleted (though all migrations remain in place since we will handle migrations as main stabilizes)
Inactive modules in lib/dal/tests were also excluded from the changes since those tests may need to re-evaluated entirely
Some exceptions to renaming were made in the resource and pkg/import modules since contextual comments are currently being used to piece those domains back together (i.e. old code from old-engine) that's been commented out means little if we rename providers for the sake of renaming them yet)
Delete unused unused attribute code (values, prototypes, etc.) since they touched the old provider system and now DependentValuesUpdate works in the new engine

Context
The original "provider" naming made the most sense back in ~Spring 2022 when they signaled that they provided values for others to consume from. Now that implicit InternalProviders do not exist in the new engine and that sockets and edges are dead, we can rename providers to "sockets". They are "just sockets" in the new engine and our ubiquitous language between all domains using them should reflect this.
Disclaimer
This is part one of a two part attempt to perform the refactor. We want to avoid having one massive unreadable PR that creates a ton of conflicts and is impossible to rebase. The follow up PR covering app/web and all other places will come soon.

bors merge

bors merge


CSS Loader and Responsive-Loader issue

In web pack 2.x, I'm having issues using responsive-loader inside of my sass files. CSS loader throws an error..
 ERROR in ./~/css-loader?{"sourceMap":true,"importLoaders":1,"modules":true,"localIdentName":"[name]__[local]--[hash:base64:5]","di
scardComments":{"removeAll":true}}!./~/sass-loader/lib/loader.js?{"sourceMap":true}!./src/views/Main/Home/style.scss
    Module not found: Error: Can't resolve './responsive-loader' in '/Users/projects/am/am-web/src/views/Main/Home'
     @ ./~/css-loader?{"sourceMap":true,"importLoaders":1,"modules":true,"localIdentName":"[name]__[local]--[hash:base64:5]","discardC
omments":{"removeAll":true}}!./~/sass-loader/lib/loader.js?{"sourceMap":true}!./src/views/Main/Home/style.scss 6:6240-6315

Any ideas? I've looked through the source but I don't understand how it handles css files...
Thanks
Webpack modules
module: {
        rules: [
            {
                test: /\.(jpg)$/,
                use: 'responsive-loader',
                include: PATHS.images,
            },
            {
                test: /\.tsx?$/,
                use: 'ts-loader',
                include: PATHS.app,
            },
            {
                test: /\.(png|jpg|svg)$/,
                use: 'url-loader?limit=8192',
                include: PATHS.app,
            },
            {
                test: /\.(jpg|png|svg)$/,
                use: 'file-loader?name=[path][name].[hash].[ext]',
                include: PATHS.images,
            }
        ]
    }

Webpack css loader rules
{
                    test: /(\.scss|\.sass)$/,
                    use: ExtractTextPlugin.extract({
                        fallback: "style-loader",
                        use: [
                            {
                                loader: 'css-loader',
                                options: {
                                    sourceMap: true,
                                    importLoaders: 1,
                                    modules: true,
                                    localIdentName: '[name]__[local]--[hash:base64:5]',
                                    discardComments: {
                                        removeAll: true
                                    }
                                }
                            },
                            {
                                loader: "sass-loader",
                                options: {
                                    sourceMap: true
                                }
                            }
                        ]

                    }),

                }

My sass
hero {
  background-image: url('../../../../assets/hero.jpg');
  background-size: cover;
  color: #fff;
  text-align: center !important;

  @include breakpoint(desktop) {
    background-image: url('responsive-loader?size=1000!../../../../assets/hero.jpg');
  }

  @include breakpoint(laptop) {
    background-image: url('responsive-loader?size=800!../../../../assets/hero.jpg');
  }

  @include breakpoint(tablet) {
    background-image: url('responsive-loader?size=600!../../../../assets/hero.jpg');
  }

  @include breakpoint(mobile) {
    background-image: url('responsive-loader?size=480!../../../../assets/hero.jpg');
  }
}

My issue was that you can't use this lib with css modules. I.e. :global blocks.

But why wouldn't that work? As far as I understand, css-loader changes semantics for url imports when in module mode but I'm not sure to what exactly (the docs are pretty confusing). Have you tried to bump importLoaders to 2? Or prefix the path with !! to make sure no other loaders apply, e.g. !!responsive-loader?....

Maybe you just have to adjust the path of the imported image when in module mode? Or does it just work when you remove the loader?


latest and edge tags are inconsistent across the project

Area for Improvement
Our usage of edge and latest container image tags is inconsistent across our repositories.
in radius-project/recipes:
latest refers to the most recent stable build, e.g. 0.37.
edge refers to the build at the top of the main branch`.
Reference: https://github.com/radius-project/recipes?tab=readme-ov-file#versioning-and-tags
Example: https://github.com/radius-project/recipes/pkgs/container/recipes%2Flocal-dev%2Fsqldatabases
in radius-project/radius:
latest refers to the build at the top of the main branch
there is no edge tag for our container images.
 I can imagine a few solutions:

For radius-project/recipes, we adopt the Radius pattern: push to latest on push to main and remove edge tag.

or

For radius-project/radius, we adopt the Recipes pattern: push to latest on release and edge on push to main

Proposed Fix
No response
rad Version
v0.37
Operating system
No response
Additional context
No response
Would you like to support us?

[ ] Yes, I would like to support you

AB#13191

This is going to require a small design review.

My 2c would be to follow the radius-project/recipes format and standard. latest is the latest release and edge is the HEAD of the main branch


Revert sriov to only one node setup

What this PR does / why we need it:
Hoping this is temporary, we revert the sriov lane to use just one note to reduce the scope of changes in order to understand ci failures.
NONE

@fedepaol do you want to merge this, or only check CI?

@slintes I'd say let's see if the kind lane gets through, if it does let's merge this so I can investigate the issue in a separate PR without making tide's life difficult

/test pull-kubevirt-e2e-kind-k8s-sriov-1.14.2

/test pull-kubevirt-e2e-kind-k8s-sriov-1.14.2

/test pull-kubevirt-e2e-kind-k8s-sriov-1.14.2

Ran it a couple of times, seems to be fine.
@slintes this is good to go


Create translation.json for polish language

Created polish language translation
Changelog Entry
Added

[Polish translation]

Thanks!


Unknown property 'scroll-snap-align'

I'm not sure I should file the issue in this repo – it's probably related to a dependency, and I don't which.
I get "Unknown property" warning when using scroll-snap-align css property in a styled component.

Why was this closed?


Train facenet on Caffe

Has anybody trained facenet on Caffe?
I trained facenet by tensorflow on Ms_celeb_1M dataset and it works well,get 99.4% accuracy on LFW.But when I use Inception_resnet_v2 on Caffe,it didn't converge,the loss is always 10+ in 28K iterations.
I don't know whether it makes some difference with BN,so I'll train the Inception_resnet_v1 again.
If anybody had successfully trained facenet on caffe,please @me,thanks a lot.

@KaleidoZhouYN  Can I see your caffe code of facenet？ I want to know how to select triplet samples..3q

@zhanglonghao1992  sorry about that,the embedding and triplet part is written by my friend.I just do the classification part.

Closed due to inactivity. Reopen if needed.


Long match expression doesn't push the brace on a new line

This is how rustfmt formats the code:
fn main() {
    if let Ok(x) = t.next
        .compare_and_swap_weak_owned(next, node, SeqCst, scope)
    {
        println!();
    }

    match t.next
        .compare_and_swap_weak_owned(next, node, SeqCst, scope) {
        Ok(_) => {}
        Err(_) => {}
    }
}

I would expect the match block to look like this:
    match t.next
        .compare_and_swap_weak_owned(next, node, SeqCst, scope)
    {
        Ok(_) => {}
        Err(_) => {}
    }

Yeah, I expect we should do this for basically all expressions with blocks


On 32bit platforms usize cannot index 64 bit vectors.

I know this was also a problem before, but on 32bit platforms usize cannot index 64 bit vectors.
Originally posted by @raphaelrobert in https://github.com/openmls/openmls/pull/318#r564735980

Since we now use the external TLS syntax crate, this is no longer an issue.


[BUG] [Android] Cannot build CHIPTest

Reproduction steps
1.Run./scripts/build/build_examples.py --target android-arm64-chip-test build
2.The build fails with "clang++: error: linker command failed with exit code 1"

(full logs attached)

Extra info:
Building the Android ChipTool using "./scripts/build/build_examples.py --target android-arm64-chip-tool build" works so:
* the Matter sources have been pulled correctly
* Matter compiler toolchain is working
* Android compiler toolchain is working

Bug prevalence
Every time
GitHub hash of the SDK that was being used
0bff5ad04c696626ce3aa8fcc5c9e229ed8bacf0
Platform
android
Platform Version(s)
No response
Anything else?
logs.txt

I also met this issue, is there anyone has solved this issue already?


Inconsistencies in user experience when creating circuits with devices.

Is your design idea/issue related to a use case or problem? Please describe.
When creating circuits for a particular device, should the users be responsible for ensuring that they only use the native gates in the circuit, and if not, first explicitly convert the circuit to one containing only native gates, using one of the existing compilation tools? Or should the device class be smart enough to do this conversion via the decompose_operation method on the device class?
Describe your design idea/issue
For example, a simple QFT circuit on 3 qubits behaves differently when created using an IonQ device vs Sycamore device.
Using IonQ device
The compilation happens as part of the construction because decompose_operation as takes care of the steps.
num_qubits = 2
qubits = cirq.LineQubit.range(num_qubits)
cirq.Circuit(cirq.QuantumFourierTransformGate(num_qubits)(*qubits), device = cirq.ionq.IonQAPIDevice(qubits))

compiles fine to the device compatible circuit
0: ───Y^0.5──────Z^0.393───XX────────YY────────ZZ─────────Z^(-9/14)───X^0.5──────S───
                           │         │         │
1: ───Z^-0.607─────────────XX^-0.5───YY^-0.5───ZZ^-0.25───X───────────Z^(9/14)───────

Using Sycamore device
The same workflow as above throws an error that the QFT gate is not supported.
num_qubits = 2
qubits = cirq.GridQubit.rect(1, num_qubits, 4, 4)
cirq.Circuit(cirq.QuantumFourierTransformGate(num_qubits)(*qubits), device = cirq.google.Sycamore)

gives
ValueError: qft((4, 4), (4, 5)) is not a supported gate

Instead, the circuit first needs to be compiled to device compatible gateset:
num_qubits = 2
qubits = cirq.GridQubit.rect(1, num_qubits, 4, 4)
circuit = cirq.Circuit(cirq.QuantumFourierTransformGate(num_qubits)(*qubits))
cirq.Circuit(cirq.google.ConvertToSycamoreGates().convert(circuit), device = cirq.google.Sycamore)

gives the correct device compatible circuit
(4, 4): ───PhXZ(a=-1,x=0.393,z=0)────PhX(0.568)^0.586───SYC───PhX(0.0677)─────────SYC───PhX(-0.599)^0.414───Z^(2/3)────────────────────PhXZ(a=0.5,x=0.5,z=0)───PhX(0.568)^0.586───SYC───PhX(0.0677)─────────SYC───PhX(-0.599)^0.414───Z^(2/3)───────────────────PhXZ(a=-0.5,x=0.5,z=0)──────PhXZ(a=-1.11e-16,x=0.724,z=0.0833)───SYC────────────────SYC───PhXZ(a=-0.0833,x=0.724,z=-0.917)───Rz(0.25π)───PhXZ(a=-0.607,x=0.5,z=0)───PhXZ(a=-0.607,x=0,z=-0.893)───
                                                        │                         │                                                                                │                         │                                                                                │                  │
(4, 5): ───PhXZ(a=0.107,x=0.5,z=0)───PhX(0.355)─────────SYC───PhX(-0.515)^0.523───SYC───Z^-0.926────────────PhXZ(a=-0.893,x=0.5,z=0)───PhX(0.355)─────────────────────────────────SYC───PhX(-0.515)^0.523───SYC───Z^-0.926────────────PhXZ(a=0.107,x=0.5,z=0)───PhXZ(a=0.542,x=0,z=0.917)────────────────────────────────────────SYC───Rx(0.189π)───SYC───PhXZ(a=0.227,x=2.22e-16,z=0.25)────Rz(0.25π)───PhXZ(a=0.554,x=1,z=0)──────PhXZ(a=-0.393,x=0,z=0.893)────

As per the discussion #3295, It looks like circuit construction with device attribute should be deprecated.

Let's change the scope of this issue to add support for an explicit compiler for IonQ as we plan on deprecating the device object in a circuit (i.e. cirq.Circuit( op_list, device = cirq.ionq.IonQAPIDevice(qubits)) would be deprecated)

@tanujkhattar Can we close this since we aren't attaching devices to circuits anymore ?


Widget - undefined 'rule' error in viewer

Describe the bug
Adding a widget to the editor works fine. But when viewing this from the Viewer I get TypeError: Cannot read properties of undefined (reading 'rule'). Likely because the rules are missing. I tried add the same widgetRules to the viewer options using widgetRules:... but did not work.
Widget markdown example $$widget0 @Firstname Lastname$$
Other markdown works but if widget is included then it breaks the Viewer and nothing is rendered.
To Reproduce
Steps to reproduce the behavior:

Create widget
View it in Viewer.

##Error
TypeError: Cannot read properties of undefined (reading 'rule') in the customInline: function()
Expected behavior
Viewer should display the widget like the editor does.
Desktop (please complete the following information):

OS: Window 10
Browser Chrome,
Version latest
@toast-ui/vue-editor v3.1.7

@digitalfuel
Viewer only uses ToastMark. So, it can't get widget rules. But, If you put the widget rule in the viewer, it is no longer a lightweight module.

So what is the correct way to display widget in viewer?

Same here. I'm trying to use the input from the user on the Editor and display it somewhere else. But I can't support any customization I added to the markdown because the viewer can't support it. Would love a "readOnly" prop to the editor. So if "weight" is not an issue I can still show a readOnly preview.

I'm also having this same issue.  There's no way to display a widget in viewer.


Where can I view errors encountered in the php-cgi-spawner.exe?

When processing a large number of requests, somewhere between NGinx and a PHP application, I get an error that I cannot find.
The NGinx access log simply says "upstream_status is 502":
{
    "time_iso8601": "2024-11-07T18:01:35+03:00",
    "time_local": "07/Nov/2024:18:01:35 +0300",
    "remote_addr": "<IP_ADDRESS>",
    "remote_port": "64432",
    "remote_user": "",
    "server_protocol": "HTTP/2.0",
    "bytes_sent": 754,
    "status": 502,
    "http_x_request_id": "",
    "http_accept": "*/*",
    "http_accept_encoding": "gzip, deflate, br, zstd",
    "http_traceparent": "",
    "http_tracestate": "",
    "request_method": "GET",
    "request_uri": "/ymap/investment_subjects_browser/geo-object-of-project/get-collection-by-area-as-jsonp?bounds=50.9248,30.9375,52.6684,33.7500&callback=id_173099169289773033535",
    "request_length": 146,
    "request_time": 2.022,
    "sent_http_content_length": "552",
    "sent_http_content_type": "text/html",
    "upstream_addr": "<IP_ADDRESS>:9000",
    "upstream_response_length": "0",
    "upstream_response_time": "2.022",
    "upstream_status": "502"
}

If you look at what is located at the "upstream_adress" specified in the log, then there is a spawner:
Proto  Local Address          Foreign Address        State
TCP    <IP_ADDRESS>:9000    <IP_ADDRESS>:0              LISTENING
[php-cgi-spawner.exe]

The target php file was not launched (there is logging on the first line, I would have seen it).
I think that the error occurred in the spawner, but I can’t find if this application has logging and how to configure it.

How do you run php-cgi-spawner?
I use this for ages with no errors:
set PHP_FCGI_MAX_REQUESTS=0
php-cgi-spawner.exe "php\php-cgi.exe -c php.ini" 9001 4 128

Over all these years php-cgi itself has managed to embed its own internal spawner, maybe you do not need to use php-cgi-spawner at all and use php-cgi for example like this:
set PHP_FCGI_MAX_REQUESTS=0
set PHP_FCGI_CHILDREN=4
cd php
start php-cgi.exe -c php.ini -b 9001

Also, regarding your question:

Where can I view errors encountered in the php-cgi-spawner.exe?

There is no error log for php-cgi-spawner as it only performs very basic tasks.

I'm using OSPanel.
OSPanel uses php-cgi-spawner to launch php.
I did not change the settings defined by the OSPanel developers.
I'm guessing that the OSPanel developers are running php-cgi-spawner incorrectly.
I will contact them with the same question.

https://github.com/OSPanel/OpenServerPanel/discussions/27

Self-Management of PHP-CGI Subprocesses: PHP modules now manage their subprocesses independently, without using php-cgi-spawner.exe, improving performance and stability.

Thank you very much for letting us know that the problem is no longer relevant.


Creating a network service discovery request

So I have an observable singleton:
/** Provides {@link BonjourEvent} observable. */
  @Provides @Singleton Observable<BonjourEvent> provideBonjourEventStream(Application app) {
    return RxBonjour.startDiscovery(app, "_myservers._tcp");
  }

"myservers" never changes.
In my fragment I have:
@Override public void onStart() {
    super.onStart();
    subscription = bonjourEventStream
        .subscribeOn(Schedulers.computation()) //
        .observeOn(AndroidSchedulers.mainThread()) //
        .subscribe(bonjourEvent -> {
            ...
         });
}

@Override public void onPause() {
    super.onPause();
    if (subscription != null && !subscription.isUnsubscribed()) subscription.unsubscribe();
  }

So when I switch the screen on an off the subscription unsubscribes and subscribes again. One out of ten times of switching the screen I have:
FATAL EXCEPTION: RxCachedThreadScheduler-15
E/AndroidRuntime: Process: com.symway.assist, PID: 24660
E/AndroidRuntime: java.lang.IllegalStateException: Exception thrown on Scheduler.Worker thread. Add `onError` handling.
E/AndroidRuntime:     at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:60)
E/AndroidRuntime:     at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
E/AndroidRuntime:     at java.util.concurrent.FutureTask.run(FutureTask.java:237)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
E/AndroidRuntime:     at java.lang.Thread.run(Thread.java:841)
E/AndroidRuntime:  Caused by: rx.exceptions.OnErrorNotImplementedException: MulticastLock under-locked RxBonjourDiscovery
E/AndroidRuntime:     at rx.Observable$29.onError(Observable.java:7498)
E/AndroidRuntime:     at rx.observers.SafeSubscriber._onError(SafeSubscriber.java:154)
E/AndroidRuntime:     at rx.observers.SafeSubscriber.onError(SafeSubscriber.java:111)
E/AndroidRuntime:     at rx.internal.operators.OperatorObserveOn$ObserveOnSubscriber.pollQueue(OperatorObserveOn.java:197)
E/AndroidRuntime:     at rx.internal.operators.OperatorObserveOn$ObserveOnSubscriber$2.call(OperatorObserveOn.java:170)
E/AndroidRuntime:     at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:55)
E/AndroidRuntime:     at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
E/AndroidRuntime:     at java.util.concurrent.FutureTask.run(FutureTask.java:237)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
E/AndroidRuntime:     at java.lang.Thread.run(Thread.java:841)
E/AndroidRuntime:  Caused by: java.lang.RuntimeException: MulticastLock under-locked RxBonjourDiscovery
E/AndroidRuntime:     at android.net.wifi.WifiManager$MulticastLock.release(WifiManager.java:2132)
E/AndroidRuntime:     at rxbonjour.internal.SupportBonjourDiscovery$2$1.call(SupportBonjourDiscovery.java:171)
E/AndroidRuntime:     at rxbonjour.internal.SupportBonjourDiscovery$2$1.call(SupportBonjourDiscovery.java:163)
E/AndroidRuntime:     at rx.Observable.unsafeSubscribe(Observable.java:7713)
E/AndroidRuntime:     at rx.internal.operators.OperatorSubscribeOn$1$1.call(OperatorSubscribeOn.java:62)
E/AndroidRuntime:     at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:55)
E/AndroidRuntime:     at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
E/AndroidRuntime:     at java.util.concurrent.FutureTask.run(FutureTask.java:237)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
E/AndroidRuntime:     at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
E/AndroidRuntime:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
E/AndroidRuntime:     at java.lang.Thread.run(Thread.java:841)

On SupportBonjourDiscovery.java:171 I see the release, so when I try to subscribe again on screen on it SHOULD go wrong but sometimes it doesn't.
Finally, I have got a question if one just shouldn't unsubscribe ever or one must not have singleton BonjourEvent observable?

I have been able to reproduce this issue. The gist of it boils down to JmDNS.close() being a method that blocks the current thread for quite a while upon cleaning up after a Support-Implementation subscription is being unsubscribed from. Because the Multicast lock previously acquired is only released after this blocking method finishes, this code may execute right before you can acquire a new wake lock.
Releasing the wake lock before the synchronously blocking close() on JmDNS fixes the issue. Also, I have changed the support implementation to only have one multicast lock.
I want to perform some more testing on this and release this in 0.3.1 shortly, if everything is working as intended!

Thank you for explanation.

By the way, you using and consuming the Observable as a Singleton provided by Dagger is totally fine, I don't see anything wrong with that!

Released in 0.3.1


Allow to build packages without config

This make possible to build a package without .pakinfo/pak.conf file, or with an incomplete one.
If the name or version keys are missing but the folder is named like <name>_<version>.dpkdir, name and version will be guessed from the folder name, if the version string in folder name is src, it will relies on git to compute the version (or use some failover version).
If there is no config file or the config file doesn't contain the game key, the game will have to be specified on command line, like --game unvanquished.

The comment sounds good, but I can't really tell for the code


CSS changes are not impacting the Threads page

The changes aren't impacting the Threads page - it's still white background with the altered colors.

Can you post a screenshot?

See attached. You can see that there is a white bar along the top as well (this also happens in the thread frame that pops out from the right hand side).

Any update on this?

That's the Search results, not the Threads pane - right?  Maybe you should edit the ticket headline.


replacing df.query in NJ scripts

This remains incredibly weird but thanks for figuring it out!

agreed--thanks so much for helping me work through it!


Made resize span for database list full height, changed cursor

In regards to #348
Grabbed the react-resize-handle class and made it 100% high so it follows the entire line.
I thought I'd try a different cursor and see how you felt about it.
To me the grab icon means "I can move this thing around the screen", not "I can resize a column" so I changed it to col-resize and gave it the !important tag to overwrite the one given by react-resizable

Yeah feels good with that cursor. And it is much better to be able to resize through the whole sidebar. Thank you!!


Fixed onerror in NeoAvatar

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #8238
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖[deprecated] Generated by Copilot at fa0c730
Enhanced the robustness and performance of the NeoAvatar component by preventing repeated error events and unnecessary image requests.

🤖[deprecated] Generated by Copilot at fa0c730

NeoAvatar fails
gracefully handles errors
removes srcset - fall

cool

seems fixed on canary tho

thanks!
pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 5.25 USD/DOT ~ 3.81 $DOT
🧗 16SpvdDgKNiQ3c53DxX7refnQcKUD3uRNim3Z1HBJLNGrtSP
🔗 0x312a05f4945778d93bcbcf43c69d429c345403778df6e339831fa53672cb75fd
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


How to link feature files

I have created a login.feature with only one login scenario
I have created another account.feature file that will create the account
Now I want to call login scenario to account.feature
Can you please help me in that?

This isn't possible, as explained here. Simply put, this is an anti-pattern and not something you should feel compelled to do.

Thank for your response and i have one more doubt do I need to cerate the folder with same name as .feature has?
if i am creating 5 feature files means i need to create 5 folder with the same name?


Remaining, not clear how to use

Hello.
Hope this is not a bug but just wrong implement config however I spent whole evening trying implement below both codes and total zero success.
- entity_id: whatever # as long as it is unique
  type: remaining_child_state
  name: Discrepancy

or
- entity_id: whatever # as long as it is unique
  type: remaining_parent_state
  name: Other

My 1st section is water meter from water supply company, middle section is subtotal water meter for how water (before boiling room) only and right section is end water meters for hot and cold water. Sometimes on end of day sum left vs right is zero, sometime on plus and sometime on minus.  But discrepences on plus or minus cannot visualization :/
Can I get help?

My code is
layout: horizontal
height: 400
unit_prefix: ""
round: 3
convert_units_to: ""
co2_intensity_entity: sensor.co2_signal_co2_intensity
gas_co2_intensity: 2351.6459999999997
min_box_size: 6
min_box_distance: 7
show_states: true
show_units: true
sections:
  - entities:
      - type: remaining_child_state
        entity_id: iiuiugig9iuiubiubiubi
        name: Discrepancy
      - type: entity
        children:
          - sensor.wmbus_reader_v2_wezel_zw_do_cwu_wifi_total
          - sensor.1_zw
          - sensor.2_zw
          - sensor.3_zw
          - sensor.4_zw
          - sensor.5_zw
          - sensor.6_zw
          - sensor.7_zw
          - sensor.8_zw
          - sensor.9_zw
          - sensor.10_zw
          - sensor.11_zw
          - sensor.12_zw
          - sensor.12a_zw
          - sensor.adm_zw
        entity_id: sensor.wmbus_reader_v2_apator_wifi_na_1_total
        color: blue
        color_on_state: false
        remaining_child_state: 1
  - entities:
      - type: entity
        children:
          - sensor.cwu_1_total_volume_m3
          - sensor.cwu_2_total_volume_m3
          - sensor.cwu_3_total_volume_m3
          - sensor.cwu_4_total_volume_m3
          - sensor.cwu_5_total_volume_m3
          - sensor.cwu_7_total_volume_m3
          - sensor.cwu_8_total_volume_m3
          - sensor.cwu_9_total_volume_m3
          - sensor.cwu_10_total_volume_m3
          - sensor.cwu_11_total_volume_m3
          - sensor.cwu_12_total_volume_m3
          - sensor.cwu_12a_total_volume_m3
          - sensor.cwu_adm_total_volume_m3
        entity_id: sensor.wmbus_reader_v2_wezel_zw_do_cwu_wifi_total
        color: red
      - type: passthrough
        children: []
        entity_id: sensor.1_zw
      - type: passthrough
        children: []
        entity_id: sensor.2_zw
      - type: passthrough
        children: []
        entity_id: sensor.3_zw
      - type: passthrough
        children: []
        entity_id: sensor.4_zw
      - type: passthrough
        children: []
        entity_id: sensor.5_zw
      - type: passthrough
        children: []
        entity_id: sensor.6_zw
      - type: passthrough
        children: []
        entity_id: sensor.7_zw
      - type: passthrough
        children: []
        entity_id: sensor.8_zw
      - type: passthrough
        children: []
        entity_id: sensor.9_zw
      - type: passthrough
        children: []
        entity_id: sensor.10_zw
      - type: passthrough
        children: []
        entity_id: sensor.11_zw
      - type: passthrough
        children: []
        entity_id: sensor.12_zw
      - type: passthrough
        children: []
        entity_id: sensor.12a_zw
      - type: passthrough
        children: []
        entity_id: sensor.adm_zw
  - entities:
      - type: entity
        children: []
        entity_id: sensor.cwu_1_total_volume_m3
        color: DarkSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_2_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_3_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_4_total_volume_m3
        color: IndianRed
      - type: entity
        children: []
        entity_id: sensor.cwu_5_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_7_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_8_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_9_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_10_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_11_total_volume_m3
        color: Salmon
      - type: entity
        children: []
        entity_id: sensor.cwu_12_total_volume_m3
        color: LightCoral
      - type: entity
        children: []
        entity_id: sensor.cwu_12a_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_adm_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.1_zw
      - type: entity
        children: []
        entity_id: sensor.2_zw
      - type: entity
        children: []
        entity_id: sensor.3_zw
      - type: remaining_child_state
        children: []
        entity_id: sensor.4_zw
      - type: entity
        children: []
        entity_id: sensor.5_zw
      - type: entity
        children: []
        entity_id: sensor.6_zw
      - type: entity
        children: []
        entity_id: sensor.7_zw
      - type: entity
        children: []
        entity_id: sensor.8_zw
      - type: entity
        children: []
        entity_id: sensor.9_zw
      - type: entity
        children: []
        entity_id: sensor.10_zw
      - type: entity
        children: []
        entity_id: sensor.11_zw
      - type: entity
        children: []
        entity_id: sensor.12_zw
      - type: entity
        children: []
        entity_id: sensor.12a_zw
      - type: entity
        children: []
        entity_id: sensor.adm_zw
      - entity_id: whatepopojpver # as long as it is unique
        type: remaining_child_state
        name: Other
type: custom:sankey-chart
title: Sankey wodociągi
energy_date_selection: true
show_names: true
show_icons: false
sort_dir: desc
wide: true
sort_by: state

You are using remaining_child_state without any children. How should the card know which entities to link it to?
Similarly, if you want to use remaining_parent_state, it must be listed in children of the relevant entities.

Ehhh, this is reason why asking, not understood how implement both :(
Now I change into this but disappear little visualization (missing extra quantity not counted) but not appear any new value. Can you help please more preciously?
layout: horizontal
height: 400
unit_prefix: ""
round: 3
convert_units_to: ""
co2_intensity_entity: sensor.co2_signal_co2_intensity
gas_co2_intensity: 2351.6459999999997
min_box_size: 6
min_box_distance: 7
show_states: true
show_units: true
sections:
  - entities:
      - type: remaining_child_state
        name: Input
        children:
          - sensor.wmbus_reader_v2_wezel_zw_do_cwu_wifi_total
          - sensor.1_zw
          - sensor.2_zw
          - sensor.3_zw
          - sensor.4_zw
          - sensor.5_zw
          - sensor.6_zw
          - sensor.7_zw
          - sensor.8_zw
          - sensor.9_zw
          - sensor.10_zw
          - sensor.11_zw
          - sensor.12_zw
          - sensor.12a_zw
          - sensor.adm_zw
        entity_id: sensor.wmbus_reader_v2_apator_wifi_na_1_total
        color: blue
        color_on_state: false
        remaining_child_state: 1
  - entities:
      - type: entity
        children:
          - sensor.cwu_1_total_volume_m3
          - sensor.cwu_2_total_volume_m3
          - sensor.cwu_3_total_volume_m3
          - sensor.cwu_4_total_volume_m3
          - sensor.cwu_5_total_volume_m3
          - sensor.cwu_7_total_volume_m3
          - sensor.cwu_8_total_volume_m3
          - sensor.cwu_9_total_volume_m3
          - sensor.cwu_10_total_volume_m3
          - sensor.cwu_11_total_volume_m3
          - sensor.cwu_12_total_volume_m3
          - sensor.cwu_12a_total_volume_m3
          - sensor.cwu_adm_total_volume_m3
        entity_id: sensor.wmbus_reader_v2_wezel_zw_do_cwu_wifi_total
        color: red
      - type: passthrough
        children: []
        entity_id: sensor.1_zw
      - type: passthrough
        children: []
        entity_id: sensor.2_zw
      - type: passthrough
        children: []
        entity_id: sensor.3_zw
      - type: passthrough
        children: []
        entity_id: sensor.4_zw
      - type: passthrough
        children: []
        entity_id: sensor.5_zw
      - type: passthrough
        children: []
        entity_id: sensor.6_zw
      - type: passthrough
        children: []
        entity_id: sensor.7_zw
      - type: passthrough
        children: []
        entity_id: sensor.8_zw
      - type: passthrough
        children: []
        entity_id: sensor.9_zw
      - type: passthrough
        children: []
        entity_id: sensor.10_zw
      - type: passthrough
        children: []
        entity_id: sensor.11_zw
      - type: passthrough
        children: []
        entity_id: sensor.12_zw
      - type: passthrough
        children: []
        entity_id: sensor.12a_zw
      - type: passthrough
        children: []
        entity_id: sensor.adm_zw
  - entities:
      - type: entity
        children: []
        entity_id: sensor.cwu_1_total_volume_m3
        color: DarkSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_2_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_3_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_4_total_volume_m3
        color: IndianRed
      - type: entity
        children: []
        entity_id: sensor.cwu_5_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_7_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_8_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_9_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_10_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_11_total_volume_m3
        color: Salmon
      - type: entity
        children: []
        entity_id: sensor.cwu_12_total_volume_m3
        color: LightCoral
      - type: entity
        children: []
        entity_id: sensor.cwu_12a_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.cwu_adm_total_volume_m3
        color: LightSalmon
      - type: entity
        children: []
        entity_id: sensor.1_zw
      - type: entity
        children: []
        entity_id: sensor.2_zw
      - type: entity
        children: []
        entity_id: sensor.3_zw
      - type: entity
        children: []
        entity_id: sensor.4_zw
      - type: entity
        children: []
        entity_id: sensor.5_zw
      - type: entity
        children: []
        entity_id: sensor.6_zw
      - type: entity
        children: []
        entity_id: sensor.7_zw
      - type: entity
        children: []
        entity_id: sensor.8_zw
      - type: entity
        children: []
        entity_id: sensor.9_zw
      - type: entity
        children: []
        entity_id: sensor.10_zw
      - type: entity
        children: []
        entity_id: sensor.11_zw
      - type: entity
        children: []
        entity_id: sensor.12_zw
      - type: entity
        children: []
        entity_id: sensor.12a_zw
      - type: entity
        children: []
        entity_id: sensor.adm_zw
      - type: remaining_child_state
        name: Other
        entity_id: sensor.wmbus_reader_v2_apator_wifi_na_1_total
type: custom:sankey-chart
title: Sankey wodociągi
energy_date_selection: true
show_names: true
show_icons: false
sort_dir: desc
wide: true
sort_by: state

You can find examples in the README and the HA forum https://community.home-assistant.io/t/anyone-using-the-sankey-chart-card/423125


SRLCompositionalGrounder crashes

In outgoingOfType(), the tok value is sometimes greater than dependencies.outgoingEdges can handle and an exception is thrown.  If the code
    if (tok >= dependencies.outgoingEdges.length)
      println("Something is very wrong!")

is added, one can catch it before it happens and inspect.  4 of 52 test files including the one attached seem to cause the problem.  The size of outgoingEdges in one case is 24.  The tokenInterval is 2-24 and it is the 24th that causes the problem.
3_ALPAN_Paper_10.txt

Can you please point me to the class where it happens?

I didn't check for multiple classes in the file.  It's in SentenceHelper in SRLCompositionalGrounder.scala.

https://github.com/clulab/eidos/blob/6ebd4b3a674294d7ca24073197c614e119c66965/src/main/scala/org/clulab/wm/eidos/groundings/grounders/SRLCompositionalGrounder.scala#L426

the dependencies array isn't shorter like the srl one, right @MihaiSurdeanu ?

It should not be. Is dependencies the syntax ones or SRL here?

Also, if this is causing so much pain, should I enforce the SRL ones to be the size of the sentence?

BTW, thanks @BeckySharp for so cleanly adding things like IndividualGrounding to the messy OntologyGrounder.scala code.  The JSONLD deserialization is ready and the present issue was only found while I was giving the serialization a good workout.

OK, i can check when i get back if this isn't solved yet -- and no, Mihai,
these are the syntax deps, not the SRLs, so no changes are needed there
On Thu, Oct 1, 2020 at 8:33 AM Keith Alcock<EMAIL_ADDRESS>wrote:

External Email
BTW, thanks @BeckySharp https://github.com/BeckySharp for so cleanly
adding things like IndividualGrounding to the messy OntologyGrounder.scala
code. The JSONLD deserialization is ready and the present issue was only
found while I was giving the serialization a good workout.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702217544, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCOHFZZWLFNVBAWXUQTSISOLJANCNFSM4R7ZQXWA
.

though fwiw you may want to add a helper method to return Array.empty when
someone tries to get the outgoing for a SRL beyond the array...?
they could all point to the same Array.empty
On Thu, Oct 1, 2020 at 9:49 AM Becky Sharp<EMAIL_ADDRESS>wrote:

OK, i can check when i get back if this isn't solved yet -- and no, Mihai,
these are the syntax deps, not the SRLs, so no changes are needed there
On Thu, Oct 1, 2020 at 8:33 AM Keith Alcock<EMAIL_ADDRESS>wrote:

External Email
BTW, thanks @BeckySharp https://github.com/BeckySharp for so cleanly
adding things like IndividualGrounding to the messy OntologyGrounder.scala
code. The JSONLD deserialization is ready and the present issue was only
found while I was giving the serialization a good workout.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702217544, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCOHFZZWLFNVBAWXUQTSISOLJANCNFSM4R7ZQXWA
.

FYI: a bug fix for this has been merged in master in processors. We now guarantee that semantic deps have the same size as the sentence.

@kwalcock: can you please test if this bug happens with the latest processors? If not, maybe we should merge the parallel branch too, and then release 8.2.2?

Yes, the testing was my plan exactly.

It may be that I am being deceived, but it doesn't seem to have worked.  I see things like words = String[30] and dependencies.outgoingEdges = Tuple2[29] and preferredSize = None.  The processor in question is a FastNLPProcessorWithSemanticRoles and it appears that the DirectedGraphs are initialized along a different route than what was changed.  They come from FastNLPProcessor.parseWithStanford which calls CoreNLPUtils.toDirectedGraph(basicDeps, in) which makes a DirectedGraph without the preferredSize.  In IntelliJ you can right click on the constructor for DirectedGraph and select "Find Usages".  "New instance creation" is probably the right category to double check.  I'm not sure which all of them are necessary to convert, but at least one is missing the new argument.

Oh, I think I know what's going on. Hang on.
On Thu, Oct 1, 2020 at 17:08 Keith Alcock<EMAIL_ADDRESS>wrote:

It may be that I am being deceived, but it doesn't seem to have worked. I
see things like words = String[30] and dependencies.outgoingEdges =
Tuple2[29] and preferredSize = None. The processor in question is a
FastNLPProcessorWithSemanticRoles and it appears that the DirectedGraphs
are initialized along a different route than what was changed. They come
from FastNLPProcessor.parseWithStanford which calls
CoreNLPUtils.toDirectedGraph(basicDeps, in) which makes a DirectedGraph
without the preferredSize. In IntelliJ you can right click on the
constructor for DirectedGraph and select "Find Usages". "New instance
creation" is probably the right category to double check. I'm not sure
which all of them are necessary to convert, but at least one is missing the
new argument.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702459177, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI75TUVUSDX3Y5M374QXSLSIUKWBANCNFSM4R7ZQXWA
.

also i’m not looking at the code, but i thought i switched to cluprocessor,
right? or am i crazy?
On Thursday, October 1, 2020, Mihai Surdeanu<EMAIL_ADDRESS>wrote:

External Email
Oh, I think I know what's going on. Hang on.
On Thu, Oct 1, 2020 at 17:08 Keith Alcock<EMAIL_ADDRESS>wrote:

It may be that I am being deceived, but it doesn't seem to have worked. I
see things like words = String[30] and dependencies.outgoingEdges =
Tuple2[29] and preferredSize = None. The processor in question is a
FastNLPProcessorWithSemanticRoles and it appears that the DirectedGraphs
are initialized along a different route than what was changed. They come
from FastNLPProcessor.parseWithStanford which calls
CoreNLPUtils.toDirectedGraph(basicDeps, in) which makes a DirectedGraph
without the preferredSize. In IntelliJ you can right click on the
constructor for DirectedGraph and select "Find Usages". "New instance
creation" is probably the right category to double check. I'm not sure
which all of them are necessary to convert, but at least one is missing
the
new argument.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702459177, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/
AAI75TUVUSDX3Y5M374QXSLSIUKWBANCNFSM4R7ZQXWA>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702470650, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCPOUUEDTYZP3B52THDSIUP6HANCNFSM4R7ZQXWA
.

Testing the fix...

Ok, can you please try now?

I was looking at EidosProcessor rather than the SRLCompositionalGrounder for the processor.  They may be getting tangled up somehow.

It doesn't look like https://github.com/clulab/processors/pull/426 fixed it.  Although eidos uses two different processors in the task, it seems that CluProcessor somehow causes the problem single handedly.  The computer may still be deceiving me, but I think that @BeckySharp 's observation above is pertinent.

So the crash is in processors on the file attached in this thread ?
On Fri, Oct 2, 2020 at 14:16 Keith Alcock<EMAIL_ADDRESS>wrote:

It doesn't look like clulab/processors#426
https://github.com/clulab/processors/pull/426 fixed it. Although eidos
uses two different processors in the task, it seems that CluProcessor
somehow causes the problem single handedly. The computer may still be
deceiving me, but I think that @BeckySharp https://github.com/BeckySharp
's observation above is pertinent.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702961731, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI75TXXKF4AOBIYVQUUBNLSIY7LRANCNFSM4R7ZQXWA
.

No, it happens in SRICompositionalGrounder.scala in SentenceHelper.outgoingOfType in which tok (tokenIndex?) is off the end of dependencies.outgoingEdges.  How it gets in the situation is the difficult question.  I notice in SentenceHelper that the length of outgoingEdges of srls and dependencies differs.  Dependencies is one shorter somehow.

Yes on the attached file and three others, though.

This means size of graphs (outgoingEdges) for ENHANCED_SEMANTIC_ROLES is not the same as for UNIVERSAL_ENHANCED.

For a CluProcessor.

Thanks! I'll debug this today.
On Fri, Oct 2, 2020 at 15:12 Keith Alcock<EMAIL_ADDRESS>wrote:

For a CluProcessor.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/issues/907#issuecomment-702979984, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI75TUGV6KTNOF6FZ56RJTSIZF6FANCNFSM4R7ZQXWA
.

I can't replicate this bug. I wrote a test for this here:
https://github.com/clulab/processors/blob/takethree/main/src/test/scala/org/clulab/processors/TestDepGraphSizes.scala
And it passes for me in this branch...

This seems to have been solved with https://github.com/clulab/processors/pull/428.


fix bug involving as patterns and list ranges

basically, if I have a pattern like (ListItem(_) L ListItem(_)) #as V, previously, L was not actually getting bound to anything, because the code that would do that is handled by the listRange function, and it thought it didn't have any such variables due to how it was previously implemented.


return @QueryResult from custom Cypher query at run-time

In my Neo4j/SDN 4 project I have a pretty complex Cypher query that can't be defined at the compilation time via SDN repository @Query.
I need to build this query at run-time and as a result I need to get a custom SDN @QueryResult object.
Right now I can't find a way how to achieve this with a Spring Data Neo4j 4.
session.query returns NULL for @QueryResult object.
Please help with this problem.

Any chance to get this feature implemented in some near time ? Unfortunately, I'm completely blocked with this right now and can't continue to work on my project without this feature :(

+1

+1

This is a nice idea.
As @QueryResult is part of Spring Data Neo4j and not OGM, we're tracking this issue over here: https://jira.spring.io/browse/DATAGRAPH-1159


Updated readme to reflect name change

Hey there!
Long time user and previous contributor. I'm submitting this PR in order to correct the inspiration of Refit to use Ana Betts' current name.

Gah! Sorry for dragging commits. I'll get this rebased onto cantor7:master first.


Add Sendable conformance

This PR adds Sendable conformance to both Netable itself, as well as the example project.
Netable
The changes in Netable were  mostly simple, with a few caveats. Every public store property within Netable has to conform to Sendable, as it's an actor. The most complicated of them is AnyPublisher and I have left a comment directly on that piece of code. In general, the Combine framework and Sendable are not a super big fan of one another. I'll discuss how this was mostly navigated lower down in the PR, but AnyPublisher threw the biggest wrench for sure.
We also previously discussed removing Sendable conformance from the requestFailureDelegate because it doesn't need to be Sendable given how it operates, but it does need to be Sendable given that it's a stored property within the Netable actor.
Example Project
Mostly of the issues were solved with declaring the code run on the MainActor, as discussed in this slack thread.
The only "odd ball" is the error service. I typically wouldn't want any service to be forced to run on the main thread, but given the nature of the error service in our case, I think it makes sense.

It looks like we've still got a whole bunch of Concurrency related warnings when building this project.
Are we able to resolve some of these, or is there some reason we can't deal with them?

The main project and both the Example project and Tests targets have complete currency checking selected but the netable target was apparently only marked minimal. I have changed this over and will fix this warnings.


When logged in as mentor only show "full" status when mentor list is full.

The class in the screenshot is full with students, not mentors.

It looks like this has been partially fixed, from what I can tell it looks like only non-mentors would see "Join Waitlist" if a class is at student capacity. But it'll still show "Volunteer" for mentors if it's at or over capacity. I'll try and add a condition that shows a waitlist option for mentors if it's at mentor capacity as well


New Drag API: DragUpdateEvent.canvasPosition is not transformed to local coordinates

Current bug behavior
The canvasPosition member variable of the DragUpdateEvent instance which is passed as a parameter to the onDragUpdate function is not transformed to local coordinates. Note that DragStartEvent.canvasPosition is actually transformed to local coordinate space.
With the old legacy Drag API everything works fine.
Expected behavior
DragUpdateEvent.canvasPosition should be transformed to local coordinates.
Steps to reproduce
When running a FlameGame not in fullscreen mode (e.g. with an AppBar at the Top), we can see the issue.
Example: https://zx9806jbx990.zapp.page/#/
Code:
import 'package:flutter/material.dart' hide Draggable;
import 'package:flame/components.dart';
import 'package:flame/experimental.dart';
import 'package:flame/game.dart';
import 'package:flutter/material.dart';
import 'package:flame/events.dart';

void main() {
  runApp(
    MaterialApp(
      home:
        Scaffold(
          appBar: AppBar(),
          body:
            GameWidget(
              game: MyGame(),
            ),

          )
      )
  );
}

class MyGame extends FlameGame {
  @override
  Future<void> onLoad() async {
    await add(TrailDragTarget());
  }
}

class TrailDragTarget extends PositionComponent
    with DragCallbacks, HasGameRef<MyGame> {
  Trail? _trail;

  @override
  void onLoad() async {
    await super.onLoad();
    anchor = Anchor.topLeft;
    position = Vector2.zero();
    size = game.size;
  }

  @override
  void onDragStart(DragStartEvent event) {
    super.onDragStart(event);
    _trail = Trail(event.canvasPosition);
    add(_trail!);
  }

  @override
  void onDragUpdate(DragUpdateEvent event) {
    _trail?.addPoint(event.canvasPosition);
  }

  @override
  void onDragEnd(DragEndEvent event) {
    super.onDragEnd(event);
    if (_trail != null) {
      remove(_trail!);
      _trail = null;
    }
  }
}

class Trail extends Component {
  Trail(Vector2 origin)
      : _path = Path()..moveTo(origin.x, origin.y),
        _points = [origin.clone()],
        _linePaint = Paint()
          ..style = PaintingStyle.stroke
          ..color = Colors.white
          ..strokeJoin = StrokeJoin.round {
  }

  final Path _path;
  final List<Vector2> _points;
  final Paint _linePaint;

  @override
  void render(Canvas canvas) {
    _linePaint.strokeWidth = 1;
    canvas.drawPath(_path, _linePaint);
  }

  void addPoint(Vector2 point) {
    if (point.isNaN) return;
    if (_points.length > 500) return;
    if (point == _points.last) return;
    _path.lineTo(point.x, point.y);
    _points.add(point);
  }
}

As you can see there is a hard cut between the canvasPosition coordinate provided in onDragStart and onDragUpdate:

Expected should be a closely straight horizontal line.
Flutter doctor output

Output of: flutter doctor -v
$ flutter doctor -v
[✓] Flutter (Channel stable, 3.10.3, on Microsoft Windows [Version
    10.0.19045.2965], locale de-DE)
    • Flutter version 3.10.3 on channel stable at
      C:\Users\<redacted>\Documents\Flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f92f44110e (3 weeks ago), 2023-06-01 18:17:33
      -0500
    • Engine revision 2a3401c9bb
    • Dart version 3.0.3
    • DevTools version 2.23.1

[...]

More environment information

Flame version: 1.7.3
Platform affected: web
Platform version affected: chrome

More information
The cause of this bug lies in the fact that the FlameDragAdapter simply forwards the DragUpdateDetails in update (here) without transforming the DragUpdateDetails.localPosition to local coordinate Space as it is done in FlameDragAdapter.start (here). As flutter does not provide a explicit DragUpdateDetails.localPosition upon construction, the DragUpdateDetails.localPosition variable equals the DragUpdateDetails.globalPosition, which is not transformed to local coordinates.
We can either transform the localPosition directly inside FlameDragAdapter.update (which requires a reconstruction of DragUpdateDetails however), or don't care about the localPosition provided from flutter at all and use globalPosition/devicePosition for all other position variables in PositionEvent such as PositionEvent.canvasPosition. We could make canvasPosition late final and calculate the transformation lazy at it is done in the old legacy Drag API with EventPosition.
NOTE:
With the old legacy Drag API everything works fine, because the DragUpdateDetails.localPosition isn't even used. Only DragUpdateDetails.globalPosition is used to populate EventPosition, which in turn keeps care of all the other transformations such as widget, viewport and game.

Thanks for a thoroughly written issue! We are quite busy this week and the next with FlutterCon that's why we are a bit slow at replying.

or don't care about the localPosition provided from flutter at all and use globalPosition/devicePosition for all other position variables in PositionEvent such as PositionEvent.canvasPosition. We could make canvasPosition late final and calculate the transformation lazy at it is done in the old legacy Drag API with EventPosition.

I think this second option probably sounds the best.
Do you want to create a PR for it? :)

Sure, i have just created a PR for this.


⚠ Bofrost.fr has degraded performance

In 260dd8c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1297 ms

Resolved: Bofrost.fr performance has improved in 4e7ca3a after 6 minutes.


Floating Popup Menu

Hi @SuavePirate,
Thank you for helping us with this great renderer.
Any thought on supporting a ListView on the ButtonClick event that opens up a dynamic floating menu [served via an ItemTemplate + dynamic data template containing floating menu (image, and label) ]  on the appropriate side on the screen depending on the button's position ?
Thanks,

I don't plan on implementing that in this control, since it isn't part of the native Android FAB, but perhaps as an extension. Waiting to see how Xamarin.Forms implements the built-in one: https://github.com/xamarin/Xamarin.Forms/pull/3295
It's not too difficult to do yourself now. Bind the icon to something dynamic, and have the click handler of the main fab show a hidden list of smaller fabs above it. Could work easily with Xamarin.Forms animations too.

@IoTFier you might get some inspiration from this one https://winstongubantes.blogspot.com/2018/01/transform-floating-action-button-into.html


Change minimum garbage_collection TTL to 1 day from 7

A very simple change to modify the minimum TTL on the garbage collector, pulling it back to 1 day from 7.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Adding PAsets, refactor some

This PR brings in the concept of a PhenotypeAssociationSet, which can be one of many for a dataset. It includes some modest code changes in how datasets and g2p relate. It will be finished when one can query for phenotype association sets (much in the same way you might search for variant sets), and the existing queries for phenotype associations accept a phenotypeAssociationSetId. I'll modify the existing tests for these changes as well. #941

You're on exactly the right path here @david4096.

Added client for searching PA sets, partial refactor of datamodel/genotype_phenotype.py more or less done. Incorporated Jerome's fixes and brought up to style concerns.

This looks good @david4096, I think you've gotten a long way here. I wonder if you're trying to bite off too much with this one though by taking on a wholesale refactor of the g2p code along with adding the new PASet type and its methods. Maybe it would be better to scale it back it a bit to just what's needed to support the PASet, and defer the big refactor until afterwards? (I definitely agree that a refactoring is required though.)

@jeromekelleher thanks for taking the time to review! I got rid of the generate functions in favor of the method being used previously (protocol.fromJsonDict).
The scaffolding is in place to add in the simulated responses and data driven tests.

+1 Clarifies contract with new AssociationSets. Addresses concerns about "middle tier" tests.  Nice!

@macieksmuga I added a few more comments :100:


docs(readme): update image url

Move to the new cms

@keithkade Can you make these PRs to the development branch first?

@jmeis Is changing the base ok, or should I open up a fresh PR?

@keithkade I think it might be better to open a fresh PR.

Opened up a new PR https://github.com/IBM/python-flask-app/pull/63


feat(@vtmn/css, @vtmn/svelte, @vtmn/react, @vtmn/vue): VtmnChip variant input match now the figma specs

Changes description

https://github.com/Decathlon/vitamin-web/issues/1455
Expected behaviour

Source : https://www.decathlon.design/726f8c765/p/08dbe1-chip/b/50988a
Context
Today, the chip have to be "selected" in order to display the cancel button

This PR aim to disable the selected on the variant input and by default, displayed the cancel button
Checklist

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[x] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[x] Check your code additions will fail neither code linting checks.
[x] I have reviewed the submitted code.
[x] I have tested on related showcases.
[x] If it includes design changes, please ask for a review design-system-core-team-design GitHub team.

Does this introduce a breaking change?

No

Hi @Tlahey, thanks for this contribution!
Can you please confirm that this PR will not cause any breaking change and that we can bump packages with a minor release for each of them?
Thanks 🙏

Hello @lauthieb
This enhancement will change the behaviour of the VpChip (caus' the select on the input disappears)
We can put a breaking change on the design but the input properties doesn't changed on the component.
The changes are only scoped on the "input" variant

Thanks @Tlahey for this explanation.
What do you mean about applying a BREAKING CHANGE for design?
You mean bump to major release your changes? Or just in Figma with a communication?
Thank you

Hello @lauthieb
Because the design now match the figma's version, the user who used the VtmnChip with a input variant will have changes when it will upgrade the version.

Hi @Tlahey,
Ok, thank you. So, I will merge it with a "BREAKING CHANGE" :)


Correct execution of test script.

Using ['run-script', 'test'] for the call is adding an additional parameter to the call. This can cause errors like below:
λ npm run-script test test

><EMAIL_ADDRESS>pretest C:\Users\bcreel\Documents\Node\muninn
> copy test\budgetsTest.sqlite test\budgets.sqlite

        1 file(s) copied.

><EMAIL_ADDRESS>test C:\Users\bcreel\Documents\Node\muninn
> tape test/tests.js | tap-difflet "test"

Usage: tap-difflet [options]

npm ERR! Windows_NT 6.1.7601

This is an easy fix; 'run-script' is not required since NPM directly supports 'test' as a command.

Is this project still being maintained?

Apologies, I've overlooked this incoming PR. Thanks for the PRs 🌹 . I'll publish a new version to the market place soon.


DE38001 Replace overflow hidden on content with something else

DE38001
This PR fixes the overflow issue in the editor guy. I also tweaked the _state stuff some so that it's consistent and more correct. I on RDP rn so I can't super well test the animation, but I did it with the 10% animation slow down thing and it looked ok.

can't add you two, but pls review @pureooze @kieranderson

@ppaskaris-d2l I tested the animations locally, they look great! 👍

oh, i forgot to feed the bot. how do i feed it now so it does a version bump?

New PR with whitespace change? 🤷♂
We should update this repo to auto release like the others lol

two steps ahead of ya 👉:sunglasses::point_right:


TypeError with theano

Hello,
I'm trying to run your training code with
python ./networkTraining.py ./LiviaNET_Config.ini 0
and I'm getting TypeError with theano 1.0.1 and python 2.7 and numpy 1.14.3

TypeError: ('TensorType(float32, scalar) cannot store accurately value 0.001, it would be represented as 0.001. If you do not mind this precision loss, you can: 1) explicitly convert your data to a numpy array of dtype float32, or 2) set "allow_input_downcast=True" when calling "function".', 0.001, 'Container name "None"')

that one was coming when callingmyLiviaNet3D.initTrainingParameters with myParserConfigIni.learning_rate
which I fixed with
np.cast["float32"](myParserConfigIni.learning_rate)
the same occurred with myParserConfigIni.momentumValue. Fixed likewise.
Unfortunately I'm now hitting the following error:

File "/usr/local/lib/python2.7/dist-packages/theano/gradient.py", line 1326, in access_grad_cache
term = access_term_cache(node)[idx]
File "/usr/local/lib/python2.7/dist-packages/theano/gradient.py", line 1162, in access_term_cache
new_output_grads)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/op.py", line 711, in L_op
return self.grad(inputs, output_grads)
File "/usr/local/lib/python2.7/dist-packages/theano/tensor/nnet/abstract_conv.py", line 2495, in grad
d_bottom = bottom.type.filter_variable(d_bottom)
File "/usr/local/lib/python2.7/dist-packages/theano/tensor/type.py", line 234, in filter_variable
self=self))
TypeError: Cannot convert Type TensorType(float32, 4D) (of Variable AbstractConv2d_gradInputs{convdim=2, border_mode='valid', subsample=(1, 1), filter_flip=True, imshp=(190, 100, 19, 19), kshp=(9, 100, 1, 1), filter_dilation=(1, 1), num_groups=1, unshared=False}.0) into Type TensorType(float64, 4D). You can try to manually convert AbstractConv2d_gradInputs{convdim=2, border_mode='valid', subsample=(1, 1), filter_flip=True, imshp=(190, 100, 19, 19), kshp=(9, 100, 1, 1), filter_dilation=(1, 1), num_groups=1, unshared=False}.0 into a TensorType(float64, 4D).

which is getting more complex and it seems to point at a general casting issue, maybe due to the newer version of theano i'm using?
Could you take a look and help me run your code?
Regards,

it seems running the code with the following env variable solves the issue:
THEANO_FLAGS='floatX=float32' python ./networkTraining.py ./LiviaNET_Config.ini 0
You might want to document that in your README :-)

Hi @iZsh,
Thank you for your comments. I am glad you found out the source of your error.
Have a look to the Readme file, on the 'known problems' section. This issue is reported there...:)
Best

oh gosh.


Isolated SPFx webparts are not loading in Microsoft Teams Desktop App

[x] Question
[ ] Typo
[x] Bug
Hi Team,

We created an isolated web part ( Just vanilla hello world web part ) and tried to load it in Microsoft teams. It worked pretty much fine in the Teams web version but not in the Teams desktop version. It just loads blank.

Here is the screenshot of fiddler trace and how web part is loading in teams (both web and desktop version).
Web Version

Desktop Version

Fiddler Trace:
Teams Desktop (vs) Web

@andrewconnell  @VesaJuvonen Please let us know if we are missing something out here..

Known issue that MSFT is working on. This is a dupe of #3819 so we'll close this issue.


Don't add "Speak selected text" to history

I use this Mac feature all the time, which utilizes the clipboard.
It would be great if there was a way to know that an entry was added to the clipboard using the speak selected text keyboard shortcut and not add those to the history.

I've tried to reproduce this behavior, but whenever the text is spoken it doesn't get added to clipboard. Neither to Maccy nor when I press CMD + V.

Strange, so it seems like only some applications use the clipboard and some do not.
So far it looks like Chrome, Firefox, VS Code, and Slack all are using the clipboard. These are all browsers or apps build with https://www.electronjs.org/

Unfortunately, I don't think I can figure out that the copied text is coming from "Speak selected text". Both in Chrome and Firefox it's copied with the following types which doesn't allow to reason about the source of copied text: ["public.html", "public.utf8-plain-text"].
I'll be happy to implement this but for now, I don't see if it's possible


Old and New Issues in Pull Request Analysis

Hello,
We have a workflow as follows:
we create a feature branch off 'main' for a sprint -> feature/sprint1 for example. Dev works on it locally and pushes to BitBucket and a creates new PR; At this point we run sonar with sonar-stash w/the following invocation:
$ sonar-scanner -Dsonar.analysis.mode=preview  -Dsonar.stash.notification=true -Dsonar.stash.project=stat -Dsonar.stash.include.existing.issues=false -Dsonar.stash.repository=StaticAnalysisPOC -Dsonar.stash.pullreque
st.id=5
We view results in BitBucket and find both the 'legacy' (pre-existing from main branch we created our feature branch from);
any assistance is appreciated.
SonarQube version: Version 6.5 (build 27846)
BitBucket version: 4.8

Can you confirm that you have ran a normal, full scan on the master branch before?
The analysis will find new issues since the last successful, full scan.

ah... excellent point... let me verify

@hax0rdlux3 Do you have any news?

Yes, sorry for the delay. We got everything working... I have a few other questions but to accomplish what I needed to do, I had to understand the workflow better, which included the running of a scan on my 'Master' (main) branch from which I created the feature branch and subsequently the PR. It works as intended.

How you got everything working please? We have similar issue. We are performing analysis on our main branch (named devel-master), put all issues are detected in pull requests instead of few of them which are really newly introduced.

@Drimix20 Are the settings for both types of scans compatible (most importantly the project key)?


Add Execution Time Measurement

Closed by https://github.com/Elisassa/Code-Formatter-Advisor/commit/2cd0189


Increase connection_timeout to PG in find end of WAL tests

And log postgres to stdout.
Probably fixes https://github.com/neondatabase/neon/issues/3778

Should I include the stderr change in my PR or did you not see my PR, or what is the plan?

will reply in your pr


Upgrade borsh to 1.0

Work on https://github.com/near/near-account-id/issues/5.

@frol I have bumped the minimum supported Rust version as borsh is currently a default feature. If we desire to do something else - let's coordinate.

All addressed!


Sanger UK Maternal-Fetal Interface data download

Hello!
I am looking to download the anndata file for this cellxgene project:
https://maternal-fetal-interface.cellgeni.sanger.ac.uk/
I don't see it in the data portal, but I see that it is using the cellxgene interface. Is there anyway to download the .h5ad file from this project?
Thank you!
Elze

Hey @elzerac !
The link you posted is a different implementation of hosted cellxgene desktop explorer (it is not maintained by the CZI team and the dataset is not hosted on https://cellxgene.cziscience.com/ ). You could try reaching out to the cellgeni group about your question.
Thanks!


New RPC to get team access request

Added teamName  argument to TeamListRequests RPC which will call new API endpoint to get access requests for a team that is: faster than the previous endpoint, and also works for subteam impadmins.
Added new argument to cli to benefit from new API as well.
(minor change :lipstick:) Changed CLI to write additional info to stderr and only use stdout for actual list of requests.
Make use of new argument in GUI. It probably needs further cleaning up, though.

cc @buoyad can you look at frontend side? thanks

@joshblum would you mind taking a look at go side?


Update tools to work in ArcGIS Pro

Had started this some time back but due to the large number of selections and size of some of the selections the performance was less than optimal. Need to get back in and review more and find a way through it.

This is resolved in the June 2016 release