Configure Google attribute mapping without marking as required

Amplify CLI Version
12.3.0
Question
I went through the following guide: https://docs.amplify.aws/javascript/build-a-backend/auth/override-cognito/#customize-amplify-generated-cognito-auth-resources-with-social-providers, but it didn't work. And in the following issue https://github.com/aws-amplify/amplify-cli/issues/11646, I saw that marking attributes as required should allow the mapping, but is there a way without marking as required?

I am looking into doing that through the post-push command hook, but I don't have access to the user pool ID.
Just mentioning it in case it helps with ideas.

I also found the following issue https://github.com/aws-amplify/amplify-cli/issues/12910.

Hey @redjonzaci, thank you for reaching out. you should be able to create a custom attribute as optional using the auth override. we have an example on https://docs.amplify.aws/javascript/build-a-backend/auth/override-cognito/#customize-amplify-generated-cognito-auth-resources

Hey @ykethan, I need to configure the Google attribute mapping, not a custom attribute on the Cognito user pool.

Hey @redjonzaci, thank you for the clarification, this looks similar to https://github.com/aws-amplify/amplify-cli/issues/12910#issuecomment-1714678419.
Update the attributes can be done when adding provider or removing a provider, it will cause the Lambda callout to fire and update the social providers with the new attribute mapping.
i was able to update the attribute, by running amplify update auth -> select the provider, then add the attribute in the cli-inputs.json in the hostedUIProviderMeta AttributeMapping then run push which should update the attributes.
do note that running an update auth may override the changes.
To understand making an attribute optional are you referring to the following?

I read in https://github.com/aws-amplify/amplify-cli/issues/11646 that if I mark attributes in Cognito as required, they will be added in the identity provider attribute mapping.
But I was looking for a way to keep the Cognito attributes optional and still add them in the identity provider attribute mapping programmatically.
What I am not getting is why would an additional amplify update auth override the attribute mapping I provided previously in the cli-inputs.json?

Also, the approach you're suggesting should work when creating new PRs?
I am testing this now and will reply again later.

Hey @redjonzaci, when we create a auth resource with amplify add auth, you should be able to remove the required attributes. by default the cognito expects an attribute with sub and should be created as optional from my testing. you could then add the additional attributes in the cli-inputs.json and push
Do you want to use the default authentication and security configuration? Default configuration with Social Provider (Federation)
 Warning: you will not be able to edit these selections.
 How do you want users to be able to sign in? Email
 Do you want to configure advanced settings? Yes, I want to make some additional changes.
 Warning: you will not be able to edit these selections.
 What attributes are required for signing up? <- removed here
 Do you want to enable any of the following capabilities?
 What domain name prefix do you want to use? attribs6b68ffc7-6b68ffc7
 Enter your redirect signin URI: http://localhost:3000/
? Do you want to add another redirect signin URI No
 Enter your redirect signout URI: http://localhost:3000/
? Do you want to add another redirect signout URI No
 Select the social providers you want to configure for your user pool: Google

on cognito

Let's say I have added given_name mapping to cli-inputs.json, will it be mapped when a new env is created?
This was my latest question.

It worked. Thanks for the support and happy holidays @ykethan!

@redjonzaci Glad to hear the suggestion worked. The amplify add env should not remove the changes. But noticed this behavior when we update the auth to add a new oauth provider the CLI overrides the cli-inputs.json file.
Closing the issue, do reach out to us if you require any assistance.


feat: adds border highlighting to cloneOrDeployProgress component

https://github.com/payloadcms/website/assets/35232443/375bb54a-159d-4186-85ea-f1cd74b28f1f
https://github.com/payloadcms/website/assets/35232443/584ffc05-0567-4545-ac89-38b50d207f52

outdated


Add whatsapp

Please add whatsapp.
Regards

I accept a good PR to add Whatsapp support

I'm also in support for this PR to add Whatsapp

I think i've achieved to apply whatsapp share link, all you need to do is to change in the followings files :-
Go to : config/laravel-share.php
Add this to services :
        'whatsapp' => [
            'uri' => 'https://wa.me/?text=',
            'extra' => ['mini' => 'true']
        ],

Go to : resources/lang/vendor/laravel-share/en/laravel-share-fa4.php
Add this to return :
    'whatsapp' => '

',
Go to : vendor\jorenvanhocht\laravel-share\src\Share.php
Add new function :
    /**
     * Whatsapp share link
     *
     * @return $this
     */
    public function whatsapp()
    {
        $url = config('laravel-share.services.whatsapp.uri') . $this->url;

        $this->buildLink('whatsapp', $url);

        return $this;
    }

Go to : vendor\jorenvanhocht\laravel-share\config\laravel-share.php
Add this to services :
        'whatsapp' => [
            'uri' => 'https://wa.me/?text=',
            'extra' => ['mini' => 'true']
        ],

Go to : vendor/jorenvanhocht/laravel-share/resources/lang/en/laravel-share-fa4.php
Add this to return :
     'whatsapp' => '

',
Finally Use it in blade :-
{!!
     Share::page('http://jorenvanhocht.be', null, [], '', '')->facebook()->twitter()->googlePlus()->whatsapp()
!!}

Hi @jorenvh! I needed this whatsapp feature and took this code above by @vipertecpro and implemented it in a new fork. You can see it in pqrs/laravel-share.
I added a test for WhatsApp and it passes. I think everything is ok but I'm fairly new to all this about forks and pull requests and don't know exactly what to do to merge it (if you want, of course).
Best!

hi @pqrs can you create a PR for that fork please?

Sure! Let me know if it’s ok.

PR #30 by @pqrs was just merged into master. A new release will be tagged later today.
Thanks again @pqrs for contributing to this package

I feel so blessed, that my code is going to be in someone else's new update (or even a small fix). That's only because the dev-master who wrote such a flexible code, if possible i would like to add all the social media around the web in it because well that's what this repo is for. Thank you so much again. Keep on updating and patching.

Thanks to you, @vipertecpro. As you see my PR is your code as you submitted so all the credit should be yours. I’m too for adding more social shares, I see someone proposed Pinterest a while ago. I’ll try when  I find some time.
Cheers!


support for nostr protocol

Would it be sensible for Atomic Data to support the nostr protocol for decentral data exchange?
I imagine that the atomic data server could act as nostr relay, accepting only data formats that it could parse semantically, and perhaps also act as a nostr client as well...

Looks interesting! I see a lot of similarity: focus on signing things, skipping the blockchaing thing but instead rely on more regular servers. Events are similar to Commits, as standardized signed entities. Major difference is that Atomic Commits are far more restrictive and structured, instead of the plain UTF-8 content used by nostr. Both have their merits, of course. Nostr is simpler, but not as suitable for structured data.
Anyway, let's consider interoperability.
The nostr protocol is pretty simple to implement. Some implementations for rust already exist. Adding this functionality to atomic-server should not be that difficult. I think most work will be reading the filter object and turning that into queries for Atomic (probably a couple of TPF queries would suffice), getting the hash, getting all event types working.
But I don't really see how both could be combined in a sensible way.
Some random thoughts:
I believe posts needs to be at least a bit standardized (more than UTF-8 strings) to provide a good UX + UI. It should support images, lists, links... Markdown / commonmark, or perhaps even HTML is a good place to start. Nostr does not have types like that.

Atomic Commits are far more restrictive and structured, instead of the plain UTF-8 content used by nostr.

I guess you are not really talking about nostr generally above, but more specifically event kind 1.
What I imagine is that Atomic would accept and serve only (made up, still to be defined as a NIP) "event kind 63" where content is JSON-LD (and likely a few other kinds too, e.g. probably also the currently defined kinds 0 and 2).

Obviously Atomic would also support that other future "event kind 482" where content is JSON-AD, I just imagine that a NIP defining a more commonly accepted format might more likely be accepted in the nostr community at first ;-)

I guess you are not really talking about nostr generally above, but more specifically event kind 1.

Yeah, you're right. Adding a new event kind would fix this issue. So... Let's say Atomic-Server only accepts event kind 482. We would have a new way of subscribing to and getting Commits. I'll have to spend some more time thinking about this, or we could have a call sometime soon to discuss this further.

I just imagine that a NIP defining a more commonly accepted format might more likely be accepted in the nostr community at first ;-)

Since JSON-AD is actually also valid JSON-LD, too, it should still count as JSON-LD! It's just a bit more constrained.

I would love a(nother) video meet - just tell me when it suits you, my calendar is wide open.
Sure, I know JSON-AD is a subset og JSON-LD - meaning that event kind 482 would be easier for you to handle, but event kind 63 would be more attractive for the nostr community to adopt


Hand moving spontaneously

During a pause between two experiments (using the WBI toolbox), two fingers of the left hand of iCubDarmstadt01 moved spontaneously (from completely extended to completely flexed).
I am not sure if this ticket is depending from the WBI-toolbox or not (might be lower-level).
This is quite awkward (and scary)...
What should I try to log next time to diagnose the possible problem?

The WBI and, as a consequence, the toolbox have nothing to do with the hands since  the version you are using is not configured to allow the control of the robot's hands, unless you specifically changed the source code. Sometimes it happens that the fingers can move a little without touching them, thus scaring a little bit the user :)

Hi Daniele,
Yes, I know that the WBI does not have direct access to control the hand.
But the one I experienced was not a little movement (i.e., noise). It was a full flexion of two fingers.
Since there are so many layers (Yarp, firmware,hardware), how can I find out what was the source of this spontaneous movement?

Some 'general' questions before starting any debug activity:

Does the issue happen frequently or it happened just once?
Is WBI somehow involved (maybe just executed once and then stopped) in this kind of issue?
Does it happen even just starting the robot from a clean configuration, without executing any user module?
What is the status of the joint shown in the robotMotorGui? is the encoder value correct? is the joint still in position mode or it is an different mode (idle / hardware fault / under calibration etc).
does the robotInterface log show any strange error/warning?
does the log of other modules show any strange error/warning? With the yarplogger tool you can save the log of a entire session (meaning that you can retrieve the log of all the the processes you launched from the startup).
is your system updated? which version of yarp/icub/fw/WBI toolbox are you using?

