text:
Identify all components of a piece

Pieces are given additional components to their numbers when they are split between counties. Coordinators need to be able to locate them
E0 U6 S9 C6: P21

A suggestion - if i could have a master list sent to me, i can go manually through any with a letter at the end and marry them up with what i have, then Kirk might be able to magic up a technique going forward so it updates in real time?

(I deleted my last comment of a few minutes ago)
I have just read Kirk's suggestion above - I have the logs of moves done since the 7th Feb and also have csv files of those, there are not a huge number so it would not take TOO long to put them together, then Kirk could perhaps work out a way of adding changes in real time as they happen, moving forward from now on

@johnt-freecen https://github.com/johnt-freecen  @rhodamackenzie https://github.com/rhodamackenzie
Yes the Coordinator will specify a year. That will set the filter.
If the file is then indexed on the piece number all associated piece numbers should appear together. (e.g. RG10_1111, RG10_1111A, RG_1111B etc) They will also be together in the ‘Home’ County of the original PARMS file before it was amended, with transferred to Counties interspersed between them. That is because they are in numerical order and the original TNA PARMS download is in numerical order. The other information requested just provides important information about the piece such as civil parishes etc.
If this is downloadable as a csv file the Coordinator can reconstruct their records from their part of this file.
At the moment a Coordinator has no easy way of knowing where parts of a piece have gone. They need a picture of their whole County. As Pat brings new Coordinators on board the move log will make less sense to them, whereas a simple, but large, table will be understandable.
I know the csv file will be large but for Somerset for instance I need to be able to see the pieces that have been transferred of created in Gloucestershire, Devon or Dorset, not just the ones that are still in Somerset. Why? Because I have the images in one block and have to manage them. If I filter by County I can no longer see the transferred pieces. If everything is in the csv file I can extract the parts of the csv file I need.
I hope that this makes sense.
Geoff
From: Kirk Dawson @.>
Sent: Sunday, May 9, 2021 7:42 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: geoffj-FUG @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] Identify all components of a piece (#1208)
@geoffj-FUG https://github.com/geoffj-FUG  @johnt-freecen https://github.com/johnt-freecen  @rhodamackenzie https://github.com/rhodamackenzie  Can I assume that the coordinator will specify a year? Will they also specify the original piece that may have had parts moved?
Without some degree of guidance it could be very time consuming task to look through all pieces to see if it has been split. We could start by finding all piece names that include and alphabetic character and look up the piece original piece number and see if they are different counties.
Alternatively perhaps we could look at combining the move results log to set up a collection that can be reviewed. Assuming that @rhoda https://github.com/rhoda  has kept them (I have not).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1208#issuecomment-835527863 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFMNCLEIK3I7LABBH53TMWV2BANCNFSM42X45UIQ .  https://github.com/notifications/beacon/AKCPIFMHWHV6QNFAJLSADEDTMWV2BA5CNFSM42X45UI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGHGSJNY.gif

@geoffj-FUG where do you want to see the action link? On the main action page or within a county. There are arguments for both options.

Kirk
My thoughts .
It is a Coordinator function.
My logic says that it should be under Manage County.
Perhaps under Manage FreeCEN2 Pieces. An additional index item at the end of
the menu 'List specific piece breakup'. The County filter should not operate
so that any piece entered for anywhere will be listed. That will help
Coordinators communicate with each other over problems and help them
identify where a piece that is not in their County on TNA came from.
Geoff
From: Kirk Dawson @.>
Sent: Monday, July 12, 2021 1:45 PM
To: FreeUKGen/FreeCENMigration @.>
Cc: geoffj-FUG @.>; Mention
@.>
Subject: Re: [FreeUKGen/FreeCENMigration] Identify all components of a piece
(#1208)
@geoffj-FUG https://github.com/geoffj-FUG  where do you want to see the
action link? On the main action page or within a county. There are arguments
for both options.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/FreeUKGen/FreeCENMigration/issues/1208#issuecomment-8779
46402> , or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKCPIFIQYEMHKIQHCLQDKRTTX
JQLPANCNFSM42X45UIQ> .
<https://github.com/notifications/beacon/AKCPIFONPGGH2IMB5BKBVL3TXJQLPA5CNFS
M42X45UI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGRKGMIQ
.gif>

<https://home.mcafee.com/utm_medium=email&utm_source=link&utm_campaign=sig-e
mail&utm_content=emailclient?utm_medium=email&utm_source=link&utm_campaign=s
ig-email&utm_content=emailclient>
Scanned by McAfee
<https://home.mcafee.com/utm_medium=email&utm_source=link&utm_campaign=sig-e
mail&utm_content=emailclient?utm_medium=email&utm_source=link&utm_campaign=s
ig-email&utm_content=emailclient>  and confirmed virus-free.

@geoffj-FUG Before I go any further have a look at test3 for a possible implementation; not fully integrated .bbut is this what you had in mind

Kirk
Coordinators don't think in Civil parishes, they thing in piece numbers. So
the most useful option is option 2 and the least useful is option 3.
Looking at the length of the result there is a problem in that I either
scroll for a long time or cannot get near the piece I need with the
elevator. It would be easier if a Coordinator could type in the primary
piece number having selected a year and then the set with that primary
number would be displayed.
So selecting 1851:
Then 1927
The results for 1927, 1927A and presumably 1927B transferred to DEV would be
displayed together with a list of the Civil parishes in each piece. (The
list below is from the PARMS breakup exercise, the civil parishes would all
appear together in each piece).
11
Ashill
HO107_1927
Chard
Ashill
OK

52
Broadway
HO107_1927
Chard
Broadway
OK

121
Cricket Malherbie
HO107_1927
Chard
Cricket Malherbie
OK

136
Donyatt
HO107_1927
Chard
Donyatt
OK

138
Dowlish Wake
HO107_1927
Chard
Dowlish Wake
OK

199
Ilminster
HO107_1927
Chard
Ilminster
OK

200
Ilton
HO107_1927
Chard
Ilton
OK

215
Kingstone
HO107_1927
Chard
Kingstone
OK

333
Seavington St Mary
HO107_1927
Chard
Seavington St Mary
OK

334
Seavington St Michael
HO107_1927
Chard
Seavington St Michael
OK

338
Shepton Beauchamp
HO107_1927
Chard
Shepton Beauchamp
OK

367
Stocklinch Magdalen
HO107_1927
Chard
Stocklinch Magdalen
OK

368
Stocklinch Ottersay
HO107_1927
Chard
Ilminster
Amend to Stocklinch Ottersey
link updated
431
West Dowlish
HO107_1927
Chard
West Dowlish
OK

448
White Lackington
HO107_1927
Chard
Ilminster
Amend to Whitelackington
link updated
58
Buckland St Mary
HO107_1927A
Chard
Buckland St Mary
OK

107
Combe St Nicholas
HO107_1927A
Chard
Combe St Nicholas
OK

218
Knowle St Giles
HO107_1927A
Chard
Knowle St Giles
OK

414
Wambrook
HO107_1927A
Chard
Wambrook
OK

449
Whitestaunton
HO107_1927A
Chard
Whitestaunton
OK

469
Yarcombe
HO107_1927A
Chard
Combe St Nicholas
DEV
Geoff
From: Kirk Dawson @.>
Sent: Tuesday, July 13, 2021 10:36 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: geoffj-FUG @.>; Mention
@.>
Subject: Re: [FreeUKGen/FreeCENMigration] Identify all components of a piece
(#1208)
@geoffj-FUG https://github.com/geoffj-FUG  Before I go any further have a
look at test3 for a possible implementation; not fully integrated .bbut is
this what you had in mind
<https://camo.githubusercontent.com/5bdcf8f0a124ce84f0d6fe81c62d6ce561180e19
ff9894f6940e2b0b17af96c8/68747470733a2f2f696d616765732e7a656e687562757365726
36f6e74656e742e636f6d2f3562363861653638346235383036626332626465666366302f383
73833343034612d323666362d343164612d616339362d363064363065326363383939>
<https://camo.githubusercontent.com/d653246a289b05be001175a4a22f75c220c78c05
bd185babe1b55327129fbc44/68747470733a2f2f696d616765732e7a656e687562757365726
36f6e74656e742e636f6d2f3562363861653638346235383036626332626465666366302f356
63331646162372d373532622d343037362d386530642d366561326634653930323131>
<https://camo.githubusercontent.com/c379cf31f1b55768ca3511d0a447ccc91aff8764
a9acd7ca060c8c54dc647875/68747470733a2f2f696d616765732e7a656e687562757365726
36f6e74656e742e636f6d2f3562363861653638346235383036626332626465666366302f323
86434373562632d313230312d346638372d383961392d653530363861326365636331>

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/FreeUKGen/FreeCENMigration/issues/1208#issuecomment-8786
88653> , or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKCPIFJ6DR4IEJEIZPQWXWDTX
OC7NANCNFSM42X45UIQ> .
<https://github.com/notifications/beacon/AKCPIFOQEXLY3P735NKVDRDTXOC7NA5CNFS
M42X45UI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGRP3TDI
.gif>

<https://home.mcafee.com/utm_medium=email&utm_source=link&utm_campaign=sig-e
mail&utm_content=emailclient?utm_medium=email&utm_source=link&utm_campaign=s
ig-email&utm_content=emailclient>
Scanned by McAfee
<https://home.mcafee.com/utm_medium=email&utm_source=link&utm_campaign=sig-e
mail&utm_content=emailclient?utm_medium=email&utm_source=link&utm_campaign=s
ig-email&utm_content=emailclient>  and confirmed virus-free.

@geoffj-FUG I am sorry to say your suggestion is not implementable as the initial piece may already have an alphabetic character associated with it. I have spent 3 days trying to find a solution and reached a brick wall and in addition the "numeric" part is not numeric and may be 1-4 characters. Some other programmer may be able to develop such code but I cannot.

@geoffj-FUG @rhodamackenzie I came at the problem a different way.
There is an Action called Locate

It takes you to a piece number entry

Locate then looks for that piece number and any others that start with that number in the complete series.

I do not know what piece moves have been done for SOM so it may not be especially enlightening. But if @rhodamackenzie has a move file for SOM we could run it on test3 so you can get a better sense of the action.
It is running on test3

i can't attach a csv file here but have sent moves files by email to Kirk and Geoff, so we can test on test3

Thanks @rhodamackenzie  run on test3
Results for HO107_933 follow

Looks good!
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Kirk @.>
Sent: 21 July 2021 00:21
To: @.>
Cc: Mackenzie, @.>; @.>
Subject: Re: [FreeUKGen/FreeCENMigration] Identify all components of a piece (#1208)
CAUTION: External email. Ensure this message is from a trusted source before clicking links/attachments. If you are concerned forward this email to @.***
Thanks @rhodamackenziehttps://github.com/rhodamackenzie run on test3
Results for HO107_933 follow
[HO107_933]https://user-images.githubusercontent.com/2874694/126407265-85a087dc-6bbb-493f-abf5-b64ed36f685a.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/FreeUKGen/FreeCENMigration/issues/1208#issuecomment-883768644, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO5M7OTCLCXZCMFW3EHM5FTTYYAIJANCNFSM42X45UIQ.
The University of Aberdeen is a charity registered in Scotland, No SC013683.
Tha Oilthigh Obar Dheathain na charthannas clàraichte ann an Alba, Àir. SC013683.

Kirk
Just tested this.
Looks good. It solves the problem.
It always helps to have different perspectives. Your solution is perfect.
Geoff

Kirk
This can be deployed and I will let the Coordinators know. I will put out an explanation that will assist with all the queries.
Geoff

repo: FreeUKGen/FreeCENMigration


text:
🛑 Minio Console Sekretariat-DJPb is down

In 1c7e84e, Minio Console Sekretariat-DJPb (https://minio-console.setditjen-djpb.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Minio Console Sekretariat-DJPb is back up in 076fc3f.

source: gharchive/issue

repo: FrideyAlpha/uptime-setditjen

license: MIT

license_type: permissive

license_source: github-api


text:
Add ShellCheck to CI

ShellCheck is a great static analyzer for shell scripts.
I propose to add it to CI.

I am willing to submit a PR after https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3548 is merged.

I don't think this must wait for #3548 though, it's even better to do this first IMO :)

authors: EvgenyOrekhov julienfalque

repo: FriendsOfPHP/PHP-CS-Fixer


text:
Priority: AlignMultilineComment should run before every PhpdocFixer

I made use of the graph I generated with https://github.com/dmvdbrugge/fixer-prio-graph
I also found a hidden priority between LineEnding and Braces, which I had to add in this PR as well, because otherwise the @Symfony_whitespace.test would fail.
Fixes #3879

I see plenty changes of getPriority methods for Fixers that are not Phpdoc*/Braces/LineEnding, without any reasoning added for those changes (keep in mind that when we do non-0 prio, we put a comment why - and those comments were either not added in your PR, or not updated while update of prio)

@keradus the answer to all your "why" is because of all the existing priorities. See the graph I mentioned, see all defined priorities, and see that all tests still pass.

See the graph I mentioned, see all defined priorities, and see that all tests still pass.

half of priorities are not covered with i-tests, eg even the one you are just saying you are fixing in this PR:

AlignMultilineComment should run before every PhpdocFixer

You added a priority relation between AlignMultilineComment and Phpdoc*Fixers, but you did not added integration tests for that.
For that, I'm not happy to change a priority without adding comment with reasoning and integration test for it.

Priority is one of the most tricky part of this tool. Maybe let's try to split this PR into atomic pieces, eg for what I understand, LineEnding and Braces can be extracted to separated PR.
Smaller PRs, easier to review & manual QA, easier to spot code changes that shall not be done or are missing tests.

Smaller PRs

This really is not that big of a PR...

You say this PR is small in terms of diff size. Indeed, it is.
Yet, I'm saying this PR is huge in terms of complexity, impact and potential risk.
Don't get me wrong - your work here is super nice, simply part of the code you are touching is super fragile, and any mistake here has big consequences.
And it's not trivial to validate if the changes are good or not. "small diff" you say? few lines to read, hard to understand the changes, impact impossible to predict, especially with missing tests.
And well, outcome of it is clear, nobody wants to review it (don't worry, I don't want to sound harsh here, this PR is on my watchlist and won't get forgotten for sure (even if I can't spend as much time with it as I would like to)), as nobody can say "it's safe to release this PR" - I'm trying to ask you for changes, that would make us more safe to release it.

I see plenty changes of getPriority methods for Fixers that are not Phpdoc*/Braces/LineEnding, without any reasoning added for those changes

As I mentioned before, it's a cascading change (again, see the graph, which visualises all current defined relations).

and for that, as it's all-in-one, hard to see out of the box which change in code is related to what change we want to accomplish.

half of priorities are not covered with i-tests

So we should write them. Please don't let that stop existing PRs that try to fix priority-problems.
(...)
Yes, sorry, none of the "special phpdoc cases" have integration tests as far as I could see.

So, if some legacy code has no tests, it's not an excuse to not create tests for new code/scenarios.
I'm not asking you to add in this PR tests for existing-not-covered cases.
I'm asking you to add them for new priority relation cases you are adding.
You added changes in "special cases" - provideFixersPrioritySpecialPhpdocCases https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/56557496da75f1feec1e12377c500f7a9b4688c2/tests/AutoReview/FixerFactoryTest.php#L219 . Those are that legacy cases without proper testing, and those are ignored in our test guard: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/56557496da75f1feec1e12377c500f7a9b4688c2/tests/AutoReview/FixerFactoryTest.php#L310 .
We shall not add more tests that are not covered by the guard.

For that, I'm not happy to change a priority without adding comment with reasoning and integration test for it.

The comment is there though. I could link to the issue as well. The reasoning I hope is clear to you: It fixes the *s of docblocks, they could be missing. And while that still is a valid docblock, there are fixers that won't run (correctly) in that case.

The comment part, especially after I will boyscout these (kudos :) ), is good part of the changes.
Lack of integration tests for newly configured priority relationship, is not.

Maybe let's try to split this PR into atomic pieces, eg for what I understand, LineEnding and Braces can be extracted to separated PR.

In theory yes, however that would block the current PR which needs to change those priorities, that's why it was found in the first place, and needed to be fixed as well.

I prefer to merge multiple small, easy and safe PRs, one after another, steps by steps, instead of having one problematic, long-living PR.

@dmvdbrugge #4245 was merged, can you please rebase this PR?

@julienfalque rebase done, had to work in a "new" (since a year ago) fixer, but all seems good again.

( i rebase to re-trigger the CIs with newest changes )

Hi @dmvdbrugge ,

AlignMultilineComment should run before every PhpdocFixer

where does this come from? I would like to right the priority integration test so we can move forward with this PR :)

a lot of integration tests were already added to cover some of those missing priority cases.
would you mind rebase and see what is still missing to cover with tests, @dmvdbrugge ?
i know it's a lot to ask, no need to rush on it ;)

I let myself integrate the integration test into the PR, with having the actual test proving the changes are solving the issue, I'm feeling much more comfortable merging this.
I synced the branch (ignore the merge commit and fabbot complain, I gonna squash it during merging anyway) and put milestone back.

Coverage remained the same at 91.901% when pulling 7919af966f3bbf838b84b2fb539858b48107c465 on dmvdbrugge:3879-AlignMultiline-before-Phpdoc into 76af6ebe7cd4fa51c6e2de5b540286642229fa0b on FriendsOfPHP:2.18.

@kubawerlos , i believe we should aim to drop numbers and have relations, like @SpacePossum described here: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5078#issuecomment-683424449

The failure we see in the test for PHP8 build is caused by not-known-before fixers relation between method_argument_space and function_declaration, and as we were not aware about the relation, we did not have explicit test for it.
Now, we accidentally discovered it with PHP8 integration test.
Having all relations discovered and documented with dedicated test is almost impossible, and changing the priorities like in this PR is putting us at risk to introduce such issues.
This time we were lucky that one of the test, even when not designed for it, discovered the problem.
FYI @dmvdbrugge @julienfalque @SpacePossum @kubawerlos

Trying to solve above issue in #5552 ,
then we should unblock this PR

Thank you @dmvdbrugge.

Thanks a lot for finishing this @keradus and my apologies for being awol.

source: gharchive/pull-request

authors: SpacePossum coveralls dmvdbrugge julienfalque keradus


text:
ExplicitStringVariableFixer - add test case for variable as an array key

Is the test case correct? Ping @Slamdunk for an answer and help with fixing if so.

Sounds legit: https://3v4l.org/dHF7s

Fix published in https://github.com/kubawerlos/PHP-CS-Fixer/pull/3
Also added test cases for "My name is $array[$foo][$bar] !": https://3v4l.org/ZhNsX

@SpacePossum not a WIP anymore

authors: Slamdunk kubawerlos


text:
Fix type null for nullable type parameters not added as type

I've started using PHP 7.1 nullable types and noticed, that Sami is not able to parse them. I saw the issue #264, but it does not fix the issue, at least not for parameters.
For example function setAvatar(?string $data) will result in a type of string, and not string|null as expected. The PHP documentation for that element is @param string|null $data.
This PR fixes this by adding the null type to the type array when resolving hints.

Thank you @CharlotteDunois.

authors: CharlotteDunois fabpot

repo: FriendsOfPHP/Sami

license: mit

license_source: bigquery


text:
Hotfix: yaml syntax error fixes

Fixes minor issues inadvertently introduced by #131
This goes on my list of "why NO to YAML" btw :-D

Thanks, good catches

@fabpot thanks for the quick merge ;-)

