check mode for ovirt-cluster-upgrade does not work

ansible-playbook upgrade.yml -e "host_names=[slot-5a,]" -C
 [WARNING]: provided hosts list is empty, only localhost is available

PLAY [oVirt infra] *************************************************************************************************************************************************************************************************

TASK [ovirt-cluster-upgrade : set_fact] ****************************************************************************************************************************************************************************
ok: [localhost]

TASK [ovirt-cluster-upgrade : Login to oVirt] **********************************************************************************************************************************************************************
ok: [localhost]

TASK [ovirt-cluster-upgrade : Get hosts] ***************************************************************************************************************************************************************************
skipping: [localhost]

TASK [ovirt-cluster-upgrade : Check if there are hosts to be updated] **********************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"failed": true, "msg": "The conditional check 'ovirt_hosts | length == 0' failed. The error was: error while evaluating conditional (ovirt_hosts | length == 0): 'ovirt_hosts' is undefined\n\nThe error appears to have been in '/tmp/ovirt-ansible/roles/ovirt-cluster-upgrade/tasks/main.yml': line 26, column 5, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n  - name: Check if there are hosts to be updated\n    ^ here\n"}

TASK [ovirt-cluster-upgrade : Logout from oVirt] *******************************************************************************************************************************************************************
ok: [localhost]
        to retry, use: --limit @/tmp/ovirt-ansible/upgrade.retry

PLAY RECAP *********************************************************************************************************************************************************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1

The problem is that the host_facts module doesn't support check_mode, so we need to add check_mode: no for that task (and any other facts task).
- name: Get hosts
  ovirt_hosts_facts:
      auth: "{{ ovirt_auth }}"
      pattern: "..."
  check_mode: no


Artifactory Gradle Task

Similar to our offerings for NPM, Docker, Maven, Go, etc, it would be ideal to have a Gradle Task as well to allow for the gathering of Build Info on Gradle builds.
Since the JFrog CLI offers build commands for Gradle, and being that the Tools Task takes care of the Maven Extractor dependency, this would fit in really well with the existing tasks.
Currently the workaround is to create a POM from the Gradle build and use the Maven Task to collect dependency information.

@AngelloMaggio,
the Artifactory Gradle task is included in the last release 1.9.0.
You can find the documentation here.
We'll appreciate your feedback for that.

@yahavi does your artifactory gradle task work with java11/ubuntu-latest for you?

@PapaWhiskey65 I am struggling with the same issue and can't seem to find any helpful examples online.

@PapaWhiskey65 I am struggling with the same issue and can't seem to find any helpful examples online. Have you managed to deploy an artifact using the Gradle task?

i was able to upload via gradle, the integration with TFS/azure never seemed to work, and another engineer came up with how to upload via TFS

I actually managed to deploy an artifact using the Artifactory Gradle Task. I had to set the artifactory plugin in my project build.gradle and use a system variable in the azure pipeline to access credentials which kinda makes the Gradle Task redundant and could have been done with a simple Gradle Task but still if someone is interested how I set up the task:

@yahavi How can I define a snapshot repository besides a releases repository? and how to define more than one repos ?


Update the npm_import documentation to have the correct self-reference.

See the reference just above, it has one less underscore.

Type of change

Documentation (updates to documentation or READMEs)

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

I believe it's actually the opposite and it should be two. Want to update the one above to two?

Done.

@kylepl looks like that file is generated and you need to change it at the source, see npm/private/npm_import.bzl

Hm, it seems like the source actually has npm__at_types_node__15 (i.e. double underscores twice). So I'm not sure why this does not match then. If there are steps that generate it, it's likely more involved then I'm willing to patch.

I think I see the bad one in the bzl file, there's also a few that correctly have the double underscore so you might just be looking at the wrong one?
If it's too much work I can try but it would be great if you could give it another shot since you have this going already 👍


Container Generator

This PR adds the long-awaited apiato:generate:container command that allows you to fully automatically create a container with all major features..

Configuration file
Migration File
Model and Migration
Routes and Requests
Actions and Tasks already implemented
Container that wires everything together..

Wooooooooooooooooooooooooha 🎉

The conflicted files

Generator/Commands/JobGenerator.php
Generator/Commands/MailGenerator.php
Generator/Commands/SeederGenerator.php
Generator/Stubs/job.stub
Generator/Stubs/mail.stub

can be resolved by using my files!
The GeneratorsServiceProvider i don't know?! Need to take a look at it..

oh yes.. and the GeneratorsServiceProvider must be overwritten with my file as well .. Sorry for that..
I changed the way the Generators are loaded (now by ::class and not name any more!)


chore(deps): upgrade to xUnit 2.1.0

chore(tools): add xUnit console runner
chore(deps): upgrade xUnit to 2.1.0
chore(deps): upgrade Shouldly to 2.8.2
chore(tools): upgrade solution to VS 2013

There's a minor package restore issue as part of the build process (which seems to then trigger a MSBuild warning): https://ci.appveyor.com/project/shiftkey/reactive-eventaggregator/build/1.1.37#L33
Not sure if you're worried about that.

Retitled PR, as the main need was to actually be able to run tests from console. Upgrading xUnit was just a needed step in the middle.

Re. the restore issue, I think it is due to the included nuget executable in .nuget folder, which by now is an old version.
The real solution to that would be to drop local nuget.exe and .nuget folder altogether, as that approach is discouraged by now. Don't remember though what happens to build.ps1.
If it's ok for you, I'd go for a merge, and possibly fix this issue in a separate commit/branch from you/my team/whoever.

@BrainCrumbz thanks for this!

This PR is just instrumental to the one we were mainly interested in, about upgrading Rx. Which is under way.


How to get the feedback of Husky velocities?

Greetings!
We send velocities commands to Husky within cmd_vel. How we can get feedback of the measured velocities of Husky? I mean the real velocities of the robot and not we have sent to it.

