Provide convenience methods for setting default values for inputs/outputs

Context
When giving the provider API a try, in the plugin to report incubating APIs, I was surprised to find that it wasn't obvious how to set a default value for an output directory, or an input file. I liked the fact that I could set all my properties as "final not null", but doing so proved to be a challenge if you also want to set a default value.
So, I found a pattern (in Kotlin) that was kind of doing what I need, and wondered if we couldn't do better:
    @OutputFile
    val htmlReportFile = newOutputFile().also {
        it.set(project.layout.buildDirectory.file("reports/all-incubating.html"))
}

The use of also lets me get the result of the newOutputFile() call, and immediately set the default value provider, but I think we should probably have a way to do it directly. Something like:
@OutputFile
val htmlReportFile = newOutputFile(project.layout.buildDirectory.file("reports/all-incubating.html"))

and probably also with an explicit provider:
@OutputFile
val htmlReportFile = newOutputFile {
   // lazy provider
}

The same is needed for input files, but inputs/outputs in general: look at this code too: https://github.com/melix/gradle-incubating-plugin/blob/master/src/main/kotlin/org/gradle/plugins/incubating/IncubatingApiAggregateReportTask.kt#L19-L22
Is there a better pattern? I think if there's a better one it needs to be documented right now, because it wasn't obvious to me, just reading the documentation.

@melix It seems that Property#convention solve your use case. Reopen an issue on gradle/gradle if you think otherwise.


Add TEPs requirements to pull request template checklist

Changes

Any user-facing changes in Tekton Pipelines need to be approved in
an implementable Tekton Enhancement Proposal.
Today, contributors may not know this requirement when they open pull
requests needing reviewers to catch it. Also, user-facing PRs can also
be mistakenly merged when the TEPs are not yet in implementable state.
In API WG on 03/07/2022, we discussed adding this requirement to the PR
checklist so that contributors are aware of it, or as a reminder.
References:

https://github.com/tektoncd/community/tree/main/teps
https://github.com/tektoncd/community/blob/main/teps/0001-tekton-enhancement-proposal-process.md

/kind misc
cc @bobcatfish
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[n/a] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

NONE

/lgtm

I like the idea of adding this item to the PR checklist, but I would change the wording of "user facing changes". For example, a bug fix is a user facing change but doesn't require a TEP. maybe link to "what type of work should be tracked in a TEP".

@lbernick good idea to clarify which kinds of changes - updated it to enhancements and linked the doc specifying what should be in a TEP

/lgtm


Enhancement: Move time-series controls to the top with other Spread Trends controls

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to scroll all the way to the bottom to have to see graphs by time period. I think it really looks out of place at the bottom. When I first used the site, I was bummed that I couldn't find any controls for seeing trends over a specific time period (say last 30 days), even though they were there at the end.
Describe the solution you'd like
I suggest moving the time series button to the top along with the rest of the 'Spread Trends' controls (like scale modes, 7-day moving average switch, state-wise filter, etc).
Describe alternatives you've considered
NA
Additional context
Suggestion:

Something like what Google has:

I think it's too late to change this now. It'll probably annoy more people than it pleases


Test with docker for Mac

Verify hmake works with Docker for Mac, not docker + docker-machine + virtualbox.

It's known current hmake doesn't work for Docker for Mac. We need to refactor the docker driver to run both with docker-machine or without docker-machine.


Add line breaks to fix heading formatting

The heading tags were not rendering correctly on the Feedback: Concepts module but I think this should fix it.

Thanks, I never noticed that. I wonder if it happened because of a change in the markdown renderer GitHub pages uses?

Actually, seems like it was the lack of a space between ### and the title


Feature request: type-specific functions composition remains type-specific

👋 From what I read of the source code, compose and andThen in  it.unimi.dsi.fastutil.Function are inherited from java.util.function.
Therefore, composing two type-specific functions returns a function that uses apply() instead of get, which means that we end up boxing/unboxing where we could remain in the type-specific world.
I'm wondering whether or not it'd be reasonable to override compose and andThen in this fashion:
// example with Int2FloatFunction
public interface Int2FloatFunction extends Function<Integer, Float>, java.util.function.IntToDoubleFunction {
	@Override
	default <V> java.util.function.Function<Integer, V> andThen(java.util.function.Function<? super Float, ? extends V> after)
        {
		if (after instanceof Float2DoubleFunction) {
			Int2DoubleFunction composed = k -> ((Float2DoubleFunction) after).get(get(k));
			return (java.util.function.Function<Integer, V>) composed;
		}
                ...
	}
}

or at least have some composition utils between fastutil types (I found such utils in https://github.com/gridtec/lambda4j for example. If you know of other libraries that implement type-specific composition, glad to have your input !)
Thanks for this awesome library btw 😉

Seems entirely sensible to me. Do you wanna try a pull request?

I don't feel 100% confortable editing https://github.com/vigna/fastutil/blob/master/gencsource.sh + I'm no big C expert (especially not with this C->Java conversion) => this may take me way too much time to get this done, at least not without help
If you help me sketch how this would look like in https://github.com/vigna/fastutil/blob/master/gencsource.sh and how/if we'd want to test this, then I could try to write the actual PR.

Note: You only need to edit the .drv files I think. Moreover, you don't need C knowledge. fastutil uses the C preprocessor to replace macros etc, but it is Java code nonetheless :-)
For example https://github.com/vigna/fastutil/blob/master/drv/AbstractFunction.drv is the base for the above code you posted. KEY_VALUE_GENERIC is replaced to <Integer, Float> for example.

Ahhh now I see the problem. The number of combinations is really bad. Well, we're used to large number of classes/methods in fastutil (I had to increase the Eclipse heap so many times..), so probably it's doable with the "fragment" approach. Let me think about it.

So, it was a bit of preprocessor bloodbath, but this is my first attempt:
https://github.com/vigna/fastutil/commit/50ec3e4b608748cf5c069f88bda12b01778c2ced
Note that this implements andThen/compose between type-specific functions, which seems to me the most interesting kind. I could also imagine a variant in which the argument is a standard Java function and we do some mild reflection to guess whether it's type-specific (as in your example). But it seems to me less interesting because we are doing in fact additional boxing and unboxing.
BTW, if anybody here is a C preprocessor wizard and can explain how to remove the ugly redundancy here
COMPOSITION(KEY_TYPE_CAP, VALUE_TYPE_CAP, Byte, bytes, GET_AFTER(Byte), get, , KEY_GENERIC, VALUE_GENERIC, VALUE_GENERIC, KEY_GENERIC)

it would be great. Ideally, GET_AFTER() should be expanded inside COMPOSITION() using the parameter T_C, and I've tried a few deferred evaluation tricks, without results. In this way it works but it's just ugly.

@16pierre , any comments?

Just tested the new API, looks great ! Thank you so much for your help 🙇

Note that this implements andThen/compose between type-specific functions, which seems to me the most interesting kind. I could also imagine a variant in which the argument is a standard Java function and we do some mild reflection to guess whether it's type-specific (as in your example). But it seems to me less interesting because we are doing in fact additional boxing and unboxing.

Indeed this sounds reasonable : the most important part is to be able to compose type-specific functions.
The use-case I saw for overriding java.util's compose was to be able to come back to type-specific functions after using a java.util.Function abstraction, without boxing. For example:
interface TransformerFactory {
     // Implementations could use type-specific functions under the hood
     Function<Integer, Float> getInt2FloatTransformer();
     Function<Float, Double> getFloat2DoubleTransformer();
}

class Foo {
     Int2DoubleFunction transformer;
     // could be type-specific

     public void bar(int param) {
         double result = typeSpecificFunction.get(param);
         ...
     }
}

public void main() {
        TransformerFactory factory = ...;

        // convertToTypeSpecific returns either the underlying type-specific function,
        // either a type-specific wrapper around java.util.Function
        Int2DoubleFunction typeSpecificFunction = Int2DoubleFunction.convertToTypeSpecific(factory.getInt2FloatTransformer().andThen(factory.getFloat2DoubleTransformer()));
       Foo foo = new Foo(typeSpecificFunction);
       ...
}

This is definitely low-priority : you made the right call to only implement the type-specific composition.
Thank you again !

I'm getting a lot of these compiling:
warning: [overloads] andThen(Float2ByteFunction) in Boolean2FloatFunction is potentially ambiguous with andThen(Float2ShortFunction) in Boolean2FloatFunction
Just wondering if we should use some special name.

I don't have a strong opinion about this, fine with muting the warnings / type-specific names.
Maybe type-specific names like andThenFloatetc. could avoid some confusion, especially when inlining FunctionalInterfaces:
// unclear type inference for x->x*3
someIntFunction.andThen(x -> x * 3)
// explicit intToFloat for x->x*3
someIntFunction.andThenFloat(x -> x * 3)

I like this idea. Sort of in line with what JDK is doing with, e.g., OptionalInt etc.

I see now the point. Yes, there's a lot of possible ambiguity which might lead to a lot of casting.

OK, the last commit changes the names. That's backward-incompatible, but it has been just a few days.

Well, in any case comments are welcome!

Great thx a ton !


Amethyst always crashes when hitting notifications

Since release 17.0 every time I want to access notifications amethyst app crashes completely, notifications not accessible anymore.
The same happens on version 0.17.1 , also clearing the cache and erasing data did not help. Going back to 0.16.2 solved the issue but of course i want the zaps!...

Should be fixed on v0.17.5, please confirm.

I had this same issue, and can confirm that it has been corrected (for my environment, anyway).

Yes solved! Thanks.


[RFC] MusicGen .to_bettertransformer() integration

Feature request
Add support for MusicGen Better Transformer integration. MusicGen is composed of three sub-models:

Text encoder: maps the text inputs to a sequence of hidden-state representations. The pre-trained MusicGen models use a frozen text encoder from either T5 or Flan-T5
MusicGen decoder: a language model (LM) that auto-regressively generates audio tokens (or codes) conditional on the encoder hidden-state representations. The pre-trained MusicGen models use the BART decoder structure
Audio codec: used to encode an audio prompt to use as prompt tokens, and recover the audio waveform from the audio tokens predicted by the decoder. The pre-trained MusicGen models use the EnCodec model

=> the text encoder uses the T5 attention module, and the MusicGen decoder uses the BART attention module. Thus, there are no extra attention layers we need to add to optimum. The audio codec is not transformer based, so we don't need to export it to better transformer.
The question is simply how to get the integration working with the sub-model structure. The config file for MusicGen is nested in the same way as the model structure, containing sub-configs for each of the three components: https://huggingface.co/docs/transformers/main/model_doc/musicgen#transformers.MusicgenConfig
=> this means that the text encoder config is accessed as config.text_encoder, and the text encoder model as model.text_encoder. Likewise, the MusicGen decoder config is accessed as config.decoder, and the text encoder model as model.decoder. We need to export the pairs of {models, configs} to their better transformer counterparts, e.g. {model.text_encoder, config.text_encoder} -> better_transformer_text_encoder, and {model.decoder, config.decoder} -> better_transformer_decoder.
Ideally, we'd like to be able to export the entire model to better transformer in one go:
from transformers import  MusicgenForConditionalGeneration

model = MusicgenForConditionalGeneration.from_pretrained("facebook/musicgen-small")
model = model.to_bettertransformer()

However. we can't simply export {model, config} like this, since the top-level config does not contain the config attributes for the sub-models. It's just a place-holder for the sub-model configs.
A simple workaround is to export the text encoder and decoder separately:
from transformers import  MusicgenForConditionalGeneration

model = MusicgenForConditionalGeneration.from_pretrained("facebook/musicgen-small")
model.text_encoder = model.text_encoder.to_bettertransformer()
model.decoder = model.decoder.to_bettertransformer()

=> but this diverges from the better transformer API
Motivation
~9M MusicGen downloads per month -> huge interest in running the model!
Your contribution
Happy to help with the integration!

cc @fxmarty


Use setup.py to install calico-diags, instead of deb/rpm incantations

As well as simplicity, this has the advantage that calico-diags will
be correctly installed (including in a virtualenv, if applicable) if
the calico code is installed using 'pip install'.

FYI this has been through a successful DevStack/Calico test now.

test this please


Middleware - Authentication

Is it possible to override the middleware authentication to redirect to a specific route?

You can listen for the error event and do whatever you want 😄
http://adonisjs.com/docs/3.2/error-and-exceptions#_catching_exception


Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I'd like to have an index that gives me a pure LLM call. I understood that EmptyIndex would give me the answer.
But I'm having always Empty Response while I'm querying a EmptyIndex.
from llama_index.core.indices import SQLStructStoreIndex, PandasIndex, EmptyIndex

prompt = "What is the capital of US?"

index = EmptyIndex(llm = wrp._service_context_llm_predictor,
                   embed_model = wrp._service_context_embedding
                  )

print(index.as_query_engine(response_mode = "generation", llm=wrp._service_context_llm_predictor).query(prompt))

Can you help me understand If this is expected. Or the solution for my question?

When I'm create a chat with an EmptyIndex, Is the expected response to be always Empty Response?
How can I get the pure LLM calls when chat with an EmptyIndex?

This code is given me Empty Response.


clarify documentation of extensible record types?

The sentences

You can also define extensible records. This use has not come up much in practice so far, but it is pretty cool nonetheless.

in the section http://elm-lang.org/docs/records#record-types were found confusing. See this discussion: https://github.com/elm-lang/elm-compiler/issues/1283#issuecomment-182682422

Also this: https://github.com/elm-lang/elm-lang.org/issues/514, where there was also confusion about whether or not adding and removing fields from record values is possible. (And the request that if not, that should be said explicitly in the documentation.)

I have updated the docs #555

Thanks @halfzebra

What about the opinion if this guy?
http://reasonablypolymorphic.com/blog/elm-is-wrong
Maybe nobody was using these extensible types since this feature does not exist in other mainstream languages (well actually Typescript allows extension of the "record" types, but not contraction as far as I know), and hence coders are unlikely to use this feature (it even does not exist in standard Haskell).
I found the paper about extensible types amazing, and thought it would be a  good fit for a language that transpiles to Javascript, since in that dynamic language fields are added and removed all the time.
Since unlike Purescript ELM does not have typeclasses, having real extensible types would be a good compromise IMHO.

That guy is the guy who opened https://github.com/elm-lang/elm-compiler/issues/1283 (if I recall correctly).


I think we should axe all usage of this in favor of JWK

          I think we should axe all usage of this in favor of JWK

Originally posted by @decentralgabe in https://github.com/TBD54566975/web5-kt/pull/146#discussion_r1402702054

i think it's safe to close this, because

the codebase has now drifted away from when this PR comment was made
we've now created our own VerificationMethod type
we are currently in the process of creating our own JWK type

if anyone disagrees, please feel free to open a new ticket with more details. thank you 🙏


Activity not logged for the bulk action

The activity is not logged for the bulk action, As an admin I wanted to update status for the users and wanted to log the activity but that won't work with whereIn method.
Here is the code for the bulk update.
User::whereIn('id', $request->get('id'))->update(['status' => 0]);
Is there any way to log such activities ?

This is because of this runs a plain SQL query and doesn't touch eloquent models at all.
You would have to retrieve models first and update them in a loop.


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 452236a:
:clock10: Full build: the average full build time is 6 min(s) 5 sec(s), this is based on the last 353 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 452236a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fullscreen art popover

Make it so that if you click on the album art, it brings it up as a popover in the middle of the screen I guess? idk, might be nice, possibly via a "fullsize" rendition on the artwork (that gets mapped to data-fullsize on the img element).

This was fixed way back in 61dbb11


Missing BC code

Hi, Thanks for your amazing work. However, it seems missing some codes about how to train the bc policies. I'm curious about what the input of bc policies is.

I would also like to request the BC code, it would help a lot with running baselines rather than re-implementing from scratch. Thank you!


Automated cherry pick of #3535: fix: remove private cloud server sku brand and provider attribute

Cherry pick of #3535 on release/2.12.
#3535: fix: remove private cloud server sku brand and provider attribute

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@020fcae). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.12   #3544   +/-   ##
==============================================
  Coverage                ?   6.97%