Ok, now I am pretty sure that it is a consequence of the WBI.

Does the issue happen frequently or it happened just once?

The issue seems to happen regularly, and it is valid for both hands

Is WBI somehow involved (maybe just executed once and then stopped) in this kind of issue?

The WBI is definitely involved

Does it happen even just starting the robot from a clean configuration, without executing any user module?

it happens after using WBI, even if no additional module is required.

What is the status of the joint shown in the robotMotorGui? is the encoder value correct? is the joint still in position mode or it is an different mode (idle / hardware fault / under calibration etc).

The motore are still in position mode and properly working (the encoder value is correct)

does the robotInterface log show any strange error/warning?
does the log of other modules show any strange error/warning? With the yarplogger tool you can save the log of a entire session (meaning that you can retrieve the log of all the the processes you launched from the startup).

I still need to run it

is your system updated? which version of yarp/icub/fw/WBI toolbox are you using?

I am using the previous version of the WBI. How can i check the versions for the other components?

I am also attaching a video https://www.dropbox.com/s/sj0x0j3r96lxa64/VID_20150122_165355.mp4?dl=0

Hi Roberto,
nice video. Just to rule out some other implications, have you tried to write a simple C++ module that does the same operations of the WBI? This might be of some help.

That is definitely interesting. You where using the old (v0.1) or new (was new_wbi_ID branch, now v0.2) version of WBI-Toolbox? In the latter case, which configuration file you where using?


Model is always Finetuned in 8bit mode, needs to not be hardcoded

https://github.com/tloen/alpaca-lora/blob/main/finetune.py#L114

I agree, but I don't think there is a major downside. The lora matricies are stored with much less quantization, so they can fill in the gaps pretty well.


Fail to CREATE_PIPE

the info of 2 iotdb cluster above ,
when i create pipe,
Msg: 1107: ProcedureId 870: Fail to CREATE_PIPE because Failed to create pipe C1toC2, details: DataNodeId: 6, Message: Failed to handle single pipe meta changes for C1toC2, because Failed to construct PipeConnector, because of java.net.UnknownHostException: iotdb-c2-1.
but i can ping the host: iotdb-c2-1

Suggest using 1.3.2


Change size and color of Prefix

Prefix changes:

Shouldn't have background color
Font size should be 14px
Should have color: #38495B;

Already implemented in flystyles


Regenerating KeyVault .Net SDK from REST API version 7.0-preview

Description

Swagger commit: https://github.com/Azure/azure-rest-api-specs/commit/cffbe0f093fc4628c60b569b6095dd0caa80da66

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please add REST spec PR link to the SDK PR
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@dragav as this needs to be merged in your own branch, could you get this approved and merged by your team members who have rights to merge it.
We will close this PR by EOW.

    /// from the issuer provider.</param>

you added  in the middle of  so it's not well formatted.

Refers to: src/SDKs/KeyVault/dataPlane/Microsoft.Azure.KeyVault/Generated/Models/IssuerParameters.cs:36 in f216934. [](commit_id = f216934db48bb78dded0509db09ff191086dcf77, deletion_comment = False)


Plugins that require a restart of job or power-cycle after install

Some plugins require a restart of a job or power-cycle after install. Ex: plugins that add streaming to the database require a restart of the MQTT streaming job.
Where should plugins tell the Pioreactor that a restart is needed? A flag file?

This can be done with post_install.sh file


Standardize Quit on Linux/Windows

@keybase/react-hackers
@MarcoPolo, this PR will affect Windows too, do you think it does the right thing there?

:mag_right: The commits 3c54cc36fe0a...9c878b1dc01c introduced some visual changes on win32:

Changed: user_search-Normal (before) (after)

Changed: Search_User_Pane-Broken_tracker_not_following (before) (after)

@MarcoPolo changed to close all windows, thanks!

👍

:+1:


BIg sur 11.0.1 Beta  not work  Crash when opened

BIg sur 11.0.1 Beta
not work
Crash when opened

I am also having regular crashes in Brackets once I upgraded to Big Sur. "Brackets Helper" repeatedly crashing.


[autoparallel] where_handler_v2

What does PR do?

simply, this PR add where_handler using refactored APIs.
In this PR, I overload post_process and register_strategy function to recover the logical strategy into physical strategy. The cost model rule is:

using logical shape in compute_cost, memory_cost, and communication_cost computaion.
using physical shape in resharding_cost computation.
Note: This rule may miscalculate some costs due to different kernel implement method, but it works fine for most of cases.

This PR has a conflict to be resolved.


Fix can't play movies on Android

Some changes for playing movies on RMMV.

Fix can't play movies on Android (videoElement.play() is required on touchend)
Make sure that wait until movie ended
on iPhone/iPod, play movie inline
Delete two global variables and a global function

Oh, I also changed index.html and add js/libs/iphone-inline-video.browser.js.
How do I change these files on this repository?

I will prepare index.html and libs directory.

I review this. great :heart:

Can you please close it and make a new PR ?
Project structure is dramatically changed.


Question about Redundancy of Virtual WAN

Its not clear from Virtual WAN documentation about how Redundancy is provided for this service
If I have Virtual WAN Hub created in a Region (US EAST)
What if the US EAST Region has an outage , will all my connections to Virtual WAN go down ?
Or
is this a Global Redundant service
So if USEAST goes down Microsoft automatically fails over the VWAN to diff (paired) region ?
Please clarify !
Thanks
Nish

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 316afee6-13d5-4b73-5224-073b35e69737
Version Independent ID: be1f6b3f-e7c1-0775-8e38-1e5044f96baa
Content: Azure Virtual WAN Overview
Content Source: articles/virtual-wan/virtual-wan-about.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@nishshah1 Thanks for the feedback! We are currently investigating and will update you shortly.

@nishshah1 , Appreciate your patience. If a specific hub goes down then other hubs in a WAN will continue to work.
VPN and ER gateways by default use availability zones if available in that particular region during the creation of these resources. But still if one of availability zone is down, then VPN or ER local hub connectivity may continue to work but there can be impact to cross hub scenarios based on the nature of outage in a region.

@nishshah1 , For Point-to-site, traffic management is built in so the remote user connects to the next available hub.
For ExpressRoute and Site-to-site, you can always create another connection to a different hub.
Within a hub services are all deployed in active-active mode so resiliency within the hub is built in. If the region were AZ, then the deployments would be in AZ.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Investigate eventsource performance for high-throughput scenarios

Determine whether moving from the WriteEvent(int eventId, params object[] args) overload is worthwhile
See: https://msdn.microsoft.com/en-us/library/system.diagnostics.tracing.eventsource.writeeventcore(v=vs.110).aspx

Dupe of https://github.com/Azure/azure-webjobs-sdk-script/issues/1362


Profile Setting Option to receive email on material Privacy and Terms changes

