I run only a eosio node locally, and another machine just receive the blocks

2018-09-05T06:29:31.826 thread-0   producer_plugin.cpp:330       on_incoming_block    ] Received block ab7bfc5a9e81b69d... #1937000 @ 2018-09-03T07:35:05.000 signed by eosio [trxs: 0, lib: 1936999, conf: 0, latency: 168866826 ms]
2018-09-05T06:29:32.183 thread-0   controller.cpp:1036           push_block           ] 3170003 producer_double_confirm: Producer is double confirming known range
producer eosio double-confirming known range
{"prod":"eosio"}
thread-0  block_header_state.cpp:158 next
2018-09-05T06:29:32.183 thread-0   producer_plugin.cpp:309       on_incoming_block    ] 3170003 producer_double_confirm: Producer is double confirming known range
producer eosio double-confirming known range
{"prod":"eosio"}
thread-0  block_header_state.cpp:158 next
rethrow
{}
thread-0  controller.cpp:1036 push_block
2018-09-05T06:29:32.183 thread-0   controller.cpp:1036           push_block           ] 3030001 unlinkable_block_exception: Unlinkable block
unlinkable block
{"id":"001d913143950247043b01aa2dc786263033152c77ef5a3e492b419cdb1c79f7","previous":"001d9130e632463987a4e0ac7939cb7eff35d18bf9faf5efe78475d59c7100a9"}
thread-0  fork_database.cpp:151 add

Do you have 2 nodes producing independently?

Thanks for your reply.   I running 2 nodes. And one is "eosio" who enable producer_plugin. The other's producer-name is like "samkunsamkun", not "eosio", But I also enable producer_plugin. Because I need to try make the second produce blocks after voting. Is it wrong?

Even  I just receive the blocks, disable producer_plugin. The same error:
2018-09-06T01:14:06.713 thread-0   controller.cpp:1036           push_block           ] 3170003 producer_double_confirm: Producer is double confirming known range
producer eosio double-confirming known range
{"prod":"eosio"}
thread-0  block_header_state.cpp:158 next
2018-09-06T01:14:06.714 thread-0   producer_plugin.cpp:309       on_incoming_block    ] 3170003 producer_double_confirm: Producer is double confirming known range
producer eosio double-confirming known range
{"prod":"eosio"}
thread-0  block_header_state.cpp:158 next
rethrow
{}

The producer-name = eosio configuration option or -p eosio command line option must appear only once.  The second node must not have that setting, or the node will attempt to produce at the same time the first node is producing, which results in duplicate blocks.  The second node should have only producer-name = samkunsamkun or -p samkunsamkun.

answered by @jgiszczak. Please ask in eosio.stackexchange.com if you have further questions.


GDScript being broken in many places after #72487

Godot version
394bb0ee2bcd863aed75e1c6799c4c8138dbf269
System information
arch linux
Issue description
#72487 broke my project in quite a few ways. I haven't yet had time to look into everything. Including here one example with assigning enum values.
SCRIPT ERROR: Parse Error: Invalid operands "Input::MouseMode" and "Input::MouseMode" for "==" operator.
SCRIPT ERROR: Parse Error: Cannot assign a new value to a constant.

Here is the full log output: https://pastebin.com/raw/TbdyfMSX
Steps to reproduce
Add the following code to a new project:
Input.mouse_mode = Input.MOUSE_MODE_VISIBLE

Minimal reproduction project
None yet

#72500 contains same bug.

cc @vnen

This PR also breaks anything using get_node or $, whether @onready is used or not.
@onready var my_node = $Button

Displays this error message despite using @onready
Parse Error: The default value is using "$" which won't return nodes in the scene tree before "_ready()" is called. Use the "@onready" annotation to solve this. (Warning treated as error.)

Thanks for finding this so quickly. I didn't have the time to test very thoroughly before sending the PR. I'll try to add these to the test cases when submitting a new version.


Pick a better default for offload_state_dict

After seeing many issues of users struggling with the offload_state_dict argument, I've decided to let it default to None and pick a default depending on the computed device map (True if there are weights offloaded to disk, False otherwise).
The same is being implemented in Transformers here.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


How to handle 2FA?

How to handle 2 factor authentication?

I have a solution for this, but the code is in a hacked-together state now. After I've cleaned it up & tested it, I'll open a pull request.
Also, it doesn't support SMS based 2FA.

Please link #169


Refactor roles

We were frequently reaching for the Role name (e.g. "admin") to decide whether a User with that role could do something. This PR replaces these string comparisons by using the Roles::<class> objects to encapsulate behaviour associated with each role.
Summary of the commits:

Replace hardcoded role strings (e.g. "normal") with the name defined in the relevant Roles::<class>
Use role predicate methods (e.g. User#admin?) where possible
Avoid hardcoding role strings in Roles::<class>.manageable_roles
Move instance methods defined in Roles::ClassMethods to module methods in Roles allowing me to remove Roles::ClassMethods.
Add Roles.find to allow us to lookup a role based on its name
Add Roles::<class>.require_2sv? to make roles responsible for knowing whether they require 2sv
Add Roles::<class>.display_name to consistently display roles in a human friendly format
Return Roles::<class> from User#role

The intention is that this is a refactoring PR and that none of the behaviour of the system should have changed.
There are lots of commits but they are all relatively small and the commit notes should (hopefully) explain the intention.
Possible future ideas:

It's not currently clear where methods relating to User#role should be defined (e.g. #admin? is defined in Roles while #role_display_name is defined in User). I think it would be an improvement if all methods that relied on User#role were defined in the same place.
Rename the users.role database column to something like role_key to indicate that it's essentially a foreign key used to lookup the corresponding Roles::<class>.

I think this would allow us to introduce a User#role= method that took a Roles::<class> and stored its name in User#role_key.

Make use of something like Whitehall's ActiveRecordLikeInterface to make the current Roles::<class> objects into instances of classes

One thing I was a bit confused about was why Roles::Normal.manageable_roles needs to be a special case, i.e. why can't it rely on the level-based logic like the other classes...?

It's because the level-based logic would mean that Roles::Normal.manageable_roles would return [Roles::Normal], implying that normal users could manage other normal users.

Thanks for reviewing, @floehopper!
I've addressed each of your comments, made some changes based on your feedback and am now getting it merged.


Transferring subscriptions to another reseller doesn't work

Hi all,
I am making an application where it should be possible to transfer existing subscriptions to another reseller.
First of all, I request the available subscriptions based on the main domain. Then I pass the subscription, which is a Subscription object, via the insert method.
Then, no matter what I try, I get a 400 error: Licensed number of seats field is read-only. I get this message when transferring 1 subscription as well as several in a batch.
Does anyone know where I should look to resolve this?
Environment details

OS: MacOS
PHP version: 8.1
Package name and version: google/apiclient v2.12

Steps to reproduce

Get subscriptions by domain
Transfer a single subscription to another reseller account

Code examples
Step 1: Getting subscriptions by domain.

$google = new Reseller($this->client);
$data = $google->subscriptions->listSubscriptions([
    'customerId' => $domain,
    'customerAuthToken' => $token,
])->getSubscriptions();

Step 2: Getting subscriptions by domain.

$google->subscriptions->insert($domain, $subscription, [
    'customerAuthToken' => $token,
]);

Subscription object (replaced some sensitive values with #)

Google\Service\Reseller\Subscription {#1859 ▼
  #internal_gapi_mappings: []
  #modelData: []
  #processed: []
  #collection_key: "suspensionReasons"
  +billingMethod: "ONLINE"
  +creationTime: "1658294337048"
  +customerDomain: "######.nl"
  +customerId: "######"
  +dealCode: null
  +kind: "reseller#subscription"
  #planType: "Google\Service\Reseller\SubscriptionPlan"
  #planDataType: ""
  +purchaseOrderId: null
  #renewalSettingsType: "Google\Service\Reseller\RenewalSettings"
  #renewalSettingsDataType: ""
  +resourceUiUrl: "https://admin.google.com/ac/billing/subscriptions?ecid=######"
  #seatsType: "Google\Service\Reseller\Seats"
  #seatsDataType: ""
  +skuId: "1010020027"
  +skuName: "Google Workspace Business Starter"
  +status: "ACTIVE"
  +subscriptionId: "4088233743"
  +suspensionReasons: null
  #transferInfoType: "Google\Service\Reseller\SubscriptionTransferInfo"
  #transferInfoDataType: ""
  #trialSettingsType: "Google\Service\Reseller\SubscriptionTrialSettings"
  #trialSettingsDataType: ""
  +"seats": Google\Service\Reseller\Seats {#1842 ▼
    #internal_gapi_mappings: []
    #modelData: []
    #processed: []
    +kind: "subscriptions#seats"
    +licensedNumberOfSeats: 5
    +maximumNumberOfSeats: 300
    +numberOfSeats: null
  }
  +"plan": Google\Service\Reseller\SubscriptionPlan {#1835 ▼
    #internal_gapi_mappings: []
    #modelData: []
    #processed: []
    #commitmentIntervalType: "Google\Service\Reseller\SubscriptionPlanCommitmentInterval"
    #commitmentIntervalDataType: ""
    +isCommitmentPlan: false
    +planName: "FLEXIBLE"
  }
  +"trialSettings": Google\Service\Reseller\SubscriptionTrialSettings {#1856 ▼
    #internal_gapi_mappings: []
    #modelData: []
    #processed: []
    +isInTrial: false
    +trialEndTime: null
  }
  +"transferInfo": Google\Service\Reseller\SubscriptionTransferInfo {#1857 ▼
    #internal_gapi_mappings: []
    #modelData: []
    #processed: []
    +currentLegacySkuId: null
    +minimumTransferableSeats: 5
    +transferabilityExpirationTime: "1667885493634"
  }
}

Step 2 response:
{
   "error":{
      "code":400,
      "message":"Licensed number of seats field is read-only",
      "errors":[
         {
            "message":"Licensed number of seats field is read-only",
            "domain":"global",
            "reason":"invalid"
         }
    ]
}

Could you please let me know what are the following in your code to triage this issue?

$this->client
function description of listSubscriptions
function description of getSubscriptions
function description of insert

@ajupazhamayil all of the above are functionalities from within this library, so I don't understand why you ask this question.

@voicecode-bv Oh sorry I got wrong repository context, I was looking at one another repository which has more than one product. Thank you for the reply.

Hi @voicecode-bv, can you please paste you stack trace when you get this error?

@yash30201 Thanks for your message. It has just been resolved. Apparently, you can't fetch the subscriptions from Google and use them directly with the transfer API calls. You have init a new Subscription model and cherry pick some details from the existing subscriptions, otherwise it just wont work. Very strange behaviour if you ask me.


Assignment Rule Header - how to disable the default rule?

Hello,
From the sample code below at https://www.salesforce.com/developer/docs/api/index_Left.htm#CSHID=sforce_api_objects_lead.htm|StartTopic=Content%2Fsforce_api_objects_lead.htm|SkinName=webhelp
It is possible to instantiate an AssignmentRuleHeader object and set useDefaultRule to true or false.
Is there any way to accomplish the same using the Toolkit for .NET?
// Create the assignment rule header and add it to the proxy binding
AssignmentRuleHeader arh = new AssignmentRuleHeader();
        // In this sample we will look for a particular rule and if found
        // use the id for the lead assignment. If it is not found we will
        // instruct the call to use the current default rule. Both these
        // values can't be used together.
        QueryResult qr = binding.query("Select Id from AssignmentRule where Name = " +
            "'Mass Mail Campaign' and SobjectType = 'lead'");
        if (qr.size == 0)
        {
            arh.useDefaultRule = true;
        }
        else
        {
            arh.assignmentRuleId = qr.records[0].Id;
        }
        binding.AssignmentRuleHeaderValue = arh;

Thanks in advance!
Armand Collin

Adding some additional info on the Assignment Rule Header, I would need to set the following:
Sforce-Auto-Assign: FALSE
From:
http://www.salesforce.com/us/developer/docs/api_rest/api_rest.pdf
Page 147
Any way to accomplish that via the Toolkit for .NET?

The salesforce developer docs cannot handle with the direct links to topcis - what a fail :P If you enter first link, use search 'AssignmentRuleHeader' and select first item on the search results - you will find the sample code.
What framework/library the salesfoce c# example is using?
 public void CreateLead()
        {
            // Create a new Lead and assign various properties
            Lead lead = new Lead();

            lead.FirstName = "John";
            lead.LastName = "Brown";
            lead.Company = "ABC Corporation";
            lead.LeadSource = "Advertisement";
            // Setting the lead source for a pre-existing lead assignment rule. This
            // rule was created outside of this sample and will assign any new leads
            // that have "Advertisement" as the LeadSource to a particular user

            // Create the assignment rule header and add it to the proxy binding
            AssignmentRuleHeader arh = new AssignmentRuleHeader();

            // In this sample we will look for a particular rule and if found
            // use the id for the lead assignment. If it is not found we will
            // instruct the call to use the current default rule. Both these
            // values can't be used together.
            QueryResult qr = binding.query("Select Id from AssignmentRule where Name = " +
                "'Mass Mail Campaign' and SobjectType = 'lead'");
            if (qr.size == 0)
            {
                arh.useDefaultRule = true;
            }
            else
            {
                arh.assignmentRuleId = qr.records[0].Id;
            }
            binding.AssignmentRuleHeaderValue = arh;

            // Every operation that results in a new or updated lead will use the
            // specified rule until the header is removed from the proxy binding
            SaveResult[] sr = binding.create(new sObject[] { lead });
            foreach (SaveResult s in sr)
            {
                if (s.success)
                {
                    Console.WriteLine("Successfully created Lead with ID: {0}", s.id);
                }
                else
                {
                    Console.WriteLine("Error creating Lead: {0}", s.errors[0].message);
                }
            }

            // This call effectively removes the header. The next lead will be assigned
            // to the default lead owner.
            binding.AssignmentRuleHeaderValue = null;
        }

Back to the toolkit - I don't know yet what the 'Assigment Rule Header' is for, but Isn't the problem is, that there is no way to set custom header by the library?
For instance: The service host client
public class ServiceHttpClient : IServiceHttpClient, IDisposable
Internally is using 'System.Net.Http.HttpClient'
But this class, or other toolkit classes does not expose any api to set custom headers by the developer?

That sample is likely working of proxy generated classes from the WSDL and SOAP endpoints.
 You could even look at the classes generated from the WSDL and use them as the basis for working with the toolkit (which largely uses the REST APIs).


PR for verifying tests

Description
Changes proposed in this pull request.
Test plan
Outline the test plan used to test this change before merging it.

Fixes #.

Codecov Report

Merging #369 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #369   +/-   ##
=======================================
  Coverage   58.42%   58.42%
=======================================
  Files          51       51
  Lines        5859     5859
=======================================
  Hits         3423     3423
  Misses       2158     2158
  Partials      278      278

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 9dfcdc2...2fa9ec3. Read the comment docs.


chore: bdd-nh-1586509315 to 0.0.1

chore: Promote bdd-nh-1586509315 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


CorpNetSaw and CorpNetPublic dont exist as service tags in the portal

Currently verifying connectivity and noted that the CorpnetSaw and Corpnetpublic service tags dont exist (well arent publicly viewable or selectable)

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

ID: fc2dd547-3ff7-b32a-ba8a-c582a721a0ee
Version Independent ID: f083bdda-0c66-af32-0d27-f535e46c96dc
Content: Connectivity architecture for a managed instance - Azure SQL Database
Content Source: articles/sql-database/sql-database-managed-instance-connectivity-architecture.md
Service: sql-database
Sub-service: managed-instance
GitHub Login: @srdan-bozovic-msft
Microsoft Alias: srbozovi

@CloudTechTravel
Thanks for the Question! We are currently investigating and will get back on this.

Its available via an ARM - but not in the portal.

@CloudTechTravel Please do let us know any further help needed with this thread.

@CloudTechTravel You can add the feedback for this feature here - https://feedback.azure.com/forums/217321-sql-database
The product team will look into the same and provide guidance. For now we are going ahead and closing this thread.


SP-1823 - Backport of PDI-13434 - Pentaho Reporting Output step - Excel output formatting is different from PRD Excel output (5.3 Suite)

do not merge it now please


Allow a least privilege model of automatic dependencies by "disabling" dependencies for individulal routers or endpoints.

Issue #2481 Mentions how it is difficult to remove a global dependency on a subset of endpoint.  The proposed solution in that defect is to add global depdencies on individual APIRouter instances and combine a set of those into the app.
This PR proposes a complimentary solution.  Anywhere where a Depends(foo) or Security(bar) is added as an item in a dependencies keyword argument, it can receive a disable=True keyword argument.
Once the final parameterless dependencies are gathered and added on a route, these are then processes and collapsed.  This allows an APIRouter to disable a global dependency which has already been added on an app object.
This allows a least privilege approach to API design, where nothing is allowed by default unless explicitly allowed on a case by case basis:  The app object can have a global dependency which requires all scopes, and then individual endpoints, for example those that deal with authentication, or other less privileged actions, can remove the more stringent scopes to allow access.
Additionally, multiple stacked calls to the same Security dependency are collapsed, merging all the provided scopes into a single list.  This allows the creation of virtual scopes, for example, prefixing a scope with a "-" do disable it, handling the processing of the all the scopes to a single call.  This is yet-another way disabling a scope requirement.
This is currently an undocumented feature request / proposal, containing tests.

Thanks for the interest and effort @kristjanvalur ! ☕
This would add some significant complexity, in particular to explain it to new developers. And it would mean that adding a dependency somewhere would no longer guarantee that it would be executed everywhere down the line.
Given that I'll pass on this one, but thanks for the interest!

@AdrianSacristanPB nope. If we think about it, a least privilege model would really mean that you would add all the things that block and restrict at the top level in your app. Even if they are only related to a small section of it. And then you would have to enable access to most of those for most of the other unrelated parts.
It would also mean that to know if a section in your code is covered by some protection, you would have to check the whole chain of included routers to make sure none is disabling it. That's kinda linear complexity (O(n)) to verify if something would apply or not. This itself could be a security risk, as people could assume that if something was applied somewhere up the chain, then it would also apply here, but that's no longer the case.
So I prefer to have related things and restrictions closer to where they apply, with the related code close together.
Now, you can probably hack around using scopes, and relax permissions with scopes. I'm pretty sure it would be a quite complicated implementation (at least mentally), but maybe you could make it work.

The case I'm trying to solve is really:  "Here I want almost the same restrictions as the rest of the app, but not quite".  Doing it currently is quite inconvenient, with multiple routers.
The reverse solution is to set no rules at the top, and make sure that you apply them at every router/endpoint.  that is too an O(n) problem and you have no guarantees there either, because you have to make sure that you don't forget to apply the rules individually.
The sort of middle ground is to have multiple routers, each with a different set of rules applied to them and join them in a top app with no rules.  This works, but it can become messy and complex.
A different way to achieve the same, or similar, results is by using only Security and make sure to extend the scopes when passing down the inheritance chain.  This PR can be modified to only join up Security scopes.  This way, the application itself can decide how to interpret added scopes.  An implementor can, for example, interpret scopes prefixed with '-' as disabling them in the security dependency.  Please also see the #5622 which merges parameterless Securitiy dependences with parameter dependencies.
An additional benefit of merging Security scopes is that it reduces calling overhead in the endpoint.  if you specify Security(foo, scopes=["a"]) at the top, and Security(foo, scopes=["b"]) lower down, the endpoint receives a single invocation of foo with scopes==["a", "b"] rather than two separate calls.
I can submit an alternate PR which only does that.


Remove modular typography scale

Our system (base 20px, scale by 1.333) works fine for 20 * 1 = 20 and 20 / 1.333 = 0.75. But every other result it produces is (a) not a useful number of rems and (b) pixel-fractional. This results in rounding which means our designs can get in a mess.
Let's instead choose nice whole pixel numbers that work well together. We can make --20px variables if we like that actually emit rems. Or we can use calc(16rem / 20). Whatever ends up making sense. But let's get practical about what actually works and banish 0.56532308rem from the CSS forever!

When we do this we should think about introducing a size between h2 and reading text. Or eliminating reading text in favor of a smaller base size. Something so that a section headed by an h2 can still have structural division like the metagame grid needs (headers for each archetype). And possibly apply this to subsections of decklists (Creastures, Spells, etc) which are currently h2.

I've already started using calc(3rem / 20) type stuff in the CSS, and maybe some bare px values? So whatever we do we should bring those into line.

One of the most prominent issues here is the 1px gap on the mobile menu between the main menu and the submenu (when it shows). This is because bottom is 3.825rem /* 0.75rem (padding-top 15) + 1rem (icon font-size) * 1.33 (line-height 27) + 0.75rem (label font-size) * 1.33 (line-height 20) + 0.75rem (padding-bottom) 15 + 0 (hide the border don't accommodate it) with some slight adjustments for browser rounding. */ which comes out at 76.5px. Which Safari and mobile Chrome decide to show 1px away from where Chrome/MacOS puts it. Nonsense.


Data Explorer catalog version display on pages

AnVIL is requesting that we implement a feature that includes a reference to which collection of datasets are in the currently displayed Data Explorer and the date of indexing.
This is to be used to help the support teams quickly identify the state of the datasets in the Data Explorer.
The format that has been suggested is:  <catalog_id>-<index_date>
An example:  anvil5-2024-03-05
The actual date used to identify the time frame of the indexing is up to the Data Explorer team.
I'm filing this against the data-browser since it will be required to display the information.  It is acknowledged that modifications may need to be made to Azul to support this.  Please file the appropriate Azul tickets.

Please note that this is a lower priority request and should not delay the AnVIL Beta, General Availability, or Azure support efforts.


Show Catalog title if available

In Data consumption:
Catalog titles are not mandatory.
However it will help the user if there is a title to show it instead of the catalog ID.

Because catalog titles are missing in the info model, the UI could only display catalog IDs.
Therefore, all resources are now directly displayed.
The catalog selection could be added again as soon as catalog titles are available in the info model.


mako: init at 1.1

mako currently works on Sway 1.0 alpha.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: mako
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: mako
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Just a note that mako is not broken per se, but just useless without a compositor that supports zwlr_layer_shell_v1 protocol

Thank you for clarifying. I suggest to leave this PR open and mark it as WIP until that compositor is available. Unfortunately there seems to be no published timeline for the sway 1.0 release.

@GrahamcOfBorg build mako

Success on aarch64-linux (full log)
Attempted: mako
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1
shrinking /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1/bin/mako
gzipping man pages under /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1/bin
patching script interpreter paths in /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1
/nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1/bin/makoctl: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1...
/nix/store/9hvcyv9gm3cg41y40b6cs9vbgxfni9md-mako-1.1

Success on x86_64-linux (full log)
Attempted: mako
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1
shrinking /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1/bin/mako
gzipping man pages under /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1/bin
patching script interpreter paths in /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1
/nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1/bin/makoctl: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1...
/nix/store/p96gppkch1b5bb84s662mcdn3gdkax3y-mako-1.1

@xeji since we have sway-beta, this is probably good to go


[move-prover] Verified the local spec of libra.move

Completed specifying and verifying the local specifications (i.e., pre- and post-conditions) of libra.move

Ported the previous specifications of the existing functions:

register (previously initialize), mint, mint_with_capability (previously mint_with_default_capability), market_cap, zero, value, split, withdraw, join, deposit, destroy_zero

Added the specifications for the newly introduced functions in the Libra module:

assert_if_registered, burn, cancel_burn, new_preburn, preburn, preburn_to_sender, burn_with_capability, cancel_burn_with_capability, publish_preburn, remove_preburn, destroy_preburn, publish_mint_capability, remove_mint_capability, preburn_value

Fixed a bug in the builtin boogie model for vector: $Vector_remove, and $Vector_swap_remove in Prelude
Revised tests/README.md

Motivation
To verify the standard Libra module/contract
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra r=DavidLDill

:pushpin: Commit 53bf56d has been approved by DavidLDill

:hourglass: Testing commit 53bf56dcb6b719110f4c3e493dc41e13eee2b2c1 with merge b29561b0ce25029d2aae1308e406c74f853b2689...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: DavidLDill
Pushing b29561b0ce25029d2aae1308e406c74f853b2689 to master...


[WIP] WDS - Anvil compatible Modal Widget

Description
This PR primarily adds the modal widget to WDS.
The following changes were made:

WidgetNameCanvas now listens to modal body scrolls to position widget name components correctly
Modal Widget is rendered as a detached widget that is outside of the layout flow of the main canvas
Main container resizer now has a higher z-index to show even if the modal is open in the preview mode
Widget selection flow in Anvil layout system has been modified to be handled in a central location (AnvilMainCanvas)
Modal widget's type in modal sagas are selected via a selector that checks for the feature flag.
Modal widget has its own preset that (at present) is similar to the Main container's layout preset

PR fixes following issue(s)
Fixes #28588
Fixes #28328
Fixes #27459
Media
https://github.com/appsmithorg/appsmith/assets/103687/bf350be4-2202-49f3-a860-3e38681ab32e
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Testing

How Has This Been Tested?

[x] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

@coderabbitai generate interesting stats about this repository from git and render them as a table.

/ok-to-test

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test<EMAIL_ADDRESS>


Issue implementing persistent state - browser storage

Great project I must say, this is one of the most amazing feature that has stormed vite. Kudos Antfu and other Collaborators.
I'm actually using vite-ssg for a new project, and everything is working out fine.
But I'm curious of how to go about implementing persistent state.
In a nutshell, my store would look like this. While I'm using vue3 reactivity out of the box, and browser-storage (by Trapcode) for handling localStorage data.
import { reactive, readonly } from "vue";
import BrowserStorage from "@trapcode/browser-storage";
// Documentation here https://www.npmjs.com/package/@trapcode/browser-storage
// Initialize
const localStore = BrowserStorage.getLocalStore("namespace");

// Enable Encryption
localStore.enableBase64Encryption(import.meta.env.PROD);

const state = reactive({
    counter: localStore.getNumber("counter", null)
})

function updateCounter($value: number) {
    state.counter = $value
    localStore.setNumber("counter", state.counter);
}

export default {
    state: readonly(state),
    updateCounter
};

It's actually difficult importing the Browser Storage package since window object doesn't exist.
Please is there a better approach to implementing this?

Hello @techieoriname , I also encountered local storage problem, can you tell me how you solved it?


Can I Customize 'From Mail' and Body of the Email ?

I tried adding as below:-
var reportCrash = new ReportCrash
            {
                CurrentCulture = new CultureInfo("en-US"),
                AnalyzeWithDoctorDump = true,
                DeveloperMessage = developerMessage,
                ToEmail =<EMAIL_ADDRESS>		FromEmail =<EMAIL_ADDRESS>		SmtpHost = "smtp.gmail.com",
		UserName =<EMAIL_ADDRESS>		Password = "--------------",
                DoctorDumpSettings = new DoctorDumpSettings
                {
                    ApplicationID = new Guid("8bac08ca-904c-4622-97f6-33f505e7da5f"),
                    OpenReportInBrowser = false
                },
            };

After adding From Email also i am getting error reports (through mail) from<EMAIL_ADDRESS>@ravibpatel

@koushik077 You can't customize the Body of the email without modifying the code inside CrashReporter.NET. You can customize From Mail though. See #4.


ODataRequestRead with https://<tenant_url>/api/v1/dwc/consumption/relational/<space_id>/<asset_id>/$metadata results in 406

The documentation here says that hitting
https://<tenant_url>/api/v1/dwc/consumption/relational/<space_id>/<asset_id>/$metadata
will retrieve metadata but the following results in a 406 response.

ODataRequestRead(
"/api/v1/dwc/consumption/relational/('$spaceId')/('$assetId')/$metadata"
.    ODataResourcePath.of(""),
.    "",
ODataProtocol.V4
).execute(httpClient)`

The reason is that the JSON looks like this
{<EMAIL_ADDRESS>"XX",     "name": "XX",     "label": "XX",     "spaceName": "XX",     "assetRelationalMetadataUrl": null,     "assetRelationalDataUrl": null,     "assetAnalyticalMetadataUrl": "XX",     "assetAnalyticalDataUrl": "XX",     "supportsAnalyticalQueries": true,     "hasParameters": false }
But the code to parse the JSON here requires it to have a value node.

May I ask, what do you expect from calling GET to an OData entity /$metadata endpoint?


Enable loop unrolling for GCC

LLVM enables loop unrolling by default at -O2 optimization level but
GCC does not enable it by default. This enables it for GCC to allow
fairer comparison.

Great, thanks!


translate.use(langCode) is not working in some cases

Current behavior

In app.component.ts I have set up the TranslateService as follows:
translate.addLangs(['en', 'bn']);
translate.use('en');

translate.onLangChange.subscribe((change) => {
      console.log('langChange', change);
})

In another module, I'm doing this to change the current language,
this.translate.use(langCode);

When I change the language to bn, it does not work.
BUT, if I write translate.use('bn'); in app.component.ts, language change starts working.
What I've noticed is that, when I set en as the language in app.component.ts the onLangChange prints an empty object as the translations property.
// in onLangChange callback
Object { lang: "bn", translations: {} }

Expected behavior

Changing language should work regardless of what language was set during initialization
How do you think that we should fix this?

Minimal reproduction of the problem with instructions

Environment

ngx-translate version: 13.0.0
Angular version: 10.1.5

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ x] Firefox version 81
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: 12.16
- Platform:   Windows

Others:

This workaround works for me (angular 11 et ionic 5) :
I force to load all possible language in app.components :
this.translate.addLangs(['fr', 'en', 'es', 'de'])
this.translate.use('en')
this.translate.use('es')
this.translate.use('de')
this.translate.use('fr')
then in other component, i can call again this.translate.use('xxx')
This issue was not present in previous version for angular 8

This workaround works for me (angular 11 et ionic 5) :
I force to load all possible language in app.components :
this.translate.addLangs(['fr', 'en', 'es', 'de'])
this.translate.use('en')
this.translate.use('es')
this.translate.use('de')
this.translate.use('fr')
then in other component, i can call again this.translate.use('xxx')
This issue was not present in previous version for angular 8

Same problem for me, the workaround of @sguilly worked. But this wasn't a problem previously.

Same problem for me, the workaround of @sguilly worked. But this wasn't a problem previously.

Hi guys! I also noticed this "issue". After a few attempts, I got it by using same loader config in forChild as in forRoot:
app.module.ts (can be core.module or whatever):
export const HttpLoaderFactory = (httpClient: HttpClient) => {
  return new ...
};

@NgModule({
  declarations: [AppComponent],
  entryComponents: [],
  imports: [
    ...,
    TranslateModule.forRoot({
      loader: {
        provide: TranslateLoader,
        useFactory: HttpLoaderFactory,
        deps: [HttpClient],
      },
    }),
  ],
  providers: [
    { provide: RouteReuseStrategy, useClass: IonicRouteStrategy },
  ],
  bootstrap: [AppComponent],
})
export class AppModule {}

app.component.ts:
export class AppComponent {
  constructor(private readonly translateService: TranslateService) {
    this.setDefaultLang();
  }

  private setDefaultLang() {
    this.translateService.addLangs(['en', 'fr', 'es-ES', 'es-419']); // array of available langs
    this.translateService.setDefaultLang('es-ES');
  }
}

lang-test-module.ts (the module from where we want to change lang):
import { HttpClient } from '@angular/common/http';
import { TranslateLoader, TranslateModule } from '@ngx-translate/core';
import { HttpLoaderFactory } from 'src/app/app.module.ts';
...
@NgModule({
  imports: [
    ...,
    TranslateModule.forChild({
      loader: {
        provide: TranslateLoader,
        useFactory: HttpLoaderFactory,
        deps: [HttpClient],
      },
    }),
  ],
  declarations: [LangTestPage],
})
export class LangTestModule {}

I don't know if it is a good solution...

@joseph-navant : your tip don't work in my code

@joseph-navant : your tip don't work in my code

Can I help you?

I have the TranslateModule.forRoot() (with loader) in my core.module (core.module is imported in app.module)
My HttpLoaderFactory is using MultiTranslateHttpLoader (https://github.com/denniske/ngx-translate-multi-http-loader)
I added available langs and default lang in app.component-ts (as u can see in the code above)
I have the TranslateModule.forChild() (with loader) in the module from I want to change lang (this module is lazy loaded)
The HttpLoaderFactory i'm using in this module is imported from my core.module (can be app.module or whatever)
In LangTestPage I get available langs and render it in radio-buttons
When I click in one of them, it calls use() from translateService and voilá

lang-test.page.ts
import { Component, OnInit } from '@angular/core';
import { TranslateService } from '@ngx-translate/core';

@Component({
  selector: 'app-lang-test',
  templateUrl: './lang-test.page.html',
  styleUrls: ['./lang-test.page.scss'],
})
export class LangTestPage implements OnInit {
  langs: string[];
  selectedLang: string;

  constructor(private readonly translateService: TranslateService) {}

  ngOnInit() {
    this.langs = this.translateService.getLangs();
    this.selectedLang =
      this.translateService.currentLang || this.translateService.defaultLang;
  }

  onLanguageChange() {
    this.translateService.use(this.selectedLang);
  }
}

lang-test.page.html (using ionic framework)
<ion-radio-group [(ngModel)]="selectedLang" (ionChange)="onLanguageChange()">
  <ng-container *ngFor="let lang of langs">
    <ion-item mode="md">
      <ion-radio [value]="lang"></ion-radio>
      <span>{{ 'SETTINGS.LANGUAGE.' + lang.toUpperCase() | translate }}</span>
    </ion-item>
  </ng-container>
</ion-radio-group>

When this code is rendered, every ion-radio value has the lang code I defined in app.component.ts. Something like this:
...
<ion-radio value="en"></ion-radio>
...
<ion-radio value="fr"></ion-radio>
...
<ion-radio value="es-ES"></ion-radio>
...
<ion-radio value="es-419"></ion-radio>
...

So, after all, the use from translateService is receiving the lang code. For example: this.translateService.use('en')

Thanks @joseph-navant, your solution worked for me.

If you use TranslateModule.forChild(), you get the default FakeTranslateLoader per here: https://github.com/ngx-translate/core/blob/ce3c70c76577550718417912516136da51922c42/projects/ngx-translate/core/src/public_api.ts#L72)
So for me, I changed my non-root modules to just be TranslateModule and the languages started to load as expected (same behavior as before.

I just came across this issue myself.
In my case, my language is set elsewhere by some other javascript so doing translateService.use inside the app module constructor is not an option because the language is not set yet.
Instead I want to do it inside the AppInitializer where my code loads the language from the backend.
Doing translate.use correctly downloads the language (I can see it in network) but all translations appear blank.
Not sure what to do, it feels very much like a bug.

I ran some more tests, it seems that just the simple act of injecting the translate service into the angular module - for whatever reason - fixes everything. Even if it is not used.
Even though I am using the injector to inject the service in the AppInitializer (which runs after the AppModule construction), it somehow initializes the service differently I guess. If anyone can explain why that is, I would be interested to learn.
So the test cases are as follows. In all cases my APP_INITIALIZER factory is the place that has an injector and sets the translate.use(X). What affects if this works or not is how AppModule constructor is setup.

TranslateService in constructor - everything works.
Injector in constructor, then injector.get(TranslateService) - does not work
Injector in constructor. then injector.get(TranslateService).use("") - does work
Nothing in constructor, nothing works.

I don't understand why injecting TranslateService directly or via injector then using it makes a difference.

Ah, it works on some reloads. So I think perhaps the bug is that you cannot set a new use language before it has loaded the default language?
I am not sure about where, but it is definately a race condition problem somehow.

Ok I found the issue.
For some unknown reason, if you try to load angular locale data (or do any import maybe?) and at the same time try to do a .use then the use will only work (properly show the translations instead of blanks) IF the use completes before the loading of the import is complete.


Outsource .gitignore parsing and fix parsing

Sorry, I've accidentally hit ctrl + return too early.

First, this PR fixes an working directory issue: globby returns paths relative to opts.cwd, but parseGitignore(…) does not use opts.cwd:
const getGitIgnores = opts => globby
	.sync('**/.gitignore', {
		ignore: opts.ignores || [],
		cwd: opts.cwd || process.cwd()
	})
	.map(pathToGitignore => {
		const patterns = parseGitignore(pathToGitignore); // does not use opts.cwd(!)

Thus, globby(…, {cwd: 'fixtures'}) may simply return '.gitignore', and therefore parseGitignore(…) will parse '.gitignore' instead of 'fixtures/.gitignore'.

Secondly, this PR fixes a Windows issue at:
const base = path.dirname(pathToGitignore);

return patterns
	.map(pattern => {
		const negate = !pattern.startsWith('!');
		const patternPath = negate ? pattern : pattern.substr(1);
		return {negate, pattern: path.join(base, patternPath)};
	})

path.dirname and path.join were used in an unfortunate way, thus pattern became a win32 styled path (i.e. using \\ instead of /), which node-glob does not support (see here for details).

@sindresorhus Okay, I believe it's ready to review.
// cc @marionebl @JonnyBurger

@JonnyBurger, @marionebl: It turned out that sorting is not need at all.
@sindresorhus: The test are failing because of https://github.com/jonschlinkert/parse-gitignore/pull/3 (parse-gitignore erroneously sorts .gitignore patterns).

@sindresorhus Okay, parse-gitignore issues and Windows issues are fixed – LGTM.
// cc @marionebl @JonnyBurger

@schnittstabil Could you outline why you chose to mix in the refactoring to use a class?

Closing in favor of https://github.com/sindresorhus/xo/pull/204. Really appreciate the work you did here @schnittstabil, even if we went in a slightly different direction :)

No problem, I've created a separated PR (#217) for the remaining changes.


OpenSSL: Visual Studio 2015 support

Current coverage is 49.20%

Merging #1334 into develop will not affect coverage as of edadc64

@@            develop   #1334   diff @@
=======================================
  Files           958     958
  Stmts         65608   65608
  Branches          0       0
  Methods           0       0
=======================================
- Hit           32282   32281     -1
  Partial           0       0
- Missed        33326   33327     +1

Review entire Coverage Diff as of edadc64

Powered by Codecov. Updated on successful CI builds.

:+1:


Remove "strip tracking parameters" option from Stealth Mode

This functionality can now be implemented as a filter list so product devs can simply enable that filter list and that's all.
Also, please do the following:

Grab the default parameters list and post it here.
We'll ask @AdguardTeam/filters-maintainers to create a new filter list for that.

@AdguardTeam/filters-maintainers what name would you like for that new list?

In fact, such a task has already been created (see https://github.com/AdguardTeam/AdguardFilters/issues/57197)

@sxgunchenko let's at least remove default hardcoded parameters as a part of this task then.

Actually, they have already been moved to the app side. There is nothing left in the CoreLibs related to the tracking parameters except the $removeparam rules processing.


CARDS-2424 and CARDS-2402: New Phone Call Follow-Up and replacement Event - Clinical Cardiac forms

Remove highlighted text - this is automatically done in the backend, don't need to show on UI

Add event type abbreviation [ex. (HF), (VHD), (CV Other)] to the Subtype question - match formatting for "Coronary Artery Disease (CAD) Subtype"

Add "the" it should read: "Why did the patient not provide consent?"

This question should be "How long was the hospital stay" not "How many days ago" this can be interpreted differently so let's stick to the specs. Also, can we add a warning if [below 1 or above 365] on the spot similar to questions in the Admin section? Right now I can add anything outside this range without a warning and save - it only show "invalid" after saving

Add text "Diagnostic Test Date" to question & remove "event date relative to enrollment or study start" as this is automatically done in the backend

Missing an answer option of "Troponin normal" under CV Other & Afib should be formatted to AFib

Remove highlighted text - this is automatically done in the backend, don't need to show on UI

Missing answer options of:

Volume overload improved
Volume overload unchanged
Volume overload worsened


Client bind interface

local_address in config.json is ignored, as is the command line option -b="" (for bind I guess).

In client, local_address in config.json is ignored, while command line option -b is respected.
What problem do you have?

Oh yeah I'm sorry, -b works. The tool I used resolved <IP_ADDRESS> to the host name so I wrongly assumed it was not bound to localhost.


WSL1 places VPN-provided DNS servers last in /etc/resolv.conf

Please fill out the below information:

Your Windows build number:

10.0.18362.535
running ubuntu-18.04

What you're doing and what's happening:

On connecting to a VPN, one VPN nameserver is being added to the end of /etc/resolv.conf

What's wrong / what should be happening instead:

The VPN's nameserver should be added at the top of /etc/resolv.conf

This is particularly pertinent with Palo Alto GlobalProtect VPN, as the Windows client by default blocks queries to DNS servers not pushed out by the VPN, completely breaking name resolution.


Feature Request: scalar-valued user-defined functions

These would be similar to the existing table-valued functions, but would return scalar values instead.

Added in #496
Should be released in v0.11 within the next day or two.


Arrow tip moving down when updating tooltip element dynamically

My tooltipster content is a div and contentCloning is set to false.
After showing the popover for a few seconds, I fetch more data from the backend and add to the div. This causes the div's height to grow, and the arrow tip to move down.
How can I fix this?

Reposition it.

tooltipster("reposition") worked like a charm. Thanks!

You're welcome !


highlight in vue3 have bug

Describe the bug
in vue3,use the code doesn't work
import codeSyntaxHighlight from '@toast-ui/editor-plugin-code-syntax-highlight';

need replace to:
import codeSyntaxHighlight from '@toast-ui/editor-plugin-code-syntax-highlight/dist/toastui-editor-plugin-code-syntax-highlight-all';

but typescript have bug, and the php code nowrap

I'll check it.


MassTransit-project.com's download link

The download link is useless, since they removed the old downloads from the repos. Should likely just point it to the build in TeamCity.
I think we should just move the wordpress site to a GitHub pages site. Apparently I can't edit the page in wordpress, I hit save and nothing happens. It's really just a static landing page anyways. The other stuff is all out dated I think anyways. Thoughts @drusellers, @phatboyg?

Looks great.....

Now the real site, thanks Travis!


[Sozo] Output block # after successful migration

When migrating world, it would be useful to show the block number the world contract was deployed on. Useful to have when starting torii with --start-block. Currently using another cli to get this value prior to migration.
Perhaps something like this?
🎉 Successfully migrated World on block #1337 at address 0x788f5fd335d29ed5f8686982079cc3aa9c82aa41968f759b2c3d0be8d5fa0c4

Great idea! @aymericdelab and I had the exact same thought yesterday

Ugh commented on the wrong issue. Meant to say it here -
Ah just occurred to me, perhaps an enhancement to sozo migrate is updating the manifest to include block_height and rpc_url after migration, this way torii just needs the manifest param to start indexing.
manifest.json
"rpc_url" : "https://rinnegan.madara.zone",
"world": {
"name": "world",
"address": "0x123",
"block_height": "1337"
"class_hash": "0x7f07c1861ee2df73f81685cdda7e51978ef0f50154a6a8c11e4af6b55d08ebf"
},

i can give this a try

@broody do you mean manifest.json generated by sozo inside ./target/dev folder? I can see world object in that file but not rpc_url

Thanks! I believe @tarrencev had some thoughts about manifest.json, could possibly address in another issue. Being able to output block height on just sozo migrate would be great for now.

@broody do you mean manifest.json generated by sozo inside ./target/dev folder? I can see world object in that file but not rpc_url.

Right, I had added that as an example to illustrate.

Also is the block_height only needed for world contract or all contracts present in that file?

From torii perspective, we just need it for world contract, but perhaps good idea to include for all.

Thanks! I believe @tarrencev had some thoughts about manifest.json, could possibly address in another issue. Being able to output block height on just sozo migrate would be great for now.

Okay then i will clean up the linked PR and change from draft to ready for review tonight

@tarrencev i am not really sure what kind of information would be useful, i can think of block_number  (as requested by @broody which already has a usecase),  block_hash, transaction_hash (in which particular contract was deployed). But not sure what other two would be used for.
Is there anything else that might be helpful?

currently schema is somewhat like this (have remove nested parts):
{
  "world": {
    "name": "$name",
    "address": "$ADDRESS",
    "class_hash": "$ADDRESS"
  },
  "executor": {
    "name": "$name",
    "address": "$ADDRESS",
    "class_hash": "$ADDRESS"
  },
  "systems": [
    {
      "name": "$name",
      "inputs": [],
      "outputs": [],
      "class_hash": "$ADDRESS",
      "dependencies": []
    }
  ],
  "contracts": [
    {
      "name": "$name",
      "class_hash": "$ADDRESS",
      "address": "$ADDRESS"
    }
  ],
  "components": [
    {
      "name": "$name",
      "members": [],
      "class_hash": "$ADDRESS"
    }
  ]
}

but i think there was a PR on sozo allowing it to deploy multiple world contracts with name as seed but do we care about old world contract after new one is deployed? same question for executor can there be multiple executor deployed and do we need to store old one in manifest after new is deployed.
here is the struct that holds Manifest file currently: https://github.com/dojoengine/dojo/blob/778ca0453639ccbf293bb6a7b0f568fb4838fea1/crates/dojo-world/src/manifest.rs#L88-L94

currently schema is somewhat like this (have remove nested parts):
i think there was a PR on sozo allowing it to deploy multiple world contracts with name as seed but do we care about old world contract after new one is deployed? same question for executor can there be multiple executor deployed and do we need to store old one in manifest after new is deployed.
here is the struct that holds Manifest file currently:
https://github.com/dojoengine/dojo/blob/778ca0453639ccbf293bb6a7b0f568fb4838fea1/crates/dojo-world/src/manifest.rs#L88-L94

I think that is a good question. Should a manifest be per chain deployment? In that case, a manifest corresponds to a particular chain. That probably makes most sense? Then we can track a single deployment per manifest

as discussed with @tarrencev manifest file related changes can be handled in a separate issue, will open one with more details.
I have clean up the linked PR for this issue and marked it for review.


Channel (EPOLL) gets closed before response is consumed #2

Netty 4.1.0.Final-SNAPSHOT
I have an unit test which I originally created for #3709 and it has started failing as of 04/04/16 (Monday). It's possible that it's related to #5061 (got merged on Friday and we pulled the latest nightly on Monday).
I'm still looking but the summary of the unit test looks like this:

Netty HTTP client
Auto-Reading is turned off
A handler that calls ctx#read() as long as !(msg instanceof LastHttpContent)
Jetty HTTP server

The test issues 50 requests in a loop (one at a time) and waits for the response. Each request opens a new connection and Jetty is explicitly flushing the response and closing the connection. The test fails if no response arrives in ~5 seconds or so. Everything runs on localhost.
It's occasionally failing if EPOLL is being used. It never fails if NIO is being used. It also appears to never fail in EPOLL or NIO if I turn auto reading on from the get go (and change the handler a little bit to accommodate for it).

@rkapsi can you check if https://github.com/netty/netty/pull/5067 fixes it ?

Checking...

@normanmaurer nope, the problems seems actually more persistent.

@Scottmitch can you check ?

yes will check

I can probably turn my test into a self-contained standalone version if needed.

@rkapsi that would help a lot !

yes please :+1:

@normanmaurer @Scottmitch please see https://github.com/Squarespace/netty-issue-5082

@rkapsi - thanks! looking now.

@rkapsi - Thanks again for the unit test! I found the issue:

The initial read for the channel never gets done because by the time it executes the channel is not active [1]
The channel is not active because it is closed [2] due to a RDHUP event we receive.
We call epollInReady before we close [3] but return early because readPending == false [4].

I pushed some new code to #5067 to address the issue ... can you please verify?
[1] https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/AbstractChannel.java#L768
[2] https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/java/io/netty/channel/epoll/AbstractEpollChannel.java#L377
[3] https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/java/io/netty/channel/epoll/AbstractEpollChannel.java#L370
[4] https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/java/io/netty/channel/epoll/AbstractEpollStreamChannel.java#L826

looks like github is experiencing some issues https://status.github.com/ ... doesn't look like the push operation made it to github's servers...

13:54 PDTGitHub.com is unavailable while we resolve an issue with one of our backend services.


Models.py is missing

Nice work! Thanks for sharing your deploying code.
For gpu tensorrt, models.py containing CLIP, VAE, and UNet is missing.  The models.py from the official Tensorrt GitHub did not work.
Here, requiring the models.py.

+1


Quickstarts School Timetabling throws error when running

I recently built a wheel from the latest commit of optapy and installed it into the virtual environment in optapy-quickstarts.  However, when I try to run the school-timetabling example, I get the following error:
16:55:13.418 [main        ] INFO  Solving started: time spent (113), best score (-38init/0hard/0soft), environment mode (REPRODUCIBLE), move thread count (NONE), random (JDK with seed 0).
Traceback (most recent call last):
  File "DefaultSolver.java", line 193, in org.optaplanner.core.impl.solver.DefaultSolver.solve
java.lang.java.lang.ClassCastException: java.lang.ClassCastException: class org.optaplanner.jpyinterpreter.types.PythonNone cannot be cast to class org.jpyinterpreter.user.domain.Timeslot$$2 (org.optaplanner.jpyinterpreter.types.PythonNone is in unnamed module of loader 'app'; org.jpyinterpreter.user.domain.Timeslot$$2 is in unnamed module of loader 'OptaPlanner Gizmo Python Bytecode ClassLoader' @3632be31)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "DefaultSolver.java", line 193, in org.optaplanner.core.impl.solver.DefaultSolver.solve
Exception: Java Exception

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/user/workspaces/optapy-quickstarts/.venv/lib/python3.10/site-packages/optapy/optaplanner_api_wrappers.py", line 419, in solve
    return _unwrap_java_object(self._java_solve(wrapped_problem))
java.lang.java.lang.IllegalStateException: java.lang.IllegalStateException: The property (timeslot) setterMethod (public void org.jpyinterpreter.user.domain.Lesson$$2.setTimeslot(org.jpyinterpreter.user.domain.Timeslot$$2)) on bean of class (class org.jpyinterpreter.user.domain.Lesson$$2) throws an exception for value (null).

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/user/workspaces/optapy-quickstarts/school-timetabling/main.py", line 80, in <module>
    solution = solver.solve(generate_problem())
  File "/Users/user/workspaces/optapy-quickstarts/.venv/lib/python3.10/site-packages/optapy/optaplanner_api_wrappers.py", line 426, in solve
    raise RuntimeError(error_message) from e
RuntimeError: An error occurred during solving. This can occur when functions take the wrong number of parameters (ex: a setter that does not take exactly one parameter) or by a function returning an incompatible return type (ex: returning a str in a filter, which expects a bool). This can also occur when an exception is raised when evaluating constraints/getters/setters.

I'll enumerate my procedure, as I may be making a mistake with building the wheel and installing into optapy-quickstarts which produces this error.

Clone optapy. From the optapy root directory, ./mvnw clean install and python -m build
Copy optapy-8.31.1b0-py3-none-any.whl to optapy-quickstarts/school-timetabling directory
From optapy-quickstarts/school-timetabling, run pip install optapy-8.31.1b0-py3-none-any.whl then python main.py

This is related to https://github.com/optapy/optapy/pull/152 , although it appears it passing None although NoneType is not assignable to Timeslot; changing the type annotation on Lesson to:
    timeslot: Optional[Timeslot]
    room: Optional[Room]

avoids the issue; my guess it is because the Timeslot and Room are set to None in the constructor (despite not being assignable to None from type annotations)

So the Optional[] typing is required because (1) this is a non-nullable planning variable and (2) it's initialized to None?  It seems like that's a very normal initialization for most under-constrained planning problems.
Would you mind clarifying how typing is used and the assumptions that jpyinterpreter has regarding None?  I couldn't find anything in the documentation about it and it appears to be essential to correct function even for non-decorated funtions/classes.  Are all typing declarations used literally by jpyinterpreter?

Typing declarations are assumed to be correct by jpyinterpreter, and are used to change generic method calls (which have to go through getattribute lookups) into specific method calls (this significantly increase speed, for school timetabling, without the Optional, it three times faster because of the specific method calls). That being said, planning variables should be allowed to take the value None even if they are not annotated Optional (this assumes simple getters and setters, which changes it from setting it to None to deleting the attribute).
This is due to how Python None works, it cannot be treated as null (i.e. isinstance(None, x) is false for all x except NoneType = type(None); whereas isinstance(null, x) is true for all x in Java; this means we cannot assign None to attributes that are not Optional).

Fixed by https://github.com/optapy/optapy/pull/156 ; ultimately, the issue was caused because the attributes of the class were typed, and the setters were not. This made optapy think None was assignable to the field, when really it was not.

Thank you once again.  I appreciate the responsiveness and clarity.


Add alias for npm global install

It should go above line 4, the comments above don't apply to this alias. Other than that, :+1:

@mcornella better ?

Yes, very :smile:
/cc @robbyrussell


Export ResizeMode and Priority types

Thanks for this awesome library.
I think that having access to those types are important so users can make wrappers and accept them on their own components maintaining type safety.
I myself am currently needing this.

LGTM!


'Type aliases can be used only in a .ts file' error popping up in Flow enabled files

From @gcazaciuc on November 8, 2016 13:18

VSCode Version:
1.7.1
OS Version:
Ubuntu 16

Steps to Reproduce:
Create a .js file and add the following code in it(the file will be processed by Flowtype):
// @flow
type LogActionsType = { type: 'NEW_LOG', payload: Array<Object> } | { type: 'LOGS_PUSHED', payload: null };
export type LogType = Object

Whenever I define a type alias the error 'Type aliases can be used only in a .ts file'  keeps showing up and is highlighed in the 'Problems' panel.
Is there any way to disable typescript ? Or prevent this kind of validation of running ?
Copied from original issue: Microsoft/vscode#15171

From @lee-fu on November 8, 2016 15:18
Got the same issue:

VSCode Version: Code 1.7.1 (02611b40b24c9df2726ad8b33f5ef5f67ac30b44, 2016-11-03T13:46:00.226Z)
OS Version: Darwin x64 15.6.0

// @Flow annotation at top of file.
and javascript.validate.enable": false in my Workspace settings.
Error highlighted:
[js] 'types' can only be used in a .ts file.
[Flow]
string: (object: Object) => Array<string>

Confirmed on latest bits. Moving to TS Server repo.

Moving back to  VS Code


antigen update doesn't work

Description
antigen update fails with a 1 exit status code.
Steps to reproduce
1 - Open terminal
2 - Run `antigen update`
3 - `Updating @master... Error! Activate logging and try again.` is displayed

Expected behavior:
- No error message is displayed and all bundles get updated accordingly.

Software version

antigen version:

Antigen develop (d1dd78b)
Revision date: 2018-01-15 14:37:21 -0300

zsh --version: zsh 5.7.1 (x86_64-pc-linux-gnu)
uname -a: Linux pensieve 5.0.7-1-MANJARO #1 SMP PREEMPT Sat Apr 6 08:50:27 UTC 2019 x86_64 GNU/Linux

Configuration

gist of .antigenrc: no such file
gist of .zshrc: https://git.io/fjmGf

More information

debug.log

Same issue here, but only with 3 of my 5 Zsh plugins, I'll paste here the output I got.
antigen update
Updating<EMAIL_ADDRESS>Error! Activate logging and try again.
Updating<EMAIL_ADDRESS>Done. Took 1s.
Updating<EMAIL_ADDRESS>Error! Activate logging and try again.
Updating<EMAIL_ADDRESS>Done. Took 1s.
Updating<EMAIL_ADDRESS>Error! Activate logging and try again.

On my laptop/ultrabook, I got no issues and it updates the 5 flawlessly, as it should.


Zelda version one is missing a Korok seed Location - Fix Urgently

Hyrule location is missing. Fix urgently

The location is added


Fix SendInBackground for iOS

Issue:
During the creation of the RaygunEnvironmentMessage on iOS some fields require that they are accessed on the Main/UI thread. Previously, the provider was silently failing on this and the crash report would never be send to Raygun. This would also cause an infinite loop if ThrowOnError was enabled.
Fix:
Construct the RaygunEnvironmentMessage on the Main/UI thread and then hand back to the thread the SendInBackground call is using.
Testing:

Confirmed <= v1.4.1 does not send crash report on iOS when using SendInBackground
Applied MainThread.InvokeOnMainThreadAsync change
Confirmed that SendInBackground works on iOS, additionally, tested that it still works on Android and other platforms

Testing of the new changes show that the ILogger spam issue goes away and the SendInBackground on iOS works


change classpath for commons-csv and commons-math3

I try to use newer version of apache-commons-csv and apache-commons-math3 build with xmvn instead of tetra.
This change the classpath of the jars which needs to adapted

Obsoleted by the merge of https://github.com/uyuni-project/subscription-matcher/pull/39


Cannot translate into llvm correctly

Hi guys,
I tried to use mcsema to translate binutils/elfedit into llvm, but I found that mcsema translates the following instruction:
mov eax, large gs:14h

as
%117 = load i32* inttoptr (i32 20 to i32*), !mcsema_real_eip !13

which is then recompiled as
mov eax, large ds:14h

Hrm, so this is probably a problem with LLVM's disassembling of memory accesses through segment registers. I've gone and transliterated your instruction down into a fossil fuel, and run through llvm-mc, and we can see the MCInst info here:
pag@sloth:~/Code$ echo "mov %gs:(14), %eax" | llvm-mc-3.9 -show-inst
	.text
	movl	%gs:14, %eax            # <MCInst #1677 MOV32rm
                                        #  <MCOperand Reg:19>
                                        #  <MCOperand Reg:0>
                                        #  <MCOperand Imm:1>
                                        #  <MCOperand Reg:0>
                                        #  <MCOperand Imm:14>
                                        #  <MCOperand Reg:33>>

I'd like to see you go into the instruction implementation of MOV32rm or something like that, and see what's missing in terms of the handling of the gs register.
The correct way for us to handle it down the line is to set the pointer address space on the memop to something like 255 or 256. This may involve adding an extra argument to the functions used to create the load and store instructions.
Please get back to me with your recommendations, or possible a PR, and we can work through integrating it.

I tried to debug with mcsema and found that in function disInstrX86 the getOpcode of mov eax, large gs:14h returns X86::MOV32o32a not the expected X86::MOV32rm. And this mov instruction has no has_imm_reference and 'has_mem_reference'. So finally the translate_MOVoa performs the following handles.
Value *addrv = CONST_V<width>(block, OP(0).getImm());
ret = doRMMov<width>(ip, block, addrv, MCOperand::CreateReg(eaxReg)) ;

And the addrv has the value of 0x14, which results in the segment fault when executing the recompiled native code.

@Epeius does this issue still happen in the llvm38 branch?


Disconnect wallet still show "Swap" instead "Connect Wallet" in button

Hi LIFI Team
I am trying to integrate widget with using my Wallet management in walletConfig.
When I disconnect wallet and set walletManagement.signer = undefined but button still show "Swap" instead "Connect Wallet".
I think has logic issue at WalletProvider
https://github.com/lifinance/widget/blob/dev/packages/widget/src/providers/WalletProvider/WalletProvider.tsx
// keep account information up to date
  useEffect(() => {
    const updateAccount = async () => {
      const account = await extractAccountFromSigner(
        walletManagement?.signer ?? signer, // I think logic here is issue
      );
      setAccount(account);
    };
    updateAccount();
  }, [signer, walletManagement]);

*signer is getting from lifiWalletManagement not from my walletManagement.signer so that why account.isActive = true => It show "Swap" instead "Connect wallet" in button
Could you help to check? thanks

Hey There!
Thanks for notifying us on that issue. We will look into that ASAP and ping back here for any updates.

Hi @Addminus. version 1.15.1 is fixed the issue.it is working now. Thanks

you have the issue on your current page  https://jumper.exchange/swap too.

@Addminus please take a look.

The fix was released and deployed 🙂


#164829209 implement commenting on articles feature

Description
As a User:
I want to be able to make a comment(s) on articles, both authored by me and those not authored by me.
Type of change

[ ] Bug fix (nonbreaking change which fixes an issue)
[x] New feature (nonbreaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes

[x] End to end
[x] Integration

Checklist:

[x] My code follows the style guide for this project
[x] I have linted my code prior to submission
[ ]  I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

How should this be manually tested?

Clone the repository
https://github.com/andela/ah-centauri-backend.git
On completing of clonning

cd ah-centauti-backend
git checkout feature/164829209-user-can-comment-on-article

Rename the .env.example file into .env and update the parameters with your own keys.

For Centauri team members I will share this keys through slack

Start development server and send the following through postman

Displaying all comments for an Article

GET /api/articles/:slug/comments
Authentication required, returns All Comment.

Add Comments to an Article
POST /api/articles/:slug/comments

Example request body:

{
  "comment": {
    "body": "His name was my name too."
  }
}
Authentication required, returns the created Comment. Required field: body

Get Comments from an Article
GET /api/articles/:slug/comments

Authentication optional, returns multiple comments.

Delete Comment
DELETE /api/articles/:slug/comments/:id

Authentication required.

Pivotal Tracker
#164829209

Looks pragmatic to me


proposal: math: add Clamp function

somethimes, I need to get a number clamped in a range. I think math package would be appropiate to add such a Clamp function.
Also, for some percentage calculation, Clamp01 method is useful as well.
(see #58144)

What are the proposed function signatures?
Can you demonstrate why it should be in the standard library other than "it is sometimes useful"? ref https://go.dev/doc/faq#x_in_std

Can you demonstrate why it should be in the standard library other than "it is sometimes useful"?

Both of these functions, are parts of so called "standard libraries" in most of programming languages I know (C#, C++, rust).
ref: https://learn.microsoft.com/en-us/dotnet/api/system.math.clamp?view=net-7.0, https://en.cppreference.com/w/cpp/algorithm/clamp, https://docs.rs/num/0.2.1/num/fn.clamp.html

Rust has it

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

math.Clamp(x, min, max float64) float64 seems okay although perhaps not above the bar.
Our current rule for the math package is that the bar is "what's in the C <math.h> library".
A generic math.Clamp that works on ints (or strings?)
would have to wait for a math/v2 or some other answer to generics in std.
math.Clamp01 seems too special-purpose either way.

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely decline. — rsc for the proposal review group

@rsc according to the discussion above, it seems that Clamp is a wanted feature. Could you please justify the refusal?

@gucio321 Nobody would propose a feature that nobody wants.  The question is not: does anybody want this feature?  If we followed that guideline, we would add every feature and the result would be an unmanageable and incomprehensible mash of features.  The question is: is this feature worth the additional complexity?
As @rsc said above, we have a simple guideline today for the math package: anything that is in the C <math.h> library.  If we're going to add math.Clamp, we need a different guideline.  What should that be?  It shouldn't be "anything that people will find useful."
Thanks.

No change in consensus, so declined.
— rsc for the proposal review group


Upgrade security of the app with a token from our server.  Passcode gets in the way when the app is deployed.

BACKGROUND

As mentioned, the passcode is very weak and vulnerable. Once it is shared, anyone who has the passcode has the ability to create a new session. Additionally, having to redeploy the app every week is a problem. We are building a version of this app that will allow several users to create and host a video-room. We need a means to allow users to start a session and gain access based on data from our token server.
The current configuration and a few blogs are misleading about the ability to set the roomname using "/room/myRoomName", but this feature only works in development mode. The deployed version requires the passcode in order to see the room and username pre-page.

Question
Is there a simple way to bypass the passcode requirement?  We are able to request a token from our server which secures usage, but the user never reaches this since we cannot bypass the passcode problem.
Thanks.

Many issues were resolved by manually uploading the token server to the Function Console. Address given in the CLI response to upload the server.


[Subtask] The client supports to operate the group

Describe the subtask
The client supports to operate the group
Parent issue
#2232

Duplicated.


Develop basic entity change tracker table.

Should seriously consider some kind of domain event system in the future.


Using Symfony mime type detection

@tractorcow: try this...

Excellent idea.
Can we change the dependencies so that the numbers match? ~2.6.0 will prevent 2.8.0 from being installed, which isn't php 5.3.3 compat.

Good idea!


Unable to initialize a new migration history

Bug description
I'm facing a problem when I want to connect the local postgreSQL db to initialize a new migration history
.env file :
DATABASE_URL="postgresql://hassan:123@localhost:5432/prismadb?schema=public"
schema.prisma file :
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

"dependencies": {
    "@prisma/client": "2.30.3",
    "prisma": "2.30.3"
  },

npx prisma -v
Environment variables loaded from .env prisma                : 2.30.3 @prisma/client        : 2.30.3
npx prisma migrate dev --name init
Environment variables loaded from prisma/.env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "prismadb", schema "public" at "localhost:5432"

Error: P3014

Prisma Migrate could not create the shadow database. Please make sure the database user has permission to create databases. Read more about the shadow database (and workarounds) at https://pris.ly/d/migrate-shadow

Original error:
Database error
Error querying the database: db error: ERROR: permission denied to create database

   0: sql_migration_connector::flavour::postgres::sql_schema_from_migration_history
             at migration-engine/connectors/sql-migration-connector/src/flavour/postgres.rs:375
   1: migration_core::api::DevDiagnostic
             at migration-engine/core/src/api.rs:89

How to reproduce

Expected behavior
No response
Prisma information

Environment & setup

OS: Mac OS
Database: PostgreSQL
Node.js version: v12.20.0

Prisma Version
2.30.3

Prisma Migrate could not create the shadow database. Please make sure the database user has permission to create databases. Read more about the shadow database (and workarounds) at https://pris.ly/d/migrate-shadow

Did you read the error message, follow the link and do what it tells you to do? That should fix the problem.

Thanks , this fix the problem
postgres=# ALTER ROLE hassan CREATEDB;
ALTER ROLE

I had only had LOGIN option.


Expand functionality of InstanceSelectEditor

The component: https://github.com/psychoinformatics-de/shacl-vue/blob/main/src/components/InstancesSelectEditor.vue
Display user-relevant information in the dropdown list
Commit https://github.com/psychoinformatics-de/shacl-vue/commit/043439ee01f8cc52f472937f5d6474fe5d0d3a6c introduced the functionality for the InstancesSelectEditor to render the list of existing nodes (in the application database) with the same type as the sh:class value of the property shape being rendered by the editor. This was a necessary first step, but has some UX shortcomings, mainly that the text of the triple's subject is displayed.
For IRI nodes this will be the IRI, which will not necessary contain user-friendly information, e.g.:

And for blank nodes, these will be the uninformative and internal blank node codes, e.g.:

These displays should be updated to show user-relevant information.
The current idea is to use traversal to grab all triples with the current subject, and display them as key-value pairs in the single list item pertaining to that node. This will have to be done in a way that does not overload the user with information, so some trimming of display properties might be necessary.
It could also be an idea to find only triples with the current subject that also qualify some preset or default criteria, e.g. that it should have a specific predicate like schema:name or dlthing:identifier. This needs further exploration.
Ability to add a new item to the list
I.e. if the user cannot find the instance they are looking for in the list, they might want to add one.
The idea is to have a default item "Add" as part of the list, which when selected will open a new form defined by the sh:class value of the property shape being rendered by the editor. Upon said form completion, the entered data are saved to the application database, the "popup" form is closed, and the new information should reactively be shown in the list.

Regarding:

Ability to add a new item to the list
I.e. if the user cannot find the instance they are looking for in the list, they might want to add one.
The idea is to have a default item "Add" as part of the list, which when selected will open a new form defined by the sh:class value of the property shape being rendered by the editor. Upon said form completion, the entered data are saved to the application database, the "popup" form is closed, and the new information should reactively be shown in the list.

this is now all possible apart from "the entered data are saved to the application database, the popup form is closed, and the new information should reactively be shown in the list."
The important next step is designing a sensible solution to the "saved to the application database" part. At the moment, any information entered into the form fields becomes part of formData on entry. When a form "Save" button is pressed, field-specific validation happens (form-level shacl validation is still a TODO) and then a new empty element of the same node type is pushed to the relevant array in formData. What actually needs to happen is to be explored further in https://github.com/psychoinformatics-de/shacl-vue/issues/29

The important next step is designing a sensible solution to the "saved to the application database" part. At the moment, any information entered into the form fields becomes part of formData on entry. When a form "Save" button is pressed, field-specific validation happens (form-level shacl validation is still a TODO) and then a new empty element of the same node type is pushed to the relevant array in formData. What actually needs to happen is to be explored further in https://github.com/psychoinformatics-de/shacl-vue/issues/29

This is also now done and closed.
With 10019c26d99a6f040c3031a267d5f420667fdddb we now have additional info about a list item that aims to alleviate the lack of info in:

Display user-relevant information in the dropdown list
Commit https://github.com/psychoinformatics-de/shacl-vue/commit/043439ee01f8cc52f472937f5d6474fe5d0d3a6c introduced the functionality for the InstancesSelectEditor to render the list of existing nodes (in the application database) with the same type as the sh:class value of the property shape being rendered by the editor. This was a necessary first step, but has some UX shortcomings, mainly that the text of the triple's subject is displayed.

From the commit message:
This addresses the issue (although not conclusively) of missing
information in a list item. The list item might only show the name
of a blank node, or of a named node, while this information might
not be enough for a user to recognize the item as the one they are
looking for. It is therefore useful to show additional information
about the item, which is now retrieved by querying the graphData for
all triples with the same subject as the item being listed

Closing this since the major objectives have been achieved.


VS Code debugger not working in iOS

Debugger stops after "app.js" and app "launchscreen" then nothing happens.
It works from terminal.
System configuration: NativeScript 2.3.0, OS X 10.12 Sierra, iOS 10

@alexrainman It seems your problem is related to https://github.com/NativeScript/ios-runtime/issues/655. The issue is related to nativescript-core-modules and there is a fix merged in the master branch of the modules. Updating to @next version of the nativescript-core-modules package should fix the issue. You can read more here on how to use the latest bits.

Closing the issue due to lack of response.
If you encounter it again reopen it, open a new one or comment with detailed steps to reproduce.


update examples of apisjson.org

update to final version
specifically
SpecificicationVersion => specificationVersion
@kinlane

ping @kinlane  ;)

Fixed in spec and sample


Crash when trying to disable DynSurroundings via fmlModState.properties

Mod Version:
1.12.2-<IP_ADDRESS>
Forge Version:
<IP_ADDRESS>05
Link to crash log (if applicable):
https://gist.github.com/0e3dbbcc34ff1f05c3deb3fd13037f27
Description:
using player's choice which configured the fmlModState.properties file
on server / world join it crashes,
seems like either fml or this mod is behaving silly

My money is on FML.  DS uses ASM to hook that routine when the DS coremod runs.  I have never disabled a mod using that feature (because I think it is incomplete).  Do you also have to disable the coremod as well?

this should work :D
{
"id": "dsurround",
"name": "Dynamic Surroundings",
"desc": "Adds a variety of sound and visual effects to improve your Minecraft experience.",
"website": "https://minecraft.curseforge.com/projects/dynamic-surroundings",
"category": "Sounds",
"extraMods": ["presets", "dynamicsurroundingscore"],
"base": false
}

@NikkyAI Have you made further progress with the changes @Zer0Lyfe pointed out?

we have dropped players choice and have not attepted anything further, the fml feature seems to not be that widely supported
i will leave ths issue open unless someone can validate that the suggested config works

@OreCruncher
For some reason my playerchoices file has seemed to start not working now but seems to be different crash. It looks like it now isn't disabling the core mod for some reason. The mods are the same version and i have tried reversing the versions as well. this is the crash log.
https://paste.dimdev.org/qawegakeqa.mccrash

I'm not going to try and do anything with this.  Apparently this feature is going to be removed in Forge 1.13.  Link here.

I spent some time looking at it figuring out that it was half way implemented to find out that it's going to be removed.  Not sure if I should be happy or sad. :)

should this be closed then ?


Link over an image

Hello,
If we add an image.
Then we click over the image and we try to include a link on it.
So..
It's adding twice the image.
Do you know how to resolve this problem please?
Best regards,
Vincent.

With my previous details are you able to reproduce this problem?


register_simp_attr with

In mathlib3 we wrote:
mk_simp_attribute monad_norm none with functor_norm

but register_simp_attr does not support a with clause.

This cannot be supported. You can't add declarations to a simp attribute until the attribute is initialized in the next file.


Bug - Change frontend API URL

Deployed the latest code to dev.jobsgowhere.com
However, it fails to fetch the response as the URL is hardcoded to the localhost.

Replace this to point the hardcoded URL to HOST_URL

Oops, https://github.com/jobsgowhere/jobsgowhere/pull/43 will fix this.

Fixed in https://github.com/jobsgowhere/jobsgowhere/pull/44.


BUG: Allow spsolve_triangular to work with matrices with zeros in the diagonal.

Reference issue
Closes https://github.com/scipy/scipy/issues/15199
What does this implement/fix?
Fixes spsolve_triangular to work with matrices that have zeros in the diagonals when specifying unit_diagonal=True.

@tupui can you review this again?

Can you also please add the example from #15199 as a test?


Store Hessian/Jacobian Expression graph?

Hi all!
Thank you for the wonderful product!  I'm using autograd to do newton optimization of a convex function (so Hessian is definite).  However, the computation of the Hessian at each iteration of the newton method is prohibitive.  For example,  in the following code
def newton(ll, z0):
grad_llambda = ag.grad(ll)
hess_llambda = ag.hessian(ll)
    z = z0
    for _ in range(max_iter):

        t0 = time.time()
        gll = grad_llambda(z)
        hll = hess_llambda(z)
        print('hess and grad time', time.time()-t0)

        t0 = time.time()
        delta = np.linalg.solve(hll, -gll)
        print('solve time', time.time()-t0)

        z = z + delta
        if np.linalg.norm(delta[0:T]) < tol:
            break
        elif _ >= (max_iter-4):
            print('ERROR: NEWTON FAILED', _)
    return z

I see "Hess and grad times" at each iteration step around 1.14 seconds and "solve times" of around 0.001 seconds.
If I were to handcode my Hessian (highly undesirable), I can't imagine hess_llambda(x) taking 1 second---the hessian is only size 325x325, and the objective function is not that complicated.
I'm guessing the computation graph of the Hessian is being recomputed each time I call hess_llambda(z), but I'm not sure, and I don't really know how to store the expression graph.  Is there a way to do this?
Also, I've tried placing
grad_llambda = ag.grad(ll)
hess_llambda = ag.hessian(ll)
outside of the function definition, and this doesn't change anything.
Thank you!

A matrix of 325 x 325 is not trivial, and could take some time to compute - thought 1 second+ does seem a bit high. Can you post your log-likelihood as well?
A parallel solution is to use a hessian-free optimization routine, like scipy's (L-)BGFS methods. These methods approximate the hessian, and though take more steps, typically are as fast or faster. Ex:
from autograd import value_and_grad
from scipy.optimize import minimize

minimize(value_and_grad(ll), z0, jac=True, method='BGFS')

Hi Cameron!
Thank you for your feeback; Here is the objective function; for context, T = 25 and ess = 13 (e.g. 13*25 = 325).
In the meantime, I'll give BFGS a try.
def log_lambda_diag_slice_x(x):
    T = np.size(robot_mu_x)

    quad_robot_mu_x = np.dot((x[:T]-robot_mu_x).T, np.dot(inv_robot_cov_x, \
                              x[:T]-robot_mu_x))

    llambda = -0.5*quad_robot_mu_x

    n = 1
    for ped in range(math.ceil(ess)):
        top = top_Z_indices[ped]

            quad_ped_mu_x[ped] = np.dot((x[n*T:(n+1)*T] - \
                 mu_linear_conditioned_x[top][buffer:]).T, \
                 np.dot(inv_ped_cov_x[top], x[n*T:(n+1)*T] - \
                 mu_linear_conditioned_x[top][buffer:]))
        n = n + 1
        llambda = llambda - 0.5*quad_ped_mu_x[ped]

    n = 1
    for ped in range(math.ceil(ess)):
        top = top_Z_indices[ped]
        log_znot_x[ped] = 0.

        quad_robot_ped_x[ped] = np.multiply(np.power(\
                             x[:T]-x[n*T:(n+1)*T], 2), inv_var_sum_x[top])
        log_znot_x[ped] = log_znot_x[ped] + np.sum(np.log1p(\
                                      -np.exp(-0.5*quad_robot_ped_x[ped])))
        n = n + 1
        llambda = llambda + log_znot_x[ped]

    return -1.*llambda

Its also worth noting that this Hessian is sparse---all the mixed derivatives with respect to ped_i and ped_j are 0 (only the rows corresponding to the robot's second derivatives are non-zero).  So the number of non-zero entries is 251325 + 13*25 = 8450.  The Hessian is of size 325**2 = 105625.
Does autograd exploit sparsity all?  I would expect the computation graph to naturally pick up on this, but I'm not sure.

I don't believe autograd detect/cares if the matrix is sparse. (I could be wrong, I'm not a core developer of autograd).
Your likelihood has some for loops and vector assignments. I would try to iron out both of these, as they are likely the cause of any slowdown.
Did the BFGS solution lead anywhere?

Hi Cameron!
thank you for your response; BFGS did perform better (both in solution quality and in speed), although the CPU usage went way up.  This algorithm is for a power constrained robot so using 3 high power cores is just not in the cards.  In any case, its a great backup and alternative!  I learned something useful which I appreciate.
I agree that the for loops and vector assignments are likely causing a slowdown, and I do have some ironing out to do.
Thank you!

Heck, if you think your likelihood has no local solutions (I'm not smart enough to identify if it does), you can even try gradient descent, which will be pretty fast (though test it) and very memory cheap.

Actually, the likelihood is convex, and so not only is the local solution the global solution, but the global optima is guaranteed to be very close to the initialization (this is a separate proof).
this is part of the reason that I'm confused that a Newton method doesn't take like, 0.0000001 seconds.  I can usually get the delta correction to within an extremely small tolerance within 1 or 2 steps of newton.  I've derived the Hessian and its pretty benign.
I'm going to handcode the Hessian in the meantime because the solution should be essentially instantaneous (this also allows me to ignore a large number of computations, e.g., the zeros.).  There's just not many computations happening.  Certainly not >1 seconds worth of computations.
But autograd is so convenient!  it's just not working correctly for my application.  Or I'm not using autograd correctly.  Since I know that autograd is the correct solution, I can at least use it to verify my hand-coded hessian (which will surely have sign errors and other errors).


(feat) Support Preserving Variable Values

Makes a minor change to the VariableAssignmentCommand to include a new boolean preserve property, which is set by placing a ? before the = in the assignment.
Updating the SamplingContext to test for the new preserve property before processing the assignment. The value being assigned is only associated with the variable if the variable did not already exist in the context. Without the preserve property, the value will always be assigned.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ff47441) 97.43% compared to head (27e0c5c) 97.44%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #115   +/-   ##
=======================================
  Coverage   97.43%   97.44%
=======================================
  Files          77       77
  Lines        3352     3362   +10
=======================================
+ Hits         3266     3276   +10
  Misses         86       86

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks!
Beyond the inline comments, could you also update the documentation to include this feature?

Where should this be added? It looks like all of the variable functionality is actually documented in the sd-dynamic-prompts project.

Where should this be added? It looks like all of the variable functionality is actually documented in the sd-dynamic-prompts project.

Heck, that's a good question. I opened (EDIT: and merged) #117 and adieyal/sd-dynamic-prompts#701 to improve and sync the docs between the two repos, so please rebase and add the new stuff into docs/SYNTAX.md :)

I updated the pull request with a new section in SYNTAX.md

@akx @adieyal are there additional changes needed?