You can get the angular velocity of each wheel from the /joint_states topic, which can be converted to the linear velocity by multiplying by the radius of the wheel.  On a standard Husky, the wheel radius is 0.1651m (see `husky_description/urdf/husky.urdf.xacro, line 78).  Obviously changes in tire pressure will affect the real-world radius, but this should give you a good approximation.
The joint_states topic will contain information like this:
header:
  seq: 3953
  stamp:
    secs: 1377
    nsecs: 860000000
  frame_id: ''
name: [front_left_wheel, front_right_wheel, rear_left_wheel, rear_right_wheel]
position: [19.085047243226207, 19.010108967194768, 13.544983137346293, 13.374554534481813]
velocity: [0.5672939631822598, 0.5672830724011997, 0.18267799254683934, 0.11022604243897091]
effort: [0.0, 0.0, 0.0, 0.0]

In the case above I am publishing the linear-x velocity of 0.1 m/s.  Multiplying 0.5672939631822598 rad/s by 0.1651 m gives 0.09366023332139109 m/s, which is within a rounding error of the published velocity.
Depending on the surface the robot is driving on there may also be external influences when it comes to converting the angular velocity to linear.  For example, loose sand or slippery surfaces may cause the robot's ground-speed to be lower than what the wheel velocities would indicate.
If you need to calculate the robot's speed relative to the environment more accurately than you can using the angular velocities of the wheels you may want to look into equipping the robot with a 2d lidar sensor and using some sort of SLAM system to calculate its speed using sensor data.

@civerachb-cpr Thank you for the detailed answer.


Add kernel modules to support the NFS4 clients

We cannot use the NFS client to share files between containers.
nginx  mount.nfs: Protocol not supported
grafana  mount.nfs: Protocol not supported

Could you please add NFS4/3 client modules similar to Jetson AGX and Xavier? https://github.com/balena-os/balena-jetson/pull/354
The concept is explained in the article: https://www.balena.io/blog/using-nfs-server-to-share-external-storage-between-containers-balena/

Adding them in https://github.com/balena-os/balena-jetson-orin/pull/9

@acostach Thank you, I have built our own image base on #9 and flashing it now to test.

Perfect, let me know if those configs are fine

@acostach
I did my own build yesterday based on the master and today based on #9. Looking at the forum and around, I followed three steps (Build, Configure and Flash). After starting device provisioned itself in the balenaStaging and downloaded the release.
Everything seems to be fine, but I can't connect to the Host OS, and it's unable to start. Any pointers on what I am missing?

Build

./balena-yocto-scripts/build/barys

Configure

BALENARC_BALENA_URL=balena-staging.com  /app/balena-cli/balena os configure --fleet orins balena-image-jetson-agx-orin-devkit-20221026121045.rootfs.balenaos-img

Flash

$ ./bin/cmd.js -f ../balena-image-jetson-agx-orin-devkit-20221026121045.rootfs.balenaos-img -m jetson-agx-orin-devkit


Spyder 5.4.5 release

This release is necessary to restore compatibility with Python 3.7, which was broken on PR #20970 by using a language feature that was added in Python 3.8.
The compatibility was restored on PR #21290.

@dalthviz, you need to checkout the new 5.4.x branch and do the release from it. After that, you need to merge that branch with 5.x and master.

Closing this to proceed with the release


Create a Security Policy

Hi, I've noticed that the security policy got lost in this recent changing in the repo. I believe it was using the github standard security policy before https://github.com/github/.github/blob/main/SECURITY.md.
Since this is an important document to ensure a safe and timely vulnerability report process, I'd like to suggest to define one for jakechampion/fetch repository.
There are a few ways to receive such disclosures:

have an email or website available to receive such reports (such as opensource-security[@]github.com); and/or
use GitHub's private vulnerability reporting feature.

Let me know if a PR suggesting a Security Policy is welcome and I'll submit it ASAP.

@joycebrum I'd be happy to accept a PR for such a document 👍


cacti train fails with WML error

When training, cacli fails with the below error
error Unable to get /v3/wml_instances/33b5e230-4230-4516-8d87-735d71b70f34 [400]: {"trace":"b401ace168081dfe7f3e517551f7b550","errors":[{"code":"invalid_request","message":"Instance in v2 plan should use v4 API."}]}

WML isn't supported at the moment, because of updates to the API, you can try Google Colab as an alternative for now


Add more features to 2024-03-01 which are same with 2023-05-02-preview

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

All changes are from previous api version 2023-05-02-preview, which includes:
https://github.com/Azure/azure-rest-api-specs/pull/24286
https://github.com/Azure/azure-rest-api-specs/pull/24459
https://github.com/Azure/azure-rest-api-specs/pull/24822

/pr RequestMerge


WS Server error, Julia 3.7, Fedora 21

Please, i need some help to with the server.
I got the following message after run a copy from the example code from the docs.
Can' t fix it, dont know whats wrong..
WARNING: Dict-based `@docstring` config is deprecated. Use keywords instead.
Listening on 34440...

signal (11): Segmentation fault
unknown function (ip: -1728454171)
write at /usr/bin/../lib64/julia/sys.so (unknown line)
write at ./iobuffer.jl:235
Segmentation fault

I am running Julia v.3.7 with Fedora 21
Thanks!

Much has changed since Julia 3.7. If you can reproduce with Julia 0.6, please include more information and we'll try to help. Closing this for now.


Assignee incorrectly set

Tool version
3.0.109
Describe the problem
In Jira (cloud) the assignee for the issue is Unassigned
In its history it was assigned to somebody but final state Unassigned
When I export it I get:
[E][12:07:34] Error mapping field 'System.AssignedTo' on item 'US-23770'.
[System.NullReferenceException] System.NullReferenceException: Object reference not set to an instance of an object.
at JiraExport.JiraMapper.MapFields(JiraRevision r):
at JiraExport.JiraMapper.MapFields(JiraRevision r)
When I import it, it has a value for Assigned To of the person who it was last assigned to
Attachments
jira-export-log-230306-120731.txt
wi-import-log-230306-120924.txt
config-agile-json.txt
Please attach the following files:

config.json
jira-export-log.txt
wi-import-log.txt

Screenshots
If applicable, add screenshots to help explain your problem.

Labeling as bug

looks like it is fixed with version 3.0.115?

@RoyalWulf I think to too.
Ran some tests for an Agile project and can confirm that the issue no longer occurs:

Let's close this one?

Issue fixed


Throw container StartExceptions instead of logging them and moving on

This change makes it so that any exceptions that come from WildFly
during container start get thrown by RuntimeServer#start instead of
just being logged by WildFly. This is especially useful in testing
scenarios where tests do container.start().stop() since that kind of
test can't otherwise detect various kinds of errors upon startup.
We may want this to just be enabled with some special testing flag, or
maybe we'd always like to throw the exception on start since Swarm is
used in more embedded scenarios. It's up for debate, so discuss before
merging.

I think it makes sense to have it active all the time


Cosmetics option disappeared?

Hi, me and my friends recently encountered an issue with the mod (I believe it is due to the recent update) where we are now unable to adjust our cosmetics from the main menu. Is this intended or is this a bug? I tried running Lethal Company with only this mod installed and it still doesn't even show the More Company title screen.
Thank you in advance, I really love this mod!

That'd likely mean that your mods aren't loading properly

This happened to me once, deleted the cache folder and that fixed it. You can give that a shot if you haven't already fixed it.


Add piece loading indicator

Need to display when pieces are loading vs playing.

Is help wanted on any of these issues?

@cdtinney I appreciate you're willingness to help. I'll be tagging any issues with the "help wanted" label if that's the case, which probably won't be very often for the time being. See CONTRIBUTING.md.


Improve GitHub Actions for pull requests

See https://github.com/Senzing/community-roadmap/issues/71

Pull requests:

https://github.com/Senzing/github-action-add-issue-to-project/pull/16


⚠ Bofrost.at has degraded performance

In 0f34171, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1472 ms

Resolved: Bofrost.at performance has improved in cbfba99 after 37 minutes.


Feat/retrieve pdbs

What type of PR is this?
Feature
What does this PR do / Why do we need it?
This PR introduces a script to review the deployed PodDisruptionBudgets (PDBs) across cluster namespaces.
It includes a "restrictive" option to retrieve only the restrictive PDBs in all namespaces.
Which Jira/Github issue(s) does this PR fix?
N/A
Special notes for your reviewer
This script is a part of the Cluster Health Check steps.
Pre-checks (if applicable)

[X] Validated the changes in a cluster
[X] Included documentation and examples

@danifernandezs did you have any further updates regarding this comment?

Hi @hectorakemp & @bmeng
The use cases of the loop that I thought of are.

If we (MCS) don't declare any namespaces, we only check those from openshift-* (I need to loop with the retrieved namespaces), to avoid PDBs from customer namespaces.
Show/Print PDBs in the same way as another script (the event one)

Hi @hectorakemp & @bmeng
The use cases of the loop that I thought of are.
* If we (MCS) don't declare any namespaces, we only check those from openshift-* (I need to loop with the retrieved namespaces), to avoid PDBs from customer namespaces.

* Show/Print PDBs in the same way as another script (the event one)

Thanks for the explanation! I think it is reasonable to loop through all the openshift-* namespaces. The number of openshift-* is not large so the api requests should be manageable.
For the "--all", I think it makes more sense to do a oc get pdb -A so that it only needs one single API request. Also I left a comment above regarding getting the pdb data in customer namespaces. I would like to know your thoughts.
Thanks!

I can modify the "all namespaces" option to use the -A flag.
The motivation behind individual API requests is to keep the output format consistent with the rest of the options.

/hold

Hi @xiaoyu74
Why this PR was flagged to be on hold???

@jaybeeunix Could you help take a look of this script from compliance pov? Thanks.

/unhold
But it needs a review from security team

@sam-nguyen7 regarding the customer data concern, what about the approach of,

If that's for health check, I think it is better to only output relevant data, eg, only output "It has bad PDBs in customer namespace"/"All PDBs look good in customer namespace".

In this case, the script will read the PDB customer namespace, but it won't output sensitive data of customer namespace.

@jaybeeunix @sam-nguyen7 - Any feedback regarding the security concerns?

@jaybeeunix @sam-nguyen7 - Any feedback regarding the security concerns?

@xiaoyu74 Sorry for not getting back to you sooner. Looking at this closer and at more osdctl logic, I'm +1 to reading the PDB in customer's namespace AND the name of the PDB as I think this will give better support. For compliance, we can justify this as an approved and procedural part of our operations on a given cluster through our tooling.

lifecycle/stale

/stale

I’m closing this pull request as no decision has been made after a year.
Additionally, I no longer work at Red Hat, so merging this code is no longer relevant to me.
Thank you for the opportunity to contribute.


enable/disable task

User savor wrote the following:
I'm trying to enable/disable scheduled task through ASP application like this:
TaskEditDialog editorForm = new TaskEditDialog(ts.RootFolder.Tasks[taskName], true, true);
editorForm.Task.Enabled = true;
but it doesn't enable the task; am I missing something?

Originally posted: 2011-07-12T09:20:59

For an ASP application, you cannot use the Windows Forms available in the TaskSchedulerEditor assembly (like TaskEditDialog). You should use the TaskScheduler assembly by itself.
To enable a task you only need a few lines of code:
Task t = ts.GetTask(taskName);
if (t != null)
{
t.Enabled = true;
t.RegisterChanges();
}
Originally posted: 2011-07-12T09:25:49

Thanks i just tried that however  on the line
t.RegisterChanges(); I get following exception:
Trigger.Repetition.Interval must be less than Trigger.Repetition.Duration under Task Scheduler 1.0.
here is what i'm doing:
TaskService ts  = new TaskService(ConfigurationManager.AppSettings["server1"],
                                     ConfigurationManager.AppSettings["userName1"],
                                     ConfigurationManager.AppSettings["domain"],
                                     ConfigurationManager.AppSettings["password1"], false);
Microsoft.Win32.TaskScheduler.Task t = (Microsoft.Win32.TaskScheduler.Task)ts.GetTask(taskName);
if (t != null)
            {
                t.Enabled = false;
                t.RegisterChanges();
            }
Originally posted: 2011-07-12T10:10:25

I believe you are using an older version of the library that had a bug. Please make sure you are using the 1.6.2 release.

Originally posted: 2011-07-13T11:20:00

I try to disable an existing schedule in Task Scheduler using these lines of code:
using (TaskService ts = new TaskService())
                {
                    Task task = ts.GetTask("TaskName");
                    task.Enabled = false;
                    task.RegisterChanges();
                }
However, it givers an error at line task.RegisterChanges().
COM exception was handled.
The task has been configured with an unsupported combination of account settings and run time options. (Exception from HRESULT: 0x80041314).
This scheduled runs under my user account.

Originally posted: 2011-07-14T19:52:53

What version of Windows are you using? If Vista or later, will you export the XML for the task and post it so I can see the properties you have set that may be causing the error?
Originally posted: 2011-07-15T13:27:40

Hi,
I use Windows XP. It can work when I run these lines of code in Windows 7, but cannot in Windows XP with td.Principal.LogonType = TaskLogonType.InteractiveToken;
By  the way, in Windows 7, I can not set td.Settings.RunOnlyIfLoggedOn =  false; with any td.Principal.LogonType. Can you give me how to create a  task in Windows 7 ( or all version of Windows) that can run when users  log off, because I need the task can run when they log off.
Thanks
Originally posted: 2011-07-16T02:28:12

You are correct, TaskLogonType.InteractiveToken only works on Vista and later. TaskDefinition.Settings.RunOnlyIfLoggedOn is only available if you force version 1 or are on XP and earlier.
However, I know of no way to trigger a task when the user logs off using the Task Scheduler.
Originally posted: 2011-07-18T12:43:26


Free text info from DrugBank

DrugBank has a bunch of extra information on their website; stuff like mechanisms of action:

I'm wondering if: a) this information exists in some downloadable part of DrugBank or b) if there were particular barriers to including this info when DrugBank was ETL'd for KG2.

Hi @dkoslicki, I think you are able to extract this information from this downloadable XML file, which needs the license to download.

Thanks @chunyuma , Mohsen and I found this and are undergoing the process of NER, identifier extraction, and alignment from all the fields in these. I'm optimistic this can be useful for future xDTD kinds of efforts

Sounds great! Happy to see this will be useful. One difficult thing I saw using this information is the mapping between this text and the node in KG2, and there are the fewer connections in KG2 between the mapped nodes. But I would be happy to share my experience with Mohsen if needed.

It does look like this information is in the DrugBank XML download:
  <mechanism-of-action>Eliglustat is a glucosylceramide synthase inhibitor used for the treatment of type 1 Gaucher disease.[L41404] Gaucher disease is a rare genetic disorder characterized by the deficiency of acid β-glucosidase, an enzyme that converts glucosylceramide (also known as glucocerebroside) into glucose and ceramide. In patients with Gaucher disease, glucosylceramide is accumulated in the lysosomes of macrophages, leading to the formation of foam cells or Gaucher cells.[L41404] Gaucher cells infiltrate the liver, spleen, bone marrow and other organs, leading to complications such as anemia, thrombocytopenia and hepatosplenomegaly.[L41404,A246384]

Eliglustat reduces the production of glucosylceramide by inhibiting glucosylceramide synthase, a rate-limiting enzyme in the production of glycosphingolipids.[L41404,A182192] This lowers the amount of glucosylceramide that is available in lysosomes, and balances the deficiency of acid β-glucosidase.[L41404,A246384]</mechanism-of-action>

I will have to brainstorm how to ETL it since the information inside of the free text is not linked to any other node IDs. I assume you want this information converted into edges?

Hi @ecwood, I agree, to convert this information into edges, we need to first resolve the mapping issue. The second issue we may need to address is extracting the correct relation logic from the free text. I have known that this is not easy because SemMedDB failed in some cases. Perhaps LLM is an option because it is smarter than the algorithm used in SemMedDB.

Unless there is a large appetite to do this, I've done this on the side/locally (to use for Pathfinder training), so at least for my purposes, I have what I need. I'd want to hear from the KG2 team if they think this is worth pursuing otherwise


Receiving 429 - Connection count reaches limit.

Connection count:
Server: 30 - Client: 0
Even when scaling up..

When you scale up, there is a DNS cache pointing to the old free tier service. So, wait a few time and then the quota will change.

That didn't help but last Friday afternoon this suddenly stopped and all was working again.


Partial validation with required-not-nullable values

There might be something i am missing but when i compare these two scenarios of posting the same empty model, i would expect them to return the same ModelState, but they don't:
    public class TestModel
    {
        [Required]
        public Guid Id { get; set; }

        [Required]
        public string Name { get; set; }
    }

... here i get a required validation error for each property.
    public class TestModel
    {
        public Guid Id { get; set; }
        public string Name { get; set; }
    }
    public class TestModelValidator : AbstractValidator<TestModel>
    {
        public TestModelValidator()
        {
            RuleFor(x => x.Id).NotEmpty();
            RuleFor(x => x.Name).NotEmpty();
        }
    }

... here i get a validation error only for the Id-property. What i would expect is to get validation errors for both properties (the same happens with .NotNull() for Name). Is there something i'm missing?

After reading a bit more, I think this is the same as #127, which means that the problem is MVC-related. I'm a bit curious though to why using [Required] works as opposed to NotEmpty()?

Yes, this is a limitation of MVC's validation infrastructure.
The difference is because DataAnnotations works by validating each property in turn, but FluentValidation expects all the properties to be set first, and then validation is run only once over the entire object. MVC's validation infrastructure doesn't work properly with FluentValidation's approach - it generates the required message for non-nullable value types, but then cancels all object-level validation (so FV never actually runs).

Thanks for a quick reply! That's good to know. Then i can either change the offending properties to be nullable or perform my own validation later (after modelbinding).


Hospitalization rate (vaccinated indiv.) not working

When setting the hospitalization rate to zero, this doe not affect the hospitalizations. The problem seems to be with the hospitalization rate for the vaccinated individuals only (to double check).

It works fine. @davidchampredon was confused! The parameter to play with for the hospitalization rate of vaccinated individuals is  vaccine effectiveness against hospitalization!!!


Best Proxy List to Use

Whats everyones Proxy List you guys are using?

I'm closing this issue because the project is currently not maintained and it will unlikely to be resolved.
For more, please read this: https://github.com/ohyou/twitch-viewer/blob/master/README.md#project-state


SnapshotNow was removed

SnapshotNow was removed from the Postgres codebase in August 2013.
<EMAIL_ADDRESS>This stops pg_schema_triggers (and CartoDB) being used with anything over 9.3.6 (barring other maintenance releases in the 9.3.x series)
Is there a plan to upgrade the extension?

I think the short term plan is to remove the dependency with this extension, we are not using it.
cc @pramsey

Remove the dependency on SnapshotNow from the extension, or remove the dependency on the extension from CartoDB?

remove the dependency on the extension from CartoDB

that's it, we are not actually using it

Given the future utility (allowing DDL by users without breaking cartodb), should I consider fixing the extension?

yep, fixing the extension would be good but removing it for the moment would solve lot of problems. I don't know if postgres is planning to support this builtin.
Could you take a look?

There's no real enhancement to event triggers in 9.4, so we'll need the extension there for sure. 9.5 may have some improvements, have to look more closely into it.

Some more background - I'm currently trying to build & run CartoDB on
Windows. Less extensions to build would be a good thing - should I skip
this one?

@voltagex you can skip this extension for sure.
You are brave :)

The current master should be supported by 9.4 and I see no changes in 9.3 that remove SnapshotNow


Adding the LPIPS metrics

I have add the lpips metrice by referring the https://github.com/xinntao/BasicSR/pull/367

editing the init.py and adding the lpips.py file which can be supported by the ymal file to test

@RebornForPower Thanks for the contribution:-)
As LPIPS takes torch tensors as inputs， we may need to directly pass torch tensors into this metric.
Let me think about it and it may involve the modification to the metric mechanism.

@RebornForPower Thanks for the contribution:-)
As LPIPS takes torch tensors as inputs， we may need to directly pass torch tensors into this metric.
Let me think about it and it may involve the modification to the metric mechanism.

why not process the img into tensor in the calculate function?


can't make imports work

Info:
libc++abi: terminating with uncaught exception of type sol::error: sol: runtime error: ...not/IdeaProjects/project-folder/dist/index.lua:6: module 'custom-module' not found:
	no field package.preload['custom-module']
	no file './custom-module.lua'
	no file '/usr/local/share/lua/5.4/custom-module.lua'
	no file '/usr/local/share/lua/5.4/custom-module/init.lua'
	no file '/usr/local/lib/lua/5.4/custom-module.lua'
	no file '/usr/local/lib/lua/5.4/custom-module/init.lua'
	no file './custom-module.lua'
	no file './custom-module/init.lua'
	no file './custom-module.lua'
	no file '/usr/local/lib/lua/5.4/custom-module.so'
	no file '/usr/local/lib/lua/5.4/loadall.so'
	no file './custom-module.so'
	no file './custom-module.so'

// src/custom-module.ts
export const helloWorld = () => 'Hello World!'

// src/index.ts
import { helloWorld } from './custom-module'

const main: IMain = {
  onRender: () => {
    helloWorld()
    console.log('done')
  },
  screen: {
    width: 600,
    height: 600,
    x: 0,
    y: 0,
  },
}

This is how it compiles:
-- dist/custom-module.lua
--[[ Generated with https://github.com/TypeScriptToLua/TypeScriptToLua ]]
local ____exports = {}
____exports.helloWorld = function() return "Hello World!" end
return ____exports

-- dist/index.lua
--[[ Generated with https://github.com/TypeScriptToLua/TypeScriptToLua ]]
local ____exports = {}
local ____custom_2Dmodule = require("custom-module")
local helloWorld = ____custom_2Dmodule.helloWorld
local main = {
    onRender = function()
        helloWorld(nil)
        print("done")
    end,
    screen = {width = 600, height = 600, x = 0, y = 0}
}
return ____exports

Question:
From what I just learned it seems that even though Lua is scripting language - by default it doesn't look for "modules" (required files) inside modules directory (dist) (I can't imagine how is that a good idea like do people really write all of their scripts inside usr/local/lib/lua/5.4/ 🤣)
How do I make this work? Is there some tstl config that I missed? 🙏

I set luaBundle to generate single file but now I get a different error:
"tstl": {
    "luaTarget": "JIT",
    "luaBundleEntry": "src/index.ts",
    "luaBundle": "bundle.lua"
}

[sol2] An error occurred and panic has been invoked: attempt to index a nil value
libc++abi: terminating with uncaught exception of type sol::error: lua: error: attempt to index a nil value

I have this code in C++:
int screenWidth = lua["main"]["screen"]["width"];
int screenHeight = lua["main"]["screen"]["height"];
int screenX = lua["main"]["screen"]["x"];
int screenY = lua["main"]["screen"]["y"];

It seems that TSTL makes the main variable local and inaccessible.
noImplicitGlobalVariables doesn't have any effect. Any ideas how do I access the main variable?

libc++abi: terminating with uncaught exception of type sol::error: sol: runtime error: ...not/IdeaProjects/project-folder/dist/index.lua:6: module 'custom-module' not found:
no field package.preload['custom-module']
no file './custom-module.lua'
no file '/usr/local/share/lua/5.4/custom-module.lua'
no file '/usr/local/share/lua/5.4/custom-module/init.lua'
no file '/usr/local/lib/lua/5.4/custom-module.lua'
no file '/usr/local/lib/lua/5.4/custom-module/init.lua'
no file './custom-module.lua'
no file './custom-module/init.lua'
no file './custom-module.lua'
no file '/usr/local/lib/lua/5.4/custom-module.so'
no file '/usr/local/lib/lua/5.4/loadall.so'
no file './custom-module.so'
no file './custom-module.so'

It lists all the locations it looks for the file. I'm guessing it cannot find custom-module because it is in the dist directory? Make sure you figure out what the root directory of the lua project is where these paths are relative to. LuaBundle would indeed also resolve this issue because there are no more  imports.
I am not sure why the bundle is giving you a nil index error, the code looks alright as far as I can tell. It's hard to see without knowing the line number where the error happens.

I have this code in C++:
int screenWidth = lua["main"]["screen"]["width"];
int screenHeight = lua["main"]["screen"]["height"];
int screenX = lua["main"]["screen"]["x"];
int screenY = lua["main"]["screen"]["y"];

It seems that TSTL makes the main variable local and inaccessible.
noImplicitGlobalVariables doesn't have any effect. Any ideas how do I access the main variable?
Or how do I make the main variable global?

You have to somehow tell TS that this variable exists globally and can be assigned to. You will usually use declarations for this that describe your lua environment. See: https://typescripttolua.github.io/docs/advanced/writing-declarations
I would expect a declaration file like:
declare var main: {
    screen: { x: number; y: number}
}

and code like:
main = { screen: { x: 10, y: 20 } };

See example in playground.


运行报错

~/tools/wechat-exporter on main > python main.py ./foo                                                                                at 19:16:41
Traceback (most recent call last):
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connection.py", line 203, in _new_conn
sock = connection.create_connection(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/util/connection.py", line 85, in create_connection
raise err
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connectionpool.py", line 790, in urlopen
response = self._make_request(
^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connectionpool.py", line 496, in _make_request
conn.request(
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connection.py", line 395, in request
self.endheaders()
File<EMAIL_ADDRESS>line 1281, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File<EMAIL_ADDRESS>line 1041, in _send_output
self.send(msg)
File<EMAIL_ADDRESS>line 979, in send
self.connect()
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connection.py", line 243, in connect
self.sock = self._new_conn()
^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connection.py", line 218, in _new_conn
raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x10474c490>: Failed to establish a new connection: [Errno 61] Connection refused
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/homebrew/lib/python3.11/site-packages/requests/adapters.py", line 486, in send
resp = conn.urlopen(
^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/connectionpool.py", line 844, in urlopen
retries = retries.increment(
^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/urllib3/util/retry.py", line 515, in increment
raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=48065): Max retries exceeded with url: /wechat/allcontacts (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x10474c490>: Failed to establish a new connection: [Errno 61] Connection refused'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/hxzhouh/tools/wechat-exporter/main.py", line 77, in
app()
File "/Users/hxzhouh/tools/wechat-exporter/main.py", line 66, in export_all
all_users = requests.get("http://localhost:48065/wechat/allcontacts").json()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/api.py", line 73, in get
return request("get", url, params=params, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/api.py", line 59, in request
return session.request(method=method, url=url, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/adapters.py", line 519, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=48065): Max retries exceeded with url: /wechat/allcontacts (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x10474c490>: Failed to establish a new connection: [Errno 61] Connection refused'))
mac m1 : 13.4.1 (22F82)

重新安装后，还是报错 ，麻烦op帮忙看看

python3 main.py ./foo
Traceback (most recent call last):

File "/opt/homebrew/lib/python3.11/site-packages/requests/models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/hxzhouh/tools/wechat-exporter/main.py", line 77, in
app()
File "/Users/hxzhouh/tools/wechat-exporter/main.py", line 66, in export_all
all_users = requests.get("http://localhost:48065/wechat/allcontacts").json()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/requests/models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

curl -i http://localhost:48065/wechat/
curl -i http://localhost:48065/wechat/search
curl -i http://localhost:48065/wechat/allcontacts

可以麻烦运行一下这几个命令，然后把结果发过来看看吗？如果真的返回了联系人数据那么务必省略一下。

closed due to inactivity

curl -i http://localhost:48065/wechat/allcontacts

运行了，只有search是工作的，其他都会报错
HTTP/1.1 501 Not Implemented
Server: GCDWebServer
Connection: Close
Date: Fri, 23 Feb 2024 19:09:33 GMT


PHPStan level 9 with bleeding edge

Update PHPStan level to 9/max with bleeding edge. You can't go any higher :-) #172 inspired me to do this, to make sure the types are correct throughout the lib.
Native types could be used instead of docblocks if support for older PHP version is dropped, but first I wanted to make sure the lib is on a highest possible PHPStan level to make adding native types a bit safer. Can add them in a next PR once this is merged for example.
This branch is based on #196 branch to make tests run on all PHP versions, probably should be rebased before merging, can take care of that.
I'm not sure what to do with the failing Style CI test that wants me to update a preexisting code like this:
- exit();
+ exit;

I can update it if it looks like what's expected.

Yes, this would be version 9, @mfn, and we would have to drop support for a bunch of PHP versions, unfortunately.

I'm all for native and strict types (both!) and I see this PR is a prerequisite for doing it.
Seems like dropping 7.1-7.3 (current usage 1.5% combined) support should not be that painful (and even the tests I've added in #196 do not run on 7.1), possibly even 7.4 (3.6%) and 8.0 (5%). Here's the version breakdown of the 8.x version, from https://packagist.org/packages/pragmarx/google2fa/php-stats#8:

I can update this PR and would be nice to merge the others as well :-) Let me know if you need a co-maintainer.

Let's try to keep 8.0 and above for 9.0 and drop support for the 7.4 and prior as 7.4 hit EOL in 2022. I can still support security issues, but not the evolution.

Got it. Do you want to create the 9.x branch so I can target it with this PR?
Would you prefer to add the native types in this PR too?

Branch 9.x created @spaze. I believe you can merge https://github.com/antonioribeiro/google2fa/pull/196 into this one, also merge 8.x back to spaze:spaze/phpstan++ to fix the conflicts with the changes that I merged recently

I just disabled Style-CI too. I will try to move to something else, as I currently prefer Prettier over any other formatter out there...

Rebased and this now targets the 9.x branch


Upgrade MongoDB connector to support latest version of MongoDB

Description
The current MongoDB connector does not support Mongo DB version 5.1.0 onwards. We need to upgrade the mongo db connector to support latest versions.
Affected Component
MI
Version
No response
Related Issues
No response
Suggested Labels
No response

A new version of connector is released to store


Unable to render anything using MDXProvider & MDXRenderer

I can't seem to render anything. I'm rebuilding an existing portfolio site from scratch using gatsby. The data is being pulled from strapi using MDXProvider and is rendered using MDXRenderer. Running this on an Ubuntu 16.04 vps with chromium installed.
strapi entry
```mermaid
graph LR
install[Install Plugin]
install --> configure[Configure Plugin]
configure --> draw[Draw Fancy Diagrams]
```    .

project.js

import { MDXProvider } from "@mdx-js/react"
import { MDXRenderer } from "gatsby-plugin-mdx"

<MDXProvider>
   <MDXRenderer>
       {data.strapiProject.childStrapiProjectContent.childMdx.body}
   </MDXRenderer>
</MDXProvider>

gatsby-config.js

{
    resolve: 'gatsby-plugin-mdx',
        options: {
           plugins: [
            {
                resolve: 'gatsby-remark-mermaid',
            }
        ],
            extensions: [".mdx", ".md"]
    },
}

Using gatsby develop I'm able to render the page without any errors. All the other elements are being shown on the page as intended.
Code block part of the HTML:

        graph LR
        install[Install Plugin]
        install --> configure[Configure Plugin]
        configure --> draw[Draw Fancy Diagrams]

The thing is I have no idea on how to identify any errors on the process. How would I go around identifying the issue on the console?

Apparently I had to include gatsby-remark-images and gatsby-remark-copy-linked-files to the plugins and now it's working as intended.
gatsby-config.js
    {
      resolve: 'gatsby-plugin-mdx',
      options: {
        gatsbyRemarkPlugins: [
        {
          resolve: 'gatsby-remark-mermaid',
        },
        {
            resolve: "gatsby-remark-images",
            options: {
              maxWidth: 1035,
              sizeByPixelDensity: true
            }
          },
          {
            resolve: 'gatsby-remark-copy-linked-files'
          }
        ],
        extensions: [".mdx", ".md"]
      }
    }

Thank you for this plugin, it's amazing!


Update Known-issues.md

removed the add-in limitation which seems to be fixed now.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Hi, @marclauinger - another one for your review.

@LolaJacobsen PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[BUG] GetMapStaticImageAsync

Library name and version
Azure.Maps.Search -1.0.0-beta.3
Describe the bug
An exception is thrown when providing the latitude and longitude as input to the GetMapStaticImageAsync function.
However, the values passed to the ReverseSearchCrossStreetAddressAsync function are functioning properly. Further information is provided below.
GetMapStaticImageAsync  exception
Service request failed.
Status: 400 (Bad Request)
Content:
{"Center":["params are out of range <-180,180> and <-85,85>"]}
Headers:
x-ms-azuremaps-region: REDACTED
X-Content-Type-Options: REDACTED
Strict-Transport-Security: REDACTED
X-Cache: REDACTED
X-MSEdge-Ref: REDACTED
Date: Sat, 25 Feb 2023 14:42:40 GMT
Content-Length: 62
Content-Type: application/json; charset=utf-8
Expected behavior
It should create an image
Actual behavior
Throw an exception
Service request failed.
Status: 400 (Bad Request)
Content:
{"Center":["params are out of range <-180,180> and <-85,85>"]}
Headers:
x-ms-azuremaps-region: REDACTED
X-Content-Type-Options: REDACTED
Strict-Transport-Security: REDACTED
X-Cache: REDACTED
X-MSEdge-Ref: REDACTED
Date: Sat, 25 Feb 2023 14:42:40 GMT
Content-Length: 62
Content-Type: application/json; charset=utf-8
Reproduction Steps
GeoPosition geoPosition = new GeoPosition(37.7749, -122.4194);
Pass GeoPosition to GetMapStaticImageAsync function
it throw an exception
Environment
.NET SDK:
Version:   7.0.200
Commit:    534117727b
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows

@rvinothrajendran We are looking into it and get back to you for any additional information.

@rvinothrajendran I believe there is a limitation on the coordinates of the center point and thus you are getting the error.  Please see below and go through the documentation over here.

Please let me know if you have any questions.


[WIP] Forward takes Variant class

One of the bigger refactors, we introduce a variant class to wrap inputs and outputs in, so you at least get run time type checking of your arguments, much like in Python. One of the things we do here is that the Variant will automatically specialize into a Variable is the function calls for it. I took the stance that if it appears in Tensor/variable.h but not Functions.h, then we should allow the function to be called. I haven't added all the function in yet.
Additionally, it introduces method changing via template magic.
Things that will work:
variant
  .getList()[0]
  .m(at::relu)
  .m(at::mse_loss, other);

Please comment on whether this is better, worse, easier, or harder to use. In general, I noticed that we had to write a little more boilerplate but it makes more intuitive sense to me.

Cool stuff! I have some concerns though. It seems that with this change you are moving all your type safety that C++ provides at compile time, to runtime. This makes for simpler code, like Python, but it also makes the code much less safe, no? Compile time type safety being one of the main advantages provided by C++. It also seems to make the code less expressive. For example this signature:
variable_list foo(Tensor x, float f) now becomes Variant foo(Variant, Variant). Don't you feel we lost something here? Apart from this, if you mix up the two arguments, you wouldn't know until runtime. What if it only fails after 3 days of training? Isn't that exactly what we're trying to avoid by using C++ instead of Python?
These are just my observations. I'm concerned because I am doing my best to make Pytorch's C++ API match your amazing existing work in autogradpp (on the functional level), so that hopefully we can soon move you guys to the official API soon (and thus provide you with much more support, and reduce the amount of effort you have to put in working around current issues in the autograd engine!). At the same time, I'm trying to design our C++ API in a way that will appeal to the wider C++ audience. I'm a bit worried about this change because I don't think our C++ users will be comfortable with an API that removes compile time type safety.
I'm sorry I didn't comment on this earlier (I'm on PTO), but I would urge you to reconsider whether this API change is the right direction for autogradpp :)

Thanks for weighing in here, @goldsborough. Here's my take on this, @ebetica can maybe elaborate a bit more.

Compile time type safety being one of the main advantages provided by C++

For us, the main argument for using C++ here is performance and the ability to efficiently use and manipulate low-level data in addition to doing tensor and NN logic.

For example this signature:
variable_list foo(Tensor x, float f) now becomes Variant foo(Variant, Variant).

As far as I understand, the variant is mostly useful for input and output arguments to forward(). We want to have it virtual, and it should not impose too many constraints on what data we can and can't pass between containers.

I'm a bit worried about this change because I don't think our C++ users will be comfortable with an API that removes compile time type safety.

Losing compile-time type-checking arguably starts with at::Tensor, no? IMHO the limited amount of compile-time checking for this is a feature rather than a hindrance :)

Can you provide me with some more context on how this change makes your code easier (just to understand)?

For me, this will be useful because I can check the inputs and outputs of user-defined models in a more robust and more expressive way (at run-time, of course). I couldn't check it at compile-time anyway (I just have ag::variable_list...).

@goldsborough @jgehring I think the crux of the issue is that our Containers are
variable_list forward(variable_list) = 0;

So we already have no compile time guarantees. This PR makes it easier to do things at run time. But, what if we just removed virtual forwards, and force users to define their own input-output contracts. Then we get compile time safety. We could even do something where container is templated on the output type:
template <typename O, typname Args...>
Container {
  O forward(Args&& params...) = 0;
  ...
}

@syhw notes that we can have both, variants for the daring and templates for the strong of will.

@ebetica and I had a discussion and we think we can have the best of both worlds. Essentially, we will take this PR, but add a little bit of extra convention to it:
// Counter-counter proposal
class Module {
  virtual Variant forward(Variant);
}
class MyContainer : Module {
  virtual Variable forward(int, Variable) {
    ...your actual, well typed code...
  }
  Variant forward(Variant v) override {
    return Variant(forward(v.getInt(0), v.get(1)));
  }
}

So if you know you have a MyContainer or subclass thereof, you can call the well-typed forward; otherwise, you can call the dynamically typed interface to implement a polymorphic trainer or something similar.

@jgehring What are best practices regarding move and copy constructors? I defined default (good for maps and stuff), copy, move, and move assignment.

Looks good; for the sake of completeness you might want to declare a default assignment operator although that's done by the compiler automatically anyway.

I don't really understand the argument that variable_list forward(variable_list) is not typesafe. It allows exclusively a list of Variables, which is 1 type. Here we're proposing a very slightly less bad thing than void*. Like, you can't pass every type in the world like with void*, but 9 different types and even a whole recursive hierarchy of these 9 different types.
@ezyang's idea is nice because it really allows you to define your forward with the concrete types (at least the forward in which you implement your code), but I much dislike putting more boilerplate on users (in having to destructure the variant in the function that calls the typed forward`).
Thought: it seems to me the original goal you had was to have a variable number of arguments, each of which can either be a variable or a scalar, right? I totally understand that this is useful. So what if we limit the Variant to be variant<Variable, Scalar> and wrap it in a way that it has variant.scalar(<index>) and variant.tensor(<index>) accessors?

