Remove array allocations

ToArray creates a new array that is the length of the memory stream contents. This is unnecessary since the data is then further copied to combine it with the total payload length. GetBuffer does not allocate and returns the backing buffer, which may be larger than the content, so it's important not to copy more bytes than what the memory stream says its length is.
2 further optimizations should be easy to do.

Use RecyclableMemoryStream instead of newing up a MemoryStream instance for every message. The package is already pulled in as a dependency of FSharp.Control.Websockets, so no extra costs.
Write 5 bytes to the memory stream first (as a reservartion of space for the length value) and then use it serialize the message. Finally, set Position to 0 and overwrite the first 5 bytes with message length. This would require an additional method in Fable.Remoting.MsgPack because the amount of bytes written per integer depends on its size, which is great in general, but here we need to rely on a fixed size to prevent array copying/shifting.

I could not test this locally but I don't expect any issues.

Thanks! I tried my best to make things performant, but it's not something I have a lot of experience with 😅.

Use RecyclableMemoryStream instead of newing up a MemoryStream instance for every message. The package is already pulled in as a dependency of FSharp.Control.Websockets, so no extra costs.

Cool, I'll check out RecyclableMemoryStream.

Write 5 bytes to the memory stream first (as a reservartion of space for the length value) and then use it serialize the message. Finally, set Position to 0 and overwrite the first 5 bytes with message length. This would require an additional method in Fable.Remoting.MsgPack because the amount of bytes written per integer depends on its size, which is great in general, but here we need to rely on a fixed size to prevent array copying/shifting.

I'd create an issue in Fable.Remoting, although I'm not sure what details I need. If you could create one I'd appreciate it so I can add that optimization when it's implemented.

On the topic of optimizations currently when deserializing I have this:
let reader = Read.Reader(input.ToArray())
let len = reader.Read typeof<uint32> |> unbox<uint32> |> int64

input <- input.Slice(len + 1L)

if len > MaxPayloadSize then
    message <- null
    false
else
    message <-
        reader.Read typeof<Msg<'ClientStreamFromApi,'ClientApi,'ClientApi,'ClientStreamToApi>>
        |> unbox<Msg<'ClientStreamFromApi,'ClientApi,'ClientApi,'ClientStreamToApi>>
        |> function
        ...

I imagine that input.ToArray() is quite bad here since that input could potentially be 2+ GB in size. Do you see any issues with something like this?
let len =
    input.Slice(0, 4).ToArray()
    |> Read.Reader
    |> fun lenReader -> lenReader.Read typeof<uint32>
    |> unbox<uint32>
    |> int64

let reader =
    let startPos =
        BitConverter.GetBytes(len)
        |> Array.filter (fun n -> n <> 0uy)
        |> Array.longLength

    input.Slice(startPos, len).ToArray()
    |> Read.Reader

input <- input.Slice(len + 1L)

if len > MaxPayloadSize then
    message <- null
    false
else
    message <-
        reader.Read typeof<Msg<'ClientStreamFromApi,'ClientApi,'ClientApi,'ClientStreamToApi>>
        |> unbox<Msg<'ClientStreamFromApi,'ClientApi,'ClientApi,'ClientStreamToApi>>
        |> function
        ...

Creating Spans by slicing is near-zero cost, but when you call ToArray on them, you'll still allocate and copy to an entirely new array, so your change pretty much does the same thing on more lines of code. input.Slice(0, 3) is a problem though, because integers of various sizes take 1-9 bytes and without inspecting the first byte (which is what the reader does) you don't know where to stop reading. That's why I said we need an extra method that will always write 5 bytes, even if the integer in question is very small. If you try to read a message larger than 65KB, you'll probably see a IndexOutOfRangeException when reading the length now.
Your original code works almost by accident. You are able to do reader.Read typeof<uint32>, because the reader only advances as  far as it is necessary to get that uint32. Next, input <- input.Slice(len + 1L) has no effect on the array you passed to the reader, so the line really does nothing. Finally, you use the same reader to deserialize the actual content. This works because reading starts where it stopped last time.
I'm not entirely positive, but I think having extra data on input never causes problems (certainly not with integers as I said above), so doing let reader = Read.Reader(input.GetBuffer()) should do the trick and avoid copying. Come to think of it though, do you even need to prepend the length to each message?

even if the integer in question is very small. If you try to read a message larger than 65KB, you'll probably see a IndexOutOfRangeException when reading the length now.

Since SignalR only allows messages up to 2GB in size we know that the length is a uint32, which is 4 bytes: [|255uy; 255uy; 255uy; 255uy|]. I'm probably misunderstanding something here, but even if it took 9 bytes wouldn't I be able to do the same thing here but by slicing 0 to 8?

Your original code works almost by accident. You are able to do reader.Read typeof, because the reader only advances as far as it is necessary to get that uint32. Finally, you use the same reader to deserialize the actual content. This works because reading starts where it stopped last time.

Yeah I was aware of that behavior, I was digging through the source code of your library when trying to get this all to work.

input <- input.Slice(len + 1L) has no effect on the array you passed to the reader, so the line really does nothing.

Sorry, I should have given some more context. input when deserializing is a byref<ReadOnlySequence<byte>> which contains at least one message (but at times multiple messages). That's why I set input once I determine the length of the message, so I can slice out the message I'm processing and later process the others available. So this change would be more to decrease the amount I have to copy to a new array.

Come to think of it though, do you even need to prepend the length to each message?

I think I may not actually need to do that anymore, I know it was a requirement when using SignalR's JS MsgPack protocol, but I'm not seeing a reason I need it anymore.

Creating Spans by slicing is near-zero cost, but when you call ToArray on them, you'll still allocate and copy to an entirely new array,

Is it possible to get a byte [] from a ReadOnlySequence<_> without doing that?

I'm probably misunderstanding something here, but even if it took 9 bytes wouldn't I be able to do the same thing here but by slicing 0 to 8?

But you don't know how many bytes the length integer occupies without checking the first Message Pack format byte. If you blindly took the first 9 bytes, you could very well consume data that is part of the serialized message, corrupting the processing.

Is it possible to get a byte [] from a ReadOnlySequence<_> without doing that?

No, spans/memory are just position and length markers in some underlying sequence of data, and since you're getting a ReadOnlySequence<byte> there's very likely no way to avoid copying here without extending the Reader API.

Oh I see what you mean. So this would always work then, right?
let len =
    input.Slice(0, 10).ToArray()
    |> Read.Reader
    |> fun lenReader -> lenReader.Read typeof<uint32>
    |> unbox<uint32>
    |> int64

let reader =
    input.Slice(0L, len).ToArray()
    |> Read.Reader

reader.Read typeof<uint32> |> ignore

This is probably not worth it since in many cases it's just a single message and is almost always quite small. Anyways, I appreciate you taking the time to respond to me!

Technically yes, but it's icky and more convoluted than the original in https://github.com/Shmew/Fable.SignalR/pull/23#issuecomment-812609916. The only way of improving this without changes in Fable.Remoting.MsgPack would be using System.Buffers.ArrayPool to requisition a byte array to copy from input to. It's still unnecessary copying, but you save an allocation. It might be a bit of an overkill as you're saying, but could also be a fun simple thing to do if you're unfamiliar with the class.
You're welcome.


Better support for Fiddler

Describe the bug
When Fiddler (or any other mitm debugging proxy) is running, az does not work.  This happens even if the Fiddler cert is trusted by the OS (Windows).
To Reproduce

Install Fiddler
Configure Fiddler to decrypt HTTPS requests
Install Fiddler cert to Windows cert store
Begin capturing requests in Fiddler
Run az login --use-device-code (chosen as a demonstration only, other commands also fail)