@Ocramius Thanks for the fix! We should have a look why the tests for #131 even passed with these issues.

Ah, sorry for not adding tests: I caught this upstream while working on
rave/security-advisories (own test suite depends on this package's
contents)
On Dec 12, 2015 20:35, "Christian Flothmann" notifications@github.com
wrote:

@Ocramius https://github.com/Ocramius Thanks for the fix! We should
have a look why the tests for #131
https://github.com/FriendsOfPHP/security-advisories/pull/131 even
passed with these issues.
—
Reply to this email directly or view it on GitHub
https://github.com/FriendsOfPHP/security-advisories/pull/132#issuecomment-164178066
.

I thought so, but it's weird that the included validation script did not spot them.

@xabbuh that's because it doesn't really check anything. The block at https://github.com/FriendsOfPHP/security-advisories/blob/7f01ec80c593ac9c234184a4fadb08bacd65a937/validator.php#L117-L125 (below):
                foreach ($branch['versions'] as $version) {
                    if ('<' === substr($version, 0, 1)) {
                        $upperBound = $version;
                        continue;
                    }
                    if ('>' === substr($version, 0, 1)) {
                        $hasMin = true;
                    }
                }

Should be something like a regex that matches:
foreach ($branch['versions'] as $version) {
    if (! preg_match('/^(\\<|\\>)(=){0,1}(\d+)(\.\d+)*$/', $version)) {
       throw new MichaelBay::explosions();
    }
}

I'll send a patch right away.

@Ocramius I would have expected the YAML parser to throw a ParseException before. If I didn't miss anything the file didn't even contain valid YAML before.

Ah never mind, the single quote is treated as part of the unquoted string. :(

@xabbuh isn't that beautiful? (hint: it isn't ;-) )

authors: Ocramius fabpot xabbuh

repo: FriendsOfPHP/security-advisories

license: unlicense


text:
Doku/README

Ein paar Infos zum Addon, wie es funktioniert, was man beachten sollte, wie man es als Dev integrieren kann (etwa für yform…), welche Systemvoraussetzungen notwendig sind, usw.

gerne stichwortartig zusammenschreiben, ich mach dann gerne den Text drum herum.

is hier noch was zu machen. ansonsten close.

Ich habe es leider nicht weiter verfolgt, weil ich zu viele andere Themen hatte. Wenn du das Gefühl hast, @darwin26, das die Doku ausreicht, dann schließe gerne.

authors: alexplusde darwin26 schuer

repo: FriendsOfREDAXO/multiupload


text:
Update ja.json

add new string and fix

Thank you for the additions and fixes!

Thank you for the additions and fixes!

repo: Frooxius/NeosLocale


text:
[BUG]

Hi. Just installed this plugin using Plugin Manager. All ok. Later installed Java, but JAVA_HOME is not set, so I added the path to the bin directory directly. Restarted Npp and now there is an error saying:
Java exec "path" does not exist
Make sure Java can be found by setting the right path in the plugins options
So, do I need to input the "path" or the whole "path\java.exe"?
Env:
win11
npp 8.5.2 64 bits
Thanks,

Hi @llagos ,
you need to set the whole path to the executable. It is also shown in the readme:

Have fun with the plugin.


text:
Create poc for new connector integrator UI module, framework agnostic

Connectors module needs to have an UI package that its framework agnostic, meaning the user can use it on React, Vue, or any other application.
This issue aims to implement a poc:

using Svelte to develop the same thing we have in "@fuels/react" module
create benchmark to compare current react module to poc svelte implementation
compare performance and compatibility

Can I jump on this task?

May I try my hand at this?

May I take this issue on?