Yeah this is totally possible I think.
template <typename T>
class Container_CRTP {
  Variant unchecked(Variant x) override {
    static_cast<T*>(this)->checked(magic_unpack(x));
  }
}

Unfortunately, the template-fu in my quick Google search seems very mysterious and perhaps difficult to write. Maybe @goldsborough wants to have a shot at it? :D I'm also not sure what the best naming scheme is. The "problem" is that the checked version cannot be defined in ContainerImpl. Only the unchecked version can be virtually defined. I think some variant of [checked, operator()] or [operator(), unchecked] seems reasonable.

template<typename T>
struct function_traits;

template<typename C, typename R, typename... Args>
struct function_traits<R(C::*)(Args...)> {
  static constexpr size_t number_of_arguments = sizeof...(Args);

  using return_type = R;
  using argument_tuple = std::tuple<Args...>;

  template<size_t n>
  using argument_type = typename std::tuple_element<n, argument_tuple>::type;
};

struct Variant {
  template<typename T>
  Variant(T) {}

  template<typename T>
  T get(size_t index) { return T(); }
};

#define TORCH_FORWARD(forward) \
  template<size_t... Is> \
  Variant variant_forward(Variant v, std::index_sequence<Is...>) { \
    return forward(v.get<function_traits<decltype(&SubModule::forward)>::argument_type<Is>>(Is)...); \
  } \
  Variant variant_forward(Variant v) override { \
    return variant_forward(v, std::make_index_sequence<function_traits<decltype(&SubModule::forward)>::number_of_arguments>{}); \
  }