==============================================
  Files                   ?     498
  Lines                   ?   83265
  Branches                ?       0
==============================================
  Hits                    ?    5810
  Misses                  ?   76961
  Partials                ?     494

Flag
Coverage Δ

#aFlag
6.97% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/skus.go
1.3% <0%> (ø)

pkg/compute/models/cloudsync.go
0% <0%> (ø)

pkg/multicloud/esxi/fakeregion.go
0% <0%> (ø)

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 020fcae...2979327. Read the comment docs.

/lgtm

/approve

/approve


$param vs $parameters

Somtimes it's the first, sometimes it's the second. We should be consistent about this.

Because it's an array, in both cases it should be at the plural: $params or $parameters.

I vote for $params because it is shorter.


[Tooltip] Moving anchor can cause tooltip to stay in view after mouse move

[x] This is a v1.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
If you have an open tooltip and an onClick handler on the anchor which moves the anchor out of focus of the mouse the tooltip should dismiss as the mouse is no longer over the element
Current Behavior
The tooltip remains open until a click somewhere else in the window is triggered, giving what appears to be a broken behaviour.
Steps to Reproduce
Codesandbox example
Click the button and see the tooltip remain visible
Context
My use case is a little more complex than the above example, but this demonstrates the base problem. My element moves when clicking, but the tooltip remains visible. There's no easy way to dismiss it other than creating a completely controlled component and handling mouse events manually.
There may be some cases where dismissing on click isn't the desired behaviour, so it may be preferable to have this as an option (e.g. dismissOnClick={true}) or exposing a call that can dismiss the tooltip (e.g. this.ref.tooltip.close())
Your Environment

Tech
Version

Material-UI
v1.4.1

React
v16.4.1

browser
Chrome 68

There's no easy way to dismiss it other than creating a completely controlled component and handling mouse events manually.

@Thomas101 There is a open / onClose / onOpen property. Controlling the tooltip should be straightforward.

That's true and what I've done for time being.
Felt a bit like a broken behaviour, and a pain to have to handle the state/pull out into a new component when tooltips are normally so zero-config :)
If it's the desired behaviour, happy to leave it there,
Thanks

@Thomas101 Well, https://github.com/mui-org/material-ui/issues/12289#issuecomment-408262317 might enable you not to have to control the tooltip. Maybe you could try this change internally?

Replacing the mouseenter / mouseleave events with mouseover / mouseout events doesn't help. I fear controlling the tooltip is your best option.

Okay, thanks for looking into it and following up.
By the way great work on material-ui. We love it :)

It's a bit hacky but you could do a toggle of the tooltip like:
setTooltipOpen(false)
setTimeout(() => setTooltipOpen(true),1)

And add a hover listener, this will unmount and mount the tooltip several time, i don't think that is the best solution, but maybe it could help


added eslint.js

Fixed styling errors in src folder
Added eslintrc.js
Made sure it works alongside ESlint plugin.

Attaching reviewers

I actually noticed that the linting hasn't been added to the workflow. Refer to this comment https://github.com/advweb-grp1/advanced-web-final-year-project/issues/19#issuecomment-1478774709

Had to run npm install eslint-plugin-import@latest --save-dev for command npm run lint to work. Can this just be added to the package.json?

Already have eslint-plugin-import in package.json

Not sure if the linting is ran when the command npm run ci part. Need to research

Not sure if the linting is ran when the command npm run ci part. Need to research


[Feature]: Generate ui tool for logs managment

Feature type
New Feature
Description
This is more a side-task, than  a real task.
Create desktop application, which will allow to manipulate logs. It should:

Map logs in real time, allowing users to see live traffic.
Allow for metadata manipulation.
Allow to remove outdated logs

Moved this task to proper project


PhysicsServer3D

Description
This issue tracks the progress of implementing the functionality of Godot's PhysicsServer3D object. Below you'll find all user-facing properties and methods, including ones inherited from relevant classes. If you notice any missing from this list, please leave a comment down below.
Status

Not supported
Not started
Partial (not functional)
Partial (functional)
Done (as much as it can be) 👈
Done (completely)

Links
Documentation: PhysicsServer3D
Inherits: N/A
Legend

🚧 indicates a partial, incomplete or broken implementation.
🛑 indicates that it's blocked or incompatible with Jolt in some way.

Methods