I'd like to handle this task.

Hi, I’d like to work on this issue to create a framework-agnostic UI package for the Connectors module.
My approach:

Build a Svelte POC replicating the "@fuels/react" module functionality.
Benchmark React vs. Svelte for performance, rendering speed, and compatibility.
Test cross-framework compatibility with React, Vue, and others.
Document findings and provide a comparison report.

I have experience building modular, framework-independent UI components and optimizing performance across various environments. I’m confident I can deliver this POC effectively.
Looking forward to your approval!

Can I try solving this issue?

I'd be happy to do this.

repo: FuelLabs/fuel-connectors


text:
Collision with rust bindings if Sway fn is called new

closes: #277
The issue
The struct in whose impl block we generate bindings for calling the contract functions also has an associated function called new that is used to initialize the struct with data needed for the actual calls -- i.e. the wallet and the contract id.
Since Rust doesn't allow for fn overload, there can only ever be one new inside that struct.
Possible approaches
I'll list the ideas that came to mind on how to solve this. They are ordered starting from the one I favor most.
Approach 1: Do nothing
Accept that we don't support contracts that have new as a function.
Approach 2: The Builder pattern
For each contract generate two structures, for example:
MyContract and MyContractBuilder.
MyContract will hold all of the bindings through which you can call your contract's functions.
MyContractBuilder will be used to construct an instance of MyContract:
let my_contract: MyContract = MyContractBuilder::new(id, wallet).build();

Pros:

Users will recognize this pattern.

Cons:

None of the traditional reasons for using the builder pattern. It is used here just to eliminate the new in MyContract.
Breaking change.

Approach 3: Have new return something other than Self
Here we create two structs MyContract which will only have the new fn, and MyContractInstance which will contain the sway function bindings.
MyContract::new will, instead of returning an instance of MyContract, it will return an instance of MyContractInstance:
let my_contract: MyContractInstance = MyContract::new(id, wallet);

Pros:

Non breaking due to type inference. (Technically it is breaking..but less of a pain to fix)

Cons:

Unheard of, that new should return something other than Self. I.e. unexpected. Maybe if we call it a factory and return a trait object :D Java flashbacks.
Difficulty finding a good name for MyContractInstance.

Approach 4: Rename collisions
If we have a collision then just rename the rust binding of the sway fn -- i.e. call it new_():
let instance = MyContract::new(id, wallet);
let result = instance.new_().call().await?;

Pros:

Non breaking

Cons:

What if you also had a sway fn called new_? Would you call the original new__ ? What if you also had new__...?
If you have Sway functions of new and new_. Will you bind the second new_ as is and change new to new__ ? Or will you bind new as new_ and change new_ to new__? I.e. do you first bind already valid names and skip over the invalid ones, then do another pass or just do it all at once. Hell to document and probably to use.

Approach 5: Field inside MyStruct
Something like this:
let instance = MyContract::new(id, wallet);
let rtn = instance.functions.new().call().await?.value;

where functions is a public field of some type that will hold all of the sway function bindings and have no new of its own.
Pros:

...
Cons:
Ugly stepping stone

Approach 6: Struct init syntax
Have the fields of MyContract made public so you can use:
let my_contract: MyContract = MyContract { wallet: some_wallet, id: some_id };

Pros:

Recognizable to users.

Cons:

Users will no longer be able to pass a String as a contract id but would rather have to create a ContractId themselves:

   let contract_id = ContractId::from_str(&some_stringy_id).expect("Invalid contract id");
   let my_contract = MyContract { contract_id ... }

We will no longer be able to do any validation on the parameters passed to MyContract.
Users might change the underlying fields whenever they wish, possibly to invalid values.
Loss of encapsulation
Breaking change

Approach 7: Dynamic function dispatch
Don't generate bindings for reserved identifiers but rather offer a mechanism to let the user call a function without type safety:
let instance = MyContract::new(id, wallet);
let result: Token = instance.dyn_call("new", vec![some_tokenizable_value, some_other_tokenizable_value]).call().await?.value;
let real_result: SomeStruct = SomeStruct::from_token(result);

Pros:

non breaking
would work for all sorts of forbidden names, especially if there are some sway finds ok but rust doesn't

Cons:

I mean... total overkill and probably a pain to implement.

Wait so I don't understand, which approach did you go with? Shouldn't this discussion be in the issue, so we can decide before doing a PR?

Wait so I don't understand, which approach did you go with? Shouldn't this discussion be in the issue, so we can decide before doing a PR?

We went with approach number 2 (you can see in the PR's code that he's using .build()). There was a lot of back-and-forth in mixed channels, including in the issue itself.
The PR itself is the suggestion; it's short enough that we can make changes if it's not the best approach. But I believe 2 (.build()) is the least painful way to achieve this. And we (@segfault-magnet and I) thought about this quite a bit! But, as always, feel free to explore and suggest better approaches if applicable.

Ok! I just thought that the summary of the solutions was the discussion itself, didn't realize it already happened. I think it would be better to document the suggested solutions in the issue itself? Rather than it getting lost in a PR. Will review soon.

LGTM! But maybe in terms of semantics (and I think it fits well with reality too!), maybe calling it Generator or something would be better? So people don't get confused by a build() method (which is a pattern in itself iirc), but call generate() which might be less confusing? Just spitballing.

Normally I'd be ok with Generator, but in this case, I prefer Builder because:

We need to generate another struct where we will move our 'constructing' functions. The same is done in the builder pattern -- i.e. you have a SomeThingBuilder.
The Builder pattern will be recognizable to users -- they'd be like 'oh, you don't instantiate contracts directly, you use a builder' -- only because of this familiarity I've added the contract_id and wallet methods to make it look like the builder pattern even further. The only thing that is missing is to have a builder() -> SomeContractBuilder associated function in SomeContract -- we can't have that since that would bring us back to square 1 -- having a reserved name we can't use in contracts. If we do a Generate thingy we'd be doing something novel -- which isn't bad per se, but why not make it more familiar.

Make sure to do a nice explanation in the squash commit message :smile:

authors: digorithm segfault-magnet vnepveu

repo: FuelLabs/fuels-rs


text:
chore: downgrading forc back to 0.48.1

Forc 0.49.1 introduced some unplanned breaking changes that made life difficult.
I'm temporarily reverting it to 0.48.1 until everything is fixed.

I had to re-run the markdown-lint workflow 5 times, and it never worked.
Weirdly enough, the PR was merged when it got 2x approvals. 🤔

repo: FuelLabs/fuels-ts


text:
Updated the sway.js file by adding mod and type in keyword

Description
fixes #4424
closes #4424
Checklist

[ ] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[ ] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[ ] I have requested a review from the relevant team or maintainers.

thanks for the PR. Would you mind removing the dep keyword as well now that we have mod?

Thanks @Yasir761. Do you mind also running ./build.sh in https://github.com/FuelLabs/sway/tree/master/scripts/highlightjs?

thanks for the PR. Would you mind removing the dep keyword as well now that we have mod?

I made change as you suggested, Please take a review.
Thank You

Thanks @Yasir761. Do you mind also running ./build.sh in https://github.com/FuelLabs/sway/tree/master/scripts/highlightjs?

I ran command 'build.sh', Please have a look.
Thank You

authors: JoshuaBatty Yasir761 mohammadfawaz

repo: FuelLabs/sway


text:
Reproduction problem

Hi @FunmiKesa
Thanks for your great work.
I tried to reproduce your work, and found an error when I tried to train the network
raise InvalidGitRepositoryError(epath)
git.exc.InvalidGitRespositoryError: /home/MOT/JLA/src
It looks like a Git problem. I update my Git and still doesn't work.
Can you please tell me how to solve this?
Thanks.

@Leo63963 What command did you run that resulted in this error?

@FunmiKesa  Thanks for your reply.
I tried to reproduce your work. I follow the instructions, clone the code, create the JLA environment, install PyTorch, and download training data and pre-trained models, which are almost the same in FairMOT.
Then I tried to train on the mix datasets (crowdhuman pre-trained model) and evaluate the performance following the command you provided as follow:
sh experiments/mot17_half_ft_ch_jla.sh
and this error came.
It seems the error happened in:
        repo = git.Repo(search_parent_directories=True)
Do you have any idea how to slove this?
Thanks.

It could be a version issue. Try  replacing  that line with
repo = git.Repo(os.getcwd(), search_parent_directories=True)
You can also check the answers here https://stackoverflow.com/questions/22081209/find-the-root-of-the-git-repository-where-the-file-lives

@FunmiKesa Thank you so much for your reply.
Unfortunately, it seems to replace that line with the following code doesn't work:
repo = git.Repo(os.getcwd(), search_parent_directories=True)
I still got the same error as before.
However, if I just remove the following codes:
https://github.com/FunmiKesa/JLA/blob/68880423f5e559e6e020079e7e82b0bb64cad882/src/lib/opts.py#L386
https://github.com/FunmiKesa/JLA/blob/68880423f5e559e6e020079e7e82b0bb64cad882/src/lib/opts.py#L387
https://github.com/FunmiKesa/JLA/blob/68880423f5e559e6e020079e7e82b0bb64cad882/src/lib/opts.py#L388
The training code is running without any errors.
Do you think I can modify the code just like this?
Thanks.

repo: FunmiKesa/JLA


text:
Complete Toolbar to mirror menu structure.

Is your feature request related to a problem? Please describe.
Compliments on the new icons, but while we are working in this area, let me add this frustration:
Open network and save network are accessible from the toolbar, but new network and save as are not...
Describe the solution you'd like
Most programs in GUI land either have the toolbar mirror the menu structure, or better yet, make it configurable.
I don't think this is an urgent request, but one to keep in mind for the upcoming GUI redesign....
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

The most important have been added.

repo: FutureAIGuru/BrainSimII


text:
user-input

user-input
http://www.w3.org/TR/2000/WD-css3-userint-20000216#user-input

This issue was imported from Google Moderator
Moderator votes: +5
Issue added by epeleg on 2012-07-24
To vote this issue up or down, simply include +1 or -1 in your comment.

-1
It's unlikely that this was ever implemented outside of Mozilla's prefixed -moz-user-input.
The spec in question was never completed and was superseded by CSS3 UI, which doesn't define such a property.

Now available at https://caniuse.com/#feat=mdn-css_properties_-moz-user-input

authors: Fyrd cvrebert

repo: Fyrd/caniuse


text:
Update css-grid.json

links:

remove articles based on old spec
remove bug reports for implemented plateform (firefox & webkit)
add mozilla article

Good changes, thanks!

authors: Fyrd yukulele


text:
New version of dokan - fixes https://github.com/G-Node/WinGIN/issues/150

Added Dokan 1.3.0

changed API to new version
changed startup script to detect new version of dokan

we are shipping it... we did even before.... the dokansetup.exe is part of wingin installer
no we dont need that.. I will fix it

repo: G-Node/WinGIN


text:
Update to new deployment

@jgiannuzzi Could you confirm that this matches the details you had? I believe this should work, but GitHub Actions are a bit of a dark art.

I believe reviewers can't see the secrets available in the new deployment.

I confirm that contents:write is enough to create releases.

Hold onto your hats!

repo: G-Research/ApiSurface


text:
The logout button is not clearly visible.

Change the icon of the logout button from the face-icon to a more identifiable icon.

?

This order is better.

authors: cityremade dbauszus-glx

repo: GEOLYTIX/xyz


text:
R21C: Needed to handle OMPS-LP-N21

Ozinfo file was missing needed components to assimilate OMPS-LP from N21. I also changed the permissions in this file and another - no needed for these text files to be executables.
These are clearly only non-zero differences when data from N21 is available; otherwise, these are zero-diff.
CHANGES AIMED AT R21C BRANCH