struct Module {
  virtual ~Module() = default;
  virtual Variant variant_forward(Variant v) = 0;
};

struct SubModule : public Module {
  int forward(double x, std::string s) {
    return x;
  }

  TORCH_FORWARD(forward)
};

blows revolver


about  detectron2    version   error ？  I use source code to install detectron2-0.6（https://github.com/facebookresearch/detectron2/releases/tag/v0.6）

v0.6  version     batch_norm   doesn't have the 'LN' method?

I compared the latest version of the code with version 0.6 and found that this method was indeed missing

Thanks for your observation.
I've checked the d2 version in my environment and it is shown as follows:

But I just built d2 from the latest source code instead of the released pre-built v0.6 one. So I checked the source code in d2 and found that its version is fixed at 0.6 since two years ago, which caused the misunderstanding.

For ViTMatte, you could build d2 from the latest source codes. If it still does not work, please let me know and I'll try to help.

Thanks for your observation. I've checked the d2 version in my environment and it is shown as follows:  But I just built d2 from the latest source code instead of the released pre-built v0.6 one. So I checked the source code in d2 and found that its version is fixed at 0.6 since two years ago, which caused the misunderstanding.  For ViTMatte, you could build d2 from the latest source codes. If it still does not work, please let me know and I'll try to help.

