Add the complex data type transformer for complex type handling

Part 1 for the complex type handling. Design doc: https://docs.google.com/document/d/1L6d0k_yF4TbFWHt5IJ5eYYjJZboRnJTBrQYRB2an9iI/edit#
Description
Add the transformer to transform the complex data type, which includes map flattening and collection unnesting.
With the transformation, the complex struct group in the meetupRsvp example is flattened into primitive types, and can be queried like the following.

release-notes
Added configs  complexTypeHandlingConfig to ingestionConfig.
for example
           complexTypeHandlingConfig: {
              "mode": "FLATTEN_MAP",
              "unnestConfig": ["group.group_topics"]
           }

When the mode is NONE (default), there is no complex type handling enabled.
When the mode is FLATTEN_MAP, then the map (or struct/record) is flattened by default.
The collections to unnest needs to be configured in option unnestConfig as a list of paths, per design doc.

Codecov Report

Merging #6845 (d394f11) into master (3e20979) will decrease coverage by 30.59%.
The diff coverage is 7.14%.

@@              Coverage Diff              @@
##             master    #6845       +/-   ##
=============================================
- Coverage     73.99%   43.39%   -30.60%
+ Complexity       12        7        -5
=============================================
  Files          1421     1423        +2
  Lines         69141    69286      +145
  Branches       9986    10024       +38
=============================================
- Hits          51159    30069    -21090
- Misses        14633    36667    +22034
+ Partials       3349     2550      -799

Flag
Coverage Δ
Complexity Δ

integration
43.39% <7.14%> (-0.17%)
7.00 <0.00> (ø)

unittests
?
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

this is awesome
lets ask some users to provide feedback on the spec?
       complexTypeHandlingConfig: {
          "mode": "FLATTEN_MAP",
          "unnestConfig": ["group.group_topics"]
       }

we

this is awesome
lets ask some users to provide feedback on the spec?

okay, any users that you have in mind?
btw, the purpose of adding mode is to allow more policies. For example, another possible default rule is not to flatten the map but instead convert it to JSON string.

       complexTypeHandlingConfig: {
          "mode": "FLATTEN_MAP",
          "unnestConfig": ["group.group_topics"]
       }

we

I was thinking if this should be part of transformConfig and model it as a transformFunction. As I user, I would like to see all the transformations in one place.
Also, is this happening before the decoder or after the decoder?
@npawar ^^

I was thinking if this should be part of transformConfig and model it as a transformFunction. As I user, I would like to see all the transformations in one place.
Also, is this happening before the decoder or after the decoder?
@npawar ^^

I see. If you look at the impl, it uses Transformer API. But I grouped the configs separately for better organization.
It happens after decoder and before other transformers, as described in the design doc.

For the complex type column, I can see 4 different operations:

SKIP (not include in the output)
FLATTEN (into key-value pairs, can generate multiple records for list)
RETAIN (as Map or List)
TO_JSON_STRING

We can have a default operation, then have an override field list for each operation.
We should probably separate the map config from the list config because I can see a common case where user only want to flatten map but not the list, so that list can be stored as MV column.
Also, I think we should make delimiter configurable. I remember there are some issues storing column name with dot in presto connector (@xiangfu0 to confirm).
So the overall config would look like:
complexTypeConfig: {
  "map": {
    "default": "FLATTEN",
    "SKIP": ["colA"],
    "TO_JSON_STRING": ["colB"],
    "delimiter": "_"
  },
  "list": {
    "default": "RETAIN",
    "FLATTEN": ["colC_listField"],
  }
}

@amrishlal, can you also help review this PR ?

I was thinking if this should be part of transformConfig and model it as a transformFunction. As I user, I would like to see all the transformations in one place.
Also, is this happening before the decoder or after the decoder?
@npawar ^^

I think it is fine to separate this flatten config from transform config. TransformConfig has a "column" -> "function" style. Whereas this flatten config seems to be for the entire record, with some special column callouts for unnesting further? So if there's multiple maps, this config will flatten them all, if I understand correctly.

For the complex type column, I can see 4 different operations:

SKIP (not include in the output)
FLATTEN (into key-value pairs, can generate multiple records for list)
RETAIN (as Map or List)
TO_JSON_STRING

We can have a default operation, then have an override field list for each operation.
We should probably separate the map config from the list config because I can see a common case where user only want to flatten map but not the list, so that list can be stored as MV column.
Also, I think we should make delimiter configurable. I remember there are some issues storing column name with dot in presto connector (@xiangfu0 to confirm).
So the overall config would look like:
complexTypeConfig: {
  "map": {
    "default": "FLATTEN",
    "SKIP": ["colA"],
    "TO_JSON_STRING": ["colB"],
    "delimiter": "_"
  },
  "list": {
    "default": "RETAIN",
    "FLATTEN": ["colC_listField"],
  }
}

SKIP (not include in the output)
Is this needed? If the column (or all descendent columns) does not show up in the schema, then it means skipped?
FLATTEN (into key-value pairs, can generate multiple records for list)
Yes, this is the current default behavior for map
RETAIN (as Map or List)
Same, this info shall be conveyed by the columns in schema
TO_JSON_STRING
I was thinking about this, and I feel it can be done via jsonFormat as your current jsonMeetupQuickStart example uses. Not sure if we shall add this syntax sugar in this config.

For the complex type column, I can see 4 different operations:

SKIP (not include in the output)
FLATTEN (into key-value pairs, can generate multiple records for list)
RETAIN (as Map or List)
TO_JSON_STRING

We can have a default operation, then have an override field list for each operation.
We should probably separate the map config from the list config because I can see a common case where user only want to flatten map but not the list, so that list can be stored as MV column.
Also, I think we should make delimiter configurable. I remember there are some issues storing column name with dot in presto connector (@xiangfu0 to confirm).
So the overall config would look like:
complexTypeConfig: {
  "map": {
    "default": "FLATTEN",
    "SKIP": ["colA"],
    "TO_JSON_STRING": ["colB"],
    "delimiter": "_"
  },
  "list": {
    "default": "RETAIN",
    "FLATTEN": ["colC_listField"],
  }
}

SKIP (not include in the output)
Is this needed? If the column (or all descendent columns) does not show up in the schema, then it means skipped?
FLATTEN (into key-value pairs, can generate multiple records for list)
Yes, this is the current default behavior for map
RETAIN (as Map or List)
Same, this info shall be conveyed by the columns in schema
TO_JSON_STRING
I was thinking about this, and I feel it can be done via jsonFormat as your current jsonMeetupQuickStart example uses. Not sure if we shall add this syntax sugar in this config.

Discussed offline and we only need the unnestFields and delimiter in the config for now:
"complexTypeConfig": {
  "delimiter": "_",
  "unnestFields": [...]
}


9-page Paper Draft