Add an option to the user's profile/settings page where they can opt into being notified by email about material (i.e. not just a typo) change to the privacy policy or terms of use.
This allows human discretion on whether or not to notification is necessary (i.e. if the mailing address changes we don't likely need to spam everyone).
This also does not complicate the registration process or interrupt site use for the majority of users who likely are not concerned about changes to the policies.

@louistatta @rbbeeston, for triggering the email notification, are we OK with a management command that's triggered manually by a dev, or do we want to implement a UI / form for this?

This would be a UI form checkbox on the User settings page
https://www.boost.io/users/me/

Yeah I have that part coded up already. But something has to actually trigger the email going out to users who have opted in.

I'm wondering, seems like we would need to have a separate page, and when it's changed would trigger it. I would suggest discussing with @sdarwin on how to go about that.

@GregKaleka
The actual messages to be sent out:
There could be a new form in the /admin/ panel, where the text content of the email is entered, and then choices of recipients such as :

all registered users
all users who have registered for Privacy and TOU updates
admins

In the short-term however, this ticket can be closed by adding a field in the db to keep track of the setting of who has registered for privacy updates, and that's all.
Implementing the next part, with a form, is "moderate" priority.
High priority are:

launch. Can the boost.io website be published on the boost.org domain tomorrow? It's ready to view by the public?
the next boost release very soon, in Dec. boost 1.87.0.  There are reports planned for the 1.87.0 release, and perhaps issues with release notes, other topics.

discuss with @rbbeeston .    If those 1 and 2 are set, then ok.

Got it - I'll put up a PR shortly, then.


Overflow in rpc:eth_call counter

This substream does a lot of eth calls, but not that many.  Seems there is an overflow in rpc:eth_call counter

https://github.com/Graph-BuildersDAO/Curve-Substream/releases/download/v0.1.6-alpha/curve-sps-v0.1.6-alpha.spkg


move population growth methods to methods vignette

possibly rename the methods vignette so more general
look for any more methodological sections to move over

done


Add warning if a local work dir is found

Description
Workflow managers commonly create a local working directory which can be sizable. When quickly developing a workflow and running locally this work dir can be created, and when run later with AGC the dir can end up being uploaded. This can make it seem like AGC hangs as it is bundling and uploading this data.
Use Case
When switching between local workflow runs and using AGC to submit to Batch.
Proposed Solution
Check if the workflow being uploaded has a common workflow work dir such as .nextflow or .snakemake, and provide a warning message to the user if one is found.
Other information

Sounds like a good feature to add.  We will include it in the backlog.

Looks like this idea has been floated before #414 discussion seems to be include a .agc-ignore file which can have these work dirs.


Is this compatible with Cardano Node >= 8.x.x ?

It is compatible at the networking level, but the still un-released Conway era changes are not yet supported at the ledger level.
Have you experienced any issues connecting to 8.x.x?

I had issues with LocalTxSubmission (Haskell), but pallas doesn't have this protocol yet, so my question can be discarded.


Fix v44 auth upgrade bug

Workaround for https://github.com/cosmos/cosmos-sdk/issues/10591 , https://github.com/provenance-io/provenance/blob/407c89a7d73854515894161e1526f9623a94c368/app/upgrades.go#L86-L122
Just do auth upgrade last.

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #618 (3960798) into main (f68d6ed) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #618      +/-   ##
==========================================
- Coverage   18.44%   18.44%   -0.01%
==========================================
  Files         172      172
  Lines       24233    24235       +2
==========================================
  Hits         4470     4470
- Misses      18998    19000       +2
  Partials      765      765

Impacted Files
Coverage Δ

app/app.go
0.55% <0.00%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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

This look as a dirty hack. I'm thinking about changing the map value type to a structure with priority.

Let's continue the discussion in the SDK issue

Seeing as this is being tested on vega upgrade as well, merging

sure, was only saying that it's feels hacky, but solves the problem. In 0.44 we don't have other solution which is not breaking :/ except the patch for x/auth.
So your one is the best so far. I have created a draft PR for 0.45 to change the structure.

ah 100% agreed, its absolutely a gross workaround lol


Material You

[x] Read CONTRIBUTING.md.

Implemented Material You.
Summary of changes:

Themed the navigation bar.
Themed Home.
Themed SearchBar.
Themed MediaPage.
Fixed a couple layout bugs.
Unhardcoded strings and dimensions.
Added a new ProgressIndicator.
Made the description scrollable.

Tested changes:
LGTM.

Why you do this GH :weary:

Do I merge?

TYSM for the reviews! <3 @boswelja @Iannnr


[2.0.0-preview017, Win10] Error with global npm install of 'prisma2' CLI

On Windows 10, I get an error immediately when I try to install the newest version of the Prisma Framework CLI (2.0.0-preview017).

"TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined"
I haven't tried yet, but I guess it won't be a problem if I try installing using WSL, as it looks like another case of a Windows-specific file access issue.

Thanks a lot for reporting @D2KX! We're looking into it and will probably release a patch tomorrow.

Hi,
Thanks for reporting this but this has been already reported once. So I am going to close this as a duplicate of #985

@pantharshit00 this issue has to do with the cache directory and is, therefore, unrelated to #985 .

@D2KX I can't reproduce it on my Windows testing machine but published a fix, which should at least fix this one (you'll still run into https://github.com/prisma/prisma2/issues/985)
Can you try out<EMAIL_ADDRESS>please?
Thanks 🙏

@D2KX I can't reproduce it on my Windows testing machine but published a fix, which should at least fix this one (you'll still run into #985)
Can you try out<EMAIL_ADDRESS>please?
Thanks 🙏

2.0.0-alpha.353 indeed fixes the problem ❤ Thank you!

Perfect! Thanks for getting back to me so quick. I'll close this one then.


Added Turkey holidays

Not implemented islamic dates, you can convert gregorian to hijri calendar

Thanks! Can you rebase from main, so I can run the tests here?

@Nielsvanpach I did it. Can you run the tests?

No library or built-in php intl functions convert dates properly for all years, including “geniusts/hijri-dates”. If you accept, it is most logical to prepare the dates between 1970 and 1938 as a constant class for Islamic holidays. Because Islamic holidays predicted and actual dates may change until the last moment.
https://en.wikipedia.org/wiki/Eid_al-Fitr?wprov=sfti1#In_the_Gregorian_calendar
https://en.wikipedia.org/wiki/Eid_al-Adha?wprov=sfti1

Hi @Nielsvanpach
PR is done. Can you check please ?
https://github.com/spatie/holidays/pull/70/files/d12bcb4a40a027e2fc8f5468f24b00f8a82a5634#diff-9692465f3ee0b4a7d6aa36598b76c3a36c88a1b36e90576bbe585872cf5854eaR10

Thanks for your work on this @thecaliskan !


Connections and Unions

Hey there! First of all thanks for this package, it really helps us to improve our development process! I have some troubles building unions and connections with unions.
I have the following gql queries:
query Search($searchTerm, $filters, $first, $last, and more...) {
  edges {
    node {
      .. on Product {
        id
      }
      .. on User {
        id
      }
    }
  }
}

or
query CurrentViewer($id, $favorites_first, and more..) {
  node(id: $id) {
    id
    favorites(first: $favorites_first) {
      edges {
        node {
          .. on Location {
            id
          }
          .. on Category {
            id
          }
        }
      }
    }
  }
}

In both cases the type Node has no props e.g.:
--- SNIP ---
@JsonSerializable(explicitToJson: true)
class Searchables with EquatableMixin {
  Searchables();

  factory Searchables.fromJson(Map<String, dynamic> json) =>
      _$SearchablesFromJson(json);

  @override
  List<Object> get props => []; # empty props
  Map<String, dynamic> toJson() => _$SearchablesToJson(this);
}
--- SNIP ---

Expectation is:
--- SNIP ---
@JsonSerializable(explicitToJson: true)
class Node with EquatableMixin {
  Node();

  factory Node.fromJson(Map<String, dynamic> json) {
    switch (json['__typename']) {
      case 'Category':
        return Category.fromJson(json);
      case 'Location':
        return Location.fromJson(json);
      default:
    }
    return _$NodeFromJson(json);
  }
}
--- SNIP ---

Hello @0x6a6f74726f!
Are you setting resolve_type_field on schema_mapping configuration to __typename?
Is this an union type?
Could you share the GraphQL schema or create a minimal reproducible repository with this issue?
Thanks!

Hi @comigor - First off, thanks for all the great work! :)
Is typename only generated for interfaces and unions? I'm using typename for caching on every object and wondering if there's a way to add it when it's specified in the queries?

@comigor yes we set the resolve_type_field according to our schema. If you need more stuff, i am happy to support!

@0x6a6f74726f Cool! I'll take a look at it when I got some time.
@michael-golfi Hey there! Thank you! For now we only generate typename for interfaces and union types "instances". We also drop any other field that starts with __ to have its code generated. Are you using typename differently? Does it agree with GraphQL protocol? Maybe we think in a way of whitelisting those fields if necessary.

I'm not sure that it's necessarily part of the protocol but I'm running queries similar to the following:
query Me {
    viewer {
        __typename
        id
        firstName
        lastName
    }
}

But the class only generates fields for id, firstName and lastName. If the query requests __typename, should it appear in the class as well?

No. It's currently, on purpose, filtering any field that starts with __.
That's why I've asked what's the use case for this, so maybe we can remove this filter.

That would be great!
I use __typename/id as a key for client side caching.

@michael-golfi I've create a new issue to not filter those fields, let's keep this one to the connection problem.
https://github.com/comigor/artemis/issues/41

Hey @jhattwich,
I've recently release 4.0.0 and this was probably fixed among other things! (And with an working example of GitHub)
Feel free to reopen thiss issue if the error still happen after bumping.


Mixin inerhitance is inconsistent when used with extends

Version
2.4.4
Reproduction link
https://jsfiddle.net/eczcLptu/2/
Steps to reproduce
Extend a component and add a mixin
What is expected?
The rule of inerhitance should be

component
extended component
mixin

What is actually happening?
The actual inerhitance is

component
mixin

This is not clearly documented and therefore a bit missleading. I am not sure wether it can technically count as a bug, but at least it is inconsistent behavoir

It's working as you expect, the mixin takes precedences, try adding something to child and child_mixin components: https://jsfiddle.net/16Lusyur/
If you put the mixin in Child, then child_mixin would overwrite it as well

No it is not. child_mixin should inerhit getMessage from Child, and then the Mixin should be merged with child_mixin, thus resulting in child_mixin's inerhited getMessage method to override the Mixin, and be the final getMessage

Unlike lifecycle hooks, methods are not merged, they are overwritten.

Options that expect object values, for example methods, components and directives, will be merged into the same object. The component’s options will take priority when there are conflicting keys in these objects:

Then the docs are inaccurate.

they're not:
const a = { foo () => 'foo' }
const b = { foo () => 'bar' }
const c = {...a, ...b}
c.foo() // bar

a and b being two methods objects

The key problem here is that apparently Vue.extend does not consistently behave with what is noted in the docs. I am very well aware that for two components this is true, but for using the combination of extends and mixin this is not what should happen in a logical OOP paradigm.

Can you explain yourself? What is inconsistent?
to be fair mixins doesn't exist in OOP, so I don't see what you're referring to

The scenario should be like this
Component (Class) A

method

Mixin

method

Component (Class) B, inerhits component A, mixin of B

method inerhited from component A
mixin should be merged. method already found because inerhited. componentB#method used

My take on this: https://jsfiddle.net/16Lusyur/1/ - but well, sure as mixins are not really™ some standard, the documentation could just say "Mixins will override anything which the component inherits via extends". I thing this is right now just undocumented.

Well. According to Wikipedia I would not say that there is no "common sense" for mixins. Also I wouldn't say that Vue.extends works consistently here. On the other hand, just documenting this more accurately would have saved some nerves.

IMO, the mixin being local, it should take precedence to the extended component, but local properties take precedence over mixins

This has nothing to do with that mixins are or what is OOP.
Different options in Vue have different merge strategies.

Lifecycle hooks are "merged" in a way that preserves all hooks in the inheritance chain, because they do not return values and instead produce side effects.

Methods of the same name simply overwrite parent value because methods can return values and it would cause ambiguity on what value to return if they are "merged" into a single function. Same for asset-registration options like directives and components.

I can see what the OP means now, essentially the problem is that the options that comes from extends should take higher priority than options coming from a mixin.

Ok, so I still think the current behavior is expected. This is because in your example the mixin is applied to ChildWithMixin instead of Child. The mixin is considered as part of ChildWithMixin's own options and would overwrite whatever is inherited from Child.
If the mixin is applied directly to Child, then you'd see child logged instead.
However, in general, I don't like the idea of over-applying OOP when composing UI components, especially with confusing overrides like this.


Write a separate script to query GitHub for "filename: CITATION.cff" and write to a file

Pagination will be relevant for getting that many api results

It may be more straightforward to select some repositories manually. In that case we won't need to deal with pagination etc.


Pie: Split draw function into sub-functions

This also fixes issues with selected-style and font-style not being properly applied

This needs tests

Can this be merged? It fixes some issues with the Pie chart


why 'center_coordinates' in read_gem(), but not in read_gef()?

What is the effect of this parameter？Should we use it？
https://github.com/STOmics/Stereopy/blob/c230d6f59e313212b9e8d2cd3f7ac19287f2b58b/stereo/io/reader.py#L1161-L1170
https://github.com/STOmics/Stereopy/blob/c230d6f59e313212b9e8d2cd3f7ac19287f2b58b/stereo/io/reader.py#L45-L53
Thanks for any reply！