it is   OK ,  use   latest(main) code,  I recommend stating on the project home page that using the latest code (especially for windows users, source code compilation is required)

@zhanghongyong123456
We have updated it. Thanks for your suggestion. 😄

@zhanghongyong123456 We have updated it. Thanks for your suggestion. 😄

Thanks again for a great project


Modules

This change is

Current dependencies on/for this PR:

main

PR #2691   👈

This comment was auto-generated by Graphite.


fix(build): create at most one esbuildService

fix #693
Also ensures that stopService will always wait for the service to actually have been created.
Otherwise, I think there might be edge cases where stopService is called before the service has actually been started, preventing proper cleanup.

This also closes #64


FontFamilyName getting overwritten

When I want to add a Text to my DxfDocument, the FontFamilyName gets overwritten.
Code to reproduce the bug:
DxfDocument doc = new DxfDocument();
TextStyle textStyle = TextStyle.Default;
textStyle.FontFamilyName = "Arial";
Text text = new Text() { Value = "Some Text", Position = new Vector3(0), Style = textStyle };
doc.Entities.Add(text);

doc.Save(@"C:\Temp\Test.dxf");

I can open the exported DXF-File with my CAD program an the text is at right position, but the Font is now "simplex"
The only workaround that I found was to add the following line after adding the entity:
text.Style.FontFamilyName = "Arial";

