Mismatch of GitHub flavored 'IMPORTANT' alert to docusaurus admonition

I think there is a mis-mapping of the mapping of the GitHub IMPORTANT alert.  It appears to be mapping to the :::warning docusaurus admonition.
Consider the following Markdown in a docusaurus site with the remark-github-admonitions-to-directives plugin installed:
> [!NOTE]
> This is a GH flavored `NOTE`

:::note
This is a Docusaurus Admonition `note`
:::

> [!TIP]
> This is a GH flavored `TIP`

:::tip
This is a Docusaurus Admonition `tip`
:::

> [!IMPORTANT]
> This is a GH flavored `IMPORTANT`

:::info
This is a Docusaurus Admonition `info`
:::

> [!WARNING]
> This is a GH flavored `WARNING`

:::warning
This is a Docusaurus Admonition `warning`
:::

> [!CAUTION]
> This is a GH flavored `CAUTION`

:::danger
This is a Docusaurus Admonition `danger`
:::

The above code produces the following output:

I believe it should map to the :::info admonition.

Hi @johnmcase, I think you're right.
My reasoning was that "important" sounds a lot more invasive than "info", so that's why I mapped it to a "warning". I think my reasoning was wrong, and it makes much more sense if all GitHub admonition types map 1:1.

@LuudJanssen would you accept a PR that would allow customizing the mapping? My use case doesn't involve Docusaurus so it would be great to have 1:1 mapping with GitHub's naming.

Hi @damianstasik, yes I would! Did you already think about what this would look like? Like in what form would you pass the mapping to the plugin?

@damianstasik, I don't know if you're still interested in this change, but I added an option to customize the mapping in #9.


formulae: fix CLI arg name

Follow-up to #1041. Fixes:
Error: CLI arg for `skip_strict?` is not declared for this command

Seen in Homebrew/homebrew-core#170246.

Whoops, thanks @ZhongRuoyu!


Nit in scatter_nd_update

"Scatter -> Scatter

@mihaimaruseac
Yes.😅
However I came across this while going through the docs and just raised a PR for the same immediately.
Thanks for the approval.

#53340


Show timestamp in the HP bar chunks

Is it a duplicate question?
I don't think so, but it might with different words.
Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Currently, the HP bar shows chunks (red or green) for the checks that the monitor has performed. If you hover over a chunck, you get the message for that check.
A nice addition could be including a timestamp in the hover message, since that helps identifying it on the list that show bellow the graph.
Describe alternatives you've considered
I currently search the list below for the corresponding report, but if you have a few reds in recently history is not that easy without counting the chuncks.
Additional context

@louislam is there a way to make dayjs globally available in the frontend? Seems a bit weird having to import and extend it everytime we need date formatting.

@louislam is there a way to make dayjs globally available in the frontend? Seems a bit weird having to import and extend it everytime we need date formatting.

For Vue, currently there is a DateTime component.
<DateTime :value="utc date supported by dayjs" />

However, there is no JS function like this, maybe I add it later.

@louislam is there a way to make dayjs globally available in the frontend? Seems a bit weird having to import and extend it everytime we need date formatting.

Just implemented a mixin, also bind to the root. You can pass any dayjs supported datetime formats in utc to the parameter.
this.$root.datetime("2021-08-17 08:40:00");
this.$root.date("2021-08-17 08:40:00");
this.$root.time("2021-08-17 08:40:00");

If possible, use it in "computed", so it can reacts when timezone changed. If not possible, use as a normal function is still acceptable, because users will not always change the timezone.


Design: how should we display subcommands?

I'm not sure how we should be handling subcommands. Should we create separate man pages? Should we create a single man page? It'd be super useful to gather prior art so we can figure out some of the options we have!
Ideally we could find a default that works for most people, and start by implementing that.
Let's start by looking at how git(1) does it.
Example: Git
Git create separate man pages for its subcommands. The git man page has GIT COMMANDS and HIGH LEVEL COMMANDS  sections listing these commands.
Each subcommand has its own page, like you'd expect from any other man page.
Screenshots
git(1) commands section

git-add(1) man page

Right now I'm thinking separate man pages is the right way to go. It's the way cargo does it too, so there's at least some established prior art!


[0.14.2] addToSchema need to run before all plugins

Instead of applying it per-plugin, it should be applied per-file, and only then to execute it.
How to reproduce this?
Use typescript-common and typescript-mongo together, and use a directive.
The schema that typescript-common gets does not contain the directives typescript-mongo adds.
It requires some refactoring, because that means that we need to load the plugin before executing it, collect all addToSchema, built a schema, and then trigger all plugins with the same shared schema.

Updated form version

✔ Parse configuration
  ❯ Generate outputs
    ❯ Generate ./src/gql/types.ts
      ✖ Load GraphQL schemas
        → Failed to load schema
        Load GraphQL documents
        Generate

 Found 1 error

  ✖ ./src/gql/types.ts
    Failed to load schema from ./src/gql/**/schema.ts:

        Type "Pagination" not found in document.
        Error: Type "Pagination" not found in document.

I began seeing this issue after updating from 1.1.0 to 1.15.0, I am using a custom package that I built ts-mongo-codegen. The module exports includes an addToSchema export however it is not found.


The SORTING tab" > Rank Model is disabled.  Doc outdated?

Under "The SORTING tab" > Rank Model, this feature was recently disabled in the query builder dialog.  Does the documentation need to reflect this?

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

ID: 4072d3b5-45f9-746a-2954-64006add2181
Version Independent ID: 66f9429e-6fd5-500b-048f-add3fdb61b5f
Content: Manage result sources - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/manage-result-sources.md
Service: sharepoint-online
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Huh? I raised is because the online documentation is wrong.
https://docs.microsoft.com/en-us/sharepoint/manage-result-sources#the-sorting-tab


Update inactive session icon so it doesn't look cut off.

Fixes #2414
Here's what it looks like in all 3 themes:

👍

👍


Incorrect schema for Microsoft.ServiceFabric/clusters/applications/services

