AttributeError：On testing with SABL head

Hi
I used SABL in faster RCNN. the training process is normal, but some wrong occured when testing. The Traceback is as below:
File "/workspace/mmdetection/mmdet/models/roi_heads/test_mixins.py", line 73, in simple_test_bboxes
    0) if bbox_pred is not None else [None, None]
0) if bbox_pred is not None else [None, None]
AttributeError: 'tuple' object has no attribute 'split'AttributeError
: 'tuple' object has no attribute 'split'

I compared the output of bbox_head and sabl_head and found the tuple which return by the reg_forward() function of mmdet/models/roi_heads/bbox_heads/sabl_head.py may account for this wrong. the simple_test_bboxes() function in mmdet/models/roi_heads/test_mixins.py expects the bbox_pred to be a tensor whose format is tensor([[…, …, …],[…, …, …],[…, …, …]]),but the bbox_pred  return by sabl_head has a format of
(
tensor([[…, …, …],[…, …, …],[…, …, …]]),
tensor([[…, …, …],[…, …, …],[…, …, …]])
)

rather a format of
tensor([
([…, …, …],[…, …, …]),
([…, …, …],[…, …, …]),
([…, …, …],[…, …, …])
])

@longyueying #3849 I have tested it in VOCDataset

@longyueying #3849 I have tested it with VOCDataset

@longyueying #3849 I have tested it with VOCDataset

