Supportability of VM/VMSS extension for AKS nodes

I have feedback from some customers regarding the following sentense.
They recognized that it is supported to install LA extensions into virtual machine / VMSS instance.
It's not clarified, but I think this is not supported. Could you please clarify the stance?
The Log Analytics agent is supported because it's an extension managed by Microsoft. Otherwise no, AKS is a managed service, and manipulation of the IaaS resources isn't supported. To install custom components, use the Kubernetes APIs and mechanisms. For example, use DaemonSets to install required components.

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

ID: d501ff67-9208-8bcb-e8df-f09e9b16a802
Version Independent ID: e3fc3851-ddc4-d26c-251d-6ae53d714081
Content: Frequently asked questions for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/faq.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the question. We are currently investigating and will update you shortly.

@yuriwoof Apologies for the delay. It is not recommended to install VMSS extensions on AKS, as mentioned in the FAQ.
We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


feat: add user roles multi select input

What does this PR do?

add multi-select options to set user roles in EditUserModal

Which issue(s) does this PR fix?

resolves issue #272

If this PR affects any API reference documentation, please share the updated endpoint references

@lakhansamani TBR


[Feature] Any way to get the logits instead of logprobs in lmdeploy?

Motivation
I now want to use lmdeploy to deploy internlm2-7b-reward.
I simply expanded the weight in v_head from [1, D] to [V, D].
If we can directly obtain the logits instead of logprobs , then we can deploy the reward model using lmdeploy easily.
Related resources
No response
Additional context
No response

from lmdeploy import pipeline
from transformers import AutoTokenizer

pipe = pipeline(model_path)
inputs = tokenizer(prompts, return_tensors='pt')
pipe.get_ppl(inputs['input_ids'].tolist()))

Sorry for late reply.
Currently, you can use pipe.get_logits to obtain the logits with seq_len x vocab_size shape. It's not thread safe, I'm not sure if this meets your needs.

Hi, guys, how to get logprobs from service? I deployed internvl2 76B with lmdeploy:
lmdeploy serve api_server \
 --server-port $PORT \
 --tp $NUM_SHARD \
 --log-level $LOG_LEVEL \
 --cache-max-entry-count $CACHE_FRACTION \
 --session-len $MAX_TOKENS \
 --backend turbomind \
 --vision-max-batch-size $V_MAX_BATCH_SIZE \
 $MODEL_DIR

and used openai sdk to chat:
        response = self.client.chat.completions.create(
            model=self.model_name,
            messages=messages,
            temperature=0.0,
            max_tokens=self.max_tokens,
            logprobs=True,
            top_logprobs=2,
        )

but I only got logprob for the best token, like:
logprob=0.0, top_logprobs=[]

so, how to get the logprobs like openai's API?
Thanks.

@heibaidaolx123
Do you mean top_logprobs should always contains the chosen token or the top_logprobs shoud be length of 2 or 3?
The format may not be completely consistent with vllm, but the logic of obtaining logprobs is similar to vllm. It use the probability after sampling. If you set temperature=0, it actually has only one candidate to sample, so the logprob should be 0. (I think select top2 from [1.0, -inf, -inf, ...] is meaningless)
For vllm, you may get something like this below, the probability of second item has been set to -inf by _apply_top_k_top_p actually.
0.0 [TopLogprob(token='I', bytes=[73], logprob=0.0), TopLogprob(token='The', bytes=[84, 104, 101], logprob=-149.8536376953125)]

For openai, if I set temperature to 0, I can't get logprob and I have no idea how they compute it.

@irexyc
Thanks for the explanation.
Havn't checked the code, but I tried vllm with params as follow:
    response = self.client.chat.completions.create(
            model=self.model_name,
            messages=messages,
            temperature=0.0,
            max_tokens=self.max_tokens,
            logprobs=True,
            top_logprobs=2,
        )

it did return top2 logprob like this:
top_logprobs=[TopLogprob(token='1', bytes=[xxx], logprob=-0.0025603154208511114), TopLogprob(token='2', bytes=[xxx], logprob=-6.705685138702393)]),

@heibaidaolx123
If you set temperature=0.0 in vllm, you will actually get temperature=1.0 instead. And vllm has a minimum limit of 0.01 on temperature. You can try temperature=0.01 in vllm.


Bound users in postgres should be in azure_pg_admin role