center_coordinates is only available for bins bin_type, each bin actually is a square, setting it to True means the central coordinate of each bin is set as its coordinate, otherwise, the upper left corner coordintates are used, due to some unknown  historical reasons, previously, if input file is a GEM, the coordinates of bins were forced to be the center of bins, but if a GEF, bins coordinates are always the upper left corner coordintates, in order to unify the GEM and GEF, we added the parameter center_coordinates to read_gem and set it to False by default so that the bins coordinates can be the upper left corner of bins and if a user wants it to be the bin center, center_coordinates can be to True.


org-mode src-blocks

are they supported?
Could not find I way to get them working
https://discuss.gohugo.io/t/org-mode-source-code-block/6755

Ask here:
https://github.com/chaseadamsio/goorgeous/issues


feat: add favicon and migration to manifest 3

Thanks for developing a great Chrome extension.

This is so kind of you to PR, everything seems to work perfectly! Thanks for the update ☺


Use torch.testing.assert_close in test_models.py

part of #3865

There are failures, some (maybe all) related to https://github.com/pytorch/vision/pull/3870, so let's wait for these to be fixed first.


Reset method isn't working

I tried to use the reset method with the only captcha that I had in my page and It's showing me this error:

Invalid ReCAPTCHA client id: 0

Same problem here. I use Recaptcha V2
Have you found any workarounds? This bug is very annoying

Can anyone fix this issue please?

Hey guys! I've encountered the same issue using react-recaptcha and decided to create my own library. It has some improvements comparing to this lib and most important - it works completely fine! 🚀
Library: https://github.com/sarneeh/reaptcha
Example: https://sarneeh.github.io/reaptcha/
Hope it helps! 😄 🎉


Update App info layout

Currently not all of the information is shown. Update they layout and discuss which information to show on the app page.
Show all projectInfo fields
Add a breadcrumb to navigate back to directory
Make some categories clickable (e.g., user clicks on ADaM and get to Directory with ADaM filter selected)?
Hide fields which are blank

I have drafted an update in the appLayout branch.

Show all projectInfo fields - there is a button, which allows to show/hide additional details. It is enabled by default in the editor preview and disabled in a standard project view. Not sure we want to show all of the information by default, because some of it feels very specific to an application stage.
Hide fields which are blank - done
Add a breadcrumb to navigate back to directory - replaced a name with breadcrums.
Make some categories clickable - I did not implement it as it feels a bit complicated. We would have to use context as we are not using redux and I do not think that this feature is so much needed.


Scala.Scala.2 version <IP_ADDRESS>0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

⚠ Accidentally modified the existing manifests. It should be closed.


Pink background instead of transparancy

#13 is about the same issue..
My transparent background color displays as pink.

I wanted to do something like Facebook does for group chats: merging participant avatars to be one image. Code I have:
//$overlayWhite = new Overlay($size,$size,new Color(255,255,255)); I get the same if I use this instead of circle.png
$background = new CircularShape(FCPATH.'public/loremipsum/img/users/circle.png');
$background->build();

foreach($users as $i => $user) {
    $file = str_replace(img_url(), FCPATH.'public/loremipsum/img/', $user->getAvatar());
    $img = new Square($file);
    $img->setDefaults();
    $img->resize($size/2,$size/2);
    //$offsetX = ($img->getWidth()-$size)/2;
    //$offsetY = ($img->getHeight()-$size)/2;
    //$img->crop($offsetX , $offsetY, 32, 32); crop don't does what I expected... Probably beacause of #16
    $img->build();

    switch($i) { $background = $background->merge($img,$w,$h); } // just to give an idea of merging, I select up to 4 avatars zo they get positioned in each corner of main image ($background)..
    // This works so not showing all code

}
return $background->getDataURI();

And another unexpected behaviour: I hoped using circularShape would cut of corners from merged squares, so the resulting image would be a circle.

We'll investigate on the problem soon..

After building image :
$background->build();
it is just a image so you can't use it as circular shape.If you want to do that create square image by using your 4 images and pass it to circular shape.
Upgrade to latest version and see if errors still there.

This is an example for create circular shape using 4 images :

    $img = new Square('./image1.jpg');
    $img->setDefaults();
    $img->resize($size/2, $size/2);
    $img->build();

    $img2 = new Square('./image2.jpg');
    $img2->setDefaults();
    $img2->resize($size/2, $size/2);
    $img2->build();

    $img3 = new Square('./image3.jpg');
    $img3->setDefaults();
    $img3->resize($size/2, $size/2);
    $img3->build();

    $img4 = new Square('./image4.jpg');
    $img4->setDefaults();
    $img4->resize($size/2, $size/2);
    $img4->build();

    $final = $img->merge($img2,125,0);
    $final = $final->merge($img3, 0,125);
    $image = $final->merge($img4,125,125);

    $circular = new CircularShape($image);
    $circular->build();

@BrainFeeder I think @antondias  is correct,  follow his code also please upgrade to latest ImageArtist Version, i'm closing this issue making as not reproducible issue


feat: add AWAY state configuration

Related Issues

Closes #96

Proposed Changes:
Adds arm AWAY configuration in the OptionsFlow step with the following logic:

After version update, the setting is empty
If the setting is not set (empty), the device arms all sectors (backward compatibility)
If the setting is set, only selected sectors are armed

Testing:

Arming the system without setting the option should arm all sectors
Arming the system with the option set, should arm only selected sectors

Extra Notes (optional):
n/a
Checklist

[x] Related issues and proposed changes are filled
[x] Tests are defining the correct and expected behavior
[x] Code is well-documented via docstrings

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.249%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
504

Relevant Lines:
536

💛  - Coveralls


PointerHoverEvent shouldn't allow non-zero pressure

Currently, PointerHoverEvent's unnamed constructor accepts a parameter pressure, and passes it to its superclass PointerEvent. Since "hover" event is defined as moving while not touching the screen, a non-zero pressure does not make sense.
We propose to break the API by removing the parameter pressure from PointerHoverEvent's unnamed constructor, instead PointerHoverEvent will always pass pressure = 0.0 to PointerEvent.
This change will break code that explicitly creates a PointerHoverEvent with pressure specified.

If it is pressure = 0.0, simply remove this line and the behavior will be the same.
If it is a non-zero pressure, consider using PointerMoveEvent instead. "Move" event is defined as moving while touching the screen, which accepts a non-zero pressure.

This change will not break code that listens to events generated by Flutter embedders, including all gestures.
A proposed PR can be found at https://github.com/flutter/flutter/pull/30414

It currently takes all mouse events, as with the down property, which would explain it. I see it more of a scale situation, where this callback overrides PointerMoveEvent (though this is not mentioned). Unless I'm misinterpreting it, this seems intentional.

It currently takes all mouse events, as with the down property, which would explain it. I see it more of a scale situation, where this callback overrides PointerMoveEvent (though this is not mentioned). Unless I'm misinterpreting it, this seems intentional.

Sorry, I don't quite get you, can you elaborate?
As far as my understanding of your point,

PointerHoverEvent does not allow customization of down, instead sets down = false.
The definition of Hover and Move are described in their doc, where hover The pointer has moved with respect to the device while the pointer is not in contact with the device." while Move is "The pointer has moved with respect to the device while the pointer is in contact with the device." They are not interchangeable.

Upon further looking, I can see I was indeed misinterpreting it. I could have sworn down returned its value, but you're right. My apologies.


Fix library install directories on Windows

Currently, OpenFHE installs all library products in the lib directory. AFAIK, this is wrong (or at least bad practice) on Windows, where the .dll files belong into the bin directory.
This PR remedies this by configuring the library installation in CMake to do The Right Thing:

On *nix systems, all library products go to lib
On Windows systems (including Cygwin and MinGW), static libraries and .dll.a or .lib libraries go to lib, while the dynamic libraries go to bin

From https://github.com/openfheorg/openfhe-development/pull/633#issuecomment-1877451241:

At the moment I am not sure how we are going to proceed with PR #634. I would need to discuss this with my colleague(s) first.

@dsuponitskiy Sure, let me know if I can be of any help. Right now, I don't see any downsides to the proposed change, since it essentially leaves all non-DLL systems unchanged and fixes only the install locations for DLL systems.
Note that the proposed change is also the default for the currently required minimum CMake version 3.5.2, while at since CMake version 3.14 it is possible to change the install location ad hoc during configure time using the CMAKE_INSTALL_XXX variables.

@dsuponitskiy Are there any updates on this issue?

@sloede - you may want to change your code to be conditional to WINDOWS
e.g. see: https://stackoverflow.com/questions/9160335/os-specific-instructions-in-cmake-how-to
if(MSVC OR MSYS OR MINGW)
    # for detecting Windows compilers
endif()

you may want to change your code to be conditional to WINDOWS

I do not think that's necessary. The proposed parameter settings should work for all Unix-like and Windows-like systems. In fact, I think it has been the default for a number of CMake releases now.

@yspolyakov are there any updates on this issue? I'd be happy to update the PR such that it can be merged eventually (I don't think there is much to it anyways)


Fix try_everything example.

Simple change log w/ logger to work.

we've gotta do something about these failing tests :(

Yep. It looks ugly 😺

oh well, this is definitely correct anyway so let's merge it and deal with the consequences later :)


home screen font on firefox

home screen font on mozilla based browser is not the same as chromium based.

fixed in #65


Give way to force a deployment after a local git push fails

e.g. in the case of #878, we end up in a state where it's hard to cause a deployment without creating a bogus commit. We need to give users an easier way to deal with this.

I remember running into this too and using git push --mirror and probably a --force too to trigger a redeploy, of course it's a very unnatural thing to do but it did work.
C:\localrepo [master]> git push
Everything up-to-date
C:\localrepo [master]> git push --mirror
Total 0 (delta 0), reused 0 (delta 0)
remote: Updating branch 'master'.
remote: Updating submodules.
remote: Preparing deployment for commit id 'b7eb192658'.
remote: Generating deployment script.
remote: Running deployment command...
remote: Handling Basic Web Site deployment.
remote: KuduSync.NET from: 'D:\home\site\repository' to: 'D:\home\site\wwwroot'
remote: Finished successfully.
remote: Deployment successful.
To<EMAIL_ADDRESS> * [new branch]      master/master -> master/master
C:\localrepo [master]>

