How does one do nested child content in the compnent derivation chain?

So, how do you go about dealing with multiple levels of child content? Lets say you have this sequence of components deriving from each other:
Modal -> DialogBox -> MyPersonalDialogBox
So DialogBox wants to put the child content of MyPersonalDialogBox inside a DIV, with a title bar on top and buttons on the bottom. Modal wants to put DialogBox's content inside himself so that he can present that content modally.
Obviously we can use containment instead of derivation, and have that working fine, so DialogBox USES a modal component, and my dialog box USES a DialogBox component. But ultimately that's not how OO should work. My dialog box 'is a' dialog box which 'is a' modal component, and we want to have virtual methods that can be called from baser classes up to more derived classes, and helper methods that can accept any modal component or any dialog box and so forth.
But I can't figure out how to deal with the layered child content when doing it this way. No matter what I do, the content ends up not rendered within the parent class' embrace but outside of it, or I end up with one ChildContent hiding the other and so forth. It's probably something so utterly obvious I can't see it.

Is there any specific reason why you want to use inheritance for this scenario? The containment model seems pretty clean:
Modal.cshtml

    @ChildContent

Dialog.cshtml
<Modal>

Dialog

    @ChildContent
</Modal>

MyDialog.cshtml
<Dialog>
    <button>Super OK!</button>
</Dialog>

If you need to treat these components generically you could just implement an interface.

It's for the reasons I indicated. There is a clear inheritance relationship, and without that, you lose a lot of the reasons that inheritance was invented: the ability to create virtual interfaces that derived classes can implement and to pass the derived class to something that takes the base class, with all of the myriad advantages that that entails. It's pretty basic OO reasoning, it seems to me.
We already have the containment version working. But if that's the only way to do it in Blazor, then I'd argue that Blazor is missing some massive benefits of OO programming.

without that, you lose a lot of the reasons that inheritance was invented: the ability to create virtual interfaces that derived classes can implement and to pass the derived class to something that takes the base class

Yes, I understand all that. I was just interested if you had specific use cases where you planned to leverage the is-a relationship established by inheritance. Instead of having a generic inheritance vs composition argument I find it helps to talk about concrete scenarios.

We do have some, but I think that the basic argument is the important one. I'm sure we could hack together something to do what we really to do, so we don't need help with that. The issue is whether Blazor is an OO component model, or not? I think it obviously needs to be.
Otherwise the argument would have to be not whether we have a specific application at the moment, but whether Blazor is going to deny application of those benefits of inheritance to all its users. And I don't think it needs to be debated how powerful those benefits are this many decades down the OO line, right? That's ultimately the issue at hand. If it's not going to support component inheritance, then that's sort of the position that, tacitly or otherwise, is being taken.

@DeanRoddey There is a strong argument for composition over inheritance. Maybe this is something you can look into. Composition over inheritance - WIKI
It is something I personally am a big fan of. It reduces coupling in a big way.

I just reject that article's premise on fundamentals. It's not like people have been hallucinating for 50 years. OO principles of inheritance and polymorphism are powerful and proven. They are at the core of most of the serious software that exists today. Of course, these days, everyone seems to be trying to push the next big thing. I'm holding out for nutmeg based programming myself.
And obviously implementing these capabilities doesn't in any way prevent you from using composition slash containment where that is most effective. If Blazor only implemented inheritance and not containment/composition, and I was arguing with you that inheritance is all you need and I prefer that, I doubt you'd accept that as a good answer. It's hard for me to see an argument for not supporting one of the most widely used and profoundly impactful software techniques of the last half century.

But... Is the issue where what can one do given a stick? Or is it, how could maybe Blazor be better than WPF, a UI framework from back when one wrote software by rubbing sticks together?

@DeanRoddey  Lot of UI component systems in this space like one of the lead component systems React, doesn't use inheritance. Blazor/Razor Components team doesn't have unlimited resources, so you have to come up with use case where inheritance has clear advantage over composition like @danroth27 said. Use case where something is extremely hard to create with composition and easy to create with inheritance. Look into Angular 2.3+, I read they added inheritance for their components in 2016.
I just feel like lot of cases, like this component system, inheritance makes things more complex and less maintainable without added nenefits.

Lots of stuff in the browser UI world has sucked tremendously from the beginning because it's been a toy system. But everyone (to my dismay) seems intent on trying to use it to write real applications (or at least as realer-like as any browser app can be, which leaves a lot to be desired.)  If that's the case, then much more serious component libraries are going to need to be created.
I've created multiple serious UI frameworks in the desktop world, and I couldn't imagine that anyone would question the value of being able to get or set the checked state of any variation of a family of checkable controls, to get/set the caption text of any in a family of text display controls, to manipulate the list items in any of a family of list based controls,  to manage the image in any of a family of image display controls, that any variation of a family of menu types can be manipulated generically, and on and on.
In some cases that's not an issue currently with Blazor, because there's not always a need for multiple layers of child content to do some of that. But as soon as you hit a need to have nested child contents, then that new control is no longer part of that derivation chain and it becomes non-interchangeable with its breathren, despite clearly conceptually being one. Manually recreating methods and delegating doesn't help because you need the common base classes to have generic manipulation.
That that would be questioned in this day and age wobbles my mind. It's at the core of being able to create reusable content.
And the thing is, I don't think it's that huge a deal. There is already a class derivation involved. It just needs a way for each layer in the chain to manage its own child content. That's all that's missing, it seems to me.
Anyway, that's all I'm going to say. I'm not going to get into an endless debate on the subject. I'd just say that, if Blazor doesn't, something else that comes afterwards probably will; and, other things being equal, it would have a substantial architectural advantage. It's not just about the resources of the Blazor team, it's also about the resources of the (many, many more) teams that use these systems and which ones make their development lives the least burdensome.
So, anyhoo, they can make of that what they will.

Before I leave it, I'll give you a very clear example, which maybe some folks here aren't considering because they look at this purely from the standpoint of someone writing applications, not someone creating tools.
At some point, presumably, someone is going to make some tools for visually building blazor based screens. This is VASTLY easier to do if there is a clear inheritance chain for related families of components, because it allows the tool to ask for configuration data in a generic way and present it without having to get into all kinds of component specific stuff, or pushing any configuration logic into the components themselves. It also allows components along the chain to override or hide or add to attributes in lower levels which can be very important.
If my touch screen UI development system didn't have this kind of clear inheritance chain the UI designer would be vastly harder to do.

@DeanRoddey Visual tools needing a clear inheritance chain is a good argument. A fruitful discussion around that concrete point could be interesting.
@danroth27 already said it:

[...]Instead of having a generic inheritance vs composition argument[...]


Responsive on schedule table (home/schedule page)

mobile

tablet

laptop <1500 resolution

It isn't fixed on #236?

second team logo and text should be inverted on screens from tablet to laptop, mobile works
padding in center colunm (time/vs/score) should be the same
mobile screens reduce the padding in center colunm

@nightmir3 2 and 3 items fixed in #276, the first one was fixed by other user.

ok, closed


update doc

php test failed
add copy from

Codecov Report

Merging #225 into async_api will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           async_api     #225   +/-   ##
==========================================
  Coverage      75.39%   75.39%
==========================================
  Files             19       19
  Lines            878      878
==========================================
  Hits             662      662
  Misses           216      216

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 cf96ad6...c636932. Read the comment docs.


Phase enum has a new variant Initialization

Phase enum has been updated with a new variant: https://substrate.dev/rustdocs/v2.0.0-rc4/frame_system/enum.Phase.html
Still need to update tests, etc.

Closing as a duplicate of https://github.com/Snowfork/go-substrate-rpc-client/pull/7


Implement cupy.show_config and cupyx.get_runtime_info