Read this. It is the same kind of problem, your Default text style is being overwritten by the one already present in your DxfDocument.

It's just like adding a block to a drawing that already has that block definition.
Running into these errors with code can be a little tricky. If your code breaks after your drawing has a text style or block incorrectly defined then you will always get that wrong definition inserting a new style or block with the same name, until you remove all references to it and then pure it out. You don't get the warning prompt
'block already exists, do you wish to redefine, when doing it directly from code.

Thank you for the quick reply.
What I don´t understand is, that its a new Document add I add a Entity. So there is nothing inside this document.
So I thought it would apply the style of the new Element.
The example with the layers is understandable. If you add the same layer again it doesn´t get overwritten.
But in my case I don´t overwrite anything, I just add a new EntityObject.
Maybe I don´t understand DXF right, but I thought that the individual options are higher ranked than the layer options.
Did I get it right, that its the normal behavior of the DXF-Definition?

There is a defualt dimstyle already present in a new doc, as is clearly stated in the link that will solve yours problem which was supplied to you.

A Text entity holds a reference to a TextStyle, so when the text is added to the document so does the style, same thing happens with its layer. All classes derived form TableObject behaves in the same way, the destination document always takes precedence. If for whatever reason you want to change this behavior it is your responsibility to check if a table already exists with the same name and act in consequence.

Thanks for your replies. I will keep that in mind.


Add warning message for discrete top-level params in MCEM.

This PR adds a warning message if a user sets up an MCEM algorithm on a model that has discrete top-level parameters.  The message reads:
Warning: MCEM cannot optimize over discrete top-level parameters. The following top-level parameters in your model are discrete: ... (parameter names here)

I thought we decided it would be an error not a warning?

Whoops, thanks @paciorek.  The warning has been changed to an error.


All requests made within nodejs time out

A brief description
I've run into an interesting issue that didn't occur in previous versions of WSL. Whenever ANY network request is made from nodejs, it either times out or throws one of two other errors I can't find information on.

Expected results
When making a request to google or another stable site a response should be returned

Actual results (with terminal output if applicable)
Seemingly randomly one of three results:

Error: connect EBADF <IP_ADDRESS>:443 - Local (undefined:undefined)
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at connect (net.js:884:16)
    at emitLookup (net.js:1013:7)
    at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:62:16)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:81:10)

Error: connect ETIMEDOUT <IP_ADDRESS>:443 - Local (undefined:undefined)
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at connect (net.js:884:16)
    at emitLookup (net.js:1013:7)
    at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:62:16)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:81:10)

or
Error: connect Unknown system error -25 <IP_ADDRESS>:443 - Local (undefined:undefined)
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at connect (net.js:884:16)
    at emitLookup (net.js:1013:7)
    at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:62:16)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:81:10)

Your Windows build number
15031.rs2_release.170204-1546

Steps / All commands required to reproduce the error from a brand new installation

node
https.get("https://www.google.com", function(err, res) { if(err) {console.log(err)} else{ console.log(res) }})

Required packages and commands to install
nodejs, npm
sudo apt-get install nodejs npm

I second this however I'm not even getting an error besides in the log. My bash terminal just freezes and then black screens upon reopening until I restart. Any command with npm does this.

Yeah, same issue for me too. Running Webpack's HMR - initial bundle appears to work fine, but the entire script just freezes after modifying a file.

This is probably #1675.

I cannot get a basic Angular package to run that's been cloned off Git. It uses npm start to trigger npm install and bower install. Also gets stuck at server initialization. If this is full featured and no bugs after beta I'll be so happy, that and tabs :P

Could this be the same issue as #1683?

@gurpreetatwal - I think it is likely.

fixed ^_^ 15042 build


Fixes double entry in Create Menu

Overview

Fixes: #6557

/azp run