Nice, never used this flag before!

wait, what does --mirror do here? The man page description doesn't seem to make sense for this scenario

Well, I'm not entirely sure why that worked, but in theory --mirror overwrites the remote repo and probably this triggered a redeploy in Kudu.
--mirror

Set up a mirror of the source repository. This implies --bare. Compared to --bare, --mirror
not only maps local branches of the source to local branches of the target, it maps all refs
(including remote-tracking branches, notes etc.) and sets up a refspec configuration such
that all these refs are overwritten by a git remote update in the target repository.

https://www.kernel.org/pub/software/scm/git/docs/git-clone.html

Hi @suwatch

We have such already. curl '<scm-uri>/deployments' -X PUT -k -data "{}"

I can't seem to get this working. Does this method still work? Or any other command has replaced this setting for Azure App Service?

Hi @suwatch

We have such already. curl '<scm-uri>/deployments' -X PUT -k -data "{}"

I can't seem to get this working. Does this method still work? Or any other command has replaced this setting for Azure App Service?

Did you try with --mirror suggested above?

Did you try with --mirror suggested above?

Yes, it gives error
error: --mirror can't be combined with refspecs
I'm using Kudu for Azure App Service deployments.

Yes, it gives error
error: --mirror can't be combined with refspecs
I'm using Kudu for Azure App Service deployments.

I suspected below also does not support refspecs.   Could you share your commit command syntax?  Also share the site name here or indirectly and the UTC time when you called below and it did not work.

We have such already. curl '/deployments' -X PUT -k -data "{}"

I suspected below also does not support refspecs.   Could you share your commit command syntax?  Also share the site name here or indirectly and the UTC time when you called below and it did not work.

We have such already. curl '/deployments' -X PUT -k -data "{}"


Error in validating transition seals

Hi @dr-orlovsky,
I continued the consecutive transfer test and found strange behavior at this step.
The condition is only valid on the first transfer. In subsequent transfers, the outpoint of the seal is always None, causing the VerifyError::WitnessNotClosingSeal. This error occurs because the seal outpoints always None after the first transfer.
For Example, the in the list below, the first blindseal causes VerifyError::WitnessNotClosingSeal:
BlindSeal { method: TapretFirst, txid: WitnessTx, vout: Vout(0), blinding:<PHONE_NUMBER>2054571931 }
BlindSeal { method: TapretFirst, txid: Txid(Txid("1ab6073b14c4232f4fac45b32c71b3fabb5234b2af8eef4aa96f72da38f2bf4d")), vout: Vout(0), blinding:<PHONE_NUMBER>2929434069 }

If we comment the step, the consignment is validated correctly.

Hi @dr-orlovsky,
Currently, I think we need an extra step on the consign operation: Add in someplace to the consignment file the prev_out txid to new transfer.
Today, the first transfer retrieves the information from Genesis->assignments, we need to add this information to validate the consignment file, we need all predecessor transitions, right?
This is one way to change WitnessTx to  prev_out txid and pass in validation.

If we comment the step, the consignment is validated correctly.

It is not; it is just not validated opening double-spending attack.
This step should work since it has to take Witness txid, i.e. construct an outpoint matching the witness transaction - and find it among inputs of the spending transaction. And the witness is constructed with the information coming from the anchor - which contains witness txid. Thus, we do not need to add anything anywhere.
https://github.com/RGB-WG/rgb-core/blob/94fd411bfe51039cc548dd5933d2c9af41637634/src/validation/validator.rs#L373C31-L376
The question remains though why the validation still fails not finding the input. Can you add dbg!(...) around self to inspect what's wrong with the witness construction

It is not; it is just not validated, opening a vector for double-spending attack.

Yes, yes, I know!
I commented on this snippet to make sure there not were other issues.  =)

The question remains though why the validation still fails not finding the input. Can you add dbg!(...) around self to inspect what's wrong with the witness construction

In general, only genesis BlindSeal has the outpoint, the others only have "Ptr::WitnessTx", where this verification fails.
BTW, I will provide this information to you ASAP.

Hi @dr-orlovsky,
Added a dbg! on BlindSeal too. See below:
[/home/crisdut/projects/contrib/rgb-core/src/validation/validator.rs:420] seal = BlindSeal {
    method: TapretFirst,
    txid: WitnessTx,
    vout: Vout(
        0,
    ),
    blinding:<PHONE_NUMBER>2054571931,
}
[/home/crisdut/projects/contrib/bp-core/seals/src/txout/witness.rs:97] self = Witness {
    tx: Tx {
        version: TxVer(
            2,
        ),
        inputs: Confined(
            [
                TxIn {
                    prev_output: Outpoint {
                        txid: Txid(
                            "9ab6b9837f9a681ecfbdeb6ec34ba002c42b23c6d43e5dd26fe0a56a726b869e",
                        ),
                        vout: Vout(
                            0,
                        ),
                    },
                    sig_script: SigScript(
                        ScriptBytes(
                            Confined(
                                [...],
                            ),
                        ),
                    ),
                    sequence: SeqNo(
<PHONE_NUMBER>,
                    ),
                    witness: Witness(
                        Confined(
                            [
                                Confined(
                                    [...],
                                ),
                            ],
                        ),
                    ),
                },
            ],
        ),
        outputs: Confined(
            [
                TxOut {
                    value: Sats(
                        9998000,
                    ),
                    script_pubkey: ScriptPubkey(
                        ScriptBytes(
                            Confined(
                                [...],
                            ),
                        ),
                    ),
                },
            ],
        ),
        lock_time: LockTime(
            0,
        ),
    },
    txid: Txid(
        "f7418e133dacad73b0d09f77c3c19cdcd486f2df2d30c1754a6b53c5cb85d909",
    ),
    proof: TapretFirst(
        TapretProof {
            path_proof: TapretPathProof {
                partner_node: None,
                nonce: 0,
            },
            internal_pk: InternalPk(
                XOnlyPublicKey(
                    53a1ec17b77db57ef39e76464555ebe9fbdfc779850b56074f040d249cdc46d5d4a06994078489cce5d654b62161ca5679af408536d7ee0716358803b93a14f1,
                ),
            ),
        },
    ),
}

[/home/crisdut/projects/contrib/rgb-core/src/validation/validator.rs:420] seal = BlindSeal {
    method: TapretFirst,
    txid: Txid(
        Txid(
            "1ab6073b14c4232f4fac45b32c71b3fabb5234b2af8eef4aa96f72da38f2bf4d",
        ),
    ),
    vout: Vout(
        0,
    ),
    blinding:<PHONE_NUMBER>2929434069,
}
[/home/crisdut/projects/contrib/bp-core/seals/src/txout/witness.rs:97] self = Witness {
    tx: Tx {
        version: TxVer(
            2,
        ),
        inputs: Confined(
            [
                TxIn {
                    prev_output: Outpoint {
                        txid: Txid(
                            "1ab6073b14c4232f4fac45b32c71b3fabb5234b2af8eef4aa96f72da38f2bf4d",
                        ),
                        vout: Vout(
                            0,
                        ),
                    },
                    sig_script: SigScript(
                        ScriptBytes(
                            Confined(
                                [],
                            ),
                        ),
                    ),
                    sequence: SeqNo(
<PHONE_NUMBER>,
                    ),
                    witness: Witness(
                        Confined(
                            [
                                Confined(
                                    [...],
                                ),
                            ],
                        ),
                    ),
                },
            ],
        ),
        outputs: Confined(
            [
                TxOut {
                    value: Sats(
                        9999000,
                    ),
                    script_pubkey: ScriptPubkey(
                        ScriptBytes(
                            Confined(
                                [...],
                            ),
                        ),
                    ),
                },
            ],
        ),
        lock_time: LockTime(
            0,
        ),
    },
    txid: Txid(
        "9ab6b9837f9a681ecfbdeb6ec34ba002c42b23c6d43e5dd26fe0a56a726b869e",
    ),
    proof: TapretFirst(
        TapretProof {
            path_proof: TapretPathProof {
                partner_node: None,
                nonce: 0,
            },
            internal_pk: InternalPk(
                XOnlyPublicKey(
                    53a1ec17b77db57ef39e76464555ebe9fbdfc779850b56074f040d249cdc46d5d4a06994078489cce5d654b62161ca5679af408536d7ee0716358803b93a14f1,
                ),
            ),
        },
    ),
}
Consignment is NOT valid
Validation failures:
- transition e9dbb0762f80b2b72bdd5789f1f25af3014fdd421e19a083f5c742b954270a00 doesn't close seal with the witness transaction f7418e133dacad73b0d09f77c3c19cdcd486f2df2d30c1754a6b53c5cb85d909. Details: the provided witness transaction f7418e133dacad73b0d09f77c3c19cdcd486f2df2d30c1754a6b53c5cb85d909 does not closes seal f7418e133dacad73b0d09f77c3c19cdcd486f2df2d30c1754a6b53c5cb85d909:0.
Validation warnings:
- duplicated terminal seal 8bNtrM7bEB6Ytb99jEibEA8FbZgzMqbJ9L36HrGf9W3o in operation e9dbb0762f80b2b72bdd5789f1f25af3014fdd421e19a083f5c742b954270a00.

Right... Very interesting.
The first transfer creates witness transaction spending 9ab6b9837f9a681ecfbdeb6ec34ba002c42b23c6d43e5dd26fe0a56a726b869e:0 output, which, I assume, contains the originally issued asset. The txid of this first witness transaction is f7418e133dacad73b0d09f77c3c19cdcd486f2df2d30c1754a6b53c5cb85d909. It also assigns all assets to the first (and the only) output of this transaction via WitnessVout seal.
Now, the second transfer tries to transfer assets not from that newly created assignment - but from 1ab6073b14c4232f4fac45b32c71b3fabb5234b2af8eef4aa96f72da38f2bf4d:0 (WTF is that and where it comes from?). Moreover, it creates a witness transaction with id 9ab6b9837f9a681ecfbdeb6ec34ba002c42b23c6d43e5dd26fe0a56a726b869e, which is not possible, since that was the original (already existing) transaction holding the very first assignment we were spending in the first transfer.
Unsuprisingly the validation fails.