N21 OMPS-LP also needs to be added to gmao_ozinfo.db (availabla.tbl and active.tbl). It will be identical to the N-SNN OMPS-LP except the start date.

Looks good to me

Actually: when I trying using these in an x-exp like settings, the code crashes in error when trying to read either one of the files in the following obs-classes:
BEGIN r21c_npp_ompslp_nc => ompslpnc_npp.%y4%m2%d2.t%h2z.nc
20130101_00z-21001231_18z 060000 /discover/nobackup/projects/gmao/input/dao_ops/ops/reanalysis/OMPS-LP/v2.6-adjusted-tint/Y%y4/M%m2/OMPS-LPoz-v2.6-adj.%y4%m2%d2_%h2z.nc
END
BEGIN m2scr_n21_ompslp_nc => ompslpnc_n21.%y4%m2%d2.t%h2z.nc
20230301_00z-21001231_18z 060000 /discover/nobackup/projects/gmao/m2scream/data/ompslp-n21/adjusted/Y%y4/M%m2/OMPS-LPoz-n21-v1.0.%y4%m2%d2_%h2z.nc
END
For the x-exp I put both N21 and NPP  together in the same experiment - the code seems to think the number of levels is 61+61=122 ... I can't put my finger one what could be wrong. I haven't tried in the R21C settings, but I checked the codes for ozone handling and they are the same between the R21C and what I am using for x-exp!

@kwargan can you point me to the scream experiment? I assume that experiment is using both sources of OMPS-LP?!

@kwargan I think I found a bug in the reading of the nc ozone files ...

@kwargan I now added to this what I believe is a fix the issue I had in trying to get more than one source of OMPS-LP processed in the code - just by augmenting the ozinfo table was enough for the code to be unhappy - even when a single source is passed gsi.rc ... the change in the Fortran code here fixes the issue. I am putting this here for M21C and also adding the same change in another PR for 5.41.x

@kwargan or @elakkraoui Can one of you approve this? This is good by me and I'd like to close on this.

Hi Matt, can you please approve this when you have a chance? This has been revised and it's also a part of what I have incorporated in the 5.41.x version of GEOSadas. Thanks.

authors: kwargan rtodling


text:
GEOS should halt nicely if running with MERRA2OX after June 2017

Every so often, @bena-nasa or myself get a ticket and the reason is that the user hit the MERRA2OX 2017 issue. The file is defined from 1979-02 to 2017-06:
pchem.species.CMIP-5.MERRA2OX.197902-201706.z_91x72.nc4

It should be feasible to do some checks in gcm_run.j to say "if date is X > 201706 and you are running MERRA2OX, exit" if you could trip this. It's not perfect, but it could help and I can think of how to do it in various ways.
The fun part, and why I need @bena-nasa help, is I can't seem to trigger this. I know it's possible, but all my attempts fail (in that the model runs and doesn't crash). I even tried running in 2019 (with Ops emissions due to the QFED v2.4r6 issue) but the model seems to run.

Update: The month matters as well. Turns out 2020-04-14 will run. not sure why. But 2020-12-14 will not!


text:
GCC9 + MOM6 Dies Reading Vegdyn

All, in trying to figure out https://github.com/GEOS-ESM/GEOSgcm/issues/177 once I got it working, I decided to run MOM6 with GCC 9. And oddly it crashes in vegdyn during the vegdyn.data read:
$ rg pe=00000 slurm-38517310.out
645:pe=00000 FAIL at line=01296    MAPL_IO.F90                              <status=5001>
902:pe=00000 FAIL at line=00981    MAPL_IO.F90                              <status=5001>
903:pe=00000 FAIL at line=05678    MAPL_Generic.F90                         <status=5001>
904:pe=00000 FAIL at line=01620    MAPL_Generic.F90                         <status=-1>

For some reason this seems familiar to me, though I can't seem to find any notes on it. @atrayano do you remember something similar somewhere? For some reason, the model dying here:
https://github.com/GEOS-ESM/MAPL/blob/2a1834a122ce5be5e0166e4463e70e78cce189fa/MAPL_Base/MAPL_IO.F90#L1293-L1300
reminds me of some issue in the past.

I believe @yvikhlya "fixed" this in the coupled model by changing/updating/fixing the boundary conditions.

This PR should fix this https://github.com/GEOS-ESM/GEOSgcm_App/pull/201. Basically, vegdyne in coupled model is now the same as in AMIP model. This works with intel compiler. I did not test a coupled model with gcc though.

repo: GEOS-ESM/MAPL


text:
DyCore GEOS-FP on Discover

Get a GEOS-FP Fortran reference at multiple resolutions:

C720: target
C180: low GPU bandwidth
C24: debug

Full configuration:

GCM

AGCM

Dycore: FV3 (Lead: William Putman)

Non-hydrostatic
Non-stretched
Non-regional

Moist (Lead: William Putman)

GFDL_1M
GF2020

Radiation: RRTMGP (Lead: Peter Norris)
Chemistry:

GOCART.data
PCHEM

OGCM: Data ocean

Final storage place: /discover/nobackup/projects/geosongpu/geos_data/geos-fp
Matt T. is working on it.

repo: GEOS-ESM/SMT-Nebulae


text:
[Title] 타이틀 화면 사운드 디자인_2

#354 에서 나온 피드백 (게임 사운드들 추가) 참고하여 사운드 디벨롭 작업

이왕이면 사람들이 알만한 고전게임 사운드들을 넣고싶어서 정안님께 추천받은 것 중 팩맘 + 버블버블 + 메탈슬러그 + 핀볼 + 그 외 게임 사운드 들을 짧게 짧게 간헐적으로 추가해봤습니다.
(노이즈를 섞어볼까도 생각했는데 지금 엠비랑 살짝 맞지 않는 것 같아서 그냥 올리는데 혹시라도 노이즈 섞인 게임 사운드를 원하신다면 이것두 다시 해서 업로드 해보겠습니닷)
컨셉 사운드 중 엠비언스로 만들어둔 사운드도 하나 추가했습니다.
BGM 제목은 Grayed Game's Whisper 입니다. 누르시면 바로 영상으로 연결됩니당. 구드에 오디오 파일도 있습니다.
(맞는 표현인진 모르겠지만 사운드트랙 다 영어로 하길래 영어로 꾸역꾸역 지엇어요..)
넣은 게임 사운드가 조금 크거나 거슬리는 부분이 있으면 말씀해주세요!
또 넣고싶은 고전게임 사운드들도 환영합니당
피드백 편하게 해주시면 감사하겠습니닷ㅎㅎ

사실 사운드만 들었을 땐 공포스러울 것 같아서 걱정했는데, 사운드가 아트랑 잘 맞아서 너무 만족합니다!
분위기는 너무 좋은데, 과거 컨셉 사운드를 제작해주셨을 때 아이디어로 넣어주신 게임 사운드를 아주 작게 재생하면 좋을 것 같다는 의견이 많이 나왔는데, 그 버전으로 제작되면 좋겠네요!

https://drive.google.com/file/d/15bAKWjsGLEzmA-UDGxZH8BSaZQolKP0x/view?usp=drive_link

링크로 찾았는데 가능하시다면 혹시 컨셉 사운드도 네이밍이 좀 더 명확할 수 있을까요?

수정이 가능하시면 수정해주시면 좋을 것 같습니다. 양이 조금만 많아져도 문제가 될 것 같아요!

컨셉 사운드는 더 만들 것 같진 않아서 양이 늘어날 것 같진 않아서요..!
일단 네이밍은 한번 생각해보고 고쳐두겠습니닷.
본문은 수정해뒀습니다!

와 미쳤다.. 피드백은 내일 하겠습니다..

너무 좋아요!! 저번에도 좋았지만 게임 사운드가 섞이니까 확실히 더 게임과 잘 어울리고 좋은 것 같습니다.
브금 제목도 너무 잘 어울리네요. Grayed Game's Whisper라니.. 엠비 언스인 만큼 정말로 이 게임이 속삭이는 소리인 것 같아요. 게임의 시작이라는 느낌이 강하게 들어서 임팩트 있는 시작이 될 수 있을 것 같고, 첫 시작인 떨어지는 사운드와도 잘 어울릴 것 같아 연결된 모습도 기대됩니다!
더 이상 하고 싶은 피드백은 딱히 없는 것 같습니다. 아트나 게임 분위기와 너무 잘 맞고 퀄리티도 좋아서 게임 시작이 기대되는 타이틀 화면인 것 같습니다!

저도 마찬가지로 너무 좋아서 피드백 드릴 건 없습니다. 공간감을 적당히 주면서 다양한 게임 사운드를 녹여낸 느낌이 정말 좋아요! 타이틀 화면과도 잘 어울리고, 마을에 떨어진 게임들의 사운드가 메아리처럼 울려 퍼지는 분위기도 정말 매력적입니다!!

파일명도 바꿔주시고 제목도 지어주시고 사운드 입힌 영상까지 올려주시다니 완벽.. ><
제목도 너무 멋진데요??!
이전 버전도 퀄리티가 워낙 좋았는데,
게임 사운드가 추가된 현재 버전은 세계관과 타이틀 아트랑 더 잘 어울리네요
굿굿~

너무 아름답습니다... 제가 그리면서 상상한 타이틀 이미지랑도 너무 잘 맞는 것 같아요 곡 제목도 너무 좋습니다!!

authors: Songkihwa Sumindd fkdl0048 roweclaw royalbluesm wooyn730


text:
decide which documentation generator to use

docosaurus, gitpages, etc.

https://rust-lang.github.io/mdBook/guide/creating.html looks good!

Maybe just co-locate the documentation with the examples in a next.js app.

Maybe just co-locate the documentation with the examples in a next.js app.

I do like this idea, Am looking at Nextra

Currently trying out Nextra over at https://use-cardano.alangaming.com/ liking it so far!

repo: GGAlanSmithee/use-cardano


text:
红点内是文字的情况下，偏移失效

如题，当红点内有内容时，view.redDotOffest失效，大神能修复下嘛

您好，因为UITabBarItem的显示文字的时候是用的系统的badge，无法设置offset，所以我想与其统一就没有左view的badge offset；
不过您提出这个需求，我暂时将offset设置成可以影响badge的offset，不知道这样做是否好，是否会影响其他用户的需求，或者说我单独增加一个badgeOffset更好？
请给出一些建议；
我暂时将改动提交到github，并暂时不加入到pod中，等此项修改完善后再更新pod
非常感谢您的建议

普通view设置文字内容时，badgeOffset也失效，我看了下是您把label的约束固定了，导致传入文字时,调整badgeOffset也失效

其实这并不会影响其他用户的需求吧，如果他不想偏移的话不设置即可，谢谢大神的回复.

我还是觉得多加一个badgeOffset比较好，因为有些人可能只想对小红点进行offset或者只想对badge进行offset，或者2个的offset不相同（根据当前项目UI设计），如果用一个offset来控制显然不是很好的方式，我又更改了一下，增加了badgeOffset的设置

你可以在Demo中的第二个item来看下效果

楼主考虑的很全面，我发现还有个小问题，就是当我badgeValue有值得时候，不是小红点时，无法隐藏了,不知我是不是没看到那个方法

是的，其实这个也是希望跟系统的UITabBarItem保持一致，系统的badge是否隐藏取决于badge是否为空，毕竟小红点是小红点，显示文字的badge是badge，都用一个方法控制不是非常的好，所以这个很难统一，怎么设计都会有使用者因为思维方式不一样觉得不对劲，所以就先这样啦，总之你可以通过badgeValue == nil来控制隐藏，试试吧

恩恩 好的

repo: GJGroup/GJRedDot


text:
Ajouts d'outils de base pour lsusb, lspci

Je viens de m'apercevoir en débugguant qu'il manquait des outils standard comme usbutils, pciutils.
Je les rajouterais dans la soirée

Il faudrait aussi ajouter GIT , ne serait ce que pour utiliser le cron qui va update la config :D

Si je n'ai pas foiré mes tests, git est normalement déjà installé et le script fonctionne OOTB, mais nous pouvons quand même le déclarer pour être tranquille. :p
Je m'en charge dans la journée

Tâches :

[ ] pciutils
[ ] usbutils
[ ] git

authors: CORAAL camini

repo: GLF-OS/Nixos-by-GLF


text:
Date and Day is incorrect

The Date and Day are incorrect in the date range picker.
For example:
September 2019 starts from Sunday(Sun) but it displays Monday.
Checked Version: 0.7.0 and 1.1.1

Hi Team,
to fix this issue in the file located
"https://github.com/GNURub/ngx-daterangepicker/blob/master/projects/ngx-daterangepicker/src/lib/ngx-daterangepicker/ngx-daterangepicker.component.ts"
replace line no 218 with the following code

Thank you!! The package related issue should be resolved by author and release new version.
If use npm install (https://www.npmjs.com/package/ngx-daterangepicker) and package.json the issue is not resolved.

repo: GNURub/ngx-daterangepicker


text:
Task09 Георгий Кашин ИТМО

Супер прикольно) но как-то его слишком уж сильно жмыхает от подсчета отброшенной тени, по виду похоже что лучи упираются в саму модель

Поправил)

