weirdness with orderBy

Latest next
I don't exactly expect this code to pass, but I also don't expect it to fail this strangely...
Test code:
https://github.com/lookfirst/mikro-example/blob/master/test/db/rig.spec.ts#L27
(note it is xtest for now)
  console.log node_modules/mikro-orm/dist/utils/Logger.js:19
    [query] begin

  console.log node_modules/mikro-orm/dist/utils/Logger.js:19
    [query] insert into `RigType` (`id`, `name`) values (1, 'rigType 1') [took 1 ms]

  console.log node_modules/mikro-orm/dist/utils/Logger.js:19
    [query] insert into `Rig` (`id`, `type`) values (1, 1) [took 1 ms]

  console.log node_modules/mikro-orm/dist/utils/Logger.js:19
    [query] commit

  console.log node_modules/mikro-orm/dist/utils/Logger.js:19
    [query] select `e0`.* from `Rig` as `e0` order by `e0`.`type` asc [took 1 ms]

Error: select `e0`.* from `RigType` as `e0` where `e0`.`id` in (1) group by `e0`.`id` order by `e0`.`0` asc, `e0`.`1` asc, `e0`.`2` asc - SQLITE_ERROR: no such column: e0.0
Error: SQLITE_ERROR: no such column: e0.0

We could sort by PK in this case (that is how it works for FilterQuery), or do you think we should add validation instead?

I think just by PK is fine since it is effectively unspecified.

The issue here is that these queries are effectively generic and dynamically built. I'm doing pagination on a table and I want to be able to click the column heading to sort the table by columns. I can just pass the list of sorts [{columnName: string, direction: string}...] to the backend without even needing to look at them.

Ok I misunderstood it for the first time, I thought the table is joined in the second query so I suggested to order by its PK, but as it is not, I will just ignore the order by in that case.
Fixed in 3.0.0-alpha.36

Confirmed


DEV-1066 fix password decoding bug

Coverage remained the same at 82.597% when pulling 1e2a01f52e2358416c1ceb6d57fd962b5f93dedd on DEV-1066 into c29a3ec2d9fe2f6c4df3b6e3e33b55f1f8782205 on master.


thread 'main' panicked at src/prover.rs:154:66:

Waiting for another program to prove...
thread 'main' panicked at src/prover.rs:154:66:
called Result::unwrap() on an Err value: Protocol(ResetWithoutClosingHandshake)
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

@rigaz29 we made some updates to error handling, are you still seeing this issue if you update? (i.e. run the curl command again)


[My Friends Page] Placeholder "Search" in the search field instead of "Search a friend"

Environment: Windows 10, Chrome 116.0.5845.97 (64 bit)
Reproducible: always
Build found: 24/08/23
Preconditions
The rigestered user is loged in
Steps to reproduce

Go to GreenCity
Click on 'My Friends Page'
Open 'All Friends' Tab

Actual result
Placeholder "Search" in the search field instead of "Search a friend".
Expected result
Placeholder 'Search a friend' in a search field.
User story and test case links
User story #2122
Labels to be added
"Bug", Priority (pri: "Low"), Severity (sev: "Minor"), Type (type: "UI")

The search field has an incorrect name.  Placeholder "Search" in the search field instead of "Search a friend".

https://github.com/ita-social-projects/GreenCity/issues/6423
https://github.com/ita-social-projects/GreenCity/issues/6465
There are other requirements in the change request


Time difference of 5 minutes

I have been noticing for quite some time, that I have a time deviation / difference between the actual time when my pets leave or enter the house through the pet door and the time, that the API returns. The time in the official SurePet App is correct, while the API returns a time stamp which is about 4-5 minutes earlier.
I already checked the time inside the docker container with docker exec -it surehub-api date and it returns the correct current time.
Anybody have an idea? Hints are much appreciated! :)

A few minutes earlier seems very strange. I will have a look and try to reproduce this by myself.

It is indeed very strange. So when the cat comes in at 2:00 pm for example, the API shows 1:55 pm.
Interestingly, when changing the cat location manually via the SurePet app, the time is correct. So the time only seems to be off, when it is changed by the cat using the door.

It is indeed very strange. So when the cat comes in at 2:00 pm for example, the API shows 1:55 pm.
Interestingly, when changing the cat location manually via the SurePet app, the time is correct. So the time only seems to be off, when it is changed by the cat using the door.

Unfortunatly I was not able to reproduce the issue. I checked the time when my cats went through the cat flap. The time returned by the API is accurate to the second. Since I query the official SureHub API directly, there is nothing I can do about the inaccuracy you described.
If it's okay with you, I would close the issue.


Page title is not meaningful - Patient Experience

Area or Component
Patient Self-Registration and Patient Results
Quick Summary
The title of the page is just "SimpleReport", which is a shared title for all pages in the patient experience.
Accessibility Violation
Page  does not identify purpose of page
https://axe.deque.com/rule-help/en/title-not-meaningful
Action Requested
Replace titles in the pages below with the following:

Terms of service: Terms of service | SimpleReport
Self-registration: Test registration form | SimpleReport
Self-registration complete: Test registration complete | SimpleReport
DOB input: Test result verification | SimpleReport
Patient results: Test result | SimpleReport

Spanish:

Terms of service: Condiciones del servicio | SimpleReport
Self-registration: Formulario de inscripción a la prueba | SimpleReport
Self-registration complete: Inscripción a la prueba completa | SimpleReport
DOB input: Verificación del resultado de la prueba | SimpleReport
Patient results: Resultado de la prueba | SimpleReport

Pages:

The Terms of Service page (shared between self-registration and results)

The patient self-registration form page

The patient self-registration completion page

The patient results DOB input

The patient results page

Source Code (optional)
Potentially the PatientApp.tsx?
Original Deque Issue

title-not-meaningful
The  element on this page does not identify the contents or purpose of the page.
Title is SimpleReport; should be SimpleReport | Self-Registration or similar
Page  does not identify purpose of page
https://axe.deque.com/rule-help/en/title-not-meaningful
serious
Yes
No
Structure
["title"]
The  element on this page does not identify the contents or purpose of the page.
SimpleReport
wcag2a,wcag242a
<EMAIL_ADDRESS>Patient Self Reg - Terms of Service
https://stg.simplereport.gov/app/register/9C7BB

2022-07-13T21:08:07.783Z

Based on existing title pattern, recommend the following titles:

Terms of service: Terms of service | SimpleReport
Self-registration: Test registration form | SimpleReport
Self-registration complete: Test registration complete | SimpleReport
DOB input: Test result verification | SimpleReport
Patient results: Test result | SimpleReport

@zdeveloper Another patient experience ticket.

Spanish translation needed

Sent to translation team through Dana Pitts and Holly Wilson this morning (10/13)

@lg-king @zdeveloper Spanish translations added to description!

@lauraykerr  Awesome! thank you!


Differences with CDN

Discussed in https://github.com/Jinntec/Fore/discussions/142

Originally posted by timathom June 23, 2022
Certain elements, such as fx-inspector and fx-dialog seem not to work when loading from CDN.

outdated - should work now


Added --fix-org-members

Added --fix-org-members option

Hi @sabre1041. Thanks for your PR.
I'm waiting for a operate-first member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign harshad16 after the PR has been reviewed.
You can assign the PR to them by writing /assign @harshad16 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tumido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [tumido]

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

/ok-to-test

/override Semantic PR

@tumido: Overrode contexts on behalf of tumido: Semantic PR

In response to this:

/override Semantic PR

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Readme: Added readme-en.md english translation

Context
For better reachability, I have added the English translation of all readme.md files in this repository.

@hhstore Can you please check this PR ? 🙏

Thank you so much for your translation work on the documentation. I took a look at it and it's fine.
I have already merged.
In addition, if there are improvements to the project code, you are also welcome to submit some PRs. I will merge them.
I've been busy with other projects recently, and haven't had time to update this template project.

I have changed the readme to English mainly.
I am adding contributors list section.
Thanks again for your work. 😄


allow non-looped cleanups

hey there!
Is it possible to run this script "one-shot" ?
I'd like to run it, and once finished, exit.

At the moment no, but a PR that adds a default "LOOP=true" flag which you can then set to "LOOP=false" would be most welcome!

awesome! thanks.

@docteurklein 1.4.1 has been released on docker hub

thanks,  that was super fast :)
already using it, seems to work great


Updates backgroundEffect code on README

@sgr-ksmt
Previously README had pullToDismiss?.background instead of pullToDismiss?.backgroundEffect which was wrong and confused me and I figured it out by looking through the source code for PullToDismiss.swift. Thanks again for this repo! 👍
Issue

Issue Number: #39

Updates the README.md so that it has the correct instructions for changing PullToDismiss's backgroundEffect.

@mding5692
I'm late...
Thanks for submitting PR!! I'll merge it! 🎉


Problem with search bar