I.e. this is an application-level bullshit, not consensus-level, i.e. the issue is in rgb-std, rgb-wallet or bitmask-core - but not in bp-core or rgb-core.
Can you pls provide details about how this transfers were created? Can I expect transactions with that IDs? Are they in testnet or your regtest?

@crisdut I need more information from you here:

Can you pls provide details about how this transfers were created? Can I expect transactions with that IDs? Are they in testnet or your regtest?

You can also check the explanation of the problem I have given above. Right now it is not clear to me how to process with the issue without understanding how these tx ids are produced and what is in the actual transactions.


Is there a way to change download directory?

is there a way to change the destination folder ?

You'll have to edit the source code for that.


Greater flexibility in prefix_util.py.em

Currently the python workspace setup scripts generated from prefix_util.py.em support two basic layouts:

With --merged-install, a merged workspace where all package installspaces are overlaid on the same FHS tree (only hooks are considered)
Without --merged-install, each package must have its installspace in an immediate subdirectory with the exact name of the package (envvar dsvs are merged, custom hooks also appended).

I would like to have a bit more flexibility here, for example the ability to move my installspaces around to arbitrary locations (fixing embedded paths as required), and then pass a list of those locations into this logic. I could also achieve what I want by having a fully separate script, and perhaps importing and reusing the other pieces of the logic from _local_setup_util_sh.py, but I'm just wanting to open a discussion here about whether others could foresee use cases for something more flexible here.
In my case, I'd like to be able to include a per-package version string/hash in the install path. Another is being able to include only a selection of packages from an isolated workspace in an underlay, for example those listed as build dependencies it a package.xml.

Adding some more colour here, my scenario looks like installing packages in paths like this:
/<prefix>/.pkgs/class_loader/cee77c5ddf3c/share/class_loader/package.xml
/<prefix>/.pkgs/cmake_modules/628981036a3c/share/cmake_modules/package.xml
/<prefix>/.pkgs/common_msgs/ca4df7d384bd/share/common_msgs/package.xml
/<prefix>/.pkgs/cpp_common/ecb34908e962/share/cpp_common/package.xml
/<prefix>/.pkgs/diagnostic_msgs/7590b8e25622/share/diagnostic_msgs/package.xml
/<prefix>/.pkgs/dynamic_reconfigure/04a0c83f4fb4/share/dynamic_reconfigure/package.xml
/<prefix>/.pkgs/gencpp/d652c2bf53b2/share/gencpp/package.xml

In my case, the list and ordering is static, so I can supply a topologically-ordered list of these and then just generate the dsv-based workspace source file by piggy-backing on the existing _local_setup_util_sh.py logic:
from _local_setup_util_sh import get_commands

with open('workspace.txt', 'r') as f:
    lines = f.read().strip().split('\n')

for line in lines:
    pkg_name, prefix = line.split(':', maxsplits=1)
    for l in get_commands(pkg_name, prefix, 'bash', 'sh'):
        print(l)

However, this leads to a later colcon overlay treating this as N layered workspaces and sourcing each one independently (which takes forever). Unfortunately, it's also not as simple as something like an in-package marker file pointing to where the setup file is, since part of the point of this approach is being able to slice and dice the contents of a particular environment.
Would we entertain a new COLCON_WORKSPACE_PATH envvar, or similar, which if set, would bypass all of the funny business going on in https://github.com/colcon/colcon-ros/blob/master/colcon_ros/prefix_path/catkin.py, and instead just source the listed workspaces' local setup files in sequence? This could be paired with a new --custom-install flag/mode for _local_setup_util_sh.py, which is passed the location of a text file containing a list (similar to the above) of packages and their locations.


propagate sort order through retrieval sequence

Description
This PR...

Fixes: #9595
Note that is only fixes the regression reported in #9595, not enhancing it or fixing any oredering bugs that were already present before 4.19

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 10967

@blueorangutan test

[SF] Trillian test result (tid-11349)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 51059 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9624-t11349-kvm-ol8.zip
Smoke tests completed. 133 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


Completion not working

Issue Description
Type: bug report
Describe what happened (or what feature you want)
Completion seems to not work

Describe what you expected to happen
I expect to have completion showing, I can't remember all subcommands (just installed)
How to reproduce it (as minimally and precisely as possible)

have a completion plugin like nvim-cmp
enter cmdline
type Octo iss

Tell us your environment
I don't think this is related but I had issues with other plugins in the past by this:
OS: Windows
Anything else we need to know?
Completion works on other plugins

Also when I enter a wrong command, nothing happens, would be nice to have a warning o error for this(vim.notify)

Also when I enter a wrong command, nothing happens, would be nice to have a warning o error for this(vim.notify)

This should be fixed, now, please let me know otherwise

Strange, I'll investigate later

Also when I enter a wrong command, nothing happens, would be nice to have a warning o error for this(vim.notify)

This should be fixed, now, please let me know otherwise

Yeah it is thanks, I feel the message a little long

Yeah it is thanks, I feel the message a little long

Yep you are right, too many actions. I shorted it a bit

If it helps, command completion is handled here:
https://github.com/pwntester/octo.nvim/blob/master/lua/octo/commands.lua#L15-L17

I was testing and seems like { complete = require("octocompletion.")... } callback isn't being called, maybe its a neovim stuff, I'm running nightly(NVIM v0.9.0-dev-77-gf175ca9f7)

Weird, just tried with NVIM v0.9.0-dev-bd7ca10 and it works for me? Does it work if you create you own command with your own completion function?

I found something very odd, may look like a joke but

it just works if I change the command name in https://github.com/pwntester/octo.nvim/blob/5191478678356c9f1de9a4135bab6756f51dcb30/lua/octo/commands.lua#L15

That is odd indeed, I have done a little refactor of the completion function. Not sure if it will fix your issue, but please give it a try and let me know if it works for you.

I think I've found the reason, seems related to cmp-cmdline, check
https://gyazo.com/c2601f1c2a2c42814627f551cb80a4af
try the minimal config:
local on_windows = vim.loop.os_uname().version:match "Windows"

local function join_paths(...)
  return table.concat({ ... }, on_windows and "\\" or "/")
end

local package_root = join_paths(vim.env.TEMP or "/tmp", "nvim", "site", "pack")
local install_path = join_paths(package_root, "packer", "start", "packer.nvim")
local compile_path = join_paths(install_path, "plugin", "packer_compiled.lua")

vim.o.runtimepath = vim.env.VIMRUNTIME
vim.o.packpath = vim.fs.dirname(package_root)

local function load_plugins()
  require("packer").startup {
    {
      "wbthomason/packer.nvim",
      -- plugins here
      "nvim-lua/plenary.nvim",
      "nvim-telescope/telescope.nvim",
      "pwntester/octo.nvim",
      "hrsh7th/nvim-cmp",
      "hrsh7th/cmp-cmdline",
    },
    config = {
      package_root = package_root,
      compile_path = compile_path,
    },
  }
  require("packer").sync()
end

local function load_config()
  -- config here
  require("octo").setup {}

  require("cmp").setup {
    snippet = function() end,
  }

  require("cmp").setup.cmdline(":", {
    sources = {
      { name = "cmdline" },
    },
  })
end

if vim.fn.isdirectory(install_path) == 0 then
 vim.fn.system { "git", "clone", "https://github.com/wbthomason/packer.nvim", install_path }
 load_plugins()
 vim.api.nvim_create_autocmd("User", {
   pattern = "PackerComplete",
   callback = load_config,
   once = true,
 })
else
   load_plugins()
   load_config()
end

Steps to reproduce

nvim -u minimal.lua
enter cmdline mode and type :Octo is

Completion wont work but if you trigger only :Octo   and press tab, it will show proper completions, can you confirm please?

Ah, again, seems like changing the command name just works

this is very very odd

Still works for me with your minimal conf. Both entering Octo is<tab> and Octo <tab>

However, as you can see in the video, cmp-cmdline is showing some options (eg: starting with r) where none of them are and Octo options, and iterating through them with <tab> gives me the real Octo commands that starts with r

Do you have a non-windows machine to test that out?

Do you have a non-windows machine to test that out?

