[Doc feature][RSC bots Phase II]

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
85
7
link
:white_check_mark:

msteams-platform/whats-new.md
88
33
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 7c4c9db:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
85
7
link
:white_check_mark:

msteams-platform/whats-new.md
88
33
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit c2c4430:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
85
7
link
:white_check_mark:

msteams-platform/whats-new.md
88
33
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 02f1c53:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
85
7
link
:white_check_mark:

msteams-platform/whats-new.md
87
36
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 45716f0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
85
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 9825755:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-messages-with-rsc.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Closing the PR


Conditions in loose threads loop does not take into account the diffe…

…rence between backtraces from ruby 2.0.0 and greater than (Fixes https://github.com/celluloid/celluloid-io/issues/165)

Nice work this is awesome.


Request for Comments: Added StringBuilder

RfC
This is a very early draft for an implementation of StringBuilder.
Initially I wanted StringBuilder to pre-allocate a list (with either a good default value or based on a parameter)
and use PyList_SET_ITEM.
typedef struct {
    Py_ssize_t _lst;
    Py_ssize_t _cap;
    Py_ssize_t _idx;
} HPyStringBuilder;

I changed it to use to match ListBuilder and TupleBuilder which are passed by value.
This forced me to use PyList_Append.
Checking the cpython implementation, it doesn't seem to be as bad as I initially thought, because
list_resize already has a
strategy for overallocation.
Just after finishing my branch, I started reading the implementation of HPyTracker which is similar to my initial idea.
On big difference between HPyTracker and this implementation for StringBuilder is that it maintains an array of HPy handles, while this implementation still uses a cpython list as a PyObject which stores other PyObject instances, similar to the other builders.

don't worry about opening PRs on the main hpy repo, you are very welcome to do so and it's easier for us to discuss design, ideas and implementations. Some random comments:

I think it should be called UnicodeBuilder, and we will also need BytesBuilder (no need to implement both at the same time, it's fine to do two separate PRs)

the most important thing to the right is the user facing API: the one you propose look reasonable, but I think it should be possible to pass a size to preallocate the buffer. E.g., currently users can call PyUnicode_FromUnicode(NULL, 10) and then fill the buffer manually: with HPy, this will be forbidden, so UnicodeBuilder needs to support this use case

the concrete implementation on CPython is less important at this stage, IMHO. Using a list of strings is good for now, but maybe in the future we will want to try different implementations (e.g. allocating a fixed-size buffer and fill it directly, using a StringIO object, etc. etc.)

as you say, you can use an approach similar to HPyTracker if you need to store multiple fields for a single builder.

OK, thanks a lot, I'm going to move discussions to the main project.


parser: support window function ast

This PR supports all window function ast.
PTAL @zz-jason @lzmhhh123 @winoros

Would you please refer the TiDB PR ? @lamxTyler
https://github.com/pingcap/parser#how-to-update-parser-for-tidb

Would you please refer the TiDB PR ? @lamxTyler
https://github.com/pingcap/parser#how-to-update-parser-for-tidb

You can use the parser branch from @lamxTyler 's repo before this PR is merged:
GO111MODULE=on go replace mod edit github.com/pingcap/parser=github.com/lamxTyler/parser@window-func-ast

@lzmhhh123

@lzmhhh123 It depends on you. If the planner needs to know if the select stmt contains any window function expression, then I will add it for you.


feat(adp-tooling): add missing adp config

#1611
https://github.com/SAP/open-ux-tools/issues/1901
Adaptation Tooling Writer for Migration
Consume the configuration settings needed in ui5.yaml and package.json in order for theVisual Editor and npm run start-editor to work in BAS.

Thanks @tobiasqueck

Please also add minify as a excluded task to the build script

Added it to the templates

Can you address the 5 sonar issues? They are for unused imports. Thanks

Thanks @docirl , i missed this.

Thank you for the reviews and testing @tobiasqueck @docirl  @devinea


add another table to store global level experiment data

We want a high level table that includes information about participants, id numbers, conditions, age, etc.
Method 1: Pass in the name of an experiment table
This is perhaps the easiest option. If you have a spreadsheet of filenames, participant id's, ages, conditions, etc, then you can pass in the filename so that it will be imported. This extra information can then be used so that the toolbox will essentially export a similar file, but with extra discounting-related information appended to it.
myData = Data(datapath,...
	'files', filenames,...
	'metaTableFile', path_to_experiment_spreadsheet);

so you would have a spreadsheet (.csv format) with something like this...

filename
id
gender
condition

AA-male-A.txt
AA
male
A

JD-male-A.txt
JD
male
A

MP-male-B.txt
MP
male
B

KR-male-B.txt
KR
male
B

OP-female-A.txt
OP
female
A

EW-female-A.txt
EW
female
A

WK-female-B.txt
WK
female
B

CD-female-B.txt
CD
female
B

Requirements

The first column must be called filename, and currently has to include the extension.
The number and names of files in this file, and the list of files passed into Data must be the same.

Method 2: Passing in a table
You can pass in a pre-built matlab Table. This might happen if you have a participant filename scheme which contains information about participant id, gender, age, condition, etc. You can then write your own function to parse all the filenames and build a matlab Table. For example, if you have a naming convention for your discounting files, such as:
AA-male-conditionA.txt
JD-male-conditionA.txt
MP-male-conditionB.txt
KR-male-conditionB.txt
OP-female-conditionA.txt
EW-female-conditionA.txt
WK-female-conditionB.txt
CD-female-conditionB.txt

Then you can write your own function to parse these filenames to result in your own table, eg. metaTable = filenames2table(filenames). In this case it might look something like:

filename
id
gender
condition

AA-male-A.txt
AA
male
A

JD-male-A.txt
JD
male
A

MP-male-B.txt
MP
male
B

KR-male-B.txt
KR
male
B

OP-female-A.txt
OP
female
A

EW-female-A.txt
EW
female
A

WK-female-B.txt
WK
female
B

CD-female-B.txt
CD
female
B

So you'd implement this in this way
metaTable = filenames2table(filenames);
myData = Data(datapath,...
	'files', filenames,...
	'metaTable', metaTable);

This approach is probably only sensible if you have categorical variables that you care about. It would get very messy to encode numerical values in a filename coding scheme.
Todo

[x] enable table to be stored in Data
[x] enable automatic exporting into a .csv file
[x] build in some asserts, because the Table being passed in must fulfil certain requirements
[x] make this alt param estimate file default behaviour, even if no metaTable inputs are provided. Do this by just setting the filename as the only column in the table
[ ] document on the wiki

This is working reasonably well now. We get an additional table exported (csv file) which includes any experiment-level variables (like condition, id, etc) as well as the discounting parameters and posterior predictive information all in one. This is nice as we can import this file directly into JASP (for example) to run analyses


[MM-46694] A/B Test: welcome post

Summary

Adds a new custom post handler for the welcome post
The welcome post content varies if the user is an admin or not
Adds track event on certain slash commands

Ticket Link
https://mattermost.atlassian.net/browse/MM-46122
Related Pull Requests

Has server changes

Screenshots
Admin:

User:

Release Note

NONE

/update-branch

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

/cherry-pick cloud

Cherry pick is scheduled.


[Feature]:  component

What problem does this feature solve?
Introduction of a new component to visualize progress in new ways as opposed to our data bars.
What does the proposed API look like?
Component to be coded only using html / svg. Proportion and size details can be found in the image below.
Explicit props:

value: number [0-100]
size?: string ["xs", "sm" (default), "md", "lg", "xl"]
color?: string // please use same color mapping logic as in ProgressBar
showLabel?: boolean
showAnimation?: boolean // please use CSS animation (fill up to threshold, same as in ProgressBar)
tooltip?: string // please use same logic as in Icon

for className prop it should be possible to adjust the size via tailwind heights (e.g. h-10)

I would like to be assigned for this one ;)

I would love to be assigned this one! :D


Support for :contains pseudo class

As this pseudo class was deprecated long ago, it would be nice to have it just match string texts.
e.g. :contains(foobar) will match
Testing foobar

this got implemented just now


[backport Blueprint] Included autogenerated code to the ign_create_docs function (#76)

Targeting ign-msgs4 for Blueprint
Signed-off-by: ahcorde<EMAIL_ADDRESS>

@osrf-jenkins run tests please


[#1309] Respect X-Forwarded-* headers

Specially X-Forwarded-proto, which controls the protocol used on redirects.
We need to redirect to the same protocol used by our proxy (HTTPS), regardless
if we are ourselves running HTTP or HTTPS.
openspending/openspending#1309

Coverage increased (+0.07%) to 75.176% when pulling 2c4548cf7d4decff8c74406e86c1a038de4d4a20 on bug/1309-respect-x-forwarded-headers into c343e783ecfbf5beac75b09ab73a2ef17e8b540d on master.


Add Help text for Verified Twitter Platform

@hroberts / @epenn -- what sort of information does the user need to enter in order pull in tweets from the verified Twitter ingest @epenn is working on? Search terms?
Am I right in thinking that this is an admin only feature--unless we enable users to bring their own API keys? It's available to everyone.

This functionality has been built out on the frontend--just needs to be turned on in the backend. We should add some helpful syntax text similar to that in the reddit pushshift platform page.

Tip/Hack: add the following to platforms_manage::_available_platforms to turn on this functionality (locally):
    info['info']["topic_platforms_sources_map"].append({
        'platform_description': 'submissions and comments from twitter.com',
        'platform_name': 'twitter',
        'source_description': 'import from the pushshift.io api',
        'source_name': 'pushshift',
        'topic_platforms_id': 2,
        'topic_sources_id': 2
    })

I want to work on this issue. Hey , would you please help me to set up python in my windows system?


chore: fix link to database update influxctl

Fixes influxctl link that pointed to list instead of update

[X] Signed the InfluxData CLA
(if necessary)
[X] Rebased/mergeable

No prob!


[Enhancement] Support microseconds for function timediff

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
Now Doris already supports DATETIMEV2 which could have precision up to microseconds. But function timediff still can not process those microseconds.

Solution
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@Gabriel39 Maybe I can have a try, assign to me, PLZ?

I try this function in mysql client
mysql> select timediff(now(3), seconds_add(now(3), 3));
+------------------------------------------+
| timediff(now(3), seconds_add(now(3), 3)) |
+------------------------------------------+
| -00:00:03                                |
+------------------------------------------+

@Gabriel39 what you expect it return?

Hi @WinkerDu , could you add my WeChat account? My ID is HongKanStation

any progress on this issue?

@Gabriel39 Sorry for late report. Currently I am quite busy on other stuff, maybe we can change assignee if appropriate.


bundle size issue

I notice when you import MUI components, you are doing import { XXX } from '@material-ui/core'. This will result in the whole MUI package to be built into the bundle.
This should be updated to import XXX from '@material-ui/core/XXX'.

Thank you for the notes, I will look into this.

I wish I could fix and send a PR but I'm quite busy right now

Updated on version 1.15.1


Images support

I suggest adding ..jpg|..png|..jpeg|..gif| to the default urlRe so that image url's will also work by default. Or maybe add an extra option to enable it.

Cleaning up issues here.
The default urlRe is now for any URL, so I believe this solves your issue.
Thanks,
Sean


Adding Slack badge to the README.md

I deployed a small app on https://abapgit-slackinviter.herokuapp.com/ that allows for people to easily join the abapGit slack. It also has a small badge that we can use in the README.md file.

thanks


chore(release): v0.7.3 🎉

:robot: I have created a release beep boop
0.7.3 (2024-06-15)
Bug Fixes

alias: correctly identify upsert varname based on alias in upsert derefs (e3e6a3c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/sql-dao-generator/releases/tag/v0.7.3 :sunflower:


It's unclear if LoggerHandler implement overload protection

Logger docs make it clear that by default there's no overload protection for handlers and imply that it's somewhat recommended to implement it. Sentry.LoggerBackend benefits  from it through LoggerBackends, but I don't believe there is any mention of overload protection in LoggerHandler docs? It'd be nice to mention it!

The :rate_limiting option is what you want (https://hexdocs.pm/sentry/Sentry.LoggerHandler.html#module-configuration). Feel free to send a PR with clearer docs if you think that would help. Thanks! 💟

@martosaur you are correct in most things you wrote. Logger backends do use Logger's overload protection, which works by dynamically switching between "async" and "sync" Logger calls to prevent overloading.
The Sentry.LoggerHandler, on the other hand, never reports messages to Sentry in a "sync" mode, only "async". But, you have rate limiting to protect you from surges. So, yes if an event passes rate limiting then there is no backpressure, but rate limiting should make that case pretty unlikely.

unless an undocumented :send_result option is set to sync (?).

That's not fair (docs) 😉

Next Steps
We can implement surge protection too if that's something we want. This is not an expected feature (as per Sentry's docs), but it might make sense, I’m not sure. Thoughts?

And are there any recommended sensible defaults for rate limiting?

Depends on your HTTP connection pool mostly.

We can implement surge protection too if that's something we want. This is not an expected feature (as per Sentry's docs), but it might make sense, I’m not sure. Thoughts?

To be honest, it feels a bit awkward to me to use rate limiting as an overload protection, as they come from different domains? Overload protection is, according to the docs, a mechanism that every erlang logger handler should ideally implement to ensure it's responsive when the system is under load. Rate limiting goal on the other hand is to not overload Sentry.
When we look at the LoggerBackends implementation (and judging by the docs, default logger_std_h handler works in a similar way) we see that it's reactive to the current state of the system: if messages piling up, we enable backpressure. Is they pile up even more, we start to drop them.
The way rate limiting is setup now, it needs an absolute value and it's really really hard to come up with one? I don't think I can predict how many log messages per second my node will generate when it's close to being overwhelmed.
Realistically though, LoggersBackend won't probably go anywhere for the time being, so it might not be worth changing stuff. But it may surely hurt handler adoption, as people like me will read through the Logger docs, see magic words "overload protection" and then look for them in every logger handler they consider hooking in 😄


Error when compiling with TS 4.5 and preserveValueImports enabled

When compiling my project with "@tauri-apps/api": "^1.0.0-beta.8",, webpack throws  three errors when compiling modules:
ERROR in<EMAIL_ADDRESS>88:9-18
[tsl] ERROR in /Users/m/Development/m2l_tina/node_modules/@tauri-apps/api/window.ts(88,10)
      TS1444: 'EventName' is a type and must be imported using a type-only import when 'preserveValueImports' and 'isolatedModules' are both enabled.

ERROR in<EMAIL_ADDRESS>88:20-33
[tsl] ERROR in /Users/m/Development/m2l_tina/node_modules/@tauri-apps/api/window.ts(88,21)
      TS1444: 'EventCallback' is a type and must be imported using a type-only import when 'preserveValueImports' and 'isolatedModules' are both enabled.

ERROR in<EMAIL_ADDRESS>88:35-45
[tsl] ERROR in /Users/m/Development/m2l_tina/node_modules/@tauri-apps/api/window.ts(88,36)
      TS1444: 'UnlistenFn' is a type and must be imported using a type-only import when 'preserveValueImports' and 'isolatedModules' are both enabled.

3 errors have detailed information that is not shown.
Use 'stats.errorDetails: true' resp. '--stats-error-details' to show it.

The solution seems an easy fix, just separate
https://github.com/tauri-apps/tauri/blob/c4bb305daeecaf4425eb63559a0ec85e0afd9a64/tooling/api/src/window.ts#L88
to
import { listen, once } from './event'
import type { EventName, EventCallback, UnlistenFn } from './event'

or the alternative version
import { type EventName, type EventCallback, type UnlistenFn, listen, once } from './event'

this has been fixed and the next release won't ship ts files anymore.

Friendly reminder, use a proper issue template next time so we can have the needed information.

Where can I find the next release? In the next branch, the window.ts is still there as part of api and shows the same mixture of imports.

Where can I find the next release?

hasn't been published yet.

In the next branch, the window.ts is still there as part of api and shows the same mixture of imports.

Our tsconfig is different than yours, so it compiles fine for us. In the next release, we will stop shipping ts files along the js files and this error will be gone on your end.


[SPARK-49171][CONNECT][DOCS] Update Spark Shell with Spark Connect

What changes were proposed in this pull request?
This PR updates Spark Connect documentation per the change in https://github.com/apache/spark/pull/47402. Now both bin/pyspark and bin/spark-shell are pretty similar.
Why are the changes needed?
So the end users can use Spark Connect.
Does this PR introduce any user-facing change?
Yes, it updates the user-facing documentation.
How was this patch tested?
Manual documentation build.
Was this patch authored or co-authored using generative AI tooling?
No.

This PR is dependent on https://github.com/apache/spark/pull/47402

Thanks for updating this, an additional question, do we have docs for bin/spark-connect-shell?

I think we should eventually remove bin/spark-connect-shell out

Merged to master.

Thank you!!


DS-3483 add support to embed linked resources collection in the HAL document

This is an initial draft that require further refinements but I suggest to have it committed asap so to work on it collaboratively.

By default now all the collection properties are embedded in response, this is for instance the case of the bitstreams list in the ItemRest;

linked entities listed in the LinksRest annotation of the repository are included only if specified in the resource wrapper instantiation and supported for embedding by the link repository (i.e the relation have suitable default or don't depend on additional parameters) - there is a working commented out code in the BrowseIndexResource to show that

This code is based on the PR #1775 and it has been already applied to the REST 7 Demo http://dspace7.4science.it/dspace-spring-rest/
you can see how the bitstreams are currently embedded in the items response
http://dspace7.4science.it/dspace-spring-rest/api/core/items
http://dspace7.4science.it/dspace-spring-rest/api/core/items/9f3288b2-f2ad-454f-9f4c-70325646dcee
http://dspace7.4science.it/dspace-spring-rest/api/discover/browses/dateissued/items

the failure reported by travis is due to a missing branch on the DSpace/DSpace repository...
At the start, I have pushed my PR branch to the wrong organisation (DSpace instead of 4Science) when I realized that, I have deleted the branch in the DSpace repository "breaking the inprogress travis build"

@abollini : I wonder if simply rebasing this against the latest master would correct the Travis issues?  I notice this includes a bunch of commits from #1775 (which was merged to master), and yet GitHub doesn't seem to realize that (as they appear in this PR as well).  Rebasing would also likely clear up the fact that this PR is really just a single commit (the last one, https://github.com/DSpace/DSpace/pull/1778/commits/244cb97959529bc573cf3101bba460de93dfa381)

This will be merged as part of #1783


first draft of artifacthub integrations

Closes #892
Signed-off-by: jetzlstorfer<EMAIL_ADDRESS>

Preview https://deploy-preview-893--keptn.netlify.app/docs/integrations/

Manually checked all commits for their signing.


[Suggestion] Medkit improvement

Already discussed in discord
Problem
Using the medkit for reviving players is hard as you can not look away for a moment (TraceLine#L136), dead corpses floats in water and it's hard to aim at them, and you can not revive players in small places #L256 I am not entirely sure if this latter one but it's worth to look at it too
Another problem would be the stucking players when they are revived, depends a lot on the map structure and where their corpses where but if a "unstuck" command or something is implemented it would be helpful.
Solution
Suggested by zehmatt a TraceHull
Alternative a bit expensive but in my personal experience and preference more reliable and fun
local SEARCH_RADIUS = 128

function SWEP:GetActorForReviving()
    local ragdoll = nil
    local owner = self:GetOwner()
    local startPos = owner:GetShootPos()
    local endPos = startPos + (owner:GetAimVector() * TRACE_LEN)

    for _, pPlayer in ipairs(player.GetAll()) do
        if pPlayer:IsRagdoll() and ( pPlayer:GetPos() - owner:GetPos() ).Length() < SEARCH_RADIUS then
            ragdoll = pPlayer
            break
        end
    end

    return ragdoll
end

Note: I don't know Lua neither the API, i did what i though was correct and didn't even tested it but you get the "prototype" to call it somehow.

Learned Lua in the night 🤓
A functional range-search would be this commit from Mikk155/Lambda though it may be overkill and sure there are better methods, yet there is the prototype of it

Learned Lua in the night 🤓 A functional range-search would be this commit from Mikk155/Lambda though it may be overkill and sure there are better methods, yet there is the prototype of it

You want to use a hull trace but that commit would also work just as well I guess the downside being that it wouldn't be lag compensated. I will update this eventually.

So far that i have seen, before d2_coast map just after exiting Ravenholm, in the train section that you have to pass ducking to avoid snipers, a friend of mine was revived where the headcrab npc is spawned and he was stuck, there were other cases but i am not entirely sure if you could replicate them as the map-structure wasn't really suspicious. if i get to do another run i'll take notes.


Try quick-xml instead of xml-rs

I've tried changing xml-rs with quick-xml to see how it performed in your simple scenario
before
test bench_sample ... bench:   6,658,267 ns/iter (+/- 509,266)

after
test bench_sample ... bench:   5,008,046 ns/iter (+/- 108,389)

quick-xml is a library I'm building which if definitely not as bullet proof as xml-rs. But for many scenario it is really faster.
Warning: there are many unwraps  in the code (you were already unwrapping event).

Welcome and thanks for the PR! Am I reading your benchmark correctly in that using quick-xml will be 1.7*10^6 nanoseconds aka 1.7 milliseconds faster per iteration?
I'm currently expecting the main runtime in opening a KeePass database to come from deriving the decryption key (i.e. doing many iterations of AES256) and not from parsing the decrypted XML since most users will have much less than 1000 entries in their database - though I don't have any data to back this up :)
I admit that the demo database included with keepass-rs is probably smaller than a typical user's real database, but do you see a realistic use case where the XML parsing will be a significant portion of the run time, thus making a faster parser necessary?

I was actually surprised by the benchs (was expecting around 0.2 - 0.3ms improvement max).
I don't know keepass enough to be certain be yes, I agree it probably doesn't matter that much. I took this as an exercise to see how difficult it was to switch from one library to the other.

Since the performance difference between the two parsers is not that important for this project, I prefer to stick with the more widely used XML parser library.
Thanks for pointing out quick-xml to me though, I'll keep it in mind for future projects - and I agree that porting between the two libraries is very easy indeed :+1:


Add Algolia settings in repo

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1490

Is this actually consumed or is it just a backup?


Implement precise snapping in the animation editor

Holding Shift while snapping is enabled will now make snapping 4 times as precise.
There may be a better precise snap modifier than 4, I just picked that one because it works reasonably well with the default step (0.1). Any suggestions that would work better with step values people often use? :slightly_smiling_face:

Thanks!


docs: Allow propery structs to contain non-struct ptrs

Other parts of blueprint have started allowing pointers to strings and
booleans. This code silently allowed them because of the nil check, but
it doesn't work if the module factory sets a default value.

LGTM, it is no worse than what is there but we should fix the type eventually

Passed travisci on my fork: https://travis-ci.org/danw/blueprint/builds/102211094


Extension does not work with remote files loaded via SSH FS

Describe the bug
When editing files loaded via a remote filesystem via the SSH FS extension most (all?) extension features are disabled. I don't get the outline, I can't jump to any definitions, I can't peek, these options aren't even present in the context menu. I'm using SSH FS to connect to some older machines that don't support Microsoft's remote SSH extension.
To Reproduce
Steps to reproduce the behavior:

Install SSH FS extension
Load a remote file
Try to use any extension features

Expected behavior
I would expect to still have these features available.
Desktop (please complete the following information):

OS: Windows 10 x64
Version 6.3.19

Additional context
I mentioned this in another issue and figured I'd actually file this report since this has been an issue I've run into daily for a long time, it would be great if I could still use this extension with remote files.

Actually after thinking about this I decided to try a work around and was able to get things working with SSH-FS for Windows. The extension seems to work fine when I use this to handle the connection instead of the SSH FS VS Code extension. So I guess I'll close this since it seems to just be an issue with that extension.

The SSH FS extension introduces a new virtual filesystem under the schema called 'ssh'.
Features in this extension are registered under the 'file' extension because it depends on the file system in order to function.
For example it uses various function in 'fs', 'path'.   These are used for caching, file searching and more.
I recommend you move from this extension to Microsoft's own remote SSH, which works much better:
https://code.visualstudio.com/docs/remote/ssh

Thanks, however that extension is not compatible with my target environment (HPUX). Like I said I was able to use SSH-FS for Windows to handle the connection outside the editor and your extension works great with that.


Add dust limit during send transactions

minimum allowed should be 550 sats

I can send 0 sats.
-I sent it from checking account and got a notification with no transaction in test account.

-when you delete something and then put a lower amount it does no validate again.

The validation should be after the button is pressed not on entry.


Mode 1331 ported ClassFileSequencer to 3.x

-added common abstract test case for sequencers
-updated ImageSequencerTest to use base test class
-fixed "package" visibility bug (was not being set properly)

Overall, I really like the changes. I do have a few suggestions/comments, and would like to hear your thoughts.


Enhancement: manage Trusted IPs in config resource

We have a use case for adding a static set of IP addresses to the Trusted IPs of all of our configs. For this, it would be very helpful to be able to manage trusted IPs when deploying configs via Terraform.
Example:
resource "doppler_config" "this" {
  project = "my-project"

  environment = "dev"
  name        = "dev"

  trusted_ips = [ "<IP_ADDRESS>", "<IP_ADDRESS>" ]
}

This would prevent us inventing some complicated workaround or automation logic, which would also likely reduce unnecessary API calls to the Add endpoint.

Here's a draft PR that adds trusted_ip as a resource type. An alternative approach would be to update the doppler_config resource to manage the trusted IP addresses instead.
https://github.com/DopplerHQ/terraform-provider-doppler/pull/61


Add serialization to the grid

An enhancement to make saving easier

Serialize -> Returns Json string (https://docs.godotengine.org/en/stable/classes/class_json.html)
Deserialize(string) -> Loads a grid's state from Json
Load(string) -> loads a grid by ID
Save(string) -> Saves a grid with the provided key

I think this would make sense in the core grid2d.cs file.

I think maybe that should be handled by the user? Don't know if it's in the scope of the add-on.


Tan

There is a feature request for Tan.
It seem most applicable to do it using constants i and e:

these formulae look normal to me. Where exactly in the codes need help?

The best place would be right between exponentiation operator^= and diffirentiation method d here https://github.com/ohhmm/openmind/blob/2ba79fbf4dcd3e91e9abc18cb6d003e67b0cfcc3/omnn/math/Valuable.h#L220

A little note about the method naming convention in this project:
If a class method modifies itself object then it should be named starting from small letter and words separated with underscores.
Otherwise if it const method which returns result by value then CamelCaseNamingNoUnderscoresPlease.

Optimizations


Add option to return bytes stream to workbooks.download() method

Hi,
It would be nice if workbooks.download() method could also return bytes stream for downloaded object. For example, add argument stream=True/False to download function that would define whether object should be downloaded into specified path or returned as a byte stream.
https://github.com/tableau/server-client-python/blob/master/tableauserverclient/server/endpoint/workbooks_endpoint.py#L134
In this case byte stream can be handled outside of library, for example it can be used to write data in AWS S3 bucket, GCS bucket or Azure Blob storage instead of local storage.
Example:
def download(self, workbook_id, filepath=None, include_extract=True, no_extract=None, stream=False):
if not workbook_id:
error = "Workbook ID undefined."
raise ValueError(error)
url = "{0}/{1}/content".format(self.baseurl, workbook_id)
if no_extract is False or no_extract is True:
    import warnings
    warnings.warn('no_extract is deprecated, use include_extract instead.', DeprecationWarning)
    include_extract = not no_extract

if not include_extract:
    url += "?includeExtract=False"

with closing(self.get_request(url, parameters={"stream": True})) as server_response:
    _, params = cgi.parse_header(server_response.headers['Content-Disposition'])
    filename = to_filename(os.path.basename(params['filename']))
    if filepath:
        download_path = make_download_path(filepath, filename)

        with open(download_path, 'wb') as f:
            for chunk in server_response.iter_content(1024):  # 1KB
                f.write(chunk)
        logger.info('Downloaded workbook to {0} (ID: {1})'.format(download_path, workbook_id))
        return os.path.abspath(download_path)
    if stream:
        return server_response.content

Agree, now I first have to write to disk and then write to a cloud storage bucket


feat: search token by name

Resolves: https://github.com/Magickbase/ckb-explorer-public-issues/issues/383

How is it going

Any update on this feature?

How is this one going?

1、For loading animation, please refer to the centered style.

2、The search toggle button style needs to be adjusted.  At the same time, when the X button appears when inputting information, the switch button will be moved. It is recommended to reserve the required position for X.
UI:

3、If the data is abnormal after searching, it should be expected that no results will be displayed.
After inputting j, the displayed result (in the name and hash) does not contain j.

4、
When there are results in the search, it is best to prompt `no more content' after the last piece of data.
Refer to the information on prd.


Define interface Comparable method compareTo as "canonical order" in Apcomplex

We often need to sort values in a list in a "canonical order" therefore we use the compareTo method of the Comparableinterface.
Can we also define a Comparable method compareTo as "canonical order" in Apcomplex?

https://github.com/mtommila/apfloat/blob/main/apfloat/src/main/java/org/apfloat/Apcomplex.java

  public int compareTo(final Apcomplex that) {
    int c = real().compareTo(that.real());
    if (c != 0) {
      return c;
    }
    if (that.imag().signum() == 0) {
      if (imag().signum() != 0) {
        return 1;
      }
    } else if (imag().signum() == 0) {
      return -1;
    }
    return imag().compareTo(that.imag());
  }

See also similar issue in the hipparchus project:

https://github.com/Hipparchus-Math/hipparchus/issues/274

I'm a bit hesitant about this since there does not seem to be a "canonical order" of complex numbers.
Even Mathematica won't compare complex numbers.
Apparently it's possible to define an infinite number of un-interesting orderings of the complex numbers, so perhaps that would be better done with a Comparator, rather than by implemeting compareTo?
This seems especially misleading in the Hipparchus reference, as complex numbers can't be turned into an ordered field. How would a complex number be a field element and comparable at the same time? So that the ordering is meaningful wrt. the field?

AT the moment I think we only need the MATHEMATICA_ORDER


add the Op div

Add the Op Div

The Op Sub and Op Tanh are added in this PR.

Because the implementation for binding Op Sigmoid is similar to other three Op in this PR. So I add this Op in the next commit.


Timestamp

To get the results from page > 1, a parameter timestamp is used by the FR24 server.
For the page 1:
https://api.flightradar24.com/common/v1/flight/list.json?query={0}&fetchBy=reg&page={2}&limit={3}&token={1}&timestamp=
For page > 1:
https://api.flightradar24.com/common/v1/flight/list.json?query={0}&fetchBy=reg&page={2}&limit={3}&token={1}&timestamp={4}
The value {4} has to be taken from the last entry of the previous page under ['time']['other']['updated'].
The 2nd hack adds a timeout when consecutive requests are sent to quickly. FR24 server responds time to time with an HTML code 402 or 400.

Thanks for this - timestamp should probably be OK to be just a field for now, will see how it evolves.
Expect to see this in a new version on pypi next week - waiting for confirmation on another fix


Modpack format path warning does not cover all possible bad paths on windows

Going off of https://learn.microsoft.com/en-us/dotnet/standard/io/file-path-formats the given warning on https://docs.modrinth.com/docs/modpacks/format_definition/#path that says to "make sure it doesn't contain .. or start with a drive name (i.e., [A-Z]:/, [A-Z]:\, and /).", If implemented at face value, would still result in multiple bad file paths being permitted such as:

\Program Files\Custom Utilities\StringFinder.exe: A relative path from the root of the current drive.
\\system07\C$\: The root directory of the C: drive on system07. (A network drive/resource)
DOS device paths such as \\.\C:\Test\Foo.txt and \\?\C:\Test\Foo.txt

This could be resolved by either adding \ as a disallowed starting character, or promoting normalization checks instead

If those paths are accepted by some launcher implementations, this should be escalated to a security advisory (CWE-35: Path transversal)

CWE-22 actually
CWE-35 is covered by the .. restriction as it's specifically about improper normalization and not generic path traversal


Development

LGTM!
Unless typer or python console scripts somehow take care of it, the only thing I would change is to make sure we always return to the original working directory, no matter what happens. Something like:
original_cwd = os.getcwd()
os.chdir(run_sub)
try:
    ....
finally:
    os.chdir(original_cwd)

you're right! and now that you pointed it out, I think I'm gonna split the actual processing from the folders part into a different function


Translate concepts/storage/storage-capacity into Japanese

Closes #23893.

/assign @inductor
Page preview: https://deploy-preview-23931--kubernetes-io-master-staging.netlify.app/concepts/storage/storage-capacity/

bells17さんの指摘以外LGTMです

/lgtm

/approve


SS14-26480 Roll Traversal Distorter into regular Artifact Analyzers

About the PR
Bit of a grab-bag of cleanup of Xenoarchaeology prompted by #26480.
Why / Balance
Xenoarch needed a bit of cleanup but a lot of the work I thought it needed was already either done or already was present before writing 26480. This covers the remaining mald points.
Technical details

Traversal distortion biases are now "up" and "down" instead of "in" and "out".
Node generation has been tidied up to make it a little clearer how it works.
Traversal Distorters have been removed from the game along with their board.
Traversal distortion is now done by the artifact analyzers by default.
Some loc strings have been made clearer.
The Abnormal Artifact Manipulation tech has been made slightly cheaper.
The aformentioned tech is now localized as Artifact Recycling, given all it does is unlock the crusher.
The Xenoarchaeology guidebook entry has been given a bit of a rewrite, putting all information into one page and making sure to cover the basics that otherwise would require someone typing "Liltenhead artifact tutorial" into Youtube.

Media

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

Changelog
:cl:

tweak: Xenoarchaeology Traversal Distorters have been rolled into the Artifact Analyzers to make artifact gameplay a bit more controlled.
tweak: The T2 Abnormal Artifact Manipulation tech has been made cheaper to compensate for losing an unlock.
tweak: The Xenoarchaeology guidebook entry has been rewritten to be more useful and comprehensive for newbies.

Note that the guidebook not mentioning the crusher is a conscious choice. The guidebook should mostly be for common roundstart questions - if the RD or someone unlocks the tech and builds the crusher, they should need to explain to their newbies what it does and how to make use of it.

modifying the traversal distorter state should be done from the analysis console UI now that it's built into the analyzer.

modifying the traversal distorter state should be done from the analysis console UI now that it's built into the analyzer.

I can agree but that is UI work I don't want to tackle in this PR (as I understand zilch of the UI code)

https://github.com/space-wizards/space-station-14/assets/732532/e1513abc-427f-440b-843d-a2d99539eb1f
I did the thing

The tooltip on that button is greatly appreciated!

The tooltip on that button is greatly appreciated!

Yeah, this is already a huge buff to Xenoarch speed but players need to know how to use it

I'm going to ignore conflcts to migration.yml as I'm vaguely sure any maintainer hitting the big green merge button can handle resolving the conflict
or pinging me to resolve the conflict

idk but it feels kinda weird with those colors, and maybe you can put the extract button to the bottom, because its like very important button


item-of-doom causes grid to crash

Issue description:
Typing certain strings in the grid interface causes the client to crash. I keep running into new ones, but so far they include "co", "br", and "one"
After a bit more testing, it seems it's not typing the string - it's displaying a certain item on screen. I can repeat the crash just by scrolling the grid display down far enough. I can't work out of a way to remove the offending item.
What happens:
Game crashes.
What you expected to happen:
Not to crash.
Steps to reproduce:

open the grid
cause item-o-doom to display in the grid, either by searching or scrolling

Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: 1.10.2 - <IP_ADDRESS>5
Refined Storage: 1.1.3

Does this issue occur on a server? [yes/no]
Have not tested, only in singleplayer.
If a (crash)log is relevant for this issue, link it here:
http://pastebin.com/QFGsjmhC

This is a forestry issue, please report to them


[derive] support generic Tree types of generic depth

Currently, it doesn't seem possible to do the following properly:
struct A<T: TreeKey<Y>, const Y: usize> {
    inner: T,
    outer: u32,
}

We would need to annotate the inner: T with #[tree(depth(Y))], but the current depth parsing requires an integer. The depth isn't known until compile time, but is known to be Y >= 1. It seems like the depth should optionally take this compile-time generic as well.

I think a blanket implementation is actually the only way. Arithmetic on const generics (i.e. impl TreeKey<{Y - 1}>) is not stable currently.

One problem would be that a generic field type may not be dominating the overall depth. Then the outer detph would not determine the inner depth in all cases. The offset would have to be constant. And Y1,Y2 combinations of a struct with multiple inner generic types with generic depth would be impossible unless Y1 = Y2 + constant.
A way to do this may be to support depth: Vec<usize> on fields and then determine the fixed offset.


Split up data parsing, modelling and rendering better

Moving this from the initial code review in #17. Currently we mix these concerns a little, but as we grow the app we'll need to refactor things so that we have a clear data parsing step, models for nodes/edges derived from BODS data and rendering code, all in their own places and wired together into the UI. This should probably be accompanied by some unit tests and an overall test of the library using something like jsdom.

As discussed with @kd-ods today, architecturally, it may be necessary to make breaking changes to the usage of this tool. The next version may involve calling a different method, or even multiple methods, rather than the current draw() method due to splitting things out.
I'm largely brainstorming at this stage, so changes to the usage may not be as significant as I'm picturing, and efforts will certainly be made to Keep It Short and Simple (KISS) and as similar to current usage as possible.


[refactor] remove atomic.h/cpp use std::atomic instead

Proposed changes
remove atomic.h/cpp use c++ standard std::atomic instead
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (No Need)
Has document been added or modified: (No Need)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM

I have a small question: why some variables are initialized to 0 and some not

I have a small question: why some variables are initialized to 0 and some not

Previous AtomicInt will  initialize value as 0， but std::atomic not, the variables not initialize as 0 will be initialized at ctor

the variables not initialize as 0 will be initialized at ctor

i get it, thanks

Merge for next remove mutex pr


[Button] Text wrap issue

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

https://codesandbox.io/s/inspiring-browser-c7qmv7?file=/Demo.tsx

Current behavior 😯

Expected behavior 🤔

Context 🔦
It looks more correct on Material UI:

https://codesandbox.io/s/peaceful-khorana-27gct9
but a lower line-height would I think feel better.

  Output from `npx @mui/envinfo` goes here.

Hi ,can I work on this if it is open?

Hi ,can I work on this if it is open?

Thanks for asking! The PR is already there, would you mind taking a look at other issues?

I checked on Material UI v6, it looks ok: https://mui.com/material-ui/react-button/#material-you-version

cc @DiegoAndai. Actually, should we rename "Material You version"  to be something like "Material UI v6 (alpha)"?

Is this issue still open, so that I can work on this.

Hey,Is this issue still open, so that I can work on this.

Hello is the issue still open can i pull and solve it ?

Hello, I'd like to inquire if the issue is still available for me to pull and work on?

Hey, @ShaikhHeeba07 @hetpatel4381 thanks for asking! We already have two undergoing PRs for the Joy UI line height on buttons, the #38696 and #39316 :)

Duplicates

[x]  I have searched the existing issues

Latest version

[x]  I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

https://codesandbox.io/s/inspiring-browser-c7qmv7?file=/Demo.tsx

Current behavior 😯

### Expected behavior 🤔

### Context 🔦
It looks more correct on Material UI:

https://codesandbox.io/s/peaceful-khorana-27gct9
but a lower line-height would I think feel better.
Your environment 🌎
npx @mui/envinfo

Hello!
Is the issue still open? If yes, can I work on it?

Hello!
Is the issue still open? If yes, can I work on it?

Hi @oliviertassinari - can you please assign this issue to me ?

Hello, I'd like to know if this issue is still persist for me to pull and work on?

hi , can i work on this

Hi, let me know if I can start working on it. Looking to start my open source journey. Thanks

@zanivan What should be the line-height for Material UI Button?

For anyone who is interested to submit a PR, here is the fix:
diff --git a/packages/mui-material/src/styles/createTypography.js b/packages/mui-material/src/styles/createTypography.js
index 277808d2a7..32b5e9f790 100644
--- a/packages/mui-material/src/styles/createTypography.js
+++ b/packages/mui-material/src/styles/createTypography.js
@@ -69,7 +69,7 @@ export default function createTypography(palette, typography) {
     subtitle2: buildVariant(fontWeightMedium, 14, 1.57, 0.1),
     body1: buildVariant(fontWeightRegular, 16, 1.5, 0.15),
     body2: buildVariant(fontWeightRegular, 14, 1.43, 0.15),
-    button: buildVariant(fontWeightMedium, 14, 1.75, 0.4, caseAllCaps),
+    button: buildVariant(fontWeightMedium, 14, 1.5, 0.4, caseAllCaps),
     caption: buildVariant(fontWeightRegular, 12, 1.66, 0.4),
     overline: buildVariant(fontWeightRegular, 12, 2.66, 1, caseAllCaps),
     // TODO v6: Remove handling of 'inherit' variant from the theme as it is already handled in Material UI's Typography component. Also, remember to remove the associated types.

For anyone who is interested to submit a PR, here is the fix:
diff --git a/packages/mui-material/src/styles/createTypography.js b/packages/mui-material/src/styles/createTypography.js
index 277808d2a7..32b5e9f790 100644
--- a/packages/mui-material/src/styles/createTypography.js
+++ b/packages/mui-material/src/styles/createTypography.js
@@ -69,7 +69,7 @@ export default function createTypography(palette, typography) {
     subtitle2: buildVariant(fontWeightMedium, 14, 1.57, 0.1),
     body1: buildVariant(fontWeightRegular, 16, 1.5, 0.15),
     body2: buildVariant(fontWeightRegular, 14, 1.43, 0.15),
-    button: buildVariant(fontWeightMedium, 14, 1.75, 0.4, caseAllCaps),
+    button: buildVariant(fontWeightMedium, 14, 1.5, 0.4, caseAllCaps),
     caption: buildVariant(fontWeightRegular, 12, 1.66, 0.4),
     overline: buildVariant(fontWeightRegular, 12, 2.66, 1, caseAllCaps),
     // TODO v6: Remove handling of 'inherit' variant from the theme as it is already handled in Material UI's Typography component. Also, remember to remove the associated types.

Hi, I am interested to Contribute with MUI and it would be an start point if this issue is not assigned to anyone
Can I work on it?

Hello everyone
can you guide me how to pass the checks please!


One source of channelgroup + more robust searching

As proposed in #53 Use a single object shared across components for being flexible about names the user may have given to something. Also expand the fallbacks from channelgroup to other common names.
This would supersede #53

Maybe this beongs in pymmcore-plus instead? but it's not 100%  to me that it would.

Codecov Report

Merging #54 (a0f625c) into main (74eb061) will increase coverage by 0.37%.
The diff coverage is 81.48%.

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   41.86%   42.23%   +0.37%
==========================================
  Files          10       10
  Lines        1247     1255       +8
==========================================
+ Hits          522      530       +8
  Misses        725      725

Impacted Files
Coverage Δ

micromanager_gui/_util.py
61.25% <78.57%> (+3.67%)
:arrow_up:

micromanager_gui/main_window.py
70.64% <83.33%> (+0.21%)
:arrow_up:

micromanager_gui/multid_widget.py
71.85% <85.71%> (+0.28%)
:arrow_up:

Continue to review full report at Codecov.

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

can you help me understand how this relates to tlambert03/pymmcore-plus#53

While the pymmcore-plus PR deals with objectives and this PR with channelgroups they are both getting at the same idea of having get___ methods that are robust to user naming. But the big open question if these robust getter methods should live in napari-micromanger or in pymmcore-plus. There are benefits to both ways but we should probably make a choice and then put all the robust getting that we need there.
This would not close #43, but if we implement this here instead of pymmcore-plus then I think we should merge this, and then rebase #43 to use this robust getter so that we have a clean centralized way of doing the fallbacks.
I think I lean very slightly towards putting this kind of stuff into pymm+ because I could imagine someone writing their own scripts that are meant to be fairly general and being annoyed that these utilities only exist in the napari-plugin. But the downside there is that it seems like these shouldn't necessarily be just mixed in with the standard pymmcore methods and maybe should get their own mmc.robust.getChannelGroup namespace or something like that.
tldr the important choice is: does this go here, or in pymmcore-plus (I'm 55% for pymm+)

I think it can be good to have this in pmmc+. As soon as it is ready and merged, I can add more "get__" methods to this, for example to get the objectives (#43) , maybe the illumination devices (#56) , etc...


Consume the new, safer Rust PKCS#11 interface into Parsec when it is available

Summary
Refactor the Parsec PKCS#11 provider so that it consumes the newer, safer types and interfaces in the Rust PKCS#11 library.
Background
The Rust PKCS#11 provider allows Parsec to work with hardware security modules and similar systems that adopt the Oasis PKCS#11 Standard.
The PKCS11 provider makes use of a suite of Rust PKCS11 bindings from the rust-pkcs11 project.
Parsec project contributors have noted some safety improvements that could be made to these interfaces. For details see: https://github.com/mheese/rust-pkcs11/issues/38
There is a fork of the PKCS#11 library created by @joechrisellis that implements the recommended changes to the Rust library: https://github.com/joechrisellis/rust-pkcs11/tree/new-abstraction
This branch reworks the library into a lower-level pkcs11-sys module, and higher-level pkcs11 module, the latter containing a new set of abstractions aimed at consumption into application code.
Once these new interfaces are made available, either by PR into the original project, or by creating a new separate project, it will be possible to refactor the PKCS#11 provider in Parsec to consume these interfaces.
This issue has been raised to track for the future.

The work on adding the safe abstraction layer is now located on the new-abstraction branch here.


Can not detect vietnamese 2 line plates

I have issue with vn2
US & EU plates working fine but not working Vietnamese 2 line plates
Any idea? Thank you

You may need to edit config file at runtime_data/openalpr.conf
Quick search revealed this file.
https://github.com/openalpr/openalpr/blob/master/runtime_data/config/vn2.conf

How to edit config at runtime_data/openalpr.conf?
Add vn2.conf below final line openalpr.conf?

I think you just replace the content of the whole file.

I replace the content of file but still not work.

I think you just replace the content of the whole file.

any idea? please

You need to read more about OpenAlpr at their homepage. Try to come up with more specific problem description. What does not exactly work? What plates? Screenshots? Two rows? Color?

@vinpro24 Did you solve it ?

@vinpro24 Did you solve it ?

@vinpro24 Did you solve it ?

Not yet


[Bug]: ImportError: cannot import name 'dependency_versions_check' from partially initialized module 'transformers'

Project Version
3.2.7
Platform and OS Version
Windows 11, 64 Bit
Affected Devices
Windows
Existing Issues
No response
What happened?
I've got problem after update:
ImportError: cannot import name 'dependency_versions_check' from partially initialized module 'transformers'
Any ideas how to fix that?
Steps to reproduce

Update applio from 3.2.6 to 3.2.7
Done, you've got problem

Expected behavior
Successeful update instead of any errors
Attachments
Traceback (most recent call last):
  File "D:\Software\ApplioRVC\Applio-3.0.6\app.py", line 22, in <module>
    from tabs.inference.inference import inference_tab
  File "D:\Software\ApplioRVC\Applio-3.0.6\tabs\inference\inference.py", line 9, in <module>
    from core import (
  File "D:\Software\ApplioRVC\Applio-3.0.6\core.py", line 21, in <module>
    from rvc.lib.tools.model_download import model_download_pipeline
  File "D:\Software\ApplioRVC\Applio-3.0.6\rvc\lib\tools\model_download.py", line 15, in <module>
    from rvc.lib.utils import format_title
  File "D:\Software\ApplioRVC\Applio-3.0.6\rvc\lib\utils.py", line 14, in <module>
    from transformers import HubertModel
  File "D:\Software\ApplioRVC\Applio-3.0.6\env\lib\site-packages\transformers\__init__.py", line 26, in <module>
    from . import dependency_versions_check
ImportError: cannot import name 'dependency_versions_check' from partially initialized module 'transformers' (most likely due to a circular import) (D:\Software\ApplioRVC\Applio-3.0.6\env\lib\site-packages\transformers\__init__.py)

Screenshots or Videos
No response
Additional Information
Any help?

help was provided at discord. https://discord.com/channels/1096877223765606521/1287022888901152830/1312712177156751420


Add key handling

Due to edge cases like keyboard-layout-changing keyboard shortcuts and dead keys, key presses and releases should (probably) be associated based on the code rather than the key.

Due to edge cases like keyboard-layout-changing keyboard shortcuts and dead keys, key presses and releases should (probably) be associated based on the code rather than the key.

For now we wll just use the logical keys to avoid having to handle different keyboard layouts


Authenticate through Nextcloud files app

Use the android library to authenticate through the Nextcloud files app.
The prefered way of authenticating should be through the Nextcloud files app. As an alternative the user should still be able to authenticate with username and app password.

Integrated nextcloud sso libraray in commit 09af350c9b5d1c7f43f3c81b5e9d91026ecdca5f. Currently sso is still non-functional. There seems to be a bug in the library (https://github.com/nextcloud/Android-SingleSignOn/issues/177) which prevents me from integrating it fully.

Hey there, I am currently getting started with nextcloud cookbook and searching for an companion app. I'm a little confused - The documentation and images say that it is possible to log in using the nextcloud files app. However, the button does not appear on my device (Nextcloud Files installed via F-Droid, this was causing errors previously). Also, I couldn't find any reference in the code. Is this already implemented and supposed to work on non-google devices?
Also, regarding the SSO API, there are other apps utilizing this API for SSO in Kotlin apps in case you are interested: https://codeberg.org/MicMun/nextcloud-cookbook/src/branch/master/app/src/main/java/de/micmun/android/nextcloudcookbook/nextcloudapi/Accounts.kt

Hi @p-fruck, this sure is confusing! I originally started working on SSO integration but never got it to work. The screenshots are quite old and from that time.
Unfortunately, I haven't gotten around to working on it since then.
TL;DR is that you cannot currently use SSO for authentication. Once integrated, it will also work on non-Google devices.

@lneugebauer wow, that reply was quick :smile: Thanks for the explanation, I'll try to use the token login then ;)


python311Packages.datasette: fix build

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262572 run on x86_64-linux 1

12 packages built:

datasette (python310Packages.datasette)
datasette.dist (python310Packages.datasette.dist)
python310Packages.datasette-publish-fly
python310Packages.datasette-publish-fly.dist
python310Packages.datasette-template-sql
python310Packages.datasette-template-sql.dist
python311Packages.datasette
python311Packages.datasette-publish-fly
python311Packages.datasette-publish-fly.dist
python311Packages.datasette-template-sql
python311Packages.datasette-template-sql.dist
python311Packages.datasette.dist


fix: quantile scale  domain calculate

Checklist

[ ] npm test passes
[ ] benchmarks are included
[ ] commit message follows commit guidelines
[ ] documents are updated

Description of change

fixd quantile scale  domain should‘t do repeated operations

需要更新一下 snapshot：删除 __tests__/integration/snapshots 下面对应的截图：

然后重新运行：npm run test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.794%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10153

Relevant Lines:
11199

💛  - Coveralls


docs: update readme and file structure

removed the outer src file as it was confusing
master => main in readme
added the helpful diagrams
Added some basic windows instructions for powershell
minor other changes to make the readme make more sense

@mefellows yeah I'm going through everything on my windows machine :)


Build issues on Docker Hub

I am experiencing a strange build error on Docker Hub that I cannot reproduce on my own machine. The minimum version of the problem takes a clean Ubuntu 14.04 and installs OpenBLAS from the master branch as follows (please find all details in the short Dockerfile):
git clone -q --branch=master https://github.com/xianyi/OpenBLAS.git
(cd OpenBLAS \
    && make DYNAMIC_ARCH=1 NO_AFFINITY=1 NUM_THREADS=32 \
        && make install)

The build on Docker Hub then fails with the following error message,
OS               ... Linux
Architecture     ... x86_64
BINARY           ... 64bit
C compiler       ... GCC  (command line : gcc)
Fortran compiler ... GFORTRAN  (command line : gfortran)
Library Name     ... libopenblasp-r0.2.15.a (Multi threaded; Max num-threads is 32)

To install the library, you can run "make PREFIX=/path/to/your/installation install".

[91m+ make install
[0mmake -j 1 -f Makefile.install install
make[1]: Entering directory `/tmp/build/OpenBLAS'
Generating openblas_config.h in /opt/OpenBLAS/include
Generating f77blas.h in /opt/OpenBLAS/include
Generating cblas.h in /opt/OpenBLAS/include
Copying LAPACKE header files to /opt/OpenBLAS/include
Copying the static library to /opt/OpenBLAS/lib
[91minstall: cannot stat ‘libopenblas-r0.2.15.a’: No such file or directory
[0mmake[1]: *** [install] Error 1
[0mmake[1]: Leaving directory `/tmp/build/OpenBLAS'
[91mmake: *** [install] Error 2
[0mRemoving intermediate container 96cd4f9c5fef
The command '/bin/sh -c bash /tmp/build_scipy_stack.sh' returned a non-zero code: 1

The complete log can be found here. It seems the build searches for 'libopenblas-r0.2.15.a' where the library is rather 'libopenblasp-r0.2.15.a'. Unfortunately, the same build passes happily fine on my own machine. For sanity check, I also forked the popular Ubuntu + OpenBLAS installation by ogrisel but I get the same error. Any idea what the issue could be?

make DYNAMIC_ARCH=1 && make install never worked for me. From Makefile.install and Makefile.system, DYNAMIC_ARCH=1 must also be set when running make install, else it build a wrong LIBNAME variable.

Yes, options given to the initial "make" do not carry over to subsequent invocations like "make test" or "make install" (cf. #651). An alternative would be to edit Makefile.rule accordingly before running make.

Ah, interesting. I adjusted the Dockerfile accordingly and now put 'DYNAMIC_ARCH=1 NO_AFFINITY=1 NUM_THREADS=32' after make install. The builds on Docker Hub are successful and I could successfully compile Numpy against OpenBLAS. Thanks a lot for your help!

Could you please close this ticket then ?


Simplify the building procedure on Win32.

The PR is too large and changes too many things. Please submit one change at a time with explanation of what it achieves.
Also note that the code changes do not respect the existing code style.

I've already split part into other pull request. review that one first.


AE2 stuff - Wireless Receiver doesn't have a config

i've been trying to change the values for the wireless receivers but i can't find anything on them. i tried looking around to try to find how to do it but can't find anything. the amount of power the wireless receivers use is insane so i wanted to change it

If you mean the wireless connectors (wireless receivers are not in AE2Stuff)
Create a file in config/AE2Stuff called config.cfg and add:
cfg Machines {
cfg Wireless {
PowerBase = 10
PowerDistanceMultiplier = 1
}
}
The formula is Power use = PowerBase + PowerDistanceMultiplier * Distance^2 (AE/t)

I'm closing this issue as part of a spring cleaning of my issue trackers from old stuff.
If you believe it's a bug that is still relevant in Minecraft 1.12 - Feel free to reopen it.

This is still present in 1.12.2. No 'config file' is generated at all.
├───AE2Stuff
│   │   readme.txt
│   │
│   └───overrides

General Versions
minecraft 1.12.2
StoneBlock-1.0.31
Mod Versions
forge-1.12.2-<IP_ADDRESS>68-universal
ae2stuff-<IP_ADDRESS>-mc1.12.2
appliedenergistics2-rv6-stable-4


Italian translation for werewolf and italian update for hunter

Italian translation for werewolf and italian update for hunter

As requested, I've added a new localization--"WOD5E.HunterDrive"--for the term specific to Hunter in the latest push to the dev branch. Let me know when you've updated the Italian localization, and I'll go ahead and merge this.


Support OKTA authentication

Added another SAML2.0 authentication method -- OKTA
Issue:
rancher/rancher#15574
Related PR:
rancher/rancher#17236

LGTM


Events system is quadratic

While discussing with @sisuresh how best to record "non-metered" optional diagnostic events, we noticed that the current event system uses MeteredVec and repeatedly does self.vec = self.vec.push_back(...). This pattern produces quadratic behaviour, as each push_back call clones the vector to add a single element on the end. We should use a normal vector here and implement metering manually (which is likely necessary anyway, to separate the costs of the optional diagnostic events from metering).

Yeah, I think we discussed before about potentially having a new "mutable metered vector" type. The current MeteredVector doesn't make clear distinction the (very important) immutability property. But we previously decided that if we made those events cheap to clone, the quadratic behavior may not be an issue but it makes metering a ton easier. This is no longer the case now with the diagnostic events.
I will create an issue for the mutable_metered_vector.

Created https://github.com/stellar/rs-soroban-env/issues/707

I've updated this to cover the current state of things, which is "non-quadratic vector that's also non-metered". needs metering.


Cache info in the editor

"when I re-open a record in the editor (I made changes and saved the record in the editor before)
it shows cached information, when I then try to save the record I get an error message.
I have to reload the web-page to get the current data."

TECH NOTES:

On 412 status code error we should automatically refresh the editor here https://github.com/inspirehep/inspirehep/blob/master/record-editor/src/app/holdingpen-editor/holdingpen-save-button/holdingpen-save-button.component.ts#L183-L196

With this pr https://github.com/inspirehep/inspirehep/pull/2036, page is redirected to the detail view when the record successfully saved. If save is failed, user should refresh the page manually when they see the error message. I am closing this issue.


Support back nav on Apple TV with menu button

First Apple TV commit... support back nav for navigators that use src/views/Header.js

@satya164 @ericvicenti this change allows StackNavigator to work as expected on Apple TV

The PR description doesn't make it clear what behavior this changes. (I'm not quite sure what TVEventHandler does). I think direct integration into the header view may not be the best idea.
Can you verify this doesn't cause any bugs on iOS or Android? Please include a test plan in future PRs.
I have an AppleTV myself so I would be curious on how to test this. Is there a getting started guide published anywhere?

@ericvicenti you can run this on Apple TV today... just go through the first StackNavigator example at https://reactnavigation.org/docs/intro/ , but in Xcode select the SimpleApp-tvOS target instead of the SimpleApp target.  When you start the app on Apple TV simulator, you will see that the "Chat With Lucy" button is highlighted, and you can press the TV remote button to push the next screen on the navigation stack.  With my change from this PR, you will also be able to use the remote's Menu button to navigate back to the home screen.  The TVEventHandler doesn't do anything on iOS or Android, so this change will have no effect on those platforms.  I'll have a look to see if there is any way to add automation for Apple TV.

@dlowder-salesforce back button handling logic lives here https://github.com/react-community/react-navigation/blob/a7b124305393a8e392621a32e49fa96a3a14c6c9/src/createNavigationContainer.js#L89
The BackAndroid module is exported from here - https://github.com/react-community/react-navigation/blob/master/src/PlatformHelpers.native.js
I think we should rename it to BackHandler or something, and implement same interface for tvOS

Thanks @satya164 for the tip, I'll take a look and rework this PR.

@dlowder-salesforce I'm pretty sure this will break things when using react-native-web and using this lib in a web-based context. We should resolve that in some fashion before we merge this.

Closing this pull request and adding the TV event handler in react-native's BackAndroid, as suggested.


How you decide to use class member, not pull them from DI?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[x] Bug
[ ] Requirement
[x] Feature or performance improvement

Question

What do you want to know?
Why you design and use non-thread-safe class members such as ContextManager.StopSpan,
which will crash production business code by throwing a InvalidException  with message "no active span"

I suggest to register your internal components with scoped lifestyle to the DI,
then create scope for every context and use them.

Bug

Which version of SkyWalking, OS and .NET Runtime?
oap version: 6.0.0-beta , not important.
agent version: 0.6.0, offical nuget package.
.NET Runtime: .NET Core <IP_ADDRESS>, with  meta package Microsoft.AspNetCore.App
Which company or project?
Secret yet.
What happen?
Agent crashes production business code.
Agent throw a InvalidException when Calling DbContext.SaveChangeAsync

Requirement or improvement

Please describe about your requirements or improvement suggestions.

Even if ITraceContext controls the scope of the span through DI, the user may destroy the lifecycle of the TraceContext in an asynchronous manner (such as Task.Run..).


Saving a script resets connection and collapes results window

Azure Data Studio Version: 1.6.0

Steps to Reproduce:

Make a connection to a server.
Write a query like SELECT 1
Run the query
Save the query as query.sql
Connection is lost and results pane collapses

Expected Result:
Results pane stays in place and connection is maintained instead of lost. This is request for PowerUser scenario since saving scripts is a common action.

I am experiencing this same behavior with the current build:  ver 1.13.0

Closing as duplicate of https://github.com/microsoft/azuredatastudio/issues/1876.  Thanks for reporting this issue!


[SPARK-14711] [BUILD] Examples jar not a part of distribution.

What changes were proposed in this pull request?
Move the spark-examples.jar from being in examples/target to examples/target/scala-2.11/jars
How was this patch tested?
Built distribution to make sure examples jar was being included in the tarball.
Ran run-example to make sure examples were run.

Test build #56114 has started for PR 12476 at commit 231ed43.

Test build #56114 has finished for PR 12476 at commit 231ed43.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56114/
Test PASSed.

@vanzin I'd appreciate if you could review this. Thanks!

Test build #56149 has started for PR 12476 at commit a9d076f.

Test build #56149 has finished for PR 12476 at commit a9d076f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56149/
Test PASSed.

Merged build finished. Test PASSed.

LGTM, merging to master.

Thanks, Marcelo. Much appreciated!


macOS: Chrome session created by init() cannot access disk(?)

Hi,
I am attempting writing a very simple code to test the web automation, which open a URL and click the a button to download a Excel file:
r.init()
r.url('https://xxxxxx')
r.wait(20)
r.click('//*[@id="ete"]')
ans = input("Wait....")
r.close()
Problem:
A Chrome session is successfully created which pointing to the target site.
A mouse click is successful emulated. Chrome shows download completed.
However, when check at the Finder, no file is saved.
I tried to add a Python statement (input) to pause the Chrome session. So I can manually operate the Chrome.
Finding:

Manually click the download button on the target website. Same symptoms. No file is wrote.
Click the "recent download history" of Chrome. All the corresponding download attempting are shown usually, as expected.
Move the mouse over the filename and click "show in the finder". No response. The target folder cannot be opened.

Seems the Chrome session which created by RPA init() cannot access to disk. If I manually open Chrome, everything works fine. The target file can be downloaded and saved.
Would you give me some hints. Thanks a lot.
The following is my environment info:
RPA
1.50.0
OS Version:
macOS 14.1.1 (23B81)
% php -v
PHP 8.2.12 (cli) (built: Oct 24 2023 19:22:16) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.12, Copyright (c) Zend Technologies
with Zend OPcache v8.2.12, Copyright (c), by Zend Technologies
Chrome version:
Version 119.0.6045.159 (Official Build) (x86_64)

Hi,
Finally I found only the 'current' directory is accessible by the Chrome. The files are saved to the 'project root', whatever we config under Chrome.

Hi @328joe thanks for raising this and updating back! Yes I think the default downloaded folder is the project root folder. But I haven't come across a user report saying that Chrome can't access other folders. It might be a new security measure in newer Chrome. Do let me know again if you run into a blocker!


Update smartplugv2_energy.yaml

For the energy sensor, change the class to energy, so that it's compatible with home assistant. I'm a beginner tell me if there are any problems I just want my plug connected to be compatible for energy in home assistant, sorry for my spelling.d

Most Tuya devices do not return useful information in this dp. They reset it to zero every time they report the value to the cloud. It is better for such devices to use an integration helper to calculate the energy based on the power sensor.

Hello, and thank you for your response and help! I understand now that the direct energy data from the plug may not be reliable. I'll try using an integration helper with the power sensor to calculate the energy. If you have any guidance on setting this up or can point me to any helpful documentation, I'd really appreciate it. Thanks again for your patience as I’m just starting out!


Reland fix 25807 implement move for sliver multibox widget

Description
The https://github.com/flutter/flutter/pull/29188 was reverted due to build failure, It should be resolve now that we can reland this pr
Related Issues
https://github.com/flutter/flutter/issues/25807

Did anything change in this PR from when it original landed?

Did anything change in this PR from when it original landed?

no


New platform tii3q98 zcu111 for testing software

Platform and runcard for tii3q98 chip qubits 1, 2, 3 controlled with ZCU111 with a basic characterization. This PR works with
qibo                      0.2.12
qibocal                   0.1.1
qibolab                   0.1.10
qibosoq                   0.1.4
qick                      0.2.249
The platform has the usual issue in this board: the polyphase filter the Qick uses in this firmware does not allow demultiplexing the 3 readouts of the qubits at the same time. I suggested using qubit 1 or qubits 2,3.
@alecandido , is this what you need to test qibosoq?

Hi @JavierSerranoGarcia, sorry for the late reply.
Yes, that's perfectly fine!
In practice, even a single qubit should be sufficient to test all the range of features, since two-qubit gates are not using any new line or feature (other than possibly the pre-distortions), so we should not need to use more than one just to test most of the code, and two only for multiplexing itself (otherwise not even that).
Btw, the PR is based on some old commit, polluting the diff with apparent removals. May I clean it up a bit, without touching tii3q98_zcu111/ at all?

@alecandido, please remove everything you think should disappear.

I just rebased this branch on 0.1, just to be ahead of that, and remove the apparent platforms' deletions (which were already removed in the updated 0.1 branch).


More cast updates

coverage: 98.561%. remained the same
when pulling 0d6dd36ff11ba31f85021d19e7e3afc1ebc32059 on jeromekelleher:improve-casts2
into 6dd3c7d9ea982003effdd5164c6096097d2c30bc on sgkit-dev:main.


Create a page for each individual song

Something is still wrong with the site generation in Vercel - it is not using getStaticPaths in song pages to know the list of song pages that it needs to generate.


Run Time Error while running SparkApplication.scala

I cloned your above project in git, did a sbt compile and sbt package, all went good.
When I try to run the StreamsApplication.scala through Spark-Submit as below, I am getting the error
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/kafka/streams/processor/TopologyBuilder
Please guide how to fix this..
============================================================
spark-submit --class com.knoldus.kafka.examples.StreamApplication --master local[4]  /Users/siva/git/kafka-streams-scala-examples/target/scala-2.12/kafka-streams-scala-examples_2.12-0.1.0-SNAPSHOT.jar
=========================================================
C:\Users\siva\git\kafka-streams-scala-examples>spark-submit --class com.knoldus.kafka.examples.StreamApplication --master local[4]  /Users/siva/git/kafka-streams-scala-examples/target/scala-2.12/kafka-streams-scala-examples_2.12-0.1.0-SNAPSHOT.jar
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/kafka/streams/processor/TopologyBuilder
at com.knoldus.kafka.examples.StreamApplication.main(StreamApplication.scala)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.apache.spark.deploy.SparkSubmit$.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:775)
at org.apache.spark.deploy.SparkSubmit$.doRunMain$1(SparkSubmit.scala:180)
at org.apache.spark.deploy.SparkSubmit$.submit(SparkSubmit.scala:205)
at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:119)
at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
Caused by: java.lang.ClassNotFoundException: org.apache.kafka.streams.processor.TopologyBuilder
at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
at java.lang.ClassLoader.loadClass(ClassLoader.java:357)

@sbklog2017 It looks like you are trying to run Apache Spark with Kafka Streams. If you run your application via sbt run, then it would run fine as application's classpath will contain all the required dependencies. However, if you run your application via spark-submit then you have to provide all the dependencies in the classpath (which are not included in the fat jar using sbt assembly). So, you have to provide them with spark-submit command, like this -
spark-submit --conf spark.driver.extraClassPath=/path/to/kafka-streams-<IP_ADDRESS>.jar:/path/to/kafka-clients-<IP_ADDRESS>.jar --conf spark.executor.extraClassPath=/path/to/kafka-streams-<IP_ADDRESS>.jar:/path/to/kafka-clients-<IP_ADDRESS>.jar --class com.knoldus.kafka.examples.StreamApplication --master local[4] /Users/siva/git/kafka-streams-scala-examples/target/scala-2.12/kafka-streams-scala-examples_2.12-0.1.0-SNAPSHOT.jar

Of course, you have to replace the Kafka JAR(s) version with the one you are using.
I hope this helps!

@sbklog2017 I hope your issue is resolved since there was no response from your side. So, closing this issue now.


rcl_interfaces: 0.6.3-0 in 'crystal/distribution.yaml' [bloom]

Increasing version of package(s) in repository rcl_interfaces to 0.6.3-0:

upstream repository: https://github.com/ros2/rcl_interfaces.git
release repository: https://github.com/ros2-gbp/rcl_interfaces-release.git
distro file: crystal/distribution.yaml
bloom version: 0.7.2
previous version for package: 0.6.2-0

Holding for release of other ROS 2 core packages.


Build(deps-dev): Bump @types/node from 17.0.22 to 17.0.23 (#3181)

Pull request opened by the merge tool on behalf of #3181

Merged in 36accdd5a9be21e42cf3c888c1a104c883064a6d


Add configuration sidebar

What this PR does:
Add configuration sidebar
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #141 (b08e8d4) into main (59ccaa5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #141   +/-   ##
=======================================
  Coverage   46.82%   46.82%
=======================================
  Files          41       41
  Lines        1640     1640
=======================================
  Hits          768      768
  Misses        821      821
  Partials       51       51

Continue to review full report at Codecov.

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


add api-version for templatespecs

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@msyyc , can you confirm if python-track2 failure is a known issue? Thanks.

@msyyc Yuchao Yan FTE , can you confirm if python-track2 failure is a known issue? Thanks.

The failure is not caused by this PR, please ignore it.

@msyyc Yuchao Yan FTE , can you confirm if python-track2 failure is a known issue? Thanks.

The failure is not caused by this PR, please ignore it.

@akning-ms , could you force merge this PR per the above comment from yuchao? Thanks.


Corrupt data from write via Telegraf procstat plugin.

I can reliably get Telegraf to write some combination of data that causes this error when attempting to read the data ERR: invalid character 'j' looking for beginning of value
Steps:

Configure Telegraf with two procstat inputs. As far as I have been able to tell it doesn't matter what the processes are, but that there are two of them. I used this config.
[[inputs.procstat]]
  exe = "telegraf"
  prefix = "telegraf"

[[inputs.procstat]]
 exe = "kapacitord"
 prefix = "kapacitord"

where exe is the name of an executing binary on your host.

Start InfluxDB/Telegraf and let a least one write happen.

Using the telegraf database run the query select * from procstat
You should get the error.

This is likely a bug in Telegraf as well as InfluxDB core.

@sparrc You might want to take a look at this too.

weird, it seems like it writes correctly at first and then gets corrupted after write ~6?
> select * from procstat
name: procstat
--------------
time			cpu_usage		dc		host	memory_rss	memory_swap	memory_vms	pid	process_name
1455315870000000000				us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315870000000000				us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper
1455315880000000000	0.09995787919900383	us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315880000000000	0.09997134762184967	us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper
1455315890000000000	0.5000649242287617	us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315890000000000	0.500063066603335	us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper

> select * from procstat
name: procstat
--------------
time			cpu_usage		dc		host	memory_rss	memory_swap	memory_vms	pid	process_name
1455315870000000000				us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315870000000000				us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper
1455315880000000000	0.09995787919900383	us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315880000000000	0.09997134762184967	us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper
1455315890000000000	0.5000649242287617	us-east-1	tars	17231872	0<PHONE_NUMBER>	882	Slack
1455315890000000000	0.500063066603335	us-east-1	tars	17231872	0<PHONE_NUMBER>	589	SpotifyWebHelper

> select * from procstat
ERR: invalid character 'j' looking for beginning of value

btw, it happens for me with just one procstat defined too

Fixed via #5726


2.1.2 Explore Node Configurations in Gaia

Research and understanding the various configuration options available for Gaia nodes. The goal is to identify the optimal settings and configurations that will support the project's requirements, including data processing, storage, and retrieval. This exploration is crucial for ensuring that the Gaia node is set up correctly and operates efficiently within the pipeline.
Resources:

Guide on how to customize a gaia node (link)
Node config recipes (link)

Starting on this

parallel to #17


Typos on description of MSDN licence types

https://docs.microsoft.com/en-us/rest/api/vsts/memberentitlementmanagement/members/get?view=vsts-rest-4.1#accesslevel
Very minor typo on Licence Type property  Description lists example licence types as  "... Visual Studio Profession, Visual Studio Enterprize"
No edit/comment option on the page for me to flag/update directly, as far as I could see?

Looks like that is over on the MicrosoftDocs/vsts-rest-api-specs repo
https://github.com/search?q=org%3AMicrosoftDocs+Enterprize&type=Code

@tedchamb would you want to fix the typo? Seems there are multiple pages for "Enterprize" issues.

Ah right @EdisonDa + @tedchamb - I created a  PR have I done that correctly?

Thank you @OraDotNetDev for your passion and contribution. Would you mind me to close this issue now? I believe the product team will review the PR and follow up with you.

Sure no worries, happy to help :+1:

Unfortunately we can not take a PR for this fix, because the swagger is generated and the source of the text comes from code that is not yet open sourced.   I appreciate you pointing out the typo, and I will get this fixed up.

@tedchamb Please inform us when the bug is fixed. I reopen the issue until it is truly fixed.


Will miss many sites

Will miss many sites
aquatone_report.html can't Show all

me too.

++++. I've found them in html/requests directory


fix(should.d.ts): remove subsequent declaration

The subsequent declaration is redundant and incorrect. See https://stackoverflow.com/a/48705389/1825329
fixes issue #147

Can you please clarify will this be allowed in ts, after this change?
import ... from 'should';

const a = '1';
a.should.be.Number();

@btd after further investigation into the issue it seems the error was due to a conflict with @types/chai and the above snippet would indeed not have worked with this change.
Collision
Both chai and this library export a should property to the global Object interface. So naturally having both type definitions in the project caused a conflict as the libraries defined a different type to the field respectively. The same name coming from the fact that both types just happen to be named the same.
So I'm closing this pull request. The solution instead is to at the very least not mix the type definitions of the two projects. If we are to continue using both in the project we need to also avoid using either libraries' global object declaration as I do not think it's obvious which library is loaded at all those times.