Все хорошо, задача зачтена, 7/7 баллов :+1:

authors: gkashin simiyutin


text:
Warp Affine tensor HIP

Adds support for warp affine tensor on hip.
Support for U8/F32/F16/I8, NCHW/NHWC/toggle, Bilinear/NearestNeighbor is added.
Adds corresponding unit tests and performance tests.

@r-abishek failing the code analysis - details

@kiritigowda Yes, actually I checked it out it. It says - Variable 'src_f6.x' is reassigned before using the previous assignment, but I do use using the previous assignment from L1763-L1768 (https://github.com/GPUOpen-ProfessionalCompute-Libraries/rpp/blob/baa063da95b36fb307ccaf71b30ac5f730973048/src/include/hip/rpp_hip_common.hpp#L1763-L1768)
Not sure if codacy understood the pointer cast on L1757

@kiritigowda Yes, actually I checked it out it. It says - Variable 'src_f6.x' is reassigned before using the previous assignment, but I do use using the previous assignment from L1763-L1768 (
https://github.com/GPUOpen-ProfessionalCompute-Libraries/rpp/blob/baa063da95b36fb307ccaf71b30ac5f730973048/src/include/hip/rpp_hip_common.hpp#L1763-L1768
)
Not sure if codacy understood the pointer cast on L1757

Thanks, I will close the issue.

@asalmanp @rrawther @r-abishek
I ran the tests for this PR and everything looks good.
The only possible concern is that some of the images have grey backgrounds instead of black. Other than that, everything looks as it should.

@paveltc Thanks! The grey backgrounds are just for i8 due to the range from -128 to 127, which is usually the case for all rpp functions.

@asalmanp @rrawther I've made the changes for the make_float4 in all applicable cases. I've also added some comments to improve readability - basically in load/store helper functions to indicate which pixel-writes are happening and in what layout (pln3/pkd3) - https://github.com/GPUOpen-ProfessionalCompute-Libraries/rpp/pull/96/commits/789abc016f2a53d7a5f500dd01ad779fdf622023

authors: kiritigowda paveltc r-abishek


text:
cryptic error for VGP with no data

Bug
VGP produces cryptic error when provided empty data
To reproduce
Minimal, reproducible example
Python 3.7.5 (default, Nov  7 2019, 10:50:52)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import gpflow
>>> import numpy as np
>>> kernel = gpflow.kernels.SquaredExponential(1.0, 0.1 * np.ones([2]))
>>> likelihood = gpflow.likelihoods.Bernoulli()
>>> gpflow.models.VGP((np.ones([0, 2]), np.ones([0, 1])), kernel, likelihood)
2020-09-09 13:04:58.751898: W tensorflow/core/framework/op_kernel.cc:1767] OP_REQUIRES failed at strided_slice_op.cc:108 : Invalid argument: slice index -1 of dimension 1 out of bounds.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/joel.berkeley/git/GPflow/GPflow/gpflow/models/vgp.py", line 77, in __init__
    self.q_sqrt = Parameter(q_sqrt, transform=triangular())
  File "/home/joel.berkeley/git/GPflow/GPflow/gpflow/base.py", line 110, in __init__
    _validate_unconstrained_value(value, transform, dtype)
  File "/home/joel.berkeley/git/GPflow/GPflow/gpflow/base.py", line 217, in _validate_unconstrained_value
    unconstrained_value = _to_unconstrained(value, transform)
  File "/home/joel.berkeley/git/GPflow/GPflow/gpflow/base.py", line 234, in _to_unconstrained
    return transform.inverse(value)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/bijectors/bijector.py", line 864, in inverse
    return self._call_inverse(y, name, **kwargs)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/bijectors/bijector.py", line 844, in _call_inverse
    return self._cache.inverse(y, **kwargs)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/internal/cache_util.py", line 268, in __call__
    return self.lookup(input, **kwargs)[0]
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/internal/cache_util.py", line 356, in lookup
    self._invoke(input, kwargs, attrs))
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/internal/cache_util.py", line 393, in _invoke
    return self._func(input, **kwargs)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/bijectors/bijector.py", line 523, in <lambda>
    inverse_impl=(lambda y, **kwargs: self._inverse(y, **kwargs)),  # pylint: disable=unnecessary-lambda
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/bijectors/fill_triangular.py", line 95, in _inverse
    return fill_triangular_inverse(y, upper=self._upper)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow_probability/python/math/linalg.py", line 993, in fill_triangular_inverse
    initial_elements = tf.reverse(x[..., -1, :], axis=[ndims - 2])
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/util/dispatch.py", line 201, in wrapper
    return target(*args, **kwargs)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/ops/array_ops.py", line 1024, in _slice_helper
    name=name)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/util/dispatch.py", line 201, in wrapper
    return target(*args, **kwargs)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/ops/array_ops.py", line 1196, in strided_slice
    shrink_axis_mask=shrink_axis_mask)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/ops/gen_array_ops.py", line 10320, in strided_slice
    _ops.raise_from_not_ok_status(e, name)
  File "/home/joel.berkeley/.venv/GPflow/dev/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 6843, in raise_from_not_ok_status
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: slice index -1 of dimension 1 out of bounds. [Op:StridedSlice] name: fill_triangular/inverse/fill_triangular_inverse/strided_slice/

Expected behavior
Either support empty data, or detail the problem in the error message
System information

GPflow version: latest develop as of writing (commit hash 7b2a0c8884dc0fe0dfc5090749b355e68c8563fb)
GPflow installed from: pip install -e . from a GitHub clone on the develop branch
TensorFlow version: 2.3.0
Operating system: ubuntu 18.04

Hm, this seems more an issue of the implementation of the underlying tensorflow_probability FillTriangular bijector:
import numpy as np
import tensorflow_probability as tfp
bij = tfp.bijectors.FillTriangular()
q_sqrt = np.ones([1, 0, 0])
bij.inverse(q_sqrt)  # fails

Whereas ideally the FillTriangular bijector would return an empty array of shape (1, 0). Note that forward mode works correctly:
assert bij.forward(np.ones([1, 0])).shape == (1, 0, 0)  # works

How often (and in what use-cases) do we expect users to provide empty data to a VGP model? What would "support empty data" look like, in a meaningful way?

ok. maybe a bug I should raise with tfp

How often (and in what use-cases) do we expect users to provide empty data to a VGP model? What would "support empty data" look like, in a meaningful way?

I'm not entirely sure, I was briefly looking at how to define GPflow models as pure priors (with no initial data). I'm not sure if there's a use case for that yet

Closing as this is an underlying tensorflow-probability issue

authors: joelberkeley-pio st--

repo: GPflow/GPflow


text:
[BUG] Mining rig doesn't seem work

Description:
The mining rig seems not to work.
When I power it up, give it a drill and a data card it cycles once (not giving any ores, just drill head circle passing once) and then stops. When I try to reinsert the data card, it doesn't even cycle once. If I remove the drill and then add it again, it says drill inactive.
Is something in my setup wrong, or is it just a bug?
Steps to Reproduce:
Minecraft 1.18.2 or 1.19.2
indrev 1.14 or 1.15
Technical Information:

Minecraft: 1.18.2 or 1.19.2
Fabric API: 0.61 for 1.19.2
Industrial Revolution: 1.14 or 1.15
Modpack: Standalone or in All of fabric 5

Details:

i fing out what is happening
if you just take out item pipes and place chest next to mining rig computer it will just place ores into chest so problem was in pipes

Nice! That's good to know - can at least use it :D
Confirmed this on AOF5 1.18.2.
THX!

repo: GabrielOlvH/Industrial-Revolution


text:
Changes to run on CUDA with TchBackend

This library is awesome, thank you. Incredibly fast and a much nicer API than alternatives.
I was hoping it would be the magic bullet that works on M2 and CUDA so that it can be deployed (running services from a MacBook seems the only option with these models!).
I tried last night on AWS with TchBackend and ran into:
Could not run 'aten::empty_strided' with arguments from the 'CUDA' backend.
After that I noticed your chunk branch used the same settings I'd used.
It looks like empty_strided isn't available on CUDA at all, and models using it need to be moved to the CPU.
Is it possible to use alternative methods in the tensor constructors (?) so that it's compatible with both WGPU and CUDA? Or do you have any pointers - did you get it working with Tch initially?

this worked - thank you.
For anyone else:
I also needed libtorch and to set LIBTORCH, which is alluded to in the ticket, but there's more details here in the tch dependency:
https://github.com/LaurentMazare/tch-rs

@antimora I get "Segmentation fault" as soon as it attempts to load a model when using webgpu on AWS G4ad instance (AMD Radeon Pro V520 GPU supporting Vulcan / OpenGl APIs).
I've tried it with AutoGraphicsApi, and Vulcan/OpenGl specifically.
Is there anything I should try or do you think this is a fatal issue atm?
My objective is to get whisper-burn deployable as a service (without shipping a Mac). I'm there with Conda on Tch backend though, but would be good to be able to use AutoGraphicsApi and PR things back into here.

@jbrough I am not sure but it's worth looking into this. Do you mind refiling with burn so the team gets needed attention? I don't think anyone else from the burn team is monitoring this repo. You can just copy your last comment. It'd be also super helpful if you could share the actual error you're seeing.

authors: antimora jbrough

repo: Gadersd/whisper-burn


text:
Investigate why support for modern kedro datastes breaks for kedro<0.19.7

Description

#590 introduced support for modern kedro datasets, i.e. with load and save defined on user side instead of private _load and _save. This breaks my tests locally for kedro<0.19.7
FYI @deepyaman, I'll take a stab at it.
Context

I should ensure retrocompatiblity, and this change isn't
Steps to Reproduce

Install the last version of master , install kedro <0.19.7, run tests.
Expected Result

Actual Result

-- If you received an error, place it here.

-- Separate them if you have more than one.

Your Environment

kedro and kedro-mlflow version used (pip show kedro and pip show kedro-mlflow):
Python version used (python -V):
Operating system and version:

Does the bug also happen with the last version on master?

@Galileo-Galilei Do you have a stack trace handy, by chance? The only thing I can imagine is if super().save and super().load aren't defined/raise attribute error, in which case may need the if condition to be if hasattr(super(), "save") and ..., etc.

There is an "obvious" (after 1 hour of debugging >< ) error in the code : This line misses a return statement and for the old dataset style, it leads to return  None when loading.
I am a bit worried that is was not caught by this test in the first place :confused:

There is an "obvious" (after 1 hour of debugging >< ) error in the code : his line misses a return statement and for the old dataset style, it leads to return None when loading.

Gah, sorry!!

No problem, I didn't see it either!

authors: Galileo-Galilei deepyaman

repo: Galileo-Galilei/kedro-mlflow


text:
ML-DSA: add hint-related functions

Closes #189.
This adds a bunch of fairly straightforward functions for extracting high and low bits from integers and then making hints out of them.
There's a little bit of funkiness due to name overloading in the spec -- the functions are defined over single values but called over vectors of polynomials of values. So there's a bunch of iteration that is all implicit in the spec. I wanted to make the call sites for these functions as simple as possible, so I contained the single-value versions of the functions inside the main functions, but it does change the definitions a bit.

Thank you for breaking this down nicely commit by commit!

authors: marsella mccleeary-galois

repo: GaloisInc/cryptol-specs


text:
How to run the project

Can you provide the steps required to run the project???

Did you have any luck with this? I'm starting to feel really dumb XD

authors: laphrodiite stzogas

repo: GanniPiece/MetU


text:
ROS TF

Use the ROS' TF library to publish and track the transformations between frames of the world, robots, and its components.
:heavy_check_mark: Publish the map, odom, and base_links frames
:heavy_check_mark: : Publish each component tf, e.g., lidar

as reference:

http://wiki.ros.org/navigation/Tutorials/RobotSetup/Odom

Done in