no :(

However, as you can see in the video, cmp-cmdline is showing some options (eg: starting with r) where none of them are and Octo options, and iterating through them with <tab> gives me the real Octo commands that starts with r

This is what I experience, it should show real completions even if you type :Octo is(wihout pressing tab), this is done if you change locally the octo command with something like Oct, which is weird

This seems to be a cmp-cmdline bug since, if you remove that part of your minimal.lua config, the completion works as expected. But yep, this is really weird, chaning the command to Octa makes it work as expected so Im wondering what that plugin is doing that can be affected by the name of the command 🤔

Well, I haven't touched my config for a good time, I just switched to lazy.nvim and seems to work fine. To be honest, I have no idea what happened lol. Thanks for taking the time to respond to this joke-look issue @pwntester


First basic features of the pokemon optimizer

Short Description: Pokemon optimizer, transfer and evolve

Transfer and evolve pokemons based on its rank in his family according to configurable sorting criteria.
Transfer and evolve to maximize xp = Keep enough basic pokemon to use your candies for evolution

TODO:

Allow deactivation of xp optimization for max level players
Support dps and moves in sorting criteria

+1 for this.

Quick question, why not improve the existing ReleasePokemon and EvolvePokemon tasks? This is awesome features, but we would have now a duplicate worker with two functions that already exist separately: transfer and evolve pokemon.

If I separate evolution and transfer, I will have to duplicate the logic since the selection of who must be evolved and who must be transfered are linked together.
I will change the EvolvePokemon task to still be compatible.
I also need to fix a few things on the current code. So please, use dry run and let me know if you see something strange in the choice of Pokemon to transfer/evolve.

For evolution, we must only keep Pokemon's with the lowest candy required for evolve.
Add one config parameter to keep at least the beat iv Pokémon of each name instead of only the best iv of the family.
Evolve only when we have lucky egg and enough evolution to cover the whole 5 minutes.

👍
@TheSavior  how do you think about this PR ? I'm starting to test locally.

I think there need a config.example, right?

I am not excited about having a new task that has duplicated logic from two of our other tasks. To follow on to @douglascamata's comment, I'm also not sure why we can't improve the tasks we have, or refactor the logic we currently have in our tasks to be able to reuse some of it in a task like this.
As this PR stands, I'd rather have this be as an external plugin than part of the core if it needs duplicated logic.

This is good for XP. @Anakin5 Merge logic with core task will be good system.
If our module separated into three kinds
Event -> Decision  ->  Action
The core will be much flexible.

There is no configuration parameters for now, but I am going to add a few.
Also I just realized I have to commit some fixes in the Pokemon.json data file.

I think this should me merged with the other workers, you could set configuration via a different setting, for example:
"optimizer" : {
    "setting 1" : True
}

Not everyone wants it on, but it could be that people want it partially on. To get this done, we will have to allow different settings for this. Still, the workers do the same thing and the logic should be kept in one place, imho.
There is another PR (#2482) which has a similar effect for the bag, perhaps these two could have merged settings to have them in one place as an optimizer?

@Anakin5 said: If I separate evolution and transfer, I will have to duplicate the logic since the selection of who must be evolved and who must be transfered are linked together.

Ok ok, then we need to find a good way to split this logic. Maybe we can create a class for filtering pokemon lists, for example. Duplicating code because otherwise I would duplicate other code is not a good excuse :P

I am fine with removing the EvolveAll and TransferPokemon tasks if you think all cannot live together. But there is no point in having tasks configurable if there is only one valid set.
Please explain why you want those 2 tasks to be split. The fact that it is like that now is not a good reason :P

@Anakin5 One reason it should be split is that some people want one feature but not the other. Also, they might want to put them in different places in the task list. Merging them into one won't let them do that.

Algo for Eevee family coming later. This one a bit tricky.

@Anakin5 This PR is currently waiting for changes. We don't want to duplicate logic that is shared between this and the other workers we currently have. If you need shared logic, it might be best to pull that out into another class that can be reused from something like this.
What might be valuable is to reduce the scope if this task. Perhaps it only replaces the existing TransferPokemon task to smartly transfer based on the number of necessary candy to evolve the ones it is keeping. Then people can use the existing Evolve task with a lucky egg.

@Anakin5 How do you feel about my last comment? I want to make sure we help you move this in a direction that will actually allow it to get merged.

How to use this feature ?

Keep best iv / cp / relative cp / across whole family
Couldn't find the correct parameter in the committed changes, nor on the config file.

Small tip: keep the methods ordered by call time. For example, if the first thing method x does is calling y, then y should be defined right below x. This makes a lot easier to read code in big files, otherwise you need a lot of scrolling all the time.

And you got merge conflicts again, sorry :/

Aaaand merge conflicts one more time. I'm so sorry that I have no time to merge your PR in time and you always get conflicts 😢

I want to get this in so you stop having merge conflicts, and then we can improve it as need be separately. It seems pretty self contained. :+1:

"sort": ["ncp"]  what is ncp?

the optimizer is only run at startup, never during run time:
2016-08-12 10:26:06,782 [PokemonCatchWorker] [INFO] [pokemon_inventory_full] Your Pokemon inventory is full! Could not catch!
After a restart it immediately evolves and transfers all pkmn.
Here is my task config:
"tasks": [
  {
    "type": "HandleSoftBan"
  },
  {
    "type": "CollectLevelUpReward"
  },
  {
    "type": "IncubateEggs",
    "config": {
      "longer_eggs_first": true
    }
  },
  {
      "type": "PokemonOptimizer",
      "config": {
        "enable": true,
        "dry_run": false,
        "evolve_only_with_lucky_egg": false
      }
  },
  {
    "type": "RecycleItems",
    "config": {
      "min_empty_space": 15,
      "item_filter": {
        "Pokeball":       { "keep" : 15 },
        "Greatball":      { "keep" : 40 },
        "Ultraball":      { "keep" : 50 },
        "Potion":         { "keep" : 0 },
        "Super Potion":   { "keep" : 10 },
        "Hyper Potion":   { "keep" : 30 },
        "Max Potion":     { "keep" : 50 },
        "Revive":         { "keep" : 20 },
        "Razz Berry":     { "keep" : 50 }
      }
    }
  },
  {
    "type": "CatchVisiblePokemon"
  },
  {
    "type": "CatchLuredPokemon"
  },
  {
    "type": "SpinFort"
  },
  {
    "type": "UpdateTitleStats",
    "config": {
        "enable": true,
        "min_interval": 10,
        "terminal_log": false,
        "terminal_title": true,
        "// stats": ["uptime", "xp_per_hour", "stardust_earned", "captures_per_hour", "level_stats" ],
        "stats": ["uptime", "xp_per_hour", "stardust_earned", "level_stats" ]
    }
  },
  {
    "type": "MoveToFort",
    "config": {
      "lure_attraction": true,
      "lure_max_distance": 2000
    }
  },
  {
    "type": "FollowSpiral",
    "config": {
      "enable": false,
      "diameter": 4,
      "step_size": 70
    }
  }
],

I really like this idea, but what do the following things mean:
"top": 1 ??
ncp ?
i would use it like this if its possible:

       "type": "PokemonOptimizer",

        "config": {
            "transfer": true,
            "evolve": true,
            "use_lucky_egg": true,
            "evolve_only_with_lucky_egg": true,
            "minimum_evolve_for_lucky_egg": 20,
            "keep": [
                {
                    "weedle": 45,
                    "evolve": true,
                    "sort": ["iv"]
                },
                {
                    "pidgey": 45,
                    "evolve": true,
                    "sort": ["iv"]
                },
                {

//below to keep every other pokemon than defined above based on IV CP, like the old release config did >//like this: "Slowbro": { "keep_best_cp": 1, "keep_best_iv": 1 },

                  "top": 2,

                    "evolve": false,
                    "sort": ["ivcp"]
                }
            ]
        }
    },

ncp stands for normalized cp. It is called cp_percent in the code actually. You can use cp_percent as well if it is more meaningful to you.
If task not running during live, it means you catch pokemon task is not updating the Pokemon inventory.

I will put some comment in the config to explain stuff a bit.

When can we expect it @Anakin5 really excited!

@Anakin5 Adding tips to explain in the config would help a lot. I have been waiting for a feature like this.
Just a few questions.

If the optimize code ONLY works at start up to evolve shouldn't there be some type of release config that runs all the time (how the code is now)? This will release lower CP or IV pokemon and not allow your pokebag to get complexly full with 200 pidgeys. If not, is there a way to automatically restart the bot when your pokebag get’s full to release and evolve to make room? I personally leave my bot running overnight and am not always there to manually restart when the bag is full.

Is there any way to use candy to max out CP (or until you run out of candy) before evolving to next level?

@nanogirl21
I added comments in the config in the next PR: https://github.com/PokemonGoF/PokemonGo-Bot/pull/3743
For your issue 1., it is not a normal behavior. This should not happen with the little change I made to the pokemon catch worker. Make sure you have that change and let us know if you still have issues.
2. No, since there is no point to use candies before evolving rather than after. We could however add a pamater that allow you to use your candies to upgrade your best pokemons.

@Anakin5  Thank you for your help. I will try to run later when I am back home. I do see that you added parameter "use_candies_for_xp". Thank you.


Code of Capture image in browser using Web Socket

Hello Sir,
AM working on this project. But, in additional I have one requirement regarding capture image from browser. Can you help me out how it can be possible to capture image from browser with PiCamera and using web Socket.
Its Urgent..
Waiting for positive response..
Thankyou

This is more a question for stackoverflow.com

Yup, definitely a question for SO - this isn't anything to do with the pistreaming project specifically so I'm closing it for now.


remove file added by mistake

this file was added by mistake during merging ADMM. This PR removes the file.

@jeverink , I made this PR, thank you for pointing this out :)


Add UI for DynGears, Spheres, Cylinders

#40 added more shapes to VertexMeshBuilder, lets expose those in the UI! And also improve the UI (#13) while we're there
The Gears UI will likely want to recommend using gears that mesh well together, such as potentially offering a "number of gears" and "gear size multiplier" rather than directly exposing Gear Radius.

Solved in eb99a5832797178873cb2c971c55597097111a9f


v1 Release Plan

Part one

[x] Test, approve and merge https://github.com/github/codescanning-jira-integration/pull/15
[x] Rename default branch to main
[x] Retain dummy “master” branch to accommodate existing workflows referencing this branch
[ ] Notify known users using this workflow for breaking changes

[ ] https://github.com/github/codescanning-jira-integration/issues/16

[ ] Refactor/Improve codebase

[ ] https://github.com/github/codescanning-jira-integration/issues/17
[ ] https://github.com/github/codescanning-jira-integration/issues/18

Part two

[ ] Update README.md
[ ] Rename repository
[ ] Release v1 tag
[ ] Smoke test v1 tag
[ ] Delete dummy master branch
[ ] Notify users to use v1 tag

Secret scanning alerts will be a part of v2, which will have a separate release issue.

v1 will be released this Thursday, 3 PM PST.


useful feature

I want to use components like function,
now there is arguments like props,
I think there could have return for components.
it's useful for somecase like datapicker.

Can you be more specific? What do you mean by "like function"?

Closing until more details are provided.


Update nextjs.page.server.mdx: Server/deployment agnostic

To give a better overview of the state of things with Next.js, under the 'Server/deployment agnostic' section.

Neat. How about this?
- (which becomes pricey at scale).
+ (which is [known](some-link) to [be](some-other-link) very [expensive](maybe-even-third-link))

Also that Vecel's pricing is opaque.

The community also made the framework-agnostic toolkit SST to ease Serverless deployment to AWS in particular (since AWS infrastructure is a pain to deal with directly/non-programmatically). So SST attempts to replicate Vercel's infrastructure and ease-of-deployment for Serverless, in a free, open-source, and self-hostable manner. Open-Next is typically used with SST if deployed to AWS Lambda.

Correct me if I'm wrong, but I think this is more of a AWS issue than a Next.js one? E.g. you'd equally may want to use SST for deploying Vike to AWS. But maybe I'm missing something 👀