Hi Folks,
today I decided to search on site (https://community.chocolatey.org/packages) by term "mouse", but happen timeout (status 524).
I know, this very crazy, but it's true.
on search

after search

I'm assuming this was a one off and we can close it?

I'm going to go ahead and close this issue, but we can reopen it if necessary.


busywork: Remove all non-determinism + New Stress targets

Description

This change set removes all* vesitiges of "non-determinism" from the
busywork/counters chaincode. When @jyellick fixed a non-determinism bug here
a few days ago, I knew there were still other instances - but thought they
would be OK. I now realize that there are cases where peers never execute even
a single transaction, even when they are not under stress. So I'm throwing in
the towel, and I have effectively rewritten this chaincode, updated the
documentation and added a test program for the new verison. To be completely
honest, the data model manipulated by this chaincode is so simple that my
"non-determinism" for error checking never caught a single bug.
The PBFT Batch protocol is working well-enough now that the busywork
stress2b test can run in its originally intended form. I've restructured the
Makefile to show this, and I've also added a new sweep1b test that also
works. The bad news is that the more complex sweep2b test fails (see #2077).
I've also included a couple of minor bug fixes and enhancements to the
busywork/counters/driver that give the PBFT Batch runs every opportunity to
succeed.
*Depends on what's considered non-deterministic, but the chaincode no longer maintains an independent idea of what the state should be.
Motivation and Context

Completely Fixes #1857
How Has This Been Tested?

I added a new test for the new counters chaincode. I've also run the new make targets interactively and verified that they work.
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Bishop Brock<EMAIL_ADDRESS>

I can confirm that stress2b completes on my laptop in Vagrant.  The sweep1b and sweep2b tests seem to have problems, I believe one of these may be #2077, and have no reason to believe it's a problem in this changeset.
Looks good to me.

Please do not merge yet. It looks like I may need to tweak some parameters to allow some of the tests to work on a laptop under Vagrant.

OK, I would like to reinstate the request that these changes be merged. For now we have to accept the fact that some of the busywork tests don't work on laptops. Perhaps the limitation of having only 1 or 2 cores to run 4 peers + 4 chaincode containers + N client processes stresses the system to the point that dropping transactions is currently unavoidable (as Jason explained to me privately).

LGTM, thanks


ValueError when batch size is 1.

Thank you for publishing this implementation of TSMixer - particularly useful to have TSMixerExt included!
I'm seeing errors when I try to call the model with a batch size of 1, and am wondering if I'm doing something wrong or whether it is a bug? For example:
from torchtsmixer import TSMixer
import torch

model = TSMixer(
    sequence_length=10,
    prediction_length=5,
    input_channels=2,
    output_channels=1
)

# Call with batch size of 2 (this works)
x_batch = torch.rand(2, 10, 2)
print(model(x_batch).shape)
# torch.Size([2, 5, 1])

# Call with batch size of 1 (this fails)
x_batch = torch.rand(1, 10, 2)
print(model(x_batch).shape)
# ValueError: Expected more than 1 value per channel when training, got input size torch.Size([1, 20, 1])

Actually, realised it is my mistake. The error is caused by the batch normalisation layers, so calling model.eval() to turn them off before passing in the batch size of 1 resolves it.


multiple gulp concat - not supported ?

Hi,
using
gulp.src("file.js")
.pipe(concat("file2.js"))
.pipe(concat("file3.js"))
.pipe(gulp.dest("file4.js"));
will not put all files into dest.
Is this expected ?
If true, why does it makes a difference when operating on streams ?
I know using  gulp.src(["file.js", "file2.js", "file3.js", "file4.js"]) will fix it.

What are you expecting to happen here?

Behaviour of gulp-concat is correct. What your code actually does is renaming file.js to file4.js:
gulp.src("file.js")             // 1
.pipe(concat("file2.js"))       // 2
.pipe(concat("file3.js"))       // 3
.pipe(gulp.dest("file4.js"));   // 4

Put file.js only into the stream
Concat all vinyl files into a single file called file2.js - the only file in the stream is file.js so your actually just renaming it to file2.js
Concat all vinyl files into a single file called file3.js - same as in step 2
Write the vinyl files from stream to a file called file4.js

I know using gulp.src(["file.js", "file2.js", "file3.js", "file4.js"]) will fix it.

This will not just fix it, it is the correct way to use streams and gulp-concat.


Update workflow MAPIv2 samples

Motivation
Based on CTC-1617, I have added code samples for both .NET and JS SDKs that were missing until now. I have also slightly updated the cURL sample to demonstrate various configurations of the updated workflow object (the same change is now represented in all .NET, JS, and cURL). Last but not least, since we effectively deprecated method listWorkflowSteps with the latest release of workflows in MAPIv2, I took my liberty and changed the JS's GetWorkflowSteps sample to use listWorkflows in the aforementioned method's stead.
Context
Add the following information:

When do you need the code to go public? Once the https://github.com/Kentico/kontent-management-sdk-net/pull/173 get published
Is the pull request complete and ready for review? Yes

I changed the codename reference in the code samples to use lowercase string instead of Article, which is not valid for codenames. Other than that, it looks fine. 👍🏻


Refactor command class hierarchy to better support engine independent commands

When implementing an engine independent command (one that performs an abstract operation on all rows), you have to copy code that's already in EngineDependentCommand::doPost. Validating the CSRF token, getting the project, creating a process, and handling the response are necessary when writing a command that performs abstract operations on all rows, too.
It also seems counter intuitive to implement engine independent commands by extending Command directly because Command has two methods for working with engine configuration:,getEngineConfig() and getEngine().
Proposed solution
From discussion in #1855 and at the suggestion of @tfmorris, I'm wondering what folks think of refactoring the Command class hierarchy to facilitate the development of new commands and refactor existing engine independent commands for maintainability.
In my mind, I think it'd be nice to have three abstract classes:

Command

EngineDependentCommand
EngineIndependentCommand

Alternatives considered

Continue extending Command: It's not hugely annoying to extend Command and override doPost when writing an engine independent command because those useful methods are hoisted in the super class. But it still feels redundant.

Extend EngineDependentOperation and ignore engineConfig when overriding EngineDependentCommand::createOperation: This is actually what ReorderColumnsCommand does.

Additional context
As a newbie to OpenRefine's codebase, my initial instinct was just to extend the EngineDependentCommand when tinkering around with a command for adding rows to a project.
Among the core commands, these engine independent commands (in particular) basically copy EngineDependentCommand::doPost or extend EngineDependentCommand and ignore engine configuration:

RenameColumnCommand
RemoveColumnCommand
ReorderColumnsCommand
JoinMultiValueCellsCommand
TransposeColumnsIntoRowsCommand

There are a number of engine independent commands which aren't project related or which don't create Operations, which is really the main benefit to using EngineDependentCommand, so I've put up a draft PR which restructures the hierarchy to move most EngineDependentCommand functionality into a new OperationCommand.
@steve-kasica Please have a look and see if something along those lines would be helpful for you

@wetneb @Abbe98 Any opinions on this request?
@steve-kasica Does the linked PR address your concerns?

Yep, I think that addresses my concerns, thanks.


added multiPart Upload and signedUrl Implementation.

@chethan-zededa s3sync file is dealing with AWS APIs. We only create the required objects in the s3api file so please move your code to s3sync.

@chethan-zededa s3sync file is dealing with AWS APIs. We only create the required objects in the s3api file so please move your code to s3sync.

changes done accordingly. please check.

@eriknordmark awaiting for your inputs.
Thanks.


Script for Zeta+ACA test

Script for Zeta+ACA test

@PikaPikaW great start towards #163 , it answered some of the highlevel technical questions for the setup. Adding @zzxgzgz @liangbin-pub to review.

Great job.
FYI, you can refer to this page for what data to POST to Zeta.

hello,everyone.Now I want to add a function in gtest that parses the JSON file from the controller, but I'm not sure which CPP file I should add it in, in aca_test_ovs_util.cpp? or something else; And should I define a parser function with TEST or a normal function, or should I define a TEST function which call the parser function, because when RUN_ALL_TESTS() runs, I think it will start executing all the functions with TEST, but I don't know in what order?

hello,everyone.Now I want to add a function in gtest that parses the JSON file from the controller, but I'm not sure which CPP file I should add it in, in aca_test_ovs_util.cpp? or something else; And should I define a parser function with TEST or a normal function, or should I define a TEST function which call the parser function, because when RUN_ALL_TESTS() runs, I think it will start executing all the functions with TEST, but I don't know in what order?

Hi, you will define all the zeta test under /test/gtest/aca_test_zeta_programming.cpp, you can do the following:
add new utility function: void aca_test_zeta_setup - parse JSON file from controller, setup the goal state and push to aca (Aca_Comm_Manager::get_instance().update_goal_state) - see this example: aca_test_1_neighbor_CREATE_DELETE
add new test case: TEST(zeta_programming_test_cases, gateway_path)
add new test case: TEST(zeta_programming_test_cases, direct_path)
and the above two test case will call aca_test_zeta_setup in the begining then execute the test

What do the fields IP and VIP in data mean when adding port?
Request: http://localhost:8080/ports
    data:
    [
      {
        "port_id": "333d4fae-7dec-11d0-a765-00a0c9342222",
        "vpc_id": "3dda2801-d675-4688-a63f-dcda8d327f50",
        "ips_port":
        [
          {
            "ip": "<IP_ADDRESS>",
            "vip": ""
          },
          {
            "ip": "<IP_ADDRESS>",
            "vip": "<IP_ADDRESS>"
          }
        ],
        "mac_port": "cc:dd:ee:ff:11:22",
        "ip_node": "<IP_ADDRESS>",
        "mac_node": "ee:dd:ee:ff:22:11",
      },
      {
        "port_id": "99976feae-7dec-11d0-a765-00a0c9341111",
        "vpc_id": "3dda2801-d675-4688-a63f-dcda8d327f55",
        "ips_port":
        [
          {
            "ip": "<IP_ADDRESS>",
            "vip": ""
          },
          {
            "ip": "<IP_ADDRESS>",
            "vip": ""
          }
        ],
        "mac_port": "6c:dd:ee:ff:11:32",
        "ip_node": "<IP_ADDRESS>",
        "mac_node": "ee:dd:ee:ff:33:11",
      }
    ]

]

just use this:
    "ips_port":
    [
      {
        "ip": "<IP_ADDRESS>",
        "vip": ""
      }
    ],

check out below to see if there is other json parser which is easier to use.
https://stackoverflow.com/questions/36471581/c-c-json-parser

I also made a fork and tried to update the json library && fix compile errors in aca_test_zeta_programming.cpp, you can take a look and pick it up from there:
https://github.com/zzxgzgz/alcor-control-agent/tree/integrationTest

Ok, thanks for your help very much, so use 'nlohmann/Json.hpp' to replace 'jsoncpp' to parse json file, commit will not fail,or i shall add nlohmann's source code in aca?

What do the fields gws in responsed data mean when adding vpc? need to put in configuration file?
data:
    {
        "vpc_id": "3dda2801-d675-4688-a63f-dcda8d327f50",
        "vni": "12345",
        "zgc_id": "f81d4fae-7dec-11d0-a765-00a0c91e6bf6",
        "name": "ZGC_test1",
        "gws":
        [
          {
            "ip": "<IP_ADDRESS>",
            "mac": "37.02.ff.cc.65.87"
          },
          {
            "ip": "<IP_ADDRESS>",
            "mac": "37.02.ff.cc.65.88"
          },
          {
            "ip": "<IP_ADDRESS>",
            "mac": "37.02.ff.cc.65.89"
          }
        ],
        "port_ibo": "8300"
    }

gws stands for gateways, those are the list of ZGC gateway IP/MAC pairs to populate OVS group/bucket

Ok, thanks for your help very much, so use 'nlohmann/Json.hpp' to replace 'jsoncpp' to parse json file, commit will not fail,or i shall add nlohmann's source code in aca?

The nlohmann/Json.hpp is already in /include/nlohmann/Json.hpp in my branch integrationTest branch. You can pull from my repo if you want to.

What do the fields gws in responsed data mean when adding vpc? need to put in configuration file?

The fields that's needed to construct goal state to ACA are below, they should be put in configuration file:
{
    "zgc_id": "f81d4fae-7dec-11d0-a765-00a0c91e6bf6",
    "gws":
    [
      {
        "ip": "<IP_ADDRESS>",
        "mac": "37.02.ff.cc.65.87"
      },
      {
        "ip": "<IP_ADDRESS>",
        "mac": "37.02.ff.cc.65.88"
      },
      {
        "ip": "<IP_ADDRESS>",
        "mac": "37.02.ff.cc.65.89"
      }
    ],
    "port_ibo": "8300"
}

hello,i am a little busy recently, very sorry. i have carefully read the changed code. So, when talking to zeta,the order is creating a ZGC firstly, getting a zgc_id returned, then creating the ZGC Nodes information in this zgc_id, then creating VPC, finally adding port information, is that right?
I have some questions about several IP entries in the Zeta data:

When creating ZGC, ip_start, IP_end refers to the IP address area that will be assigned to ZGC Node in ZGC ? I installed Zeta on machine 247 and 248 here, so the address area should be written as 172..16.62.247-248. Is that right ?
When creating ZGC Node, is IP_contorl ZGC Node's ip address? I only have two machines install Zeta, so I should create two nodes ?
When creating VPC, the gws returned is the ZGC Gateway's information (Is ZGC Node and ZGC Gateway a concept?).Are these IP addresses corresponding to the IP address area given when creating ZGC? and this information should be placed in the configuration file ?

Hi Wanchen,
1.no, the ip range given to ZGC is for ZGC gateway entrances on tenant network(vxlan), not the one for management (such as you ssh into it), they are of different network
2. No, if you have two machine to run Zeta, you will have one for Zeta management (serve the RESTful APIs), and the other as ZGC "node", which serve as gateway for tenant network traffic
3. yes, the gws list is same as the range given at ZGC create, with additional info with MAC

Hi @PikaPikaW,
I made some changes to the aca test code, and made some minor modifications to the pseudo controller. Please take a look and consider pulling my changes.
Thanks

Okay, I think i get the idea, so:

There will be one node in each ZGC for communication with management (restful apis), and other nodes as the gateway
ZGC Node that communicates with Management will have multiple IPs, IP that communicates with Manangement (IP used when ssh into it), and IP that communicates with ACA (IP in Tenant Work)

If I understand correctly, then:

The node in ZGC that is responsible for communicating with Management gets the information that needs to be added, how other nodes get information, whether they communicate with this node or also with Management (in cases script not used)?
Ip_control is the IP that ZGC Node communicates with Management?

@er1cthe0ne, okey

okay, @zzxgzgz

i think the IP_node in the responed port information should be the ip of the vm host. Why is this ip different from the Ip information in ACA_nodes

In addition, in your experimental environment, how many Zeta gateway nodes and ACA compute nodes respectively?

i think the IP_node in the responed port information should be the ip of the vm host. Why is this ip different from the Ip information in ACA_nodes
In addition, in your experimental environment, how many Zeta gateway nodes and ACA compute nodes respectively?

@PikaPikaW

Yes and no. The ip_node should be one of the IP of the interface you use for tenant network of that host. Also, are you guys using VMs as ACA_nodes? We're using physical hosts for ACA nodes, and those nodes don't run any VMs, so I'm not sure if I'm understanding the the ip of the vm host correctly.
In our experimental environment, we have one physical node to deploy zeta and the pseudo controller, one physical node to run 6 VMs as zeta gateway nodes, one physical node to be the master of the zeta gateway cluster, and two physical nodes as ACA compute nodes.

Well, thanks for your explanation very much.so pseudo controller communicates with Zeta Control plane, adding information(step 1-4).Then

does ZGC Nodes need to save the added information? If need, should the master of the Zeta Gateway cluster send the information, or pseudo Contoller (in addition, the master of the Zeta Gateway cluster and ZGC nodes communicate in the ZGC_network?)
does ZGC nodes need to install Zeta?
Does the pseudo Controller's physical machine also need to install Zeta (I don't know what Zeta repo for Zeta Deployment means)?
so,in node's information, ip_control is the ip of ZGC Node in tenant_network, which also corresponds to the ip area in zgc's information. However, in the figure, it seems that the ip of each ZGC Node's ens4 is not in the ip area (ip_start :<IP_ADDRESS>; Ip_end: <IP_ADDRESS>)
In our experimental environment, the current environment is the same as the previous installation, as shown in the figure below, which also uses two physical machines as ACA Nodes and does not run any vm.ZGC also uses two physical machines with Zeta installed and does not use any vm. If one is the master of the Zeta Gateway cluster, then there is only one ZGC Node