thanks a lot.  the solution in your PR
 bbox_pred = (
                bbox_pred, ) if bbox_pred is not None else [None, None]```
doesn't split bbox_pred according to the batchsize.
Do you set batchsize as 1 or 2?

No, my batch size is 4, and tested it with 2 gpus

No, my batch size is 4, and tested it with 2 gpus
samples_per_gpu  =  4?

No, my batch size is 4, and tested it with 2 gpus
samples_per_gpu  =  4?

yes

Ok，I will try this. thanks

@longyueying #3849 I have tested it with VOCDataset

thanks a lot. the solution in your PR
 bbox_pred = (
                bbox_pred, ) if bbox_pred is not None else [None, None]

doesn't split bbox_pred according to the proposals number of each image.
Do you set batchsize as 1 or 2?

an easy example:
a = torch.rand(5,5)
a.split(5,0) == (a,)

the number of proposal == bbox_pred[0].shape[0]
so bbox_pred.split() == (bbox_pred, )

@longyueying hello, here is the latest code #3853

@yuzhj ok，this modification seems to be right. thanks a lot!


Handle FormUrlEncoded content in the request

Handle form data in a request

thanks cam. will deploy this now


Replace Konnect w/Konnect Cloud, add UTM params

Replace all "Konnect” to “Konnect Cloud”
Make all links to Konnect Cloud have utm tracking parameters https://konnect.konghq.com/register/?utm_medium=Referral&utm_source=Github&utm_campaig[…]y&utm_content=konnect-promo-in-gateway&utm_term=get-started

Summary

Full changelog

[Implement ...]
[Add related tests]
...

Issue reference

Fix #[issue number]

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


Revise README to be more concise

@JohnathonBowers,
Can you add a Readme for this Github App. Now, the usage of this is much more significant. It is not only for creating changesets. His functionality could be expanded to many tasks, such as for example handle the release notes process. I would think adding the readme we already have as functionalities of the bot and introduce this tool as a bot for automation for OpenSearch project.
What do you think?

Closing this issue as it was resolved by PR #33 .


Missing 1.0.9 release

The 1.0.9 release is mission here on GH would it be possible to add it?

Oh, it should be there, what's missing? I see the 1.0.9 tgz file, for example.

hmm I can not see it on https://github.com/libvips/php-vips-ext/releases

Ah I get it now. It is available directly from the master branch.

Ah OK, yes, I don't think the releases are used much, perhaps they should be disabled.

GitHub recently added a new "Watch" option "Release Only - Be notified of new releases, and when participating or @mentioned." which is quite useful to distro package maintainers.
To my knowledge it is triggered by new released getting added to the releases page so it might be worth keeping around.

OK, that sounds useful. I'll tag 1.0.10 then, that should update the releases page.

Also, there is no tag for 1.0.9, which should be there, I think. :)
I missed the 1.0.9 (and 1.0.10) releases, because I would have never checked for the archives in the actual repository. That's what tags, releases and the associated artifacts on GH are actually really useful for.


Necessary to have separate logl and logp functions with PTSampler?

When using EnsembleSampler one includes both the log-prior and log-likelihood terms into the lnpostfn function. However, for PTSampler one is supposed to supply logl and logp functions for the likelihood and prior.
Is it still reasonable to make a function that calculates the log-posterior, supply it as the logl positional parameter and use a logp function that returns 0? Are there situations when this is reasonable, and situations when it's not reasonable?

In practice, this will often work but you'll have to be careful about interpreting the marginalized likelihood integral that you get out because it isn't defined with improper priors.
I actually don't actually use the PTSampler much so @farr or @willvousden might have more intuitive comments.

Pretty much what Dan said, plus a bit: You will also have to be careful about the adapting temperature scheme in some versions of the PTSampler.  It sets the highest temperature to infinity (i.e. draws from the prior), and therefore can never converge in your problem since the prior is improper.  Even if you don't use the adaptive temperature scheme, you will find that you don't see convergence at the high-temperature end to a common distribution (this is a common test employed to see if you have set Tmax large enough) since there is no such common distribution.    What's so bad about breaking out the prior in your problem?

Thanks for the help everyone. The context is that we use emcee within the lmfit curvefitting project as a way of estimating probability distributions for the parameters. Each parameter can have lower and upper bounds, but there is no assumption on the distribution of the parameter within the bounds if they exist. The user provides an objective function that either returns a vector or a single float. If it returns a vector then it's assumed to be the residuals for the curvefitting problem (a. la. scipy.optimize.leastsq).  If a float is returned then that's taken to either be the log-posterior probability, or a chi2 value.  In any case these values can be used to calculate a log-posterior probability to supply to EnsembleSampler.
The trouble comes with PTSampler; the user doesn't provide a function to calculate the log-prior. Rather they provide something that I can use to calculate the log-posterior with (probably with un-normalised prior if there aren't upper and lower bounds), or they may have gone to the effort of supplying the actual log-posterior (in which case the log-prior term should be zero).
I could ask them to provide a log-prior function, but when you get to that stage I may as well just advise them to use emcee directly because the convenience factor of a curvefitting package is reaching its limit.

Yeah, in this case I would just add your own flat prior, with bounds in the dimensions provided by the user (note that the prior for PT sampling doesn't have to be normalised---even if you are interested in the normalised evidence) and no bounds elsewhere, and hope that the fitting goes well.


Full support for metabolite data

Just setting this issue up to hold all the various miscellaneous tasks necessary to support metabolite data. (Large tasks should still get their own issues.)
A few I can think of now:

Changing x-axis of rank plot to not say "taxa"
Changing search functionality re: metabolite naming conventions
Changing search labels ("Filter numerator to taxa that...") to not say "taxa"
Not saying "Microbe Selections", etc

Maybe just saying "taxa/metabolites" or "microbes/metabolites" is just a good compromise. Or we could have the user specify in the input command if metabolites or taxa are involved, and adjust all these UI elements accordingly... but there very well may be some datasets where both metabolites and taxa are ranked, so that might be an ineffective solution.

WAIT you can just say "features" instead of "taxa" or "metabolites" that makes everything a billion percent easier

All that's left is testing that this works (see #49). Therefore, I'm closing this issue.


Trigger screenshot after failed expect

Take screenshots after each failed expect than only after a failed spec.

Already using it, but i am getting 2 screenshots for each failing spec, though I'd prefer to have only one.


Fix log typos for "Successful"

Saw your webinar on plugins and thought I'd pitch in on this issue #65
Great presentation! Hope to see more of your work in the community soon 🙌

Thank you so much, both for the PR and the feedback on the webinar :D


Unknown lagging for about 100ms every time the distillator run a recipe

There is a lagging for about 100ms every time the distillator run a recipe. Here is the spark linkage: https://spark.lucko.me/X5PvYQ7I5C

The distillation issues are gonna be fixed in 1.3


Title field should also search subtitle

Right now it searches main title only

The logical way to fix this was by implementing a search field in the solr config, so you also get #221

Tested with "valuable," "elements," and "treatise." Title field is now searching both main title and subtitle. Also tested that keyword fielded search still works.


Grade school exercise improvement

The test case getsStudentsInAGrade was intended to test the return value of the grade(grade:Int) when there are multiple students in different grades.
However, this test case was checking for the whole school db, calling the db() method, instead of calling the grade(grade:Int) method.
I aligned the test getsStudentsInAGrade to call the intended method.

@araknoid Sorry, I'm gonna close this for now due to inactivity.
Feel free to reopen this PR!


[Bug report] In parameter sweep, specifying subsys_update_info does not work as expected when deepcopy = True.

Describe the bug
When I set deepcopy = True and specify subsys_update_info when initializing the parameter sweep, the sweep can't recognize the subsystem objects in the subsys_update_info. The ParameterSweep object will misinterpret the subsys_update_info and think the bare spectrum don't need to be updated. In the end, the bare spectrum are calculated once and copied for the rest of the sweep, resulting in the incorrect result.
To Reproduce
tmon = scq.Transmon(
    EJ=8.0,
    EC=0.3,
    ng=0.3,
    ncut=50,
    truncated_dim=5,
    id_str="tmon"
)

hspace = scq.HilbertSpace([tmon])

def update_hilbertspace(EJ):
    tmon.EJ = EJ

sweep = scq.ParameterSweep(
    hspace,
    paramvals_by_name={
        "EJ": [7, 8, 9],
    },
    update_hilbertspace=update_hilbertspace,
    evals_count=3,
    subsys_update_info={"EC": ["tmon"]},
    deepcopy=True,
)

print(sweep["evals"])

We expect seeing three different list of evals, while the current code gives the repeated eval lists.

Might be related to #209. It is possible that both are the same issues. This example is perhaps a nice starting point.

Hey @saipavanc. I made some mistakes in uploading the code to reproduce the error in this issue. The argument in creating the ParameterSweep object should really be subsys_update_info={"EJ": [tmon]}.
Your correction seems to look fine - using id_str to identify the subsystem. However, scqubits actually accept the definition of two qubits with the same id_str if the user specifies. I feel this is a potential issue and scqubits should not allow it.
One potential solution is to do a few checks in creating the ParameterSweep object, for repetitive id_str and illegal subsys_update_info.

Or just for the purpose of a valid parameter sweep, can we just run through the subsystem_list of a Hilbert space and find repetitive names?

it is now fixed in #252

Thanks @Harrinive !


resizable widget not working

Hello,
I am so sorry to bother, but can you give me a specific idea of what I need in place to get the widget vertical and horizontal resize to work ?
In my application, I can only resize the widget's width but I lost the ability to resize the height (vertical).
I'm searching high and low throughout the demo code (apart from my application) to see where the resizable feature kicks in - i.e. in the demo code I downloaded, the resizing work fine for all widgets.
I see jquery.ui.resizable.css, but I cannot figure out how to make that work in my application
example widget definitions I've used as a template to work from :
     (function () {
'use strict';
angular.module('rage')
    .factory('widgetDefinitions', ['RandomDataModel', 'ColumnChartDataModel', 'TreeGridDataModel', widgetDefinitions])

/* define the widget definitions  */
function widgetDefinitions(RandomDataModel, ColumnChartDataModel, TreeGridDataModel) {
    var size = {width: '30%',height: '60px'};
    return [
       {
           name: 'datamodel',
           directive: 'wt-scope-watch',
           dataAttrName: 'value',
           dataModelType: RandomDataModel
       },
      {
          name: 'resizable',
          templateUrl: 'app/shared/template2/resizable.html',
          attrs: {
              class: 'demo-widget-resizable'
          }
      }
    ];
}
 })();

thanks for your time...
Bob

I can literally see the resizer class binded to the DOM elements in F12 Inspector:
   .widget .widget-s-resizer

I can also see the .widget-resizer-marquee div changing dynamically as I resize. Yet still, the resize bounced back to its original height.


Invalid strings should turn red

Expected behavior:
When someone inputs and invalid string the string should turn red.
Screenshots:
expected:

current:

Fixed in v0.7.11.


Review TileDB-SOMA for spec compliance

Once TileDB-SOMA RC0 is code-complete, review the TileDB-SOMA public API to ensure compliance with the SOMA abstract specification w.r.t class names, class properties, and method names , and method signatures.
The review team is:
@atolopko-czi , @mlin , @thetorpedodog , Pablo Garcia-Nieto, and Nicholas Sofroniew
TBD: division of labor.
Any discrepancies found can simply be noted on this issue as comments. @atolopko-czi will coalesce into one or more issues for any fixes that may be needed.
We may need to review some discrepancies with the larger team if it is not clear whether it is the implementation or the spec that should be corrected.

@johnkerl @@thetorpedodog We still have many "⚠" notes in the spec.  Do we need to address these before performing a spec compliance check or proceed as is?

[ ] "Issue: are there parts of the Arrow type system that we wish to explicitly exclude from SOMA?"
[ ] "Issue: it would be a good idea to factor SOMAExperiment and SOMAMeasurement into separate sections."
[ ] "[SOMADataFrame] To be further specified: all methods need specification"
[ ] "[SOMADenseNDArray] To be further specified: this is incomplete."
[ ] "[Operation: read()] Issue: support for other formats, such as Arrow Table, is under discussion."
[ ] "[General Utilities] To be further specified"
[ ] "[Value Filters] To be further specified"
[ ] "[Other Issues (open issues with this doc)] Issues to be resolved: ..."

Tracking review progress and action items here.

cc @bkmartinjr

@thetorpedodog @mlin Found and fixed some discrepancies in spec in https://github.com/single-cell-data/SOMA/pull/145, but looking for feedback on how to handle the following additional discrepancies:

[ ] Collection.add_new_collection: cls param: cls (impl) vs type (spec)
[ ] Should spec mention closed and mode properties? (e.g. it does mention context property)
[ ] DataFrame.write: values param only admits pa.Table (impl) vs spec admits pa.Table or pa.RecordBatch

Couple more discrepancies I noted -- the SOMACollection spec says that has() and iterator are available operations, but this is currently not true. I am not sure whether there is a plan to add these in the future or not, but right now it will output a has no attribute error if those methods are called on a Collection object.


Merge latest commits from Sufia

:face_with_head_bandage:

I think this will need to be rebased on #575 in order for tests to pass.


Support optional arguments

We should distinguish between optional and required fields in the UI.
Right now all fields are optional, so technically "support non-optional arguments" is probably a better issue name.

superseded by #38


[NuclideReason] File highlight defaults to "Regular Expression (babel)" instead of reason

Who has regex files anyway? Maybe we can just remove the *.re suffix from the regex syntax (assuming that's what's causing this)?

Should take 5mins

Man, software engineering...
It's most likely caused by this. And I don't think atom has a way of specifying priority. I've disabled language-babel for now.

Opened gandm/language-babel#194 to see if they actually need the re extension.

Awesome, closing!


[ruff-0.8] [ruff] Stabilise parenthesize-chained-operators (RUF021)

Summary
This rule has been in preview for a long time and its implementation has been stable for a long time. There are no open issues about the rule and no known issues in the implementation. The documentation seems clear.
It's an opinionated stylistic rule, but also probably a change that most people would agree with (in our subjective opinion).
Test Plan
We'll wait to see what the ecosystem check says!

Linter (stable)
ℹ ecosystem check detected linter changes. (+63 -0 violations, +0 -0 fixes in 9 projects; 45 projects unchanged)

As expected, this impacts quite a few projects. But from a quick skim, these all look like "true positives", and I think changing the code to conform to the rule would result in more readable code in all places.


Docs on client

Co-Authored-By: Ryu JuHeon<EMAIL_ADDRESS>

Summon @SaidBySolo

Summon @SaidBySolo

?

?

@sannoob

@sannoob


Add planner name to trajectory info

Add planner id to SubTrajectory SolutionInfo.

Why not optionally set (or add to) the comment instead of creating a new field?
Also, the pipeline planner already has a property "planner" which is basically what you implement via getPlannerId.
I would propose to implement this one in the other planners as well and use the property from there instead of implementing it again.
I did not review the rest of this patch.

Thanks for your review!

Why not optionally set (or add to) the comment instead of creating a new field?

I think it is nicer to have a specifically named field rather than an arbitrary "comment" field for this data

Also, the pipeline planner already has a property "planner" which is basically what you implement via getPlannerId.

The refactored version of the PipelinePlanner we're using here does not. See PR against main mtc repository for details

@sjahr: I'm going through all commits PickNik committed to the ros2 branch. For this one, could you please argue why this additional field is actually needed? How do you handle generators, which also generate trajectories but do not have a planner necessarily?

I think it is useful to include information about the planner that created a solution into the solution info. We're using it e.g. for debugging and introspection. I did not think about the generator use-case, but I think in that case the field could be left empty or populated with something like "generated" (depending on how it was actually created). Before, the parallel planning API I guess it was possible to infer the planner that was used from the pipeline id but now, the pipeline planner uses multiple algorithms and without this field it would be harder to say which one actually provided the solution. Just out of curiosity: Are there any trajectory generator stage examples in the mtc main repository?


Bump version to 0.3.5

Could you merge this and publish a new version to get the byteorder bump to 1.0 in Servo please?

Sure, although it seems that you didn't actually bump the byteorder version :)

Thanks!

Done, 0.3.5 is published.


is this possible

i have a bot on telegram alredy but i want to know is it possible to get message from discord post it to telegram and then get the reply from the bot on telegram and post it on discord?

No, bot's don't see messages from other bots on telegram.
https://core.telegram.org/bots/faq#why-doesn-39t-my-bot-see-messages-from-other-bots


mWeb - Part of the screen is covered with black rectangle, scrolling is stuck on MC page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.57-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5176953
Issue reported by: Applause - Internal Team
Action Performed:

Navigate to https://staging.new.expensify.com/
Sign in page
Type email address and tap Done to exit keyboard
Tap continue to go to MC page
Try to swipe up and down and observe the page

Expected Result:
Expensify animation is shown under Sign in module on the MC entering page
Actual Result:
Part of the screen is covered with black rectangle on MC page. Scrolling gets stuck after swiping up and down a few times
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/3e764934-9411-47c4-87ff-05ea5deeffc8
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Part of the screen is not visible when going to the magic code input page.
What is the root cause of that problem?
This only happens if we open a keyboard first on the login/email page. The sign in page enable shouldEnableMaxHeight which uses the height from useWindowDimensions. In useWindowDimensions, every time we focus on an input, we set the height immediately from cachedViewportHeightWithKeyboardRef value.
https://github.com/Expensify/App/blob/10454f06334ae3d07d4a28a81c5c79f3d2112c53/src/hooks/useWindowDimensions/index.ts#L51-L58
cachedViewportHeightWithKeyboardRef holds the height of the window when a keyboard is shown.
https://github.com/Expensify/App/blob/10454f06334ae3d07d4a28a81c5c79f3d2112c53/src/hooks/useWindowDimensions/index.ts#L97-L102
For example, if the window height is 700, but when the keyboard shows it becomes 400, then cachedViewportHeightWithKeyboardRef value is 400. In our case, when we go to the magic code input, we auto-focus on the input and the height from cachedViewportHeightWithKeyboardRef is set as the useWindowDimensions height.
It was first added in https://github.com/Expensify/App/issues/33546 to update the height quickly without waiting for window.visualViewport resize event (which rn-web uses to update the dimensions). However, in our case, the soft keyboard doesn't show (it's a known issue on safari if the focus isn't directly triggered by user action), so resizing it doesn't make sense.
What changes do you think we should make in order to solve the problem?

I don't think we can know whether a keyboard will show from a focus event or no, so I'm suggesting removing shouldUseCachedViewportHeight from SignInPage.
https://github.com/Expensify/App/blob/10454f06334ae3d07d4a28a81c5c79f3d2112c53/src/pages/signin/SignInPage.tsx#L295-L298
shouldUseCachedViewportHeight is currently being used in SignInPage and WorkspaceInviteMessagePage. It wasn't applied globally because it causes issues on some pages. So, they only add it for WorkspaceInviteMessagePage. But then, in this issue, we add shouldUseCachedViewportHeight for SignInPage too.
Is it safe to remove shouldUseCachedViewportHeight from SignInPage?
The proposal there explain why we need to use shouldUseCachedViewportHeight, but the root cause is actually unclear. Also, if we look at the PR: https://github.com/Expensify/App/pull/44999, it actually addressed the visual viewport scroll issue, instead of the wrong visual viewport height which is mentioned on the proposal. (btw, the issue is that a blank space appears above the keyboard)
And yes, the issue is about the visual viewport scroll. The visual viewport scroll issue is well explained in this other issue.
And it also happens on iOS safari, which isn't fixed yet. (more explanation below)
https://github.com/user-attachments/assets/cc667376-2400-4fce-9f29-43bbbb38d405
So I believe we can safely remove shouldUseCachedViewportHeight since that doesn't fix the issue where it was added and just causes a worse issue.

For the iOS safari viewport scroll issue, we actually already has a solution for that, for all web, that is to scroll it back to 0,
https://github.com/Expensify/App/blob/10454f06334ae3d07d4a28a81c5c79f3d2112c53/src/hooks/useTackInputFocus/index.ts#L31-L39
except for iOS chrome which was added in https://github.com/Expensify/App/pull/44999.
However, there looks like a bug on iOS safari where if the visual viewport offsetTop is smaller than 59, then scrollTo doesn't do anything. We can apply the same workaround as iOS chrome, but it causes multiple scrolling.
https://github.com/user-attachments/assets/cad01308-13d2-4046-97ba-b4e0e1809125
Another workaround for iOS safari only is to scroll to -1.
 else if (Browser.isMobileSafari() && (window.visualViewport?.offsetTop ?? 0) > 1) {
    window.scrollTo({top: -1});
} else {

I think I agree with @bernhardoj 's investigations on the issue. But I'll need to test the solutions they proposed to avoid unnecessary back-and-forth.

I think we can go with @bernhardoj 's proposal - Removing shouldUseCachedViewportHeight and the workaround for mobile Safari make sense to me as it's an edge case of mobile Safari.
🎀👀🎀 C+ reviewed

PR is ready
cc: @eh2077

Looks like the useOnyx migration causes the issue. I'll open the PR without doing the migration.

New PR is ready
cc: @eh2077

Requested in ND.

I was OOO last week; I will complete this today.

Payment summary:
Contributor proposal: @bernhardoj owed $250 via NewDot $250 PENDING
@eh2077 owed $250 via NewDot C+ $250 PENDING

$250 approved for @eh2077

$250 approved for @bernhardoj


build: move number-format and time-format to core

🏠 Internal
Follow up for #729 , merging more packages to @superset-ui/core. This time the d3 formatters.

@kristw Tests are failing because encodable depends on an older version of @superset-ui/number-format. What do you think is a good way to reduce such inter-dependencies?

@kristw Tests are failing because encodable depends on an older version of @superset-ui/number-format. What do you think is a good way to reduce such inter-dependencies?

I am working to decouple encodable from superset-ui. Should take a few more days.

Wonder if there is an easy way to disable certain plugin building / testing temporarily to unblock you.

Wonder if there is an easy way to disable certain plugin building / testing / publishing temporarily to unblock you.

We can move the packge to temporary-plugins/? The only block package is presets-chart-xy, which we only use for Boxplot in Superset.

Wonder if there is an easy way to disable certain plugin building / testing / publishing temporarily to unblock you.

We can move the packge to temporary-plugins/? The only block package is presets-chart-xy, which we only use for Boxplot in Superset.

sgtm

There are a couple of other plugins also use encodable. It's too much work to move them (and their storybook examples) back and forth. I'll just leave this PR here and work on merging other packages instead.


feat: adding conflict error to error definition

Description
Adding new HTTP return code (409) which will be returned for conflict
References
First step in addressing https://github.com/openfga/openfga/issues/1080
Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Updated to avoid introducing new internal code


feat: Add multi get

Add MultiGetAsyc following the example of multi_get in the Python SDK.
Closes #52

@poppoerika is test failure because of issue in alpha still?

@eaddingtonwhite - It failed because of the alpha issue, but it looks like the issue has been resolved?
At least, now it's passing all the tests.


[BUGFIX] fix pass name

Description
Minor fix related to SwitchTrainOp pass's name.
Checklist

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

cc @awslabs/raf-reviewer

Thanks @yetiansh


Pushwoosh initing without calling Pushwoosh

Ok, this is extremely annoying. After installing Unity SDK when I run the app for the first time I get a popup asking if I want push notifications enabled. Basically the first line on the log is:
[UIApplication(Pushwoosh)] Pushwoosh: Initializing application runtime
This is horrible. I didn't even call anything on pushwoosh on my scripts yet. I don't want that popup at all until I decide to ask for push notifications permissions (usually after a few steps on the game). Why would you guys ask pushwoosh to display the notification without our consent?

duplicate: https://github.com/Pushwoosh/pushwoosh-unity/issues/3


[HW]

Name: Farha Parveen Padiyathu Puthenkattil Fazal
Email<EMAIL_ADDRESS>Linkedin Profile: www.linkedin.com/in/farhaparveenpf
Attach the homework step I screenshots below:

![step1](https://user-images.githubusercontent.com/56920928/126179798-bc3065e7-7dad-454f-b5d7-82c03719425b.jpeg)
Attach the homework step II screenshots below:
<SCREEN

SHOTS>

Hey Farha Parveen Padiyathu Puthenkattil Fazal, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/QhaGhU1xQBuo9flptsZm7Q


Feature: Working on an idea for having a visual tool for beginners to configure webpack setup

I have been working on an idea for providing a visual tool for web pack setup.
For a beginner, one should go through all the docs to understand and do a minimal setup of web pack.
I think learning process should be simple. I am working on a visual tool/playground for web pack, where the user can select options to generate a config file. Once he generates the config file, he can copy it and paste this file into his project with few modifications.
In this way, a beginner can easily understand web pack and he can visually see options along with output.
I wanted the community to give their opinions and pain points so that collectively we can give better options to this new tool.
If I raise a PR will it be accepted and if anyone has any opinions or use cases??
Overall trying to make Web pack more reachable and understandable to everyone. :relaxed:

If you tool is good and popular, feel free to add a link to the documentation.


[core-paging] Update changelog for release

Packages impacted by this PR
core-paging
Issues associated with this PR
Describe the problem that is addressed by this PR
Update changelog for release
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Stage 1

UI https://github.com/kiegroup/backstage-plugins/pull/13
Create a way to execute the stored scorecards from the database and store the result in the db as well #17
Create a way to define scorecards and put them inside a database #16

@lucamolteni ok updated.


Fix DLQ reason and description not work

Description
Resolve #40951
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Hi, @saragluna .
Please help to review this PR.

LGTM, but can we hold until this #41131 gets merged, and change this PR's target branch to the main branch?

@saragluna .
Sure, will do.

Closing in favor of https://github.com/Azure/azure-sdk-for-java/pull/41172


Error while cmake a program

Hi @tonyp7,
I want to ask some question about cmake. What is payload for cmake this program?
I try to cmake but not success.
$ sudo cmake -DVENDOR=espressif -DBOARD=esp32_devkitc -DCOMPILER=xtensa-esp32 -DCMAKE_TOOLCHAIN_FILE='/home/monhack/esp/esp-idf/tools/cmake/project.cmake' -DCMAKE_C_COMPILER='/home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc' -DCMAKE_CXX_COMPILER='/home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-g++' -DCMAKE_ASM_COMPILER='/home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc' -DCMAKE_MAKE_PROGRAM='/home/monhack/.espressif/tools/ninja' -DAFR_BOARD='espressif.esp32_devkitc' -S . -B wifi_1

CMake Error at CMakeLists.txt:5 (include):
  include could not find load file:

    /tools/cmake/project.cmake

CMake Warning at /home/monhack/esp/esp-idf/tools/cmake/idf.cmake:24 (message):
  IDF_PATH environment variable not found.  Setting IDF_PATH to
  '/home/monhack/esp/esp-idf'.
Call Stack (most recent call first):
  /home/monhack/esp/esp-idf/tools/cmake/project.cmake:7 (include)
  wifi_1/CMakeFiles/3.18.1/CMakeSystem.cmake:6 (include)
  CMakeLists.txt:7 (project)

-- The C compiler identification is GNU 5.2.0
-- The CXX compiler identification is GNU 5.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - failed
-- Check for working C compiler: /home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
-- Check for working C compiler: /home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc - broken
CMake Error at /usr/local/share/cmake-3.18/Modules/CMakeTestCCompiler.cmake:66 (message):
  The C compiler

    "/home/monhack/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/monhack/Documents/Kerjaan/esp32-wifi-manager/examples/default_demo/wifi_1/CMakeFiles/CMakeTmp

    Run Build Command(s):/home/monhack/.espressif/tools/ninja cmTC_90512/fast && ninja: error: loading 'build.ninja': No such file or directory

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:7 (project)

-- Configuring incomplete, errors occurred!
See also "/home/monhack/Documents/Kerjaan/esp32-wifi-manager/examples/default_demo/wifi_1/CMakeFiles/CMakeOutput.log".
See also "/home/monhack/Documents/Kerjaan/esp32-wifi-manager/examples/default_demo/wifi_1/CMakeFiles/CMakeError.log".

please help me to solve this problem, Thanks :)

Why don't you use ESP-IDF to build project?
$idf.py build

Thanks @hanhdt,
i follow you instruction, but error
/home/pckharisma-2/esp/esp-idf/tools/idf.py build flash monitor -p /dev/ttyUSB0
Setting IDF_PATH environment variable: /home/pckharisma-2/esp/esp-idf
WARNING: Support for Python 2 is deprecated and will be removed in future versions.
The following Python requirements are not satisfied:
gdbgui==<IP_ADDRESS>
pygdbmi<=<IP_ADDRESS>
python-socketio<5; python_version<="2.7"
kconfiglib==13.7.1
Please follow the instructions found in the "Set up the tools" section of ESP-IDF Getting Started Guide
Diagnostic information:
    IDF_PYTHON_ENV_PATH: (not set)
    Python interpreter used: /usr/bin/python
    Warning: python interpreter not running from IDF_PYTHON_ENV_PATH
    PATH: /home/pckharisma-2/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/pckharisma-2/esp/xtensa-esp32-elf/bin

ESP-IDF v4.4-dev-4-g73db14240-dirty

finally solved


CASMNET-690 Add CMN for bifurcated CAN and make VLAN handling consistent

Summary and Scope
CMN
The CMN (Customer Management Network) will provide ingress administrative access to the system instead of the CAN as part of the CAN bifurcation process.  The CAN will remain in place, but be relegated to non-administrative (ie. regular user) traffic. This changes adds plumbing for the CMN.
Default VLAN: 6
Default IPv4 Subnet: <IP_ADDRESS>/24
New files:
        dnsmasq.d/
            CMN.conf
        pit-files/
            ifcfg-vlan006

New structure in sls_input_file.json:
        "CMN": {
            "Name": "CMN",
            "FullName": "Customer Management Network",
            "IPRanges": [
                "<IP_ADDRESS>/24"
            ],
            "Type": "ethernet",
            "ExtraProperties": {
                "CIDR": "<IP_ADDRESS>/24",
                "VlanRange": [
                    6
                ],
                "MTU": 9000,
                "Subnets": [
                    {
                        "FullName": "CMN Static Pool MetalLB",
                        "CIDR": "<IP_ADDRESS>/28",
                        "Name": "cmn_metallb_static_pool",
                        "VlanID": 6,
                        "Gateway": "<IP_ADDRESS>"
                    },
                    {
                        "FullName": "CMN Dynamic MetalLB",
                        "CIDR": "<IP_ADDRESS>/25",
                        "Name": "cmn_metallb_address_pool",
                        "VlanID": 6,
                        "Gateway": "<IP_ADDRESS>"
                    },
                    {
                        "FullName": "CMN Bootstrap DHCP Subnet",
                        "CIDR": "<IP_ADDRESS>/24",
                        "IPReservations": [...]

VLAN Handling
Previously VLANs from cli input were not consistently honored in the network VlanRanges or other structures.  This changeset also provides consistent defaults handling in structures, overrides via cli-input and output that has far less inconsistent and unused data.
VlanRange is now consistent with cli input and used subnet VLANs:
        "HMN": {
            "Name": "HMN",
            "FullName": "Hardware Management Network",
            "IPRanges": [
                "<IP_ADDRESS>/17"
            ],
            "Type": "ethernet",
            "ExtraProperties": {
                "CIDR": "<IP_ADDRESS>/17",
                "VlanRange": [
                    4
                ],
                "MTU": 9000,
                "Subnets": [...]

OTHER:

Yaml output of network files has been removed.  Few people or scripts used this and those scripts are being changed.
CLI network parameters that clash for CMN and CAN are now handled with appropriate error messaging (e.g. if CAN or CMN static/dynamic pools are not in the respective subnet a prescriptive error occurs.

Issues and Related PRs

Resolves CASMNET-690

Testing
LIST THE ENVIRONMENTS IN WHICH THESE CHANGES WERE TESTED.
Tested on:

Generation and SLS load on Odin and Hela.
Generation and SLS load on Surtur.

Were the install/upgrade based validation checks/tests run?(goss tests/install-validation doc) YES
Was a fresh Install tested? YES
Was an Upgrade tested?      NO - handled by another process.
Was a Downgrade tested?     NO
Risks and Mitigations
This is required for Bifurcated CAN in CSM 1.2.  Deployment earlier than this should not cause any impacts as the new structures will not be used by anything.

@trad511 can you rebase this PR?

@trad511 can you rebase this PR?

Looks like there's two issues, the extra commits and the conflicting Jenkins.github file.

@trad511 these changes overall look O.K. but I need some education about this NIC. The PR says it replaces the CAN, but I still see the CAN NIC in this code.
Will the NCN have these four now?
* `vlan002`

* `vlan004`

* `vlan006`

* `vlan007`

@taylorludwig @denniswalker we may need to update GOSS to check for VLAN006.

@rustydb I've updated the PR text to this same effect but CMN replaces the CAN for administrative access (ssh, admin APIs etc...).  The CAN remains, but will be for users running jobs.  A future PR will pare down CAN IP's for access, but that's a breaking change so avoided here.

Great. Thanks, just taking this into account for the NIC renaming. I'm good with my review.

I had used the web gui to update your branch, but that does a fast-forward merge.. so I rebased and fixed it.


Math 1458

Tests and fix for MATH-1458

Coverage increased (+0.01%) to 90.023% when pulling d97cc96e285b6cf382d96f7213e5a62355bc0aa2 on aherbert:MATH-1458 into f43069ac6d281b8367dad6f78def4b8336a11ff0 on apache:master.


On Firefox, the Encounter Generator doesn't use the party size/levels

I tried this with both a Generic Party and a Detailed party.  In both cases,  I get an encounter generated for a party of a single level 1 PC.
On Chrome, everything works perfectly.
Test Machine
MacOS: Ventura 13.2.1
Firefox  Version:  110.0.1
Steps to Reproduce

Open FF and navigate to https://koboldplus.club/
Set party to 4 level 10 players, and include XP (or don't, whatever)
Select Encounter Type
Click the Roll button.

On FF,  regardless of the number and level of players,  I always get just a small handfull of CR 1/8 monsters.

We've made some improvements and changes over the last few months that appear to have fixed this, at least when I try in Firefox on my end. That said, let us know if you're still seeing it! Happy to take a closer look.


"docker daemon" deprecation message doesn't use the new versioning scheme

https://github.com/docker/docker/blob/17.03.x/cmd/docker/daemon_unix.go#L27
Mentions docker 1.16, should instead be 17.XX.

Fixed in https://github.com/docker/docker/issues/31409, thanks for reporting!


Attributed String is not working with PDFTableCell

Hello,
I want to display hyperlink with text. So I used attributed string. It is not working with PDFTableCell.
Please review following code :
let content = PDFTableContent(type: .attributedString, content: content)
let cell = PDFTableCell(content: content, alignment: alignment)
So can you please help me ?

Are you using the develop branch?

We are using pod.
pod 'TPPDF'

Hyperlinks are not released yet (see the CHANGELOG on develop)
Therefore to use this feature you would have to use the develop branch, which contains unstable code. Please see here how to do that:
http://guides.cocoapods.org/using/the-podfile.html

Hello
It is working fine with below code
let title = NSMutableAttributedString(string: "(txt)", attributes: [
.foregroundColor:  UIColor.blue,
.link: "http://www.google.com"
])
But getting crash with below code
let title = NSMutableAttributedString(string: "(txt)", attributes: [
.foregroundColor:  UIColor.blue,
.link: "http://www.google.com",
.underlineStyle: NSUnderlineStyle.single
])
Getting crash at : PDFCalculations.swift
internal static func calculateTextFrameAndDrawnSizeInOnePage(frame: CGRect,
text: CFAttributedString,
currentRange: CFRange) -> (CGRect, CTFrame, CGSize) {
let framesetter = CTFramesetterCreateWithAttributedString(text)
}
How do I give underline to text ?
Please help me

Hi there, I just released 2.0.0 where this is could be resolved.
Please take a look and let me know if it is still an issue

@vimal-bhadra anthing new on this issue?

I'll close this for inactivity. If it's still relevant with the newest TPPDF version, please reopen


How to change image watermark and text  postion in screen in OpenGlRtmp.java

How to change image watermark and text  postion in screen in OpenGlRtmp.java

To change watermark you have this methods:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L568
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L583
This work for ImageStreamObject, GifStreamObject and TextStreamObject.
If you want change position of Android view filter you need create a Layout set your view in the layout and send layout to filter instead of only your view.
Example send Button in center of the screen.
In XML:
<RelativeLayout
      android:layout_width="match_parent"
      android:layout_height="match_parent"
      android:id="@+id/rl_to_send"
      >
    <Button
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:text="Text to send"
        android:layout_centerInParent="true"
        />
  </RelativeLayout>

In Java code send RelativeLayout:
AndroidViewFilterRender androidViewFilterRender = new AndroidViewFilterRender();
androidViewFilterRender.setView(findViewById(R.id.rl_to_send));
rtmpCamera1.setFilter(androidViewFilterRender);


fix: sorting by user for Query History list view

SUMMARY

Fixing a bug that snuck in with https://github.com/apache/incubator-superset/pull/11702 breaking sorting by user in the list view.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A
TEST PLAN

manually verify that sorting by user in Query History does not result in an api exception.

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11897 (fd5e3e3) into master (1842c56) will decrease coverage by 4.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11897      +/-   ##
==========================================
- Coverage   67.64%   63.46%   -4.18%
==========================================
  Files         927      461     -466
  Lines       45014    28368   -16646
  Branches     4308        0    -4308
==========================================
- Hits        30448    18005   -12443
+ Misses      14462    10363    -4099
+ Partials      104        0     -104

Flag
Coverage Δ

cypress
?

javascript
?

python
63.46% <ø> (-0.68%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/db_engine_specs/hive.py
54.08% <0.00%> (-29.97%)
:arrow_down:

superset/db_engine_specs/presto.py
69.76% <0.00%> (-12.53%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/utils/celery.py
96.42% <0.00%> (-3.58%)
:arrow_down:

superset/examples/world_bank.py
97.10% <0.00%> (-2.90%)
:arrow_down:

superset/examples/birth_names.py
96.51% <0.00%> (-2.33%)
:arrow_down:

superset/views/database/mixins.py
80.70% <0.00%> (-1.76%)
:arrow_down:

superset/result_set.py
96.69% <0.00%> (-1.66%)
:arrow_down:

... and 475 more

Continue to review full report at Codecov.

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


cloud-controller-manager exited

Environmental Info:
K3s Version:

root@<hidden>:/home/devuser# k3s -v
k3s version v1.25.4+k3s1 (0dc63334)
go version go1.19.3

Node(s) CPU architecture, OS, and Version:

AWS EC2 instance:
root@<hidden>:/home/<hidden># cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.6 LTS"
root@<hidden>:/home/<hidden>#
root@<hidden>:/home/<hidden># uname -a
Linux <hidden> 5.11.0-1016-aws #17~20.04.1-Ubuntu SMP Thu Aug 12 05:39:36 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
root@<hidden>:/home/<hidden># cat /proc/cpuinfo | grep 'vendor' | uniq
vendor_id	: AuthenticAMD
root@<hidden>:/home/<hidden># cat /proc/cpuinfo | grep 'model name' | uniq
model name	: AMD EPYC 7R32
root@<hidden>:/home/<hidden># cat /proc/cpuinfo | grep processor | wc -l
32

Cluster Configuration: 1 server only

Describe the bug:

cloud-controller-manager exited due to Apr 20 13:02:35 <hidden> k3s[84782]: time="2023-04-20T13:02:35+02:00" level=error msg="cloud-controller-manager exited: unable to load configmap based request-header-client-ca-file: configmaps \"extension-apiserver-authentication\" is forbidden: User \"k3s-cloud-controller-manager\" cannot get resource \"configmaps\" in API group \"\" in the namespace \"kube-system\"", the role was created at "2023-04-20T13:02:30Z" and the role binding was created at "2023-04-20T13:02:36Z". It means that at the moment when cloud-controller-manager tries to get config map, it has not permissions due to lack of role binding. And it causes that taint node.cloudprovider.kubernetes.io/uninitialized=true:NoSchedule remains on the node and pods stay in pending state.
Restarting the k3s service resolves the issue, cloud-controller is restarted and the role binding already exists, that is probably why it is working after k3s restart, but it is not how it should work.
Steps To Reproduce:

Installed K3s:

INSTALL_K3S_VERSION=v1.25.4+k3s1 INSTALL_K3S_EXEC="--disable traefik --service-node-port-range 30000-30050
--kube-apiserver-arg=enable-admission-plugins=NodeRestriction,ServiceAccount --kube-apiserver-arg=request-timeout=5m0s
--kube-apiserver-arg=audit-log-path=/var/lib/rancher/k3s/server/logs/audit.log
--kube-apiserver-arg=tls-cipher-suites=TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305,TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305,TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384
--kube-apiserver-arg=audit-log-maxbackup=10 --kube-controller-manager-arg=terminated-pod-gc-threshold=1000
--kube-controller-manager-arg=leader-elect-lease-duration=30s --kube-controller-manager-arg=leader-elect-renew-deadline=20s
--kube-controller-manager-arg=bind-address=<IP_ADDRESS> --kube-scheduler-arg=bind-address=<IP_ADDRESS>
--kubelet-arg=streaming-connection-idle-timeout=5m --kubelet-arg=rotate-server-certificates=true
--kubelet-arg=eviction-hard=imagefs.available<20Gi,memory.available<100Mi,nodefs.available<20Gi,nodefs.inodesFree<5%
--kubelet-arg=eviction-minimum-reclaim=imagefs.available=1Gi,nodefs.available=1Gi" ./k3s_installer.sh

I do not have steps to reproduce, as it occurs periodically.
Expected behavior:

cloud-controller-manager running properly, maybe it should ensure that role and role binding are created before trying to get the configmap.
Actual behavior:

cloud-controller-manager exited, it was not able to get the required config map due to lack of permissions.
Additional context / logs:

role details:
root@<hidden>:~# kubectl get role -n kube-system extension-apiserver-authentication-reader -o yaml
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
  annotations:
    rbac.authorization.kubernetes.io/autoupdate: "true"
  creationTimestamp: "2023-04-20T13:02:30Z"
  labels:
    kubernetes.io/bootstrapping: rbac-defaults
  name: extension-apiserver-authentication-reader
  namespace: kube-system
  resourceVersion: "176"
  uid: bd705934-437e-45e4-9233-b6982d11d1d0
rules:
- apiGroups:
  - ""
  resourceNames:
  - extension-apiserver-authentication
  resources:
  - configmaps
  verbs:
  - get
  - list
  - watch

role binding details:
root@<hidden>:~# kubectl -n kube-system get rolebinding -o yaml k3s-cloud-controller-manager-authentication-reader
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  annotations:
    objectset.rio.cattle.io/applied: <hidden>
    objectset.rio.cattle.io/id: ""
    objectset.rio.cattle.io/owner-gvk: k3s.cattle.io/v1, Kind=Addon
    objectset.rio.cattle.io/owner-name: ccm
    objectset.rio.cattle.io/owner-namespace: kube-system
  creationTimestamp: "2023-04-20T13:02:36Z"
  labels:
    objectset.rio.cattle.io/hash:<PHONE_NUMBER>c5482413c7f05e837e9b88f02ad052
  name: k3s-cloud-controller-manager-authentication-reader
  namespace: kube-system
  resourceVersion: "247"
  uid: b8168fc0-0d35-461a-b5a4-8d41a7215c4d
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: extension-apiserver-authentication-reader
subjects:
- apiGroup: rbac.authorization.k8s.io
  kind: User
  name: k3s-cloud-controller-manager
  namespace: kube-system

journalctl logs related to cloud controller:
root@<hidden>:~# journalctl -u k3s.service | grep cloud-controller
Apr 20 13:02:25 <hidden>k3s[84782]: time="2023-04-20T13:02:25+02:00" level=info msg="certificate CN=k3s-cloud-controller-manager signed by CN=k3s-client-ca@1681995745: notBefore=2023-04-20 13:02:25 +0000 UTC notAfter=2024-04-19 13:02:25 +0000 UTC"
Apr 20 13:02:25 <hidden> k3s[84782]: time="2023-04-20T13:02:25+02:00" level=info msg="Running cloud-controller-manager --allocate-node-cidrs=true --authentication-kubeconfig=/var/lib/rancher/k3s/server/cred/cloud-controller.kubeconfig --authorization-kubeconfig=/var/lib/rancher/k3s/server/cred/cloud-controller.kubeconfig --bind-address=<IP_ADDRESS> --cloud-config=/var/lib/rancher/k3s/server/etc/cloud-config.yaml --cloud-provider=k3s --cluster-cidr=<IP_ADDRESS>/16 --configure-cloud-routes=false --controllers=*,-route --kubeconfig=/var/lib/rancher/k3s/server/cred/cloud-controller.kubeconfig --leader-elect=false --leader-elect-resource-name=k3s-cloud-controller-manager --node-status-update-frequency=1m0s --profiling=false"
Apr 20 13:02:31 <hidden> k3s[84782]: time="2023-04-20T13:02:31+02:00" level=info msg="Waiting for cloud-controller-manager privileges to become available"
Apr 20 13:02:35 <hidden> k3s[84782]: unable to load configmap based request-header-client-ca-file: configmaps "extension-apiserver-authentication" is forbidden: User "k3s-cloud-controller-manager" cannot get resource "configmaps" in API group "" in the namespace "kube-system"
Apr 20 13:02:35 <hidden> k3s[84782]: Error: unable to load configmap based request-header-client-ca-file: configmaps "extension-apiserver-authentication" is forbidden: User "k3s-cloud-controller-manager" cannot get resource "configmaps" in API group "" in the namespace "kube-system"
Apr 20 13:02:35 <hidden> k3s[84782]:   cloud-controller-manager [flags]
Apr 20 13:02:35 <hidden> k3s[84782]:       --leader-elect-resource-name string        The name of resource object that is used for locking during leader election. (default "cloud-controller-manager")
Apr 20 13:02:35 <hidden> k3s[84782]:   -h, --help                             help for cloud-controller-manager
Apr 20 13:02:35 <hidden> k3s[84782]: time="2023-04-20T13:02:35+02:00" level=error msg="cloud-controller-manager exited: unable to load configmap based request-header-client-ca-file: configmaps \"extension-apiserver-authentication\" is forbidden: User \"k3s-cloud-controller-manager\" cannot get resource \"configmaps\" in API group \"\" in the namespace \"kube-system\""
Apr 20 13:02:36 <hidden> k3s[84782]: time="2023-04-20T13:02:36+02:00" level=info msg="Slow SQL (started: 2023-04-20 13:02:33.991100151 +000 CEST m=+15.104670164) (total time: 2.104021479s): INSERT INTO kine(name, created, deleted, create_revision, prev_revision, lease, value, old_value) values(?, ?, ?, ?, ?, ?, ?, ?) : [[/registry/clusterrolebindings/k3s-cloud-controller-manager-auth-delegator 1 0 0 0 0 [<hidden>]
Apr 20 13:02:36 <hidden> k3s[84782]: I0420 13:02:36.095642   84782 trace.go:205] Trace[1885216390]: "Create etcd3" audit-id:239071e7-361e-4035-83d9-c50e59baa653,key:/clusterrolebindings/k3s-cloud-controller-manager-auth-delegator,type:*rbac.ClusterRoleBinding (20-Apr-2023 13:02:33.990) (total time: 2105ms):

maybe it should ensure that role and role binding are created before trying to get the configmap.

It does, actually.
https://github.com/k3s-io/k3s/blob/944f811dc5341349372ccf57706f2aa65e13ce04/pkg/daemons/control/server.go#L380-L392

root@<hidden>:/home/<hidden># grep MemTotal /proc/meminfo
MemTotal:       130524204 kB

root@<hidden>:/home/<hidden># grep 'cpu cores' /proc/cpuinfo | uniq
cpu cores	: 16

root@<hidden>:/home/<hidden># lshw -c storage -c disk
  *-storage:0
       description: Non-Volatile memory controller
       product: Amazon.com, Inc.
       vendor: Amazon.com, Inc.
       physical id: 4
       bus info: pci@0000:00:04.0
       version: 00
       width: 32 bits
       clock: 33MHz
       capabilities: storage pciexpress msix nvm_express bus_master cap_list
       configuration: driver=nvme latency=0
       resources: irq:11 memory:fd010000-fd013fff
     *-nvme0
          description: NVMe device
          product: Amazon Elastic Block Store
          physical id: 0
          logical name: /dev/nvme0
          version: 1.0
          serial: vol0b802cac479ac41bb
          configuration: nqn=nqn.2014.08.org.nvmexpress:1d0f1d0fvol0b802cac479ac41bbAmazon Elastic Block Store state=live
        *-namespace
             description: NVMe namespace
             physical id: 1
             logical name: /dev/nvme0n1
             size: 550GiB (590GB)
             capabilities: partitioned partitioned:dos
             configuration: logicalsectorsize=512 sectorsize=512 signature=0f25cffe
  *-storage:1
       description: Non-Volatile memory controller
       product: NVMe SSD Controller
       vendor: Amazon.com, Inc.
       physical id: 1f
       bus info: pci@0000:00:1f.0
       version: 00
       width: 32 bits
       clock: 33MHz
       capabilities: storage pciexpress msix nvm_express bus_master cap_list
       configuration: driver=nvme latency=0
       resources: irq:0 memory:fd018000-fd01bfff memory:fe800000-fe801fff
     *-nvme1
          description: NVMe device
          product: Amazon EC2 NVMe Instance Storage
          physical id: 0
          logical name: /dev/nvme1
          version: 0
          serial: AWS2236BE4FA42C13BFE
          configuration: nqn=nqn.2008-08.com.amazon.aws:ec2-nvme-instance:AWS2236BE4FA42C13BFE state=live
        *-namespace
             description: NVMe namespace
             physical id: 1
             logical name: /dev/nvme1n1
             size: 838GiB (900GB)
             configuration: logicalsectorsize=512 sectorsize=512

Currently, I m not able to attach full logs from journalctl, as the error is once appearing and once not.
@brandond

@brandond I do not have access there currently, trying to get one.
I found another reproduction of the issue, and I have gathered logs.
k3s.log

yeah something is definitely wrong with your disk. Just creating the very first row in the database during initial setup caused a slow SQL error.
Apr 24 11:54:04 <hidden> k3s[84855]: time="2023-04-24T11:54:04+02:00" level=info msg="Configuring sqlite3 database connection pooling: maxIdleConns=2, maxOpenConns=0, connMaxLifetime=0s"
Apr 24 11:54:04 <hidden> k3s[84855]: time="2023-04-24T11:54:04+02:00" level=info msg="Configuring database table schema and indexes, this may take a moment..."
Apr 24 11:54:07 <hidden> k3s[84855]: time="2023-04-24T11:54:07+02:00" level=info msg="Database tables and indexes are up to date"
Apr 24 11:54:09 <hidden> k3s[84855]: time="2023-04-24T11:54:09+02:00" level=info msg="Slow SQL (started: 2023-04-24 11:54:07.941613409 +0200 CEST m=+7.038912399) (total time: 1.536657724s): INSERT INTO kine(name, created, deleted, create_revision, prev_revision, lease, value, old_value) values(?, ?, ?, ?, ?, ?, ?, ?) : [[compact_rev_key 1 0 0 0 0 [] []]]"
Apr 24 11:54:09 <hidden> k3s[84855]: time="2023-04-24T11:54:09+02:00" level=info msg="Kine available at unix://kine.sock"

1.5 seconds to insert the very first row in the databse, before ANYTHING else is running, is bonkers slow.


Ensure AnimationPlayer evaluate animations when autoplay is enabled and node becomes ready

Fixes #83326.
The issue was introduced in the refactoring done in 1b95827d3ef244de322b0c16deb49fefe48ed1a1.

Thanks! And congrats for your first merged Godot contribution :tada:


[Bug]: RuntimeError on A800 using vllm0.6.1.post2

Your current environment

The output of `python collect_env.py`
Collecting environment information...
PyTorch version: 2.4.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.30.0
Libc version: glibc-2.35

Python version: 3.10.14 (main, May  6 2024, 19:42:50) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-4.18.0-372.9.1.el8.x86_64-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 12.1.105
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA A800 80GB PCIe
GPU 1: NVIDIA A800 80GB PCIe
GPU 2: NVIDIA A800 80GB PCIe
GPU 3: NVIDIA A800 80GB PCIe
GPU 4: NVIDIA A800 80GB PCIe
GPU 5: NVIDIA A800 80GB PCIe
GPU 6: NVIDIA A800 80GB PCIe
GPU 7: NVIDIA A800 80GB PCIe

Nvidia driver version: 535.54.03
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Address sizes:                   43 bits physical, 48 bits virtual
Byte Order:                      Little Endian
CPU(s):                          128
On-line CPU(s) list:             0-127
Vendor ID:                       HygonGenuine
BIOS Vendor ID:                  Chengdu Hygon
Model name:                      Hygon C86 7385 32-core Processor
BIOS Model name:                 Hygon C86 7385 32-core Processor
CPU family:                      24
Model:                           2
Thread(s) per core:              2
Core(s) per socket:              32
Socket(s):                       2
Stepping:                        2
BogoMIPS:                        3999.97
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid amd_dcm aperfmperf pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb hw_pstate ssbd ibpb vmmcall fsgsbase bmi1 avx2 smep bmi2 rdseed adx smap clflushopt sha_ni xsaveopt xsavec xgetbv1 xsaves clzero irperf xsaveerptr arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif overflow_recov succor smca sme sev sev_es
Virtualization:                  AMD-V
L1d cache:                       2 MiB (64 instances)
L1i cache:                       4 MiB (64 instances)
L2 cache:                        32 MiB (64 instances)
L3 cache:                        128 MiB (16 instances)
NUMA node(s):                    8
NUMA node0 CPU(s):               0-7,64-71
NUMA node1 CPU(s):               8-15,72-79
NUMA node2 CPU(s):               16-23,80-87
NUMA node3 CPU(s):               24-31,88-95
NUMA node4 CPU(s):               32-39,96-103
NUMA node5 CPU(s):               40-47,104-111
NUMA node6 CPU(s):               48-55,112-119
NUMA node7 CPU(s):               56-63,120-127
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Retpolines, IBPB conditional, STIBP disabled, RSB filling
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected

Versions of relevant libraries:
[pip3] numpy==1.26.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.1.105
[pip3] nvidia-cuda-nvrtc-cu12==12.1.105
[pip3] nvidia-cuda-runtime-cu12==12.1.105
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.555.43
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] nvidia-nvjitlink-cu12==12.5.82
[pip3] nvidia-nvtx-cu12==12.1.105
[pip3] pyzmq==26.2.0
[pip3] sentence-transformers==3.0.1
[pip3] torch==2.4.0
[pip3] torchvision==0.19.0
[pip3] transformers==4.44.2
[pip3] triton==3.0.0
[conda] numpy                     1.26.4                   pypi_0    pypi
[conda] nvidia-cublas-cu12        <IP_ADDRESS>                 pypi_0    pypi
[conda] nvidia-cuda-cupti-cu12    12.1.105                 pypi_0    pypi
[conda] nvidia-cuda-nvrtc-cu12    12.1.105                 pypi_0    pypi
[conda] nvidia-cuda-runtime-cu12  12.1.105                 pypi_0    pypi
[conda] nvidia-cudnn-cu12         <IP_ADDRESS>                 pypi_0    pypi
[conda] nvidia-cufft-cu12         <IP_ADDRESS>                pypi_0    pypi
[conda] nvidia-curand-cu12        <IP_ADDRESS>               pypi_0    pypi
[conda] nvidia-cusolver-cu12      <IP_ADDRESS>               pypi_0    pypi
[conda] nvidia-cusparse-cu12      <IP_ADDRESS>               pypi_0    pypi
[conda] nvidia-ml-py              12.555.43                pypi_0    pypi
[conda] nvidia-nccl-cu12          2.20.5                   pypi_0    pypi
[conda] nvidia-nvjitlink-cu12     12.5.82                  pypi_0    pypi
[conda] nvidia-nvtx-cu12          12.1.105                 pypi_0    pypi
[conda] pyzmq                     26.2.0                   pypi_0    pypi
[conda] sentence-transformers     3.0.1                    pypi_0    pypi
[conda] torch                     2.4.0                    pypi_0    pypi
[conda] torchvision               0.19.0                   pypi_0    pypi
[conda] transformers              4.44.2                   pypi_0    pypi
[conda] triton                    3.0.0                    pypi_0    pypi
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.6.1.post2@9ba0817ff1eb514f51cc6de9cb8e16c98d6ee44f
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    GPU1    GPU2    GPU3    GPU4    GPU5    GPU6    GPU7    NIC0    NIC1    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      PIX     PXB     PXB     SYS     SYS     SYS     SYS     SYS     SYS     24-31,88-95     3               N/A
GPU1    PIX      X      PXB     PXB     SYS     SYS     SYS     SYS     SYS     SYS     24-31,88-95     3               N/A
GPU2    PXB     PXB      X      PXB     SYS     SYS     SYS     SYS     SYS     SYS     24-31,88-95     3               N/A
GPU3    PXB     PXB     PXB      X      SYS     SYS     SYS     SYS     SYS     SYS     24-31,88-95     3               N/A
GPU4    SYS     SYS     SYS     SYS      X      PIX     PXB     PXB     SYS     SYS     56-63,120-127   7               N/A
GPU5    SYS     SYS     SYS     SYS     PIX      X      PXB     PXB     SYS     SYS     56-63,120-127   7               N/A
GPU6    SYS     SYS     SYS     SYS     PXB     PXB      X      PXB     SYS     SYS     56-63,120-127   7               N/A
GPU7    SYS     SYS     SYS     SYS     PXB     PXB     PXB      X      SYS     SYS     56-63,120-127   7               N/A
NIC0    SYS     SYS     SYS     SYS     SYS     SYS     SYS     SYS      X      PIX
NIC1    SYS     SYS     SYS     SYS     SYS     SYS     SYS     SYS     PIX      X

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

NIC Legend:

  NIC0: mlx5_0
  NIC1: mlx5_1

Model Input Dumps
No response
🐛 Describe the bug
Example of command:
python benchmark_throughput.py --trust-remote-code  --enforce-eager --dtype float16  --num-prompts 1 --input-len 2 --output-len 128 --model Llama-2-7b-chat-hf
Output:
Namespace(backend='vllm', dataset=None, input_len=2, output_len=128, model='Llama-2-7b-chat-hf', tokenizer='Llama-2-7b-chat-hf/', quantization=None, tensor_parallel_size=1, n=1, use_beam_search=False, num_iters_warmup=1, num_prompts=1, seed=0, hf_max_batch_size=None, trust_remote_code=True, max_model_len=None, dtype='float16', gpu_memory_utilization=0.9, enforce_eager=True, kv_cache_dtype='auto', quantization_param_path=None, device='auto', num_scheduler_steps=1, use_v2_block_manager=False, enable_prefix_caching=False, enable_chunked_prefill=False, max_num_batched_tokens=None, download_dir=None, output_json=None, distributed_executor_backend=None, load_format='auto', disable_async_output_proc=False, async_engine=False, disable_frontend_multiprocessing=False)
Traceback (most recent call last):
File "/data/test/benchmark_throughput.py", line 617, in
main(args)
File "/data/test/benchmark_throughput.py", line 382, in main
elapsed_time = run_vllm(*run_args)
File "/data/test/benchmark_throughput.py", line 98, in run_vllm
llm = LLM(
File "/opt/miniconda3/envs/vllm/lib/python3.10/site-packages/vllm/entrypoints/llm.py", line 178, in init
self.llm_engine = LLMEngine.from_engine_args(
File "/opt/miniconda3/envs/vllm/lib/python3.10/site-packages/vllm/engine/llm_engine.py", line 547, in from_engine_args
engine_config = engine_args.create_engine_config()
File "/opt/miniconda3/envs/vllm/lib/python3.10/site-packages/vllm/engine/arg_utils.py", line 843, in create_engine_config
device_config = DeviceConfig(device=self.device)
File "/opt/miniconda3/envs/vllm/lib/python3.10/site-packages/vllm/config.py", line 1094, in init
self.device = torch.device(self.device_type)
TypeError: device() received an invalid combination of arguments - got (bool), but expected one of:

(torch.device device)
didn't match because some of the arguments have invalid types: (bool)
(str type, int index = -1)

Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

can you debug to see why it errors? it looks irrelevant to the gpu, but may be some bugs in the benchmark script.

llm = LLM(
model=model,
tokenizer=tokenizer,
quantization=quantization,
tensor_parallel_size=tensor_parallel_size,
seed=seed,
trust_remote_code=trust_remote_code,
dtype=dtype,
max_model_len=max_model_len,
gpu_memory_utilization=gpu_memory_utilization,
enforce_eager=enforce_eager,
kv_cache_dtype=kv_cache_dtype,
quantization_param_path=quantization_param_path,
device=device,
enable_prefix_caching=enable_prefix_caching,
download_dir=download_dir,
enable_chunked_prefill=enable_chunked_prefill,
max_num_batched_tokens=max_num_batched_tokens,
distributed_executor_backend=distributed_executor_backend,
load_format=load_format,
num_scheduler_steps=num_scheduler_steps,
use_v2_block_manager=use_v2_block_manager,
disable_async_output_proc=disable_async_output_proc,
)
During the initialization of LLM, an error occurred. It seems that the initialized values have not been obtained and are all None, such as device and dtype.


Look at how to speed up the builds

There's a potential solution https://github.com/ros2/rosidl/pull/769 for speeding up the build.
That might be a good solution or some other one such that the message generation phase goes faster and isn't noticable when building.

Lets start with making a good diagram of how things work, and how they are extended to generate protobuf generation.
Then make a parallel diagram of how the new system works and how it would be extended for generating protobuf.
Keeping in mind that there might be new generators of interest such as a rust generator in the future.

Hook registration on CodeGenerator project build
sequenceDiagram
    participant CodeGenerator_cmake
    participant ament_package
    participant ament_register_extension
    CodeGenerator_cmake->>ament_package: CONFIG_EXTRAS
    ament_package->>ament_register_extension: rosidl_generate_idl_interfaces, cmake_code_hook
    Note right of ament_package: Execute a cmake.in template with variables and a cmake code hook.

CURRENT
Hook execution and code generation
sequenceDiagram
    participant project_with_rosidl_files
    participant rosidl_generate_interfaces
    participant rosidl_generate_idl_interfaces_HOOK
    participant generate_arguments_file
    participant rosidl_generator
    project_with_rosidl_files->>rosidl_generate_interfaces: path_to_idl_files
    loop ForEach Hook
        rosidl_generate_interfaces->>rosidl_generate_idl_interfaces_HOOK: path_to_idl_files, parameters
        rosidl_generate_idl_interfaces_HOOK->>generate_arguments_file: template_parameters
        activate generate_arguments_file
        generate_arguments_file->>rosidl_generate_idl_interfaces_HOOK: arguments_file
        deactivate generate_arguments_file
        rosidl_generate_idl_interfaces_HOOK->>rosidl_generator: arguments_file
    end

This will be great to add to documentation either here: https://github.com/ros2/rosidl?tab=readme-ov-file
Or where it links to at https://docs.ros.org/en/rolling/Concepts/Advanced/About-Internal-Interfaces.html

Drafted a PR for ros2_documentation for old and new approach. Initial testing of new approach going well.
Do some initial benchmarking to validate speedup and that it works. Only some of the implementation, only rosidl_introspection_c and introspection_cpp are ported at the moment. The others rely on the backwards compatibility. We can show speedup for these and open tickets to accelerate the other groups.
We'll work to land the new method and then port the protobuf version to the new format. Showing speedup here will be a good case for others to use the new generator.

Documentation PRs created at my local fork for review:

https://github.com/gonzodepedro/ros2_documentation/pull/1
https://github.com/gonzodepedro/ros2_documentation/pull/2

Issues Encountered

Not sure if mermaid is support on .rsl files
Commited the mermaid sources and also a locally generated svg file.

For Mermaid support we should try to use the sphinx plugin: https://github.com/mgaitan/sphinxcontrib-mermaid

sequenceDiagram
    participant project_with_rosidl_files
    participant rosidl_generate_interfaces
    participant rosidl_create_type_descriptions_extensions_HOOK
    participant rosidl_generate_idl_interfaces_HOOK
    participant rosidl_write_generator_arguments_extensions_HOOK
    participant generate_arguments_file
    participant rosidl_generator
    project_with_rosidl_files->>rosidl_generate_interfaces: path_to_idl_files
    loop ForEach rosidl_create_type_descriptions_extensions
        rosidl_generate_interfaces->>rosidl_create_type_descriptions_extensions_HOOK: path_to_idl_files, parameters
    end
    loop ForEach rosidl_write_generator_arguments_extensions
        rosidl_generate_interfaces->>rosidl_write_generator_arguments_extensions_HOOK: path_to_idl_files, parameters
        activate generate_arguments_file
        rosidl_write_generator_arguments_extensions_HOOK->>generate_arguments_file: path_to_idl_files, parameters
        generate_arguments_file->>rosidl_write_generator_arguments_extensions_HOOK: arguments_file
        deactivate generate_arguments_file
        rosidl_write_generator_arguments_extensions_HOOK->>rosidl_generate_interfaces: arguments_file
    end
    rosidl_generate_interfaces>>rosidl_generator: arguments_files
    rosidl_generate_interfaces->>rosidl_generate_idl_interfaces_HOOK: generated_files

I made changes to both PRs to reflect comments and work done by Tully to integrate mermaid into sphinx.

https://github.com/gonzodepedro/ros2_documentation/pull/1
https://github.com/gonzodepedro/ros2_documentation/pull/2

Currently, the sequence of events to generate idl interfaces looks something like this:
    sequenceDiagram
    participant project_with_rosidl_files
    participant rosidl_generate_interfaces
    participant rosidl_generate_idl_interfaces_HOOK
    participant generate_arguments_file
    participant rosidl_generator
    project_with_rosidl_files->>rosidl_generate_interfaces: path_to_idl_files
    loop ForEach Hook
        rosidl_generate_interfaces->>rosidl_generate_idl_interfaces_HOOK: path_to_idl_files, parameters
        rosidl_generate_idl_interfaces_HOOK->>generate_arguments_file: template_parameters
        activate generate_arguments_file
        generate_arguments_file->>rosidl_generate_idl_interfaces_HOOK: arguments_file
        deactivate generate_arguments_file
        rosidl_generate_idl_interfaces_HOOK->>rosidl_generator: arguments_file
    end

If we take a deeper look into the last activity (rosidl_generator), we can decompose it into another series of interactions:
    sequenceDiagram
        participant rosidl_generator
        participant python_interpreter
        participant emInterpreter
        rosidl_generator->>python_interpreter: arguments_file
        Note right of rosidl_generator: Kickstarting Python interpreter is an expensive task
        python_interpreter->>emInterpreter: template, arguments

As you can see in the diagram above, calling rosidl_generator which generates code from templates and parameters, requires to kick-start a python interpreter, which is an expensive operation in terms of time and resources. If you take a look at the first diagram, you will see that this happens inside a loop. Which means that the expensive operation will be executed once per interfaces per package.
This PR, allows to defer the call to the python interpreter for a later time, where all the interfaces and packages have already been executed. To do that, each interface package will, instead of calling the rosidl_generator directly, they will generate an arguments file and return it. When all interface packages are called and all arguments files are generated, then the code will call the rosidl_generator once, passing all the arguments_files.
As you will see in the diagram below, the original loop is now separated into three loops, the first one gets everything set, the second one generates the arguments files and the last one does all the post processing after the source code is generated. But the interpreter is instantiated only once, outside of any loop.
    sequenceDiagram
    participant project_with_rosidl_files
    participant rosidl_generate_interfaces
    participant rosidl_create_type_descriptions_extensions_HOOK
    participant rosidl_generate_idl_interfaces_HOOK
    participant rosidl_write_generator_arguments_extensions_HOOK
    participant generate_arguments_file
    participant rosidl_generator
    project_with_rosidl_files->>rosidl_generate_interfaces: path_to_idl_files
    loop ForEach rosidl_create_type_descriptions_extensions Hook
        rosidl_generate_interfaces->>rosidl_create_type_descriptions_extensions_HOOK: path_to_idl_files, parameters
    end
    loop ForEach rosidl_write_generator_arguments_extensions Hook
        rosidl_generate_interfaces->>rosidl_write_generator_arguments_extensions_HOOK: path_to_idl_files, parameters
        activate generate_arguments_file
        rosidl_write_generator_arguments_extensions_HOOK->>generate_arguments_file: path_to_idl_files, parameters
        generate_arguments_file->>rosidl_write_generator_arguments_extensions_HOOK: arguments_file
        deactivate generate_arguments_file
        rosidl_write_generator_arguments_extensions_HOOK->>rosidl_generate_interfaces: arguments_file
    end
    rosidl_generate_interfaces->>rosidl_generator: [arguments_files]
    loop ForEach rosidl_generate_idl_interfaces Hook
        rosidl_generate_interfaces->>rosidl_generate_idl_interfaces_HOOK: generated_files
    end

@tfoote PTAL at comment above, as a candidate for the description of the PR

Top level items would be great to have verbs with descriptions

aka Building a Package with message definitions", "invoking rosidl_generate_interfaces"

It would be nice to see what the outputs are "inputs => output file type/format"

Include the last stage that gets blown out in the feature on the diagram (add one more row/cycle showing the invocation of the generator in the top diagram)

Reworking this to be same language as the upstream PR to be contributed there.
Goal for next week extended documentation and benchmark showing upstream benefits to promote that PR out of draft status.

It would be good to benchmark the processes. I found some resources such as:
https://stackoverflow.com/a/67012173/604099

For reference other generators
rosidl_generator_java
https://github.com/ros2-java/ros2_java/blob/main/rosidl_generator_java/rosidl_generator_java/__init__.py
rclrs
https://github.com/ros2-rust/ros2_rust/blob/main/rosidl_generator_rs/rosidl_generator_rs/__init__.py
rosjava did catchup generation https://github.com/rosjava/rosjava_bootstrap/tree/kinetic/message_generation/src/main/java/org/ros/message

@gonzodepedro can you check out chris's concerns in https://github.com/ros2/rosidl/pull/769#issuecomment-1738050659

Top priorities rebase the rosidl change to be able to retest it. Work on documentation before landing.

Mermaid documentation https://mermaid.js.org/syntax/sequenceDiagram.html

PR for rosidl "rebase" https://github.com/DanMesh/rosidl/pull/1

Also validating it with drake-ros to make sure it's working with non cmake projects

The drake_ros work isn't using the command line interface, but introspecting/loading the
Instead make a python test in the typesupport libraries. Do this for protobuf_tyepsupport
And in the type adapter libraries and then our adapters
If the pattern is easy enough to replicate we can PR it for other typesupports in the future.


componentWillReceiveProps changes the State but shouldComponentUpdate does not consider this change

Hello,
I use componentWillReceiveProps to change the State before rendering and also shouldComponentUpdate to determine whether to render (based on the State that componentWillReceiveProps does change):
componentWillReceiveProps(nextProps) {
var newValue = //do stuff here to calculate newValue ;
this.setState({stateVar: newValue});
}

shouldComponentUpdate(nextProps, nextState) {
        var shouldUpdate = nextState.stateVar !== this.state.stateVar;
        return !!shouldUpdate;
    }

Based on my tests, the nextState.stateVar is still the old one, despite the componentWillReceiveProps did change it.
May I assume that this is not possible or is this a bug?
Thank you.

Is this radium specific? If not, please post your question to stackoverflow.

You are right, I saw a similar post here and thought I could ask, I have just created a stackoverflow post.

Going to close this out. Best of luck with your issue!


⚠ Bofrost.es has degraded performance

In e5e570f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1729 ms

Resolved: Bofrost.es performance has improved in ee40a2a after 6 minutes.


Bug - Application Timeline - Duplicate text

Description
There is duplicate text in EN and FR on the Start Your Application portion of the Job Poster Application.
Steps to Reproduce

Go to Job Poster Application
See duplicate text

Images

Fixed in #5356


Only cancel scan if job is cancelled and ignore cancel output

Description
The post job was trying to cancel the scan even if the pipeline was not cancelled. This PR fixes this behavior by only cancelling the scan if the pipeline was canceled. Also, the pipeline will not fail if the post job cancel fails.
References
N/A
Testing
N/A
Checklist

[x] I have added documentation for new/changed functionality in this PR (if applicable).
[x] I have updated the CLI help for new/changed functionality in this PR (if applicable).
[x] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used

Checkmarx AST – Scan Summary & Details – 55afcbc6-e33e-4d31-89d6-bdece6082297
No New Or Fixed Issues Found


Allow Authentication to be configured in the helm deployment

It would be nice if the config items for SSO could be provided in the helm values (and/or via secrets). This would allow SEQ to be rollout out fully configured. E.g:

Authentication provider
Directory (tenant) ID
Application (client) ID
Client secret (as a secret)

https://docs.datalust.co/docs/azure-active-directory

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Thanks for the suggestion! This is on our radar but needs some supporting infrastructure in Seq, so isn't one we can turn around quickly right now.

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


feat(i18n) : enabling last part of Italian curriculum

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

based on our discussion in chat, let's re-open this once the related superBlocks are translated.


Error compiling for board Arduino 101

When is run test code in arduino 101
Arduino: 1.8.2 (Windows 10), Board: "Arduino/Genuino 101"

In file included from C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral\src/utility/lib_aci.h:34:0,

                 from C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral\src/nRF8001.h:9,

                 from C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral\src/BLEPeripheral.h:26,

                 from C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral\examples\test\test.ino:12:

C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral\src/utility/hal_platform.h:75:6: error: #error "Unsupported platform"

     #error "Unsupported platform"

      ^

Multiple libraries were found for "BLEPeripheral.h"
 Used: C:\Users\Faris\OneDrive\Documents\Arduino\libraries\BLEPeripheral
 Not used: C:\Users\Faris\AppData\Local\Arduino15\packages\Intel\hardware\arc32\2.0.2\libraries\CurieBLE
exit status 1
Error compiling for board Arduino/Genuino 101.

Also seeing this issue.

any solutions...?

Please see the read me for compatible hardware: https://github.com/sandeepmistry/arduino-BLEPeripheral#compatible-hardware
For the 101, you'll have to use the CurieBLE library.

am using the CurieBLE library provided with the intel 101 board definition , but still getting the error .

@salmanfarisvp
I had included CurieBLE.h via the menus. When I did that there was a whole stack of includes. I'm guessing you may have done the same?
When I removed all the extras I started seeing no errors in compile time. I've yet to complete my mission and make my garage door controlled from my phone, but I thought I should mention my experience.


Distribution bundle bwc tests

Description
There is a framework for BWC tests on distribution level as well which will set up the test cluster with the bundle distribution including all plugins engineering effectiveness team builds.
It will be beneficial during the release cycle since we will be testing the exact distribution that we would ship to the customers.
This PR consist configuration to run bwc tests on distribution bundle. The PR is on Branch 2.x
The bwc distribution will fail on neural search for opensearch versions <2.11 because of the bug from the ml-commons.
https://github.com/opensearch-project/ml-commons/issues/1838
rg.opensearch.neuralsearch.bwc.SemanticSearchIT > testSemanticSearch_E2EFlow FAILED
    org.opensearch.client.ResponseException: method [POST], host [http://<IP_ADDRESS>:39449], URI [/neuralsearch-bwc-testsemanticsearch_e2eflow/_search?size=1], status line [HTTP/1.1 500 Internal Server Error]
    {"error":{"root_cause":[{"type":"illegal_state_exception","reason":"unexpected byte [0x48]"}],"type":"illegal_state_exception","reason":"unexpected byte [0x48]"},"status":500}
        at __randomizedtesting.SeedInfo.seed([9C3D08353B6D4C7A:2A2FA7AD9C7ED3BD]:0)
        at app//org.opensearch.client.RestClient.convertResponse(RestClient.java:376)
        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:346)
        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:321)
        at app//org.opensearch.neuralsearch.BaseNeuralSearchIT.search(BaseNeuralSearchIT.java:417)
        at app//org.opensearch.neuralsearch.BaseNeuralSearchIT.search(BaseNeuralSearchIT.java:378)
        at app//org.opensearch.neuralsearch.BaseNeuralSearchIT.search(BaseNeuralSearchIT.java:359)
        at app//org.opensearch.neuralsearch.bwc.SemanticSearchIT.validateTestIndexOnUpgrade(SemanticSearchIT.java:79)
        at app//org.opensearch.neuralsearch.bwc.SemanticSearchIT.testSemanticSearch_E2EFlow(SemanticSearchIT.java:46)

Issues Resolved
564
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

./gradlew bwcTestSuite -Dtests.security.manager=false -PcustomDistributionDownloadType=bundle

> Configure project :
Download https://raw.githubusercontent.com/opensearch-project/security/main/bwc-test/src/test/resources/security/esnode.pem
Download https://raw.githubusercontent.com/opensearch-project/security/main/bwc-test/src/test/resources/security/esnode-key.pem
Download https://raw.githubusercontent.com/opensearch-project/security/main/bwc-test/src/test/resources/security/root-ca.pem
Download https://raw.githubusercontent.com/opensearch-project/security/main/bwc-test/src/test/resources/security/kirk.pem
Download https://raw.githubusercontent.com/opensearch-project/security/main/bwc-test/src/test/resources/security/kirk-key.pem
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 8.4
  OS Info               : Linux 5.10.205-172.807.amzn2int.x86_64 (amd64)
  JDK Version           : 21 (Amazon Corretto JDK)
  JAVA_HOME             : /local/home/varunudr/.sdkman/candidates/java/21.0.1-amzn
  Random Testing Seed   : 135916AFF3FBB0A6
  In FIPS 140 mode      : false
=======================================

> Task :generatePomFileForNebulaPublication
Maven publication 'nebula' pom metadata warnings (silence with 'suppressPomMetadataWarningsFor(variant)'):
  - Variant testFixturesApiElements:
      - Declares capability org.opensearch:neural-search-test-fixtures:<IP_ADDRESS>-SNAPSHOT which cannot be mapped to Maven
  - Variant testFixturesRuntimeElements:
      - Declares capability org.opensearch:neural-search-test-fixtures:<IP_ADDRESS>-SNAPSHOT which cannot be mapped to Maven
These issues indicate information that is lost in the published 'pom' metadata file, which may be an issue if the published library is consumed by an old Gradle version or Apache Maven.
The 'module' metadata file, which is used by Gradle 6+ is not affected.

> Task :qa:restart-upgrade:testAgainstOldCluster
Feb 01, 2024 1:30:57 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

> Task :qa:restart-upgrade:testAgainstNewCluster
Feb 01, 2024 1:32:06 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

> Task :qa:rolling-upgrade:testAgainstOldCluster
Feb 01, 2024 1:33:02 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

> Task :qa:rolling-upgrade:testAgainstOneThirdUpgradedCluster
Feb 01, 2024 1:33:55 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

> Task :qa:rolling-upgrade:testAgainstTwoThirdsUpgradedCluster
Feb 01, 2024 1:34:24 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

> Task :qa:rolling-upgrade:testRollingUpgrade
Feb 01, 2024 1:34:53 PM sun.util.locale.provider.LocaleProviderAdapter <clinit>
WARNING: COMPAT locale provider will be removed in a future release

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

For more on this, please refer to https://docs.gradle.org/8.4/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

BUILD SUCCESSFUL in 5m 2s
38 actionable tasks: 13 executed, 25 up-to-date


Incorrect syntax for OpenID Connect trust policy

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-amazon-web-services
What part(s) of the article would you like to see updated?
I want to update the code snippet in section Configuring the role and trust policy to match the syntax provided by AWS in Using multiple keys and values.
Basically add ForAllValues: in front of StringEquals.
Additional information
No response

Glitch in the matrix, Trust Policy editor is no longer complaining about it.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 29eb576.


Proof and provenance docs

cUrl docs for provenance and ledger API
sdk docs for provenance and ledger API
documents about skip-list, merkle-tree, merkle-patricia-trie and overview document about proofs
Signed-off-by: Gennady Laventman<EMAIL_ADDRESS>

@gennadylaventman Please rebase


CollectionViewCell dynamic heights

Creating dynamic height with a fixed width just sure ain't easy in objective C. Here are all the UICollectionViewCells that need to be dynamic to support multil-line text:

[ ] Campaign Detail: Presignup Campaign Cell (LDTCampaignListCampaignCell)
[x] Campaign Detail: Doing/Completed Campaign Cell (LDTCampaignDetailCampaignCell) - #480
[ ] Campaign Detail: Self reportback, Reportback gallery (LDTCampaignDetailReportbackItemCell)

We don't run into this trouble as much on the Profile because it's implemented using a UITableView.

Removed item

Campaign List: Expanded Campaign Cell (LDTCampaignListCampaignCell)

from this issue description, as we'll soon be removing the Main Feed screen.

Not working for multi-line captions on the Campaign Detail.


CI Generator

This pull request includes configuration for running lint, code style checks, and tests within your CI environment. Feel free to commit any additional changes to the shift-71818 branch.
Before merging, you need to:

Review all pull request comments for additional changes
Ensure your CI build is running successfully

:x: The CI Generator does not support versions of PHP prior to 7.3. Since Shift only detected older PHP versions within your composer.json, your jobs have been configured to run PHP 7.3.

:warning: Shift was unable to detect any PHP extensions within your composer.json. Your jobs have been configured to use the PHP extensions required by Laravel.

:information_source: Shift configured your jobs to use MySQL 8.0. You may adjust the configuration if you are using a different version of MySQL or a different database system.

:information_source: Shift detected you have a Laravel application, but do not have an existing code style.

:information_source: Shift configured Composer to use the COMPOSER_AUTH environment variable. If your project contains private dependencies, you should create a secret for your repository named COMPOSER_AUTH with the JSON configuration for Composer.

:information_source: By default, Shift configured your jobs to use an official PHP image. However, if you are familiar with Docker, you may improve performance by using your image with all your dependencies prebuilt.

:alembic: This Shift is still being refined. Please report any issues or suggestions to<EMAIL_ADDRESS>Your feedback is what helps improve the experience for everyone.

:information_source: Shift detected your application only has the default tests. You may quickly start adding tests by using the Test Generator to automatically create the model factories, test classes, and sample tests cases for your Laravel application.


Implement logout command

What should be done?
https://www.mongodb.com/docs/manual/reference/command/logout/
We should implement that command despite it being deprecated (it's needed for tests).
It should be implemented in the common handler, with other handlers using a common implementation.
It should call ConnInfo.SetAuth("", "") to remove authentication information.
Where?

https://github.com/FerretDB/FerretDB/blob/main/internal/handlers/handlers.go
https://github.com/FerretDB/FerretDB/tree/main/internal/handlers/common
https://github.com/FerretDB/FerretDB/blob/main/internal/handlers/common/msg_listcommands.go
https://github.com/FerretDB/FerretDB/tree/main/integration

Definition of Done

all handlers updated;
integration/compatibility tests added/updated;
spot refactorings done;
user documentation updated.

@AlekSi Can I work on this one??

Sure!