I did some minor changes, feel free to revert any of it. (Your writen language is esquiste btw. 👌)

Thanks. I implemented your suggestions. All should be resolved and ready to merge now.

Is this change ok for you?

As always: feel free to disagree with my other changes as well.


@davidmarkclements - can you push the latest change to npm?

Hi David
I've merged a fix - can you push that to npm?
Alternatively, please add me as a collaborator for prompt-sync on npm so I can update and maintain
Thanks

hey @yegodz this is published –
@0x00A are we cool to add @yegodz as npm collab?

thanks @davidmarkclements


Fix md2tex fontenc

Remove
\usepackage[T1]{fontenc}

from highlight style, doesn't work with xelatex.

Worked around the issue using pdflatex or luatex.


RevGAN not listed?

The first table demonstrating capabilities of different kind of generative models tells that GAN is irreversible, probably in general sense. But note that prior to this work (Ouderaa & Worrall, 2019) published reversible GAN (RevGAN) for image-to-image translation.
Please add this information somewhere in the text.

Closing because of no response.

Closing because of no response.


Show previous versions of sprig game in editor (like in onedrive) and add undo/redo button

Due to the autosave feature, instead of pulling the (newer) online version of my game I wrote on another device, it just saved the local one, resulting in a lost of progress. With the possibilities to undo, or having an older version; this problem wouldn't have happened.
Obviously the possibility to undo/redo changes and/or the possibility to check out previous versions is extremely practical

@leomcelroy Do you support an undo/redo feature?  If so can you specify exactly how it should work in this thread here?

I think this could add quite a bite of complexity to the backend. If someone wants to propose how it could work I'm not entirely opposed to it but at the moment I'm going to suggest we not implement this.
It does seem like an unfortunate issue that you encountered. If there exists a newer version of your game it should not be overwritten. How did you manage to open an old version of the game on a separate device?

I think this could add quite a bite of complexity to the backend. If someone wants to propose how it could work I'm not entirely opposed to it but at the moment I'm going to suggest we not implement this.
It does seem like an unfortunate issue that you encountered. If there exists a newer version of your game it should not be overwritten. How did you manage to open an old version of the game on a separate device?

I had the browser tab still open, and when I got on the device with the older version, and I opened it, it just auto-saved it, overwriting the online version with the local one

given @leomcelroy's feedback, I am assigning this issue directly to you @jakjakob.  My stance on this issue is that it is not actionable until you (or someone else) can propose a concrete implementable feature that @leomcelroy can sign off on.
After you add commentary here describing the feature, please assign this issue to @leomcelroy for his consideration.

hey @jakjakob is this still something you'd be interested in working on?

hey @jakjakob is this still something you'd be interested in working on?

I'm currently busy, so I won't do much about it in the next period. @JosiasAurel you are free to work on it

Would it be possible to save them with timestamp (and then obv. make it so that there is a way to get to them)? Just some thoughts, never got the time to look if actually possible to realise
(side note: should this not be closed as not planned instead of completed??)
since I am already here, would it be possible to make an undo/redo feature on the editor itself (client side - only keeps track of changes made during that session)?


Request for SQLCommand or extend the Query object

In the current NAV 2018 on-premises we make use of SQL Command to produce high performing queries and more flexible queries than the current NAV Table and Query objects allow us.
Typical TSQL commands we use are

Like statement
More sophisticated OR statements
inner join
cross apply
union
Date formula's
Intermediate results in TempTables.
Join over different companies

If we want the current functionality written 100% using C/AL code, we have super bad performance, because it will end up in more and more sub queries and nested loops.

I agree.  This or expand the "Query" object to facilitate more advanced queries.

Sounds like a great idea! Could I ask either of you to post that idea to to Business Central - Ideas? You'll have better luck getting it implemented and noticed by other partners if you post it there, as this GitHub repository only handles issues regarding our set of library functions in (C/)AL.

Done:
https://experience.dynamics.com/ideas/idea/?ideaid=972a4dde-fc90-e911-80e7-0003ff68e84c

Great! Thanks :) Now let's hope for a lot of votes to get this pushed up in the backlog!


Data Science with Python : Principal Component Analysis

Description 📜
Completed the documentation of Principal Component Analysis that comprises of its objectives, important terminologies, its uses, PCA Algorithm, step by step working, conditions of PCA, and sample implementation with Python
Fixes #1034

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[X] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[X] Datascience with Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[X] I follow Contributing Guidelines & Code of conduct of this project.
[X] I have performed a self-review of my own code or work.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generates no new warnings.
[X] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Thank you so much @prathimacode-hub ma'am. It means a lot.


[vite] Error when evaluating SSR module

Please provide the environment you discovered this bug in.
Node: v18.11.0
OS: macOS Ventura 13.0.1
Chip: Apple M1
Which area/package is the issue in?
create-analog
Description
Installed fresh proyect of analog using npm init analog
Got error when opening http://localhost:5173/
Please provide the exception or error you saw
oguajardo@Otoniels-MacBook-Air Projects % npm init analog
✔ Project name: … test
✔ Select a template: › Analog
✔ Select a variant: › angular-v15

Scaffolding project in /Users/oguajardo/Projects/test...

Initializing git repository:

Done. Now run:

  cd test
  npm install
  npm run dev

oguajardo@Otoniels-MacBook-Air Projects % cd test/
oguajardo@Otoniels-MacBook-Air test % npm i
npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm WARN deprecated<EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead

added 1614 packages, and audited 1615 packages in 1m

134 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
oguajardo@Otoniels-MacBook-Air test % npm run dev

><EMAIL_ADDRESS>dev
> ng serve

The '@analogjs/platform' successfully started.
The server endpoints are accessible under the "/api" path.
  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
10:46:31 PM [vite] Error when evaluating SSR module<EMAIL_ADDRESS>failed to import "rxjs"

10:46:31 PM [vite] Error when evaluating SSR module ~analog/entry-server:

node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

TypeError: Method Promise.prototype.then called on incompatible receiver [object Object]
    at ZoneAwarePromise.then (<anonymous>)
    at node:internal/per_context/primordials:440:35
    at new Promise (<anonymous>)
    at new SafePromise (node:internal/per_context/primordials:408:29)
    at node:internal/per_context/primordials:440:9
    at Array.map (<anonymous>)
    at arrayToSafePromiseIterable (node:internal/per_context/primordials:437:5)
    at node:internal/per_context/primordials:453:21
    at new Promise (<anonymous>)
    at primordials.SafePromiseAll (node:internal/per_context/primordials:452:3)

Node.js v18.11.0
oguajardo@Otoniels-MacBook-Air test %

Other information
About 3 weeks ago, I initialized another analog proyect and worked flawlessly
This are the dependencies on the package-lock of that proyect,
 "dependencies": {
    "@analogjs/content": "^0.1.8",
    "@analogjs/router": "^0.1.0-alpha.8",
    "@angular/animations": "^15.0.0",
    "@angular/common": "^15.0.0",
    "@angular/compiler": "^15.0.0",
    "@angular/core": "^15.0.0",
    "@angular/forms": "^15.0.0",
    "@angular/platform-browser": "^15.0.0",
    "@angular/platform-browser-dynamic": "^15.0.0",
    "@angular/platform-server": "^15.0.0",
    "@angular/router": "^15.0.0",
    "@nrwl/angular": "^15.7.0",
    "front-matter": "^4.0.2",
    "marked": "^4.2.12",
    "prismjs": "^1.29.0",
    "rxjs": "~7.5.6",
    "tslib": "^2.4.0",
    "xp.css": "^0.2.6",
    "zone.js": "~0.11.8"
  },
  "devDependencies": {
    "@analogjs/platform": "^0.1.0-beta.18",
    "@angular-devkit/build-angular": "^15.0.0",
    "@angular/cli": "~15.0.0",
    "@angular/compiler-cli": "^15.0.0",
    "@nrwl/vite": "^15.7.0",
    "jsdom": "^20.0.0",
    "nx": "^15.7.0",
    "typescript": "~4.8.4",
    "vite": "^4.0.3",
    "vitest": "^0.25.8"
  },
  "engines": {
    "node": ">=16.0.0"
  }

And this is the new project package-lock, I noted in the new proyect it uses "latest" instead of an specific version, genuinely don't know if this has something to do
"dependencies": {
        "@analogjs/router": "latest",
        "@angular/animations": "^15.0.0",
        "@angular/common": "^15.0.0",
        "@angular/compiler": "^15.0.0",
        "@angular/core": "^15.0.0",
        "@angular/forms": "^15.0.0",
        "@angular/platform-browser": "^15.0.0",
        "@angular/platform-browser-dynamic": "^15.0.0",
        "@angular/platform-server": "^15.0.0",
        "@angular/router": "^15.0.0",
        "@nrwl/angular": "^15.7.0",
        "rxjs": "~7.5.6",
        "tslib": "^2.4.0",
        "zone.js": "~0.11.8"
      },
      "devDependencies": {
        "@analogjs/platform": "latest",
        "@angular-devkit/build-angular": "^15.0.0",
        "@angular/cli": "~15.0.0",
        "@angular/compiler-cli": "^15.0.0",
        "@nrwl/vite": "^15.7.0",
        "jsdom": "^20.0.0",
        "nx": "^15.7.0",
        "typescript": "~4.8.4",
        "vite": "^4.0.3",
        "vitest": "^0.25.8"
      },

I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Duplicate of #203
This was a bug that has since been fixed in Angular. Try Node v18 13.0
https://github.com/analogjs/analog/issues/203#issuecomment-1399145967

Thanks @brandonroberts !


chore(build): publish universal wheel

Run setup.py like this:
$ python setup.py sdist bdist_wheel upload

You need access to https://pypi.python.org/pypi/simpl; Maintainer at least? Can we document $ python setup.py sdist bdist_wheel upload somewhere too along with the process for bumping the version and any tags we create?

@ziadsawalha https://github.com/checkmate/simpl/wiki#cutting-a-release


feat: add push providers

Add push providers

Agreed with Ferhat that we won't add tests until we don't generate test credentials and it can be merged as it is for now