cupy.show_config is a counterpart for numpy.show_config that prints configuration information to stdout.
Note that cupy.show_config prints both compile-time and runtime info, whereas numpy.show_config only prints compile-time info.
cupyx.get_runtime_info is a hidden API to be used in Chainer (see PR https://github.com/chainer/chainer/pull/4298)

LGTM. Please fix style error.

Thanks, fixed.

jenkins, test this please

LGTM!


Caddy seems to pretend it can't access folders the calling user owns

Caddy 0.11.0
Caddy doesn't seem to want to read from a directory it can read from. The calling user is www-data and when the service from systemd runs, and it stats the directory from the Caddyfile and acts as if it can't read from it. The resulting error is this:
Error during parsing: Unable to access root path '/home/www-data/www': stat /home/www-data/www: permission denied
The www-data user is the owner for both the group and user, and recursively as well with 0755 and 0775 permissions respectively. Which makes this confusing as the user literally has all the access to these directories, but still acts as if it can't find them or has no permission to.
Running Caddy with the same commandline arguments while logged into www-data has no issue, and everything is served as normal.
Caddyfile:
# Default Configuration

subdomain.domain.tld
{
  tls<EMAIL_ADDRESS>                   # Enable SSL
  ext .html .php                           # Clean URLs
  root /home/www-data/www                  # Declare Root Folder
  log /usr/local/log/caddy-www.log         # Create Access Log
  {
    rotate
	{
	  size 1024
	  age 365
	  keep 999
	}
  }
}

SystemD Service:
[Unit]
Description=Better Caddy Startup
Documentation=https://caddyserver.com/docs
After=network-online.target
Wants=network-online.target systemd-networkd-wait-online.service

[Service]
Restart=on-failure

ExecStart=/usr/local/bin/caddy -log /usr/local/bin/caddy-main.log -agree=true -conf=/usr/local/etc/caddy/Caddyfile -root=/home/www-data<EMAIL_ADDRESS>ExecReload=/bin/kill -USR1 $MAINPID

User=www-data
Group=www-data

Environment=HOME=/var/www

PrivateTmp=true
PrivateDevices=true
ProtectHome=true
ProtectSystem=full

ReadWriteDirectories=/var/www
ReadWriteDirectories=/home/www-data/www

CapabilityBoundingSet=CAP_NET_BIND_SERVICE
AmbientCapabilities=CAP_NET_BIND_SERVICE
NoNewPrivileges=true

RuntimeDirectory=/home/www-data

[Install]
WantedBy=multi-user.target

Keep in mind, this is on a completely new, from scratch, clean installation of Debian 9.4 on a VPS, so there should literally be nothing interfering with this. I'm pretty sure it's an issue with the Go Runtime or an issue with Caddy's stat call (I might try removing the fail case), but I'm not completely sure yet.

I figured it out.
ProtectHome=true
This was the offending line in the service file. Guess I should have looked closer then to think it was the fault of stat or the runtime.


[SPARK-47752][PS][CONNECT] Make pyspark.pandas compatible with pyspark-connect

What changes were proposed in this pull request?
This PR proposes to make pyspark.pandas compatible with pyspark-connect.
Why are the changes needed?
In order for pyspark-connect to work without classic PySpark packages and dependencies.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Yes, at https://github.com/apache/spark/pull/45870. Once CI is setup there, it will be tested there properly.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @itholic @zhengruifeng

LGTM when CI pass

Merged to master.


Allow multiple environments in the same account with cdk-pivot role

Feature or Bugfix
IMPORTANT

⚠ ⚠ ⚠ IMPORTANT: Users MUST upgrade their data.all Environments and Datasets if they want to enable multiple_environments_same_account pivot roles! We need to make it very clear in the release notes. Otherwise a good hint that indicates if an environment is updated is that when opening the Environment/Stack tab they get an error of the type: An error occurred (AWSResourceNotFound) when calling GET_STACK_LOGS operation: Logs could not be found for this stack

⚠ ⚠ ⚠ IMPORTANT: For data.all CREATED datasets, this PR introduces a change in the IaC that BREAKS the KMS policy for folder and bucket sharing. The share will appear as unhealthy if we verify it and will be fixed by re-applying the share. To automate this process we recommend users to:
- 1. Update data.all to new version
- 2. Update all Environment and Dataset stacks
- 3. trigger the share-verifier task in ECS
- 4. trigger the share-reapplier task in ECS introduced in PR #1151

⚠ After upgrading, there will be some useless LakeFormation permissions granting permissions to the deleted dataallPivotRole-cdk because LakeFormation does not delete the permissions of deleted IAM roles. In addition, Lake Formation keeps the old pivot role as datalake admin flagged as invalid (see pic below). The permissions do not really harm anyone and can be deleted manually. If you are a user and are struggling cleaning-up or wish more support, please open a GitHub issue referencing this PR and we will address your challenges.

Detail
Problem statement
data.all =<2.2.0 is not designed to allow the creation of multiple data.all Environments in the same AWS account. However, there are customers that store and work with data in the same AWS account but in different regions. This PR is meant to enable that use-case solving its 2 blockers:

If using cdk-pivot roles, the second environment stack fails to deploy because there is already an existing dataallPivotRole-cdk IAM role created in the first environment stack. Explained in detail: #936
If using manually created pivot roles, there is a limit ~5regions that can be onboarded. The failure is caused by the inline policies that CDK adds to the pivot role for the DLQ SQS queues and X-trace. Explained in detail: #1048

Design
Our recommendation is to use cdk-created pivot roles because their policies can be more restrictive and manual errors of deploying CFN stacks are avoided. To solve #936 and use CDK pivot role in a multi-environment same account setup we have 2 options:

re-use the same IAM pivot role for all environments in the account: environment 1 deploys the pivot role nested stack and the subsequent environments detect that there is an IAM pivot role already created and use the existing one. The main advantage is its simplicity, we keep using the same IAM role and there is no need for backwards compatibility or changes in the code to assume a different role. But it has some important drawbacks:

it does not solve #1048
issues with deletion of environment 1 ---> handle environment 2 updates. Environments depend on each other :'(
permissions of the pivot role refer to environment datasets.... ---> how will the pivot role get updated for other environments?? Not as simple as it looks.

deploy one IAM pivot role for each environment. Each environment deploys a separate pivot role stack. All role assumption boto3 calls need to be updates to use a different pivot role depending on the environment. It keeps each pivot role restricted to its environment, there is no coupling between environments and no #1048 policy limit issues. As disadvantages we can point out:

more code changes, passing info between calls and boto3 methods
backwards compatibility with original dataallPivotRole-cdk role.

Design decisions
Despite having disadvantages, this PR implements option 2 because it is more robust in the long-term. It also has advantages in terms of security as it "isolates" regions; we can take it further an define pivot role policies with the specified region. In addition, we reduce the impact of IAM size service quotas as each role is dedicated to one environment only.
To minimize the code changes and impact, I have used the region to differentiate between pivot roles in the same account. At first I added the environmentUri to the pivot role name, but it made it very difficult to use in the boto3 aws clients. We would need to modify all the client calls to input the environmentUri. That's why I re-used the region which was already a parameter of most clients.
This PR does not handle multiple environments using the manual pivot role. Meaning that it does not expect that each environment has a dedicated dataallPivotRole-region. It works as usual with one dataallPivotRole for all environments. We want to favor CDK-pivot roles and slowly move away from manual pivot roles.
Backwards compatibility
As preventive measurement, this PR introduces a config.json parameter to configure the usage of cdk_pivot_role_multiple_environments_same_account. The default value is False, thus the changes introduced in this PR are not mandatory and customers can migrate whenever they have the need. UPDATE FROM REVIEW: If we see that the backwards compatibility is smooth we will discard the parameter. This is a good enhancement and does not remove any functionality for customers.
TESTING
On an existing deployment with 2 environments and datasets (imported and created) and multiple shares, update the codebase. All modules are active and enabled in the environments.
Part 1: dataallPivotRole-cdk continues working

[X] Update environment with no parameter cdk_pivot_role_multiple_environments_same_account--> check it updates and keeps deploying the dataallPivotRole-cdk
[X] Update environment with cdk_pivot_role_multiple_environments_same_account= False --> check it keeps deploying the dataallPivotRole-cdk
[X] Check IAM permissions of dataallPivotRole-cdk`(still contain KMS and S3 statements relative to imported dataset) and ensure it is the LF admin and has dataset permissions in LF.
[x] Create new dataset --> check new permissions are added to dataallPivotRole-cdk
[X] CloudFormation get stack (in Environment view) --> boto3 calls use the remote_session pointing at dataallPivotRole-cdk

Part 2: dataallPivotRole-cdk-region migration infrastructure

[X] change cdk_pivot_role_multiple_environments_same_accountto True and update environment. Environment updates the pivot role stack and creates NEW dataallPivotRole-cdk-region - as expected, in CloudFormation we see the message Requested update requires the creation of a new physical resource; hence creating one
[X] IAM permissions of dataallPivotRole-cdk-region --> check that imported datasets in the environment are present in the policy statements
[X] Ensure dataallPivotRole-cdk-region is a LF admin - it happens right after environment update
[X] Update created dataset stack and check that dataallPivotRole-cdk-region  has LF permissions to the dataset. Check that KMS key policy includes the new pivot role. Finally check that the trust policy of the dataset IAM role includes the new pivot role:
[X] Update imported dataset stack and check that dataallPivotRole-cdk-region  has LF permissions to the dataset. Bucket policy and KMS policies are not managed by data.all in the creation of dataset. Finally check that the trust policy of the dataset IAM role includes the new pivot role

Part 3: dataallPivotRole-cdk-region migration data sharing

[X] Click on verify permissions for one of the shares (imported dataset) ---> ⚠ verification shows table as unhealthy, folder appears as healthy. When users click on re-apply share, it grants permissions to the new pivot role in the cross account database and resource link and it appears as healthy again. See detail in Issues found section.
[X] Add new table to share in the same share --> check that share is successful
[X] Add bucket to share --> check that share is successful - KMS policy contains new pivot role!
[X] Revoke one table  --> check share is successful - to ensure LF permissions work
[X] Revoke one folder  --> check share is successful - to ensure LF permissions work
[X] Revoke one bucket  --> check share is successful - to ensure LF permissions work
[X] click on verify permissions in the other share (created dataset) --> ⚠  in the dataset_stack, the KMS key policy has been wiped out, so bucket and access point shares are unhealthy!  (This part is fixed, see details below) After re-applying shares:

 Failed to get kms key id of alias/loc-new-uqo3m5oh: An error occurred (AccessDeniedException) when calling the DescribeKey operation: User: arn:aws:sts::796821004569:assumed-role/dataallPivotRole-cdk-eu-west-1/dataallPivotRole-cdk-eu-west-1 is not authorized to perform: kms:DescribeKey on resource: arn:aws:kms:eu-west-1:796821004569:key/d00a9763-7fd0-4845-81d6-863d20714888 because no resource-based policy allows the kms:DescribeKey action

Part4: dataallPivotRole-cdk-region in boto3 calls - especially IAM and Quicksight that required some additional development.

[X] IAM - Add consumption role and check its policies attached
[X] CloudFormation - get stack logs in any Stack tab - check client
[X] Athena dataset client - Preview table
[X] Glue dataset client - Start crawler
[X] Glue profiling client - Run profiling job --> it triggers the profiling job but there is an issue unrelated to this PR related to the profiling script int he S3 bucket of the environment: key: profiling/code/jars/deequ-2.0.0-spark-3.1.jar.The specified key does not exist
[X] S3 location client - Create folder - check that s3 prefix gets created in S3
[X] S3 dataset client - Get Dataset AWS creds - also checks that the dataset role can be assumed by the new pivot role --> for backwards compatibility dataset stack MUST be updated.
[X] LF dataset clients +  S3 bucket policy client - Create Dataset
[X] IAM + data sharing clients (KMS, LF, RAM, S3, GLUE) - Share new items, revoke items, create new share. --> same error on KMS key for bucket share.
[x] Quicksight - Create Quicksight session in dashboards - it checks not only the client but the Quicksight identity region
[ ] SageMaker Notebooks - create and open Notebook 🔴 Error: 'Could not find any VPCs matching...' it fails on creation
[X] SageMaker ML Studio + EC2 base client - create and open ML Studio user
[X] Athena - run query in Worksheets
[ ] Remote session in Pipelines - Create Pipeline 2 types: CDK and non-CDK (tested below)

Additional testing in AWS + support PRs (Update April 8th) - deploy https://github.com/data-dot-all/dataall/pull/1151 as well

[X] Deploy in AWS CICD pipeline succeeds
[x] Revoke shared bucket without verify/reapply (after update environments and datasets)
[x] Revoke shared folder without verify/reapply (after update environments and datasets)
[x] Revoke shared table without verify/reapply (after update environments and datasets) - I checked the logs and before the revoke it grants the pivot role permissions to the db
[X] Verify shared table succeeds and does not fail because of missing pivot role permissions (test Fix in PR: #1149 )
[X] Deploy #1150 - Verify shared folder and bucket shows unhealthy for data.all created datasets, admins go to data.all and trigger ECS task share-reapplier, verify shared folder and bucket again, it should be healthy (Test #1150) 🚀

I triggered the share-verifier task manually and only the expected broken shares (created datasets - folder/bucket sharing) appeared as unhealthy
Other less important items - open issues if needed

[X] Test Notebooks --> all good, it must have been something related to my local environment
[X] Test Pipelines --> all good we can get credentials
[ ] Test profiling jobs ---> another different error ImportError: cannot import name 'ColumnProfilerRunner' from 'awsglue.transforms' (/opt/amazon/lib/python3.6/site-packages/awsglue/transforms/__init__.py) --> open github issue

Open Issues, remarks and questions

[X] ISSUE TO FIX:  Folder and bucket permissions get wiped out when the dataset stacks get updated with the new KMS statements (see the issues above). Users need to re-apply the shares because after the upgrade they will be broken. ⚠ in order for this PR to be released we need to implement some ECS task or automation to re-apply all failed shares ---> Issue Using PR https://github.com/data-dot-all/dataall/pull/1151 this issue can be remediated

[X] TO DECIDE: Should we make this feature configurable? If so, what is the deprecation timeline for dataallPivotRole-cdk? ---> At the moment: Yes, this can be configured so that users can decide when to do the upgrade. We will use the maintenance window implementation to update users about this change.

[X] ISSUE TO FIX: Bucket sharing or folder sharing in testing part3 will fail for imported datasets because the statement in the KMS policy DATAALL_KMS_PIVOT_ROLE_PERMISSIONS_SID is skipped in dataall/modules/dataset_sharing/services/share_managers/s3_access_point_share_manager.py:535 keeping the reference to the old pivot role. One quick fix is to rename the Sid, but that will leave orphan resources referencing the old pivot role. A better fix is to ensure that the role is that sid is the one of the pivot role. --> 🟢 fixed

[X] ISSUE TO FIX: Missing kms:DescribeKey permissions for pivot role from KMS key policy making the share fail. I am not sure how this issue did not surface before. I think it has to due with the by default permissions in KMS vs the ones applied by data.all.  Failed to get kms key id of alias/loc-new-2-eo336u1r: An error occurred (AccessDeniedException) when calling the DescribeKey operation: User: arn:aws:sts::796821004569:assumed-role/dataallPivotRole-cdk-eu-west-1/dataallPivotRole-cdk-eu-west-1 is not authorized to perform: kms:DescribeKey on resource: arn:aws:kms:eu-west-1:796821004569:key/bb37c05a-0588-45f2-a9e4-4075392ee5e1 because no resource-based policy allows the kms:DescribeKey action  --> permissions added in this commit

[X] ISSUE TO FIX: Table health verification shows unhealthy after update: Requestor arn:aws:iam::SOURCE_ACCOUNT:role/dataallPivotRole-cdk-eu-west-1 missing LF permissions: ALL for Glue DB Target: loc_imported_c2_kms_URI Requestor arn:aws:iam::TARGET_ACCOUNT:role/dataallPivotRole-cdk-eu-west-1 missing LF permissions: ALL for Glue DB Target: loc_imported_c2_kms_URI_shared  The issue is solved when we click on Re-Apply share but it is a bit annoying that customers need to re-apply all their shares. I think we should introduce an ECS task to re-apply all shares. @anushka-singh already suggested this feature for other purposes --->based on @noah-paige's review, I think the best fix will be to ensure that the pivot role gets granted all the permissions in the verification task, that way the verification won't fail and the re-apply won't be necessary for cases in which the missing pivot role permissions are the source problem. ---> Fix in PR: #1149

Less important

[X] ISSUE UNRELATED TO THIS PR: Run profiling job --> it triggers the profiling job but there is an issue unrelated to this PR related to the profiling script in the S3 bucket of the environment: key: profiling/code/jars/deequ-2.0.0-spark-3.1.jar.The specified key does not exist ---> confirm and create issue

[X] ISSUE TO INVESTIGATE: the permissions of the pivot role after the shares are reapplied seem incomplete:
  --> 🟢 It does not seem to have an impact on shares, all table sharing keeps on working as expected.

Relates

#936
#1048

Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)? No

Is the input sanitized? n/a
What precautions are you taking before deserializing the data you consume? n/a
Is injection prevented by parametrizing queries? n/a
Have you ensured no eval or similar functions are used? n/a

Does this PR introduce any functionality or component that requires authorization? No

How have you ensured it respects the existing AuthN/AuthZ mechanisms? n/a
Are you logging failed auth attempts? n/a

Are you using or adding any cryptographic features? No

Do you use a standard proven implementations?n/a
Are the used keys controlled by the customer? Where are they stored?n/a

Are you introducing any new policies/roles/users? Yes

Have you used the least-privilege principle? How? -----> 🖐 We use a dedicated IAM role for each data.all Environment onboarded to data.all. This role can access data in its region only. This way the blast radius of any attack is limited to a single AWS region. This PR improves the user experience to use cdk-Pivot role, which encourages users to avoid manual pivot role that has more open permissions and is more error prone. -----> This PR also introduces a tiny change on the IAM permissions of the Lambdas and ECS tasks of the backend. They can now assume dataallPivotRole-cdk-* to account for the multiple region roles. This change is still restrictive enough

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Went through the PR and left some minor comments. Overall the PR makes sense to me, my comments for the Issues to Decide below:

(1) Health verification shows unhealthy after update...
Confirming my understanding first:

This would not just be for tables but also for folders and buckets right?

For Tables - it shows unhealthy but the shares still work (missing PivotRole permissions that can be fixed on next reapply)
For Folder/Buckets - these share items fail once dataset_stack updated and KMS wiped, right? (fixed on reapply as. well)

Question:
- Have we tested switch to new pivotRole naming and then immediately revoke table share item (I think should work but worth testing?)
Ideas:

I do agree I think we should have a reapply ECS task that could be run once for all shares in unhealthy status rather than handling each one individually. I think we need to better understand how this would be used, as in should this be ad-hoc triggered, scheduled,  paired with share verify task (maybe initially keep it as its own ECS task triggered on-demand?)

Secondly, should we even be checking PivotRole permissions for verify_shares table items? PivotRole missing LF permissions should not interfere with the table item share and we tend to grant PivotRole LF permissions (if missing) before adding, removing, or re-applying table items?

(2) Orphaned LF Permissions to Old Pivot Role

I am a bit surprised that we do not overwrite the old pivotRole name as admin based off of the logic in dataall/modules/datasets/cdk/assets/lakeformationdefaultsettings/index.py (this CR depends on pivot_role meaning the new pivot role must exist and the old should not and thus the old pivot role name should not be a DataLakeAdmins passed to put_data_lake_settings()

Maybe we can fix logic there with a 1-liner to remove invalid role as Admin
I think that would suffice for this PR

For notebook stack creation failing with Error: 'Could not find any VPCs matching...' - would this not be due to missing permissions or similar on the cdk execution role if anything (and not the pivot role)?

Hello @dlpzx Thanks for prioritising and the details.
Attached image is my understanding about the feature

About open issues, remarks and questions
TO DECIDE: Should we make this feature configurable? If so, what is the deprecation timeline for dataallPivotRole-cdk?

Yes should be made configurable.
As soon as the feature is ready we are planning to adapt it.
By end of Q3 we should have upgraded all our work environments so as long is it is available till Q3 end we are Good

ISSUE TO FIX: Folder and bucket permissions get wiped out when the dataset stacks get updated with the new KMS statements (see the issues above). Users need to re-apply the shares because after the upgrade they will be broken. ⚠ in order for this PR to be released we need to implement some ECS task or automation to re-apply all failed shares ---> Issue

By Q2 end we have plans to rollout for stakeholder use so if we get it before Q2 End should be Good

ISSUE TO DECIDE: orphan permissions pointing at the old cdk role in LF: LakeFormation does not delete the permissions of deleted IAM roles, so the old pivot role keeps having LF permissions and is still the datalake admin after it has been deleted. It appears as invalid. Should we do anything about it? Users can just clean-up the permissions and remove the deleted role from their accounts, but it is true that it is a bit annoying. We can at least offer some sort of script or at least instructions

We haven't faced this issue, It is Low priority for us. Instructions should suffice

Hi @sandeephs1 thanks for the diagram :) Yes, that is exactly the expected behavior. We will consider your feedback, with the feature as is now I think your requirements for Q2-Q3 are satisfied

Tested deploying this PR and ensuring:

[x] CICD Pipeline Successful
[x] Env Updates Successful (on stack updater task run)
[x] Dataset Updates Successful (on stack updater task run)
[x] Table Shares still work
[x] Folder Share works after verify + reapply
[x] New Env in same account, dif. region works

Left one minor comment on IAM policy permissions - once addressed I think good to approve!


Printing model name

@smilesun is it possible to print also the model name in this statement? With multithreading it is not possible to say to which algorithm this print refers.
https://github.com/marrlab/DomainLab/blob/37504b0d51b241076ab86374dfa5727bfec7b315/domainlab/compos/exp/exp_main.py#L50

8136a87


Refine flaky tests introduced by #586

#586 introduced some flaky tests like using time.sleep(3000).

done in #615


Bundle appmap-java agent v1.15.4

Can be tested by launching the IDE with the plugin and running a run configuration with the AppMap executor.
I verified that the bundled jar is the new version.

Note that releases may be disabled right now because of the Travis
credentials leak issue
On Thu, Dec 8, 2022 at 8:45 AM Joachim Ansorg @.***>
wrote:

Merged #176 https://github.com/getappmap/appmap-intellij-plugin/pull/176
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/getappmap/appmap-intellij-plugin/pull/176#event-7989227994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC63D45JHCP7WO5BGDODWMHRBJANCNFSM6AAAAAASYETDCU
.
You are receiving this because you are subscribed to this thread.Message
ID: <getappmap/appmap-intellij-plugin/pull/176/issue_event/7989227994@
github.com>

:tada: This PR is included in version 0.21.1 :tada:
The release is available on:

v0.21.1
GitHub release

Your semantic-release bot :package::rocket:


hand-foot syndrome

Preferred term label: hand-foot syndrome
Synonyms: palmar-plantar erythrodysesthesia
Definition (free text, please give PubMed ID, if applicable, in format PMID:#######): a side effect of some chemotherapy drugs that can cause redness, swelling and blistering on the palms of the hands and soles of the feet.
Parent term (use OLS, or your favorite ontology browser): skin disorder
Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?
Your nano-attribution (ORCID): 0000-0001-7151-1615

x-refs:
NCIT:C27177
UMLS:C0549410
MESH:D060831
synonym:
Palmar-Plantar Erythrodysthesia (NCIT:C27177)
definition:
A condition characterized by redness, pain, swelling, and tingling in the palms of the hands or the soles of the feet. It may appear as a side effect to chemotherapy agents.
NCIT:C27177


Invalid column type from bcp client for colid 7.

I am trying to insert 39M of records into table which contains 375 columns. I created this table exactly like the source (stream file .ss) and still getting the colid 7. or colid 8 match, i have googled this issue but no luck. even I tried to changed the column to nullable.
This is the error I am getting from my databricks notebook.
Please use this Error ID when contacting your Administrator for assistance. EID:(717910bd397f4c189533e8c87e72b767)

Unfortunately without a repro or more details, it is very hard for the community to provide further input. Let us know if you have a more complete example to demonstrate this issue, else we will have to close this issue by Sept 1st, 2020.

Closing due to inactivity.


Add multi-user file upload

file uploading added to multi-user-integration tho it needs some testing -- I don't have any files to test with


Cannot run

Hi there,
Thank you for developing this amazing tool. I've tried to run the container however, I am getting this error:
/usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/lib/wraith/wraith.rb:166:in verbose': undefined method []' for nil:NilClass (NoMethodError)
from /usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/lib/wraith/validate.rb:13:in validate' 	from /usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/lib/wraith/cli.rb:131:in block in capture'
from /usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/lib/wraith/cli.rb:28:in within_acceptable_limits' 	from /usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/lib/wraith/cli.rb:130:in capture'
from /usr/local/lib/ruby/gems/2.1.0/gems/thor-0.19.1/lib/thor/command.rb:27:in run' 	from /usr/local/lib/ruby/gems/2.1.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in invoke_command'
from /usr/local/lib/ruby/gems/2.1.0/gems/thor-0.19.1/lib/thor.rb:359:in dispatch' 	from /usr/local/lib/ruby/gems/2.1.0/gems/thor-0.19.1/lib/thor/base.rb:440:in start'
from /usr/local/lib/ruby/gems/2.1.0/gems/wraith-3.1.0/bin/wraith:5:in <top (required)>' 	from /usr/local/bin/wraith:23:in load'
from /usr/local/bin/wraith:23:in `'
ERROR: unable to find config at config.yaml
I am running the container as follows: alias docker-wraith="docker run -P -v /path/to/your/project/wraith:/wraithy -w='/wraithy' bbcnews/wraith"
docker-wraith capture config.yaml
and in the directory that i'm passing to the container I have a file called config.yaml
Do you know what i'm doing wrong?
Thanks
Regards,
Sergiu

To help us debug, can you paste the contents of your YAML file here?

Hi there,
Thank you for your help, however I have now fixed the issue. It was due with the way I was trying to running. Doing lots of researching and combining bits and pieces of how to run containers now it is working how it should, however I would suggest putting a better 'how to run it section' for docker & wraith beginners as myself.
Thanks,
Regards,
Sergiu


Bug: Creating or modifying wallet address is currently broken and does not have proper error handling

Describe the background or context
I spent over three hours trying to verify my PR and get 'QA' and I stumbled against a serious bug.
There are two problems:

Inserting or modifying any database row with type defined as a fixed length char is currently broken and allows only 1 character. I verified this with supabase dashboard. The root cause is https://github.com/PostgREST/postgrest/issues/2861 which was fixed but the fix is not available yet in the workspace.

Error handling is supabase adapter is broken and it must be fixed e.g.

https://github.com/ubiquity/ubiquibot/blob/development/src/adapters/supabase/helpers/client.ts#L165
https://github.com/ubiquity/ubiquibot/blob/development/src/adapters/supabase/helpers/client.ts#L170
https://github.com/ubiquity/ubiquibot/blob/development/src/adapters/supabase/helpers/client.ts#L178
https://github.com/ubiquity/ubiquibot/blob/development/src/adapters/supabase/helpers/client.ts#L193
Modifying db entries return data or error, but nothing is done when error happens. The functions happily return with just logger traces and the bot claims that everything went well, see https://github.com/gitcoindev/ubiquibot/issues/6#issuecomment-1678456596 and bot comments below. Only when I locally modified the source to insert "1" as wallet address, it was truly successful.
Describe the solution

Use variable char in DB, even for address and validate variables length where needed:

https://github.com/PostgREST/postgrest/issues/1586#issuecomment-696821113

Implement a proper error handling in supabase adapter.

This a known issue. Solution for 2. will be resolved by #545

Good to hear that @whilefoo

I will make a PR that solves both issues because I don't know how long #545 will take to merge

I will make a PR that solves both issues because I don't know how long #545 will take to merge

Normally we prefer to have unique pull requests for unique issues. This is for auditing reasons. You should try and do a unique pull request for this issue by cherry picking the changes from your other pull request. It's fairly straightforward to do if you use a tool like gitgraph for vscode once the code is already done.


discussion

Hello, thank you for your open source work!
I got an error running the sample：
AssertionError: Do not find any checkpoint under the dir ./simulation/pretrained_promptEHR.
Could you give me some hints

I just run the code, it does not produce the same error. Can you try to remove the folder simulation in this directory and try it again?


CascadingDropDown extender control continally refreshing in Firefox browser 65.0.1

The AJAX Control Toolkit \18.1.1
The installation method

[ ] Installer

Browser(s) used
IE, Edge, Chrome, Safari are okay, Firefox 65.0.1 is not okay.
A site deployment method

[ ] VS development web-server,
[ ] IIS

I have been using the CascadingDropDown extender control for several years now in an ASP.NET web site using version 15.1.4
The scenario in which the control is used is like select Make of Car in first dropdown, then select Model in the second.
Just recently a user who prefers Firefox said the dropdown controls continually refresh several times a second as of about 22 Feb 2019 after Firefox updated to V65.0.1
They said it worked fine up until then. In fact he went to another PC with earlier version of Firefox and it worked, updated Firefox and the refreshing started.
I just downloaded the latest version of AjaxControlToolkit (v18.1.1) on my development system and it does the same thing.
I notice that while it is refreshing (several times a second) the IIS Express worker process is busy until the page is closed.
Works fine in other browsers such as IE, Edge, Chrome, Safari.
It would appear the problem is Firefox but I thought the issue should be raised here as well.

How to reproduce this?
I'm using Firefox, and the CascadingDropDown demo doesn't send any excessive requests.

I agree the demo doesn’t produce the rapid refreshing and works as intended.
My implementation is much more complex than that reading a database and all wrapped up in a user control.
(It has been working for over 5 years – first complaint was when a user upgraded Firefox to 65.0.1)
My implementation uses:

In the master page which is the same as the CascadingDropDown demo https://ajaxcontroltoolkit.devexpress.com/CascadingDropDown/CascadingDropDown.aspx  uses.
When I get time, I shall strip the control down bit by bit until it works in an effort to identify the problem.
Kind Regards,
Roger.
From: AlekseyMartynov<EMAIL_ADDRESS>Sent: Tuesday, 19 March 2019 1:30 AM
To: DevExpress/AjaxControlToolkit<EMAIL_ADDRESS>Cc: RogerMCA<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [DevExpress/AjaxControlToolkit] CascadingDropDown extender control continally refreshing in Firefox browser 65.0.1 (#477)
How to reproduce this?
I'm using Firefox, and the CascadingDropDown demo https://ajaxcontroltoolkit.devexpress.com/CascadingDropDown/CascadingDropDown.aspx  doesn't send any excessive requests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/DevExpress/AjaxControlToolkit/issues/477#issuecomment-473933495 , or mute the thread https://github.com/notifications/unsubscribe-auth/AuZ3dICS1mnCSUhRPlHkljpR10aw_Yudks5vX6L7gaJpZM4b4pJV .  https://github.com/notifications/beacon/AuZ3dKbQnB4NVMhj2YN3s_ugw6Hi1h11ks5vX6L7gaJpZM4b4pJV.gif

I did some testing and found it was the AutoPostBack property set to True on the DropDownList control, which I need, so have implemented a workaround for the time being.
Thanks for your response.
Kind Regards,
Roger.
I agree the demo doesn’t produce the rapid refreshing and works as intended.
My implementation is much more complex than that reading a database and all wrapped up in a user control.
(It has been working for over 5 years – first complaint was when a user upgraded Firefox to 65.0.1)
My implementation uses:

In the master page which is the same as the CascadingDropDown demo https://ajaxcontroltoolkit.devexpress.com/CascadingDropDown/CascadingDropDown.aspx  uses.
When I get time, I shall strip the control down bit by bit until it works in an effort to identify the problem.
Kind Regards,
Roger.
From: AlekseyMartynov<EMAIL_ADDRESS>Sent: Tuesday, 19 March 2019 1:30 AM
To: DevExpress/AjaxControlToolkit<EMAIL_ADDRESS>Cc: RogerMCA<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [DevExpress/AjaxControlToolkit] CascadingDropDown extender control continally refreshing in Firefox browser 65.0.1 (#477)
How to reproduce this?
I'm using Firefox, and the CascadingDropDown demo https://ajaxcontroltoolkit.devexpress.com/CascadingDropDown/CascadingDropDown.aspx  doesn't send any excessive requests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/DevExpress/AjaxControlToolkit/issues/477#issuecomment-473933495 , or mute the thread https://github.com/notifications/unsubscribe-auth/AuZ3dICS1mnCSUhRPlHkljpR10aw_Yudks5vX6L7gaJpZM4b4pJV .  https://github.com/notifications/beacon/AuZ3dKbQnB4NVMhj2YN3s_ugw6Hi1h11ks5vX6L7gaJpZM4b4pJV.gif

@RogerMCA: I have the EXACT same issue. Could you please post your workaround?

That control has a special check for Opera and Safari which should also apply to Firefox.
Try the following:

Add the AjaxControlToolkit.StaticResources nuget package to your project
Modify line of 340 of Scripts\AjaxControlToolkit\Debug\CascadingDropDown.debug.js

from
if (list && (Sys.Browser.agent !== Sys.Browser.Safari) && (Sys.Browser.agent !== Sys.Browser.Opera)) {
to
if (list && (Sys.Browser.agent !== Sys.Browser.Safari) && (Sys.Browser.agent !== Sys.Browser.Opera) **&& (Sys.Browser.agent !== Sys.Browser.Firefox)**) {

Do a similar modification to the Release version of the same file.

@manuelra
Thank you. I'm reopening the ticket.

As the author said it's because of the "AutoPostBack" property set to true. I replaced it with the following client script called onchange of the dropdown, which will simply do a postback when any value is selected:
`var lastId = null;
function doPostbackForDropDown(control) {
if (!control) return;
if (control.value != '' && lastId != control.value) {
	setTimeout('__doPostBack(\'' + control.name + '\',\'\')', 0);
	lastId = control.value;
}

}`
Version used of ACT: 7.0607

To reproduce in the demo, remove AsyncPostBackTrigger:
--- a/AjaxControlToolkit.SampleSite/CascadingDropDown/CascadingDropDown.aspx
+++ b/AjaxControlToolkit.SampleSite/CascadingDropDown/CascadingDropDown.aspx
@@ -42,7 +42,6 @@
             <asp:Label ID="Label1" runat="server" Text="[No response provided yet]" />
         </ContentTemplate>
         <Triggers>
-            <asp:AsyncPostBackTrigger ControlID="DropDownList3" EventName="SelectedIndexChanged" />
         </Triggers>
     </asp:UpdatePanel>
 </asp:Content>

Then select a make and a model.

We fixed the issue in the master branch. Check the 'Updating the Project from CI Builds' guide to learn how to update the project before an official AJAX Control Toolkit release is available.


Implement ability for project properties to suppress and elevate warnings

Introduce MSBuildTreatWarningsAsErrors property that when set to true will treat all warnings as errors when building a project.
Introduce MSBuildWarningsAsErrors property which is a semicolon delmited list of warning codes to treat as errors when building a project.
Introduce MSBuildWarningsAsMessages property which is a semicolon delimited list of warning codes to treat as low importance messages.
This allows users to control warnings at the project level via standard MSBuild properties.  They can be set in a project, an import, or from the command-line.
The limitation here is that the warnings can only include ones that occur during a build.  This is because the properties are not read until after parsing so warnings generated during parse/evaluation happen too soon.  For these warnings, users will only be able to treat them as errors/messages with the /WarnAsError and /WarnAsMessage command-line arguments.
Closes #1886

FYI @jaredpar

Nice!


Card auto fill broken on Chrome iOS

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/generous-community
Click on pay
Scroll down to card details

Pressing your card information doesn’t result in entering data into the form, there’s no response.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: iPhone XS
OS: iOS 13.3.11
Browser Chrome
Version 80.0.398795

Additional context
You need cards saved in chrome to see this feature

A $100 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $100 when the Pull Request is reviewed, accepted and merged. More info
This bounty is currently reserved to participants of the Open Source Festival event. If not completed there, it will be later open to everyone.

It's now open to everyone.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/generous-community
Click on pay
Scroll down to card details

Pressing your card information doesn’t result in entering data into the form, there’s no response.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: iPhone XS
OS: iOS 13.3.11
Browser Chrome
Version 80.0.398795

Additional context
You need cards saved in chrome to see this feature

@chmac Could you add a screenshot?

@leonardohof Unfortunately I don't have credit cards in chrome on my iPhone. It was a friend trying to support our campaign at an event who told me about it, without any other cards on hand, there was no way for them to support the campaign. I'm asking if they can forward a screenshot, but I guess it's the absence of something happening rather than the presence of a problem, so it might be hard to get a screenshot which captures this essence. It's hard to take a picture of something that doesn't exist, etc...

Hi @znarf @chmac , looks like this isn't straightforward to reproduce. I tried with a card I had saved to the page mentioned ( https://opencollective.com/generous-community ) with my phone on:

Device: iPhone 8 (not same phone as mentioned)
OS: iOS 13.3.1
Browser Chrome
Version 80.0.3987.95
which in my opinion is quite similar to the device browser described and the autofill worked successfully as shown in the screengrab below.

@brymut Thanks for investigating this so thoroughly. I'm gonna close this issue now on the basis that it was probably a one off, or at least hard to reproduce, issue. Impressed with the depth of investigation and feedback here. Thanks. ❤


Fix doc bug for LDAPS access from internet.

#sign-off

Leave a comment

Moving fix to private repo.


[WIP] update init of qiskit.pulse

Summary
This adds all necessary modules to create Schedule in qiskit.pulse.__init__
Details and comments

@itoko Please add comments if we need to change module names. And when need some more imports.


jsonld/sign endpoint cannot handle unicode characters

If I want to sign a verifiable presentation that contains none ascii characters I get the following error response:
'ascii' codec can't encode character '\\xfc' in position 576: ordinal not in range(128)
Steps to reproduce:

Get your verkey e.g. from /wallet/did/public
Insert the verkey into the json below, and post ist to /jsonld/sign

{
  "verkey": "myVerkey",
  "doc": {
    "credential": {
      "@context": [
        "https://www.w3.org/2018/credentials/v1"
      ],
      "type": [
        "VerifiablePresentation"
      ],
      "verifiableCredential": [
        {
          "@context": [
            "https://www.w3.org/2018/credentials/v1",
            "https://raw.githubusercontent.com/iil-network/contexts/master/masterdata.jsonld"
          ],
          "type": [
            "VerifiableCredential",
            "MasterDataCredential"
          ],
          "credentialSubject": {
            "type": "LegalEntity",
            "legalName": "Würthemberger Test Corp"
          }
        }
      ]
    },
    "options": {
      "verificationMethod": "something"
    }
  }
}

https://github.com/hyperledger/aries-cloudagent-python/pull/687 addresses


chore: Test Runners [DO NOT MERGE]

Description

Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Nix doesn't seem to be working on aarch64-darwin mac CI machines -- going to terminate this experiment as x86-64 still takes a long time


No shared state across components using SSR syntax ($store.$db().model())

I'm having a strange issue with the SSR syntax since switching from the ES6 static import syntax.  My components each seem to be accessing a difference instance of the store bizarrely, see very simplified example:
QuoteList (parent):
computed: {
  Quote() {
    return this.$store.$db().model('quotes');
  },
},
mounted() {
  this.Quote.insertOrUpdate({
    data: {
      name: 'test1',
      company_id: 555,
    }
  });

  const records = this.Quote().all();
  console.log(records); // [{ name: 'test1', company_id: 555 }]
}

Quote (child):
computed: {
  Quote() {
    return this.$store.$db().model('quotes');
  },
},
mounted() {
  console.log(this.Quote.all()); // [] (EMPTY)
}

Is this intended for the SSR way?  Seems to defeat the object of using Vuex in the first place.
Using within Quasar framework.

This is a duplicate of #694. Which do you want to keep?

This is a duplicate of #694. Which do you want to keep?

Apologies, I got a 404 on submit so assumed it hadn't sent, have closed the other issue.

The code above is obviously very simplified, in reality it's a large scale app with 20+ models but it's happening in numerous places.
This particular example is basically a data table with 50 rows (aka QuoteList), each row clickable which brings up the child (actually a spawned dialog) with the singular information for the row.
I've added some logging to demo this, but you can see things are loaded in the right order and populated correctly etc.  Is there anything I can log which could identify a given instance of a database/store?

This was caused by exporting the store as a function in Quasar framework as opposed to exporting the store itself, nothing to do with using the SSR syntax, sorry for the inconvenience.


We auto create role

Summary

Checklist

[ ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

[ ] Have you tested the changes? (pytest & dpytest)
[ ] Have you followed PEP-8 for naming and styling?
[ ] Has your code been properly documented with RestructuredText docstrings?
[ ] Have you added your changes to CHANGELOG.md under the [Unreleased] heading?
[ ] If your code added new bot commands, have you updated documentation.json?

[ ] All of this code is your own, or follows the author repo's licence.

To add onto the testing portion, have a look at the automated tests made on python 3.7 as some of the new tests failed. Another issue is the code coverage but this can be rectified by ensuring your tests cover as much as possible.


Installation as a standalone project : Could not find package forikal-uk/ping-drive with stability stable.

Steps to Recreate
Look at the instructions to install as a standalone project.
I go to my projects folder
$ pwd
/Users/jw/Documents/Projects/XmlAuthoring

Create a directory as destination.
$ mkdir ping-drive

Then get composer to create the project:
$ composer create-project forikal-uk/ping-drive ping-drive

What occurs
Results in:

  [InvalidArgumentException]
  Could not find package forikal-uk/ping-drive with stability stable.

create-project [-s|--stability STABILITY] [--prefer-source] [--prefer-dist] [--repository REPOSITORY] [--repository-url REPOSITORY-URL] [--dev] [--no-dev] [--no-custom-installers] [--no-scripts] [--no-progress] [--no-secure-http] [--keep-vcs] [--no-install] [--ignore-platform-reqs] [--] [<package>] [<directory>] [<version>]

Expected
I expect no errors.

Here are my notes as I experienced the issue.

Maybe this can be fixed by pushing a new tag to packagist or adding a line into the composer.json ?

A repository needs to be published to Packagist to be available through Composer.
You have published the repository so the issue is resolved.

Thanks.


Added target for object file

Useful for use in bigger non-makefile (cmake/meson) projects

Sorry, GitHubs web IDE ruined that tabulator

Would you mind adding a comment explaining that it's not used in the bare-bones tree but can be used in the other applications you were thinking?

Err I can just accept and add.

Fixed in main.


Nuxt DevTools throw error

Actual Behavior
Nuxt DevTools only show white page and error Uncaught ReferenceError: __NUXT_ASYNC_CONTEXT__ is not defined is shown in browser console

Expected Behavior
Nuxt DevTools work without error
Reproduction
https://stackblitz.com/edit/nuxt-starter-wvxxjo?file=nuxt.config.ts
Dependencies
 "nuxt": "^3.9.3",
  "@vue-email/nuxt": "^0.8.10"

can you please try 0.8.11

can you please try 0.8.11

Thanks, it works with 0.8.11. 🚀💪🏻


Observable of FDB Signals Targets

Future work includes further data analysis to identify additional patterns and insights, the development of a trading strategy based on the analysis findings, and the backtesting and live trading of the strategy to evaluate its performance. The script serves as a valuable tool for improving trading strategies and decision-making by providing insights into the effectiveness of SELL signals generated by the "Fractal Divergent Bar" indicator within the "Big Alligator" tool.

Adding the Tide Alligator Indicator and New Backtesting SignalsOverviewThe plan is to introduce a new Alligator indicator, called the "Tide Alligator," and incorporate new backtesting signals for fractal divergent bars. This will involve using three columns for the Tide Alligator: tide lips (tlips), tide teeth (tteeth), and tide jaw (tjaw).OpenAI Platformplatform.openai.comOn May 28, 2024, at 23:04, Guillaume Descoteaux-Isabelle @.***> wrote:
Future work includes further data analysis to identify additional patterns and insights, the development of a trading strategy based on the analysis findings, and the backtesting and live trading of the strategy to evaluate its performance. The script serves as a valuable tool for improving trading strategies and decision-making by providing insights into the effectiveness of SELL signals generated by the "Fractal Divergent Bar" indicator within the "Big Alligator" tool.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


[release-1.26] Packaged Traefik 2.9.4 has incorrect access logs

Backport #7301

Validated on commit e8d9ed79841859a7ecf70039416e1af5b7a91bdb
$ k3s -v
k3s version v1.26.4+k3s-e8d9ed79 (e8d9ed79)
go version go1.19.8

Environment Details

Infrastructure

[ ] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.2 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Cluster Configuration:

1 server

Testing Steps

Install the k3s using commit
Look for the Traefik 2.9.4 & Chart 21.2.0

Validation Results:

$ kubectl get node,all -A -o wide | grep "2.9.10"
kube-system   deployment.apps/traefik                  1/1     1            1           10m   traefik                  rancher/mirrored-library-traefik:2.9.10   app.kubernetes.io/instance=traefik-kube-system,app.kubernetes.io/name=traefik
kube-system   replicaset.apps/traefik-57c84cf78d                  1         1         1       10m   traefik                  rancher/mirrored-library-traefik:2.9.10   app.kubernetes.io/instance=traefik-kube-system,app.kubernetes.io/name=traefik,pod-template-hash=57c84cf78d

$ kubectl describe pod/traefik-57c84cf78d-k9nm7 -n kube-system | grep "traefik"
Name:                 traefik-57c84cf78d-k9nm7
Service Account:      traefik
Labels:               app.kubernetes.io/instance=traefik-kube-system
                      app.kubernetes.io/name=traefik
                      helm.sh/chart=traefik-21.2.1_up21.2.0
Controlled By:  ReplicaSet/traefik-57c84cf78d
  traefik:
    Image:         rancher/mirrored-library-traefik:2.9.10
    Image ID:      docker.io/rancher/mirrored-library-traefik@sha256:aaec134463b277ca7aa4f88807c8b67f2ec05d92a8f0432c0540b7ecc8fe724a
      --entrypoints.traefik.address=:9000/tcp
      --providers.kubernetesingress.ingressendpoint.publishedservice=kube-system/traefik
  Normal  Scheduled  11m   default-scheduler  Successfully assigned kube-system/traefik-57c84cf78d-k9nm7 to ip-172-31-33-229
  Normal  Pulling    11m   kubelet            Pulling image "rancher/mirrored-library-traefik:2.9.10"
  Normal  Pulled     11m   kubelet            Successfully pulled image "rancher/mirrored-library-traefik:2.9.10" in 2.97682403s (2.976831523s including waiting)
  Normal  Created    11m   kubelet            Created container traefik
  Normal  Started    11m   kubelet            Started container traefik

$ helm ls -A
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /etc/rancher/k3s/k3s.yaml
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /etc/rancher/k3s/k3s.yaml
NAME       	NAMESPACE  	REVISION	UPDATED                                	STATUS  	CHART                      	APP VERSION
traefik    	kube-system	1       	2023-05-15 20:20:03.658372718 +0000 UTC	deployed	traefik-21.2.1+up21.2.0    	v2.9.10
traefik-crd	kube-system	1       	2023-05-15 20:20:00.331370837 +0000 UTC	deployed	traefik-crd-21.2.1+up21.2.0	v2.9.10


Hardhat: upgrade Solidity version

Currently, we are having Solidity 0.6.12 set for hardhat compilation options. We need to upgrade it to Solidity ^0.8

I don't know if I understand your question, but I know that you can explicitly set solidity version in the in config file like this...
module.exports = {
solidity: "0.8.1", // or any other version
};
And inside of the solidity file you can set pragma solidity to ^0.8....


Mirror transport

Here's an attempt at #292. Apt has a "mirror transport" instead of http/https. In brief testing, it appears to allow more explicit configuration of mirror priority, and Ansible doesn't trip over itself when a mirror is unavailable.
If merged, closes #292

This breaks the Mint Update Manager, so it may not be usable.

This also seems to prevent Ansible from finishing the Packer build, so I'm inclined to scrap the idea at this point.


[Tool Removal] Game Controller Tester

This tool was useful originally to play around with Plug and Play Controllers on various desktop applications.
It should be removed from editor and turned into a standalone application/ tool and beautified some.

Winter 2024 as not priority and is a tool removal from editor to be placed in separate program.


Information missing in Aggregate and Disk dashboards when compared with 1.6

Is your feature request related to a problem? Please describe.
We are missing the disk utilization graph and highlight panel in the Aggregate dashboard like the harvest 1.6 has in the dashboard: NetApp Detail: Disk and Cache Layers. Additionally we will be interesting to recover the graph for User Read/Write IOP Size, this information could feed the dashboard of Disk, like happened in the past with this information which was in the dashboard: NetApp Detail: Disk and Cache Layers.
Also we missed other relevant information for us in the Cluster Dashboard and is the highlight
Describe the solution you'd like
Add new information for the dashboard for Aggregate and Disk.
For the dashboard Aggregate:
- Disk utilization
,for the Disk dashboard:
- User read/write IOP Size
and the last in the Cluster dashboard:
- Highlights
Additional context
I can do add the information for the disk utilization by my self but the problem is everytime a new release is coming, I will lose the changes that I did in my dashboard since I upload again the new panels with the information, changes or something do it by you. In the case of the User Read/Write IOP Size I don't know if I have that information in my current instance of harvest. Refer to the Cluster highlights is the same issue that the disk utilization.

Disk utilization and highlight of disk utilization too.

User Read/Write IOP Size

Cluster highlight

For the Disk utilizationpanels, we already have them in Disk dashboard.
Above right table has instant data whereas below left panel has time range data.
Earlier, topk wasn't supported in them, Now made the appropriate changes to support topk.

For the User read/write IOP Size panels, Added them in Disk dashboard just below the Disk utilization panel.

For the Highlights panels, Added them in Cluster dashboard.

Hi @faguayot

Understood your perspective. I would leave this panel in Disk dashboard as-is and I have added new panels in Aggregate dashboard which would be aggregation at aggr level and not MAX.

This is PR: https://github.com/NetApp/harvest/pull/1320/. It has dashboard changes along with minor template change. So, you would need to do both in your system for local testing.

It's same Cluster dashboard. Just added these panels in Highlights, rest all panels are as-is.

I wouldn't say they are wrong, just they would change unit based on the latency value. If the value > 1000 then grafana would show in s(sec) and not ms(millisec)

hello @Hardikl
Checking again the units for the graph, I think those are in a incorrect units. We don't have any perfomance issues in this cluster or others and 1minute of latency is too much. We haven't seen this metrics never.

thanks for the follow-up @faguayot. I wanted to make sure, you running nightly with Hardik's changes?

Hello @cgrinds
No, I am not running the nightly build but now that I am sure that the unit in this dashboard for the 22.08 version and in the Top Latency was wrong, so I corrected by myself.
Thanks.

Thanks again for bringing to our attention


[oap-native-sql][Scala] Adding metrics for input/output batches

Adding metrics for input/output batches
Signed-off-by: Yuan Zhou<EMAIL_ADDRESS>

Thanks. LGTM.


device_handle of LoadedImage is private

AFAIK getting SimpleFileSystem protocol needs device_handler of LoadedImage, but it's not marked as public. Is there another way to get SFS protocol?
fn prepare_file_system(image: &Handle, system_table: &SystemTable<Boot>) -> file::Directory {
    let loaded_image = system_table
        .boot_services()
        .handle_protocol::<loaded_image::LoadedImage>(*image)
        .expect_success("Failed to load image");

    let simple_file_system = system_table
        .boot_services()
        .handle_protocol::<fs::SimpleFileSystem>(loaded_image.into_inner().device_handle) // error: field `device_handle` of struct `uefi::proto::loaded_image::LoadedImage` is private
        .expect_success("Failed to prepare simple file system.");

    let simple_file_system = unsafe { &mut (*simple_file_system.get()) };

    simple_file_system
        .open_volume()
        .expect_success("Failed to open volume.")
}

None that I know of. AFAIK, it is safe to read device handles, but not to write to them, so you may want to resolve this by simply adding a read-only accessor to LoadedImage that returns the device handle.

I've added a getter for the respective handle in 522b7287addc52e3aba940ccf0a5ab5dd3b79955. Will also publish a new version of the crate with this change.


Fix validation failure when uploading to AppStore in Xcode 15.3

Fixes #19

Waiting for https://github.com/yeatse/opencv-spm/actions/runs/8619452675


feat: use CDN location for script

FullStory is now hosting the FullStory script (fs.js) on Google’s Content Delivery Network (CDN) and wants users to use that new location. More info: https://help.fullstory.com/hc/en-us/articles/360041242094

Related to issue #20

Closed via a different PR.


KL loss compare to Keras

Thank you the nice tutorial and supporting code. I made a plot (attached) of KL Loss vs iterations of your implementation and that of Keras (blog, code). Could you please provide insight as to why the KL loss for your implementation is going up?

Not immediately evident based on just the plot--although one big difference is the number of latents.  Initialization can also play a role.  Note that the KL loss going up simply means that more information is being encoded in the latents; as long as the error is going down more than the KL loss goes up, then the VAE is learning the distribution.  In fact, this is often the behavior you want from a VAE, since you expect the VAE to encode more in its latent variables over time.


Distinguish extension body in new tabs

This PR adds styling to distinguish the extension body in new tabs:

add box-shadow to root element
set root element height to also be 600px

I am not sure why the root element was set to be 2px less than the body, I've changed it to match the body at 600px. Let me know if it is a requirement, and I can change it back!

Thanks for the initiative @wahidrahim. I believe the plan for the full screen display, was to:

have the wallet stretch to fill window height completely.
and for gradient color of header to occupy full window width.

Can we at least do 1 in this PR.

Thanks for the details @remon-nashid that's a much nicer approach!
Drafting this PR for now until the requirements are met.


Wd14 tagger re-runs the same image

Why is the tagger re-running even if I keep the same image loaded (already tagged in a previous run)? Is there a way to prevent it?

i think this problem is important,
if resolved, can save lot of time when using workflows, especially the ones have lots of nodes was set after the wd14 tagger.

i think adding "
@classmethod
def IS_CHANGED(s, image):
    image_path = folder_paths.get_annotated_filepath(image)
    m = hashlib.sha256()
    with open(image_path, 'rb') as f:
        m.update(f.read())
    return m.digest().hex()

"
to line 204 of "E:\ComfyUI_Mini-1\ComfyUI\custom_nodes\ComfyUI-WD14-Tagger\wd14tagger.py" will work.
i will try it later.

Tried to implement IS_CHANGED but the image always comes None in the argument; also tried to store the image hash when tag is called but when IS_CHANGED is called, it's also None.
Found out that apparently the back-end isn't authoritative, removing this code from the wd14tagger.js solved for me
https://github.com/pythongosssss/ComfyUI-WD14-Tagger/blob/262cfdbd16d5892c0a8fcb2d61439610854aeaea/web/js/wd14tagger.js#L96-L101

This should be fixed


Feedback for Lab 1.4

Clean up developer lab language to be clearer.

Converting to draft while I figure out the USER_NUMBER thing.

Fixed the USER_NUMBER issue by oc whoami and then using sed.  I just tested this in my cluster and looks like it works fine.


Lowercase Paths during Lookup

Hi All,
I came across an issue where any conda environments with uppercase characters were not able to start a kernel in Jupyter. Eventually I found out that it is due to nb_conda_kernels is making the path lowercase.
In manager.py, line 218:
kernel_dir = dirname(spec_path).lower()

When I manually remove the .lower() portion - things start working as expected. Because I am on a case-sensitive OS as part of a HPC cluster (Ubuntu Linux), I was confused to find this.
Is nb_conda_kernels intended to be only used on Windows or other case-insensitive systems? Either way, even in a case-insensitive OS, mismatched uppercase/lowercase chars shouldn't stop the application from working. Is this something that should be removed? Happy to submit a PR if so, but it feels like I'm missing something so I'll wait for now.

Same here.
Removing the .lower() portion fixes linux functionality when path contains uppercase letters.


[Feature] Generate openapi schema without running server

Is your feature request related to a problem
Not a bug, just a problem with my CI system.
I want to be able to programmatically access the openapi schema and save it to a file without running the web server
The solution you would like
Ideally, I'd like to write a script that would output the openapi schema without running the webserver.
from myapp.api import api
json.dump(
  get_schema_from_app(api),
  open('openapi_schema.json', 'w')
)

Describe alternatives you've considered
I've considered running the app in my CICD workflow and requesting the schema via http, but that seems like a lot of overhead, and I'd bet the functions to do what I want are already in the code and just need to be invoked in the right order.
Additional context
Nothing to add, other than I'm gonna open a PR and see if I can figure it out.

Duplicate of #1173.

@phy25, thanks for the link, it was exactly what I needed. I'll close out.

Thanks for the help here @phy25 ! :clap: :bow:
Thanks for reporting back and closing the issue @BenDavidAaron  :+1:


Add custom resource name to BOSH connection

Hi.
Thanks for a great little library.  It has really helped me out in setting up Rails and XMPP.
I needed the ability to set the resource name when initialising the BOSH connection.  I have added the simple code to make that happen.  You just need to add the name at the end if the jid like so:
<EMAIL_ADDRESS>This is the same way Strophe works when creating a connection.  I also updated the Readme to include the above example.
Thanks again.
Kind Regards
Hamza Khan-Cheema

Thanks!


Example

I'll be glad to have some examples to test your library ;)

Hi there !
I´m so glad you want to use this library despite its early stage. However, I´m afraid there´s still a lot work to be done before this library could be shared. Maybe in a near future.

Looks like everything is doing well, I'm still looking at this lib ;)

Mee too,. it sounds very abitious and interresting!

@fabiomsr  - I attempted to email you as well.  I've made some significant headway in my path_measure branch.  I'm still waiting for a PR in engine to land that adds some features this needed to get moving forward.  Are you still looking to maintain this project?

After discussing with @fabiomsr, I've taken on maintaining this.  It's been published to pub based on my fork.  There's an example of usage in that repository.


update 500 page http://pre-live.jncc.gov.uk/error-page.html

this page needs its footer updating to be the same as the live site
the google tag manager code needs to be added - same tag manager code that is in the current site
and the text needs to be changed to:
Sorry we were unable to complete your request but our technical staff have been notified of the issue and will work to resolve it. If the problem persists please feel free to contact us.
Note: contact us needs to be a link to /contact/?ref=500
also could you confirm how the technical staff are going to be notified of this error

http://dev-website.jncc.gov.uk/

Hi @CathyJinJNCC happy for you to deploy to live


require-default-props without prop-types

Is it possible to add new rule for to use require-default-props without checking prop-types?
We have rather big project with defaultProps and it would be very sad to add redundant prop-types everywhere.

How are they redundant?
propTypes tells React what kind of thing to expect (if it's required, or if it's not undefined); defaultProps tells React what to substitute if the prop provided is undefined.

In a lot of cases list of default props is good enough. When really many components already has default props described it is not so necessary to spend time on propTypes.
But eslint do nothing when some of them changes and it is sad for me

In no cases do I think defaultProps is good enough - without propTypes, the wrong value can be passed and you'll never know it.
It is always necessary to spend time on propTypes. It would be far better to never use defaultProps, then to ever omit propTypes.


File ".js" extension is not being resolved

The issue
Code works during dev, but fails during SSG build. The culprit is the missing file extension when importing.
Stacktrace
[vite-ssg] An internal error occurred.
[vite-ssg] Please report an issue, if none already exists: https://github.com/antfu/vite-ssg/issues
(node:54643) UnhandledPromiseRejectionWarning: Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/alexgrozav/Workspace/inkline.io/node_modules/@inkline/inkline/components/IAlert/manifest' imported from /Users/alexgrozav/Workspace/inkline.io/.vite-ssg-temp/main.mjs
Did you mean to import @inkline/inkline/components/IAlert/manifest.js?
    at new NodeError (internal/errors.js:322:7)
    at finalizeResolution (internal/modules/esm/resolve.js:308:11)
    at moduleResolve (internal/modules/esm/resolve.js:731:10)
    at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:842:11)
    at Loader.resolve (internal/modules/esm/loader.js:89:40)
    at Loader.getModuleJob (internal/modules/esm/loader.js:242:28)
    at ModuleWrap.<anonymous> (internal/modules/esm/module_job.js:76:40)
    at link (internal/modules/esm/module_job.js:75:36)

This didn't happen in 0.16.2.

@alexgrozav the new version 0.17.2 by default is esm (also supports cjs) while 0.16.2 is only cjs, so you should add the extension. You can try changing the format to  cjs on ssgOptions but there is a problem with the binary vite-ssg and maybe build will not work, I have fixed it on #160 (and a few other things):

@alexgrozav try with cjs format with the latest version released

Thank you for the heads up! I'm updating the package to use esm properly.

how to solve it :  Unknown file extension ".vue"


Create User model

Create the user database model in the models > user.model.js file. The information regarding the same is available in the README.md file.

claim

Congratulations on your first assigned issue. You have 48 hours for this task.
Happy Contributing!!

claim

@Aman1143 Issue has already been assigned

@BuddyLongLegs Should I do some basic validation for name and email like required/trim or just the basic structure for the model is acceptable??

Sure, doing so is recommended


AE49_181106_213349.529

Type: Missing
Summary:
License
Details:
Add MIT
Resolution:
https://jquery.org/license/
Affected definitions:

jQuery 2.1.3

This is under both since this package contain both jQuery files, that are licensed under MIT, and custom code that is licensed under BSD-3-Clause per https://nugetpackages.codeplex.com.


rollup throws a lot of warnings

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
Just wondering why is rollup throwing so many warnings that don't make sense:
><EMAIL_ADDRESS>ionic:serve /Users/rolandjitsu/Projects/infarm/operator
> ionic-app-scripts serve

[14:49:53]  ionic-app-scripts 0.0.45
[14:49:53]  watch started ...
[14:49:53]  build dev started ...
[14:49:53]  clean started ...
[14:49:53]  clean finished in 6 ms
[14:49:53]  copy started ...
[14:49:53]  transpile started ...
[14:49:57]  transpile finished in 4.10 s
[14:49:57]  rollup started ...
[14:49:58]  copy finished in 4.24 s
[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports 'TextAst'
            from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'BoundTextAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports 'AttrAst'
            from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'BoundElementPropertyAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'BoundEventAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'ReferenceAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports 'VariableAst'
            from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports 'ElementAst'
            from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'EmbeddedTemplateAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'BoundDirectivePropertyAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'DirectiveAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports 'ProviderAst'
            from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'ProviderAstType' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'NgContentAst' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'PropertyBindingType' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:00]  rollup: Conflicting namespaces:
<EMAIL_ADDRESS>re-exports
            'templateVisitAll' from both
<EMAIL_ADDRESS>            (will be ignored) and
<EMAIL_ADDRESS>[14:50:10]  rollup finished in 12.91 s

It basically tells that two files (which is actually the same file - the strange part) export the same namespace and then it gets imported in index.
This might not be the place to create this issue, but I want confirmation that it's not some config in config/rollup.config.js that causes this.
What behavior are you expecting?
It should not throw any of these warnings.
Steps to reproduce:
1.
2.
3.
insert any relevant code between the above and below backticks

Which @ionic/app-scripts version are you using?
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Note the the production build is actually not throwing any of these.

This is an issue Angular introduced. Should be fixed in the next Angular release. I'm going to close this for now as there's nothing we can do. We could swallow the warning but we don't think that's a good idea at this time.
Thanks,
Dan

I see, maybe it is already fixed as Ionic 2.0.0-rc.3 is still fixed on Angular 2.1.1 and Angular has a few more releases since then.


Asks "Do you want to allow this app from an unknown publisher to make changes to your computer?" every single time

Windows 10 prompts me with this question every single time. If I highlight multiple folders and change their color all at once, it will ask me once for each folder, one after the other.
If that were solved, this app would be perfect!

Thanks. That's the User Account Control (UAC) dialog box that comes up because Folcolor needs to run as administrator.
Maybe there is a way to not have the UAC dialog box not come up, like signing the executable a special Microsoft way.
Other possibilities are using some sort of hacks/exploits which I don't want to use.
What you can do, at least temporary, if you have a lot of folders you want to set. Search the Windows/Cortana box for "UAC" and run the settings. If you drop it all the way down you won't get the notification anymore. But this is not so good long term because it's good to know when programs want to run as admin. You should comeback later and set it to at least the last of the four levels so you'll get notified again.


Contribution guide

Hi guys, seems awsome and I'd love to contribute my few cents.
Is there a recommended workflow for contributions? I'd like to ask b4 I start working on a feature.
Right now I have some refactoring ideas. Would you prefer discussing them in advance or should I just make the changes and discuss them in a PR?
Mainly I would

rename the methods of the HttpHelper class from camelCase to pep8_case
make the Client class a sublcass of the HttpHelper to avoid the awkard calls such as client.http.get_something() (The cleaner version would be client.get_something().
some minor changes (like turning the HttpHelper.tokenheader() into a @property. Imo it makes more sense to use it like HttpHelper.tokenheader

This would change the api alot and break lot of existing code so I'm asking in advance if you are ok with that.
That's what I got just through quickly skimming the source, feel free to disagree with anything :beer:

Hi @pocin , welcome!
As far as contribution workflow, I think we can probably at this get away with just using the PR
comment/review system.
I'm not a python person per se (as is probably obvious by the source:) so there isn't really any strict way I have in mind of how this client should be, other than that it should be a client for the keboola storage api (http://docs.keboola.apiary.io/#).
As for your 3 suggestions above, yes, yes, and yes:)   and thank you!
Looking forward to making this thing pleasant and useful together, cheers 🍺

fwiw, I tagged it as 0.0.1 so we can properly version for breaking changes.

i'd just add that it would be nice to keep the method names (and arguments) aligned with the API, and ideally with the PHP client https://github.com/keboola/storage-api-php-client/blob/master/src/Keboola/StorageApi/Client.php so that there is a e.g a listBuckets (or list_buckets to be PEP8) method and not getBuckets method.

@pivnicek (I suppose you have write access to this repo) - can you make a dev branch so we don't have to merge directly into the master? Once the dev is mature enough, we can "release" a new version.

@pocin you should have an invite link in the mail to join the repo with write.  I've also sent it to you in a slack PM just in case.

FYI: i made a couple of issues and also a milestone 1.0.0 - i consider the issues for that milestone to be important for real reliable production use. If i forgot something, feel free to add. I do not consider full API coverage to be important for milestone 1.0.0, but maybe some parts might be worth?

@odinuv, I notice you use a nice commit message structure, like tag: description. Do you have documentation for that style of commit message? I might try using it too.

I tihnk I'm using these https://seesparkbox.com/foundry/semantic_commit_messages I'm not overly strict about it thoug :)

materialized in a361476b0c9b867220750eccd4278ddde94864ab


Fixes links to API documentation

Hi folks, I noticed the links to API documentation were giving 404s, so I've updated to hopefully appropriate new URLs.

Thanks for your helping in keeping out documentation up-to-date


How to quote your work?

Referring to this https://github.com/cerlymarco/linear-tree/issues/13
From my point of view, you can simply cite the project reporting the link and the name of the library.
All the best


feature request: Set config and cache flags via env variables

following up the thread in #226
(created for tracking)
be able to set --config and --cache flags using environment variable since it would allow users to set them in a Dockerfile:
ENV DEVBOX_CONFIG=.decontainer/.devbox.json
ENV DEVBOX_CACHE=/devbox/.devbox

please keep open


nixos/acme: Remove an incorrect assertion from tests

Motivation for this change
nixosTests.acme was failing, because commit 3a2e0c36e79cecaf196cbea23e75e74710140ea4 has removed
--reuse-key from default renew options, yet the tests still expected keys not to change.
Things done
This assertion is now removed, as they are supposed to change on each renew/change.
There is no assertion to replace the removed one, because I don't think there's anything to replace—the test already checks that the new certificate works, so e.g. comparing that the hashes are different doesn't make much sense.

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test acme

FWIW: Should have looped in the ACME team to consider backporting this.


test_select sporadically fails on x86-64 Linux

Just randomly hit this after the test seemed to get stuck:
thread 'sys::test_select::test_select' panicked at 'assertion failed: `(left == right)` (left: `1`, right: `0`)', test/sys/test_select.rs:52

Please post this to #679.


how to convert the result to bvh file?

does there any instructions?

SMPL and SMPLX authors shall know better...
But I did find this project and I think you can dig in their codes to change smpl to bvh.

@jinfagang this may help you

oh, thank u for the info!


[f8a] Update the list of deployment units and configs for workers

Downstream job which runs E2E tests needs to know which units from saas-analytics should be deployed and which deployments to rollback, if the tests failed.
This PR adds units/configs which are not new, but were previously missing.

@kbsingh the best way is to merge it, immediately test it and fix it, if it doesn't work :) Worst thing that can happen is that all master branch builds for workers will start failing.

@msrb lets sync up in the AM and merge it together, test and handle case

Friday afternoon, the ideal time to break f8a CI. Let's do this :) (I am serious) I can fix it if needed and we won't disrupt people's work.

Worst thing that can happen is that all master branch builds for workers will start failing

I'm not completely sure this PR is causing it, but E2E test have been failing since yesterday with:
Error from server (NotFound): deploymentconfigs.apps.openshift.io "bayesian-worker-priority-dispacher" not found

I'm not completely sure this PR is causing it, but E2E test have been failing since yesterday

That's only E2E for workers, you are right. The other failures (majority of them) look unrelated:
Failing scenarios:

  features/component_search.feature:27  Check the component search functionality for existing component from the npm ecosystem

  features/component_search.feature:35  Check the component search functionality for existing component from the pypi ecosystem

  features/component_search.feature:43  Check the component search functionality for existing component from the maven ecosystem

  features/gremlin.feature:49  Basic check for Gremlin ability to search for given package in any versions for the Maven ecosystem

  features/gremlin.feature:234  Basic check for Gremlin ability to search for given package in the Maven ecosystem


Wrong ontology for norbornene and premutilin

These relationships are wrong because alkenes are acyclic while norbornene and premutilin are cyclic.
norbornene (CHEBI:52286) is a alkene (CHEBI:32878)
premutilin (CHEBI:142455) is a alkene

Thanks. Now corrected.


decomp: finish misty-teetertotter | misty-warehouse

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.998%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test/offline/offline_test_main.cpp
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
39298

Relevant Lines:
57793

💛  - Coveralls