Exposing container image registry

To push container images into the OpenShift environment of CRC, we need to expose the container registry.
Ref: https://docs.openshift.com/container-platform/4.1/registry/accessing-the-registry.html

We are now already exposing it but need to put document around how to access it.

This does not work with CRC:
$ oc get nodes
NAME                 STATUS    ROLES           AGE       VERSION
crc-2n9vw-master-0   Ready     master,worker   19d       v1.14.6+6ac6aa4b0
$ oc debug nodes/crc-2n9vw-master-0
error: cannot debug crc-2n9vw-master-0: unable to extract pod template from type *v1.Node
I know that this is not a CRC issue but nobody seems to take notice although an issue has been opened: https://github.com/openshift/origin/issues/23371

closing this issue since we are already exposing the registry.

Hi, just a small comment about this: in most cases, developers who are using CRC will not want to access the registry from within the cluster, but form their own machines. Pointing to the documentation about accessing the registry from the cluster is really not that helpful. At least in my case I was looking for a quick and fast solution to build an image locally (in CRC's host machine) and push it to the OCP registry.

@lsalotto-ingenia with exposed registry you can do that, do you mean that it should be part of the document?

I have documented external access to registry in one of my projects: https://github.com/IBM/cloud-native-starter/blob/master/documentation/OS4Requirements.md#access-the-openshift-internal-image-registry
Its a little more complicated for CRC because it uses self-signed certificates. Maybe this is helpful?

@Harald-U Thanks for the steps you pointed out, since crc for local development you can just add that registry route default-route-openshift-image-registry.apps-crc.testing as insecure one, in podman there is already check --tls-verify which you can set to false and it works without any change, in case of docker it does require manual step to add the route as insecure registry one.


[WIP] Responsive css

Pull Request
Add more responsive for mobile devices
Description:
Self explanatory
Related Issue:
Closes #66
It is a WIP

Hey,
I feel we should add a hamburger for the nav links (just for mobile devices) but I am not very sure. Plus, this is my first Vue project. I haven't found adding CSS difficult but I am not sure how to add hamburger menu (or any other functional code) here. Any tips/help?

Hey,
I feel we should add a hamburger for the nav links (just for mobile devices) but I am not very sure. Plus, this is my first vue project. Styling not any issues but I am not sure how to add hamburger menu here. Any tips/help?

I can tell you I've done exactly this in this project https://github.com/Nexmo/deved-platform maybe check out the header component and see how I've done it :)


chore(release): v2.18.1

Problem
new patch release

Codecov Report

Merging #1114 (cefce3b) into main (6b2206a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##             main    #1114   +/-   ##
=======================================
  Coverage   93.74%   93.74%
=======================================
  Files         151      151
  Lines        6269     6269
  Branches     1858     1858
=======================================
  Hits         5877     5877
  Misses        373      373
  Partials       19       19

Continue to review full report in Codecov by Sentry.

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


Issue with documentation for "Migrations in Team Environments"

Maybe I'm wrong, but I believe that this section in docs is wrong and can cause issues: Migrations in Team Environments
We should not merge two migrations into main branch just like that, as the designer file of the later migration can be wrong, and using remove-migration command can revert the snapshot file to the wrong state.
There are at least two issues that were reporting this behavior #8880 and #23911

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

ID: 0fbabb60-978b-99b3-0376-b2af83765680
Version Independent ID: fcfb2618-1415-4173-5210-ddf06ec9342e
Content: Migrations in Team Environments - EF Core
Content Source: entity-framework/core/managing-schemas/migrations/teams.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: avickers

Related: https://github.com/dotnet/EntityFramework.Docs/issues/3560


ExpressionVisitor.visit(CustomExpression)

It would be nice to have a
void visit(CustomExpression ce)

in ExpressionVisitor such that custom implementations could be visited as part of the processing.
Useful for
List<Expression> newList = new ArrayList<>();
if (expression instanceof JdbcParameter)
   newList.add(new MyCustomExpression());
else
  newList.add(expression);

update.setExpressions(newList);

I am not sure if I understand you. You want one additional method that will only be called if one uses your custom class? What is the difference in overriding e.g. Expression visitors Expression visit method? I don't see any advantage.

The Update.getColumns() and Update.getExpressions() isn't visited in a "pair" so in
UPDATE MY_TABLE SET COL1 = ?, COL2 = ?, ...

the visit(JdbcParameter) doesn't have a visit(Column) before it. One can fix that with keeping track of the number of parameters processed and manually getting the correct column.
The WHERE is processed correctly.
Using custom deparser's is likely a better solution, so I think we can close this.

After digging deeper I recognized, that the Update deparser does not use the ExpressionVisitor, to work on the columns. Is that the problem you described as "not visited in pair"?
That is definitly a visitor problem and therefore a bug that should be resolved.

Could be, Update.getExpressions() and Update.getWhere() are visited.
https://github.com/jesperpedersen/postgres-tools/blob/master/QueryAnalyzer/QueryAnalyzer.java#L365-L376
because
https://github.com/jesperpedersen/postgres-tools/blob/master/QueryAnalyzer/QueryAnalyzer.java#L354
isn't called until Update.getWhere()

Could you check it. I corrected the visitor calling for update statements. If you find similar problems for other kind of statements, please file an issue.

:+1:


Add tables for other EC2 services

Add as many tables for EC2 services as possible. The important ones include Key Pairs, AMIs, VPCs, Security Groups, etc., that is anything required to actually create EC2 instances.

Done in 2b03b097aba6188b13c531c639794aed8afb660e


download notification doesn't show up

Steps to reproduce
Pause a download
In the new release 1.9.3 when you pause the download notification it switchea and shows up cancel and continue button but in the new preview r508 it doesn't happen
Expected behavior
Notification shows up with cancel and continue options
Actual behavior
No notification shows up
And you have to go to the download section in app in order to continue or cancel downloads
Crash logs
No response
Tachiyomi version
App version: 1.9.1 preview 508
Android version
Android 12
Device
Samsung s10
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.9.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Probably fixed in the recent builds, either that or you disabled the notification

Probably fixed in the recent builds, either that or you disabled the notification

I'm on r508 I even uninstalled and reinstalled the app and the issue persist

https://user-images.githubusercontent.com/91851160/236650655-f073a037-9aee-4318-8000-a54e2efd46d7.mp4

@jobobby04
sorry for the tagging I just don't know if you're subscribed this post

Does it happen in the latest build(build 512?)

Yes, although the recording was on 508 the same thing happens on r512

Idk if this helps but on 1.9.3 it works
https://user-images.githubusercontent.com/91851160/236686291-197258ea-0aa4-472f-b01f-82bd873449db.mp4


Fixed Java 8 support

Thanks for the contribution! We just put in a fix for Java 8 similar to yours, which also relies on getting Java 8 from the Director yum repo.


Incorrect label for federal territory.

Previously in #14 and #17  which added information on Kuala Lumpur, Labuan and Putrajaya, the administrative division was incorrectly labeled as "State of Malaysia". They are however not considered to be states but as federal territories instead.
The correction would be as follows (for the administrative division)
Labuan - Federal Territory of Labuan
Kuala Lumpur  - Federal Territory of Kuala Lumpur
Putrajaya - Federal Territory of Putrajaya
This will also line up the entries so that the data is similar to the ones in the all.json file.

That is indeed correct. I will merge this PR.


[WIP] An RSpec file for testing HTTP library behavior with 'no_proxy' environment variable

#3293 related?

This looks to have been superseded by other PRs, and so I'm going to close it out. If that's not the case please feel free to reopen and finish this up.


Explain the implementation of total orders command

Sequence diagrams for total orders command:

Updated sequence diagram

Codecov Report

Merging #282 (b6210ce) into master (47e934c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #282   +/-   ##
=========================================
  Coverage     66.27%   66.27%
  Complexity      837      837
=========================================
  Files           135      135
  Lines          2758     2758
  Branches        333      333
=========================================
  Hits           1828     1828
  Misses          815      815
  Partials        115      115

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/ModelManager.java
66.91% <0.00%> (ø)

...java/seedu/address/logic/commands/HelpCommand.java
100.00% <0.00%> (ø)

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 47e934c...b6210ce. Read the comment docs.


bug: Solved mac only lazyvim repeatedly cloning on start

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
10.1
Operating system/version
MacOs 11.5
Describe the bug
Just posting incase someone else runs into this, at it was hard to track down. I had an old install of lazyvim worked great on my linux machines. Was moving to a mac and got a bug where lazyvim would just clone itself over and over again on start. Everything worked find on the linux setups but the issue was on mac only.
The solution was removing lazyvim/lazyvim from my plugin list.
Steps To Reproduce
solved
Expected Behavior
Solved
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- add any other plugins here
{lazyvim/lazvim} -- remove this
  },
})

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer

The Lazyvim starter template doesn't include such a line as can be seen here. You most likely put it there yourself. Not a Lazyvim bug.


nuspec files from msbuild/csproj if not provided

If no nuspec files are already provided, it provides them using msbuild artifacts and csproj settings
Add unit tests
Update change logs

I didn't tests with ILRepack. I had a error while building from a fresh clone. I trust the unit tests. :-)


Rethink crate packaging and the build.rs script.

I'd like to hear from Rust users before this is merged. Ping @kirk-baird, @paulhauner, @michaelsproul, @dignifiedquire, ...

That's good by me :)

Thanks for the feedback! Just in case for reference. Primary DEP_BLST_* consumer will be CUDA, more specifically host-side data pre- and postprocessing.

Merged. Thanks, everybody!


[Question] How to integrate flutter on exsiting app？

I hava an app which most pages written by native code. Now I want to integrate flutter, in order to write new pages.  I have read flutter doc, for example:
Android
public class MainActivity extends FlutterActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    GeneratedPluginRegistrant.registerWith(this);
  }
}

this activity run with main.dart automatically.
But if I have mainA.dart and mainB.dart, how to assign mainA.dart to AActivity, assign mainB.dart to BActivity. Or how to specify which dart file to load?
many thanks.

I believe this is a dup of #14821