PLEASE FINISH WRITING YOUR ASSIGNED SECTIONS BEFORE SUNDAY, 11/29, so Jane can proofread/organize as needed. The fresh report outline and .tex files should be going up as a pull request very soon, sorry about the delay. If your section already exists in some form from the previous draft, the .tex file for the section is just copied over. If not, it will be blank.
In addition to the assigned sections outlined below, we also need to go over/update the abstract, intro, data, and discussion sections. Jane is happy to do that when she organizes/proofreads on Sunday, but you are all more than welcome to contribute too.
Methods:
Smoothing (Rachel)
Convolution and time correction (Ben)
GLM (Jane)
Normality checks/assumptions (Kent)
Hypothesis testing (Hiro)
Benjamini-Hochberg (Rachel)
Clustering (Hiro)
Results:
GLM (Jane/Ben)
Hypothesis testing -- normality and Benjamini-Hochberg are lumped in here too (Hiro/Rachel/Kent)
Clustering (Hiro)
Appendix (If you can fit everything you want to say gracefully in the main paper, it's fine not to do these):
Convolution analysis (Ben)
Benjamini-Hochberg (Rachel)
Clustering (Hiro)
Time series (Jane)
NOTE: If you are using citations in your appendix section, please uncomment the 3 other lines in the make file  associated with your section. If you're not using citations, it's fine to leave the make file as is and I'll clean it up. But to get your citation references to render correctly, you'll need to uncomment the 3 other lines for your section.

@benjaminleroy
@reychil
@kentschen
@hiroto-udagawa


Using Wave for public website recommended?

Hi, just asking if I should / can use Wave for a tiny public website hosted within IIS (or not)

"Serious" site on WAVE web server is built to be accessed through reverse proxy.
If it is a "tiny" site, then you can use WAVE instead of IIS.
Make sure you protect all ports with firewall though.


Controllers should recover from failures

All controllers should continually drive towards desired state, even in the face of failures.  They should retry, operations should be idempotent, they should be able to handle partially applied state, etc.
From the roadmap:

Retry/recovery from failure, idempotence, dealing with partial state

This will be some fairly advanced logic to make the controllers smart enough to know how to fix error conditions on their own without human intervention.

I think we could proooobably call this fixed once all controllers are ported to the generic managed resource reconciler.


When using clone: false watchers are not triggered with correct entities

Hello,
I have a watcher on a table and the db is initialized with clone: false. If I, for example, upsert an entity, the callback of the watcher is called without the changed entity. It works if clone is set to true (which is the default).

Sorry, can't reproduce your issue :/ Does your setup differ in any significant way from the testcase below? It passes for me.
let users: User[];
let userTable: Table<User, "id">;
let fn: jest.Mock<void, [User[]]>;

describe("without clone", () => {
  beforeEach(async () => {
    users = generateRandomUsers();
    const db = createDB({ clone: false });
    userTable = createTable<User>(db, "users")();
    await insertMany(userTable, users);

    fn = jest.fn<void, [User[]]>();
  });

  it("should trigger watch if a matching entity is inserted (#23)", async () => {
    await watch(userTable, { where: { name: { in: ["Alice", "Eve"] } } }, fn);
    const eve: User = { id: "300", name: "Eve", age: 5 };
    await upsert(userTable, eve);

    expect(fn.mock.calls.length).toBe(2);
    expect(fn.mock.calls[1][0].sort(sortById)).toStrictEqual(
      [...users.filter(u => u.name === "Alice" || u.name === "Eve"), eve].sort(sortById)
    );
  });
});

Ok, I figured out the root of the problem. If I change a property of an object while not using clone and the upsert the object, the watcher does not return it as changed since the object is the same in the table already.. :sweat_smile:

Modifying objects returned from the DB if you're using clone: false will almost certainly result in bugs. Afaik I can't really prevent such modifications (apart from systems like change tracking which would require additional overhead). clone: false gives you a performance "boost" by not having to clone returned objects, but almost requires you to uses immutable objects.
If you have code that changes properties like this:
const user = await one(userTable, { id: "300" });
user.name = "Eve";
await upsert(userTable, user);

consider rewriting it to look like this:
const user = await one(userTable, { id: "300" });
await upsert(userTable, { ...user, name: "Eve" });

I know that running into bugs like this is surprising, but this is the price you have to pay for an API that works without clone :/
Since I don't expect there to be a follow up, I'm closing this issue.

Since it doesn't really make sense to use it with clone: false anyway, maybe the option should be removed completely?


Use --no-deploy after the fact

Is your feature request related to a problem? Please describe.
Unfortunately our cluster was deployed using the k3s defaults before we really understood the implications of using Traefik's default config. Now, we'd like to use Nginx instead of Traefik.
Is it possible to remove Traefik and/or other services deployed by default without removing the cluster and starting over?
Describe the solution you'd like
A command or process to add --no-deploy to a pre-existing cluster.
Describe alternatives you've considered
We've considered uninstalling the cluster and starting over, but this seems fairly drastic.

"kubectl delete -n kube-system helmcharts traefik" would remove the default traefik.

Thanks!

Is it possible to do the same for servicelb?
Trying kubectl delete -n kube-system helmcharts servicelb I get this error:
Error from server (NotFound): helmcharts.helm.cattle.io "servicelb" not found

unless you have services with type=loadbalancer, i think you can just reboot with --no-deploy servicelb

What do you mean by "reboot"?

What do you mean by "reboot"?

would also be interested how to reboot with --no-deploy-servicelb

I had success removing servicelb by executing the install command I originally used again, and then restarting the k3s service.
Any stray servicelb pods can then be manually deleted if they don't disappear.

Thanks @radicand, that worked.
To confirm, with a pre-existing cluster I ran the following:
curl -sfL https://get.k3s.io | sh -s - server --no-deploy traefik --no-deploy servicelb
sudo k3s kubectl get pods --all-namespaces --watch
sudo k3s kubectl delete -n kube-system helmcharts traefik

This will clean up the cluster, but if you reboot... helm will re-install traefik. To resolve this, you'll also need to remove/rename the manifest file:
sudo mv /var/lib/rancher/server/manifests/traefik.yaml /var/lib/rancher/server/manifests/traefik.yaml.bak


Improve token counting for messages with package

Purpose
As I needed our token counting and message building functionality for another sample, I moved it to a separate package. That also gave me the opportunity to increase test coverage and receive community contributions for improving it, so it is more accurate than what's currently in this repo (notably, it can now count tools, which it was ignoring before).
You can see the interface for the other package here:
https://pypi.org/project/openai-messages-token-helper/
This PR deletes the code/tests from this repo and replaces them with equivalent functionality in that package.
Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[X] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.
[ ] Yes
[X] No

Type of change
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[X] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Code quality checklist
See CONTRIBUTING.md for more details.

[X] The current tests all pass (python -m pytest).
[ ] I added tests that prove my fix is effective or that my feature works: There is already a test, test_chat_with_long_history, that effectively uses the functionality.
[X] I ran python -m pytest --cov to verify 100% coverage of added lines
[ ] I ran python -m mypy to check for type errors
[X] I either used the pre-commit hooks or ran ruff and black manually on my code.

I've merged from main, this is ready for review.


Fails to prove Twitter identity (for a public account)

I tried to prove my Twitter identity: https://twitter.com/jeehoon_kang/status/1670464797668958209
But it doesn't work: Keybase says it fails.
I tried multiple times revoking and retrying again.
I wonder how I can proceed. Thanks!

I just changed my account and tries to reprove Twitter identity: https://twitter.com/jeehoon_kang/status/1671686973491417089
But it still fails...

There might be a possibility that their Twitter API keys might have some issues due to recent Twitter API pricing changes.

I'm also having this issue.
https://twitter.com/MatioWhho/status/1678116101866295296?s=20
https://keybase.io/matiowhho/sigchain#ee8c7ef3dfbea5739edd2c8e0850dce49350690badddbb104d230ebd4d3499f60f

reddit fail
https://www.reddit.com/r/KeybaseProofs/comments/15bneex/my_keybase_proof_redditvonheiising/
twitter fail
https://twitter.com/unclegeekbrasil/status/1684786422371295232?s=20

Not working for me either https://github.com/keybase/client/issues/25759#issuecomment-1826795820


Nothing is saved in router if there is no port

Date - 12/3/19
Steps:

Go to Create Router
Add Name, App and Version but leave port blank
Create
Edit or look at YAML

Results:  The App and Version are not saving.

Date - 12/5
Verified fixed


About trams and ships

Players can't take trams or boats. The original geyser doesn't have this problem

I will look into a fix when I next update

I believe the problem is caused by heath123's move fixes
I have got the boats to work some what
You have to spam the enter and exit buttons a couple of times then it will allow you to stay sitting in the boat

is this still a issue?

I believe this issue is now fixed


Interactive shell in the test process is erratic due to stdin being piped to both the server and the test processes

Is this a bug report or a feature request?
Bug report
If this is a bug report, please provide as much info as possible

version: 2.0.0
platform: Windows 10 > WSL2 > Ubuntu 22
expected behavior > the shell in the test command works fine
actual behavior > the shell's input is glitchy

When using a test program which has interactive shell, the interactive shell will behave in an erratic way: some characters are not registered on keypress. It's randomly about 50% chance that the character is written to the test process's stdin. The exact case I'm running is with the 'smashtest' package. It's not really relevant, but for completeness, here is my command line:
start-test 'PORT=5001 npm start | dev-null' tcp:<IP_ADDRESS>:5001 'PORT=5001 smashtest tests/*.smash'
Smashtest is running a Node.js built-in REPL instance. I have debugged the problem and found that start-server-and-test uses stdio: 'inherit' on both the server and the test child processes, which doesn't make sense, as we can't control both. If I set the server to stdio: ['ignore', 'inherit', 'inherit'], the problem goes away.
I assume people didn't run into this problem because they either used this library with a non-interactive headless test session, or with a GUI test program.
I've made a PR which fixes the issue: https://github.com/bahmutov/start-server-and-test/pull/369

closed in #369


EKS Nodes NotReady with node to node encryption enabled.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When setting encryption.nodeEncryption to true we noticed that all of our EKS nodes can join the cluster but are always NotReady. The cilium agents are unable to start and any attempts to contact the node fail. Here are our helm values.
  values:
  - eni:
      enabled: true
      iamRole: <iam role arn>
      subnetIDsFilter: <subnet ids>
      awsReleaseExcessIPs: true
    ipam:
      mode: eni
    tunnel: disabled
    egressMasqueradeInterfaces: eth0
    tls:
      enabled: true
      secretsBackend: k8s
    encryption:
      enabled: true
      nodeEncryption: true
      type: ipsec
      ipsec:
        keyFile: keys
        secretName: cilium-ipsec-keys
        mountPath: /etc/ipsec

Are we missing any configuration options we are missing that would allow us to enable node to node encryption on EKS?
Cilium Version
1.11.0
Kernel Version
Linux 5.4.156
Kubernetes Version
1.21
Sysdump
cilium-sysdump-20211222-142036.zip
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hello, can you provide the logs of the cilium agent that it's not able to become ready?
Also, I've noticed from the sysdump that you have provided that you are using ArgoCD. There have been a couple issues opened by users regarding ArgoCD. For example https://github.com/cilium/cilium/issues/14550 and https://github.com/cilium/cilium/issues/17349. You might want to have a look at those as well.

Might be a duplicate of https://github.com/cilium/cilium/issues/18323

@aanm I am on the same team as @Ian-Greene and am working on this too.  We actually are not able to access logs for pods on the node.  When the instance comes up logs are available via the Kubernetes api (i.e. via kubectl) but after about 30s these become inaccessible due to connection timeouts.  I would note that we are unable to ssm onto the nodes as well with this configuration applied.
I would also note that we are not deploying Cilium via ArgoCD, and we are not using the filter configs mentioned in https://github.com/cilium/cilium/issues/18239.

@aanm I am on @Ian-Greene's team working on this as well.
I am unable to provide logs because the logs become unavailable via the Kubernetes api (i.e. via kubectl) for these pods about 10 seconds after they start trying to come up -- we start getting connection timeouts.  We are also unable to ssm onto the nodes when this occurs due to connection timeouts.
I'd also note that Cilium is not being deployed via ArgoCD, and that I do not think we are using the problematic filter configs mentioned in https://github.com/cilium/cilium/issues/18239.

@aanm I have the same issue and the subnetfilter issue doesn't seem to be related. I do not have subnet filters enabled and after installing cilium with nodeEncryption enabled, the nodes become unreacheable, even after uninstalling cilium. The nodes have to be replaced.


Host can change currency

Flarum has recently moved to their own fiscal host from OSC. Because they are in Europe, their currency is showing in Euros, but they actually want to stay with US dollars. How can they change the currency? I can't find this setting in the Host or Collective settings.

A few unwanted side effects:

Backers were stored with USD, see this.
Backers on a tier no longer are on a tier. Seeing active backers is now only possible using the API.

I am seriously hoping we didn't just lose all recurring donations, because that would be a huge hit to our financial stability and thus our sustainability. We are moving to get one core developer to work for us part time as a freelancer, we need all these funds for running costs and his salary!
To be honest, I actually wrote @alanna (who could have never known this) about my fears that something bad could happen and demanded clarification about the exact procedure. Even though the procedure was simple, it struck me as odd that after changing fiscal host I could only "add funds" in Euro's and no longer as dollars. If I had known in advance it would have made this a lot easier and maybe I would have decided against switching to our own fiscal host. This exact thing happened with Tenancy back then, so no one actually tested this and documented the oddities of doing the procedure. For a platform working with money I'm just hoping that at some point you will see the wisdom of making things like this less of a surprise to anyone.
tldr: Can anyone tell me:

Can we (easily) switch back to USD if we wanted to? It really doesn't matter to Stripe, they will accept USD from OC and then pay us to our bank account in EUR.
Did we just lose all recurring contributions?
Can someone push the recurring backers that were on a tier, back into a tier?

This is not meant as a rant, even though it feels a bit like it. I do love what OC is and does, I just feel disappointed (again).

Can we (easily) switch back to USD if we wanted to? It really doesn't matter to Stripe, they will accept USD from OC and then pay us to our bank account in EUR.

Yes. I think keeping everything in USD is the sensible choice if the goal is to keep the current recurring contributions running. What behavior would the existing contributors expect?

Did we just lose all recurring contributions?

No. According to the documentation, the subscriptions should be kept https://docs.opencollective.com/help/collectives/change-fiscal-host#what-happens-to-my-recurring-financial-contributions-followers-of-updates-and-transaction-history
Seems you created new Tiers when switching in euros (and deleting the old ones), the new Tiers are the one displayed and no one subscribed to them. The old tiers are deleted and not appearing, the attached subscriptions are still there.

Can someone push the recurring backers that were on a tier, back into a tier?

Yes, that can be done.

Let's troubleshoot this issue over our support channels, happy to help on Slack or<EMAIL_ADDRESS>

To come back to this ticket original purpose, this is not something that we want automated or done by the users.

Collectives should always use the Host currency for accounting
Collectives with past transactions should not be allowed to change their currency (they should be able to swap hosts but not with to an Host with a different currency)
Hosts with past transactions should not be allowed to change their currency

This rules are dictated by the current limitations of our systems. In an ideal world we would be able to provide such flexibility.


Add eBPF for network monitoring based on containermgr

Heads-up, I rebased #738 and made some changes that introduced a couple conflicts. git rebase --onto origin/sharnoff/neonvm-runner-crictl HEAD^^ should mostly do the trick.

@save-buffer remind me please, this is blocked on two things: https://github.com/neondatabase/cloud/issues/11655 and availability of cgroups v2 in the Amazon Machine Image, right?

Yes that's correct

I'll close it then for the time being.


MSDKUI-1836: [Design - Android] UI review for GuidanceManeuverView

GuidanceManeuverView - match parents - Content should be top aligned
GuidanceManeuverView - no data & updating state - text should be aligned with the icon

Codecov Report

Merging #95 into master will increase coverage by 1.74%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #95      +/-   ##
============================================
+ Coverage     72.89%   74.63%   +1.74%
+ Complexity     1636      995     -641
============================================
  Files           129       69      -60
  Lines          5630     3931    -1699
  Branches        870      436     -434
============================================
- Hits           4104     2934    -1170
+ Misses          858      750     -108
+ Partials        668      247     -421

Impacted Files
Coverage Δ
Complexity Δ

...com/here/msdkui/guidance/GuidanceManeuverView.java
84.15% <100%> (+0.26%)
30 <0> (ø)
:arrow_down:

...e/msdkuiapp/routing/RoutingInstructionsFragment.kt

...java/com/here/msdkuiapp/common/PermissionsUtils.kt

.../com/here/msdkuiapp/routing/OptionPanelFragment.kt

...ere/msdkuiapp/routing/RouteManeuverListFragment.kt

.../here/msdkuiapp/routing/SubOptionPanelPresenter.kt

...m/here/msdkuiapp/routing/SubOptionPanelFragment.kt

...msdkuiapp/guidance/GuidanceNextManeuverFragment.kt

...com/here/msdkuiapp/routing/OptionPanelPresenter.kt

...m/here/msdkuiapp/position/AppPositioningManager.kt

... and 51 more

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 9437029...a651a54. Read the comment docs.


about left right eye landmarks question

In all of the landmark connection vector found it FaceMesh_Map.jpg and Contour keypoints. The landmark for the left eye is: left eyes:
. The landmark for the right eye is: right eyes:

But in iris_tracking_gpu.pbtxt. define left eye boundary is (in L84-L96):

define right eye boundary is (in L98-L110):

In other words, the landmarks in pbtxt and the left and right eye landmarks defined by keypoints are inconsistent. Where is wrong?

Yes, I also noticed this inconsistency when I was trying to change the range of the eye_boundary_landmarks. Did you find the correct landmarks_indices mapping yet?

@04633435 I think Contour keypoints is right, because it matches the mirror image we perceive. So currently in the graph, I adjust it according to it. I haven't waited for the official reply yet, I'm still waiting for the official explanation.

@04633435 I think Contour keypoints is right, because it matches the mirror image we perceive. So currently in the graph, I adjust it according to it. I haven't waited for the official reply yet, I'm still waiting for the official explanation.

Thank you for your reply!
I just look through the Contour keypoints, but there are only 229 points, half of the vertices excluding the iris landmarks. Seems like there are some points missing. Do you have any clue?

@04633435 Face Mesh has a total of 468 points. iris is an extra 10 points(left 5, right 5). Not included in FaceMesh.
In iris_tracking_gpu.pbtxt ConcatenateNormalizedLandmarkListCalculator code can be found.
Note: left iris landmarks and right iris landmarks the order is different, you have to distinguish it yourself. I made changes in my app, so this part can’t be shared with you.

Hi, @lilinxiong, I solved my problem. And for the problem you mentioned above, in L84-L96, it selects out the inner corner and outer corner landmarks of the left eye, and within the subgraph iris_landmark_landmarks_to_roi.pbtxt of the iris_landmark module, these two landmarks are used to create a square boundary around the left eye area. So in L84-L96, there are only landmark 33 and 133, not the whole left eye contours landmarks.

@04633435 I don’t think we are talking about the same issue.
Face Mesh has a total of 468 points. iris is an extra 10 points(left 5, right 5). ........

My answer here is that the iris coordinates are added to the content related to FaceMesh landmark.
Including the order of iris coordinates and the position added to FaceMesh.
And for the problem you mentioned above, in L84-L96, it selects out the inner corner and outer corner landmarks of the left eye.....

As for what you are answering now, I think you only need to exchange the subscript values of left and right in the original pbtxt.
The original intent of this issue is that the left and right in pbtxt are written inversely, so even after swapping left and right, the result is still the opposite. You can verify it yourself.
I'm not sure, whether you know what I want to express, but the final conclusion is: even after you swap left and right, their final value is unchanged, just changing here will not take effect. Other places need to be changed.

Hi @lilinxiong ,
Are you still looking for the resolution on this issue.

Hello @lilinxiong,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediaPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions.


RethinkDB support

Will rethinkdb ever be supported by Lucid?
If not in the near future, how would one go about using RethinkDB within the adonisjs

I don't think I will be adding support for same. Rethink DB has some official listing for wrappers for Node.js https://www.rethinkdb.com/docs/nodejs/


Adds stamina damage to the toy sword

About the PR

Gives 8 stamina damage to the toy sword, so you can have duels with your friends instead of endlessly hitting eachother with plastic. Was going to be 5 since it doesn't deal slight brute like boxing gloves but the toy hammer also deals 8 stam so whatever.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Gave the toy sword slight stamina damage, like the boxing gloves and toy hammer.

Don't do this. It can become a mass-produced stun weapon.
Instead, make a recipe for training Bokken that needs wood and supplies from botany.

Don't do this. It can become a mass-produced stun weapon.

Since when you can produce it?
Its a rare drop off arcades and happy honks.

Don't do this. It can become a mass-produced stun weapon.

Don't do this. It can become a mass-produced stun weapon.

Boxing gloves and toy hammers do the same stamina damage as what I added and are way easier to get and i'll be surprised if you can find someone using any of them as actual stun weapons.

Also to note, you need 15 melee hits from a toy e-sword to actually stun someone, even if its mass produced it wouldn't really matter.

real

this is fine, it cant be produced and it has precedent from others


ccm creates cluster with Murmur3Paritioner minimum token

When I run the following commands to create a cluster:
ccm create test
ccm populate test -n 3
ccm start test
ccm node2 nodetool ring

I get in the following ring:
Datacenter: datacenter1
==========
Address    Rack        Status State   Load            Owns                Token
<PHONE_NUMBER>618258602
<IP_ADDRESS>  rack1       Up     Normal  46.57 KB        66.67%              -9223372036854775808
<IP_ADDRESS>  rack1       Up     Normal  46.56 KB        66.67%              -3074457345618258603
<IP_ADDRESS>  rack1       Up     Normal  46.58 KB        66.67%<PHONE_NUMBER>618258602

The assigned token for one of the nodes is -9223372036854775808 which is Java's Long.MIN_VALUE. This token shouldn't be assignable as it's used in queries to wrap around the entire token ring (get entire range of tokens).

CCM has always worked like this. Are you sure the minimum token is unassignable? I'm aware it's used in queries to wrap around the entire token ring.


Could you add select from dropdown using value in ElementActions

I'm dealing with a dropdown has the same exact text in all options
Describe the solution you'd like
I need to use the "value" attribute by using ElementActions.select()

the same method handles values natively... the value you provide to this method is searched and if it is found as a text or as a value it is selected.
look here for more info:
https://github.com/MohabMohie/SHAFT_ENGINE/blob/master/src/main/java/com/shaft/gui/element/ElementActions.java#L959
specially this condition:
https://github.com/MohabMohie/SHAFT_ENGINE/blob/master/src/main/java/com/shaft/gui/element/ElementActions.java#L980
kindly try it and let me know.

Closing this issue for lack of response. Please feel free to comment and reopen it if needed.


Css height control

Thumbnails always at 16x9 ratio
Thumbnails on cards will always be centered and cropped to 16x9, regardless of the dimensions of the original image.

Just tested it, works great!


Anchore CLI is not working

Is this a request for help?:

Is this a BUG REPORT or a FEATURE REQUEST? (choose one):

Version of Anchore Engine and Anchore CLI if applicable:
What happened:
What did you expect to happen:

Any relevant log output from /var/log/anchore:
What docker images are you using:
How to reproduce the issue:
Anything else we need to know:
Hello,
For a demo setup I want to use Anchore for container vulnerability testing. However, I am not able to run the Anchore CLI with the error:
DEBUG:anchorecli.clients.apiexternal:As Account = None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): localhost:8228
Error: could not access anchore service (user=admin url=http://localhost:8228/v1/): HTTPConnectionPool(host='localhost', port=8228): Max retries exceeded with url: /v1/ (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f585bcc61f0>: Failed to establish a new connection: [Errno 111] Connection refused'))
Please note that I am able to browse the APIs, however:
http://localhost:8228/v1
http://localhost:8228/v1/images/list
Though I don't find any image, at least I can run the API service successfully.
Kindly let me know if I am missing anything.

Hello @cjmakwana, thank you for filing the ticket. Can you share some context by answering the following questions

Where is anchore-engine deployed  - docker-compose or k8s
What is the exact anchore-cli command used and where was it executed from
Was there any additional configuration for anchore-cli such as environment variables overriding the url, username, password etc


Bluetooth: controller: split: Fix Conn Param Req procedure stall issue

Fix an issue wherein local or remote initiated Connection
Parameter Request procedure would stall without generation
of LE Connection Update Complete HCI event because a local
or remote initiated PHY Update procedure has overwritten the
currently active Link Layer Control Procedure type.
Related to #12139
Signed-off-by: Alexander Svensen<EMAIL_ADDRESS>

Does this need to be 1.14 back-ported?

Does this need to be 1.14 back-ported?

@ioannisg : No, this is a "forward port" of a fix which was done in the legacy controller into the split link layer. The split link layer is not considered supported in 1.14


no public directory ?

Hi ! noob here. i have no idea of what i'm doing honestly. i'm on a debian 8 srv with a LAME config and i manage to run the serv. but after a black page with what needed to login, i have this error.
file_get_contents(/var/www/koel/public/build/rev-manifest.json): failed to open stream: No such file or directory

here is my koel folder and i don't have any public folder

i don't remember having any error on the installation but as i have say, i really don't know what i'm doing in web.

gulp problem. you do not need to run this root. also consider using docker.

as i say, i'm a complete ignorant. so, i read that Gulp is a javascript task runner that lets you automate tasks, so don't get the why. and i will learn to use docker in the future, but it's way to high for me now.

npm install should get everything in place.


Use composeExtensions to preserve existing Python package overrides

Resolves #183
This PR addresses the scenario where the qchem overlay is applied after another overlay that overrides Python packages. Currently, the previously-applied overrides will be lost due to Python overrides not composing by default (https://github.com/NixOS/nixpkgs/issues/44426).
The following is a minimal reproduction:
let
  qchemOverlay = import ./overlay.nix;

  myOverlay = _: prev: {
    python3 = prev.python3.override {
      packageOverrides = (_: _: { foo = 42; });
    };
  };

  pkgs = import <nixpkgs> {
    overlays = [
      myOverlay
      qchemOverlay
    ];
  };
in
pkgs.qchem.python3.pkgs.foo

Currently, this expression fails to evaluate:
$ nix eval -f test.nix
error: attribute 'foo' missing

This PR modifies the qchem overlay to preserve existing overrides, using the variant of override that accepts a function from old to new overrides and lib.composeExtensions. This is similar to the approach used in tweag/jupyterWith.

Currently, this expression fails to evaluate:
$ nix eval -f test.nix
error: attribute 'foo' missing

nix-build test.nix works with this (e.g. use a real package):
let
  qchemOverlay = import ./overlay.nix;

  myOverlay = _: prev: {
    python3 = prev.python3.override {
      packageOverrides = (_: p: { foo = p.numpy; });
    };
  };

  pkgs = import <nixpkgs> {
    overlays = [
      myOverlay
      qchemOverlay
    ];
  };
in
pkgs.qchem.python3.pkgs.foo

nix-build test.nix works with this (e.g. use a real package):

@markuskowa ah, that's a more realistic reproduction!
Just to clarify, I assume you mean this works only on the PR branch? (Before the patch, this also fails for me, as expected)

Yes, it works on the PR branch, and now also on master.


Can this method be used to synthesize panoramas?

Hello, this method is great! ! !
I would like to know if this method can be used to composite a panorama？

As far as I understand it, the difference between a panorama and the 360 degree scenes that we evaluate on is that a panorama is typically background-only, and could probably be modeled more simply than a full Plenoxel grid (e.g. the foreground grid could be low resolution).

I see. So can I generate a panorama directly with your code? If so how can I set the parameters?

There is an option to render only the background of a scene so technically it might be possible.
But the dataset you need to train a Plenoxel grid will be more then enought to create a panorama with much higher resolution and better quality in seconds using existing software like ptgui.

When I train the TanksAndTempleBG dataset, I always run out of memory when the resolution is [640, 640, 640]. How can I adjust the parameters?
The configuration I train on is a GTX Titan with 12212MB of video memory.

To reduce the memory requirements there are a few things to try. You can try reducing the final resolution, changing the optimizer (from RMSProp to SGD; SGD uses much less memory but also require more tuning of learning rate), and/or increasing the regularizers to encourage more sparsity.

I tried it, but got really bad results, I was wondering what graphics card memory you guys are using


error on refresh page with unplugin-vue-router

Describe the bug
i'm building vue app using unplugin-vue-router.
the problem is occured if we had dynamic path containing symbol e.g<EMAIL_ADDRESS>, this cause an error when we refreshing the page, it has no problem on the first redirect using spa.
i've open issue on https://github.com/posva/unplugin-vue-router/issues/34 but it seems like the problem is on the vite site
Reproduction
https://github.com/zynth17/unplugin-vue-router-bug-dynamic-path
System Info
System:
    OS: Linux 5.10 Ubuntu 20.04.4 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 7 4800H with Radeon Graphics
    Memory: 11.59 GB / 11.99 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
    Yarn: 1.22.17 - ~/.nvm/versions/node/v16.13.2/bin/yarn
    npm: 8.1.2 - ~/.nvm/versions/node/v16.13.2/bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^3.0.0 => 3.0.0
    @vitejs/plugin-vue-jsx: ^2.0.0 => 2.0.0
    vite: 3.0.0 => 3.0.0

Used Package Manager
pnpm
Logs
vite:spa-fallback Rewriting GET / to /index.html +0ms
vite:time 15.48ms /index.html +0ms
vite:spa-fallback Not rewriting GET /manage-people/manage-user/user/detail/dummystaff%2540mail.com?tab=user because the path includes a dot (.) character. +40ms
vite:time 3.05ms /manage-people/manage-user/user/detail/dummystaff%2540mail.com?tab=user +27ms
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #2415


[in_app_pur] Add screenshots to pubspec.yaml

Capturing from offline discussion: is there a way we can add this to the package's page but not the search results, since the image isn't really useful in the search context?

Capturing from offline discussion: is there a way we can add this to the package's page but not the search results, since the image isn't really useful in the search context?

Thanks for adding the comment here. At this point there is not, unfortunately.
I agree in the consideration that this package does not offer a visual component, and hence might not be the best candidate for including screenshots. However, a screenshot might still give some value in the search context.
In any case, I think we should drop this PR for now, and see how the use of screenshots will evolve.

@stuartmorgan can you clarify how you find this "not useful"? It seems very useful to me to be able to see the payment UI when searching, as that is a good indication of the functionality of the plugin.

My understanding from the UX studies we did, visuals were desired as a way of confirming that a certain package is what the developer is looking for. So white the UI may be OS provided, a screenshot of it is a very strong visual clue to a developer that they have found the thing they are looking for.
cc @InMatrix to get his input

That makes sense to me in general, but my assumption is that "for making in-app purchases through the App Store and Google Play" is so unambiguous that a search listing screenshot is just going to be noise in this particular case. But if my mental model on this is wrong we can certainly reopen the PR.

Let's consider this pub.dev query: https://pub.dev/packages?q=In-App+Purchase. The in_app_purchase package is the 6th result.

Will providing a screenshot of in_app_purchase in the results page help the user distinguish the package from the rest of the search results?
What if all top 10 packages on that search results page provided screenshots? Will they look sufficiently different to provide value to the user?

Not knowing enough about the other packages returned from the query, I don't have good answers to these questions. However, others in the thread might have a better idea.

From a quick skim:

The first one looks like it's probably wrapping the same libraries that in_app_purchase is, so my comment above about UI not being a distinguishing factor would almost certainly apply here.
The second is one of the platform implementations of in_app_purchase; it should not be higher in search. An implementation package (one with implementation in the pubspec) should never be higher relevance for a generic search than the app-facing package, so that's a search ranking problem more than anything else.
The third one looks like a UI library, where screenshots would help distinguish.
I'm not sure what the fourth one is so I'm not sure
The fifth isn't a real package, it's an ad for a Udemy course. With 7 likes and 26% popularity. Maybe screenshots would help, but as above that seems like a way to try to paper over a significant ranking issue.

Ranking issues aside (they are orthogonal to the issue), likely the first one on the list and our in purchase package would both use a screenshot of the native payment UI. I think that would be hugely helpful in having those two stand out against the others.

Okay, we can give this a shot then. This will just need to be updated to re-bump the version.

@szakarias can you help with that update?


Add cargo watch

Describe the problem you are trying to solve

During development you are very often in the process of making changes and then build or run your project or run your tests. This always costs some time, therefore an automation would be useful, to save some time and increase the developer experience
Describe the solution you'd like

I'd like to have a built-in watcher similar to nodemon for NodeJS or like this project https://github.com/passcod/cargo-watch
This would allow you to run your whole project if you change something or you can also specify to run only tests when your files change:
cargo watch -x run
cargo watch -x test
Since this is completely optional to use, developers are not forced to use the suggested watcher.
Notes

@Zerotask, I usage the cargo-watch package and is like nodemon. Why do you think that is necessary include this on cargo if the package cargo-watch solve this problem?
When I need make any tests without cargo, I run on homemade script like this:
while inotifywait -e close_write *; do rustc main.rs; done

Sorry if I did not understand correctly your request.

Yes, I also mentioned this package and the reason why I think it should be added to the default toolchain of cargo.
If a command is often used, one could consider to add it to the default toolchain.

Yes! I understand your wish now and this make sense. I use every day cargo watch too.

Just note that Cargos has a function list_files respecting package.include and package.exclude. This might be worth a reuse than inventing a brand new file listing mechanism. There are also change detection instructions in build scripts we need to consider. O
To be fair, I don't know any details of cargo-watch. epage's suggestion seems like a great starting point to me.

Another thing I'd add is to study multiple forms of prior art for features, pitfalls, etc so and report on them and what you suggest we should apply from them.
We do this when adding commands but I suspect a --watch flag has enough little pieces of policy to be worth it as well.

While this might only be implemented a bit at at time, we should consider what all commands should support this.
Speaking of a bit at a time, a plan for implementing this, at least from MVP to what you care about.

I think the MVP should only care about of watching for files changes under src (or respectives if not a binary package) and then recompile it as of cargo run

High-priority

@zzz6519003 it helps nothing leaving a comment like “High-priority”. At this moment, cargo-watch and its underlying watchexec-cli are good alternatives before a first-party subcommand comes out. They both work seamlessly to me.

Copied from epage's comment:

By "adding" cargo watch to cargo, I think integrating it as a --watch flag is included in that. Probably first thing is a thorough understanding of cargo watch and what works well and doesn't. Besides having a general idea that you want this, we need an understanding of what that means in practice. The discussion should likely continue on that linked issue.

I think it is what to do next if someone wants to give it a try. Further more, you might want to go through a (pre-)RFC process or similar to solicit community feedback. In addition, commands other than build can also benefit from watch mode, so we should make the scope of the MVP and the future extension clearer. For example, do we want to have Cargo in daemon mode? Is it related to how watch mode is integrated? As a good reference, you can check how cargo <file> -Zscript has been done.

Triage: in terms of technical details, the MVP needn't be a perfect one. I'd say the efforts level almost reaches “hard” but let's make it a medium one for now. @rustbot: label +E-medium +S-needs-design

@zzz6519003 it helps nothing leaving a comment like “High-priority”. At this moment, cargo-watch and its underlying watchexec-cli are good alternatives before a first-party subcommand comes out. They both work seamlessly to me.

Copied from epage's comment:

By "adding" cargo watch to cargo, I think integrating it as a --watch flag is included in that. Probably first thing is a thorough understanding of cargo watch and what works well and doesn't. Besides having a general idea that you want this, we need an understanding of what that means in practice. The discussion should likely continue on that linked issue.

I think it is what to do next if someone wants to give it a try. Further more, you might want to go through a (pre-)RFC process or similar to solicit community feedback. In addition, commands other than build can also benefit from watch mode, so we should make the scope of the MVP and the future extension clearer. For example, do we want to have Cargo in daemon mode? Is it related to how watch mode is integrated? As a good reference, you can check how cargo <file> -Zscript has been done.

Triage: in terms of technical details, the MVP needn't be a perfect one. I'd say the efforts level almost reaches “hard” but let's make it a medium one for now.
@rustbot label: +E-medium +S-needs-design

Hey found out about this thread from someone linking me https://blog.rust-lang.org/inside-rust/2024/02/13/this-development-cycle-in-cargo-1-77.html — happy to answer any queries y'all might have.


Fix backslash in CSS urls on Windows

Do not use path.join because it introduces backslashes on Windows platforms. Fixes #114 .

@daniel-rikowski Are you able to add changelog entry to https://github.com/shakacode/shakapacker/blob/master/CHANGELOG.md
Good to merge after that

Of course, didn't notice the request earlier...


Allow package.json as an alternative to separate config file

I'm looking for ways to use this package with least clutter.
I also want to have .markdownlint.json a separate file, for better tools integration. (For example, vscode extension supports only that file.)
Since I only need .markdownlint-cli2.jsonc for ignores, I would prefer to have it inside package.json, like many other tools allow.
This will also hopefully help to prevent the globs from getting overlooked when there is a time for them to be updated.

Now that the VS Code extension supports .markdownlint-cli2.jsonc, are you still interested in this? So far you're the only person in 5 months to ask. :)

This is cool.
I have to reevaluate now whether I can comfortably switch from CLI1 in my small projects.
Can use single config file now - check.
Have to explicitly maintain globs for markdownlint - I think I'm more relaxed about it now.

Great! FYI, you can set globs and ignores both in the same .markdownlint-cli2.jsonc file if you want an empty command line.
https://github.com/DavidAnson/markdownlint-cli2#markdownlint-cli2jsonc

I migrated from -cli to -cli2 in my packages and pretty happy about it, except some vscode extension issue.
As a result, I have no need in this feature anymore. Feel free to close it or keep it for others consideration.

Great! I'll close for now since it only came up once in 6 months.

I also need this feature (config section inside package.json). How about davidtheclark/cosmiconfig?

Yes why not reopen this? It was not actually implemented.


[UI] Cluster - Cannot RKE cluster as cluster owner when admin created the cluster

Similar issue: https://github.com/rancher/rancher/issues/12038
Version: master-head (v2.3) (8/8/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps and results:

Create a cluster (without CT) as an admin. Make a user "test1" a cluster owner.
Log in as the "test1" user, try to edit the Cluster. Rancher UI thinks it's a custom cluster so they can't properly edit the cluster.

Create another cluster (this time with a simple CT) as an admin. Make user "test1" a cluster owner.
Log in as the "test1" user, try to edit the Cluster. We have access to the cluster template (even though we did not share the CT) per https://github.com/rancher/rancher/issues/21473 so we don't need to be shared the CT.
Same issue, "test1" user sees cluster as custom and when edits cluster gets "Custom" cluster UI for editing (asking to chose roles and update nodes with new commandline).

I made a fix here that changed this but forgot that the previous view was deemed acceptable in 2.2. I will revert this change to get us back to the old behavior.
In order to fix this whole custom thing we need a way for either the node-template to be shared or at the very least a way to figure out what provider the cluster is on without access to the nodes.

Version: master-head (v2.3) (8/12/19)
Still not quite right, the revert was an attempt to let us have the workaround in https://github.com/rancher/rancher/issues/12038 where we pick the rke cluster and can then edit properly. The main issue is we don't have access to the node template, which is a known issue. The past way we could from the UI pick what kind of rke cluster it is and then edit properly.
When I log in as the "test1" user and try to edit the cluster I just see a blank page now:

@cjellick Can I get the nodeTemplate->driver copied onto the nodePool so this can be improved instead of just reimplementing the asking for what driver to use thing we had in 2.2?

Backend PR merged to master.

Version: master-head (v2.3) (9/3/19)
I tested both scenarios. I didn't identify any issues.
Scenario 1:
Create a cluster (without using a template).

Make sure you add "test1" user as a cluster owner
Log in as "test1" user and edit the cluster - we now are able to edit without any issues.
Change something and ensure the change takes and the cluster updates accordingly.
Smoke test to ensure the cluster is still in a good state and usable.

Scenario 2:
Same steps as above, except create the CT as admin and create the cluster with this CT as admin.
Make sure to add "test1" user as a cluster owner and test editing again.
In both scenarios this now works without any issues. The "test1" cluster owner user is able to edit a cluster whether or not it was created with a CT without any issues.


makes date recomputation optional when invalid date is entered (Resubmitting PR1158)

This is resubmitting PR #1158 , but now passes the JSLint and unit tests for the current build.

Thanks for resubmitting the PR! Can you also add the docs for this new option?

Of course. will do that asap.

+1


Adjusting the minLength for Brazil numbers

Adjusting the minimum number of digits for Brazil, because there are numbers that contain 10 digits.
Mobile phones have changed to 11 digits, but there are still home phones that contain 10 digits
@all-contributors please add @adilsonjuniordev for code

Waiting..

@adilsonjuniordev Is there any external source like Wikipedia or a reputable outlet to support your claim?

@keezysilencer Yes. I will send here:
https://en.wikipedia.org/wiki/Telephone_numbers_in_Brazil
10 digits (landlines)
11 digits (mobiles)

@keezysilencer
Telephone formats are standardized by ANATEL, which is Brazil's national telecommunications agency.  There are the two formats below:
AA NNNN-NNNN (landlines)
AA 9NNNN-NNNN (mobile lines)

Waiting...

Waiting....

use intl_mobile_field package


Feature request: allow adding separators in the left hand services column

I would like to be able to visually separate my "work" services from my "personal" services inside the Franz window, and it seems like the easiest way to do that would be to allow adding a simple horizontal separator. I've mocked up the sort of thing I mean; see the image below.

I'm going to add "groups" soon. On top of having dividiers, you'll be able to enable/disable groups and provide overrides for options like notifications, audio, ...

Is there an ETA for when this feature would be added?
Loving FRANZ btw!

I'm with Benjamin, also loving Franz here ;)
I would totally use groups/folders. I don't have too much need for separator, but I guess they could be useful too.

Yeah, as I'm moving more into Franz, removing all those pesky pinned tabs from Chrome, I'd also love this feature. A lot of these things are memory hogs (Slack, Gmail, FB messenger) and being able to toggle off "home" / "work" stuff easily to recover that memory, and remove unwanted distractions during the work day / weekend would be good
(right now, I use two chrome profiles, and minimize or shut down the chrome window in question)

Unstale

Guess we're not getting separators then?

@adlk said in https://github.com/meetfranz/franz/issues/1352 that this is "coming soon". But I wonder if it will also come with disabling multiple grouped services at once, since I do not have enough memory to spare for all my services on my laptop.


Resolves: Add native GitHub continuous code security and quality analysis

add codeql-analysis.yml which automatically enables CodeQL code security and quality scanner. It executes on every push commit, PR, manually and every day at 8:00AM UTC. A scan check can be viewed on commits and PRs. All alerts and fix suggestion can be viewed at Security tab -> Code scanning alerts -> CodeQL

Resolves #169

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: szlatkow sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[Explorer] Some problem for question 84ec06ef-8a00-48f3-9ab7-4741b1fd32af: tidb主干分支代码都是哪些年写的？

Hi, I have some problems with the question 84ec06ef-8a00-48f3-9ab7-4741b1fd32af tidb主干分支代码都是哪些年写的？ (errorType = none):

Generated SQL
WITH
  main_branch_code AS (
    SELECT
      ge.created_at,
      ge.additions,
      ge.deletions
    FROM
      github_events ge
    WHERE
      ge.repo_id = (
        SELECT
          repo_id
        FROM
          github_repos
        WHERE
          repo_name = 'pingcap/tidb'
        LIMIT
          1
      )
      AND ge.type = 'PushEvent'
      AND ge.action = 'pushed'
      AND ge.number = 'master'
  )
SELECT
  DATE_FORMAT(mbc.created_at, '%Y') AS year,
  SUM(mbc.additions) AS additions,
  SUM(mbc.deletions) AS deletions
FROM
  main_branch_code mbc
GROUP BY
  year
ORDER BY
  year ASC
LIMIT
  20

Chart:
{
  "chartName": "LineChart",
  "title": "Code written in the master branch of tidb",
  "x": "year",
  "y": [
    "additions",
    "deletions"
  ]
}

Result:
// Fields
  [
  {
    "name": "year"
  },
  {
    "name": "additions"
  },
  {
    "name": "deletions"
  }
]

// First result (Totally 0 rows)
  undefined

We can't answer question about "主干分支" as there is no branch info in table github_events, :-)


Update our README files to reflect the Scope concept.

Coverage remained the same at 78.598% when pulling 413bd58c61bd24d0361715c17f8cc5be0fe2e197 on carlosalberto:v0.31.0_README_update into 818ae53041df95c274398f99e56536bcef0fa4b1 on opentracing:v0.31.0.

Thanks Carlos! I'm going to merge this in, as it's easy to make edits via additional PRs.


E0001 Error for SAR Apps

cfn-lint version:  cfn-lint 0.15.0
Description of issue
While running cfn-lint on a nested CloudFormation stack containing AWS::Serverless::Application resources, it errors with:
Resources:
  AppName:
    Type: AWS::Serverless::Application
    Properties:
      Location:
        ApplicationId: arn:aws:serverlessrepo:us-west-2:<my-account-id>:applications/<app-name>
        SemanticVersion: 1.0.0
      Parameters:
        Debug: 'True'
        MemorySizeMB: 128
        TimeoutSec: 300

$ cfn-lint template.yml

E0001 Resource with id [AppName] is invalid. Application with id arn:aws:serverlessrepo:us-west-2:<my-account-Id-hosting-the-app>:applications/<app-name> could not be found.
template.yml:1:1

Supporting evidence:

The Application definitely exists, since the template runs with no issues.
I have admin permissions with the current user to make, update, view, delete the app.

Interesting. Validation of serverless resources occurs via the SAM validation libraries. Do you happen to have the SAM CLI installed? I'm curious what results you get validating through the CLI.

@cmmeyer I get the following:
$ sam validate --template infra.yml
infra.yml is a valid SAM Template

@cmmeyer we actually use their Translator.  https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/transform.py#L118  Before SAM SAR we would validate the translated templated from SAM.   What I think we need to figure out is what Translator does with a SAM SAR application.  My basic understanding is in the end its just a nested template.  My hope is it isn't reaching out to SAM SAR to validate and pull information about the application.

Oddly enough, I'm having another issue with a Serverless template running in CI:
E0001 Error transforming template: 'module' object has no attribute 'logging'

Really not sure where to start with this one. It's on the latest version of cfn-lint too... Any ideas @cmmeyer ?

. @Dunedan figured out the logging issue:
https://github.com/aws-cloudformation/cfn-python-lint/pull/702
So it looks like you may have gotten sam-translator 1.10.0 in the mix at some point. I wonder if the other issues may stem from that as well.

Not solving it, but is it an idea to output the transformed template with --debug?

Playing around with this and I think I got it.
*(I'm running sam-translater 1.9.0, so it's not related to 1.10.)

The error is actually coming from SAM since it's a E0001  error.
SAM actually does stuff in AWS, so your AWS Credentials need to be valid to retrieve the data needed (I get a Resource with id [AppName] is invalid. User: XXX is not authorized to perform: serverlessrepo:CreateCloudFormationTemplate on resource: arn:aws:serverlessrepo:us-west-2:<my-account-id>:applications/<app-name> error when running this sample (least privilege here))

Based on this:

I created a PR to clarify the transformation. Added output of the transformed template (if successful)
and the region to the debug output (-d or --debug) and added an explicit Error transforming template to the error message.
Can you check the privileges on your side? I think it has to do something with that

Can you check the privileges on your side? I think it has to do something with that

I'm using admin privs, so it can't be that.

I think @fatbasstard meant your AWS privileges.  Since it looks like SAR is causing it to reach out to AWS you will need credentials and privileges to query it.  We have tried to remain completely offline during all of our linting but this new feature in SAM has changed that.  I want to dig a little farther into why its doing that, etc.  To see if we can stop it or prevent it.

Might even be the region.. Which region is the function in, and against what region is the linter running?
It might search for the function in another region, which causes the "cannot find it". The linter is running against us-east-1 if no region is specified

My guess is we are going to have to translate this before we go to SAM to prevent it reaching out to the account.  This is an example of what gets output on full transformation.  The Template URL is probably provided by the call out to the serverless repo.  The good part is we don't worry about the validity of the Url only the properties and their values for formatting reasons.   What I'm proposing is we translate AWS::Serverless::Application to its equivalent resource AWS::CloudFormation::Stack voiding the TemplateURL.   Or maybe there is a clever way to provide the feedback to the call that samtranslator is making to provide it a generic TemplateURL.
"Bucket": {
      "Type": "AWS::CloudFormation::Stack",
      "Properties": {
        "TemplateURL": "https://awsserverlessrepo-changesets-plntc6bfnfj.s3.amazonaws.com/AIDAJEF4MAIJQFOIKLXX6-51d20263-e5dc-461c-8d34-029afde3a90f.yaml?X-Amz-Security-Token=FQoGZXIvYXdzEAoaDLPbjmWojhcH4U1rcyKCApkuQ%2ByITP3xcIIJAQO5Ehty%2F6GWw2rHaNaxECDXgItWZh1Q6cPt0HXeTEWo8nGuohPU6IluBfzuuFfh5c6gXC3JXzygRxbloe3rSljQaI3G3ior7LqTEARPcauKlEV1uSwQ%2BS0PsiJfv8QNms0%2BV46w8iPrgDBwTrTJt%2FgLhHykWM15M4nbql1OmlC4x4866A%2FqQ5Qwtgc60Q%2FWxu4sqEj9J6y9ElwGsLUAe5oZTp%2FcnOzmkEh%2FwDfeBBq60tcZpCm0YAKHU6HB9G39Uhkze%2Bps3Y1CgTR6PHO2a71oHJJKKP88QNzIJhzTjoLXjqnSJhv8O7aaREQhqd6lVDMKBync%2FCimuZ%2FkBQ%3D%3D&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Date=20190312T164245Z&X-Amz-SignedHeaders=host&X-Amz-Expires=21599&X-Amz-Credential=ASIAQ63C33MXBRRTMRDP%2F20190312%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Signature=40f006d885d7f4b2af8da077e1ef09d9c5ecda6c4d3bab5ae288d1bb9713fb65",
        "Parameters": {
          "BucketName": "test-123456789012-us-east-1"
        },
        "Tags": [
          {
            "Value": "SAM",
            "Key": "lambda:createdBy"
          },
          {
            "Value": "arn:aws:serverlessrepo:us-east-1:123456789012:applications/test",
            "Key": "serverlessrepo:applicationId"
          },
          {
            "Value": "0.0.1",
            "Key": "serverlessrepo:semanticVersion"
          }
        ]
      }
    }

Found a way to translate this.  Release v0.17.0 is being built now and will have this fix.

Oddly enough, I'm having another issue with a different SAM template running in CI (no nested stacks or anything, just some AWS::Serverless resources):
E0001 Error transforming template: 'module' object has no attribute 'logging'

Really not sure where to start with this one. It's on the latest version of cfn-lint too... Any ideas @cmmeyer ?

I have the same issue as @jacknagz .
I am using cfn-lint 0.22.0 for AWS:Serverless:Function as below:
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31 # It transforms SAM syntax to Lambda function CloudFormation template: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/transform-aws-serverless.html
Resources:
  MyLambda:
    Type: AWS::Serverless::Function # More info about Function Resource: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#awsserverlessfunction
    Properties:
    ...

@kddejong Does it make sense to re-open this issue?

Do you know what version of aws-sam-translator you have?

@kddejong  I am using aws-sam-translator 1.12.0.

We still have this issue, has a fix been release yet? Thanks

@dinvlad what version of aws-sam-translator and cfn-lint are you using?
@cjongseok are you still having this issue?  We require 1.15.1 now.

@kddejong OK, I think I misdirected the issue. We stumbled upon on this in the regular sam validate, not cfn-lint. The latter seems to work fine!


Install all header for ctkMacroBuildLib

default qt version set to qt5
install all headers with ctkMacroBuildLib and  maintain the directory heirarchy

Closing. Superseded by https://github.com/commontk/CTK/pull/1150
The update to ctkMacroBuildLib were not included since there is already install rules for explicitly installing *.h and *.tpp files. Instead of adding another rule the existing rule should be fixed.
https://github.com/commontk/CTK/blob/c57612f76b7743b76878be73aaea27936ee1d7c5/CMake/ctkMacroBuildLib.cmake#L200-L206


LED Drivers for dummies

Hi,
I took the example 2 (for Arduino) and placed it on my esp32. I changed the number of leds and the gpioNum.
The result is a Guru Meditation Error and 24 red leds flickering.
Is there some more guidance on how to use this lib properly?
Thanks!

I found this lib that looks a bit easier to me. https://github.com/RoboticsBrno/SmartLeds/tree/master/examples/basicExample
However, it throws the same error, so must be something else.

Are you powering the LEDs with a separate power supply? The red flicker suggests the LEDs don't have sufficient power: the typical ESP32 breakout board won't have anywhere near enough power on its own to drive 24 LEDs at anything but the lowest brightness levels, if that.
If power is too low it may in turn cause a brown-out on the device, leading to errors (not sure if that's what the error is about though).

Thanks! It's maybe is not, since i've a separate power supply and the colors are more or less fine with the FastLed lib until i use WiFi, which then indicates it is a power supply problem )-;
Also it might be the hardware or the logic converter.

Finally it was: bad soldering on the second ground pin of the converter, plus calling a non existing pixel, plus certain colors needing too much power (and others are fine). Setting the brightness to 25 "solved" this. I'm so glad this is just a hobby!

Sounds painful but I'm glad you found the root cause!


Game crashes at rhythm minigame

What bug did you encounter?
The game crashes whenever I try to play the rhythm minigame.
Steps to Reproduce
As soon as I click on any one of the songs in the rhythm mini game, it immediately displays the attached screenshot. I can safely rollback, but I can't do anything in the minigame.
Screenshots

Additional Information
Here's the full error log:
I'm sorry, but an uncaught exception occurred.

While running game code:
  File "game/script.rpy", line 1229, in script call
    call day_activity_choices from _call_day_activity_choices_6
  File "game/scripts/labels/day_activity_choices.rpy", line 15, in script call
    call day_activity_relax from _call_day_activity_relax
  File "game/scripts/labels/day_activity_choices.rpy", line 167, in script call
    call day_activity_video_game from _call_day_activity_video_game
  File "game/scripts/labels/day_activity_choices.rpy", line 287, in script call
    call rhythm_game_entry_label from _call_rhythm_game_entry_label
  File "game/scripts/rhythm_minigame.rpy", line 573, in script
    call screen rhythm_game(rhythm_game_displayable)
  File "renpy/common/000statements.rpy", line 569, in execute_call_screen
    store._return = renpy.call_screen(name, *args, **kwargs)
  File "game/scripts/rhythm_minigame.rpy", line 67, in execute
    screen rhythm_game(rhythm_game_displayable):
  File "game/scripts/rhythm_minigame.rpy", line 67, in execute
    screen rhythm_game(rhythm_game_displayable):
  File "game/scripts/rhythm_minigame.rpy", line 81, in execute
    vbox:
  File "game/scripts/rhythm_minigame.rpy", line 101, in execute
    showif rhythm_game_displayable.has_game_started:
  File "game/scripts/rhythm_minigame.rpy", line 103, in execute
    text _('Score: [score+str]'):
NameError: Name 'score+str' is not defined.

-- Full Traceback ------------------------------------------------------------

Full traceback:
  File "game/script.rpy", line 1229, in script call
    call day_activity_choices from _call_day_activity_choices_6
  File "game/scripts/labels/day_activity_choices.rpy", line 15, in script call
    call day_activity_relax from _call_day_activity_relax
  File "game/scripts/labels/day_activity_choices.rpy", line 167, in script call
    call day_activity_video_game from _call_day_activity_video_game
  File "game/scripts/labels/day_activity_choices.rpy", line 287, in script call
    call rhythm_game_entry_label from _call_rhythm_game_entry_label
  File "game/scripts/rhythm_minigame.rpy", line 573, in script
    call screen rhythm_game(rhythm_game_displayable)
  File "renpy/ast.py", line 2010, in execute
    self.call("execute")
  File "renpy/ast.py", line 1998, in call
    return renpy.statements.call(method, parsed, *args, **kwargs)
  File "renpy/statements.py", line 278, in call
    return method(parsed, *args, **kwargs)
  File "renpy/common/000statements.rpy", line 569, in execute_call_screen
    store._return = renpy.call_screen(name, *args, **kwargs)
  File "renpy/exports.py", line 3132, in call_screen
    rv = renpy.ui.interact(mouse="screen", type="screen", roll_forward=roll_forward)
  File "renpy/ui.py", line 298, in interact
    rv = renpy.game.interface.interact(roll_forward=roll_forward, **kwargs)
  File "renpy/display/core.py", line 3276, in interact
    repeat, rv = self.interact_core(preloads=preloads, trans_pause=trans_pause, pause=pause, pause_start=pause_start, **kwargs)
  File "renpy/display/core.py", line 3694, in interact_core
    root_widget.visit_all(lambda i : i.per_interact())
  File "renpy/display/core.py", line 568, in visit_all
    d.visit_all(callback, seen)
  File "renpy/display/core.py", line 568, in visit_all
    d.visit_all(callback, seen)
  File "renpy/display/core.py", line 568, in visit_all
    d.visit_all(callback, seen)
  File "renpy/display/screen.py", line 432, in visit_all
    callback(self)
  File "renpy/display/core.py", line 3694, in <lambda>
    root_widget.visit_all(lambda i : i.per_interact())
  File "renpy/display/screen.py", line 443, in per_interact
    self.update()
  File "renpy/display/screen.py", line 631, in update
    self.screen.function(**self.scope)
  File "game/scripts/rhythm_minigame.rpy", line 67, in execute
    screen rhythm_game(rhythm_game_displayable):
  File "game/scripts/rhythm_minigame.rpy", line 67, in execute
    screen rhythm_game(rhythm_game_displayable):
  File "game/scripts/rhythm_minigame.rpy", line 81, in execute
    vbox:
  File "game/scripts/rhythm_minigame.rpy", line 101, in execute
    showif rhythm_game_displayable.has_game_started:
  File "game/scripts/rhythm_minigame.rpy", line 103, in execute
    text _('Score: [score+str]'):
  File "renpy/text/text.py", line 1551, in __init__
    self.set_text(text, scope, substitute)
  File "renpy/text/text.py", line 1670, in set_text
    i, did_sub = renpy.substitutions.substitute(i, scope, substitute)
  File "renpy/substitutions.py", line 270, in substitute
    s = formatter.vformat(s, (), kwargs)
  File "/home/tom/ab/renpy-build/tmp/install.linux-x86_64/lib/python2.7/string.py", line 563, in vformat
  File "/home/tom/ab/renpy-build/tmp/install.linux-x86_64/lib/python2.7/string.py", line 585, in _vformat
  File "/home/tom/ab/renpy-build/tmp/install.linux-x86_64/lib/python2.7/string.py", line 646, in get_field
  File "/home/tom/ab/renpy-build/tmp/install.linux-x86_64/lib/python2.7/string.py", line 605, in get_value
  File "renpy/substitutions.py", line 228, in __getitem__
    raise NameError("Name '{}' is not defined.".format(key))
NameError: Name 'score+str' is not defined.

Windows-10-10.0.19041
Ren'Py <IP_ADDRESS>5
Learn to Code RPG 1.0
Wed Jan 19 12:17:26 2022

Hi @DuxVictorum Would you mind trying the latest build on itch? I think this has been fixed in 266da11ba1849cd31f0f22f3a866dbfe44fb1b88

Yup, I reinstalled and everything works well now!
Mark
On Mon, Jan 24, 2022, 10:17 PM Lynn Zheng @.***> wrote:

Closed #62 https://github.com/freeCodeCamp/LearnToCodeRPG/issues/62.
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/LearnToCodeRPG/issues/62#event-5947037677,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARLRI5FCX7NF5XVMZ4LKP7DUXYP5NANCNFSM5MKVHRFQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Is PlinianCore ratified by TDWG?

Hello, I am trying to figure out if PlinianCore is now officially approved by the TWDG executive committee. I can see from here: https://github.com/tdwg/PlinianCore#latest-developments-and-current-status-2022 that it was about to be submitted in 2021. Where can I find out more?
Thanks!

Apologies for the delay in responding. The required documentation was submitted to the TDWG Secretary last month (January 2024). We are currently waiting for the Executive Committee's decision (typically, it takes about 30 days). Following the path towards ratification of a TDWG standard, if the response obtained is positive, the Executive Committee will appoint a Review Manager, and the process will move to the Public Review phase (more information about the TDWG process at https://www.tdwg.org/about/process/).


Seems like a bug: Create a new datastore in put_node

This line seems like a bug: It attempts to create a new datastore in the put_node method
https://github.com/UKP-SQuARE/square-core/blob/3f5d675b5c70aeebf7f3855e73c3bff00d837dee/datastore-api/app/routers/kgs.py#L227

@SevenDaysDA

Please keep these two functions separate ("adding nodes" vs "adding datastores/kgs") and just return an error if the datastore does not exist

It seems that put_nodedoes not have anything to do with adding nodes


logging: Fix initial log level in shell

Shell log backends were initialized from shell context. After
lowering logger thread priority order has been flipped. That
revealed a bug where shell logger backend was enabled before
backend ID's has been assigned during logger initialization.
ID assignment is moved to log backend enabling function making
it independent of order of initialization.
Fixes #11866.
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

fyi @rlubos @jfischer-phytec-iot @jukkar Fix for logs disabled by default when shell was used.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@258fd2d). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master   #11870   +/-   ##
=========================================
  Coverage          ?    51.3%
=========================================
  Files             ?      231
  Lines             ?    28738
  Branches          ?     7154
=========================================
  Hits              ?    14744
  Misses            ?    11182
  Partials          ?     2812

Impacted Files
Coverage Δ

subsys/logging/log_core.c
74.67% <100%> (ø)

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 258fd2d...b88538e. Read the comment docs.


suprising behavior of cupy.array on numpy views with base array of a different dtype

The following example illustrates that creating CuPy array from a NumPy view where the underlying array has a different base dtype can result in unexpected behavior. Here is a minimal concrete example for a boolean view of a uint8 array. In NumPy calling np.array() creates a copy. In CuPy, the created array has flags.owndata=True as expected, so it looks like it is a copy, but the underlying data still correspond to the view! This can be seen if converting back from bool->uint8 via astype, values > 1 occur!

Conditions (you can just paste the output of python -c 'import cupy; cupy.show_config()')

CuPy 9.0.0a2

Code to reproduce

import numpy as np
import cupy as cp

# boolean view of a uint8 array
a = np.array([0, 3, 1, 2, 0, 1], dtype=np.uint8)
a_view = a.view(np.bool)
print(f"a_view.flags = {a_view.flags}")

# calling np.array makes a copy as expected
b = np.array(a_view)  # will make a copy!
print(f"b.flags = {b.flags}")
print(f"b.astype(int) = {b.astype(int)}\n")  # only 1 and 0 as expected

# create CuPy array from the NumPy view
gpu_a = cp.array(a_view)
print(f"gpu_a = {gpu_a}")
print(f"gpu_a.astype(int) = {gpu_a.astype(int)}")   # unexpected values of 2 and 3!
print(f"gpu_a.flags = {gpu_a.flags}")  # flags look like it is a copy (owndata=True)

gives
a_view.flags =   C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

b.flags =   C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

b.astype(int) = [0 1 1 1 0 1]

gpu_a = [False  True  True  True False  True]
gpu_a.astype(int) = [0 3 1 2 0 1]
gpu_a.flags =   C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : True

Possible Solution
Possibly need a check on the input object in cp.array that looks something like the following
if isinstance(obj, numpy.ndarray) and not obj.flags.owndata and obj.base.dtype != obj.dtype:
    obj = numpy.copy(obj)

or if a copy is not requested then perhaps creating the cupy.ndarray from obj.base and then create a view?

Thanks for reporting! (and sorry for late response)
It looks the copied b has the same binary with a in NumPy,
>>> b = np.array(a_view)
>>> b.view(np.uint8)
array([0, 3, 1, 2, 0, 1], dtype=uint8)

so cupy.ndarray.astype may have some miss compatibility. Would you also check the result above?

Thanks for reporting! (and sorry for late response)
It looks the copied b has the same binary with a in NumPy,
>>> b = np.array(a_view)
>>> b.view(np.uint8)
array([0, 3, 1, 2, 0, 1], dtype=uint8)

so cupy.ndarray.astype may have some miss compatibility. Would you also check the result above?

cupy.ndarray.astype does elementwise_copy to copy underlying data:
https://github.com/cupy/cupy/blob/cff7da1130587d814b744293e81ca64c22081839/cupy/_core/core.pyx#L466
On the other hand, numpy.ndarray.astype seems to do a more complicated thing than I thought:
https://github.com/numpy/numpy/blob/a11945d7dc930afe47e1cf7f81cb69f2958f4c1f/numpy/core/src/multiarray/array_assign_array.c#L117-L125
/* Get the function to do the casting */
NPY_cast_info cast_info;
if (PyArray_GetDTypeTransferFunction(aligned,
                    src_strides_it[0], dst_strides_it[0],
                    src_dtype, dst_dtype,
                    0,
                    &cast_info, &needs_api) != NPY_SUCCEED) {
    return -1;
}

I think we can make a special path for boolean in CuPy, and it's would be enough here.

Now CuPy is fixed, but NumPy reads bytes for float16 (numpy/numpy#19514).
>>> gpu_a.astype(int)
array([0, 1, 1, 1, 0, 1])
>>> a_view.astype(int)
array([0, 1, 1, 1, 0, 1])
>>> gpu_a.astype('float16')
array([0., 1., 1., 1., 0., 1.], dtype=float16)
>>> a_view.astype('float16')
array([0., 3., 1., 2., 0., 1.], dtype=float16)