You will see this:
Please ensure you have network connection. Error detail: HTTPSConnectionPool(host='login.microsoftonline.com', port=443): Max retries exceeded with url: /common/oauth2/devicecode?api-version=1.0 (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')],)",),))

Expected behavior
az begins old login flow.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Windows, installed via the msi a couple of weeks ago
Additional context
There are two semi-workarounds that are insufficient:

AZURE_CLI_DISABLE_CONNECTION_VERIFICATION.  This is UNSAFE and should not be used.
REQUESTS_CA_BUNDLE.  This allows me to specify a path to the Fiddler cert and az will now work when Fiddler is running, however it will no longer work while Fiddler is not running.  Given that a typical developer will turn Fiddler on and off multiple times throughout the day it is not feasible to expect to constantly flip environment variables.

Please either use the OS cert store or allow a way to support this scenario.

Alternatively is there a way to tell az not to use the system proxy? ie Never use Fiddler?

We've found a solution that solves this for our team:  set NO_PROXY=1.

@lmazuel, does requests package support to read OS's certificate store?
Based on this Python issue it doesn't; however this SO thread mentioned it is possible, but is it reliable?

@yugangw-msft no, requests does not support it. If I understood correctly, @zooba is working on this.

I am closing this issue as it is clearly not an issue CLI can address and we have a document which calls out all required steps to get this working.


Accessibility Update

What is the expected behavior?
when a user is moving about the calendar via a keyboard (or some other a11y device), the calendar needs to set "aria-selected" for the currently selected item and "aria-current" for the current item (day, month, year, whichever).
What is the current behavior?
A user can see the currently selected and the current date, but users who are visually impaired and screen readers cannot.
What are the steps to reproduce?

be focused within the open calendar and moving about the calendar via keyboard.
the aria-selected attribute is never being set
-- nor is aria-current

Which versions of angular-bootstrap-datetimepicker, OS, TypeScript, browsers are affected?

angular-bootstrap-datetimepicker: ^2.2.1
OS: Windows (not sure about others)
Browsers: Firefox and Chrome
TypeScript: 4.2

Is there anything else we should know?

Accessibility testing was done by Deque (https://www.deque.com/). We attempted to fix it but there were not enough sufficient @Output events to let us set these attributes ourselves. Deque's accessibility testing tool gave us the following output:

Description
The selected state of an element is not conveyed to screen reader users.
Element name: Current date and selected date example "9".
Location: On the calendar widget.
Additional details: Current date and selected dates are not announced to the screen reader users.
Recommendation to fix
RULE :
The name, role, value, states, and properties of user interface components MUST be programmatically determinable by assistive technologies.
HOW TO FIX:
Fix this issue by using ONE of the following techniques:

Use the aria-selected attribute with the following four ARIA roles: gridcell, option, row, tab.
Use the aria-current attribute to indicate the element that represents the current item within a container or set of related elements, such as the currently selected page, step, location, date, or time.

REFERENCE:
WAI-ARIA Recommendation: https://www.w3.org/TR/wai-aria-1.1/#aria-selected
WAI ARIA Authoring Practices: https://www.w3.org/TR/wai-aria-practices-1.1/#tabpanel
WAI-ARIA Recommendation: https://www.w3.org/TR/wai-aria-1.1/#aria-current
BACKGROUND:
States and properties are attributes used to convey essential information about an element to screen readers and other assistive technologies. Some roles require certain state and property information - such as the checked/unchecked state of a checkbox. Native HTML elements provide those required states and properties, so nothing more needs to be done. If you create a custom version of a native HTML element or a custom control or widget that does not have a native HTML equivalent, you must add the relevant states and properties using ARIA.

Thanks @SmittyShmee! I'm super happy to see a11y testing happening! Sorry I missed this!
I have not verified this but yet but I will shortly.
Can you tell me a bit more:
What version of Angular are you using?
Do you have a timeline where you need this fixed?

Hello @dalelotts,
The testing was done on an Angular 12 build (12.0.2). The framework we include angular-bootstrap-datetimepicker in will be getting an update to Angular 13 (13.1.1) in the not-to-distant future.
I know there is another round of a11y testing set to be done, just not sure when it's scheduled for.
Thanks.


fix: fix command all duplicates

:tada: This PR is included in version 5.21.2 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.0.0-rc.5 :tada:
The release is available on:

npm package (@prerelease dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


backdrop can't cover the StatusBar

What I want is like this:

What I got is like this:

How can I make the backdrop cover the StatusBar?

See #50 😢

@mmazzarolo @YanmaY  do you have the solution? could you share the code snippet here? I'm also having this issue. The modal doesnt cover the statusbar

@alvintheodora react-native-modal-translucent


[BUG] ETXTBSY when launching

Environment

chrome-aws-lambda Version: 5.5.0
puppeteer / puppeteer-core Version: 5.5.0
OS:  MacOs Catalina 10.15.7
Node.js Version:  v12.16.3
Lambda / GCF Runtime:  Node.js 12.x

Expected Behavior

Simple lambda reads HTML from event.body and returns PDF stream. Intended to be wired up to api-gateway.
With a test event it will just log the binary results.
Current Behavior

Does not produce PDF and throws error
2021-01-15T22:49:17.389Z	c37e163e-a8d2-468a-b3e7-eb8a49172ecf	ERROR	exception caught  Error: spawn ETXTBSY
    at ChildProcess.spawn (internal/child_process.js:407:11)
    at Object.spawn (child_process.js:553:9)
    at BrowserRunner.start (/var/task/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:79:34)
    at ChromeLauncher.launch (/var/task/node_modules/puppeteer-core/lib/cjs/puppeteer/node/Launcher.js:84:16)
    at async Runtime.getpdf [as handler] (/var/task/index.js:13:19) {
  errno: 'ETXTBSY',
  code: 'ETXTBSY',
  syscall: 'spawn'
}

Steps to Reproduce

Everything is available here: https://github.com/hamiltonca/html-to-pdf-lambda
# Assuming a new clone/install
npm install --only=prod
zip -r <zipFile> . -x '.git*'
<!--
```js
const chromium = require('chrome-aws-lambda')
async function getpdf(event, context) {
    console.log('## ENVIRONMENT VARIABLES: ' + JSON.stringify(process.env))
    console.log('## CONTEXT: ' + JSON.stringify(context))
    console.log('## EVENT: ' + JSON.stringify(event))

    let browser;
    try {
        const reqContent = event.body
        console.log("request content: " + reqContent)
        console.log("launching browser...")
        const executablePath = await chromium.executablePath
        browser = await chromium.puppeteer.launch({
            args: chromium.args,
            defaultViewport: chromium.defaultViewport,
            executablePath: executablePath,
            headless: true,
            ignoreHTTPSErrors: true
        });
        console.log("browser launched, setting up page.");
        let page = await browser.newPage();
        await page.setContent(reqContent)
        console.log("page content set... generating pdf")
        const pdf = await page.pdf({
            format: "letter",
            printBackground:true
        })
        const retContent = formatResponse(pdf)
        console.log("retContent: " + JSON.stringify(retContent))
        return retContent

    }
    catch(e) {
        console.error("exception caught ", e)
        return formatError(e)
    }
    finally {
        if (browser) {
            console.log("closing browser")
            await browser.close()
            console.log("browser closed, returning content")
        }
    }
}
function formatResponse(pdf) {
    console.log("formatting response")
    return {
        "statusCode": 200,
        "headers" : {
            "Content-Type": "application/pdf"
        },
        "body" : pdf
    }
}
function formatError(e) {
    console.log("formatting error response")
    return {
        "statusCode" : 500,
        "headers" : {
            "Content-type" : "text/plain"
        },
        "body" : "An error occurred." + e.toString()

    }
}
exports.handler = getpdf

-->
Possible Solution

I think ETXTBSY implies some resource cannot be attained because of an exclusive lock.
Again the entire project for this is at https://github.com/hamiltonca/html-to-pdf-lambda
Hope you can help me with this. Please let me know if you need more info.
Thanks.

Is there any possibility I could get some assistance with this? Thanks

Is there any possibility I could get some assistance with this? Thanks

Apologies for only now getting to this @hamiltonca, work has been keeping me busier than usual.
I cannot repro your issue, using your code verbatim (without event data) I get the expected output:
{
  "statusCode": 200,
  "headers": {
    "Content-Type": "application/pdf"
  },
  "body": {
    "type": "Buffer",
    "data": [
      37,
      80,
      68,
      ...
      69,
      79,
      70
    ]
  }
}

You are right, ETXTBSY  signifies that a file is busy.
This usually happens when either:

you call chromium.executablePath (which inflates the Chromium binary) while puppeteer is using it
you don't await chromium.executablePath

You seem to be doing everything right and I'm able to successfully run your code, so I would check for lingering references to chromium.executablePath - it could be that you are calling it more than once and overlooked it perhaps?


How can I get a transparent view?

You can use background removal tools (SAM, rembg, etc.) to remove the background.


fix: correct conversion of AsyncAPI types into Java types

I think the existing implementation was confusing "format" and "type", comparing the type property of AsyncAPI data objects with values that are supposed to be formats (e.g. "password").
I've reimplemented this based on the table in
https://www.asyncapi.com/docs/reference/specification/v2.5.0#dataTypeFormat
I'm using the optional format specifier where possible, as it's more specific - and if we can't find anything matching using that I'm falling back to the more general type specifier.
Signed-off-by: Dale Lane<EMAIL_ADDRESS>
Description

...
...
...

Related issue(s)

This was raised in Slack at https://asyncapi.slack.com/archives/C023GJWH33K/p1665939249920639?thread_ts=1665899981.402679&cid=C023GJWH33K

/rtm


[FEA] Reduce speedup factors for complex string expressions (regexp/split)

Current speedup factor is defaulting to 4 for expressions, but we want it lowered for complex string functions like regexp and split.

Expressions to reduce default speedup factor:
RegExpExtract
RegExpExtractAll
RegExpReplace
RLike
StringSplit

Leaving speedup factors as defaults for now as benchmarks showed good acceleration on normal-sized strings


feat: stop sending user and reaction for reactions

Is there a reason that we need to send these? If we don't need them for anything we should not send them so nobody gets upset (it is also just kinda a waste). @rishiosaur are you using these for anything?
Signed-off-by: Matt Gleich<EMAIL_ADDRESS>

give me 10 minutes or so to deploy it...


Upgrade dependencies, expand tests, fix formating, and bump to v1.0.2

This pull request cleans up the code base and updates the dependencies to the latest versions.  Here's a list of what's included:

Upgrade RSpec and Child Process to latest versions
Add a default Rake task to run tests
Migrate all tabs to spaces
Restructure version so it can be imported into gemspec
Reformat gemspec to be more readable and future-proof
Expand test based on some common cases from jruby-sandbox

Thanks :+1:


workbench.main.js:91

Visual Studio Code Version : [ 1.4.0]
Code Sync Settings Version : [ 2.2.3]
Operating System : [ Windows 10]
Please make sure to write the versions and console logged error picture or copied text.

This mistake was every time when I open vscode. I manually choose Sync: Update / Upload, and mistake has gone

Looks like the process was not completed but u started it again.
Look forward to this problem in future update
So the issue is solved?

Yes, no error anymore


fix note about alpha status of featuregate

ConfigurableFSGroupPolicy has been enabled by default and in beta status since kubernetes v1.20.  Updated the docs to reflect that.

I signed it

Hi @RileyMShea .
When you have a moment, would you review the suggestions and rebase? Thanks!

Hi @RileyMShea, this PR needs a rebase almost 100 days ago. If there's no action within 7 days, a PR wrangler may close this PR.

👋 @RileyMShea .
I plan to close this pull request.
Feel free to reopen this PR and continue to work on the changes.
/close


A Minor Improvement to the test cases in AppResponseTest

I would like to propose a minor improvement that could apply to two test cases in AppResponseTest(testSetExceptionWithEmptyStackTraceException and testAppResponseWithEmptyStackTraceException). I will use testSetExceptionWithEmptyStackTraceException as an example.
        Throwable throwable = buildEmptyStackTraceException();
        if (throwable == null) {
            return;
        }

On line 68-71, it uses an if condition to check if a precondition is violated, i.e. “buildEmptyStackTraceException fail to construct NPE” If so, the test case return silently without any further execution. There could be two drawbacks to this:

The if and return make the logic of the test case implicit
It may still be worthwhile to execute the test case and see what happens.

The assume function, introduced after JUnit 4&5 is designed to check for test pre-condition. As mentioned in the document:
Assume functions is a set of methods useful for stating assumptions about the conditions in which a test is meaningful. A failed assumption does not mean the code is broken, but that the test provides no useful information.
The if block could be replaced by After refactoring:
        Throwable throwable = buildEmptyStackTraceException();
        assumeNotNull(throwable);

This makes the logic more explicit. Although this is a minor change, it can be applied to the two test cases in this AppResponseTest .

Could you please file a pull request to add these improvements directly to Dubbo?

I'm interested in contributing to this issue, so before I start working it, would you mind sparing your time explaining what the issue is about and pointing me to some resources to get started.

@sreenath-tm Sorry for taking the chance. I just submit the PR for this Issue, but if you are still interested in this PR, please feel free to review and leave comments in the PR.


Added a CodeChef problem!

Please do check and merge!

Okay so I will add the logic, please guide me once if I will have to again fork the repo or what? How should I go about it for the changes?

@ananya0504  No need to fork again. You can work on your existing fork. Just modify your code and commit your changes.
If you face any more issues, feel free to ask :)

Okay so I have commited the changes, added the comments. Now but after committing it did not show me options to make a PR again!

@ananya0504  No you needn't have to send another PR. You just made you changes and by commiting them, you updated it here itself. I can see them and they're good now :)

Wont you label it as hacktober fest accepted?

@ananya0504  That label is needed only if I am not approving/merging your changes. Since I approved and merged them, it is considered as Hacktoberfest accepted and will be counted.  Reference: https://hacktoberfest.digitalocean.com/hacktoberfest-update


Validate include parameter

When the include parameter is allowed in a request it must be validated, ideally in the corresponding request class (ListRequest, GetRequest, …) by following the path and checking if each property is set as readable. If this is not possible it needs to be done in the DynamicTransformer.
Things may be simpler to implement if the ExposableRelationshipTypeInterface interface is removed first.
See https://jsonapi.org/format/#fetching-includes

Note that depending on the implementation of this issue, the psr/log dependency may be fully removed, as it is currently needed in the DynamicTransformer only.


Update available icon on containers

Hello. I have seen several requests for some kind of automatic container update feature. I thought I would add to those. I don't necessarily want automatic updating, but I would like Portainer to check for updates and if one is available, display an icon on the container (maybe next to the check box?). This way I would know that an update is available and I can choose if and when to update.
Thanks,
Chris

As you mentioned, this question has been asked multiple times before. Closing this as a duplicate of #3434 .


replace registerUserWithUserId with loginUserWithUserAttributes

[Intercom] WARNING: The method '+[Intercom registerUserWithUserId:]' has been deprecated and will be removed in a future release. Please use +[Intercom loginUserWithUserAttributes:success:failure:]' instead.

fixed on v4


Defer loading of config until after rails had a chance to set ENV vars

Before we would initialize the config very early in the CLI, so rails apps would not get a chance to set ENV vars in their initializers.

I’m a bit skeptical of an app mutating its own env, but I’m fine with this change 👍


Update gtest config documentation

Following up on fkie/catkin_lint#76 and ros-controls/ros_control#408:
This doc is a bit out of date, and the rosunit dependency is no longer needed for the catkin_add_XXXtest() commands.

Please update the same docs in the sibling directory for format1 accordingly.

Done in 41b6692. (Force pushed to squash changes to both files into the same commit)

Thanks for the patch.

Force pushed to squash changes to both files into the same commit

Don't worry about this in the future. Most PRs are being squash merged anyway. And when PRs get bigger and are being iterated on it is usually more valuable to be able to only review the incremental change since the last review.

Don't worry about this in the future. Most PRs are being squash merged anyway

Good call, had my mindset on ros_control where we haven't been squashing PRs.
Thanks for the speedy review & merge :)


cannot access '.': Permission denied (os error 13)

On ubuntu 18 I am getting this error
cannot access '.': Permission denied (os error 13)

Getting the same error. Version 0.14.0 installed with snap.

Hi!
duplicate of #79


🐞 [Bug]: Unable to add caprover worker from deployment list

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
I deployed a caprover instance with worker and tried to add another worker from the deployments list but it failed.
I think the issue happens because we use the worker name instead of the leader name.
Steps To Reproduce

Go to dashobard
deploy caprover
wait till the deployment finishes and add a worker from deployed list

which network/s did you face the problem on?
Dev
version
1f123d2
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

tried same flow, got this error

https://github.com/threefoldtech/tfgrid-sdk-ts/pull/3363

Verified, Dev, e87174a
successfully added a worker from the deployed list.


improved the code using code climate

used code climate to solve the issues and improved the code. Related to #84

@smcgregor made it as it was before

A few more small comments. If changed, this is ready to test then merge.

@smcgregor done.

Just one more change.

If you don't comment about the changes you made to the PR, it won't make its way back into the review queue.

@smcgregor I'll make sure I comment from the next time. The latest commit has changes to maintain the semantics you mentioned above.

The code here looks good to go. I'll test it to make sure then merge.

I had to squash the commits, this is merged:
https://github.com/privly/privly-chrome/commit/8f98450a651e48a597a0405d7ad5b0572bb814a6


Create a rule that "ensures that no component tests are extending a core test base class".

Feature request
Title kinda says it all, related to https://www.drupal.org/project/drupal/issues/3400366

Thoughts: https://www.drupal.org/project/drupal/issues/3400366#comment-15313129


While using Zoom.us, MuteMe doesn't appear to mute.

Reproduction steps:
I've tested in Zoom's settings before a conference and found that even while MuteMe was active, My voice was still picked up while testing the input.

I'm having the same issue! Yet, sometimes it does work and sometimes not.

This issue still persists with v.1.2.0

@johncschuster @BalassaSamuelson
Are you using a headset or external microphone?

@proAlexandr I've changed my setting to Same as System and will let you know if I still experience issues. Thanks!

@proAlexandr thanks for that! I'll give it a try and let you know if it keeps happening.

We carefully tested zoom behaviour in the new version of the app which currently we distribute as a separate application https://mutify.app/.

You can try it for free with 7-days trial and get 50% discount with promo code: MUTEME
However, the existing opensource version will stay forever free.

@johncschuster I voluntarily maintain this repository here: https://github.com/ImBIOS/mute-me


#1104 Sorting error in gauge chart

Description
게이지차트의 오름차순 / 내림차순의 미동작 에러를 해결합니다.

Related Issue : #1104

How Has This Been Tested?

차트화면에서 게이지차트를 선택합니다.
선반에 diemsnion / measure를 각각 하나씩 올립니다.
measure의 오름차순 / 내림차순에 따라 차트가 정렬되는지 확인합니다.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Context

run build
deploy to 4

@juheeko9

@ufoscw 수정하였습니다.

run build
deploy to 4


sys-fs/vdo-kmod-<IP_ADDRESS> fails to build: /usr/src/linux-6.6.13-gentoo/Makefile:768: arch/amd64/Makefile: No such file or directory

I tried compiling sys-fs/vdo-kmod-<IP_ADDRESS> against my running kernel. Sources are present, fully configured and compiled.
This is on an x86_64 machine and I am not cross-compiling the kernel.
Error message / build log:
>>> Running pre-merge checks for sys-fs/vdo-kmod-<IP_ADDRESS>
 * Determining the location of the kernel source code
 * Found kernel source directory:
 *     /usr/src/linux
 * Found sources for kernel version:
 *     6.6.13-gentoo
 * Checking for suitable kernel configuration options ...                                                                                [ ok ]

>>> Emerging (1 of 1) sys-fs/vdo-kmod-<IP_ADDRESS>::inode64-overlay
 * vdo-kmod-<IP_ADDRESS>.tar.gz BLAKE2B SHA512 size ;-) ...                                                                                [ ok ]
 * Determining the location of the kernel source code
 * Found kernel source directory:
 *     /usr/src/linux
 * Found sources for kernel version:
 *     6.6.13-gentoo
 * Checking for suitable kernel configuration options ...                                                                                [ ok ]
>>> Unpacking source...
>>> Unpacking vdo-kmod-<IP_ADDRESS>.tar.gz to /var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work
>>> Source unpacked in /var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work
>>> Preparing source in /var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work/kvdo-<IP_ADDRESS> ...
>>> Source prepared.
>>> Configuring source in /var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work/kvdo-<IP_ADDRESS> ...
>>> Source configured.
>>> Compiling source in /var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work/kvdo-<IP_ADDRESS> ...
make -j4 -C /usr/src/linux M=/var/tmp/portage/sys-fs/vdo-kmod-<IP_ADDRESS>/work/kvdo-<IP_ADDRESS>
make: Entering directory '/usr/src/linux-6.6.13-gentoo'
/usr/src/linux-6.6.13-gentoo/Makefile:768: arch/amd64/Makefile: No such file or directory
make[1]: *** No rule to make target 'arch/amd64/Makefile'.  Stop.
make: *** [Makefile:234: __sub-make] Error 2
make: Leaving directory '/usr/src/linux-6.6.13-gentoo'
 * ERROR: sys-fs/vdo-kmod-<IP_ADDRESS>::inode64-overlay failed (compile phase):

Ugh, I haven't looked at this ebuild in a while.
I started the creation but I didn't finish, now I'm a little busy.


Enforce 500 are internal_server_error

We want all our 500 to be internal_server_error and vice-versa. This PR enforces it.

papertrail: in reactions.ts, this PR also removes the 500 error on null reactions

(null was not possible on that reactions endpoint)


std.algo.cmp.clamp docu

Returns and Params for clamp

LGTM, thanks!

Auto-merge toggled on

thanks

thanks


[Admin cabinet] Add to the "Position" names division in Ukrainian and English

[[Admin cabinet] Add to the "Position" names division in Ukrainian and English #5972]
(https://github.com/ita-social-projects/GreenCity/issues/5972)
Now Position have names division in Ukrainian and English
Added new Dto: PositionWithTranslateDto which returns map: :
Changed: New field in Position, PositionDto: "nameEN"

Please fix code smell


Incorrect data for AN draw on 2016-05-19

The draw number is 150104 but it should be 15104.

NLCB seems to have corrected the error so I fixed it by deleting the old incorrect value.
DELETE FROM results where draw=150104;


Terraform Registry

Any chance this provider can be published in Terraform Registry for easy install in tf 0.13?

I'll look into it.

Released today @t3mi let me know if you run into any issues.


python: migrate away from crypt library

crypt is removed in Python 3.13 which is currently in beta.

I suppose we should replace it with hashlib?

ah, if we already use that, sure! I was thinking that maybe passlib would be an option too, but best to keep dependencies low.

wait, we are doing our crypto with a random library with 2 github stars?
Maybe we should use passlib instead? It's recommended in the crypt docs: https://docs.python.org/3/library/crypt.html

I am not sure this whole crypt->passlib or other mechanism is really worth it.
Chatmail is based on debian 12, and that uses python 3.11.
By the time debian packages python 3.13 and we switch to debian 13 or so,
we might have ported our python code to Rust already.
So i suggest to close both this issue and the PR, and revisit it next year, or in the context of a Debian 13 upgrade of chatmail.

Who knows, maybe there is a better crypt replacement then.

yeah, makes sense I guess

So i suggest to close both this issue and the PR, and revisit it next year, or in the context of a Debian 13 upgrade of chatmail.

We're still working on it because it is basically finished and only the CI is broken. If we realize it is not actually finished we will probably close it.


Email or Password incorrect.

I keep getting Email or Password incorrect. Please visit https://ionic.io for help. I'm using the right user name and password.

Uh, same here

2015 and I still have this problem. I can login via website successfully tho


Saving checkpoints at interrupt

Thank you for providing example implementations!
I was wondering what signal is sent to the docker container when spot training job are interrupted. Is it SIGKILL or SIGTERM with some grace period (https://docs.aws.amazon.com/sagemaker/latest/APIReference/API_StopTrainingJob.html)?
I was looking to implement a signal handler which, on SIGTERM, saves the latest checkpoint to S3. That way, resume happens from the exact point in time.
Is this possible? Do we need to account for the time it takes for the uploader service to upload the content of /opt/ml/checkpoints to the checkpoint_s3_uri?
Any guidelines on how to resume from the latest stop point is much appreciated

@eitansela please lmk if this is not the correct location to ask this question and I can close this issue :)

@rst0git I don't think a signal handler can be attached to a SIGKILL signal (https://man7.org/linux/man-pages/man7/signal.7.html). Once a SIGKILL is sent, the process is terminated immediately. Based on the sagemaker docs, SIGTERM is only sent once with a grace period of 120s.

You should save checkpoint to /opt/ml/checkpoints after each EPOC, and SageMaker takes care to copy it to checkpoint_s3_uri for you. It is not a matter of speed because if it is a long training job of few hours or few days, why a SIGTERM will help here? If a Spot goes down, you lose few minutes of training and resume back after you have a new Spot, from the last checkpoint.

So you are right in that I will only lose a few minutes of training if I'm training on one node.
However, if I am training on p4d/p5 instances which have a > 20% interruption rate in most regions and if I'm doing multi node training, then if one node is reclaimed, the whole job needs to be paused.
In this case, there can be too many interrupts.
Also, in general, its preferable to not lose training progress that a job may have made.
Currently we try to counter the lose progress issue by checkpointing frequently but that also has a cost (especially for large models) so its much more convenient if we get some sort of a signal that tells us that our job is going to be interrupted. If SIGTERM is that signal based on the docs, then we can save and resume from same point.


org.hl7.fhir.dstu3.model.DomainResource#listChildren() does not contain the super.listChildren() call

As a result the "new Patient().children()" method does not return any of: id, meta, implicitRules, language
None of the Resource#listChildren() elements.
DomainResource should rather read:
protected void listChildren(List<Property> childrenList) {
  super.listChildren(childrenList);
  ...
  ...

FYI that method is not a part of HAPI's suported API. It comes from the RI structures but it does not get used by HAPI's parser and isn't guaranteed to work or continue to exist long term.
If you want a proper listing of child elements you'd need to use HAPI's metamodel via the FhirContext#getResourceDefinition() method.

Aaargh.   :-)
Relying on this gives you a much cleaner encoder. I have merged the XmlParser and BaseParser into one class and then removed the IParser interface end everything you don't need for creating a record (meaning an ArrayList of columns where sub-structures are nested ArrayLists).
The result of that is working an 60'613 characters of code. That is a lot. Especially as it contains expensive operations like Class.cast() as well.
In comparison, using the Patient.children() method is 100 lines of code. You simply check if the property has a value, if not skip it. I would think you can speed up the JsonParser/XmlParser significantly by using that.
The reading of json/xml probably as well.
Am I missing something?

The children() method is used by the RI parser, and it is indeed cleaner. The problem is that it isn't able to handle a number of scenarios: Using HAPI's annotations it is possible for user code to create resource subclasses that override fields in the base classes to specify custom datatypes or change cardinality, add extension fields, etc. Those changes would not be visible to the parser if it relied purely on children().
Naturally things ..could.. be shifted so that such structures are required to modify children() but we have always opted for a messier parser in exchange for cleaner user code.


docs: typo fix Update README.md

I know it's a tiny typo, but my love for Base runs so deep that I can't help but want everything to be flawless!
"NON- INFRINGEMENT" – there’s an extra space between "NON-" and "INFRINGEMENT." Corrected to "NON-INFRINGEMENT."
Based.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


separate lock screen and wallpaper

I would like the ability to give a separate prompt for my lock screen and my wallpaper. I like my lock screen and my wallpaper to have different themes.

That needs a significant rewrite, so it's gonna take a while. But in the meantime, if you'd like to set your lock screen background in a different way (manually, using a different app...), you can enable live wallpaper in the app's settings and set it only for your homescreen.

Yeah. I figured that was going to be the case. It is a feature request after all. I'll create a few more issues for other features that I (and hopefully other people) would like, but they are feature "requests" not feature "demands" so feel free to say if they don't match your vision of the app.


[Enhancement]: Page Number Section Design Improvement

Is your enhancement request related to a problem? Please describe.➕
No. It's an enhancement in the existing design.
Describe the solution you'd like
Working on the styling of this section, I can make it look even better.
Current Design

Expected Updated Design

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @iamRabia-N , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#1740

Hey @iamRabia-N ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @iamRabia-N , You can't have another issue before completing the previous one 😀 you already have the following 1 open issues 👀 ! : #1740

Oh. Kindly review it so that I can work on this one.


Understanding BootStandalone

I'm having trouble understanding how BootStandalone works. Does anyone have sample code that I could filch? I can't find anything on the webs.
It looks like it can define a different loop function at boot time. From there, I would expect that I could have some offline (standalone) functionality while waiting for wifi and mqtt connections. Once these are obtained, I would assume that the normal operating mode (and loop function) would take over.
Do I have the gist of this? Sample code is worth a thousand words.

check out #125

Yes, that's where I started. Marvin said there was a setStandalone in version 2. It's also in the v2 docs, but it's not in the code. There's only one reference to that, and it's in keywords.txt.
I found BootStandalone, but there's no documentation on it. In perusing the code, I found a rich set of functions that allow for setting the boot behavior, but I can't figure out how to make it work. I could go through the code and try to understand how it works, but someone's gotta have an implementation of this functionality that would help me kickstart the feature.
So that's what I'm looking for: some samples of the functionality or perhaps documentation that shows how it works.

Yes, that's where I started. Marvin said there was a setStandalone in version 2. It's also in the v2 docs, but it's not in the code. There's only one reference to that, and it's in keywords.txt.
I found BootStandalone, but there's no documentation on it. In perusing the code, I found a rich set of functions that allow for setting the boot behavior, but I can't figure out how to make it work. I could go through the code and try to understand how it works, but someone's gotta have an implementation of this functionality that would help me kickstart the feature.
So that's what I'm looking for: some samples of the functionality or perhaps documentation that shows how it works.

Is setStandalone actually implemented? What would it actually do?
As far as I can tell from the code, the only option is to use it with setHomieBootModeOnNextBoot and setHomieBootMode, which would either wait for or force a reboot.

Well, the setHomieBootModeOnNextBoot PR indeed changed the standalone behavior. To be honest, I'll have to take a look to understand more deeply how it works now. 😅

Just came across this today. I was looking at having a node running without a MQTT broker.
(I'm having a test creating a clock, so networking is fine for NTP but not yet using more than that).
So we have :
http://marvinroger.github.io/homie-esp8266/develop/advanced-usage/standalone-mode/
https://github.com/marvinroger/homie-esp8266/blob/develop/src/Homie.cpp#L271
https://github.com/marvinroger/homie-esp8266/blob/develop/src/HomieBootMode.hpp#L5
As i'm not that used to Arduino/C++, I struggled to find how to use enum, I ended up to do:
// Arduino Setup Handler
void setup() {
  // When not using MQTT, set as standalone
  Homie.setHomieBootMode(HomieBootMode::STANDALONE);

  // Some more code here
}

As I'm not yet used to PR in GitHub, I'll leave it like this for now but will probably get back some day soon.


Update Web3Modal Events

Requirements

Update the MODAL_OPEN and MODAL_CLOSE Events to include a new property 'Connected' which can be either true(wallet is connected) or false (no wallet is connected)
Add new events for the following:

CLICK_CHOOSE_NETWORK
SELECT_NETWORK
CLICK_ACTIVITY

Details can be found in this spreadsheet:
https://docs.google.com/spreadsheets/d/1F07sUJhq29S0v5UTe_Q2SZinNQ9tGN9tTN_cOdcZUZk/edit#gid=0

Applied naming changes and added descriptions in spreadhsheet
CLICK_CHOOSE_NETWORK => OPEN_NETWORKS_VIEW
SELECT_NETWORK => SWITCH_NETWORK
CLICK_ACTIVITY => OPEN_ACTIVITY_VIEW


This instruction fails on colab: pip install -r requirements.txt

Hello, the command fails if run on colab.
This is what I get:
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Collecting git+https://github.com/thu-ml/zhusuan.git (from -r requirements.txt (line 13))
Cloning https://github.com/thu-ml/zhusuan.git to /tmp/pip-req-build-csj2aoui
Running command git clone --filter=blob:none --quiet https://github.com/thu-ml/zhusuan.git /tmp/pip-req-build-csj2aoui
Resolved https://github.com/thu-ml/zhusuan.git to commit 4386b2a12ae4f4ed8e694e504e51d7dcdfd6f22a
Preparing metadata (setup.py) ... done
Collecting<EMAIL_ADDRESS>(from -r requirements.txt (line 14))
Cloning https://github.com/haowen-xu/tfsnippet.git (to revision v0.2.0-alpha1) to /tmp/pip-req-build-iwf0sus7
Running command git clone --filter=blob:none --quiet https://github.com/haowen-xu/tfsnippet.git /tmp/pip-req-build-iwf0sus7
Running command git checkout -q 7b43abdbdd29f1914dbc11b961b5d45b9de76653
Resolved https://github.com/haowen-xu/tfsnippet.git to commit 7b43abdbdd29f1914dbc11b961b5d45b9de76653
Preparing metadata (setup.py) ... done
Collecting six==1.11.0
Downloading six-1.11.0-py2.py3-none-any.whl (10 kB)
Collecting matplotlib==3.0.2
Downloading matplotlib-3.0.2.tar.gz (36.5 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 36.5/36.5 MB 9.9 MB/s eta 0:00:00
Preparing metadata (setup.py) ... done
Collecting numpy==1.15.4
Downloading numpy-1.15.4.zip (4.5 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 4.5/4.5 MB 16.0 MB/s eta 0:00:00
Preparing metadata (setup.py) ... done
Collecting pandas==0.23.4
Downloading pandas-0.23.4.tar.gz (10.5 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 10.5/10.5 MB 17.4 MB/s eta 0:00:00
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Preparing metadata (setup.py) ... error
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

I too have the same problem with both Colab and my laptop (Windows 10).


Fix name collisions nodes.v1.core with longhorn crd name 'nodes'

What this PR does / why we need it:
Longhorn has its own CRD names like nodes.longhorn.io. And its short name is nodes.
This causes trouble listing kubectl get nodes.
Using long hand name like "nodes.v1." fixes it.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Steps to reproduce
Does this PR introduce a user-facing change?

Fixed a bug where migration to containerd from docker will fail due to incorrectly listing node using incorrect Group.

Does this PR require documentation?
No

I'm not sure why this would clash? Shouldn't Kubernetes api-discovery resolve the core/v1 apigroup/version first when evaluating resources? This seems like a bug in kube to me.

Also, how do you reproduce this? I have longhorn installed on a cluster with Kube 1.22 and I don't see this issue.

Relevant:

There's a proposal to enable kubectl to warn the user when there is a resource name/type conflict: https://github.com/kubernetes/kubernetes/pull/110218

Also, how do you reproduce this? I have longhorn installed on a cluster with Kube 1.22 and I don't see this issue.

Migrating to containerd causes it.

I'm not sure why this would clash. Shouldn't Kubernetes api-discovery resolve the core/v1 apigroup/version first when evaluating resources? This seems like a bug in kube to me.

Probably a bug on kube. and interestingly core is an empty string :)


Merge for multiple PRs

Hi duilio,
I'm sorry I didn't reply for those pull requests in time, I was just too busy these two weeks.
Since there are 4 pull requests that are waiting for updating README, I just created a new PR, merged that four branches, and updated README at once. You can close that four PRs.

Hi Stephen, I'm even later as you notice :-)
I've merged some other PR in the meantime and unluckily this has has some conflicts now :-(

Also you may add your name in the author list ;-)


Use rustls for tls implementation

This patch switches tls backend to rustls from openssl. This makes cross-compiling to arm and other architecture easier by not requiring openssl headers. Also the rustls implementation is more future promising in term of security and safety.
I will be using this branch in my own setup. If the change makes sense, I will finish some minor TODO items to improve the code quality.
Thank you for this excellent tool!

Thanks for your contribution! Shifting to rustls has been on my mind for pretty long time. But there are two main blockers:

rustls doesn't support cert for ip, which has been an issue for years, waiting for the maintainer to fix.

Rummors are that rustls makes it harder to cross compile for minor platforms. E.g. mips. However, I've not checked whether rustls compile for all platforms that rathole currently supports

---Original---
From: "Ning @.>
Date: Fri, Apr 8, 2022 23:19 PM
To: @.>;
Cc: @.***>;
Subject: [rapiz1/rathole] Use rustls for tls implementation (PR #152)
This patch switches tls backend to rustls from openssl. This makes cross-compiling to arm and other architecture easier by not requiring openssl headers. Also the rustls implementation is more future promising in term of security and safety.
I will be using this branch in my own setup. If the change makes sense, I will finish some minor TODO items to improve the code quality.
Thank you for this excellent tool!
You can view, comment on, or merge this pull request online at:
  https://github.com/rapiz1/rathole/pull/152
Commit Summary
18d38c6  refactor: use rustls for tls implementation
File Changes
(3 files)
 M     Cargo.lock     (363)

 M     Cargo.toml     (5)

 M     src/transport/tls.rs     (127)

Patch Links:
https://github.com/rapiz1/rathole/pull/152.patch
https://github.com/rapiz1/rathole/pull/152.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I'm thinking about adding a compile flag ( features, in cargo's terms) to compile with rustls, instead of replacing openssl. We can use this as meditation since rustls doesn't fit all our requirements (see above). Are you still interested in this work?

@rapiz1 I'm no longer working on this. So feel free to pick it up for further development. Using feature flag for rustls makes a lot of sense. Thank you!

Interesting. I am struggling with rustls now. Glad to find this PR.

rustls 0.22 introducing some breaking changes that invalidates this code.
I am discovering how to fix. May take longer than I expected.

merged via https://github.com/rapiz1/rathole/pull/330


Support standalone Redis (non-cluster)

It is useful for developers if the same client can connect to a standalone node, not only a cluster. For this, we need to handle an error response form CLUSTER SLOTS and handle this as if the whole cluster is handled by a single node.
The requirement that the cluster needs to have at least two masters for 'cluster_ok' may need to be reconsidered.

@zuiderkwast is it possible to connect to a standalone node, with PR https://github.com/Ericsson/ered/pull/41 ?

Even without #41, it's possible to connect to a single node using the ered_client module. #41 does make things a little more clear though with functions like ered:connect_client for connecting to a single instance.
It's not possible to use a standalone instance as if it were a cluster though, with the same functions, but actually I don't think we want to do this automatically. Cluster and standalone deployments are very different. Standalone mode allows SELECT and cross-slot commands, for example, which are forbidden in cluster mode.

@zuiderkwast Thanks a lot for your useful information! Currently we're using eredis in VerneMQ so we cannot connect to a Redis cluster yet. And ered's stated goal is to replace both previous libraries, isn't it. In any case, I'll try to do some testing based on your suggestions.

Yes, ered implements the functionality for single instance and cluster, but the API has a preference for cluster. This was done before I was the maintainer of this project. Now, I've been trying to make the API look nice for single instance too.
An idea: We use a tuple like {cluster, Pid} or {standalone, Pid} as the ered instance reference, returned by ered:connect functions. Then we can use the same functions like ered:command/N for both of them and it can dispatch to the right module based on the tuple. That's future work, if you like the idea. (ssl:connect and gen_tcp:connect return a tuples like this, so it's not that strange.)
I've been busy with other work though.

@zuiderkwast I suppose the client always knows whether it's connecting to a standalone or a cluster. Your suggestion sounds great to me, in any case!

I've now written my plugin using ered:start_link/2 as main entry point. As expected, this only works for a cluster, not a standalone (standalone gets the unmapped_slots error). A question, though: with the idea proposed above, will ered be able to figure out by itself whether it connects to a standalone or a cluster?

A question, though: with the idea proposed above, will ered be able to figure out by itself whether it connects to a standalone or a cluster?

It would be possible to auto-detect, but I think it's better that users connect explicitly to cluster or standalone. I'm thinking ered:connect(Host, Port, Opts) for standalone and ered:connect_cluster(AddressesAndPorts, Opts) for cluster. After that, ered:command(EredRef, Command) and similar can be used for both kinds of instances.

Sounds great. That interface is perfect at least for me, as the client (the plugin) always knows whether it connects to a cluster or not.
Let me know if I can test  (or do anything helpful). No worry & no hurry for anyone :)
I think it'll be a cool library to use and like the unified approach in it.


Migrate article: Having trouble creating or upgrading to a larger plan?

Migrate the content from this article to fit the documentation portal format https://help.aiven.io/en/articles/5240633
(Assign this issue to yourself before you start, and remember to mention the issue number when you open your pull request)

Hi @lornajane, I've create a pull request for this issue
https://github.com/aiven/devportal/pull/1150


Add this New YouTube Link: https://bit.ly/CodeINYouTube and remove th…

…e previous link from the footer
🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

hello there i have change the youtube link thanks

changes all link in all pages


notFoundRedirectNamed issue

This is not a bug, but a bad behaviour IMO.
On a web browser, when routing to an inexistent route, it gets redirected correctly to notFoundRedirectNamed, but the browser url got stuck with the wrong route. Is there a way to get it changed to the notFoundRedirectNamed value?
Thanks
BeamerDelegate(
        initialPath: initialPath,
        notFoundRedirectNamed: initialPath,
        locationBuilder: RoutesLocationBuilder(
          routes: routes,
        ).call,
        setBrowserTabTitle: true,
        guards: guards,
      )

@busslina So you mean the url on top of the browser does not change when notFoundRedirectNamed is ran ?
The way i see it is to just update the browser title if tghe notFoundRedirectNamedis used?

So you mean the url on top of the browser does not change when notFoundRedirectNamed is ran ?

Exactly.

The way i see it is to just update the browser title if tghe notFoundRedirectNamedis used?

The browser title is working fine.
Also, guard redirections also works fine. It's just the case of a not found route. It gets well redirected but it keeps the inexistent url path on the browser address. It's acting like an alias, which could be cool if any want it, but also it must be provided a way to refresh with the actual path.

So you mean the url on top of the browser does not change when notFoundRedirectNamed is ran ?

Exactly.

The way i see it is to just update the browser title if tghe notFoundRedirectNamedis used?

The browser title is working fine.
Also, guard redirections also works fine. It's just the case of a not found route. It gets well redirected but it keeps the inexistent url path on the browser address. It's acting like an alias, which could be cool if any want it, but also it must be provided a way to refresh with the actual path.

For now you can change it yourself... but it needs to be fixed.
You can do that before you return it with the notFoundRedirectNamed: ...

You can do that before you return it with the notFoundRedirectNamed: ...
But notFoundRedirectNamed is just a String. How can I change there the browser path?

You can do that before you return it with the notFoundRedirectNamed: ...

But notFoundRedirectNamed is just a String. How can I change there the browser path?

Do you want to change the browsor path without changing the app state ?

Seems this issue got resolved "by itself". So I'm closing


STORM-1020 - Added Javadoc to Fields and ITuple.

Broke combined PR into distinct branches.

@rmkellogg
+1 Could you upmerge?
Hint: Modifying CHANGLOG.md would make conflicts.

Having difficulties with merge operation.  Pardon me.

OK, then I'll merge this by cherry-picking c2fa25f.

Thank you.  Will try to make it easier next time.

@rmkellogg I merged into master by cherry-picking. Could you close this PR?


MorphTo Refactor

Type of PR:

[ ] Bugfix
[x] Feature
[x] Refactor
[ ] Code style update
[ ] Build-related changes
[ ] Test
[ ] Documentation
[ ] Other, please describe:

Breaking changes:

[x] No
[ ] Yes

Details

Implementing changes discussed in the following draft PR: https://github.com/vuex-orm/vuex-orm-next/pull/107
We are going to want to squash and merge this since I am a frequent committer 🙃

Amazing job 👏 this looks and works like a charm 👍

Thanks! Couldn't have done it without @kiaking and you 😄

Merged into #106. @kiaking can review and manage from there 👍
Thanks for all your hard work @ryandialpad 🙌


Deployment clone action and networking slowness injection

TL;DR
This adds a new action - cloning a deployment to add failure into a copy, rather than modifying the existing software. It also lays foundation for various mutators, with a first batch of:

environment variables mutation (for example override REDIS_PORT to a proxied one)
traffic control mutation (add latency to the whole pod)
toxiproxy mutation with automatic redirection via iptables to selectively create proxies and configure network issues on them

That gives the user two different ways of achieving safe-ish (only the copy is affected) network slowness simulation (L3).
Details
This is still a little rough around the edges and various things should be made configurable, but the concept is proven.

[x] add the schema changes for the new clone action
[x] implement the cloning mechanism for deployments
[x] implement the label modifications
[x] implement the mutations of environment variables
[x] implement copying labels from services' selectors
[x] implement traffic control mutation
[ ] add tc presets like latency, loss, throttle
[x] add traffic control docs and example usage
[x] implement toxiproxy mutation
[x] add toxiproxy docs and example usage
[x] implement automatic cleanup
[ ] test coverage
[x] upgrade kubernetes version to 12.0.10a1 to leverage Kubernetes 1.16

Also note, that because toxiproxy is a little silly and allows for populating the proxies at startup, but not the toxics, I'm abusing the startup probe a little bit, to actually populate the toxiproxy with the user-requested stuff.
Testing
See the added docs for example usage. You will be able to see these pods with network slowness automatically created for you:

Resolves https://github.com/powerfulseal/powerfulseal/issues/285

Looking for feedback and reviews on this one :)


tesla-control: fix climate-set-temp command wrong conversion equation

Description
fix climate-set-temp command wrong conversion equation

Previously, sending a climate-set-temp command in "f or F" units set the cabin temp to HIGH.
Only "c or C" units were accepted Now command sets cabin to appropriate temperature in both units.

Fixes # (issue)
Type of change
Please select all options that apply to this change:

[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation update

Checklist:
Confirm you have completed the following steps:

[x] My code follows the style of this project.
[x] I have performed a self-review of my code.
[ ] I have made corresponding updates to the documentation.
[ ] I have added/updated unit tests to cover my changes.

Thanks!


[Storage All Service] Find best default value for block size

Will be aligned with .Net

Will be aligned with .Net


Relations in Structure seems not to be working

Hi,
I added a Relation field in a Section: Type Structure and I get "no relations found".
I did the same with a Section: Type Channel and Single and it works fine.
Using Craft <IP_ADDRESS>, I did a test also with Craft 3.1.22 and I have the same issue.
Do you have any suggestions? Could it be a bug?
Thank you very much
Steve

@steve71p I Just did a test and seems to work fine. Are entries in your structure related somewhere? It will show no relations found if they are not.

Hi, Yes I do have entries related, it's strange but if with you it's working I will do a double check and see if I'm not doing something wrong.

No worries, let me know if there is an issue. It would also be good to know if entries are being related within another field like Neo or Super Table.

I have a channel "teachers" and I have a stucture "courses". In an entry "course" I have an "entries field" to select a teacher.
In the channel "teacher" I add your relation field but it doesn't show the related course.
Keep in mind I'm in a multisite enviroment.

@steve71p I tried to replicate this issue but no success so far. Could you please send me a test db which I run on my machine to replicate this issue?


port_scanner - Use of uninitialized value in subroutine entry

I get the following error when running port_scanner against localhost:
$ ./port_scanner --ip <IP_ADDRESS>

./port_scanner  Version 0.02  Source: github.com/dnmfarrell/Penetration-Testing-With-Perl

Wed Mar  9 15:52:40 2016

Starting port scan: type: tcp, flags: syn, 50 procs, 1 (secs) delay
Source host: XX.XX.XX.XX:42283, target host: <IP_ADDRESS>

Use of uninitialized value in subroutine entry at ./port_scanner line 154, <$port_file> line 19941.
Use of uninitialized value in subroutine entry at ./port_scanner line 154, <$port_file> line 19941.
p is not of type pcap_tPtr at ./port_scanner line 154, <$port_file> line 19941.

Sorry the docs aren't clearer, you need root privileges. Are you using system Perl or a locally installed Perl? This should work for both:
sudo $(which perl) port_scanner --ip <IP_ADDRESS>

If system Perl that can be shortened to:
 sudo perl port_scanner --ip <IP_ADDRESS>

I'll add a check for root privileges and an exit with appropriate error message if not run as root.

Fixed with latest commit


Too much crash

**Fedoraware Version Date: August 30, 2022 **
Windows Version: Windows 11 Home Single Language
Configs used:
cfgs.zip
Description:
Either by joining or retrying
Steps to reproduce:

[ ] With the steps below I am able to consistently reproduce this issue.


Allow disambiguating paths in explorer and source control views

Context:

we use many small repos organized in a hierarcy
the basenames of some repos repeat in different places in the hierarchy, e.g. .../k8s/alpha refers to the infrastructure code of the kubernetes cluster named alpha and .../mysql/alpha is the infrastructure code of the mysql cluster named alpha
inside those repos we use git submodules with shared code
many submodules repeat from repo to repo (since that is what they are for) with the same names - so inside every cluster repo there will be modules/linux-base and modules/saas etc
our workflow relies on being able to quickly open a terminal in the project folder (so the correct environment is configured)
for this reason we add the repos to the workspace individually
this results in many ambiguos folders in the explorer pane, but especially in the source control repositories pane (not only top level repositories with repeating names, but also submodules with even more repetition)

Feature suggestion:

explorer: allow the user to enable a "disambiguate paths" or something like that - whenever two folders in a workspace are named the same show just enough parent folders in their title so they become distinct
source control repositories:

same as explorer
sort mode where submodules always come after their parent repository. you would think current "sort by path" achieves that but somehow it doesn't: first all top level repos are listed and then their submodules
perhaps too much to ask, but may be a mode where source control repositories shows only repositories from the workspace folder to which the file in the active editor belongs - so when I edit something from .../k8s/alpha I only want to see the repository .../k8s/alpha and its submodules
or: a way to filter the source control pane on repositories with changed files (because I probably want to commit those soon)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add boolean dynamic ValueAccessor method

Adds a method to return a boolean from a ValueAccessor. Resolves #1152 1152

Thanks! 🙂


Missing Subscribe button in newsletter form (Magento 2.3.2)

When module enabled then "Subscribe" button is hidden in newsletters subscription form.
It is because of CSS class "sr-only".
https://github.com/magento/magento2/blob/2.3.2/app/code/Magento/Newsletter/view/frontend/templates/subscribe.phtml#L36

Hello,
on my page the subscription button has disappeared after upgrading to Magento 2.3.2
Do you know the solution to the problem?

Hi,
this issue (https://github.com/magento/magento2/issues/24027) is already fixed in 2.3-develop branch.
You can apply this patch manually for now or remove the sr-class with custom javascript.

Thank you :)


When commenting, automatically "save" a task

When you comment on a task, it is likely that you may want to return to it later. This is made much easier when the task is saved automatically.

I still think this makes sense. @nrotstan @mgcuthbert your thoughts?

Let's keep this one on the radar. I hope this would be a smaller enhancement?

This is now obsolete because we moved commenting  to the post completion dialog. (Although you can still add a comment widget to your task window layout)


Cannot set array as value

I am trying to set array values to a field, but its always passed as a string instead of an array.
    this.form.setValue({
      'request_types': ['hotel', 'flights']
    });

@jagadeshanh In your case request_types is formArray right? If so I know the issue, is because currently setValue accepts only string not array or objects, can you submit PR for this?

What was the reason for converting the value to string
this.formGroup.get(values).setValue(value.toString());

I know there was an issue and I solved it with toString() but I don't remember what was the issue! However, change if you can make it work properly.


Attempt to boost startup time with SnoopPrecompile

@Quarlos this demonstrates a simple implementation of SnoopPrecompile. Using this for precompilation (i.e., Pkg.precompile()) reduces startup time for me by about 33%. Want to have a look at this and see if you can improve it at all?
Here's the code I tested this on:
@time begin
    import Random: seed!
    using SymbolicRegression
    seed!(0)
    X = randn(Float64, 2, 3); y = X[1, :];
    options = Options(progress=false, unary_operators=(cos,), max_evals=10, verbosity=0)
    hof = EquationSearch(X, y, options=options, multithreading=true);
    nothing
 end

The one strange thing is that it seems to complain about me redefining operators on the Node{T} type compared to the precompiled version:
WARNING: Method definition +(T1, SymbolicRegression.CoreModule.EquationModule.Node{T2}) where {T1<:Real, T2<:Real} in module OptionsModule at /Users/mcranmer/Documents/SymbolicRegression.jl/src/Options.jl:560 overwritten on the same line (check for duplicate calls to `include`).
  ** incremental compilation may be fatally broken for this module **

I do this redefinition because the Node{T} uses an enum over operators, so for things like print to actually be helpful, I have to redefine them every time an Options is created. So, for using this in practice, it would probably be a good idea to prevent SnoopCompile from precompiling those functions.
Fixed the above issue with an extra parameter that disables it!
cc @ChrisRackauckas - thanks for the recommendation.

Probably the simplest way to fix these precompile warnings is to add a flag to Options that simply tells it to not re-define the base functions.

New version in #161


gradle exception

gradle 2.10
gradle --stacktrace myCucumberSuite

org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':myCucumberSuite'.
...
Caused by: java.lang.NoClassDefFoundError: org/gradle/internal/logging/progress/ProgressLoggerFactory
at com.commercehub.gradle.cucumber.CucumberTask.runTests(CucumberTask.groovy:35)
at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:75)

As this project is now deprecated, CommerceHub will not be contributing further on this ticket.  Please feel free to fork the project and work on it further yourself under the terms of the project license if desired.


new version

This issue tracker is for problems with the Open Distro for Elasticsearch documentation: inaccuracies, content requests, etc.
If you need help with troubleshooting, see the community issue tracker or the forums.
Thanks so much for helping keep us organized. Please delete this text before posting your issue.

Hello I was using opendistro for elasticsearch at latest version on test server.
my question is, Now elasticsearch latest version is 7.10.0 and latest opendistro includes 7.9.3 (which is current version for me)
If I wanna use opendistro security plugin in my new cluster elasticsearch 7.10.0 , should i wait for new update in opendistro for elasticsearch or can i use opendistro_security_<IP_ADDRESS> plugin in elasticsearch 7.10.0 ?
Thanks


Place google map into Unity

RFS google maps building is in Blender. Move to Unity and replace MapBox, as a new branch.

[ ] Find correct export format
[ ] Import into Unity
[ ] Edit scale
[ ] Hide mapbox

Is this done? It looked p good on the demo?


NIFI-7030 Add Kerberos principal and password properties to Solr proc…

…essors.
This allows authenticating to a kerberized Solr using a principal and password, as an alternative to the keytab credential service.
I tested this with docker-kdc, Solr 8.4.1, and ZooKeeper 3.4.6, all running locally.
The Solr reference guide has good instructions on how to configure everything:
https://lucene.apache.org/solr/guide/8_4/kerberos-authentication-plugin.html#kerberos-authentication-plugin

Pushed commits that address review comments from @alopresto


How Should the SWG Do Its Work?

How do we work together and develop the specification. A variety of techniques including face-to-face meetings, telephone/web conference and online tools such as GitHub have been suggested. How do these fit together and integrate with OGC procedures?

Working with SWG members and OGC staff, we have created a specialized subgroup to complete the conceptual model design. Participation is open to all SWG members but the work is conducted in separate bi-weekly two-hour online meetings. GitHub is used as an organizing framework, with recorded issues and comments driving the work.


Invisible activity is always created on background

When asking for permissions, the invisible activity is always created, this might cause some issues if Dexter is called from the onCreate or onResume methods of your own activity (as it will force another call to onCreate & onResume once the inner activity is dismissed). We should only create the inner Activity iff ActivityCompat.requestPermissions is going to be called.
For calling checkSelfPermission we should use the provided application context.

Duplicated #47


testrpc always creates exactly 10 accounts upon startup

The "-a" option does not seem to change the number of accounts created on startup
Expected Behavior
Running "testrpc -a 100" creates 100 accounts at startup
Current Behavior
Running "testrpc -a 100" creates 10 accounts at startup. Omitting "-a 100" also creates 10 accounts at startup.
Steps to Reproduce (for bugs)

Run "testrpc -a 100"
Observe / verify by attaching a console that only 10 accounts are available

The problem is here:
https://github.com/ethereumjs/testrpc/blob/master/lib/statemanager.js#L61
this.total_accounts is always null.  It should be options.total_accounts:
var total_accounts = options.total_accounts || 10;

This fixes the problem

I noticed that issue and included the fix into PR #165.
Once the PR is merged, testrpc -t 100 will have the expected result.

PR #167 also improves that.  Samples in the PR comments.

By the way @riggs-, I was very confused as well as the doc did not match the code but here is how the options work:

-a / --account is to provide a list of accounts (not a number)
-t / --total_accounts is the number of accounts you want.

Once my PRs are merged, you can use the --help option to remember and -t 100 to get what you need.

Thanks for helping and investigating. I hope the pull requests solving the problem will be merged soon. For now I can live with a locally patched testrpc, but eventually this should make it into the next release.

@riggs- I am very confused atm regarding the options but if you test with 3.0.0, the -a flag will work as you expect it to work (unlike what I wrote above).

Fixed by -a in current version.


Fix link after site was renamed in #1540

Issue
After the Themeing guide was renamed to Theming in #1540, there’s a broken link in the Setup guide.
Solution
Fix link.
I already pushed that change to the docs. So there’s nothing more to do.

awesome! thanks for the fix


AboutActivity now uses Chrome Custom Tabs for website of libraries

@nickbutcher please review and merge if you think it is a valuable enhancement

Thanks for the PR, another nice change. Have added one v minor comment.

@nickbutcher done with the changes. This is ready to merge!

@nickbutcher ping

@nickbutcher I made the changes in the third commit. Let me know what do you think. Thanks!

I will squash the 1st and 3rd commit before merging.

Also, Now that I am passing the activity, do you still want to have the fallback to                         context.startActivity(new Intent(Intent.ACTION_VIEW, uri)); if the host Activity is null?

@nickbutcher done with changes. Thanks!

Thanks for the changes! LGTM.


feat: ignore universe levels when creating canonicalizer key

closes #3848

We already have a PR for this: #3853


[consensus] Expose Tendermint config CreateEmptyBlocks and CreateEmptyBlocksInterval

fix #923
Signed-off-by: phymbert<EMAIL_ADDRESS>

Thanks!


Kontena Web UI?

Hi,
I think I must be missing something as the website and some other areas of kontena platform point to "web ui", how can one access it?
I have installed a master and grid, and even deployed a test app as well as some examples... but having an "overall view" of what is going on is quite tedious without a lot of automating with other tools so far... I understand kontena has a dashboard of sorts?  How does one access that?
from the Kontena Platform Services: "Web UI
Beautiful web based user interface"

https://cloud.kontena.io
If you are using Kontena as the authentication source for your master you should already have an account.

Hmm.. I selected standalone...
How can we run the UI offline or at least without internet access?  Some of our deployments are intranet only and the monitoring wont have access to the web.
Thanks!

@RVN-BR Currently there is no standalone version of the UI available for on-premise setups.


Commercial license checking

Hi there,
Loving the package! Thanks so much for all the work you put into this. This really makes license checking a breeze in comparison to other packages.
I really like the summary view!
I think it would be great if we could run legally --commercial and have legally warn us about any dependencies whose licenses prohibit commercial-use (since this is probably the number one reason people check licenses).
We could use the following website to check (in advance) which licenses prohibit commercial use:
https://tldrlegal.com
What do you think about this? Looking forward to hearing your thoughts.

Thanks, this would be nice and would be a great addition, but I think that implementing this would be a similar task that the whole project so I do not plan on adding something like this anytime soon. Would you like to develop this?

Thanks for your response! At the moment I don't think I'll find the time to complete this though.
Do you think this should be part of legally or an external module that makes use of legally?

Both are possible, but legally is not built in a way that allows for easy extension (but it does allow for easy modification) as it's a public-facing tool, not a library.
And that is the basic reason why I created legally, to check (and avoid) for commercial and share-alike licenses. I've read enough about compatibility, besides what I search when I don't know it, to love MIT while disliking the GPL so for me that's enough. However, ideally this would be a commercial, share alike, attribution, etc checker.

I see. Yeah, I've done the same research, but every once in a while you end up with a package using some weird, uncommon license (i.e. AFL 2.1 / CC0 / CC BY 4.0) and it would be nice to know immediately how restrictive that license is.
Anyhow, I'll see if I find the time to develop anything like this and let you know if I do.

@franciscop I went ahead and developed this! 😁
It's called tldrlegal and it provides the following output, using legally to both determine the dependencies' licenses and build the output tables:

Check it out and let me know what you think! 👍

That is... really awesome!! I knew that making my projects modular would help someone some day, I see your var legally = require('legally/lib/legally'); which returns the packages with their licenses even though it is not documented. I will add your project to the readme of legally.
One of the main features for legally is to be able to check a package before installing it with legally PACKAGENAME. I do it based on minimist as can be seen here and here (remote package installation in temp folder), in case you are considering it.

@franciscop Glad you like it! 😄
Yeah, I found the "unofficial" API extremely easy to use, thanks for making it so modular! The table generator is also awesome.
Regarding remote checking, that's cool! I'll consider adding it in the future to tldrlegal as well :10

@franciscop FYI I also wrote a blog post and mentioned legally :)
https://eladnava.com/check-your-dependencies-license-requirements-with-tldrlegal/