hello,when i am running the script,met some problem:

After zeta is installed, there will be a default zgc information, I want to delete this zgc, but using API can not delete,i think before delete zgc,we should clean its node information,However, there is no id information of node when using get zgc.
When adding vpc, the message sent does not include zgc_id, so which zgc ip area does the returned gws correspond to?

because I cannot delete the default ZGC, so after adding VPC, the returned gws corresponds to the IP area of the zgc that existed by default at the beginning(not the new zgc i add) I don't know how to solve this problem ?

@PikaPikaW I just read your latest commit, and I'd like to suggest you not to put your username/password into the json file, as now you exposed their login info and it is unsafe. Please find a way to fix this.

Hello, may I ask, what dependencies should I delete before I delete port. Because I want to delete vpc, I think delete vpc information needs to delete the port related to it firstly, and the port deletion always fails, prompt 500. In the information returned by GET ports, what do eps and host_id stand for? I understand from the api source code that these two fields also have GET operation, but there is no DELETE and POST operation.

Hello, may I ask, what dependencies should I delete before I delete port. Because I want to delete vpc, I think delete vpc information needs to delete the port related to it firstly, and the port deletion always fails, prompt 500. In the information returned by GET ports, what do eps and host_id stand for? I understand from the api source code that these two fields also have GET operation, but there is no DELETE and POST operation.

Can you try to check the logs for the zeta-manager pods?
You can do kubectl get pods to find all pods in your zgc cluster, then kubectl logs zeta_manager_pod_name to check the logs.

@liangbin-pub,hello,the zgc control node has been deployed,the machine's port is  8221

@PikaPikaW Thanks, I checked zeta repo is good, thanks

Hello, I would like to confirm that now tower1 221 is used as Zeta Control Plane Server and an ACA Node. So tower2 222 is used as another ACA node or used to deploy ZGC nodes? and the other two servers <IP_ADDRESS> and <IP_ADDRESS> how to allocate?

No, tower 1 is used for:
ACA parent, Pseudo controller and Zeta deployer (which deploys zeta to remote servers)
Tower 2 is used for: ACA child
Server 1 is used for ZGC nodes (will deploy 6 VMs there) as ZGC nodes (<IP_ADDRESS>-<IP_ADDRESS> on tenant network and <IP_ADDRESS>-<IP_ADDRESS> on management network)
Server2 is used for Zeta control plane (K8S cluster) which offers Zeta REST API @ http://<IP_ADDRESS>/
Hope this is clear