[x] area_add_shape
[x] area_attach_object_instance_id
[x] area_clear_shapes
[x] area_create
[x] area_get_collision_layer
[x] area_get_collision_mask
[x] area_get_object_instance_id
[x] area_get_param
[x] area_get_shape
[x] area_get_shape_count
[x] area_get_shape_transform
[x] area_get_space
[x] area_get_transform
[x] area_remove_shape
[x] area_set_area_monitor_callback
[x] area_set_collision_layer
[x] area_set_collision_mask
[x] area_set_monitor_callback
[x] area_set_monitorable
[x] area_set_param
[x] area_set_ray_pickable
[x] area_set_shape
[x] area_set_shape_disabled
[x] area_set_shape_transform
[x] area_set_space
[x] area_set_transform
[x] body_add_collision_exception
[x] body_add_constant_central_force
[x] body_add_constant_force
[x] body_add_constant_torque
[x] body_add_shape
[x] body_apply_central_force
[x] body_apply_central_impulse
[x] body_apply_force
[x] body_apply_impulse
[x] body_apply_torque
[x] body_apply_torque_impulse
[x] body_attach_object_instance_id
[x] body_clear_shapes
[x] body_create
[x] body_get_collision_layer
[x] body_get_collision_mask
[x] body_get_collision_priority
[x] body_get_constant_force
[x] body_get_constant_torque
[x] body_get_direct_state
[x] body_get_max_contacts_reported
[x] body_get_mode
[x] body_get_object_instance_id
[x] body_get_param
[x] body_get_shape
[x] body_get_shape_count
[x] body_get_shape_transform
[x] body_get_space
[x] body_get_state
[x] body_is_axis_locked
[x] body_is_continuous_collision_detection_enabled
[x] body_is_omitting_force_integration
[x] body_remove_collision_exception
[x] body_remove_shape
[x] body_reset_mass_properties
[x] body_set_axis_lock
[x] body_set_axis_velocity
[x] body_set_collision_layer
[x] body_set_collision_mask
[x] body_set_collision_priority
[x] body_set_constant_force
[x] body_set_constant_torque
[x] body_set_enable_continuous_collision_detection
[x] body_set_force_integration_callback
[x] body_set_max_contacts_reported
[x] body_set_mode
[ ] 🛑 body_set_omit_force_integration
[x] body_set_param
[x] body_set_ray_pickable
[x] body_set_shape
[x] body_set_shape_disabled
[x] body_set_shape_transform
[x] body_set_space
[x] body_set_state
[x] body_test_motion
[x] box_shape_create
[x] capsule_shape_create
[x] concave_polygon_shape_create
[x] cone_twist_joint_get_param
[x] cone_twist_joint_set_param
[x] convex_polygon_shape_create
[ ] 🛑 custom_shape_create
[x] cylinder_shape_create
[x] free_rid
[x] generic_6dof_joint_get_flag
[x] generic_6dof_joint_get_param
[x] generic_6dof_joint_set_flag
[x] generic_6dof_joint_set_param
[ ] 🛑 get_process_info
[x] heightmap_shape_create
[x] hinge_joint_get_flag
[x] hinge_joint_get_param
[x] hinge_joint_set_flag
[x] hinge_joint_set_param
[x] joint_clear
[x] joint_create
[ ] 🛑 joint_get_solver_priority
[x] joint_get_type
[x] joint_make_cone_twist
[x] joint_make_generic_6dof
[x] joint_make_hinge
[x] joint_make_pin
[x] joint_make_slider
[ ] 🛑 joint_set_solver_priority
[x] pin_joint_get_local_a
[x] pin_joint_get_local_b
[x] pin_joint_get_param
[x] pin_joint_set_local_a
[x] pin_joint_set_local_b
[x] pin_joint_set_param
[x] separation_ray_shape_create
[x] set_active
[x] shape_get_data
[x] shape_get_type
[x] shape_set_data
[x] slider_joint_get_param
[x] slider_joint_set_param
[ ] 🛑 soft_body_get_bounds
[x] space_create
[x] space_get_direct_state
[x] space_get_param
[x] space_is_active
[x] space_set_active
[x] space_set_param
[x] sphere_shape_create
[ ] 🛑 world_boundary_shape_create

This tracking issue has served its purpose and will now be closed. 🫡
Any remaining work has had new issues created for it.


[Event hubs] V2 stack onboarding

Tracking work item: https://github.com/Azure/azure-sdk-for-java/issues/38476

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

[linking related work] Additional improvements for caching (request response channel, sessions caching) in v2 https://github.com/Azure/azure-sdk-for-java/pull/39107


Tweaks for ASB pubsub

Do not attempt to reconnect when context was canceled
Message finalization uses a background context and not the running ctx, in case the app is shutting down

Although the context isn't cascaded from Dapr, this component does use a context internally and it's canceled when the context is closed: https://github.com/dapr/components-contrib/pull/1729/files#diff-8f0a750f05222eb118aded3b8c6c3a926a2564220b5a9c2fd06e6e75c7180aafL615

@ItalyPaleAle but how is the context closed right now?

It's created in the Init method
https://github.com/dapr/components-contrib/blob/6b99b02a2d36b69999957e3c790b46a15069e1dc/pubsub/azure/servicebus/servicebus.go#L305
It's closed in the Close method, which is invoked by Dapr at shutdown.
https://github.com/dapr/components-contrib/blob/6b99b02a2d36b69999957e3c790b46a15069e1dc/pubsub/azure/servicebus/servicebus.go#L621
Internally in the component, the context is used to cancel subscriptions.
When Dapr adds support for cascading the context, this should work transparently without changes.


Retry submission on no resource left.

