Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Minor Changes

aaeb7ecf3: When newer documentation available but not built, show older documentation while async building newer
TechDocs backend: /sync endpoint added to support above, returns immediate success if docs don't need a build, returns delayed success after build if needed
TechDocs backend: /docs endpoint removed as frontend can directly request to techdocs.storageUrl or /static/docs
3139f83af: Add sticky sidebars and footer navigation links to TechDocs Reader

Patch Changes

ea9d977e7: Introduce workaround for admonition icons of MkDocs.
2aab54319: TechDocs: links at sidebar and bottom reset scroll position to top
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Minor Changes

aaeb7ecf3: When newer documentation available but not built, show older documentation while async building newer
TechDocs backend: /sync endpoint added to support above, returns immediate success if docs don't need a build, returns delayed success after build if needed
TechDocs backend: /docs endpoint removed as frontend can directly request to techdocs.storageUrl or /static/docs

<EMAIL_ADDRESS>Patch Changes

4d248725e: Make the backend plugin template use the correct latest version of express-promise-router

<EMAIL_ADDRESS>Patch Changes

fcc3ada24: Reuse ResponseErrorList for non ResponseErrors
4618774ff: Changed color for Add Item, Support & Choose buttons with low contrast/readability in dark mode
df59930b3: Fix PropTypes error with OverflowTooltip component
Updated dependencies [76deafd31]
Updated dependencies [01ccef4c7]
Updated dependencies [4618774ff]

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

76deafd31: Changed the signature of createRoutableExtension to include null
01ccef4c7: Introduce useRouteRefParams to core-api to retrieve typed route parameters.
Updated dependencies [4618774ff]

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

2c525f85e: (fix) Adds locationAnalyzer to default-app template
The locationAnalyzer was missing from the default-app template.
This resulted in 404 errors in newly bootstrapped backstage applications,
when adding components without configuration.
To fix this in an existing backstage application, the locationAnalyzer needs
to be carried from the builder to the router in the
packages/backend/src/plugins/catalog.ts file.
   const builder = new CatalogBuilder(env);
   const {
     entitiesCatalog,
     locationsCatalog,
     higherOrderOperation,
+    locationAnalyzer,
   } = await builder.build();
   // ...
   return await createRouter({
     entitiesCatalog,
     locationsCatalog,
     higherOrderOperation,
+    locationAnalyzer,
     logger: env.logger,
   });

f88fe9dd9: Adds plugin-org and more capability to the default EntityPage to display Users, Groups and Systems.
To update an existing application, add the org plugin:
cd packages/app
yarn add @backstage/plugin-org

Then add the example systems locations to your app-config.yaml:
catalog:
  rules:
-    - allow: [Component, API, Group, User, Template, Location]
+    - allow: [Component, System, API, Group, User, Template, Location]
  locations:
    # Backstage example components
    - type: url
      target: https://github.com/backstage/backstage/blob/master/packages/catalog-model/examples/all-components.yaml

+    # Backstage example systems
+    - type: url
+      target: https://github.com/backstage/backstage/blob/master/packages/catalog-model/examples/all-systems.yaml
+
    # Backstage example APIs

Additionally, the default app sidebar was updated to parity with the Backstage
repo. You can see these changes in the template
App.tsx
referencing a new Root component.
Finally, compare your packages/app/src/components/catalog/EntityPage.tsx to
EntityPage
from the @backstage/create-app default template to pick up additional
changes there.

4d248725e: Update the create-app template to use the correct latest version of express-promise-router.
To apply the same change in your own repository, update all of your repo's dependencies on express-promise-router to "^4.1.0".

Updated dependencies [9f2e51e89]

Updated dependencies [01ccef4c7]

Updated dependencies [4d248725e]

Updated dependencies [aaeb7ecf3]

Updated dependencies [449776cd6]

Updated dependencies [91e87c055]

Updated dependencies [ea9d977e7]

Updated dependencies [fcc3ada24]

Updated dependencies [687f066e1]

Updated dependencies [2aab54319]

Updated dependencies [113d3d59e]

Updated dependencies [f47e11427]

Updated dependencies [4618774ff]

Updated dependencies [3139f83af]

Updated dependencies [c862b3f36]

Updated dependencies [4d248725e]

Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

4618774ff: Changed color for Add Item, Support & Choose buttons with low contrast/readability in dark mode

<EMAIL_ADDRESS>Patch Changes

449776cd6: The auth config types now properly accept any declared auth environment. Previously only development was accepted.
The audience configuration is no longer required for GitLab auth; this will default to https://gitlab.com

<EMAIL_ADDRESS>Patch Changes

687f066e1: Update badge dialog formatting
4d248725e: Get the current entity using useEntity from @backstage/plugin-catalog-react
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

4d248725e: Temporarily add UNSTABLE_extraContextMenuItems to the entity layout, so that we could detach the catalog plugin from the dependency on the badges plugin
687f066e1: Add icon for entity badge menu
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

f47e11427: Log how many repositories were actually matching in GithubDiscoveryProcessor

c862b3f36: Introduce pagination in the /entities catalog endpoint.
Pagination is requested using query parameters. Currently supported parameters, all optional, are:

limit - an integer number of entities to return, at most
offset - an integer number of entities to skip over at the start
after - an opaque string cursor as returned by a previous paginated request

Example request:
GET /entities?limit=100
Example response:
200 OK
Content-Type: application/json; charset=utf-8
Link: </entities?limit=100&after=eyJsaW1pdCI6Miwib2Zmc2V0IjoyfQ%3D%3D>; rel="next"
<more headers>