Possibly a good candidate also for some build time validator (to ensure that we don't have another thing that creates a double menu entry)
Obviously separate change.

@Railboy, can you rebase this change to prerelease/2.2.0_stabilization? If you do not have the time, we can merge here and cherry-pick.
Thanks!

@wiwei I'll also update the service extension wizard so it adds a correct create asset menu attribute.

@davidkline-ms I'd prefer if you cherry-picked since it's such a tiny change.

I'd prefer if you cherry-picked since it's such a tiny change.

Will do once this passes validation


open injector failed

I have downloaded the latest version,but still the same problem，I suspect it's a problem with my computer, but I don't know how to solve it

check if .net libraries are installed on  system

check if .net libraries are installed on system

If what you say is .net framework, the version is 4.8. The weird thing is that it still worked a while ago

is there any log files the injector save on my system?  I can offer to you

is there any log files the injector save on my system? I can offer to you

im not sure but windows may automatically log in event viewer

is there any log files the injector save on my system? I can offer to you

im not sure but windows may automatically log in event viewer

I try running a demo using Visual Studio to inject the dll and it works but throws an error like this, I think this might be because of the new version of  LOL?

thats right, you have to update patterns how to do it you can find in this issue #2

I assume OP is playing on Tencent server.
So just rename the target dll int ohid.dll and put it under Game Folder. League of Legends.exe will load it automaticlly.
This method works on global versions too.

install .net framework 4.8 and c++ redistributable 2015-2023


Add VersionedApp

Add VersionedApp which now contains version which has been moved out of App
Allows users to getApp and then updateApp using the same object without causing a 400 (Marathon does not allow updates or creates to include version field)

This may be a moot point because uris and fetch cannot both be set on POST/PUT but are set on GET so we'll have to do some munging on our side anyways.
Can someone give some clarity on why both uris and fetch are used? Which should I be deleting?

@lewisheadden  fetch preferred over uris
https://github.com/mesosphere/marathon/blob/master/docs/docs/rest-api/public/api/v2/types/app.raml#L291-L294


New UAA user doesn't have cloud_controller permission

Hi Dear all,
I am trying to use the latest Cloud Foundry and UAA, but I found that I cannot create new user correctly. Because new user doesn't have the cloud_controller.read and cloud_controller.write authorities than old UAA version.
Such as I execute:  cf create-user<EMAIL_ADDRESS>passw0rd
Then I login by using it, it reported:
API endpoint: https://api.arm-dev.cc (API version: 2.92.0)
User: bmxsvt1@test
No org or space targeted, use 'cf target -o ORG -s SPACE'
FAILED
Error finding available orgs
Invalid auth token: Invalid Auth Token
Then I checked with CF_TRACE=true, I found that the authorities are different than before:
Old UAA Version:
{"access_token":"[PRIVATE DATA HIDDEN]","token_type":"[PRIVATE DATA HIDDEN]","refresh_token":"[PRIVATE DATA HIDDEN]","expires_in":599,"scope":"openid uaa.user cloud_controller.read password.write cloud_controller.write","jti":"c5084ab1ae10407aba2bb7825e7552af"}
New UAA Version:
{"access_token":"[PRIVATE DATA HIDDEN]","token_type":"[PRIVATE DATA HIDDEN]","refresh_token":"[PRIVATE DATA HIDDEN]","expires_in":599,"scope":"password.write openid uaa.user","jti":"f5ea18f47e4345e9a447188883871c0c"}
I also cannot see these two authorities in uaa.log:
[2017-11-08 08:52:59.732] uaa - 7318 [https-jsse-nio-8443-exec-3] ....  INFO --- Audit: ClientAuthenticationSuccess ('Client authentication success'): principal=cf, origin=[remoteAddress=<IP_ADDRESS>, clientId=cf], identityZoneId=[cf]
[2017-11-08 08:52:59.975] uaa - 7318 [https-jsse-nio-8443-exec-3] ....  INFO --- Audit: UserAuthenticationSuccess ('bmxsvt1@test.com'): principal=e7ea814e-a5c1-4cc6-af8b-cd1bd9af2045, origin=[remoteAddress=<IP_ADDRESS>, clientId=cf], identityZoneId=[cf]
[2017-11-08 08:53:00.128] uaa - 7318 [https-jsse-nio-8443-exec-3] ....  INFO --- Audit: TokenIssuedEvent ('["password.write","openid","uaa.user"]'): principal=e7ea814e-a5c1-4cc6-af8b-cd1bd9af2045, origin=[client=cf<EMAIL_ADDRESS>identityZoneId=[cf]

But after I can manually add by using:
uaac member add cloud_controller.read<EMAIL_ADDRESS>uaac member add cloud_controller.write<EMAIL_ADDRESS>
This user works correctly as before.
But new user still have this issue.
I am not sure if it is related with the first change in new UAA release: https://github.com/cloudfoundry/uaa-release/releases/tag/v43
Or do you know how to add these two authorities as default so that I can create new user with two authorities?
Or is there any error in my configuration for new UAA release?
Thanks a lot all!

I found that I create a new zone cf and no cc permission in new zone default groups.
I need add it by using uaac curl -X PUT /identity-zones/cf
Thanks!

Typically, CC works against the default identity zone (/identity-zones/uaa). Other newly created identity zones aren't meant to be understood by CC by default. You should be updating users in the default zone (uaa) for CC to use.


component [spreadsheetLibrary.Spreadsheet] has no function with name [getActiveSheetIndex]

I have some issues with several functions of workbook that can't be found, like the getActiveSheetIndex() or the getLastRowNum() function. The errors do throw inside the library. I'm not sure if this might be related to my code set up, or if this is an bug in you library. That's why I open an issue and ask for help here. Hopefully you can help me, as I really like the work you did with the spreadsheet library.

I'm using the version 0.7.8 of the library and run it under CF 10 and Lucee 4.5.3.

Can you post the code you are using to call the library when it throws these errors?

The error message suggests you're passing in the library instance itself rather than a workbook object to whichever public library method you're using.

For sure.
I took a further look and tried to run the sample from your README, and I was having the same issue with this sample:
<cfscript>
spreadsheet=New spreadsheetLibrary.Spreadsheet();
data=QueryNew( "First,Last","VarChar,VarChar",[ [ "Susi","Sorglos" ],[ "Frumpo","McNugget" ] ] );
workbook=spreadsheet.new();
spreadsheet.addRows( workbook,data );
</cfscript>

This is the code I'm running:
spreadsheet=New spreadsheetLibrary.Spreadsheet();
queryData = spreadsheet.read(src="#theFile#", format="query", headerrow=1);
...
// Setting query data
...
spreadsheet.addRows(workbook=spreadsheet, data="#queryData#")

And I guess you are right, that I passed the spreadsheet instead of the workbook, but I tried the same thing like in your sample, by creating a new workbook instance and passing it to the addRows method, but I experienced no difference.

Can you post your revised code where you create the new workbook and pass it into addRows() as per the sample?

My revised code looks like
spreadsheet=New spreadsheetLibrary.Spreadsheet();
queryData = spreadsheet.read(src="#theFile#", format="query", headerrow=1);
...
// Generate Query data here
...
workbook=spreadsheet.new();
spreadsheet.addRows(workbook=workbook, data="#queryData#")

And like I said I actually get the same error with your sample:
sheet=New spreadsheetLibrary.Spreadsheet();
data=QueryNew( "First,Last","VarChar,VarChar",[ [ "Susi","Sorglos" ],[ "Frumpo","McNugget" ] ] );
workbook=sheet.new();
sheet.addRows( workbook,data );
```

Thanks already a lot for taking  a look and your time in advance

Strange. Just tested the sample code again and it's working fine for me. If you're getting the same error as before then there must still be something wrong with the workbook argument you are passing into addRows(). Try dumping that variable before you pass it into that function, i.e.
spreadsheet=New spreadsheetLibrary.Spreadsheet();
queryData = spreadsheet.read(src="#theFile#", format="query", headerrow=1);
// Generate Query data here
workbook=spreadsheet.new();
dump( workbook );
abort;
spreadsheet.addRows(workbook=workbook, data="#queryData#")

What do you get?

I agree with you.
I get a workbook that, for me, looks ok:

Also I can see an existing function getActiveSheetIndex() there.

Try aborting aborting immediately after the addRows() call.
spreadsheet=New spreadsheetLibrary.Spreadsheet();
queryData = spreadsheet.read(src="#theFile#", format="query", headerrow=1);
// Generate Query data here
workbook=spreadsheet.new();
spreadsheet.addRows(workbook=workbook, data="#queryData#");
abort;

Do you still see the error?

I tried to debug this and came to  the code line 1598 in the Library where the function private any function getActiveSheet( required workbook ) starts and dumped the workbook here as well, and it seems to be fine at that point. Anyway the function that the error was complaining about wasn't there. For some reason, after playing around and after some back and forth this works now. The really weird thing is that the code that I call, as well as the code of the library now has the same status like at the point where I shared my code (and yes, I'm sure it is the same :) ). I guess that my issue might have something todo with the caching of my server, wich shouldn't be the case as well as I restarted the server several times and cleared the cache of the browser as well..
Anyways I'm closing this now and really appreciate your help.
I'll reopen this if I figure out more or get the error once again.
Thanks a lot for your time and help!


Copy all /oauth2/login/ cookies to WebView

... instead of trying to single out the sessionid cookie; since we don't know it's name ahead of time.
Fixes https://openedx.atlassian.net/browse/MA-1689

@BenjiLee Please review?

Looks good to me!

:+1:

:cookie: :japanese_ogre:


(WIP) Utilize a separate history table for (un)claiming incidents

WIP because performance is very bad when tested against prod dataset.

Avoid UPDATE'ing the active and owner_id columns of the incident
table by the API

Instead, insert either "claim" or "unclaim" actions into a history
table

Show the claim/unclaim actions for a given incident on the incident
view page

Closing until I can fix the perf here.


How can I run LSTM using my own data

Please help

If you have your own text data then use library like NLTK to do the preprocessing part like removing stop words , tokenization , and then use pre-trained Glove embedding or word2vec and then proceed in the same manner as given in this repo. Hope this helps.


Feature Request: Share note outside of app

Problem:  I frequently share things from within my Twitter app to people outside of the app via email, or Signal, or other apps on my phone.  Amethyst does not have an external app share feature.  Amethyst does not have a webapp interface, so there is nowhere for it to link to in order to reference posts.
Solution:

Add a configuration parameter for the user's favorite webapp Nostr client.  I personally use Iris.to
Add a share entry in the three dots button at the bottom of each post that allows you to send some pre-formatted data to another app on the user's device, linking the note in the configured webapp's URI format for direct view of a note.

I frequently share Tweets in this manner via GMail, Signal, Slack, and Discord, but the Share option should really work with any app that the user can select from the device.  Also provide a "Copy Link" option under the Share menu.

NIP-21 would be the correct way to do this, but it would only work for people who already had a nostr client registered to handle nostr: URIs. It probably makes sense to allow copying a web URL until NIP-21 is more widely adopted at least.
This should be a relatively quick fix, but again I don't know where to put the URL setting as we don't have a settings dialog @vitorpamplona, ideas?

I am more worried about how to add this to the UI.. we have way too many options already.. with too many complaints of Amethyst been too complicated.

This is partly why I suggested #216. We could probably also merge the "Copy" methods in the same way, and add a Share button. So the result for the menu would be:

Follow
Copy

Note Text
Author PubKey
Note ID

Share (opens usual share dialog, sends to snort)
Block & Hide User
Report

Spam / Scam
Hateful speech
Impersonation
Nudity / Porn
Illegal Behavior

I know it's more taps but you can't have everything within 1 tap when we need so many options. And we do need them IMO, but the goal is to make it intuitive so that it doesn't “feel” complicated.

@vitorpamplona I was thinking we could use a more "action menu" style for the long click on a note, and have different options to whatever is in the 3-dots dropdown. Here is a concept:

The actual buttons and icons should be chosen more carefully, of course.
I have seen some apps do this before where there is a "Select" action that opens the text in a plain text viewer for using the normal system select & copy tools. It's slightly clunky, but it works when the actual content is complicated (as in this case with multiple Texts).
What do you think about this concept, and if you like it, what would you like to see in the action menu popup (long click) vs. the 3 dots dropdown?

I hate DropDown menus, so this is def better. The question is will it fit width-wise on most small screens? I had some issues with the Zap button already...

Right now it's 64 x 4 = 256 dp. Small screens are typically at least 320 dp, so it should be fine. (In theory we can check the screen size and shrink if needed, but probably not worth it to start with.)


Setting noSwiping to true has no effect

noSwiping doesn't appear to have any effect. Simple as.
https://codesandbox.io/embed/reactidswiper-demo-53uko?fontsize=14&hidenavigation=1&theme=dark

works for me

Doesn't work for me as well. Any news? Also, I have rebuildOnUpdate set to true, but still no luck.

same, doesnt work for me on 4.0.0


Azure API Management - Questions

What would you like to know?

Can i publish GraphQL Endpoints via API-Management or only REST is supported
Is it mandatory to create a Product in API Management, or i can just publish my APIs without Product setup
Can i enforce policies at the API Management level or operation level

hey friend - I'm afraid that I can't help much with this question so I'll have to refer you to https://twitter.com/AzureSupport.


nodemcu ans azure event hub

hello
Is it possible to use nodemcu to connect the azure event hub. instead of  Azure IoT hub.  i am currently able to connect raspberry pi to azure event hub, i want to move my project from raspberry pi to nodemcu.
can anybody help me with that if it is possible.

Hi @rishikeshayre1310
we are now mostly focused on Azure IoT Hub for the connectthedots project, but let me recommend looking into the Event Hubs SDK: https://github.com/azure/azure-event-hubs.
I have not tested the port on nodemcu myself but I am sure this is doable

Thank you @olivierbloch
I have successfully implemented interfacing Nodemcu with Azure Event Hub.
PS: if anyone is still looking for similar solution please go through this


jsonschema V4.18.0 breaks openapi-spec-validator

When pulling the recent version of https://github.com/python-jsonschema/jsonschema it breaks this tool:
AttributeError: module 'jsonschema._utils' has no attribute 'load_schema'

jsonschema breaking version: https://github.com/python-jsonschema/jsonschema/releases/tag/v4.18.0
Edit: I've looked into it a bit further and believe the issues is with the openapi-schema-validator version of jsonschema:
https://github.com/python-openapi/openapi-schema-validator/blob/master/pyproject.toml#L58

Turns out we have some hardcoded versions, apologies for this ticket.


Holding a scope attachment gives an ability icon that implies it can be used by hand (it cant)

Description
As expained in title. The 2x scope attachment (i think the 4x as well)) when held in hand NOT attached to a rifle will display the action for activating the scope. This action has no effect, as the scope cannot be used without a rifle. The action should not appear for attachments that are not usable by hand.
Reproduction
Grab scope attachment in hand. Scope ability appears on ability button sidebar thingy. Works with a gun in the other hand as well, tested with 2x in hand and rifle that had 4x on it. Pressing ability for detached scope has no effect.
Screenshots

Additional context
As a sidenote, i think it would be an interesting feature to be able to use the scopes without the gun, like a makeshift binoculars.
Ok thats all <3

It's a side-effect of the way certain toggleable attachments are handled. It was either that or not having the action when the scope is attached to the gun and only making it work through hotkeys.
I'll try to see if I can change this when I make the next attachments PR. Agreed on making scopes usable in hand, though I'm not sure that's going to be allowed while we're porting 1:1. I'll ask smug about it later.


olm: update vulnerability description

Description of changes
Since the known vulnerability was added, additional information has been
published by upstream about why they believe the vulnerability to not be
exploitable over the network:
https://matrix.org/blog/2024/08/libolm-deprecation/
This commit updates the text of the vulnerability warning with
additional information from the blog post as well as a link to the blog
post.
Signed-off-by: Sumner Evans<EMAIL_ADDRESS>
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The wording in this PR seems to conflict with upstream's statements to me:

Upstream [...] has stated that the library should not be used going forward

vs.

Given the above, in the context of Matrix, libolm is currently safe to use in a practical sense (with the above caveats). However, we strongly encourage all app developers to chart a path away from libolm in favour of vodozemac.

I definitely don't see this as implying that they expect distributors or end users to take action, and marking the packages as unstable has the IMO fairly severe consequence of resulting in them not being built by Hydra.

The wording is inherited from https://github.com/NixOS/nixpkgs/pull/334638, and is based on the upstream statement in the libolm repository:

As such as of July 2024, libolm is now officially deprecated - please do not use it going forwards.  The Matrix cryptography team sadly does not have bandwidth to maintain both vodozemac and libolm, and all of our maintenance effort will go into vodozemac.

so it’s almost a direct quote. Not much to say beyond that as all sides were pretty thoroughly hashed out in the original PR.

FWIW I don’t have any objection to building libolm on Hydra (we have a mechanism for exceptions to the blanket exclusion of marked‐insecure packages from Hydra builds), but I think we concluded there was no need to because none of the reverse dependencies are particularly heavy builds. If there are clients that would be particularly painful for users to build (maybe Nheko, since it’s C++?), then it seems reasonable to consider adding olm to the list of exceptions.

If there are clients that would be particularly painful for users to build (maybe Nheko, since it’s C++?), then it seems reasonable to consider adding olm to the list of exceptions.

Exactly: the nheko developers said they don't have immediate plans to switch to vodozemac; no one has shown how to practically exploit these "vulnerabilities" and building nheko is quite resource intensive. I'm going to add an exception.

An exception for the Hydra permitted insecure packages list would be okay in my view. That means it can still be built on Hydra but the user has to override the warning to use it. Hiding the message entirely on an ad-hoc basis is not.

An exception for the Hydra permitted insecure packages list would be okay in my view. That means it can still be built on Hydra but the user has to override the warning to use it. Hiding the message entirely on an ad-hoc basis is not.

So, basically here we have both the upstream developers of the library and the client saying these vulnerabilities are basically non-issue; yet we are breaking setups unless they read a super technical wall of text, follow the instructions and accept the shame of having vulnerable software in their computer.
Okay, I guess this is useful if you're someone like Ed Snowden, but for the average Joe, it's a terrible user experience.

(Replied in https://github.com/NixOS/nixpkgs/pull/341831#issuecomment-2351584809.)


Snippet able to be commentted?

i created a snippet view as below

i can see the preview/status button, but how can i enable the comment as below? i cannot find example, anyone can help?

Hi @salfe, we use either StackOverflow  or the Wagtail Slack for support requests like this. Please check there. If you believe we’re missing this feature or there’s a bug, please take the time to use one of our pre-set issue templates.

Hi @salfe, we use either StackOverflow or the Wagtail Slack for support requests like this. Please check there. If you believe we’re missing this feature or there’s a bug, please take the time to use one of our pre-set issue templates.

It is really strange of you, why i cannot post such issue here?

@salfe We use Github issues as a planning tool for keeping track of things to be fixed or improved - each issue is a task for a developer to work on. A support question is not the same thing as a developer task - it might translate to an improvement to the software, but it might just be someone misunderstanding how to use it, or it might be a duplicate of a problem we already know about. Often, a question is better answered by someone in the community rather than a core developer, so that's what our support channels are for. For these reasons, we don't use Github issues as a helpdesk - it makes it less effective as a planning tool.


Omnibox search glyph alignment off

(Should re-check once we update to latest select library - not at all sure it will fix, but worth landing the dropdown refactor #663 first)

We've dropped Blueprint Select in favor of react-select as our core combo, and as a result this omnibox component is no longer included by default with Hoist.  Closing as a result.
We can revisit / open a new ticket when we have a concrete need for an omnibox in an app on an updated version of the toolkit.


Let's encrypt CA websocket fails on Android api < 20

Is this a bug report?
Yes
Environment
Environment:
OS: macOS Sierra 10.12.6
Node: 9.3.0
Yarn: 1.3.2
npm: 5.6.0
Watchman: 4.7.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 2.1 AI-143.2915827
Packages: (wanted => installed)
react: 16.0.0-beta.5 => 16.0.0-beta.5
react-native: 0.49.3 => 0.49.3
Target Platform: iOS (10.3)
Steps to Reproduce
I have a node server which uses Let's Encrypt CA. I try to connect to it using websockets.
On iOS and newer Android versions (API  > 20) it connects successfully.
When running on older Android versions (both on simulator and actual devices) it never connects.
I have tried to connect using older Android version to a server that's not using Let's encrypt CA, and it also connects successfully.
The error I get when using this:
    ws.onerror = (e) => {
      // an error occurred
      console.log(e.message);
    };

is Connection closed by peer
and this:
    ws.onclose = (e) => {
      // connection closed
      console.log(e.code, e.reason);
    };

logs undefined, undefined
Expected Behavior
Connect to the server
Actual Behavior
Fails
Reproducible Demo
    var ws = new WebSocket('wss://example.now.sh');

    ws.onerror = (e) => {
      // an error occurred
      console.log(e.message);
    };

    ws.onclose = (e) => {
      // connection closed
      console.log(e.code, e.reason);
    };

It seems it is indeed a problem with Okhttp.
Based on this comment I need to set up a custom trust store that trusts LetsEncrypt like this example.
@tuncaulubilge Do you have an idea how can I achieve this in react native?

There was a way to instantiate your own OkHttpClient and provide it to react-native, but OkHttpClientProvider has been broken for some time. Here is the issue: https://github.com/facebook/react-native/issues/13487
Another option is to fork the react-native repo and add your custom trust store to okhttp

@tuncaulubilge Thank you very much for your help!
I changed my files as mentioned in this commit and mentioned in this comment
As far as I understand I also need to change MainReactPackage in MainApplication.java and NetworkingModuleUtils.java as mentioned in this comment
But then how can I instantiate your own OkHttpClient and provide it to react-native in order to provide a custom trust store?

I fixed it based on this comment and this gist.

This issue should be reopened.

@svlada From what I understood, it's more a problem with okhttp and android that a problem with react native

@perrosnk please provide your solution. Or example please

@shamilsun put this into your MainApplication.java file:
  private void upgradeSecurityProvider() {
    ProviderInstaller.installIfNeededAsync(this, new ProviderInstallListener() {
        public void onProviderInstalled() {
          // You can do something here
        }

        public void onProviderInstallFailed(int errorCode, Intent recoveryIntent) {
            GooglePlayServicesUtil.showErrorNotification(errorCode, MainApplication.this);
        }
    });
  }

I hope that helps


Update speech-sdk.md

Add link to Voice Assistant documentation.
Fix weird sentence about custom voices with regards to voice assistants.

@dargilco : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Replace feather

We are using the feather icons in various places. They are pretty, but they can only either be used by including the svg in the html source, which clutters the code, or by using the data-feather="circle" attribute on an element and then running feather.replace(), which is cumbersome, because that means we have to run feather.replace() everytime we change something in a DOM where an icon could be affected.
Maybe we can replace it with something that can be used without svg and javascript, directly in html? (bootstrap icons comes to mind, but maybe there are better solutions nowadays...)

replace with something more usable like bootstrap? Or material?

 ATM (to my knowledge) we don't use that many different icons anyway.
Embedding-wise and styling wise they are quite the same.
Material Icons

link

Bootstrap Icons

link

Feather Icons

link
According to my sources, the full package of the icons is quite "big" to load, bigger than the others at least. So as we use not that many Icons, maybe we should go for a smaller package.

Font Awesome Icons

link (pre-selected a free tier set, but there are others)

–––
I would suggest to discuss which library to choose and how to unify existing Icons in an upcoming JFX.
Django-Icons
There is also this library, but all it does is to allow you to define one or multiple icon sets you want to use (f.e. one of the above) and then it creates Templatetags for you, so you can embed icons via those Templatetags. Seems just like syntactic sugar to me. But I include it here if you want to have a look (I just took a brief look):

PyPi Django Icons
Django Icons Docs

decided for material icons
@gregorpirgie to create a PR for changing icons to Google Material

I created a sample html page and loaded all existing feather icons that we currently use (in apis-core-rdf - tempaltes) into a table. You find the closest Material Icons I could find next to it. Where multiple alternatives existed, I created additional columns.
I am not opinionated about this, but if somebody prefers a variant, please comment here.

While replaciong the feather icons, I saw that some templates also use font-awesome icons.
For example this template: relations_detail_generic.html
'fa' and 'fas' signify the font-awesome icons.
Not many templates use it, but it's not the only one. Not sure if those templates are actively used though.
I will leave those untouched for the moment. Just note that there is at least one other icon library implemented in our codebase.

@gregorpirgie Thanks! It looks like the material icons are a wee bit smaller - can we preview how it looks on a random APIS page (the static html of a detail page for example) if it's not too much trouble?

@gythaogg Sure! Here is a direct comparison for detail and edit pages. Note that the "open-book" icon (the bibsonomy reference Icon) is not yet updated, as this needs an integration of the new material icons in the bibsonomy module (I created an issue for that).
Without tinkering with the size of the icons, the material icons are actually a bit bigger.  But you can style the material icons with css (color, size, etc) also.
Old feather Icons:

New Material Icons:

Right! Material looks good to me.