From @krancour on September 28, 2017 20:17
Without this, users cannot enable extensions, and the inability to do so rules out too many common use cases. (i.e. There's too much off the shelf software that requires various postgres extensions. GitLab CE is a great example.)
Unfortunately, this is not easily accomplished at this time because, while it is possible to assign new users to the azure_pg_admin, one must be a superuser to revoke the role. And even the administrative user is not a superuser. Inability to revoke the role from a bound user means it is also impossible to drop a user upon unbind since a drop role (user) command implicitly drops all the role's (user's) roles (group's).
(PostgreSQL uses "role" too much. 😛 )

Note that this issue impacts the Gitlab CE chart, so it would be helpful to figure it out sooner rather than later

How did you solve this?

@yves-vogl see #51. I instead of handing out near-superuser level of access upon bind, the provisioning parameters were expanded to permit the party requesting provisioning to specify what extensions are needed. These get pre-installed by the broker upon provisioning.


Support for ActiveBackup for Business and ActiveBackup for Office365

Hi i've added a little support for ActiveBackup for Business and ActiveBackup for Office365
I only have added the functions I needed, it is not complete.
These are the functions (for the moment)

GetOffice365_ListTasks
GetOffice365_GeneralLog
GetBusiness_ActivityLog
GetBusiness_InfoList

Is it ok if i push my branch?  (don't have much experience with git and other peoples projects)

I never received an answer from the owner of the original Synology.net.
But i have a c# app running to monitor different Synology's at clients locations.
How do you suggest i share this with you?
I'll have a look later today or tomorrow on how i could push this repo as a public repo.

First thank you for reply
I just need you to help me about the active backup API if you have a documents or the methods name to get all info for tasks, reports.

I've uploaded my library at:   https://github.com/lucwuyts/ADE-SynologyAPI
I discovered the parts i implemented by using Fiddler to look at the original Synology communications.
I only build and tested what i needed for myself.  So this is not complete (yet)


v4.1.1 has inconsistent types with express v5

@slack/bolt version
4.1.1
Your App and Receiver Configuration
N/A
Node.js runtime version
v21.7.3
Steps to reproduce:
Install @slack/bolt latest which installs express v5. The express types are set to ^4 in @slack/bolt's package.json but these aren't aligned with the version of Express being used.
Expected result:
@types/express should be 5.0.0 if Express v5 is used as far as I know.
Actual result:
@types/express v4 is used even though express v5 is installed.
Requirements
For general questions/issues about Slack API platform or its server-side, could you submit questions at https://my.slack.com/help/requests/new instead. :bow:
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hey @nickdirienzo 👋  Thanks for catching this!
As I understand, the major version of @types/express should match express which is done in #2355. That PR also moves this into devDependencies since I believe it's used during development of bolt-js itself and not during app development.
Could you share how you stumbled into this? I'm wondering if it was caused by incompatible @types/express and express versions, but haven't run into this myself 🤔

Thanks for jumping on this so quickly @zimeg! I didn't realize @types/express was pinned to ^4, so in our package.json we had it on 5.0.0 because of the express 5 upgrade. It looks like that PR should resolve the typing issue 🙏
What we're seeing in our development experience is we have a function like this:
import { json as JsonMiddleware, type IRouter, Router } from "express";

// This is getting typed with v5
export const setupApiRoutes = (router: IRouter) => {
	router.use("/v1", apiRouter);
};

Trying to pass in receiver.router causes TS to think a v4 Express router is being passed in even though it expects a v5 one... 😅
import { App, ExpressReceiver } from "@slack/bolt";

const receiver = new ExpressReceiver({
	signingSecret: env.SLACK_SIGNING_SECRET,
	clientId: env.SLACK_CLIENT_ID,
	clientSecret: env.SLACK_CLIENT_SECRET,
	stateSecret: env.SLACK_STATE_SECRET,
	scopes,
	installerOptions: {
		directInstall: true,
	},
	installationStore: InstallationStore,
});

// This errors for us in our IDE
setupApiRoutes(receiver.router);

🎉 Thanks so much for sharing this snippet! I can confirm the latest changes fix the compilation error this was causing.
We have a release candidate prepared for the next patch version in #2347 which is undergoing testing now, but I'm hoping to land this change with the next release soon!

So speedy! Thanks for following up on this so quickly!


skipping N labels in the x axis is dynamic

The datasets I am using are often very large, (one point every minute for a day or week). It is a pain to need to calculate the number labels to skip based on screen size and number of datapoints by hand every time.

Thanks @flatiron32, this makes users life much easier!


List Setup box cannot be closed

Expected behavior:
When clicking the Cancel or X button on List Setup box, it should disappear without saving any changes.
Actual behavior:
The List Setup box does not disappear when clicking on the Cancel or X button.
Reproduce steps:
Login to admin dashboard, find a list (orders, menu items or whatever), click Setup button on header, the List Setup box appears. Click Cancel or X button on that box.
Version:
3.0.4.beta.22
Additional Information:
N/A

Works fine for me, do you have errors in your browser console log?

There isn't any error in the console log. Quite strange. Have installed 2 copies both are the same. Also tried different browser.

Same behavior for me.

Does it work on a different browser?

Does it work on a different browser?

No I've tried Google Chrome, Microsoft Edge, Firefox and Safari, none of them works.

Works for me on firefox and chrome

Perhaps some files missed from the latest release?

Probably, but you should see an error in your console.log if its a case of missing files

Sam - Open the setup box, untick one of the options, then try cancel or close the box - it won't work. You have to re-tick the option and press apply to close the window. There is nothing showing in Console, which I find very strange? Almost as though there is no action attached to the buttons when a change has been made. To be fair, the end result that you are forced to do in order to close the window, is the same as pressing cancel or x but I understand how some users may find it frustrating.

Thank you for the input.
Truly strange 🤯. I've tried both admin orders and reservations page, open the setup modal, tick one of the options and I'm still able to close the modal. Tried on both firefox and google chrome. And yeah, there's no event attached to the checkboxes or action attached to the buttons so it's a bit tricky to debug, can't even guess what it could be lol.
Good thing there's a workaround until we find a solution.

The alternative, to retick the item and apply, gives the same result...so for now, you probably have more important things to be on with :)

I now just refresh the whole page to close it lol

Yea re-ticking the item makes no difference on my end, I guess we can manage with the workarounds until there's a fix 🤷♂was hoping to have this fixed in the next update.

I get this from time to time but it's definitely a bootstrap modal bug instead of a TI one - the close buttons aren't selectable at all even with a right click and inspect. We may just need to upgrade bootstrap modal?

Coolio, I'll do the bootstrap update. I have some UI improvements planned for the first RC.

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.

This issue looks fixed in v4. Closing, but please reopen if it still occurs.


Add ETCD Chart

Description
Create etcd chart chatflow
Related Issues
#2112
Checklist

[x] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[x] Build pass
[ ] Documentation
[ ] Code format and docstrings

Once Merged will move etcd chart chatflow issue to Verification

Once Merged will move etcd chart chatflow issue to Verification


fixFix cluster key column duplicated in value columns group

What problem does this PR solve?
Issue Number: close #44609
Related pr: #40372
Problem Summary:
key_columns will be used after move.
This will cause skip in construction of value_columns useless.
cluster key  column will be duplicated added in value column group.
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


Finalize all layers

For #3039
Change List

Finalize all layers (will call buffer.delete and model.delete) upon Deck finalization

Coverage decreased (-0.008%) to 62.464% when pulling 427d0b315ae21abe292f7d81632b1eb33e5ad82d on x/memory-leak-fix into 1222ed1c88ddf0f101674c2a81734c96ace7bd90 on master.


Topics

Hi there, is there a way to set topics for a Jocko server either at runtime or through configuration options?

Hey, I started on writing a command. E.g. go run cmd/createtopic/main.go --topic my_topic --partitions 1


Fix readme, bump deps

Fixes #544

Codecov Report
Base: 58.46% // Head: 58.46% // No change to project coverage :thumbsup:

Coverage data is based on head (7336244) compared to base (555a1fc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #545   +/-   ##
=======================================
  Coverage   58.46%   58.46%
=======================================
  Files          23       23
  Lines        1777     1777
=======================================
  Hits         1039     1039
  Misses        738      738

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Feature Request: Install Script/Documentation Improvements

Is your feature request related to a problem? Please describe.
Users are having issues with the setup and definition of the IRR.IRRSMO00.PRECHECK resource in the XFACILIT class.
Describe the solution you'd like
We should potentially have a script that could define this resource so that auditing would flag attempts to use it when it fails.
Describe alternatives you've considered
Another alternative would be to DIRECTLY REFERENCE this general resource in the install steps in the documentation.
Additional context
N/A

Implemented in #57


Forwarded Ip is ignored

If Traefik is configured as LB and with externalTrafficPolicy: Local, the Ip address logged (and i guess used for verification) is a internal cluster ip, instead of the real client ip.
time="2019-02-20T17:29:52Z" level=warning msg="Error validating csrf cookie: Invalid CSRF state value" RemoteAddr="<IP_ADDRESS>:58794" csrf=ASDF state=
time="2019-02-20T17:30:45Z" level=error msg="Invalid cookie: Invalid cookie mac" RemoteAddr="<IP_ADDRESS>:58794"

The "RemoteAddr" is read directly from the request, so this would be the expected behaviour: https://github.com/thomseddon/traefik-forward-auth/blob/master/main.go#L23
Perhaps this could be improved in this scenario!
I want to test this with kubernetes a bit more, so this would be one to look at whilst that's being done.

I can actually use docker-compose-dev.yml to reproduce this issue.

create OAuth API, configure http://localhost:8085/_oauth
update the traefik-forward-auth env setup
docker-compose -f docker-compose-dev.yml up
update virtual host and access http://localhost:8085

I can always see
traefik-forward-auth_1  | time="2019-04-16T03:27:53Z" level=error msg="Invalid cookie: Invalid cookie mac" RemoteAddr="<IP_ADDRESS>:47630"

http request would be like below:
Cookie: _ga=GA1.1.1551400415.1547069710<EMAIL_ADDRESS>Host: whoami.localhost.com

I turned on the DEBUG log:
traefik_1               | time="2019-04-16T03:45:22Z" level=debug msg="Creating server server-example-whoami2-1-d0bbe061a5e0438aae1a72e9084b9114 at http://<IP_ADDRESS>:80 with weight 1"
traefik_1               | time="2019-04-16T03:45:22Z" level=debug msg="Creating route route-frontend-Host-whoami-localhost-org-1 Host:whoami.localhost.org"
traefik_1               | time="2019-04-16T03:45:22Z" level=info msg="Server configuration reloaded on :80"
traefik_1               | time="2019-04-16T03:45:22Z" level=info msg="Server configuration reloaded on :8080"
traefik-forward-auth_1  | time="2019-04-16T03:45:33Z" level=error msg="Invalid cookie: Invalid cookie mac" RemoteAddr="<IP_ADDRESS>:50662"
traefik_1               | time="2019-04-16T03:45:33Z" level=debug msg="Remote error http://traefik-forward-auth:4181. StatusCode: 401"

Now I comment out the lines:
    # [entryPoints.http.auth.forward]
    # address = "http://traefik-forward-auth:4181"
    # authResponseHeaders = ["X-Forwarded-User"]

Seems working for my favor now.

I've just pushed a commit that should fix this: https://github.com/thomseddon/traefik-forward-auth/commit/f1ba9b5ac49abac977651552193b3ea87166b423
I will tag and release this next week :)

This is included in v2 which has been released today


Could not find the CDS virtual entity application role

Hi,
We have followed this step up correctly up till the step to assign CDS virtual entity role to the new user, but we couldn't find it.
However, we found these roles

Are those the correct replacement for CDS virtual entity role? Since CDS is an old term used before new term (Dataverse) is introduced.
However if we use that, we could not get the result in advanced find in the next step. It ended up as an error.
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/power-platform/enable-virtual-entities#generate-virtual-entities

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

ID: 8065cd89-f253-c50b-ed2e-7b84e5b2409c
Version Independent ID: 8c7e311a-582c-1ac6-328e-be817a62924c
Content: Configure Dataverse virtual entities - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/power-platform/admin-reference.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@Jared @.***>

From: v3ux @.>
Sent: Wednesday, June 29, 2022 3:38:26 AM
To: MicrosoftDocs/dynamics-365-unified-operations-public @.>
Cc: Sunil Garg @.>; Mention @.>
Subject: [MicrosoftDocs/dynamics-365-unified-operations-public] Could not find the CDS virtual entity application role (Issue #4062)
Hi,
We have followed this step up correctly up till the step to assign CDS virtual entity role to the new user, but we couldn't find it.
However, we found these roles
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F108313663%2F176416866-590b0030-ec4c-49eb-b87a-e2c6eec9835b.png&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=9eFfh1B%2FTPZvbs5gGlQVuJCKk2RFudQkAF6w9Smphw0%3D&reserved=0
Are those the correct replacement for CDS virtual entity role? Since CDS is an old term used before new term (Dataverse) is introduced.
However if we use that, we could not get the result in advanced find in the next step. It ended up as an error.
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/power-platform/enable-virtual-entities#generate-virtual-entitieshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Ffin-ops-core%2Fdev-itpro%2Fpower-platform%2Fenable-virtual-entities%23generate-virtual-entities&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=zv2xNfgM5Fz%2FW8Kb%2Bs12iGwNtMYYYMOmzeQc6uFMMdQ%3D&reserved=0

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

ID: 8065cd89-f253-c50b-ed2e-7b84e5b2409c
Version Independent ID: 8c7e311a-582c-1ac6-328e-be817a62924c
Content: Configure Dataverse virtual entities - Finance & Operations | Dynamics 365https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Ffin-ops-core%2Fdev-itpro%2Fpower-platform%2Fadmin-reference%23grant-app-permissions-in-finance-and-operations-apps&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=UbX%2F%2Bkh0HEDu14pzvo783xtPg2LvbFma9kNucnP%2Ffrs%3D&reserved=0
Content Source: articles/fin-ops-core/dev-itpro/power-platform/admin-reference.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FDynamics-365-Unified-Operations-Public%2Fblob%2Fmain%2Farticles%2Ffin-ops-core%2Fdev-itpro%2Fpower-platform%2Fadmin-reference.md&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2BoXarashuySKIr6P5UWLK7fggb9Nw%2B6fUF%2FgwQkCsm0%3D&reserved=0
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @Sunil-Garghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSunil-Garg&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=xz5%2B%2BpmbU%2BtS5VLm74n0uQTxidcFbrvNNCJVk%2Bq5e1I%3D&reserved=0
Microsoft Alias: sunilg

—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fissues%2F4062&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=VLTKjM7vPZX8CzlTdWJ1k%2BmBgIJ%2FY4UJqC6QJD1NsuE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHNKDW4D5HZTP5UY6O55KSLVRQRSFANCNFSM52FHXSPA&data=05|01|sunilg%40microsoft.com|1f291ce9436240bf00e608da59bb807b|72f988bf86f141af91ab2d7cd011db47|1|0|637920959107688100|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=YPT51hy4Tvk0Wbb8UeDDPWNSRa6k4TuxulmhzTnp%2FPI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Yes, this should have been changed. The names of the security roles were updated when the product name changed from "Common Data Service" to "Dataverse". The role to use for this scenario is "Dataverse Virtual entity integration app". I've submitted a PR to update the documentation.

@jaredha - Thank you for taking care of this so quickly. I have merged your PR and the article changes will show in the next few hours. I will close this issue.


⚠ ClamAV has degraded performance

In 7958008, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 625672f.


Conflicting warnings regarding space character within a regular expression

I use a small regular expression /[\ \t]+$/gm to remove trailing spaces/tabs from each line of a multi-line string, and it was previously accepted without issue from JSLint. (I can test and see which version this behaviour changed, if you wish.)
With the current version (2017-01-20), however, the line
let a = /[\ \t]+$/gm;

will trigger the warning Unexpected '\' before ' '.
Removing the '\' escape character before the space, as recommended
let b = /[ \t]+$/gm;

will instead trigger the warning Expected '\' before ' '.
Which of these warnings is correct?

Putting a space in a regexp is asking for trouble. Instead us \u0020 or \s, or even better, use the trim method.

Sorry for not being too clear, but the issue I was trying to bring up wasn't related to the use of spaces, but instead the conflicting/incorrect warnings. If \s or \u0020 is recommended, the warning should try to indicate this, rather than suggesting to add/remove the '\' character.


Character Browser & Character Creator cannot be accessed

Recently, ever since June 28 this year, I somehow was not able to access the character creator or the character browser on Wrapper: Offline v2.0.0. Every time I visit the link for both webpages, it shows the Chrome error "This page isn’t working - localhost didn’t send any data. ERR_EMPTY_RESPONSE". I didn't configure or modify any of the cc_store files. When I tried to fix the issue, I searched up ways to fix this crash on all browsers, everything has been attempted and I still can't access the char creator and browser. Is there a fix for this? I suppose it's something with Node.JS but I'm not quite sure how to fix this issue.

Nevermind, I removed the footer for the main 3 pages and forgot to remove it for the Char pages. Case closed!


Making GroupBy extensible

Fixes https://github.com/Kotlin/dataframe/issues/871
Creating a way to allow Kandy to create a custom GroupBy class by the GroupByInternal interface

Publishing dev version: 0.14.0-dev-4148

second attempt at 0.14.0-dev-4161

I'll merge it after https://github.com/Kotlin/dataframe/pull/877


[Xcode] Supports Xcode 8 / Swift 2.3

This build settings allows the frameworks to be built using Xcode 8 and Swift 2.3 As a bonus, a side-effect is that Carthage will continue to work when using Xcode 8.

Cool, I actually haven't tested against 2.3. Is all the current code compatible with 2.3?
Will merge if tests pass.

Locally testing, two iOS tests are failing, both related to attributed string calculations:

testAttributedTextDefaultFont;

_("117.5") is not equal to ("114.5") +/- ("1e-05") _

testAttributedStringCustomFontPartialRange

("171.0") is not equal to ("168.0") +/- ("1e-05")

I noticed in the latter text case, there's a conditional code path between iOS 9 & 8. For the time being, we can close this PR; I will continue to use a fork.

Thanks for the idea. I created #37 to make the tests more robust and to support Xcode 8.


On the hot corner "Mission control " to hide the desktop icon

"On the hot corner "in Window switch mode" to hide the desktop icon"
In this mode, the hidden icon only shows the wallpaper, because this can keep the screen clean.
Elementary OS

Mac OS

Probably need to fork KWin for that. Dynamic workspaces would be good as well.

I don't quite understand what the actual feature request is.

I don't quite understand what the actual feature request is.

Hide icon on this mode.

I think you mean Fiker's desktop icons should hide in expose mode.
Is that it?

Yep

For this to work, Filer would need to be aware of what KWin is doing.
Filer is independent software from KWin (which comes from KDE). I don't want to introduce KDE dependencies into Filer, hence it is unlikely that Filer will somehow react to what KWin is doing.
Hence, closing for now; especially since this does not affect functionality negatively.


docs(configuration): add updated_option

Check List
Please read and check followings before submitting a PR.

[ ] I want to publish my theme on Hexo official website.

[ ] I have read the theme publishing doc.
[ ] name is unique.
[ ] link URL is correct.
[ ] preview URL is correct.
[ ] preview URL web site is rendered correctly.
[ ] Add a screenshot to source/themes/screenshots.
[ ] Screenshot filename is same as value of name.
[ ] Screenshot size is 800 * 500.
[ ] Screenshot file format is png.

[ ] I want to publish my plugin on Hexo official website.

[ ] I have read the plugin publishing doc.
[ ] name is unique.
[ ] link URL is correct.

[x] Others (Update, fix, translation, etc...)

Languages:
[x] en English
[x] ko Korean
[x] pt-br Brazilian Portuguese
[x] ru Russian
[x] th Thai
[x] zh-cn simplified Chinese
[x] zh-tw traditional Chinese

https://github.com/hexojs/hexo/pull/4278

@curbengh Would you mind reviewing this?


Annoying sound in dashboard

Can you please add an ability to disable that "block found" sound from account settings? Or at least remember its state between dasboard page refresh? Because it is so annoying. And when i have many pools opened, it is like tortures.

Try this #2510

Was merged into the development branch so feel free to try it out there. Will be merged into master someday.


Add histogram_stdvar function

Addresses https://github.com/prometheus/prometheus/issues/11278
Only variance, since std dev can be derived by square rooting variance.

Thanks, looks plausible to me, but CC @beorn7 as the author of the issue and the native histogram guy :)

Btw. I see that for some other things like stdvar_over_time(), we use Kahan summation to keep errors smaller (https://github.com/prometheus/prometheus/blob/bb90379163afe24d60c5afae999af4b8f202d1e0/promql/functions.go#L653-L654), is that applicable here as well?

Thanks for the tip and the cc (I don't have the permission to add him as reviewer), now using kahan summation as requested.

@beorn7 I added you as a reviewer just because this is so native-histogram-related that I wouldn't want to merge it without your "ok" :)

@beorn7 PTAL


Update to rxjs 5.5 and lettable operators

https://github.com/ReactiveX/rxjs/blob/master/doc/lettable-operators.md
Also had to bump the TypeScript version for this to build, which in turn required bumping the Angular version, then bumped the zone.js version because why not.
(Diff against #16 to see the actual changes made here, or just look at the last commit of this PR.)

Sry, will close this too, since it's mixed with other stuff.


Support direct calls to a contract without going through an account contract

All contracts are equal, so we could start a tx by calling any method on any contract. A good example for this is any "poke"-like method that can be called by anyone. This probably needs a new wallet abstraction that does not have an entrypoint associated, and just forwards the request.
Example:
contract Foo {
    function poke() {
        // can be called by anyone directly
    }
}

@PhilWindle said done/duplicate.


add zstd compression support

Was unable to extract a fedora rpm using this tool due to their move to zstd to compress the embedded cpio archive.
This change just adds the de-compressor for zstd.

Thanks for the contribution! This would add a hard requirement on CGo that wasn't needed before. Could you please swap out a suitable pure replacement and/or put the DataDog dependency into a separate file guarded by a cgo build tag?

Thanks for the contribution! This would add a hard requirement on CGo that wasn't needed before. Could you please swap out a suitable pure replacement and/or put the DataDog dependency into a separate file guarded by a cgo build tag?

@mtharp https://github.com/klauspost/compress/tree/master/zstd seems like a decent good implementation, however since this returns a io.Reader instead of a io.ReadCloser, I believe it would leak goroutines without additional changes, although the cgo solution leaks memory also right now. :neutral_face:
no-cgo branch

@mtharp https://github.com/klauspost/compress/tree/master/zstd seems like a decent good implementation, however since this returns a io.Reader instead of a io.ReadCloser, I believe it would leak goroutines without additional changes, although the cgo solution leaks memory also right now. :neutral_face:
no-cgo branch

Take a look at e5cf0f808585c10c36b3c133f0a524b93afd34d1 and see if that does what you need. I pulled in both implementations depending on whether cgo is on, and added calls to Close in each place the decompressor is used. Your patch changing the signature to ReadCloser would be the best way if it were new code but I don't want to break the call signature of cpio. I also added a test to verify that no goroutines leak.

Take a look at e5cf0f808585c10c36b3c133f0a524b93afd34d1 and see if that does what you need. I pulled in both implementations depending on whether cgo is on, and added calls to Close in each place the decompressor is used. Your patch changing the signature to ReadCloser would be the best way if it were new code but I don't want to break the call signature of cpio. I also added a test to verify that no goroutines leak.

I believe this solves my needs, and wants are in your TODO list for v2, so I think I am good.

I believe this solves my needs, and wants are in your TODO list for v2, so I think I am good.


FileSystem scrolls when moving a dock tab below it

Tested versions
v4.2.2.stable.official [15073afe3],v4.3.beta2.official [b75f0485b]
System information
Godot v4.2.2.stable - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce RTX 3070 (NVIDIA; <IP_ADDRESS>99) - 11th Gen Intel(R) Core(TM) i9-11900K @ 3.50GHz (16 Threads)
Issue description
When rearranging the dockable tabs, if the filesystem tab is docked above others, then when moving a lower tab the filesystem will scroll down.
Steps to reproduce
Have FileSystem tab on upper dock.
Any tab on lower dock, below FileSystem.
Then move that lower tab, and the scroll will occur.
Here is a video showing it with the MRP:
https://github.com/godotengine/godot/assets/43964088/a872e809-7a5e-4e64-b4cc-5e328ce20b1e
Minimal reproduction project (MRP)
MRP from the video example with a folder of 100 empty txt files:
MRP-filesystem-scroll.zip

I think it's caused by a feature of the tree where the scroll is automatic when the dragged object is "near" the tree.
On my computer the scroll_border is 40 pixels:

I'm not sure how or if this should be fixed.
https://github.com/godotengine/godot/assets/81109165/219b4c2d-0b50-4b8b-bca3-11cafcd80126

@Hilderin I see what you mean. Not sure if there is a way to prevent non-files from causing the scroll, meaning have it only scroll for files, not everything. Or as an easy fix, can make an editor options toggle to turn off drag scroll. So if I am dragging something I can just scroll manually instead.


Training via CPU instead of GPU

Would this be possible, I have around 48gb of system ram but my GPU is a notebook M1000M laptop GPU which has 2GB which is insufficient, I understand CPU training would take significantly longer but still wish to switch as I don't have the graphics hardware.
WIN 10 X64
VSCode, IPYNB jupyter notebook

This is for the train.py file, there are multiple instances of cuda being used in the code and I'm just wondering if there was a simple way to switch over to entirely CPU processing

Unless there is a way to get past this issue:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 58.00 MiB (GPU 0; 2.00 GiB total capacity; 1.66 GiB already allocated; 0 bytes free; 1.72 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

To my knowledge nobody has attempted to do CPU training. Typically 16GB VRAM is needed for training. If you really want to try doing it on CPU I would try removing the cuda() functions in train.py that are used for moving tensors to the GPU.


Add Support for piñata IPFS gateways

Add support to host pinned IPFS files via Piñata gateways. These gateways will show IPFS content faster and without rate limits. So we can use IPFS as storage solution for NFTz.me and others as well. Will also greatly improve rendering of existing IPFS NFT projects like spookies and pixelpirates (those have ratelimits now via cloudflare ipfs)

Hi @superzordon this pr was approved for bitclout. Can you also approve? cloudflare-ipfs.com is marked as dangerous in Edge for  example. Thanks

@superzordon thanks for accepting the PR. Any eta on when this will be active?


CPU profiler doesn't produce file for std

I found when using python as the driver program with CPU profiler enabled with LD_PRELOAD, the profiler file will not be generated.
Example code to reproduce
if __name__ == '__main__':
    env = os.environ.copy()
    proc = subprocess.Popen(
        ["bazel-bin/external_profiling/cpp_subprocess"],
        env=env,
        # If remove the later two lines, the CPU profiler file will be generated without any problem.
        stdout=subprocess.PIPE,
        stderr=subprocess.PIPE)

    proc.wait()

Command to reproduce:
LD_PRELOAD="/usr/lib/x86_64-linux-gnu/libprofiler.so.0" CPUPROFILESIGNAL=12 CPUPROFILE=/tmp/cpu_profile.out bazel-bin/experimental/external_profiling/python_driver

It implies something isn't right with your environment variables. I'd suspect CPUPROFILESIGNAL.
https://github.com/gperftools/gperftools/blob/285908e8c7cfa98659127a23532c060f8dcbd148/src/profiler.cc#L199
Make sure you understand what is it that you want and debug it first without python driver.


Feature request: Watch unit tests

It would be nice to be able to do something like npm run unit:watch to improve the workflow of writing unit tests. Is something like this in the roadmap? Thanks!

just do npm run unit --no-single-run (or was it npm run unit -- --no-single-run ?)
It's a Karma option: https://karma-runner.github.io/1.0/config/configuration-file.html

Awesome! (It's the second one, npm run unit -- --no-single-run, btw). Thanks a lot.


Tell user what kind of resource is not enough when 'GangAllocation failed'.

As below, 'GangAllocation failed'. We'd better display which resource is in short.

https://github.com/Microsoft/pai/issues/824


undefined: os.ReadFile

When I try to install, I'm getting the following:
GOPATH=/usr go get github.com/r3kzi/clamav-prometheus-exporter
# github.com/prometheus/procfs/internal/util
usr/src/github.com/prometheus/procfs/internal/util/parse.go:69:15: undefined: os.ReadFile
usr/src/github.com/prometheus/procfs/internal/util/parse.go:78:15: undefined: os.ReadFile
usr/src/github.com/prometheus/procfs/internal/util/readfile.go:36:9: undefined: io.ReadAll

This is solved by using a newer version of go, and instead doing:
GOPATH=/usr go install github.com/r3kzi/clamav-prometheus-exporter@latest

Perhaps the README.md should be updated to indicate the minimal version and these instructions.


Configuration file behaving weirdly + bow issue

Hello, i have some serious issues with the configuration file, first is that for some reason the numbers are not adding up.
With 1.0 cost i can do 5 melee attacks with a regular netherite sword
With 0.8 its almost 35 attacks, i thought it would be 20% more attacks then regular "default" config, but seems that's not how it works.
Don't get me wrong, 35 attacks is about what i wanted, but reducing the cost by 20% and that increasing the amount of uses in 1000% is way too weird of a configuration, its totally not proportional.
I first tried to change the default numbers  to 0.1 0.3 or 0.5 (to increase number of attacks to around 20) but all of those settings resulted in absolute 0 consumption - infinite attacks (also i had to restart the world for changes to apply on single player)
Also when using bows, the configuration behaves even worse... even with the config set to 0.0 which should make it not consume any stamina at all, holding the bow on "aiming" mode still drains the same stamina it did with default settings and makes it impossible to use the bow (3 uses/max charge shots and it drains all base stamina ,unrealistic and bugged)
Is it possible to get some help on this? I really wanted the stamina system to balance melee/ranged classes the same way i did with mana for my mage classes in my modpack, but this mod is not working right now..

Also the game crashed when i tried to change the effect values, something related to string input or something, idk, it crashed instantly, can provide more details on that later, but its not my main focus here.

Help please T-T

Nevermind, edited the mod and setting the bow value to 0 worked inside the jar file..
Please don't add a source on curseforge if you don't plan to manage your mods issues.. a month is too long to not even reply


fixes #172 and #169: NewsPlease.from_urls() - use multiprocessing

also, do not attempt to extract info from empty html responses
ref: https://github.com/fhamborg/news-please/pull/173#issuecomment-1697734223

Thanks for the implementation, @arcolife. Code works for me.
This is my cpu core usage:

I don't know how many cores can/should it use in parallel because html parsing is a quick process but it does appear quicker.

@fhamborg There are times when we get 503 error because of server issue. We can add a retry extraction feature here like below:
    def __download_with_retries(self, url, local_filepath, max_retries=10, retry_delay=5):
        retry_count = 0
        while retry_count < max_retries:
            try:
                urllib.request.urlretrieve(url, local_filepath,reporthook= self.__on_download_progress_update)
                return local_filepath
            except urllib.error.HTTPError as e:
                if e.code == 503:  # Service Unavailable
                    print(f'HTTP Error 503: Service Unavailable - Retrying in {retry_delay} seconds')
                    time.sleep(retry_delay)
                    retry_count += 1
                else:
                    raise  # Re-raise the exception if it's not a 503 error
        print('Max retries reached, unable to download file')
        return None

    def __download(self, path):
        """
        Download and save a file locally.
        :param url: Where to download from
        :return: File path name of the downloaded file
        """
        local_filename = urllib.parse.quote_plus(path)
        local_filepath = os.path.join(self.__local_download_dir_warc, local_filename)

        if os.path.isfile(local_filepath) and self.__reuse_previously_downloaded_files:
            self.__logger.info("found local file %s, not downloading again due to configuration", local_filepath)
            return local_filepath
        else:
            # cleanup
            try:
                os.remove(local_filepath)
            except OSError:
                pass

            # download
            if self.__s3_client:
                with open(local_filepath, 'wb') as file_obj:
                    self.__s3_client.download_fileobj(self.__cc_bucket, path, file_obj)
                return local_filepath
            else:
                url = self.__cc_base_url + path
                self.__logger.info('downloading %s (local: %s)', url, local_filepath)
                local_filepath = self.__download_with_retries(url, local_filename)
                self.__logger.info('download completed, local file: %s', local_filepath)
                return local_filepath

It's a small feature. Can/should that be added to this same PR which @arcolife has initiated or should it be added separately (if it can be added)?

thanks for the PR! :)

@copperwiring thanks for the confirmation!
re: cpu cores in your screenshot: maybe there weren't as many links? It's been a while since I ran this on a full-blown testbed. But it did use all the cores on my end last time (as you've seen in those other screenshots back then)
re: retry: You could add the retry in a separate PR if it's not a non-issue :)


remove unused func - fix from https://github.com/mattermost/mattermost-webapp/pull/1976

Summary
remove the function which should be removed from https://github.com/mattermost/mattermost-webapp/pull/1976
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

@crspeller i will cherry pick this to release branch or dont need?

@cpanato The original PR you reference is not on the release branch, so why would this need to be cherry picked?

Ahh thanks for let me know @crspeller


Abstract out reliance on DRF response structure

Currently we are assuming that list responses follow a Django REST Framework list response structure, e.g.:
{
    "count": 26,
    "next": null,
    "previous": null,
    "results": [
        {},
        {}
    ]
}

We are automatically pulling out results as the objects of interest. This will break for any API that just returns results as a simple list, or with a different key.
Some options to resolve this could be:

specifying a keyPath in the schema (e.g., for DRF the keyPath would be results)
do the above and also save the meta data
something else?

Ah, I see we kept the keyPath logic in the templates. @rmutter I'm happy to add this back to the parser and update the templates if you think it's the way to go.

@baylee I don't know if there would preferably be a global default as well as a way to configure it per endpoint. Globally may be the more valuable feature to implement first.


BUG: Model.clientModel.service.finish_modification() causes error since version "1.18.0"

Describe the bug
Model.clientModel.service.finish_modification() causes an error since version "1.18.0" of the rfem package.
To Reproduce
Steps to reproduce the behavior:
Create a structure with loads and combinations and result combinations.
call Model.clientModel.service.finish_modification() to finish modification and run calculation.
Expected behavior
I expect, as what happened in version "1.17.0" the calculation to run, not give an error.
Desktop (please complete the following information):

Version of WIN10 OS: [WIN 10 20H2]
Version of RFEM6
Version of tool 1.18.0

Hi @Dervis-vL,
this is usually sign of problem inside the routine, not with finish_modification function itself. If you can share the script here or send it to<EMAIL_ADDRESS>I can investigate further.

Should be resolved. No script was sent and nobody else reported this issue.


For pylsp server command PylspInstall python-lsp-black does not work out of the box

(thank you for your work on the installer)
Describe the issue
For the pylsp server after running :PylspInstall python-lsp-black does not use the black formatter. It uses the autopep8 and yapf formatters instead because :LspInstall pylsp installs the python lsp with the [all] descriptor (python-lsp-server[all] ).
Links
nvim-lsp-installer PylspInstall code
python-lsp-server
python-lsp-black
System info (please complete the following information):

OS: Linux and OSX
Neovim version: v0.6.1

Checkhealth output (:checkhealth nvim-lsp-installer):
Linux
nvim-lsp-installer: require("nvim-lsp-installer.health").check()
========================================================================
## nvim-lsp-installer report
  - OK: neovim version >= 0.6.0
  - WARNING: **Ruby**: not available
  - WARNING: **RubyGem**: not available
  - WARNING: **Composer**: not available
  - WARNING: **PHP**: not available
  - OK: **gzip**: `gzip 1.11`
  - OK: **curl**: `curl 7.80.0 (x86_64-pc-linux-gnu) libcurl/7.80.0 OpenSSL/1.1.1l zlib/1.2.11 brotli/1.0.9 zstd/1.5.0 libidn2/2.3.2 libpsl/0.21.1 (+libidn2/2.3.0) libssh2/1.10.0 nghttp2/1.46.0`
  - OK: **wget**: `GNU Wget 1.21.2 built on linux-gnu.`
  - OK: **bash**: `GNU bash, version 5.1.12(1)-release (x86_64-pc-linux-gnu)`
  - OK: **tar**: `tar (GNU tar) 1.34`
  - OK: **sh**: `Ok`
  - OK: **Go**: `go version go1.17.5 linux/amd64`
  - OK: **node**: `v17.2.0`
  - OK: **python3**: `Python 3.9.5`
  - OK: **java**: `Ok`
  - OK: **javac**: `javac 17.0.1`
  - OK: **pip3**: `pip 21.1.1 from /home/binam/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pip (python 3.9)`
  - OK: **npm**: `8.1.4`

OSx
nvim-lsp-installer: require("nvim-lsp-installer.health").check()
========================================================================
## nvim-lsp-installer report
  - OK: neovim version >= 0.6.0
  - WARNING: **Composer**: not available
  - OK: **bash**: `GNU bash, version 5.1.8(1)-release (x86_64-apple-darwin19.6.0)`
  - OK: **tar**: `bsdtar 3.3.2 - libarchive 3.3.2 zlib/1.2.11 liblzma/5.0.5 bz2lib/1.0.6`
  - OK: **curl**: `curl 7.64.1 (x86_64-apple-darwin20.0) libcurl/7.64.1 (SecureTransport) LibreSSL/2.8.3 zlib/1.2.11 nghttp2/1.41.0`
  - OK: **wget**: `GNU Wget 1.21.2 built on darwin20.4.0.`
  - OK: **Ruby**: `ruby 2.6.3p62 (2019-04-16 revision 67580) [universal.x86_64-darwin20]`
  - OK: **sh**: `Ok`
  - OK: **gzip**: `Ok`
  - OK: **Go**: `go version go1.17.2 darwin/amd64`
  - OK: **node**: `v17.3.0`
  - OK: **python3**: `Python 3.9.1`
  - OK: **PHP**: `WARNING: PHP is not recommended`
  - OK: **RubyGem**: `3.0.3`
  - OK: **pip3**: `pip 20.2.3 from /Users/bkayastha/.pyenv/versions/3.9.1/lib/python3.9/site-packages/pip (python 3.9)`
  - OK: **npm**: `8.3.0`
  - OK: **javac**: `Ok`
  - OK: **java**: `Ok`

Replication Steps
:LspInstall pylsp
:PylspInstall python-lsp-black

Create a temporary python file

Run the formatter
:lua vim.lsp.buf.formatting()

Expected behavior
Expected (single quotes are replaced with double quotes):

Actual (single quotes are kept as is):

Workaround
DEAR PEOPLE FROM THE FUTURE: Here's what we've figured out so far (In case there is someone out there like who is specifically running neovim, nvim-lspconfig, nvim-lsp-installer, pylsp, python-lsp-black, and black :)
After I understood that PylspInstall wax creating the virtualenv in ~/.local/share/nvim/ I was able to fix my issue by uninstalling yapf and autopep8 specifically:
cd ~/.local/share/nvim/lsp_servers/pylsp/venv/bin
./pip3 uninstall yapf autopep8

It started working after that.
Possible Solutions
These are just off the top of my head, there might be other solutions as well. I also don't know what the "right" solution is.
Modify  :LspInstall pylsp  to not install with [all] extras.
python-lsp-server[all] would become python-lsp-server
:heavy_plus_sign:   The black formatter plugin won't conflict with the yapf and autopep8 when it's installed
:heavy_minus_sign:  We'll need to add extra commands that allow the user to install python-lsp-server[yapf] and python-lsp-server[all]
:heavy_minus_sign:  It won't include the linters and other packages by default (see pip freeze packages below)
Modify PylspInstall to uninstall other formatters when it installs black
:heavy_plus_sign:  "It just works" for the user
:heavy_plus_sign: It will include packages from [all] like the formatter
:heavy_minus_sign: There's no way to "undo" adding the formatter for black.
:heavy_minus_sign:  You also still run into the issue when one project uses black, but another may not.
Allow PylspInstall to specify which virtualenv to install the plugins
:heavy_plus_sign:  Pylsp can be configured to have per project configurations when these projects have virtualenvs.
:heavy_plus_sign: Can use the default virtualenv (the current one in .loacl) when one is not provided
:heavy_minus_sign: You still own't be able to run black in the default virtualenv
:heavy_minus_sign:  This is probably more of a feature request an a bugfix
pip freeze info
pip freeze for pip install python-lsp-server[all]
astroid==2.9.3
autopep8==1.6.0
flake8==4.0.1
isort==5.10.1
jedi==0.18.1
lazy-object-proxy==1.7.1
mccabe==0.6.1
parso==0.8.3
platformdirs==2.4.1
pluggy==1.0.0
pycodestyle==2.8.0
pydocstyle==6.1.1
pyflakes==2.4.0
pylint==2.12.2
python-lsp-jsonrpc==1.0.0
python-lsp-server==1.3.3
rope==0.22.0
snowballstemmer==2.2.0
toml==0.10.2
typing-extensions==4.0.1
ujson==5.1.0
wrapt==1.13.3
yapf==0.32.0

pip freeze for pip install python-lsp-server[yapf]
jedi==0.18.1
parso==0.8.3
pluggy==1.0.0
python-lsp-jsonrpc==1.0.0
python-lsp-server==1.3.3
ujson==5.1.0
yapf==0.32.0

pip freeze for pip install python-lsp-server
jedi==0.18.1
parso==0.8.3
pluggy==1.0.0
python-lsp-jsonrpc==1.0.0
python-lsp-server==1.3.3
ujson==5.1.0

Hello! Thanks for the detailed report with suggestions! I know pylsp allows toggling certain behavior by providing your own configuration (they've got it documented here).
I see they also provide a JSON schema for this configuration, so I just pushed a commit that incorporates it into the installer UI, so you can explore it from within neovim as well:

I wonder if you'll be able to do what you want to by just changing some configurations, for example setting
settings = {
    pylsp = {
       plugins = {
          pycodestyle = { enabled = false },
          yapf = { enabled = false }
       }
    }
}

Oh my gosh yes it works! That's a much better solution 😅
I was trying to do that earlier but I missed the "enabled" 🤦 . I was doing yapf = false instead of yapf = {enabled = false}
fyi - I only needed to disable yapf and didn't need to touch pycodestyle.
The UI is awesome~
You're the best, thank you so much!

@williamboman May I ask you that if I want this behaviour that is described in the link (with image): https://github.com/hrsh7th/nvim-cmp/issues/823
Then which option should I enable?

@binamkayastha And now with pylsp my completion menu doesn't show items (e.g. get) from the library requests:


Go/Nogo for release 0.24

milestone: https://github.com/CenterForOpenScience/osf.io/issues?q=milestone%3A"0.24.0+(OSF+Files%2C+Treebeard%2C+MFR)"+is%3Aclosed
changelog: https://github.com/CenterForOpenScience/osf.io/blob/develop/CHANGELOG
Deployment actions:

[x] Change Tokumx log path (to prevent OS disk from filling up).
[ ] Update python requirements
[ ] Install mfr requirements
[ ] Clear mfr cache
[ ] Clear mfr static assets website/static/mfr/*
[ ] Put up maintenance page (nginx)
[ ] Build assets
[ ] Take down maintenance page
[ ] Restart nginx
[ ] Restart celery
[ ] Restart uwsgi

Post release actions:

[ ] Manual regression testing
[ ] Update dev docs for installing/running waterbutler service

ping @JeffSpies @GaryKriebel @lbanner @caileyfitz @nicipfeiffer @brianjgeiger @caneruguz @chrisseto @jmcarp @icereval

GO :raised_hands:

GO

A long awaited, and my last :+1: GO!

:+1:

GO from Treebeard/Fangorn/Project Organizer
Harry's issue can't be replicated for now and is really minor (one line css that works with IE 5.5).
Treebeard Tests repaired and running and is now part of Travis, (build passing).

No argument -- 'Tis a GO! :+1: Woo Hooo! @nicipfeiffer
..... Let's make sure @JeffSpies gets a chance to review and weigh in -- Thanks!

The issue with water butler servers going offline was tracked down to an auto update settings and FastStart option in the CSF Firewall, both have been set disabled (h/t @chrisseto).
Also a status page was added in water butler today allowing us to monitor the workers are online and accessible via the haproxy load balancer.

Thank you @icereval  -- I know you noted it this morning -- just to reiterate for all: We've got multiple folk w/access to the status page + auto-notification to ping (via Text? Email?) if a server / workers down, correct?

Brian G. is a go according to his text message.
On Thu, Jan 29, 2015 at 6:37 PM, Michael Haselton<EMAIL_ADDRESS>wrote:

The issue with water butler servers going offline was tracked down to an
auto update settings and FastStart option in the CSF Firewall, both have
been set disabled (h/t @chrisseto https://github.com/chrisseto).
Also a status page was added in water butler today allowing us to monitor
the workers are online and accessible via the haproxy load balancer.
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/1635#issuecomment-72126841
.

@lbanner We have email notifications now. Michael will be setting up SMS notifications as well.

go given verbally

Released!

@lbanner - Yes we have multiple dev's with access to the status page, monitoring has been setup to email on warnings/changes and page if it is critical.

Thank you @icereval


Graphics card / os dependencies

I want to try this.
Will I be able to build / compile / run on a macbook ?
What about windows ?

Hi @joeblew99
For OS X / macOS you'll need to install the latest GLFW 3.3 from master https://github.com/glfw/glfw
and prepare MoltenVK https://moltengl.com/moltenvk/ SDK beforehand so CMake could find it.
There is a Makefile https://github.com/vulkan-go/demos/blob/master/vulkancube/vulkancube_desktop/Makefile to show how to properly invoke go install specifying the path to GLFW.
For Windows you don't need to compile MoltenVK and can use GLFW 3.2.1 distro from the site http://www.glfw.org then just specify paths in Makefile or run commands by hand, specifying paths to GLFW distro folders.
In all cases you will run XXX_desktop demos.

Thanks @xlab
Am disappointed they charge so much for the license.  Grumble grumble ....
Will have a play anyway.
Benchmark of OpenGL versus Vulkan Spinning the same triangle will be a useful thing

Benchmark of OpenGL versus Vulkan

Don't expect much from benchmarks at this stage, however I wonder what the difference will be.

Am disappointed they charge so much for the license.

You can use it absolutely free, but there will be a watermark. For playing around this is feasible. And MoltenVK is not a real driver like NVIDIA, so it would still be not fair to benchmark using MoltenVK.

Benchmark. I expected it to not be optomised. The reason i want to ad it is so we can run the benchmarks as we code things and se if changes to the code are impacting perf. Good regression testing best practice. Having a Spinning Triangle in golang / openGl is also a nice benchmark for Us and others to see how it compares too.
Could you add the Dependencies to the README so others also see this ?

Could you add the Dependencies to the README so others also see this ?

Done


updated Dockerfile and README.md

Hello,
after cloning the repository i wanted to try the filebin using docker and noticed that probably something is broken.
docker build -t filebin .

Here's the docker log
Sending build context to Docker daemon   55.4MB
Step 1/22 : FROM alpine:edge
 ---> 003bcf045729
Step 2/22 : MAINTAINER Sebastian Rakel<EMAIL_ADDRESS> ---> Using cache
 ---> 6fe437802c72
Step 3/22 : RUN apk add --no-cache bash php7 py-pygments py2-pip imagemagick php7-gd nodejs composer php7-pdo_mysql php7-exif php7-ctype php7-session git php7-fileinfo msmtp
 ---> Running in 9fc0618396d4
fetch https://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
ERROR: unsatisfiable constraints:
  py2-pip (missing):
    required by: world[py2-pip]
The command '/bin/sh -c apk add --no-cache bash php7 py-pygments py2-pip imagemagick php7-gd nodejs composer php7-pdo_mysql php7-exif php7-ctype php7-session git php7-fileinfo msmtp' returned a non-zero code: 1

Probably the package py2-pip needs to be replaced with py-pip.
I also noticed that the file docker/README.md had a broken link to the Dockerfile so i added that change too.
The MAINTAINER reference is also depreciated in Dockerfiles so I changed that to the LABEL maintainer=
Best regards.

Thanks! I also found another issue with the database initialization during the container startup that should now be fixed.


Add Dry out x Mr.Beast by Lopzinc

Song Name
Dry out x Mr.Beast
Artist Name
Lopzinc
Source
Youtube
Youtube Link
yvq7CMAGQWDwswY0
Direct File Link
No response
Song ID
3
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 4

reject
invalid yt link


Add helper function to store/retrieve partition config from context

What changed?

Add helper function to store/retrieve partition config into/from context
Update frontend workflowHandler to retrieve partition config from context and set it to the requests to history
Add a middleware (ZonalPartitionConfigMiddleware) as an example to demonstrate how to store partition config into context
Update clusterRedirectionHandler to set a header to indicate which cluster the requests are from for all auto-forwarding requests
Add a middleware (ForwardPartitionConfigMiddleware) which forwards partition config to remote cluster

ForwardPartitionConfigMiddleware must be used if we want to enable tasklist isolation feature. ZonalPartitionConfigMiddleware can be replaced with a different implementation. For example, if zonal isolation is enabled at network layer, we can create a middleware which derives the zone which the requests are from on the server side, which is more reliable than letting client set a header indicating which zone the requests are from.

Why?
To provide a way to set isolation groups for frontend requests

How did you test it?
unit test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01880309-10ab-4eeb-a50b-7aabaeeeb457

97 of 103 (94.17%)  changed or added relevant lines in 4 files are covered.
77 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.02%) to 57.197%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/frontend/clusterRedirectionHandler.go
33
35
94.29%

common/rpc/middleware.go
35
39
89.74%

Files with Coverage Reduction
New Missed Lines
%

service/history/queue/timer_queue_processor_base.go
1
77.92%

common/task/weightedRoundRobinTaskScheduler.go
2
89.12%

service/matching/matcher.go
2
90.65%

common/persistence/statsComputer.go
3
94.64%

service/history/task/task_util.go
4
75.14%

service/history/task/transfer_standby_task_executor.go
4
86.93%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
6
59.1%

common/persistence/nosql/nosqlplugin/cassandra/workflowUtils.go
21
74.32%

common/persistence/nosql/nosqlplugin/cassandra/workflowParsingUtils.go
34
79.18%

Totals

Change from base Build 018802e6-6913-48bd-a84d-dbf07b294aa6:
-0.02%

Covered Lines:
86193

Relevant Lines:
150696

💛  - Coveralls


instaloader: 4.9.6 -> 4.10.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for instaloader is: Download pictures (or videos) along with their captions and other metadata from Instagram
meta.homepage for instaloader is: https://instaloader.github.io/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.10.3 with grep in /nix/store/ca3xjrnni7ycmlsvn0lkf5hxa58brpky-python3.11-instaloader-4.10.3
found 4.10.3 in filename of file in /nix/store/ca3xjrnni7ycmlsvn0lkf5hxa58brpky-python3.11-instaloader-4.10.3

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
instaloader

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/1fc17df80cc872e1551237be14641b64590fba85#instaloader

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ca3xjrnni7ycmlsvn0lkf5hxa58brpky-python3.11-instaloader-4.10.3
ls -la /nix/store/ca3xjrnni7ycmlsvn0lkf5hxa58brpky-python3.11-instaloader-4.10.3/bin

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

1 package built:

instaloader

Maintainer pings
cc @Creator54 for testing.

LGTM


Configuration with react

Can anyone please give me an example of using this plugin with React?
Have no idea about how to configure it, thanks!

Please use stackoverflow or gitter for questions


Using Procfile.dev tries to parse as a Proxfile

Hi there,
Just tried to start up prox using a file called Procfile.dev and I get the following output:
$ prox --procfile Procfile.dev
[ERROR] Failed to parse Procfile: failed to decode Proxfile as YAML: yaml: unmarshal errors:
  line 1: field web not found in type prox.Proxfile
  line 2: field worker not found in type prox.Proxfile

Hi Drew, the issue seems to be that prox is only trying to parse the Procfile format if the name exactly matches  Procfile: https://github.com/fgrosse/prox/blob/main/cmd/prox/main.go#L106-L108
I'm currently rather busy but feel free to open a PR to change this to a prefix match instead if it helps your usecase :)

I just released v1.1.0 which includes the improved behavior.


Add pin override for NUCLEO_F429ZI.

This is needed as the regular configuration does not work with this board.

A quick question, is this only needed when ethernet is in use? It seemed to work fine in the default pinout otherwise.

The image looks good, however the CI test shield is not publicly available nor there is requirement to use it.
Any SD card reader would work, as long as these lines are available through the SPI bus: SCLK, MISO, MOSI, CS. This is the key info to add to the docs.

Hey @JanneKiiskila, @Timjun01, this is low priority, but we're confused why this workaround is needed.
We followed the pins on the CI shield, but it looks like nothing conflicts with the NUCLEO board if you're only using the SD card. Is there a conflict with another peripheral?
cc @kegilbert


[ChannelTabs] Tabs don't update.

Before you submit an issue please restart your discord. If the issue is still there feel free to write a bug report.
Describe the bug
A bit too confusing to explain, but, when you change channels in a tab it doesn't update. When you swap to that tab it sometimes sends you to the channel it shows, not the last you've opened.
To Reproduce
Open a new tab(it would be your friends list)
Switch to some channel in the newly opened tab.
Switch to a different tab and then back to your new one. You're back in your friends list.
Expected behavior
The tab header changes to whatever channel you switched to, and when you switch to that tab it stays on that channel.
Screenshots

The right most tab is active. But it shows a different channel. When i swap from it and back, it returns me to the channel it shows on the top, not the latest channel i had open.
Information (please complete the following information):

OS: windows

Plugin Version: 2.6.13

Discord version:  stable 328697 (38204c9) Host 1.0.9163 x64 (52153) Build Override: N/A Windows 11 64-bit (10.0.22631)

Compact Mode enabled? no

Update: I rebooted my client, and it stopped working completely.

can confirm it doesn't work anymore

can confirm, too. Changing the location doesnt update the tab location. Switch to a new tab and than back, sends me back to an old location.
That's the error which came up when chaning the location in the tab
web.cde6776….js:12 [PluginManager] Unable to fire onSwitch for ChannelTabs v2.6.13

 TypeError: Cannot read properties of undefined (reading 'getIconURL')
    at getCurrentIconUrl (ChannelTabs.plugin.js:1306:20)
    at eval (ChannelTabs.plugin.js:3438:20)
    at Array.map (<anonymous>)
    at ChannelTabs.onSwitch (ChannelTabs.plugin.js:3430:43)
    at N.onSwitch (renderer.js:5:30384)
    at a.emit (renderer.js:5:368671)
    at a.dispatch (renderer.js:5:19988)
    at renderer.js:5:20161
    ```

Can only upload/share their config files for those that deleted the plugin and reinstalled it?

For those who want to keep using the plugin till its update you can change Line 1304 From:
return guild.getIconURL(40, false) || DefaultUserIconBlue;

To:
return guild ? guild.getIconURL(40, false) : DefaultUserIconBlue;

So the icons will be default ones, but its useable again.

Sweet work around. I'll gladly give up server icons for functionality until this is officially patched.

@samfundev This is a critical break, any chance you can take a look at this?

same here
also can't open a favorite in a new tab by right clicking it

Also same issue, across all my devices which use the plugin

More proper temp fix here:
Open ZeresPluginLibrary, go Line 381
change it from:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds");},

to:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds", "getRoles");},

should also fix multiple other plugins at the same time.

More proper temp fix here: Open ZeresPluginLibrary, go Line 381 change it from:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds");},

to:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds", "getRoles");},

should also fix multiple other plugins at the same time.

this fixed other plugins for me but didnt fix channeltabs

More proper temp fix here: Open ZeresPluginLibrary, go Line 381 change it from:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds");},

to:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds", "getRoles");},

should also fix multiple other plugins at the same time.

This fixed it for me!

More proper temp fix here: Open ZeresPluginLibrary, go Line 381 change it from:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds");},

to:
    get GuildStore() {return _webpackmodules__WEBPACK_IMPORTED_MODULE_1__["default"].getByProps("getGuild", "getGuildIds", "getRoles");},

should also fix multiple other plugins at the same time.

Thanks, fixed it for me! :)

The fix for this has been released in v2.6.14. It has not been approved yet, but feel free to download the latest version in the mean time.

Fix did not work for me with just the 2.6.14 update; I also had to apply the Zeres Lib fix from https://github.com/samfundev/BetterDiscordStuff/issues/12#issuecomment-2395289463


Typo in 'Business continuity and database recovery - SQL Server'

Typo, can't edit document as content isn't on github.
Typo: "availability group as one of the instances hosing either a primary or secondary replica"
Fix: "availability group as one of the instances hosting either a primary or secondary replica"
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 76208061-3c5d-758a-6704-dd7f546d5545
Version Independent ID: 6ea99630-2750-17b2-b64b-b6fa8464e1dc
Content: Business continuity and database recovery - SQL Server - SQL Server
Content Source: docs/database-engine/sql-server-business-continuity-dr.md
Product: sql
Technology: availability-groups
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@connormoolman -- Connor, thank you for catching that.  I'm updating the article.

@WilliamAntonRohm -- Hi, it still seems like the Typo is there in case you have forgotten, cheers.

@connormoolman Thank you for your contribution. The typo in question was in a difficult-to-find file and it took several attempts to get a fix in. I ended up having to do it through an internal process. You should see the correct text in the next day or so.


Stricter pinning for occt

Must rebuild smesh because OCCT 7.7.1 breaks ABI compatibility. See https://github.com/conda-forge/occt-feedstock/issues/97

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@looooo when you have time please check this one

@conda-forge-admin, please rerender

I now have permission to merge but I see some checks failed, @realthunder do you know how to fix this?

@conda-forge-admin, please restart ci

We also need to patch @looooo 's fork of SMESH repo.

@looooo does not seem to be available to merge stuff, and I think it would be best to get these things merged upstream anyways. Do you mind making that PR to https://github.com/trelau/SMESH instead? As a workaround this is now pointing to your smesh repo but I will change to point again to upstream once we get that merged there.


[Looking for CSS Expert] The Heartbeat bar so strange in some browsers and DPI

I called it a "beat" for each green block. Beat is a div with width/height. However, in some cases, the beat's width is different from each others. No matters how I set the width, it do not work with all DPI.
Enlarge image: 1920x1080 @ 100dpi  (Windows Server 2019  / Chrome via RDP)

https://github.com/louislam/uptime-kuma/blob/f84135ca675944fd59f2927d5fdd747fc9eba2c6/src/components/HeartbeatBar.vue#L95-L102
Edit: JSFiddle: https://jsfiddle.net/tqkermc5/21/

In CSS, "px" doesn't actually mean device pixels, instead the browser will handle display scaling and rendering to the screen, and if the elements does not align to physical pixels, do sub-pixel rendering to improve the result. Since you used non-integer values for the width, it's very likely that the start of each beat does not align to the start of the physical pixels, and somehow the sub-pixel rendering does not handle this situation well. Here I tested width: 6, margin: 2 and IMO it looks better and does not suffer from the problem.

original
integer width & margin

Yes, width: 6, margin: 2 is good for 100DPI
However, 125 DPI becomes like this.

Also, tried many other units, no luck to make them to be even width of all beats in all DPIs.

Added
https://jsfiddle.net/tqkermc5/21/

It seems that there is no universal solutions for this, I handled by each DPI.
https://github.com/louislam/uptime-kuma/commit/cc25787878c37655ea6ce0cfe4d4fb1e3f224252


Heuristic sampler vs. PhenotypeFilter

Both MTC_Strategy.HEURISTIC_SAMPLER and genophenocorr.analysis.PhenotypeFilter use CohortAnalysisConfiguration.min_perc_patients_w_hpo property to remove the HPO terms present in fewer patients than the threshold value.
We seem to be doing the same thing twice.
Let's investigate if these are two separate concerns or if we should remove PhenotypeFilter.

I think that for simplicity we should just offer the HeuristicFilter, but let's call the Heuristicfilter the PhenotypeMtcFilter, that is a better name. So -- we should
1- Delete the Phenotype filter
2. Rename HEURISTIC_SAMPLE to PHENOTYPE_MTC_FILTER

Fixed by #221


Video plays all at the same time

When I put the video data with in a list of 2+ items, when you click in one of them with autoplay=true all videos plays at the same time.
Also I noticed the id for  tag is the same for all.

Hi, I will look into it, that's a serious issue.

Confirming the issue, happens to me as well.

Think you can fix it @GenaBitu ?

Closing for inactivity - should be fixed already.


ViewPager as stream info with swipe support

Closed as I will implement a user interface better suited for more streams


Still in active development?

Looks like not much weekly activity on master since June. Project still going strong?

It's actively used and supported.
What about development:

I'm waiting for spreading GATs to fix borrowing fetching API (https://github.com/loyd/clickhouse.rs/issues/24).
I'm planning (and have already performed some groundwork) to move to RowBinaryWithNamesAndTypes in order to support conversions between types (https://github.com/loyd/clickhouse.rs/issues/10).


Modify the UI of events log switch tab

Describe the bug
Currently the switch tab of events & daily road and filters button in the consultation page are utilizing two separate rows, unnecessarily
To Reproduce
Steps to reproduce the behavior:

Go to the patient tab
click on a active consultation
See error

Expected behavior

[ ] keep the daily round & events and filter button to a single row
[ ] Remove the sort-by button in the events green border to make it similar to the filter button in daily rounds
[ ] When the sort by is active in events, make it the primary color to show sort by is active

Screenshots

https://github.com/user-attachments/assets/42e51659-108f-4f5f-81bb-ce4e391b7ce5
I have brought them to single row by creating filter ui elements in the parent component (removed the logic related to them in child components where they were present initially) and passed the corresponding query values of both filter ui as props to components which render the result obtained from filtering. Adjusted the grid to 3 columns. Please let me know if this is what you wanted ? and happy to modify if anything @rithviknishad

The "sort by" and "filter" is NOT a tab right? Can we keep it outside the tab switch?

The "sort by" and "filter" is NOT a tab right? Can we keep it outside the tab switch?

https://github.com/user-attachments/assets/9a32ff26-1b2f-42e5-a1ed-f6ff424a4851
There is some inconsistency in the appearance of filter buttons. How do we make it better ? any inputs @rithviknishad

We could use a Popover with 3-dot menu or filters icon for both. And keep the sort by as an option (select menu) in the popover instead.

We could use a Popover with 3-dot menu or filters icon for both. And keep the sort by as an option (select menu) in the popover instead.

is this how you wanted @rithviknishad ? Let me know if any changes required.
https://github.com/user-attachments/assets/d098d94f-47b3-4176-a166-bb1a0b8dc4b3

This seems to be nested popovers which was not what I intended.
Let's inline the filter contents to the first popover itself.
Also could you make the 3-dot menu to use to <ButtonV2>? The current button doesn't seem to be clickable.

This seems to be nested popovers which was not what I intended.
Let's inline the filter contents to the first popover itself.
Also could you make the 3-dot menu to use to <ButtonV2>? The current button doesn't seem to be clickable.

Sure ill change.

Checkout SelectMenuFormField. Lets use Popover for both sort by and filter. The sort by popover can contain SelectMenuFormField as it's child.

Also let the 3-dot button's variant be "secondary", also ensure it's square.

Hey @JOSHIK27
Any updates on this? Stuck somewhere?

Apologies for the delay. I could not find SelectMenuFormField anywhere in neither headless ui elements nor the codebase. Checked with headlessui documentation. Am i missing something ? Could you please direct me with this element. I'll give u the update about pr eod.

My bad, it's SelectFormField

My bad, it's SelectFormField

thank you. Will look into it

@nihal467 whats the status here? Also link the PR

@bodhish i think you mis tagged me, @rithviknishad @JOSHIK27 can you share the status on this issue and link the PR

@JOSHIK27 the recording shared in the last comment looks fine to me


Back to dev again

PR checklist
Merge of patch content from master and version bump to 1.1dev

[x] This comment contains a description of changes (with reason).
[x] Make sure your code lints (nf-core lint).
[x] CHANGELOG.md is updated.

I am a bit confused is to a version bump from a new release back to dev or from a prep for a new release?

We had a patch because of Megatest issues, then when we merged the changes back into dev the version number 1.0.1 got merged as well and we need it to be 1.1dev here for the time being

Template merge hell has started here, can someone more experienced help?

Force-merging with failing tests – the errors are due to unmerged 3.0 template


Fix up PHPStan errors.

Road to level 8

The other changes look good to me.

Codecov Report
Base: 88.56% // Head: 69.15% // Decreases project coverage by -19.41% :warning:

Coverage data is based on head (34752b5) compared to base (011aaa7).
Patch coverage: 52.63% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             master    #1955       +/-   ##
=============================================
- Coverage     88.56%   69.15%   -19.41%
+ Complexity     8051     8049        -2
=============================================
  Files           243      240        -3
  Lines         24590    24546       -44
=============================================
- Hits          21777    16976     -4801
- Misses         2813     7570     +4757

Flag
Coverage Δ

5-max
69.15% <52.63%> (-19.41%)
:arrow_down:

7.4
69.15% <52.63%> (-19.41%)
:arrow_down:

agnostic
?

mysql
69.15% <52.63%> (+0.07%)
:arrow_up:

pgsql
?

sqlite
?

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

Impacted Files
Coverage Δ

...ivable/ArchivableBehaviorObjectBuilderModifier.php
0.00% <0.00%> (-97.47%)
:arrow_down:

...hivable/ArchivableBehaviorQueryBuilderModifier.php
0.00% <0.00%> (-95.75%)
:arrow_down:

...l/Generator/Behavior/Delegate/DelegateBehavior.php
0.00% <0.00%> (-89.52%)
:arrow_down:

...nable/VersionableBehaviorObjectBuilderModifier.php
0.00% <0.00%> (ø)

src/Propel/Generator/Model/Column.php
79.56% <ø> (-16.40%)
:arrow_down:

...Propel/Generator/Model/ConstraintNameGenerator.php
0.00% <0.00%> (-85.72%)
:arrow_down:

src/Propel/Generator/Model/Diff/ColumnDiff.php
75.00% <0.00%> (-21.97%)
:arrow_down:

src/Propel/Generator/Platform/PgsqlPlatform.php
0.00% <0.00%> (-97.57%)
:arrow_down:

...e/ActiveQuery/Criterion/AbstractModelCriterion.php
21.21% <0.00%> (ø)

...ActiveQuery/SqlBuilder/AbstractSqlQueryBuilder.php
97.72% <ø> (ø)

... and 157 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@mringler @dereuromark can we move forward here?

@michbeck It looks like @dereuromark has left the project, and it looks like there is no successor at Spryker at the moment. I have contacted the owner, and he told me that the project will keep being maintained by Spryker, but I haven't heard back if/when there will be someone who actively maintains Propel2, i.e. merges this PRs.
For what it's worth, I am very sorry for the situation! I really appreciate the work you put into this in the previous PRs, and I would have really liked to see this merged, preferably in acceptable time.

Best wishes

@mringler thank you for the reminder and your patience! ❤ I really hope you'll keep contributing after this, I'm really sorry that you had to wait, I know from personal experience that it is frustrating when your contributions won't move forward. 😞
I'll take this over temporarely until I figure out who will be our new internal owner. Your PR looks good to me! 💪 There is just one comment from Mark, to which I agree, that needs to be fixed but we could do that as well.
@michbeck can you maybe take care of it tomorrow? I'll approve the merge as soon as this has been fixed by someone.

@gechetspr this is ready for merge now.


[RHCLOUD-20671] refactor: centralize tenancy lookups on the identity struct

Until now, we were "individually" looking up the tenants in three ways:

By creating an identity struct from the EBS account number.
By creating an identity struct from the OrgId.
By passing the identity struct directly.

Since the third way includes the two above, I thought it might be a good
idea to just use the identity struct for everything, and to forget about
pulling specific data for the tenancy.
This will make the "x-rh-identity" header the favoured in case we
receive it along the "ebs account number" and "orgid" headers.
Links
[[RHCLOUD-20671]

This needs rebase.

I like this refactor - if we can only focus on one field in the context that would be better. I think the ultimate refactor in this line is combining this + our internal PKs for tenant/user into a struct. But this is a good first step.

Yep, initially I developed it that way, but I think I was going off rails with the purpose of the PR, so I decided to just stick to this little refactor.


What about incomplete dates

That happens in Muskrat captures in Flanders, Belgium (year only)

The current solution is to assign them to the first day of the year, which is imperfect.

Some possible approaches at: https://stackoverflow.com/questions/2971198/how-to-deal-with-partial-dates-2010-00-00-from-mysql-in-django and https://softwareengineering.stackexchange.com/questions/194286/how-do-you-store-fuzzy-dates-into-a-database
Might be useful to store all occurrence dates as range (when querying, see if occurrence range and query range overlap?)

Note that this can affect the assignment to a occurrence over time chart: #22


You can share your dataset ?

... You can provide dataset ???
Thanks you

The dataset is included in the data folder.
There is no real dataset link available, you have to sign a release agreement and send it to a certain link, the agreement can be found here: http://www.cbsr.ia.ac.cn/english/FASDB_Agreement/Agreement.pdf

Hi thanks you I have a
ROSE dataset but accuracy is very low ??? I using Face_aligment ???
ROSE dataset provide many videos with other FPS (each video I  get 10 frame)

Hi, I have a problem to get face anti-spoofing data. Can anyone share them?
Many thanks.


Luke: Data retriever service

Added new field to define_replay

This is not the architecturally correct way to do it. For LCA this is ok, but in the longer term, we need to manage our data archives more explicitly. We had lots of design for this in R1. Let's revisit after LCA.


Collect stats on sample_caller runs and optionally report them

This PR adds light instrumentation to the sample_caller runs, where exceptions and total items scraped are counted for each scraper ('module_string'). At the end of the run, if the--report option is enabled, a report.html file is created in the project root directory.
An example of a real report that this generates is here
I believe this adds obvious value over trying to run sample_caller one scraper at a time and "see" if it failed or not.
Also jinja2 is added only as a dev dependency.

Update: fixed a bug with the number of "non-error" reports and a CSS bug.

Solid, and super helpful. @arderyp, you might find this interesting/useful. New flag on sample_caller.py that'll generate a report!

yeah, thanks @audiodude for doing this, its an awesome feature!

@mlissner is this deployed and will our email/slack report include the new info?

Well...it's available as part of sample_caller.py, but it's not something that's in the daily emailer. We could probably upgrade one with the other though.
Another thought is that github actions allows you to run code periodically. Maybe it's doable to run this that way once/day (or faster), and to upload the report somewhere...somehow?


Pathplanner closes immmediately

Whenever I open PathPlanner, the app closes instantly. It doesn't matter if I use the one from Github, AppStore, or directly build it from source. Does anyone knows how to fix this?
The only "insight" I have is that when I build it from source, it opened normally, but as soon as I opened a robot project, it closed. It is important to note that the file I chose was the deploy file, by error. Other times I have used Pathplanner, I have correctly opened everything

I am using Mac M1 by the way

Can you run defaults delete com.mjansen4857.pathplanner in the terminal then try again.

It could also be a corruption in one of your existing path files in the deploy folder for that project. This fits with the symptoms you are describing above.
Try taking all of the files out of the deploy folder and restarting the app.  If that works, close PP and copy one path back in at a time, opening and closing PP each time.

If you're building from the current main branch that shouldn't be an issue anymore but for older versions, yeah.

Yeah I did all of the above and it worked. Thanks a lot!


add only valid json option when parsing json.

Hi,
this is my attempt to fix #121 #79 #65 #52,

@lucacervello hmm.. I am concerned that this approach incurs the cost for the check for all parsing.
How would you feel about adding a new parser in core that does the check, and the other parsers can continue to ignore it? Maybe parse-exact?

@dakrone I understand your concern and i'm ok with adding parse-exact to core.

@lucacervello great, is that something you'd like to do as part of this PR?

@dakrone yes definitely.

@dakrone is only one parse-exact enough ? or we want:

parse-string and parse-string-exact
parse-string-strict and parse-string-strict-exact
parse-stream and parse-stream-exact
parse-smile and parse-smile-exact

If we want an exact function for every parse
Maybe a macro with-exact or exact that wrap that parsing functions is a better solution.
Something like this
(with-exact
  (parse-string "{\"foo\": 1}"))

I think with-exact can works but i'm not a macro expert 😄

I think that would work, as long as the macro simply added the validation after the parse-string function instead of modifying it directly. Want to give it a shot?

Yes, sure.

@dakrone i think the "macro approach" can't work. I need to access the JsonParser object and i can't from outside.
Maybe an exact function for every parse is a better idea.

Okay, I'm okay either way. If we do have to duplicate each, maybe we can just put them in a different namespace and name them the same thing, rather than add more things to the core one?
Maybe cheshire.exact? What do you think?

I'm okay with it. I'll give it a try.

@dakrone I've done only parse-string and parse-string-strict.
I don't know if translate parse-stream and parse-smile make any sense.
There are cases where the stream isn't a valid json ?

Merged this, thanks so much @lucacervello! I'll release 5.8.0 with this shortly


[RESOLVED] S100-Santiago has been restarted (uptime < 10m)

Problem started at 21:01:00 on 2023.05.28
Problem name: S100-Santiago has been restarted (uptime < 10m)
Host: S100-Santiago
Severity: Warning
Operational data: 00:00:16
Original problem ID: 22883
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23522&eventid=22883

Problem has been resolved in 10m 2s at 21:11:02 on 2023.05.28
Problem name: S100-Santiago has been restarted (uptime < 10m)
Host: S100-Santiago
Severity: Warning
Original problem ID: 22883


Error

Hi, I'm Rishi K. Bose and I use your program for my laptop with UEFI. However, the program has a date when it will expire.

Hi @rishikbose25,
Yes, the machine owner key certificate will expire after a given time, by default it seems to be 100 years, so please check your system clock is correct. You can check the validity period using:
sudo openssl x509 -text -in /var/lib/shim-signed/mok/MOK-Kernel.pem

        Validity
            Not Before: Oct 29 11:15:23 2021 GMT
            Not After : Oct  5 11:15:23 2121 GMT

If you customised the name of the certificate, you will need to provide the path to the MOK pem bundle you use to sign kernels with.
If your certificate has expired, then you will need to recreate the MOK using the Creating a MOK for kernel signing.
Please note, you will also need to enrol the key into the shim and then re-install any kernels you want signed with the new key. You can alternatively manually sign the kernel images. You will need to sign the kernel images with the new key after you've created the new key and imported it into the shim using the mok-setup.sh script.

Also, can you please give me the error that you're receiving?

Thanks.

Please let me know if you have an error to share, I'll close this issue for now.

@rishikbose25 I just had a thought, I guess it's possible you're using the old version of the script. The mainline server address changed and the script was updated. It might give an error that the TLS certificate of the old server is invalid prior to throwing a 404 error. Please update your signing script from the main branch and it should resolve the issue for you. The information is at the top of the README indicating that issue. Cheers.


Cannot 'Create a Slack Bot User'

Following the documentation, I have deployed my chatbot on Azure and am trying to connect this chatbot with Slack using the Azure Bot Service Portal. There is a mismatch between what I can do when creating a Slack app and the steps specified by the documentation. The relevant part of the documentation is at: https://docs.microsoft.com/en-us/azure/bot-service/bot-service-channel-connect-slack?view=azure-bot-service-4.0&tabs=abs.
Details

The documentation has a step 'Create a Slack Bot User'. Slack refers to the bot user feature being in a new place - App Home:

Upon clicking on the App Home tab, there is no bot user feature section. Instead,
Slack states that I first need to assign a scope to my bot token:

The current Microsoft documentation does not specify what scopes are required.
Thank you in advance for your help on this issue - it is much appreciated.

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

ID: 97b7eb7d-843a-774e-3ee4-8f3126cfea20
Version Independent ID: 5d501ffc-819f-0634-671c-13d432aa7f28
Content: Connect a bot to Slack - Bot Service
Content Source: articles/bot-service-channel-connect-slack.md
Service: bot-service
GitHub Login: @arturl
Microsoft Alias: arturl

Duplication of issue https://github.com/MicrosoftDocs/bot-docs/issues/1897. The problem has been addressed.
The fix will be available in the next updated documentation

The next documentation update to be made available soon.


Make link to programmatic auth more obvious

PR summary
The README had some links that pointed at external URLs about other authentication modes rather than env var configuration but it was not so obvious that includes the programmatic auth mode as well. The aim of this PR to highlight those links by creating a new section for them.
I also considered it is time to refactor the README file a little because the lines were getting longer and longer, and it was tough to read the lines at maintenance/review. The long links now also have a reference instead of inline linking. I was trying to keep the <80 char/line length, I broke this rule only if an inseparable link was in that line.
I put full stop where it was not before, and fixed some grammar issues.
I also found some markdown linting  and go fmt issues:

missing whitespaces from the beginning of a line if the list item was broken down into 2 lines
go fmt found unnecessary whitespace in the examples

PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the
Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
The README had some links that pointed at external URLs about other authentication modes but it was not so obvious that includes the programmatic auth mode as well.
What is the new behavior?
The README contains a new section that highlights the links of external configuration and the programmatic auth mode.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

disregard my comments below if they fall under the linting issue "go fmt found unnecessary whitespace in the examples".

Yeah, they fall under it.
Thanks for the reviews @emlaver and @ricellis! I'm now about to merge this


upgrade systemd to 239

systemd 239 was released on Jun 22. Would be nice to upgrade before we fork off 18.09.

@Mic92 has already opened a PR at NixOS/systemd#21.

I am finished with the pull request. I will add it to nixpkgs, once my changes are merged into the fork.

systemd is now in staging: https://github.com/NixOS/nixpkgs/pull/44451

Great, thanks! Let's keep this issue open until it hits master.

it's in master now, and did hit unstable channels too if I'm not mistaken.

Not yet released to the channel, but on master which is enough to close this! :beers:


Consider integrating with NativeScript?

Hi, thanks for the initiative of starting this project. I too am looking for an approach to build cross-platform native desktop apps with TypeScript/JavaScript and Angular.
I was wondering, have you considered integrating your efforts of building a NodeGUI Angular Renderer with the NativeScript project?
This could really be a game changing opportunity to elevate the use of these technologies to build native apps for any device!
Thank you for your initiative creating this project. I will be following closely :-)

Qt5 is supported for mobile app development as well, so integrating with NativeScript wouldn't make any sense.