Until https://github.com/aws/aws-parallelcluster/issues/3576 get's resolved, is there a config that can make slurm/parallelcluster retry to create instances until one is available instead of terminating it?
This is problematic especially if that task also has dependencies, then the whole chain of jobs are rejected.
(Example failure message:
Retries
2022-05-09 18:01:44,275 - [slurm_plugin.instance_manager:add_instances_for_nodes] - ERROR - Encountered exception when launching instances for nodes (x1) ['gpu-dy-g4dnxlarge-1']: An error occurred (InsufficientInstanceCapacity) when calling the RunInstances operation (reached max retries: 1): We currently do not have sufficient g4dn.xlarge capacity in the Availability Zone you requested (us-east-1a). Our system will be working on provisioning additional capacity. You can currently get g4dn.xlarge capacity by not specifying an Availability Zone in your request or choosing us-east-1b, us-east-1c, us-east-1d, us-east-1f.

Hi lletorn,
The current behavior has retries. When there is insufficient capacity error, the cluster will try to relaunch compute nodes after a period of time (2 minutes by default). During these retries, the job is re-queued as long as --no-requeue option is not used.
According to my understading, your job failed. Did the job get requeued?
Hanwen,
Thank you!

I saw the documentation that says it requeues on NODE_FAIL if no-requeue is set, but this was when using SPOT instances. I haven't found any documentation that talks about ON-DEMAND nodes hitting InsufficientInstanceCapacity.

Forgot to mention I'm running pcluster 3.1.2

By default, we allow requeue regardless of SPOT or ON-DEMAND.
Could you provide the command you used for srun or sbatch and your /opt/slurm/etc/slurm.conf?
Thank you!

There you do:
srun -p gpu --time=04:00:00 -c 4 --pty bash -i

Hi @lletourn
to have the job requeued, you should use the sbatch command, --no-requeue option is not supported by srun command.

The srun command is designed for interactive use, with someone monitoring the output. The output of the application is seen as output of the srun command, typically at the user's terminal. The sbatch command is designed to submit a script for later execution and its output is written to a file. [...] Another significant difference is in fault tolerance. Failures involving sbatch jobs typically result in the job being requeued and executed again, while failures involving srun typically result in an error message being generated with the expectation that the user will respond in an appropriate fashion.

Source: Slurm documentation
Let us know if it helps.
Enrico


--server-connect-attribute private_ip_address no longer works

Version:
knife-ec2 2.0.0
chef 16.2.50
Environment:
Mac OSX 10.15.6 on ruby 2.6.2
Scenario:
Running knife ec2 server create with the --server-connect-attribute private_ip_address does not work correctly and chef bootstrap tries to connect over SSH to the public dns name instead.  This appears to be a regression since I just updated from version 0.13.0 of knife-ec2.
Steps to Reproduce:
Run knife ec2 server create with the the option --server-connect-attribute private_ip_address.  You can see from the log output that the chef bootstrap portion is incorrectly connecting using the public dns name:
Waiting for EC2 to create the instance
<<REDACTED>>
Public DNS Name: ec2-35-160-149-120.us-west-2.compute.amazonaws.com
Private IP Address: <IP_ADDRESS>
Waiting for sshd access to become available
SSH Target Address: <IP_ADDRESS>(private_ip_address)
................done
SSH Target Address: <IP_ADDRESS>(private_ip_address)
Connecting to ec2-35-160-149-120.us-west-2.compute.amazonaws.com using ssh
WARN: [SSH] connection failed, terminating (#<Net::SSH::ConnectionTimeout: Net::SSH::ConnectionTimeout>)
ERROR: Train::Transports::SSHFailed: SSH session could not be established

It appears the ssh connection test is correctly using the private ip address, but the bootstrap portion is incorrectly using the public dns name.
Expected Result:
The chef bootstrap should be using the private ip address instead.  In the example above, the output should say the following instead: Connecting to <IP_ADDRESS> using ssh
Actual Result:
The chef bootstrap uses the public dns name instead, as shown in the log message:
Connecting to ec2-35-160-149-120.us-west-2.compute.amazonaws.com using ssh
[What actually happens after the reproduction steps?]
The chef bootstrap tries to connect to the public dns address instead of the private ip address and fails because in our environment we do not allow connecting from the public interface.
Possible fix?
Removing/commenting out these lines: https://github.com/chef/knife-ec2/blob/master/lib/chef/knife/ec2_server_create.rb#L1401-L1411 appears to fix the problem for my specific use case.  However, I'm not sure what the intention is behind this code and if removing it has any unwanted side effects on other use cases.  Happy to submit a PR though if you think removing these lines is sufficient.

Hello knife-ec2 developers,
Is this ticket likely to get attention in the near future? This is still very much a problem from what I can tell in knife-ec2 gem version 2.0.4. Is Chef/Opscode considering abandoning support for EC2? I only ask because issues like this going unaddressed for (close to) two months makes me question if the software is still supported.
Here's my output showing the knife EC2 code failing to respect the --server-connect-attribute flag properly:
+ knife ec2 server create -I ami-0f82752aa17ff8f5d --connection-user ubuntu --ssh-identity-file /home/rkeiii/.ssh/bks-ec2-202004.pem --ssh-key bks-ec2-202004 --subnet subnet-f11eae9f -g sg-441efa2b -g sg-e8238687 -g sg-e94df086 -g sg-da3aedb5 --associate-eip <IP_ADDRESS> -N app5.qa -f t2.large -E qa --ebs-size 15 --ebs-volume-type gp2 --run-list 'role[xenial_app_class]' --bootstrap-template ../bootstrap/chef-full.erb --iam-profile ec2-role --server-connect-attribute private_ip_address

Waiting for EC2 to create the instance
Instance ID: i-03d3fdd2458e788cd
Flavor: t2.large
Image: ami-0f82752aa17ff8f5d
Region: us-east-1
Availability Zone: us-east-1d
Security Group Ids: sg-e94df086, sg-e8238687, sg-441efa2b, sg-da3aedb5
IAM Profile: ec2-role
AWS Tags: Name: app5.qa
SSH Key: bks-ec2-202004
T2/T3 Unlimited: Disabled

Subnet ID: subnet-f11eae9f
Tenancy: default
Public IP Address: <IP_ADDRESS>
Private IP Address: <IP_ADDRESS>

Waiting for sshd access to become available
SSH Target Address: <IP_ADDRESS>(private_ip_address)
...done

SSH Target Address: <IP_ADDRESS>(private_ip_address)
Connecting to ec2-54-211-181-96.compute-1.amazonaws.com
<EMAIL_ADDRESS>Permission denied (publickey,password).
ERROR: Train::Transports::SSHFailed: SSH command failed (command failed: ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o BatchMode=yes -o LogLevel=ERROR -o ForwardAgent=no -i /home/rkeiii/.ssh/bks-ec2-202004.pem<EMAIL_ADDRESS>-p 22 -W ec2-54-211-181-96.compute-1.amazonaws.com:22)


Add boto3-pip entry

PyPI project URL: https://pypi.org/project/boto3/

Hi Tully, how can we depend on the latest version of boto3 in a ROS package? The version in Xenial (1.2.2) is far behind pip (1.9.35) and missing essential features.

If you need it you can locally create your own rosdep rules. But we don't want to distribute that to the community due to it potentially breaks anything that depends on the system packaged version.


Add a filter for Route53 API results

Route53 API can return larger names than requested when there is no exact match. This PR adds a filter ensuring that resolve method only returns a list when there is an exact match with the requested name and entry type.
I tried to reproduce the problem with tests, but the mocked Route53 does not return larger names.
This PR has been applied to our internal deployment and is working as expected. There are more information on how to reproduce the error on the bug report #1236.
Fixes #1236.

Hey @pedrokiefer, we require all commits to be signed before merging a PR, and it looks like your first commit for this branch isn't signed. Could you open a duplicate PR and sign all the commits? You could also try some git wizardry to retroactively sign the commit for this PR, but it will probably be more of a hassle than opening a new PR.

Hey @pedrokiefer, moving current PR to v0.19.1 milestone. Once we have a new PR (or updated commit), we will merge the changes.

I’ll update the PR later tonight.

Hey @pedrokiefer, we have moved current PR to v0.19.2 milestone. Once we have a new PR (or updated commit), we will merge the changes.

Sorry for the delay, just signed the commit.


Feature Request: Single Import Usage?

Currently the getting started example is this:
import com.softwaremill.sttp._
implicit val backend = HttpURLConnectionBackend()
sttp.get(uri"http://httpbin.org/ip").send()

Any chance we could add a convenience import to avoid the implicit val?
// brings in an `implicit val backend = HttpURLConnectionBackend()` automatically
import com.softwaremill.sttp.default._
sttp.get(uri"http://httpbin.org/ip").send()

Would make sttp much more convenient to use from e.g. the Ammonite REPL

Interesting idea. Definitely possible - some changes would be needed (e.g. moving the contents of the package object to a trait, and extending it in both packages), but nothing major. Probably the package could be called sth indicating that it brings in a synchronous backend (import com.softwaremill.sttp.sync._?).
One worry I would have, though, is that using sttp in a REPL and in a "normal" application would be different. In REPL you would import the package that brings in the synchronous backend, while in a normal application you would import the one which requires a backend to be defined. Don't you think that this could confuse users?
A solution here could be to name the package so that it indicates that it's a "quick" or "REPL-only" solution, so maybe:

import com.softwaremill.sttp.quick._
import com.softwaremill.sttp.repl._

?

Sure the exact package name doesn't matter to me, any of your suggestions work

I'm not sure if there are other stateless backends too (e.g. an async one?) but for all the stateless ones it would make sense to add a default "no config" import to use them easily


Navbar Buttons not interactable in Projects Page.

Description
Navbar Buttons not interactable in Projects Page as can be seen in the video .Will make those buttons working and also add hover effect on them.
https://github.com/Opentek-Org/opentek/assets/74114936/c13247f0-5230-49dc-9e20-0bcaf26ddb86
Screenshots
No response
Additional information
No response
Do you want to work on this issue?

[X] I want to work on this issue.

@AnirudhPanda


Discussion model benchmarks

This issue is to discuss ideas around standard model benchmarks. Here is my suggestion, as a starting point for the discussion. The main idea is to group benchmarks for a specific model, and by a specific model I mean with all the options (e.g. thermal, degradation...) fixed. Then, for each model we test various:

Parameter sets
Mesh sizes
Solvers & settings
CC discharge and various experiments

I think for the moment we can just change one setting at the time (the default could be the first setting of each group). We can later think how to sweep two or more variables (e.g. parameter sets & mesh sizes).
In terms of implementation, we can define a base class to which we can pass the model we want to study, and all benchmarks can be inherited from there. If we keep notation consistent, it should be fairly easy to write a script to process the JSON files.
Thoughts?

What's the plan with this?

I guess it is on hold (I opened this issue more as a discussion rather than a specific task). Saying that, I realise we should probably convert it into a discussion and follow it up there.


Reconsider need for macOS 10.14-specific DisplayLink vsync workaround?

During research for #33116 I encountered suggestions that the underlying V-Sync issue was fixed in a subsequent minor OS point release. [1]
This made me wonder if the workaround added via https://github.com/godotengine/godot/commit/b53f2d1d598193ab1d1c02345fc321be254364ca was still required.
My primary reason for asking is that I have suspicions that the DisplayLink-based workaround may be causing Godot editor and/or games to hang (when V-Sync is enabled) when my MacBook Pro on Mac OS X 10.9.5 goes to sleep automatically.
I did some preliminary investigations which suggested the DisplayLink thread may not survive the sleep/wake cycle but haven't had the chance to investigate properly.
Next steps:

Someone may wish to test if the DisplayLink patch is still necessary for 10.14+
At some point I may be able to dig deeper into the sleep issue I observed.

[1] e.g.

https://github.com/openframeworks/openFrameworks/issues/6146#issuecomment-470499466
https://twitter.com/FlohOfWoe/status/1103296098834305026
https://github.com/openframeworks/openFrameworks/issues/6146#issuecomment-519195854
https://github.com/glfw/glfw/issues/1337#issuecomment-476482317
https://github.com/cocos2d/cocos2d-x/issues/19080#issuecomment-478640731
https://stackoverflow.com/questions/52938516/opengl-not-rendering-on-macos-mojave#52938517
https://hg.libsdl.org/SDL/rev/73f3ca85ac0e

[2] Here are some other recent Mac OpenGL patches I noticed in other projects in case we encounter the same issues at some point:

https://github.com/wxWidgets/wxWidgets/pull/1617 & "wxGLCanvas macOS texture issues with 10.13 and lower"
https://github.com/wxWidgets/wxWidgets/pull/1354
https://github.com/qt/qtbase/commit/f39230fcac4de01f26945bde16c3a10c5ac74afb

[3] Random other semi-related links I encountered:

https://stackoverflow.com/questions/20083027/nsopenglview-and-cvdisplaylink-no-default-frame-buffer#44301471
https://stackoverflow.com/questions/54891914/nsopenglview-drops-its-nsopenglcontext-on-macos-10-9

Edit: Link to Mac OS sleep-related issue which might be related: https://github.com/godotengine/godot/issues/28039

cc: @BastiaanOlij due to being author of the original patch

SDL have removed it in https://hg.libsdl.org/SDL/rev/1b6f67e84802

If Apple fixed the issue I don't see any reason why we shouldn't undo the DisplayLink fix. It really was just a bandaid imho.

BTW in the unlikely case this information is useful to someone in future, as noted in the above issue, at some point I discovered that almost no CVDisplayLink implementations correctly handle a situation that can occur (via Fast User Switching) where the link is destroyed & then a new link created later:

"In macOS 10.4 and later, the display link thread is automatically stopped if the user employs Fast User Switching. The display link is restarted when switching back to the original user. "
-- https://developer.apple.com/documentation/corevideo/1457281-cvdisplaylinkstop?language=objc

As I understand it this was the cause of a Godot hang I encountered (at the time) of uncertain origin but turned out to be related to the use of Fast User Switching while Godot was running.


Maximum callstack exceeded

error: Uncaught RangeError: Maximum call stack size exceeded
    const row = new this(...cells);
                ^
    at Function.from (https://deno.land/x/cliffy@v0.20.1/table/row.ts:27:17)
    at TableLayout.createRow (https://deno.land/x/cliffy@v0.20.1/table/layout.ts:156:16)
    at TableLayout.spanRows (https://deno.land/x/cliffy@v0.20.1/table/layout.ts:131:29)
    ...
    at TableLayout.spanRows (https://deno.land/x/cliffy@v0.20.1/table/layout.ts:148:17)
    at TableLayout.createLayout (https://deno.land/x/cliffy@v0.20.1/table/layout.ts:54:36)
    at TableLayout.toString (https://deno.land/x/cliffy@v0.20.1/table/layout.ts:32:40)
    at Table.toString (https://deno.land/x/cliffy@v0.20.1/table/table.ts:126:48)
    at Table.render (https://deno.land/x/cliffy@v0.20.1/table/table.ts:131:22)
    at file:///Users/jb/Downloads/table.ts:80:6

code
// Imports
import { parse } from<EMAIL_ADDRESS>import { resolve } from<EMAIL_ADDRESS>import {
  Cell,
  Row,
  Table,
} from<EMAIL_ADDRESS>import { green, red } from<EMAIL_ADDRESS>
interface Item {
  at: Date;
  type: string;
  description: string;
  total: number;
  fee: number;
  receiver?: string;
  account: string;
}

const items: Item[] = [];

for (const filename of Deno.args.map((filename) => resolve(filename))) {
  const data = JSON.parse(Deno.readTextFileSync(filename)) as Item[];
  items.push(...(data.map((value) => (value.at = new Date(value.at), value))));
}

console.log(items.filter((item) => item.total === undefined));

const data = items.sort((a, b) => a.at.getTime() - b.at.getTime()).map(
  (
    { account, at, description, fee, total, type, receiver },
    _1?: unknown,
    _2?: unknown,
    netto = total - fee,
  ) => {
    return [
      Cell.from(account).align("center"),
      Cell.from(
        at.getDate().toString().padStart(2, "0") + "/" +
          (at.getMonth() + 1).toString().padStart(2, "0") + "/" +
          at.getFullYear(),
      ),
      Cell.from(type).align("center"),
      Cell.from(
        (total > 0 ? green : total < 0 ? red : (s: string) => s)(
          (total < 0 ? total * -1 : total).toFixed(2),
        ),
      ).align("right"),
      Cell.from(
        (fee > 0 ? green : fee < 0 ? red : (s: string) => s)(
          (fee < 0 ? fee * -1 : fee).toFixed(2),
        ),
      ).align("right"),
      Cell.from(
        (netto > 0 ? green : netto < 0 ? red : (s: string) => s)(
          (netto < 0 ? netto * -1 : netto).toFixed(2),
        ),
      ).align("right"),
      Cell.from(description).align("left"),
      Cell.from(receiver || "").align("left"),
    ];
  },
);
(Error as any).stackTraceLimit = 99999;
try {
  new Table()
    .header(Row.from([
      Cell.from("Account").align("center"),
      Cell.from("Dato").align("center"),
      Cell.from("Type").align("center"),
      Cell.from("Totalt Beløp").align("center"),
      Cell.from("Avgift").align("center"),
      Cell.from("Netto").align("center"),
      Cell.from("Beskrivelse").align("left"),
      Cell.from("Konto").align("center"),
    ]))
    .body(data)
    .border(true)
    .render();
} catch (error) {
  throw error;
}

I have added a performance improvement in #327 which solves the issue for tables without col and row span. For col and row span it's simply to much data. Maybe i can solve the issue with an async method or by chunking the data with an async itarator.
Does the fix solve the issue for you? it's currently available in the main branch.

Yup it works now


Getting version numbers set by the build farm

Summary of the Pull Request
Staying draft until we can adjust updating detection.
Since i did a bad branch with #6777, recopied
As outlined in https://github.com/microsoft/PowerToys/wiki/Versioning, we are moving the versioning e2e on the build farm.  From now on there is a single spot that will set the version number and all local dev builds will be 0.0.1
This also sets up the start of having application titles / systray and other spots with the dev environment.  After doing a bit of investigation, this will require a bit more work.
This is also the start of #873 which then will let us start shifting to allow a more complete revision number.  As this has pretty wide reaching changes, i'm just getting the build farm up and functional with this PR.
The script runs local as well as on the farm due to the relative path.
For the farm, this was done by adding in an additional step to execute the PowerShell script and pass in the var's set in ADO.
tools/build/versionSetting.ps1
-versionNumber "$(BuildVersion)" -DevEnvironment "$(DevEnvironment)"

PR Checklist

[ ] Applies to #873
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

As mentioned in the closed PR, we will also have to modify the auto-update code to make sure local builds don't auto-update.

@enricogior and @ryanbodrug-microsoft build farm worked.  You can see 0.0.2 from screenshot

@enricogior can we get that for 0.25 in a different PR and get this and that at the same time?  i'll open a new issue for that

@crutkas
sure.

someone needs to approve the PR :)
i won't check this in until we snap 0.25


feat: allow to pass response function

Changes:
Allow response to be a function. This is useful to be able to mock same API call returning different value when using POST requests.

@moroine thanks for implementing this feature. I believe this feature will fix #44.
I am merging this PR. I have to add few tests for the function types before the release


Feature/Submit an advice

Context
GDG Algiers' discord community members love getting advice from the bots there. Brainy's ones are brilliant, yet we almost have seen all of them and the funny  command has started losing its shine.
In order to get it back, we need more advice in the set.
The idea is to let the community members contribute with their own piece of advice and to make this process less technical than doing pull requests.
Proposition
Creating a new command that enables a member in the server to submit an advice that will be added to the list. It is important to include a validation step in the process, to avoid adding irrelevant advice or inappropriate ones.
The suggested workflow

The user runs the command in Brainy's DM in this format :

   $submit <the advice>

The user gets a message confirming the receipt of the advice and saying that it will be validated soon.
The person in charge of validation is notified and is able to validate the advice or reject it.
The user receives a message with the result of the validation.
The advice is added to the set, if validated.

Remaining issues

The validation step is mandatory to preserve the community guidelines, yet since it had to be manual, it can be overwhelming for the person in charge. How can we make it less annoying ?

Tasks

[ ]  Propose a workflow for the validation step
[ ]  Write the code for the new command
[ ]  Test the functioning
[ ]  Release it

Hello There, I think it's a very awesome Idea to allow community members to contribute to our advice space. As we are currently using a google sheet for storing advice I think we should now try moving to database access to better organize it?
For the review and validation part, I thought about having a channel that serves as a log for all the feature submissions and moderators can validate, cancel or edit a submission. What do you think about it @CerineBnsd @Aymen311 ?

Hello Akram! thanks for the your suggestions, I think they are great and should be taken into consideration while developing this feature.


feature request: add "easy" API to use cfn-lint as a library

Note: I use the term "easy" api as inspired by the libcurl-easy API which performs most setup steps automatically for the caller. Without this easy API the user must manually initialize all options required.
This issue is a feature request to add a "simple" or "easy" API to cfn-lint so that it can be used as a library within other applications. The goal would be that without any extra options set, one could call a hypothetical cfnlint.simple.run() on a dict or alternatively bytes-like object and return a list of lint violations. There would be a simple way to set options like ignoring rules.
Having the easy api as part of the project will help all users avoid re-inventing this core functionality.
Context:
Our pipeline generates CloudFormation templates via Troposphere as part of a suite of test fixtures. Each fixture could benefit from a cfn-lint step. Today these fixtures are generated dynamically and assert statements are called for each fixture. An easy api would would allow us to set up a parameterized test for linting as well. Without round-tripping the fixture to a file and without the tremendous overhead of calling cfn-lint via subprocess.
Alternatively if there are any hints on how we should implement this ourselves, that would be useful too. I am a bit lost: I would like to use as high level functions as possible but there is a loads() call made implicitly within cfnlint.core.get_template_rules(). Maybe I monkeypatch this to return the template dict I already have?

Hi, https://github.com/aws-cloudformation/cfn-lint/pull/2285 has now been merged, does https://github.com/aws-cloudformation/cfn-lint/blob/main/docs/getting_started/integration.md#getting-started fit your use case?

@LaurenceWarne That looks really good, thank you so much!  FWIW I eventually fumbled my way to:
def get_cfn_lint_matches(test_name, test_input):

    try:
        template = cfnlint.decode.cfn_yaml.load(condenser_output)
    except Exception as e:  # pylint: disable=broad-except
        LOG.error("Skipping cfn-lint validation due to an internal error.\n")
        LOG.error(str(e))
        return []

    # Initialize the ruleset to be applied (some excludes)
    # FIXME initialize .cfnlintrc.yaml correctly.
    rules = cfnlint.core.get_rules([], ["W3011", "W3005"], [], [], False, [])

    # Default janus region
    regions = ["us-west-2"]

    # Runs Warning and Error rules
    matches = cfnlint.core.run_checks(test_name, template, rules, regions)

    return matches

@evilensky should have this officially released in pypi this week


feat(autoware_universe_utils): reduce dependence on Boost.Geometry

Description
This PR is part of https://github.com/autowarefoundation/autoware.universe/pull/7778, and adds the following functions to reduce dependence on Boost.Geometry.

Boost.Geometry  (boost::geometry)
alternative  (autoware::universe_utils)
supported types

correct()
correct()
polygon

equals()
equals()
point & point / polygon & polygon

intersects()
intersects()
segment & segment / polygon & polygon

within()
within()
point & polygon / polygon & polygon

See the original PR for details and notes.
Related links
Parent Issue:

https://github.com/autowarefoundation/autoware.universe/issues/8128

How was this PR tested?
colcon test --packages-select autoware_universe_utils

Benchmark result with 500 randomly generated polygons with 9 vertices:

function
Boost
self-impl.
task

intersects()
195.41ms
145.11ms
check if 500 polygons intersect (runs totally 250,000 times)

within()
842.81ms
120.63ms
check if each of 500 polygons is within 500 polygons

Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@mitukou1109 Thanks! But could you fix pre-commit.ci error?


Add -hack-initializers to work around a driver bug

Use -hack-initializers to work around driver bugs that cause the
get_global_size builtin function to return the wrong value if used with
a non-constant dimension argument.
The underlying issue is that SPIR-V Private variable initializers may
not take effect.  The workaround is to start every kernel with an
explicit store to write the initializer value.  This is only required
for a compiler-generated variable that stores the workgroup size, only
used to implement the get_global_size builtin.

Tested on Release and Debug builds.
Necessary for some driver versions.
LGTM.


Android Error: animateToRegion

I am getting the following error in Android;
Attempt to invoke virtual method 'void com.google.android.gms.GoogleMap.animateCamera( com.google.android.gms.maps.CameraUpdate, int, com.google.android.gms.maps.GoogleMap $CancelableCallback)' on a null object reference
animateToRegoin AirMapView.java:457

I had not enabled the Google Maps API in the Google developer console. The issue is resolved.


Create a TraceListener that writes to Stackdriver Logging

This will allow ASP.NET apps to easily report trace information to the Stackdriver Logging API.
cc: @benwulfe @jskeet @csells

It looks like we will be able to just write out to the console.


updated amidst (4.1)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as 445f4edc617877dc707334fc224c927731a4391f.


[bug fix] Skip the pod inaccessible for istioctl ps --xds-via-agent

This PR proposes to skip the pod inaccessible for the command istioctl ps --xds-via-agent.
When there is a pod which is terminating or not running yet, such pod can cause an error when creating a port-forward.
During traversing pods for istioctl ps --xds-via-agent, it blocks to go with the other healthy pods.
So, this PR skips such inaccessible pods in the traversing loop.

In response to a cherrypick label: new pull request created: #41738

In response to a cherrypick label: new pull request created: #41739

In response to a cherrypick label: new pull request created: #41740


WEBPAGE: Confirm related sightings page*

Admin will see related sightings detected by algorithm, and can confirm or deny if they think the sightings are the same.
*need to figure out which fields to show to the admin, and how related sightings will be displayed in the exported data

Let admin choose which field is the ultimate field?
merging somehow


useStyles hook

🚀 Feature Proposal
An optional useStyles('CustomComponent', props) hook alternative to styled HOC/decorator for custom components.
Motivation
A hook could lead to cleaner api/less typing/less wrapper components in component tree when you inspect...
Example
I have a somewhat working implementation below. I needed StyleConsumerService and MappingContext to get it working, but they aren't exported at package level so the direct file imports feel a little brittle since a new version update could change internal structure.
import { useContext, useState, useMemo } from 'react';
import { useTheme } from '@ui-kitten/components';
import { StyleConsumerService } from<EMAIL_ADDRESS>import { MappingContext } from '@ui-kitten/components/theme/mapping/mappingContext';

export function useStyles<P = any>(componentName: string, sourceProps: P): any {
  const [interaction, dispatch] = useState([]);
  const theme = useTheme();
  const mappingContext = useContext(MappingContext);
  const service = useMemo(() => new StyleConsumerService(componentName, mappingContext), [
    componentName,
    mappingContext,
  ]);
  const defaultProps = useMemo(() => service.createDefaultProps(), [service]);
  const props = { ...defaultProps, ...sourceProps };
  const style = service.createStyleProp(props, mappingContext, theme, interaction);

  // return eva object:
  return {
    theme,
    style,
    dispatch,
  };
}

Usage
export const CustomComponent = props => {
    // needs the props to determine variants:
    const eva = useStyles('CustomComponent', props);
    // convert eva mapping token values to style object:
    const styles = getComponentStyles(eva.style);

    // eva.dispatch([Interaction.HOVER]); possible to change interaction state and trigger rerender styles lookup

    return <View style={styles.container}>...</View>;
}

// vs styled hoc:

const CustomComponent = props => {
   const evaStyles = props.eva.style;
   // convert eva mapping token values to style object:
   const styles = getComponentStyles(evaStyles);
   return <View style={styles.container}>...</View>;
}
const StyledCustomComponent = styled('CustomComponent')(CustomComponent);
export { StyledCustomComponent as CustomComponent };

There's already an open issue for this here.

There's already an open issue for this here.

Ah yeah, that overlaps and has more fleshed out examples of the hook. If they keep that issue open for the hook, then this one should be closed. I'll leave this issue open for now in case they close the other issue because the original question has a solution that doesn't require the hook (styled decorator as HOC as you posted - I'm also using this approach on my project)

Pasting the comment I wrote for the related issue:
is this something that could be added to ui kitten? I think it would be nice to have. I changed it a bit to make the prop types remain:
import { StyledComponentProps } from '@ui-kitten/components/theme/style/styled';
import { useCallback, useContext, useMemo, useState } from 'react';
import { MappingContext } from '@ui-kitten/components/theme/mapping/mappingContext';
import { ThemeContext } from '@ui-kitten/components/theme/theme/themeContext';
import { StyleConsumerService } from<EMAIL_ADDRESS>
export function useStyled<T>(name: string, sourceProps: T): T & StyledComponentProps {
  const [interaction, setInteraction] = useState([]);
  const mappingStyle = useContext(MappingContext);
  const theme = useContext(ThemeContext);
  const service = useMemo(() => new StyleConsumerService(name, mappingStyle), [name, mappingStyle]);
  const defaultProps = useMemo(() => service.createDefaultProps(), [service]);
  const computedProps = useMemo(() => ({ ...defaultProps, ...sourceProps }), [defaultProps, sourceProps]);
  const dispatch = useCallback((newInteraction: any) => setInteraction(newInteraction), [setInteraction]);
  const style = useMemo(() => service?.createStyleProp(computedProps, mappingStyle, theme, interaction), [
    service,
    computedProps,
    mappingStyle,
    theme,
    interaction
  ]);

  return useMemo(() => ({
    ...computedProps,
    eva: { theme, style, dispatch }
  }), [computedProps, theme, style]);
}

Usage:
import React from 'react';
import { View } from 'react-native';
import { FooterStyle } from './Footer.style';
import { ExternalLink } from '../ExternalLink';

import { useStyled } from '../../lib/hooks/useStyled';

interface FooterProps { }

export function Footer(props: FooterProps) {
  const { eva: { style = null } = {} } = useStyled('Footer', props);

  return (
    <View style={[FooterStyle.container, style]}>
      <ExternalLink href="https://google.com/">Go to link!</ExternalLink>
    </View>
  )
}


Make automatic

Closes #7 and #88

This should work 🤞  , but we won't know until it runs.
Last thing is that we need to check the version is actually updated before making a new release.  This was a manual process before.

🤞


Incorrect build instruciton in readme?

The readme states
https://github.com/digital-asset/daml-finance/blob/d867151d8edf8801e6cbfe4881d60fd1e6b5d9b0/README.md?plain=1#L37-L41
Shouldn't the command be make build-all?

It should be make build-all. If you have time, raise a PR for for this.


[gardening] Change static public to public static for consistency

Before the changes:

git grep "public static" | wc -l: 2007
git grep "static public" | wc -l: 17

@swift-ci smoke test

Excellent
⛵


⚠ Samsung has degraded performance

In 3184244, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2027 ms

Resolved: Samsung performance has improved in c304514.


[sil-mode] Add highlighting for {alloc,project,open}_existential_box.

Title says it all.

@swift-ci Please smoke test and merge


Removes random delay when renewing certificates with the renew now button

Resolves https://github.com/jc21/nginx-proxy-manager/issues/238

This is an automated message from CI:
Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1567
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.


Complete Python Assignment-2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Thanks for your advice! I have updated the codes for the assignment 2. Please review. Thanks!


Board Review: .NET Azure.Core

Contacts and Timeline

Main contacts: @annelo-msft @christothes @JoshLove-msft
Expected code complete date: 10/28
Expected release date: 11/3

About the client library

Name of the client library: Azure.Core
Languages for this review: .NET

Artifacts required (per language)
Please read through “API Review” section here to understand how these artifacts are generated. It is critical that these artifacts are present and are in the right format. If not, the language architects cannot review them with the SDK Team’s API review tool.
.NET

APIView Link: Nullable response

Link to Champion Scenarios/Quickstart samples: https://github.com/Azure/azure-sdk-for-net/issues/27852

APIView Link: RetryPolicy

Link to Champion Scenarios/Quickstart samples: https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/core/Azure.Core/samples/Configuration.md#setting-a-custom-retry-policy

RequestUriBuilder

ResourceIdentifier.TryParse

Scheduled: Oct 31st 2:05PM - 3:00PM PST

@JoshLove-msft The APIView links don't seem to resolve to a valid page. If you get a chance please can you double check them? Thanks!

Updated, though I'm not sure why the links keep becoming invalid. I'm linking to the diff in the auto-review.

Recording (MS INTERNAL ONLY)


Fixed equals method in TreeGen

It seems to be a bit broken, it is comparing this same list twice.

Thanks! The equals method looks like it was automatically generated by an IDE, I wonder how this bug snuck in.


No syntax highlight at some points in js files

VSCode Version: 1.4.0
OS Version: OS X Yosemite 10.10.5

Hi,
using VSCode to write some javascript, sometimes it ends up with no syntax highlight at some points in the file.
Here is an example of what I'm talking about :

The code is correctly formated and functional, and it actually have a correct highlight on other editors like atom for example). I'm using the dark+ theme and I've already tried to install some other themes and adding highlight snippets to see if they made a difference but the problem was still here.
Hope you can help

I have the same problem after updating from version 1.0 to 1.4. All worked without problems in 1.0.

I have the same problem in my typescript files.. and i found it just occur in 1.4 , and now  i use back to v1.3.

Duplicates #9937


Release 0.11.0

Closes https://github.com/chentsulin/electron-react-boilerplate/issues/808 https://github.com/chentsulin/electron-react-boilerplate/issues/833 #909 #905 #874
Remember to mark off https://github.com/chentsulin/electron-react-boilerplate/issues/504

Hey amila, just tested it on a fresh clone and everything was working as supposed to!

@uppe-r of dev-master or master?

dev-master

@williamoliveira @uppe-r found an issue when reloading after navigating to a certain route. To reproduce, click on 'counter' and reload.

@amilajack You're right. This happened to me before and I kinda ignored it since I was testing the old content loading bug.
Probably has something to do with #884?

master was using a hash router, on dev-master its an html5 one

@williamoliveira +1 for a PR :)


Show that duplicate nullifier errors are propagated through the transaction verifier

There are currently no tests to check that duplicate nullifier errors from the state are being propagated through the transaction verifier correctly.
~But since our error handling is generic, we don't need to write new tests, we can just point to existing tests that together cover this kind of error handling.~
We are missing test coverage for error conversions from zebra-state to zebra-consensus.

I think we can respond to this issue by finding existing tests that cover this kind of error handling.
For example:

unit tests for duplicate nullifier errors
integration tests for any state errors in mempool transaction validation or block validation

a good example would be submit block or block proposal RPC errors

full zebrad integration tests that run the syncer and inbound block verify services, occasionally encounter errors, and handle them correctly

Since most of our error handling doesn't consider the specific type of error, this should be enough to show that nullifier errors are handled correctly.

Is this related to #6338 ? Or a different audit issue

Is this related to #6338 ? Or a different audit issue

This is a different issue that wasn't in the findings but was brought up during the audit sync.

@arya2 we think you might be the best person to do this ticket?

mempool_request_with_immature_spent_is_rejected checks that ValidateContextErrors are converted correctly, though it doesn't check downcasts from BoxErrors so we may want to add a test for that.
https://github.com/ZcashFoundation/zebra/blob/8a9113c4e2fc3cd551f5cf94c3b8177591a54eae/zebra-consensus/src/transaction/tests.rs#L611-L662

though it doesn't check downcasts from BoxErrors so we may want to add a test for that.

Seems like it might be a low priority, but that's @dconnolly's call.

@arya2 what's the complexity of this fix and what would be the effort required to address this audit comment? Do we need to review the issue estimate? Feel free to update it to what you think is a more accurate reflection of the size of this issue.

what's the complexity of this fix and what would be the effort required to address this audit comment? Do we need to review the issue estimate? Feel free to update it to what you think is a more accurate reflection of the size of this issue.

The estimate is still accurate.


Async/await support

Since async/await support in Node is on the horizon -- when will DocumentDB support it? Since async/await is just sugar on top of promises, I guess the real question is -- will DocumentDB support promises instead of callbacks?

DocumentDB has a promises based npm package as well.
GitHub repo : https://github.com/Azure/azure-documentdb-node-q
npm package : https://www.npmjs.com/package/documentdb-q-promises
Let me know if that works for you.
cc: @moderakh

Ah, didn't know that. What's the reasoning behind going with Q instead of native promises, backwards compatibility? Do you know if Q is fully compatible and will work with async/await?

As an alternative, I wrote a wrapper around the Node.js API specifically for use with TypeScript and/or ES6 (Babel, ES2015, whatever). Link: https://github.com/jcormont/documentdb-typescript
This does support async/await, as well as incorporating a more streamlined API that is somewhat inspired by the object oriented .NET API. Just take a look at the examples in the readme file. Perhaps this is closer to what you are looking for!

Looks nice @jcormont!


Deleting events causes announcement generation to fail

seems transient thought

Epiphany: It's because deleting an event doesn't cause the cache to reset which it should.


[bugfix] allow jsdoc author tag (#4849)

PR checklist

[x] Addresses an existing issue: fixes #4849
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
Allows usage of JSDoc @author tag.
CHANGELOG.md entry:
[bugfix] no-redundant-jsdoc: allow @author tag

Thanks for your interest in palantir/tslint, @petersandor! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

The build with typescript@next is now failing since it now uses the 3.7 beta that was released yesterday.

We can ignore the typescript@next failures in this PR. They're unrelated: #4863.

thanks for the PR @petersandor!

Can this be backported to 5.x?


chore: fix absolute routes

Description
Prevents the issue of the base URL having a trailing slash.
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update
[ ] This change requires installing new dependencies

nice!


Documentation for Example App credentials/ setup

The credentials for example apps could be explained in more detail to help people get up to speed with using the example app without facing errors/ build failures.

Users need to set a MAPBOX_DOWNLOADS_TOKEN but this is not documented, so gradle doesn't work when I followed the readme instructions. This can be the same, but it can also be different to MAPBOX_ACCESS_TOKEN. Other questions arose too: Which scopes should I give to the downloads token? Will only Downloads:read work, or do I need navigation:download, vision:download, etc?

Which google maps API should I enable and use for GOOGLE_MAPS_API_KEY? I guess the Google geolocation API is the correct one, but there is also "Google Maps SDK for Android". I think it would be good to be clear which API needs to be enabled for that API key, giving minimum access possible.

We ask users to update gradle.properties with sensitive credentials. I believe this may put users at risk, they might accidentally commit gradle.properties because this is a file that is shared between developers. One suggestion I had would be to gitignore local.properties, and then load local.properties in gradle and ask users to specify credentials in this file instead. Here is an example to load from that file: https://stackoverflow.com/questions/43921078/android-reading-from-local-properties

I did manage to build it, but instead of 2 minutes it took me about 20 minutes. I'm happy to improve the documentation with my findings, if you don't mind 😃.

Thanks for picking this up! We definitely should clarify which scope we need for the token - this is a great catch, and something we should add to both this and to the main docs.
On the second point - we actually don't suggest to put the secrets in the local grade.properties, instead we suggest putting them in the gradle.properties in the home folder which I don't think many users are likely to check into git repo:

The following secrets need to be injected into Gradle by either storing them in ~/.gradle/gradle.properties,


Allow doppler websocker server host to be configured

This PR allows the doppler service to be run on a specific interface (i.e. localhost)
@mdellilo @aemengo

@mdelillo this is scheduled and should be picked up shortly.

@mdelillo your changes have been merged in.


Question from the optimizing Scrapy with Playwright for Concurrent Page Handling and Response Capture with async_generator TypeError

Context
I am developing a scraper using Scrapy and Playwright in Python, where I need to open multiple web pages with different profiles (e.g., "EXTRACTOR 1", "EXTRACTOR 2", "EXTRACTOR 3") to expedite the scraping process. Each page is managed with its own logic and is expected to capture specific responses.
Current Workflow

Within a Scrapy spider, I'm using Playwright's async_api to initiate new browser pages in a given context. This involves dynamically opening multiple pages for different profiles, such as "PAGE 1", "PAGE 2", and "PAGE 3".
For each of these new pages, I navigate to a specific URL using Playwright's goto method and perform profile-specific adjustments.
On each page, I implement a handle_request_logic method. This method sets up a request listener using page.on("request", self.handle_request) and triggers certain actions on the page, like clicking and waiting for specific elements.
In the handle_request_logic method, I use Scrapy's FormRequest to send modified POST requests based on the page interactions. The goal is to capture and process the responses using Scrapy’s callback mechanism.

Problem
I am encountering difficulties in integrating Playwright's asynchronous page handling with Scrapy's workflow. Specifically, I am experiencing a TypeError: object async_generator can't be used in 'await' expression when trying to manage multiple pages concurrently with Playwright within a Scrapy spider and ensuring that responses captured by scrapy.FormRequest are processed correctly.
Question
What is the recommended approach for managing multiple pages concurrently with Playwright within a Scrapy spider, ensuring that the responses captured through scrapy.FormRequest are appropriately processed?
Simplified Code Example
async def parse_coupon(self, response):
    profiles = ["PAGE 1", "PAGE 2", "PAGE 3"]
    context = response.meta["playwright_page"].context
    for profile_name in profiles:
        page = await context.new_page()
        await page.goto("https://example.com")
        await self.handle_request_logic(page, profile_name) # I hope to obtain items from each page, but I'm not sure how to do it.
        await page.close()

async def handle_request_logic(self, page, profile_name):
    # Page setup and logic
    # ...
    # yield scrapy.FormRequest(...)

I appreciate any guidance or suggestions for improving this workflow and ensuring effective and efficient scraping.

Hi, I wanted to share a solution approach I've found for the problem I described. While I'm not entirely certain if it's the best method, it seems to work effectively in my context.
Solution I Found
To handle multiple asynchronous Playwright pages within a Scrapy spider, I devised a method where I accumulate a list of scrapy.FormRequest objects in the handle_request_logic method. This process involves collecting all necessary requests per page and then iterating over this list to yield each request to Scrapy's engine. Here's how I implemented it:
async def parse_coupon(self, response):
    profiles = ["PAGE 1", "PAGE 2", "PAGE 3"]
    context = response.meta["playwright_page"].context
    all_requests = []
    for profile_name in profiles:
        page = await context.new_page()
        await page.goto("https://example.com")
        requests = await self.handle_request_logic(page, profile_name)
        all_requests.extend(requests)
        await page.close()

    for request in all_requests:
        yield request

async def handle_request_logic(self, page, profile_name):
    # Page setup and logic
    requests = []
    # ... logic to create scrapy.FormRequest ...
    requests.append(scrapy.FormRequest(...))
    return requests

This approach seems to resolve the issues I was facing, particularly the async_generator error, and integrates well with Scrapy's processing workflow.
Seeking Feedback
While this method works for my project, I'm not sure if it's the optimal solution. I'm posting it here in case it helps others facing similar challenges, and I would greatly appreciate any feedback, suggestions, or alternative approaches that might be more efficient or effective.
Thanks for any insights you can provide!

Sounds right. I think the parse_coupon callback could be simplified a bit by letting scrapy-playwright handle the page creation without accessing the context directly. You could also just return the request list, no need to iterate over it.
The following might not match your existing code entirely, but could serve to illustrate:
class MySpider(scrapy.Spider):
    def previous_callback(self, response):
        profiles = ["PAGE 1", "PAGE 2", "PAGE 3"]
        for profile_name in profiles:
            yield scrapy.Request(
                url="https://example.com",
                callback=self.parse_coupon,
                meta={"playwright": True, "playwright_include_page": True},
                cb_kwargs={"profile_name": profile_name},
            )

    async def parse_coupon(self, response, profile_name):
        page = response.meta["playwright_page"]
        requests = await self.handle_request_logic(page, profile_name)
        await page.close()
        return requests

    async def handle_request_logic(self, page, profile_name):
        ...

In your example, wouldn't executing the previous_callback function result in only the first profile being run? Since when performing the for loop, it would return the value of the first yield because it's not asynchronous, right?

No, the three requests are enqueued sequentially in the for loop, but their downloading and processing is handled asynchronously by Scrapy.

I understand that this should work ... but in my case that did not work :cry:
Thank you very much

One thing that could be causing only the first request to work is Scrapy's duplicate filtering, see https://docs.scrapy.org/en/latest/topics/settings.html#dupefilter-class

OMG!!! It was that! Thank you so much! ❤ ❤ ❤

Glad to hear, I edited my original comment to prevent future confusion. Should this issue be closed then?

yes, thank you very much for the help


[Bug]: Shop page bug!

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When click to the shop page it is not working keeps on loading. Below is the Screenshot provided:

Expected behavior
The page should be loaded and visible when clicked.
Add ScreenShots
No response
On which device are you experiencing this bug?
Windows
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Is there an existing issue for this?

[x]  I have searched the existing issues

Describe the bug
When click to the shop page it is not working keeps on loading. Below is the Screenshot provided:
Expected behavior
The page should be loaded and visible when clicked.
Add ScreenShots
No response
On which device are you experiencing this bug?
Windows
Record

[x]  I have read the Contributing Guidelines
[x]  I'm a GSSOC'24 contributor
[x]  I have starred the repository

Hello @KAmaL-senpai, Go for it
All the Best


Added proposal table

Follow up PR to devEnvironment, this creates a new table (hardcoded for now) for discussion

How can we ensure that the proposals repo remains the source of truth? In other words, how can we make sure the website is never out of sync with the repo?

@ljharb We'll have to manually maintain data here, in order to give more explanations of features than we have in the proposals repository, but also more concise than you can find from current READMEs.

I wonder if we could have a build process in this repo that only ends up rendering proposals’ extra info when the proposal is in the main repo, and that renders a placeholder section for proposals in the main repo that lack extra info here?

@ljharb We were thinking about making this dynamic as well, once we have a better idea of what is needed. For the moment we are just seeing what this looks like as a proof of concept and help us figure out exactly what information we need, and where best to store that information

I have no bandwidth due to a few confs. Should be better next week.

Closing this, as it has been rebased and was necessary for @Regaddi 's pr -- @xtuc feel free to branch off of master. if there are any issues with conflicts let me know


Bump minimum terraform version for modules to 3.116

Is your feature request related to a problem?
The current version does not meet the min requirements for the code we use within vmseries module.
Describe the solution you'd like
Bump minimul required azrerm provider version.
Describe alternatives you've considered.
No response
Additional context
No response

We'll bump straight to AzureRM v4, PR #86 ready for review.


To able to write own serialize and deserialize for relay id.

Product
Hot Chocolate
Is your feature request related to a problem?
Can we add a custom serialize and deserialize function for the ID in the relay? This feature can be useful if you want to make the ID in the URL more user-friendly and SEO-friendly. For example, instead of having a URL like /organization/T3JnYW5pemF0aW9uCmRhbmd1bGFy, you can have a URL like /organization/angular. To do this, you would need to create a custom function that can take the ID and convert it to a more readable format and another function that can take the readable format and convert it back to the original ID. This way, you can use the custom functions in your front-end code to handle the ID encoding and decoding.
The solution you'd like
A class we can pass will encode and decode relay id.

This is already implemented:
services
    .AddGraphQLServer()
    .AddIdSerializer<YourIdSerializer>()
    ....


Add always_flush option

Add the option to flush the IOStream after printing the message.

Thanks.
Can you describe the situation where you needed this? It would be nice to add some documentation for this option and as part of that, describe in which circumstances people might want it.

This option is useful, if you run your code as batch job on a cluster, where your output is redirected to a file. Without flushing, the output may appear in these files just when your program is finished. See e.g https://discourse.julialang.org/t/workers-do-not-show-any-io-when-using-a-cluster/40955


[ISSUE #6180] Polish the issue_template

Make sure set the target branch to develop
What is the purpose of the change

fix #6180
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #6181 (3312b5a) into develop (7cfffe7) will increase coverage by 0.04%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #6181      +/-   ##
=============================================
+ Coverage      43.18%   43.23%   +0.04%
- Complexity      8852     8857       +5
=============================================
  Files           1094     1094
  Lines          77153    77153
  Branches       10063    10063
=============================================
+ Hits           33321    33359      +38
+ Misses         39665    39626      -39
- Partials        4167     4168       +1

Impacted Files
Coverage Δ

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
80.35% <0.00%> (-5.36%)
:arrow_down:

...ketmq/proxy/grpc/v2/channel/GrpcClientChannel.java
49.07% <0.00%> (-2.78%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
82.21% <0.00%> (-2.41%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
33.42% <0.00%> (-1.66%)
:arrow_down:

.../rocketmq/broker/filter/ConsumerFilterManager.java
72.19% <0.00%> (-0.90%)
:arrow_down:

...che/rocketmq/acl/plain/PlainPermissionManager.java
78.22% <0.00%> (-0.54%)
:arrow_down:

...mq/client/impl/producer/DefaultMQProducerImpl.java
46.22% <0.00%> (-0.25%)
:arrow_down:

...main/java/org/apache/rocketmq/store/CommitLog.java
65.13% <0.00%> (-0.11%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (ø)

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Instantiate Per Codeblock Refresh Disabling

Simple solution and only applicable to dataviewjs.
Allows for a comment within the codeblock—disable-refresh—to disable automatic refreshing for select dataviewjs queries. Thus, preventing flashing of rendered diagrams yet still allowing the functionality of automatic view refreshing.
Example:
```dataviewjs
// disable-refresh
await dv.view("diagram", 0)
```

Fixes: https://github.com/blacksmithgu/obsidian-dataview/issues/2396

I think this is a great solution but disable-refresh is somewhat generic tag and so it has a nonzero potential for accidental enabling. Have you considered calling it something like "dataview-disable-refresh"?

@reply2za For sure, in the original bug ticket my initial thought was to make it similar to typescripts ignore, so potentially changing this to @dataviewjs-disable-refresh which then removes the potential for accidental enabling. I'll go ahead and make this change.


Removed inline CSS

Date: 17 Oct 2024
Developer Name: Uttam Rao

Issue Ticket Number
#458
Description
Removed inline CSS
Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[ ] Yes
[x] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots

Test Coverage
Screenshot Attached
Additional Notes
NA

Please break the PR as I cannot review too big a Pr

@iUttamRao this is a very big pr please cherry pick the commits and break it into smaller prs


Failure has occurred while loading a type.

Using Chromapper version 0.10.763 (can't find any download option for any other version of release 10)
This happens when I click "audio" in the editor.
[Log] Failure has occurred while loading a type.
[Exception] [System.TypeLoadException] Failure has occurred while loading a type.
at Automapper.Methods.Onset.GetMap (System.String audioPath, System.Single bpm) [0x001e4] in <69b56cfb71a54a01b3ef5134118325ef>:0
at Automapper.Automapper.Audio () [0x0000a] in <69b56cfb71a54a01b3ef5134118325ef>:0
at Automapper.UI.UI.b__4_20 () [0x00000] in <69b56cfb71a54a01b3ef5134118325ef>:0
at UnityEngine.Events.InvokableCall.Invoke () [0x00010] in C:\build\output\unity\unity\Runtime\Export\UnityEvent\UnityEvent.cs:178
at UnityEngine.Events.UnityEvent.Invoke () [0x00022] in C:\build\output\unity\unity\Runtime\Export\UnityEvent\UnityEvent\UnityEvent_0.cs:58
at UnityEngine.UI.Button.Press () [0x00027] in /home/jenkins/agent/workspace/ChroMapper/checkout/Library/PackageCache/com.unity.ugui@1.0.0/Runtime/UI/Core/Button.cs:70
at UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) [0x00010] in /home/jenkins/agent/workspace/ChroMapper/checkout/Library/PackageCache/com.unity.ugui@1.0.0/Runtime/UI/Core/Button.cs:114
at UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) [0x00001] in /home/jenkins/agent/workspace/ChroMapper/checkout/Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:57
at UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) [0x0006f] in /home/jenkins/agent/workspace/ChroMapper/checkout/Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:272

I'm on CM stable v0.10.763, latest release <IP_ADDRESS> work fine for me. Try to clean up your plugin folder, and only have Automapper.dll inside it to see if it work properly. Also, make sure that your CM folder is somewhere that can't create issue, like on the Desktop for example.

Should be fixed now, note that the plugin only work with OGG right now.


Get corrections in JSON output

Hi! We're working on a tool named coala which is designed to provide good user interaction to any static code analysis tool. That way people writing code analysis can concentrate on just doing that and not rewriting user interaction again and again for each language (coala is language independent), OTOH the user only needs to learn one tool.
Because our goal is to reduce redundancy and not create some, we're integrating other tools like rubocop so they feel native for coala users.
For that we'd love it if we could get the fixes associated with the issues. In our JSON output we e.g. ship a unified diff with it: http://pastebin.com/RSArMFvS , other linters just give something like a start and end position and a replacement.
Would it be possible for you to ship the fixes with your JSON output?
(Video demo of coala at https://asciinema.org/a/37941 , if you have more questions just ask me or drop by at https://gitter.im/coala-analyzer/coala )

RC allows you to create a custom "formatter" class, which controls the CLI output. It sounds like writing your own formatter is what is needed for your use case.

@alexdowad looking at http://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Offense it doesn't seem like a custom formatter would even have access to that information

@sils1297 OK, you are right.
One (big) open project for RC is to provide an API which can be used to invoke it programmatically. When that is done, perhaps we can pass back the information you are asking for.

IMO if you make that information available it should be contained in the JSON output right away, as people usually request JSON when they want a structured dump of all the information for the results.

There hasn't been much activity on this ticket and our Core Team is spread too thin on so many tasks, so I'll just close it for the sake of having a cleaner lists of tasks to focus on. We'd gladly review a PR, but it's unlikely that'd we're going to tackle this ourselves in the foreseeable future.


docs: youtube guide

Hey this is the youtube guide

Thanks for the review, will change it later. Will merge this for now as needed


Changes in footer

Adding more resources as well as social media icons.
I work on this issue under LGM-SOC'21 participant..please assigned me

I would like to work on this issue under LCM Soc21 please assign to me.

Check #17


remove unused methods

line details : https://github.com/libuparayil/networking-huawei/blob/4c808cd1b60540dc3bfbfcce6e40694ab9f5542c/networking_huawei/common/ac/client/restclient.py#L181
below methods are not used  please check and remove
# Check whether the http response is failure ir not
def httpError(self):
    pass

# http Operationtimeout
def httpTimeout(self):
    pass

# Post the message
def post(self):
    pass

ok


[Question] Why the rule Identifier isn't showing in some cases?

I know I can desactivate ordering rules (or other kind of rules) to avoid warnings (SA1201     ElementsMustAppearInTheCorrectOrder in this case):

But why it's not showing the rule identifier? I'm guessing that StyleCopAnalyzer doesn't want to mess with my code that much or something like that.
(I'm sorry if this is not the place for this kind of question)

The message displayed is the formatted message that we pass to Roslyn. It would be possible to add the identifier to the formatted message. Marking the issue with the enhancement / discussion tag to see if there is consensus about doing this.

@vweijsters We should do the same thing the IDE does for all other messages, which means two things:

We shouldn't manually add the ID ourselves, as the compiler doesn't do that
If the situation changes, it would change within Roslyn itself and not within this project specifically

Here is the issue tracker for the project which is responsible for this situation:
https://github.com/dotnet/roslyn/issues


ctrl+d shortcut being blocked by continue

Describe the bug
Whenever I want to use vscode's ctrl+d shortcut, continue debug opens. This is not something I want.
Expected behavior
ctrl+d is one of the most commonly used vscode shortcuts. You cannot own this as an extension. With this extension installed, I cannot use the editor as I am used to.
Screenshots

@aykutkardas You're right, this has to change—I just pushed a new version, will be ready in ~15min. I changed the shortcut to ctrl+shift+r. This in my understanding is a much more acceptable keybinding?

Yes, great choice ⚡

Awesome, thx for the feedback on this! Shortcuts that are both available and ergonomic are few and far between. Closing this issue, but if problems with other shortcuts arise, you're welcome to open another


Fix notebook extension aliasing

This fixes NameError: name 'PrintReportNotebook' is not defined error on import.
This situation may happen when ipykernel is used directly without installing ipywidgets (e.g., #%% cells by VS Code's Python extension).

/test