Update:
I lost connectivity to server 2, so I'm using server1 (50.7) to deploy ZGC nodes (VMs), it's done, total 6 VMs.
To re-deploy (normally not needed), login and in ~/kvm, run "sudo ./kvm_deploy.sh 6", these will delete old VMs and re-deply.
Once Qianyu brings back server2 (fresh install ok), Zeta control plane services (providing REST API http://<IP_ADDRESS>/)can be deployed on it following following step:

login tower1 (<IP_ADDRESS>)
cd ~/zeta
./deploy/full_deploy.sh -d lab

Then follow ACA instructions to deploy ACA on both tower1 (parent) and tower2 (child)
When using Pseudo controller, make sure change following parameters:

ip_start and ip_end in ZGC create API call, give it <IP_ADDRESS>-<IP_ADDRESS>   (16 unused IPs in <IP_ADDRESS>/24)
when adding AGC nodes (6 times for the 6 ZGC node VMs), ip_control is <IP_ADDRESS> for node1, <IP_ADDRESS> for node2, etc, up to node 6; id_control and pwd_control van put any string, not used; int_tenant put "ens4", inf_zgc put "ens3", you need to get MAC address of each ZGC node VM's ens3, ens4 interface and fill in as mac_zgc and mac_tenant.

per 12-21-2020 open source community discussion, closing this older PR. Please rebase with the latest code and create new PR for the next code change. https://github.com/futurewei-cloud/alcor-control-agent/blob/master/src/README.md


Support DbContext.FromSqlRaw

Today there is a DbContext.Set<T>().FromSql<T>(), the problem is that when u use this , your sql should map to entity or a quey type that DbContext know about how to map the property.
That add may overhead for devs to just use group by , join  for some temporary query.
normally we just switch to use Dapper ,  but the good of .FormSql<T>() is you can then add .Where(Expression) to it.
I'm suggestion add a new method that create IQuerable<T> directly from sql,  and the property map must use Sql to match model  instead of use model to match sql so there no need to configurate the query model in the dbContext.
DbContext.FromSqlRaw<T>(string sql)
DbContext.FromSqlRaw<T>(string sql, T objectProjection> // some idea of dynamic/anonymous support , .FromSql("SELECT **", new { Name= "", Age = 0, BirthDay = (DateTime?)null })

Duplicate of #10753


Vulkan: StandardMaterial3D alpha scissor does not cast shadows

Godot version
Calinou Nightly build 5710adda7
System information
Macbook Pro 2019 16"
Issue description
I've been trying to get basic alpha textures to cast shadows from a SurfaceTool geometry to no avail. They receive them no problems, and when i turn alpha off, the object then shadows, but not with Alpha on.

Steps to reproduce
Create a flat geo with SurfaceTool, or with ArrayMesh ()
Give it a material and texture with an alpha
Minimal reproduction project
No response
18_StandardMaterialShadowBug_GDv4.0.zip

Try enabling opaque prepass on the material. See also https://github.com/godotengine/godot/issues/17567.

I have a similar issue when importing a mesh with an alpha texture in Godot 4. When enabling depth pre-pass i see the shadow, but its loosing alpha on the texture. I guess its an AMD Radeon issue.

Turns out that it's any Mesh with alpha textures!
I've uploaded the minimum reproduction in the original post.

Yep, could be a Mac thing. I get this error whenever i try to edit the transparency.

Nope. Not just a Mac/Radeon thing, I've just tested the same project on my Surface Pro and it's not giving me any shadows on StandardMaterial3D with alphas on their either.

I've got the same issues in (Godot_v4.0-dev.20210916_win64) downloaded from tuxfamilly
Windows 10
AMD Ryzen 9 3900X
NVIDIA GeForce RTX 3060

I opened #55084 with some fixes.
It would be nice if you guys could confirm whether it fixes all your issues or there are things still unresolved.


Manually implement Default

I broke chrono's MSRV with the unstable-locales feature in #8.
error[E0658]: deriving `Default` on enums is experimental
     --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/pure-rust-locales-0.8.0/src/lib.rs:55118:23
      |
55118 | #[derive(Copy, Clone, Default, PartialEq, Eq, Hash)]
      |                       ^^^^^^^
      |
      = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
      = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

This fixes the build for rust 1.61 and adds a CI check for the generated code on rust nightly, beta and 1.61.0.
Can we release this as 0.8.1 to make the CI for chrono green again?

I think you could yank the other and release this one

Should set the rust-version in the Cargo metadata, I think you won't have to suppress the lint then.

Should set the rust-version in the Cargo metadata, I think you won't have to suppress the lint then.

Of course, thank you for reminding me! That is good to set anyway. I'll use the lowest rust version that currently works and supports rust-version, 1.56.0

I think you could yank the other and release this one

Not building with an old rust version doesn't seem quite bad enough for yanking I think.

Published.


Refactor SSH init code, fix directory creation for TrustedUserCAKeys file

Follow #20289

#20289

In #20289, the ~/.ssh is not created automatically, but that PR is not ideal: indeed, only the GlobalInitInstalled function should prepare the SSH files for external server or starts the builtin server.
This PR should make the logic clear and reduce some dependencies between modules.
Major changes:

trustedUserCaKeys is removed, use SSH.TrustedUserCAKeys directly, it has been parsed by ini module correctly
introduce ssh.Init, move the SSH init code from routers/init.go to it
ssh.Init will start builtin SSH server or prepare external SSH server files

This PR could also to be considered as a bug fix for #20289, otherwise the os.WriteFile(fname/*ca file*/, might fail.
And a strange FIXME:

FIXME: why 0o644 for a directory ..... os.MkdirAll(setting.SSH.KeyTestPath, 0o644)

Codecov Report

Merging #20299 (c3b347e) into main (711cbcc) will decrease coverage by 0.00%.
The diff coverage is 47.24%.

@@            Coverage Diff             @@
##             main   #20299      +/-   ##
==========================================
- Coverage   46.91%   46.91%   -0.01%
==========================================
  Files         973      976       +3
  Lines      134721   135007     +286
==========================================
+ Hits        63204    63334     +130
- Misses      63768    63911     +143
- Partials     7749     7762      +13

Impacted Files
Coverage Δ

cmd/dump_repo.go
0.00% <0.00%> (ø)

cmd/restore_repo.go
0.00% <0.00%> (ø)

models/issues/issue_project.go
30.00% <0.00%> (-2.24%)
:arrow_down:

models/unittest/testdb.go
14.38% <0.00%> (-1.59%)
:arrow_down:

models/user/user.go
51.54% <0.00%> (-2.81%)
:arrow_down:

modules/indexer/code/elastic_search.go
1.12% <0.00%> (ø)

modules/markup/html.go
35.67% <0.00%> (-0.05%)
:arrow_down:

modules/markup/markdown/toc.go
0.00% <0.00%> (ø)

modules/migration/null_downloader.go
31.42% <0.00%> (ø)

modules/setting/i18n.go
100.00% <ø> (ø)

... and 78 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 a9e66cf...c3b347e. Read the comment docs.


I18n for connections

Pending a fix due to
React Hook "useTranslation" cannot be called at the top level. React Hooks must be called in a React function component or a custom React Hook function.

We have a t function for testing which is exported from test/utils

Updated with proposed changes. Still something going on. I just get the error in the console missingKey en connections add_friend add_friend, and no clue why that's happening.

get the error in the console missingKey en connections add_friend add_friend, and no clue why that's happening.

Ahh I see where the issue is - it's because we share some connections related strings on the user/profile page (like the "add friend" button), and we aren't including the connections namespace in SSR in the relevant routing file under the pages folder.
https://github.com/Couchers-org/couchers/blob/093d88dd06a83d9f2f4316fe05275da1ec6ad3c8/app/web/pages/user/[[...slug]].tsx#L16-L24
https://github.com/Couchers-org/couchers/blob/093d88dd06a83d9f2f4316fe05275da1ec6ad3c8/app/web/pages/profile/index.tsx#L7-L15
If you update these to include "connections" namespace it should do the trick.

No idea why this is failing :S

@Jesus82 app/web/features/connections/constants.ts is still present but empty - it should be deleted (typescript expects every .ts file to either export or import something)

Looks like there's still a loose constant in the tests since CI is failing on the following error:
features/messages/groupchats/GroupChatView.test.tsx(386,59): error TS2304: Cannot find name 'SEND'.

I checked that out, and I cannot see what the error is. Also I didn't touch that file in this PR :S

Looks like there's still a loose constant in the tests since CI is failing on the following error:
features/messages/groupchats/GroupChatView.test.tsx(386,59): error TS2304: Cannot find name 'SEND'.

That was a tricky one, as I thought that was related with a translated constant, but it was not. That is solved now, but test still failing.

Looks like there's still a loose constant in the tests since CI is failing on the following error:
features/messages/groupchats/GroupChatView.test.tsx(386,59): error TS2304: Cannot find name 'SEND'.

That was a tricky one, as I thought that was related with a translated constant, but it was not. That is solved now, but test still failing.

The failing test is unrelated to your change - it's a known flaky one that's been popping up recently :/


Roll back to REST API endpoints

Core engine and tests are working smoothly but documentation (Which is auto-generated using Swagger) is generating wrong APIs for Resourses that have 2 routes as shown in the picture below, So still working on it.
(Closes #16 )

Thank you @farbodahm for your attention :)


SameSite

Is it possible to set the SameSite option on a cookie, and if so, how do I do it? I didn't see it listed under the set options.

Yes the option is supported and was missing in the TypeScript typing until the last version. I've updated the readme with the information. Thank you for letting me know! 👍


Extras - Switch for Fallback Internet not available/showing

Describe the bug
I have successfully configured the plugin, allowing me to get switches for Fritzbox on/off, LED and reconnect besides others. All works as expected, switches appear in the Home app and also work.
However, when tryiing to configure the Option->Extras->Fallback Internet and select it to be a switch and corresponding states for on and off, such a switch NEVER shows up anywhere in homekit or the home app. I also tried different apps (Eve) to no avail.
To Reproduce
Homebidge config extract:
"extras": {
"fallbackInternet": {
"active": true,
"provider": "DEUTSCHE TELEKOM",
"onState": "1",
"offState": "3",
"accType": "switch"
I tried various other values for on/off state (0,2) with no change, it just does not work for me, no switch is shown/coming up.
Expected behavior
A switch to be able to turn fallback internet on/off should be aviailable in the home app or homebridge.
AVM Device

Fritz OS 7.50, Fritzbox 6591

-Homebridge: 1.6.0, Node.js: 18.14.2
Additional context
When making this Extras option a characteristic, I am able to get info on fallback internet status shown, but is read only.

Same for me
"I tried various other values for on/off state (0,2) with no change, it just does not work for me, no switch is shown/coming up"


Add attribute/sensor/etc for whether or not a cleaning is automatic/scheduled or manually started

Does Dreame's API expose anything that would indicate whether the current cleaning task was started from a scheduled job? It would be nice to have Home Assistant send a notification when a scheduled task starts, since the DreameHome app doesn't do this.

This is great, thanks!


fix functions that aggregate to include the aliases in their params

This should fix the validation not being aware of legal aliases.
fix #2494
@DanCech will this break anything else that relies on .params ? i see this property is used in quite many places but didn't investigate them all

The only problem I have with this approach is that we don't distinguish between the standard and deprecated options.  It might be better if we sort the aggFuncs then sort the aliases separately and append them.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2496   +/-   ##
=======================================
  Coverage   80.01%   80.01%
=======================================
  Files          85       85
  Lines        9131     9131
  Branches     1948     1948
=======================================
  Hits         7306     7306
  Misses       1544     1544
  Partials      281      281

Impacted Files
Coverage Δ

webapp/graphite/render/functions.py
96.26% <100%> (ø)
: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 a4941cc...f384575. Read the comment docs.

the aliases are deprated? ok here you go.


Followers list dropped when you follow somebody

This issue also happens with relay list. As NIP-02 said: "Every new contact list that gets published overwrites the past ones, so it should contain all entries. Relays and clients SHOULD delete past contact lists as soon as they receive a new one._ something related to this on library makes you follow only one npub and "forget" or delete the others you were following.
Could be related with max bytes accepted by relays? I don't know but I will appreciate help with this issue! I hope I can understand if it is a bug or it is clients fault.

The issue is not related to this library. It is some client somewhere doing things wrong and overwriting your follow lists. Use https://coracle.social/ or https://github.commikedilger/gossip.


sendPhoto never returns in some circumstances

System: Raspberry 3 Model B with Raspbian
In my python script I send a photo with:
bot.sendPhoto(chatid,("webcam.jpg", cameraImage), message)
Normally it works, and if there isn't internet connection it retries to connect for 4 times , and after (about 2 minutes) it launches an exception. Perfect.
But during the retry attempts of sending, if the connection comes back, it locks up and the function sendPhoto() never returns (I wait for about 10 minutes...).
When it start to sending image without internet connection:
Converted retries value: 3 -> Retry(total=3, connect=None, read=None, redirect=0)
......connection fails, retrying...
-> internet connection comes back
Starting new HTTPS connection (2): api.telegram.org
--> No more logs, the script is blocked
I tried to recreate this scenario many times and it happens frequently (not always).

You are testing your bot very thoroughly :smiley:
I also did my test on Raspbian Jessie, Pi 3, and cut internet connection by unplugging the LAN cable between my home router and modem. Unfortunately, I was not able to reproduce your situation. I varied the delay of re-plugging the LAN cable, and it either retried successfully (if re-plugged soon enough) or raised a MaxRetryError (if re-plugged too late), but never hung forever.
I tried on both Python 2.7 and 3, combined no less than 10 times. So, I have no more meaningful things to say .... maybe your modem or router does not handle re-established connections properly?

Yes... :)
During my tests I don't unplug the LAN cable to simulate the disconnection, but I disconnect internet connection inside my router web interface. In this way I simulate the ADSL is down. Unplugging the LAN cable is not the same thing: it lost the ip address and when re-plugging the cable it re-establish it making a dhcp request.
When I re-establish internet in my router web interface the connection come back correctly I tested it in raspberry too.
I don't understand what happens in that moment in raspberry to block the request.
How can I log it at deeper level?

I believe one needs to generate network timeouts to replicate this problem which are quite uncommon.


2022050802

Closes #4805
Closes #4806 Another PR
Closes #2210 Already removed
Closes #4810
Closes #4803
Closes #4739
Closes #4429
Closes #4428
Closes #4570
Closes #4569
Closes #4566
Closes #4565
Closes #4564
Closes #4563
Closes #4562
Closes #4561
Closes #4560
Closes #4559
Closes #4558
Closes #4557
Closes #4556
Closes #4555
Closes #4554
Closes #4553
Closes #4552
Closes #4549
Closes #4551
Closes #4550
Closes #4803

Can you please look into issue #4276


Introduce new image tagging strategy

Description of the change
Introduce new tagging strategy.
So far we do not change the :latest tag, it will stay on ansible v7 to avoid introducing BC breaks.
Check out the readme for more details.
Fix #19 and #16
Type of change

[ ] Bug fix (non-breaking change that fixes an issue);
[x] New feature (non-breaking change that adds functionality);
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected);
[ ] Other (miscellaneous, GitHub workflow changes, changes to the PR template);

Checklists
Development

[ ] Parts of this pull request impacting core (user-facing, documented) product functionality have test coverage;

Code review

[x] This pull request has a descriptive title and sufficient context for a reviewer. There may be a screenshot or screencast attached;
[ ] This pull request is no longer a draft;
[ ] You have reviewed this Pull Request yourself;

@michalg9 Could you have a look at this PR when you'll find some time? 🙏🏻


Add optional argument to specify base in Num.fromString

Depends on #984.
Allows Num.fromString to take an optional argument to specify radix.
Currently, I set it to throw errors if the radix is invalid (greater than 36 or less than 2 and unequal to 0) however some may prefer Num.fromString to return null instead (JavaScript appears to do this, though they also don't appear to check that the radix is an integer or that the first argument is even a string!).
For now, I'll mark this PR as draft until #984 is accepted or declined.

This is something I've been meaning to suggest myself so you've beaten me to it :)
Regardless of the fate of  #984, I still think it's worth doing.
I agree that it's better to throw an error if the base is invalid rather than return null.

Thinking some more about my last point, perhaps an entirely new method Num.toBaseString(radix) would be better.

I prefer an overloaded Num.toString(radix).

I prefer an overloaded Num.toString(radix)

Well, it would be more symmetric with what we already have.
Another alternative would be String.fromNum(num, radix) though that would need to be static.


Use psycopg2 instead of psycopg2-binary.

This PR changes DefectDojo to use the psycopg2 package instead of psycopg2-binary.
This fixes a problem with running DefectDojo on aarch64 machines, with the main motivation here being able to do dev work on an Apple Silicon Macbook.
The psycopg2 package also recommends build from source instead of relying on the binary package.

The binary package is a practical choice for development and testing but in production it is advised to use the package built from sources.

There was some talk about moving to psycopg3 recently. That may be the better approach here

I agree with @Maffooch - if we're going to change how we handle that dependency, might as well upgrade to psycopg3

I'm not familiar enough with the DefectDojo codebase to make that change, since I believe the APIs aren't compatible, so someone else will have to take that on. :)

@Ayrx thank you for your PR, we're going to work internally on going to psycops3, so I think adding the binary is moving in the wrong direction.

It occured to me that you will have the same issue with psyocpg3. The recommended installation method for production sites is the local installation method which still requires libpq-dev.
https://www.psycopg.org/psycopg3/docs/basic/install.html#local-installation

@Ayrx so? are modifying your PR to switch to psyocpg3 or we abandon this PR?

Thanks @Maffooch for doing the research on this :+1:


Fix utxo query result and test cases, PLT-1540, and PLT-1536

Fixes are:

fix the broken property tests in Utxo
remove the equivalent test criteria, replace with sort/compare
add isToVacuume switch indicating SQLite vacuum operation. The switch is not in CLI, and set to True. In test we set that to False.
fix the Query results
Balance the in-memory Utxos in regards to Spent
minor change in the JSON form of the UtxoResponse.
Pre-submit checklist:
Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated
[ ] Important changes are reflected in changelog.d of the affected packages

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

First pass, I'll take a more in-depth look later. Which test actually makes the query fail without the fix?

@kayvank

@koslambrou  Roundtrip store & retrieve test(s) were failing for depth greater than the number of events. In particular, this one always failed.
I saved the failure and will attach it to the ticket

The other PR has been merged: https://github.com/input-output-hk/plutus-apps/pull/1034, so a rebase is necessary.

created https://input-output.atlassian.net/browse/PLT-5460 for the toVacuum flag


Unable to call instance method from C

Hey everyone. I just can't seem to figure out how to call a instance method from C. Here's a minimal example:
Here is my wren code
class A {
  construct new() {}
  b(){
    System.print("I was called yeeehaa!")
  }
}
var a = A.new()
ForeignFuncs.callB(a)

An here is my C code
static void callback(WrenVM* vm){
  WrenHandle* callHandle = wrenMakeCallHandle(vm, "b()");
  WrenHandle* instHandle = wrenGetSlotHandle(vm, 1);
  wrenSetSlotHandle(vm, 0, instHandle);
  wrenCall(vm, callHandle);
}

However this does not seem to work. Anyone has an idea why this is not working?
Wren-Error in module '(null)' line -1: Fn does not implement 'b()'.
Wren-Error in module 'main' line 22: (script)
Segmentation fault

`

Are you trying in debug? One important not is that you can't call from a foreign function. WrenCall inside foreign is an assert (which you'd see in debug)

Hi ruby, thanks for pointing this out. Yep, I did not build in debug mode but this is most likely a re-entrancy issue - which, as pointed out, is not possible.
However, I cant figure out how to mitigate this. In my current setup, wren is running my main loop. So there is no way around re-entrancy. E.g. main_loop (wren) -> poll_events (c) -> input_callback (c) -> wren_method (wren) !! - reentrancy
I guess a work-around for this, is to move the main loop to c, right?

There is AFAIK no real solution to this problem with current API. The only
practical solution is to make C code less smart, pull the event from C to
wren an do the callback from there.
I'm trying to solve this problem since the introduction of libuv dependency
to wren. It introduced some hacky code to circumvent the problem, that is
not really safe and reusable.


What version of Power Bi?

I have installed Power Bi (the version released in May 2019) on my computer. Your Power Bi template cannot be opened due to version compatibility.
Installed Power Bi Product Version:
2.69.5467.1801 (19.05) (x64)
Error Message:
'..\FlowViz - Agile Template.pbit' is incompatible with your current version of Microsoft Power BI Desktop. Please install the latest version and try opening the document again.

The prerequisite comments in the description say to “make sure you have the latest version of Power BI desktop downloaded” - it’s a hyperlink too to link to where you can download it 🙂

Closing as not an issue with the template, will edit instructions on Azure DevOps marketplace so it's clearer :)


Update isovar usage

@iskandr is changing a few things around. Also, his comment from before:

My only suggestion would be to maybe increase the minimum MAPQ to 1 and if you're ultimately using the counts of neoepitopes with >= 3 spanning reads then this will be very sensitive to the degree of exonic coverage. Maybe normalize by number of reads mapping to exons?

Also see https://github.com/hammerlab/bladder-analyses/issues/4#issuecomment-222227666 (sorry to link to a private repo).


Issue: Party Changing Feature, party members not loaded in map

when changing party members using the toybox feature the new party members seem to remain loaded in their original location, teleport feature doesnt seem to address this either, travelling across the different areas of the 4th chapter's region doesnt seem to fix it either

Is this while on the global map or in an area?

I had reported the same issue (#317), so I closed that one. It happens at least in areas. Bag of Tricks had the same issue, but had an extra button, to "activate" the characters, that would fix it.


Fix of issue #787

Fixes #787
Converted the /contact route handler to use async/await for better readability and handling of asynchronous operations.
Cached the Nodemailer transporter outside of the route handler to reuse it for sending emails.

@himanshu-03 hii, please review and merge


Support NetBSD guests

Assemble the correct commands to boot NetBSD guests.

NetBSD 9+ supports UEFI. This is a "won't do".


Define a Simple Data Model and OData Service with CDS

Tutorial URL: https://developers.sap.com/tutorials/odata-05-data-model-service.html
Please specify the step you are referring to 6
Hi DJ,
Are the Products still available or is this now all Bookshop data model?
thx, gm

Hi @greg2git - the Products are from Northwind, a sample OData service supplied by Oasis. This is separate to and independent from the Bookshop samples you're more often likely to see in CAP examples like this one. Both are great though as they provide a simple but interesting enough data model.


Guesslang in VS Code

Hi there,
My name is Isidor and I work on VS Code.
We have the following problem:
Novice user create a new untitled files and start typing and they have no clue that they have to set the language mode to get all the language smartness that VS Code provides.
Thus we were thinking to use some smart language detection so we could automatically set the language for the user.
I was doing a bit of online research and I came across this project - looks very cool!
Is it possible to somehow have this work as a node module instead of python? Since then we could consume it easily in VS Code and things might just work. Even cooler would be if it worked in the browser.
Let me know if you are interested we can also setup a meeting where I could explain our use case in more detail.
Thanks!

Hello @isidorn
Actually, I tried to convert Guesslang model to Javascript two years ago because I wanted to create an Atom extension, a VS Code extension and a Javascript front-end library.
But it was a real challenge to convert the model to TensorFlow.js because several elements of the models where not implemented in TensorFlow.js at that time. I tried different options like simplifying Guesslang model, splitting the model, compiling the missing elements from C++ to Web Assembly, etc... While some solutions "worked", the result was way too buggy and dirty to ship.
By the way, sorry for the late answer.

It would be nice to have the insight of someone who managed to convert a canned TensorFlow model to TensorFlow.js.

@yoeo thanks for your answer.
We are actually working with the Tensorflow team so they support some of the missing funcitonality so we can convert this model to JS. Please check out this issue https://github.com/tensorflow/tfjs/issues/4838
Let me also ping the Tensorflow team again so we try to get some progress here. It would be super cool to have this in VS Code.

@yoeo the Tensorflow team have just updated TensorflowJS so now it is possible to run your model in the browser, for more details checkout this comment https://github.com/tensorflow/tfjs/issues/4838#issuecomment-850730654
We will look into adopting this in VS Code next milestone in June.
In the meantime is it possible for the Model to be updated to also Classify JSON? This is a very common language for our users and it would be great if the model could support it.

Hi @isidorn that's really good news.
Yes it is possible to add JSON.
However, it will take some time to generate a new training dataset that includes JSON and other requested languages like VisualBasic, Pascal, Kotlin, XML, YAML, etc... By the way the part that takes the most time is actually downloading ~1TB of repositories for Github.

@yoeo cool, it would be really useful for us to add JSON when possible.
Makes sense that the 1TB download is the slowest part...
Thanks a lot and I will provide more feedback in a couple of weeks when I try to integrate all of this into VS Code.

There's progress on our side and we are looking into adding this to our VS Code product. More details can be found here https://github.com/tensorflow/tfjs/issues/4838#issuecomment-859275362 and https://github.com/microsoft/vscode/issues/118455
Two things we would still really like to improve in order to ship this:

[ ] more languages support (as we discussed above)
[ ] Is it possible to further compress the model? Current size is 1.1 MB and we want to ship this as part of every VS Code.

@yoeo if you do not have time can you give us some instructions on how to do the first. So we might also put a help wanted in the VS Code repository, since we have lots of contributors maybe somebody will volunteer.
Thanks a lot for this great model

We were able to compress to model using gzip to be 20kb. So we are good regarding the size.
However improving the model for more languages would be fantastic.

Wow I wonder how you were able to compress the model that much, that's insane!

more languages support (as we discussed above)

I'll try experiment that this week end.

can you give us some instructions on how to do the first

Of course. I use this tool to build the dataset: https://github.com/yoeo/guesslangtools
You can find how to use it on the README page but here's a quick guide on how to add new language with this tool:
# Install Gesslang & GuesslangTools inside your virtualenv in developer mode
git clone git@github.com:yoeo/guesslang.git guesslang
cd guesslang/
pip install -e .
cd ..

git clone git@github.com:yoeo/guesslangtools.git guesslangtools
cd guesslangtools/
pip install -e .
cd ..

# Add the new languages to the language mapping
vi guesslang/guesslang/data/languages.json
cp  guesslang/guesslang/data/languages.json guesslangtools/guesslangtools/data/languages.json

# Build the dataset (might take few days, depending on your Internet connection)
DESTINATION_PATH=...  # dataset directory, 1TB of free space recommended
gltool $DESTINATION_PATH

# Train the model (might take few hours, depending on your computer speed)
guesslang --train $DESTINATION_PATH --model ./new_model/

# Play with the new model
echo '
#include <stdio.h>

int main(int argc, char* argv[])
{
  printf("Hello world");
}
' | guesslang --model ./new_model/  # Should output "Programming language: C"

Thanks for the updates.

Hello, just an update.
I'm trying to add the following languages to Guesslang:

Assembly
CSV
Dart
Fortran
Groovy
INI
JSON
Julia
Kotlin
Pascal
TOML
TypeScript
VBA
XML
YAML

I built the list according the requests https://github.com/yoeo/guesslang/issues/24 https://github.com/yoeo/guesslang/issues/23 https://github.com/yoeo/guesslang/issues/19 , Tiobe language index and Stackoverflow popular languages .
But there are few issues:

some language datasets that I'm generating are too specific (ex: many TOML files that I'm getting are Cargo.toml files).
and I mostly work on it during week-ends, so things may go slowly.

Therefore my current strategy is to add the "simplest" languages first then bump Guesslang and take some time to work on the more tricky languages.
Thanks.

@yoeo Thanks a lot for looking into this and for providing an update.
Starting with the simplest languages makes good sense to me.
As for the compression: the model after being converted for TensorFlowJS was .json and that seems to be easily compressible.
If needed I can put help-wanted on the vscode issues, and somebody from the community can also help here. Just let me know...
We would love to ship this feature in July / August, but at the end of the day there is no rush, we would like to get it right and there are other things we need to look at

Hi @isidorn
I've made some progress during last couple of weekends. There is now a development version of the model that supports most of the languages listed above (including JSON). You can try it here https://github.com/yoeo/guesslang/pull/33
There are still issues that I need to solve before merging:

The dataset download was taking forever :hourglass_flowing_sand: I had to rewrite my dataset creation tool to speed things up a little. This work is still ongoing... https://github.com/yoeo/guesslangtools/pull/41. the "Pascal" language training dataset is broken, I'll have to generate a new ones
I didn't find enough training files for some languages like "TOML", "INI". I still have hope though

@yoeo this is great work, thanks a lot for the update 👏
I think @tylerleonhardt plans to jump on this next milestone (July), so the timings seem to align. I think we still have to streamline the conversion to TensorFlowJS as captured here.
Next three weeks I will be on vacation, so expect slow responses from me.

Hi @isidorn , @TylerLeonhardt
I just finished adding the new languages support to Guesslang https://github.com/yoeo/guesslang/pull/33
It now supports Guesslang 54 programming languages (24 more than before):

Languages

Assembly
Batchfile
C
C#
C++

Clojure
CMake
COBOL
CoffeeScript
CSS

CSV
Dart
DM
Dockerfile
Elixir

Erlang
Fortran
Go
Groovy
Haskell

HTML
INI
Java
JavaScript
JSON

Julia
Kotlin
Lisp
Lua
Makefile

Markdown
Matlab
Objective-C
OCaml
Pascal

Perl
PHP
PowerShell
Prolog
Python

R
Ruby
Rust
Scala
Shell

SQL
Swift
TeX
TOML
TypeScript

Verilog
Visual Basic
XML
YAML

Feel free to tell me if you have feedbacks about this new model.

Being able to tell the difference between Matlab Objective-C and Julia is killer, thanks*e(6)!

@yoeo this is amazing, thanks a lot 👏  We really appreciate your help.
I just came back from vacation, but @tylerleonhardt was working on this and there is already a prototype of this in VS Code. More details can be found here https://github.com/microsoft/vscode/issues/129004
In short if you want to try it out you should set workbench.editor.untitled.experimentalLanguageDetection in vs code settings.
We are about to pick up your latest model.

I think we can go ahead and close this issue now :) the guesslang model now ships in VS Code and beyond misc improvements to the model, there's no more action items here.

In short if you want to try it out you should set workbench.editor.untitled.experimentalLanguageDetection in vs code settings.
Thanks for the info. You've got a new beta-tester :slightly_smiling_face:

I'm closing this issue and do not hesitate to create new ones for improvement requests.

For reference here's the test plan item on the vscode side that has good steps on how to setup https://github.com/microsoft/vscode/issues/129436


feat: add sandbox generation PE-1440

There appears to be an empty file committed: sanbox_test.dart


Search page title does not change with area aggregates (county, state)

Right now, if you switch to viewing demands for your county or your state, the title of the list/search page still always says "Town of  demands".
Need to make the title reflect the currently selected area, such as "County of Middlesex demands" or "State of Massachusetts demands".
Also, we should attempt to make the title smart enough to say "Town of" vs. "City of" if we have that data available.

Linked to #24

Fixed in #28


Sync with FENRIR

pqmf
inference.py

Version of python must be >= 3.9 cause TTS is tested on Ubuntu 18.04 with python >= 3.9, < 3.12


Restored terminals in editor area use default (?) profile instead of the correct one

Type: Bug

Ensure "terminal.integrated.defaultLocation": "editor" is set.
Open Remote SSH session (I'm not sure whether this is actually required or not, but I use VS Code with Remote sessions exclusively)
Choose Terminal: Create New Terminal (With Profile), select the appropriate profile and confirm.
Restart VS Code
Observe a terminal will be restored in the editor area but in my case it's using the "bash" profile rather than the profile that I chose earlier.

VS Code version: Code 1.83.1 (f1b07bd25dfad64b0167beb15359ae573aecd2cc, 2023-10-10T23:57:32.750Z)
OS version: Darwin arm64 21.6.0
Modes:
Connection to 'SSH: homer.kelman.com' could not be established
Remote OS version: Linux x64 4.18.0-477.13.1.el8_8.x86_64

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
1, 7, 7

Memory (System)
16.00GB (0.08GB free)

Process Argv
--crash-reporter-id 4abd401a-b2b8-4d50-b481-1e179f7002ef

Screen Reader
no

VM
0%

Connection to 'SSH: homer.kelman.com' could not be established

Item
Value

Remote
SSH: cal-devel21

OS
Linux x64 4.18.0-477.13.1.el8_8.x86_64

CPUs
12th Gen Intel(R) Core(TM) i7-12700 (20 x 2100)

Memory (System)
30.84GB (21.31GB free)

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

Bookmarks
ale
13.4.2

FreeMarker
dco
0.0.9

vscode-position
jtr
1.1.2

jupyter-keymap
ms-
1.1.2

remote-containers
ms-
0.315.1

remote-ssh
ms-
0.106.5

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.81.8

vscode-remote-extensionpack
ms-
0.24.0

remote-explorer
ms-
0.4.1

remote-server
ms-
1.4.3

open-in-browser
tec
2.0.0

asciidoctor-vscode
asc
3.1.5

atlascode
atl
3.0.7

gitlens
eam
14.4.0

jbang-vscode
jba
0.9.0

vscode-position
jtr
1.1.2

vscode-docker
ms-
1.27.0

fabric8-analytics
red
0.7.0

java
red
1.23.0

vscode-commons
red
0.0.6

vscode-rsp-ui
red
0.24.9

vscode-server-connector
red
0.26.12

vscode-xml
red
0.26.1

vscode-yaml
red
1.14.0

LiveServer
rit
5.7.9

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.2.30

vscode-gradle
vsc
3.13.0

vscode-java-debug
vsc
0.54.0

vscode-java-dependency
vsc
0.23.1

vscode-java-pack
vsc
0.25.14

vscode-java-test
vsc
0.40.0

vscode-maven
vsc
0.42.0

vscode-proto3
zxh
0.5.5

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30856252
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
282f8724:30602487
89544117:30613380
a9j8j154:30646983
showlangstatbar:30737416
03d35959:30757346
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
copilotsettingt:30859503
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
dsvsc015:30845448
h48ei257:30863626
pythontbext0:30864172

This is essentially a duplicate of #190749 but I was unable to add a note confirming it's still an issue as it was locked.

@Tyriar I can confirm this is still an issue today in 1.83.1.  I assume the code in the Insider build you mentioned back on Aug 18 would be in the September release?

/duplicate #190749

@meganrogge Is it possible to get the original issue (#190749) unlocked as well so I can add a note on there?


Fix for #389

Thank you @alimbada for opening this PR. I've made a similar change that passed the commit lint test.


Fix bug that skips other selectors if ToEndpoints is empty

policy: Add test for non-empty ToCIDR and empty ToEndpoints
policy/api: Fix bug that skips other selectors if ToEndpoints is empty

Second commit for convenience:

policy/api: Fix bug that skips other selectors if ToEndpoints is empty
As reported in GH-33432, do not skip other selectors in the case
ToEndpoints is empty. For example, if the user defined a ToCIDR with an
empty ToEndpoints, then
getDestinationEndpointSelectorsWithRequirements() /
GetSourceEndpointSelectorsWithRequirements() returns nil, causing the
ToCIDRs section to be skipped. Instead, return the aggregated selectors,
which aggregates selectors such as ToCIDRs, prior to the call to the
aforementioned functions inside SetAggregatedSelectors().

Fixes: e97df7badd ("policy: Do not select any identity with ingress empty slices")
Fixes: 6ccd044b05 ("policy: Do not select any identity with egress empty slices")
Fixes: https://github.com/cilium/cilium/pull/29608

Fixes: https://github.com/cilium/cilium/pull/29608
cc @pippolo84

/test

/test

I investigated in more detail the issue and I believe this is not what we want.
This fix successfully equates the two cases:

apply of the policy:

apiVersion: "cilium.io/v2"
kind: CiliumNetworkPolicy
metadata:
  name: test
spec:
  endpointSelector:
    matchLabels:
      run: tmp-shell
  egress:
  - toCIDR:
    - <IP_ADDRESS>/32
    toEndpoints: [] # <-- this
    toPorts:
      - ports:
          - port: "80"

apply of the policy with a nil toEndpoints selector, then update it with an empty non-nil toEndpoints selector, ending up with the same policy as in case 1

But it does that in a way that reverts what https://github.com/cilium/cilium/pull/29608 fixed. The failed tests reflects this (as an example, see TestMergeDenyAllL3 - case 1B, that was introduced in https://github.com/cilium/cilium/pull/29608).
In other words, as suggested in the issue, the correct behavior should be that the two scenarios above ("direct" apply of the final version of the policy and apply+update) both lead to the policy denying connections to <IP_ADDRESS>.
Quoting from the issue:

Expected
The behaviour should be consistent regardless of the order of operation on a policy.
Following result is what a user should experience at all times for above policy with toEndpoints: [] in 1.16:
$ curl -m 1 <IP_ADDRESS>
curl: (28) Connection timed out after 1002 milliseconds

$ curl -m 1 <IP_ADDRESS>
curl: (28) Connection timed out after 1001 milliseconds

This is not happening with the proposed fix, that allows the curl <IP_ADDRESS> in both case 1) and 2).
To get the expected behavior I think the solution should be this one, where we change the logic to compare EgressCommonRule and IngressCommonRule instances, taking into account the different semantic of nil and empty slices and propagating the update in the scenario above as expected.

Closing in favor of https://github.com/cilium/cilium/pull/33506


[stable/redis] error: failed switching to "redis": unable to find user redis: no matching entries in passwd file

Describe the bug
Master and Slave pods keep crashing due to the below error.
Version of Helm and Kubernetes: Kubernetes: v1.14, Helm v2.9.1
Which chart: stable/redis
What happened:
Mohammeds-MacBook-Pro:further-exchange mfadin$ kubectl logs gogs-redis-master-1
INFO  ==> ** Starting Redis **
error: failed switching to "redis": unable to find user redis: no matching entries in passwd file

Mohammeds-MacBook-Pro:further-exchange mfadin$ kubectl logs gogs-redis-slave-0
INFO  ==> ** Starting Redis **
error: failed switching to "redis": unable to find user redis: no matching entries in passwd file

What you expected to happen: Pods to run
How to reproduce it (as minimally and precisely as possible):
helm upgrade  --tls gogs-redis --set master.persistence.size=20Gi --set master.persistence.storageClass=ibmc-file-bronze --set securityContext.enabled=true --set securityContext.fsGroup=0 --set securityContext.runAsUser=0  --set volumePermissions.enabled=true  stable/redis

Anything else we need to know: None

Hi @MohammedFadin ,
Thank you very much for reporting the issue. I can replicate it and it seems it happens when you deploy the Chart as root. But also I can see it doesn't appear when using docker-compose as root.
We will investigate the issue and let you know about the workaround.
Again, thank you for reporting.

Hi @MohammedFadin ,
Is there any reason you have to run Redis as root?
You can see in the documentation the following:

Redis does not requires root privileges to run. It is recommended to run it as an unprivileged redis user that is only used for this purpose.

Thank you @miguelaeh !
Our Volume Storage requires root user privileges only to read and write.

Hi @MohammedFadin ,
When that PR will be merged it should solve your problem.
Thank you again for reporting.


Update Site for 3.0.0 Release

Update http://www.mikekoch.me/ets2-mobile-route-advisor for the 3.0.0 release

I have a problem with the translation in the new version. In place of TimeRemaining does not change the values translated.

@raflix261 , please create a separate issue for your problem, as this is not related to this current issue.


Extending support to Mongoid 5

Hey guys! I've made the changes suggested by @tboerger. The tests are all green and this should extend support to Mongoid 5 version

:+1: so it looks like it is compatible to mongoid 5

Awesome! Please merge them in that case. And thanks for your help. Really appreciate it

@Mohakjuneja waiting also for a response of @johnnyshields as he's still actively using this gem ;)

Great, nice job on your first PR @Mohakjuneja !

Thank you @johnnyshields :) All credit to you guys!


AddingCasAuthInfoDAO

Required for seed project.
Also fixed a bug with correct username retrieval.

This would pull the CAS module if a user selects the persistence module, even if CAS isn't needed. I'm not sure what the best approach would be. Please give me some days.
But please could you submit a pull request, which fixes only the bug with username retrieval you have encountered.

Perhaps if the persistence guice module could be overriden, or perhaps made private and explicitly included?
I could then build my own DelegableAuthInfoRepository in the seed project.

I think we should remove the automatic binding for the persistence memory module. Then everyone can bind the DAO's as one likes. I would provide a default Guice module in every SBT module which serves as an example or which can be used standalone. What do you think?

Is that to say that each Guice module would become private to each project, it would then be used thus:
install(new PersistenceModule())
?

No, it's not private. I'm not sure if private makes sense, because then nobody can use it.

I've removed the default binding #469
Now this pull request should be divided into two parts. The first part should fix the bug with the username retrieval you have encountered. The second should create a new module called silhouette-cas-persistence-memory which contains the persistence stuff. What do you think?

Apologies I wasn't clear I mean a PrivateModule:
https://google.github.io/guice/api-docs/latest/javadoc/index.html?com/google/inject/PrivateModule.html
Unless I've misunderstood how this works.

I think this is to confusing for the most of the users.

@arjunsol Any news here? I plan to release an RC1 at the beginning of the next week, and it would be great if we can get this merged.

I'll add the bug fix in a separate pull request today.
I have my doubts on creating a module to contain a few classes, for one it's not really scalable should other providers need to be added. It feels like it's a case for MultiBinder.
The memory module adds all default DAO's to the binder.
DelegableAuthInfoRepository then has Set[InMemoryAuthInfoDAO] injected in its constructor.
Any project using the persistence projects can add to the multibinder as needed.
Alternative the cas project could supply the DAO in the same fashion.
Thoughts?

@arjunsol The MultiBinder sounds great!

Yes I thought so, ok, I'll submit two pulls. One bug fix and one for swapping to MultiBinder. Haven't done a PoC yet but it looks like it would work.

@arjunsol Any news? I'm ready to release the RC1!

I've been looking at this today, having trouble getting the MultiBinder to work with the generics in DelegableAuthInfoDAO
I can get it to compile using:
ScalaMultibinder.newSetBinder[DelegableAuthInfoDAO[_]](binder)
However Guice then fails to inject, even if I use @Named

Please let us remove the provideAuthInfoRepository method form the PersistenceModule and let us provide only the bindings for the DAOs. So the user can decide which DAOs he need and provide a binding for the AuthInfoRepository in the main module of the application.
The memory modules are anyway placeholder modules which gets replaced in real applications.

OK, that makes sense.

If you cannot find the time I could create the pull request, if this is OK for you! What do you think?

See #477

Apologies I have been away for the past week. I see you made the change, looks good. I'll keep an eye out for RC1 as I see the seed is using it. Then I'll add CAS to the seed project.

I've removed all the in memory based modules. Please see #479 for a detailed explanation.
I've already released RC1 and I've also adapted the seed templates! An announcement comes in the next few days.

Right you are, I'll add the CAS provider to the seed.


Update README.md

Chuyển code sang dạng .mobileconfig

AimLockSettings

Enabled

FOV
90
Smoothness
5
TargetPriority
closest

Targets

Type
enemy
Distance
100
Weapon
M1887

Type
enemy
Distance
100
Weapon
UMP

Type
enemy
Distance
100
Weapon
MP40

Actions

Type
aimAssist
Duration
0.1
Weapon
M1887

Type
aimAssist
Duration
0.1
Weapon
UMP

Type
aimAssist
Duration
0.1
Weapon
MP40

Type
triggerBot
Delay
50
Weapon
M1887

Type
triggerBot
Delay
50
Weapon
UMP

Type
triggerBot
Delay
50
Weapon
MP40

Mới chuyển code sang dạng code c#

Player

weapons

name
UMP

name
MP40

name
M1887

name
SVD

name
DESERT EAGLE

name
MP5

name
M500


mdns_scanner crash if started without any network

When I am starting mDNS search without any connected network I am expecting it to return an empty result.
But I am getting crash that is not being handled.
Log from my program (running on dart native):
Unhandled exception:
SocketException: Send failed (OS Error: Network is unreachable, errno = 101), address = <IP_ADDRESS>, port = 5353
#0      _NativeSocket.send (dart:io-patch/socket_patch.dart:1213:34)
#1      _RawDatagramSocket.send (dart:io-patch/socket_patch.dart:2450:15)
#2      MDnsClient.lookup (package:multicast_dns/multicast_dns.dart:225:14)
#3      MdnsScanner._findingMdnsWithAddress (package:network_tools/src/mdns_scanner/mdns_scanner.dart:71:54)
<asynchronous suspension>
#4      MdnsScanner.searchMdnsDevices (package:network_tools/src/mdns_scanner/mdns_scanner.dart:41:29)
<asynchronous suspension>
#5      CompaniesConnectorConjector.searchAllMdnsDevicesAndSetThemUp (package:cbj_hub/infrastructure/devices/companies_connector_conjector.dart:144:43)
<asynchronous suspension>

Handling this error inside package can hide the fact that app is not connected to network. Empty Result can also mean that there is no device and you are connected to network. Empty result can be ambiguous in internet issue case.

Yeah just rethrow it. Developers will handle it.

I think no need to catch and rethrow if we are not doing anything extra in handling of exception. Users can themselves catch it.


fix: upload svg conformation

fix #649

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


submit with or without fence will use same amount of time

I found submit with or without fence will have different performance on windows or android device. But on mac, two of them is same, Is that because metal make no difference for that? all command buffer in metal can wait until complete.

Do you have further details? Between what points are you measuring performance times? Or are you talking about just frame-rate? And if so, without a fence, how are you guarding resources, etc?

I use https://github.com/google/uVkCompute/blob/main/uvkc/benchmark/dispatch_void_shader.cc to test submit with fence. I found when I remove fence, the submit will be more faster. I can use next submit to wait this submit by using semaphore.


chore(dev-deps): update ledger-cosmos-go from v0.12.2 to v0.12.4 and ledger-go from v0.14.1 to v0.14.3

Please see:

https://github.com/cosmos/cosmos-sdk/issues/18319
https://github.com/cosmos/cosmos-sdk/issues/18423#issuecomment-1804129041

@jkilpatr

Hey thanks for creating this patch! I'll review it tomorrow.


RuntimeError when following the Hello World tutorial

I'm going through the Hello World tutorial and I've literally just copied and pasted all the sample code. I'm just trying to get something up and running, but I'm getting the error in the attached picture when I open the project on my local host.
Here's what I've done:

All the sample code is copied and pasted
wasm files are in the root directory
I'm trying to load an ifc from a folder called "models" which is in the root directory
I've tried it on Chrome V104 and Edge V104
I'm running the latest version of NodeJS

::take

🟢🟢🟢

::take

Hi, @aka-blackboots! Thanks for taking this bounty! The due date is December 19, 2022 UTC.
If you need to submit some pull requests (PR) to complete the tasks, make sure that the last and only the last PR has a title that either starts with the bounty ID or is exactly the same as the bounty name. After the PR is merged, this bounty’s status will automatically changed to done.
If you do not need to make a PR, tell the manager @agviegas to run ::done command after your tasks is confirmed to be done.
Good luck!

::drop

🟢🟢🟢

::drop

Hi, @aka-blackboots! Thanks for giving it a try! It’s now once again available for anyone to take.

::take

🟢🟢🟢

::take

Hi, @rrdls! Thanks for taking this bounty! The due date is December 27, 2022 UTC.
If you need to submit some pull requests (PR) to complete the tasks, make sure that the last and only the last PR has a title that either starts with the bounty ID or is exactly the same as the bounty name. After the PR is merged, this bounty’s status will automatically changed to done.
If you do not need to make a PR, tell the manager @agviegas to run ::done command after your tasks is confirmed to be done.
Good luck!

Hello, @agviegas  !
I tried to reproduce the reported error and this error is occurring because the user put the directory of the 'wasm' files incorrectly or put the wrong 'wasm' files. The directory should be entered in the following line of code shown in the "Hello World" example:
ifcLoader.ifcManager.setWasmPath("static/wasm/");
Could I have more information about the code used?

::done

🟢🟢🟢

::done

Hi, @rrdls! Thanks for your contributions! Please submit an expense to IFC.js Open Collective. Then, tell us the invoice number via the ::expense::_____ command (replace the _____ with the invoice number).

::expense::112800

🟢🟢🟢

::expense::112800

Hi, @rrdls! Thanks for the confirmation! We’ll proceed to review the expense. Once it’s approved, the payment will be scheduled.


Add YAML files re-writing upon cluster setup

Previously, we stored Knative/Istio YAML files with our modifications in place, which is prone to errors. Hence, I wrote a generator, which is based on YQ YAML processor, that applies various updates to YAML files. For example, setting CPU and memory quotas, nodeSelector, CIDR.
More details can be found in the vHive PR, which contains the pre-requisite changes.

Replaced by merged PR #320.


python38Packages.mautrix: 0.10.2 -> 0.10.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.mautrix is: "A Python 3 asyncio Matrix framework."
meta.homepage for python38Packages.mautrix is: "https://github.com/tulir/mautrix-python"
meta.changelog for python38Packages.mautrix is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.3 with grep in /nix/store/q1vrwdl8d1rkl1mcbqiqji3w2jk719rq-python3.8-mautrix-0.10.3

found 0.10.3 in filename of file in /nix/store/q1vrwdl8d1rkl1mcbqiqji3w2jk719rq-python3.8-mautrix-0.10.3

directory tree listing: https://gist.github.com/a1bf3c11e9847eb2bf3eb0f13600aae9

du listing: https://gist.github.com/f3183bcc907cbd2c48976b4f0801250b

Rebuild report (if merged into master) (click to expand)

22 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mautrix-signal
mautrix-telegram
python38Packages.mautrix
python38Packages.mautrix-appservice
python39Packages.mautrix
python39Packages.mautrix-appservice

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q1vrwdl8d1rkl1mcbqiqji3w2jk719rq-python3.8-mautrix-0.10.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.mautrix https://github.com/r-ryantm/nixpkgs/archive/2f83defdc64c529534d481dd4ae6a0f41dce76c6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q1vrwdl8d1rkl1mcbqiqji3w2jk719rq-python3.8-mautrix-0.10.3
ls -la /nix/store/q1vrwdl8d1rkl1mcbqiqji3w2jk719rq-python3.8-mautrix-0.10.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

mautrix-signal
mautrix-telegram
python38Packages.mautrix
python39Packages.mautrix

Maintainer pings
cc @nyanloutre @ma27 for testing.

Result of nixpkgs-review pr 134114 at 2f83defd run on aarch64-linux 1

4 packages built successfully:

mautrix-signal
mautrix-telegram
python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)


How to start and stop WinAppDriver programmatically using java

How we can start and stop WinAppDriver programmatically using java

Hi @surajwaits
I'm not an experienced Java developer, but here's what I did for my toy project.
// START
ProcessBuilder pBuilder = new ProcessBuilder("PATH_OF_WIN_APP_DRIVER");
pBuilder.inheritIO();
Process p = pBuilder.start();

// STOP
p.destroy