text:
consider putting the example app within the test/dummy/* app

https://github.com/Gaurav0/ember-cli-jquery-ui
This keeps it self contained, and visiting ember server in the addon will display the demo.
This also becomes a great place to add unit/integration/acceptance tests

@Gaurav0 I don't think he meant for it to be served with heroku. He was talking more about actually adding a dummy application so that when you checkout your project and run ember serve you can see the dummy app working.

There are no points on Ember Observer for having a dummy application.
I really don't want to maintain a dummy app in addition to my separate example application.

Oh, I didn't see you had a example app as well.
Also, my mistake about the emberobserver points.

authors: Gaurav0 seawatts stefanpenner


text:
Rewrite of sd3latentselectres.py to overcome issue#1

Complete rewrite for several issues. Adds a multiply factor, which may not be needed but need to see SD3 capabilities.

I've mostly used the code fixes in here, however I didn't see the point of multiply factor (which is what I wanted to remove) as SD3 can only work at it's native resolution (1 megapixel), upscaling has to be done with tiles.

authors: GalaxyTimeMachine GavChap


text:
Dom Exception when using keyboard to remove first tag

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request/question

Current behavior
I am using angular material2 beta 3 library. I have a sidebar widget in which I am using the ng2-tag-input component.

Select the first suggestion.
Then press backspace 2 times. 1st time to select the chip, 2nd time to remove the chip.

I get an error
core.es5.js:1084 ERROR DOMException: Failed to execute 'animate' on 'Element': Partial keyframes are not supported.
    at WebAnimationsPlayer._triggerWebAnimation (http://localhost:5000/dist/vendor.js:103429:35)
    at WebAnimationsPlayer.init (http://localhost:5000/dist/vendor.js:103412:29)
    at InjectableAnimationEngine.DomAnimationEngine._queuePlayer (http://localhost:5000/dist/vendor.js:102703:16)
    at InjectableAnimationEngine.DomAnimationEngine.animateTransition (http://localhost:5000/dist/vendor.js:102618:14)
    at InjectableAnimationEngine.DomAnimationEngine.setProperty (http://localhost:5000/dist/vendor.js:102489:18)
    at AnimationRenderer.setProperty (http://localhost:5000/dist/vendor.js:90507:26)
    at setElementProperty (http://localhost:5000/dist/vendor.js:9893:19)
    at checkAndUpdateElementValue (http://localhost:5000/dist/vendor.js:9812:13)
    at checkAndUpdateElementInline (http://localhost:5000/dist/vendor.js:9752:24)
    at checkAndUpdateNodeInline (http://localhost:5000/dist/vendor.js:12439:23)
    at checkAndUpdateNode (http://localhost:5000/dist/vendor.js:12414:16)
    at prodCheckAndUpdateNode (http://localhost:5000/dist/vendor.js:12867:5)
    at Object.eval [as updateRenderer] (ng:///Ng2DropdownModule/Ng2DropdownMenu.ngfactory.js:300:5)
    at Object.updateRenderer (http://localhost:5000/dist/vendor.js:12784:70)
    at checkAndUpdateView (http://localhost:5000/dist/vendor.js:12386:14)

I am using angular 4. I am using the chrome browser.

Are there any custom styles applied?

The tag-input component is sitting inside the md-sidenav component. There are no custom styles applied to the tag-input component.

Also note, that this issue only happens when a chip is removed via backspace. If I use the mouse, everything works great.

@vadim82 does it also happen if the tag-input is not inside md-sidenav? If it doesn't, I am not sure if the issue is in this component.

Fair point. I tried using this component inside an angular-material2 md-card component and had the same issues. So it seems like the 2 libraries do not play nice together. Something to do with the animate library. Its possible that it is an angular material issue, but I am only seeing this error in the ng2-tag-input component, not in any other ones. So I was just wondering if maybe there is some animation being triggered that is causing this error.

That error usually occurs when the animated component doesn't have a size pre-defined. I struggle to understand though why this would happen :(

Is there some kind of flag we can set to disable animations? At least that would allow us to continue using this library.

@vadim82 I haven't found none that would allow me to optionally include animations. I'd suggest you to fork and remove the animations, or maybe extend the component and override the animations

Having exact same issue. Tag-input is not inside an angular material component

authors: Gbuomprisco meleskim vadim82


text:
Does the popover/menu component support interaction with elements on outside clicks on web?

It seems like the popover/menu component currently acts like a modal. Is it possible to add functionality for users to interact with content outside the menu when the menu is open?
This is the current behavior of react-native-popper by @intergalacticspacehighway. I'm currently implementing the functionality in a custom  Menu component but it would be nice if that were supported out of the box.

@cmaycumber Currently, the outside interactions are prevented as a transparent backdrop component is added. We'll add the mentioned feature in Menu/Popover components in the future.

When the popover is controlled by the "isOpen" prop, we can't dismiss the popover anymore by cliking outside of it.
To achieve that you have to implement the onClose prop with your setIsOpen state callback.

repo: GeekyAnts/NativeBase


text:
Icon in InputLeftElement of a input field not animating properly

Description
I have a login page where i have some input fields with icons on left. I am doing some animation (used react-native-reanimated) when a user focus on that input field. While doing this animation icons of input field going out of the screen+input field.
CodeSandbox/Snack link
https://snack.expo.dev/@rassemdev/login-page
Steps to reproduce

Load the page
Focus on input field

NativeBase Version
3.2.1
Platform

[X] Android
[ ] CRA
[ ] Expo
[X] iOS
[ ] Next

Other Platform
No response
Additional Information
Those are my dependencies:
"dependencies": {
"@react-native-community/datetimepicker": "^3.5.2",
"@react-native-community/masked-view": "^0.1.11",
"@react-native-community/netinfo": "^6.0.0",
"@react-navigation/material-top-tabs": "^6.0.6",
"@react-navigation/native": "^5.9.4",
"@react-navigation/stack": "^5.14.5",
"moment": "^2.29.1",
"native-base": "^3.2.1",
"react": "17.0.1",
"react-native": "0.64.2",
"react-native-app-auth": "^6.4.0",
"react-native-gesture-handler": "^1.10.3",
"react-native-pager-view": "^5.4.9",
"react-native-reanimated": "^2.8.0",
"react-native-responsive-screen": "^1.4.2",
"react-native-safe-area-context": "^3.2.0",
"react-native-screens": "^3.4.0",
"react-native-sqlite-storage": "^5.0.0",
"react-native-svg": "^12.1.1",
"react-native-tab-view": "^3.1.1",
"react-native-vector-icons": "^8.1.0",
"recyclerlistview": "^3.0.5",
"styled-components": "^5.3.0",
"styled-system": "^5.1.5"
},

Hi @rassemdev, Thanks for reporting the issue. We will look into it.


text:
Additional methods for GsPostgresConnection

Add the following methods to GsPostgresConnection:
#commitTransaction
#rollbackTransaction

Implement as specified

repo: GemTalk/GemConnect-for-Postgres


text:
"'end" button does not work reliably

If you have some text, e.g. type in a workspace
aaa
bbb
ccc

and your cursor is at the beginning of the bbb row, and you press end, your cursor moves to the end of the ccc row.  Lo these many years and many applications and the end of the bbb row has always up to now been where I would have got to.
It seem to work correctly in a pharo playground...

This helped complete the fix for earlier bug regarding copying source outside pharo and getting cr's in the text on linux.
When you type in a pharo workspace, the Enter key will insert a cr but we use lf's to avoid the above-mentioned problem. Now end/home buttons check for the first cr or lf to cover both types of text.
Not sure I'm super-thrilled with the mixing of end of line characters but it seems to work.

authors: LisaAlmarode ericwinger

repo: GemTalk/JadeiteForPharo


text:
Add addon installation support

Allows user to select addons to install along with the base mod.

Thank you very much for this Pull Request! I am honestly surprised to have any contributors for this little project.
Which mods do you run in the game? I've not tried any yet. Which would you recommend? The forums are currently down so I cannot currently download any.
As for the change in this pull request I'm interested to know your thoughts here and have a few questions to consider:
UI Layout
For the users workflow I had currently assumed that the most important thing they would be doing is selecting the patch location. This is why the patch selection panel is placed at the top of the UI. For this change you have moved the panel below the Minecraft home selection. Is that to bring the association between the patch and the Addons closer together?
Addon Order
Another question I had was around the ordering of addons when building the patched Jar. Does the order matter for addons? Do they make unique changes or are they infact overlapping and have dependencies between addons?
Patching and Upgrading
Another question I was considering was around users who patch with addons and then at a later point in time update but forget or do not apply the same addons. Would this result in problems with that players save games? Whilst I am not suggesting we should aim to fix all eventualities however it might be good to warn the user that they are patching an install that previously had addons installed. It might be good to track the addons patched in the version file.
For these questions, if there are any code changes coming out of them, would you like to make those changes or would you like me to take on those? This is the first open source project pull request I've worked with so I'm not entirely sure on the best approach or indeed what you would prefer?

"Which mods do you run in the game?"

The main mods I use are the Skin Fix mod (since the server older versions of Minecraft get their skins from is outdated) and the CraftGuide mod (to see recipes in-game).

UI Layout

Basically what you said, to bring the association between patches and addons together. Of course, if you believe another layout is better I'm all for it, it was just personal preference.

Addon Order

I'm actually not 100% sure if order matters. I suspect it might not, especially with the little amount of addons present for BTW, I just added it in case. I'll admit the way I implemented the ordering isn't very clean though.

Patching and Upgrading

For client side addons this shouldn't be an issue, and I know for some addons that add blocks, the blocks will simply disappear from the world when not installed, though I can't say for certain that for all addons there won't be a problem. Nonetheless I do agree that it's a good idea to warn the user.

"For these questions [...]"

I'm not sure if it comes across from the code, but I am not very experienced in Java at all, and this was the first time I've ever had to work with Observers, so figuring that out took a while :P. This is basically also my first pull request I've done, so I'm not sure what the best approach is either, however I am more than willing to apply any changes you want to the code.

Now the BTW forums are back online I've taken time to look through a number of mods and try out your changes. They work very well so thank again for putting this together!
Also I've been thinking about how best to collaborate on this change. I was going to propose the following changes with rationale for them:

Switch the position of the "Minecraft Home" section to below the "Better Than Wolves Addons": This achieves the objective of placting the most important parts (Patch & Addons) at the top of the UI.

Remove the Move Up/Move Down buttons: From what I have learned so far there is no dependencies between the mods. Simplifying the UI to just the things the user needs will make it more approachable for new users.

Remove the folder selection functionality once a zip is selected. At the moment I cannot find any mod that needs this feature. Simplification will help the user here again. Less decisions is better for users who might be new to the game.

The goal of this installer is to make it as easy to install the mod as possible so that there is a chance that new players can join this great mod.
If these changes sound ok to you then I would be happy to work with you on them.

Thanks for getting back, and I'm happy to make most of these changes. However both the Deco Addon and the Jorge's Miscellaneous Addon have separate client and server files, which is why I had to add the folder selector for the zip. Unless there's a way to determine if the given directory is for the client, I don't know how else to solve this problem.
Apart from that, I'll get started on the rest of the changes as soon as I'm free.

Thanks for getting back, and I'm happy to make most of these changes. However both the Deco Addon and the Jorge's Miscellaneous Addon have separate client and server files, which is why I had to add the folder selector for the zip. Unless there's a way to determine if the given directory is for the client, I don't know how else to solve this problem.
Apart from that, I'll get started on the rest of the changes as soon as I'm free.

So I've gone ahead and added a bit of code that minimises the need for the user to use the zip folder selection. It basically searches the zip for any .class files in the root folder, and if it finds any it assumes the folder to be extracted is the root folder. I also check if there are any folders the Deco Addon and Jorge's Miscellaneous Addon use for their client files, and if found, extracts from these files.
I don't imagine there will be many other edge cases given the lack of addons being developed, so this will probably be enough for now.

So I've gone ahead and added a bit of code that minimises the need for the user to use the zip folder selection. It basically searches the zip for any .class files in the root folder, and if it finds any it assumes the folder to be extracted is the root folder. I also check if there are any folders the Deco Addon and Jorge's Miscellaneous Addon use for their client files, and if found, extracts from these files.
I don't imagine there will be many other edge cases given the lack of addons being developed, so this will probably be enough for now.

Amazing work and good decisions there. I'll give the changes a test and then merge this into the code base.

Amazing work and good decisions there. I'll give the changes a test and then merge this into the code base.

Ok, all looks good. Thank you again for this. I'll merge this in now and work on adding some credits so your name can appear there.

Ok, all looks good. Thank you again for this. I'll merge this in now and work on adding some credits so your name can appear there.

authors: GencoreOperative Joel-Paul

repo: GencoreOperative/btw-installer


text:
Cannot delete v2 files

SDK version: 3.0.4
Platform: Web (JS)
Whenever you try to delete a file using the deleteFile method, it throws the following error:
Cannot read properties of undefined (reading 'owner-account-pubkey')

That is when attempting to delete a file stored on a v2 storage account using the following method call:
deleteFile(account, url, 'v2')

Now, when trying to delete a file stored on a v1 storage account using the deleteFile method and passing the v2 version as the parameter as:
deleteFile(account, url, 'v2')

It throws the following error:
Invalid account discriminator

However, when trying setting the version to v1 as:
deleteFile(account, url, 'v1')

To further attempt to delete a file stored on a v1 storage account, the signature request pops up, and when approving it, the file gets successfully deleted, which makes sense as its a file stored on a v1 storage account.
What it's weird is that it worked with a signature request (not a transaction) and it's a v1 file, and v2 files are failing with either the v1 and v2 parameters.

This should be resolved!

authors: bntzio tracy-codes

repo: GenesysGo/shadow-drive


text:
🛑 WWWW xyz is down

In 8c9b3e2, WWWW xyz (https://www.crushing.xyz) was down:

HTTP code: 502
Response time: 2849 ms

Resolved: WWWW xyz is back up in d88bcf6 after 8 minutes.

repo: GentlemanHu/own-status


text:
🛑 WeChat Blog is down

In 9fa033d, WeChat Blog (https://ai.crushing.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: WeChat Blog is back up in 118cb2e after 5 minutes.


text:
🛑 AI flowise is down

In 83fee6a, AI flowise (https://aiflow.crushing.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: AI flowise is back up in 77cf8d4 after 24 days, 23 hours, 25 minutes.


text:
Device screen not opening

INFO: scrcpy 1.14 https://github.com/Genymobile/scrcpy
3604 KB/s (33142 bytes in 0.008s)
[server] INFO: Device: Xiaomi POCO F1 (Android 10)
That's it nothing showed up after this

Not even "device disconnected" ?
Is there any error in the output of adb logcat?
Does it work with v1.13 (the previous version)?

How to check if there is error in adb logcat

After typing adb logcat it keeps on typing something I couldn't understand but no errors

log.txt
This is the log file please help me

repo: Genymobile/scrcpy


text:
How can it keep screen on while connecting (avoid screen off),and back to normal while disconnected.

[Y ] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
NO
Describe the solution you'd like
A clear and concise description of what you want to happen.
Can it be done automaticly?
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
NO
Additional context
Add any other context or screenshots about the feature request here.
NO

Could you check with the fix from #1670 on dev branch?
You could try with scrcpy-server posted here (for another issue, but it includes this fix): https://github.com/Genymobile/scrcpy/issues/1468#issuecomment-691715573
Thank you.


text:
Scrcpy is not supporting touch event in landscape orientation

Scrcpy is not supporting touch event through mouse in Landscape in Android Tablet
Environment
-Windows 10

v1.1.9
Windows release

v21H1(OS Build 19043.1200)
Android version: [7]

Describe the bug
Scrcpy is not supporting touch event in landscape orientation
terminal message
[server] WARN: Ignore touch event, it was generated for a different device size

I want to use it in landscape. Scrcpy only supports touch event in portrait and it is forcing me to use in portrait, which I don't like...

If you encounter this message, you probably have two lines on starting, similar to this:
INFO: Initial texture: 1080x2336
INFO: New texture: 888x1920

The reason is that the device sends a video stream at a size different from the requested one.
Limit the video size to what it automatically sends, and everything will work again.
See https://github.com/Genymobile/scrcpy/issues/2031#issuecomment-758567488 and a973757fd141fc07ef81314d350bd4f4de64f59c for explanations.

@rom1v
How to do that?

For example scrcpy -m 1920 if the max supported dimension is 1920.
If you don't know how command line works: https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#command-line-on-windows

If I set it to 592, length automatically changes from 1024 to 336...
It's not working out...

What did you do exactly?

Initial texture : 1024x600
New texture : 1024x593
I gave the command scrcpy -m 593

-m means max size, so you don't want the biggest dimension to be 592 (it's probably 592, not 593, isn't it?)
The problem is that your device decides by itself to use 592 instead of 600 (because it's a multiple of 16).
Try scrcpy -m1010, that might force the other dimension to be 592 (but then that might cause other problems) :/

I tried this @rom1v .It works now, Thanks a lot..


text:
Allow audio, image, and video objects to be uploaded to the media gallery

Overview
Update GalleryForm.js to allow audio (mp3, mp4), image (jpeg, gif, png, svg) and video (mpeg, mp4) objects to be uploaded to the gallery.
Description of Changes

Allow  @type  of updateMediaGalleryItem to be 3DModel (existing), AudioObject, ImageObject, or VideoObject
Let format be set according to file types and encoding mappings found in this sheet
Accept .glb, .usdz, .mp3, .mp4, .jpeg, .gif, .png, .svg, & .mpeg file types in the file selection modal
Update the File Format dropdown to include the new file type options following the existing format: e.g., ".gif (image/gif)"

Closed with https://github.com/Geo-Web-Project/cadastre/pull/84

repo: Geo-Web-Project/cadastre


text:
Put package requirements in setup.py

Unfortunaetly the setup.py does not state the requirements.
So once you install the package in a virtual environment you have to manually figure them out.
Best to figure them out is to create a new virtualenvironment and do some tests.
If i have some time I might do a PR.
In general I am a bit confused as the documentation link https://georaster.readthedocs.io/en/latest/api.html "edit on github" is not pointing to this repro but to https://github.com/atedstone/georaster/blob/master/docs/api.rst which repro is the most current one?

@atedstone are you also editing the repro?

Hi,
Thanks for the feedback. It would indeed be interesting to add this. @atedstone, you might know better but let me know if you want me to add this.
Alternatively, the requirements are stated in the conda requirement file here: https://github.com/GeoUtils/georaster/blob/master/docs/rtd_environment.yml
Regarding the repositories issue, both Andrew and I have our fork of the package, but the one in GeoUtils is meant to be the stable version. So you're right, the documentation should point to GeoUtils, not Andrew's own.
@atedstone, I guess you're the only one who can make the change?

@CarliJoy thanks for spotting this, I'll update the documentation link asap. The version on GeoUtils is indeed the stable version and is the only place where we periodically releases.
@adehecq would you mind looking at the requirements for virtualenv installation?
We've prioritised packaging for conda-forge so far, so the package requirements are sorted here - @adehecq take a look at https://github.com/conda-forge/georaster-feedstock/blob/master/recipe/meta.yaml which should help to improve our setup.py.

Unfortunaetly the setup.py does not state the requirements.
So once you install the package in a virtual environment you have to manually figure them out.

There is now a requirements.txt file.
However, due to the complexities of GDAL installations (which I don't believe that a requirements.txt file can solve), we still recommend that georaster is installed from conda-forge.

Best to figure them out is to create a new virtualenvironment and do some tests.
If i have some time I might do a PR.
In general I am a bit confused as the documentation link https://georaster.readthedocs.io/en/latest/api.html "edit on github" is not pointing to this repro but to https://github.com/atedstone/georaster/blob/master/docs/api.rst which repro is the most current one?

The link is now fixed.

authors: CarliJoy adehecq atedstone

repo: GeoUtils/georaster


text:
fix(bindings): renamed some libs to avoid same naming in different pr…

…ojects.

:tada: This PR is included in version 7.0.5-rc.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: BotellaA MelchiorSchuh

repo: Geode-solutions/OpenGeode-IO


text:
Latest version for Windows?

Any chance to get the Windows build updated to latest version? I see it's a couple version behind.

Yes, I'll make a build for Windows for the next version

Any update on this?
The command line version of 1.1 for Windows freezes on meshes that work in 1.3 for OSX.
I am also having difficulty building 1.3 from source as the documentation for how to link missing dependancies is not clear.

Has anyone managed to build the 1.3 for windows? Having the same issue.

@thundaBYR I haven't been able to find the time yet, in the meantime, might be worth looking here: https://github.com/GeometryCollective/boundary-first-flattening/issues/55

@thundaBYR I haven't been able to find the time yet, in the meantime, might be worth looking here: #55

Latest version 1.5 can be found here


text:
Not recognizing InsufficientMaterial EndGame

Describe the bug
A board with insufficient material is not being recognized as such, and its IsEndGame is false, etc.
To Reproduce
var board = ChessBoard.LoadFromFen("2b5/4kB2/8/8/8/8/5K2/8 b - - 0 16");
if (!board.IsIendGame)
    throw new ();

Expected behaviour
Boards such as the one used in the reproduction example should be recognized as having insufficient material to proceed, and the game drawn, with board.EndGame.EndgameType == EndgameType.InsufficientMaterial, and with its board.ToPgn() string being given the ½-½, etc., etc.
Screenshots
N/A
Additional context
I know that you probably have a much better way to fix this in the library, but in case it is in any way useful, here is the code I added as a workaround in my application:
private static bool IsDrawnForInsufficientMaterial(this ChessBoard board)
{
    //	Insufficient Material isn't properly detected.  Honestly, I checked. Do it ourselves.
    //	If there are any pawns, rooks, or queens on the board, there is sufficient material.
    var fen = board.ToFen().Split()[0];
    if ("PQR".Any(fen.ToUpper().Contains))
	return false;
    // Otherwise, it's just bishops, knights, and kings.  Well, yeah, kings -- ignore them....
    var remainingPieces = new string(fen.Where(static c => c is > '8' and not ('K' or 'k')).ToArray());
    //	Kings only or King against a single bishop or knight is drawn.
    //	Otherwise, except for a bishop apiece, it's sufficient.
    //	In B vs b, it's a draw if both bishops are on light squares or both on dark squares.
    return remainingPieces.Length < 2
        || remainingPieces is "Bb" or "bB"
        && Regex.Replace(fen, "[2468]", string.Empty).Select(static (c, i) => c is 'B' or 'b' ? i : 0).Sum() % 2 is 0;
}

Well, i know about this one...
Also 50 moves rule and repetition is not supported yet
Didn't expect to someone really using this lib haha :)
I cannot promise to make it shortly but I'll do my best!

Thanks so much!  And I guess this is what you get for putting out quality software!  :-)
BTW, I'm sorry to report that I have another very hairy bug to report to you, but it evades an easy way to reproduce it, so I have a nasty workaround in place.  I'll let you know at some point all about it, once it stops evading my capture.  Basically, there is a situation where a valid ChessBoard (created from .LoadFromFen), when told to .Move(oneOfItsOwnMovesReturnedFromBoard.Moves()) returns false and doesn't update the board.  From all evidence, the only time I am seeing this is when the move is an en passant capture which (when coming out of board.Moves()) is not fully created -- with no .CapturedPiece, no .IsEnPassantCapture, etc.)

I'll take a look at this one

Hello -- I just realized that my workaround code for InsufficientMaterial is...well, insufficient.  Unless I am mistaken, one player could have his King and only any number of bishops (say, five or six of them, even!) against an opposing king (who could himself have an army of bishops), and the game would be drawn for insufficient material if ALL of the bishops were on light squares or all on dark-squares.  So we will both need better code than I provided in this ticket!  :-)

Thanks so much! And I guess this is what you get for putting out quality software! :-)
BTW, I'm sorry to report that I have another very hairy bug to report to you, but it evades an easy way to reproduce it, so I have a nasty workaround in place. I'll let you know at some point all about it, once it stops evading my capture. Basically, there is a situation where a valid ChessBoard (created from .LoadFromFen), when told to .Move(oneOfItsOwnMovesReturnedFromBoard.Moves()) returns false and doesn't update the board. From all evidence, the only time I am seeing this is when the move is an en passant capture which (when coming out of board.Moves()) is not fully created -- with no .CapturedPiece, no .IsEnPassantCapture, etc.)

I have tested it and found a bug with SAN-notation, its already fixed. I suppose that CapturedPiece property is being set correctly, perhaps you are using an old version of library because IsEnPassantCapture property does not exist in the last one...

Yeah, I realized after submitting the report that .IsEnPassantCapture is an
extension method that I wrote.  :-)
On Fri, Jan 6, 2023 at 12:47 PM Sviatoslav @.***> wrote:

Thanks so much! And I guess this is what you get for putting out quality
software! :-)
BTW, I'm sorry to report that I have another very hairy bug to report to
you, but it evades an easy way to reproduce it, so I have a nasty
workaround in place. I'll let you know at some point all about it, once it
stops evading my capture. Basically, there is a situation where a valid
ChessBoard (created from .LoadFromFen), when told to
.Move(oneOfItsOwnMovesReturnedFromBoard.Moves()) returns false and doesn't
update the board. From all evidence, the only time I am seeing this is when
the move is an en passant capture which (when coming out of board.Moves())
is not fully created -- with no .CapturedPiece, no .IsEnPassantCapture,
etc.)
I have tested
https://github.com/Geras1mleo/Chess/blob/master/ChessUnitTests/UnitTests.cs#L475
it and found a bug with SAN-notation, its already fixed. I suppose that
CapturedPiece property is being set correctly, perhaps you are using an old
version of library because IsEnPassantCapture property does not exist in
the last one...
—
Reply to this email directly, view it on GitHub
https://github.com/Geras1mleo/Chess/issues/9#issuecomment-1374052829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASF4VN3THE23TQXICXI3JE3WRBZEFANCNFSM6AAAAAATHGCE4U
.
You are receiving this because you authored the thread.Message ID:
@.***>

I looked at your unit test code, and dang, I promise I run the same thing
and I do not get a pass.  I'm on release 1.0.2 -- is that not the latest?
On Fri, Jan 6, 2023 at 1:45 PM Manus Hand @.***> wrote:

Yeah, I realized after submitting the report that .IsEnPassantCapture is
an extension method that I wrote.  :-)
On Fri, Jan 6, 2023 at 12:47 PM Sviatoslav @.***>
wrote:

Thanks so much! And I guess this is what you get for putting out quality
software! :-)
BTW, I'm sorry to report that I have another very hairy bug to report to
you, but it evades an easy way to reproduce it, so I have a nasty
workaround in place. I'll let you know at some point all about it, once it
stops evading my capture. Basically, there is a situation where a valid
ChessBoard (created from .LoadFromFen), when told to
.Move(oneOfItsOwnMovesReturnedFromBoard.Moves()) returns false and doesn't
update the board. From all evidence, the only time I am seeing this is when
the move is an en passant capture which (when coming out of board.Moves())
is not fully created -- with no .CapturedPiece, no .IsEnPassantCapture,
etc.)
I have tested
https://github.com/Geras1mleo/Chess/blob/master/ChessUnitTests/UnitTests.cs#L475
it and found a bug with SAN-notation, its already fixed. I suppose that
CapturedPiece property is being set correctly, perhaps you are using an old
version of library because IsEnPassantCapture property does not exist in
the last one...
—
Reply to this email directly, view it on GitHub
https://github.com/Geras1mleo/Chess/issues/9#issuecomment-1374052829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASF4VN3THE23TQXICXI3JE3WRBZEFANCNFSM6AAAAAATHGCE4U
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi again --
I updated my own workaround code that handles InsufficientMaterial per the realization I made a comment about earlier.  In case it is of any help to you, this is what I am now using:
//	Insufficient Material isn't properly detected.  Honestly, I checked. Do it ourselves.
//	If there are any pawns, rooks, or queens on the board, there is sufficient material.
var fen = board.ToFen().Split()[0];
if ("PQR".Any(fen.ToUpper().Contains))
	return false;
// Otherwise, it's just bishops, knights, and kings.  Well, yeah, kings -- ignore them....
var remaining = new string(fen.Where(static c => c is > '8' and not ('K' or 'k')).ToArray());
//	Kings only or King against a single bishop or knight is drawn.
//	Otherwise, except for only bishops, it's sufficient.
//	If bishops only, it's a draw if they're all on the same color.
var count = remaining.Length;
return count < 2
	|| remaining.All(static c => c is 'B' or 'b')
	&& new[] { 0, count }.Contains(Replace(fen, "[2468]", string.Empty).Select(static (c, i) => c is 'B' or 'b' ? i + 1 : 0)
																       .Where(static i => i is not 0)
																       .Count(static i => i % 2 is 0));

Hi again --
I updated my own workaround code that handles InsufficientMaterial per the realization I made a comment about earlier. In case it is of any help to you, this is what I am now using:
//	Insufficient Material isn't properly detected.  Honestly, I checked. Do it ourselves.
//	If there are any pawns, rooks, or queens on the board, there is sufficient material.
var fen = board.ToFen().Split()[0];
if ("PQR".Any(fen.ToUpper().Contains))
	return false;
// Otherwise, it's just bishops, knights, and kings.  Well, yeah, kings -- ignore them....
var remaining = new string(fen.Where(static c => c is > '8' and not ('K' or 'k')).ToArray());
//	Kings only or King against a single bishop or knight is drawn.
//	Otherwise, except for only bishops, it's sufficient.
//	If bishops only, it's a draw if they're all on the same color.
var count = remaining.Length;
return count < 2
	|| remaining.All(static c => c is 'B' or 'b')
	&& new[] { 0, count }.Contains(Replace(fen, "[2468]", string.Empty).Select(static (c, i) => c is 'B' or 'b' ? i + 1 : 0)
									   .Where(static i => i is not 0)
									   .Count(static i => i % 2 is 0));

Yes, thank you!

I can confirm that 1.0.3 fixed my en passant issue (which I reported as issue #10), and also was indeed the cause of the "hairy bug" that I mentioned earlier in this thread, and that you commented about, mentioning that your fix to en passant San may have addressed it.  It sure did!  Thank you once again!!!  I am busy happily removing workarounds from my application now!

If you are looking for a good test position for any change you make to InsufficientMaterial calculation, you can get one by using my website that uses your library -- http://104.168.200.10:43860 --, changing the "Vocabulary" to PGN, and then giving it this problem to solve:
.. xx .x
xx xx xx
xx xx xx
&B    c&
xx xx xx
xx xx xx
/

It will take it a while (about 90 seconds) to solve it, but then it will give you the sequence of moves, and resulting FEN, that this string produces, which ends in an InsufficientMaterial position that your library does not detect (but my workaround code then does).

Thank you for working this!  I look forward to the NuGet update that includes the new code!!

Hello -- Do you have a date on which you plan to deploy this fix in a new NuGet release?  Or should I download the branch source and use it that way?  (I would rather, of course, use it as a NuGet package.)
Thanks,
Manus

Hello -- Do you have a date on which you plan to deploy this fix in a new NuGet release?  Or should I download the branch source and use it that way?  (I would rather, of course, use it as a NuGet package.)
Thanks,
Manus

Today I've added threefold repetition rule and I will add 50 moves rule tomorrow, so you can expect new release tomorrow evening.
Cheers,
Sviatoslav

Wow, that is great!  Thank you so much!

Wow, that is great! Thank you so much!

New release with endgame rules, use AutoEndgameRules prop to configure!
Cheers,
Sviatoslav

Hello --
I am afraid that even with NuGet version 1.0.4, InsufficientMaterial is not being properly recognized.  Here is a code-snippet show this:
var board = ChessBoard.LoadFromFen("2b5/4kB2/8/8/8/8/5r2/4K3 w - - 0 1");
board.Move("Kxf2");
if (board.EndGame is not { EndgameType: EndgameType.InsufficientMaterial })
	throw new ("InsufficientMaterial not recognized");


text:
cd into the ui folder before doing npm install for ui

You need to cd ui folder,before installing packages.

Remove lock file

I'm having a deja-vu.
didn't you already made that PR this morning?

repo: GerevAI/gerev


text:
why the Basicconv have not the ReLU function ?

thanks for your excellent, but I find your released code  in  fsnet.py , the BasicConv2d do not have the ReLU ?

Thank you for your attention to our work. I did not use it in the FSNet. So what's your problem for this point?
Best regards,
Ge-Peng.

repo: GewelsJI/FSNet