The template example under section Add a new application to your Resource Manager template seems like it might be outdated/incorrect?
For example, I don't see a property instanceCount defined for the schema of type Microsoft.ServiceFabric/clusters/applications/services like is shown here:
"apiVersion": "2019-03-01",
     "type": "Microsoft.ServiceFabric/clusters/applications/services",
     "name": "[concat(parameters('clusterName'), '/', parameters('applicationName'), '/', parameters('serviceName'))]",
     "location": "[variables('clusterLocation')]",
     "dependsOn": [
         "[concat('Microsoft.ServiceFabric/clusters/', parameters('clusterName'), '/applications/', parameters('applicationName'))]"
     ],
    "properties": {
        "provisioningState": "Default",
        "serviceKind": "Stateless",
       "serviceTypeName": "[parameters('serviceTypeName')]",
    ❌"instanceCount": "-1",

The schema definition I'm looking at is here:
https://raw.githubusercontent.com/Azure/azure-resource-manager-schemas/master/schemas/2019-03-01/Microsoft.ServiceFabric.json

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

ID: f979abbb-2b62-ef4b-9cce-48bd08e0d5dc
Version Independent ID: ef3e1a6a-5801-c4bc-277e-7a307cebc8cb
Content: Deploy and upgrade applications and services with Azure Resource Manager
Content Source: articles/service-fabric/service-fabric-application-arm-resource.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: atsenthi

Nevermind, I do see it in the schema, I made a mistake!

@ryandle Glad to know that you have self resolved the issue. Thanks for the update.


Multiple parallel downloads using TransferUtility hanging UI thread

Hi We are using the latest version of Android SDK ie 2.4.4 and are downloading about 20 files.
We see that the max concurrent downloads can be 5 which internal can use max number of HTTP connections which we leave it to the default ie 10.
With this setup the UI thread hangs up completely. If I reduce the number of connections to 3 then it is a little better, but overall the app becomes unresponsive. I could not find a way to reduce overall thread count but I suspect it is the number of connections causing this.
Our block of code is quite simple as shown in Demo, that is use transfer utility to create a download and attach an observer.
We use Cognito Authentication and that part works as expected.

I see the following warning in the Console,
W/art: Long monitor contention event with owner method=int com.amazonaws.mobileconnectors.s3.transferutility.TransferDBBase.update(android.net.Uri, android.content.ContentValues, java.lang.String, java.lang.String[]) from TransferDBBase.java:181 waiters=0 for 288ms

Hi @kvasukib i also have the same problem.
My version is:
compile 'com.amazonaws:aws-android-sdk-s3:2.6.+'
Here is my console:
12-04 14:07:04.515 28325 31431 W art     : Long monitor contention event with owner method=int com.amazonaws.mobileconnectors.s3.transferutility.TransferDBBase.update(android.net.Uri, android.content.ContentValues, java.lang.String, java.lang.String[]) from TransferDBBase.java:181 waiters=0 for 382ms

Hi @sriram-kakarala, Sorry for the delayed reply. We released 2.6.16 with enhancements for stability in TransferUtility. Now you have a flag to customize the size of the thread pool for the transfers. We have documented it here: https://docs.aws.amazon.com/aws-mobile/latest/developerguide/how-to-transfer-files-with-transfer-utility.html#native-transfer-utility-options Please let us know if this works for you.


Tutorial Request for Validation on Original Image Spacing

To achieve the full pipeline, it is better to have a complete tutorial for model training, validation, inference on original spacing (and optional validation on the original spacing). And it would be helpful to add contents in the following tutorial.
https://github.com/Project-MONAI/tutorials/blob/master/3d_segmentation/spleen_segmentation_3d.ipynb
Thanks! @yiheng-wang-nv

Hi @wyli , @Nic-Ma , @rijobro , should I add a new notebook, or update spleen_segmentation_3d.ipynb directly?

I'm not sure that inference using the original spacing is a good idea. If you've trained your network with certain sized voxels, you'll want to infer with those size voxels, too. Do you mean performing the inference on the under-sampled images, and then using the inverse transforms to resample them back to the original spacing?

There is a good example of reversing the model output back to the original spacing. It would be helpful to further calculate the validation accuracy of the original spacing data. Because this will be a fair way to compare models trained at different image spacings. Moreover, it is better to combine the solutions into a complete tutorial rather than fragments (at least for one task). Thank you! @rijobro


Possible NestedSet::moveAsFirst() bug

When use this function it sets wrong lft and rgt values. (in example 12, 13 but it must be 10, 11)

There is no more moveAsFirst method inside behavior for Yii 2 release version. Also working approach was changed so there is now tree integrity garantee. We can complettely forget about troubles like this.

There is no more moveAsFirst method inside behavior for Yii 2 release version. Also working approach was changed so there is now tree integrity garantee. We can complettely forget about troubles like this.

And how now i must move node as first?

And how now i must move node as first?


[Documentation] Add another model to the support table.

The Tovendor Mini Smart Home Camera appears to be a rebranded Yi model. It runs the 9.0.41* firmware and is compatible with the b091qp firmware. It's serial number/ID starts with A001E. Cheers.

Thank you. I will add it to the readme.


How to serve a dspy model

Have a simple model all configured but cannot get it to work on a Flask/FastAPI server. I have looked at similar issues brought up but nothing conclusive on if the problem was addressed. At bottom issue seemed to be something with threading. Can someone give me an example of how to do a simple forward pass with a Flask server?

Are you able to specify your error? I had an issue with Rlock/cloudpickle a few weeks ago and was able to avoid the bug by rewriting the module's deep_copy method.

`from flask import Flask, request
from functools import lru_cache
import os
from dotenv import load_dotenv
load_dotenv()
app = Flask(name)
counter = {"api" : 0}
RAG IMPORT
import dspy
set up llm and retrieval model
ollama_model = dspy.OllamaLocal(model="mistral:7b", max_tokens=500)
colbertv2 = dspy.ColBERTv2(url='http://localhost:8893/api/search')
dspy.settings.configure(lm=ollama_model, rm=colbertv2)
class GenerateAnswer(dspy.Signature):
"""Answer questions as a TA giving hints"""
context = dspy.InputField(desc=" ... ")
question = dspy.InputField()
answer = dspy.OutputField(desc=" ... ")

class RAG(dspy.Module):
def init(self, num_passages=3):
super().init()
    self.retrieve = dspy.Retrieve(k=num_passages)
    self.generate_answer = dspy.ChainOfThought(GenerateAnswer)

def forward(self, question):
    context = self.retrieve(question).passages
    prediction = self.generate_answer(context=context, question=question)
    return dspy.Prediction(context=context, answer=prediction.answer)

rag = RAG()
@lru_cache(maxsize=1000000)
def api_search_query(query):
pred = rag(query)
return {"query": query, "answer": pred.answer, "context": pred.context}

@app.route("/api/search", methods=["GET"])
def api_search():
if request.method == "GET":
counter["api"] += 1
print("API request count:", counter["api"])
return api_search_query(request.args.get("query"))
else:
return ('', 405)
if name == "main":
app.run("<IP_ADDRESS>", port=int(os.getenv("PORT")))
`

as simple as this, flask server just hangs won't launch

@JamesScharf does this help specify anything/was it similar to your error?


remove header on antlr

Coverage remained the same at 88.432% when pulling 616e0f660d9bbd9bf278a4386043708d077c124d on remove-antlr-header into 1f3a5f877720545346652a4c701edb299c91a8ce on async_phase1.


Update to terraform-provider-datadog 2.13.0

https://github.com/pulumi/pulumi-datadog/blob/726371ecb509a928e7fe64c609667b39dbed67da/provider/go.sum#L551
Could you please update it to https://github.com/DataDog/terraform-provider-datadog/releases/tag/v2.13.0 ?
Thanks

Hi @XBeg9
This provider was upgraded already as part of v2.8.0
Can you tell me if this isn't the case?
Paul


Code | Alphabetic Pattern 86

@aryashah2k Kindly assign this one to me.

@aryashah2k Kindly assign this one to me.


krita: 4.4.8 -> 5.0.0, krita-beta: 5.0.0-beta2 -> 5.0.0-beta5

Motivation for this change
Updated krita and krita-beta to the latest version.
Release notes: https://krita.org/en/krita-5-0-release-notes
Tested by drawing the Nix logo.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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

[x] 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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

gmic-qt-krita
krita
krita-beta

Maybe just set krita-beta = krita for now? I don't think there is point in keeping old beta around.

Do you think I should just alias krita-beta? or maybe keep it like how it is now for when the beta channel will be used again?

Given that beta.nix is almost completely a copy of default.nix I`d say we can just delete it and alias for now. Not a strong opinion though.

Given that beta.nix is almost completely a copy of default.nix I`d say we can just delete it and alias for now. Not a strong opinion though.

Alright, I aliased it.


[FEATURE] Adapt filters area to new design (UI)

Is your feature request related to a problem? Please describe.
The current filter area does not support the inclusion of more filters without breaking the design
Describe the solution you'd like
Make more flexible filter area
Describe alternatives you've considered
Make a collapsable filter area and include some styling to make the area lighter

[x] Make area collapsable
[x] Include new styles and sizes
[x] Include filters in records column
[x] Adapt layout to two columns

@leiyre what is the status of this issue? We are working toward new release 1.19.0 and I would like to know if this could be part of that release.


Version 0.25.0

Bumps to 0.25.0
Bug fixes
Attempting to deploy from a directory that only contains secrets.ejson will no longer fail deploy (#416)
Resources which don't exist will be reported as "Not Found" rather than "Unknown"
Other
Update kubeclient gem to 4.2.2. Note this replaces the KubeclientBuilder::GoogleFriendlyConfig class with KubeclientBuilder::KubeConfig (#418)

Is it possible to wait on some of the PRs which appear to be very close to being merged?

Is it possible to wait on some of the PRs which appear to be very close to being merged?

I'd rather cut this as a separate version so that administrators can pick up the apply-errors-can-leak-secrets patch we just merged without also picking up the secret pruning (which may require preparatory action on their part). In general I don't have a problem with releasing multiple versions close together in time as long as each contains a worthy changeset.


Nudge Releases have extra .

Running the Nudge.download recipe with AutoPkg, it uses GitHubReleasesInfoProvider processor to scrape the releases. It seems to expect the version to be in the format of v1.1.* instead of the current v.1.1.* This causes the version to output from the processor as .1.1.*, which doesn't match the apps bundle version.
Would it be possible to update the version tags to not include the extra leading v.1.1* and instead format it as v1.1.*?
Thanks
Rusty

The versioning has always been 4 digits. Sounds like I need to look at the autopkg recipe.

Sorry, it's not the four digits... It's the prefix of '.' between the "v" and "1.1.1.*". Does that make sense?

I see it now. Thanks for reporting this!

I think this should be closed. It looks like I'm using an outdated GitHubReleasesInfoProvider which doesn't properly remove the "v." and instead just returns the [1:] from the tag string.
Please feel free to close this now.
Thanks!

I actually think this is a bug. I've fixed it with https://github.com/macadmins/nudge/pull/253

Here's the AutoPkg discussion on it: https://github.com/autopkg/autopkg/issues/746

So it was my fault :) I've fixed it anyway.

I didn't even notice that until you mentioned it :) Thank you so much! It's working perfectly for me now.

I went backwards in time and re-issued all the old github tags as well. Looks like this issue has existed since the very first nudge release.


Add light sensitivity to config. Add option to draw black to screen when low light level hit.

This is just  a couple of small potential changes.
The first adds configuration of the light sensor to the config file. I have set it to the same value it was hardcoded to (4.0).
The second may only help people with the first batch of grows where the screen does not go off. This adds a new option to the config file (disabled by default) which will just draw a black screen in low light level situations.
I don't know the specifics of the issue with the first batch of grow hats but if its that calling the backlight function does not work (and it works with the new ones then this can be added.

Thanks for taking the time to PR this! I've got to find the time to test and merge - juggling all the things, as always - but your teaser looked good! Drawing a blank screen probably makes sense in either case.
 I think that's the only way to get them moving.
Something like:
git commit --allow-empty -m "Trigger GitHub actions"

Done. Its alive!


Add Rust bindings for Flash Translation Layer

I added the Rust binding module for creating a Rust-based Flash Translation Layer module.

Need to create a new repository.
https://kornel.ski/rust-sys-crate


Exit with nonzero code on failure

This follows the example at https://github.com/mochajs/mocha/wiki/Using-mocha-programmatically
Fixes #102, though I haven't figured out how to write a test to check for the correct behaviour.

Add via slightly different implementation on master. Cheers!


Update TargetPlatform enum to include linux_arm64 and windows_arm64 entries

These will be required to specify arm64 build targets. The desktop_device base class should also be updated to return this instead of the hardcoded x64 platform arch, using the detection logic from https://github.com/flutter/flutter/issues/69213

Hi this should work to determine the linux architecture but i'm not really sure how to test
@override
  HostPlatform get hostPlatform {
    if (_hostPlatform == null) {
      String unamePath;
      String getconfPath;
      if (which('uname') == null) {
        // Fallback to known install locations.
        for (final String path in <String>[
          '/usr/bin/uname',
          '/bin/uname',
        ]) {
          if (_fileSystem.isFileSync(path)) {
            unamePath = path;
          }
        }
      } else {
        unamePath = 'uname';
      }
      // This if is redundant because of the else
      if (unamePath == null) {
        throwToolExit('uname not found. Try adding it to your PATH environment variable.');
      }
      if (which('getconf') == null) {
        // Fallback to known install locations.
        for (final String path in <String>[
          '/usr/bin/getconf',
          '/bin/getconf',
        ]) {
          if (_fileSystem.isFileSync(path)) {
            getconfPath = path;
          }
        }
      } else {
        getconfPath = 'getconf';
      }
      // This if is redundant because of the else
      if (getconfPath == null) {
        throwToolExit('getconf not found. Try adding it to your PATH environment variable.');
      }
      RunResult arm64Check = _processUtils.runSync(<String>[unamePath, '- m']);
      // On arm64 "uname -m will output aarch64 or an armv8 variant if processor is arm64"
      if (arm64Check.exitCode == 0 && (arm64Check.stdout.trim().startsWith('arm') || arm64Check.stdout.trim().startsWith('aarch')))
        // getconf is used to determine whether the kernel is 32 or 64 bit
        arm64Check = _processUtils.runSync(<String>[getconfPath, 'LONG_BIT']);
      if (arm64Check.exitCode == 0 && arm64Check.stdout.trim() == "64"){
        _hostPlatform = HostPlatform.linux_arm64;
      }
    } else {
      _hostPlatform = HostPlatform.linux_x64;
    }
    return _hostPlatform;
  }

Hi this should work to determine the linux architecture but i'm not really sure how to test
@override
  HostPlatform get hostPlatform {
    if (_hostPlatform == null) {
      String unamePath;
      String getconfPath;
      if (which('uname') == null) {
        // Fallback to known install locations.
        for (final String path in <String>[
          '/usr/bin/uname',
          '/bin/uname',
        ]) {
          if (_fileSystem.isFileSync(path)) {
            unamePath = path;
          }
        }
      } else {
        unamePath = 'uname';
      }
      // This if is redundant because of the else
      if (unamePath == null) {
        throwToolExit('uname not found. Try adding it to your PATH environment variable.');
      }
      if (which('getconf') == null) {
        // Fallback to known install locations.
        for (final String path in <String>[
          '/usr/bin/getconf',
          '/bin/getconf',
        ]) {
          if (_fileSystem.isFileSync(path)) {
            getconfPath = path;
          }
        }
      } else {
        getconfPath = 'getconf';
      }
      // This if is redundant because of the else
      if (getconfPath == null) {
        throwToolExit('getconf not found. Try adding it to your PATH environment variable.');
      }
      RunResult arm64Check = _processUtils.runSync(<String>[unamePath, '- m']);
      // On arm64 "uname -m will output aarch64 or an armv8 variant if processor is arm64"
      if (arm64Check.exitCode == 0 && (arm64Check.stdout.trim().startsWith('arm') || arm64Check.stdout.trim().startsWith('aarch')))
        // getconf is used to determine whether the kernel is 32 or 64 bit
        arm64Check = _processUtils.runSync(<String>[getconfPath, 'LONG_BIT']);
      if (arm64Check.exitCode == 0 && arm64Check.stdout.trim() == "64"){
        _hostPlatform = HostPlatform.linux_arm64;
      }
    } else {
      _hostPlatform = HostPlatform.linux_x64;
    }
    return _hostPlatform;
  }

I'm working to support arm64 Linux desktop at the moment. #61221

I'm working to support arm64 Linux desktop at the moment. #61221

This is done.


Fix typos

Description
Found two misspellings.
Motivation and context
Show what typos can do.
Checklist:

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[ ] I have created a branch for this patch/feature.
[ ] I have added tests to cover my changes.
[ ] If my change requires a change to the documentation, I have updated it accordingly.

There is a test called testFindWithMisspelingHandler (with 1 l)

Good catch, thanks !

➕ There is a test called testFindWithMisspelingHandler (with 1 l)

Quite ironic for a spellchecking lib :sweat_smile:

Glad to contribute.

You find tools for caring about byte in this repository https://github.com/szepeviktor/byte-level-care/tree/master/.github/workflows


SiteId placeholder is not working in TelemetryTopicTemplate using Dapr transport

Describe the bug
If I set the --SiteId commandline parameter to "xyz" and use it in the --TelemetryTopicTemplate commandline parameter like "events.{SiteId}.{PublisherId}" it does not get replaced. Its still SiteId. PublisherId works as expected.
To Reproduce
Steps to reproduce the behavior:

Set commandline parameters to
--SiteId=xyz
--PublisherId=0815
--TelemetryTopicTemplate=events.{SiteId}.{PublisherId}
--DefaultTransport=Dapr
Run OpcPublisher

Expected behavior
The message topic should be "events.xyz.0815" but is "events.SiteId.0815"
Additional context
Using mcr.microsoft.com/iotedge/opc-publisher:2.9.4-preview3

Fix will be in preview4.


Undefine Object#send method to call JavaScript send method

Background
The following ruby.wasm code will cause an error.
ws = JS.global[:WebSocket].new("ws://localhost:9292")
ws[:onopen] = -> (event) {
  ws.send("Hello")
}

Error: /bundle/gems/js-2.6.2/lib/js.rb:184:in `method_missing': undefined method `Hello' for an instance of JS::Object (NoMethodError)

This error is caused by the Object#send method being called instead of the send method of the JavaScript WebSocket object.

Given that method_missing API is just a shorthand of JS::Object#[], it might be too much to undefine such conventional methods. Could you collect some case studies on how other similar method_missing users (e.g. ActiveRecord, PyCall.rb, etc...) address this kind of name conflict?

I have collected case studies.
In ActiveRecord, the send column is defined as a method using define_attribute_methods. In other words, the Object#send method is overridden.
In PyCall, if there is a name conflict with the send method, the Object#send method is given priority. To call the send method of a Python object, use PyColl.getattr as follows.
require 'pycall'
client = PyCall.import_module 'http.client'
conn = client.HTTPConnection.new('www.example.com')
PyCall.getattr(conn, 'send').("GET / HTTP/1.0\r\n\r\n".b)

I like the way Rails does it. So I defined the JS::Object#send method.

Sorry for my late response. I have been thinking about this, how about inheriting BasicObject instead of Object? If we do that, we don't need to care about most of conventional methods (not only for send) except for a few like send.
It slightly breaks API compatibility but shouldn't be a big deal considering its use cases.

how about inheriting BasicObject instead of Object?

I think it's an interesting idea. I'll try to see how it's implemented.

It looks like it can be successfully implemented by inheriting the BasicObject.
I will create a new pull request with a new implementation.
I close this pull request.


[cloud_firestore] metadata is missing for QuerySnapshot class

Use case
I have disabled the persistent mode in my app and I am looking for solution to identify unsuccessful firestore read.
Seems like firestore always returns successful result (via Future / Stream) in query case.
The metadata should contain isFromCache property which can indicate if the read was failed or not.
The metadata is already implemented in other sdks such as js,dart (html) and more (see references)
https://pub.dev/documentation/firebase/latest/firebase_firestore/QuerySnapshot-class.html
https://firebase.google.com/docs/reference/js/firebase.firestore.QuerySnapshot

Proposal
Please implement it, Thanks a lot

@shasaf1986
The issue at https://github.com/flutter/flutter/issues/32789 has been closed and moved here. Future collaboration on this issue will be done here.

Hey all, as part of our on-going work for #2582, this has been resolved in our Firebase Firestore rework (#2913) (metadata is now available on the QuerySnapshot class) - which has been merged into master. We'll look at publishing some prereleases in the next few days. Thank you


Unable to authenticate on Mongo DB - 'fluent-plugin-mongo' version '0.7.15'

Hi,
I am unable to authenticate against the MongoDB instance. It says - "fluent/log.rb:382:fatal: Could not authorize user admin on database admin". Pasting the match tag for your reference.
I was able to connect to the same mongo instance using Studio 3T providing the same credentials as below and authentication is happening perfectly fine.
Match:
<match mongo.**>
@type mongo
user admin
password ********
database admin
host abcd23756
port 27017
collection docker

Note: I have docker image in my local and working fine with the local mongo instance which has no credentials. But the issues arrises when authenticated against other mongo instance that needs credentials.
Error Logs:
2019-03-19 05:16:23 +0000 [info]: #0 fluent/log.rb:322:info: starting fluentd worker pid=35 ppid=6 worker=0
2019-03-19 05:16:23 +0000 [fatal]: #0 fluent/log.rb:382:fatal: Could not authorize user admin on database admin.
2019-03-19 05:16:23 +0000 [info]: fluent/log.rb:322:info: Worker 0 finished unexpectedly with status 1
2019-03-19 05:16:23 +0000 [info]: fluent/log.rb:322:info: gem 'fluent-plugin-mongo' version '0.7.15'
2019-03-19 05:16:23 +0000 [info]: fluent/log.rb:322:info: gem 'fluentd' version '1.2.4'
2019-03-19 05:16:23 +0000 [info]: fluent/log.rb:322:info: adding match pattern="mongo.**" type="mongo"
** Notice: The native BSON extension was not loaded. **
  For optimal performance, use of the BSON extension is recommended.

  To enable the extension make sure ENV['BSON_EXT_DISABLED'] is not set
  and run the following command:

    gem install bson_ext

  If you continue to receive this message after installing, make sure that
  the bson_ext gem is in your load path.

This doesn't follow contribution guide. This is fluentd core repository.
Don't post 3rd party plugin issue: https://github.com/fluent/fluentd/blob/master/CONTRIBUTING.md


[lecture9-2] [0619] Inquiry on Parameter Reduction in CNNs

I am writing to inquire about the specific reasons for the reduction in the number of parameters as discussed on page 65 of the lecture notes on Computer Vision.
In the lecture, it is mentioned that the number of parameters decreases significantly in certain convolutional neural network (CNN) architectures. I would like to understand this in more detail and see some specific examples of the reduction in parameter count.
Could you please elaborate on the following points regarding the reduction in the number of parameters?

Group Convolution: How does the use of group convolution contribute to the reduction in the number of parameters? Could you provide an example of how this is implemented and its impact on the overall network efficiency? Specifically, I would like to see a comparison of the number of parameters before and after applying group convolution.

Depthwise Convolution: The lecture notes mention depthwise convolution as a method to approximate standard convolution. How does this approach reduce the parameter count, and what are the trade-offs in terms of performance and accuracy? Again, a numerical example showing the reduction in the number of parameters would be very helpful.

@zxcv3296 Thank you for the comment! (and would you send your name by mail<EMAIL_ADDRESS>Group convolution is an important technique for reducing the number of parameters in a convolutional neural network. By using N groups, group convolution approximates the standard convolution operation with N smaller convolutions, each having input and output channels equal to  C_in/N and C_out/N, respectively. Consequently, each smaller convolution has (1/N)^2 the parameters of a standard convolution. Since there are N such smaller convolutions, the total number of parameters is reduced to 1/N of the original.
Depthwise convolution is a special case where N=C_in. Therefore, the total reduction in parameters is 1/C_in. This method significantly reduces the number of parameters but does not mix information across channels, so it is usually combined with pointwise convolution (1x1 convolution).
Have a nice vacation!


Remove Faraday User-Agent headers from mocks

This should clear up errors related to our having upgraded Faraday.

Looks good. When the build finishes, I'll :merge:.


chore: remove open collective entry.

The open collective link (https://opencollective.com/serverless-dotenv-plugin) is broken. I'm thinking of removing it from this file and the repo page unless you have concerns @colynb.

Yes thanks. I turned that service off a while ago.
On Fri, Feb 5, 2021 at 7:52 PM Mark Tse<EMAIL_ADDRESS>wrote:

@neverendingqs https://github.com/neverendingqs requested your review
on: #85
https://github.com/neverendingqs/serverless-dotenv-plugin/pull/85
chore: remove open collective entry..
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/neverendingqs/serverless-dotenv-plugin/pull/85#event-4298672934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIAEJA4PEGOAFKUPWBGO3LS5SVFLANCNFSM4XFZGPQQ
.


Improve tuple & array inter-assignability

Purpose
$subject
Fixes #17891
Remarks
#17891

Codecov Report

Merging #18333 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18333   +/-   ##
=======================================
  Coverage   15.06%   15.06%
=======================================
  Files          48       48
  Lines        1248     1248
  Branches      192      192
=======================================
  Hits          188      188
  Misses       1046     1046
  Partials       14       14

Continue to review full report at Codecov.

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


Max number of iteration reached for pairwise_align()

I always got this warning when using pairwise_align() and I got the same warning when setting pi12 = pst.pairwise_align(slice1, slice2,numItermax=1000000)
and my two consecutive slices were very poorly aligned, Is that the problem?
RESULT MIGHT BE INACURATE
Max number of iteration reached, currently 100000. Sometimes iterations go on in cycle even though the solution has been reached, to check if it's the case here have a look at the minimal reduced cost. If it is very close to machine precision, you might actually have the correct solution, if not try setting the maximum number of iterations a bit higher

Hi,
A poor alignment is most likely not due to the max number of iterations. The POT solver is likely getting stuck at a local minima. I would recommend trying to vary levels of alpha (e.g. `alpha = 0.1, 0.01, 0.001).
Are your spatial slices slightly symmetric by chance? Intuitively, the more symmetric the shape of the tissue the harder it is to find the optimal rotation based solely spatial information. For example, if the spots between layers formed a perfect square and you were only looking at spatial coordinates, you'd have no idea which 90 degree rotation would give the optimal rotation. This is also why in the case above you could see better results at lower alpha (giving more weight to gene expression).
In situations like these, if you know the proper orientation (to some degree), another option would be to try roughly rotating the spatial coordinates yourself before running PASTE.


RossTalk module GPI issue

Ross module sends incorrect GPI data. When sending "1" as the GPI, it sends "01" according to Wireshark. So it seems 1-9 are being sent as 01-09. RossTalk should accept single digits as a GPI. I'm getting this info directly from a Ross employee who says that GPI should be alphanumeric, which means "01" does not equal "1".

It only seems to except numbers and not letters.
There is no function to "Trigger GPI By Name" that I can see.

@CoBot3000 which version of Companion are you running?

@josephdadams I am running v1.3.0

@CoBot3000 try 1.4 or 2.0.

https://builds.bitfocus.io/companion/branch/companion-1.4.0-backport-c6a13a9-osx.zip
https://builds.bitfocus.io/companion/branch/companion-1.4.0-backport-c6a13a9-win64.exe
—
W
On Wed, 11 Sep 2019 at 22:38, Joseph Adams<EMAIL_ADDRESS>wrote:

@CoBot3000 https://github.com/CoBot3000 try 1.4 or 2.0.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bitfocus/companion-module-rosstalk/issues/12?email_source=notifications&email_token=AADIDEVKEQJSZETOUJBXAJTQJFJMRA5CNFSM4IO23C32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6P2GXI#issuecomment-530555741,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADIDEUAHFHD7QWGWKC5763QJFJMRANCNFSM4IO23C3Q
.

--
William Viker

Awesome thanks guys.. I have installed v2.0.0
It has the function where you can take the arrows out which is what I need.
I'm sorry if I sound a little noobish.
Now, I have put in the Ultrix Salvo name and it doesn't seem to work. I have tried with the number in the name and in the parameter.
Any thoughts?
Have added image:


chore(deps): 1.3.x fix fix CVE-2024-45590, CVE-2024-43799, CVE-2024-45296

chore(deps): 1.3.x fix fix CVE-2024-45590, CVE-2024-43799, CVE-2024-45296
Description
Please explain the changes you made here.
Manual backport of https://github.com/janus-idp/backstage-showcase/pull/1619
WIP:  Waiting on https://github.com/janus-idp/backstage-showcase/pull/1527 to be backported
Which issue(s) does this PR fix

Fixes #?
https://issues.redhat.com/browse/RHIDP-3917
https://issues.redhat.com/browse/RHIDP-3945
https://issues.redhat.com/browse/RHIDP-3897

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Manual cherrypick for #1527 can be found here

+1, though I notice in addition to the CVEs you've also bumped to "express": "4.21.0".

Just noting here, that many of the CVEs are dependencies of express so it was just a matter of deleting the old version and re-running yarn install to upgrade to the latest version.


Allow admins to create reputation groups in the dashboard

Remind me, when is this due? Not during the current grant period right?

Remind me, when is this due? Not during the current grant period right?

In Dec/Jan, after we complete Goals 2 and 4.


Test case with Serty data

Each Laitos acts as a depot and has one truck+trailer available (1+2 container spots).
In the simulation, containers on the sites need to be deducted.
For each order timestamp, change container at the site from 0 to 97% minus possible weekend. Linear increase of waste 1% per day.
Test response time effect wrt number of kilometers = when bringing the order, initiate with 95% instead of 97% (or even 93%).
(Keeping in mind, what we actually looking for is the HCT routes, locations)


Set up Discord

What if you could vote/battle IN DISCORD?!?!?! But first pass would just be a community for folks

Done!


SimpleNeo4j\Client candidate for builder pattern

Hello,
Typodeaf from devRant here.
Looking at the test for the Client I saw a possibility to implement a builder for the Client.
I think this would contribute to readability, especially since I have a feeling the args list will grow.
Wdyt?
I'm imagining defining different Clients like this:
$builderOne = new SimpleNeo4j\ClientBuilder('some_host');
$client = $builderOne->username('some_user')->password('pw')->build();

$builderTwo = new SimpleNeo4j\ClientBuilder('some_host');
$client = $builderTwo->guzzle($http_client)->build();

or
$client = SimpleNeo4j\Client::createBuilder('some_host')->username('some_user')->password('pw')->build();

$client = SimpleNeo4j\Client::createBuilder('some_host')->guzzle($http_client)->build();

Thanks for the suggestion!
I was circling back and starting to make tickets for this tonight. I think you're definitely right in that the args list for the constructor for client will get out of hand.
I think the best approach might be to do it Guzzle-style, where anything with over a few parameters takes a config array. Builder is pretty cool, but I think for configuring a database client it's easier to be able to pass in an array of config options that you can store in your app config like many apps do.
Please let me know your thoughts.

Yes that is a good point. A config array is probably better, especially if you need re-instantiate. What I prefer about the builder is that it allows for the IDE to provide hints about the available configuration parameters without having to look into the constructor manually and that you don't need any constructor code to ensure the required configuration is set.
But now that I think about it you can also have a default array parameter with all the available configuration keys set to null.
I think both of them will serve equally well for this purpose and, as you mentioned, sticking to the Guzzle standard probably makes more sense for the library consumers.


Tools for interaction: Hide Controller On Grab and Use Only If Grabbed

I'm working on a script to extend VRTK_InteractableObject for the interaction with other objects (should I put the scrpt in Script/Tools/VRTK_Tool.cs?), along with some animation and sound effects.
Here you can see the example scene developed to test this feature (not yet ready for a PR):

Two main issues arose with this work, the first one regards hiding the controller on grab only for some objects, the second one preventing an object to be used if not grabbed.
Hiding the controller on grab only for some objects (Hide Controller On Grab)
There are different types of objects, those that does fit into the controller shape (e.g. drills, electrical screwdrivers, light sabers...), and those that don't fit (e.g. large tools or simple movable objects), according to the correlation between the controller to the virtual object shape (and buttons).
In each of these types the controller is expected to be hidden (objects that fit) or not (objects that don't fit).
Why don't move Hide Controller On Grab (hideControllerOnGrab) from Grabbing Interactable Object (VRTK_InteractGrab) to Interactable Object (VRTK_InteractableObject)?
For example in VRTK_InteractGrab this could coded like this:
var grabbedObjectScript = grabbedObject.GetComponent<VRTK_InteractableObject>();
...
hideControllerOnGrab = grabbedObjectScript.hideControllerOnGrab;

Another chance is to define the default behaviour for the controller and let the grabbed object override this if a specific behaviour is defined:
var grabbedObjectScript = grabbedObject.GetComponent<VRTK_InteractableObject>();
...
var grabbedToolScript = grabbedObjectScript as VRTK_Tool;
if (grabbedToolScript!=null) hideControllerOnGrab = grabbedToolScript.hideControllerOnGrab;

This let the controller (the VRTK_InteractGrab script) check if the grabbed object is a tool, in that case the flag "Hide Controller On Grab" of the tool would be used instead of the one of VRTK_InteractGrab (with this last solution should I use the same name hideControllerOnGrab as in VRTK_InteractGrab?).
Preventing an object to be used if not grabbed (Use Only If Grabbed).
This is a feature required by many real tools (you should not use a drill if you didn't grab it).
I think this is more a generic feature, so I'd put it into VRTK_InteractableObject rather than in the tool script (e.g. VRTK_Tool).
For example put in DoStartUseObject() (in VRTK_InteractUse):
VRTK_InteractableObject vrobj = touchedObject.GetComponent<VRTK_InteractableObject>();
if (vrobj.useOnlyIfGrabbed && !vrobj.IsGrabbed()) return;

There's already an issue raised for overriding hide controller on certain objects:
https://github.com/thestonefox/SteamVR_Unity_Toolkit/issues/79
So I'd say your solution shouldn't be an extension into your own thing, make it part of the Interactable Object as this should be a core feature.
Also, same for Use only If Grabbed make this a core feature as it's useful for everyone.
That should answer the original question of where to put the extended file :) don't put it anywhere and just do the work on the interactable object :)

There's already an issue

Yes, I missed that issue, now I read it and proposed another option.

where to put the extended file :) don't put it anywhere and just do the work on the interactable object :)

I can apply the 2 changes (Hide Controller On Grab and Use Only If Grabbed) to VRTK_InteractableObject, but what about animations sounds and so on? Shuld I put everything into VRTK_InteractableObject.cs?
In the Inspector we will find a very long list of features, even if not used. This is related to the issue #60. By the way, am I missing some work already in progress for this topic?
Anyway I arranged things to let the script use a feature only if the corresponding item is set. For example a scaling is applied only if the related Transform is set in the Inspector.
With this structure an option could be to write an Editor script devoted to make each section disappear if the related target is not set (e.g. if no Transfom is set for scaling do not display scaling values, etc.).

If you're making an example with animations and sounds then put the script inside Examples/Resources/Scripts
I wouldn't suggest having animations and sound effects as a default core feature.
You're right about the crazy long list of params. I think this can be resolved with a custom inspector and put things into foldouts and hiding things as you say.
I keep meaning to get round to looking into how custom inspectors work and writing it. So if that's something you want to take a look at as well then that would be great!

Got it!
Anyway I will think about the inspector later, I'm going to work on the VRTK_InteractableObject script now.

Cool :)
Yeah the inspector is one of those things that probably needs doing, just never has the time for the love and attention it deserves :)

I need to add also two options in VRTK_InteractableObject to disable the teleport when grabbing or using an object:
        public bool disableTeleportOnGrab = false;
        public bool disableTeleportOnUse = false;

what do you think about it?
In case you agree with this change should I add it to the same PR where I put "Hide Controller On Grab and Use Only If Grabbed"?

I started with this PR #348, the example with sounds and animation will come with another PR.

fixed https://github.com/thestonefox/SteamVR_Unity_Toolkit/pull/348


Denon quick select state text

Requirements

#261
Add state text for current quick select mode.

Testing concluded that it is not possible to detect the current quick select state using the Denon AVR integartion.


Update window function: lead, lag

Changelog category (leave one):

New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...

Information about CI checks: https://clickhouse.tech/docs/en/development/continuous-integration/

Can one of the admins verify this patch?

test this please with 906

test this please with 906

@lgbo-ustc 代码冲突需要解决一下

move to a new pr #322


Fix example in readme

Fix example

A http.Client parameter is only necessary on version 1 of Elastic. Version 2 has a different way of configuring settings. The Wiki explains that. Are you still on version 1?

Am 05.04.2015 um 21:41 schrieb Artur Cygan<EMAIL_ADDRESS>Fix example
You can view, comment on, or merge this pull request online at:
https://github.com/olivere/elastic/pull/60
Commit Summary
Update README.md
File Changes
M README.md (2)
Patch Links:
https://github.com/olivere/elastic/pull/60.patch
https://github.com/olivere/elastic/pull/60.diff
—
Reply to this email directly or view it on GitHub.


Enable indexing

Enable usage with indexing together with calling the get method, example:
 shelf.get(5) == self[5]

Fixed in PR #8


Rule to suggest using attributes if available

Per https://github.com/openshift/openshift-docs/pull/60604#issuecomment-1565370504
I can't tell if this is working locally or not, but figured I will put it up at least

Last commit gets basically everything working right, except for things like Red Hat Advanced Cluster Management (RHACM). Will poke at it further later.

Now we're getting funky. We want to catch terms that are not preceded by other terms. Regex required. See for example: https://github.com/redhat-documentation/vale-at-red-hat/blob/main/.vale/styles/RedHat/CaseSensitiveTerms.yml#L17

So, most recent commit works for me, with this little bit of weirdness:
When I hit Ctrl+S in the invalid test file, it alternates which rule it grabs, seemingly randomly. So like in the screenshot above, sometimes it will underline the full string Logical volume manager storage (LVM Storage), sometimes just the LVM Storage part. I haven't done enough with this to know if that is normal? If not, I guess yeah needs moar regex. I can come back to it to learn more :nerd_face:

So, most recent commit works for me, with this little bit of weirdness:
When I hit Ctrl+S in the invalid test file, it alternates which rule it grabs, seemingly randomly. So like in the screenshot above, sometimes it will underline the full string Logical volume manager storage (LVM Storage), sometimes just the LVM Storage part. I haven't done enough with this to know if that is normal? If not, I guess yeah needs moar regex. I can come back to it to learn more nerd_face

Need to work on the regex. Will have a peep again tomorrow.

So the problem now is that the Vale regex terms in this rule are giving inconsistent results. open a cmd at the fixtures folder, and run vale . a few times. you will see that you get different results.
✔ 0 errors, 0 warnings and 82 suggestions in 2 files.
✔ 0 errors, 0 warnings and 76 suggestions in 2 files.
etc
The regex hits need to be unique or this won't work.

Basically, it's the Venn Diagram problem You have various terms that are returning hits inside other terms. Vsle doesn't know what to do.

Per conversation today, will comment out some problematic ones so we can get helpful ones working. The more complex things can be a target for improvement later.

Ok, I have commented out the ones that I think resolve the conflicts best. I refreshed the save a bunch of times and no longer got different hit numbers - looks consistent to me but definitely a thing worth a second check because it seems like some easy stuff to miss.

Yes please - squash them

@jeana-redhat @aireilly - how is this one progressing? I think it would be useful to get it merged (I'd also like to talk about it at an upcoming Vale talk for a VF2F for the Telco team)

@jeana-redhat @aireilly - how is this one progressing? Does it need further reviews? I think it would be useful to get it merged (I'd also like to talk about it at an upcoming Vale talk for a VF2F for the Telco team)

I ran out of time to work on it, but I should probably just carve out some time and try to get it through. I don't recall the state of things now but will refresh myself this week and see about getting it done

IIRC, last time I reviewed there was one or two small things left. Very close to being ready IMO.

merged in #692


Refactor Languages into Defintions and extend english definitions

Coverage decreased (-0.8%) to 99.223% when pulling cbd1bb245fd5f925a452502bd83cba2bbde25b22 on feature/refactor-language-definitions into 8da40c7c093740f3514f826ead7d403fdc3ccb64 on master.


Somewhere between 5th Oct and 21st Oct, go mod tidy stopped getting run

Package ecosystem
Go mod
Package manager version
Go 1.17
Language version
Go 1.17
Manifest location and content prior to update

dependabot.yml content
version: 2

updates:
  - package-ecosystem: "gomod"
    directory: "/"
    schedule:
      interval: "weekly"

Updated dependency

What you expected to see, versus what you actually saw

Native package manager behavior

Images of the diff or a link to the PR, issue or logs

🕹 Bonus points: Smallest manifest that reproduces the issue

Somewhere between the 5th Oct and the 21st Oct, dependabot seems to have stopped running go mod tidy for PRs to our repos. I can't see anything significant that changed in our repos. It appears to be happening for both internal projects and public ones.
Last PR where it behaved properly (5th Oct): https://github.com/circleci/ex/pull/92
First PR where it stopped tidying (21st Oct):  https://github.com/circleci/ex/pull/114
Whole bunch of PRs where I had to manually go mod tidy:

https://github.com/circleci/ex/pull/122
https://github.com/circleci/ex/pull/126
https://github.com/circleci/ex/pull/133
https://github.com/circleci/ex/pull/134
https://github.com/circleci/ex/pull/135

Thanks for reporting this @pete-woods, my first guess is that dependabot encounters an error on these repo's when running go mod tidy, which we currently ignore, let me see if that's the case and if I can find out what the errors are

I'm not seeing any errors in the public-facing logs, FWIW: https://github.com/circleci/ex/network/updates/235635214

I'm not seeing any errors in the public-facing logs, FWIW: https://github.com/circleci/ex/network/updates/235635214

Yeah we currently swallow those entirely, it would probably be nice to at least log those when they occur

I guess it must do (or at least use the latest Go), as we have successfully generated PRs while we were using Go 1.17 in the mod file.

We currently run dependabot on go 1.17 fwiw

FYI, I tried running go mod tidy in a vanilla Go docker image, with a commit I know that dependabot failed on:
% docker run --rm -v $PWD:/go/src -ti golang:1.17 bash -c 'cd src && go mod tidy'
go: downloading gotest.tools/v3 v3.0.3
go: downloading github.com/cenkalti/backoff/v4 v4.1.2
go: downloading github.com/honeycombio/beeline-go v1.3.1
go: downloading golang.org/x/sync v0.0.0-20201020160332-67f06af15bc9
go: downloading github.com/gin-gonic/gin v1.7.4
go: downloading github.com/DataDog/datadog-go v4.8.3+incompatible
go: downloading github.com/rollbar/rollbar-go v1.4.2
go: downloading github.com/hellofresh/health-go/v4 v4.5.0
go: downloading github.com/jolestar/go-commons-pool/v2 v2.1.1
go: downloading github.com/makasim/amqpextra v0.16.4
go: downloading github.com/streadway/amqp v1.0.1-0.20200716223359-e6b33f460591
go: downloading github.com/alecthomas/kong v0.2.18
go: downloading github.com/honeycombio/dynsampler-go v0.2.1
go: downloading github.com/honeycombio/libhoney-go v1.15.6
go: downloading github.com/google/go-cmp v0.5.6
go: downloading github.com/klauspost/compress v1.13.6
go: downloading github.com/jmoiron/sqlx v1.3.4
go: downloading github.com/lib/pq v1.10.3
go: downloading github.com/go-redis/redis/v8 v8.11.4
go: downloading github.com/michaelklishin/rabbit-hole/v2 v2.11.0
go: downloading github.com/Microsoft/go-winio v0.5.0
go: downloading go.opentelemetry.io/otel v1.0.0
go: downloading go.opentelemetry.io/otel/trace v1.0.0
go: downloading github.com/pkg/errors v0.9.1
go: downloading github.com/felixge/httpsnoop v1.0.2
go: downloading github.com/gin-contrib/sse v0.1.0
go: downloading github.com/mattn/go-isatty v0.0.14
go: downloading github.com/golang/protobuf v1.5.2
go: downloading github.com/stretchr/testify v1.7.0
go: downloading github.com/golang/mock v1.4.4
go: downloading go.uber.org/goleak v1.0.0
go: downloading github.com/fortytw2/leaktest v1.3.0
go: downloading gopkg.in/alexcesaro/statsd.v2 v2.0.0
go: downloading github.com/facebookgo/muster v0.0.0-20150708232844-fd3d7953fd52
go: downloading github.com/vmihailenco/msgpack/v5 v5.3.4
go: downloading github.com/DataDog/zstd v1.4.8
go: downloading github.com/go-sql-driver/mysql v1.6.0
go: downloading github.com/mattn/go-sqlite3 v2.0.3+incompatible
go: downloading golang.org/x/xerrors v0.0.0-20200804184101-5ec99f83aff1
go: downloading github.com/onsi/ginkgo v1.16.4
go: downloading github.com/onsi/gomega v1.16.0
go: downloading github.com/cespare/xxhash/v2 v2.1.2
go: downloading github.com/dgryski/go-rendezvous v0.0.0-20200823014737-9f7001d12a5f
go: downloading golang.org/x/sys v0.0.0-20210910150752-751e447fb3d0
go: downloading google.golang.org/grpc v1.40.0
go: downloading github.com/json-iterator/go v1.1.10
go: downloading github.com/ugorji/go/codec v1.1.7
go: downloading gopkg.in/yaml.v2 v2.4.0
go: downloading github.com/go-playground/validator/v10 v10.4.1
go: downloading google.golang.org/protobuf v1.26.0
go: downloading golang.org/x/lint v0.0.0-20190930215403-16217165b5de
go: downloading github.com/facebookgo/clock v0.0.0-20150410010913-600d898af40a
go: downloading github.com/facebookgo/limitgroup v0.0.0-20150612190941-6abd8d71ec01
go: downloading github.com/davecgh/go-spew v1.1.1
go: downloading github.com/pmezard/go-difflib v1.0.0
go: downloading github.com/stretchr/objx v0.2.0
go: downloading gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b
go: downloading github.com/vmihailenco/tagparser/v2 v2.0.0
go: downloading github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd
go: downloading github.com/modern-go/reflect2 v1.0.1
go: downloading gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c
go: downloading github.com/go-playground/universal-translator v0.17.0
go: downloading github.com/leodido/go-urn v1.2.0
go: downloading golang.org/x/crypto v0.0.0-20210920023735-84f357641f63
go: downloading github.com/go-playground/assert/v2 v2.0.1
go: downloading github.com/go-playground/locales v0.13.0
go: downloading github.com/facebookgo/ensure v0.0.0-20200202191622-63f1cf65ac4c
go: downloading github.com/nxadm/tail v1.4.8
go: downloading golang.org/x/net v0.0.0-20210913180222-943fd674d43e
go: downloading github.com/kr/pretty v0.2.1
go: downloading golang.org/x/tools v0.0.0-20201224043029-2b0845dc783e
go: downloading github.com/facebookgo/stack v0.0.0-20160209184415-751773369052
go: downloading github.com/facebookgo/subset v0.0.0-20200203212716-c811ad88dec4
go: downloading gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7
go: downloading github.com/kr/text v0.1.0
go: downloading golang.org/x/text v0.3.7
go: downloading github.com/fsnotify/fsnotify v1.4.9
go: downloading github.com/cespare/xxhash v1.1.0
go: downloading github.com/ugorji/go v1.1.7
pete@Petes-MBP ex % g diff
diff --git a/go.sum b/go.sum
index 2fd4410..356e484 100644
--- a/go.sum
+++ b/go.sum
@@ -20,8 +20,6 @@ github.com/armon/consul-api v0.0.0-20180202201655-eb2c6b5be1b6/go.mod h1:grANhF5
 github.com/beorn7/perks v0.0.0-20180321164747-3a771d992973/go.mod h1:Dwedo/Wpr24TaqPxmxbtue+5NUziq4I4S80YR8gNf3Q=
 github.com/beorn7/perks v1.0.0/go.mod h1:KWe93zE9D1o94FZ5RNwFwVgaQK1VOXiVxmqh+CedLV8=
 github.com/bradfitz/gomemcache v0.0.0-20190913173617-a41fca850d0b/go.mod h1:H0wQNHz2YrLsuXOZozoeDmnHXkNCRmMW0gwFWDfEZDA=
-github.com/cenkalti/backoff/v4 v4.1.1 h1:G2HAfAmvm/GcKan2oOQpBXOd2tT2G57ZnZGWa1PxPBQ=
-github.com/cenkalti/backoff/v4 v4.1.1/go.mod h1:scbssz8iZGpm3xbr14ovlUdkxfGXNInqkPWOWmG2CLw=
 github.com/cenkalti/backoff/v4 v4.1.2 h1:6Yo7N8UP2K6LWZnW94DLVSSrbobcWdVzAYOisuDPIFo=
 github.com/cenkalti/backoff/v4 v4.1.2/go.mod h1:scbssz8iZGpm3xbr14ovlUdkxfGXNInqkPWOWmG2CLw=
 github.com/census-instrumentation/opencensus-proto v0.2.1/go.mod h1:f6KPmirojxKA12rnyqOA5BBL4O983OfeGPqjHWSTneU=

Here's the commit: https://github.com/circleci/ex/commit/41374783a6dc854f9aae8fc3751c8e938105bc7f

One thing I noticed when trying to run an update is that dependabot seems to run into issues with<EMAIL_ADDRESS>=== github.com/gin-gonic/gin (1.7.4)
 => checking for updates 4/20
 => latest available version is 1.7.6
 => latest allowed version is 1.7.6
 => requirements to unlock: own
 => requirements update strategy:
 => updating github.com/gin-gonic/gin from 1.7.4 to 1.7.6
 => handled error whilst updating github.com/gin-gonic/gin: dependency_file_not_resolvable {:message=>"go<EMAIL_ADDRESS>reading gopkg.in/alexcesaro/statsd.v2/go.mod at revision v2.0.0: git ls-remote -q origin in /home/dependabot/go/pkg/mod/cache/vcs/b21a1f53c1523fdeb566aed8131de2b9fe1ca244709afb058906cee2b66ac19a: exit status 128:"}

Not sure if that's related yet, we currently have an issue with getting dependabot to run against a specific commit for go_modules specifically (it's not related, and happens because we perform a shallow clone on the repo, which does not support providing a commit), so let me clone the repo and reset it to a commit we know dependabot failed on, and see if I can figure out what it run into.

The error I'm getting when running go mod tidy -e  (also without the -e flag) is:
go: downloading github.com/streadway/amqp v1.0.1-0.20200716223359-e6b33f460591
go: downloading github.com/google/go-cmp v0.5.6
go: downloading github.com/michaelklishin/rabbit-hole/v2 v2.11.0
go: downloading github.com/go-redis/redis/v8 v8.11.4
go: downloading go.opentelemetry.io/otel v1.0.0
go: downloading github.com/stretchr/testify v1.7.0
verifying<EMAIL_ADDRESS>checksum mismatch
	downloaded: h1:P3fq6xFRZ8FWOCsjgrAW4FezfIpyc1bcKIp1Qkf/egk=
	go.sum:     h1:v/Jtrr0FY82pITY3VFhIDaXCllPCTGpGCIM2U505Row=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

The difference between your local run and what dependabot does, is that dependabot currently always sets GOPRIVATE=*. If I omit that, it succeeds and stores this new value in the modcache, but we currently don't have a way to intelligently disable that env variable in dependabot.
Any thoughts on why we're seeing this error?

I can confirm that enabling GOPRIVATE immediately shows the same issue for me:
% docker run --rm -v $PWD:/go/src -e 'GOPRIVATE=*' -ti golang:1.17 bash -c 'cd src && go mod tidy'
go: downloading gotest.tools/v3 v3.0.3
go: downloading github.com/cenkalti/backoff/v4 v4.1.2
go: downloading github.com/honeycombio/beeline-go v1.3.1
go: downloading golang.org/x/sync v0.0.0-20201020160332-67f06af15bc9
go: downloading github.com/gin-gonic/gin v1.7.4
go: downloading github.com/hellofresh/health-go/v4 v4.5.0
go: downloading github.com/DataDog/datadog-go v4.8.3+incompatible
go: downloading github.com/rollbar/rollbar-go v1.4.2
go: downloading github.com/jolestar/go-commons-pool/v2 v2.1.1
go: downloading github.com/makasim/amqpextra v0.16.4
go: downloading github.com/streadway/amqp v1.0.1-0.20200716223359-e6b33f460591
go: downloading github.com/honeycombio/dynsampler-go v0.2.1
go: downloading github.com/honeycombio/libhoney-go v1.15.6
go: downloading github.com/google/go-cmp v0.5.6
go: downloading github.com/klauspost/compress v1.13.6
go: downloading github.com/alecthomas/kong v0.2.18
go: downloading github.com/jmoiron/sqlx v1.3.4
go: downloading github.com/lib/pq v1.10.3
go: downloading github.com/michaelklishin/rabbit-hole/v2 v2.11.0
go: downloading github.com/go-redis/redis/v8 v8.11.4
go: downloading go.opentelemetry.io/otel v1.0.0
verifying<EMAIL_ADDRESS>checksum mismatch
        downloaded: h1:P3fq6xFRZ8FWOCsjgrAW4FezfIpyc1bcKIp1Qkf/egk=
        go.sum:     h1:v/Jtrr0FY82pITY3VFhIDaXCllPCTGpGCIM2U505Row=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

I have raised an issue against that library: https://github.com/michaelklishin/rabbit-hole/issues/219

Thanks @pete-woods!

Feels like the only things that can be done on the dependabot end are better reporting / handling of errors when updating deps. Would have been great to see a red mark in the GitHub UI somewhere.

Yeah, surfacing errors is definitely something that we can improve the UI/UX of, in this case treating it as a hard failure is probably not the right thing to do, since there are cases in which it's just not possible for us to run go mod tidy, and we'd prefer to still open a PR in those cases.
Once we get a chance to dig into this though, I'll make sure we also think about surfacing non-fatal errors/issues though 👍
Even just adding an entry in the log explaining we couldn't run tidy would be an improvement here IMO.

Thanks for circling back round to this! 🎉


surface-control: 0.4.3-1 -> 0.4.3-2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/linux-surface/surface-control/releases.
meta.description for surface-control is: "Control various aspects of Microsoft Surface devices on Linux from the Command-Line"
meta.homepage for surface-control is: "https://github.com/linux-surface/surface-control"
meta.changelog for surface-control is: ""
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.3-2 with grep in /nix/store/v5fndiip54h4ppga418ia6i4n6v1xd2s-surface-control-0.4.3-2
found 0.4.3-2 in filename of file in /nix/store/v5fndiip54h4ppga418ia6i4n6v1xd2s-surface-control-0.4.3-2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
surface-control

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v5fndiip54h4ppga418ia6i4n6v1xd2s-surface-control-0.4.3-2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v5fndiip54h4ppga418ia6i4n6v1xd2s-surface-control-0.4.3-2
ls -la /nix/store/v5fndiip54h4ppga418ia6i4n6v1xd2s-surface-control-0.4.3-2/bin

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

1 package built:

surface-control

Maintainer pings

@ofborg build surface-control surface-control.passthru.tests


[1.19.2]: High tick spike when cutting down trees

Is this caused by dynamic trees or an interaction with https://modrinth.com/mod/vmp-forge?
(Processing time by performance mods can't accurately be judged since they route certain logic through their own methods)

i guess thats one way to deal with issues lmao
not using the mod anymore so w/e


Allow configuration of cell debugger

In some cases (e.g., https://github.com/microsoft/debugpy/issues/1166#issuecomment-1374221154) it would be useful if we could configure the cell debugger. For example, in the case pointed out above, it would be nice if we could set "subProcess": false for preventing the debugger from crashing (as it currently does not support subprocess debugging).

That issue will be fixed. Is that what this is about or is there anythign else you are asking to configure?

Mainly about this issue, but I imagine there may be other cases where configuration of the cell debugger might be useful.

I don't think I can tackle this without a concrete use case, I will get to the subprocess issue as soon as I'm able though


Automated cherry pick of #6609: fix host name validator

Cherry pick of #6609 on release/3.2.
#6609: fix host name validator

Codecov Report

Merging #6612 into release/3.2 will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.2    #6612      +/-   ##
===============================================
- Coverage         7.03%    7.03%   -0.01%
===============================================
  Files              650      650
  Lines           111812   111821       +9
===============================================
  Hits              7869     7869
- Misses          103334   103343       +9
  Partials           609      609

Flag
Coverage Δ

#aFlag
7.03% <0.00%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/namevalidator.go
0.00% <0.00%> (ø)

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.48% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/buckets.go
1.17% <0.00%> (ø)

pkg/compute/models/hosts.go
0.30% <ø> (ø)

pkg/compute/models/skus.go
1.16% <0.00%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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


[rails] Update 7.0.3, 6.1.6, 6.0.5, 5.2.8

https://github.com/rails/rails/releases/tag/v7.0.3
https://github.com/rails/rails/releases/tag/v6.1.6
https://github.com/rails/rails/releases/tag/v6.0.5
https://github.com/rails/rails/releases/tag/v5.2.8

@BiNZGi Thank you


WIP: EN-27397 Some performance improvements to the aggregate method.

[ ] styleCheck passes
[ ] unit tests pass
[ ] integration tests pass
[ ] manual tests pass
[ ] API docs updated

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.

This build looks good.


Switch to TypeScript

This will allow us to make the code typesafe and will also declaration files (.d.ts). This is important for consumers to see what package exports and what methods are available on the class (e.g. they can be public or private). Public methods will be start and destroy for our consumers (some article on the declaration files: https://morioh.com/p/33bca6a30f8d).

This is a great idea, it's something I've been wanting to do but haven't gotten around to it.


[FEATURE] Allow Dictionary Inputs for Complex Types in SDK

Problem Statement
The SDK currently requires users to create specific object types (like EndpointCoreConfigInput, AiGatewayConfig, RateLimit, EndpointTag) when e.g. creating a serving endpoint (sticking to this as endpoint creation/updating and gateway configuration is my current use case). This can be cumbersome and somewhat challenging to learn.
For instance, creating a serving endpoint currently looks something like:
from databricks.sdk.service.serving import (
    ServedEntityInput,
    EndpointCoreConfigInput,
    ExternalModel,
    ExternalModelProvider,
    OpenAiConfig,
)

create_endpoint_config = EndpointCoreConfigInput(
    name="dr-gateway-demo",
    served_entities=[
        ServedEntityInput(
            name="<name>",
            external_model=ExternalModel(
                name="<model-name>",
                provider=ExternalModelProvider.OPENAI,
                task="llm/v1/chat",
                openai_config=OpenAiConfig(
                    openai_api_key=f"{{{{secrets/{SECRETS_SCOPE}/{SECRETS_KEY}}}}}",
                ),
            ),
        )
    ],
)

Even for someone experienced with the API, it takes a lot of extra time consulting docs making sure the classes are properly instantiated.
This is made more challenging by how deeply nested these types are (see above).
Proposed Solution
Allow the SDK to accept both custom objects and plain dictionaries for complex type parameters, with a recursive helper function that converts dictionaries to appropriate object types while handling nested structures and type compatibility. For example, the create() method could be updated to accept either custom objects or dictionaries (resulting in the following signature):
def create(self,
           name: str,
           config: Union[EndpointCoreConfigInput, Dict[str, Any]],
           *,
           ai_gateway: Optional[Union[AiGatewayConfig, Dict[str, Any]]] = None,
           rate_limits: Optional[Union[List[RateLimit], List[Dict[str, Any]]]] = None,
           route_optimized: Optional[bool] = None,
           tags: Optional[Union[List[EndpointTag], List[Dict[str, Any]]]] = None) -> Wait[ServingEndpointDetailed]:
    # Implementation that handles both object and dictionary inputs

Additional Context
One of the use cases this enables is making it much easier to update configs—you can just .update() the corresponding dict. It would also:

Make it more similar in usage to the MLflow deployments SDK (at least for model serving), which does let you pass configs as plain dicts
Make objects you might send via a REST API request with the requests library more compatible with the SDK, without a tricky translation to specific object types.

Completely agree. One thing I do as a workaround is use the from_dict methods provided by the data classes (e.g. https://github.com/databricks/databricks-sdk-py/blob/79b096fb414d87ea5c8d4484687544ce63bbe266/databricks/sdk/service/jobs.py#L1545)


Object.prototype.toString() cannot be used to detect object class

The section titled Using toString() to detect object class says:

toString() can be used with every object and allows you to get its class.

which contradicts this prominent note in ECMA-262:

The above definition of toString ... does not provide a reliable type testing mechanism for other kinds of built-in or program defined objects.

This seems to be fixed already.
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/toString$revision/1629396

Looks like this was fixed already.
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/toString$revision/1629396

@MendyBerger yup, looks like this was fixed indeed. Thanks!


Color the answerer's name in comments colorrs all answerers names on any answer

This answer illustrates the issue.
Youstay Igo posted the answer, Lord of Eden posted the question. However, in the comments on Youstay Igo's answer, both Youstay Igo and Michael Richardson's names are colored. As it turns out, Michael Richardson posted one of the other answers to the question.
I didn't look into the code for this issue at all yet, so I'm not sure what the underlying problem might be.

@SnoringFrog That's what the feature's supposed to do: it colours all answerer's names in the comments...

(and as for your other issues - I'll look at them at the weekend! :)

Oh, gotcha. I thought it was only supposed to color their name on their own answers. My mistake then.

no problem! I find the current behaviour better because you can tell when someone who has contributed something is saying something rather than those people who just say "this is wrong. you suck. I don't the know the answer, but blah blah blah" :P


Children is been added twice

Bevy version
0.3 (commit fae628797fb2dba7a95a323c44c0118875b87f1c)
Operating system & version
Fedora 31
What you did
Spawn a scene with a hierarchy
let mut world = World::default();
let world_builder = &mut world.build();

let components = (
    GlobalTransform::default(),
    Transform::default(),
    Children::default(),
);
world_builder
    .spawn(components.clone())
    .with_children(|builder| {
        builder.spawn(components.clone()).with_children(|builder| {
            builder.spawn(components.clone()).with_children(|builder| {
                builder.spawn(components.clone()).with_children(|builder| {
                    builder.spawn(components.clone()).with_children(|builder| {
                        builder.spawn(components.clone());
                    });
                });
            });
        });
    });

commands.spawn_scene(scenes.add(Scene::new(world)));

What actually happened
Each children will be added twice to it's parent Children component, this causes a massive slowdown caused by the transform_propagate_system witch will then need to update each node hierarchy multiple times.
Additional information
I propose 3 things:

Adding a property ignore to the Children smallvec so it won't get serialized once it get's transformed into a Scene
Fixing WorldBuilder to not insert entities into the Children component
Changing children smallvec to pub(create) and adding a impl Deref<Target = [Entity]> for Children. If the used change this children component the side effects will be devastating

I didn't spot this error on the latests commit

Before anything else I recommend making Children.0 and PreviousParent.0 pub(crate).
@cart  I found 3 main ways of fixing this issue, but not sure witch one is best:

Check if the entity was already added, if so do nothing (silliest)
impl FromResources and MapEntities for PrevousParent then registry it appropriately (less impact on code structure, but won't fix serialized scenes)
Don't save or map Children entities (break codes that spawns a scene before EVENT and use a entity Children component of that said scene before POST_UPDATE)

My main goal is to make the behaviour more consistent and easer to explain: the Children component is a helper component used by the bevy_transform crate, any changes to the hierarchy will only be reflected in the Children component in the next frame;
For that reason I more inclined to go with 3, (none of theses changes will affect the behaviour of the WorldBuilder.with_children function and friends as it already set Children + PreviousParent)
Opinions?


updated frontend to display one Critical Update checkbox for subscribed users

@blueQcloud can you deploy this version in dev/test namespace so we can see the changes?


User model and basic apis

related to #19 and #20

basic APIs introduced: ../api/user/getall/, /api/users/get/<id>/


how to use ReplyKeyboardMarkup?

im new to coding and dont get my bot working! i use telepot. i want my bot to show a customized keyboard , with (for example) 2 fields for yes and no, after he gets the command to do that (for example: /key).
unfortunately i cant find other help on the net that i can understand. maybe there is someone out there who can show me the right code.
thanks guys

Hello,
this type of question is better suited on stackoverflow.com than here. here is mostly for problems specifically with telepot. e.g. something that doesn't work as expected or a bug or something that is missing from telepot can be reported here.
here is an example code to send a keyboard:
import sys
import time
import telepot
from telepot.namedtuple import ReplyKeyboardMarkup, KeyboardButton

def on_chat_message(msg):
    content_type, chat_type, chat_id = telepot.glance(msg)
    print('Chat Message:', content_type, chat_type, chat_id)

    if content_type == 'text':
        if msg['text'] == '/key':
            bot.sendMessage(chat_id, 'testing custom keyboard',
                            reply_markup=ReplyKeyboardMarkup(
                                keyboard=[
                                    [KeyboardButton(text="Yes"), KeyboardButton(text="No")]
                                ]
                            ))

TOKEN = sys.argv[1]  # get token from command-line

bot = telepot.Bot(TOKEN)
print('Listening ...')
bot.message_loop({'chat': on_chat_message}, run_forever=True)

you can view all telegram bot api methods here:
https://core.telegram.org/bots/api#available-methods
thay all work in telepot with the same parameters.

How can I close the keyboard?

msg = bot.sendMessage(...)
msg_ident = telepot.message_identifier(msg)
bot.editMessageReplyMarkup(msg_ident, reply_markup=None)

Extract the message identifier from the sent message, then editMessageReplyMarkup(). Haven't tested it, but should not be far off.

how i can have a reply keyboard with indistinctive buttons!
i want to load my buttons from database and i don't know how many value is in my database!
c.execute("select * from admin WHERE id='%s'" % chat_id)
channel=ReplyKeyboardMarkup(keyboard=[
[KeyboardButton(text=c.fetchall()[0])]], resize_keyboard=True)
i try this but i know is not true!
its just for intention of me


iComfort S30

Hey,
I have seen some previous discussion on supporting the iComfort S30 that uses https://www.lennoxicomfort.com/ instead of https://www.myicomfort.com/
Just wondering if there is support for the S30 and/or are you willing to add support if I can provide access to my thermostat for testing purposes.

Hey @rumym, sorry for the late response. I didn't see this message until now. I can look into it, but the API might be completely different for the S30. With any luck, it would be more or less the same.

Thank you! There are quite a few folks who are looking for this and if you are able to solve it then that would be really amazing!

I did some poking around and it looks like the lennoxicomfort.com website might be sharing some the same backend infrastructure as myicomfort.com. However, I can't do too much investigation without a thermostat to register with. I looked on eBay to see if I could maybe get a used S30, E30, or M30 thermostat for cheap... but they're not cheap enough for me to buy for this. If I had access to an account with a registered thermostat, I would probably be able to get support fully added. I'll keep my eye out for a used thermostat for cheap.

@rumym I re-read your original message and realized you offered to let me access your thermostat account. That would help immensely for me to add support for the S30 and likely the M30 and E30 thermostats as well. Send me an email at<EMAIL_ADDRESS>and can figure out a good time for me to do some investigating with your account.


Sky soarer fall damage always mitigated

Somebody should close this issue report for not filling out the template properly. ;)

Somebody should close this issue report for not filling out the template properly. ;)

@Vaelzan What template? 🦘

@Vaelzan What template? 🦘

You'll have to try harder than that!

Sorry?

Sorry?


Hardlight bow buff

Changed the following damages for hardlight bow:
Base - 30 Heat -> 50 Heat
Disabler - 60 Stam -> 45 Stam
Fire - Damage unchanged, deals 3 fire stacks rather than 1.5
Ice - 20 cold -> 5 cold. Cools target down by around double previous amount.
X-ray - 24 heat -> 35 heat.
Bees - Summons 8 bees rather than 4.
Why / Balance
As discussed in Discord with other players, the hard light bow is extremely weak for an 80TC purchase, so we tested and played with numbers to make it a bit better. Before going further, keep in mind that the bow is extremely difficult to land hits with and extremely punishing for missing them. Before this change, it was out DPS'd by the viper, a 14TC purchase.

The base arrow was made to two hit unarmored targets, with a kill time of around 3 seconds if one has perfect accuracy. If the target is moving, it is very difficult to attain this.

The disabler arrow was made to stun in three hits rather than two, making it not the best option available. However, it is still far worse than a disabler.

Fire damage was unchanged, but the firestacks were extremely underwhelming, now it is more of a threat if not extinguished and comparable with the base arrow.

Ice was changed to be more of a utility arrow than a true damage monster, it no longer does 20 true damage and now chills targets enough to slow them down and make them easier to hit with follow-ups.

Xray arrows received a slight damage increase, which will be reduced upon the introduction of any x-ray eyewear.

Bees are by far the most glass cannon arrow type, one mistake and they kill you rather than your target, doubling the amount of bees to amplify this effect further.

I tested the bow ingame multiple times before this and could confidently say it was the most fun weapon to play with, but the least viable. After testing it in dev environment it feels much better without being overwhelming. Overall these tweaks will bring it to a comparable level to the other 80TC weapon, the desword.

Just make it cost 60 tc...

I don't understand why you buff regular arrows but nerf disabler ones. Non-lethal options should be better.
8 bees means 60 pierce damage to a single person and you don't even need to aim directly at to hit them. You can shoot through glass//airlocks/firelocks so bees don't sting you. I'd rather nerf bee aggression and juke range so they don't attack the shooter too often.
Nerfing cold damage is not a good idea because there should be an option against people in atmos suits. Also cold is easily healable by sleeping on a bed so 20 damage is fine.

Nerfing cold damage is not a good idea because there should be an option against people in atmos suits.

Oh, I thought atmos suits have 90% heat protection, they actually have like 20.

armok trying to balance traitor... :goberserk:

Xray mode is literally broken thing, why do you need to buff it? And with bees - why not increase the damage on bees instead of their amount? This will lead to more server lags.
Anyway, i saw a lot of comments in discord that this bow now is balance broken weapon (xray and explosion modes).

Keep in mind this is the server that sec can instantly beat any antag but nukie by injecting ephe and throwing a single flash bang

Ok, you also need to change disabler arrow selectable ammo proto description because it says that it stuns in two hits.

Ok, you also need to change disabler arrow selectable ammo proto description because it says that it stuns in two hits.

Thanks for letting me know, on it

@Aviu00 do you have any idea how I could make the bomb arrow only explode upon hitting a player but otherwise fizz out?

@Aviu00 do you have any idea how I could make the bomb arrow only explode upon hitting a player but otherwise fizz out?

It would require making another component and system to trigger on projectile mob hit instead of TriggerOnCollide.

Put up for admin review.

Put up for admin review.

any shot I could be added to the convo


Problems with building release in project including exprotobuf

Some change that occured after commit 58f5d44 causes mix release task (exrm version 0.19.9) to fail on building tar archive. Issue has been narrowed down to exprotobuf in current HEAD. After reverting to mentioned commit everything started to work again.

Is this resolved after #34 was merged?

Nope, problem here stands. Here is the trace:
==> Generating release...
{{case_clause,
     {'EXIT',
         {function_clause,
             [{filename,join,[[]],[{file,"filename.erl"},{line,393}]},
              {erl_tar,split_filename,4,[{file,"erl_tar.erl"},{line,471}]},
              {erl_tar,create_header,3,[{file,"erl_tar.erl"},{line,400}]},
              {erl_tar,add1,4,[{file,"erl_tar.erl"},{line,323}]},
              {systools_make,add_to_tar,3,
                  [{file,"systools_make.erl"},{line,1879}]},
              {lists,foreach,2,[{file,"lists.erl"},{line,1337}]},
              {systools_make,'-add_applications/5-fun-0-',6,
                  [{file,"systools_make.erl"},{line,1569}]},
              {lists,foldl,3,[{file,"lists.erl"},{line,1262}]}]}}},
 [{systools_make,'-add_applications/5-fun-0-',6,
      [{file,"systools_make.erl"},{line,1569}]},
  {lists,foldl,3,[{file,"lists.erl"},{line,1262}]},
  {systools_make,add_applications,5,[{file,"systools_make.erl"},{line,1568}]},
  {systools_make,mk_tar,6,[{file,"systools_make.erl"},{line,1562}]},
  {systools_make,mk_tar,5,[{file,"systools_make.erl"},{line,1538}]},
  {systools_make,make_tar,2,[{file,"systools_make.erl"},{line,336}]},
  {rlx_prv_archive,make_tar,3,[{file,"src/rlx_prv_archive.erl"},{line,83}]},
  {relx,run_provider,2,[{file,"src/relx.erl"},{line,308}]}]}
==> Failed to build release. Please fix any errors and try again.

@rafal-radziszewski I can't reproduce this with a new project. Could you share your mix.exs and how you are invoking the release task?

This related issue may help
https://github.com/bitwalker/exrm/issues/299

Closing this as it's a hard limit imposed on us by systools when unpacking releases. Further work on fixing this in the future will be handled via the exrm/distillery projects.


System.InvalidOperationExceptionAn attempt was made to transition a task to a final state when it had already completed.

Bug
System.InvalidOperationExceptionAn attempt was made to transition a task to a final state when it had already completed.
Raw
System.Threading.Tasks.TaskCompletionSource.SetResult(TResult result)<368820a9888f43ddb85d18e87189adbf>:0
Plugin.Permissions.PermissionsImplementation.OnRequestPermissionsResult(int requestCode, string[] permissions, Permission[] grantResults)<47bb53b40691435f96893bccf92fcd93>:0
Sensus.Android.AndroidMainActivity.OnRequestPermissionsResult(int requestCode, string[] permissions, Permission[] grantResults)<23556fa8db484a1f9fb76c2fcebf4564>:0
Android.App.Activity.n_OnRequestPermissionsResult_IarrayLjava_lang_String_arrayI(IntPtr jnienv, IntPtr native__this, int requestCode, IntPtr native_permissions, IntPtr native_grantResults)<148efd2f7a03496fb40611cf5f4f0bb1>:0
at (wrapper dynamic-method) System.Object:d20dfcf5-62be-4e27-8c5a-6c2b9ce6303b (intptr,intptr,int,intptr,intptr)
Version Number of Plugin:  1.1.7
Device Tested On:  Android M
Simulator Tested On:  None
Expected Behavior
No exception.
Actual Behavior
Exception.
Steps to reproduce the Behavior
Not sure. Got this in Insights.

How did you call it? This probably means you called a method twice without awaiting.
Or this means you didn't put the code in your main activity to pass up the event.

I obtain permission with the following call:
await CrossPermissions.Current.RequestPermissionsAsync(permission)
I don't call this method anywhere else, although it's possible for this line of code to run concurrently in multiple threads. Is this a problem?
I've got the following in my main activity:
        public override void OnRequestPermissionsResult(int requestCode, string[] permissions, Permission[] grantResults)
        {
            PermissionsImplementation.Current.OnRequestPermissionsResult(requestCode, permissions, grantResults);
        }

Looks correct to me. Perhaps the user pressed some button that trigger the call multiple times? That may cause something like it. Out of the box I have never seen something like this from normal use.

The calls don't result from user interaction. It's quite possible for them to be performed concurrently across threads. I'll try restricting concurrent executions for this code.

Perhaps there is something I can do in the code to stop it...
Essentially what I do here is cancel the current one if a new one comes in...
https://github.com/jamesmontemagno/PermissionsPlugin/blob/master/src/Plugin.Permissions.Android/PermissionsImplementation.cs#L120-L124
So perhaps it is a weird timing issue?

Yeah maybe. I'm pretty sure that await CrossPermissions.Current.RequestPermissionsAsync(permission) is being called within multiple threads. If you can handle it that would be great.

I don't think i can handle multiple requests at the same time persay based on my design of the library... if I could get a small repo somehow then I could test for it.
I could see how it could come up though if you call it form message center or something like that...
Need some threading experts.

Would it be as straightforward as guarding RequestPermissionsAsync with a SemaphoreSlim instance variable on PermissionsImplementation?  You can await the SemaphoreSlim's availability with its WaitAsync method.

This was my issue and SemaphoreSlim solved it for me. Thanks for the tip!

Getting the same problem.
Semaphore just on RequestPermissionsAsync or also other methods as well (e.g. CheckPermissionStatusAsync) ?

More of a "me too" post. I'm seeing the same issue with PermissionsPlugin occuring when using xam.plugins.geolocator (not hitting permissions myself).
It has only occurred on the Android side (so far), not on the iOS side.

Exception: Message: An attempt was made to transition a task to a final state when it had already completed.; StackTrace: at System.Threading.Tasks.TaskCompletionSource`1[TResult].SetResult (TResult result) [0x00013] in <06ef333e5e2644c1b4965d5d27f9ec90>:0 at Plugin.Permissions.PermissionsImplementation.OnRequestPermissionsResult (System.Int32 requestCode, System.String[] permissions, Android.Content.PM.Permission[] grantResults) [0x0007d] in <47bb53b40691435f96893bccf92fcd93>:0 at UnderSea.Droid.MainActivity.OnRequestPermissionsResult (System.Int32 requestCode, System.String[] permissions, Android.Content.PM.Permission[] grantResults) [0x00005] in :0 at Android.Support.V4.App.FragmentActivity.n_OnRequestPermissionsResult_IarrayLjava_lang_String_arrayI (System.IntPtr jnienv, System.IntPtr native__this, System.Int32 requestCode, System.IntPtr native_permissions, System.IntPtr native_grantResults) [0x00038] in <7e083afc7c9045f59e01d9c14931060d>:0 at (wrapper dynamic-method) System.Object:8d1d0ec0-02e8-48c3-9419-cf668207bd90 (intptr,intptr,int,intptr,intptr); Source: mscorlib;

Would need sample code, you may be calling it multiple times so put some checks around it.

Just re-checked the code, and yes it was being called twice on app launch. Removed the duplicate call which got rid of the error.

If you want to trap this exception just go to the MainActivity where OnRequestPermissionsResult is overriden and put a try...catch around: PermissionsImplementation.Current.OnRequestPermissionsResult(requestCode, permissions, grantResults);
This way you avoid a crash.