Specifying an dart file entry point is not yet supported but in progress (@chinmaygarde can correct me if I'm wrong). Please append your request and follow progress on #14821.


Migrate ClusterClaim to ClusterProperty

Describe what you are trying to solve
Antrea Multi-cluster is using ClusterClaim CRD to allow user to claim a cluster in a ClusterSet. We was following the definition in KEP2149, but the CRD is changed from ClusterClaim to ClusterProperty, ClusterClaim is no longer used in upstream. It's better to follow upstream definition in Antrea Multi-cluster.
Describe the solution you have in mind

Remove ClusterClaim and add ClusterProperty CRD.
Support both ClusterClaim and ClusterProperty CRDs (preferred).

Describe how your solution impacts user flows
This will impact the way how to initialize a ClusterSet from user perspective. We should cover following cases when we do MC controllers refinement.

Migrate existing ClusterClaim to ClusterProperty and watch ClusterProperty change and update only after migration
ClusterProperty controller should cover all current ClusterClaim's use case.

Hi,
Here is the cope of move ClusterClaim to ClusterProperty issue:

Docs: docs/multicluster docs
Designed the new ClusterProperty which conforms to the requirements, and the generated codes
New ClusterProperty webhook and its unit-tests. If we want support both ClusterClaim and ClusterProperty,
we also need a convert here.
Multicluster/build yamls and config yamls
Multicluster/controllers of ClustersetController in both Leaders and Members, MemberClusterAnnounceController, ClusterProperty itself Controller.
Pkg/client versioned informers and listers, which can be generated automatically
Integration’s test codes and yamls
Antctl ’s destroy command, init command, join command, cleanup command, deploy leader cluster command and deploy member cluster command, and the common file. And change the test files.  If we want support both ClusterClaim and ClusterProperty, we need to find a way to deal with two CRD here.

The biggest variable now is whether we should support both versions, if the answer is yes, what approach we should take. We should convert the old ClusterClaim forcefully or not, if we support both version, do we want two CRDs running in the same cluster? Do we want the users to run the  migrate code to ClusterClaim to ClusterProperty at last? I think a discussion should be done if we want to do this.
Thanks,
Bangqi
From: Lan @.>
Date: Tuesday, March 7, 2023 at 9:32 PM
To: antrea-io/antrea @.>
Cc: Bangqi Zhu @.>, Assign @.>
Subject: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
!! External Email
Describe what you are trying to solve
Antrea Multi-cluster is using ClusterClaim CRD to allow user to claim a cluster in a ClusterSet. We was following the definition in KEP2149https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkubernetes%2Fenhancements%2Ftree%2Fmaster%2Fkeps%2Fsig-multicluster%2F2149-clusterid&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=V9L3xWhPKurefj8yXHDH51U9630xYBoqSOlSMC8%2BwSM%3D&reserved=0, but the CRD is changed from ClusterClaim to ClusterProperty, ClusterClaim is no longer used in upstream. It's better to follow upstream definition in Antrea Multi-cluster.
Describe the solution you have in mind

Remove ClusterClaim and add ClusterProperty CRD.
Support both ClusterClaim and ClusterProperty CRDs (preferred).

Describe how your solution impacts user flows
This will impact the way how to initialize a ClusterSet from user perspective. We should cover following cases when we do MC controllers refinement.

Migrate existing ClusterClaim to ClusterProperty and watch ClusterProperty change and update only after migration
ClusterProperty controller should cover all current ClusterClaim's use case.

—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fantrea-io%2Fantrea%2Fissues%2F4686&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4tZXYLP%2BuBedlvkRcNXP1xuvWIKstyMFR1WA6u3p52U%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNNN6WGU3O3VXRBHHXER33W3AKYBANCNFSM6AAAAAAVTK62DI&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LbFU0L087WBhH4BScZtWMvYQBw3ozt68I8MsYztybC8%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.

@Jianjun @.>/ @Lan @.> Could you please direct @Bangqi @.***> for the right approach?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Bangqi Zhu @.>
Sent: Monday, March 13, 2023 10:55:36 PM
To: antrea-io/antrea @.>; Akshay Katrekar @.>; Krishna Miriyala @.>
Cc: Lan Luo @.>; Jianjun Shen @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
Hi,
Here is the cope of move ClusterClaim to ClusterProperty issue:

Docs: docs/multicluster docs

Designed the new ClusterProperty which conforms to the requirements, and the generated codes

New ClusterProperty webhook and its unit-tests. If we want support both ClusterClaim and ClusterProperty,
we also need a convert here.

Multicluster/build yamls and config yamls

Multicluster/controllers of ClustersetController in both Leaders and Members, MemberClusterAnnounceController, ClusterProperty itself Controller.

Pkg/client versioned informers and listers, which can be generated automatically

Integration’s test codes and yamls

Antctl ’s destroy command, init command, join command, cleanup command, deploy leader cluster command and deploy member cluster command, and the common file. And change the test files.  If we want support both ClusterClaim and ClusterProperty, we need to find a way to deal with two CRD here.

The biggest variable now is whether we should support both versions, if the answer is yes, what approach we should take. We should convert the old ClusterClaim forcefully or not, if we support both version, do we want two CRDs running in the same cluster? Do we want the users to run the  migrate code to ClusterClaim to ClusterProperty at last? I think a discussion should be done if we want to do this.
Thanks,
Bangqi
From: Lan @.>
Date: Tuesday, March 7, 2023 at 9:32 PM
To: antrea-io/antrea @.>
Cc: Bangqi Zhu @.>, Assign @.>
Subject: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
!! External Email
Describe what you are trying to solve
Antrea Multi-cluster is using ClusterClaim CRD to allow user to claim a cluster in a ClusterSet. We was following the definition in KEP2149https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkubernetes%2Fenhancements%2Ftree%2Fmaster%2Fkeps%2Fsig-multicluster%2F2149-clusterid&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=V9L3xWhPKurefj8yXHDH51U9630xYBoqSOlSMC8%2BwSM%3D&reserved=0, but the CRD is changed from ClusterClaim to ClusterProperty, ClusterClaim is no longer used in upstream. It's better to follow upstream definition in Antrea Multi-cluster.
Describe the solution you have in mind

Remove ClusterClaim and add ClusterProperty CRD.
Support both ClusterClaim and ClusterProperty CRDs (preferred).

Describe how your solution impacts user flows
This will impact the way how to initialize a ClusterSet from user perspective. We should cover following cases when we do MC controllers refinement.

Migrate existing ClusterClaim to ClusterProperty and watch ClusterProperty change and update only after migration
ClusterProperty controller should cover all current ClusterClaim's use case.

—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fantrea-io%2Fantrea%2Fissues%2F4686&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4tZXYLP%2BuBedlvkRcNXP1xuvWIKstyMFR1WA6u3p52U%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNNN6WGU3O3VXRBHHXER33W3AKYBANCNFSM6AAAAAAVTK62DI&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LbFU0L087WBhH4BScZtWMvYQBw3ozt68I8MsYztybC8%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.

Ideally, we should do mirroring of CRDs - using a mirroring controller to convert old CRDs to new ones, and other code just handles new CRDs. You can check the following PRs to learn more:
Remove all legacy (*.antrea.tanzu.vmware.com) APIs by antoninbas · Pull Request #3299 · antrea-io/antrea (github.com)https://github.com/antrea-io/antrea/pull/3299
Rename API groups from *.antrea.tanzu.vmware.com to *.antrea.io by hongliangl · Pull Request #1799 · antrea-io/antrea (github.com)https://github.com/antrea-io/antrea/pull/1799
However, I did not look into ClusterProperty to know if it is easy to be mirrored to ClusterClaim. Could you evaluate the approach?
In any case, I feel we could just support the new CRDs with antctl, and document only the new CRDs in documentation. Users can switch to old version antctl/documents for old CRDs.
Jianjun

From: Krishna Miriyala @.>
Sent: Tuesday, March 14, 2023 8:14
To: Bangqi Zhu @.>; antrea-io/antrea @.>; Akshay Katrekar @.>; Jianjun Shen @.>; Lan Luo @.>
Cc: Lan Luo @.>; Jianjun Shen @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
@Jianjun @.>/ @Lan @.> Could you please direct @Bangqi @.***> for the right approach?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Bangqi Zhu @.>
Sent: Monday, March 13, 2023 10:55:36 PM
To: antrea-io/antrea @.>; Akshay Katrekar @.>; Krishna Miriyala @.>
Cc: Lan Luo @.>; Jianjun Shen @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
Hi,
Here is the cope of move ClusterClaim to ClusterProperty issue:

Docs: docs/multicluster docs

Designed the new ClusterProperty which conforms to the requirements, and the generated codes

New ClusterProperty webhook and its unit-tests. If we want support both ClusterClaim and ClusterProperty,
we also need a convert here.

Multicluster/build yamls and config yamls

Multicluster/controllers of ClustersetController in both Leaders and Members, MemberClusterAnnounceController, ClusterProperty itself Controller.

Pkg/client versioned informers and listers, which can be generated automatically

Integration’s test codes and yamls

Antctl ’s destroy command, init command, join command, cleanup command, deploy leader cluster command and deploy member cluster command, and the common file. And change the test files.  If we want support both ClusterClaim and ClusterProperty, we need to find a way to deal with two CRD here.

The biggest variable now is whether we should support both versions, if the answer is yes, what approach we should take. We should convert the old ClusterClaim forcefully or not, if we support both version, do we want two CRDs running in the same cluster? Do we want the users to run the  migrate code to ClusterClaim to ClusterProperty at last? I think a discussion should be done if we want to do this.
Thanks,
Bangqi
From: Lan @.>
Date: Tuesday, March 7, 2023 at 9:32 PM
To: antrea-io/antrea @.>
Cc: Bangqi Zhu @.>, Assign @.>
Subject: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
!! External Email
Describe what you are trying to solve
Antrea Multi-cluster is using ClusterClaim CRD to allow user to claim a cluster in a ClusterSet. We was following the definition in KEP2149https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkubernetes%2Fenhancements%2Ftree%2Fmaster%2Fkeps%2Fsig-multicluster%2F2149-clusterid&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=V9L3xWhPKurefj8yXHDH51U9630xYBoqSOlSMC8%2BwSM%3D&reserved=0, but the CRD is changed from ClusterClaim to ClusterProperty, ClusterClaim is no longer used in upstream. It's better to follow upstream definition in Antrea Multi-cluster.
Describe the solution you have in mind

Remove ClusterClaim and add ClusterProperty CRD.
Support both ClusterClaim and ClusterProperty CRDs (preferred).

Describe how your solution impacts user flows
This will impact the way how to initialize a ClusterSet from user perspective. We should cover following cases when we do MC controllers refinement.

Migrate existing ClusterClaim to ClusterProperty and watch ClusterProperty change and update only after migration
ClusterProperty controller should cover all current ClusterClaim's use case.

—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fantrea-io%2Fantrea%2Fissues%2F4686&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4tZXYLP%2BuBedlvkRcNXP1xuvWIKstyMFR1WA6u3p52U%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNNN6WGU3O3VXRBHHXER33W3AKYBANCNFSM6AAAAAAVTK62DI&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LbFU0L087WBhH4BScZtWMvYQBw3ozt68I8MsYztybC8%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.

Sounds good. I would suggest to consider several smaller PRs which are easier to review.
Jianjun

From: Bangqi Zhu @.>
Sent: Wednesday, March 15, 2023 2:23
To: Jianjun Shen @.>; Krishna Miriyala @.>; antrea-io/antrea @.>; Akshay Katrekar @.>; Lan Luo @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
Hi Jianjun, I got how to use this mirroring controller, since it are some changes between ClusterProperty and ClusterClaim, like Well known properties and apiVersion, I think I need about 3-4 weeks to deal with this issue, after this maybe there will be some changes but won’t block me from other issues.
Thanks!
Bangqi
From: Jianjun Shen @.>
Date: Tuesday, March 14, 2023 at 11:16 AM
To: Krishna Miriyala @.>, Bangqi Zhu @.>, antrea-io/antrea @.>, Akshay Katrekar @.>, Lan Luo @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
Ideally, we should do mirroring of CRDs - using a mirroring controller to convert old CRDs to new ones, and other code just handles new CRDs. You can check the following PRs to learn more:
Remove all legacy (*.antrea.tanzu.vmware.com) APIs by antoninbas · Pull Request #3299 · antrea-io/antrea (github.com)https://github.com/antrea-io/antrea/pull/3299
Rename API groups from *.antrea.tanzu.vmware.com to *.antrea.io by hongliangl · Pull Request #1799 · antrea-io/antrea (github.com)https://github.com/antrea-io/antrea/pull/1799
However, I did not look into ClusterProperty to know if it is easy to be mirrored to ClusterClaim. Could you evaluate the approach?
In any case, I feel we could just support the new CRDs with antctl, and document only the new CRDs in documentation. Users can switch to old version antctl/documents for old CRDs.
Jianjun

From: Krishna Miriyala @.>
Sent: Tuesday, March 14, 2023 8:14
To: Bangqi Zhu @.>; antrea-io/antrea @.>; Akshay Katrekar @.>; Jianjun Shen @.>; Lan Luo @.>
Cc: Lan Luo @.>; Jianjun Shen @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
@Jianjun @.>/ @Lan @.> Could you please direct @Bangqi @.***> for the right approach?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Bangqi Zhu @.>
Sent: Monday, March 13, 2023 10:55:36 PM
To: antrea-io/antrea @.>; Akshay Katrekar @.>; Krishna Miriyala @.>
Cc: Lan Luo @.>; Jianjun Shen @.>
Subject: Re: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
Hi,
Here is the cope of move ClusterClaim to ClusterProperty issue:

Docs: docs/multicluster docs

Designed the new ClusterProperty which conforms to the requirements, and the generated codes

New ClusterProperty webhook and its unit-tests. If we want support both ClusterClaim and ClusterProperty,
we also need a convert here.

Multicluster/build yamls and config yamls

Multicluster/controllers of ClustersetController in both Leaders and Members, MemberClusterAnnounceController, ClusterProperty itself Controller.

Pkg/client versioned informers and listers, which can be generated automatically

Integration’s test codes and yamls

Antctl ’s destroy command, init command, join command, cleanup command, deploy leader cluster command and deploy member cluster command, and the common file. And change the test files.  If we want support both ClusterClaim and ClusterProperty, we need to find a way to deal with two CRD here.

The biggest variable now is whether we should support both versions, if the answer is yes, what approach we should take. We should convert the old ClusterClaim forcefully or not, if we support both version, do we want two CRDs running in the same cluster? Do we want the users to run the  migrate code to ClusterClaim to ClusterProperty at last? I think a discussion should be done if we want to do this.
Thanks,
Bangqi
From: Lan @.>
Date: Tuesday, March 7, 2023 at 9:32 PM
To: antrea-io/antrea @.>
Cc: Bangqi Zhu @.>, Assign @.>
Subject: [antrea-io/antrea] Migrate ClusterClaim to ClusterProperty (Issue #4686)
!! External Email
Describe what you are trying to solve
Antrea Multi-cluster is using ClusterClaim CRD to allow user to claim a cluster in a ClusterSet. We was following the definition in KEP2149https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkubernetes%2Fenhancements%2Ftree%2Fmaster%2Fkeps%2Fsig-multicluster%2F2149-clusterid&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=V9L3xWhPKurefj8yXHDH51U9630xYBoqSOlSMC8%2BwSM%3D&reserved=0, but the CRD is changed from ClusterClaim to ClusterProperty, ClusterClaim is no longer used in upstream. It's better to follow upstream definition in Antrea Multi-cluster.
Describe the solution you have in mind

Remove ClusterClaim and add ClusterProperty CRD.
Support both ClusterClaim and ClusterProperty CRDs (preferred).

Describe how your solution impacts user flows
This will impact the way how to initialize a ClusterSet from user perspective. We should cover following cases when we do MC controllers refinement.

Migrate existing ClusterClaim to ClusterProperty and watch ClusterProperty change and update only after migration
ClusterProperty controller should cover all current ClusterClaim's use case.

—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fantrea-io%2Fantrea%2Fissues%2F4686&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4tZXYLP%2BuBedlvkRcNXP1xuvWIKstyMFR1WA6u3p52U%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNNN6WGU3O3VXRBHHXER33W3AKYBANCNFSM6AAAAAAVTK62DI&data=05|01|zbangqi%40vmware.com|53578e19c68a43ff397f08db1f968dee|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|638138503730198560|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=LbFU0L087WBhH4BScZtWMvYQBw3ozt68I8MsYztybC8%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.

We decide to remove ClusterClaim to simplify ClusterSet setup, so close this issue.


Update Issue and PR Templates

Closes #17

Following discussion with @al-fisher this PR is being closed due to the changes made by the documents that were existing as this is a fork. This needs to be treated as an exception to our usual workflow and the PR will be redone.


Blank resources list after clicking the X in the blue header bar

v1.1.0-rc.6 build 21-ad9123a
I opened a Titus tN file and ran the validator. I then clicked on the X in the blue header bar to select a different project and found the resource list to be blank.
I was able to get things working again by clicking on Organization in the stepper.
I repeated the same steps but this time did not see any problems.

@cckozie should this go in the QA watchlist?

@birchamp - I'd rather not put it there as I like to use that lane for things I expect to get resolved in the not to distant future. I suspect this is a problem we're not very likely to see again, but wanted to document it incase it does reappear. I'm fine with moving it to the icebox if you'd prefer to get it out of new issues.

I tried several times to replicate, but could not. Then I looked closer at the error message on the screenshot. There are 8 name resolution errors, one per resource type. This is followed by 8 low level HTTP or Network errors:

4 name resolution errors
4 server not found errors

Hypothesis: the server briefly offline/unavailable


Send an email to the astropy community about the first interim finance committee meeting

Several initial decisions and transparency channels were decided on at the first finance committee meeting, a message should go out to the astropy community about these and how people can keep up on continuing activitities.
These messages can only be set up once the channels are all up and working though.

I wanted to make a few additions to the wiki in support of this mail (primarily, adding the Finance Ctte charge), but I don't have access. Should I have? If not, what should our workflow be? I don't think it's possible to make a PR against a wiki.

I received the email in my inbox, so this is obviously done.
Thanks @jdswinbank !


Custom Device Wizard package should have a VeriStand Custom Device Development Tools as dependent package

Is your feature request related to a problem? Please describe.
If you only install the Custom Device Wizard from the release package, the Wizard will be placed in the Template project of LabVIEW but the generation will take arround 5 min and will fail at the end with error 7 on "Copy Template Files.vi"
Describe the solution you'd like
Make this package part of the dependencies so the user don't need to install it manually
Describe alternatives you've considered
Make the explanation a bit more clear of what dependencies need to be installed
Additional context
Add any other context or screenshots about the feature request here.

Creating a dependent package does not eliminate the need to manually install the dependent package because there is not a feed containing all the required packages, but #50 will prevent installation of the wizard package if the other package is not installed and the release page has been updated to include prerequisite installers like the README does.


shared-mime-info-2.0_2 making poudriere effectively unusable

Prerequisites

[X] Have you checked for an existing issue describing your problem?
[X] Are you running the latest version?
[X] Is your ports tree recent?
[X] Is your FreeBSD Host on a supported release?

Describe the bug
lib/build-depend stage slows massively post shared-mime-info being installed. Runs that use to complete in under 24 hours will now run well over 100 hours with out showing noticable progress as some ports will be stuck for hours in the lib/build depend stage constantly rebuilding the shared mime-info DB.
For example kf5-kirigami2-5.87.0 took a bit over 3 hours to build here in my current run. Nearly all that time was '/usr/local/bin/update-mime-database /usr/local/share/mime'.
[12stableAMD64-head-job-05] |   `-- Installing qt5-widgets-5.15.2_3...
[12stableAMD64-head-job-05] |   | `-- Installing atk-2.36.0...
[12stableAMD64-head-job-05] |   | `-- Extracting atk-2.36.0: .......... done
[12stableAMD64-head-job-05] |   | `-- Installing gtk3-3.24.30...
[12stableAMD64-head-job-05] |   |   `-- Installing adwaita-icon-theme-40.1.1...
[12stableAMD64-head-job-05] |   |   | `-- Installing gdk-pixbuf2-2.40.0...
[12stableAMD64-head-job-05] |   |   |   `-- Installing shared-mime-info-2.0_2...
[12stableAMD64-head-job-05] |   |   |   `-- Extracting shared-mime-info-2.0_2: ......... done
Building the Shared MIME-Info database cache
[12stableAMD64-head-job-05] |   |   |   `-- Installing tiff-4.3.0...
[12stableAMD64-head-job-05] |   |   |   | `-- Installing jbigkit-2.1_1...
[12stableAMD64-head-job-05] |   |   |   | `-- Extracting jbigkit-2.1_1: .......... done
Building the Shared MIME-Info database cache
[12stableAMD64-head-job-05] |   |   |   `-- Extracting tiff-4.3.0: .......... done
Building the Shared MIME-Info database cache
[12stableAMD64-head-job-05] |   |   | `-- Extracting gdk-pixbuf2-2.40.0: .......... done
Building the Shared MIME-Info database cache
[12stableAMD64-head-job-05] |   |   | `-- Installing gtk-update-icon-cache-3.24.26_1...
[12stableAMD64-head-job-05] |   |   | `-- Extracting gtk-update-icon-cache-3.24.26_1: .......... done
Building the Shared MIME-Info database cache
Generating GTK icon cache for /usr/local/share/icons/hicolor
[12stableAMD64-head-job-05] |   |   | `-- Installing pango-1.48.7...
[12stableAMD64-head-job-05] |   |   |   `-- Installing cairo-1.17.4,3...
[12stableAMD64-head-job-05] |   |   |   | `-- Installing pixman-0.40.0_1...
[12stableAMD64-head-job-05] |   |   |   | `-- Extracting pixman-0.40.0_1: .......... done
Building the Shared MIME-Info database cache
Generating GTK icon cache for /usr/local/share/icons/hicolor
[12stableAMD64-head-job-05] |   |   |   `-- Extracting cairo-1.17.4,3: .......... done
Building the Shared MIME-Info database cache
Generating GTK icon cache for /usr/local/share/icons/hicolor
[12stableAMD64-head-job-05] |   |   |   `-- Installing fribidi-1.0.11...
[12stableAMD64-head-job-05] |   |   |   `-- Extracting fribidi-1.0.11: .......... done
Building the Shared MIME-Info database cache
Generating GTK icon cache for /usr/local/share/icons/hicolor
[12stableAMD64-head-job-05] |   |   |   `-- Installing libXft-2.3.3...
[12stableAMD64-head-job-05] |   |   |   `-- Extracting libXft-2.3.3: .......... done
Building the Shared MIME-Info database cache
Generating GTK icon cache for /usr/local/share/icons/hicolor

How to reproduce
1: grab the current ports tree
2: start a build of anything that wants lots of stuff post installing shared-mime-info, such as x11-toolkits/kf5-kirigami2.
3: Watch it hang on effectively hang lib/build depend install stage.
Expected behavior

Screenshots

Environment

Host OS [e.g. 12.2 amd64]: FreeBSD vixen42.vulpes.vvelox.net 12.2-STABLE FreeBSD 12.2-STABLE stable/12-n233932-9419d273e47 vixen42  amd64
Jail OS [e.g. 12.0 powerpc]: FreeBSD vixen42.vulpes.vvelox.net 12.2-STABLE FreeBSD 12.2-STABLE stable/12-n233932-9419d273e47 vixen42  amd64
Browser: [e.g. chrome, safari]: n/a
Poudriere Version [e.g. 3.3.1 or git hash or port version]: 3.3.7_1
Ports branch and revision [e.g. 2020Q3 r550754]: 7aa3d635d2f57370b1ebbf63b09354a4ac3a1545

Additional context
Not sure how we want to proceed given this is a combination of bad port design and how we are using pkg to install stuff. I am fairly certain this is not a pkg bug, but would really need to dig in and find where/how pkg is being called here.

hmm... okay did not think we were doing all the pkg stuff in a single throw but looks like I was wrong, should of dug into this before opening this...
should now be fixed with the new pkg version...
    ports-mgmt/pkg: update to 1.17.5

    Changes from 1.17.4 to 1.17.5
    - pkg add: only runs triggers at the end of the equivalent of a transaction
    - pkg search: fix the combinaison of -q and -o
    - pkg fetch: correctly exit with error on fetch failures
    - Improve error message when failing to drop privileges


Organize application configuration

The HTML templates are now parameterized with app config variables

appName (e.g., BioSimulations, BioSimulators)
appName left (e.g., Bio)
appName right (e.g., Simulations, Simulators)
appUrl (e.g., https://biosimulations.org/)
apiUrl (e.g., https://api.biosimulations.org/)
submitAppUrl (e.g., https://submit.biosimulations.org)
webserviceUrl (e.g., https://dispatch.biosimulations.org)
issueUrl (e.g., https://github.com/biosimulations/Biosimulations/issues/new/choose)
pullsUrl (e.g., https://github.com/biosimulations/Biosimulations/pulls)
email (e.g<EMAIL_ADDRESS><EMAIL_ADDRESS>icon location

Each place in the .ts files with the comment // TODO: get from app config needs to be connected to a new application configuration

For now, I'm starting to put the values in biosimulations/apps/*/src/app/config. This will need to be renamed and reformatted. At least this collates the values into one place.

Added in dev-config branch


Hide error_chain boiler plate by default in all examples

Modified all examples to hide noisy error_chain related code
resolves: https://github.com/brson/rust-cookbook/issues/106

Wow this is going to be way more readable. Thanks!


Lack of documentation on how to use the website for new officers

Aside from whatever documentation we, as officers, have tried to compile regarding the website’s functionality and features, there is little in the way of meaningful, clear documentation that is known to and available to the officer corps. Not everyone who takes on a leadership is going to be familiar with Markdown, and most people will not have experience working with our website specifically. Having clear documentation would make it much easier for newer officers to effectively use the website.
Comment from Atishay @atish3 (F21 Service Coordinator):

This should ideally be prepared by the current website maintainers.

is there any documentation on how to run this website locally on say my computer? just to test out changes and updates to the website


{KeyVault} Modify private link commands to align with storage

Support wait command in az keyvault private-endpoint-connection group;
Add examples in help.py;
Change the command name from az keyvault private-link-resource show to az keyvault private-link-resource list;
Change the parameter name approval-description/rejection-description to description;
Use the validate/transform functions from core.utils instead of self-implementations.

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

KeyVault private link


feat(angular-table) refactor flex-renderer to use signal

convert all inputs to signal
re-render when content is updated (is this expected?)
make rendering more flexible by handling function based content first

My flex-render implementation volutely used the ""old"" approach due to non-interchangeable differences between signals execution and ngDoCheck/ngOnChanges.
You need to use ngDoCheck and propagate change detection to columns that renders template and components.
In the previous implementation, we were not re-rendering all from scratch. vcr.clear() was called on every content change, which should be updated only if you update your column definition variable. Instead, we need to mark the view dirty if the context value reference changes.
You can do some test with the row-selection example which is probably broken with this implementation

@merto20 @riccardoperra  This seems like a breaking change. Since its still early in this adapter's life, if we see this change as necessary to performance, I'm open to it, but would want to be clear on the trade offs. And before merging, all docs and examples would need to be updated, and we would probably even need a section in the flex render docs explaining the breaking change.
An alternative approach we could take, would be to export a new signalFlexRender (or similar) method with the different implementation suggested here. No breaking changes, but an alternative that's available.

@KevinVandy Using signals input or decoration based approach doesn't change how the consumer uses the flexRenderDirective, the signature is the same. What really differs is when the component renders and when it's updated.
In my company we have something similiar to the flexRender directive that we use to render with the same api components/templates/strings/functions and we had no performance issue at all using ngDoCheck/ngOnChanges. Even the official angular directive implementation still relies on that approach (ng_template_outlet.ts and ng_component_outlet.ts) which is extensively tested

@riccardoperra row-selection example project is not working on my side even without my changes. Is it working for you?

There are maybe some cache issues. you can check of the stackblitz example in the docs https://tanstack.com/table/latest/docs/framework/angular/examples/row-selection

@riccardoperra i've tried different machine, I'm not able to run any examples  using the main branch. tried --skip-nx-cache option while building the packages but no avail.

You have to build all packages before running examples. We can discuss in the TanStack discord if you are still running into any issues. We can update the contributing guide too if it's not accurate.

@KevinVandy I did build all the packages. Even tried --skip-nx-cache command thinking it was due to caching issue.
Pls add me to the channel, @merto200. thanks

@KevinVandy i'm able to run the examples now using a different linux distro. I don't how it happened, but it is a pain when your dev environment got affected like that.

@riccardoperra @KevinVandy The reason why I didn't encounter the error when I tested grouping example project was because it didn't able to reload using the latest packages build. Now, i have to manually clear the cache before running the example project again to make sure it loads the latest changes.
Please check my latest commits. I've removed the FlexRenderComponent and injectFlexRenderContext. This is a significant change that simplifies the component creation process. By allowing consumers to use signal inputs and component types directly, we're streamlining the development workflow and potentially reducing the learning curve. It's always beneficial to have a more natural and intuitive way of building components, as it can lead to better maintainability and easier collaboration.

My flex-render implementation volutely used the ""old"" approach due to non-interchangeable differences between signals execution and ngDoCheck/ngOnChanges.
In the previous implementation, we were not re-rendering all from scratch. vcr.clear() was called on every content change, which should be updated only if you update your column definition variable. Instead, we need to mark the view dirty if the context value reference changes.
You need to use ngDoCheck in order mark the view as dirty manually to all rendered templateRefs/components that may have been changed
You can do some test with the row-selection example which is probably broken with this implementation

I've been using signals in directives. This simplifies the implementation as we don't need to care for changeDetection anymore.

What really differs is when the component renders and when it's updated. I didn't notice any benefit at all having signals with flexRender, the view is marked as dirty later and you may have ui synchronization issues, unless you handle it manually (which you do anyway with the other approach)

this should not be an issue since Angular table fully utilizes signals.

I'm a little confused with this PR. This now looks to be a large breaking change.

@KevinVandy @riccardoperra  Apologies for the oversight, I've just noticed that the Angular Table has already been shipped. To address this, I'm going to break the current PR into two separate ones. I will introduce support for custom component-based rendering with signal inputs, another option to using FlexRendererComponent and injectFlexRenderContext approach.

@KevinVandy we can revisit this PR again if and when we support signal-based flex-renderer-directive.
This the PR I created for to support signal-based component. https://github.com/TanStack/table/pull/5576

Closing, but as mentioned above, concepts from this PR could be pointed to the v9 alpha branch


Double Device Names For Entity

Describe the bug
I noticed after the latest update that all Alarm.com Z-Wave devices now show in Home Assistant with their names twice for the device name. Example: Living Room Light now shows as light.Living Room Light Living Room Light
What's odd is that when I go do the Devices list in Home Assistant the names show correct (IE: once) so the entity ID isn't changed just what shows in the cards on the dashboard
To Reproduce
Steps to reproduce the behavior:

Link device to alarm.com
Add z-wave device
View device name
Add device to dashboard in a card and see that device name shows twice for Entity (but not entity ID) as well
See error on card display name

Expected behavior
Should show device name only once for Entity.
Screenshots

Home Assistant Version:
Home Assistant 2023.7.3
Frontend 20230705.1 - latest
Additional context
Workaround is to manually put in the name to overwrite the default display.

This is fixed in v3.0.7. Can you update and confirm?

This is fixed in v3.0.7. Can you update and confirm?

Yup, I can confirm


Using cookie as one of the request token source

On top of authorization header, I have added cookie as one of the request token sources.
Also, I change the composer file to use php-jwt version 4.0.x instead of 3.0.x

Hey thanks for the pull request. Didn't expect I'll get one.

It's on v0.7.0


NBA.getGames() goes wrong.

NBA.getGames() goes wrong.

Similar issue
TypeError: Cannot read property 'sports_content' of undefined

Oops, NBA.getGamesFromDate() goes wrong.
Please run nba-go again.
If it still does not work, feel free to open an issue on https://github.com/xxhomey19/nba-go/issues

+1 - same error as the above

Same error, help us!!!

Seems like they've changed the API endpoint since about March - any date beyond the 16th of March 2020 either redirects to that day's scoreboard or returns a 404. http://data.nba.com/data/5s/json/cms/noseason/scoreboard/20200328/games.json
Wonder if someone could fork this or the NBA package itself and point it to the V2 endpoint? https://data.nba.net/prod/v2/20200830/scoreboard.json

same here. would be glad if this issue can be solved, i want to use it

still not working using macOS 14.1 Sonoma


次字幕字体大小问题 (Subtitle font size issue)

1
first thanks author:
Subtitle font size issue,
I selected the maximum, but the two subtitles are not the same size

1

Hi, thanks for the report! Could you tell me your browser and screen resolution? And just to confirm: the size setting does work?
I will look into improving it, it has been on my todo list for a while. Maybe make it a slider or an input box so you can choose the exact size you want.

Hi, thanks for the report! Could you tell me your browser and screen resolution? And just to confirm: the size setting does work?您好，感谢您的报告！您能告诉我您的浏览器和屏幕分辨率吗？只是为了确认：尺寸设置是否有效？
I will look into improving it, it has been on my todo list for a while. Maybe make it a slider or an input box so you can choose the exact size you want.我会考虑改进它，它已经在我的待办事项清单上有一段时间了。也许将其设置为滑块或输入框，以便您可以选择所需的确切尺寸。

Chrome   ,     1920 X 1080

Sorry for the long wait. I reworked the font size settings in the latest version (v.2.3.5). You should now be able to increase the font size more. Could you check if this solves your issue?

yes, the require is solve, thanks author

Sorry for the long wait. I reworked the font size settings in the latest version (v.2.3.5). You should now be able to increase the font size more. Could you check if this solves your issue?抱歉让您久等了。我在最新版本（v.2.3.5）中重新设计了字体大小设置。您现在应该能够进一步增加字体大小。您能检查一下这是否解决了您的问题吗？

yes, the require is solve, thanks author


Feature/generator

Add multiple types to marshaling.
Add a code generator for marshaling and unmarshaling structs, slices and maps.

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #40    +/-   ##
======================================
  Coverage     100%   100%
======================================
  Files          30     30
  Lines        3403   3561   +158
======================================
+ Hits         3403   3561   +158

Impacted Files
Coverage Δ

encode_array.go
100% <100%> (ø)
:arrow_up:

encode_number_int.go
100% <100%> (ø)
:arrow_up:

encode_number_uint.go
100% <100%> (ø)
:arrow_up:

encode.go
100% <0%> (ø)
:arrow_up:

decode_number.go
100% <0%> (ø)
:arrow_up:

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 9f5ebac...ea31b29. Read the comment docs.


Passing variables

How can i pass variable into TwigColumn?

Variables is auto passing. You can check in twig via {{ dump() }}

Curious - was the documentation on this unclear?
https://omines.github.io/datatables-bundle/#twigcolumn

Its not unclear, but for me it looks like a workaround if i need to pass a controller variable to the template.
now i have to inject the twig environment in my controller action and call
$twig->addGlobal('name', $variable );
It would be nice to just have
$table->add(
            'name',
            TwigColumn::class,
            ['variables' => ['name' => $variable],
...

or something like that


Overlays vanished at random time and have not come back

Stop the runtime data being written to the module config file as it can lead to corruption, save it in a separate file instead
Discussed in https://github.com/AllskyTeam/allsky/discussions/3454

Originally posted by STLS-1 March  4, 2024
As in subject, My overlays simply vanished at around 12:35PM yesterday and have not come back. No clue why. I have not touched anything with overlays once I got them working. I rebooted and no change. I removed overlay items to see if there was one that suddenly went bad.. NO change.

Same thing is happeing to me using V2024.xx.xx branch. For some reason overlay module is getting disabled by something.

It seems my issue of disapearing overlay is caused by some stuck of system:
2024-03-24T02:15:09.077479+01:00 allsky allsky[885]: STARTING EXPOSURE at: 2024-03-24 02:15:09   @ 2.8 sec 2024-03-24T02:15:09.992652+01:00 allsky allsky[25652]: ERROR: Module allsky_overlay.py will be disabled, it took 33.54 seconds; max allowed is 30 seconds 2024-03-24T02:15:16.307289+01:00 allsky allsky[885]:   > Saving NIGHT image 'image-20240324021509.jpg'

You can adjust the maximum allowed - see the documentation.  33 seconds is a long time, do you have a lot of text and/or images in your overlay, or have a slow Pi?

Closing as the watchdog has been removed


Constraints problem when showing / dismissing

The Picker gives a constraints warning when showing and dismissing the view, and it also shows a dirty line when it closes

2015-03-23 10:21:42.915 RMPickerViewController-Demo[4943:195982] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want. Try this: (1) look at each constraint and try to figure out which you don't expect; (2) find the code that added the unwanted constraint or constraints and fix it. (Note: If you're seeing NSAutoresizingMaskLayoutConstraints that you don't understand, refer to the documentation for the UIView property translatesAutoresizingMaskIntoConstraints)
(
"<NSLayoutConstraint:0x7fa449556c50 V:|-(10)-[UILabel:0x7fa449549cc0'This is an example title....']   (Names: '|':_UIVisualEffectContentView:0x7fa449552040 )>",
"<NSLayoutConstraint:0x7fa449556ca0 V:[UILabel:0x7fa449549cc0'This is an example title....']-(10)-|   (Names: '|':_UIVisualEffectContentView:0x7fa449552040 )>",
"<NSLayoutConstraint:0x7fa449556120 V:[UIVisualEffectView:0x7fa449553d70]-(10)-[UIVisualEffectView:0x7fa449552480]>",
"<NSLayoutConstraint:0x7fa449556170 V:[UIVisualEffectView:0x7fa449552480(44)]>",
"<NSLayoutConstraint:0x7fa4495561c0 V:[UIVisualEffectView:0x7fa449552480]-(0)-|   (Names: '|':UIView:0x7fa44954ecb0 )>",
"<NSLayoutConstraint:0x7fa449556210 V:[UIVisualEffectView:0x7fa449553d70(216)]>",
"<NSLayoutConstraint:0x7fa449556a50 V:|-(0)-[UIVisualEffectView:0x7fa449552940]   (Names: '|':UIView:0x7fa44954ecb0 )>",
"<NSLayoutConstraint:0x7fa449556aa0 V:[UIVisualEffectView:0x7fa449552940]-(10)-[UIVisualEffectView:0x7fa449553d70]>",
"<NSAutoresizingMaskLayoutConstraint:0x7fa44980b570 h=-&- v=-&- _UIVisualEffectContentView:0x7fa449552040.height == UIVisualEffectView:0x7fa449551030.height>",
"<NSAutoresizingMaskLayoutConstraint:0x7fa4498045c0 h=-&- v=-&- UIVisualEffectView:0x7fa449551030.height == _UIVisualEffectContentView:0x7fa4495413d0.height>",
"<NSAutoresizingMaskLayoutConstraint:0x7fa44985a980 h=-&- v=-&- _UIVisualEffectContentView:0x7fa4495413d0.height == UIVisualEffectView:0x7fa449552940.height>",
"<NSLayoutConstraint:0x7fa4497cf810 UIView:0x7fa44954ecb0.bottom == UIView:0x7fa449440060.bottom - 10>",
"<NSLayoutConstraint:0x7fa4497d4420 V:[UIView:0x7fa449440060]-(0)-[UIView:0x7fa44954ecb0]>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x7fa449556ca0 V:[UILabel:0x7fa449549cc0'This is an example title....']-(10)-|   (Names: '|':_UIVisualEffectContentView:0x7fa449552040 )>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKit/UIView.h> may also be helpful.
2015-03-23 10:21:42.916 RMPickerViewController-Demo[4943:195982] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want. Try this: (1) look at each constraint and try to figure out which you don't expect; (2) find the code that added the unwanted constraint or constraints and fix it. (Note: If you're seeing NSAutoresizingMaskLayoutConstraints that you don't understand, refer to the documentation for the UIView property translatesAutoresizingMaskIntoConstraints)
(
"<NSLayoutConstraint:0x7fa449556120 V:[UIVisualEffectView:0x7fa449553d70]-(10)-[UIVisualEffectView:0x7fa449552480]>",
"<NSLayoutConstraint:0x7fa449556170 V:[UIVisualEffectView:0x7fa449552480(44)]>",
"<NSLayoutConstraint:0x7fa4495561c0 V:[UIVisualEffectView:0x7fa449552480]-(0)-|   (Names: '|':UIView:0x7fa44954ecb0 )>",
"<NSLayoutConstraint:0x7fa449556210 V:[UIVisualEffectView:0x7fa449553d70(216)]>",
"<NSLayoutConstraint:0x7fa449556a50 V:|-(0)-[UIVisualEffectView:0x7fa449552940]   (Names: '|':UIView:0x7fa44954ecb0 )>",
"<NSLayoutConstraint:0x7fa449556aa0 V:[UIVisualEffectView:0x7fa449552940]-(10)-[UIVisualEffectView:0x7fa449553d70]>",
"<NSAutoresizingMaskLayoutConstraint:0x7fa4498045c0 h=-&- v=-&- UIVisualEffectView:0x7fa449551030.height == _UIVisualEffectContentView:0x7fa4495413d0.height>",
"<NSAutoresizingMaskLayoutConstraint:0x7fa44985a980 h=-&- v=-&- _UIVisualEffectContentView:0x7fa4495413d0.height == UIVisualEffectView:0x7fa449552940.height>",
"<NSLayoutConstraint:0x7fa4497cf810 UIView:0x7fa44954ecb0.bottom == UIView:0x7fa449440060.bottom - 10>",
"<NSLayoutConstraint:0x7fa4497d4420 V:[UIView:0x7fa449440060]-(0)-[UIView:0x7fa44954ecb0]>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x7fa449556120 V:[UIVisualEffectView:0x7fa449553d70]-(10)-[UIVisualEffectView:0x7fa449552480]>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKit/UIView.h> may also be helpful.
2015-03-23 10:21:42.921 RMPickerViewController-Demo[4943:195982] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want. Try this: (1) look at each constraint and try to figure out which you don't expect; (2) find the code that added the unwanted constraint or constraints and fix it. (Note: If you're seeing NSAutoresizingMaskLayoutConstraints that you don't understand, refer to the documentation for the UIView property translatesAutoresizingMaskIntoConstraints)
(
"<NSLayoutConstraint:0x7fa4497cf810 UIView:0x7fa44954ecb0.bottom == UIView:0x7fa449440060.bottom - 10>",
"<NSLayoutConstraint:0x7fa4497d4420 V:[UIView:0x7fa449440060]-(0)-[UIView:0x7fa44954ecb0]>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x7fa4497d4420 V:[UIView:0x7fa449440060]-(0)-[UIView:0x7fa44954ecb0]>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKit/UIView.h> may also be helpful.

Which version of Xcode / iOS SDK are you using?

Is this problem still present on version 1.4.0 and above?

Closing due to no response


Aerospike Object - Allow disabling SHM by passing FALSE as shm value.

I don't think this change is necessary since shared memory is disabled by default. See here

We have it enabled. However in code there we need to connect to multiple
clusters, we disable it.
ср, 18 дек. 2019 г., 2:15 AM dwelch-spike<EMAIL_ADDRESS>
I don't think this change is necessary since shared memory is disabled by
default. See here https://www.aerospike.com/docs/client/c/usage/shm.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aerospike/aerospike-client-php/pull/43?email_source=notifications&email_token=AEMMG76WFNUVA2FILBGTYL3QZFTQZA5CNFSM4JIXSPXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHEMY5I#issuecomment-566807669,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMMG73IRAJVGHZILFLKZW3QZFTQZANCNFSM4JIXSPXA
.

If you have a unique shm_key for each cluster, you can connect multiple clients to the those clusters with shared memory enabled. As long as the clients are configured to use the appropriate shm_keys they will not interfere with each other.

In such a case We'll need to manage several SHM addresses, which is not safe. Taking the C library history, into consideration where SHM address was changed due to changes in internal binary format makes it even more dangerous. The point is simplicity w/o config files diversifications along servers parks. If this patch is so 'disliked', then there is no much I can do to convince you. At worst case  it will remain local extra patch.

From the client config documentation:

shm_key
Explicitly set the shm key for this client.
If use_shared_connection is not set, or set to False, the user must provide a value for this field in order for shared memory to work correctly.
If , and only if, use_shared_connection is set to True, the key will be implicitly evaluated per unique hostname, and can be inspected with shm_key() .
It is still possible to specify a key when using use_shared_connection = True.

I'm not sure where the disconnect is - you already have full control to configure the behavior you want in the application. And if you're modifying the application for this already, I don't see how it's more or less dangerous than other options.
We're trying to avoid having multiple paths to do the same thing. Like you mentioned, you can fork the client.

What a client for Python has to do with one for PHP?
PHP client relies on C client library and uses default shared memory address. The limitation of PHP client is that you can only disable SHM globally in configuration of the extension, and not in runtime in object. This path allows to disable it.
"If , and only if, use_shared_connection is set to True, the key will be implicitly evaluated per unique hostname, and can be inspected with shm_key() ."
Nice for Python. No such logic in PHP extension...

@vityank I guess I shouldn't have been looking into it at a weird hour while jetlagged.  I worked ont he Python and PHP clients  in the past, so got some wires crossed.
Still need to understand where the  'safety' issue is. Your application has full control over whether SHM is used or not. You already have aerospike.shm.use = false (and it's the default!) so being able to also set aerospike.shm = false to achieve the same thing gives you no more functionality, and it's odd to have two ways  that  are so similar to get the same result.
aerospike.shm.use is  the top thing that controls if SHM is use or not. Changing the aerospike.shm.key doesn't modify this behavior,  so it should be adequate.

Even if "use_shm" is enabled in the php.ini, it will still be disabled by default in the clients unless a "shm" array is passed in the config. See this test for an example. You can disable shm on a client by client basis by omitting the shm array in the config.

Interesting, it didn't worked for me then I developed the Aspike
connector...
The false patch worked.
The bug i tried to avoid, which was tested by our own autetests was as
following:

$x=connect to cluster A
$y=connect to unreachable cluster B

W/o patch, even empty shm $y will end as connection to cluster B.
сб, 21 дек. 2019 г., 1:23 AM dwelch-spike<EMAIL_ADDRESS>
Even if "use_shm" is enabled in the php.ini, it will still be disabled by
default in the clients unless a "shm" array is passed in the config. See
this test
https://github.com/aerospike/aerospike-client-php/blob/5d61b2d76bdfdbf09daac160e27a1111dc788657/src/tests/Connection.inc#L437
for an example. You can disable shm on a client by client basis by omitting
the shm array in the config.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerospike/aerospike-client-php/pull/43?email_source=notifications&email_token=AEMMG74MBLOPAR3WZNSNZSDQZVHVLA5CNFSM4JIXSPXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHON73A#issuecomment-568123372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMMG77IJ4PNLZACRX353P3QZVHVLANCNFSM4JIXSPXA
.

Mistyped. $y ended as connection to cluster A, and this is incorrect
behaviour.
сб, 21 дек. 2019 г., 1:53 AM Виктор К<EMAIL_ADDRESS>
Interesting, it didn't worked for me then I developed the Aspike
connector...
The false patch worked.
The bug i tried to avoid, which was tested by our own autetests was as
following:

$x=connect to cluster A
$y=connect to unreachable cluster B

W/o patch, even empty shm $y will end as connection to cluster B.
сб, 21 дек. 2019 г., 1:23 AM dwelch-spike<EMAIL_ADDRESS>
Even if "use_shm" is enabled in the php.ini, it will still be disabled by
default in the clients unless a "shm" array is passed in the config. See
this test
https://github.com/aerospike/aerospike-client-php/blob/5d61b2d76bdfdbf09daac160e27a1111dc788657/src/tests/Connection.inc#L437
for an example. You can disable shm on a client by client basis by omitting
the shm array in the config.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerospike/aerospike-client-php/pull/43?email_source=notifications&email_token=AEMMG74MBLOPAR3WZNSNZSDQZVHVLA5CNFSM4JIXSPXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHON73A#issuecomment-568123372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMMG77IJ4PNLZACRX353P3QZVHVLANCNFSM4JIXSPXA
.

We have shm.use=true in our config, as most of the time we need it.
Nevermind.
сб, 21 дек. 2019 г., 12:51 AM Ronen Botzer<EMAIL_ADDRESS>
@vityank https://github.com/vityank I guess I shouldn't have been
looking into it at a weird hour while jetlagged. I worked ont he Python and
PHP clients in the past, so got some wires crossed.
Still need to understand where the 'safety' issue is. Your application has
full control over whether SHM is used or not. You already have aerospike.shm.use
= false (and it's the default!) so being able to also set aerospike.shm =
false to achieve the same thing gives you no more functionality, and it's
odd to have two ways that are so similar to get the same result.
aerospike.shm.use is the top thing that controls if SHM is use or not.
Changing the aerospike.shm.key doesn't modify this behavior, so it should
be adequate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerospike/aerospike-client-php/pull/43?email_source=notifications&email_token=AEMMG724NSRD7IEJ2YKH3JLQZVD7HA5CNFSM4JIXSPXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHOML3Q#issuecomment-568116718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMMG77DSJGX2AG5CQLJET3QZVD7HANCNFSM4JIXSPXA
.


Add req argument to custom success/error

Fixes https://github.com/pinojs/pino-http/issues/208

No problem. Should err still be first in that case?

If it's there, yes!

I swapped the order where I added req. Should I also add to customLogLevel? Does it still make sense to have err in a different order between customErrorMessage and customLogLevel?
customLogLevel?: ((res: ServerResponse, error: Error) => pino.LevelWithSilent) | undefined;
customReceivedMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
customSuccessMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
customErrorMessage?: ((error: Error, req: IncomingMessage, res: ServerResponse) => string) | undefined;

I also noticed that the docs have customProps while the Typescript types have reqCustomProps (and the code supports both). Perhaps if we're bumping the major version, this is a good time to rename one of them?

Go ahead and fix customLogLevel and resolve the problem with the types! Thanks for the help!

Ok, we now have this:
customLogLevel?: ((req: IncomingMessage, res: ServerResponse, error: Error) => pino.LevelWithSilent) | undefined;
customReceivedMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
customSuccessMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
customErrorMessage?: ((error: Error, req: IncomingMessage, res: ServerResponse) => string) | undefined;
customProps?: ((req: IncomingMessage, res: ServerResponse) => object) | undefined;

customErrorMessage still has error first, but I think it makes sense in that case.

Do you know when you might merge and release this?

Monday or Tuesday if @jsumners  doesn't beat me to it


fix: conditional advanced/required options, require input on required fields in Add Node modal

conditional render of advanced and required options
require input on all required input fields before moving onto next screen (solves #407 )
moved advanced options right under required options like spec

Again, great work!!


chore: update CODEOWNERS

📬 Code coverage report
🟢 Merging this PR to main will change coverage by 0.0% (100.0%)
No new files


How to use TPKeyboardAvoiding in swift

I want to use in my swift project . could u please suggest me ?. Thanks in advance

Hope it helps.

https://developer.apple.com/library/prerelease/ios/documentation/Swift/Conceptual/BuildingCocoaApps/MixandMatch.html

Add a bridging header for each of the .h files. Then you can use Storyboard to change the custom class to the relevant class.

These answers seem sufficient. I've used this in several Swift projects without any issues. If you hit a particular problem, please open another issue.


Add purely static export

Is your feature request related to a problem? Please describe.
Perseus is currently not technically JAMstack because it doesn't support eliminating server-side code. Many Perseus sites only rely on build-time rendering strategies, and performance would be improved if we could export to purely static files.
Describe the solution you'd like
The Perseus CLI should have an export command that creates a purely static set of files that can be served from anywhere, like a CDN, without any dependency on a server.
Describe alternatives you've considered
Zola is a great alternative in the meantime, but being able to have this work with Perseus would be a huge plus, and would get the project even with NextJS on JAMstack support.
Additional context
Add any other context or screenshots about the feature request here.

In theory, this should be able to be done by adding an extension of the build process for exporting, and shouldn't actually need any changes to the client-side app shell.

Okay, one slight pitfall of any approach to this is that we have to write the initial load files as .html files, which means whatever web server is used has to strip those extensions. For example, this causes things to break with Python's http.server. Somewhat ironically, Vercel's serve works perfectly! This problem should be very easy to solve though for whatever custom server ends up being set up.


provision-aws-cluster.sh vague error

Recently, the provision-aws-cluster.sh script got a new check at the end. It runs deisctl list repeatedly, looking for a successful exit. It times out after a few minutes.
https://github.com/deis/deis/blob/master/contrib/aws/provision-aws-cluster.sh#L150
The first few times I got this message, I thought the entire cluster install had failed, and I destroyed and re-installed the cluster multiple times. Later, I realized that this message is actually harmless.
I'm sure this is here for a good reason, but the messaging doesn't make any sense. Is it an error if it times out? If so... what are we supposed to do? If it's a warning, then put it in yellow. But in any event, give some clear indication of what the user needs to do.
In my tests, it appears that the only role for this is to delay until running the deisctl command to set the proxy type on nginx. If that's the case, let's make this explicit so that the user knows what to do.

The goal of the deisctl loop is to ensure provisioning waited until etcd2 is actually up and running. Checking fleet is an indirect way of doing that.
If you don't have the provisioning key available through ssh-agent, this will time out and print a red error that we couldn't verify core services are running. (Previously, if you don't have the correct key available, this would print a red error that proxyProtocol couldn't be enabled.) We could suggest in both error messages that the user run ssh-add.

That would be better. And that probably explains the error I got. I wish I'd captured the output, because it looked like Christmas -- red and green all over. But I had no idea what to do.
If the deisctl list thing is a real error, why don't we just stop the script and print an error with the kind of message you suggest? That would definitely be more helpful. Right now, it prints that vague error and then goes on to try to set proxyProtocol anyway.

Well...both are errors, but both are generally recoverable. If they succeed, we know the cluster's etcd2 is happy and proxyProtocol was set. If they fail, usually you can ssh-add and try again to set proxyProtocol, and you're fine.
Maybe we should make the first one a yellow warning and keep the second one red, then add a suggestion about ssh-add to the second error message? I can see that it's confusing to print the green "Your cluster is running" message in between these two errors as well; maybe that should go last?


Dev branch - addressing issue #69 workdir used incorrectly in agent construction

[Short description explaining the high-level reason for the pull request]
Additions

Removals

Removed line 82: relative_to = model_conf['workdir']

Changes

Edited line 85 from, self._model.model.resolve_paths(relative_to) to self._model.model.resolve_paths(self.job.workdir)

Testing

After receiving errors about 'workdir' not being in model_conf, edits above were made. To test, reran calibration scripts, and error no longer appeared.

Screenshots
Notes

Running on Ubuntu 22.04

Todos

Checklist

[x ] PR has an informative and human-readable title
[ x] Changes are limited to a single goal (no scope creep)
[ x] Code can be automatically merged (no conflicts)
[ x] Code follows project standards (link if applicable)
[ x] Passes all existing automated tests
[ x] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[x ] Linux
[ ] MacOS

Thanks @Ben-Choat !


Word cloud showing value(weight) with Key(word)

Hi, Wordcloud is showing numbers along with words. The numbers are display in small size. How can I hide the numbers (i.e. the value field)?
Here's my code:
const chart = new Chart(document.getElementById(visual.VisualId).getContext("2d"), {
type: "wordCloud",
data: {
labels: words2.map((d) => d.key),
datasets: [
{
label: "",
data: words2.map((d) =>d.value)
}
]
},
options: {
color: ThemeCols[0].toString(),
fit:true,
padding: 2,
title: {
display: false,
},
plugins: {
legend: {
display: false
}
}
}
});

I am using ChartJs version 3.5.1
and word cloud version 3
Thanks

I found a solution, reason for the issue was "Data labels plugin". I disabled the DataLabels plugin and the issue was resolved!
Here is fix for anyone facing the same issue.
const chart = new Chart(document.getElementById(visual.VisualId).getContext("2d"), {
type: "wordCloud",
data: {
labels: words2.map((d) => d.key),
datasets: [
{
label: "",
data: words2.map((d) => d.value)
}
]
},
options: {
color: ThemeCols[0].toString(),
fit: true,
padding: 2,
title: {
display: false,
},
plugins: {
datalabels: {
display: false,
},
legend: {
display: false
}
}
}
});


If you want to use GithubLink component, you must specify: owner, repo and branch.

https://opensource.huntersofbook.com/global/i18n
refresh page and see
github: https://github.com/huntersofbook/huntersofbook/tree/main/docs

Hello, what is the issue ?

Please see configuration docs https://docus.dev/introduction/configuration

@esoubiran-aneo The codes are already open with the settings. I shared the repo above. I wrote what the problem was. This error pops up when I refresh the page.

Yes, you need to update this object: https://github.com/huntersofbook/huntersofbook/blob/main/docs/app.config.ts#L26
Using this part of the table

Exemple: https://github.com/aneoconsulting/armonik-docs-theme/blob/main/app.config.ts#L23


Implement End To End Tests

Summary
This PR implements end to end tests to the platform, and modifies the CI workflow to run Docker Compose as part of the test suite.

This PR is way too big. Will be addressed by other (smaller) PRs.


Pages & Posts routings are interfering.

Hello everyone.
Child pages and child posts are interfering....
When i lead to a child post it is considered has a child page.
http://stackoverflow.com/questions/37740469/ui-router-pages-posts-routing-are-interfering
Cheerz !

Ok...  Re-ordering states declaration makes it work.


Build failure: sunshine

Steps To Reproduce
Steps to reproduce the behavior:

Update flake.lock to latest nixos-unstable
services.sunshine.enable = true;

Build log
please ignore the indentation:)
       last 25 log lines:
       >       |
       > In file included from /build/source/src/upnp.cpp:5:
       > /nix/store/vmzxminj8mwfw81whay2524c4b6w321q-miniupnpc-2.2.8/include/miniupnpc/miniupnpc.h:122:1: note: declared here
       >   122 | UPNP_GetValidIGD(struct UPNPDev * devlist,
       >       | ^~~~~~~~~~~~~~~~
       > /build/source/src/upnp.cpp: In member function 'void upnp::deinit_t::upnp_thread_proc()':
       > /build/source/src/upnp.cpp:334: error: too few arguments to function 'int UPNP_GetValidIGD(UPNPDev*, UPNPUrls*, IGDdatas*, char*, int, char*, int)'
       >   334 |         auto status = UPNP_GetValidIGD(device.get(), &urls.el, &data, lan_addr.data(), lan_addr.size());
       >       |
       > /nix/store/vmzxminj8mwfw81whay2524c4b6w321q-miniupnpc-2.2.8/include/miniupnpc/miniupnpc.h:122:1: note: declared here
       >   122 | UPNP_GetValidIGD(struct UPNPDev * devlist,
       >       | ^~~~~~~~~~~~~~~~
       > [ 68%] Building C object CMakeFiles/sunshine.dir/generated-src/xdg-output-unstable-v1.c.o
       > [ 70%] Building C object CMakeFiles/sunshine.dir/generated-src/wlr-export-dmabuf-unstable-v1.c.o
       > [ 72%] Building CXX object CMakeFiles/sunshine.dir/src/platform/linux/wlgrab.cpp.o
       > /build/source/src/upnp.cpp: At global scope:
       > /build/source/src/upnp.cpp:47: warning: 'std::string_view upnp::status_string(int)' defined but not used [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wunused-function-Wunused-function8;;]
       >    47 |   status_string(int status) {
       >       |
       > [ 75%] Building CXX object CMakeFiles/sunshine.dir/src/platform/linux/wayland.cpp.o
       > make[3]: *** [CMakeFiles/sunshine.dir/build.make:104: CMakeFiles/sunshine.dir/src/upnp.cpp.o] Error 1
       > make[3]: *** Waiting for unfinished jobs....
       > make[2]: *** [CMakeFiles/Makefile2:191: CMakeFiles/sunshine.dir/all] Error 2
       > make[1]: *** [CMakeFiles/Makefile2:198: CMakeFiles/sunshine.dir/rule] Error 2
       > make: *** [Makefile:190: sunshine] Error 2

Additional context
I think it's the same issue as #326140, UPNP_GetValidIGD expects more arguments
Notify maintainers
@devusb

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
~
󰘧 nix-shell -p nix-info --run "nix-info -m"
fetching path input 'path:/nix/store/dk2rpyb6ndvfbf19bkb2plcz5y3k8i5v-source'
warning: Nix search path entry '/nix/var/nix/profiles/per-user/root/channels' does not exist, ignoring
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.7, NixOS, 24.11 (Vicuna), 24.11.20240703.9f4128e`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Lix, like Nix) 2.90.0-rc1`
 - nixpkgs: `/nix/store/dk2rpyb6ndvfbf19bkb2plcz5y3k8i5v-source`

Add a :+1: reaction to issues you find important.

#326057 should solve it for 0.23.1 -- upstream has already fixed compatibility with new miniupnpc in https://github.com/LizardByte/Sunshine/pull/2782 as well, but the patch is not directly compatible with 0.23.1 -- next release should be able to work with new and old miniupnpc.

my bad, I missed that PR. and yes, it works after applying. I should close this issue.


Feature/updated docs including amazon

I merged this in the other PR, so I can add everything at once


Consume smithy-runner 3.5.5

In order to keep builds deterministic, and reduce the risk of future build failure, please pin your smithy runner to the currently defaulted runner.
For more information please see the email 'Smithy Runner Recommendation' or reach out to Travis Reed in hipchat.
Requested by: @travisreed-wf
@travisreed-wf

Raven
Number of Findings: 0


Feature: analysis jobs items crud

This pull request supersedes #288
This feature branch adds in support for Analysis Jobs' Items. In particular, this branch has the code, logic, and tests, for CRUD operations. It DOES NOT include actual integration with Analysis Jobs; that is when an AnalysisJob is created it should also create AnalysisJobsItems - that functionality will come in a later request.
This pull request also unifies the analysis results controller with the new analysis_jobs_items controller. All results, whether system or AnalysisJob generated are accessible via the same REST endpoint with same properties available.
This work lays out the foundation for resolving #192.
Fixes #274.

Coverage decreased (-1.2%) to 84.539% when pulling 931066b5eb30925e34e659bdff91e5b55abce764 on feature-analysis-jobs-items-crud into 16d1de08ec0ba134b14d9e6a6bbeaaa08b23e261 on develop.

Coverage increased (+0.4%) to 86.096% when pulling fbe08eede6c23fb316198760e23afa9816e637f5 on feature-analysis-jobs-items-crud into 16d1de08ec0ba134b14d9e6a6bbeaaa08b23e261 on develop.


Avoid executing filters on all candidates

Dynamic version selection via version listing causes every version to be considered a candidate.
Context
A combination of large numbers of exclude rules to support our Ivy to Maven migration and the implementation of our  alignment feature using ranges results in a very large number of calls to quite expensive content filtering for each candidate version. This avoids this by filtering as each candidate is considered (ignore the times here, it's a tracing profiler, the call counts are what's important):

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


--sample-config success, then ends with crash

Running email --sample-config writes a sample config, then crashes something like this:
...
  },
  "keys": {
    "testkey": "1PAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA9I"
  },
  "serialized_creds": {
    "_comment": "This gets populated programatically"
  }
}
Traceback (most recent call last):
  File "/usr/local/bin/email", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/site-packages/emailer/shell.py", line 53, in main
    config_obj, creds = get_config_and_creds(options.config_dir)
  File "/usr/local/lib/python3.7/site-packages/emailer/shell.py", line 35, in get_config_and_creds
    config_obj = config.load_from_file(config_path)
  File "/usr/local/lib/python3.7/site-packages/emailer/config.py", line 81, in load_from_file
    raise InvalidFileError(config_path)
emailer.config.InvalidFileError: None

Trying to get it set up in the next 30 min so I probably won't debug this right now.  Just logging the issue.

Using docker python:alpine environment (3.7)

Ah... this happens when there is no config file to load at all. Now to fix it without introducing unnecessary conditionals in shell.py :/


audio: NewPlayer should be an independent function

https://github.com/hajimehoshi/ebiten/commit/9a18ffe5637106fa3e065e8656a63cea39a5a517 Fixed this


host/ble_gap.c : Check allowed random address

Add a check to validate whether the random address being set is allowed as per spec

Hi @sjanc , can you please review this change.

I think this should be in some common code eg ble_hs_id.c ?  Some helpers for generating and internal helpers for validating type of addresses.
There are similar checks for static random when calling ble_hs_id_set_rnd() etc

I think this should be in some common code eg ble_hs_id.c ? Some helpers for generating and internal helpers for validating type of addresses. There are similar checks for static random when calling ble_hs_id_set_rnd() etc

Thanks for the review . I have made a generic function to check if passed address is as per spec or not.

@sjanc , Please let me know if you have any more suggestions / comments.

@sjanc ping

Hi @sjanc  , please take a look


cannot import name _minibatch

Hi, I am unable to run the pyxmeans benchmark after installation. It fails with this error:
cannot import name _minibatch
I'm using the anaconda distribution of python with Numpy 1.8.2 and compiling with gcc 4.9 on a mac. The package compiled and installed successfully, with some warnings generated.

@mynameisfiber I have the same problem. No errors during installation. Installed simply with python setup.py install.

This is probably happening because you are trying to import pyxmeans while inside of the pyxmeans directory.  Python will try to load the package in the current directory instead of looking for the installed version.
electron ~/projects:$ python -c 'import pyxmeans'
electron ~/projects:$ cd pyxmeans
electron ~/projects/pyxmeans:(master)$ python -c 'import pyxmeans'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "pyxmeans/__init__.py", line 1, in <module>
    from . import _minibatch
ImportError: cannot import name _minibatch

Please tell me if this is infact the problem!


Cards: CodeSandbox

This issue is a reminder that once we've completed debugging our Codesandbox issue, this component needs to be included in our CodeSandbox repo.

https://codesandbox.io/s/productive-card-3qln0
https://codesandbox.io/s/expressive-card-jeq35


Connecting, disconnecting and then reconnecting to VoiceChannel results in panic writing to closed channel

Hi,
I've mostly cribbed the example code for your voice-receive example and embedded it in a bot to allow me to control it.. but despite following the example where you close(*discordgo.VoiceConnection.OpusRecv) and then *discordgo.VoiceConnection.Close() (and I then s.ChannelVoiceJoinManual(r.Guild.ID, "", true, false) to make the bot leave the voice channel... there isn't an explicit ChannelVoiceLeave() it seems?)
On starting speaking, it panics:
panic: send on closed channel

goroutine 70 [running]:
github.com/bwmarrin/discordgo.(*VoiceConnection).opusReceiver(0xc0000de420, 0xc000366088, 0xc0000a3b00, 0xc0004ea480)
        /home/azekeil/go/pkg/mod/github.com/bwmarrin/discordgo@v0.23.2/voice.go:840 +0x2d8
created by github.com/bwmarrin/discordgo.(*VoiceConnection).onEvent
        /home/azekeil/go/pkg/mod/github.com/bwmarrin/discordgo@v0.23.2/voice.go:441 +0x6d8
exit status 2

Have I done something wrong?
Thanks in advance, appreciate your library!

I have same problem too.


[cherry]fix(core): Add a new parameter worker_event_max_payload to kong.con…

…f (#11214)

With a hard-coded payload size, for some use cases like uploading a big OpenAPI spec in DevPortal or updating a big config entry for plugins, they can not work as expected. With the new parameter, the user can decide the payload size to meet their needs.

In this PR, a new parameter, worker_events_max_payload is added, which allows to specify the payload size the worker_events lib can accept. The default size is 64k, and the max allowed to set is 16M Bytes.
The corresponding PR for worker_events lib is #37
FTI-4963

add changelog entry

Update kong.conf.default

add test case and bump lua-resty-events

correct the default value, and add an entry for bumping the version of lua-resty-events

append PR number to the changelog entry of lua-resty-events

correct the spec test
style

Update CHANGELOG.md

Summary
https://github.com/Kong/kong/pull/11214
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference

Fix #[issue number]

wrong cherry-pick


firebase_messaging onMessage callback not fired for "data" message on iOS

I have the same problem with onMessage not being called on data messages under iOS.  In all test cases the Android and iOS flutter app is running in foreground.  Messages with the "notification" work fine on both Android and iOS but when I remove the notification element and send just a "data" message, then the iOS application does not receive the onMessage callback.
In my case I am using python on my back-end server to publish notification messages.  The python code looks like:
                msg = firebase_messaging.Message(
                    android=firebase_messaging.AndroidConfig(
                        priority='high', ttl=30.0),
                    apns=firebase_messaging.APNSConfig(
                        headers={'apns-priority': '10'}),
                    data=data_message,
                    # notification=firebase_messaging.Notification(
                    #    title='GC Runner data',
                    #    body=data_message['op']),
                    token=fcm_token)
                rc = firebase_messaging.send(msg)

Everything works fine (iOS and Android receive onMessage callback) when the notification parameter is passed into the Message constructor.  When I comment out this parameter (as shown) then the iOS app stops receiving the onMessage callback.
Here is my flutter doctor output:
C:\projects\mobile\golfdb>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v1.4.9-hotfix.1, on Microsoft Windows [Version 10.0.17134.706], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[√] Android Studio (version 3.3)
[√] VS Code, 64-bit edition (version 1.33.1)
[√] Connected device (2 available)

• No issues found!

Here is my pubspec.yaml:
C:\projects\mobile\golfdb>type pubspec.yaml
name: golfdb
description: A new Flutter project.

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# Read more about versioning at semver.org.
version: 1.0.0+14

environment:
  sdk: ">=2.1.0-dev.5.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  intl: ^0.15.7

  shared_preferences: ^0.4.3
  flutter_secure_storage: ^3.1.2
  device_info: ^0.2.1
  http: ^0.11.3
  cached_network_image: ^0.5.1
  flutter_cache_manager: ^0.2.0
  uuid: ^1.0.3
  charts_flutter: ^0.5.0
  flutter_markdown: ^0.2.0
  path_provider: ^0.4.1
  audioplayers: ^0.7.8
  keyboard_visibility: ^0.5.1

  firebase_core: ^0.3.4
  firebase_messaging: ^4.0.0+3
  firebase_analytics: ^2.1.1+3

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.2

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_launcher_icons: "^0.7.0"

flutter_icons:
  image_path: "icons/icon-test.png"
  android: true
  ios: true

# For information on the generic Dart part of this file, see the
# following page: https://www.dartlang.org/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  # assets:
  #  - images/a_dot_burr.jpeg
  #  - images/a_dot_ham.jpeg
  assets:

See https://github.com/flutter/flutter/issues/28196

@cgfischer was the issue solved?

I believe that it has been solved in a newer version.  I have not been able to verify yet because attempts to upgrade have failed due to dependencies.

Hi @cgfischer
given your last message I feel safe to close this issue,
if you disagree please write in the comments
and I will reopen it.
Thank you