[{"metadata":{...

Note the Link header. It contains the URL (path and query part, relative to the catalog root) to use for requesting the next page.
It uses the after cursor to point out the end of the previous page. If the Link header is not present, there is no more data to read.
The current implementation is naive and encodes offset/limit in the cursor implementation, so it is not robust in the face of overlapping
changes to the catalog. This can be improved separately in the future without having to change the calling patterns.

Updated dependencies [4d248725e]

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

01ccef4c7: Introduce useRouteRefParams to core-api to retrieve typed route parameters.
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

d10ea17c9: fix missing type exports of public components
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

01ccef4c7: Introduce useRouteRefParams to core-api to retrieve typed route parameters.
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

01ccef4c7: Introduce useRouteRefParams to core-api to retrieve typed route parameters.
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

a2a3c7803: Bump @kubernetes/client-node from ^0.13.2 to ^0.14.0.
Updated dependencies [a2a3c7803]
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

a2a3c7803: Bump @kubernetes/client-node from ^0.13.2 to ^0.14.0.

<EMAIL_ADDRESS>Patch Changes

9f2e51e89: Fixes bug in the github:publish action causing repositories to be set as private even if the visibility is set to internal
91e87c055: Add inputs for action fetch:cookiecutter: copyWithoutRender, extensions, imageName
113d3d59e: Added a publish:file action to use for local development. The action is not installed by default.

<EMAIL_ADDRESS>Patch Changes

4d248725e: Bump to use the in-repo latest backend-common, and the correct version of express-promise-router

<EMAIL_ADDRESS>Patch Changes

4d248725e: Bump to use the in-repo latest backend-common, and the correct version of express-promise-router

<EMAIL_ADDRESS>Patch Changes

322e01a95: Support SonarQube project keys containing special characters like colons.
Updated dependencies [01ccef4c7]
Updated dependencies [fcc3ada24]
Updated dependencies [4618774ff]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

36d933ec5: Forward authorization on todo backend requests

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [a2a3c7803]
Updated dependencies [687f066e1]
Updated dependencies [d10ea17c9]
Updated dependencies [01ccef4c7]
Updated dependencies [4d248725e]
Updated dependencies [4d248725e]
Updated dependencies [aaeb7ecf3]
Updated dependencies [ea9d977e7]
Updated dependencies [fcc3ada24]
Updated dependencies [687f066e1]
Updated dependencies [2aab54319]
Updated dependencies [4618774ff]
Updated dependencies [3139f83af]
Updated dependencies [4d248725e]
Updated dependencies [df59930b3]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [a2a3c7803]
Updated dependencies [9f2e51e89]
Updated dependencies [4d248725e]
Updated dependencies [aaeb7ecf3]
Updated dependencies [449776cd6]
Updated dependencies [91e87c055]
Updated dependencies [36d933ec5]
Updated dependencies [113d3d59e]
Updated dependencies [f47e11427]
Updated dependencies [c862b3f36]

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@backstage/silver-lining @backstage/techdocs-core I can haz +1 here?

Tugboat has finished building the preview for this pull request!
Link:

https://pr5020-mj8g1fnqhvikzgzhriyz7ffo4iokjfsx.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/605cb9ee437dd1ad99f749dc

Tugboat has finished building the preview for this pull request!
Link:

https://pr5020-mj8g1fnqhvikzgzhriyz7ffo4iokjfsx.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/605cb9ee437dd1ad99f749dc

Tugboat has finished building the preview for this pull request!
Link:

https://pr5020-mj8g1fnqhvikzgzhriyz7ffo4iokjfsx.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/605cb9ee437dd1ad99f749dc


Update handlers.py

To execute job with user's environment variables.
Fixed error /bin/sh: 1: papermill: not found issue(https://github.com/tiburon-security/jupyterlab_scheduler/issues/2)

Thanks for the pull request. What OS were you using when you ran into the environment issue? I only tested on Ubuntu so I'm curious and using the official jupyterlab docker image.

I'm using official jupyterhub on ubuntu 20.04 without docker (installed with anaconda) :)


Feature: Webhook Notification

How about adding a simple Webhook Notification? Nothing fancy, just a URL which is requested with GET on a a "try once, fail/success once" basis. For safety, basic auth would't be hard to add as well.
This could trigger some external automation an event on a gateway, home automation or such.

yeah, its easy. we can do it.

I need this too.

this feature is released in v1.11.4

Cool, thanks a bunch @moonrailgun !!


Room floors should be dark when underground and bright when on the surface level.

Related to #12 and #16.

Almost done.
There are these "default" styled underground rooms we must still consider though. Perhaps re-randomize until we have no default style?

The "default" styled rooms were mostly due to a bug in color::shade_color() in Termin8or which is now fixed.


[freeCodeCamp] Add points badge

Adds a badge for points a user has on freeCodeCamp. (I'm tempted to say the badge should refer to challenges rather than points, but I'm not 100% sure if completing challenges is the only way to get points.)
I'm unsure about the use of the word "points" in this, as it is indeed "points", but the term is too abstract. Any advice or opinions?
As displayed on a user's profile:

As displayed in JSON:
"points": 934,

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @SethFalco!

  Generated by :no_entry_sign: dangerJS against 66c8efa4723e02921145b73f9d09e090f0be854d


Mixed managed virtual network and Azure virtual network information

[Enter feedback here]
The following step links to the instructions for using a workspace in a managed virtual network.

-> https://learn.microsoft.com/en-us/azure/machine-learning/tutorial-create-secure-workspace?view=azureml-api-2

However, the steps below are for an Azure virtual network and are inconsistent with the step described above.

-> https://learn.microsoft.com/en-us/azure/machine-learning/how-to-secure-training-vnet?view=azureml-api-2&tabs=cli%2Crequired

As a rule of thumb, I think it's only supported for use in Azure virtual networks, not managed virtual networks, but please fix.
Probably the link in the first image is incorrect, and the correct one is as follows?
wrong:
https://learn.microsoft.com/en-us/azure/machine-learning/tutorial-create-secure-workspace?view=azureml-api-2
correct:
https://learn.microsoft.com/en-us/azure/machine-learning/tutorial-create-secure-workspace-vnet?view=azureml-api-2

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

ID: 540ed864-89a2-83b0-597e-5e7772fd2d71
Version Independent ID: 225d44b2-09b5-f3b6-0299-93f91fb3799a
Content: Network isolation in batch endpoints - Azure Machine Learning
Content Source: articles/machine-learning/how-to-secure-batch-endpoint.md
Service: machine-learning
Sub-service: core
GitHub Login: @santiagxf
Microsoft Alias: fasantia

@damochiz
Thanks for your feedback! We will investigate and update as appropriate.

@damochiz
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


Fix neuclir queries

Remove unecessary quotes from neuclir queries

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

Coverage data is based on head (65e6ee5) compared to base (dc0b7ff).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2054   +/-   ##
=========================================
  Coverage     58.93%   58.93%
  Complexity     1135     1135
=========================================
  Files           187      187
  Lines         10960    10960
  Branches       1487     1487
=========================================
  Hits           6459     6459
  Misses         4011     4011
  Partials        490      490

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

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

I've added separately to my own PR.


Fix dropdown potentially mutating transforms when dealing with async bindable changes

Manifested in the following case (TestSceneFullscreen):
Unhandled exception. osu.Framework.Graphics.Drawable+InvalidThreadForMutationException: Cannot mutate the Transforms on a Loaded Drawable while not on the update thread. Consider using Schedule to schedule the mutation operation.
   at osu.Framework.Graphics.Drawable.EnsureMutationAllowed(String action) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/Drawable.cs:line 2618
   at osu.Framework.Graphics.Drawable.EnsureTransformMutationAllowed() in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/Drawable.cs:line 2589
   at osu.Framework.Graphics.Transforms.Transformable.AddTransform(Transform transform, Nullable`1 customTransformID) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/Transforms/Transformable.cs:line 354
   at osu.Framework.Graphics.TransformableExtensions.TransformTo[TThis](TThis t, Transform transform) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/TransformableExtensions.cs:line 66
   at osu.Framework.Graphics.TransformableExtensions.TransformTo[TThis,TValue,TEasing](TThis t, String propertyOrFieldName, TValue newValue, Double duration, TEasing& easing, String grouping) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/TransformableExtensions.cs:line 53
   at osu.Framework.Graphics.TransformableExtensions.FadeColour[T,TEasing](T drawable, ColourInfo newColour, Double duration, TEasing& easing) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/TransformableExtensions.cs:line 539
   at osu.Framework.Graphics.TransformableExtensions.FadeColour[T](T drawable, ColourInfo newColour, Double duration, Easing easing) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/TransformableExtensions.cs:line 313
   at osu.Framework.Graphics.UserInterface.Dropdown`1.DropdownMenu.DrawableDropdownMenuItem.UpdateBackgroundColour() in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 508
   at osu.Framework.Graphics.UserInterface.Dropdown`1.DropdownMenu.DrawableDropdownMenuItem.OnSelectChange() in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 502
   at osu.Framework.Graphics.UserInterface.Dropdown`1.DropdownMenu.DrawableDropdownMenuItem.set_IsSelected(Boolean value) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 449
   at osu.Framework.Graphics.UserInterface.Dropdown`1.DropdownMenu.<>c__DisplayClass12_0.<SelectItem>b__0(DrawableDropdownMenuItem c) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 382
   at osu.Framework.Extensions.IEnumerableExtensions.EnumerableExtensions.ForEach[T](IEnumerable`1 collection, Action`1 action) in /Users/dean/Projects/osu-framework/osu.Framework/Extensions/IEnumerableExtensions/EnumerableExtensions.cs:line 23
   at osu.Framework.Graphics.UserInterface.Dropdown`1.DropdownMenu.SelectItem(DropdownMenuItem`1 item) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 380
   at osu.Framework.Graphics.UserInterface.Dropdown`1.selectionChanged(ValueChangedEvent`1 args) in /Users/dean/Projects/osu-framework/osu.Framework/Graphics/UserInterface/Dropdown.cs:line 288
   at osu.Framework.Bindables.Bindable`1.TriggerValueChange(T previousValue, Bindable`1 source, Boolean propagateToBindings, Boolean bypassChecks) in /Users/dean/Projects/osu-framework/osu.Framework/Bindables/Bindable.cs:line 289
   at osu.Framework.Bindables.Bindable`1.SetValue(T previousValue, T value, Boolean bypassChecks, Bindable`1 source) in /Users/dean/Projects/osu-framework/osu.Framework/Bindables/Bindable.cs:line 102
   at osu.Framework.Bindables.Bindable`1.TriggerValueChange(T previousValue, Bindable`1 source, Boolean propagateToBindings, Boolean bypassChecks) in /Users/dean/Projects/osu-framework/osu.Framework/Bindables/Bindable.cs:line 284
   at osu.Framework.Bindables.Bindable`1.SetValue(T previousValue, T value, Boolean bypassChecks, Bindable`1 source) in /Users/dean/Projects/osu-framework/osu.Framework/Bindables/Bindable.cs:line 102
   at osu.Framework.Bindables.Bindable`1.set_Value(T value) in /Users/dean/Projects/osu-framework/osu.Framework/Bindables/Bindable.cs:line 95
   at osu.Framework.Platform.SDL2DesktopWindow.updateWindowSpecifics() in /Users/dean/Projects/osu-framework/osu.Framework/Platform/SDL2DesktopWindow.cs:line 1018

Not sure if this fix will be accepted, but the alternative is ensuring that all input bindables for Current are only updated on the update thread. I'm not sure if this is a good restriction to have in place.

Not too sure about this one, for two reasons:

While framework generally didn't do this, game-side we have quite a few cases wherein bindable sets are protected with schedules to ensure they are only ever fired from the update thread. And sure, they're more local, but I presume this case is associated with CurrentDisplayBindable, which is something that I feel is likely to see much use on a framework level and as such it probably makes sense to schedule writes to it too, at which point this diff becomes no longer required until this comes up again, at which point we can re-evaluate.
If the above fails to convince and this is deemed the right way to go - any reason to do Schedule() rather than Scheduler.AddOnce()?

To confirm, your thinking is that dropdowns should not be protected and rather the input bindable should already be update-thread-safe in all usages?
The reason I didn't do the scheduling is that right now the SDL class has no knowledge of the update thread, so this would be a completely new flow. Also it could probably bubble out to more than this one bindable, and in a scenario where there's already one or two levels of schedule/queueing logic. not 100%.

Not too sure about this one, for two reasons:

While framework generally didn't do this, game-side we have quite a few cases wherein bindable sets are protected with schedules to ensure they are only ever fired from the update thread. And sure, they're more local, but I presume this case is associated with CurrentDisplayBindable, which is something that I feel is likely to see much use on a framework level and as such it probably makes sense to schedule writes to it too, at which point this diff becomes no longer required until this comes up again, at which point we can re-evaluate.

If the above fails to convince and this is deemed the right way to go - any reason to do Schedule() rather than Scheduler.AddOnce()?

I think AddOnce will work.

Upon having a short look at how we have things structured in framework I'm going to back down on the first point because it's not going to be easy to marshal that bindable's updates onto the update thread. And, also, a similar thing was done in the past for slider bars (#4458), so I don't really seem to have much of a leg to stand on in this regard.
As for the latter I'll push a change to use AddOnce() to speed things up.


Composite key with Alias does not works

Environment

Version: v0.9.12
Hardware: same as #51
keymap.rb

puts "== DEBUG PAD =="

# Initialize a Keyboard
kbd = Keyboard.new

# Initialize GPIO assign
kbd.init_pins(
  [ 27 ], # row0, row1,... respectively
  [ 6,7,8,9 ]  # col0, col1,... respectively
)

kbd.add_layer :default, %i(
  KC_A SELECT_ALL BACK_LONG BACK_SHORT
)

kbd.define_composite_key :SELECT_ALL, %i(KC_LGUI KC_A)
kbd.define_composite_key :BACK_LONG,  %i(KC_LGUI KC_LBRACKET)
kbd.define_composite_key :BACK_SHORT, %i(KC_LGUI KC_LBRC)

kbd.start!

How to reproduce

tap BACK_LONG => It works 🙆♂
tap BACK_SHORT => Nothing reported 🤔

When tap BACK_SHORT, errors was printed.
debug print:
Suspending keymap.
Trying to reload keymap.
keymap.rb found.
== DEBUG PAD ==
Initializing Keyboard.
Initializing GPIO.
Starting keyboard task ...
Undefined local variable or method '+' for NilClass
Undefined local variable or method '*' for NilCUndefined local variable or method '+' for NilClass
Undefined local variable or method '*' for NilCUndefined local variable or method '+' for NilClass
Undefined local variable or method '*' for NilCUndefined local variable or method '+' for NilClass
Undefined local variable or method '*' for NilCUndefined local variable or method '+' for NilClass
Undefined local variable or method '*' for NilC

@arai-ta Thanks, I reproduce that. Will fix.
BTW, what is the expectation of the BACK_LONG shortcut?
Is it

command + 左角かっこ ([)：前のフォルダに移動します。

which is described in https://support.apple.com/ja-jp/HT201236 ?
(I'm not using a mac)


A sentence in the pricing page of Private DNS is wrong.

I found a sentence which has "Managed Preview" in public document for Private DNS pricing. I know Private DNS has already been GA now. Could you revise that document ?
https://azure.microsoft.com/en-us/pricing/details/dns/
Zones can be either public or private, where Private DNS Zones (in Managed Preview) are only visible to VMs that are in your virtual network.

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

ID: 7d766d17-0650-407e-1aed-d94966ea4932
Version Independent ID: 0254b3d9-adaf-8292-c3d3-93cb665b7504
Content: What is Azure Private DNS?
Content Source: articles/dns/private-dns-overview.md
Service: dns
GitHub Login: @rohinkoul
Microsoft Alias: rohink

@hisasima , Thank you for your feedback. This GitHub issue is used to report issues which is related to Azure Docs repo.
But Thank you for letting us know. I will work internally to get the pricing page updated.
@rohinkoul , Can you please take a look ?

Thanks for pointing this out.
<EMAIL_ADDRESS>Could you help with fixing this. Private DNS is GA so we should remove “managed preview” from the text.
<EMAIL_ADDRESS>From: hisasima<EMAIL_ADDRESS>Sent: Tuesday, March 31, 2020 12:15 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Rohin Koul<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [MicrosoftDocs/azure-docs] A sentence in the pricing page of Private DNS is wrong. (#51332)
I found a sentence which has "Managed Preview" in public document for Private DNS pricing. I know Private DNS has already been GA now. Could you revise that document ?
https://azure.microsoft.com/en-us/pricing/details/dns/https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fazure.microsoft.com%2Fen-us%2Fpricing%2Fdetails%2Fdns%2F&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946116770&sdata=TavSM8QU2%2Bt%2FS5fQVxGbjyNa7VLJ%2BwfDT6QBh7P8Tco%3D&reserved=0
Zones can be either public or private, where Private DNS Zones (in Managed Preview) are only visible to VMs that are in your virtual network.

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

ID: 7d766d17-0650-407e-1aed-d94966ea4932
Version Independent ID: 0254b3d9-adaf-8292-c3d3-93cb665b7504
Content: What is Azure Private DNS?https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fdns%2Fprivate-dns-overview&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946126765&sdata=U8XRnm0pQdgc4bHSrr8%2B3%2BN7AckrfaDckn%2BHBKtqks0%3D&reserved=0
Content Source: articles/dns/private-dns-overview.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fazure-docs%2Fblob%2Fmaster%2Farticles%2Fdns%2Fprivate-dns-overview.md&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946126765&sdata=zimwmT%2BbjlaBTZsZYwZD8mvTlzBRst5T6tRqWNo%2Fca4%3D&reserved=0
Service: dns
GitHub Login: @rohinkoulhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frohinkoul&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946136761&sdata=Yx2QB2uBCy3S5GXnWkUmDyJQeDodNM0Lc7O5cU7nGxo%3D&reserved=0
Microsoft Alias: rohink

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F51332&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946136761&sdata=nhaC0A4Uf0O7o03qTmOcElXL63S6LqSxWKPKZxwdu%2F0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK2EQHXT2SVXY4SJ23IUNA3RKGJ6XANCNFSM4LXJQ4CA&data=02|01|Rohin.Koul%40microsoft.com|031d81d244274a1772ad08d7d5433526|72f988bf86f141af91ab2d7cd011db47|1|0|637212356946146759&sdata=OTKVmLoQhPjRJr6K8rOfTpgz3vpzq4InnxVTYkiY4N0%3D&reserved=0.

@hisasima , Thank you for your feedback. The documentation will be updated shortly.


Add Jenkins configuration for rostam

This is just for testing triggering of builds for now. Configurations are still missing.

Jobs are now working by the way, but I only have one configuration in place. My plan is to fill the gaps between the configurations tested on Piz Daint, i.e. test:

clang 6 to 9 with one Boost version each
gcc 8 to 9 with one Boost version each
cuda 11 with gcc 10
all configurations with MPI (some with OpenMPI and some with MPICH) and release/debug
at least one config with cuda on and distributed runtime off (as in #4839).

@hkaiser, @biddisco, @rori, @sithhell please comment if you think there are other "special" configurations that you'd like to see tested.
I had in mind that these configurations would be enough even for master, and that they would be run for each and every PR (we'll have to see how rostam keeps up with that; these are more configurations per PR, but fewer than before on master).

retest

This is now ready to go (at least the basic functionality works, we may tweak it over time). The CUDA tests don't seem to run correctly, but that is for a separate PR, if we decide to fix it...

looks like the test is failing on daint -
https://cdash.cscs.ch/testDetails.php?test=24783653&build=118817
I wonder if I need to make sure that srun is used in place of mpi, or if the test is failing because the localities is wrong (it is using 1 instead of 2). I will check the test results of this PR when they come in ...

looks like the test is failing on daint -
https://cdash.cscs.ch/testDetails.php?test=24783653&build=118817
I wonder if I need to make sure that srun is used in place of mpi, or if the test is failing because the localities is wrong (it is using 1 instead of 2). I will check the test results of this PR when they come in ...

I think you meant rostam ;) I think this result is before your fix and the number of localities were incorrectly set to first 2 and then 1. Let's wait for the next round to see if that fixes it.

Looks like rostam might be only allocating one node, then the test asks for 2 and an error is being triggered. Not completely sure actually, but can add a flag to the rostam setup to allow 2 ranks per node just in case?

@biddisco I was just looking at this and ended up disabling the test for now so that we can get this in, but I'd appreciate your help in figuring out if it's the configuration or the test that's not working. We should still re-enable it before the release. The build does get two nodes allocated and all other "normal" MPI parcelport tests do pass (although they may still only be using one node, but definitely two ranks since some of the tests require that and would fail otherwise). Does the test pass for you locally? Actually, it would probably be enough to allocate a single node and use two ranks/localities on the node for tests to avoid reserving two full nodes, but that should be unrelated to this.

I don't think disabling the test is a good idea. On rostam I see different invocations (mpich/openmpi) and I'm not sure which one to actually look at to diagnose what's wrong. is it possible to run the tests by hand from the jenkins build dir and see what's up? (clearly no, if the test is disabled).

looking at this page
https://cdash.cscs.ch/testSummary.php?project=2&name=tests.unit.modules.async_mpi.mpi_ring_async_executor&date=2020-07-22
it would seem that hpx compiled with mpich (bottom of page) is/was passing the test, but with openmpi failing and when the number of ranks was set to 1, it failed with both.
All the gcc-10 builds appear to have passed, but builds with clang and earlier gcc failed. What I can't see is which ones explicitly enabled async_mpi and other options etc. the gcc-10 build were all launched with the correct N=2 mpi ranks, but others with the same hash appear to have N=1 - which may be the bug we have just addressed with the locality setting.
Enabling the test again would help  to spot a pattern of fails/passes

@biddisco found a problem in the async_mpi test. I'll merge this as it is and he will re-enable the test in a later PR.


feat(linter): eslint-plugin-jest: no-deprecated-function

Rule Detail: link

Wow, we only have one rule left to complete #492!

I'll try it in next two days.


[DOCS] Document Dijkstra Shortest Path

Documentation Dijkstra Shortest Path
The goal of this issue if the improve the documentation of an existing algorithm. This documentation should go into the algorithms section of our docusaurus docs.
More detail on how to build the documentation locally can be found on the wiki.
An existing documentation page exists, which should be extended. The page can be found under docs/docs/algorithms/shortest-path/dijkstra.md.
The corresponding algorithm is called dijkstra_shortest_path and can be found under include/graaflib/algorithm/shortest_path.h.
Documentation Contents
The documentation entry should adhere to the following template:
# [ALGORITHM_NAME]
- A short description of the algorithm, what does it do and in which use cases is it used.
- What are the limitations of the algorithm, does it work on both directed and undirected graphs? Does it consider edge weights, and if yes, does it support negative weights?
- Link to the wikipedia entry on the algorithm.

## Syntax
- A code block with the syntax of the algorithm. This should now include any javadoc comments.
- An explanation of the parameters (including template parameters) and the return type.

## (Optional) See also
- If there are similar algorithms, or we have a slightly different version of the same algorithm, you can link it here.
- If we have an example (found on the example page of the docs) which uses this algorithm, you can link it here.

Hi, I would like to help but I have seen there is already a "docs: add documentation for dijkstra (#66)" added. Does this make this task obsolete, or is it to further improve dijkstra.md? I tried to install yarl, unfortunately without success. Is this mandatory or is it just to edit/enhance the markdown file.

Hi @MichaeINeumann, thanks for your interest! You are correct, in the other PR we added a documentation entry for both versions of the Dijkstra algorithm. I did however forget to close this ticket 😬
If you are interested in picking up a documentation ticket, maybe one of the following could be of interest:

#60
#58
#57
#56

I would highly recommend installing yarn when working on the documentation, as it allows you to serve the documentation locally. This makes it very easy to observe the effect of your changes. If you have npm installed on your system you can install yarn using:
npm install --global yarn

On another note, I see that all algorithm tickets are assigned. I plan on adding more this afternoon in case you are interested.


Avoid exception when tearing down a router with string location

The current teardown static method of the Router component is:
teardown: function() {
              location.removeChangeListener(this.changeListener);
          }

The removeChangeListener function being absent on the String class, this will raise an exception when a router with "static" location is tear down. I'd suggest this simple change:
teardown: function() {
            if(location.removeChangeListener)
            {
              location.removeChangeListener(this.changeListener);
            }
          }

Would it be reasonable to add this check?
edit I have gotten myself out of the situation where this was a problem for me, but the issue might still stand anyway?

@olance This is fixed in master.

ok thanks :)


Upload path changes on 1.10.14

Hello,
We use mgob image 1.9.12-all and tried to upgrade to 1.10.14-all version and use Azure as storage backend.
with 1.9 version, backup were uploaded in storage/ path but after upgrade to 1.10, backup are in tmp/
Is it a normal behaviour ?
Thanks

this is related to this https://github.com/maxisam/mgob/issues/40
It is intended behavior. If you set a local path, it should move the backup to the path you set.

there are 2 things might relate to this

make sure retention > 0
https://github.com/maxisam/mgob/blob/366207a0954fbfdf684cf0726c8b14e71f94ee08/charts/mgob/values.yaml#L68

make usre storagePath for app config is pointing to /storage


System Alerts | Remove "time to disappear" progress bar |1|

When a system alert appears the "time to disappear" progress bar moves from right to left. We want to remove this bar.

 System Alerts | Remove "time to disappear" progress bar |1|


DE simulation

Adds all parts required for slugs to work.
Adds DE2, DE6 and BE2 defaults.
Adds reorderable array field utility for custom editors, allowing quick reordering of elements with 2 buttons (useful in execution order).

Now includes the rest of the DE simulation components.


[DOCFIX] Update Configuring-Alluxio-with-secure-HDFS.md of Chinese version

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12601/
Test PASSed.


make use of micro/go-log

In favour of "libraries not logging" I've defined a universal log interface https://github.com/go-log/log to help solve this problem and a micro scoped logger https://github.com/micro/go-log for all micro libraries. This is the first PR to shift to using that logger. This change actually changes nothing since the underlying implementation still uses the stdlib log package. In the grand scheme of things people can freely set the logger using micro/go-log.SetLogger.

Looking good. I don't know if  I am overlooking something, but I don't see that you are setting the DefaultLogger. So shouldn't the logger by default be the noopLogger and it won't log anything?

https://github.com/micro/go-log/blob/master/log.go#L10


Implement logger message for enhanced logging

Each route should be named (or randomly named) and this should be reflected in the logs. Logger message will enable this to be a much easier.

Didn't realise this had already been done

Didn't realise this had already been done


chore(release): v2.15.1

Problem

Solution

Additional Notes

Links
Ticket

GitHub issue:
Other links
Verification
Manual tests

Automated tests

[ ] Unit tests added/updated
[ ] E2E tests added/updated
[ ] N/A - (provide a reason)
[ ] deferred - (provide GitHub issue for tracking)

Housekeeping

[ ] No non-essential console logs
[ ] All new files contain license notice

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #1066 (61ed94d) into main (e12116a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main    #1066   +/-   ##
=======================================
  Coverage   93.81%   93.81%
=======================================
  Files         124      124
  Lines        5850     5850
  Branches     1785     1785
=======================================
  Hits         5488     5488
  Misses        344      344
  Partials       18       18

Continue to review full report in Codecov by Sentry.

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


scrabble-score: capitalize oxyphenbutazone

A history:

1d941e524d25b40d3e8ab452dc1a945423da897b: MULTIBILLIONAIRE
9541d7b7b835fb98eef32c281a8931f34b73322b: oxyphenbutazone
(because multibillionaire was too long!)
But since MULTIBILLIONAIRE was our only word with capital letters, now
we've lost the capability to test solutions for them! So let's bring
it back by putting OXYPHENBUTAZONE in all caps!

Test version is bumped for this.
Closes #270

Thank you!


Exercise: Write chunk(-like data structure) to disk and read it using mmap.

Adapted according to feedback from @tjjordan and me.

Merged master to have a working CI config.

Closing for now to clean up PR board.


Using reactive variable to set date on timepicker, but date doesn't show.

I have a full calendar plugin, and I set the doc attribute of the quickForm template as the event I just clicked. The form sets the title and description, but not the start and end dates. It comes from the helper as "invalid date", but it is a valid date in the event variable, coming from the calendar plugin.

Closing all issues. The plugin was a proof of concept which others have forked and made better alternatives.


listener Socket closes die to inactivity

After a while the listener stops after grateful closure of connection with closure code 1000 due to inactivity.
Could it be, the websocket needs some ping/pong to keep connection alive?

Are you referring to an approach similar to this? https://websockets.readthedocs.io/en/stable/topics/timeouts.html#keepalive-in-websockets

Yes, that's what I refer to. But indeed I do not know whether dirigera hub actually supports or needs it. I just see my event listener being closed after a while when no traffic is exchanged with closure code 1000

I don't experience that problem but that is probably because I have an air quality sensor that reports changes very frequently (several in a minute).

I suggest then publishing a ping message every once in a while. I will try to verify if this works. Never experienced this also due to having the air quality sensor.

Well in fact, since last weeks I don't see the connection drop anymore.
Maybe with the new software IKEA dropped or enlarged the timeout. For the
moment, I guess it could be considered as non issue
Nicolas Hilberg @.***> schrieb am Do., 26. Okt. 2023,
20:04:

I suggest then publishing a ping message every once in a while. I will try
to verify if this works. Never experienced this also due to having the air
quality sensor.
—
Reply to this email directly, view it on GitHub
https://github.com/Leggin/dirigera/issues/27#issuecomment-1781589226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6ATSSTNEZTD45A3G4ON3VLYBKQZHAVCNFSM6AAAAAA4BXDKQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBRGU4DSMRSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

The behaviour reoccured and I see disconnection due to inactivity of the listener each 50 to 80 minutes when no updates occur. So maybe the ping would still be a good option.
At the moment I workaround this by reconnecting on on-disconnect

@alexkno79 I found that web socket has a ping interval parameter.
I set it default to 60 seconds now, but it's exposed so it can be changed.
If you find out a better default let me know.

Thanks!
I updated and keep an eye in it. Usually it happens overnight when no device is acting. I'll report back

I see the pings and the pongs but still disconnections.
So pinging seem not to keep the channel open. On the other hand, i have a third party zigbee outlet which sends a signal each 5 minutes, this keeps the connection stable :)

Apologies for the necro post,
I've been tinkering the past few days trying to find why exactly my script was locking up and I've managed to narrow it down to this module.
My theory is that the issue does indeed still persist where the event listener does eventually die out once no actions have occured for some time.
Is there a general concensus on how we should go about keeping it alive?


Create Initial Docs

A new docs site likely needs to be spun up. This should include at a minimum some basic installation, getting started, and a few guides.
Additionally, we need some API docs, which means writing a lot of docstrings (or getting copilot to help 😉)

would love to contribute to agent docs writing/generating, and site publishing. Let me know how I could get started. @logan-markewich

@peteryxu while trying to avoid having too many docs sites, considering integrating the docs into llama-index docs.
I'm thinking the docs will mostly live in this repo, and the build step for the llama-index docs will clone this and copy over some info
Still need to try and get that working 😅
For docs in general, I'd really like to focus on a good quickstart (maybe the current readme isnt good enough), theory/structure overview, api docs (so good docstrings, since it's generated from that) and good examples.

that's sounds a good plan. I can start thinking about some good quickstarts, and arch overview.

For general DocOps automation & process, have you looked at https://www.komment.ai/#product ?  @logan-markewich

Docs are now available at https://docs.llamaindex.ai/en/latest/module_guides/llama_deploy/ we can call this issue done, will open a new one in case later we want to have a dedicated website to serve Llama Deploy docs exclusively


Error when building the fonts on Linux

It seems that either the README is out of date or there's a bug in the script because when you try to run it you get:
Traceback (most recent call last):
  File "scripts/instantiate-code-fonts.py", line 33, in <module>
    fontPath = sys.argv[2]
IndexError: list index out of range

And it all seems to be related to this line introduced in the latest commit: https://github.com/arrowtype/recursive-code-config/blob/main/scripts/instantiate-code-fonts.py#L33
EDIT: There also seems to be an issue with the line endings of scripts/build-all.sh because if you try to run it as-is in linux you get:
scripts/build-all.sh: line 2: $'\r': command not found
scripts/build-all.sh: line 4: $'\r': command not found
: invalid optionl.sh: line 5: set: -
set: usage: set [-abefhkmnptuvxBCHP] [-o option-name] [--] [arg ...]
scripts/build-all.sh: line 6: $'\r': command not found
scripts/build-all.sh: line 23: syntax error near unexpected token `$'do\r''
'cripts/build-all.sh: line 23: `for config in $configs; do

This gets fixed by changing the line endings from CRLF to simply LF

For the first issue, the script changed recently, to take in a source font-data path. In order to generate the font again, you'll need to run this instead:
python3 scripts/instantiate-code-fonts.py <premade-configs/casual.yaml> <font-data/Recursive_VF_1.081.ttf>

It's mentioned here.
I've pushed a PR fix that picks a default path. This is only if @arrowtype feels like keeping the previous api makes sense where we don't have to explicitly specify the path.

@arrowtype I just tried with a freshly cloned repo and it worked out the box. Good job! 👍

That's amazing news – thanks for checking and letting me know, @larsbs!


Merge pull request #10 from andela-jomadoye/feature/143585203/build-f…

…rontend-components-with-effective-UI/UX
Feature/143585203/build frontend components with effective ui/ux

Changes Unknown when pulling 89e91f64be17ab049344c1eaa7b73991a448ceb5 on develop into ** on chore/143585017/writing-tests**.


Cannot customize button border color

In the current _buttons.scss file, the same color is sent in as background and border color. This was not the case earlier where we could specify in our customization specific border and background colors.
We need to be able to customize border colors separate from the background. Otherwise why have borders at all?
I find this related to #23418

Although the border is defined at the mixin:
https://github.com/twbs/bootstrap/blob/v4-dev/scss/mixins/_buttons.scss#L9
It's set as the same color of the background here:
https://github.com/twbs/bootstrap/blob/v4-dev/scss/_buttons.scss#L55
I think this is a bug. The solutions I can think of are:

Make the border a darker shade of the background and keep this very nice definition of btns based on just one color set in the maps.
Create a border variables (I don't like it)

What do you think?

I described the same problem here https://github.com/twbs/bootstrap/issues/23126

@andresgalante  In my opinion this is one of the weakness with the new variable system. It is structured a lot better than earlier, but it isn't as flexible as it was, limiting us to either accept the default variations of basic color settings or to be forced to create our own definitions of the classes which I think will create more confusion, larger style sheets, more error prone and harder to read.
As for your suggestions, I think that the first one isn't an option. In our case, we want the background to be one color and the outer to be a completely other one ($white background and $blue border) and as far as I understand your suggestion it won't give me that.
The only way this can be solved for the usage I have is to be able to configure colors by themselves, and the only way is to have variables. However, instead of having lots of key/value pairs, why not use key/array structures instead? Like this: (I do not know scss that well, so just look at this example as pseudo code)
$buttons = (
    primary: (
        color: $white,
        background: theme-color("primary"),
        border: theme-color("primary"),
        /* and other button related parameters */
   )
)

Then generate buttons based on this with:
@each $color, $values in $buttons {
  .btn-#{$button} {
    @include button-variant($values);
  }
}

and let button-variant read the configuration from $values?
As I said, it may not work since I don't know scss that well, but if it works it's worth a try.
I did get the issue described by @wojtask9 today and was planning to check it out tomorrow but it is clearly already registered. But I think that a similar construct like this (if it works) could be applied to lots of the similar issues.

@dozer75 you are absolutely right it is limiting and my solution wouldn't help much.
A better mapping can be the solution but I am afraid that the more scss abstract we generate the harder it would be for people to understand what's happening under the hood and reduce contributions. The beauty of bootstrap is that it is easy to read in a way.

@andresgalante Actually I think that this abstraction will make it easier to read because users of the scss wont need to go into the source to find out why for example a button get's a specific border or foreground color or an alert get's it's border color or foreground color as we have to do now. By having it structured in the variables files with logical variable names it can be seen there. I have used two full days now with the current scss files to try to understand what's happening under the hood and it is quite confusing with all these auto calculated variants inline in functions. On the plus side it has given (or forced :) ) me the opportunity to learn scss more in-depth than I ever had before.

You are right @dozer75 I think it's worth the try, but it's not really my call 😄
If you do it I would love to review that PR and learn from it.

I found this because after updating from Alpha 6 to Beta I noticed that my own Button variations looked completely messed up.
It appears as though I can't directly control the text colour because of color-yiq and I can't find any documentation on how that works so I'm kind of stuck.
Even having $brand-primary removed seemed like something that should have happened well before alpha 6.
I feel like the mixins still need a lot more inline documentation.

color-yiq is define in mixins/_functions.
@mixin color-yiq($color) {
  $r: red($color);
  $g: green($color);
  $b: blue($color);

  $yiq: (($r * 299) + ($g * 587) + ($b * 114)) / 1000;

  @if ($yiq >= 150) {
    color: #111;
  } @else {
    color: #fff;
  }
}

it works by contrasting the RGB values. If the output is >=150, the contrast is too hight and the text colour is set to 111.
It's a nightmare trying to override mixins in Bootstrap at the moment!
#23499

@andresgalante My problem is that I don't have that much time, but I'll give it a try to structure something however not sure when I can get it done. There is a lot of parameters that would have to be restructured around. But maybe start small to get some idea.

@dozer75 before you embark in something like this, I'd wait for the blessing from one of the core team, specially if you don't have much time.

@andresgalante Ok, but I am a bit curious myself, so I will try to do a small prototype portion to see if it even is possible in a neat clean way :) It will also be a good experience anyway to be able to handle bootstrap in the future by learning more of the internals.

@andresgalante I pushed a branch to a bootstrap fork on my account: https://github.com/dozer75/bootstrap/tree/enhance-button-variables Don't want to PR it as it is just to show one (of many) solutions to the issue.
I can't find really a good way to do it, one is to push lots of variables into _variables.scss, but I found that it messed it up quite a lot. So I went for this one where the theme button maps was filled with default values if they doesn't exist.
Most of the changes are in mixins/_buttons.scss, but also in the ordinary _buttons.scss (just calling to the mixins in _buttons.scss) and _variables.scss (comments on how to customize).

I just ran into this as well. The variable changes are really bad, I was looking forward to v4 and now I'm not.

Closing as dupe of #23418.

@mdo I could be missing something but this is a different issue than the one you linked to when closing as a dupe. I checked #23418 and #23611 (which supposedly resolved the former), and didn't find a solution for customizing the button border color.
Should this ticket be reopened?

@joshbuckley I agree. Customising the border colour is a seperate issue to #23418.

Any progress here?


'npm start' does not work locally

I am trying to work on a PR and I can't get the editor to run locally. Everything appears to install just fine, though. A window opens at http://localhost:3000/ but nothing loads. Any ideas would be appreciated.
Node: 6.10.0
npm: 3.10.6
OS: Windows 7 Enterprise
Here are the console errors:
warning.js:35 Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined.  You likely forgot to export your component from the file it's defined in. Check the render method of `Example`.
    at invariant (invariant.js:44)
    at ReactCompositeComponentWrapper.instantiateReactComponent [as _instantiateReactComponent] (instantiateReactComponent.js:74)
    at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:366)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:257)
    at Object.mountComponent (ReactReconciler.js:45)
    at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:370)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:257)
    at Object.mountComponent (ReactReconciler.js:45)
    at mountComponentIntoNode (ReactMount.js:104)
    at ReactReconcileTransaction.perform (Transaction.js:143)

I figured it out! I removed "^" from all the dependencies in package.json because those minor version upgrades since this was last updated looks like it caused imports to break.
I have been under the impression that as a community, we're frowning upon using anything but exact dependencies in npm? Maybe this would work if I was using yarn? I'd like to work a PR for this but want to get some clarification around this first. Maybe it's just an update to the README?

Thanks @abobwhite, @gabrielrtakeda will try to resolve this bug and at the same time will upgrade slatejs #35.

Hi @abobwhite! I am glad that you have contacted us! I think it is really good that the community help improving the repository and I am very happy that caught your eyes! We really need people that want to contribute too and your help is very appreciate!
About the bug issue with npm, I tested locally installing the dependencies with npm (v5.0.4) and the problem is really with npm, because when installing with yarn, we already the version of packages that is working actually in the project.. I have dived a little more deep into the code and founded that the error was produced by the way that we are using the react-router package which have changed its router API.. I will push a commit to fix this and will have a look into the PR that you have opened..
Again, thanks for the contribution! (:

Thanks @gabrielrtakeda and @lpirola I went ahead and tried yarn yesterday and got it to work in the meantime. Thanks for updating npm support, too :)

We thank you for the help.

I would suggest while fixing this to remove "^" from the npm dependencies. While in theory, it's handy, so many libraries are not respecting sem-ver and so minor/patch versions are making incompatible, breaking changes. Which in turn make this library slave to those versions. I would recommend using no qualifier on the dependencies and just keeping them exact so you know it will work.
Also - I would recommend including in the README which version(s) of node/npm are compatible with the development of this editor.

Rebase 👇


Rubocop

We started with:
9 files inspected, 252 offenses detected
Now we have:
79 files inspected, 94 offenses detected

@geisasantos Can u review this PR?

@n3k00n3 sounds great, go ahead!

My only doubts were about the lines ajustment, that Brena already talked about it. So, about the tip, guess if we can Travis ou CircleCi to our repo, so the tests can run automatically. What do you think @n3k00n3 and @monteirobrena ?


Epmlabsbrn 298 make a get end-point which would return folders from aws s3

EPMLABSBRN-298
Description: BackEnd: make a get end-point which would return folders from s3

could you please add raml description of new end-point


Add list table

Could you add some description and usage of it?

:shipit:

would you mind adding some documentation about the public components (props needed, purpose, etc) being added and how they are used to the description of the PR?

@adampickeral Do you have an example of what you'd like to see? It seems like the documentation we have is primarily the Demo classes, which I've provided. Also, it probably shouldn't reside in the PR, but the wiki is currently empty, so opinions on where it should go?

when we have a larger feature being added such as this, it's good to have documentation in the PR (or in an issue proposing the change) about the approach that is being taken. It makes reviewing the approach at a higher level easier, and it also makes it easier for anyone to update the official documentation after the feature is merged in.
https://github.com/rackerlabs/canon-react/pull/27
https://github.com/rackerlabs/canon-react/issues/55

@adampickeral Gotcha. Thanks for the examples. I'll add something similar.

:+1:

LGTM

Great work, @wpwood ! This looks awesome. :+1:


Need Xcode 6.4 support

Hi,
It was working fine on Xcode 6.3.2 but with the release of 6.4 it stopped working. Can you please add the UDID so it can work again? Thanks!

Run this shell:
https://gist.github.com/alexlee002/b87dbeac1f9fd896174b

Just submitted a pull request for this:  https://github.com/yuhua-chen/MCLog/pull/24
In the meantime, you can use my fork for Xcode 6.4 support: https://github.com/mverderese/MCLog


dotnetCorePackages: refactor aliases to microsoft naming scheme

Description of changes
Related Issue: https://github.com/NixOS/nixpkgs/issues/260915
Not sure we need any message to user, basically would be seen a stable version bump to anyone referencing the existing name.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

This will break dozens of packages... aliases.nix is meant for deprecated aliases

This will break dozens of packages... aliases.nix is meant for deprecated aliases

So it's not an actual alias? I saw numerous references that clang does this with.

Use of aliases from aliases.nix isn't allowed in nixpkgs, it's for out of tree uses.

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

28 packages built:

dotnet-aspnetcore
dotnet-aspnetcore-lts
dotnet-aspnetcore-preview
dotnet-runtime
dotnet-runtime-lts
dotnet-runtime-preview
dotnet-sdk
dotnet-sdk-lts
dotnet-sdk-preview
fable
famistudio
godot3-mono
godot3-mono-debug-server
godot3-mono-export-templates
godot3-mono-headless
godot3-mono-headless.dev
godot3-mono-headless.man
godot3-mono-server
godot3-mono.dev
godot3-mono.man
lidarr
nuget-to-nix
pbm
pre-commit
pre-commit.dist
prowlarr
radarr
readarr

Result of nixpkgs-review pr 260946 run on aarch64-darwin 1

13 packages built:

dotnet-aspnetcore
dotnet-aspnetcore-lts
dotnet-aspnetcore-preview
dotnet-runtime
dotnet-runtime-lts
dotnet-runtime-preview
dotnet-sdk
dotnet-sdk-lts
dotnet-sdk-preview
pre-commit
pre-commit.dist
prowlarr
radarr

Lost interest in dealing with this. Thought it seemed like a no brained quick help PR.


How to trun on evaluation during the training.

I want to turn on the evaluation during the training process.

Please set config.TEST.EVAL_PERIOD value, e.g. 100 means after training 100 iters, test one time.

Thank you very much!


it doesnt show the option of new project and external class

Issue Type: Feature Request
it doesnt show the option of new project and external class
VS Code version: Code 1.54.1 (f30a9b73e8ffc278e71575118b6bf568f04587c8, 2021-03-04T22:38:31.419Z)
OS version: Windows_NT x64 10.0.19042

VS Code doesn't have a 'project' concept quite like you may be familiar with from other tools.
Have you seen https://code.visualstudio.com/docs/editor/multi-root-workspaces ?
Also, please clarify what you mean by 'external class'.


Cordova camera plugin doesn't work with $ionicActionSheet

When I'm trying to use Cordova camera plugin in buttonClicked function within $ionicActionSheet this doesn't work.
This is my code:
$ionicActionSheet.show({ 			buttons: [ 				{ text: 'Camera' } 			], 			titleText: 'Subir evidencia', 			buttonClicked: function(index) { 		    if (window.cordova) { 		      var options = { 		        quality: 75, 		        destinationType: Camera.DestinationType.DATA_URL, 		        sourceType: Camera.PictureSourceType.CAMERA, 		        allowEdit: true, 		        encodingType: Camera.EncodingType.JPEG, 		        targetWidth: 300, 		        targetHeight: 300, 		        popoverOptions: CameraPopoverOptions, 		        saveToPhotoAlbum: false 		      }; 		      var base64 = $cordovaCamera.getPicture(options).then(function (imageData) { 		        return "data:image/jpeg;base64," + imageData; 		      }, function (err) { 		        console.log({ title: 'Error!', template: 'Problemas al conectar con la camara!'}); 		        console.log({ title: 'Error!', template: err}); 		      }); 		    } else { 		        console.log({ title: 'Error!', template: 'Cordova initializing problem!'}); 		    } 			} 		});
this gives to me the following message 'Camera cancelled'

Hello! Are you still having this issue? Thanks!

Hello! As it has been a little while since there was any activity on this issue i will be closing it for now. Thanks for using ionic!

i facing this problem when i inspect it in browser
Object {title: "Error!", template: "Cordova initializing problem!"}


heroku deploy cannot find bower

the line "postinstall": "../../node_modules/bower/bin/bower install" mentioned in the readme doesn't work for me, it can't find bower. where is this directory supposed to come from?

btw it works when I change it to just bower install without any directories.

Thanks for bringing this up.

Howdy,
I'm running into a similar problem trying to get our app deployed on Heroku.
Ember CLI recommends installing bower globally and it seems like bower doesn't get installed as part of the default Heroku node buildpack. I've tried setting NPM_CONFIG_PRODUCTION=false as recommended in the docs, but that doesn't make a difference.
Do I need to install bower globally in the preinstall or postinstall hook of the root package.json?
We're using:

Node 4.1.0
NPM 2.14.3
Ember CLI 1.13.8


Logger suggestion - Set time zone for logs

Would like an option to change the time zone shown in the logs.

Great suggestion, will be added soon


Migrates Project dashboard Inventory Item status mapper for OBC

Signed-off-by: Bipul Adhikari<EMAIL_ADDRESS>

/lgtm


Problem with SAML broadcast Logout request NameId format!

I am duplicating request for the reason that no one answer my questions in original thread!!! It was closed in may 22 and we don't have any information of progression! In august I checked state of functionality by myself and found that problem is still there! Now I need to know when it will be fixed and what a priority of that bag?

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

ID: 76d4eac0-25d3-b5d5-f58e-cdd1024f32b4
Version Independent ID: 7771f280-3701-4f88-cb9f-add968d494de
Content: Azure Single Sign Out SAML Protocol
Content Source: articles/active-directory/develop/single-sign-out-saml-protocol.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@FunkEinstein Thanks for the feedback! We are currently investigating and will update you shortly.

@FunkEinstein I got an update that the work is still in progress. We will update the thread as soon as the work is finished regarding this.

@ManojReddy-MSFT what is the estimated time for resolution?

@FunkEinstein I am afraid I do not have an ETA for this. The Product is actively working on this and is planning to preview this feature as soon as possible.

@ManojReddy-MSFT I see. Ok, I'll wait.

@FunkEinstein I got a confirmation that a fix for this bug has been deployed by the engineering team. Can you test this again in your environment and let me know if it's working as expected now?

@ManojReddy-MSFT I will test it tomorrow!

@ManojReddy-MSFT so I tested this out and old issue is gone, yay.
But I found another problem! After my service processed SAML LogoutRequest it create SAML LogoutResponse and send it into url that specified in SAML metadata in SingleLogoutService. This behavior specified by official SAML Specification:
"When either the principal invokes a logout at the session authority, or a session participant sends a logout request to the session authority specifying that principal, the session authority SHOULD send a  message to each session participant to which it provided assertions containing authentication statements under its current session with the principal, with the exception of the session participant that sent the  message to the session authority. It SHOULD attempt to contact as many of these participants as it can using this protocol, terminate its own session with the principal, and finally return a  message to the requesting session participant, if any."
But after service send created LogoutResponse I see next message in logout form:
Sign in
Sorry, but we’re having trouble signing you in.
AADSTS7500513: The message type 'Microsoft.AzureAD.Protocols.Saml2.LogoutResponse' is not a supported type of SAML request. Supported SAML requests are AuthnRequest and LogoutRequest.

@FunkEinstein Thanks for the confirmation. I will get back to you regarding the new error.

@ManojReddy-MSFT so is there any information about new error?

@FunkEinstein
I am assuming your service is initiating the logout request. In this case, Azure AD sends the logout response to your service after processing the logout request,
If some other service sends a logout request and Azure is broadcasting its own logout request, to your service (assuming your service shares the same session) then Azure AD is not expecting another response back.  Your service should not send any response back.
In the statement from SAML specification,  the session authority is Azure AD and it will send a logout request to other services and will send a log out response to the service that initiated the single logout request.
I hope this clarifies.

Yeah, you right, I demonstrated an unsuitable passage. But still, specification says that I MUST respond with LogoutResponse:
"3.7.2 Element
The recipient of a  message MUST respond with a  message,
of type StatusResponseType, with no additional content specified.
..."
Here is another reference to the fact that session participant must respond after receiving LogoutRequest:
"<IP_ADDRESS> Session Authority Rules
...
In the event that not all session participants successfully respond to these  messages(or if not all participants can be contacted), then the session authority MUST include in its
 message a second-level status code of urn:oasis:names:tc:SAML:2.0:status:PartialLogout to indicate that not all other session participants successfully responded with confirmation of the logout.
..."

@FunkEinstein I understand. Unfortunately, Azure AD currently cannot acknowledge this logout response and that's most likely the reason for the error you are receiving.
If this is a requirement, I would recommend submitting a feature request here.

@FunkEinstein We will now proceed to close this thread. If you have further questions, please tag me in the comments and I will gladly continue the conversation.

This is a requirement of SAML specification, not my! Your realization isn't fully corresponds to specification, don't you think that it's need to be fixed?!

@FunkEinstein I agree with your concern. This part of the SAML specification is currently not supported.
However, our product team is working on this and this will be supported in the future. I do not have any ETA's regarding this.
Submitting a feature request will let the product team prioritize this over other features.


Incorrect output values in issue_metrics.json

Incorrect output values for average_time_to_first_response and average_time_to_close in issue_metrics.json
The actual values in issue_metrics.json are shown below, but the timedelta map values have been output.
{
    "average_time_to_first_response": "{'avg': datetime.timedelta(seconds=50124), 'med': datetime.timedelta(seconds=10122), '90p': datetime.timedelta(days=1, seconds=15064)}",
    "average_time_to_close": "{'avg': datetime.timedelta(seconds=69549), 'med': datetime.timedelta(seconds=19616), '90p': datetime.timedelta(days=2, seconds=78098)}",
    "average_time_to_answer": "None",
    "average_time_in_labels": {

https://github.com/github/issue-metrics/blob/9291773ac94d1e6e9afa852d792e92230f882582/json_writer.py#L85C22-L85C22
In the above code, the variable average_time_to_first_response used to be of type timedelta, but it has changed to type map, so  need to specify the element to get.

fixed in #163

I also added #164 support for median and 90th percentile.
Please merge the ones you like.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 273 ms

Resolved: ClamAV performance has improved in e9fc9c7 after 46 minutes.


Is ability to paste data into a select2 for auto-tokenization broken?

I need the ability to paste data into a select2 data input field, that would be auto-tokenized into tags.  Is this ability supported?  I seem to have read on Issue #3458 that Pasting is currently not creatingTags.  Is this true, or has this since been corrected?  If it does support this, could you please provide a code sample on how to enable this?  I didn't see one posted on the example page.
Would appreciate any support this!
Thanks.

Yes.
This is a duplicate of https://github.com/select2/select2/issues/3458.


ffmpeg EACCES error

Describe The Bug:
Snapshot or video stream fails.
To Reproduce:
Every time I open the camera in Home app
Expected behavior:
Snapshots should show in the app and video stream in the app when opening the camera.
Logs:
[6/24/2020, 9:08:27 PM] [Security Cameras] Snapshot from Driveway at 480:270
[6/24/2020, 9:08:27 PM] [Security Cameras] An error occurred while making snapshot request
[6/24/2020, 9:08:35 PM] [Security Cameras] [FFMPEG] Failed to start stream: spawn /homebridge/node_modules/homebridge-camera-ffmpeg/node_modules/ffmpeg-for-homebridge/ffmpeg EACCES

Homebridge Config:
{
    "bridge": {
        "name": "Homebridge 9347",
        "username": "0E:BC:CB:CA:93:48",
        "port": 53253,
        "pin": "031-45-154"
    },
    "platforms": [
        {
            "name": "Security Cameras",
            "cameras": [
                {
                    "name": "Driveway",
                    "manufacturer": "Reolink",
                    "model": "RLC-410-5MP",
                    "firmwareRevision": "v<IP_ADDRESS>8_19061407",
                    "videoConfig": {
                        "source": "-rtsp_transport tcp -re -i rtsp://motion:motioneye@<IP_ADDRESS>:554/h264Preview_01_sub",
                        "stillImageSource": "-i http://<IP_ADDRESS>/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=&user=motion&password=motioneye",
                        "maxStreams": 2,
                        "maxFPS": 15,
                        "maxBitrate": 300,
                        "vcodec": "copy",
                        "videoFilter": "none",
                        "audio": false,
                        "debug": false
                    }
                }
            ],
            "platform": "Camera-ffmpeg"
        }
    ],
    "accessories": []
}

Screenshots:
n/a
Environment:

Node.js Version: v12.18.1
NPM Version: v6.14.5
Homebridge Version: 1.1.1
Homebridge Camera FFmpeg Version: 2.0.0
Homebridge Config UI X Plugin Version: 4.23.0
Operating System: Docker on Raspberry Pi 4

Closing this per #554.

I have upgrade to latest v2.1.1 of the plugin and this issue still exists:
[7/10/2020, 7:10:12 AM] [Security Cameras] Snapshot from Driveway at 480:270
[7/10/2020, 7:10:12 AM] [Security Cameras] ffmpeg -i http://<IP_ADDRESS>/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=&user=motion&password=motioneye -frames:v 1 -f image2 -
[7/10/2020, 7:10:12 AM] [Security Cameras] An error occurred while making snapshot request
[7/10/2020, 7:10:12 AM] [Security Cameras] Error: spawn /homebridge/node_modules/homebridge-camera-ffmpeg/node_modules/ffmpeg-for-homebridge/ffmpeg EACCES
    at Process.ChildProcess._handle.onexit (internal/child_process.js:267:19)
    at onErrorNT (internal/child_process.js:469:16)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  errno: 'EACCES',
  code: 'EACCES',
  syscall: 'spawn /homebridge/node_modules/homebridge-camera-ffmpeg/node_modules/ffmpeg-for-homebridge/ffmpeg',
  path: '/homebridge/node_modules/homebridge-camera-ffmpeg/node_modules/ffmpeg-for-homebridge/ffmpeg',
  spawnargs: [
    '-i',
    'http://<IP_ADDRESS>/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=&user=motion&password=motioneye',
    '-frames:v',
    '1',
    '-f',
    'image2',
    '-'
  ]
}

It turned out to be a disk permission issue (the mounted disk had noexec flag). I removed it following this: https://forums.plex.tv/t/pms-cannot-transcode-any-file-conversion-failed-the-transcoder-crashed-or-failed-to-start-up/227898/9

Glad you were able to solve it.


TestKit permanently takes 900MB+ of /tmp/ on testing with different Gradle versions

Expected Behavior
Using TestKit should not permanently take 1GB of RAM.
Current Behavior
TestKit permanently takes 1GB of RAM when running tests with different Gradle versions.
Context
Running regression tests with TestKit using 6 different Gradle versions takes 900MB+ in /tmp/.gradle-test-kit-foobar/. It's a temporary storage, however, on modern Linux distributions /tmp/ is a virtual file system kept entirely in memory. As TestKit doesn't seem to free that space (which in general makes sense) that ~1GB or RAM is effectively lost until restart (or manual cleanup). Having "only" 16GB of RAM (with Gradle and Idea) it occasionally is a problem for me.
The majority of space it taken by generated-gradle-jars (120MB+ per version). I like the fact that TestKit separates the Gradle instance that is used for tests, however, some people might (optionally) prefer to keep it somewhere in ~/.gradle/, especially if used on the developer workstation. It would also slightly reduce the execution time of the first run (after power-on/restart).
$ du -h --max-depth=2 /tmp/.gradle-test-kit-foobar/
13M	/tmp/.gradle-test-kit-foobar/caches/jars-8
141M	/tmp/.gradle-test-kit-foobar/caches/6.5-milestone-1
143M	/tmp/.gradle-test-kit-foobar/caches/6.4
131M	/tmp/.gradle-test-kit-foobar/caches/6.0.1
0	/tmp/.gradle-test-kit-foobar/caches/transforms-2
176M	/tmp/.gradle-test-kit-foobar/caches/5.6.4
161M	/tmp/.gradle-test-kit-foobar/caches/5.0
4,0K	/tmp/.gradle-test-kit-foobar/caches/transforms-1
5,1M	/tmp/.gradle-test-kit-foobar/caches/modules-2
2,3M	/tmp/.gradle-test-kit-foobar/caches/jars-3
28K	/tmp/.gradle-test-kit-foobar/caches/journal-1
150M	/tmp/.gradle-test-kit-foobar/caches/4.10.3
919M	/tmp/.gradle-test-kit-foobar/caches
40K	/tmp/.gradle-test-kit-foobar/native/5fef6b658a83c5bcc1b372000e2bec0065fa8a789a588892999cc2c26706b2e4
36K	/tmp/.gradle-test-kit-foobar/native/481b90ef2c2b5ab96622bab5346fdaaafe5c91895d4d4de1c4fe4b2e89a8011f
40K	/tmp/.gradle-test-kit-foobar/native/30
272K	/tmp/.gradle-test-kit-foobar/native/jansi
36K	/tmp/.gradle-test-kit-foobar/native/25
424K	/tmp/.gradle-test-kit-foobar/native
236K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/6.5-milestone-1
236K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/6.4
232K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/6.0.1
220K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/5.6.4
216K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/5.0
216K	/tmp/.gradle-test-kit-foobar/test-kit-daemon/4.10.3
1,4M	/tmp/.gradle-test-kit-foobar/test-kit-daemon
921M	/tmp/.gradle-test-kit-foobar/

There is also a related issues #12745 about cluttering up /tmp with the gradle-worker-classpath* files.

Hi @szpak,
You can change the default using GradleRunner.withTestKitDir(File).

Thanks @eskatos. It should help in that case.
I wonder, if there is anything else that could be done to make it more intuitive in general?

That option can currently be found by looking at the TestKit API. Maybe adding something to the user guide to make it more discoverable would have helped you find it?

Good idea with the user guide. I wasn't aware of it.

From https://github.com/gradle/gradle/issues/1675#issuecomment-289753752 I know there is also an ability to set it in the project configuration via org.gradle.testkit.dir. However, it doesn't work if put globally in ~/.gradle/gradle.properties as those properties are not available for TestKit by default (which in general is good). Rewriting that property in the project (if available) is some option, but it is not perfect.

We should consider changing the convention for where TestKit writes state to, as /tmp isn't really a good choice in many cases. It should probably default to somewhere under the build directory.
Also, to avoid duplicate work and storage for things that build logic tests as generally not concerned with, we might also consider reusing the main Gradle caches for certain things, such as dependency resolution state, downloaded artifacts, generated Jars and so on. A good choice might be to reuse the main caches for everything except configuration files and daemon state.

For context, the original choice was influenced by the test runtime not reliably knowing where the build dir is given it is configurable and there's no guarantee that Gradle bootstrapped the test runtime and could therefore specify where it is (e.g. system property).
I think it would make sense for Gradle to specify a path inside $buildDir to the test runtime and have TestKit use that. When that's not set, it could either then fallback to /tmp or assume that $buildDir is new File("build"). It might also be worth using a directory inside rootProject.buildDir depending on exactly what this ends up getting used for.
As part of this, we might want to reconsider explicitly setting the gradle.user.home for the test runtime so that any custom user home can be found: https://github.com/gradle/gradle/blob/d0f5ac620da4d4410285e864069830a7675b8732/subprojects/test-kit/src/main/java/org/gradle/testkit/runner/GradleRunner.java#L127

Any thoughts in 2022 on an ability to keep it in some (other) user defined place?


pytorch 1.3 test error

in build progress  warning  this message：
/home/chenjun/Deformable-Convolution-V2-PyTorch-pytorch_1.0.0/src/cuda/deform_conv_cuda.cu:219:2309: warning: 鈥[01mT* at::Tensor::data() const [with T = float]鈥is deprecated [-Wdeprecated-declarations]
/home/chenjun/anaconda3/envs/torch13/lib/python3.7/site-packages/torch/include/ATen/core/TensorBody.h:303:1: note: declared here
when run test.py:
ImportError: /home/chenjun/anaconda3/envs/torch13/lib/python3.7/site-packages/DCN-1.0-py3.7-linux-x86_64.egg/DCN.cpython-37m-x86_64-linux-gnu.so: undefined symbol: cudaSetupArgument

I encountered the same problem, have you solved it now?

I also encountered the same problem with pytorch 1.7, python 3.8. Have you solved it?


ci: Remove 10.15 runner for test

Issue #, if available:
Description of changes:
Remove OS 10.15 version test since breaking change not supported in 10.15
Testing done:

[x] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Tests failed due to timeouts. Fixed in #329 (not merged yet). Overriding and merging.


Revert "Use feature macro to check for support of filesystem"

Reverts Microsoft/vcpkg#5651

Resolves #5764


Publish Docker images on the Hub or Github Container Registry

This would allow people to download pre-built images. The dependencies for this library are complex and sometimes conflict (e.g. when using conda/mamba), so using Docker avoids that problem.
You could add a Github Action to automatically build and push the images on pushes/PRs to main or new release tags.

Thanks for your suggestion.
Will add it on the to-do-list.
Also reducing dependency complexity will be of top-priority.

Resolved with PR #232


Use 201 status code for put license

Scope & Purpose
Backport of https://github.com/arangodb/arangodb/pull/16177

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

[ ] https://jenkins.arangodb.biz/view/Documentation/job/arangodb-ANY-examples/1945/


Create openmc installation scripts

This issue can be closed when scripts that install and build OpenMC such that all tests pass exist. This may depend on building and installing HDF5 and h5py from source.

Update: 7 tests are failing. 3 value failures and 4 technical failures

Test: 2 group CMFD with CMFD Feedback

Assertion error: Results do not agree

Test: Plot file string hash test

Attribute error: ‘Dataset’ object has no attribute ‘value’

Test: Overlap plot test

Attribute error: ‘Dataset’ object has no attribute ‘value’

Test: Voxel plot test

Attribute error: ‘Dataset’ object has no attribute ‘value’

Test: Track to vtk test

AssertionError: poly.pvtp file not found
Attribute error: ‘Dataset’ object has no attribute ‘value’

Test: Statepoint restart test

Assertion error: Results do not agree

Test: Triso test

Assertion error: results do not agree


Bug: request_meta is not available in server.request_context

Describe the bug
This makes it impossible to build a server with progress notifications
To Reproduce
Steps to reproduce the behavior:
do a tool_call for example:
{
"method": "tools/call",
"params": {
"name": "6a41a978-0a31-4eaf-9d0c-431a5f784bc9",
"arguments": {
"input_value": "3*2"
},
"_meta": {
"progressToken": 0
}
}
}
server.request_context.meta is None
Expected behavior
meta should include the progressToken
It's not getting assigned here https://github.com/modelcontextprotocol/python-sdk/blob/main/src/mcp/server/init.py#L447

Good catch. Do you mind having a go at create a PR? Happy to review.


Fix Zabbix installation on Bionic

On bionic, the database schemas are not installed on containerized
systems such as LXC or docker images because they contain a dpkg
configuration which excludes file installation to /usr/share/doc/*.
This breaks the role because dpkg filters the Zabbix database schemas
that are typically installed to this path.
Type of change
Bugfix Pull Request
Fixes an issue
On a bionic install in affected environments, the following is observed:
# dpkg -S /usr/share/doc/zabbix-server-mysql/create.sql.gz
zabbix-server-mysql: /usr/share/doc/zabbix-server-mysql/create.sql.gz
# ls /usr/share/doc/zabbix-server-mysql/create.sql.gz
ls: cannot access '/usr/share/doc/zabbix-server-mysql/create.sql.gz': No such file or directory
# ls /usr/share/doc/zabbix-server-mysql
changelog.Debian.gz  copyright

This causes the task zabbix_server : Get the file for create.sql >= 3.0 to fail due to the error ls: cannot access '/usr/share/doc/zabbix-server-mysql/create.sq*': No such file or directory
The reason the file is not present is because dpkg is configured to filter installations to /usr/share/doc/* in /etc/dpkg/dpkg.cfg.d/excludes, specifically with the line:
# Drop all documentation ...
path-exclude=/usr/share/doc/*

To fix this issue, we can drop the following line in the file before installing the zabbix-server-{mysql,pgsql} packages:
path-include=/usr/share/doc/zabbix*

, which will allow the Zabbix package to install the schemas as usual to these paths.
Please see upstream bug https://support.zabbix.com/browse/ZBX-14820 for more information.

Thanks! 👍


Feature videos stretch on the homepage

Huhm will the object-contain work better than object-fill?

Huhm will the object-contain work better than object-fill?

I dont even remember what i did lol!

https://github.com/longtoZ/score-gen-2/blob/main/client/src/components/home/Features.jsx
Hể

Ngủ đi bro
Giấc ngủ ngàn thu
Cùng cái project này


Functions provided as arguments not passed to component

Summary
When rendering a component with arguments accepting a function, storybook strips this and only assigns nil.
Based on this recommendation from the Phoenix framework this is a valid approach for unified communication, it works fine but just not in Storybook.
Ref: https://hexdocs.pm/phoenix_live_view/Phoenix.LiveComponent.html#module-unifying-liveview-and-livecomponent-communication
Steps to reproduce
Create a component:
defmodule Test do
  use Phoenix.component

  attr(:func, :any)

  def render(assigns) do
    ~H"""

<%= func.("hello world")  %>

    """
  end
end

Storybook:
defmodule Storybook.Test do
  use PhoenixStorybook.Story, :live_component

  def component, do: Test

  def variations do
    [
      %Variation{
        id: :default,
        attributes: %{
          func: fn _ -> "override text" end
        }
      },
      %Variation{
        id: :default,
        attributes: %{
          func: &display_something/1
        }
      },
    ]
  end

  defp display_something(_item), do: "override text"
end

Expected
To see the function being called and having my custom text returned
Actual Result
Error is returned because function is nil
** (exit) an exception was raised:
    ** (BadFunctionError) expected a function, got: nil

I never envisaged passing functions as assigns, but I agree it should work.
I'll see what I can do

You can make it work using late evaluation
defmodule PhoenixStorybookDemo.Components.Function do
  use PhoenixStorybook.Story, :component
  alias PhoenixStorybookDemo.Components.{Function, FunctionComponent}

  def function, do: &FunctionComponent.fun/1

  def variations do
    [
      %Variation{
        id: :anonymous_function,
        attributes: %{
          func: {:eval, ~s|fn _ -> "override text" end|}
        }
      },
      %Variation{
        id: :local_function,
        attributes: %{
          func: {:eval, ~s|&#{Function}.display_something/1|}
        }
      }
    ]
  end

  def display_something(_item), do: "override text"
end

Let me know if this is working for you.
You can then go ahead and close the issue.

Is it working for you?
I was able to test it again on core_component's table function, it was doing okay 👌
Feel free to reopen if necessary


Update interface.py

much better INDI tune for RAV4TSS2
Safety Param 73
Re-flash panda to use

Have you had anyone test this to make sure others agree it's an improvement?


Method that allows to load a json config file by name

Hi!
I miss a method that would allow to load json configuration by a file name. Method should do environment override how it is done for appsettings.json.
LogManager.Setup().LoadConfigurationFromJson("nlog.json");
Why we need this?
NLog config might be quite big in json and it might be a good idea to move it from appseetings.json in a separate file.
Here I created extension method to demonstrate the idea:
public static ISetupBuilder LoadConfigurationFromJson(this ISetupBuilder setupBuilder, string fileName, string? environment = null)
        {
            if (environment == null)
                environment = Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT");
            string noextension = Path.GetFileNameWithoutExtension(fileName);
            IConfigurationBuilder configurationBuilder = new ConfigurationBuilder().SetBasePath(Directory.GetCurrentDirectory()).AddEnvironmentVariables("ASPNETCORE_").AddEnvironmentVariables("DOTNET_")
                .AddJsonFile(fileName, false, false)
                .AddJsonFile($"{noextension}.{environment}.json", optional: true, false)
                .AddEnvironmentVariables();
            IConfigurationRoot config = configurationBuilder.Build();
            if (config.GetSection("NLog")?.GetChildren().Any() ?? false)
            {
                return setupBuilder.SetupExtensions(delegate (ISetupExtensionsBuilder e)
                {
                    e.RegisterNLogWeb();
                }).LoadConfigurationFromSection(config, "NLog");
            }
            throw new Exception("Config section not found");
        }

Closing issue since https://github.com/NLog/NLog.Web/pull/805 has started.

After looking at the code, then I think it is just easier if you do this in your own application:
var config = new ConfigurationBuilder().AddJsonFile("nlog.json").Build();
NLog.LogManager.Setup().SetupExtensions(e => e.RegisterNLogWeb()).LoadConfigurationFromSection(config);


--no-bruteforce does not work with a list of usernames and passwords.

If we use --no-bruteforce to test 1st user with 1st password, it only works with a file, would be good to have the same feauture if we provide a list of users and passwords instead of a file.
In the following example the 1st user and 1st password match and the 2nd user and 2nd password, but --no-bruteforce doesn't have an effect in this case, only if using a file.
crackmapexec smb <IP_ADDRESS> -u engels kiosko -p "Inlanefreight19!" "SimplePassword3!" --no-bruteforce --continue-on-success
SMB         <IP_ADDRESS>  445    DC01             [*] Windows 10.0 Build 17763 x64 (name:DC01) (domain:inlanefreight.htb) (signing:True) (SMBv1:False)
SMB         <IP_ADDRESS>  445    DC01             [+] inlanefreight.htb\engels:Inlanefreight19!
SMB         <IP_ADDRESS>  445    DC01             [-] inlanefreight.htb\engels:SimplePassword3! STATUS_LOGON_FAILURE
SMB         <IP_ADDRESS>  445    DC01             [-] inlanefreight.htb\kiosko:Inlanefreight19! STATUS_LOGON_FAILURE
SMB         <IP_ADDRESS>  445    DC01             [+] inlanefreight.htb\kiosko:SimplePassword3!

cat userfound.txt
engels
kiosko

cat passfound.txt
Inlanefreight19!
SimplePassword3!

crackmapexec smb <IP_ADDRESS> -u userfound.txt -p passfound.txt --continue-on-success --no-bruteforce
SMB         <IP_ADDRESS>  445    DC01             [*] Windows 10.0 Build 17763 x64 (name:DC01) (domain:inlanefreight.htb) (signing:True) (SMBv1:False)
SMB         <IP_ADDRESS>  445    DC01             [+] inlanefreight.htb\engels:Inlanefreight19!
SMB         <IP_ADDRESS>  445    DC01             [+] inlanefreight.htb\kiosko:SimplePassword3!

Hello,
Thanks for the issue, it is now fixed on the last public release of CrackMapExec https://github.com/mpgn/CrackMapExec v6.0.0
Regards,
mpgn


No install.sh file

File from readme is not there anymore MhcVizPipe_install.sh.
If you follow the readme it is even more confusing because you get the html for the 404.

Yikes, sorry. The installation process changed a while ago and doesn't use the MhcVizPipe_install.sh script anymore. The wiki has the correct instructions. Either I forgot to update the readme or I accidentally uploaded an old version. Thanks for bringing this to my attention.
I will update the readme as soon as possible. In the meantime, please see the instructions here: https://github.com/CaronLab/MhcVizPipe/wiki
Let me know if you run into any more issues.
Best wishes,
Kevin


make BaseAddress optional

Usefull in environment where the BaseAddress is retreived dynamically (service discovery, local load balancing) and still keeping only one HttpClient

Seems okay to me. I assume all tests are still passing?

Yes indeed

@MaxDeg One last thing before we merge it - can you please add a unit test or two around this property?

I think this pull request must be cancelled.
When trying to write some tests I faced an issue with HttpClient. The SendAsync version that refit use is not overridable. This methods perform check or HttpRequestMessage  to ensure the request uri is Absolute or that a BaseAddress is defined on HttpClient.

Is there another approach that would meet your objectives?

Well to be honest I don't see any except setting a fake BaseAddress to override.

Closing for now


Add Mi:dm model

PR Description:
I've updated the README to include Mi:dm (7B).

Model
Link

KT-AI/midm-bitext-S-7B-inst-v1
🤗 huggingface

LGTM Thank you!